bug: checker = true and : hangs on windows 11

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.11.0-dev-345+g3e6cec0be
Operating system/version
Windows 11
Describe the bug
When checker = { enabled = true}or you hit :, neovim automatically hangs, taking up 1 cpu.
Steps To Reproduce

Set checker = { enabled = true}
Hangs after around 0.5sec, with one cpu at 100%

Expected Behavior
Neovim does not hang
Repro
local lazypath = vim.fn.stdpath("data") .. "/lazy/lazy.nvim"
if not (vim.uv or vim.loop).fs_stat(lazypath) then
  local lazyrepo = "https://github.com/folke/lazy.nvim.git"
  local out = vim.fn.system({ "git", "clone", "--filter=blob:none", "--branch=stable", lazyrepo, lazypath })
  if vim.v.shell_error ~= 0 then
    vim.api.nvim_echo({
      { "Failed to clone lazy.nvim:\n", "ErrorMsg" },
      { out, "WarningMsg" },
      { "\nPress any key to exit..." },
    }, true, {})
    vim.fn.getchar()
    os.exit(1)
  end
end
vim.opt.rtp:prepend(lazypath)

require("lazy").setup({
  spec = {
    -- add LazyVim and import its plugins
    { "LazyVim/LazyVim", import = "lazyvim.plugins" },
    -- import/override with your plugins
    { import = "plugins" },
  },
  defaults = {
    -- By default, only LazyVim plugins will be lazy-loaded. Your custom plugins will load during startup.
    -- If you know what you're doing, you can set this to `true` to have all your custom plugins lazy-loaded by default.
    lazy = false,
    -- It's recommended to leave version=false for now, since a lot the plugin that support versioning,
    -- have outdated releases, which may break your Neovim install.
    version = false, -- always use the latest git commit
    -- version = "*", -- try installing the latest stable version for plugins that support semver
  },
  install = { colorscheme = { "tokyonight", "habamax" } },
  checker = { enabled = true }, -- automatically check for plugin updates
  performance = {
    rtp = {
      -- disable some rtp plugins
      disabled_plugins = {
        "gzip",
        -- "matchit",
        -- "matchparen",
        -- "netrwPlugin",
        "tarPlugin",
        "tohtml",
        "tutor",
        "zipPlugin",
      },
    },
  },
})

Shouldn't you post this on lazy.nvim repo?

Yes, please open this in the lazy.nvim repo.
And do include steps to reproduce, since LazyVim works fine for me on Windows.


Cannot call method "getRange" of null

Hi there,
i've tried to follow the easy guide, but I'm stuck with this error when I try to run overview.gs, and I'm only left with an empty sheet (only details got somehow populated with some data).
If I try to run the overview.gs, i get that error on lines 44 and 45
sheetDashboard.getRange("B1").setValue(lifeTimeData); //write lifetime data, in MWh, to dashboard sheet

sheetDashboard.getRange("D1").setValue(lastYearData); //write last year data, in kWh, to dashboard sheet

any hints?

I'm not sure exactly what the issue is, but I started by trying to build my own sheet and adding the code to it. That was a mistake - I got the same errors you did. When I copied the example sheet over to my drive and put in my Site ID, API Key, and Inverter Serial it worked just fine.

After learning a bit more about how this works, it looks like the above error happens when the cells you're trying to paste into don't exist. Make sure there are enough destination cells to paste all of your data and you should be good.


Create a documentation site for the ACA-Py that incorporates and organizes the current Markdown documentation

Create a documentation website for ACA-Py that provides both a useful structure for the documentation, adds introductory material to organize the information for different audiences and incorporates the many existing markdown documents we have in the repo.

For AFJ we use https://docusaurus.io/, source code can be found here: https://github.com/hyperledger/aries-javascript-docs/

@amanji has been experimenting with Material for Mkdocs since Hyperledger has the "insider" edition, and Docusaurus, because of the AFJ success with it.  Leaning is Docusaurus, perhaps built into ACA-Py repository, or perhaps as a separate site.  One of the features we'd like to keep is the generated internal ACA-Py code documentation that we get from Sphinx and ReadTheDocs, but the main focus of the site will be developers getting started with ACA-Py, vs. contributors/maintainers.

I think this is resolved by #2079

Soon🤞. Need to create a workflow for a multi branch build before that can be merged.

First cut of site launched currently at: https://hyperledger.github.io/aries-acapy-docs
I’ll be changing the URL to a custom domain later today, and adding the 0.8.0 docs, then the 0.7.5, and likely the last of the 0.6.X line of docs as well.
I’ll probably do some more maintenance on the docs in main.
Feedback welcome!


Fix indentation and enable syntax highlighting

When you browse the source of the python client on Github the first example you meet has the indentation wrong. This is a shame.
Also, I believe the readability can be improved by utilizing the syntax highlighting supported by Github.

Hi can you please sign here https://ravendb.net/contributors/cla/sign

@IdanHaim Done.


Add brew integration description

Just adding a brew integration description.

Merged with master in order to pass the tests.
In any case, this PR is just documentation, it does not touch the code really, but it looks better this way ;)


Camera normal in z direction

As commented in this line, why is the normal -1 instead of 1 in z?
In my case, using 1 gives the correct results. And thanks for your user-friendly work!

It wasn't my choice to change it to -1. We had someone previously refactor the code and this was one of the changes made. My guess is that it might be a specific configuration to cater for how certain cameras/lidars pick up the chessboard (maybe the rotation convention/ left-hand coordinate system or something). Though I apologize I don't have more time to look into this.
For others who are facing similar issues, it's good that this may be a potential fix.


Leaflet Pins

Make pins different between hatcheries and offices.
It would also be super cool if we could make clicking a pin on the map highlight the info in the table.
Set boundaries so the map cannot scroll outside of the PNW.  Picky, but yah.

Current two problems with this request. First, the only data available to leaflet right now is the location name, so making them appear different based on an attribute requires additional information sourced from related tables.  Locations are currently related to two child tables (facility and task) that will need to be linked.  Second, the facility table doesn't house any information about facility type that could be used to distinguish hatcheries from offices.  That field could be added but it doesn't exist now.


version: post-release bump

Signed-off-by: Lars Holm Nielsen<EMAIL_ADDRESS>

Coverage remained the same at 76.696% when pulling 8d1a66e0c6847c462b3924a1adbf54a4abcfd54d on lnielsen:postrel into 9e6f407a1b44a78059a1f80501b21d2ddc3cb261 on inveniosoftware:master.

Coverage remained the same at 76.696% when pulling 869beb5626f83697f9180ab1c78e7bf0dbda399a on lnielsen:postrel into 75dcbf7a19e99b1846e2de507381adaa15abbf6b on inveniosoftware:master.


Add each method to Travel

Hello,
This PR adds the each method to Travel class. It works on any iterable, including CarbonPeriod, which could be useful.
Thanks,
Otto

Thank you @ottoszika ❤


fix: copy static prerendered routes to the static directory

This PR does the following before processing a directory for functions:

Checks for any prerendered route configs.
Checks if prerendered routes already exist in the static assets directory.
Copies prerendered routes to the static assets directory.
Returns a filtered list of paths to process that excludes the valid prerendered routes.
Applies overrides to the build output for prerendered routes, like what would normally happen with static pages in the build output config.
Adds tests for the new behaviour.

Example screenshots of the result from running the pre-release.


removed platform name

automatic PR created to update the booster #9

automatic PR created to update the booster https://api.github.com/repos/snowdrop/spring-boot-http-booster/pulls/50

automatic PR created to update the booster https://api.github.com/repos/openshiftio-vertx-boosters/vertx-http-booster/pulls/10

merged: https://github.com/openshiftio/appdev-documentation/commit/de43cc39a8d667bed4b40b2e9554a9b4ba3f34a7

automatic PR created to update the booster https://github.com/snowdrop/spring-boot-http-booster/pull/51

automatic PR created to update the booster https://github.com/openshiftio-vertx-boosters/vertx-http-booster/pull/11

automatic PR created to update the booster https://github.com/wildfly-swarm-openshiftio-boosters/wfswarm-rest-http/pull/16


Conversion to Ketting

@mattbishop I was gonna adopt eslint from ketting with these settings:
https://github.com/badgateway/ketting/blob/main/.eslintrc.json
One thing is that they do include and auto-fix semicolons, but I can change the settings to not do that. Let me know if you're down with eslint and if you prefer to take all semi-colons out. I slightly prefer them but definitely not enough to argue about it =)

Forgot to fix the unittests, will take some more time because they rely on Undici's mocking support (which is neat!) but this uses node-fetch under the hood and there's no equivalent =(

I like eslint, as long as we can burn the semicolons with fire! I spent
many years writing C and Java, so it reminds me of where I am to not use
semicolons.
On Wed, Mar 15, 2023 at 4:57 PM Evert Pot @.***> wrote:

@mattbishop https://github.com/mattbishop I was gonna adopt eslint from
ketting with these settings:
https://github.com/badgateway/ketting/blob/main/.eslintrc.json
One thing is that they do include and auto-fix semicolons, but I can
change the settings to not do that. Let me know if you're down with eslint
and if you prefer to take all semi-colons out. I slightly prefer them but
definitely not enough to argue about it =)
—
Reply to this email directly, view it on GitHub
https://github.com/evently-cloud/cli/pull/9#issuecomment-1470998688, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACY3BLTFSWMXOHUYQG5BRDW4JJNJANCNFSM6AAAAAAV4JQRQU
.
You are receiving this because you were mentioned.Message ID:
@.***>

I like eslint, as long as we can burn the semicolons with fire! I spent
many years writing C and Java, so it reminds me of where I am to not use
semicolons.

Alright! Will take some getting used to =)

I'll do that in a later PR so this one doesn't get too noisy.

Hey @mattbishop ,
I tried to fix the unittests but ran into all kinds of strange issues with types. I suspect that this is because partially the codebase thinks it's working with the new built-in Request and Response classes from fetch, and partially those from node-fetch.
I haven't really had issues with this in the past, because they are similar enough. One area they distinctly aren't is how they do streams, which is where all my issues are.
So one idea I have is to stop using node-fetch. This is possible if I release Ketting 8 as a beta. I've already made this change here, but I wanted to run this by you in case you believe this may constitute scope creep.
Pretty frustrating TBH! You can see also how I've tried to implement mocking functionality with the ledger:download test. Not super happy with the verbosity there so I thought it might also be helpful there to add some utilities that make it easy to see if an incoming HTTP request roughly matches a pattern.
LMK, so far I've spent about 6 hours on the entire project.

@mattbishop ready for a deeper review, I did all the above and everything now works the way I wanted.
I will follow up with another PR that removes all the semicolons (permanently)


in profile section save data should be visible to user after input so that whenever user want to edit it should be edited

@RilAbbSnehaM you can start working now

@Jiggy9 the Firebase Storage security rules are not allowing the user to upload files.for this, you need to update your Firebase Storage security rules to allow authenticated users to upload files.

@Jiggy9 the Firebase Storage security rules are not allowing the user to upload files.for this, you need to update your Firebase Storage security rules to allow authenticated users to upload files.

Mail or LinkedIn me, I'll give you  access to firebase you can modify it accordingly.

can u snd me linkedin id @Jiggy9

can u snd me linkedin id @Jiggy9

You can check out it on my GitHub profile

@Jiggy9 I snd u request u can accept and msg me I unable to msg u bcoz it req premium

Uploading WhatsApp Video 2024-10-10 at 01.37.51_c6a512a4.mp4…
done

@Jiggy9 close this issue


Removed original AB3 commands that are not needed.

Coverage increased (+2.0%) to 54.494% when pulling eb90b5887c0c481c3dcd05ef77cd944b93f94bec on kianhong95:RemoveOldCommands into 2feed6f17b1e2ad045a383a186a6251cc7ad7021 on CS2113-AY1819S1-F09-2:master.


Upgrade to Gatsby 3

Currently we are at Gatsby 2, which holds back our image processing.
By upgrading to Gatsby v3, we can:

[ ] Upgrade sharp and corresponding image packages(gatsby-plugin-sharp, gatsby-transformer-sharp, gatsby-image, etc.)
[ ] Upgrade to Node version 16(Currently at 15)

Issues that will need to be resolved when we upgrade:

[ ] Upgrading to v3 will likely break our custom caching mechanism, we'll need to make sure it's working and correct any issues that may arise.

@julieg18 this is a great summary, thanks! I wonder if we have some ticket related to caching already (cc @rogermparent )?

I couldn't find one, but there are quite a few issues, I could have easily missed one 🤔

#1063 is an old ticket about image caching that can provide some context, particularly the latest comment from me on there can be considered a call for this Issue (though I think it was still Gatsby 2 then)

our caching mechanism still breaks when deploying. Though I'm not sure if its for a different reason than last time we attempted this.

When I mention the caching mechanism failing, I'm specifically referring to how after all errors are squashed and builds are successful, cache is invalidated on every build. Builds succeed, but take over twice the amount of time because all images are re-processed after each build.
There were regular errors like that one on the first attempt, but even after solving all of those the caching mysteriously didn't work despite us using a hand-rolled version of effectively the same cache preservation method other services like Netlify successfully do (upload public and .cache after build, download before next build).


Fix connection status badge display

Fixes https://github.com/microsoft/azuredatastudio/issues/11873
Don't use absolute positioning so that the badge can be indented properly.

Coverage decreased (-0.002%) to 40.257% when pulling 6cc7375bdccc752b80268620139b053e0b3e0fc0 on chgagnon/fixBadges into d96e83c3f03a58084835d5bae2eb6dfb412fa3f2 on main.


RDKDEV-991 - Add additional methods in UsbAccess plugin

Reason for change: unmountUSB, getUSBSize, getFileInfo, getMountedInfo, getPartitionInfo, getPartitionCount, getDeviceCount methods added in UsbAccess plugin. Support for 2 USB also added.
Risks: Low
Test Procedure: Use curl commands and verify all the commands give proper responses
From: lanyujie<EMAIL_ADDRESS>zhangguangchen<EMAIL_ADDRESS>niuben<EMAIL_ADDRESS>

Review on changes done by @srikanth-vv in https://github.com/rdkcentral/rdkservices/pull/5144
Added and verified the suggested changes and got them approved by @srikanth-vv

USB Access Plugin will be deprecated soon and there are new plugins being developed in its place. It would be worth taking these changes to the new plugin than merging these changes into usb access plugin, as it brings low value. Please hold on with this change until new plugins are introduced.


@font-face supported？

This is amazing , but is css3 @font-face supported? The doc doesn't mentioned?

Radium doesn't have any explicit support for @font-face.. We definitely could add something similar to the @keyframes helper.

We could add @font-face support, but I don't know if we really need to. I don't think we would gain much from handling it in Radium.

@alexlande I would benefit from using this in my custom <Text /> and <Header /> components to remain completely independent and modular.
Currently, I have to add the @font-faces in my html head's inline styles, which leaves my components coupled. Can you think of a current Radium way to get around this? (I'm still learning Radium)

No, there's nothing built in to Radium to support that use case. I do understand the inclination for all components to be self-contained, but I don't know if it's a common enough case to add it to the lib.
I would expect the vast majority of apps to handle their own webfont loading (either through a font service or through their own @font-face declarations, which might be inlined in the head, or lazy loaded, or etc)

That's fair. I think Aphrodite has a more applicable use case than mine for why this feature might be worth adding: https://github.com/Khan/aphrodite#font-faces

FWIW you should be able to do this fairly easily with a Radium plugin.

@ianobermiller Good idea - thanks! I'll let you know if I end up with anything worth sharing.

Maybe that support could be helpful when you are working on a library of components instead of the app itself. A company might ensure implementing that library, that all the company's apps match the corporate UX guidelines. And that frequently involves the font family.
The plugin approach souns interesting.

I've been searching for a way to include base64 encoded fonts into react so that I don't need to use a font-service, include the clunky base64 code in the index.html file or separate stylesheet. By storing them as data, they would load with the page, avoiding the fonts popping in after the fact on slower connections. The software I'm writing is lightweight enough (thanks to Radium) that fonts are currently the only piece of the puzzle requiring me to load a separate css file.


Retrieve email messages from Outlook with smtp addresses of TO & CC fields

[Enter feedback here]

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

ID: 34835c44-604e-9820-db62-c7fe2abdf1bf
Version Independent ID: 8734bb69-fa2b-4d92-d595-b1d763108caf
Content: Outlook - Power Automate
Content Source: articles/ui-flows/desktop/actions-reference/outlook.md
Service: flow
GitHub Login: @mariosleon
Microsoft Alias: marleon

%CurrentEmail['To']% fetches email address in X500 format, need an option to either convert it to smtp format.

@malayvir Thank you for your feedback. This appears to be a feature request, please go ahead and share this with the product team in the following link: https://powerusers.microsoft.com/t5/Power-Automate-Ideas/idb-p/MPAIdeas


[AMBARI-23112] [trunk] Some libraries are not relocated in ambari-metrics-common.

What changes were proposed in this pull request?
Fixed shading to include all of jackson and commons jars.
How was this patch tested?
Ran 'mvn clean package' and verified the JAR contents.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/1560/
Test FAILed.
Test FAILured.


Docker gui not showing with wayland and Fedora 38

With Wayland and Fedora 38 the container will start just fine, but the gui never pops up. No obvious error codes or stack trace. When I switch over to x11 everything works just fine. I tried removing the Wayland check and just forcing the xhost +local part of the code to run at the end, but issue persists.
Sorry I don't have more details or a solution. Not a blocker for me, but figured I would call it out.

What graphics card? (nvidia/others)
What container solution? (docker/podman)
What command is run to start the container?

i.e podman run -d --rm --privileged -e DISPLAY=:0 -v /tmp/.X11-unix:/tmp/.X11-unix -v /run/user/1000/pulse:/run/user/1000/pulse -v zwift-netbrain:/home/user/Zwift --userns=keep-id --gpus all docker.io/netbrain/zwift:latest

Graphics Card: NVIDIA Corporation GP107GLM [Quadro P2000 Mobile]
Kernel driver in use: nouveau
Container solution: docker (same result with podman though)
Command to start container
`#!/usr/bin/env bash
set -x
Set the container image to use
IMAGE=${IMAGE:-docker.io/netbrain/zwift}
The container version
VERSION=${VERSION:-latest}
CONTAINER_TOOL=docker
Check for updated container image
if [[ ! $DONT_PULL ]]
then
$CONTAINER_TOOL pull $IMAGE:$VERSION
fi
VGA_DEVICE_FLAG="--device /dev/dri:/dev/dri"
Start the zwift container
CONTAINER=$($CONTAINER_TOOL run
-d
--rm
--privileged
-e DISPLAY=$DISPLAY
-v /tmp/.X11-unix:/tmp/.X11-unix
-v /run/user/$UID/pulse:/run/user/1000/pulse
-v zwift-$USER:/home/user/Zwift
$([ "$CONTAINER_TOOL" = "podman" ] && echo '--userns=keep-id')
$VGA_DEVICE_FLAG
$IMAGE:$VERSION)
Allow container to connect to X
xhost +local:$($CONTAINER_TOOL inspect --format='{{ .Config.Hostname  }}' $CONTAINER)
`

could you try to run the following command and see what you get?
# podman
podman run -it --rm --privileged -e DISPLAY=unix$DISPLAY -v /tmp/.X11-unix:/tmp/.X11-unix --userns=keep-id --entrypoint=bash netbrain/zwift
sudo apt update && sudo apt -y install mesa-utils
glxinfo | egrep "(glx)|(rendering)"
glxgears
exit

# docker
docker run -it --rm --privileged -e DISPLAY=unix$DISPLAY -v /tmp/.X11-unix:/tmp/.X11-unix  --entrypoint=bash netbrain/zwift
sudo apt update && sudo apt -y install mesa-utils
glxinfo | egrep "(glx)|(rendering)"
glxgears
exit

You might also have to do xhost + before running the abovementioned.

Tried the docker version.
The glxinfo | egrep "(glx)|(rendering)" didn't output anything.
glxinfo by itself just outputs:
user@a294d42ac8a6:~$ glxinfo
name of display: :0
And this output from glxgears
user@a294d42ac8a6:~$ glxgears
X Error of failed request:  BadAlloc (insufficient resources for operation)
Major opcode of failed request:  147 ()
Minor opcode of failed request:  7
Serial number of failed request:  41
Current serial number in output stream:  43

Since your using nouveau you might need to add --device /dev/dri:/dev/dri, not sure tho.

But in any case if you can't get proper output from glxinfo or start glxgears, then something is wrong with the system setup.
What is your docker version? and are you running docker-desktop? There has been other users who had problems when running fedora and docker-desktop. But everything worked when installing the docker-engine
https://docs.docker.com/engine/install/fedora/

--device needs to be at the start, not the end.

glxinfo seems to just hang on me until after a few minutes I give in and kill it.
user@26728f9912e6:~$ glxinfo
name of display: :0
^C

Also just to re-iterate. I am perfectly happy using xorg. I would only recommend spending time on this if you think it might be a systemic issue you are interested in.

might have been related to #44 , could you check after the next build?


fix: added comments for Bot Analytics

Added comments
Fixed test cases

:tada: This PR is included in version 0.42.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Scalar and git workspaces

In the concept of git workspaces, do we register only the clone or the workspace or both?

You should only need to register the base, and then all the other worktrees will benefit automatically. This is because all of the object data and refs are shared across worktrees.

Will there be issue is someone register the workspace instead of the base or both?  Can scalar recognize workspace and indicate that they are already registered or something like that? As of today, scalar list report both when registering both.

I believe Scalar works with both, but I'm not sure without doing careful testing. If the scalar register command succeeds, then I would imagine that the rest works. Let me know what you find out after trying it.


Use Azure Identity to Authenticate with Microsoft Graph API

Hey, I just made a Pull Request!
Replaced MSAL with Azure Identity to support additional authentication methods with
Microsoft Graph - in particular Managed Identity.
If clientId and clientSecretare configured, the plugin will continue to behave as before using those credentials.  If those values are ommitted, thenDefaultAzureCredential` is choose the best method of authentication.
This is particularly helpful locally as if the Azure CLI or Azure Powershell are
installed, Azure Identity will use those to authenticate, without the developer
needing to configure anything.  In those cases, the following config will be
sufficient for a working graph plugin.
catalog:
  provider:
    microsoftGraphOrg:
      default:
        tenantId: ${AZURE_TENANT_ID}
        user:
          filter: accountEnabled eq true and userType eq 'member'
        group:
          filter: securityEnabled eq false and mailEnabled eq true and groupTypes/any(c:c+eq+'Unified')

Signed-off-by: Alex Crome<EMAIL_ADDRESS>
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

I think this PR is code complete, but I do need to do a bit of further work on the docs - what documentation should go in the plugin's readme vs documentation site (i.e. https://backstage.io/docs/integrations/azure/org ).  Right now the docs site is a bit broken for users on the stable version of backstage as it points the readme for master which has a signfiicantly overhauled configuration scheme.

@freben I've pushed the doc updates to go along with this.


Close on navigate

One more little tweak before I can replace the account menu with this.

Awesome :shipit:


Please publish a new version to npm

There is a bug in the version 0.4.0 published to npm, but which is fixed in commit  bc3cfcf5c2b35235b3dadc76f34fe49ff86a9f65.
The error causes a warning message "Warning: Invalid argument supplied to oneOf, expected an instance of array" to be printed to the console.

Wasn't watching this repository for some reason, I'll take care of it. Luckily it wasn't a form breaker or anything.
Please note that 1.0.0 may contain breaking changes, I'm not sure yet, but the internals of this are going to change a bit with the release of WP Libre Form 2.0.

Reopen this if 0.4.1 doesn't fix your issue.


Architectural pass-through

Now that we have shell features (menus, preview etc) and app features (field editing, navigation), we can firm up the application boundaries between Shell and Apps.

Tidy up existing app

Acceptance

[ ] We are all satisfied (esp @ionab and @stephenwf) that architectural separation of concerns is correct

Update wiki descriptions if they are not correct.
https://github.com/digirati-co-uk/iiif-manifest-editor/wiki/Shell etc


Create a release

following https://help.github.com/articles/creating-releases/
This is useful, as future commits might be less stable than the current one.

You might be tempted to delay it until contigs visualization. However, I recommend you make the current HEAD commit (reads visualization) version 0.1.0 and, the future contigs release as 0.2.0.

Done


Change asset and asset version keys to ints

Fusion uses strings to refer to assets and asset versions (the string corresponds to the asset's location under the asset root). This results in lots of duplicate, possibly long strings since every asset needs to hold the string referring to every other asset it has a relationship with (children, parents, inputs, and listeners). For large builds this can take lots of memory and use extra processing time since string comparisons are inefficient. We should update Fusion to use ints instead of strings when referring to assets and just store a single table relating strings to assets. This should reduce memory consumption and improve performance.

Closed via PR #1266


selected parameter doesn't work

As described in Label Studio — Choice Tag for Single Choice Labels, I would like to define a choice that will be selected by default.
I tried to define selected in the Choice tag, but it doesn't work as expected.
<!-- { "data": {"text": "Esse é um exemplo estúpido, então obrigado por nada a<EMAIL_ADDRESS>} -->
<View>
    <Text name="text" value="$text" />
    <View style="box-shadow: 2px 2px 5px #999; padding: 20px; margin-top: 2em; border-radius: 5px;">
        <Header value="Is this text toxic?" />
        <Choices name="is_toxic" toName="text" choice="single-radio" required="true" showInLine="true">
            <Choice value="Yes" hotkey="y" selected="true" />
            <Choice value="No" hotkey="n" />
        </Choices>
    </View>
    <View visibleWhen="choice-selected" whenTagName="is_toxic" whenChoiceValue="Yes">
        <Header value="Select categories" />
        <Choices name="Sentiment" toName="text" choice="multiple" showInLine="true">
            <Choice value="Identity Attack" />
            <Choice value="Insult" />
            <Choice value="Profane" />
            <Choice value="Racism" />
            <Choice value="Sexism" />
            <Choice value="Sexually Explicit" />
            <Choice value="Xenophobia" />
        </Choices>
        <Header value="There's a specific target?" />
        <Choices name="target" toName="text" choice="single-radio" showInLine="true">
            <Choice value="Individual" hotkey="I" />
            <Choice value="Group" hotkey="G" />
            <Choice value="Other" hotkey="O" />
        </Choices>
        <Header value="What words are toxic/offensive?" />
        <Labels name="BadWords" toName="text">
            <Label value="Curse words" hotkey="C" background="darkorange" />
        </Labels>
    </View>
</View>

Expected behavior
I would like that the choice "Yes" in the first question can be selected automatically when labeling start.
Screenshots

Environment (please complete the following information):

OS: Docker (Deployed in Heroku)
Label Studio Version: 1.4.0

Hi, @DougTrajano ! We've fixed such bug in previous version, so that's weird. I'll check it, thank you for notice!


Recipe for Revert Holy Ghrazi Rapier is incorrect

The recipe for reverting holy ghrazi rapier is currently:
holy rapier + holy kit -> ghrazi rapier
The holy kit should be in the output, not the input.
https://github.com/oldschoolgg/oldschoolbot/blob/06ed17ffd9ff8e4877b4445f6392abd24cc5d399/src/lib/data/creatables/tob.ts#L167

I also just noticed that there is no recipe for:
Holy sanguinesti staff (uncharged) -> Holy sanguinesti staff
Not sure if this is intentional, but I believe this means you can't use holy sang staff for CoX, etc.


Feature/auto change meaning field for FLEx

For a project that has been created by FLEx
and has just been received from USB, internet or ChorusHub
if there is no .WeSayConfig then project is new from FLEx so automatically set the meaning field to gloss
If the project has already had interaction with WeSay
then the person with FLEx should do the migration manually
for WS-472
3e7503e
@glasseyes
glasseyes
don't ask question but auto change to gloss mf

This change is

Reviewed 8 of 8 files at r1.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

src/WeSay.ConfigTool/ConfigurationWindow.cs, line 297 at r1 (raw file):

		/// </summary>
		/// <returns>true if the project was sucessfully opend</returns>
		public bool OpenProject(string path, bool newClone)

I noticed that most of the calls to this method pass "false" to the "newClone" parameter. You might want to consider giving this parameter a default value of "false". It is entirely up to you though. I just wanted to make you aware of the option.

src/WeSay.Project/WeSayWordsProject.cs, line 284 at r1 (raw file):

				return false;
			}
			var reader = XmlReader.Create(liftPath);

I believe that XmlReader implements IDisposable. If that is true, this should be wrapped in a using statement.

Comments from Reviewable

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

src/WeSay.ConfigTool/ConfigurationWindow.cs, line 297 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
I noticed that most of the calls to this method pass "false" to the "newClone" parameter. You might want to consider giving this parameter a default value of "false". It is entirely up to you though. I just wanted to make you aware of the option.

Done.

src/WeSay.Project/WeSayWordsProject.cs, line 284 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
I believe that XmlReader implements IDisposable. If that is true, this should be wrapped in a using statement.

Done.

Comments from Reviewable

Reviewed 2 of 8 files at r1, 3 of 6 files at r2.
Review status: all files reviewed at latest revision, 3 unresolved discussions.

src/WeSay.ConfigTool/ConfigurationWindow.cs, line 296 at r1 (raw file):

		///
		/// </summary>
		/// <returns>true if the project was sucessfully opend</returns>

typo "opened"

src/WeSay.ConfigTool/ConfigurationWindow.cs, line 297 at r1 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
I noticed that most of the calls to this method pass "false" to the "newClone" parameter. You might want to consider giving this parameter a default value of "false". It is entirely up to you though. I just wanted to make you aware of the option.

I agree a default param would be nice

Comments from Reviewable

Review status: 4 of 5 files reviewed at latest revision, 3 unresolved discussions.

src/WeSay.ConfigTool/ConfigurationWindow.cs, line 296 at r1 (raw file):
Previously, megahirt (Chris Hirt) wrote…
typo "opened"

Done.

src/WeSay.ConfigTool/ConfigurationWindow.cs, line 297 at r1 (raw file):
Previously, megahirt (Chris Hirt) wrote…
I agree a default param would be nice

still Done ;).

Comments from Reviewable

Reviewed 1 of 6 files at r2, 2 of 2 files at r3.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

Comments from Reviewable


Add range widget

see https://github.com/opendatakit/xforms-spec/issues/109

MVP done, but #528, #527 remaining


byrd: init at git-20200312

Motivation for this change
byrd is a simple terminal emulator.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 110831 run on Other x86_64-linux 1

1 package built:

byrd

That said, the resulting binary is mostly useless without the corresponding Byrd.conf also shipped (as perhaps an example.)

Result of nixpkgs-review pr 110831 run on Other x86_64-linux 1

1 package built:

byrd

That said, the resulting binary is mostly useless without the corresponding Byrd.conf also shipped (as perhaps an example.)

Revised.

Revised.

this was a sample PR to demonstrate the process of adding something to nixpkgs for someone. i do not think this is worth merging


polysemy-plugin: fix build

Motivation for this change
This is the second half of this change:
https://github.com/NixOS/nixpkgs/pull/83294#event-3171471057
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[X] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Done.
The Polysemy people are already aware of this, but presumably this takes time. But will monitor and push if necessary.

The Polysemy people are already aware of this, but presumably this takes time. But will monitor and push if necessary.

Please put a link in the comment you added to the upstream issue where this is discussed.
This will help the nixpkgs haskell maintainers know when this override can be removed.

This mentions both Stackage and the plugin version issue:
https://github.com/polysemy-research/polysemy/issues/322

Wait, I'll update the patch, too.

Wait, I'll update the patch, too.

@GrahamcOfBorg build haskellPackages.polysemy-plugin

@mikesperber You'll also have to mark polysemy-plugin as not broken.  See this video for instructions:
https://discourse.nixos.org/t/video-tutorial-how-to-fix-broken-haskell-packages-in-nix/3968

Done also.

@mikesperber When trying to build polysemy-plugin on this branch, I get the following build failure:
$ nix-build -A haskellPackages.polysemy-plugin --arg config '{allowBroken = true;}'
these derivations will be built:
  /nix/store/a9wnl65pksj4xli9z6mc9a3k4y38gifr-polysemy-plugin-<IP_ADDRESS>.drv
building '/nix/store/a9wnl65pksj4xli9z6mc9a3k4y38gifr-polysemy-plugin-<IP_ADDRESS>.drv'...
setupCompilerEnvironmentPhase
Build with /nix/store/dhnspmn5hc312lm46a98w4n1bz27bp40-ghc-8.8.3.
unpacking sources
unpacking source archive /nix/store/j8xpmj53gvxz1k7i04dqk0a9jq2565ng-polysemy-plugin-<IP_ADDRESS>.tar.gz
source root is polysemy-plugin-<IP_ADDRESS>
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file polysemy-plugin-<IP_ADDRESS>/test/VDQSpec.hs
patching sources
compileBuildDriverPhase
setupCompileFlags: -package-db=/build/setup-package.conf.d -j4 -threaded
[1 of 1] Compiling Main             ( Setup.hs, /build/Main.o )
Linking Setup ...
configuring
configureFlags: --verbose --prefix=/nix/store/3i318ghbzmjh6jljhjd2jwq2rw6l572s-polysemy-plugin-<IP_ADDRESS> --libdir=$prefix/lib/$compiler --libsubdir=$abi/$libname --docdir=/nix/store/75canmpmxy208kqyqgvl68diqy96lskr-polysemy-plugin-<IP_ADDRESS>-doc/share/doc/polysemy-plugin-<IP_ADDRESS> --with-gcc=gcc --package-db=/build/package.conf.d --ghc-option=-j4 --disable-split-objs --enable-library-profiling --profiling-detail=exported-functions --disable-profiling --enable-shared --disable-coverage --enable-static --disable-executable-dynamic --enable-tests --disable-benchmarks --enable-library-vanilla --disable-library-for-ghci --ghc-option=-split-sections --extra-lib-dirs=/nix/store/49x12wk4l8c0lh97crvwhjbzfxrq1fa2-ncurses-6.2/lib --extra-lib-dirs=/nix/store/hfs34n1yww7ba1m8gbrwrwh6v1y269m6-libffi-3.3/lib --extra-lib-dirs=/nix/store/l9nfr0znv7qibrapym32qj7dxmynfh84-gmp-6.2.0/lib --extra-lib-dirs=/nix/store/dhnspmn5hc312lm46a98w4n1bz27bp40-ghc-8.8.3/lib
Using Parsec parser
Configuring polysemy-plugin-<IP_ADDRESS>...
CallStack (from HasCallStack):
  die', called at libraries/Cabal/Cabal/Distribution/Simple/Configure.hs:1022:20 in Cabal-<IP_ADDRESS>:Distribution.Simple.Configure
  configureFinalizedPackage, called at libraries/Cabal/Cabal/Distribution/Simple/Configure.hs:475:12 in Cabal-<IP_ADDRESS>:Distribution.Simple.Configure
  configure, called at libraries/Cabal/Cabal/Distribution/Simple.hs:625:20 in Cabal-<IP_ADDRESS>:Distribution.Simple
  confHook, called at libraries/Cabal/Cabal/Distribution/Simple/UserHooks.hs:65:5 in Cabal-<IP_ADDRESS>:Distribution.Simple.UserHooks
  confHook, called at libraries/Cabal/Cabal/Distribution/Simple/UserHooks.hs:65:5 in Cabal-<IP_ADDRESS>:Distribution.Simple.UserHooks
  configureAction, called at libraries/Cabal/Cabal/Distribution/Simple.hs:180:19 in Cabal-<IP_ADDRESS>:Distribution.Simple
  defaultMainHelper, called at libraries/Cabal/Cabal/Distribution/Simple.hs:125:42 in Cabal-<IP_ADDRESS>:Distribution.Simple
  defaultMainWithHooks, called at src/Distribution/Extra/Doctest.hs:170:27 in cabal-doctest-1.0.8-2wQRz9ec1yB5lMDRmEgBWB:Distribution.Extra.Doctest
Setup: Encountered missing or private dependencies:
polysemy >=<IP_ADDRESS>

builder for '/nix/store/a9wnl65pksj4xli9z6mc9a3k4y38gifr-polysemy-plugin-<IP_ADDRESS>.drv' failed with exit code 1

It looks like you'll also have to update the version of polysemy that polysemy-plugin is using?

Argl, I had polysemy = super.polysemy_1_3_0_0 sitting somewhere else.
Patch updated.
I am sorry I'm such a klutz about this. Next time will be better hopefully.

Looks good. Thanks for working through this with me!


Black Copper MINI Thermal Printer BC-P58B supported

This plugin is also support Black Copper MINI Thermal Printer BC-P58B.
Thank you giorgiofellipe

Thank you.


Releases contain the .github directory and .gitignore file

There's no reason to ship those in the Release zips.

Resolved for current (1.7) release, will ensure future releases are cleaned of these files.


RectangularElevationBehavior - Deprecated since version 1.1.0.

You need to update the repository.
The current version of kiwi is 1.1.1
class kivymd.uix.behaviors.elevation.RectangularElevationBehavior(**kwargs)
Deprecated since version 1.1.0.
Use CommonElevationBehavior class instead.

@psiu7217 Yeah I tried it recently after many days and realised the same.

Updated it now.


Fix healthcheck for the example-backend

Hey, I just made a Pull Request!

I was adding a healthcheck to @thefrontside's instance of Backstage and noticed the example provided here returns a false positive because of its 404 page:

If you create a new backstage app, build the image without actually adding a health check, and deploy it to a local cluster with the startup probe configured:
...
spec:
  containers:
    - name: backstage
      image: backstage:x
      imagePullPolicy: Never
      startupProbe:
        httpGet:
          path: /healthcheck
          port: http

You should be able to see that the probe succeeds despite there not being a /healthcheck route. Then if you update the deployment so that the probe's path is /api/whatever, the probe will then correctly fail.
Approach

I replaced the current healthcheck route to use the apiRouter instead
Changed the default path of createStatusCheckRouter() from /healthcheck to / (otherwise the default path would end up being /api/healthcheck/healthcheck)
Updated the documentation in Deploying with Kubernetes and Observability

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Hm wait, are you saying that DESPITE the health check plugin being in place, you can get a false positive? Or does this problem only happen if you explicitly do not have the health check plugin added in your backend at all?

Looking forward a bit I think we'll encourage some of these internal endpoints to be exposed through separate listeners, so I actually think it makes sense to keep it at the root for now tbh. Health checks doesn't quite feel like /api level functionality.
I'm struggling a bit to understand the actual issue here tbh, is the confusion arising from setting up health checks towards an instance that does not have health checks installed, or something else?
As a side note I'd always encourage health check clients to check more than just the status code, preferably even that it's hitting the expected service.

I'm feeling that the documentation is quite clear that you need to add the health check yourself. I guess something we could consider here is whether we should add the health check to the @backstage/create-app template?
Just to clarify as well, what you end up getting without a health check endpoint installed is index.html, due to the HTML5 mode routing.
I'm still not entirely clear on the issue here though tbh, because the default health check behaves pretty much exactly like index.html anyway? Was the issue that you configured a custom health check but you had forgot to register the health check router in the top-level router?

I'm still not entirely clear on the issue here though tbh, because the default health check behaves pretty much exactly like index.html anyway? Was the issue that you configured a custom health check but you had forgot to register the health check router in the top-level router?

My concern is that the default behaviour of a probe automatically succeeds with or without a health check route. Whether or not someone implements their own health check logic, shouldn't a health probe fail unless the status is explicitly successful?
So instead of:

Deploy Backstage and assume a "successful deployment" is fine unless if and when there's a health check that tells us otherwise

It should be:

Deploy Backstage only when a healthcheck is successful; otherwise assume something is wrong

That way if probes are configured on a deployment and someone does forget to register their healthcheck plugin, they won't mistake the default success as their own health check working properly.

I made diagrams.
Currently:

What I'm proposing:


squid: 3.5.15 -> 3.5.17 (resolving CVE-2016-3947, CVE-2016-3948, CVE-2016-4051, CVE-2016-4052, CVE-2016-4053, CVE-2016-4054)

Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] OS X
[x] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

LGTM


Status of recordings

Would it be possible to add a field to the kapacitor list recordings results that indicates whether a recording has completed? Recordings can take a long time to create and it would be nice to know which ones have finished so they can be replayed. I know that I can do an HTTP GET for /task?id=81a...323 and wait for it to return, but that's not really a good option. This is especially true in cases where I'd like to report recording results on a webpage immediately while many recordings are being created simultaneously.
If I can add an "I wish" feature request, too, it would be cool if that completion status also indicated how much of the recording has been processed. It may not be perfect, but just a simple percentage of the time duration has been covered so far would suffice.

I think something like this would be possible, and would also play nice with #283

This was fixed as part of the API rewrite.


Create/Upload fallback resources for missing fields such as no photo or location data.

Handle unexpected auth return data.

Commit 7465039 fixes.


Error while trying to run the project

Analyzer Feedback from IntelliJ
Version information

IDEA IC-172.3968.16
1.21.0-dev.10.0
IC-172.3968.16, JRE 1.8.0_152-release-915-b11x64 JetBrains s.r.o, OS Mac OS X(x86_64) v10.12.6 unknown, screens 1440x900  Retina

Exception
Please append the contents of:
file:///private/var/folders/z0/c42m1s012k5_yr_9v22q1trw0000gn/T/report.txt
Dart analysis server, SDK version 1.21.0-dev.10.0, server version 1.17.0, FATAL error: Failed to handle request: {clientRequestTime:<PHONE_NUMBER>574, params: {file: /Users/shajeelafzal/Documents/flutter_workspace/cricket_app_flutter/lib/main.dart, offset: 47}, method: edit.getFixes, id: 46}
NoSuchMethodError: The method 'getLocation' was called on null.
Receiver: null
Tried calling: getLocation(47)
#0      Object._noSuchMethod (dart:core-patch/object_patch.dart:44)
#1      Object.noSuchMethod (dart:core-patch/object_patch.dart:47)
#2      EditDomainHandler.getFixes.<getFixes_async_body> (package:analysis_server/src/edit/edit_domain.dart:208)
#3      _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:27)
#4      _rootRunUnary (dart:async/zone.dart:1158)
#5      _CustomZone.runUnary (dart:async/zone.dart:1037)
#6      _FutureListener.handleValue (dart:async/future_impl.dart:131)
#7      _Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:637)
#8      _Future._propagateToListeners (dart:async/future_impl.dart:667)
#9      _Future._complete (dart:async/future_impl.dart:467)
#10     _SyncCompleter.complete (dart:async/future_impl.dart:52)
#11     AnalysisServer.getResolvedCompilationUnit.<getResolvedCompilationUnit_async_body> (package:analysis_server/src/analysis_server.dart:809)
#12     Future.Future.microtask.<anonymous closure> (dart:async/future.dart:144)
#13     _rootRun (dart:async/zone.dart:1146)

This has likely been fixed in the latest dev releases. Please upgrade to a newer version of the SDK and let us know if you're still seeing this issue. Thanks!


Add autoconfiguration for ReactivePulsarListener

Closed via https://github.com/spring-projects-experimental/spring-pulsar/commit/4944be6fecc2afc61b6150c6769821bef0717149
@cbornet there was a failing test that I adjusted in follow on commit https://github.com/spring-projects-experimental/spring-pulsar/commit/498728a06fb468f3d3fcbed72c854584e10cf71a . The issue was that the mocked AdaptedReactivePulsarClientFactory was not setup to return anything from .create(PulsarClient.class) and this was previously fine before some other bean needed the RPC.


hack: Add very basic debug info

This branch is for testing of lowering of DWARF info in hugr-llvm. This implementation is quite hacky; we should wait until #257 is done to properly implement this.
This branch adds debug info to the following nodes:

Module (requires passing an explict file parameter when constructing a GuppyModule)
FuncDefn
Most ExtensionOps

Codecov Report
Attention: Patch coverage is 95.23810% with 1 line in your changes missing coverage. Please review.

Project coverage is 92.59%. Comparing base (6698b75) to head (a023228).
Report is 5 commits behind head on main.

Files
Patch %
Lines

guppylang/hugr_builder/hugr.py
94.11%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #336      +/-   ##
==========================================
- Coverage   92.67%   92.59%   -0.08%
==========================================
  Files          45       45
  Lines        5105     5148      +43
==========================================
+ Hits         4731     4767      +36
- Misses        374      381       +7

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


PDFJS.webViewerLoad is not a function

I have include PDF.js and also set the url of pdf.worker.js but still i am getting this error in console. What i am doing wrong ?

@jasny @KillaBee1985 @pranjal-goswami @subdee @sven4ask is anybody here to address this issue ?

This ticket is a duplicate of https://github.com/legalthings/angular-pdfjs-viewer/issues/38


[YUNIKORN-2418] Improve clarity in Features docs

What is this PR for?
This PR updates some wording in the features documentation to improve clarity and understanding.
What type of PR is it?

[ ] - Bug Fix
[ ] - Improvement
[ ] - Feature
[x] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

Todos

[x] - Add JIRA issue once ASF self service has created me an account

What is the Jira issue?

Open an issue on Jira https://issues.apache.org/jira/browse/YUNIKORN/
Put link here, and add [YUNIKORN-Jira number] in PR title, eg. [YUNIKORN-2] Gang scheduling interface parameters

How should this be tested?
n/a
Screenshots (if appropriate)
n/a
Questions:

[ ] - The licenses files need update.
[ ] - There is breaking changes for older versions.
[ ] - It needs documentation.

Please open a jra for tracking the changes, see https://yunikorn.apache.org/community/how_to_contribute for the process

Thanks, updated.

@alex-stiff thanks for the PR, I've always believed we need something like that.
Which part of the documentation have you read already?

@alex-stiff thanks for the PR, I've always believed we need something like that.
Which part of the documentation have you read already?

Mostly this page - as I go through the rest of it I'll be taking a look at the other docs pages as well.


Change column names in StatsBomb data_type in plot_pass()

Every other pitch plot function accepts column names as (x, y, finalX, finalY) for even statsbomb datasets, but plot_pass accepts the standard statsbomb names (location.x, location.y). This might be confusing so should be changed

This isnt a change for the purpose of preference, Its an error in the code from the commit that added the new version of the function to the package

Had to shift the theme colors to the beginning of the function as they weren't being registered for the Opta plot type

Had to shift the theme colors to the beginning of the function as they weren't being registered for the Opta plot type

ooh that's why it was on top, that's my bad -- i didn't check .
and well, our tests also don't cover appearance, which is something we can do with https://github.com/r-lib/vdiffr but we can talk about that later

Its a good thing we're expanding tests. Wouldn't have found an issue if I wasn't trying to include the opta data_type within the tests


adding time schema validation to other interface method schemas

Addresses #324 by updating interface-method schemas that have timestamps without the date-time schema reference
After digging into this, it appears a lot of progress has already been made on timestamp validation in #147

Codecov Report

Merging #580 (5360ebf) into main (bea7bdf) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #580   +/-   ##
=======================================
  Coverage   97.68%   97.68%
=======================================
  Files          66       66
  Lines        7932     7932
  Branches     1150     1150
=======================================
  Hits         7748     7748
  Misses        178      178
  Partials        6        6


Propose Logo

Hi, @BharathVishal. I wanted to contribute to your project and I've designed a logo for your project. if you need anything change please let me know.

Best Regard, Mirza Zulfan

Hi @mirzazulfan,
The first logo is good. If would be good if you could remove the gradient effect in the battery icon and make it flat. Thanks.

Thanks for your response @BharathVishal . Im glad you liked it. Do you mean like this ?

I'll sent all file and create a pr. What do you think?
Thank you. Best Regard.

Hi mirzazulfan,
The second logo(the having the red background) would be apt if you could change the background color of it to #2ecc71(emerald green), signifying a battery's color. Thanks.

@BharathVishal .. Like this ?

Thanks.

Exactly, it's apt. Thanks for the logo. Feel free to add the logo and create a pull request.

I'll upload the files and create a pr. Thank you.

Thanks @mirzazulfan ! :) 👍

Hi @BharathVishal ,, i've created a pr #2 . Good luck with your project.
Best Regard. :+1:


Type conflict with NServiceBus.Order

Much better thanks, otherwise one needs to deal with using statements and weird compilation errors.


Flight State Machine

Flight State Machine
Problem
Need ability to determine what state of the competition the UAS is in.
Solution
Implement a state machine to track and switch between the states of the competition.

Current implementation of Argonia code does not require a State Machine


Page scrolls down on drag & drop

Tested in Chrome 39 and Safari 7.1.2

I prevented selectize in drag_drop plugin definition from scrolling down by adding a scroll lock found on http://stackoverflow.com/a/3656618/3250797


build dev failed: Cannot read property 'indexOf' of undefined

From @stefdelec on December 18, 2016 19:11
Impossible to "serve".
I got this error only on brand new project but not older ones.
build dev failed: Cannot read property 'indexOf' of undefined
Ionic version:
Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.4
Ionic CLI Version: 2.1.17
Ionic App Lib Version: 2.1.7
Ionic App Scripts Version: 0.0.47
ios-deploy version: 1.8.6
ios-sim version: 5.0.6
OS: OS X El Capitan
Node Version: v6.9.2
Xcode version: Xcode 8.1 Build version 8B62
npm 4.0.5
[ ] bug report
$ ionic serve

> ionic-hello-world@ ionic:serve /Users/Stephane/Downloads/MyProject
> ionic-app-scripts serve

[20:09:16]  ionic-app-scripts 0.0.47
[20:09:16]  watch started ...
[20:09:16]  build dev started ...
[20:09:16]  clean started ...
[20:09:16]  clean finished in 4 ms
[20:09:16]  copy started ...
[20:09:16]  transpile started ...
[20:09:20]  build dev failed: Cannot read property 'indexOf' of undefined
[20:09:20]  dev server running: http://localhost:8100/

[20:09:20]  copy finished in 4.32 s
[20:09:20]  watch ready in 4.34 s

Debug file:
0 info it worked if it ends with ok
1 verbose cli [ '/usr/local/bin/node',
1 verbose cli   '/usr/local/bin/npm',
1 verbose cli   'run',
1 verbose cli   'ionic:build',
1 verbose cli   '--' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'preionic:build', 'ionic:build', 'postionic:build' ]
5 info lifecycle ionic-hello-world@~preionic:build: ionic-hello-world@
6 silly lifecycle ionic-hello-world@~preionic:build: no script for preionic:build, continuing
7 info lifecycle ionic-hello-world@~ionic:build: ionic-hello-world@
8 verbose lifecycle ionic-hello-world@~ionic:build: unsafe-perm in lifecycle true
9 verbose lifecycle ionic-hello-world@~ionic:build: PATH: /usr/local/lib/node_modules/npm/bin/node-gyp-bin:/Users/Stephane/Downloads/MyProject/node_modules/.bin:/Library/Frameworks/Python.framework/Versions/3.5/bin:/Library/Frameworks/Python.framework/Versions/3.5/bin:/Library/Frameworks/Python.framework/Versions/2.7/bin:/Library/Frameworks/Python.framework/Versions/3.4/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin:/usr/local/share/dotnet:/usr/local/git/bin:/Library/Frameworks/Mono.framework/Versions/Current/Commands
10 verbose lifecycle ionic-hello-world@~ionic:build: CWD: /Users/Stephane/Downloads/MyProject
11 silly lifecycle ionic-hello-world@~ionic:build: Args: [ '-c', 'ionic-app-scripts build' ]
12 silly lifecycle ionic-hello-world@~ionic:build: Returned: code: 1  signal: null
13 info lifecycle ionic-hello-world@~ionic:build: Failed to exec ionic:build script
14 verbose stack Error: ionic-hello-world@ ionic:build: `ionic-app-scripts build`
14 verbose stack Exit status 1
14 verbose stack     at EventEmitter.<anonymous> (/usr/local/lib/node_modules/npm/lib/utils/lifecycle.js:255:16)
14 verbose stack     at emitTwo (events.js:106:13)
14 verbose stack     at EventEmitter.emit (events.js:191:7)
14 verbose stack     at ChildProcess.<anonymous> (/usr/local/lib/node_modules/npm/lib/utils/spawn.js:40:14)
14 verbose stack     at emitTwo (events.js:106:13)
14 verbose stack     at ChildProcess.emit (events.js:191:7)
14 verbose stack     at maybeClose (internal/child_process.js:877:16)
14 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:226:5)
15 verbose pkgid ionic-hello-world@
16 verbose cwd /Users/Stephane/Downloads/MyProject
17 error Darwin 15.6.0
18 error argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "ionic:build" "--"
19 error node v6.9.2
20 error npm  v3.10.9
21 error code ELIFECYCLE
22 error ionic-hello-world@ ionic:build: `ionic-app-scripts build`
22 error Exit status 1
23 error Failed at the ionic-hello-world@ ionic:build script 'ionic-app-scripts build'.
23 error Make sure you have the latest version of node.js and npm installed.
23 error If you do, this is most likely a problem with the ionic-hello-world package,
23 error not with npm itself.
23 error Tell the author that this fails on your system:
23 error     ionic-app-scripts build
23 error You can get information on how to open an issue for this project with:
23 error     npm bugs ionic-hello-world
23 error Or if that isn't available, you can get their info via:
23 error     npm owner ls ionic-hello-world
23 error There is likely additional logging output above.
24 verbose exit [ 1, true ]

Copied from original issue: driftyco/ionic#9692

From @jayeshanandani on December 21, 2016 2:27
You can try by npm install @ionic/app-scripts@latest --save-dev

From @stefdelec on December 19, 2016 12:14
I am not using anything.
I creating a new project and then serve.
ionic start myproject tabs --v2 --ts
...
cd myproject
ionic serve

From @lufias on December 19, 2016 2:20
Are you using ion-img? Using debugger, my indexOf error was thrown trying to load an ion-img src using a promise..
https://github.com/driftyco/ionic/issues/9691

From @jayeshanandani on December 20, 2016 18:7
Please try by updating ionic-app-scripts to 0.0.48 and ensure you have deleted node_modules folder and reinstalled again for safety check.

From @stefdelec on December 20, 2016 20:25
how to update  ionic-app-scripts ?
npm  ionic-app-scripts --update ?

I got this error:
"npm WARN<EMAIL_ADDRESS>requires a peer of @angular/core@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of @angular/compiler@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of @angular/compiler-cli@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of @angular/tsc-wrapped@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of rxjs@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of typescript@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of zone.js@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed."

So I did:
npm install -g typescript
and again
npm install @ionic/app-scripts@latest --save-dev
Still having the issue...

@stefdelec : what typescript version are you using?

tsc -v
Version 2.1.4

@stefdelec : can you please try with typescript version 2.0.9 as I have seen quite a few issues with 2.1.x in angular and other places round.

macbook:~ Stephane$ npm install<EMAIL_ADDRESS>/Users/Stephane
+-- UNMET PEER DEPENDENCY @angular/compiler@*
+-- UNMET PEER DEPENDENCY @angular/compiler-cli@*
+-- UNMET PEER DEPENDENCY @angular/core@*
+-- UNMET PEER DEPENDENCY @angular/tsc-wrapped@*
+-- UNMET PEER DEPENDENCY browserify@>= 6.x
+-- UNMET PEER DEPENDENCY rxjs@*
+--<EMAIL_ADDRESS>`-- UNMET PEER DEPENDENCY zone.js@*

npm WARN enoent ENOENT: no such file or directory, open '/Users/Stephane/package.json'
npm WARN<EMAIL_ADDRESS>requires a peer of @angular/core@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of @angular/compiler@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of @angular/compiler-cli@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of @angular/tsc-wrapped@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of rxjs@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of zone.js@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of browserify@>= 6.x but none was installed.
npm WARN Stephane No description
npm WARN Stephane No repository field.
npm WARN Stephane No README data
npm WARN Stephane No license field.

macbook-air-de-stephane:~ Stephane$ npm install<EMAIL_ADDRESS>-g
/usr/local/bin/tsc -> /usr/local/lib/node_modules/typescript/bin/tsc
/usr/local/bin/tsserver -> /usr/local/lib/node_modules/typescript/bin/tsserver
/usr/local/lib
`--<EMAIL_ADDRESS>
macbook-air-de-stephane:~ Stephane$ npm install @ionic/app-scripts@latest --save-dev -g
/usr/local/bin/ionic-app-scripts -><EMAIL_ADDRESS>/usr/local/lib
+-- UNMET PEER DEPENDENCY @angular/compiler@*
+-- UNMET PEER DEPENDENCY @angular/compiler-cli@*
+-- UNMET PEER DEPENDENCY @angular/core@*
+-- UNMET PEER DEPENDENCY @angular/tsc-wrapped@*
+--<EMAIL_ADDRESS>| `-- UNMET PEER DEPENDENCY<EMAIL_ADDRESS>+-- UNMET PEER DEPENDENCY rxjs@*
+-- UNMET PEER DEPENDENCY typescript@*
`-- UNMET PEER DEPENDENCY zone.js@*

npm WARN<EMAIL_ADDRESS>requires a peer of @angular/core@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of @angular/compiler@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of @angular/compiler-cli@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of @angular/tsc-wrapped@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of rxjs@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of typescript@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of zone.js@* but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

@stefdelec : It seems you dont' have any angular dependencies installed. Please share you package.json file.

It is a brand new projet from
ionic start myproject tabs --v2 --ts

@stefdelec : That's fine but from your output it seems npm is not installing all dependencies needed and hence the error.

@stefdelec: Please update ionic-app-scripts to 0.0.48 in package.json. Delete npm_modules, .tmp folder if any and run the application again. Ensure you don't have any errors while npm install command.

Almost there : new error
`MacBook-Air-de-Stephane:myproject Stephane$ ionic serve

ionic-hello-world@ ionic:serve /Users/Stephane/Documents/APPs/myproject
ionic-app-scripts serve

keywords if/then/else require v5 option
[17:27:58]  ionic-app-scripts 0.0.48
[17:27:58]  watch started ...
[17:27:58]  build dev started ...
[17:27:58]  clean started ...
[17:27:58]  clean finished in 4 ms
[17:27:58]  copy started ...
[17:27:58]  transpile started ...
[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.collection.d.ts, line: 24
All declarations of 'size' must have identical modifiers.
  L25:  interface WeakMapConstructor {

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.collection.d.ts, line: 30
All declarations of 'prototype' must have identical modifiers.
  L30:  declare var WeakMap: WeakMapConstructor;

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.collection.d.ts, line: 44
        All declarations of 'prototype' must have identical modifiers.

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.collection.d.ts, line: 54
L43:      new (values?: T[]): Set;
L44:      readonly prototype: Set;
        All declarations of 'size' must have identical modifiers.

  L54:  interface WeakSetConstructor {
  L55:      new (): WeakSet<any>;

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.collection.d.ts, line: 60
All declarations of 'prototype' must have identical modifiers.
  L59:  declare var WeakSet: WeakSetConstructor;

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.collection.d.ts, line: 73
All declarations of 'prototype' must have identical modifiers.
[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.core.d.ts, line: 17
        Duplicate identifier 'PropertyKey'.

  L16:  * Returns the index of the first element in the array where predicate is true, and undefined
  L17:  * otherwise.
  L18:  * @param predicate find calls predicate once for each element of the array, in ascending

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.core.d.ts, line: 95
All declarations of 'name' must have identical modifiers.
  L94:      imul(x: number, y: number): number;

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.core.d.ts, line: 216
[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.core.d.ts, line: 251
All declarations of 'EPSILON' must have identical modifiers.
 L217:        * Returns a Boolean value that indicates whether a value is the reserved value NaN (not a

        All declarations of 'MAX_SAFE_INTEGER' must have identical modifiers.

 L251:    * Converts A string to an integer.
 L252:    * @param s A string to convert into a number.

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.core.d.ts, line: 258
All declarations of 'MIN_SAFE_INTEGER' must have identical modifiers.
 L257:      parseInt(string: string, radix?: number): number;

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.core.d.ts, line: 403
All declarations of 'flags' must have identical modifiers.
 L402:  interface RegExpConstructor {
 L403:      new (pattern: RegExp, flags?: string): RegExp;

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.iterable.d.ts, line: 24
L404:      (pattern: RegExp, flags?: string): RegExp;
        All declarations of 'iterator' must have identical modifiers.

  L24:  interface IterableIterator<T> extends Iterator<T> {
  L25:      [Symbol.iterator](): IterableIterator<T>;

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.iterable.d.ts, line: 29
All declarations of 'value' must have identical modifiers.
  L28:  interface Array<T> {

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.promise.d.ts, line: 69
L29:      /** Iterator */
L30:      Symbol.iterator: IterableIterator;
        All declarations of 'prototype' must have identical modifiers.

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.symbol.d.ts, line: 29
L69:   * Creates a Promise that is resolved with an array of results when all of the provided Promises
L70:   * resolve, or rejected when any Promise is rejected.
        All declarations of 'prototype' must have identical modifiers.

  L29:    * Returns a key from the global symbol registry matching the given Symbol if found.
  L30:    * Otherwise, returns a undefined.

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.symbol.wellknown.d.ts, line: 24
All declarations of 'hasInstance' must have identical modifiers.
  L24:      readonly replace: symbol;

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.symbol.wellknown.d.ts, line: 30
        All declarations of 'isConcatSpreadable' must have identical modifiers.

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.symbol.wellknown.d.ts, line: 36
L30:      readonly search: symbol;
        All declarations of 'match' must have identical modifiers.

  L36:      readonly species: symbol;

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.symbol.wellknown.d.ts, line: 42
        All declarations of 'replace' must have identical modifiers.

  L42:      readonly split: symbol;

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.symbol.wellknown.d.ts, line: 48
All declarations of 'search' must have identical modifiers.
  L48:      readonly toPrimitive: symbol;

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.symbol.wellknown.d.ts, line: 54
All declarations of 'species' must have identical modifiers.
  L54:      readonly toStringTag: symbol;

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.symbol.wellknown.d.ts, line: 60
All declarations of 'split' must have identical modifiers.
  L60:      readonly unscopables: symbol;

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.symbol.wellknown.d.ts, line: 66
        All declarations of 'toPrimitive' must have identical modifiers.

  L67:  interface Array<T> {

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.symbol.wellknown.d.ts, line: 72
All declarations of 'toStringTag' must have identical modifiers.
  L72:  [Symbol.unscopables](): {
  L73:      copyWithin: boolean;

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.symbol.wellknown.d.ts, line: 78
All declarations of 'unscopables' must have identical modifiers.
  L77:  findIndex: boolean;
  L78:  keys: boolean;
  L79:  values: boolean;

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.symbol.wellknown.d.ts, line: 82
        All declarations of '[Symbol.toStringTag]' must have identical modifiers.

  L83:  interface Date {

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.symbol.wellknown.d.ts, line: 142
All declarations of '[Symbol.toStringTag]' must have identical modifiers.
            rface Math {
 L143:      readonly [Symbol.toStringTag]: "Math";

[17:28:04]  typescript: node_modules/typescript/lib/lib.es2015.symbol.wellknown.d.ts, line: 161
All declarations of '[Symbol.toStringTag]' must have identical modifiers.
  Returns the target object.
  L21:  * @param target The target object to copy to.
  L22:  * @param source1 The first source object from which to copy properties.

        All declarations of 'name' must have identical modifiers.

  L84:  * otherwise.
  L85:  * @param predicate find calls predicate once for each element of the array, in ascending
  L86:  * order, until it finds one where predicate returns true. If such an element is found, find

        Subsequent variable declarations must have the same type. Variable '[Symbol.unscopables]' must be of type '{
        copyWithin: boolean; entries: boolean; fill: boolean; find: boolean; findIndex: boolean; keys: ...', but
        here has type 'any'.

 L146:        * Creates an array from an array-like object.

        All declarations of 'flags' must have identical modifiers.

 L261:  * number. Only finite values of the type number, result in true.
 L262:  * @param number A numeric value.

        All declarations of 'EPSILON' must have identical modifiers.

 L275:  * to a number. Only values of the type number, that are also NaN, result in true.
 L276:  * @param number A numeric value.

        All declarations of 'MAX_SAFE_INTEGER' must have identical modifiers.

 L310:  * If this argument is not supplied, strings with a prefix of '0x' are considered hexadecimal.
 L311:  * All other strings are considered decimal.

        All declarations of 'MIN_SAFE_INTEGER' must have identical modifiers.

 L318:    * Returns the number of leading zero bits in the 32-bit binary representation of a number.
 L319:    * @param x A numeric expression.

        Subsequent variable declarations must have the same type. Variable '[Symbol.toStringTag]' must be of type
        '"Symbol"', but here has type 'string'.

 L457:      for(key: string): symbol;

        All declarations of '[Symbol.toStringTag]' must have identical modifiers.

 L457:      for(key: string): symbol;

        All declarations of 'prototype' must have identical modifiers.

 L464:      keyFor(sym: symbol): string;

        All declarations of 'hasInstance' must have identical modifiers.

 L493:        * A regular expression method that replaces matched substrings of a string. Called by the

        All declarations of 'isConcatSpreadable' must have identical modifiers.

 L499:        * A regular expression method that returns the index within a string that matches the

        All declarations of 'iterator' must have identical modifiers.

 L505:        * A function valued property that is the constructor function that is used to create

        All declarations of 'match' must have identical modifiers.

 L511:        * A regular expression method that splits a string at the indices that match the regular

        All declarations of 'replace' must have identical modifiers.

 L517:        * A method that converts an object to a corresponding primitive value.Called by the ToPrimitive

        All declarations of 'search' must have identical modifiers.

 L523:        * A String value that is used in the creation of the default string description of an object.

        All declarations of 'species' must have identical modifiers.

 L529:        * An Object whose own property names are property names that are excluded from the with

        All declarations of 'split' must have identical modifiers.

 L535:        * Non-standard. Use simple mode for core-js symbols. See https://github.com/zloirock/core-js/#caveats-when-using-symbol-polyfill

        All declarations of 'toPrimitive' must have identical modifiers.

 L540:    * Non-standard. Use setter mode for core-js symbols. See https://github.com/zloirock/core-js/#caveats-when

        All declarations of 'toStringTag' must have identical modifiers.

 L545:  declare var Symbol: SymbolConstructor;
 L547:  interface Object {

        All declarations of 'unscopables' must have identical modifiers.

 L552:      hasOwnProperty(v: PropertyKey): boolean;

        Subsequent variable declarations must have the same type. Variable '[Symbol.toStringTag]' must be of type
        '"Math"', but here has type 'string'.

 L608:      size: number;

        All declarations of '[Symbol.toStringTag]' must have identical modifiers.

 L608:      size: number;

        Subsequent variable declarations must have the same type. Variable '[Symbol.toStringTag]' must be of type
        '"JSON"', but here has type 'string'.

 L612:  [17:28:04]  typescript<EMAIL_ADDRESS>line: 680

22mnew <K, V>(): Map<K, V>;
L613:  new <K, V>(iterable: Iterable<[K, V]>): Map<K, V>;
L614:  prototype: Map<any, any>;
        All declarations of '[Symbol.toStringTag]' must have identical modifiers.

 L612:  new <K, V>(): Map<K, V>;
 L613:  new <K, V>(iterable: Iterable<[K, V]>): Map<K, V>;
 L614:  prototype: Map<any, any>;

        All declarations of 'size' must have identical modifiers.

 L628:  interface SetConstructor {
 L629:      new <T>(): Set<T>;

        All declarations of 'prototype' must have identical modifiers.

 L634:  declare var Set: SetConstructor;

        All declarations of 'size' must have identical modifiers.

 L644:  new <K, V>(): WeakMap<K, V>;
 L645:  new <K, V>(iterable: Iterable<[K, V]>): WeakMap<K, V>;
 L646:  prototype: WeakMap<any, any>;

        All declarations of 'prototype' must have identical modifiers.

 L651:  interface WeakSet<T> {
 L652:      add(value: T): WeakSet<T>;

        All declarations of 'prototype' must have identical modifiers.

 L665:  // #############################################################################################
 L666:  // ECMAScript 6: Iterators

[17:28:04]  typescript<EMAIL_ADDRESS>line: 692
[17:28:04]  typescript<EMAIL_ADDRESS>line: 804
L667:  // Modules: es6.string.iterator, es6.array.iterator, es6.map, es6.set, web.dom.iterable
        All declarations of 'prototype' must have identical modifiers.

 L681:  interface Iterable<T> {

        All declarations of 'value' must have identical modifiers.

 L691:      [Symbol.iterator](): IterableIterator<string>;

        All declarations of 'prototype' must have identical modifiers.

 L803:  all<T1, T2, T3, T4, T5, T6, T7>(values: [T1 | PromiseLike<T1>, T2 | PromiseLike<T2>, T3 | PromiseLike<T3>, T4 | PromiseLike <T4>, T5 | PromiseLike<T5>, T6 | PromiseLike<T6>, T7 | PromiseLike<T7>]): Promise<[T1, T2, T3, T4, T5, T6, T7]>;
 L804:  all<T1, T2, T3, T4, T5, T6>(values: [T1 | PromiseLike<T1>, T2 | PromiseLike<T2>, T3 | PromiseLike<T3>, T4 |
 L805:  all<T1, T2, T3, T4, T5>(values: [T1 | PromiseLike<T1>, T2 | PromiseLike<T2>, T3 | PromiseLike<T3>, T4 | PromiseLike <T4>, T5 | PromiseLike<T5>]): Promise<[T1, T2, T3, T4, T5]>;

[17:28:04]  transpile failed
[17:28:04]  dev server running: http://localhost:8100/
[17:28:04]  copy finished in 5.67 s
[17:28:04]  watch ready in 5.69 s
^[^C
MacBook-Air-de-Stephane:myproject Stephane$`

I believe this issue is resolved now.
Thanks,
Dan

Nope. It is not I am still running into the issue even after updating package.json and ionic-script 0.0.48

@stefdelec,
Did you modify the lib option at all?
It should look like this:
https://github.com/driftyco/ionic2-app-base/blob/master/tsconfig.json#L7-L10
Thanks,
Dan

Nop I havn't modified anything anywhere. But if I put the provided tsconfig in the link, doesn't work. If I remove
"    "lib": [
"dom",
"es2015"
],"
It is transpiling. And Ionic is serving.

@stefdelec I got the same error, tried few hours to figure it out, it's related to lib option . your solution work perfectly

Ahh, It could be because @types/core-js which is likely providing duplicate definitions for ES2015. I recommend removing those Types. You shouldn't need them as you should never be interacting directly with the polyfills.
Thanks,
Dan


assignment expression does not return value.

assignment expression should return value (ex. 5 for "a=5") according to ECMA-262 spec.
In rapidus, example codes (listed below) are correctly parsed, but go panic while bytecode execution.
no value seems to push to VM.state.stack in VM.set_name() function.
ex)
console.log(a=5)
a=b=5

Hmm, It seems I completely don't know JS.
Appreciate your reporting.

Fixed.

works fine!
thank you.


"Plugin's runtime wasn't added" in Webpack 2

<EMAIL_ADDRESS>is working for me with<EMAIL_ADDRESS>but when I upgrade to any of the newer Webpack versions I get this error:
Error: offline-plugin: Plugin's runtime wasn't added to one of your bundle entries. See this https://goo.gl/YwewYp for details.
    at Compiler.<anonymous> (/Users/nickstevens/Repos/surge-ui/node_modules/offline-plugin/lib/index.js:279:20)
    at next (/Users/nickstevens/Repos/surge-ui/node_modules/webpack/node_modules/tapable/lib/Tapable.js:104:14)
    at callback (/Users/nickstevens/Repos/surge-ui/node_modules/copy-webpack-plugin/dist/index.js:71:17)
    at Object.finallyHandler (/Users/nickstevens/Repos/surge-ui/node_modules/copy-webpack-plugin/node_modules/bluebird/js/main/finally.js:39:23)
    at Object.tryCatcher (/Users/nickstevens/Repos/surge-ui/node_modules/copy-webpack-plugin/node_modules/bluebird/js/main/util.js:26:23)
    at Promise._settlePromiseFromHandler (/Users/nickstevens/Repos/surge-ui/node_modules/copy-webpack-plugin/node_modules/bluebird/js/main/promise.js:510:31)
    at Promise._settlePromiseAt (/Users/nickstevens/Repos/surge-ui/node_modules/copy-webpack-plugin/node_modules/bluebird/js/main/promise.js:584:18)
    at Promise._settlePromises (/Users/nickstevens/Repos/surge-ui/node_modules/copy-webpack-plugin/node_modules/bluebird/js/main/promise.js:700:14)
    at Async._drainQueue (/Users/nickstevens/Repos/surge-ui/node_modules/copy-webpack-plugin/node_modules/bluebird/js/main/async.js:123:16)
    at Async._drainQueues (/Users/nickstevens/Repos/surge-ui/node_modules/copy-webpack-plugin/node_modules/bluebird/js/main/async.js:133:10)
    at Immediate.Async.drainQueues [as _onImmediate] (/Users/nickstevens/Repos/surge-ui/node_modules/copy-webpack-plugin/node_modules/bluebird/js/main/async.js:15:14)
    at tryOnImmediate (timers.js:543:15)
    at processImmediate [as _immediateCallback] (timers.js:523:5)

The runtime is included like this in my app's index file:
import OfflinePlugin from 'offline-plugin/runtime'
OfflinePlugin.install()

My webpack.config.js looks like this:
const { resolve } = require('path')
const webpack = require('webpack')
const ExtractTextPlugin = require('extract-text-webpack-plugin')
const HtmlWebpackPlugin = require('html-webpack-plugin')
const BrowserSyncPlugin = require('browser-sync-webpack-plugin')
const CopyWebpackPlugin = require('copy-webpack-plugin')
const Dotenv = require('dotenv-webpack')
const OfflinePlugin = require('offline-plugin')

require('dotenv').config({
  path: process.env.CONFIG
})

const HOST = process.env.HOST || 'localhost'
const PORT = process.env.PORT || 8080
const PROXY = `http://${HOST}:${PORT}`

const removeEmpty = array => array.filter(i => !!i)

module.exports = () => {
  return {
    entry: {
      app: process.env.NODE_ENV === 'dev' ?
        ['./index.js', 'webpack/hot/only-dev-server', 'webpack-dev-server/client?http://localhost:8080']
        : './index.js',
      vendor: ['react', 'redux', 'react-redux', 'axios', 'react-router', 'react-router-redux', 'immutable', 'react-dom']
    },
    output: {
      filename: process.env.NODE_ENV === 'dev' ? 'bundle.[name].js' : 'bundle.[name].[hash].js',
      path: resolve(__dirname, 'dist'),
      pathinfo: process.env.NODE_ENV === 'dev',
      publicPath: process.env.BASE_URL ? process.env.BASE_URL : '/'
    },
    resolve: {
      extensions: ['.jsx', '.scss', '.js', '.json']
    },
    context: resolve(__dirname, 'src'),
    devtool: process.env.NODE_ENV === 'dev' ? 'eval' : 'source-map',
    devServer: {
      historyApiFallback: true
    },
    bail: process.env.NODE_ENV !== 'dev',
    module: {
      loaders: [
        {
          test: /\.js$/,
          loader: 'react-hot!babel-loader!eslint',
          exclude: resolve(__dirname, "node_modules")
        },
        {
          test: /\.jsx$/,
          loader: 'react-hot!babel-loader?presets[]=react,presets[]=es2015-webpack,presets[]=stage-2,plugins[]=transform-class-properties!eslint',
          exclude: resolve(__dirname, "node_modules")
        },
        {
          test: /\.scss$/,
          loader: ExtractTextPlugin.extract('css-loader?sourceMap&modules&importLoaders=1&localIdentName=[name]__[local]___[hash:base64:5]!postcss-loader!sass-loader')
        },
        {
          test: /\.css$/,
          loader: ExtractTextPlugin.extract('css-loader')
        },
        {
          test: /\.icons\.(js|json)$/,
          loader: 'style-loader!css-loader!fontgen-loader'
        },
        {
          test: /\.json$/,
          loader: 'json-loader'
        },
        {
          test: /\.(jpe?g|png|gif|svg)$/i,
          loader: 'url?limit=10000!img?progressive=true'
        },
      ]
    },
    plugins: removeEmpty([
      new webpack.DefinePlugin({
        'process.env': {
          'NODE_ENV': JSON.stringify(process.env.NODE_ENV),
          'SURGE_HOME_URL': JSON.stringify(process.env.SURGE_HOME_URL),
          'SURGE_API_URL': JSON.stringify(process.env.SURGE_API_URL),
          'BASE_URL': JSON.stringify(process.env.BASE_URL)
        }
      }),
      new webpack.HotModuleReplacementPlugin(),
      new HtmlWebpackPlugin({
        template: './index.html'
      }),
      new BrowserSyncPlugin(
        {
          host: HOST,
          port: PORT,
          proxy: PROXY
        },
        {
          // prevent BrowserSync from reloading the page
          // and let Webpack Dev Server take care of this
          reload: false
        }
      ),
      new ExtractTextPlugin({
        filename: 'styles.[hash].css',
        allChunks: true
      }),
      new webpack.optimize.CommonsChunkPlugin({
        name: 'vendor',
      }),
      new CopyWebpackPlugin([
        {
          context: process.env.NODE_ENV === 'production' ?
            './browser/production/' :
            './browser/staging/',
          from: {
            glob: '**/*',
            dot: false
          }
        }
      ]),
      new OfflinePlugin({
        externals: ['https://fonts.googleapis.com/icon?family=Material+Icons'],
        ServiceWorker: {
          navigateFallbackURL: '/'
        }
      })
    ])
  }
}

Any ideas why this error is occurring even though I am definitely including the offline-plugin runtime? Thanks!

@2WheelCoder thanks for reporting this. Might be related to: https://github.com/NekR/offline-plugin/issues/148 somehow, but I'm not sure yet. I'll have to try with your config.

@2WheelCoder I don't see any problems with it in<EMAIL_ADDRESS>Can you try in it? Also, how do you run your webpack build?

@2WheelCoder Also, are you running it with dev-server? offline-plugin doesn't work in dev-server. At least it's not supported yet and it worked before that was coincidence.

@NekR
Getting the same error.
Seems that this error appears only when using import instead of System.import.
But this feature was introduced only in webpack beta28 https://github.com/webpack/webpack/releases/tag/v2.1.0-beta.28

@maslianok did you delete your comment here? Can you confirm that you are also using webpack-dev-server?

@NekR Sorry, I found that import doesn't work for me at all. So I think it's not a problem of offline-plugin.
No, I'm testing in production env.

@maslianok sure, no problem at all :-)

@2WheelCoder oh, I was able to repro this. Accidentally. I know what's going on and it's related to webpack 2 itself. In webpack 2, all loaders have to have full names, i.e. -loader isn't added automatically. In your project I found at least 2 such loaders where you don't add that prefix: url and eslint. If you will add it, then everything magically should start working.
That's so weird behavior and I'll try to resolve it on webpack side. Thanks for reporting this!

Okay, I made it a bit different type of errors so it now more obvious what happens and doesn't exit whole program (since doesn't shallow webpack's -loader error), but that still webpack's issue and I hope it will be resolve prior webpack 2 goes to stable.
Commit: https://github.com/NekR/offline-plugin/commit/b84f4f1808b6cdd8b56c9f1ac8bd9d53200ed689

Thanks @NekR! I thought I had updated the plugins that needed -loader appended, but it appears that all of them needed it and I had only updated a few. Everything seems to be working smoothly now with<EMAIL_ADDRESS>I am indeed using it with webpack-dev-server, but I hadn't had any issues before this. Seems to be working fine with webpack-dev-server now. This issue did not seem to be affected by it because I still received the error when running my staging and production builds without webpack-dev-server.


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/VSC-Service-Account/docs-archive-test-source.git

Docs Build status updates of commit c671539:
:clock10: Pending: waiting for processors (38 builds ahead of you)
:warning: Docs Build is busy, currently there are 38 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit c671539:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_master_2020-11-15-17-11-57 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


cc_proxy: Move to proxy v3.0

This patch updates the cc-proxy implementation of the proxy, relying
on latest proxy changes (version 3.0).

I have added do-not-merge label because we have to wait for https://github.com/clearcontainers/proxy/pull/11 to be merged before we merge this one. Indeed, we don't want to break virtcontainers until the proxy 3.0 is completely merged and ready.

There is a "detail" that needs figuring out still, I talked about it very briefly at the f2f and filed an issue for that: https://github.com/clearcontainers/proxy/issues/21.
Right now, there's no buffering of stdout data in the proxy, so the shim needs to be ready to receive stdout before execcmd is issued.

@dlespiau about clearcontainers/proxy#21, we are already doing what you described for the create/start sequences in runtime 3.0 + virtcontainers.
But I agree that regarding exec, it is more complicated as we don't have 2 separate commands to:

first: retrieve the token only
second: exec the exec command

I am not sure we want to modify virtcontainers so that we split this thing, and maybe the proxy should queue the output for a bit in case of exec (for a bit means not for a long time because shim connection should happen very quick). And thinking about future, maybe one shim will handle a pod as Manohar mentioned it :)
@sameo what is your opinion about this issue ?

@sboeuf @dlespiau See my comment on clearcontainers/proxy#21

For the record, this looks good to me, the interface definition is orthogonal to the PR as @jodh-intel noted.
lgtm

lgtm.

Now that we have the proxx 3.0 ready, I have rebased this PR and re-uploaded it. We should merge it :)

Coverage decreased (-0.2%) to 66.363% when pulling de46d018960ffaf0f6a161fa4121d610d55fd06c on sboeuf/move_to_proxy_3_0 into 932b63e7e0bbb180cabc4f0ff35b36aa91af61e2 on master.


Make the readthedocs download-able as a PDF

Is there a reason it's not? I'd like to download it to my tablet reader (a low-powered eink device) and be able to reference it from there. The web browsing feature (even if it's local) leaves a lot to be desired on some of those devices

I'll take a look - I think this is configurable in readthedocs, though it may require some work to get the PDF formatting to look OK

Hey there! I'm just following up on this. It seems like hte build was successful :) ? Would this be merge-ready soon(ish)?
Thanks for being so quick on this

The build was successful, but as indicated at the link above, CI builds skip the PDF to avoid timeouts. So to actually test this, I need to merge it, which I strongly suspect will cause a timeout in our readthedocs build (which occasionally times out as it is), at which point I'll most likely have to revert the change.
I just haven't had a block of time yet where I can deal with the potential churn, but I'll try to give it a shot this week.

Absolutely no worries! Worst case scenario I can look into it and see if I can build it on my end

OK, I gave it a try... good news is that it didn't time out. Bad news is that the PDF build failed because of SVG graphics used in the included jupyter notebooks (raw build log here: https://readthedocs.org/api/v2/build/16428989.txt). The problematic images are the "Open In Colab" badges at the top of each notebook rendered in the documentation.
It looks like the suggested fix is to use image directives in sphinx, but for us this is not possible because the source is in ipynb markdown and is automatically converted to rst via myst-nb, and the point of these badges is to be viewable in the ipynb itself.
We might be able to address this by switching these from SVG to PNG. Quite annoying that the only way to check is to merge a PR...

There are a large number of other errors there as well, including unicode that's unsupported by LaTeX.
I think given the amount of trial-and-error that would be required here, it's probably not worth spending time on unless we get a lot more interest in PDF docs. I'm going to abandon this and revert the initial change.
I'll leave this open in case anyone else wants to attempt going down the road of generating PDF from our sphinx docs.

Note that if your only goal is to be able to read the docs offline, you can always download the HTML zip version:

@IanQS , I actually think the suggestion by @jakevdp can be used to generate a PDF as follows:

Download the zip of the documentation as suggested by @jakevdp

Open the index.html in the zip:

This HTML file is different from what is shown on read the docs, in that it presents the complete documentation as a single page. So now we can do print as pdf. However, before doing that, open the developers console in your browser, for instance in Chrome: right click, Inspect. Go to the elements tab, and search for "Skip to main content". Right click on that element, and delete it:

Next, go to Print, Print as PDF. If you use chrome, go to more settings, and select "None" for margins. Loading can take a minute. Click save.


chore: Squash size components

What this PR does / why we need it:
Pickup work from #74 and #73 and finish it.
Release note:

None

For some reason this isn't causing lint.sh to fail:
$ make
./scripts/cri.sh  "./scripts/lint.sh"
./common-instancetypes/instancetypes/u/1/u1.yaml
  18:51     error    no new line character at the end of file  (new-line-at-end-of-file)

For some reason this isn't causing lint.sh to fail:
$ make
./scripts/cri.sh  "./scripts/lint.sh"
./common-instancetypes/instancetypes/u/1/u1.yaml
  18:51     error    no new line character at the end of file  (new-line-at-end-of-file)

https://github.com/kubevirt/common-instancetypes/pull/89

/approve
/lgtm

/test pull-common-instancetypes-kubevirt-functest


List Jackson's apps on solidproject.org

They are staged at https://github.com/solid/solid.github.io/pull/213/files#diff-82552945e34fa472e90ebb8de48a818bR17-R18 but not deployed yet.

yes so they are on https://solid.github.io/use-solid/apps but not on https://solidproject.org/use-solid/apps. So they will go live with next week's planned deploy.


Keeping attributes when using DataArray.astype

Hi all,
I was wondering whether there is an easier way to keep the attributes when using the DataArray.astype method?
import xarray as xr

# DataArray with attributes
da = xr.DataArray(
    [[0, 1, 2], [0, 1, 2]], attrs={"attr1": "value1"}
)

# the attributes are not passed over to new_da
new_da = da.astype(float)
# I have to set the attributes by myself
new_da.attrs = da.attrs.copy()

This is just one extra-line, but I have to keep track of the old DataArray and it may become unhandy if I do many astype calls.
Any hints?

Note: currently astype is directly assigning a method onto Dataset/DataArray (ugh!).
https://github.com/pydata/xarray/blob/dc3eebf3a514cfdc1039b63f2a542121d1328ba9/xarray/core/ops.py#L39
It should really be switched to use a more transparent definition, e.g., a method defined in a base class, with the implementation defined by using apply_ufunc:
https://github.com/pydata/xarray/blob/6402391cf206fd04c12d44773fecd9b42ea0c246/xarray/core/common.py#L748
This would make it quite easy to add keep_attrs=True.

I have a version of this working, but to get tests to pass I had to add the same behavior for Variable types (as the method was no longer being added from  NUMPY_UNARY_METHODS). I don't think I have a very good picture of the proper use of Variables in the internal api, so I wasn't sure if it made sense to extend the behavior therein. Also I should say that on the first pass I had to do this outside of the apply_ufunc mechanism, as apply_ufunc doesn't keep attrs for Variables (thus inspiring my question above).

OK, sounds good with adding this to Variable, too. It's the internal data structure that we build most of xarray's higher level labeled arrays on: http://xarray.pydata.org/en/stable/internals.html#variable-objects
Feel free to open an incomplete PR with your current progress and I am happy to advise

Above PR is a first draft. It would seem that the kwargs for the dask array method are a subset of the numpy array method, so I based docstring on these. Happy to do something else though if that makes sense.

Just run into this issue


test: improve selectors suite

The tests that are commented out are breaking the parser or doesn't output correctly.
E.g:
p:first {} // breaks.
&:nth-child(even of li, .item) {color: blue} // Doesn't break, but the output is wrong (&:nth-child(evenofli, .item))

Opened a PR migrating those tests into native: https://github.com/davesnx/styled-ppx/pull/259