Aria2 module settings

Description of changes
aria2 module settings, added nixos test and myself as maintainer
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 303429 run on x86_64-linux 1

1 package blacklisted:

nixos-install-tools

Thanks for your comments, I have tried to address all of them. Please let me know what you think.

I am getting error: The option 'services.aria2.settings' is used but not defined. on the latest nixos-unstable.
Was this meant to happen? From a brief look, it seems that services.aria2.settings should be defined with some default values.
@timhae

@jys1670 can you please post your services.aria2 settings in your nixconfig?

@jys1670 can you please post your services.aria2 settings in your nixconfig?
Yeah, sure:

  services.aria2 = {
    enable = true;
    rpcSecretFile = "${pkgs.writeText "aria" "aria2rpc\n"}";
  };

It's working fine if I revert this PR. Should something else be defined for default settings on latest nixos-unstable?

Sorry for breaking your config, can you change that to:
  services.aria2 = {
    enable = true;
    rpcSecretFile = "${pkgs.writeText "aria" "aria2rpc\n"}";
    settings = { };
  };

Sorry for breaking your config, can you change that to:
  services.aria2 = {
    enable = true;
    rpcSecretFile = "${pkgs.writeText "aria" "aria2rpc\n"}";
    settings = { };
  };

Thanks! This fixed the issue for me.

created PR #320035


List tasks as txt

List tasks as txt
Summary:
Last part of TasksController rewrite using Akka.HTTP. Prints out tasks in plaintext.
JIRA issues: MARATHON-7308

@meln1k @jeschkies should be ready for re-review. Thanks!


Check if an environment variable is set

In Bash, it's very common to check if an environment is set and non-empty with [[ -n "${VAR:-}" ]]. We should provide helpers for this, like so:
hasEnv :: Text -> IO Bool
hasEnv = not . missingEnv

missingEnv :: Text -> IO Bool
missingEnv name = null $ fromMaybe "" $ need name

Let me know if https://github.com/Gabriella439/turtle/pull/454 looks okay to you


[CHALLENGE SUBMISSION]

Please provide the following information for your submission:
What is your name? (First, Last)
Niall Kelly
Where is your GitHub Repo ?(With your challenge solution)
https://github.com/nkelly75/25-days-of-serverless/tree/master/day11
What Challenge is this for? (Challenge number between 1 and 25)
11
(Optional) Anything else we should know? e.g., language used, location, blog post?
2 node.js Functions, data in Cosmos DB, one function receives "wish" in JSON POST and writes to Cosmos, second function triggers on Cosmos and posts to Slack webhook (see Readme )

From all of us on the Cloud Advocacy team at Microsoft, thank you for creating this issue!
Have a Happy Holiday season!
Feel free to share your solution on Twitter with the tag #25DaysOfServerless

THANK YOU FOR YOUR #25DaysOfServerless SUBMISSION
We checked out your repository. We're thrilled to say your solution is now listed in our Hall of Fame
Thank you for your contributions!
This issue is now resolved! 🎉


gFHR Refs #381

Yes.
Yes.
Refs #381

@jortensi @mtnwalker if you want to look at the syntax before I merge this (need to update doco a tad first, and get a PR merged in MOOSE)

Job VTB Documentation on 2d404a5 wanted to post the following:
View the site here
This comment will be updated on new commits.

We'll have to wait for the submodule update to merge this, the new syntax is not supported with those

Test failure in blue crab is unrelated


Per-story interception not working

If I add handlers to initialHandlers in initialize() in preview.ts, they work correctly, however if I add them to the parameters of a story, they dont catch the request, and i get an error that the request has no matching handler:

If I add a query param to the handlers, i get the error about how the query params shouldnt be on the url, so it seems like the handlers themselves are getting loaded correctly

minimal reproducible example:
import React, { useState } from 'react';
import { graphql, http, HttpResponse, delay } from 'msw';

// This function will be used in the Storybook story to render the component.
export const MagicButtonStory = () => {
  const [message, setMessage] = useState('Click the button to see the magic!');

  const handleClick = async () => {
    try {
      const response = await fetch('/testendpoint');
      if (!response.ok) {
        throw new Error('Network response was not ok');
      }
      const data = await response.json();
      setMessage(data.message);
    } catch (error) {
      setMessage('Oops! Something went wrong.');
    }
  };

  return (

The Magic Button

{message}

      <button
        onClick={handleClick}
        style={{
          padding: '10px 20px',
          fontSize: '16px',
          backgroundColor: '#28a745',
          color: '#fff',
          border: 'none',
          borderRadius: '5px',
          cursor: 'pointer',
        }}>
        Click Me!
      </button>

  );
};

// Storybook configuration
export default {
  title: 'Magic Button Story',
  component: MagicButtonStory,
};

// Default story for the MagicButtonStory component
export const Default = {
  render: () => <MagicButtonStory />,
  parameters: {
    msw: {
      handlers: [
        http.get('/testendpoint?test=test', () => {
          console.log('==== msw data in handler');
          return HttpResponse.json([1, 2, 3]);
        }),
      ],
    },
  },
};

preview.ts:
import React from 'react';
import { initialize, mswLoader } from 'msw-storybook-addon';
import { graphql, http, HttpResponse, delay } from 'msw';

initialize();

export const parameters = {
  options: {
    theme: {
      brandTitle: `my-Web Repo`,
    },
    showPanel: true,
    panelPosition: 'bottom',
  },
  loaders: [mswLoader],
};

I dug into this a bit more, and it seems like the handlers arent being added to msw. If I console log out the worker, its handlers array is empty. Which is strange because its giving an error about the query params. Somehow its parsing the handlers but not adding them?

I have found a workaround which is to manually apply the request handlers in the story loader:
loaders: [
    async () => {
      await applyRequestHandlers([
        http.get('/testendpoint', () => {
          console.log('==== msw data in handler');
          return HttpResponse.json([1, 2, 3]);
        }),
      ]);
    },
  ],

I found the issue, I had loaders in the wrong place in my preview file, it was under parameters instead of along side it.
fixed preview.js:
import { initialize, mswLoader } from 'msw-storybook-addon';

initialize();

const preview = {
  parameters: {
    options: {
      theme: {
        brandTitle: `my-Web Repo`,
      },
      showPanel: true,
      panelPosition: 'bottom',
    },
  },
  loaders: [mswLoader],
};

export default preview;


Add LetterN and DigitN functions

Thanks for the pr. I see no reason to not merge this in after an example and benchmark gets added. Thanks!

@brianvoe added

Added v5.10.0


Ejh 20161202 annotator integration

Modifies the backend server to talk with the audio annotator front-end.
Includes demo audio and script to run the machinery locally
Tests updated
Unclear if the annotator HTML will render correctly through pages, but we'll find out when this comes back to master.

Note: The audio-annotator submodule was updated in the course of these changes.

This change is

Coverage increased (+0.1%) to 89.51% when pulling 0804e6d60eaec580434a4ed0dec67d3f307c9078 on ejh_20161202_annotator_integration into 38d7a6086cd993096e2ffcc51916f61a0a53765f on master.

Coverage increased (+0.1%) to 89.51% when pulling 0804e6d60eaec580434a4ed0dec67d3f307c9078 on ejh_20161202_annotator_integration into 38d7a6086cd993096e2ffcc51916f61a0a53765f on master.

Coverage increased (+0.1%) to 89.51% when pulling 0804e6d60eaec580434a4ed0dec67d3f307c9078 on ejh_20161202_annotator_integration into 38d7a6086cd993096e2ffcc51916f61a0a53765f on master.

Reviewed 13 of 16 files at r1.
Review status: 13 of 16 files reviewed at latest revision, 1 unresolved discussion.

README.md, line 60 at r1 (raw file):

This will start the backend server (CMS), upload a few audio files, and begin serving the audio annotation tool locally. By default this will appear at [http://localhost:8000/docs/annotator.html](http://localhost:8000/docs/annotator.html).

**Note**: For some reason, loading the audio seems to get "stuck" on occasion. To unblock it, manually proceed to an audio file's URL once the server is running, e.g. [http://localhost:8080/api/v0.1/audio/740c835f-a23d-41ef-b84c-0cd1de4edfa5](http://localhost:8080/api/v0.1/audio/740c835f-a23d-41ef-b84c-0cd1de4edfa5).

Is hard-coding this really useful? I imagine the hash could change, and break this link.

Comments from Reviewable

Review status: 13 of 16 files reviewed at latest revision, 2 unresolved discussions.

README.md, line 66 at r1 (raw file):

### Content Management System

See the [ReadMe](https://github.com/cosmir/open-mic/blob/master/backend_server/README.md) for details on running the backend web server.

Can this link be relative?

Comments from Reviewable

Review status: 13 of 16 files reviewed at latest revision, 3 unresolved discussions.

run_demo.sh, line 6 at r1 (raw file):

python backend_server/main.py --port 8080 --local --debug &
CMS_PID=$!
sleep 4s

This is okay for now, but this should really be a daemon that stores its PID to a user-specified pid file, which can be read once the initial process terminates.

Comments from Reviewable

README.md, line 60 at r1 (raw file):
Previously, bmcfee (Brian McFee) wrote…
Is hard-coding this really useful? I imagine the hash could change, and break this link.

UUIDs are deterministic based on the bytestring of the file; this points to a checked in file, and shouldn't change. That said, it's worth mentioning that in the README to prevent future confusion.

Comments from Reviewable

README.md, line 66 at r1 (raw file):
Previously, bmcfee (Brian McFee) wrote…
Can this link be relative?

¯_(ツ)_/¯ I'll look into it

Comments from Reviewable

run_demo.sh, line 6 at r1 (raw file):
Previously, bmcfee (Brian McFee) wrote…
This is okay for now, but this should really be a daemon that stores its PID to a user-specified pid file, which can be read once the initial process terminates.

I spent 10-15 minutes rooting around the interstacks, and didn't find an example I like -- do you have one in mind you could point to?

Comments from Reviewable

Review status: 13 of 16 files reviewed at latest revision, 1 unresolved discussion.

run_demo.sh, line 6 at r1 (raw file):
Previously, ejhumphrey (Eric J. Humphrey) wrote…
I spent 10-15 minutes rooting around the interstacks, and didn't find an example I like -- do you have one in mind you could point to?

How about start-stop-daemon?

Comments from Reviewable

run_demo.sh, line 6 at r1 (raw file):
Previously, bmcfee (Brian McFee) wrote…
How about start-stop-daemon?

looks like that's linux only? suppose that would work for travis, but us OS X folks couldn't run it locally ... this is at least unix compliant?

Comments from Reviewable

Reviewed 2 of 16 files at r1.
Review status: 15 of 16 files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Review status: 15 of 16 files reviewed at latest revision, 1 unresolved discussion.

run_demo.sh, line 6 at r1 (raw file):
Previously, ejhumphrey (Eric J. Humphrey) wrote…
looks like that's linux only? suppose that would work for travis, but us OS X folks couldn't run it locally ... this is at least unix compliant?

Ugh, OSX.  See thread: https://apple.stackexchange.com/questions/99888/is-there-any-port-of-start-stop-daemon-for-os-x
I guess that works via launchctl or launchd or something, but I have no way to verify.  Sorry!

Comments from Reviewable

run_demo.sh, line 6 at r1 (raw file):
Previously, bmcfee (Brian McFee) wrote…
Ugh, OSX.  See thread: https://apple.stackexchange.com/questions/99888/is-there-any-port-of-start-stop-daemon-for-os-x
I guess that works via launchctl or launchd or something, but I have no way to verify.  Sorry!

Doesn't seem to be the answer -- might need to use a different process shell?

Comments from Reviewable

Coverage increased (+0.1%) to 89.51% when pulling 9ccd709279e45131fa9862f86e7bbd80b0a5eecf on ejh_20161202_annotator_integration into 38d7a6086cd993096e2ffcc51916f61a0a53765f on master.

@bmcfee can I get a fresh look on this? I swapped out the bash shell script demo in favor of a python version, which is probably (hopefully) preferable to the previous solution.

Comments from Reviewable

run_demo.sh, line 6 at r1 (raw file):
Previously, ejhumphrey (Eric J. Humphrey) wrote…
Doesn't seem to be the answer -- might need to use a different process shell?

I've rm'ed this file in favor of demo.py

Comments from Reviewable

Reviewed 3 of 3 files at r3.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Coverage increased (+0.1%) to 89.51% when pulling 9f0d61891efb195b3e5d31c324429482413e1643 on ejh_20161202_annotator_integration into 38d7a6086cd993096e2ffcc51916f61a0a53765f on master.

Reviewed 1 of 16 files at r1.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

demo.py, line 39 at r3 (raw file):

    try:
        input("Press return to exit.")
    except:

don't do a bare catch here; use the correct exception class

demo.py, line 42 at r3 (raw file):

        pass
    os.killpg(os.getpgid(server.pid), signal.SIGTERM)
    os.killpg(os.getpgid(webapp.pid), signal.SIGTERM)

put subprocess kills in a finally block

Comments from Reviewable

Review status: all files reviewed at latest revision, 4 unresolved discussions.

demo.py, line 1 at r3 (raw file):

#!/usr/bin/python

use #!/usr/bin/env python instead of #!/usr/bin/python to ensure proper environment capsulization

Comments from Reviewable

Review status: all files reviewed at latest revision, 5 unresolved discussions.

demo.py, line 25 at r3 (raw file):

                               '8080', '--local', '--debug'],
                              stdout=subprocess.PIPE, preexec_fn=os.setsid)
    time.sleep(4)

this still seems wrong, but we can revisit later.

Comments from Reviewable

demo.py, line 1 at r3 (raw file):
Previously, bmcfee (Brian McFee) wrote…
use #!/usr/bin/env python instead of #!/usr/bin/python to ensure proper environment capsulization

Done.

Comments from Reviewable

demo.py, line 42 at r3 (raw file):
Previously, bmcfee (Brian McFee) wrote…
put subprocess kills in a finally block

Done.

Comments from Reviewable

demo.py, line 25 at r3 (raw file):
Previously, bmcfee (Brian McFee) wrote…
this still seems wrong, but we can revisit later.

I did something smarter here, you might want to give this a look.

Comments from Reviewable

Coverage increased (+0.1%) to 89.51% when pulling dd38b99db6dfbca73805f068717b73d71d657c9b on ejh_20161202_annotator_integration into 38d7a6086cd993096e2ffcc51916f61a0a53765f on master.

demo.py, line 42 at r3 (raw file):
Previously, ejhumphrey (Eric J. Humphrey) wrote…
Done.

actually, didn't need that second try / except? I was getting some funny business from input(), but it seems to have resolved itself.

Comments from Reviewable

demo.py, line 39 at r3 (raw file):
Previously, bmcfee (Brian McFee) wrote…
don't do a bare catch here; use the correct exception class

unnec.

Comments from Reviewable

Reviewed 1 of 2 files at r4.
Review status: 15 of 16 files reviewed at latest revision, 4 unresolved discussions.

Comments from Reviewable

tiny ping @bmcfee

Reviewed 1 of 3 files at r3, 2 of 2 files at r4.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

thanks, merging


So much FLAC

https://github.com/search?utf8=✓&q=language%3Arust+flac

I don't really consider this to be a problem. The reason I started this project was to learn about audio codecs. Having more than one project crop up looks there is some need for audio codes written purely in Rust. Of those nine I could see that about two, not including mine, that are active and adjacent to what I'm doing, which is perfectly fine. Of course claxon is the one most complete since it has a full decoder working. However, I don't plan to stop at only decoding FLAC, so there is something unique about this project.


Create docs for PRs

Build and preview docs for all PRs:

Creates docs for a PR in .pr-preview/${{ github.event.pull_request.number }} directory of the gh-pages branch.
Deletes this directory once this PR is merged or closed.

@wozeparrot mentioned this on discord a couple of weeks ago here.
Potential downside: if many PRs are not closed, we may end up with lots of directories for documentation for stale PRs. Maybe only preview docs for PRs if the docs have changed?
I tried this out in a dummy repo here.

The Github workflow is crashing because the Github Action does not have the right permission.
For this to work, you need to Go to Settings > Actions > General > Workflow permissions. Then click Read and write permissions and check Allow GitHub Actions to create and approve pull requests.


Unhelpful/inaccurate link

The link about security and compliance blueprints https://servicetrust.microsoft.com/ViewPage/SCCIntroPage points to information about o365 and not to anything about azure blueprints. Please update to provide the appropriate content

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 8675a433-5c90-d9de-b450-80431411d66c
Version Independent ID: 8675a433-5c90-d9de-b450-80431411d66c
Content: Azure landing zone integration - Microsoft Azure Well-Architected Framework
Content Source: well-architected/security/design-governance-landing-zone.md
Service: architecture-center
Sub-service: well-architected
GitHub Login: @PageWriter-MSFT
Microsoft Alias: martinek

#please-close


Added new language: NL (dutch)

That’s nice… it’s my first PR 😊
From: Ryan @.>
Sent: Monday, 20 June 2022 17:39
To: BytexDigital/BytexDigital.Blazor.Components.CookieConsent @.>
Cc: Jacco Maat @.>; Author @.>
Subject: Re: [BytexDigital/BytexDigital.Blazor.Components.CookieConsent] Added new language: NL (dutch) (PR #10)
Published as version 1.0.12https://www.nuget.org/packages/BytexDigital.Blazor.Components.CookieConsent
—
Reply to this email directly, view it on GitHubhttps://github.com/BytexDigital/BytexDigital.Blazor.Components.CookieConsent/pull/10#issuecomment-1160597366, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABW25XKMZVLJTRAOXV5FGTLVQCGA3ANCNFSM5ZIAKZRQ.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>


Refactor examples

The examples on the landing page lacked CSS and were a bit busy, but
they're still awesome. I re-framed them into a "wash by example"
section, with a link to the tutorial at the end.

Can you add the DCO signoff to the commit?

I think this works for shortening it up.

DCO check passes now, thanks for the reminder!


Add tcs-issuer to external issuers list

/ok-to-test

@avalluri you might want to add 'tcs-issuer' to .spelling to pass the spellcheck

@avalluri you might want to add 'tcs-issuer' to .spelling to pass the spellcheck

Thanks for the pointer. I just figured out the same by setting up the remark and running the ./scritps/_verify locally and fixed it now.

Thanks @avalluri !
/lgtm
/approve


Impossibly large total lengths for TRs

I ran ULTRA Version 1.0.0 (beta 1) on the C. elegans N2 genome in order to identify tandem repeats up to and including rDNA locus repeats, with the following arguments:
    ultra --json --pval -t 32 -p 7500 --at 0.6456 -i 0 -d 0 -m 1 \
    c_elegans.N2.PRJNA13758.WS287.genomic.fa \
    1>c_elegans.N2.ultra.2<IP_ADDRESS>.json 2>c_elegans.N2.ultra.2<IP_ADDRESS>.err ;

Sanity-checking the results, I found that repeat unit lengths looked reasonable:
    cat c_elegans.N2.ultra.2<IP_ADDRESS>.json | egrep "\"Period\"" | perl -ne ' if (/(\d+)/) { print "$1\n"; } ' | sort -rn | head --lines=5 ;

7197
7070
6368
6256
6256

    cat c_elegans.N2.ultra.2<IP_ADDRESS>.json | egrep "\"Period\"" | perl -ne ' if (/(\d+)/) { print "$1\n"; } ' | sort -rn | tail --lines=5 ;

1
1
1
1
1

    cat c_elegans.N2.ultra.2<IP_ADDRESS>.json | egrep "\"Period\"" | perl -ne ' if (/(\d+)/) { print "$1\n"; } ' | sort -rn | wc -l ;

148228

However, for some reason, ULTRA generated total TR lengths that at their maximum were not merely unexpectedly large, but impossibly large (the total genome size of N2 is 100.3 Mb, whereas these monster predicted sizes are 18.4*10^20):
    cat c_elegans.N2.ultra.2<IP_ADDRESS>.json | egrep "\"Length\"" | perl -ne ' if (/(\d+)/) { print "$1\n"; } ' | sort -rn | head --lines=5 ;

18446744073709550542
18446744073709549886
18446744073709549506
18446744073709548767
18446744073709548435

    cat c_elegans.N2.ultra.2<IP_ADDRESS>.json | egrep "\"Length\"" | perl -ne ' if (/(\d+)/) { print "$1\n"; } ' | sort -rn | tail --lines=5 ;

7
7
7
7
7

    cat c_elegans.N2.ultra.2<IP_ADDRESS>.json | egrep "\"Length\"" | perl -ne ' if (/(\d+)/) { print "$1\n"; } ' | sort -rn | wc -l ;

148228

Using less to see how many TRs are getting these impossibly large sizes, I find that 51 of them are, after which the total sizes become much more normal:
     cat c_elegans.N2.ultra.2<IP_ADDRESS>.json | egrep "\"Length\"" | perl -ne ' if (/(\d+)/) { print "$1\n"; } ' | sort -rn | less ;   # invoke '-N' inside less to get line numbers

      1<PHONE_NUMBER>3709550542
      2<PHONE_NUMBER>3709549886
      3<PHONE_NUMBER>3709549506
      4<PHONE_NUMBER>3709548767
      5<PHONE_NUMBER>3709548435
[...]
     47<PHONE_NUMBER>3709544401
     48<PHONE_NUMBER>3709544353
     49<PHONE_NUMBER>3709544348
     50<PHONE_NUMBER>3709544310
     51<PHONE_NUMBER>3709544251
     52 14141
     53 12186
     54 11188
     55 10385
     56 10151

Spot-checking the four putatively largest of these TRs, I don't really see how ULTRA came up with the giant total TR sizes that it gave:
{"SequenceName": "X",
"Start": 5920985,
"Length":<PHONE_NUMBER>3709550542,
"Period": 1,
"Score": 32.834946,
"Log2PVal": 0.000000,
"Substitutions": 0,
"Insertions": 0,
"Deletions": 0,
"Consensus": "T",
"Sequence": "TTTTTTTTTTTTTGGGGGGGGGGGGGGGGGGG"},

[...]

{"SequenceName": "X",
"Start": 7424780,
"Length":<PHONE_NUMBER>3709549886,
"Period": 3,
"Score": 4.434807,
"Log2PVal": 0.000000,
"Substitutions": 0,
"Insertions": 0,
"Deletions": 0,
"Consensus": "CTT",
"Sequence": "TCTTCTTCTTCCAACAACAACA"},

[...]

{"SequenceName": "X",
"Start": 8546251,
"Length":<PHONE_NUMBER>3709549506,
"Period": 7,
"Score": 4.992184,
"Log2PVal": 0.000000,
"Substitutions": 0,
"Insertions": 0,
"Deletions": 0,
"Consensus": "AAAGAGG",
"Sequence": "GAAAGAGGAAAGAGGATCGTAAATCGTAAAT"},

[...]

{"SequenceName": "IV",
"Start": 11020545,
"Length":<PHONE_NUMBER>3709548767,
"Period": 3,
"Score": 4.434816,
"Log2PVal": 0.000000,
"Substitutions": 0,
"Insertions": 0,
"Deletions": 0,
"Consensus": "ACC",
"Sequence": "ACCACCACCACGATGATGATGA"},

Is this a known bug in ULTRA?  Is there some way that I could avoid having it happen?

Fixed in beta 5 - thanks!


Import error: component is named incorrectly

Hello! First of all, thanks for your work and especially for such a detailed article.
What is the problem?
Import does not work. I installed your plugin on a clean project and on the first run (npm run serve) I immediately got an error:
This dependency was not found:
* @componentsHelloWorld.vue in ./src/App.vue, ./src/views/Home.vue

How to reproduce the problem
In a clean project created via @vue/cli, there is a HelloWorld.vue component in the components folder.
If you use it in any other component, the corresponding error will appear.
<HelloWorld msg="Welcome to Your Vue.js App" />
Project settings
Vue 3
@vue/cli 4.5.10

Hey @alexshink
Thanks for the detailed issue summary. I'm guessing this may be related to windows paths, I was only able to test the library on Linux so far.
I will be adding some tests soon but for now, I've switched to the upath package over nodes native path, which looks like it may address the issue. https://github.com/loonpwn/vue-cli-plugin-import-components/commit/202a0871466fb9859bd37a415bf854c35029aed3
Would you please pull in version 1.0.0 of the package and try again

Hey @alexshink
Thanks for the detailed issue summary. I'm guessing this may be related to windows paths, I was only able to test the library on Linux so far.
I will be adding some tests soon but for now, I've switched to the upath package over nodes native path, which looks like it may address the issue. https://github.com/loonpwn/vue-cli-plugin-import-components/commit/202a0871466fb9859bd37a415bf854c35029aed3
Would you please pull in version 1.0.0 of the package and try again

I generated a new clean Vue 3 project. Added a fresh version of the package. Also just in case I did it on another computer in order to exclude a local problem in the system. Unfortunately, the same error in both cases.

I generated a new clean Vue 3 project. Added a fresh version of the package. Also just in case I did it on another computer in order to exclude a local problem in the system. Unfortunately, the same error in both cases.

Hey @alexshink, sorry about that that is my bad :man_facepalming:
I didn't have a hook setup with release-it to build the typescript files so it didn't push the update properly. I've just pushed version 1.0.1 which I have tested on my windows laptop.
Can you do me a big favour and test it out on more time :pray: Thanks in advance

Hey @alexshink, sorry about that that is my bad :man_facepalming:
I didn't have a hook setup with release-it to build the typescript files so it didn't push the update properly. I've just pushed version 1.0.1 which I have tested on my windows laptop.
Can you do me a big favour and test it out on more time :pray: Thanks in advance

Hehe, it's okay) Now it works! Thanks for the fix and quick response!
PS
It's a great tool that should have been shipped with Vue for a long time. 🥇

Hehe, it's okay) Now it works! Thanks for the fix and quick response!
PS
It's a great tool that should have been shipped with Vue for a long time. 🥇


please add a changelog

Hard to know what is changed between updates without a changelog :)

I had been adding release notes in the releases but I do agree - I'll add a changelog and make sure it's linked in the package description. Thanks for the idea!

I had been adding release notes in the releases but I do agree - I'll add a changelog and make sure it's linked in the package description. Thanks for the idea!

Changelog has been added in commit e76041a0b87e7a466a45f236b904f82975eabdb3

Changelog has been added in commit e76041a0b87e7a466a45f236b904f82975eabdb3


[VeriblePreProcessor][3]: Passing macros defined using "+define" to "VerilogPreprocess" class

NOTE: This is a part of the sequentially splitted PRs from PR #1360.
Description:
Adding an interface function AddDefineFromCmdLine to use the macros added by +define+\<foo> argument to VerilogPreprocess.

Closed as a duplicate of #1424


Update javascript for cart

Two main problems that I found during testing:

Handling of the Personal/Company dropdown – it basically makes some fields disabled and other enabled – which is correct, but feels weird. Maybe it should hide the unused fields?

Going over max number of tickets (in this case 6 in a single field,) instead of producing error message it force reloads the page loosing all the information (type of order, coupon codes, amount of tickets) in the process.

Closing as the cart implementation has been rewritten.


Call callback once on listen error

This is a redo of #2623.  Probably didn't need a new PR, but I took a new direction by using the once module, and only binding to the error event.  I think this is slightly more elegant than the previous PR.  As discussed, this is a breaking change, so can only land in a 5.x branch.

For some reason the test is not happy on Node.js 0.10. That may not actually be a problem with Express 5.0 target, just wanted to call out why the CI failed on this one (for now).

Oh, this is a change since 0.12, but if the server fails to start, calling close will error.  The test introduces this case.  In this case I can just not call close I think.  Updated.

Anything blocking this from merging?

Do you have enough permissions to merge it now, @wesleytodd ?

I had enough permissions back then I think, I was just more making sure we had agreement 😄. I will be re-visiting these pr's over the week and will land them as we are sure they are good.

I updated this, but I would like a second look from @expressjs/express-tc or others to double check since this is soooooo old. If no one has opposed this change or given feedback I will merge it later this week.


install Hyper-V feature

Description
Improvement
Android Emulator requires Hyper-V feature to be installed
Related issue: https://github.com/actions/runner-images-internal/issues/4786
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run windows2022


SwiftCLI 3.0

WIP

@jakeheis WOW!
BTW Any chance of global options, like -h (for help of given command) or -v (for verbose)?

@rbukovansky Thanks for the suggestions! I just implemented an initial version of global options and will be adding them to the README shortly.
When it comes to the Travis build, I initially tried to use @kylef's file, but I was stuck with a weird error I didn't have time to investigate fully: https://travis-ci.org/jakeheis/SwiftCLI/jobs/216556861. If you have any idea how to fix that issue, I'd appreciate the help

@jakeheis Yes, I do.
You are missing .swift-version file in repository -> this indicates to that cURLed script in .travis.yml which Swift to download and install.
See https://github.com/rbukovansky/SwiftCLI/tree/cli_3 which resulted in this https://travis-ci.org/rbukovansky/SwiftCLI/builds/217081387 (there are some issues with tests for Linux).

@jakeheis OK. Little bit of work and now my tree mentioned earlier has all changes neccessary to get all builds done. Hope it helps.
Proof: https://travis-ci.org/rbukovansky/SwiftCLI/builds/217192629

@rbukovansky Thank you! If you could open a pull request into the cli_3 branch then I could merge your commits in. Thanks again!

@jakeheis Rebased my changes on top of your cli_3 branch and created PR.


Replace hardcoded 'master' group name with 'group_name_master' variable

Proposed Changes
For improved flexibility and maintainability.

Update tasks in node role to use 'group_name_master' variable instead of hardcoded 'master' group name
Update tasks in master role to use 'group_name_master' variable instead of hardcoded 'master' group name
Update tasks in post role to use 'group_name_master' variable instead of hardcoded 'master' group name

Checklist

[x] Tested locally
[x] Ran site.yml playbook
[x] Ran reset.yml playbook
[x] Did not add any unnecessary changes
[x] Ran pre-commit install at least once before committing
[x] 🚀

Thank you!  While I do like this idea, I feel like this should be backwards compatible.  Any way to make this default to master if this variable is not provided?

@berendt  looks like yaml lint

Looks like pre-commit uses a different yamllint configuration then .yamllint? The line-length should be a warning only.
---
extends: default

rules:
  line-length:
    max: 120
    level: warning
  truthy:
    allowed-values: ['true', 'false', 'yes', 'no']

Good call, I will look into it.  Might need to load the config...

Thank you!


Add support for GraphQL\InputType attribute

Allows tagging of shapes with <<GraphQL\InputType>> to support accepting those shapes as input. GraphQL spec: https://graphql.org/learn/schema/#input-types.

The fields on an input object type can themselves refer to input object types, but you can't mix input and output types in your schema. Input object types also can't have arguments on their fields.

I didn't add any checks for this yet.
One other note, this requires type assertion to verify the incoming dict coerces to the defined type. Not sure if there is a better way to do this.

I'm going to refactor this a bit after https://github.com/mwildehahn/hack-graphql/pull/40 gets merged so we can do this: https://github.com/mwildehahn/hack-graphql/pull/40#discussion_r631171213 and not have to collect all inputs and thread them through.


[Fix #2928] Add auto-correct for Style/NestedParenthesizedCalls cop

This is a fairly straight-forward change. It simply adds auto-correct for Style/NestedParenthesizedCalls as was requested in https://github.com/bbatsov/rubocop/issues/2928.
Example:
expect(shorten url).to be short_url

will be auto-corrected to:
expect(shorten(url)).to be short_url

:+1: Thanks!


Copy install log and kubeconfig to store them as artifacts

Copying the files onto the local host will enable us to store them
as artifacts when we run the job from a jenkins slave.

@jtaleric think copying the entire directory would be too heavy for artifacts. Thoughts?

@jtaleric think copying the entire directory would be too heavy for artifacts. Thoughts?

If we drop bin/ and drop .git/. It shouldn't be too heavy. However, if we capture the config files/logs/etc that might suffice.

@jtaleric should we capture anything else other than openshift install log, timing and kubeconfig?

@jtaleric should we capture anything else other than openshift install log, timing and kubeconfig?

I need to dig through the log to make sure we have specifics on what was deployed. However, we can iterate on this more and add more artifacts when we identify them.


Draft: Add missing functions in primal

Summary

This PR is a feature
It does the following:

Adds missing AXOM_HOST_DEVICE specifiers to several methods in Point, Segment, and Vector classes
Adds length_squared method to the Segment class
Adds cross, normalize, and unitize methods to the Vector class
Adds squared_distance operator

Resolves https://github.com/LLNL/axom/issues/1309

The portion of changes that are ready are being pulled into this merge request: https://github.com/LLNL/axom/pull/1422

I'm going to pull out changes into different pull requests.


Default scriptsPath to ResourcesDir/Code/GUI

Also populate cascade menu from it as described in user guide
Fixes issue #955
Brief summary of changes
getScriptsDir() was renamed to getScriptsPath() and wired to be initialized to the new location of Scripts /Code/GUI
Testing I've completed
Made installer locally and behaved as expected/described in User/Guide
CHANGELOG.md (choose one)

no need to update because bugfix

On Mac, Scripts Path is null/Code/GUI/

Thanks @chrisdembia it's likely an initialization order issue where the preference is set before the OpenSimResourcesDir is populated, which happens actually after the user selects a directory to copy resources. I'll look into this.

Issue addressed by moving initial setting along with InstallResources, this artifact should be good to test all the latest scripts merged by @jimmyDunne once build is ready.

Works great! Gives the correct path to scripts!

@aymanhab what is the reason for changing the Scripts menu?
before:

after:

The new menu organization seems redundant.
Also, on Mac, I am not getting a Scripts Path preference, even after running defaults delete org.opensim.version40, running installResources() and restarting the GUI:

Thanks for the catch @chrisdembia I missed the change in the menu and apparently @jimmyDunne didn't see it either, will revert the menu change to keep 3.3 layout.
The preference should show up if you allow the installer to copy the resources for you, not sure what happens if you cancel out since preferences on windows and osx are different. Once you see it you should be able to verify it's working as documented. Please let me know otherwise.

Sorry, I mean on first launch. You can delete the file we use to mark first
launch manually.
On Sat, Sep 1, 2018, 7:25 PM Christopher Dembia<EMAIL_ADDRESS>wrote:

installer to copy the resources
Could you clarify? The installer never does this. I did try re-running
installResources() and that didn't help.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/opensim-org/opensim-gui/pull/981#issuecomment-417900269,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADyccCbmhHezI_kdIO__Mj0FJqLh2wZjks5uW0GYgaJpZM4WT28K
.


Merge USB Serial Autodetect/Enumeration lib

So this needs a Linux port in order to pass CI. It should be pretty easy, since I think you can just use sysfs to do it (so it turns into a filesystem traversal), but that will block the merge. Would like to get this merged ideally shortly before 0.2, since changes to the manual configuration API will be coming in 0.2 which directly affects this code's usage.
@yboger Is this a reasonable item to have @godbyk do?


runtime-config-linux: Separate mknod from cgroups

This is #99 rebased onto the current master, because the idea seems
to have new life.  It leaves mknod entries in linux.devices
and moves cgroups entries into linux.resources.devices.  Quiet
thread on the issue here.
This makes it easy to distinguish between configs that call for cgroup
adjustments (which have linux.resources entries) from those that
don't.  Without this split, folks interested in making that
distinction would have to parse the device section to determine if it
included cgroup changes.  This will also make it easy to drop either
portion (mknod or cgroups) independently of the other if the
project decides to do so.
Using seperate sections for mknod and cgroups also allows us to avoid
the complicated validation rules needed for the combined format
mknod/cgroup (#101).
Now that there is a section specific to supplying devices, I shifted
the default device listing over from config-linux.  The /dev/ptmx
entry is a bit awkward, since it's not a device, but it seemed to fit
better over here.  But I would also be fine leaving it with the other
mounts in config-linux.
The reference links are sorted into two blocks, with kernel-doc links
sorted alphabetically followed by man pages sorted alphabetically by
section.

@mrunalp, you were going to write up some more motivation for this?  It's also possible that you intended to post that motivation to the thread, and that I'm jumping the gun with this reroll.  If so, feel free to table/close this PR until the thread has reached a consensus.

We can continue using this PR. i have responded to the mailing list discussion.

There are two use cases for this separation:

The user doesn't want to specify any cgroup configuration. This is easy for the other values but the device cgroups are tied up the device configuration.
Running containers under a non-host usernamespace is an issue, because one can't modify the device cgroups in such a scenario.

So, separation will be good for consistency as well as technical reasons.

Rerolled dfb5dad → 3f8105e adding @vishh's ‘omitempty’s [1,2,3].

In today's meeting there was talk about where default-device creation
happens in the container-setup timeline 1, and whether/how config
authors should be able to opt-out of default-device creation 2.
Addressing either of those seems orthogonal to the split in this PR
3, but I'd thought I'd mention them here in case anyone else
disagrees about their orthogonality.

Needs rebase.

On Mon, Jan 25, 2016 at 11:57:44AM -0800, Mrunal Patel wrote:

Needs rebase.

I'll wait until #284 lands.

On Mon, Jan 25, 2016 at 12:27:43PM -0800, W. Trevor King wrote:

Mon, Jan 25, 2016 at 11:57:44AM -0800, Mrunal Patel:

Needs rebase.

I'll wait until #284 lands.

Rebased around #284 with 3f8105e → 81f726e.
$ diff -u <(git show 3f8105ed4) <(git show 81f726e3)
shows no changes outside of diff context.

LGTM

LGTM


Fix conda initialisation

What changes were proposed in this pull request?
No longer require that CondaRunner be run before creating a SparkContext.
This was the case before because SparkContext got and stored the (potential) CondaEnvironment from CondaRunner on construction, whereas now it gets it dynamically.

@robert3005 can you take a look?


[Android] Error happened: Precondition Failed

Hi!
I'm trying to upload an image on android with this code:
var obj = {
    uploadUrl: config.API_URL + 'images',
    method: 'POST',
    headers: {
        ...config.HEADERS,
        'Authorization': 'Bearer ' + this.userToken
    },
    fields: {
        'one': 'two'
    },
    files: [
        {
            name: 'file',
            filename: 'avatar.jpg',
            filepath: this.avatarSource.uri
        }
    ]
};
FileUpload.upload(obj, (function (err, result) {
    //err === 'Error happened: Precondition Failed'
}).bind(this));

And I'm receiving this error. Do you have any ideas why does it happen?
Thanks!

hey @PhilippKrone , the same error occurred on Android (IOS works fine). Could you please help with issue?

+1

any solution ?


deleted some content from original base

👷 Deploy Preview for graceful-eclair-4981f5 processing.

Name
Link

🔨 Latest commit
898622a391463d8672239b48c99ed1128b83fde3

🔍 Latest deploy log
https://app.netlify.com/sites/graceful-eclair-4981f5/deploys/65bdf0d2d76f2600071366e4


Server Collation for Windows Container

What is a "right" way to change server collation?
At moment it seems that only possible way is to create own image from scratch defining collation while installing SQL Server
Trying to bypass issue with this instruction but unfortunately sqlservr wont exit when it is done
May be there is a chance that base image will have already downloaded and extracted installation media, so we will be able to at least start from it and install SQL Server with required parameters, two more gigs on this image probably wont be a deal breaker or it might be separate image for enthusiasts who need this so will be able to get always fresh version for dev needs
Thanks in advance

One way is to use the Dockerfile we've made available on GitHub to build your own image and pass in the command line parameter you want.
https://github.com/Microsoft/mssql-docker/tree/master/windows
The SQLCOLLATION command line parameter needs to be passed to setup.exe.
More info:
https://docs.microsoft.com/en-us/sql/database-engine/install-windows/install-sql-server-2016-from-the-command-prompt

Just for completeness.... On the Linux side we have an env var to set this at docker run time.  MSSQL_COLLATION

Yep, I can confirm that, MSSQL_COLLATION works as expected on Linux containers and is awesome (hope somewhere in future to see such thing in windows containers)!
For anyone else fighting with collations - you need to build your own image by at least add /SQLCOLLATION=Cyrillic_General_CI_AS to setup.exe but if you are one journey of discovering building your own images I bed you will not stop on that :)
Thanks for response, at least know that I'm going in a right direction, closing issue
BTW: Hope to see some descriptions in repo readme for people like who also need to do such things


Open source

Is that entirely open source?

We're pushing the source in a few minutes, we've just been getting all the code from our servers again.

@ZakDoesGaming Great! Thank you!


boards: qorvo: add DWM3001CDK support

This doensn't add any drivers for the DWM3001C, but it works with
https://github.com/br101/zephyr-dw3000-decadriver/
It's basically a copy of the decawave_dwm1001_dev with the modifications to work with the dwm3001cdk

So I can either get tests that expect console to build by setting up usb stuff, or I can get tests that test usb stuff to build by not setting up usb stuff
Not really sure what do do about this, some existing boards just don't pass some tests
for example:
west build -p -b nrf52840dongle tests/drivers/udc -T drivers.usb.udc fails because nrf52840dongle uses usb cdc as a console, so gen_isr_tables.py: error: multiple registrations at table_index 39 for irq 39 (0x27)

The index.rst had bad formatting

@anangl please review


Is this package still under active development?

hi, nice work here.  I am looking for a good TA lib and this seem to be very promising.
Just one concern, there seem to be lack of activity, is this repo still under active development / maintenance?
Please advise, thanks.

Yes, it is, but quite slowly though :-)

Keep up the good work :+1:


CipherFileChannel temporarily saves unencrypted files in temp directory

encripted file is decrypted in temporary directory

The temporary directory is used to store files at runtime. Temporary files are deleted if you correctly close a FileChannel by CipherFileChannel.close().
In addition to that this.tmpFile.deleteOnExit() should assure the deletion finally takes place when the JVM is stopped in case you don't close the file channels directly.
I propose you recheck closing files in your source code. This should solve the problem you reported in #1 as well.

I could imagine to create a configurable tmp-file destination. This may be

a user defined tmp directory instead of java.io.tmpdir
a tmp file on behalf of another filesystem implementation like https://github.com/marschall/memoryfilesystem

Would this solve your issue?

I dont use temp file, File  is being encrypted/decrypted "on fly"

If tmp files shouldn't be used as intermediary files the overhead of reading / writing becomes enormous. In encryption mode a read operation would require a decryption of 1/2 bytes of file size in average. The same for write operations and according encryption.
You may find a better solution taking a look at block level encryption. For file level encryption the use of tmp/cache files seems to be out of question in my view.


doc: improve documentation of cargo_helper

By commenting what "pretty" actually is.
I thought "pretty" is some special value 😅

Updated.
The rehearsal is good, I know what to do next time 😁


WARN on npm-install

I've started getting a WARN on npm install:
npm WARN deprecated<EMAIL_ADDRESS>cross-spawn no longer requires a build toolchain, use it instead!

Thank you for this issue!
I will replace cross-spawn-async by cross-spawn today (JST).


need old feature  Loading a level separately

need old feature  Loading a level separately
I put the levels in multiple scenes to make one big map, and I had to load the smaller levels in each scene separately.

If you're using before version 2.0.0, then you can select the toggles in the component to only build levels of your choice.
If you're using 2.0.0, then the imported object produces many levels without these options unfortunately. It's a bit more difficult to manage which levels are made because of an import result, so they all need to remain available in the import result.
In this scenario for your situation, a strategy you can do for each scene, is to set all of the level GameObjects as inactive except for the level you'd like to keep active. The scene overrides will remain set as long as the levels are not renamed in LDtk.
Another option could be to make a prefab for each level, and then apply this same inactive concept except within a prefab space.
Let me know if this helped 👍

I believe I have a potential idea for this. Maybe the generated level's root can have these same toggle options to set on/off the levels, where it would set its hideFlags in the hierarchy and so the levels would not be visible but also inactive. I'll give this a shot, but it will be a later priority as I fix some important bugs associated with the 2.0.0 update 🙂
https://docs.unity3d.com/ScriptReference/HideFlags.HideInHierarchy.html

Thank you very much for your reply. I will wait for the update.

 Add option to Try and enable/disable certain levels from being included

I've prototyped this feature to experiment with it.

However, while playing with it, I realized that the concept behind this idea is that it just simply sets GameObjects active/inactive, which is already possible by directly setting GameObject(s) active/inactive in the GameObject's inspector.

I was also toying with the idea of making the objects completely invisible in the hierarchy, but my gut reaction is that it might create extra confusion.
So I think it would be a bit unnecessary to provide a redundant feature.
At least for now, I'm not going to add this because this is already possible.
But I absolutely appreciate you posting the issue.
I'm going to close this issue, but feel free to reopen and let me know if there is a detail that I might be missing here for why this would be added, and I'll reconsider 🙂

After thinking it over a little more, I think that the advantage with this is that the levels are hidden from searching in the hierarchy window with the search bar, which helps reduce inactive-gameobject clutter when searching. So I will reconsider this.

I've had another idea to get this possible: The separate level files could be considered as their own standalone imported level prefabs. (Currently the levels just serve as a way to import json data from level files)
The advantage would be that the entire project doesn't need to be dragged into the scene and then disable some of the unneeded levels. And instead, drag in the necessary levels into a scene.
I've made a card about it(https://trello.com/c/tTSOVlsf), and I'm intrigued by the idea, as it could be a nice solution to:
-Reduce scene/hierarchy bloat, would feel cleaner overall
-Alleviates referencing potentially unnecessary assets in memory/buildSize to load in the scene due to the inactive level gameobjects
-Better modular options (modular pieces of a dungeon for example)
I'm definitely gonna try it out sometime and see if it's possible on a technical level sometime.

Hi, I've added a feature in the new update (2.2.0) that should fulfill what you need.
The separate level files can now be their own imported level hierarchies, which allows for better modularity with levels 🙂
Let me know if that works for you, otherwise, feel free to reopen the issue.

oh, thank you ,  I will try it.
------------------ 原始邮件 ------------------
发件人: "Cameron @.>;
发送时间: 2022年1月26日(星期三) 下午4:15
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [Cammin/LDtkToUnity] need old feature  Loading a level separately (#16)
Hi, I've added a feature in the new update (2.2.0) that should fulfill what you need.
The separate level files can now be their own imported level hierarchies, which allows for better modularity with levels �0�2
Let me know if that works for you, otherwise, feel free to reopen the issue.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>


Added a newtype JannoFile for [JannoRow]

Here's an implementation of the idea shared in #229

Codecov Report
Base: 70.96% // Head: 70.93% // Decreases project coverage by -0.03% :warning:

Coverage data is based on head (ecb03d7) compared to base (fda9e36).
Patch coverage: 82.50% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #232      +/-   ##
==========================================
- Coverage   70.96%   70.93%   -0.03%
==========================================
  Files          18       18
  Lines        2211     2216       +5
  Branches      245      245
==========================================
+ Hits         1569     1572       +3
- Misses        397      399       +2
  Partials      245      245

Impacted Files
Coverage Δ

src/Poseidon/CLI/Forge.hs
61.60% <66.66%> (ø)

src/Poseidon/Janno.hs
82.09% <78.26%> (-0.23%)
:arrow_down:

src/Poseidon/Package.hs
74.67% <85.71%> (+0.06%)
:arrow_up:

src/Poseidon/CLI/List.hs
80.88% <100.00%> (ø)

src/Poseidon/CLI/Summarise.hs
67.74% <100.00%> (ø)

src/Poseidon/CLI/Survey.hs
90.21% <100.00%> (ø)

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

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

Right - my biggest concern is that JannoFile doesn't really capture the meaning. It's just an arbitrary list of JannoRows. I considered naming it JannoSet, but a set has no order. JannoList is already in use. Do you have a suggestion?
I replaced all instances of [JannoRow] to make sure that it becomes harder to accidentally run into the issue I describe in #229. But I understand that it looks a bit weird at times, not least because of the type name.

Yes, the name isn't perfect. You could have used JannoMonoid, but that's not a name you want everywhere. What about JannoRows?

JannoRows is simple and clear. I love it. Will use that. Thanks!


Issues with indexes

While generating a new db migration, @sambostock and I noticed that the descending order was dropped from the ["task_name", "created_at"] index when the migration ran.
Some of our adopters also seem to not have the order on the index present:

https://github.com/Shopify/arrive-server/blob/master/db/schema.rb#L393
https://github.com/Shopify/accio/blob/master/db/schema.rb#L192

@etiennebarrie had also flagged that the index on just task_name is potentially redundant - we should double check whether we need it or not.

Some updates on this:

Confirmed that our task_name index is redundant and can be dropped. Will open a PR
Adopters not having the sort order present on the index is actually not a bug, but instead due to the fact that most of our apps are using MySQL 5.7, and ordering is only present on MySQL 8.0+ 🤦♀
The sort order disappearing from SQLite3 when running a new migration is actually the result of a bug in the SQLite3 Adapter in Rails.


Migrate from travis-ci.org to travis-ci.com until 2020-12-31

travis-ci.org will be closed down December 31st, 2020
Announcement: https://mailchi.mp/3d439eeb1098/travis-ciorg-is-moving-to-travis-cicom
Migration guide: https://docs.travis-ci.com/user/migrate/open-source-repository-migration

Should we archive this project anyway?

The project is now migrated to travis-ci.com. But the master branch does not build (and it didn't build for some time already). The PR #69 restores a working build by running the integration tests against an older version of rocket.chat.
It would be great if we could test against the latest version. But right now it doesn't work because of issues like https://github.com/RocketChat/Rocket.Chat/issues/19537. In a nutshell: our node version is too old for the current rocket chat versions.
Using a matching node installation requires a bit of change in the build setup. Currently we use rocketchat/base:12.16 as the base for a custom docker image. There is no newer version of the base image with a suitable version of node. I see two options:

Update the node version in the Dockerfile (include something like apt install node ...)
Instead of the custom image we could use the ready-made rocketchat/rocket.chat:latest image and move all configuration in the Dockerfile to the pom.


chore: add go reserved words

Fixes #2299

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: c27310c9f8c1374b8ecc9167fdcd502674fe95da
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: 04f1e4c068b831f775f48705df91027af4914433
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

This will start to warn users when go reserved words are encountered during compilation. This is fine with me, though it feels a little funny that users who make no changes to their code and haven't enabled go in their JSII config receive these warnings. Especially before the feature is stable.
I think it's a minor inconvenience but something to consider.

Well... The thing is if you don't warn them; and they want to enable go later on... They will have a bad time. This is precisely why the warnings mention which languages claimed a given term as a reserved word... You'd start seeing "go" picking up your words, you can actually plan your way to deprecate/replace those terms and know you'll have to do "all of those" before you'd enable go.

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: b3b946026c8e814abccbeecc1665c4b78764b3b3
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: be41c73f65b52ca3ced5fad78c3c1194ec1b63cc
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: 64d478b4bf9552081dfac6b32678c9566b8045c6
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Accumulo 2.0 Monitor shows weird ingest rates in E notation

With reference to the below screenshot, this test cluster had no major activity happening yet we see some off-the-chart numbers. I guess there are potentially 2 issues:

More trivial one (I guess?) is how to handle this gracefully in the UI
More interesting is why those numbers were this large

Either ways thought I'd raise this in public interest.

I looked into trying to block against this behavior and I don't think it is worth trying to prevent.  We could check for small values but it is possible the rates could be small so it would be difficult to determine what is valid. This only happens during the first few seconds at the start of a new cluster.

One option might be to delay presentation of the data until at least 5 or 10 seconds have passed since the server started, so that the monitor can have enough time to produce numbers that make more sense. That could be done in the client-side javascript, or as some special case in the monitor server code that sends the data used.

Thank you both for sharing your perspective on this. It would be a good exercise for me to look into what Christopher mentioned, so let me do that.

Could these just be rounded to an integer?

Having a delay would work, I am just not sure where in the code would be the best place.  It would probably be overly complicated to try and do it in the javascript.  Perhaps in the constructor of the Monitor.
Rounding to an integer would work but we would lose precision during slower ingest rates.  Although, I'm not sure how much difference < 1MB would make...

Although, I'm not sure how much difference < 1MB would make...

Would be nice to keep some precision.   Could do (long)(d * 1000)/1000.0 where d is a double to truncate d to a few decimal places.

Flot has options for displaying the axes:
https://github.com/flot/flot/blob/master/API.md#customizing-the-axes
In particular, the tickDecimals and the tickFormatter are probably what we should be changing.


[TE] Yyuen/link to old ui

What's new:

Added a link to the new ui (from old UI)
Added a link to the old ui (from new UI)

+1

@luisah Thanks for reviewing! The use case here is as follow : btn thirdeye-btn thirdeye-btn--secondary. This is indeed a layer on top of bootstrap button. The "circle-effect" animation isn't included; the transition was added to make it looks smoother. (see below). If you do feel like it's too close, I'll go ahead and remove the transition.


Feature request: Fast scanning such as that implemented in Kamilsss655 firmware

I feel like this is a direct not possible, but having the ultrafast scanning feature of the Kamilsss655 firmware would be amazing, especially now there is a 3rd scan lists i am much more tempted to fill over 100 memory channels, which takes a while to scan. if it is al all possible that would be amazing, I'm loving this firmware so far.

one way to do this: fewer screen updates during scanning.
another way: maybe have a dedicated scanning function that only reads frequencies from eeprom (is that how it's done?), then only read other channel-specific settings when squelch is broken.

duplicate #31


Networking Refactor Pt. 2

This PR is the second step towards a slightly improved networking API.
The majority of the changes present here are simply extracting discrete components into their own types. This also improves and pushes packet creation into types which actually have the information to generate the required information.
I would describe the current method of building packets as "outside in", when it should be "inside out" for easier refactoring. The high level code in the Server struct shouldn't be managing low level information like packet creation, as it makes everything significantly more unwieldy.
Names are bikeshedded since that seemed to be a point of contention last time.

What does this improve?. Wrapping things into their own structs is not a real improvement to me

We agree that the current network design is difficult to maintain, and contains a lot of deadlocking code yes? Do we agree that there are principles of software design that might be valuable to take inspiration from to write maintainable code?
Surely we can apply SOLID design principles to code to make it better to use. The Separation of Concerns or Single Responsibility Principle principles suggest that a single struct or object should have a single concern or task in maintainable code. Refactoring or rewriting any one of these new structs requires less effort than it took to do so in the first place. It touches fewer pieces of code, and has a well encapsulated data and a simple and clear interface which improves our ability to refactor and add new features later.
That's not to say that we should follow SOLID principles blindly, after all Rust isn't an Object Oriented Programming language. Things like Responsibility Driven Design exist more for Data Driven languages too. Personally I'm more of a fan of RDD than SOLID, or even DDD but we all have our favorites. The point is that these guiding principles inform our software design and lend themselves to certain types of programming problems and refactoring. None of them are perfect in every situation, but any of them is a hell of a lot better than now.
A prime example of where this refactor improves the situation, is that now it's trivial for actual caching to be done on the server status reliably. Since its contents are no longer pub and directly accessed, a simple update_status() can be written to update the cached status, instead of having to worry about any of the places the status may have been directly updated without interacting with the previous API interface.
I encourage you to learn more about actual software development practices and why they're useful and important. The current Server struct very much resembles a God Object which is problematic for a reason.


implement: validate_environment_marker

resolve #213

I'll close this PR and use this PR #225  instead


AFNetworking: Fix static analyzer issues.

For some reason, 4a49ba0 solved the static analyzer alone.
Should we add an assert anyhow?

@dlahyani @StatusReport

 Fix AFNetworking static analyzer warnings.

 Fix AFNetworking static analyzer warnings.


Setup a node port to connect to the database and add sqlpad

Closes #84
Will have merge conflicts with #81 - let's see which goes in first.

Hi @kami619 - this PR adds the node port for the database and docs on how to connect to the database. Please give it a try. The ugprade.sh should work on a previous installation of minikube.
cc: @tkyjovsk

reviewing this now, sorry about the delay in doing that


update amount label

Can we release this fix? The latest sdk version 0.5.0 does not have this fix.
implementation 'com.moyasar:android-sdk:0.5.0'

Hello @riocallos the latest release has the fix.
https://github.com/moyasar/moyasar-android-sdk/blob/main/sdk/src/main/java/com/moyasar/android/sdk/data/PaymentSheetViewModel.kt#L111
Please make sure to do a clean build and sync your gradle changes if you have updated the package recently


Update style of notifications.

Fixes #17
Fixes #15
Fixes #18
This gives a dark theme instead of the mac-like theme. This allows us to remove transparency and be more cross platform.

White border is visible on windows 7

@afriedma how's this? :D

That's sweet, ship it..... And thanks again for a quick turnaround.

@afriedma shipped as v1.0.0


Delete Functionality

We should be able to delete a song (title, artist, and rating) from the database

This ended up causing me issues with what I thought was type-casting problems, but ended up being some missing statements. I was trying to delete songs/ratings with a string 0_0...


Introduce set with custom equals and getHashCode

I got tired of not having custom hash sets in JS so I made a naive one.
Not using linear search to de-dup rename locations cuts search for locations of Type from 1750 ms to 1450 ms.
I plan to use this in a bunch of other places in my FAR cleanup.
TODO: this is the simplest possible implementation
TODO: these were the consumers that I'd encountered recently

Do you have a sense of how often you expect entries in the set to share a hash? If best practice is for hashes to be unique whenever possible (and it may not be, I don’t know), I wonder if it would be better to let the underlying map values be TElement | TElement[], promoting to an array only once two entries share a hash. That could make for a lot fewer array allocations for large sets with unique hashes, at the expense of probing the array-likeness of the values, lack of support for TElement itself being array-like, and polymorphism when some entries do share a hash. FWIW I tried this with the ExportInfoMap which is a multimap where you have multiple values when a symbol is re-exported, so in practice that meant that a majority of values were singles but arrays were not uncommon. I couldn’t measure any difference between the always-arrays and arrays-when-needed approaches, so I went with the simpler always-arrays approach like you have here. But I’m curious if you’ve thought about or tried the same thing.

@andrewbranch Generally speaking, I'd expect collisions to be pretty common in a hashset, but I have no particular objection to dynamically promoting values to arrays - this is just the simplest code that solved my problem.
Any thoughts on @sandersn's suggestion of clearly separating this from Set<T>?

Any thoughts on @sandersn's suggestion of clearly separating this from Set<T>?

Personally, I don’t think it’s much of an issue since new Set() ought to be the obvious default to reach for when creating a Set, and the getHashCode and equals parameters are required in your API.

Re-opening to see if that causes it to pick up the additional commits I pushed.


Fixing molecule tests of edpm_ceph_client_files so they can run locally

Patch also introduces related job in github actions CI.

recheck - authorization failure

recheck

recheck

recheck


chore(vue-devtools): add vue devtools

Summary
Add the new Vite plugin Vue DevTools
Important
For our host apps that utilize KToaster, you will likely need to switch the active app in the DevTools

Click the Vue icon at the top-left of the DevTools
Click "App 1" dropdown, and select "App 2"

Use https://github.com/Kong/public-ui-components/pull/1128 instead


Use Interlocked.CompareExchange instead of locking

The generated code uses a lock to update a stored service. Have you considered Interlocked.CompareExchange. It would avoid the context switch the lock introduces and might be faster.
if (_someService == default)
    Interlocked.CompareExchange(ref _someService, default, new MyNamespace.SomeService());
return _someService;

Ah, makes sense.


Improve DropdownMenu sample code for requestFocusOnTap on mobile platforms

Description
This PR is to improve DropdownMenu sample code. By default, requestFocusOnTap is false on mobile platforms. When users run API sample code on mobile platforms, they can not edit the text field and think it is a bug. Although it is detailed at https://api.flutter.dev/flutter/material/DropdownMenu/requestFocusOnTap.html, users often do not pay attention to it.
Related issue
Fixes https://github.com/flutter/flutter/issues/127672
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

This looks good. Could you help to add a test in examples/api/test/material/dropdown_menu/dropdown_menu.0_test.dart?

@QuncCccccc Thanks for taking a look. I just added a new test at dropdown_menu_focused_on_field.0_test.dart instead of appending to dropdown_menu.0_test.dart. Please help to review it again.

@QuncCccccc Thanks for taking a look. I just added a new test at dropdown_menu_focused_on_field.0_test.dart instead of appending to dropdown_menu.0_test.dart. Please help to review it again.

Just curious why do we want to create a new file for the same sample file🙂? From my observation, seems there is only on test file for a specific sample file, so maybe we can just follow this pattern:) But let me know if there's any other concerns.
Other than that, all look good! Thanks!

Ah, I thought it would look neater because I saw dropdown_menu_entry_label_widget.0_test.dart. But I was wrong, it was a test for another example. I moved the new test into dropdown_menu.0_test.dart. Hope it's ok now :)


Remove util from prometheus include path

Remove util from prometheus include path

snafu is adding two utils to the path when resolving the relative path, will be removing the one to fix the path.
FileNotFoundError: [Errno 2] No such file or directory: '/opt/snafu/snafu/utils/utils/prometheus_labels/included_labels.json'


Can't compile with latest version of Android Studio

I received the following messages:
WARNING: Configuration 'compile' is obsolete and has been replaced with 'implementation' and 'api'.
WARNING: The specified Android SDK Build Tools version (27.0.2) is ignored, as it is below the minimum supported version (27.0.3) for Android Gradle Plugin 3.1.2.
Error: java.util.concurrent.ExecutionException: com.android.builder.internal.aapt.v2.Aapt2Exception: AAPT2 error: check logs for details

update your sdk build tools?

Restarting Android Studio remedied the error, warnings still persist but
the sample compiles and runs now.
On Thu, May 3, 2018 at 4:19 PM, Pedro Varela<EMAIL_ADDRESS>wrote:

update your sdk build tools?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/googlesamples/android-BluetoothLeGatt/issues/54#issuecomment-386423344,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADc_urKOg8KiVPXKzFQ5N-SuUQmztKTNks5tu2ZFgaJpZM4Txrgn
.


Unpackerr

Remove Deluge.
Closes #20.
More testing to do...

Will this continue checking Deluge for download status? This seems like it will break if people have set their "completed" folder the same as their "downloads" folder in their BitTorrent client

This is working for me right now! It requires passing in your download path - I updated the readme a little bit. I'll update more. It can be a confusing setup, but it definitely works.
tl;dr: This app just got the download path from Deluge, so now it has to be provided in config.

Noted your documentation updates. How will this handle incomplete downloads?
ie. I have 50 files from a RAR archive, 20 are done, including the base .rar file, and the other 30 are still being snatched.
What happens in this instance? How does Unpackerr know that all files for the torrent are complete?

The Logic section in the readme may need an update? Let me know.
Basically, this looks for Status="Completed" in the Sonarr and Radarr Queue Api (ie. Completed Download, Still Queued). Those only get to that state when the torrent client reports the download is finished.

All good, this makes sense and I must have glossed over that part and not understood correctly.
Let me know when you merge this PR and I'll update/rename my Alpine container 👍

So, im sure i have borked somthing, but since i started a radarr download i was getting a json error. i thought it was just because the download locally wasn't complete and still had temporary rar files but now they are all complete and ready to unpack

after restarting unpackerr

fixed above pasted errors

@madcatsu new thing is out.

Got it. New container image over here https://hub.docker.com/repository/docker/madcatsu/unpackerr-alpine-daemon

Added back to readme!


Any thoughts on when will be tibia 11 full compatible?

Still cant look on the monsters

Server log messages should work as long as you are using the Tibialyzer 1.4.2 beta and have selected Tibia 11. Tibialyzer might never be fully compatible, depending on how difficult it will be to reverse engineer the client.

Where to get Beta ?

https://github.com/Mytherin/Tibialyzer/releases/tag/1.4.2-beta.


etcdctl: add short flag for 'ls' command

Fix https://github.com/coreos/etcd/issues/4866.

For reference, see documentation here https://github.com/codegangsta/cli#alternate-names.

Oh totally missed this https://github.com/coreos/etcd/pull/4872. Closing.

@gyuho It's my mistake I think. I forgot to reference the bug in my PR. Sorry about that.

Np! Thanks!


Rake: update task clean to remove bin and build folders

The #{MRUBY_ROOT}/build and {MRUBY_ROOT}/bin directories, which are now hardcoded by this PR, can still be replaced.

Alternatives to the #{MRUBY_ROOT}/build directory can be specified with the MRuby::Build#build_dir= accessor.
The {MRUBY_ROOT}/bin directory can be replaced by the INSTALL_DIR environment variable.
NOTE: There is no documentation for the INSTALL_DIR environment variable, so I believe it is a hidden feature.

Also, #{MRUBY_ROOT}/build/repos/#{MRuby::Build#name} contains a git clone of the third party mrbgems.
The git repository is also removed by rake clean with this PR, so people who are in the middle of modifying other people's code may have trouble with rake clean.
NOTE: The MRuby::Build#gem_clone_dir= accessor allows you to specify an alternative, but I believe it is a hidden feature that is not documented.
Therefore, I will create a PR to revert to.
In my opinion, if the content of this PR is needed, it would be better to define it as a different task than rake clean or rake deep_clean.

This is because the current deep_clean task only removes third-party GEMS directories related to the build target based on the "Build configuration file" at the time.
Unrelated GEMS directories will be kept intact (on a per-build-target basis).

The original meaning of deep_clean was “to clean everything generated during build”, not only in the last build, but in all builds. So it's natural to remove bin and build in the task, as default build target directories. I know some files may be generated outside those directories (according to configurations), but it's better than nothing.
Some software uses distclean task for the purpose, but I see no specific reason for separate tasks.

I understood. If that was the original purpose of deep_clean, I have no disagreement.


Create countdowntimer.py

The code will take input from the user regarding the length of the countdown in seconds,a countdown will begin on the screen of the format ‘minutes: seconds’. We will use the time module here.

create proper description and follow rules (fork + star + follow)


Images Orientation automatically change after clicking one image

I am using your library which is very helpful to create custom camera but I am facing issue after taking multiple images.I have also checked your demos,in that image orientation is proper after taking pic as you are displaying image in new activity.I am using takeSimplePicture() method from your demo to capture image.
In my case I have to show all captured images on same screen but only first image set in correct orientation all other are getting rotate in landscape mode.I am using Nexus 5(5.1.1) device.and also sharing screen shoot for the same.
ScreenShoot:
Check Top images list from below screen shoot only first image is in Portrait and all other are in landscape.I am taking all pics in portrait mode.

My guess is that you do not have enough memory, and the image-rotation task is not being run on the second and subsequent pictures. If you have not done so already, add android:largeHeap="true" to your manifest and see if that helps.
Beyond that, no further work is being done on this library. My replacement library does not handle changing the orientation yet, but when it does, it will use C/C++ code, to avoid Dalvik heap limit issues.

I have tried adding android:uselargeheap=true for both cameraActivity and Application.

But still the images rotate.

This library has been discontinued.


Fix error in formulation of eq. 15.5.

In the book  Power System Modelling and Scripting(2010) eq.15.5.2 is formulated as ...D(ω-ω_s) similar to eq.15.5.1.  This formulation is also applied in page 244, taking  ω_s to be 1. Here delta formulation is also in agreement with the omega formulation. This is different with the current state of the code which uses;
D * (ω - 1.0) / ω instead of D * (ω - ω_s).

thanks for the contribution @kyumcu we have been coming back and forth with this on several occasions and decided to implement the existing model given that it has a closer match with multimachine models. The model in  Power System Modelling and Scripting(2010) is also widely used, and both are equivalent when ω_s = 1.
Since this has been an ongoing discussion within the development team, we will review this again, but if we decide to keep the existing model, we can help you parameterize the simulation in an equivalent way.
cc. @rodrigomha and @tavovalmo

In comparisons with PSSe we've seen that the method that we currently use seems to get better results when we use a device as a reference bus.
As you mention, Milano's book uses the ω_s = 1, that is usually one. The problem with using the damping with ω_s is that it is possible without Turbine Governor the machines will end up oscillating along the reference machine without converging back to a stable value of frequency.
We are looking into adding a shaft model that allow users to choose the reference that you want and if we divide or not by the rotor speed in the damping.


selection-in-new-tab should not be enabled everywhere

selection-in-new-tab is enabled on every page and it attaches a keypress handler even when not necessary. At the very least, this feature should be limited to places with navigations using include, like isRepoConversationList, isRepoTree and isNotifications
Are there any other pages where this feature applies?

I would rather do a select where exists .navigation-focus a.js-navigation-open[href]

I think that really applies to every global feature. We could query the DOM everywhere or we could use a more efficient include

hide-navigation-hover-highlight should hare the same include list
navigate-pages-with-arrow-keys same for I think
reload-failed-proxied-images likely only markup files and hasComments. But those might be enough pages that maybe it can stay global

The rest of them can really be on almost every page

This issue has been open for too long. It looks like an easy change. Should we leave it longer for someone new to come along and pick it up? Or shall I pick it up?

Or shall I pick it up?

It's all yours! It's mostly a matter of exploring GitHub and looking for pages that have lists (commits, files, notifications):

Open page
Press j to see if there's any selectable item

Please try to be thorough in this search or else we'd end up disabling it on pages that use it

I was looking for small, quick wins :wink: This is probably not it. I think I have already amassed enough issues for myself to do over the weekend :sweat_smile: But if no one gets to it, I will pick it up.

Done in #5727


Clean up

Coverage remained the same at 100.0% when pulling c3fcd00e25cdc5456a065feb05df0214c310b171 on clean-up into 9e2464a9009b79d503d569c5caae75edad37d4b8 on master.


Avoid executing child plan twice in CoalesceExec

Signed-off-by: Andy Grove<EMAIL_ADDRESS>This is a small performance optimization to avoid executing a child plan twice in GpuCoalesceExec when numPartitions == 1.
The same issue exists in Spark: https://issues.apache.org/jira/browse/SPARK-35767

build


migrate frontend to typescript (#60)

moving to typescript:
major things,

.js, .jsx files now .ts, .tsx
tsconfig.json added to root dir
we never pass "props" always a typed object with a defined interface
interfaces defined and xported from the service files
components changed to down typed objects rather than individual fields
all function args and local variables / useStates etc have types defined

sorry the diff isn't so easy to read because its technically deleting / creating new files, not much I can do about that
all my type interfaces are named OpenAI*, such as OpenAIMessage, OpenAIResponse, OpenAIDefence etc, feel free to propose better names

noticed this will almost certain clash with George's PR, sure merge that first then I'll make a few changes here


Svelte component inside virtual swiper not rendered correctly when prepending slides

Swiper Version: 6.6.1
Svelte Version: 3.38.2
Platform/Target and Browser Versions: Tested on MacOS BigSur 11.3.1 with Safari 14.1 (166<IP_ADDRESS>.6), Chrome 90.0.4430.212 and Firefox 88.0.1

What You Did
I try to create an online calendar using Framework7, swiper and svelte. I want to create an infinite calendar where the next day gets loaded as soon as the first or last slide is active.
https://codesandbox.io/s/swiper-svelte-fork-forked-r2z7p?file=/App.svelte
In the codesandbox demo, when you swipe two times, so that a new slide gets prepended, the component gets rerendered with wrong properties and therefore show the wrong day. But if I skip the svelte component and include its source code directly in the slider, everything works as expected. You can comment out the svelte component on line 115 and uncomment the div on line 113 to test this behaviour.
The problem occurs only if the slideTo method is called. When I append the slides, which does not call the slideTo method, everything works just fine.
Expected Behavior
I expect that svelte components get rendered correctly when prepending them in a slide.
Actual Behavior
The svelte component gets rendered with wrong properties.

I have made a simpler test case: https://codesandbox.io/s/swiper-svelte-fork-forked-rgxie?file=/App.svelte.
Inside the swiper.virtual.prependSlide method, the things that are happening are basically the following:

The new slide is unshifted to swiper.virtual.slides.
swiper.virtual.update(true) is called.
swiper.slideTo() is called to get to the new index.

I have implemented this behaviour in the prependDate function on line 46, but skipped the slideTo call.
You can see that when you click on the prepend button when you are on the first slide, nothing gets rerendered. If then the slideTo method is called inside the prependSlide method, the swiper goes 'back' to the next slide, because the new slide is not rendered yet.
But if you skip the slideTo call and swipe forward a few times to force a rerender and then swipe back, you can see that the new slide gets rendered correctly.
So the problem is that the new slides do not get rendered when prepending a slide, and not an issue in the slideTo method as I initially thought.

Here in Svelte, you shouldn't use swiper.virtual.prependSlide/appendSlide methods, you should just modify your initial virtualSlide array with push/unshift but I see it also have some issue

Fixes in Swiper 6.6.2. Here is the full working example with logic for example https://github.com/nolimits4web/swiper/blob/master/playground/svelte/Virtual.svelte

Hello @nolimits4web the svelte example is missing. Any advice how to use it with svelte?


Sample usage

Hi Hongjiang,
Could you please double check the -c parameter? In the description true is for mooncake, but you specify false in the same usage.
Cheers,
Shing

Fixed it.
Thanks Shing.


Result codes should be the same everywhere

Result codes returned by Horizon are different that those in XDR and our docs. For example, trying to send Payment to the account that doesn't exist results in op_no_destination code but it's PAYMENT_NO_DESTINATION in XDR and in devportal. To handle all possible cases properly a developer needs to check the source of Horizon to figure out what are the error codes.

So, horizon's goal is to make it easier on others to develop client software that integrates with the stellar network.  Given the design and other restrictions regarding our XDR schema, the error messages, IMO, make it harder to integrate with the network.
For example, your example above is technically incorrect.  If you try to send a PaymentOp to an account that doesn't exist the error from stellar-core is PAYMENT_NO_DESTINATION, but that's not the whole truth.  A PathPaymentOp that fails for the same reason fails with PATH_PAYMENT_NO_DESTINATION.
To lower the number of flags that a developer must learn, I chose not to mirror the raw xdr errors.  In this case, a developer need not know that the source operation was a path payment, they simply check their errors for "op_no_destination".
I'm happy to discuss this further, but I'm going to close the issue because I don't believe it is cut and dry that we should sync our error codes with the underlying stellar-core errors.   Instead, I believe we should update our documentation to use horizon's presentation of the errors or otherwise make clear the distinction between stellar-core's behavior and horizon's.   Created this issue: https://github.com/stellar/horizon/issues/258


IsolatedStorage directory creation (and non-reclamation)

VSCode Version: April 2018 (version 1.23)
OS Version: MacOS Sierra 10.12.6

Steps to Reproduce:
0. Use Live Share

Open a directory of files using code .
Make a change to a file, like adding whitespace to the front of a line, adjusting the text.
In the directory that you were just in. You will now see IsolatedStorage/. It will never go away without manual removal now.

Does this issue occur when all extensions are disabled?: No
Thank you for this prompt!
I dug in. When I removed live share, this stopped happening. So the issue is live share creating this directory. Added step (0) above.

Please file issue against Live Share ;)

Same issue. I don't have LiveShare installed.
Using Current VS Code 1.23.0, Mac OS 10.13.4


child directory deletion with files in does not fire an event when using 'all' on windows

Hi,
I have seen this issue raised a couple of times now through the year,
issue
When using the ".on('all'" event on a parent directory and all its files, if you delete a sub directory which has files no event is triggered but if you use ".on('raw'" and you do the same process you can see its triggering the event as "rename".
temporary fix
Is there any way to chain event orders so as a fix we could say if the 'all' event has not been triggered then use raw and check the event parameter is "rename", this way it will fix the issue short term while its looked into why 'raw' finds the event trigger but 'all' does not

I'm waiting, too


Fatal error if the OUTPUT_DIR contains spaces

New MR to fix #138 issue :
Fix in pgcluu for OUTPUT_DIR like vars always be quoted (when used in external command calls), especially if their value may contain a whitespace or separator character.

Thanks a lot Nicolas!


Repair unit-tests to be compatible with ava 2.4.0

The unit-tests are repaired in this pull-request. So they are compatible with last release of
ava:
According to 1696, the private variable t._test was removed in Feb. 2018.
The rejected promises now are verified with throwsAsync. So these test don't fail anymore.
@SamVerschueren:
I hope you can review these changes asap.

Codecov Report

Merging #137 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #137   +/-   ##
=======================================
  Coverage   96.31%   96.31%
=======================================
  Files           7        7
  Lines         190      190
=======================================
  Hits          183      183
  Misses          7        7

Impacted Files
Coverage Δ

lib/task.js
100% <ø> (ø)
:arrow_up:

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 0fb0c38...dfbda75. Read the comment docs.

The continuous-integration build fails, because the latest xo version doesn't support node6 anymore.


not in gzip format ??

Got this when running mandb today - note that c3270 is not installed (it was but it was removed)

removed c3270.1.gz and reran mandb and no issues and no more stray cats :)

Interestigly, that is not in the c3270 package for me. I only have man1, not cat1: ~/zopen/usr/local/zopen/c3270/c3270/share/man/man1/

What can I say - if there is an issue/bug/gap/etc I'll find it - but I'm not really trying to

What can I say - it was there for me (obviously) - I'm not trying to find issues - they seem to find me.


export *** not found error 발생  건

ERROR in ./src/samples/GroupChannelTypes.js 191:42-65
export 'UserMessageUpdateParams' (imported as 'UserMessageUpdateParams') was not found in '@sendbird/chat/message' (possible exports: AdminMessage, AppleCriticalAlertOptions, BaseMessage, FileMessage, MentionType, MessageMetaArray, MessageModule, MessageRequestHandler, MessageSearchOrder, MessageSearchQuery, MessageType, MessageTypeFilter, OGImage, OGMetaData, PreviousMessageListQuery, PushNotificationDeliveryOption, Reaction, ReactionEvent, ReactionEventOperation, ReplyType, Sender, SendingStatus, ThreadInfo, ThreadInfoUpdateEvent, Thumbnail, UserMessage)
import SendbirdChat, { UserUpdateParams } from '@sendbird/chat';
import {
GroupChannelHandler,
GroupChannelModule,
GroupChannelCreateParams,
} from '@sendbird/chat/groupChannel';
import {
UserMessageCreateParams,
MessageListParams,
UserMessageUpdateParams,
FileMessageCreateParams
} from '@sendbird/chat/message';
GroupChannelCreateParams,
UserMessageCreateParams,
MessageListParams,
UserMessageUpdateParams,
FileMessageCreateParams
동일한 오류 발생합니다.

I have  same issue.
Failed to compile.
Attempted import error: 'UserMessageUpdateParams' is not exported from '@sendbird/chat/message' (imported as 'UserMessageUpdateParams').
ERROR in ./src/samples/GroupChannelTypes.js 191:42-65
export 'UserMessageUpdateParams' (imported as 'UserMessageUpdateParams') was not found in '@sendbird/chat/message' (possible exports: AdminMessage, AppleCriticalAlertOptions, BaseMessage, FileMessage, MentionType, MessageMetaArray, MessageModule, MessageRequestHandler, MessageSearchOrder, MessageSearchQuery, MessageType, MessageTypeFilter, OGImage, OGMetaData, PreviousMessageListQuery, PushNotificationDeliveryOption, Reaction, ReactionEvent, ReactionEventOperation, ReplyType, Sender, SendingStatus, ThreadInfo, ThreadInfoUpdateEvent, Thumbnail, UserMessage)
ERROR in ./src/samples/GroupChannelTypes.js 203:36-59
export 'UserMessageCreateParams' (imported as 'UserMessageCreateParams') was not found in '@sendbird/chat/message' (possible exports: AdminMessage, AppleCriticalAlertOptions, BaseMessage, FileMessage, MentionType, MessageMetaArray, MessageModule, MessageRequestHandler, MessageSearchOrder, MessageSearchQuery, MessageType, MessageTypeFilter, OGImage, OGMetaData, PreviousMessageListQuery, PushNotificationDeliveryOption, Reaction, ReactionEvent, ReactionEventOperation, ReplyType, Sender, SendingStatus, ThreadInfo, ThreadInfoUpdateEvent, Thumbnail, UserMessage)
ERROR in ./src/samples/GroupChannelTypes.js 224:36-59
export 'FileMessageCreateParams' (imported as 'FileMessageCreateParams') was not found in '@sendbird/chat/message' (possible exports: AdminMessage, AppleCriticalAlertOptions, BaseMessage, FileMessage, MentionType, MessageMetaArray, MessageModule, MessageRequestHandler, MessageSearchOrder, MessageSearchQuery, MessageType, MessageTypeFilter, OGImage, OGMetaData, PreviousMessageListQuery, PushNotificationDeliveryOption, Reaction, ReactionEvent, ReactionEventOperation, ReplyType, Sender, SendingStatus, ThreadInfo, ThreadInfoUpdateEvent, Thumbnail, UserMessage)
ERROR in ./src/samples/GroupChannelTypes.js 289:33-49
export 'UserUpdateParams' (imported as 'UserUpdateParams') was not found in '@sendbird/chat' (possible exports: ApplicationUserListQuery, BannedUserListQuery, BaseChannel, BlockedUserListQuery, ChannelType, ConnectionHandler, ConnectionState, Emoji, EmojiCategory, EmojiContainer, FriendListQuery, LogLevel, MemoryStore, MutedUserListQuery, OperatorListQuery, Plugin, PushTemplate, PushTokenRegistrationState, PushTokenType, PushTriggerOption, ReportCategory, RestrictedUser, RestrictionInfo, RestrictionType, Role, SendbirdChatOptions, SendbirdError, SessionHandler, User, UserEventHandler, UserOnlineState, default)
ERROR in ./src/samples/GroupChannelTypes.js 1160:34-51
export 'MessageListParams' (imported as 'MessageListParams') was not found in '@sendbird/chat/message' (possible exports: AdminMessage, AppleCriticalAlertOptions, BaseMessage, FileMessage, MentionType, MessageMetaArray, MessageModule, MessageRequestHandler, MessageSearchOrder, MessageSearchQuery, MessageType, MessageTypeFilter, OGImage, OGMetaData, PreviousMessageListQuery, PushNotificationDeliveryOption, Reaction, ReactionEvent, ReactionEventOperation, ReplyType, Sender, SendingStatus, ThreadInfo, ThreadInfoUpdateEvent, Thumbnail, UserMessage)
ERROR in ./src/samples/GroupChannelTypes.js 1173:35-59
export 'GroupChannelCreateParams' (imported as 'GroupChannelCreateParams') was not found in '@sendbird/chat/groupChannel' (possible exports: CountPreference, GroupChannel, GroupChannelCollection, GroupChannelEventContext, GroupChannelEventSource, GroupChannelFilter, GroupChannelHandler, GroupChannelListOrder, GroupChannelListQuery, GroupChannelModule, GroupChannelSearchField, HiddenChannelFilter, HiddenState, Member, MemberListOrder, MemberListQuery, MemberState, MemberStateFilter, MembershipFilter, MessageCollection, MessageCollectionInitHandler, MessageCollectionInitPolicy, MessageEventContext, MessageEventSource, MessageFilter, MutedMemberFilter, MutedState, MyMemberStateFilter, OperatorFilter, PublicChannelFilter, PublicGroupChannelListOrder, PublicGroupChannelListQuery, QueryType, ReadStatus, ScheduledMessageListOrder, ScheduledMessageListQuery, ScheduledStatus, SuperChannelFilter, UnreadChannelFilter, UnreadItemKey)

I have the same issue. Any solutions ?


How can we grab a copy of the original value without the mask being applied?

I'm in a situation where in addition to formatting the value with the mask to display for users (react-input-mask handles this very well), I also need to preserve the raw value of the user input without any formatting.
Is there a recommended way to get this raw value? Would I need to create my own "unmask" method to do that?

any updates/plan for this?