firebase_messaging 10.0.2 - FirebaseMessaging.onMessageOpenedApp is never triggered

Bug report
FirebaseMessaging.onMessageOpenedApp is never triggered, even when I click notification (app is killed), then app opens.
I can see in flutter logs
this is working fine FirebaseMessaging.onBackgroundMessage(_firebaseMessagingBackgroundHandler);
This is never getting executed
FirebaseMessaging.onMessageOpenedApp.listen((message) {
      _logger.d('onMessageOpenedApp: ${message.data}');
      handle(message);
    });

Versions
flutter --version
Flutter 2.2.2 • channel stable • https://github.com/flutter/flutter.git
Framework • revision d79295af24 (11 days ago) • 2021-06-11 08:56:01 -0700
Engine • revision 91c9fc8fe0
Tools • Dart 2.13.3

firebase_messaging: ^10.0.2
Usage
main.dart
void main() async {
  await _initializeFirebase(config.isProduction);
// below stuff not relevant as we initialise right away firebase

  runZonedGuarded(() {
    runApp(AppConfigWidget(
      config: config,
      child: BlocProvider<AppBloc>(
        create: (context) => appBloc,
        child: App(
          locale: locale,
        ),
      ),
    ));
  },
      config.isProduction
          ? FirebaseCrashlytics.instance.recordError
          : _reportCrash);
}

Future<void> _initializeFirebase(bool isProduction) async {
  await Firebase.initializeApp();
  await FirebaseCrashlytics.instance
      .setCrashlyticsCollectionEnabled(isProduction);
  if (isProduction) {
    // Pass all uncaught errors from the framework to FirebaseCrashlytics.
    Function originalOnError = FlutterError.onError;
    FlutterError.onError = (FlutterErrorDetails errorDetails) async {
      await FirebaseCrashlytics.instance.recordFlutterError(errorDetails);
      // Forward to original handler.
      originalOnError(errorDetails);
    };
  }

  PushNotificationsManager();
  FirebaseMessaging.onBackgroundMessage(_firebaseMessagingBackgroundHandler);
  FirebaseMessaging.onMessageOpenedApp.listen((message) {
   // THIS IS NEVER TRIGGERED!!!
    getLogger('app_runner').d('Firebase.onMessageOpenedApp: ${message.data}');
// The call below does app internal logic, just parses notification data and redirects to other view (which worked fine before //upgrade to 10.0.2
    PushNotificationsManager.handle(message);
  });
}

/// Define a top-level named handler which background/terminated messages will
/// call.
/// To verify things are working, check out the native platform logs, flutter logs
Future<void> _firebaseMessagingBackgroundHandler(RemoteMessage message) async {
  await Firebase.initializeApp();
  getLogger('app_runner').d('Firebase.onBackgroundMessage: ${message.data}');
}

As mentioned _firebaseMessagingBackgroundHandler this works fine !!! when app is closed/killed in flutter logs command the logs appear, but when clicking the notification we expect onMessageOpenedApp.listen callback to run, and to read same notification there, then handle it in our extra class and redirect user to proper view
Also what is difference between getInitialMessage and onMessageOpenedApp
Flutter doctor
flutter doctor -v
[✓] Flutter (Channel stable, 2.2.2, on macOS 11.4 20F71 darwin-x64, locale en-DE)
    • Flutter version 2.2.2 at /Users/kristi.jorgji/flutter
    • Framework revision d79295af24 (2 weeks ago), 2021-06-11 08:56:01 -0700
    • Engine revision 91c9fc8fe0
    • Dart version 2.13.3

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/kristi.jorgji/Library/Android/sdk
    • Platform android-30, build-tools 29.0.3
    • ANDROID_HOME = /Users/kristi.jorgji/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5, Build version 12E262
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Ultimate Edition (version 2020.2.3)
    • IntelliJ at /Users/kristi.jorgji/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.57.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (3 available)
    • SM J510FN (mobile)                 • badea702      • android-arm    • Android 7.1.1 (API 25)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86    • Android 7.0 (API 24) (emulator)
    • Chrome (web)                       • chrome        • web-javascript • Google Chrome 91.0.4472.114

• No issues found!

same issue here

@kristijorgji I am facing the same issue. Can you please tell me how you solved this?


cannot resolve dispatchClick

I cannot call the dispatchClick() function from the onBindViewHolder of my adapter class. What else can I use to paste the selected item to the edittext?

I solved it..


Transpose input to match changing on indices in nextsimdg

After PR https://github.com/nextsimhub/nextsimdg/pull/373 domain_decomp produces incorrect meta data when x and y dims were switched (for rectangular shaped grids) -- even when supplying --dim0 y and --dim1 x options.
E.g. when running the testRectGrid_MPI test from nextsimdg we have a rectangular grid of dimensions:
$ ncdump RectGrid_test.nc
netcdf RectGrid_test {
[...]
group: data {
  dimensions:
  	x = 25 ;
  	y = 15 ;
  	z = 1 ;
  variables:
  	double cice(y, x) ;
  		cice:missing_value = -2.03703597633449e+90 ;
[...]

However if we run decomp we get the following meta data (notice that globalX, globalY and other related dimensions are the wrong way round).
$ mpirun -n 2 ./decomp --grid RectGrid_test.nc --dim0 y --dim1 x && ncdump partition_metadata_2.nc
Partitioning total time: 8.8619e-05 (secs)
Partitioning Statistics:
[...]
 STATS DETAIL count:  min 180  max 195  avg 187.500000  imbal 1.040000
 STATS DETAIL wdim = 0; no detail available
$ ncdump partition_metadata_2.nc
netcdf partition_metadata_2 {
dimensions:
	globalX = 15 ;
	globalY = 25 ;
[...]
  data:
   global_x = 0, 0 ;
   global_y = 0, 12 ;
   local_extent_x = 15, 15 ;
   local_extent_y = 12, 13 ;
  } // group bounding_boxes
[...]
}

This PR fixes the issue and the correct meta data is produced.
$ mpirun -n 2 ./decomp --grid RectGrid_test.nc && ncdump partition_metadata_2.nc
[...]
netcdf partition_metadata_2 {
dimensions:
	globalX = 25 ;
	globalY = 15 ;
[...]
   global_x = 0, 12 ;
   global_y = 0, 0 ;
   local_extent_x = 12, 13 ;
   local_extent_y = 15, 15 ;
  } // group bounding_boxes
[...]
}

Notice I have omitted the --dim0 and --dim1 flags.
This "fix" has been implemented by transposing the input data when it is read-in from the netcdf file specified by --grid.
This was the simplest way to get everything working without breaking the tests, the library or the stand-alone binary.
Whilst this works for now I think a proper solution should be found because I think it could be confusing in future.
Unfortunately I think we need to switch everything else -- instead of the input i.e., dim0 <-> dim1, x <-> y etc.
This is not as trivial as I first thought however as there are many places where this needs to be carefully checked.

I haven't even looked at the code yet, but I think we had the same problem in the nextsim-DG test with the MacOS test failing to extract correctly into homebrew/cask. I will try to find a) if I am indeed remembering correctly and then b) what the solution was.

I haven't even looked at the code yet, but I think we had the same problem in the nextsim-DG test with the MacOS test failing to extract correctly into homebrew/cask. I will try to find a) if I am indeed remembering correctly and then b) what the solution was.

Solved with commit e59851e.


Update intrinsics readme.md file

Existing Sample Changes
Description
Restructured to match new template. Corrected links. Updated branding. Corrected some formatting.
Fixes Issue#
External Dependencies
N/A
Type of change
Please delete options that are not relevant. Add a 'X' to the one that is applicable.

[x] Documentation

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] VSCode

@ishaghosh27  please review.
@IgorOchocki  and @jimmytwei added for notifications.


Timed GPIO Interrupt (IDFGH-11339)

Is your feature request related to a problem?
I'm looking for a way to grab a timer value upon triggering an interrupt on a GPIO pin
Unfortunately the standard gpio interrupt hooks are not high enough priority to activate on time during other computations
I attempted to write a high priority interrupt in assembly, and it did work, but it also interferes with the standard C gpio interrupt functions (gpio_install_isr_service). And I certainly can't write the rest of my gpio interrupt code in assembly
I'm working with Seeed Studio Xiao ESP32-S3 and ESP32-C3 for my use case, but I barely know enough to program it for the ESP32-S3
Here is the test C code (ESP32-S3):
/*
 *
	This tests the GPIO interrupt latency using a HLI
	that directly sets the register

*/
#include <stdio.h>
#include "sdkconfig.h"
#include "freertos/FreeRTOS.h"
#include "freertos/task.h"
#include "esp_system.h"
#include "spi_flash_mmap.h"
#include "driver/gpio.h"
#include "esp_intr_alloc.h"
#include "driver/timer.h"
#include "driver/uart.h"
#include <string.h>
#include "soc/rtc.h"
#include <sys/time.h>
#include <string.h>

#include "esp_log.h"

#include "rom/ets_sys.h"
#include "soc/periph_defs.h"

#include "soc/timer_group_reg.h"

unsigned int * TIMR_ADDR_LO = (unsigned int*)TIMG_T0LO_REG(0);
unsigned int * TIMR_ADDR_HI = (unsigned int*)TIMG_T0HI_REG(0);
unsigned int * UPDATE_ADDR = (unsigned int*)TIMG_T0UPDATE_REG(0);

#define GPIO_INPUT_IO_1     7
#define GPIO_INPUT_PIN_SEL1  (1ULL<<GPIO_INPUT_IO_1)

#define INTR_NUM 31
#define CPU_CORE 0

volatile uint64_t timer0_value = 0;

static void gpio_task_example(void* arg) {
	while(1) {
		*UPDATE_ADDR = 1;
		uint64_t lastValue = *TIMR_ADDR_LO;
		printf("LOW TIMER ADDR: %X\n", (unsigned int)TIMR_ADDR_LO);
		fprintf(stdout, "GPIO Reg value = %lli - %lli\n", lastValue, timer0_value);
		vTaskDelay(pdMS_TO_TICKS(1000));
	}
}

void app_main() {
    timer_config_t config = {
        .divider = 80,
        .counter_dir = TIMER_COUNT_UP,
        .counter_en = TIMER_START,
        .alarm_en = TIMER_ALARM_DIS,
        .auto_reload = TIMER_AUTORELOAD_DIS,
        .clk_src = TIMER_SRC_CLK_APB
    };
    timer_init(TIMER_GROUP_0, TIMER_0, &config);

	gpio_config_t io_conf;
	//interrupt of rising edge
	io_conf.intr_type = GPIO_INTR_POSEDGE;
	//bit mask of the pins, use GPIO4/5 here
	io_conf.pin_bit_mask = GPIO_INPUT_PIN_SEL1;
	//set as input mode
	io_conf.mode = GPIO_MODE_INPUT;
	//enable pull-up mode
	io_conf.pull_up_en = 0;
	io_conf.pull_down_en = 1;
	ESP_ERROR_CHECK(gpio_config(&io_conf));

	gpio_set_level(GPIO_INPUT_PIN_SEL1, 0);

	ESP_INTR_DISABLE(INTR_NUM);
	intr_matrix_set(CPU_CORE, ETS_GPIO_INTR_SOURCE, INTR_NUM);
	ESP_INTR_ENABLE(INTR_NUM);
    // fprintf(stdout, "This tests the interrupt latency, plug in a fungen to pin 17 and observe on pin 4!\n");

    xTaskCreatePinnedToCore(gpio_task_example, "gpio_task_example", 2048, NULL, 10, NULL, 0);

	while(1) {
    	// fprintf(stdout, "This tests the interrupt latency, plug in a fungen to pin 17 and observe on pin 4!\n");
		vTaskDelay(pdMS_TO_TICKS(1000));
	}
}

Here is the assembly:
#include <xtensa/coreasm.h>
#include <xtensa/corebits.h>
#include <xtensa/config/system.h>
#include "freertos/xtensa_context.h"
#include "esp_private/panic_reason.h"
#include "sdkconfig.h"
#include "soc/soc.h"
#include "soc/gpio_reg.h"
#include "soc/dport_reg.h"

#define TIMR_ADDR_LO 0x6001F004
#define TIMR_ADDR_HI 0x6001F008
#define TIMR_ADDR_UPD 0x6001F00c
#define GPIO_PIN 7

#define L5_INTR_STACK_SIZE  12
#define LX_INTR_A15_OFFSET 0
#define LX_INTR_A14_OFFSET 4
#define LX_INTR_A13_OFFSET 8
    .data
_l5_intr_stack:
    .space      L5_INTR_STACK_SIZE

    .section .iram1,"ax"
    .global     xt_highint5
    .type       xt_highint5,@function
    .align      4
xt_highint5:

	movi    a0, _l5_intr_stack // assuming a0 saved restored by EXCSAVE_5 at the end of this
    s32i    a15, a0, LX_INTR_A15_OFFSET
    s32i    a14, a0, LX_INTR_A14_OFFSET
    s32i    a13, a0, LX_INTR_A13_OFFSET

    /* Clear interupt on GPIO17 */
    movi    a14, GPIO_STATUS_W1TC_REG
    movi    a15, (1 << GPIO_PIN)
    s32i    a15, a14, 0
    memw

    movi a14, TIMR_ADDR_UPD    //Timer 0 Update address into a10
    s32i a0, a14, 0         //A write instruction copies the timer to the timer registers
    movi a14, timer0_value  //timer0_value address into a11
    movi a15, TIMR_ADDR_LO    //Timer 0 lower 32 bits address into a15
    l32i a13, a15, 0
    s32i a13, a14, 0        //Store lower 32 bits to timer0_value
    movi a15, TIMR_ADDR_HI    //Timer 0 higher 32 bits address into a15
    l32i a13, a15, 0
    s32i a13, a14, 4        //Store lower 32 bits to timer0_value
    memw

   	l32i    a15, a0, LX_INTR_A15_OFFSET
    l32i    a14, a0, LX_INTR_A14_OFFSET
    l32i    a13, a0, LX_INTR_A13_OFFSET

    rsr     a0, EXCSAVE_5 // restore a0
    rfi     5

    .global ld_include_xt_highint5
ld_include_xt_highint5:

Describe the solution you'd like.
A high level library that captures a timer upon triggering a gpio interrupt at highest priority
Describe alternatives you've considered.
I did try using the PWM capture library, but it too returned substantial variation in latency when the CPU is loaded. Also it's not available on the ESP32-C3
Additional context.
No response

I did try using the PWM capture library, but it too returned substantial variation in latency when the CPU is loaded.

The CPU latency should not affect the captured timestamp, because it's done by hardware.
If you have the chance to use esp32c6, it has a new peripheral called "Event Task Matrix", which can also do the same job. We can use any GPIO event to trigger the timer's task (in this case, the task is "capture the timestamp").

I shall check my PWM code and get back to you. Thank you for the response

Also, if you need to capture a PWM waveform, this can be done pretty accurately using the RMT peripheral.

PWM is definitely worked as needed with the ESP32S3
I shall check out the RMT peripheral on the ESP32C3 and report back


Update README.md

Update readme to explain to Unity3D users how to use this plugin for their project

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Doesn`t show users when adding member to team/project/office

Now it is:

But it should be:

If all the members are already added to the project/team then it'll show none, it shows user who are not added yet

It's fresh migration-seed. Screenshots are from headquarter, only admin is a member.
In that case, admin shouldn't apear in list.
In commit 'Add direct messaging functionality #109',
'data' was swaped with users and that causing an error...

Can I take this issue to solve?

yes sure


Remove depth limit in Travis CI to pass incremental statistics test

The Travis CI builds are currently failing because one of the tests, test_incremental_stats, attempts to reference an older commit even though Travis CI only retrieves the last 3 commits.
This pull request changes the Travis CI configuration to retrieve all commits, thereby fixing the builds. I haven't noticed a strong increase in build time; the time to download submodules dominates the total download time anyway.
I'm not sure this is the appropriate fix as I don't understand why it worked before... It does fix the build though :-)
Template doesn't apply.

And I think Travis CI has a very good connection to Github.com :-)


Improve auto generated resource docs

More minor updates to improve our docs generation
Signed-off-by: Tim Smith<EMAIL_ADDRESS>

There's some new spellcheck violations here:
/workdir/lib/chef/resource/bash.rb:83:38 - Unknown word (sstephenson)
/workdir/lib/chef/resource/bash.rb:139:16 - Unknown word (jxvf)


Flatpickr has a bug in time-only mode

I fixed a bug in https://github.com/flatpickr/flatpickr/pull/1938 so if this is merged you should update the flatpickr dependency immediately.
When you use the sw-datepicker like this:
<sw-datepicker dateType="time"
               v-bind:label="$t('my-date')"
               v-bind:config="{dateFormat: 'H:i:S'}"
               v-bind:required="true"
               v-model="entity.timeString">
</sw-datepicker>

it throws the following exceptions:
vue.esm.js?a026:1897 TypeError: Cannot read property 'children' of undefined
    at updateWeekdays (flatpickr.js?cf06:1345)
    at eval (flatpickr.js?cf06:2157)
    at Array.forEach (<anonymous>)
    at Object.set (flatpickr.js?cf06:2157)
    at VueComponent.updateFlatpickrInstance (index.js?a739:1)
    at VueComponent.handler (index.js?a739:1)
    at Watcher.run (vue.esm.js?a026:4571)
    at flushSchedulerQueue (vue.esm.js?a026:4313)
    at Array.eval (vue.esm.js?a026:1989)
    at flushCallbacks (vue.esm.js?a026:1915)

Hi @JoshuaBehrens
I've created a ticket for this: https://issues.shopware.com/issues/NEXT-4882
and subscribed to your flatpickr PR to get notified as soon as it is merged.
If we don't get back to you please remind us in this thread as soon as your PR is merged

Merged it is. No release yet

Release with my fix is public https://github.com/flatpickr/flatpickr/releases/tag/v4.6.3

Sorry it took so long, but I'm on it. Will probably be merged today

Nice. Thanks :blue_heart:


1 project per sample

Hello
Please make one project per sample, it is impossible to understand without knowing how to use Nez wich is not what a sample is made for

Each individual sample has all of its files conveniently located in the Scenes/SAMPLE_NAME folder as you can see here. There are no plans to make an individual project for each of them. It just doesnt make much sense to have that many projects for simple sample Scenes.


No stable API to access

The lib is currently missing a stable API that can be used at the moment.
I see two decent ways to handle it.

Build a mock/stub of all the needed API functions.
Wait till the entire library is done.

The API is pretty much done for the current version of the library


UI feedback when a pattern fails to compile

Maybe even include the GLSL error msg

Right now the node will delete itself if it fails to compile, but we should display (part of) the error message.


Fix README to match code files

Made the build script executable for Linux in order to match the README instructions
Changed the instructions for the web server port to reflect what's in launchSettings.json

Thanks for your contribution


Issue #80 Mark all as Read

Added an options menu in the MyCourses fragment which marks all modules of all sections of all courses as read.
Currently, this process takes a noticeable amount of time with large number of modules. I'm assuming it's the
Realm transactions which need to be done for each module. If possible, this should be made asynchronous so that
UI feels more snappy.

As you mentioned, Realm is doing the transactions on the UI thread, which is undesirable. How about we leave this PR open for a while till you are confident enough to add Rx support? Then the call can be made in the back, and observed on main.
You probably won't need too much Rx experience something of this level. A major Rx based rewrite can be pushed in a separate PR.

As you mentioned, Realm is doing the transactions on the UI thread, which is undesirable. How about we leave this PR open for a while till you are confident enough to add Rx support? Then the call can be made in the back, and observed on main.

I had looked into this further and from preliminary profiling, with around 110 or so unread modules, the entire process would take around 300ms. For every day purposes, this should be alright, presuming, at max there'll be 10-15 modules that need to be marked as read.

PR #245 migrates MyCoursesFragment to Kotlin. The head of this PR is based on that PR and hence this should only be merged in after that one.
This PR now uses Kotlin coroutines as a way of asynchronously marking all courses as read instead of introducing RxAndroid.


Convert BaseStorageIntegrationTest to a JUnit 5 extension

Better control over test lifecycle.

Does not require extending any particular base test class.

Let's merge this part now to enable repo reorg. I will do another PR for follow-up test improvements

Merging now to unblock repo reorg. I'll fix the cluster size handling in a follow-up RP.


Feat/answer to comments

My proposal for https://github.com/RedHatQE/pylero/issues/171.

PS: CI Pipeline is failing due to https://github.com/psf/black/issues/4175 :(

The update lgtm.
I saw the createComment API is marked as Deprecated by addComment, so this change is necessary and we could keep the create_comment function name for backward compatibility for our users.
For the Black and reorder incompatible issue, could you revert the change with pre-commit component changes until there is a solution for that, thanks.

Thanks, lgtm and test also pass


Lane Invasion when reversing doesn't work ?

Hi !
Lane Invasion Sensor doesn't recognize lane mark when going reverse...  Possible bug ?
Standard manual_control.py from Carla <IP_ADDRESS>

Note: it did works when half car when over the lane.

Standard manual_control.py from Carla <IP_ADDRESS> as well, looks fine on Town06...
Maybe try a little more distance?

what do you means by a little more distance?

when driving with low speed and some angles of the car, the sensor doesn't "see" the lane...

@Axel1092 could you have a look at this please?

Hi all, I have checked the issue and the problem is that the lane invasion sensor relies on the OpenDRIVE data to detect the lane markings but these are not exported within junctions. The example you are showing a situation which is labelled as junction in OpenDRIVE (even if only the opposite side of the road has connections with other roads.


should text shadow affect calculated color contrast?

Under AX_COLOR_01, it mentions that:

The Web Content Accessibility Guidelines 2.0 recommendation is that text has a contrast ratio of at least 4.5:1, or 3:1 for large text.

and it seems that the tool is looking at the contrast difference between the text and the background color.
However, even if you have white text on white background, the text is still readable if a text-shadow is set (effectively creating a background of a different color than the element's background).
To illustrate, consider that even though white text on white background has no difference in color and no contrast, it is still visible:

See example on codepen
Using the NoCoffee Chrome extension and setting contrast loss to 40, you can still see the white-on-white text with just the text-shadow property set:

The question was asked on the WCAG mailing list:

Can someone use a text shadow in order to meet the colour contrast requirement?

which led to this response:

Yes if done correctly, it can be used to create the contrast needed.

WGAG gives an example [found via this question on stack overflow] of using this method to increase contrast: "the background around the letters can be chosen or shaded so that the letters maintain a 4.5:1 contrast ratio with the background behind them even if they do not have that contrast ratio with the entire background."
p.s. for the record, I'm not meaning to have white on white but rather something like this:

It's close-but-no-cigar:

(It has text-shadow: 0px 0px 1px rgb(55, 90, 99) with color: #f7f6f6 on background: #39acc9.)

Adding a text-shadow allows me to pass the color contrast tests (4.5:1 ratio) with Color Contrast Analyzer but it looks like the Accessibility Dev Tools ignores text-shadows.


Now able to use and create custom tensorflow heads, embedders, and middleware.

Ref #134
I modified the following classes:

HeadParameters
MiddlewareParameters
InputEmbedderParameters

Adding a path property (or function as I mention in challenges).
Then I modified:

GeneralTensorFlowNetwork.get_input_embedder
GeneralTensorFlowNetwork.get_middleware
GeneralTensorFlowNetwork.get_output_head

To use these paths instead of their own local's.
I moved a local dictionary inside GeneralTensorFlowNetwork.get_input_embedder called mod_names to embedder_parameters.MOD_NAMES so that it's more accessible.
Challenges

InputEmbedderParameters.path can not be a property like the rest. You can call it with emb_type and the path will be created. But that's different than how most path's are made.

Pytest
I ran pytest locally and do not see any dramatic changes in the number of passing tests.

Can one of the admins verify this patch?

I couldn't find a similar area of code for mxnet to fix. If it exists I'll fix that too.
Also the CI does not appear to be passing master at the moment. It would be a bit difficult to replicate the environment on my system, but for the ones that dont have extra dependencies they pass last time I checked.
Will this be part of a versioned release?
Get Outlook for Androidhttps://aka.ms/ghei36

From: Gal Leibovich<EMAIL_ADDRESS>Sent: Monday, December 3, 2018 7:28:01 AM
To: NervanaSystems/coach
Cc: Ryan Peach; Author
Subject: Re: [NervanaSystems/coach] Now able to use and create custom tensorflow heads, embedders, and middleware. (#135)
@galleibo-intel approved this pull request.
LGTM. Need to have tests passing, and rebase.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/NervanaSystems/coach/pull/135#pullrequestreview-180741278, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AOJrye1aZvXMN4J0pgrvpGutr8s_wFsiks5u1RjRgaJpZM4Y8ksM.

Yep the CI wasn't initially set to run on forked builds (just changed and kicked off a build for this and other open PRs).

I could also run via the Amazon released docker container if that has all the dependencies. I heard about this project and am using it via the new Sagemaker RL.

Thanks for fixing the CI much appreciated.

What is your opinion on InputEmbedderParameters.path?

Although it differs from the solution used on other modules, I think that having it as method makes sense in this InputEmbedder case.
If you could rebase your branch, we can merge it to master.


[Issue] Disabled options on the original select element are visible/selectable on Tags UI

When the original select element has one or more disabled options, after the Tags UI gets populated items matching the originally disabled options will be still visible and selectable thus leading to potential validation issues.
Users should not be able to select an option that is disabled, Tags should provide a way to either disable or hide options as needed.

fixed in https://github.com/lekoala/bootstrap5-tags/releases/tag/1.4.3

Wow, superb. Thanks!


Adds more testing scenarios

To cover the functionality of this library I'd like to add more test scenarios that target different usage of the classes it patches.
Roadmap

[x] http.get
[x] http.request
[x] https.get
[x] https.request
[x] XMLHttpRequest
[x] fetch

This setup is not perfect, as it only tests the "full" scenarios: requests with query params, body. It would be nice to have one "plain" request, and one "full" request scenarios.


Upgrade to Angular 12

Upgraded to Angular 12, Structure is little but different but see it makes sense to have it like this.

Can somebody take care of this please.

It seems the current project has been abandoned. It might be time to create a fork that can hopefully be maintained?

I tried to use your fork. I am trying to customise the library. But when I am trying to build it, it creates blank QueryBuilderModule like:
export declare class QueryBuilderModule {
}

Try this https://www.telerik.com/kendo-angular-ui/components/filter/

I made this fork, not maintaining it beyond my own needs ATM though. https://github.com/raysuelzer/ngx-angular-query-builder


[Issue #52] Add schema content + presetNodes ModelObject constructor

Adds a new constructor to ModelObject allowing the use of schema content with preset nodes and nodes to build.
Description
As requested in the issue, there should be a constructor allowing prefetched schemas.
Related Issue

#52
Motivation and Context
Discussed in the issue.
How Has This Been Tested?

As I've not found tests for the others constructors involving a resource location, i have not figured out a way to test this overload. Any tips on this?
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

I'm writing the test case for my constructor, and I've stumbled on two issues. The first one is that an old test was not passing. I've managed to fix it on my last commit. The second issue is that the ModelObject(String modelString, boolean useRequiredOnly) constructor apparently leaves the object in an invalid state. I'm still working to see if there's something I can do. Any hints on that will be appreciated.

In does that ModelObject constructor work outside of your static constructor?  I would test that first.
Can you post your stack error, if you're getting one, or any additional information about how you know the instance is in an invalid state?
Can you post your test code?

I've checked out the develop branch from the main repo to run the test, and the test fails:
Tests run: 14, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 2.565 sec <<< FAILURE!
testStringJsonSchema(com.adobe.ride.utilities.model.test.ModelObjectTest)  Time elapsed: 0.003 sec  <<< FAILURE!
java.lang.NullPointerException
        at java.base/java.io.Reader.<init>(Reader.java:167)
        at java.base/java.io.InputStreamReader.<init>(InputStreamReader.java:113)
        at org.apache.commons.io.IOUtils.copy(IOUtils.java:2272)
        at org.apache.commons.io.IOUtils.toString(IOUtils.java:1041)
        at com.adobe.ride.utilities.model.test.ModelObjectTest.testStringJsonSchema(ModelObjectTest.java:94)

In my last commit (d20318b) I've managed to get my test working.

I've reverted the change to the existing test (see my comments in the resolved section above for details.), and added a new test for the code I've introduced. Everything seems correct now, so tell me if there's still anything to be improved. Thanks for your attention.

Thanks for running this down!  Can you do me a favor a write up a new issue regarding the Java 10+ issue?  We'll need to address it at some point.
Taking a final look now!

Now that you have an approved PR, you're welcome to have a Ride T-shirt, if you like.  I'll just need to know where to send it.  You can send me the address through the linked in link on the readme (so you don't have to publicly broadcast it).
If not, that's ok too.
Thanks again for contributing!  Looking forward to more ;)

I can't seem to find the link for informing the address. Where is it?

Hello.  Sorry for the delay.  Here is it: https://www.linkedin.com/in/ted-casey-0074064/
Thanks again!


Solve the issue with last letter of GuiMessageBox

Solves #315 by adding 2 to textWidth.
All other controls that includes text have 2 pixel extra when measuring text width but somehow it's omitted in GuiMessageBox.

@hanaxar thanks for the review!


Voices don't work

Describe your bug here.
I want to make a joke song chart but the voices don't work, other voices worked for different songs but this one just doesn't.
Command Prompt/Terminal logs (if existing)
no

Are you modding a build from source or with Lua?
Source, also I'm on Psych 0.7.3
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
yes this is a copy of my mod that im working on i added many new things

Describe your bug here.
I want to make a joke song chart but the voices don't work, other voices worked for different songs but this one just doesn't.
Command Prompt/Terminal logs (if existing)
no

Are you modding a build from source or with Lua?
Source, also I'm on Psych 0.7.3
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
yes this is a copy of my mod that im working on i added many new things

i had to reopen it for it to work 😭


fix(eslint-config-react-app): adjust TS override pattern to match files outside of current working directory

I'll start with that this is speculative pull request. I didn't find issue related to it, but given minimal code change I figured I might as well start with change proposal.
I'm not exactly sure how intentional it is currently for pattern to start with globstar here (initial pull request for TS support in this eslint preset already had it - https://github.com/facebook/create-react-app/pull/6513) and later on there were just few changes that unified it a bit, but didn't touch **/ part of pattern.
Impact that **/ prefix has is that eslint will not match files if they are outside of cwd. This can happen if you are working with yarn workspaces and your setup doesn't compile TS files and leave them as is.
Please check https://runkit.com/pieh/5e7150eb43c9c70019a74c16 for very simplified illustration of problematic scenario (with some parts of eslint inlined there with links to source), so it's easier to see what's going on. You can play with override pattern there to see that "TS path outside project directory" match changes when you remove **/ prefix from false to true
Ref: Issue on gatsbyjs/gatsby about TS linting in workspaces

It's not stale ;)

It's not stale ;)


See tasks after CTF is over

Would be nice to turn the tasks into read-only mode once the CTF is over so teams can still see the challenge description, link and how many teams solved it.
It'd really help when writing writeups :smiley:

I based this on https://ctf.internetwache.org but I can't find the code responsible for that, so it might be their modification.

@arturo182 Yes, this is indeed our modification. We'll create a fork, push our modifications and create a PR eventually.

@gehaxelt you the man!


Use env var for site url / redirect

This update makes use of the existing RAZZLE_SITE_URL env variable.  We'll need to update that to allow the site to redirect to a "generic" not found and maybe error page as needed.
@pcraig3 to be discussed should we keep the "base domain" alive to host generic pages (no location etc... needed)?

Grr, all of our heroku deploys are dependent on the PR number.
ie → https://ircc-development-pr-406.herokuapp.com/
If we don't want to add some logic to try and treat the heroku config differently (which I don't think we should spend a lot of time with personally), we could just hardcode this into Heroku as rescheduler-dev or something?

Question: does the site URL look like?

https://rescheduler-dev.cds-snc.ca
rescheduler-dev.cds-snc.ca

If we do the first one, it's hard for us to add locations in.
If we do the second one, we have to add in the https (or not, if we're on localhost) ourselves.
I'm thinking option 2.

Ya let's go with option 2

Cool, updated in Heroku as well.

I don't think this will get our heroku deploys to actually work again, but at least they won't immediately fail to build.


appaloosa 1.3

Adding .apk support
New Pull Request Checklist
Please mark the points which you did / accept.

[x] I read the Step Development Guideline
[x] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[ ] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[x] I read and accept the Abandoned Step policy

Sorry, forgot to mention: please use a three component version number - instead of 1.3 use 1.3.0


Make code style of hasBOM() consistent with getBOMCharsetName()

This is a minor, stylistic patch.
Background fact:  After the first call to getBOM(), the expressions getBOM() and byteOrderMark are equivalent.  (This fact isn't documented in the code, though.)
getBOMCharsetName() first calls getBOM() for side effect, then uses byteOrderMark within its body.
By contrast, hasBOM() mixes the two forms.
This patch changes hasBOM() to use the same code style as getBOMCharsetName().  In addition to consistency throughout the file, the refactored code makes it obvious that there is no null pointer exception, without the reader having to re-derive the background fact.

Coverage remained the same at 90.056% when pulling 46a2c6bebe30989d51d59e14def38a82e8edfd78 on mernst:consistent-check-and-use into 733dc26a281e235d52c965b6053a4e16bd4f85de on apache:master.

JIRA ticket: https://issues.apache.org/jira/browse/IO-553

Thank you!


FRC wingspan, no background

chris fix lines 80-something, whatever looks sketchy as hell

I fixed it guys


Support custom username field in logout

Add support for custom username fields in logout as well. This is the final missing piece, and is a companion to #36.

@skorokithakis Can you complete this PR by yourself and release on pypi? Btw, __dict__ used in current implementation too.
https://github.com/stochastic-technologies/django-loginas/blob/master/loginas/utils.py#L44


Shader Language: Implement read-only mecanism on builtins.

Fixes #14449.

@AlmightyScientist nice job! :)

I updated the PR (squashed) to make clang happy xD.
@Krakean Thanks :)
We should open an enhancement issue so :D
You mean auto-complete mecanics, or just the visual aspect of it? :)

@AlmightyScientist

You mean auto-complete mecanics, or just the visual aspect of it? :)

More likely both. Currently writing shaders in godot shader editor is not very handy due to semi-working auto-complete for constants/builtins & context-aware things :-
Just write a few yourself, you will feel it that autocompletion requires some attention :)

@Krakean

Just write a few yourself, you will feel it that autocompletion requires some attention :)

You're completely right. I think we should at first enhance Shader Compiler semantics (maybe we could decrease the time between Ctrl+S and effective compilation).

Hey, had a fix myself underway for this, so just commited it but thanks a lot for the contribution though. I'm surprised you could figure out how shader compiler works that easily.

You're welcome :)


Documentation Improvement

More required libraries are to be added in the requirements.txt file along with the existing libraries, for Face-X Implementation

U can add


Validate tileoffset table size

Applies similar logic to #824's ScanLine handling to prevent excessive memory allocation when reading truncated Tiled and DeepTiled files.
If the dataWindow and tileDescription indicate there are over a million tiles in level 0 of the file, will scan forward to check the file is at least large enough to contain enough tiles for that level. This prevents small specially crafted OpenEXR files causing large memory allocations in the library.
Addresses https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=25648

This looks fine to me


Incorrect type argument in a generic closure in DDC

The test (will be submitted as tests/language_2/closure_type_arguments_test.dart):
import "package:expect/expect.dart";

typedef MapFunc<S1, S2> = void Function(Map<S1, S2> arg);

class A<P> {
  final List barTypeArguments = [];

  void foo<Q, Q1 extends P, Q2 extends Q, Q3 extends P, Q4 extends Q>() {
    void bar<T1 extends P, T2 extends Q>(Map<T1, T2> arg) {
      barTypeArguments..add(T1)..add(T2);
    }

    // Call with explicit type arguments.
    bar<Q1, Q2>(null);

    // No explicit type arguments - should be instantiated to bounds.
    bar(null);

    // Partial tear-off instantiation.
    MapFunc<Q3, Q4> instantiated = bar;
    instantiated(null);
  }
}

abstract class MyIterable implements Iterable {}

main() {
  final a = new A<num>();
  a.foo<Iterable, int, List, double, MyIterable>();
  Expect.listEquals(
      [int, List, num, Iterable, double, MyIterable], a.barTypeArguments);

  // Test instantiation to bounds in the enclosing method.
  dynamic b = new A<int>();
  b.foo();
  Expect.listEquals(
      [int, dynamic, int, dynamic, int, dynamic], b.barTypeArguments);
}

Output:
Runtime error:
Expect.listEquals(at index 1, expected: <dynamic>, actual: <Object>) fails
at Object.dart.throw (http://<IP_ADDRESS>:44619/root_build/gen/utils/dartdevc/kernel/amd/dart_sdk.js:4212:29)
    at Function._fail (http://<IP_ADDRESS>:44619/root_build/gen/utils/dartdevc/pkg_kernel/expect.js:461:17)
    at Function.listEquals (http://<IP_ADDRESS>:44619/root_build/gen/utils/dartdevc/pkg_kernel/expect.js:260:25)
    at closure_type_arguments_test.main (http://<IP_ADDRESS>:44619/root_dart/out/ReleaseX64/generated_compilations/dartdevk-checked-sdk/tests_language_2_closure_type_arguments_test/closure_type_arguments_test.js:82:20)

Full log

It looks like this is DDC+Kernel.
This is probably the Object vs dynamic issue with respect to something that doesn't have an explicit bound (in this case, Q). If I recall correctly, Kernel added an API to distinguish between <Q extends Object>, <Q extends dynamic>, and <Q>, but I have not updated DDC+Kernel to use it yet. Prior to that, we did not have enough information to distinguish these 3 cases, so we could not implement the desired instantiate to bounds behavior.

Simplified repro:
main() {
  f<T>() => T;
  print(f());
  print((f as dynamic)());
}

it should print dynamic for both, but DDC+Kernel prints Object for the second one. This is because the runtime type information stores Object for the bound of T (which is the correct bound, but not correct for default instantiation).

https://dart-review.googlesource.com/c/sdk/+/72081


Idea: include option for 'Dunkelflaute' similar to the 'Cold snap' option

I have often been told that the electricity system needs to be robust enough to provide security of supply even during a so-called Dunkelflaute. This German term describes a once-in-so-many-years event in which there is little sunshine an no wind in a country or area (of Europe).
I think the ETM can be improved by including an option for this, just like the current option for 'Cold snap' included at the behest of Gasunie. Of course, the cold snap and Dunkelflaute are likely to be related, and therefore not independent, but the extent to which they are I leave to be researched.
To be researched:

Just what happens during a Dunkelflaute to wind and solar curves, or even temperature curves?
Does a Dunkelflaute significantly impact the average temperature in a year in a predictable way?
How often does such an event occur and how long does it last?
etc..

Notifying @ChaelKruip @jorisberkhout @michieldenhaan @MartLubben @DorinevanderVlies

I happen to know that Ecofys and CIEP have published on this topic or have expertise in this matter

Closing, this is now supported! https://github.com/quintel/etmodel/commit/f72e9edd9663ff7b756afec0bb126f09a6ae2ca1


Add CloudTrail logging

Apparently CloudTrail is enabled by default for AWS accounts, so we should be fine with this.


Unsupported Units CF

I'm in the US (Washington state to be exact) and it look like CF (cubic feet) is not a supported unit. Complete error message included below. The website and my phone app report volume in gallons, but it looks like the meters are natively reporting CF. This is with the most recent code base.
Logger: homeassistant
Source: custom_components/eyeonwater/eow.py:155
Integration: EyeOnWater ([documentation](https://github.com/kdeyev/eyeonwater))
First occurred: 9:42:02 PM (2 occurrences)
Last logged: 9:44:47 PM
Error doing job: Task exception was never retrieved

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 182, in async_refresh
    await self._async_refresh(log_failures=True)
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 283, in _async_refresh
    self.async_update_listeners()
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 110, in async_update_listeners
    update_callback()
  File "/config/custom_components/eyeonwater/sensor.py", line 85, in _state_update
    self._state = self.meter.reading
  File "/config/custom_components/eyeonwater/eow.py", line 155, in reading
    raise EyeOnWaterAPIError(f"Unsupported measurement unit: {read_unit}")
custom_components.eyeonwater.eow.EyeOnWaterAPIError: Unsupported measurement unit: CF

Fixed. Please update and verify.
Thank you for reaching out!

Yes, this fixed it. Thanks for the quick response.


Unrecognized syntax in tarmac file

tarmac-browser doesn't like the following entry:

2884500 ps  ES<PHONE_NUMBER>000188:9e6703e0) O el3h_s:         FMOV     d0,xzr
R V0<63:0><PHONE_NUMBER>000000
R V0<127:64><PHONE_NUMBER>000000

$ ./tarmac-browser tarmac._CLUSTER_g_core_0__g_little_u_helios_core_u_vcpu_u_cpu_u_helios_dpu_u_follow_.t0.log
index file tarmac._CLUSTER_g_core_0__g_little_u_helios_core_u_vcpu_u_cpu_u_helios_dpu_u_follow_.t0.log.index does not exist; building it
tarmac._CLUSTER_g_core_0__g_little_u_helios_core_u_vcpu_u_cpu_u_helios_dpu_u_follow_.t0.log:199: Unrecognised token
                    R V0<63:0><PHONE_NUMBER>000000
                        ^

Thanks for the report. Can you tell us what Tarmac producer emitted that snippet?

The tarmac file was generated by the Neoverse E1 cycle model.


Running two services test with different volume to be run in parallel

Prerequisites

[X] I am using the latest version of Locust
[X] I am suggesting a new feature, not asking a question

Description
We are evaluating locust tool for performance testing, for a use case containg multiple services I would like to try and implement a locust solution. Say we have multiple services (AIP), each services will have its own volume to test, One service  with 10k volume in an hr and another services with 3k volume in an hour. Wants to run both in parallel as separate/multiple users. How this can be implemented. Please share a basic model code to achive this load/volume for the User/script, each users will run parallel, if required we can have multiple users. Appreciate your help.

Easiest would be to make two User classes, one for each service.
Use wait_time = constant_pacing(x) on both user classes to constrain the number of task iterations per second for each user to one per x (x needs to be bigger than the max expected response time).
Then launch (wanted throughput) * x number of users of each class (by setting the weight of service A to 10 and service B to 3). Or launch the same amount of users of both classes and adjust wait times to get the desired request duration.
locust-plugins has a wait_time function called constant_ips for ”global” request rate limiting, you might want to explore that one as well.


Add any remaining hardcoded metadata options for testcases

Have gone through the current 13 test cases, need to add:

[ ] Breaking Waves
[ ] Flexible structures

Added both and wrote up documentation for this on the wiki


Add option to not indent conditional compilation blocks

When adding conditional compilation to a large block of code it is often undesirable to have that large block be indented.

Would this also mean this formatting would work:
func foo() {
#if BAR
    print("hi")
#endif
}

Or would it de-ident it to:
func foo() {
#if BAR
print("hi")
#endif
}

There's no ability yet for lines starting with # to be always be flush-left; this change only affects the indentation of the contents of a conditional compilation block.
So if you write this:
func foo() {
#if BAR
  print("hi")
#endif
}

Then the only possibilities are this if the configuration option is true (default):
func foo() {
  #if BAR
    print("hi")
  #endif
}

and this if it's false:
func foo() {
  #if BAR
  print("hi")
  #endif
}


Feature/sample function greeter azure

This PR includes code for greeting sample for azure function.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: whoswendy:x: Wendy HuWendy Hu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


NO_PROXY support

can these utilities respect the NO_PROXY environment variable in the docker_rules? or is there a way to specify this ?

None of these are compatible with http it seems. Is there a reason why this can't be supported through an option?


Blog post about new docs

Pretty self-explanatory. I dated it July 5 so we can push stuff on July 4, blog about it July 5 (and it won't be on a holiday).

@facebook-github-bot shipit


open to close transfer more smoothly

add code to initialize static var "prevDuty" in function

When change speed in closed-loop, it’s necessary non-blocking function.


translating to japanese

I'll translate the rest of the pages to Japanese. r? @chikoski may help reviewing.

Let's see what @chikoski says; happy to merge whenever you think it's ready.

Thank you for involving me. I have added some comments to the translations.
From my Japanese localization experience, I can say URLs in links should not contain locales, such as ja-JP. This is because users would like to  acquire the pages in their favorite locales, which should be set as their default locales in browsers' settings.

I'll reflect reviews after all the pages are roughly translated to Japanese, so go ahead reviewing without caring about conflicts between new review and review reflection.

@chikoski @sinkuu I found that faq is too big and it worth little to block merging until it gets completed. Let's incrementally translate it.
Thus, please start reviewing the other pages.

Thanks again @KeenS! Is this ready to go?

@alan-andrade I'd like to wait for @chikoski 's comment. Please stay a while.

Sorry for keep you waiting, @KeenS
I totally agree with you to translate FAQ incrementally.

@chikoski Thank you. And can it be considered you mean nothing to say with the translation?
@alexcrichton Please go ahead to merge.

:eyes:

Ah sorry was quite busy these past few days, thanks!


Crash when creating task: TimeZoneNotFoundException

On http://allready-d.azurewebsites.net/ or when running locally I get the following error when trying to create an activity:

Looks like it might just be that the Create method in TaskController.cs needs updating to set a value for TimeZoneId on the view model.

I will take a look at this.

Fixed in PR #407


--latest-stamps and resuming

There's still one small problem with --latest-stamps, when used together with interrupting and resuming a download. And I'm not sure what would be the best way to fix it.
In the first run, if the download is interrupted, then the last timestamp is not stored, because the exception handling skips the code that writes the timestamp. But this is actually a good thing.
In the next run, continuing the interrupted download, the last timestamp is still the same of the first run. As I said, this is good because if the time were saved and used in this second run, nothing would be downloaded.
However, when this second run finishes, it stores as last timestamp the time of this second run, so any media included between both runs would be ignored in the next update.

One possibility would be to store in the json file the timestamp, but I'm not sure how could this be done. It would require some huge and intrusive changes.

Another idea, that seems more feasible, is to store not the current timestamp in the stamps file, but the timestamp of the newest media downloaded. But this would require having NodeIterator cache the first item it returns (or at least its timestamp) so that it can be queried afterwards, and including this timestamp in FrozenNodeIterator so that it is stored and loaded in subsequent runs.

Please let me know what you think and if there's another way to solve this problem.

Hi! Thanks a lot for sharing this observation. @Thammus and I just thought about it and come up with the following suggestion that is very close to your second idea:
Besides post-timestamp (or last_scraped as the variable is called in the code), add a field to the latest stamps file start_of_first_incomplete_run, which stores either a datetime or None and defaults to None.
After the call to posts_download_loop, update these two fields in the latest stamps file in a manner that depends on (a) whether we resumed or not in the current run and (b) whether the current run was aborted or terminated normally (completed):

Resumed?
Completed?
Action to latest_stamps

Not resumed
Completed
last_scraped := Date of first post, start_of_first_incomplete_run := None

Not resumed
Aborted
leave last_scraped unchanged, start_of_first_incomplete_run := Date of first post

Resumed
Completed
last_scraped := start_of_first_incomplete_run, start_of_first_incomplete_run := None

Resumed
Aborted
leave last_scraped unchanged, leave start_of_first_incomplete_run unchanged

Date of first post is the date of the first post that the iterator returns in the current run. I think one could easily implement a copy function to NodeIterator, which just copies a NodeIterator. Then, the date of the first post is next(copy(node_iterator)).date_utc.
Whether the current run resumed is start_of_first_incomplete_run is not None.
Whether the current run completed or not can be determined by whether a KeyboardInterrupt or AbortDownloadException is raised in posts_download_loop. They must be re-raised when handling the exception.
I think this will give us a quite efficient implementation that will not redownload posts unnecessarily, it will not leave out posts, it does not depend on the correctness of the system time, and continues to cleanly store the latest-stamps-related information in the latest stamps file.

A copy function would work, but wouldn't it cause at least one more request to IG's API to fetch the first page again? Given the frequency of "need to wait X seconds" and even the occasional 429, it seems preferable to avoid queries as much as possible.
If the date of first post can be obtained, I don't think it's necessary to store it as start_of_first_incomplete_run. As long as last_scraped is not updated when the download is aborted, and on the next run the same iterator is used (which will return the same date of first post, regardless of how this is implemented), then that date would be stored when the download completes, which is what's necessary. Or am I missing something?

A copy function would work, but wouldn't it cause at least one more request to IG's API to fetch the first page again?

No. The first request is done in __init__. If copy copies all the instance's fields of a completely initialized NodeIterator, no extra request is needed.

As long as last_scraped is not updated when the download is aborted, and on the next run the same iterator is used (which will return the same date of first post, regardless of how this is implemented), then that date would be stored when the download completes, which is what's necessary.

On the next run the same iterator does not neccessarily return the same date of the first post, since new posts might have been created in the time between the runs. That is why we suggested to store the date of the newest post of the first incomplete run.

I think I understand, but copy would duplicate the iterator from the point it's at, so if 5 items have been returned, and copy is called, both the copy and the original would return item 6. For our specific case, it would work (as only as copy is called just after the iterator is created), but maybe this is somewhat limiting for other possible uses.
I have written a proof of concept in #1219, but using a different strategy. The iterator itself caches the date of the first item returned, and this value is also saved and restored. Having to cache that value is not super elegant, but the code seems to work.


Adds right-click options to Configuration side bar

Closes #13003
Based off the issue above, I came up with the concept of right-clicking the side bar wrench (Configuration) so it will appear these options:

"Restore default plugins" (enable enabledByDefault plugins & disable everything else including external plugins);
"Disable all plugins" (except Configuration & Dev tools if enabled);
"Enable all plugins" (except GPU plugins);
"Revert changes" (this will basically revert the plugin status after using one of the above options to last stored).

I think this is more than just a master switch/using rs=vanilla , specially the "Restore default plugins" option might come in handy on discord #support.

Disabling every plugin is a bad idea because it will make people think they can do that as a substitute to reproducing a bug on vanilla. Enabling everything is also a bad idea, as the things that are disabled by default generally have a good reason to be and may actively conflict with other plugins.
Having a restore default plugins button seems ok, but I don't think this is a good place for it. Maybe it could live in the info panel.

As suggested, removed everything but restore default plugins which I added to the info panel.

And also a confirmation dialog:


Upgrading vscode-azureappservice package to T2 + split utils packages

I'll try to call attention to any files where something more interesting than updates to imports occurred.

Were the changes to triage-actions/**/*.js files intentional?

@alexweininger That's Git trying oh-so-hard to be helpful. Every time I push a commit from my Windows machine it reapplies these changes. My plan was to un-change them one last time from a Unix machine before submitting.


Update Java and Maven version

This pull request aims to adopt a more widely used Java version (11, open-jdk). Furthermore, it fixes an outdated dependency to a maven binary, which was not being hosted anymore.
Additionally, the readme was updated to encourage the use of Github Open ID Connect Web Identities, rather than relying on long-living AWS credentials.

@roechi Sorry, it took long for me to get to this. Great PR. I have reviewed this, and I agree with the upgrades and approach to using OpenId Connect.
Let me know if it's ready for merging and I can take care of it.


Webscrapp from chess manager tournaments that are listed on another webpages

I need to figure that out how to implement in code to intelligent discover how many subpages result set has on website end to include it in URL  (it works  for example as parameter &offset=50 )

With the following commit  support has been added


Adjust some debug levels

This is the first issue I found on stage. The affectedBy method print tons of INFO log and flushed everything away. We have a dedicated API to check it in 2.0. So we don't need the INFO log anymore.

simple log change. I merge it.


URL rewrite rules

One possible reason for the failure is URL rewrite rules in the web.config and the error message provides no details in such case.
I had <system.webServer><rewrite> section in web.config to implement automatic redirection from HTTP to HTTPS and it prevented debugging. As I do not really need that on the development machine I moved that setting to the web.Whatever.config files used when publishing and this way things work.
You may want to add this to the list of possible causes.

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

ID: 12b3da70-ab86-495d-abfe-384f369660fe
Version Independent ID: 2456d73a-68bd-cc6b-bff0-e7666e53f404
Content: Error: Unable to Start Debugging on the Web Server - Visual Studio
Content Source: docs/debugger/error-unable-to-start-debugging-on-the-web-server.md
Product: visual-studio-windows
Technology: vs-ide-debug
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@JendaPerl Thank you for the feedback, assigning this to the author for review.

@JendaPerl What was the full error message that you saw? Some of the more detailed messages described in the article mention URL rewrite rules, so it would be helpful to know the more exact message in order to update the doc. Thanks!

I'm sorry, I did not notice your response.
I added the section back to the web.config and tried to debug the project. I got a dialog box with title "Microsoft Visual Studio" and text "Unable to start debugging on the web server. Operation not supported. Unknown error: 0x80004005. Click Help for more information."
The Help loads the https://docs.microsoft.com/en-us/visualstudio/debugger/error-unable-to-start-debugging-on-the-web-server page.
I did not find anything related in the Event Viewer, but maybe I just did not know where to look.

Thanks for responding. So, you used the Event Viewer to get details on the error?

Oh, misread that. You tried the Event Viewer, but it didn't help. Looks like I need a new entry for "Operation not supported".

Exactly. The "Operation not supported. Unknown error: 0x80004005" was all the error details I could find anywhere. Luckily I remembered the recent addition of the rewrite section in the web.config and tried to remove it and that fixed the problem.
Please add this into the list of possibly causes of the error in the document. :-)

Updating doc with new error message.


Remove 3.6 support / discontinue less available platforms

Codecov Report

Merging #1267 (94b396f) into master (4d7d6b6) will decrease coverage by 0.83%.
The diff coverage is 94.79%.

@@            Coverage Diff             @@
##           master    #1267      +/-   ##
==========================================
- Coverage   97.28%   96.44%   -0.84%
==========================================
  Files          67       85      +18
  Lines        4235     5936    +1701
==========================================
+ Hits         4120     5725    +1605
- Misses        115      211      +96

Impacted Files
Coverage Δ

tests/test_binary.py
100.00% <ø> (ø)

httpie/compat.py
31.11% <27.90%> (-68.89%)
:arrow_down:

tests/conftest.py
77.14% <58.33%> (-9.82%)
:arrow_down:

httpie/output/lexers/http.py
59.09% <59.09%> (ø)

tests/test_ssl.py
91.01% <66.66%> (-3.93%)
:arrow_down:

httpie/output/lexers/common.py
75.00% <75.00%> (ø)

httpie/output/lexers/metadata.py
81.25% <81.25%> (ø)

httpie/manager/__main__.py
82.35% <82.35%> (ø)

httpie/models.py
93.93% <88.88%> (-3.43%)
:arrow_down:

tests/test_uploads.py
97.66% <92.15%> (-2.34%)
:arrow_down:

... and 67 more

Continue to review full report at Codecov.

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


Warning: Unknown event handler property onInteraction for Tab in React 16.8.6

This is my first time using the sandbox. Very cool. You can see the warning in the console.
https://codesandbox.io/s/rmwc-sandbox-ywo3f

What RMWC Version are you using [major.minor.patch]:
5.5.1

Name your build system [Webpack, Rollup...]:
create-react-app webpack

Describe the bug with as much detail as possible:
When adding an event handler onInteraction to a Tab, a warning appears in the console.
React seems to not want an unexpected attribute on the DOM.

I'm also using an onActivate event handler on the TabBar, and there is no warning shown for that handler.
I'm pretty new to React, so I'm doing my best to guess what might be happening. Perhaps RMWC needs to process and remove the attribute before writing the DOM element.

What happened, and what was supposed to happen:
Expected to see no warning.

Thanks so much for taking a look!

Yeah generally those types of bugs are from a prop that should have been removed before they where passed on to a dom element. Thanks for reporting, I’ll take a look.

Fixed in 5.6.0

Thanks so much!
On Jul 21, 2019, 4:21 PM -0400, James Friedman<EMAIL_ADDRESS>wrote:

Fixed in 5.6.0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Shutdown hang

Using poweroff causes the kernel to panic and the emulator to hang at 100% CPU usage. The following screenshot shows sudo strace poweroff through SSH and the kernel panic dumped to the screen.

Confirmed as a known issue in QEMU at https://bugs.launchpad.net/qemu/+bug/1840777

Confirmed as a known issue in QEMU at https://bugs.launchpad.net/qemu/+bug/1840777

For the time being there's probably nothing we can or should do in this project to fix it.


Not getting correct GoogleOath2 details when signing up by OAuth access token

Full details located on stackoverflow here:
http://stackoverflow.com/questions/28237885/python-social-auth-not-getting-correct-googleoath2-details
I want to login a user using the python-social-auth functionality for Google Plus signin in Django. When logging in from my website, everything works fine and the correct details are added to the database.
However, I want to authenticate from my Android application as well. The user logs in in the application, which then sends the access token to the django API, which handles the login process in the following code, adapted from the documentation:
@csrf_exempt
@serengeti_api_request
@psa('social:complete')
def login_social_token(request, backend):
# Ensure the token has been specified.
token = request.META.get('HTTP_ACCESSTOKEN')
if token is None:
raise SerengetiApiRequestException('Access token is missing!')
# Login the user for this session
user = request.backend.do_auth(token)
if user is None:
    raise SerengetiApiRequestException('Could not authenticate user!')

login(request, user)

# Store the email address if one has been specified (e.g. Twitter)
email = request.META.get('HTTP_EMAIL')
if email is not None:
    user.email = email
    user.save()

# Prepare the parameters to be returned
response = dict({
    'id': user.id,
    'first_name': user.first_name,
    'last_name': user.last_name,
    'api_key': request.session.session_key,
})

# Return a 200 status code to signal success.
return HttpResponse(json.dumps(response, indent=4), status=200)

When logging in from the website, the social_auth_usersocialauth table contains:
id | provider      | uid       | extra_data
.....................................................................................
10 | google-oauth2 |  | {"token_type": "Bearer", "access_token": "", "expires": 3600}
However, when logging in from the application using the above function, the operation completes ok, but the entry in the table looks like this:
id | provider      | uid     | extra_data
.....................................................................................
10 | google-oauth2 |  | {"access_token": "", "expires": null}
Also, the auth_user table contains a username like eeed494412obfuscated48bc47dd9b instead of the Google Plus username and the email field is empty.
What am I doing wrong and how can I obtain the same functionality as I get on the website?
I would like to mention that I have implemented Facebook and Twitter authentication from the Android application, which call the above-mentioned function and store the correct details, only Google Plus is causing problems.

I'm getting the same problem with the extra_data = {"access_token": "", "expires": null}, What am I doing wrong? i had very similar configuration. Why did you close the issue?

@zetahernandez I closed it because it wasn't really an issue with PSA. Have a look at my answer to the stackoverflow question linked above.


Rename inconsistent whitelisted_params and allowed_params methods >> permitted_params

In advance of an error correction PR that will need review

Coverage: 93.36%. Remained the same when pulling 8381e148193975eaeac8b24145385dff3a201990 on nimmo-rename-permitted-args-params into eeae8ea12421e3e2bb77bbb3dfea1e65df7aa49c on main.


Update insomnia to 5.16.1

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

https://www.virustotal.com/#/file/aa5ba1636e5cbea72b547ea7bd32bacdec296b0c3f8c3230d2bebc434af348a6/details
aa5ba1636e5cbea72b547ea7bd32bacdec296b0c3f8c3230d2bebc434af348a6  insomnia--5.16.1.dmg

Executable=/Volumes/Insomnia 5.16.1/Insomnia.app/Contents/MacOS/Insomnia
Identifier=com.insomnia.app
Format=app bundle with Mach-O thin (x86_64)
CodeDirectory v=20200 size=272 flags=0x0(none) hashes=3+3 location=embedded
Signature size=8916
Authority=Developer ID Application: Gregory Schier (7PU3P6ELJ8)
Authority=Developer ID Certification Authority
Authority=Apple Root CA
Timestamp=30 Apr 2018 at 22:50:22
Info.plist entries=23
TeamIdentifier=7PU3P6ELJ8
Sealed Resources version=2 rules=13 files=11
Internal requirements count=1 size=176

/Volumes/Insomnia 5.16.1/Insomnia.app: valid on disk
/Volumes/Insomnia 5.16.1/Insomnia.app: satisfies its Designated Requirement


Support plugging in your own OpenTelemetry exporter

[as opposed to using an APM vendor's agent that wraps this project and embeds the vendor's exporter]
This is a bit tricky because the OpenTelemetry API and SDK that the agent use are shaded, so the exporter must be shaded as well to match.
There are (at least) two options for shading the user exporter:

Provide a tool that end users can run to shade the OpenTelemetry exporter that they want to use
Shade the OpenTelemetry exporter dynamically at runtime before loading it

What if we took the more painless route of using the native OpenTelemetry Collector protocol and have people use a collector "sidecar" to do the translation to the target protocol?

I think that allowing people to use OTel without a collector and sending to their vendor of choice is still very important. I'd hate to have to stand up a collector to do local development, for example.

I'm taking a stab at this. I'll implement some kind of prototype we can use as basis for further design. Please assign this to me!

I came up with a prototype design for this. Here are some of the considerations:
Early or late shading?
The best spot to load custom exporters seems to be in the TracerInstaller class under agent_tooling. Loading it in this context means that the OpenTelemetry API needs to be shaded. This can be done in three ways: At exporter build time, using some CLI tool on a vendor JAR or dynamically at runtime. Each approach has its pros and cons.
Exporter build time
This involves adding a mandatory list of relocation statements to the gradle script that builds the exporter. Since the OpenTelemetry API is brought in as a compileOnly, the only effect this would have is to change every reference to the API to a shaded one.
Pros

Easy to implement
Requires no additional code
Requires no costly processing at agent load time

Cons

Requires awareness of the auto agent from exporter developers (probably needed anyway, see below)
Exporters will have to be recompiled with a new version of their gradle script if the auto agent ever changes its shading.

CLI Tool
With this approach, we would provide a simple CLI tool that allows users to apply the required shading etc to any exporter JAR.
Pros

Relieves exporter developers from having to deal with auto agent specifics.
Requires no costly processing at agent load time

Cons

Doubtful that it's possible to completely relieve the exporter developers from any awareness of the auto agent, since we need a way to specify bootstrapping etc.
An additional tool to write and maintain

Runtime shading
Here, shading would be done at exporter load time and the exporter could be made completely unaware of the auto agent (restrictions apply, see below).
Pros

Relieves exporter developers from having to deal with auto agent specifics.
Exporters won't break if we change the shading rules.
No extra tools to run

Cons

Doubtful that it's possible to completely relieve the exporter developers from any awareness of the auto agent, since we need a way to specify bootstrapping etc.
Relatively difficult to implement
ByteBuddy doesn't seem to have this type class transformation functionality out-of-the-box (I could be wrong here). ASM has it, but don't want to drag in another bytecode manipulation package.
Processing could be quite costly and we're trying to reduce the load time.

Current design
Shared library
The exporter and the auto agent need to share a very small package in order to obtain a factory, share configuration etc. This is currently done through the exporter-support module in the auto agent. This should probably be moved to opentelemetry-java once we've nailed down the design.
The package contains two classes:
ConfigProvider a simple interface with basic getters that allows the exporter to read arbitrary configurations from the agent. A DefaultConfigProvider implementation exists that provides configurations from the agent config.
ExporterFactory interface that specifies the creation of a SpanExporter based on a ConfigProvider. The exporter project must implement an instance of this and map the configuration to the programmatic way of creating an exporter.
Bootstrapping
In order to be compatible, an exporter needs to provide two things (in addition to the factory described above).

A Bootstrap class. This can be any class that implements a getFactory() method.
A Ota-Bootstrap-Class proprety in the manifest that contains the fully qualified name of the bootstrap class.

The bootstrap procedure in the auto agent works as follows:

Read the ota.exporter.jar property
Create a URLClassLoader based on that property.
Load the MANIFEST.MF and extract the ``Ota-Bootstrap-Class``` property.
Using the newly created ClassLoader, try to load the class referred to by the manifest property.
Using reflection, call the getFactory() method to get a factory.
Construct a ConfigProvider
Call fromConfig() on the factory.
Register the exporter with the tracer.

Sample exporter
A sample exporter is available under the dummy-exporter module. It simply prints the spans to stdout. This exporter is needed for testing, but should probably be moved under an existing module.
Current code:
https://github.com/prydin/opentelemetry-auto-instr-java/tree/prydin-exporter-experiment

would the exporter-build-time option mean that non-auto-instrumentation users would have to deal with oddball packaging?

Good question! I think I would generate two separate jars. One standard jar with no weirdness and one that has the auto stuff in it. That's what I'm planning to do with the vendor specific one I'm working on now.

Of course, the special auto jar could be generated using a separate gradle task so it's not done by default, but the logic would be the same.

Cool. I looked at your dummy exporter. Seems like this is not a lot to ask of people who want their exporters to work auto-magically with the auto-instrumentation system. Would they also need to provide a standard configuration mechanism for their exporter?

All they have to do is to ready properties of their choice from the ConfigProvider they get passed. Then they can use builders or whatever they want to set it up. Like this:
public SpanExporter fromConfig(ConfigProvider config) {
    WavefrontSpanExporter.Builder b = WavefrontSpanExporter.Builder.newBuilder();
    String proxy = config.getString(PROXY, null);
    String url = config.getString(WAVEFRONT_URL, null);
    if (proxy != null) {
      if (url != null) {
        throw new IllegalArgumentException(
            "Settings " + PROXY + " and " + WAVEFRONT_URL + " are mutualy exclusive");
      }
      WavefrontSpanExporter.ProxyClientBuilder pb = b.proxyClient(proxy);
      pb.metricsPort(config.getInt(METRICSPORT, 2878));
      pb.tracingPort(config.getInt(TRACEPORT, 30000));
      pb.flushIntervalSeconds(config.getInt(FLUSH_INTERVAL, 5));
      return pb.build();
    } else if (url != null) {
      String token = config.getString(TOKEN, null);
      if (token == null) {
        throw new IllegalArgumentException(
            "Setting " + TOKEN + " must be specified for direct connections");
      }
      WavefrontSpanExporter.DirectClientBuilder db = b.directClient(url, token);
      db.flushIntervalSeconds(config.getInt(FLUSH_INTERVAL, 5));
      return db.build();
    } else {
      throw new IllegalArgumentException(
          "Either " + PROXY + " or " + WAVEFRONT_URL + " need to be specified");
    }
  }

The capability of configuring an exporter from some kind of config should, in my opinion, be inherent to opentelemetry-java at some point. But that's above my pay grade right now. :)

Hmm. It doesn't really make sense to have exporters have to depend on code in the auto-instrumentation project (ConfigProvider, etc). Can you make a proposal (issue) in the java project for having this ConfigProvider be a part of the SDK?

 They should definitely be part of the SDK. I just had issues linking the auto instrumenter to an unreleased jar in my file system, so I resorted to this in order to produce a PoC. I forgot to mention that! Nothing should ever be dependent on the auto instrumenter in my opinion. It's a black box.

I will open an issue once we decide that this is the way forward. Let's discuss on the Thursday call!

Seems like this is not a lot to ask of people who want their exporters to work auto-magically with the auto-instrumentation system

This seems reasonable to me also.
One thought so far, can we use SPI / service loader instead to look up and instantiate the bootstrap class?

 I wanted to make this configurable from system attributes and/or environment variables only.

Instead of requiring shading at all, could the exporter be wrapped in an adapter/bridge at load time?

Instead of requiring shading at all, could the exporter be wrapped in an adapter/bridge at load time?

Not sure I understand how that would work. Can you elaborate?

Re: ServiceLoader, it would just be used for the Bootstrap Class discovery (replacing the Ota-Bootstrap-Class MANIFEST entry). It wouldn't affect the configuration proposal.

Re: ServiceLoader, it would just be used for the Bootstrap Class discovery (replacing the Ota-Bootstrap-Class MANIFEST entry). It wouldn't affect the configuration proposal.

Hmmm... Yeah, that might actually work. Trying it out...

hits head I just realized that the ClassRemapper from ASM is available through ByteBuddy. OK, then the dynamic runtime shading thing isn't that hard to do I suppose.

Pushed code based on feedback from @trask, @tylerbenson and @jkwatson

Using the SPI pattern with ServiceLoader
Implemented runtime shading

Have a peek at https://github.com/prydin/opentelemetry-auto-instr-java/tree/prydin-exporter-experiment of you like!

This all sounds good to me. Can you submit a PR for more detailed review? Thx!

It sounds like remapping at load time is the way to go.  Nice job.  One thing to consider... what happens if the exporter exists on a user's classloader and isn't on the system classpath (eg spring boot jar)?

Initial implementation completed by #159 and #171


Failed to connect: Unable to create ProxyServer with ProxyManager: com.cognifide.aet.proxy.RestProxyManager

Hi
I'm trying to run your tools but I have found an error during execution suite.xml sample.
COMPARED: [success: 0, total: 0] ::: COLLECTED: [success: 0, total: 1]
[Collector-4] Couldn't process following url http://www.cognifide.com because of error: Failed to connect: Unable to create ProxyServer with ProxyManager: com.cognifide.aet.proxy.RestProxyManager
If I'm trying to run the test without proxy set I get no proxy error but get "Processing error occurred" visible in test report and the same info in execution log ('COMPARED: [success: 0, total: 0] ::: COLLECTED: [success: 0, total: 1]')
Can you please advise?

Hi,
could you please tell:

what version of AET you use
how did you setup AET instance (Docker / Vagrant / Manual)
attach your suite.xml

Currently there is too little information to say more.

Hi @Skejven,

what version of AET you use ( latest downloaded from repo )
how did you setup AET instance (Docker / Vagrant / Manual) (same way how was described in your tutorial, however, I had an issue at the end which after using  vagrant up command bellow notification pup-up:

Vagrant was unable to mount VirtualBox shared folders. This is usually because the filesystem "vboxsf" is not available. This filesystem is made available via the VirtualBox Guest Additions and kernel module. Please verify that these guest additions are properly installed in the guest. This is not a bug in Vagrant and is usually caused by a faulty Vagrant box. For context, the command attempted was:
mount -t vboxsf -o uid=1000,gid=1000 keys /keys

above issue was solved after vagrant plugin repair and vagrant provision
I have same versions of vbox and others tools as described in manual.

attach your suite.xml

default suite from your manual:
https://github.com/Cognifide/aet/wiki/AETIn10Minutes
Thanks for replying.

Could you pleas ssh to your vagrant instance and check connectivity to http://www.cognifide.com (e.g. with curl or wget)? It looks like there is some problem with network.
If you fail to connect to the server, please try restarting the VM. If there will be still problem, please remove the vagrant instance and start it over.
You may also want to try https://github.com/Skejven/aet-docker if vagrant setup fails.


self: shrubs render themselves

blue is good for overlays
mast is good for recursive rendering
??? is good for non-overlay, non-recursive rendering
Add a third rendering methodology, self, which simply passes http requests to the shrub specified in the path. If Sky doesn't know about any other strategies for this path (which will typically be the case), it should use self as the default rendering framework.

The Sky navbar now displays three hardcoded renderer options: hawk, tree, and self.
Next Up:

For apps we want to keep around, rewrite all hawk renderers  to self and obfuscate this option from the UI, routing to self as the default
If a dev wants to use mast as their default renderer (or blue, for whatever reason), they should poke that strategy into Sky in that imp's init arm
If default rendering nacks, redirect to tree
For overlays and other renderers, users should still be able to type in a strategy, and if this succeeds, there should be a UI element for saving it (as well as un-saving)


Search Service Contributor is both "Generally available" and "Preview"

The table on this page mentioned that this role is "Generally available and preview". This is because this role gives permissions to both the management API and the search API and the latter only supports Azure AD authorisation as a preview feature?

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

ID: 2293a04c-2b13-4087-0b20-eebf2278f522
Version Independent ID: 9fc429b6-1f91-74e0-3f35-3b0e54ecd213
Content: Use Azure RBAC roles - Azure Cognitive Search
Content Source: articles/search/search-security-rbac.md
Service: cognitive-search
GitHub Login: @HeidiSteen
Microsoft Alias: heidist

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

@pnarsi Yes, that's right.  Search Contributor is GA for "control plane" tasks that go through ARM, like service creation and configuration.  Search Contributor is Preview for "data plane" tasks that target content -- create/delete index, indexers, data sources, etc.  It's a hybrid role.  Using Azure roles for "data plane" is currently in preview, no matter what role definition is used in the assignment.

@pnarsi  I fixed the docs and the updates will be live later today, around 3:00 p.m. PT.   The fix puts GA and preview on separate lines and explains the difference.  Thanks for filing this issue. Combining preview and GA was confusing.  I'm going to close this issue because I think it's resolved, but if you have more feedback on the update, please feel free to reopen it.
#please-close


Updated request a speaker w/ static content

Updated request a speaker template w/ static content instead of relying on the CMS content
Additions

None

Removals

None

Changes

Replaced placeholder content with true content

Testing

navigate to /events/request-a-speaker/

Review

@KimberlyMunoz
@anselmbradford
@sebworks

Preview

Preview this PR without the whitespace changes
Notes

RaS is not related to an office, using static content until further notice
All content copied from RaS sub-page and FAQ on WordPress

:+1: The spacing looks a little weird in the first half of the page, but that's just due to the length of the content.

Minor: but it looks like there should be a little bit of whitespace between "Don't see your question here..." and the expandable faqs.
[ghe]/flapjack/event/blob/gh-pages/design/Request_a_speaker_page/Request_a_speaker_JS_CLEARED-01.png

@KimberlyMunoz I'm going to push off the spacing thing to @sebworks FAQ macro. Once he's finished with that, I'll submit another PR updating the Event RaS FAQ.

:+1:

:+1:


Ansible Lint improvements

Ignore Jinja spacing rule that caused noise in output
Update git checkouts to improve role speed and use sparse checkouts
Ignore the Prometheus Postgres Exporter which is a submodule and cannot be
linted by our rules

Closes #525

/merge


Height limit of 256 chops off blocks when anchored above

When anchoring above 256 blocks, all blocks 256 and up and chopped off the ship and disappear.

256 is the build limit for Minecraft itself, so there can't be any blocks above that height. Maybe needs a disassemble height limit so that you can't disassemble a ship if any part of it is above 256.

Pretty much yeah. This is a MovingWorld issue by the way, so I'm closing
this issue and I ask that you open a new issue in the MovingWorld repo.
On Sun, Jun 14, 2015 at 9:31 PM, Peter McMurray<EMAIL_ADDRESS>wrote:

256 is the build limit for Minecraft itself, so there can't be any blocks
above that height. Maybe needs a disassemble height limit so that you can't
disassemble a ship if any part of it is above 256.
—
Reply to this email directly or view it on GitHub
https://github.com/darkevilmac/archimedes-ships-plus/issues/27#issuecomment-111919228
.

Well no issue was opened, it's patched out in the 1.8 build anyway though.

Yeah sorry about that, I'm actually in a middle of finals week so I haven't had time for much of anything
Sent from my iPhone
On Jun 18, 2015, at 3:36 AM, Benjamin K<EMAIL_ADDRESS>wrote:

Well no issue was opened, it's patched out in the 1.8 build anyway though.
—
Reply to this email directly or view it on GitHub.


Levelized Price of Resource Metric in HERON

Pull Request Description
What issue does this change request address?
#300
What are the significant changes in functionality due to this change request?
HERON now allows users to select Levelized Cost as a metric calculated in the inner optimization step. Can be selected as an optimization metric in opt mode or an economic metric in sweep mode.

Levelized Cost is calculated via TEAL's NPV_search metric

this solves for a multiplier on a cashflow price which results in NPV equal to some target
if the target NPV is 0 (the default), the resulting multiplier is the break-even or levelized cost

Users can specify a nonzero target if desired, but would result in something different from "levelized cost"

Users can set the base price of the cashflow to 1 (default) or have an additional ValuedParam as the price, like an ARMA or fixed value

in this case, the result is strictly a multiplier on the cashflow price and is NOT intended to be a levelized cost

How to use it? Depends whether a user is running an optimization or a sweep.

If optimizing (note that the target parameter is optional):
<Case>
   <mode>opt</mode>
   ...
   <optimization_settings>
      <opt_metric target='0'>LC</opt_metric>
      <stats_metric>expectedValue</stats_metric>
   </optimization_settings>
   ...
</Case>

If running a sweep (note that the target parameter is optional):
<Case>
   <mode>sweep</mode>
   ...
   <economics>
      <EconMetrics>
          <LC target='0'></LC>
      </EconMetrics >
   </economics>
   ...
</Case>

User must also specify which cashflow contains the levelized cost

applied as a node within the <reference_price> node, replacing a ValuedParam entry<CashFlow>
   ...
   <reference_price>
      <levelized_cost/>
   </reference_price>
   ...
</CashFlow>

currently NOT available (but coming soon) for:

One-Time cash flows (e.g., CAPEX)
Cashflows with variable ValuedParam as Drivers (e.g., a capacity variable that is optimized in the outer RAVEN step)

NOTE: mult_target parameter is now deprecated

Levelized Cost requires a different inner objective function that is automatically applied. It is nonlinear, so Levelized Cost only works currently with the IPOPT solver

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[x] 1. Review all computer code.
[x] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[x] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[x] 4. Automated Tests should pass.
[x] 5. If significant functionality is added, there must be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large tes.
[x] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[x] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[x] 8. If an analytic test is changed/added, the the analytic documentation must be updated/added.
[x] 9. If any test used as a basis for documentation examples have been changed, the associated documentation must be reviewed and assured the text matches the example.

Looks like we don't get ipopt on the Windows computer; however, it looks like we're requesting it as part of the dependencies, so that's odd.

This looks like it makes progress towards  #300 without completing it, so I did not mark it for closure. Let me know if I'm wrong about that!
Otherwise LGTM, tests pass, merging. If you want to announce this feature (and its limitations) on the discussion board, that would be great!


generate and support arm binaries

is it possible for you to build and support binaries for arm6/7 and arm64 as well please

You can easily compile arm binaries using go. Since I have an arm64 available I compiled a latest siasync version, you can find it on my fork here https://github.com/vcomtec/siasync/tree/1.1.0-vcomtec.

You can easily compile arm binaries using go. Since I have an arm64 available I compiled a latest siasync version, you can find it on my fork here https://github.com/vcomtec/siasync/tree/1.1.0-vcomtec.

Thanks


Deleting a message can race with various updates

Anything which updates the message is potentially subject to a race with deleting it.  We have observed it with literal message updates as well as mobile message notifications.
In order to prevent this, the message needs to be originally selected with SELECT FOR UPDATE to take a write lock.

I wonder if we could do this within the access_message_by methods?

I wonder if we could do this within the access_message_by methods?

That has been my concept for fixing this - I'm not super happy with it because it'll require adding a bool flag (or even ttwo? one for "select message for update" and one for "select usermessage for update", since both can be useful) to the function, but it seems the most reasonable approach

Yeah; probably we need two flags to encode those ideas.  There's only about 10 call points to access_message(, so this might not be that much work to migrate.  Maybe lock_message and lock_usermessage would be reasonable names for the flags?

Can I take up this issue if it is still available?

@MSurfer20 Someone else started working on this as far as I know, let me verify.

#18559 added the ability for access_message to lock, but missed some callsites to it -- for instance, handle_push_notifications.  We should do another pass of looking at access_message callsites.

Fundamentally, check_update_message, which is the main work from the update message backend, calls access_message without taking a lock:
https://github.com/zulip/zulip/blob/38a4751b7bb8535416569bb5f41b246f83ac64c2/zerver/actions/message_edit.py#L904-L919
This means that deletes can still race with updates, causing the update to 500 with Save with update_fields did not affect any rows.
The aforementioned scan of access_message callsites for locations which need to add locking is still true.

#26266 fixed the check_update_message codepath.

I did a skim of all existing access_message() calls, and the only ones that don't take a lock are indeed doing read operations; so I'm going to close this as resolved.


Hangs when run from lambda

I run this code from my desktop and works great, but the same code, even with Admin role, hangs.
Below are the lines where I use FluentMetric
    m = FluentMetric().with_namespace(CWNAMESPACE).with_stream_id(streamid)
    m.log(MetricName='HealthyServerCount-Port'+str(port), Value=numpassed, Unit='Count')```

Never mind. This is a function of lambda in a VPC. It needs special infrastructure to egress to AWS services, not just IGW that is already there and enjoyed by Ec2.  My bad.