fix(docs) WithPusher has been deprecated

WithPusher has been deprecated, WithExporter should be used.

lgtm, just need the CLA

lgtm, just need the CLA

Thanks, I've signed it two times, but it doesn't seem to change the status here on the PR :/

Hi! Could it be that controller.WithExporter isn't available in opentelemetry-go 0.16.0? (the version currently used in the example).
Faced that error walking through the example, which is fixed going back to controller.WithPusher.
Didn't find the new WithExporter function on the 0.16.0 tree and the change only seems to be mentioned in the 0.19.0 release:

"go.opentelemetry.io/sdk/metric/controller.basic".WithPusher is replaced with WithExporter to provide consistent naming across project. (#1656)

Anyway, I dropped a PR to explore the upgrade to opentelemetry-go to 0.19.0 if you're interested!


Updated Count

Basically it should indicate how many files have been updated/added in the repo

@dhairyaostwal  do you mean updated/added count wrt the last commit made to that repo?
@puf17640  I'd like to work on it. it that's okay.

@dhairyaostwal  do you mean updated/added count wrt the last commit made to that repo?
@puf17640  I'd like to work on it. it that's okay.

@dhairyaostwal do you mean updated/added count wrt the last commit made to that repo?
@puf17640 I'd like to work on it. if that's okay.

Sure man! Thanks for asking! 👍

@dhairyaostwal do you mean updated/added count wrt the last commit made to that repo?
@puf17640 I'd like to work on it. if that's okay.

Sure man! Thanks for asking! 👍


fix(connector-corda): container image kotlin compilation fails in model

Describe the bug
The corda connector jvm backend's container image build is failing on DockerHub due to a compilation error in the generated model code.
This is likely related to the recent downgrade to the standard version of the openapi generator (from 5.0.0 to 5.0.0-beta2).
To Reproduce
Build the container image on DockerHub or locally and observe it failing.
Expected behavior
Build does not fail.
Logs/Stack traces
Step 5/24 : RUN gradle build
---> Running in f1dbb3aad89c
Welcome to Gradle 6.8!
Here are the highlights of this release:
- Faster Kotlin DSL script compilation
- Vendor selection for Java toolchains
- Convenient execution of tasks in composite builds
- Consistent dependency resolution
For more details see https://docs.gradle.org/6.8/release-notes.html
Starting a Gradle Daemon (subsequent builds will be faster)
> Task :compileKotlin
e: /kotlin-spring/src/main/kotlin/org/hyperledger/cactus/plugin/ledger/connector/corda/server/model/JarFile.kt: (29, 5): This type has a constructor, and thus must be initialized here

> Task :compileKotlin
FAILED

FAILURE:
Build failed with an exception.

* What went wrong:

Execution failed for task ':compileKotlin'.

> Compilation error. See log for more details

* Try:

Run with --stacktrace
 option to get the stack trace. Run with --info or
--debug option to get more log output. Run with
--scan to get full insights.

* Get more help at
https://help.gradle.org

Deprecated Gradle features were used in this build, making it incompatible with Gradle 7.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/6.8/userguide/command_line_interface.html#sec:command_line_warnings

BUILD FAILED in 3m 41s

1 actionable task: 1 executed
Removing intermediate container f1dbb3aad89c
The command '/bin/sh -c gradle build' returned a non-zero code: 1

Screenshots
N/A
Cloud provider or hardware configuration:
N/A
Operating system name, version, build:
N/A
Hyperledger Cactus release version or commit (git rev-parse --short HEAD):
0.4.0
Hyperledger Cactus Plugins/Connectors Used
Corda
Additional context
There's another issue open with globally upgrading the openapi generator to 5.0.0 so that would likely solve this bug as well implicitly.
cc: @takeutak @sfuji822 @hartm @jonathan-m-hamilton @AzaharaC @jordigiam @kikoncuo @jagpreetsinghsasan

I'll be having a look at this!


Add Make Sum Divisible by P

@Arshad561
@abhisheksingh789
@aman247av
add Make Sum Divisible by P

add the problem link at the top of file

Added


Removing special branch for unpinned pipeline

Change Description
Removing special branch for unpinned pipeline
Consensus Changes

[ ] Consensus Changes

API Changes

[ ] API Changes

Documentation Additions

[ ] Documentation Additions

Looks as if the unpinned is pulling the proper image again and it's failing for a legitimate reason (something having to do with fc)


File upload exercise answer should not read contents when listing files

Currently the file contents are read in get_returned_files, update to use get_lexer_for_filename instead and check for side-effects of this change

New version of user exercise answers page no longer uses this method, check if there are other instances where it's used but shouldn't


[Error] following model_kwargs are not used by the model: ['image_sizes'

Describe the issue
Issue: i am using the bellow code, a modified version of cli.py code, to query a local image.
import sys
sys.path.append(os.getcwd()+ "/LLaVA/")
import argparse
import torch

from llava.constants import IMAGE_TOKEN_INDEX, DEFAULT_IMAGE_TOKEN, DEFAULT_IM_START_TOKEN, DEFAULT_IM_END_TOKEN
from llava.conversation import conv_templates, SeparatorStyle
from llava.model.builder import load_pretrained_model
from llava.utils import disable_torch_init
from llava.mm_utils import process_images, tokenizer_image_token, get_model_name_from_path, KeywordsStoppingCriteria

from PIL import Image

import requests
from PIL import Image
from io import BytesIO
from transformers import TextStreamer

def load_image(image_file):
    if image_file.startswith('http://') or image_file.startswith('https://'):
        response = requests.get(image_file)
        image = Image.open(BytesIO(response.content)).convert('RGB')
    else:
        image = Image.open(image_file).convert('RGB')
    return image

model_path="liuhaotian/LLaVA-Lightning-MPT-7B-preview"
model_base=None
load_8bit=False
load_4bit=False
temperature = 0.2
max_new_tokens = 512
image_file = "/localhome/local-vishkumar/gen-ai-app/streams/llava_logo.png"

disable_torch_init()

model_name = get_model_name_from_path(model_path)
tokenizer, model, image_processor, context_len = load_pretrained_model(model_path, model_base, model_name, load_8bit, load_4bit)

if "llama-2" in model_name.lower():
    conv_mode = "llava_llama_2"
elif "mistral" in model_name.lower():
    conv_mode = "mistral_instruct"
elif "v1.6-34b" in model_name.lower():
    conv_mode = "chatml_direct"
elif "v1" in model_name.lower():
    conv_mode = "llava_v1"
elif "mpt" in model_name.lower():
    conv_mode = "mpt"
else:
    conv_mode = "llava_v0"

if conv_mode is not None and conv_mode != conv_mode:
    print('[WARNING] the auto inferred conversation mode is {}, while `--conv-mode` is {}, using {}'.format(conv_mode, conv_mode, conv_mode))
else:
    conv_mode = conv_mode

conv = conv_templates[conv_mode].copy()
if "mpt" in model_name.lower():
    roles = ('user', 'assistant')
else:
    roles = conv.roles

image = load_image(image_file)
image_size = image.size

image_tensor = process_images([image], image_processor, model.config)
if type(image_tensor) is list:
    image_tensor = [image.to(model.device, dtype=torch.float16) for image in image_tensor]
else:
    image_tensor = image_tensor.to(model.device, dtype=torch.float16)

inp = "Describe the image"
print(f"{roles[0]}: {inp}")
print(f"{roles[1]}: ", end="")

if image is not None:
    # first message
    if model.config.mm_use_im_start_end:
        inp = DEFAULT_IM_START_TOKEN + DEFAULT_IMAGE_TOKEN + DEFAULT_IM_END_TOKEN + '\n' + inp
    else:
        inp = DEFAULT_IMAGE_TOKEN + '\n' + inp
    conv.append_message(conv.roles[0], inp)
    image = None
else:
    conv.append_message(conv.roles[0], inp)
conv.append_message(conv.roles[1], None)
prompt = conv.get_prompt()

input_ids = tokenizer_image_token(prompt, tokenizer, IMAGE_TOKEN_INDEX, return_tensors='pt').unsqueeze(0).to(model.device)
stop_str = conv.sep if conv.sep_style != SeparatorStyle.TWO else conv.sep2
keywords = [stop_str]
streamer = TextStreamer(tokenizer, skip_prompt=True, skip_special_tokens=True)

with torch.inference_mode():
    output_ids = model.generate(
        input_ids,
        images=image_tensor,
        image_sizes=[image_size],
        do_sample=True if temperature > 0 else False,
        temperature=temperature,
        max_new_tokens=max_new_tokens,
        streamer=streamer,
        use_cache=True)

outputs = tokenizer.decode(output_ids[0]).strip()
conv.messages[-1][-1] = outputs

print("\n", {"prompt": prompt, "outputs": outputs}, "\n")

Error Message:
user: Describe the image
assistant: Traceback (most recent call last):
  File "/localhome/local-vishkumar/gen-ai-app/src/test.py", line 100, in <module>
    output_ids = model.generate(
  File "/usr/local/lib/python3.10/dist-packages/torch/utils/_contextlib.py", line 115, in decorate_context
    return func(*args, **kwargs)
  File "/localhome/local-vishkumar/.local/lib/python3.10/site-packages/transformers/generation/utils.py", line 1307, in generate
    self._validate_model_kwargs(model_kwargs.copy())
  File "/localhome/local-vishkumar/.local/lib/python3.10/site-packages/transformers/generation/utils.py", line 1122, in _validate_model_kwargs
    raise ValueError(
ValueError: The following `model_kwargs` are not used by the model: ['image_sizes'] (note: typos in the generate arguments will also show up in this list)

Can anyone please tell me why i am getting this issue??

changing the model to model_path="liuhaotian/llava-v1.5-13b" resolved the issue and I'm getting the image description now.
Seems like there is bug in LLaVA-Lightning-MPT-7B-preview model path.

getting this also to llava-hf/llava-v1.6-mistral-7b-hf

getting this error in MPT-7b model too


InfluxQL Time Selection not working with Group By Clause

Steps to reproduce:
I have a measurement ("machine") with two values ("temp", "humidity") and one tag ("batch_no").
How can I get the temperature, humidity and the time at the start of each batch? Ideally I want to be able to select first("time"), but that's not possible. The time is returned anyway, but not as expected:
SELECT first("temp") AS "Temperature", first("humidity") AS "Humidity"
FROM "machine"
WHERE time > now()-1h
GROUP BY "batch_no"
ORDER BY time

Expected behavior:
I expect the following results:

Time
batch_no
Temperature
Humidity

10:00
1
80
39

10:12
2
78
40

10:23
3
81
40

Actual behavior:
I am getting the following results:

Time
batch_no
Temperature
Humidity

10:28
1
80
39

10:28
2
78
40

10:28
3
81
40

The time is the current time and not the time of each group. It works when I select one value only:
SELECT first("temp") AS "Temperature"
FROM "machine"
WHERE time > now()-1h
GROUP BY "batch_no"
ORDER BY time

Time
batch_no
Temperature

10:00
1
80

10:12
2
78

10:23
3
81

I might be missing something, but this behaviour seems strange.
Is it really not possible to get the first time (and the last time) for each group?
Environment info:

System info: Linux 5.4.0-1045-aws aarch64
InfluxDB version: InfluxDB v1.8.4 (git: 1.8 bc8ec4384eed25436d31045f974bf39f3310fa3c)

SELECT *
FROM "machine"
WHERE time > now()-1h
GROUP BY "batch_no"
ORDER BY time asc
limit 1

Perfect, thanks very much!


Update abstractNonProvisionedController (for aws elb)

See https://github.com/cloudsoft/brooklyn-aws-elb/ for an example usage of this. That is not in Brooklyn core because it depends on the aws-sdk, which would pull in > 10MB of additional dependencies.

LGTM.  agree @neykov's comment needs addressing, also the constructors change will have tiny conflict w #11 as you noted, but otherwise good to merge.

Thanks all - changes incorporated, jenkins build confirmed; merging now.


Use unauthenticated images for go-toolset.

The issue resolved by this Pull Request:
Related https://github.com/opendatahub-io/data-science-pipelines-operator/issues/177
Description of your changes:
changing the go-toolset to use the registry that doesn't require authentication same as for the ubi image further down the dockerfile.
Testing instructions
none needed, the images are identical.
Checklist

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

/lgtm
/approve


Added redux logger middleware

I prefer to use this way to monitor my state and I think it would be good candidate for the starter kit since it shows how the action modified the state object.
Allows people to see Redux without getting Redux Dev tools installed.
Demonstrates exactly how to add middleware within compose so should be easier for people to add something like thunk.
Bumped the react-redux version to avoid an npm dependency warning for react 0.14

Since this is a teaching/starting point for react, I recommend updating docs to reflect what the new package is doing. This way even if the pull request does not get approved it can help someone integrate into their project and understand why they may want to integrate it.

Yep, I forgot to update it. Thanks.
@coryhouse If you think something like this needs a larger documentation call out like your 'What about Redux Devtools' section or if it is potentially annoying to people and need to know how to undo it, let me know and I can update the FAQ further.

Hrmm. Why would one want to use this when the Redux dev tools (which are already supported) provide this information and so much more?

It's not going to be a huge value add, and might boil down to just my preference.
I prefer to see this output in my console over using Redux dev tools because it shows me the previous state, the action and the mutated state from the reducer. I find it easier to inspect my state from the console than dev tools.
Obviously you don't get timetravel debugging features of dev tools, but I rarely use that and if I do I still keep dev tools running.
Also, I thought to get dev tools working you still need to install a monitor like the Chrome extension, so I figured the logger would add some value without additional dependencies.

I can totally see how this can be useful, but I'm concerned about the amount of noise it creates within the console. I agree it's a useful tool, but I don't believe it should be enabled by default since I'd only use it in a debugging situation. That's why I prefer continuing to recommend the Redux dev tools instead. I'm concerned logging everything to the console could annoy as many people as it pleases. And since it's trivial to add, I think I'll leave it out for now.
That said, thanks for the PR. :)

fair enough


uninitialized constant ProMotion::TableViewCell::TableViewCellModule (NameError)

Getting issue related to https://github.com/rubymotion/BubbleWrap/issues/337
# Gemfile (with RubyMotion 2.31)
gem "ProMotion", "~> 2.0.0"
gem "bubble-wrap", "~> 1.7.0"

# Rakefile
require 'bubble-wrap/core'

Gives uninitialized constant BubbleWrap::Constants (NameError)
Adding app.detect_dependencies = false to Rakefile causes:
uninitialized constant ProMotion::TableViewCell::TableViewCellModule (NameError)

Is there any way of using these together?

Hi @chrise86, this issue sounds similar to #515. Are you still having this issue with the latest versions of ProMotion and BubbleWrap?

Hi @andrewhavens as mentioned in the other issue, not touched any of this since so closing :+1:


CommonGen data statement

This proposes a first version of a Dataset Sheet for the CommonGen Dataset.
cc @sebastianGehrmann

@sebastianGehrmann here's the updated version following the new template!


How to download outdated versions of Tensorflow in Google Colab?

Click to expand!
Issue Type
Build/Install
Have you reproduced the bug with TF nightly?
Yes
Source
source
Tensorflow Version
tf 2.5
Custom Code
Yes
OS Platform and Distribution
Colab
Mobile device
No response
Python version
3.11
Bazel version
3.7.2
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
I have a project using Mask RCNN, and the project only works with Tensorflow 2.5, but this version is outdated and the pip doesn't support this version anymore, when I'm trying to build on colab using the "Build from Source" documentation,
When I try to use bazel I get an error while trying to build it
!bazel build --config=cuda //tensorflow/tools/pip_package:build_pip_package

Also when I try to use pip
!pip install<EMAIL_ADDRESS>
I get the following error
Looking in indexes: https://pypi.org/simple, https://us-python.pkg.dev/colab-wheels/public/simple/
Collecting<EMAIL_ADDRESS>  Cloning https://github.com/tensorflow/tensorflow.git (to revision r2.5) to /tmp/pip-req-build-f6mzej4m
  Running command git clone --filter=blob:none --quiet https://github.com/tensorflow/tensorflow.git /tmp/pip-req-build-f6mzej4m
  Running command git checkout -b r2.5 --track origin/r2.5
  Switched to a new branch 'r2.5'
  Branch 'r2.5' set up to track remote branch 'r2.5' from 'origin'.
  Resolved https://github.com/tensorflow/tensorflow.git to commit 959e9b2a0c06df945f9fb66bd367af8832ca0d28
ERROR<EMAIL_ADDRESS>does not appear to be a Python project: neither 'setup.py' nor 'pyproject.toml' found.

Standalone code to reproduce the issue
You can reproduce it by using this colab

https://colab.research.google.com/github/Huxwell/caffe-tf-torch-darknet-onnx/blob/main/tensorflow_from_source.ipynb

Relevant log output
No response

Hi @luishbastos ,
Colab recently stopped support for Tensorflow Versions older than 2.8v. You can't download older versions directly with pip install tensorflow==x . The command !pip install<EMAIL_ADDRESS>not even working with TF 2.12 versions which might be due to the way TF repo constructed,since setup.py not located at the root of tensorflow project directory but under sub directories at path here.
Tensorflow team not actively supporting older TF versions and this might be the reason why Colab dropped support for TF<2.8 versions. Nevertheless you may ask the reason for same at googlecolab repo also if required.
However for your case you may use Anaconda environment/jupyter notebook and install TF 2.5 version by pip install tensorflow==2.5.However we are not supporting older versions now and we recommend to use latest versions and we can be helpful in case of any problems with latest versions.
Thanks!


Support for Laravel 8 - Unable to open CountriesSeeder.php

Since Laravel8, seeders have been moved to database/seeders
See here: https://laravel.com/docs/8.x/seeding
However currently the migration creation file tries to create the seeder at:
$seeder_file = $this->laravel->path."/../database/seeds/CountriesSeeder.php";
So this is failing with the error:
  fopen(../database/seeds/CountriesSeeder.php): failed to open stream: No such file or directory

  at vendor/webpatser/laravel-countries/src/commands/MigrationCommand.php:119

Here is a workaround.

Create the file CountriesSeeder.php at database/seeders/ like below:

`<?php
namespace Database\Seeders;
use DB;
use Illuminate\Database\Seeder;
use Webpatser\Countries\Countries;
class CountriesSeeder extends Seeder {
/**
* Run the database seeds.
*
* @return void
*/
public function run() {
//Empty the countries table
DB::table(\Config::get('countries.table_name'))->delete();
    //Get all of the countries
    $countryModel = new Countries();
    $countries = $countryModel->getList();
    foreach ($countries as $countryId => $country){
        DB::table(\Config::get('countries.table_name'))->insert(array(
            'id' => $countryId,
            'capital' => ((isset($country['capital'])) ? $country['capital'] : null),
            'citizenship' => ((isset($country['citizenship'])) ? $country['citizenship'] : null),
            'country_code' => $country['country-code'],
            'currency' => ((isset($country['currency'])) ? $country['currency'] : null),
            'currency_code' => ((isset($country['currency_code'])) ? $country['currency_code'] : null),
            'currency_sub_unit' => ((isset($country['currency_sub_unit'])) ? $country['currency_sub_unit'] : null),
            'currency_decimals' => ((isset($country['currency_decimals'])) ? $country['currency_decimals'] : null),
            'full_name' => ((isset($country['full_name'])) ? $country['full_name'] : null),
            'iso_3166_2' => $country['iso_3166_2'],
            'iso_3166_3' => $country['iso_3166_3'],
            'name' => $country['name'],
            'region_code' => $country['region-code'],
            'sub_region_code' => $country['sub-region-code'],
            'eea' => (bool)$country['eea'],
            'calling_code' => $country['calling_code'],
            'currency_symbol' => ((isset($country['currency_symbol'])) ? $country['currency_symbol'] : null),
            'flag' =>((isset($country['flag'])) ? $country['flag'] : null),
        ));
    }
}

}
`

Now run php artisan db:seed --class=CountriesSeeder

I had the same error. Adding the file worked, but as you can see in the first error, it's referring to the wrong path.
`  fopen(../database/seeds/CountriesSeeder.php): failed to open stream: No such file or directory
at vendor/webpatser/laravel-countries/src/commands/MigrationCommand.php:119`
I went to the file MigrationCommand.php and changed the line with the path from  $seeder_file = $this->laravel->path . "/../database/seeds/CountriesSeeder.php"; to  $seeder_file = $this->laravel->path . "/../database/seeders/CountriesSeeder.php";.
Mind the path in the first one still being called seeds in stead of seeders.

That will fix this issue too. @chadriae
However, changing that will affect other third-party libraries with a seeder in the correct location.
Also, I wouldn't ever recommend changing vendor files directly. Or else they will be overriden by composer.  Plus you should avoid tracking vendor with Git.


RSPEED-270 - Add sqlalchemy db connection

[x] added sqlalchemy
[x] added connection to postgres database based on env variables
[x] changed endpoint to use tsvector search instead of naive keyword search

Codecov Report
Attention: Patch coverage is 61.11111% with 14 lines in your changes missing coverage. Please review.

Project coverage is 51.78%. Comparing base (dd4a78f) to head (2ed850b).

Files with missing lines
Patch %
Lines

app/v1/released/endpoints.py
41.66%
7 Missing :warning:

app/crud.py
37.50%
5 Missing :warning:

app/database.py
75.00%
2 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main       #3       +/-   ##
===========================================
- Coverage   76.53%   51.78%   -24.75%
===========================================
  Files           9       10        +1
  Lines          98      112       +14
===========================================
- Hits           75       58       -17
- Misses         23       54       +31

Flag
Coverage Δ

unittests
51.78% <61.11%> (-24.75%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


TIME and TIMETZ types are reported as Type 1111 (OTHER)

Driver version
<IP_ADDRESS>
Redshift version
PostgreSQL 8.0.2 on i686-pc-linux-gnu, compiled by GCC gcc (GCC) 3.4.2 20041017 (Red Hat 3.4.2-6.fc3), Redshift 1.0.25109
Client Operating System
Windows 10
JAVA/JVM version
17-ea+16-1315
Table schema
CREATE SCHEMA test_time_columns;

CREATE TABLE test_time_columns.Test(
    MyTimestamp TIMESTAMP,
    MyTimestampTz TIMESTAMPTZ,
    MyTime TIME,
    MyTimeTz TIMETZ
);

Problem description
Calling DatabaseMetaData.getColumns() on test_time_columns.Test returns a ResultSet where the DATA_TYPE column lists 1111 for all TIME and TIMETZ columns.

Expected behaviour: getColumns() provides appropriate type ints for these types. 92 for both is one option (matches PostgreSQL driver). 92 for TIME and 2013 for TIMETZ would also seem reasonable. Note that TIMESTAMPTZ columns are marked as 2014 (PostgreSQL reports them as 93).
Actual behaviour: TIME and TIMEZ are reported as 1111 (OTHER)
Error message/stack trace: None
Any other details that can be helpful: This is not the end of the world; our application can add a Redshift-specific workaround to map based on TYPE_NAME in this case. But it would be preferable to have correct type ints. Other applications may require appropriate type ints to operate correctly.

JDBC trace logs

Reproduction code

Thanks for reporting issue. We will take a look and get back to you.

Thanks! FYI: It looks like this Convert data types documentation page states that the Amazon Redshift JDBC driver version 2.0 should report TIME & TIMETZ columns as SQL_TYPE_TIME (92) and TIMESTAMP & TIMESTAMPTZ columns as SQL_TYPE_ TIMESTAMP (93).

We fix the issue and it will show as  92 (TIME) and 2013 (TIMETZ). We will release this fix in <IP_ADDRESS> by the end of the month.

Sounds good to me... thanks for the super fast response! Might want to update that documentation page to match this behavior.


Add a slider to resize video / panel

Purpose
On the live view, the panel on the right has a default size ratio (1/3 of the parent, and so the video on the left takes the 2/3 remaining) and can't be adjusted. This PR aims to resolve this situation by adding a slider between the panel and the video, in order to be able to re-size the panel to fit the desired size ratio of the user.
Furthermore, a button is added to be able to open and close the chat.
Proposal

I think you forget to push some commits because there is nothing to test here.


404-ing

Looks like the path to the CSS file was changed, but Obsidian is expecting obsidian.css, so installing the theme now fails.

fixed!
i made a copy of the theme and renamed it to obsidian.css and it now works in the community themes again.


add new lenguage - terraform

A new language, please have a look and let me know if anything is missing.

I could do with some help.
JS is not my cup of tea and I have stuck on testing.
Please help.

Hi @WinOpsDBA, we are in the process of moving new languages to their own repos. If I made you a repository under the hljs organization, would you be willing to maintain this language?
Here is what a typical repository would look like:
https://github.com/highlightjs/highlightjs-solidity
And then you can publish the language on npm for people to use.

Hi @marcoscaceres, I can do language syntax etc. however I am new to js and will require lots of learning and help ;)
I am up for the challenge. Let's give it a go and will see.

Looks like what you have there is ok so far. All you really need in the repository in the terraform.js file. Testing is, of course, nice to have.
Doing a testing setup should be fairly straight forward. I'll see if I can find time to do it from one of the other languages, and the you can just reuse everything you've already got here.
I'd suggest to start just moving everything over... then we put all the bits together.

Sounds good, please send me details for a repository and advise which one should I use as a reference.

Done: https://github.com/highlightjs/highlight-terraform/invitations

Hi @marcoscaceres I have done my 1st commit when you have a second could you have a look, please?
Could you let me know how to test it etc.
I am on uncharted waters here so any feedback is much appreciated.

Sent :) Have another PR for linting the code, but will send that one after as it depends on the first.


Resolved #141: handle h11.ConnectionClosed

Resolved #141

Coverage decreased (-0.03%) to 58.268% when pulling c64d6be8741512ae9e9671f768560216a0929d94 on handle-h11-closed-#141 into d5cd9a227f2673d9d598cfbc6e41ffa5ae24a050 on master.

Merged, thanks.


[PROBLEM] 43. Multiply Strings

Difficulty
Medium
Problem Description
Given two non-negative integers num1 and num2 represented as strings, return the product of num1 and num2, also represented as a string.
Note: You must not use any built-in BigInteger library or convert the inputs to integer directly.
Example 1:

Input: num1 = "2", num2 = "3"
Output: "6"

Example 2:

Input: num1 = "123", num2 = "456"
Output: "56088"

Constraints:

1 <= num1.length, num2.length <= 200
num1 and num2 consist of digits only.
Both num1 and num2 do not contain any leading zero, except the number 0 itself.

Link
https://leetcode.com/problems/multiply-strings/

Please assign this issue to me @godkingjay


fix: Removed '/' from image url (instructions) of Test in Cat Photo Step 12.

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #51459

Hey @adiyadav123
This PR seems to make similar changes as the existing PR #51470. As such, we are going to close this as a duplicate.
If you feel you have additional changes to expand upon this PR, please feel free to push your commits and request this PR be reopened.
Thanks again! 😊

If you have any questions, feel free to ask questions on the "Contributors" category on our forum or the contributors chat room.


[WIP] Watchlist Feature and Built-in Page (PR 2)

Supercedes https://github.com/beakerbrowser/beaker/pull/1210
Still todo:

[x] Update the error page to prompt to use the watchlist
[ ] URLs that have hostnames don't currently work

I noticed that adding URLs with domain names currently generates an internal error. It's straightforward to solve the error: we need to resolve the domain names first.
However, we're going to need to add code to handle when domain-name resolution fails. For that, we're going to need to periodically attempt to resolve the domain name.

Merged! Thanks @Septimus for your work on this PR.


Use new writeCharacteristic method for API Level >= 33

Thank you for this PR and sorry for the late reply.
I will accept it and refactor slightly before the next release.

@all-contributors please add @td-tomasz-joniec  for code


RCP support added for gp712 and 7015m

Addition of a RCP build for QPG7015M and GP712 Qorvo platforms
Changes in glue code to support RCP properly.
Spinoff from https://github.com/openthread/openthread/pull/6076 to pass cla check

From codecov report:
 27 untracked files changed.
No tracked files changed.

However, this check shows decrease in code coverage and fails.

From codecov report:
 27 untracked files changed.
No tracked files changed.

However, this check shows decrease in code coverage and fails.


Outdated 2017-09-20.pem CAfile?

Hello!
Today we got a new certificate for our endpoints, and sending request from Chrome works just fine. When we tried to send a request from Insomnia the TLS handshake failed with the following trace:
* ALPN, offering h2
* ALPN, offering http/1.1
* successfully set certificate verify locations:
*   CAfile: /tmp/insomnia_2020.5.2/2017-09-20.pem
*   CApath: none
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
* TLSv1.3 (IN), TLS handshake, Server hello (2):
* TLSv1.3 (IN), TLS handshake, Encrypted Extensions (8):
* TLSv1.3 (IN), TLS handshake, Certificate (11):
* TLSv1.3 (OUT), TLS alert, unknown CA (560):
* SSL certificate problem: unable to get local issuer certificate
* Closing connection 7

This is the certificate shown in the Chrome certificate dialog:

One thing that seems odd is the use of the 2017-09-20.pem CAfile, which looks like very outdated.
If this is indeed the issue, any plans to update this CAfile?

Yes, it's currently slated for the next release 2020.6


SUBMARINE-793. Drop old Ranger version support

What is this PR for?
More recently, the precommit takes hours to complete, a lot of it is spent on running UT for different combination of Hadoop, Spark and Ranger.
We currently run against Ranger 1.0, 1.1, 1.2 and 2.0. I propose we retain 1.2 and 2.0 and drop 1.0 and 1.1. Dropping these two will save around 40 minutes per precommit (out of more than 3 hours).
What type of PR is it?
[Improvement]
Todos

[ ] - Task

What is the Jira issue?
https://issues.apache.org/jira/projects/SUBMARINE/issues/SUBMARINE-793
How should this be tested?
https://travis-ci.org/github/aeioulisa/submarine/builds/767357154
Screenshots (if appropriate)
Questions:

Do the license files need updating? Yes/No
Are there breaking changes for older versions? Yes/No
Does this need new documentation? Yes/No

LGTM @yaooqinn thoughts?

SGTM


lineHeight doesn't work in TextInput

For example when I set lineHeight style for TextInput it has no effect on app (and no warning is being shown aswell).

To add to @mikollo's issue, I've created an example on RNP
https://rnplay.org/apps/F75ppA
Here's the code...
var SampleApp = React.createClass({

  render: function() {
    var loremText = "Lorem ipsum dolor sit amet, consectetur adipiscing elit. Quisque quis lacinia sapien, in bibendum est. "

    return (
      <View style={styles.container}>
        <Text style={styles.lorem}>
          {loremText}
        </Text>

        <TextInput style={[styles.lorem,styles.input]} multiline="true" value={loremText} />
      </View>
    );
  }
});

var styles = StyleSheet.create({
  container: {
    flex: 1,
    paddingTop: 30,
    alignItems: 'center',
    backgroundColor: '#d3c012',
  },
  lorem: {
    textAlign: 'left',
    color: '#000',
    fontSize: 19,
    marginBottom: 5,
    lineHeight: 50,
  },
  input: {
   marginTop: 50,
   padding: 5,
   height: 150,
   borderColor: 'purple',
   borderWidth: 2,
  }
});

AppRegistry.registerComponent('SampleApp', () => SampleApp);

And here's a screenshot...

Some other styling doesn't work for TextInput, so far I encountered:
fontStyle: 'italic' - doesn't do anything,
textAlign: 'center' - shifts it to the right side

Me too, made a post on Product Pains. https://productpains.com/post/react-native/textinput-lineheight-not-respected/

@katrinanova that is being fixed in https://github.com/facebook/react-native/issues/2140 (related PR: https://github.com/facebook/react-native/pull/7186)

Awesome if that this is fixed on master. FYI: It's not working on v0.24.1

I can confirm that this issue is still present in RN 0.30. Setting lineHeight on TextInput changes height of the element itself but doesn't change spacing between lines of text.

@grabbou Your screenshot from April 24th only confirms what @roman01la verified, that lineHeight affects the height of the element, but not the spacing between the lines. Were you able to verify with multiple lines of text as well?

Nope, I've never had such need unfortunately. If you could try adding a broken multi-line demo to UIExplorer inside a PR, that could help fixing it?

@grabbou is that really necessary? I provided a copy/pastable example above with a linked example https://rnplay.org/apps/F75ppA

That rnplay link does not work for me, and the issue is old, and some of these issues appear to be resolved, so I am going to close this. I think folks should feel free to open a new issue if we have a working repro link for a specific still-existent problem.

the rnplay link was using the old way of requiring react native at the top. I've tried converting it to the newer import, but it's still not working. I'm not sure how they're handling these sort of breaking changes and I don't see anyway to change which react-native version to use.
That said, I'm disappointed by the trend to close issues that aren't actually fixed, moving some to product pains. Just because they're old doesn't mean they're no longer a problem. I get that it's tough to keep up with so many issues. Many very useful PRs have gone stale for the same reason. But closing them without actually fixing them or merely moving them to product pains essentially just "sweeps them under the carpet". out of sight, out of mind. They still exist, so someone else will be forced to file a "fresh" issue at some point.
I'd love to see a better effort to actually fix issues. This would probably have to start with a strategy which tackles PRs more aggressively. I believe the community would be more likely to fix the issues if they were confident their fixes would land. Furthermore, I believe the community is more than willing to jump in and help, but someone has to organize the masses so that we're all working in concert with one another. I don't know how to go about this, but I'm confident we have more than enough manpower...which means we're severely lacking in leadership.

I totally agree.
For issues - we closed about 400 issues over the past week and a half, so I think it's inevitable that some will be mistakes and should actually be open. Some kind of get "stuck", like this one is marked as needing a response from the author. In this case it is great that you have an updated repro. Would you mind just filing that as a new issue? - I think it will just be very hard to take action based on anything buried this deeply in this issue.
But overall pull requests should be the highest priority. The lost value from a good pull request going stale is much higher than the lost value from a good issue going stale. From Oct 19 - Oct 28 we've gotten from 210 open PRs to 157, which I would still describe as "not good" but it's a decent step in the right direction. Part of that has been closing stale pull requests but another part has been pinging the right code reviewer, or making sure we provide the simple feedback quickly like getting through CLA and continuous integration problems.
Are you interested in helping out here, perhaps in some community-organizey way? If yes, feel free to email me at lacker at fb dot com.

I'd be happy to open a new issue. That makes sense.
And yes, I'd be happy to help out in anyway I can. I've signed up at codetriage.com, but that only has limited impact. I'd love to be part of something more organized, a united front... rather than a bunch of us running around and independently trying to do something, and oftentimes doing it differently. Much like rowing a boat, we will go farther faster if we all stroke in unison together.
Regarding the PRs, I'd focus on incoming PRs. We should land any new PRs as quickly as humanly possible. It will take much longer to land older PRs since they'll likely require rebasing, which may not be that simple. By landing new PRs quickly, we set ourselves up for success going forward. We'll lose a lot of great work, but overall this should work in our collective favor.
Just my 2 cents. I'll send you an email, and if you agree... I'd be happy to help.

Hey guys, just noticed this thread. I'd love to at least subscribe to any thread regarding React Native and feature requests / bug reports.  Agree the issue experience could be better, let's figure out something that works.

Issue present in 0.45.0

So for those still having this problem - @dmr07, I found this solution worked for me - https://stackoverflow.com/a/37882782/4392314.  Ensure you set paddingVertical: 0

@cjcheshire Thanks a lot. Combining this with lineHeight got me the desired result.

For temp solution, I wrap <Text/> in <TextInput/>.  Make sure the <TextInput/> doesn't specify value props.
<TextInput
  onChangeText={(text) => {this.setState({content: text});}}
  style={{lineHeight: 28}}>
  <Text>{this.state.text}</Text>
</TextInput>

preset in 0.48.0


Possible improvements

@chanks I've just encountered presentation of Thomas Munro from PGConf that lists possible improvements on top of advisory locks. Could you take a look?
https://www.pgcon.org/2016/schedule/attachments/414_queues-pgcon-2016.pdf

I skimmed it quickly but didn't really discover anything I wasn't aware of. SKIP LOCKED is new but I don't think it's an inherently better technique - see #158 for discussion.
Was there something specific in that presentation that you thought was worth using?

Maybe creating NOTIFY channel for each connected worker and make each worker to LISTEN to its channe (and ping random worker if job is available). This could be functionality on top of current mechanism, to prevent unnecessary queries when there are no jobs (most of the time?)

The 1.0 branch does basically that. I haven't had time to work on it recently, though.

It's almost a total rewrite, btw, so don't expect to be able to just start using it without issues.

ok then :)


Change light config. value_template -> state_value_template

Breaking change in HASS v.2022.2. The lights are disovered with this.

Will push this in next release. Many thanks!


what licensing is the github of the code in?

what licensing is the github of the code in?

Hi @hiqsociety , will add that.


Support field help text

It would be great to be able to set help text on a field in migrations. Documentation and a code search suggests this is not currently possible.

It is but not documented properly unfortunately (I'll file a ticket to fix that)
Here's a sample payload which achieves this (the helpText is defined within the settings of the editor interface control definition):
{
  "sys": {
    ...
    },
    "contentType": {
      "sys": {
        ...
      }
    }
  },
  "controls": [
    {
      "fieldId": "slug",
      "widgetId": "slugEditor",
      "settings": {
        "helpText": "Slug used for URLs"
      }
    }
  ]
}

Now that editor interfaces can be changed via the migration CLI, this is the last missing piece for us.  This is our only remaining manual task after running migrations.

This PR adds support for settings including help text and enabling bulk editing:
https://github.com/contentful/contentful-migration/pull/97


pico: generic st7789 rotation config

The values are not exactly the same, but the scan order was previously somewhat inconsistent.
Wrote this patch a few weeks ago then forgot about it...

Ooh, this makes sense! Nice, thanks!

That sound like a "you should merge this" to me. :laughing:
(Should've mentioned that I tested this on three displays with different default rotations)


Skipping frames while resizing a transparent window

Describe the bug
Skipping frames while resizing a transparant window
Affected platforms
Select one of the platforms below:

Desktop

Versions

Kotlin version*: 1.9.21
Compose Multiplatform version*: 1.5.11
OS version(s)* (required for Desktop and iOS issues): Arch Linux 6.7.0-arch3-1
OS architecture (x86 or arm64): x86-64
JDK (for desktop issues): 17.0.10 temurin

To Reproduce
import androidx.compose.foundation.layout.fillMaxSize
import androidx.compose.material.Surface
import androidx.compose.ui.Modifier
import androidx.compose.ui.graphics.Color
import androidx.compose.ui.window.Window
import androidx.compose.ui.window.application

fun main() = application {
    Window(onCloseRequest = ::exitApplication, title = "Window", undecorated = true, transparent = true) {
        Surface(color = Color.Green, modifier = Modifier.fillMaxSize()) { }
    }
}

Expected behavior
Without skipping frames while resizing

Made reproducible sample here: https://github.com/dima-avdeev-jb/issue-4153
Tested on Manjaro Linux, and found some janks while resizing.
@GazimSoliev Can you please recoed a video for better understanding?

I tried recorded, screenrecorder can't catch that cuz of bitrate, but in reality compose skips 1-2 frames while resizing that makes window invisible, espaseally hard UI window

and yeah, it's so glitch

On my Manjaro Linux I haven't encountered any problems with the invisible window.

On my Manjaro Linux I haven't encountered any problems with the invisible window.

I noticed this in XORG session, it doesn't exist in Wayland

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Add Firefox Add-on mention

The Add-on seems to also use the domains array.

Probably better to link to the addon's GitHub page to match the Chrome extension link: https://github.com/jdm-contrib/justdelete.me-firefox-addon

Probably better to link to the addon's GitHub page to match the Chrome extension link: https://github.com/jdm-contrib/justdelete.me-firefox-addon

Yep, agreed.


chore(main): release 1.2.3

:robot: I have created a release beep boop
1.2.3 (2023-01-09)
update

更新订单列表组件 (3b19e22)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/School-of-Website-Engineering/Back-end-management-system-E-commerce/releases/tag/v1.2.3 :sunflower:


Updating Profile - Saving information

The only things that are allowing me to save are:
1 - Profile Picture
2 - Gender
3 - Name
4 - Business Name
5 - About me information.
Everything else goes away.

My top priority is getting the rest of the update profile functional. Then will focus on more of the UI and adding actual values to the input list

Business info now updates. Will be available on next deploy


Templating and deterministic maps

In yesterday's meeting, we discussed the widespread desire for this spec to handle deterministic, evenly spaced-out chunks within a file, rather than explicitly enumerating each chunk and its offsets. We would need some sort of support for templates and symbolic expressions. For a 1D file, it could be something like
"{i}", ["s3://bucket/path/file.nc", "3200 * {i}", 3200]

or
"{i}", ["s3://bucket/path/file.nc", "{itemsize} * {i}", 3200]

where {itemsize} would get filled with 3200
For ND it would be harder because you need to know the array shape:
"{j}.{i}", ["s3://bucket/path/file.nc", "{itemsize} * ({j} * Nx + {i})", 3200]

I can't think of a clever way around that.
Ideas? @manzt? @joshmoore?

In sketching this out, we may also want to consider:

key names that are regex, which is a well-defined thing
expressions are jinja-like, including support for functions
expressions are simple, like "itemsize * i" (actually, if we know itemsize, I would replace it before saving)

I wonder, is there an overlap with the discussion about coordinates specified programatically in xarray?
To what extent does this the putative option of binary storage for offsets (i.e., not json, perhaps zarr itself) mitigate the problem, due to efficient compression of integers?
Sorry if that's too many thoughts for the limits of this issue.

Given the complexity, we might also consider punting on this and releasing an initial spec that requires explicit keys.

A spec can be added to easier than removed from ...

Martin suggested my adding feedback I received from the @bioformats team here. I asked in a general way if there were anything needed for the spec to maximize the number of formats that we could support. There's hope that some files for some of the proprietary formats will be readable, but just taking the most prevalent (TIFF), there are a large number of edge cases that would require a callback function of some form. So a fourth argument:
(name, templated-size, templated-offset, processing-function)

since data that is interleaved, reversed, sample packed, etc. might need reversing, striding, etc. Details can be found in TiffParser.getSamples. Obviously, this is in no way a MUST (all for KISS) but it would be interesting to hear if a minimal library of such functions would help in other domains as well.
cc: @melissalinkert @dgault @sbesson @cgohlke @manzt


Feature request: expose the setting for notification icon colour on Android

I need to set the colour for the notification icon and I noticed it is not exposed in this package. Is it possible to add that in?

Hi @yixiaoyx,
Thanks for reaching out. We'd be glad to assist with your inquiry.
Please consult this article for a solution:
https://support.pushy.me/hc/en-us/articles/360044212951-How-can-I-customize-the-Android-notification-on-Flutter-
Please let me know if there's anything else we can help with.

Hi @yixiaoyx,
Thanks for reaching out. We'd be glad to assist with your inquiry.
Please consult this article for a solution:
https://support.pushy.me/hc/en-us/articles/360044212951-How-can-I-customize-the-Android-notification-on-Flutter-
Please let me know if there's anything else we can help with.


Rustup for rust-lang/rust#84401

cc rust-lang/rust#84741

:pushpin: Commit b30c5fef5b3654ada90856067240e7b451b8b973 has been approved by RalfJung

:hourglass: Testing commit b30c5fef5b3654ada90856067240e7b451b8b973 with merge 41f3fe64317a6ef144d2ac33e4e5870d894d6038...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 41f3fe64317a6ef144d2ac33e4e5870d894d6038 to master...


Use the environment variable GIDIR for installation

This saves you from having to rewrite ext/grruby/extconf.rb.

fix GIDIR -> GRDIR


Small fixes and improvements in models.py

I was testing my migration script and found some bugs in representations (calling non-existing attributes)

@lachmanfrantisek I had not noticed that, sorry

@lachmanfrantisek I had not noticed that, sorry

No need to sorry, I discovered that during the work on some fix and haven't created an issue. The simplification of create* methods should be still valid. I hope it will be merged soon so you can rebase.


Undefined offset: 2

When downloading certain emails "in vendor/eden/mail/src/Imap.php at line 1371".
Here's what imap pushes to Eden
array('header' => '', 'headers' => object(stdClass), 'tos' => array(), 'ccs' => array(), 'bccs' => array(), 'ID' => array())
Annoying since i cannot download whole mailbox that contains less than 90 mails.

function imap_rfc822_parse_headers($header)

in imap.php
If htmlentities (line 1367) shits itself on, for example, mails that were denied by x-spam all hell breaks loose.

@wolf3r Can you please provide a screenshot of your error?

is there any way to minimize it??


Fix array blocks, add new for loop blocks

Blockly drop:

Fixes issue with losing shadow blocks in arrays. (https://github.com/Microsoft/pxt-blockly/pull/102)
Fixes alignment issue of flyout buttons in Edge and Firefox: (https://github.com/Microsoft/pxt-blockly/pull/101)
Add data attribute for use in comment decompilation on the workspace comment block (https://github.com/Microsoft/pxt-blockly/pull/103)
Duplicate on drag support: https://github.com/Microsoft/pxt-blockly/pull/104

New array blocks:

adding a variables setter to make it easier to create a new list
once you reach more than 5 items, the list becomes vertical.
adding a new item to the list adds a shadow block of type number or string depending on index 0's type. Only supports number and string, otherwise empty.

New for iterator blocks:

deprecating old blocks
fixes issues around deleting field variables that delete the entire block (https://github.com/Microsoft/pxt/issues/4106)
allows dragging a new variable getter just by dragging the variable out of the block (rather than having to get it from the flyout)
potential to use this feature further in custom blocks

Checked they decompile in docs :)


[BUG] kube-vip requires additional RBAC

Describe the bug

I've provisioned a new cluster with Rancher 2.8.3 in Harvester 1.3.0, v1.28.8+rke2r1 added Deployment and LoadBalancer service.
I've also applied patch for kube-vip to have replicas in all nodes and add svc_election: true environment variable. This did not change the outcome as I've tested later on.
When service is created, kube-vip logs look like this:
time="2024-04-24T00:14:03Z" level=info msg="Starting kube-vip.io [v0.6.0]"
time="2024-04-24T00:14:03Z" level=info msg="namespace [kube-system], Mode: [ARP], Features(s): Control Plane:[false], Services:[true]time="2024-04-24T00:14:03Z" level=info msg="No interface is specified for VIP in config, auto-detecting default Interface"
time="2024-04-24T00:14:03Z" level=info msg="prometheus HTTP server started"
time="2024-04-24T00:14:03Z" level=info msg="kube-vip will bind to interface [enp1s0]"
time="2024-04-24T00:14:03Z" level=info msg="Starting Kube-vip Manager with the ARP engine"
time="2024-04-24T00:14:03Z" level=info msg="beginning watching services, leaderelection will happen for every service"
time="2024-04-24T00:14:03Z" level=info msg="starting services watcher for all namespaces"
time="2024-04-24T00:14:03Z" level=info msg="[endpoint] watching for service [echo] in namespace [test]"
E0424 00:14:03.799663       1 retrywatcher.go:130] "Watch failed" err="unknown (get endpoints)"
E0424 00:14:04.800262       1 retrywatcher.go:130] "Watch failed" err="unknown (get endpoints)"
E0424 00:14:05.800789       1 retrywatcher.go:130] "Watch failed" err="unknown (get endpoints)"
E0424 00:14:06.801337       1 retrywatcher.go:130] "Watch failed" err="unknown (get endpoints)"
E0424 00:14:07.802185       1 retrywatcher.go:130] "Watch failed" err="unknown (get endpoints)"
E0424 00:14:08.803655       1 retrywatcher.go:130] "Watch failed" err="unknown (get endpoints)"
E0424 00:14:09.803082       1 retrywatcher.go:130] "Watch failed" err="unknown (get endpoints)"
E0424 00:14:10.803704       1 retrywatcher.go:130] "Watch failed" err="unknown (get endpoints)"
E0424 00:14:11.804210       1 retrywatcher.go:130] "Watch failed" err="unknown (get endpoints)"
E0424 00:14:12.804512       1 retrywatcher.go:130] "Watch failed" err="unknown (get endpoints)"
E0424 00:14:13.804255       1 retrywatcher.go:130] "Watch failed" err="unknown (get endpoints)"
E0424 00:14:14.804677       1 retrywatcher.go:130] "Watch failed" err="unknown (get endpoints)"

To Reproduce
Steps to reproduce the behavior:

Update harvester-cloud-provider with Rancher as defined here:
https://github.com/harvester/harvester/issues/4678#issuecomment-2068690105

Bootstrap FluxCD with rbac-missing branch:
https://github.com/DovydasNavickas/harvester-kube-vip/tree/rbac-missing

NB: I tried without the kube-vip patches in the repo and the result is the same.
I've also added IPPool in Harvester for kube-vip to be able to get external IP from it:
apiVersion: loadbalancer.harvesterhci.io/v1beta1
kind: IPPool
metadata:
  labels:
    loadbalancer.harvesterhci.io/global-ip-pool: "false"
  name: kube-vip-test
spec:
  ranges:
    - subnet: <IP_ADDRESS>/16
      gateway: <IP_ADDRESS>
      rangeStart: <IP_ADDRESS>
      rangeEnd: <IP_ADDRESS>
  selector:
    network: infrastructure/80-dc
    scope:
      - guestCluster: kube-vip-test
        namespace: test
        project: "*"

Expected behavior

No RBAC errors would be present.
Support bundle

I don't think it helps a lot here, because the problem seems to be inside the Guest Kubernetes Cluster itself. I can generate it if it helps.
Environment

Harvester ISO version: 1.3.0
Rancher: 2.8.3
Kubernetes: v1.28.8+rke2r1
Underlying Infrastructure:
Baremetal Supermicro

Adding these RBAC resources:
apiVersion: v1
kind: ServiceAccount
metadata:
  name: kube-vip
  namespace: kube-system
---
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRole
metadata:
  annotations:
    rbac.authorization.kubernetes.io/autoupdate: "true"
  name: system:kube-vip-role
rules:
  - apiGroups: [""]
    resources: ["services/status"]
    verbs: ["update"]
  - apiGroups: [""]
    resources: ["services", "endpoints"]
    verbs: ["list", "get", "watch", "update"]
  - apiGroups: [""]
    resources: ["nodes"]
    verbs: ["list", "get", "watch", "update", "patch"]
  - apiGroups: ["coordination.k8s.io"]
    resources: ["leases"]
    verbs: ["list", "get", "watch", "update", "create"]
  - apiGroups: ["discovery.k8s.io"]
    resources: ["endpointslices"]
    verbs: ["list", "get", "watch", "update"]
---
kind: ClusterRoleBinding
apiVersion: rbac.authorization.k8s.io/v1
metadata:
  name: system:kube-vip-binding
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: system:kube-vip-role
subjects:
  - kind: ServiceAccount
    name: kube-vip
    namespace: kube-system

The errors go away instantly:

cc @w13915984028 @starbops


Google closure compiler doesn't have --verbose.  This means that it simpl

Google closure compiler doesn't have --verbose.  This means that it simply prints "--verbose" is not a valid option and doesn't write the result to standard out.

Will merge it soon, thanks.

Merged in 8c16870. Thanks again.


ORC-1190: Fix Writer benchmark initialization

What changes were proposed in this pull request?
Fix writer benchmark initialization
How was this patch tested?
This does not affect main ORC or tests, only the benchmark suite. I tested by packaging and running java -jar hive/target/orc-benchmarks-hive-1.9.0-SNAPSHOT-uber.jar read-all data, observing that it no longer failed.

@dongjoon-hyun I agree we should have an early exception if the intended benchmark is misconfigured, but shouldn't we be able to run a benchmark individually as long as it is correctly configured?
Right now I can't run any of the benchmarks - you can verify this behavior. For instance, trying to run the "read-all" benchmark fails simply because we can't initialize the irrelevant writer one to check if its name is equal to "read-all".

I agree with your user perspective. However, I prefer to push the community fix it immediately because we are the developers. :)

BTW, FYI, we are trying to add a test coverage for benchmark. Here is the AS-IS progress. We can add your bug report here.
https://github.com/apache/orc/blob/dc2d86487e6e4479e8d674e64a851dd1a8da74aa/java/CMakeLists.txt#L79-L92

@dongjoon-hyun Do you prefer this solution?

@dongjoon-hyun done.

Thank you for updating. Let me verify manually in this evening.
Since we don't have a test coverage in our CI, this should be tested manually.

BTW, it would be better if you narrow down more into dumpDir. The current title is too broad. :)
Fix Writer benchmark initialization

BTW, cc @autumnust and @pgaref , too.

Merged to main/1.8/1.7.


Map MySQL JSON to Presto JSON

Implements https://github.com/prestodb/presto/issues/12806
TestPostgreSqlTypeMapping has this test "\"text with \\\" quotations and ' apostrophes\"", but mysql's escape logic is little different from postgresql and presto. Then, I changed the test case to "\"text with ' apostrophes\""
presto> select CAST('"text with \" quotations and '' apostrophes"' AS JSON);
                       _col0
---------------------------------------------------
 "\"text with \\\" quotations and ' apostrophes\""

postgresql> select CAST('"text with \" quotations and '' apostrophes"' AS JSON);
                    json
---------------------------------------------
 "text with \" quotations and ' apostrophes"

mysql> select CAST('"text with \" quotations and '' apostrophes"' AS JSON);
ERROR 3141 (22032): Invalid JSON text in argument 1 to function cast_as_json: "The document root must not follow by other values." at position 13.

mysql> select CAST('"text with \\" quotations and '' apostrophes"' AS JSON);
+---------------------------------------------------------------+
| CAST('"text with \\" quotations and '' apostrophes"' AS JSON) |
+---------------------------------------------------------------+
| "text with \" quotations and ' apostrophes"                   |
+---------------------------------------------------------------+

The travis failed by No output has been received in the last 10m0s

Merged, thanks!


undefined reference to `pread' (IDFGH-1211)

esp-idf a20d02b7f
trying to use pread function from esp-idf/components/newlib/include/sys/unistd.h yields a link error:
undefined reference to `pread'

regular read function works fine

Thanks @olegantonyan. Yes, the implementation of pread() is missing from or virtual filesystem layer. We will implement it.


Nano Every (ATMega4809) + Due (SAM3) support and some HAL restructuring

Hi Nathan,
Here's my monstrosity. I'm not convinced that my architecture changes would make it better, so maybe it should be a separate thing, but I really don't want to deal with the package management, so here we are.

Nano Every was working before too,
I cleaned up the Due code. The basic idea is that it doesn't have to be low level anymore as the MCU has enough capacity to power through heavier stuff, thus the relevant functions are just wrappers around the factory Serial and SerialUSB library functions. The one magic trick used is that via a macro gcc demotes the ISR routines to regular functions which will be called whenever Camino wants to read or write, or if there's a buffer overflow, on the next main loop cycle.
Camino.cpp went back being a cpp (without any significant mods), and I added an arch.cpp too. There is still code in arch.h, and may not be a lot more readable, but at least not inside a preprocessor macro.
The fundamental problem is that the Arduino environment doesn't allow the library and the user code to mingle at compile time. One solution is, as you did it, to put everything in the user code by including it in the header files. This here instead leaves as much as possible in the library part, just implements minimum necessary snippets in the headers imported from the user code and relies on extern declarations in the library to pick up user configured code during linking.
On the AVR versions there should be one or two extra clock cycles tops overhead.
E.g. the ISR routines take an extra instruction due to the aliasing method, or instead of #defining the buffer array size, the whole buffer is defined in the user header, and linked together as an extern variable reference.
The hearder files also check if they were included from library or user code. It could be split, but this is more compact.
I renamed the PORT macro as it had a conflict with a factory library used for the SAM3.

I didn't touch the original AVR arch code as I can't test it. I ordered an Arduino UNO and a Mega 2560. Considering how deep I am in the rabbit hole with this I might as well complete it.

Now I feel a bit better about this. Using the UNO R3 and the Mega2560 I reworked the sections for the older devices too. I also separated the arch.h into arch.h + arch_user.h. Latter contains definitions influenced by user code macros.
acrh.cpp, arch.h, camino.cpp, camino.h, options.h are pure library code.
Test.ino + tesy.py passes on the:
Nano Every - USB Port
DUE - Native USB Port
DUE - Programming USB Port
UNO R3 - USB Port
Mega 2560 R3 - USB Port
(I didn't try any of the TTL serial pins, but theoretically they should work.)
For the older AVR devices I changed the deprecated macros to the new stile. E.g. sbi, cli
This way "wiring_private.h" doesn't have to be included.

Thanks, lots of improvements here. I'll have time to review it in detail this weekend.
At first glance, it's a little hard for me to follow all the externs/files.
I'm wondering---what's the justification to bring back .cpp files? I know that header-only libraries are bad practice, but I thought the downsides only surface in larger projects. It seems like a header-only library would make this a lot cleaner

Let me start with that I'm not an expert by any means. I'm an engineer not a CS person.
The header only way isn't necessarily bad, I guess what I disliked the most is having to put significant chunks of code into macros. It would be entirely possible to declare functions for every variable piece and just call the right one picked by an #ifdef tree.
And at that point if you have to have the small sections as functions, it might well declare them extern and implement them in a different file.
The value in the library is obviously the comms part, with packet handling, callable management, timeouts etc. I wanted to keep that as "normal" as possible. In a sense of "hey here's this awesome library: Camino.cpp + Camino.h" this is what 99% of people care about and it looks feels smells quacks like some normal vanilla c++ library. "and then there those other files included, it's because the Arduino environment is ... special, but this way the library still provides customization options."
This gets philosophical at this point, and it's your code.
Maybe in order to drop one file, arch_user.h could be merged into options.h, that way one can say that everything that is related to user parameters is in one header file and the naming is less confusing also.
IDK.

I did those changes. Maybe a bit better.
Moved arch_user into options.h
Also got rid of some of the cross includes and associated checks between camino.h and arch.h
One thins I don't know what to do with is the macro IN_CAMINO_LIBRARY that's used to tell whether camino.h was included from camino.cpp or from the sketch. If latter, camino.h will will include options.h
I wish there was a preprocessor directive to easily tell apart the two cases without a prior #define

I am also no C expert, thanks for bearing with me (:
I think you're right to split into the user-included Camino.h header, and actual cpp library. My main gripe is also with the IN_CAMINO_LIBRARY macro, which stems from reusing the same Camino.h in the cpp implementation. Here's my idea:
The camino library Camino_lib.cpp includes its own header Camino_lib.h. This header file declares all options-driven variables/functions extern.
A user #defines any options, and then #includes "Camino.h". This Camino.h contains the implementations for all the extern variables/functions in Camino_lib.h. That's mapping the ARDUINO_xxx/PORT to the correct InitPort implementation; setting the byte packetDataArray to the right size; etc., as you've already done.
So we'd have three files total: external Camino.h; internal Camino_lib.h/Camino_lib.cpp
Do you think this would work? I'm unfamiliar with ISR_ALIAS and if it works with this scheme

Yep, that sounds great. Merging the library cpps and hs will make some of the back and forth includes less confusing. I've been thinking more about this and I think if there was a proper way somebody would've done it by now. I played around with the thought of defining function templates in the library and letting the compiler create every possible version (e.g. for all serial ports) then by only picking one from the library the linker would throw out the rest. Then I realized this wouldn't work with buffer sizes and the other options.
I also think I made a mistake in my last commit. I hastily merged stuff into options.h and didn't test with everything after. (classic mistake) This might've made the ISR_ALIAS more confusing, because I somehow left out an explicit declaration of the vector functions being extern to the user code.
To address your concern that should work just fine.
In camino.cpp I replaced ISR(USART_UDRE_vect) with ISR(Camino_ByteSent_vect).
On platforms where the ISR macro is not defined (ARM) it will simply turn into a regular function: void Camino_ByteSent_vect()
On AVR it will still create an ISR routine, but it will put it at the end of the vector table with no actual interrupts belonging to it.
ISR_ALIAS is normally used to allow multiple interrupts being handled by the same routine. Let's say you do ISR(USART0_UDRE_vect) then use ISR_ALIAS(USART0_UDRE_vect, USART0_RX_vect) to tell the build chain, oh btw call the same function on this other interrupt too, instead of putting a new entry into the vector table.
In my case the difference is that the first interrupt is not real thus could never happen, only the one I aliased to it.
Maybe a less confusing possibility would be to just make a simple function in camino_lib.cpp: Camino_ByteSent
then entirely define the ISR in camino.h as
extern void Camino_ByteSent();
ISR(USARTx_UDRE_vect){Camino_ByteSent_vect();}


added template prototyping route and template

This creates a route ("/template_prototyping/") and template (/app/templates/template_prototyping.html) not linked from anywhere else, that can be used for prototyping different interface elements without touching the other templates. This is a place where we can work out some of the details of an template include fragment, or form fields, etc.
Here is the code for the route:
# TEMPLATE PROTOTYPING
# This is a dev-only route for prototyping fragments of other templates without touching
# them. The url should not be linked anywhere, and ideally it should be not be
# accessible in the deployed version.
@app.route('/template_prototyping/')
def template_prototyping():
    return render_template('template_prototyping.html')

And the entire template as it currently stands for this merge:
{% extends "base.html" %}
{% block content %}

Template Prototyping Page

Here you will find sketches of interface components

{% endblock %}

Fine by me - considering all of the routes I've made have been prototypes :wink:

Just to be clear on the purpose of this, this is where I was hoping to test out some different patterns for conditional field display.


stream_list: Use delegate instead of default for tippy.

Followup to https://github.com/zulip/zulip/pull/32285#discussion_r1837276700

Merged, thanks @evykassirer and @amanagr!


Write page layout pattern guidelines

User Story

As a [user role below]:
user of ibm.com

I need to:
As an adopter I need to see examples of good layout design

so that I can:
create a strong user flow and page narrative

https://ibm.box.com/s/iqapt5viffo3ha69nh2i5bwx5wbj9tr2
Tasks

[ ] Document page layout design decisions and usage guidelines
[ ] Write guidelines for adopters to use
[ ] Collaborate with Jenny

Acceptance criteria

[ ] Guidelines can be added to cupcake website and/or Carbon's site
[ ] The website publishing issue is created

Closing this issue because it is a duplicate of https://github.com/carbon-design-system/ibm-dotcom-library-website/issues/288


Protect against segfaults calling destroyed closures

This commit updates the drop glue generated for closures to simply
ignore null pointers. The drop glue can be called in erroneous
situations such as when a closure is invoked after it's been destroyed.
In these cases we don't want to segfault and/or corrupt memory but
instead let the normal error message from the invoke glue continue to
get propagated.
Closes #1526

But now, there was an other bug:
Error: closure invoked recursively or destroyed already
    at module.exports.__wbindgen_throw (/Users/ling/Codelab/tswasm/rust_assembly/pkg/rust_assembly.js:390:11)
    at wasm_bindgen::throw_str::hda700a983b9b66a8 (wasm-function[121]:5)
    at <dyn +core::ops::function::FnMut<()>+Output = R as wasm_bindgen::closure::WasmClosure>::describe::invoke::h72c772442484abe4 (wasm-function[51]:65)
    at Function.cb (/Users/ling/Codelab/tswasm/rust_assembly/pkg/rust_assembly.js:401:20)
    at listOnTimeout (internal/timers.js:531:17)
    at processTimers (internal/timers.js:475:7)

for issue RuntimeError: memory access out of bounds #1526

@NateLing I believe that is a legitimate bug in that branch (if it's the same as I remember) which is JS calling a destroyed closure.


Improve the SegmentWriter's HashMap

As tantivy's SegmentWriter are using a memory arena, tantivy is not using rust standard library hashmap. The current implementation is using very simplistic linear probing with a djb2 hash.
If possible improve the quality of the code there (without hurting performance), and replace linear probing with something more memory efficient (robin hood hashing?).

It might be more important than I initially thought. See #135.
Performance are degrading extremely fast when indexing a single field with a very high cardinality.

Oops ... Apparently there is no condition over the saturation of the hashmap :|

I'm not sure RobinHood is a good fit when you don't store the hashes (you need the hash of other items on insert/delete and optionally on lookup). But like you said, memory is at a premium.

Good point. What do you think would be the best hashmap impl here?

It should be properly measured, but it seems to me collision are extremely expensive when indexing the movielens dataset. One strategy we could use to mitigate this cost would be to add some info within the table (1 or 2 extra byte of hash for instance) to make sure that handling most collision do not require to jump in memory to compare the actual strings.

We also might want to express the concept of saturation as a collision rate.

taking over this issue

Actually I think we are ok on that one now.


FR: Remove/replace 'space' in {{keywords}} tags: frontmatter yaml citation-plugin

Is Your Feature Request Related to a Problem? Please Describe.
using the obsidian-citation plugin and Better BibLatex export, I would like the frontmatter under the the {{keywords}} (technically {{entry.data.fields.keywords}} ) to output the keywords with spaces as either with an underscore or no space at all,
Describe the Solution You'd Like
Removes spaces from keywords exported from zotero, to either underscores, or no space at all
Please include an example where applicable:
tags: Cannabinoid Receptor Agonists,Cannabinoid Receptor Antagonists,Cannabinoids,Drug Discovery,Endocannabinoids,Humans,Ligands,Piperazines,Protein Binding,Pyrazoles,Receptors,Cannabinoid,Receptors,G-Protein-Coupled

tags: Cannabinoid_Receptor_Agonists,Cannabinoid_Receptor_Antagonists,Cannabinoids,Drug_Discovery,Endocannabinoids,Humans,Ligands,Piperazines,Protein_Binding,Pyrazoles,Receptors,Cannabinoid,Receptors,G-Protein-Coupled

Describe Alternatives You've Considered
Doing it by hand each time :/
Additional Context
Ideally, I would like comma separation of keywords to replaced by spaces...... Or rather than 'underscore's  replacing the 'space's, the spaces are removed, and the comma's are replaced with spaces
tags:  Cannabinoid_Receptor_Agonists Cannabinoid_Receptor_Antagonists Cannabinoids Drug_Discovery Endocannabinoids Humans Ligands Piperazines Protein_Binding Pyrazoles Receptors Cannabinoid Receptors G-Protein-Coupled

OR
Cannabinoid_Receptor_Agonists, Cannabinoid_Receptor_Antagonists, Cannabinoids, Drug_Discovery, Endocannabinoids, Humans, Ligands, Piperazines, Protein_Binding, Pyrazoles, Receptors, Cannabinoid, Receptors, G-Protein-Coupled

@Gornag , would an option to remove/replace spaces in tags when commas and or array syntax is used satisfy what you are looking for? It sounds like that is what is being asked, but I wanted to make sure.

Sorry for the delay.
Thank you for taking the time to reply.
Yes, I believe that would satisfy what I'm looking for!
Thanks :)

+1
This would be very helpful for tags coming in from citation managers like Zotero. Ideally we could select specific YAML fields so that spaces aren't removed from author names.


E-mail subject?

Is their are sample of the E-mail from the Action Group?
I would like to know how the E-mail subject looks like.

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

ID: 2fa0b27d-b18c-22a9-4e7d-cb3a1978338f
Version Independent ID: 58732c0e-494c-ac9b-115a-f4a2482d6cb5
Content: Create and manage action groups in the Azure portal
Content Source: articles/monitoring-and-diagnostics/monitoring-action-groups.md
Service: azure-monitor
GitHub Login: @dkamstra
Microsoft Alias: dukek

@KeiroKamioka0000
Thanks for your feedback! We will investigate and update as appropriate.

#please-close


AssertionError: AWQ kernels could not be loaded.

Hi, I ran your example code and encountered the following error. How can I resolve it?
Even after installing autoawq-kernels, I still get the same error.
AssertionError: AWQ kernels could not be loaded. Please install them from https://github.com/casper-hansen/AutoAWQ_kernels

My env package versions are as following:

python 3.10
transformers 4.37.2
autoawq 0.2.2
autoawq-kernels 0.0.5

Did you try to install the kernels as suggested by the error message? Most likely, an error happened during installation due to your environment not meeting the requirements. Please retry and paste the logs

Thank you for your help! I resolved this problem, so I close this issue.

same promblem, how to solve it @Sangh0

@nguyenhoanganh2002  Refer to this issue #367


Use kubectl in EjsonSecretProvisioner

tl;dr kubeclient isn't working for some of our production clusters, so this switches EjsonSecretProvisioner over to kubectl--which is what the rest of the deploy process uses--for the time being.

When I rolled out https://github.com/Shopify/kubernetes-deploy/pull/81 in shipit, I discovered that kubeclient does not seem to be working with our GCP clusters. It is failing with an SSL error on client.discover (see this deploy). This may be a further piece to the work @xldenis already did in https://github.com/Shopify/kubernetes-deploy/pull/88; he is investigating.
The most meaningful change is that we need to write a tempfile to use kubectl apply for secret creation/update. Note that although the unit tests required surgery, the integration tests all passed as-is.
Why not wait for the kubeclient fix?
The current consensus is still that we want to use a gem rather than kubectl for everything we can (which doesn't include "apply" operations) in the long run, but I didn't realize that none of the tasks that already use kubeclient had been tested in our production Shipit environment. This feature is high-priority, and switching to kubeclient is far from it. With the problems we've had, we may even want to reconsider this particular gem choice.
cc @Shopify/cloudplatform

Updated


Developers should have a way of mocking the API

Is your feature request related to a problem? Please describe.
I'm always frustrated when the API is mandatory to run the project for local development.
Describe the solution you'd like
Developers should have a way of mocking the API if they're only developing the UI.

@fabio-franco I made a mistake opening this issue. It's already implemented: https://github.com/IBM/libras-ui/blob/202dbbc0422fdd5c0505e1b7e3b15e374865b667/src/content/UserPage/UserPage.js#L20

It seems the API mock has problems. Let's reopen this issue.


feat(felte): add event dispatchers

This adds another way to handle custom controls without needing to pass props to lower components by allowing to dispatch "input", "change" and "focusout" events from a custom field.
The proposed documentation for the feature can be seen here

Codecov Report

Merging #32 (72a9626) into main (465c987) will decrease coverage by 0.33%.
The diff coverage is 92.30%.

@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
- Coverage   99.09%   98.76%   -0.34%
==========================================
  Files          34       35       +1
  Lines         995     1049      +54
  Branches      332      346      +14
==========================================
+ Hits          986     1036      +50
- Misses          8       12       +4
  Partials        1        1

Impacted Files
Coverage Δ

packages/felte/src/custom-events.ts
91.89% <91.89%> (ø)

packages/felte/src/helpers.ts
98.49% <92.50%> (-0.42%)
:arrow_down:

packages/felte/src/index.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

This approach does not provide the DX I'd like. I'm closing this in favour for using another alternative. Currently exploring the idea of a createField/useField export.


Build error with GCC9 on aarch64 and armv7

Build error with GCC9 on aarch64 and armv7 (native compilation) with armnn version 19.02.
The error log is:
[  421s] In file included from /home/abuild/rpmbuild/BUILD/armnn-19.02/src/backends/aclCommon/test/CreateWorkloadClNeon.hpp:7,
[  421s]                  from /home/abuild/rpmbuild/BUILD/armnn-19.02/src/backends/neon/test/NeonCreateWorkloadTests.cpp:10:
[  421s] /home/abuild/rpmbuild/BUILD/armnn-19.02/src/armnn/test/CreateWorkload.hpp: In instantiation of 'std::unique_ptr<_Tp> {anonymous}::CreateMergerWorkloadTest(armnn::IWorkloadFactory&, armnn::Graph&, const armnn::TensorShape&, unsigned int) [with MergerWorkload = armnn::NeonMergerWorkload; armnn::DataType DataType = armnn::DataType::Float32]':
[  421s] /home/abuild/rpmbuild/BUILD/armnn-19.02/src/backends/neon/test/NeonCreateWorkloadTests.cpp:627:75:   required from 'void CreateWorkloadNeon::NeonCreateMergerWorkloadTest(std::initializer_list<unsigned int>, unsigned int) [with MergerWorkloadType = armnn::NeonMergerWorkload; armnn::DataType DataType = armnn::DataType::Float32]'
[  421s] /home/abuild/rpmbuild/BUILD/armnn-19.02/src/backends/neon/test/NeonCreateWorkloadTests.cpp:641:97:   required from here
[  421s] /home/abuild/rpmbuild/BUILD/armnn-19.02/src/armnn/test/CreateWorkload.hpp:1095:36: error: moving a local object in a return statement prevents copy elision [-Werror=pessimizing-move]
[  421s]  1095 |     return std::move(workloadMerger);
[  421s]       |                                    ^
[  421s] /home/abuild/rpmbuild/BUILD/armnn-19.02/src/armnn/test/CreateWorkload.hpp:1095:36: note: remove 'std::move' call
[  421s] /home/abuild/rpmbuild/BUILD/armnn-19.02/src/armnn/test/CreateWorkload.hpp: In instantiation of 'std::unique_ptr<_Tp> {anonymous}::CreateMergerWorkloadTest(armnn::IWorkloadFactory&, armnn::Graph&, const armnn::TensorShape&, unsigned int) [with MergerWorkload = armnn::NeonMergerWorkload; armnn::DataType DataType = armnn::DataType::QuantisedAsymm8]':
[  422s] /home/abuild/rpmbuild/BUILD/armnn-19.02/src/backends/neon/test/NeonCreateWorkloadTests.cpp:627:75:   required from 'void CreateWorkloadNeon::NeonCreateMergerWorkloadTest(std::initializer_list<unsigned int>, unsigned int) [with MergerWorkloadType = armnn::NeonMergerWorkload; armnn::DataType DataType = armnn::DataType::QuantisedAsymm8]'
[  422s] /home/abuild/rpmbuild/BUILD/armnn-19.02/src/backends/neon/test/NeonCreateWorkloadTests.cpp:656:105:   required from here
[  422s] /home/abuild/rpmbuild/BUILD/armnn-19.02/src/armnn/test/CreateWorkload.hpp:1095:36: error: moving a local object in a return statement prevents copy elision [-Werror=pessimizing-move]
[  422s] /home/abuild/rpmbuild/BUILD/armnn-19.02/src/armnn/test/CreateWorkload.hpp:1095:36: note: remove 'std::move' call
[  422s] cc1plus: all warnings being treated as errors
[  422s] make[2]: *** [src/backends/neon/test/CMakeFiles/armnnNeonBackendUnitTests.dir/build.make:66: src/backends/neon/test/CMakeFiles/armnnNeonBackendUnitTests.dir/NeonCreateWorkloadTests.cpp.o] Error 1
[  422s] make[2]: Leaving directory '/home/abuild/rpmbuild/BUILD/armnn-19.02/build'
[  422s] make[2]: *** Waiting for unfinished jobs....

Hi,
the issue is that the -Werror=pessimizing-move compilation flag is creating an error, that's not a required flag when building ArmNN, can you turn it off and try again?
Best regards,
Mike

the issue is that the -Werror=pessimizing-move compilation flag is creating an error, that's not a required flag when building ArmNN, can you turn it off and try again?

I would say the compilation flag shows an actual error, so, I would prefer to fix it, instead of trying to ignore it.
What about removing std::move and just return the value?

Good point, I'll create a ticket and add it to our roadmap.

Here is what I tried:
--- armnn-19.02.orig/src/armnn/test/CreateWorkload.hpp	2019-02-28 13:19:24.000000000 +0100
+++ armnn-19.02/src/armnn/test/CreateWorkload.hpp	2019-05-29 17:17:11.912401239 +0200
@@ -1092,7 +1092,7 @@ std::unique_ptr<MergerWorkload> CreateMe
     auto workloadMerger = MakeAndCheckWorkload<MergerWorkload>(*merger, graph, factory);
     BOOST_TEST_CHECKPOINT("created merger workload");

-    return std::move(workloadMerger);
+    return workloadMerger;
 }

 template <typename PreCompiledWorkload, armnn::DataType dataType>

But the error found here: https://github.com/ARM-software/armnn/issues/140#issuecomment-498190719 may be related

Hi,
this build issue should have been fixed by https://review.mlplatform.org/c/ml/armnn/+/1333 I think that it should also have fixed the unit test issue in #140

Fixed in 19.08


Make Storybook deployable

We want to share our custom components displayed as stories in Storybook via URL.
Use Vercel or something similar for fast and non-configurable deployment? Or Azure?

Dependet on #180

We want to share our custom components displayed as stories in Storybook via URL.
Use Vercel or something similar for fast and non-configurable deployment? Or Azure?

I assume that we deploy it as a Docker container on Azure as with the rest of our setup.


fix mutatingwebhook patch problem

when rootCABundle rotates, mutatingwebhook fails to pick up the latest one.  This PR utilize the keycertBundleWatcher to fix the problem.

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

Test name
Commit
Details
Rerun command

release-notes_istio
0a2bbaf91a3faf794d2f8dd757751c72e30d23e4
link
/test release-notes_istio

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

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

Test name
Commit
Details
Rerun command

release-notes_istio
0a2bbaf91a3faf794d2f8dd757751c72e30d23e4
link
/test release-notes_istio

lint_istio
0a2bbaf91a3faf794d2f8dd757751c72e30d23e4
link
/test lint_istio

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

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

Test name
Commit
Details
Rerun command

lint_istio
0a2bbaf91a3faf794d2f8dd757751c72e30d23e4
link
/test lint_istio

release-notes_istio
c28d91f64d1f8fa8fc646d82f3e6c3c28ed0ea5b
link
/test release-notes_istio

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

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

Test name
Commit
Details
Rerun command

release-notes_istio
c28d91f64d1f8fa8fc646d82f3e6c3c28ed0ea5b
link
/test release-notes_istio

gencheck_istio
c28d91f64d1f8fa8fc646d82f3e6c3c28ed0ea5b
link
/test gencheck_istio

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

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

Test name
Commit
Details
Rerun command

gencheck_istio
c28d91f64d1f8fa8fc646d82f3e6c3c28ed0ea5b
link
/test gencheck_istio

release-notes_istio
574c714408362502cdeb125bfdc9d0ccb188b201
link
/test release-notes_istio

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

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

Test name
Commit
Details
Rerun command

release-notes_istio
b81d8dfc785e03861beee49a0b5f0ce042628c65
link
/test release-notes_istio

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

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

Test name
Commit
Details
Rerun command

release-notes_istio
b81d8dfc785e03861beee49a0b5f0ce042628c65
link
/test release-notes_istio

integ-pilot-multicluster-tests_istio
b81d8dfc785e03861beee49a0b5f0ce042628c65
link
/test integ-pilot-multicluster-tests_istio

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

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

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
b81d8dfc785e03861beee49a0b5f0ce042628c65
link
/test integ-pilot-multicluster-tests_istio

release-notes_istio
0d6f1027dc1c3dc430c3224d06b4b10c7a1a002a
link
/test release-notes_istio

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

/test release-notes_istio


add registry option to controller

When passing an OCI url, e.g. oci://docker.io/foo/bar:1.2.3, to launching pods, we already have an option to pass it --registry=local so that it uses the local registry. This has been useful for images that are built locally and not available on the remote registry hub.
This adds the same --registry option to eveimage-update, so we can do the same.

Just tested, does the right thing. Actually, it was really nice to be able to load an image from local docker to local eden registry and then tell eve to pull it for baseos.


Fix placement groups

I'm working with ~20-22 nodes and everytime, I re-created the whole cluster, it failed because of wrong placement_group mappings (it tried to map >10 nodes at one placement_group)
As far as I could calculate, the placement_group logic is starting one ID above 0 (1) and accidentally used the terraform overflow-logic (if n+1 is longer than the array-size, start from 0)
I think I solved it by slightly replace the calculation :-)

I cannot update this PR before tomorrow evening (in about 20hours), sry :-/

No worries @nupplaphil, thanks good, thanks!!

Am awaiting confirmation to merge and release! Just try the above proposed changes with +20 setup please, and confirm all good! :) Whenever you can...

@nupplaphil What I would like you is test the above. With 20-something nodes, the bill can go up fast! But since you use that already... And it's your PR, I though you could test it.

I did a small test, and the floor function works great here, indeed. Thanks for this fix @nupplaphil!


Avoid using the ambiguous \h shorthand character

Depending on the regular expression engine used, \h does not always
mean the same.
Atom uses an Oniguruma engine, but github.com relies on a PCRE
engine.

looks small is big :) thanks!

The main atom grammar that I think everybody uses is here: https://github.com/crucialfelix/atom-supercollider
Could you maybe submit the pr to there ?
Also for these changes you just merged to be used by anybody with language-supercollider installed you have to publish a new release.
I don't even remember if it was me that published this. I think I did and then moved it to the supercollider org.

GitHub only uses this grammar, not the one at https://github.com/crucialfelix/atom-supercollider. If you want to fix the other one, you just have to replace all occurrences of \\h with [0-9A-Fa-f].

Really, they use this one ? This is definitely an atom package. Where exactly does github specify that or have it registered ?
I published 0.2.1 https://atom.io/packages/language-supercollider in any case.
I will do the change on atom-supercollider
Thanks

Really, they use this one ? This is definitely an atom package. Where exactly does github specify that or have it registered ?

There: https://github.com/github/linguist/tree/master/vendor.
GitHub can use Atom, Sublime Text, and TextMate packages.


Document add_row()

I hope that's enough and I follow the code style close enough (code style isn't my thing lol)

Might as well document how to then access the text from the second row


Inline style for Skeleton

https://github.com/yandex-cloud/uikit/blob/da9f256ac9fa521fe616bfa2b1d013068db74a2a/src/components/Skeleton/Skeleton.tsx#L8-L10
Hi, I would like to use skeleton to show preload and I wondered if it could be passed styles prop directly, or maybe width/height props. It seems tiresome to create new classes for each element to stub.
Or maybe there are some best practices on how to use this skeleton

@ierehon1905  Hi! It seems legit. Can you send PR?


Update tool-chaosblade-install-and-uninstall.md

Fix package name

@xujpxm Please sign the DCO as required


Nav Responsive Collapse too aggressive?

Hi:
I am curious about other bootstrap users thoughts on the responsive top nav collapse. Personally I think it is too aggressive and should only collapse when less than 768px.
Bootstrap gods... what was your reason for making this that aggressive? Is there a way to make this configurable without having to hack about the core bootstrap css? Maybe there is a mechanism to only make it collapse if there are too many nav menus? (Would probably require a JS Plugin).
Let me know your thoughts?
Regards:
DigitalJohn

I have a few thoughts about this, but nothing we can address right now. For now, if you want to change that, modify the responsive.less file and rebuild. I made it 980px to start because so many folks just completely fill up their navbar with as much as they can rather than selectively choosing what to add. I'll swing back around to this in 2.1 hopefully.


Cosmos v4 RFC

There are no immediate plans for a v4 Cosmos SDK, but we are starting to collect some changes that would be nice to include in an eventual v4 release:

Remove ClientSideMetrics https://github.com/Azure/azure-sdk-for-js/pull/5381
Remove TimeSpan
Force all queries to use the query plan. Remove the ability to query the gateway directly or expose on another method
Rename some client methods https://github.com/Azure/azure-sdk-for-js/issues/6150
Rename some client options https://github.com/Azure/azure-sdk-for-js/issues/6149
Rewrite retry logic to give better user access. Expose shouldRetry function
Ability to set throttle retry count: https://github.com/Azure/azure-sdk-for-js/issues/9176

Additional suggestions welcome!

Migrate to azure core logging library https://github.com/Azure/azure-sdk-for-js/issues/9250

Suggest throwing instead of allowing requests we already know are bad to fail from auth tokens being null: https://github.com/Azure/azure-sdk-for-js/issues/9511

Drop support for < TS 3.5 https://github.com/Azure/azure-sdk/pull/1805

Investigate removing node-fetch in favor of @azure/core-http https://github.com/Azure/azure-sdk-for-js/issues/11229#issuecomment-701534390

Consider https://github.com/Azure/azure-sdk-for-js/issues/14033.


Create project to host "framework like" classes

Classes like ByteArrayReadStream and BufferRepresentation should not be in GameServer or Network.
Likewise, "resource pools" are so  generic and "framework-like" that we should create a project to host such classes.

Done in #41


feat(Chromium): roll chromium to r513435.

This roll brings in a bunch of important patches:

crrev.com/512647 Changed headless browser profile dir to use Default profile path
crrev.com/512760 DevTools: stop idleness detector when pending navigation commits
crrev.com/512905 DevTools: introduce Page.getFrameTree
crrev.com/513373 DevTools: report loaderId in the lifecycle events
crrev.com/513419 DevTools: introduce Page.setLifecycleEventsEnabled
crrev.com/513422 DevTools: return loaderId from Page.navigate

Fixes #921

Exciting! This is a breaking change for people using headless user data dirs.


NameError: name 'jsondata' is not defined

Upon starting the cheat I get an error that says: "File "C\users\Myusername\Downloads\BlooketHack-master\main.py", line 117, in
for question in jsondata['questions']:
NameError: name 'jsondata' is not defined"
It stops there and nothing happens. If anyone can tell me how to fix this I would love it.

Someone else had the same issue I think, while I haven't been able to conduct real research into the problem. I might have a theory on how to fix it but not sure yet. I'm gonna ask a few questions, please answer them to the best of your ability but know that you don't have to answer if you feel a question is invasive.
How fast is your internet connection?
What python version are you running?
What Blooket game mode where you attempting to run?
What Chrome version are you on?

I would recommend using python 3.7+ as I know it should work for every version after that. There shouldn't be an issue there. Your chrome version matches with the driver so that's good. Crypto worked last time I tried it but I can test again. Administrator shouldn't be an issue at all. I can go through the code later and try to make it work.
Another question, are you using any Chrome extensions that could interfere with the cheat?

I hink I fixed the issue causing this yesterday. Let me know if it works.

how get to hack i dont know how


koa and sqlite3 node REST endpoint returning only Database object instead of data

Having an issue on a koa route where the data is not being returned correctly, instead getting the instance of the Database object.
Requesting data like so from koa route:
db.connection.all('SELECT * FROM meter_reads', [], (error, data) => data);
Unwanted result:
Database { open: true, filename: ':memory:', mode: 65542 }
More information/code here:
https://stackoverflow.com/questions/52518527/koa-and-sql3lite-node-rest-endpoint-returning-only-database-object-instead-of-da

ctx.body = await db.connection.all('SELECT * FROM meter_reads', (err, data) => data)

'all' does not return any value ( it returns the result by calling the given callback) so you are actually awaiting 'db.connection' in this line of code
there are some libraries based on node-sqlite3 providing promise based API and more
e.g
https://www.npmjs.com/package/sqlite
or mine:
https://www.npmjs.com/package/sqlite3orm


Haiku

Hi - is there a command line option to say the thanks/praise to my coffeeshop friends in the form of a haiku?
Thanks in advance.

@Qix- open your text editor and replace text. That's super-easy-twenty-strings-bash-script, you cat do anything what you want :)
For example, playing music:
afplay ~/Music/LMFAO\ -\ Sorry\ For\ Party\ Rocking.mp3 > /dev/null 2>&1

or shutdown your laptop, if you want.

I don't think Vim supports haikus :(


MISO not updated on falling edge of CS when CPHA=1

When using the shift() function MISO data is not shifted on the falling edge of CS. Its only shifted on the first edge SCLK.

The _shift function is a utility function. If you want to read some data on the falling edge of CS, you will need to implement the behavior yourself. Please refer to the slave examples provided to see how this is done.