Bump version

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@7cd43d5). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #1   +/-   ##
=======================================
  Coverage        ?   70.00%
=======================================
  Files           ?        2
  Lines           ?       10
  Branches        ?        0
=======================================
  Hits            ?        7
  Misses          ?        3
  Partials        ?        0

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


Upgrade to Chromium 83.0.4103.97

Addresses #1059. I will let you know once I have a successful build.

My build succeeded; feel free to merge.


fix require library error

wow, you merged the other one pretty quick! 😁
this actually fixes it!
I found the answer here:
http://guides.rubygems.org/make-your-own-gem/#requiring-more-files/
Code for your package is placed within the lib directory. The convention is to have one Ruby file with the same name as your gem, since that gets loaded when require 'hola' is run. That one file is in charge of setting up your gem’s code and API.

so what happened is that, by convention, it tries to load the file lib/<name-of-gem>.rb so if you don't have it, it raises a LoadError


Create a login/signup page.

Adding an authorization page will be better for user to keep themselves registered here. Website quality will reach it's max . Please assign this task to me. I will create both the pages.

Like this.

Adding an authorization page will be better for user to keep themselves registered here. Website quality will reach it's max . Please assign this task to me. I will create both the pages.  Like this.

Oky I will assign you all the best


🚀 feat(apply): add parametric json configuration support

As a developer: I don't want to put my connector configs in the company repository with database passwords or sensitive data.
Description: Pass parameters to config for sensitive data when creating and updating the connector.
I add a new argument --config-parameters to apply command.
Let's say we have a connector config like the below. If I put this configuration in the repository, this does not allow for security. I can set sensitive parts to blank for the repository but if I put this configuration to pipeline or automation for generic configuration that will not work I must create many same configuration file just for one field and this is not cool.
{
  "name": "my-connector",
  "config": {
    "connector.class": "io.debezium.connector.mysql.MySqlConnector",
    "database.user": "debezium-user",
    "database.password": "password",
    ...
  }
}

So, I can put parameters to these fields and I can set them from the command line
{
  "name": "my-connector",
  "config": {
    "connector.class": "io.debezium.connector.mysql.MySqlConnector",
    "database.user": "$#1",
    "database.password": "$#2",
    ...
  }
}

And I can run this command;
kcctl apply -f test-connector.json -ca "debezium-user password"

Or I can use for duplicate fields
{
  ...
  "key.converter": "io.confluent.connect.avro.AvroConverter",
  "key.converter.schema.registry.url": "$#1",
  "value.converter": "io.confluent.connect.avro.AvroConverter",
  "value.converter.schema.registry.url": "$#1"
  ...
}

Hey @muhammedikinci, sorry for the late feedback, only seeing this now. It's an interesting proposal, I'm curious though what advantages you see over the secret provider mechanism built into Kafka Connect itself? In particular, when using that, you also won't expose any secrets when displaying connector configuration.

Thank you for your interest. You are right, actually, I didn't see the secret provider. It resolves all these problems.


Trigger watch on page load

Hi
Thanks for the great code!
Am I right to say that the viewport-watch is first triggered on scroll?
Is there any way to trigger it on page load?
Also, can you please give some examples of Manual watcher toggling with the controllerAs syntax?
Toda Raba!
Steven

for the moment I have resorted to
$window.scollTo(0,1)
in several critical pieces of my code - it's a bit ugly!
Thanks again for the great code, it made my sluggish code super fast.
I'm still learning but at some point I'll try and solve this issue for you and put in a pull request.


Update limits

Preview is available here:http://developers-pr996.previews.kube001.services.stellar-ops.com

Core PR for the new limits, https://github.com/stellar/stellar-core/pull/4491

Preview is available here:http://developers-pr996.previews.kube001.services.stellar-ops.com

Preview is available here:http://developers-pr996.previews.kube001.services.stellar-ops.com

@anupsdf I think that's probably a good idea. We can split them up in a separate PR.

Preview is available here:http://developers-pr996.previews.kube001.services.stellar-ops.com

We probably should merge this PR on October 9th when the new limits go live.


sort.rs: do not change the output delimiter

This commit makes sort to use the same delimiter
in the output as in the input.
The particular use-case was to sort a passwd file,
where the delimiter must be a colon (:).

This change is

What do you think about introducing a new option for the output delimiter?
(While I still consider this as a sneaky side-effect, I can accept your argument about the sane defaults.)

What do you think about introducing a new option for the output delimiter?

That's what xsv fmt is for.

(While I still consider this as a sneaky side-effect, I can accept your argument about the sane defaults.)

To do otherwise is untenable without some other form of configuration (such as an env var) that applies to all xsv commands in the pipeline. With the way xsv currently works, you need to, in some cases, modify the delimiter in the first command and if the standard format is undesirable (probably unlikely), then you can use xsv fmt to reformat it. In some cases, you may need xsv input at the beginning of a pipeline for more archaic formats that require changes other than just the delimiter. But still, it's always at the beginning and end, and otherwise everything Just Works.

Hmph, I thought this side effect will mess up my passwd file, but with xsv fmt it works perfectly (i.e. the unnecessary quotations were removed).
Thanks for the clarification, in this case the pull request is no longer relevant, and I'm closing it.


Command window shows list of submodules after write command

If I open a file inside a git repository with submodules and then type :w<CR> or :write<CR>, then the command window closes for a split second and then shows a list of all submodules

For some reason it works fine with other commands like :wall.

Thanks for logging the issue @mabuchner ! That would be... very annoying.
It looks like it might be coming from the extension host. Just curious - is this an open source project I can try to repro on? And what extensions are you using?

boost is a bit infamous for it's use of submodules:
git clone git@github.com:boostorg/boost.git
cd boost
git submodule update --recursive --depth=1 --init
oni2 .

Here it takes a few seconds before the problem starts to occur. As soon as the status bar shows some git hashes it happens.

Just open any file (e.g. index.htm) and type :w<cr>.

I currently only have the following plugin for CMake support installed:
https://marketplace.visualstudio.com/items?itemName=twxs.cmake


[TC-SU-2.1] - SDK needs to implement the feature of  queryImage command sent once during the required time

Test case [TC-SU-2.1] - is not verifying all the test step mentioned in the test paln.step no.2 needs to be implement in the SDK.
Test step needs to be implement  - We have to verify that the OTA-P sends the  QueryImage command  only once in that 1 minute interval.

Fixed here: https://github.com/project-chip/connectedhomeip/issues/13782


SS-45255: Remove empty spaces at the horizontal right end of the sequence block

JIRA: SS-45255
Primary: @karry08
This removes extra empty spaces at the right end of the sequence block, giving more space for the viewer.
Screenshots
Before:

After:

LGTM

Merged locally with 98ad554a35e88644b66378624f0954dfbafaa989.


Upload stream and get response body

I'm having problem with uploading a stream and reading response body. I tried many combinations for couple of days but nothing worked as expected. This is my function:
    postFile(url: string, filePath: string): Promise<any> {
        return new Promise((resolve, reject) => {
            const post = request.post({
                url: url,
                headers: this._headers,
                jar: this._jar,
                simple: false,
            });
            post.on('response', function(response){
                console.log(response.body);
                resolve(response.body);
            });
            fs.createReadStream(filePath).pipe(post);
        });
    }

The problem here is that response.body returns empty. The shocking part was that when I try to listen it with fiddler and add proxy: 'http://<IP_ADDRESS>:8888' to the options, then the response body returns as expected, I have no idea why it does... Hope you can help me because I searched both request and request-promise issues and found nothing. All I need is uploading a file as stream and get response body.
Edit: At first I tried this, which should also work:
    postFile(url: string, filePath: any): Promise<any> {
        return request.post({
            url: url,
            headers: this._headers,
            jar: this._jar,
            simple: false,
            body: fs.createReadStream(filePath),
        });
    }

But it doesn't, simply returns empty body. Only thing that worked was body: fs.readFileSync(filePath) which reads the file as a buffer, not suitable for my case. I need to upload large files like ~50MB.

The second code snippet in this example uploads a file using a stream. Does this help?

That example uploads using multipart upload. The host I am uploading requires whole body as a file stream with Content-Type: image/jpeg, thus I have to pass stream as a body object. I don't understand why passing buffer to body works but stream does not. I'm totally stuck at this point.

Seems like a request issue. Closing this...
https://github.com/request/request/issues/3108

Indeed, request itself would need to add support for that. But since you need to pass the full image in the body, it means that you have to have the image fully in your memory. So a stream doesn’t really add any benefits. Instead, you could just use fs.readFile(...) and take care of the encoding.

Well for uploading image, I could use that, but in another request I need to upload a video... I hope they can provide me a workaround.


Passing data to lightbox

[x] Feature Request.

I want to pass data attribute to the lightbox, would be nice if i can expand the attributes.
Like:
Open

My fault, it already works 👍


TextInput.hint_text too long for TextInput.size_hint_x locks the app

If I have a TextInput whose size_hint_x makes it too small to hold all its hint_text, my app freezes (apparently in an infinite loop) instead of truncating the hint_text as I'd expect.
Steps to reproduce:

Make a TextInput, give it long hint_text and small size_hint_x (or put it in a BoxLayout so that a "large" size_hint_x still makes it too small)
Launch app

Expected behavior: get an app window that I can interact with
Actual behavior: app eats 100% CPU and doesn't display anything
Example code:
from kivy.base import runTouchApp
from kivy.lang import Builder
from kivy.uix.boxlayout import BoxLayout

class TestBoxLayout(BoxLayout):
    pass

Builder.load_string("""
<TestBoxLayout>:
    orientation: 'vertical'
    Widget:
        id: foo
    Widget:
        id: bar
    BoxLayout:
        TextInput:
            hint_text: 'Some hint text'
            size_hint_x: 0.5
        Widget:
            id: spam
        Widget:
            id: eggs
""")

if __name__ == '__main__':
    runTouchApp(TestBoxLayout())

It works as expected with 1.9.1.

Introduced by https://github.com/kivy/kivy/pull/4186.
@kived ping.

@kived has found the buggy commit: 29e7f851a8ae58ba018c9a0a3d639b8a20c0b69e.

This is happening again, but I made (I think) a more general fix in #4445


refs #61 Handle simple text response.

Pull Request Test Coverage Report for Build 60

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 86.598%

Totals

Change from base Build 58:
0.4%

Covered Lines:
64

Relevant Lines:
69

💛  - Coveralls


Why is query execution time so high ?

Why does same query take around 300 ms more than actual execution time when using Mysql native driver for Nodejs , even with or without using "create pool" options?
Please refer highlighted section in below attached screenshot

Also for native driver execution time, please see below attached screenshot:

Codebase for node.js Mysql Native driver
db.js

var mysql = require('mysql');
var connectionpool = mysql.createPool({
    connectionLimit: 100, //important
    host: 'localhost',
    user: config.development.username,
    password: config.development.password,
    database: config.development.database,
    multipleStatements: true,
});

    exports.getConnection = function(callback) {
        connectionpool.getConnection(callback);
    };

emp.js
var connections = require('../config/db');
     var pre_query = new Date().getTime();
        var sqlstatements = "select  * from employees where recordstate<>'raw'and  DATE(createdAt) " + "between ('1982-03-24') and ('2017-04-23')    order by employeesID desc limit 20 offset 0;" + "select COUNT(*) as count   from employees where recordstate<>'raw'and  " + "DATE(createdAt) between ('1982-03-24') and ('2017-04-23')    order by employeesID desc";
        connections.getConnection(function(err, c) {
            console.log(sqlstatements)
            c.query(sqlstatements, function(err, projects) {

                console.log(err);
                if (!err) {
                    c.release();

                    var red = new Object();
                    red.rows = JSON.parse(JSON.stringify(projects[0]));
                    red.count = JSON.parse(JSON.stringify(projects[1]))[0].count;
                    var post_query = new Date().getTime();
                    // calculate the duration in seconds
                    var duration = (post_query - pre_query) / 1000;
                    console.log(duration)
                        // console.log(red);
                    res.json(red);

                }
            })
        })

Interesting, I'm not sure. Unless you can track down where exactly in the code the time is being taken, I can do it, but need to get enough information to set up the same thing you have there, if you can provide the details.

Hi @dougwilson ,
thanks for response and i think the setup would be any node.js version and  provided code  is decoupled , which in this scenario makes it pretty straight forword .
make two files db config.js and emp.js and place the  provided code according and only thing missing is mysql table dump which i am providing right now.
please refer below gdrive link for the same
https://drive.google.com/file/d/0B_sJE1z-87iTVXpISGFxZmlrRTA/view?usp=sharing

Thanks, @matt212 ! I don't have anything to open .rar files, can you provide in a more common format? Perhaps just a .zip?

@dougwilson ,
Sure , please refer below zip containing employee data for evaluation purpose
https://drive.google.com/file/d/0B_sJE1z-87iTM0ViZTRWVjFaN1E/view?usp=sharing

Thanks! While I was waiting, I did as you instructed and created a db.js with your code and a emp.js with the code. I tried to run emp.js and got the following error. What am I doing wrong?
$ node emp
module.js:471
    throw err;
    ^

Error: Cannot find module '../config/db'
    at Function.Module._resolveFilename (module.js:469:15)
    at Function.Module._load (module.js:417:25)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/mysql-1706/emp.js:1:81)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)

it require path of config db.js thats it !
make config folder or place where-ever one would seems fit and place the path in "require" statement !
please refer db.js code snippet

Ok, so I changed the first line of emp.js to var connections = require('./db'); and then I installed npm install mysql, but now I get the following error:
$ node emp
/mysql-1706/db.js:5
    user: config.development.username,
          ^

ReferenceError: config is not defined
    at Object.<anonymous> (/mysql-1706/db.js:5:11)
    at Module._compile (module.js:570:32)
    at Object.Module._extensions..js (module.js:579:10)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Module.require (module.js:497:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/mysql-1706/emp.js:1:81)
    at Module._compile (module.js:570:32)

hi @dougwilson  ,
you need to put your mysql credentials  there in place of "config.development" part of db.js code snippet !

Yes, I see now. I don't really have a lot of time right now, I thought it would be simple to run, but seems like I have to struggle to run, get error, fix error, run, get error, fix error, over and over. I'm happy to take a look, but please just provide the necessary instructions with complete, runnable code and when you do that, I'll come back another day and take a look. Thanks!

I tried messing around a little more before heading out, and got it to run, but I'm seeing an execution time of 0.177 (177 ms), which is way less than 300ms. I'm a bit unclear on what I'm supposed to be looking for, can you explain a bit more on what I'm supposed to be seeing when running your code / what you're trying to figure out? Here is the output of running the emp.js file against the dump you provided:
$ node emp
select  * from employees where recordstate<>'raw'and  DATE(createdAt) between ('1982-03-24') and ('2017-04-23')    order by employeesID desc limit 20 offset 0;select COUNT(*) as count   from employees where recordstate<>'raw'and  DATE(createdAt) between ('1982-03-24') and ('2017-04-23')    order by employeesID desc
null
0.177
/mysql-1706/node_modules/mysql/lib/protocol/Parser.js:79
        throw err; // Rethrow non-MySQL errors
        ^

ReferenceError: res is not defined
    at Query._callback (/mysql-1706/emp.js:21:21)
    at Query.Sequence.end (/mysql-1706/node_modules/mysql/lib/protocol/sequences/Sequence.js:86:24)
    at Query._handleFinalResultPacket (/mysql-1706/node_modules/mysql/lib/protocol/sequences/Query.js:137:8)
    at Query.EofPacket (/mysql-1706/node_modules/mysql/lib/protocol/sequences/Query.js:121:8)
    at Protocol._parsePacket (/mysql-1706/node_modules/mysql/lib/protocol/Protocol.js:280:23)
    at Parser.write (/mysql-1706/node_modules/mysql/lib/protocol/Parser.js:75:12)
    at Protocol.write (/mysql-1706/node_modules/mysql/lib/protocol/Protocol.js:39:16)
    at Socket.<anonymous> (/mysql-1706/node_modules/mysql/lib/Connection.js:103:28)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:188:7)

And if I move the var pre_query = new Date().getTime(); line down so it doesn't include the time setting up the TCP connection, handshaking with the MySQL sever, and performing the authentication and setting setup, I get that the query took 0.013 seconds. Here is the emp.js file I'm using for that:
var connections = require('./db');
        var sqlstatements = "select  * from employees where recordstate<>'raw'and  DATE(createdAt) " + "between ('1982-03-24') and ('2017-04-23')    order by employeesID desc limit 20 offset 0;" + "select COUNT(*) as count   from employees where recordstate<>'raw'and  " + "DATE(createdAt) between ('1982-03-24') and ('2017-04-23')    order by employeesID desc";
        connections.getConnection(function(err, c) {
     var pre_query = new Date().getTime();
            console.log(sqlstatements)
            c.query(sqlstatements, function(err, projects) {

                console.log(err);
                if (!err) {
                    c.release();

                    var red = new Object();
                    red.rows = JSON.parse(JSON.stringify(projects[0]));
                    red.count = JSON.parse(JSON.stringify(projects[1]))[0].count;
                    var post_query = new Date().getTime();
                    // calculate the duration in seconds
                    var duration = (post_query - pre_query) / 1000;
                    console.log(duration)
                        // console.log(red);
                    res.json(red);

                }
            })
        })

And it's output:
$ node emp
select  * from employees where recordstate<>'raw'and  DATE(createdAt) between ('1982-03-24') and ('2017-04-23')    order by employeesID desc limit 20 offset 0;select COUNT(*) as count   from employees where recordstate<>'raw'and  DATE(createdAt) between ('1982-03-24') and ('2017-04-23')    order by employeesID desc
null
0.013
/mysql-1706/node_modules/mysql/lib/protocol/Parser.js:79
        throw err; // Rethrow non-MySQL errors
        ^

ReferenceError: res is not defined
    at Query._callback (/mysql-1706/emp.js:21:21)
    at Query.Sequence.end (/mysql-1706/node_modules/mysql/lib/protocol/sequences/Sequence.js:86:24)
    at Query._handleFinalResultPacket (/mysql-1706/node_modules/mysql/lib/protocol/sequences/Query.js:137:8)
    at Query.EofPacket (/mysql-1706/node_modules/mysql/lib/protocol/sequences/Query.js:121:8)
    at Protocol._parsePacket (/mysql-1706/node_modules/mysql/lib/protocol/Protocol.js:280:23)
    at Parser.write (/mysql-1706/node_modules/mysql/lib/protocol/Parser.js:75:12)
    at Protocol.write (/mysql-1706/node_modules/mysql/lib/protocol/Protocol.js:39:16)
    at Socket.<anonymous> (/mysql-1706/node_modules/mysql/lib/Connection.js:103:28)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:188:7)

@dougwilson ,
if you refer mysql workbench query it is zero ms in nodejs object it is coming to near 285 to 300 ms in my machine ,

another note
thanks for pointing out below
a. time setting up the TCP connection
b. handshaking with the MySQL sever
c. performing the authentication and setting setup
what is tentative ratio of query execution in mysql database and nodejs object  including aforementioned factors
ratio example :-   1ms in mysql database then 2 ms in nodejs object

I don't have MySQL workbench, so in order to compare the times, can you explain exactly what they are including in their times? In the 0.013 seconds I posted above, that includes allocating memory in Node.js, then constructing the packet buffers to send to the MySQL server, then the network transfer time of sending the packet, then the query execution time, then the time of the results transfer back, then more memory allocation in Node.js, then trying to parse the received packets, then the time to cast the types into native JavaScript types, and the overheads with executing in a non-compiled language. It's not clear to me what you're trying to compare, unless you're asking where the 13 ms of overhead is coming from, but to do all of that in 13 milliseconds seems good to me. Do you have any suggestions of where to optimize out the 13 milliseconds from the Node.js code? From timing on my machine, 10ms of that was the network transfer time, leaving only 3 milliseconds within Node.js itself.

As for why you're getting near 285 to 300 ms on your machine, I really have no idea. I asked if you could explain how to get the same setup, but you said it didn't matter, so I just used what I had at hand. Without being able to reproduce the near 285 to 300 ms or get access to your machine, I don't really have a good way to determine where that time is coming from. I just ran the code you provided.

what is query is in a nutshell
a.why is native driver for node.js slower as compared to query execution in database.
b.what would be tentative ratio of query execution in mysql database and nodejs native driver  including below factors

time setting up the TCP connection
handshaking with the MySQL sever
performing the authentication and setting setup
4.allocating memory in Node.js,
5.then constructing the packet buffers to send to the MySQL server,
then the network transfer time of sending the packet,
then the query execution time, then the time of the results transfer back,
then more memory allocation in Node.js,
9.then trying to parse the received packets,
10.then the time to cast the types into native JavaScript types,
if I had any pointers, i would have mentioned in post !
like 1 ms in mysql server and 2 ms in nodejs object due to above afoementioned factors
Kinda curious , there has to be some standard ratio !

I already provided the details. If you're looking for the specific time on your machine, you'll need to run a profiler on your Node.js application. Here is the answers for all those on my machine from my run of 117ms which included all the steps you mentioned:

30ms
12ms
62ms
<1ms (anything taking les than 1ms just shows as 0ms)
<1ms
4ms
<1ms then 6ms
<1ms
<1ms
<1ms

Since all those <1ms added up into 3ms total, it's a bunch of fractions spread around.
I hope this helped!

Another aspect which i have observed when query execution after idle time it takes more time , then
after it executes like seamlessly any pointers.

Another aspect which i have observed when query execution after idle time it takes more time , then after it executes like seamlessly any pointers.

Hm, I'm not sure. If you can provide some code to run with details instructions so I can reproduce, I can then see where the time is being used.

plus config of your machine and OS ?

Woah , that is some Heavy Rig,  Kudos , but also  thanks for taking out time for explaining the intricate details of functional aspect of native driver !
Sure give me  a day or two will share across project via bitbucket , case in point you can provide me with your   bitbucket username or emailid in order for me to share the repo !

Hi @matt212 yea, just whatever I was issued for work. Unfortunately I do not have a bitbucket account, I'm just here on GitHub.

shoot, no worries will share it through gdrive in  .zip archive mode .


Pass checkstyle properties through

Needed for warnings-ng change

Should it be propagated to the Plugin POM?

Should it be propagated to the Plugin POM?

it's a little bit weird here, if you don't set them explicitly it seems to works fine.
I tried on azure-keyvault plugin as well and that one doesn't set the config, possibly a fix would have been to remove the config but I didn't find that till after.
different ways of solving it i guess.

plugin pom doesn't have checkstyle afaik?

plugin pom doesn't have checkstyle afaik?

Looks like we removed the checkstyle profile at some point.
I think it would be great to add it again so that plugin maintainers can easily enable it if needed (e.g. a profile with checkstyle rules file activator, or a property managing the skip flag)


Feature Request: Delete Media?

Any thoughts on deleting media as well?

That would be a nuce feature to have as well I imagine.

this was implemented in #8


Add lpthread flag to support systems that have implicitly pthread

Android's bionic comes with pthread built in, and hence -lpthread is invalid. This adds a flag, such that
cabal -f-lpthread, allows this package to be cross compiled to android.

Thanks!
I wonder if this sort of thing would be better left to an autoconf script? That way, we wouldn't need to force cross-compiler users to turn on an extra flag.

By the way, I fixed the Travis failures in acf1b3bb4e8ddd3466a20904ccddd7dc2e3329d8. Can you rebase your PR on top of that?

I'll try to add this via autoconf. The AX_PTHREAD macro should work I hope. Will also do the rebase.

What is the status of this, @angerman?

If it would be more convenient for you, we could merge this as-is, and defer the autoconf stuff for later. But I'd hate to leave this PR lingering (there are also conflicts with master now), so we should decide one way or the other.

Closing due to inactivity (see also tweag/inline-java/pull/70 and tweag/inline-java/issues/92).


Starting with cli

Hello,
I am starting with cli package. Can you share few example (Git URL) command line utilities built with cli package.
I am new to Go.
This will help with understanding correct usage of cli.
Appreciate your time.

Hi @deepujain,
You can see a complete list of public projects that import cli here: https://godoc.org/github.com/urfave/cli?importers
I poked at a couple and might recommend https://github.com/nicerobot/gister/blob/master/gister.go as a simple usage to start with.
Let me know if this helps!

Given that this is from last year, I think I'm comfortable closing it 🙂 feel free to re-open / open a new issue / comment in support if there's still interest here!

I did end up using it in my project. Fantastic work.
On Sat, Aug 17, 2019 at 2:24 AM lynncyrin<EMAIL_ADDRESS>wrote:

Closed #577 https://github.com/urfave/cli/issues/577.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/urfave/cli/issues/577?email_source=notifications&email_token=AADDJ6PGUQJBPGA7JGW6NNDQE67VFA5CNFSM4C2JJNP2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOTDI2T4Q#event-2563877362,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADDJ6JR3RFUSHZ6TIEVGODQE67VFANCNFSM4C2JJNPQ
.

--
Deepak


Customise AEM package name based on the profile

Hi,
is there anyway to customise CRX package name based on profiles like dev, qa or prod ?
like how we have below for maven plugin 👍

Thanks,
Venkat

in Gradle you have almost unlimited ways to do it.
common practice of users using GAP is sth like this
aemCompose {

def profile = project.findProperty('aem.profile')

classifier = profile // this will add a suffix to zip file name
// see https://docs.gradle.org/current/dsl/org.gradle.api.tasks.bundling.Zip.html#org.gradle.api.tasks.bundling.Zip:archiveName for other fields that you can use here and wanna adapt

if (classifier == 'content) {
    includeProject ':content:demo'
    includeProject ':content:init'
} if (classifier == 'app) {
    includeProject ':app:core'
    includeProject ':app:common'
} else {
    includeSubprojects()
    /* same as specified explicitly
    includeProject ':app:core'
    includeProject ':app:common'
    includeProject ':content:demo'
    includeProject ':content:init'
    */
}

}

Please let me know if it is ok for you.

Hi, i am asking about the package name - ${build.project.name}-${build.package.app}-v${project.version}
Currently it is creating based on groupid and artfifact id but is there any way to control that based on profile ?

in gradle there is no such thing like profile. you have properties that comes from gradle.properties file or command line. you could also dynamically read properties by using if statement and `apply from 'my.properties'. what you will use is all upon your preferences.
package name could be directly set as
aemCompose {
   archiveName = "${project.group}-${project.name}-${project.version}.${ext}" // enforcing name directly
   baseName = 'aaa' // will replace "${project.group}-${project.name}' to 'aaa'
}

according to your case, I am guessing that you want to change conditionally baseName in case of command line parameter or parameter set on environment at location ~/.gradle/gradle.properties
please read https://docs.gradle.org/current/userguide/build_environment.html

closing, I hope my anwer helped you. if you need sth more, don't hesitate to reopen :)


[multistage][cleanup] clean up PhysicalPlanContext usage

PhysicalPlanContext contains lots of stage execution metadata
it is being instantiated from OpChainExecutionContext as well as from ServerPlanRequestContext
both should only encapsulate the object instead of knowing the constructor. Otherwise it is creating 3 times more code changes when PhysicalPlanContext needs modification.
This PR cleans it up

Codecov Report

Merging #10950 (2c80f45) into master (84bc45f) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##           master   #10950     +/-   ##
=========================================
  Coverage    0.11%    0.11%
=========================================
  Files        2188     2134     -54
  Lines      117758   115251   -2507
  Branches    17796    17494    -302
=========================================
  Hits          137      137
+ Misses     117601   115094   -2507
  Partials       20       20

Flag
Coverage Δ

integration1temurin11
?

integration1temurin17
?

integration1temurin20
?

integration2temurin11
?

integration2temurin17
?

integration2temurin20
?

unittests1temurin11
?

unittests1temurin17
?

unittests1temurin20
?

unittests2temurin11
?

unittests2temurin17
?

unittests2temurin20
0.11% <0.00%> (ø)

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

Impacted Files
Coverage Δ

...va/org/apache/pinot/query/runtime/QueryRunner.java
0.00% <0.00%> (ø)

...ot/query/runtime/plan/OpChainExecutionContext.java
0.00% <0.00%> (ø)

.../pinot/query/runtime/plan/PhysicalPlanContext.java
0.00% <0.00%> (ø)

...runtime/plan/pipeline/PipelineBreakerExecutor.java
0.00% <0.00%> (ø)

.../runtime/plan/server/ServerPlanRequestContext.java
0.00% <0.00%> (ø)

...ry/runtime/plan/server/ServerPlanRequestUtils.java
0.00% <0.00%> (ø)

.../runtime/plan/server/ServerPlanRequestVisitor.java
0.00% <0.00%> (ø)

.../pinot/query/service/dispatch/QueryDispatcher.java
0.00% <0.00%> (ø)

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


[RFC-62] Diagnostic Reporter

Change Logs
With the development of hudi, more and more users choose hudi to build their own ingestion pipelines to support real-time or batch upsert requirements.
Subsequently, some of them may ask the community for help, such as how to improve the performance of their hudi ingestion jobs? Why did their hudi jobs fail? etc.
For the volunteers in the hudi community, dealing with such issue, the volunteers may ask users to provide a list of information, including engine context, job configs,
data pattern, Spark UI, etc. Users need to spend extra effort to review their own jobs, collect metrics one buy one according to the list and give feedback to volunteers.
By the way, unexpected errors may occur at this time as users are manually collecting these information.
Obviously, there are relatively high communication costs for both volunteers and users.
On the other hand, for advanced users, they also need some way to efficiently understand the characteristics of their hudi tables, including data volume, upsert pattern, and so on.
In order to expose hudi table context more efficiently, this RFC propose a Diagnostic Reporter Tool.
This tool can be turned on as the final stage in ingestion job after commit which will collect common troubleshooting information including engine(take spark as example here) runtime information and generate a diagnostic report json file.
Or users can trigger this diagnostic reporter tool using hudi-cli to generate this report json file.
Impact
no impact
Risk level: none | low | medium | high
Choose one. If medium or high, explain what verification was done to mitigate the risks.
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

Could we also mention the need of obfuscation of enterprise informations such ip address, hoatname, bucket, tables or columns names and so on ?

Hi @codope And @YuweiXiao Really appreciate for your efforts here!
All comments are addressed. PTAL!

Could we also mention the need of obfuscation of enterprise informations such ip address, hoatname, bucket, tables or columns names and so on ?

Sure we can collect these info such as schema in this report!

@zhangyue19921010 thanks for taking this up! Some high level thoughts:

hudi commit metadata vs hudi metrics: if users enable diagnostic reporter, should we have a config to include metrics reporter's data? metrics system is good at showing the trends but hard to cross-check against commit metadata. so regardless of enabling metrics reporter or not, diagnostic reporter can collect metrics and save to report dir, just like a csv/json metrics reporter. We can also refine what goes to metrics and what goes to commit metadata, to keep the responsibilities clear and reporting data organized.
consolidate with error table: RFC-20 this is a long-pending feature that also aims to assist investigation. diagnostic reporter should be aware of error table settings and zip the error table if configured so. Size could be a concern, so configuration can be given to zip the whole table, or sample records, or skip error table completely. Also it requires some config to allow masking any fields. Taking a step further, we can also make error table one of the diagnostic reporting features. They have similar storage structures: can be local to the hudi table or global to the whole platform.
work with metadata table: you've already mentioned collecting stats by listing the file system. diagnostic reporter should also be aware of the presence of metadata table and zip the table or extract relevant data - fallback to file system listing if not present.

Thanks @xushiyan for your advice!
Will have a deep look and expand this rfc asap!


Add test watcher package

Description
Add a test watcher mix package: https://github.com/lpil/mix-test.watch
Checklist

[x] Everything passes when running mix test 😏
[x] Ran mix format
[x] No frontend compilation warnings

nice, seems useful!


Add sign-in section on home page

I want to do under JWOC

I want to work on this issue under JWOC.

I would like to work on this issue under JWOC.

I want to work for on this issue under jwoc , please assign me

Sorry for the delay from my end in assigning the issue, @Keshab1113. If anybody wants to work on this issue, reply to this message and I'll assign the issue.
Also, can you be a bit clearer on the changes you suggest?


[move] Fix typo for ( in TextMate language

Due to some lazy copy-pasting on my part, the TextMate language incorrectly tokenized the ( in <identifier>( as "less", rather than correctly as "left-parenthesis".

/land

:broken_heart: Test Failed - ci-test

/land


Setup automated testing using Appveyor and Travis CI

we do have a travis script set up for rudimentary testing - which is unfortunately broken at the moment. @michaeltryby has a better testing framework that he's said he would like to contribute, so i suggest waiting to hear from him on that issue.

@AbelHeinsbroek @samhatchett I have some work on numerical regression testing that I could share. It's based on python nrtest. It's pretty much the same thing I just checked in for SWMM except for EPANET. It works great for benchmarking engine work locally. We will have to adapt it to the CI workflow but I think its a solid foundation for doing that.
Sorry for the delay. I am officially working on EPANET now.

The tools needed to run nrtest are staged here. I can checkin a stub for epanet-nrtestsuite as well.
I've done a little refactoring on the outputapi cause I wasn't doing the opaque pointer the right way. I can also switch the python wrapper from ctypesgen to SWIG. What do you think?

Turns out handling opaque pointers in SWIG is a bit tricky: SWIG Opaque Pointer

thinks for digging into this @michaeltryby ! while not trivial, at least it's possible with swig. I say we go ahead with it. based on some of the discussions here, it sounds like using swig as the generic wrapper interface is a good move. Since that is perhaps a separate issue from just doing automated tests, I will create a new issue for that and tag it on the roadmap. Still open to discussion of course, if anyone has feedback!

@samhatchett I would like to check in my current nrtest tooling. Do you have a preference what branch I pull against?

@michaeltryby please update from feature-nrtest and PR to that branch. thx 😉

nrtest works great for bench marking EPANET locally. I believe we will have to resolve the following issues in order to get nrtest running under CI:

Comparison tolerance rtol and atol for nrtest are specified from the command prompt. It would be better if comparison tolerances were persisted with the test descriptions. Values passed from the command prompt should override those defined with the test description. So either nrtest needs to be forked and modified or the run script called from CI can call each test individually with tolerances.

To run an executable nrtest requires the absolute path to be defined in the app description or it must be on the system path. nrtest is designed to run a setup script prior to calling the executable. To run under CI the setup script would need to add the executable to the system path.

Python based EPANET test tools need to be built, packaged, and installed prior to running nrtest.  The Python environment on the CI server has to be configured to satisfy test tool dependencies.

Write a run script for the CI server to execute (see above).

I'm sure there are others I can't think of ...

Thoughts?

looks like this issue is basically resolved. further enhancements to testing can be fielded in other issues. thanks all!


sample run gives No sequences found matching given filter criteria

The environment of my computer is Red Hat Enterprise Linux Server release 6.2 (Santiago), perl 5.10.1, ncbi-blast+2.2.30 and samtools1.2.
I ran command proovread --sample-run in bash. It gave out the following error messages several times:
Can't open perl script "/public/opt/sc/program/proovread/bin/SeqChunker-perl": No such file or directory
but continued running. At last it gave out the following message:
#------------------------------------------------------------------------------#
[17:30:44] No sequences found matching given filter criteria
[15-05-13 17:30:44] [siamaera] Input neither FASTA nor FASTQ
#------------------------------------------------------------------------------#
[Wed May 13 17:30:44 2015] Cleaning up
Wed May 13 17:30:44 2015] Total time spent 00:00:37
[Wed May 13 17:30:44 2015] Done
file pr-Fa.chim.tsv and pr-Fa.trimmed.fq has nothing in it, and pr-Fa.untrimmed.fq has all the sequences. Mask rate is 0.0%.
I don't know what to do now. Can you give me any suggestions?

There seems to be a problem with resolving the actual path to the SeqChunker-perl binary. First, give this dirty quick fix a try:
cd /public/opt/sc/program/proovread/bin/
ln -s ../util/SeqChunker/bin/SeqChunker-perl

Now, let's try to get to the real problem. The location of SeqChunker-perl is determined by proovread/bin/SeqChunker, which actually should be a symlink to ../util/SeqChunker/bin/SeqChunker. I can think of two reasons causing your problem:

Check if proovread/bin/SeqChunker really is a symlink on your system. If you used some form of reverse coping on proovread it is possible that the symlink wasn't preserved and instead the actual file was copied. It happens for example with scp -r, try to use rsync instead (https://smaftoul.wordpress.com/2008/08/13/recursive-scp-and-symlinks/).
The determination of the actual location of the SeqChunker binary fails on your systems for some reason - but let's look into this if 1. does not pan out.

I'm assuming you were able to resolve the problem and hence will close the issue.
Cheers

Sorry. Yes the rsync method is working and the sample run has passed. I has been running the project data and it consumes much time and make me wait a long time.
Sorry again.


[Recoverable Error] Argument 1 passed to SilverStripe\Config\MergeStrategy\Priority::mergeArray() must be of the type array

Just installing a fresh version of SS4 on my virtual machine via composer, everything working fine in terms of installing and website loading - but when I try to login to the CMS I am getting this error:
[Recoverable Error] Argument 1 passed to SilverStripe\Config\MergeStrategy\Priority::mergeArray() must be of the type array, string given, called in /var/www/src/vendor/silverstripe/framework/src/Core/Config/Middleware/InheritanceMiddleware.php on line 43 and defined
GET /Security/login?BackURL=%2Fadmin%2Fpages%2F
Line 59 in /var/www/src/vendor/silverstripe/config/src/MergeStrategy/Priority.php
Is this a known issue?

Do you have any custom YML config or modules?
What's the output of composer show?

No feedback in 11 days, closing


Relative/absolute record name conversions

Hi,
Over in the libdns repo, we're discussing standardizing the use of relative or absolute record names in libdns.Record structs. The current consensus is that they should be made relative (i.e. for a record named sub.example.com in a zone called example.com, the name should become sub). This may affect your provider implementation.
When you have a chance, would you please check out this issue? This is the last call for feedback before we decide and I push the changes, and I want to make sure we get it right for the majority of DNS provider implementations so we don't have to change it again.
Link: https://github.com/libdns/libdns/issues/12#issuecomment-756135908
Thank you!

Thanks for your notification. I'll stay tuned with the conversation of libdns.Record.Name.

@viscropst just a reminder that this needs to be addressed. The changes to libdns have been merged, and the latest beta of Caddy uses the changes. This library needs an update to be made compatible, I think. A user seems to be having trouble with it: https://caddy.community/t/how-to-make-caddy-use-not-443-port-for-https/12314

Okay,I'll address and solve this later. Thanks for reminding!

(Sorry, forgot about this issue!)
After much discussion we settled on using relative names with conversion functions.
Thanks for maintaining your libdns repo!


Add time slicing to Sequence and DataCollection

This PR fixes #161.
I fixed the slicing issue by adding custom select_by_time() methods to DataCollection and Sequence.

Looks good to me (although do see my comment on TimeIndexer). @wbaccinelli ?

Looks good, I also approved the changes.

If you agree, @DaniBodor and @wbaccinelli , I'll go ahead and merge this one.

If you agree, @DaniBodor and @wbaccinelli , I'll go ahead and merge this one.

Go ahead


[Bug]: Pan/Tilt Camera doesn't show stream

What happened?
After installing and configuring the plugin using ADMIN guest credentials, Homebridge shows the attached log error. No stream is visible, but a "home security" accessory is added in Homekit.
Toggling the "Livestream from Camera device" has no visible effect.
Some additional details:
-Using the Homebridge docker image to run it.
-Using 64-bit Raspbian
Please let me know if I can provide more useful info.
Device Type
Indoor Camera (Supported)
Plugin Version
v1.0.x (Supported)
HomeBridge Version
v1.3.x (Supported)
NodeJS Version
v16 (Supported)
Operating System
Raspbian
Relevant log output
[EufySecurity] Device looks station but it's not could imply some errors Type: 31

just curious: why would you need homebridge for the pan/tilt camera? it has native homekit support.
i have three of them and i actually added them to the exclusions of this plugin so it doesn't interfere with native homekit

@eulisef
For me I bring my 2K P&T in using HomeBridge, they were in my Home before but I had to remove them for a problem I was having and now they will add back but will not stream and the only option is to wipe and recreate my home and that I am not willing to do as I would have to re-add everything and re-set up all my scenes and automations.

just curious: why would you need homebridge for the pan/tilt camera? it has native homekit support.
i have three of them and i actually added them to the exclusions of this plugin so it doesn't interfere with native homekit

just curious: why would you need homebridge for the pan/tilt camera? it has native homekit support. i have three of them and i actually added them to the exclusions of this plugin so it doesn't interfere with native homekit

I have three cameras at another house that isn't homekit compatible. I'd like to pull those into my homekit setup. So I've added them and have the same issue. Just another use case for you

Please try with latest version.

just curious: why would you need homebridge for the pan/tilt camera? it has native HomeKit support. I have three of them and I actually added them to the exclusions of this plugin so it doesn't interfere with native homekit

Having 2k  Pan/Tilt support is useful to me as I don't have a HomeKit hub. This means I can have my Eufy cam in HomeKit and still retain essential features like recording, object based detection etc

just curious: why would you need homebridge for the pan/tilt camera? it has native HomeKit support. I have three of them and I actually added them to the exclusions of this plugin so it doesn't interfere with native homekit

Having 2k Pan/Tilt support is useful to me as I don't have a HomeKit hub. This means I can have my Eufy cam in HomeKit and still retain essential features like recording, object based detection, zones etc

That is genius, didn't think about that. If this plugin ever gets up to par, I might actually go that route!

Same here: I have three 2Ks and I use one of them as a doorbell cam via HomeBridge and the homebridge-ffmpeg plugin.  Works a treat and it means I get 2K streaming and recording as well as the Eufy features which HomeKit would otherwise disable.
Apart from the lack of HKSV, it's pretty much the best of both worlds.

Please try with latest version.

Already tried with the latest version. The problem persists.

@hoaxlayer Please provide logs for your bug. Preferably with enabledetailedlogging set to 2.

Same issue, here is the debuglog

Same as above, I just see the camera as a security toggle but not as a camera in homekit. I have the Livestream from Camera device enabled in the plugin config.

As per your feedback I'm closing this one. Please open another issue for the ffmpeg


Adding "Run In Colab" Option for Keras Documentation similar to that in TF.Org

URL(s) with the issue:.
Please provide a link to the documentation entry, for example: https://keras.io/guides/customizing_what_happens_in_fit/
Description of issue (what needs to be changed):.
The Tutorials and Guide of Keras Documentation can have a button named "Run in Colab", like we have it in the Tutorials in the Tensorflow Site.

There is a button at the top of each notebook called View in Colab.


Please cut a new release

Problem Statement
Hey there, I would like to update my other Sentry packages but currently can't because sentry-electron uses the old SDK version, Would be nice if this can be coordinated in the future!
Thank you!
Solution Brainstorm

Release new sentry-electron version
In the future, always release a new sentry-electron version when sentry-javascript releases a new version with a newer SDK

Hey @Stanzilla, we'll get another release out with the latest sdk version.

In the future, always release a new sentry-electron version when sentry-javascript releases a new version with a newer SDK

We can't guarantee immediate releases because of maintainer bandwidth, but we'll try our best to release in a timely manner. If you would like to help expedite, PRs are always welcome (usually with JS SDK bumps comes some small changes to internals or refactors to tests).

Hey @Stanzilla, we'll get another release out with the latest sdk version.

In the future, always release a new sentry-electron version when sentry-javascript releases a new version with a newer SDK

We can't guarantee immediate releases because of maintainer bandwidth, but we'll try our best to release in a timely manner. If you would like to help expedite, PRs are always welcome (usually with JS SDK bumps comes some small changes to internals or refactors to tests).

Oh is that something you guys would accept? Just bumping the SDK version? Good to know!

yes! example PR here: https://github.com/getsentry/sentry-electron/pull/912

Apologies! The JavaScript SDKs are updated, we just needed a release #924

Thank you!

Once this issue closes, the new release will be published!

Released! https://github.com/getsentry/sentry-electron/releases/tag/5.1.0

Hey, me again, would love to get a new release that is compatible with the latest SDK. Always a bit of a pain if the two projects desync and you can't update Sentry because the Electron plugin will break otherwise


Missing tiles in raster reprojection example

Running into quite a few situations where no tiles are displayed for certain areas (see screenshot).
cc @petrsloup

After some debugging I realized this is actually not an issue with the reprojection.
I same the same problem at http://openlayers.org/en/master/examples/wms-custom-tilegrid-512x256.html
Looks like there is some problem (temporary?) with the WMS itself:
e.g. http://demo.boundlessgeo.com/geoserver/wms?SERVICE=WMS&VERSION=1.3.0&REQUEST=GetMap&FORMAT=image%2Fpng&TRANSPARENT=true&LAYERS=topp%3Astates&TILED=true&WIDTH=512&HEIGHT=256&CRS=EPSG%3A3857&STYLES=&BBOX=-7623269.642878361%2C3833530.4571513445%2C-6370925.371454033%2C4459702.5928635085 returns transparent image (and sometimes even java.lang.NullPointerException if you keep refreshing).

Ah interesting, hadn't noticed this on my end.

Thanks for your quick feedback!


Package causes some atom features get stucked

Steps to reproduce

install & enable package
try to search a phrase in project
try to show list of calls to deprecated methods
disable package and try these steps again

Expected results

phrase is being searched and results are shown in the search tab
tab with list of calls to deprecated methods opens

Actual results

phrase is being searched, but result window doesn't show any result
tab with list of calls doesn't open
after disabling package everything works as expected

I think the package obstructs atom somehow from refreshing tab or opening new one.
The bug does not appear after fresh install, you are suppose to use it for a while, maybe it is connected with some cache memory?
Ask me for more details.

I am unable to reproduce the issue you are seeing. Any additional information you can provide would be great.

@jakesankey I will try to reproduce it. Now I'm running the package for a day w/o crash, so it may have been fixed by the last Atom release.

Please let me know and close out the issue if it is indeed resolved..

I'm not able to reproduce this again, so I think it has been fixed with Atom 0.176 release. Closing.


==> Pouring ncurses--6.5.x86_64_linux.bottle.tar.gz Error: No such file or directory @ rb_check_realpath_internal - /etc/ld.so.conf

brew gist-logs <formula> link OR brew config AND brew doctor output
It won't accept all four so I did each one individually.

❯ brew gist-logs htop
Error: No logs.

❯ brew gist-logs bmon
Error: No logs.

❯ brew gist-logs nload
Error: No logs.

❯ brew gist-logs mtr
Error: No logs.

Verification

[X] My brew doctor output says Your system is ready to brew. and am still able to reproduce my issue.
[X] I ran brew update and am still able to reproduce my issue.
[X] I have resolved all warnings from brew doctor and that did not fix my problem.
[X] I searched for recent similar issues at https://github.com/Homebrew/homebrew-core/issues?q=is%3Aissue and found no duplicates.

What did you expect to happen?
Finish installation.
Step-by-step reproduction instructions (by running brew commands)
As above

brew doctor
Your system is ready to brew.

brew config
HOMEBREW_VERSION: 4.3.24
ORIGIN: https://github.com/Homebrew/brew
HEAD:<PHONE_NUMBER>62c32fc2365e8e9ff0b1507a98925e
Last commit: 7 days ago
Core tap JSON: 29 Sep 04:03 UTC
HOMEBREW_PREFIX: /home/linuxbrew/.linuxbrew
HOMEBREW_CASK_OPTS: []
HOMEBREW_DISPLAY: :0
HOMEBREW_EDITOR: nano
HOMEBREW_MAKE_JOBS: 16
Homebrew Ruby: 3.3.4 => /home/linuxbrew/.linuxbrew/Homebrew/Library/Homebrew/vendor/portable-ruby/3.3.4_1/bin/ruby
CPU: 16-core 64-bit zen
Clang: 18.1.8
Git: 2.46.1 => /bin/git
Curl: 8.10.0 => /bin/curl
Kernel: Linux 6.10.11-303.current x86_64 GNU/Linux
OS: Solus (resilience)
Host glibc: 2.40
/usr/bin/gcc: 14.2.0
/usr/bin/ruby: 3.2.3
glibc: N/A
gcc@11: N/A
gcc: N/A
xorg: N/A

Great, thanks. Can you try to reproduce the error with --debug? e.g., brew install --debug htop

htop seems to install OK.  Do you have an ld.so.conf?

ls /etc/ld.so*
/etc/ld.so.cache

Package glibc has file /usr/share/defaults/etc/ld.so.conf

Thanks for the prompt fix @carlocab!

Just sent a small thank you on OpenCollective (GH no longer supports PayPal).


s/22.04/24.04

While at it, cleanup how the per-script test image is defined.

@tomponline I didn't get around to figuring the DNS resolution issue with tests/network-ovn when using 24.04 images so I've revert to using 22.04 for that single test until I can revisit this.
I think it's still worth merging as-is.


⚠ Ixtanet SAT has degraded performance

In 2301ab9, Ixtanet SAT ($MEDIA_SAT) experienced degraded performance:

HTTP code: 200
Response time: 21219 ms

Resolved: Ixtanet SAT performance has improved in 0d2eac5 after 290 days, 8 hours, 11 minutes.


xbox classic games restarting

Xbox 360 Slim
Matrix Glicher RGH 1.2-v2
Corona
My xbox classic games stopped working after switching to AuroraCrashPatcher.
I did a test, that's how remove the plugin back to work
greetings from Brazil

Xbox 360 Slim Matrix Glicher RGH 1.2-v2 Corona
My xbox classic games stopped working after switching to AuroraCrashPatcher. I did a test, that's how remove the plugin back to work
greetings from Brazil

You need it to update db images, after you can disable it.
On last dash I don't  have this problem

I believe the idea is to always keep active instead of deactivating

I believe the idea is to always keep active instead of deactivating

Correct; best to just leave it running in the background all the time and simply boot with no plugins (by opening the disc tray when booting) if planning to play classic games.

I believe the idea is to always keep active instead of deactivating

Correct; best to just leave it running in the background all the time and simply boot with no plugins (by opening the disc tray when booting) if planning to play classic games.

Thanks for answering
do you plan to fix this error with classic xbox games?

I have no plans on updating this patch further. Aurora pushed an official hotfix update earlier today (0.7b-2), and as I am now directly involved with the Aurora development, there shouldn't be any need for a third-party patch plugin should any issues arise in the future.
As of the time of this writing, this plugin should only be necessary if you're using FSD, which has not received an official update as of yet. If only using Aurora, you can remove it from your launch plugins once Aurora is updated.
For more info or to download the update, see the official post on the Aurora newsfeed at http://phoenix.xboxunity.net/#/news.

thanks for everything
I think the time has come to switch to AURORA.


Is there a WebGL only version of PIXI?

Now that WebGL is widely supported (97%+) I think it makes sense for PIXI to have a WebGL-only version (for both speed and bundle size reasons).
Can it be compiled so that the dist bundle doesn't include any Canvas-specific code (eg. removes everything related to the canvas renderer).
And as a "larger" question, do you think Canvas fallback is still needed?

Agreed! We have designed this for v5 which is currently on the “next” branch. The default build of PixiJS v5 will not have canvas, which is now treated as second class.

I just realised after asking the question that one use case for Canvas rendering might be server-side rendering. But I don't know if anyone actually used, uses or will use PIXI for server-side rendering (on servers without GPUs).
Still, the first question is still unanswered: can the current build be bundled without the Canvas renderer?

Current, no. This one works without canvas: https://github.com/pixijs/pixi.js/tree/next . Even more, if you try "pixi-legacy" build from that branch, it wont work because CanvasRenderer is botched and we havent fixed it yet.


[Enhancement] When using Attributify mode, windi generate mutiple times css style --> Lag

Hope your team can fix it!

First of all, thank you for raising the issue, and secondly, I hope you understand that the most important thing to raise the issue is to give information that can reproduce the situation, such as the platform you are using(vue/svelte/webpack). Because many of the compatibility issues belong to the plugin, and do not need to be solved in the windicss core.


Dropzone

This pull requests makes the entire Edit page a drop-able zone for image uploads.  There is also a button a user can click to select a file for upload.  Multiple files can be dropped at once.
Uploaded files will be sent vi webdav to a configurable webdav server. The URL of the image will then be associated with the event via the Image feature.  It should be possible to implement other uploaders than webdav, but I figure that's a decent place to start.
Along with this is the addition of loading css files from a feature, as well as loading more than one js or css file.
There is a README in features/uploads/README.md  with more details.

awesome feature! lgtm!


Implement support for non-JS users

Why
We've accidentally discovered that this template doesn't render well if the JavaScript is disabled.
Following this blog post:

How many people are missing out on JavaScript enhancement? by Peter Herlihy

we believe it should be resolved in order to keep up with Using progressive enhancement.

What's the issue with rendering? The content should be just fine, though it will fall back to a non flexbox layout with a non-fixed left hand nav and header.

Hey,
Sorry. That's the issue I'm referring to. It appears you know about this - is that something scheduled or did you decide against fixing it?

Good catch, thanks. I've opened a PR which should improve things a bit.


Week 2 Plans and Tangibles

Todo

[x] Brainstorm mini project to cover the JS hacks presented on teacher site

[x] JS API: Utilize external API to print motivational quotes at the top of the page.
[x] JS Input: Utilized input boxes and adding new elements onto the document.

[x] Do something extra, have the program detect whenever a new table row is completed, then add on a new row with comment boxes. Scan the new row for completion

[x] JS Output using jQuery: Implement a dataTable embed using jQuery, allow for column sorting, multiple pages, and searching
[x] JS Storage using localStorage: Use the localStorage feature in modern browsers to store persistent user data for the user

[x] Auto-generate the existing user data from JSON stored in localStorage

Project would simulate the old CSP project we created last year. Just more lightweight and implemented into the blog

[x] Record video timelapse showing collaboration on project
[x] Utilizing technology such as live share to facilitate group collaboration and peer programming
[x] Stylize the todo list using CSS

Bugs

[x] localStorage would not properly store user data

fixed, bad semicolon.

[x] Bad starting index for table

fixed, stored index_count in localStorage

[x] jQuery dataTable would not render properly, adds additional "No data in table" row in table even when data is present

Table structure was bad, new rows were being added to <thead> element instead of <tbody>. Locating this element fixed the issue.

[x] Todo list would not render properly on Github pages, Jekyll rendering is bad

The todo list had JS comments in the code which commented out the rest of the program. Comment was removed in order for program to work as everything in the script tag was interpreted as one line

Here is feedback from ChatGPT for the code I wrote with Ethan.
I've reviewed your code and here are some observations and suggestions:
1. Global Variables: You have a few global variables like index_count, row, and idxCount. It's generally a good practice to limit the use of global variables and declare them within the scope where they are needed. Consider using function parameters and return values to pass data between functions.
2. Variable Declarations: Always declare variables using var, let, or const to avoid accidental global scope issues. Make sure to declare variables like row and idxCount using var, let, or const to avoid potential issues.
3. Inconsistent Naming: Your variable naming is somewhat inconsistent. Some variables use underscores (class_name), while others use camelCase (jsonParsed). Stick to a consistent naming convention, preferably camelCase for JavaScript variables.
4. Code Structure: Your code could benefit from better organization and comments to improve readability and maintainability. Consider adding comments explaining the purpose of each function and the logic within it.
5. LocalStorage Usage: Be cautious when using localStorage for storing data. It has a limited storage capacity, and storing large amounts of data can lead to issues. Consider using other storage options like IndexedDB for larger data sets.
6. Error Handling: There's no error handling in your code. When working with external APIs like the one in getNewQuote(), it's essential to handle potential errors gracefully.
7. HTTP Requests: You're using the XMLHttpRequest API for making HTTP requests. Consider using the more modern fetch API for better readability and compatibility.
8. Hardcoded API Tokens and URLs: Avoid hardcoding sensitive information like API tokens directly in your code. Store them in environment variables or configuration files for security reasons.
9. Magic Values: There are several hard-coded values like the API URL and keys. It's a good practice to define such values as constants at the top of your script to make it easier to update them in the future.
10. HTML Structure: Your code assumes the existence of an HTML table with specific cell structures (table1, rows, and cells). Ensure that your HTML structure matches the code's expectations.


Update swiftlint

Why
Adopt a better approach to swiftlint
Changes
None
Tasks

[X] Add relevant tests, if needed
[X] Add documentation, if needed
[X] Update README, if needed
[X] Ensure that all the examples (as well as the demo) work properly

👍


release: Pelorus Operator version 0.0.8

[x] I have read Pelorus Contributing guidelines

Linked Issues

related to #1050
Pelorus Operator 0.0.8 release
Release for version 0.0.8 of Pelorus Operator.
Testing Instructions
To test the release candidate, run the command the CI posted in this PR.
Also, read, review and update release body marked as drafted.

/hold Merge only on Monday

/lgtm

/unhold


Way to identify mergeable page

Currently we identify whether to merge responses or not by checking if response has GetHtmlPartialUrl method (/Server/Launcher.json.cs#L78
I wonder if it is the best way to do so, as I believe that someone would like to return a JSON without such method, and personally I does not find adding such method intuitive, and I've get stuck few times wondering why my app output does not appear as mergeable result.
Maybe we could use MergeableJson class or json.mergable flag for such cases?

@tomalec is this issue still up to date?

@warpech It's solved by your implicit standalone mode, isn't it?
The only question is what about improvements of check whether it's a partial or not.

If using implicit standalone mode, the app does not have to care if it is running standalone. Implicit standalone mode will wrap it.
Right now the check is based on the presence of <!doctype, which might be improved in future: https://github.com/Starcounter/Starcounter/blob/81c9a245f2f3769cb19bb2e2ad0c3d69a14b19b8/src/Starcounter/Middleware/PartialToStandaloneHtmlProvider.cs#L105


Saving model with inlined resource

Supposing I have the following model:
restmod.model('/rest/documents').mix({
	attachments: {
		belongsToMany: restmod.model('/rest/attachments'),
	}
});

When I fetch a document, I get the following responce from the api:
{
	id: 1,
	attachments: [{ id: 1 }, { id: 2 }]
}

When I save this document, I want all inline data to be posted as is. But instead request data looks like:
{
	id: 1,
	attachmentIds: [1, 2]
}

Is it by design? If it is can I somehow override this behaviour?

It is by design, yes, but a possible workaround would be:
restmod.model('/rest/documents').mix({
    attachments: {
        belongsToMany: restmod.model('/rest/attachments')
    },
    $hooks: {
      'before-render': function(_raw) {
         _raw.attachments = this.attachments.$encode();
         delete _raw.attachmentIds;
      }
    }
});

ok. thx for the hint.


Update PagerRenderer.php

Some time , somebody need 'total' and 'pageCount'
Each pull request should address a single issue and have a meaningful title.
Description
Explain what you have changed, and why.
Checklist:

[ ] Securely signed commits
[ ] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[ ] Conforms to style guide

---------Remove from here down in your description----------
Notes

Pull requests must be in English
If the PR solves an issue, reference it with a suitable verb and the issue number
(e.g. fixes 12345)
Unsolicited pull requests will be considered, but there is no guarantee of acceptance
Pull requests should be from a feature branch in the contributor's fork of the repository
to the develop branch of the project repository

Before ,like this:
previous 1 ... 2 3 4 [5] 6 7 8 ...  100 Next     or     previous 1 ... 93 94 95 [96] 97 98 99 ... 100 Next
Then:
previous 1 2 3 4 [5] 6 7 8 ...  100 Next     or     previous 1 ... 93 94 95 [96] 97 98 99 100 Next

If I want to get
total:100 previous 1 2 3 4 [5] 6 7 8 ... 100 Next  or  total:100 previous 1 ... 93 94 95 [96] 97 98 99 100 Next
This kind of demand is more common

total:100 previous 1 2 3 4 [5] 6 7 8 ... 100 Next page:5/100 or total:100 previous 1 ... 93 94 95 [96] 97 98 99 100 Next page:96/100

I see the difference here in the output you're presenting but still... you have to find the way to accomplish it without changing parameters for public methods, like makeLinks() and displayLinks(). Making changes to the public methods will result in broken code for other users.

This has been unattended for almost a month, a BC break, and has merge conflicts. Closing this.


fix(packge.json config): configuration method for plugins with options

fix #86

@pselle I did a quick cleanup and comment add


Improve readme

I changed the readme to Markdown, fixed some spelling and grammar issues, in addition to what looks like a quoting mistake in the example command.

I am really not a fan of markdown, but since the readme is only used for the repo (and not included in some release archive) and you did some nice improvements I've decided that my dislike is not sufficient reason to not merge it :)


feat: add footer

To do this well would require overriding PageFrame and adding the Footer inside there. The site already has a footer with pagination, etc. and the custom footer would only have the header links presenented in a slightly different way. I think it's not needed that much.


Scanpy version pinned to >=1.5.1 necessary?

Hey!
I was wondering whether the scanpy version pin to >=1.5.1 is necessary. Do you rely on e.g., the location of the stored connectivities matrix somewhere? We are working with version 1.4.6 in our benchmarking project at the moment which contains code that is not compatible with 1.5.1 yet.

Hi @LuckyMD, it seems like we bumped scanpy with our 0.6.6 release. @galenxing do you remember why?
For your project, since you're just using scVI it should be fine to use version 0.6.5, which has scanpy > 1.4 requirement.

Hey @adamgayoso! Thanks so much for the quick reply.
We noticed there was only _highly_variable_genes from scanpy in here. We are however now also considering to test scANVI for which there was apparently a bug until 0.6.6. So we're unfortunately a bit stuck on this. Do you think it would be possible to remove the requirement? Otherwise we would just make a fork without it and use that i guess (a bit more messy for a benchmark though).

I see, we will test out a 0.6.7 with removing these requirements.

Thanks a lot!

We should be able to make a release tomorrow.

You are awesome :). That might be just in time to set up our runs before i head on vacation tomorrow :).

0.6.7 is now on pypi

Thanks!


failing tests

On running app.js, I get hit with:
webapp_1    |   6 failing
webapp_1    |
webapp_1    |   1) CashflowyService #findSimilarTransactions UPI payment:
webapp_1    |      TypeError: Cannot read property 'original_amount' of undefined
webapp_1    |       at Object.findSimilarTransactions (api/services/CashflowyService.js:157:37)
webapp_1    |       at Context.<anonymous> (test/services.test.js:38:48)
webapp_1    |       at process.topLevelDomainCallback (domain.js:120:23)
webapp_1    |
webapp_1    |   2) CashflowyService #findSimilarTransactions NEFT income salary:
webapp_1    |      TypeError: Cannot read property 'original_amount' of undefined
webapp_1    |       at Object.findSimilarTransactions (api/services/CashflowyService.js:157:37)
webapp_1    |       at Context.<anonymous> (test/services.test.js:38:48)
webapp_1    |       at process.topLevelDomainCallback (domain.js:120:23)
webapp_1    |
webapp_1    |   3) CashflowyService #findSimilarTransactions UPI incoming:
webapp_1    |      TypeError: Cannot read property 'original_amount' of undefined
webapp_1    |       at Object.findSimilarTransactions (api/services/CashflowyService.js:157:37)
webapp_1    |       at Context.<anonymous> (test/services.test.js:38:48)
webapp_1    |       at process.topLevelDomainCallback (domain.js:120:23)
webapp_1    |
webapp_1    |   4) CashflowyService #findSimilarTransactions Netbanking outgoing:
webapp_1    |      TypeError: Cannot read property 'original_amount' of undefined
webapp_1    |       at Object.findSimilarTransactions (api/services/CashflowyService.js:157:37)
webapp_1    |       at Context.<anonymous> (test/services.test.js:38:48)
webapp_1    |       at process.topLevelDomainCallback (domain.js:120:23)
webapp_1    |
webapp_1    |   5) CashflowyService #findSimilarTransactions Mobile banking IMPS outgoing:
webapp_1    |      TypeError: Cannot read property 'original_amount' of undefined
webapp_1    |       at Object.findSimilarTransactions (api/services/CashflowyService.js:157:37)
webapp_1    |       at Context.<anonymous> (test/services.test.js:38:48)
webapp_1    |       at process.topLevelDomainCallback (domain.js:120:23)
webapp_1    |
webapp_1    |   6) CashflowyService #findSimilarTransactions NEFT incoming zerodha:
webapp_1    |      TypeError: Cannot read property 'original_amount' of undefined
webapp_1    |       at Object.findSimilarTransactions (api/services/CashflowyService.js:157:37)
webapp_1    |       at Context.<anonymous> (test/services.test.js:38:48)
webapp_1    |       at process.topLevelDomainCallback (domain.js:120:23)

This is running over Docker, but it shouldn't be related as far as I can see. I could fix this with a small fix:
 var findSimilarTransactions = function(options, callback) {
-	// console.log('this is the thing that is done');
-	var t=options.t;
+	console.log(options);
+	let t = options;
+	if (options.t) {
+		t = options.t;
+	}

But it failed again:
webapp_1    |
webapp_1    |   1) CashflowyService #findSimilarTransactions UPI payment:
webapp_1    |      TypeError: Cannot read property 'join' of undefined
webapp_1    |       at Object.findSimilarTransactions (api/services/CashflowyService.js:167:42)
webapp_1    |       at Context.<anonymous> (test/services.test.js:38:48)
webapp_1    |       at process.topLevelDomainCallback (domain.js:120:23)
webapp_1    |
webapp_1    |   2) CashflowyService #findSimilarTransactions NEFT income salary:
webapp_1    |      TypeError: Cannot read property 'join' of undefined
webapp_1    |       at Object.findSimilarTransactions (api/services/CashflowyService.js:167:42)
webapp_1    |       at Context.<anonymous> (test/services.test.js:38:48)
webapp_1    |       at process.topLevelDomainCallback (domain.js:120:23)
webapp_1    |
webapp_1    |   3) CashflowyService #findSimilarTransactions UPI incoming:
webapp_1    |      TypeError: Cannot read property 'join' of undefined
webapp_1    |       at Object.findSimilarTransactions (api/services/CashflowyService.js:167:42)
webapp_1    |       at Context.<anonymous> (test/services.test.js:38:48)
webapp_1    |       at process.topLevelDomainCallback (domain.js:120:23)
webapp_1    |
webapp_1    |   4) CashflowyService #findSimilarTransactions Netbanking outgoing:
webapp_1    |      TypeError: Cannot read property 'join' of undefined
webapp_1    |       at Object.findSimilarTransactions (api/services/CashflowyService.js:167:42)
webapp_1    |       at Context.<anonymous> (test/services.test.js:38:48)
webapp_1    |       at process.topLevelDomainCallback (domain.js:120:23)
webapp_1    |
webapp_1    |   5) CashflowyService #findSimilarTransactions Mobile banking IMPS outgoing:
webapp_1    |      TypeError: Cannot read property 'join' of undefined
webapp_1    |       at Object.findSimilarTransactions (api/services/CashflowyService.js:167:42)
webapp_1    |       at Context.<anonymous> (test/services.test.js:38:48)
webapp_1    |       at process.topLevelDomainCallback (domain.js:120:23)
webapp_1    |
webapp_1    |   6) CashflowyService #findSimilarTransactions NEFT incoming zerodha:
webapp_1    |      TypeError: Cannot read property 'join' of undefined
webapp_1    |       at Object.findSimilarTransactions (api/services/CashflowyService.js:167:42)
webapp_1    |       at Context.<anonymous> (test/services.test.js:38:48)
webapp_1    |       at process.topLevelDomainCallback (domain.js:120:23)

so I'm reporting it instead.

Seems like test cases are failing. Few fixtures are hard coded which won't work for other users. Will fix the issue.

Any way for me to bypass the tests while trying to run the app?

On 15-Feb-2019, at 13:00, akash patro<EMAIL_ADDRESS>wrote:
Seems like test cases are failing. Few fixtures are hard coded which won't work for other users. Will fix the issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

comment out from line no. 321 to 812 in test/config/services/CashflowyService.test.js

@captn3m0, @aptro - is this resolved?

I got the app running post it, but couldn't login.
There was however this error:
postgres_1  | 2019-02-19 19:58:08.261 UTC [31] STATEMENT:  SELECT "email"."email", "email"."token", "email"."details", "email"."id", "email"."createdAt", "email"."updatedAt", "email"."user" FROM "public"."email" AS "email"

migrations aren't running looks like. I didn't have time to debug further after this.
I can file a PR with my Docker Compose setup, which does make development slightly easier though.

@captn3m0 will look into the docker compose setup. Can you file a PR?
There's an issue for the same https://github.com/alexjv89/cashflowy/issues/24


Code generator does not generate proper code for timestamps when using Aurora

Describe the bug
When using TIMESTAMP(0) or TIMESTAMP(3) in the SQL database, codegen interprets that as AWSTimestamp, which seems right, but the problem is that when using "SELECT * FROM User", all TIMESTAMP fields are printed like:
"2019-12-19 16:59:09.966"
Because of that, the mapping fails. with:
"Can't serialize value (/loggedUser/dob) : Unable to serialize 2019-12-19 16:59:09.966 as a valid timestamp. Ensure that the value provided is an Integer that lies within the limits specified in this scalar's description."
I found a workaround to fix this temporally, basically, doing this:
SELECT cast(UNIX_TIMESTAMP(dob) as unsigned) as dob FROM User WHERE user = "whatever"
That's working, but the problem is that I cannot use "SELECT * FROM User" in the velocity templates, which is very annoying, especially when you add new fields and you have to modify several templates. This is prone to error.
How to fix that? I don't that, but the second issue is exactly the same as this ticket?
https://github.com/aws-amplify/amplify-cli/issues/3049
Amplify CLI Version
You can use amplify -v to check the amplify cli version on your system
To Reproduce
Just use Aurora serveless as a data source and generate the API using amplify command line tool.
Make sure you use TIMESTAMP(0) in your entities.
Now, call a method from API.swift that uses an entity with timestamp.  It will fail with the error mapping I said above.
Expected behavior
The code generation should generate code that fixes this issue automatically. How? To be honest, I don't know. I think the trick is in the "SELECT *" part.
Desktop (please complete the following information):

OS: Mac
Node Version: v13.2.0

Additional context
The "SELECT * FROM..." issue is the same as:
https://github.com/aws-amplify/amplify-cli/issues/3049

related https://github.com/aws-amplify/amplify-cli/issues/2199


literal handling

We must differentiate between a constant literal and a column reference in a projection.
The following changes should be implemented.
There is a difference between

SELECT name
SELECT "name"
SELECT 'name'

For 1, name is a column reference, and must be resolved to a column of the input of the projection. If no such column exists, an error is returned.
For 2, "name" is a constant literal or column reference, meaning that if there exists a column name in the input of the projection, it is resolved to that column. If not, it is interpreted as a constant literal.
For 3, 'name' is a constant literal, which in this case is of type String.

TODO:

[x] adapt parser ( #56 )
[x] adapt parse tree ( #56 )
[ ] adapt AST ( #57 )
[ ] adapt compiler ( #57 )
[ ] adapt engine ( #58 )

For the AST, I'd like two new interfaces and three new structs.
type (
    ColumnReferenceExpression interface { ... }
    LiteralExpression         interface { ... }

    ConstantLiteral struct {}                  // implements LiteralExpression
    ConstantLiteralOrColumnReference struct {} // implements both LiteralExpression and ColumnReferenceExpression
    ColumnReference struct {}                  // implements ColumnReferenceExpression
)


JWT Leeway

is there any way (setting?) to set JWT leeway?

Nope, but this is where you can implement it: https://github.com/jazzband/djangorestframework-simplejwt/blob/ff598307692391ab29d01404fa0bff1440fad244/rest_framework_simplejwt/backends.py#L79
You'd need to override any parts mentioning a token backend though. Probably best if you made a PR and put this in the settings as DECODE_LEEWAY as being either integer or timedelta.


Fixes LB certificate tests failure

Whenever the path to the project includes a URL encodable character like whitespace, these tests fail to find the local path to the certs.
This is because getClass().getResource() returns a URL object and that is used directly to initialize a local file path.

LGTM.


experimental: add utility for extracting top projects

Also adds argparse logic.
This is working towards end-to-end combination with core OFG.
Ref: https://github.com/google/oss-fuzz-gen/issues/450
Signed-off-by: David Korczynski<EMAIL_ADDRESS>

/gcbrun skip


Rate limit checks fail when there is no rate limiting

I am trying to use this exporter against a GitHub Enterprise instance. This server does not have rate limiting enabled, which appears to be causing the following error when checking for a rate limit: Error gathering Data from remote API: strconv.ParseFloat: parsing "": invalid syntax.

Here is the output of curl to the rate limit api. Some details have been removed for privacy.
curl -v https://server/api/v3/rate_limit
*   Trying x.x.x.x...
* TCP_NODELAY set
* Connected to server (x.x.x.x) port 443 (#0)
> GET /api/v3/rate_limit HTTP/1.1
> Host: server
> User-Agent: curl/7.51.0
> Accept: */*
>
< HTTP/1.1 404 Not Found
< Server: GitHub.com
< Date: Tue, 09 May 2017 21:24:57 GMT
< Content-Type: application/json; charset=utf-8
< Content-Length: 122
< Status: 404 Not Found
< Cache-Control: no-cache
< X-GitHub-Media-Type: github.v3
< Access-Control-Expose-Headers: ETag, Link, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval
< Access-Control-Allow-Origin: *
< X-GitHub-Request-Id: 8f609e15-d338-4927-bd8f-ea969a18b96d
< Content-Security-Policy: default-src 'none'
< Strict-Transport-Security: max-age=31536000; includeSubdomains
< X-Content-Type-Options: nosniff
< X-Frame-Options: deny
< X-XSS-Protection: 1; mode=block
<
{
  "message": "Rate limiting is not enabled.",
  "documentation_url": "https://developer.github.com/enterprise/2.8/v3"
}
* Curl_http_done: called premature == 0
* Connection #0 to host server left intact

Thanks for the heads up @bmanuel.
I've not had the ability to test against GitHub Enterprise, only the cloud instance.
I'll have a look at making the code for this passively fail and to log out the error only.

I've just pushed an image to test infinityworks/github-exporter:04. Would you mind giving it a once over? If all looks good i'll look to merge today.

@Rucknar Thanks for the quick fix. It appears to be working as expected.


RRateLimiter.trySetRate does not reset all rate limiter state despite documentation

Expected behavior
RRateLimiter.trySetRate's docstring claims that it:

Initializes RateLimiter's state and stores config to Redis server.

and that it will

@return {@code true} if rate was set and {@code false} otherwise

Hence, one would expect that whether or not the rate config was updated, the rate limiter state would be reset.
Actual behavior
However, RedissonRateLimiter.trySetRate does not delete the value and permit keys ({xxx}:permits and {xxx}:value), unlike RedissonRateLimiter.setRate. Hence, it is possible for the value and permit state to remain un-reinitialized and invalid even after calling trySetRate. This causes issues when, for example, upgrading redisson to a version where there have been changes to the permits serialization format.
Steps to reproduce or test case
Code snippet:
RRateLimiter rateLimiter = redisson.getRateLimiter(ofRateLimiter);
if (!rateLimiter.trySetRate(RateType.OVERALL, 10L, 1L, RateIntervalUnit.HOURS)) {
  RateLimiterConfig currentConfig = rateLimiter.getConfig();
  boolean rateLimiterConfigLooksFine = currentConfig != null
    && currentConfig.getRateType() == RateType.OVERALL
    && currentConfig.getRate().equals(10L)
    && currentConfig.getRateInterval().equals(RateIntervalUnit.HOURS.toMillis(1L));

  if (!rateLimiterConfigLooksFine) {
    rateLimiter.setRate(RateType.OVERALL, 10L, 1L, RateIntervalUnit.HOURS);
  }
}

// In 3.16.3, adds permit values in the struct format `fI`.
// In 3.30.0, expects permit values in the struct format `Bc0I`, which fails and throws RedisException: bad argument #2 to 'unpack'.
rateLimiter.acquire();

Install a redisson version prior to a rate limiter internal state format change, e.g. 3.16.3 which was before this change.
Run the snippet, then keep Redis running.
Upgrade the redisson version to a version after that internal state format change, e.g. 3.30.0.
Run the snippet without having restarted Redis.

Redis version
6.x
Redisson version
Upgrading from 3.16.3 to 3.30.0.
Redisson configuration
N/A
Thoughts
Both of these options seem reasonable here, not sure if there's a better way:

Either update the documentation for trySetRate to call out that the state could be invalid if it returns false (if it returns true the rate limit is not present and is created fresh).
Or update trySetRate to also delete the permits and values keys just like setRate.

I have same issue with rate limiter by upgrade redisson from 3.16.1 to 3.30.0.
There does not seem to be a straightforward solution. So I tried this by catch the RedisException and then call setRate to reset the rate limiter, although the previous rate limiter data would be lossed.
public boolean tryGetToken(String redisKey, int rateIntervalSeconds, int rate) {
    RRateLimiter rateLimiter = redissonClient.getRateLimiter(redisKey);

    rateLimiter.trySetRate(RateType.OVERALL, rate, rateIntervalSeconds, RateIntervalUnit.SECONDS);

    try {
        return rateLimiter.tryAcquire(1);
    } catch (RedisException ex) {
        // 1. check the exception is bad argument?
        // reference: https://github.com/redisson/redisson/issues/5900
        if (ex.getMessage().startsWith("ERR user_script:1: bad argument #2 to 'unpack' (data string too short)")) {
            log.info("Rate limiter {} initialized by old version redisson, as the exception is: {}", redisKey, ex.getMessage());
            rateLimiter.setRate(RateType.OVERALL, rate, rateIntervalSeconds, RateIntervalUnit.SECONDS);

            // 2. call acquire again.
            return rateLimiter.tryAcquire(1);
        } else {
            throw ex;
        }
    }
}

the rate limiter state would be reset.

state can be cleared only if rate was updated. I'll update javadocs. In your case setRate usage is the only right solution.


Confusing blit

@attilaz @bkaradzic , sorry to disturb you again.
I would like to ask why reading data back from textures requires blit once? What is the role of blit? If we don’t call blit, what will happen? Sorry, I don’t understand this principle.
Is it possible to just call readTexture to get the data without calling the blit method?

No. You're blitting from GPU to CPU texture. CPU doesn't have access to GPU memory, and blit is necessary to move memory from GPU RAM to CPU RAM. You also want to minimize amount of data you're transferring and instead blitting whole texture you can blit only parts you actually need for your app.

@bkaradzic , But for opengles on Android platform,  I need to paste texture A somewhere in texture B, On the ios platform I can use the blit method to achieve。 but I can't use the blit method on opengles. In addition updateTexture2D method can only be updated using Memory。 Can you give us some suggestion?

On GLES keep copy of TextureB in memory, and then outside of bgfx paste textureA into it, and then use updateTexture2D to upload it.
GLES just lacks blit functionality, so you need to implement fallback code path.


Create a CI check to ensure new rules are in index.js & README

Will help us avoid issues like #136.

For the list of doc pages in the readme, couldn't we automatically update it when generating the doc ?

Yes, that would work too. Although we may want to re-group the list at some point so just having it as check might be more straightforward.


Fixed Buffer Reaching Limit

It now limits the max length of the message sent and when the buffer reaches the limit it deletes the longest message in the buffer. Give me money.

Also, I won't give you money :3

Looks good, will merge.


Some SceneTransforms functions docs do not correctly reflect implementation

What happened?
The JSDoc for certain functions in the cesium\packages\engine\Source\Scene\SceneTransforms.js have JSDoc that don't completely describe the possible return values.
The affected functions are:
SceneTransforms.worldToWindowCoordinates
SceneTransforms.wgs84ToWindowCoordinates
SceneTransforms.worldToDrawingBufferCoordinates
SceneTransforms.wgs84ToDrawingBufferCoordinates
Each of these functions have JSDoc describing their return types as some variation of the following:
@returns {Cartesian2} The modified result parameter or a new Cartesian2 instance if one was not provided.  This may be undefined if the input position is near the center of the ellipsoid.

It is noted that each of these functions can potentially return an undefined value, but this possibility is not encoded in the type definition of the @return. Including the undefined value as a union with the successful return type is a potential resolution to this: @return {Cartesian2|undefined}.
The typescript signatures of these functions are built from the JSDoc, so when building typescript definitions from the source, the signatures of these functions end up not accurately describing the potential for failure that they have. Any consumers of cesium that are accessing these function might end up with unexpected crashes because of it.
Reproduction steps

Run the script to build typescript definitions: npm run build-ts
Inspect cesium\packages\engine\index.d.ts and notice that the type declarations for the above functions do not correctly reflect the implementations

Sandcastle example
No response
Environment
Browser: N/A
CesiumJS Version: cbd13cee0959d2de6e01f355dd1cd73342467102
Operating System: Windows 11 Pro

Thanks for pointing this out @sudo-jaa! Since some of those function were just added, this should be fixed before the next release.


Demo server broken ?

When browsing
http://neuspell.github.io/
"Impossible to connect to
http://ogma.lti.cs.cmu.edu:5000/
"

Hi @WilliamTambellini The server will be down for some days now. However, you can easily run the flask-server on your own machine by following the demo setup page here. The code can run on CPU as well as GPU.


[UI] Please add a refresh button to a Project Results view

Such a button is sourly missed, for quickly refreshing a project's results view, without refreshing the whole URL via the browser's refresh button.
Maybe add a Refresh timer too?

@edagan I'll do a quick PR for this

@JonathanRosado - do you have something, or should I address it to @AndreeaDora as well?

PR #125 was merged.


Cognito client x-amzn-errortype fix

Issue #, if available:
fixes #874
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1008 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1008   +/-   ##
=======================================
  Coverage   87.12%   87.12%
=======================================
  Files          81       81
  Lines        3960     3960
  Branches      749      749
=======================================
  Hits         3450     3450
  Misses        487      487
  Partials       23       23

Continue to review full report at Codecov.

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

Also maybe related to #994