text:
Clipboard updates

Fixes #2763
This PR contains three commits

The first of these adds more checking to a Format Data Response PDU. In particular, if the client responds with CB_RESPONSE_FAIL for any requested format, the X11 client is informed that the requested selection is not available, rather than the message being ignored. Additionally, pasted text is now given its own function, and more error checking is added.
The second of these only advertises text to the X11 clients when a text format (or a file list) is advertised by the Windows client.
The third of these is a tidy-up. There's a lot more that can be done in this area, but I've stuck with the minimum so that a backport to v0.9.x is feasible.

With these commits, when a picture is selected on the Windows client, the following can be seen using xclip:-
$ xclip -o -selection clipboard -t TARGETS
TARGETS
TIMESTAMP
MULTIPLE
image/bmp
$ xclip -o -selection clipboard -t STRING
Error: target STRING not available

Thanks for the fix, I'll test. I think backport to v0.9 is necessary.

The patches I've added cherry-pick nicely on to v0.9, which is one reason I didn't make more changes in this area! I'll do some more tidy-ups on devel only.
Since the cherry-pick was so trivial, I've added #2767 against v0.9. Test whichever one works best for you and I'll apply any fixes to both.

LGTM.
% xclip -o -selection clipboard -t TARGETS
TARGETS
TIMESTAMP
MULTIPLE
image/bmp

% xclip -o -selection clipboard -t STRING
xclip: Error: window id 0x1c00001 cannot convert CLIPBOARD selection to target 'STRING'


text:
Channels improvements

Check for empty channel names, they are unusable. Jump Desktop client send empty channel names for disabled channels. That made xrdp think we got more channels then really negotiated via  Client MCS Channel Join Request PDUs. That fix makes Jump Desktop client work with xrdp (should resolve #594)

Respect xrdp.ini channel blocking settings. The restriction was only applied on client to server channel data.  block PDUs coming from chansrv if channel is not allowed. This change can help with Parallels Client, which doesn't support our rdpsnd channel (#608). One can now block rdpsnd channel in xrdp.ini so Parallels Client will not drop the connection.

+1 awesome, I like the second fix more than the first.
@proski only allow channel one way won't work for clipboard issue as acks are used in protocol.

I  also think the first change is not the long term solution. I think a. better solution will require to improve the xrdp <-> chansrv protocol. We should initialize channels by handling channel join pdus.

source: gharchive/pull-request

repo: neutrinolabs/xrdp

license_type: permissive

license_source: bigquery


text:
Script hangs at "Selecting best server based on latency..."

Hi,
I am running this on a OpenWRT box. The script many times hangs (i.e. doesn't complete even after 5 minutes and I cancel it).
./speedtest
Retrieving speedtest.net configuration...
Retrieving speedtest.net server list...
Testing from North East Dataa Network (xx.xx.xx.xx)...
Selecting best server based on latency...

I found that one of the 10 servers was not responding during latency test so script was waiting indefinitely. When I ran the loop only for 5 servers , the speedtest-lite completes.
Is there a way to add a timeout if a server doesn't respond?

source: gharchive/issue

repo: neutronth/speedtest-lite


text:
NVSHAS-8284 Warning status for some Compliance checks which however were satisfied

Summary

Improve the warning messages on docker bench

I have reviewed Hank's changes which mainly added the missing warning messages. It should be good to merge the code.

authors: lsongsuse pohanhuangtw

repo: neuvector/neuvector

license_source: github-api


text:
Match magnolify-tensorflow behavior w.r.t. nested types

Magnolify flattens nested types as "field_a.field_b". Need to think about how we handle this here.

More specifically one can write nested case class as Parquet file with parquet-types and might want to read them as TF Example which doesn't support nesting.
Alternatively we can ask users to always do case class -> Example using magnolify-tensorflow and then write flattened parquet-tensorflow instead.

repo: nevillelyh/parquet-extra


text:
About forking, source code history and AUR pkgbuild

Hello,  I have seen this from the new AUR pkgbuilds

This IS a fork, and is reflected as such in ./LICENSE

https://github.com/newm-next/newm-next/commit/e951c326bd72fa4fbe9e161533745abb21dee2f8

When making a fork please use Import a repository and preserve the previous development history instead of creating e new repository and squashing everything into a single commit, your repo is only 7 commits so it is easy to restart again from a proper fork
The pkgbuild have some faults; I could help improving it when the above is fixed

Hello, you are right I kinda did a dirty fork.  I'll fix this ASAP.
Probably not as clean as needed, but the previous git history will be there.
That would be awesome to have a proper PKGBUILD. It's my first attempt and it was based on the previous one "newm-atha-git".

repo: newm-next/newm-next

license: MIT


text:
[Snyk] Security upgrade ubuntu from latest to kinetic

This PR was automatically created by Snyk using the credentials of a real user.Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Changes included in this PR

testdata/acceptance/deb.meta.dockerfile

We recommend upgrading to ubuntu:kinetic, as this image has only 10 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Some of the most important vulnerabilities in your base image include:

Severity
Priority Score / 1000
Issue
Exploit Maturity

371
Improper Validation of Integrity Check Value SNYK-UBUNTU2204-SYSTEMD-5721324
No Known Exploit

371
Improper Validation of Integrity Check Value SNYK-UBUNTU2204-SYSTEMD-5721324
No Known Exploit

371
Improper Validation of Integrity Check Value SNYK-UBUNTU2204-SYSTEMD-5721338
No Known Exploit

371
Improper Validation of Integrity Check Value SNYK-UBUNTU2204-SYSTEMD-5721338
No Known Exploit

371
Improper Validation of Integrity Check Value SNYK-UBUNTU2204-SYSTEMD-5721345
No Known Exploit

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

Codecov Report

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

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff            @@
##             master      #16   +/-   ##
=========================================
  Coverage          ?   70.24%
=========================================
  Files             ?        9
  Lines             ?     1109
  Branches          ?        0
=========================================
  Hits              ?      779
  Misses            ?      204
  Partials          ?      126

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

authors: codecov-commenter nr-security-github

repo: newrelic-forks/nfpm


text:
Emit performance data

Emit performance data (see also here) for consumption by Nagios pnp4graph. Changes the following normal output:
$ ./check_docker -base-url="http://localhost:1234"
OK: 1 containers

to this:
$ ./check_docker -base-url="http://localhost:1234"
OK: 1 containers | 'containers'=1;;;;

Requires go_nagios support https://github.com/newrelic/go_nagios/pull/1

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Rob Chekaluk seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Rob Chekaluk seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

authors: CLAassistant claassistantio rchekaluk


text:
Stuck in cert manager command completion while trying to install kubernets agent opeartor for AWS EKS fargate container

I am trying to install the Kubernetes agent operator in the AWS EKS cluster with the fargate container.
I am following this article to install the operator.
I want to integrate New Relic to collect APM and AWS EKS fargate container logs in New Relic.
The below command execution is getting stuck and the job runs for 5 min and then gets terminated with a timeout error.
helm install cert-manager jetstack/cert-manager \
    --namespace cert-manager \
    --create-namespace \
    --set crds.enabled=true

The same command works for the minikube Kubernetes cluster in the local machine.
Please let me know if I am missing something to make the cert-manager job completed.

@AbhiBhalani11 I'm sorry to hear you're having trouble. This isn't an issue with the New Relic Node Agent, so I've transferred it to the k8s-agents-operator repo.

Can anyone have a solution to this issue? I am stuck in the production setup.

Hi @AbhiBhalani11
We currently don't officially support AWS EKS Fargate for our K8s Agent operator.
Although we're very close to release a new version that don't require cert-manager that may solve your needs.
cc @dbudziwojskiNR @ramkrishankumarN

How close is close? Just wondering if I should wait it out to deploy operator across the fleet or wait for the non-cert-manager dependent version.

Hi @brenwhyte! We've actually already released support for non-cert-manager install starting in v0.13.0. This said, I would recommend installing v0.14.0 as it has support for annotation-less install :)

The timing couldn't have been more perfect, checking this out now 👍

Hey o/
I'm using the nri-bundle chart, how can I speficy v0.14.0 version of the k8s-agents-operator subchart?
I don't think I can.
Using Flux to deploy the charts

Hi @brenwhyte! We rollout releases over the course of a couple days. The nri-bundle chart released yesterday contains v0.14.0 of the operator. Please let me know if anything else comes up!

Hi @brenwhyte! Just wanted to check in if you were able to install the operator? Cheers!

authors: AbhiBhalani11 brenwhyte dbudziwojskiNR mangulonr mrickard


text:
feat(synthetics): addition of mutations for the monitor downtime feature

TBD

Codecov Report
Attention: 240 lines in your changes are missing coverage. Please review.

Comparison is base (5f87eef) 39.08% compared to head (e069c99) 37.33%.

Files
Patch %
Lines

pkg/synthetics/synthetics_api.go
0.00%
152 Missing :warning:

pkg/synthetics/synthetics_api_monitor_downtime.go
0.00%
88 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1069      +/-   ##
==========================================
- Coverage   39.08%   37.33%   -1.75%
==========================================
  Files          82       83       +1
  Lines        5133     5373     +240
==========================================
  Hits         2006     2006
- Misses       2977     3217     +240
  Partials      150      150

Flag
Coverage Δ

unit
37.33% <0.00%> (-1.75%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Reopening via #1070 to view code coverage accurately

authors: codecov-commenter pranav-new-relic

repo: newrelic/newrelic-client-go


text:
chore: update to readme

this PR updates the readme and fixing some MD linter errors.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

authors: CLAassistant jpvajda


text:
README is out of date

https://github.com/newrelic/newrelic-lambda-extension/blob/main/examples/terraform/nodejs/README.md is out of date
[NOTE]: # ( ^^ Provide a general summary of the issue in the title above. ^^ )
Description
While trying to implement the example found in https://github.com/newrelic/newrelic-lambda-extension/blob/main/examples/terraform/nodejs I noticed that the instructions are out of date.

The instructions refer to running newrelic-lambda install, but I believe the correct command is newrelic-lambda integrations install and it does not look like --enable-license-key-secret flag is still valid.
Inside main.tf

data "aws_iam_policy" "newrelic_license_key_policy" {
  arn = "arn:aws:iam::${data.aws_caller_identity.current.account_id}:policy/ViewNewRelicLicenseKey"
}

throws an error since the policy created by the step in #1 has a different name.
There may be other problems and it would be great to get them fixed.

This issue was fixed here

authors: chaudharysaket rmclaughlin-nelnet

repo: newrelic/newrelic-lambda-extension


text:
Upgraded to Django 4 and other related modules also

Upgraded to Django 4.0.3 and other modules to the latest version.
Signed-off-by: Pratyush1606 jaiswalpratyush2015@gmail.com

@pombredanne Can you please review this pr?

@pombredanne I have merged it. Can you please approve running workflows?

@pombredanne  @tdruez Some tests are failing because Django 4.0+ requires Python 3.8+ and those tests are having Python versions below that. Can you tell me how to proceed further?

@tdruez So should I start working on reviewing those security details or move to other issues for right now?

@Pratyush1606 I would not consider the upgrade complete without addressing those points.

@tdruez I will look into those points and will make changes.

Upgraded to Django 4.0.3 and other modules to the latest version.
Changes in Django 4.0 releases:

Python compatibility
Django 4.0 supports Python 3.8, 3.9, and 3.10. I have removed Python 3.6 and Python 3.7 from setup.cfg and updated CI

zoneinfo default timezone implementation
The Python standard library’s zoneinfo is now the default timezone implementation in Django. I have removed pytz from the requirements. zoneinfo is part of the Python standard library from Python 3.9. The backports.zoneinfo package is automatically installed alongside Django if you are using Python 3.8.
The present version of the codebase is using the timezone imported from django.utils for getting the present time using timezone.now() which works fine as previous in this newer version. Reference

Minor Features

https://docs.djangoproject.com/en/4.0/releases/4.0/#csrf
The changes mentioned in this feature are related to CSRF_TRUSTED_ORIGINS settings which our codebase is not concerned with.
https://docs.djangoproject.com/en/4.0/releases/4.0/#requests-and-responses
The changes mentioned in this feature are related to SECURE_CROSS_ORIGIN_OPENER_POLICY settings which also doesn't affect any change in our dependencies.
https://docs.djangoproject.com/en/4.0/releases/4.0/#csrf-trusted-origins-changes and https://docs.djangoproject.com/en/4.0/releases/4.0/#configuring-it-may-now-be-required
For the changes in CSRF_TRUSTED_ORIGINS settings, no changes are we are including csrf_token in every form requests like of POST methods.
https://docs.djangoproject.com/en/4.0/releases/4.0/#securitymiddleware-no-longer-sets-the-x-xss-protection-header
No changes are required.

Resolves #384
Signed-off-by: Pratyush1606 jaiswalpratyush2015@gmail.com

@tdruez I went through the release of Django 4.0 and compared and analysed the changes. I have made the changes. I have also updated the PR description according to that only. Can you please have a look?

@Pratyush1606 thanks for your contribution. See https://github.com/nexB/scancode.io/pull/418 for some extra changes and the merge into main branch.
Cheers 👍


text:
Update nix flake

Not yet functional:
warning: Git tree '/home/aengelen/dev/vulnerablecode' is dirty
error: builder for '/nix/store/mcyd7cfny2p4rzlfax3kwypn5a03ihv2-mach_nix_file.drv' failed with exit code 1;
       last 10 log lines:
       > Mach-nix version: master
       > Python: 3.8.5
       > Cause: Requirements conflict: cryptography (<SpecifierSet('==36.0.2')>,)
       > The requirements which caused the error:
       >   cryptography (<SpecifierSet('==36.0.2')>,)
       >
       > The given requirements might contain package versions which are not yet part of the dependency DB
       > currently used. The DB can be updated by specifying 'pypiDataRev' when importing mach-nix.
       > For examples see: https://github.com/DavHau/mach-nix/blob/master/examples.md
       >
       For full logs, run 'nix log /nix/store/mcyd7cfny2p4rzlfax3kwypn5a03ihv2-mach_nix_file.drv'.
(use '--show-trace' to show detailed location information)

Thanks!

@raboof thanks for your contribution, please add your sign-off DCO is failing.

Hey - it was great to meet in Brussels!
Unfortunately it looks like this PR is still not quite functional. I think we should probably abandon this PR and replace mach-nix with 'manual' nix packaging or dream2nix.

authors: TG1999 pombredanne raboof

repo: nexB/vulnerablecode


text:
Functional Module

Functional Module for Project;

User Authentication:- The registration and Login system has been added so that only authenticated users (Admin, Doctor, Patient) can perform task.

Doctor Module: Add Doctor, View Doctors, Search Doctors

Appointment Module: Add Appointment, View Appointments, Treatment Appointment

Patient Module: Add Patient, View Patients

This is done. I created this bcz it was important.

repo: next-demo/doctor-appointment-service


text:
Add FormattedDatetimeConverter

Link
https://github.com/Rapptz/discord.py/pull/7312

unsure ab this one, maybe a util maybe not
discord still not having a proper ui to make a timestamp doesnt make this very user friendly

repo: nextcord/nextcord


text:
Render large JSON files in a memory-efficient way

The usual pattern to render a JSON file is to create the equivalent data structure in Groovy code, render it to a JSON string, and write the entire string to a file. For large runs with thousands of tasks, the JSON string could be quite large and cause Nextflow to run out of memory.
First we need to evaluate whether this is actually a real problem. Do some large runs and see how large the resulting prov reports are. If they get into the 100 MB - 1 GB range, then we should probably optimize the rendering code.
The memory-efficient approach is to write the JSON output directly and save it to the file in pieces, so that we never have to allocate the entire report in memory and the memory usage does not increase with the number of tasks / outputs.

rnaseq-nf BCO is ~7 KB
nf-core/rnaseq (-profile test) BCO is ~386 KB

repo: nextflow-io/nf-prov


text:
Implement 'retiring' experiments

This PR replaces #199, which was prematurely merged and then reverted.

[x] Add experiment retired DB flag setting to ResourceManager
[x] Add seperate 'Live' and 'Retired' tabs to /dashboard/experiment_list
[x] Add API endpoint to get and set retired state
[x] Add a way to retire/unretire experiments to the dashboard

This PR adds the following API endpoints:

POST /dashboard/experiment/<exp_uid>/retire, optionally taking the form-data parameter retired:bool (default: True)

It also implements several miscellaneous /dashboard/experiment_list enhancements:

Cleaned up page markup
Rewrote scripts for the dashboard to use idiomatic JS and make use of jQuery's functionality, since we already have it included on every page
Fixed responsiveness on the dashboard - the table now properly collapses and horizontal-scrolls

The interface
'Live' tab

'Retired' tab

Retiring experiments

video (webm)

Tested with SITE_KEY enabled; all the changes in this PR continue to work.

LGTM

authors: liamim stsievert

repo: nextml/NEXT


text:
🛑 Nextpost.tech is down

In 04ca723, Nextpost.tech (https://nextpost.tech) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Nextpost.tech is back up in 2c99bcb after 11 minutes.

repo: nextpost-tech/status


text:
nf-core/modules guidelines missing when: requirement

The current modules guidelines does not appear to mention the required when: statement
https://nf-co.re/developers/adding_modules#new-module-guidelines-and-pr-review-checklist
even though it results in a linting error with nf-core tools when missing.
This should be documented

There is a small part about the when: statement mentioned in the guidelines:
_The process definition MUST NOT change the when statement. when conditions can instead be supplied using the process.ext.when directive in a configuration file.
process {
withName: 'FOO' {
ext.when = !params.skip_module
}
withName: 'BAR' {
ext.when = { meta.single_end }
}
}_


text:
implement sharding of tests

Adapted from https://github.com/nf-core/sarek/pull/1668 by @edmundmiller | @maxulysse

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] If necessary, also make a PR on the nf-core/rnaseq branch on the nf-core/test-datasets repository.
[ ] Make sure your code lints (nf-core pipelines lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

Let's put it aside for now, I think it's the way to go, but let's try to see if https://github.com/nf-core/rnaseq/pull/1437 is not fixing our issues already

repo: nf-core/rnaseq


text:
Create a sample plugin generator

With yeoman

Completed this with https://github.com/nfarina/homebridge/pull/1577.

repo: nfarina/homebridge


text:
*.neoncloud.io domains: register IPs for all masters?

It looks like we're only registering the IP address for the first master for the generated *.neoncloud.io domain.  This means that the cluster will not be reachable when the first master is down.
We should register all of the master IPs with the domain and then check that kubectl and helm will fail-over to another master (I'm betting that they don't).
NOTE: This isn't and issue for cloud clusters because the load balancer distributes traffic to healthy masters.

This doesn't really make any sense.  Users with on-premise clusters should really deploy a load balancer or router/LB to handle this.
In the future, we may add some router integrations to configure this automatically, if there's demand.

repo: nforgeio/neonKUBE


text:
Update demo

It could be helpful to extend the demo and show the sidebar example from [#1] too.
I guess the right thing to do here would be to make a 'master' page that links to demo nr. 1, nr. 2 and so on.

I just sent you a new example file linked from the existing index.html. Be careful before merging because I'm not sure if http://ng-milk.github.io/angular-sticky-navigation-directive/ will work properly OOTB. Check #3 for details.

Awesome that you made the demo.
gh-pages works ok, thanks for the heads up :-)

authors: danmindru llattes

license: mit


text:
Fix path error in rpm build script used by Jenkins

Didn't catch this path error in my first update

Coverage increased (+0.0%) to 63.23% when pulling d96bb8da353548008cb817135de494dad73de9fc on fix-rpm.sh into fa67466ab271c38a3ae552b0c7a62f56bfa3a282 on master.

authors: coveralls spohnan

repo: ngageoint/geowave


text:
[UX] Active chain visual cues

Currently it is unclear if any chain is running and if so, which one unless you look at the TERMINAL tab.
Example of a running chain

The status bar at the bottom says "Neo: Not connected". I have actually no idea what it means or if it is supposed to say something about the chain status. If I open the TERMINAL tab we can see there's actually a chain running

Ideas
Based on the following view

I have the following ideas

Change the play button to a stop button for the chain that is running
Highlight the background color to green or some colour to indicate it's actively running (this is also helpful if for whatever reason the chain dies)
From the context menu hide Start blockchain ... options and only show Stop blockchain when it's running and vice versa.
a. Perhaps even hide all options that cannot be used until the chain is running to keep the context menu clean.
b. Alternatively, disable/grey out options that cannot be used in the current state
I'm a fan of reducing visual clutter so I'd even suggest having a button in the title bar (view/title group:navigation) that allows you to show only the active server.

I added code to the settings.json file which changes the colors on the status bar at the bottom of the VS Code border to green when NEO is connected to the blockchain, white when connecting, and orange to when it is disconnected. We can change the color to whatever we want. Please have a look.

The colouring of this bar can be an extra (if we understand what the real purpose of this bar is), but I think the original idea of changing the icons for the entries in the blockchain tree is more powerful and self-explanatory

Erik, as you can see from my screenshot, no inline icon is showing up in my sidebar view for a 'start' or 'stop' for the chain. I can't figure seem to figure out why. Can you point to me where in the codebase the inline icon resides for this 'start'/'stop' icon for the chain?

Anyways, I do think the colored bar below does a pretty good job already of alerting the user as to whether or not they are connected. The green color is quite eye-catching as opposed to the orange color. Have you been able to try it yet?

The reason you’re not seeing a “play” button is because you’re selecting Neo Mainnet. You can’t start or stop that, neither can you start/stop Neo Testnet. It will therefore only show on local networks.
You wrote “…alerting the user as to whether or not they are connected”.  I don’t even know what “connected” means on this status bar. 2 months ago when i first opened this issue i didn’t know, now i still don’t know. The fact that we have to look at the source code to learn what it actually means already says it is failing from a UX perspective.
Even without defining what “connected” exactly means, the fact that we can run multiple chains simultaneously but this bar can only show the status of 1 chain means it is the wrong place to try and say something about chain status. Having said that i think we should actually completely remove this item from the bottom status bar.

On 30 May 2022, at 15:55, Rob L. @.***> wrote:

Erik, as you can see from my screenshot, no inline icon is showing up in my sidebar view for a 'start' or 'stop' for the chain. I can't figure seem to figure out why. Can you point to me where in the codebase the inline icon resides for this 'start'/'stop' icon for the chain?
Anyways, I do think the colored bar below does a pretty good job already of alerting the user as to whether or not they are connected. The green color is quite eye-catching as opposed to the orange color. Have you been able to try it yet?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

Erik, I think we're having a misunderstanding here of what 'Active chain' means. The start and stop function doesn't imply starting or stopping the chain itself. What it implies is starting or stopping an active connection from the user to the chain itself.
If you are gathering from the UX that pressing the 'play' button from the UX is going to actually start or stop the mainnet or testnet or private chain, then yes, that is indeed a fail from the UX perspective!
I'm not so sure that one would want to connect to multiple chains simultaneously, as that could lead to a lot of confusion, no?

authors: ixje robliou


text:
Feature: What about hints in IDE?

Hello. It would be very convenient to find the desired translation in .json files by ctrl + click on the key in selectTranslate () or in templates.
I'm using Phpstorm and at the moment I have this problem:

Should I expect any plugins for Jetbrains IDEs or other?
Maybe there are some other approaches?
Thanks!!!

In an unrelated project for work, I've solved this concern with straight plain old TypeScript bindings. Here is what the developer sees in Webstorm, and VS Code is rather similar. No plugins, no fuss trying to guess obscure keys. Bringing typesafe i18n makes it easier for developers.

That would be an incredible feature. Maybe autogenerate TypeScript Bindings just as the i18-extract? Don't know if there is another way

FYI i18n-ally is a VSCode extension and supports this (at least in templates).

This is definitely possible with TypeScript, I have seen this within: https://github.com/mozilla/node-convict/tree/master/packages/convict but I dont remember how this was called

authors: MickL NiklasPor alex-kovalchuk mattmutt xjuanc

repo: ngneat/transloco


text:
Add global PersistentKeepalive setting to wg.conf template.

If set, this will insert PersistentKeepalive into the server side config Peer sections. PersistentKeepalive is not permissible under Interface sections. Only Peer sections.
Refer to man page or source:
https://git.zx2c4.com/wireguard-tools/tree/src/config.c
Resolves issue #432.

Tested on b55543f424b4ebffe0080446816fff7b5aa191d1 and verified.

Done in #515

authors: ngoduykhanh systemcrash

repo: ngoduykhanh/wireguard-ui


text:
Looking for a Cascaded multicheckbox

Hello everybody,
Is there is any example of a cascaded multicheckbox similar to the cascaded select
I tried something like this based on the cascaded select but i don't think its the way to do it :
  form = new FormGroup({});
  model: any = {};
  options: FormlyFormOptions = {
    formState: {
      selectOptionsData: {
        teams: [
          { key: '1', value: 'Bayern Munich', sportId: '1' },
          { key: '2', value: 'Real Madrid', sportId: '1' },
          { key: '3', value: 'Cleveland', sportId: '2' },
          { key: '4', value: 'Miami', sportId: '2' },
        ],
        players: [
          { key: '1', value: 'Bayern Munich (Player 1)', teamId: '1' },
          { key: '2', value: 'Bayern Munich (Player 2)', teamId: '1' },
          { key: '3', value: 'Real Madrid (Player 1)', teamId: '2' },
          { key: '4', value: 'Real Madrid (Player 2)', teamId: '2' },
          { key: '5', value: 'Cleveland (Player 1)', teamId: '3' },
          { key: '6', value: 'Cleveland (Player 2)', teamId: '3' },
          { key: '7', value: 'Miami (Player 1)', teamId: '4' },
          { key: '8', value: 'Miami (Player 2)', teamId: '4' },
        ],
      },
    },
  };

  fields: FormlyFieldConfig[] = [
    {
      key: 'sport',
      type: 'multicheckbox',
      templateOptions: {
        label: 'Sport',
        options: [
          { key: '1', value: 'Soccer' },
          { key: '2', value: 'Basketball' },
        ],
        valueProp: 'key',
        labelProp: 'value',
      },
    },
    {
      key: 'team',
      type: 'multicheckbox',
      templateOptions: {
        label: 'Team',
        options: [],
        valueProp: 'key',
        labelProp: 'value',
      },
      expressionProperties: {
        'templateOptions.options': 'formState.selectOptionsData.teams.filter(team => team.sportId === model.sport)',
        // reset model when updating select options
        'model.team': `this.field.templateOptions.options.find(o => o.id === model.team) ? model.team:null`,
      },
    },
    {
      key: 'player',
      type: 'multicheckbox',
      templateOptions: {
        label: 'Player',
        options: [],
        valueProp: 'key',
        labelProp: 'value',
      },
      expressionProperties: {
        'templateOptions.options': 'formState.selectOptionsData.players.filter(player => player.teamId === model.team)',
        // reset model when updating select options
        'model.player': `this.field.templateOptions.options.find(o => o.id === model.player) ? model.player:null`,
      },
    },
  ];

Thank you in advance

almost the same except the expressionProperties check should be adjusted for multicheckbox type as the model can be an array or an object (based on templateOptions.type) so if it's an object then you have to change the find condition into:
{
  key: 'team',
  type: 'select',
  expressionProperties: {
    'templateOptions.options': 'formState.selectOptionsData.teams.filter(team => model.sport && !!model.sport[team.sportId])',
  },
}

here is a simple example: https://stackblitz.com/edit/angular-afbvdz

Thank you for you help and your quick response.

weird, in stackblitz it works but in my project its not working, is there a difference between this code with bootstrap and with material?
I'm using Material and i have an error when i execute this code :
 form = new FormGroup({});
  model: any = {};
  options: FormlyFormOptions = {
    formState: {
      selectOptionsData: {
        teams: [
          { id: '1', name: 'Bayern Munich', sportId: '1' },
          { id: '2', name: 'Real Madrid', sportId: '1' },
          { id: '3', name: 'Cleveland', sportId: '2' },
          { id: '4', name: 'Miami', sportId: '2' },
        ],
      },
    },
  };

  fields: FormlyFieldConfig[] = [
    {
      key: 'sport',
      type: 'multicheckbox',
      templateOptions: {
        label: 'Sport',
        options: [
          { id: '1', name: 'Soccer' },
          { id: '2', name: 'Basketball' },
        ],
        valueProp: 'id',
        labelProp: 'name',
      },
    },
    {
      key: 'team',
      type: 'multicheckbox',
      templateOptions: {
        label: 'Team',
        options: [],
        valueProp: 'id',
        labelProp: 'name',
      },
      expressionProperties: {
        'templateOptions.options': 'formState.selectOptionsData.teams.filter(team => model.sport && !!model.sport[team.sportId])',
        // reset model when updating select options
        'model.team': `this.field.templateOptions.options.find(o => o.id === model.team) ? model.team:null`,
      },
    },

  ];

  submit() {
    alert(JSON.stringify(this.model));
  }

i'm having this error when i execute it
FormlyFieldMultiCheckbox.html:3 ERROR Error: Cannot find control with unspecified name attribute

I think the model.sport doesn't get bind with the value when the sport is checked that why the error occur. i think what missing is to put on the model.sport the value of the checked value only when it's checked but i don't know how to do that with formly

After multiple test i confirm that the problem is not from the code but its from the formlyMaterial who's not supporting the multicheckbox cascaded. he runs very well with the select cascaded

just checked and it seems working https://stackblitz.com/edit/angular-6utyyj
Note:  the reset model should be adjusted
which version of formly do you use ?

i'm using @ngx-formly/core@4.8.2

multicheckbox in V4 doesn't support dynamic options, either update to V5 or use a custom field type https://github.com/ngx-formly/ngx-formly/blob/master/src/material/multicheckbox/src/multicheckbox.type.ts

authors: aitboudad boufenzi

repo: ngx-formly/ngx-formly


text:
JSON-Schema dependencies of boolean type doesn't hide on false

Hi,
I try to set up a form with json-schema with dependencies. I set up checkbox input which on true should reveal another input and it works for the reveal but when I uncheck the input doesn't disappear.
If the
Minimal Reproduction
Here is a fork from your JSON-schema stackblitz with simple setup
https://stackblitz.com/edit/angular-quodaz
Schema is in simple.json file
If it is not a bug could you help me to setup that?

use oneOf: https://stackblitz.com/edit/angular-quodaz-wvabux
note: require v5.5.5

authors: NiestrojMateusz aitboudad


text:
fix: script error when calendar list is not set(refs: #81, #122)

Please check if the PR fulfills these requirements

[x] It's submitted to right branch according to our branching model
[x] It's right issue type on title
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[x] It does not introduce a breaking change or has description for the breaking change

Description
refs: #81, fixes: #122

Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

retest this please


text:
Client APIs name improvements

Chang the naming of the generated client APIs to be "SeviceName MethodName" so it is easier for clients to use.
The containing folder names in the project are also updated.
Naming for streaming services has also been improved as follows:

"Begin ServiceName MethodName"
"End ServiceName MethodName"
"ReadStream ServiceName MethodName"
"WriteStream ServiceName Method"

I am not too happy with the ReadStream and WriteStream names but didn't have a name that I thought was much better.  I am willing to change those before this PR goes in.  I would very much like these names to be settled.

We can consider
"ReadStream ServiceName MethodName" -> "ResponseStreamWrite SerivceName MethodName"
"WriteStream ServiceName MethodName" -> "RequestStreamRead SerivceName MethodName"
If that makes it better, otherwise its ok.

authors: ccifra ni-sujain

repo: ni/grpc-labview


text:
Disable intermittent test

Pull Request
🤨 Rationale
Test is intermittent. See https://ni.visualstudio.com/DevCentral/_workitems/edit/2632606
Related https://github.com/ni/nimble/issues/1761
👩💻 Implementation
Commented out.
🧪 Testing
N/A
✅ Checklist

[x] I have updated the project documentation to reflect my changes or determined no changes are needed.

@m-akinc don't need to wait for @jattasNI to review

authors: m-akinc rajsite

repo: ni/nimble


text:
Update the palettes and scripting APIs to address API review feedback

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
Updates the API and palettes to address API review feedback. This includes significant changes to the palette structure, as well as a number of VI renames.
Not all feedback is addressed in this review, in order to keep the number of changes remotely manageable.
The following feedback has not yet been addressed and is being deferred to a different pull request, as the majority of the VIs in the repository will be affected:

Remove the ref suffix from parameter names
Remove the (no error) suffix from error in parameter names
Use lower case (or at least consistent case) parameter names
Use the VeriStand item ref typedef everywhere, instead of raw U64s

Additionally, the examples will need to be reviewed and updated in a separate pull request.
Why should this Pull Request be merged?
This addresses a large portion of the feedback from the API review process.
What testing has been done?
Ran unit tests, tested the build and palettes.

[ ] Review Routing and Faulting Hardware API and remove unused VIs. I am looking at Module.

[ ]  Review Routing and Faulting Hardware API and rename/remove relevant VIs. Get Model for example.

I was originally thinking we could remove this VI, as I did not think that we were using the model in RaF (since there was no setter in the hardware API that SLSC Switch was using), but it appears that we are using the model in tip strips as of https://github.com/ni/niveristand-routing-and-faulting-custom-device/pull/156. Moreover, the RaF HW API is is reading the model from a property that is not reserved for RaF use (i.e. it is not prefixed with Routing and Faulting.), and so SLSC Switch isn't aware that Routing and Faulting is actually using this property.
I think we should consider adding a setter for the ~model~ product name to the HW API, and properly reserving it if we intend to read from this property in RaF. This probably means mutation in SLSC Switch to change the model property to be properly prefixed, and probably some backwards-compatibility handling in RaF.

authors: csjall rtzoeller


text:
[Feature Request] Grid Layout

I think this could be cool to have a grid layout as it's a layout "often" used in GUI apps.

@JulienLecoq Definitely agree! I think there's certainly a Grid and/or Table layout that would work well with the other features in clay.
For transparency around my priorities at the moment:

Debug Tooling (https://github.com/nicbarker/clay/pull/15) - should ship today or tomorrow
Zig bindings
A "wrapping" container - think something like flex-wrap in CSS. This is pretty important as it blocks the creation of "rich text", clickable links in text paragraphs etc.
Grid or table view

So likely it'll be a month or two before it starts to get looked at 🙂

I wrote an algorithm a few years ago for masonry layouts (grids where each element has fixed width but variable height, like a photo gallery with different aspect ratios ) that I haven't seen anywhere even in CSS: it solves a problem that if you have a particular ordering in mind (like search results, most relevant to least relevant), the algorithm CSS uses places the next object on the shortest column but that sometimes breaks the logical flow (see https://youtu.be/W91Wst6fMNU?t=447)
I don't know C, would it be at all useful to propose the algorithm in pseudocode or python?

authors: Greedquest JulienLecoq nicbarker

repo: nicbarker/clay

license: Zlib


text:
Error in OpenCL.dll

Hi!
I run
nheqminer.exe -t 8 -l europe.equihash-hub.miningpoolhub.com:20570 -u user.worker -p x

on windows 7 system from latest build. But this computer have videocard from IBM. I got an error

the entry point in the procedure could not be found, clReleaseDevice

I copy opencl.dll in miner folder, but this not work.

Copy OpenCL.dll from https://github.com/nicehash/NiceHashMiner/releases/download/1.7.3.11/NiceHashMiner_v1.7.3.11.zip and place it in the same directory as nheqminer.exe
The DLL needs to be included in the release zip

https://github.com/computeronix/nheqminer/blob/master/OpenCL.dll?raw=true

OpenCL solver are dropped from nheqminer. Latest release.
Closing issue...

authors: S74nk0 Slavenin computeronix

repo: nicehash/nheqminer


text:
Remove additional config on coverage

Describe your changes
Type of change
Fix an issue where coverage report isn't generated on codecov
Please delete options that are not relevant.

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

Recordings / Screenshots (if any)

No error bug to fix on codecov side. Closed.


text:
how to use a different user other than getcurrentuser

i have a important scenario.  I see this code is useful when we are impersonating as a logon user from the current user that is logged in..
but i have a different scenario my issue is i have a domain1/user1 ( which is a logon user) i want to change to domain2/user2 ( which is not a current user that is logged in ) how can i change the code or use a different method. any input will be greatly useful.

I assume you are referring to this: https://github.com/nicholasdille/PowerShell/blob/master/Impersonation/Invoke-ImpersonatedCommand.ps1. Correct?

If so, the prerequisite for this is that you are able to log on with user2\domain2 on the same machine as domain1\user1, because the code extends a user's security token.

I assume you are referring to this: https://github.com/nicholasdille/PowerShell/blob/master/Impersonation/Invoke-ImpersonatedCommand.ps1. Correct?

I assume you are referring to this: https://github.com/nicholasdille/PowerShell/blob/master/Impersonation/Invoke-ImpersonatedCommand.ps1. Correct?

I looked at this script..
https://github.com/nicholasdille/PowerShell/blob/master/Impersonation/Test_ImpersonationContext.ps1
in the script that you are referencing above you are using following lines. GetCurrent() is automatically taking the logged in user. i want to change to a different user without passing password.
like domain2/user2 ... can i do that ???
$identityName = [System.Security.Principal.WindowsIdentity]::GetCurrent().Name
Write-Host "Current Identity: $identityName"
$newIdentity = New-Object System.Security.Principal.WindowsIdentity( $tokenHandle )
$context = $newIdentity.Impersonate()

$identityName = [System.Security.Principal.WindowsIdentity]::GetCurrent().Name
Write-Host "Impersonating: $identityName"

authors: kriskoneru nicholasdille

repo: nicholasdille/PowerShell


text:
App Shortcut Icon creation

Hi. I am implementing app shortcuts in one of my project, however, I am having trouble generating those icons' vector drawables. I want a music note (#FF0000) inside the circle (#F5F5F5).
I don't know how to proceed and put the music note inside circle with padding all around it. The icon I am creating have the note at the top-left corner. I don't know what path should I write to draw it in the centre.
Kindly help me out.
<vector
xmlns:android="http://schemas.android.com/apk/res/android"
android:width="48dp"
android:height="48dp"
android:viewportWidth="48"
android:viewportHeight="48">

<path android:fillColor="#f5f5f5"
    android:pathData="M24,24m-22,0a22,22 0,1 1,44 0a22,22 0,1 1,-44 0" />

<path android:fillColor="#FF0000" android:pathData="M12,3V12.26C11.5,12.09 11,12 10.5,12C8,12 6,14 6,16.5C6,19 8,21 10.5,21C13,21 15,19 15,16.5V6H19V3H12Z" />

</vector>

Here you go (I created it in Sketch, exported as SVG and then imported into Android Studio):
<vector
    xmlns:android="http://schemas.android.com/apk/res/android"
    android:width="48dp"
    android:height="48dp"
    android:viewportWidth="48"
    android:viewportHeight="48">
  <path
      android:pathData="M24,24m-22,0a22,22 0,1 1,44 0a22,22 0,1 1,-44 0"
      android:fillColor="#F5F5F5"/>
  <path
      android:pathData="M24,14L24,24.55C23.41,24.21 22.73,24 22,24C19.79,24 18,25.79 18,28C18,30.21 19.79,32 22,32C24.21,32 26,30.21 26,28L26,18L30,18L30,14L24,14Z"
      android:fillColor="#FF0000"/>
</vector>

Thank you very much

authors: nickbutcher niteshvuliv

repo: nickbutcher/plaid


text:
Complete the proxy support

Re #294 which adds proxy support for the async version of telepot.
Use the default route as a fallback for an invalid proxy.

Anybody knows when pull requests about proxy issues will be integrated in the API and published?

@das7pad, thanks for the work. Can you promise that you will ensure the correct working of proxy support for both the traditional (urllib3) and async (aiohttp) version? I can check the logic, but I don't want to take on the responsibility of testing them. I appreciate anyone's help in that regard.
If I can find someone to take on that responsibility, proxy support will be merged and added in the upcoming release, which will also include support for Bot API 3.4. Regardless of proxy support, expect the upcoming release on or before Oct 25.

@yakumoklesk, after some considerations, I have decided the existing methods of using proxy is too ad-hoc. If we want to give both traditional and async version proxy support, at least I should make it consistent. I have done that in the newest version 12.4. Please refer to #309 for usage.

authors: das7pad nickoala yakumoklesk

repo: nickoala/telepot


text:
Review update on cables

Valid from and valid to times should reflect changes made.

solved in the last 4 commits, as of this one 71aaddbfb795201b1950cff191ec1471c605e27e

repo: niclabs/net-viz


text:
Missing license

Please add a license to this project, consider MIT
https://choosealicense.com/licenses/mit/

Added the MIT license

authors: AThraen nilsree

repo: nicolaayan/EpiserverCmsAudit


text:
Rendering Problem

Hello,
I've copy pasted the example code from the the README.md.
It is working actually but the problem is the size of the generated image.
It looks like this:

but actually it should look like this:

I don't know what is wrong. The code is a simple copy paste from the examples and looks like this:
        Graph g = graph("example1").directed()
                .graphAttr().with(RankDir.LEFT_TO_RIGHT)
                .with(
                        node("a").with(Color.RED).link(node("b")),
                        node("b").link(to(node("c")).with(Style.DASHED))
                );
        try {
            Graphviz.fromGraph(g).height(100).render(Format.PNG).toFile(new File("example/ex1.png"));
        } catch (IOException e) {
            e.printStackTrace();
        }

This bug appears with the latest versions. After rolled back to v0.8.0, it's working properly.

I changed something in 0.8.4 regarding the image size.
But I cannot reproduce this issue neither with 0.8.0 nor with 0.8.10. The resulting image has always 100px height.
Maybe it's the graphviz version you have on your machine, can you try adding Graphviz.useEngine(new GraphvizV8Engine()) at the beginning?

I found an issue that could cause your problem. It is fixed in version 0.11.0. Please check it out.

hi,
how can i do to start and configure the Graphviz.useEngine () method?

Just call Graphviz.useEngine(new GraphvizV8Engine()) at the beginning of your program.

Additionally in version 0.11.1, the image sizing is improved and maybe thus the issue is solved?

Closing this. Feel free to reopen if the problem is still here.


text:
HelpDown: съедается пробел если строка в списке начинается с линка

# Contents

a. text
   [link](https:\\)

@Contents
$ #Contents#

   1. §¦text~link~@https:\@

Ожидаемое поведение: пробел между text и link.

Cannot reproduce, I have the space
.Language=English,English
.Options CtrlStartPosChar=§¦

@Contents
$ #Contents#

 a. text ~link~@https:\@

The intermediate HTML

Contents

a. text link

У вас почему-то не определился список.

md
# Contents

- text
   [link](https:\\)

html

Contents

text link

hlf
.Language=English,English
.Options CtrlStartPosChar=§¦

@Contents
$ #Contents#

   • §¦text ~link~@https:\@

pandoc --wrap=preserve
Intermediate html:

Contents

text
link

Проверьте v1.3.16, должно работать, но у меня это редкий юз-кейс, может, что пропустил.

Спасибо!

repo: nightroman/FarNet


text:
Don't allow entry when entry is already at maximumFractionDigits

This makes it so there isn't any weird rounding when entering more decimal places than allowed. Now it just prevents entering more decimal places once the maximum has been entered.

Summary
This fixes the "weird rounding"
Configuration

Build Trio dev commit c870190d to SE phone running iOS 17.5.1
Find a "weird rounding" example.
Then apply the patch and test.

Tests
Example:
Type a value with multiple decimal places into

bolus entry with straight dev:

type 1.555555 and it provides: 1.56, 1.57, 1.58, 1.59, 1.6 etc

bolus entry with patch

type 1.555555 and it provides: 1.55 and that's it

Merging with two approvals.

authors: MikePlante1 bjornoleh marionbarker

repo: nightscout/Trio


text:
Fix code scanning alert - Inefficient regular expression

Tracking issue for:

[ ] https://github.com/nihilor/flgs/security/code-scanning/1

Regular expression changed.

repo: nihilor/flgs


text:
Apply the header to the documentation site

The documentation site (currently hosted at https://ds-nihr.vercel.app/) does not show the DS's own NIHR-branded header.
Definition of Done

Apply the DS's own NIHR-branded header component to the documentation site

What does DS's "own NIHR-branded" look like?

NIHR logo usage awaits resolving license usage.

repo: nihruk/design-system


text:
Any plans to implement more scaling options/target resolutions?

waifu2x-caffe seems to support some robust resolution output options as seen here, whereas vulkan only supports 2x. Just wondering. Thanks.

i'm very agree with it. it's can't get qulity with normal method
when you want to stretch image, like 4:3 to 16:9

authors: Tenome aaaboypop


text:
1.8.3: can't retrieve Letsencrypt certs

It seems to be VestaCP issue, but may be not:
At the start of container i have:
[i] running /bin/vesta-auto-ssl.sh
[i] Creating cert for 'admin' domain 'mydomain.nl'
Error: Let's Encrypt new auth status 429
[err] cert not found '/home/admin/conf/web/ssl.mydomain.nl.pem'

It works well in vestacp:latest image.
Also, I'cant get certs for any domain I added:
Error: Let's Encrypt validation status 400

Sorry, it is my fault:
{
"type": "urn:ietf:params:acme:error:rateLimited",
"detail": "Error creating new order :: too many certificates already issued for exact set of domains: mydomain.nl: see https://letsencrypt.org/docs/rate-limits/",
"status": 429
}

repo: niiknow/vestacp


text:
Is there an official training code available now?

Thank you for your excellent work! Is there any official training code available now?

Excellent work!! Thanks for your open source, it will greatly assist my work!! 😊

repo: niki-amini-naieni/CountGD


text:
Loosen the test for "text-shadow: 1px 1px 1px"

phantomjs on Debian GNU/Linux gives a slightly different but equivalent result

rgb(0,0,0) is not equivalent to rgba(0,0,0,0)


text:
mkv to mp4 remux for dolby vision profile 8 constructs incorrect metadata.

MediaInfo.txt

Send sample mkv plz.

https://file.kiwi/qzTzhXZR#dStMpyaMR-q74P9B4r1lVg

I couldn't download it. You can just split a little clip for me (use MKVToolNix).

minisample.zip

This is caused by mp4muxer and I don't know why...

okay. Thank you for your review.

This is caused by mp4muxer and I don't know why...

I have an idea I'd like to suggest.
https://www.bento4.com/documentation/mp4mux/
How about using this instead of mp4muxer?

This is caused by mp4muxer and I don't know why...

I have an idea I'd like to suggest. https://www.bento4.com/documentation/mp4mux/ How about using this instead of mp4muxer?

I will consider it.

@bluewidy Try latest version
https://github.com/nilaoda/DoViMuxer/releases

Now this works great. Thank you!

authors: bluewidy nilaoda

repo: nilaoda/DoViMuxer


text:
老大，我的批处理脚本问题在哪，提示地址无效，单独下又可以

"N_m3u8DL-CLI_v2.9.9.exe" "https://valipl-vip.cp31.ott.cibntv.net/65728168E47307146D96062DC/05007E00006180E7678BB780000000D37DEB01-5DFD-4ADA-9B11-4C926004C320_video.m3u8?ccode=0502&duration=1477&expire=18000&psid=8d83f425f6491d3ca73e46f1be977d1443346&ups_client_netip=db90a4af&ups_ts=1640710268&ups_userid=1104021290&apscid=&mnid=&umt=1&type=cmaf4hd3&utid=PoU7Gvh4ux0CAXp2qYX6eieM&vid=XNTE5OTQ4NTQ4MA%3D%3D&s=eaac6f7662474171bcda&iv=1&sp=&t=6d456319bfd897e&cug=2&bc=2&si=3&eo=1&vkey=Bea549ae8d29bd7880214f79c2c881cf7" --saveName 10 --enableDelAfterDone --disableDateInfo --useKeyBase64 i1xw6NuhZGaY4xiZrQxpWA==
只列出了一条,而且我在使用gui填key解密时不可以，提示数据块不完整
用指令下载并解密时又正常 望解答

BAT中需要把%替换为%%

authors: nilaoda xiaoxiaoliya


text:
old icon in "moved" banner occasionally covering new icon

when viewing a moved account, the icon of the old account sometimes covers the icon of the new account.
this is a screenshot from 433.world, running v1.9.4:

and this is from i1.no, running v2.0.0-prerelease:

interestingly, on zeppelin.flights, which presumably also runs v1.9.4, this looks just fine:

(in all three cases the account previewed is the old FediTips account, @feditips@mstdn.social, although I've also seen this occur with other moved accounts.)

Ah, yes, I see where the issue is.
Thanks for reporting!

authors: nehemiagurl nileane

repo: nileane/TangerineUI-for-Mastodon


text:
You're calling a NumPy function on a CVXPY expression.

When I try to apply the trained method afhmm to new data I get the following error:

Traceback (most recent call last):
File "/home/habring/anaconda3/envs/nilm-contrib/lib/python3.7/multiprocessing/process.py", line 297, in _bootstrap
self.run()
File "/home/habring/anaconda3/envs/nilm-contrib/lib/python3.7/multiprocessing/process.py", line 99, in run
self._target(*self._args, **self._kwargs)
File "/home/habring/anaconda3/envs/nilm-contrib/lib/python3.7/site-packages/nilmtk_contrib/disaggregate/afhmm.py", line 157, in disaggregate_thread
total_observed_reading+=cvx_state_vectors[appliance_id]@means_vector[appliance_id]
File "/home/habring/anaconda3/envs/nilm-contrib/lib/python3.7/site-packages/cvxpy/expressions/expression.py", line 721, in array_ufunc
raise RuntimeError(NUMPY_UFUNC_ERROR)
RuntimeError:
You're calling a NumPy function on a CVXPY expression. This is prone to causing
errors or code that doesn't behave as expected. Consider using one of the
functions documented here: https://www.cvxpy.org/tutorial/functions/index.html

I think I have narrowed down the issue already. Apparently in line 157 in nilmtk_contrib/disaggregate/afhmm.py you do something like
numpy array += cvxp variable @ numpy array
I think since the left-hand side of the += is a numpy array, the operator @ is interpreted as a numpy function and then the error is raised.
Suggestions are welcome.
Best,
Andi

I could resolve this by donwgrading cvxpy to version 1.0.

I could resolve this by donwgrading cvxpy to version 1.0.

Hi, I encountered exactly the same issue, I tried downgrading by
pip install -I cvxpy==1.0.xx
or pip install cvxpy==1.0.xx,
I tried a lot of 1.0 versions, all the installations are failed. Could you tell which version you installed?

Hi,
I'm not 100% sure anymore. But maybe try the following.

Create a fresh conda environment.
Use conda to install scikit-learn=0.21, keras=2.2.4, and cvxpy=1.0.0.
Afterwards install nilmtk-contrib via conda install -c conda-forge -c nilmtk nilmtk-contrib

Also, generally when I am forced to work with conda, like in this case, since nilmtk uses conda as an environment manager, I try to stick to conda for installing packages if it is not necessary to invoke pip. I don't know if this makes a difference but you could try.
Best,
Andi

Thank you for your env info, the DSC finally can run now, but the AFHMM and AFHMM+SAC cannot work anymore... Maybe because of the downgrading of Keras? I'm on the verge of breaking down now...

repo: nilmtk/nilmtk-contrib


text:
🛑 Coasthill NO-MRA-EC-LE is down

In 602da4b, Coasthill NO-MRA-EC-LE ($URI_443_SERVER_3_NC_WEST) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Coasthill NO-MRA-EC-LE is back up in 9aca4a4.

repo: nilsjha/upptime-pub


text:
Change the name of the bucket to be in par with the changes on main repo

The name of the buckets needs to be changed from having the suffix nimbella-io to support Bring Your Own Domain feature where the client can get their custom domain names.
To be merged with #2101 from the main repo

Can this be merged?

Can you check the failing tests?

Do you want to merge this before 2101?

authors: Tejusbharadwaj jthomas rabbah

repo: nimbella/nimbella-sdk-python


text:
more efficient dcat-ddirch conjugacy

This creates 'ncat=length(prob)' as altParam for dcat so that ncat can be used instead of length(prob) in dcat-ddirch conjugate sampler. This should save some copying of the prob values in the conjugate sampler code, though as the code stands in the PR we will still dynamically calculate length(prob) when getParam(..., 'ncat') is called.
To hard-code in the actual length(prob), I think I could intervene in buildNodeFunctions just after:
     altParams <- BUGSdecl$altParamExprs
     altParams <- lapply(altParams, nimble:::insertSingleIndexBrackets, modelDef$varInfo)

and simply do 'altParams$ncat <- eval(altParams$ncat)'.  However, before I do that, could either @perrydv or @danielturek fill me in on what the 'insertSingleIndexBrackets' is doing here?

@perrydv can certainly comment with more authority, but per my understanding, all model variables are internally stored as vectors, or arrays with dimension >= 2.  Meaning, there's nothing (in a model) that's actually a scalar.  Accordingly, when we have
a <- ...
b ~ dxxx(...)

in model code, both a and b will eventually be represented as length-1 vectors in the underlying C++.  So for consistency, we alter the model code, so that it's:
a[1] <- ...
b[1] ~ dxxx(...)

to agree with this from the beginning of the model processing stages.  This holds for any appearances of a, b, and any such (originally declared) scalar variables.  So we need to process all the model code to insert [1] after such variable names.  This happens a lot of places, including in the declaration and processing of altParam code.
Not sure I'm totally following your suggestion about "intervening in the buildNodeFunctions function", but I would be extremely hesitant to insert anything in there that's specific to the processing of dcat distribution.  Right?
Also, this is merely a suggestion on your PR.  Happy to go with whatever you prefer, but thoughts on re-naming ncat to k?  Up to you.  I understand ncat is for "number of categories", but seems confusingly similar to dcat, rcat, etc.  k seems like a nice variable name to indicate this fixed number.  But like I said, merely a thought, I'm ok either way.

Thanks I had forgotten about the [1] insertion. Clear now.
Yes, I was proposing an "if(dist == 'dcat') {} ' insertion in buildNodeFunctions, and agree it is ugly.
Yes, 'k' is probably better.

This is passing locally on my machine so not sure what is up with Travis.
I changed 'ncat' to 'k' and hoping that the new Travis build will pass tests.

authors: danielturek paciorek

repo: nimble-dev/nimble


text:
[#352] Fix: ArgumentError: unknown keyword: :desired_capabilities when running system tests with the headless_chrome driver

Close #352

What happened 👀

Remove the deprecated desired_capabilities from the capybara config

Insight 📝
The desired_capabilities was deprecated quite some time ago and ended up being removed from the newer versions of our dependencies.
We did not find another place to configure the logs (ALL everywhere). But we believe such a configuration is not a must-have and we can do without it. Feel free to challenge and suggest improvements! :)
Proof Of Work 📹

Before
After

@hoangmirs @carryall @junan @longnd Any comment? :)

repo: nimblehq/rails-templates


text:
Remaining word count is not correct when switching start team

Start a new game in which red goes first.
Note that the words remaining are correct - 9 for red and 8 for blue.
Start a new game in which blue goes first.
Note that the words remaining are incorrect - there should be 9 for blue and 8 for red, but it is the other way around.

We love playing this for team building at work.  We often play 2 or 3 games in a row which is how we found this bug.  Whenever blue goes first, the words remaining counts are wrong.

In updateScore() it seems to always subtract from blue.
//subtract 1 for non-starting team
	if($('.redStarts') === 1){
		redScore--;
	} else {
		blueScore--;
	}

fixed: https://github.com/ninjabunny/KodeNames/commit/97d46b417b04d63402c10026232e2d06d0648677
@ig-ealbert where do you work? are they hiring? I will play codenames with you hahaha :D

@ninjabunny I work at Infragistics.  We have job openings listed here.  We'd be happy to play with you! ^_^
Thanks for fixing this so quickly!  Should I verify and close the issue?

Yes. Thanks!

Verified!

authors: ig-ealbert ninjabunny

repo: ninjabunny/KodeNames


text:
Allow matching on whole words only

Add a search option to Containing() and ContainingAll() to allow for matching on whole words only
Given "I like coffee" is present in a string
When searching for "fee" with WholeWord search options selected
Then no data is returned
Given "I like coffee" is present in a string
When searching for "coffee" with WholeWord search options selected
Then data is returned

New feature added to SearchExtensions, version 1.6.
http://jnye.co/Posts/2044/search-extensions-whole-word-matching

repo: ninjanye/SearchExtensions


text:
Carthage build error

Hello,
I have an issue on executing carthage command to add your library on my project, see the message below:
`
*** Building scheme "NVActivityIndicatorViewAppExtension" in NVActivityIndicatorView.xcodeproj
Build Failed
Task failed with exit code 65:
/usr/bin/xcrun xcodebuild -project /Users/marcoslacerda/my-project/Carthage/Checkouts/NVActivityIndicatorView/NVActivityIndicatorView/NVActivityIndicatorView.xcodeproj -scheme NVActivityIndicatorViewAppExtension -configuration Release -derivedDataPath /Users/marcoslacerda/Library/Caches/org.carthage.CarthageKit/DerivedData/9.2_9C40b/NVActivityIndicatorView/4.1.0 -sdk iphoneos ONLY_ACTIVE_ARCH=NO BITCODE_GENERATION_MODE=bitcode CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY= CARTHAGE=YES archive -archivePath /var/folders/_c/3nj34dfs13qb1ny20c_g06qm0000gn/T/NVActivityIndicatorView SKIP_INSTALL=YES GCC_INSTRUMENT_PROGRAM_FLOW_ARCS=NO CLANG_ENABLE_CODE_COVERAGE=NO (launched in /Users/marcoslacerda/my-project/Carthage/Checkouts/NVActivityIndicatorView)
This usually indicates that project itself failed to compile. Please check the xcodebuild log for more details: /var/folders/_c/3nj34dfs13qb1ny20c_g06qm0000gn/T/carthage-xcodebuild.L1CU2i.log
`
This is a part the content of log file:
`
/Carthage/Checkouts/NVActivityIndicatorView/NVActivityIndicatorView/NVActivityIndicatorView/NVActivityIndicatorView.swift:336:20: error: use of unresolved identifier 'NVActivityIndicatorAnimationCircleStrokeSpin'
return NVActivityIndicatorAnimationCircleStrokeSpin()
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
** ARCHIVE FAILED **
The following build commands failed:
CompileSwiftSources normal arm64 com.apple.xcode.tools.swift.compiler
CompileSwift normal arm64
(2 failures)
`
Do you have any idea of this issue?
Thanks for your help
[UPDATE]
If i use the version 3.7.0 the problem not occurred
=D

I see the same thing.
Xcode 9.2, High Sierra

@marcoslacerda: It seems related to #186, and the fix in #187 has been merged to master. Could you check again?

@ninjaprox Thank you for your help
But the problem persist yet.
If i use the code below in my Carthage file the error on build occurred:
github "alickbass/NVActivityIndicatorView"
If i use the code on #186 the problem not occurred.
Do you have Any idea ?
Thanks

@marcoslacerda: Please refer to Cartfile spec. By using either github "ninjaprox/NVActivityIndicatorView" or github "alickbass/NVActivityIndicatorView", it refers to the latest version 4.1.0 which's having the issue. At the moment, you can fix this by specifying the branch github "ninjaprox/NVActivityIndicatorView" "master". I'll release a new version with the fix by today.

Ok
I'm using :
github "ninjaprox/NVActivityIndicatorView"
In my Cartfile, i'll wait for your release, thank you!

@marcoslacerda: 4.1.1 is out 🚀 . Please help check.

@ninjaprox As for me, the latest version 4.1.1 builds fine in Carthage now.  Thank you!

I'm sorry for the delay in returning.
Version 4.1.1 is working perfectly with Carthage now, thanks for the support.
Thanks

authors: alexn-spire marcoslacerda ninjaprox

repo: ninjaprox/NVActivityIndicatorView


text:
[RESOLVED] Unavailable by ICMP ping

Problem started at 21:12:14 on 2023.07.27
Problem name: Unavailable by ICMP ping
Host: MF-FG
Severity: High
Operational data: Down (0)
Original problem ID: 33175
Event details in Zabbix: http://192.168.232.99/tr_events.php?triggerid=23130&eventid=33175

Problem has been resolved in 9h 57m 0s at 07:09:14 on 2023.07.28
Problem name: Unavailable by ICMP ping
Host: MF-FG
Severity: High
Original problem ID: 33175


text:
[RESOLVED] High ICMP ping response time

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

Problem has been resolved in 4m 0s at 10:08:14 on 2023.05.06
Problem name: High ICMP ping response time
Host: MF-FG
Severity: Warning
Original problem ID: 20375


text:
MNT: Update Ubuntu, FreeSurfer, AFNI and Convert3D

Changes proposed in this pull request

Update Ubuntu to 22.04 LTS
Update FreeSurfer to 7.3.2
Update FSL to 6.0.6.2, download only needed packages from conda
Update AFNI installation to work on 22.04
Update ANTs to 2.4.3
Update Convert3D to 1.3.0, from conda-forge

Removing ANTs bump for now. AFAIK there are no outstanding issues we've been waiting on a new ANTs for.

ICA-AROMA is the only thing stopping us from upgrading FSL: #2936.

@mgxd I would appreciate a review on this one today, if possible. It would be good to do all local testing on updated dependencies.

have you tested on any non-CI datasets?

I have tested on one. It will be good to do some more, but the changes relative to 7.2 were not massive.

I think having a FreeSurfer include file, rather than exclude,

The majority of the change was just running sort on the file. I think an include would be more brittle. Here, a name change means we are over-inclusive, while an include file would reverse that.

repo: nipreps/fmriprep


text:
ShakeDetion should work in background

Do you want to request a feature or report a bug?
Feature
What is the current behavior?
ShakeDetection is working when app is open. I want it even though app is killed.
If the current behavior is a bug, please provide the steps to reproduce.

Any logs, error output, bugreport etc?

What is the expected behavior?
Any other comments?
Please help to work with Shake Sensor in Background too.
What versions of software are you using?

Device Information:
Samsund Galext J8

Android Version:
Android 9.0

This can not be done anymore, from Android 9 onwards.
Read more: https://developer.android.com/about/versions/pie/android-9.0-changes-all#bg-sensor-access

There is also written like:

If your app needs to detect sensor events on devices running Android 9, use a foreground service.

True, but that is not a implementation I see being bundled in the library. Activity/Services are not supposed to be bundled. They are a client side implementation. The Aim of Sensey is to detect the Gesture, how and where it is needs to be detected is a app developer decision.

Yeah, you are right. This is really great library.
Thank you @nisrulz for your time.

authors: nisrulz pratikbutani

repo: nisrulz/sensey


text:
circle ci broken by latest matplotlib release

the example plot_localizer_surface_analysis fails with matplotlib 3.1
since the latest matplotlib release, setting the aspect of an Axes3d results in
a error, which caused an error in nilearn's plot_surf_stat_map. The problem
has been fixed in nilearn:
https://github.com/nilearn/nilearn/pull/2070
but the fix has not been released yet.
maybe the simplest strategy is to pin the circle ci matplotlib to an older
version until the next nilearn release.

@kchawla-pi do you want to take care  of that ?

Sure. We already fixed this in Nilearn, it is not a difficult thing.
On Wed, Jun 12, 2019, 08:28 bthirion notifications@github.com wrote:

@kchawla-pi https://github.com/kchawla-pi do you want to take care of
that ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nistats/nistats/issues/358?email_source=notifications&email_token=AB6SXRHUECYU5ZQ4PQRMAE3P2CJPRA5CNFSM4HXDZ4M2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXPMJCI#issuecomment-501138569,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB6SXRHNVOQYDVKUDCKGAPLP2CJPRANCNFSM4HXDZ4MQ
.

Since the code that has been rendered incompatible is in Nilearn (fixed in master, not release), I have pinned MPL==3.0.3 in CircleCI Config.

authors: bthirion jeromedockes kchawla-pi

repo: nistats/nistats


text:
Windows: fix bootstrap

It looks like the use of the method is_tty in Nit tools triggered the FFI compilation of the posix module which fails on Windows. This module imports some headers that are unavailable in the msys2 environment, as used by our test servers.

:robot: test windows please

🤖 test this please

Won't this cause extra warnings to appear for the Password and Group classes on Windows? Is there a way we could only include both these classes and the related header files on POSIX environments (like platform or anything)?

Yeah, don't use Password or Group on Windows. Luckily, the C code of unused FFI methods is not compiled so you can import posix (or core) and it compiles, but GCC reports errors if you use them.
We should write alternative implementations for Windows as I did in exec and file. Or remove the posix module from core,  its main services may be a bit too POSIX for portable code. It looks like isatty falls back on the ISO C++ conformant _isatty.

I changed to fix to move posix out of core.
🤖  test windows please

:robot: test windows please

This should be it...
🤖 test windows please

... test windows please

repo: nitlang/nit


text:
implement event publisher (a.k.a. Zeebe exporter)

the engine is meant to be embedded.
that said, there's a need/appetite to bring transparency into the engine, by allowing users to have a "view" on their processes and states.
One of the initial ideas/goals of this project is to emit the same events, as the popular Camunda Cloud a.k.a. Zeebe BPMN Engine.
Zeebe uses a protobuf based exporter protocol.
The idea for this library is, to provide an optional module so that developers can opt-in to emit events like Zeebe does with the ultimate goal to use their Simple Monitor.

Had to switch the milestone to 0.4.0, because Hazelcast's Go client (which is required for sending events to Hazelcast, the default exporter for Zeebe) does miss RingBuffer support, see https://github.com/hazelcast/hazelcast-go-client/issues/751

I've contributed to Hazelcast Go Client and started in a feature branch to experiment with event publishing.
See https://github.com/nitram509/lib-bpmn-engine/tree/experimental-zeebe-exporter


text:
lorri Crash Report

Describe the bug
The lorri daemon crashed, yielding the attached crash report from three threads:

Metadata

reports.zip

$ lorri info --shell-file shell.nix
lorri version: 1.5
GC roots exist, shell_gc_root: /Users/chris/Library/Caches/com.github.nix-community.lorri.lorri.lorri/gc_roots/d44ed7205e17d524d463782543f64e72/gc_root/shell_gc_root

$ uname -a
Darwin ume.local 21.3.0 Darwin Kernel Version 21.3.0: Wed Jan  5 21:37:58 PST 2022; root:xnu-8019.80.24~20/RELEASE_X86_64 x86_64 i386 Darwin

This bug is fixed in 1.7.0, which is in nixpkgs as of NixOS 24.05. Thank you for your patience!

authors: chrisandreae nyarly

repo: nix-community/lorri


text:
fix: Set BatchMode for ssh-copy-id

ssh-copy-id can hang indefinitely at the SSH password prompt if Server is configured to allow KeyboardAuthentication and PubKeyAuth fails.
SSH BatchMode instructs the ssh client to disable all interactive user prompts like password and host key confirmation.
Using the terraform module without BatchMode can result in a terraform apply run hanging "forever" at the password prompt.  With BatchMode enabled the terraform apply run fails correctly.

Digging through the code a bit more I suspect this could happen during all steps of nixos-anywhere that use SSH.
I guess the question is if running nixos-anywhere from the CLI should support the use-case of entering the password and this issue would be better fixed by the terraform module setting BatchMode=yes via the --ssh-option parameter?
Please advice.

repo: nix-community/nixos-anywhere


text:
Add wrappers for some time-related syscalls

gettimeofday, ~clock_gettime~, ~clock_settime~, ~clock_getres~, nanosleep and clock_nanosleep seem to be supported by various *nixes (at least Linux and FreeBSD), but bindings are not included in this crate.
I'd be happy to open a PR adding these bindings.

Check again.  https://github.com/nix-rust/nix/pull/1281 just added some of those.

Darn. I must have searched issues and not pull requests. Regardless, some are still not implemented (the original issue has been edited).

Why would you ever use gettimeofday in new code, instead of clock_gettime?  And why would you ever need nanosleep instead of thread::sleep?

My use case is rather niche - I'm instrumenting syscalls, which is quite possibly the only reason to use the first two functions in new code. clock_nanosleep could use a clock other than CLOCK_MONATONIC, which is what std frequently uses (std uses plain nanosleep for thread::sleep).
Do you think it would be worthwhile to at least implement clock_nanosleep?

Do you think it would be worthwhile to at least implement clock_nanosleep?

Yep.

authors: asomers maxbla

repo: nix-rust/nix


text:
Added wrapper around getrandom syscall

Hi,
Added wrapper around Linux's getrandom syscall as mentioned in #1826.
pub fn getrandom(buffer: &mut [u8], mode: RandomMode) -> Result<isize>

returns number of bytes written
Thanks :)
Signed-off-by: Stefin stefin@pm.me

@stefins actually, is there a compelling reason to include these changes in nix, instead of just depending on the getrandom crate?

@rtzoeller apart from adding a wrapper around getrandom this doesn't have any other compelling reason. Since libc support getrandom I thought this will be a good addition for the nix crate

@rtzoeller apart from adding a wrapper around getrandom this doesn't have any other compelling reason. Since libc support getrandom I thought this will be a good addition for the nix crate

The getrandom crate already provides a wrapper around the libc function, including aggregating similar functions from multiple platforms. In this case I'd rather not attempt to duplicate their work, which is excellent.
If a compelling reason comes up for why the getrandom crate cannot be used and a separate wrapper is strongly preferable, I'd be happy to see this PR re-opened. I'm going to close it for now though.

authors: rtzoeller stefins


text:
Add IpMtu sockopt

Resolves https://github.com/nix-rust/nix/issues/1864

I'm not sure why some linux tests are failing, they say ENOPROTOOPT, is networking not enabled on those architectures?

The architecture's where the tests are failing are the ones where we use qemu user-mode emulation in CI.  Lots of stuff isn't enabled in qemu.   You should add this to your test:
#[cfg_attr(qemu, ignore)]

The clippy failure is fixed on master.
bors retry

authors: ShadowJonathan asomers


text:
Add Iteration Statements

for, while and do-while loops should be supported.

[ ] for
[ ] while
[ ] do-while

added in (1b9808e727e4b275d93d78abb64805e95b42452a)

repo: niyrme/NyrLang


text:
add GZIP compression

closes #3

Pull Request Test Coverage Report for Build 219

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 89.674%

Totals

Change from base Build 211:
0.0%

Covered Lines:
165

Relevant Lines:
184

💛  - Coveralls

authors: coveralls nlohmann

repo: nlohmann/crow


text:
Typo in cmake/test.cmake

Description
At line 260 in cmake/test.cmake there is this code
        include(CheckTypeSize)
        check_type_size("size_t" sizeof_size_t LANGUAGE CXX)
        if(sizeof_size_t AND ${sizeof_size_t} EQUAL 4)
            message(STATUS "Auto-enabling 32bit unit test.")
            set(${build_32bit_var} ON)
        else()
            set(${build_32bit_var} OFF)
        endif()

The conditional check
        if(sizeof_size_t AND ${sizeof_size_t} EQUAL 4)

The first condition if(sizeof_size_t is always TRUE, since it is comparing against the string value "sizeof_size_t". To check if the string is empty (meaning check_type_size could not return a value for size_t)  it should be
        if(${sizeof_size_t} AND ${sizeof_size_t} EQUAL 4)

If for some reason the size of size_t is unknown (for example, using exotic toolchains where stddef.h might not be visible during the initial cmake evaluation) this will cause the invocation of cmake to fail.
Reproduction steps
Replace the call to check_type_size with
set(sizeof_size_t "")

Expected vs. actual results
Running cmake . should not fail.
Minimal code example
No response
Error messages
if given arguments:

    "sizeof_size_t" "AND" "EQUAL" "4"

  Unknown arguments specified

### Compiler and operating system

gcc 11.2.0, Linux

### Library version

3.11.2

### Validation

- [X] The bug also occurs if the latest version from the [`develop`](https://github.com/nlohmann/json/tree/develop) branch is used.
- [ ] I can successfully [compile and run the unit tests](https://github.com/nlohmann/json#execute-unit-tests).

Good catch!

Hi @nlohmann can you please assign me

Go ahead!

Hi @nlohmann I'm new to CMake can you suggest to me how to setup development environment on my local machine and how to run the project

No, I cannot assist you there. Please let me know if I should un-assign you.

Hey @nlohmann, can I do it?

Go ahead!

authors: akbhuker m-hilgendorf nlohmann theevilone45

repo: nlohmann/json


text:
Add Vcpkg port version badge

I'm the author of vcpkg.link and I'm trying to improve the discoverability of Vcpkg ports. From my experience, such package managers have been helping developers to integrate libraries into their projects while keeping them up to date.
Therefore, this PR aims to add a shields.io badge that provides at a glance in the README which version of the library is available on Vcpkg.
Moreover, it provides a link to a page that contains extra details about the Vcpkg port of the library and how to use it.

Pull request checklist
Read the Contribution Guidelines for detailed information.

[x]  Changes are described in the pull request, or an existing issue is referenced.
[x]  The test suite compiles and runs without error.
[x]  Code coverage is 100%. Test cases can be added by editing the test suite.
[x]  The source code is amalgamated; that is, after making changes to the sources in the include/nlohmann directory, run make amalgamate to create the single-header files single_include/nlohmann/json.hpp and single_include/nlohmann/json_fwd.hpp. The whole process is described here.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Specifically, I am aware of compilation problems with Microsoft Visual Studio (there even is an issue label for this kind of bug). I understand that even in 2016, complete C++11 support isn't there yet. But please also understand that I do not want to drop features or uglify the code just to make Microsoft's sub-standard compiler happy. The past has shown that there are ways to express the functionality such that the code compiles with the most recent MSVC - unfortunately, this is not the main objective of the project.
Please refrain from proposing changes that would break JSON conformance. If you propose a conformant extension of JSON to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

Coverage: 100.0%. Remained the same when pulling aad12e5f1098966667b9b6ae992c46148af74e8c on njakob:vcpkg-badge into 546370c9e778d99e7176641123e5cc1d0b62acab on nlohmann:develop.

Thanks!

authors: coveralls njakob nlohmann


text:
Adding appropriate message for nltk.download when loading pre-trained models

Following up on #1787, it would be great if the users download specific corpora / models instead of abusing nltk.download('all').
@alexisdimi gave a good suggestion to warn the user with the appropriate package to download when an error is raised at model/data loading.
For the various model, the message would have to be added when the error is raised while loading the model.
To start off, these should be the more popular models that users might load

punkt / sent_tokenize:  https://github.com/nltk/nltk/blob/develop/nltk/tokenize/init.py#L84
pos_tag: https://github.com/nltk/nltk/blob/develop/nltk/tag/init.py#L84
ne_chunk: https://github.com/nltk/nltk/blob/develop/nltk/chunk/init.py

Additionally, a quick search on https://github.com/nltk/nltk/search?utf8=✓&q="from+nltk.data+import+load"&type= reveal several more.

Resolved in #1806

repo: nltk/nltk


text:
Re-running each MCMC saved sample to get a report file in Hake assessment

Imported from redmine, Issue #75754
Opened by @k-doering-NOAA on 2020-03-03
Status when imported: New
This first came up in #75174, but is a separate topic from the original issue being discussed there.
Kelli noted:

Thanks Ian for thinking of the hake assessment. We do in fact re-run each saved sample with bias adjustment
ramp = 1.0 to generate a report file that can later be read in. Does anyone know how other Bayesian assessments
written in ADMB get around this issue? Also, what other stocks turn on MCMC within an SS model?

and Rick responded:

Kelli and Ian,
I think we need a new issue for this topic of re-running posteriors to get
full report. I had no idea that was being done. Is there more that can be
added to posteriors.sso to obviate that need?

@kellijohnson-NOAA
We need to decide if it is better to find a way to write each MCEVAL's complete, but customized, report.sso to a separate folder
or to create a new report function with newly specified output that is called only during MCEVAL

Infrastructure wise, I am guessing that just writing Report.sso for every iteration would be the easiest, but I am unsure is this is limiting time and size wise?

time and space is not insignificant.  But customized selection of individual elements of report.sso is already a SS feature.  Seems feasible to create capability so that a different (leaner) set of elements gets written during MCEVAL.

the mkdir() function in C++ will create sub-directories.  Need to investigate whether or not it is cross-platform (iOS and linux)
@nschindler-noaa

Okay. the syntax for creating new folders is
mkdir("geeksforgeeks", 0777)
seen in the following code:
if (mkdir("geeksforgeeks", 0777) == -1)
cerr << "Error :  " << strerror(errno) << endl;
else
    cout << "Directory created";

On Thu, Oct 14, 2021 at 12:13 PM Kathryn Doering @.***>
wrote:

Reopened #70
https://github.com/nmfs-stock-synthesis/stock-synthesis/issues/70.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nmfs-stock-synthesis/stock-synthesis/issues/70#event-5465870042,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARUZACWBT7XFK7SIRW74U3LUG4TVLANCNFSM4TLVRGOQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
Neal Schindler
Programmer Analyst, Caelum
NWFSC (206) 860-3407
cell (206) 909-0505

=(((<  >=(((<  >=(((`<

=(((`<

Thanks Neal.
Then need to change to that as the output directory.  I found the code below and perhaps could modify from there:
#ifdef _WIN32
#include <direct.h>
// MSDN recommends against using getcwd & chdir names
#define cwd _getcwd
#define cd _chdir
#else
#include "unistd.h"
#define cwd getcwd
#define cd chdir
#endif
#include
char buf[4096]; // never know how much is needed
int main(int argc , char** argv) {
if (argc > 1) {
std::cout  << "CWD: " << cwd(buf, sizeof buf) << std::endl;
// Change working directory and test for success
if (0 == cd(argv[1])) {
  std::cout << "CWD changed to: " << cwd(buf, sizeof buf) << std::endl;
}

} else {
std::cout << "No directory provided" << std::endl;
}
return 0;
}

OK.  Cycling back to the main question before starting any coding:  Is it better or easier to implement the MCEVAL reports as:
a)  specific report name, like report_mcev****.sso, in current folder
b)  report.sso in new subfolder named mcev****
c)  report.sso written in append mode (like data.ss_new) like a separator named  mcev****
In all cases, a user-specified subset of all report.sso elements could be implemented.
Inclusions of reports other than report.sso should also be considered.

I will do an experiment with the mkdir, chdir code to be sure that a chdir during a run does not totally mess with what ADMB is expecting.  But not yet 100% convinced that this is the best approach among the three  I outlined above.

Thanks Ian.  I do see this as a viable option.  I worry that the chdir() will not work with ADMB during a run because ADMB is writing others items in append mode for each mceval call (i.e. the write to posteriors.sso which is in the original directory).
Regarding the need for compreport.sso - let's start a separate conversation on this.  I think the r4ss processing of compreport.sso is redundant with what SS3 is doing and reporting in report.sso to the fit_age_comp section.  We can tweak that write to fit_age_comp to make it do exactly what is needed.

Hi Rick,
That's reasonable to not worry about CompReport.sso for now. The image below is from the beginning of this thread on VLab (https://vlab.noaa.gov/redmine/issues/75754#note-3, but the images didn't get transferred to github).
I think getting either the posterior medians or the 95% intervals requires one row of output per age bin as in CompReport.sso rather than one row per age comp observations as in FIT_AGE_COMPS. However, the intervals are trivially small in almost all cases and my memory is that the posterior medians are so close to the MLE that I'm guessing it would be fine to skip this step and not bother with CompReport.sso files from all the mceval steps. This figure based on CompReport.sso was originally created only because those files were already available from the inefficient status-quo workflow used for hake.

Apologies Neal, I had some time and found something that works.  Claims to be cross-platform.
#include <sys/types.h>
#include <sys/stat.h>
struct stat pathinfo;
adstring pathname;
if( stat( "./ssnew", &pathinfo ) != 0 )
{
  pathname="";
}
else
  {
    pathname="./ssnew/";
  }

for (Nudat=1;Nudat<=N_nudata;Nudat++)
{
if(Nudat==1)
{
anystring=pathname+"data_echo.ss_new";
report1.open(anystring);
...

Discussions about subfolders and files names should now take place in #226 ; this issue should now be only used to discuss the original issue of saving each MCMC sample to a get report file

@Rick-Methot-NOAA regarding files needed for hake. Chris Grandin reminded me today that we also use compreport.sso for Composition Database in addition to report.sso.

authors: Rick-Methot-NOAA iantaylor-NOAA k-doering-NOAA kellijohnson-NOAA nschindler-noaa

repo: nmfs-stock-synthesis/stock-synthesis


text:
Generalize skill info

Currently the effects of attacks and skills are hardcoded in a switch statement in AttackSys.ts. Let's generalize attack and skill data. Consider how far towards generalized actions the new generalized skill data structure should be.
The UI will need a way to consistently retrieve relevant data for skills.

At some point this was already completed.

repo: nmkataoka/adv-life


text:
Admin UI: tier quota definition is not deleted from the database after removing it from a tier

Readiness checklist

[ ] I added/updated unit tests.
[ ] I added/updated integration tests.
[X] I ensured that the PR title is good enough for the changelog.
[X] I labeled the PR.

Description
Prior to fixing it the attempt at deleting the tier quota definition in the Admin UI would imply orphan it, i.e. set its TierId column value to null, making it in effect "orphaned" (not connected to any Tier entity). Database operation trigger is introduced that is activated after updating tier quota definition and which deletes the said definition should it turn out to be orphaned.

@NikolaVetnic The title of the PR is a bit messed up.

"Neither [...] nor [...] are not merged [...]"

"Neither" and "are not" is a double negation.
I don't see how "merged" would make sense in this context

Question: did you do a manual test?

Literally the first thing I did :) I'll do it once again just to be on the safe side.

authors: NikolaVetnic tnotheis

repo: nmshd/backbone


text:
json2tbl leaves extra padding at the end

currently affecting btlsys, gourmet, condition info and chapter save. shouldn't be a big deal but it's nice to investigate to iron them out finally.

Ah no sorry, the padded zeros come from the decrypted files. When compiling them json2tbl just don't read them since the files usually end with text which ends with a single 0.
Pic below with the recompiled file at the top and the decrypted file (from korean tables) at the bottom

Extra padding is for Blowfish encryption. Not a bug


text:
Layer properties as map

Update properties internally to be set with std::map<std::string, std::any> with upgrade to c++17.

:octocat: cibot: Thank you for posting issue #269. The person in charge will reply soon.

This will be handled with #404 different strategy that ensures compile time type

authors: kparichay taos-ci zhoonit

repo: nnstreamer/nntrainer


text:
[Fix] Flatten realizer to maintain original name

Dependency of the PR
Commits to be reviewed in this PR
[Fix] Flatten realizer to maintain original name
This patch changes flatten realizer to maintain original node name.
Please see the included test patch to get the concept.
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 #1800. 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/.

:octocat: cibot: @zhoonit, nntrainer/compiler/flatten_realizer.cpp includes bug(s). Please fix incorrect coding constructs in your commit before entering a review process.

:octocat: cibot: @zhoonit, nntrainer/compiler/flatten_realizer.cpp includes bug(s). Please fix incorrect coding constructs in your commit before entering a review process.

:octocat: cibot: @zhoonit, 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://nnstreamer.mooo.com/nntrainer/ci/repo-workers/pr-checker/1800-202112171508390.023124933242798-ae2618092d1b815ec9fc3a97dfb3c9445d41de89/.

authors: taos-ci zhoonit


text:
Fixed incorrect matching expression for webhook

This commit is to fix incorrect usage of the regular expression
when the wehook handler (PHP) checke "@open sesame" tag.

Input (PR title): a model is updatable - @open sesame 12/03 16:47
Before this PR: "@open sesame 12/03 16:4"
After  this PR: "@open sesame 12/03 16:47"

Signed-off-by: Geunsik Lim geunsik.lim@samsung.com

:memo: Version: 1.3.20190927. Thank you for submitting PR #598. 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/.

authors: leemgs taos-ci

repo: nnsuite/TAOS-CI


text:
ArgumentException when reloading a file with the same name

https://github.com/no00ob/LCSoundTool/blob/4e0a3f1593f21ea75cbcd5b2cfc02dcff91ce5f7/SoundTool.cs#L880-L881
If a mod was to reload a file with the same filename, this throws an
ArgumentException: An item with the same key has already been added. Key: XYZ
Probably would be more sane to check if this clip is being reloaded and replace the type.
Hypothetical solution
if (result != null)
-     clipTypes.Add(result.GetName(), audioType);
+     clipTypes[result.GetName()] = audioType;

Hey! Thanks for letting me know and thank you very much for the fix. I'll merge it into the next update.