Ionic: Uncaught Error: API request /v1/profile missing parameter identity_id

Hi,
I'm trying to use Branch for my Ionic app but it's impossible... I receive this error when trying to set the identity or generating a link (this one is from setIdentity):
Uncaught Error: API request /v1/profile missing parameter identity_id
The init is done correctly as I don't receive any error. But what is strange here is that the result of the init function is an empty structure:
{"data":null,"data_parsed":null,"has_app":null,"identity":null,"referring_identity":null,"referring_link":null}
The init is done in $ionicPlatform.ready and the createUrl is done via a user action (so after init).
What am I doing wrong ?
Thanks !

Hi @Gp2mv3 - That's quite strange.

Can you enable debug (branch.setDebug(true);) and share the logs?
Can you also share how you're initing the library? Here's a Cordova example: https://github.com/BranchMetrics/Cordova-Ionic-PhoneGap-Deferred-Deep-Linking-SDK/blob/master/cordova-testbed/www/js/index.js#L38

Lastly, how did you add the module? NPM or from the Github repo?

Hi, thanks for this fast answer.
branch.setDebug(true); doesn't do anything (it seems). Where are the logs normally (Logcat or Browser logs) ?
The init is done like this:
 $ionicPlatform.ready(function() {
    if (window.cordova) {
      branch.setDebug(true);
      branch.init(Config.branchKey, function(err, data) {
        console.log("branch ready");
        console.log(err);
        console.log("data "+JSON.stringify(data));
      });

      $ionicPlatform.on('resume', function(err, data) {
        branch.init(Config.branchKey, function(err, data){
          console.log("branch resume");
        });
      });
    }
  });

Where Config.branchKey is the test key. Then the url generation is done via
      branch.link({
        channel: 'shareMenu',
        feature: 'group',
        stage: 'share group',

        data: {
          "$deeplink_path": "app/group/"+$scope.group._id
        }
      }, function(err, link){
        if(err) {
          alert(err);
        }
        else {
          window.plugins.socialsharing.share($scope.group.name, null, null, link);
        }
      });

Thanks !

Hmm.. that's right. I think debug mode was never implemented in this version. We're about to roll out an entirely new version of the Cordova SDK with all of this implemented properly.
@Gp2mv3 And is this with the Test version of Nightspot? I see a few successful opens, and a link that was created in the live view. https://dashboard.branch.io/#/liveview/events/view This would indicate it's working some of the time...

Can you try changing the init method to:
 branch.init(Config.branchKey, { }, function(err, data) {

Yes it's for NightSpot but I'm still getting this error and I
Even with branch.init(Config.branchKey, { }, function(err, data) {...
What's weird is that I see in the Dashboard that I opened the app multiple times and that Identities are stored as well. I recognize my phone in the events. So maybe the error is a false alarm ?

Of, by enabling eval, it's working so it's the exact same issue as issue #30.


{webapp} Addition to description of list-runtime command

Fixes https://github.com/Azure/azure-cli/issues/21728 .
Addition to description of list-runtime command.
Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

webapp

Closing this PR as the fix will be done on other PR https://github.com/Azure/azure-cli/pull/21771


add SUPPORT.md

https://help.github.com/articles/adding-support-resources-to-your-project/

This doesn't really seem necessary for the time being. Closing for now.


TLS ClientHello and TLS Alert not Parsed

I have a small network dump that only consists of a few packets. In total there are 2 TLS packets included in this dump:

a ClientHello
a TLSAlert

When I iterate over the dump the TLS layer will not be parsed but instead will be displayed as "Payload". When I use this payload to create a new packet like this:
newPacket := gopacket.NewPacket(tcpLayer.LayerPayload(), layers.LayerTypeTLS, gopacket.Default)
it is successfully parsed as a TLS packet. I double checked that DecodeStreamsAsDatagrams is set to true. I am bit confused, because every other dump I had so far worked like a charm. What would be the best way to debug this?

I just happen to notice that the function func (t *TCP) NextLayerType() gopacket.LayerType deduces the next layer based on the port (e.g. 53 for DNS, 443 for HTTPS). As my target port is 8443, it was not included in the list of ports that have are related to LayerTypeTLS.
Suggestion
Would it be feasible to add an additional  check for the bytes indicating  TLS content? As it might be problematic for some applications I would additionally suggest another DecodeOption like ForceTLSDecoding to signal that an additional check for TLS content is wanted.


Add and implement (output) DatasetVersionDatasetFacet for Spark

This commit adds DatasetVersionDatasetFacet to OpenLineage spec.
It implements this facet for v2 writes in Apache Spark for output datasets, where provider is Iceberg or Delta.
In both cases, we use their implementation of TableCatalog to get their internal dataset versions.
The implementation utilizes new facet builder API for output dataset facets. This allows the main class of this commit: DatasetVersionOutputDatasetFacetBuilder to listen only on SparkListenerSQLExecutionEnd event, which is emitted only when writing to dataset, both data and metadata is complete. Other events, like SparkListenerJobEnd do not have this property.
This PR will someday be followed with implementation of this facet for input datasets. The main blocker is that we can't use current structure of CustomFacetBuilder that attaches resulting facet to any dataset. There can be multiple input datasets, and they can have different versions, or have no explicit version. Issue is described here: https://github.com/OpenLineage/OpenLineage/issues/484
Signed-off-by: Maciej Obuchowski<EMAIL_ADDRESS>

There can be multiple input datasets, and they can have different versions, or have no explicit version. Issue is described here:

My thinking here was that the facet builder could use the input Stage's RDD to trigger when the facet should be generated for an input dataset.


try-catch and using statements in exception handling

It says, "Use a try-catch statement for most exception handling" and proceeds to show an example where the exception could be avoided by doing bounds checks. I don't think this should be a recommended coding convention since exception-based programming is more expensive than mitigating exceptions that can be prevented.
If anything, this should demonstrate catching an exception that you can't prevent (i.e. SocketException in some cases), and not catching the System.Exception type but the most relevant derived exception as a best practice.

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

ID: 40f42506-f4a5-859a-0266-ded002056d83
Version Independent ID: f940f2ad-e13d-4d37-6439-f8afcbc4e4c7
Content: C# Coding Conventions
Content Source: docs/csharp/fundamentals/coding-style/coding-conventions.md
Product: dotnet-csharp
Technology: csharp-fundamentals
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

These are generally good suggestions @bitm0de
I'm not convinced that SocketException or something similar is the best option. I'd rather pick something more common, to avoid the need to explain more about the conditions which cause the exception.

These are generally good suggestions @bitm0de
I'm not convinced that SocketException or something similar is the best option. I'd rather pick something more common, to avoid the need to explain more about the conditions which cause the exception.

You mention that it is not the best option, although no alternative suggestion is made. SocketException is rather common in networking, and an exhaustive list of preconditions doesn't have to be provided to make a point. One probable cause is to state the closing of the connection from the server side. The real point being, to use a try-catch as the solution in an appropriate context for catching the exception (i.e. when you can't mitigate it in most circumstances).
The current example provides no explanation about what would cause an IndexOutOfRangeException, or whether it's the most appropriate way to handle the underlying cause of the exception rather than preventing it. Thus, it does seem odd to suggest the need for listing the causes for a SocketException, even if it is arguably a better use of a try-catch.


Verify hash in CI

@drazisil Talked about it in #243

Tests fail 🛑
Please upgrade the hash.

@Slamdunk It looks like the version of shasum on Travis doesn't have the --strict flag. Thoughts?

Try sha1sum command:
sha1sum --check --strict SHA1SUM

Modernization done.

One more package and we are there
https://manpages.ubuntu.com/manpages/bionic/en/man1/shasum.1p.html

All green 🍏

@drazisil Could you merge it?


Repo sync for protected CLA branch

The pull request is created from master637408018997748916 to master to fix git push error for protected CLA branch

@cmcclister : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Protocol versus API standardization

For video ingestion, the case for protocol standardization seems solid - distinct implementations (e.g. mobile, browser, server) need to interoperate.
For media distribution to a browser where the code for the service and player are written by the same entity, the case for protocol standardization seems less clear to me.  The major interop question is whether APIs are implemented in an interoperable way between browsers.
I'd note that WebRTC datachannel is today very widely used for media distribution (particularly in cloud gaming) without an apparent need for standardization of the media encapsulation.  Instead, the focus has been on API interoperability (particularly differences in browser MSE implementations).  Why is QUIC different?

I don't think that the service and the player are the same entity is a safe assumption necessarily. There are a bunch of player vendors that don't have an associated service, and there are services that work with 3rd party players (such as IVS).

Yeah I was going to say the exact same thing.
There are many services that offer HLS/DASH endpoints and expect you to use your own player. In fact that's kind of the point; it's easy to host the media files via a commodity CDN and watch via some arbitrary player (ex. hls.js, VLC, exoplayer, avplayer).

Propose to change the charter text - omit the word container, replace with media format


Error Message in antsMultivariateTemplateConstruction2.sh

Hello,
I have successfully run antsMultivariateTemplateConstruction.sh several times, but I consistently get errors when attempting to run antsMultivariateTemplateConstruction2.sh. It appears the error is occurring at line 266: "imagelist=(ls ${outputname}template${whichtemplate}*WarpedToTemplate.nii.gz)". It seems that when this line is first run, the only file which has been created in my output directory is "${outputname}template0.nii", so it throws an error when the specified file above is not found.
Previously, I also received an error at line 218, stating that "AverageImages: command could not be found." Originally, the script read "AverageImages $dim $output 0 ${images[]}" at line 218. After replacing this with ANTSPATH appended, i.e., "${ANTSPATH}/AverageImages $dim $output 0 ${images[]}", the script seemed to run normally past this point.
Here is the Cygwin output I receive as the first error occurs:

Thanks,
Elliot

Regarding the second issue---usually ANTSPATH is specified in the PATH, which is why this bug was missed.  However, it's fixed now.
I'm not sure about the first issue.  Make sure you're using the latest ANTs from the repo and let me know if you can reproduce the error on this  2-D example, specifically, the brain slices example using this script.

I'm pretty sure this is fixed.  Might need to reopen if the issue still persists.

Apologies for the delayed response. It looks like I receive a similar error message when running this on the template building example. I used the following command "
${ANTSPATH}/antsMultivariateTemplateConstruction2.sh -d 2 -o T1_ -i 4 -g 0.25 -j 2 -c 0 -k 2 -w 1x1 -f 8x4x2x1 -s 3x2x1x0 -q 100x70x50x10 -n 1 -r 1 -l 1 -m CC[2] -t TimeVaryingVelocityField[0.25,3,3,0,0,0] OASIS*.nii.gz" from the TemplateMultivariateBSplineSyN.sh script. I've successfully completed the 2D brain slices example before using the command in templateCommandSyN.sh however, which calls antsMultivariateTemplateConstruction.sh instead of antsMultivariateTemplateConstruction2.sh.
Below is the error message I received this time:

It looks like my directory where the brain slices are now has "job_0_0" files with "metriclog" files, as well as "T1_template0OASIS-TRT-20-16Slice121Repaired.nii" files, "T1_template1OASIS-TRT-20-10Slice121LaplacianRepaired.nii" files, and a "T1_template1.nii". However, there don't appear to be any files which say "...Warp.nii.gz", "...GenericAffine.mat", or "WarpedToTemplate.nii.gz", as the error message states. Also, it looks like the N4Bias correction step ran successfully, albeit with some error message periodically (see below).

Finally, if it makes a difference, I am running ANTs on windows.
Thanks,
Elliot

How recent is your ANTs build?  Are you using the latest version along with the latest scripts?

Yes, it looks like it may possibly be a version compatibility issue. I initially downloaded and compiled the most recent release from source code using mingGW, CMake, ITK4, and zlib. Although it seemed to work fine for some applications, it may not have compiled completely correctly, so I tried using the pre-compiled Windows version on Sourceforge. BTW, I noticed that Sourceforge currently has a very old version set as the default download for Windows users (it says "Looking for the latest version? Download ANTs-1.9.x-win32.exe (48.0 MB)" from June 2011. This download is getting a fair amount of downloads per week currently based on the site's statistics, and this is what I initially downloaded by mistake. I found it had very limited functions compared to recent releases, both in terms of the C++ applications and shell scripts, and I'm not sure if it was really functional anymore. I fear that a lot of unaware Windows downloaders may be wasting their time on this version. Are there any plans in the near term to update the pre-compiled Windows to version 2.1?
Afterwards, I found and downloaded the most recent Windows version, ANTs-1.9.v4-win32.exe, from Dec. 2012, and updated this with the latest shell scripts and other files. This seems to have mostly full functionality, and recently I've been able to run antsMVTC.2 using both SyN and TimeVaryingVelocityField for transformation methods. However, it still throws the same error for B-Spline SyN. Is this likely due to version incompatibility? antsMVTC.sh seems fully functional. (Note: when running these examples, I noticed that the scripts in the brain slice template building example have their transformation methods mismatched: the templateCommandMultivariateBSplineSyN.sh script is currently set to use TimeVaryingVelocityField[0.25,3,3,0,0,0] and the templateCommandTimeVarying.sh script is set to use BSplineSyN[0.1,26,0].)
Thanks,
Elliot

However, it still throws the same error for B-Spline SyN. Is this likely due to version incompatibility?

Probably.

you can find releases here: https://github.com/stnava/ANTs/releases


deprecate getExpenseForViewer

This Pr is a follow up to this PR: https://github.com/opencollective/opencollective-api/pull/2567
Also, I updated the front end in this PR: https://github.com/opencollective/opencollective-frontend/pull/2575

@znarf was it specified that we wanted to remove getExpenseForViewer?

@Betree it's now effectively useless and should be removed. However, I have problems with other aspects of this PR and have yet to provide clear feedback: modification to dataLoader, straight removal of properties instead of deprecation.

@znarf @Betree Friendly ping.

@znarf Done.

/cc @Betree


custom url for whisper api

I run whisper in docker and I would like to automatically generate transcripts through the plugin myself instead of having to do so manually.

Great, can you share the docker build?
Is there an API available in the docker container that allows you to interact with whisper from outside the container?

I just ran the commands in the readme: https://github.com/ahmetoner/whisper-asr-webservice
The API is pretty simple, you upload the file and tell it which format to return.

Ah I was hoping there was an existing docker image you used to run whisper. In order to run the commands within the plugin, it's necessary to create a REST API within the docker container to interact with Whisper, like running the model and getting the result. A downloadable docker image will need to be created to make it distributable. That's a good chunk of work and it'll be a while until I get to it. If you'd like to contribute that would be great too.

I don't think it's a good idea to automatically spin up a docker image from within obsidian. I think it would be best to require users to spin up what I linked themselves, which is quite easy with docker desktop. https://github.com/djmango/obsidian-transcription does this, but it's flaky from my testing. Not that I know anything about the internals of this plugin currently, but it seems moderate to run a command on an audio file that saves the json result next to it in-tree.

The plugin would never spin up a docker container automatically.
The overlap between the number of users who can spin up a docker container and use it, but who do not know how to install Whisper on their own machine is likely small, so the use case for using a docker container would be to support people who a) cannot install python and Whisper, b) can install docker and one image, and c) cannot interact with docker. So the workflow I would want to implement would be to support accessing docker via an API, which is necessary for the plugin anyway.
Writing an API will probably take 4 hours, and as you have seen from the other image it can be finicky. This is the majority of the work - create a container, preinstall Whisper, create a REST API, and publish the container. Hooking it up in the plugin will be straightforward after that. I likely won't get to this for a while.

Thanks for that clarification. I'm one of those people who can't install python and whisper, because I can not figure out python's dependency management and environments and so on and so forth. I guess I also don't know the difference between a docker container and a docker image.
Thanks for considering this issue.


Issue/4401 resource actions join table (#4707)

Pull request opened by the merge tool on behalf of #4707

Merged into branches iso5 in 8fd6d0c68273de93f8282a5c05dd6a83c0890ee5


Support explain_weights(pipeline)

Towards fixing #15, allowing explain_weights for Pipelines. Does not handle explain_predictions; tracking the provenance of features back to parts of text may be tricky in a convoluted pipeline/union structure.
No tests yet.
Introduces new eli5.transform.transform_feature_names singledispatch, which could do with more implementations.
I think the exact forms of the names should remain experimental... or indeed we could refuse to provide default implementations for anything but vectorizers, selectors, Pipeline and FeatureUnion, letting users register their own.
Rubbish example:
from sklearn import *
from eli5 import explain_weights
from IPython.display import display
est = pipeline.make_pipeline(pipeline.FeatureUnion([('words', pipeline.make_pipeline(feature_extraction.text.CountVectorizer(),
                                                                                feature_extraction.text.TfidfTransformer())),
                                                    ('chars', feature_extraction.text.CountVectorizer(analyzer='char'))]),
                             feature_selection.SelectKBest(feature_selection.chi2, k=10),
                             linear_model.LogisticRegression())
bunch = datasets.fetch_20newsgroups(categories=['alt.atheism', 'talk.religion.misc', 'rec.autos'])
est.fit(bunch.filenames, bunch.target)

display(explain_weights(est, target_names=bunch.target_names))

I'm not sure I'll be able to complete this patch any time soon, though I am keen to have something like this merged! Help is welcome.

Codecov Report

:exclamation: No coverage uploaded for pull request head (pipeline@4f93e7f).

Continue to review full report at Codecov.

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

[Note: @kmike's wish that we only calculate feature names when we've determined that they are important requires on knowing dependencies between input and output features for each transformer, i.e. something like https://github.com/scikit-learn/enhancement_proposals/pull/5]

Thanks @jnothman! If you don't have time we can pick up the development. I've left some comments in order to not forget about them.

How much do you think we should prefabricate feature name designs and how much leave it to the user? As I said, we could import implementations of transform_feature_names for Pipeline, FeatureUnion, SelectorMixin by default, and leave others unimplemented; or provide default implementations in some optional import like eli5.sklearn.experimental. the issue is that sometimes a user will  to see "topic1" for brevity, but a list of the top features on the topic would be more helpful!

It may similarly be useful to show "scale(featurename)" but some users would rather the scale transform elided. There are many ways to name features.
Which of course leads to the possibility of feature descriptions, but I think that can wait for later

More untested code pushed :)

I now see that it should be trivial to support the same feature in explain_prediction

How much do you think we should prefabricate feature name designs and how much leave it to the user? As I said, we could import implementations of transform_feature_names for Pipeline, FeatureUnion, SelectorMixin by default, and leave others unimplemented; or provide default implementations in some optional import like eli5.sklearn.experimental. the issue is that sometimes a user will to see "topic1" for brevity, but a list of the top features on the topic would be more helpful!

I think providing as much default feature names as possible is a good thing. As for topics, what do you think about showing only a few top features? eli5.formatters.html could allow to expand topics on a client, or just show more words in title attribute.

It may similarly be useful to show "scale(featurename)" but some users would rather the scale transform elided. There are many ways to name features.
Which of course leads to the possibility of feature descriptions, but I think that can wait for later

Good points.
I think providing defaults like scale(featurename) is fine; customizability can be added / documented later if there is demand.

@kmike what do you propose to move this forward? (I've noticed not a great deal of eli5 work lately in general.) Are we happy with the API and I should find time to write tests?

@jnothman I'd love to have it moved forward :) To merge it we need tests. I like the API; transformer coverage is great, and we can always improve coverage in future.
I'd also register everything by default, without marking it as experimental and requiring user to call a register function. This way explain_weights works in more cases, and if users want to override the way features are formatted it is the same amount of work (register their own function).
Other than that it looks fine.

i think that might be for an old conception. I'm slowly working through
tests and docs, but there's a lot to test. starting bottom up.
On 6 Apr 2017 7:42 pm, "Mikhail Korobov"<EMAIL_ADDRESS>wrote:

@kmike commented on this pull request.
In eli5/sklearn/explain_weights.py
https://github.com/TeamHG-Memex/eli5/pull/158#discussion_r110119142:

@@ -412,3 +414,19 @@ def _features(target_id):
method='linear model',
is_regression=True,
)

+@explain_weights.register(Pipeline)
+def explain_weights_pipeline(estimator, feature_names=None, **kwargs):

last_estimator = estimator.steps[-1][1]
transform_pipeline = Pipeline(estimator.steps[:-1])
if hasattr(feature_names, 'get_feature_names'):

What is this condition for? I'd expect feature_names to be a list or
FeatureNames instance.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/TeamHG-Memex/eli5/pull/158#pullrequestreview-31266529,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEz68CpZLPDj5cb_H6s9HRHot-gbIMgks5rtLNugaJpZM4LyOOJ
.

@jnothman thanks for the update! What do you think about merging it partially, e.g. only the API part, Pipeline, FeatureUnion, and maybe a couple of basic transformers, to make it easier to test & document?

might be sensible. Either way I don't think I'll have anything for the next
week
On 6 Apr 2017 10:00 pm, "Mikhail Korobov"<EMAIL_ADDRESS>wrote:

@jnothman https://github.com/jnothman thanks for the update! What do
you think about merging it partially, e.g. only the API part, Pipeline,
FeatureUnion, and maybe a couple of basic transformers, to make it easier
to test & document?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/TeamHG-Memex/eli5/pull/158#issuecomment-292152280,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEz6-6fIV89IJivgwtPoJq6ylNlPe00ks5rtNPxgaJpZM4LyOOJ
.

I'll pull out a PR with just the basic (explain_weights_pipeline plus transform_feature_names defined for feature selection and pipeline).

Where does further documentation belong?

@jnothman what further documentation are you thinking of? I think an example of how to register your own transform_feature_name handler may go to sklearn library docs.

Yes, fair enough.
On 3 May 2017 at 22:39, Mikhail Korobov<EMAIL_ADDRESS>wrote:

@jnothman https://github.com/jnothman what further documentation are
you thinking of? I think an example of how to register your own
transform_feature_name handler may go to sklearn library docs.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/TeamHG-Memex/eli5/pull/158#issuecomment-298898784,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEz6yGad0xV8h6iK0P3rg60VDwTfQ2xks5r2HV2gaJpZM4LyOOJ
.


Let user use system installed mkl-dnn, googltest, rapidcheck ?

I seriously do NOT wanna use the third-party submodules. Is it possible to let CMakeLists.txt to search for my local installed packages?
Thank you

@jiapei100 It is possible, but, the version of system installed mkl-dnn may be incompatible with current version of ideep(Chainer Backend for Intel Architecture).


Backgrounding the app without canceling auth flow

Describe the feature request?
Hey Okta team,
When user clicks on Login button, we are setting our loading state to true and call WebAuthenticationClient's login which then launches the chrome custom tab. If user closes the tab, we catch the FlowCancelledException which we use to reset the loading state.
If the user backgrounds the app and then use the app launcher icon to relaunch the app, the custom tab gets destroyed and it leaves the user in perpetual loading state. Is there a callback that can be used here to reset loading state when the ForegroundActivity gets destroyed?
New or Affected Resource(s)
User loading state
Provide a documentation link
No response
Additional Information?
No response

Adding comment to get attention

Hi @satyajitvure91, this issue may be happening if you are executing this function from a viewModelScope or lifecycleScope. Those coroutine scopes get destroyed when the user backgrounds the app. I have to investigate this issue further, but does overriding ViewModel.onCleared function help with this issue in this case?
There isn't a callback currently, but I can emit an event in the SDK's EventCoordinator whenever ForegroundActivity gets destroyed.

Thank you @rajdeepnanua-okta. I can follow the solution you've suggested and update you


.gitignore not reflecting live

Hi,
Steps to reproduce
1 - Open folder project
2 - Modify the .gitignore file, adding an entry: my_temp_folder
3 - my_temp_folder still visible
Expected:
Folder hidden because entry was added to .gitignore list
Currently:
This is reflecting now until editor window is reopened, so my question is there any way to handle this without closing and opening again the window ?

I have a similar problem that might be related.
When i remove an item (eg dist) from the .gitignore file.
It should show up on the tree view, because its not being ignored anymore.
But it does not show up on tree view.
I have to close and reopen my ide for changes to take effect.
I have also tried all combinations of Hide Ignored Names in the tree-view options, and Exclude VCS Ignored Paths.  None of these take effect until i close and reopen.
atom version: 1.5.3
os: windows 10

@asotog Do you mean that the file is still showing in the Tree View? You could try collapsing and expanding the root folder. That should refresh it.
@damieng Can you reproduce what @germ13 is describing?

Unless we're monitoring .gitignore for changes I can see how this would happen.

We should be. This works on OS X for me. The tree view updates as soon as I save the .gitignore file.

Yeah it works when you collapse and expand again the folder

I tried collapsing and expanding the folder and it solved the issue as well.
@damieng can you give me more info on where to look at in the atom code (or docs) to enable monitoring of the .gitignore file.  A more natural solution than collapsing and expanding the folders would be better.
thanks.

Are you editing the .gitingore in Atom? Or are you editing the .gitignore in some other manner?

I am updating the .gitignore within Atom. No other manner.

Ok, then there shouldn't be anything else you need to do to enable monitoring. It should just work.

seems non-intuitive to have to collapse and expand tree to see effects of saving the .gitignore file.
seems like it should refresh/reflect immediately upon save.

My statement about collapsing and expanding the root folder was specifically to address the question:

is there any way to handle this without closing and opening again the window ?

It wasn't a suggestion that this is acceptable. If I thought it was acceptable or intended, I would have closed the issue after giving the explanation.

Could the linux label be added as well? This is also reproducible on Ubuntu 15.10 with all versions of Atom up to and including 1.6.0-beta6.

@lee-dohm understood. was just making sure we were on same page.

This also happens on OSX.

@lordmat0 Please give some more information. All indications are that this does not reproduce on OS X.

@lee-dohm I think when it upgraded to 1.6.0 it broke a lot of things, so I have now done a clean install of Atom. Unfortunately I forgot to test this so for now I'll take back my comment of it not working on OSX and report back on Tuesday if it's the case for OSX.


Improve the enabled / disabled styling of the send message button

At the moment it's not easy to see when sending is disabled...

Fixed in d114ef8


Update Demo01JavaRunner.java

fix broken locator in sample test
Description
Thanks for contributing this Pull Request. Make sure that you submit this Pull Request against the develop branch of this repository, add a brief description, and tag the relevant issue(s) and PR(s) below.

Relevant Issues : (compulsory)
Relevant PRs : (optional)
Type of change :

[ ] New feature
[ ] Bug fix for existing feature
[ ] Code quality improvement
[ ] Addition or Improvement of tests
[ ] Addition or Improvement of documentation

@MohabMohie thanks for the heads up !


we need a multiple res image mixin

aka retinaImg and/or media-conditional resolutions

we can use http://bourbon.io/docs/#retina-image


use the latest go version to build server/cli

This PR is for:

Workflow

@zhangzujian Please correct the PR title to follow the rules specified here: https://github.com/deepflowio/deepflow/blob/main/commit-template


Include cross-links in man pages

Make dh_golang and dh-make-golang easier to discover by people eading man pages about Go packaging in Debian.
Current versions for reference:

https://manpages.debian.org/unstable/dh-golang/Debian::Debhelper::Buildsystem::golang.3pm.en.html
https://manpages.debian.org/unstable/dh-golang/dh_golang.1p.en.html
https://manpages.debian.org/unstable/dh-make-golang/dh-make-golang.1.en.html

Related: https://salsa.debian.org/go-team/packages/dh-golang/-/merge_requests/24

@foka Would you be the best person to review this?

@anthonyfok Would you be the best person to review/approve this?


r3:do(compile). kills embedded yaws service

I'm humbly hoping someone here may advise or comment on this yaws-related ticket which involves rebar3 https://github.com/erlang/rebar3/issues/1549
I'm using the latest versions of rebar3 and yaws 2.0.4
Because I cannot use rebar3's reloading behaviour I've been starting a new erl shell for every change I make, which is slow and cumbersome.

I'm guessing you built Yaws with HiPE enabled? If so, try again without it.

I'll take a look later this evening and let you know if HiPE is enabled. I used rebar3 to include yaws and didn't explicitly include or disclude it myself.
There are some responses in the rebar3 issues link that include other hypotheses.
Thanks.

I'm not sure how rebar3 is managing the Yaws compilation but it appears that HiPE would need to be explicitly disabled with something like ./configure --disable-hipe. I will ask the rebar3 thread if there is a "rebar3 way" of doing this.

In an attempt to move this forward, here's a repost of what I had on the rebar3 issue:

My bet there is on Yaws not coping with sequential hot-code loads and one of the modules getting killed. You can see process <0.320.0> getting killed (I assume this is gserv). The parent (I assume this is yaws_serv) is holding a reference to <0.320.0> in its state before terminating with reason restartme.
You can see this taking place here in the code: https://github.com/klacke/yaws/blob/b32f7d49eff120f7860453a488acbd358beb5763/src/yaws_server.erl#L421-L430
I don't know that too much can be done if this server dies.


root.hints permissions

sometimes, during interface reset, unbound config is regenerated/reloaded and I'm getting errors in unbound log like this:
 error: could not read root hints /root.hints: Permission denied

it looks like there could be 2 reasons for that:

race condition: /var/unbound/unbound.conf file is regenerated , which triggers unbound process to reread it and thus reload root.hints, but at the same time new root.hints is created
actually wrong permissions, default ones are 0600, so if there is some mismatch between file owner and unbound user - unbound cannot read it

so, by moving root.hints generation before unbound.conf generation and setting permissions to 0644 we can eliminate both reasons
usually rename preserves source file permissions, this is why permissions are set before rename, but to be 101% sure - chmod is used again after rename

I don’t mind adding Chloe but moving the spot is probably not a good idea when we already talk race conditions.

I don’t mind adding chmod but moving the spot is probably not a good idea when we already talk race conditions.

@fichtner why? by moving root.hints file generation before we make sure that during main conf file creation hints are already there and nobody is touching them

The history of this repeated tweaking is reason enough to be careful if you want to read it don’t take my word for it.

The history of this repeated tweaking is reason enough to be careful if you want to read it don’t take my word for it.

@fichtner  I'm not sure that I understand you, what should I read? why moving the spot is not good idea?


Kalman filter

Reference Issues/PRs
https://github.com/alan-turing-institute/sktime/issues/1011
What does this implement/fix? Explain your changes.

Does your contribution introduce a new dependency? If yes, which one?

What should a reviewer concentrate their feedback on?

Any other comments?

PR checklist

For all contributions

[ ] I've added myself to the list of contributors.
[ ] Optionally, I've updated sktime's CODEOWNERS to receive notifications about future changes to these files.
[ ] I've added unit tests and made sure they pass locally.
[ ] The PR title starts with either [ENH], [DOC] or [BUG] indicating whether the PR topic is related to enhancement, documentation or bug

For new estimators

[ ] I've added the estimator to the online documentation.
[ ] I've updated the existing example notebooks or provided a new one to showcase how my estimator works.

Hi @fkiraly :)
Here is the Kalman Filter draft PR we spoke about.
About denoising (given control variables [u], measurements [z], produce denoised z) with Kalman Filter -
Can it be implemented by first using filter() from pykalman, then using the output as input for the rts_smoother() (implementation of the Rauch-Tung-Striebal Kalman smoother)  from FilterPy?
Lines 347-358 in file kalman_filter.py
By your advice about making one class per interfaced package, 'FilterPy' is only used in the lines mentioned above and can be removed easily.

Can it be implemented by first using filter() from pykalman, then using the output as input for the rts_smoother()

In theory, that would probably work, looking at the docs.
However, I would strongly advise against using both packages in the same class. Hopefully there is a way to smoothe/denoise using pykalman alone?

PS: the tests are failing since you didn't implement get_test_params.

All tests pass, great!

(let me know when you think this is ready for review)

We were happy to do it🙂
Thank you for all the help, it is much appreciated.


Update ArraySink interface usage: property name "array", not "a"

Codecov Report

Merging #109 into master will not change coverage.
The diff coverage is 50%.

@@           Coverage Diff           @@
##           master     #109   +/-   ##
=======================================
  Coverage   63.98%   63.98%
=======================================
  Files          35       35
  Lines        1688     1688
  Branches      125      125
=======================================
  Hits         1080     1080
  Misses        608      608

Impacted Files
Coverage Δ

lib/confluence/failure_to_ship.es6.js
100% <ø> (ø)
:arrow_up:

lib/client/api_confluence.es6.js
10.34% <0%> (ø)
:arrow_up:

lib/confluence/api_velocity.es6.js
98.33% <0%> (ø)
:arrow_up:

lib/client/api_matrix.es6.js
59.71% <0%> (ø)
:arrow_up:

lib/confluence/metric_computer.es6.js
95.91% <100%> (ø)
:arrow_up:

lib/confluence/aggressive_removal.es6.js
100% <100%> (ø)
:arrow_up:

lib/confluence/browser_specific.es6.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


dotnet: consider emitting namespace, class, method, and field/property names referenced by an assembly

(1) e.g. emit the following for System.IO.File::OpenRead:
- namespace: System.IO
- class: File
- method: OpenRead

(2) be more explicit to avoid FPs resulting from naming conflicts, e.g.:
- namespace: System.IO
- class: System.IO.File
- method: System.IO.File::OpenRead

and for properties, e.g. (more discussion in #939):
- namespace: System.Net
- class: System.Net.WebRequest
- property: System.Net.WebRequest::Method

I like option 2 the best, and I think this would apply to both file and function scopes and both internal and external references.

we can then write rules like:
rule:
  meta:
    name: manipulate files in dotnet
    namespace: host-interaction/file-system
    scope: function
  features:
    - and:
      - class: System.IO.File

Do we need the namespace field for (2)? Can you think of rules where we would use that in rules right now?

Including the namespace for imported functions like System.IO.File::OpenRead is probably overkill. However, including user-defined namespaces may facilitate malware family detection. I don't have a specific example for the latter.

I propose the following:

emit class, method, and property/field names for imported classes
emit namespace, class, method, and property/field names for user-defined classes

Ah, yeah, that could be neat to detect on namespaces. For simplicity I'd then say we emit namespace, class, method, and property/field names for all classes?!

I agree; we should keep it simple by extracting the same for all classes. so, emit the following for all classes referenced at file and function scope:

namespace
class
method/field/property

see #939 for discussion on emitting references to properties/fields.

this may require introducing new features, including:

namespace
class
field/property??
method??

see #939 for field/property.
unsure if we need method or if we can simply reuse api.

see #939 for property access.
#1030 covers namespace/class/method.


fix: allow specifying id attribute for blockquotes and tables

I added missing id attribute.
Test markdown:
> p
{#asdf .sec}

| | |
|-|-|
|test|test2|
{.cls #tbl _thead=false}

Rendered result:

p

        test
        test2

I only tested with #id syntax, but this should also work with id= syntax.

The table should already support id, could you please confirm it and revert the change of table hook.

I removed the table hook from the PR. Thank you for letting me know about that!

Thanks.


Show a happy message when there are no savings.

Closes #992

Bit fugly, but...

Great! Is it possible to keep the "learn more" link so people can see what this is all about?


Fixed steps for confusing apps UI in on premise SharePoint

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/publish/publish-task-pane-and-content-add-ins-to-an-add-in-catalog.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Allow one to sync non-bare local git repository

One can currently manage bare local git repositories with modulesync but
cannot manage non-bare git repositories. This commit aims to add this
feature.
If I was to have several local directories (from git clone), I can't keep
them in sync with modulesync as git clone would complain about :
fatal: '/tmp/puppet/openstack/puppet-nova.git' does not
appear to be a git repository
fatal: Could not read from remote repository.
Yet '/tmp/puppet/openstack/puppet-nova' is a valid path with a .git
directory in it.
This is due to the '.git' appended at the end when protocol is file.

:+1:


support calculate function coverage rate

It will display as format
int getX(int x) 1x [100.0%]
if ( x > 0) { 1x
    return 1; 1x
}
return 0; 1x

Signed-off-by: FANG.Ge<EMAIL_ADDRESS>

@xiaoyun94 Is there any chance for you to work on the requested changes?

@JacquesLucke Isn't that feature included in version 0.5.0  (the one still missing on open-vsx ;-) ? In this case the issue could be closed.


Scrape and analyze tourism data

Who is interested, pls comment :)
@pratimakandel - if you have time may i request you to scrap factsheet 1, 2 and 3 from this doc: http://tourism.gov.np/files/statistics/2.pdf
Pls submit a pull request to our data repo: https://github.com/Code4Nepal/data
@CodingWithLaurence are you interested?

Huge thanks to @pratimakandel for scraping the data: https://github.com/Code4Nepal/data/tree/master/datasets/Tourist 2018


Updates around Build Tool Tasks

Changes/additions under the Build tool tasks section:

Consolidated use of header levels.
Added information about the version parameters for the tool installer task
Added WhoAmI task
Added parameters for the Checker task
Consolidated naming of the tasks with what they are called when working with pipelines

@rappen Because of changes that have been applied since you logged this PR (fie was split in two), I had to manually apply your changes. The updates have been published. I am closing this PR.
Thanks for the suggestions.

Thanks for the follow-up, @phecke !


fix: fix typos

Thanks for fixing the typo!
Let's ping and assign @cedoor for review.

Thanks for fixing the typo! Let's ping and assign @cedoor for review. Test pipeline is failing but I don't think it is related to the changes introduced by the PR.thinking

I also don't think it's related to this PR. I'll merge it anyway.


Know that PG's IS DISTINCT FROM may never return null

Resolves #2602

Refactor infix_operator to only express the expression SQL type and bounds once
Add capability to have a constant (or different nullability constraints) to infix_operator (undocumented for now)
Use new feature of infix_operator in IS DISTINCT FROM and IS NOT DISTINCT FROM

While I really appreciate that you work on fixing all this issues, I'm quite sure that opening all those PR's that fast is a good way to burn me out on maintaining diesel. Each time my backlog grows I'm asking myself How I should ever handle that on my own and unfortunately there is no one else I can share that work with. It's probably a good idea to give my some time to work through all those PR's and issues before opening any new, even if that likely means waiting a few weeks.

While I really appreciate that you work on fixing all this issues, I'm quite sure that opening all those PR's that fast is a good way to burn me out on maintaining diesel. Each time my backlog grows I'm asking myself How I should ever handle that on my own and unfortunately there is no one else I can share that work with. It's probably a good idea to give my some time to work through all those PR's and issues before opening any new, even if that likely means waiting a few weeks.

You have all my moral support with regards to the amount of open stuff. ^^
If you want to add me to the reviewers ping so I can try and help review some of the PRs as well that's fine by me.
I had to implement these anyway because I wanted to finally get rid of #2274, which we still had as we were pinned to the commit just before #2182 was merged because of the follow-up issues. (Now we're about to move to a hopefully temporary fork of the latest master which just has these few PRs merged in advance.)
That being said, once they are implemented I can think of no better place to tidy them than in the open-awaiting-for-review PRs list, should they not be reviewed right away because we don't have enough time to allocate to them (which is completely understandable, thanks a lot for all the time you are putting in Diesel in any case <3).

If you want to add me to the reviewers ping so I can try and help review some of the PRs as well that's fine by me.

I've added you to the corresponding team. Any help reviewing open active PR's is definitively welcome.

I am actually relying on the return type of this being billable now. 😓

@pksunkara I'm not sure how that would be an argument to not fix this issue. That's just the cost of using a unreleased (and therefore not finished) version of diesel.

Why we cannot write this change as
infix_operator!(IsDistinctFrom, " IS DISTINCT FROM ", Bool, backend: Pg);

by just specifying a return type?

Because the way infix_operator is currently written, whatever the input type, it will be made Nullable based on the whether the input arguments are nullable:
https://github.com/diesel-rs/diesel/blob/06494e8bdfe774bac9be5cab488b3d0cbec97f4e/diesel/src/expression/operators.rs#L229-L237
Not setting the return type is just like setting it to Bool
https://github.com/diesel-rs/diesel/blob/06494e8bdfe774bac9be5cab488b3d0cbec97f4e/diesel/src/expression/operators.rs#L220-L222
I beleive that was a smart choice in general, because most of the SQL operators match that behavior, e.g.
https://github.com/diesel-rs/diesel/blob/06494e8bdfe774bac9be5cab488b3d0cbec97f4e/diesel/src/pg/expression/operators.rs#L20-L22
Which is why my approach was to add support for specifying "please don't do that".

Which is why my approach was to add support for specifying "please don't do that".
Another approach would have been to write what the macro generates manually for both these operators and any other one that also needs this and we might not have noticed yet, reducing the complexity of that macro but increasing that of defining such an operator. I would probably have gone for that approach if there was only one, but because there were at least 2 I was considering writing a small macro for implementing both, so I thought I might as well add support for this in the original macro.

OK, that makes sense now, should have looked at this macro a bit more in detail. Nevertheless I do not thing it's that common that a operator has this behavior, therefore I think we should not mention this feature as part of the documentation and we should not consider that feature to be part of the public API. (Mainly because I feel that the current implementation does not really fit into the API provided by the macro + I do not see a clear way to improve that API.)

Can we expand this same usage to IS NULL and IS NOT NULL too?
https://github.com/Ten0/diesel/blob/fix_2602_is_distinct_from_is_never_null/diesel/src/expression/operators.rs#L485-L486

Can we expand this same usage to IS NULL and IS NOT NULL too?
https://github.com/Ten0/diesel/blob/fix_2602_is_distinct_from_is_never_null/diesel/src/expression/operators.rs#L485-L486

IS NULL/IS NOT NULL already has the correct output, as it's declared as a postfix_operator, whose output's nullability does not depend on the nullability of the arguments:
https://github.com/diesel-rs/diesel/blob/e8c8d167d56ad12a271535f9027a7e9768ce93f8/diesel/src/expression/operators.rs#L331-L355
(It's not an infix_operator so it doesn't go through the same macro.)

Considering this is deemed finished and was validated twice a week ago, is it ok to merge it, or are we waiting for additional input?

 As mentioned already too often: This is a free time project for me, so if I don't have time or motivation to work on it I'm not doing anything which includes merging PR's.

 As mentioned already too often: This is a free time project for me, so if I don't have time or motivation to work on it I'm not doing anything which includes merging PR's.

 As mentioned already too often: This is a free time project for me, so if I don't have time or motivation to work on it I'm not doing anything which includes merging PR's.

Sorry, I didn't mean to put any pressure: it's completely ok if this has to wait. I just wanted to make sure this wouldn't stale due to a misunderstanding on who should click merge, given it's approved and I now have the button.

 As mentioned already too often: This is a free time project for me, so if I don't have time or motivation to work on it I'm not doing anything which includes merging PR's.

Sorry, I didn't mean to put any pressure: it's completely ok if this has to wait. I just wanted to make sure this wouldn't stale due to a misunderstanding on who should click merge, given it's approved and I now have the button.

I just wanted to make sure this wouldn't stale due to a misunderstanding on who should click merge, given it's approved and I now have the button.

I would be surprised if you have merge rights already. As far as I know you are part of the diesel/reviewers team. Persons in that team get pinged by requesting a review from the corresponding team, otherwise there are no other rights. Merging PR's is a right that is granted to people in the diesel/contributors and diesel/core team. The normal process to get into one of those teams is by showing via review/contributions that you understand larger parts of the code base and have gained some trust of the members of the core team.

I just wanted to make sure this wouldn't stale due to a misunderstanding on who should click merge, given it's approved and I now have the button.

I would be surprised if you have merge rights already. As far as I know you are part of the diesel/reviewers team. Persons in that team get pinged by requesting a review from the corresponding team, otherwise there are no other rights. Merging PR's is a right that is granted to people in the diesel/contributors and diesel/core team. The normal process to get into one of those teams is by showing via review/contributions that you understand larger parts of the code base and have gained some trust of the members of the core team.

I would be surprised if you have merge rights already. As far as I know you are part of the diesel/reviewers team.

That makes sense semantically, which is why I was surprised I now have the button. ^^
I haven't tried clicking it though, so maybe it would tell me I don't have rights if I did.

I would be surprised if you have merge rights already. As far as I know you are part of the diesel/reviewers team.

That makes sense semantically, which is why I was surprised I now have the button. ^^
I haven't tried clicking it though, so maybe it would tell me I don't have rights if I did.

@Ten0 I've rechecked that now twice. That sounds strange, as you are not in the correct team, but let's just try that out on the next merge or so (I will just ping you there :shrug:)

@Ten0 I've rechecked that now twice. That sounds strange, as you are not in the correct team, but let's just try that out on the next merge or so (I will just ping you there :shrug:)


UK police data api links added

This is the site for open data about crime and policing in England, Wales and Northern Ireland.
You can download street-level crime, outcome, and stop and search data in clear and simple CSV format and explore the API containing detailed crime data and information about individual police forces and neighbourhood teams.
You can also download data on police activity, and a range of data collected under the police annual data requirement (ADR) including arrests and 101 call handling.
All the data on this site is made available under the Open Government Licence.

I've merged this but it doesn't seem to be in the Readme...

@toddmotto Thanks for the merge.  Anyways, created another one [ https://github.com/toddmotto/public-apis/pull/51 ]


If a projector countdown is projected and then stopped, the countdown does not stop on the projector

I cannot reproduce this.


Access denied due to invalid VCC-API-KEY. Make sure to provide a valid key for an active application.

Currently getting this error message upon starting the image. Has the VCC-API-KEY been invalidated?

Please pull the latest image and read the release notes

Right, just figured that out myself. So, never mind, sorry. You should mention that in the README.md too. Thanks for your great work!

No problem! Feel free to open a new issue if something isn't working.
Please don't forget to comment or close #2 if the raspberry build is working!

Yeah will do. I still have to check on #2.


Improve Player and profile

This feature is one of the most requested feature by different players and testers. Under this issue, following goals are expected to fulfill.

[x] Adding titles for players based on different activities they perform.
[x] Improving the !p or the !profile command. (Currently we need to type the fullname of a player to get information about him/her. We are planning to make a default to this command so that if a player run only the !p  command it should display his/her profile)
[ ] Improving the help so that it is easy to know more about profile and !p command

Closing this issue with the commit 285e769


simple code in JavaScript project cordova

hi
can you send me a simple code that work in java script project cordova
thank alot

Please check if this is working for you:
https://github.com/paystory-de/thermal-printer-cordova-plugin/issues/15#issuecomment-1128100770
https://github.com/paystory-de/thermal-printer-cordova-plugin/issues/9#issuecomment-993658943


Add games

[ ] Byzanz
[x] Love letter
[ ] Air, land, and sea
[ ] Parade
[ ] Red7
[ ] The crew

Message sent to RagManu re their parade art.

Check out squib and game-icons.net. They might be useful.

For the record, here's some legal guidelines re copyrights / trademarks / patents in games:

Thread on BGG.
Answer on StackExchange.


Pass the right display type to the widget element.

In this PR

While helping someone which uses a projection element with a custom display type to render each content, we needed to pass the right diplay type to widget elements.

Another suggestion

When in Design mode, because the actual Projection.Design.cshtml only uses the ContentItems collection, to prevent many handler / driver from being executed, we could use this in ProjectionElementDriver.OnDisplaying().

// after this line
context.ElementShape.ContentItems = contentItems;
// Nothing more to do if we are in "Design" mode.
if (context.DisplayType == "Design") {
    return;
}

But i didn't include this in this PR because maybe we still want to be able to override `Projection.Design.cshtml` if someone want to render more things in design mode ... So let me know.

Best

What the link between Projections, Widgets, Display Type and "Design" ?

When you edit a layout part (displayType = Design) that has a projection element which queries on contents which have widget elements ...
But regarding the Design mode it's only a suggestion, right now this PR only pass the right displayType to widget elements when rendering in the front end.


Load PS/2 keyboard mapping as a Grub module

Shouldn't be that hard, just need to read more on OSDEV about Grub first. Apperently you can just cast a *const T to a &'static T which should help greatly. Also the new keyboard changes should fit in perfectly with that.
Once I do this I'll just add another makefile option for keyboard layout e.g. kb={us,dvorak,ect.}

Fixed with 6ea074c


Ubuntu login

I've just finished the Quick Start Guide, and I'm attempting to start up my new VM from VirtualBox, but I don't know what the Ubuntu login is :(

It looks like you may have used VirtualBox and installed a new Ubuntu virtual machine.
Drupal VM builds its own fresh VM, so all you need to do (after shutting down the VM that you had built with VirtualBox) is, inside your terminal (or Cygwin/Cmder on Windows, or even PowerShell or Git Bash/Git Shell), cd into the Drupal VM download directory, then run vagrant up.
You shouldn't ever need to use VirtualBox directly for anything.
Also, if you're still having any trouble, please post what OS you're using, what happened to get you to the point where you are, etc.—it's hard to diagnose any issues just from a screenshot.

If you want to open a shell on the VM you can SSH into it using vagrant ssh. As @geerlingguy said there's no need to use VirtualBox, everything is done through Vagrant.
Ofc there is still a way you could access it through the VirtualBox GUI but that's probably not what you're actually look for. If you for some reason really do want to, the username is vagrant and there should be a private key generated in ~/.vagrant.d/insecure_private_key (I'm not sure the path is the same on Windows).
Unfortunately I'm not sure how Windows users generally use the VM, but personally I never even SSH into my VMs. After having run vagrant up the first time everything is already configured so the only thing I do 95% of the time is let my code be transfered through the shared folder. I guess front-end tooling might be more difficult on Windows though so that might be a reason for you to SSH into it.

Please open a new issue if you're finding any other issues with the setup process; thanks!

Thank you both for your help!  I will do better with providing key details when I submit issues in the future.
I have used VB in the past for Drupal development, and I did everything inside my VMs, so I imagined this as the same concept, but with everything set up for me so I don't have to configure my LAMP environment, etc., on my own.  In other words, I imagined that after getting myself set up with Drupal VM, I would be in the same sort of position as if I manually created a VM in Virtual Box and set up LAMP and everything, and then just carried on with my normal Drupal stuff, inside the VM -- with the additional functionality of portability/repeatability of the environment.  I hope I'm making some sense :)
I probably misunderstood the intended use of this tool and was trying to make it do things it wasn't meant to do -- but I'm going to keep at it at least a little while longer.  Again, thank you so much for your prompt and patient support!

For future readers, I found insecure_private_key in C:\Users\alison\.vagrant.d\.

@alisonjo2786 you could still use it with your old workflow. To enable GUI mode for the VM you can create a Vagrantfile.local next to the regular Vagrantfile with this content:
config.vm.provider :virtualbox do |v|
  v.gui = true
end

If you have that and run vagrant up, the VirtualBox GUI should open.

That's awesome, thank you so much, @oxyc !
Question about the insecure_private_key -- I have used RSA keys for authentication when connecting via SSH, but I'm not sure how to do it in my Ubuntu Login situation, or is that not what you meant?

I've never tried it but I assume if you create the Vagrantfile.local once you run vagrant up it will open a session with the user already signed it.
I'm not sure how to do this through VirtualBox directly.

As far as I could tell it did not start a session, but, I did find that the Ubuntu login is simply vagrant / vagrant, so that's something.
Now I have been welcomed and presented with a simple Ubuntu prompt:
vagrant@ubuntu:~$
so that's something!


Change Link UUID examples to uuidv4 format and remove the UUID regex pattern

The specification has examples in old format, but new UUIDs are generated using uuidv4 library.

[ ] The examples should be changed to eg. '11bf5b37-e0b8-42e0-8dcf-dc8c4aefc000'.
[ ] The link UUID pattern should be removed from specification (if needed, the application can do the validation explicitly using import { isUuid } from 'uuidv4';

Issue#1 : For the service , /v1/regard-application , in the examples of the requestbody of the callbacks

LinkUpdateNotificationCausesRequestForLinkEndPoints , shall be changed
from
link-uuid: 'alt-2-0-1-op-link-051'
to
link-uuid: '11bf5b37-e0b8-42e0-8dcf-dc8c4aefc000'
CyclicOperationCausesRequestForListOfLinks, shall be changed
from
link-uuid-list:
- 'alt-2-0-1-op-link-051'
- 'alt-2-0-1-op-link-145'
to
link-uuid-list:
- '11bf5b37-e0b8-42e0-8dcf-dc8c4aefc000'
- '11bf5b37-e0b8-42e0-8dcf-dc8c4aefc001'
CyclicOperationCausesRequestForLinkEndPoints, shall be changed
from
link-uuid: 'alt-2-0-1-op-link-051'
to
link-uuid: '11bf5b37-e0b8-42e0-8dcf-dc8c4aefc000'

Issue#2: For service /v1/regard-updated-link ,

Pattern shall be removed for request body attribute link-uuid  as per the  discussion made in the issue https://github.com/openBackhaul/ApplicationLayerTopology/issues/262 to change the uuid format.
Example shall be modified from
link-uuid: 'alt-2-0-1-op-link-051' to
link-uuid: '11bf5b37-e0b8-42e0-8dcf-dc8c4aefc000'

Changed as proposed.


MonGo Search

whilst your site is still unavailable, is this deployed on github anywhere? or instructions to self host

I've deployed the site with github pages. Thank you for feedback!


I meet a problem in training "No such file or directory: 'ckpt/TATT/model_best_acc_0.pth' ", how can I solve it ?

Thanks for you work.I meet a problem in training.How can I solve it?
First, I meet this problem.
No such file or directory: 'ckpt/TATT/

Then, I make a directory named 'TATT' in 'ckpt', but I meet another problem
No such file or directory: 'ckpt/TATT/model_best_acc_0.pth'

I'm working on my graduation project, and I need to reproduce your code, so it's important to me.Thanks for your work, and looking forward to your reply!

I solved the problem, which was caused by my lack of cuda memory


Missing slash in generated url for actions

The code generated for Power Automate has an issue when building the url for the action to perform. There is a missing Slash

Deployment: Online
DB Version: 9.2.22033.152
Connection Controls Version: 1.2022.4.48
XrmToolBox Version: 1.2022.2.54
Tool Version: 1.2021.12.1

I tried to reproduce it on a few organizations but didn't have the issue above. It's probably related to ConnectionDetail.WebApplicationUrl that I'm using for the base URL (which has / in the end in my case).
Is it maybe better to put ConnectionDetail.OriginalUrl instead and add / at the end since it is the same but without the ending slash?

I guess the best is to test if the url ends with a slash and add it if it is missing

Good point, I'll make it that way.


Indentation issues result when more complicated option aliases are used

Demo (using Ruby 2.7.4):
require 'thor'

class MyThor < Thor
  default_command :go
  desc '', 'Do some stuff'
  long_desc <<~LONG_DESC
    Do some stuff.
  LONG_DESC

  option "verbose", desc: "verbose logging", aliases: %w(-v), type: :boolean
  option "debug", desc: "debug mode", type: :boolean

  # If this next option is commented out, the 'help' output lines them up nicely
  #
  # Options:
  #   -v, [--verbose], [--no-verbose]  # verbose logging
  #       [--debug], [--no-debug]      # debug mode
  #
  # If it is present, the indentation is uneven:
  #
  # Options:
  #   -v, [--verbose], [--no-verbose]                # verbose logging
  #           [--debug], [--no-debug]                # debug mode
  #   -p, --post-to-gh, [--post-to-github=USERNAME]  # post to a GitHub repository
  #
  # Preferred output:
  #
  # Options:
  #   -v, [--verbose], [--no-verbose]                           # verbose logging
  #       [--debug], [--no-debug]                               # debug mode
  #   -p, [--post-to-gh=USERNAME], [--post-to-github=USERNAME]  # post to a GitHub repository

  option "post-to-github", desc: "post to a GitHub repository", aliases: %w(-p --post-to-gh), banner: 'USERNAME'

  def go
    puts("Options: #{options.inspect}")
  end
end

MyThor.start(ARGV)

The options work as expected. It's only the "help go" output that has an issue.
Thanks!

Would be great to see a pull request fixing this! 😄


repo clone problem

Hello,
i've just cloned your repo but i get this error:
Failed to find target with hash string 'android-23' in: C:\Users\othman\AppData\Local\Android\Sdk
can you please tell me how to solve this?

Have you installed Android 6.0 on your computer?: https://developer.android.com/studio/releases/platforms#6.0

@nya-elimuai Thank you!
but now there is a new error in the app build.gradle , "Failed to resolve: com.android.support:appcompat-v7:23.3.0"
any idea what this could be?

Check whether you have updated libraries and android studio. It worked with me.

same error comes when i was trying
"Failed to resolve: com.android.support:appcompat-v7:23.3.0"


write_spss drops first dichot on some MRSETs

write_spss sometimes results in a SAV file in which some MRSETs are missing their first dichotomous variable reference.

After a lot of investigating it turns out this behaviour is not controlled by quantipy, but instead by savReaderWriter.
It turns out MRSETS don't get correctly constructed when there is more than 1 string variable also in the output, but it has nothing to do with the data in the string variable or its label. There's an obscure connection between there being string variables (actually, more than 1 of them) and MRSETS  sometimes (no discernable pattern) missing their first dichot reference.
The solution to this problem is (unfortunately) to always export string columns separately to the main data.

Resolved by #310.


fix inCompartment for a collection of owners

concerning issue: https://github.com/jamesagnew/hapi-fhir/issues/778

Sorry, some merge error caused the closing of this PR. Reopen in https://github.com/jamesagnew/hapi-fhir/pull/804


v1.9: Update Cilium base images

Update v1.9 base runtime images to pick up the latest security updates.

No need to run full CI; smoke tests successfully pulled new image, built it, and checked conformance with basic connectivity suite. Good to merge.


Not able to see aws account details in security monkey dashboard

Please make sure that you have checked the boxes:

[x] Review the Quickstart guide
[x] Search for both open and closed issues regarding the problem you are experiencing
[x] For permissions issues (Access Denied and credential related errors), please refer to the requisite docs before submitting an issue:
AWS, GCP, OpenStack, GitHub

Description of issue:
Hi Team,
I have installed security monkey by following the blog https://github.com/Netflix/security_monkey/blob/develop/docs/iam_aws.md, can able to view the dashboard, have added one test account to view the detail.
But i am not able to see any information on Dashboard for the  account what i have configured, can anyone guide on how to fix or is there anything which i am missing in configuration.

Please review: https://github.com/Netflix/security_monkey/blob/develop/docs/troubleshooting.md#common-issues

Hi Mike,
I have loaded  the supervisor worker, scheduler and ui as documented, but i am not able to start these processes under scheduler. I am not able to find the log for this, can you help me out in fixing this.

All instances must be launched with the SecurityMonkeyInstanceProfile IAM role as documented in the AWS IAM documentation.


No Transparency on Windowlogged Panes (1.19.2, 0.1.1)

Title is self explanatory

Here is my modlist, incase one of them is the problem
Create Pals Dev.txt

Also have this issue for vanilla dyed glass

Nevermind, I've found the culprit.
It turns out it's Sodium, (or the forge port, embeddium)
This is an issue with Sodium has with transparency frequently.


etcd 3.0.1 cannot run normal on Windows 7 64-bit

https://github.com/coreos/etcd/issues/5852
after  manually renamed default.etcd/member/wal.tmp to default.etcd/member/wal. Then etcd will start normally.
but can not provide the service
E:\etcd>etcdctl member list
this conmand run hang
http://localhost:2379/
run hang  in IE
E:\etcd>etcdctl set /testdir/testkey "Hello world"
run hang
E:\etcd>etcd
2016-07-06 15:54:12.173012 I | etcdmain: etcd Version: 3.0.1
2016-07-06 15:54:12.254269 I | etcdmain: Git SHA: a4a52cb
2016-07-06 15:54:12.254269 I | etcdmain: Go Version: go1.6.2
2016-07-06 15:54:12.254269 I | etcdmain: Go OS/Arch: windows/amd64
2016-07-06 15:54:12.255248 I | etcdmain: setting maximum number of CPUs to 4, total number of available CPUs is 4
2016-07-06 15:54:12.255248 W | etcdmain: no data-dir provided, using default data-dir ./default.etcd
2016-07-06 15:54:12.255248 N | etcdmain: the server is already initialized as member before, starting as etcd member...
2016-07-06 15:54:12.262101 I | etcdmain: listening for peers on http://localhost:2380
2016-07-06 15:54:12.265038 I | etcdmain: listening for client requests on localhost:2379
2016-07-06 15:54:12.270912 W | snap: skipped unexpected non snapshot file db.lock
2016-07-06 15:54:12.270912 I | etcdserver: name = default
2016-07-06 15:54:12.270912 I | etcdserver: data dir = default.etcd
2016-07-06 15:54:12.271891 I | etcdserver: member dir = default.etcd/member
2016-07-06 15:54:12.271891 I | etcdserver: heartbeat = 100ms
2016-07-06 15:54:12.271891 I | etcdserver: election = 1000ms
2016-07-06 15:54:12.271891 I | etcdserver: snapshot count = 10000
2016-07-06 15:54:12.271891 I | etcdserver: advertise client URLs = http://localhost:2379
2016-07-06 15:54:12.275807 I | etcdserver: restarting member 8e9e05c52164694d in cluster cdf818194e3a8c32 at commit index 0
2016-07-06 15:54:12.278744 I | raft: 8e9e05c52164694d became follower at term 0
2016-07-06 15:54:12.282660 I | raft: newRaft 8e9e05c52164694d [peers: [], term: 0, commit: 0, applied: 0, lastindex: 0, lastterm: 0]
2016-07-06 15:54:12.318883 I | etcdserver: starting server... [version: 3.0.1, cluster version: to_be_decided]
2016-07-06 15:54:12.320841 E | etcdserver: cannot monitor file descriptor usage (cannot get FDUsage on windows)
2016-07-06 15:54:19.339292 E | etcdserver: publish error: etcdserver: request timed out
2016-07-06 15:54:26.358722 E | etcdserver: publish error: etcdserver: request timed out
2016-07-06 15:54:33.381089 E | etcdserver: publish error: etcdserver: request timed out
2016-07-06 15:54:40.400519 E | etcdserver: publish error: etcdserver: request timed out
2016-07-06 15:54:47.421907 E | etcdserver: publish error: etcdserver: request timed out
2016-07-06 15:54:54.441337 E | etcdserver: publish error: etcdserver: request timed out

I rebuilt etcd just now from master (commit 234c30c) and got the following similar output (but with more verbose errors):
C:\etcd\bin>etcdctl cluster-health
member 8e9e05c52164694d is healthy: got healthy result from http://localhost:2379
cluster is healthy

C:\etcd\bin>etcdctl member list
client: etcd cluster is unavailable or misconfigured; error #0: client: endpoint http://localhost:2379 exceeded header timeout

C:\etcd\bin>etcdctl set /testdir/testkey "Hello world"
Error:  client: etcd cluster is unavailable or misconfigured; error #0: client: endpoint http://localhost:2379 exceeded header timeout

The server output itself seems equivalent:
C:\etcd\bin>etcd
2016-07-06 01:48:41.889331 I | etcdmain: etcd Version: 3.0.0+git
2016-07-06 01:48:41.889831 I | etcdmain: Git SHA: 234c30c
2016-07-06 01:48:41.890331 I | etcdmain: Go Version: go1.6.2
2016-07-06 01:48:41.890331 I | etcdmain: Go OS/Arch: windows/amd64
2016-07-06 01:48:41.890831 I | etcdmain: setting maximum number of CPUs to 8, total number of available CPUs is 8
2016-07-06 01:48:41.891331 W | etcdmain: no data-dir provided, using default data-dir ./default.etcd
2016-07-06 01:48:41.891831 N | etcdmain: the server is already initialized as member before, starting as etcd member...
2016-07-06 01:48:41.895332 I | etcdmain: listening for peers on http://localhost:2380
2016-07-06 01:48:41.895832 I | etcdmain: listening for client requests on localhost:2379
2016-07-06 01:48:41.901832 W | snap: skipped unexpected non snapshot file db.lock
2016-07-06 01:48:41.901832 I | etcdserver: name = default
2016-07-06 01:48:41.902332 I | etcdserver: data dir = default.etcd
2016-07-06 01:48:41.902332 I | etcdserver: member dir = default.etcd/member
2016-07-06 01:48:41.902832 I | etcdserver: heartbeat = 100ms
2016-07-06 01:48:41.902832 I | etcdserver: election = 1000ms
2016-07-06 01:48:41.903333 I | etcdserver: snapshot count = 10000
2016-07-06 01:48:41.903833 I | etcdserver: advertise client URLs = http://localhost:2379
2016-07-06 01:48:41.904833 I | etcdserver: restarting member 8e9e05c52164694d in cluster cdf818194e3a8c32 at commit index 65
2016-07-06 01:48:41.904833 I | raft: 8e9e05c52164694d became follower at term 2
2016-07-06 01:48:41.905333 I | raft: newRaft 8e9e05c52164694d [peers: [], term: 2, commit: 65, applied: 0, lastindex: 65, lastterm: 2]
2016-07-06 01:48:41.920835 I | etcdserver: starting server... [version: 3.0.0+git, cluster version: to_be_decided]
2016-07-06 01:48:41.921335 E | etcdserver: cannot monitor file descriptor usage (cannot get FDUsage on windows)
2016-07-06 01:48:41.921835 I | membership: added member 8e9e05c52164694d [http://localhost:2380] to cluster cdf818194e3a8c32
2016-07-06 01:48:41.921835 N | membership: set the initial cluster version to 3.0
2016-07-06 01:48:42.421898 I | api: enabled capabilities for version 3.0
2016-07-06 01:48:42.706935 I | raft: 8e9e05c52164694d is starting a new election at term 2
2016-07-06 01:48:42.706935 I | raft: 8e9e05c52164694d became candidate at term 3
2016-07-06 01:48:42.707435 I | raft: 8e9e05c52164694d received vote from 8e9e05c52164694d at term 3
2016-07-06 01:48:42.707435 I | raft: 8e9e05c52164694d became leader at term 3
2016-07-06 01:48:42.707935 I | raft: raft.node: 8e9e05c52164694d elected leader 8e9e05c52164694d at term 3
2016-07-06 01:48:42.708935 I | etcdserver: published {Name:default ClientURLs:[http://localhost:2379]} to cluster cdf818194e3a8c32
2016-07-06 01:48:42.708935 I | etcdmain: ready to serve client requests
2016-07-06 01:48:42.709435 N | etcdmain: serving insecure client requests on localhost:2379, this is strongly discouraged!

I removed my original reply - it confirmed the problem you're seeing but as it became a bit intermittent I  decided to delve deeper. I think I have your answer (though this goes back to the need to document the Windows use case for the Windows deliverables).
localhost and <IP_ADDRESS> aren't always quite the same on Windows 7 and newer: for me, localhost is the IPv6 ::1: while the only IPv4 loopback is <IP_ADDRESS>.  My hosts file (C:\Windows\System32\drivers\etc\hosts) is a bit ambiguous on this point, having both the ipv4 and ipv6 loopbacks:
<IP_ADDRESS>       localhost
::1             localhost

I tried combinations of endpoints - etcd and etcdctl both using localhost, one using localhost and the other using <IP_ADDRESS>, but the only way I got it to work consistently was to explicitly use only the  <IP_ADDRESS> IPv4 endpoints. Therefore, you'd start the server with:
etcd --debug --advertise-client-urls http://<IP_ADDRESS>:2379
and access it like so:
etcdctl --debug --endpoints http://<IP_ADDRESS>:2379 <command>
e.g.:
etcdctl --debug --endpoints http://<IP_ADDRESS>:2379 member list
etcdctl --debug --endpoints http://<IP_ADDRESS>:2379 set /message "Hello world"
This is incomplete, as you'll probably need to also set the other parameters that refer to localhost to <IP_ADDRESS> as well... but for one etcd server and one etcdctl client this will suffice.
Note: debug option was added for extra info.

One interesting side-note - if there is no service on port 4001, calls via etcdctl using the default values often take an extra second while it tries and fails to use that particular endpoint (if it polls it first). I'd think this would slow things down a bit if nothing else...

Finally, something that gets to the heart of how weird localhost can be when it's defined for both protocols. I started etcd with no special options, so it refers to localhost, and then ran the following commands:
etcdctl with localhost alone simply never works:
etcdctl --debug --endpoints http://localhost:2379 set /message "Hello world"
start to sync cluster using endpoints(http://localhost:2379)
cURL Command: curl -X GET http://localhost:2379/v2/members
got endpoints() after sync
Cluster-Endpoints:
cURL Command: curl -X PUT /v2/keys/message -d "value=Hello+world"
Error:  client: etcd cluster is unavailable or misconfigured; error #0: unsupported protocol scheme ""

error #0: unsupported protocol scheme ""

Neither does etcdctl with <IP_ADDRESS> alone:
C:\bin\etcd3>cd /D C:\bin\etcd3 && etcdctl --debug --endpoints http://<IP_ADDRESS>:2379 set /message "Hello world"
start to sync cluster using endpoints(http://<IP_ADDRESS>:2379)
cURL Command: curl -X GET http://<IP_ADDRESS>:2379/v2/members
got endpoints(http://localhost:2379) after sync
Cluster-Endpoints: http://localhost:2379
cURL Command: curl -X PUT http://localhost:2379/v2/keys/message -d "value=Hello+world"
Error:  client: etcd cluster is unavailable or misconfigured; error #0: client: endpoint http://localhost:2379 exceeded header timeout

error #0: client: endpoint http://localhost:2379 exceeded header timeout

But combine them and they often work - as long as it happens to poll (and fail) localhost first:
etcdctl --debug --endpoints http://localhost:2379,http://<IP_ADDRESS>:2379 set /message "Hello world"
start to sync cluster using endpoints(http://<IP_ADDRESS>:2379,http://localhost:2379)
cURL Command: curl -X GET http://<IP_ADDRESS>:2379/v2/members
got endpoints(http://localhost:2379) after sync
Cluster-Endpoints: http://localhost:2379
cURL Command: curl -X PUT http://localhost:2379/v2/keys/message -d "value=Hello+world"
Error:  client: etcd cluster is unavailable or misconfigured; error #0: client: endpoint http://localhost:2379 exceeded header timeout

error #0: client: endpoint http://localhost:2379 exceeded header timeout

etcdctl --debug --endpoints http://localhost:2379,http://<IP_ADDRESS>:2379 set /message "Hello world"
start to sync cluster using endpoints(http://localhost:2379,http://<IP_ADDRESS>:2379)
cURL Command: curl -X GET http://localhost:2379/v2/members
cURL Command: curl -X GET http://<IP_ADDRESS>:2379/v2/members
got endpoints(http://localhost:2379) after sync
Cluster-Endpoints: http://localhost:2379
cURL Command: curl -X PUT http://localhost:2379/v2/keys/message -d "value=Hello+world"
Hello world

It's quite repeatable if the two attempts are in the same program instance, and makes me wonder if this might be a golang quirk as well, if not just a Windows thing.

thanks  MartyMacGyver  for deep dive into the issue.
but the configuration and commands cannot run right in my env.
hosts file configed as follow:
<IP_ADDRESS>  localhost
start etcd:
E:\etcd>etcd --debug --advertise-client-urls http://<IP_ADDRESS>:2379
2016-07-08 09:32:45.902446 I | etcdmain: etcd Version: 3.0.1
2016-07-08 09:32:45.989933 I | etcdmain: Git SHA: a4a52cb
2016-07-08 09:32:45.989933 I | etcdmain: Go Version: go1.6.2
2016-07-08 09:32:45.989933 I | etcdmain: Go OS/Arch: windows/amd64
2016-07-08 09:32:45.989933 I | etcdmain: setting maximum number of CPUs to 4, total number of available CPUs is 4
2016-07-08 09:32:45.989933 W | etcdmain: no data-dir provided, using default data-dir ./default.etcd
2016-07-08 09:32:45.989933 N | etcdmain: the server is already initialized as member before, starting as etcd member...
2016-07-08 09:32:45.997887 I | etcdmain: listening for peers on http://localhost:2380
2016-07-08 09:32:46.005840 I | etcdmain: listening for client requests on localhost:2379
2016-07-08 09:32:46.013793 W | snap: skipped unexpected non snapshot file db.lock
2016-07-08 09:32:46.013793 I | etcdserver: name = default
2016-07-08 09:32:46.013793 I | etcdserver: data dir = default.etcd
2016-07-08 09:32:46.021747 I | etcdserver: member dir = default.etcd/member
2016-07-08 09:32:46.029700 I | etcdserver: heartbeat = 100ms
2016-07-08 09:32:46.029700 I | etcdserver: election = 1000ms
2016-07-08 09:32:46.030694 I | etcdserver: snapshot count = 10000
2016-07-08 09:32:46.038648 I | etcdserver: advertise client URLs = http://<IP_ADDRESS>:2379
2016-07-08 09:32:46.051572 I | etcdserver: restarting member 8e9e05c52164694d in cluster cdf818194e3a8c32 at commit index 0
2016-07-08 09:32:46.051572 I | raft: 8e9e05c52164694d became follower at term 0
2016-07-08 09:32:46.051572 I | raft: newRaft 8e9e05c52164694d [peers: [], term: 0, commit: 0, applied: 0, lastindex: 0, lastterm: 0]
2016-07-08 09:32:46.075433 I | etcdserver: starting server... [version: 3.0.1, cluster version: to_be_decided]
2016-07-08 09:32:46.083386 E | etcdserver: cannot monitor file descriptor usage (cannot get FDUsage on windows)
2016-07-08 09:32:53.219578 E | etcdserver: publish error: etcdserver: request timed out
2016-07-08 09:33:00.347819 E | etcdserver: publish error: etcdserver: request timed out
E:\etcd>etcdctl --debug member list
start to sync cluster using endpoints(http://<IP_ADDRESS>:2379,http://<IP_ADDRESS>:4001)
cURL Command: curl -X GET http://<IP_ADDRESS>:2379/v2/members
cURL Command: curl -X GET http://<IP_ADDRESS>:4001/v2/members
and then hang
E:\etcd>etcdctl --debug set /message "Hello world"
start to sync cluster using endpoints(http://<IP_ADDRESS>:2379,http://<IP_ADDRESS>:4001)
cURL Command: curl -X GET http://<IP_ADDRESS>:2379/v2/members
cURL Command: curl -X GET http://<IP_ADDRESS>:4001/v2/members
hang
but stop etcd
etcdctl --debug member list and etcdctl --debug set /message "Hello world" stopped with the following message
Error:  dial tcp <IP_ADDRESS>:4001: connectex: No connection could be made because the target machine actively refused it.
E:\etcd>etcd --debug --advertise-client-urls http://<IP_ADDRESS>:4001
2016-07-08 10:02:05.564094 I | etcdmain: etcd Version: 3.0.1
2016-07-08 10:02:05.658249 I | etcdmain: Git SHA: a4a52cb
2016-07-08 10:02:05.666095 I | etcdmain: Go Version: go1.6.2
2016-07-08 10:02:05.666095 I | etcdmain: Go OS/Arch: windows/amd64
2016-07-08 10:02:05.673942 I | etcdmain: setting maximum number of CPUs to 4, total number of available CPUs is 4
2016-07-08 10:02:05.673942 W | etcdmain: no data-dir provided, using default data-dir ./default.etcd
2016-07-08 10:02:05.681788 N | etcdmain: the server is already initialized as member before, starting as etcd member...
2016-07-08 10:02:05.689634 I | etcdmain: listening for peers on http://localhost:2380
2016-07-08 10:02:05.697481 I | etcdmain: listening for client requests on localhost:2379
2016-07-08 10:02:05.705327 W | snap: skipped unexpected non snapshot file db.lock
2016-07-08 10:02:05.713173 I | etcdserver: name = default
2016-07-08 10:02:05.713173 I | etcdserver: data dir = default.etcd
2016-07-08 10:02:05.713173 I | etcdserver: member dir = default.etcd/member
2016-07-08 10:02:05.721020 I | etcdserver: heartbeat = 100ms
2016-07-08 10:02:05.721020 I | etcdserver: election = 1000ms
2016-07-08 10:02:05.721020 I | etcdserver: snapshot count = 10000
2016-07-08 10:02:05.721020 I | etcdserver: advertise client URLs = http://<IP_ADDRESS>:4001
2016-07-08 10:02:05.728866 I | etcdserver: restarting member 8e9e05c52164694d in cluster cdf818194e3a8c32 at commit index 0
2016-07-08 10:02:05.728866 I | raft: 8e9e05c52164694d became follower at term 0
2016-07-08 10:02:05.728866 I | raft: newRaft 8e9e05c52164694d [peers: [], term: 0, commit: 0, applied: 0, lastindex: 0, lastterm: 0]
2016-07-08 10:02:05.756328 I | etcdserver: starting server... [version: 3.0.1, cluster version: to_be_decided]
2016-07-08 10:02:05.768097 E | etcdserver: cannot monitor file descriptor usage (cannot get FDUsage on windows)
2016-07-08 10:02:12.801326 E | etcdserver: publish error: etcdserver: request timed out
2016-07-08 10:02:19.838478 E | etcdserver: publish error: etcdserver: request timed out
2016-07-08 10:02:26.872688 E | etcdserver: publish error: etcdserver: request timed out

There may be other problems with 3.0.1 that are confounding this.
Also - is 2.3.7 working for you?

thanks MartyMacGyver .
for security reason, I can not access dropbox in our office
I will install dropbox in my home and test your compiled etcd windows version
I have not test 2.3.7 version

It's probably easier to simply build your own copy of it from here using Go 1.6.2. Clone this repo to an NTFS drive and run "build.bat". As long as you have PowerShell and Git installed that should work.

Also, if it helps, here's a link that may be easier to access:
https://drive.google.com/open?id=0B3dTI7CdidfqRG1Xb2NGS2ZpZms

@sinsharat I know you have Windows env. Can you check if this is still a problem for etcd?

@xiang90 sure will look into it.

@xiang90 : The following were my observations with etcd Version: 3.1.0-alpha.1+git:

wal issue referenced in earlier issue #5852 doesn't exist anymore.
with etcd V3 api (ETCDCTL_API=3) there are no issues, it works perfectly fine.
There is no issue with etcd server startup when using v2 version api, also etcdctl works fine when complete endpoint is specified. For example :
PS C:\sharat\workspace\go-projects\src\github.com\coreos\etcd\bin> .\etcdctl.exe --endpoint=http://localhost:2379 member list 8e9e05c52164694d: name=default peerURLs=http://localhost:2380 clientURLs=http://localhost:2379 isLeader=true PS C:\sharat\workspace\go-projects\src\github.com\coreos\etcd\bin> .\etcdctl.exe --endpoint=http://localhost:2379 set foo bar bar
The issue for client for v2 api not working on starting default etcd is only when the endpoint is not specified. It gives the below error:
`PS C:\sharat\workspace\go-projects\src\github.com\coreos\etcd\bin> .\etcdctl.exe member list
client: etcd cluster is unavailable or misconfigured; error #0: client: endpoint http://localhost:2379 exceeded header timeout

PS C:\sharat\workspace\go-projects\src\github.com\coreos\etcd\bin> .\etcdctl.exe set foo bar
Error:  client: etcd cluster is unavailable or misconfigured; error #0: client: endpoint http://localhost:2379 exceeded header timeout
error #0: client: endpoint http://localhost:2379 exceeded header timeout`

@xiang90 Also my suggestion is since default host is not supported on windows, better to modify logging code at places where we print <IP_ADDRESS> by default.  May be by using windows specific files to avoid confusion while printing details during etcd startup. Also if you want i can look into the reason for etcdctl not working without endpoint for v2 version.
Need your opinion on this.
Thanks!

@sinsharat Thanks for checking this.

Also my suggestion is since default host is not supported on windows, better to modify logging code at places where we print <IP_ADDRESS> by default.

How about adding windows specific code to change the default host to <IP_ADDRESS> from localhost? Go supports build tag, so it is pretty easy to achieve.

@xiang90 yes that's exactly what i had meant by adding windows specific files. I was already going through the code and had observed that the values are not hard-coded. Hence i referred to the earlier logs and figured out that there is already some code written for the same. Will analyze this in detail in the morning, fix it and raise PR for this.
Thanks!

@sinsharat Sounds great! Thanks!

@sinsharat Shall we close this issue? It seems etcd now can run on windows?

@xiang90 yes already tested. With the fix provided it works fine. It can be closed.
Thanks!


Unhide manage species lists link

The "Manage Species List" link was not showing up for users who owned no species lists, but who were on projects which did.  Instead of adding even more expensive and complex logic to decide whether to show the link, I just unhid it for all logged-in users.
Also, since I was in the area, I checked on the "View Notifications" page.  It has never once been used in the past two years.  So I ripped out all of that code throughout.  With many apologies to Nathan.  I admit to partial responsibility, since I did not understand the ultimate goal of this Notifications code right from the beginning, and I think that's at least partly why it failed to get properly utilized.  But in any case, a high priority these days is to simplify show_observation, and this moves us in that direction at least.  If and when we do finally decide to pursue Nathan's goal of handling notifications better, we can redesign it in such a way to minimize the impact on show_observation -- that is, minimize the number of times and places we need to do queries to figure out if the user has unshown notifications.

Coverage increased (+0.02%) to 93.059% when pulling 0ddbc9d4afe8a3f7f6f6c88c4bb252451e10a11d on unhide-manage-species-lists-link into 217ce4bed9ff92f3d11bf7e3a625cc290b173a50 on master.


chore: go-demo-1571047433 to 0.0.2

chore: Promote go-demo-1571047433 to version 0.0.2

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Added parameter based timeouts setting + missing stream.close()

Current coverage is 91.44% (diff: 50.00%)

Merging #11 into master will decrease coverage by <.01%

@@             master        #11   diff @@
==========================================
  Files            17         17
  Lines          2562       2561     -1
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           2343       2342     -1
  Misses          219        219
  Partials          0          0

Powered by Codecov. Last update 00b8e37...cfe6c53


[fftw3[openmp]] build failure

Host Environment

OS: Windows 10 1909
Compiler: MSVC 19.25.28611

To Reproduce
Steps to reproduce the behavior:
./vcpkg install fftw3[openmp]
Failure logs
config-x64-windows-out.log
install-x64-windows-dbg-out.log

openmp does not work in "windows"
vcpkg install fftw3[core]
or
fftw3[threads,core,avx2,sse2,avx,sse]

Related: https://github.com/FFTW/fftw3/issues/142


ImportError: cannot import name 'string_int_label_map_pb2' from 'object_detection.protos' (..\object_detection\protos_init_.py)

`---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
 in
11
12 # Import utilites
---> 13 from utils import label_map_util
14 from utils import visualization_utils as vis_util
15
~\Desktop\Mask Detector\master\research\object_detection\utils\label_map_util.py in
25 import tensorflow as tf
26 from google.protobuf import text_format
---> 27 from object_detection.protos import string_int_label_map_pb2
28
29
ImportError: cannot import name 'string_int_label_map_pb2' from 'object_detection.protos' (..\object_detection\protos_init_.py)
`
I have tensorflow version '1.15.0'

Download that .py file and put in protos folder
On Tue, 26 May, 2020, 11:08 PM Sunny Kumar<EMAIL_ADDRESS>wrote:

`---------------------------------------------------------------------------
ImportError Traceback (most recent call last)
in
11
12 # Import utilites
---> 13 from utils import label_map_util
14 from utils import visualization_utils as vis_util
15
~\Desktop\Mask
Detector\master\research\object_detection\utils\label_map_util.py in
25 import tensorflow as tf
26 from google.protobuf import text_format
---> 27 from object_detection.protos import string_int_label_map_pb2
28
29
ImportError: cannot import name 'string_int_label_map_pb2' from
'object_detection.protos' (..\object_detection\protos_init_.py)
`
I have tensorflow version '1.15.0'
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Spidy20/face_mask_detection/issues/9, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKA3TJALPW7EXDKQ7GIHQWLRTP473ANCNFSM4NKYXHBQ
.

Now this error..
`~\Desktop\Mask Detector\master\research\object_detection\protos\string_int_label_map_pb2.py in
19   syntax='proto2',
20   serialized_options=None,
---> 21   create_key=_descriptor._internal_create_key,
22   serialized_pb=b'\n2object_detection/protos/string_int_label_map.proto\x12\x17object_detection.protos"\xc0\x01\n\x15StringIntLabelMapItem\x12\x0c\n\x04name\x18\x01 \x01(\t\x12\n\n\x02id\x18\x02 \x01(\x05\x12\x14\n\x0c\x64isplay_name\x18\x03 \x01(\t\x12M\n\tkeypoints\x18\x04 \x03(\x0b\x32:.object_detection.protos.StringIntLabelMapItem.KeypointMap\x1a(\n\x0bKeypointMap\x12\n\n\x02id\x18\x01 \x01(\x05\x12\r\n\x05label\x18\x02 \x01(\t"Q\n\x11StringIntLabelMap\x12<\n\x04item\x18\x01 \x03(\x0b\x32..object_detection.protos.StringIntLabelMapItem'
23 )
AttributeError: module 'google.protobuf.descriptor' has no attribute '_internal_create_key'`

Can you please share your protoc and protobuf version that you are using.

You need to put this file in protos folder
https://github.com/datitran/object_detector_app/blob/master/object_detection/protos/string_int_label_map_pb2.py
On Wed, May 27, 2020 at 1:03 PM Sunny Kumar<EMAIL_ADDRESS>wrote:

Can you please share your protoc and protobuf version that you are using.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Spidy20/face_mask_detection/issues/9#issuecomment-634483547,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKA3TJCTQWU43DYSL545DM3RTS63RANCNFSM4NKYXHBQ
.

Thank you for the quick response. The code is working after I upgraded my protobuf.
I have tested on couple of images and number of detection is more than 1. sharing the image for reference.


chore: promote jx-aspnet-app002 to version 0.0.17

this commit will trigger a pipeline to generate the actual kubernetes resources to perform the promotion which will create a second commit on this Pull Request before it can merge

jx-aspnet-app002
Changes in version 0.0.17
Chores

release 0.0.17 (jenkins-x-bot)
add variables (jenkins-x-bot)

Other Changes
These commits did not use Conventional Commits formatted messages:

Revert "修改应用" (huyc04)
Update README.md (hughexp)
修改应用 (huyc04)

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


[DPE-4956] Stabilise integration tests

See https://github.com/canonical/kafka-k8s-operator/pull/125 for the changelog
In addition, here are the additional fixes specific to VM:

Fixed jmx_prometheus jar path
Used workload.restart instead of start in balancer/start event handler. This is fine (and recommended) to use start on k8s because it does restart the service, whereas on VM we did not pick up the configuration changes

I'm not really sure I can provide any sensible comment here. I'm happy for @marcoppenheimer and @zmraul to review this, given they have waaay more background and expertise on the code base / integration testing issues