Running time of HVM asymptotically slower than Agda/Haskell

When running this HVM code:
(If 1 t e) = t
(If 0 t e) = e

(Filter p Nil) = Nil
(Filter p (Cons x xs)) = (If (p x) (Cons x (Filter p xs)) (Filter p xs))

(Append Nil ys) = ys
(Append (Cons x xs) ys) = (Cons x (Append xs ys))

(Bind Nil f) = Nil
(Bind (Cons x xs) f) = (Append (f x) (Bind xs f))

(Range a b) = (Go (- (+ 1 b) a) a)

(Go 0 b) = Nil
(Go a b) = (Cons b (Go (- a 1) (+ 1 b)))

(Alltriples top) = (Bind (Range 1 top) (@z (Bind (Range 1 z) (@y (Bind (Range 1 y) (@x (Cons (Triple x y z) Nil)))))))

(Cartesian (Triple x y z)) = (== (+ (* x x) (* y y)) (* z z))

(Triples top) = (Filter (@x (Cartesian x)) (Alltriples top))

(Sumall Nil) = 0
(Sumall (Cons x xs)) = (Sumall.Go x xs)
    (Sumall.Go (Triple x y z) xs) = (+ (+ (+ x y) z) (Sumall xs))

(Main n) = (Sumall (Triples n))

and this Agda code:
{-# OPTIONS --guardedness #-}

open import Agda.Builtin.Nat
open import Agda.Builtin.Bool
open import Agda.Builtin.Unit
open import Agda.Builtin.String
open import Agda.Builtin.List

open import Function.Base using (_$_)
open import IO
open import System.Environment
open import Data.Nat.Show using (show; readMaybe)
open import Data.Maybe.Base using (fromMaybe)
open import Data.List.Base using (head; drop)

variable A B : Set

data MyList (A : Set) : Set where
  [] : MyList A
  _::_ : A → MyList A → MyList A

infixr 5 _::_

if_then_else_ : Bool → A → A → A
if true  then x else y = x
if false then x else y = y

id : A → A
id x = x

filter : (A → Bool) → MyList A → MyList A
filter p [] = []
filter p (x :: xs) = (if p x then (x ::_) else id) (filter p xs)

_++_ : MyList A → MyList A → MyList A
[]        ++ ys = ys
(x :: xs) ++ ys = x :: (xs ++ ys)

_my>>=_ : MyList A → (A → MyList B) → MyList B
[]        my>>= f = []
(x :: xs) my>>= f = f x ++ (xs my>>= f)

range : Nat → Nat → MyList Nat
range x y = go (suc y - x) x
  where
    go : Nat → Nat → MyList Nat
    go zero    _ = []
    go (suc m) n = n :: go m (suc n)

record Triple : Set where
  constructor triple
  field fst snd trd : Nat

alltriples : Nat → MyList Triple
alltriples top = range 1 top my>>= λ z → range 1 z my>>= λ y → range 1 y my>>= λ x → (triple x y z) :: []

pythagorean : Triple → Bool
pythagorean (triple x y z) = x * x + y * y == z * z

triples : Nat → MyList Triple
triples top = filter pythagorean (alltriples top)

sumall : MyList Triple → Nat
sumall [] = 0
sumall (triple x y z :: xs) = x + y + z + sumall xs

test1 = sumall (triples 200) -- evaluates to 33638

main : Main
main = run $ do
  args <- getArgs
  let nstr = fromMaybe "10000" (head args)
  let n = fromMaybe 10000 (readMaybe 10 nstr)
  let res = sumall (triples n)
  putStrLn (show res)

I get these running time:

Why is this the case? Shouldn't HVM always have the same or better time asymptotics as Haskell/others? Is this an HVM bug or is this a problem of Interaction Nets?

Shortly: I don't know. HVM is merely a fast implementation of the optimal algorithm, as described on the literature. I've mailed related researchers, but didn't get a reply on this specific issue. For what I've read, there should be no such case, so I thought it was a bug on HVM, but I'm surprised it also happens with BOHM. See #60.

Sorry to reopen, but this paper was brought to my attention recently and may be relevant:
https://discourse.haskell.org/t/high-order-virtual-machine-hvm-an-optional-ghc-like-runtime-for-rust-with-many-comparisons-to-ghc/4118/4


Fix default cost of ContractCallQuery

Describe the bug

ContractCallLocal is $0.001
ContractCall e $0.05

In the packages/abstract-signer/src.ts/index.ts the DEFAULT_HEDERA_CALL_TX_FEE is set to 143083413 which is around 14 HBARs.
Lets aim to have it as close to the actual cost as possible.
Environment:
Everywhere
Search Terms
call cost, hbar

From the user:


System arguments color valid values differ from LabelComponent's scheme valid values

👋🏻 Something I noticed when trying to coordinate an Octicon's color with a LabelComponent's scheme was that the valid values for each aren't the same. Compare:
color:

Text color. One of :accent, :attention, :closed, :danger, :default, :done, :inherit, :muted, :on_emphasis, :open, :severe, :sponsors, :subtle, or :success.

-- https://primer.style/view-components/system-arguments#color
scheme:

One of :accent, :attention, :danger, :default, :done, :info, :orange, :primary, :purple, :secondary, :severe, :sponsors, :success, or :warning.

-- https://primer.style/view-components/components/label
Common values:

:accent
:attention
:danger
:default
:done
:severe
:sponsors
:success

Differences:

:subtle is a valid color but is not a valid scheme
:closed is a valid color but is not a valid scheme
:inherit is a valid color but is not a valid scheme
:muted is a valid color but is not a valid scheme
:on_emphasis is a valid color but is not a valid scheme
:open is a valid color but is not a valid scheme
:subtle is a valid color but is not a valid scheme
:info is a valid scheme but is not a valid color
:warning is a valid scheme but is not a valid color
:secondary is a valid scheme but is not a valid color
:purple is a valid scheme but is not a valid color
:primary is a valid scheme but is not a valid color
:orange is a valid scheme but is not a valid color

I wanted to bring this up because I think some/all of the different values could actually probably work for both components, just based on my understanding that all these colors are coming from Primer variables like in https://primer.style/css/utilities/colors.

Hi @cheshire137, thanks for reporting this! It definitely seems like something that should get addressed. We're going to move this into our backlog to explore improving it in the future.

Not stale

Are we still not stale, @lesliecdubs? The bot is insistent. 😅

@cheshire137 I think this is still a valid issue, though it's unlikely we will have resources within Primer Engineering to address it in the near future. Please feel free to reopen in the "Unprioritized backlog" if you'd like to keep this on the radar for future improvement.

Please feel free to reopen in the "Unprioritized backlog" if you'd like

Thanks! Sorry, is that a milestone/project/repo/label?

Sorry for the obtuse instruction 🤦🏻♀ I will go ahead and reopen this issue and move it to the "Unprioritized backlog" on our team project board.


AllowDrag and AllowDrop

Hi, great project!...
We have been trying it out and a couple of features it would be nice to have are the ability to disable sorting of nodes, perhaps using an item on individual nodes, e.g. allowDrag: false
Also, the ability to check if dropping is allowed on a particular target, maybe using a new method like allowDrop, e.g.
allowDrop: (event, movedNode, targetParentNode) => {
     if (targetParentNode.data.title == 'NoDrop') return false;

     return true;
}

If the function returns false, maybe a new class is applied to the target node whilst dragging so it could be styled to show dropping is not allowed, e.g. tree__node--drop-disallowed, and after dropping it would not action the move and would return the tree to the state prior to dragging.
Is there any current way to achieve the above, or is this something you might consider including in a future release?

There is already a confirm option that takes a validation function as value.


Toggle test_brt shortcut mode from command line

The test_brt function can run either with mocked, partially-processed inputs, (shortcut mode) or by running the full batchruntomo command. Need to be able to control running either in one or the other mode. Currently it's done by commenting out code, which is error-prone.

Withdrawing this Issue as OBE. We will move away from the shortcut BRT with Prefect 2.

we should no longer be running the mocked outputs I think, it's quicker to generate new outputs with the given BRT params.

Agreed. I was thinking we would pull the utils.run_brt() when we move to Prefect 2. Or we could just pull it now and remove the @pytest.mark.slow mark.


Backport of 2613 in stable-3.0

Backport of #2613 in stable-3.0

jenkins test luminous-ansible2.4-centos7_cluster

jenkins test luminous-ansible2.4-docker_cluster


Added transitionend event

Looks good to me. I'm trying to work out whether it's a breaking change or not though! I think it might be due to the change in type synonym, but then it's a row type, so maybe not...

@garyb It is a breaking change, I think; consider the case where someone had written code like InteractiveEvents (onTransitionEnd :: I); I think this would be an error now.


How to filter by catalog properly?

Hi, I'm using this as an example:
params = {
    "search_text": "Ticket to Ride",
    "catalog[]": 2311,
    "order": "newest_first",
    "page": 1
}

However, I can’t seem to find the right way to filter by category. Changing the catalog ID always seems to return the same products. I’ve tried using "catalog" with and without [], as well as providing the ID with commas or within brackets.
Could you please guide me on how to filter by category properly?
By the way, it’s great to meet another board game enthusiast!

Hi!
Thanks. Unfortunately, board games are quite an expensive hobby 🥲 and we need to find the best way to spare 🤣
I normally use catalog_ids to filter by category.


hut: 0.3.0 -> 0.4.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/hut/versions.
meta.description for hut is: A CLI tool for Sourcehut / sr.ht
meta.homepage for hut is: https://sr.ht/~emersion/hut/
Updates performed

Golang update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.4.0 in filename of file in /nix/store/jb3av1k6gwa0xm19z44dh9lyhy4f2wdd-hut-0.4.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
hut

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/e37a3c3b506a5c4bc2e83f4a896dffa1360f86e0#hut

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/jb3av1k6gwa0xm19z44dh9lyhy4f2wdd-hut-0.4.0
ls -la /nix/store/jb3av1k6gwa0xm19z44dh9lyhy4f2wdd-hut-0.4.0/bin

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

1 package built:

hut

Maintainer pings
cc @fgaz for testing.

I don't see any major/breaking changes https://git.sr.ht/~emersion/hut/refs/v0.4.0 so let's backport


sas supports raw permissions

closes https://github.com/Azure/azure-sdk-for-cpp/issues/1032

/azp run cpp - storage

/azp run cpp - storage


README.md: add Alternatives section

I love to add cross-references. But I think KeePass will not accept my PR to mention this repo :(

If we add alternatives, the list would be a loooooot longer and I don't even think keepassxc is an alternative to this project. That's a password manager that has totp capabilities, but most if not all password managers has totp capabilities. I don't think adding that to the readme has any value. Adding tools like oathtool, WhyNotHugo/totp-cli, Google Authenticator, Authy, or even aram-hamo/totp-cli (which is a wrapper around oathtool) would have some value, but adding a password manager is like saying Chrome and Firefox are alternatives to wget and curl, indeed they both can download a webpage, but I wouldn't say they are alternatives to each other.
This project is not and will not be a password manager, only totp tokens following the gnu principal of "doing one thing that can be combined with other tools and do not try to do more things".
In my opinion if you say something is an alternative solution to something, you can create a table with various features to list which one is implemented in each of them, and show how good or bad the implementation is. For totp you can list only one thing that you can split up if you try very hard like "store totp secret token", "generate totp token from secret", "delete stored totp secret token". But that's it. For keepassxc that list is a lot longer and 90% of it is totally irrelevant to totp-cli.
From the front page:

KeePassXC: Cross-platform Password Manager
totp-cli: Authy/Google Authenticator like TOTP CLI tool written in Go.

We can add an Alternatives section, I'm happy to include them, but only with alternative tools, tools living in the same space, and not tools/programs that has similar functionality as a tiny fraction of their full set of features. For example lists see previous comment.
What is not an alternative software: LastPass, Bitwarden, KeePass, etc.


Fix subscribe panic

Handle errs from request after ApplyOperation is executed
Prevent panic when "var f *fieldToExec" is nil
Added field "name" on pack error

What is the panic that you faced before? Can you, please, provide a simple example to reproduce the issue that this PR fixes?

There are conflicts to be resolved.


Update method for fwspp object

Store as an attribute the datetime a query was initiated.  Subsequently use this to pass to the get_* functions to, I suspect, profoundly reduce the size and increase the speed of query updates.

[ ] GBIF API accepts a lastInterpreted parameter
[x] No obvious solution in BISON SOLR API
[ ] Maybe datemodified in iDigBio API, but need to check further
[ ] Perhaps lastindexed is appropriate VertNet parameter?
[ ] No obvious EcoEngine solution, but probably not a huge concern
[ ] No obvious AntWeb solution, but likely not a concern

No longer needed for BISON.


Add React rules for eslint

Right now we only have some simple rules for JSX and React. We want to enforce a strict standard also for JSX so please add some rules that enforce a nice style for JSX.

I have added some but we can definetly improve here :).

Thanks! :)

Closing this issue for now :)


polymer 1.0

is it working with polymer 1.0 ?
because i'm getting the error (i'm using branch 1.0.2 of gwt-polymer)
Have you got already a compiled jar ?
[17:44:15] Starting 'gwt-api:generate-imports-map'...
[17:44:15] Generating element interface from date-input.json
stream.js:94
throw er; // Unhandled stream error in pipe.
^
SyntaxError: Unexpected end of input
at Object.parse (native)
at C:\Users\jim\git\gwt-polymer\gwt-polymer-elements\gulpfile.js:78:61
at wrappedMapper (C:\Users\jim\git\gwt-polymer\gwt-polymer-elements\node_modules\map-stream\index.js:84:19)
at Stream.stream.write (C:\Users\jim\git\gwt-polymer\gwt-polymer-elements\node_modules\map-stream\index.js:96:21)
at write (C:\Users\jim\git\gwt-polymer\gwt-polymer-elements\node_modules\gulp\node_modules\vinyl-fs\node_modules\through2\node_modules\readable-stream\lib_stream_readable.js:623:24
)
at flow (C:\Users\jim\git\gwt-polymer\gwt-polymer-elements\node_modules\gulp\node_modules\vinyl-fs\node_modules\through2\node_modules\readable-stream\lib_stream_readable.js:632:7)
at DestroyableTransform.pipeOnReadable (C:\Users\jim\git\gwt-polymer\gwt-polymer-elements\node_modules\gulp\node_modules\vinyl-fs\node_modules\through2\node_modules\readable-stream
lib_stream_readable.js:664:5)
at DestroyableTransform.emit (events.js:104:17)

This branch is not working yet.
Will keep you informed when we finish.

Hi there,
we released gwt-polymer elements v<IP_ADDRESS>-alpha1 last friday. It's also available through Maven.
Please let us know how it works for you.

hi. great,
demo seems to work but npm install giving errors: here is the end of npm-debug file
info<EMAIL_ADDRESS>Failed to exec postinstall script
42042 verbose stack Error<EMAIL_ADDRESS>postinstall: node_modules/gwt-api-generator/bin/gwt-api-generator.js --package=PolymerElements/paper-elements#1.0.1,PolymerElements/iron-elements#1.0.0 --pom  --artifactId=gwt-polymer-elements --groupId=com.vaadin.polymer && mvn clean install javadoc:jar
42042 verbose stack Exit status 1
42042 verbose stack     at EventEmitter. (C:\Program Files\nodejs\node_modules\npm\lib\utils\lifecycle.js:213:16)
42042 verbose stack     at EventEmitter.emit (events.js:110:17)
42042 verbose stack     at ChildProcess. (C:\Program Files\nodejs\node_modules\npm\lib\utils\spawn.js:14:12)
42042 verbose stack     at ChildProcess.emit (events.js:110:17)
42042 verbose stack     at maybeClose (child_process.js:1015:16)
42042 verbose stack     at Process.ChildProcess._handle.onexit (child_process.js:1087:5)
42043 verbose pkgid<EMAIL_ADDRESS>42044 verbose cwd C:\Users\jim\git\gwt-polymer-elements
42045 error Windows_NT 6.3.9600
42046 error argv "C:\Program Files\nodejs\\node.exe" "C:\Program Files\nodejs\node_modules\npm\bin\npm-cli.js" "install"
42047 error node v0.12.2
42048 error npm  v2.7.4
42049 error code ELIFECYCLE
42050 error<EMAIL_ADDRESS>postinstall: node_modules/gwt-api-generator/bin/gwt-api-generator.js --package=PolymerElements/paper-elements#1.0.1,PolymerElements/iron-elements#1.0.0 --pom  --artifactId=gwt-polymer-elements --groupId=com.vaadin.polymer && mvn clean install javadoc:jar
42050 error Exit status 1
42051 error Failed at the<EMAIL_ADDRESS>postinstall script 'node_modules/gwt-api-generator/bin/gwt-api-generator.js --package=PolymerElements/paper-elements#1.0.1,PolymerElements/iron-elements#1.0.0 --pom  --artifactId=gwt-polymer-elements --groupId=com.vaadin.polymer && mvn clean install javadoc:jar'.
42051 error This is most likely a problem with the gwt-polymer package,
42051 error not with npm itself.
42051 error Tell the author that this fails on your system:
42051 error     node_modules/gwt-api-generator/bin/gwt-api-generator.js --package=PolymerElements/paper-elements#1.0.1,PolymerElements/iron-elements#1.0.0 --pom  --artifactId=gwt-polymer-elements --groupId=com.vaadin.polymer && mvn clean install javadoc:jar
42051 error You can get their info via:
42051 error     npm owner ls gwt-polymer
42051 error There is likely additional logging output above.
42052 verbose exit [ 1, true ]

Couldn't find the original issue, but there seems to be a number of projects in Github having issues running post install scripts with npm in Windows. Apparently it can't resolve the relative path to the script properly. Added an explicit call to node to remedy the situation.

Great it s working now.
Maven build failed because i haven't gpg but i will download it.
Super. Congratulations for your library.

You can remove the build related plugins from pom.xml if you're just interested in compiling the library for local use. GPG is only needed for deployment, we're considering on extracting those to a separate pom.xml


make eta_reduction less aggressive

Hi! Thank you for this very useful tool.
I found the eta-reduction to be too aggressive for all cases, reducing code readability in complex lambda expressions. When there are more arguments to the call in the lambda body, explicit arguments indicate more useful information to the reader. My idea is to add a restriction that makes it only run for the most primitive expressions. Is that an idea worth pursuing?
The test fixture is a bit contrived. Real-world code can be more complex.

hi, thanks for the PR, i think i do agree with the changes here, the reduced form leads to too much mental gymnastics sometimes.

thanks for the contribution!


update go to the latest version

https://github.com/stolostron/backlog/issues/26062

Signed-off-by: Chunxi Luo<EMAIL_ADDRESS>(cherry picked from commit a32482ab0a825b8f86b78c636a2688e7200bf08c)

/cherry-pick release-2.6


Ambari views

I have included the recipe to install, setup and start Ambari-server, Tez view and start Application TimeLineServer.

Closing for cleanup. Feel free to reopen if this work gets picked up later on.


add rule (vue-strong): Component name casing in templates

https://vuejs.org/style-guide/rules-strongly-recommended.html#component-name-casing-in-templates

For this we have to check all tags in descriptor.template.ast?.children and we have to have a list of html and svg element names like in https://github.com/vuejs/eslint-plugin-vue/blob/master/lib/rules/component-name-in-template-casing.js


added submodule for koron/go-ssdp

With the submodule specified in .gitmodules the docker hub build works again.
The go-ssdp subproject commit was updated to point to the currently newest commit.

Looks good, thank you for your work @maaximal .
@tombowditch I'm merging this as we desperately need it.


Fix incorrect detection of silenced errors (by the @ operator)

As can be found here the @ operator no longer silences all error types starting with PHP 8: https://php.watch/versions/8.0/fatal-error-suppression.
Initially I thought the following was happening but that is not true: So when capturing a silenced error the error_reporting() will not report 0 on PHP 8 but 4437 (which is E_ERROR | E_PARSE | E_CORE_ERROR | E_COMPILE_ERROR | E_USER_ERROR | E_RECOVERABLE_ERROR, check for yourself: https://maximivanov.github.io/php-error-reporting-calculator/).
When capturing a silenced error the error_reporting() will not report 0 on PHP 8 but E_ERROR | E_PARSE | E_CORE_ERROR | E_COMPILE_ERROR | E_USER_ERROR | E_RECOVERABLE_ERROR without the ones you excluded in an earlier error_reporting() call yourself (for example error_reporting(E_ALL & ~E_USER_ERROR) would result in error_reporting() being: E_ERROR | E_PARSE | E_CORE_ERROR | E_COMPILE_ERROR | E_RECOVERABLE_ERROR when handling a silenced error on PHP 8 (missing the E_USER_ERROR)).
The current behaviour makes it so that error_reporting() is leading in determining what is a "silenced error" which is simply not true under PHP 8 (or 7 for that matter).
Currently when you set error_reporting(E_ALL & ~E_NOTICE) and 'error_types' => E_ALL it will currently report all E_NOTICE as silenced errors (thus not report them at all) which is incorrect and something I consider to be a bug and the incorrect behaviour.
This PR aims to resolve the above.
There is 1 edge case in this "detection" of silenced errors. If the user set's their error_reporting() to 0 on PHP 7 and below or anything only containing the error types that cannot be silenced on PHP 8 and up we detect all errors as silenced, but I consider this acceptable since this was always the way it was before the broken code was introduced. If we find this unacceptable we need to remove the silenced errors detection all together.

It looks like as @ste93cry mentioned I overlooked that the error_reporting() set when reporting on silenced errors is not fixed as I thought, so I created a workaround for that too (the joys of error handling code).
Also to show that the difference between what PHP 7 and 8 reports but with out current code they are both wrong I have this little proof of concept (adapted from @ste93cry): https://3v4l.org/nct1E (I would expect it to report the silent status correctly which is does not).
I have also tried to add more tests and better word what we I am doing and why I am dong it.

We should backport those tests in the 2.x branch and apply fixes if needed.

There is no need to backport because this was introduced in #1087 and that change only went in 3.x so we're good!

FYI: this is a bugfix and as such should have been merged into the master branch, not the develop branch. I cherry-picked the patch into the master branch with 2c59c6c


Rename "Feature" to "Features" in test folder

Should be plural. No idea why it's singular.

@driesvints Why should it be plural? All I see in the community is the folders with a singular name

@MichaelDeBoey you're right! No idea why I thought it would be different.

Just looked at the issues I could maybe help with and found this one a bit odd that's why I was asking, just to be sure 🙂
Saw you also changed it for the rewrite of the portal (laravelio/portal#241) 🙂
Keeping small things like this consistent through the whole community is something I see more when browsing more through interesting project 🙂


arch-riscv: Fatal if specify RVV in RV32 mode

Currently we only support RVV in RV64 mode, if users want to run RISC-V
in RV32 mode, the should disable RVV manually.
Change-Id: I1f224d2b927d576d13991aa440c9e2c2c3969f6a

Not intent to be merged


Keras fails to feed ragged/sparse inputs with correct input placeholder

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes, custom subclassed model
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Google Colab
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: No
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): v1.12.1-17764-gae26958 2.1.0-dev20191111
Python version: Google Colab py3
Bazel version (if compiling from source): No
GCC/Compiler version (if compiling from source): No
CUDA/cuDNN version: Google Colab
GPU model and memory: Google Colab

Describe the current behavior
Error raised when trying to feed ragged/sparse tensors to model with tf.data.Dataset
Describe the expected behavior
There should be no error
Code to reproduce the issue
https://colab.research.google.com/drive/1Ysbe9uxWLn1eFWNMax9SAHewHMrXwG-R
Other info / logs
---------------------------------------------------------------------------
InvalidArgumentError                      Traceback (most recent call last)
<ipython-input-4-ed795b275880> in <module>()
----> 1 sparse_model = MyModel(False)
      2 sparse_input = my_dataset(False)
      3
      4 sparse_model.compile(optimizer='Adam', loss='mse')
      5 sparse_model.fit(sparse_input)

13 frames
<ipython-input-2-45855ec3bda4> in __init__(self, ragged_on, **kwargs)
     13     outputs = tf.keras.layers.GlobalAveragePooling1D()(dense)
     14
---> 15     super(MyModel, self).__init__(inputs=inputs, outputs=outputs)

/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/keras/engine/training.py in __init__(self, *args, **kwargs)
    144
    145   def __init__(self, *args, **kwargs):
--> 146     super(Model, self).__init__(*args, **kwargs)
    147     _keras_api_gauge.get_cell('model').set(True)
    148     # initializing _distribution_strategy here since it is possible to call

/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/keras/engine/network.py in __init__(self, *args, **kwargs)
    167         'inputs' in kwargs and 'outputs' in kwargs):
    168       # Graph network
--> 169       self._init_graph_network(*args, **kwargs)
    170     else:
    171       # Subclassed network

/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/training/tracking/base.py in _method_wrapper(self, *args, **kwargs)
    455     self._self_setattr_tracking = False  # pylint: disable=protected-access
    456     try:
--> 457       result = method(self, *args, **kwargs)
    458     finally:
    459       self._self_setattr_tracking = previous_value  # pylint: disable=protected-access

/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/keras/engine/network.py in _init_graph_network(self, inputs, outputs, name, **kwargs)
    270
    271     if any(not hasattr(tensor, '_keras_history') for tensor in self.outputs):
--> 272       base_layer_utils.create_keras_history(self._nested_outputs)
    273
    274     self._base_init(name=name, **kwargs)

/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/keras/engine/base_layer_utils.py in create_keras_history(tensors)
    185     keras_tensors: The Tensors found that came from a Keras Layer.
    186   """
--> 187   _, created_layers = _create_keras_history_helper(tensors, set(), [])
    188   return created_layers
    189

/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/keras/engine/base_layer_utils.py in _create_keras_history_helper(tensors, processed_ops, created_layers)
    245           else:
    246             with ops.init_scope():
--> 247               constants[i] = backend.function([], op_input)([])
    248       processed_ops, created_layers = _create_keras_history_helper(
    249           layer_inputs, processed_ops, created_layers)

/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/keras/backend.py in __call__(self, inputs)
   3725         value = math_ops.cast(value, tensor.dtype)
   3726       converted_inputs.append(value)
-> 3727     outputs = self._graph_fn(*converted_inputs)
   3728
   3729     # EagerTensor.numpy() will often make a copy to ensure memory safety.

/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/eager/function.py in __call__(self, *args, **kwargs)
   1527       TypeError: For invalid positional/keyword argument combinations.
   1528     """
-> 1529     return self._call_impl(args, kwargs)
   1530
   1531   def _call_impl(self, args, kwargs, cancellation_manager=None):

/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/eager/function.py in _call_impl(self, args, kwargs, cancellation_manager)
   1567       raise TypeError("Keyword arguments {} unknown. Expected {}.".format(
   1568           list(kwargs.keys()), list(self._arg_keywords)))
-> 1569     return self._call_flat(args, self.captured_inputs, cancellation_manager)
   1570
   1571   def _filtered_call(self, args, kwargs):

/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/eager/function.py in _call_flat(self, args, captured_inputs, cancellation_manager)
   1668       # No tape is watching; skip to running the function.
   1669       return self._build_call_outputs(self._inference_function.call(
-> 1670           ctx, args, cancellation_manager=cancellation_manager))
   1671     forward_backward = self._select_forward_and_backward_functions(
   1672         args,

/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/eager/function.py in call(self, ctx, args, cancellation_manager)
    521               inputs=args,
    522               attrs=("executor_type", executor_type, "config_proto", config),
--> 523               ctx=ctx)
    524         else:
    525           outputs = execute.execute_with_cancellation(

/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/eager/execute.py in quick_execute(op_name, num_outputs, inputs, attrs, ctx, name)
     65     else:
     66       message = e.message
---> 67     six.raise_from(core._status_to_exception(e.code, message), None)
     68   except TypeError as e:
     69     keras_symbolic_tensors = [

/usr/local/lib/python3.6/dist-packages/six.py in raise_from(value, from_value)

InvalidArgumentError:  You must feed a value for placeholder tensor 'data/values' with dtype float and shape [?]
	 [[node data/values (defined at <ipython-input-2-45855ec3bda4>:15) ]] [Op:__inference_keras_scratch_graph_22]

Function call stack:
keras_scratch_graph

Issue replicating for the given code. Thanks!

Original issue is still here with tf-nightly, but this workaround works https://github.com/tensorflow/tensorflow/issues/27170#issuecomment-589546804

Does it still fail? Looks like it's fixed in nightly -- even the colab suggests so

@shkarupa-alex
I have tried in colab with TF version 2.3 and i am not seeing any issue. Please,find the gist here.Please, verify once and close the issue. Thanks!


Webhooks

I have added some more webhook notifications.

Did I address everything? Not to rush, just want to make sure I didn't miss anything.

Hi @wesdean, there are still some updates needed. See my last comment. Thanks!

Cool, this is almost done! It just needs tests (which were probably removed by the rebase, because I know they were there before) and let's rename the old dunning file _deprecated and name the new one without any suffix. Thanks!

Hi, any updates for this PR?
May I help somehow @wesdean ?

@skix123 Help would be great. I've had too much else going on to wrap this up. Just look at Kate's last comment.
Cool, this is almost done! It just needs tests (which were probably removed by the rebase, because I know they were there before) and let's rename the old dunning file _deprecated and name the new one without any suffix. Thanks!

I don't understand this one:

let's rename the old dunning file _deprecated and name the new one without any suffix

Can you explain @katenicoletti ?

Updated version in PR #72
@wesdean good job!

I'm going to close this PR and review #72. Thanks for your contributions everyone!


Type completions in jsDoc @augments and @implements tags.

Suggestion
Add code completions in jsDoc @augments and @implements clauses.
Use Cases
Anybody using the new @implements and @augments jsDoc tags will benefit from this.
Examples
Inside the {} triggering code completions should show types.
/**
 * @augments {}
 * @implements {}
 */
declare class B {
    b(): void;
    i(): void;
}

Checklist
My suggestion meets these guidelines:

[X] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[X] This wouldn't change the runtime behavior of existing JavaScript code
[X] This could be implemented without emitting different JS based on the types of the expressions
[X] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[X] This feature would agree with the rest of TypeScript's Design Goals.

This is a flow-on from #36292 cc: @sandersn

I have a PR ready to go for this one the @implements  (#36292) PR makes it in.

It turns out that Typescript doesn't even support this properly — there aren't even any (good) completions in Typescript for class B implements /**/. And it's not a completion location at all, so you have to request completions manually in the first place.
We can wait to fix this issue until completions for Typescript interfaces is available, at which point it should be fairly simple.


Add a default folder for when there is no note open

I recently discovered the full power of this plugin for navigating with the keyboard. It's quickly moved to my most used and loved Obsidian plugin. Thank you!
Here is a suggestion to make it even better: add the ability to configure additional hotkeys that open directly to specific folders.
As an example I have an inbox folder where all new notes go, including externally added clippings. Every so often I process those notes (GTD style) making additional notes and moving thing to the approriate place. When I move a note it's usually into a deeply nested folder structure, and I want go right to the next note in the inbox. Right now I have to open the root folder (CMD+U in my setup) then navigate via the arrow keys. It would be terrific if I could just hit CMD+U and have it already at the fist note in the inbox instead of in the root.

Hm.  How about I add a URL action to open a specific folder?  Then you can use the URI Commands plugin to set up as many specific-folder commands as you like.

That's an interesting idea. I think it would work for me personally, though I can't imagine many other situations where a URL action would be helpful for this plugin, (that doesn't mean they don't exist).
My only other thought is that passing a hotkey to a URL action might feel a little slow compared to a direct call. I haven't done any plugin creation personally, or used the URI Actions plugin, so that's only my uneducated opinion.

You might find it a better workflow to do something like:

Set up a processing-specific workspace using the built-in workspaces plugin
In that workspace, either set the search to path:My/Inbox/Folder or pin a note in one pane that has an embed of that search (or a similar dataview)

Now, when you want to process, just open that workspace and there's your files to be processed, and clicking on any item will open it in the first unpinned pane.  When you move the item out of your folder, the search or dataview will update, with that item gone from it.
When you're done processing, flip back to your standard workspace, et voila.
To be fair, this process does require you to mouse-select your next file to work on, since search and dataview don't support keyboard selection.  However, if you pin a note that's in the folder you want, you can keyboard back to that pane and then open quick explorer and you're in business.  So three or so keystrokes rather than 1, but still 100% keyboard.

My current keyboard driven strategy is already totally tolerable. This would only make it that much faster. I like the idea of using a workspace though.

Well, since the URI thing isn't something you're interested, I'm going to go ahead and close this.  My only other suggestion besides that and the workspace thing is that you make a QuickAdd or Templater macro/userscript to jump to the next available file in your inbox, then bind that to a hotkey.  Or make an index file in your inbox, star it with the stars plugin, then use the hotkeys-for-stars plugin to define a hotkey to jump to that file.  Or...  well, there are lots of ways to handle it, but I don't think the idea of a default folder makes any sense -- the default folder is just the vault root.

I am interested in the URI option. It would be far more flexible than what I was originally thinking of.
I've been testing the waters with both QuickAdd and Templater recently so I'll look into options for using them as well.
I wasn't aware of the hotkeys-for-stars plugin. It turns out the creator also made Hotkeys for specific files which doesn't quite solve my problem but is a good start and super useful in other contexts.

Hm.  How about I add a URL action to open a specific folder?  Then you can use the URI Commands plugin to set up as many specific-folder commands as you like.

Hello @pjeby and @montemartin !
Forgive @pjeby for step into this conversation.
I just wanna say that an URI action to open folder in QE is an amazing feature. It's the of feature that fits perfectly with QE !

URI Commands to specific folders (as @pjeby said)
QuickAdd Multi Choice to create collections of URI for specific purposes/workflows;

If you don't mind, I just wanna add 2 suggestions to this feature:
01. Parameter for bottom
Would be amazing to have a parameter in the URI to define the focus of the selector when opening the folder.

classic example would be the Daily Notes folder, where the most recent notes are in the bottom

I'm assuming that the default position is top
02. Variable for the Active Folder {{folder_-1}} {{folder_-2}} ...
I will try to explain with my poor English.
Imagine this folder structure for the note Guggenheim.md
Continent\North America\Country\USA\State\NY\City\New York\Museums\Guggenheim.md
Guggenheim.md is my ACTIVE NOTE
I want to "travel" between the folders of my active note
To go to the Museums folder I just hit QE: Browse to Current Folder
BUT With a {{variable}} we can create URI to quickly go to other folders related to the active note

folder -1 is New York;
folder -2 is NY;
folder -3 is State ... and so on

Would be great to have a dynamic URI applied to the active note :)
Sorry for the long text !
Have a great day!

It would probably simpler to just allow relative paths, i.e. . for current folder, .. for folder above, and so on.
As far as the bottom thing is concerned, I'm not really sure.  The command I had in mind is effectively "explore this item", which means if you target a folder, the selected item would be the folder, in its parent folder, not a random position in the target folder.  I'd need to add something like folder/ to mean "show the contents of the folder", in which case nothing would be selected and you could go either up or down or type something.
Also, I have noticed, since beginning to use the URI Commands plugin, that running commands is slow because they shell out to the OS and then send a message back to the current process.  (There is a way to speed that up significantly but it requires using a currently-undocumented API in Obsidian.)

It would probably simpler to just allow relative paths, i.e. . for current folder, .. for folder above, and so on.

100% agree much easier to understand :)

As far as the bottom thing is concerned, I'm not really sure.  The command I had in mind is effectively "explore this item", which means if you target a folder, the selected item would be the folder, in its parent folder, not a random position in the target folder.  I'd need to add something like folder/ to mean "show the contents of the folder", in which case nothing would be selected and you could go either up or down or type something.

Would be amazing to have URI without bottom/top @pjeby !!!!

My idea was to show the variety of subset of features that we could have from the main feature which is URI for QE!

Whatever you decide to approach, I know that will be the best for the users and for QE :)

Also, I have noticed, since beginning to use the URI Commands plugin, that running commands is slow because they shell out to the OS and then send a message back to the current process.  (There is a way to speed that up significantly but it requires using a currently-undocumented API in Obsidian.)

Good to know @pjeby !
In my specific case I use the plugin Shell Commands to run my URI (created using Advanced URI plugin) instead of URI Commands.

https://github.com/Taitava/obsidian-shellcommands

From my experience SC is fast and flawless :)

I would use Shell Commands plugin for the QE URIs

Thanks for your replying and for considering URI for QE !
Have a great day!

Actually, it turns out I was incorrect about the performance issue: I was projecting due to the earlier comment by @montemartin, as I only just set up a URI command this weekend and then thought it was kind of slow to respond.  But it turns out that it's actually the Obsidian hotkey settings page that's slow to load for some reason, as every other URI I've set up since then appears to run instantly.

Thanks for the info @pjeby !
When something behaves different from what we expect in Obsidian, we have so many variables to investigate to try to find a solution.
I'm glad to know that URI Command is working great!


react hooks support?

Cool lib, thanks!
How can I use hooks, for example useState?

Thanks @crazy4groovy !
Great question. I assume you are talking about the jsxLoader. Yes there are several different methods:
Example using Global React Object
// This exits at the top of the file:
const useState = React.useState;
const useReducer = React.useReducer;

https://awesome-web-react.js.org/examples/state-management/react-hooks.htm
Source
https://github.com/dataformsjs/awesome-web-react/blob/master/examples/state-management/react-hooks.jsx
Docs
This topic in the doc shows several additional options, for example if coming from node/webpack and copying and pasting the a file for browser then simple find and replace methods can be used to handle existing code.
https://github.com/dataformsjs/dataformsjs/blob/master/docs/jsx-loader.md#handling-node-importexport-statements-and-browser-exportsrequire
Here are a few more examples using popular hook libraries:a
React Hook Form
https://awesome-web-react.js.org/examples/forms/react-hook-form.htm
React Query
https://awesome-web-react.js.org/examples/component-and-hook-libraries/react-query.htm
Basically JSX Loader simply transforms the JSX to JS for modern browsers (happens in milliseconds) and if a old browser is used (IE 11, older iPhones with Safari, etc) then the JSX Loader will download Babel Standalone and use that for the compiler.
As for the React Components currently only Components have been published for DataFormsJS. For example <JsonData> which is used to easily specify JSON Web Services and pass the data to child components:
function DataPage() {
    return (
        <JsonData
            url="https://www.dataformsjs.com/data/geonames/countries"
            isLoading={<ShowLoading />}
            hasError={<ShowError />}
            isLoaded={<LazyLoadDataPage />}
            loadOnlyOnce={true} />
    );
}

Thanks, const useState = React.useState is what I needed! (You may want to consider adding that in the docs or an example somewhere.)
👍

Thanks @crazy4groovy
Your advice helped me improved the main readme doc so I gave you credit for the idea in the  https://github.com/dataformsjs/dataformsjs/blob/master/CHANGELOG.md on the release that I just published!
It was buried in the detailed docs before but this makes it much clearer I think for someone who see's the project for the first time.
Thanks again!


Yet another round of battling building and testing containers on Windows

The Problem/Issue/Bug:
We seem to never be able to build and test our containers on WIndows and elsewhere.
Trying again.
How this PR Solves The Problem:
Manual Testing Instructions:
Automated Testing Overview:

Related Issue Link(s):
Release/Deployment notes:

Much pain, much learning. I think this is ready for a quick review. Mostly I learned about agent setup, and may still have some work to do on that, maybe an agent qualifier script that makes sure git works right and all the right tools are installed?

I'm probably good with this now, my energy may have run its course again :)


Pass thread id to task ?

This is a very nice up to date version, thanks for making this available.
In my use case, I would like the task to access a structure based on the
thread id - is is possible to pass this to the task ?

never mind, figured it out :)

nope, still can't figure it out

Doesn't std::this_thread::get_id() work?

Thanks! I was hoping to get the thread number from the pool, i.e. 0,1,..... num_threads.
But, I guess I can use the thread id for my purpose : access per-thread data.


Ensure template_path always uses "/" to match jinja

This PR shoudl fix #1545
The existing code substituted / for , assuming this was the right behaviour for windows. But on Windows, Jinja still uses / for the template list - See https://github.com/pallets/jinja/blob/896a62135bcc151f2997e028c5125bec2cb2431f/src/jinja2/loaders.py#L225

Aha I understand the problem now! https://github.com/pallets/jinja/issues/1378#issuecomment-812410922

Jinja template names/paths are not always filesystem paths. So regardless of the OS Jinja always uses forward slashes.

I was hoping to get the test suite running on Windows before merging this PR but that looks like it will be a BIG job, see:

#1627

So I'm going to merge this one as-is for the moment.


size

how can i justify rating size,
it look so big

We were just having a similar discussion (see issue #7). I will be looking at ways to import only the required components without making the installation complicated.


Add context propagation to rclone

Changes:

Change rclone/fs interfaces to accept context.Context
Update interface implementations to use context.Context
Change top level usage to propagate context to lover level functions

Context propagation is needed for stopping transfers and passing other request-scoped values.
Fixes: #3257

Thank you for this, will merge now :-)
Alas it is going to break a lot of pending changes, but that can't be helped :-(


Fix Firestore build broken by M70 version changes

#no-changelog

cc: @mikehaney24 See the question above.

LGTM
Can we merge GoogleDataTransport and GoogleDataTransportCCTSupport? When would you ever want one without the other? (I should add that I mean this for the future--not this PR.)

The idea is that one could implement their own backend and share the bulk of the transport code. There are plans for this in the works, so I'd prefer to keep them separate for now and reassess later.


fix(XHR Failed to construct Response): the status provided 0 is outside the range 200 599

Fixes #335

this fix address the below issue:

I can help you with rebasing this so don't worry. Just ping me once you think I can look at this. Thanks!

@smo043, there's been quite fundamental changes to the XHR interceptor but what you're proposing here still makes sense. I've switched to a Proxy-based interception and I see you've already rebased. Thanks.

I can confirm that if request fails then xhr.status will be rightfully 0. I think we still attempt to construct a Response in that scenario? That'd be incorrect.
This is how we can emulate request failure:
interceptor.on('request', () => {
  throw new Error('Network error')
})

This will halt the request (readyState 4) and its status will remain at 0, which is correct. That indicates that request never took off.
I'm curious if we still attempt to create a Fetch Response in that case 🤔

@kettanaito - Amazing I'll test it today and update you

@kettanaito - I tried with the latest version. I blocked the url in the browser's network tab, and this time there is no response with a status code.
I have also noticed that I can't import presets or ClientRequestInterceptor as per documentation.
Error:

package path ./ClientRequest is not exported from package @mswjs/interceptors (see exports field in @mswjs/interceptors/package.json
browserInterceptors preset: Cannot find module @mswjs/interceptors/presets/browser

Released: v0.22.10 🎉
This has been released in v0.22.10!

📄 Release notes
📦 npm package

Make sure to always update to the latest version (npm i @mswjs/interceptors@latest) to get the newest features and bug fixes.

Predictable release automation by @ossjs/release.


fix: error message typo

What kind of change does this PR introduce?
Minor Fix: Typo on error message.
What is the current behavior?
N/A
What is the new behavior?
N/A
Additional context
N/A

Thanks!


Fix Excel.Databar typo

The Excel.Databar object is defined with a lower B (not DataBar).

Learn Build status updates of commit c0ec223:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/Excel.Databar.md
:white_check_mark:Succeeded
View

api/Excel.FormatConditions.AddDatabar.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Thanks for all the fixes, @lucafrance! Cheers, Linda


TransactionRecord API changes

deprecated getTransfers() in favor of field transfers
(breaking-beta) renamed memo to transactionMemo

@mehcode moving Transfer out of the class is a breaking change, should I add a deprecation notice?

Yes. Just in case they happen to be using that class definition.


Add support for non utf-8 bytes

Hi, I'm using this library for AWS managed prometheus remote write API.
API requires encoded protobuf message body. But, this library supports only &str type.
I changed body type to &[u8] to supporting non utf-8 bytes.

@psnszsn Colud you review this PR?

@snpkx Thanks!


Review Options without #[serde(skip_serializing_if)]

There's at least one (here).

I fixed both cases in that file, any other occurrence doesn't really need this issue to be open to have someone fix it.


bug caused by .gitignore or 32-bit used on x64?

git-tfs version <IP_ADDRESS> (TFS client library <IP_ADDRESS> (MS)) (32-bit)
I'm using VS2015, does it correspond to TFS client library <IP_ADDRESS> (MS)?
Weird, my system is 64-bit. Why does it automatically find 32-bit git tfs?
How can I force git-tfs to use 64-bit version?

I have .gitignore with contents like the following:
*.js
!some_js_to_unignore.js
... and many others

Now I find that my git repo seems corrupted, in that some of the JS changes made by my colleague doesn't exist in my local git.
Command run:git tfs checkout 43594 -d
No authors file used.
git-tfs version <IP_ADDRESS> (TFS client library <IP_ADDRESS> (MS)) (32-bit)
Looking for changeset 43594 in git repository...
 => Commit found! hash: f87447395860016e0a2bf2972aaabeb8e741828d

And I can only see one file change in git, as follows
git show f8744 --name-only
commit f87447395860016e0a2bf2972aaabeb8e741828d (HEAD)
Author: ...
Date:   ...

    ....
    git-tfs-id: [https://xxx.com/DefaultCollection]$/AD Metrics Experimental/ux-separation-branch;C43594

Metrics.Server/App/pages/xorb/index.html

But actually it should have two file changes

My colleague's box is working fine. His env:
>git tfs version
git-tfs version <IP_ADDRESS> (TFS client library <IP_ADDRESS> (MS)) (64-bit)
C:\tools\gittfs\git-tfs.exe

Come on. Some one helps me! This issue caused my so much trouble.

I'm running git tfs verify, and it shows that some of the files differ.
$ git tfs verify
 Comparing TFS changeset 43620 to git commit 18a4b848f86a8199e02c86fbd716a16cd3becbd8
FacetedSearch\MultiFacetLuceneNet\MultiFacetLucene\MultiFacetLucene.csproj differs.
FacetedSearch\MultiFacetLuceneNet\MultiFacetLucene\MultiFacetLucene.csproj.vspscc differs.
Metrics.Server\App\services\dataservice.js differs.
Metrics.Server\Content\health3.css differs.
.........

Git-tfs is built to run in 32bit (but that should not be your problem).
It seems that you choose to ignore js files. What do you expect if they are ignored by git-tfs?
If that annoy you a lot, dig in the code and try to fix the bug.
Otherwise use 0.25 that don't have this behavior...

@pmiossec  Hi, thanks for your response!

I ignore most of the js files (auto-generated from typescript) using *.js in .gitignore, and un-ignore some legacy js files using !legacy-code.js. I think git-tfs shouldn't ignore those legacy-code.js, but it did somehow.
I switched to 0.25, but how can I recover those TFS checkins into my local git? git tfs pull or git tfs fetch do nothing.

My current ENV.
$ git tfs info

git version 2.15.1.windows.2

git-tfs version <IP_ADDRESS> (TFS client library <IP_ADDRESS> (MS)) (64-bit)
 C:\Richard\Tools\GitTfs-0.25.1\git-tfs.exe

$ git tfs checkout 43633 -n
9c3d96294603f1049ea75f05470c8147a8671310
// So, C43633 = 9c3d96294603f1049ea75f05470c8147a8671310
// Let's see what is in 9c3d9
$ git show 9c3d9 --name-only
commit 9c3d96294603f1049ea75f05470c8147a8671310
Author: xxxx
Date:   xxx

    Revert sanitization undo

    git-tfs-id: [https://admetrics.visualstudio.com/DefaultCollection]$/AD Metrics Experimental/ux-separation-branch;C43633

// Nothing here!!

This command is doing it:
https://github.com/git-tfs/git-tfs/blob/master/doc/commands/reset-remote.md
But due to optimization in git-tfs, be sure to reset the remote (with this command) and the local branch (to not see bad commits any more) before fetching again.

Hi,
have the same problem with .gitignore and reported it on issue #1121
My solution so far is to keep with version 0.25.1.

It seems that my problem is solved using @pmiossec 's way.

// Reset git (not sure this is needed, I did it anyway)
git checkout 337d32
Note: checking out '337d32'.

You are in 'detached HEAD' state. You can look around, make experimental
changes and commit them, and you can discard any commits you make in this
state without impacting any branches by performing another checkout.

If you want to create a new branch to retain commits you create, you may
do so (now or later) by using -b with the checkout command again. Example:

  git checkout -b <new-branch-name>

HEAD is now at 337d32d3... Add missed key to cscfg
// Re-pull the change sets
$ git tfs pull
Working with tfs remote: default => $/AD Metrics Experimental/ux-separation-branch
Fetching from TFS remote 'default'...
C43579 = eef5ae97a2be3c6894914c56ab456000f660baf5
....


Syntax error in SamplerV2 examples

URL to the relevant documentation
https://docs.quantum.ibm.com/api/qiskit/primitives#overview-of-samplerv2
Select all that apply

[X] typo
[X] code bug
[ ] out-of-date content
[ ] broken link
[ ] other

Describe the fix.
In the example code, print functions have redundant parentheses and run is missing closing parenthesis.
print(f"The primitive-job finished with result {job_result}"))

# run a sampler job on the parameterized circuits
job2 = sampler.run([(pqc, theta1), (pqc2, theta2)]
job_result = job2.result()
print(f"The primitive-job finished with result {job_result}"))

Looks like this was fixed in https://github.com/Qiskit/qiskit/pull/12587

Indeed, thanks for checking @abbycross!


ext-authz: fix to set the proper timeout for gRPC case

This match the same timeout we had set for HTTP.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[x] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[x] Does not have any changes that may affect Istio users.

@yangminzhu: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
155b15715f9d5c0b13b40914d10a1146e3b509e3
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

In response to a cherrypick label: new pull request created: #30800

so timeout is the minimum of cluster timeout and timeout here?
On Wed, Feb 10, 2021 at 7:04 PM Istio Automation<EMAIL_ADDRESS>wrote:

In response to a cherrypick label: new pull request created: #30800
https://github.com/istio/istio/pull/30800
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/30795#issuecomment-777176606, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXM4OZMJDJSAPBN753LS6NCLJANCNFSM4XN6QKUA
.

My understanding from experiment, It appears they control different
aspects. Cluster one is control connection timeout. This one is how long
client envoy can tolerate, for each outbound grpc request can be non
responding.
On Thu, Feb 11, 2021, 8:34 AM John Howard<EMAIL_ADDRESS>wrote:

so timeout is the minimum of cluster timeout and timeout here?
On Wed, Feb 10, 2021 at 7:04 PM Istio Automation<EMAIL_ADDRESS>
wrote:

In response to a cherrypick label: new pull request created: #30800
https://github.com/istio/istio/pull/30800
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/30795#issuecomment-777176606, or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAEYGXM4OZMJDJSAPBN753LS6NCLJANCNFSM4XN6QKUA
.

—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/30795#issuecomment-777624306, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMVXB7GN6SQW4NYKJN6QYDS6QBKBANCNFSM4XN6QKUA
.

so timeout is the minimum of cluster timeout and timeout here?
…
On Wed, Feb 10, 2021 at 7:04 PM Istio Automation @.***> wrote: In response to a cherrypick label: new pull request created: #30800 <#30800> — You are receiving this because your review was requested. Reply to this email directly, view it on GitHub <#30795 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEYGXM4OZMJDJSAPBN753LS6NCLJANCNFSM4XN6QKUA .

yes, but we will need to investigate more if that achieves the same effect, we may need to make the timeout` configurable if not.


Reconfigure the route generate for hapi

Instead of using predefined handlers, create the handlers in their specific form. This way we can allow for manual configuration of the routes. A possible example of this would look like this:
server.route({
    method: 'get',
    path: '/<model>/{id}',
    config: {
        handler: roadworkInstance.handlers.findOne,
        auth: 'roadwork-authentication'
    }
});

The routeGenerator has been rewritten now and will return the routeOptions instead of preregistering the routes themselves. The registration of the routes now happens in the Roadwork.js file. This might be changed in the future, but for now this will stay like this, therefor the issue stays open


Implement Path Transport Extension in Go

Implement the Path Transport Extension, as described in the book (section 15.1.4, p. 351)

The path transport extension is an end-to-end extension that allows communicating end hosts to pass or update a path. It is designed to transport a forwarding path (i.e., a path in the data-plane format, see Section 15.1.3). The transported path can be accompanied with a SCION address, to indicate that the end host can be reached via another address and path to support host mobility.  [...].
The path transport extension that conveys a path in the data-plane format (set of info (INF) and hop (HF) fields) is depicted in Figure 15.8.  The extension contains the field SrcType, which indicates what type of SCION address the path is prepended with.  A source can inform a destination that the source is reachable through another address via the SrcAddr field (ISD and AS identifiers concatenated with a host address). If necessary, padding is added so that the full path is aligned to an 8-byte boundary (Section 15.1.3 describes details of the data-plane path format).  The Path field is not mandatory and an empty path can be sent, and through such a construction a source can indicate a new SCION address

I no longer have interest in this. The Path Transport Extension as described above has various issues and it's probably best to look for different approaches where this could have been applicable.


Retry polling event queue always, with backoff

Fixes: #184
Along the way, add some infrastructure to help us better write tests for asynchronous code, and in particular to better use package:fake_async.

This now has some conflicts; would you mind resolving those?

Sure — done.

Merged. Thanks for the review!


Only one characteristic can notify at a time

Unless I am mistaken, the following code only allows one notification to be active at a time:
/// Notifies when the characteristic's value has changed.
  /// setNotification() should be run first to enable them on the peripheral
  Stream<List<int>> onValueChanged(BluetoothCharacteristic characteristic) {
    return FlutterBlue.instance._characteristicNotifiedChannel
        .receiveBroadcastStream()
        .map((buffer) => new protos.OnNotificationResponse.fromBuffer(buffer))
        .where((p) => p.remoteId == id.toString())
        .map((p) => new BluetoothCharacteristic.fromProto(p.characteristic))
        .where((c) => c.uuid == characteristic.uuid)
        .map((c) {
          characteristic.updateDescriptors(c.descriptors);
          characteristic.value = c.value;
          return c.value;
        });
  }

For many BLE devices it is necessary to listen to notifications from multiple characteristics at the same time. Not to mention from multiple devices!

This allows for multiple characteristics, since they are being filtered first by device ID and then by characteristic uuid.
Have you experienced an issue with this? If so, I can investigate.

It looks like .receiveBroadcastStream() will override the previous platform channel if it's called more than once.  Good catch!
I'll use this opportunity to refactor some event channels to utilize the main method channel (#11).
Fix on the way.

That was quick! I can confirm it works for me now. Thanks :)


Implement MAT-5640 MAT-5658

MADiE PR
Jira Ticket: MAT-5640
(Optional) Related Tickets:
Jira Ticket: MAT-5658
Summary
All Submissions

[x] This PR has the JIRA linked.
[x] Required tests are included
[x] No extemporaneous files are included (i.e Complied files or testing results)
[x] This PR is into the correct branch.
[x] All Documentation as needed for this PR is Complete (or noted in a TODO or other Ticket)
[x] Any breaking changes or failing automation are noted by placing a comment on this PR.

DevSecOps
If there is a question if this PR has a security or infrastructure impact, please contact the Security or DevOps engineer assigned to this project to discuss it further.

[ ] This PR has NO significant security impact (i.e Changing auth methods, Adding a new user type, Adding a required but vulnerable package)
[ ] All CDN/Web dependencies are hosted internally (i.e MADiE-Root Repo)

Reviewers
By Approving this PR you are attesting to the following:

Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
The tests appropriately test the new code, including edge cases
If you have any concerns they are brought up either to the developer assigned, security engineer, or leads

I am guessing its https://jira.cms.gov/browse/MAT-5648 rather than 5658


Add Supported platforms doc

In order to execute McThings we suppport:

Spigot + https://www.spigotmc.org/resources/raspberryjuice.22724/: Tested with 1.15.2
Forge (1.12.2) + https://github.com/arpruss/raspberryjammod
Mintest (5.1) + https://github.com/arpruss/raspberryjammod-minetest

This documentation is critical and must be added to the repository

Done!
https://github.com/juntosdesdecasa/mcthings/blob/develop/USAGE.md
This doc is critical for adoption so it is great to have it and we will need to maintain it.
Not entered in a lot of details because for newbies it will be nearly imposible to install all by themselves. We need to think about it. Probably the Minetest option is the easiest one.


Replace examples on home page

Currently, the home page example charts are not working. We need to replace them with new ones, ideally with static data, so they don't depend on live cubes.

Hi @ortnever, unfortunately the first two examples don't work ("Keine Daten") – could you take a look at this?
Or should we create the examples ourselves based on the data from these cubes?

I came up with these examples (tried to match the data shown on the chart with descriptions provided in the examples), but I'm not sure if they are fine with you.
I would appreciate it if you could guide me into a correct direction. Thanks!

Here the URLs for the examples on the homePage.
I admit that the graphs are not really nice but they make sense. We will replace them with more attractive graphics when we have more cubes published.
If there is a problem, just tell me.
https://test.visualize.admin.ch/en/v/CrTq9JmpiXt-
https://test.visualize.admin.ch/de/v/8MrC1M0sFrd1

Hi @ortnever,
thank you for providing new examples! The second one works, but unfortunately this is what is being displayed when I open the first link:

Could you take a look at this?
After creating a new chart based on this example and playing around with the filters, some data showed up, but I am not sure what was your vision for the chart there.

once again, the URLs for the first example.
https://test.visualize.admin.ch/en/v/fq-xjBXGpxgg
I don't know why the link created yesterday no longer shows the data today. Maybe it's because I created a new version? Strange. I'll try to do some tests about that.

with "created a new Version" I mean I published a new Version of the cube 1 hour ago. But normaly, the "old" URLs should work and display the new Version of the Cube..

Another example from Pierre for the homepage. Maybe instead of Bodenbelastung, so there are no two bar charts.
https://test.visualize.admin.ch/de/v/scBBn-4e4o3p


telemetry: include Azure CLI acquisition information in the user-agent

For us to know how people acquired CLI, like pip, bundle, curl? We need this information to help out with the decision that whether and when CLI can be Python3 only
//CC: @marstr

Additionally, it will allow our update message to be more useful when users run az --version.

Adding a little more context on how we've talked about doing this:
Many of our release channels create their own launcher script. We do this to accommodate different manners of acquiring Python. However, we can take advantage of this by setting an environment variable as we invoke the CLI.
Take our RPM spec for example:
https://github.com/Azure/azure-cli/blob/e241fd56c07a03445ea5d7db99a776696acfa3b3/scripts/release/rpm/azure-cli.spec#L69
would be come:
printf '#!/usr/bin/env bash\nAZ_INSTALLER=RPM %{cli_lib_dir}/bin/python -Esm azure.cli "$@"' > %{buildroot}%{_bindir}/az

In Homebrew, our launcher would go from:
https://github.com/Azure/azure-cli/blob/a3cad860fe0e61f003ff4f3efdf0c4d3b955ebc1/scripts/release/homebrew/docker/formula_template.txt#L39-L42
to:
    (bin/"az").write <<~EOS
      #!/usr/bin/env bash
      AZ_INSTALLER=brew #{libexec}/bin/python -m azure.cli \"$@\"
    EOS

We could then take the value of AZ_INSTALLER and append it to our user agent.

this information would be helpful for az feedback as well!

add to S164.


Replace linux_cross with add_architecture

As described in PR #19, this PR eliminates linux_cross (currently unused in reality) in favor of an add_architecture variable, better named to explain its purpose and implemented correctly.

Before going ahead with this, please let me check that the changes don't trigger unnecessary rebuilds of the i386 toolchain. Having implemented correctly linux_cross may have this side effect (in which case I must remove the just-replaced add_architecture from that target).

All fine. The rebuilds are avoided by various target replacements. I confirm that this PR doesn't change anything functionally. (Also, removing add_architecture would not have been a good fix in any case as it would have prevented from actually adding the i386 architecture in containers which do need it).

Thanks @NoisyCoil !

No problem! Rebasing the other MR on this one.


It's slow!

On Monday we will talk about: http://arborjs.org/docs/barnes-hut
and here: https://arxiv.org/pdf/astro-ph/0505010.pdf

https://docs.scipy.org/doc/scipy/reference/generated/scipy.spatial.KDTree.html

find your particles at r<r_0
build KDtree masses
use them for forces


Validate  or Normalize typo in contracts addresses for subgraph.yaml

See Code:
specVersion: 0.0.1
schema:
  file: ./schema.graphql
dataSources:
- kind: ethereum/contract
  name: DAOFactory
  source:
    address: " 0x595B34C93aA2c2ba0A38daEEdE629a0DfBDcc559" # Mainnet: 0x595B34C93aA2c2ba0A38daEEdE629a0DfBDcc559
    abi: DAOFactory

That extra space in the contract address is allowable, and the subgraph gets created, but then a weird error appears: Error deploying the subgraph: subgraph resolve error: parse error: Invalid character 'x' at position 2. This error message is really unclear so we should fix this.
We should normalize the contract address or else validate it ahead of time. Validating is probably good enough.

@davekaj Is this still a problem?

No, this is the error message now:
  Path: dataSources > 0 > source > address
  Contract address is invalid:  0xc0a47dFe034B400B47bDaD5FecDa2621de6c4d95
  Must be 40 hexadecimal characters, with an optional '0x' prefix.

I think this is sufficiently descriptive!


settimeevents should be sent on serverside

The ppobyter/events/timedevent.py class  and all classes that inherit from it should be replaced with normal events, and the scheduling of this should happen server side. This should be sent to all clients just like swarms, goldrushes etc.

world blessing & drop rate blessing should still be implemented, but that requires further decoding of ppo's communication protocol.


Updates to the README.md

Should include wording on Scylla Cloud connectivity (can be added later)
Need to use the same license as upstream (Apache Cassandra)

Should include wording on Scylla Cloud connectivity (can be added later)
Need to use the same license as upstream (Apache Cassandra)

Added license information

LGTM


Drop 'old' LLVM versions

This should enable us to switch to C++17 as suggested by @gebart in the offline (slack) discussion.
cc @stanislaw

Nice that you were able to remove some of the abstractions that were no longer needed :+1:

Did you pick version 6.0 as the lowest version because that is what is available in Ubuntu 18.04 LTS and Debian stable repos?
From a user perspective I think it makes sense to support what is easily installable in the most common "stable" distribution channels (Ubuntu, Debian, Homebrew, maybe Centos)

@gebart that's the first one that compiled with -std=c++17, at least that was the case with libirm.


Efficent way to delete element

Hi,
i was trying to create a function called deleteOlderThanOneMinute() that will delete the element that have a value older than a minute but i don't figure it out a correct and fast way to handle this task, can you tell me how can i do it? Following the code that i actually use to test your library.
import (
	"fmt"
	"time"
	csmap "github.com/mhmtszr/concurrent-swiss-map"
)

var m = csmap.Createstring, int64,
	csmap.WithSizestring, int64,
)

func main() {
	hash := "hash1"
	timestamp := time.Now().Unix()
	m.Store(hash, timestamp)
	hash2 := "hash2"
	timestampOld2Minute := time.Now().Add(-2 * time.Minute).Unix()
	m.Store(hash2, timestampOld2Minute)
	hash3 := "hash3"
	timestampOld3Minute := time.Now().Add(-3 * time.Minute).Unix()
	m.Store(hash3, timestampOld3Minute)
	count := m.Count()
	println("count:", count)
	m.Range(func(key string, value int64) (stop bool) {
		fmt.Printf("Key: %s, Value: %d\n", key, value)
		return false // continue
	})
	deleteOlderThanOneMinute()
}

Hi,
Thanks for the contribution. time.AfterFunc can be used here but this feature is out of the context of concurrent-swiss-map. You may fork and create your own cache library or wrap the library.


Huge issue with time-only DateTimeField and DST

Hi,
I have a huge issue with time-only DateTimeField in a productive site running Orchard 1.9.x.
The issue is that time-only fields are always stored as UTC (converted from CurrenetTimeZone at moment of saving/publishing a content item) time but rendered as local time (at the moment of displaying the content item) and this depends if DST is active for moment of creation and/or displaying the field.
First for clarification:
Central European Summer Time (CEST) = Central European Daylight Saving Time (CEDT= CET DST). CEDT = UTC + 2.
Central Summer Time (CEST) = Central European Daylight Saving Time (CEDT= CET DST). CET = UTC + 1.
Example:
At my Event content type, I have one date-only field for the date and one time-only field for the time. I took this approach since it should be possible for an event to have no time.
Regard the following 4 cases all dealing with creation of a time for a CET date - they differ only in creation date (whether CET or CEDT) and display date (whether CET or CEDT)
Case 1: On November 2015 (CET) create an event with a date for May 2016 (CEDT). Set the time-only field to 10:00 a.m. Since creation is while CET it is stored as 09:00 a.m. (UTC=CET-1).
Case 1a: Look at the item while CET, e. g. January 2016: time is displayed correctly, since CET = UTC+1 ==> 10:00 a.m.
Case 1b: Look at the item while CEDT, e. g. May 2016: time is displayed incorrectly, since CEDT = UTC+2 ==> 11:00 a.m.
Case 2: On April 2016 (CEDT) create an event with a date for May 2016 (CEDT). Set the time-only field to 10:00 a.m. Since creation is while CEDT it is stored as 08:00 a.m. (UTC=CEDT-2).
Case 2a: Look at the item while CEDT, e. g. May 2016: time is displayed correctly, since CEDT = UTC+2 ==> 10:00 a.m.
Case 2b: Look at the item while CET, e. g. January 2017: time is displayed incorrectly, since CET = UTC+1 ==> 09:00 a.m.
I took a look at the implementation in DefaultDateLocalizationService:ConvertFromLocalizedString and DateTimeFieldDriver:Display/Editor.
I think to fix the issue we need to add "options.EnableTimeZoneConversion = false;" for time-only fields in the DateTimeFieldDriver:Display and in the DateTimeFieldDriver:Editor methods.
Totally there are two occurences where I applied added here disabling the time zone conversion:
if (settings.Display == DateTimeFieldDisplays.TimeOnly) {
options.EnableCalendarConversion = false;
options.EnableTimeZoneConversion = false;
options.IgnoreDate = true
}
Of course with this fix all existing time-only field values stored in DB are also incorrect and need to be adapted.
Please give me feedback if this is a suitable fix or if there is any better way to fix it.
Thanks,
Dieter

/cc @DanielStolt Haven't looked in the detailed yet, assume you might be interested too.

It's complicated, took several reads, but I think maybe I understand the issue now. :)
A time-only field has no concept of a date, so how do we decide if the time is DST or not? I guess the approach I took when implementing this was to base it on the current date, at the time of storage/retrieval.
I don't agree that the correct solution is to remove time zone conversion. We should, however, remove DST conversion since the concept does not even apply if there is no date.
I'm wondering if perhaps one solution would be to simply make it deterministic, i.e. when storing/retrieving a time-only value, always base the conversion on a fixed date, such as DateTime.MinValue.Date. This would ensure that, whenever you save the field and whenever you render it, the result would always be the same with respect to DST, and the time zone conversion would still work as expected.
What do you think of that @dpomt?

Interestingly enough, I was just looking at this field for an unrelated issue, and discovered that it was storing different information based on the settings (Date only) which I think is wrong. DateTime should only store a date time, and display the time only if required, which would solve this issue too. Or if we care about only a Time that is invariant, then a new field is necessary. So I think we should update the field to store a full date and time. The migrations would be transparent as today's date would be taken into account during next load/save.

@sebastienros I see problems with your suggestion. Which date do we store? Next time you save the content item, a different date would be stored, and then depending on DST differences between different time zones you would still see inconsistencies.
Do you see a problem with my suggestion?

I see the issue with my suggestion, yes I agree with your comment, but it still means to read the settings in the field then.
I really think the ultimate solution should be to have a DateTime field and a Time field (which would be time zone insensitive, just two ints)
Let's do with your suggestion though, this is the best non-breaking change we can do right now.

@DanielStolt  I agree using a fixed date would be a solution and is better than removing time Zone conversion in general.
@sebastienros  I also agree that it would be even better to have a separate time field.
For my special usage having two DateTime fields for an event begin date/time, another Option would be to have a Checkbox in DateTime Editor to allow the user to flag it as an all day event. Right now, it is not possible to not specifiy the time portion within DateTime picker. If you leave it empty now it is stored as 00:00:00 UTC and then gets converted to e. g. UTC+2 when re-displayed in CEDT.
I am still hoping that we get rid of DST here in Germany ;) There are discussions on that.

@sebastienros and @dpomt OK, assigning this to myself for implementation of the fixed date fix. As for the new separate "static time of day" field, that's better targeted at dev and perhaps should get its own separate issue to track.

https://github.com/OrchardCMS/Orchard/issues/6781

Did you think about providing a migration step for fixing probably incorrect values?
I did not find a suitable way to do this. For my productive site I corrected the value depending if a content item what created in DST or not. But there are IMO more cases than that (depending whether a content item was just created or duplicated, changed later or even if it was created prior to 1.8.x etc.). I ended up in sending mails to all persons that created events to check and adapt their event times.

Should it be on the dev miletone, it seems the commit is on dev.

@sebastienros It was committed on 1.10.x and merged to dev.


Installation issue

Problem encountered on https://dotnet.microsoft.com/learn/dotnet/hello-world-tutorial/install
Operating System: Linux Ubuntu 18.10 - x64
I have run all the commands given in instruction still the system is showing following error.
@ubuntu:~$ sudo apt-get install dotnet-sdk-2.2
Reading package lists... Done
Building dependency tree
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies:
dotnet-sdk-2.2 : Depends: aspnetcore-runtime-2.2 (>= 2.2.4) but it is not going to be installed
Depends: dotnet-runtime-2.2 (>= 2.2.4) but it is not going to be installed
E: Unable to correct problems, you have held broken packages.

I don't know what the problem is but can you update the Ubuntu?

Can you run these diagnosis steps and share the results?
# Shows the repositories you have configured. Source: https://askubuntu.com/a/741948
grep -r --include '*.list' '^deb ' /etc/apt/sources.list /etc/apt/sources.list.d/
# Show your distro version.
cat /etc/os-release
# This install command will show exactly which dependency can't be met.
sudo apt-get install dotnet-runtime-deps-2.2

This normally happens when the repository you're using doesn't match the distro or specific distro release.

Hi @dagood. I have updated the system a few days back. That may be the problem. I am sharing the result of the commands you have given to me.
suraj@ubuntu:~/Coding/codingit$ grep -r --include '*.list' '^deb ' /etc/apt/sources.list /etc/apt/sources.list.d/
/etc/apt/sources.list:deb http://es-mirrors.evowise.com/ubuntu/ disco main restricted
/etc/apt/sources.list:deb http://es-mirrors.evowise.com/ubuntu/ disco-updates main restricted
/etc/apt/sources.list:deb http://es-mirrors.evowise.com/ubuntu/ disco universe
/etc/apt/sources.list:deb http://es-mirrors.evowise.com/ubuntu/ disco-updates universe
/etc/apt/sources.list:deb http://es-mirrors.evowise.com/ubuntu/ disco multiverse
/etc/apt/sources.list:deb http://es-mirrors.evowise.com/ubuntu/ disco-updates multiverse
/etc/apt/sources.list:deb http://es-mirrors.evowise.com/ubuntu/ disco-backports main restricted universe multiverse
/etc/apt/sources.list:deb http://archive.canonical.com/ubuntu disco partner
/etc/apt/sources.list:deb http://es-mirrors.evowise.com/ubuntu/ disco-security main restricted
/etc/apt/sources.list:deb http://es-mirrors.evowise.com/ubuntu/ disco-security universe
/etc/apt/sources.list:deb http://es-mirrors.evowise.com/ubuntu/ disco-security multiverse
/etc/apt/sources.list.d/google-chrome.list:deb [arch=amd64] http://dl.google.com/linux/chrome/deb/ stable main
/etc/apt/sources.list.d/microsoft-prod.list:deb [arch=amd64] https://packages.microsoft.com/ubuntu/18.10/prod cosmic main
/etc/apt/sources.list.d/rednotebook-ubuntu-stable-disco.list:deb http://ppa.launchpad.net/rednotebook/stable/ubuntu disco main
suraj@ubuntu:~/Coding/codingit$ cat /etc/os-release
NAME="Ubuntu"
VERSION="19.04 (Disco Dingo)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 19.04"
VERSION_ID="19.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=disco
UBUNTU_CODENAME=disco
suraj@ubuntu:~/Coding/codingit$ sudo apt-get install dotnet-runtime-deps-2.2
[sudo] password for suraj:
Reading package lists... Done
Building dependency tree
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies:
dotnet-runtime-deps-2.2 : Depends: libssl1.0.0 but it is not installable
E: Unable to correct problems, you have held broken packages.
suraj@ubuntu:~/Coding/codingit$

Thanks @surajwate! That is indeed the problem. You're on Ubuntu 19.04, but have the 18.10 Microsoft repository installed:
/etc/apt/sources.list.d/microsoft-prod.list:deb [arch=amd64]
  https://packages.microsoft.com/ubuntu/18.10/prod cosmic main
...
VERSION="19.04 (Disco Dingo)"

Running the 19.04 instructions should upgrade your Microsoft repository and work:
wget -q https://packages.microsoft.com/config/ubuntu/18.10/packages-microsoft-prod.deb
sudo dpkg -i packages-microsoft-prod.deb

sudo apt-get update
sudo apt-get install dotnet-sdk-2.2

(I filed https://github.com/dotnet/core/issues/2728 to track getting instructions for 19.04 on dot.net.)
The mismatched repository vs. distro causes a problem because some Ubuntu packages are available in 18.10 but not 19.04, and dotnet-runtime-deps-2.2 has to match. There are plans to make the dotnet-runtime-deps packages work across multiple versions but we haven't got that into a 2.1/2.2 release yet. (https://github.com/dotnet/core-setup/issues/5812)

@dagood thanks for the help.

@dagood  is there an eta when (dotnet/core-setup#5812) will be released?

It's merged for the next servicing updates, 2.1.12 (https://github.com/dotnet/core-setup/pull/6246) and 2.2.6 (https://github.com/dotnet/core-setup/pull/6247). The current plan has those releasing in July (although it may change).

I guess I could just build the dotnetcore-runtime-deps myself If I want
them sooner or would I then have to build everything myself?
Davis Goodin<EMAIL_ADDRESS>schrieb am Di., 28. Mai 2019, 17:03:

It's merged for the next servicing updates, 2.1.12 (dotnet/core-setup#6246
https://github.com/dotnet/core-setup/pull/6246) and 2.2.6 (
dotnet/core-setup#6247 https://github.com/dotnet/core-setup/pull/6247).
The current plan has those releasing in July (although it may change).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dotnet/core/issues/2707?email_source=notifications&email_token=AEMNB7NMCNXWBZ6D255YCK3PXVCTPA5CNFSM4HMISBM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWMNK7A#issuecomment-496555388,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEMNB7P22IARY7GPIFHKLY3PXVCTPANCNFSM4HMISBMQ
.

Since the dotnet-runtime-deps package doesn't contain anything (it's only used for the dependencies it declares), building it yourself should work fine.
Depending on your use case, it might be cleaner to make sure you have the dependencies installed yourself and ignore dotnet-runtime-deps. Or, it would arguably be the cleanest to just install from the tarball for now.

Which tarball so you mean? Could you please elaborate?
Davis Goodin<EMAIL_ADDRESS>schrieb am Di., 28. Mai 2019, 18:20:

Since the dotnet-runtime-deps package doesn't contain anything (it's only
used for the dependencies it declares), building it yourself should work
fine.
Depending on your use case, it might be cleaner to make sure you have the
dependencies installed yourself and ignore dotnet-runtime-deps. Or, it
would arguably be the cleanest to just install from the tarball for now.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dotnet/core/issues/2707?email_source=notifications&email_token=AEMNB7PUSKBXSBSBVKQDXBDPXVLVFA5CNFSM4HMISBM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWMVBVY#issuecomment-496586967,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEMNB7PVU33ZWGNRNJU5OPLPXVLVFANCNFSM4HMISBMQ
.

On https://dotnet.microsoft.com/download you'll see a table a bit lower on the page--I'm referring to the "Binaries" row. There's some instructions to follow after you click a download link, or you can also take a look at https://github.com/dotnet/core/blob/master/Documentation/linux-setup.md.
You can also check out if Snaps suit you!

Thanks I will try it. 👍
Davis Goodin<EMAIL_ADDRESS>schrieb am Di., 28. Mai 2019, 23:27:

On https://dotnet.microsoft.com/download you'll see a table a bit lower
on the page--I'm referring to the "Binaries" row. There's some instructions
to follow after you click a download link, or you can also take a look at
https://github.com/dotnet/core/blob/master/Documentation/linux-setup.md.
You can also check out if Snaps
https://github.com/dotnet/core/blob/master/Documentation/linux-setup.md#installation-using-snap
suit you!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dotnet/core/issues/2707?email_source=notifications&email_token=AEMNB7MYEMHBNPMR3OALV4LPXWPS3A5CNFSM4HMISBM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWNPWOY#issuecomment-496696123,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEMNB7JJJKJKHH4LFYQLN63PXWPS3ANCNFSM4HMISBMQ
.

Tried installing dotnet via the tarball and it just worked. Just FYI I am
running PureOS(Debian 10(Testing)based distro). 👌 Thanks for pointing me
in the right direction.
Torben Clasen<EMAIL_ADDRESS>schrieb am Mi., 29. Mai 2019, 08:25:

Thanks I will try it. 👍
Davis Goodin<EMAIL_ADDRESS>schrieb am Di., 28. Mai 2019,
23:27:

On https://dotnet.microsoft.com/download you'll see a table a bit lower
on the page--I'm referring to the "Binaries" row. There's some instructions
to follow after you click a download link, or you can also take a look at
https://github.com/dotnet/core/blob/master/Documentation/linux-setup.md.
You can also check out if Snaps
https://github.com/dotnet/core/blob/master/Documentation/linux-setup.md#installation-using-snap
suit you!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dotnet/core/issues/2707?email_source=notifications&email_token=AEMNB7MYEMHBNPMR3OALV4LPXWPS3A5CNFSM4HMISBM2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWNPWOY#issuecomment-496696123,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEMNB7JJJKJKHH4LFYQLN63PXWPS3ANCNFSM4HMISBMQ
.