Indels in HapMap file

Hi,
does indel coded as "--" are intentionally not supported? I have HapMap file, which work properly with TASSEL, however after running MVP.Data() on it, indels are coded same as alternative allele. For exaple line:
[1] "S01_17126" "A/-"       "01"        "17126"     "+"         NA          NA          NA          NA          NA          NA          "--"         [13] "AA"        "--"        "AA"        "--"        "AA"        "--"        "AA"        "AA"        "--"        "--"        "AA"        "AA"         [25] "--"        "AA"        "AA"        "AA"        "AA"        "--"        "AA"        "AA"        "AA"        "AA"        "AA"        "AA"         [37] "--"        "AA"        "--"        "--"        "AA"        "--"        "--"        "--"        "--"        "--"        "--"        "AA"         [49] "--"        "--"        "AA"        "--"        "AA"        "AA"        "AA"        "--"        "AA"        "AA"        "AA"        "AA"         [61] "AA"        "--"        "--"        "--"        "AA"        "--"        "AA"        "--"        "AA"        "--"        "AA"        "AA"         [73] "--"        "--"        "--"        "--"        "--"        "AA"        "AA"        "--"        "--"        "--"        "--"        "--"         [85] "--"        "--"        "--"        "--"        "AA"        "--"        "--"        "AA"        "--"        "--"        "AA"        "AA"         [97] "--"        "--"        "AA"        "AA"        "AA"        "--"        "--"        "--"        "AA"        "AA"        "AA"        "AA"        [109] "AA"        "AA"        "AA"        "AA"        "AA"        "--"        "AA"        "--"        "AA"        "--"        "AA"        "AA"        [121] "--"        "AA"        "AA"        "AA"        "AA"        "AA"        "--"        "AA"        "AA"        "AA"        "--"        "AA"        [133] "AA"        "--"        "AA"        "AA"        "AA"        "AA"        "AA"        "AA"        "AA"        "AA"        "--"        "--"        [145] "--"        "AA"        "AA"        "AA"        "AA"        "AA"        "--"        "AA"        "AA"        "--"        "AA"        "AA"        [157] "--"        "--"        "--"        "AA"        "--"        "--"        "AA"        "AA"        "AA"        "--"        "AA"        "AA"        [169] "AA"        "--"        "--"        "--"        "AA"        "AA"        "AA"        "AA"        "AA"        "--"        "--"        "AA"        [181] "AA"        "AA"        "AA"        "--"        "--"        "AA"        "--"        "AA"        "--"        "--"        "AA"        "AA"        [193] "--"        "AA"        "AA"        "AA"        "--"        "AA"        "--"        "AA"        "AA"        "AA"        "AA"        "--"        [205] "AA"        "--"        "--"        "--"        "AA"        "--"        "AA"        "--"        "--"        "--"        "--"        "AA"        [217] "--"        "A-"        "--"        "AA"        "AA"        "AA"        "AA"        "AA"        "AA"        "--"        "--"        "--"        [229] "--"        "AA"        "--"        "--"        "AA"        "--"        "--"        "--"        "--"        "AA"        "--"        "--"        [241] "AA"        "--"        "AA"        "AA"        "AA"        "AA"        "--"        "AA"        "--"        "--"        "--"        "-A"        [253] "AA"        "AA"        "-A"        "--"        "AA"        "--"        "AA"        "AA"        "AA"        "--"        "AA"        "AA"        [265] "AA"        "AA"        "--"        "AA"        "--"        "AA"        "AA"        "AA"        "--"        "--"        "--"        "AA"        [277] "--"        "AA"        "AA"        "--"        "--"        "--"        "AA"        "--"        "--"        "--"        "--"        "--"        [289] "AA"        "AA"        "AA"        "AA"        "AA"        "AA"        "--"        "AA"        "AA"        "--"        "AA"        "AA"        [301] "AA"        "--"        "AA"        "AA"        "--"        "AA"        "AA"        "AA"        "--"        "--"        "AA"        "--"        [313] "AA"        "AA"        "--"        "AA"        "--"        "AA"        "--"        "--"        "AA"        "AA"        "AA"        "AA"        [325] "--"        "--"        "AA"        "--"        "--"        "--"        "--"        "--"        "AA"        "--"        "--"        "AA"        [337] "--"        "--"        "--"        "--"        "--"        "--"        "AA"        "--"        "AA"        "AA"        "--"        "--"        [349] "AA"        "--"        "AA"        "--"        "AA"        "AA"        "--"        "AA"        "AA"        "AA"        "AA"        "AA"        [361] "--"        "AA"        "AA"        "--"        "--"        "AA"        "--"        "AA"        "AA"
in big.matrix object created with MVP.Data looks like this:
[1] 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0  [76] 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 [151] 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 [226] 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 [301] 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0
In map file alleles are properly coded:
S01_17126     1 17126   A   -
Rest SNPs looks fine. I have used most recent version rMVP installed from github.
Best,
Marcin

Sorry, it’s hard to believe that I haven’t noticed this issue for such a long time.
Your guess is correct. The data conversion format module of rMVP does not consider InDel. For Hapmap Diploid format, any non-ATGC character will be treated as Missing.

This issue is closed because it is no longer active.


adding functionality of "contact us" and "about"

I would like to enhance the functionality of "contact us" and "about" section
please assign to me

Ok Sir I work on it
On Wed, Oct 2, 2024, 5:13 PM ANSHIKA-26 @.***> wrote:

Assigned #68 https://github.com/ANSHIKA-26/WordWise/issues/68 to
@vararaghu002 https://github.com/vararaghu002.
—
Reply to this email directly, view it on GitHub
https://github.com/ANSHIKA-26/WordWise/issues/68#event-14487384551, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A45ZWGZGAY6I6RDGSBKEUEDZZPL4PAVCNFSM6AAAAABPHPUVJOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJUGQ4DOMZYGQ2TKMI
.
You are receiving this because you were assigned.Message ID:
@.***>

kindly assign the label as gssoc ext under hactoberfest and level


The 1000sats and 1inch keys are not accessible in type script

You can access like obj["1000sats"]


scripts: zephyr_module: Add URL, PURL and version to SPDX

Improve the SPDX with the current values:

URL: extracted from git remote. If more than one remote, URL is not set.
Version: extracted from git rev-parse (commit id).
PURL: generated from URL and Version.

For zephyr, the tag is extracted, if present, and replace the commit id for the version field.
Since official modules does not have tags, tags are not yet extracted for modules.
This PR follows this issue and PR #66182 and #66495.

@mbolivar-ampere I don't understand why the pipeline fails. I've updated files containing zephyr_modules.py call with the --zephyr-base parameters (which is now required), including twister.yaml.
Could you help me ?
Thanks

If you think --zephyr-base parameter should not be required`, let me know (it would solve the pipeline issue).
Since there is this line of code, it seems that it should be provided each time:
https://github.com/zephyrproject-rtos/zephyr/blob/9569e4410475fb62125e769a13222882902d8c75/scripts/zephyr_module.py#L587-L589

--zephyr-base parameters in zephyr_module.py should be required but Github actions seems to use external twtister.yaml (and not the one from my branch) for the pipeline which do not provide --zephyr-base parameter.
I added the --zephyr-base arguments everywhere, so it can be marked as required in zephyr_module.py when twister.yaml from this PR is merged (and used in CI).

Forget all I said about --zephyr-base, it might be more complicated that I thought and this should not be in this PR, so I removed this modification from the PR.
Sorry about this.

PR was splited but everything is now present in the same PR.
Closed in favor of PR https://github.com/zephyrproject-rtos/zephyr/pull/66495


chore: Upstream ByteSlice from mathlib

This does not upstream String.toAsciiByteArray, which belongs elsewhere with other ASCII utilities.

This seems limited compared to a would-be ByteSubarray. Maybe rename it to ByteArray.Iterator by analogy with String.Iterator? In any case, it's not clear this is needed but there should be an alternative such as ByteSubarray.

I created an alternative #851 that is closer to core's Subarray in the hope to advance the discussion.

Closing in favour of #851.


priority signal bug

I think I have come across a bug in the priority signal. I did a series of unit tests to narrow it down and this one failed to fill 5 slots.
[Test]
public function positive_to_negative_priority_values():void
{
_signal.addWithPriority(testFunction_1, 1);
Assert.assertEquals(_signal.numListeners, 1);
_signal.addWithPriority(testFunction_2, 0);
Assert.assertEquals(_signal.numListeners, 2);

_signal.addWithPriority(testFunction_3, -1);
Assert.assertEquals(_signal.numListeners, 3);

_signal.addWithPriority(testFunction_4, 0);
Assert.assertEquals(_signal.numListeners, 4);

_signal.addWithPriority(testFunction_5, 1);
Assert.assertEquals(_signal.numListeners, 5);

}
if you use 1, 0 then -1 as signal priorities in that order in a proper test case it should be reproducible.
I love signals btw. building a game with them :)

I can't replicate this, this my diff (sorry it's not clean, I pulled from the upstream at the same time)
https://github.com/stickupkid/as3-signals/commit/b6c19b396aee01885ed9a685414dd4362398df7c

I did some more digging, I was using the 0.9-beta so I was missing the following change:
https://github.com/robertpenner/as3-signals/issues/56
the test passes after that.

Can you close this issue. Cheers


Upgrade Guava and don't use deprecated methods like Objects#ToStringHelper

Our Guava dependency is pretty old right now. When we bring in Iceberg as a dependency, we hit NoSuchMethodErrors when calling the Objects#toStringHelper method, which was removed in later versions of Guava. It's been replaced by MoreObjects#toStringHelper.
There's two matters to consider here:

Should we bump Guava? We're using 23.6.1: https://github.com/palantir/incubator-iceberg/pull/8
Let's minimize our use of Guava's deprecated APIs. We can start with toStringHelper and perhaps add linting rules to find others.

I am +1 on this. @rdblue what do you think?

I definitely agree about minimizing the use of deprecated APIs that we know will be a problem for people that upgrade.
My concern about upgrading the version we use is that it could cause problems downstream. Do we expect everyone to shade guava? Also, now that we are using Caffeine, should we consider moving off of Guava?
I think we mainly use Guava for 4 things:

Caches - replaced by Caffeine
Iterables and Iterators
Preconditions - easily replaced
Maps, Lists, etc. for new and immutable containers

If we can find a good alternative for iterables/iterators that hopefully doesn't require too much of the streams API, maybe we should consider using that. We could also build alternatives of those for Iceberg; we already have a fair number of them in CloseableIterable.

My concern about upgrading the version we use is that it could cause problems downstream

Could we shade our Guava dependency in our artifacts?  We'd have to make sure that guava is bundled in our artifacts and shaded and is not fetched transitively, whereas the rest of the dependencies can be fetched transitively.  Shadow plugins allows us to do this. This could be some work and could change how our artifacts are fetched . Users would have to rely on the different classifier or configuration for the artifacts for example.

We could do something similar to what Avro and Parquet do -- build a shaded Jar and rewrite everything to depend on that. I'd rather not do that if we can cleanly move off of Guava, though. Does anyone agree with the idea of trying to move off?

Do you mean building alternatives ourselves?

For preconditions and iterators/iterables, we would probably build alternatives. A lot of those are already implemented in CloseableIterables. Preconditions are easy to add, too.
The main annoyance is the container types, including immutable variants. Is there an alternative for those?

To be clear I think it's appropriate to use Guava's APIs that are marked as stable, not deprecated, and not experimental / beta. Guava should be committed to API stability and thus it shouldn't be a problem if the Guava dependency is swapped from under the hood.
The issue is that we were using deprecated methods that were removed.
I wonder if we can add a linting check for this.

After a little searching, I don't see good alternatives for the immutable collections. I think we should move forward with upgrading Guava and not using deprecated methods.

+1.
I'll get a PR going for this (the intellij support for iceberg is buggy for this project. It picks up an older version of guava and doesn't find symbol for toStringHelper)

Thank you @SurenNihalani!


zip.AddFile undefined when running "go get github.com/claudiodangelis/qr-filetransfer"

I'm opening this issue because:

[X] I have found a bug
[ ] I want to request a feature
[ ] I have a question
[ ] Other

I am trying to install the binary with go get github.com/claudiodangelis/qr-filetransfer, and getting the following error:
/Users/issmirnov/go/github.com/claudiodangelis/qr-filetransfer/content.go:54:7: zip.AddFile undefined (type *archivex.ZipFile has no field or method AddFile)

It is coming from here.
I have reproduced this on two systems: ubuntu and OSX.
OSX:
$ go version
go version go1.10.3 darwin/amd64

$ echo $GOPATH
/Users/issmirnov/go

$ sw_vers
ProductName:	Mac OS X
ProductVersion:	10.13.2
BuildVersion:	17C88

Ubuntu:
$ go version
go version go1.9.4 linux/amd64

$ echo $GOPATH
/Users/issmirnov/go

$ uname -r
Linux mysecrethostname 4.4.0-116-generic #140-Ubuntu SMP Mon Feb 12 21:23:04 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

$ $ cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=16.04
DISTRIB_CODENAME=xenial
DISTRIB_DESCRIPTION="Ubuntu 16.04.5 LT

I believe the cause of this bug is this commit: https://github.com/jhoonb/archivex/commit/1eb6e91aa0b4b9b1623fdf3a2a38142b4afe4b63
It removes the AddFile method, and replaces it with Add.

Thanks for reporting @issmirnov, I will look at it ASAP

Having the same issue with the current version:
$ go get github.com/claudiodangelis/qrcp
github.com/claudiodangelis/qrcp/util
dev/go/src/github.com/claudiodangelis/qrcp/util/util.go:38:7: zip.AddFile undefined (type *archivex.ZipFile has no field or method AddFile)

Hi @vizzah, sorry to hear, which version of go are you using?

Same issue here "go version go1.14.2 darwin/amd64"

Hi @vizzah, sorry to hear, which version of go are you using?

go version go1.12.8 linux/amd64
thanks

Same issue on: go version go1.14.2 linux/amd64
Thank you!

Same issue on go version go1.14.2 android/arm64

I get this error too with go version go1.13 linux/amd64 .  However, if I clone the repo and do go build, it builds successfully, and qrcp seems to work.

Should be all good now, the issue was the one pointed out by @issmirnov, which I totally forgot about.
Please let me know if everything works as expected, and thanks everyone for taking the time to report the problem.

Awesome works like a charm now


viz.py no result

After making the same 2.7 os.makedirs() change to viz.py as in #2,
% python viz.py --load 0.677756249905 --same
/Users/priot/anaconda/lib/python2.7/site-packages/matplotlib/font_manager.py:273: UserWarning: Matplotlib is building the font cache using fc-list. This may take a moment.
warnings.warn('Matplotlib is building the font cache using fc-list. This may take a moment.')
%
And no obvious action.
ls -l saved_models/6_8_128_cpu/0.677756249905
-rw-r--r--  1 priot  staff  433348 Aug 29 19:21 saved_models/6_8_128_cpu/0.677756249905
OSX Darwin/Macbook Pro

Viz.py dumps images to the visualization folder. I'll add this to the readme.

Even better have it print something when it quits.


[Bug] Long poll requests timing out after 20 seconds

Long poll requests definitely last longer than this under load so something is capping the metric, guessing something related to Trend/Time metrics in k6 -> prometheus.
Seems that the SDK is timing out GetWorkflowExecutionHistory requests after 20 seconds. I think it should be 65, so not sure what's going on here.

Upon further investigation it seems that this is a timeout being set on the requests by the SDK rather than a metric issue. I've not yet been able to find where this is coming from.

The 20 second timeout is expected behaviour for get history requests. The stalls in performance I saw were due to low task poller counts which meant the workers would often timeout waiting for empty task queue partitions.


GraphDB Acid Benchmark Implementation

Thanks! Is this ready to merge?

Yes, this is ready to merge.


Install .NET 6 on runners

Anticipating the upgrade to .NET6 , ensure that it is available on runners.

I've tested this change on pulumi-tls. Full upgrade instructions in https://github.com/pulumi/home/issues/2531

This is a preparatory change it should be no-op for existing code. It makes sure .NET 6.0 is available in the GHA environment. Once this is applied, providers may continue upgrading to newer pulumi/pkg that will actually update provider .NET SDK dependency to .NET 6.0.

Thanks for the quick review @lblackstone


Support for stream keyword with new-expr in incremental parser

Description:
Only the object type descriptor is supported along with the new-expr. Need to support stream type descriptor as well.
Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

@dulvinw is this issue still valid?
Can you give a sample code?

I think this is already fixed when I look at https://ballerina.io/learn/by-example/streams.html.
For example  var oddNumberStream = new stream<int, error>(oddGen);


Keep only pool APY without wallet connected different than with

Without a wallet being connected the pool APY that is displayed is different than with a wallet connected.
It doesn't matter which wallet is connected, in all cases it goes from ~5% to ~10%.
Without:

With

Hi! thanks for reporting an issue!
I was not able to recreate this bug locally and on mainnet. Could you please double-check with the current dashboard at https://dashboard.keep.network/liquidity?

Hi RC,
I believe it was fixed in #2425. At least I'm not able to recreate it myself anymore as well.

Perfect! Thanks!
Closing this issue since we are not able to recreate described bug- fixed in #2425.


Update to Storage Page

I found that the Storage page was not auto updating the local and session variables on the page on keyup as expected. I've updated the page to work with this now.
This is my first pull request ever Remy, so if I've made a balls of it, reject it and I'll try and redo it.
I added a function to force the content to update in those two onpage variables. This gets called when the "keyup" eventListener is triggered.

Second request adds comments and tidy's up changes to .htaccess to make it work locally for me.


If/How to support extra supporting data besides metrics

E.g. trace id for a specific quantile.

Created from https://github.com/RichiH/OpenMetrics/issues/22

More specifically, annotations or non-identifying information about a specific data point or range of data points.


Feature/php tidy

I tried to tidy up the PHP a bit here in line with spec.
I'm still a little unsure as to why the key string or what I see as the event namespace was so hardcoded, i'm not an expert so perhaps you could clarify. I've changed the structure and readme in-line with how I believe it could work better.
Tabs to 4 spaces
Constants to class variables
Block comments
Emit arguments passed inline rather than stripped out of the args

It should in essence not change the way the class already works, just tidies it up. Let me know what you think...

Thanks for the PR!
I like moving the constants and doc comments, but I'm not very fond of tab spacing or newline before {, for example.
Could you maybe suggest these changes in smaller pieces?

@rase- This would be awesome for your library. I know that following the PSR standards can be weird at first, but you will get the hang of it. :+1: for the PR.

src/binary.php file names should use StudlyCase, therefore src/Binary.php.


quassel-daemon unnecessary qt5-base dependency

Issue description
quassel-daemon depends on qt5-base, which pulls in wayland, libX11, gtk and all sorts of stuff that I definitely don't want to have on my headless server. ArchLinux solves this by including qt5-base only as a runtime dependency for the quassel client packages:
https://git.archlinux.org/svntogit/community.git/tree/trunk/PKGBUILD?h=packages/quassel
Steps to reproduce
Enable the quassel service
services.quassel.enable = true;
Technical details

system: "x86_64-linux"
host os: Linux 4.14.79, NixOS, 18.09.1763.fbb7dbdb95d (Jellyfish)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.1.3
channels(root): "nixos-18.09.1763.fbb7dbdb95d"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Nevermind, qt5-base is an indirect dependency of the quassel-core package on ArchLinux. Still, it doesn't pull in quite as much unnecessary things.


feat(styles, documentation): styles update text input

Waiting for #2978 to be merged because the icon in the validation message gets added in that PR and the icon inside the input field gets deleted in this one.

@davidritter-dotcom, blocking PR is merged.


Use bech32 in bns addresses

Closes #452

Any idea what this Travis fail means? Looks related to the change
1. Integration tests with bov+tendermint : returns empty list when getting an unused account
   Error: Invalid checksum for<PHONE_NUMBER>02030303040404050505050a0a0a0a
       at Object.decode /home/travis/build/iov-one/iov-core/node_modules/bech32/index.js:102:24
       at Function.decode /home/travis/build/iov-one/iov-core/packages/iov-encoding/src/encoding.ts:173:35
       at Object.decodeBnsAddress /home/travis/build/iov-one/iov-core/packages/iov-bns/src/util.ts:25:17
       at BnsConnection.<anonymous> /home/travis/build/iov-one/iov-core/packages/iov-bns/src/bnsconnection.ts:161:36

Re-opened to trigger a fresh CI build after Github issues yesterday


Fixed sox recording

Fixed arguments for sox recording. Sox is now default record program. Added option "audioType", that allows receive stream with any supported audio type.

Hey Guys please merge this PR the package is totally stopped.

@gillesdemey I believe this might be an issue we've been bitten by too, bother you for a patch. I'm happy to help pitch in, in whatever way I can :smile:

Will be fixed in https://github.com/gillesdemey/node-record-lpcm16/pull/39

@gillesdemey :+1: thank you for the hard work.


Importing database from original amazon-echo-ha-bridge?

This is a question... not a bug report:
I was happily entering all of my devices into the configurator in the original HUE emulator that this project is forked from, and it was working well until Echo stopped being able to import devices (I don't know if I reached some limit on the number of devices, but it can no longer find any devices, even though I can see the log entry showing it's initiating the request).  While I was hunting for a solution I came across this much fuller-featured version of the app, but I'm wondering if there's a simple way to preserve all of the work I did entering the device info into the original app.  I've got roughly 50 devices and am not eager to have to re-enter them all manually.  I captured the data I entered via the api/devices page, so I do have a text file with all of my devices listed.  I'm interfacing with openHAB, so I'm using the manual add tab to get my devices into this emulator. Thanks.

So, it is very easy. The capture you did from the api/devices call is the base format of the device.db file. Save that data to a file without any extra returns at the ens and put that file in a direcory and point the bridve ar for the device db. Th bridge will load it.

Well, it wasn't quite as simple as a copy and paste, but I did finally get it to import the old database.  I had to remove ,"httpVerb":null,"contentType":null,"contentBody":null from each record before the new ha-bridge would recognize the file as valid.
HOWEVER, I'm still running into problems with Alexa telling me it hasn't found any connected devices.  Through some trial and error, it appears Echo cannot deal with more than 47 devices in the HUE bridge emulator.  If I stick to 47 it imports fine into Echo, but if there are >47 devices in the emulator, Echo fails to import any of them.  Has anyone else run into this?  I tried changing my 48th device to a different one just in case that was the problem, but it didn't make a difference.

Apparently I'm not the first to run into a limit, but for some reason the limit I'm hitting is higher than what others reported.  See: https://github.com/armzilla/amazon-echo-ha-bridge/issues/12 for some additional info on this.  It's not clear to me that the limit is in Echo... it could be something in the bridge emulator.
Is there a way to disable to UPNP listener in your version of the ha-bridge?  It appears I will need to run two instances of the bridge to get around the limitation in Echo, and I'm guessing only one instance at a time should be listening for Echo to query the database?  According to jlippold at the above URL, you don't want to leave it listening all the time anyway because Echo will end up refreshing the list without your permission.

It is a limitation with the echo as it has occurred with smartthings and wink. It seems it is based on the connection to a specific hub instance.
The Multiple bridge solution works great and many use this method to get past the limitiation. You just have to change 2 settings, the upnp response port and the web port for each bridge you run. UPNP is already shareable on a given device.

This seems like a great feature to have built-in: automatically enumerate the devices across different UPnP ports.
How difficult do you think this would this be to support?

I can confirm it is a limit of the Echo, when IoT connections were first available on the Echo it had a 12 (or so) device limit, that Amazon was well aware of, and 'fixed' with code. Going by others who have hit the discovery limit you are pretty fortunate, many others have found the limit in the high 20's.  I too use the bridge with mostly short custom URLs to interface with my X10 equipment (along with a Harmony Hub configuration) and have not yet hit a limit with 43 configured devices.  Most who have reported discovery limits are using Vera device configurations which include more config data, Crush (@bwssystems) and I have discussed what we think is behind this seemingly random limit.  He is way more familiar with the discovery process, and has stated that the Echo only uses the device IDs for discovery so the config of the devices does not come into play, but I can't help but wonder why those with setups like ours have higher limits than those with Vera configured devices.

Since I'm going to have to run 2 instances of the ha-bridge anyway, I guess I should split the devices evenly between the two so I have less risk of some future problem that spontaneously lowers the (apparently high) limit I've found through testing.  It would be nice if Alexa would actually tell you that you've exceeded the limit instead of acting like she didn't even find the bridge.
What about the issue of turning off UPNP after the initial setup?  Is this not something that I should be worried about, or is there already built-in support for this in ha-bridge?  I didn't see it on the Bridge Control page.  Is the command line option for this still embedded in ha-bridge like it was the the project ha-bridge was forked from?

One more question... I noticed that the device IDs from the original emulator are significantly longer than the device IDs in this version of ha-bridge.  Since I'm going to have to manually manipulate the database to split it in two, I can easily edit the device IDs as well.  Is there any reason to have long numbers or can I safely use something simple like 1,2,3,4,... ?  Put differently, do the IDs have to be unique only within the list of ha-bridge entries, or do they need to be unique in the entire universe of devices that Echo knows about?

As for turning off UPnP after the initial setup, what if we mimic the way that the Hue bridge does this? Alexa instructs you to press the button on the Hue bridge to enable discovery of the devices.
Would it be better to have a button on the web UI that allows the software to broadcast the devices for 60-90 seconds, or whatever amount of time that the Hue bridge does? Maybe something configurable to enable/disable manual broadcasting, and to set the time limit might be helpful.

Yet another question... When I built my initial database using the original bridge emulator, it assigned a deviceType of "switch" to each device.  I noticed that ha-bridge assigns a "custom" deviceType if I build the record on the Manual Add page.  How/where is this deviceType used?  I'm sending all of my bridge output to openHAB, and so far it seems to turn my lights on and off with either deviceType, but I'm wondering if there's a reason I should have them set one way vs the other.  FWIW, most of my physical devices are Dimmers, with a few Switches mixed in.

So, the first question you had was, why not enumerate the devices over the ports if given. Unfortunately, I use a framework, sparkjava, to implement the rest api's. It only allows me to have one configured web port per process. To emulate multiple bridges, it needs to be a different URL as the echo recognizes this as I tried. So that is out.
There is no need to turn off UPNP in any process. They all reply as individual devices that the echo sees.
As far as the ID's, You could assign any number you want. The random generator I use starts around 9 digits so you could go less than that.
The deviceType is a carry over from the original bridge and I use other methods to detect what to do.

Is this all set now?

I have split my devices into two groups and I set up two instances of the emulator.  Echo is able to discover all of my devices now, and things seem to work pretty well, but I noticed that the iPad Alexa app shows many of my devices as "Offline".  I have tried "forgetting" all devices via the iPad app and re-discovering them. After some period of time (less than an hour I believe), all of the devices from one or the other instance of ha-bridge will grey out and display "Offline" (all devices from the other instance of ha-bridge continue to show as online).  I can still use these Offline devices from Echo, and after using a device it will revert back to an Online status in the Alexa app for a while, but after some additional period of time it goes back to Offline again.  Only the device(s) I actually use will revert to Online... the rest in that group still show Offline.  It doesn't seem to affect my ability to control devices, but I find it a little disconcerting that Echo seems to think all of the devices in one of my ha-bridge groups is Offline.
I don't know if this is related, but I have also noticed that sometimes the "discovery" process will only pick up one of the two bridge's devices.  I eventually got all devices from both instances to be discovered, but I don't remember if I had to stop the ha-bridge process and restart it to get discovery to work.  I've had this happen several times as I was experimenting, so it's not a one-off problem.
So I think things are running well even if I have a strange Offline status problem.  I really appreciate the effort you put into creating this streamlined app.  Right now I'm running it on my OS X machine, but I plan to migrate to a Raspberry Pi at some point, so the lightweight app will be a big plus there.

I forgot to mention one other thing... Since I had to edit the database to get it to import, I started by separating each device onto its own line in the text file.  I did this to make it easier to read the file and perform the edits.  I fully expected I would have to put it all back on one line to get it to import into ha-bridge, but I was pleasantly surprised to find that ha-bridge is able to read the file with the line breaks in it.  If you use the GUI in ha-bridge to make any database changes, it gets written back out as a single line, but for those users who might find it easier to maintain their list of devices using a text editor, this is something they might like to know about.

Glad it is all working for you. The Offline/Online status is something others have seen as well, but they still work. Also, the discovery process without all devices I have encountered. These things are all in the echo as far as I can tell.
Thanks for the tip on the  db files. That will be something we add to the help we are creating.
Closing as this is resolved.


Support for stdin?

Hey there, thanks for working on this project. I'm curious if you'd be open to support stdin, similar to how flake8 does it? (i.e. support - as the file name and a flag to identify the filename for error messages -- --stdin-display-name)
I can send a PR if that's acceptable!

Yeah that sounds reasonable to me!


VHDL: Add private, view keywords; Distinguish attribute from keyword

I'm not familiar with the language, but based on the file history, I presume that any changes to the language after 2015 are not reflected. There's been at least one new version since then (VHDL-2019, some highlights summarised on another site), and brings at least two new keywords: private and view. I'd confirm with the language reference manual but it costs like $500 and it's not quite worth the cost for what I'll use it for :P
I've also separated the attributes from the keywords, and used a generic pattern instead of a name list for the attributes (similar to Ada).

The coverage test failing isn't your fault.
That test uses a few different strategies to determine whether a pattern has been covered. The most basic strategy is to simply check that a regex produced some token. However, it uses a different strategy for keyword lists. It requires all keyword in the list to procure a token.
The test now fails because you simplified the keyword enough for the coverage test to recognize it as a keyword list, and now it wants you to test all of them.
I'm sorry about that, but could you please add the missing keywords to the test?

but it costs like $500

lol. Yeah, that's not quite worth it.

I've also separated the attributes from the keywords, and used a generic pattern instead of a name list for the attributes

Sounds good.

Ooh that's interesting! I'll update the tests shortly. My apologies for not running all the tests locally!

Ah hmm... I did not realise that library and use are also contained the in keyword list, I thought they were only considered constants.
I'll update the keyword test, but I'm not sure whether to remove constant, or allow library and use to be assigned both constant and keyword, as they are both standard tokens. I'm leaning towards removing constant, but I wonder why those two are constants as well. I might have to look for whatever VHDL docs I can find about it.

@RunDevelopment I see! Thanks for searching that up :)
I ran the test on my laptop, it timed out the first time, but all of it passed after plugging my laptop in.

Thank you!

I ran the test on my laptop, it timed out the first time

Do you remember which one timed out?

Ah no, not really. I just tried to run it again after unplugging it, but it hasn't failed again yet.
I tried to google some of the words I remember seeing in the error, this question on StackOverflow seems to match it.
As for the file, I believe it was coverage.js, which makes sense because I only ran npm run regex-coverage.

A timeout on coverage? Huh, that's strange. The test just runs all languages tests, so it doesn't do any heavy computations. Anyway, I'll keep that in mind for the future.


README is slightly incomplete

There seems to be a step missing from the README. Enabling workload identity on an existing cluster (here) misses a step mentioned in the Google docs here.
tldr; you need to also enable workload-metadata on the nodepools, only updating the cluster is insufficient.
It'd be great to add this step to the docs since it had me stumped for a couple of days.

I had exactly issue today and did a troubleshooting for 2-3 hours to get it resolved. I was about to open this issue but found someone had raise this.


build without docker

(This came from discussion at Edinburgh Docker meetup)
Need to have the ability to build in a container (without Docker socket access). Example is in AWS Codebuild pipeline.
Currently Docker is needed for some output formats, more work needs doing on the linuxkit based outputs but these do work with qemu.
Secondly it is needed to extract image filesystems. We should switch to containerd for these; either using image store, or filesystem as now, or filesystem layers that get mounted later.

Ongoing work on this in linuxkit


fix: use a slice directly in the extended_commitment_factory

The patch based on https://github.com/tari-project/bulletproofs-plus/pull/12 to eliminate values copying.

Hi @DenisKolodin please point cargo.toml to tag = "v0.0.3" ?


Run builds on every PR change

Description
This to allow enabling branch protection

I do not think

Description
This to allow enabling branch protection

I do not think we need to run build on every PR to be comfortable with branch protection for admins. Indeed, if we are in a bind, an admin can always go to settings, allows admins to bypass branch protection, fix things.
That being said, running build on every PR may be a good idea in its own right. I am in favor of it.

Description
This to allow enabling branch protection

I do not think we need to run build on every PR to be comfortable with branch protection for admins. Indeed, if we are in a bind, an admin can always go to settings, allows admins to bypass branch protection, fix things.
That being said, running build on every PR may be a good idea in its own right. I am in favor of it.

I misunderstood the description. This is only to run the existing PR workflow even if we only modify readme files, to allow merging PRs because of the check.

Since we require a Test to pass prior to merging for all contributions (including administrators), this change is necessary.


Initial work supporting generic EventGridEvent

Fixes #11.

@drub0y,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.
It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot


Does video quality of "video only"+audio have difference with ALREADY merged video

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[ ] I've searched the bugtracker for similar questions including closed ones

Question

YouTube-dl provides multiple formats of videos.
For instance, when I download 720p video, I could select format 22(already merged), but I could select 136 and merge it with other audio formats such as 140.
I want to compare the amount of "LOSS" of videos.

Already merged format: -f 22
Video only format: -f 136
post-merged video format(Video only + audio): -f 136+140

If there are any data or ways to check this, please let me know.

Obviously you won't get files byte to byte equal but they will likely be somewhat close.

@dstftw
Do you mean they are almost same so it is meaningless to talk about the difference of loss amount?


pipenv: add back wrapper

Turns out that when creating a virtualenv, it ignores what has been injected
into sys.path with the magical sed expression. Also, NIX_PYTHONPATH is
not used.
Likely a subprocess is invoked in which case both are no longer visible.
That's the reason PYTHONPATH needs to be used.

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
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 nix-review --run "nix-review wip"
[ ] 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
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @kaali

I tested this and now we get ModuleNotFoundError: No module named 'pip._internal.main' error again (related to #71178). But it does fix the missing virtualenv in #71771

Let me guess, you are using nix-shell?

Let me guess, you are using nix-shell?

No. I'm installing pipenv in my system configuration.
To repeat you can try this Pipfile:
[[source]]
name = "pypi"
url = "https://pypi.org/simple"
verify_ssl = true

[packages]
spacy = "~=2.1"

[requires]
python_version = "3.7"

One without any dependencies works fine.

Closing in favor of https://github.com/NixOS/nixpkgs/pull/71800.


Simplify parallel processing to solve some intermittent failures

Implements our own forking to avoid intermittent Parallel::DeadWorker issues
Better visibility of errors surfaced by parallel processes

Use https://github.com/github/octocatalog-diff/pull/111/files?w=1

Fixes https://github.com/github/octocatalog-diff/issues/104


Update to align with user-supplied storage keys

You haven't aligned the document with the new user-supplied storage keys, still says "coming soon"
Existing data you say "However, data that were already present will not be encrypted" but the screen shot says it will be encrypted via a background process, which is correct?

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

ID: e7e05a67-3121-bbc2-7a46-637199104ea3
Version Independent ID: 2802344c-0571-bdac-4755-580f75f404b0
Content: Azure Storage Service Encryption for Data at Rest | Microsoft Docs
Content Source: articles/storage/common/storage-service-encryption.md
Service: storage
GitHub Login: @lakasa
Microsoft Alias: lakasa

@phatcher
Thanks for the feedback! We are currently investigating and will update you shortly.

@phatcher I have assigned the issue to the author to investigate and update as appropriate.

@phatcher - We fixed this back in March. My apologies that we didn't respond to you before now.
I made one more minor clarification in the article. If you see any further issues, please let us know.
#please-close

@phatcher We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Make system actor names consistent

Some are prefixed with $, some other are not.
Let's stick with the $ prefix and use for all system actors
$invalidator
activator
partition-activator
$proxy-activator
pubsub-delivery
partition-activator
$proxy-activator
gossip
pubsub-delivery
server_seed

Done


timer and pot not showing

after playing about with it for the last 2hrs i got the site to work this far
http://prntscr.com/95b8ux
but trying to get the pot to show with the timer to
all the chat works fine and saves the steam trade urls so as far as i can work out its not database side of things
any idea's ?

Do you have the bot running?

yes bot is running tho cant trade with it for 7 days due to only signed in with it for the 1st time on my  dedicated server

Ehm for me everything is loading tho... pot starts when someone goes in and timer when another one joins the pot.

oh so its when someone bets the pot comes up i didnt no that XD sorry guys
i fort it was like some where it loads up with out anyone bettings


Allow to "merge" captured scripts

Current behavior
Currently, the tag helper renders captured script blocks exactly as they were captured.
For example, if we capture the following scripts:
<script capture='Foo'>
 console.log('Foo 1');
</script>
<script capture='Foo'>
 console.log('Foo 2');
</script>
<script capture='Foo' src='some good CDN' integrity='sha256-bla' crossorigin='anonymous'></script>

they will be rendered into 3 script blocks as the were captured.
Proposed change
Allow script blocks to be marked with merge attribute.
Here is the same example with a merge attribute:
<script capture='Foo' merge>
 console.log('Foo 1');
</script>
<script capture='Foo' merge>
 console.log('Foo 2');
</script>
<script capture='Foo' src='some good CDN' integrity='sha256-bla' crossorigin='anonymous'></script>

Note that first two script blocks have merge attribute and the thrid does not. The render output for this should be like this:
<script>
 console.log('Foo 1');
 console.log('Foo 2');
</script>
<script src='some good CDN' integrity='sha256-bla' crossorigin='anonymous'></script>

Additionaly, add auto-merge='true' attribute to the render method, to instruct the render tag helper to merge script blocks that contain JS code and don't merge script references.
For example:
<script capture='Foo'>
 console.log('Foo 1');
</script>
<script capture='Foo'>
 console.log('Foo 2');
</script>
<script capture='Foo' src='some good CDN' integrity='sha256-bla' crossorigin='anonymous'></script>

...
<script render='Foo' auto-merge='true'></script>

that will produce the same output as if the first two blocks were marked as "mergable".
<script>
 console.log('Foo 1');
 console.log('Foo 2');
</script>
<script src='some good CDN' integrity='sha256-bla' crossorigin='anonymous'></script>

Done in version 0.3.0


[Docs]: update snacks in Fundamentals

Added missing snacks for examples in Fundamentals docs.

Thanks a ton for this contribution!
I'm extremely thankful for these changes and proud of these great lines of code!
I strongly encourage you for further development and can't wait for the next great changes! Do not give up doing a good job!
😍


Confirm Volunteer Involvement for ITD Launch and Document Access

Description:
With the ITD program launching this Friday, we need to confirm which volunteers will be involved across all participating regions. It would also be valuableto confirm each volunteer’s role and location for the launch, ensuring the information is accurate and readily accessible for reference during and after the event. This will also help streamline any last-minute communication needs.
Acceptance Criteria:

[ ] Volunteer Confirmation: Confirm the list of volunteers actively participating in the ITD launch, along with their assigned regions and roles (e.g., mentors, workshop facilitators, code reviewers).
[ ] Centralised Document: Ensure this information is recorded in a central, easily accessible document

Could classplanner help with this? @CodeYourFuture/class-planner bringing this use case to your attention

@MarianaVazquezCYF are there any updates on this?

Mariana is using this document, Amym while Class Planner is not there yet. https://docs.google.com/spreadsheets/d/1xGDPty4h3G4UQjse2cmsS_g7WXDiJTqu5XeOQ1AG-RQ/edit?gid=906027144#gid=906027144

Hey yes, I had a meeting with class planner team and they will make some changes to help!
We have almost all the teams, we will have them ready for next week!
Question: which document are you talking about? Centralised Document: Ensure this information is recorded in a central, easily accessible document?  this is the one I am using: https://docs.google.com/spreadsheets/d/1xGDPty4h3G4UQjse2cmsS_g7WXDiJTqu5XeOQ1AG-RQ/edit?gid=148920607#gid=148920607 you will see in person and online community leaders there!


[LOWPRI][FUTURE]Support empty variant type, and empty record type.

@objmagic added these: https://twitter.com/objmagic/status/927026404331479041
We have one issue with syntactic real estate for empty records. Currently we use {} to represent empty Object, but I don't really think it's used that frequently - we merely needed {} to parse to something so that when people are writing dummy function bodies, it doesn't error. I think {} would be a better candidate for empty record, now that it's supported.
It will take some time for these empty variant types/record types to work their way into the OCaml ecosystem so support for this is low pri right now, but it's good to be aware of it.

A working branch is here. It still needs some polish and I need to send it to experienced maintainer for a preview. Below is an example
For the following program:
type o = A
type t = | ;;
let f (x:t) = match x with _ -> . ;;
type m = { };;
let x = { }
type n = m
let y:n = x
module X = struct type g = { }  end
let g = { }
let n = let open X in { }
let h = n

The signature will be:
type o = A
type t = |
val f : t -> 'a
type m = { }
val x : m
type n = m
val y : n
module X : sig type g = { } end
val g : m
val n : X.g
val h : X.g

I'm still confused about how this is useful in practice. Care to TL;DR?

Here is an example of why empty variants could be useful.
For empty record, as Yallop mentioned in the above Mantis ticket, compiling empty Protobuf enum could be one example.
I admit that the real-world use cases are pretty rare. So the main purpose of these additions are to make language more consistent.

I'm using an empty type to mark constructors in a variant as not-yet-implemented. When I'm ready to implement the constructor I change it from Foo of empty to Foo of int (say), and the compiler will give me a bunch of errors to guide me in implementing the Foo-constructor. At least that's the idea. Not sure if this is a good approach or not.

It seems empty record is unlikely to be allowed. Jacques showed his arguments here: https://github.com/ocaml/ocaml/pull/1536#issuecomment-353221867

Empty variants are implemented in #2487


[azure-*-cpp] Update to April Releases

Update vcpkg ports for Azure SDK release. This release may contain multiple ports.

Adding azure-core to release
cc: @antkmsft

Adding azure-core-amqp to release
cc: @antkmsft

Adding azure-data-tables to release
cc: @gearama

@WangWeiLin-MV, thank you for the suggestion, but please let us not have that - we have automation that handles generation of this file (we've had similar conversation in the PR with one of your team members about a year ago).
In our case, the REF there does not always have to have the "azure-...-${VERSION}" format (in our daily pipelines, it is just a SHA, for example).

@WangWeiLin-MV, I am not seeing info:reviewed tag, so I'm just checking, if you think we are good to go?
If there are more questions, or suggestions, or if more time is needed - that's ok, I'll be happy to respond especially if anything is needed from us, but if not, then ideally it would be great we are ready when there is the next slot for PR merges, and our port updates are picked up along with the other PRs.

@antkmsft Sorry for the long wait, I have approved and added the tag.

@WangWeiLin-MV No problem. Thank you!

Wow, thank you so much @WangWeiLin-MV for the deep dive, analysis, and your PRs https://github.com/Azure/azure-sdk-for-cpp/pull/5514 and https://github.com/Azure/azure-sdk-for-cpp/pull/5516! Really over the top, above and beyond!
I will test them both (for https://github.com/Azure/azure-sdk-for-cpp/pull/5516, we have a dummy package that we can run through the release automation and "release" it to see if everything is working).
I will test them both today, and will update this PR later in the day. If https://github.com/Azure/azure-sdk-for-cpp/pull/5516 works as well, I will that the ${VERSION} change as well - it looks like we'll need to re-run x-add-version for this PR anyway, and since our build scripts will also be updated, I will take that another change as well.
With dependencies, great catch.
Thanks again!

@WangWeiLin-MV, I have applied your feedback to this PR.


[question] Flower power

If I understood well, flowers are supposed to spawn only if they are less than 4 in a 9x9x9 cube
@Gael-de-Sailly can probably confirm that.
But strangely, it seems that it is not always applied... I planted only 4 flowers, and here is the result:

On the surface you can see ~40 flowers for a 25x20 area. Is there anything wrong, or am I paranoid?
the goal is to know if I can enhance my flower factory with more than 3 ones. The best would be to be able to produce every sort of them.

I think it's like "démineur", a flower can have only 8 other flowers in a 9x9 field.
If it's calculated like this, the flowers you see are possible.
Need someone to confirm that :)

Do not understand "9 blocks far", This 9x9x9 cube is from (x-4, y-4, z-4) to (x+4, y+4, z+4)

ok, then I can make new plans for my factory. thanks :)

A good answer is given, i closed the issue.


(typescript) Required prop can be undefined when props is declared outside defineComponent

Version
3.0.5
Reproduction link
https://codesandbox.io/s/loving-frost-l1gox?file=/src/index.ts
Steps to reproduce
follow the link

What is expected?
props.requiredValue shouldn't be inferred as undefined.
What is actually happening?
Opposite

When declaring it outside you need as const:
const props = {
  requiredValue: {
    type: Array as PropType<string[]>,
    required: true
  }
} as const;

When declaring it outside you need as const:
const props = {
  requiredValue: {
    type: Array as PropType<string[]>,
    required: true
  }
} as const;

thanks!

thanks!

Hi, sorry to comment this closed issue but I can't find a way to send props to a custom defineComponent  function, e.g. :
import { defineComponent } from 'vue'
import Parent from './Parent'

export const props = {
  msg: {
    type: String,
    required: true
  }
} as const

function customComponent(props: {}) {
  return defineComponent({
    extends: Parent,
    props
  })
}

export default customComponent(props)

Is there any way to do that easily without using a custom DefineComponent type (I tried but without success for the moment) ?

Well, it was not so difficult (thanks to pikax on discord) :
function customComponent<P extends Readonly<ComponentPropsOptions> = {}>(
  props: P
) {
  return defineComponent({
    extends: Parent,
    props
  })
}


Enable Sparse matrix functions/structures

After NamedArrays is added, DataGraphs.jl will probably need multiple dispatch to manage sparse matrices

In lieu of NamedArrays.jl, I could consider using OrderedDicts. I may need to decide between OrderedDicts and NamedArrays


Some suggestions...

Hi,
I would like to suggest following improvements...

Remove unnecessary using statements

In Setting.cs

        public object GetValue(Type type)
        {
            if (type == null)
                throw new ArgumentNullException("type");

            if (type.IsArray)
                throw new InvalidOperationException("To obtain an array value, use GetValueArray() instead of GetValue().");

            if (this.IsArray)
                throw new InvalidOperationException("The setting represents an array. Use GetValueArray() to obtain its value.");

            return **type.IsArray** ?
                GetValueArray(type.GetElementType()) :
                CreateObjectFromString(mRawValue, type);
        }

It is not required to check this condition.

In Setting.cs, following method looks similar to above method. So can it be refactored?

        public T GetValue<T>()
        {
            var type = typeof(T);

            if (type.IsArray)
                throw new InvalidOperationException("To obtain an array value, use GetValueArray() instead of GetValue().");

            if (this.IsArray)
                throw new InvalidOperationException("The setting represents an array. Use GetValueArray() to obtain its value.");

            return (T)CreateObjectFromString(mRawValue, type);
        }

In Setting.cs, private int CalculateArraySize() method, following are the issues...

add check; arrayEndIdx >= arrayStartIdx (true)
            // Naive algorithm; assume the number of array element delimiters
            // equals the number of elements + 1.
            for (int i = 0; i < mRawValue.Length; ++i)
                if (mRawValue[i] == Configuration.ArrayElementSeparator)
                    ++arraySize;

i = arrayStartIdx; i < arrayEndIdx; ++i

Configuration.ArrayElementSeparator should not be space char

            if (arraySize == 0)
            {
                // There were no element separators in the array expression.
                // That does not mean that there are no elements.
                // Check if there is at least something.
                // If so, that is the single element of the array.
                for (int i = arrayStartIdx + 1; i < arrayEndIdx; ++i)
                {
                    if (mRawValue[i] != ' ')
                    {
                        ++arraySize;
                        break;
                    }
                }
            }
            else **if (arraySize > 0)**
            {
                // If there were any element separators in the array expression,
                // we have to increment the array size, as we assumed
                // that the number of element separators equaled the number of elements + 1.
                ++arraySize;
            }

this check is not required.

In ConfigurationReader.cs...

In private static Setting ParseSetting(string line, int lineNumber) method...
           string settingValue = line.Substring(indexOfAssignOp + 1, line.Length - indexOfAssignOp - 1);
line.Substring(indexOfAssignOp + 1) will do
and similar corrections required in some other place too...
Thanks

Correction...
type.IsArray
Bold marking within code blocks is not shown correctly.

Correction...
In Setting.cs, private int CalculateArraySize() method, following are the issues...
add check; arrayEndIdx > arrayStartIdx (true)

Hey there, sorry for the late reply.

Seems fair.

This check is actually necessary, because if the user calls GetValue(typeof(int[])), he expects an int[] to be returned by that call. But when calling GetValue(typeof(int)), he expects a non-array value, not an array value with a single element.

Same reason as in 2. Also, it cannot be refactored, because C# does not allow function distinction by return type. See the signature of GetValue:    T GetValue<T>()
We can't have that and T[] GetValue<T>()
How should the compiler know which one to choose when calling (for example) GetValue<int>()?
If such a thing as T[] GetValue<T[]> was possible in C#, this wouldn't be a problem. But C# unfortunately doesn't allow that. So we have to have GetValue and GetValueArray.

I don't understand what you mean, sorry. What would the fully corrected code look like?

That is correct, thanks!

Cheers

Hi,
In Setting.cs
        public object GetValue(Type type)
        {
            if (type == null)
                throw new ArgumentNullException("type");

            if (type.IsArray)
                throw new InvalidOperationException("To obtain an array value, use GetValueArray() instead of GetValue().");

            if (this.IsArray)
                throw new InvalidOperationException("The setting represents an array. Use GetValueArray() to obtain its value.");

            return type.IsArray ?
                GetValueArray(type.GetElementType()) :
                CreateObjectFromString(mRawValue, type);
        }

type.IsArray check is already performed, hence it is not required tobe checked in return statement
It should be...
        return CreateObjectFromString(mRawValue, type);

Corrected code...
        private int CalculateArraySize()
        {
            if (string.IsNullOrEmpty(mRawValue))
                return -1;

            int arrayStartIdx = mRawValue.IndexOf('{');
            int arrayEndIdx = mRawValue.LastIndexOf('}');

            if (arrayStartIdx < 0 || arrayEndIdx < 0)
                return -1; // Not an array.

            // Added
            if (arrayEndIdx < arrayStartIdx)
                return -1;

            // There may only be spaces between the beginning
            // of the string and the first left bracket.
            for (int i = 0; i < arrayStartIdx; ++i)
                if (mRawValue[i] != ' ')
                    return -1;

            // Also, there may only be spaces between the last
            // right brace and the end of the string.
            for (int i = arrayEndIdx + 1; i < mRawValue.Length; ++i)
                if (mRawValue[i] != ' ')
                    return -1;

            int arraySize = 0;

            // Naive algorithm; assume the number of array element delimiters
            // equals the number of elements + 1.

            // Added
            for (int i = arrayStartIdx + 1; i < arrayEndIdx; ++i)
                //for (int i = 0; i < mRawValue.Length; ++i)
                if (mRawValue[i] == Configuration.ArrayElementSeparator)
                    ++arraySize;

            if (arraySize == 0)
            {
                // There were no element separators in the array expression.
                // That does not mean that there are no elements.
                // Check if there is at least something.
                // If so, that is the single element of the array.
                for (int i = arrayStartIdx + 1; i < arrayEndIdx; ++i)
                {
                    if (mRawValue[i] != ' ')
                    {
                        ++arraySize;
                        break;
                    }
                }
            }
            else /*if (arraySize > 0)*/ // Added
            {
                // If there were any element separators in the array expression,
                // we have to increment the array size, as we assumed
                // that the number of element separators equaled the number of elements + 1.
                ++arraySize;
            }

            return arraySize;
        }

I have marked changes with // Added tag
also, Configuration.ArrayElementSeparator should not be  space char

Hey,
you are right, the check in GetValue() is not needed.
However, the check in CalculateArraySize() is not needed, because arraySize won't be negative, only >= 0.
CalculateArraySize() will get a new implemented soon anyway, which will also take inner structures into account. Example: { {"Test1", 10}, {"Test2", 20} } will be a valid array with 2 elements.
So for now, the current CalculateArraySize() implementation is good enough.


[docs] ROCm 6.0 supported OSs in documentation don't match the repository

Suggestion Description
RHEL 8.7 is not supported anymore at least when I look into the repository (https://repo.radeon.com/amdgpu/6.0/rhel) but is listed as version to install (https://rocm.docs.amd.com/projects/install-on-linux/en/latest/how-to/native-install/rhel.html).
Also the github link (http://github.com/ROCm/rocm-install-on-linux) is a 404 not found.
Operating System
No response
GPU
No response
ROCm Component
documentation

The link (http://github.com/ROCm/rocm-install-on-linux) is still broken.


feat: dms serverless replication

what
Something @Gowiem was previously working on, DMS Serverless replication, which is the same as the migration replication task but doesn't need an instance because it's serverless.
Thought it would be a good addition here with the other resources.
I've added the module and also an example within the examples/complete using the DBs created there as well.
references
https://docs.aws.amazon.com/dms/latest/userguide/CHAP_Serverless.html

cc @AdamTylerLynch, if I'm remembering correctly, you were looking Serverless DMS support within Cloud Posse modules! Check this out and let @oycyc or I know if there are any enhancements you would want to make prior to merge 👍

/terratest

/terratest

@oycyc Terratest is failing because this IAM role name is hard-coded
https://github.com/cloudposse/terraform-aws-dms/blob/main/modules/dms-iam/main.tf#L38
https://github.com/cloudposse/actions/actions/runs/9781915755/job/27007106494#step:10:2666

TestExamplesComplete 2024-07-03T17:10:30Z logger.go:66: │ Error: creating IAM Role (dms-access-for-endpoint): operation error IAM: CreateRole, https response error StatusCode: 409, RequestID: cb6a7c41-7de6-4ae2-a46e-31833fb2394a, EntityAlreadyExists: Role with name dms-access-for-endpoint already exists.

It's not necessarily related to this PR, but we unfortunately need to fix the tests to merge this PR

@oycyc Terratest is failing because this IAM role name is hard-coded
https://github.com/cloudposse/terraform-aws-dms/blob/main/modules/dms-iam/main.tf#L38 https://github.com/cloudposse/actions/actions/runs/9781915755/job/27007106494#step:10:2666

TestExamplesComplete 2024-07-03T17:10:30Z logger.go:66: │ Error: creating IAM Role (dms-access-for-endpoint): operation error IAM: CreateRole, https response error StatusCode: 409, RequestID: cb6a7c41-7de6-4ae2-a46e-31833fb2394a, EntityAlreadyExists: Role with name dms-access-for-endpoint already exists.

It's not necessarily related to this PR, but we unfortunately need to fix the tests to merge this PR

Looking into this!

/terratest

@oycyc Terratest is failing because this IAM role name is hard-coded
https://github.com/cloudposse/terraform-aws-dms/blob/main/modules/dms-iam/main.tf#L38 https://github.com/cloudposse/actions/actions/runs/9781915755/job/27007106494#step:10:2666

TestExamplesComplete 2024-07-03T17:10:30Z logger.go:66: │ Error: creating IAM Role (dms-access-for-endpoint): operation error IAM: CreateRole, https response error StatusCode: 409, RequestID: cb6a7c41-7de6-4ae2-a46e-31833fb2394a, EntityAlreadyExists: Role with name dms-access-for-endpoint already exists.

It's not necessarily related to this PR, but we unfortunately need to fix the tests to merge this PR
@milldr
Can you rerun the test? Just pushed a commit that should fix that, and also the other error with name exceeding the character limit.

@oycyc Terratest is failing because this IAM role name is hard-coded
https://github.com/cloudposse/terraform-aws-dms/blob/main/modules/dms-iam/main.tf#L38 https://github.com/cloudposse/actions/actions/runs/9781915755/job/27007106494#step:10:2666

TestExamplesComplete 2024-07-03T17:10:30Z logger.go:66: │ Error: creating IAM Role (dms-access-for-endpoint): operation error IAM: CreateRole, https response error StatusCode: 409, RequestID: cb6a7c41-7de6-4ae2-a46e-31833fb2394a, EntityAlreadyExists: Role with name dms-access-for-endpoint already exists.

It's not necessarily related to this PR, but we unfortunately need to fix the tests to merge this PR

@milldr  Can you rerun the test? Just pushed a commit that should fix that, and also the other error with name exceeding the character limit.

/terratest

@milldr ah I see why it wasn't already previously using the null label, the role name has to explicitly hard coded like dms-vpc-role. (https://repost.aws/questions/QU61eADUU7SnO-t7MmhxgfPA/dms-service-roles#AN4e6MtkTATtyEN2THAO6csA)
It's not being referenced by any of the resources provisioned in this module but it's being used by AWS so now Terratest is failing changing the name

But if it's explicitly using that name in the codebase, it fails it's already created in Terratest's environment (didn't get destroyed in previous run?)
I reverted it for now.

/terratest

Related discussion about test failure started in this PR https://github.com/cloudposse/terraform-aws-dms/pull/30#issuecomment-2321292307

/terratest

/terratest

@oycyc tests are failing due to serverless config creation time out:
TestExamplesComplete 2024-09-01T11:28:41Z logger.go:66: module.dms_serverless_replication_config.aws_dms_replication_config.default[0]: Still creating... [48m50s elapsed]
panic: test timed out after 1h0m0s

Have you run them locally? Does it behave another way?


Geometry attributes/values. TS wrapper on Geopandas

Use geopandas as alternative to pandas.
If geopandas detected, export to geoparquet instead of parquet for attributes

If I understood correctly, this is automatically done if a non-None geometry is provided after #58, right @sphamba ?
Basically if df_attrs is a GeoDataFrame, we are thus using GeoDataFrame.to_parquet, that already takes care of writing the geometry columns. The issue is whether we should use "geoarrow" instead of the default "WKB" (see the geometry_encoding argument at https://geopandas.org/en/stable/docs/reference/api/geopandas.GeoDataFrame.to_parquet.html). Any ideas on this @ghiggi ? In any case, I doubt this is any performance bottleneck for most use cases and I would like to avoid any premature optimization, but I can (i) test the effects with a proper benchmark and (ii) add a to_parquet_kwargs argument to the to_tstore methods which is forwarded to df_attrs.to_parquet.

@martibosch Yes, loading a TSDF (or converting into a TSDF) with geometry data will create a TSDF wrapper around a Geopandas dataframe. For writing geometry attributes, we are indeed using GeoDataFrame.to_parquet, and for reading, we attempt reading the attributes dataframe with geopandas.read_parquet, which fails if there is no geometry attributes.


Please invite me to the GitHub Community Organization

Please invite me to the GitHub Community Organisation.

Name:

Discord Username(if exists):

Additional Context:

Hi @clefcodex.
Please, I don't know if you read the above message.
An invitation has been sent to your github registered email address. So, please accept it to be a recognised member on the page.
Also, joing the discord server is pertinent, that's one of the places where we collaborate. Here is the link https://discord.gg/c6RhGwcP5b
And lastly follow the community handle on Twitter https://twitter.com/ChryzHub
Thanks 🙏
Anticipating to see you amongst us


Removed docker folder as it is no longer needed

I've already moved the info in README to the wiki.

Please send PR against wip branch


Fix numbers being reported as the keys of objects to TypeScript - this is invalid.

Currently, the TypeScript definitions include a number of Record<string|number, ...>.  This is invalid as the keys of an object can't be numbers - if you put a number in, the key is coerced to a string.  This causes a build error on depending upon webpack if skipLibCheck isn't enabled in your tsconfig.json.

What kind of change does this PR introduce?

I've removed the number option from the key type parameter of every Record<...> I can find.
Did you add tests for your changes?

I do not believe that the TypeScript definitions have test coverage to update.  I am not sure how I would do this without adding a large number of build dependencies.
Does this PR introduce a breaking change?

This is a type change, but shouldn't break anything - type checks for libraries must have been turned off to use these types up until now anyway.
What needs to be documented once your changes are merged?

I am unaware of a need to document these changes.

I am unsure why the lint test is failing - I have executed the command it suggests and it makes no further changes.

Looks like the type generation made an unexpected change.  Reverted that, let's see if that fixes the tests.

This is invalid as the keys of an object can't be numbers - if you put a number in, the key is coerced to a string.

What is error?

node_modules/webpack/types.d.ts:746:12 - error TS2344: Type 'string | number' does not satisfy the constraint 'string'.
  Type 'number' is not assignable to type 'string'.

746  ): Record<string | number, Record<string, (string | number)[]>>;
               ~~~~~~~~~~~~~~~

node_modules/webpack/types.d.ts:793:12 - error TS2344: Type 'string | number' does not satisfy the constraint 'string'.
  Type 'number' is not assignable to type 'string'.

793  ): Record<string | number, (string | number)[]>;
               ~~~~~~~~~~~~~~~

node_modules/webpack/types.d.ts:799:12 - error TS2344: Type 'string | number' does not satisfy the constraint 'string'.
  Type 'number' is not assignable to type 'string'.

799  ): Record<string | number, Record<string | number, string>>;
               ~~~~~~~~~~~~~~~

node_modules/webpack/types.d.ts:799:36 - error TS2344: Type 'string | number' does not satisfy the constraint 'string'.
  Type 'number' is not assignable to type 'string'.

799  ): Record<string | number, Record<string | number, string>>;
                                       ~~~~~~~~~~~~~~~

node_modules/webpack/types.d.ts:803:12 - error TS2344: Type 'string | number' does not satisfy the constraint 'string'.
  Type 'number' is not assignable to type 'string'.

803  ): Record<string | number, boolean>;
               ~~~~~~~~~~~~~~~

node_modules/webpack/types.d.ts:1020:15 - error TS2344: Type 'string | number' does not satisfy the constraint 'string'.
  Type 'number' is not assignable to type 'string'.

1020  hash: Record<string | number, string>;
                   ~~~~~~~~~~~~~~~

node_modules/webpack/types.d.ts:1021:22 - error TS2344: Type 'string | number' does not satisfy the constraint 'string'.
  Type 'number' is not assignable to type 'string'.

1021  contentHash: Record<string | number, Record<string, string>>;
                          ~~~~~~~~~~~~~~~

node_modules/webpack/types.d.ts:1022:15 - error TS2344: Type 'string | number' does not satisfy the constraint 'string'.
  Type 'number' is not assignable to type 'string'.

1022  name: Record<string | number, string>;
                   ~~~~~~~~~~~~~~~

node_modules/webpack/types.d.ts:1034:13 - error TS2344: Type 'string | number' does not satisfy the constraint 'string'.
  Type 'number' is not assignable to type 'string'.

1034  id: Record<string | number, (string | number)[]>;
                 ~~~~~~~~~~~~~~~

node_modules/webpack/types.d.ts:1035:15 - error TS2344: Type 'string | number' does not satisfy the constraint 'string'.
  Type 'number' is not assignable to type 'string'.

1035  hash: Record<string | number, string>;
                   ~~~~~~~~~~~~~~~

Found 15 errors.

I imagine most people wouldn't have all of the strict correctness checks enabled in tsconfig.json - they're off by default.

Please provide them

I've listed the errors above - are those "them" in "Please provide them"?  Thanks.

@jameswilddev I mean please provide ts config

Aha!  Sorry.  Here it is:
{
  "compilerOptions": {
    "charset": "utf8",
    "composite": false,
    "declaration": true,
    "diagnostics": false,
    "emitBOM": false,
    "incremental": false,
    "inlineSourceMap": false,
    "inlineSources": false,
    "jsx": "preserve",
    "listFiles": false,
    "module": "CommonJS",
    "newLine": "LF",
    "noEmit": false,
    "noEmitHelpers": false,
    "noEmitOnError": true,
    "noImplicitAny": true,
    "noImplicitThis": true,
    "noUnusedLocals": true,
    "noUnusedParameters": true,
    "noResolve": false,
    "noStrictGenericChecks": false,
    "skipDefaultLibCheck": false,
    "skipLibCheck": true,
    "preserveConstEnums": false,
    "preserveSymlinks": false,
    "preserveWatchOutput": true,
    "pretty": true,
    "removeComments": false,
    "isolatedModules": false,
    "sourceMap": false,
    "suppressExcessPropertyErrors": false,
    "suppressImplicitAnyIndexErrors": false,
    "stripInternal": true,
    "target": "ES2018",
    "experimentalDecorators": true,
    "emitDecoratorMetadata": true,
    "moduleResolution": "node",
    "allowUnusedLabels": false,
    "noImplicitReturns": true,
    "noFallthroughCasesInSwitch": true,
    "allowUnreachableCode": false,
    "forceConsistentCasingInFileNames": true,
    "traceResolution": false,
    "allowJs": false,
    "noErrorTruncation": false,
    "allowSyntheticDefaultImports": false,
    "listEmittedFiles": false,
    "disableSizeLimit": true,
    "strictNullChecks": true,
    "importHelpers": false,
    "alwaysStrict": true,
    "strict": true,
    "strictBindCallApply": true,
    "downlevelIteration": true,
    "strictFunctionTypes": true,
    "strictPropertyInitialization": true,
    "esModuleInterop": false,
    "allowUmdGlobalAccess": false,
    "keyofStringsOnly": true,
    "declarationMap": false,
    "resolveJsonModule": true,
    "lib": [
      "ES2018",
      "DOM"
    ]
  }
}

Changing skipLibCheck from false to true should reproduce the above error.

Ah coverage checks have failed.  I don't have access to see the results but it's likely flagging up that I've changed comments as those won't be possible to hit in tests.

I've signed the CLA - it's just not flagged up yet.

"keyofStringsOnly": true is the culprit in this case. Note that it's not a "strict correctness check". It enables an old pre-TypeScript-2.9 (2018) behavior. See here. webpack doesn't need to be compatible with that old behavior.
As for the correctness checks, it's recommended to use "strict": true. It's a master switch that enables them all at once.

This is invalid as the keys of an object can't be numbers

Yet, on the type level, TS makes a distinction between string and number keys. E.g., [0]['0'] yields a compilation error.

Why do you think Record<string|number, ...> is invalid? The type definition explicitly allows that:
type Record<K extends string | number | symbol, T> = { [P in K]: T; }

I also don't see problems doing that:
https://www.typescriptlang.org/play?#code/G4QwTgBAHgXBBKBTAxgezAEwDwDsCuAtgEaJgA0E+xpAfBALwQDeAvgNwCwAUN1ANoBGALoMIAzjy78AzCMYAmNkA
Maybe you are using the deprecated keyofStringsOnly option?
https://www.typescriptlang.org/tsconfig#keyofStringsOnly

Hmm - that's not what I'm seeing on TypeScript 4.2.3 if I F12 on Record<...> in VS Code:
/**
 * Construct a type with a set of properties K of type T
 */
type Record<K extends keyof any, T> = {
    [P in K]: T;
};

I also find it interesting that they're allowing applications to treat object keys as numeric when they're clearly not:
> typeof Object.keys({ 1: null })[0]
'string'

I'll admit though that I don't keep up with modern ES.
If you think the current behaviour is correct feel free to reject the PR.

From typescript perspective it's valid. From Typescript-side property names can be string | number | Symbol.


[Helm] Updated the helm package to deploy the upgraded blockscout version

Description
Updated the Helm Package to deploy the new version of Blockscout
Tested
No yet tested. Before testing, https://github.com/celo-org/blockscout/pull/8/ has to be merged, the new image generated, adapt the image tag to latest or configure a different docker tag in the helm variables, and then deploy to verify.
Related issues

Fixes #994, #998, #1001

Backwards compatibility
The deployment changes a bit the approach (one unique docker image for the two containers, the indexed and the web, with a different command.

@diminator Why did you assign me to this PR? Looks like I already approved it and no changes happened since then

Ok, no worries. If all is good then @jcortejoso can merge


After disaster recovery , the unhealthy etcd was still not able to recover.

Rancher Version: v1.1.1-rc1
etcd version used - llparse/etcd:v2.3.6-4
Steps to reproduce the problem:
Have a kubernetes setup with  2 hosts.
The host where etcd_1 was running went from "Reconnecting" -> "Active" state.
etcd_1 container keeps restarting (Initializing -> stopped)
In this state , though I was able to query for existing pods/services. But not able to create new ones .

Attempted disaster recovery:

Exec into etcd_2 container (Which is the healthy one) and run disaster command.
Restart etcd_2
Restart etcd_1

Even after this , etcd_1 continues to be stuck in Initializing -> stopped loop.

@LLParse: I tested the same scenario with etcd version used - llparse/etcd:v2.3.6-5.
I am still able to reproduce the issue.
etcd_1 continues to be "Initializing" state even after Disaster Recovery steps was performed
Logs from etcd_1 container:
7/14/2016 1:45:29 PM+ META_URL=http://rancher-metadata.rancher.internal/2015-12-19
7/14/2016 1:45:29 PM++ wget -q -O - http://rancher-metadata.rancher.internal/2015-12-19/self/stack/name
7/14/2016 1:45:29 PM+ STACK_NAME=Kubernetes
7/14/2016 1:45:29 PM++ wget -q -O - http://rancher-metadata.rancher.internal/2015-12-19/self/container/create_index
7/14/2016 1:45:29 PM+ CREATE_INDEX=24
7/14/2016 1:45:29 PM++ wget -q -O - http://rancher-metadata.rancher.internal/2015-12-19/self/container/name
7/14/2016 1:45:29 PM+ NAME=Kubernetes_etcd_1
7/14/2016 1:45:29 PM+ export ETCDCTL_ENDPOINT=http://etcd.Kubernetes:2379
7/14/2016 1:45:29 PM+ ETCDCTL_ENDPOINT=http://etcd.Kubernetes:2379
7/14/2016 1:45:29 PM+ '[' 1 -eq 0 ']'
7/14/2016 1:45:29 PM+ eval node
7/14/2016 1:45:29 PM++ node
7/14/2016 1:45:29 PM++ '[' -d /data.backup/member ']'
7/14/2016 1:45:29 PM++ '[' -d /data/member ']'
7/14/2016 1:45:29 PM++ giddyup leader check
7/14/2016 1:45:29 PM+++ grep Kubernetes_etcd_1
7/14/2016 1:45:29 PM+++ etcdctln member list
7/14/2016 1:45:29 PM+++ target=0
7/14/2016 1:45:29 PM++++ seq 1 5
7/14/2016 1:45:29 PM+++ for j in '$(seq 1 5)'
7/14/2016 1:45:29 PM++++ seq 1 2
7/14/2016 1:45:29 PM+++ for i in '$(seq 1 $SCALE)'
7/14/2016 1:45:29 PM+++ giddyup probe http://Kubernetes_etcd_1:2379/health
7/14/2016 1:45:29 PM+++ '[' 1 == 0 ']'
7/14/2016 1:45:29 PM+++ for i in '$(seq 1 $SCALE)'
7/14/2016 1:45:29 PM+++ giddyup probe http://Kubernetes_etcd_2:2379/health
7/14/2016 1:45:30 PM+++ '[' 1 == 0 ']'
7/14/2016 1:45:30 PM+++ '[' 0 '!=' 0 ']'
7/14/2016 1:45:30 PM+++ sleep 1
7/14/2016 1:45:31 PM+++ for j in '$(seq 1 5)'
7/14/2016 1:45:31 PM++++ seq 1 2
7/14/2016 1:45:31 PM+++ for i in '$(seq 1 $SCALE)'
7/14/2016 1:45:31 PM+++ giddyup probe http://Kubernetes_etcd_1:2379/health
7/14/2016 1:45:31 PM+++ '[' 1 == 0 ']'
7/14/2016 1:45:31 PM+++ for i in '$(seq 1 $SCALE)'
7/14/2016 1:45:31 PM+++ giddyup probe http://Kubernetes_etcd_2:2379/health
7/14/2016 1:45:32 PM+++ '[' 1 == 0 ']'
7/14/2016 1:45:32 PM+++ '[' 0 '!=' 0 ']'
7/14/2016 1:45:32 PM+++ sleep 1
7/14/2016 1:45:33 PM+++ for j in '$(seq 1 5)'
7/14/2016 1:45:33 PM++++ seq 1 2
7/14/2016 1:45:33 PM+++ for i in '$(seq 1 $SCALE)'
7/14/2016 1:45:33 PM+++ giddyup probe http://Kubernetes_etcd_1:2379/health
7/14/2016 1:45:33 PM+++ '[' 1 == 0 ']'
7/14/2016 1:45:33 PM+++ for i in '$(seq 1 $SCALE)'
7/14/2016 1:45:33 PM+++ giddyup probe http://Kubernetes_etcd_2:2379/health
7/14/2016 1:45:34 PM+++ '[' 1 == 0 ']'
7/14/2016 1:45:34 PM+++ '[' 0 '!=' 0 ']'
7/14/2016 1:45:34 PM+++ sleep 1
7/14/2016 1:45:35 PM+++ for j in '$(seq 1 5)'
7/14/2016 1:45:35 PM++++ seq 1 2
7/14/2016 1:45:35 PM+++ for i in '$(seq 1 $SCALE)'
7/14/2016 1:45:35 PM+++ giddyup probe http://Kubernetes_etcd_1:2379/health
7/14/2016 1:45:35 PM+++ '[' 1 == 0 ']'
7/14/2016 1:45:35 PM+++ for i in '$(seq 1 $SCALE)'
7/14/2016 1:45:35 PM+++ giddyup probe http://Kubernetes_etcd_2:2379/health
7/14/2016 1:45:36 PM+++ '[' 1 == 0 ']'
7/14/2016 1:45:36 PM+++ '[' 0 '!=' 0 ']'
7/14/2016 1:45:36 PM+++ sleep 1
7/14/2016 1:45:37 PM+++ for j in '$(seq 1 5)'
7/14/2016 1:45:37 PM++++ seq 1 2
7/14/2016 1:45:37 PM+++ for i in '$(seq 1 $SCALE)'
7/14/2016 1:45:37 PM+++ giddyup probe http://Kubernetes_etcd_1:2379/health
7/14/2016 1:45:37 PM+++ '[' 1 == 0 ']'
7/14/2016 1:45:37 PM+++ for i in '$(seq 1 $SCALE)'
7/14/2016 1:45:37 PM+++ giddyup probe http://Kubernetes_etcd_2:2379/health
7/14/2016 1:45:37 PM+++ '[' 1 == 0 ']'
7/14/2016 1:45:37 PM+++ '[' 0 '!=' 0 ']'
7/14/2016 1:45:37 PM+++ sleep 1
7/14/2016 1:45:38 PM+++ '[' 0 == 0 ']'
7/14/2016 1:45:38 PM+++ echo No etcd nodes available
7/14/2016 1:45:38 PM++ '[' '' '!=' '' ']'
7/14/2016 1:45:38 PM++ runtime_node
7/14/2016 1:45:38 PM+++ giddyup service containers --exclude-self
7/14/2016 1:45:38 PM++ for container in '$(giddyup service containers --exclude-self)'
7/14/2016 1:45:38 PMWaiting for lower index nodes to all be active
7/14/2016 1:45:38 PM++ echo Waiting for lower index nodes to all be active
7/14/2016 1:45:38 PM+++ wget -q -O - http://rancher-metadata.rancher.internal/2015-12-19/self/service/containers/Kubernetes_etcd_2/create_index
7/14/2016 1:45:38 PM++ ctx_index=4
7/14/2016 1:45:38 PM++ '[' 4 -lt 24 ']'
7/14/2016 1:45:38 PM++ giddyup probe http://Kubernetes_etcd_2:2379/health --loop --min 1s --max 60s --backoff 1.1
7/14/2016 1:45:39 PMHTTP 503
7/14/2016 1:45:39 PM
7/14/2016 1:45:41 PMHTTP 503
7/14/2016 1:45:41 PM
7/14/2016 1:45:43 PMHTTP 503
7/14/2016 1:45:43 PM
7/14/2016 1:45:45 PMHTTP 503
7/14/2016 1:45:45 PM
7/14/2016 1:45:47 PMHTTP 503
7/14/2016 1:45:47 PM
7/14/2016 1:45:49 PMHTTP 503
7/14/2016 1:45:49 PM
7/14/2016 1:45:52 PMHTTP 503
7/14/2016 1:45:52 PM
7/14/2016 1:45:54 PMHTTP 503
7/14/2016 1:45:54 PM
7/14/2016 1:45:57 PMHTTP 503
7/14/2016 1:45:57 PM
7/14/2016 1:46:00 PMHTTP 503```

Observed the same issue in the scenario below:
Started with 3 nodes. Added pods
Turn off 2 hosts
Performed disaster recovery steps. Was able to create more pods after this.
Turned on the hosts. The etcds were stuck in initializing  state.
Bring down 1 host where etcd_1 is running.
At this point , we are able to query existing pods/service.But not able to create new ones.
Bring up host.
After this etcd_1 is able to start successfully. Able to create new pods and services successfully.

Have a kubernetes setup with 5 hosts.
Bring down 2 hosts where etcds are running.
At this point , there is no attempt made to bring up etcd in the 2 remaining hosts (because of recreateOnQuorum set to 2).
Delete both the hosts that are currently down (deactivate and delete).
This results in 2 etcds being created in the remaining 2 hosts.
These etcds are stuck in "Initializing" state.
At this point , we are able to query for existing pods/services ..but not able to create new ones.
Perform DR.
On the etc that is still healthy , execute disaster.
Restart the etc container.
After a while, both the etcds that are stuck in "Initializing" state become "healthy". We are able to create  new pods/services after this.

Tested with updated qa-catalog.
Started with 3 nodes. Added pods
Turn OFF 2 hosts
Performed disaster recovery steps. Was able to create more pods after this.
Turned ON the hosts. Kubernetes was functional. I was able to create pods successfully.


Vickel: Watch Buds[\W_]+Culture[\W_]+Store(?!.com(?<=budsculturestore.com))

Vickel requests the watch of the watch_keyword Buds[\W_]*+Culture[\W_]*+Store(?!\.com(?<=budsculturestore\.com)). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Buds[\W_]*+Culture[\W_]*+Store(?!\.com(?<=budsculturestore\.com)) has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Update README.md

Purpose

...
Found that without the py launcher in the path, the azd up is failing. I feel that we need to add a bit more clarification on py launcher dependency for azd up, so adding the following: "Important - Please note that azd up would look for py launcher, make sure py launcher is in the path system environment variable. After successful installation of Python (with py launcher selected in the installer), put the py location (i.e., C:\Windows) in the system path variable.

Please review and let me know if we can add this change. It took me some time to figure this out, hopefully will save some end user time seeing the same behavior. One of my customers had this issue. Even though I had Python in path, azd up kept failing and only way to fix this was to put the py launcher location in the path system variable.
Does this introduce a breaking change?

[ ] Yes
[ x] No

Pull Request Type
What kind of change does this Pull Request introduce?
Documentation

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ x] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code
No code changes

Test the code

What to Check
Verify that the following are valid

...
Review the information added in the README.md and test if needed. Please reach out, if I can help with the clarification

Other Information

I don't have the py launcher and was able to run this template successfully, what OS are you running on?


Verify immmunefi reporting on LPToken total supply 1 wei bug

Adds the POC contract and tests it on forked mainnet against empty FRAX/USDC pool.

Closing this since it has been verified.


User gets kicked out by owner application

Sometimes a users gets kicked out of the application and is routed to the start page when trying to create a new need. => Unauthorized
configRouting.js:165 GET https://satsrv07.researchstudio.at/owner/rest/users/isSignedIn 401 (Unauthorized)accessControl @ configRouting.js:165(anonymous function) @ configRouting.js:88$broadcast @ angular.js:7889transitionTo @ angular-ui-router.js:1512go @ angular-ui-router.js:1421(anonymous function) @ router-middleware.js:14dispatch @ applyMiddleware.js:35(anonymous function) @ account-actions.js:75 utils.js:370 Uncaught (in promise) Error: Unauthorized(…)checkHttpStatus @ utils.js:370 messaging-agent.js:73 checking heartbeat presence:  22
Maybe related log in owner log:
27.9.16 15:43:49.476 [Thread-10] WARN  w.o.m.OwnerWonMessageSenderJMSBased - Could not register with default won node https://satsrv07.researchstudio.at/won/resource java.lang.NullPointerException: null         at won.protocol.util.RdfUtils.getNodeForPropertyPath(RdfUtils.java:663) ~[won-core-0.2-SNAPSHOT.jar:na]         at won.protocol.util.RdfUtils$4.visit(RdfUtils.java:682) ~[won-core-0.2-SNAPSHOT.jar:na]         at won.protocol.util.RdfUtils$4.visit(RdfUtils.java:679) ~[won-core-0.2-SNAPSHOT.jar:na]         at won.protocol.util.RdfUtils.findFirst(RdfUtils.java:860) ~[won-core-0.2-SNAPSHOT.jar:na]         at won.protocol.util.RdfUtils.findFirst(RdfUtils.java:867) ~[won-core-0.2-SNAPSHOT.jar:na]         at won.protocol.util.RdfUtils.getNodeForPropertyPath(RdfUtils.java:678) ~[won-core-0.2-SNAPSHOT.jar:na]         at won.protocol.util.RdfUtils.getURIPropertyForPropertyPath(RdfUtils.java:465) ~[won-core-0.2-SNAPSHOT.jar:na]         at won.protocol.jms.ActiveMQServiceImpl.getBrokerEndpoint(ActiveMQServiceImpl.java:134) ~[won-core-0.2-SNAPSHOT.jar:na]         at won.owner.messaging.OwnerProtocolCommunicationServiceImpl.configureCamelEndpoint(OwnerProtocolCommunicationServiceImpl.java:153) ~[won-owner-0.2-SNAPSHOT.jar:na]         at won.owner.messaging.OwnerProtocolCommunicationServiceImpl.register(OwnerProtocolCommunicationServiceImpl.java:89) ~[won-owner-0.2-SNAPSHOT.jar:na]
After a restart of the owner things are working normal again ...
=> Is the reason that the registration at the wonnode was working then?

The stack trace from above but with line-breaks:
configRouting.js:165
GET https://satsrv06.researchstudio.at/owner/rest/users/isSignedIn 401 (Unauthorized)
accessControl @ configRouting.js:165
(anonymous function) @ configRouting.js:88
$broadcast @ angular.js:7889
transitionTo @ angular-ui-router.js:1512
go @ angular-ui-router.js:1421
(anonymous function) @ router-middleware.js:14
dispatch @ applyMiddleware.js:35
(anonymous function) @ account-actions.js:75
utils.js:370
Uncaught (in promise) Error: Unauthorized(…)
checkHttpStatus @ utils.js:370

didnt have this problem for long time anymore => close this ticket?

yes, i agree


choose chi_sim.traineddata will crash

I don't know if chi_sim.traineddata is too big(40+M) , can you deal this problem

Which device do you use? It seems that 40+M is not a big deal for any Apple device.
What error messages do you get? Can you show a code snipet you try?

I just tested chi_sim on both simulator and actual device without any issues. What's the error message that you get?

@xiaoxingan, can you ensure that you don't use Cube mode. It seems that Cube mode is no longer supported. Refer to #140 for more details.

@kevincon, let's close this as well.

G8RecognitionOperation *operation = [[G8RecognitionOperation alloc] initWithLanguage:@"eng+chi_sim+chi_tra"];
operation.tesseract.engineMode = G8OCREngineModeTesseractCubeCombined;
2015-09-04 10:13:15.299 CamClaim[15569:4962459] BSXPCMessage received error for message: Connection interrupted
Using default language params
Cube ERROR (CubeRecoContext::Load): unable to read cube language model params from /private/var/mobile/Containers/Bundle/Application/FEF9236E-7977-4038-864C-D220533359C1/CamClaim.app/tessdata/chi_sim.cube.lm
Cube ERROR (CubeRecoContext::Create): unable to init CubeRecoContext object
init_cube_objects(true, &tessdata_manager):Error:Assert failed:in file tessedit.cpp, line 203

G8RecognitionOperation *operation = [[G8RecognitionOperation alloc] initWithLanguage:@"eng+chi_sim+chi_tra"];
operation.tesseract.engineMode = G8OCREngineModeTesseractCubeCombined;
2015-09-04 10:13:15.299 CamClaim[15569:4962459] BSXPCMessage received error for message: Connection interrupted
Using default language params
Cube ERROR (CubeRecoContext::Load): unable to read cube language model params from /private/var/mobile/Containers/Bundle/Application/FEF9236E-7977-4038-864C-D220533359C1/CamClaim.app/tessdata/chi_sim.cube.lm
Cube ERROR (CubeRecoContext::Create): unable to init CubeRecoContext object
init_cube_objects(true, &tessdata_manager):Error:Assert failed:in file tessedit.cpp, line 203

G8RecognitionOperation *operation = [[G8RecognitionOperation alloc] initWithLanguage:@"eng+chi_sim+chi_tra"];
operation.tesseract.engineMode = G8OCREngineModeTesseractCubeCombined;
2015-09-04 10:13:15.299 CamClaim[15569:4962459] BSXPCMessage received error for message: Connection interrupted
Using default language params
Cube ERROR (CubeRecoContext::Load): unable to read cube language model params from /private/var/mobile/Containers/Bundle/Application/FEF9236E-7977-4038-864C-D220533359C1/CamClaim.app/tessdata/chi_sim.cube.lm
Cube ERROR (CubeRecoContext::Create): unable to init CubeRecoContext object
init_cube_objects(true, &tessdata_manager):Error:Assert failed:in file tessedit.cpp, line 203

G8RecognitionOperation *operation = [[G8RecognitionOperation alloc] initWithLanguage:@"eng+chi_sim+chi_tra"];
operation.tesseract.engineMode = G8OCREngineModeTesseractCubeCombined;
2015-09-04 10:13:15.299 CamClaim[15569:4962459] BSXPCMessage received error for message: Connection interrupted
Using default language params
Cube ERROR (CubeRecoContext::Load): unable to read cube language model params from /private/var/mobile/Containers/Bundle/Application/FEF9236E-7977-4038-864C-D220533359C1/CamClaim.app/tessdata/chi_sim.cube.lm
Cube ERROR (CubeRecoContext::Create): unable to init CubeRecoContext object
init_cube_objects(true, &tessdata_manager):Error:Assert failed:in file tessedit.cpp, line 203

G8RecognitionOperation *operation = [[G8RecognitionOperation alloc] initWithLanguage:@"eng+chi_sim+chi_tra"];
operation.tesseract.engineMode = G8OCREngineModeTesseractCubeCombined;
2015-09-04 10:13:15.299 CamClaim[15569:4962459] BSXPCMessage received error for message: Connection interrupted
Using default language params
Cube ERROR (CubeRecoContext::Load): unable to read cube language model params from /private/var/mobile/Containers/Bundle/Application/FEF9236E-7977-4038-864C-D220533359C1/CamClaim.app/tessdata/chi_sim.cube.lm
Cube ERROR (CubeRecoContext::Create): unable to init CubeRecoContext object
init_cube_objects(true, &tessdata_manager):Error:Assert failed:in file tessedit.cpp, line 203

G8RecognitionOperation *operation = [[G8RecognitionOperation alloc] initWithLanguage:@"eng+chi_sim+chi_tra"];
operation.tesseract.engineMode = G8OCREngineModeTesseractCubeCombined;
2015-09-04 10:13:15.299 CamClaim[15569:4962459] BSXPCMessage received error for message: Connection interrupted
Using default language params
Cube ERROR (CubeRecoContext::Load): unable to read cube language model params from /private/var/mobile/Containers/Bundle/Application/FEF9236E-7977-4038-864C-D220533359C1/CamClaim.app/tessdata/chi_sim.cube.lm
Cube ERROR (CubeRecoContext::Create): unable to init CubeRecoContext object
init_cube_objects(true, &tessdata_manager):Error:Assert failed:in file tessedit.cpp, line 203
Cube mode is no longer supported for Chinese ???

G8RecognitionOperation *operation = [[G8RecognitionOperation alloc] initWithLanguage:@"eng+chi_sim+chi_tra"];
operation.tesseract.engineMode = G8OCREngineModeTesseractCubeCombined;
2015-09-04 10:13:15.299 CamClaim[15569:4962459] BSXPCMessage received error for message: Connection interrupted
Using default language params
Cube ERROR (CubeRecoContext::Load): unable to read cube language model params from /private/var/mobile/Containers/Bundle/Application/FEF9236E-7977-4038-864C-D220533359C1/CamClaim.app/tessdata/chi_sim.cube.lm
Cube ERROR (CubeRecoContext::Create): unable to init CubeRecoContext object
init_cube_objects(true, &tessdata_manager):Error:Assert failed:in file tessedit.cpp, line 203
Cube mode is no longer supported for Chinese ???

I know the G8OCREngineModeTesseractCubeCombined mode is the most accurate. If I only need recognize English, this mode is ok. But if I need recognize Chinese, this mode can not be used. Anybody helps?

@brave1024, is it still an issue for you?

@brave1024 u dont have enough resource for engine "Cube". Modify the engine to tesseract only will fix the problem..  .traineddata file just for tesseract engine

__block G8Tesseract * tesseract = [[G8Tesseract alloc]initWithLanguage:@"chi_sim" engineMode:G8OCREngineModeTesseractOnly];
chi_sim can't using G8OCREngineModeTesseractCubeCombined,@xiaoxingan，

Closing an issue

G8Tesseract *test=[[G8Tesseract alloc]initWithLanguage:@"eng+chi_sim" engineMode:G8OCREngineModeTesseractOnly];
//    test.engineMode=G8OCREngineModeTesseractOnly;
test.delegate=self;
[test setImage:[UIImage<EMAIL_ADDRESS>]];
[test setRect:CGRectMake(20, 20, 100, 100)];
[test recognize];
Error:
actual_tessdata_num_entries_ <= TESSDATA_NUM_ENTRIES:Error:Assert failed:in file ../../ccutil/tessdatamanager.cpp, line 53
I need recognize Chinese, Anybody helps?

Same issue here for Korean language cannot use cubeCombined with Korean, Please any one have resource or link of resource for cubeCombined Korean language  Tesseract 3.03-rc1 ?

@fb2012 i am Chinese......but I also this bug error...so Have you solved the problem?
emmm my engilsh is not good,i hope you can understand.....


How to change inference labels

Which "classes file" does the program use for inference?
So I get this result for example:

It is not taking the files with the annotations for some reason.
Should we change some configuration for the inference step @FateScript ?

Labels is provided here named COCO_CLASSES, plz check if it fits your self defined dataset.