segmentGrow example

Hi, I appreciate your abstraction lib.
I'm trying to use this segmentGrow facility but don't know how to adapt to my scenario. Can you provide an example?
I got this plunker to show you what I'm doing. It is a street route, it is buggy but it is a start.

Hi,
segmentGrow is something I'm not really happy with.
I was looking for a way to animate the drawing of a straight line, so I invented this segmentGrow thing. What it does is that it takes a target Point as value, and adds many small segments to the Path until it reaches said target Point. It kinda works on small Paths, but it's a huge performance hit if you have many Paths animated that way.
Anyway you wanted an example, so there you go:
http://jsbin.com/gafaro/1/edit?js,output
  var line = new paper.Path();
  line.strokeColor = "black";
  line.add(new paper.Point(20,20));
  line.animate({
    properties: {
      segmentGrow: new paper.Point(20,100) // target
    },
    settings: {
      duration: 400,
      complete: function() {
        this.animate({
          properties: {
            segmentGrow: new paper.Point(30,20) // new target
          }
        });
      }
    }
  });

Hope this helps.
Cheers


Vectorized aggregation

This changes aggregation to work with shares in $\mathbb{F}_2$ rather than $\mathbb{F}_p$ (required to support DZKP-based malicious security), and vectorizes it.

there are plenty of changes coming from 1008 that add some noise - can you rebase it on top of main after #1008 is merged?

Capturing some open items:

[ ] Test for aggregate_values
[ ] Saturating addition
[ ] dummy_fn changes (Default? rename?)
[ ] Aggregation transpose test

Double-posting a comment I made in a thread here for better visibility:
My plan is to defer this [using select in move_value_to_bucket] and saturating addition to a follow-up. I am working on the (rather gruesome) merge conflicts presently, but I'm hoping this can be ready tonight or tomorrow.

I think it's fine to defer the saturation addition as well.

https://github.com/private-attribution/ipa/pull/1046 has the promised follow-up changes.


Update distributed-tracing-instrumentation-walkthroughs.md

Summary
Include link cap in documentation.
Fixes https://github.com/dotnet/docs/issues/39397

Internal previews

📄 File
🔗 Preview link

docs/core/diagnostics/distributed-tracing-instrumentation-walkthroughs.md
Adding distributed tracing instrumentation

ping @tommcdon
This LGTM. Can you verify that it's correct?


Mails going into spam folder

Hi,
I'm new to this. I install Postal today. Thanks for the good documentation.
I setup my first testserver and send out an testmail. This mail are going directly into the spam folder of the receipient.
Any ideas why?
When I send through MailGun, I have no issues.
Torsten

Send over some headers of the email and I might be able to tell you why that's happening.

Hmm ... I just setup an 2nd domain and there it's working fine.
I will check it with a few more domains into the nexr days. If I got still errors, I will post the headers.
Thanks.

Righto.


Support for loading templates from the file system or from an embedded representation in the app's binary.

This PR contains the following changes:

Integration of default diagnostic templates into the application binary.
A new command weaver diagnostic init to locally create diagnostic templates, allowing the user to override them or define new targets.
A mechanism to detect the presence of local templates for rendering diagnostic messages.
An infrastructure to test the execution of CLI commands, their outputs, and their exit codes.

Closing: #167

Looks like the docker build also needs tobe updated for the rename of the templates.


zephyr: allow dynamic numeration of flash_areas

Zephyr flash_map reworks caused that areas id exact number are
assigned dynamically.
This patch i counterpart to
https://github.com/zephyrproject-rtos/zephyr/pull/8837
Merge after above PR.
Signed-off-by: Andrzej Puzdrowski<EMAIL_ADDRESS>

Can be merged! Related Zephyr PRs were just merged to the zephyr master.

@mbolivar  ^^ Plasn to do it ASAP.

@carlescufi Can you approve?


Endpoint for Elasticsearch does not work

Hi,
in the configuration files of the kube-burner we have this   configuration point

esServers: [http://elastic-elk.apps.rsevilla.kube-burner.com].

The thing is that the address http://elastic-elk.apps.rsevilla.kube-burner.com does not lead anywhare.
Perhaps there is a new updated address?

Hi,
in the configuration files of the kube-burner we have this configuration point
* esServers: [http://elastic-elk.apps.rsevilla.kube-burner.com].

The thing is that the address http://elastic-elk.apps.rsevilla.kube-burner.com does not lead anywhare. Perhaps there is a new updated address?

That's a fake address just for demonstration purposes.


Added argument to initialize Euclidean metric for samplers

Submisison Checklist

[x] Run tests: ./runCmdStanTests.py src/test
[x] Declare copyright holder and open-source license: see below

Summary:
This is the CmdStan interface component of: https://github.com/stan-dev/stan/pull/2260
This should allow initialization of Euclidean metric for samplers with an R dump file
Questions:

Where should the metric_file argument go? I think it makes sense under "init" but I didn't put it there cause that would mean breaking a bunch of other stuff (that depends on passing in init=whatever arguments). I just made it a high level argument for now.

I only added check-that-it's-working tests. Lemme know if I should add check-if-it's-failing tests. I figured the Stan tests should handle most of that though (I didn't really add any functionality beyond the interface)

Should the unit_e samplers allow initialization of their Euclidean metrics? The interface isn't there for them.

Nomenclature-wise, are we setting the metric? Or are we initializing the Euclidean metric? Or are we setting a Euclidean metric? Or what is the verbage equivalent to "setting the mass matrix".

How to Verify:
./runCmdStanTests.py src/test/interface/metric_test.cpp
Side Effects:
Documentation:
I still need to do it
Reviewer Suggestions:
@mitzimorris or @sakrejda, whoever feels inclined
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): University of California, Santa Barbara
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

the challenge in adding this, as you've discovered - is the way that the command.hpp code sets up the cascade of dependent arguments.  if you look at the current cmdstan manual (which needs to updated as part of the PR),  the set of arguments for which passing in a metric file is a valid are only "sample algorithm = hmc"
method = sample (Default)
        sample
          num_samples = 1000 (Default)
          num_warmup = 1000 (Default)
          save_warmup = 0 (Default)
          thin = 1 (Default)
          adapt
            engaged = 1 (Default)
            gamma = 0.050000000000000003 (Default)
            delta = 0.80000000000000004 (Default)
            kappa = 0.75 (Default)
            t0 = 10 (Default)
            init_buffer = 75 (Default)
            term_buffer = 50 (Default)
            window = 25 (Default)
          algorithm = hmc (Default)
            hmc
              engine = nuts (Default)
                nuts
                  max_depth = 10 (Default)
              metric = diag_e (Default)
              stepsize = 1 (Default)
              stepsize_jitter = 0 (Default)

given this, you could add argument "metric_file" as a subargument to argument "hmc"  - you should be able to check that when the metric is "diag_e" the metric file is diagonal, and when the metric is "dense_e" the metric file is dense, and then you should be able to make this feature more general.

valid are only "sample algorithm = hmc"

Hmm, that does make sense.

which needs to updated as part of the PR

Will do. Was just being lazy.

This is ready for review

I read through this but I'd like to do one more pass before calling it
good.  I'll do that tomorrow.
On Sat, Oct 21, 2017, 11:39 AM Ben Bales<EMAIL_ADDRESS>wrote:

This is ready for review
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/cmdstan/pull/576#issuecomment-338411036, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAfA6auNDxq8fOFs7Y2tt0xpVi4kZCETks5suhAhgaJpZM4P5jhC
.

Only comment I had was maybe say in the doc that metric is used as a starting point for adaptation when both are specified (assuming this is what happens?)

Good point. I ran methods with adaptation and they didn't fail, but I need to make sure the adaptation isn't simply ignoring the provided initial matrix and re-adapting a new one. I'll check that and update the doc to make it clear.

now that we've got this all plumbed through, I'm still trying to make sense of the use cases for this feature:

the model is fully converged, we want to run the fitted model to generate more samples - in this case, additional param adapt engaged=0 will stop adaptation and go straight to sampling.  in which case, the fully consistant config requires:

random seed=<seed>
metric_file=<filename>
stepsize=<stepsize>

the model is taking a long time to converge, we want to checkpoint where it's at and restart.

running tests for case (2), I'm not sure adaptation is respecting stepsize.
here's my test metric file - same as used for stan unit test:
inv_metric <- structure(c(0.787405, 0.884987, 1.19869),.Dim=c(3))";

using model stan/src/test/test-models/good/mcmc/hmc/common/gauss3D.stan

regarding my previous comment, fix for problems w/r/t stepsize and adaptation should go in at the stan services level - not a cmdstan problem.

Use case (1) and (2), definitely.  A third case is external algorithms---Aki needs that for something, I believe.

Use case 3: 1) Fit the model to data A; 2) pretend like fitting the model to data A + B where B is much much smaller won't change posterior geometry much; 3) fit the model to data A + B without adaptation in parallel since now there's no adaptation to worry about (you could still do multiple chains with different starting points).

w/r/t testing - do we have tests for use cases 2 and 3?
I've edited my comment above re usecases w/ my attempts to convince myself that this feature works for use case 2, and I'm not sure it does.

Shouldn't tests be in stan::services? Or do you mean in general showing that they are worthwhile use cases?

yes, tests should be in stan::services.
current tests in stan::services test that sampler initializes itself with pre-specified metric - a unit test at the feature level, not a functional test at the use case level.

Only comment I had was maybe say in the doc that metric is used as a starting point for adaptation when both are specified (assuming this is what happens?)

I checked this. The way adaptation works, the provided metric is tossed if adaptation is enabled. So the way I have the docs written now is wrong. If someone provides a metric, adaptation of that metric should be disabled (otherwise it's misleading). I can make these options mutually exclusive.
Is there a way to separately turn off timestep and metric adaptation? For my use case for this I was hoping I could just leave the timestep adaptation to Stan (and just provide the metric).
Looking at the code it seems like either they both happen or neither happen: https://github.com/stan-dev/stan/blob/develop/src/stan/mcmc/hmc/nuts/adapt_diag_e_nuts.hpp#L31
Should we add this in? Or make it so that if someone provides a metric, they're liable for the timestep as well?
Either way, don't merge this pull :P.

hmm - if that's what the code is doing, that contradicts what Michael said here:

The stepsize parameter defines the initial step size from which the
algorithm begins.  If adaptation is engaged then this is quickly modified,
but it can be helpful to start with a small step size on particularly nasty
problems to facilitate adaptation.

(https://groups.google.com/forum/#!topic/stan-users/O-PNZhzVjTI)

OTOH, the current Stan manual says this:

Stan can be configured with a user- specified step size or it can estimate an optimal step size during warmup using dual averaging

@bbbales2 - you're misreading the code -

Looking at the code it seems like either they both happen or neither happen: https://github.com/stan-dev/stan/blob/develop/src/stan/mcmc/hmc/nuts/adapt_diag_e_nuts.hpp#L31

the variable named adapt_flag_ gets changed by the sampler during its run.
the stepsize can be set at initialization:
https://github.com/stan-dev/stan/blob/476975dacfc13ff7a1aec4cf23ff0fd11a64caea/src/stan/mcmc/hmc/base_hmc.hpp#L152

this PR looks good, and I believe the stan code for the samplers will do the right thing.  however, it would be good for the docs to spell out the 3 use cases and appropriate config:
(1)  specify stepsize, metric, set "adapt engaged=0".  the appropriate "num_samples" is determined by desired precision of your quantity of interest which in turn depends on "N_eff" for that QoI.
(2) specify stepsize, metric and any other non-default settings used in initial run
(3) depends on external algorithm

This is correct — unless you set adapt engaged=0 the step size and metric will
used only to initialize the adaptation routine.  If they are good values then they
should not change much.  This is the desired behavior and especially useful
if people want to use this feature to set informed guesses for the inv_metric
on new runs, as opposed to just restarting.

On Oct 24, 2017, at 7:14 PM, Mitzi Morris<EMAIL_ADDRESS>wrote:
this PR looks good, and I believe the stan code for the samplers will do the right thing. however, it would be good for the docs to spell out the 3 use cases and appropriate config:
(1) specify stepsize, metric, set "adapt engaged=0". the appropriate "num_samples" is determined by desired precision of your quantity of interest which in turn depends on "N_eff" for that QoI.
(2) specify stepsize, metric and any other non-default settings used in initial run
(3) depends on external algorithm
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/stan-dev/cmdstan/pull/576#issuecomment-339164401, or mute the thread https://github.com/notifications/unsubscribe-auth/ABdNlifo6AHbVAxu2Y9UpijS_FaTNbVSks5svm9LgaJpZM4P5jhC.

doc looks great!  very clear.

Np thanks for the review

#563


remove parent_id from layer model

Relates to #954

retest this please

retest this please

retest this please

retest this please

@juanignaciosl CR please.
It will be followed by a migration that removes DB field, but to avoid AR issues in this case better to first remove all code usages.

Also, it only covers BE removal of the field, I guess FE part would be ok but backend still returns the values whenever needed so nothing should break.

:+1: after column AR disabling

retest this please


New version: Microsoft.Gaming.TAK.CLI version 11.16.0

Checklist for Pull Requests

[ ] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

FYI - similar to previous release (https://github.com/microsoft/winget-pkgs/pull/155638) we don't expect this to return 0 when executed by the pipeline automation as this is a CLI tool.
cc @stephengillie

Automatic Validation ended with:

Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Packages\Microsoft.Gaming.TAK.CLI__DefaultSource\tak.exe returned exit code: 1
Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Links\tak.exe returned exit code: 1

(Automated response - build 887.)

FYI - similar to previous release (#155638) we don't expect this to return 0 when executed by the pipeline automation as this is a CLI tool.
cc @ stephengillie

Yes, this is common. Sorry for the delay.

https://github.com/microsoft/winget-pkgs/issues/157749


Fix ShowMoreTokensLink linking to empty ERC-721

https://lw-oasis-explorer-frontend.pages.dev/emerald/account/0xba504818fdd8d3dba2ef8fd9b4f4d5c71ad1d1d3

Click "+1 more" in EVM tokens
Opens empty ERC-721
expected it to show 4th ERC-20

Closed via https://github.com/oasisprotocol/explorer/pull/135


Release Google.Cloud.BigQuery.DataTransfer.V1 version 4.1.0

Changes in this release:
New features

Added Location API methods (commit 7e0e4ab)

Documentation improvements

Updated comments (commit 7e0e4ab)

Created release for Google.Cloud.BigQuery.DataTransfer.V1-4.1.0


Cannot use Graph Explorer when I have MFA

Describe the bug
Does not do MFA - so deniced access
To Reproduce

Log in with an Azure AD account that has MFA turned on
Try to run a Graph API

Expected behavior
Prompt for MFA when logging in to Graph Explorer
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 10
Browser Chrome
Version 97

Additional context
results returned (404)
"error": {
"code": "Unauthorized",
"message": "Additional claims required {"access_token":{"capolids":{"essential":true,"values":["d5f8fe6b-0605-40b8-9e25-e37a364553f3"]}}}",
"innerError": {
"date": "2022-02-10T15:36:43",
"request-id": "859efc1f-7a89-48be-b658-864a93702632",
"client-request-id": "c04cf16c-53e9-f96c-6902-795bdff01915"
}
}

Hey @i-m-batman, sorry about this.
Were you able to log in?
The error response you have included suggests you logged in but couldn't access some resources in your tenant because of permissions restrictions. Which query did you run when you got this error?
Please add some screenshots for more context

I was able to get in later, but not sure why it happened (sorry). Should know better and have snagged screenshots.
I will update if it happens again (with pictures)
thanks for getting back to me.


Lack of documentation for data_convertor

Description
data_convertor is the core of our convertor tasks. Without data_convertor, we have to care about apiClient, scheduler, db and so on. Right now, we can implement convertor tasks with only consideration of business.
But data_convertor is hard to understand how to use data_convertor
data_convertor path: plugins/helper/data_convertor.go
Describe the solution you 'd like

understand data_convertor
write a doc to contain the following:

describe how data_convertor works
write a doc to describe how to implement convertor with data_convertor

I'll pick it

@xgdyp is working on this issue. As some new features are not released, this might take some time


🐛 [FirebaseAuth] userChanges returns an anonymous user after calling linkWithCredential on iOS

Bug report
userChanges stream returns an anonymous user after calling linkWithCredential() to promote a current anonymous user to a permanent user.  This only happens on iOS. userChanges returns a correct signed user on Android.
Steps to reproduce
Steps to reproduce the behavior:

Sign in anonymously by calling signInAnonymously()
Promote the anonymous user to a permanent user

final email =<EMAIL_ADDRESS>final password = '123456';
final credential = EmailAuthProvider.credential(email: email, password: password);
FirebaseAuth.instance.currentUser.linkWithCredential(credential);

Listen to FirebaseAuth.instance.userChanges() and log the user.

Expected behavior
The returned user should be the signed user. But it returns an anonymous user on iOS. It works fine on Android.

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
[✓] Flutter (Channel stable, 1.22.2, on Mac OS X 10.15.4 19E287, locale ko-KR)
	• Flutter version 1.22.2 at /Users/*******/Developer/flutter
	• Framework revision 84f3d28555 (4 days ago), 2020-10-15 16:26:19 -0700
	• Engine revision b8752bbfff
	• Dart version 2.10.2

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
	• Android SDK at /Users/*******/Library/Android/sdk
	• Platform android-29, build-tools 29.0.3
	• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
	• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
	• All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
	• Xcode at /Applications/Xcode.app/Contents/Developer
	• Xcode 11.5, Build version 11E608c
	• CocoaPods version 1.9.3

[✓] Android Studio (version 3.6)
	• Android Studio at /Applications/Android Studio.app/Contents
	• Flutter plugin version 45.1.1
	• Dart plugin version 192.7761
	• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[✓] Connected device (1 available)
	• ****** iPhone (mobile) • 48006f05f9f84f9f0771134bb1d76016876695a3 • ios • iOS 13.5.1

[✓] Flutter (Channel stable, 1.22.2, on Mac OS X 10.15.4 19E287, locale ko-KR)
	• Flutter version 1.22.2 at /Users/******/Developer/flutter
	• Framework revision 84f3d28555 (4 days ago), 2020-10-15 16:26:19 -0700
	• Engine revision b8752bbfff
	• Dart version 2.10.2

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
	• Android SDK at /Users/******/Library/Android/sdk
	• Platform android-29, build-tools 29.0.3
	• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
	• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
	• All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
	• Xcode at /Applications/Xcode.app/Contents/Developer
	• Xcode 11.5, Build version 11E608c
	• CocoaPods version 1.9.3

[✓] Android Studio (version 3.6)
	• Android Studio at /Applications/Android Studio.app/Contents
	• Flutter plugin version 45.1.1
	• Dart plugin version 192.7761
	• Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[✓] Connected device (1 available)
	• ****** iPhone (mobile) • 48006f05f9f84f9f0771134bb1d76016876695a3 • ios • iOS 13.5.1

• No issues found!

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
Dart SDK 2.10.2
Flutter SDK 1.22.2
myapp 1.0.0+1

dependencies:
- carousel_slider 2.2.1 [flutter]
- cloud_firestore 0.14.1+3 [flutter meta quiver firebase_core firebase_core_platform_interface cloud_firestore_platform_interface cloud_firestore_web]
- device_preview 0.4.8 [font_awesome_flutter http freezed_annotation json_annotation flutter flutter_localizations meta shared_preferences shared_preferences_macos]
- equatable 1.2.4 [collection meta]
- extended_image 1.1.1 [flutter extended_image_library meta]
- file_picker 2.0.8 [flutter flutter_web_plugins flutter_plugin_android_lifecycle plugin_platform_interface]
- firebase_analytics 6.0.2 [meta flutter firebase_core firebase_analytics_web firebase_analytics_platform_interface]
- firebase_auth 0.18.1+2 [meta firebase_core firebase_core_platform_interface firebase_auth_platform_interface firebase_auth_web flutter]
- firebase_core 0.5.0+1 [firebase_core_platform_interface flutter quiver meta firebase_core_web]
- firebase_crashlytics 0.2.1+1 [flutter stack_trace firebase_core firebase_core_platform_interface firebase_crashlytics_platform_interface]
- firebase_messaging 7.0.3 [meta flutter firebase_core]
- firebase_storage 4.0.1 [flutter firebase_core]
- flutter 0.0.0 [characters collection meta typed_data vector_math sky_engine]
- flutter_bloc 6.0.4 [flutter bloc provider]
- flutter_linkify 3.1.3 [flutter linkify]
- flutter_signin_button 1.1.0 [flutter font_awesome_flutter]
- flutter_widget_from_html 0.5.1 [cached_network_image chewie flutter flutter_layout_grid flutter_svg flutter_widget_from_html_core html url_launcher video_player webview_flutter]
- fluttertoast 7.0.4 [flutter flutter_web_plugins]
- get_it 4.0.4 [async meta]
- google_sign_in 4.5.4 [google_sign_in_platform_interface flutter meta google_sign_in_web]
- http 0.12.2 [http_parser path pedantic]
- image_cropper 1.3.1 [flutter]
- intl 0.16.1 [path]
- material_design_icons_flutter 4.0.5555 [flutter]
- path 1.8.0-nullsafety.1
- path_provider 1.6.18 [flutter path_provider_platform_interface path_provider_macos path_provider_linux path_provider_windows]
- share 0.6.5+2 [meta mime flutter]
- shared_preferences 0.5.12+2 [meta flutter shared_preferences_platform_interface shared_preferences_linux shared_preferences_macos shared_preferences_web shared_preferences_windows]
- shimmer 1.1.2 [flutter]
- timeago 2.0.27
- url_launcher 5.5.3 [flutter url_launcher_platform_interface url_launcher_web url_launcher_linux url_launcher_macos]
- webview_flutter 1.0.1 [flutter]

dev dependencies:
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters charcode collection matcher meta source_span stream_channel string_scanner term_glyph typed_data]

transitive dependencies:
- async 2.5.0-nullsafety.1 [collection]
- bloc 6.0.2 [meta]
- boolean_selector 2.1.0-nullsafety.1 [source_span string_scanner]
- cached_network_image 2.3.2+1 [flutter flutter_cache_manager octo_image]
- characters 1.1.0-nullsafety.3
- charcode 1.2.0-nullsafety.1
- chewie 0.9.10 [open_iconic_flutter video_player wakelock flutter]
- clock 1.1.0-nullsafety.1
- cloud_firestore_platform_interface 2.1.2 [flutter meta collection firebase_core plugin_platform_interface]
- cloud_firestore_web 0.2.0+4 [flutter flutter_web_plugins firebase http_parser meta firebase_core cloud_firestore_platform_interface js]
- collection 1.15.0-nullsafety.3
- convert 2.1.1 [charcode typed_data]
- crypto 2.1.5 [collection convert typed_data]
- csslib 0.16.2 [source_span]
- extended_image_library 0.3.0 [flutter path_provider http_client_helper crypto path]
- fake_async 1.2.0-nullsafety.1 [clock collection]
- ffi 0.1.3
- file 5.2.1 [intl meta path]
- firebase 7.3.0 [http http_parser js]
- firebase_analytics_platform_interface 1.0.3 [flutter meta]
- firebase_analytics_web 0.1.1 [flutter flutter_web_plugins firebase firebase_analytics_platform_interface meta]
- firebase_auth_platform_interface 2.1.1 [flutter meta firebase_core plugin_platform_interface]
- firebase_auth_web 0.3.1+1 [flutter flutter_web_plugins firebase meta http_parser intl firebase_core firebase_auth_platform_interface js]
- firebase_core_platform_interface 2.0.0 [flutter meta plugin_platform_interface quiver]
- firebase_core_web 0.2.0 [firebase firebase_core_platform_interface flutter flutter_web_plugins meta js]
- firebase_crashlytics_platform_interface 1.1.1 [flutter meta collection firebase_core plugin_platform_interface]
- flutter_blurhash 0.5.0 [flutter meta]
- flutter_cache_manager 1.4.2 [flutter path_provider uuid http path sqflite pedantic clock file rxdart]
- flutter_layout_grid 0.10.2 [collection flutter meta quiver]
- flutter_localizations 0.0.0 [flutter intl characters collection meta path typed_data vector_math]
- flutter_plugin_android_lifecycle 1.0.11 [flutter]
- flutter_svg 0.18.1 [path_drawing xml vector_math meta flutter]
- flutter_web_plugins 0.0.0 [flutter characters collection meta typed_data vector_math]
- flutter_widget_from_html_core 0.5.1 [flutter html]
- font_awesome_flutter 8.8.1 [flutter]
- freezed_annotation 0.11.0+1 [collection json_annotation meta]
- google_sign_in_platform_interface 1.1.2 [flutter meta quiver]
- google_sign_in_web 0.9.2 [google_sign_in_platform_interface flutter flutter_web_plugins meta js]
- html 0.14.0+4 [csslib source_span]
- http_client_helper 0.2.1 [http]
- http_parser 3.1.4 [charcode collection source_span string_scanner typed_data]
- js 0.6.2
- json_annotation 3.0.1
- linkify 2.1.0
- matcher 0.12.10-nullsafety.1 [stack_trace]
- meta 1.3.0-nullsafety.3
- mime 0.9.7
- nested 0.0.4 [flutter]
- octo_image 0.3.0 [flutter flutter_blurhash]
- open_iconic_flutter 0.3.0 [flutter]
- path_drawing 0.4.1+1 [vector_math meta path_parsing flutter]
- path_parsing 0.1.4 [vector_math meta]
- path_provider_linux 0.0.1+2 [path xdg_directories path_provider_platform_interface flutter]
- path_provider_macos 0.0.4+3 [flutter]
- path_provider_platform_interface 1.0.3 [flutter meta platform plugin_platform_interface]
- path_provider_windows 0.0.4+1 [path_provider_platform_interface meta path flutter ffi win32]
- pedantic 1.9.0
- petitparser 3.1.0 [meta]
- platform 2.2.1
- platform_detect 1.4.0 [meta pub_semver]
- plugin_platform_interface 1.0.2 [meta]
- process 3.0.13 [file intl meta path platform]
- provider 4.3.2+1 [flutter nested collection]
- pub_semver 1.4.4 [collection]
- quiver 2.1.3 [matcher meta]
- rxdart 0.24.1
- shared_preferences_linux 0.0.2+2 [file flutter meta path path_provider_linux shared_preferences_platform_interface]
- shared_preferences_macos 0.0.1+10 [shared_preferences_platform_interface flutter]
- shared_preferences_platform_interface 1.0.4 [meta flutter]
- shared_preferences_web 0.1.2+7 [shared_preferences_platform_interface flutter flutter_web_plugins meta]
- shared_preferences_windows 0.0.1+1 [shared_preferences_platform_interface flutter ffi file meta path path_provider_platform_interface path_provider_windows]
- sky_engine 0.0.99
- source_span 1.8.0-nullsafety.2 [charcode collection path term_glyph]
- sqflite 1.3.1+1 [flutter sqflite_common path]
- sqflite_common 1.0.2+1 [synchronized path meta]
- stack_trace 1.10.0-nullsafety.1 [path]
- stream_channel 2.1.0-nullsafety.1 [async]
- string_scanner 1.1.0-nullsafety.1 [charcode source_span]
- synchronized 2.2.0+2
- term_glyph 1.2.0-nullsafety.1
- test_api 0.2.19-nullsafety.2 [async boolean_selector collection meta path source_span stack_trace stream_channel string_scanner term_glyph matcher]
- typed_data 1.3.0-nullsafety.3 [collection]
- url_launcher_linux 0.0.1+1 [flutter]
- url_launcher_macos 0.0.1+7 [flutter]
- url_launcher_platform_interface 1.0.8 [flutter meta plugin_platform_interface]
- url_launcher_web 0.1.3+2 [url_launcher_platform_interface platform_detect flutter flutter_web_plugins meta]
- uuid 2.2.2 [crypto convert]
- vector_math 2.1.0-nullsafety.3
- video_player 0.10.12+5 [meta video_player_platform_interface video_player_web flutter]
- video_player_platform_interface 2.2.0 [flutter meta]
- video_player_web 0.1.4 [flutter flutter_web_plugins meta video_player_platform_interface]
- wakelock 0.1.4+2 [flutter]
- win32 1.7.3 [ffi]
- xdg_directories 0.1.0 [path process flutter]
- xml 4.5.1 [collection convert meta petitparser]

I'm seeing this as well. It breaks my entire authentication flow; I'm effectively unable to update my app state when an anonymous user registers with an email and password. Is there a workaround? Or a fix coming soon? I've tried calling  signInWithEmailAndPassword after linkWithCredential; doing so generally doesn't result in the email-authenticated user appearing in userChanges (but for some reason, sometimes does).
The above is specific to iOS - I haven't seen it in Android (where userChanges seemingly always gets updated with the email-authenticated user right after the call to linkWithCredential).
I'm using 0.18.4+1.

@jesper372 You can call FirebaseAuth.instance.currentUser.reload() after linkWithCredential(). It returns the correct signed user.

@jezsung Thanks!

Hey @jezsung, is this still an issue on the latest auth package?
  firebase_core: "^1.1.0"
  firebase_auth: "^1.1.3"

@russellwheatley Sorry for the late reply, I haven't used the userChanges stream in the latest development. I found it updating the user's status based on the provided stream has some tradeoff and I eventually

Sorry I accidentally closed the issue. Please re-open it.


Introduce a documentation decorator and parse the TypeScript AST to find doc-decorated methods.

This PR:

Adds a script that parses the AST starting from 'index.ts', looking for "@doc" decorators and building a JSON representation of the API (schema in api-view.ts).
Adds a simple mustache template which renders this JSON. This is subject to change, but is extremely low overhead so will be easy to rip out (adding vue prerender adds a lot of dependencies and complexities). The majority of the complexity of this change is in the AST parser, so this shouldn't be an issue. This lives in website/ which is described below.
Adds a website/ directory - this is where we'll work on the brand new website. We can generate demos separately from the homepage / API reference (instead of homepage being a special demo).
Currently outputs the generated HTML into /tmp/deeplearnjs-new-website/api/index.html. This is also subject to change as we build the new website.
Links methods to github URLs for the method.

Example: http://nsthorat.github.io/api.html
Screenshot (lots of documentation needs to be written):

This change is

 This is amazing! Great job figuring out the ts compiler API !

Reviewed 21 of 26 files at r1, 5 of 5 files at r2.
Review status: all files reviewed at latest revision, 9 unresolved discussions.

scripts/site/make-api.sh, line 1 at r2 (raw file):

#!/usr/bin/env bash

let's move scripts/site/* to website/*. This way we keep anything related to website in one folder that can easily be moved to its own repo.

scripts/site/make-api.sh, line 2 at r2 (raw file):

#!/usr/bin/env bash
# Copyright 2017 Google Inc. All Rights Reserved.

s/2017/2018/

scripts/site/make-api.sh, line 23 at r2 (raw file):

node_modules/.bin/mkdirp $OUT_PATH

./node_modules/.bin/tsc ./scripts/site/make-api.ts --target es6 --module commonjs

How about using ts-node (https://github.com/TypeStrong/ts-node). It works by registering the TypeScript compiler for the .ts extension, allowing node to execute ts files. Also, in that case you can remove make-api.sh and move mkdirp to the make-api.ts file

scripts/site/make-api.ts, line 22 at r2 (raw file):

import * as mustache from 'mustache';
import * as ts from 'typescript';
import {DocHeading, DocMethod, DocMethodParam, Docs, DocSubheading} from './view-api';

I'm surprised that the linter doesn't complain about this line being longer than 80

scripts/site/make-api.ts, line 27 at r2 (raw file):

const DOCUMENTATION_DECORATOR = 'doc';
const API_TEMPLATE_PATH = './website/api/index.html';
const SRC_ROOT = 'src/';

remove SRC_ROOT and inline src/ inside PROGRAM_ROOT since it's only used once

scripts/site/make-api.ts, line 37 at r2 (raw file):

if (argv.rootPath == null) {
  throw new Error(
      `No rootPath provided. Please provide a github root path with --rootPath.`);

Rename this flag to repoPath, or similar in order to differentiate it from GITHUB_ROOT, or better maybe is to remove this flag and take cwd as the rootPath. You kinda already do this when calling this script from make-api.sh.

scripts/site/make-api.ts, line 77 at r2 (raw file):

const docs: Docs = {
  headings: docHeadings

move this above, next to docHeadings

src/math/reduction_ops.ts, line 287 at r2 (raw file):

Normalization

Maybe mean, std, moments etc should go in section "Averages". Idea is from: https://docs.scipy.org/doc/numpy-1.13.0/reference/routines.statistics.html (see headers)

src/math/softmax.ts, line 89 at r2 (raw file):

   *     which indicates the last dimension.
   */
  @doc('Operations', 'Classification')

This actually gets exposed as dl.losses.softmaxCrossEntropy while the current doc generator shows it as dl.softmaxCrossEntropy. If there is a way to detect that automatically, that would be great. Otherwise we can hard-code it in decorator. Something like: @doc('Losses', 'Classification', 'losses') 3rd argument being a namespace that gets appended to dl.

Comments from Reviewable

Review status: 1 of 25 files reviewed at latest revision, 9 unresolved discussions, some commit checks failed.

src/math/reduction_ops.ts, line 287 at r2 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…

Normalization

Maybe mean, std, moments etc should go in section "Averages". Idea is from: https://docs.scipy.org/doc/numpy-1.13.0/reference/routines.statistics.html (see headers)

Talked offline, mirrored TF:
https://www.tensorflow.org/api_docs/python/tf/nn/moments

src/math/softmax.ts, line 89 at r2 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
This actually gets exposed as dl.losses.softmaxCrossEntropy while the current doc generator shows it as dl.softmaxCrossEntropy. If there is a way to detect that automatically, that would be great. Otherwise we can hard-code it in decorator. Something like: @doc('Losses', 'Classification', 'losses') 3rd argument being a namespace that gets appended to dl.

Done.

scripts/site/make-api.sh, line 1 at r2 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
let's move scripts/site/* to website/*. This way we keep anything related to website in one folder that can easily be moved to its own repo.

Done.

scripts/site/make-api.sh, line 2 at r2 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
s/2017/2018/

Done.

scripts/site/make-api.sh, line 23 at r2 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
How about using ts-node (https://github.com/TypeStrong/ts-node). It works by registering the TypeScript compiler for the .ts extension, allowing node to execute ts files. Also, in that case you can remove make-api.sh and move mkdirp to the make-api.ts file

Done.

scripts/site/make-api.ts, line 22 at r2 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
I'm surprised that the linter doesn't complain about this line being longer than 80

Done.

scripts/site/make-api.ts, line 27 at r2 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
remove SRC_ROOT and inline src/ inside PROGRAM_ROOT since it's only used once

Forgot I did this so that we could pull "src" off the path in the display. This is used now.

scripts/site/make-api.ts, line 37 at r2 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
Rename this flag to repoPath, or similar in order to differentiate it from GITHUB_ROOT, or better maybe is to remove this flag and take cwd as the rootPath. You kinda already do this when calling this script from make-api.sh.

Done.

scripts/site/make-api.ts, line 77 at r2 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
move this above, next to docHeadings

Done.

Comments from Reviewable


Fix variance

Closes #3

Hi @RustyYato is cell-project still maintained - should people be using it ? cheers

Thanks for the ping @pinkforest .
This slipped under my radar. I'm passively maintaining cell-project. This crate should be fine to use.
@SoniEx2 I'm fine with raising MSRV for the fix here. Thanks for the PR

We believe this fix does not raise the MSRV. However, it is unclear to us whether it's fully sound - specifically, due to provenance, which is an area of unsafe that's still in flux (see also UCG). Ideally this would be using addr_of_mut!, but we do believe the current code goes out of its way to correct for provenance, so it might be okay after all. We don't enough about provenance to be able to tell, tho.

I'm going to switch to use addr_of_mut in the next release and add miri test to CI, so I think that should be fine.
I'm aware of the UCG's discussions on provenance and how much of it is in flux 😄.

Hi @RustyYato would you like to send advisory on informational = unsound PR to RustSec ? thx!

Done in https://github.com/rustsec/advisory-db/pull/1391


0.2.7 release candidate test

Do not merge - this is to check that the release will build with the current set-up for all desired systems.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

When defining a source/url please add a sha256, sha1 or md5 checksum (sha256 preferably).


Microsite: Update jira-dashboard.yaml

Broken url for icon
Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

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

@benjdlambert: One more fix for jira-dashboard. Ping @fridajac

Thanks!


Update TF Provider versions

@nsarupr the provider version was included in a test, pls update the test with the new value pls

@nsarupr the provider version was included in a test, pls update the test with the new value pls

@juandiegopalomino This has been addressed. Although the Azure CI is failing due to the password issue.


Toggle some feature on/off using command line script. E.g. keyboard manager for work / game profile

Description of the new feature / enhancement
Enable or disable the keyboard manager by command line options, so that user can make a automated script to toggle module on/off. Right now the only way to do that is open the GUI and click the on/off button.
Scenario when this would be used?
I use the same PC for coding and gaming. While it is handy to swap the Tab key and Escape key while coding, this must be disable every time I play steam game, this setting will cause problems while gaming. It would be very good feature if we can quickly enable or disable it by a shortcut or script.
Supporting information
No response

+1 on toggling the keyboard manager via command prompt.  I'd like to set up a script that runs when I plug my laptop into the docking station - I've got a Mac external keyboard and I use PowerTools to remap some of the keys.  But when I'm not using the docking station, I'm using the built-in laptop keyboard and I then want keyboard manager off.  I could set up an automated script to turn keyboard manager on and off, if PowerToys supported the command line toggling.

+1 on toggling the keyboard manager via command prompt. I'd like to set up a script that runs when I plug my laptop into the docking station - I've got a Mac external keyboard and I use PowerTools to remap some of the keys. But when I'm not using the docking station, I'm using the built-in laptop keyboard and I then want keyboard manager off. I could set up an automated script to turn keyboard manager on and off, if PowerToys supported the command line toggling.

This is exactly the situation I was about to come in and ask for - I'm also switching between a docking station + external keyboard and just being on the go and this would be so helpful!


glob configuration as an array

:zap: Summary
I'd love to see the glob option have support for an array of multiple globs
glob:
  - '*.{yml,yaml}'
  - .ansible-lint

Value
It can be rather difficult to construct or maintain a singular glob that selects all the files a single command can operate on, and that holds especially true for multi-language formatters like prettier.
This is one of the globs that I've had in one of my repos:
glob: '{*.{css,md,html,json,jsonc,code-snippets,rb,xml,plist,yml,yaml,ansible-lint}{,.tmpl},{Gem,Brew}file}'

It's 101 characters on its own, and still requires some excludes excludes to skip files that I know it should skip but still let match the glob because it's easier to edit that way.
It'd be much easier to split that up into six globs, like so:
glob:
  - '*.{css,md,html,json,jsonc,rb}'
  - '*.{xml,plist,yml,yaml}{,.tmpl}'
  - .vscode/*.code-snippets
  - .ansible-lint
  - Gemfile
  - Brewfile

And it'd be much easier to update in the future to work with more files.
Behavior and configuration changes
The glob option would be able to be a string with a single glob (to avoid breaking existing configs), or an array of multiple globs.

Nice suggestion! I'm not sure I have enough time to make it until 2025, but if you want, you can submit a PR :+1:


ref(*): Refactor the whole thing

This is refactor of the whole shooting match.  It's stable... I get all tests passing well in excess of 95% of the time.
Note to reviewers: Rather than looking at the diffs, I'd just suggest evaluating the branch as a whole.  Consider running the tests locally to assert they work as advertised.
Highlights:

Parallelized by default.

make test-integration defaults to parallel execution
Serialized execution needs to be explicitly requested with make test-integration-serial
Parallel execution defaults to 15 nodes

Began using Ginkgo's SynchronizedBeforeSuite to manage one-time setup for the parallelized test suite.  This cut down on repetitive, unnecessary, and doomed (but not fatal) attempts to register the admin user.
Careful management of shared state.  Ginkgo seems to run nodes as separate processes, so global variables need to be managed carefully.  All of these have been aggregated into a single settings package to avoid polluting the tests package and to keep test code expressive.
Careful refactoring of contexts / specs.  There were cases where we were satisfying a "heavyweight" requirement like "an app exists and has already been deployed," where the fact that the app had already been deployed was actually inconsequential to the functionality under test.  Avoiding unnecessary deployments helped improve overall stability.
More deis builds:create (aka deis pull)!  Why?  For many tests that have a condition / context that specifies an application exists and has already been deployed, this is a faster way to satisfy that prerequisite than by having the builder build pushed code.  (Don't worry.  There is still a test that specifically exercises that functionality, but we just avoid that anywhere where there's a faster means of satisfying "an app exists and is already deployed.")  It would seem these faster, lighter deploys have been good for overall stability.

Note that I have a hard-coded sleep for 10s after a deis builds:create.  I know, I know... sleeps in the tests are evil, right?  Or are they?  In 132/134 tests we're not testing deis builds:create itself, but as I said, we're using it to satisfy the condition "an app exists and has already been deployed."  I think it's more than fair to assume an "existing app" has been around for at least 10 seconds (in real life, an "existing app" has likely existed for much longer).  So the code reflects this... and this also had a drastic effect on improving the stability of the tests.

Frequently used commands (and complementary assertions) that are often used to fulfill contexts / conditions like "with an existing user" or "who owns an existing app," have been extracted into their own packages.  This has helped to avoid polluting the tests package.
Large specs that had too much going on in them were broken up into smaller, more focused specs.
I eliminated separate files for admin and healthcheck since, unlike the rest of the *_test.go files, they didn't neatly align with an describe a deis subcommand.  The specs within those files were not removed.  They were transplanted to other files where they seemed to fit more appropriately.
Specs that were in some way redundant were eliminated.
Verbiage for contexts and specs was altered to be more expressive.
Tests mostly clean up after themselves!  This is not 100% foolproof because if the tests are interrupted, cleanup doesn't happen.  Some test failures (on the rare occasion they now occur) can also cascade and cause some cleanup failures.  I still have an expectation that @arschles k8s cleanup code will be re-introduced at some point, in addition to the cleanup I have already included.  Bottom line... tests should be run with a clean k8s cluster and a fresh Workflow install, but the cleanup that gets done right now is good enough that when you're running tests locally, you can reuse your cluster and your Workflow install over and over and over again without much junk accumulating.  Locally, this has also helped stability.

Things I didn't change or fix:

There's still plenty of opportunity to pay down DRY exceptions in the *_test.go files.
I didn't add any new tests (only refactored existing ones or busted up big ones into smaller specs).  IMO, there are a lot of things that are still untested.  For instance, for some subcommands, we test help, but for most, we don't.  There should also probably be more failure / abuse cases for users who don't have permissions on an app (for instance) trying to do things to/with that app.  Adding in more of these tests will actually further improve stability.  (If some percentage of the nodes are executing trivial tests, it reduces odds that heavy-hitting, resource intensive tests are monopolizing the nodes.)

One thing I removed, in part because it didn't make sense to me: I did remove some specs having to do with arbitrarily complex usage of multiple domains and certificates.  Smaller, more focused specs would be better than an epic.  I do, however, feel that the important functionality is still tested and that I didn't actually sacrifice coverage by eliminating that big spec.
Bonus: Containerized e2e tests work now.  (It seems to me they didn't quite work before.)
To run:
$ export DEIS_CONTROLLER_URL=http://deis.whatever.io
$ make bootstrap
$ make docker-build docker-test-integration

Caveat: Any credentials referenced by your kubectl configuration must live somewhere within ~/.kube
Other notes: Some of the environment variables that drive all of this have changed, so this PR isn't going to pass in Jenkins with the job as is.
This is all far from perfect, but I'll let the stability speak for itself.  Didn't want to delay PR'ing it any longer since I know people are eagerly awaiting a suite that will pass the PRs we've all got queued up.
If I am on PTO Friday, I do not mind if someone else takes this over.

I haven't not read the code yet but based on the description a few cert + domain tests were removed, I'm not very sure that is wise. @slack and I came up with 5 uses cases for when I implemented this in the Controller and the same use case were applied to the router.
They are complicated but this is the only point in our testing cycle where we test all the complexities of wildcard and SAN and other fun fun things.

The one issue I see (and I think you alluded to) is that these tests use DEIS_CONTROLLER_URL to discover the router, which afaik will break these running inside a pod. We could change the test chart to define this new env var (i.e. value: $(DEIS_ROUTER_SERVICE_HOST):$(DEIS_ROUTER_SERVICE_PORT), but unless there was another motivation for this change, I'd rather keep it the same as before.

Since a few have asked about it, I'll defend my rationale for use of the DEIS_CONTROLLER_URL variable over DEIS_ROUTER_SERVICE_HOST  and DEIS_ROUTER_SERVICE_PORT.  When the router's doing its job, it's transparent.  It's awkward to ask for the host and port of the router (and expecting that the host is actually the subdomain for the controller).  One would never tell a real user to carry out deis register using the router's host:port.  A real user would be instructed to use the controller's address.  My transition to that bew variable was to provide a more accurate emulation of real life use.  If there are technical reasons to not do that, we can discuss, but the change was carefully thought out.

@helgi we can put more cert tests back.  What I was trying to express was that an arbitrarily complex test case like "test and app with two certs and three domains, where one domain is a wildcard and so is one cert" is an epic... it doesn't exercise any specific conditions that cannot be exercised by smaller, more focused specs.  BDD is about story telling and a spec such as that is like Batman v Superman... too many supetflous scenes that don't advance the story.  I'd be very happy to work with you to come up with smaller, more focused specs that exercise any cert-related use cases that you feel are not being addressed.

FYI, I've manually kicked off the CI pipeline on this commit again, so we should get e2e test job result back using the ci/jenkins/pr status context.  (Test job itself is here https://ci.deis.io/job/workflow-test-pr/469/console)

@krancour if you want to go ahead with DEIS_CONTROLLER_URL then you're going to have to figure out a way to get this to work with the workflow-e2e chart, as with this PR it breaks because it cannot discover where the controller lives inside the cluster.

@bacongobbler, I don't quite understand this:

because it cannot discover where the controller lives inside the cluster

Can you elaborate?

@bacongobbler @krancour it appears communications are working fine with current commit changes: https://ci.deis.io/job/workflow-test-pr/469/console (aside from actual test error, the tests are running properly)

><> cat ~/.helm/workspace/charts/workflow-beta1-e2e/manifests/deis-tests-pod.yaml | grep "image:"
    image: <IP_ADDRESS>:5000/deis/deis-e2e:git-0152535
    image: busybox
><> helm install workflow-beta1-e2e
---> Running `kubectl create -f` ...
pod "workflow-beta1-e2e" created

---> Done

Because $DEIS_CONTROLLER_URL is not present in the pod, it will fail.

@bacongobbler... we can fix that in the chart if need be.  I thought you were trying to express that even after such a change there would be technical difficulties with the approach.  fwiw, I have always run these remotely and have not tried running them from the chart yet.

@vdice... do the Jenkins jobs deploy e2e tests to a cluster using the chart?  Or do the jobs run the tests remotely against a cluster?

@krancour the former; the job runs tests using the tests chart.

@vdice, then I suspect complimentary chart changes actually are in order.

@bacongobbler @vdice @arschles
The concerns about finding the correct URL for the controller (which only happens to be a nip.io URL involving the router's service address) have been addressed by adding the following bit of code to the Makefile:
ifndef DEIS_CONTROLLER_URL
ifdef DEIS_ROUTER_SERVICE_HOST
export DEIS_CONTROLLER_URL=http://deis.${DEIS_ROUTER_SERVICE_HOST}.nip.io
endif
endif

The end result is that a user executing the tests remotely can specify the controller URL (which feels natural) via the DEIS_CONTROLLER_URL env var.  If this is not set, the Makefile will attempt to construct this URL using the DEIS_CONTROLLER_URL env var.
I don't both checking the port because the router's service always listens on port 80.  There is no scenario where it doesn't.  Even when HTTPS is enforced, the router's service still forward traffic from 80 to 8080 within the container and a redirect to HTTPS is enforced there.  Long story short-- 80 is always available.
As @bacongobbler had pointed out, the chart wasn't calling make.
That is also changed here in the Dockerfile:
CMD ["/usr/bin/make", "test-integration"]

And a complementary change in the chart:
https://github.com/deis/charts/pull/214
This setup has the added benefit that all details of how tests are executed are now consolidated in deis/workflow-e2e, which hopefully insulates the chart from any need for further changes if / as that changes.

@krancour I still don't see the technical rationale for using a new env variable, regardless of where it points. The existing ones are provided by Kubernetes, which allows someone to run these tests in isolation, without a chart, as long as they're in the same namespace as a working deis install.
The reasons you gave are philosophically valid, but (1) they suggest leaving out the router entirely, which would make these tests not end-to-end anymore (maybe I misunderstood you here) and (2) practically speaking, they will add another step (set the correct env variable) before we can run tests
(2) is the technical reason for not making the env var name change.

@arschles if you'll bear with me, I do think you're misunderstanding the change a bit.  I'll try to respond to your concerns-- which I appreciate you raising.

The existing ones are provided by Kubernetes, which allows someone to run these tests in isolation, without a chart, as long as they're in the same namespace as a working deis install.

This won't change, as I'll demonstrate.

The reasons you gave are philosophically valid

Thank you.  At the risk of being pedantic, I believe that how things are named is always of tremendous consequence.  The first time I ran this test suite (remotely) and was asked to provide the router's IP and port, I couldn't at all understand how that information was relevant.  The tests interact with the controller.  The fact that traffic flows through the router on its way to the controller is largely inconsequential.  A real world analogy: If you want to send me a letter, you use my address; not the address of the nearest post office.

they suggest leaving out the router entirely

I believe this is a misunderstanding.  The router is not bypassed in these tests.  We'll come back to this...

practically speaking, they will add another step (set the correct env variable) before we can run tests

It doesn't.  As of Friday afternoon, I amended this PR to include the following code in the Makefile:
ifndef DEIS_CONTROLLER_URL
ifdef DEIS_ROUTER_SERVICE_HOST
export DEIS_CONTROLLER_URL=http://deis.${DEIS_ROUTER_SERVICE_HOST}.nip.io
endif
endif

If DEIS_CONTROLLER_URL is already set (e.g. by someone executing the test suite remote), the rest of this block is ignored.  The variable that person would have set makes sense, as per the naming philosophy we discussed.
If DEIS_CONTROLLER_URL is not set, an attempt is made to construct an appropriate value from the DEIS_ROUTER_SERVICE_HOST, if that second environment variable does exist.  And as you have pointed out, that variable is provided automatically by Kubernetes.
Ultimately, the only variable that anyone executing tests remotely cares about is the only variable that the test suite itself cares about-- the one that makes sense-- DEIS_CONTROLLER_URL.  In the case of someone executing the test suite within a cluster (with or without using the chart to facilitate installation), this variable's value is constructed automatically.
Coming back to the following:

they suggest leaving out the router entirely

In the case of the user executing the test suite remotely, the router is the single point of ingress into their cluster.  The value they would set for DEIS_CONTROLLER_URL must necessarily resolve to the load balancer in front of the router(s), or else their Workflow installation wouldn't be functioning at all.
In the case of a user executing the test suite within a Kubernetes cluster, the automatic construction of the DEIS_CONTROLLER_URL env var's value involves the router's service IP.  So in both cases, the traffic flowing through the controller does indeed flow through the router.

@krancour that clears things up completely, thanks! I'll test these locally now.

@krancour I was able to run these tests against my personal testing cluster. Since it was a relatively smaller cluster, I had to run with only 2 nodes. Only 2 errors occurred, and as we talked about privately, they may be DNS related (possibly nip.io?). See below for test output:
ENG000656:workflow-e2e aaronschlesinger$ GINKGO_NODES=2 make test-integration
ginkgo -slowSpecThreshold=120.00 -noisyPendings=false -nodes=2 tests/
Running Suite: Deis Workflow
============================
Random Seed:<PHONE_NUMBER>
Will run 134 of 137 specs

Running in parallel across 2 nodes

••••••••••••••P
------------------------------
• [SLOW TEST:190.383 seconds]
deis limits
/Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/limits_test.go:104
  with an existing user
  /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/limits_test.go:102
    who owns an existing app that has already been deployed
    /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/limits_test.go:100
      that user can set a CPU limit on that application
      /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/vendor/github.com/onsi/ginkgo/ginkgo_dsl.go:354
------------------------------
$ deis auth:register http://deis.<IP_ADDRESS>.nip.io --username=test-13336 --password=asdf1234<EMAIL_ADDRESS>http://deis.<IP_ADDRESS>.nip.io/v2/ does not appear to be a valid Deis controller.
Make sure that the Controller URI is correct, the server is running and
your client version is correct.
Error: Get http://deis.<IP_ADDRESS>.nip.io/v2/: dial tcp: lookup deis.<IP_ADDRESS>.nip.io on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:49619-><IP_ADDRESS>:53: i/o timeout
$ deis auth:cancel --username=test-97929 --password=asdf1234 --yes
Error: Not logged in. Use 'deis login' or 'deis register' to get started.

• Failure in Spec Setup (BeforeEach) [10.125 seconds]
deis limits
/Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/limits_test.go:104
  with an existing user
  /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/limits_test.go:102
    who owns an existing app that has already been deployed [BeforeEach]
    /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/limits_test.go:100
      that user can unset a memory limit on that application
      /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/vendor/github.com/onsi/ginkgo/ginkgo_dsl.go:354

      No future change is possible.  Bailing out early after 10.099s.
      Expected
          <int>: 1
      to match exit code:
          <int>: 0

      /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/cmd/auth/commands.go:49
------------------------------
••••••••••••••••••s•^R
•
------------------------------
• [SLOW TEST:137.505 seconds]
deis ps
/Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/ps_test.go:177
  with an existing user
  /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/ps_test.go:175
    who owns an existing app that has already been deployed
    /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/ps_test.go:173
      that user can restart that app's processes
      /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/vendor/github.com/onsi/ginkgo/extensions/table/table.go:96
        restarts all of 3
        /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/vendor/github.com/onsi/ginkgo/extensions/table/table_entry.go:46
------------------------------
•••••••••••
------------------------------
• [SLOW TEST:139.905 seconds]
deis ps
/Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/ps_test.go:177
  with an existing user
  /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/ps_test.go:175
    who owns an existing app that has already been deployed
    /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/ps_test.go:173
      that user can restart that app's processes
      /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/vendor/github.com/onsi/ginkgo/extensions/table/table.go:96
        restarts all of 3 by type
        /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/vendor/github.com/onsi/ginkgo/extensions/table/table_entry.go:46
------------------------------
••••••••
------------------------------
$ deis auth:register http://deis.<IP_ADDRESS>.nip.io --username=test-43494 --password=asdf1234<EMAIL_ADDRESS>Registered test-43494
Error: Post http://deis.<IP_ADDRESS>.nip.io/v2/auth/login/: dial tcp: lookup deis.<IP_ADDRESS>.nip.io on <IP_ADDRESS>:53: read udp <IP_ADDRESS>:59254-><IP_ADDRESS>:53: i/o timeout
$ deis auth:cancel --username=test-54860 --password=asdf1234 --yes
Error: Not logged in. Use 'deis login' or 'deis register' to get started.

• Failure in Spec Setup (BeforeEach) [12.743 seconds]
deis tags
/Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/tags_test.go:177
  with an existing user
  /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/tags_test.go:142
    who owns an existing app that has already been deployed [BeforeEach]
    /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/tags_test.go:140
      and a tag has already been added to the app
      /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/tags_test.go:138
        that user can unset that tag from that app
        /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/vendor/github.com/onsi/ginkgo/ginkgo_dsl.go:354

        No future change is possible.  Bailing out early after 12.567s.
        Expected
            <int>: 1
        to match exit code:
            <int>: 0

        /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/cmd/auth/commands.go:49
------------------------------
••••••••••••••••••••P•••••••••••••
------------------------------
• [SLOW TEST:120.912 seconds]
deis releases
/Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/releases_test.go:94
  with an existing user
  /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/releases_test.go:92
    who owns an existing app that has already been deployed
    /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/releases_test.go:90
      and that app has three releases
      /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/releases_test.go:88
        that user can roll the application back to the second release
        /Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/vendor/github.com/onsi/ginkgo/ginkgo_dsl.go:354
------------------------------
••••••••••••••••••••••••••••••••••••••••P••

Summarizing 2 Failures:

[Fail] deis limits with an existing user [BeforeEach] who owns an existing app that has already been deployed that user can unset a memory limit on that application
/Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/cmd/auth/commands.go:49

[Fail] deis tags with an existing user [BeforeEach] who owns an existing app that has already been deployed and a tag has already been added to the app that user can unset that tag from that app
/Users/aaronschlesinger/gocode/src/github.com/deis/workflow-e2e/tests/cmd/auth/commands.go:49

Ran 134 of 137 Specs in 2443.886 seconds
FAIL! -- 132 Passed | 2 Failed | 3 Pending | 0 Skipped

Ginkgo ran 1 suite in 40m46.05314335s
Test Suite Failed
make: *** [test-integration] Error 1

Note the Makefile has now been amended to remove any special support for executing tests in parallel.  If you want to execute tests in serial, set the GINKO_NODES env var to 1.

I had a run with only two failed specs (both of which seem to be timeout related). I think we need to get this merged ASAP

@krancour Let's update the README when this is nearing 'code complete'... minimally describing that tests run in parallel by default and how one can run tests serially (GINKGO_NODES=1)

I tested and confirmed last night these tests work for me as well. @arschles those do indeed look like DNS issues with nip.io and I have a fun fix for that. ;)
Funny how CI is failing the PR, which fixes CI.

 I always forget to update the docs...

Per @vdice's suggestion, I want to update the README before merging this.  Not updating it was an oversight.

https://github.com/deis/controller/issues/552 will probably be fixed by this

@krancour code LGTM. I have 2 more godoc related comments (https://github.com/deis/workflow-e2e/pull/157#discussion_r59436476 and https://github.com/deis/workflow-e2e/pull/157#discussion_r59436460), so I'm LGTMing this assuming those will be fixed
:shipit:

@arschles fixing now.


Reduce number of calls to server

This plugin is very cool, I liked it a lot.
However I'm having a couple of issues.
1st, I noticed that the plugin queries the server too many times, which generates a huge load on the bridge, and floods the action history on Verisure's panel.
I have homebridge-gsh, i don't know if maybe it is influencing this behavior.
I see that there is is a pooling interval setting, to improve this perhaps the plugin should debounce status updates by the same time as the pooling interval, or perhaps to a fixed rate. Ex: if the plugin is requested to update the status 200 times within 10 seconds, only do one request, and return the same value during the timeout period.
Homebridge also reports that the status request take too long to respond, thats probably because of how long the api normally takes to respond. A possible solution to this is return the current status immediately, and request a status update, and provide it in a future status request.
2nd, I could not change the alarm mode, it reported the current status correctly, but I tried disarming it, and then the plugin stopped responding, I checked the Logs to see if there were some errors, but there was nothing there. Maybe the disarm API changed? I took a look at the code, but didn't manage to figure it out yet. I might contribute if I find something else about it.

HI, some evolution for you ? have you try to make it work ?
I really need this plugin.. and if you can or someone or together we can make it work great and better go !
I'm sure there is a lot of people they wan't to see it work great.

HI, some evolution for you ? have you try to make it work ?
I really need this plugin.. and if you can or someone or together we can make it work great and better go !
I'm sure there is a lot of people they wan't to see it work great.

Nah, I have up on that route, Securita's back end and communication with the alarm central is just too slow to be reliable.
I am working however in an alternative solution, but its very hacky, basically my plan is to use a ESP32 microcontroller to interface with the remote controller buttons and LEDs, and then use it as an actual home kit sensor.
Seems like it will work nicely, since the Remotes middle button asks for the status, I can even check if there is a zone open via the remote.

@joaodforce Caching and debouncing the arm state could make it work more reliable.
A combination of optimistic updates and caching, keeping the polling interval and adding debounce timeout as config options.
PS. I have a feeling that the SD installation communicates via SMS, which may explain the unreliable and slow interface. DS.


Problem sort icicle chart

Don't sort by label cant, how I fix it correct order?

Hmm, could you post the JavaScript from the Script? I think you might be able to sort it in JS.

I'm going to close this in a day since I haven't gotten a response @vperilla95.


Fixing dockerfile

One liner to properly add the package sources to the docker image.

LGTM - I assume the fix works! ;)

I hope so! Does for me at least 👍


Add JMOD signing retry and check curl rc

Add retry to the JMOD Windows&Mac signing, as a workaround until EF upstream issue is resolved: https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/issues/3758
This is likely to fail during the release builds with the current failure rate.

run tests


Automatic error report

<!---   *** BEFORE SUBMITTING: PASTE CLIPBOARD HERE TO COMPLETE YOUR REPORT ***   ---!>

I'm closing this issue because there was no information provided. If you encounter another error please open a new issue.


request for configurable stickers

If you are ok with that and assuming it is not a big problem, would you consider to follow Josef´s steps regarding number of stickers (1-54)? In such case some of us would not be forced to rewrite all 54 letters once again ;-)
Be assured that it is not a big deal for me to rewrite those 54 letters for one cube in case you decide to leave it as it is now (i.e. different number for different sticker between you and Josef).

It should not be a big problem and I'm ok with it, but I can't find on Josef's page how those parameters are supposed to be given. I've checked http://software.rubikscube.info/AnimCube/
There is says "The value of this parameter must have exactly 54 characters"

There is a self-explanatory picture of it in a facelet parameter section. You could also make it as an option if you want to use your order... Similar to values 0, 1, 2 etc. in case of Josef's applet.

 I just realized that this could be easily solved by combination of your parameters since I already know the end state of cube and algorithm needed to get the cube there. So sorry for bothering you for no reason.

Well I was thinking a bit further than that. I realized you could just use an algorithm. However, e.g. I was trying some F2L code and I realized it would be helpful in some cases to show only the inserted pair. I could of course make special cases for that though, like F2L-FL, F2L-FR, F2L-BL, F2L-BR. What do you think?

I can not speak for others, but for me it is not useful at all. I am using/describing intuitive method, so I certainly do not need that. Others, however, could be glad. My first thought in mind is to use slightly different colors - see Josef´s two applets in initrevmove parameter. Slightly different colors could work as highlighter...
That´s why I find move counter so important. With normal colors, you can just describe that in moves no. 1-4 the edge is paired-up with corner and in moves 5-8 the pair is inserted to the last slot.

forgot to close issue


key_scores_utils ImportError

Steps to reproduce

Clone anls_star 0.0.11 (commit 97cfec4)
Install anls_star in a fresh venv with pip install -r requirements_dev.txt
Run a benchmark, e.g. python3 src/benchmark_vrdu_ad_buy.py gpt-4o simple

Output:
  File ".../anls_star_metric/src/anls_star.py", line 16, in <module>
    from . import key_scores_utils as ksu
ImportError: attempted relative import with no known parent package

Attempts to fix
Removing the problematic import (L16) and all lines where it's used fixes the issue. It is thus isolated to the key_scores_utils introduced in 0.0.11.
Additional information

Bug does not occur when installing from PyPI
Tested on Python 3.11 and 3.12

Whats the status with this error @phschoepf ?


Create course browser page

As a user, I want to be able to select a class so I can see a detailed description of the selected class.
[Story Points: 7]
[Priority: Low]
[Risk: Low]

Take closer look at "add-course" structure (backend). @eyadsw

https://youtu.be/gIWRi2tAwyU


Session folder name in the tab title.

Hi
do you think you could add an option to add the folder name on tab name ?
many of the session names i use are the same and by adding the folder name in the tab will help me identify which session I am working on.
I have attched a draft of my idea.
if the folder name or the session name  is too long or there are too many tabs open it can only show few letters as this would be enough to know which tab belongs to which session.

We'll see about getting something added to help you out with this. I am thinking a user definable string with expandable variables to give everyone flexibility to have the tab naming the way they prefer.


middleman 4.0.0.rc.1 support

I tried to use this gem in a middleman 4.0.0.rc.1 project and looks this gem still not support the next version.
Bundler could not find compatible versions for gem "middleman-core":
  In Gemfile:
    middleman-sprockets (>= 4.0.0.rc.1) ruby depends on
      middleman-core (>= 4.0.0.rc.1) ruby

    middleman (>= 4.0.0.rc.1) ruby depends on
      middleman-core (= 4.0.0.rc.1) ruby

    middleman-sprockets (>= 4.0.0.rc.1) ruby depends on
      middleman-core (>= 4.0.0.rc.1) ruby

    middleman-syntax (>= 0) ruby depends on
      middleman-core (~> 3.0.0) ruby

I'm not sure what'll be necessary to make it support 4.0. Would you like to check out a local copy and give it a try?

After this little modification is working for me locally.
https://github.com/szines/middleman-syntax/commit/74926e6ccc62d37750ed8956f08849b7b00ae795


WIP - Rop experiment

#464
Submitting to get some feedback.
I've applied (at least tried to) Railway Oriented Programming to two commands: Convert from nuget and Simplify. The results / errors of a command are handled by call-site in PublicApi.
Pros

I find the code a bit easier to reason about
No modification in file system is performed until the command returns success result

Cons

code overhead for the infrastructure
in order to fully benefit from the technique, probably all commands ought to be transformed to use it

There are some new changes here.
I introduced inline operators for Rop module - hopefully they're not to mystifying.
There's the new concept of PaketEnv - hopefully this can be used by other commands in future as well
@forki do you think we can merge it, or shall I do some more cleanups?


Gitops : Error creating : pods xxxx is forbidden

While I was trying to deploy the Event and the AI-Manager I stumbled across an issue regarding permissions that results into a failed to create x.
I added a screenshot in the attachments regarding the error. I get the same error on multiple resources that are trying to create objects.
I already checked that my argo-cd has te required cluster-role bindings. Just to make sure, I added a screenshot of the yaml file  of this role binding aswell.
Hoping someone can help me resolve this issue !
Thanks in advance.
Kind regards,
Gilles

@Gilles-Plaquet can you provide more info, at which step you failed based on the document at here https://ibm.github.io/cp4waiops-gitops/docs/how-to-deploy-cp4waiops-35 ? What is your OCP version?

@gyliu513 the current Openshift version is 4.8.39. In the documentation it stated that i should be above 4.5 so i guess that should be fine.
I was able to create the ceph-cluster, and the shared  application without any issues ( all app details seem to be healthy there)
I guess its the moment it started installing the AI-Manager. Then i started to get the issue stated above.
Hope this information helped.

@Gilles-Plaquet can you login to your ocp cluster and run the command oc get po -n cp4waiops to check the pod status? If there are some pods not running, can you run oc logs for one of the not running pod and append the log here?

@gyliu513  there are none.
in the error message he is telling that he can't create pods. thats why we don't see anything i think.

I went to check in the openshift  interface and then i see this :

@gyliu513 exactly, that was my reasoning as well... permission issue but i have all the cluster permissions.
Thanks already for the help !
I also ran the command and everything in the rook-ceph cluster looks fine(too me) .

@Gilles-Plaquet let me check more with @huang-cn and @morningspace , they are located in China, and hope we can give you more info tomorrow, thanks!

thanks a lot already !

@Gilles-Plaquet
I see you mentioned that you are deploying both Event and the AI-Manager. May I know which install option you are taking, e.g.: to install it one by one, or use the all-in-one template. Also, may I know which release you deploy. Can you share the outputs of oc get csv under namespace cp4waiops and ibm-common-services?

I used the one by one installation since, the other one was in technical preview. I opted for release 3.5.

oc get csv -n cp4waiops output:

oc get csv -n ibm-common-services

@morningspace Thanks a lot!
In case a webex,zoomcall,....  is easier to help solve the issue, that is possible ofcourse !

@Gilles-Plaquet I don't understand why  there's this runAsUser: Invalid value: 1001 error appears here,  the AIOPS should not use runAsUser scc option at all, it shouldn't specify any UID value and let OCP to allocate one.  I'm wondering if the cagalog image in this env is the same as in ours?
Could you run commands below to check the catalogsource image and operator scc settings?
oc -n openshift-marketplace get catalogsource ibm-operator-catalog -oyaml|grep image:

oc -n cp4waiops get deploy iaf-core-operator-controller-manager -oyaml|grep -v 'f:securityContext'|grep securityContext  -A8

oc -n ibm-common-services get deploy ibm-common-service-operator -oyaml|grep -v 'f:securityContext'|grep securityContext  -A8

oc -n openshift-marketplace get catalogsource ibm-operator-catalog -oyaml|grep image:

oc -n cp4waiops get deploy iaf-core-operator-controller-manager -oyaml|grep -v 'f:securityContext'|grep securityContext  -A8

oc -n ibm-common-services get deploy ibm-common-service-operator -o yaml|grep -v 'f:securityContext'|grep securityContext  -A8

Currently the namespace is not existing however yesterday it was, see the post above. this might be since openshift was unable to install the operator.

@Gilles-Plaquet the AIOps never uninstall ibm-common-services components unless you remove them manually, it is weird, we can talk next Monday to dig more, hope it is OK. Thanks!


WordPress on App Engine Standard for PHP 7.2 can't continue the installation step

Hello,
I follow the step on https://github.com/GoogleCloudPlatform/php-docs-samples/tree/master/appengine/php72/wordpress
After #gcloud app deploy app.yaml cron.yaml
Then access the site, and try to continue the installation step, it shows the error message as below:
Warning: require(/srv/wp-admin/index.php): failed to open stream: No such file or directory in /srv/gae-app.php on line 53 Fatal error: require(): Failed opening required '/srv/wp-admin/index.php' (include_path='.:/usr/lib/php') in /srv/gae-app.php on line 53
Please advise, thanks.

Interesting, for some reason wp-admin/index.php is not present, but it absolutely should be. Something went wrong in the installation process. What files are in your project directory?
I wonder if when WordPress was downloaded and unzipped, it failed to copy the files.

/php-docs-samples/appengine/php72/wordpress$ ls -la
total 132
drwxr-xr-x  6 roger roger  4096 Aug 30 00:58 .
drwxr-xr-x 16 roger roger  4096 Aug 30 00:56 ..
-rw-r--r--  1 roger roger   253 Aug 30 00:56 composer.json
-rw-r--r--  1 roger roger 78417 Aug 30 00:56 composer.lock
drwxr-x---  3 roger roger  4096 Aug 30 01:43 datar-wordpress-www
drwxr-xr-x  2 roger roger  4096 Aug 30 00:56 files
-rw-r--r--  1 roger roger    21 Aug 30 00:56 .gitignore
-rw-r--r--  1 roger roger   917 Aug 30 00:56 phpunit.xml.dist
-rw-r--r--  1 roger roger  7236 Aug 30 00:56 README.md
drwxr-xr-x  2 roger roger  4096 Aug 30 00:56 test
drwxr-xr-x 17 roger roger  4096 Aug 30 00:56 vendor
-rw-r--r--  1 roger roger  4861 Aug 30 00:56 wordpress.php
/php-docs-samples/appengine/php72/wordpress$ ls -la datar-wordpress-www/
total 40
drwxr-x--- 3 roger roger 4096 Aug 30 01:43 .
drwxr-xr-x 6 roger roger 4096 Aug 30 00:58 ..
-rw-r----- 1 roger roger 1033 Aug 30 00:59 app.yaml
-rw-r----- 1 roger roger   93 Aug 30 00:59 cron.yaml
-rw-r----- 1 roger roger 1667 Aug 30 00:59 gae-app.php
-rw-r----- 1 roger roger  584 Aug 30 00:59 .gcloudignore
-rw-r----- 1 roger roger  295 Aug 30 00:59 php.ini
-rw-r----- 1 roger roger 4605 Aug 30 00:59 wp-config.php
drwxr-xr-x 3 roger roger 4096 Aug 30 00:58 wp-content
roger@datar-www:~/php-docs-samples/appengine/php72/wordpress$

Looks like wp-admin did not make it into your directory. I have no idea why. Try rerunning it and seeing if an error or warning is thrown in the installation process, or just download WordPress manually from wordpress.org and unzip it, and then use the following cmmand:
php wordpress.php update /path/to/unzipped-wordpress-project

I also get this error, I have tried to reinstall with no luck, still get a fatal error. files in my dir are
app.yaml  cron.yaml  gae-app.php  php.ini  wp-config.php  wp-content

or just download WordPress manually from wordpress.org and unzip it, and then use the following cmmand:

How would one do this via the cloud shell?

Ok, so after hammering on this. I got wpadmin to load in, however I still get the error
Warning: require(/srv/index.php): failed to open stream: No such file or directory in /srv/gae-app.php on line 53 Fatal error: require(): Failed opening required '/srv/index.php' (include_path='.:/usr/lib/php') in /srv/gae-app.php on line 53 I'm thinking this may be a slightly different error, so ill open a new ticket and reference

Hey @coreysyms! So it's possible this is because the files are failing to copy to your project directory. I've added a branch to this repository that hopefully fixes this problem. Try the following on the command line:
# CD into where you downloaded this repository
cd /path/to/php-docs-samples
# CD into the wordpress sample directory
cd appengine/php72/wordpress
# checkout the branch where I've added the fix
git checkout fix-wordpress-rename
# pull down the new composer shizzle
composer install
# Now try to run the create command again!
php wordpress.php create

Let me know if this works better for you, or if you still get the error!

@bshaffer Perfect! this runs with no errors now, thanks!

Yay! Thank you! Please let us know if you experience any other issues!

@bshaffer I have also made from PHP 7.2 in march of 2019 project from docs-samples and https://cloud.google.com/community/tutorials/run-wordpress-on-appengine-standard by upgrading existing project. I have wp-admin folder in my project. I installed google cloud storage from github repo first but now I want integrate the gae plugin - but I cant activate it.
Warning: require_once(google/appengine/api/urlfetch_service_pb.php): failed to open stream: No such file or directory in /srv/wp-content/plugins/appengine-plugin/modules/urlfetch.php on line 72 Fatal error: require_once(): Failed opening required 'google/appengine/api/urlfetch_service_pb.php' (include_path='.:/usr/lib/php') in /srv/wp-content/plugins/appengine-plugin/modules/urlfetch.php on line 72
Additonally can't run caching plugins on as described.


one out of two devices (BLE 4.0 and BLE 4.1) cant find the service data while the other can

Hello, I'm using noble and bleno as bluetooth central and peripheral.
During testing I could connect to the laptop with BLE 4.0 running bleno from my desktop with BLE 4.1 running noble
but always got an empty array when running discoverServices.
After running the server on the pc with BLE 4.1 and the client on my laptop with BLE 4.0
I was able to get the service array without any problems.
I was wondering if this is a known limitation e.g. if this could be a limitation of my laptops bluetooth chip ?
Edit:
I just realized  that I got some usb_binding 1.2 build errors on my laptop, could this be a problem in this case ?
Edit2:
Okay this is not the problems origin, it compiles now and the problem sill exists.
Edit3:
This is from the bleno page:

If you're using noble and bleno at the same time, connected BLE devices may not be able to retrieve a list of services from the BLE adaptor. Check out noble's documentation on bleno compatibility

I was unter the impression that this does only apply when trying to scan and advertise at the same time, while I wasn't running bleno and noble at the same time during my tests, I do plan to switch between them to first scann then advertise and the other way around if required. This shouldn't be a problem ?
Edit4:
I might have missed it but other if I don't get an error where would I find some logs ?
Edit5:
I will try this, maybe thats the reason why one of the devices cant get the service data.

Reporting all HCI events
By default noble waits for both the advertisement data and scan response data for each Bluetooth address. If your device does not use scan response the following environment variable can be used to bypass it.
sudo NOBLE_REPORT_ALL_HCI_EVENTS=1 node .js

This was a bug I my own code, sorry for the inconvenience.


Fix inferred-reference-bug

See issue #181 -- reference is sometimes incorrectly attributed to NCBI36 when (for example) "b36
is in the file path.
This PR attempts to address that by hard-coding the sort order of references to check, so that search looks for GRCh38 before GRCh37 (for example) & looks for human references before mouse references.

This change is

Coverage remained the same at 88.789% when pulling a05dfe0c61e81aa3fa1287fcc4f21535e555f12a on fix-reference-bug into 73f5d812e201ee77d10acf041ed746c485f03775 on master.

The current code is pretty fragile but I'm worried that fixing it via an enforced sort order leaves us exposed to the same kinds of bugs. What if a GRCh37 VCF had a reference path with "job38" in it?
Maybe one alternative is to (1) try inferring from filename before using the directory and (2) only try looking for reference names at the start of strings between separators (e.g. "grch38p2.fasta") but don't allow open ended string containment (i.e. "b37" shouldn't match "job37").
This is a bunch of hacky heuristics but hopefully will work a larger fraction of the time.

This makes a lot of sense, thanks for reviewing. I wonder if it is
reasonable to infer the reference from the directory at all, if the file
doesn't match? Also, perhaps we should give a warning if either we see more
than one possible match, or no match on file name.
On Sep 19, 2016 8:46 PM, "Alex Rubinsteyn"<EMAIL_ADDRESS>wrote:

The current code is pretty fragile but I'm worried that fixing it via an
enforced sort order leaves us exposed to the same kinds of bugs. What if a
GRCh37 VCF had a reference path with "job38" in it?
Maybe one alternative is to (1) try inferring from filename before using
the directory and (2) only try looking for reference names at the start of
strings between separators (e.g. "grch38p2.fasta") but don't allow open
ended string containment (i.e. "b37" shouldn't match "job37").
This is a bunch of hacky heuristics but hopefully will work a larger
fraction of the time.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hammerlab/varcode/pull/182#issuecomment-248170892,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4XPVBslJAFemUKAcBa4F6rrCDGoQMUks5qry0AgaJpZM4KBEgK
.

Coverage decreased (-1.3%) to 87.472% when pulling 8d53b1acde0027857e3dd7229c375424ce6f8d9f on fix-reference-bug into 73f5d812e201ee77d10acf041ed746c485f03775 on master.

LGTM except that such tricky-looking logic deserves a few unit tests.

Yeah - you're absolutely right. I will put together a few test input
strings, with the idea that it can be extended as additional problem
references are found.
Is it worth maybe splitting off any of these elements into separate
functions? Ie, the part that picks the "most recent", or the matching on
dict + aliases?
You'll note I didn't implement the re to match the first part of the
filename; figured I should get this logic down before adding that element.
Thx for the feedback!
On Thu, Sep 22, 2016 at 10:45 AM Alex Rubinsteyn<EMAIL_ADDRESS>wrote:

LGTM except that such tricky-looking logic deserves a few unit tests.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/hammerlab/varcode/pull/182#issuecomment-248925033,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4XPaBX1guq1kPAOBumoUCmDHijkDneks5qspSbgaJpZM4KBEgK
.

It sure seems like the repeated recency code could benefit from being pulled out into a helper function (and then tested in isolation).
Also, I think that you might get away with not needing the recency logic if you instead implement some sort of delimiter splitting over parts of the filename. But I'm happy with any solution that solves your job36 problem and is easy to extend/modify later.

@jburos Just checking in, how is this branch doing?

@iskandr I added a few test cases & separated the recency bit into a function as recommended.
Will wait to see how tests finish. Thanks & LMK your thoughts --

Coverage increased (+0.1%) to 88.922% when pulling 048c8cbcc63cdac119036642877a8cf25cd7489e on fix-reference-bug into 73f5d812e201ee77d10acf041ed746c485f03775 on master.

Coverage increased (+0.1%) to 88.922% when pulling 048c8cbcc63cdac119036642877a8cf25cd7489e on fix-reference-bug into 73f5d812e201ee77d10acf041ed746c485f03775 on master.

Coverage increased (+0.4%) to 89.143% when pulling 048c8cbcc63cdac119036642877a8cf25cd7489e on fix-reference-bug into 73f5d812e201ee77d10acf041ed746c485f03775 on master.

Coverage increased (+0.4%) to 89.143% when pulling 048c8cbcc63cdac119036642877a8cf25cd7489e on fix-reference-bug into 73f5d812e201ee77d10acf041ed746c485f03775 on master.


Update ru.json

I translated BD Emotes system to Russian, and some other fixes

https://github.com/BetterDiscord/BetterDiscord/blob/main/CONTRIBUTING.md#translations


PVC reflection: add missing filter annotation and fix typo

Description
This PR fixes a bug in the PVC reflection logic, adding the volume.beta.kubernetes.io/storage-class annotation to the filter list in order to prevent the propagation of the incorrect storage class name. In addition, it fixes a typo in a method name.
Fixes #(issue)
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

[X] Manually, on kind (v1.23)

/test

/merge


unexpected behavior (please doc or fix)

this.model.set("title", this.$el.find('input').val()); // new value
this.model.set("text", this.$el.find('textarea').val()); // value not changed

var changed = this.model.changedAttributes();
console.log(changed) // changed is false!

but if
this.model.set({
  title: this.$el.find('input').val(), // new value
  text: this.$el.find('textarea').val() // value not changed
})

var changed = this.model.changedAttributes();
console.log(changed) // changed is valid!

Doesn't look inconsistent to me. I the first case you're getting changedAttributes for the last this.model.set call, which didn't change anything. In the second case you're getting it for the only this.model.set call that was made, in which title changed.


.gitignore files do not show in file explorer

Issue Type: Bug
Open any project that has been initialized with git init, place a .gitignore file in it, and it will not show
I am running the latest Code-Insiders build (through the Arch Linux AUR package, visual-studio-code-insiders-bin)
If you try to add a .gitignore file, it will say .gitignore file already exists
VS Code is detecting the .gitignore file, but it is not showing it
.gitignore is not in my Excludes list in the settings.json file
VS Code version: Code - Insiders 1.69.0-insider (fbf1cf3832d43088e27837dbb68d24ab65a098c1, 2022-06-17T05:17:59.312Z)
OS version: Linux x64 5.18.5-arch1-1
Restricted Mode: No

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-1195G7 @ 2.90GHz (8 x 2680)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
2, 1, 1

Memory (System)
31.07GB (21.48GB free)

Process Argv
--no-sandbox --unity-launch --crash-reporter-id 534ad51f-3a50-4c19-83b9-fe3bf8f15214

Screen Reader
no

VM
0%

DESKTOP_SESSION
plasma

XDG_CURRENT_DESKTOP
KDE

XDG_SESSION_DESKTOP
plasma

XDG_SESSION_TYPE
x11

Extensions (29)

Extension
Author (truncated)
Version

npm-intellisense
chr
1.4.2

doxdocgen
csc
1.4.0

es7-react-js-snippets
dsz
4.4.3

gitlens
eam
12.1.1

vscode-npm-script
eg2
0.3.25

copilot
Git
1.30.6165

go
gol
0.33.1

cloudcode
goo
1.18.3

discord-vscode
icr
5.8.0

better-cpp-syntax
jef
1.15.18

cmake-language-support-vscode
jos
0.0.4

file-downloader
min
1.0.12

mindaro
min
1.0.120220125

vscode-docker
ms-
1.22.0

vscode-dotnet-runtime
ms-
1.5.0

vscode-kubernetes-tools
ms-
1.3.10

python
ms-
2022.8.0

vscode-pylance
ms-
2022.6.20

jupyter
ms-
2022.5.1001601848

remote-containers
ms-
0.238.2

remote-ssh
ms-
0.82.1

remote-wsl
ms-
0.66.3

cmake-tools
ms-
1.11.26

cpptools
ms-
1.10.7

cpptools-extension-pack
ms-
1.2.0

vscode-yaml
red
1.8.0

LiveServer
rit
5.7.5

svelte-vscode
sve
105.17.0

cmake
twx
0.0.17

(1 theme extensions excluded)

A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368cf:30146710
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256197
vslsvsres303:30308271
pythonvspyl392:30422396
pythontb:30258533
pythonptprofiler:30281269
vshan820:30294714
pythondataviewer:30285072
vscod805cf:30301675
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30404738
wslgetstarted:30449409
vscscmwlcmt:30465136
cppdebug:30492333
vscaat:30438846
pylanb8912:30496796
vsclangdf:30492506

Looks like a recent change means .gitignore now always gets hidden in Explorer unless you have set "explorer.excludeGitIgnore": true, in which case the .gitignore file shows but none of the .gitignored files do.
@lszomoru this means that people who don't want to use the new explorer.excludeGitIgnore setting will no longer see their .gitignore file in Explorer. Is this intentional?

Looks like a recent change means .gitignore now always gets hidden in Explorer unless you have set "explorer.excludeGitIgnore": true, in which case the .gitignore file shows but none of the .gitignored files do.
@lszomoru this means that people who don't want to use the new explorer.excludeGitIgnore setting will no longer see their .gitignore file in Explorer. Is this intentional?

It seems like setting that to true fixes the issue, but I do need to see the files in my .gitignore, such as .env files that contain environment variables.
Is there anyway I can see the .gitignore file, with it showing my excluded files?

I do not think this is intentional, as I can see many issues this could arise because of this

This is an issue...

I saw that even having a folder without Git initialized in it still results in the .gitignore being hidden.  Seems like it's a default behavior and the explorer.excludeGitIgnore setting changes this behavior.  I tried messing with the Files: Exclude setting and adding a negated glob (or whatever that's officially called) to get the gitignore back, but that didn't work.

@lszomoru Any update on where this bug report is going?

Adding @lramos15 as he worked on this new functionality.

Issue has been fixed!

Glad to hear it! Sorry for the few bumps along the way :)


add basic testing of webchecks

Thanks for the PR! For now frontman have all kind of edge cases inside the example.json – so we just use it and confirm results. But we agreed with @thorstenkramm that we will need to write tests with CI instead as soon we finish all the features.
According to the PR. I can merge it now. But the right approach is to use the methods directly instead of relying on builded binaries. E.g. using fm.runWebCheck directly in the test and compare the output

@requilence yep I agree, running the binary is not ideal. I forgot to add a note that a binary must be installed.
I did it this way to avoid redoing all the parsing/setup work which is done in https://github.com/cloudradar-monitoring/frontman/blob/master/cmd/frontman/main.go#L53
Maybe we can wrap this somehow to make it easier to bring up the app from code.

Closing this for the moment. Not happening