system-test flake for 'ipro resoruces tagged with s-maxage, CC: s-maxage=50, max-age=200

TEST: ipro resources tagged with s-maxage, CC: s-maxage=50, max-age=200
Fetching http://localhost:8080/mod_pagespeed_test/ipro/cc200sma50/example.css until it's optimized...
(response body will be in /tmp/mod_pagespeed_test.jmarantz/apache/fetched_directory/wget/check_ipro_s_maxage.16452)

.Timed out: never got to being optimized
FAILed input: HTTP/1.1 200 OK
Date: Fri, 24 Jun 2016 14:09:40 GMT
Server: Apache/2.2.29 (Unix) mod_ssl/2.2.29 OpenSSL/1.0.1j DAV/2 mod_fcgid/2.3.9
Content-Length: 41
X-TestHeader: hello
Last-Modified: Sun, 10 Apr 2016 17:06:43 GMT
ETag: "1de6652-29-5302473d3b77a"
Accept-Ranges: bytes
X-Extra-Header: 1
Vary: Accept-Encoding
Cache-Control: s-maxage=10, max-age=200
X-Content-Type-Options: nosniff
Content-Type: text/css
/home/jmarantz/dev/chango/google3/third_party/pagespeed/automatic/system_test_helpers.sh:547: fail
/home/jmarantz/dev/chango/google3/third_party/pagespeed/automatic/system_tests/smaxage.sh:81: check_ipro_s_maxage
/home/jmarantz/dev/chango/google3/third_party/pagespeed/automatic/system_tests/smaxage.sh:136: source
/home/jmarantz/dev/chango/google3/third_party/pagespeed/automatic/system_test_helpers.sh:240: run_test
/home/jmarantz/dev/chango/google3/third_party/pagespeed/automatic/system_test.sh:40: source
/home/jmarantz/dev/chango/google3/third_party/pagespeed/system/system_test.sh:14: source
/home/jmarantz/dev/chango/google3/third_party/pagespeed/apache/system_test.sh:95: main
     failure at line 81
in 'ipro resources tagged with s-maxage, CC: s-maxage=50, max-age=200'
FAIL.
make[1]: *** [apache_debug_smoke_test] Error 1
make: *** [apache_vm_system_tests] Error 2
exit_status=2

What's in /tmp/mod_pagespeed_test.jmarantz/apache/fetched_directory/wget/check_ipro_s_maxage.16452 ?

It's very surprising that it only printed one "." before "Timed out".  It has a 20s timeout, and each time through the loop it prints a ".".  If I'm reading this right, then, it did two loads, and both received unoptimized data.  But how would that take 20s+?

sorry, the log file got erased when I re-ran the test.
On Fri, Jun 24, 2016 at 11:00 AM, Jeff Kaufman<EMAIL_ADDRESS>wrote:

It's very surprising that it only printed one "." before "Timed out". It
has a 20s timeout, and each time through the loop it prints a ".". If I'm
reading this right, then, it did two loads, and both received unoptimized
data. But how would that take 20s+?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/pagespeed/mod_pagespeed/issues/1326#issuecomment-228416624,
or mute the thread
https://github.com/notifications/unsubscribe/AB2kPT-rc_3qae6rdrIkaaexqIN05fdRks5qPBtBgaJpZM4I97Vn
.


Feature/441

Issue #, if available:
#441
Description of changes:
This adds support for custom API key expiration dates as well as supports auto-refreshing API keys on push. This also changes CLI behaviour in that now it now merges instead of overwriting the parameters.json file. This also adds a new parameter named APIKeyExpirationEpoch that can be used to tweak API keys as follows:
Resets the API Key to expire 1 week after the next amplify push
{
  "APIKeyExpirationEpoch": "0"
}

Do not create an API key
{
  "APIKeyExpirationEpoch": "-1"
}

Set a custom API key
{
  "APIKeyExpirationEpoch": "1544745428"
}

would make the key expire Thursday, December 13, 2018 11:57:08 PM GMT.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This was rebased incorrectly.

Is the issue #441 fixed?

Is there an example of this ?


Can we strip the symbols of hip kernel?

Hi, guys...
I am trying to add HIP support to our existing Cmake project. We have to strip the symbols in the generated .so/executable. But after stripping the symbols, the executable throws an exception.
Device name Device 6860
terminate called after throwing an instance of 'std::exception'
  what():  std::exception
[1]    22966 abort (core dumped)  ./hip_cmake_test

System information

Have I written custom code: Yes
main.cpp

#include <iostream>

#define __HIP_PLATFORM_HCC__

// hip header file
#include "hip/hip_runtime.h"
#include "transpose.cuh"

#define WIDTH 1024

#define NUM (WIDTH * WIDTH)

// CPU implementation of matrix transpose
void matrixTransposeCPUReference(float* output, float* input, const unsigned int width) {
    for (unsigned int j = 0; j < width; j++) {
        for (unsigned int i = 0; i < width; i++) {
            output[i * width + j] = input[j * width + i];
        }
    }
}

int main() {
    float* Matrix;
    float* TransposeMatrix;
    float* cpuTransposeMatrix;

    float* gpuMatrix;
    float* gpuTransposeMatrix;

    hipDeviceProp_t devProp;
    hipGetDeviceProperties(&devProp, 0);

    std::cout << "Device name " << devProp.name << std::endl;

    int i;
    int errors;

    Matrix = (float*)malloc(NUM * sizeof(float));
    TransposeMatrix = (float*)malloc(NUM * sizeof(float));
    cpuTransposeMatrix = (float*)malloc(NUM * sizeof(float));

    // initialize the input data
    for (i = 0; i < NUM; i++) {
        Matrix[i] = (float)i * 10.0f;
    }

    // allocate the memory on the device side
    hipMalloc((void**)&gpuMatrix, NUM * sizeof(float));
    hipMalloc((void**)&gpuTransposeMatrix, NUM * sizeof(float));

    // Memory transfer from host to device
    hipMemcpy(gpuMatrix, Matrix, NUM * sizeof(float), hipMemcpyHostToDevice);

    // Lauching kernel from host
    exec(gpuTransposeMatrix, gpuMatrix, WIDTH);

    // Memory transfer from device to host
    hipMemcpy(TransposeMatrix, gpuTransposeMatrix, NUM * sizeof(float),    hipMemcpyDeviceToHost);
    // CPU MatrixTranspose computation
    matrixTransposeCPUReference(cpuTransposeMatrix, Matrix, WIDTH);

    // verify the results
    errors = 0;
    double eps = 1.0E-6;
    for (i = 0; i < NUM; i++) {
        if (std::abs(TransposeMatrix[i] - cpuTransposeMatrix[i]) > eps) {
            errors++;
        }
    }
    if (errors != 0) {
        printf("FAILED: %d errors\n", errors);
    } else {
        printf("PASSED!\n");
    }

    // free the resources on device side
    hipFree(gpuMatrix);
    hipFree(gpuTransposeMatrix);

    // free the resources on host side
    free(Matrix);
    free(TransposeMatrix);
    free(cpuTransposeMatrix);

    return errors;
}

MatrixTranspose.cpp
#include <iostream>

#include "hip/hip_runtime_api.h"
#include "hip/hip_runtime.h"
#define WIDTH 1024

#define NUM (WIDTH * WIDTH)

#define THREADS_PER_BLOCK_X 4
#define THREADS_PER_BLOCK_Y 4
#define THREADS_PER_BLOCK_Z 1

// Device (Kernel) function, it must be void
__global__ void
__attribute__((visibility("default")))
matrixTranspose(float* out, float* in, const int width) {
    int x = hipBlockDim_x * hipBlockIdx_x + hipThreadIdx_x;
    int y = hipBlockDim_y * hipBlockIdx_y + hipThreadIdx_y;

    out[y * width + x] = in[x * width + y];

}

void exec(float* gpuTransposeMatrix, float* gpuMatrix, const int width) {
    // Lauching kernel from host
    hipLaunchKernelGGL(
        matrixTranspose,
        dim3(WIDTH / THREADS_PER_BLOCK_X, WIDTH / THREADS_PER_BLOCK_Y),
        dim3(THREADS_PER_BLOCK_X, THREADS_PER_BLOCK_Y), 0, 0,
        gpuTransposeMatrix, gpuMatrix, WIDTH);
}

transpose.cuh
#define THREADS_PER_BLOCK_X 4
#define THREADS_PER_BLOCK_Y 4
#define THREADS_PER_BLOCK_Z 1

void exec(float* out, float* in, const int width);

CMakeList.txt
cmake_minimum_required(VERSION 2.8.3)

if(NOT DEFINED HIP_PATH)
    if(NOT DEFINED ENV{HIP_PATH})
        set(HIP_PATH "/opt/rocm/hip" CACHE PATH "Path to which HIP has been installed")
    else()
        set(HIP_PATH $ENV{HIP_PATH} CACHE PATH "Path to which HIP has been installed")
    endif()
endif()
set(CMAKE_MODULE_PATH "${HIP_PATH}/cmake" ${CMAKE_MODULE_PATH})

project(12_cmake)

find_package(HIP QUIET)
if(HIP_FOUND)
    message(STATUS "Found HIP: " ${HIP_VERSION})
else()
    message(FATAL_ERROR "Could not find HIP. Ensure that HIP is either installed in /opt/rocm/hip or the variable HIP_PATH is set to point to the right location.")
endif()

set(MY_SOURCE_FILES MatrixTranspose.cpp)
set(MY_TARGET_NAME MatrixTranspose)
set(MY_HIPCC_OPTIONS )
set(MY_HCC_OPTIONS )
set(MY_NVCC_OPTIONS )

set (CMAKE_LINKER "/opt/rocm/bin/hipcc")
set (CMAKE_CXX_LINK_EXECUTABLE "<CMAKE_LINKER> <FLAGS> <CMAKE_CXX_LINK_FLAGS> <LINK_FLAGS> <OBJECTS> -o <TARGET> <LINK_LIBRARIES>")

set_source_files_properties(${MY_SOURCE_FILES} PROPERTIES HIP_SOURCE_PROPERTY_FORMAT 1)
hip_add_library(${MY_TARGET_NAME} ${MY_SOURCE_FILES} HIPCC_OPTIONS ${MY_HIPCC_OPTIONS} HCC_OPTIONS ${MY_HCC_OPTIONS} NVCC_OPTIONS ${MY_NVCC_OPTIONS})
set (SOURCES main.cpp)
#add_custom_target(combined COMMAND ar -x $<TARGET_FILE:MatrixTranspose>
#                WORKING_DIRECTORY ${CMAKE_BINARY_DIR}
#                DEPENDS MatrixTranspose
#                )
#add_library (hip_cmake_lib STATIC main.cpp)
#add_dependencies(hip_cmake_lib MatrixTranspose)
add_executable (hip_cmake_test main.cpp)
target_link_libraries (hip_cmake_test MatrixTranspose hip_hcc)

ROCm version

Package: rocm-libs
Version: 2.3.14
Priority: optional
Section: devel
Maintainer: Advanced Micro Devices Inc.
Installed-Size: 13.3 kB
Depends: rocfft, rocrand, hipblas, rocblas
Homepage: https://github.com/RadeonOpenCompute/ROCm
Download-Size: 766 B
APT-Sources: http://repo.radeon.com/rocm/apt/debian xenial/main amd64 Packages
Description: Radeon Open Compute (ROCm) Runtime software stack

hip version

HIP version  : 1.5.19123

== hipconfig
HIP_PATH     : /opt/rocm
HIP_PLATFORM : hcc
CPP_CONFIG   :  -D__HIP_PLATFORM_HCC__=   -I/opt/rocm/include -I/opt/rocm/hcc/include

== hcc
HSA_PATH     : /opt/rocm/hsa
HCC_HOME     : /opt/rocm/hcc
HCC clang version 9.0.0 (/data/jenkins_workspace/compute-rocm-rel-2.3/external/hcc-tot/clang 785f31db116e742ac53d052e207979869a857d1a) (/data/jenkins_workspace/compute-rocm-rel-2.3/external/hcc-tot/compiler 87f982f8ce2b85ce824f91bf8c2c90f6843a50a3) (based on HCC 1.3.19115-9b3a740-785f31d-87f982f )
Target: x86_64-unknown-linux-gnu
Thread model: posix
InstalledDir: /opt/rocm/hcc/bin
LLVM (http://llvm.org/):
  LLVM version 9.0.0svn
  Optimized build.
  Default target: x86_64-unknown-linux-gnu
  Host CPU: haswell

  Registered Targets:
    amdgcn - AMD GCN GPUs
    r600   - AMD GPUs HD2XXX-HD6XXX
    x86    - 32-bit X86: Pentium-Pro and above
    x86-64 - 64-bit X86: EM64T and AMD64
HCC-cxxflags :  -hc -std=c++amp -I/opt/rocm/hcc/include -I/opt/rocm/includeHCC-ldflags  :  -hc -std=c++amp -L/opt/rocm/hcc/lib -Wl,--rpath=/opt/rocm/hcc/lib -ldl -lm -lpthread -lhc_am -Wl,--whole-archive -lmcwamp -Wl,--no-whole-archive
...
== Linux Kernel
Hostname     : #####
Linux qiang 4.13.16-041316-generic #201711240901 SMP Fri Nov 24 09:02:42 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 16.04.5 LTS
Release:        16.04
Codename:       xenial

Reproduce

Firstly, compile the source code

cmake .
make

Then, strip the symbols of the executable

strip hip_cmake_test

And run the executable

./hip_cmake_test

Describe the current behavior

the executable throws an exception described above

Describe the expected behavior

strip the DSO/executable, and executable runs correctly.

Additional information

The exception is thrown at `hip/hcc_detail/functional_grid_launch.cpp:108'

    if (it == function_names().cend()) {
        hip_throw(std::runtime_error{"Undefined __global__ function."});
    }

Please assiste how to resolve this problem
Thanks and Regards.

@xiaocenxiaocen Apologies for the lack of response.  Do you still need assistance with this ticket? Thanks!


Use 2 separate pools to fix a problem in ParquetReaderBenchmark.cpp

Problem description:
Tried to run the ParquetReaderBenchmark(velox_dwio_parquet_reader_benchmark) and got below error:
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".

Breakpoint 1, facebook::velox::detail::veloxCheckFail<facebook::velox::VeloxRuntimeError, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&> (args=..., s="Memory pool addAggregateChild operation is only allowed on aggregation memory pool: Memory Pool[default_leaf_0 LEAF root[__default_root__] parent[__default_root__] MALLOC no-usage-track thread-safe]<u"...) at /root/velox/./velox/common/base/Exceptions.h:69
69	    LOG(ERROR) << "Line: " << args.file << ":" << args.line
E0811 15:19:40.412384 481990 Exceptions.h:69] Line: /root/velox/velox/common/memory/MemoryPool.cpp:313, Function:addAggregateChild, Expression:  Memory pool addAggregateChild operation is only allowed on aggregation memory pool: Memory Pool[default_leaf_0 LEAF root[__default_root__] parent[__default_root__] MALLOC no-usage-track thread-safe]<unlimited max capacity unlimited capacity used 0B available 0B reservation [used 0B, reserved 0B, min 0B] counters [allocs 0, frees 0, reserves 0, releases 0, collisions 0])>, Source: RUNTIME, ErrorCode: INVALID_STATE
terminate called after throwing an instance of 'facebook::velox::VeloxRuntimeError'
  what():  Exception: VeloxRuntimeError
Error Source: RUNTIME
Error Code: INVALID_STATE
Reason: Memory pool addAggregateChild operation is only allowed on aggregation memory pool: Memory Pool[default_leaf_0 LEAF root[__default_root__] parent[__default_root__] MALLOC no-usage-track thread-safe]<unlimited max capacity unlimited capacity used 0B available 0B reservation [used 0B, reserved 0B, min 0B] counters [allocs 0, frees 0, reserves 0, releases 0, collisions 0])>
Retriable: False
Function: addAggregateChild
File: /root/velox/velox/common/memory/MemoryPool.cpp
Line: 313
...
Program
received signal SIGABRT, Aborted.
__pthread_kill_implementation (no_tid=0, signo=6, threadid=140737309463040) at ./nptl/pthread_kill.c:44

Stack:
facebook::velox::detail::veloxCheckFail<facebook::velox::VeloxRuntimeError, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&> (\root\velox\velox\common\base\Exceptions.h:69)
facebook::velox::ram2::MemoryPool::addAggregateChild (\root\velox\velox\common\memory\MemoryPool.cpp:313)
facebook::velox::parquet::writer::Writer (\root\velox\velox\dwio\parquet\writer\Writer.cpp:137)
std::make_unique<facebook::velox::parquet::Writer, std::unique_ptr<facebook::velox::dwio::common::WriteFileDataSink, std::default_delete<facebook::velox::dwio::common::WriteFileDataSink> >, facebook::velox::parquet::WriterOptions&> (\usr\include\c++\11\bits\unique_ptr.h:962)
ParquetReaderBenchmark::ParquetReaderBenchmark (\root\velox\velox\dwio\parquet\tests\reader\ParquetReaderBenchmark.cpp:56)
run (\root\velox\velox\dwio\parquet\tests\reader\ParquetReaderBenchmark.cpp:251)
run_BigInt_Filter_0_Nulls_0_next_5k_dict (\root\velox\velox\dwio\parquet\tests\reader\ParquetReaderBenchmark.cpp:369)
operator() (\root\velox\velox\dwio\parquet\tests\reader\ParquetReaderBenchmark.cpp:369)
operator() (\usr\local\include\folly\Benchmark.h:204)
std::__invoke_impl<folly::detail::TimeIterData, folly::addBenchmark<<lambda(unsigned int)> >(char const*, folly::StringPiece, <lambda(unsigned int)>&&)::<lambda(unsigned int)>&, unsigned int>(std::__invoke_other, struct {...} &) (\usr\include\c++\11\bits\invoke.h:61)
std::__invoke_r<folly::detail::TimeIterData, folly::addBenchmark<<lambda(unsigned int)> >(char const*, folly::StringPiece, <lambda(unsigned int)>&&)::<lambda(unsigned int)>&, unsigned int>(struct {...} &) (\usr\include\c++\11\bits\invoke.h:116)
std::_Function_handler<folly::detail::TimeIterData(unsigned int), folly::addBenchmark<<lambda(unsigned int)> >(char const*, folly::StringPiece, <lambda(unsigned int)>&&)::<lambda(unsigned int)> >::_M_invoke(const std::_Any_data &, unsigned int &&) (\usr\include\c++\11\bits\std_function.h:291)
std::function<folly::detail::TimeIterData (unsigned int)>::operator()(unsigned int) const (\usr\include\c++\11\bits\std_function.h:590)
folly::runBenchmarkGetNSPerIteration (\root\velox\folly\folly\Benchmark.cpp:169)
folly::(anonymous namespace)::runBenchmarksWithPrinter (\root\velox\folly\folly\Benchmark.cpp:682)
folly::runBenchmarks (\root\velox\folly\folly\Benchmark.cpp:756)
main (\root\velox\velox\dwio\parquet\tests\reader\ParquetReaderBenchmark.cpp:378)

Solution:
I see there are only 2 kinds of memory pool:
/// Defines the kinds of a memory pool.
  enum class Kind {
    /// The leaf memory pool is used for memory allocation. User can allocate
    /// memory from this kind of pool but can't create child pool from it.
    kLeaf = 0,
    /// The aggregation memory pool is used to manage the memory pool hierarchy
    /// and aggregate the memory usage from the leaf pools. The user can't
    /// directly allocate memory from this kind of pool but can create child
    /// pools from it.
    kAggregate = 1,
  };

so I changed the code in ParquetReaderBenchmark.cpp to use 2 separate pools and fixed that problem (follow the code in E2EFilterTest.cpp):
    rootPool_ = memory::defaultMemoryManager().addRootPool("ParquetReaderBenchmark");
    leafPool_ = rootPool_->addLeafChild("ParquetReaderBenchmark");

This PR is part of work to solve https://github.com/facebookincubator/velox/issues/6070

@yingsu00 Ok, done. I 'rebase commits' and 'push force' to this PR.

@Yuhta Will you be able to review this PR? Thanks!

This should already have been merged in a77b987c81c4d0f9b5f8b02d614a481c541c5b58, not sure why the PR is not closed automatically

Closing as it's already merged a77b987c8


Empty response fetching from static data source

I'm going through the tutorial, and while each piece is explained well, I can't get it to work end-to-end.
Here's my attempt:
package main

import (
	"bytes"
	"context"
	"fmt"

	"github.com/wundergraph/graphql-go-tools/v2/pkg/ast"
	"github.com/wundergraph/graphql-go-tools/v2/pkg/astparser"
	"github.com/wundergraph/graphql-go-tools/v2/pkg/asttransform"
	"github.com/wundergraph/graphql-go-tools/v2/pkg/engine/datasource/staticdatasource"
	"github.com/wundergraph/graphql-go-tools/v2/pkg/engine/plan"
	"github.com/wundergraph/graphql-go-tools/v2/pkg/engine/resolve"
	"github.com/wundergraph/graphql-go-tools/v2/pkg/operationreport"
)

func main() {

	staticDataSource, err := plan.NewDataSourceConfiguration(
		"StaticDataSource",
		&staticdatasource.Factory[staticdatasource.Configuration]{},
		&plan.DataSourceMetadata{
			RootNodes: []plan.TypeField{{
				TypeName:   "Query",
				FieldNames: []string{"hello"},
			}}},
		staticdatasource.Configuration{
			Data: `{"hello":"world"}`,
		},
	)
	if err != nil {
		panic(err)
	}

	config := plan.Configuration{
		DataSources: []plan.DataSource{staticDataSource},
		Fields: []plan.FieldConfiguration{{
			TypeName:  "Query",
			FieldName: "hello",
			Path:      []string{"hello"},
		}},
		IncludeInfo: true,
	}

	operationDocument := ast.NewSmallDocument()
	operationDocument.Input.ResetInputString("query MyQuery { hello }")

	schemaDocument := ast.NewSmallDocument()
	schemaDocument.Input.ResetInputString("type Query { hello: String }")

	report := &operationreport.Report{}
	operationName := "MyQuery"
	parser := astparser.NewParser()

	parser.Parse(operationDocument, report)
	if report.HasErrors() {
		panic(report.Error())
	}

	parser.Parse(schemaDocument, report)
	if report.HasErrors() {
		panic(report.Error())
	}

	asttransform.MergeDefinitionWithBaseSchema(schemaDocument)

	planner, err := plan.NewPlanner(config)
	if err != nil {
		panic(err)
	}

	executionPlan := planner.Plan(operationDocument, schemaDocument, operationName, report)
	if report.HasErrors() {
		panic(report.Error())
	}

	resolver := resolve.New(context.Background(),
		resolve.ResolverOptions{
			MaxConcurrency: 1024,
		})

	ctx := resolve.NewContext(context.Background())

	switch p := executionPlan.(type) {
	case *plan.SynchronousResponsePlan:
		out := &bytes.Buffer{}
		err := resolver.ResolveGraphQLResponse(ctx, p.Response, nil, out)
		if err != nil {
			panic(err)
		}
		fmt.Println(out.String())
	}
}

I don't get any panic, but the resulting output gives:
{"errors":[{"message":"Failed to fetch from Subgraph 'StaticDataSource' at path 'query', empty response."}],"data":null}

What am I missing?  Thanks.

I think this is missing in your execution plan:
https://github.com/wundergraph/graphql-go-tools/blob/95ebc94f0714ff8fba1f50d7357e94eca9d10753/v2/pkg/engine/datasource/graphql_datasource/graphql_datasource_federation_entity_interfaces_test.go#L34

Sorry, I don't understand.  I'm just using the built-in static data source, as shown in the tutorial.  PostProcessing appears to be an option on FetchConfiguration, which isn't exposed to me as far as I can tell.
Or are you suggesting it should be added here:
https://github.com/wundergraph/graphql-go-tools/blob/95ebc94f0714ff8fba1f50d7357e94eca9d10753/v2/pkg/engine/datasource/staticdatasource/static_datasource.go#L53-L58

FWIW, I tried creating my own data source based on the static data source, so I could set PostProcesing.
type MyFactory struct {
	staticdatasource.Factory[staticdatasource.Configuration]
}

type MyPlanner struct {
	config staticdatasource.Configuration
	staticdatasource.Planner[staticdatasource.Configuration]
}

func (f *MyFactory) Planner(logger abstractlogger.Logger) plan.DataSourcePlanner[staticdatasource.Configuration] {
	return &MyPlanner{}
}

func (p *MyPlanner) ConfigureFetch() resolve.FetchConfiguration {
	return resolve.FetchConfiguration{
		Input:      p.config.Data,
		DataSource: staticdatasource.Source{},
		PostProcessing: resolve.PostProcessingConfiguration{
			SelectResponseDataPath:   []string{"data"},
			SelectResponseErrorsPath: []string{"errors"},
		},
	}
}

It had no affect.  The result is the same.

Still haven't figured out how to fix it correctly, but through debugging I found that by the time it gets to loading data from the data source here:
https://github.com/wundergraph/graphql-go-tools/blob/master/v2/pkg/engine/datasource/staticdatasource/static_datasource.go#L68-L71
The input slice is empty.  If I manually set it to the original data (ie. input = []byte({"hello":"world"})) then the rest of it works and I get the correct output.
I think there's a bug somewhere, but I've not figured out whether its in my code or in the library.

I stumbled upon a test that uses the static data source, BenchmarkExecutionEngineV2.
It's currently failing, but I was able to get it to pass with two small changes:
https://github.com/wundergraph/graphql-go-tools/blob/95ebc94f0714ff8fba1f50d7357e94eca9d10753/v2/pkg/graphql/execution_engine_v2_test.go#L1626-L1628
change to:
staticdatasource.Configuration{
	Data: `{"hello": "world"}`,
},

and then:
https://github.com/wundergraph/graphql-go-tools/blob/95ebc94f0714ff8fba1f50d7357e94eca9d10753/v2/pkg/graphql/execution_engine_v2_test.go#L1640
Set DisableDefaultMapping to false (or remove line)
The test then passes.
From there, I was able to adapt that to an end-to-end version of what I was originally trying to do:
package main

import (
	"context"

	"github.com/jensneuse/abstractlogger"
	"github.com/wundergraph/graphql-go-tools/v2/pkg/engine/datasource/staticdatasource"
	"github.com/wundergraph/graphql-go-tools/v2/pkg/engine/plan"
	"github.com/wundergraph/graphql-go-tools/v2/pkg/graphql"
)

func main() {
	schema, err := graphql.NewSchemaFromString(`type Query { hello: String}`)
	if err != nil {
		panic(err)
	}

	dsCfg, err := plan.NewDataSourceConfiguration(
		"MyStaticDataSource",
		&staticdatasource.Factory[staticdatasource.Configuration]{},
		&plan.DataSourceMetadata{
			RootNodes: []plan.TypeField{
				{
					TypeName:   "Query",
					FieldNames: []string{"hello"},
				},
			},
		},
		staticdatasource.Configuration{
			Data: `{"hello": "world"}`,
		},
	)
	if err != nil {
		panic(err)
	}

	engineConf := graphql.NewEngineV2Configuration(schema)
	engineConf.SetDataSources([]plan.DataSource{
		dsCfg,
	})
	engineConf.SetFieldConfigurations([]plan.FieldConfiguration{
		{
			TypeName:  "Query",
			FieldName: "hello",
		},
	})

	ctx := context.Background()
	logger := abstractlogger.NoopLogger

	engine, err := graphql.NewExecutionEngineV2(ctx, logger, engineConf)
	if err != nil {
		panic(err)
	}

	req := graphql.Request{
		Query: "{hello}",
	}

	writer := graphql.NewEngineResultWriter()
	err = engine.Execute(ctx, &req, &writer)
	if err != nil {
		panic(err)
	}

	result := writer.String()
	println(result)
}

And it works!
{"data":{"hello":"world"}}

A few questions:

This approach seems a lot simpler than the one in the tutorial.  Which is preferred? Are there pros/cons to either approach?
What is it about this approach that works, while the other one doesn't?
I notice I can remove the call to engineConf.SetFieldConfigurations entirely, and everything still works.  Is that just because it's using default field configurations and the root nodes configuration of the data source metadata?  (I'm a bit confused why I need both.)
I presume these are just growing pains and related to still being in RC status.  Is there a roadmap/timeline to release? I'm not sure if I can consider v2 production-ready, or if I should stick to v1?

Thanks.

both approaches are fine, you usually use default field mapping with GraphQL data sources and disable it for everything else
not exactly sure tbh, I don't have the time to deeply investigate your config
it's because default config is off and no nesting
v2 is used in production heavily, but we don't manually use these low-level APIs. If you're interested to learn how we use the "engine" ourselves, check out Cosmo Router

Closing this because I've figured out how to use the execution/engine package instead.  It mostly works great.  If y'all find some time, you should update your readme tutorial to demonstrate using it.  😃


Ensure mockito tests without warnings for JDK21+

In JDK21+ instrumentation agents such as mockito and jmockit can only be used when they are properly attached with a -javaagent tag on the JVM command line for security reasons. This should be implemented for:

Automated maven test when the code is installed or deployed by including it in the pom.xml
Manual Run as... Maven test in Eclipse
Run as... JUnit test in Eclipse
Coverage as... JUnit test in Eclipse

The maven handling can probably be automated through the pom.xml file. The Eclipse commands might need JRE settings in Eclipse that can be explained in the OTS manual.
Note that this has to be tested with JDK-17 (current version), JDK-21 (giving a warning), and later JDK's (forbidding instrumentation without the command line adaptation).
The error/warning is:
Mockito is currently self-attaching to enable the inline-mock-maker. This will no longer work in
future releases of the JDK. Please add Mockito as an agent to your build what is described in
Mockito's documentation:
https://javadoc.io/doc/org.mockito/mockito-core/latest/org/mockito/Mockito.html#0.3
WARNING: A Java agent has been loaded dynamically (C:\Users\alexandv\.m2\repository\net\
         bytebuddy\byte-buddy-agent\1.15.4\byte-buddy-agent-1.15.4.jar)
WARNING: If a serviceability tool is in use, please run with -XX:+EnableDynamicAgentLoading
         to hide this warning
WARNING: If a serviceability tool is not in use, please run with -Djdk.instrument.traceUsage
         for more information
WARNING: Dynamic loading of agents will be disallowed by default in a future release

The Mockito manual explicitly addresses this issue: https://javadoc.io/doc/org.mockito/mockito-core/latest/org/mockito/Mockito.html (section 0.3). As a result, we can easily enable the agent instrumentation in Maven:
<plugin>
  <groupId>org.apache.maven.plugins</groupId>
  <artifactId>maven-surefire-plugin</artifactId>
  <version>${maven.surefire.version}</version>
  <configuration>
    <runOrder>alphabetical</runOrder>
    <!-- Sets the VM argument line used when unit tests are run. -->
    <argLine>${surefireArgLine} -javaagent:"${settings.localRepository}"
         /org/mockito/mockito-core/${mockito.version}/mockito-core-${mockito.version}.jar</argLine>
    <!-- Skips unit tests if the value of skip.unit.tests property is true -->
    <skipTests>${skip.unit.tests}</skipTests>
    <!-- Excludes integration tests when unit tests are run. -->
    <excludes>
      <exclude>**/IT*.java</exclude>
    </excludes>
    <!-- Make sure every unit test is run as a separate process to avoid static leaks between tests -->
    <forkCount>1</forkCount>
    <reuseForks>false</reuseForks>
  </configuration>
</plugin>

Notes:

The use of ${mockito.version} takes care of updating the mockito version automatically when the dependency changes.
The use of "${settings.localRepository}" takes care of individual computers and operating systems for the location of the .m2/repository folder.
The quotes take care of possible spaces in the repository path.
The line break in the argLine tag is not present in the file, and just there to view it properly here in the issue.

Tested the Run as... Maven test with JDK17 and JDK21. Both work without warnings about the agent. The warning that does occur a few times is:
OpenJDK 64-Bit Server VM warning: Sharing is only supported for boot loader
classes because bootstrap classpath has been appended.

All articles on the web about this warning state that these can be safely ignored.

The Run as... JUnit test and Coverage as... JUnit test in Eclipse are a bit more tricky. None of the settings how to run a unit test can be saved in such a way that all developers will be updated with these settings using git. Automatic updating of the mockito version based on the pom file is also impossible. Therefore, the only way to handle this is a manual setting that has to be documented. This manual setting can be applied on two levels:

Indicate for the Eclipse-wide JDK21+ compiler in the Eclipse preferences that it has to use a -javaagent VM argument when executing the JDK. Possible side effects: this setting is also used when executing other code such as a demo. Note that this setting does not transfer to other projects. This is by far the easiest setting to apply.
Creating a specific Run Configuration per project that uses mockito with the -javaagent setting. This is quite a lot more work, but it has the advantage that demo's and other types of runs will not attach the agent. Note that separate configurations have to be built per project for Run as... Unit test, Coverage as... Unit test and Debug as... Unit test. Also, when running a unit test for only one class, the -javaagent option will not be attached.  Therefore, only option 1 will be detailed below.

The -javaagent  VM setting should only be used for a JRE/JDK version 21 or higher. The setting that can be used is:
-javaagent:"${env_var:HOME}"/.m2/repository/org/mockito/mockito-core/5.14.2/mockito-core-5.14.2.jar

The "${env_var:HOME}" takes care of linking to the home directory of the user. The quotes handle possible spaces in the path.
The setting can be applied in Eclipse through: Window - Preferences - Java - Installed JREs. In the Installed JREs, choose every compiler that is Java 21 or higher, and press 'Edit':

Here, add the above VM argument, and press Finish and Apply and Close. Now, all runs with JDK-21 in this project will attach the mockito agent when executing unit tests and coverage tests on any of the projects or classes in the entire project.

Unit tests and test coverage work as intended with the setting. The warning has disappeared.

Documentation has been created:


Better-scheduled parallel tests

@rbuckton mentioned that if he were to rewrite the parallel test runner, he would make it use a work stealing system, rather than partitioning work in advance; to help improve thread utilization near the end of a test run. This is effectively that.
This pulls the import parts of our parallel test script into the test runner itself, and then reworks the parallel runner to use a central queue for distributing work to the worker threads, while also sorting the work by file size (approximating how long the test will take to execute - the largest tests execute first). The improved utilization and load balancing takes the end-to-end time of gulp runtests-parallel --tests=rwc (now that that is functional) down from 13 minutes to 8 minutes on my workstation - I can only assume the effect is even more pronounced with more cores at the disposal of the test runner.
The primary downside to this system is that the tests aren't actually run under mocha, but rather a shim which polyfills the behaviors we use. This is so it immediately executes the tests on discovery (since we run one file at a time on-demand), but the downside is that timeouts aren't currently supported in my shim (so a test can just hang if it is malfunctioning).

Couple of notes without looking at the code yet:

The travis failure is probably due to stale cache of some kind; I had to wipe my node_modules folder to get a build.
runtests-parallel is much slower for our test suite. It goes from about a minute on my machine to over 2 minutes.

Now that you have my old 24-thread machine up and running again, it would be interesting to compare results from it as well.

Also, I don't care about how many tests have passed so far, just how many have failed.

With the message-batching change, the PR is as fast (or maybe a little faster) than the current runtests-parallel for the test suite.

@rbuckton done.

This actually makes tests slower on my desktop. From the prior commit (c522f379b20b2be70873ca6b61dd8be7be1c76bf), it took 2 minutes 24 seconds. With this one (d1c4754b37fb49e3f3d9f73dcc8fc6810a1e081e), it takes 2 minutes 48 seconds.

On my laptop, the time went up from 2min 41s to 3m 10s. My laptop has 8 cores like my desktop but only 8GB of memory while my desktop has 16GB.

@andy-ms How many workers does your machine use? It is possible that the batch ratio is a bit suboptimal for a given thread count (ie, pre-batching too many or too few tests, resulting in idle threads).

Hmmm.... I'll make the batch parameter tuneable. The worst case (pre-batching all tests) should approximately equal the old parallel runner in speed (since there will be many idle threads) - and right now it is hardcoded to batch 90% of the available filesize into the first messages. OFC, since filesize isn't a perfect approximator for test time, this isn't perfect... I have some ideas on how to improve it a bit more, though.


frp: 0.46.1 -> 0.47.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/fatedier/frp/releases.
meta.description for frp is: Fast reverse proxy
meta.homepage for frp is: https://github.com/fatedier/frp
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.47.0 with grep in /nix/store/gwl5vqgnnahzjy5rag5b1prxzbpsnfrg-frp-0.47.0
found 0.47.0 in filename of file in /nix/store/gwl5vqgnnahzjy5rag5b1prxzbpsnfrg-frp-0.47.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
frp

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/gwl5vqgnnahzjy5rag5b1prxzbpsnfrg-frp-0.47.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A frp https://github.com/r-ryantm/nixpkgs/archive/b5fa1434696c86576da1462db2d005ea2e93501d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gwl5vqgnnahzjy5rag5b1prxzbpsnfrg-frp-0.47.0
ls -la /nix/store/gwl5vqgnnahzjy5rag5b1prxzbpsnfrg-frp-0.47.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

frp

Maintainer pings
cc @Br1ght0ne for testing.

Result of nixpkgs-review pr 215942 run on aarch64-darwin 1

1 package built:

frp


Track resource consumption for query and fetch phases

Description
Part 1: This is first phase towards providing visibility into the most memory/compute heavy queries
This change measures memory and cpu-time used during query and fetch phases. I am leveraging the single threaded execution model to track resources consumed by the thread executing the query/fetch phase.
Signed-off-by: Ankit Malpani<EMAIL_ADDRESS>Issues Resolved
Search memory tracking
Check List

[x] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@malpani I believe there are some overlaps with https://github.com/opensearch-project/OpenSearch/pull/1555 (please correct me if I am wrong). It seems like you are focusing on search only, but by and large, the query and fetch phases are just variations of SearchShardTask, may be it is better to report this stats on task level instead? What do you think?

@reta - yes they are related, @sruti1312 and I work on the same team and are collaborating to roll resource tracking out. To keep diffs small, we have split this out into multiple atomic changes instead of raising a giant merged PR that will be hard to review -

(Consumer - Task framework) Extending task framework to track resource consumption by tasks - https://github.com/opensearch-project/OpenSearch/pull/1555
(Producer - query resource tracking) This PR - measuring resources for search related tasks
Merging - merged code will finally look like https://github.com/malpani/OpenSearch/tree/stat-task-framework
(Consumer - develop sinks) Logging every task will have a perf hit, we will be providing options eg. a default top N resource consumers based sink
5 (Producer - measure more frequently) - Instead of this PR just measuring at the end of the task, we will track resource usage on a per phase basis within a task eg. aggregation/reduce etc.
Consumer - system index sink) - this is optional and instead of having just log as a sink, this will write docs into a system index

Coming back to your question - can this be purely done from task side - the answer is no. eg. Bulk tasks work very differently from search, so it will be upto different task executors to define how to track their resource usage. Given indexing usage is very easy to determine as a function of the incoming payload as against search where most of the problems arise and a small payload can cause extensive usage), the focus is currently on query resource observability. Hope this helps clarify where we are headed with this.

Thanks @malpani, it makes sense. Just to highlight some gaps: it may not be possible to capture memory and CPU consumption accurately since additional executors may be involved along the way. Fe, we are exploring the experimental Apache Lucene support of the concurrent segments search (https://github.com/opensearch-project/OpenSearch/pull/1500) and it uses different executor behind the scene to do that. The resources consumed by those pooled threads won't be captured during query / fetch phases. Hope sounds reasonable, thank you.

@reta - thanks for the heads up, i skimmed through the PR and when concurrent segments search is enabled, the current form for tracking resource consumption of local thread will not work as multiple threads will be involved. My initial thought to have this work in that world will be a wrapped ResourceTrackingQuery whose createWeight could trigger the resource collector object instantiation and eventually they will need to be summed up, i implemented something similar for ultrawarm. However, that can be explored more when the experimental tag for concurrent segment search gets removed.

I'm happy to merge if @reta is A-OK with the change, LMK?

@dblock yes, I am A-OK, thanks for asking!

@zelinh can you please help with the whitesource failure in this PR?

Referring to #1593. We may ignore the WhiteSource check for OpenSearch repo for now. There is an issue on WhiteSource side and they are working to fix it.

@malpani  should I merge it as is or are you making changes suggested by @Bukhtawar?

start gradle check

@dblock i will have one more revision to incorporate feedback from @andrross and @Bukhtawar . Also, we now have 2 similar PRs for resource tracking - one is for https://github.com/opensearch-project/OpenSearch/issues/1042  and the current one is for query observability. There is similar logic in both in terms of relying on resource usage deltas via ThreadMXBean and while both can get merged independently on the short term, i do think we can consolidate things.

closing this out in favor of a merged approach with https://github.com/opensearch-project/OpenSearch/issues/1042


container.lookup is not a function<EMAIL_ADDRESS>

This addon is using the Application Initializers API, which doesn't have access to app.lookup anymore. It should probably be migrated to be an instance initializer to access app.lookup as it fails in new ember versions.
It's fixed as easily as renaming app/initializers to app/instance-initializers I believe.

+1 this is also happening to me after migrating to 2.2.0

@luismendozamx feel free to look how i fixed it here. https://github.com/ZackMattor/ember-cli-bugsnag
If you wanna test it...
    "ember-cli-bugsnag": "https://github.com/ZackMattor/ember-cli-bugsnag.git#v0.0.12",

Awesome! Thank you @ZackMattor

Also... it looks like https://github.com/twokul/ember-cli-bugsnag/pull/22 might fix it too.


Is it working now?

Great to see such a project! Is it working now?

@hsluoyz Almost. Jetbrains request a small change. I will resubmit and it should be publicly available!

Looking forward to it!

@hsluoyz this plugin has been on the market for some time now. It supports the latest two major versions: 201 and 202. Would it be possible to put a page on the Casbin website noting this plugin?

@will7200 great work! Will do it definitely. Can you provide me a web page link to that plugin?

@hsluoyz yeah!the jetbrains link to is: https://plugins.jetbrains.com/plugin/14809-casbin

@will7200 already added:

You can also send a PR to: https://github.com/casbin/casbin-website to add a detailed page for it. I want the users to know it more :)


HTTPS with HTTP11Connection

Hi All,
What configurations (especially tls versions) do we need to use secure option on HTTP11Connection.
I keep getting error (check the last paragraph)
Client:
ssl_context = hyper.tls.init_context()
ssl_context.load_cert_chain(certfile='/mnt/QUIClientServer0/cert.crt', keyfile='/mnt/QUIClientServer0/cert.key')
ssl_context.load_verify_locations(cafile='/mnt/QUIClientServer0/cert.pem')

connection = hyper.HTTP11Connection('<IP_ADDRESS>', ssl_context= ssl_context, secure= True, port=9000)
        http11_conn = connection.request('GET',parse_url.path)
        f_conn=connection.get_response()

Server:
Caddy server
https://<IP_ADDRESS>:9000, https://www.yo.org, http://<IP_ADDRESS>, https://<IP_ADDRESS>:443
tls cert.crt cert.key {

protocols tls1.0

}

ERROR:
 File "/mnt/QUIClientServer0/download_test_https1.py", line 28, in get_file
    http11_conn = connection.request('GET',parse_url.path)
  File "/usr/local/lib/python3.4/dist-packages/hyper/http11/connection.py", line 251, in request
    self.connect() File "/mnt/QUIClientServer0/download_test_https1.py", line 28, in get_file
    http11_conn = connection.request('GET',parse_url.path)
  File "/usr/local/lib/python3.4/dist-packages/hyper/http11/connection.py", line 251, in request
    self.connect()
  File "/usr/local/lib/python3.4/dist-packages/hyper/http11/connection.py", line 198, in connect
    sock, proto = wrap_socket(sock, self.host, self.ssl_context)
  File "/usr/local/lib/python3.4/dist-packages/hyper/tls.py", line 46, in wrap_socket
    ssl_sock = _ssl_context.wrap_socket(sock, server_hostname=server_hostname)
  File "/usr/lib/python3.4/ssl.py", line 365, in wrap_socket
    _context=self)
  File "/usr/lib/python3.4/ssl.py", line 601, in __init__
    self.do_handshake()
  File "/usr/lib/python3.4/ssl.py", line 828, in do_handshake
    self._sslobj.do_handshake()
ssl.SSLError: [SSL: WRONG_VERSION_NUMBER] wrong version number (_ssl.c:600)

  File "/usr/local/lib/python3.4/dist-packages/hyper/http11/connection.py", line 198, in connect
    sock, proto = wrap_socket(sock, self.host, self.ssl_context)
  File "/usr/local/lib/python3.4/dist-packages/hyper/tls.py", line 46, in wrap_socket
    ssl_sock = _ssl_context.wrap_socket(sock, server_hostname=server_hostname)
  File "/usr/lib/python3.4/ssl.py", line 365, in wrap_socket
    _context=self)
  File "/usr/lib/python3.4/ssl.py", line 601, in __init__
    self.do_handshake()
  File "/usr/lib/python3.4/ssl.py", line 828, in do_handshake
    self._sslobj.do_handshake()
ssl.SSLError: [SSL: WRONG_VERSION_NUMBER] wrong version number (_ssl.c:600)

Hi All,
I figured out that the error was that alpn on Caddy server is h2 by default, changed it to http/1.1 and it works perfectly.
Thanks


Deflake AndroidEntityHandleManagerTest

Related bugs:
http://b/156993444

Was the test already not flaky, so this is now just cleanup + remove Ignore?

Yeah, pretty much... its flakiness was likely fixed with all the other stuff that was fixed in the last couple of months. Passed 10000/10000 times, twice.


Audio "playing" flag is the wrong type.

Describe the bug
Audio "playing" flag is the wrong type.
To Reproduce
Steps to reproduce the behavior:
from reflex import rx

def play_audio():
    audio.playing = True

button = rx.button('Play', onClick=play_audio)
audio = rx.audio(
    url="https://www.learningcontainer.com/wp-content/uploads/2020/02/Kalimba.mp3",
    playing=False,
)

rx.render(button, audio)

Expected behavior
I expected to be able to switch the flag from False to True and instead got: "TypeError: Invalid var passed for prop playing, expected type <class 'str'>, got value False of type <class 'bool'>."
Screenshots

Specifics (please complete the following information):

Python Version: 3.11.5
Reflex Version: 0.2.8
OS: Windows 11
Browser (Optional):

Additional context
Add any other context about the problem here.

You have to use State if you want to control the displayed component.
class State(rx.State):
    playing: bool = False

    def play_audio(self):
        self.playing = True

@page(route="/")
def index():
    return rx.fragment(
        rx.audio(
            url="https://...",
            playing=State.playing
        ),
        rx.button("Play", on_click=State.play_audio)
    )

Thanks for the detailed reply! This addresses the bug I was getting, but I still think the docs should be clear that this is a bool and not a string, right?

Yeah the type needs to be fixed on that prop:
https://github.com/reflex-dev/reflex/blob/864a66b20d9c26bbb4fbf674171857e288ab455a/reflex/components/libs/react_player.py#L23-L24
Good first issue, if you wanted to contribute a PR.

My apologies, I missed this comment. Thanks for such a quick fix. Will contribute a PR next time if I find something ;)


Move graph refresh specs under providers/inventory

Move graph refresh specs under providers/inventory

@miq-bot assign @agrare
@miq-bot add_label refactoring

not needed in the end


[IOS][webP] webP show blank on IOS 13.6

Describe the bug
webP show blank on IOS 13.6.
To Reproduce
display webp image on IOS 13.6.
IOS >= 14 and Android shows well.
IOS 13.6 shows blank.
Expected behavior
webP shows well on IOS 13.6.
Dependency versions

React Native version: 0.66.4
React version: 17.0.2
React Native Fast Image version: 8.5.11
cocoapods 1.11.3
IOS 13.6

podfile:
require_relative '../node_modules/react-native/scripts/react_native_pods'
require_relative '../node_modules/@react-native-community/cli-platform-ios/native_modules'

platform :ios, '12.0'

ENVFILES = {
  'Debug' => '$(PODS_ROOT)/../../.env.dev',
  'Release' => '$(PODS_ROOT)/../../.env.prod',
  'Daily' => '$(PODS_ROOT)/../../.env.daily',
}

def react_native_post_install(installer)
	installer.pods_project.targets.each do |target|
		target.build_configurations.each do |config|
		  # Set 9.0 as min version for all
		  if Gem::Version.new('9.0') > Gem::Version.new(config.build_settings['IPHONEOS_DEPLOYMENT_TARGET'])
			config.build_settings['IPHONEOS_DEPLOYMENT_TARGET'] = '9.0'
		  end
      if target.name == 'react-native-config'
        config.build_settings['ENVFILE'] = ENVFILES[config.name]
      end
		end
	end
end

target 'app' do
  config = use_native_modules!

  pod 'GoogleMLKit/BarcodeScanning', '2.3.0'

  use_react_native!(
    :path => config[:reactNativePath],
    # to enable hermes on iOS, change `false` to `true` and then install pods
    :hermes_enabled => false
  )

  permissions_path = '../node_modules/react-native-permissions/ios'
  pod 'Permission-Camera', :path => "#{permissions_path}/Camera"
  pod 'Permission-FaceID', :path => "#{permissions_path}/FaceID"
  pod 'Permission-Notifications', :path => "#{permissions_path}/Notifications"

  pod 'RNFS', :path => '../node_modules/react-native-fs'

  target 'appTests' do
    inherit! :complete
    # Pods for testing
  end

  # Enables Flipper.
  #
  # Note that if you have use_frameworks! enabled, Flipper will not work and
  # you should disable the next line.
  use_flipper!({ 'Flipper-Folly' => '2.5.3', 'Flipper' => '0.75.1', 'Flipper-RSocket' => '1.3.1' })

  post_install do |installer|
    react_native_post_install(installer)
    __apply_Xcode_12_5_M1_post_install_workaround(installer)
  end
end

Solved, need to call
 SDImageWebPCoder *webPCoder = [SDImageWebPCoder sharedCoder];
 [[SDImageCodersManager sharedManager] addCoder:webPCoder];

in the AppDelegate.m

@LouisWT @DylanVann even though I call I still don't see webP images being loaded in an online simulator. ( tested on iOS 12.5.5, 12.5.7 and 13.6 )
Is there anything else needed ?


Unable to add new master to cluster with embedded etcd

Environmental Info:
K3s Version:
k3s -v
k3s version v1.20.4+k3s1 (838a906a)
go version go1.15.8
Node(s) CPU architecture, OS, and Version:
Linux andreasp61 5.4.0-74-generic #83-Ubuntu SMP Sat May 8 02:35:39 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Cluster Configuration:
5 masters, now trying to add 2 more masters
Describe the bug:
When trying to add a 6th node to the cluster i'm getting the following error:
{"level":"info","ts":"2021-07-08T12:43:50.702+0200","caller":"raft/node.go:325","msg":"raft.node: d7b511c51f0b845f elected leader be23a81d48f61f6b at term 57"}
{"level":"info","ts":"2021-07-08T12:43:50.724+0200","caller":"etcdserver/server.go:716","msg":"initialized peer connections; fast-forwarding election ticks","local-member-id":"d7b511c51f0b845f","forward-ticks":8,"forward-duration":"4s","election-ticks":10,"election-timeout":"5s","active-remote-members":5}
{"level":"warn","ts":"2021-07-08T12:44:00.677+0200","caller":"clientv3/retry_interceptor.go:62","msg":"retrying of unary invoker failed","target":"passthrough:///https://<IP_ADDRESS>:2379","attempt":0,"error":"rpc error: code = DeadlineExceeded desc = context deadline exceeded"}
INFO[2021-07-08T12:44:00.677875833+02:00] Failed to test data store connection: context deadline exceeded
{"level":"warn","ts":"2021-07-08T12:44:00.678+0200","caller":"grpclog/grpclog.go:60","msg":"grpc: addrConn.createTransport failed to connect to {https://<IP_ADDRESS>:2379  0 }. Err :connection error: desc = "transport: authentication handshake failed: context canceled". Reconnecting..."}
{"level":"warn","ts":"2021-07-08T12:44:05.674+0200","caller":"etcdserver/server.go:2068","msg":"failed to publish local member to cluster through raft","local-member-id":"d7b511c51f0b845f","local-member-attributes":"{Name:andreasp61-f0c52b9d ClientURLs:[https://<IP_ADDRESS>:2379]}","request-path":"/0/members/d7b511c51f0b845f/attributes","publish-timeout":"15s","error":"etcdserver: request timed out, possibly due to connection lost"}
{"level":"warn","ts":"2021-07-08T12:44:11.609+0200","caller":"grpclog/grpclog.go:60","msg":"grpc: addrConn.createTransport failed to connect to {https://<IP_ADDRESS>:2379  0 }. Err :connection error: desc = "transport: authentication handshake failed: context deadline exceeded". Reconnecting..."}
{"level":"warn","ts":"2021-07-08T12:44:15.609+0200","caller":"clientv3/retry_interceptor.go:62","msg":"retrying of unary invoker failed","target":"passthrough:///https://<IP_ADDRESS>:2379","attempt":0,"error":"rpc error: code = DeadlineExceeded desc = context deadline exceeded"}
ERRO[2021-07-08T12:44:15.610140689+02:00] Failed to check local etcd status for learner management: context deadline exceeded
{"level":"warn","ts":"2021-07-08T12:44:15.610+0200","caller":"grpclog/grpclog.go:60","msg":"grpc: addrConn.createTransport failed to connect to {https://<IP_ADDRESS>:2379  0 }. Err :connection error: desc = "transport: authentication handshake failed: context canceled". Reconnecting..."}
{"level":"warn","ts":"2021-07-08T12:44:15.678+0200","caller":"clientv3/retry_interceptor.go:62","msg":"retrying of unary invoker failed","target":"passthrough:///https://<IP_ADDRESS>:2379","attempt":0,"error":"rpc error: code = DeadlineExceeded desc = context deadline exceeded"}
INFO[2021-07-08T12:44:15.678236945+02:00] Failed to test data store connection: context deadline exceeded
{"level":"warn","ts":"2021-07-08T12:44:15.678+0200","caller":"grpclog/grpclog.go:60","msg":"grpc: addrConn.createTransport failed to connect to {https://<IP_ADDRESS>:2379  0 }. Err :connection error: desc = "transport: authentication handshake failed: context canceled". Reconnecting..."}
Steps To Reproduce:

Installed K3s
Add 5 master nodes
Attempt to add a 6th node to the cluster

Expected behavior:
Successfully added the node to the cluster.
Actual behavior:
The new node is not joining the cluster correctly.
Additional context / logs:
The cluster is running behind a corporate proxy, but the proper env vars are being set correctl(http_proxy, https_proxy, no_proxy).

How many servers are you planning on adding to your cluster? Each member you add to the etcd cluster adds a fair bit of overhead; etcd does not recommend any more than 7 for event the most fault-tolerant of use cases. 5 should be more than enough for most folks.
From the logs, it looks like the new member is unable to connect to the other servers in the cluster. Have you checked the logs on the other nodes?
{"level":"warn","ts":"2021-07-08T12:44:05.674+0200","caller":"etcdserver/server.go:2068","msg":"failed to publish local member to cluster through raft","local-member-id":"d7b511c51f0b845f","local-member-attributes":"{Name:andreasp61-f0c52b9d ClientURLs:[https://<IP_ADDRESS>:2379]}","request-path":"/0/members/d7b511c51f0b845f/attributes","publish-timeout":"15s","error":"etcdserver: request timed out, possibly due to connection lost"}

We're just trying to replace 2 if the current masters with 2 new machines. To do this, we wanted first to add the 2 new members and then remove the old ones, in order to not reduce the availability of the masters.
I've been reviewing the logs in the existing masters, and found this relevant log line:
Jul 13 08:52:28 qradafsp25 k3s[1862482]: time="2021-07-13T08:52:28.784736884+02:00" level=info msg="Cluster-Http-Server 2021/07/13 08:52:28 http: TLS handshake error from <IP_ADDRESS>:51624: remote error: tls: bad certificate"
Seems like a problem with the certificates generated for authentication with the etcd cluster. Any clue on how to adress this?

That's a normal message, you will see that once each time a server or agent starts up, as it checks to see if the cluster CA is trusted by the OS CA bundle. If you're seeing it constantly then something might be wrong, but it's not an error if you see it occasionally when nodes are starting up.
Can you confirm that the required ports are open on both the old and new nodes?

I'm experiencing this as well now on v1.20.7+k3s1 (k3OS 0.20.7-k3s1r0). The same base image used to work before to join nodes, but on an attempt to rejoin a node, the logs look like in this issue. Except I'm trying to rejoin the third out of three etcds so currently running on two :-/
I can confirm that in this state, curl https://<IP_ADDRESS>:2379 on the joining node as well as accessing the port from externally, hangs indefinately. The ports are in listeing state by the k3s process and not blocked by anything.
Just checked with v1.21.1+k3s1 (k3OS 0.21.1-k3s1r0). That one seems to work fine.

@mstarostik can you post logs from both nodes?

@brandond unfortunately not; upgraded all nodes to 1.21.1+k3s1 by now and the logs are gone.
And now in this setup I can't reproduce it anymore, both versions join just fine.
From memory:
no (obvious) line suggesting a root cause
etcdctl member list shows the new node as a learner for a short while
then it gets booted off again, after existing nodes experience timeouts trying to connect to :2379 on the new node
new node stays in a loop trying to reach <IP_ADDRESS>:2379
kubelet not started
sorry, that's as much as I can tell anymore

Same error here, also after updating to v1.21.4+k3s1.
New node
Logs in node that I want to add as a server to the cluster:
ERRO[2021-09-16T12:41:33.318032754+02:00] Failed to check local etcd status for learner management: context deadline exceeded
{"level":"warn","ts":"2021-09-16T12:41:33.318+0200","caller":"grpclog/grpclog.go:60","msg":"grpc: addrConn.createTransport failed to connect to {https://<IP_ADDRESS>:2379  <nil> 0 <nil>}. Err :connection error: desc = \"transport: authentication handshake failed: context canceled\". Reconnecting..."}
{"level":"warn","ts":"2021-09-16T12:41:33.331+0200","caller":"clientv3/retry_interceptor.go:62","msg":"retrying of unary invoker failed","target":"passthrough:///https://<IP_ADDRESS>:2379","attempt":0,"error":"rpc error: code = DeadlineExceeded desc = context deadline exceeded"}
INFO[2021-09-16T12:41:33.331427438+02:00] Failed to test data store connection: context deadline exceeded
{"level":"warn","ts":"2021-09-16T12:41:33.331+0200","caller":"grpclog/grpclog.go:60","msg":"grpc: addrConn.createTransport failed to connect to {https://<IP_ADDRESS>:2379  <nil> 0 <nil>}. Err :connection error: desc = \"transport: authentication handshake failed: context canceled\". Reconnecting..."}
{"level":"warn","ts":"2021-09-16T12:41:33.979+0200","caller":"grpclog/grpclog.go:60","msg":"grpc: addrConn.createTransport failed to connect to {https://<IP_ADDRESS>:2379  <nil> 0 <nil>}. Err :connection error: desc = \"transport: authentication handshake failed: context deadline exceeded\". Reconnecting..."}
{"level":"warn","ts":"2021-09-16T12:41:38.326+0200","caller":"etcdserver/server.go:2068","msg":"failed to publish local member to cluster through raft","local-member-id":"adea52081ea2c35d","local-member-attributes":"{Name:andreasp35-aba1c9b4 ClientURLs:[https://<IP_ADDRESS>:2379]}","request-path":"/0/members/adea52081ea2c35d/attributes","publish-timeout":"15s","error":"etcdserver: request timed out"}
{"level":"warn","ts":"2021-09-16T12:41:48.317+0200","caller":"clientv3/retry_interceptor.go:62","msg":"retrying of unary invoker failed","target":"passthrough:///https://<IP_ADDRESS>:2379","attempt":0,"error":"rpc error: code = DeadlineExceeded desc = context deadline exceeded"}

It seems etcd do not start properly in the new node.
root@andreasp35:~# ETCDCTL_ENDPOINTS='https://<IP_ADDRESS>:2379' ETCDCTL_CACERT='/var/lib/rancher/k3s/server/tls/etcd/server-ca.crt' ETCDCTL_CERT='/var/lib/rancher/k3s/server/tls/etcd/server-client.crt' ETCDCTL_KEY='/var/lib/rancher/k3s/server/tls/etcd/server-client.key' ETCDCTL_API=2 etcdctl member list
client: etcd cluster is unavailable or misconfigured; error #0: client: endpoint https://<IP_ADDRESS>:2379 exceeded header timeout

root@andreasp35:~# ETCDCTL_ENDPOINTS='https://<IP_ADDRESS>:2379' ETCDCTL_CACERT='/var/lib/rancher/k3s/server/tls/etcd/server-ca.crt' ETCDCTL_CERT='/var/lib/rancher/k3s/server/tls/etcd/server-client.crt' ETCDCTL_KEY='/var/lib/rancher/k3s/server/tls/etcd/server-client.key' ETCDCTL_API=3 etcdctl member list
Error:  context deadline exceeded

Although it is able to connect and add correctly to the ETCD cluster at first moment. But after local stops responding, it deletes from the etcd cluster.
Current server node
Logs in current server node:
Sep 16 12:39:38 qradaasp09 k3s[1281082]: time="2021-09-16T12:39:38.285244113+02:00" level=info msg="Cluster-Http-Server 2021/09/16 12:39:38 http: TLS handshake error from <IP_ADDRESS>:42184: remote error: tls: bad certificate"

@roger21gm can you attach the complete logs from the "current server node" as well as the output of kubectl get node -o yaml ?

Node list: nodes.txt

I use this documentation to add new master to the cluster:
https://rancher.com/docs/k3s/latest/en/installation/ha-embedded/
K3S_TOKEN=SECRET k3s server --server https://<ip or hostname of server1>:6443

@roger21gm it looks to me like you have 6 nodes in your cluster. One of them is not configured the same as the rest; and none of them appear to have K3S_URL or --server set. How exactly have you been adding nodes to this cluster, and why do you have so many servers?
    name: qasp04
      k3s.io/node-args: '["server","--flannel-backend","none","--cluster-cidr","<IP_ADDRESS>/16","--service-cidr","<IP_ADDRESS>/16","--cluster-dns","<IP_ADDRESS>","--disable","traefik,servicelb"]'
      k3s.io/node-env: '{"K3S_DATA_DIR":"/var/lib/rancher/k3s/data/bc7e12eeb86c005efd104d57a8733f0e9fbf2ede3571bad3da2cd0326b978aed","K3S_TOKEN":"********"}'
--
    name: qasp09
      k3s.io/node-args: '["server","--flannel-backend","none","--cluster-cidr","<IP_ADDRESS>/16","--service-cidr","<IP_ADDRESS>/16","--cluster-dns","<IP_ADDRESS>","--disable","traefik,servicelb"]'
      k3s.io/node-env: '{"K3S_DATA_DIR":"/var/lib/rancher/k3s/data/bc7e12eeb86c005efd104d57a8733f0e9fbf2ede3571bad3da2cd0326b978aed"}'
--
    name: qasp25
      k3s.io/node-args: '["server","--flannel-backend","none","--cluster-cidr","<IP_ADDRESS>/16","--service-cidr","<IP_ADDRESS>/16","--cluster-dns","<IP_ADDRESS>","--disable","traefik,servicelb"]'
      k3s.io/node-env: '{"K3S_DATA_DIR":"/var/lib/rancher/k3s/data/bc7e12eeb86c005efd104d57a8733f0e9fbf2ede3571bad3da2cd0326b978aed","K3S_TOKEN":"********"}'
--
    name: qasp34
      k3s.io/node-args: '["server","--flannel-backend","none","--cluster-cidr","<IP_ADDRESS>/16","--service-cidr","<IP_ADDRESS>/16","--cluster-dns","<IP_ADDRESS>","--disable","traefik,servicelb"]'
      k3s.io/node-env: '{"K3S_DATA_DIR":"/var/lib/rancher/k3s/data/bc7e12eeb86c005efd104d57a8733f0e9fbf2ede3571bad3da2cd0326b978aed"}'
--
    name: qfsp03
      k3s.io/node-args: '["server","--server","https://<IP_ADDRESS>:6443","--token","********"]'
      k3s.io/node-env: '{"K3S_DATA_DIR":"/var/lib/rancher/k3s/data/bc7e12eeb86c005efd104d57a8733f0e9fbf2ede3571bad3da2cd0326b978aed"}'
--
    name: qfsp23
      k3s.io/node-args: '["server","--flannel-backend","none","--cluster-cidr","<IP_ADDRESS>/16","--service-cidr","<IP_ADDRESS>/16","--cluster-dns","<IP_ADDRESS>","--disable","traefik,servicelb"]'
      k3s.io/node-env: '{"K3S_DATA_DIR":"/var/lib/rancher/k3s/data/bc7e12eeb86c005efd104d57a8733f0e9fbf2ede3571bad3da2cd0326b978aed","K3S_TOKEN":"********"}'

Hi @brandond, I try to add the server to the cluster using this command:
k3s server --server https://<IP_ADDRESS>:6443 --flannel-backend=none --cluster-cidr=<IP_ADDRESS>/16 --service-cidr=<IP_ADDRESS>/16 --cluster-dns=<IP_ADDRESS> --disable traefik,servicelb

And that's the output I get:
out.txt
Related to other servers, I joined to the cluster the first time using the command above, and then I ran the service using systemctl. This is the content of /etc/systemd/system/k3s.service.env
K3S_TOKEN=<TOKEN>
no_proxy=<IP_ADDRESS>/16,<IP_ADDRESS>/16, *.lc.es, gitlab.lc,registrypro.lc,registry-s1.lc,registry-s2.lc.es, kubernetes.default.svc, *.csrt.csrt, <IP_ADDRESS>/23, <IP_ADDRESS>/24
https_proxy="<proxy>"
http_proxy="<proxy>"

I would honestly recommend against adding any more servers at this point. Etcd recommends no more than 7 servers for the very largest of workloads, and only with appropriately sized hardware.

https://etcd.io/docs/v3.3/faq/#what-is-maximum-cluster-size
A 5-member etcd cluster can tolerate two member failures, which is enough in most cases. Although larger clusters provide better fault tolerance, the write performance suffers because data must be replicated across more machines

What I'm trying to do is add two servers and remove two old ones. I don't want more than 5 masters, but i dare not take out masters until I manage to add new ones.

I got those errors because the ports 2379 and 2380 weren't opened.


How to scroll to a component from another component?

I got the app.component.html having smth like this
<md-sidenav-container (pan)="swipeSidenav($event)" mode="over">
  <md-sidenav #sidenav>
    <app-sidenav-content></app-sidenav-content>
  </md-sidenav>
  <nav class="mat-elevation-z4">
    <app-top-nav></app-top-nav>
  </nav>
  <header>
    <app-intro></app-intro>
  </header>
  <main>
    <app-primary-offer></app-primary-offer>
    <app-contact-form></app-contact-form>
    <app-offer></app-offer>
    <app-projects id="awesomePart"></app-projects>
  </main>
</md-sidenav-container>

And I want to scroll to projects so I assign the ID to it, but the button itself is inside app-top-nav component
<button pageScroll href="#awesomePart" fxHide fxShow.gt-md md-button>Offers</button>
So when pressing it nothing happens. I generated the project with Angular-Cli and added it dependency in the app.modules.ts
import { Ng2PageScrollModule } from 'ng2-page-scroll';
...
imports: [
   ...
    Ng2PageScrollModule.forRoot()
  ],

So, how do I make it work with communicating between components? I tried using a shared service
constructor(private pageScrollService: PageScrollService, @Inject(DOCUMENT) private document: any) { }

public goToPart(): void {
    const pageScrollInstance: PageScrollInstance = PageScrollInstance.simpleInstance(this.document, 'awesomePart');
    this.pageScrollService.start(pageScrollInstance);
  }

and then using it within the component where button click occurs, but it throws me
Scrolling not possible, as we can't find the specified target
Even though I know the id is correct.

To add to the message. document.getElementByid('id') finds it perfectly fine.

Hi,
the second parameter of the simpleInstance method is similar to a css selector. Add the # in front of the element id:
const pageScrollInstance: PageScrollInstance = PageScrollInstance.simpleInstance(this.document, '#awesomePart');

In future versions it might be possible to select scrollTargets based on other thinks than the id.

Hello,
There is no more error in the console about missing target but it doesn't scroll at all when I press the button. Am I not getting smth?

I assume you have to use a custom scrollView, as the material2 sidenav directive introduces a non-trivial DOM hierarchy: https://github.com/Nolanus/ng2-page-scroll/wiki/Using-the-right-scrolling-view

Hello again,
Apologies for missing. I tried to determine, which element had $0.scrollTop equal to a non-zero value but all of them returned 0 o_O so I am not really sure to which element I should even bind it.
My app.component.html looks like this
<md-sidenav-container (pan)="swipeSidenav($event)" mode="over">
  <md-sidenav #sidenav>
    <app-sidenav-content></app-sidenav-content>
  </md-sidenav>
  <nav class="mat-elevation-z4">
    <app-top-nav></app-top-nav>
  </nav>
  <header>
    <app-intro id="home"></app-intro>
  </header>
  <main>
    <app-primary-offer id="core"></app-primary-offer>
    <app-contact-form id="sign" [captcha]="captchaValue" (requestCaptcha)="captchaRef.execute()"></app-contact-form>
    <app-offer id="offers"></app-offer>
    <app-projects id="projects"></app-projects>
  </main>
</md-sidenav-container>
<re-captcha #captchaRef="reCaptcha" size="invisible" siteKey="secret" (resolved)="captchaValue = $event"></re-captcha>

Check that plunkr using md-sidenav from material2 library: https://plnkr.co/edit/mmBS1AZv7Jd7eCTrFCOZ?p=preview
The scrollable view is the div.mat-sidenav-content the sidenav module introduces while creating itself. Due to that is's not possible to set an id on the object and you have to get it manually in the component to pass it to the PageScrollInstance.

Freaking brilliant. Thanks a lot. I've been trying to wrap my head around it for hours.


Speedup 2x tests

On my machine, this patch speedup tests execution from 90 seconds to 50 secs.

Whaaaaat. What improvement are you seeing? What do Django folks say about switching the hasher for tests?

With this patch, it doesn't take so long to run tests. From my point of view, this is an improvement. :)
Switching hasher for tests is a tip from Django doc.

@pbanaszkiewicz, could we merge this with the next release? This'll make my life much easier.

I'm going to merge it soon, just let me test it first.

Indeed this brings decent speedup: 27s vs 17s for me.
Merging with single change (comment about that conditional in the settings).
@chrismedrela If you want further speed up the tests, you can try the new multithreaded mode available since Django v1.9 I believe.
Thanks for this contribution!

@pbanaszkiewicz, do you mean executing python3 manage.py test --parallel=4? On my machine, I get NotImplementedError: subtests aren't supported at this time error from multiprocessing library.

@chrismedrela yes, but I didn't claim it was painless :-)

Tests now throw segmentation fault everytime.


BUG:Cookie Box Not Scrollable, Accept/Reject Buttons Not Visible

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The cookie consent box on the homepage is not scrollable, preventing users from seeing or interacting with the "Accept" or "Reject" buttons. This issue hinders users from making their cookie preferences, affecting their browsing experience.
Expected Behavior: The cookie consent box should allow scrolling or size should increase so that users can view and click the "Accept" or "Reject" buttons to set their preferences.
i would like to work on it please it to me under GSSOC'24EXTD, hacktoberfest'24 and level labels.
@anuragverma108
https://github.com/user-attachments/assets/2f2446b4-8ea8-43e2-bf52-33ae749bb203
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@sailaja-adapa
You closed the issue but i didn't get any update about my pr you said u will merge it can you please tell me about the pr update i just wanna know to do better next time.

@abhi03ruchi
please update me about my pr on this issue

Oh I am sorry @Chitrakadyan2005 I didn't seen msg
When coming to this PR the issue has already been done and it was now perfectly fine try to work on new issue like social icons were colliding with Navbar


TFRS - Hide Category D selection after director decision on credit transfer

Describe the task
Hide the Category D selection sentence in the UI for IDIR users once the director has made a decision on the credit transfer, either approved or declined it.
Purpose
The IDIR user is not able to select this field once the credit transfer has been processed, therefore it should be hidden from the UI. Currently it looks selectable.
Acceptance Criteria

[ ] first
[ ] second
[ ] third

Screenshot:

Hey team! Please add your planning poker estimate with Zenhub @jig-patel @prv-proton @tim738745


Fix for #46

@civfanatics/civfanatics-devs Ping

let's update the PR summary to reflect its purpose, and also fill out its description with "what" and "why".


Getting Error after launching  launch_hw_emu.sh

The error is:
[C 10:01:23.052 NotebookApp] Bad configuration encountered during initialization: No such notebook dir: ''/usr/share/example-notebooks''
xinit: giving up
xinit: unable to connect to X server: Connection refused
xinit: server error
I also tried running the command, but it stuck after the kernel graph ran. Additionally, I have attached the screen shot.
Please provide appropriate steps for successful emulation testing

I also tried running the command but it stuck after kernel graph ran

Ignore the "Notebook" errors by pressing "enter".

The process you do is correct, the printing information means the program has already under simulation. However since the iteration number is 100 which is too large for this case in simulation, it needs too much time to get the result. Try below and wait for tens of minutes:

./hostexe mm_hw_emu.xclbin 1024 1024 1024 1 0

The output shown in the snapshot is as follows: is this the correct output?

Right, this is as expected.


bug fix : fault tolerance , not retry did not reply error message

Can one of the admins verify this patch?

Pull Request Test Coverage Report for Build 1781

0 of 12 (0.0%)  changed or added relevant lines in 1 file are covered.
18 unchanged lines in 6 files lost coverage.
Overall coverage increased (+0.1%) to 62.175%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core/handler/loadbalance_handler.go
0
12
0.0%

Files with Coverage Reduction
New Missed Lines
%

core/handler/loadbalance_handler.go
1
31.4%

core/registry/healthz.go
1
43.84%

core/lager/lager.go
1
76.0%

core/invoker.go
2
50.0%

core/handler/transport_handler.go
3
22.89%

core/lager/logrotate.go
10
39.07%

Totals

Change from base Build 1779:
0.1%

Covered Lines:
5523

Relevant Lines:
8883

💛  - Coveralls


LGTM.com - false positive (user-provided value)

Description of the false positive
It is not detecting that I validate that 'user-provided value'. I make a request to my database and its result is stored into the app variable. That contains an array of allowed values and is checked in line 28. I use a function to check if it is inside that array (case insensitive).
URL to the alert on the project page on LGTM.com
https://lgtm.com/projects/g/Mc-Auth-com/Mc-Auth-Web/snapshot/41ec351596d9eaf086d8530beb383734cadebce9/files/routes/oAuth2.js?sort=name&dir=ASC&mode=heatmap#x2ee0a58e9646f1d3:1

Thank you for your report! We will look into improving our query to detect your validation logic.


fix(action/inject): prevent cross-seed inject if no client

Also updated getClient() to return null if no client is configured.

It's necessary for the blocklist branch as the client get's used outside of injection, but I added it here so the diff is easier.

If you want a simpler diff, would you mind making the case for that in a separate PR?

It's very straight forward so I don't think it's worth it. I might move all the new functions added to clients to it's out PR though. That would make the rest easier to review.


[Suggestion] (Though im kinda tarded) Weapon on ground chams

I have no clue if that is possible, nor if you could do it, but I love using this cheat purely for visual purposes because it's so refined when it comes to that. Keep up the awesome work daniel, we all really love your work!!!!!

There is Glow available for this!

Yes, there is glow, but chams fill up the whole weapon. I know that there are different types of glows that fill up the weapon, but they are seeable through a wall. Chams would look a lot better nonetheless with these sexy Osiris chams

You might look at chams.cpp there is a caption where weapons get rendered chams. Its a good trial to make WeaponChams on the ground instead stacked at localplayer.
void Chams::renderWeapons(void* ctx, void* state, const ModelRenderInfo& info, matrix3x4* customBoneToWorld) const noexcept
{
    const auto localPlayer{ interfaces.entityList->getEntity(interfaces.engine->getLocalPlayer()) };
    if (!localPlayer->isScoped()) {
        auto applied{ false };
        for (size_t i = 0; i < config.chams[WEAPONS].materials.size(); i++) {
            if (config.chams[WEAPONS].materials[i].enabled) {
                if (applied)
                    hooks.modelRender.callOriginal<void, void*, void*, const ModelRenderInfo&, matrix3x4*>(21, ctx, state, info, customBoneToWorld);
                applyChams_2(config.chams[WEAPONS].materials[i], false, localPlayer->health());
                applied = true;
            }
        }
    }
}

I don't know if there's anything else required, but I've checked for strstr "_dropped.mdl" which does return true, but when applying the chams it doesn't actually do anything, is there anything special required?

I don't know if there's anything else required, but I've checked for strstr "_dropped.mdl" which does return true, but when applying the chams it doesn't actually do anything, is there anything special required/am I doing something wrong?

dropped weapons start with "w_", "_dropped" is only for the C4 afaik

dropped weapons start with "w_", "_dropped" is only for the C4 afaik

From what I tested it doesn't matter, unless there's a special modifier (like mat_wireframe 1), weapons arent passed through DM/DME.

Did a little research, a free cheat (open source) has weapon on ground chams. If you guys wanted to look into it, its on multiple forums.  Nonetheless thanks for contributing in this topic.


Theming update

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/graycoreio/daffodil/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?
The palettes are not cohesive, and yellow is used as a primary color when it should be supplemental.
Fixes: N/A
What is the new behavior?
Update the color palettes to be more reflective of our theming practices. The default hues sits close to the 4:5 contrast ratio.
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

@damienwebdev documentation added.


[BEAM-12572] Fix failing python examples tests in Dataflow runner

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Python Examples_Dataflow

Run Python Examples_Direct

Run Python Examples_Spark

Run Python Examples_Flink

Run Python Examples_Flink

Run Python Examples_Spark

Run Python Examples_Direct

Run Python Examples_Dataflow

Run Python Examples_Dataflow

Run Python Examples_Spark

Run Python Examples_Direct

Run Python Examples_Flink

Run Python Examples_Dataflow

@tvalentyn could you help me to review this?
With these changes now Dataflow_examples job is green.
I disabled https://github.com/apache/beam/blob/1b69bf3533c828c7a1ee20074bf51914885328e8/sdks/python/apache_beam/examples/complete/top_wikipedia_sessions_test.py#L105 because the output is inconsistent between Dataflow vs [Direct, Spark, Flink]

Jobs results:

Dataflow

Run Python Examples_Flink

Run Python Examples_Spark

Run Python Examples_Direct

Run Python Examples_Dataflow

Will this fix https://ci-beam.apache.org/job/beam_PostCommit_Python_Examples_Dataflow/56/ ?

Will this fix https://ci-beam.apache.org/job/beam_PostCommit_Python_Examples_Dataflow/56/ ?

Yes, here is a successful run https://ci-beam.apache.org/job/beam_PostCommit_Python_Examples_Dataflow_PR/12/

LGTM, thanks!

Thanks!


Add an opinion about making scripts chatty

I'm generally in favor of keeping the opinions section pithy, but I think this may be a fit.

Use real logging, not print statements (we have some boilerplate)

easy redirect to multiple places
timestamps and module for free
easy to see what happens on someone else's instance

Include tqdm by default

+1 for tqdm
I am having a hard time imagining a python data science project which could not benefit from using tqdm

Added some of this in the v2 scaffold. Closing for now.


lots of stuff

yikes

:tada: This PR is included in version 0.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Improvement over audio component

As per the design introduced in #4563 for an enhanced version of browsing results with sensitive images, the audio component requires some changes.
Description
The three variants of the component documented in the Design Library need to allow hiding the image beneath a blur layer that has the bg-blur variable applied. There are no changes over current interactions.
Screenshots from the design file made for #4563

XL breakpoint
XS breakpoint

Additional context
The tokens applied are according to #3592 project

Implementation
If shouldBlur is true, the VAudioThumbnail in VRowLayout component:
https://github.com/WordPress/openverse/blob/975c543a221a19169c7bde5f31ea7a8eafb1f977/frontend/src/components/VAudioTrack/layouts/VRowLayout.vue#L11
should be replaced with a div with bg-blur class and a VIcon with eye-open as the name. Sample use of VIcon:
https://github.com/WordPress/openverse/blob/975c543a221a19169c7bde5f31ea7a8eafb1f977/frontend/src/components/VSafetyWall/VSafetyWall.vue#L56


Tweak api

This Pull request:

Remove Max Parameter in API
Fix bug in benchmark

@lfousseaq  it's seem like I cannot add you as reviewer


What about parameters?

I have a postman collection, and have requests like /profile/umpirsky in it, where umpirsky is username parameter. Concrete value for parameters is what is expected in postman collections.
I export this, and import in Insomnia, then export from there and use swaggyermania to export it to swagger UI format. But then I have no parameters in swagger UI.
How do you usually manage this? Keep /profile/{username} placeholders in Insomnia? What about paremeter type, description...?
Thanks for the nice tool!

Keep /profile/{username} placeholders in Insomnia
At this moment, the tool doesnt support parameter type & description, but im working on it :)


Symbol submission

Removes unnecessary indentation.
Fixes log location.

facepalm That'll teach me to not copy/paste when I should have...
:+1:

:)


After building with cmake, how do you load the build onto an emulator?

I managed to successfully build using cmake, but I am having trouble figuring out how to load the code onto an emulator through android studio.

I'm not sure what you mean by "the code".  If you are referring to the real-time FaceFilter app, I don't think it will be easy to run that in an emulator.  You would have to create some kind of camera interface to support testing.   There are unit tests that run the drishti library using the Android QEMU emulator.  If you are looking for the latter, I can provide some links.

Yes, the links will be helpful. Also, sorry for not clarifying, but I did mean the real time FaceFilter App. Also for the camera interface you mentioned, do you mean that I would have to link the emulator to a webcam? or do you mean that I would have to develop some sort of UI to support a camera?

or do you mean that I would have to develop some sort of UI to support a camera?

AFAIK, the Android emulator doesn't come with camera support.  I did check at one point to see if end-to-end application level testing would be possible.   I think you would have to write some code to play back video files or something like that.
Why do you need it?
If you manage your unit tests with gauze, it manage the emulator for you:
https://github.com/hunter-packages/gauze
This  allows testing OpenGL ES shader pipeline in Travis CI.
See the  docs (in particular the Amdroid section):
https://gauze.readthedocs.io/en/latest/prereq/android.html
Example (gauze_main())
https://github.com/elucideye/drishti/blob/378725771b0b4353e5b3fa93f32175023dfc0e0f/src/tests/drishti/test-drishti.cpp#L38
Or any of the tests in the gauze repo:
https://github.com/hunter-packages/gauze/tree/master/test/gauze

Thank you, I'll take a look at the documentation. I'm running it on an emulator because I don't have an android phone handy. As for why I'm interested in running the example code, I want to see how well the library runs on mobile to see if I can apply the library to a research project I'm a part of. My plan would be for the code to run in the background to calculate distance between the phone and face and to also possibly calculate gaze. The calculated information would then be used for image processing variables.


How  to manage an asyncronous addValidator for a namedContext?

Let's say I wanted to validate my whole form, but needed to make a call to the back end to do it.

To add a custom validation function that is called for all keys for a specific SimpleSchema instance, use mySimpleSchema.addValidator(myFunction).

// validate an address by looking up the whole address through a meteor method
AddressSchema.addValidator(function () {
  Meteor.call('Address._validateAddress', this.value, function (err) {
    // ???
  });
});

While the docs show how to use it for a single key, it doesn't seem very clear how to use it for an entire form. How would this be done?

To help me catch up, I'm closing some old issues. If you still need help, post a comment asking to reopen.
Also, this is more of a collection2 or autoform question, since simpleschema could simply be run in the method and then send the invalidKeys back to the client.


Issue47 (a sub issue to 46) change btnLocationClick to extract row number correctly

btnLocationClick (event, buttonId) {   var rowNumber = buttonId.slice(-1); needs to change to extract more than 1 digit

script.js line 192     var chosenPropertyRowNumber = buttonId.match(/\d+/);


Customize lint vars output, improve messaging

Originally added in #24860, the output of the messaging was confusing to me at first, stating that a variable was being used once rather than it's unused. The former sent me down a tailspin of thinking my editor's find function was broken :D.
While I was in there, I stylized the output a bit, too.

Please look at the package I've made and submit patches there. This script
is obsolete and I will replace it.
I have a few ideas for the symbols yeah. And I already have added colors
there. But feel free to submit patches so that we are done sooner and
switch to it faster here :)
On Dec 22, 2017 20:18, "Mark Otto"<EMAIL_ADDRESS>wrote:

@mdo commented on this pull request.
In build/lint-vars.js
https://github.com/twbs/bootstrap/pull/25049#discussion_r158537344:

@@ -27,7 +27,7 @@ function findUnusedVars(dir) {
process.exit(1)
}

console.log(Finding unused variables in "${dir}"...)

console.log(\x1b[1m%s\x1b[0m, \n➜ Finding unused variables in "${dir}"...)

I'd rather not add those unicode stuff because it's not cross-platform.
Is there a cross-platform solution you can recommend? I assume you mean
Mac and Windows, correct?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/twbs/bootstrap/pull/25049#discussion_r158537344, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAVVtbSWhabsQIvtfdGhgCAkaqAx1GXxks5tC_KMgaJpZM4RKasR
.


Charge Costs integration - Grafana and API

This PR isn't ready for merging yet, however I did want to consult on the API routing so that I understand the overall best direction before working on the controllers and functions.
At the moment, we don't have a web controller for object classes represented in the API other than on a per-Car basis. Although charges could be associated to cars, I don't really see a reason to do so (this could be filtered out/in of the dataset, and in my view the more relevant pivot is the address rather than car), so I have represented these 2 tables separately - one controller for charges and another for addresses.
Does that make sense, or would you see this arranged in another fashion?

I should add, I also modified the column headings in the charges.json grafana dashboard to stop them bunching up on each other after adding the Cost column. The purpose is to make some a little less verbose but for the headings to still make sense, if any are unclear I can fix them up

I could imagine that due to the potentially large number of stored addresses the GET /address route would need to be paginated.  What do you say we embed the address in the charges (instead of returning an address_id) and only add the charges_controller? The embedded address could contain only the most important attributes e.g. display_name, latitude and longitude. That way one requests could return all information and it would save additional work.

Sounds good, @adriankumpf . Makes sense to me.
I am closing this PR off as there's a merge conflict with the charge stats grafana template and I want to avoid it being an issue again if there are any more updates, so I'll split the grafana template changes from the API changes.


Store user's Ethereum address in DB to support BlockExplorer

To support BlockExplorer omisego/blockexplorer/issues/14

We can get this info by joining transactions and txoutputs table and looking at txoutputs.owner.


Always notify term change on RequestVote.

Overview
Previously there was a bug that ignored new terms from incoming vote requests when the sender's log was out of date. This fixes it so that new terms are saved even if the vote is rejected.

+1, makes sense to me.


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 1178 ms

Resolved: Bofrost.be performance has improved in 097b103.


Fix URLs of mirror and dark site

https://dark.namu.wiki isn't available now, so I replaced it with https://namu.mirror.wiki/dark. Also http://namu.mirror.wiki/wiki is now replaced with https://namu.mirror.wiki/w.

Thanks, @yous!


Implement Pull Requests history

It would be nice to have history of changes in the pull requests as it is in gitlab, for example: if someone pushes commit with force, it would be nice to have diff of what was before this push, so, probably, we should store changes for that and notify user on each push event as in gitlab.

@lafriks I am not sure I understand what you were saying.

I asked if anyone has ideas on how to store/get previous changes that have been removed from branch by force push

What about storing them in the gitea's database?

The question is about the record format, I believe.
I don't have the experience with git database format, but gerrit or gitlab formats should do.
Particularly, each PR version can be saved as a hidden branch; the comments to the commits would be stored the way they are now.
You may even want to reimplement gerrit's master feature: use or calculate a mapping between different versions of same commit across PR versions in Change-Id: field, and display a history of a commit.

Actually I just discovered this issue. I think this is the same as the one I opened: #12604.
The related work in progress pull request is: #12665

Yay, thanks a ton!


Feature/nix

Note: with this the tool-versions code to set up the correct go version also becomes obsolete since we can just use the Nix environment for that

Note: with this the tool-versions code to set up the correct go version also becomes obsolete since we can just use the Nix environment for that

I will add a task to get nix added to this properly. I think it needs to be placed in the setup and not the runner with the ability to use nix or the old until more people are trained on nix and more projects having working nix files for test.


[6.5.x] RHBPMS-4249, JBPM-5319 - Transaction exception for Per Process Instan…, RHBPMS-4247, JBPM-5317 - Failed to upgrade process instances in Maria

@sutaakar and here is back port to 6.5.x

jenkins retest this


WARNING: Using outdated gnuplot, use v4.0 for best results

After runs gnuplot --version, mummerplot.pl will depending on the return value print:
WARNING: Using outdated gnuplot, use v4.0 for best results

https://github.com/gmarcais/mummer/blob/v4.0.0beta/scripts/mummerplot.pl#L1502
First, this is confusing as the message appears if using more recent versions of gnuplot.
Second, should it say v4.0 or later? Is gnuplot 5.0.x acceptable?

Thanks for the bug report.
This message will only show up if gnuplot version 3 or earlier is the default version. It will not show up for version 4 or 5. I modified the message to say v4 or later. Yes, gnuplot 5 is acceptable.


updated profile information

Updated profile information

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: ritzmodi sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

:x: Validation status: errors

File
Status
Preview URL
Details

rd/ritesh-modi.yml
:x:Error

Details

rd/mamadou-bah.yml
:x:Error

Details

rd/joel-oleson.yml
:x:Error

Details

rd/udi-dahan.yml
:x:Error

Details

rd/mateusz-kozlowski.yml
:x:Error

Details

rd/dan-hermes.yml
:x:Error

Details

rd/sebastian-fyda.yml
:x:Error

Details

rd/igor-abade.yml
:x:Error

Details

rd/nishith-pathak.yml
:x:Error

Details

rd/reza-rad.yml
:x:Error

Details

rd/paula-januszkiewicz.yml
:x:Error

Details

rd/amit-bansal.yml
:x:Error

Details

:x:Error

Details

rd/ritesh-modi.yml

[Error] Unable to load file: ritesh-modi.yml via processor: Profile: Validation against "https://dotnet.github.io/docfx/schemas/v1.0/schema.json#" failed:
Invalid type. Expected Number but got Array. Path 'location.lat'.
Invalid type. Expected Number but got Array.  Expected String but got Null. Path 'facebook'.
Invalid type. Expected String but got Null. Path 'twitter'.
Invalid type. Expected String but got Null. Path 'linkedin'.

rd/sebastian-fyda.yml

[Error] Unable to load file: sebastian-fyda.yml via processor: Profile: sebastian-fyda.yml is not in supported format: (Line: 18, Col: 18, Idx: 592) - (Line: 18, Col: 19, Idx: 593): While parsing a block mapping, did not find expected key.

rd/igor-abade.yml

[Error] Unable to load file: igor-abade.yml via processor: Profile: igor-abade.yml is not in supported format: (Line: 18, Col: 18, Idx: 572) - (Line: 18, Col: 19, Idx: 573): While parsing a block mapping, did not find expected key.

rd/nishith-pathak.yml

[Error] Unable to load file: nishith-pathak.yml via processor: Profile: nishith-pathak.yml is not in supported format: (Line: 37, Col: 19, Idx: 2524) - (Line: 37, Col: 20, Idx: 2525): While parsing a block mapping, did not find expected key.

rd/reza-rad.yml

[Error] Unable to load file: reza-rad.yml via processor: Profile: reza-rad.yml is not in supported format: (Line: 18, Col: 18, Idx: 562) - (Line: 18, Col: 19, Idx: 563): While parsing a block mapping, did not find expected key.

rd/paula-januszkiewicz.yml

[Error] Unable to load file: paula-januszkiewicz.yml via processor: Profile: paula-januszkiewicz.yml is not in supported format: (Line: 18, Col: 18, Idx: 618) - (Line: 18, Col: 19, Idx: 619): While parsing a block mapping, did not find expected key.

rd/amit-bansal.yml

[Error] Unable to load file: amit-bansal.yml via processor: Profile: amit-bansal.yml is not in supported format: (Line: 18, Col: 18, Idx: 578) - (Line: 18, Col: 19, Idx: 579): While parsing a block mapping, did not find expected key.

[Error] mamadou-bah.yml is not in supported format: (Line: 18, Col: 18, Idx: 577) - (Line: 18, Col: 19, Idx: 578): While parsing a block mapping, did not find expected key.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:x: Validation status: errors

File
Status
Preview URL
Details

rd/amit-bansal.yml
:x:Error

Details

rd/joel-oleson.yml
:x:Error

Details

rd/udi-dahan.yml
:x:Error

Details

rd/paula-januszkiewicz.yml
:x:Error

Details

rd/dan-hermes.yml
:x:Error

Details

rd/sebastian-fyda.yml
:x:Error

Details

rd/mamadou-bah.yml
:x:Error

Details

rd/mateusz-kozlowski.yml
:x:Error

Details

rd/nishith-pathak.yml
:x:Error

Details

rd/reza-rad.yml
:x:Error

Details

rd/igor-abade.yml
:x:Error

Details

:x:Error

Details

rd/amit-bansal.yml

[Error] Unable to load file: amit-bansal.yml via processor: Profile: amit-bansal.yml is not in supported format: (Line: 18, Col: 18, Idx: 578) - (Line: 18, Col: 19, Idx: 579): While parsing a block mapping, did not find expected key.

rd/joel-oleson.yml

[Error] Unable to load file: joel-oleson.yml via processor: Profile: joel-oleson.yml is not in supported format: (Line: 18, Col: 18, Idx: 578) - (Line: 18, Col: 19, Idx: 579): While parsing a block mapping, did not find expected key.

rd/udi-dahan.yml

[Error] Unable to load file: udi-dahan.yml via processor: Profile: udi-dahan.yml is not in supported format: (Line: 18, Col: 18, Idx: 567) - (Line: 18, Col: 19, Idx: 568): While parsing a block mapping, did not find expected key.

rd/paula-januszkiewicz.yml

[Error] Unable to load file: paula-januszkiewicz.yml via processor: Profile: paula-januszkiewicz.yml is not in supported format: (Line: 18, Col: 18, Idx: 618) - (Line: 18, Col: 19, Idx: 619): While parsing a block mapping, did not find expected key.

rd/dan-hermes.yml

[Error] Unable to load file: dan-hermes.yml via processor: Profile: Validation against "https://dotnet.github.io/docfx/schemas/v1.0/schema.json#" failed:
Invalid type. Expected String but got Null. Path 'facebook'.
Invalid type. Expected String but got Null. Path 'twitter'.
Invalid type. Expected String but got Null.

[Error] amit-bansal.yml is not in supported format: (Line: 18, Col: 18, Idx: 578) - (Line: 18, Col: 19, Idx: 579): While parsing a block mapping, did not find expected key.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Please also fill in the last two parts:
lat: [decimal latitude]
long: [decimal longitude]