Roles and Environments should have identical implementations

They're basically the same right now, but environments support cookbook version pinning and roles do not (at least not without the hacky equality pinning in the run_list).  If they were normalized, then the distinction between them would go away.  Environments could ideally then be just dropped and sites could use role[production]. role[development], etc.  We'd still probably keep them around because people want a thing called 'environment' that matches their 'development/test/integration/production' workflow.
Right now environments are bad because its a 1-1 mapping and results in sites having to decide up front what their strategy will be and then they will get locked in.  You see sites where they start mixing in datacenters and canaries and all kinds of other dimensions into the environment in order to get their deployment cells.
The difference is that before you might have a server with:
run_list:  role[foo_app]
chef_environment:  production-seattle-canary

The result is a large number of environments to manage as we have the cross product of all the dimensions expressed in the environment.  Each item in the cross product is another cell to manage.
And afterwards:
run_list:  role[foo_app], role[seattle], role[canary]
chef_environment: production

Now we don't have to manage and name the cross product of the things, we just manage the axes and then they get mixed in.  That keeps 'environment' as the name for the thing people expect (or they could drop it entirely), and then also allows the rest of the matrix to expand organically.  If the company fragments and they need to add logical business units to the run_list to segregate companies-within-a-company then they can just add another role to all the servers that maps to the business units, etc.

:+1:

Gonna close this since it's not really a ChefDK thing. If you use policyfiles, then you're not using roles or environments, so that sorta fixes the issue. That said, we're probably gonna need some sort of policyfile composition story (maybe also a service discovery thing to replace environment attributes) before most organizations will be able to migrate to them, so maybe it's worth some kind of near-term updates to the non-policyfile way of running Chef to improve the story here. But the RFC process is probably a better place to propose and track that.


Can a calculation item be hidden?

What are you trying to achieve?
It appears that the 'Calculation Item' is the only object in the table construct that cannot be hidden. Is this a limitation in tabular or Tabular Editor?
What have you tried so far?
Edited the .bim file in notepad and added a clause to the calculation item for IsHidden = true. However, then the .bim file couldn't load into tabular editor. That being the case, I would think this is a limitation within tabular, not Tabular Editor.

Calc items cannot be hidden. They are rows in a table - if you want certain calc items to not be visible in a slicer, then add a calc column to the calc group table and use that column to filter your slicer.

Got it, makes sense. Thanks!


Internal datatypes

Problem: Two different datatypes of TESTed are mapped on the same datatype in a programming languages. When that datatype is returned only one datatype from TESTed could be provide.
Consequence: Possible mismatch with datatype expected by the TESTed value evaluator.
Some examples:

Java/Kotlin:

TESTed type1
TESTed type 2
Java/Kotlin type

uint8
int16
short

uint16
int32
int

uint32
int64
long

uint64
bigint
BigInteger

double_extended
fixed_precision
BigDecimal

Python/JavaScript: no datatype for individual characters, both uses the datatype for strings.

Possible solution:
Possibility to provide more then one datatype from TESTed when passing a data value to TESTed from the evaluation code.
Only for datatypes from TESTed that mapped to the same datatype in the programming language.
Example solution:

Numbers Java/Kotlin:

{
    "data": 255,
    "type": ["uint8", "int16"]
}

{
    "data": 256,
    "type": "int16"
}

Characters and String Python/JavaScript:

{
    "data": "a",
    "type": ["char", "text"]
}

{
    "data": "ab",
    "type": "text"
}

An example of a failing example in Java and Kotlin:
- tab: "Feedback"
  contexts:
    - expression: '5 :: uint8'
      return-raw: '5 :: uint8'

Feedback result Java:
{"description": {"description": "(short) 5", "format": "java"}, "command": "start-testcase"}
{"expected": "(short) 5", "channel": "return", "command": "start-test"}
{"message": "Verwachtte waarde van type AdvancedNumericTypes.U_INT_8, maar was type AdvancedNumericTypes.INT_16.", "command": "append-message"}
{"generated": "(short) 5", "status": {"enum": "wrong", "human": "Returnwaarde heeft verkeerd gegevenstype."}, "command": "close-test"}
{"command": "close-testcase"}

The indented logic for type comparisons

Check if the type of the two values match. The following procedure is used:

If the expected value's type is a basic type, the actual value is reduced to it's basic type, after which both types are checked for equality.
If the expected value's type is an advanced type, there are two possibilities:
a. If the language of the submission supports the advanced type, both types are checked for equality.
b. If the language of the submission only supports a reduction of the type, all types are reduced to their basic type, after which equality is checked.
An error occurs if the language explicitly does not support the advanced type.

I think this system allows us to solve the raised issues by fixing the configs of the types:

The uint* types are currently listed as supported in Java; however I think they should be reduced, since there isn't really an uint8 type. In the example, the returned type would be int16, but the expected type uint8 is not fully supported in Java, so it would reduce both to integer, and compare those.
If we want to support using strings as char (e.g. in Python), we should move the char type to the advanced types, and define it as a specialized type of the basic type text.

I'm not against changing how this works, but I'm not convinced listing all possible advanced types is the best solution from the perspective we want to have as little as possible language specific stuff in the testplan. While the advanced types are not language specific, their inclusion is.
For example, this would work until we add a Java/Kotlin exercise, which is something we want to avoid.
{
    "data": 256,
    "type": "int16"
}

I'm not against changing how this works, but I'm not convinced listing all possible advanced types is the best solution from the perspective we want to have as little as possible language specific stuff in the testplan. While the advanced types are not language specific, their inclusion is.

It's not adding this functionality to the testplan, it's adding a custom (de)serialization format for internal usage in the judge.

It's not adding this functionality to the testplan, it's adding a custom (de)serialization format for internal usage in the judge.

So this would only be added when generating json-values in the generated code (e.g. generated Java-code -> judge)?
If so, it could be a good solution, but I'm not entirely convinced the extra complexity has advantages against falling back on the basic types for advanced types that don't have a one-on-one mapping.

So this would only be added when generating json-values in the generated code (e.g. generated Java-code -> judge)?

Indeed this would only be added when generating json-values in the generated code.


Fix json_parse bad return

In webview.h json_unescape function, the last character of out is set to '\0'. So its size is not r but r + 1.

@zserge seems you fixed the problem in another way in 16a331de00e7da02b7f562f0d93927dd212d184e.


Change matching pattern on settings.gradle

Link doesn't work on f8app,
as it is, android projects don't have to contain the matching string and then the linking would not work, I've switched it to the much simpler new-line.
If you can find a better common minimum string for settings.gradle please do.
Commit Msg:
Current string, include ':app'\n, is not a minimum common string.
Instead a simple \n will do.

@Kureev Mind taking a look?

I just scaffolded a new react-native create app. Line 3 of settings.gradle file in android folder: include :app. Not sure if we have any plans to support various custom configurations. If you want to aim to process any settings.gradle file possible, most probably you end up with just adding new lines to the end of file (that's the safest way).
tl;dr: don't think this patch is required, but don't think it break anything as well.

Hi @Kureev , thanks for answering.
It doesn't work for facebook feature app f8app
And might cause confusion.
Other then that you're right and can close if you want.

No, I think you're right. \n should do the trick.

@facebook-github-bot shipit


linting and added fix for session cookie

Hi,
I had problems with the KEMP loadmaster and a session cookie (lmdata) that was generated during login. The session cookie was somehow after logout (timeout in browser) not cleared by the browser and interrupts the login afterwards. So in case the cookie was not correct terminated, the OWA/ECP login was stuck and was looping on the SSO login page. The support of KEMP able to reproduce this issue by inserting the cookie via browser extension, but doesn't provide a fix. After some days of research I was able to create a workaround with these SSO pages and added a function which clears the cookie during loading.
May this could be helpful for other users which got the same issues.
Thanks for providing this great guide.
Regards Enrico

@altima This is great. Thank you for contributing. 👍


Cannot map properties that have differt names and init access modifier

Hi,
I have a problem with version 7.4. If I want to map two properties with different names and the destination has  a different name and the init modifier, Mapster throws a NullReferenceException. This happens when in the configuration of Mapster I set PreserveReference to true.
Here's the code
{
    public Mapper Mapper { get; }

    public MapsterTest()
    {
        var config = new TypeAdapterConfig();
        config.Default.Settings.PreserveReference = true;
        config.Scan(Assembly.GetAssembly(typeof(MapsterTest)));
        config.Compile();
        Mapper = new Mapper(config);
    }
}

internal class MyClass
{
    public int MyProperty { get; set; }
}

internal class DtoClass
{
    public int MyPropertyDto { get; init; }
}
internal class Register : IRegister
{
    void IRegister.Register(TypeAdapterConfig config)
    {
        config.NewConfig<MyClass, DtoClass>()
            .Map(x => x.MyPropertyDto, x=> x.MyProperty);
    }
}

And here's the mapping

var from = new MyClass { MyProperty = 1};

var to = mapster.Mapper.Map<DtoClass>(from);

(The problem does not manifest if the properties have the same name or another modifier (set; or private set;)
This code works fine with version 7.3

linked #422
@nicoarm93  @andrerav   location of this bug found )

In my case similar regression (Mapster throws a NullReferenceException) between 7.4.0 and 7.3.0 is not related to PreserveReference. It's about init only properties, when mapping to target object.
Below code works fine with Mapster 7.3.0, but doesn't work with Mapster 7.4.0.
using Mapster;

TypeAdapterConfig<Core.RootSource.NestedSource, Core.RootDestination.NestedDestination>
    .NewConfig()
    .Map(dest => dest.NameDest, src => src.NameSrc);

var dest = new Core.RootDestination { MyProperty = null };

var mapped = Core.RootSource.Create().Adapt(dest);

Console.WriteLine(mapped);

namespace Core
{
    public class RootSource
    {
        public required NestedSource? MyProperty { get; set; }

        public static RootSource Create()
        {
            return new RootSource
            {
                MyProperty = new NestedSource
                {
                    NameSrc = "NotEmptyString"
                }
            };
        }

        public class NestedSource
        {
            public required string NameSrc { get; set; }
        }
    }

    public class RootDestination
    {
        public NestedDestination? MyProperty { get; set; }

        public override string ToString()
        {
            return $"{nameof(RootDestination)}.{nameof(MyProperty)}={MyProperty?.NameDest}";
        }

        public class NestedDestination
        {
            public string? NameDest { get; init; }
        }
    }
}


Focus cncf-infra/prow-config to just contain the configuration for Prow

The Prow deployment and verify-conformance config is currently located in infra/aws/manifests and the content of that folder should be all that the prow-config repo contains.
The configuration should be managed by FluxCD.

closing since verify-conformance will likely run in an existing cluster along side existing config management (via Prow)


RFE: search for incident error message text

Motivation / Use Case
The "Incident view" shows all incidents. It would be very helpful to search for specific text in incident/error messages.
Feature description

the incident view offers a form field, where text can be entered

the text is meant to be a partial match for error messages in incidents
not case sensitive

entering such a text, is an equivalent for filtering just the incidents with this text included

@saig0 What's your view?
PS: I'm willing to contribute, as usual ;)

@nitram509 sounds good to me :+1:
Looking forward to see your contribution :rocket:


Transfer-Encoding: Chunked - Inaccurate and sometimes incomplete using HttpRequestStreamReader

From @VAchris on August 15, 2018 11:9
https://github.com/aspnet/HttpAbstractions/blob/bfa183747f6fb528087554c3d6ec58ef05f1c10a/src/Microsoft.AspNetCore.WebUtilities/HttpRequestStreamReader.cs#L322
Ok, so lets set the stage. I have an ApiController that accepts a POST request. I will always expect the POST  to include the http headers Transfer-Encoding: Chunked and Content-Type: application/json. The POST body will typically contain a large JSON file that is sent from a central gateway (similar to a DMZ) that sits between the LAN and WAN.
The central gateway doesn't store and forward; it just forwards using chunked transfer encoding to my receiving ApiController which writes the POST body to a file.
Issue: How do I detect the true EOF when chunked transfer encoding is used?
What I've tried:
var streamReader = new HttpRequestStreamReader(Request.Body, Encoding.UTF8);
if(streamReader.Peek() >= 0) { ... }
However there are times when .Peek() will return -1 even though only a small portion of the POST body transfer is complete.
I've also tried a do while loop using .Read() however I get similar results to .Peek()
var postBody = streamReader.ReadToEnd();
This works however I can't and shouldn't read the entire contents; I need to be able to stream the POST body and immediately write to a file, especially for large POSTs.
Interestingly: .Peek() and .Read() consistently both return -1 after 8,192 bytes have been read from the stream.
Copied from original issue: aspnet/HttpAbstractions#1031

Can you provide a sample project that reproduces the problem?

Closing this as we haven't heard from you and generally close issues with no response after ~7 days. Please feel free to comment if you're able to get the information we're looking for and we can reopen the issue to investigate further!


feat: Add Node, Deno, Fastro, Fastify and Express

Fixes #4

Ready for review 👍 (i had the time so thought i might as well tackle the issue anyway)


Customize Chip Colors According to Room Labels

Idea Description
In the DroidKaigi app, we have the opportunity to enhance the visual differentiation of room labels by changing the color of the corresponding Chips.
Expected Behavior
Chips should have distinct colors based on the room labels they represent.
Current Behavior
All Chips have the same color, regardless of room label.
Steps to Reproduce
Provide steps on how to reproduce this issue...
Additional Information
Provide any additional information or screenshots...
Reference images and links
https://www.figma.com/file/MbElhCEnjqnuodmvwabh9K/DroidKaigi-2023-App-UI

I would like to challenge this issue 🙋.
Also, sorry if this is an odd question, but am I correct in understanding that the room color is branched by the name (en) of the RoomResponse, as in the 2022 application?

I think we are having a extention function for room for this year. Can check it?

I would like to use this 🙏
https://github.com/DroidKaigi/conference-app-2023/blob/192521ac03a3c5c499a5019bfc49be2094a36eeb/core/model/src/commonMain/kotlin/io/github/droidkaigi/confsched2023/model/RoomIndex.kt#L6

Thank you 🙇


doc: 👷 update ci add "doc" label and remove dependabot

Why need this change? / Root cause:

Add CI auto lebel "doc"
Remove dependabot because its functionality overlaps with RENOVATE-BOT.

Changes made:

CI

Test Scope / Change impact:

Issue

✅ Knip Scan Result for https://github.com/Game-as-a-Service/Game-Lobby-Web/pull/366/commits/2e501fe4b492c8588536c5824b11fb998921e5b5

Unused files (9)

Unused files (9)

.bundlewatch.config.js

components/shared/Chat/ChatContent.tsx

components/shared/Chat/ChatMessage.tsx

components/shared/Chat/index.tsx

configs/i18nConfigs.ts

lighthouserc.js

reset.d.ts

scripts/knipScanReporter.js

scripts/lhciScanReporter.js

Unused dependencies (2)

Unused dependencies (2)

@svgr/webpack  package.json

sharp          package.json

Unused devDependencies (5)

Unused devDependencies (5)

@actions/github             package.json

@octokit/core               package.json

@storybook/blocks           package.json

@storybook/testing-library  package.json

@total-typescript/ts-reset  package.json

Unused exported types (2)

Unused exported types (2)

ChatProps        type  components/shared/Chat/v2/Chat.tsx

ChatHeaderProps  type  components/shared/Chat/v2/ChatHeader.tsx

Configuration issues (1)

Configuration issues (1)

Unused item in ignoreDependencies: @next/bundle-analyzer

🤖 Lighthouse Scan Result for https://github.com/Game-as-a-Service/Game-Lobby-Web/pull/366/commits/2e501fe4b492c8588536c5824b11fb998921e5b5

/rooms

Metric
Value

Performance
73

Seo
78

Accessibility
78

HTML Report for LHCI Scan
Report Link

/rooms/abc

Metric
Value

Performance
76

Seo
78

Accessibility
64

HTML Report for LHCI Scan
Report Link

/login

Metric
Value

Performance
47

Seo
80

Accessibility
94

HTML Report for LHCI Scan
Report Link


[WIP][ISSUE #4437]Xss plugin

It is still under development.
#4437

@iwangjie pls resolve conflicts.


Comments aren't recognised or formatted

First: thank you for picking this up! The original was good but it's broken now, so glad there's an alternative.
I did notice that comments formatted to Cooklang specs are either hidden in reading mode, or are parsed as cooking steps.
Comments added with "--" between the instruction and the comment, or block comments on a single line with "[-" and "-]" at the start and end are not visible in Reading view.
Block comments split over multi lines display in Reading mode with the first line appended to the end of the last Step, and subsequent lines are parsed as Steps too. In my file I had a single-line block comment between Step 8 and a multi-line block comment; the Reading mode dropped the single-line block comment and chucked the first line of the multi-line at the end of Step 8.
Usages for comments are Descriptions, or Cooking Notes that aren't part of the general recipe method flow (like, "If you use fridge cold eggs, you will notice the batter is thicker. It will still work just fine, but room temperature eggs is better – it makes the batter pourable into the pan. Cake may also need an extra 5 minutes in the oven.").

Hi! Thanks for the issue. I think that the comments aren't supposed to be rendered. I try this "recipe" on the sandbox and don't see the comment https://biowaffeln.github.io/cooklang-parser/
-- Don't burn the roux!

Mash @potato{2%kg} until smooth -- alternatively, boil 'em first, then mash 'em, then stick 'em in a stew.

Do you have any rendering example? Maybe I could add an icon to indicate that there's a comment or something (or an option on settings to show them)...

Thanks for the issue! I'll try to reproduce it first and then fix it. It seems a parser problem, but let me check.

As you can see, it's the parser that doesn't interpret the multiline comments properly. I'll try to do a PR there, although I see that the last commit was 9 months ago...

OK, so the PR is here: https://github.com/cooklang/cooklang-ts/pull/25
When it's accepted, I'll update the version I'm using so it works. I hope it's accepted some time soon...

Done! You can try version 0.0.12 now


[ES2017/ES8] spec violation

@ariya

//| can use 'yield' identifier outside generators.
async yield => 1
//| should handle export - throws  "Unexpected token function"
export async function foo() { }

//| should handle export default - throws  "Unexpected token"
export default async function() { }

//| cannot combine with generators - throws  "Unexpected token"
async function* foo() { }

And this should throw syntax error, but does not
//| syntax error if `=>` didn't exist.
async ({a = b})

//| - cannot use 'await' inside async functions.
(async function foo() { return {await} })

//| cannot use 'await' inside async functions.
async await => 1

//| cannot use 'await' inside async functions.
async (await) => 1
//| cannot use 'await' inside async functions.
async ({await}) => 1
//| cannot use 'await' inside async functions.
async await => 1

//| cannot use 'await' inside async functions.
async ({a: await}) => 1
//| cannot use 'await' inside async functions.
async ([await]) => 1

There are loads of other similar combinations, but this one is what I had time to find for now.

The await keyword is a reserved word, and should throw if {sourceType: 'module'}. But it throws for both SourceTypes.

Thanks for the report @nowindowsowrking! The coverage of early error is not complete yet, as acknowledged in the implementation (PR #1597).

@nowindowsowrking  For clarity, I may need to edit some of the descriptions to follow our guide on Filing a Defect (feel free also to adjust the report yourself).


Invalid pointer when precompiled with SnoopPrecompile

I tried using SnoopPrecompile.jl on a method that called Voronoi from DirectQhull.jl. Unfortunately, it seems there is some pointer loaded during module initialization that does not persist between different loads of the package: I wonder if these pointers should be initialized in an __init__ function?
Representative error message below:
ERROR: pointerref: invalid pointer
Stacktrace:
  [1] unsafe_load (repeats 2 times)
    @ .\pointer.jl:111 [inlined]
  [2] qh_visit_voronoi(qh_ptr::Ptr{DirectQhull.qhT}, ridges::DirectQhull.RidgesT, vertex_ptr::Ptr{DirectQhull.QHvertexT{HD}}, vertexA_ptr::Ptr{DirectQhull.QHvertexT{HD}}, centers_ptr::Ptr{DirectQhull.QHsetT{Ptr{DirectQhull.QHfacetT{HD}}}}, unbounded::UInt32)
    @ DirectQhull C:\Users\tchr\.julia\packages\DirectQhull\Ngrfk\src\DirectQhull.jl:1334
  [3] qh_eachvoronoi_all
    @ C:\Users\tchr\.julia\packages\DirectQhull\Ngrfk\src\DirectQhull.jl:677 [inlined]
  [4] qh_get_voronoi_diagram(qh_ptr::Ptr{DirectQhull.qhT}, num_input_pnts::Int64, #unused#::Val{3})
    @ DirectQhull C:\Users\tchr\.julia\packages\DirectQhull\Ngrfk\src\DirectQhull.jl:1382
  [5] DirectQhull.Voronoi(pnts::Matrix{Float64}, qhull_options::Vector{String})
    @ DirectQhull C:\Users\tchr\.julia\packages\DirectQhull\Ngrfk\src\DirectQhull.jl:532
  [6] Voronoi
    @ C:\Users\tchr\.julia\packages\DirectQhull\Ngrfk\src\DirectQhull.jl:508 [inlined]
  [7] wignerseitz(basis::ReciprocalBasis{2}; merge::Bool, Nmax::Int64)
    @ Brillouin.WignerSeitz C:\Users\tchr\Dropbox (MIT)\Projects\photonic-topo-sym\julia\Brillouin\src\WignerSeitz.jl:119
  [8] wignerseitz
    @ C:\Users\tchr\Dropbox (MIT)\Projects\photonic-topo-sym\julia\Brillouin\src\WignerSeitz.jl:104 [inlined]
  [9] _precompile(sgnum::Int64, Rs::Vector{SVector{2, Float64}}, Dᵛ::Val{2})
    @ Main .\REPL[6]:5
 [10] macro expansion
    @ .\REPL[8]:2 [inlined]
 [11] top-level scope
    @ .\timing.jl:273 [inlined]
 [12] top-level scope
    @ .\REPL[8]:0

Well, I can't infer from that error message what could cause it.
The qhull memory is reserved abd initialized in the Voronoi constructor.
The failing unsafe_load comes from a julia function (qh_visit_voronoi) that is given to qhull as cfunction.
The invalid pointer should be allocated inside qhull and given to qh_visit_voronoi as parameter.
Maybe the SnoopPrecompile has some incompatibility with the giving julia function as parameter to qhull. At least that is only thing I can think of now.

After trying for ~1 hour, I somehow also cannot reproduce this anymore: sorry about the noise - I'll close this again.
For posterity, a possible MWE could have been something like this (but works fine for me now...):

Generate a new project, say DebugSnoop.jl, with dependencies on SnoopPrecompile.jl and DirectQhull.jl
Put this in /src/DebugSnoop.jlmodule SnoopDebug

using SnoopPrecompile
using DirectQhull

f(x) =  Voronoi(x .* rand(2,5))
@precompile_all_calls begin
    f(2)
end
end # module

See if using DebugSnoop crashes - now it doesn't.
Check which methods got precompile by doing (before using DebugSnoop):using SnoopPrecompile
SnoopPrecompile.verbose[] = true
include("src/DebugSnoop.jl")


[SPARK-25815][k8s] Support kerberos in client mode, keytab-based token renewal.

This change hooks up the k8s backed to the updated HadoopDelegationTokenManager,
so that delegation tokens are also available in client mode, and keytab-based token
renewal is enabled.
The change re-works the k8s feature steps related to kerberos so
that the driver does all the credential management and provides all
the needed information to executors - so nothing needs to be added
to executor pods. This also makes cluster mode behave a lot more
similarly to client mode, since no driver-related config steps are run
in the latter case.
The main two things that don't need to happen in executors anymore are:

adding the Hadoop config to the executor pods: this is not needed
since the Spark driver will serialize the Hadoop config and send
it to executors when running tasks.

mounting the kerberos config file in the executor pods: this is
not needed once you remove the above. The Hadoop conf sent by
the driver with the tasks is already resolved (i.e. has all the
kerberos names properly defined), so executors do not need access
to the kerberos realm information anymore.

The change also avoids creating delegation tokens unnecessarily.
This means that they'll only be created if a secret with tokens
was not provided, and if a keytab is not provided. In either of
those cases, the driver code will handle delegation tokens: in
cluster mode by creating a secret and stashing them, in client
mode by using existing mechanisms to send DTs to executors.
One last feature: the change also allows defining a keytab with
a "local:" URI. This is supported in client mode (although that's
the same as not saying "local:"), and in k8s cluster mode. This
allows the keytab to be mounted onto the image from a pre-existing
secret, for example.
Finally, the new code always sets SPARK_USER in the driver and
executor pods. This is in line with how other resource managers
behave: the submitting user reflects which user will access
Hadoop services in the app. (With kerberos, that's overridden
by the logged in user.) That user is unrelated to the OS user
the app is running as inside the containers.
Tested:

client and cluster mode with kinit
cluster mode with keytab
cluster mode with local: keytab
YARN cluster with keytab (to make sure it isn't broken)

Test build #98333 has started for PR 22911 at commit 7f86e23.

Test build #98333 has finished for PR 22911 at commit 7f86e23.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98333/
Test PASSed.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/4680/

@ifilonenko @skonto

Just a note, this PR has some overlaps with https://github.com/apache/spark/pull/22915. But just after a brief skim, I am a bit hesitant on storing keytabs in secrets, but it might have to be done for client mode support.

But whether to store it in secrets is not a question. You either store it in a secret or you don't support the keytab/principal feature in Spark at all, and we can delete a bunch of code here.

But can we, for a specific mode, client or cluster, tune whether the keytab is put into a secret? Right now, if a keytab is specified, regardless, a secret is created. I do wish to support keytab/principal so I think that storage in secrets should be fine.

To address the comments I had to change a whole bunch of things; mainly because I noticed that the client and cluster paths were really different, when they should for the most part be similar.  Check the updated PR description for details.

Test build #98832 has started for PR 22911 at commit a129314.

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/5029/

Test build #98833 has started for PR 22911 at commit d8416cd.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5029/
Test PASSed.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/5030/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/5030/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5030/
Test PASSed.

Test build #98838 has started for PR 22911 at commit d4a84bc.

Test build #98833 has finished for PR 22911 at commit d8416cd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98833/
Test PASSed.

Test build #98842 has started for PR 22911 at commit 7e9b3a2.

Test build #98846 has started for PR 22911 at commit 78b76a8.

Test build #98838 has finished for PR 22911 at commit d4a84bc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98838/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5032/
Test FAILed.

Test build #98842 has finished for PR 22911 at commit 7e9b3a2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

--keytab is a simpler way IMO to enable long running spark on k8s than DT renew approach

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5033/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5036/
Test FAILed.

retest this please

Test build #98880 has started for PR 22911 at commit 78b76a8.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5058/
Test FAILed.

test this please

Test build #98885 has started for PR 22911 at commit 78b76a8.

Merged build finished. Test FAILed.

Test build #98880 has finished for PR 22911 at commit 78b76a8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98880/
Test PASSed.

Test build #98885 has finished for PR 22911 at commit 78b76a8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98885/
Test PASSed.

Test build #99404 has started for PR 22911 at commit 88f1bb5.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/5477/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/5477/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5477/
Test PASSed.

Test build #99404 has finished for PR 22911 at commit 88f1bb5.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99404/
Test FAILed.

Test build #99627 has started for PR 22911 at commit 05333ea.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/5684/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/5684/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5684/
Test PASSed.

there was a seemingly corrupt xml file in the jenkins worker, I removed it and will retest.

retest this please

on a non-testing not, any further feedback here?

Test build #99630 has started for PR 22911 at commit 05333ea.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/5686/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/5686/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5686/
Test PASSed.

Test build #99842 has started for PR 22911 at commit 4143603.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/5866/

Test build #99842 has finished for PR 22911 at commit 4143603.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class ArrowCollectSerializer(Serializer):
class CSVInferSchema(val options: CSVOptions) extends Serializable
class InterpretedSafeProjection(expressions: Seq[Expression]) extends Projection
sealed trait DateTimeFormatter
class Iso8601DateTimeFormatter(
class LegacyDateTimeFormatter(
class LegacyFallbackDateTimeFormatter(
sealed trait DateFormatter
class Iso8601DateFormatter(
class LegacyDateFormatter(
class LegacyFallbackDateFormatter(

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99842/
Test FAILed.

retest this please

Test build #99932 has started for PR 22911 at commit 4143603.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/5938/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/5938/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/5938/
Test PASSed.

Test build #99932 has finished for PR 22911 at commit 4143603.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class ArrowCollectSerializer(Serializer):
class CSVInferSchema(val options: CSVOptions) extends Serializable
class InterpretedSafeProjection(expressions: Seq[Expression]) extends Projection
sealed trait DateTimeFormatter
class Iso8601DateTimeFormatter(
class LegacyDateTimeFormatter(
class LegacyFallbackDateTimeFormatter(
sealed trait DateFormatter
class Iso8601DateFormatter(
class LegacyDateFormatter(
class LegacyFallbackDateFormatter(

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99932/
Test PASSed.

Test build #100047 has started for PR 22911 at commit cce3f1d.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/6036/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/6036/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6036/
Test PASSed.

Test build #100047 has finished for PR 22911 at commit cce3f1d.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100047/
Test FAILed.

No more comments here and I plan to push this tomorrow based on the +1s I have.

Hi @vanzin , great work!  I'd just been trying to get this working and was happy to find your PR.  I'm having some trouble getting it working, though - not sure whether an issue in my env or if there's a condition that's not getting caught, hoping you or others could shed some light:
Summary:

Executors don't seem to be logging in under any scenario using the delegation token
In client mode, driver connects to hdfs and gets delegation token but the executors don't seem to be using it
In cluster mode, seems driver pod doesn't get hadoop configs mapped in (this had been working in previous version) so doesn't try using kerberos keytab.  Once I manually add the hadoop configs the driver is able to connect to hdfs, but the executors fail

Environment overview:

Kerberized HDFS in traditional hadoop cluster
Launching spark into separate kubernetes cluster, trying to access a parquet dataset

Steps:

Cloned your branch, commit cce3f1dd32c204f5c57b3db3a79d49895512f0a5 from today
Compiled, created docker images
kinit-ed
Tried running a small python program to access hdfs, launching in local, cluster, and client modes

Results - Local mode
./bin/spark-submit \
--master local[4] \
--name spark-hdfs \
--principal<EMAIL_ADDRESS>\
--keytab keytab.kt \
https://xxx.com/spark/hdfs_test.py

works, program runs and is able to load parquet and print the first row
Results - Client mode
./bin/spark-submit \
--master k8s://https://$KUBERNETES_SERVICE_HOST:$KUBERNETES_SERVICE_PORT \
--deploy-mode client \
--conf spark.driver.host=$(getent hosts $HOSTNAME | cut -f1 -d' ') \
<pod-spec-details>
--conf spark.kubernetes.kerberos.krb5.path=/path/krb5.conf \
--conf spark.kerberos.keytab=keytab.kt \
--conf<EMAIL_ADDRESS>\
https://xxx.com/spark/hdfs_test.py

Driver is able to login to the hdfs server, acquires a delegation token, starts executors
Executors go to connect to HDFS, error out with the following:
2018-12-14 00:13:41 WARN  Client:683 - Exception encountered while connecting to the server : org.apache.hadoop.security.AccessControlException: Client cannot authenticate via:[TOKEN, KERBEROS]
2018-12-14 00:13:41 ERROR Executor:95 - Exception in task 0.3 in stage 0.0 (TID 3)
org.apache.spark.SparkException: Exception thrown in awaitResult:
        at org.apache.spark.util.ThreadUtils$.awaitResult(ThreadUtils.scala:226)
        at org.apache.spark.util.ThreadUtils$.parmap(ThreadUtils.scala:290)
...
Caused by: java.io.IOException: Failed on local exception: java.io.IOException: org.apache.hadoop.security.AccessControlException: Client cannot authenticate via:[TOKEN, KERBEROS]; Host Details : local host is: "hdfs-test-1544746404469-exec-1/<IP_ADDRESS>"; destination host is: "xxx.com":5555;
        at org.apache.hadoop.net.NetUtils.wrapException(NetUtils.java:776)
        at org.apache.hadoop.ipc.Client.call(Client.java:1480)
        at org.apache.hadoop.ipc.Client.call(Client.java:1413)

Seems like they're not trying to connect with the delegation token?
Results - Cluster mode
./bin/spark-submit \
--master k8s://https://$KUBERNETES_SERVICE_HOST:$KUBERNETES_SERVICE_PORT \
--deploy-mode cluster \
<pod-spec-details>
--conf spark.kubernetes.kerberos.krb5.path=/path/krb5.conf \
--conf spark.kerberos.keytab=keytab.kt \
--conf<EMAIL_ADDRESS>\
https://xxx.com/spark/hdfs_test.py

Driver and executor pods are launched
Driver tries to connect to hdfs with SIMPLE authentication
Found that hadoop configs weren't being propagated to driver pod, even when adding the --conf spark.kubernetes.hadoop.configMapName option
When I manually added the configs and set the driver environment variable HADOOP_CONF_DIR, driver now is able to connect to hdfs and get delegation token, but executors have the same problem observed in client mode

I've been running tests with the Spark HdfsTest example; maybe pyspark does something different. I'll try that tomorrow.
I haven't really tried the config map route, only HADOOP_CONF_DIR.

This might be a bug, I've complained in the past that HDFS stuff doesn't seem to work unless you set HADOOP_CONF_DIR in the submission clients environments

This one might have been just a bad merge on my part. Without integration tests this is hard to catch (and I was having trouble with minikube yesterday).
I'll try out my tests on minikube tomorrow.

Test build #100118 has started for PR 22911 at commit ccb3956.

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/6103/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6103/
Test PASSed.

Test build #100118 has finished for PR 22911 at commit ccb3956.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100118/
Test FAILed.

Had a chance to try it - the driver pod is now getting the keytab mounted and is able to generate delegation token, so that's looking good.  I'm still having an issue where the executors seem to not be using the dt and so are getting denied access - are they also supposed to have hadoop_conf_dir set?

@vanzin - how are you compiling spark?  Specifically, what version of hadoop are you testing with?  I'm working with v2.7.7, wondering if there are changes in later versions to the ipc.client and usergroupinformation classes that may handle delegation tokens differently

I've tried this again in all configurations I know (and also pyspark) and it's working for me:

cluster mode: keytab, local: keytab, no keytab
client mode: keytab / no keytab, client in my laptop, client in k8s pod
spark-shell and pyspark shell with tgt

I'm building Spark normally. Make sure your images are updated in docker (remember that the pyspark image is not build by default anymore).
You should see a log message like this in the executors:
2018-12-14 18:52:00 INFO  SparkHadoopUtil:58 - Updating delegation tokens for current user.

Update - my setAuthenticationMethod change didn't work - after calling that I see that the executor reports it's trying to use TOKEN method, but it's still getting rejected with an error that it can't login via TOKEN, KERBEROS:
18/12/14 23:26:01 DEBUG UserGroupInformation: PrivilegedActionException as:user (auth:TOKEN) cause:org.apache.hadoop.security.AccessControlException: Client cannot authenticate via:[TOKEN, KERBEROS]
18/12/14 23:26:01 DEBUG UserGroupInformation: PrivilegedAction as:user (auth:TOKEN) from:org.apache.hadoop.ipc.Client$Connection.handleSaslConnectionFailure(Client.java:651)
18/12/14 23:26:01 WARN Client: Exception encountered while connecting to the server : org.apache.hadoop.security.AccessControlException: Client cannot authenticate via:[TOKEN, KERBEROS]

As a test I tried mounting in my keytab and hadoop_conf_dir into the executors and setting KRB5CCNAME env var (via pod template) - this results in the main user being kerberized, then I commented out the "runassparkuser" section of the coarsescheduler, and now the executors are able to successfully login to HDFS and grab the parquet data.
It looks like there's some incompatibility in my environment with how the delegation tokens are being used - I think this shouldn't block this merge if it's working for others - is there any other venue people would suggest I continue trying to debug?

I suggest filing a bug or starting a mailing list thread. But really I doubt anyone will be able to help if you only provide the errors, without explaining your environment and, especially, the code you're running.

retest this please

Test build #100257 has started for PR 22911 at commit ccb3956.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/6229/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/6229/

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/6229/
Test PASSed.

Test build #100257 has finished for PR 22911 at commit ccb3956.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/100257/
Test PASSed.

The main two things that don't need to happen in executors anymore are:
adding the Hadoop config to the executor pods: this is not needed
since the Spark driver will serialize the Hadoop config and send
it to executors when running tasks.

gentle ping @vanzin
seems the executor still need hadoop config.


INDArray shape() return int[] instead of int cause Nd4j.zeros() error

https://gist.github.com/olafhel/566ab93b2ecbd65c80e4

caused by f139952229efad88a1e0e3f5ba6f8d2fc0099ff6

Actually the problem occur when building dl4j-spark-ml.

Fixed  in deeplearning4j/deeplearning4j@0850a4fced80d0b27d655d2f13d17f02c74a4d6f


Submodules of submodules are not merged into parent repository

If you run git-submodule-rewrite on a submodule that has its own submodules they will be lost after the consolidation, rather than being merged into the parent repository's .gitmodules.
(Yes, we did have far too many submodules!)

Oh, wow.  Yeah, I don't currently support this.


Fix github link text

I have no idea if this works (haven't tested) -- this website is pretty complex. Should fix the button text, though! :wink:

Awesome!


Update to Pytorch 1.0

Pyro currently works with PyTorch 0.4.0 (0.4.1 support was abandoned #1267).
Tasks

[x] #1431 Update code
[x] Switch networkx dependency to 0.2.2 (see #1191)
[ ] Finish release notes
[ ] Cut a 0.2.2 release

Closing in favor of #1570


Change ./reactivesearch directory to chorus_ui

Change the demo directory to make it easier for others to know where the ui code is.

renamed to chorus_ui


Change color list/methods to LED* to match hardware labels

This PR change the previous LEDs methods, to use board labels instead colors.

[ ] https://github.com/bluerobotics/navigator-rs/issues/32

thks, all fix applied

@patrickelectric It's rebased


Provide a mechanism to to report disabled tests

TestNG Version

Note: only the latest version is supported

7.4.0
Currently TestNG does not have a way of reporting tests that were disabled. It should be available through either the current reporting mechanisms or through some other means so that TestNG users can keep a track on disabled tests as well.

Verified this in the latest master branch (which will likely be released as 7.8.0. This capability exists as of today via
org.testng.ITestContext#getExcludedMethods and then filtering out methods where org.testng.ITestNGMethod#getEnabled returns false. Those would be the methods which have been :

annotated using @Ignore (or)
have the setEnabled=false via the @Test annotation directly or indirectly via an IAnnotationTransformer implementation.


G cloud links

Delivers: https://www.pivotaltracker.com/story/show/93097694

Story to prefix search and services pages with the framework name

As currently we only have a single framework this is hardcoded to g-cloud

Added redirects to the existing urls to the new ones

Decorated these old routes with the deprecated annotation

Added tests for new and old endpoints

Fixed bug where empty keywords caused an error.

Can't be merged since it's behind master now, but other than that LGTM :+1:
It's great that we're transitioning links to url_for in templates!


MakieLayout: sequence of specifying layouts matters?

Hi, trying to use makielayout and want to use it in a nested way.
IMHO the following should work, and it would allow to define subscenes  of various kinds and allow to arrange them in an arbitrary fashion:
using MakieLayout, GeometryBasics

function should_work()
    scene, layout = layoutscene(resolution=(1000,500))
    layoutL=GridLayout()
    layoutR=GridLayout()
    volmesh=GeometryBasics.Mesh([Point3f0(0,0,0),Point3f0(1,1,1)],Vector{GLTriangleFace}(undef,0))
    function make_scene!(scene,layout,color)
        myscene= layout[1,1] = LScene(scene, camera = cam3d!, raw = false,scenekw = (clear = true,))
        myslider=layout[2,1] = LSlider(scene, range = 4:1:10000, startvalue = 4,width = Auto())
        mesh!(myscene,volmesh)
        node=Node(rand(myslider.value[],3))
        scatter!(myscene,lift(a->a,node),color=color);
        onany(myslider.value) do v
            node[]=rand(v, 3)
        end
    end
    make_scene!(scene,layoutL,:blue)
    make_scene!(scene,layoutR,:red)
    layout[1, 1] = layoutL
    layout[1, 2] = layoutR
    scene
end

This gives:

What does work is:
function does_work()
    scene, layout = layoutscene(resolution=(1000,500))
    layoutL=GridLayout()
    layoutR=GridLayout()
    volmesh=GeometryBasics.Mesh([Point3f0(0,0,0),Point3f0(1,1,1)],Vector{GLTriangleFace}(undef,0))

    mysceneL= layoutL[1,1] = LScene(scene, camera = cam3d!, raw = false,scenekw = (clear = true,))
    mysceneR= layoutR[1,1] = LScene(scene, camera = cam3d!, raw = false,scenekw = (clear = true,))
    mysliderL=layoutL[2,1] = LSlider(scene, range = 4:1:10000, startvalue = 100,width = Auto())
    mysliderR=layoutR[2,1] = LSlider(scene, range = 4:1:10000, startvalue = 100,width = Auto())

    mesh!(mysceneL,volmesh)
    nodeL=Node(rand(mysliderL.value[],3))
    scatter!(mysceneL,lift(a->a,nodeL),color=:blue);
    onany(mysliderL.value) do v
        nodeL[]=rand(v, 3)
    end

    mesh!(mysceneR,volmesh)
    nodeR=Node(rand(mysliderR.value[],3))
    scatter!(mysceneR,lift(a->a,nodeR),color=:red);
    onany(mysliderR.value) do v
        nodeR[]=rand(v, 3)
    end

    layout[1, 1] = layoutL
    layout[1, 2] = layoutR
    scene
end

It gives:

So in the moment it appears that I am forced to fill the layout grid row by row.
Bug or feature ?

This
LScene(scene, camera = cam3d!, raw = false,scenekw = (clear = true,))

should be
LScene(scene,scenekw = (camera = cam3d!, raw = false, clear = true))

Does that fix your problem?

Thanks - it looks right now, but at the expense of not being able to rotate the left plot. The sliders work though.
In your layout example from zulip  I can have two 3D scatter plots working.
In fact I would like to get rid of the need to use sliders as they eat up screen real estate.
I'd rather use arrow keys or the mouse (with "mode switching" through some hotkeys) to control things - main purpose is here interactively controlling isolevels or isosurfaces in 3D.  For this purpose there should be a way to distinguish the LScene where they happen.

ok figured  this out so far - just check if mouse position is in pixel area....
working now  to catch up with the big update....

ok figured  this out so far - just check if mouse position is in pixel area....
working now  to catch up with the big update....


Add and use FeeType enum

This change is

Codecov Report

Merging #900 (dad2005) into dori/rename-trait (17069b8) will decrease coverage by 0.04%.
The diff coverage is 54.54%.

@@                  Coverage Diff                  @@
##           dori/rename-trait     #900      +/-   ##
=====================================================
- Coverage              69.70%   69.67%   -0.04%
=====================================================
  Files                     42       42
  Lines                   5741     5744       +3
  Branches                5741     5744       +3
=====================================================
  Hits                    4002     4002
- Misses                  1367     1370       +3
  Partials                 372      372

Files Changed
Coverage Δ

crates/blockifier/src/block_context.rs
43.75% <33.33%> (-17.37%)
:arrow_down:

crates/blockifier/src/transaction/objects.rs
44.44% <80.00%> (+5.73%)
:arrow_up:

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


add fulcio-grpc as knative service

Signed-off-by: Bob Callaway<EMAIL_ADDRESS>
Summary

Release Note

Documentation

@vaikas I think once this is merged (assuming you're OK with it), we'll need to cut 0.4.13 release and then bump the references inline up to 0.4.13 and then cut another release or update the tag?

Thanks @bobcallaway I have no problem at all running parallel Fulcio, though shortly I'd like to still investigate the use the same port for both grpc/http :)
Something like this (that just doesn't rely on HTTP/2 for muxing):
https://github.com/chainguard-dev/go-grpc-kit


[WFLY-16193] Upgrade netty from 4.1.72 to 4.1.75

Fixes https://issues.redhat.com/browse/WFLY-16193

@pferraro @ehsavoie Could you review and approve, please? All links are in the Jira.

:+1: wrt Infinispan.

@ehsavoie Could you have a look at this, please? I'm hoping to backport it to 26.1.1 in time.


feat: add option to run with coverage #279

I'm not sure what you git process is and how this build, version bumps or publishing work so I hope this PR is ok it is just code changes.
I didn't add any new tests because it is pretty tricky trying to write tests for jestRunner.ts because you can't easily mock service calls. I think this might be a good follow up PR.

i removed the additional options from the context-menu to not spam the context-menu with to many options.
its still available in command-palette and configurable for code-lens.
i also adjusted some labels to make it more clear.
thx for your contribution.

That sounds good to me thanks @firsttris


Add 'arm-latest' tag

This is for 'ubuntu.1404.arm32.open' Helix queue arm Tegra machines.
Usage in dotnet/coreclr netci.groovy will be like:
Utilities.setMachineAffinity(job, 'Ubuntu', 'arm-latest')

@mmitche @jashook PTAL
I don't have write access to this repo.
If this is taken, how and when does it get deployed?

@BruceForstall This is not needed.  In the netci.groovy file, you simple use:
setMachineAffinity('ubuntu.1404.arm32.open')
No changes in dotnet-ci needed.

If I just call:
Utilities.setMachineAffinity(job, 'Ubuntu', 'ubuntu.1404.arm32.open')

then it will call:
String machineLabel = Agents.getAgentLabel('Ubuntu', 'ubuntu.1404.arm32.open')

which looks like it will assert if it doesn't find the tag:
def machineLabel = versionLabelMap.get(version, null)
assert machineLabel != null : "Could not find version ${version} of ${osName}"

On the other hand, I suppose I could skip the Utilities.setMachineAffinity() call altogether and just do:
        job.with {
            label('ubuntu.1404.arm32.open')
        }

But that seems like maybe short-circuiting the abstraction layer.
Comments?

Ahh yeah no overload for that.  just use
job.with {
  label('ubuntu.1404.arm32.open')
}

The abstraction layer isn't doing much here.


[BUG] props is not defined on shortcut elements

Hey - Aren't your missing this.props instead of props?
export default class VBox extends Component {
  render() {
    return <Box {...props} column />
  }
}

Also, why not creating pure function components such as:
export default (props) => (<Box {...props} column />);

Oh damn, I missed that while transforming them, they've been pure functions before, but that does not play well with hot reloading which was the reason why I used classes again.
If you like you could submit a pull request, else I will do it when I am back home in 3 days :p

Haha true that pure functions don't work well with HMR, I know they're working on it though! However, out of curiosity, would that mean that having a pure function VBox would break HMR for any component using a VBox (even if that component is not a pure function)? I'll do the PR but I won't be able to test it :S


Doubt about replacing rules

I'm doing a digital twin with fuzzy logic. Anyways, sometimes I need to replace a rule for another that represents better the dynamic of the system. Is it possible to do this replacement without load all the rules every time I need to update the rules? It's different of just "append" the rules, I would need a way to replace a specific rule for another.
I thank you for the package, it's very simple to use and I already thank you for the attention!

Hi @JoseVinicius1998 , thanks for your interest in simpful!
@aresio just added a new method to the FuzzySystem object that allows you to replace rules.
It works like this:
FS.replace_rule(i=1, rule="IF (oxygen IS high) THEN (power IS medium)")
where i is the index of the rule to be substituted. Rules are stored in the object in the same order as they are added.
You can also check the current rule base with get_rules.

Excellent, that's very important to save time during update the system's rule. And, another asking: what is the fastest way to defuzzy the system's inputs? I'm using the Mandani_inference() method. It's very strong method, but a little slow to my application, once i have 6 inputs (yes, i know, that's a little much) and i spending about 2 seconds to defuzzy


getStockTransactions() can return null wallet_tx_id

A band-aid solution for expected return values of wallet-tx-id
Doesn't change the logic of backend (ie. still treated as empty string)
May want to expand upon later in development

DON'T MERGE:
found some faulty logic

Fixed the issue.
We will need to add more error checking to the engine to handle unexpected values somewhere down the line


Create wi_to_dwc.R

@peterdesmet, Here is the report from the gbif validator on my dataset:
https://www.gbif.org/tools/data-validator/6467d85c-2ec3-4db5-982f-832de2e75ce4.
Let me know if you have feedback. I'm not really sure about deploymentRemark.

Apparently I need to be logged in to see that report. Can you send me another way?

Ignore, I can see it now.

Could you send me the produced file(s)?

Further feedback on the mapping:

[ ] type: use StillImage after all (based on feedback from @tucotuco), unless the source is a video file
[ ] rightsHolder: can this be derived from the project metadata
[ ] datasetID: I would use the project ID or URL here (if no DOI is available)
[ ] collectionCode: I would set this to Wildlife Insights (as a virtual collection) or (if known by that acronym) WI (currently does not seem to be used by other collection)
[ ] recordedBy: what person is this, the person who set up the camera? I left this one out, because it's a machine obs
[ ] behavior: can this be recorded in WI?
[ ] occurrenceRemarks: I would put individual_animal_notes here
[ ] eventID: either imageID or sequenceID, not deploymentID
[ ] parentEventID: deploymentID, not projectID
[ ] habitat: is this info available in WI (other than featuretype)
[ ] samplingEffort: write as YYYY-MM-DDTHH:MM:SSZ/YYYY-MM-DDTHH:MM:SSZ (or just the dates), currently 2020-11-10 UTC--2021-02-10 UTC
[ ] locality: why are those the same values as eventID? Do they always look
[ ] locationRemarks: I would put featureType here
[ ] coordinateUncertaintyInMeters: is it possible to estimate this for all records?
[ ] dateIdentified: is this information available?
[ ] scientificName: can this be placed above kingdom?

Further feedback on the mapping:

[x]  type: use StillImage after all (based on feedback from @tucotuco), unless the source is a video file
[ ]  rightsHolder: can this be derived from the project metadata
There is a project_admin_organization (e.g. A Rocha Kenya) and project_admin (e.g. Raphael Nussbaumer)(as well as email). But this is different I thought?
[ ]  datasetID: I would use the project ID or URL here (if no DOI is available)
The only field which might be helpful is : data_citation (e.g. "Nussbaumer, Raphaël\nKirao, Lennox\nJackson, Colin. 2019. Last updated July 2022. Dakatcha Woodland Biodiversity Assessment. http://n2t.net/ark:/63614/w12001317. Accessed via wildlifeinsights.org on 2022-07-26."). What do you think of extracting  http://n2t.net/ark:/63614/w12001317 ? Is this equivalent to a doi?
[x]  collectionCode: I would set this to Wildlife Insights (as a virtual collection) or (if known by that acronym) WI (currently does not seem to be used by other collection)
[x]  recordedBy: what person is this, the person who set up the camera? I left this one out, because it's a machine obs
It's also empty in my dataset, but I guess that it's possible to set it up on WI. I leave it out for now.
[ ]  behavior: can this be recorded in WI?
Not directly. There is the remark field but no designated one. Delete?
[x]  occurrenceRemarks: I would put individual_animal_notes here
[x]  eventID: either imageID or sequenceID, not deploymentID
Just to confirm that we have image_id on both eventID and occurenceID
[x]  parentEventID: deploymentID, not projectID
[x]  habitat: is this info available in WI (other than featuretype)
no, not that I can find.
[ ]  samplingEffort: write as YYYY-MM-DDTHH:MM:SSZ/YYYY-MM-DDTHH:MM:SSZ (or just the dates), currently 2020-11-10 UTC--2021-02-10 UTC
[x]  locality: why are those the same values as eventID? Do they always look
No, this is particular to our case where we set up camera at new location every single time. (Don't ask me why....)
[x]  locationRemarks: I would put featureType here
[ ]  coordinateUncertaintyInMeters: is it possible to estimate this for all records?
Not sure how to do that generically... In our case, we measure position with the phone gps and then manually include the information on WI. But different location source would lead to different uncertainty, no?
[ ]  dateIdentified: is this information available?
Strangely not. It is visible on the platform, but I cannot find it in the exported file. I've sent a request to WI about this.
[x]  scientificName: can this be placed above kingdom?

rightsHolder: This is one of the fields I let the user set in the function write_dwc(), because it's hard to derive from metadata
datasetID: Yes http://n2t.net/ark:/63614/w12001317 would be very good as a datasetID (it's stable and it references a study/dataset uniquely)
behavior: delete then
eventID: "Just to confirm that we have image_id on both eventID and occurrenceID" yes correct. Since you are using image_id, I'm assuming that (for this dataset perhaps), you are not grouping observations (into one) for sequences?
coordinateUncertainty: This is a really useful field to populate. 30 would be good default value when positions were recorded using GPS, so I would use that for your dataset. A bit trickier to use though as a generic value for the function (and other WI datasets).

@peterdesmet , I've added rightsHolder and coordinateUncertainty as parameter. What to you think of my suggestion to have WI project_admin_organization as default?

Yeah, that's probably a good default.

Are you not getting any mediaID returned by the WI export?

Or the timestamp (xmp:CreateDate) of the image? Which I assume is the timestamp of the observation?

Are you not getting any mediaID returned by the WI export?

How would it be different from image_id?

Or the timestamp (xmp:CreateDate) of the image? Which I assume is the timestamp of the observation?

timestamp is used for the eventDate.
I've set up my project as image based. Maybe sequence based project have a different export format?

[ ] Can you add mediaID to Audubon and populate it with image_id (it's ok that this is a duplicate of occurrenceID)
[ ] Can you add CreateDate to Audubon and populate it with timestamp?
[ ] Can you correct typo in filename to tests/testthat/test-write_dwc_wi.R
[ ] Can you rename directory to wi_export_sample?


rewrite the ruleset

Summary of the Pull Request

PR Checklist

[ ] Closes: #xxx
[ ] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[ ] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

I'm trusting this is valid. A bit of escription would be nice, though :)


Test suite doesn't tsc type-check

The current test suite under spec/ is being transpiled to JS and executed, but no type-checking is currently being enforced. As a result of that, lots of typing errors have been introduced into the test suite.
Repro:
yarn tsc -p spec/tsconfig.json

Expected: no type errors
Actual: lots and lots of errors
Once this problem is resolved, the tsc typecheck should be added to the CI suite so that we can avoid introduction of new errors in future commits.

@david-driscoll is working on this. https://github.com/ReactiveX/rxjs/pull/3329 . I've accidentally closed https://github.com/ReactiveX/rxjs/issues/3020 .

I think we can use this one until it's resolved, #3020 doesn't have additional detail specifically worth to reopen.

I can't cope any longer with the test suite's not being type checked. I see that #3329 has been closed:

... in favor of a more incremental approach.

I'm going to start work on an incremental approach now. Unless anyone has objections, I plan on doing the following:

Create a temporary spec/tsconfig.check.json file.
Specify an include in said file so that files can be incrementally checked and fixed. E.g. 'include': ['spec/operators/a*.ts']
Create a temporary script named test_check (or whatever) in the package.json to run a no-emit  tsc build that uses spec/tsconfig.check.json.
Include npm test_check in the .travis.yml configuration file so that the CI build runs it.
Incrementally update the -spec.ts files so that they pass the check. That is, update them with individual PRs or, at least, in very small batches.
When all -spec.ts files pass, remove the temporary scripts from the package.json and the .travis.yml files, delete the spec/tsconfig.check.json file and set the TS_NODE_TYPE_CHECK environment variable in the test script so that ts-node will throw an error if TypeScript errors are introduced into the tests.

Once the test suite is again type checked, I'd like to extend the type tests to include snippet-based testing, as  discussed here and as done here.

👍  Let me know if you have any questions.  I had worked in a bunch of changes before there were some large refactors.
Interestingly most of the required changes to fix the type checking (and enhance the typings themselves) usually boiled down to some simply regex replace with a little bit of manual clean up.

It seems Microsoft have come through with the goods and built a tool for testing type expectations and errors. This looks like it will be pretty useful: Microsoft/dtslint


Sorting Numbers with commas

It makes sense that the sorting algorithm can distinguish that 40000 > 100.
But, for end users, this [front-end] framework should really be able to determine that 40,000 > 100.
The sort currently puts the whole numbers (without commas) at the beginning of the list when sorting in descending order.

100
40,000

Yeah! It does! It's so annoying! How do we get around this?

I ultimately dropped this library from production and replaced it with DataTables.   It does everything that listjs does and is better maintained.

maybe you can do like this
userList.sort('sortPrice', {   sortFunction: function(a) {     let prices = parseFloat(a.values().sortPrice);     a.values().sortPrice = prices;     return a.values().sortPrice;   } });
thank you
i hope this can help you


PXBLUE-2652 Rebranded px blue references

Fixes # PXBLUE-2652

Changes proposed in this Pull Request:

pxblue ==> brightlayer-ui
@pxblue ==> @brightlayer-ui

Update to use latest @brightlayer-ui/chartjs package instead of the beta.


Add code formatting

Are you open to format the code with luafmt?
The code is a bit messy, with mixed 2 and 4 space indentation etc.
No judgment what so ever, I'm just super excited about this and want to help out.
And consistent code would help make contributions easier.

Sure, feel free to send a PR reformating code where needed. I tried to use 2 spaces indentation, but it may not be 💯 consistent. Also, Im new to lua so feel free to suggest any refactoring to get the code more idiomatic.


Register AuthenticatableType in scheme

In #3056 if didn't register the new AuthenticatableType in the scheme. This PR fixes it

/lgtm
/approve
cc @pierDipi


feature (ServiceActionDisabledModal) Moves copy text to icon

This PR removes the "copy" text to copy a command and instead adds an icon to do this. This was a request from the Design team. I've gone over it with them and gotten sign-off on the implementation.
Before

After

Checklist

[ ] Did you add a JIRA issue in a commit message or as part of the branch name?
[ ] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

@rcorral - flicts!

Works great!

LGTM when the tests all passed. I have made a change to remove css specificity.

✅  x 2 + CI is 🍏  = Merge


[Refactor][meta-format-change] Change routine load serialize/deserialize format to json

Fixes #issue

[FE PR Coverage Check]
:disappointed: fail : 50 / 85 (58.82%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/load/routineload/PulsarRoutineLoadJob.java
0
2
00.00%
[83, 88]

:large_blue_circle:
com/starrocks/load/routineload/PulsarProgress.java
0
1
00.00%
[46]

:large_blue_circle:
com/starrocks/server/GlobalStateMgr.java
0
1
00.00%
[1416]

:large_blue_circle:
com/starrocks/journal/JournalEntity.java
0
6
00.00%
[534, 535, 536, 545, 546, 547]

:large_blue_circle:
com/starrocks/load/routineload/RoutineLoadManager.java
8
29
27.59%
[691, 707, 708, 709, 710, 711, 712, 713, 714, 718, 721, 722, 723, 724, 726, 727, 728, 731, 732, 734, 736]

:large_blue_circle:
com/starrocks/persist/EditLog.java
2
4
50.00%
[1741, 1742]

:large_blue_circle:
com/starrocks/load/routineload/RoutineLoadJob.java
19
21
90.48%
[1761, 1762]

:large_blue_circle:
com/starrocks/load/loadv2/LoadJobFinalOperation.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/load/FailMsg.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/load/routineload/KafkaProgress.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/load/loadv2/LoadJob.java
13
13
100.00%
[]

:large_blue_circle:
com/starrocks/load/routineload/KafkaRoutineLoadJob.java
2
2
100.00%
[]

:large_blue_circle:
com/starrocks/load/EtlStatus.java
2
2
100.00%
[]

:large_blue_circle:
com/starrocks/load/loadv2/BulkLoadJob.java
1
1
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 78 / 89 (87.64%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/server/GlobalStateMgr.java
0
1
00.00%
[1416]

:large_blue_circle:
com/starrocks/journal/JournalEntity.java
0
6
00.00%
[534, 535, 536, 545, 546, 547]

:large_blue_circle:
com/starrocks/persist/EditLog.java
2
4
50.00%
[1741, 1742]

:large_blue_circle:
com/starrocks/load/routineload/RoutineLoadManager.java
26
28
92.86%
[726, 727]

:large_blue_circle:
com/starrocks/load/routineload/RoutineLoadJob.java
21
21
100.00%
[]

:large_blue_circle:
com/starrocks/load/routineload/PulsarRoutineLoadJob.java
2
2
100.00%
[]

:large_blue_circle:
com/starrocks/load/loadv2/LoadJobFinalOperation.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/load/FailMsg.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/load/routineload/KafkaProgress.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/load/loadv2/LoadJob.java
13
13
100.00%
[]

:large_blue_circle:
com/starrocks/persist/gson/GsonUtils.java
5
5
100.00%
[]

:large_blue_circle:
com/starrocks/load/routineload/PulsarProgress.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/load/routineload/KafkaRoutineLoadJob.java
2
2
100.00%
[]

:large_blue_circle:
com/starrocks/load/EtlStatus.java
2
2
100.00%
[]

:large_blue_circle:
com/starrocks/load/loadv2/BulkLoadJob.java
1
1
100.00%
[]


Issue with custom css

What version of starlight are you using?
0.24.1
What version of astro are you using?
4.8.6
What package manager are you using?
pnpm
What operating system are you using?
windows
What browser are you using?
Chrome
Describe the Bug
Steps:

Create a css file src/styles/custom.css:

.neutral-badge {
	border-color: var(--sl-color-gray-5);
	background-color: var(--sl-color-gray-6);
	color: var(--sl-color-text);
}

Update astro.config.mjs:

starlight({
	title: 'My Docs',
	customCss: ['./src/styles/custom.css'],
}),

Use Badge component:

<Badge class="neutral-badge" text="Custom" />

Result:
When after build, this src/styles/custom.css is not work. dev mode is work.
Reason:
Badge component use scoped styles, but src/styles/custom.css was bundle to a link tag, like <link rel="stylesheet" href=".astro/xxx.css" />.
Scoped styles will take precedence.
The same is true for other Starlight user-components, which can't override style by customCss
My workaround
Import custom.css in single page(.mdx or .astro)
---
title: Example Guide
description: A guide in my new Starlight docs site.
---

import { Badge } from '@astrojs/starlight/components'
import "../../../styles/custom.css"

<Badge text="Main" />

<Badge class="neutral-badge" text="Custom" />

The above code will bundle custom.css as an scoped style.
It works! Now we have correct custom style.
Issues

I only imported custom.css in one page, but it build for every page. I dont know how it works.
Any other solutions for custom styles? Whether the customCss property can be specified, whether it should be packaged as an scoped style or as an external style link. Like Astro's inlineStylesheets option?

You can reproduce it in my GitHub repository and this docs PR.
Link to Minimal Reproducible Example
https://github.com/liruifengv/badge-test
Participation

[ ] I am willing to submit a pull request for this issue.

I seems get it.

I only imported custom.css in one page, but it build for every page.

Import custom.css in page and use customCss together, will package both to scoped style.

Any other solutions for custom styles? Whether the customCss property can be specified, whether it should be packaged as an scoped style or as an external style link. Like Astro's inlineStylesheets option?

The customCss option only should be use for customize theme by override style vars in props.css.
When user want change the user-components style, should be wrapped for Starlight's user-components instead of use the customCss option.


Smarti Widget doesn't load results when open on message

When typing and sending a message with the knowledgebase-tab open the widget results don't get refreshed automatically.

After closing and reopening the knowledgebase-tab the results are visible:

ToDo
The widget should refresh the analysis results automatically even with open knowledgebase-tab

@ThomasRoehl - which version did you use here? I'm not able to reproduce it with smarti 7fa0edf1 and rocket.chat redlink-gmbh/Rocket.Chat@38dd8aca
end-to-end tests also work: 43 passed, 1 pending

@ja-fra I can't reproduce it as well. Seems to be fixed with the newer version.


Intersection returns a point that doesn't intersects with any of the two geometries

Expected behavior and actual behavior.
Intersection should return geometry whose points intersects with both of the geometries
Steps to reproduce the problem.
from shapely import from_wkt
import matplotlib.pyplot as plt

line2 = "LINESTRING<PHONE_NUMBER>479876 6131642.230652019,<PHONE_NUMBER>223561 6131690.780868441)"
poly2 = "POLYGON ((829479.0270571347 6131645.850985122,<PHONE_NUMBER>511323 6131646.586646849,<PHONE_NUMBER>118628 6131647.689983897,<PHONE_NUMBER>678958 6131648.479587758,<PHONE_NUMBER>909479 6131649.481861163,<PHONE_NUMBER>49062 6131650.435463399,<PHONE_NUMBER>713882 6131650.439804772,<PHONE_NUMBER>618099 6131649.443423415,<PHONE_NUMBER>113995 6131648.132067959,<PHONE_NUMBER>722032 6131646.922705633,<PHONE_NUMBER>117242 6131645.919191298,<PHONE_NUMBER>231631 6131645.484549604,<PHONE_NUMBER>571347 6131645.850985122))"
l2 = from_wkt(line2)
p2 = from_wkt(poly2)

x, y = p2.exterior.coords.xy
plt.plot(x, y)
x, y = l2.coords.xy
plt.plot(x, y)
plt.show()

x,y= l2.intersection(p2).coords.xy
plt.plot(x, y)
x, y = p2.exterior.coords.xy
plt.plot(x, y)
plt.show()

print(l2.intersection(p2).wkt)
# Returns LINESTRING<PHONE_NUMBER>912745 6131645.8797281,<PHONE_NUMBER>370405 6131650.436346635)

print(p2.contains(from_wkt("Point(829478.9907912745 6131645.8797281)")))
# Returns False
print(p2.contains(from_wkt("Point(829479.0711370405 6131650.436346635)")))
# Returns True

Operating system
Ubuntu 22.04.1 LTS
Shapely version and provenance
2.0.1 installed from PyPI using pip

I think this is an upstream GEOS issue; intersection and predicates are coming directly from there.  My understanding is that this is due to floating-point precision issues.  Related comment GEOS #585
This also fails the intersects predicate for one of the points, but passes if you use the dwithin predicate to allow some minor differences:
>>> i = shapely.intersection(p2, l2)
>>> shapely.intersects(p2,shapely.get_point(i, 0))
False
>>> shapely.intersects(p2,shapely.get_point(i, 1))
True

>>> shapely.dwithin(p2, shapely.get_point(i, 0), 1e-9)
True
>>> shapely.dwithin(p2, shapely.get_point(i, 1), 1e-9)
True


Angular range problem

Thanks for your contribution! In the following code does 360 means 2pi?
leapct.set_conebeam(numAngles, numRows, numCols, pixelSize, pixelSize, 0.5*(numRows-1), 0.5*(numCols-1), leapct.setAngleArray(numAngles, 360.0), 1100, 1400)
I'm wondering what if I want to use the sparse-view or limited-angle setting how do I change the code. And how to adjust the start_angle, cause in TIGRE it's easy to do that as following.
start_angle = np.deg2rad(140)
end_angle = start_angle + 2 * np.pi
angles = np.linspace(start_angle, end_angle, 984, dtype=np.float32)
And for the iterative algorithm, I saw you have SART reconstruction and TV denoising, is it easy to combine them together to do the sparse-view reconstruction? Like usually when we use SART we will add one regularization term.

Thanks, it worked!


QuestionnaireResponse.age_at_response duplicates Visit.age_at_visit_start linked through QuestionnaireResponse.associated_visit

Does QuestionnaireResponse.age_at_response duplicate Visit.age_at_visit_start because these are linked through QuestionnaireResponse.associated_visit?

Currently QuestionnaireResponse.associated_visit is an optional attribute to handle the potential use case of questionnaires administered outside of an obviously associated visit.  Also, it is conceivable that a visit spans numerous days (e.g. an inpatient stay) and that a questionnaire may be administered on one day of a multiple-day visit and one wants to be able to track on which particular day a response is given.  In either of these use cases, having the ability to provide an QuestionnaireResponse.age_at_response would be useful.


Loosen apache-airflow dev dependency

Codecov Report

Merging #24 (3e8ebea) into main (75aaa0d) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #24   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          112       112
=========================================
  Hits           112       112

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 75aaa0d...3e8ebea. Read the comment docs.


Extension host terminated unexpectedly

VSCode Version:  1.18.1
OS Version: Windows 10 Home 1703

Steps to Reproduce:

Install ms-python extension from Microsoft
Wait for an amount of time after opening Python file

Reproduces without extensions: No
I don't think it's the python extension causing the issue because I haven't been able to use VS code for months because of the "Extension host terminated unexpectedly" with or without the python extension. This is a clean install of VSCode and the only extension I have is the python one from Microsoft.
I'm getting this error on the console.
`
/C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:9 Extension host terminated unexpectedly.
e.doShow @ /C:/Program Files/Microsoft VS Code/resources/app/out/vs/workbench/workbench.main.js:9'

Reproducalbe steps would be great.
Also can you reproduce without any extension.
Also you could use vsocde insiders, f1 > runtime extensions to detect what extension might be causing this

I am getting the same problem.

VS Code:
Version 1.19.0-insider
Commit 8a12b294de610aab61c32c02c7c148d981cf07e4
Date 2017-12-04T05:14:19.063Z
Shell 1.7.9
Renderer 58.0.3029.110
Node 7.9.0
Architecture x64
OS:

Steps to Reproduce:

Install ms-python extension from Microsoft
Create new empty py file.
Wait for an amount of time after opening Python file. Error appears.

Reproducible with all extensions disabled for the workspace
Error in developer tools console:

Please file these issues to the extension here https://github.com/DonJayamanne/pythonVSCode

@isidorn
It's not just that extension. I've tried it with many other VS Code extensions and I have been getting that error for months. Except you're claiming that many major VS Code extensions are faulty.

@oluwatimilehin please file issues for all extensions that are causing this.
As I already mentioned you can get vscode insiders and try the F1 > runtime extensions to more easily detect which extension is to blame. Thanks


It is not possible to retrieve large (e.g. 100k) events from the provider service

What happened:
It is not possible to retrieve large (e.g. 100k) events from the provider service. At a certain point in time i get an "unexpected status code: 502 Bad Gateway" errror.
What you expected to happen:
Get all the results in timely manner.
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know:
The problem is with the offset feature (https://www.postgresql.org/docs/current/queries-limit.html) that we use for chunked retrieval of events. This gets very inefficient with large result sets.
Environment:

provided in https://github.com/gardener/falco-event-provider/pull/37


Telerik Removal

Describe the bug
DNN v10 is going to fully remove Telerik from it's installations.  The DNN Telerik Identifier Module is reporting the Events module is using the Telerik libraries.
To Reproduce
Steps to reproduce the behavior:

Download and install the module above.
Run the module and it reports the Events module as using Telerik library.

Additional context
DNN v9.8.0 makes Telerik removal a manual option currently.  v10 is going to be automatic and when this occurs, the Events module will be broken.

Related to #85 I am not sure if it's the only usage though.

This subject is already covered by #85 and a known required enhancement for long.
Any donations of code is welcome


Types with IEnumerable properties

I have a scenario in which I want to generate a excel sheet for a list of object of a certain type, this type itself contains properties that are of type IEnumerable , the create method works fine for basic types , but outputs the number of elements in the enumerable properties .

Yes, that is the current intended behavior. Enumerable properties will receive a count in the output. What would you expect the spreadsheet cell to contain for an enumerble property?


How to disabled the Link and when returning data reenable the Link?

I want to do this:

<Link to={{ pathname }}>some things</Link>
when I click:

event.preventDefault();
get some data
reenable the Link.

How to achieve it?

For what you're trying to do, you will need to write an asynchronous link component. You can basically copy the source for the <Link>, but edit handleClick to not perform the navigation until your async code has finished (either using a Promise or a callback).
handleClick = e => {
  event.preventDefault();
  this.props.onClick()
    .then(resp => {
      // do something with the loaded data
      // then call history.push
    });
}

If you have any more usage questions, please use either StackOverflow or Reactiflux because those are better setup to handle these types of questions. The issues section here is intended for bug reports.

@pshrmn thanks


What is the difference between the tcpreassembly and reassembly libraries?

The tcp packet reassembly library is divided into two parts as in the title. Why?

"reassembly" is basically "tcpassembly" v2.


Pixeebot/drip 2024 02 09 pixee python/secure random

Description

Updated the Snorkell DocGen Client configuration in the GitHub Actions workflow file.
Modified the _model_manager.py file to import the secrets module instead of random module and use secrets.SystemRandom() instead of random.Random() for generating random integers.
Updated the conftest.py file to import the secrets module and use secrets.SystemRandom().choices() instead of random.choices() for generating random choices.

Do you want to include any other information in the pull request description?

Hello @2lambda123! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file vizro-core/tests/unit/vizro/models/_controls/conftest.py:

Line 22:80: E501 line too long (94 > 79 characters)


Add disjointWith parsing

OWL file now has disjointWith keyword which requires parsing

I don't know if this helps, but with PaNET commit dfce0a0d9d,  we are now generated both the old-style output and the output having run it through a reasoner.
You should find this "reasoned output" as an artefact from the CI/CD pipelines: they are kept for some 7 days.
The reasoned output isn't part of the v1.1.0 release asserts, but we should include it as part of the next release's set of assets.

Perhaps the easiest way to get the reasoner output is to use the CI/CD-generated artefact.
The general procedure is:

go to PaNET main page
Click on the green tick near the top of the page, showing the CI/CD result of the most recent commit.
Against the "panet-build / build (push)" row, select "Details" link.
Select " Summary" link (from left-hand side navigation bar).
Under "Artifacts", select the "PaNET" artefact.  This will trigger the download of the PaNET.zip file.
Within PaNET.zip file, the file PaNET_reasoned.owl contains the reasoned output.

For the most recent commit, here is the PaNET.zip file.  However, this link will likely break, soon.

mmm the reasoned looks very similar to the plain .owl, what's the main advantage to use that? So, apart from downloading the owl and doing the diffs by hand, you don't plan to keep the commited versions of the owl file, apart from attaching to releases?

so the reasoner also has the grandchildren computed and referenced as subclassOf?

so the reasoned also has the grandchildren computed and annotated with subclassOf grandparent?

Exactly.
This is the RDFS entailment: subClassOf is transitive and the reasoner provides the transitive closure of these relationships.
The next PaNET version should provide both a "concise" and a "reasoned" version of PaNET.  For some software (like ontologies-api) there's currently no big difference.  This is because we're largely limited to RDFS entailment and (IIRC) ontologies-api implements support for RDFS entailment.
However, in the future, we might use more OWL features (moving away from only using RDFS relationships) to express more complex relationships in a manageable way.  When doing this, the distinction between the "concise" and "reasoned" versions will likely matter to ontologies-api.

ah thanks! So that might be quite useful indeed, I have been thinking that pan-api should rely on existing reasoners rather than parsing the owl itself. Would it make sense to make the ...reasoned.owl file part of the release assets?

pan-api should rely on existing reasoners

Yes, I think this would be a good idea.  But, it's easy for me to say this, when I don't have to do the work!

Would it make sense to make the ...reasoned.owl file part of the release assets?

Yes, I would say so.  The the primary reason for updating the build container (to run the result through a reasoner and save the result) was to include the reasoned output as a release asset.
I'd say that, assuming it isn't urgent, we can do this as part of the next PaNET release.


Dialog Directing Nodes

Graphic way of making statements.
Keep code way for users that want it.
Add classes that allow to use:

predefined menu scene, add menu_id to menu() keywords
predefined say box, add say_id to say() keywords
predefined ask box, , add ask_id to ask() keywords

changed today.

It seems there is no reason for that in with our new thread dialogue approach.


Can deepexpainer  calculate the shap value of lstm that is used for regression?

Thank you for your uesful tool.In the readme,there is a example that use deep DeepExplainer to calculate the shap value of LSTM.But that LSTM is used for classification.If I train an LSTM for regression ,can I use DeepExplainer to calculate its shap value.If yes,is it the same with classification?

Hello!
Did you find an answer to that question yet, i.e., have you tried it out? I also plan on doing that and I am looking for possible explainers that can do this.
Best


Districts not Syncing

If you create any districts and try to rename, it looks like not names are not syncing properly between host and client and the name will disappear.

After testing again, it still works for me. Could you please provide the exact steps that led to this problem? Also: Did you have any other mods or DLCs installed?

Cannot reproduce


Fix building with vanilla gjson

This template has been broken since we merged https://github.com/kubewarden/go-policy-template/pull/22
Building the policy via make was broken because of the following reasons:
That's because:

The vendored files were not updated
The containerized build relied on tinygo 0.18.0, which didn't support some of the features requested by the vanilla gjson

This PR address both issues by:

Update the vendored directory and doing a cleanup of it via go mod tidy
Upgrade to latest release of tinygo: 0.23.0

Updating to tinygo 0.23 causes also a compilation error inside of the container because the git executable is not found. That's because upstream Go, since release 1.18, embeds the git repository details inside of the final binaries, as reported inside of the release notes:

The go command now embeds version control information in binaries. It includes the currently checked-out revision, commit time, and a flag indicating whether edited or untracked files are present. Version control information is embedded if the go command is invoked in a directory within a Git, Mercurial, Fossil, or Bazaar repository, and the main package and its containing main module are in the same repository. This information may be omitted using the flag -buildvcs=false.

We don't want to add the git binary to the official tinygo container images, hence we're turning off this feature. It's important to note that tinygo build command doesn't support the buildvcs flag. However setting the flag via the GOFLAGS environment variable works.
As a final note, in order to prevent this kind of breakages from happening again, this PR enables the GitHub action that runs the tests and e2e tests against the reference policy contained inside of this repository

CC @floriankoch


Check key in correct locale

Currently it checks it in currentLocale even if locale parameter is explicitly given.

Will check
On Jul 15, 2017 12:42, "Edho Arief"<EMAIL_ADDRESS>wrote:

@nanaya https://github.com/nanaya requested your review on:
rmariuzzo/Lang.js#49 https://github.com/rmariuzzo/Lang.js/pull/49 Check
key in correct locale as a code owner.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rmariuzzo/Lang.js/pull/49#event-1165337900, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAa2HzMPhmOYHoXWcVVvCmDdVVk22KIHks5sOOwNgaJpZM4OZEBT
.

Thank you @nanaya for helping maintaining this project. :-)


robots.txt

We should have a robots.txt that blocks old versions of playwright from being indexed.

Seems fixed in #210


[UriError]: Scheme is missing

Hi,
I just installed this extension and got the error message as below.
[Error - 10:12:51 PM] Server initialization failed.
Message: Request initialize failed with message: [UriError]: Scheme is missing: {scheme: "", authority: "", path: "null", query: "", fragment: ""}
Code: -32603
How could I get more information to inspect why it can't be initialized?
VScode information:

Version: 1.39.2
Commit: 6ab598523be7a800d7f3eb4d92d7ab9a66069390
Date: 2019-10-15T15:33:00.827Z
Electron: 4.2.10
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Darwin x64 19.0.0

extension version: 1.1.0

Can you set ycmd.debug to see if there are more logs?
Also what's the version of your vscode and extension?

Hi @richard1122 ,
I got the same message in "Output" after setting ycmd.debug.
The version is added in my post.

@hsnusonic Hi, I can repro it sometimes.
I'll try to fix it these days.
Thanks.


Add required attribute to examples

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Create template for writing page template guidance

Create template for the team to use to write documentation for GC Design System's page templates.
To do

[ ]  Document page template page elements and instructions
[ ]  Explore format
[ ]  Produce prototype

Outcomes

Make it easier for team members to contribute design system documentation

Deliverables

Prototype template for page templates

To do

[ ]  Document page template page elements and instructions
[ ]  Explore format
[ ]  Produce prototype

Outcomes

Make it easier for team members to contribute design system documentation

Deliverables

Prototype template for page templates

Started template draft based on the basic page template structure so far.

Template draft complete.
Needs review/team feedback, especially @MatildePerrusclet and @realdylanzheng


NEP17.cs: "ManifestExtra" compile Error

requires: using Neo.SmartContract.Framework.Attributes;
this problem is present in different code sections in the documentation, not just this file.

w.r.t. compilation with rev 3.1.0


Create cloudstream.io

Milehigh.me

[x] I understand that this repository is auto-generated and my pull request may not be merged

Changes being requested
Additional context & links

https://platform.openai.com/docs/assistants/overview/agents


addCollisionShape documentation in .cpp is incorrect

From RigidBody.cpp for RigidBody::addCollisionShape:
/// When you add a collision shape to the body, an internal copy of this
/// collision shape will be created internally. Therefore, you can delete it
/// right after calling this method or use it later to add it to another body.

This is contrary to the HTML documentation, which states that only a pointer is copied in:
First note that when you add a collision shape to a body, the shape will not be copied internally.  The shape must exist during the whole lifetime of the body.

which is the actual behaviour.

That's right. I will fix this in the next release of the library.
Thanks a lot !

The version v0.8.0 of ReactPhysics3D has been released. In this version, this issue is now fixed.
Thanks a lot for reporting this issue.