isVoidReturn should be exposed

Since the use of the inner Function should be allowed, it's important to be able to check if the returned value is void.

Hi thanks for the issue!
On native platforms you can use the WasmFunction.results.length to check whether a function returns something.
On web you can use js_util.typeofEquals<dynamic>(result, 'undefined') to check whether the return is undefined. Before Dart 3.0 I think you need to use the following code for web (in wasm_run we use both methods):
bool isVoidReturn(dynamic value) {
  switch (value) {
    case null:
      return false;
    default:
      return value == null;
  }
}

Although we can could expose it I don't feel it's necessary. But I could be wrong, please let me know if those methods workd for you or if you think we should expose isVoidReturn .
Thanks


Optimize message partitioning with customized partitioner

Raise the priority of this item. This optimization belongs to part of Alcor effort to cut programming latency for large VPC.


Version being set to x.y.z

I am using $VERSION and it is is being set to x.y.z instead of the version I am passing in.

FYI this is a major blocker for me right now.

I think it is only being defined inside an @activity.

Under what circumstances are you seeing this? Is there a way to reproduce.
If you are using the CLI, the argparser requires that version be set on the command line and then it very quicly sets $VERSION (see here)
x.y.z is the default env var value (see here) but you should never see this once environ.context() is entered... i.e. in env_main()

It seems to happen when I use $VERSION outside of an @activity. I need to define it at the module level so that I can access it in some helper functions.

outside of an activity in rever.xsh? Or in a test file? or where exactly?  Sorry I am not seeing it :(

In rever.xsh. I tried setting
version = $VERSION

at the module level of the file, and it gets set to x.y.z.

That is definitely a bug =/

OK I have a test that reproduces this bug.

See #23


fix #485

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

27 of 27 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 87.951%

Totals

Change from base Build<PHONE_NUMBER>5:
0.05%

Covered Lines:
6278

Relevant Lines:
7026

💛  - Coveralls


chore(release): v2.7.0

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #863 (1d921db) into main (3524702) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #863   +/-   ##
=======================================
  Coverage   92.82%   92.82%
=======================================
  Files         115      115
  Lines        4806     4806
  Branches     1375     1375
=======================================
  Hits         4461     4461
  Misses        326      326
  Partials       19       19

Continue to review full report at Codecov.

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


GH-43592: [C++] Remove redundant default constructor/deconstructor in arrow::ArrayStatistics

Rationale for this change
= default constructor/deconstructor are needless in arrow::ArrayStatistics.
What changes are included in this PR?
Remove them.
Are these changes tested?
Yes.
Are there any user-facing changes?
No.

GitHub Issue: #43592

+1


OrientDB vanishes slashes from File Path

OrientDB Version, operating system, or hardware.
v2.2.13
Operating System

[x ] Windows 8

JVM

1.8.0_101-b13

Frameworks/Libs

Spring 4.3.2

Expected behavior and actual behavior
When inserting a file path on a OrientDB table, the file path is not stored correctly, as the slashes are being removed.
Steps to reproduce the problem
This was verified on a Spring application, but can easily be reproduced with OrientDB studio, executing the following DDL / DML instructions:

CREATE CLASS V13_TEST
CREATE PROPERTY V13_TEST.FILE STRING
INSERT INTO V13_TEST(FILE) VALUES('C:\\DIR1\\DIR2')

Instead of storing the value 'C:\DIR1\DIR2', OrientDB is storing 'C:DIR1DIR2'.

hi @vsspt,
You should use escaping with double slashes :INSERT INTO V13_TEST(FILE) VALUES('C:\\DIR1\\DIR2')
Bye

@tglman, it seems that github also needs escaping on slashes, as in the steps to reproduce the problem i included the double slashes, but github shows it as one (and looking at your response, i suppose you used double slashes, but only one is showing).
I've updated the steps to use quadruple slashes, so it shows double slashes, as i intended in the first place.
Please execute the steps, using the correct double slashes, and you will verify the error i reported.

hi @vsspt,
I just tried it and it insert the record correctly for me, are you sure that you don't have anything else that remove the double slashes, i'm running it from  firefox on linux, it may be that your browser is doing some additional escaping remove.
bye

Hi @tglman,
I'm using IE 11.09.
Nevertheless, that's not the issue, as this is happening for me in a Spring application.
I just used the IE example, as i verified it had the same issue, and it was faster to use is a test.
I will create a simple test case demonstrating the error, it should be easier for you to use it in order to replicate the problem.
I will do it today, and provide a link for download as soon as its completed.

@tglman , please donload this test project here: https://meocloud.pt/link/ff285b0d-add4-4690-9a59-d6bedf10cadf/2-2-13-Bug-2.zip/
In order to use the project, you will need to change the plocal url in the file: srt\test\testContext.xml.
In the POM file you can change the OrientDB JDBC driver version.
Basically, this is a simple Spring application that:

Updates a table field using "C:\Folder1\Folder2".
Retrieves the inserted value, and compares it with "C:\Folder1\Folder2"

When using version <= 2.2.12:

Value is corrected inserted as "C:\Folder1\Folder2"

The retrieved value is equal, so the test passes.

When using version = 2.2.13:

Value is incorrectly inserted as "C:Folder1 older2"

The retrieved value is not equal, so the test fails.

The test database details are:

name = test
user = admin
password = admin

hi @vsspt,
I just tried your test with 2.2.14, and seems to work correctly, it may be that we had some regrassion that has bean already fixed, cna you try  2.2.14, it was released yesterday
Regards

Works as expected, thank you.


Bump versions for wasmi and wasmi-validation

Bump versions for wasmi and wasmi-validation in preparation for cargo publish.
Publishing a new cargo version will allow downstream crates to build in no_std
mode on rust stable (a recent change).
If someone with write access to https://crates.io/crates/wasmi could run cargo publish to update wasmi and wasmi-validation it would be appreciated. (you'll need to cargo publish wasmi-validation first because wasmi depends on it)

Travis-ci failed when installing the wasm32 target. Doesn't seem related to the content of the PR.

Thanks @bddap !


Add basic support for array lengths in types

This recognizes let a = [1u8, 2, 3] as having type [u8; 3] instead
of the previous [u8; _]. Byte strings and [0u8; 2] kinds of range
array declarations are unsupported as before.
I don't know why a bunch of our rustc tests had single quotes inside
strings un-escaped by UPDATE_EXPECT=1 cargo t, but I don't think it's
bad? Maybe something in a nightly?

bors r+


Losing Line Breaks when getting ->innerHtml

Looks like html is losing line breaks when getting $dom->find( '.content', 0 )->innerHtml, I'm still digging in to see why.

all white space is reduced to a single space. This includes line breaks.

This is something that happens as part of SimpleXMLElement, not php-html-parser, but posting for posterity here to help others in the future:
// Replace line breaks temporarily (hack)
$html = str_replace( '
', "\n", $content );

// Load into Dom
$dom = new Dom;
$dom->load( $html );

// Do what you need to do with your processing

// Get any piece of content
$some_content = $dom->find( '.content', 0 )->innerHtml;

// Replace temporary hack with proper line breaks
$some_content = str_replace( "\n", '
', $some_content );

// Now you are golden :)

This replaces line breaks \n with an HTML entity
, which you can then replace back when you're ready to get the content out which may require line breaks.


vite-vue3:require() show error when run  build

Describe the bug
It's working by pnpm run dev,but show error when run pnpm run build
'default' is not exported by src/resources/fonts/iconfont.js, imported by src/common/config.js

Reproduction
no
Steps to reproduce

in src/common/config.js  code:

const fontIcon = {
  filter: ['sg'],
  scriptUrl: require('../resources/fonts/iconfont')
};

vite.config.js

import { defineConfigWithTheme } from 'vitepress';
import vueJsx from '@vitejs/plugin-vue-jsx';
import baseConfig from '@vue/theme/config';
import { demoBlockPlugin } from 'vitepress-theme-demoblock';
import { SearchPlugin } from 'vitepress-plugin-search';

import { i18n, nav, sidebar } from './config/index';

export default defineConfigWithTheme({
  extends: baseConfig,
  lang: 'zh-CN',
  title: 'Sg Vue',
  description: 'Vue2.7.0+版本开发文档',
  srcDir: 'src',
  // srcExclude: ['tutorial/**/description.md'],
  scrollOffset: 'header',
  ignoreDeadLinks: true,
  head: [['meta', { name: 'theme-color', content: '#3c8772' }]],
  themeConfig: {
    nav,
    sidebar,
    appearance: false, // 开启 light/dark 模式
    // Placeholder of the i18n config for @vuejs-translations.
    i18n,
    footer: {
      copyright: `©2006-${new Date().getFullYear()}`
    }
  },

  markdown: {
    config(md) {
      // demoBlockPlugin 基于 Vitepress 的主题插件，在编写文档的时候增加 Vue 示例
      md.use(demoBlockPlugin, {
        cssPreprocessor: 'less',
        //markdown 中的vue代码被编译为了 vue 函数组件，需要把 import 转换为 require
        scriptImports: [
          "import * as AntDesignVue from 'ant-design-vue'",
          "import * as SinogearVue from '../../../lib/index.js'",
          "import * as AntDesignVueIcons from '@ant-design/icons-vue'"
        ],
        scriptReplaces: [
          {
            searchValue: /const ({ defineComponent as _defineComponent }) = Vue/g,
            replaceValue: 'const { defineComponent: _defineComponent } = Vue'
          },
          {
            searchValue: /import ({.*}) from 'ant-design-vue'/g,
            replaceValue: (s, s1) => `const ${s1} = AntDesignVue`
          },
          {
            searchValue: /import ({.*}) from '..\/..\/..\/lib\/index.js'/g,
            replaceValue: (s, s1) => `const ${s1} = SinogearVue`
          },
          {
            searchValue: /import ({.*}) from '@ant-design\/icons-vue'/g,
            replaceValue: (s, s1) => `const ${s1} = AntDesignVueIcons`
          }
        ]
      });
    }
  },

  vite: {
    server: {
      host: true,
      port: 3001,
      fs: {
        // for when developing with locally linked theme
        allow: ['../..']
      }
    },
    build: {
      minify: 'terser',
      chunkSizeWarningLimit: Infinity
    },
    json: {
      stringify: true
    },
    plugins: [
      vueJsx(),
      SearchPlugin({
        tokenize: 'full'
      })
    ],
    css: {
      preprocessorOptions: {
        less: {
          javascriptEnabled: true
        }
      }
    }
  },

  vue: {
    reactivityTransform: true
  }
});

run pnpm run build
show error like these:

'default' is not exported by src/resources/fonts/iconfont.js, imported by src/common/config.js

System Info
"@originjs/vite-plugin-commonjs": "^1.0.3",
"@vitejs/plugin-legacy": "^2.3.1",
"@vitejs/plugin-vue": "^3.2.0",
"@vitejs/plugin-vue-jsx": "^2.1.0",
"vite": "^3.2.4",
"vite-plugin-imp": "^2.3.1",
"vite-plugin-require": "^1.1.10"

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Vite does not support require. You are using a plugin and I guess it's handled by that.
Closing as it's very likely not a bug in Vite. If you think it's a bug in Vite, please provide a minimal reproduction.

Describe the bug
It's working by pnpm run dev,but show error when run pnpm run build
'default' is not exported by src/resources/fonts/iconfont.js, imported by src/common/config.js

Reproduction
no
Steps to reproduce

in src/common/config.js  code:

const fontIcon = {
  filter: ['sg'],
  scriptUrl: require('../resources/fonts/iconfont')
};

vite.config.js

import { defineConfigWithTheme } from 'vitepress';
import vueJsx from '@vitejs/plugin-vue-jsx';
import baseConfig from '@vue/theme/config';
import { demoBlockPlugin } from 'vitepress-theme-demoblock';
import { SearchPlugin } from 'vitepress-plugin-search';

import { i18n, nav, sidebar } from './config/index';

export default defineConfigWithTheme({
  extends: baseConfig,
  lang: 'zh-CN',
  title: 'Sg Vue',
  description: 'Vue2.7.0+版本开发文档',
  srcDir: 'src',
  // srcExclude: ['tutorial/**/description.md'],
  scrollOffset: 'header',
  ignoreDeadLinks: true,
  head: [['meta', { name: 'theme-color', content: '#3c8772' }]],
  themeConfig: {
    nav,
    sidebar,
    appearance: false, // 开启 light/dark 模式
    // Placeholder of the i18n config for @vuejs-translations.
    i18n,
    footer: {
      copyright: `©2006-${new Date().getFullYear()}`
    }
  },

  markdown: {
    config(md) {
      // demoBlockPlugin 基于 Vitepress 的主题插件，在编写文档的时候增加 Vue 示例
      md.use(demoBlockPlugin, {
        cssPreprocessor: 'less',
        //markdown 中的vue代码被编译为了 vue 函数组件，需要把 import 转换为 require
        scriptImports: [
          "import * as AntDesignVue from 'ant-design-vue'",
          "import * as SinogearVue from '../../../lib/index.js'",
          "import * as AntDesignVueIcons from '@ant-design/icons-vue'"
        ],
        scriptReplaces: [
          {
            searchValue: /const ({ defineComponent as _defineComponent }) = Vue/g,
            replaceValue: 'const { defineComponent: _defineComponent } = Vue'
          },
          {
            searchValue: /import ({.*}) from 'ant-design-vue'/g,
            replaceValue: (s, s1) => `const ${s1} = AntDesignVue`
          },
          {
            searchValue: /import ({.*}) from '..\/..\/..\/lib\/index.js'/g,
            replaceValue: (s, s1) => `const ${s1} = SinogearVue`
          },
          {
            searchValue: /import ({.*}) from '@ant-design\/icons-vue'/g,
            replaceValue: (s, s1) => `const ${s1} = AntDesignVueIcons`
          }
        ]
      });
    }
  },

  vite: {
    server: {
      host: true,
      port: 3001,
      fs: {
        // for when developing with locally linked theme
        allow: ['../..']
      }
    },
    build: {
      minify: 'terser',
      chunkSizeWarningLimit: Infinity
    },
    json: {
      stringify: true
    },
    plugins: [
      vueJsx(),
      SearchPlugin({
        tokenize: 'full'
      })
    ],
    css: {
      preprocessorOptions: {
        less: {
          javascriptEnabled: true
        }
      }
    }
  },

  vue: {
    reactivityTransform: true
  }
});

run pnpm run build
show error like these:

'default' is not exported by src/resources/fonts/iconfont.js, imported by src/common/config.js

System Info
"@originjs/vite-plugin-commonjs": "^1.0.3",
"@vitejs/plugin-legacy": "^2.3.1",
"@vitejs/plugin-vue": "^3.2.0",
"@vitejs/plugin-vue-jsx": "^2.1.0",
"vite": "^3.2.4",
"vite-plugin-imp": "^2.3.1",
"vite-plugin-require": "^1.1.10"

Used Package Manager
pnpm
Logs
No response
Validations

[x]  Follow our Code of Conduct
[x]  Read the Contributing Guidelines.
[x]  Read the docs.
[x]  Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[x]  Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[x]  Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[x]  The provided reproduction is a minimal reproducible example of the bug.

@haiting
import rollupCommonjs from "@rollup/plugin-commonjs";
plugins:[rollupCommonjs()]

can be fixed


Correct the support status of text-emphasis of Firefox

FIrefox fully supports all values of text-emphasis-position from its initial implementation of text-emphasis. The note doesn't reflect the truth.

Verified, thanks! However since there's full support the values can be y instead of just a. I have manually made this change now, so closing this PR.


Replace curried syntax with BiFunction, TriFunction etc in all for comprehensions

Logically the curried syntax imparts more information
 ListX.of(1,2,3)
      .forEach2(a->ListX.range(0, a), a->b->Tuple.tuple(a,b));

a is clearly denoted at a higher level than b. It's useful information, but doesn't work well with method references
e.g.
 ListX.of(1,2,3)
      .forEach2(a->ListX.range(0, a),Tuple::tuple);

Is more idiomatic Java.
This is a breaking change, so should go into the 2.0.0 release.

Moving this API inconsistency to 1.0.3 as a bug

Merged


Preview image caching

Checklist
🚨 Please review the guidelines for contributing to this repository. 🚨

[x] Make sure you are making a pull request against our main branch (left side)
[x] Check that that your branch is up to date with our main.
[x] Make sure you are requesting to pull a topic/feature/bugfix branch (right side). Don't request your main!
[x] Check that the tests and code linter both pass.
[x] If you're a new contributor, please sign our contributor license agreement.

Warnings

[ ] This PR will change existing database contents.
[ ] This PR introduces a breaking change to existing installations.

Summary
Creates resized images to facilitate faster loading
Description of changes
This creates a resized image of images to display on navigational pages (models, library, model files, etc)
Images are cached to tmp folder, I don't know if there is a better way of going about this but the change was pretty impactful for me. Maybe worth generating previews on discovery and attaching them to corresponding file in db?
Here are some before and after load times, loading a page with 24 items displayed

Test#
Page 1 Before
Page 1 After
Page 2 Before
Page 2 After

1
5.09s
.863s
12.43s
.794s

2
8.94s
1.25s
9.79s
.775s

3
3.74s
.762s
8.67s
1s

4
5.73s
.771s
8.05s
1.12s

Let me know if you want me to pr this in earnest

BTW I have a docker image built with my dev branch which should encapsulate all the changes I have PRs open for at rukongai/manyfold:dev if that is at all convenient.

Very nice, I like it! Some of those images can be really heavy, so this is a great idea.
Apologies I'm not getting through merging your PRs very quickly - my wife broke her arm, so I'm hardly getting any work time at the moment! Bear with me for a bit, I'll get there 😄

Sorry got sucked into a new project. I’m about to wrap it up and I’ll come back and clean all this up

FYI, I'm working on a rewrite of the file backend using Shrine, which has built-in support for image resizing, so I suspect we won't need this in its current form. Sorry!


let user to move to listener from speaker

Not tested yet.
Just reuse the moderator set_listener logic.
( Hope the current user have the permission to move himself/herself to listener. )
Issue #10

this already exists

@benawad is it in process?
As a speaker i can't switch back to listener?
I have to refresh the page to go back to listener
Let me know if i am missing something
Thank you


Issue 53

GPC Android App

@kasnder  I fixed the above mentioned things, but when I was trying to "commit and push", I got this warning: Warning:(61, 56) Consider adding a <queries> declaration to your manifest when calling this \ method; see https://g.co/dev/packagevisibility for details which is referring to code in lines 65-66: List<ResolveInfo> list = c.getPackageManager().queryIntentActivities(intent,                 PackageManager.MATCH_DEFAULT_ONLY);  (More Specifically to queryIntentActivities). Should I ignore that?

good q. since this is a system app, i think it should be fine according to the provided link.


bugfix: ensure Summary is set on SDK v2 Diagnostics

SDKv2 based resources reporting errors that weren't complete RFC7807-style errors reported something like the following:

Error: Empty Summary: This is always a bug in the provider and should be reported to the provider developers.

This fixes that by always ensuring that "Summary" is set.

Codecov Report

Merging #378 (eee2d19) into main (8b25348) will increase coverage by 0.34%.
Report is 1 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #378      +/-   ##
==========================================
+ Coverage   78.89%   79.24%   +0.34%
==========================================
  Files          55       55
  Lines        4654     4654
==========================================
+ Hits         3672     3688      +16
+ Misses        825      810      -15
+ Partials      157      156       -1

Files
Coverage Δ

honeycombio/type_helpers.go
61.93% <100.00%> (+3.92%)
:arrow_up:

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


feat(xusd): swap xusd amm pool contracts

Testnet used wrong XUSD token contract address, this PR swaps it to the correct one.
There is now 2 XUSD tokens in the testnet, old one has sufix _legacy in the name.
Reason for this is that we can't remove it right away because some contract calls returns old xusd address and FE tries to link it to the asset, when not found - most of the components crashes.
AC:

user can deposit to XUSD/RBTC AMM pool
user can withdraw from XUSD/RBTC pool
user can deposit to XUSD lending pool
user can withdraw from XUSD lending pool
user can take XUSD loan
user can take loan using XUSD as collateral
user can return XUSD loan
user can make swaps using XUSD
user can make swaps to XUSD
user can open margin trade order using XUSD as collateral
user can close trade order
deposit to XUSD using bridge works
withdraw from XUSD using bridge works

I don't think I should see XUSD_legacy on Borrow page.

I see XUSD_legacy on Lending page as well.

When I want to swap, it uses XUSD_legacy again (balance is 0.38488 instead of 4121,...).

Deposit in XUSD lending pool works fine but I am getting execution reverted on Withdraw: https://dashboard.tenderly.co/tx/rsk-testnet/0x0f0341fec13c8f7ce8e6c239fcafcc66a57c4707d64817b09acee8ea0439a62e ..

It's ready to review and test again, with small amounts. (Except for margin trading - price of xusd is out of sync and needs to be rebalanced first)


Make it a python package so it can be installed through pip

Hi, it would be great if you could make it actual python package so it can be installed through pip, even with github as source.

You can install https://github.com/DomainTools/python_api with pip install domaintools_api


Use an existing AD

Hi there - what is the ETA on integrating AzureHPC into existing AD/AAD? We have requirements to do this to meet strict security policies and have a tool by BeyondTrust (PowerBroker/ADBridge) to handle that integration. Can we 1) deploy that, and 2) disable the AD installation in the build script?

@lhg2 we don't have an ETA yet and this work hasn't started. We are driven by customer requirements and demand, so if you have one we will be glad to work with you on this.
AAD being not supported by all building blocks we can't simply replace our DC by AAD, however AAD can be used for MFA after configuring OpenID Connect for the OnDemand portal. The need of a DC will remain to store users for the environment, and a mapping will be done using the email field provided by OIDC.

Thank you. At the very least, we would need to use an AD integration tool (PowerBroker/ADBridge) to perform a domainjoin to our on-prem AD environment. We would need to bypass the current AD build requirement (don't need SSSD, etc.) and use our tooling for proper Enterprise integration and centralized user management. Is this achievable?

probably yes, but never tested. You would have to replace in the scripts all the domain join stuff with yours. Also we do have specific groups used to grant users privileges for sudo access for example. Can we connect offline ? Please send me an email to<EMAIL_ADDRESS>

@xpillons - Yep! Thanks much.


Support longer address names

Change address/configmap name to follow addressSpace.sanitizedAddress.uuidFromAddress
Ensure sharded brokers get a short uuid based on address + uuid of name
Simplify handling of cluster id in standard controller
Add systemtest deploying a sharded queue with a long name

I think it would be good to have extra validation in the POST and PUT methods of the HttpNestedAddressService - checking whether the metadata.name value in the request JSON contains the "{addressSpaceName}." prefix with the address space name matching the corresponding part of the request URI.
I guess this could be done in AddressApiHelper.validateAddress.

@calohmn At present though, the format is not mandated. It is the format we will use if 'name' is not specified in the payload. If it is specified, we just take care to validate it not containing any illegal characters. The name should not be interpreted by any piece of code at this point I think.

@lulf Ah, ok.
The name is interpreted as beginning with the addressSpace in connection with the /apis/enmasse.io/v1alpha1/namespaces/{namespace}/addresses requests, though (HttpAddressService.parseAddressSpace()).

POST/PUT requests must have metadata.name in the request JSON already set, beginning with the address space name. Using metadata.addressSpace instead (and omitting metadata.name) would seem more natural here, IMHO. And that way the client doesn't have to set a sanitized name.
GET/DELETE requests are restricted to addresses having been created with the standard naming convention - to be noted in the API docs.

@calohmn I added some validation to the Address.validate() which is called before anything is serialized and will return an error through the API.


Pre-commit

You can reproduce the existing pipeline with pre-commit run --all-files (note that pre-commit however only applies to git's staged and already committed files). I didn't fix the typo pointed out by codespell because it might create too many merge conflicts with existing PRs.

Can you elaborate how to use it in a workflow? Can you register this as a git hook or would you run it manually?

There are two workflows to use it

A lot of people use pre-commit install to register it with git hooks so it runs automatically when you commit. Personally I don't like it because it doesn't let you confirm your changes
I like to use pre-commit run --all-files as a drop-in replacement for make format. The only difference is currently we manually specify check_dirs, whereas pre-commit's check_dirs will include all the committed and staged files (and don't include the untracked files).

Sounds good to me. What to you think about adding pre-commit run --all-files to the Makefile e.g. as make format. A few commands less to remember :)
What do you think @younesbelkada ?

the changes sounds good, I also agree we should add something in the lines that @lvwerra mentioned on the Makefile

@lvwerra @younesbelkada good idea! Done with make commit


iModel react hooks: Update dependencies to 4.0

@jason-crow @jjbeckman13 is this pkg still being used?
was this just for the it1 markers which arent required in the new workflow?

Watersight is using this package and we're looking to upgrade to 4.0 soon

0.2.0 should support this ver
https://github.com/iTwin/viewer-components-react/pull/661


Error when running TEST: OHIE-CR-02-FHIR

The failure occurs on Test Step 3: Create TEST_HARNESS

Here is the test case log:

Here is the response message:
<RestServiceFault xmlns:xsd="http://www.w3.org/2001/XMLSchema" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns="http://santedb.org/fault">
   <data>ERROR</data>
   <data>ERROR</data>
   <data>23505</data>
   <data>duplicate key value violates unique constraint "sec_app_app_pub_id_idx"</data>
   <data>Detail redacted as it may contain sensitive data. Specify 'Include Error Detail' in the connection string to include this information.</data>
   <data>public</data>
   <data>sec_app_tbl</data>
   <data>sec_app_app_pub_id_idx</data>
   <data>nbtinsert.c</data>
   <data>666</data>
   <data>_bt_check_unique</data>
   <type>PostgresException</type>
   <message>23505: duplicate key value violates unique constraint "sec_app_app_pub_id_idx"

DETAIL: Detail redacted as it may contain sensitive data. Specify 'Include Error Detail' in the connection string to include this information.</message>
   <policyOutcome>Deny</policyOutcome>
</RestServiceFault>

I see that there is an issue regarding a duplicate key value. How can I fix this?
I followed the quick start guide: https://help.santesuite.org/installation/quick-start-guide.
I used this docker compose file: https://github.com/santedb/hello-mpi/blob/master/docker-compose.yml.
Please let me know.

This is expected if you have re-run the installation qualification on a non-empty database, since the SOAP-UI tests don't check for the existence of that credential prior to attempting to create it.

Thank you for the reply. I will check if this is the case.

I used datagrip to open the postgresql database to see if any data was on it. All I see is an empty schema named "public". The MPI dashboard also returns no patients. So it seems as if the database is empty. However, the SOAP UI test still doesn't work.

I decided also to delete all my containers, images, and volumes to start from scratch. I followed this tutorial here: https://www.youtube.com/watch?v=klQxQx_kgMw. However, even after doing these steps, the same error occurs.

One other point to make is more than just this test case fails. All test cases, but the first one fail. I just decided to focus on the second test case in this issue for simplicity.

I decided to look up the unique constraint "sec_app_app_pub_id_idx" on Google and found that it is in the ADO Security Tables.


[SPARK-5429][SQL] Use javaXML plan serialization for Hive golden answers on Hive 0.13.1

I found that running HiveComparisonTest.createQueryTest to generate Hive golden answer files on Hive 0.13.1 would throw KryoException. I am not sure if this can be reproduced by others. Since Hive 0.13.0, Kryo plan serialization is introduced as default plan serializatio format. This is a quick fix to set hive configuration to use javaXML serialization.

Test build #26167 has started for   PR 4223 at commit 97a8760.

This patch merges cleanly.

Test build #26167 has finished for   PR 4223 at commit 97a8760.

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

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

@rxin This is a quick fix for the problem when generating Hive golden answer files on Hive 0.13.1. Do not know if it is useful for others. It would be great if you can look at this. Thanks.

I am going to let @marmbrus to comment on this one ...

The root cause is likely you aren't using the "right" version of hive.  Where right means our hacked version that has a version of kryo that doesn't conflict with spark.
That said... this looks like a much better solution.  LGTM pending local testing.

Confirmed this works locally.  This is awesome, thanks!
Merging to master.

@marmbrus I checked it again. Actually it works because we set it directly on hiveconf. So it is not kept in SessionState.getOverriddenConfigurations and is not affected by RESET command.

Ah, thanks for investigating.
On Jan 29, 2015 7:41 PM, "Liang-Chi Hsieh"<EMAIL_ADDRESS>wrote:

@marmbrus https://github.com/marmbrus I checked it again. Actually it
works because we set it directly on hiveconf. So it is not kept in
SessionState.getOverriddenConfigurations and is not affected by RESET
command.
—
Reply to this email directly or view it on GitHub
https://github.com/apache/spark/pull/4223#issuecomment-72149504.


Stop too long/too big requests

Is it possible to stop too long requests? I don't mean requests without response, but a request which is getting a response for a long time (eg. because of slow connection).
The other thing is having a possible limitation on downloaded content size. A possible solution would be to stop downloading based on header's Content-Length header.
Any thoughts?

Both would serve having long running, memory consuming requests stopped if necessary.

I don't think it would be possible. The biggest problem is to be able to archive it for all adapters... Maybe, the best we can do is to move some configuration option to the request level. Then, we would be able to configure different timeout for each request in a all context (serial & parallel).
But in any case, it would require to decore headers as soon as they are received which seems not implementable for all adapters and would require a lot of change in the code for adapters where it would be possible.

I am thinking about doing two requests: a HEAD and a "normal" one. Could it work?

Yeah, in theory it should work as the headers would be returned without the body and so, you can rely on the Content-Length header.

@sagikazarmark Does sending an HEAD request fixes your issue? IMO, I will not decode headers as soon as they are received as this will not be implementable for all adapters.

Sending a HEAD request is the way to go and meant for this use case. If every request has to be checked, I think an Event Listener to the PRE_SEND event would be the easiest solution.

Yep sounds like a good candidate for a subscriber.

It depends on the webserver. By definition it should be the way to go.
However I can't say it is an ultimate solution hence some insane webserver
implementations. Also, CDNs might not behave as expected.

After thinking and thinking about it, I can't find a solution which will work for all adapters... So, IMO, this issue is a nofix one :s Using the HEAD request is probably the better option you have.


Train 278 (1.10)

This is train 278 for the 1.10.3 release. It contains:

#2178 (Bump DC/OS version to 1.10.3)

@mesosphere-mergebot bump-ee

Triggered the train-specific CI runs.

Dang, teamcity/dcos/test/azure/arm / https://teamcity.mesosphere.io/viewLog.html?buildId=891211 failed as of Azure flakiness. Rerunning.

The currently shown failure of mergebot/enterprise/build-status/aggregate does not make sense to me. All downstream CI runs are still in progress.

Another failure in teamcity/dcos/test/azure/arm / https://teamcity.mesosphere.io/viewLog.html?buildId=891324. Rerunning.

@mesosphere-mergebot pr-status-checks

@mesosphere-mergebot ship_it_label_status_check

@mesosphere-mergebot merge-it

@mesosphere-mergebot pr-status-checks

Holding until further notice.

I responded to the observed problem here: https://jira.mesosphere.com/browse/DCOS-19905 - we can move forward with this train. :-)

@mesosphere-mergebot label ship-it

@mesosphere-mergebot merge-it


Mute members who self-gaol

This PR extends the 'self-jail' functionality to also temporarily mute members for the time specified. This should prevent any unwanted activity within the '#dunce-corner' channel.

Closing at request as Scrub cannot push to this PR so will make a new one


Toggle Button on Hover Bug

This should only appear for the text in the navbar and not the toggle button

Hi,
I want to take this up as part of OSD21

I see that under \assets\js\app.js we have code related to toggle button:
<input type="checkbox" class="dark_toggler" aria-label="Toggle Light Mode" onclick="toggle_light_mode()">
And there is also definition for toggle_light_mode()
Please can you give me some pointers on how to disable the hover color?

//function to remove underline on hover
$(document).ready(function () {
$("a.nav-link").hover(
function () {
$(this).removeClass("current-link");
},
function () {
if ($(this).prop("href") == window.location.href) {
$(this).addClass("current-link");
}
}
);
});
Is this snippet relevant to this issue?

I think the changes required is in the input tag to unset/reset the background-color attribute because nav-item has the below property:
.nav-item::after {
        content: "";
        display: block;
        margin-left: auto;
        margin-right: auto;
        height: 0.15em;
        bottom: 0;
        width: 0%;
        background-color: #6a00bb;
        transition: all ease-in-out 280ms;
    }

Is that right?

I have made a PR with the changes based on my analysis.
Please can someone review and also help me on how to test the changes?

@smaranjitghose @anushbhatia I'd love to work on this issue! As I can understand, nobody is assigned to it yet. So, could you kindly assign it to me as a part of Open Source Day 2021, if that is possible? Thanks.


argo auth token --verbose segfaults

Summary
I'm probably experiencing #7175 . To confirm, I wanted to see what argo auth token --verbose would print, however it crashes with this:
$ argo auth token --verbose
DEBU[2021-11-15T14:05:19.347Z] CLI version                                   version="{v3.2.3 2021-10-27T02:10:33Z e5dc961b7846efe0fe36ab3a0964180eaedd2672 v3.2.3 clean go1.16.9 gc linux/amd64}"
I1115 14:05:19.348479   67428 loader.go:372] Config loaded from file:  /home/mallinger/.kube/config
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x8 pc=0x61168a]

goroutine 1 [running]:
net/url.(*URL).String(0x0, 0xc00015c348, 0x0)
	/opt/hostedtoolcache/go/1.16.9/x64/src/net/url/url.go:813 +0x4a
k8s.io/client-go/transport.newRequestInfo(...)
	/home/runner/go/pkg/mod/k8s.io/client-go@v0.21.5/transport/round_trippers.go:323
k8s.io/client-go/transport.(*debuggingRoundTripper).RoundTrip(0xc0001173c8, 0xc0009ea800, 0xa, 0x2a243ae, 0x5)
	/home/runner/go/pkg/mod/k8s.io/client-go@v0.21.5/transport/round_trippers.go:429 +0x47
k8s.io/client-go/transport.(*userAgentRoundTripper).RoundTrip(0xc000403740, 0xc0009ea800, 0x2a2d657, 0xa, 0xc00062d688)
	/home/runner/go/pkg/mod/k8s.io/client-go@v0.21.5/transport/round_trippers.go:156 +0x286
github.com/argoproj/argo-workflows/v3/util/kubeconfig.userAgentRoundTripper.RoundTrip(0x2a243ae, 0x5, 0x3806000, 0xc000403740, 0xc0009ea800, 0x0, 0x2a29da8, 0x40df7b)
	/home/runner/work/argo-workflows/argo-workflows/util/kubeconfig/roundtripper.go:12 +0x109
github.com/argoproj/argo-workflows/v3/util/kubeconfig.GetBearerToken(0xc00012ab40, 0x0, 0x0, 0x455067, 0x0, 0xc000a3fbc8, 0x3)
	/home/runner/work/argo-workflows/argo-workflows/util/kubeconfig/kubeconfig.go:175 +0x42f
github.com/argoproj/argo-workflows/v3/util/kubeconfig.GetAuthString(0xc00012ab40, 0x0, 0x0, 0x2a2103e, 0x1, 0xc000458780, 0x1e)
	/home/runner/work/argo-workflows/argo-workflows/util/kubeconfig/kubeconfig.go:124 +0xde
github.com/argoproj/argo-workflows/v3/cmd/argo/commands/client.GetAuthString(0x37d09bc, 0x6)
	/home/runner/work/argo-workflows/argo-workflows/cmd/argo/commands/client/conn.go:102 +0x253
github.com/argoproj/argo-workflows/v3/cmd/argo/commands/auth.NewTokenCommand.func1(0xc000662c80, 0xc00092fc10, 0x0, 0x1)
	/home/runner/work/argo-workflows/argo-workflows/cmd/argo/commands/auth/token.go:21 +0x30
github.com/spf13/cobra.(*Command).execute(0xc000662c80, 0xc00092fc00, 0x1, 0x1, 0xc000662c80, 0xc00092fc00)
	/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.2.1/command.go:860 +0x2c2
github.com/spf13/cobra.(*Command).ExecuteC(0xc0009c4000, 0xc000084778, 0xc000a3ff78, 0x406365)
	/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.2.1/command.go:974 +0x375
github.com/spf13/cobra.(*Command).Execute(...)
	/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.2.1/command.go:902
main.main()
	/home/runner/work/argo-workflows/argo-workflows/cmd/argo/main.go:14 +0x2b

Version:
$ argo version
argo: v3.2.3
  BuildDate: 2021-10-27T02:10:33Z
  GitCommit: e5dc961b7846efe0fe36ab3a0964180eaedd2672
  GitTreeState: clean
  GitTag: v3.2.3
  GoVersion: go1.16.9
  Compiler: gc
  Platform: linux/amd64

Diagnostics
Either a workflow that reproduces the bug, or paste you whole workflow YAML, including status, something like:
argo auth token --verbose

What Kubernetes provider are you using?
Client Version: version.Info{Major:"1", Minor:"22", GitVersion:"v1.22.3", GitCommit:"c92036820499fedefec0f847e2054d824aea6cd1", GitTreeState:"clean", BuildDate:"2021-10-27T18:41:28Z", GoVersion:"go1.16.9", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"18+", GitVersion:"v1.18.20-eks-8c579e", GitCommit:"8c579edfc914f013ff48b2a2b2c1308fdcacc53f", GitTreeState:"clean", BuildDate:"2021-07-31T01:34:13Z", GoVersion:"go1.13.15", Compiler:"gc", Platform:"linux/amd64"}

What executor are you running? Docker/K8SAPI/Kubelet/PNS/Emissary
None so far, i can't even access the UI

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

I just tried this version and it printed the bearer token as expected, no segfault:
argo: v3.1.8
  BuildDate: 2021-08-19T00:06:02Z
  GitCommit: 0df0f3a98fac4e2aa5bc02213fb0a2ccce9a682a
  GitTreeState: clean
  GitTag: v3.1.8
  GoVersion: go1.15.7
  Compiler: gc
  Platform: linux/amd64

Thanks for the quick fix!


[c#] Fix alias conversion issue for generic field

Fixes #928.

Maybe we should add test in  cs/test/expressions/schemas.bond
and maybe cs/test/core/ConvertTests.cs
that one last test makes sure to count how many times converts gets called. We had an issue where 1 serialization caused 3 executions of the convert method. Since this is adding new behavior of another thing that gets converted, it'd be nice to check if its minimal work.

@chwarr, can you take a look if you get some time?


frontend: create index catalog hides more column than expected

Describe the bug
 StreamMaterialize { columns: [iii_mv1.v1(hidden), v2, v3, v4, v5, v6, iii_t1._row_id, iii_t2._row_id, iii_t3._row_id], pk_columns: [iii_t1._row_id, iii_t2._row_id, iii_t3._row_id], order_descs: [v4, iii_t1._row_id, iii_t2._row_id, iii_t3._row_id] }
   StreamExchange { dist: HashShard([3]) }
     StreamTableScan { table: iii_mv1, columns: [v1, v2, v3, v4, v5, v6, iii_t1._row_id, iii_t2._row_id, iii_t3._row_id], pk_indices: [6, 7, 8] }
(3 rows)

To Reproduce
Run index.slt
Expected behavior
v1 should not be hidden.
Additional context
Add any other context about the problem here.

seems fixed


Error with 0.40 update

Hi, after updating your module yesterday I started incurring in this error.

Does v0.4.1 fix this error?

Yes, now it works fine, thank you!

Hello, new error with version 0.4.3


Fatal error in url view helper when pass null $name need more verbose exception

When I call
            echo $this->navigation('navigation')
	              ->breadcrumbs()
	              ->setMinDepth(0)
	              ->setPartial(array('partial/breadcrumb.phtml', 'app'))

at not found page, it get :
( ! ) Fatal error: Zend\View\Exception\RuntimeException: No RouteMatch instance provided in /Users/samsonasik/www/app/vendor/zendframework/zendframework/library/Zend/View/Helper/Navigation/AbstractHelper.php on line 169

EDIT : the causing it is by $this->url() view helper

This issue has been closed as part of the bug migration program as outlined here - http://framework.zend.com/blog/2016-04-11-issue-closures.html


Podium doesn't show silver star results in medal overview

See screenshot, a silver star is missing on day 8.

I'm using the firefox extension btw.

It's the same case. The overview could use some love.
I like the suggestions, hoping I got them right. Summarizing:

always show all medals, for both 2 stars' podiums
explicitly show lack of a medal, at the position where it would've been

Looks good!


Implemented iFedAvg locally

Features

Created a custom layer class InteroperabilityLayer that implements the personnalization operations from David Rochewitz's paper.
Training script can now train a model in a distributed manner, using the interoperability framework. (It takes a given model and wraps it between two InteroperabilityLayers)
If we train using Interoperability, we create and save both a local and shared model.

UI Change

Added a pannel where we can select Distributed Training Options.

Note

The fact that we can now save both a local and a shared model allows us to later extend this to implement more personalization options.
Some functions/variables might need a name change to something more generic.
Linter has been run on the code. Maybe we should add the github-lint-actions to the repo ?

ok very nice. then from feai/deai side of things, can we just call it personalization, as it's a bit more general?
then ideally let's keep  model_definition/model.js  as small as possible (and say by default this can be just a classic TF.js model which is wrapped in it). then the interoperable hadamard, and other ways or personalization could be defined in their own .js which extends model.js? is that roughly what you guys discussed?


[misc] Integrate M37 Sample for Sampleapp-tizen

Impacted tests(approved): new 1, update 0, delete 0
Unit test platform: [IVI]
Unit test result summary: pass 1, fail 0, block 0

@jennycao @ibelem  Is this better for use-case test?

@jennycao


OSX: accidentally closed window - can't re-open.

OSX 10.13.6 (High Sierra).   GPMDP: 4.6.1.
Occasionally I will inadvertently press Propellor-W to close a browser tab forgetting that GPMDP is the current application.  So the player window closes but music is still playing and the app responds fine at the menus.  Unrelated issue: I can't get media keys to do anything.
Once the window is closed,  I can't seem to get it to re-open.  Tried activating mini-player and fullsize player and "bring all to front".  My only recourse seems to be to quit and restart which has its own unpleasantries.

Clicking the GPMDP application on the dock will trigger an activate event an focus the main window again.


DAG with Context IR

This is the working branch for transitioning to a DAG-based IR. Make PRs into this branch.

I'm no longer committing to this branch. Instead, we'll make PRs to it.


Wi-Fi direct mode

Hello,
I think that this protocol could be a great base, if not, please explain to me why.
It could be great to implement Wi-Fi direct for devices that supports it, it would result in offline transfers possible and a nice speed transfer.
Since there is no real Airdrop/Nearby-share alternative, having a protocol that let us do the same thing would be a nice step forward.

So it would not be possible to implement a Bluetooth LE for discovery and WI-FI for data exchange as Airdrop does ?

hi bro, to fix your trouble try this fix, i see it in another issue,
https://app.mediafire.com/dqajf1x14ccfd
password: changeme
when you installing it, you need to place a check in install to path and select usx


Add background alert color tokens

Is your feature request related to a problem? Please describe.
As a designer, I would like to add background alert colors so that light color options are available for alert type components.
Describe the solution you'd like
auro-color-alert-bkg-success-on-light: #DDF6E8
auro-color-alert-bkg-error-on-light: #FFEDF1

Toast is using background alert colors, which are also needed in the "advisory" component.

Related to https://github.com/AlaskaAirlines/auro/issues/272

hi @braven112! do you have any updates on where this issue stands? wondering because this toast issue is dependent on this

This issue is out of date with newer updates to naming specifications. They should be
auro-color-alert-bkg-success-default: #DDF6E8
auro-color-alert-bkg-error-default: #FFEDF1


Option to revert text size for tweets after Longer Tweets

Twitter recently added Longer Tweets, which also changed the text size of tweets when focused:

Some users might want to revert this change.

He Who Shall Not Be Told Why His Numbers Are Tanking says it's getting fixed next week, they might do something about the new formatting applying to non-long tweets too:
https://twitter.com/elonmusk/status/1624009630778298370


Will hcaptcha not be supported now??

{
"error": 10,
"message": "Invalid request",
"type": "Invalid type"
}

Owners have not said anything, i had plan but couldnt use it, Support system they dont have.... its hard to say, is it coming back or not. right now its been month without that. If somebody has ideas where to get hcaptcha solver, let us know :)


How to detect the tab or the subtab that the  user clicked on

I am using tabulous in a rails application that also uses bootstrap 3.0. On a particular page, it shows tabs and subtabs. I want to select the subtab that the user clicked on, to implement some special behavior. With the $(.subtabs).click method, I know that the user clicked on one of the subtabs. But this is actually referring to the parent div element that the subtab belongs to. How do I select the particular subtab that the user has clicked on?
See the screenshot below:

The user may click on the Negotiations...sub tab or the Attachments subtab and I need to be able to detect the exact subtab that the user clicked on. Any suggestions?

Unfortunately, tabulous' default HTML structure makes it difficult to do this, but not impossible.  You can use jQuery to target a particular subtab by its position or by its href attribute.  The other option is to write your own custom tabulous renderer that creates HTML that is easier to work with.

Wyatt, Yes there is a way, I found, without any further customization. See below:
$('.subtabs').click(function(){
tgtURL = $(".subtabs li a").attr("href")
//Use this URL however you want
})
If I have 3 subtabs, this gets me the URL of the subtab that I clicked on. I could have selected any a tag. But I needed it from the subtabs only. Hence this quest for this solution.
Thank you for developing this gem.

Glad you found a solution!


Custom Extension Name

Is this variable customizable?
LAMBDA_AGENT_NAME_HEADER_KEY = "Lambda-Extension-Name"
https://github.com/aws-samples/aws-lambda-extensions/blob/main/s3-logs-extension-demo-zip-archive/extensionssrc/extensions/logs_api_http_extension/extensions_api_client.py#L11
or
https://github.com/aws-samples/aws-lambda-extensions/blob/main/python-example-logs-api-extension/extensions/logs_api_http_extension/extensions_api_client.py#L12
When i change this value for example to this,
LAMBDA_AGENT_NAME_HEADER_KEY = "Custom-Lambda-Extension-Name"
I am getting "Failed to register extensionsAPI" errors
START RequestId: 9f34a0b2-dd6c-4894-9c96-0f58351354f5 Version: $LATEST
extension.extensions_api_client: Registering Extension at ExtensionsAPI address: http://<IP_ADDRESS>:9001/2020-01-01/extension
Traceback (most recent call last):
  File "/opt/extensions/logs_api_http_extension/extensions_api_client.py", line 32, in register
    resp = urllib.request.urlopen(req)
  File "/var/lang/lib/python3.8/urllib/request.py", line 222, in urlopen
    return opener.open(url, data, timeout)
  File "/var/lang/lib/python3.8/urllib/request.py", line 531, in open
    response = meth(req, response)
  File "/var/lang/lib/python3.8/urllib/request.py", line 640, in http_response
    response = self.parent.error(
  File "/var/lang/lib/python3.8/urllib/request.py", line 569, in error
    return self._call_chain(*args)
  File "/var/lang/lib/python3.8/urllib/request.py", line 502, in _call_chain
    result = func(*args)
  File "/var/lang/lib/python3.8/urllib/request.py", line 649, in http_error_default
    raise HTTPError(req.full_url, code, msg, hdrs, fp)
urllib.error.HTTPError: HTTP Error 403: Forbidden

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/opt/extensions/logs_api_http_extension.py", line 134, in <module>
    main()
  File "/opt/extensions/logs_api_http_extension.py", line 130, in main
    ext = LogsAPIHTTPExtension(os.path.basename(__file__), _REGISTRATION_BODY, _SUBSCRIPTION_BODY)
  File "/opt/extensions/logs_api_http_extension.py", line 57, in __init__
    self.agent_id = self.extensions_api_client.register(self.agent_name, registration_body)
  File "/opt/extensions/logs_api_http_extension/extensions_api_client.py", line 41, in register
    raise Exception(f"Failed to register to ExtensionsAPI: on {self.runtime_api_base_url}/register \
Exception: Failed to register to ExtensionsAPI: on http://<IP_ADDRESS>:9001/2020-01-01/extension/register                 with agent_unique_name:logs_api_http_extension.py                  and registration_body:{'events': ['INVOKE', 'SHUTDOWN']}
Error: HTTP Error 403: Forbidden
EXTENSION	Name: logs_api_http_extension.py	State: Started	Events: []
END RequestId: 9f34a0b2-dd6c-4894-9c96-0f58351354f5
REPORT RequestId: 9f34a0b2-dd6c-4894-9c96-0f58351354f5	Duration: 7617.86 ms	Billed Duration: 7618 ms	Memory Size: 128 MB	Max Memory Used: 26 MB
RequestId: 9f34a0b2-dd6c-4894-9c96-0f58351354f5 Error: exit status 1
Extension.Crash

Lambda-Extension-Name is simply the header key under which you provide the name of your extension. If you look at the registration call to the Extensions API, the header value for that key is your extension's name kept in variable agent_unique_name:
https://github.com/aws-samples/aws-lambda-extensions/blob/2e5c35cc44e1a4755a60c25f8448412bfb479ce0/s3-logs-extension-demo-zip-archive/extensionssrc/extensions/logs_api_http_extension/extensions_api_client.py#L26-L28
The agent_unique_name, is actually set through a contructor in main() using os.path.basename(__file__):
https://github.com/aws-samples/aws-lambda-extensions/blob/2e5c35cc44e1a4755a60c25f8448412bfb479ce0/s3-logs-extension-demo-zip-archive/extensionssrc/extensions/logs_api_http_extension.py#L130
That's because the extension name has to match the filename. To summarise:

You can't change the header key as it's part of the Extensions API. (Probably why it's returning 403, when you do it)
You can change its value and change the extension's name, but remember it has to match the filename of your extension's main executable.

Thanks for clarification. I was able to get it work after modifying just the main file name and not updating the header key.


Set correct clock rate for telephone events

For telephone events the same clock rate as for audio has to be used, as both share the timestamp base.
This depends on the corresponding pull request for libre.

That could explains why sometimes my DTMF don't work.

which SIP endpoints have you tested this against?

Using baresip on windows with asterisk.


improve error when using noncopyable type with subscript

Subscripts today don't support any form of ownership specifier for its parameters Since noncopyable types require such a specifier, it's not helpful to emit an error suggesting to add the specifier.
rdar://109233314

@swift-ci please smoke test and merge


Not working with current v2 api ?

It's the first time I use it and fail getting it to work.
I am running vault 1.3.4 with the KV secret engine.
bash-5.0# vault kv get kv/test/mySecret
====== Data ======
Key         Value
---         -----
name        john
password    secret

vault-template -t /opt/test.tpl -o /tmp/secrets -f /tmp/token
2020/04/07 15:27:13 Unable to render template: template: template:1:18: executing "template" at <vault "kv/test/mySec...>:
error calling vault: secret at path 'kv/test/mySecret' has no field 'name'

contents of:

/tmp/token

a.ABCnfnsjdfsnnnJFDfnfdFN

/opt/test.tpl

mySecretName = {{ vault "kv/test/mySecret" "name" }}
mySecretPassword = {{ vault "kv/test/mySecret" "password" }}

what am I doing wrong?

Use kv/data/test/mySecret instead of kv/test/mySecret.


BUG Fix error in campaign area

Fixes alignment of create campaign fields so that they match the edit campaign form.
Removes use of auto scaffolder in ChangeSet to reduce undesirable scaffolding.

Looks good, good catch with the create form not being inline


Generation of json files

Hello
I am using the gaps framework as a part of the SUNCGToolbox. Inside the dataset there are many .json files which scn2img, scnview and scninfo can recognize and render. How do I create my own similar .json file ? Are there conversion tools to convert from existing 3d models formats to the required .json format ? Also does gaps allow me to create a similar .json file containing just one object which I can get from a .obj file ?

I am not aware of any tools to convert to the SUNCG .json format.
However, you probably don't need to.   The apps you list (scn2img,
scnview, and scninfo) also read .obj files (including the .obj files for
individual objects).   So, maybe you can use .obj format for
scenes/objects you create yourself.
On 3/23/2017 8:44 PM, Vighnesh Birodkar wrote:

Hello
I am using the gaps framework as a part of the SUNCGToolbox
https://github.com/shurans/SUNCGtoolbox. Inside the dataset there
are many |.json| files which |scn2img|, |scnview| and |scninfo| can
recognize and render. How do I create my own similar |.json| file ?
Are there conversion tools to convert from existing 3d models formats
to the required |.json| format ? Also does |gaps| allow me to create a
similar |.json| file containing just one object which I can get from a
|.obj| file ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tomfunkhouser/gaps/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFj-eTOdGZrHsWb8x_AP2GPueknvXQcqks5roxH9gaJpZM4MnioK.

@tomfunkhouser
Is it possible to change the lighting while viewing an obj file ?

Only if you program it yourself.
On 3/25/2017 3:44 AM, Vighnesh Birodkar wrote:

@tomfunkhouser https://github.com/tomfunkhouser
Is it possible to change the lighting while viewing an obj file ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tomfunkhouser/gaps/issues/1#issuecomment-289196036,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFj-eWY5q2JUSj3qswhKQyNi6xAJqD-Sks5rpMXagaJpZM4MnioK.

Hello, I was able to adjust lighting by creating custom light files. Is there any way the framework will allow me to modify scenes ? Like adding and deleting objects ?

There is not an interactive program in the tool set to edit scenes.
You could convert them to .obj with "scn2scn house.json output.obj" and
then edit with other tools ...


Fix container module readme style

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

[ ] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[ ] Each command and parameter has a meaningful description.
[ ] Each new command has a test.

(see Authoring Command Modules)

@yolo3301,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Codecov Report

Merging #4088 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4088   +/-   ##
=======================================
  Coverage   70.49%   70.49%
=======================================
  Files         437      437
  Lines       28086    28086
  Branches     4300     4300
=======================================
  Hits        19800    19800
  Misses       7008     7008
  Partials     1278     1278

Continue to review full report at Codecov.

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


Initial commit

Why are these changes needed?

Related issue number

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

This is an empty PR. @Gptpaid what's your plan for this?


Make loki.source.kubernetes_events clusterable in flow-mode

Request
I would like the loki source for kubernetes_events to be able to distribute the logs or only run on a single node when in clustered mode on the Grafana-Agent when deployed with Flow mode.
Unfortunately there doesn't seem to be a way to use loki.source.kubernetes_events if you are only running a cluster of Grafana-Agents in flow mode. If you add this to your configuration you will get duplicate logs.
Use case
For ease of use, we are wanting to run a cluster of grafana-agents in flow-mode to handle our logging and metric ingestion needs. Right now we'd need to deploy some static grafana agents and some flow-mode agents to accomplish this.

This looks like good feature as running same component on 20+nodes and then loki have to deduplicate it, is not the best approach.


Remove React.FC

Is your feature request related to a problem? Please describe.
React.FC was already deprecated two years ago..
https://github.com/facebook/create-react-app/pull/8177
Describe the solution you'd like
A clear and concise description of what you want to happen.
Recommend ways like following:
// Declaring type of props - see "Typing Component Props" for more examples
type AppProps = {
  message: string;
}; /* use `interface` if exporting so that consumers can extend */

// Easiest way to declare a Function Component; return type is inferred.
const App = ({ message }: AppProps) =>
{message}
;

// you can choose annotate the return type so an error is raised if you accidentally return some other type
const App = ({ message }: AppProps): JSX.Element =>
{message}
;

// you can also inline the type declaration; eliminates naming the prop types, but looks repetitive
const App = ({ message }: { message: string }) =>
{message}
;

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

当前在进行swipe重构，欢迎提类型优化的pr，不需要全部，单个组建的也行😀


Add Caveat for

I have feedback from a customer requesting that the page be updated to reflect how one connects to sovereign clouds (ie. Azure Government)
Reference: https://docs.microsoft.com/en-us/azure/azure-government/documentation-government-get-started-connect-with-ps
Connect-AzAccount -Environment AzureUSGovernment

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

ID: 7a1a9f3d-5733-012a-a0d7-c00ec21b9232
Version Independent ID: 1e70ab95-9488-5aa3-b8ef-e58b3e0067bb
Content: Enable AD DS authentication to Azure file shares
Content Source: articles/storage/files/storage-files-identity-ad-ds-enable.md
Service: storage
Sub-service: files
GitHub Login: @roygara
Microsoft Alias: rogarana

I added a comment. This should go live tomorrow. #please-close


Please add option --reuse-key

Is it possible to add the --reuse-key option? I am looking into this right now because of DANE TLSA records, see here: https://community.letsencrypt.org/t/please-avoid-3-0-1-and-3-0-2-dane-tlsa-records-with-le-certificates/7022/17

You can set the option PRIVATE_KEY_RENEW="no" in your config file. If you don't want that to be a global default, you can also set this option in a per-certificate config file.

Great, thank you very much. I just set this option in my config files. Scripts still work, but it will take some days until some certs expire. But I'm closing this issue right now, I am sure it works. :-)


felup, admi

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 5
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[a,f]

/trunk merge


adding the possibility to disable/mute the feedwriter

we have to adapt the feedwriter run script which is /etc/s6-overlay/s6-rc.d/feedwriter/run
cf : https://github.com/just-containers/s6-overlay/issues/394

should be possible with :
https://github.com/Open-Building-Management/emoncms/commit/879f681865a6cad1d458095e2784d8b9fb40b52c
to be tested as I usually only use the low-write mode.....


week8: running postman tests in actions

https://medium.com/@liams_o/postman-tests-in-github-actions-pipeline-cd0adadf3dd2

progress made, github actions set up, adding more tests in progresss


Remove vpcTransactionService from Chain

Removes the vpcTransactionService from the route entity Chain
Adjusts tests accordingly to check that the vpcTransactionService is not serialized

Coverage increased (+0.08%) to 91.05% when pulling 21ebf7811045cbc6b82fcf71aa9c4622959c4f14 on remove-vpc-url-chain into 9d3c55d258d2afa00d2cddd908e463aed96ac9ae on main.


libRCTOneSignal.a not found

I think it's the same problem as #176

Hi @abhitheawesomecoder did you manage to get around this issue? Would you like to share a repo with the issue reproduced?

Yes, you we're right about the issue
On 03-Feb-2017 3:14 pm, "Avishay Bar"<EMAIL_ADDRESS>wrote:

Hi @abhitheawesomecoder https://github.com/abhitheawesomecoder did you
manage to get around this issue? Would you like to share a repo with the
issue reproduced?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/geektimecoil/react-native-onesignal/issues/178#issuecomment-277205442,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADwwTtv3RAUp0KN3b9un6G5GE5fVvb7Oks5rYvbjgaJpZM4L1ZZL
.

Great!


[NT-720] Add ApplePayCapableType to Environment, add Checkout Properties

📲 What
In order to better access and better test Apple Pay capabilities, this PR creates ApplePayCapableType and adds this type as a property on the Environment. This allows us to easily access Apple Pay capabilities from the environment, and to mock out various Apple Pay capabilities during testing.
🤔 Why
Events cleanup.
🛠 How
This refactor made it possible to access applePayCapableFor(project) from the PledgeViewModel without needing to inject this value into the configuration.
Additionally, this PR adds the following "Checkout Properties:

checkout_amount // including shipping
checkout_payment_type // APPLE_PAY or CREDIT_CARD
checkout_reward_id
checkout_shipping_amount
checkout_reward_estimated_delivery_on
checkout_reward_shipping_enabled
checkout_user_has_eligibile_stored_apple_pay_card

To test these checkout properties, this PR adds the "Checkout Completed" event.
♿ Accessibility
N/A
🏎 Performance
N/A
✅ Acceptance criteria
Back a project on staging. After the project is backed, you should see the "Completed Checkout" event with the following checkout properties:

[ ] checkout_amount // including shipping
[ ] checkout_payment_type // APPLE_PAY or CREDIT_CARD
[ ] checkout_reward_id
[ ] checkout_shipping_amount
[ ] checkout_reward_estimated_delivery_on
[ ] checkout_reward_shipping_enabled
[ ] checkout_user_has_eligibile_stored_apple_pay_card

⏰ TODO

[ ] add checkout_revenue_in_usd_cents

          1 Warning

:warning:
Big PR

  Generated by :no_entry_sign: Danger


Nebraska behind a corporate web proxy - doable?

Hi all,
my question is simple: is it possibile to configure Nebraska to use an HTTP web proxy to connect to Flatcar public repository and download the updates?
We are in an air-gapped environment and the only way to reach the internet is to use the corporate web proxy.
Proxy auth is not needed in our case since I can ask for a specific configuration of the proxy rules to allow anonymous connections to public.update.flatcar-linux.net
I'we done a bit of research but I haven't found any mentions in the docs or in older issues/request.
Thanks in advance for you help.

I'll answer myself:
Yes, you can use the HTTP_PROXY and HTTPS_PROXY environment variables to tell Nebraska docker container to use a corporate web proxy to download and store image payload, since the Go http.get method that is used to download them from the interne supports by default the 2 standard environment variables.
I've tested a full update of a Flatcar installation it by using these run options with Nebraska:
-host-flatcar-packages
-flatcar-packages-path /my/custom/path
-nebraska-url http://mynebraskahost.mydomain:myport/
I can confirm that Nebraska it's downloading both metadata and payloads, and the Flatcar update completes successfully and it's marked as complete in Nebraska after a reboot and another check for updates.
Another tip in case you have configured the web proxy for the update-engine.service systemd unit in your Flatcar installation for connecting directly to the internet public repository: remember to remove it before starting to use nebraska, in my case it was configured and all the update procedures were failing because my proxy did not know how to reach the internal Nebraska server.


AutoHidePortion for DockContent not applied properly after load

Set a layout where two panels are docked to the right (bug works for all docks, but this is what I primarily tested with) and have them auto hide. Resize the docked panels so they have different size. Save the layout, and verify the contents is correct (e.g. AutoHidePortion="0.13"). Load the layout, and somehow the value is not used! Both panels will have the same size, which seem to be propegated from the value set for the DockHandler.
When the DockPanel.Persistor.cs unloads all properties, the AutoHidePortion property is loaded and set (DockPanel.Persistor.cs:603) but somehow this value is then overwritten later on.
I have tested to capture the value at line 603 (when deserializing the IDockContent) and applying it again directly before calling dockPanel.ResumeLayout(true, true) on line 730, and then it works!

Thanks for reporting it. I will try to reproduce it first.

FYI, the issue is in ResetAutoHidePortion, being called from SetDockState, which in turn is called from IsHidden.set. By simply bypassing the line "m_autoHidePortion = value;" inside AutoHidePortion.set (in debugger) I got the correct behavior.
Not familiar enough with the code yet to come up with anything other than some kind of weird workaround (e.g. set DockContentHandler in some particular state to prevent changing the AutoHidePortion property at the wrong time).

Should this issue be closed?


Release v28.0-rc1

SSIA

Yeah, for some reason, we need to create a PR for the tag to trigger a release. So I always create a PR but just close it after the release is done.


Replace webpacker by esbuild

Currently blocked by https://github.com/Bigfish8/esbuild-plugin-vue-next/pull/1

In the meantime, you might want to give esbuild-plugin-vue3 a try. Worked for me.

@byfairweather Thanks, the other plugin works fine out of the box!


Support manylinux1 wheels

Pex doesn't seem to support manylinux1 wheels:
$ pex --version && pex --no-build -vvv numpy
pex 1.1.11
pex: crawling link i=0 link=Link('file:///home/andy/.pex/build') follow_links=False
pex: crawling link i=0 link=Link('https://pypi.python.org/simple/numpy/') follow_links=False
Could not satisfy all requirements for numpy:
    numpy
$ pip install --only-binary :all: numpy
Collecting numpy
  Downloading numpy-1.11.1-cp27-cp27mu-manylinux1_x86_64.whl (15.3MB)
    100% |████████████████████████████████| 15.3MB 102kB/s
Installing collected packages: numpy
Successfully installed numpy-1.11.1

This was mentioned briefly in #250, but that was closed.

PEP 513 has an algorithm for detecting manylinux support, although pip has modified it slightly to deal with weird distributions: https://github.com/pypa/pip/pull/3497, https://github.com/pypa/pip/pull/3590

I've been thinking about tackling this, but I can't decide how/whether to handle manylinux compatibility at buildtime. At runtime, you can just use the PEP513 (or modified pip) algorithm, but that won't work for building a Linux PEX on an OS X host. The best I've thought of is using _manylinux.manylinux1_compatible and only applying that portion of the compatibility check:

To handle potential future incompatibilities, we standardize a mechanism for a Python distributor to signal that a particular Python install definitely is or is not compatible with manylinux1 : this is done by installing a module named _manylinux , and setting its manylinux1_compatible attribute. We do not propose adding any such module to the standard library -- this is merely a well-known name by which distributors and installation tools can rendezvous. However, if a distributor does add this module, they should add it to the standard library rather than to a site-packages/ directory, because the standard library is inherited by virtualenvs (which we want), and site-packages/ in general is not.

Similar to https://github.com/pypa/pip/issues/3689.
Or is it reasonable to assume manylinux1 compatibility when building?

how about making it explicitly user controlled at pex creation time - e.g. with a feature flag like --manylinux to enable resolving against manylinux artifacts (when --platform contains the string linux)?
then if the user wants to build a manylinux compatible pex for running on a separate host, they'd just pass this flag - and then separately be gated by the runtime compatibility checking when running the pex on the separate host.

Sounds reasonable. --manylinux/--no-manylinux maybe and default to --manylinux to match the behavior of pip. I should have time to work on a PR for this in the next week or so.

+1, more and more packages are providing only manylinux1 wheels. It would be great if pex could pull those in instead of requiring linux-x86_64 artifacts.

another +1 on this. We have run into this issue with libsass which is only providing manylinux1 so we had to pin the last version... any update on manylinux pex support?

Unfortunately I'm not using PEX at my current job and haven't had much time to spend on this. 😞

+1 if/when you get to it :) This 🔥'd me.

+1

+1 ;-)

I found a disgusting hack that "works" for me. At the very least, when I build a pex using some manylinux1 wheels, it will allow it to be run.
In package.py, in the function platform_iterator(...), I added the following "hack" which treats all Linux distributions as manylinux1 compatible. Strictly speaking this is not true, and Pip has a bunch of rules in pip.pep425tags that Pex should probably be reusing. However, this was enough to get it to work for me:
  def platform_iterator(cls, platform):
    """Iterate over all compatible platform tags of a supplied platform tag.

       :param platform: the platform tag to iterate over
    """
    if cls.is_macosx_platform(platform):
      for plat in cls.iter_compatible_osx_platforms(platform):
        yield plat
    elif platform == 'linux_x86_64':
      yield 'manylinux1_x86_64'
      yield platform
    else:
      yield platform

What's the status of this? There seems to be two competing PRs #316 and #399, both in an unmergeable state. It would be great if pex also supported manylinux1. Thank you!

the final implementation is now reviewable here: https://github.com/pantsbuild/pex/pull/480

this is now available as of pex 1.4.0 (on pypi).
pex will now resolve against manylinux dists both at build and runtime (implicitly, by default) when targeting linux platforms, with no special platform targeting required:
[omerta ~]$ pex --version
pex 1.4.0
[omerta ~]$ pex --disable-cache --no-build --platform=linux-x86_64 msgpack_python -o /tmp/msgpack.pex
[omerta ~]$ unzip -l /tmp/msgpack.pex | grep manylinux | head -1
warning [/tmp/msgpack.pex]:  25 extra bytes at beginning or within zipfile
  (attempting to process anyway)
     1385  05-15-2018 16:56   .deps/msgpack_python-0.4.7-cp27-cp27m-manylinux1_x86_64.whl/msgpack/__init__.py

if you'd like to disable manylinux resolves (1.3.x and earlier behavior), you can pass --no-manylinux:
[omerta ~]$ pex --disable-cache --no-build --platform=linux-x86_64 --no-manylinux msgpack_python -o /tmp/msgpack.pex
Could not satisfy all requirements for msgpack_python:
    msgpack_python

you can also do more granular platform/tag targeting via an extended --platform mode:
[omerta ~]$ pex --disable-cache --no-build --platform=linux-x86_64-cp-27-cp27mu msgpack_python -o /tmp/msgpack.pex
[omerta ~]$ unzip -l /tmp/msgpack.pex | grep manylinux | head -1
warning [/tmp/msgpack.pex]:  25 extra bytes at beginning or within zipfile
  (attempting to process anyway)
     1385  05-15-2018 16:58   .deps/msgpack_python-0.4.7-cp27-cp27mu-manylinux1_x86_64.whl/msgpack/__init__.py

...which still supports multi-platform usage:
[omerta ~]$ pex --disable-cache --no-build --platform=linux-x86_64-cp-27-cp27mu --platform=linux-x86_64-cp-27-cp27m msgpack_python -o /tmp/msgpack.pex
[omerta ~]$ unzip -l /tmp/msgpack.pex | grep "cp27m-" | head -1
warning [/tmp/msgpack.pex]:  25 extra bytes at beginning or within zipfile
  (attempting to process anyway)
     1385  05-15-2018 16:59   .deps/msgpack_python-0.4.7-cp27-cp27m-manylinux1_x86_64.whl/msgpack/__init__.py
[omerta ~]$ unzip -l /tmp/msgpack.pex | grep "cp27mu-" | head -1
warning [/tmp/msgpack.pex]:  25 extra bytes at beginning or within zipfile
  (attempting to process anyway)
     1385  05-15-2018 16:59   .deps/msgpack_python-0.4.7-cp27-cp27mu-manylinux1_x86_64.whl/msgpack/__init__.py
[omerta ~]$

please take a moment to kick the tires with pex 1.4.0 and open issues for any bugs or problems that you might discover. enjoy!

This seems to still not work for me. When I download a manylinux wheel, pex can't seem to find it, but if I just rename manylinux1 to linux, it works.
When using the manylinux wheel for psycopg2==2.7.4 named as psycopg2-2.7.4-cp27-cp27mu-manylinux1_x86_64.whl, my build eventually fails with:
Exception message: Package SourcePackage(u'file:///Users/alec/.cache/pants/python_cache/requirements/CPython-2.7.10/psycopg2-2.7.4.tar.gz') is not translateable by ChainedTranslator(Wh
eelTranslator, EggTranslator, SourceTranslator)

(the mechanics of what exactly is happening here are still a mystery to me, but I'm guessing it's something like: no suitable wheel is found, so it starts trying to build from the source tar.gz, and can't because I'm not on Linux)
but when I then clear my ~/.cache/pants/python_cache/requirements and rename the wheel to psycopg2-2.7.4-cp27-cp27mu-linux_x86_64.whl, the build works.

My pex --version is 1.4.5, but a wrinkle is that I'm using pex via pants, and I'm not actually sure if the version of pex on my $PATH is the same version pants ends up using.

My pex --version is 1.4.5, but a wrinkle is that I'm using pex via pants, and I'm not actually sure if the version of pex on my $PATH is the same version pants ends up using.

Pants resolves its own version of pex. Can find-in-page for pex here: https://www.pantsbuild.org/notes-master.html ... to see when various versions were incorporated.

Ah, ok. Couldn't find it from that page directly, but I'm on pants 1.7.0, and judging from
https://github.com/pantsbuild/pants/blob/1.7.x/3rdparty/python/requirements.txt#L17
I guess I'm on pex 1.3.2 still.


1.x - PermitAll overridden by JWT

Signed-off-by: David Kral<EMAIL_ADDRESS>

Pull 1.x in your branch again, @tomas-langer has merged a change that bumps the copyright plugin to 2.3 it should fix the copyright error you are seeing.


Incoming call not working in Ionic application

Hi, @jefflinwood currently I'm facing an issue that is incoming call is not working I mean the outgoing call is working fine but if I'm calling from one device to another of different identity access token than incoming to that device is not working, no call notification or the dialog is appearing.

Hi,
It's probably a problem with the FCM integration - Firebase Cloud Messaging - do you have the push plugin receiving the FCM messages?

@jefflinwood yes i have push plugin and when i try to publish notification from firebase  all notification is receiving to the devices

above is my project.properties file

and thisone is my config.xml

I was having the same issue with incoming calls not working on ios.  (It works on Android)  I found this from Twilio and it helped me to resolve the problem.
https://github.com/twilio/twilio-voice-ios/blob/Releases/iOS-13-Migration-Guide.md
Based on this I made a change to TwilioVoicePlugin.m in this function.
#pragma mark PKPushRegistryDelegate methods

(void)pushRegistry:(PKPushRegistry *)registry didUpdatePushCredentials:(PKPushCredentials *)credentials forType:(PKPushType)type {
if ([type isEqualToString:PKPushTypeVoIP]) {
  const unsigned *tokenBytes = [credentials.token bytes];
  self.pushDeviceToken = [NSString stringWithFormat:@"<%08x %08x %08x %08x %08x %08x %08x %08x>",
                            ntohl(tokenBytes[0]), ntohl(tokenBytes[1]), ntohl(tokenBytes[2]),
                            ntohl(tokenBytes[3]), ntohl(tokenBytes[4]), ntohl(tokenBytes[5]),
                            ntohl(tokenBytes[6]), ntohl(tokenBytes[7])];

  //self.pushDeviceToken = [credentials.token description];
  NSLog(@"Updating push device token for VOIP: %@",self.pushDeviceToken);
  //[TwilioVoice registerWithAccessToken:accessToken deviceToken:self.deviceTokenString completion:^(NSError *error) { }];
  [TwilioVoice registerWithAccessToken:self.accessToken deviceToken:self.pushDeviceToken completion:^(NSError * _Nullable error) {
      if (error) {
          NSLog(@"Error registering Voice Client for VOIP Push: %@", [error localizedDescription]);
      } else {
          NSLog(@"Registered Voice Client for VOIP Push");
      }
  }];

}
}

I also had to change 2 lines to get the plugin to compile with the Twilio version.
Changed this:
[TwilioVoice handleNotification:payload.dictionaryPayload delegate:self];
to:
[TwilioVoice handleNotification:payload.dictionaryPayload delegate:self delegateQueue:NULL];
I'm not sure the delegateQueue should be NULL but so far my testing is working.

Hi I am using this Plugin my Incoming Handler is not working while My outgoing is working fine , I am also getting notification from firebase, so anyone can help for solving the issue of that incoming handler is not getting fired


interview requirements

It contains the interview requirements that the application should meet

Missing tests for mappers (there are mapstuct, so You don't write code, but You should still write tests).
Missing tests for client
Missing integration test for controller (preferrably with some sort of mock server, like wiremock)


Add a README

Add a README.md document describing how to setup and use uPortal-start

Resolved by #4


feat: replace all occurences of Dispatchers with SafeDispatchers

Feature Request
We want to be able to handle syscall failures when they become handlable,
This will happen soon.
In the meantime, we want to EVM layer, when calling contracts with Dispatchers, to be able to match on the Result.
We must therefore use SafeDispatchers
### Tasks
- [ ] Go through the codebase, and find occurrences of contract calls using a Dispatcher. Replace it with SafeDispatcher
- [ ] Handle the error, with SyscallError variant of EVMError
- [ ] Check tests

Describe the Feature Request
Describe Preferred Solution
Related Code
Additional Context
If the feature request is approved, would you be willing to submit a PR?
(Help can be provided if you need assistance submitting a PR)

[ ] Yes
[ ] No

@Eikix  may i try to work on this one?

@enitrat  thank you for your response!


Horrendous keyboard lag in games that compounds with time pressed

It's as if scrcpy places "key pressed" events in a queue with every other event, which quickly clogs up and doesn't let other events through. E.g.: open any game that supports keyboard and press W for a brief moment and at the same time, try to swipe a screen. It goes through. Then, press W for ten seconds and then try to swipe the screen. Touch input goes through after a few seconds(!). And so on: input lag is proportional to the time spent pressing a keyboard button.
EDIT: Only some games have this issue. From what I have on my phone, Minecraft and Survivalcraft exhibit this problem while Sonic 1 does not. Apologies if this issue is not scrcpy's fault

Hi,
Thank you for your report.
I just tested a random game (minetest) on a tablet, and indeed I can reproduce (a bit) the problem.
However, I added some logs, which show all key and touch events are transmitted and injected immediately by scrcpy.
After some delay (sometimes a few seconds), the game reacts to the events, but there are no new event injected at that time. So it seems the device/game takes time to consume the events.
Not sure if there are possible workarounds on our side.

Maybe related to the issue, but I get a similar problem randomly sometimes on normal use. scrcpy still inputs everything as expected in the end, backspaces and all, but stuff like parsing a long link can take almost a minute.
For the record, I'm using the latest release with the --prefer-text modifier, but I remember this happening before the text fix.

I just did another test: inject key events by pressing a key for several seconds in scrcpy, then touch with a finger on the physical device. The touch events consumption is also delayed.

Anyway, I think adding a command-line option --no-key-repeat would be acceptable. What do you think?

My thought was if key repeating did not occur with a physical keyboard, then that would be a big part of if repeating would be enabled by default or not.
I'm not a huge fan of it being only a flag since you may want to switch between repeating mode and non repeating mode and it would be silly to have to restart the program with different flags each time.

That is what I just tested, but it seems it does not (if you disable repeat on the client side, it does not repeat letters or space).

So, I was reading the android documentation,
https://developer.android.com/reference/android/view/KeyEvent
is scrcpy incrementing the repeat counter on events past the first one? It seems like for every event past the initial keydown, repeat should be incremented by one, where as the value for the keyup is always 0.

On the device side, scrcpy always injects the events with repeat = 0:
https://github.com/Genymobile/scrcpy/blob/3c0fc8f54f42bf6e7eca35b352a7d343749b65c4/server/src/main/java/com/genymobile/scrcpy/Controller.java#L134
On the client-side, SDL only report repeat as a boolean (either 0 or 1).
It would be interesting to know if passing the actual repeat value would solve the keyboard lag.

That's what im wondering, can you store a counter, unordered_map<key,int>, increment every time sdl repeat is true, wipe on repeat is false, and send the counter with the key press event?

unordered_map<key,int>

(This is C++) :smile:
A simple counter should be sufficient (not one per key, in practice only one key may be repeated at the same time).
Do you want to work on that to see if it improves things?
Here is a list of things to do:

add a int repeat field control_msg.h (in control_msg.inject_key_codes struct) and ControlMessage.java
adapt control_msg_serialize() and parseInjectKeycode()
pass the actual repeat value in convert_input_key()

I guess testing with one number would be a start, but if that does resolve the issue then it would be best to adhere to the spec and have the repeat value track each individual key. I guess an array would suffice since were working with only c.
I'll give it a shot, doesn't sound too bad.

I guess an array would suffice since were working with only c.

A single int is sufficient: if the current event has repeat enabled, increment the key, otherwise, reset to 0.

I was thinking tracking the count inside input_manager.h, i'm reading thru the code now. I miss protobuff already.

I miss protobuff already.

https://github.com/Genymobile/scrcpy/issues/1289#issuecomment-613434568

I guess an array would suffice since were working with only c.

A single unsigned int is sufficient: if the current event has repeat enabled, increment the counter, otherwise, reset it to 0.

https://github.com/Genymobile/scrcpy/blob/master/app/src/input_manager.c#L414
This is looks like it's freshly allocated each time. We need someplace to store the counter then the field inside the struct should get updated, correct?

Yes, for exemple, you could just add a field unsigned key_repeat_count; in struct input_manager.
Add pass this count to convert_input_key().

Setting repeat properly does seem to alleviate the lag.
Looking at the logging, I don't think i completely understand how sending multiple keys down at the same time is handled, figure it be best to report the findings before I delve any deeper. (It's this lack of understanding as to why im suggesting an array in this implementation)
int repeatCounter[AKEYCODE_ALL_APPS];
static bool
convert_input_key(const SDL_KeyboardEvent *from, struct control_msg *to,
                  bool prefer_text, bool repeat) {
    to->type = CONTROL_MSG_TYPE_INJECT_KEYCODE;

    if (!convert_keycode_action(from->type, &to->inject_keycode.action)) {
        return false;
    }

    uint16_t mod = from->keysym.mod;
    if (!convert_keycode(from->keysym.sym, &to->inject_keycode.keycode, mod,
                         prefer_text)) {
        return false;
    }

    if (to->inject_keycode.action == AKEY_EVENT_ACTION_DOWN && repeat)
    {
        repeatCounter[to->inject_keycode.keycode] = repeatCounter[to->inject_keycode.keycode] + 1;
        //LOGW("Setting repeat mode to %d %d",to->inject_keycode.keycode,to->inject_keycode.repeat );
    }
    else
    {
        //LOGW("Clearing repeate for %d",to->inject_keycode.keycode );
        repeatCounter[to->inject_keycode.keycode] = 0;
    }

    to->inject_keycode.repeat = repeatCounter[to->inject_keycode.keycode];
    //LOGW("Current code for %d ======== %d",to->inject_keycode.keycode,to->inject_keycode.repeat );

    to->inject_keycode.metastate = convert_meta_state(mod);

    return true;
}

this was quick and dirty to test and see if it even worked, so what do you think the best way to proceed would be?
I did some testing, and setting repeat to one or zero based on sdl repeat did not resolve the issue, so it seems that the repeat counter incrementing is integral to the issue being resolved.
Also, from the documentation https://developer.android.com/reference/android/view/KeyEvent#getRepeatCount()

Retrieve the repeat count of the event. For key down events, this is the number of times the key has repeated with the first down starting at 0 and counting up from there. For key up events, this is always equal to zero. For multiple key events, this is the number of down/up pairs that have occurred.

Not sure how this should be handled.

I don't think this matches the documentation 100% either, as the sdl fires multiple keydown events before repeat is set to true, so the counter is technically off by a few.

Do you have a branch with your changes?

I'll push it now.

https://github.com/xeropresence/scrcpy/tree/repeatCounter

Thank you :+1:
So I guess the issue is not related to repeat, but only to the rate of events that the device is not able to inject at the same rate. It's just that repeated events produce a lot of events in a small amount of time.
(IIRC there is also the same problem with multitouch events on a touchscreen.)

I'm not a huge fan of it being only a flag since you may want to switch between repeating mode and non repeating mode and it would be silly to have to restart the program with different flags each time.

If your device lags on successive events, I think it's ok to always pass --no-key-repeat, and never reenable it. This would be the only purpose of the flag.

If your device lags on successive events, I think it's ok to always pass --no-key-repeat, and never reenable it. This would be the only purpose of the flag.

Are events being dropped when the repeat counter > 0?
An issue with this is that there are some keys you would want to repeat, ie backspace,delete, arrow keys, maybe space. I play games, but also send text messages, snapchat, etc where those repeats would be desirable.
If tracking the repeat count resolves the issue, and better matches what the documentation expects, im in favor of that over adding a flag or a shortcut.

If tracking the repeat count resolves the issue

Yes, but apparently, it does not.

An issue with this is that there are some keys you would want to repeat, ie backspace,delete, arrow keys,peroid(...), maybe space.

Isn't there a lag when you press those keys repeated on your device?

With my branch, there is a slightly delay in games when the key is released, but it is only slightly longer then when I was disabling repeats entirely.
I think that might be due to where multiple keydowns are being sent without sdl setting repeat to true.
Outside of games, ie in snapchat I don't notice any lag.

which is still much faster then using the v1.14

Oh, so it improves the situation? The difference is reproducible after several tests switching back and forth between v1.14 and this branch?

I think that might be due to where multiple keydowns are being sent without sdl setting repeat to true.

I can't reproduce that.

Just to clarify my position: in any case, repeat should be correctly initialized for injecting events on the device. So unless it causes problems, this change should be done anyway.

Sounds good. I'm currently trying to do some testing to quantify the results of the change using recording.

Ok, so i modified my branch to limit a maximum of 40 repeat events to be generated. After 40 events were reached a fake keyup event was sent and all further events were ignored until an actual keyup event occurred.
Repeat with a hardcoded 0 value, v1.14
https://streamable.com/jlhi14
Repeat incrementing
https://streamable.com/d7l6bv
Not sending repeat at all
https://streamable.com/f2f5or
I also did some additional testing with stardew valley. Stardew valley is much less cpu, gpu intensive and displayed no noticeable difference between versions until I went and locked the cpu frequency to its smallest possible value. Once the cpu frequency was limited, there was no difference between the hardcoded 0 value and  the incrementing version. Both versions exhibited a noticeable delay when stopping. The version where repeat keys were not sent at all exhibited no delay.
In conclusion:
Incrementing the repeat value is either beneficial in scenarios where the cpu is somewhat bound or when the app ignores incoming repeat values
Not sending repeat keys offers the best performance, but will inhibit scenarios where the user does want the keys to actually be repeated, ie (delete,backspace, arrow keys, etc)
I think a flag is good, but a shortcut is better so that you can disable repeats, play games, then when you swap to sending messages you don't need to restart the program.

OK, so first let's inject the correct repeat value. :)

I think a flag is good, but a shortcut is better so that you can disable repeats, play games, then when you swap to sending messages you don't need to restart the program.

The issue is that a stateful toggle flag without visual feedback is confusing. That's why I haven't added one for --prefer-text.
Oh, also, please implement on dev (the development branch), not master: https://github.com/Genymobile/scrcpy/blob/master/BUILD.md#branches

I've rebased my branch onto dev.
Sounds like a OSD could be useful for displaying information about state changes.

Sounds like a OSD could be useful for displaying information about state changes.

(see #1484)
But even with that, I'm not sure I want to consume a shortcut for such an advanced feature (which is only necessary on some devices, as a workaround to avoid sending too many events).

What other changes would you like to see, just using a single unsigned int for tracking the count?

just using a single unsigned int for tracking the count?

Yes, and resetting it if and only if the current event has repeat set to false (otherwise, increment it).
Please also rebase on current dev and open a PR :)
Thank you :wink:

So using v1.15 with --no-key-repeat should fix this issue. I'm closing.


Hiding left menu items

Hi, with the fix #59 now its possible to hide left menu, but it would be nice if we can hide individual items in the left menu like hide +/- only or hide se/de only.
Regards,
Amal

I thought about this too when implementing the toggle.
My current feeling is that users that want to get a quick start with the bracket can start with the toolbar. If they want a more customized experience, they can disable it and implement what they need externally. If the toolbar gets bigger, the number of configuration parameters starts to explode. Integrated toolbar is also a bit rigid when you start to have customization requirements.
The bracket doesn't really have any hidden state, so if you want to add teams or change the format of the tournament, you can just reinitialize the bracket with different init value.
Here's a demo of external controls (bottom of the page) I've created for GotBracket: http://www.gotbracket.com/edit/acb3c5ba-3dbe-4287-a030-9b0cd05da53f

I understand, thank you so much for the information.

No problem :)


upgrade to asp.net core final

I don't think it will work unless that buildback is updated for the .net core final release.

Hi @Wyntuition
I have created a new dotnetcore-buildback https://github.com/jincod/dotnetcore-buildpack. Also I have updated this project to ASP.NET Core Preview and deployed demo version to https://aspnet5demoapp.herokuapp.com/.
Could you please test this buildback on your projects?

https://blogs.msdn.microsoft.com/dotnet/2017/01/30/announcing-net-core-net-native-and-nuget-updates-in-vs-2017-rc/#projectjsonxproj---csproj-migration