text:
not picking up method signatures that span multiple lines

This works:
  extractEmailsFromString(text, options = { isWrappedLessThanGreaterThan: false, }) {
    if (options.isWrappedLessThanGreaterThan) {
      return text.match(/(<[a-zA-Z0-9._-]+@[a-zA-Z0-9._-]+\.[a-zA-Z0-9_-]+>)/gi);
    }
    return text.match(/([a-zA-Z0-9._-]+@[a-zA-Z0-9._-]+\.[a-zA-Z0-9_-]+)/gi);
  }

But this doesn't work:
  extractEmailsFromString(text, options = {
    isWrappedLessThanGreaterThan: false,
  }) {
    if (options.isWrappedLessThanGreaterThan) {
      return text.match(/(<[a-zA-Z0-9._-]+@[a-zA-Z0-9._-]+\.[a-zA-Z0-9_-]+>)/gi);
    }
    return text.match(/([a-zA-Z0-9._-]+@[a-zA-Z0-9._-]+\.[a-zA-Z0-9_-]+)/gi);
  }

I know ctags works on a line by line basis, so is this just not possible to implement? Thanks for making this project!

Yeah, in cases where there is no function or : or = I need to be sure that this is a definition and not a call and for that I must find the ) {.
I don't think I've ever tried to use \n in this context but I can see this becoming very complicated very quickly. I will see what I can do.

authors: dyllandry romainl

repo: romainl/ctags-patterns-for-javascript


text:
Update dependencies

Upgrade closure-compiler and webpack and downgrade css-loader to 0.14.5 to improve performance

@jmandreslopez Thanks! Published as v2.1.4

source: gharchive/pull-request

license: mit

license_type: permissive

license_source: bigquery


text:
Prevent unicode character be rendered as emoji

Closes #720
Result:

Looks good! Thank you! I researched this a bit and this looks like the best solution. Note that the preview example is actually generated from real output, we'll probably need to automatically insert this for certain unicode characters in the HTML renderer. I can't imagine us needing to update it though for a while.

authors: paulomenezes sebmck

repo: romefrontend/rome

license: MIT

license_source: github-api


text:
能否在基本数据栏添加导入原数据功能

您的功能请求是否与问题有关？ 请描述一下。

在基本数据栏有导出原数据功能，因为一些原因，比如我从Google备份恢复后发现基本数据被清空，而我恰好不久前导出过数据，既然有导出，能不能添加导入功能，谢谢
描述你想要的解决方案

描述您考虑过的替代方案

其他附加信息

导入数据很难避免用户使用fake数据导入，这点值得在开发之前做详细探讨，特别是在目前插件截图在某种程度上是求药的必备。
毕竟目前导出的格式较为简单，且没有校验，直接修改重新导入即相当于对历史数据篡改。
虽然在某种程度上使用F12一样可以对输出结果进行fake，但是在没有导入原数据功能之前，插件本身记录的数据较难以修改。

导入数据很难避免用户使用fake数据导入，这点值得在开发之前做详细探讨，特别是在目前插件截图在某种程度上是求药的必备。
毕竟目前导出的格式较为简单，且没有校验，直接修改重新导入即相当于对历史数据篡改。
虽然在某种程度上使用F12一样可以对输出结果进行fake，但是在没有导入原数据功能之前，插件本身记录的数据较难以修改。

正因如此，才一直没做导入功能。如果真要做，导出的数据结构需要重新设计一下，得好好讨论讨论。

导入数据很难避免用户使用fake数据导入，这点值得在开发之前做详细探讨，特别是在目前插件截图在某种程度上是求药的必备。
毕竟目前导出的格式较为简单，且没有校验，直接修改重新导入即相当于对历史数据篡改。
虽然在某种程度上使用F12一样可以对输出结果进行fake，但是在没有导入原数据功能之前，插件本身记录的数据较难以修改。

正因如此，才一直没做导入功能。如果真要做，导出的数据结构需要重新设计一下，得好好讨论讨论。

好的，感谢解惑

source: gharchive/issue

authors: Rhilip XHLYDZ ronggang

repo: ronggang/PT-Plugin-Plus


text:
Support node@12

Fixes https://github.com/ronomon/base64/issues/1
Based on: https://github.com/bcoin-org/bcrypto/issues/7#issue-377078516
Please note, I don't have experience with node-gyp bindings, just changed that and tested locally - everything seems to be working. But if there's anything more required, please let me know.
Also fixed following warnings in node@10:

btw. maybe we could set up a CI build for testing compatibility with future node versions?

can we get this merged?

@ronomon Can you merge this please?

This package seems all but abandoned.

authors: AfzalivE HimbeersaftLP jakubwolny marin-liovic


text:
#007 Release 1st Stable Version - v0.0.2

[x] make required changes in setup.py (update version & development status - beta)
[x] push changes to github
[x] roll-out stable version & pass required comments
[x] test functionalities on google colab

New Stable Version Release: v0.0.2
First bug-free version v0.0.2
PyPI: https://pypi.org/project/chatInsights/0.0.2/
Full Changelog: https://github.com/ronylpatil/chatInsights/releases

repo: ronylpatil/chatInsights


text:
🛑 TROJAN 🇨🇦 Canada CAO 1 is down

In 520b218, TROJAN 🇨🇦 Canada CAO 1 (https://cat-1.opensvr.net/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TROJAN 🇨🇦 Canada CAO 1 is back up in de21753.

repo: roosterkid/opentunnel-status-server


text:
🛑 XRAY 🇸🇬 Singapore SGD 1 is down

In 0c644d9, XRAY 🇸🇬 Singapore SGD 1 (https://sgx-3.openv2ray.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: XRAY 🇸🇬 Singapore SGD 1 is back up in 39e74b7.


text:
🛑 SSH 🇮🇩 Indonesia IDA 2 is down

In 1a3ceee, SSH 🇮🇩 Indonesia IDA 2 (http://ids-4.opensvr.net:8080/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: SSH 🇮🇩 Indonesia IDA 2 is back up in db490ed.


text:
🛑 XRAY 🇨🇦 Canada CAO 1 is down

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

HTTP code: 0
Response time: 0 ms

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


text:
🛑 TROJAN 🇬🇧 United Kingdom UKO 2 is down

In 0a579ea, TROJAN 🇬🇧 United Kingdom UKO 2 (https://ukt-2.opensvr.net/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TROJAN 🇬🇧 United Kingdom UKO 2 is back up in 3f22dda.


text:
🛑 V2RAY 🇮🇩 Indonesia IDA 2 is down

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

HTTP code: 0
Response time: 0 ms

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


text:
🛑 SSH 🇮🇩 Indonesia IDA 1 is down

In 64149b4, SSH 🇮🇩 Indonesia IDA 1 (http://ids-2.optnl.com:8080/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: SSH 🇮🇩 Indonesia IDA 1 is back up in 3e40ffb.


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

In 29e8a27, TROJAN 🇺🇸 United States USF 1 (https://ust-1.optnl.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TROJAN 🇺🇸 United States USF 1 is back up in cdd065a.


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

In 3f288f8, PPTP 🇺🇸 United States USF 1 (http://usp-1.optnl.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: PPTP 🇺🇸 United States USF 1 is back up in 2a2c0c1.


text:
🛑 XRAY 🇫🇷 France FRF 1 is down

In f978b12, XRAY 🇫🇷 France FRF 1 (https://frx-1.openv2ray.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: XRAY 🇫🇷 France FRF 1 is back up in 8c19202.


text:
🛑 XRAY 🇩🇪 Germany DEH 1 is down

In c54aac7, XRAY 🇩🇪 Germany DEH 1 (https://dex-1.openv2ray.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: XRAY 🇩🇪 Germany DEH 1 is back up in cf7914a.


text:
🛑 TROJAN 🇸🇬 Singapore SGA 1 is down

In 3acc4df, TROJAN 🇸🇬 Singapore SGA 1 (https://sgt-3.opensvr.net/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TROJAN 🇸🇬 Singapore SGA 1 is back up in ea6e63d.


text:
🛑 TROJAN 🇸🇬 Singapore SGA 1 is down

In 8857610, TROJAN 🇸🇬 Singapore SGA 1 (https://sgt-3.opensvr.net/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TROJAN 🇸🇬 Singapore SGA 1 is back up in 79da024.


text:
🛑 TROJAN 🇮🇩 Indonesia IDJ 1 is down

In a53d531, TROJAN 🇮🇩 Indonesia IDJ 1 (https://idt-1.opensvr.net/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: TROJAN 🇮🇩 Indonesia IDJ 1 is back up in 2a5206f.


text:
🛑 SSH 🇸🇬 Singapore SGD 1 is down

In 12d2323, SSH 🇸🇬 Singapore SGD 1 (http://sgs-5.opensvr.net:8080/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: SSH 🇸🇬 Singapore SGD 1 is back up in 8ebed75.


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

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

HTTP code: 0
Response time: 0 ms

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


text:
SQL query cache + performance tuning

Self explanatory.

MariaDB already has the query cache on by default and it's set to 16MB which is a sane default (especially on a common 1GB server for example). I don't think there's really any common performance tuning to do I think we're good for default setups.

authors: nathanielks swalkinshaw

repo: roots/bedrock-ansible


text:
Bisq changed to tor onion v3

Bisq v1.7.4 will support v3 onions. Will be out in a week.
v2 onions will still work, but they getting generally more unreliable.

Tested it with Bisq release/v1.7.4 and it works.

Will be part of v1.7.1rc3

authors: Emzy rootzoll

repo: rootzoll/raspiblitz


text:
README.md references to IS8601 should be ISO8601

There are a few placed in README.md which refer to "IS8601". They should refer to "ISO8601", which matches the #define.

Fixed... See https://github.com/ropg/ezTime/commit/8e8a459a09fcced5486b16853d447182b53d4e07

repo: ropg/ezTime


text:
Fix Nosetests invocation in CI

More python3/python2 confusion.

Currently, I have no idea how to force travis to run flake8 for python3 and not ruin the test file for python2.
This currently just fixes the linter errors I get when I run it locally.

I think we'll just stick with python2 flake8 and handle the python3 flake8 problems (which seem to only reflect import orders) on a when-noticed basis.

Nevermind, turns out I was missing a plugin for flake8.

repo: ros-infrastructure/superflore


text:
Fix flake8 failure

Signed-off-by: Anas Abou Allaban allabana@amazon.com
Issue #, if available:. N/A
Description of changes: Fix flake8 failure.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The linter is now failing because of the known issue with ament_flake8 (ros-tooling/aws-oncall#118).


text:
find_object_2d: 0.6.2-0 in 'lunar/distribution.yaml' [bloom]

Increasing version of package(s) in repository find_object_2d to 0.6.2-0:

upstream repository: https://github.com/introlab/find-object.git
release repository: https://github.com/introlab/find_object_2d-release.git
distro file: lunar/distribution.yaml
bloom version: 0.6.1
previous version for package: null

@matlabbe FYI: If you release from a new branch you will need to bump the minor version number as only one version of the code can exist on the upstream repository for a given tag. Otherwise catkin_prepare_release will fail next time you prepare a release for both kinetic and lunar with the same version number.
(see ros_tutorials for an example of package releasing from a different branch every ROS distro and bumping the version number accordingly)

Thx for the warning. I use the same version of the standalone (no ROS) version of Find-Object to be easier to track which version is actually used in ROS (if I have issues opened on a particular version of Find-Object, standalone or ROS).
I do the same with RTAB-Map, the ros package uses the version of the standalone version to track the issues instead of the ROS distros.
I didn't have problems in the past with bloom doing so though, I'll see the next release if it is too problematic. If it fails, I'll know why!

authors: matlabbe mikaelarguedas

repo: ros/rosdistro


text:
mavros: 0.26.3-0 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository mavros to 0.26.3-0:

upstream repository: https://github.com/mavlink/mavros.git
release repository: https://github.com/mavlink/mavros-release.git
distro file: melodic/distribution.yaml
bloom version: 0.6.6
previous version for package: 0.26.2-0

libmavconn
* Prevent MAVConnTCPClient::do_recv and MAVConnTCPServer::do_accept from running after destruction has begun
* libmavconn/CMakeLists.txt: link mavconn-test against pthread
* Contributors: mlvov

mavros
* test: Fix sensor orientation. RPY 315 was removed in recent mavlink.
  https://github.com/mavlink/mavlink/commit/3d94bccfedc5fc7f2ffad247adecff0c2dc03501
* lib: update generated entries
* Contributors: Vladimir Ermakov

mavros_extras
* fixup! b43279058a3029c67ea75b1ecb86442c9dc991d4
* mavros_extras/log_transfer: Log transfer plugin
* Contributors: mlvov

mavros_msgs
* fixup! 5a4344a2dcedc157f93b620cebd2e0b273ec24be
* mavros_msgs: Add msg and srv files related to log transfer
* Contributors: mlvov

test_mavros

No changes

Forgot to say "Yes" when i do releases...


text:
marti_messages: 0.8.0-0 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository marti_messages to 0.8.0-0:

upstream repository: https://github.com/swri-robotics/marti_messages.git
release repository: https://github.com/swri-robotics-gbp/marti_messages-release.git
distro file: melodic/distribution.yaml
bloom version: 0.6.7
previous version for package: 0.7.0-0

marti_can_msgs

No changes

marti_common_msgs
* Merge pull request #96 <https://github.com/swri-robotics/marti_messages/issues/96> from matt-attack/add-psuedo-service
* Add ServiceHeader message
* Contributors: Matthew Bries, P. J. Reed

marti_nav_msgs

No changes

marti_perception_msgs

No changes

marti_sensor_msgs

No changes

marti_status_msgs

No changes

marti_visualization_msgs

No changes

Holding for sync

authors: pjreed tfoote


text:
geometry2: 0.6.5-0 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository geometry2 to 0.6.5-0:

upstream repository: https://github.com/ros/geometry_experimental.git
release repository: https://github.com/ros-gbp/geometry2-release.git
distro file: melodic/distribution.yaml
bloom version: 0.6.8
previous version for package: 0.6.4-0

geometry2

No changes

tf2

No changes

tf2_bullet

No changes

tf2_eigen

No changes

tf2_geometry_msgs
* Fix python3 import error
* Contributors: Timon Engelke

tf2_kdl

No changes

tf2_msgs

No changes

tf2_py

No changes

tf2_ros
* Protect the time reset logic from a race condition.
  Fixes #341 <https://github.com/ros/geometry2/issues/341>
  This could incorrectly trigger a buffer clear if two concurrent callbacks were invoked.
* Contributors: Tully Foote

tf2_sensor_msgs

No changes

tf2_tools

No changes

Holding for sync exception requested.


text:
aws-robomaker-small-warehouse-world: 1.0.1-1 in 'foxy/distribution.yaml' [bloom]

Increasing version of package(s) in repository aws-robomaker-small-warehouse-world to 1.0.1-1:

upstream repository: https://github.com/aws-robotics/aws-robomaker-small-warehouse-world.git
release repository: https://github.com/aws-gbp/aws_robomaker_small_warehouse_world-release.git
distro file: foxy/distribution.yaml
bloom version: 0.10.7
previous version for package: null

@cheng-kevin would you mind merging master into your branch? That should fix CI.

authors: cheng-kevin hidmic


text:
Revert "libcamera: 0.2.0-1 in 'humble/distribution.yaml' [bloom]"

Reverts ros/rosdistro#39494
I would like to revert this back to version 0.1 as it causes issues with the RHEL build and crashes with some Raspberry Pi cameras.

Closing and reopening to hopefully kick CI.

authors: christianrauch clalancette


text:
Lost documentation from the previous version of ROS2 for installing on debian

I'm installing ROS2 From Source on Rasbian. So far I have managed it through the pyenv installation, but I managed to find
this page that seems to be have been lost during revisions. Link Here
Despite it's age it has been extremely useful (with minor modifications with the export thing, which has been useful as I had a problem with sourcing keys.
Is there any particular reason it got deleted? I'm willing to contribute a guide if nessecary.

Is there any particular reason it got deleted? I'm willing to contribute a guide if nessecary.

It was renamed to http://docs.ros.org/en/humble/Installation/Ubuntu-Install-Debians.html

authors: avgsurfman clalancette


text:
Clarifications to the Windows Foxy Binary Install

This PR clarifies some issues that tripped me up during my install. Haven't built locally since the changes are small, but I can.

@binitshah friendly ping

Sorry, thanks for the ping. Will update tonight @clalancette

Built and visualized locally.

authors: binitshah clalancette


text:
Update README

Update README - Add missing info about installing pygments.rb and add other useful steps to run poroject

Hi,
I was plauing with your theme today. I had problem running it right away. The problem was that gem pygments.rb was missing.
Therefore I decided to write more in depth README.
I hope you will find that helpful, I do :)
Cheers!

Brilliant, thanks!

authors: pavelbinar rosario

repo: rosario/kasper


text:
عدم تشخیص ای ندا در نرمال سازی

عبارتی همانند ترسم که به کعبه نرسی ای اعرابی یا امثال آن را به کعبه‌ای تبدیل می‌کنند. حقیقتا اگر دستور زبان بهتر بلد بودم خودم حلش میکردم ولی راه حلی نیافتم.

پس شما هم راه حل دیگه ای پیدا نکردید. چون کاربر متاسفانه متوجه نیست. و باید خودم یک راه حلی پیدا کنم. ممنون جواب دادید. کلا راهی برای تشخیص تفاوت این دو ای با توجه به قواعد زبان فارسی وجود داره؟

با توجه به قواعد زبان فارسی سخت هست. به نظرم راه حلش دادن هر دو حالتش به یک مدل زبانی برای تشخیص نوع درست هست.

repo: roshan-research/hazm


text:
I'm Having trouble accessing the textbook answers (on chrome)

The extension doesn't seem to be working on the quizlet page, and it just stays as if the extension wasn't there. What should I do?

Turns out I just had to reset the site data, it's working fine now


text:
Support custom 404 previews

Hello, thanks for making this great Action.
I'm trying to test adding a new 404 page and wondered if I'm going about it the wrong way.
The docs for GitHub Pages say that you can have a custom 404 page by putting a 404.html in the root of the deployment.
If I create a 404 page and add it as part of a PR, I can see it by going to https://[owner].github.io/[repo]/pr-preview/pr-[number]/404.html but it does not function as a catch-all, for example when I go to https://[owner].github.io/[repo]/pr-preview/pr-[number]/this/page/does/not/exist I see the default GitHub Pages 404 instead of mine. Would it be possible to get this working?

I don't think so, no. 404s get redirected to 404.html at the root of your site.
If you were using e.g. an Apache web server, you could change the config to have a different 404 page per directory. GitHub Pages doesn't allow you that level of customisation.

authors: rossjrw xander-marjoram

repo: rossjrw/pr-preview-action


text:
Add a config file to allow the user to set defaults

This will mean the user doesn't have to enter command line arguments each time they run the program.
If command line arguments are given, they should override the values in the config file.
Format will most likely be YAML for readability.

Further research suggests TOML may be a better file format.

repo: rosswf/drawing-splitter


text:
Systemd

I'm having an issue running this with systemd. do you have an example you can share?

I haven't configured an instance of this with systemd, so unfortunately I can't help you at present. I will leave this issue open though, as it would be nice to have a working example. If you can update with specific details of your issue and any resolution, that may help someone!

Just set this up on a pi as a systemd service, here's my setup:
Download and Install
git clone https://github.com/rouyng/tncexporter
cd tncexporter
pip3 install -r requirements.txt

Create Systemd Service
sudo nano /etc/systemd/system/tncexporter.service

[Unit]
Description=TNC Exporter
After=network.target
StartLimitIntervalSec=0

[Service]
Type=simple
Restart=always
RestartSec=1
User=pi
WorkingDirectory=/home/pi/tncexporter
ExecStart=/usr/bin/env python3 -m tncexporter --latitude xx --longitude xx

[Install]
WantedBy=multi-user.target

NOTE 1: replace --latitude xx --longitude xx with your own values for distance metrics to work.
NOTE 2: set User and WorkingDirectory to your values. User is important for python deps to load.

Enable and Start Service
sudo systemctl enable tncexporter.service
sudo systemctl start tncexporter.service
sudo systemctl status tncexporter.service

Metrics should now be available at: http://127.0.0.1:9110/metrics

Just set this up on a pi as a systemd service, here's my setup:
Download and Install
git clone https://github.com/rouyng/tncexporter
cd tncexporter
pip3 install -r requirements.txt

Create Systemd Service
sudo nano /etc/systemd/system/tncexporter.service

[Unit]
Description=TNC Exporter
After=network.target
StartLimitIntervalSec=0

[Service]
Type=simple
Restart=always
RestartSec=1
User=pi
WorkingDirectory=/home/pi/tncexporter
ExecStart=/usr/bin/env python3 -m tncexporter --latitude xx --longitude xx

[Install]
WantedBy=multi-user.target

NOTE 1: replace --latitude xx --longitude xx with your own values for distance metrics to work.
NOTE 2: set User and WorkingDirectory to your values. User is important for python deps to load.

Enable and Start Service
sudo systemctl enable tncexporter.service
sudo systemctl start tncexporter.service
sudo systemctl status tncexporter.service

Metrics should now be available at: http://127.0.0.1:9110/metrics

Hi Liam,
Do you mind if I incorporate your instructions here into the project documentation?
Thanks
Ross

Sounds good to me 😃 thanks!

authors: liamcottle rouyng wolfman2g1

repo: rouyng/tncexporter


text:
Add persisting and point-evaluating contexts

Example usage of PersistingContext and EvaluatingContext. See also TestWorkflow and TaskRunnerEntrypoint in the PR.
PersistingContext persistingContext = new PersistingContext(
    basePath, TaskContext.inmem());
TaskContext context = MemoizingContext.composeWith(persistingContext);

// evaluate, this will persist all tasks
Value<?> value = context.evaluate(task);

// get all persisted file paths
Map<TaskId, Path> filePaths = persistingContext.getFiles();

EvaluatingContext evaluatingContext = new EvaluatingContext(
    basePath, MemoizingContext.composeWith(TaskContext.inmem()));

// point-evaluate specific task
Value<?> value = evaluatingContext.evaluateTaskFrom(persistedFilePath);

@fabriziodemaria @bergman PTAL

Codecov Report

Merging #24 into master will decrease coverage by 3.5%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##             master      #24      +/-   ##
============================================
- Coverage     84.19%   80.69%   -3.51%
- Complexity      121      136      +15
============================================
  Files            20       24       +4
  Lines           481      606     +125
  Branches         20       26       +6
============================================
+ Hits            405      489      +84
- Misses           71      108      +37
- Partials          5        9       +4

Impacted Files
Coverage Δ
Complexity Δ

...java/io/rouz/flo/freezer/TaskRunnerEntrypoint.java
0% <0%> (ø)
0 <0> (?)

...ain/java/io/rouz/flo/context/AwaitingConsumer.java
60% <0%> (-4.29%)
7 <0> (ø)

...main/java/io/rouz/flo/freezer/LoggingListener.java
68.75% <68.75%> (ø)
2 <2> (?)

...in/java/io/rouz/flo/freezer/PersistingContext.java
81.39% <81.39%> (ø)
9 <9> (?)

...in/java/io/rouz/flo/freezer/EvaluatingContext.java
82.6% <82.6%> (ø)
4 <4> (?)

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 f83337a...7afda05. Read the comment docs.

authors: codecov-io rouzwawi

repo: rouzwawi/flo


text:
Upgrade grpc and kotlin

grpc = 1.15.1
kotlin = 1.2.71
kotlinx-coroutines-core = 0.30.2

fixes #5

repo: rouzwawi/grpc-kotlin


text:
extra input fields not being rendered.

If you add extra hidden input fields they are not being rendered like so.
  <dropzone id="fileDropZone" url="/api/upload">
                        <input type="hidden" name="trackCategory" v-model="track.trackCategory">
                        <input type="hidden" name="Authorization" value="bearer:this.$auth.token('login-as')"
                    </dropzone>

is it committed to the repo yet?

Its committed to repo but not released on npm

Solved in latest release.

authors: kyoukhana vrajroham

repo: rowanwins/vue-dropzone


text:
Error: Request failed with status code 400

Timbot is starting.
Connecting to Discord...
[Discord] Logging in...
[Sellout] Sellout list initialized from remote, 946 items
[Discord] Bot is ready; logged in as Xesh Bot#7579.
[Discord]  -->  Member of server SAKNΞ's server, target channel is #stream-announcements
[Discord] Discovered 1 channels to announce to for stream-announcements.
[StreamActivity] Cleared current activity.
[TwitchMonitor] Configured stream status polling for channels: <some_twitch_channel_name>, <some_twitch_channel_name> (60000ms interval)
[Twitch]  ▪ ▪ ▪ ▪ ▪  Refreshing now (Initial refresh after start-up)  ▪ ▪ ▪ ▪ ▪
[TwitchApi] API request failed with Helix error: Invalid login names, emails or IDs in request (Bad Request/400)
[TwitchMonitor] Error in users refresh: Error: Request failed with status code 400
at createError (C:\Users\SAKNE\Desktop\bots\Xesh Bot\timbot\timbot\node_modules\axios\lib\core\createError.js:16:15)
at settle (C:\Users\SAKNE\Desktop\bots\Xesh Bot\timbot\timbot\node_modules\axios\lib\core\settle.js:17:12)
at IncomingMessage.handleStreamEnd (C:\Users\SAKNE\Desktop\bots\Xesh Bot\timbot\timbot\node_modules\axios\lib\adapters\http.js:236:11)
at IncomingMessage.emit (events.js:323:22)
at endReadableNT (_stream_readable.js:1204:12)
at processTicksAndRejections (internal/process/task_queues.js:84:21)

The important part is in the error from the Twitch API (Helix):
Invalid login names, emails or IDs in request

Most likely there is an invalid character in the stream names you're monitoring. If you share your config we can take a look, but in general: make sure you're using the channel names as they appear in the twitch.tv URL.

authors: roydejong sakne

repo: roydejong/timbot


text:
Example usage - how would a request to the example server look like?

Hi there,
I was trying to use this package in a server that I already have and use it to connect to a Azure PaaS SSAS offering. I was wondering how would a request look like for the example included in this project - https://github.com/rpbouman/xmla4js/blob/master/examples/example-xmla4js-nodejs-app.js? I understand that the server in the example is taking the incoming URL and building everything off of that. It would be great to see an example of what that request/incoming URL would look like or how should that be structured. Please advise.
Thanks,
Shani.

It has been a while so I maybe wrong but it would be something like:
GET http:///example-xmla4js-nodejs-app.js?url=<url of your XML/A service>
this would do a DISCOVER_DATASOURCES XML/A request and return the results (https://docs.microsoft.com/en-us/sql/analysis-services/schema-rowsets/xml/discover-datasources-rowset?view=sql-analysis-services-2017). Using the result of this response gives you the available catalogs, and possibly a new value for the url
GET http:///example-xmla4js-nodejs-app.js/<CATALOG_NAME>?url=<url of your XML/A service>
will do a DISCOVER request for MDSCHEMA_CUBES (https://docs.microsoft.com/en-us/sql/analysis-services/schema-rowsets/ole-db-olap/mdschema-cubes-rowset?view=sql-analysis-services-2017)
GET http:///example-xmla4js-nodejs-app.js/<CATALOG_NAME>/CUBE_NAME?url=<url of your XML/A service>
will do a DISCOVER request for the MDSCHEMA_DIMENSIONS in the specified cube (https://docs.microsoft.com/en-us/sql/analysis-services/schema-rowsets/ole-db-olap/mdschema-dimensions-rowset?view=sql-analysis-services-2017)
...and I think you'll get the picture now - adding more path elements will let you get hierarchies, levels, members and properties as well.
For executing MDX, you can do a request like this:
GET http:///example-xmla4js-nodejs-app.js/<CATALOG_NAME>?url=<url of your XML/A service>&mdx=
Valid content types for the response are:
"text/csv","text/plain" for CSV output,
"text/html", "application/xhtml+xml" for (X)HTML output.
"text/xml", "application/xml" for XML output (pass through XML received from the provider)
"text/json", "application/json" ,
"text/javascript": toJavaScript,
"application/javascript": toJavaScript
You can pass these either through the Accept request Header, or as the "format" parameter in the query, like so:
GET http:///example-xmla4js-nodejs-app.js?url=<url of your XML/A service>&format=
(Specifying format as query option will override the Accept header)

BTW nobody is actively updating the npm package. It was created at a time when someone convinced me node was all the shit and I should package for that. So I did but I never got any signals people were actually using it. So, please let me know if you are interested in an updated package for node, or otherwise, just grab the latest version and import it yourself in your node project.

Thanks for your updates Ronald. Much appreciated. I'll try the URL formats you suggested. I was planning on leveraging this package directly in another NodeJS server from where I will be calling the XMLA service exposed by Azure SSAS service. As such, the calls will not be URL based. I'll have to build the requests, set necessary headers (auth tokens at the very least) and then execute some DAX/MDX queries against the cubes.
If you do have any updated Node packages, please do share. I'm using the currently published package for now (version 0.0.2).
Thanks again for your response.
Shani.

@shani117, my suggestion is that you don't use the published node package, but grab either the Xmla.js from src or the compiled or minified one from the js directory. Xmla4js is arm compliant so it should not be any issue to require it in your node script.
The "URL-based calls" are only an illustration for the node js example. They are not indicative of any kind of real-world solution - it is merely a sample to illustrate use in node.
For the regular api, just checkout the doc/api docs, that's what they are there for. The browserbased samples also illustrate the use of the actual API.


text:
🛑 Maxx Techno Cast is down

In 4af4439, Maxx Techno Cast (http://maxxtechnocast.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Maxx Techno Cast is back up in 8de104b.

repo: rpharaniya/websites-uptime-monitor


text:
🛑 Amey Transport is down

In 8ae65e9, Amey Transport (https://ameytransport.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Amey Transport is back up in 76f13a6 after 3 minutes.


text:
🛑 Digital Global Hub is down

In 3b2a984, Digital Global Hub (https://digitalglhub.com/) was down:

HTTP code: 500
Response time: 219 ms

Resolved: Digital Global Hub is back up in aee3521 after 6 minutes.


text:
use prefixed ui-bootstrap directives

the new ui-bootstrap 0.14 uses the prefix uib- before all directives.

Yep you are correct.

Fixed in v1.7.0

authors: Jandalf rpocklin

repo: rpocklin/ui-router-tabs


text:
Aula de Linguagem Online

Aula de linguagem Markdown

Estou aprendendo linguagem Markdown no Curso em Vídeo de Git e GitHub
Podemos misturar configurações
Lista Numerada:

Teste
Teste2

Teste3
Teste4

Lista Demarcada

Teste
Teste

Teste

Lista de Tarefas:

[x] Criar página principal
[ ] Criar a página da loja
[ ] Receber pagamento

Acesse meu GitHub.

Tabela

Num
Nome
Nota

1
Gustavo
8,5

2
José
10

Comando
Não entendo o comando document.getElementById()

Olha meu progama em Python
num = int(input('Digite um valor: ')
if num % 2 == 0:
    print(f'O valor {num} é PAR')
else:
    print(f'O valor {num} é ÍMPAR')

repo: rposilva/OlaMundo


text:
Error in nearBaitAnalysis

Hi,
I did the near bait analysis and I get an error when the window size is between 3-5.  If I increase it to 10 it works.  The error is:
Error in if (as.character(windows[i, 1]) == chr & windows[i, 2] <= end &  :
missing value where TRUE/FALSE needed
Do you have any idea what it could possible be? Thanks.

Just realized that it does work when k=3 but it gives error with k=4 or 5.  I'm using a 4 cutter dpnii, do you know why this happens?


text:
Missing state type in a snippet in StateProvider documentation

StateProvider documentation states in a snippet in line 52:
final productsProvider = StateNotifierProvider<ProductsNotifier>((ref) => ProductsNotifier());
https://github.com/rrousselGit/river_pod/blob/master/packages/riverpod/lib/src/state_provider.dart
Is there a state type missing or intention was to use different provider?

The documents are not yet updated to version 1.0

Got that, I think this one is wrong anyway.

This is incorrect. The docs are up-to-date, and that snippet has a bug. I'll fix it

authors: Hamza-Imran75 rrousselGit swavkulinski


text:
Providers Doesnt Dispose After  Navigator.of(context).pushReplacement

I have multiple multiple AutoDisposeFutureProvider widget tree. After some specific network response i use Navigator.of(context) .pushReplacement for return the login page. Then i logged in again but  all FutureProviders states still alive, not disposed.
How can i archive dispose all used provider in widget tree with after use Navigator.pushReplacement. Am i doing something wrong?
Thanks for all your efforts on this awesome package after Provider

Hi there,
it would help if you post a reproducible example if you're submitting a bug issue.
Anyways, if you expect your Providers to be disposed (as you say: no one listens to them anymore), but they aren't, unexpectedly, then someone is still listening to those providers. The autoDispose clause will dispose a provider iff no one listens to them.
More debugging tools are on their way (by reading this): they should help you debugging your code quicker.

Here is the reproducible example
main.dart
import 'dart:convert';
import 'package:dio/dio.dart';
import 'package:dio_http_formatter/dio_http_formatter.dart';
import 'package:flutter/material.dart';
import 'package:flutter_riverpod/flutter_riverpod.dart';
import 'package:logger/logger.dart';

void main() {
  runApp(
    const ProviderScope(
      child: MyApp(),
    ),
  );
}

class MyApp extends ConsumerWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context, WidgetRef ref) {
    dio = ref.watch(networkManagerProvider).dio;

    return MaterialApp(
      title: 'Riverpod Autodispoe Bug Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Autodispoe Bug Demo'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key, required this.title}) : super(key: key);

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            ElevatedButton(
              onPressed: () {
                Navigator.of(context).push(MaterialPageRoute(builder: (context) => const TodoPage()));
              },
              child: const Text("Login"),
            )
          ],
        ),
      ),
    );
  }
}

class TodoPage extends ConsumerStatefulWidget {
  const TodoPage({Key? key}) : super(key: key);

  @override
  ConsumerState<ConsumerStatefulWidget> createState() => _TodoPageState();
}

class _TodoPageState extends ConsumerState<TodoPage> {
  @override
  Widget build(BuildContext context) {
    final todoListFuturePr = ref.watch(todosListFutureProvider);

    return Scaffold(
      appBar: AppBar(
        title: const Text("Todo List"),
      ),
      body: todoListFuturePr.when(
        data: (data) {
          return ListView.builder(
            itemBuilder: (context, index) {
              final item = data[index];
              return Padding(
                padding: const EdgeInsets.all(8.0),
                child: Card(
                  child: InkWell(
                    onTap: () {
                      Navigator.of(context)
                          .push(MaterialPageRoute(builder: (context) => ItemDetailsPage(item.id.toString())));
                    },
                    child: ListTile(
                      title: Text(item.title ?? '-'),
                      subtitle: Text(item.id?.toString() ?? '-'),
                    ),
                  ),
                ),
              );
            },
          );
        },
        error: (error, stackTrace) => Center(
          child: Text("$error"),
        ),
        loading: () => const Center(
          child: CircularProgressIndicator(),
        ),
      ),
    );
  }
}

class ItemDetailsPage extends ConsumerStatefulWidget {
  final String id;
  const ItemDetailsPage(this.id, {Key? key}) : super(key: key);

  @override
  ConsumerState<ConsumerStatefulWidget> createState() => _ItemDetailsPageState();
}

class _ItemDetailsPageState extends ConsumerState<ItemDetailsPage> {
  @override
  Widget build(BuildContext context) {
    final todoItemFuturePr = ref.watch(todosItemFutureProvider(widget.id));

    return Scaffold(
        appBar: AppBar(
          title: const Text("Item Details"),
        ),
        body: Center(
          child: todoItemFuturePr.when(
            data: (data) {
              return Column(
                mainAxisAlignment: MainAxisAlignment.center,
                children: [
                  OutlinedButton(
                      onPressed: () {
                        Navigator.of(context).push(MaterialPageRoute(
                          builder: (context) => const MyHomePage(title: "Login"),
                        ));
                      },
                      child: const Text("Go to LoginPage with pushreplacement")),
                  Padding(
                    padding: const EdgeInsets.all(8.0),
                    child: Card(
                      child: ListTile(
                        title: Text(data.title ?? '-'),
                        subtitle: Text(data.id?.toString() ?? '-'),
                      ),
                    ),
                  ),
                ],
              );
            },
            error: (error, stackTrace) => Center(
              child: Text("$error"),
            ),
            loading: () => const Center(
              child: CircularProgressIndicator(),
            ),
          ),
        ));
  }
}

final networkManagerProvider = Provider<NetworkManager>((ref) {
  return NetworkManager();
});

late Dio dio;

class NetworkManager {
  Dio dio = Dio();

  NetworkManager() {
    initDio();
  }

  initDio() {
    dio.options.sendTimeout = 8000; //8s
    dio.options.receiveTimeout = 8000; //8s
    dio.options.contentType = 'application/json';
    dio.interceptors.addAll([
      HttpFormatter(),
    ]);

    logger.i("Dio initialized");
  }
}

final todosListFutureProvider = FutureProvider.autoDispose<List<Todo>>((ref) async {
  try {
    final response = await dio.get("https://jsonplaceholder.typicode.com/todos",
        options: Options(responseType: ResponseType.plain));
    final todoList = todosListFromJson(response.data);
    ref.onDispose(() {
      logger.i("todosListFutureProvider disposed");
    });
    return todoList;
  } catch (e, s) {
    return Future.error(e, s);
  }
});

final todosItemFutureProvider = FutureProvider.autoDispose.family<Todo, String>((ref, id) async {
  try {
    final response = await dio.get("https://jsonplaceholder.typicode.com/todos/$id",
        options: Options(responseType: ResponseType.plain));
    final todoItem = Todo.fromJson(jsonDecode(response.data));
    ref.onDispose(() {
      logger.i("todosItemFutureProvider disposed");
    });
    return todoItem;
  } catch (e, s) {
    return Future.error(e, s);
  }
});

var logger = Logger();

List<Todo> todosListFromJson(String str) => List<Todo>.from(json.decode(str).map((e) => Todo.fromJson(e)));

class Todo {
  int? userId;
  int? id;
  String? title;
  bool? completed;

  Todo({this.userId, this.id, this.title, this.completed});

  Todo.fromJson(Map<String, dynamic> json) {
    userId = json['userId'];
    id = json['id'];
    title = json['title'];
    completed = json['completed'];
  }
}

pubspec.yaml
name: riverpod_dispose_bug
description: A new Flutter project.

publish_to: 'none'

version: 1.0.0+1

environment:
  sdk: ">=2.16.1 <3.0.0"

dependencies:
  flutter:
    sdk: flutter

  cupertino_icons: ^1.0.2
  dio: ^4.0.4
  flutter_riverpod: ^1.0.3
  dio_http_formatter: ^2.1.0
  logger: ^1.1.0

dev_dependencies:
  flutter_test:
    sdk: flutter
  flutter_lints: ^1.0.4

flutter:
  uses-material-design: true

Hi there,
As expected, I don't think Riverpod is bugged. Riverpod is behaving as expected.
On the "Go to LoginPage with Pushreplacement" button's onPressed event, you're just pushing a route.
This implies that:

The widget tree up until that point isn't disposed, since you're not using a pushReplacement, but instead just a `push: this causes a new route to be added on top of everything else, meaning you're not disposing anything, and therefore your tree is still listening to the providers you've created, and therefore they're not disposed as intended by Riverpod's documentation;
Even if you'd use pushReplacement, you're just replacing the last page you're viewing. You'd need to replace all and then push the login page. This means you'd need to Navigator.of(context).popUntil((route) => false) and then Navigator.of(context).push(... your route ...).

Keep in mind that these kind of questions usually belong to stackoverflow.
Also, and this is just my curiosity, why are you using dio = ref.watch(networkManagerProvider).dio, even though that variable is global? If you wanted to use dio as a Singleton, why not just initialize it outside?
Hope this helps (note: I'm just trying to be helpful, I'm not a maintainer or anything)

Closing as this is working as expected, as explained previously 😄
Hopefully the new devtools will help debugging this when they are available.

authors: lucavenir rrousselGit smurat

repo: rrousselGit/riverpod


text:
How to do not download image judge by the image's size?

New Issue Checklist

[ ] I have read and understood the CONTRIBUTING guide
[ ] I have read the Documentation
[ ] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
ios

Platform Version
e.g. 8.0

SDWebImage Version
4.0.0-beta2

Integration Method
e.g. carthage / cocoapods / manually

Xcode Version
e.g. Xcode 7.3

Repro rate
e.g. all the time (100%) / sometimes x% / only once

Repro with our demo prj
e.g. does it happen with our demo project?

Demo project link
e.g. link to a demo project that highlights the issue

Issue Description and Steps
Please fill in the detailed description of the issue (full output of any stack trace, compiler error, ...) and the steps to reproduce the issue.
If the image's size is very large, I use :
[_iconView sd_setImageWithURL:[NSURL URLWithString:model.lml_iconName] placeholderImage:img_placeholder_circle_header];

Sometims my app will crash.
So, how to judge the image's size if the size is in limit, I can setImageWithURL, if not I do not set the setImageWithURL. Because the method will download image, the image is too large, it can not download, or app will crash.

You can try this
[image sd_setImageWithURL:avatarPath  placeholderImage:nil options:SDWebImageScaleDownLargeImages];
Flag SDWebImageScaleDownLargeImages can help you.

repo: rs/SDWebImage


text:
Umbrella header warning for the FLAnimatedImage

New Issue Checklist

[x] I have read and understood the CONTRIBUTING guide
[x] I have read the Documentation
[x] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
iOS

Platform Version
10.0

SDWebImage Version
4.0.0

Integration Method
Carthage

Xcode Version
Xcode 8.2.1

Repro rate
all the time (100%)

Issue Description and Steps
Hello,
I am using version 4.0 of the framework. I have this Swift compiler warning:
/project_path/<module-includes>:1:1: Umbrella header for module 'SDWebImage' does not include header 'FLAnimatedImage.h'
Is FLAnimatedImage supposed to be added separately and how do I get rid of this warning?
thanks!

@ivan-konov please fill in the info required in the ISSUE TEMPLATE

There you go.

This is a useful read on the issue:
http://stackoverflow.com/questions/32923483/umbrella-header-for-module-myframework-does-not-include-header-otherframework
People who use the "raw" source code should be able to fix it themselves. I prefer to use Carthage to add frameworks to my projects so I would wait for it to be fixed in the repo :)

Hi guys,
It has been a while with this. What is the status of this issue?
Do you need help?
I can not use your library because of "Umbrella header for module 'SDWebImage' does not include header 'FLAnimatedImage.h''.
Thank you very much,
CALINOIU Laura

@ivan-konov @tualatrix (who created #1781) @lauracalinoiu this problem is tricky.
When installing via Carthage, indeed you get FLAnimatedImage as well. But with CocoaPods, you can just use the Core subspec which means no FLAnimatedImage.
Also, if you do install FLAnimatedImage as a dependency of SDWebImage via CocoaPods, this path <SDWebImage/FLAnimatedImage.h> will not work.
Not sure what to do here.
CC @mythodeia @skyline75489

Actually I think I found a way.
#if __has_include(<SDWebImage/FLAnimatedImage.h>)
#import <SDWebImage/FLAnimatedImage.h>
#endif

#if __has_include(<SDWebImage/FLAnimatedImageView.h>)
#import <SDWebImage/FLAnimatedImageView.h>
#endif

It will check if the inclusion is available.
Because the issue of direct inclusion was not so easy. It might have worked for Carthage alone, but for people installing via CocoaPods, it would not have worked.
And we cannot use the Project visibility for FLAnimatedImage.h and FLAnimatedImageView.h since people using Carthage might need to reference FLAnimatedImage entities directly. For WebP, since we hide all the implementation details, we don't need to make any of those headers Public so the umbrella warning does not apply there.

Nice, thanks a lot! :)

authors: bpoplauschi ivan-konov lauracalinoiu


text:
Weird characters in message list

Inside the message list, the message preview contains weird characters. But when I opened the full message, the message content rendered normally.

I'm using the latest version 0.1.5.

OS version?
App version?
OS language?
Does messenger.com render in the same way in Safari?

authors: rsms zulhfreelancer

repo: rsms/fb-mac-messenger


text:
display nested exception causes

fixes #2044

Duplicate of #2074 .  :(

Thanks for the effort, I am going to go with #2074 for now but we appreciate the though!

@rubysolo sorry I noticed your branch after doing all of the work :(

No worries!

authors: JonRowe mrageh rubysolo

repo: rspec/rspec-core


text:
Rename option for FailureList formatter to "failures"

The name was not matching case statement in formatters.rb and it is much
simplier to have an argument with that name.
Discussion:

https://github.com/rspec/rspec-core/pull/2624/files#r284776400

Thanks


text:
Use latest ruby version in ci

I updated some Ruby version.
And, I think Ruby2.4.1 spec should pass always.

Thanks, I'd already made all the pulls to the individual repos off my own but forgotten to push. The removed "failure" is just clean up, has no affect as we're running a different version of 2.4, so I won't repush those.

authors: JonRowe takkanm

repo: rspec/rspec-dev


text:
Add documentation to recently added ActionMailbox methods

Per request in https://github.com/rspec/rspec-rails/pull/2119#issuecomment-491050524

@benoittgt - looks like CI isn't running on these branches, and script/run_build is failing for me locally for reasons I haven't dug into yet. Please let me know if y'all need something more than or different from this.

Thanks a lot @jamesdabbs for the quick PR. 🤜 🤛

authors: benoittgt jamesdabbs

repo: rspec/rspec-rails


text:
Extracting EncodedString specs from #134

See https://github.com/rspec/rspec-support/pull/134
I need help with the matcher
Also, I decided not to include the new differ specs in here,
to reduce complexity, but am open to adding it.

I opened a PR against your branch that refactors the matcher and does a couple other small changes:
bf4/rspec-support#1

I opened a PR against your branch that refactors the matcher and does a couple other small changes:

oh fun. Will review

Didn't finish addressing comments tonight.  Merged and amended last commit in your PR.  Will finish and push code 'tomorrow' :zzz:

Didn't work on it today.  Probably won't again till Sunday.

That's fine :).

Rebased off of current master and force pushed
Add new commits to PR per discussion for easy review, if ok, would like to discuss which concepts in the pr deserve their own commit and how your PR into my PR should appear. (if you don't care, I'm comfortable just doing something).
Also, I looked into windows encodings more and think I'd like to document how to set the default external encoding that the suite expects on posix and windows, see https://github.com/rspec/rspec-support/pull/151#discussion_r22637355 and https://github.com/rspec/rspec-support/pull/151#discussion_r22991439.

@myronmarston This is ready for review, when you get the chance.  (Also see previous comment).

Merged.  Thanks, @bf4!

:rainbow: :fireworks: :horse: :100: yay!

repo: rspec/rspec-support


text:
failed to convert this.on function.

js:
this.on('complete', this.onSave, this);

cs:
@on_ "complete", @onSave, this

Dupe of #95, which is now fixed. Cheers!


text:
sparklyr fails to create temp table in _join(copy=TRUE) with dplyr and Spark 2.0.0

sparklyr fails to create temp table in _join(copy=TRUE) with dplyr and Spark 2.0.0.  Example with outcome in link.

Thank you, working as expected now, using latest saprklyr/dplyr and Spark 2.2:
> d1 <- copy_to(sc,data.frame(x=c(1,2),y=c('a','b')),'d1')
> d2 <- data.frame(y=c('a','b'),z=c(3,4))
> d1 %>% dplyr::inner_join(d2,by='y',copy=TRUE)
# Source:   lazy query [?? x 3]
# Database: spark_connection
      x     y     z
  <dbl> <chr> <dbl>
1     1     a     3
2     2     b     4

However, this is most likely fixed in sparklyr/dplyr and unrelated to Spark version.

authors: JohnMount javierluraschi

repo: rstudio/sparklyr


text:
Override canceling matrix job if only one of them fail.

[x] I have searched the issues of this repo and believe that this is not a duplicate.

Feature Request

By default, GitHub actions cancels all in-progress jobs in the matrix if any matrix job fails.

Docs: https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#jobsjob_idstrategyfail-fast

However, I'm not sure if GitHub actions will continue the next matrix job with versions that did pass in the previous one or the whole job will be canceled anyways.

repo: rszamszur/fastapi-mvc


text:
Contacts are not loading in IOS

Actual Behaviour
Data is not retrieving in ios and it is continuously showing spinner and in ios i am not getting the permission alert
expected behaviour
Permission alert should be called and later data should be displayed
React-Native Version:0.59.8
Code:
import React, { Component } from "react";
import SectionListContacts from "react-native-sectionlist-contacts";
import {
  View,
  Text,
  StyleSheet,
  ActivityIndicator,
  Platform,
  PermissionsAndroid
} from "react-native";
import Contacts from "react-native-contacts";
let selectedArray = [];
export default class SampleContact extends Component {
  constructor(props) {
    super(props);
    this.state = {
      getContact: [],
      spinner: true
    };
    this._renderHeader = this._renderHeader.bind(this);
    this.renderItem = this.renderItem.bind(this);
  }

  async componentWillMount() {
    if (Platform.OS === "android") {
      PermissionsAndroid.request(PermissionsAndroid.PERMISSIONS.READ_CONTACTS, {
        title: "Contacts",
        message: "This app would like to view your contacts."
      }).then(() => {
        this.loadContacts();
      });
    } else {
      this.loadContacts();
    }
  }

  _renderHeader = params => {
    console.log("---custom-renderHeader--", params);
    return (
      <View>
        <Text>{params.key}</Text>
      </View>
    );
  };

  renderItem = (item, index, section) => {
    console.log("---custom-renderItem--", item);

    return (
      <View>
        <Text>{item.name}</Text>
        {item.number.map((res, i) => {
          return <Text key={i}>{res.number}</Text>;
        })}
        {item.email.map((res, i) => {
          return <Text key={i}>{res.email}</Text>;
        })}
      </View>
    );
  };

  loadContacts() {
    Contacts.getAll((err, contacts) => {
      if (err === "denied") {
        console.warn("Permission to access contacts was denied");
      } else {
        console.log("contacts123" + JSON.stringify(contacts));
        contacts.map((res, i) => {
          selectedArray.push({
            id: res.recordID,
            name: res.givenName,
            number: res.phoneNumbers,
            email: res.emailAddresses
          });
        });
        console.log("selectedArray456" + JSON.stringify(selectedArray));
        this.setState(
          {
            getContact: selectedArray,
            spinner: false
            /////  page: 0
          },
          () => {
            console.log(
              "FinalSortList123",
              JSON.stringify(this.state.getContact)
            );
          }
        );
        console.log("spinner232435", this.state.spinner);
      }
    });
  }
  render() {
    //alert("hi");
    //console.log("1245", this.state.getContact);
    return (
      <View style={styles.container}>
        {this.state.spinner ? (
          <ActivityIndicator color="black" size="large" />
        ) : (
          <SectionListContacts
            ref={s => (this.sectionList = s)}
            sectionListData={this.state.getContact}
            initialNumToRender={this.state.getContact.length}
            showsVerticalScrollIndicator={false}
            SectionListClickCallback={(item, index) => {
              console.log("---SectionListClickCallback--:", item, index);
            }}
            renderHeader={this._renderHeader}
            renderItem={this.renderItem}
            otherAlphabet="#"
            showAlphabet={false}
          />
        )}
      </View>
    );
  }
}
const styles = StyleSheet.create({
  container: {
    flex: 1
  }
});

I'm also getting the same issue in React-Native Version:0.59.5

the project only works on latest react native. try updating your project to rn 60 or downgrade your rnc version


text:
Search isn't indexing the index page of nested pages in htmldir builder

You can test with this project
https://anymail.readthedocs.io/en/latest/search/?q=Sending+email&check_keywords=yes&area=default
There aren't results from this page https://anymail.readthedocs.io/en/latest/sending/
This is kind of related to the doctype that we want to remove https://github.com/rtfd/readthedocs.org/issues/4638 and kind of the same as https://github.com/rtfd/readthedocs.org/issues/5254
In a more minimal example this is more visible, our search returns 0 results and fallback to sphinx.
The explanation is that we try to index the file sending.fjson but that file doesn't exists, what we want is to index sending/index.fjson

Hrm, I thought we fixed this here: https://github.com/rtfd/readthedocs.org/blob/e4966e51a521a5cae44842aa87bb870f59b13b36/readthedocs/projects/models.py#L1149-L1159

We shouldn't use doctype anymore :).
I think basename is sending only, that's why.

How do you figure out if you should check for an HTMLDir path without doctype? I guess we could just try both, but that feels like it would lead to other bugs.

That's what I want I figure out now. If there isn't a clean way, we have the doctype from the config module in the database (old projects don't have this, but we could create a migration).

Seems like this should work?
In [6]: for p in HTMLFile.objects.filter(project__slug='anymail', version__slug='latest', path='sending/index.html'):
   ...:     print(p.json_file_path)
   ...:
readthedocs.projects.models:1154[6212]: INFO Adjusted json file path: sending/index -> sending
/home/docs/checkouts/readthedocs.org/media/json/anymail/latest/sending.fjson

Oh, the filename is /home/docs/checkouts/readthedocs.org/media/json/anymail/latest/sending/index.fjson -- that's the issue.

Seems like the logic is broken, and without it it would work 😢

Seems like this logic works for everything except the index file... There's some weird edge case here:
In [1]:
   ...: import os
   ...: for p in HTMLFile.objects.filter(project__slug='anymail', version__slug='latest', path__startswith='sending'):
   ...:     print(p.json_file_path)
   ...:     print(os.path.exists(p.json_file_path))
   ...:
   ...:
readthedocs.projects.models:1154[9789]: INFO Adjusted json file path: sending/templates/index -> sending/templates
/home/docs/checkouts/readthedocs.org/media/json/anymail/latest/sending/templates.fjson
True
readthedocs.projects.models:1154[9789]: INFO Adjusted json file path: sending/tracking/index -> sending/tracking
/home/docs/checkouts/readthedocs.org/media/json/anymail/latest/sending/tracking.fjson
True
readthedocs.projects.models:1154[9789]: INFO Adjusted json file path: sending/django_email/index -> sending/django_email
/home/docs/checkouts/readthedocs.org/media/json/anymail/latest/sending/django_email.fjson
True
readthedocs.projects.models:1154[9789]: INFO Adjusted json file path: sending/signals/index -> sending/signals
/home/docs/checkouts/readthedocs.org/media/json/anymail/latest/sending/signals.fjson
True
readthedocs.projects.models:1154[9789]: INFO Adjusted json file path: sending/exceptions/index -> sending/exceptions
/home/docs/checkouts/readthedocs.org/media/json/anymail/latest/sending/exceptions.fjson
True
readthedocs.projects.models:1154[9789]: INFO Adjusted json file path: sending/anymail_additions/index -> sending/anymail_additions
/home/docs/checkouts/readthedocs.org/media/json/anymail/latest/sending/anymail_additions.fjson
True
readthedocs.projects.models:1154[9789]: INFO Adjusted json file path: sending/index -> sending
/home/docs/checkouts/readthedocs.org/media/json/anymail/latest/sending.fjson
False

Yeah, index is the only one affected.

It's because sending.rst or sending/index.rst both lead to the same file :/

There isn't a sending.rst file, just sending/index.rst https://github.com/anymail/django-anymail/blob/master/docs/sending/index.rst

Right, but both files could lead to the same output file, so there's no way to know which it came from :/

authors: ericholscher stsewd


text:
Footnotes have excessive margins

I don't know if I'm doing something wrong with my markup, but footnotes have very large spacing compared to their size.
6. **remote:** Remove old key from ``~/.ssh/authorized_keys``.
7. **client:** Delete old key files.

.. [1] https://wiki.mozilla.org/Security/Guidelines/OpenSSH#OpenSSH_client
.. [2] https://wiki.mozilla.org/Security/Guidelines/OpenSSH#OpenSSH_server

Produces
  …

remote: Remove old key from ~/.ssh/authorized_keys.

client: Delete old key files.

  …

  …

Which under sphinx-rtd-theme looks like this:

Ran into the same issue today and could 'quick-fix' it by removing/(wrap comments around)
margin-bottom: 24px;.
of the css class
.wy-table-responsive
I'm pretty sure this is not the best way of handling this so it would be great if any of the devs could chime in and do a proper fix.

Yes, thanks. I'm not sure if other elements might be affected by this though (not that I saw any), as the developers probably have their reasons for this margin. I hope they give this a second look, as I understand too little of the theme to justify submitting a pull request.

I think the problem is that each footnote (same for citations) is turned into a table with a single row, instead of a table with multiple rows. Since this is done by docutils, it can not be easily changed from Sphinx or the html theme.
Each table is wrapped in a div to make it responsive, which has a non-zero margin-bottom, which is the cause of this spacing.
I have a possible workaround, will file a PR


text:
Fix mp.release()

In the reset method; i think you shouldn't release the mediaplayer object. based on the mediaplayer life graph, you can simply call mediaplayer.reset(); after that the state should allow to play next video.. the benefit of the current approach is that you don't have to release the mediaplayer and recreate it.. that should save some time in playing next video.. what do you think?

Fixed.

repo: rtoshiro/FullscreenVideoView


text:
URL is dummy but receving following error

MediaPlayer: setDataSource: IOException! uri=http://example.in/1494927784.mp4
java.io.FileNotFoundException: No content provider: http://example.in/1494927784.mp4
at android.content.ContentResolver.openTypedAssetFileDescriptor(ContentResolver.java:1137)
at android.content.ContentResolver.openAssetFileDescriptor(ContentResolver.java:975)
at android.content.ContentResolver.openAssetFileDescriptor(ContentResolver.java:902)
at android.media.MediaPlayer.setDataSource(MediaPlayer.java:992)
at android.media.MediaPlayer.setDataSource(MediaPlayer.java:966)
at com.github.rtoshiro.view.video.FullscreenVideoView.setVideoURI(FullscreenVideoView.java:814)
at com.petnod.activity.VideoviewActivity.videoPlay(VideoviewActivity.java:70)
at com.petnod.activity.VideoviewActivity.onCreate(VideoviewActivity.java:37)
at android.app.Activity.performCreate(Activity.java:6005)
at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1111)
at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2446)
at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2555)
at android.app.ActivityThread.access$800(ActivityThread.java:176)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1437)
at android.os.Handler.dispatchMessage(Handler.java:111)
at android.os.Looper.loop(Looper.java:194)
at android.app.ActivityThread.main(ActivityThread.java:5576)
at java.lang.reflect.Method.invoke(Native Method)
at java.lang.reflect.Method.invoke(Method.java:372)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:956)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:751)

Hi @sonaypsi
Does it work with native VideoView?

authors: rtoshiro sonaypsi


text:
Feature Request: 'Codequery-Build' cmd triggers (user-modifiable) script to build cq.db...

Instead of requiring the user to manually launch all the cmds that aggregate and assemble the cq.db database for CodeQuery to use, I propose the plugin install the following user-modifiable scripts:

cscope-build.sh
ctags-build.sh
cq-build.sh

cq-build.sh:
#!/bin/sh
cd ${CQ_WS_DIR}   # = <workspace>/.vscode/codequery
exec ${CQ_PLUGIN_BIN}/cscope-build.sh
exec ${CQ_PLUGIN_BIN}/ctags-build.sh
${CQ_PLUGIN_BIN}/cqmakedb -s ${CQ_WS_DIR}/cq.db -c ${CQ_WS_DIR}/cscope.out -t ${CQ_WS_DIR}/tags -p

cscope-build.sh:
#!/bin/sh
find ${WS} -iname "*.c"    > ${CQ_WS_DIR}/cscope.files
find ${WS} -iname "*.cpp" >> ${CQ_WS_DIR}/cscope.files
find ${WS} -iname "*.cxx" >> ${CQ_WS_DIR}/cscope.files
find ${WS} -iname "*.cc " >> ${CQ_WS_DIR}/cscope.files
find ${WS} -iname "*.h"   >> ${CQ_WS_DIR}/cscope.files
find ${WS} -iname "*.hpp" >> ${CQ_WS_DIR}/cscope.files
find ${WS} -iname "*.hxx" >> ${CQ_WS_DIR}/cscope.files
find ${WS} -iname "*.hh " >> ${CQ_WS_DIR}/cscope.files
cscope -cbf ${CQ_WS_DIR}/cscope_c.out

# Uncomment below if using pycscope
# find ${WS} -iname "*.py"    > ${CQ_WS_DIR}/pycscope.files
# pycscope -i ${CQ_WS_DIR}/pycscope.files -f ${CQ_WS_DIR}/cscope_py.out

# Uncomment below if using starscope
# find ${WS} -iname "*.rb"    > ${CQ_WS_DIR}/cscope.files
# find ${WS} -iname "*.go"    >> ${CQ_WS_DIR}/cscope.files
# find ${WS} -iname "*.js"    >> ${CQ_WS_DIR}/cscope.files
# starscope -e cscope,${CQ_WS_DIR}

ctags-build.sh:
#!/bin/sh
ctags --fields=+i -n -L ${CQ_WS_DIR}/cscope.files -f ${CQ_WS_DIR}/tags

Hopefully this is enough to give you a rough idea of what I mean.  The key is that the user can adjust locations, etc. as needed to integrate with other tools and systems they might be using, etc.

I need to think about this more carefully, because this extension supports multiple programming languages  not just C/C++, and also multiple OS platforms including Windows. So, we need many many scripts.

@gwiede Feature implemented in codequery4vscode 0.8.0. You've got to write your own rebuild script and put it in a standard place, and it can be called by the extension through the command palette.


text:
Rubicon-adapter: Integration questions

Hi,
Can you point me to the right documentation or the person to answer a question regarding rubicon adapter integration?
We got the following config for s2s adapter at client side
pbjsConfig.s2sConfig = {
    accountId: '1',
    bidders: ["rubicon"],
    enabled: true,
    timeout: Settings.prebidBidderTimeoutMS,
    adapter: 'prebidServer',
    endpoint: 'https://pbs.xxxxxxxx.com:8080/openrtb2/auction',
    cookieSet: false
};

The server forwards it correctly, but receives a http 400 from rubicon. I guess we need syncEndpoint or cookieSetUrl specified as well. What is the correct value for that?
Secondly, we get some errors back from Rubicon. Is this the right place to discuss that or should we go through the account manager route?

If anyone is blocked on similar issues, here are the relevant docs

https://github.com/rubicon-project/prebid-server-java/blob/master/docs/endpoints/cookieSync.md
https://github.com/rubicon-project/prebid-server-java/blob/master/docs/developers/cookie-syncs.md
https://github.com/rubicon-project/prebid-server-java/blob/master/docs/developers/gdpr.md

Closing the issue

repo: rubicon-project/prebid-server-java


text:
? repetition causes matched nodes to be returned as an array

https://nodepattern.herokuapp.com/?p=(
  def %24_method_name
  (args)
  (or-asgn %24(ivasgn _ ...) ...)%3F%0A)&ruby=def%20foo%0A%20%20%40bar%20%7C%7C%3D%20baz%0Aend

The pattern is reduced from what I'm trying to write. I'd like to use OR {} but because each branch will contain different amount of captures, I write multiple ? matchers after each other. * and + can return 0-n and 1-n matches so it totally makes sense to return an array for these. + on the other hand is ony 0-1 so I'd expect this to be either be the node or nil.
Can't change this now without a major version bump of course and the solution for me is to just call flatten somewhere in the code but I wanted to see what others think. Does my "complaint" make sense?

Does my "complaint" make sense?

It does.

Can't change this now without a major version bump of course

Well, there's a major version in the works... :-)

authors: Earlopain marcandre

repo: rubocop/rubocop-ast


text:
Edge case for StringReplacement: Use delete instead of gsub cop

Expected behavior
Using gsub instead of delete for integers expressed as unicode characters should trigger this offense
Actual behavior
No offense is registered
Steps to reproduce the problem
str.gsub(8203.chr('UTF-8'),'') # should register offense and suggest use of delete

RuboCop version
1.56.4 (using Parser 3.2.2.3, rubocop-ast 1.29.0, running on ruby 3.2.2) [x86_64-linux]
  - rubocop-capybara 2.18.0
  - rubocop-minitest 0.31.1
  - rubocop-performance 1.19.1
  - rubocop-rake 0.6.0
  - rubocop-sequel 0.3.4

There may also be unicode character substitution edge cases in the gsub vs. tr version of this cop too.

FWIW it is possible to match the two forms of integer coercion to an encoded character string using Integer#chr with a mere 103 regexps.
str = '55_295.chr(Encoding::UTF-8)'
str1 = "0.chr('ASCII-8BIT')"

int_to_char_encoding_regexps = Encoding.list.map { |enc| /\d{0,2}_?\d{1,3}\.chr\((Encoding::)?'?#{Regexp.quote(enc.to_s)}'?\)/ }

puts int_to_char_encoding_regexps.any? { |r| r.match? str } # => true
puts int_to_char_encoding_regexps.any? { |r| r.match? str1 } # => true

I'm sure there's a more elegant way.

repo: rubocop/rubocop-performance


text:
Cop Idea: DuplicateAssociation cop

Recently we had a few PRs created that defined the same association. They were both in code review at the same time, so both added the association that did not previously exist. When both got merged, we didn't realize that we now had the same association defined twice in 1 model. It would be nice if there were a cop that caught this and pointed it out.
For example:
belongs_to :foo
belongs_to :bar
belongs_to :blah
belongs_to :foo # <----- Duplicate

has_many :somethings

I expect this should also cover cases where the definitions refer to the same field but aren't identical, e.g.:
belongs_to :foo
belongs_to :foo, required: false

Wondering how it works. Maybe duplicate declarations complement each other, like e.g. new-style validations do?
belongs_to :user, -> { joins(:friends) }
belongs_to :user, foreign_key: :account_id, class_name: 'Account'
belongs_to :user, counter_cache: true
belongs_to :user, touch: :employees_last_updated_at

If not, why Rails doesn't warn of a duplicate declaration?
Maybe it's like that to be able to group common options?
belongs_to :user, foreign_key: :account_id, class_name: 'Account'
with_options counter_cache: true do
  belongs_to :user
  belongs_to :book
end

What if belongs_to is defined on the parent class, but one decided to duplicate this declaration for explicitness in the child class?
It's fine to tag identical declarations, though. Unless one of them is scoped inside with_options.

Yeah, multiple belongs_to :foo lines, regardless of the options hash or a with_options are a duplicate association. Since :foo is the name of the association, you can't define it twice. I'm not actually sure what Rails does if you try to. I know with scopes it logs a warning that you're redefining an existing scope. Do associations work the same way? I assume it either uses the first or last definition, but does not merge the definitions.
I could see this sort of thing actually just being a Rails PR, to error on association redefinition. But considering a scope redefinition is just a log line, they might not be up for an error.
There's certainly a lot of cases that a cop couldn't cover. It wouldn't be able to tell if you're redefining an association that's already defined in a parent class or a concern, for example. But it would help with for the same file at least.

Apparently, the last declaration wins. This is quite unfortunate due to the case of with_options and limits its application. However, this behaviour is understandable for the case with redefining the association in a child class.
Since the latter case is legitimate, including prepended/included concerns, it's also understandable  why Rails doesn't raise an error or does not warn.
Indeed, defining the same association twice in the same scope is a mistake, as the first declaration will be disregarded, and it's at best a redundant statement. Including the case when one of them is inside with_options.
Would you like to contribute a cop for it provided all necessary guidance?
I couldn't find a resembling cop that you could mimic. This one might be of some help to get an idea how to detect.
There are also duplicate helpers, not sure if they will come handy, though.
A node matcher that you could use is
ASSOCIATIONS = Set.new(%i[belongs_to has_one has_many has_and_belongs_to_many])
def_node_matcher :association?, <<~PATTERN
  (send nil? $#ASSOCIATIONS $_ ...)
PATTERN

Then in the cop:
def on_class(node)
  associations = node.children.map { |child| association?(node) }
  # reject [nil, nil] ones
  # => [[:has_many, :books], [:belongs_to, :city]]
  # now, find duplicates and add an offence:
  add_offense(node) do |corrector|
    # remove duplicates except for the last one
  end
end

The case with with_options might not be worth the effort.

@pirj I made a PR for this. #602

authors: andyw8 natematykiewicz pirj

repo: rubocop/rubocop-rails


text:
Run ActionCable in App

If you are using a server that supports the Rack socket hijacking API, Action Cable can run alongside your Rails application. For example, to listen for WebSocket requests on /websocket, specify that path to config.action_cable.mount_path

https://github.com/rails/rails/tree/v5.0.0.rc2/actioncable#in-app
看起来是可以直接和主应用跑在同一个实例里面的。

简化好多，学到了。

authors: greatghoul huacnlee

repo: ruby-china/ruby-china


text:
Fix detecting of commit message in Git::Lib::process_commit_log_data

Your checklist for this pull request
🚨Please review the guidelines for contributing to this repository.

[x] Ensure all commits include DCO sign-off.
[x] Ensure that your contributions pass unit testing.
[ ] Ensure that your contributions contain documentation if applicable.

Description
If the commit doesn't contain a message, parsing the output of "git log"
results in an error, which occures because the end of the commit message
can't be detected.
fixes #481

@rvodden @perlun @tarcinil, please can somebody review this pull request?
PS: build passed https://travis-ci.org/github/ruby-git/ruby-git/builds/716900598, I'm not sure why the status is not updating

I re-ran the build and the status was updated.  Not sure why it wasn't updating before.

@kpd200081 can you please add a failing test that this change fixes?
You should probable create your own test fixture instead of changing existing fixtures. If you don't have time to figure this out, I can help out -- just let me know.  Here is what I told someone else:

Reusing the default test fixture (test/files/working/dot_git) breaks tests that don't have anything to do with the change you are making. To me this is tech debt that is piling up with each change like this one. The next PR might have to correct all the same tests plus the new ones introduced in this PR. Additionally, I think merge conflicts will be more of a problem if everyone is trying to change the same test fixture for their tests.
You can see an example of using a different fixture in tests/units/test_diff_non_default_encoding.rb for using the fixture tests/files/encoding.  It would probably be good to create a new git working copy setup just for your test instead of copying an existing one that has a lot of setup for other things.

@jcouball, I'm not sure if a test is needed for this case.
Anyway, I'm not familiar with Ruby tests, so if you think it's necessary it would be nice if you could help with that.

@jcouball I added a test that falling without this fix, but I had to change the commit method a bit, take a look please
As we can see in travis it fail because run on published version of gem (I don't know why, locally I have same behavior)

@jcouball I apply suggestions to test name and opts type documentation, also I reply to other questions.

@jcouball should I rebase it?

@jcouball I fix commit author and rebase to remove merge commit, now  I think nothing block from accepting PR.

repo: ruby-git/ruby-git


text:
Given does not use proper nesting in declared(params)

When using given block inside a nested requires block the validation works properly. But when trying to access these required params they are not nested in the requires block. Following spec (spec/grape/validations/params_scope_spec.rb) illustrate the problem
 it 'includes the nested parameter within #declared(params)' do
      subject.params do
        requires :bar, type: Hash do
          optional :a
          given :a do
            requires :b
          end
        end
      end
      subject.get('/nested') { declared(params).to_json }

      get '/nested', bar: { a: true, b: 'yes' }
      expect(JSON.parse(last_response.body)).to eq('bar' => { 'a' => true, 'b' => 'yes'})
    end

This fails with following message:
expected: {"bar"=>{"a"=>true, "b"=>"yes"}}
     got: {"b"=>nil, "bar"=>{"a"=>"true"}}

(compared using ==)

Diff:
@@ -1,2 +1,3 @@
-"bar" => {"a"=>true, "b"=>"yes"},
+"b" => nil,
+"bar" => {"a"=>"true"},

I tried different options (repeating the requires block in the given block) but nothing seems to work. I also cannot find the root of the problem but I guess it has to do with the following MR: https://github.com/ruby-grape/grape/pull/1047

Add optional :b:
  requires :bar, type: Hash do
    optional :a
    optional :b
    given :a do
      requires :b
    end
  end

subject.get('/nested') { declared(params).to_json }
get '/nested', bar: { a: true, b: 'yes' }
=>  {"b":null,"bar":{"a":"true","b":"yes"}}

The declared method returns only declared params?
"b":null outside bar is caused by including_missing: true here:  https://github.com/ruby-grape/grape/blob/master/lib/grape/dsl/inside_route.rb#L28 not sure why.

@stjhimy Ha thanks that seems to solve the issue.
But the syntax is not really clear, why do I need the extra optional :b? Now I have defined :b two times.
I thought (and from the documentation) that I should only need the requires :b

I think that given :a requires :b in this case should automatically cause an optional :b. I wonder whether there's any case where we would not want that? If not this is a valid improvement / feature request.

I still think this is a bug and not really a feature request. Following example right from the specs
subject.params do
  optional :a
  given :a do
    requires :b
  end
end
subject.get('/test') { declared(params).to_json }

Why does this work out of the box without the optional :b where the nested example does require the optional: b to work? Clearly thats a bug to me and not a feature request.
I think that given :a requires :b implies that we ONLY require :b when a is provided, In this case I could use the given as a switch for my params. It's also possible that I only want to optional :b when a is provided. I don't want to allow optional :b as long as a is not provided!
If its implemented that given :a requires :b always implies an optional :b  then the following should be invalid: given :a optional :b since its implied already.
Anyway quoting from the docs:
Suppose some of your parameters are only relevant if another parameter is given; Grape allows you to express this relationship through the given method in your parameters block, like so:

This implies for me the behaviour given :a requires :b without the implicit optional :b

Ok I am with you wrt bug. Would love a fix.

authors: JanStevens dblock stjhimy

repo: ruby-grape/grape


text:
Infinite loop if the connection is lost

https://github.com/ruby-jp/websocket-client-simple/blob/17eadab39d586d879a4e7ba1572516154c742ff7/lib/websocket-client-simple/client.rb#L46-L51
When the TCPSocket is closed from the server side, getc always returns nil, so this code is an infinite loop. I am not sure what the intent of this code is, but it is definitely a bug and I am reporting it anyway.
Note that reading one letter at a time by using getc is not a good idea in terms of efficiency.

Thank you for report.

Note that reading one letter at a time by using getc is not a good idea in terms of efficiency.

We knew that. Reported at #11 .

authors: mame unasuke

repo: ruby-jp/websocket-client-simple


text:
Cannot use external metadata

Given a directory that includes both a data.csv and an alternate-data.csv, and given code that looks something like this:
require 'linkeddata'

RDF::Tabular::Reader.open("alternate-data.csv", metadata: "data.csv-metadata.json") do |reader|
  puts reader.inspect

  reader.each do |statement|
    puts statement.inspect
  end
end

and a data.csv-metadata.json that looks something like this:
{
  "@context": "http://www.w3.org/ns/csvw",
  "url": "data.csv",
  "tableSchema": {
    "primaryKey": "id",
    "columns": [
      {
        "titles": "id",
        "name": "id",
        "datatype": "string",
        "required": true
      }]
}

The code executes, but it processes data.csv, not alternate-data.csv.
If, in lib/rdf/tabular/reader.rb I comment out the the last half of line 135, so
 @options[:original_input] = @input #unless @options[:metadata]

instead of
 @options[:original_input] = @input unless @options[:metadata]

it functions correctly.
It looks like there's an edge case where that and line 183 don't work together in this case.  The tests pass either way, but it seems unlikely that it was just left in accidentally.

It should detect an error, as the metadata must reference the CSV it is used with. In this case, you referenced both the metadata and input, but they disagree.
I know it would be useful to be able to re-use metadata with different CSV files, but this is specifically disallowed by the spec (it was an original design goal, but proved to be problematic).
The last half of line 35 is when metadata is either specified, or has been detected and we restart using that metadata, so it is intentional.

Thank you for clarifying.  I'll work around it on my end, then—still figuring out how all this works.

The requirements of locating metadata, and in turn, having the metadata identify the CSV do may this more challenging than other RDF formats, unfortunately.

I know it would be useful to be able to re-use metadata with different CSV files, but this is specifically disallowed by the spec (it was an original design goal, but proved to be problematic).

There is a way to reuse the table schema definitions, namely by putting the schema in a separate file and referencing it for each of the tables. Your metadata would like these two files:
{
  "@context": "http://www.w3.org/ns/csvw",
  "tables": [
   {
    "url": "data.csv",
    "tableSchema": { "@id": "schema.json" }
   },
   {
    "url": "alternate-data.csv",
    "tableSchema": { "@id": "schema.json" }
   }
  ]
}

{
  "@id": "schema.json",
  "primaryKey": "id",
  "columns": [
      {
        "titles": "id",
        "name": "id",
        "datatype": "string",
        "required": true
      }
  ]
}

Thanks!  That's a good workaround.

authors: bencomp gkellogg workergnome

repo: ruby-rdf/rdf-tabular

license: unlicense


text:
Org home page: update step 2 copy

Please update the text for step 2 to say 'Complete questionnaire'

@fivefootbot All yours. Thanks!

repo: rubyforgood/homeward-tails


text:
README: Add note about DB fix for Apple Silicon

Description

My colleague @mgriffin encountered this problem, and solved it with the change mentioned. Here's fuller error output from him:
[IMPORTANT]
Don't forget to include the Crash Report log file under
DiagnosticReports directory in bug reports.

FATAL: prematurely zombied
[39268] - Worker 0 (PID: 39439) booted in 0.05s, phase: 0
/Users/mike/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/pg-1.5.6/lib/pg/connection.rb:695: [BUG] Segmentation fault at 0x0000000103ba8abc
ruby 3.2.2 (2023-03-30 revision e51014f9c0) [arm64-darwin23]

-- Crash Report log information --------------------------------------------
   See Crash Report log file in one of the following locations:
     * ~/Library/Logs/DiagnosticReports
     * /Library/Logs/DiagnosticReports
   for more details.
Don't forget to include the above Crash Report log file in bug reports.

-- Control frame information -----------------------------------------------

I've also very slightly tweaked a couple of other bits of the README to hopefully make it clearer.
Type of change

Documentation update

How Has This Been Tested?

Local setup

I tried this out on Mac and it was fine. I'm good with moving this to database.yml. Feel free to repurpose this PR to do that.

Ran into this issue today ended up  adding export PGGSSENCMODE="disable" to my .zshrc to fix this.

Thanks all. I've moved the change to database.yml in f6c1debbc2c695d0ffc4c998777241ab00d9a07d

authors: dorner sampart vsrnth

repo: rubyforgood/human-essentials


text:
Published build for 16.10.0.0-x86_64-darwin-20

Hello,
I have been having issues on my machine trying to use the latest mini_racer (0.6.2) which depends on the latest libv8-node. I am able to use an older version of mini_racer which uses libv8-node 15.14.0.1, and on my machine, a build specifically for darwin-20.
*** LOCAL GEMS ***

libv8-node (16.10.0.0 ruby x86_64-darwin, 15.14.0.1 x86_64-darwin-20)

I'm thinking if you were to publish a darwin-20 specific gem it may address the problems I am experiencing. Thanks!

for reference https://github.com/rubyjs/mini_racer/issues/185

Note to people watching these issues & PRs-- We are trying to release new versions of libv8-node and mini_racer and would love your help testing for stability in your products. Here's the main issue thread where we are tracking this:
https://github.com/rubyjs/mini_racer/issues/277
Please let us know the results of your tests in our branches so we can do the actual releases on those branches with confidence.

authors: scruff311 seanmakesgames tisba


text:
TypeApplications for shorter/nicer code

(no code change, but update example) The example has

 , constant "+"   ((+)  :: Int -> Int -> Int)

I think this looks much nicer
{-# language TypeApplications #-}
...
 , constant "+"   ((+)  @Int)

(code change) currently, we need a proxy argument

  , instances = [ reifyListable (0::N), ...

the "proxy" role could be made more clear by writing undefined::N, or by Proxy :: Proxy N, but I think this would be best:
  , instances = [ reifyListable @N, ...

This might require AllowAmbiguousTypes.

I think this looks much nicer
{-# language TypeApplications #-}
...
 , constant "+"   ((+)  @Int)

That's a good suggestion.  I have added a note on the README mentioning this.
In favour of backwards compatibility, I usually try to stay away from language extensions where possible when writing libraries.  Speculate does work on GHCs down to version 7.8 and I suspect it could be backported further to 7.6 and 7.4 with minor changes.  (cf. TypeApplications started exiting in GHC 7.8)

the "proxy" role could be made more clear by writing undefined::N, or by Proxy :: Proxy N, but I think this would be best:

The first option, undefined :: N is the one I use most frequently in the examples folder.  I'm aware of Proxy but ended up deciding to go for a more "old-school" solution.
Incidentally: I'm from the time before TypeApplications, I even created a LeanCheck module with a "DSL of sorts" for type binding.   The original example would become:
, constant "+" ((+) -:> int)

But of course, in today's times, it is better to use the more widely known TypeApplications pragma.  This is more of a curiosity now.

authors: jwaldmann rudymatela

repo: rudymatela/speculate


text:
[Bug]: ValueError: Tool with name functions not found, some queries will result in this error, while others won't.

Bug Description

I have modified according to this page, adding unique names to the metadata.
But the error still happend.

I have modified all the EnginTool's ToolMetadata
I also print the metadata for the retrieval step, ensuring that the names in the metadata are included.

I follow the these two Example, got same error.
Multi-Document Agents (V1)
Multi-Document Agents
Version
0.10.38
Steps to Reproduce
response = agent.query("my_query")
Relevant Logs/Tracbacks
No response

Hi, I have verified my tools names I am using in my queries match the names defined in the ToolMetadata.

But the error still happend.

I'm not sure what LLM you are using, but it's just hallucinating a tool name, which can happen

Hi @logan-markewich, I use AzuerOpenAI gpt3.5, so could this be an issue with the LLM API?

Hi @dosu , some queries won't result in this error, but some querirs will happen..
this won't happen

this happen