text:
Platform backlog meta

Objectives
The company OKRs and engineering OKRs outline the following objectives and key results:

Shareable code artifacts

Data to dashboard via files
Rehydrate project via a single CLI command
CI workflow for artifact deployment

High quality experience

Remove Node.js

Migrate data modeller and profiling to APIs
Migrate CLI to Go

Improved error handling
Solid unit and end-to-end testing

Work breakdown
Code artifacts

[ ] #1174
[ ] Design end-to-end API workflow for manipulating and applying file artifacts (incl. invalid state)
[ ] Adapt migration and repo APIs to support several different file artifact representations (mapping up to internal objects)
[ ] Refactor/implement migrations and refresh logic for YAML artifacts

Modeller on APIs

[ ] #1175

CLI

[ ] #1176
[ ] Add ability to serve the web-local SPA
[ ] Add ability to start runtime
[ ] Setup GoReleaser for cli with nightly and/or preview builds

Avoiding new tech debt

[ ] Merge relevant parts of https://github.com/rilldata/rill-developer/pull/1064
[ ] Refactor functionality in server into a "services" layer + expand current tests
[ ] Rename drivers to a more appropriate term (such as persistence, stores, infra)
[ ] Add README in drivers explaining the stores concepts
[ ] Split runtime.proto into multiple modules
[ ] Add request validation using https://github.com/bufbuild/protoc-gen-validate
[ ] Setup e2e server tests using https://github.com/grpc/grpc-go/tree/master/test/bufconn
[ ] Setup pagination on all relevant endpoints in accordance with these guidelines
[ ] Consider running gRPC using https://github.com/bufbuild/connect-go
[ ] Configure Druid driver to not build Docker image on every test
[ ] Submit Druid authentication issue to calcite-avatica-go (regarding https://github.com/begelundmuller/calcite-avatica-go/commit/52480d9968a90893976c582449a56f2fa6c85275)
[ ] Revisit dry runs in OLAP queries – do we need them? If yes, it should check args and return schemas
[ ] Check design of the rill.runtime.Type protobuf message against Apache Arrow's schema type (link)
[ ] Disable SQL in runtime builds (for now)
[ ] Add attribution and tests to package pkg/sharedlibrary (it's loosely adapted from https://github.com/ying32/dylib)
[ ] Setup http://golangci-lint.run

Connectors

[ ] #1179
[ ] Add unified validation across frontend/backend (e.g. using JSON schema)
[ ] Support globs for local, S3 and GCS
[ ] Support NDJSON files
[ ] #1182

Better error messages

[ ] Research current vs. desired error messages from runtime
[ ] Refactor inline errors to error consts
[ ] Move to a complex error message type that can include things like stack traces, lists of errors, human-friendly messages, suggestions, docs links, etc.
[ ] Adapt gRPC error handling to emit extra error details (see these guidelines)

Cloud ready

[ ] runtime: Setup configurable CORS that works for both local and remote deployments
[ ] runtime: Add JWT-based auth inspired by https://docs.temporal.io/server/security
[ ] runtime: Setup standalone Docker image
[ ] control plane: Rename server-cloud to admin
[ ] control plane: Research and MVP authorization
[ ] meta: Design async task architecture for runtime

as I learned, protoc-gen-validate doesn't support Javascript

as I learned, protoc-gen-validate doesn't support Javascript

Yeah, but we don't need it on the client side – it's only needed to validate request parameters on the server.

@begelundmuller I am new to this repo, and after going through the tasks to understand more about this project in depth.
I feel split runtime. proto in multiple modules is a good first issue to understand more about the project.
can I pick the item and go ahead?

Hey @antimpatel, thank you for your interest in contributing – and sorry that I missed your comment. The proto refactor impacts a number of outstanding PRs, so it's something we'd prefer to do on the team. If you're still interested in contributing, I would suggest https://github.com/rilldata/rill-developer/issues/1265 as a good first issue :)

authors: antimpatel begelundmuller egor-ryashin

repo: rilldata/rill-developer


text:
Remove this unnecessary cast to double (Group - 3)

Describe the technical debt
Addressed a sonar issue where the unnecessary cast to double used which redundant.
Location : src/main/java/de/dennisguse/opentracks/data/ContentProviderUtils.java

The redundant cast should not be used.
The potential consequences of not addressing this issue :

Reduced code understanding and code readability
Inconsistent naming
Increased potential for errors or bugs

This needs to be fixed to:

Improve code quality and maintainability
Reduce risk of errors or bugs

Proposed solution
Removed the unnecessary cast to 'double'

Technical information
Device : Pixel_3a
OS : Android 12
OpenTracks version : v3.2.4
OpenTracks commit id : accc200
Commit : Click here

Solved and committed to the branch. Remaining to merge in the main

Solved and committed to the branch. Remaining to merge in the main.

source: gharchive/issue

repo: rilling/OpenTracksConcordia

license_type: permissive

license_source: github-api


text:
hotfix

Hello,
please note: fixed nNum, used only on Windows, no need for declaration globally

Thanks for the update

source: gharchive/pull-request

authors: Ilir-Liburn MahmoudFayed

repo: ring-lang/ring

license: mit

license_source: bigquery


text:
Add mdx_include to the mkdocs installing documentation

We missed the mdx_include package that's needed to run the mkdocs server. so I've added it and joined all pip dependencies installation into a single line command.

@grokify hey, may I ask you for a review?

repo: ringcentral/engage-digital-api-docs

license: MIT


text:
Refactor PlayOrResumeOptions

Since the json attributes of the PlayOrResumeOptions must be disjointed, two new classes have been implemented to keep unnecessary attributes out of the request body. This is the consequence of #202.
Furthermore, in order to test PUT, POST or DELETE http methods, a very rudiment interceptor has been added.

@hayribakici please check the conflicts, then I will merge it :)

I don't see any conflicts here 🤔

What the heck, it told me I couldn't merge because of conflicts. Now it says it's fine on my side, too. Sorry.

authors: hayribakici rinukkusu

repo: rinukkusu/spotify-dart


text:
Membuat RESTful API

Saya ingin membuatkan REST API untuk file-file json ini. Apakah boleh ?

Oke makasih Mas

repo: rioastamal/quran-json


text:
Add fecth data workflow and automate it

Similar to https://github.com/riparias/pov-ias-plants-occurrences/issues/1

Solved by #9

repo: riparias/pov-fishes-occurrences


text:
Mapping of fields in EKC sheet

What do EKC and GEP mean? Answer from mail of Frank:

EKC MAF betekent ecologische kwaliteitscoëffiënt (=indicator) voor macrofyten
Voor kunstmatige en sterk veranderde (natuurlijke) waterlopen is het toegestaan om de beoordeling aan te passen.
Dan wordt een goed ecologisch potentieel bepaald.
Praktisch wordt de indicator bij macrofyten anders bepaald dan bij natuurlijke waterlichamen:
Er wordt rekening gehouden met bepaalde ingrepen voor bepaalde nuttige doelen van het waterlichaam.

Mapping follows. VMM review required.

The columns index GEP, index ZONDER gep and klasse can be mapped in DwC MeaurementOrFact extension. Here below the proposed mapping.
Mapping index GEP
DwC term measurementType: 'ecologic quality coefficient with GEP'
DwC term measurementValue: values in column index GEP
DwC term measurementUnit: NULL
Mapping index ZONDER gep
DwC term measurementType: 'ecologic quality coefficient without GEP'
DwC term measurementValue: values in column index ZONDER gep
DwC term measurementUnit: NULL
Mapping klasse
DwC term measurementType: 'ecologic quality coefficient class'
DwC term measurementValue:

Original values (NL)
proposed values (EN)

SLECHT
poor

ONTOEREIKEND
insufficient

MATIG
fair

GOED
good

GOED EN HOGER
very good

DwC term measurementUnit: NULL
Questions for VMM:

do you agree on the English translations in measurementType?
do you agree on the English translations of klasse values?

Bevestigingscode MF is for judging the EQR (EKC). Is it representative (R) or not (NR).
If NR the EQR are considered unusable and not reported.

Class GOED EN HOGER does not mean very good, but good.
Artificial and heavily modified (natural) waterbodies do not get the class very good (ZEER GOED), being the reason for adding EN HOGER.
ZEER GOED is also possible, for (not or barely modified) natural waterbodies, translating as "very good".

empty=0
Deelmonster Geen Organisme = 1 have been omitted from the data iirc.
Not sure whether that is the right thing to do. Should we or how could we report there is nothing on this place at that time?

informix_vhaw_status

informix_vhaw_status

code
omschrijving

NAT
Natuurlijk

KUNST
Kunstmatig

DEFAU
Nog geen statuut toegekend

This is the Status of the waterbody.
It combines with "sterk veranderd" (or not) for status Natuurlijk.
(Sterk veranderd is another property of the waterbody on which the site is situated, not a measurement nor an evaluation.)
Therefor we have defined a Statuut in our Datawarehouse:

VHA   Waterlichaam Statuut
VHA   Waterlichaam Status Code
VHA   Waterlichaam Status Omschrijving
VHA   Waterlichaam Sterk Veranderd

Default
DEFAU
Nog geen statuut toegekend
0

Default

0

Kunstmatig
KUNST
Kunstmatig
0

Natuurlijk
NAT
Natuurlijk
0

Sterk Veranderd
NAT
Natuurlijk
1

index ZONDER gep? Or index GEP?
The relation with statuut (status + sterk veranderd) is thus:
When the waterbody Status is artificial or the waterbody has been heavily modified, you are allowed to give another assessment. This is index GEP.
[The reference is not anymore the natural ecological conditions but a good ecological potential in the given (unfavorable) conditions.
Possibilities are to change the calculation of the EKC or to change the limits for evaluation or ...
For EKC MAF the calculation changes and the limits for evaluation stay.]

type
status
sterk veranderlijk
owl (= code of waterlichaam)
waterlichaam
are properties of the site (Meetplaats)
They are informative to the index because all this information is needed to calculate 'index zonder gep' (= 'natural' EKC) and if permitted 'index gep'

Translation of EKC is EQR or Ecological Quality Ratio.

authors: damianooldoni flavens-vmm

repo: riparias/vmm-macrophytes-occurrences


text:
"error_code": 17, "error_message": "InsufficientNetworkMod"

request
curl -X GET
http://localhost:5005
-H 'Content-Type: application/json'
-d '{
"method": "account_info",
"params": [
{
"account": "rBG...27b",
"strict": true,
"ledger_index": "current",
"queue": true
}
]
}'
response
{
"result": {
"error": "noNetwork",
"error_code": 17,
"error_message": "InsufficientNetworkMode",
"request": {
"account": "rBG...27b",
"command": "account_info",
"ledger_index": "current",
"queue": true,
"strict": true
},
"status": "error"
}
}
server_info
{
"result": {
"info": {
"build_version": "0.80.0",
"closed_ledger": {
"age": 15,
"base_fee_xrp": 1e-05,
"hash": "13FB...817",
"reserve_base_xrp": 200,
"reserve_inc_xrp": 50,
"seq": 29
},
"complete_ledgers": "empty",
"hostid": "localhost",
"io_latency_ms": 1,
"last_close": {
"converge_time_s": 1.999,
"proposers": 0
},
"load": {
"job_types": [
{
"in_progress": 1,
"job_type": "clientCommand"
}
],
"threads": 8
},
"load_factor": 1,
"network_ledger": "waiting",
"peers": 0,
"pubkey_node": "n9M...4AXg",
"pubkey_validator": "none",
"published_ledger": "none",
"server_state": "full",
"state_accounting": {
"connected": {
"duration_us": "48017447",
"transitions": 1
},
"disconnected": {
"duration_us": "1160612",
"transitions": 1
},
"full": {
"duration_us": "902968886",
"transitions": 1
},
"syncing": {
"duration_us": "0",
"transitions": 0
},
"tracking": {
"duration_us": "0",
"transitions": 0
}
},
"uptime": 952,
"validation_quorum": 5
},
"status": "success"
}
}
the same happens with all request

Noticed that “reserve_base_xrp” says “200”. It’s supposed to be 20.

You server has no peers. Most likely, it's a network connectivity problem but it could also be that it's not configured correctly.

Closing this for now. If you are still having trouble after fixing networking connectivity issues, please open a new request with more details.

I have same problem:
{
"id" : 1,
"result" : {
"error" : "noNetwork",
"error_code" : 17,
"error_message" : "InsufficientNetworkMode",
"request" : {
"account" : "r.....Cn",
"command" : "account_info"
},
"status" : "error"
}
}
server_info:
{
"id" : 1,
"result" : {
"info" : {
"build_version" : "0.80.0",
"closed_ledger" : {
"base_fee_xrp" : 1e-05,
"hash" : "517....9",
"reserve_base_xrp" : 200,
"reserve_inc_xrp" : 50,
"seq" : 10
},
"complete_ledgers" : "empty",
"hostid" : "ip....3",
"io_latency_ms" : 1,
"last_close" : {
"converge_time_s" : 1.999,
"proposers" : 5
},
"load" : {
"job_types" : [
{
"job_type" : "untrustedProposal",
"per_second" : 21
},
{
"in_progress" : 2,
"job_type" : "ledgerData",
"waiting" : 4
},
{
"in_progress" : 1,
"job_type" : "clientCommand"
},
{
"job_type" : "transaction",
"per_second" : 5
},
{
"job_type" : "fetchTxnData",
"per_second" : 6
},
{
"avg_time" : 8,
"job_type" : "trustedValidation",
"peak_time" : 46,
"per_second" : 1
},
{
"job_type" : "writeObjects",
"peak_time" : 4,
"per_second" : 2
},
{
"avg_time" : 3,
"job_type" : "acceptLedger",
"peak_time" : 9
},
{
"job_type" : "trustedProposal",
"per_second" : 1
},
{
"job_type" : "peerCommand",
"per_second" : 346
},
{
"job_type" : "diskAccess",
"peak_time" : 1,
"per_second" : 1
},
{
"avg_time" : 18,
"job_type" : "SyncReadNode",
"peak_time" : 228,
"per_second" : 1
},
{
"job_type" : "AsyncReadNode",
"peak_time" : 45,
"per_second" : 2710
},
{
"job_type" : "WriteNode",
"per_second" : 2
}
],
"threads" : 8
},
"load_factor" : 1,
"network_ledger" : "waiting",
"peers" : 10,
"pubkey_node" : "n9...P",
"pubkey_validator" : "none",
"published_ledger" : "none",
"server_state" : "connected",
"state_accounting" : {
"connected" : {
"duration_us" : "22728764",
"transitions" : 1
},
"disconnected" : {
"duration_us" : "1274750",
"transitions" : 1
},
"full" : {
"duration_us" : "0",
"transitions" : 0
},
"syncing" : {
"duration_us" : "0",
"transitions" : 0
},
"tracking" : {
"duration_us" : "0",
"transitions" : 0
}
},
"uptime" : 24,
"validation_quorum" : 3
},
"status" : "success"
}
}
Restarted rippled and it helped for some time, but now the problem is back again; network and accesses on the server are fine. But there is high CPU usage by rippled process..  Server has 4 Core Xeon and not that much transactions to deal with, so don't know from where is there such a high load.

Based on the output above, I don't see an explanation for why you had high CPU usage and a delay syncing to the network.  Feel free to re-open the issue if you experience a similar problem and we can try to iterate on the root cause.

The problem was that the server could not keep up with ledger synchronizations: rippled started and looped in restart after 1 minute interval, debug log was showing that it was trying to keep up with ledger versions, but could not finish the process; then re-initialization again... Even after upgrading AWS EC2 instance to higher specs - there still was a problem. After that we've deleted the local database and tried to start from scratch. That helped, even downgrading the instance kept the syncs without problems. So now I have lost local ledger history before the time of the last start, but that is way smaller problem then completely stopped service.

What type of EC2 instances are you using? Are you using NuDB backend or RocksDB?

RockDB and instance was t2.xlarge. Changed to c3.large in the end.

Thanks.  Did it ever actually crash? Any ERR messages in the logs?

I see you are ok for now; no need to respond if you don't want to.

I was checking debug log, can't really remember if it had ERR messages, but there were many messages like:
NetworkOPs:WRN We are not running on the consensus ledger
I know that this on is typically normal and we must wait for getting to the consensus, but I was waiting more than an hour + restarted the rippled and tried to wait again with no luck. The old log is rewritten now, so can't really recheck to answer properly.

repo: ripple/rippled

license: ISC


text:
add failure metadata in upgrade tests

Part of the dtest failure audit. Adds metadata to the upgrade_tests module.

+1

Thanks, merging. (I should note that I've run these changes locally and they're fine.

authors: mambocab ptnapoleon

repo: riptano/cassandra-dtest


text:
Transfer records are added to the wrong end of the FIFO?

Chapter 1, "Introduction", says:

CTR defines a circular (FIFO) buffer.

and:

Recorded transfers are inserted to the head of the buffer,

Usually entries are said to be added to a FIFO (a queue) at the tail and removed from the head.  (Think of what it means to be at the head of a queue.)
Presumably, for "RAS emulation mode", recorded transfers are added at the head of the buffer, since that is a stack (LIFO).

Interesting, I have always said it the other way, where the tail pointer is the oldest entry (read pointer) and the head pointer is the newest (or where the next will be written).
On a quick search, I see docs that agree with me:
https://www.kernel.org/doc/html/v6.1/core-api/circular-buffers.html
And with you:
https://www.cs.umb.edu/~kamaral/cs240/lectures/CS240-Lecture-17.pptx#:~:text=Here we include two indexes,the freshest element will go.
And some that, confusingly, seem to flip-flop:
https://www.baeldung.com/cs/circular-buffer
We could opt to avoid confusion by omitting references to head/tail altogether, saying something like:

Recorded transfers are inserted at the write pointer, which is then incremented, while older [...]

Do you think this warrants a new release version?

On a quick search, I see docs that agree with me:

IMHO, some mistakes are common, but still wrong.
While Wikipedia isn't guaranteed to be perfectly authoritative either, nevertheless see:
https://en.wikipedia.org/wiki/FIFO_(computing_and_electronics)
https://en.wikipedia.org/wiki/Queue_(abstract_data_type)
Your suggested fix seems good to me.

Do you think this warrants a new release version?

I don't.  I think this easily counts as an editorial improvement that can be made at any time.

Okay, I'll add the clarification at the conclusion of public review.

authors: bcstrongx jhauser-us

repo: riscv/riscv-control-transfer-records


text:
Update register name order

https://github.com/riscv/riscv-isa-manual/issues/1352

@kersten1 Should we also update all other chapters to make it uniform

@ved-rivos I'm working on it!

@aswaterman This should be all the titles. I'll do another looksie after these are out of my optics :)

repo: riscv/riscv-isa-manual


text:
getchar/putchar interface

I notice that these functions take or result in an int.  Are these interfaces intended to take only 7-bit ASCII, or should they be Unicode codepoints?
The reason I'm asking is because of getchar's use of -1 to indicate failure to retrieve a character.  With a wrapper around getchar, you can use this to implement a function can_getchar, which returns true if and only if there's a character waiting, but which does not pop the character input queue.  However, doing so involves locally caching the retrieved character, which seems like it should be the job of whatever is implementing the SBI.
Maybe three functions are to be preferred?

int getchar() -- retrieves the next console character.  Never blocks; -1 if not possible to grab character.
int can_getchar() -- true if and only if getchar() will yield valid data.
putchar(int) -- sends a character out to the console.

Just an idea.

Never mind; I missed that they are spec'd specifically to send or receive bytes, which implies ASCII only operation.  Use of -1 as a sentinel is safe then.

Byte-based non-ASCII encodings such as UTF-8 should also work.

Mnemonically, then, I'd recommend changing the names of the functions to sbi_console_{get|put}byte instead of sbi_console_{get|put}char, to make the distinction between characters and their encodings more clear.

authors: neuschaefer sam-falvo

repo: riscv/riscv-sbi-doc


text:
Doesn't download videos anymore

It doesn't download videos anymore.  Says something about url can't be downloaded with excuses about private link or something, but it has always worked before and now doesn't work for any videos on youtube, rumble, etc...

Hi @SpiritualCharlesDarwin could you send me video url so that i can check the issue. Thanks!

I just tried this one and it doesn't work:
https://www.youtube.com/watch?v=fWheqQLWrXY&t=424s
--Dennis
On Wed, Jun 19, 2024 at 4:30 AM Rishabh Bhardwaj @.***>
wrote:

Hi @SpiritualCharlesDarwin https://github.com/SpiritualCharlesDarwin
could you send me video url so that i can check the issue. Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/rishabh3354/VIZTUBE/issues/4#issuecomment-2178333796,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUERFISKBUPHTUZ63AKS2ITZIFMS3AVCNFSM6AAAAABJQ4KUKKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZYGMZTGNZZGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

For me it is working fine and i just downloaded with viztube
This might be country restricted video from youtube. Which country are you from? i checked with India

My VPN is set to Las Vegas, and that seems to be the problem.  I turn it
off and then it downloads, as I am in Canada and it is not restricted
here.  When I put the VPN back on, it will stop the download right in the
middle of operation.  Turn it off, and I can download again.  But, this
video is NOT restricted in the USA, as far as I know.  Were you able to
download from USA?
--Dennis
On Wed, Jun 19, 2024 at 5:40 AM Rishabh Bhardwaj @.***>
wrote:

Screenshot.from.2024-06-19.17-08-26.png (view on web)
https://github.com/rishabh3354/VIZTUBE/assets/26843758/a1510998-1fc4-4f88-88f8-fc2f86d8ffe1
For me it is working fine and i just downloaded with viztube
This might be country restricted video from youtube. Which country are you
from? i checked with India
—
Reply to this email directly, view it on GitHub
https://github.com/rishabh3354/VIZTUBE/issues/4#issuecomment-2178484276,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUERFIU3NHYOB5EHNRJSTVLZIFU4LAVCNFSM6AAAAABJQ4KUKKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZYGQ4DIMRXGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Looks like vpn issue, here i don't have any issues. I connected to the USA , worked fine

Ok I'll update VPN or re-install, or perhaps contact them. Helps to narrow
it down. Thanks for the info.
On Wednesday, June 19, 2024, Rishabh Bhardwaj @.***>
wrote:

Looks like vpn issue, here i don't have any issues. I connected to the USA
, worked fine
—
Reply to this email directly, view it on GitHub
https://github.com/rishabh3354/VIZTUBE/issues/4#issuecomment-2179790733,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUERFIRB63LVFU6ACKZUISDZIJMIVAVCNFSM6AAAAABJQ4KUKKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZZG44TANZTGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thanks for reporting. it is now fixed in new version of the app. i am closing this issue for now.


text:
Fix encoding problem

Add the following lines after import sys:
reload(sys) sys.setdefaultencoding('utf8')

Probably not a good idea - http://stackoverflow.com/a/3828742/1866159

repo: rishirdua/facebook-birthday-response


text:
Support format & override in runner

Currently it's put in bin, but maybe it can be put in runner, so that users who implement custom engines can use that feature 🤔

BTW, maybe we can add some mechanism to let bin load custom engines ?! 🤪

supported by #130

repo: risinglightdb/sqllogictest-rs


text:
bug: panic on schema mismatch when sink into table with array of struct

Describe the bug
Refer to the produce and error messages below. Still investigating.
Specially, the table's column data types changed after executing sink into table.
Error message/log
thread 'rw-streaming' panicked at src/stream/src/executor/wrapper/schema_check.rs:48:29:
schema check failed on ExecutorInfo { schema: Schema { fields: [id:Varchar, parameters:Struct(StructType { field_names: ["name", "string"], field_types: [Varchar, List(Struct(StructType { field_names: ["value"], field_types: [Varchar] }))] }), _row_id:Serial] }, pk_indices: [], identity: "Project 2C00000000" }: column type mismatched at position 1: expected Some(Struct(StructType { field_names: ["name", "string"], field_types: [Varchar, List(Struct(StructType { field_names: ["value"], field_types: [Varchar] }))] })), found Some("List")
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

*** await tree context of current task ***

Actor 44: `CREATE TABLE t ("id" CHARACTER VARYING, "parameters" STRUCT<"name" CHARACTER VARYING, "string" STRUCT<"value" CHARACTER VARYING>[]>)` [67.554s]
  Epoch 7521805286637568 [534.547ms]
    Materialize 2C00000005 [534.547ms]
      RowIdGen 2C00000004 [534.547ms]
        Union 2C00000003 [534.547ms]  <== current
          Merge 2C00000002 [534.547ms]

To Reproduce
CREATE SOURCE s (raw bytea) WITH (connector = 'kafka', kafka.brokers = '127.0.0.1:51349', topic = 'json-any') FORMAT PLAIN ENCODE BYTES;

CREATE MATERIALIZED VIEW parsed AS (SELECT CONVERT_FROM(raw, 'utf8')::JSONB AS payload FROM s);

CREATE TABLE t (
    "id" VARCHAR
    , "parameters" STRUCT<
        "name" VARCHAR
        , "string" STRUCT<
            "value" VARCHAR
        >
    >[]
);

SELECT definition FROM rw_tables;

CREATE SINK sk INTO t AS (SELECT
        payload ->> 'id' AS "id",
        (jsonb_populate_record(NULL::STRUCT<
                parameters STRUCT<
                        "name" CHARACTER VARYING,
                        "string" STRUCT<"value" CHARACTER VARYING>
                >[]
        >, payload)).parameters AS "parameters"
        FROM parsed)
with (
        type='append-only',
);

SELECT definition FROM rw_tables;

Expected behavior
It should NOT panic.
The table definition should NOT change as follows:
- CREATE TABLE t ("id" CHARACTER VARYING, "parameters" STRUCT<"name" CHARACTER VARYING, "string" STRUCT<"value" CHARACTER VARYING>>[])
+ CREATE TABLE t ("id" CHARACTER VARYING, "parameters" STRUCT<"name" CHARACTER VARYING, "string" STRUCT<"value" CHARACTER VARYING>[]>)

How did you deploy RisingWave?
./risedev p
The version of RisingWave
v2.0.1
Additional context
#14063

Seems to be a sqlparser bug on struct & array:
echo "CREATE TABLE t (\"id\" CHARACTER VARYING, \"parameters\" STRUCT<\"name\" CHARACTER VARYING, \"string\" STRUCT<\"value\" CHARACTER VARYING>>[])" | cargo run --bin sqlparser

repo: risingwavelabs/risingwave


text:
Cannot run live server in either browser

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[x ] Bug report  <!-- Please search GitHub for a similar issue or PR before submitting -->
[ ] Feature request
[ ] Documentation issue or request
[ ] Other: <!-- Please describe: -->

Current behavior

Installed addon, rightclick html and ran live server. Firefox/chrome gets "Page not working, invalid response. ERR_INVALID_HTTP_RESPONSE". I tried changing the settings to load in both firefox and chrome
Expected behavior

HTML page should be displayed, works fine if I click my html page manually
Environment
Browser:

- [x ] Chrome (desktop) version 69.0.3407
- [x ] Firefox version 62.0.3
- [ ] Safari (desktop) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:

- Live Server: <!-- Check which version is installed -->
- Platform:  <!-- Mac, Linux, Windows -->
- Visual Studio Code: <!-- `code --version` --> 5.1.1

Others

I'm also experiencing the same problem and decided to do some testing across a few setups to see if it was consistent across all stacks.
I've isolated the problem down to MacOS Mojave as the culprit. I upgraded to Mojave from High Sierra, so I'm not sure if the upgrade causes the liver server auto-reload problem, meaning it maybe okay on a fresh install of Mojave.
Here's what I tested…

Mojave - VS Code(1.28.1) - Chrome (69.0.3497...) - Does NOT Live Reload
Mojave - VS Code(1.28.1) - Safari (12.0.14606...) - Does NOT Live Reload
High Sierra - VS Code(1.28.1) - Safari (12.0.13606...) - Live Reload Works!
Windows 10 - VS Code(1.28.1) - Chrome - Live Reload Works!

Not sure why Mojave would be blocking live reload, but it seems to be doing something.

authors: BJSebastian Leeoku

repo: ritwickdey/vscode-live-server


text:
fix: wait for stop task to finish before allowing starting new task panel task

Fixes RVTEE-656

#296  👈
main

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

Join @NathanFlurry and the rest of your teammates on  Graphite

repo: rivet-gg/plugin-godot


text:
Guava fault 1 trigger tests differ between Java 7-8

The Guava faults - contributed to the bug mining branch - were gathered using the JDK 8 compiler. To retain compatibility, we seek identical trigger test results whether using JDK 7 or 8.
All Guava faults EXCEPT #1 meet this goal. However, Guava fault #1 demonstrates different results. Using JDK 8, the command ''defects4j test'' returns the following:
Failing tests: 5
  - com.google.common.cache.LongAdderTest
  - com.google.common.collect.MinMaxPriorityQueueTest::testRandomAddsAndRemoves
  - com.google.common.collect.MinMaxPriorityQueueTest::testRandomRemoves
  - com.google.common.collect.MinMaxPriorityQueueTest::testRemoveRegression
  - com.google.common.reflect.ClassPathTest::testExistsThrowsSecurityException

(Two of those - the first and last - also fail on the fixed version. The remaining three are the trigger tests.)
Using JDK 7,  instead, 370 "tests" fail. These are all test classes, rather than individual test cases, and are almost all of the test classes.
How to address this ties into recent discussions on the future of Defects4J. One option:

Keep faults 2-9 (and add in the separate faults 10-11, which require JDK7) in the core Java 7 version of Defects4J. Faults 2-9 have the same test behavior for both JDK7 and 8 .
Add faults 1-9 to the JDK8 branch. Faults 1-9 were mined using JDK 8 and have no issues.

Note - the recently-contributed JacksonCore faults offer identical test results in JDK7 and 8. No concerns there.

Following changes made in porting Guava to the main branch, Guava 1 now works in Java 7 as well.


text:
Fix Calc's evaluation of ValueScope values inside objects or lists

For instance, upper(slice($*, 1)) doesn't work because the array elements are ParameterValue type and the convertCase function doesn't do the "getValue()" call on them.  There are likely many, many places like this in the latest code where I am not dealing with ValueScope objects correctly.

Along the way, maybe we could rename "evaluateFunction" in CalcObjectVisitor to just "evaluate" (as we make sure to call it in all the right places where it has been missing already).

What remains to do is to closely examine all the other possible places where "evaluate" might need to be called (spec. for ValueScope or ParameterScope objects) but isn't.  I don't have any fail cases right now, so we might need to do some intense work to find them (if any exist).

repo: rlwhitcomb/utilities


text:
🛑 5typos.ne Blog is down

In a1020f9, 5typos.ne Blog (https://5typos.net) was down:

HTTP code: 0
Response time: 0 ms

Resolved: 5typos.ne Blog is back up in 58f831d after 4 minutes.

repo: rmateu/statuspage


text:
When linking two long ranges, not longer is scroll-able

This simple note will break scrolling in live preview mode.
![[Mark 14#60..72]]

![[Mark 14#60..72]]

after the second link is added and it loads (there seems to a delay of 5 seconds) the page will auto scroll to the top of the note and you are unable to scroll to the bottom of the page again.

This seems to not be an issue anymore with Obsidian version 1.4.14

repo: rmellmer/obsidian-link-range


text:
SysExMessage::Buffer requires Default, limiting buffer size to 32 bytes

Hi there!
Thank you so much for all the hard work that's gone into NIH-plug and its ecosystem.
I'm just wondering about the Default requirement on the SysExMessage::Buffer type, effectively limiting the size to 32 bytes because of delightful Rust array shenanigans. I removed this bound locally: everything compiles and all of the tests still pass.
This Default doesn't seem necessary here, since the buffer is custom-created via calls to to_buffer. The code internals clearly don't rely on this bound and making the type more permissive should be strictly non-breaking. For instance, you could still call Buffer::default() on smaller buffers as before (if you're not doing something more direct like [a, b, c, d, ..]), whereas [0; N]` or similar would make larger arrays doable.
As for the motivation: for my plugin work, I need comparatively huge SysEx buffers (typically 1kB+) to collect incoming notation/articulation/arbitrary text information from the DAW, update this data and add/modify MIDI events in various ways based on the data contained therein. For debug builds I use a human-readable format but switch to a more compact binary representation for release. It's all working phenomenally well so far, especially with VST3, and I would really like to use the official repo instead of my patched version.
If there are no objections, I'd be happy to submit a pull request for this one-liner.
Thanks again,
Iain

The Default constraint is indeed no longer necessary, good catch. It was part of an older design where the equivalent of SysExMessage::to_buffer() would fill in existing slice which meant that the caller had to construct that slice first. Fixed in 727b15aa9304918f0098d6be6ac406ea444737d5. Do keep in mind that if you need to accept 1 KiB SysEx events your plugin's memory consumption will grow by a couple megabytes to be able to handle events that large.

authors: ijijn robbert-vdh

repo: robbert-vdh/nih-plug


text:
Disable Aliases from plugins I didn't enable

It appears that a bunch of aliases have been enabled upon installing oh-my-zsh, even though the plugin files in which they are declared were never included in my .zshrc
How can I disable all those aliases?
P.S.: I noticed several similar issues but none actually discussing this specific issue.

git is by default enabled if that's the one you're referring to, check your plugins=() array. If that's not it post your zshrc file.

authors: DaniSpringer mcornella

repo: robbyrussell/oh-my-zsh


text:
Create README.md

Added README for UFW plugin issue: [Good first PR] Add READMEs to undocumented plugins #7175

Thanks!

authors: ajmorozoff mcornella


text:
Preventing buttons for consecutive calls at DataGrid/DataForm

Prevent buttons from making a second request when Ajax Call is made

New button component added for async calls. It prevents multiple actions on ajax calls.

authors: fatihtotrakanli serayuzgur

repo: robeio/robe-react-ui


text:
change/prettify README (no parcel anymore)

added "endOfLine": "auto" for eslint for VSC in Windows.
allow /dist in github, otherwise users would have to download the repo and build it themselves. Closes #146
prettier changed the Readme quite a bit..

Or maybe we could use Github Actions to build the repo and write to the /dist folder?


text:
Make CVXOPT optional

@robertmartin8 That's a good point. When running the discrete allocation without any MI solvers (or only ECOS_BB), cvxpy raises a very informative error, detailing the issue. It also provides two quick fixes to the user:
Quick fix 1: if you install the python package CVXOPT (pip install cvxopt),
then CVXPY can use the open-source mixed-integer solver `GLPK`.

Quick fix 2: you can explicitly specify solver='ECOS_BB'. This may result
in incorrect solutions and is not recommended.

One could even go one step further and catch it early, but I do not think that would add a lot of value.
Another approach would be to switch the default MI solver to either CPLEX or XPRESS. Both run the tests just fine and I never had an issue with installation. Check the cvxpy Docs for the limitations. Even though they are installing the free editions by default, some users might have issues with closed source dependencies for various reasons.
Originally posted by @phschiele in https://github.com/robertmartin8/PyPortfolioOpt/issues/284#issuecomment-779853255

New thread (based on #284) to reduce clutter.
I think the simplest fix is just to make CVXOPT optional like matplotlib and sklearn. Then, in the body of lp_portfolio, I'll check if GLPK_MI is installed and raise a warning/error if not.
It would indeed be easier to just wrap CVXPY's error message, but I don't think that's possible given the current design of lp_portfolio: it requires an argument specifying the solver, so there is no situation in which problem.solve() will be called with no explicit solver.

@robertmartin8 Sounds like a good plan.
I'd probably not check for GLPK_MI  directly, but just if the specified solver (which must not be None), so advanced users can provide their own solver (this is already how it is implemented).
If one would like to allow None (i.e. let CVXPY check for available solvers), one would just have to change one line:
if solver not in cp.installed_solvers():

to
if solver is not None and solver not in cp.installed_solvers():

Both approaches seem fine to me.


text:
Single checkbox filter in group

Available?

1

2

By default, all items should be shown
If you check the filter, only the first item should be shown.
If you uncheck the filter, all items should be shown.

Fixed by b783b5aec9348f90ac595884b62339c22d11db6d
Test still missing

Now closed by https://github.com/robertpainsi/filtering.js/commit/7b5df3903958bf2a992ae2833aefd444e610c14f


text:
🛑 KickerTracker App is down

In 6da7e2e, KickerTracker App ($KICKERTRACKER_APP_URL) was down:

HTTP code: 0
Response time: 0 ms

Resolved: KickerTracker App is back up in cab4ffd after 2 hours, 22 minutes.

repo: robingenz/upptime


text:
Grabbing 52 week high and 52 week low

Is is possible to grab the 52 week high and low for a given stock.
For example:
rh.quote_info('AAPL')['52wk_high']
>> '334.34'

yes

fundamentals  = trader.fundamentals('DIS')
low = fundamentals['high_52_weeks']
high = fundamentals['low_52_weeks']

I'm going to consider this issue closed.

repo: robinhood-unofficial/Robinhood


text:
Make 'container' element follow 'overlay' rather than being nested within it

This is a workaround for an issue in the underlying micromodal library, present in version 0.4.10.
Upgrading from 0.4.8 to 0.4.10 is desirable, as it fixes an issue where a click to dismiss the Overlay propagates through to any underlying clickable elements.
This should fix #6.
Further notes on the changes are added as inline PR comments.

This looks great @Holf, thanks a lot! I need a bit of time (couple of weeks maximum) to give this a review and merge it — I'll get to it as soon as I can. Sorry!
In case you're in a rush, you should (I think) be able to already use this in your projects (assuming you're using npm or similar) by switching the dependency to:
- "svelte-micromodal": "^0.0.8"
+ "svelte-micromodal": "Holf/svelte-micromodal#6-make-overlay-and-container-peer-elements"

Cool, and no rush at all. 👍

Thanks a lot for your contribution @Holf, and thanks for your patience! This looks great — releasing the new version to npm in a minute :slightly_smiling_face:

Thank you, and no problem at all. Pleasure doing business with you. 😎
Do let me know if you spot any issues with it.

I will! Likewise if you spot any other issues with the library :slightly_smiling_face:

authors: Holf robinmetral

repo: robinmetral/svelte-micromodal


text:
Environment based value direct substitution in charts not working

I'd like to use the environment functionality, with a values file, to template our charts per environment. I don't need to template values in the helmfile, only the referenced charts. I'd like to leave the helmfile largely untouched if possible.
The issue is that when i reference a value inside a chart from an environment inside my helmfile - see deployment.yaml below, there is no value provided to them. i.e the output is  (note lines 7 and 8 below ) - UNLESS i reference the value inside the release stanza for a given chart helmfile (i.e assign it to an intermediate value inside the release)
value-from-env-values.yaml: {{ .Values.envInlineValue }}
value-from-env-values-file.yaml: {{ .Values.envFileValue }}

Building dependency release=test, chart=test-chart
Templating release=test, chart=test-chart
# Source: test-chart/templates/deployment.yaml
value-from-chart-values.yaml: 1
value-from-release: build
value-from-release-file: release-bar
value-from-env-values.yaml:
value-from-env-values-file.yaml:
value-from-env-values-via-release-value.yaml: Some build
value-from-env-values-file-via-release-value.yaml: build-bar
value-from-env-templated-release-values-file.yaml:

(The naming of the fields and their values reflect where the field values are derived from)

I'm running >  helmfile -e build template

My test helmfile and chart is
helmfile.yaml
environments:
  build:
    values:
    - build-env.yaml
    - envInlineValue: "Some build"
    missingFileHandler: Error

---

releases:
  - name: test
    chart: test-chart
    values:
    - releaseInlineValueTemplatedEnvObjectName: {{ .Environment.Name }}
    - release-env.yaml
    - releaseInlineValueTemplatedEnvObject: {{.Environment.Values.envInlineValue }}
    missingFileHandler: Error

test-chart/templates/deployment.yaml
value-from-chart-values.yaml: {{ .Values.replicaCount }}
value-from-release: {{ .Values.releaseInlineValueTemplatedEnvObjectName }}
value-from-release-file: {{ .Values.releaseFileValue }}
value-from-env-values.yaml: {{ .Values.envInlineValue }}
value-from-env-values-file.yaml: {{ .Values.envFileValue }}
value-from-env-values-via-release-value.yaml: {{ .Values.releaseInlineValueTemplatedEnvObject }}
value-from-env-values-file-via-release-value.yaml: {{ .Values.releaseInlineValueTemplatedEnvObjectFromFile }}
value-from-env-templated-release-values-file.yaml: {{ .Values.releaseTemplatedEnvFileValue }}

all-values:
  {{ toYaml . }}

values.yaml
# Default values for test-chart.
# This is a YAML-formatted file.
# Declare variables to be passed into your templates.

replicaCount: 1

image:
  repository: nginx
  pullPolicy: IfNotPresent

build-env.yaml
envFileValue: "build-bar"

release-env.yaml
releaseFileValue: "release-bar"

I've tried this test with:

Environment block first and last
With and without '---' seperator
default environment and named environments
With the --log-level command (and debug)

When i inspect the rendered chart toYaml field, in values i see only
Values:
  image:
    pullPolicy: IfNotPresent
    repository: nginx
  releaseFileValue: release-bar
  releaseInlineValueTemplatedEnvObject: Some build
  releaseInlineValueTemplatedEnvObjectFromFile: build-bar
  releaseInlineValueTemplatedEnvObjectName: build
  replicaCount: 1

i.e the values defined in the environment have not been passed (envFileValue: "build-bar")
When i run with the debug command I can see that it looks like it's generating values - and storing them in /tmp/.... (this is the output without the --- divider - it differs from the output with the --- divider, which has no reference "to replaced s to workaround" ). I've looked at these files - none of these contain envFileValue despite the logging showing that it did "load" these values
i.e merged environment: &{build map[envFileValue:build-bar envInlineValue:Some build] map[]}
processing file "helmfile.yaml" in directory "."
first-pass rendering starting for "helmfile.yaml.part.0": inherited=&{build map[] map[]}, overrode=<nil>
first-pass uses: &{build map[] map[]}
first-pass rendering output of "helmfile.yaml.part.0":
 0:
 1: releases:
 2:   - name: test
 3:     chart: test-chart
 4:     values:
 5:       - releaseInlineValueTemplatedEnvObjectName: build
 6:       - release-env.yaml
 7:       - releaseInlineValueTemplatedEnvObject:  <no value>
 8:       - releaseInlineValueTemplatedEnvObjectFromFile:  <no value>
 9:     missingFileHandler: Error
10:
11: environments:
12:   build:
13:     values:
14:     - build-env.yaml
15:     - envInlineValue: "Some build"
16:     missingFileHandler: Error

replaced <no value>s to workaround https://github.com/golang/go/issues/24963 to address https://github.com/roboll/helmfile/issues/553:
  strings.Join({
        ... // 5 identical lines
        "      - releaseInlineValueTemplatedEnvObjectName: build",
        "      - release-env.yaml",
-       "      - releaseInlineValueTemplatedEnvObject:  <no value>",
-       "      - releaseInlineValueTemplatedEnvObjectFromFile:  <no value>",
+       "      - releaseInlineValueTemplatedEnvObject:  ",
+       "      - releaseInlineValueTemplatedEnvObjectFromFile:  ",
        "    missingFileHandler: Error",
        "",
        ... // 6 identical lines
  }, "\n")

envvals_loader: loaded build-env.yaml:map[envFileValue:build-bar]
first-pass produced: &{build map[envFileValue:build-bar envInlineValue:Some build] map[]}
first-pass rendering result of "helmfile.yaml.part.0": {build map[envFileValue:build-bar envInlineValue:Some build] map[]}
vals:
map[envFileValue:build-bar envInlineValue:Some build]
defaultVals:[]
second-pass rendering result of "helmfile.yaml.part.0":
 0:
 1: releases:
 2:   - name: test
 3:     chart: test-chart
 4:     values:
 5:       - releaseInlineValueTemplatedEnvObjectName: build
 6:       - release-env.yaml
 7:       - releaseInlineValueTemplatedEnvObject:  Some build
 8:       - releaseInlineValueTemplatedEnvObjectFromFile:  build-bar
 9:     missingFileHandler: Error
10:
11: environments:
12:   build:
13:     values:
14:     - build-env.yaml
15:     - envInlineValue: "Some build"
16:     missingFileHandler: Error

envvals_loader: loaded build-env.yaml:map[envFileValue:build-bar]
merged environment: &{build map[envFileValue:build-bar envInlineValue:Some build] map[]}
Building dependency release=test, chart=test-chart
exec: helm dependency build test-chart
exec: helm dependency build test-chart:
processing 1 groups of releases in this order:
GROUP RELEASES
1     test

processing releases in group 1/1: test
worker 1/1 started
worker 1/1 finished
successfully generated the value file at release-env.yaml. produced:
releaseFileValue: "release-bar"
Templating release=test, chart=test-chart
exec: helm template test test-chart --values /tmp/values488756425 --values /tmp/values726486932 --values /tmp/values876768739 --values /tmp/values216311782
exec: helm template test test-chart --values /tmp/values488756425 --values /tmp/values726486932 --values /tmp/values876768739 --values /tmp/values216311782: ---
# Source: test-chart/templates/deployment.yaml
value-from-chart-values.yaml: 1
value-from-release: build
value-from-release-file: release-bar
value-from-env-values.yaml:
value-from-env-values-file.yaml:
value-from-env-values-via-release-value.yaml: Some build
value-from-env-values-file-via-release-value.yaml: build-bar
value-from-env-templated-release-values-file.yaml:

#all-values:
 # `{`{ toYaml . `}`}

---
# Source: test-chart/templates/deployment.yaml
value-from-chart-values.yaml: 1
value-from-release: build
value-from-release-file: release-bar
value-from-env-values.yaml:
value-from-env-values-file.yaml:
value-from-env-values-via-release-value.yaml: Some build
value-from-env-values-file-via-release-value.yaml: build-bar
value-from-env-templated-release-values-file.yaml:

#all-values:
 # `{`{ toYaml . `}`}

I'm using:
helmfile version v0.99.0
helm version
version.BuildInfo{Version:"v3.0.3", GitCommit:"ac925eb7279f4a6955df663a0128044a8a6b7593", GitTreeState:"clean", GoVersion:"go1.13.6"}
What am i missing ? Is this by design ? Are the "values" passed to the rendering of the helmfile, and related files meant to be passed through to the helm charts ? Apologies if i'm missing something obvious !!
helmfile-test.zip

I am seeing the same behavior on 0.101.0

I'm not sure if this is by design, but it bit me too. I was hoping to utilize bases to implement an environment layer that could be a reusable way to reference helmfile-specific values to reduce boilerplate for requiring the helmfile authors at our org.
Here's what I was trying to do:
→ tree .
├── helmfile.yaml
└── values
    ├── development.yaml
    ├── production.yaml
    └── staging.yaml

→ tree ../defaults/
├── layers
│   └── environments.yaml
└── values
    ├── development.yaml
    ├── production.yaml
    └── staging.yaml

→ cat ../defaults/layers/environments.yaml
environments:
  staging:
    values:
    - ../defaults/values/staging.yaml
    - values/staging.yaml
...

```shell
→ cat ../defaults/values/staging.yaml
global:
  environment: staging
...

bases:
- ../defaults/layers/environments.yaml

releases:
- name: app
  chart: repo/app
  version: "~0.1.0"

The hope was that we could simplify the dev experience by providing an "automatic" way to reference global variables and to have helmfile env-specific values files integrated into a release's values using a simple helmfile -e staging... syntax. Unfortunately, I see that the values assigned in the environments are only accessible to the helmfile itself and not the helm chart.
My hope is that environment values can be more useful. At the very least, they need better documentation.

@alisters Hey! Sorry for the slow response.

i.e the values defined in the environment have not been passed (envFileValue: "build-bar")

Yeah this is by design.
The environment values are completely different concept than chart values. That's why the name doesn't contain "chart".
I admit this is confusing but it was unavoidable to support the case that you wanna have "intermediate" environment values that you don't want to pass-though to charts without any processing.

At the very least, they need better documentation.

@irlevesque I've been constantly hearing people asking this - but honestly I'm not sure where/how to improve.
Environment values concept is already documented in https://github.com/roboll/helmfile#environment-values and it doesn't say it is propagated to chart values.
Also - for your specific example, you should use --- after bases. Otherwise you're falling into the chicken-and-egg prob between environment values and your desired state. That is, helmfile.yaml needs environment values loaded in order to render all the template expressions. But you need to render the helmfile.yaml to load environment values.
bases:
- ../defaults/layers/environments.yaml

---

releases:
- name: app
  chart: repo/app
  version: "~0.1.0"

Also, propagating environment values to chart values is as easy as:
bases:
- ../defaults/layers/environments.yaml

---

releases:
- name: app
  chart: repo/app
  version: "~0.1.0"
  values:
  - {{ .Values | ndeint 4 }}

authors: alisters cdunford irlevesque mumoshu

repo: roboll/helmfile


text:
console prints spam

Hi!

I suppose that's not good to print a message in console every time a winch is used. Console is for debugging, testing, scripts cheating and other abnormal activities monitoring, not for standard activities. Another way could be a flying-text entity use (I use one entity prototype for all mod messages) - works fine for such rare messages.
If my humble opinion is completely wrong at least a mod setting per player to turn those hints off could solve the problem.
P.S. What do you think about moving winch out of items into pure shortcuts?

Looks like there are some useful messages for typical user inaccuracy mistakes. But can be easily displayed via player.create_local_flying_text .
For example
use
player.create_local_flying_text({
				position = position,
				time_to_live = 250,
				text = {"vehicle-wagon2.train-in-motion-error"}
			})

instead of
player.print{"vehicle-wagon2.train-in-motion-error"}

This has been implemented for the 2.0 release


text:
Fix URL to "How to resolve" section of the README

When GitHub auto-generates IDs on headings in Markdown files, the slug it creates is word-separated by hyphens, not underscores. This PR changes the anchor in the URL that's displayed in the output when there is unused code found.
Before
how to resolve: https://github.com/robotdana/leftovers/tree/v0.5.2/README.md#how_to_resolve

After
how to resolve: https://github.com/robotdana/leftovers/tree/v0.5.2/README.md#how-to-resolve

Commits;

Change underscores to hyphens in URL to README

oh thank you, i noticed this a while ago but forgot to fix it.
please rebase so tests can run (and i think there's probably a couple tests that need updating)

Oh thanks for catching those others!

authors: robotdana veganstraightedge

repo: robotdana/leftovers


text:
Process: Better support on Jython 2.7 (termination, signals, pid)

Jython 2.7 beta 3 added initial support for Popen.terminate and Popen.send_signal. Due to JVM limitations they won't support everything normal Python supports, but we should get at least basic support for Terminate Process and Send Signal To Process. Hopefully we could also get support for Popen.pid.
This issue covers verifying the aforementioned enhancements by running acceptance tests with Jython 2.7b3+. Possible problems need to be reported to Jython project and patches provided when feasible, Process code adjusted where possible, and possible limitations documented.
Concrete tasks related to this issue:

[x] Workaround Popen.terminate closing PIPEs (http://bugs.jython.org/issue2218). Reported separately as #1870, fixed by 092e238 and included already in RF 2.8.7.
[x] Implement Popen.pid (http://bugs.jython.org/issue2221)
[x] Implement forceful kill outside Windows (http://bugs.jython.org/issue2220)
[x] Implement signal support (http://bugs.jython.org/issue2219)
[x] Document that child processes cannot be terminated or signals send to them.
[x] Document that graceful termination is not supported on Windows.
[x] Mark tests that cannot pass due to Jython/JVM limitations non-critical.

I implemented PID support to Jython (http://bugs.jython.org/issue2221). It will be part of Jython 2.7 beta 4 and requires no changes to Robot side. I'm now looking at Popen.send_signal and Popen.kill and both look doable.

This pull requests fixes Jython's Popen.kill and Popen.send_signal:
https://github.com/jythontools/jython/pull/14
No other changes are required to make related functionality work with Process library, so they will work also with earlier RF versions. Will keep this issue open until the PR is merged.

The aforementioned Jython pull request has been merged and related issues closed. Also this issue can now be considered done.

repo: robotframework/robotframework


text:
Add Ressource - amount left or ressource.isNotDepleted

Now that ressources deplete, the player needs a way to check for that. Otherwise the robots continue to walk to the marker and serve no purpose.

implemented in 2dbfa6a9aaaded00b71f797483bda765300f31e1

repo: robotopia-x/robotopia


text:
Suggestion: Grey-out skill instead of hide

I've been looking for months for a plugin that's similar to this one, however with the ability to grey-out a skill instead of hiding them. I'm playing skill-restricted accounts (an 807-UIM for example) and it would be super awesome if it would be possible to grey-out the skills I'm "not allowed" to skill instead of hiding them.
Like the way F2P greys out the member skills.

I think the plugin has the basics for this already but I'm not a good enough coder to modify it myself so I'm really hoping you're willing to add this feature to the plugin. I also think it might make the plugin appeal to a wider audience.
Hope to hear from you!

Added in v1.1, waiting for approval here: https://github.com/runelite/plugin-hub/pull/3841


text:
consider using krew-plugin-release to bump krew-index

Hi Rob
I've created this github action krew-plugin-release, that can be used to auto-bump the new version in krew-index when you publish a new release of your plugin.
Right now I am using this to publish the modify-secret krew plugin.
If this is something that interests you, I will be happy to open a PR for you.
Thanks
Rajat Jindal

Hey @rajatjindal, this looks really cool, would love a PR if you have time!

awesome. thank you. I have started work on it, and will submit the PR as soon as I test it once.

Between i should clarify that i changed the implementation from a github action (required a token of user) vs a webhook based implementation (use krew-release-bot account) to open the pr
This was changed based on discussions with the krew team

authors: rajatjindal robscott

repo: robscott/kube-capacity


text:
New rdf4h release coinciding with xmlbf 0.6 release

The new XML parser depends on monad transformer additions added by @wismill to the xmlbf library. This dependency will be reflected in the release of 0.6 of the xmlbf library.
Once xmlbf-0.6 has been uploaded to hackage, the stack.yaml file for rdf4h should add xmlbf-0.6 as an extra dependency, removing the information about the xmlbf git repo and commit ID, since stack will find xmlbf-0.6 on hackage.

Also update the version bounds in the cabal file

There wasn't a xmlbf release reflecting the monad transformer needed by rdf4h, so instead two files Xmlbf.hs and Xeno.hs were copied into this repository, temporarily hopefully:
https://github.com/robstewart57/rdf4h/commit/7cc70978b6001e426fe9224c2334bc6a1687dd71


text:
PrometheusNotFound: when running the Rescan from the UI.

Describe the bug
In the UI under Efficiency when you select cluster to rescan and then click Rescan you get an error in the logs that says PrometheusNotFound: Prometheus url could not be found while scanning in default cluster
I am using an external prometheus
To Reproduce
Steps to reproduce the behavior:

Go to 'Efficiency`
Select a cluster from the Cluster to rescan dropdown.
Click on Rescan'
Wait for message to popup saying the job failed.
See error in logs

Expected behavior
I would expect this to rescan the cluster
Desktop (please complete the following information):

OS: MacOS
Browser Chrome
Version 113.0.5672.92

Hey, @mmulligan03
What prometheus installation do you have on your cluster? Does it work with robusta itself?

Closing this issue because no answer received
And it actually refers to an old Robusta UI, not a KRR itself

repo: robusta-dev/krr


text:
Sia/Decred Luxor Mining: Error submitting solution - Unauthorized User

Hi @robvanmieghem,
thanks for your work on this miner! I was playing around and tried to use it for mining Sia and Decred on a MacPro but unfortunately I don't get credited for my hashrate/share nor will it be shown on any mining pools like siamining, nanopool or luxor. I've installed gominer and I'm able to start gominer successfully wit the following command:
gominer -url stratum+tcp://eu.luxor.tech:3333 -user <wallet_address>.<worker_name>
2018/01/21 15:27:50 Platform Apple
2018/01/21 15:27:50 1 device(s) found:
2018/01/21 15:27:50 0 - GPU - AMD Radeon HD Tahiti XT Prototype Compute Engine
2018/01/21 15:27:50 Starting SIA mining
2018/01/21 15:27:50 0 - Initializing GPU - AMD Radeon HD Tahiti XT Prototype Compute Engine
2018/01/21 15:27:50 Connecting to eu.luxor.tech:3333
2018/01/21 15:27:50 0 - Global item size: 268435456 (Intensity 0 ) - Local item size: 256
2018/01/21 15:27:50 0 - Initialized  GPU - AMD Radeon HD Tahiti XT Prototype Compute Engine
2018/01/21 15:27:50 0 - No work ready
2018/01/21 15:27:51 ERROR fetching work - No job received from stratum server yet
2018/01/21 15:27:51 Stratum server changed difficulty to 25.1
2018/01/21 15:27:51 New job received from stratum server
2018/01/21 15:27:51 Authorization of <wallet_address>.<worker_name> : true
2018/01/21 15:27:52 0 - Continuing
...
0-867.9 Total: 867.9 MH/s  2018/01/21 15:31:20 New job received from stratum server
0-868.1 Total: 868.1 MH/s  2018/01/21 15:31:42 0 - Yay, solution found!
0-867.3 Total: 867.3 MH/s  2018/01/21 15:31:42 0 - Error submitting solution - Unauthorized User
0-867.8 Total: 867.8 MH/s  2018/01/21 15:31:50 New job received from stratum server
0-867.9 Total: 867.9 MH/s  2018/01/21 15:32:50 Stratum server changed difficulty to 7.209574468085107
2018/01/21 15:32:50 New job received from stratum server
0-867.9 Total: 867.9 MH/s  2018/01/21 15:33:03 0 - Yay, solution found!
0-867.4 Total: 867.4 MH/s  2018/01/21 15:33:03 0 - Error submitting solution - Unauthorized User
0-868.0 Total: 868.0 MH/s  2018/01/21 15:33:06 0 - Yay, solution found!
0-867.2 Total: 867.2 MH/s  2018/01/21 15:33:06 0 - Error submitting solution - Unauthorized User
...

I'm not sure if there is maybe a problem in communicating with the stratum server or have I missed something? I've also tested my MacBook and got the same errors. Where Luxor at least can find my address but shows 0 MH/s, Nanopool an Siamining don't even find my address at all.

i have found the save problem with siamining.com that stats can not sync for
This address has not submitted any shares yet. If you have just started mining, please come back in a minute.

authors: drobakowski redcatmiss

repo: robvanmieghem/gominer


text:
impl add/sub/mul wrapped for u128/i128

The change here implements add/sub/mul wrapped variants for u128/i128 by exporting the relevant bitcode functions and calling them. I'd like to implement them as asm, but it involves exposing multiple variants of add/sub/mul asm instructions and I will defer that to a subsequent PR, leaving this one the minimal change to get this working.
fixes #7253 , fixes #7251

I'm thinking we should cleanup some of the related tests in gen_num with this change to cover more of the different types.  They're a bit spread out and it's hard to see we've covered the different combinations. WDYT?

I can give it a shot. For example of add, I can move all the add_wrap_* tests to the same place and check similar spread of cases as the add_saturated test? I think for F32/F64/Dec there should be no wrapping behaviour, and currently Num.addWrap throws a type error when you try with Frac * types.

authors: lukewilliamboswell shua

repo: roc-lang/roc


text:
Remove nvim-cmp sources

Remove nvim-cmp's sources as they are better documented on the plugin's wiki and other completion plugins' sources aren't documented.

I think this makes sense, although you missed one.

It isn't a source, just a utility function. which is why I didn't remove it and why it isn't documented in nvim-cmp's wiki.

repo: rockerBOO/awesome-neovim


text:
migrate2rocky.sh error CAn;t get package what provides /etc/yum.repos.d/rhel8.repo

Please help as I am getting the following error.
[root@1111 rocky-tools]# ./migrate2rocky/migrate2rocky.sh -r
migrate2rocky - Begin logging at Tue 08 Mar 2022 01:21:09 PM IST.
Updating Subscription Management repositories.
Unable to read consumer identity
This system is not registered with an entitlement server. You can use subscription-manager to register.
Removing dnf cache
Preparing to migrate Red Hat Enterprise Linux 8.5 (Ootpa) to Rocky Linux 8.
Determining repository names for Red Hat Enterprise Linux 8.5 (Ootpa)......
Found the following repositories which map from Red Hat Enterprise Linux 8.5 (Ootpa) to Rocky Linux 8:
Red Hat Enterprise Linux 8.5 (Ootpa)  Rocky Linux 8
InstallMedia-AppStream                appstream
InstallMedia-BaseOS                   baseos
Getting system package names for Red Hat Enterprise Linux 8.5 (Ootpa)........Error: No Matches found
Can't get package that provides /etc/yum.repos.d/rhel8.repo.
An error occurred while we were attempting to convert your system to Rocky Linux. Your system may be unstable. Script will now exit to prevent possible damage.
A log of this installation can be found at /var/log/migrate2rocky.log
The following is log
migrate2rocky - Begin logging at Tue 08 Mar 2022 01:21:09 PM IST.
Updating Subscription Management repositories.
Unable to read consumer identity
This system is not registered with an entitlement server. You can use subscription-manager to register.
Removing dnf cache
Preparing to migrate Red Hat Enterprise Linux 8.5 (Ootpa) to Rocky Linux 8.
Determining repository names for Red Hat Enterprise Linux 8.5 (Ootpa)......
Found the following repositories which map from Red Hat Enterprise Linux 8.5 (Ootpa) to Rocky Linux 8:
Red Hat Enterprise Linux 8.5 (Ootpa)  Rocky Linux 8
InstallMedia-AppStream                appstream
InstallMedia-BaseOS                   baseos
Getting system package names for Red Hat Enterprise Linux 8.5 (Ootpa)........Error: No Matches found
Can't get package that provides /etc/yum.repos.d/rhel8.repo.
An error occurred while we were attempting to convert your system to Rocky Linux. Your system may be unstable. Script will now exit to prevent possible damage.

The issue is likely that your RHEL subscription has expired. If you're using the free "Red Hat Developer Subscription for Individuals" then see https://developers.redhat.com/articles/renew-your-red-hat-developer-program-subscription# for information on how to re-register. If you have a paid subscription then you'll need to renew it in the access portal. If you wish you can probably convert an expired paid subscription to a free developer subscription by first subscribing from the link at https://developers.redhat.com/blog/2016/03/31/no-cost-rhel-developer-subscription-now-available?source=sso#what_you_get then following the directions to re-register above.

I found a work around. First shifted to AlmaLinux and then to Rocky.
Alma Linux does not depend on subscription of RHEL for migration.

authors: pajamian vipinbhidwaria

repo: rocky-linux/rocky-tools


text:
Added SHA384

Added SHA-384 to get closer to enabling yet another TLS 1.3 cipher suite. Essentially identical to SHA-512 but with smaller digest size, different IVs and truncated output. Tested against FIPS 180-4 test vectors in SHA384ShortMsg.rsp (added to expected test results)

Code which is common to all hash algorithms (including RFSB) should go in the private part of SPARKNaCl.Hashing (like the TS64 function, for example.). Code that is common to all variants of SHA2 should go in SPARKNaCl.Hashing.SHA2_Common. Does that make sense?

That should work, and would be my preference as well.

I agree - I think those short names came from TweetNaCl, so aren't really appropriate here, so please rename them as you see fit.

I won't be able to look at it this week, if you have bandwidth and would like to take a look at the refactor sooner I'd welcome the help!

A new branch is probably sensible!

I think the default is "portable and should work on any 32 or 64 bit machine (but favour 32-bit if and where you can...)" Does that make sense?

If you can push your commits to this PR branch I'm happy to continue working on where you left off.

OK just added you as a contributor!

I think I agree with Max - we already have more than one "family" of hash functions in the library - SHA2 and RFSB, so the package structure should reflect that.

authors: docandrew rod-chapman

repo: rod-chapman/SPARKNaCl


text:
Provider code called in rspec testing

PUP-6067 exposed 2 things:

The pip3 upgrade actually wasn't complete and caused issues in spec testing on OS X for RHEL specific code.
The pip, first provider, fallback was actually masking the fact that rspec-puppet is using that to mock out some facts about providers. When patch for the above bug gets merged into 4.4.1 it will be hidden again but issue is should rspec-puppet be calling that provider code.
When compiling a catalog on master (Linux) it shouldn't be calling and looking for binaries appropriate for another platform, therefore testing on Mac shouldn't be looking for /bin/rpm, /bin/yum, /bin/mount, /bin/umount.

I think this is a long time bug exposed with the pip3 upgrade incompleteness.

Hi, are you sure you are in the right repo? I do not understand what you are talking about, and PUP-6167 does not exist.

@DavidS yes, updated the link

This seems to be a variation of #352 . Besides deeply intrusive mocks that disconnect most of puppet from the host system, there is not much that can be done at this point in time.

@DavidS can you explain how is this a variation of #352, that seems to be literally a puppet_gem install and test.
This is a package resource like:
package {'jenkins':
  ensure => '1.6.23-1',
}

since pip was the first provider on my system before and supported versionable this worked.
Now with 4.2.0 -> 4.4.0 pip provider isn't supporting versionable (this is being fixed in that bug ticket).
So without it rspec-puppet is looking for /bin/rpm or /bin/yum on the system to use that package provider in order to determine if it's versionable supporting or not.
I believe the facts supplied for kernel, osfamily, operatingsystem, etc in the test should be enough to determine which package provider to use vs looking for the binaries of them. As you said stubbing out these for defaultprovider isn't a long term solution because then you are left stubbing out things like /bin/mount and /bin/umount for mount {'..':} resources, and so on.

I believe the facts supplied for kernel, osfamily, operatingsystem, etc in the test should be enough to determine which package provider to use vs looking for the binaries of them.

Emphasis on "should". Puppet is not implemented this way. rspec-puppet currently has no mechnisms in place working around this globally, and it would be a massive effort to do so.
To workaround this specific puppet bugs that affect a large number of tests, I would not be opposed to a patch that sets the default package provider to something other than pip on OSX for the affected puppet versions.

I was just reminded that #192 is another instance where the system running the specs leaks inexorably into the environment.

@DavidS Is there a way to mock up the provider in the rspec-puppet tests to exist? I don't follow PUP-6067 well enough to see how to transform that code into an rspec test to help it pass.
/cc @bastelfreak

@rnelson0 the solution is in https://tickets.puppetlabs.com/browse/PUP-6067?focusedCommentId=276716&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-276716 : Michael directly stubs the default provider on the package type.
If you look into http://rspec.info/documentation/3.4/rspec-mocks/#label-Test+Doubles you can provide a double as provider, which would allow you to replace everything at that level, at the cost of more stubbing.
Another complexity you might run into is the unholy mix of mocha and rspec-mocks. If you can, I'd recommend moving to rspec mocking syntax before starting this endeavor. See https://github.com/puppetlabs/puppetlabs_spec_helper/blob/917fd0c45a60539433c50763f7d96eb2c40a3539/lib/puppetlabs_spec_helper/puppet_spec_helper.rb#L122

This should now be resolved by #498

authors: DavidS cdenneen rnelson0 rodjek

repo: rodjek/rspec-puppet


text:
signal([]) not update state

Hi!
Thank you for your great package!
I spent a few hours on the problems below.
This doesn't update the state.
final list = signal(<String>[]);
list.value.add('newElement');

This updates state.
final list = listSignal(<String>[]);
list.add('newElement');

Of course, the document mentions listSignal, but newcomers may miss it.
I think some improvements will help them, so I opened this issue.
These are my ideas. If you have any thoughts, please let me know.

Update document
Change API name
create signals linter

Thank you.

There is a signals_lint package which we could update to include this. But this is a problem anything in dart would face with pass by value or reference.
This also would only not update with mutable lists as immutable variants you would need to create and pass a new list with the element.
Same think would apply with riverpod, value notifier, or block/cubit

If you really rely on collection identity you can build with
https://pub.dev/packages/fast_immutable_collections
It will trigger on assignation since instances are immutable.

authors: Solido mafreud rodydavis


text:
Sort styles’ keys before rendering class names

Description
There is an issue with server-side rendering due to keys being sorted differently on Node and the client (non-Blink based). This pull-request intends to sort object keys before outputting to class names to avoid that React mismatch.
Packages

fela@6.2.3

This pull-request needs to change its base branch once a branch rooted from fela@6.2.3 has been created on rofrischmann/fela.
Versioning
Patch
Checklist
Quality Assurance

[x] The code was formatted using Prettier (yarn run format)
[x] The code has no linting errors (yarn run lint)
[ ] All tests are passing (yarn run test)
Some tests are not passing but they don’t appear to be related to my changes.
[x] There are no flow-type errors (yarn run flow)

Changes

[x] Tests have been added/updated
[x] Documentation has been added/updated
[x] My changes have proper flow-types

@rofrischmann I’m not sure I understand why the tests are failing… Could you put me on the right track so I can fix them and we can get this merged?

repo: rofrischmann/fela


text:
parent_id in Files.create() should be folder_id

This style is more consistent since we are referring to the company and project with company_id and project_id

Resolved with #27

repo: rogers-obrien-rad/ProPyCore


text:
Remove zlib dependency, MSVC compatbilty

Could you please add an optional #ifdef to avoid zlib dependency for those looking to use only uncompressed npy save/load? There is no need for zlib.h for just *.npy files.
Something like:
#define CNPY_UNCOMPRESSED_ONLY //Disable compressed archive functionality and zlib dependency
#include "cnpy.h"

Also for MSVC users, the use of fopen, scanf/sprintf causes compiler issues. Adding the #pragma warning(disable:4996) fixes the problem, but it is recommended to replace these unsafe functions with the safer versions.

I wonder if the project is still active, the last commit was 2 years ago!

Same problem here. Did you have any success with removing the zlib dependency? @sayan1an

@grau4 I ended up removing all *.npz functionality by removing all functions which required zlib. It's fairly simple.

repo: rogersce/cnpy


text:
">" isn't properly escaped when contained in a string that is exported to rbxm causing import to fail

We're using remodel 0.8.1.
We have a rich-text string in the game that contains  to indicate a line break.
When remodel exports this, it exports it incorrectly: The < is escaped correctly as < but the > is not escaped at all. This causes the resulting XML to be invalid and rojo fails to import it. Removing the / before the > doesn't make a difference.
The original rbxlx file contains this:
<string name="Text">This is some
Text with a new line in</string>
The generated rbxmx contains this:
<string name="Text">This is some
Text with a new line in</string>
Attached is a minimal reproduction of this issue.
I'm not sure if this lies in remodel directly or the underlying libraries it uses - or the underlying xml serialisation libraries they use! I've tried digging through them to find out but it's tricky to follow and I'm not very familiar with rust.
new-line-bug.zip

I've dug into this further and it appears to be a known issue in xml-rs. It seems the issue is particularly with strings that contain /> (not just >). It's valid for > not to be escaped - but xml-rs incorrectly thinks that it isn't valid and fails to load it. So, this is just a deserialisation issue.

Thank you for investigating this and finding an upstream issue! I'm going to move this issue into rojo-rbx/rbx-dom because this likely impacts Rojo as well.

authors: LPGhatguy eAi

repo: rojo-rbx/remodel


text:
java.util.NoSuchElementException: No value present

SpongeAPI version: spongeforge-1.11.2-2227-6.0.0-BETA-2157.jar
PlaceholderAPI version: Placeholders-3.4.jar
Plugins installed:
AntiWDL-1.1-all.jar
API6-BetterBoom.jar
AutoBroadcasterSponge-0.2.1.jar
CommandSigns-API-6.0.0-1.3.1.jar
CustomPlayerCount.-.1.1.jar
EasyScoreboards.jar!
EconomyLite-5.0.0-2.9.0.jar
FlexibleLogin.jar
foxcore-0.10.3-208-server.jar
foxguard-0.20.2-api5-347-server.jar
Graveyards-v2.1.3.jar
helpme-5.0.0-0.2.2.jar
holograms-2.1.1.jar
IPLog-0.1.1.jar
JobsLite-1.7.0.jar
Launchpad-v1.0-FIXED.jar
MiscStuff-1.0-SNAPSHOT.jar
Nucleus-0.23.2-1.11.2-S6.0-plugin.jar
ore-plugin-1.1.0-SNAPSHOT.jar
PermissionsEx-Sponge.jar
Placeholders-3.4.jar
projectportals-5.1.0-0.13.6.jar
spongeforge-1.11.2-2227-6.0.0-BETA-2157.jar
Synonym-1.0.1.jar
TitleMOTD.-.1.0.jar
treepers-5.1.3.jar
twentyticks.jar
Vigilate-1.0.2.jar
walletdrop-5.1.0-0.2.1.jar
worldedit-sponge-6.1.7-SNAPSHOT-dist.jar
Error log [if any]: http://pastebin.com/EywHhmLh
Description:
Steps to reproduce: First start the server, after upgrading the plugin

That's a problem with Sponge.

Fixed by 17d6184 and 0c78488.

authors: Wundero dualspiral ziceptor


text:
839 send email from catalog

Description
Send an email when a new reviewer is added/modified.
This PR sends a "reviewer updated" email for admin/reviewer POST.  This requires env variables SENDER_EMAIL (@illinois.edu) and SENDER_PASSWORD (password for the sender email) to be configured. This branch is diverged from 864-add-email-reviewer branch.
Fixes #839
Review Time Estimate
Please give your idea of how soon this pull request needs to be reviewed by selecting one of the options below. This can be based on the criticality of the issue at hand and/or other relevant factors.

[ ] Immediately
[x] Within a week
[ ] When possible

Type of changes
Please select a relevant option:

[ ] Bug fix (non-breaking change which fixes an issue).
[x] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected).
[ ] Other (any another change that does not fall in one of the above categories.)

Checklist:
Please select all applicable options:

[ ] I have signed the Rokwire Contributor License Agreement (CLA). (Any contributor who is not an employee of the University of Illinois whose official duties include contributing to the Rokwire software, or who is not paid by the Rokwire project, needs to sign the CLA before their contribution can be accepted.)
[x] I have updated the CHANGELOG.
[ ] I have read the Contributor Guidelines.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] My change requires updating the documentation.
[ ] I have made necessary changes to the documentation.
[ ] I have added tests related to my changes.
[ ] My changes generate no new warnings.
[ ] New and existing unit tests pass locally with my changes.
[ ] Any dependent changes have been merged and published in downstream modules.

I had this error. Maybe my setting is wrong?
Traceback (most recent call last):
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\app.py", line 2463, in __call__
    return self.wsgi_app(environ, start_response)
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\app.py", line 2449, in wsgi_app
    response = self.handle_exception(e)
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask_cors\extension.py", line 165, in wrapped_function
    return cors_after_request(app.make_response(f(*args, **kwargs)))
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\app.py", line 1866, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\_compat.py", line 39, in reraise
    raise value
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\app.py", line 2446, in wsgi_app
    response = self.full_dispatch_request()
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\app.py", line 1951, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask_cors\extension.py", line 165, in wrapped_function
    return cors_after_request(app.make_response(f(*args, **kwargs)))
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\app.py", line 1820, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\_compat.py", line 39, in reraise
    raise value
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\app.py", line 1949, in full_dispatch_request
    rv = self.dispatch_request()
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\app.py", line 1935, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "C:\VirtualEnv\rokwire\lib\site-packages\connexion\decorators\decorator.py", line 68, in wrapper
    response = function(request)
  File "C:\VirtualEnv\rokwire\lib\site-packages\connexion\security\security_handler_factory.py", line 381, in wrapper
    return function(request)
  File "C:\VirtualEnv\rokwire\lib\site-packages\connexion\decorators\uri_parsing.py", line 149, in wrapper
    response = function(request)
  File "C:\VirtualEnv\rokwire\lib\site-packages\connexion\decorators\validation.py", line 193, in wrapper
    response = function(request)
  File "C:\VirtualEnv\rokwire\lib\site-packages\connexion\decorators\parameter.py", line 115, in wrapper
    return function(**kwargs)
  File "C:\Workspace-rokwire\rokwire-building-blocks-api\contributions\api\controllers\contributions.py", line 206, in post
    send_email_new_contribution(reviewer['githubUsername'])
  File "C:\Workspace-rokwire\rokwire-building-blocks-api\contributions\api\controllers\contributions.py", line 989, in send_email_new_contribution
    adminutils.send_email(dataset[0]['email'], subject, message)
  File "C:\Workspace-rokwire\rokwire-building-blocks-api\contributions\api\utils\adminutils.py", line 79, in send_email
    connection.login(mail_from, password)
  File "C:\Python37\lib\smtplib.py", line 730, in login
    raise last_exception
  File "C:\Python37\lib\smtplib.py", line 721, in login
    initial_response_ok=initial_response_ok)
  File "C:\Python37\lib\smtplib.py", line 642, in auth
    raise SMTPAuthenticationError(code, resp)
smtplib.SMTPAuthenticationError: (535, b'5.7.3 Authentication unsuccessful [CH2PR14CA0038.namprd14.prod.outlook.com]')

Should we need to provide the password? Can we use something like this?
import smtplib
smtpServer='smtp.yourdomain.com'
fromAddr='from@Address.com'
toAddr='to@Address.com'
text= "This is a test of sending email from within Python."
server = smtplib.SMTP(smtpServer,25)
server.ehlo()
server.starttls()
server.sendmail(fromAddr, toAddr, text)
server.quit()

Should we need to provide the password? Can we use something like this? I've using this kind of thing but tried to avoid using the password. This is what I got from the Internet and it might work this way as well if we can use some kind of smtp server.
import smtplib
smtpServer='smtp.yourdomain.com'
fromAddr='from@Address.com'
toAddr='to@Address.com'
text= "This is a test of sending email from within Python."
server = smtplib.SMTP(smtpServer,25)
server.ehlo()
server.starttls()
server.sendmail(fromAddr, toAddr, text)
server.quit()

The current code requires a sender email id and password. The method described above would also work if we have an SMTP server. Since we utilize the university server, we will just require an email id and password. This has been setup and configurations shared by Sandeep in a PEAR message.

Should we need to provide the password? Can we use something like this? I've using this kind of thing but tried to avoid using the password. This is what I got from the Internet and it might work this way as well if we can use some kind of smtp server.
import smtplib
smtpServer='smtp.yourdomain.com'
fromAddr='from@Address.com'
toAddr='to@Address.com'
text= "This is a test of sending email from within Python."
server = smtplib.SMTP(smtpServer,25)
server.ehlo()
server.starttls()
server.sendmail(fromAddr, toAddr, text)
server.quit()

The current code requires a sender email id and password. The method described above would also work if we have an SMTP server. Since we utilize the university server, we will just require an email id and password. This has been setup and configurations shared by Sandeep in a PEAR message.

Tried with the below code based on https://answers.uillinois.edu/illinois/page.php?id=47615
import smtplib
smtpServer='smtp.office365.com'
fromAddr='netId@illinois.edu'
toAddr='netId@illinois.edu'
text= "This is a test of sending email from within Python."
server = smtplib.SMTP(smtpServer,587)
server.ehlo()
server.starttls()
server.sendmail(fromAddr, toAddr, text)
server.quit()

Resulted in "Client not authenticated to send mail." error.
stmp.illinois.edu is not present.

I had this error. Maybe my setting is wrong?
Traceback (most recent call last):
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\app.py", line 2463, in __call__
    return self.wsgi_app(environ, start_response)
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\app.py", line 2449, in wsgi_app
    response = self.handle_exception(e)
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask_cors\extension.py", line 165, in wrapped_function
    return cors_after_request(app.make_response(f(*args, **kwargs)))
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\app.py", line 1866, in handle_exception
    reraise(exc_type, exc_value, tb)
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\_compat.py", line 39, in reraise
    raise value
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\app.py", line 2446, in wsgi_app
    response = self.full_dispatch_request()
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\app.py", line 1951, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask_cors\extension.py", line 165, in wrapped_function
    return cors_after_request(app.make_response(f(*args, **kwargs)))
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\app.py", line 1820, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\_compat.py", line 39, in reraise
    raise value
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\app.py", line 1949, in full_dispatch_request
    rv = self.dispatch_request()
  File "C:\VirtualEnv\rokwire\lib\site-packages\flask\app.py", line 1935, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "C:\VirtualEnv\rokwire\lib\site-packages\connexion\decorators\decorator.py", line 68, in wrapper
    response = function(request)
  File "C:\VirtualEnv\rokwire\lib\site-packages\connexion\security\security_handler_factory.py", line 381, in wrapper
    return function(request)
  File "C:\VirtualEnv\rokwire\lib\site-packages\connexion\decorators\uri_parsing.py", line 149, in wrapper
    response = function(request)
  File "C:\VirtualEnv\rokwire\lib\site-packages\connexion\decorators\validation.py", line 193, in wrapper
    response = function(request)
  File "C:\VirtualEnv\rokwire\lib\site-packages\connexion\decorators\parameter.py", line 115, in wrapper
    return function(**kwargs)
  File "C:\Workspace-rokwire\rokwire-building-blocks-api\contributions\api\controllers\contributions.py", line 206, in post
    send_email_new_contribution(reviewer['githubUsername'])
  File "C:\Workspace-rokwire\rokwire-building-blocks-api\contributions\api\controllers\contributions.py", line 989, in send_email_new_contribution
    adminutils.send_email(dataset[0]['email'], subject, message)
  File "C:\Workspace-rokwire\rokwire-building-blocks-api\contributions\api\utils\adminutils.py", line 79, in send_email
    connection.login(mail_from, password)
  File "C:\Python37\lib\smtplib.py", line 730, in login
    raise last_exception
  File "C:\Python37\lib\smtplib.py", line 721, in login
    initial_response_ok=initial_response_ok)
  File "C:\Python37\lib\smtplib.py", line 642, in auth
    raise SMTPAuthenticationError(code, resp)
smtplib.SMTPAuthenticationError: (535, b'5.7.3 Authentication unsuccessful [CH2PR14CA0038.namprd14.prod.outlook.com]')

error handled in SMTP exceptions

modifications done. ready for review

modified title to avoid confusion

Please get the latest from develop branch. Please see above. Thanks.

Hi, @minump, I just have a few more questions.  I will list them in separate comments.
Could you please add a boolean configuration variable so that we can turn email notifications off, say in the development instance? You could use one like the DEBUG config parameter. Thanks.

Currently POST /reviewers endpoint is slow because it's also trying to establish a connection during request handling. As a future improvement, maybe you could look into establishing a connection when the container starts and then use the existing connection to send the notification or something similar. What do you think?

Please also get the latest from develop branch. Thanks.

Hi, @minump, I just have a few more questions. I will list them in separate comments.
Could you please add a boolean configuration variable so that we can turn email notifications off, say in the development instance? You could use one like the DEBUG config parameter. Thanks.

NOTIFICATION_ENABLED env variable added. True value will send notifications.

Currently POST /reviewers endpoint is slow because it's also trying to establish a connection during request handling. As a future improvement, maybe you could look into establishing a connection when the container starts and then use the existing connection to send the notification or something similar. What do you think?

Yup. Can try that. Can first establish connection when contribution is started and then use it for sending notification for new reviewers and new contribution, then quit connection once contribution is closed. The issue with this might be that server connection might be idle for a longer duration and there might be a time limit and the server disconnects by itself. We can surely catch this error and reconnect again if necessary.

Currently POST /reviewers endpoint is slow because it's also trying to establish a connection during request handling. As a future improvement, maybe you could look into establishing a connection when the container starts and then use the existing connection to send the notification or something similar. What do you think?

Yup. Can try that. Can first establish connection when contribution is started and then use it for sending notification for new reviewers and new contribution, then quit connection once contribution is closed. The issue with this might be that server connection might be idle for a longer duration and there might be a time limit and the server disconnects by itself. We can surely catch this error and reconnect again if necessary.

Okay. Thanks. Yes, connections will need to be sustained. Please just create a follow-up feature at this point. We can look at the remaining tasks during the next sprint planning and decide the ones to work on. Thanks.

@minump, could you please also get the latest from the develop branch? Thanks.

Created new issue for the same

authors: minump sandeep-ps ywkim312

repo: rokwire/rokwire-building-blocks-api


text:
Error with readable-stream

When I use readable-stream with the rollup-plugin-commonjs it throws:
TypeError: The super constructor to "inherits" must not be null or undefined
    at Object.inherits (util.js:969:11)
    at Object.<anonymous> (/home/ttimm/dev/workspaces/rollup-readable-stream/dist/index.js:1129:8)
    at Module._compile (module.js:643:30)
    at Object.Module._extensions..js (module.js:654:10)
    at Module.load (module.js:556:32)
    at tryModuleLoad (module.js:499:12)
    at Function.Module._load (module.js:491:3)
    at Function.Module.runMain (module.js:684:10)
    at startup (bootstrap_node.js:187:16)
    at bootstrap_node.js:608:3

at runtime.
I've created a repo and tested it with babel & typescript.
https://github.com/TobiasTimm/rollup-typescript-readable-stream

In case someone else stumbles into this issue.  This isn't really a problem with rollup-plugin-commonjs and more just an issue between rollup not supporting all circular dependencies and readable-stream not removing the circular dependency because they are valid under the ES module spec.  They both have issues tracking this:
https://github.com/rollup/rollup/issues/1507
https://github.com/nodejs/readable-stream/issues/348
There is workaround mentioned in the first issue by timdp which replaces readable-stream with stream and it works most of the time for me.

Hey folks (this is a canned reply, but we mean it!). Thanks to everyone who participated in this issue. We're getting ready to move this plugin to a new home at https://github.com/rollup/plugins, and we have to do some spring cleaning of the issues to make that happen. We're going to close this one, but it doesn't mean that it's not still valid. We've got some time yet before the move while we resolve pending Pull Requests, so if this issue is still relevant, please @ me and I'll make sure it gets transferred to the new repo. :beer:

authors: shellscape tobiastimm westonpace

repo: rollup/rollup-plugin-commonjs


text:
componentWillUnmount is not a function

Hi,
Can you provide more info? Such as

React version
react-portal-tooltip version

Thanks

react: 15.4.2
react-portal-tooltip version: 0.14
I'm using a single page application. When I change routes the ToolTip crashes.

@elihofni Is it possible to make a Codesandbox showing the issue? I'm not able to replicate on my local.

https://codesandbox.io/s/542kny3r3p

Anyhow, I had to do a few wierd code to make the ToolTip suits to me, like, I wanted a content update inside it, so I had to hide-n-show it. (It would be cool to pass a prop for this)

Not understanding, can you explain? (Maybe even update the codesandbox? 😉)

Other point, I wished the ToolTip to show on click and hide on click somewhere out of it, so I managed a WatchOutsideClick component. (Would be nice to pass a prop for this too)

Probably won't be done as per #29

I'm closing this issue unless anything else comes up...
As far as the hide-n-show thing, please open a new issue for that if you want!

Cool, I've forked this repo and fixed the 'bugs' I said. As soon as possible I return the changes as a PR if you want

authors: Redmega elihofni tanqhnguyen

repo: romainberger/react-portal-tooltip


text:
Blur effect bug in the main Activity

Steps to reproduce
Open the main Activity to see the wallpaper in full
Lock the screen
Unlock again
The wallpaper gets blurred and stuck in that state.
After this it cannot be scrolled and double click does not work to unblur it.

Interestingly, I can't seem to reproduce this on the current master. Please reopen it if you see it crop up in the future (preferably in a beta build so that it can be fixed before going to production).

I can reproduce it with the latest beta. When is the next beta release from Master?

Muzei 2.5 Beta 1 is out now!

I got that too! But then if you switch to the Sources tab and back to Muzei, the blur goes away

Could it be device dependant? I just updated to latest beta and the bug is still here. I have OnePlus 5 with Oreo.

I'm not having any luck reproducing this or figuring out how this is happening. Could you try (in this order):

Restarting your device
Switching to a different wallpaper, then re-activating Muzei

Did both, still there. I will check with a different  device

authors: ianhanniballake maheshgaya tasomaniac

repo: romannurik/muzei


text:
🛑 V2RAY 🇺🇸 United States USF 2 is down

In 2888b8e, V2RAY 🇺🇸 United States USF 2 (https://usv-2.openv2ray.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: V2RAY 🇺🇸 United States USF 2 is back up in 5337173.

repo: roosterkid/opentunnel-status-server


text:
🛑 TROJAN 🇸🇬 Singapore SGO 1 is down

In efdf861, TROJAN 🇸🇬 Singapore SGO 1 (https://sgt-2.opensvr.net/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TROJAN 🇸🇬 Singapore SGO 1 is back up in 7576d27.


text:
🛑 XRAY 🇨🇦 Canada CAO 1 is down

In c3a7547, XRAY 🇨🇦 Canada CAO 1 (https://cax-1.openv2ray.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: XRAY 🇨🇦 Canada CAO 1 is back up in 6a01fe0.


text:
🛑 XRAY 🇺🇸 United States USF 1 is down

In e187cde, XRAY 🇺🇸 United States USF 1 (https://usx-1.openv2ray.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: XRAY 🇺🇸 United States USF 1 is back up in d7c8158.


text:
🛑 V2RAY 🇮🇩 Indonesia IDA 1 is down

In e9a4e79, V2RAY 🇮🇩 Indonesia IDA 1 (https://idv-1.openv2ray.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: V2RAY 🇮🇩 Indonesia IDA 1 is back up in b6e8269.


text:
🛑 V2RAY 🇮🇩 Indonesia IDA 2 is down

In 24f3845, V2RAY 🇮🇩 Indonesia IDA 2 (https://idv-3.openv2ray.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: V2RAY 🇮🇩 Indonesia IDA 2 is back up in 3679723.


text:
🛑 XRAY 🇬🇧 United Kingdom UKH 1 is down

In 39c17d7, XRAY 🇬🇧 United Kingdom UKH 1 (https://ukx-1.openv2ray.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: XRAY 🇬🇧 United Kingdom UKH 1 is back up in 4d9ae27.


text:
🛑 XRAY 🇨🇦 Canada CAO 1 is down

In eb4495f, XRAY 🇨🇦 Canada CAO 1 (https://cax-1.openv2ray.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: XRAY 🇨🇦 Canada CAO 1 is back up in ace3975.


text:
🛑 TROJAN 🇭🇰 Hong Kong HKE 1 is down

In 88b8ba5, TROJAN 🇭🇰 Hong Kong HKE 1 (https://hkt-1.opensvr.net/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TROJAN 🇭🇰 Hong Kong HKE 1 is back up in fe748a3.


text:
🛑 V2RAY 🇺🇸 United States USF 2 is down

In 914bd4f, V2RAY 🇺🇸 United States USF 2 (https://usv-2.openv2ray.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: V2RAY 🇺🇸 United States USF 2 is back up in 93701c0.


text:
acceptTap doesnt seem to work anymore

I installed 2.0 and acceptTap doesnt seem to close the drawer anymore. As I understand it, acceptTap allows you to close the drawer by tapping on the area of the screen that is not overlaid by the drawer.

Tried lots of things, but it really doesn't work. I see this happen in the phone log when I tap the area next to the open menu:
May  3 12:37:53 Marcs-MacBook-Pro assertiond[88244]: assertion failed: 15E56a 13C75: assertiond + 12188 [8CF1968D-3466-38B7-B225-3F6F5B64C552]: 0x1

Ok, got it working again. I had
panCloseMask={1}

Not sure if this worked before and stopped working after

Glad you got it working. Right now panCloseMask handles both integers (pixels) and decimals (treated as %). Do you think it would be more intuitive if we only accept one number format and allow the rest to be handled with callback style?

Not sure. But the way it is, is indeed counter intuitive

repo: root-two/react-native-drawer


text:
make default jenkins launcher file into template

This will use the hiera value jenkins_java_args to support upping memory limits for the jenkins process.
Fixes #90

Tested live