text:
No auto-rebase after @dependabot merge command

Related to: https://github.com/dependabot/feedback/issues/365

Package manager/ecosystem
javascript, npm
Manifest contents prior to update
https://github.com/Maxim-Mazurok/google-api-typings-generator/blob/master/.github/dependabot.yml
What you expected to see, versus what you actually saw
Every week, dependabot open a couple of PRs.
I have auto-merge action, which adds "@dependabot merge" comment to all minor update PRs automatically. The problem is that only the first one gets merged. Other ones need to be rebased, but they never do it automatically. I have to manually comment "@dependabot rebase" on every PR, which defeats the purpose.
I want dependabot to automatically rebase my PRs even if I have commented "@dependabot merge".
I believe that this was implemented as part of https://github.com/dependabot/feedback/issues/365 and now it doesn't work, so this may be a regression.
Images of the diff or a link to the PR, issue or logs

All 4 got opened yesterday:
This one got merged: https://github.com/Maxim-Mazurok/google-api-typings-generator/pull/443
These 3 are stuck unless I manually comment "@dependabot rebase"
https://github.com/Maxim-Mazurok/google-api-typings-generator/pull/441
https://github.com/Maxim-Mazurok/google-api-typings-generator/pull/442
https://github.com/Maxim-Mazurok/google-api-typings-generator/pull/444

@greysteil do you know anything about this, by chance?

Afraid not - I haven't worked on Dependabot for the last couple of years! It's in the safe hands of @hmarr and @asciimike now, and this looks like a very clear bug report for them and the team to dig into.

Hi @Maxim-Mazurok, this has been open for a while, apologies for that.
The PRs you linked to have since been rebased and merged, so it's a little hard to dive into what happened.

I want dependabot to automatically rebase my PRs even if I have commented "@dependabot merge".

It's good to know that dependabot will only rebase if there are any merge conflicts, but not when the ref is out of date. Do you happen to know if this was the case? Has this happened since?

@jurre thank you for picking up this one.
Unfortunately, this was very frustrating for me, so instead of relying on @dependabot merge command, I started using Kodiak merge.automerge_dependencies. Here's the commit in which I switched: https://github.com/Maxim-Mazurok/google-api-typings-generator/commit/40d393bdb1e41d72599250c5366c1e83b8eac5ce
Now it works as expected.
As you can see, dependabot opened two PRs last Saturday: https://github.com/Maxim-Mazurok/google-api-typings-generator/pull/478 and https://github.com/Maxim-Mazurok/google-api-typings-generator/pull/479
First, I have approved https://github.com/Maxim-Mazurok/google-api-typings-generator/pull/478#pullrequestreview-632887234
Then I approved https://github.com/Maxim-Mazurok/google-api-typings-generator/pull/479#pullrequestreview-632887257
Kodiak merged the first one: https://github.com/Maxim-Mazurok/google-api-typings-generator/pull/478#event-4577066254
Then, Kodiak rebased the second one: Maxim-Mazurok/google-api-typings-generator@429318c (#479)
And Kodiak merged it: https://github.com/Maxim-Mazurok/google-api-typings-generator/pull/478#event-4577066254
This is precisely what I wanted Dependabot, paired with ahmadnassri/action-dependabot-auto-merge) to do.
That being said, I'll try to create reproduction in the new repo.

I forgot to mention my repo settings:
  Protect matching branches
   [x] Require pull request reviews before merging:
    Required approving reviews: 1
    [x] Dismiss stale pull request approvals when new commits are pushed
   [x] Require status checks to pass before merging
    [x] Require branches to be up to date before merging
    Status checks found in the last week for this repository:
     [x] test

These options are checked. Everything else is disabled.

I tried to reproduce it here: https://github.com/Maxim-Mazurok/dependabot-3132-repro but I couldn't.
Because of the https://github.com/ahmadnassri/action-dependabot-auto-merge/issues/60 issue, I had to change from on: pull_request to on: pull_request_target.
Everything works as expected:

3 PRs opened: https://github.com/Maxim-Mazurok/dependabot-3132-repro/pull/8 and https://github.com/Maxim-Mazurok/dependabot-3132-repro/pull/9 and https://github.com/Maxim-Mazurok/dependabot-3132-repro/pull/10
https://github.com/Maxim-Mazurok/dependabot-3132-repro/pull/8 got merged
https://github.com/Maxim-Mazurok/dependabot-3132-repro/pull/9 and https://github.com/Maxim-Mazurok/dependabot-3132-repro/pull/10 rebased
https://github.com/Maxim-Mazurok/dependabot-3132-repro/pull/9 got merged
https://github.com/Maxim-Mazurok/dependabot-3132-repro/pull/10 rebased
https://github.com/Maxim-Mazurok/dependabot-3132-repro/pull/10 got merged

And all of that even without the need to manually approve PRs, perfect! (even though not very secure)
I'm going to try this approach on my main repo and will let you know how it goes after the weekend. I do updates on it every Saturday.

I'm expecting this to have been resolved, I'm going to close the issue out for now, but please reopen if you run into things @Maxim-Mazurok!

It's only Friday here in Sydney, so I haven't confirmed if it works. Will reopen in case if it doesn't.

Unfortunately, this week I received only one dependency update (https://github.com/Maxim-Mazurok/google-api-typings-generator/pull/481) so couldn't verify rebasing functionality. Will have to wait for the next Saturday.

@jurre any ETA on when I could manually merge stuck PRs? Don't want to do that before your investigation.

@jurre following up, as the issue has been reopened 19 days ago and now I have 4 PRs stuck. Take a look, please, before I manually merge these PRs. Thanks!

@jurre bumping this, since 3 PRs are still stuck, see https://github.com/Maxim-Mazurok/google-api-typings-generator/pulls?q=is%3Apr+is%3Aopen+sort%3Aupdated-desc+author%3Aapp%2Fdependabot

unsure if related, but on a private repo we see that dependabot doesn't work the whole day - we merge a pr and than it takes hours before dependabot rebase the others.

I wish it was the case with my repo. PRs never get rebased for me :(
@jurre can you please confirm that this issue is not abandoned? I really want to help you by not rebasing PRs manually, but if this issue is abandoned - I'll have to look for alternative solutions. Cheers!

@jurre, unfortunately, I haven't heard back from you in a while, but I see from your GitHub profile that you're active. So I'm assuming that this issue is abandoned and I will be reverting back to using https://github.com/chdsbd/kodiak to auto-merge dependabot PRs

@dependabot help

We're also getting the same issue in some of our private Discovery repos!
In one of these projects, I ended up using a solution described at the bottom of this community post: How can I stop babysitting Dependabot?
It would be great if someone from Github could assist debugging the problem. It is sad to see that @Maxim-Mazurok's efforts to reproduce the problem have been done in vain.

https://github.com/marketplace/actions/gha-auto-dependabot-rebase seems like a pretty nice hack, thanks for sharing!
I've been using Kodiak for a while now and it worked flawlessly for me. Almost every Saturday I get a couple of dependencies auto-updated, rebased, and merged.
My configs: https://github.com/Maxim-Mazurok/google-api-typings-generator/blob/master/.github/dependabot.yml and https://github.com/Maxim-Mazurok/google-api-typings-generator/blob/master/.kodiak.toml
The only weird issue is that dependabot refuse to add "automerge-kodiak" label to PRs... And for whatever reason that doesn't stop Kodiak from auto-merging them anyway :D So somehow it works...

Actually, in this PR I somehow had some conflicts that kodiak wasn't able to resolve using rebase. But dependabot isn't just doing git rebase, as far as I understand, it's doing npm update from master discarding original PR changes.
So I will be adding gha-auto-dependabot-rebase into the mix to handle rebasing.

Closing out as stale

authors: BriceTossim Maxim-Mazurok abdulapopoola bdo greysteil jurre

repo: dependabot/dependabot-core


text:
Please don't open dependabot PRs during working hours

From @rdunlop on November 27, 2017 16:44
Usually, dependabot opens PRs for my projects between 2-3am, which is great, because no one is working. My CI system has enough capacity/time to work through the PRs at its leisure.
A few times (today being one of them), the PRs were opened starting around 10:30am. This is very problematic because there are now developers doing development work, and they want the CI system's capacity for themselves.
If there is a dependabot infrastructure issue preventing PRs from being opened at the normal 2am time, please DO NOT open them during normal business hours. That hurts. It makes my developers less happy with dependabot, because it makes their development iteration cycles longer...as they have to wait for dependabot's builds before their builds run.
Copied from original issue: dependabot/feedback#56

Understood, thanks @rdunlop - this was a manual run (on my side) to make up for a couple of missed updates this morning, but in future I'll wait until the following day. I'm also thinking about making Dependabot more timezone aware, so that it can be more considerate of CI capacity wherever teams are based.
And sorry! :octocat:

From @rdunlop on November 27, 2017 16:45
Was there a dependabot issue this morning? (or is another developer in my organization clicking "run now" on all my repos?)

source: gharchive/issue

license: MIT

license_type: permissive

license_source: github-api


text:
Rsync deployment failing to use hostname value in yaml hosts inventory file

Q
A

Issue Type
Bug

Deployer Version
5.1.1

Local Machine OS
MacOS Sierra 10.12.5

Remote Machine OS
CentOS 7.1

Description
Release v5.1.1 (relating to #1271) appears like it may have broken rsync process amongst hosts.
Here's a quick summary..
On my system, v5.1.1 appears to attempt to connect to the  host, rather than the hostname specified
Steps to reproduce
Set YAML inventory hosts to the following and use rsync recipe for deploy
production:
  hostname: host1
  user: user1
  forwardAgent: true
  multiplexing: false
  stage: production
  deploy_path: "~/deploy"

Upon deployment, at the Rsync stage, the command fails to connect to the host, due to the rsync attempting to connect to hostname "production" instead of the expected hostname "host1"
[Symfony\Component\Process\Exception\ProcessFailedException]
  The command "rsync -rz --delete --exclude='.env*' --exclude='.git*' --exclude='deploy.php' --exclude='hosts.yaml' --exclude='node_modules' '/example/build/current/' 'user1@production:~/deploy/releases/20170626055543/'" failed.
  Exit Code: 255(Unknown error)
  Working directory: /example
  Output:
  ================
  Error Output:
  ================
  Host key verification failed.
  rsync: connection unexpectedly closed (0 bytes received so far) [sender]
  rsync error: unexplained error (code 255) at /BuildRoot/Library/Caches/com.apple.xbs/Sources/rsync/rsync-51/rsync/io.c(453) [sender=2.6.9]

It should be working correctly: https://github.com/deployphp/deployer/blob/master/src/Host/Host.php#L67
Please, provide reproducible example with deploy.php and inv.yml

@antonmedv - please see example deploy.php + hosts.yaml is attached below. I've also updated the ticket to reflect this is affecting v5.1.3 (which was the version i had installed during this mockup test).
Have not confirmed this, but i think i had another project using "upload" method not affected by this issue, so it may be rsync recipe specific.
Example deploy.php
<?php

namespace Deployer;

use Deployer\Task\Context;

require 'recipe/common.php';
require 'recipe/rsync.php';

set('ssh_type', 'native');
set('ssh_multiplexing', true);
set('writable_mode', 'chmod');
set('default_stage', 'production');
set('repository', 'ssh://git@example.com/path/repo.git');
set('rsync', [
    'exclude' => [
        '.env*',
        '.git*',
        'deploy.php',
        'hosts.yaml',
        'node_modules',
    ],
    'exclude-file'  => false,
    'include'       => [],
    'include-file'  => false,
    'filter'        => [],
    'filter-file'   => false,
    'filter-perdir' => false,
    'flags'         => 'rz', // Recursive, with compress
    'options'       => ['delete'],
    'timeout'       => 600,
]);

// Set the release build identifer as the date
$releaseBuild = date('YmdHis');

// set the global release name based on release build number
set('release_name', $releaseBuild);

set('deploy_path', __DIR__ . '/.build');
set('rsync_src', __DIR__ . '/.build/current');

// Hosts
inventory('hosts.yaml');

desc('Cleanup the local deployment');
task('local:cleanup', function () {
    invoke('cleanup');
})->local();

desc('Unlock the local deployment build');
task('local:deploy:unlock', function () {
    invoke('deploy:unlock');
})->local();

desc('Build the project locally');
task('build', function() {
    invoke('deploy:prepare');
    invoke('deploy:lock');
    invoke('deploy:release');
    invoke('deploy:update_code');
    invoke('deploy:vendors');
    invoke('deploy:symlink');

    // Set the local rsync source to the current path
    set('rsync_src', '{{release_path}}');
})->local();

desc('Deploy the project');
task('deploy', [
    'deploy:prepare',        // Create dirs on server
    'deploy:lock',           // Lock deploys on server
    'deploy:release',        // Release number on server
    'rsync',                 // Send files to server
    'deploy:writable',       // Ensure paths are writable on server
    'deploy:shared',         // Shared and .env linking on server
    'deploy:clear_paths',    // Clear paths specified in config
    'deploy:symlink',        // Symlink /current on server
    'deploy:unlock',         // Unlock deploys on server
    'cleanup',               // Cleanup old releases on server
    'local:cleanup',         // Cleanup old releases on local
    'success',
]);
// Build the release locally
before('deploy', 'build');

// If deploy fails automatically unlock.
after('deploy:failed', 'deploy:unlock');
after('deploy:unlock', 'local:deploy:unlock');

Example hosts.yaml
example.com.au:
    hostname: fe1.prod.example.net.au
    port: 22
    user: example
    stage: production
    branch: master
    deploy_path: ~/deploy
    pty: true

staging.example.com.au:
    hostname: fe1.staging.example.net.au
    port: 22
    user: example
    stage: staging
    branch: master
    deploy_path: ~/deploy
    pty: true

dev1.staging.example.com.au:
    hostname: fe1.staging.example.net.au
    port: 22
    user: example_dev1
    stage: dev1
    branch: master
    deploy_path: ~/deploy
    pty: true

When attempting to deploy to stage "staging" or "dev1", during Rsync, the command initiated attempts to rsync with hostname "staging.example.com.au" or "dev1.staging.example.com.au" - the expected behaviour is that this would rsync with hostname "fe1.staging.example.net.au", as outlined in the hosts.yaml preferences.
Example output
➤ Executing task build
✔ Executing task deploy:prepare
✔ Executing task deploy:lock
✔ Executing task deploy:release
✔ Executing task deploy:update_code
✔ Executing task deploy:vendors
✔ Executing task deploy:symlink
✔ Ok
✔ Executing task deploy:prepare
✔ Executing task deploy:lock
✔ Executing task deploy:release
➤ Executing task rsync
The authenticity of host '[dev1.staging.example.com.au]:22 ([x.x.x.x]:22)' can't be established.

Debug mode is showing the following command issued...
rsync -rz -e 'ssh -p22' --delete --exclude='.env*' --exclude='.git*' --exclude='deploy.php' --exclude='hosts.yaml' --exclude='node_modules' '/path/to/.build/current/' 'example_dev1@dev1.staging.example.com.au:~/deploy/releases/20170707060849/'

@antonmedv - found where this is occurring - looks like https://github.com/deployphp/recipes/blob/master/recipe/rsync.php#L139 is the culprit. Issuing a PR now.
    $host = $server->getHostname();

Should be
    $host = $server->getRealHostname();

I'm closing this issue as no activity for more then 2 weeks. I hope you found a solution (please, share with us).
If you still have any problems feel free to open a new issue.

authors: antonmedv matthew-muscat

repo: deployphp/deployer


text:
Using ssh-agent on Windows?

Is it possible to use deployer with ssh-agent on Windows?
When I am connecting with identityFile('~/.ssh/id_rsa.pub', '~/.ssh/id_rsa', 'MyStrongPassword') it's ok. Deployment is successful.
But it's not secure to store MyStrongPassword here. I am trying to use forwardAgent() and all I get is:
Warning: fsockopen(): unable to connect to unix://C:/Users/8523~1/AppData/Local/Temp/ssh-lZz6mszVrxCO/agent.2700:0 (Unable to find the socket transport "unix" - did you forget to enable it when you configured PHP?) in path\to\vendor\deployer\phpseclib\phpseclib\System\SSH\Agent.php on line 97

Notice: Unable to connect to ssh-agent (Error 11073052: Unable to find the socket transport "unix" - did you forget to enable it when you configured PHP?) in path\to\vendor\deployer\phpseclib\phpseclib\System\SSH\Agent.php on line 99

[RuntimeException]
 Unable to login with the provided credentials.

I am using https://msysgit.github.io/ and ssh-agent works great. I just typing "ssh user@hostname" and connection is successful.
Also I can use "pageant" agent (Putty). It works great too.

Error 11073052: Unable to find the socket transport "unix" - did you forget to enable it when you configured PHP?

Couldn't find any information about unix sockets on Windows.

Is it possible to use unix sockets on Windows? [I think answer is NO]
Is it possible to use another socket transport to connect to ssh-agent on Windows?

I have exact same problem on Windows.
@frutality

When I am connecting with identityFile('~/.ssh/id_rsa.pub', '~/.ssh/id_rsa', 'MyStrongPassword') it's ok.

I even could not deploy with identityFile().
When using identityFile(), how could deployment server be authenticated by git server? Did you upload private key to deployment server?

Why is this closed? I'm having same issue.

using windows
everything works with putty+pageant
everything works with ssh+ssh-agent
deployer doesn't work, same error as described above.

stream_get_transports() returns
string (3) "tcp"
string (3) "udp"
string (3) "ssl"
string (5) "sslv3"
string (5) "sslv2"
string (3) "tls"
string (7) "tlsv1.0"
string (7) "tlsv1.1"
string (7) "tlsv1.2"

How exactly do I "enable unix:// socket transport" in my php.ini in windows?
If unix:// transport is not available, maybe you could implement some other way of using ssh-agent (or pageant) - the way default utilites like ssh do it.

@dima-stefantsov this issue is out of date.
You can make a new issue if you have problem.
Please show me your enviroment.
Current i am using forward agent success on windows (git bash + ssh-agent).

Hi @oanhnn, thanks for your reply.
With stream_get_transports() I have shown that I don't have unix:// transport available/installed. That is certainly the reason it fails for me.
I am using latest wamp on windows, php 5.6 or 7.0.8. Since it works for you, you must have unix:// transport. How do I get one?

Problem with ssh in php is what native extension ext-ssh for php does not support this, and pure php implementation of ssh (phpseclib) supports it only on unix-like systems.
So here a few way to solve this issue:

Implement forwardAgent in ext-ssh.
Add windows support to phpseclib.
Or add new ssh transport to deployer.

I'm going to do last one in next v4. But do not know then it will be done.

Reopened in #808

authors: dima-stefantsov elfet frutality oanhnn vanquang9387


text:
The way to 100% test coverage (part 1)

Q
A

Bug fix?
Yes

New feature?
No

BC breaks?
No

Deprecations?
No

Fixed tickets
N/A

I've started to fix tests to get 100% coverage. It's important image thing as for me.

I added each class refactoring as separate commit os it's convenient for cherry-pick.

First of all, thank you for your contribution to the project. I'm really appreciate that.
In open source there is a simple rule: no is temporary, yes is forever. If i say yes to this PR, we'll need to support those tests for every changes in recipes, and them are rapidly comes in. Those tests may bring 100% code coverage, but in coast of maintenance and slowing changes adoptions. In this project i nwhat ever aims to 100% code coverage. In fact i'm going to separate tests in two layers: one on unit tests for core (all what's in src and responds for task running and configuration) and integration tests for all recipes, tests what will be testing critical for recipes stuff, like success deploy, rollback, etc (i already start working on this, but code not in public for now). So, as maintainer of Deployer, i'm saying no to this PR for now.

@elfet Bad news for me.
Do you have a written list of main goals, principles and development trends of the project? Don't want to waste time for useless things.

@elfet

What're main features that you're planning in v5 and working right now?
What do you recommend to do for contributors?

Main feature it's deprecating other ssh clients and leaving only native ssh, so i want to fix as much as possible in v4 (while there is still way of using it if there is a bug).
For now, there is two labels: good for beginner and advanced+level. If you are planning something big it's good to open issue to discus it.

source: gharchive/pull-request

authors: elfet pluseg


text:
Change EventDispatcher versions

Q
A

Bug fix?
Yes

New feature?
No

BC breaks?
No

Deprecations?
No

Fixed tickets
N/A

We are using a project with Symfony 2.7 . The Deployer without namespaces cause conflict with the hamcrest/hamcrest-php library. The event dispatcher version is too strict with the ^3.1.

CI fails.

Yes, because there are some error in original code: https://github.com/deployphp/deployer/commits/4.x  There are many red X .
The CI response: https://travis-ci.org/deployphp/deployer/jobs/161881831
Your requirements could not be resolved to an installable set of packages.
  Problem 1
    - phpunit/phpunit 5.5.5 requires php ^5.6 || ^7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
    - phpunit/phpunit 5.5.4 requires php ^5.6 || ^7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
    - phpunit/phpunit 5.5.3 requires php ^5.6 || ^7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
    - phpunit/phpunit 5.5.2 requires php ^5.6 || ^7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
    - phpunit/phpunit 5.5.1 requires php ^5.6 || ^7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
    - phpunit/phpunit 5.5.0 requires php ^5.6 || ^7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
    - phpunit/phpunit 5.4.8 requires php ^5.6 || ^7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
    - phpunit/phpunit 5.4.7 requires php ^5.6 || ^7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
    - phpunit/phpunit 5.4.6 requires php ^5.6 || ^7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
    - phpunit/phpunit 5.4.5 requires php ^5.6 || ^7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
    - phpunit/phpunit 5.4.4 requires php ^5.6 || ^7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
    - phpunit/phpunit 5.4.3 requires php ^5.6 || ^7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
    - phpunit/phpunit 5.4.2 requires php ^5.6 || ^7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
    - phpunit/phpunit 5.4.1 requires php ^5.6 || ^7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
    - phpunit/phpunit 5.4.0 requires php ^5.6 || ^7.0 -> your PHP version (5.5.38) does not satisfy that requirement.
    - Installation request for phpunit/phpunit ~5.4 -> satisfiable by phpunit/phpunit[5.4.0, 5.4.1, 5.4.2, 5.4.3, 5.4.4, 5.4.5, 5.4.6, 5.4.7, 5.4.8, 5.5.0, 5.5.1, 5.5.2, 5.5.3, 5.5.4, 5.5.5].

The PHPUnit 5.x doesn't support the PHP 5.5 . Shall I remove this version from .travis.yml or will you do it?

I made a different branch...


text:
Update mfetch to get the wm in a simpler way

Hey guys! Just wanted to say, that I love this project, I really like the aesthetic and the simple code, I decided to make it even simpler and get the wm without depending of ~/.xinitrc (and it saves 1 line). I hope you guys like this pr, this is how it looks.

I also played around with the README.md, you can change it if you want.
Greetings,
Jimmysit0.

This assumes that XDG variables are set, which isn't always the case.

repo: depsterr/mfetch


text:
🛑 Derailed Home is down

In 69aa7f6, Derailed Home (https://derailed.one) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Derailed Home is back up in 3b272b7 after 326 days, 20 hours, 21 minutes.

repo: derailed-org/derailed-oss-status


text:
🛑 Derailed Home is down

In ffe9863, Derailed Home (https://derailed.one) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Derailed Home is back up in 9ee13d3.


text:
Puppet stuck modifying rpc services on every run with NFS v4 client.

I setup an NFS v4 client using the following basic configuration.
class { '::nfs':
  client_enabled      => true,
  nfs_v4_client       => true,
  nfs_v4_idmap_domain => $facts['domain'],
}

Nfs::Client::Mount <<| nfstag == 'utilsrv1' |>>

Puppet seems to be stuck permanently making the following correct changes on every run.
Service[rpcbind.service]
enable changed 'true' to 'false'
Service[rpcbind.socket]
enable changed 'false' to 'true'
I'm running Puppet Enterprise 2016.5.2 and my client server is running Oracle Linux 7.4.

Hi,
thanks for reporting this bug. This should be the same bug like described in #70, and could be fixed by splitting the rpcbind service and socket like in https://github.com/derdanne/puppet-nfs/blob/41d1cc694539a0c69fdb3dccf710b26d57e06896/manifests/params.pp#L154.
I have no experience with Oracle Linux. Which $::osfamily does it use? Would you be able to add the support of Oracle Linux as a pull request?
Best regards,
Daniel

Oracle Linux 7 is an RHEL 7 clone except that we are using their Unbreakable Linux kernel instead of the Red Hat compatible kernel. The rpcbind.service is also set to indirect, but I'm curious about rpcbind.socket since I'm showing it is enabled, but Puppet keeps saying it is changing enable from false to true.
[root@neil-test www]# systemctl is-enabled rpcbind.service
indirect
[root@neil-test www]# systemctl is-enabled rpcbind.socket
enabled
[root@neil-test www]# puppet agent -t
Info: Retrieving pluginfacts
Info: Retrieving plugin
Info: Loading facts
Info: Caching catalog for neil-test
Info: Applying configuration version '1518702490'
Notice: /Stage[main]/Nfs::Client::Service/Service[rpcbind.service]/enable: enable changed 'true' to 'false'
Notice: /Stage[main]/Nfs::Client::Service/Service[rpcbind.socket]/enable: enable changed 'false' to 'true'
Notice: Applied catalog in 5.80 seconds

I will need some time to setup a testing environment with Oracle Linux, unfortenately i can't test versus PE.

I can't imagine Oracle Linux would operate any different vs RHEL in this regard. If there are any changes you would like me to test, I'm open to helping.

Oracle Linux unfortenately does operate differently than RHEL 7, but it could also be PE which behaves differently. When you look at #55, you'll see that we've applied a fix for handling rpcbind.service and rpcbind.socket within RHEL 7 and CentOS 7.

Which version of Puppet are you testing against? The version of Puppet in PE 2016.5.2 is 4.8.

Here's what I see in your params.pp for RedHat family.
        $client_nfsv4_services      = { 'rpcbind.service' => {
                                           ensure => 'running',
                                           enable => false,
                                         },
                                         'rpcbind.socket' => {
                                           ensure => 'running',
                                           enable => true,
                                         },
                                       }

Based on that, what I am seeing makes perfect sense. If I manually disable rpcbind.service, it disables rpcbind.socket. So you are effectively disabling and then enabling rpcbind.socket on every run.
I did a little searching and found this.
https://github.com/ghoneycutt/puppet-module-rpcbind/issues/27
This indicates that Puppet 4.9 changed the behavior for indirect services so that enable = false, so that should solve this problem. Since I'm still on Puppet 4.8, it would appear it is time for me to upgrade Puppet.

OK, thanks for investigatting. Can you please report back, if the update to a Puppet version >=4.9 does fix this issue?

Is rpcbind service still needed for NFSv4?

This issue should be fixed with PR #81, can you verify if the latest release also works for you?

I am experiencing the same issue with CentOS 7.4 and Puppet 5.5 (community version).  I will try pulling the PR#81 code to see if that solves the issue for me.

Hi @dgoldsmith, could you approve, this is working for you?

Sorry for the lack of reply.  Yes, using PR #81 solved the issue for me.

Thanks!

authors: derdanne dgoldsmith jurim76 nvergottini

repo: derdanne/puppet-nfs

license_source: bigquery


text:
Plugin causes folds to not work properly.

I've been using the plugin since started using Scala and it is awesome, thanks for the work! Recently I started using Intellij for jumping into function definitions and noticed that the imports in the file are folded. So I've added a syn region foldImports start="import" end=/import.*\n^$/ fold keepend configuration to my .vimrc file to fold import statments. However I noticed that line of code did not work with vim-autoclose or vim-scala. When I removed the plugins the fold worked as expected. Here are examples of it not folding and folding. You can ignore the fold on the object in the first image.
Not folding.

Folding

I'm not sure what you're asking for here.  You want imports to be folded by default?

He added a custom line of config to get imports to fold automatically, but when vim-scala is active for scala filetype that custom syntax rule doesn't work. I can replicate that, but I'm not sure why it happens yet…

Is there a way to turn off the folds created by vim-scala such that I'm able to use the custom definition to fold on imports?
@derekwyatt That was unclear, basically what @ches mentions above. When vim-scala is active the syntax rule I defined doesn't work properly.

authors: Petesta ches derekwyatt

repo: derekwyatt/vim-scala


text:
Small video screen on chrome and android

Checklist:

Please try updating to the latest available version.
Please try clearing your browser cache.

[REQUIRED] Card diagnostic information:

card_version: 6.0.2
browser: >-
  Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like
  Gecko) Chrome/131.0.0.0 Safari/537.36
date: 2024-12-10T16:13:55.070Z
lang: en
timezone: Europe/Amsterdam
git:
  hash: 216ded2
  build_date: Tue, 10 Dec 2024 02:14:31 GMT
  commit_date: Mon, 9 Dec 2024 18:13:23 -0800
ha_version: 2024.12.1
integrations:
  reolink:
    detected: true
  frigate:
    detected: true
    version: 5.5.1
  hass_web_proxy:
    detected: false
  motioneye:
    detected: true
config:
  type: custom:frigate-card
  cameras:
    - camera_entity: camera.voortuin
      live_provider: ha
  view:
    default: live
  menu:
    style: outside
  performance:
    profile: high
    style:
      border_radius: true
      box_shadow: true

[REQUIRED] Description of problem:
When i open HA on android very often 9/10 the image of the feed is small. i need to click the fullscreen button and than size it down again to make it visible.  This hapens also on google chrome.

[OPTIONAL] Last working release (if known):
Unknow it was happening in version 5 also.
[OPTIONAL] Javascript errors shown in the web inspector:

[OPTIONAL] Additional information:

When i open HA on android very often 9/10 the image of the feed is small. i need to click the fullscreen button and than size it down again to make it visible. This hapens also on google chrome.
Unknow it was happening in version 5 also.

Android and Chrome. In both v5 and v6. I have not seen this before, and Android and Chrome are very commonly used with the card -- makes me wonder if you have some other card/Javascript/setting that is causing breaking this.
Perhaps try making a brand new dashboard, with a single view, and a single Frigate card (simplest config). Do you still see this issue?

HI @dermotduffy,
Thank you for your reply,
When i updated to 6.0.3 everything looks like its working again.

repo: dermotduffy/frigate-hass-card


text:
Automerge partykit integration

Description of the Problem / Feature

Scale automerge sync server with Partykit and cloudflare workers

Explanation of the solution

Disable WebSocket Sync server used with Automerge repo in Desci-repo project
Implement Custom Partykit WebSocket client Adapter and Postgres Storage adapter that integrates well with Automerge repo and cloudflare sync server.
Setup local cloudflare (wrangler/workered) deployment with docker/docker compose for local development setup
Add Github actions for deployment of cloudflare workers on staging and production
Setup/deploy hyperdrive database tunnel and test

Todo checklist

[x] Integrate party module into local dev setup
[x] Setup party module k8s deployment for staging and prod
[x] Integrate partySocket in desci repo (manage connections properly) or connect through desci-server

Pre-deployment merge todos

[ ] Update kubernetes Vault with new cloudflare websocket server routes (PARTY_SERVER_URL)
[ ] Add cloudflare Api token secret to k8s vault (PARTY_SERVER_TOKEN)
[ ] Set API_TOKEN value on deployed worker secrets and redeploy
[ ] Set ENABLE_PARTYKIT_FEATURE=0 on staging and prod vault, after deployment of cloudflare workers, update env to flag to 1

stale

repo: desci-labs/nodes


text:
feat: setup app wiring

Description
Closes: #XXXX
This PR setup app v2 app wiring feature.

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[x] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[x] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] followed the guidelines for building modules
[ ] included the necessary unit and integration tests
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[x] reviewed "Files changed" and left comments if necessary
[x] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed ! in the type prefix if API or client breaking change
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

@RiccardoM Rebased

@RiccardoM Rebased

repo: desmos-labs/desmos


text:
ERR_BUFFER_OUT_OF_BOUNDS

Getting error unable to authenticate with Rcon, definitely attempting to connect as can see all the connections starting and ending in the log however it's not authenticating and is giving the error message in title.
This is stopping all messages from Discord not to post to the server.
Password is correct.

`[INFO] Using the Discord bot to send messages
[INFO] Using configuration for local log file at "/home/mcserver/serverfiles/logs/latest.log"
[INFO] Authenticated with 85.159.211.247:25575
[ERROR] Could not send password to Rcon server!
RangeError [ERR_BUFFER_OUT_OF_BOUNDS]: Attempt to write outside buffer bounds
at boundsError (internal/buffer.js:47:11)
at checkBounds (internal/buffer.js:30:5)
at checkInt (internal/buffer.js:37:3)
at writeU_Int32LE (internal/buffer.js:647:3)
at Buffer.writeInt32LE (internal/buffer.js:816:10)
at Rcon.sendPackage (/home/mcserver/shulker/build/Rcon.js:144:14)
at Rcon. (/home/mcserver/shulker/build/Rcon.js:110:61)
at step (/home/mcserver/shulker/build/Rcon.js:33:23)
at Object.next (/home/mcserver/shulker/build/Rcon.js:14:53)
at /home/mcserver/shulker/build/Rcon.js:8:71
[ERROR] Could not auth with the server!
RangeError [ERR_BUFFER_OUT_OF_BOUNDS]: Attempt to write outside buffer bounds
at boundsError (internal/buffer.js:47:11)
at checkBounds (internal/buffer.js:30:5)
at checkInt (internal/buffer.js:37:3)
at writeU_Int32LE (internal/buffer.js:647:3)
at Buffer.writeInt32LE (internal/buffer.js:816:10)
at Rcon.sendPackage (/home/mcserver/shulker/build/Rcon.js:144:14)
at Rcon. (/home/mcserver/shulker/build/Rcon.js:110:61)
at step (/home/mcserver/shulker/build/Rcon.js:33:23)
at Object.next (/home/mcserver/shulker/build/Rcon.js:14:53)
at /home/mcserver/shulker/build/Rcon.js:8:71
[DEBUG] Sending command "/tellraw @a [{"color": "white", "text": " Working really?"}]" to the server
Unexpected rcon response -1 2
[DEBUG] Received [04:24:02] [RCON Listener #2/INFO]: Thread RCON Client /85.159.211.247 started
[DEBUG] Regex could not match the string:
Received: "[04:24:02] [RCON Listener #2/INFO]: Thread RCON Client /85.159.211.247 started", Regex matches lines that start with: "[Server thread/INFO] [.]:"
[ERROR] Could not send command!
Server sent no request in 5 seconds
[DEBUG] Rcon closed!
[DEBUG] Received [04:24:07] [RCON Client /85.159.211.247 #13/INFO]: Thread RCON Client /85.159.211.247 shutting down
[DEBUG] Regex could not match the string:
Received: "[04:24:07] [RCON Client /85.159.211.247 #13/INFO]: Thread RCON Client /85.159.211.247 shutting down", Regex matches lines that start with: "[Server thread/INFO] [.]:"
`

Apologies for the delay. Could you provide what server version and/or software you were running? Not sure what is the problem here.

license: isc


text:
Cron NOT working

Discussed in https://github.com/orgs/deta/discussions/250

Originally posted by luizfelipevbll June  5, 2022
I am using using deta 1.1.0 with Python with the exactly same code in the docs
Every time I try to run the code I get this error: ImportError: cannot import name 'App' from 'deta'
from deta import App
from fastapi import FastAPI

app = App(FastAPI())

@app.get("/")
def http():
    return "Hello Deta, I am running with HTTP"

@app.lib.cron()
def cron_job(event):
    return "Hello Deta, I am a cron job"

I checked the deta code in deta/init.py
This piece of code in deta lib is not working:

The exception raised is No module named 'detalib'

hi, left you a response https://github.com/deta/docs/discussions/250#discussioncomment-2889621

authors: abdelhai luizfelipevbll

repo: deta/docs


text:
新增第七章翻译

专有名词 InnoDB 增译，专有名词 Mysql Shell 保留，比较性部分使用括弧增补，其余部分直译，参考信息中的三级目录保留，不做翻译。

请求协助关闭昨日PR，今日我已经修改了URL以及其他内容，推送到自己仓库，发现无法在上次的PR状态下继续推送合并版本，以下为更新内容：

去掉未翻译部分的URL
将方括号从英文替换为中文
增加一行换行
增加约两个空格
因担心URL后跟中文句号会导致浏览器解析错误，遂跟随原文格式
删除第三十四行错误文本“666666”

Sent from Mail for Windows
From: Moon Wu
Sent: Thursday, April 13, 2023 10:19 AM
To: dev2007/mysql8-manual
Cc: Caesar-Victory; Author
Subject: Re: [dev2007/mysql8-manual] 新增第七章翻译 (PR #3)
@dev2007 commented on this pull request.
建议PR只提交自己完成的部分。感谢。
In docs/7/7.1/backup and recovery type.md:

@@ -0,0 +1,100 @@
+# 7.1 备份和恢复的数据类型
建议翻译完成一篇文档，一小节或者一单后再提交 PR。要不然会混入很多这种英文的进来。即使它没有添加在目录中。
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


text:
[0.5pt] Create MusicAlbum class (Team member #2)

Create MusicAlbum class in a separate .rb file.

Hello @kene-creator
can you please have a look at this pull request
https://github.com/devMunyi/catalog-of-my-things/pull/75

repo: devMunyi/catalog-of-my-things


text:
Support for ESP DEVKIT V1 (generic from ali express)

I have one of these: https://www.aliexpress.com/item/3256801743620798.html?spm=a2g0o.order_list.0.0.120a1802llozYz (the DEVKIT V1), which I thought was a DIOT, but I'm not certain anymore. It's been flashing as the DIOT just fine, but who knows. Anyway, I've flashed it and it's discovering the bot, but it won't discover the commands like my wrover will.
Here's a screenshot from MQTT explorer with only relevant parts (note how the bottom is missing (set = ):

They are the both connecting to the same device (but even when I only had the bottom one on, it didn't work.
Here's my config (expires in a month, but I can always repaste if needed): https://pastebin.com/7TbCT4FJ
Logs from serial monitor (with debug on): https://pastebin.com/jviG7Mtc
Without debug, it basically is just:
2:58:30.327 -> lld_pdu_get_tx_flush_nb HCI packet count mismatch (1, 2)
22:58:30.664 -> lld_pdu_get_tx_flush_nb HCI packet count mismatch (1, 2)
22:58:31.991 -> lld_pdu_get_tx_flush_nb HCI packet count mismatch (1, 2)
22:58:33.069 -> lld_pdu_get_tx_flush_nb HCI packet count mismatch (1, 2)
22:58:34.727 -> lld_pdu_get_tx_flush_nb HCI packet count mismatch (1, 2)
22:58:35.021 -> lld_pdu_get_tx_flush_nb HCI packet count mismatch (1, 2)
22:58:36.353 -> lld_pdu_get_tx_flush_nb HCI packet count mismatch (1, 2)

So I'm thinking that this is an issue with something going wrong with the model maybe? But I'm too new to microcontrollers to know. Any thoughts? Ways I can help you debug?

it all looks good from here. you can see a "REQUESTSETTINGS" command was sent and success so I think youve compile and uploaded correctly. "REQUESTSETTINGS" is also a command sent to the bot to get firmware, holdsecs values. It just doesn't make the bot move. The rssi of -83 isnt great but it did at least work once
what seems to be the issue is qith MQTT broker somewhere. if you sent a command, no matter what it should show in the 'set' topic. Even with the esp32 unplugged as this is done by the MQTT broker
you can try using a separate MQTT username and password for the 2nd esp32

Hmmm, having same generic ESP-32 38Pin and just finished setting everything up, and it's working, but there is really big delay between commands sent from ESP to Switchbot Curtain, and having same log as @Jacksonbm1
21:27:35.377 -> lld_pdu_get_tx_flush_nb HCI packet count mismatch (1, 2)
basically this start to spam serial port once command is sent

@scetu those logs are normal and sent from the NimBLE library. No way to turn them off from what I can tell. They aren't sent from my code directly
what is the rssi linkquality value for the curtain device. This value shows the BLE signal quality. How close is the esp32 to the curtain? the closer the better

@devWaves ESP is 3-4 meters from curtains, without obstacles

@scetu you might have a dud esp32. -88 is not a good signal. You want something at least in the -70s
You are sure the esp32 you have doesn't need an external antenna plugged into it?

@devWaves ok, I will investigate it more tommorow, or will try to move it further. Thanks for suggestion, I will let you know, I've sent command from HA and curtains did not respond at all but I will check it tommorow and this was in log
Logger: homeassistant.components.mqtt.discovery
Source: components/mqtt/discovery.py:107
Integration: MQTT (documentation, issues)
First occurred: 12:37:46 AM (12 occurrences)
Last logged: 12:37:48 AM

Received message on illegal discovery topic 'homeassistant/sensor/Závěsy_-_Levá_strana/linkquality/config'
Received message on illegal discovery topic 'homeassistant/sensor/Závěsy_-_Levá_strana/illuminance/config'
Received message on illegal discovery topic 'homeassistant/sensor/Závěsy_-_Levá_strana/position/config'
Received message on illegal discovery topic 'homeassistant/binary_sensor/Závěsy_-_Pravá_strana/calibrated/config'
Received message on illegal discovery topic 'homeassistant/binary_sensor/Závěsy_-_Levá_strana/calibrated/config'

@scetu spaces in the names are replaced with _ and you have a space before and after the - . HA might also might not like the special characters. These would be from HA, not the esp32
not sure what HA doesn't like but I would suggest a name like Zavesy_Leva_strana  if you are having issues

@devWaves ok after playing a bit, deleting and renaming device, it works more responsively - there is sometimes delay 2-3 secs, but it works and no errors in logs and all commands are executed in queue.
Thanks for your great work!

Huh, really weird, it works now fine after plugging back in. Didn't change anything to my knowledge.

Nvm. It doesn't work. What ends up happening is I have two esp32s. The first works, but the second (the one with the code here) doesn't seem to actually work. The weird thing is that it's information shows up in mqtt explorer, but it seems like it's trying to send the command through the wrong esp. When you say:
"you can try using a separate MQTT username and password for the 2nd esp32, but it probably won't fix things as the issue is with MQTT receiving the set command, not the esp32"
Isn't this the password for my broker (set up through HA)? Why would it be different? Maybe that'll solve.

@Jacksonbm1 if you ever loaded multiple esp32s with the same devices that could mess things up. If you aren't using MQTT for other things the easiest things to do is delete everything under the homeassistant and switchbot MQTT topics using MQTT Explorer (with the esp32s unplugged), then plug them back in. This will start things from scratch It is possible you have a device incorrectly associated to the wrong esp32 topic
You can create different username/passwords for MQTT HA by simply creating a new user in HA. Use those credentials

@scetu the delay of 2-3 secs is caused by poor signal. A signal of something like -50rssi will have no delay (or close to it)

Ok I will try that. Little scared to do that bc it might nuke my setup, but I will give it a go

Yeah I've been doing that. I got it all working (I did, unfortunately reflash the wrong esp32, but that's fine, I'm 99% of the way to getting that one back online to what it's original purpose was). The great thing is, I got the bots working. Two were trying to connect to the same device and I think that was causing problems! I think it's now fixed, though! Thank you for your help! Also, semi-possible that one of them had a bad bluetooth receiver (I shelved it because I thought it was causing the issues, but I'm thinking it actually wasn't)

authors: Jacksonbm1 devWaves scetu


text:
Permissions

Users who do not have permissions to delete things can do so through the checkbox lists and selecting "Delete selected".

Thanks, I will have a look over it. Or you can fork and create a pull request with the solution

deleteAny()
Add deleteAny() to your policies. That should help. I could provide a PR

authors: Hussam92 creatobg mihaisolomon

repo: devaslanphp/project-management


text:
dataview代码能否隐藏

obsidian中有dataview, 发布到官网能否显示dataview检索的内容呢?
如果不能,dataview代码是不是可以隐藏呢? 目前发布到官网,就是显示dataview的代码

https://www.weqoocu.com/11410.html

dataview 的话，首先我要试试能不能去手动调用并且拿到结果插入到笔记里面，否则就只能移除了

dataview 的话，首先我要试试能不能去手动调用并且拿到结果插入到笔记里面，否则就只能移除了

是的,期待大神的解决方案

dataview 的话，首先我要试试能不能去手动调用并且拿到结果插入到笔记里面，否则就只能移除了

期待能解析dataview, 或者隐藏dataview， 文章中有dataview发布代码出来，太影响视觉了

求支持dataview内容读取出来，一起发布出去

authors: devbean musicleaf weqoocu

repo: devbean/obsidian-wordpress


text:
Nix feature doesn't work on GitHub Codespaces universal image

Cross-post of existing issue report for visibility: https://github.com/NixOS/nix/issues/6680#issuecomment-1311559039

There seems to be some sort of interaction with the universal image here that needs to be investigated.  The Dev Container Feature does work with other images (as you can see in the test battery).  Will investigate further.

Is there a workaround to this? Until February 14th it was working for us on base:jammy, but now it gives a similar error and falls back on the fallback image.

@CSchank Hmm. I am currently able to build using mcr.microsoft.com/devcontainers/base:jammy and mcr.microsoft.com/devcontainers/base:bullseye. Its possible there was an issue with a specific version of nix.

authors: CSchank Chuxel NickHu

repo: devcontainers/features


text:
🛑 Auth API is down

In 96be6f9, Auth API (https://auth.melodymocktail.com) was down:

HTTP code: 502
Response time: 91 ms

Resolved: Auth API is back up in 6709efb after 4 hours, 28 minutes.


text:
Official adapter for enzyme

IMHO,
If enzyme-adapter-preact-pure is the official adapter for enzyme,
I think it is better to indicate it in the release note.

Are you asking us just to add a link to this package in the release notes? I did also submit a PR to the Enzyme project updating the link in the official Enzyme docs, but they haven't had a new release yet.

Are you asking us just to add a link to this package in the release notes?

Yes, I am.
It makes the package to be easily found.

The adapter needs to be mentioned in the preactjs.com docs when it is updated for Preact v10. In the meantime I have updated the changelog for the 10.0.0-beta.0 release with a link as you suggested. Thanks for the feedback 🙂

repo: developit/preact

license: mit


text:
Fix codecov action in main branch

What does this PR do?
Fixes the codecov action in main, which is currently failing.
Controller tests that use envtests depend on CRDs being available to install into the test cluster. Since these CRDs are not committed to the repository, it is necessary to download them for envtest tests to function correctly.
What issues does this PR fix or reference?
N/A
Is it tested? How?
Tested by pushing changes my fork's main branch.
PR Checklist

[ ] E2E tests pass (when PR is ready, comment /test v8-devworkspace-operator-e2e, v8-che-happy-path to trigger)

[ ] v8-devworkspace-operator-e2e: DevWorkspace e2e test
[ ] v8-che-happy-path: Happy path for verification integration with Che

This PR also installs goimports in the main GH action to avoid unnecessarily reformatting code (make fmt defaults to using go fmt ./... if goimports is not installed)

/retest

Codecov in main is now working: https://github.com/devfile/devworkspace-operator/actions/runs/3733560143/jobs/6334506572

repo: devfile/devworkspace-operator


text:
default impl is missing for ListSystemSettings { }

default implementation is missing for ListSystemSettings object.
we are trying to call.
ListSystemSettings::default()
but it is not properly handle the call.

issue resolved.

repo: device-automation-bus/dab-adapter-rs


text:
리스트 추가 요청입니다. @beyondthedream

이름: Yena Kwon
Nomad Study 기수: 1기
노마드 코더 프로필: https://nomadcoders.co/users/ynkwon114ve29h
깃 프로필: https://github.com/beyondthedream
감사합니다 데비난님~~~
Originally posted by @beyondthedream in https://github.com/devinan/nomadstudy-github-list/issues/3#issuecomment-1189851058

안녕하세요 @beyondthedream 님, 한국시간 오늘 오후 9시에 master 브런치로 등록 하겠습니다.
노마드 스터디 깃헙 리스트에 추가 요청해주셔서 감사합니다.🥳

master 브런치로 등록 처리 완료했습니다.

repo: devinan/nomadstudy-github-list


text:
build not working with Java11

Building devon4j wiki with JDK 11 error leads to problems with maven-antrun-plugin. Mainly due dropped Nashorn support in JDK 11.

org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal org.apache.maven.plugins:maven-antrun-plugin:1.8:run (pre-process-asciidoc) on project devon4j-doc: An Ant BuildException has occured: The following error occurred while executing this line:
D:\workspace\cg\java_11\Devon-dist_3.0.0\workspaces\devon4j-wiki\devon4j.wiki\target\antrun\build-main.xml:15: Unable to create javax script engine for javascript
around Ant part ...... @ 5:30 in D:\workspace\cg\java_11\Devon-dist_3.0.0\workspaces\devon4j-wiki\devon4j.wiki\target\antrun\build-main.xml
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:215)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:156)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:148)
at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:117)
at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:81)
at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build (SingleThreadedBuilder.java:56)
at org.apache.maven.lifecycle.internal.LifecycleStarter.execute (LifecycleStarter.java:128)
at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:305)
at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:192)
at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:105)
at org.apache.maven.cli.MavenCli.execute (MavenCli.java:956)
at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:288)
at org.apache.maven.cli.MavenCli.main (MavenCli.java:192)
at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0 (Native Method)
at jdk.internal.reflect.NativeMethodAccessorImpl.invoke (NativeMethodAccessorImpl.java:62)
at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke (DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke (Method.java:566)
at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced (Launcher.java:289)
at org.codehaus.plexus.classworlds.launcher.Launcher.launch (Launcher.java:229)
at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode (Launcher.java:415)
at org.codehaus.plexus.classworlds.launcher.Launcher.main (Launcher.java:356

There is already an open issue for it below
https://issues.apache.org/jira/browse/MNG-6275

Maybe we can find an alternative to JavaScript for doing the advanced replacements:
https://github.com/devonfw/devon-docgen/blob/1520c04ad107c032d44440ba690072c1a8b52a2a/pom.xml#L189
We can check what ant supports. Maybe Java via groovy or beanshell is also possible.

Done with PR #6

repo: devonfw/devon-docgen


text:
mysql/oracle/postgres/sqlserver schema's are not working

currently we have schema for mysql/postgres/oracle/sqlserver but they are not working
Only h2 and hana schema's are working

Not valid anymore as other DB support apart from h2 is removed.

authors: ssarmokadam vapadwal

repo: devonfw/my-thai-star


text:
Documentation -divided the README file into parts

Created new files of documentation added to new folder documentation.
Modified README file.

Also there seems to be some merge conflict here, could you resolve this @bartoszbicz96 ?

@bartoszbicz96 thanks for this PR.
However, was there any dicussion or rationale in advance to do this?
Many github projects want to show the main aspects and features at a glance in the REAME as this is the first thing people find on github.
Was it too overwhealming?
Should we should ask the community wheter it is better to have it all in one place or split it up into tiny little bits?
Further we should also consider document generation for this repo like we have for all other projects.
Therefore we need to consider include vs. link, etc..

@hohwille
The goal here was to have the documentation of this repo included in the devonfw guide. I thought splitting the different chapters of our readme into seperate files would make things easier for us with the document generation, as it is that way in other repositories too.
Maybe we could have the first few chapters (Motivation, Installation, Configuration, devonfw Java Quality Profile) displayed normally in the readme, and reference the documents for the different rule types with a link?

Thanks for the explanation. I got the rationale now and indeed we agreed in devonfw to put all documentation into a documentation folder and follow common practices.
As always we observe things in devonfw to grow over time. Therefore we should also consider that it is time to say that we not only have "just a README" here but some real documentation that will get more complex over time. In that case we should follow how it is done in other repos like devon4j or ide where we generate a PDF documentation as part of the build process and also deploy this documentation alongside with every release:

We need to restructure a little and introduce a parent pom with pom packaging
this one should include the current plugin and the documentation as modules
the documentation then also becomes a pom.xml that derives from docgen.
as a result we then get the documentation alongside as e.g. here: https://repo1.maven.org/maven2/com/devonfw/tools/ide/devonfw-ide-doc/2020.08.002/devonfw-ide-doc-2020.08.002.pdf

Would it be possible to update this PR accordingly?

Perfect, thanks for the pointers. Sure, I will connect with Bartosz and we will get on the tasks.
So just to clarify: Is the readme fine as it is in this PR?

authors: bartoszbicz96 hohwille lmarniazman


text:
Add git-lfs to docker action

:memo:  Brief description
Currently the action-commit-push doesn't support repos that have git lfs enabled.  It would be great if the Dockerfile could also install git-lfs so that it won't fail on commits.
https://github.com/devops-infra/action-commit-push/blob/21398281c939ea77c66416756ee63001f51c0a13/Dockerfile#L54 could simply add git-lfs

:warning: Checklist

[ X] Provided a clear and concise description of what the issue is.
[ X] Given a clear and concise description of what is expected.
[ ] Proposed a clear and concise description of any alternative solutions or other features.
[ X] Added any other context or screenshots about the feature request.
[ ] Associated pull request has been already created and link was provided.

Check CONTRIBUTING.md and CODE_OF_CONDUCT.md for more information

Hello. Good idea. I've added it in the newest version v0.5.

repo: devops-infra/action-commit-push


text:
delete parameters

make it possible to delete a parameter, if it exists

at specific level, or all levels of hierarchy (global, device, region)

repo: devsim/devsim


text:
Stdout not showing in light background terminal.

What happened?
When running devspace, output within "inverted commas" isn't visible.
Not sure but https://github.com/devspace-sh/devspace/blob/main/pkg/devspace/dev/dev.go#L36 explicitly mentioned white color is causing the issue
			ctx.Log().Infof("DevSpace UI available at: %s", ansi.Color("http://"+serv.Server.Addr, "white+b"))
dev:app Selected pod voyager-devspace-test-vc-devspace-657b784b97-sgtjt
dev:app open  Opening 'http://localhost:8089/' as soon as application will be started
dev:app ports Port forwarding started on: 3000 -> 3000, 9229 -> 9229
dev:app sync  Sync started on: ./server/dist <-> /home/ubuntu/deployment/voyager/dist
dev:app sync  Waiting for initial sync to complete
dev:app sync  Initial sync completed
dev:app ssh   Port forwarding started on: 10548 -> 8022
dev:app ssh   Use 'ssh app.devspace.devspace' to connect via SSH

It looks something like this in white background.

Looks completely fine with dark background

What did you expect to happen instead?
Wanted stdout to be visible.
How can we reproduce the bug? (as minimally and precisely as possible)
run commands in light background terminal

Sorry, but I don't think this solves the problem, because sometimes this tool is used internally by another script.  I've also seen that the code supports the NO_COLOR env variable, but instead of using a specific colour or cleaning, wouldn't it be better to just make text bold when you want to highlight something? You project is used by many people, and I think everyone would appropriate some accessibility.
Thank you for supporting this project

authors: batizhevsky murarisumit

repo: devspace-sh/devspace


text:
⚠ Mobitopup has degraded performance

In 7178f35, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 6243 ms

Resolved: Mobitopup performance has improved in 1c2290d.


text:
Middleware and Addon ergonomics

I submitted PR #1510  already for adding keys for all request types via the existing hooks middleware.
Some ideas and questions arose in the process, and I'd like to continue contributing.
Is there, or should/could there be a way to disable a previously enabled middleware ?
eg: db.unUse('dbcore','hooks')
How does a contributor and/or maintainer decide if an addon belongs:

in a src/subfolder,
in add-ons (and thus needs the extra package management hastle),
or in a separate repo

??
which order do middleware modules run?
how do promises work in there? (what's the story with PSD eg here)
note: i consciously chose issue over discussion for this, as i hope it will lead to more task checkboxes and more PRs

eg: db.unUse('dbcore','hooks')

@dfahlander  is there such a thing?


text:
Updating values not all ways working

Hi
Great library.
I have a problem with a browser app that I have built for chrome or Firefox.
On a android tablet I have a set of buttons that I press as I pass through the different stages.
When I hit the button it updates the time in the indexeddb and then rebuilds the button to reflect the next stage.
I am finding that indiscriminately it doesn't update the details. As I hit the buttons sometimes it takes 2 or 3 hits to move on to the next stage.
Do i need to reload/refresh my Indexeddb? Can dexie force a reload of the indexeddb?
Any ideas would be appreciated.
Thanks

Read this great article by @nolanlawson:
https://pouchdb.com/2015/05/18/we-have-a-problem-with-promises.html
It's really about understanding promises for understanding how to use the Dexie api or any other Promise based api.

Thanks David. I will
On Fri, Apr 15, 2016 at 2:07 AM, David Fahlander notifications@github.com
wrote:

Read this great article by @nolanlawson https://github.com/nolanlawson:
https://pouchdb.com/2015/05/18/we-have-a-problem-with-promises.html
It's really about understanding promises for understanding how to use the
Dexie api or any other Promise based api.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/dfahlander/Dexie.js/issues/235#issuecomment-210022156

--
Regards
George Malkoun
0402 133 820

authors: dfahlander gmalkoun


text:
IndexedDB is getting bigger on each page refresh

Version: 2.0.3 (from npm)
Structure:
"version": 0.1,
"stores":{
	"cache": "path",
	"upstreamDB": "id,tipo,nombre,coordenadas,localidad",
	"localidadesDB": "id_loc,nombre"
}

There are another properties inside of the upstreamDB, one is a image stored as a blob and another a string from URL.createObjectURL
The script that runs on each refresh does a fetch of about 75 images at a 600x600 res. The cache table only stores cache when there isn't a ServiceWorker (what's not the case now).
The blob and the url reference are stored on the IDB using Dexie with another properties (some of them indexed), but all of them are short strings, booleans and numbers (all <100).
Before it saves all of this data to the IDB, does a db.upstreamDB.clear() The app takes 8MB on first load and it increases (I've noticed that when reached 1300MB) on every reload.

Not sure if this you are referring to app memory or storage size. App memory should not increase with Dexie as it does no caching. Storage size in IndexedDB is not released immediately when you clear a table. It seems IndexedDB does not delete the content that you clear but just mark it as deleted. I suppose it will release the storage eventually when reaching a certain percent of the quota, but this is just a theory though. Most databases works the same - deletion is lazily done for performance reasons.
Not sure if this answers the question enough, but this is an area where I've got similar questions about before and it may depend on what browser implementation your app is running on.

Yes, question solved. It seems to be more likely an upstream issue. I'm
using Google Chrome Stable. I've no tested it on another browsers. The
issue is with the persistent storage. Thanks for replying so fast.
El mar., 15 may. 2018 16:55, David Fahlander notifications@github.com
escribió:

Not sure if this you are referring to app memory or storage size. App
memory should not increase with Dexie as it does no caching. Storage size
in IndexedDB is not released immediately when you clear a table. It seems
IndexedDB does not delete the content that you clear but just mark it as
deleted. I suppose it will release the storage eventually when reaching a
certain percent of the quota, but this is just a theory though. Most
databases works the same - deletion is lazily done for performance reasons.
Not sure if this answers the question enough, but this is an area where
I've got similar questions about before and it may depend on what browser
implementation your app is running on.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dfahlander/Dexie.js/issues/699#issuecomment-389197445,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARxPWIrdt7UgRxc1JGQtFpuA5B2Ysmeaks5tyuxygaJpZM4T_Yvf
.

authors: dfahlander mrvik


text:
style(frontend): remove unnecessary margin bottom in info

Motivation
See screenshots
Changes

span instead of p

Screenshots
Before

After

Duplicated by #2668.

repo: dfinity/oisy-wallet


text:
feat(frontend): blockstream API

Motivation
Unfortunately, we cannot get both transactions and latest block data from Blockchain API since they have a strict API rate limit (1 request per 10 seconds). Therefore, we need to use another provider to get the height of the latest mined BTC block. Llorenc already sent Blockstream API for security review, we will merge it only after it's approved. The code can be already reviewed, though.

LGTM, thx
That said my review is closed to a rubber stamp this time.
Unrelated but related: Can we potentially remove blockchain.info to just use blockstream.info ?

Probably we could, yes. But according to security, we should use two endpoints to fetch the transaction and compare the results before showing them to the user.

@lmuntaner gotcha, thanks for the explanation. That wasn't clear for me reading the description of the PR I have to say.


text:
MIR Navigation start_planner issues.

I get this error when trying to run the roslaunch mir_navigation start_planner.launch.

[FATAL] [1614779970.506633859, 19.521000000]: Failed to create the nav_core_adapter::LocalPlannerAdapter planner, are you sure it is properly registered and that the containing library is built? Exception: Could not find library corresponding to plugin mir_dwb_critics::PathDistPrunedCritic. Make sure the plugin description XML file has the correct name of the library and that the library actually exists.
[move_base_node-3] process has died [pid 10889, exit code 1, cmd /opt/ros/kinetic/lib/move_base/move_base map:=/map odom:=odom_comb marker:=move_base_node/DWBLocalPlanner/markers __name:=move_base_node __log:=/home/user/.ros/log/8d7433ac-7c28-11eb-9552-0242c0a8d007/move_base_node-3.log].
log file: /home/user/.ros/log/8d7433ac-7c28-11eb-9552-0242c0a8d007/move_base_node-3*.log

When I look in the files it looks like I am missing some header files aswell not sure if it is just reference errors in my ide or if this is something faulty aswell. I also had to use the catkin_make_isolated instead of the normal catkin_make since it complained about the workspace containing non-catkin packages in it.
CMake Error:

Base path: /home/user/catkin_ws
Source space: /home/user/catkin_ws/src
Build space: /home/user/catkin_ws/build
Devel space: /home/user/catkin_ws/devel
Install space: /home/user/catkin_ws/install

Running command: "make cmake_check_build_system" in "/home/user/catkin_ws/build"

-- Using CATKIN_DEVEL_PREFIX: /home/user/catkin_ws/devel
-- Using CMAKE_PREFIX_PATH: /home/user/catkin_ws/devel;/opt/ros/kinetic
-- This workspace overlays: /home/user/catkin_ws/devel;/opt/ros/kinetic
-- Using PYTHON_EXECUTABLE: /usr/bin/python
-- Using Debian Python package layout
-- Using empy: /usr/bin/empy
-- Using CATKIN_ENABLE_TESTING: ON
-- Call enable_testing()
-- Using CATKIN_TEST_RESULTS_DIR: /home/user/catkin_ws/build/test_results
-- Found gtest sources under '/usr/src/gmock': gtests will be built
-- Found gmock sources under '/usr/src/gmock': gmock will be built
-- Using Python nosetests: /usr/local/bin/nosetests-2.7
-- catkin 0.7.20
-- BUILD_SHARED_LIBS is on
-- BUILD_SHARED_LIBS is on
-- ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
-- ~~  traversing 9 packages in topological order:
-- ~~  - mir_robot (metapackage)
WARNING: The CMakeLists.txt of the metapackage 'mir_robot' contains non standard content. Use the content of the following file instead: /home/user/catkin_ws/build/catkin_generated/metapackages/mir_robot/CMakeLists.txt
-- ~~  - mir_msgs
-- ~~  - sdc21x0
-- ~~  - mir_description
-- ~~  - mir_gazebo
-- ~~  - mir_navigation
-- ~~  - mir_actions
-- ~~  - mir_driver
-- ~~  - mir_dwb_critics
-- ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
CMake Error at /opt/ros/kinetic/share/catkin/cmake/catkin_workspace.cmake:95 (message):
This workspace contains non-catkin packages in it, and catkin cannot build
a non-homogeneous workspace without isolation.  Try the
'catkin_make_isolated' command instead.
Call Stack (most recent call first):
CMakeLists.txt:63 (catkin_workspace)
-- Configuring incomplete, errors occurred!
See also "/home/user/catkin_ws/build/CMakeFiles/CMakeOutput.log".
See also "/home/user/catkin_ws/build/CMakeFiles/CMakeError.log".
Makefile:318: recipe for target 'cmake_check_build_system' failed
make: *** [cmake_check_build_system] Error 1
Invoking "make cmake_check_build_system" failed

Both errors are weird, and I can't reproduce them. What's your ROS version? Which Git Commit are you on? (roscd mir_robot && git log | head -n1)

I am on the latest commit also tried the one before that and still had the same issues.
user:~$ roscd mir_robot && git log | head -n1commit 9e3ec2cd0102b1140f09ffc765ae5f61b406a59e
I am using the kinetic version. But when I do "rosversion ros" I get  I am currently using ROS through app.theconstructsim.com. Not sure if that would be an issue? Everything else works with the robot and I can control it with the speed controls.

Can you run echo $ROS_DISTRO?

Also:
cat /home/user/catkin_ws/build/catkin_generated/metapackages/mir_robot/CMakeLists.txt
roscd mir_robot
cat CMakeLists.txt

Can you run echo $ROS_DISTRO?
kinetic

cat /home/user/catkin_ws/build/catkin_generated/metapackages/mir_robot/CMakeLists.txt
cmake_minimum_required(VERSION 2.8.3)
project(mir_robot)
find_package(catkin REQUIRED)
catkin_metapackage()
roscd mir_robot
cat CMakeLists.txt
cmake_minimum_required(VERSION 3.5.1)
project(mir_robot)
find_package(catkin REQUIRED)
catkin_metapackage(

I don't see the problem. The only difference is the cmake_minimum_required, and that should be okay. At least catkin_make doesn't complain on my machine. It looks like this is not a bug in mir_robot, so I'm afraid I can't help you.

Could I be that I do not have all the required packages installed? Any easy way to check this?

Issue was that I didnt do the Preliminaries so I was missing packages thank you for the help!


text:
JAXOptMinimize.eval_and_update() has no forward_mode_differentiation keyword argument

Hi!
I got the error below when running the Jaxoplanet transit fitting tutorial. I'm guessing this is due to an API change in how numpyro does optimization and SVI.
A simple fix that worked locally was to add that keyword argument in JaxOptMinimize.eval_and_update(), set it to false by default, and raise an error if it's set to True (so that an attempt to use it does not fail silently). I can submit a PR if this this is a good fix (I don't know if jaxopt supports forward differentiation like numpyro's solvers, in which case I guess it would be better to make it do that instead of raising an error).
Thanks!

Full error output
TypeError                                 Traceback (most recent call last)
Cell In[7], line 22
      8     init_params = {
      9         "t0": T0,
     10         "logP": jnp.log(PERIOD),
   (...)
     15         "u": U,
     16     }
     17     run_optim = numpyro_ext.optim.optimize(
     18         model,
     19         init_strategy=numpyro.infer.init_to_value(values=init_params),
     20     )
---> 22 opt_params = run_optim(jax.random.PRNGKey(5), t, yerr, y=y)

File ~/repos/astro/exoplanet-dev/numpyro-ext/src/numpyro_ext/optim.py:86, in optimize.<locals>.run(rng_key, *args, **kwargs)
     84 state = svi.init(init_key, *args, **kwargs)
     85 for _ in range(num_steps):
---> 86     state, _ = svi.update(state, *args, **kwargs)
     87 info = getattr(state.optim_state[1], "state", None)
     88 params = svi.get_params(state)

File ~/repos/astro/exoplanet-dev/jaxoplanet/venv/lib/python3.11/site-packages/numpyro/infer/svi.py:285, in SVI.update(self, svi_state, forward_mode_differentiation, *args, **kwargs)
    273 rng_key, rng_key_step = random.split(svi_state.rng_key)
    274 loss_fn = _make_loss_fn(
    275     self.loss,
    276     rng_key_step,
   (...)
    283     mutable_state=svi_state.mutable_state,
    284 )
--> 285 (loss_val, mutable_state), optim_state = self.optim.eval_and_update(
    286     loss_fn, svi_state.optim_state, forward_mode_differentiation=forward_mode_differentiation
    287 )
    288 return SVIState(optim_state, mutable_state, rng_key), loss_val

TypeError: JAXOptMinimize.eval_and_update() got an unexpected keyword argument 'forward_mode_differentiation'

Thanks! Adding ref to related issue here: https://github.com/exoplanet-dev/jaxoplanet/issues/160
If you'd be up for opening a PR, that would be great! I think that it's possible that we could actually pass the forward_mode_differentiation argument where needed, rather than throwing an error, but I don't feel too strongly about that, and I rather get something working!
Furthermore, it looks like jaxopt is being deprecated, so we shouldn't spend too much time thinking about this just now.

authors: dfm vandalt

repo: dfm/numpyro-ext


text:
Using proxies the browser steps arent loading (ERR_TUNNEL_CONNECTION_FAILED)

Describe the bug
Fetch method: Playwright Chromium/Javascript via 'ws://playwright-chrome:3000/?stealth=1&--disable-web-security=true'
When using Brightdata proxies the browser steps are not loading. The browserless/chromium docker exits with: Failed to load resource: net::ERR_TUNNEL_CONNECTION_FAILED.
When using the proxied watch without browser steps it works fine.
So, my workaround is now:

Set up watch without proxy
Select browser steps without proxy
Turn on the proxy

I can live with that but I thought I should make a note here.
Version
0.40.1.1
To Reproduce
Add watch
Use Playwright fetch method
Select Brightdata proxy
Try to use browser steps
https://changedetection.io/share/ARfnCoA0Ho4a
Expected behavior
I expect the browser steps to load eventually.
Screenshots
n/a
Desktop (please complete the following information):

OS: Synology docker
Browser: Brave

thanks for the solid report!

@opicron when this happens, can you see something in the output logs like Browser Steps: UUID {} Using proxy {}" ? can you paste it? is that value in what it says what you configured?

I will check shortly, sorry for delay

I am available to debug now. I don't seem to find any log file either in the docker or in the datastore folder. How should I activate logging?

Suprising when I select price only on a entry, with no proxy selected, the error also happens.

I'm having the same issue on version 0.41. Browser steps always throws a 400 error. Also non proxied requests all fail, but the visual filter selector still works.

Closed by this commit, you can use the latest docker image or master branch code https://github.com/dgtlmoon/changedetection.io/commit/cf3f3e44977f72c12d73b5807c4998993d236332

authors: dgtlmoon opicron pedrogius


text:
Add date below the message

Hi I want to know the date when the text is sent, its more of a feature request.
Example)   {type: 'text', author: "them", data: { text: "You've got to have a story.", date: "Tue, 25 Jul 2017 22:53:41 GMT" }}

That's a good idea @vkhattar -- let's think about the other kinds of dates we might need so we get the names right (eg sent, received, read, etc) and add this as a feature req.
It won't make it into the next version, but it may appear later.
Important to note however, you can have your message objects have whatever fields you want. react-chat-window will just ignore extraneous ones. I want to leave this open because I think the UI should have the option of showing some date indicators, but it's not clear what ones or what that might look like.

I did add something locally, but could not make a pull request as I did not fork it. For now i am just adding the date with the text, it looks ugly but does get the job done.

@dharness -- Any update on this feature? I'm planning to use this library for one of my projects and would require this feature as part of it. If you can guide me through the implementation, I can develop it for you and create a PR too :)

I'm pretty swamped, but I'd recommend putting it here:
https://github.com/dharness/react-chat-window/blob/master/src/components/Messages/index.js

I'd like to open this to opinions and suggestions for the design. Should the date appear for every message? Above the bubble, below it, with a time, with the day of week, with relative times (5 minutes ago, yesterday), only at intervals of greater than X time, should there be a break in the convo when it's a new day... there's a lot we need to specify if we want to see this feature!

authors: dharness eswarasai heatherbooker vkhattar

repo: dharness/react-chat-window

license: Unlicense


text:
Check for invalid frame length

https://tools.ietf.org/html/rfc6455 writes the "the most significant bit [of frame length] MUST be 0" so update the code to check for that.

Credit to 4cad@silvertoque for telling me about this.

Credit to Dane (4cad@silvertoque) for telling me about this.

repo: dhbaird/easywsclient


text:
Manifest format improvements

Updated the manifest format and abstracted manifest {de}serialization into a helper module:

two manifest types: "source" and "binary"
neon.type always explicitly says which it is
full validation of the JSON data to avoid sloppy failures during deserialization
relationship between source and binary manifests is much easier to follow in the logic

Should also add legacy compatibility for the binary manifest format that puts the data under neon.binary, since there are published packages with that format.

repo: dherman/neon-rs


text:
Added synchronous GRPC call option for row commit

This pull request adds an async parameter to the commit method that allows to choose between synchronous and asynchronous GRPC calls.
When executing a couple of hundred write commits per second, the commit method that performs an async GRPC call sometimes never returns. The problem doesn't occur when using the synchronous GRPC call.
We haven't identified the root cause yet, but it might be an issue in the GRPC library itself, since we ran into the same problem with a custom Java Bigtable client we wrote.

I'm adding @nathanielmanistaatgoogle since he is the author of the gRPC Python client.

@joerideturck Thanks for using this library! (And filing this issue.)
A few things to note:

We are slowly moving the contents of this library from here into the official gcloud-python so trying somewhat to avoid changes here (though I think this one can go in)
We'll be switching from gRPC alpha to gRPC beta soon, and this may change some of the behavior. IIRC the reason I used async originally was because the gRPC alpha client fails to relay error messages. @nathanielmanistaatgoogle tells me this is fixed in the beta client.
If in fact sync and async should both be used, this is an option we'd likely want to add to all methods (I can handle that after this PR goes in, just worth nothing that you may have stumbled on a useful feature for the whole library)

Are you using the happybase package or the main gcloud_bigtable package? I'd love to hear some design feedback (e.g. #9) either way.

We're using the gcloud_bigtable package. We haven't really used it extensively up until now (but we plan on using and testing it a lot more in the next couple of weeks). The general design (and documentation) is quite clear and the only feedback that I can give right now is that unifying commit() and commit_modifications() seems like a good idea; I think it's a bit confusing to have 2 separate methods to basically do the same thing. I'll post more feedback in #9 when we use the library more.

@nathanielmanistaatgoogle Can you help review this. I'm not sure what the implications are, though I'd guess that @joerideturck has been using this code, so it almost certainly works as expected.

(The spirit of my three inline comments apply to all the functions changed in this pull request, not just the one function on which they were actually placed.)

@joerideturck Check out the latest version of HEAD. async is gone now and the library is upgraded to use the beta version of gRPC

repo: dhermes/gcloud-python-bigtable


text:
README refers to closed cargo PR

README says

Support for a local registry is being added in https://github.com/rust-lang/cargo/pull/2361 and will be able to redirect all Cargo downloads/requests to a registry stored locally.

That PR is closed, not merged.
The PR refers to https://github.com/rust-lang/cargo/issues/2111 which was resolved when https://github.com/rust-lang/cargo/pull/2811 was merged.
Digging through the PR a bit more I find https://github.com/rust-lang/cargo/pull/2857 , which was merged and it looks like that is the feature the README in this repo should be mentioning instead of PR 2361, and I guess the tense of that sentence needs to change, and maybe the content of the sentence can also be updated.

Maybe the sentence can be reframed around https://doc.rust-lang.org/cargo/reference/source-replacement.html

repo: dhovart/cargo-local-registry


text:
Modify the prevent_destroy toggle script for OSX

The -i flag doesn't work the same in the GNU and BSD implementations of sed, so I have to resort to some tomfoolery to work around that.
The BSD implementation of sed also doesn't allow the \s notation I was using for whitespace matching, so I modified the regex to use [[:space:]] instead.

GIve this a try, @dav3r.  Let me know if it works or not.

Totally awesome, dude! :surfing_man:


text:
feat(feed-item-row): DLT-1725 add noInitials to feed item row

feat(feed-item-row): DLT-1725 add noInitials to feed item row

Obligatory GIF (super important!)

:hammer_and_wrench: Type Of Change

These types will increment the version number on release:

[x] Feature

:book: Jira Ticket
https://dialpad.atlassian.net/browse/DLT-1725
:book: Description
Add noInitials prop into feed-item-row. Works the same way it does in leftbar contact row
:bulb: Context
Needed to have the leftbar and feed item row be in sync.
:pencil: Checklist

For all PRs:

[x] I have ensured no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).
[x] I have reviewed my changes.
[x] I have added all relevant documentation.
[x] I have considered the performance impact of my change.

For all Vue changes:

[x] I have made my changes in Vue 2 and Vue 3. Note: you may sync your changes from Vue 2 to Vue 3 (or vice versa) using the ./scripts/dialtone-vue-sync.sh script. Read docs here: Dialtone Vue Sync Script

:camera: Screenshots / GIFs

Added test.

repo: dialpad/dialtone


text:
docs: update typography docsite table

Moved from https://github.com/dialpad/dialtone/pull/971
Description
updating images → svg tokenized
new table with text styles
Pull Request Checklist

[ ] Ask the contributors if a similar effort is already in process or has been solved.
[ ] Review the contribution guidelines.
[ ] Use staging as your pull request's base branch. (All PRs using production as its base branch will be declined).
[ ] Ensure all gulp scripts successfully compile.
[ ] Update, remove, or extend all affected documentation.
[ ] Ensure no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).

Obligatory GIF (super important!)

Approving to not block.
Would be cool to have a design review here, this is a @fede-dp PR.

Approved to avoid blocking

Forgot to put this PR into draft, this is still not finished and @fede-dp is on vacations I think, so don't review this, please.

@fede-dp @yorbi-dp @francisrupert Ping to move forward or close this PR.

authors: iropolo juliodialpad


text:
json file from server not working...

I am trying to access music json file from my own server but its not working only assest file json working. Please give me brief details about this feature or fix this issue thank you!

Sorry but I didn't get you can you some send some more details. Could you please provide audio file server URL by which you are playing the song. For Audio Streaming we need a simple audio URL like 'http://storage.googleapis.com/automotive-media/Jazz_In_Paris.mp3' it does not depend on assets or server. I have used as assets for only demo purpose.

i want to say .... i will put json list of music on anywhere online and then access directly inside the application.
Means, when I update online file of json music list like new music or replace any link inside file online also update list of music in application. For now if i want too add any new music in json every time need to update app. So please add online json file support like other apps.
I hope you understand what I want too say........
Thanking you for quick replay!

Hello MohdShamim, have you been able to find a solution fetching your data from your own remote server?

authors: MohdShamim111 dibakarece zinobraskey

repo: dibakarece/DMAudioStreamer


text:
Add scope guard/defer

Add mechanism like go's defer

Closed by #31

repo: dice-group/dice-template-library


text:
Form checkmark icon not converted

When a PDF form with checkboxes checked is converted to an image, those checkmarks icons are missing.
I tried to combine disableFontFace and useSystemFonts options but without success. Any solution to this? Thanks!

When a PDF form with checkboxes checked is converted to an image, those checkmarks icons are missing. I tried to combine disableFontFace and useSystemFonts options but without success. Any solution to this? Thanks!

Could you share such file with me?

Sorry but I can't share the file itself for property reasons, however I can share some more details. Seem to be related to a missing standard font FoxitDingbats.
This is how the character looks like in the PDF:

Log:
Warning: fetchStandardFontData: failed to fetch file "FoxitDingbats.pfb" with "UnknownErrorException: The standard font "baseUrl" parameter must be specified, ensure that the "standardFontDataUrl" API parameter is provided.".
Whe I tried to set standardFontDataUrl following messages:
Warning: fetchStandardFontData: failed to fetch file "foxitDingbats.pfb" with "UnknownErrorException: Unable to load font data at https://unpkg.com/pdfjs-dist@2.14.305/standard_fonts/.
Warning: fetchStandardFontData: failed to fetch file "foxitDingbats.pfb" with "UnknownErrorException: Unable to load font data at ../node_modules/pdfjs-dist/standard_fonts/.

Please try v3.3.0

authors: dichovsky froston

repo: dichovsky/pdf-to-png-converter


text:
fix：处理由于依赖 bpmn/ids 在其 1.0.4 构建时会出现构建失败进而导致 logiflow 使用异常的问题

fix：处理由于依赖 bpmn/ids 在其 1.0.4 构建时会出现构建失败进而导致 logiflow 使用异常的问题（ https://github.com/bpmn-io/ids/issues/8 ）(#1228 )。
产生该问题的原因：logicflow  更新的时候遇到其内部引用的 bpmn-io/ids 依赖在由 ids 依赖 1.0.3 升级到 1.0.4时，ids 依赖构建失败，导致安装了 logiflow/extension 的工程启动失败。这是 bpmn-io/ids 1.0.4 构建失败的 bug#8 导致的（ https://github.com/bpmn-io/ids/issues/8 ）。
解决办法：logicflow/core 和 logicflow/extension 的 package.json 的 ids 的版本改为固定版本 1.0.3，有需要时再主动更新。同时更新 logicflow/extension 的 package-lock.json。

@ZhangChongLing 感谢PR，我看ids已经出1.0.5修复这个问题了，现在构建应该没有问题了。

@ZhangChongLing 感谢PR，我看ids已经出1.0.5修复这个问题了，现在构建应该没有问题了。后面我们考虑将ids在extension包中内置。

我package-lock.json安装的1.0.5版本 依然会报错误 #1243

@ZhangChongLing 感谢PR，我看ids已经出1.0.5修复这个问题了，现在构建应该没有问题了。后面我们考虑将ids在extension包中内置。

我package-lock.json安装的1.0.5版本 依然会报错误 #1243

先固定版本试试？这个问题后面会修复

@ZhangChongLing 感谢PR，我看ids已经出1.0.5修复这个问题了，现在构建应该没有问题了。后面我们考虑将ids在extension包中内置。

我package-lock.json安装的1.0.5版本 依然会报错误 #1243

先试试固定版本？这个问题后面会修复

请放心，您的邮件已收到。

authors: ZhangChongLing jasmine-song towersxu wumail

repo: didi/LogicFlow


text:
chameleon cml dev的时候 是否可以监听文件修改自动编译

chameleon cml dev的时候 是否可以监听文件修改自动编译

可以

repo: didi/chameleon


text:
对AWS SES的支持

What would you like to be added: 邮件配置中加入对AWS SES的支持，目前使用AWS的邮件认证用户名配置无法通过gomail的校验，使用邮箱又通不过认证。
Why is this needed: 公司的邮件服务使用的AWS，希望可以继续使用。

是smtp协议么

是smtp协议么

是的，smtp协议，但是认证的用户名是一串字母，配置到系统的时候无法通过gomail的邮箱格式校验

手工改gomail的代码，去掉格式验证代码重新编译试试

authors: FredCof UlricQin

repo: didi/nightingale


text:
[sdk] Add client support for get_account_by_version

We previously added support for getting an account at a historical version at the json-rpc server layer here: https://github.com/diem/diem/pull/7983
This PR finally adds support to the client. Here, we add a new method get_account_by_version for convenience, while the actual MethodRequest::GetAccount enum just gets an extra Option<u64> version parameter.

/land

repo: diem/diem