No explicit documentation for .docker/config.json usage for registry auth

If this is a duplicate: i'am sorry, but i couldn't find any help in the docs...
I have the following setup:
Jenkins runs a gradle-task inside docker (so i use docker.sock). From the gradle-job, i start my Integration test using testcontainers.
My problem starts, when i want to use private repositories from jffrog (public images works fine). I tried to set the environment variables to the docker command which runs the gradle command via:
-e \"registryUsername=foo\" -e \"registryPassword=bar\" -e \"registryUrl=https://companyname.jfrog.io\"
Inside the container which runs gradle, i can see that the values are set correctily. But none of the values are picked up from testcontainers, which results in an error, complining about not be able to load the images...
What am i missing ? I found this change log but no docs how to configure it...
Any help is welcome, thank you!

Ok, i found a solution:
Create this file ~/.docker/config.json via the following command in the jenkins, before the tests are started via gradle:
sh 'echo "{\\"auths\\":{\\"https://company-docker.jfrog.io\\":{\\"auth\\": \\"\$(echo -n \$JFROG_USR:\$JFROG_PWD | base64)\\"}}}" > ~/.docker/config.json'
I really think, that this should be documented in testcontainers, even if it's part of docker-java.

Hi @athlordJojo, thanks for the remarks and providing the solution with an example.
Do you think we should duplicate this into our docs, or just add references to docker-java?

Hey @kiview !
I really think, that this should be part of testcontaines documentation.
For example: I didn't even knew, that textcontainers uses docker-java, so i didn't looked at there documentation.

Yes, this definitely makes sense.
But we could just link to https://github.com/docker-java/docker-java#properties-docker-javaproperties for example.
By the way, the $HOME/.docker/config.json is a general Docker approach, not docker-java specific I just realized: https://docs.docker.com/engine/reference/commandline/login/

ok, so will add the documentation ? becasue then we could close this issue.

I'd agree that this needs to be in the documentation. If anyone would like to raise a PR to add, it would be appreciated!


[keras compatibility] tf.keras.regularizers.l2() cannot be used in tf.get_variable()

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): custom code
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 1.10.1
Python version: 3.6.4
OS Platform and Distribution: N/A
Bazel version: N/A
CUDA/cuDNN version: N/A
GPU model and memory: N/A
Mobile device: N/A
Exact command to reproduce: Source code below.

Describe the problem
tf.keras.regularizers.l2() cannot be used in tf.get_variable(). Currently I have to use tf.contrib.layers.l2_regularizer() in tf.get_variable().
Note that , tf.keras.regularizers.l2() seems to work well in tf.layers.Dense().
In general, I expect that all keras functions like regularizers, activations, initializers could be used for tensorflow.
Source code / logs
x = tf.get_variable(shape = [dim_in, dim_out],
    initializer=tf.keras.initializers.glorot_uniform(),
    regularizer=tf.keras.regularizers.l2(lambda))

Traceback:

File "/Users/mac/anaconda3/lib/python3.6/site-packages/tensorflow/python/ops/variable_scope.py", line 1467, in get_variable
aggregation=aggregation)
File "/Users/mac/anaconda3/lib/python3.6/site-packages/tensorflow/python/ops/variable_scope.py", line 1217, in get_variable
aggregation=aggregation)
File "/Users/mac/anaconda3/lib/python3.6/site-packages/tensorflow/python/ops/variable_scope.py", line 527, in get_variable
aggregation=aggregation)
File "/Users/mac/anaconda3/lib/python3.6/site-packages/tensorflow/python/ops/variable_scope.py", line 481, in _true_getter
aggregation=aggregation)
File "/Users/mac/anaconda3/lib/python3.6/site-packages/tensorflow/python/ops/variable_scope.py", line 930, in _get_single_variable
loss_name = loss.name
AttributeError: 'float' object has no attribute 'name'

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
OS Platform and Distribution
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce
Mobile device

@tranhungnghiep It looks like the issue only happens when tf.keras.regularizers.l2(l=0.) is used. In that case regularization = 0. is returned directly. Otherwise it should be fine:
https://github.com/tensorflow/tensorflow/blob/609b2ce3fe8ebecf4031670b8c2186468369b0ba/tensorflow/python/keras/regularizers.py#L56-L62
Created a PR #22478 for the fix.

@yongtang Yes that should fix it. Thanks.

@yongtang Thanks for the PR!


Decoder sequence question

I'm planning to introduce a additional piece of functionality to the decoder of a protocol.
Given the GL100 Protocol would this be the correct way to add extra functionality?
                pipeline.addLast("frameDecoder", new CharacterDelimiterFrameDecoder(1024, '\0')); //already there
                pipeline.addLast("stringEncoder", new StringEncoder());//already there
                pipeline.addLast("stringDecoder", new StringDecoder());//already there
                pipeline.addLast("objectDecoder", new Gl100ProtocolDecoder(Gl100Protocol.this));//already there
                pipeline.addLast("objectDecoder", new MyCustomDecoder(Gl100Protocol.this));//my extra decoder

This MyCustomDecoder would then have a decode(object obj) method with the decoded object from the previous decoder right? Like the decorator pattern?
Is my intuition correct about this?

I must not be so lazy.. I just tried it. This is possible as long as you use unique names for the decoders. So myEncoder would work. :)


Tracking issue for Regional Keyboard Support

COMMUNITY-DRIVEN WORK TRACKING ISSUE
Summary of feature
Today, Silk.NET has no official support for receiving KeyDown (et al) events in any keyboard layout other than US (even though the SDL backend accidentally mandatorily supports it, but this is a bug with that backend).
This has been requested lots of times, and even got to a point where a proposal made it to review but was ultimately rejected as we wanted to think of a new approach using scancodes.
This line of work is that new approach. With this, a set of methods are exposed to convert scancodes to Keys and back.
This is a big feature that will be entirely community-driven, so I have made a tracking issue and a branch develop/2.0/community_region_specific_input which all intermediate work will go into and, when the feature is complete, will be merged in (unsquashed).
Please read these Discord messages in the Silk.NET Discord server:

https://discord.com/channels/521092042781229087/607634593201520651/926654719001247754
https://discord.com/channels/521092042781229087/607634593201520651/926656904594350130
https://discord.com/channels/521092042781229087/607634593201520651/926657474193412117
(and possibly surrounding messages on the above)

Development

[ ] [Not Started] Formalize a proposal using the proposal process - (unassigned)

NB: this should be the only PR against the main branch

[ ] [Not Started] Review proposal in a working group meeting (work can continue on the other items while this happens) - @dotnet/silk-dotnet
[ ] [Not Started] Add new IKeyboard APIs and expand Key as necessary - (unassigned)
[ ] [Not Started] Implement new IKeyboard APIs in GLFW - (unassigned)
[ ] [Not Started] Implement new IKeyboard APIs in SDL - (unassigned)
[ ] [Not Started] Merge develop/2.0/community_region_specific_input into main

Milestones

[ ] Release with Regional Keyboard Support

Pinging @ThomasMiz as area owner. Note that this is fully community-driven work, and will not be done by the Silk.NET team unless sponsored to do so.
@Beyley Please let me know if you want to officially put your name one or more of these items. You can do multiple items in one PR, or one PR for each item. Your choice! Thanks for help shaping this in the Discord.

Pinging @ThomasMiz as area owner. Note that this is fully community-driven work, and will not be done by the Silk.NET team unless sponsored to do so.
@Beyley Please let me know if you want to officially put your name one or more of these items. You can do multiple items in one PR, or one PR for each item. Your choice! Thanks for help shaping this in the Discord.

Note: 2.15 finally fixes the backend differences which technically made it so you don't need this proposal. This has always been a bug ever since 2.0 was released, but it is now fixed.


Spring cloud function server up time is more in aws lambda

Issue: Spring cloud function server up is delay.
Description: Implemented spring cloud function and deployed into AWS lambda.. Some times more request are coming, new execution service will start in aws lambda.. That's taking more than 9 seconds.. So i'm getting gateway timeout exception in apigateway and some performance issues.
Versions:  Spring boot starter parent is 2.5.0
spring-cloud-function-dependencies: 3.1.3
spring-loud-dependencies: 2020.0.2
Note:  Used @Lazy but getting same time to start up.
Questions:

is there any way to make start application in milliseconds? what configuration i need to change?
Can make instance running quickly like Nodejs?. Do i need to upgrade or any extra configuration I need to configure?

Please suggest me on this. Appreciate your help on quick response.
Thanks a lot.

Hi @jaganm2018,
@olegz has a sample from SpringOne last year which uses Spring Cloud Functions with Spring Native.
https://github.com/olegz/springone2021
This should give you cold start times similar to Nodejs.

Thanks Msailes. I'm going to refer this project..
https://github.com/olegz/springone2021/tree/main/aws-native/function-aws-native-springone
Please let me know if i'm wrong.
Once again thanks for giving reply.

Please try this more formal sample - https://github.com/spring-projects-experimental/spring-native/tree/main/samples/cloud-function-aws.
The sample in my repo hasn't been maintained since SpringOne

After reading it some more indeed I would suggest native approach if you want fast startups

Closing it due to lack fo followup from the original reporter


DeepLink flow broken with WalletConncect Kotlin Dapp

The deep link flow is broken. When requesting the connection from Kotlin Dapp to Alpha Wallet the session proposal dialog is not shown (see the 1st video)

After upgrade SDK, it's working now.


Add estraverse-fb and remove failing react/prefer-stateless-function rule

For this boilerplate to work out of the box estraverse-fb needs to be added to the package.json.
 Or the components re-written to be stateless.
I choose the quicker option :)

Thanks for trying out the code in the repo @ro-savage!
I'm puzzled by the issues you've seen – the Travis build works fine, and it works locally as well.
A couple of questions:

What OS are you using?
And what versions of node and npm?
why do you think estraverse-fb is missing? (haven't had any warnings or errors either locally, or on Travis AFAIK)

Our local setup is OSX 10.11.3, node 5.3.0 (the Travis build specifies 5.4.0) and npm 3.3.12. It might be the case that we need to add a Prerequisites section to the README.
Matt

Hi,
Did a little bit more research.
The estraverse-fb missing issue is related to babel-eslint requiring a package that no longer exists in eslint. The way to resolve is either to install estraverse-fb package directly, or lockdown eslint to 2.2.0 (or add the package to babel-eslint?)
 I have locked down eslint to 2.2.0 for my other projects for now.
Just in case, for the other questions
OSX <IP_ADDRESS>
NPM 3.6.0
Node 5.7.0

@ro-savage: thanks for digging, and finding the cause. I removed my node_modules directory, re-ran npm install, and am now seeing the error you described (the version of eslint installed is 2.3.0).
I tend to agree with you and @jhabdas: the better option is to not upgrade until the problem is solved.
Thanks for the offer to make changes; I think it will be quicker for me to make the change to package.json to lock eslint at version 2.2.0 instead of ^2.2.0.

What I've also done is to lock eslint-config-airbnb to version 6.0.2, which fixes the eslint errors. We may go to more recent versions in the future.
The merged PR is #82.
Thanks for your help!

Awesome. Thanks.
This boilerplate you put together is really useful for getting a better understand of how to put together a more real world application than other tutorials/boilerplates out there. Keep up the good work!

Great to hear. I've learned a lot myself!


Modify facet-search sort at query time

Related product team resources: PRD (internal only)
Related spec: WIP
Motivation

Required by users
Usage

https://meilisearch.notion.site/API-usage-sort-facet-search-at-query-time-2c920089bbcc43c5a94185b6cf0f6a6f?pvs=25
TODO

[ ] Merge changes into main
[ ] Update the spec

Reminders when modifying the Setting API

[ ] Ensure the new setting route is at least tested by the test_setting_routes macro
[ ] Ensure Analytics are fully implemented

[ ] /settings/my-new-setting configurated in the make_setting_routes macro
[ ] global /settings route configurated in the update_all function

[ ] Ensure the dump serializing is consistent with the /settings route serializing, e.g., enums case can be different (camelCase in route and PascalCase in the dump)

Special cases when adding a setting for an experimental feature

[ ] ⚠ API stability: The setting does not appear on the main settings route when the feature has never been enabled (e.g. mark it Unset when returned from the index in this situation. See an example)
[ ] The setting cannot be set when the feature is disabled, either by the main settings route or the subroute (see validate_settings function)
[ ] If possible, the setting is reset when the feature is disabled (hard if it requires reindexing)

Impacted teams

@meilisearch/docs-team

For security reasons, will not be done in v1.8.0, we need more exploration on this


Ionic 3.X Weird Extra Top Padding For Ion-Content

Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[X] 3.x
[ ] 4.x
I'm submitting a ...  (check one with "x")
[X] bug report
[ ] feature request
Current behavior:
Since upgrading to ionic-angular 3.7.1, I've been experiencing a weird top padding issues with ion-content.
Initially the view loads fine, however after interacting with the view like rebinding data or refreshing using the ion-refresher, the top of the ion-content gets shifted down with extra top padding, which appears the same height as the status bar.
Oddly enough, text that has user-select: text enabled in the CSS to allow copy & paste, is being highlighted where the input is suppose to be rather than where it is.
Expected behavior:
The content padding should be consistent, and not changed after interacting with the view.
Steps to reproduce:
Before

After

Hello! Thank you for opening an issue with us!
Please check that you are using WKWebView and not UIWebView.
If you have this plugin installed, you are using WKWebView (which is what you want)
https://github.com/ionic-team/cordova-plugin-ionic-webview
Using WKWebView has a few caveats that may need to be dealt with in your app. However it is the proper way forward for many reasons.
http://ionicframework.com/docs/wkwebview/
If you are using WKWebView and still are having the issue, would you be able to provide a sample application via GitHub that demonstrates the issue you are having?
Thanks for using Ionic!

Thanks @kensodemann for the reply!
I tried to upgrade to WKWebView, unfortunately it didn't seem to fix the issue. I removed /platforms and /plugins, re-added iOS platform but still not luck.
One app I'm developing, I can upgrade to WKWebView, so hopefully this can fix my issue. However another app I'm building still needs to use UIWebView because I ran into some CORS and local files issues with WKWebView.
Is there any way to fix this issue so it doesn't happen in either WKWebView or UIWebView?

For the CORs issues, have you tried this? http://ionicframework.com/docs/wkwebview/#cors

Is there any way to fix this issue so it doesn't happen in either WKWebView or UIWebView?

According to my understanding of issue #12958 (which I think is the same as what you are seeing), no. It is due to a bug in UIWebView (which is an Apple thing that we can do nothing about, and it is the older of the web views and Apple has been suggesting ppl use WKWebView since iOS 8 or 9, so I don't really see it getting much love from Apple, honestly).
However, if updating to WKWebView did not fix the issue for you, there may be more going on. Can you duplicate this in a sample application? Basically, start with whatever starter is closest to your app's architecture and add the minimal stuff required to get it to happen and then upload to GitHub?  Unless, of course, your app itself is already in a public repo.
One other thing to check would be that everything here is handled: https://blog.ionic.io/ios-11-checklist/

Thanks @kensodemann, I'll review https://blog.ionic.io/ios-11-checklist/ again to see if I missed something. It does seem like a similar issue to https://github.com/ionic-team/ionic/issues/12958.
Another weird thing I noticed, when the extra spacing appears, it's like the content behind has moved down but the touch area has not. For example, to select a ion-item row, you actually need to click above the ion-list, the same above of spacing that it's been shifted down.

This should be address via
npm i<EMAIL_ADDRESS>

This will be the nightly version

When you'll release this version? I mean 3.7.2 or like that? @mhartington

Weird @mhartington, even with the nightly build it's still happening for me.
The view loads fine but any kind of rebinding to data, the extra space appears. However using the ion-refresher control, the extra space disappears again.

Here's the details of the package.json and config.xml in case it helps.
{
  "name": "our-app",
  "version": "0.12.0",
  "author": "Us",
  "homepage": "http://www.website.com",
  "private": true,
  "scripts": {
    "build": "ionic-app-scripts build",
    "clean": "ionic-app-scripts clean",
    "lint": "ionic-app-scripts lint",
    "ionic:build": "npm run build",
    "ionic:serve": "npm run serve",
    "ionic:lint": "npm run lint"
  },
  "dependencies": {
    "@angular/animations": "4.4.4",
    "@angular/common": "4.4.4",
    "@angular/compiler": "4.4.4",
    "@angular/compiler-cli": "4.4.4",
    "@angular/core": "4.4.4",
    "@angular/forms": "4.4.4",
    "@angular/http": "4.4.4",
    "@angular/platform-browser": "4.4.4",
    "@angular/platform-browser-dynamic": "4.4.4",
    "@ionic-native/app-availability": "^4.3.1",
    "@ionic-native/app-version": "^4.3.1",
    "@ionic-native/call-number": "^4.3.1",
    "@ionic-native/camera": "^4.3.1",
    "@ionic-native/contacts": "^4.3.1",
    "@ionic-native/core": "^4.3.1",
    "@ionic-native/deeplinks": "^4.3.1",
    "@ionic-native/device": "^4.3.1",
    "@ionic-native/diagnostic": "^4.3.1",
    "@ionic-native/file": "^4.3.1",
    "@ionic-native/geolocation": "^4.3.1",
    "@ionic-native/in-app-browser": "^4.3.1",
    "@ionic-native/is-debug": "^4.3.1",
    "@ionic-native/keyboard": "^4.3.1",
    "@ionic-native/native-storage": "^4.3.1",
    "@ionic-native/network": "^4.3.1",
    "@ionic-native/sim": "^4.3.1",
    "@ionic-native/social-sharing": "^4.3.1",
    "@ionic-native/splash-screen": "^4.3.1",
    "@ionic-native/sqlite": "^4.3.1",
    "@ionic-native/status-bar": "^4.3.1",
    "@ionic-native/themeable-browser": "^4.3.1",
    "@ionic-native/transfer": "^3.14.0",
    "@ionic/storage": "2.0.1",
    "call-number": "git+https://github.com/Rohfosho/CordovaCallNumberPlugin.git",
    "cordova-android": "^6.2.3",
    "cordova-ios": "~4.5.2",
    "cordova-custom-config": "^4.0.2",
    "cordova-plugin-app-version": "^0.1.9",
    "cordova-plugin-appavailability": "^0.4.2",
    "cordova-plugin-camera": "^2.4.1",
    "cordova-plugin-compat": "^1.2.0",
    "cordova-plugin-contacts": "^2.3.1",
    "cordova-plugin-device": "^1.1.6",
    "cordova-plugin-file": "^4.3.3",
    "cordova-plugin-file-transfer": "^1.6.3",
    "cordova-plugin-geolocation": "^2.4.3",
    "cordova-plugin-ionic-keyboard": "^2.0.4",
    "cordova-plugin-ionic-webview": "^1.1.16",
    "cordova-plugin-is-debug": "^1.0.0",
    "cordova-plugin-nativestorage": "^2.2.2",
    "cordova-plugin-network-information": "^1.3.3",
    "cordova-plugin-sim": "^1.3.3",
    "cordova-plugin-splashscreen": "^4.0.3",
    "cordova-plugin-statusbar": "^2.2.3",
    "cordova-plugin-themeablebrowser": "^0.2.17",
    "cordova-plugin-whitelist": "^1.3.2",
    "cordova-plugin-x-socialsharing": "^5.2.1",
    "cordova-sqlite-storage": "^2.0.4",
    "cordova.plugins.diagnostic": "^3.7.1",
    "es6-promise-plugin": "^4.1.0",
    "ionic-angular": "nightly",
    "ionic-plugin-keyboard": "^2.2.1",
    "ionicons": "3.0.0",
    "moment": "^2.18.1",
    "ng-lazyload-image": "^3.3.1",
    "rxjs": "5.4.3",
    "sw-toolbox": "3.6.0",
    "zone.js": "0.8.18"
  },
  "devDependencies": {
    "@ionic/app-scripts": "nightly",
    "typescript": "~2.3.2"
  },
  "cordovaPlugins": [
    "cordova-plugin-device",
    "cordova-plugin-whitelist",
    "cordova-plugin-statusbar",
    "cordova-plugin-splashscreen"
  ],
  "cordovaPlatforms": [
    "ios",
    {
      "platform": "ios",
      "version": "",
      "locator": "ios"
    }
  ],
  "description": "Rollcall App",
  "cordova": {
    "plugins": {
      "cordova-plugin-app-version": {},
      "cordova-plugin-appavailability": {},
      "cordova-plugin-camera": {
        "CAMERA_USAGE_DESCRIPTION": "This app would like to access your camera.",
        "PHOTOLIBRARY_USAGE_DESCRIPTION": "This app would like to access your photo library."
      },
      "cordova-plugin-contacts": {
        "CONTACTS_USAGE_DESCRIPTION": "This app would like to access your contacts."
      },
      "cordova-plugin-device": {},
      "cordova-plugin-file": {},
      "cordova-plugin-file-transfer": {},
      "cordova-plugin-geolocation": {
        "GEOLOCATION_USAGE_DESCRIPTION": "This app would like to access your location."
       />
        <variable name="PHOTOLIBRARY_USAGE_DESCRIPTION" value="This app would like to access your photo library." />
    </plugin>
    <plugin name="cordova-plugin-contacts" spec="^2.3.1">
        <variable name="CONTACTS_USAGE_DESCRIPTION" value="This app would like to access your contacts." />
    </plugin>
    <plugin name="cordova-plugin-device" spec="^1.1.6" />
    <plugin name="cordova-plugin-file" spec="^4.3.3" />
    <plugin name="cordova-plugin-file-transfer" spec="^1.6.3" />
    <plugin name="cordova-plugin-geolocation" spec="^2.4.3">
        <variable name="GEOLOCATION_USAGE_DESCRIPTION" value="This app would like to access your location." />
    </plugin>
    <plugin name="cordova-plugin-is-debug" spec="^1.0.0" />
    <plugin name="cordova-plugin-nativestorage" spec="^2.2.2" />
    <plugin name="cordova-plugin-network-information" spec="^1.3.3" />
    <plugin name="cordova-plugin-sim" spec="^1.3.3" />
    <plugin name="cordova-plugin-splashscreen" spec="^4.0.3" />
    <plugin name="cordova-plugin-statusbar" spec="^2.2.3" />
    <plugin name="cordova-plugin-whitelist" spec="^1.3.2" />
    <plugin name="cordova-plugin-x-socialsharing" spec="^5.2.1" />
    <plugin name="cordova-sqlite-storage" spec="^2.0.4" />
    <plugin name="cordova.plugins.diagnostic" spec="^3.7.1" />
    <plugin name="cordova-plugin-ionic-keyboard" spec="^2.0.4" />
    <plugin name="cordova-custom-config" spec="^4.0.2" />
    <plugin name="cordova-plugin-themeablebrowser" spec="^0.2.17" />
    <plugin name="cordova-plugin-ionic-webview" spec="^1.1.16" />
    <plugin name="call-number" spec="git+https://github.com/Rohfosho/CordovaCallNumberPlugin.git" />
</widget>

And here's the details from ionic info.
cli packages: (/usr/local/lib/node_modules)

    @ionic/cli-utils  : 1.15.0
    ionic (Ionic CLI) : 3.15.0

global packages:

    cordova (Cordova CLI) : 7.1.0

local packages:

    @ionic/app-scripts : 3.0.0-201710161925
    Cordova Platforms  : ios 4.5.2
    Ionic Framework    : ionic-angular 3.7.1-201710241735

System:

    Android SDK Tools : 26.0.2
    ios-deploy        : 1.9.2
    ios-sim           : 6.1.2
    Node              : v7.2.1
    npm               : 3.10.10
    OS                : macOS Sierra
    Xcode             : Xcode 9.0 Build version 9A235

Environment Variables:

    ANDROID_HOME : /Library/Android

Misc:

    backend : legacy

I can confirm same issue and this is fixed in WKWebView, however same display issue persists CDVUIWebViewEngine.
Easy to produce just create blank app and put in home.html
<ion-content >     <ion-searchbar showCancelButton=true [debounce]="600"  ></ion-searchbar>                    </ion-content>
And as soon as you tap in the search bar the content moves down about 20px and sometimes when you go back to previous pages they actually bump up under header and partially hidden.
We cannot move to WKWebView until it works properly with cookies (mostly works on device, but always fails on simulator) so we would be grateful to have CDVUIWebViewEngine working again as it used too.

Hmm, I am unable to replicate at all with ionic-angular 3.8.0 (just released)

I tried with 3.8.0, problem still exists.
But note it does not exist on simulator only on real device. Tested with two iPhone 7/7+ both running IOS 11.0.3.  And only with CDVUIWebViewEngine.

Also getting this on 3.8.0.
Problem seems to be a style applied to .scroll-content:
padding-top: 20px;
padding-top: constant(safe-area-inset-top);
padding-top: env(safe-area-inset-top);
If you set padding-top: 0 !important it goes away

Thats not really the proper solution.
So it's only visible if there is no ion-header? That css is only applied when ion-content is the first element. I will test that out.

Hmm, Im not able to replicate it still. This is looking fine with the latest release.

Yes works fine in EMULATOR. Does not work on real device. And I am using default ion-header per start blank project.

Alright, I check it out
Mike Hartington

On Oct 27, 2017, at 10:05 AM, sjdrew<EMAIL_ADDRESS>wrote:
Yes works fine in EMULATOR. Does not work on real device. And I am using default ion-header per start blank project.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

In case it helps, I found that adding no-bounce to ion-content prevents the issue.
Steve.

Actually forget that, after cold starting the app with the "no-bounce" it still has the problem.

@mhartington did you try testing on a simulator other than iPhone X?
I've been having my issues on iOS 11 on iPhone SE, iPhone7, etc emulators.

Yes, I've been using iphone 6/7, but sim and real devices
Sim

Device

If someone can replicate this, please provide a minimal project. Otherwise, Im going to have to close this as I cannot reproduce at all.

Here is how I can reproduce it. However for some reason generating a new blank project today it only happens about 1 in 3 tries as opposed to an upgraded one.

create blank starter project

within ion-content tag add a search bar (or any input) <ion-searchbar showCancelButton=true [debounce]="600"  >

add to config.xml

ionic cordova run ios --prod   (seemed to happen more with --prod)

See screen shot and notice the cursor flashes above the input field. This is the behaviour I now see in a few apps after upgrading ionic and running ios 11.0.3.  Simulator works fine.

Ah, alright @sjdrew so the main issue here is UIWebview. Using WKWebview is the solution to this.

I wish it were that easy, have bunch of apps to convert to WKWebView, but still have issues with WKWebView that will take lots of work to workaround and previous cookie handling and app state already in UIWebView so it is not a practical solution.
Is there any workaround that you can think of, I see many people running into this issue as know of many in same situation as us that WKWebview will not work for them.
I did not read anything that said UIWebview was no longer supported as documentation shows how to use either view.
Thanks.

I wish it were that easy, have bunch of apps to convert to WKWebView,

WKWebview is mostly a drop in replacement for UIWebview, and anything that you need to know is covered in this doc.  http://ionicframework.com/docs/wkwebview/

Is there any workaround that you can think of

Unfortunately, we're at the mercy of UIWebview/Apple here. Because it is a Core piece of iOS, we're not able to make any modification to it or change it behaviour. The way to "fix" this is to use WKWebview.

I did not read anything that said UIWebview was no longer supported

You can use UIWebview, but there are several major issues that will come from it. Especially with iOS 11 and the new safe-area margins, which get reported incorrectly (again at the native level so we cannot fix/hack to work).
I will note, I tried this again with UIWebview and a prod build of an app but could not reproduce with the steps provided. It seems very inconsistent (pretty much UIWebview's main issue)

OK Thanks, I understand. Probably be prepared for this topic to come up a lot for anyone upgrading and using cookies in their app.
FYI this issue https://github.com/ionic-team/cordova-plugin-ionic-webview/issues/22  …  Is one of the main reasons stopping people from running WKWebView (from my experience), if someone could fix the cookie issue that would mean WKWebview would be a working drop in replacement.
If I take any of my apps that were using cookies and convert to WKWebview they fail in the simulator 100%.
Steve.
From: Mike Hartington<EMAIL_ADDRESS>Reply-To: ionic-team/ionic<EMAIL_ADDRESS>Date: Monday, October 30, 2017 at 10:09 AM
To: ionic-team/ionic<EMAIL_ADDRESS>Cc: Steve Drew<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [ionic-team/ionic] Ionic 3.X Weird Extra Top Padding For Ion-Content (#13229)
I wish it were that easy, have bunch of apps to convert to WKWebView,
WKWebview is mostly a drop in replacement for UIWebview, and anything that you need to know is covered in this doc. http://ionicframework.com/docs/wkwebview/
Is there any workaround that you can think of
Unfortunately, we're at the mercy of UIWebview/Apple here. Because it is a Core piece of iOS, we're not able to make any modification to it or change it behaviour. The way to "fix" this is to use WKWebview.
I did not read anything that said UIWebview was no longer supported
You can use UIWebview, but there are several major issues that will come from it. Especially with iOS 11 and the new safe-area margins, which get reported incorrectly (again at the native level so we cannot fix/hack to work).
I will note, I tried this again with UIWebview and a prod build of an app but could not reproduce with the steps provided. It seems very inconsistent (pretty much UIWebview's main issue)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

OK good news. Just upgrade to iOS 11.1 and problem is gone, so far tried many times to reproduce and no display issues. (With UiWebView)
Steve.

@sjdrew @mhartington just some followup.
On one project, upgrading and switching to WKWebview fixed the padding/spacing issue I was experiencing.
However on another project, which sets statusBar.overlaysWebView to true or false in different pages based on whether there is a ion-navbar, the issue still appears to be happening.
So perhaps there is still a lingering issue, with both UiWebView and WKWebview on iOS 11 with overlaysWebView?

Unfortunately, the complications that cordova-plugin-ionic-webview (or WKWebView) creates with cookies, XHR requests, etc... make it a deal breaker. Does anyone know how to fix the space at the top of the app without using cordova-plugin-ionic-webview?

my two cents to this issue:
I have WKWebView plugin installed. I was still seeing this
padding-top: calc(16px + 20px) in the web inspector.
To fix this, all I did was move padding attribute from <ion-content> to scroll-content as
.scroll-content {
padding: 16px;
}

and I did observe the issue again in any ios versions or model (including iphone X).


added vault user instead of root

continuing my crusade of getting rid of root user in our containers, I believe this should work as long as people don't mount their secrets in a weird place that isn't /etc/secrets

@Joseph-Irving looks sensible. Could you squash your commits?

@vaijab oops, yeah of course, done.

@Joseph-Irving ... Your'll have to double check this doesn't create permissions issues between containers ..

@gambol99 @Joseph-Irving: If the UID is 1000 then please merge this.  Bump the version to cover the potential BC break and then roll it out where you can.


App build failing after adding gradle dependency

Upon adding implementation 'in.payu:payu-checkout-pro:1.8.3' I'm not able to run my project. I don't have jCenter()
FAILURE: Build completed with 7 failures.
1: Task failed with an exception.

What went wrong:
Execution failed for task ':app:checkCarefirstDebugAarMetadata'.

Could not resolve all files for configuration ':app:carefirstDebugRuntimeClasspath'.
Could not resolve com.paytm:pgplussdk:1.4.4.
Required by:
project :app
Could not resolve com.paytm:pgplussdk:1.4.4.
Could not parse POM https://tokbox.com/maven/com/paytm/pgplussdk/1.4.4/pgplussdk-1.4.4.pom
Already seen doctype.
Could not resolve com.paytm:pgplussdk:1.4.4.
Could not get resource 'http://tokbox.com/maven/com/paytm/pgplussdk/1.4.4/pgplussdk-1.4.4.pom'.
Could not HEAD 'http://tokbox.com/maven/com/paytm/pgplussdk/1.4.4/pgplussdk-1.4.4.pom'.
Connect to tokbox.com:80 [tokbox.com/<IP_ADDRESS>] failed: Operation timed out (Connection timed out)
Could not resolve com.github.barteksc:android-pdf-viewer:2.8.2.
Required by:
project :app
Could not resolve com.github.barteksc:android-pdf-viewer:2.8.2.
Could not get resource 'https://jitpack.io/com/github/barteksc/android-pdf-viewer/2.8.2/android-pdf-viewer-2.8.2.pom'.
Could not HEAD 'https://jitpack.io/com/github/barteksc/android-pdf-viewer/2.8.2/android-pdf-viewer-2.8.2.pom'. Received status code 401 from server: Unauthorized
Could not resolve com.github.barteksc:android-pdf-viewer:2.8.2.
Could not parse POM https://tokbox.com/maven/com/github/barteksc/android-pdf-viewer/2.8.2/android-pdf-viewer-2.8.2.pom
Already seen doctype.
Could not resolve com.github.barteksc:android-pdf-viewer:2.8.2.
Could not get resource 'http://tokbox.com/maven/com/github/barteksc/android-pdf-viewer/2.8.2/android-pdf-viewer-2.8.2.pom'.
Could not HEAD 'http://tokbox.com/maven/com/github/barteksc/android-pdf-viewer/2.8.2/android-pdf-viewer-2.8.2.pom'.
Connect to tokbox.com:80 [tokbox.com/<IP_ADDRESS>] failed: Operation timed out (Connection timed out)
Could not resolve com.android.volley:volley:1.1.1.
Required by:
project :app > com.google.android.libraries.places:places:2.4.0
Could not resolve com.android.volley:volley:1.1.1.
Could not parse POM https://tokbox.com/maven/com/android/volley/volley/1.1.1/volley-1.1.1.pom
Already seen doctype.
Could not resolve com.android.volley:volley:1.1.1.
Could not get resource 'http://tokbox.com/maven/com/android/volley/volley/1.1.1/volley-1.1.1.pom'.
Could not HEAD 'http://tokbox.com/maven/com/android/volley/volley/1.1.1/volley-1.1.1.pom'.
Connect to tokbox.com:80 [tokbox.com/<IP_ADDRESS>] failed: connect timed out

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
==============================================================================

2: Task failed with an exception.

What went wrong:
Execution failed for task ':app:mergeCarefirstDebugResources'.

Could not resolve all files for configuration ':app:carefirstDebugRuntimeClasspath'.
Could not resolve com.paytm:pgplussdk:1.4.4.
Required by:
project :app
Could not resolve com.paytm:pgplussdk:1.4.4.
Could not parse POM https://tokbox.com/maven/com/paytm/pgplussdk/1.4.4/pgplussdk-1.4.4.pom
Already seen doctype.
Could not resolve com.paytm:pgplussdk:1.4.4.
Could not get resource 'http://tokbox.com/maven/com/paytm/pgplussdk/1.4.4/pgplussdk-1.4.4.pom'.
Could not HEAD 'http://tokbox.com/maven/com/paytm/pgplussdk/1.4.4/pgplussdk-1.4.4.pom'.
Connect to tokbox.com:80 [tokbox.com/<IP_ADDRESS>] failed: Operation timed out (Connection timed out)

Hi @vikash010890, your logs do not appear to be from a flutter project as such I'm going to close this issue. If your project is indeed a flutter project, please provide the following information and I will reopen this issue.

The output of flutter doctor -v
The full output of flutter run -v (kindly put this in a .txt file)
Your pubspec.lock and pubspec.yaml
Your android/build.gradle and android/app/build.gradle files

Thank youj


multi-token mapping for runnables.rs (#10070)

implement #10070 in runnables

r? @Veykril

Looks good! I haven't figured out a way to test this yet, since "Peek Related Tests" does not work on first and second in the following code:
// ...

Does it depend on some other aspect of this ticket or another missing feature?

Does it depend on some other aspect of this ticket or another missing feature?

I don't think it's part of #10070, but I'm not really sure what exactly the problem is either

bors d+
Looks like CI will do the remaining review, feel free to r+ once that passes

Squashing should fix the CI error.

bors r+

But you didn't squash..

But you didn't squash..

sorry - how should I fix this now?

bors r-

Untested:
$ git rebase -i @~4
# edit the file to put `p` or `pick` on the last three commits (if vim starts, you'll need to press `i` to insert, then quit with `ESC` and  `:wq`
$ git log # check that there's only one commit
$ git show # check that the commit looks correct
$ git push --force-with-lease

Umm, that didn't come out right.

Ah, sorry, that should have been fixup, not pick.

Try:
$ git reset --hard e2b6b0a6bba3b1ef56461d01ead89a69ad17549a
$ git show # make sure it looks correct
$ git push --force-with-lease

Untested:
# ...

sorry, I wasn't clear in my phrasing, I wanted to ask how to undo the bors action 😆  - that seems resolved after your r-?
(rebase help is also always appreciated, but I think I made it work now, after making it much worse temporarily)

bors r+

bors r-
I already filed and r+ed #10188.

sorry, I wasn't clear in my phrasing, I wanted to ask how to undo the bors action laughing - that seems resolved after your r-?

Oh, sorry.


v2.10.1-dev

v2.10.1:
New
None
Fix

Ignore changes made to metadata.user_data in any instance, since changing the value will destroy and recreate the instance.

resource "oci_core_instance" "instances" {
  ...
  ...
  metadata = {
    ssh_authorized_keys = each.value.autherized_keys
    user_data           = lookup(each.value.optionals, "user_data", null)
  }
  lifecycle {
    ignore_changes = [
      metadata["user_data"]   <------------------------------ note this
    ]
  }
}

Breaking Changes
None

@Twirlyz  please update the releases.md file https://github.com/Binsabbar/oracle-cloud-terraform/pull/73/files#diff-1d2b91d957f8612017f196e6cc2e83c33bcfe3fd348dc3619e9e9672e797d46f


[SVG] vphantom produces too much width

E.g., $a\vphantom {pretend something is here}b$ produces a wide output in SVG output (whereas HTML/CSS and CommonHTML output will not).

I can't reproduce your result.  The example works as expected in all three output renderers for me in Firefox and Chrome on Mac OS X.  What browser and OS are you using?

I ran into this on mathjax-node; I do see it on Chrome as well. The output on http://cdn.mathjax.org/mathjax/latest/test/sample-dynamic-2.html is ~190px for SVG but ~18px on HTML.
To clarify: the "space" is after the b (or "overlapping" it I suppose).

OK, I think I understand what you are seeing.  The problem is that the bounding box information for the expression is built up as the expression is generated, so the size of the phantom text is incorporated into the box before the phantom action occurs, and that size is maintained, so the SVG output thinks the output is wider than the final expression's visible extent.  The <svg> element is supposed to get a right margin that should compensate for that extra width, but that doesn't seem to be making it through to the output in MathJax-node (but does in the browser, which is why I wasn't seeing it).
I'll look into possible solutions.

Sorry, thought this was a mathjax-node complaint, but I see it is for MathJax itself.

PS, the problem with the missing right-margin is resolved in this patch of mathjax-node, so if you actually are using MathJax-node, and you are using the develop branch, then you should be getting the right-margin that compensates for the extra width.

The difficulty is that the \vphantom produces
<mpadded width="0">
  <mphantom>
     ...
  </mphantom>
</mpadded>

The <mphantom> should have the full width of its content, as it does here, and the <mpadded> should keep track of the overflow, since it doesn't know that the content isn't being displayed, and if it were displayed (as in the output of \rlap{} for example), you would want the overflow to be part of the final svg (with right-margin to compensate).
Fixing this would require maintaining yet another value in the bounding box data (already a source of some performance issues) that would provide an "ignorable width" so that mpadded nodes could use to clip the results when their contents are hidden and the width is specified.  Note, however, that overflow an occur on all four sides (top, bottom, and left, as well as right), so to do it right, clipping values for all four would be needed, adding more overhead, though the right-hand clipping would be the most important case.
Is the right-margin compensation not sufficient?

The problem is that the bounding box information for the expression is built up as the expression is generated, so the size of the phantom text is incorporated into the box before the phantom action occurs, and that size is maintained, so the SVG output thinks the output is wider than the final expression's visible extent.

Thanks for the background.

Sorry, thought this was a mathjax-node complaint, but I see it is for MathJax itself.

Apologies for not being clearer myself. I first saw it on mathjax-node and then checked MathJax itself and saw it there, too.

if you actually are using MathJax-node, and you are using the develop branch, then you should be getting the right-margin that compensates for the extra width.

Hm. I am using mathjax-node#develop and see the problem there. I'll investigate and file a bug downstream.

Fixing this would require maintaining yet another value in the bounding box data (already a source of some performance issues) that would provide an "ignorable width" so that mpadded nodes could use to clip the results when their contents are hidden and the width is specified. Note, however, that overflow an occur on all four sides (top, bottom, and left, as well as right), so to do it right, clipping values for all four would be needed, adding more overhead, though the right-hand clipping would be the most important case.
Is the right-margin compensation not sufficient?

Thanks for the assessment. I don't have an opinion at this time. Let's discuss this F2F perhaps.

In testsuite:
MathMLToDisplay/issue1637.html


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.15.17 to before v5.15.25",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=e23f075d77987de4215c8e0696f28bcc707506f7",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=ddd46059f7d99119b62d44c519df7a79f2e6a515"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "e23f075d77987de4215c8e0696f28bcc707506f7",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "ddd46059f7d99119b62d44c519df7a79f2e6a515",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "iwlwifi: fix use-after-free\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.15.25 by commit ddd46059f7d99119b62d44c519df7a79f2e6a515, it was introduced in version v5.15.17 by commit e23f075d77987de4215c8e0696f28bcc707506f7. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2022-1000456


Does not work with bluetooth keyboard.

Does not work with bluetooth keyboard.
The same bluetooth keyboard works in 'KeyWriter' well.

Hi there! Thanks for posting this issue. If you have time, please try this binary in the branch 1-multiple-keyboards, to address more complex USB setups.
It may solve the problem simply; I'm using it myself but haven't merged it into the main branch yet, just to avoid disrupting other users.
Let me know if it works for you, and thank you again!

Thanks for you reply. It's still not work. I can change the page by using left and right array key of my keybord, but it could not  typing words.


Charge and Drive Details no information in dashboard

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Charge Details and Drive Details dashboard are not showing end total information such as
in Drive Details dashboard:

Distance is N/A
Energy used is N/A
Duration is N/A

in Charge Details dashboard:

Added is N/A
Cost is N/A

Expected Behavior
Show information in N/A result in dashboard
Steps To Reproduce

Go to Internal/Drive Details Dashboard
Select Date range

Relevant log output
Not so sure what to look in the logs

Screenshots

Additional data
No response
Type of installation
Docker
Version
v8.5.6

IMHO you are supposed to access that via the "Drives" dashboard.

IMHO you are supposed to access that via the "Drives" dashboard.

This is correct. If you want to see multiple drives or charges, use Trip

IMHO you are supposed to access that via the "Drives" dashboard (resp. via "Charges")

IMHO you are supposed to access that via the "Drives" dashboard.

This is correct. If you want to see multiple drives or charges, use Trip

OK, I can confirm that it works when going from ex. Drives Dashboard>Select or click in one history I can see information.


Keyboard as an optional Controller

Hi there, it would be nice if there was an option to have an optional controller bound to the keyboard. It would be reported as just another controller, obviously with minimal features (no vibro, no analog). I know that libgdx already has support for keyboard input but it's a bit ugly to have different mechanics for the two inputs. Ideally one could implement a two player games that can be played with a keyboard and a gamepad (as well as with two gamepads) using the same code, just handling two Controller objects.
Another benefit is that button mapping would be standardized to the same keys for any game.
Or maybe something like this already exists ?
Thanks

It does not exist, I made something like that in my own game (lightblocks) by abstracting the input method. But it is more fitting into the gdx-controllerutils project as it already provides a mapping feature. Feel free to add it there.


Cannot load zsh auto-completion

If I tried what docs suggest it doesn't work at all:
$ cat ~/.zshrc
 eval (env _PIPENV_COMPLETE=source-zsh pipenv)

Then I tried following:
$ cat ~/.zshrc
 eval "$(env _PIPENV_COMPLETE=source-zsh pipenv)"

Now I get following error:
zsh compinit: insecure directories, run compaudit for list.
Ignore insecure directories and continue [y] or abort compinit [n]?

Hey @Zhomart, this was a feature implemented by @kennethreitz and I'm not entirely sure how it was originally supposed to function. He may have some insight on this.

the only documented (and tested) way of doing this is for fish, currently.

I'd look here for guidance: https://github.com/click-contrib/click-completion

I'm going to close this out since there doesn't seem to be anything further actionable here. Please feel free to let us know if you have any more information, questions, or reproducible examples, @Zhomart.


Dispatcher TTL feature no longer works in AEM 6.2

Required Information

[X] AEM Version, including Service Packs, Cumulative Fix Packs, etc: AEM6.2GA
[X] ACS AEM Commons Version: 3.14.6
[X] Reproducible on Latest? yes/no

Bug Description
The Dispatcher TTL feature no longer works on AEM 6.2 due to a regression bug caused by the commit in #975.
https://github.com/Adobe-Consulting-Services/acs-aem-commons/commit/040365685114d14613f3553bb2d4d3144b09c03c
Steps to Reproduce
Try using the ACS Commons TTL features on AEM6.2GA with ACS Commons 3.14.6.  It no longer works, this is due to the commit from #975.  If you install ACS Commons 3.8.2 to the instance instead then it works since the deprecated "pattern" configuration property is used instead.
https://adobe-consulting-services.github.io/acs-aem-commons/features/dispatcher-ttl/index.html

@andrewmkhoury I can't reproduce this issue. Can you clarify what exact steps you used to reproduce? I tested the Max Age filter (on 6.2 GA w/ ACS AEM Commons 3.14.6) and see the Cache-Control header being set correctly:
$ curl -H"Server-Agent: Communique-Dispatcher" -u admin:admin -v http://localhost:4502/content/geometrixx/en.html 2>&1 | grep Cache-Control
< Cache-Control: max-age=60

Can you also provide the output of /system/console/status-httpservice ?

@andrewmkhoury remember the dispatcher http request header must be present for the filter to process the request; this is often overlooked when testing directly against  Aem. Iirc this header requirement is customizable in the osgi service.

FWIW, the header requirement is not currently customizable, but it is easy to set when using curl (see example above).


feat(pose_estimator_arbiter): add pose_estimator_arbiter to realize multi localizer

Description
This PR implement a mechanism to launch multiple pose_estimators, and allow for pausing and resuming specific ones according to the situation.
The package README describes the purpose in detail.
Related links
[Proposal] Runtime Localization Module Switching for Multi-Pose Estimators #3878
Tests performed
I have confirmed that ndt/yabloc/eagleye/artag is available as before with logging_simulator.launch.xml .
Also, I confirmed that by specifying multiple pose_estimators at startup, pose_estimator_arbiter is launched and arbitrates the pose_estimators according to the situation.
sample data
sample data (rosbag & map)
The rosbag is simulated data created by AWSIM. The map is an edited version of the original map data published on the AWSIM documentation page to make it suitable for multiple pose_estimators.
how to launch
When start a single pose_estimator:
ros2 launch autoware_launch logging_simulator.launch.xml \
    map_path:=$HOME/MAP vehicle_model:=sample_vehicle sensor_model:=awsim_sensor_kit \
    pose_source:=ndt # or yabloc, eagleye or artag

When start multiple pose_estimator:
ros2 launch autoware_launch logging_simulator.launch.xml \
    map_path:=$HOME/MAP vehicle_model:=sample_vehicle sensor_model:=awsim_sensor_kit \
    pose_source:=ndt_yabloc_eagleye_artag # or ndt_yabloc, eagleye_artag and so on

expected behavior
When start a single pose_estimator, it launches the same nodes as before.
When start multiple pose_estimator, it works as follows.

https://github.com/autowarefoundation/autoware.universe/assets/24854875/62b26fdd-dcf0-4b1c-a1a0-ecd633413196

[!IMPORTANT]
We can visualize the debug prints of pose_estimator_arbiter at any time with the following command
(I recommend enabling logging, because with the default rviz settings, we do not know which pose_estimator is running.)

ros2 service call /localization/pose_estimator_arbiter/config_logger \
    logging_demo/srv/ConfigLogger \
   '{logger_name: localization.pose_estimator_arbiter , level: debug}'

[!IMPORTANT]
Eagleye requires a GNSS doppler, but AWSIM does not generate it. So I have stored my pseudo-generated dopplers in rosbag. Because of that, Eagleye's estimated position is not accurate.

Notes for reviewers

[!TIP]
If you want to create an rviz screen like the demonstration video, please visualize the following topics.
/localization/pose_estimator_arbiter/debug/marker_array
/localization/pose_estiamtor/points_aligned
/localization/pose_estimator/yabloc/image_processing/lanelet2_overlay_image
/localization/pose_estimator/yabloc/pf/predicted_particles_marker
/localization/pose_estimator/ar_tag_based_localizer/debug/image
/localization/eagleye/pose_with_covariance/to_relay

Sorry for the large number of commits.

Interface changes
The default behavior remains the same.
Users can now specify multiple pose_estimators for pose_source . By default, ndt is launched.

Effects on system behavior
The default behavior remains the same.
Users can now specify multiple pose_estimators for pose_source . By default, ndt is launched.

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

Hi. All reviewers.
I am going to remove the pcd_occuancpy, artag_position rules from the main stream and implement rules that switch all pose_estimators by reference to the vector map.
This will simplify the source code and README.

I start this modification, so I revert this PR back to draft once.


Automate git submdoules updating

Following discussion from #1238 and #1234.
The published documentation on the website is stored in babel/babel repo (which simplify maintenance). We are currently updating the git submodules manually each time Babel documentation files were updated.
Possible solutions:

@babel-bot could push in this repository fresh submodules when documentation was updated in Babel/babel. In which branch should we push?

Enable the solution above in babel/babel.github.io CI. We need to figure out how to build and publish the website through the CI (not travial when using Jekyll and gh-pages).

We need to figure out how to build and publish the website through the CI

What about moving from GitHub Pages to a more powerful host like Netlify? Then you can have a build script that does whatever you need, without having to hack around GitHub Pages' many limitations 😃

Is it just for the documentation? Instead of adding the entire Babel repo as a submodule, we could just pull the required Markdown files (eg. through their raw Github URLs). If we move the site to Netlify, we can do that as part of the build script. This is much more maintainable/scalable than hacking random things like auto-updating the Git submodules.

I'll try to take a look at this tonight and see how difficult it'd be to remove the usage of submodules where possible.

yeah this was just for the readme and we wanted the version on master instead of npm since could be older, but ideally we would have versioned docs

Submitted a PR (#1293) to download the readmes from GitHub via their raw.githubusercontent.com URLs.  This adds around one second to the build time for the site, but we'll easily save several seconds by avoiding the submodule updates so it's still a net win.

Thanks for your work @Daniel15!


Add a frontend classifier module

Before throwing the sentence at Genie/decanlp (and until decanlp is trained with enough tasks that it can handle everything), we should run a simple frontend classifier to distinguish:

commands
questions
chatty text
insults / spam
other

@gcampax I'm going to try to do this. Also, do you know of any datasets which contain data classified by commands, questions, or spam?

I don't know of a single dataset that contains all of these.
We have our own dataset that contains commands, but there are probably larger datasets out there you can leverage.
There are a lot of datasets such as WebQuestions or WikiQA (maybe others) that contains questions.
For chatty text, I would look into end-to-end dialogue papers, for example those using the Switchboard or Ubuntu Tech Support datasets, and from those see if there are truly free text datasets we can leverage.
For spam I don't know of a good dataset of the top of my head. There are spam datasets of emails but they are not as relevant. OTOH, spam is usually recognizable by pattern, for example because it contains questionable links, or weird capitalization. You might be able to do something clever about it.
Other is of course other, and that would be where the fun would be - for example using confidence scores to detect out of distribution data.

Fixed by #294


[BUG] Incorrect subimages for TX files

Describe the bug
Traversing subimages in TX files (multipage TIF) cause an infinitive loop.
It is not returning a correct ImageSpec for the given page.
To Reproduce
Steps to reproduce the behavior:
auto in = OIIO::ImageInput::open("normal.tif");
auto spec = in->spec();
int subimage = 0;
while (spec.format != OIIO::TypeUnknown)
{
  //...
  spec = in->spec(++subimage); // next subimage
}

Expected behavior
It works correctly for multipart EXRs - the loop ends by returning OIIO::TypeUnknown format if subimage not exists.
It also looks that the spec is not updated after calling in->spec(++subimage)
Evidence
normal.zip
Platform information:

OIIO branch/version: 2.1.17
OS: Windows 10 x64
C++ compiler: VS2019 x64
Any non-default build flags when you build OIIO:

Yes, I can reproduce.  Looking into it...

A combination of a bug and a misunderstanding.
When a TIFF file is found to contain the tag TIFFTAG_PIXAR_TEXTUREFORMAT, it's recognized as a MIP-mapped texture, with the multiple TIFF directories interpreted as MIPmap levels of a single "subimage" (whereas without that tag, the image is treated as if it has multiple subimages, but they aren't MIPmapped).
The full signature of the spec() function is:  ImageInput::spec(int subimage, int miplevel=0)
I can't tell if you (a) want to walk through what OIIO considers subimages, and expect to see just ONE (which will have many MIP levels, but you aren't iterating over those in your example); or (b) if you want to walk through the MIP levels.
If (b), then you want your loop to look like this:
int miplevel = 0;
while (spec.format != OIIO::TypeUnknown)
{
  //...
  spec = in->spec(0, ++miplevel); // next MIP level
}

However, if you really did want (a), then you are doing it correctly and there is a bug -- totally my fault -- in ImageInput::spec(), in the specific case where it's a "mip-mapped" TIFF file but you are passing an invalid subimage request. I will post a patch immediately, stay tuned...

Proposed fix: #2723

(a) is right :)
I'm using the subimage for multipart exrs and I don't need to walk through the MIP levels...


The to_windows method yields an odd error and doesn't generate the windows.

Here's the sample data I'm using:  https://www.dropbox.com/scl/fi/r3zpba1gknl3d0hvxencz/linref_bug.zip?rlkey=d4y1lcun3urm07f6pb6w4m9ki&dl=0
For some reason, I can't get the to_windows method to work with this particular input data.
I'm not sure what's so special about this GeoDataFrame, but when I run the to_windows method on it, I get an error message that sounds like its trying to interpret the beg column as a string instead of a double.
import geopandas as gpd
import linref as lr

# Input layer is in EPSG:3081
srcc_missing = gpd.read_file('c:/temp/linref_bug.gpkg')

# Making the Events Collection object
srcc_missing_ec = lr.EventsCollection(srcc_missing.copy(),
                                      keys=['RIA_RTE_ID'],
                                      beg='START',
                                      end='END',
                                      geom='geometry')

# Trying to make the new non-overlapping windows.
srcc_windows_ec = srcc_missing_ec.to_windows(length=100,
                                             steps=1,
                                             fill='balance')

The code above yields the following error traceback:
Traceback (most recent call last):

  File "C:\Users\diasf\AppData\Local\Temp\ipykernel_18048\1850216886.py", line 1, in <cell line: 1>
    srcc_windows_ec = srcc_missing_ec.to_windows(length=100,

  File "C:\ProgramData\Anaconda3\envs\gpd\lib\site-packages\linref\events\collection.py", line 1146, in to_windows
    df = df.astype(dtypes, copy=False)

  File "C:\ProgramData\Anaconda3\envs\gpd\lib\site-packages\pandas\core\generic.py", line 6226, in astype
    res_col = col.astype(dtype=cdt, copy=copy, errors=errors)

  File "C:\ProgramData\Anaconda3\envs\gpd\lib\site-packages\pandas\core\generic.py", line 6240, in astype
    new_data = self._mgr.astype(dtype=dtype, copy=copy, errors=errors)

  File "C:\ProgramData\Anaconda3\envs\gpd\lib\site-packages\pandas\core\internals\managers.py", line 448, in astype
    return self.apply("astype", dtype=dtype, copy=copy, errors=errors)

  File "C:\ProgramData\Anaconda3\envs\gpd\lib\site-packages\pandas\core\internals\managers.py", line 352, in apply
    applied = getattr(b, f)(**kwargs)

  File "C:\ProgramData\Anaconda3\envs\gpd\lib\site-packages\pandas\core\internals\blocks.py", line 526, in astype
    new_values = astype_array_safe(values, dtype, copy=copy, errors=errors)

  File "C:\ProgramData\Anaconda3\envs\gpd\lib\site-packages\pandas\core\dtypes\astype.py", line 299, in astype_array_safe
    new_values = astype_array(values, dtype, copy=copy)

  File "C:\ProgramData\Anaconda3\envs\gpd\lib\site-packages\pandas\core\dtypes\astype.py", line 230, in astype_array
    values = astype_nansafe(values, dtype, copy=copy)

  File "C:\ProgramData\Anaconda3\envs\gpd\lib\site-packages\pandas\core\dtypes\astype.py", line 170, in astype_nansafe
    return arr.astype(dtype, copy=True)

ValueError: invalid literal for int() with base 10: '0.0'

The to_windows method seems to work perfectly well for another dataset, but something about this is tripping it up. Am I missing something just super simple and obvious?
Thanks!

@phildias Thanks for bringing this up, I'll take a look!
Could you confirm what version of linref you're using?
Could you provide the output of srcc_missing_ec.info()?

Opened a new branch to address this item. @phildias please review the PR and ensure that this fixes your issue. If it does, please approve and I'll merge.

Issue solved with PR #5.


AppendToBlob Example not fully async

The example code on the page doesn't seem to be fully async.
In particular, the async variants of the following two methods aren't used:
CreateIfNotExists => CreateIfNotExistsAsync
AppendBlock => AppendBlockAsync
I appreciate that it's only an example but the copying from the input MemoryStream to a temporary buffer seems a little inefficient.

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

ID: 589ba022-a885-4e6e-8acc-946d732400a3
Version Independent ID: f73863d2-f2eb-ded0-a207-65c7d70ebf02
Content: Append data to a blob with .NET - Azure Storage
Content Source: articles/storage/blobs/storage-blob-append.md
Service: storage
Sub-service: blobs
GitHub Login: @pauljewellmsft
Microsoft Alias: pauljewell

@Zerantal
Thanks for your feedback! We will investigate and update as appropriate.

@Zerantal
I'm going to assign this to the document author so they can take a look at it accordingly.
@pauljewellmsft
Can you please check and add your comments on this doc update request as applicable.

@Zerantal Thanks so much for this feedback. The code sample in the article has been updated with a few changes, including the consistent use of async methods.
Note that there's a feature request logged for AppendBlock* to consume BinaryData rather than only Stream. We'll update the docs if that's supported in the future.
Thanks again for contributing to the docs. Closing the issue for now, but please reach out if you have further comments on this item.
#please-close


Add support for fuzzing

Use cargo-afl or something similar, experiment with different fuzzers.
Make a nightly CI run, as fuzzing typically likes to take time.
The goal is to find crashes and edge cases.
Before going for a fuzzer that we should have property based tests in place, see #16
Hook up the validator to the output (#18 ), to also try finding cases where we do not crash but generate schema that does not validate.

This should be solved for all crates - see https://github.com/balena-io-modules/balena-temen/issues/6


Expandable doc: Close button hit zone misplaced

VSCode Version: Code - Insiders 1.13.0-insider (afe47dc84467b487ce0079ff72f55e7ef4f7ebca, 2017-05-30T10:34:18.253Z)
OS Version: Darwin x64 15.6.0
Extensions: none

Testing #27315
When I click the close button directly, it is not hit. I have to click slightly to the left. The hover indication of the button reflects that behavior. It seems the hit zone is moved to the left.


Implement Reseedable::curve and Reseedable::terrace

The Reseedable class has methods and operators which allow it to be modified and combined with other Reseedables. Module parameters are passed in these methods rather than set after construction. Most of the non-generator libnoise Modules have a corresponding method. Two libnoise modifier modules do not have a corresponding method: Terrace and Curve.
Both of these modules have a variable-length parameter set consisting of a set of control points. Normally these are set one at a time in separate function calls. Since Reseedable::curve and Reseedable::terrace would need to take all the parameters in one call, the control points should probably be passed as a pair of iterators.

Implemented in https://github.com/serin-delaunay/Wangscape/tree/curve-terrace; needs tests.


Use real-world references examples in kitchen-sink

This way it's easier to check if fields are mapped correctly to styled version. We'd like to get this right for the September Texture release.
@Melissa37 could you update the following file with using real world ref examples instead. Only one reference per type for now, with as much as fields filled out as possible. It needs to conform to the dar spec (as seen in the current tagging of the file)
https://raw.githubusercontent.com/substance/texture/master/data/kitchen-sink/manuscript.xml

Hi Michael
Sure, @JGilbert-eLife and I will look through these, but we need to understand your @id= logic on the <ref> as we don't understand it and don't approve of what we're seeing here :-)
Happy to have a call to discuss if that would help, but we cannot review with these ids as they currently stand.
Thanks!
M

We just need unique ref id's for the example, no special logic. You an choose them freely.

Sorry for the delay responding to this, @JGilbert-eLife will try to do this today!

Hi @michael -
I'm attaching my draft from today here - I'd like to take another look on Monday to make sure I haven't missed anything but please feel free to examine this for any glaring problems now!
manuscript.xml.zip
A few things jumped out at me while I was doing this:

Books - the DAR spec (https://github.com/substance/dar/blob/master/DarArticle.md) refers to 'title' but it should be, correctly, as in the example file - 'source'
Preprints - they seem to be missing from DAR - is that intentional? There was an arXiv article in the original file as a report; this is an incorrect treatment of a preprint!
Webpage - there's no provision for accessed date in DAR. Is that correct? This is something we've been treating as pretty important given the rapidity with which websites can change or disappear.
The DAR spec allows day and month for many more kinds of references than seems likely to find a use - for example, it's unlikely that the precise date will be needed for book - and even for journal articles, which do have precise publication dates, they are usually cited by year alone. This is an observation more than a criticism though - better to have and not need!

Will go through this when it's not Friday afternoon to make sure there's nothing I've overlooked!
Thanks,
James

@JGilbert-eLife so we want to have preprint as one available type?

@michael - yes. Certainly, eLife needs this reference type and I think without it, we'd be preventing users from properly citing a reference type that is increasing in importance.
Here's an example form our Kitchen Sink - broadly similar to a journal but without volume and page-range information.
<!-- Reference to a Preprint: bioRxiv -->
    <ref id="bib3">
        <element-citation publication-type="preprint">
            <person-group person-group-type="author">
                <name>
                    <surname>Bloss</surname>
                    <given-names>CS</given-names>
                </name>
                <name>
                    <surname>Wineinger</surname>
                    <given-names>NE</given-names>
                </name>
                <name>
                    <surname>Peters</surname>
                    <given-names>M</given-names>
                </name>
                <name>
                    <surname>Boeldt</surname>
                    <given-names>DL</given-names>
                </name>
                <name>
                    <surname>Ariniello</surname>
                    <given-names>L</given-names>
                </name>
                <name>
                    <surname>Kim</surname>
                    <given-names>JL</given-names>
                </name>
                <name>
                    <surname>Judy Sheard</surname>
                    <given-names>J</given-names>
                </name>
                <name>
                    <surname>Komatireddy</surname>
                    <given-names>R</given-names>
                </name>
                <name>
                    <surname>Barrett</surname>
                    <given-names>P</given-names>
                </name>
                <name>
                    <surname>Topol</surname>
                    <given-names>EJ</given-names>
                </name>
            </person-group>
            <year iso-8601-date="2016">2016</year>
            <article-title>A prospective randomized trial examining health care utilization in individuals using multiple smartphone-enabled biosensors</article-title>
            <source>bioRxiv</source>
            <pub-id pub-id-type="doi">10.1101/029983</pub-id>
        </element-citation>
    </ref>

<!-- Reference to a Preprint: arXiv.org, no DOI -->
    <ref id="bib9">
        <element-citation publication-type="preprint">
            <person-group person-group-type="author">
                <name>
                    <surname>Cartwright</surname>
                    <given-names>A</given-names>
                </name>
            </person-group>
            <year iso-8601-date="2016">2016</year>
            <article-title>The Venus Hypothesis</article-title>
            <source>arXiv.org</source>
            <ext-link ext-link-type="uri" xlink:href="https://arxiv.org/abs/1608.03074">https://arxiv.org/abs/1608.03074</ext-link>
        </element-citation>
</ref>

@JGilbert-eLife do preprints have an issue property?

@michael - As far as I can tell, most preprint servers don't have issues, volumes or page ranges. I think we're probably safe to exclude all three properties for now.
Some do have non-DOI IDs, but I think that's usually a component of the URL.

Ok I will remove those then from the schema. What concerns me a bit, is that CSL/JSON does not have a special type for preprints. They use the article type. See: https://twitter.com/zuphilip/status/1037663548762017794
Was wondering if we were able to ensure 100% mapping between our JATS-serialisation and CSL/JSON. To make sure we can use Citeproc processing for citation styles in the future.
Thoughts?

For the webpage accessed date, what element would you suggest for tagging that?

It seems obvious but just to be sure: year,month,day we will use to tag when the webpage has been published (only works for things like blog posts, articles that have a date). And accessedDate to capture when it has been looked up.

I have reached out to CSL and cc'd @michael. Let's see what they say?

@Melissa37 thanks!
And do you also agree that book - and journal articles would not need month and day properties? I will remove them then.

@michael yes I agree that book and journal don't need month and day.
However, it would be good to get a consensus from others in the consortia because we don't have experience in humanities and it might be different there

Ok, I will open a separate issue for that.

@JGilbert-eLife
Makes sense. I will add date-in-citation. I will only consider the iso-8601-date attribute. The content of the element is redundant and can be generated.
Just to verify the spec:
person-group[person-group-type='author']?, article-title?, uri?,year?, month?, day?,
date-in-citation[iso-8601-date=yyyy-mm-dd], publisher-loc*, source?

is publisher-loc valid here?
do we also need publisher-name*?

I would argue that you don't need a publisher location for a website :-)

@Melissa37 then what if the webpage is: https://elifesciences.org/articles/37267 and I didn't see there's also a DOI ? ;)

@JGilbert-eLife In the case of date-in-citation this is fine. I mean I do see a bit of inconsistency with the explicit year,month,day tags and a combined iso-8601-date.
However it brings me to the idea that in Texture internally we could potentially treat all date-related fields as single iso-8601 strings.
Then we could just display one field for editing a date which can be filled out as "2017" or "2017-04" or "2017-04-23" instead of individual inputs for year,month,day. Could that make any sense? In the XML still we could have year/month/day separately. Just an idea...  to simplify/normalize the use of "dates" Texture.
@oliver---- what do you think?

@michael it would be production level QC to ensure https://elifesciences.org/articles/37267  is tagged as a journal article and not a website. I think it would be very rare for an author to submit a journal article reference without the relevant other metadata to indicate it is a journal :-)

I was just joking. ;) I'm stripping the publisher-loc element from webpage.

I'm known for my lack of a sense of humour!
Thanks :-)

@michael I am fine with a unified internal format. Of course There are these rare cases where a date is expressed verbally, where we might have problems with that, but for that we gonna find a solution.

@Melissa37 these jokes ain't funny... :D
@oliver---- need to talk more about that, but the intent would be to normalize all date-related fields in Texture and have consistent UX for them. Thinking of a date picker that allows you to either select a year, a year-month, or a year-month-day.

@JGilbert-eLife I'd propose that we follow CSL/JSON's example and use 'article' as publication type, instead of 'preprint'. Preprint to me is more like a temporary phenomenon, but in fact this is just an article that has not been published in a journal. However we can use preprint as a label if that's making things easier to understand for users.
I've almost finished addressing all issues mentioned in the thread, I will soon provide you a link so you can play with the new reference editing/rendering based on real examples.

That sounds good to me
On Sun, Sep 9, 2018 at 6:30 PM Michael Aufreiter<EMAIL_ADDRESS>wrote:

@JGilbert-eLife https://github.com/JGilbert-eLife I'd propose that we
follow CSL/JSON's example and use 'article' as publication type, instead of
'preprint'. Preprint to me is more like a temporary phenomenon, but in fact
this is just an article that has not been published in a journal. However
we can use preprint as a label if that's making things easier to understand
for users.
I've almost finished addressing all issues mentioned in the thread, I will
soon provide you a link so you can play with the new reference
editing/rendering based on real examples.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/substance/texture/issues/677#issuecomment-419731459,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFxYJFKuCwm9YPrcVdDF6kMqBjgOzfcnks5uZVA-gaJpZM4WLJbS
.

--
Melissa Harrison, Head of Production Operations, eLife
eLife Sciences Publications, Ltd
1st Floor
24 Hills Road
Cambridge
CB2 1JP
01223 855345
orcid.org/0000-0003-3523-4408
Receive content alerts
http://www.elifesciences.org/crm/civicrm/profile/create
Browse articles http://elife.elifesciences.org/browse
eLife Sciences Publications, Ltd is a limited liability non-profit
non-stock corporation incorporated in the State of Delaware, USA, with
company number 5030732, and is registered in the UK with company number
FC030576 and branch number BR015634 at the address First Floor, 24 Hills
Road, Cambridge CB2 1JP.

Hey @JGilbert-eLife @Melissa37 !
I need some help regarding the examples you provided:
You specified pub-type book 3 times. However we want to have each pub type used just once. Can you help me getting proper examples for the following types that are currently missing:

'magazine'
'chapter'

      <ref id="r3">
        <element-citation publication-type="book">
          <person-group person-group-type="author">
            <name>
              <surname>Buzsaki</surname>
              <given-names>G</given-names>
            </name>
          </person-group>
          <year>2006</year>
          <source>Rhythms of the Brain</source>
          <publisher-loc>New York</publisher-loc>
          <publisher-name>Oxford University Press</publisher-name>
          <pub-id pub-id-type="doi">10.1093/acprof:oso/9780195301069.001.0001</pub-id>
          <pub-id pub-id-type="isbn">978-0195301069</pub-id>
        </element-citation>
      </ref>
      <ref id="r4">
        <element-citation publication-type="book">
          <person-group person-group-type="editor">
            <name>
              <surname>Chmeil</surname>
              <given-names>N</given-names>
            </name>
          </person-group>
          <edition>2</edition>
          <year>2008</year>
          <month>05</month>
          <day>26</day>
          <source>An Introduction to Work and Organizational Psychology: A European
            Perspective</source>
          <publisher-loc>Oxford</publisher-loc>
          <publisher-name>Wiley-Blackwell</publisher-name>
          <pub-id pub-id-type="isbn">978-1405132763</pub-id>
        </element-citation>
      </ref>
      <ref id="r5">
        <element-citation publication-type="book">
          <person-group person-group-type="author">
            <name>
              <surname>Coyne</surname>
              <given-names>JA</given-names>
            </name>
            <name>
              <surname>Orr</surname>
              <given-names>HA</given-names>
            </name>
          </person-group>
          <person-group person-group-type="editor">
            <name>
              <surname>Otte</surname>
              <given-names>D</given-names>
            </name>
            <name>
              <surname>Endler</surname>
              <given-names>JA</given-names>
            </name>
          </person-group>
          <year>1989</year>
          <month>08</month>
          <day>22</day>
          <chapter-title>Two rules of speciation</chapter-title>
          <source>Speciation and its consequences</source>
          <fpage>180</fpage>
          <lpage>207</lpage>
          <publisher-loc>Sunderland, MA</publisher-loc>
          <publisher-name>Sinauer Associates</publisher-name>
          <pub-id pub-id-type="isbn">978-0878936588</pub-id>
        </element-citation>
      </ref>

@michael - Sorry! One of those should be chapter (the one featuring a chapter, of course!).
I think I was thrown by your original file having multiple books in it; I assumed that meant you needed the variants for this type. The reason I included two instances of book is that we allow books with only authors and books with only editors. These are two distinct cases and I wasn't sure I could convey that with just the one instance. Happy to remove the editor example if need be.
I'll put in a magazine example too; sorry I missed this, it's not a type we've used so I didn't have any live content to hand.
James

@michael - me again. Here's the updated manuscript.xml document. Based on previous discussion, I've removed the month and day elements and I have added an article/preprint example. I took out the extra book with editors reference; let me know if that should go back!
Chapter has been corrected, magazine has been added.
Please let me know if anything else needs sorting out.
manuscript.zip

Thank you @JGilbert-eLife! I will integrate the changes now. I have some tasks for you, but we can discuss later in the call. :)

Closing this issue, as we have real examples now. @JGilbert-eLife if there are any problems, please open a separate issue and we will try to address those asap.


Add an "Actions" header to the new member tables

The actions in the row only show up on :hover.

I have a new set up for a click-able row that could replace this situation for a single "edit" action, like this one:

This is fixed by always showing the buttons rather than only on hover.


Improving the UI of the landing page and about section

I would like to the the overall ui of home and about section.
Here is my design.
Moksh_new.pdf
Hope this helps!

can u please join discord channel for discussion on UI
awesome work

I'm already in the Moksh's discord channel.
Okay then i'll have a discussion regarding ui there and if you approve i'll start working.

Yes and it's a request from our side to not start working on any issue before getting assigned.

Yes and it's a request from our side to not start working on any issue before getting assigned.

Yes and it's a request from our side to not start working on any issue before getting assigned.

Hi, @shivani-tripurari are u still working on this issue? If so can you please tell what are the issues are resolved and what are the issues are remaining.!

Hi, @shivani-tripurari are u still working on this issue? If so can you please tell what are the issues are resolved and what are the issues are remaining.!

I'm working on other designs as the admins did not confirm about the final design


Some fixes for the incremental xDS implementation

Here is what has been done:

Merge with the upstream master (since the original PR was open TTL was added to the upstream implementation. I fixed code conflicts, but didn't add TTL to the incremental version yet)
The code now respects ResourceNamesSubscribe field and responds only with the requested resources. Here is how it works:

First, the server identifies whether the current request is a wildcard link
Next, we create the list of resources the client is interested in together with the current client version link In order to create this list we take into account ResourceNamesSubscribe, ResourceNamesUnsubscribe and InitialResourceVersions  fields (previously InitialResourceVersions  was ignored)
I updated StreamState struct to include only those 2 fields - this is our primary interface between the server and the cache. Those are the only 2 fields that are needed for the cache to identify what resources to respond with.

I removed nonce from the StreamState and used this fields to store nonce. Previously those fields were unused, which was confusing. Also, this matches the SOTW implementation.
I moved the logic that compares the resource versions and identifies what resources we should include in the response from the SetSnapshot method to the createDeltaResponse method. This was done because this logic should be shared for 2 cases:

When SetSnapshot is called
When we immediately respond to the delta discovery request link Previously in this case we didn't compare old and new resource version and always responded with all resources.
Also, the comparison logic was updated to use the new StreamState

Now we populate RemoveResources field in case when a resource is gone from the server link

A few additional notes:

I updated only v3 implementation. My suggestion is to delete incremental support for v2 completely because v2 will be deprecated in Q1 2021. If the maintainers aren't ok with this I can merge the same changes to the v2.
Both unit and integration tests work after these changes. I also did extensive testing on our staging environment.
We really want this to get merged to the upstream because we don't want to support a fork I'll try to ping some people in the envoy slack and will help with answering review comments and providing necessary fixes.

2020/12/22 02:36:40 server callbacks fetches=0 requests=0 responses=0 delta-requests=62 delta-responses=37

This is great! I tested and pulled this into my own local dev and all looks great. If all is good i'd like to merge this into my fork, anything else we should tackle before? I was thinking we could implement Mux and Linear in another PR or after my initial PR gets merged in. Will definitely try and bump the original PR for review so we can get that logic in

I just let the unit tests run for a bit and I noticed I got this panic:
goroutine 131 [chan receive, 10 minutes]:
testing.(*testContext).waitParallel(0xc0002aff80)
        /usr/local/go/src/testing/testing.go:1008 +0xa7
testing.(*T).Parallel(0xc00019b700)
        /usr/local/go/src/testing/testing.go:815 +0x1ed
github.com/envoyproxy/go-control-plane/pkg/cache/v3.TestLinearConcurrentSetWatch.func1.1(0xc00019b700)
        /go-control-plane/pkg/cache/v3/linear_test.go:223 +0x52
testing.tRunner(0xc00019b700, 0xc0003cb6e0)
        /usr/local/go/src/testing/testing.go:909 +0xc9
created by testing.(*T).Run
        /usr/local/go/src/testing/testing.go:960 +0x350

goroutine 132 [chan receive, 10 minutes]:
testing.(*testContext).waitParallel(0xc0002aff80)
        /usr/local/go/src/testing/testing.go:1008 +0xa7
testing.(*T).Parallel(0xc00019b800)
        /usr/local/go/src/testing/testing.go:815 +0x1ed
github.com/envoyproxy/go-control-plane/pkg/cache/v3.TestLinearConcurrentSetWatch.func1.1(0xc00019b800)
        /go-control-plane/pkg/cache/v3/linear_test.go:223 +0x52
testing.tRunner(0xc00019b800, 0xc0003cb720)
        /usr/local/go/src/testing/testing.go:909 +0xc9
created by testing.(*T).Run
        /usr/local/go/src/testing/testing.go:960 +0x350

goroutine 133 [chan receive, 10 minutes]:
github.com/envoyproxy/go-control-plane/pkg/cache/v3.verifyResponse(0xc00019b900, 0xc00036a000, 0x0, 0x0, 0x1)
        /go-control-plane/pkg/cache/v3/linear_test.go:36 +0x65
github.com/envoyproxy/go-control-plane/pkg/cache/v3.TestLinearConcurrentSetWatch.func1.1(0xc00019b900)
        /go-control-plane/pkg/cache/v3/linear_test.go:238 +0x416
testing.tRunner(0xc00019b900, 0xc0003cb760)
        /usr/local/go/src/testing/testing.go:909 +0xc9
created by testing.(*T).Run
        /usr/local/go/src/testing/testing.go:960 +0x350
FAIL    github.com/envoyproxy/go-control-plane/pkg/cache/v3     600.033s

I don't think this has anything to do with the incremental changes but was just curious if you're seeing this

Hi, @alecholmez. I am back from PTO and going to start working on this again.
I didn't know about the versioning script. My plan is to manually modify the script to copy from v3 to v2, fix Code generated by create_version. DO NOT EDIT. comment and revert the script.  Then we should get identical v2/v3 implementation.
I want to start working on mux and linear caches sometime soon, but my suggestion is to merge this first and then add this functionality in a subsequent PR, either to this repo or to the upstream directly.
I didn't get the timeout while running make test on my branch, all packages, including github.com/envoyproxy/go-control-plane/pkg/cache/v3 run successfully, how can I reproduce this?

I applied the same changes to v2 API and run create_version.sh as well as make test to make sure that nothing was broken.


doc-folder-reorg

Documentation folder reorganization.

I'm puzzled why there are changes to non-document files (build scripts etc.). Has this been synced to main ? Please do not merge until this mystery is solved.

Obsolete


[documentation] Renamed function names in 13-actions chapter of book to reduce confusion

Used different names for add_todo server fn vs add_todo action as using the same name can cause confusion to newcomers to the framework.
On the first readthrough, seeing the same name being used for both action and the server function tripped me up a little.

Thanks very much!


Fixed incorrect code in README

Hello, @alialtaf9! This is your first Pull Request that will be reviewed by Ebert, an automatic Code Review service. It will leave comments on this diff with potential issues and style violations found in the code as you push new commits. You can also see all the issues found on this Pull Request on its review page. Please check our documentation for more information.


Any Entity Command - Project Failed to Build

Hello,
I'm having a problem when I try to run any entity command.
The only message I got is "The project 'xxxx' failed to build.", but if I try to build my entire solution or each project separately all of them build correclty.
What can I do here? I have a project in prod where I need to add a migration but I'm blocked now.
Technical Info:
Package Manager Console Host Version <IP_ADDRESS>48
Visual Studio Community  2017 - 15.7.1
Entity Framework 6.2.0

EF Team Triage: This issue is lacking enough information for us to be able to effectively triage it. In particular, it is missing the following information requested in the new issue template. Can you please provide this information?

Steps to reproduce

Ideally include a complete code listing that we can run to reproduce the issue.
Alternatively, you can provide a project/solution that we can run.
BTW we're not just doing this to be mean :smile:... we get a lot traffic on this project and it takes time to attempt to reproduce an issue based on fragments of information. In addition, our attempt is often unsuccessful as the exact conditions required to hit the issue are often not explicitly included in the code provided. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we ask that folks give us a self-contained way to reproduce an issue.
For a guide on submitting good bug reports, read Painless Bug Tracking.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


could mix javascript & native iphone library to talk to mixpanel?

I asked a question on stackoverflow, but thought this might be better place to ask.
http://stackoverflow.com/questions/34504690/mixpanel-for-hybrid-app
In short, can I use mixpanel-javascript for certain events and mixpanel-iphone for other events from a hybrid app?

Hi @pcompassion, Jared from Mixpanel support here... while I've not yet heard of anyone using both libraries simultaneously, off the top of my head, I can't imagine any problems. It seems to me like it would work just fine but only testing will confirm. I would recommend taking a crack at it and if you run into any problems or have questions shoot me an email at<EMAIL_ADDRESS>either way I'd love to know how it turns out.

@jaredmixpanel Thanks for the answer. will do report if I test it out.

Hi @pcompassion, how the integration of both iOS and JS SDK is doing ?
Are users tracking the same way even if the event is generated from a webview or a native screen of you hydrid app ?
Finally what's happening with funnel if the user path crosses webviews and then native ? Can you track this user as a same person ?
Thanks


[SPARK-4119][SQL] Remove unused HIVE_DEV_HOME related variable

What changes were proposed in this pull request?
According to the Spark SQL Internals wiki, Spark uses HIVE_HOME or HADOOP_HOME instead HIVE_DEV_HOME during generating golden answer files since Hive 0.13.1 (June 2014). Also, Spark uses Hive 1.2 since Spark 1.5.0 (Sep. 2015). This PR removes HIVE_DEV_HOME-related variable.
How was this patch tested?
This is about test suites.

Hi, @liancheng .
Could you review this PR?

Test build #63514 has started for PR 14578 at commit 00714ef.

Can we close this? I'm not sure if pull requests like this adds much value since it has 0 user facing benefit and almost no benefit to developers either. Also we should just get rid of TestHive, etc in the future and this will become irrelevant.

Sure. I see. I'm just interested in SPARK-4119 and want to close that in some way.
I will close this PR.
BTW, if possible, could you close SPARK-4119, too?

Oh, you did close already. Thank you!

Test build #63514 has finished for PR 14578 at commit 00714ef.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63514/
Test PASSed.

PS thank you @dongjoon-hyun for putting some time into reviewing old issues to see if they can be fixed, are fixed already, or are obsolete. It is helpful.


[Question] how to disallow accessing the local file

Hi,
We are able to see the contents of local files when accessing them through a sample html file in headless mode of playwright for chrome browser on windows. Is there any switch or a flag that can block the same?
Here is the index.html file:
<!DOCTYPE html>

win.ini

Result with playwright(headless execution of chrome):

In further investigation, we tried to access a local file through localhost where, in case of playwright's headless flag false for chrome we were not able to see its contents while in case of headless flag true for chrome, we were able to see the contents. Could you tell us the reason why this might be happening?
Here is the screenshot for that case -
Playwright with Headless flag true for chrome browser

Playwright with Headless flag false for chrome browser

Do you run with MS Edge, what command do you use? It maybe that you have some enterprise policy in place that causes that behavior. Try running with bundled chromium it should be able to load local files in both headed and headless modes.

Hi Yury,
We are running with Google Chrome,  and we want to restrict the access of local file ie., we don't want our browser to be able to load the file's contents in the iframe.
This is more important to us and we wanted to know how we can achieve that
Get Outlook for Androidhttps://aka.ms/ghei36

From: Yury Semikhatsky @.>
Sent: Wednesday, September 1, 2021 2:12:47 AM
To: microsoft/playwright-java @.>
Cc: Tushita Aggarwal @.>; Author @.>
Subject: Re: [microsoft/playwright-java] how to disallow accessing the local file (#588)
Do you run with MS Edge, what command do you use? It maybe that you have some enterprise policy in place that causes that behavior. Try running with bundled chromium it should be able to load local files in both headed and headless modes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fplaywright-java%2Fissues%2F588%23issuecomment-909619939&data=04|01|tuaggarw%40adobe.com|7d73ce63370c49a9ef0708d96cbfe4a9|fa7b1b5a7b34438794aed2c178decee1|0|0|637660393736465762|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=qpiv2d9XT1JjXhNW7cPf1LZfC3uJjJmBAK4JdOuL%2FAg%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAG6EFIA6YEGY5UA3ST7QQLTT7U5EPANCNFSM5DDVCTSA&data=04|01|tuaggarw%40adobe.com|7d73ce63370c49a9ef0708d96cbfe4a9|fa7b1b5a7b34438794aed2c178decee1|0|0|637660393736475752|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=o0H7nzeA2w4%2FWI6Srauh%2BwNoA7g9JZZeJuWIoKQhEQo%3D&reserved=0.
Triage notifications on the go with GitHub Mobile for iOShttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fapps.apple.com%2Fapp%2Fapple-store%2Fid1477376905%3Fct%3Dnotification-email%26mt%3D8%26pt%3D524675&data=04|01|tuaggarw%40adobe.com|7d73ce63370c49a9ef0708d96cbfe4a9|fa7b1b5a7b34438794aed2c178decee1|0|0|637660393736475752|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=6u%2BiDtPLUkAjNf2DGPP5YeLVCp5For7atbtazzxunfo%3D&reserved=0 or Androidhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fplay.google.com%2Fstore%2Fapps%2Fdetails%3Fid%3Dcom.github.android%26referrer%3Dutm_campaign%253Dnotification-email%2526utm_medium%253Demail%2526utm_source%253Dgithub&data=04|01|tuaggarw%40adobe.com|7d73ce63370c49a9ef0708d96cbfe4a9|fa7b1b5a7b34438794aed2c178decee1|0|0|637660393736485747|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=O2iCh14DAxIQJbHkXNNvVfIweXjfmIZwg3qpV0wCYGY%3D&reserved=0.

By default Chromium doesn't allow load file url into iframe and you should see the following error in the console 'Not allowed to load local resource: file:///....'  That's what's happening when I try to load the page you posted above from http://localhost:8907/a.html, is not happening for you? If so could you run with DEBUG=pw:browser and post the output here? Whether file urls are allowed from pages loaded from file:// urls is controlled by --allow-file-access-from-files flag in Chrome.

Closing per the comment above.