text:
Confusing behavior of ignore-mode "download"

If ignore-mode "download" is set, the common sense expectation is non-visible articles not to exist in local cache. However, when set after the articles have already been downloaded, they remain.
This is not really a bug, but it deserves some attention. Possible solutions:

The downloaded (and invisible) articles should be deleted to match the expectation of a clean local cache (perhaps after a warning and confirmation after enabling the ignore-mode)

An additional option to deal properly with such situation. For example: ignore-mode "delete" which would do the same which "download" does plus also clean up local cache from old articles matching the killfiles.

Tested in:
 emanruse commented Dec 15, 2020

newsboat 2.21.0 - https://newsboat.org/
Copyright (C) 2006-2015 Andreas Krennmair
Copyright (C) 2015-2020 Alexander Batischev
Copyright (C) 2006-2017 Newsbeuter contributors
Copyright (C) 2017-2020 Newsboat contributors

Newsboat is free software licensed under the MIT License. (Type `newsboat -vv' to see the full text.)
It bundles:
- JSON for Modern C++ library, licensed under the MIT License: https://github.com/nlohmann/json
- optional-lite library, licensed under the Boost Software License: https://github.com/martinmoene/optional-lite
- expected-lite library, licensed under the Boost Software License: https://github.com/martinmoene/expected-lite

newsboat 2.21.0
System: Linux 5.9.12-1-default (i686)
Compiler: g++ 10.2.1 20200825 [revision c0746a1beb1ba073c7981eb09f55b3d993b32e5c]
ncurses: ncurses 6.2.20201031 (compiled with 6.2)
libcurl: libcurl/7.73.0 OpenSSL/1.1.1h-fips zlib/1.2.11 libidn2/2.3.0 libpsl/0.21.1 (+libidn2/2.3.0) libssh/0.9.5/openssl/zlib nghttp2/1.41.0 (compiled with 7.73.0)
SQLite: 3.34.0 (compiled with 3.33.0)
libxml2: compiled with 2.9.10

You're right, this option could've been more intuitive. Out of the approaches you listed, I'd prefer the second one, because it's backwards-compatible and lets people choose between keeping old items and deleting them retroactively.
Actually, the more I think about it, the more I wonder why Newsboat even has ignore-mode "display". Isn't it the same as filtering the feed? I'll have to dive into the Git history to see how this feature got developed, before I can decide what to do with it going further.

I'd prefer the second one

Agreed.

Actually, the more I think about it, the more I wonder why Newsboat even has ignore-mode "display". Isn't it the same as filtering the feed?

Yes.

Okay, so ignore-mode "display" was added in Newsbeuter 2.2 via https://github.com/newsboat/newsboat/commit/1c8b9e440c438e0a51ee0657c72f2cb903a1cc39 to fix https://code.google.com/archive/p/newsbeuter/issues/52. The discussion in that issue draws a clear analogy with filters, and even suggests using pre-defined filters for that.
At the time, apparently no-one was inconvenienced by ignore-mode "download" not removing articles that are already in the cache.
I think it's fine to add ignore-mode "delete" (name subject to bikeshedding still), so I'm re-labelling this as an enhancement.

Sounds good.

authors: Minoru emanruse

repo: newsboat/newsboat


text:
Error when building with Unity 5.5

Hi,
first I'd like to let you know that this is fantastic!
However I'm having issues when Building with Unity 5.5. In editor.log, I get the following messagen:
Assets/3rd Party/Tools/Runtime Animation Recorder/UnityAnimSaver/UnityAnimationRecorder.cs(123,3): error CS0103: The name `AssetDatabase' does not exist in the current context
Thanks!

Hi,
Because the AssetDatabase is one of UnityEditor's function, so it can't be import in the build.
I updated a small fix, which allows the building process, but the function will not working in the build though.
I will try to find a replacement of AssetDatabase function in the future. But for now, you can't record into .anim in the build. If you really want to do the recording in the build, you can use the maya recorder though.

Awesome man, thank you so much. I wasn't sure this repo is still being maintained.
I don't need the function in the build, however would like to be able to build my project with you script imported.
Thanks!

haha you're welcome!

source: gharchive/issue

license: MIT

license_type: permissive

license_source: github-api


text:
Added identifier for package nodes

The nodes that includes packages will now have '(P)' appended in their filename.

@Rishav159 Thanks! @jdaguil your to review

@Rishav159 could there be another way may be with an icon for the node rather than adding extra characters to the node name?

@jdaguil what do you think?

@pombredanne @jdaguil I changed the color of the text-background to green instead of appending '(P)' in the filenames, to identify if the files contain package information.
Please review.

Hi @Rishav159 I'll review this soon. For any pull requests, we would want to merge them into the develop branch and not master.

@jdaguil Okay. I will make the PR again to the develop branch.

source: gharchive/pull-request

authors: Rishav159 jdaguil pombredanne

repo: nexB/aboutcode-manager

license_source: bigquery


text:
Improve RTD installation documentation

https://scancodeio.readthedocs.io/en/latest/installation.html
it states "PostgreSQL" is optional, but if I follow the instruction without installing the postgreSQL, I will get the following error:
~/scancode.io$ make postgres
-> Configure PostgreSQL database
-> Create database user 'scancodeio'
sudo -u postgres createuser --no-createrole --no-superuser --login --inherit --createdb scancodeio || true
sudo: unknown user: postgres
sudo: unable to initialize policy plugin
sudo -u postgres psql -c "alter user scancodeio with encrypted password 'scancodeio';" || true
sudo: unknown user: postgres
sudo: unable to initialize policy plugin
-> Drop 'scancodeio' database
sudo -u postgres dropdb scancodeio || true
sudo: unknown user: postgres
sudo: unable to initialize policy plugin
-> Create 'scancodeio' database
sudo -u postgres createdb --encoding=utf-8 --owner=scancodeio scancodeio
sudo: unknown user: postgres
sudo: unable to initialize policy plugin
Makefile:95: recipe for target 'postgres' failed
make: *** [postgres] Error 1

Although it's obvious make postgres will fail as no postgres is installed, I found it somehow confusing with the "optional" word in the installation. Perhaps we should make some notes about the requrirement to have postgres or sqlite installed for the desire database?

PostgreSQL installation is optional when using SQLite as a database. It is required for using PostgreSQL as the local database (but it may not be required neither if your database is located on another server for example).
Anyway, I agree with you, we should improve the documentation being more explicit to remove any confusions.

@hyounes4560 I would say that SQLite is a single user, single threaded database. Therefore it should only be used for small scale one-off testing IMHO. Any serious usage would need a PostgreSQL instance for production use.

authors: chinyeungli pombredanne tdruez


text:
Support ABOUT files for code that is patched or vendored

We should likely consider ABOUT files as "curations" for packages and they may likely override other scans/manifest data collection.
Here is a design:
Using ABOUT file for overries and curations
Say I have reviewed and curated the origin and license of this JAR
https://repo1.maven.org/maven2/log4j/log4j/1.2.13/log4j-1.2.13.jar
This is stored in an ABOUT file and license file this way, side by side:

apache-2.0.LICENSE
log4j.NOTICE
log4j-1.2.13.jar.ABOUT

The ABOUT file has this content:
about_resource: log4j-1.2.13.jar
name: log4j
version: 1.2.13
download_url: https://repo1.maven.org/maven2/log4j/log4j/1.2.13/log4j-1.2.13.jar
package_url: pkg:maven/log4j/log4j@1.2.13
license_expression: apache-2.0
notice_file: log4j.NOTICE
licenses:
  - key: apache-2.0
    name: Apache License 2.0
    file: apache-2.0.LICENSE

These could live anywhere in the codebase inputs, typically in the FROM in a d2d pipeline for a start, but they could be in the TO or added as an extra input.
I would like that:

the package with this ABOUT file is added to the project packages
any "log4j-1.2.13.jar" present in the "TO" side in a d2d is mapped to this ABOUT file and being for this package (including any extracts)
no more matching, mapping or scanning should take place and the involved should be marked with a ststus that tells they have been processed

the package with this ABOUT file is added to the project packages

In that example, about_resource: log4j-1.2.13.jar, the log4j-1.2.13.jar does not exists in the from/ side, right? And only the .LICENSE, . NOTICE , and .ABOUT files are expected to be available in from/?

This is correct. If it existed on the From/ side we can decide later what to do

any "log4j-1.2.13.jar" present in the "TO" side in a d2d is mapped to this ABOUT file and being for this package (including any extracts)

What value are we using to map, the about_resource extact filename?

For a start, yes we are mapping on he exact filename. We may extend this later if this is not good enough (may be using a path pattern)

the package with this ABOUT file is added to the project packages

Do we want to create the DiscoveredPackage even if no about_resource instances were found on the to/ side?

No we do not want to create the DiscoveredPackage in this case.

Implemented in https://github.com/nexB/scancode.io/pull/778/

authors: pombredanne tdruez


text:
[Step1] 지하철역 인수 테스트 작성

안녕하세요.
Step1 지하철역 인수 테스트 작성에 대한 코드 리뷰 요청드립니다.
RestAssured를 사용하는 부분이 코드가 길어져 가독성을 해친다고 생각해서 메소드를 분리해봤는데 이렇게 진행하는 것이 일반적인지 궁금합니다.
또, 메소드를 분리하다보니 createStation 메소드가 이름이 겹쳐 의도치 않은 오버로딩이 되었는데요.
메소드 이름을 바꾸자니 의도를 명확하게 보여주기보다는 헷갈릴 만한 소지가 생긴다고 생각해서 일단 그대로 두었습니다.
이런 경우에는 공통으로 사용되는 (여기에서는 제가 분리한) 메소드들을 클래스로 위치 자체를 분리하는 것이 더 좋을까요?
리뷰어님의 의견이 궁금합니다.
리뷰 미리 감사드립니다! 잘 부탁드려요 🙇🏻♀

RestAssured를 사용하는 부분이 코드가 길어져 가독성을 해친다고 생각해서 메소드를 분리해봤는데 이렇게 진행하는 것이 일반적인지 궁금합니다.

인수테스트 작성 방법에는 규칙은 따로 없습니다!
말씀해주신대로 가독성이 해친다고 생각하시면 메서드 분리를 잘해주셨다고 생각합니다 :)
인수테스트는 개발자가 아닌 도메인 이해관계자들이 모두 모여서 보았을때 무엇을 테스트하려고 했는지 알 수 있도록 작성 해주시면 좋을 것 같습니다. 그렇기에 해주신 것처럼 가독성과 재사용성을 우선으로 메서드 추출해서 검증해주시면 좋을 것 같아요!

또, 메소드를 분리하다보니 createStation 메소드가 이름이 겹쳐 의도치 않은 오버로딩이 되었는데요.
메소드 이름을 바꾸자니 의도를 명확하게 보여주기보다는 헷갈릴 만한 소지가 생긴다고 생각해서 일단 그대로 두었습니다.
이런 경우에는 공통으로 사용되는 (여기에서는 제가 분리한) 메소드들을 클래스로 위치 자체를 분리하는 것이 더 좋을까요?
리뷰어님의 의견이 궁금합니다.

사실 이 부분도 정답이 없는 부분이라 참고만 해주시면 감사하겠습니다 :)
테스트 케이스 메소드 이름과 given절을 위한 제공 데이터의 createStation 메서드가 오버로딩 된다는 뜻이군요!
이 부분은 크게 상관이 없다고 생각합니다 😄
만약 여러 인수테스트에서 공통적으로 사용이 된다면 테스트픽스쳐 같은 공통 테스트 클래스에 위치시켜도 좋다고 생각합니다 :)

authors: ohtaeg ychxexn

repo: next-step/atdd-subway-admin


text:
🚀 1단계 - 경로 조회 타입 추가

안녕하세요, 이슬님. 1단계 미션 진행완료하여 리뷰 요청드립니다.
[개발 사항]

[x] 경로 조회 시 최소 시간 기준으로 조회할 수 있도록 기능을 추가하세요.
[x] 노선추가 & 구간 추가 시 거리와 함께 소요시간 정보도 추가하세요.

[x] 노선 등록 시, 소요 시간 정보를 보내도록 수정
[x] 구간 등록 시, 소요 시간 정보를 보내도록 수정

[x] 역과 역 사이에 구간을 등록할 시, 기존 구간의 소요 시간도 함께 변경해야함.
ex) 강남역 - 선릉역 (소요시간 8분), 새로운 구간 강남역 - 역삼역 (소요시간 3분) -> 강남역 - 역삼역 (소요시간 3분), 역삼역 - 선릉역 (소요시간 5분

이제 교육 기간이 마무리되어 리뷰어 분들께서 의무적으로 미션에 대한 리뷰를 안해주셔도 되는 것으로 알고 있는데, 해당 주차 미션까지 완주하고 싶어 추가 리뷰 요청은 boorwonie(류성현) 님께 여쭤보면 되는 것인지 질문드립니다!

피드백주신 내용 반영하여 수정 후, 재리뷰 요청드립니다!

repo: next-step/atdd-subway-fare


text:
Step0 수행해봤습니다

안녕하세요 리뷰어님 :) 스탭0 진행해보았습니다.
과제 진행하면서 mock을 이용하여 가짜 협력객체를 만들어 stubbing 할 때, 어떤 메소드를 stub할지 계속 구현해놓으신 코드를 보게 되어것 같아서 찝찝한 마음이 있었습니다.  이게 아무래도 이미 구현해놓으신 샘플코드가 있어서 더 그랬던 것일까요....?
아니면 원래 stub을 하려면 세부 구현에 의존할 수 밖에 없는 것인가요? 😭

stub을 하려면 세부 구현을 알아야만 할 수 있습니다, 지금 고민하시는 부분은 stub을 통해 편하게 테스트 해볼 수 있는 점을 경험해보며 찝찝함 없이 하셔도 될 것 같습니다! :)

authors: heesunjeong junwoochoi

repo: next-step/atdd-subway-path


text:
[그럴듯한 인프라 만들기] STEP2 : 배포하기

java 경험이 없어서 조금 헤맸네요!
다음 주에는 step3 해보도록 노력하겠습니다 :D

말씀해주신 내용 해결했습니다 :D 감사합니다!

repo: next-step/infra-subway-deploy


text:
Bug: junit.xml not generated

Description of the issue
Description:
I have a multi-module project using workspaces. In the workspace cargo.toml I have a [profile.dev.junit] block giving a path for junit.xml. When I run cargo nextest run, it is successfully running the tests for the sub-module containing some tests, but it is not writing a junit.xml cargo file anywhere. It is warning me that the manifest key is not used. This may be my user error, puting the config in the wrong place?
A representative output is:
$ cargo nextest run
    Blocking waiting for file lock on package cache
    Blocking waiting for file lock on package cache
warning: /home/nmrp3/devel/turingatemyhamster/ssnet/Cargo.toml: unused manifest key: profile.dev.junit
warning: function `resolve_to_download_url` is never used
   --> src/ssnet_lib/src/biogrid_cache.rs:289:10
    |
289 | async fn resolve_to_download_url(client: &reqwest::Client, url: &str, resource: &str) -> Result<Option<Str...
    |          ^^^^^^^^^^^^^^^^^^^^^^^
    |
    = note: `#[warn(dead_code)]` on by default

warning: `ssnet_lib` (lib) generated 1 warning
warning: `ssnet_lib` (lib test) generated 1 warning (1 duplicate)
warning: unused variable: `latest_remote_release`
  --> src/biogrid_cache/src/main.rs:33:21
   |
33 |         if let Some(latest_remote_release) = cache.remote().await?.version(&latest_remote_version) {
   |                     ^^^^^^^^^^^^^^^^^^^^^ help: if this is intentional, prefix it with an underscore: `_latest_remote_release`
   |
   = note: `#[warn(unused_variables)]` on by default

warning: `biogrid_cache` (bin "biogrid_cache" test) generated 1 warning
    Finished `test` profile [unoptimized + debuginfo] target(s) in 0.09s
------------
 Nextest run ID efb5be7a-b625-413a-bef9-8c65533f2970 with nextest profile: default
    Starting 10 tests across 2 binaries
        PASS [   0.004s] ssnet_lib biogrid_cache::tests::version_eq
        PASS [   0.004s] ssnet_lib biogrid_cache::tests::version_from_str_works
        PASS [   0.004s] ssnet_lib biogrid_cache::tests::version_from_str_not_numeric
        PASS [   0.005s] ssnet_lib biogrid_cache::tests::version_ord_major
        PASS [   0.005s] ssnet_lib biogrid_cache::tests::version_ord_point
        PASS [   0.005s] ssnet_lib biogrid_cache::tests::version_ord_minor
        PASS [   0.005s] ssnet_lib tests::it_works
        PASS [   0.007s] ssnet_lib biogrid_cache::tests::version_from_str_wrong_number_of_components_fails
        PASS [   0.008s] ssnet_lib biogrid_cache::tests::parse_version_handles_single_digit_versions
        PASS [   0.009s] ssnet_lib biogrid_cache::tests::parse_version_handles_multi_digit_versions
------------
     Summary [   0.009s] 10 tests run: 10 passed, 0 skipped

My workspace is:
[workspace]
members = [
    "src/ssnet_lib",
    "src/biogrid_cache",
    # Add other binaries here
]

resolver = "2"

[workspace.dependencies]
ssnet_lib = { path = "src/ssnet_lib" }
reqwest = { version = "0.12.7", features = ["json"] }
tokio = { version = "1", features = ["full"] }
indicatif = "0.17"
log = "0.4"
env_logger = "0.11.5"
clap = { version = "4", features = ["derive"] }
colored = "2"
serde = { version = "1.0", features = ["derive"] }
serde_json = "1.0.128"
select = "0.6.0"
regex = "1.10.6"

[profile.dev.junit]
path = "junit.xml"
store-success-output = false
store-failure-output = true

The cargo for the library with tests is:
name = "ssnet_lib"
version = "0.1.0"
edition = "2021"

[dependencies]
reqwest = { workspace = true }
tokio = { workspace = true }
log = { workspace = true }
indicatif = { workspace = true }
serde = { workspace = true }
select = { workspace = true }
regex = { workspace = true }

Expected outcome
I expected a junit.xml file to be generated under the target directory.
Actual result
It was not generated.
Nextest version
cargo-nextest 0.9.79 (ec7861ab7 2024-10-02)
release: 0.9.79
commit-hash: ec7861ab7c27a63f036eca5fb3bbe67bbe1c0dd1
commit-date: 2024-10-02
host: x86_64-unknown-linux-gnu

Additional context
No response

Hi, thanks for filing the report! Nextest configuration goes under .config/nextest.toml — see https://nexte.st/docs/configuration/.

Thank you. I was looking at https://nexte.st/docs/machine-readable/junit/ and I had not spotted that the link to the nextest configuration was outside of the main cargo toml config. It's working as expected for me now. Perhaps in the "example configuration" on that junit page, use the code fence feature where it lets you name the file, and make it /.config/nextest.toml for clarity?

Agreed -- done!

authors: drdozer sunshowers

repo: nextest-rs/nextest


text:
🛑 /getMusicFeed - Get music feed by URL or ID is down

In 7154768, /getMusicFeed - Get music feed by URL or ID (https://tiktok.nextpost.tech/getMusicFeed?query=7129970471625362182&cache_timeout=0&license_key=$API_KEY) was down:

HTTP code: 400
Response time: 5461 ms

Resolved: /getMusicFeed - Get music feed by URL or ID is back up in 0fe11ac after 5 minutes.

repo: nextpost-tech/tiktok-status


text:
XIOS: Private methods for converting between strings and durations

We already have private methods for converting between strings and dates:
std::string convertXiosDatetimeToString(const cxios_date datetime, const bool isoFormat = true);
cxios_date convertStringToXiosDatetime(const std::string datetime, const bool isoFormat = true);

We should implement methods for converting between strings and durations, too.

Closed by #637.

repo: nextsimhub/nextsimdg


text:
fix server deletion api, add issue templates

Fix https://github.com/nezhahq/nezha/issues/489

用户前端导流到 nezha-dash

用户前端导流到 nezha-dash

已经改了 看看

authors: naiba uubulb

repo: nezhahq/nezha


text:
Update for Dockerfile

This addresses issue #17 , and should provide a cleaner conda environment.

Aha cool - so the entire Dockerfile needed replacing! Sorry, I thought that #17 was just about adding the one conda clean command..

Ahhhh, wait - I did already do this in #12, I'm not going crazy 😛
I think your fork must have been out of date..

Ok, changed / merged things. Sorry, should've updated my fork beforehand -.-

No worries!

:rofl:

authors: apeltzer ewels sven1103

repo: nf-core/cookiecutter


text:
Running on Windows

Hello!
How can I start on bootup in Windows?
How start, stop, restart and control status?
Joachim

@Joey-1970: You can create a start skript over a bacht file. I have write a tool to start Homebridge "invisible" on windows.
What do you mean by: How to start, stop, restart and control status?
Greeting
Patcher

Hello Patcher,
thanks for your reply. May you show me your startscript?
I want to get regularly control the state (running/not running) of the homebridge from another application (like 'sudo /etc/init.d/homebridge status' on Raspberry Pi).
And - if this possible - check the log or error messages (like 'tail -f /var/log/homebridge.log' on Raspberry Pi).
Joachim

Here ist my batch (start.bat) file:
echo off
cd C:\Users\xxx\AppData\Roaming\npm\node_modules\homebridge\bin
cls
node homebridge
You are from Germany?
Greeting
Patcher

Hello Patcher,
thanks for your script, it is very helpful. I try to install it on my system.
Yes, I am from Germany. Why?
Joachim

Hello Joachim,
i come from Germany and thy name is typically German ;)
If you want, I can tell you later yet another way of booting Homebridge?!
Greeting
Patcher

Hello Patcher,
this would be very nice!
Joachim

Hello Joachim,
sorry, today I had no time. I think I have time tomorrow night.
Greeting
Patcher

Hello Patcher,
maybe this Link will help us to take (a little) control of the homebridge?
https://www.npmjs.com/package/node-windows
Joachim

Hello Joachim,
sorry for my late answer. I have write a new version of my program to start homebridge on Windows.
You can download it from the Annex.
I hope it match your requirements?!
Greeting
Patcher
shb.zip

Many, many thanks!!!
Unfortunely I haven't the time to test it. A question before I do it: Can I start and stop up over command line too? I need to control from another application... ;-)
Joachim

Hello Joachim,
Currently you can not. What do you imagine that? About TCP or Shell?
Greeting
Patcher

Hello Patcher,
I think that is equal for me. The Homebridge runs on the same WHS 2011 as my smart-home-application (IP-Symcon).
So I think I can go both ways.
Joachim

Hello Joachim,
now i have make a new version with TCP-control and Tray-control. You can download it from the Annex. Please read the "_Readme.txt" for details.
I hope it match your requirements?!
Greeting
Patcher
Download: shb.zip

Hello Patcher,
many thanks for your commitment!!!!! I will test  it tomorow... ;-))
There is a interesting discussion on issue #420 ...
Joachim

...I could't wait until tomorow - look very nice!!!
Joachim

Hello Joachim,
I think you can now close this issues - or?
Greeting
Patcher

Hello Patcher,
yes, thank you very much for support!
Joachim

repo: nfarina/homebridge


text:
Are the priors in the gamma_model.stan truncated?

In https://github.com/nfidd/nfidd/blob/459d1bfcc5424a1e36f193ce7a575fc8d7ee48cd/sessions/R-Stan-and-statistical-concepts.qmd#L154, you state that the priors are truncated. I thought they would be if they were defined using T[] like here https://github.com/nfidd/nfidd/blob/459d1bfcc5424a1e36f193ce7a575fc8d7ee48cd/stan/lognormal.stan#L14.

You're right! The truncation should be added to alpha and beta

authors: jamesmbaazam sbfnk

repo: nfidd/nfidd


text:
Release/5.3.0

Add new functionality to inject custom tags string in the head through HelmetsOpenedVisor element.
This closes #323 .
I've rebased it here instead of https://github.com/nfl/react-helmet/pull/496 .

Looks like something wrong with test/build environment. I've passed all local tests successfully. @KraigWalker  @cwelch5 or somebody else could you please take a look, I have no idea how to fix it.

Made some mistakes when exporting Helmet. So, close this and open another PR.

repo: nfl/react-helmet

license: mit


text:
XCP VM tools

VM tools are not getting installed for XCP/XenServer nodes.

CLOSING: These are being installed but XenCenter seems to report that they aren't installed for more advanced CPUs with better virtualization capabilities.

Have yet another look at this.

Required XenServerImageHost changes:

[ ] There's no need to uninstall hv-kvp-daemon.service.  This is no longer installed.
[ ] We're mounting the guest-tools ISO from /dev/dvd; this needs to be /dev/cdrom

FIXED

authors: jefflill marcusbooyah

repo: nforgeio/neonKUBE


text:
WIP: fix antimeridian queries via second r-tree

Add a second r-tree indexed with extents in the range [0, 360] and use that r-tree when the extent query crosses the antimeridian.
resolves #468

This is marked as WIP because it should not be merged until it is compared with the other solution PR.

Closing because we are going with #470 instead.

repo: ngageoint/opensphere


text:
Specify supported njs version.

https://unit.nginx.org/scripting/
Warning

Unit 1.31+ doesn’t support pre-0.8 njs [versions](https://nginx.org/en/docs/njs/changes.html); please update.

Because of the change of njs 0.8.2, Unit doesn't support this version, it looks like we need to specify which versions we support.

@hongzhidao I'd be happy to help out with this issue.
Would updating Unit 1.31+ doesn’t support pre-0.8 njs -> Unit 1.31+ doesn’t support pre-0.8.2 njs resolve the issue? Or can you help me understand the change you'd like to see?

Hi @Dean-Coakley
Thanks. Actually, 0.8.0-0.8.2 works.
@thresheek could you help check it?

authors: Dean-Coakley hongzhidao

repo: nginx/unit-docs


text:
[enhancement] any plans to make it for real-time dev?

Do you have any plans to do it for real time developing?
At the moment it takes too much time for compiling new bundles and serving in a browser.
I think all of us wants to see best performance in building and new nice features like HMR, etc. (what we see in @AngularClass/angular2-webpack-starter)
Because of this fact above it becomes useless example although you made great work!

Hi @IPRIT, thanks for pointing out this topic.
Actually I was thinking about the same thing on the way home from gym - so, great minds think alike ;)
Although it's leas than just a couple of hours of work, this week I won't be able to provide that. But next week, I'll for sure publish an update containing the HMR capabilities.
Also, we need other tweaks both to reduce the size of output (especially server.js) and decrease the build times with & without HMR.

Hi! Anything new on this request?
Great work man 👍

Hi @wozniakm, thanks for your appreciation 👍
Just updated the repo with some utils from @nglibs and their loaders at the server-side, and currently working on caching the XHR/fs requests.
I think after this, HMR has the first priority - so we're not far from there. And then I hope I'll make several contributions to @angular/universal with these lessons learned.

Sounds great. If you can prepare some roadmap I would love to help you with developing that "starter" pack.

Thanks @wozniakm, I and all people benefiting from this repo would really appreciate that.
Currently we've got more or less no roadmap, except the items on the issue list. Feel free to add new items with enhancement tag/label, so I will do that and let's progress with them.
Actually, just after the cache implementation new items will occur.

Hi @fulls1z3, is there any new news?

Hi @IPRIT, I made a few tries on it.
First tried the HMR from @AngularClass, but it simply didn't work. Then made another try using webpack-hot-middleware. Seemed like it was working, but it simply re-triggers the webpack process from all over.
I've read the issues about webpack-hot-middleware and they say it could happen because we use nodemon. So basically it would take a few more time until we got it running.
Meanwhile, I didn't commit that code because it would create a huge mess for the people who are already benefiting from this repo. If you're interested I can hereby share a zipped version of this repo and you can create a PR if you have the opportunity.

hi @IPRIT @MuhammadShaban @wozniakm, just pushed the latest changes including HMR support. Refer the instructions at the README and give it a try when u've got available time. Awaiting feedbacks 👍

thanks @IPRIT, i will try it as soon as possible.

authors: IPRIT MuhammadShaban fulls1z3 wozniakm

repo: nglibs/universal-example-app


text:
Add Abbrevation option to gender

Description
gender({ abbrev: true }) // F M
Proposed solution
No response
Alternatives considered
No response
Do you want to create a pull request?
No

I would like to do, can you tell me where in the code, we have to add this

https://github.com/ngneat/falso/blob/main/packages/falso/src/lib/gender.ts

At which line i have to add? @NetanelBasal

I am really interested to contribute , just need guidance as i am new to this? if you have any references , please give i will read and then contribute @NetanelBasal

https://github.com/ngneat/falso/blob/main/CONTRIBUTING.md

@RishiKumarRay Have a look at the language/language-codes implementation: https://github.com/ngneat/falso/pull/66/files
You can implement gender abbreviations in a similar way. 🙂

Please assign this to me.

authors: NetanelBasal RishiKumarRay dsa0x michaelxvoelker

repo: ngneat/falso


text:
Multiple allowed IPs not in the wg config

If I add multiple Allowed IPs for a client, I only can see first one appears in the config

However if I add a second one to the field "Extra Allowed IPs" it works, but I don't see it on the web page only "Allowed IPs"

this Bug already exists!!

authors: einbit vlad-infra-admin

repo: ngoduykhanh/wireguard-ui


text:
Added items

Added assorted barrows items, Changed Dart animation, Added dragon knife special attack and lastly blazing blowpipe + venator bow. Sorry about that

I forgot to push the darts change as well whenever i restarted it

Can you squash this into one commit and get rid of the 3 "changed darts as well" commits?

I have 0 clue how to do that man

Alright, I'll merge it and fix it up on my end.


text:
Timing issue - with Validator & Change Expression

I've got 2 fields, A - with a validation fn & B - with a change expression.
On change A - triggers B's change expression
I need A's validation fn to wait for B's change expression to complete - as A needs the latest value from B.
Is there a way to control the timing so A fires after B or any chance of adding a timing option?

Declare the validation in their parent field in order to get notified whenever one of them is updated, see https://formly.dev/examples/validation/matching-two-fields

authors: aitboudad warrendodsworth

repo: ngx-formly/ngx-formly


text:
Ionic pro - deploy

We are looking to setup ionic pro within our project but we are having a few issues. Has anyone had any luck with this?
The issues we have faced are due to the folder structure. Ionic looks for src/app/main.ts but it is actually in src/main.ts. I am not able to find any configuration settings for this. If I move this file I then get this error which I am unable to resolve.

ionic-app-script task: "build"
TypeError: Cannot read property 'kind' of undefined
TypeError: Cannot read property 'kind' of undefined
    at declareSymbolAndAddToSymbolTable (/.../node_modules/typescript/lib/typescript.js:21251:31)
    at bindPropertyOrMethodOrAccessor (.../node_modules/typescript/lib/typescript.js:22225:19)
    at bindWorker (.../node_modules/typescript/lib/typescript.js:21778:28)
    at bind (.../node_modules/typescript/lib/typescript.js:21627:13)
    at bindSourceFile .../node_modules/typescript/lib/typescript.js:20036:17)
    at Object.bindSourceFile (.../node_modules/typescript/lib/typescript.js:19979:9)
    at initializeTypeChecker (.../node_modules/typescript/lib/typescript.js:47371:20)
    at Object.createTypeChecker (.../node_modules/typescript/lib/typescript.js:24920:9)
    at Object.getTypeChecker (.../node_modules/typescript/lib/typescript.js:74466:79)
    at moduleNameResolvesToAmbientModuleInNonModifiedFile (.../node_modules/typescript/lib/typescript.js:74222:88)

Any advise would be great.

The Ionic CLI cannot build Angular CLI project (seems that it's what you're trying to do from your error), but you don't need the Ionic CLI to use Ionic pro, see: https://ionicframework.com/docs/pro/deploy/builds/
Their build system juste use the npm run build script and expect an output www folder, so that should work.

The issue was because we did not have the project in the root folder of our repo and the combination of that and not using the ionic cli was causing me problems.
Thanks for your help.

authors: mhuntley sinedied

repo: ngx-rocket/generator-ngx-rocket


text:
MissingTranslationHandler fires even when there is a translation available

I'm submitting a ...  (check one with "x")
[x] bug report => check the FAQ and search github for a similar issue or PR before submitting
[ ] support request => check the FAQ and search github for a similar issue before submitting
[ ] feature request

Current behavior
Missing translation handler fires for all translations.
Expected/desired behavior
I would expect it not to.
Reproduction of the problem
export class MyMissingTranslationHandler implements MissingTranslationHandler {
  handle(params: MissingTranslationHandlerParams) {
    console.log(params);
    return params.translateService.instant('NOTIFICATIONS.GLOBALS.UNKNOWN_ERROR');
  }
}

    TranslateModule.forRoot({
      loader: {
        provide: TranslateLoader,
        useFactory: HttpLoaderFactory,
        deps: [HttpClient]
      },
      missingTranslationHandler: {provide: MissingTranslationHandler, useClass: MyMissingTranslationHandler}
    }),

Please tell us about your environment:

ngx-translate version: 9.1.1

Angular version: 5.2.4

Browser: all

At which time are you setting the language / default language?
How are you using the translations?

authors: CodeAndWeb chrillewoodz

repo: ngx-translate/core


text:
Subscriptions are failing to update (Windows/Flutter)

This was originally reported by @escamoteur via Discord.
After running the software, leaving it for a bit and coming back, subscriptions were no longer updating, and mutations hang.
I'm attempting to reproduce now.

I have been unable to reproduce on macOS, using the same app that Thomas is working on. I'm getting my Windows machine setup to see if I can repro there.

I'm having the same problem; I'm on macos. Subscriptions updates are not consistent. They ran a couple of time and then just stopped updating. Sometimes they update after much time (3 to 5 min after the mutation has been executed). Here is my code:
  @override
  void initState() {
    super.initState();

    final subscription = client.subscribe(SubscriptionOptions(document: gql(r'''
        subscription members {
          members {
            id
          }
        }
      ''')));

    subscription.listen((res) {
      if (res.data != null) {
        print("${res.data}"); // <-- this is inconsistent
      }
    });
  }

Thanks @seyaobey-dev.
Is the mutation also taking place via the same app? I'd like to build something as close as I can to attempt a repro.

I found the bug. I'm working out a fix now.

I reported my findings to graphql, but I'm preparing an interim fix. I'll be pushing it soon.

I published a number of pre-release versions that should fix the issue. I'll be promoting them to a stable version soon.
nhost_flutter_auth    1.0.3-dev.1
nhost_flutter_graphql 1.0.3-dev.1
nhost_gql_links       1.0.1-dev.2
nhost_graphql_adapter 1.0.3-dev.2
nhost_sdk             1.0.3-dev.1

Still not quite there, but I'm focused on it.

Yes, OK, so the prereleases I mentioned earlier do not work. I misunderstood what my test harness was reporting.
But I have found the issue (I'm not disconnecting the socket properly), and I'll have something up by EOD. Thanks for your patience.

Hey folks,
I've pushed a new set of pre-release versions that are creeping towards stability. While I still anticipate issues after some of the problems I've observed in the link libraries, I have an early early report from @MaxSchilling suggesting that he is no longer encountering issues with subscriptions, after multiple token refreshes.
I also sent a PR over to the graphql repo to include stack traces in more of the exception they emit...because when these exceptions do happen, their source can be next to impossible to determine. Hopefully that will be accepted.
@escamoteur, unfortunately, this does not yet resolve issues with Donation Tracker. I'm going to have to do more link surgery to figure out what's happening there.
nhost_flutter_auth    1.0.3-dev.2
nhost_flutter_graphql 1.0.3-dev.2
nhost_gql_links       1.0.1-dev.4
nhost_graphql_adapter 1.0.3-dev.4
nhost_sdk             1.0.3-dev.2

OK, @escamoteur's issues were unrelated, and https://github.com/gql-dart/gql/issues/227, while problematic, isn't blocking, so we are now published as stable.
nhost_flutter_auth    1.0.3
nhost_flutter_graphql 1.0.3
nhost_gql_links       1.0.2
nhost_graphql_adapter 1.0.4
nhost_sdk             1.0.3

Please post back to this bug if there are any problems.

authors: seyaobey-dev shyndman

repo: nhost/nhost-dart


text:
🛑 Core is down

In a03f25a, Core ($CORE_URL) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Core is back up in 45422c4 after 12 minutes.

repo: nhs-work/test-upptime


text:
:globe_with_meridians: Remove opening times and fix lon/lat

I've also made the output alphabetised.

:rocket: deployment of profiles succeeded (http://profiles-pr-171.dev.beta.nhschoices.net)

repo: nhsuk/profiles


text:
Update licenses, packages, other docs in repo for open source usage

😯 Problem to Solve
Updates are needed to get our repo ready for being a public repo
💁 Proposed Solution/Tasks

[ ] Update repo to use an MIT license
[ ] Update package.json files license field correctly
[ ] Source code files have comments describing their licensing/copyright if necessary
[x] Add Repo links to communication channels for bugs and discussion
[x] Add Repo contributing docs describe contribution model (fork vs branch, contributor license agreement if needed)

At one point we had a thought that our icons might need different licensing than our code and binaries (e.g. creative commons instead of MIT). I don't see that listed anywhere in this issue. @rajsite or @nate-ni or @fredvisser  do you know if we ever reached a decision on whether that was necessary and if so what work needs to happen?

It was simpler, for the moment, to use a single license type so I had proposed that it all be under MIT. If there is a significant reason to start with a different license for images assets, we can talk about it. We can also update the licensing later if we have to.
For getting started, it seemed to make things more complicated than they needed to be, so I abandoned considering multiple license types.

If the answer to this post on Stack Exchange is correct, this is a fine starting point.

There is nothing in principle that would make the MIT license inapplicable to images, as long as they are subject to copyright. One thing to consider, though, is that the MIT license is not very frequently used for artistic works or presentations. So if you want to make it easy for your users to use some pictures in their works, you will do them a favor by dual-licensing them under a Creative Commons license. If you are the copyright holder, you can simultaneously license your work under as many licenses as you wish...Dual-licensing the pictures has the benefit that your project still uses the same license for all files but also offers using a more “natural” license for pictures for those users who would benefit from this.

If the answer to this post on Stack Exchange is correct, this is a fine starting point but we may end up also using CC if our clients aren't satisfied by MIT.

There is nothing in principle that would make the MIT license inapplicable to images, as long as they are subject to copyright. One thing to consider, though, is that the MIT license is not very frequently used for artistic works or presentations. So if you want to make it easy for your users to use some pictures in their works, you will do them a favor by dual-licensing them under a Creative Commons license. If you are the copyright holder, you can simultaneously license your work under as many licenses as you wish...Dual-licensing the pictures has the benefit that your project still uses the same license for all files but also offers using a more “natural” license for pictures for those users who would benefit from this.

Agree that MIT is probably fine to start. Other icon system licenses:

Material Icons usesApache 2.
Font Awesome uses CC BY 4.

Licenses have been updated. All packages reflect an MIT license on npmjs.org or nuget.org where appropriate.

authors: jattasNI nate-ni rajsite

repo: ni/nimble


text:
race condition fix

get_current_frames() can crash because the list of active threads can change between calling threading.enumerate() and later sys._current_frames()
Following is one way to refactor to avoid that crash. Basically, the thread has to exist in both sets of data to qualify to be returned from the function
def get_current_frames() -> Dict[int,Dict[str,Any]]:
    """Return current threads prepared for
    further processing.
    """
    threads = {thread.ident: thread for thread in threading.enumerate()}
    frames = {}
    for thread_id, frame in sys._current_frames().items():
        thread = threads.get ( thread_id, None )
        if thread is not None:
            frames[thread_id] = {
                'frame': thread2list(frame),
                'time': None,
                'id': thread_id,
                'name': thread.name,
                'object': thread
            }
    return frames

This is fixed in #91.

Thanks, this should be uploaded as v2.0.6 https://pypi.org/project/hanging-threads/

authors: niccokunzmann remdragon


text:
在Mac下使用多个SSH key | 天意人间舫

https://blog.lovejade.cn/2016/01/27/ssh-mac.html

ssh 准备连接远程服务器却遭提示 Permission denied (publickey) , 这是由于您没有将公钥( publickey )  添加到本地 ssh 环境造成的，或者是由于多日未 进行ssh 登录操作，本地 publickey 失效造成的。我的mac os x 环境隔几天没有登录ssh 就会报 “Permission denied ” 啦。只要 使用 ssh-add 命令再次添加一下公钥即可。


text:
Wands fail at very large world coordinates

Note: I'm writing this up for a patron of AoF7... English is their second language and are not comfortable in it.
At very large distances, the holograms the wands draw to preview where blocks will be placed get wonky and "wavy"... animatedly so. It becomes difficult to accurately place blocks.
In the patrons words (they use computer translation, I was told):

the problem is the hologram
You have to be extremely careful where you press otherwise it will always put everything in the wrong place
It feels like you only have an extremely small area where you can press

Some examples they shared:
Worst Case They Had

Milder

The phenomena was gone when getting down to X / Z: ~50,000 / ~50,000.

I'll look into it

authors: SubordinalBlue nicguzzo

repo: nicguzzo/wands


text:
Update binaries to 0.12.5 version

Hi @nicholasklick! Here is updated versions of binaries. Initially the PR was requests from https://github.com/nicholasklick/wkhtmltoimage-binary/issues/12, but since it won't fix the problem of the issue I'm not sure if we should close the issue.

Thanks for this. I will try to review this and get it pushed tonight.

Please merge this if seems fine. I also need this

repo: nicholasklick/wkhtmltoimage-binary


text:
🛑 Authinator is down

In 2b04ee5, Authinator (https://www.authinator.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Authinator is back up in 1d1ba2e.

repo: nickadam/status


text:
run python script telegram bot with webhook

I'm writing a telegram bot by python telepot package.I used getupdate method, so I want to use webhook method ,how can i do it?
I can set webhook another  script that is very simple on my host, but i dont know how convert getupdate to webhook.i use telepot.

For webhook usage, please refer to the webhook examples.
Simply put, instead of using MessageLoop, you use OrderedWebhook:
from telepot.loop import OrderedWebhook

def handle(msg):
    # ......

bot = telepot.Bot(TOKEN)
webhook = OrderedWebhook(bot, handle)

webhook.run_as_thread()

And then, you have to make the webserver, on receiving some HTTP requests, feed data to the OrderedWebhook object. Using flask as an example, you may do something like this:
from flask import Flask, request

app = Flask(__name__)

@app.route('/webhook_path', methods=['GET', 'POST'])
def pass_update():
    webhook.feed(request.data)
    return 'OK'

Good luck.

thank you, is there another way?my server doesn't support flask.

My example illustrates the principle. It's your job to find a way do that in your own webserver.

yes, thank you.you are all right
On Thursday, July 20, 2017 12:40 PM, Nick Lee <notifications@github.com> wrote:

My example illustrates the principle. It's your job to find a way do that in your own webserver.—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hello I need your help please guide me.first I have to thanks to your good example in github about telepot.I write a script that wich i use telepot and handle(msg). I want to set web hook for it , you already told me i have to use flask and orderwebhook in my script to set webhook,my server doesnt support flask and I dont have another way to set webhook for my script.please help me to set webhook on my telegram bot.thank you.

So, which webserver are you using?

I use heliohost.org site as my web server.
On Sunday, July 23, 2017 3:55 PM, Nick Lee <notifications@github.com> wrote:

So, which webserver are you using?—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hello, I can't understand the right way to send certificate with setWebhook method.
Many Thanks!

authors: Duiesel nickoala samira4237

repo: nickoala/telepot


text:
anyof syntax

I am trying to validate a list of different dicts.
Can't seem to find it in the docs. So I hope I put it here....
Trying to validate the following:
{
    "products": [
    {
        "title": "test"
    },
    {
        "name": 1
    }]
}

But below schema don't work.
schema1 = {
    'products': {'type': 'list',
                 'schema': {'type': 'dict',
                            'anyof': [
                                {'name': {'type': 'number'}},
                                {'title': {'type': 'string'}}
                            ]}
                 }
}

Anyone a clue how to do this ?

found it:
schema1 = {
    'products': {'type': 'list',
                 'schema': {
                     'anyof': [
                         {'type': 'dict', 'schema': {'name': {'type': 'number'}}},
                         {'type': 'dict', 'schema': {'title': {'type': 'string'},
                                                     'value': {'type': 'number'}}}
                     ]}
                 }
}

repo: nicolaiarocci/cerberus

license: isc


text:
Return the selected attribute names using clickCallback

Hi there,
I'm trying to return the attribute names using clickCallback - i.e. when you click on a cell it returns the Column and Row titles that particular cell belongs to. Is there anyway of doing this?
Thank you,
Kind Regards,
Dan

Sorry I figured it out using the filters object. Thanks.

repo: nicolaskruchten/pivottable


text:
Allow Laravel 6.0

As there is no breaking change for illuminate/database from 5 to 6, allowing Laravel 6.0 is OK

please merge this

Hi @veneliniliev for the record, for now I use my fork, you can use it too. I will keep it until it's merged:
composer.json
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/kylekatarnls/searchable"
        }
    ],
    "require": {
        "nicolaslopezj/searchable": "dev-patch-1",
    },

authors: kylekatarnls veneliniliev

repo: nicolaslopezj/searchable


text:
Responsive issues

[x] User card username too long
[x] Post item content too long

Done

repo: nicolasparada/nakama


text:
events.endTurn() doesn't work with multiplayer: local

I'm trying to get Local Master multiplayer working, looking at
https://boardgame.io/#/multiplayer
In the version  in the codesandbox,

the onClick handler for the TicTacToeBoard is
 onClick(id) {   if (this.isActive(id)) {     this.props.moves.clickCell(id);   } }
and we have  set
flow: {   movesPerTurn: 1, ... }
so that the framework knows to end the turn after every move.

This works as expected, you can play as both players by alternating between the first client and the second client.
However, if we modify the code as follows, to match with the beginning tutorial at  https://boardgame.io/#/tutorial:

Adding an explicit endTurn() to the click handler:
 onClick(id) {  if (this.isActive(id)) {    this.props.moves.clickCell(id); this.props.events.endTurn();  } }
Removing the movesPerTurn property from flow,

then the game stops working. Clicking on the first client results in error:
ERROR: invalid stateID, was=[1], expected=[0]
and it seems as the turn doesn't end.
Shouldn't both methods work?
Note: If we run without local multiplayer mode by using
const App = Client({   game: TicTacToe,   board: TicTacToeBoard, }); export default App;
Then everything works fine.

Thanks for the report.
This is due to a race condition where the client is not seeing the new game state between the move and the end turn event. I'll take a closer look soon.

A workaround for now (until this gets fixed) would be to wrap the endTurn in a setTimeout like this:
this.props.move.clickCell(id);
setTimeout(() => this.props.events.endTurn(), 0);

Fixed in v0.31.5.
Can you upgrade and try?

authors: nicolodavis peelin


text:
[Pseudo Code] Catch Kappa Instances

Catches kappa regardless of position in the string.

IT BROKE THE DAMN BOT LMAO

license: ISC


text:
The calculation of state_mean and state_std in d4rl_info.py

Hello there, we are currently trying to use the code to give some other dataset a go, like walker2d-random-v2. So we would like to kindly ask how state_mean and state_std in d4rl_info are calculated, or whether there is any open data for these values. Thank you very much!

They are calculated from the dataset.
Refer to 1st block of 'dataset' section in the ipynb file.
you can remove the last 4 print and assert statements, instead add the calculated stats to D4RL_DATASET_STATS dictionary.

They are calculated from the dataset. Refer to 1st block of 'dataset' section in the ipynb file.
you can remove the last 4 print and assert statements, instead add the calculated stats to D4RL_DATASET_STATS dictionary.

Thanks for ur reply, that helps a lot :)


text:
Tagset

Hi,
what is the full tagset used by the parser?
Thanks!

We use the Penn Treebank to train -- here is one page that explains the labels used: http://www.surdeanu.info/mihai/teaching/ista555-fall13/readings/PennTreebankConstituents.html
You can also access parser._label_vocab and parser._tag_vocab (for the newer models only); these are private but you can print them to get a sense of what the full set of labels is.

Thanks Nikita! Exactly what I needed.
Can I assume the "Treebank-3" https://catalog.ldc.upenn.edu/LDC99T42 was used in training?

That's correct

authors: bustrofedico nikitakit

repo: nikitakit/self-attentive-parser


text:
onclone callback removed?

Has the onclone  callback removed? If yes, what's the recommended way of manipulating the cloned document?
JSFiddle: http://jsfiddle.net/w8woox8c/3/
Specifications:

html2canvas version tested with: 1.0.0-alpha.9
Browser & version: Chrome 63
Operating system: Windows 8, Ubuntu 17.10

Any news on that topic?

@niklasvh This looks promising, thank you very much! I'll give it a shot and I'll get back at you.

Updated to 1.0.0-alpha.10 but still not working for me. Am I doing something wrong, here?
    html2canvas(this.exportSlide, {
      canvas:  this.exportCanvas,
      logging: false,
      width: 1080,
      height: 1080,
      scale:   1,
      onclone: function(doc) {
        console.log('onclone fired');
      }
    }).then(() => {
      this.exportAnchor.href = this.exportCanvas.toDataURL();
      this.exportAnchor.download = "SlideExport";
      this.exportAnchor.click();
    });

Trying to use the solution in #1448

Also no worky for me. Onclone doesn't seem to do anything.

@JasonTheAdams , @moettinger I've looked into my code and it works just fine with version 1.0.0-alpha.10.
Can you try wrapping this into setTimeout with, let's say 1 second delay?
If that doesn't work, I'll provide you with my working code...

Hi @markokoleznik! Sorry, which part, exactly, are we wrapping in a timeout? The entire html2canvas call or just the onclone function?

But If I modify the element inside the onclone, it never reflects in the generated PDF. I am not using html2canvas directly, I am using it through html2pdf which uses jsPDF and html2canvas inside. I am not sure how to make it work.

@agiratech-vigneshm and others who might find this useful.
I have been digging around to find a solution to this and was disappointed that I could find none. Here is my solution after going through html2pdf code. Unfortunately they don't document the lib very concisely and this is why we are glob-trotting looking for answers like this.
import $ from 'jquery';
html2pdf()
        .set({
          margin:      [10, 10],
          filename:    `blah.pdf`,
          // don't use html2canvas onclone here
        })
        .from('source')
        .toContainer()
        .then(()=>{
          const doc = $('.html2pdf__container');
          doc.find('.remove-on-export').remove();
          doc.find('.hide-on-export').hide();
          doc.find('.show-on-export').show();
        })
        .save();

This enables me to manipulate the document however I like as you would in onclone.

But If I modify the element inside the onclone, it never reflects in the generated PDF. I am not using html2canvas directly, I am using it through html2pdf which uses jsPDF and html2canvas inside. I am not sure how to make it work.

Thank you for answer it helped me a lot. And you are right about docs.

authors: JasonTheAdams agiratech-vigneshm emahuni erensertkaya markokoleznik moettinger sagrawal-idrc


text:
Laravel 5.3 compatibility added

I tested this with Laravel 5.3.21.

I tested this via a patch file on some basic functionality in a Laravel v5.3.19 project and didn't have any issues.

The unit tests for this are failing because of the differences between Laravel 5.2 & 5.3, have a look at the changes on my latest comment to see what I had to fix:
https://github.com/Stubbs/laravel5-jsonapi/commit/5af2634e1345beae257c644c5cdef1345db095f7

Stubbs@5af2634 is working good for me 👍

authors: Stubbs aethr dneckel mspiderv


text:
Package structure issue

I've received this warning during an installation
Downloading https://github.com/nim-appkit/values using git    Warning: Package 'values' has an incorrect structure. The top level of the package source directory should contain at most one module, named 'values.nim', but a file named 'values_test.nim' was found. This will be an error in the future.

As you can see from the commit log, this library is ancient and I haven't used Nim in years.
I'm happy to merge PRs but won't do any fixes myself.

authors: theduke vulogov

repo: nim-appkit/values


text:
Implement new package scanner

Changes to the original package scanner:

Can compare two package lists and only check the modified packages.
Checking the availability of URLs is now a runtime option.
Checks not just the web URL for availability but also docs and download URLs.
All URLs are ensured to be https.
License checking has been removed as it produced too many false positives and lead to merges being delayed by months because of a failed CI status.
Package names are checked against a character whitelist. This can be used in the future to make sure names are valid identifiers in Nim.
Duplicate package names are reported for all occurences of the name instead of only starting at the second occurence. Their URLs are printed to help with resolving the situation.
The special tag "deleted" marks a package as having its repository deleted. Its entry will not be checked for errors anymore.
All errors of a package are reported, not just the first.

Changes to the workflows:

The on-push workflow no longer checks URLs. It still checks all package entries for mistakes that can be detected without network requests.
The on-pull-request workflow does check URLs for availability, but it only processes modified or new packages.
A periodic URL check of all packages is possible but not implemented.

You removed the GPL-3 header. That's fine if you basically reworte it from scratch but it's quite mean esp wrt to @FedericoCeratto

I'm merging this since it is a great improvement.

authors: Araq Yepoleb ringabout

repo: nim-lang/packages


text:
Fix buffering in extractFile

Fixes #14

Changed following @Araq's suggestion

repo: nim-lang/zip


text:
[gh385] Minify the CSS and JS for production build

close #385
What happened 👀
We customize the build process between development and production, as we want the production build to be as minimal as possible.
Insight 📝
With JS, we config the esbuild configuration:

Add minify option to be true in the case we don't have the --watch option (production)

With CSS, we separate two scripts:

build:css for the production that included the '--style=compressed' option.
build:css-dev for the development environment, without the compressed option.

Proof Of Work 📹

Minify JS
Minify CSS

Generated package.json

Generated Procfile.dev

@sanG-github rebase needed 🙇

@tyrro rebase needed and then we can merge this one 🚀

@malparty , I'll tag @sanG-github so that he can do the rebasing for me 🙈 😏

authors: malparty sanG-github tyrro

repo: nimblehq/rails-templates


text:
install_debian: Cleanup and add Linux Mint

TODO: Merge install_debian with main install script.

@windowz414 done with ❤

repo: nimsandu/spicetify-bloom


text:
Any way to ask ninja to prioritise a file in compile order?

(Asked the same question over at cmake, and it was suggested I look to ninja.)
I was pointed at an old (open) issue. I'm not sure if it applies directly to what I'm asking or if anything was implemented.
I have a large project with many, many files.
One file takes a very long time to compile - about 2.5 minutes. I don’t have control over that file because it’s a third party dependency.
The build (across cores) looks like this:

With fewer cores, it’s possible this file starts late in the compile which extends the compile time a lot.
Is there a nice way to ask ninja to compile this file as early as possible?

I guess your build.ninja is generated by CMake?
Unfortunately there's no way without touching the build manifest. -> #232.

Yes it is generated by cmake.
So I would need something like #1333 to land here, and then for cmake to take advantage of that when generating the build.ninja file?
Just trying to understand the scope of the problem.
Thank you!

You wouldn't need to wait for CMake, as you can specify the target order when calling ninja (if #1333 was merged). If I understand the PR correctly it would also notice the long compile time and build the target first on subsequent builds.

Wow - that sounds like it would be an amazing addition to ninja.
Thanks Jan!

authors: asmaloney jhasse

repo: ninja-build/ninja


text:
More accessible coloring on Team Scores

#redScore and #blueScore now have background colors for the teams, as the fonts being the colors are difficult to differentiate on poorly calibrated monitors.

I just did this as a "first pass" since it personally affected some of the people I played with.
Assuming you want some work done on this, I was planning on improving some more Accessibility/styling things, such as:

Clearer board borders (for which team starts)
Option to change team colors (for those with different types of color blindness)
Hideable "controls" region (for those with smaller screens / want a clearer UI)

For sure open to more suggestions as well.

repo: ninjabunny/KodeNames


text:
[RESOLVED] High ICMP ping response time

Problem started at 08:25:14 on 2023.05.22
Problem name: High ICMP ping response time
Host: MF-FG
Severity: Warning
Operational data: Value: 695ms
Original problem ID: 21551
Event details in Zabbix: http://192.168.232.99/tr_events.php?triggerid=23132&eventid=21551

Problem has been resolved in 6m 0s at 08:31:14 on 2023.05.22
Problem name: High ICMP ping response time
Host: MF-FG
Severity: Warning
Original problem ID: 21551


text:
throw exceptions while reading picture position from Excel (2.4.1-2.5.3)

test-Pic.xlsx

进过进一步的测试，发现从Excel中复制的图片时，报以上错误。从其他地方复制或剪切的图片或从文件中加载的图片正常。

#403

The answer to your question exists in xl/drawings/drawing1.xml. This is an onecellanchor block.

if you want to get the exact position of a picture, you have to use ext:cx and ext:cy. For details, you can read http://officeopenxml.com/drwPicInSpread-oneCell.php

authors: dragon-p tonyqus

repo: nissl-lab/npoi


text:
Any way to change smart plug "In Use" threshold?

Basically, I have a consumer that uses the smart plug and while it's on sleep mode sometimes I'm getting InUse=true, maybe because of the power supply drawing higher current on some random points. I believe it's drawing less than it's on.
this isUse value affects my automation. I'm getting a false positive.
so if I will be able to change the threshold it will be great.
btw I'm using Eve app
Thanks!

That's actually a good idea...
for now, it's recommended to use it in parallel with homebridge-delay-switch

mmmmmm.... I'm trying to understand how can It be useful in my case... What did you think? 🤔

Yeah, it fixed the issue using homebridge-delay-switch.. but added more automations..

repo: nitaybz/homebridge-switcher-platform


text:
add open telemetry tracing

Closes: https://github.com/nitrictech/node-sdk/issues/149

Codecov Report
Patch coverage: 87.42% and project coverage change: +0.08 :tada:

Comparison is base (13a77b3) 85.81% compared to head (b8e447f) 85.90%.

:exclamation: Current head b8e447f differs from pull request most recent head 6052bdb. Consider uploading reports for the commit 6052bdb to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #170      +/-   ##
===========================================
+ Coverage    85.81%   85.90%   +0.08%
===========================================
  Files           82       83       +1
  Lines         1445     1518      +73
  Branches       143      207      +64
===========================================
+ Hits          1240     1304      +64
- Misses         203      208       +5
- Partials         2        6       +4

Impacted Files
Coverage Δ

src/api/documents/v0/collection-group-ref.ts
89.65% <ø> (ø)

src/api/documents/v0/documents.ts
100.00% <ø> (ø)

src/api/queues/v0/queues.ts
96.34% <ø> (+0.18%)
:arrow_up:

src/api/secrets/v0/secrets.ts
97.26% <ø> (+0.07%)
:arrow_up:

src/resources/bucket.ts
74.35% <0.00%> (+0.67%)
:arrow_up:

src/api/errors/deadline-exceeded.ts
25.00% <25.00%> (ø)

src/resources/topic.ts
86.95% <33.33%> (+0.28%)
:arrow_up:

src/resources/secret.ts
78.37% <45.45%> (+0.60%)
:arrow_up:

src/faas/v0/start.ts
33.60% <57.14%> (+1.10%)
:arrow_up:

src/resources/collection.ts
77.77% <66.66%> (+1.03%)
:arrow_up:

... and 37 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

authors: HomelessDinosaur codecov-commenter

repo: nitrictech/node-sdk


text:
How can I override the normal controls?

I'm trying to change how the save button looks in the editor but I simply can't find a way where I could just override the Save icon.
I'm trying to build a customControl but the props returned don't suit what I need to do. All I need is to have the same behavior as the Save button.
I can't use the editor ref and can't save the state returned in the callback. I have a lot of editors across my app and I need a one-for-all configuration.

Hi @Kauyumari
You cannot "override" the default controls but you can replicate all default controls functionalities with the provided API. Also you can add set the id property if you have multiple editors.
If you want to create your own Save button, you can use a callback custom control and when you click it you will get all necessary parameters to replicate the functionality.
This is the onClick event API:
(editorState: EditorState, name: string, anchor: HTMLElement | null) => EditorState | void
Just use the editorState instance to replicate the internal save feature.
Find a full example of callback custom controls here.
Hope this helps you out!

authors: Kauyumari niuware

repo: niuware/mui-rte


text:
Revert #16

Somehow #16 makes cargo test unstopped.
Issued: https://github.com/laysakura/beam/issues/17
I revert #16 for now.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

oops, selected wrong repo for base branch

repo: nivaldoh/beam


text:
plugins/avante: init

Add support for avante.nvim, an AI companion plugin.
cc @synthe102 for testing
Fixes #2096

@mergifyio queue

Thanks for the review !

Has anyone tested the diff bindings ? Can't get them to work

authors: GaetanLepage elythh

repo: nix-community/nixvim


text:
先决条件

[x] 我已尝试更新模板版本

[X] 我已检索模板手册

[X] 我已检索项目 wiki

[X] 我已确认这个请求没有在其他 issues中提出过。

现有模板的不便之处
为了避免页眉出现文字重叠的问题，我使用了模板中的短标题功能。
\section[短标题]{这是我非常非常非常非常完整的标题}
但也因此导致目录显示了短标题而难以表达其原有含义。

提出可能的解决方案
是否可采用文档选项或者其它形式使用户可选择目录使用完整标题呢？
提出考虑过的方案
No response
额外信息
No response

需要修改 \chapter、\section 等命令的底层实现。
清华这边的格式允许页眉换行（https://github.com/tuna/thuthesis/issues/735），作为参考。

不错的方案👍。
不过南大这边的模板中页眉是“左章右节”的方案，若采用THU的方法应需要对章和节的标题均有允许换行的实现。

请看 https://tex.stackexchange.com/questions/6862/how-can-i-display-a-short-chapter-name-in-the-header-and-a-long-chapter-name-in，用下面这两个命令即可覆盖页眉页脚的标题设置：
\chapter[长长长长长长长长长长长长长长长长长长长长长长长长目录标题]{长长长长长长长长长长长长长长长长长长长长长长长长标题}
\chaptermark{页眉短标题}

\section[长长长长长长长长长长长长长长长长长长长长长长长长目录标题]{长长长长长长长长长长长长长长长长长长长长长长长长标题}
\sectionmark{页眉短标题}

njuthesis 现在对 twoside 模式的设置是奇数页呈现节标题，偶数页呈现章标题，所以也很好奇你目前是否用的是 oneside 模式，或者有没有用上最新版。新版手册中也给出了页眉内容的定制方式。

感谢，完美解决问题。👍
另，以上内容确实是设置了 oneside 模式。

authors: atxy-blip vill-jiang zepinglee

repo: nju-lug/NJUThesis


text:
Added new library 'prop-types'

#33

I suggest to make all these changes in one PR, there is no need to split it to multiple =)

sorry, I just made the changes in the online editor and there did not seem to be a way to merge the patches into on PR

You can choose your patch branch and edit more files.

updated #37 closing other PR's

Thanks 👍

authors: aribronstein nkbt

repo: nkbt/react-page-click


text:
Webpack v2 support

Add support for resolve.modules.
Supersedes #39.

@nkt Can you take a look ?

Thanks!

authors: MatthieuLemoine nkt

repo: nkt/atom-autocomplete-modules


text:
EEPROM not written to flash anymore

The simulated EEPROM seems to be written to flash no longer.
Parameters revert to older values after a reset of the device.
These older values were written at some point, so writing to flash used to work.
Suggestion: write small FW to just read/write single value to flash, to verify this still works.
Workaround for powersensor: run psconfig every time after connecting/resetting the device.

Location of problem: EE_Init does not return HAL_FLASH_ERROR_NONE

Running into PAGE_FULL in EE_VerifyPageFullWriteVariable as part of EE_Init; case VALID_PAGE, Page0 valid, Page1 receive.
A call to EE_Format (temporarily made public) seems to have fixed it for now.
ToDo: Find out when/why this is necessary.

Consider using EEPROM support built into stm32duino, see https://github.com/frankpolte/stm32duino/tree/master/hardware/arduino/avr/libraries/EEPROM


text:
Repository is almost 450MB

The size of this repository is 436MB unpacked (although the download is 'only' ~150MB), which is insane for a tiny JSON parsing library. There are multiple cases where (IMO) the size can be trimmed down massively without actually impacting the repo.
The .git directory is 178MB, likely since you have a lot of changed binary files. You can easily slash the size of this directory by ~90% using git repack (which has drawbacks for repo history, but unless you really care about specific info about changes from years ago this is fine).
Why do you have a 23MB file of 1000000 signed integers in test/data/regression/signed_ints.json Do any of those ints besides the first few thousand really contribute any code coverage? I'd wager not. If the purpose of that many is to make sure the program can handle really large JSON files, why is there also another 23MB of unsigned integers? And 22MB of floats? Having lots of tests is good but this is seriously affecting users with slow internet speed who want to contribute to the project.

Duplicate of #96.

authors: nlohmann toomanybananas

repo: nlohmann/json


text:
Build error  when modify value

Hi, all
When build code ,got this error: ,bug use g++ -fpermissive test.cpp can build success and work success, how to fix this bug

The follow is my code :
`
#include
#include
#include <nlohmann/json.hpp>
using namespace nlohmann;
int main()
{
constexpr auto jsfile = "/home/parallels/nlohmanntest/sensors.json";
std::ifstream jsonFile(jsfile);
if (!jsonFile.is_open()){
std::cout << "Unable to open json file" << std::endl;
}
auto sensors = json::parse(jsonFile, nullptr, false);
if (sensors.is_discarded()){
std::cout << "Invalid json - parse failed" << std::endl;
}
for (const auto& sensor : sensors){
    const json newType = (long int)130;
    sensor["sensorType"] = newType;
}
jsonFile.close();

std::ofstream newFile(jsfile, std::ios::trunc | std::ios::out | std::ios::in);
newFile << sensors.dump(4) << std::endl ;

return 0;

}
`
thanks a lot~~

I find the error : for (const auto& sensor : sensors) ==> for (auto& sensor : sensors),
so close the issue


text:
Allow parsing from std::byte containers

Fixes the lexer to read from containers of std::byte.
Closes #2546.
Todo:

[ ] add regression test

Coverage remained the same at 100.0% when pulling 5dd06714b13b375acdc987b9261679d0c331b9ed on std_bytes into dfedefb99369dabc86e307ba7edaeb4e9116720d on develop.

authors: coveralls nlohmann


text:
nist_score vs bleu_score's SmoothingFunction.method3

The 3.2.5 release introduced the nltk.translate.nist_score module, but I was under the impression that the BLEU score using nltk.translate.bleu_score.SmoothingFunction.method3() for smoothing was the NIST metric, as its docstring says "NIST geometric sequence smoothing". Are these two totally different things? They do give very different scores:
>>> from nltk.translate import nist_score, bleu_score
>>> hypothesis1 = ['It', 'is', 'a', 'guide', 'to', 'action', 'which','ensures', 'that', 'the', 'military', 'always','obeys', 'the', 'commands', 'of', 'the', 'party']
>>> reference1 = ['It', 'is', 'a', 'guide', 'to', 'action', 'that','ensures', 'that', 'the', 'military', 'will', 'forever','heed', 'Party', 'commands']
>>> nist_score.sentence_nist([reference1], hypothesis1)
0.05876787197888857
>>> bleu_score.sentence_bleu([reference1], hypothesis1, smoothing_function=bleu_score.SmoothingFunction().method3)
0.41180376356915777

The docstring for nist_score.sentence_nist() says it comes from mteval-14.pl, while the Chen & Cherry 2014 paper says their method 3 comes from mteval-v13a.pl. Perhaps we could just update the docstrings of the respective functions so it's more clear that the smoothing function and the NIST metric are not the same, so that people who aren't already familiar with the implementation (like me) are not confused.

Sadly, NIST score is still broken. Although the NIST tokenizers are working fine. https://github.com/nltk/nltk/pull/1840
But I think it's a good idea to clean up the documentation too.
The SmoothingFunction().method3 is a smoothing method from NIST but NIST score reports a totally different score from BLEU, it doesn't scale to the 0-0.1 range too. Smoothing only changes the precision part of the score but the NIST has a different brevity penalty and also a different way to compute the final score.
You can take a look at the examples from the files in nltk_data/models/wmt15_eval, to download, >>> nltk.download('models/wmt15_eval'.
To summarize =)

NIST is a different score from BLEU
NIST don't range from 0-1.0
NIST smoothing in BLEU is introduced in mteval-v*.pl to smooth BLEU and NIST precision
NIST comes with its own brevity penalty different from BLEU
BLEU uses geometric mean of the ngram overlaps, NIST uses arithmetic mean.

Thanks for the explanation. Sorry I missed #1840 when looking for discussion about this---my search criteria included is:issue.
I'm glad you're working on nist_score, because otherwise I would have continued thinking that I was computing the NIST score when I was just using NIST smoothing :)

Resolved. NIST was completed in #1840 a couple of month before =)

authors: alvations goodmami

repo: nltk/nltk


text:
feat: getcharid and getguildinfo base command

Description
Implementing missing features for Kafra NPC.
To-do

[x] cutin
[ ] percentheal
[ ] viewpoint
[x] implode
[ ] countitem
[ ] delitem
[ ] basicskillcheck
[ ] getskilllv
[x] getcharid
[ ] getguildinfo
[ ] savepoint
[ ] emotion
[ ] logmes

@nmeylan There's any easier way to test these commands on compile time?

And how about commands that isn't implemented at rAthena codebase?

Nevermind, it belongs to Hercules and not rAthena.

@nmeylan There's any easier way to test these commands on compile time?

You mean at build time with test? Unfortunatly i have not setup unit test for this native function handler method, but it would be nice to have. I am afraid that we add more and more feature to this function without tests, it will be harder to catch up

Are you sure that the function "logmes" is placed in any Kafra's script? I took a look into rAthena and Hercules src's and didn't found anything related there.
Maybe it will have a specific usage that you're planning?

repo: nmeylan/rust-ro


text:
Add error for range of month inputs (must be 1.0 - 12.99)

Thanks Ian, for pointing this out!!
Ian wrote:
The User Manual has example month inputs including "spawning biomass is calculated at this time of year (1.5 means January 15)", "Survey timing is now cruise-specific and specified in units of months (e.g., April 15 = 4.5)", and "The above example specifies settlement to mid-May (month 5.5)".
I never thought about it before, but Max Cardinale (@akatan999) asked about how to associate something with the end of the year.
I found this conversion: temp1=max(0.00001,(month-1.0)/12.); //  month as fraction of year
in `get_data_timing(): https://github.com/nmfs-stock-synthesis/stock-synthesis/blob/main/SS_global.tpl#L213
which implies that the range of month inputs should start at 1.0 as in the example text.
So should December 31 be entered as 12.99 (which would convert using the above code to a fraction of 0.9992), or is 13.0 an acceptable input?  Also, should there be a warning for inputs greater than the upper limit (e.g. >13.0 or >=13.0)?
KD and RM thought it would be good to add a warning or error:
Based on the code, it seems like if using 13.0, it would convert to temp1 = 1.0, which wouldn't be right?
Agree with Rick, the warning (or even error) is a good idea!
Originally posted by @k-doering-NOAA in https://github.com/nmfs-stock-synthesis/doc/issues/93#issuecomment-1030182264

I could try this, if Rick approves. Or, if @kellijohnson-NOAA feels like this would be a good one to learn @Rick-Methot-NOAA 's workflow, feel free to reassign.

done.  slipped this one in with the bioecon branch

authors: Rick-Methot-NOAA k-doering-NOAA

repo: nmfs-stock-synthesis/stock-synthesis


text:
How do you make a bounded up-down counter without modulo?

Modulo operators for negative numbers are a known issue, and the operator is not implemented on stable.
What is the work around for a decrement to underflow around an arbitrary bounds?
This test shows the modulo producing strange results on decrement that disappear if you force the range into positive.
from nmigen import *
from nmigen.back.pysim import Simulator

class counter_thing(Elaboratable):
    def __init__(self, max_count, rst):
        self.dir = Signal(1)
        self.mod_pos = Signal(range(max_count), reset=rst)
        self.mod_neg = Signal(range(max_count), reset=rst)
    def elaborate(self, platform):
        m = Module()
        m.d.sync += self.mod_pos.eq((self.mod_pos + max_count - 1) % max_count)
        m.d.sync += self.mod_neg.eq((self.mod_neg - 1) % max_count)
        return m

def test():
    count = rst
    for _ in range(2*max_count):
        yield
        count = (count - 1) % max_count
        print(str((yield dut.mod_pos)) + ', ' + str((yield dut.mod_neg)) + ', ' + str(count))
        assert (yield dut.mod_pos) == count # passes
        assert (yield dut.mod_pos) == (yield dut.mod_neg) # fails after underflow

if __name__ == "__main__":
    max_count = 17
    rst = 13
    dut = counter_thing(max_count, rst)
    sim = Simulator(dut)
    sim.add_clock(1)
    sim.add_sync_process(test) # or sim.add_sync_process(process), see below
    sim.run()

What is the work around for a decrement to underflow around an arbitrary bounds?

Assuming you are interested in a counter that increments or decrements by 1 each cycle, something like this would give you arbitrary overflow bounds:
class BoundedCounter(Elaboratable):
    def __init__(self, min, max):
        self.min = min
        self.max = max

        self.up = Signal(1)
        self.count = Signal(range(min, max + 1))

    def elaborate(self, platform):
        m = Module()
        with m.If(self.up):
            with m.If(self.count == max):
                m.d.sync += self.count.eq(min)
            with m.Else():
                m.d.sync += self.count.eq(self.count + 1)
        with m.Else():
            with m.If(self.count == min):
                m.d.sync += self.count.eq(max)
            with m.Else():
                m.d.sync += self.count.eq(self.count - 1)
        return m

Does this answer your question?

I'd suggest to avoid the % and / operators in hardware design (when you divide by a number that is not a power of 2). division is slow and expensive in hardware, it would take more than one clock at a rate users usually want to work, that is why usually backends don't implement that in synthesizable code regardless of what language you would use.

authors: BrettRD DaKnig whitequark

repo: nmigen/nmigen


text:
Fix {r,w}_level in AsyncFIFOBuffered

This Fixes #485 as per discussion in https://freenode.irclog.whitequark.org/glasgow/2020-10-22#1603367304-1603369561

This should be rebased onto master now (which includes the other PR)

Thanks!

authors: anuejn whitequark


text:
feat/starlark scripting

Starlark Scripting Support:

Added Starlark scripting support to the project.
Updated scripting/scripting.go to include functions for running Starlark programs and handling Starlark scripts.
Added a new file scripting/starlark_wrapper.go to define Starlark built-ins and helper functions for converting between Starlark values and Go values.
Updated sample to include the sample scripts for starlark

Code Refactoring:

Replaced the deprecated ioutil.ReadFile with os.ReadFile in server/config.go for reading configuration files.
Updated go.mod to include the go.starlark.net dependency.

Good job this works.

repo: nmondal/goscow


text:
Linechart: Dataset labels and highlight colors

Great library, not sure if this has been mentioned before but I noticed two things when adding data to a empty line chart:

Dataset labels are not available in the points themselves, meaning I can't make use of them in the tooltip template. I have to iterate through all points per dataset to add a datasetLabel property.
Highlight colors are not copied to the internal datasets, meaning the highlight rollover stops working when adding data. I have to iterate through all points per dataset to add the highlightFill and highlightStroke properties from the dataset objects I initially used to construct the line chart.
Would be useful if you could reset a bunch of data to a empty chart (0,0) with the points animating from the bottom rather than the side.

@Maxdw Huzzah! The first alpha of Chart.js 2.0 has landed and should fix this issue. Check out the release and try it out! We've got a lot of momentum right now, so please help us test so we can launch 2.0 Gold by the end of the month.
https://github.com/nnnick/Chart.js/releases/tag/v2.0-alpha
I'm closing this issue for now, but if you have implementation questions or find bugs, please create a jsfiddle and post the link here and we'll reopen this issue and get it fixed.

authors: Maxdw derekperkins


text:
[API] Add layer visitor for the model

Dependency of the PR
Commits to be reviewed in this PR
[API] Add layer visitor for the model
This patch add visitor for the model api
Self evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee jhoon.it.lee@samsung.com

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1788. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnstreamer.mooo.com/.

authors: taos-ci zhoonit

repo: nnstreamer/nntrainer


text:
[FIX] modified for checking weight grad

This path checks requested memory is weight gradient which information will be used for planning.

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2264. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

authors: jihochu taos-ci


text:
[activation] Move activation functions to activation file

Move activation operator functions to activation file as static members
Rename sigmoidePrime to sigmoid Prime
Signed-off-by: Parichay Kapoor pk.kapoor@samsung.com

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #339. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnsuite.mooo.com/nntrainer/ci/repo-workers/pr-checker/339-202007201656420.02003002166748-73be585d5e9e51aec074ab2025bc09a8efd726af/.

authors: kparichay taos-ci


text:
Include asymmetric parametric wind capability

Include asymmetric parametric wind capability

@pvelissariou1
Please update this issue following our meeting today.

[ ] validation test case for ADC+PAHM through nupoc caps for Sandy

[ ] symmetric and asymmetric runs

[ ] validation maxelev against HWM

[ ] time series against COOPS stations

Asymetric vortex implementation completed.

repo: noaa-ocs-modeling/PaHM


text:
Debugger window cuts off top (debugger toolbar and first few lines not visible)

I just restarted my machine because of an update and suddenly the debugger window will not display the debug tools (pause, step over etc.) and the top of my source file is not visible even when scrolled to the top. Please see attachment. My guess is that the update to chrome 44 caused this.
node version: 0.10.39
node-inspector version: 0.11.1
Mac OS: 10.10.4 (Yosemite)
Chrome version: 44.0.2403.89 (64-bit)

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/24787092-debugger-window-cuts-off-top-debugger-toolbar-and-first-few-lines-not-visible?utm_campaign=plugin&utm_content=tracker%2F195817&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F195817&utm_medium=issues&utm_source=github).

I was able to fix this issue by clearing the local storage.

OK, this is still an issue. After debugging for a while it comes back. It seems to happen when I load a larger source file in the source view.

I think I see what is going on. When the info in the right pane (Call stack, Breakpoints, etc) is expanded and there is a lot of stack frames for example or breakpoints, it is pushing the hight of the overall frame above the viewport. There are no scrollbars so the only way to see tools is to collapse the pane with the long data. This is why clearing by localstorage helped, as in that particular case it cleared by breakpoints which were numerous.

I also have this problem!
There are any fix  ?!

I'm having a similar problem. I never had this problem before. I assume there is something wrong with scrolling and scrollbars.

I encountered a similar issue as well. In my case, I have a lot of files in the project that I run the debugger on, so the left pane ends up with a lot of nodes at various levels. When this happens, the only way to access nodes beyond the extent of the page is to arrow down the list with the keyboard, but then this shifts everything else, as well. This issue did just appear, so I do imagine it has to do with some change in Chrome 44.

I have the same issue with 0.11.2 as well as 0.11.1. Also at some point of loading (node --debug-brk myapp.js) the garbage collector is busy for 90 seconds hitting ~97% cpu, which for that time it's not useable and I must wait. I downgraded to 0.11.0 and encountered no issues on Win7 with node 0.12.7.

Same problem here with scrolling (and also the starting point that @chk- mentioned).

This is caused by a breaking change in Chrome 44 and flex display.  We found this at our company since it affected our app as well.  The relevant Chrome issue is filed here: https://code.google.com/p/chromium/issues/detail?id=502422  but they seem reluctant to fix it as apparently it matches FF behaviour.
The fix for us was to alter the CSS and add min-width: 0 and min-height: 0 to every container element that had display: flex.  I tried this on the HTML of the debugger window and it fixed the problem.

So should node inspector add the CSS fix? It sounds like the safer option. Right now, it's almost impossible to use node inspector.

@juuxstar thanks! Applying those styles to the main div (first and only child of ) really helped, not sure if the layout issues are completely gone, but at least node-inspector isn't impossible to use now.

+1 Thanks @juuxstar @tlancina The fix works, I've made a pull request with it.

authors: CodeFred akras14 asgerhallas bigeasy chk- ghendricks juuxstar mnaughto roysG tlancina

repo: node-inspector/node-inspector


text:
v5.1AuthorizationOptions.scopeBreaking Change

related: #305, #306
Version 5.1 changed the accepted type of AuthorizationOptions.scope from string[] to string, which is a breaking change.

introduced in #267
The only sensible approach is to accept both strings and arrays of strings for this argument.

Resolved by #309

Will publish this with a new minor release, once we got #314 merged.

authors: Lordfirespeed dhensby jankapunkt


text:
Add message catalog for Japanese translation

I added the Japanese message catalog to the swagger node. It can also solve the following error when Node-RED starts in the Japanese browser environment.
12 Nov 16:31:19 - [info] フローを開始します
12 Nov 16:31:19 - [info] フローを開始しました
12 Nov 16:31:19 - [info] サーバは http://127.0.0.1:1880/ で実行中です
Error: ENOENT: no such file or directory, stat '/Users/yokoi/.node-red/node_modules/node-red-node-swagger/swagger/locales/ja/swagger.json'

@dceejay Thank you!

repo: node-red/node-red-node-swagger


text:
For long running node installs show a message

Current Behavior
If a node install takes a long time the UI just returns to the list of nodes in the catalogue and allows the currently installing node button to be pressed again.
Expected Behavior
The UI should show a message saying the install will continue in the background at the the node will be added to a palette once complete.
Installing other (or attempting to install the same node again) nodes should be blocked until the first completes
Steps To Reproduce
On a slow machine (limited memory and CPU resources) install a large node e.g openapi-red and wait
Example flow
NA
Environment

Node-RED version: 3.1.7
Node.js version: 18.x
npm version: 9.x
Platform/OS: All
Browser: NA

Should this move to using the editor messaging API rather than wait on a long HTTP request to return (or time out in this case?
It already updates the UI when/if the install completes cleanly, just not if it fails.

OK, so the timeout is actually coming from the reverse proxy, Standalone Node-RED doesn't timeout it waits forever.
Question now is do we extend the reverse proxy timeout or still try and handle this better in Node-RED?

repo: node-red/node-red


text:
JavaScript editor using Chrom browser on Mac OS has issue for editing.

One of the user of Node-RED reported that JavaScript editor ( function node ) is not usable. There is a problem in cursor position. For example, put the mouse cursor among the code, and then try to edit goes to different place.
Env: Mac OS
Browser: Chrom
If you need additional details, I can ask the developer to report the detail.

Function node editor seems to work fine for me on latest OSX / Chrome / Node-Red.

Please provide lots more information, such as steps to recreate, versions of node-red, chrome etc.
I develop using chrome on osx and have never seen anything like this.

@ibmamnt,
any more info on this ? or can we close ?

Oh, sorry. I asked the user to input. But looks like he did not respond to write error. OK to close for now.

My friend and I meet the same problem on Ubuntu system, I also try with firefox, but seems useless.
Chrom Version: 51.0.2704.79 Ubuntu 15.10 (64-bit)
node-red version: 0.17.3

I have the same problem. Chrome and Firefox on Arch-Linux. I suspect something with the ACE Editor not using a monospaced font like here: https://stackoverflow.com/questions/15183031/ace-editor-cursor-behaves-incorrectly.

I found the culprit: It is a problem with monospaced fonts. However if I change the css to just font-family: monospace it does not work. The original line is: Menlo, Consolas, 'DejaVu Sans Mono', Courier, monospace !important and if I remove Menlo,  it works. Maybe there is a non-monospaced font named Menlo on Linux? If I use monospace in Inkscape it does not look monospace, so maybe my font configuration is messed up.

good debugging. thanks

authors: PearceDuan dceejay ibmamnt knolleary shemethy t-oster


text:
Fix down arrow triggering menu in search box

fixes #3491

[x] Bugfix (non-breaking change which fixes an issue)

Proposed changes
Only show the drop down in the search menu if the search term is blank
Fixes the issue where after you have typed something in the action bar search field & you press down to select a found item, the dropdown menu is shown!
NOTE: this is targeted to dev because the addition of the dropdown menu in the action bar search field is currently in dev branch.
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Coverage decreased (-0.04%) to 68.721% when pulling 5e197713ffcb61420965e4937fef06383effd52b on Steve-Mcl:fix-down-arrow-in-search into cb88409102e9ecf4ff7521579f1b77cf0cbf8774 on node-red:dev.

authors: Steve-Mcl coveralls


text:
Using navigator.usb in the browser

Hi there!
With node-usb v2 now supporting the WebUSB API, would it maybe be possible to have the library use navigator.usb when available? I would love to be able to use the same code in Node.js, Electron and the browser, and at the moment I'm having to use what feels like a hacky workaround:
let isBrowser, usb = null;

if (typeof navigator !== 'undefined') {
  const userAgent = navigator.userAgent.toLowerCase();
  isBrowser = userAgent.indexOf(' electron/') === -1 && typeof window !== 'undefined';
} else {
  // Node.js process
  isBrowser = false;
}

if (!isBrowser) {
  // For Node.js and Electron
  const { webusb } = await import('usb');
  usb = webusb;
} else {
  usb = navigator.usb; // Yay, we're using WebUSB!
}

Instead, I would just like to do an import { webusb } from 'usb'; and have it use navigator.usb in the browser and node-usb in Node.js and Electron.

Hmm, interesting suggestion.
One drawback would be the user doesn't have control over which implementation is returned (but TBF, I can't think of an environment where both would be available).

Something like #467?

Yes, that looks like it will work just fine!

authors: gniezen thegecko

repo: node-usb/node-usb


text:
Out of bounds memory access

cjs-module-lexer has a stack depth of 2048:
https://github.com/nodejs/cjs-module-lexer/blob/dd9d9f9d9546e2b000d02f1aa038900f170cd438/src/lexer.c#L8
this creates two stacks:
https://github.com/nodejs/cjs-module-lexer/blob/dd9d9f9d9546e2b000d02f1aa038900f170cd438/src/lexer.c#L36-L37
However, when pushing to these stacks, there is no bounds check performed. For example:
https://github.com/nodejs/cjs-module-lexer/blob/dd9d9f9d9546e2b000d02f1aa038900f170cd438/src/lexer.c#L164
If you run with a maliciously crafted input, you can overflow the stack and cause memory corruption.
import { parse, init } from 'cjs-module-lexer';

await init();
console.log(parse('exports.foo = 2;\n' + '{'.repeat(3069) + '}'.repeat(3069)));

Running this causes a Bus error: 10 crash.
This can also cause a crash of node itself by writing the above contents to a file and importing it via ESM.

Thanks for digging into this. The expectation for protection during development was always that it would only crash the Wasm process and therefore we'd wrap that in a JS error and simply catch it. Wasm errors causing process errors is not something I expected though, in which case we need to introduce these protections carefully.
We can PR the stack checks in the right places and backport across all active release lines as it's a patch. Would be nice to ensure we catch everything at once if possible, but we can also do it gradually. I'll try and put a Sunday afternoon into this soon when I can. PRs very welcome too.

authors: devongovett guybedford

repo: nodejs/cjs-module-lexer


text:
Add a Gitter chat badge to README.md

nodejs/docs now has a Chat Room on Gitter
@bengl has just created a chat room. You can visit it here: https://gitter.im/nodejs/docs.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

LGTM, but I've never really used gitter before. At the worst, this change is super easy to revert. So, let's try it out! :smile:

LGTM. :smile_cat:

authors: Qard drewfish gitter-badger

repo: nodejs/docs


text:
chore(ci): add "publish" GHA

Todo:

[ ] Set up repo secret for codemod
[x] Each recipe being published needs to be bundled and then sent to the codemod registry. Thereafter, the bundle generated by CI can be discarded.

npm i should fix the CI

There are a few issues blocking this:

bundling via esbuild encounters a platform-specific node addon used by ast-grep, which can't be bundled

codemod publish doesn't have a programmatic API, so it'll need leverage spawn

codemod publish can handle the bundling but doesn't support distributed config, and the bundle it builds errors (transformer cannot be null)

there's not much documentation around what it's doing (they're working on it)

the bundle it builds errors (transformer cannot be null)

This turned out to be a red-herring: it was actually throwing when it got to module.register('@nodejs-loaders', import.meta.url) because when codemod publish builds the bundle (also via esbuild), esbuild does not realise there is a dependency it needs to include. https://github.com/evanw/esbuild/issues/4016

authors: AugustinMauroy JakobJingleheimer

repo: nodejs/userland-migrations


text:
Test Issue created at Thu, 18 Nov 2021 10:40:02 GMT

Test issue body  Thu, 18 Nov 2021 10:40:02 GMT

Comment on issue at Thu, 18 Nov 2021 10:40:06 GMT

repo: nodemationqa/nodeQA