Submitting a JOSS paper

I've just pushed a draft of a paper for submission to JOSS in parallel with the release of version 3.0. I'd like to invite members of the community to join as an author on this paper if you're are interested. Please reply to this thread with your full name, affiliation, and (optionally) ORCID in the format needed by JOSS if you would like to join as an author. Pull requests or comments about the manuscript are also welcome! For example, my info is:
name: Daniel Foreman-Mackey
orcid: 0000-0003-0872-7098
affiliation: Center for Computational Astrophysics, Flatiron Institute

I'd like to submit this on 28 Oct 2019 so let me know if you're interested in joining, but need more time to decide.
Pinging committers with at least 10 lines (but others are also welcome if you feel that you have contributed!):

@farr
@manodeep
@davidwhogg
@jeremysanders
@eteq
@joezuntz
@pkgw
@andyfaff
@bencebeky
@dstndstn
@drphilmarshall
@willvousden
@terhardt
@Christopher-Bradshaw
@ipashchenko
@OriolAbril
@ericdill
@mindriot101
@nkern
@migueldvb

@dfm Thanks! Yes, I would like to be listed as an author on the paper.
name: Manodeep Sinha
orcid: 0000-0002-4845-1228
affiliation: "1, 2"

affiliations:
 - name: Centre for Astrophysics & Supercomputing, Swinburne University of Technology
   index: 1
 - name: ARC Centre of Excellence for All Sky Astrophysics in 3 Dimensions (ASTRO 3D)
   index: 2

That's a very kind invitation, I would like to accept.
name: Andrew R. J. Nelson
orcid: 0000-0002-4548-3558
affiliation: "1"

affiliations:
 - name: Australian Nuclear Science and Technology Organisation, NSW, Australia
   index: 1

Thank you for the kind invitation and for drafting the paper. I'd feel honoured to accept.
name: Tobias Erhardt
orcid: 0000-0002-6683-6746
affiliation: Climate and Environmental Physics and Oeschger Center for Climate Change Research, University of Bern, Bern, Switzerland

This is very kind of you, thank you very much.  If you're happy to include me I'd be happy to accept.
name: Joe Zuntz
orcid: 0000-0001-9789-9646
affiliation: Institute for Astronomy, University of Edinburgh, Edinburgh, EH9 3HJ, UK

Thank you, Dan. I would like to accept your kind invitation.
name: Ilya Pashchenko
orcid: 0000-0002-9404-7023
affiliation: P.N. Lebedev Physical Institute of the Russian Academy of Sciences, Moscow, Russia

Thanks for the invitation!
  - name: Peter K. G. Williams
    orcid: 0000-0003-3734-3587
    affiliation: "1, 2"

affiliations:
 - name: "Center for Astrophysics | Harvard & Smithsonian"
   index: 1
 - name: American Astronomical Society
   index: 2

Thanks, Dan---I would be happy to be a co-author for v3.
name: Will M. Farr
orcid: 0000-0003-1540-8562
affiliation: "1, 2"

affiliations:
 - name: "Department of Physics and Astronomy, Stony Brook University, Stony Brook NY 11794, United States"
   index: 1
 - name: "Center for Computational Astrophysics, Flatiron Institute, 162 5th Ave, New York NY 10010, United States"
   index: 2

Thanks for the invitation! My orcid and affiliation are:
  - name: Miguel de Val-Borro
    orcid: 0000-0002-0455-9384
    affiliation: Planetary Science Institute, 1700 East Fort Lowell Rd., Suite 106, Tucson, AZ 85719, USA

Thank you so much for the invitation, it would be an honor.
- name: Oriol Abril Pla
  orcid: 0000-0002-1847-9481
  affiliation: Universitat Pompeu Fabra, Barcelona

@dfm: This is too kind; I have done nothing for this update, except slow you down by creating obstacles in the rest of your life. I am perfectly happy to live in the acknowledgements. But if you want me in the author list I'm:
name: David W. Hogg
orcid: 0000-0003-2866-9403
affiliation: Center for Computational Astrophysics, Flatiron Institute
affiliation: Center for Cosmology and Particle Physics, Department of Physics, New York University

That's a kind offer @dfm . The only thing I've done on this repo is a tiny bit of work on the CI parts. I hardly think that warrants inclusion on your manuscript. I'd like to let the folks who've contributed meaningfully to the code base get the credit they deserve.

Thanks very much for the invitation! I feel like I haven't really provided anything significant for the code (just an example), so I don't think I'm justified becoming an author.

Well, if you think an example is good enough, then I'd be happy to be involved. My details are
name: Jeremy S. Sanders
orcid: 0000-0003-2189-4501
affiliation: Max Planck Institute for Extraterrrestrial Physics

Hey @dfm, I appreciate the offer, but my contribution was trivial and so I don't think I should be on the paper.
Thanks, and congrats!

@dfm Similarly with me, I don't think I contributed to the code base in a meaningful way. Please feel free to not add me.
Thanks

@aarchiba: Would you like to be a co-author on this submission as well given your work on #319?

If I can get the code working and worth merging, and if you feel like it's
a significant contribution, sure. I actually looked to see whether I hadn't
added something to do with MPIPool but it doesn't show up in the history so
probably I just hacked something together for my own use. I don't need to
be "author n+1 who barely did anything" just because I fixed a bug, but I
am willing to do some work on the paper text if that would help, so feel
free to use your judgement as to who you want to include.
Also, because I bet you don't hear it enough and I know I don't say it
enough: thank you for writing an extremely useful package! We used it in
our test-of-gravity paper and it did sterling work under trying
circumstances. I also took your advice to heart and released the entire
source code, no matter how grotty: https://github.com/aarchiba/triplesystem
Anne
On Thu, 24 Oct 2019 at 14:42, Dan Foreman-Mackey<EMAIL_ADDRESS>wrote:

@aarchiba https://github.com/aarchiba: Would you like to be a co-author
on this submission as well given your work on #319
https://github.com/dfm/emcee/pull/319?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dfm/emcee/issues/316?email_source=notifications&email_token=AABBNIYDXOLZXIBIU6KRLCDQQGQ2VA5CNFSM4JB7HH5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECFCFQY#issuecomment-545923779,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABBNIY3XX5HBARC44AJIXDQQGQ2VANCNFSM4JB7HH5A
.

After reviewing what I actually contributed, I think I am more justified to reside in the acknowledgements. But I will leave that decision to you and I am happy to provide feedback on the draft though I have no experience with the format/content that JOSS requires.

@aarchiba: I think that contributing a bug fix is certainly an important contribution and I'd be happy to have you as a co-author, but it's up to you!
@terhardt: Same to you! The format for a JOSS paper is pretty simple: https://joss.readthedocs.io/en/latest/submitting.html#what-should-my-paper-contain and comments are definitely welcome!
@ all: I'm hoping to submit by the end of the day today (NYC time), but we'll be able to make changes during the review so please send along comments, questions, suggestions, etc.

I'd like to directly encourage @aarchiba to join the paper. That was a non-trivial fix. Of course I am also just an amateur here, and all hail @dfm!

Agreed!!
I submitted the paper and it's now under review here: https://github.com/openjournals/joss-reviews/issues/1840
But please continue to send suggestions and/or your author info if you'd like to be a co-author! We can add names during the review.

Thank you, you're very kind, I'd like to accept.
name: Anne M. Archibald
orcid: 0000-0003-0638-3340
affiliation: University of Newcastle

Very kind of you to offer!  I just looked at the contributions graphs, and I haven't done anything since 2012, so no need to include me.  Congrats on the paper!

@ all: The JOSS review is just about finished so please let me know if you have any last comments about the paper or if you want to be added (or removed) from the author list. I'll try to get everything wrapped up for this review before tomorrow, but let me know if you need more time. Thanks!

PS the review is happening here: https://github.com/openjournals/joss-reviews/issues/1864

@dfm Nothing from me. Thanks again for adding me to the author-list!

The paper has been accepted! Time to update our CVs:
https://joss.theoj.org/papers/10.21105/joss.01864

Great news

The paper has now been posted to arXiv: http://arxiv.org/abs/1911.07688
Let me know if you would like the password to claim ownership of the paper on arXiv.

Hi Dan,
Just on general principles, I try to claim all my papers on the arXiv---so
I'd like to claim ownership of this one.
Cheers,
Will
On Mon, Nov 18, 2019 at 10:35 PM Dan Foreman-Mackey <
<EMAIL_ADDRESS>wrote:

The paper has now been posted to arXiv: http://arxiv.org/abs/1911.07688
Let me know if you would like the password to claim ownership of the paper
on arXiv.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dfm/emcee/issues/316?email_source=notifications&email_token=AAAV7QADJ27DGP2LNQCGYIDQUNNI5A5CNFSM4JB7HH5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEMXTWA#issuecomment-555317720,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAV7QEXRQO3UEL4CXXHPLLQUNNI5ANCNFSM4JB7HH5A
.

@dfm Same here - will you please share the paper password.

@dfm Password please too (and thanks!)

@dfm Me too, please.

@dfm  I'd like to claim it as well if you don't mind.


Fix degenerate arc issue

@microsoft-github-policy-service agree

I checked locally that the unit test fails without the bug fix code change, so it is successfully testing the issue

Thanks for the contribution! I'm wondering if we should put the guard on the svg import code. 🤔

I’m not sure quite what you mean, but if it’s helpful for evaluating what to do here, the reason I needed this change is because I need to round trip a makerjs model through an svg path, and depending on the exact situation, makerjs produces a model that holds these degenerate arcs which it was exporting successfully and then failing on import.

I see. If possible, a repro of that degeneration might be helpful.

Thanks! Published 0.18.1


Docs

Codecov Report

Merging #3 (9baa785) into main (e2d534f) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main       #3   +/-   ##
=======================================
  Coverage   99.10%   99.10%
=======================================
  Files          21       21
  Lines        1563     1563
=======================================
  Hits         1549     1549
  Misses         14       14

Impacted Files
Coverage Δ

Sources/Sass/Color.swift
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Update cs-221-reflex-models.md template

Hi @shervinea,
I notice that this line which is a list of titles is missing Residual section.

Thanks @rootonchair for the pull request. This indeed appears as something that should have been there. Thanks!


Pipeline: self-signed-cert git integration and other settings

Address issue: https://github.com/rancher/docs/issues/1018

LGTM


BlockUseHandler

Mistake

BlockUseHandler doesn't exist anymore and ItemUseHandler is depreciated.

My proposed solution

correct the handler to whatever it was changed to
Wiki page

4a right click with item or click on a block [ItemUseHandler and BlockUseHandler]
Slimefun Version

	<groupId>com.github.TheBusyBiscuit</groupId>
	<artifactId>Slimefun4</artifactId>
	<version>master-RC-7-ga1952ed-1279</version>

They both still exist and are not deprecated.
You are importing them from the wrong location, they are in io.github.thebusybiscuit.slimefun4.core.handlers
Also, you are building against an insanely old version, use "RC-14" as the version.


Superfluous #num in examples/logic.eve?

Is #num in  examples/logic.eve#L4 redundant:
 [#div #num text: "If you see this, it's an error."]

It's not used in that example, but there's nothing wrong with it. It helps make that div more specific as now you could find all the divs generated by that with a rule like so:
Find num divs
  div = [#div #num]

It'll be a pretty common pattern to have combinations of tags and names as a way to classify different objects in the system.

OK, I'll close this issue.


tree expand subfolder doesn't work anymore

VSCode Version: Update 1.27.2
OS Version: mac 10.13.6

Steps to Reproduce:

Option + click folder with subfolder doesn't expand subfolders.

*Collapsing all folder works fine with Option + click, when it is expanded.

Does this issue occur when all extensions are disabled?: Yes

Dup of https://github.com/Microsoft/vscode/issues/9456


exit but not use all memory

when run sample with Vulkan in archlinux(with 16G memory),
It panic when try to create_buffer, but from memory monitor, not all memory used.
when try to use OpenGL backend, can sample the image, but it's pixel all dark.
thread 'main' panicked at 'wgpu error: Validation Error

Caused by:
    In Device::create_buffer
      note: label = `Buffer Src`
    Not enough memory left

', /home/ensonmj/Codes/wgpu/wgpu/src/backend/direct.rs:3095:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Do you have 16 GB of GPU VRAM or 16 GB CPU RAM? SD requires at least 8 GB of GPU VRAM.


A JavaScript Error ocurred in the main process

When I run this app, Show this error message
Can tell me how to solve this error ,tks

Hi, thank you for creating an issue, I think you need to enable Aero. This windows feature is required in order to create transparent windows.

thanks so much, I can run this app


bump patch for crate

just a suggestion, as in msys2 we want to have fix for *-gnullvm target applied at stable so we can update it for needed packages (e.g. starship)

Is this something you have experience with and can attempt? Maybe it's as easy as updating a submodule. Even if not, maybe it's an experience worth having as I am typically relying on update-contributions for this and am happy to get these released as often as they happen.

we need an update to compile the crate for aarch64-pc-windows-gnullvm target (MSYS2). an old version included arm64_neon.h which is MSVC only. so there was a build failure

I can contribute if needed only tomorrow

Yes, please feel free to contribute the update. Unless I am missing something, I'd expect it to pass CI without hitches, and thus is eligible for a release.

The update has already happened in https://github.com/rust-lang/libz-sys/pull/162
It's just not published as a new version and that would be the only necessary action.

Even better! Then it would be best if someone could create a PR with the version change so I can merge it and create a new release. If I would create that PR I can't review it myself which has the potential for delays.


Incorrect default mfa_configuration value of OFF instead of NONE in latest amplify_outputs.json

Environment information
System:
  OS: macOS 14.4.1
  CPU: (10) arm64 Apple M1 Pro
  Memory: 211.27 MB / 32.00 GB
  Shell: /bin/zsh
Binaries:
  Node: 18.19.0 - /opt/homebrew/opt/node@18/bin/node
  Yarn: 1.22.19 - /opt/homebrew/bin/yarn
  npm: 10.2.3 - /opt/homebrew/opt/node@18/bin/npm
  pnpm: undefined - undefined
NPM Packages:
  @aws-amplify/backend: 1.0.3
  @aws-amplify/backend-cli: 1.0.4
  aws-amplify: 6.3.6
  aws-cdk: 2.145.0
  aws-cdk-lib: 2.145.0
  typescript: 5.4.5
AWS environment variables:
  AWS_STS_REGIONAL_ENDPOINTS = regional
  AWS_NODEJS_CONNECTION_REUSE_ENABLED = 1
  AWS_SDK_LOAD_CONFIG = 1
No CDK environment variables

Description
The amplify_outputs.json value for auth configuration  contains the following field/value"mfa_configuration": "OFF".  This is causing Amplify configuration error on Android native applications due to a deserialization error for the incorrect/unsupported value.  The expected value is NONE.
Auth backend definition:
export const auth = defineAuth({
  loginWith: {
    email: true,
  },
});

Here's the generated auth configuration in amplify_outputs.json:
  "auth": {
    "user_pool_id": "us-east-1xxxxxxx",
    "aws_region": "us-east-1",
    "user_pool_client_id": "xxxxxxxxx",
    "identity_pool_id": "us-east-1:xxxxxxxx",
    "mfa_methods": [],
    "standard_required_attributes": [
      "email"
    ],
    "username_attributes": [
      "email"
    ],
    "user_verification_types": [
      "email"
    ],
    "mfa_configuration": "OFF",
    "password_policy": {
      "min_length": 8,
      "require_numbers": true,
      "require_lowercase": true,
      "require_uppercase": true,
      "require_symbols": true
    },
    "unauthenticated_identities_enabled": true
  },

Noticed this in the amplify_outputs.json. Marking this as bug.

note: this is missing from the parsing logic in aws-amplify https://github.com/aws-amplify/amplify-js/blob/de6b06db80b02687e597aceaa2c33f65171a6389/packages/core/src/parseAmplifyOutputs.ts#L328-L335. Despite the schema specifying NONE as a valid option in place of OFF, OFF is passed through from the CFN value to the generated outputs file, and the library is correctly parsing it as "OFF"


OpenAPI specification is missing info property

Describe the bug
The generated Swagger OpenAPI specification is missing the required info attribute.
To Reproduce
Steps to reproduce the behavior:

Go to datahub
Append /openapi/v3/api-docs to URL

Expected behavior
OpenAPI specification requires the info property. see here
The info property is described here
Additional context
If the info property is not available in the json file. The specifcation cannot be imported to Azure API Management

Hi @jan-wacker,
Can you please guide what needs to be added in the "title" and "version" tags of the info property. Additionally, can you also please provide the values of any tags of the info property which are required to be added.
Thanks

As described in the OpenAPI Requirements here the following tags are required:

title: string The title of the API.
version: string The version of the OpenAPI document (which is distinct from the OpenAPI Specification version or the API implementation version).

"title": "DataHub OpenAPI",
"version": "1.0.1"

Version tag needs to be generated.

Hi @jan-wacker,
I have created a PR for this issue.
Thanks


Safari BigInt Polyfill needed

A polyfill for BigInt / bigInt is required for Safari to enable the negotiator to work on Iphone and older Mac OS's such as Catalina.

we have multiple ways to solve it, but do we really need it?
@colourful-land , do we need bigInt for ticketClass and ticketId? maybe<PHONE_NUMBER>740991 (JS MAX_SAFE_INTEGER = 2**53 - 1) can be enough for devcon?


Duplicating class names: MProductGroupBuilder

v0.8.0, There are

com.kazurayam.materialstore.reduce.MProductGroup.Builder, which is a inner public static class
com.kazurayam.materialstore.reduce.MProductGroupBuilder, which is an external public class

The names are too similar, too confusing.
Should be changed.

MProductGroupBuilder --- this name is long, difficult to remember, does not imply what it does. Better class name is deserved.

Why not rename com.kazurayam.materialstore.reduce.MProductGroupBuilder to com.kazurayam.materialstore.reduce.Reducer.

renamed tod "Reducer" at v0.8.3-SNAPSHOT


python310Packages.openstacksdk: 0.103.0 -> 1.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.openstacksdk is: An SDK for building applications to work with OpenStack
meta.homepage for python310Packages.openstacksdk is: https://github.com/openstack/openstacksdk
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 1.0.0 with grep in /nix/store/cx9qrnchp3f5393i9vbfffsksw5wqvh1-python3.10-openstacksdk-1.0.0
found 1.0.0 in filename of file in /nix/store/cx9qrnchp3f5393i9vbfffsksw5wqvh1-python3.10-openstacksdk-1.0.0

Rebuild report (if merged into master) (click to expand)

19 total rebuild path(s)

19 package rebuild(s)

First fifty rebuilds by attrpath
heatclient
ironicclient
manilaclient
openstackclient
python310Packages.openstacksdk
python310Packages.osc-lib
python310Packages.python-heatclient
python310Packages.python-ironicclient
python310Packages.python-manilaclient
python310Packages.python-openstackclient
python310Packages.python-swiftclient
python311Packages.openstacksdk
python311Packages.osc-lib
python311Packages.python-heatclient
python311Packages.python-ironicclient
python311Packages.python-manilaclient
python311Packages.python-openstackclient
python311Packages.python-swiftclient
swiftclient

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/cx9qrnchp3f5393i9vbfffsksw5wqvh1-python3.10-openstacksdk-1.0.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.openstacksdk https://github.com/r-ryantm/nixpkgs/archive/b68532f4e618866b079983da70b1aecba21a3525.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/cx9qrnchp3f5393i9vbfffsksw5wqvh1-python3.10-openstacksdk-1.0.0
ls -la /nix/store/cx9qrnchp3f5393i9vbfffsksw5wqvh1-python3.10-openstacksdk-1.0.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

7 packages failed to build:

python311Packages.openstacksdk
python311Packages.osc-lib
python311Packages.python-heatclient
python311Packages.python-ironicclient
python311Packages.python-manilaclient
python311Packages.python-openstackclient
python311Packages.python-swiftclient

7 packages built:

heatclient (python310Packages.python-heatclient)
ironicclient (python310Packages.python-ironicclient)
manilaclient (python310Packages.python-manilaclient)
openstackclient (python310Packages.python-openstackclient)
python310Packages.openstacksdk
python310Packages.osc-lib
swiftclient (python310Packages.python-swiftclient)

Maintainer pings
cc @emilytrau @SuperSandro2000 for testing.

darwin is broken for a longer time, feel free to update the test exclusion for it, I don't have the time to debug this.
python 3.11 should be marked broken because a core openstack dependency does not work with it and it it probably going to take a bit until it is.


Configure automated testing in Travis

So that pull requests etc are automatically tested.
(You'll need to visit https://travis-ci.org to actually enable the build hook)

@paulfurley If you can bump this so 3.3 is changed to 3.4, I'll merge it.

Done :)
On 29 Jun 2015 16:44, "Steven Maude"<EMAIL_ADDRESS>wrote:

@paulfurley https://github.com/paulfurley If you can bump this so 3.3
is changed to 3.4, I'll merge it.
—
Reply to this email directly or view it on GitHub
https://github.com/scraperwiki/data-services-helpers/pull/24#issuecomment-116737498
.

Thanks! Just need someone to actually enable Travis; I'm not sure how I'd get access to do that @drj11 @scraperdragon.


Copy-DbaDatabase fails when destination is a mountpoint

On version 1.0.123
The destination server have the same folder structure as the source, but one of the destination folders is a drive mapped to a path (which we use on all our SQL Servers instead of drive letters)
Copy-DbaDatabase -Source sqlmigratesql2.agr.bah -Destination sqlmigratesql4.agr.bah -DestinationSqlCredential $sacred -AllDatabases -DetachAttach -ReuseSourceFolderStructure -Reattach
First a warning is displayed:
WARNING: [08:58:26][Mount-SqlDatabase] Issue mounting database. | Unable to open the physical file
"c:\mssql\crmdata\agrtmp_Primary.mdf". Operating system error 2: "2(The system cannot find the file specified.)".
CREATE DATABASE failed. Some file names listed could not be created. Check related errors.
Type                    Name                 Status  Notes

Database                xxxxxxxbooking       Skipped Already exists on destination
Database (DetachAttach) agrtmp               Failed  Failed to attach database to destination
And when trying to do that database the following error occurs:
Start-BitsTransfer : The destination volume has changed. If the disk is removable, it might have been replaced with a d
ifferent disk. Reinsert the original disk and resume the job.
At C:\Program Files\WindowsPowerShell\Modules\dbatools\1.0.123\allcommands.ps1:5327 char:29

...             Start-BitsTransfer -Source $from -Destination $remotefile ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [Start-BitsTransfer], Exception
FullyQualifiedErrorId : StartBitsTransferCOMException,Microsoft.BackgroundIntelligentTransfer.Management.NewBits
TransferCommand

It seems that the Start-BitsTransfer is the culprit and the code for the transfer (Start-SqlFileTransfer) never falls back to the copy-item command in the third try-catch block (which it seems the code intended to)
Trying the command below result in the same error
Start-BitsTransfer -Source .\myfile.txt -Destination \sqlmigratesql4.agr.bah\c$\mssql\crmdata\myfile.txt
System, Administrators and MSSQLSERVER have full permissions on the folder (and mapped drive) - relaxing the permissions doesn't solve the BITS problem, so the bug is that the Copy-Item never gets executed

The intended behavior would be achieved by adding -ErrorAction Stop to the Start-BitsTransfer commands in this senario, but that might not cover all needs


Favorites Folder for quick access to commonly used resources

Is your feature request related to a problem? Please describe.
As a user, it is hard to sometimes search for resources; if many things I work on are on the stack. Even if I search, I may misclick and accidently do something wrong to a resource.
Describe the solution you'd like
A favorites directory, in which I can favorite resources to be housed in there. For example, if there are 200 lambdas; but I only work on three of them; I would favorite the three and it would be stored under Favorites -> AWS Lambda.
(Nice to have) The favorited resources are stored locally in a JSON object so I can delete them from the folder from there.
Describe alternatives you've considered
I have used search as well, but if things are named similar it can lead to errors. I am only human and can click the wrong thing.
With a favorite's directory, I can favorite test resources so even if I misclick; I know it will not affect production.
Additional context
Thats it, very awesome vs code extension.

Thanks for the feature request @hanifzaijohn  - to clarify, are you referring to the AWS Toolkit for Visual Studio, or the AWS Toolkit for VS Code?

@awschristou It is for https://marketplace.visualstudio.com/items?itemName=AmazonWebServices.aws-toolkit-vscode

Thanks for clarifying. I will move this issue to the AWS Toolkit for VS Code repo.


[Regostry Preview] Convert data (binary) values

Description of the new feature / enhancement
Currently the data values (Multi_SZ, Expand_SZ, ...) are shown with their binary data in the data grid. If possible we should show them with user readable data.
UPDATE: We can convert everything that is not a string to the same layout as used in Regedit. For the complex types binary and multi_sz we could add a small preview dialog to show the string translation too.
Scenario when this would be used?
To see what value is really set.
Supporting information
No response

This has been worked on for the 0.74.0 release. Please update PowerToys to the latest version. 😉
https://github.com/microsoft/PowerToys/releases


Added a progress bar in skills and language section in resume_builder.html and designed using css

🛠 Fixes Issue (Number)

👨💻 Changes proposed

✅ Check List (Check all the applicable boxes)

[x] My code doesn't break any part of the project (Zero Octave-Javascript-Projects).
[x] This PR does not contain plagiarized content.
[x] My Addition/Changes works properly and matches the overall repo pattern.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

Helpful


Added Dark and Light mode feature

@MOHAMMED-IQRAMUL brother i have added dark and light mode feature , now please merge it brother

Resolve #35

Resolve #35


Router ignore  tag when use pushState = true

Router ignore <base/> tag when pushState = true I have to add:
config.options.root = document.getElementsByTagName('base')[0].href.replace(location.protocol + "//" + location.host, "");
To get work it.
The documentation says nothing about it.

@bryanrsmith Can you include this in the router docs you are working on?

Unless this is something which was changed, I don't have to do this in order to get it working. However the base has to be as:
<base href="/" />

I have:
<base href="/au/">

But doesn't work if I don't set options.root.

@tolemac To clarify, are you saying that if you set a base tag you expect the options.root to be set automatically from that?

The documentation (http://aurelia.io/hub.html#/doc/article/aurelia/framework/latest/cheat-sheet/7 section Configuring PushState) says you have to add a base tag and configure the router this way:
export class App {
  configureRouter(config) {
    config.title = 'Aurelia';
    config.options.pushState = true;
    config.map([
      { route: ['welcome'],    name: 'welcome',     moduleId: 'welcome',      nav: true, title:'Welcome' },
      { route: 'flickr',       name: 'flickr',      moduleId: 'flickr',       nav: true, auth: true },
      { route: 'child-router', name: 'childRouter', moduleId: 'child-router', nav: true, title:'Child Router' },
      { route: '',             redirect: 'welcome' }
    ]);
  }
}

Setting options.pushState to true, it says nothing about options.root.
Then I had understood that it should be enough, therefore Yes, I expect the options.root to be set automatically.
The framework is ignoring base tag or the docs are wrong ... IMHO ;)

Updated the docs concerning this setting. It will go out in the next docs update release this week.


Smoldyn simulation failing

Failed sim:  https://run.biosimulations.dev/simulations/60ca36c5ec4fbe336094932b
Log:
INFO:    Using cached SIF image
(EE)
Fatal server error:
(EE) Server is already active for display 1
        If this server is no longer running, remove /tmp/.X1-lock
        and start again.
(EE)
/usr/local/lib/python3.9/site-packages/biosimulators_utils/warnings.py:31: BioSimulatorsWarning: The SED document is potentially incorrect.
  - Model `model` may be invalid.
    - The model file `model.txt` may be invalid.
      - No validation is available for models encoded in `urn:sedml:language:smoldyn`
  warnings.warn(termcolor.colored(message, Colors.warning.value), category)
/usr/local/lib/python3.9/site-packages/biosimulators_utils/warnings.py:31: BioSimulatorsWarning: The COMBINE/OMEX archive may be invalid.
  - The SED-ML file at location `./simulation.sedml` may be invalid.
    - Model `model` may be invalid.
      - The model file `model.txt` may be invalid.
        - No validation is available for models encoded in `urn:sedml:language:smoldyn`
  warnings.warn(termcolor.colored(message, Colors.warning.value), category)
 succeeded

  Executed 1 tasks and 2 outputs:
    Tasks:
      Succeeded: 1
      Skipped: 0
      Failed: 0
    Outputs:
      Succeeded: 2
      Skipped: 0
      Failed: 0

Bundling outputs ...
Cleaning up ...

============= SUMMARY =============
Executed 1 SED documents:
  SED documents (1):
    Succeeded: 1
    Skipped: 0
    Failed: 0
  Tasks (1):
    Succeeded: 1
    Skipped: 0
    Failed: 0
  Outputs (2):
    Succeeded: 2
    Skipped: 0
    Failed: 0

/xvfb-startup.sh: line 6: kill: (3413906) - No such process
srun: error: xanadu-55: task 0: Exited with exit code 1
srun: Terminating job step 3065144.1

Potential solution here: https://github.com/ssandrews/Smoldyn/pull/91

The same solution should be copied to

[ ] Biosimulations/tools/xvfb-startup.sh
[ ] Biosimulators_pyNeuroML/scripts/xvfb-startup.sh

The Smoldyn image should be fixed by biosimulators/Biosimulators#255

@bilalshaikh42 could you try the Smoldyn run again with the revised image?

Updated error
=============Running docker image for simulator=============
INFO:    Using cached SIF image
(EE)
Fatal server error:
(EE) Server is already active for display 1
	If this server is no longer running, remove /tmp/.X1-lock
	and start again.
(EE)
/usr/local/lib/python3.9/site-packages/biosimulators_utils/warnings.py:31: BioSimulatorsWarning: The SED document is potentially incorrect.
  - Model `model` may be invalid.
    - The model file `model.txt` may be invalid.
      - No validation is available for models encoded in `urn:sedml:language:smoldyn`
  warnings.warn(termcolor.colored(message, Colors.warning.value), category)
/usr/local/lib/python3.9/site-packages/biosimulators_utils/warnings.py:31: BioSimulatorsWarning: The COMBINE/OMEX archive may be invalid.
  - The SED-ML file at location `./simulation.sedml` may be invalid.
    - Model `model` may be invalid.
      - The model file `model.txt` may be invalid.
        - No validation is available for models encoded in `urn:sedml:language:smoldyn`
  warnings.warn(termcolor.colored(message, Colors.warning.value), category)
 succeeded

  Executed 1 tasks and 2 outputs:
    Tasks:
      Succeeded: 1
      Skipped: 0
      Failed: 0
    Outputs:
      Succeeded: 2
      Skipped: 0
      Failed: 0

Bundling outputs ...
Cleaning up ...

I see the message, but the simulation runs and the final error appears to be gone. Does the run actually fail?
To cleanup the error message, we could try adding the following before Xvfb $DISPLAY
if [ -f "/tmp/.X1-lock" ]; then
  rm /tmp/.X1-lock
fi

Yes, the run step is marked as failed on slurm
On Thu, Jun 17, 2021 at 7:36 PM Jonathan Karr @.***>
wrote:

I see the message, but the simulation runs and the final error appears to
be gone. Does the run actually fail?
To cleanup the error message, we could try adding the following before Xvfb
$DISPLAY
if [ -f "/tmp/.X1-lock" ]; then
rm /tmp/.X1-lockfi
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/biosimulations/Biosimulations/issues/2641#issuecomment-863627245,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHX4FIFHSVS5VXR52UVQIS3TTKBGDANCNFSM462PETWQ
.

I think this can be closed now.
I mistakenly pushed the same image again. I fixed this. I was able to run a simulation successfully (https://run.biosimulations.org/simulations/60cca9fe69720db446b98c68).
Regarding the first error (If this server is no longer running ...), I haven't been able to replicate this locally. I don't see this message in the log for 60cca9fe69720db446b98c68 either. If we see this again, I think we could try running with a different value of the DISPLAY environment variable: singularity run --env "DISPLAY=:1000" ...? If that helps, we can reflect that change in the Docker image.

I believe this is fixed.


[Screen Reader - Open and close New Tab]: Same name as "Windows Powershell" is defined for all the tabs.

User Experience:
Users who rely on screen reader would be impacted and not able to get which tab is open as same name "Windows Powershell" is defined for all the tabs.
Test Environment:
OS: Win 10 2004 Build (20262.1010)
App: Windows Terminal
App version: 1.4.3243.0
Tool: Accessibility Insights for Windows
Repro Steps:

Open Windows terminal app.
Press "Ctrl+Shift+T"  to open multiple tabs.
Now run accessibility insight tool on the application.
Observe the issue.

Actual Result:
Same name as "Windows Powershell" is defined for all the tabs. So when screen reader focus navigates to any tab then narrator just announces it as "Windows powershell editing space".
Expected Result:
Unique name should be defined for each tabs.
For e.g. Name should be defined as "Windows powershell 1" for 1st tab item and so on.
MAS Reference:
https://microsoft.sharepoint.com/:w:/r/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={248054a6-5e68-4771-9e1e-242fb5025730}&cid=e71aec14-4173-423e-aecb-2472bbc751e8

MAS4.1.2_Same name defind for each tab.zip

Isn't this by-design? The client application (powershell.exe) is responsible for setting the title of the tab, not the Terminal.

@Chandresh1007 yes, please clarify. What happens in a normal web browser when the same tab is opened multiple times?

@DHowett We agree that web browsers also don't handled multiple tabs with same name in a different way, but here, it's easy to fix the bug by prefixing the string such as: "tab 1", "tab2" in front of the console processes such as 1st powershell session etc.
In current scenario, user won't be able to differentiate between different tabs while switching.
Also, a command should be added so that user can rename the tab as per his/her comfort.

Also, a command should be added so that user can rename the tab as per his/her comfort.

That already exists in a bunch of different forms. There's:

the renameTab action
the openTabRenamer action
The user can right-click the tab to open the renamer:
They can just double-click the tab
They could also set the title from their shell

@zadjii-msft But how do keyboard user can access that as Keyboard tab focus does not navigate to the Tabs. Is there any key defined for renaming the tab?

Not by default, but the user can bind any key they want to renameTab/openTabRenamer. They can also use the action as it's bound in the Command Palette to access the action even without binding a key to it:

@zadjii-msft
It's absolutely fine, however, we'll be very thankful to you folks if a default keybinding could be added to this action.
It will enhance the user experience as user won't have to manually asign the keybinding since it is quite a common action.

Not by default, but the user can bind any key they want to renameTab/openTabRenamer. They can also use the action as it's bound in the Command Palette to access the action even without binding a key to it:

Feature team does not consider this a bug. It is the same as navigating between multiple PowerShell windows.
It may be a duplicate of #8474 (“screen reader does not announce position of tabs.”)
Hence Closing the bug.


Update LDK for to fix inflight updates issue

This is to fix: https://github.com/MutinyWallet/mutiny-web/issues/1032
commit is here: https://github.com/MutinyWallet/rust-lightning/commit/0e2c116d1fa9d8db25e0364466dcacf4c7366c33
waiting on @thebluematt's confirmation but this should be all we need to get this user unblocked

Confirmed with Matt on discord


refactor hmr config parse feature

What problem does this feature solve?
In the current project, the function of loading hmr configuration is too complicated. I hope that the relevant functions can be reconstructed into the following logic:

use user config. when hmr_port is be used by other sw, throw error (in this case, farm cannot guarantee the availability of self-increasing ports. eg. reverse proxy)
in default, hmr use 'auto' path and port (determined by browser,same as what browser use (not farm's default config))

more discussion is welcome, if there are no other comments I will submit a PR later.😊
What does the proposed API look like?
nothing changed

PR welcome, as long as there's not breaking changes


Production administration not working

Hi,
Your demo site is not working, the administration page throws following error
index.js:4 Uncaught ReferenceError: LIB_VERSION is not defined
at Object.UlOv (index.js:4)
at o (bootstrap f0ffbbae6c894842955b:54)
at Object.YVUD (Search.vue:25)
at o (bootstrap f0ffbbae6c894842955b:54)
at Object.1 (backend.fe9554f2f51fe7cedb14.js:1)
at o (bootstrap f0ffbbae6c894842955b:54)
at window.webpackJsonp (bootstrap f0ffbbae6c894842955b:25)
at backend.fe9554f2f51fe7cedb14.js:1
You need to update the vue-chartjs to v3.2.1

Thx 4 feedback bro !
Fixed.


TypeError: t.pcap['First packet time'] is undefined

When i press "Show the full report" button, browser debug console shows error:
TypeError: t.pcap['First packet time'] is undefined. ...................................................................................vue.runtime.esm.js:3047:16
r report.vue:1
VueJS 14
click report.vue:1
VueJS 33
Any ideas how to fix it?

I also delayed my comment. All errors disappeared (which were in the previous post, too). Actions that helped:

author corrected requirements.txt file
laptop with fresh install Raspberry Pi Desktop (bullseye 32bit). Everything went smoothly without any extra steps.
another external usb wireless adapter. The previous one supported AP mode and worked well in linux and windows, but for some reason there were problems with SpyGuard. Maybe a modification or a different driver was needed.


[import wizard] add multiVarCol mapping functionality

adds the mapping functionality for a multi var col template

Notes:

does not include making the UI pretty or layout changes from the mock

downloaded tmcf:

This is very cool!
I wonder can you add some kind of test from UI change to the TMCF text? This will be very helpful for future changes.

We actually have tests for all the util functions (the functions that generate the TMCF, example observations, validation, etc): https://github.com/datacommonsorg/website/tree/master/static/js/import_wizard2/utils
And because the UI is going to be updated, maybe better to add tests for interacting with UI later?

This is very cool!
I wonder can you add some kind of test from UI change to the TMCF text? This will be very helpful for future changes.

We actually have tests for all the util functions (the functions that generate the TMCF, example observations, validation, etc): https://github.com/datacommonsorg/website/tree/master/static/js/import_wizard2/utils
And because the UI is going to be updated, maybe better to add tests for interacting with UI later?

That SG. I feel eventually we want to have some golden .tmcf files for testing. Even UI changes, but they would always generate the same TMCF.

ah yes, golden tmcf files makes sense!
Thanks for the review!


Romoving users with a certain role

I am trying to to finish my database seeder. I would like to know if there are any methods that elets all the users from a certain role or group

You mean delete all users from the database which are in some role? Or you mean remove users from a role, but keep them in the database?
For first, for v1.0 you can use Roles.getUsersInRole to get a cursor of all users and then you can iterate over and remove them.
For second you could combine this with removeUsersFromRoles.
In v2.0 you can use Roles._usersInRoleQuery to get a query which you can use directly to remove users from the database.


HOMEMATICIP_LOCAL-XmlRPC-Server received no events.

DONT'T DELETE THIS.
Please answer all questions as good as possible.
Language: english/german
Make sure you have read the documentation at https://github.com/danielperna84/custom_homematic#custom_homematic.
Make sure you have read the release notes at https://github.com/danielperna84/custom_homematic/releases.
Make sure you are running the latest version of the custom_component (and Home Assistant) before reporting an issue.
custom_component version (if applicable):
1.17.0
known last working custom_component version (if applicable):
1.x.x
Home Assistant version (if applicable):
2022.9.6
CCU version:

OS: RaspberryMatic
Version: <IP_ADDRESS>20831

Problem-relevant configuration:
Do you use tls?
no
Do you use callback?
no
Do you use username and password?
yes
Which interfaces do you use (hmip/bidcos/wired)?
bidcos
Describe the bug
I get multiple times a day this HA notification:
HOMEMATICIP_LOCAL-XmlRPC-Server received no events.
What triggers this notification?
How to disable it?
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Screenshots (if applicable)

CCU firewall
integrations view
device view
enity view

Relevant HA log entries (full log)

Diagnostic Information (available on every device)

Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.

Please add the log entry (full log)

What triggers this notification?
We EXPECT Events from CCU within 5 minutes. If this is missing, we create a message.

How to disable it?

Restart CCU?
Restart HA?

Please add the log entry (full log)

What triggers this notification?
We EXPECT Events from CCU within 5 minutes. If this is missing, we create a message.

How to disable it?

Restart CCU? Restart HA?

Restart of neither CCU nor HA doesn‘t help

Please add the log entry (full log) !!!

Please add the log entry (full log) !!!

ok,
hopefully you can find something?
Homematic_Log.txt

Looks like HA does not receive events (Updates) from CCU for  the HmIP Interface.
Check the logs on your CCU.
Are the devices accessible(controllable) from CCU WebUI?

I think it's a missconfiguration from my side. I do not have HmIP devices. I've reconfigured the HA Integration and checked only the box for port 2001 Bidcos-RF and not for HmIP.
Maybe this solves the issue?
Devices can be controlled from HA and CCU WebUI.

Yes


Incorrect difference when comparing varchar key between Oracle and RDS Postgres

Giving inaccurate differnce when comparing the varchar key value  between  a table in Oracle and RDS Postgres.

Orgin difference shows correct , destination difference  just dumps the whole key list even though they exist

The same issue I found between Postgres and BigQuery. More details are here.

relates to https://github.com/datafold/data-diff/issues/550

No Worries, i have found a alternate way of finding the differences without the data-diff. The package will be of no use if it cannot follow some basic rules when comparing..

Hey @skmanchala can you add some detail?
This is not enough to be actionable on its own:

Giving inaccurate differnce when comparing the varchar key value between a table in Oracle and RDS Postgres.
Orgin difference shows correct , destination difference just dumps the whole key list even though they exist

However this scenario may become apparent in the resolution of #550

We have a Varchar primary key in both RDS  Postgrest and Oracle which i am using to compare the data, as mentioned previously , the record in source not in target shows up correcly  but after that it shows all the records in destination not present in source

Hi @skmanchala,
I'm sorry for the delay in following up on this. Thank you for taking the time to raise this issue!
We made a hard decision to sunset the data-diff package and won't provide further development or support.
If that's of interest, over the past few months, we have rewritten the diffing engine in Datafold Cloud and solved many issues that existed in this package's diffing algorithm.
-Gleb


Remote dev: Local images in hovers aren't loading properly

Issue Type: Bug
Originally reported: https://github.com/eamodio/vscode-gitlens/issues/734#issuecomment-492055255
In GitLens, I am loading an svg image off disk (installed with the GitLens vsix) for the presence indicator dot, which works fine normally, but when used with remote dev it shows as a broken image: (the You are offline part is the alt-text of the image)

Here is the code I'm currently using for it.
https://github.com/eamodio/vscode-gitlens/blob/d8654c6cd1465da8f5290bd9b5bb6e775cc6f679/src/git/formatters/commitFormatter.ts#L137-L140
Is this something I can change/fix somehow?
VS Code version: Code - Insiders 1.35.0-insider (94f7de61ece171f4390ece488a220d22cbb0fddd, 2019-05-23T05:19:33.101Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz (8 x 4008)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.93GB (16.66GB free)

Process Argv

Screen Reader
no

VM
0%

I think you can use Data URLs. You can load an image file on a remote machine, and pass the image to VSCode as a Data URL. For example,
const dataurl = 'data:image/svg+xml;base64,' + fs.readFileSync(filename).toString('base64')
const imgMarkdown = `![AltText](${dataurl})`

We can use Data URLs in markdown on VSCode although that is not explicitly documented.
I hope Data URLs in markdown will be explicitly documented and supported. That seems essential for extensions to work with VS Code Remote Development.

@tamuratak That's a great idea! Thanks!


Job Output Directory Support For Genie 3.0

This pull request enables handling of the job output directory for Genie 3.0.
Since Genie 3.0 runs using Spring Boot, which includes an embedded Tomcat, instead of within an independent Tomcat process the output directory processing had to change.
In Genie 2.x we would configure the job output directory as another "web application" via symlink to a directory on the filesystem within the tomcat webapps directory. From there Tomcat would handle serving that content using the DefaultServlet and enabling directory listing.
This wouldn't work out of the box in Spring Boot. Also the old solution kicked the user out of the Genie context which would cause a lot of problems for the new security put in place via #180. Basically the anyone could have accessed the output files which obviously could contain data that people shouldn't have access to.
This PR contains code which adds a REST API under /api/v3/jobs/{id}/output/** which handles listing directories and serving static files. Static file support is done using Springs' ResourceHttpRequestHandler. This leaves in place all the advanced file handling they already have. Support for directories is accomplished by extending that class and checking if the requested resource on disk is a directory. If it is the handling bypasses Springs implementation and can return either a HTML or JSON representation of the directory contents.
The other important feature of this PR is forwarding of requests for these static resources. In Genie 2.x due to the design explained above with the symlinked directory there was no control in Genie what resources the users were trying to request as there was no application code around the requests. This PR adds a feature that allows admins to turn on or off Job Output Directory forwarding. When run as a cluster this allows users to send requests for resources to any Genie node and have it forwarded to the node where the resource "should" be (unless it was deleted by some janitor type process). This can be turned off if the job directory is backed by some shared NFS type directory like EFS. If it is on though it removes the "stickiness" of needing to send requests to the proper nodes in Genie 2 and allows running Genie behind a load balancer or similar.
The HTML output of the directories are likely to change but want to get this code merged into the baseline to avoid conflicts with other development efforts going on.

Current coverage is 75.11%

Merging #183 into develop will decrease coverage by -2.95% as of 01b4644

@@            develop    #183   diff @@
=======================================
  Files            86      91     +5
  Stmts          2585    2724   +139
  Branches        231     247    +16
  Methods           0       0
=======================================
+ Hit            2018    2046    +28
- Partial          50      52     +2
- Missed          517     626   +109

Review entire Coverage Diff as of 01b4644

Powered by Codecov. Updated on successful CI builds.

Looks good.It's weird there is no library out there just lets us do the listing automatically instead of us rendering the page in Genie. Will work well for us for the time being though.


chore: fix test, add some options

Added some options, tests & file descriptions.
Fixed existing tests and some little issue.
Updated deps, gas, tsd.

@all-contributors please add @PierreDemailly for code, test


ubuntu-bionic-arm64 build failed with "175 errors, 61 warnings"

Problem
Build "ubuntu-bionic-arm64" target failed, with logs "175 errors, 61 warnings" at last.
I aslo checked td-agent/apt/repositories/ directory, it's empty.
The builder (container) is still running at last.
Build environment
my vps host is ubuntu bionic on x86_64 platform, 1CPU and 1 GB RAM.
How to reproduce
Follow the instructions in Readme to build
export TARGET_BASE="ubuntu-bionic"
sudo apt install qemu-user-static
cd /path/to/td-agent-builder
cp /usr/bin/qemu-aarch64-static td-agent/apt/${TARGET_BASE}
rake apt:build APT_TARGETS="${TARGET_BASE}-arm64"

Expected
Build successfully
Actual
Build failed
Builder logs
I check the builder logs using "docker logs -f  860a5cd80e74" and find there are some ruby extensions failed.

At last, it failed with "175 errors, 61 warnings"

This is the log file dumped by "docker logs  860a5cd80e74 > td-agent-builder.log "
td-agent-builder.log

Build "ubuntu-bionic-arm64" target failed, with logs "175 errors, 61 warnings" at last.

These warnings and errors are reported by a package checking tool (lintian), building the package itself is succeeded . Most of warnings or errors of lintian is false-positive or ignorable. Same warning or errors are reported also on my environment which can build the package correctly.

This is the log file dumped by "docker logs 860a5cd80e74 > td-agent-builder.log "
td-agent-builder.log

It seems that building the package is finished by not yet copied to the host.
Here is the tail of my log on finishing successfuly to build the package.
N: 236 tags overridden (175 errors, 61 warnings)
Finished running lintian.
/build
cd -
cd -

my vps host is ubuntu bionic on x86_64 platform, 1CPU and 1 GB RAM.

The memory amount might not be enough.

The memory amount might not be enough.
Thanks, that may be the case. The instance's configuration is too low and take much time to build. I will upgrade it later and try to build again.

And another question: can i costomize gem plugins into td-agent using the builder.
Acctually, i'm compiling a fluentd container image from kolla project on platform ububtu/arm64. I checked the fluentd's Dockerfile , it needs more gem plugins. and i want to bundler these into td-agent to. How can i do it ?

And another question: can i customize gem plugins for td-agent using the builder?
Acctually, i'm compiling a fluentd container image from kolla project on platform ubuntu/arm64. I checked the fluentd's Dockerfile , it needs more gem plugins. and i want to bundle these into td-agent . How can i do it ?

See https://github.com/clear-code/td-agent-builder#how-to-bump-up-the-package-version

Edit td-agent/plugin_gems.rb
Commit it by git commit -a
Rebuild the package

If you can't build it even if you increase amount of memory, please reopen this issue.

I upgrade the instance with 2 CPU and 4GB RAM and rebuild td-agent. There seems nothing improved except the time cost. The log is still same as before.
N: 236 tags overridden (175 errors, 61 warnings)
Finished running lintian.

That is a bit different from yours. There is no
/build
cd -
cd -

It seems that building the package is finished but not yet copied to the host.

I log into the builder (container), and  "td-agent_3.7.1-1_arm64.deb" is in /build directory. Is the package size (9.8M) ok ? If it's ok, i can copy it to host by "docker cp".

Is the package size (9.8M) ok ?

OK, it's expected size range.


#244 check the JQ version before converting the instances

Fixes #244

PS: Can you add a short description to the branch name, ie, frosner/issue-244-check-jq-version instead frosner/issue-244?  Makes it easier to checkout the branch when you've forgotten the issue number, and makes git branch -a more informative :blush:


Updating project dependencies & CI/CD scripts

NPM libraries and project setup in package.json hasn't been changed
for a long time....
Reviewing all the existing libraries, removed unnecessary libs and upgraded
all remaining to recent versions (#177). Took the opportunity to refresh the CI/CD
setup. Allowed me to remove lots of old style configuration.
Added a new node_modules size check from #158 & #159

This is ready for review. Highlights include....

Upgraded all libs to most recent versions, removed lots of unnecessary dev dependencies.
Add safety check on node_modules size to CI/CD to stop accidental issues.
Re-factored CI/CD setup to be much simpler.

I've got more changes to make before the release but didn't want this PR to get even bigger...


About valuation

您好，我有两个问题想请教您一下：
1.代码中评估用的“score”计算方法似乎没有在论文中体现，请问这个评价指标如何去理解，pth模型在各个数据集上的“score”值的测试能否提供官方数据；
2.论文中F1和auc的值的计算，您是否能提供一下测试程序，计算结果似乎有一定偏差。
Hello, I have two questions that I would like to ask you:
1.The "score" calculation method used for evaluation in the code does not seem to be reflected in the paper. How to understand this evaluation indicator, and can official data be provided for testing the "score" values of the PTH model on various datasets;
2.Can you provide a testing program for the calculation of F1 and AUC values in the paper? The calculation results seem to have some deviation.

可以要一份评估代码吗？谢谢！


worker: remove libvirt from optional dependencies

We no longer require libvirt in the worker image for virtualized
testbots, remove it from optional dependencies to save time.
Change-type: patch
Signed-off-by: Joseph Kogut<EMAIL_ADDRESS>

@balena-ci rebase


cargo-expand: 1.0.90 -> 1.0.91

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/dtolnay/cargo-expand/releases.
meta.description for cargo-expand is: Cargo subcommand to show result of macro expansion
meta.homepage for cargo-expand is: https://github.com/dtolnay/cargo-expand
meta.changelog for cargo-expand is: https://github.com/dtolnay/cargo-expand/releases/tag/1.0.91
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.0.91 with grep in /nix/store/63wk8j3999d65j0q1h72f1npvvllqcal-cargo-expand-1.0.91
found 1.0.91 in filename of file in /nix/store/63wk8j3999d65j0q1h72f1npvvllqcal-cargo-expand-1.0.91

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

cargo-expand
irust

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/63wk8j3999d65j0q1h72f1npvvllqcal-cargo-expand-1.0.91 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cargo-expand https://github.com/r-ryantm/nixpkgs/archive/195ab8b0ee16d8db6f5c465b4c53962a59c6d947.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/195ab8b0ee16d8db6f5c465b4c53962a59c6d947#cargo-expand

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/63wk8j3999d65j0q1h72f1npvvllqcal-cargo-expand-1.0.91
ls -la /nix/store/63wk8j3999d65j0q1h72f1npvvllqcal-cargo-expand-1.0.91/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 2 packages built:

cargo-expand
irust

Maintainer pings
cc @figsoda @xrelkd for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

@NixOS/nixpkgs-merge-bot merge


LIFX Exception in callback _SelectorDatagramTransport._read_ready()

Home Assistant release with the issue:
0.98.2
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):
HassOS 2.12 rpi3
Hass.io supervisor: Version 185
Component/platform:
https://www.home-assistant.io/components/lifx/
Description of problem:
Turned on a LIFX Z and saw the log littered with the traceback below. Same call stack each time. Once per second until light was disconnected from power.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
#LOAD PLATFORM
lifx:

Traceback (if applicable):
2019-09-04 08:45:27 ERROR (MainThread) [homeassistant.core] Error doing job: Exception in callback _SelectorDatagramTransport._read_ready()
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/asyncio/events.py", line 88, in _run
    self._context.run(self._callback, *self._args)
  File "/usr/local/lib/python3.7/asyncio/selector_events.py", line 965, in _read_ready
    self._protocol.datagram_received(data, addr)
  File "/usr/local/lib/python3.7/site-packages/aiolifx/aiolifx.py", line 170, in datagram_received
    getattr(self,setmethod)(response)
  File "/usr/local/lib/python3.7/site-packages/aiolifx/aiolifx.py", line 989, in resp_set_multizonemultizone
    self.color_zones[i] = resp.color[i-resp.index]
IndexError: list assignment index out of range

Additional information:

What do you mean by "turned on" ... does this happen all the time or just this one time?
How many zones does the LIFX Z have?

It's reproducible. Just plug in to wall and wait about a minute before the logs start. There are seven 1M strands attached to the controller.

I am sorry, I do not see that issue. My LIFZ Z just has two 1M segments though. I am using LIFZ 2, firmware 2.77.
If you can dump the values of i, resp.index and resp.count or do a Wireshark packet capture, probably we can fix this.


Added More Courses/Cards in Courses Page

Description
Hey @thestarsahil
I have added more Courses/Cards in Courses Page

Fixes #945

Screenshots

Checklist

[X] Tests have been added or updated to cover the changes
[X] Documentation has been updated to reflect the changes
[X] Code follows the established coding style guidelines
[X] All tests are passing

@thestarsahil Please merge this PR


Make pylint happy

PPO tuner is not touched because I can't understand reported functions. @QuanluZhang
I have not installed all tuner dependencies and disabled import-error locally, so it may be incomplete.

@liuzhe-lz  Just to clarify, which modules is this PR fixing?

With the output on nni SDK with current settings:
python3 -m pylint src/sdk/pynni/nni
************* Module nni.bohb_advisor.config_generator
src/sdk/pynni/nni/bohb_advisor/config_generator.py:258:12: E1137: 'return_array' does not support item assignment (unsupported-assignment-operation)************* Module nni.compression.tensorflow.builtin_quantizers
src/sdk/pynni/nni/compression/tensorflow/builtin_quantizers.py:30:4: W0235: Useless super delegation in method '__init__' (useless-super-delegation)src/sdk/pynni/nni/compression/tensorflow/builtin_quantizers.py:55:4: W0235: Useless super delegation in method '__init__' (useless-super-delegation)************* Module nni.compression.tensorflow
src/sdk/pynni/nni/compression/tensorflow/__init__.py:2:0: W0401: Wildcard import builtin_pruners (wildcard-import)
src/sdk/pynni/nni/compression/tensorflow/__init__.py:3:0: W0401: Wildcard import builtin_quantizers (wildcard-import)
************* Module nni.compression.torch.builtin_quantizers
src/sdk/pynni/nni/compression/torch/builtin_quantizers.py:30:4: W0235: Useless super delegation in method '__init__' (useless-super-delegation)
src/sdk/pynni/nni/compression/torch/builtin_quantizers.py:56:4: W0235: Useless super delegation in method '__init__' (useless-super-delegation)
************* Module nni.compression.torch
src/sdk/pynni/nni/compression/torch/__init__.py:2:0: W0401: Wildcard import builtin_pruners (wildcard-import)
src/sdk/pynni/nni/compression/torch/__init__.py:3:0: W0401: Wildcard import builtin_quantizers (wildcard-import)
************* Module nni.curvefitting_assessor.curvefunctions
src/sdk/pynni/nni/curvefitting_assessor/curvefunctions.py:103:4: E1111: Assigning to function call which doesn't return (assignment-from-no-return)
************* Module nni.curvefitting_assessor.model_factory
src/sdk/pynni/nni/curvefitting_assessor/model_factory.py:311:12: E1111: Assigning to function call which doesn't return (assignment-from-no-return)
************* Module nni.evolution_tuner.test_evolution_tuner
src/sdk/pynni/nni/evolution_tuner/test_evolution_tuner.py:54:32: C0326: Exactly one space required after comma
                "_type":"choice","_value":["Adam", "SGD"]
                                ^ (bad-whitespace)
src/sdk/pynni/nni/evolution_tuner/test_evolution_tuner.py:26:0: C0411: standard import "from unittest import TestCase, main" should be placed before "import numpy as np" (wrong-import-order)
************* Module nni.gp_tuner.target_space
src/sdk/pynni/nni/gp_tuner/target_space.py:142:61: E1102: self.dim is not callable (not-callable)
************* Module nni.hyperopt_tuner.hyperopt_tuner
src/sdk/pynni/nni/hyperopt_tuner/hyperopt_tuner.py:59:20: E1111: Assigning to function call which doesn't return (assignment-from-no-return)
************* Module nni.networkmorphism_tuner.graph
src/sdk/pynni/nni/networkmorphism_tuner/graph.py:801:8: E0401: Unable to import 'keras' (import-error)
src/sdk/pynni/nni/networkmorphism_tuner/graph.py:905:4: E0401: Unable to import 'onnx' (import-error)
************* Module nni.networkmorphism_tuner.layers
src/sdk/pynni/nni/networkmorphism_tuner/layers.py:33:4: W0235: Useless super delegation in method '__init__' (useless-super-delegation)
src/sdk/pynni/nni/networkmorphism_tuner/layers.py:412:4: W0235: Useless super delegation in method '__init__' (useless-super-delegation)
src/sdk/pynni/nni/networkmorphism_tuner/layers.py:471:4: W0235: Useless super delegation in method '__init__' (useless-super-delegation)
src/sdk/pynni/nni/networkmorphism_tuner/layers.py:528:4: E0401: Unable to import 'keras' (import-error)
src/sdk/pynni/nni/networkmorphism_tuner/layers.py:544:4: E0401: Unable to import 'keras' (import-error)
************* Module nni.networkmorphism_tuner.test_networkmorphism_tuner
src/sdk/pynni/nni/networkmorphism_tuner/test_networkmorphism_tuner.py:195:27: C0326: Exactly one space required after comma
        tuner.add_model(0.8,  0)
                           ^ (bad-whitespace)
src/sdk/pynni/nni/networkmorphism_tuner/test_networkmorphism_tuner.py:196:27: C0326: Exactly one space required after comma
        tuner.add_model(0.9,  1)
                           ^ (bad-whitespace)
************* Module nni.ppo_tuner.distri
src/sdk/pynni/nni/ppo_tuner/distri.py:153:15: E1120: No value for argument 'mask_npinf' in constructor call (no-value-for-parameter)
src/sdk/pynni/nni/ppo_tuner/distri.py:153:15: E1120: No value for argument 'nsteps' in constructor call (no-value-for-parameter)
src/sdk/pynni/nni/ppo_tuner/distri.py:153:15: E1120: No value for argument 'size' in constructor call (no-value-for-parameter)
src/sdk/pynni/nni/ppo_tuner/distri.py:153:15: E1120: No value for argument 'is_act_model' in constructor call (no-value-for-parameter)
************* Module nni.ppo_tuner.ppo_tuner
src/sdk/pynni/nni/ppo_tuner/ppo_tuner.py:540:0: C0330: Wrong continued indentation (remove 2 spaces).
                                        mb_dones, last_values, self.inf_batch_size)
                                      | ^ (bad-continuation)
src/sdk/pynni/nni/ppo_tuner/ppo_tuner.py:239:12: E1137: 'mb_advs' does not support item assignment (unsupported-assignment-operation)
src/sdk/pynni/nni/ppo_tuner/ppo_tuner.py:584:27: W1202: Use % formatting in logging functions and pass the % parameters as arguments (logging-format-interpolation)
src/sdk/pynni/nni/ppo_tuner/ppo_tuner.py:585:80: C1801: Do not use `len(SEQUENCE)` to determine if a sequence is empty (len-as-condition)
src/sdk/pynni/nni/ppo_tuner/ppo_tuner.py:25:0: W0611: Unused import os (unused-import)
src/sdk/pynni/nni/ppo_tuner/ppo_tuner.py:29:0: W0611: Unused import json_tricks (unused-import)
************* Module nni.ppo_tuner.util
src/sdk/pynni/nni/ppo_tuner/util.py:59:4: W0612: Unused variable 'nbatch' (unused-variable)
************* Module nni.smac_tuner.smac_tuner
src/sdk/pynni/nni/smac_tuner/smac_tuner.py:282:20: E1111: Assigning to function call which doesn't return (assignment-from-no-return)

------------------------------------------------------------------
Your code has been rated at 9.85/10 (previous run: 9.39/10, +0.46)

@leckie-chn For your pylintrc output:

PPO tuner is not covered in this PR
Unit tests should not be required to pass pylint
I disabled useless-super-delegation until we find a way to write derived constructor docstring
unsupported-assignment-operation and assignment-from-no-return are because numpy not marked as generated-members in rc file
The other three files strangely passed pylint on my machine and will be updated soon

@leckie-chn For your pylintrc output:

PPO tuner is not covered in this PR
Unit tests should not be required to pass pylint
I disabled useless-super-delegation until we find a way to write derived constructor docstring
unsupported-assignment-operation and assignment-from-no-return are because numpy not marked as generated-members in rc file
The other three files strangely passed pylint on my machine and will be updated soon

@liuzhe-lz https://github.com/microsoft/nni/blob/041c3f0e2d1f0020f8c30e422094a07bb88b9b2d/pylintrc#L45 Here we have a rule for disabling check for Unit tests.

@leckie-chn Suggest use ignore-patterns.

@leckie-chn For your pylintrc output:

PPO tuner is not covered in this PR
Unit tests are not required to pass pylint (though recommended maybe)
I disabled useless-super-delegation until we find a way to write derived constructor docstring
unsupported-assignment-operation and assignment-from-no-return are because numpy not marked as generated-members in rc file
wildcard-import is disabled for __init__.py in latest pylint, see here
The last problem in GP tuner is strangely ignored by latest pylint (fixed below)

output w/ pylint 2.4.3, removing all UTs and PPO modules:
************* Module nni.bohb_advisor.config_generator
src/sdk/pynni/nni/bohb_advisor/config_generator.py:312:69: E1136: Value 'train_configs.shape' is unsubscriptable (unsubscriptable-object)
src/sdk/pynni/nni/bohb_advisor/config_generator.py:313:72: E1136: Value 'train_configs.shape' is unsubscriptable (unsubscriptable-object)
src/sdk/pynni/nni/bohb_advisor/config_generator.py:321:11: E1136: Value 'train_data_good.shape' is unsubscriptable (unsubscriptable-object)
src/sdk/pynni/nni/bohb_advisor/config_generator.py:321:39: E1136: Value 'train_data_good.shape' is unsubscriptable (unsubscriptable-object)
src/sdk/pynni/nni/bohb_advisor/config_generator.py:323:11: E1136: Value 'train_data_bad.shape' is unsubscriptable (unsubscriptable-object)
src/sdk/pynni/nni/bohb_advisor/config_generator.py:323:38: E1136: Value 'train_data_bad.shape' is unsubscriptable (unsubscriptable-object)
************* Module nni.compression.tensorflow.builtin_quantizers
src/sdk/pynni/nni/compression/tensorflow/builtin_quantizers.py:30:4: W0235: Useless super delegation in method '__init__' (useless-super-delegation)src/sdk/pynni/nni/compression/tensorflow/builtin_quantizers.py:55:4: W0235: Useless super delegation in method '__init__' (useless-super-delegation)************* Module nni.compression.torch.builtin_quantizers
src/sdk/pynni/nni/compression/torch/builtin_quantizers.py:30:4: W0235: Useless super delegation in method '__init__' (useless-super-delegation)     src/sdk/pynni/nni/compression/torch/builtin_quantizers.py:56:4: W0235: Useless super delegation in method '__init__' (useless-super-delegation)     ************* Module nni.metis_tuner.metis_tuner
src/sdk/pynni/nni/metis_tuner/metis_tuner.py:358:43: E1133: Non-iterable value results_outliers is used in an iterating context (not-an-iterable)   ************* Module nni.networkmorphism_tuner.graph
src/sdk/pynni/nni/networkmorphism_tuner/graph.py:801:8: E0401: Unable to import 'keras' (import-error)
src/sdk/pynni/nni/networkmorphism_tuner/graph.py:905:4: E0401: Unable to import 'onnx' (import-error)
************* Module nni.networkmorphism_tuner.layers
src/sdk/pynni/nni/networkmorphism_tuner/layers.py:33:4: W0235: Useless super delegation in method '__init__' (useless-super-delegation)
src/sdk/pynni/nni/networkmorphism_tuner/layers.py:412:4: W0235: Useless super delegation in method '__init__' (useless-super-delegation)
src/sdk/pynni/nni/networkmorphism_tuner/layers.py:471:4: W0235: Useless super delegation in method '__init__' (useless-super-delegation)
src/sdk/pynni/nni/networkmorphism_tuner/layers.py:528:4: E0401: Unable to import 'keras' (import-error)
src/sdk/pynni/nni/networkmorphism_tuner/layers.py:544:4: E0401: Unable to import 'keras' (import-error)

unsubscriptable-object is caused by https://github.com/PyCQA/pylint/issues/3139
For disabling useless-super-delegation if you want to locally disable it, please add the comment #pylint: disable=useless-super-delegation.
not-an-iterable is the 1 true real warning that needs to be fixed.

@leckie-chn

See our old pylintrc file. generated-members=numpy can stop pylint to analyze numpy types.
I believe it's a good practice to refine parent class' docstring. And I don't want to make good things verbose. If a class has a useless __init__ without docstring, it will be later detected by missing-docstring.
It looks like an iterable to me. results_outliers is returned by gp_oulier_detection.outlierDetection_threaded, which is defined in nni/metis_tuner/Regression_GP/OutlierDetection.py. The function returns either a list or None, and there is a is not None checking. So this should be a pylint bug.


Use SXSSFWorkbook in XlsExporter

to improve memory usage when exporting xlsx files
indirectly related to https://github.com/OpenRefine/OpenRefine/issues/2593

Nice! It's convenient that the API remains the same even in streaming mode.
Looking at main/tests/server/src/com/google/refine/exporters/XlsExporterTests.java we do not seem to have tests for XLSX export so I am a bit worried about test coverage here (although it's not your fault of course). So we need to test this manually and add a unit test which covers the change.

There is XlsxExporterTests and this tests xlsx cases.
The XlsExporterTests (for xls files) seems not to have much coverage.

Ah thanks, funny that they were split into two test classes, that's something I should clean up. Yes overall we have a pretty pathetic test coverage in this project. Ok, I'll test this manually and merge.


Back-porting fix #669 to new class in feature branch

NetflixOSS » eureka » eureka-pull-requests #471 SUCCESS
This pull request looks good


Update golang version to 1.11

Updating golang version to 1.11 hoping to use new features (e.g. modules).

My personal recommendation go switch to ffmpeg-debian (I also maintain them) for sake of stability and glibc compatibility.

@BattleBas tags were updated, but I created another tag: 4.0-golang-1.11 specifically for your demand.

@denismakogon  thank you very much!
Would it be possible to also update your gocv-alpine image on hub.docker.com ? I was hoping to use the updated ffmpeg image with it.

Yup,’gonna take care of that today.


⚠ Bofrost.lu has degraded performance

In 9e0cc7a, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1215 ms

Resolved: Bofrost.lu performance has improved in 7d52de9.


How to do this if the url contains a parameter?

How do I make this work with a path variable? Suppose the URL is /user/123 for example.
The controller would have something like
@PostMapping("/user/{userId}")
public String updateUser(@PathVariable("userId") long id, @Valid PersonForm personForm, BindingResult bindingResult) {
  if (bindingResult.hasErrors()) {
            return "form";
        }

        return "redirect:/user/{userId}";
}

I tested this and the redirect in the success case works, but the error case seems to work at first, but if you then update the form again and submit again, it does not go through the same controller method. So how to make this work?

Not sure I follow the bit about "does not go through the same controller". If you POST to the same endpoint, it has no choice but to do so, I should think. Maybe you could fork the project and push a change to your fork so we can see what you mean?

Hi,
I created a fork of the project to better explain what I mean.
You can find the changes here: https://github.com/wimdeblauwe/gs-validating-form-input/commit/b567f27b51189df9b584caf58d5a71cc82783ada
You can check out the branch pathvariable-example from https://github.com/wimdeblauwe/gs-validating-form-input to test it.
This is what happens:

I open the url http://localhost:8080/user/123 in my browser. This gives me an empty form.
I press on 'Submit'. This shows the validation messages as expected. If I type something invalid, the form data is retained. The url stays at http://localhost:8080/user/123
Press 'Submit' again with invalid data. The url jumps to http://localhost:8080/user/0 suddenly.

In my actual application, the url jumps to http://localhost:8080/user/, but that might be because I use objects as the path variable and not a long.
I would appreciate you having a look to the changes and let me know if I am doing something wrong or if there is maybe a bug somewhere?
There is also a question on this on SO (http://stackoverflow.com/questions/18137326/springmvc-controller-how-to-stay-on-page-if-form-validation-error-occurs) I found, but I think the answers could be better.

In your @GetMapping you set the "id" property on the form object, but in the @PostMapping you don't. I think that's all there is to it.

But then why does it work the first time I submit? Why does the url stay correct there?

Because that was the URL that you submitted the form to. But the id is 0, so the HTML is different the second time you see that form, and it submits to the wrong URL on the third visit.

ok. Works if I set the path variable on the form object in case the validation fails. Thx for helping me out!


Ping is not a constructor

I am trying to use ping in my Angular/Cordova application in the following way:
let ipList = [{ query: 'www.tiste.org', timeout: 1, retry: 3, version: 'v4' },
            { query: 'www.somesite.com', timeout: 2, 'retry': 3, version: 'v6' }];
            let success = function (results) {
                console.log(results);
            };
            let err = function (e) {
                console.log('Error: ' + e);
            };

            let p = new Ping();
            p.ping(ipList, success, err);

But am getting the error:
ERROR TypeError: Ping is not a constructor

Actually, creating this plugin, I had no idea how to properly require or import modules while using cordova (and I still don't have much clue about it). This script was mainly built to be used as an inline <script> and to expose some globals, which seems to be the problem when you try to import the script with a more modern import approach.
Will try to investigate more about importing this module, but feel free to open a PR if you know how to do it properly.

Hmm, I'm now trying to use it as:
<script type="text/javascript" src="../node_modules/cordova-plugin-ping/www/ping.js"></script>
from my index.html
but am getting the error:

Yes!!! I have figured it out. Turns out I was doing too much for the importing part of it. No need to import it at all. Just needed to specify declare var Ping; at the top of my file so typescript wouldn't throw errors.
If I get some time, maybe I can create a type definition file for TypeScript and I could submit a PR.

Also for any future person using Angular with this, I had to enclose the content of the success callback in a ngZone like so (don't forget to import ngZone from @angular/core and inject it into your component):
        // Use the Ping plugin to determine if the domain name of the specified
        // host can be resolved. Try using both IPv4 and IPv6.
        let ipList = [
            { query: entry.host, timeout: 2, retry: 2, version: 'v4' },
            { query: entry.host, timeout: 2, retry: 2, version: 'v6' }
        ];

        let ping = new Ping();

        ping.ping(ipList, success => {
            component.ngZone.run(() => {
                let requestList: any[] = success;
                 ... // The rest of your handling goes here.
           }); // End of Zone
        }, err => console.log(err));

@crlang44 glad you could make it work and sorry not to have been able to help much...
Sure, you can open a PR regarding TypeScript usage. As you may have noticed, I'm not familiar (yet?) with TypeScript, but wanting to learn it for some time, so reviewing your eventual PR may not be immediate.
Thanks for your investigation anyway

did you have to declare the variable ngZone in the constructor?
I also have a problem with "component" ("can not find name 'component'")


Rename project from 'smlj' to 'morel'

Rename project from 'smlj' to 'morel'.

Fixed in 11d1846dc8bbef691fb81a4c08cd3a2cddfb5fd6.
When we release morel-0.2 we can change the remaining references to smlj in Maven central.

Test scripts (see https://github.com/julianhyde/morel/tree/master/src/test/resources/script) still have the suffix .sml. This is deliberate. The scripts contain the language Standard ML (albeit with extensions).


Added new mouse layout to support mouse Button4 & Button5

Added new mouse layout: 'Layout_Buttons1-5_Forward+Back', for a standard 5 button mouse, adding Button4 & Button5

Thank you for your pull request; however, it’s not necessary as you can customize your own mouse layout.
Please refer to the instruction video for detailed guidance:
https://youtu.be/ClGztJtR2rQ?si=5fMRxXHRkDIzGMA0

The most the layout customization video doesn't show how to add mouse button 4 and 5 to a layout and none of the built-in layouts included these buttons. Am I missing something?

If the keys are "unlocked" you can hover above them and click the wanted keybind. The text changes but the icon does only change when you save it.
https://youtu.be/ClGztJtR2rQ/&t=0m24s

Got it figured out.  Thanks


v0.0.40

Codecov Report

Merging #96 (5d17d71) into main (dbc0204) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #96   +/-   ##
=======================================
  Coverage   82.48%   82.48%
=======================================
  Files          12       12
  Lines         925      925
  Branches      261      261
=======================================
  Hits          763      763
  Misses        162      162

Continue to review full report at Codecov.

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


Windows: use $LIBRARY_PREFIX and autoreconf

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Develop Index.html- hero section.

im working on  footer section but the hero section disable my footer section. i can not see with hero es able; that is a problem or a error or is a problem with my footer section.

Hi, i have just checked develop and the layout seems to flow as expected, please review and if its fixed close this issue, if not add instructions on how to replicate your issue.


Add complete information for each card and level

[{
"Name": "Archers",
"idName": "archers",
"Rarity": "Common",
"type": "Troop",
"Cost": "3",
"Range": "5",
"DeployTime": "1",
"HitSpeed": "1.2",
"arena": 0,
"elixirCost": 3,
"description": "A pair of unarmored ranged attackers. They'll help you with ground and air unit attacks, but you're on your own with coloring your hair.",
"statistics":   [
    {"Levels": "1", "Hitpoints":"125", "Damage": "40", "DPS": "33"},
    {"Levels": "2", "Hitpoints":"137", "Damage": "44", "DPS": "36"},
    {"Levels": "3", "Hitpoints":"151", "Damage": "48", "DPS": "40"},
    {"Levels": "4", "Hitpoints":"166", "Damage": "53", "DPS": "44"},
    {"Levels": "5", "Hitpoints":"182", "Damage": "58", "DPS": "48"},
    {"Levels": "6", "Hitpoints":"200", "Damage": "64", "DPS": "53"},
    {"Levels": "7", "Hitpoints":"220", "Damage": "70", "DPS": "56"},
    {"Levels": "8", "Hitpoints":"241", "Damage": "77", "DPS": "64"},
    {"Levels": "9", "Hitpoints":"265", "Damage": "84", "DPS": "70"},
    {"Levels": "10", "Hitpoints":"291", "Damage": "93", "DPS": "77"},
    {"Levels": "11", "Hitpoints":"320", "Damage": "102", "DPS": "85"},
    {"Levels": "12", "Hitpoints":"351", "Damage": "112", "DPS": "93"},
    {"Levels": "13", "Hitpoints":"386", "Damage": "123", "DPS": "102"}
]
}

]

+1

It is a great suggestion, but unluckily there is a LOT of different options like hitpoints, damage, dps, etc. and as Supercell updates the balance of the game really often it is quite difficult for me to keep that information updated.
I will leave this issue open, feel free to submit a pull request if you want help with this, I will try to work on this feature in the future.
Thanks!

@martincarrera Yes, it's almost impossible to do so unless people can contribute info or something like that which is not needed.

At least, could be possible to have cards level at tournament rules? It should be easier

does anyone have an idea where the game store card stats? I can decode csv files and gather that info.
another option is getting them from the wikia. I can also do that but we would echo any errors they have in their database so I'd prefer the 1st option.

@barreeeiroo There's too many cards, it takes a lot of time, but if other people can update info in like a separated page/web that @martincarrera can approve the changes it would be good and less time consuming.

@MaherFa if the game doesn't want others to access data/API it's kinda pointless to try to.
The best we can come up with is this repo.
I don't get it, clash of clans API is out why there's no official API for clash royale 😐

I'm already on it, I decoded characters.csv and found so many ordered useful information. I will not leave this issue until I have them all ordered in a table and ready to be implemented, It'll take some time tho. I will keep you guys updated here.

Option 1: Getting info from Wikia
I managed to write a simple code that extracts stats and info from wikia page tables without the images in th, it also includes name, idName and statsDate to keep track of latest update.
if we agree to use this method I will write special rules store numerical values as integers instead of strings
how to test:
go to http://clashroyale.wikia.com/wiki/Knight, open console and paste the following code
  var myRows = [];
  var $headers = $("#unit-statistics-table th");
  var $rows = $("#unit-statistics-table tr:not(:first-child)").each(function(index) {
      $cells = $(this).find("td");
      myRows[index] = {};
      $cells.each(function(cellIndex) {
          myRows[index][$($headers[cellIndex]).clone().children().remove().end().text().trim().toLowerCase().split(" ").join("-")] = $(this).text().trim();
      });
  });

  var myObj = {}
  myObj.name = $(".header-title h1").text().trim();
  myObj.idName = $(".header-title h1").text().trim().toLowerCase().split(" ").join("-")
  $("#unit-attributes-table tr th").each(function(index) {
      var thisTH = $(this).clone().children().remove().end().text().trim().toLowerCase().split(" ").join("-")
      var thisTD = $("#unit-attributes-table tr td").eq(index).text().trim()
      myObj[thisTH] = thisTD
  });
  myObj.stats = myRows;
  myObj.statsDate = new Date().toISOString()
  console.log(myObj);

you will get and object called myObj that looks like this:
  {
    "name": "Knight",
    "idName": "knight",
    "cost": "3",
    "hit-speed": "1.1 sec",
    "speed": "Medium (60)",
    "deploy-time": "1 sec",
    "range": "Melee",
    "target": "Ground",
    "count": "x1",
    "transport": "Ground",
    "type": "Troop",
    "rarity": "Common",
    "stats": [
        {
            "level": "1",
            "hitpoints": "660",
            "damage": "75",
            "damage-per-second": "68"
        },
        {
            "level": "2",
            "hitpoints": "726",
            "damage": "82",
            "damage-per-second": "74"
        },
        {
            "level": "3",
            "hitpoints": "798",
            "damage": "90",
            "damage-per-second": "81"
        },
        {
            "level": "4",
            "hitpoints": "877",
            "damage": "99",
            "damage-per-second": "90"
        },
        {
            "level": "5",
            "hitpoints": "963",
            "damage": "109",
            "damage-per-second": "99"
        },
        {
            "level": "6",
            "hitpoints": "1,056",
            "damage": "120",
            "damage-per-second": "109"
        },
        {
            "level": "7",
            "hitpoints": "1,161",
            "damage": "132",
            "damage-per-second": "120"
        },
        {
            "level": "8",
            "hitpoints": "1,273",
            "damage": "144",
            "damage-per-second": "130"
        },
        {
            "level": "9",
            "hitpoints": "1,399",
            "damage": "159",
            "damage-per-second": "144"
        },
        {
            "level": "10",
            "hitpoints": "1,537",
            "damage": "174",
            "damage-per-second": "158"
        },
        {
            "level": "11",
            "hitpoints": "1,689",
            "damage": "192",
            "damage-per-second": "174"
        },
        {
            "level": "12",
            "hitpoints": "1,854",
            "damage": "210",
            "damage-per-second": "190"
        },
        {
            "level": "13",
            "hitpoints": "2,039",
            "damage": "231",
            "damage-per-second": "210"
        }
    ],
    "statsDate": "2017-03-22T08:20:55.031Z"
}

Option 2: Getting info from game files
I managed to extract and decode official csv files, I narrowed down related files to these:
https://github.com/MaherFa/temp-csv-files
characters.decoded.csv
icludes data like Rarity SightRange DeployTime ChargeRange Speed Hitpoints HitSpeed LoadTime Damage 	DamageSpecial CrownTowerDamagePercent
• DPS is calculated using HitSpeed and Damage
• Ranged units doesn't have Damage value, but they have a Projectile value that can be used to get data from projectiles.decoded.csv
projectiles.decoded.csv
       icludes data about ranged units and some direct damage spells
character_buffs.decoded.csv
       icludes information about elements that affect characters like Rage, Heal, Poison and Freeze
buildings.decoded.csv
       buildings
area_effect_objects.decoded.csv
       area effect spells and cards
• some cards are listed with different names from game names, for example Executioner is called AxeMan
• stats are listed for Level 1 cards, sowe need to figure out the formula (multiplayer) to calculate stats for each level.
I believe this method is much better than the first option but it'll require more initial work. but once we have the script that extracts data from files we can automate the whole proccess for future updates.

@martincarrera I have no experience with MongoDB, but I can loop through wikia pages and generate a JSON with all the info they have, and I think it should be easy to merge the result with the existing database. what do you think?

Hi, I think the best way should be the first one.
If you could provide me a JSON with the complete data of all the cards, I can update the API model to support that information and update the API.
I could try to create a tiny script to run your code, but as I don't have a lot of time this days, it may take a couple of weeks.
Thanks!

already on it, I'm writing a node.js script that takes card names from an array and generate wikia links and http rquest them. I'm also using jsdom and jquery to run the code inside of node.
it should be ready within 48 hours max, depending on my free time.
Some mico issues:
• should key names be like hit-speed or hitSpeed?
• should I use damage-per-second/damagePerSecond or dps
• should speed include an integer or a string or both?  I think using a number is better for combaring cards
• I'm also thinking about removing the unit sec from time values and maybe multiply by 1000 to get rid of the decimals so "1.1 sec" would become 1100

Use camelCase if it's possible, such as hitSpeed instead of hit-speed.
dps should be ok
Take into account that some cards may have a range in some attributes, such as the inferno damage 20-400 instead of a simple value, or the goblin gang hitpoints 80/52. In this cases, use a string if possible.
I think the best would be to multiply it by 1000 so that all units are in milliseconds.

Could you add the attack type? Like if it attacks is area type or single objective

@barreeeiroo I'll focus on gathering data present in wikia page tables for now. but I will definitely look into this later.

@martincarrera Full JSON file is ready, where should I upload it?

Could you create a public gist and share the url?
I can work on that.
Thanks!

Here it is
it's a duplicate of your cards.json with same order and length with new keys.
also I spent some time to create a help file to clear everything for users, including units, types and special cases.
one thing to note, there was two cards with "needs confirmation" tags that we might need to confirm later.
Mega Minions Damage and Dark Prince Hitpoints
I hope you guys take a look at the file and point any possible issues and I'll work on them.
but YEAH we finally have it!!!!!

Is there a way to get tournament standards stats?
Like turLevel:  9 and we'll know to get level 9.

@amirm3hdi
I don't know if Martin have plans of implementing this but I see no point in adding this to every card, maybe a separate call with a value for each rarity?
{
    common: 9,
    rare: 7,
    epic: 4,
    legendary: 1
}

for others looking for a way to handle this, you can use conditions with hard coded indexes to get what you want for now:
console.log(getTournyStats(cardObject))

function getTournyStats(myCard) {
    switch (myCard.rarity) {
        case "Common":
            return myCard.stats[8];
            break;
        case "Rare":
            return myCard.stats[6];
            break;
        case "Epic":
            return myCard.stats[3];
            break;
        case "Legendary":
            return myCard.stats[0];
            break;
    }
}

@MaherFa I agree, a tiny code can achieve that.

@martincarrera
Can you add soon the levels to your API? All members of my clan are waiting for the levels of each card to add to my bot ;)
https://gist.github.com/MaherFa/2d143da0a707caa7532dfff708643a6d

@martincarrera if that JSON is hard to work with I can help if you point me to the recommended format you like the data in. I'm also perfecting the StatGrab bot for future updates, requests are welcomed.

added all available icons to be used with stats (https://github.com/martincarrera/clash-royale-api/pull/75/commits/1d27efaf2e9da731cb3cc69016df433bf063d8b8)

Should be a exception for in range for Mortar
range [ number ] Except "Goblin Gang" and Mortar
range [ string ] Goblin Gang, Mortar separated by a "/" min and max range values.
It says, 311 and should be "3/11"
Or something like that.

Is this still happening? Is anyone actively working on this api?

Hi @BenC14! I am working on this API on my free time - and to be honest I don't have much since a couple of months ago.
I will try to make this feature as soon as possible, but I can't promise any date or anything.
I will post any news in this thread.
Thanks!


[Feature] Update At.js dependency to upstream [OSF-6948]

Purpose
At.js was using our fork of the library until a fix for the contenteditable bugs was taken integrated. The changes were merged 🎉  so we don't need the fork anymore.
Changes

point to upstream instead of fork

Side effects
None
Ticket
https://openscience.atlassian.net/browse/OSF-6948

@mfraezz, this should actually work now 👍


display.get_screens() returns NotImplementedError:abstract

Trying to get a screen does not work, display.get_screens() returns NotImplementedError:abstract
pyglet version: 1.5.8
IDE: Spyder
Traceback (most recent call last):
File "C:\Users\x.spyder-py3\HelloWorld.py", line 12, in
screen = display.get_screens()
File "C:\Program Files\WPy64-3850\python-3.8.5.amd64\lib\site-packages\pyglet\canvas\base.py", line 98, in get_screens
raise NotImplementedError('abstract')
NotImplementedError: abstract

Just tried this out and it seemed to work fine:
Python 3.8.6 (tags/v3.8.6:db45529, Sep 23 2020, 15:52:53) [MSC v.1927 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import pyglet
>>> window = pyglet.window.Window()
>>> display = window.display
>>> display.get_screens()
[Win32Screen(x=0, y=0, width=1920, height=1080)]

How are you grabbing your display object?

Thanks LennyPhoenix. Intially, I tried this call (in Spyder) and got the reported error:
display = pyglet.canvas.Display()
screen = display.get_screens()
When I try this call - or the one you posted - in the Python shell and in Pyzo, it works. No idea what causes the error in Spyder.