After I update from 1.24.0-10.2.pre to 1.25.0-8.1.pre,  My APP with permission_handler 5.0.1+1 compiled failed

It output a lot of error saying that something wrong with androidx.***
After I downgrade back to 1.24.0-10.2.pre, everything is OK.

I use Windows7 OS:
C:\geanapp\gphone>flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel beta, 1.25.0-8.1.pre, on Microsoft Windows [Version 6.1.7601], locale zh-CN)
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[√] Chrome - develop for the web
[√] Android Studio (version 4.1.0)
[√] VS Code (version 1.52.1)
[√] Connected device (2 available)
• No issues found!
The following is the output when I compile under 1.25.0-8.1.pre:
Launching lib\main.dart on M2002J9E in debug mode...
lib\main.dart:1
FAILURE: Build failed with an exception.

What went wrong:
Could not determine the dependencies of task ':permission_handler:compileDebugAidl'.

Could not resolve all task dependencies for configuration ':permission_handler:debugCompileClasspath'.
Could not resolve androidx.lifecycle:lifecycle-common:{strictly 2.0.0}.
Required by:
project :permission_handler
Cannot find a version of 'androidx.lifecycle:lifecycle-common' that satisfies the version constraints:
Dependency path 'com.baseflow.permissionhandler:permission_handler:1.0-SNAPSHOT' --> 'io.flutter:flutter_embedding_debug:1.0.0-92ae191c17a53144bf4d62f3863c110be08e3fd3' --> 'androidx.lifecycle:lifecycle-common:2.2.0'
Constraint path 'com.baseflow.permissionhandler:permission_handler:1.0-SNAPSHOT' --> 'androidx.lifecycle:lifecycle-common:{strictly 2.0.0}' because of the following reason: debugRuntimeClasspath uses version 2.0.0
Dependency path 'com.baseflow.permissionhandler:permission_handler:1.0-SNAPSHOT' --> 'io.flutter:flutter_embedding_debug:1.0.0-92ae191c17a53144bf4d62f3863c110be08e3fd3' --> 'androidx.lifecycle:lifecycle-common-java8:2.2.0' --> 'androidx.lifecycle:lifecycle-common:2.2.0'
Dependency path 'com.baseflow.permissionhandler:permission_handler:1.0-SNAPSHOT' --> 'io.flutter:flutter_embedding_debug:1.0.0-92ae191c17a53144bf4d62f3863c110be08e3fd3' --> 'androidx.lifecycle:lifecycle-runtime:2.2.0' --> 'androidx.lifecycle:lifecycle-common:2.2.0'

Hi @heroqin
This will be fixed in the next beta release, meanwhile, you can either upgrade to dev.
I see there's an open issue addressing the case you described. https://github.com/flutter/flutter/issues/72627
Please follow up on that issue, I'm closing the current one as a duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you


GO:0000187 | activation of MAPK activity

GO:0000187 | activation of MAPK activity
do we need this term? The only think that is annotated to it is
MF MAP kinase kinase activity
(its a MF in the BP ontology)

Here's my proposal:

[ ] obsolete
GO:0000187 | activation of MAPK activity
and  GO:0007257 activation of JUN kinase activity
and suggest 'consider' protein kinase activator activity

[ ] obsolete  GO:0071883 activation of MAPK activity by adrenergic receptor signaling pathway
No annotations; corresponds to a GPCR pathway

What do you want to do with

GO:0000199 activation of MAPK activity involved in cell wall organization or biogenesis
GO:0071508 activation of MAPK activity involved in conjugation with cellular fusion
?

Thanks, Pascale

Obsolete?
These are GO-cams

ok great

I'll start with those with few annotations

GO:0000199 activation of MAPK activity involved in cell wall organization or biogenesis
GO:0071508 activation of MAPK activity involved in conjugation with cellular fusion

Dear all,
The proposal has been made to obsolete

GO:0000199 activation of MAPK activity involved in cell wall organization or biogenesis and
GO:0071508 activation of MAPK activity involved in conjugation with cellular fusion

The reason for obsoletion that those are pre-composed terms that should be captured as a GO-CAM model. There are 3 EXP annotations to 'activation of MAPK activity involved in cell wall organization or biogenesis' by AspGD, and no annotations to 'activation of MAPK activity involved in conjugation with cellular fusion'.
There are no mappings to those terms; those terms are not in any subsets.
We are opening a comment period for this proposed obsoletion. We’d like to proceed and obsolete this term on April 23. Unless objections are received by April 23, 2021, we will assume that you agree to this change.
You can comment on the ticket here: https://github.com/geneontology/go-ontology/issues/20242
Thanks, Pascale
Thanks, Pascale

@marekskrzypek Can you please look at the 3 AspGD annotations?

AspGD is no longer maintained. I can certainly take a look at these items, but I can't convert them to a GO-CAM model, even if it sounds reasonable to me; all I can do is to delete them.

Those terms also have no annotations:
GO:0000198 | activation of MAPKK activity involved in cell wall organization or biogenesis' | 0
GO:0071509 | activation of MAPKK activity involved in conjugation with cellular fusion' | 0
GO:0000197 | activation of MAPKKK activity involved in cell wall organization or biogenesis' | 0
GO:0071510 | activation of MAPKKK activity involved in conjugation with cellular fusion' | 0
GO:0000200 | inactivation of MAPK activity involved in cell wall organization or biogenesis' | 0
GO:0000200 | inactivation of MAPK activity involved in conjugation with cellular fusion' | 0
GO:0035423 | inactivation of MAPK activity involved in innate immune response' | 0

Hi @marekskrzypek  you would not lose any information if you use the suggested replacements.

Hi Marek, you can just use
GO:1903139 positive regulation of cell wall integrity MAPK cascade, which means essentially the same thing.

Oh, yes, makes sense. Thanks, Val


Expose port required or not?

Hi, first thanx for amazing stuff!
Anyway: in readme, there is notice that

The containers being proxied must expose the port to be proxied, either by using the EXPOSE directive in their Dockerfile or by using the --expose flag to docker run or docker create.

I'm using nginx-proxy via docker-compose (v3.3) but it works even without exposing ports, somehow automatically. Is this correct? Is there some way to allow only some ports?
Also, in this project's readme the is section "Docker Compose", but whoami container is NOT exposing any ports...
Thanx a lot for clarification... and maybe there should be some mention in readme file when it's required and when not.

Hi, you missed line 9 - see https://github.com/jwilder/whoami/blob/master/Dockerfile#L9

Hi matt, ssry for newbie question... i thout that when working with docker-compose, that only ports listed in docker-compose are exposed. But it seems that ports are exposed:

as defined in Dockerfile
and as deffined in docker-compose.yml
right?
Thanx a lot!

Hi,
I got nginx-proxy up and running. Still I'm not sure why I want to have EXPOSEd Ports (actually ports definition in docker-compose.yml).
I understand that the nginx-proxy is using that information to build /etc/nginx/conf.d/default.conf. However, as I defined VIRTUAL_PORT=xy I'm not sure its really required?
I mean - I wouldn't really care - but EXPOSing Ports means they are accessible from the outside? E.g. if I use nginx-proxy to EXPOSE 443 - why would my webservers still sit with exposed HTTP ports?
I'm sure that's a missunderstanding on my end - that's why I'm asking before trying to tinker stuff..
Thanks ;-)

@maximkrusina and @Trashmee, you are both correct.  EXPOSE does almost nothing at all - it just hints at what ports might be used, without actually binding them to the host (that's what PORT does, as @Trashmee noted).  As far as I know, you cannot un-expose a port (I filed a bug about this with Docker years ago), so yes, every Dockerfile in the chain that led you to your image, and your docker-compose.yml can add on more exposed ports.

Thank you all! :)

I am attempting to use the nginx-proxy to manage my docker servers on a host, some of these servers require extra open ports for other actions performed by the server, e.g. a splunk server container is a good example.  I use the splunk server to collect logs as forward logs the container has all the ports exposed on the docker file, but when I check the server with docker inspect I get the secondary port listed as null.
"NetworkSettings": {
"Bridge": "",
"SandboxID": "e6d664795404344aedc1c6b832acf9104009e8d560a1fb4f8f017fcba8687d29",
"HairpinMode": false,
"LinkLocalIPv6Address": "",
"LinkLocalIPv6PrefixLen": 0,
"Ports": {
"1514/tcp": null,
"8000/tcp": [
{
"HostIp": "<IP_ADDRESS>",
"HostPort": "8080"
}
],
"8088/tcp": null,
"8089/tcp": null,
"8191/tcp": null,
"9997/tcp": null
also following the commands in the Readme I created VIRTUAL_HOST for each port using the form  -e
VIRTUAL_HOST=splunk2.xxx.mycompany.com:8088,  splunk2.xxx.mycompany.com:9997, splunk2.xxx.mycompany.com:8191 as well as using the individual Virtual_host as in -e VIRTUAL_HOST=splunk2.xxx.mycompany.com:8080  -e VIRTUAL_HOST=splunk2.xxx.mycompany.com:9997  -e VIRTUAL_HOST=splunk2.xxx.mycompany.com:8191 and neither method makes the ports accessible from containers and other computers using the http collector which runs on port 8088 and in both instances I get the null port issue listed above.  is there a way I can resolve this to make it viable.


added servant-0.4 template

This PR adds a slightly bigger template for servant-0.4.
I put it in a subdirectory, because I thought it would make sense to start to organize the templates a bit, but I don't know if this'll actually work with stack new. I can move it to the root dir, of course.

I'll merge it and see if it works.

Related: I've created a servant-postgresql template for servant 0.5 & was wondering about a couple things:

Would this be accepted? Are there some sort of minimum requirements for new templates or will you accept anything useful & working? Is there a limit to how indepth a template can be(maybe that's more an on individual basis)?
Is there a way to compose templates? So this template & my template could reuse the Test file instead of having to duplicate the code?
Is there some sort of style guide for the final app? For instance, when specifying the executable, should Main.hs live in app/ or bin/ or exe/ or something else?
Is there a way to get stack to add servant-0.5 & servant-server-0.5 to the extra-deps in stack.yml when generating the app? Or should I rely on the cabal file & expect people to add --solver to their stack new command?

I will make a separate PR when I have some extra time.

@mgsloan: I've put the file in the root directory. I've also renamed it to indicate that it uses hspec-wai-json.
@prikhi: See a related discussion here: https://groups.google.com/d/topic/haskell-stack/sOI5rY28G3E/discussion

@prikhi Not really the best place to discuss, I have opened a few issues:
https://github.com/commercialhaskell/stack-templates/issues/55
https://github.com/commercialhaskell/stack/issues/1994
https://github.com/commercialhaskell/stack-templates/issues/56

I'd really prefer a short name.  Imagine someone typing this in!  We do not have shell autocompletion of template names yet.  servant or servant-0.4 is fine.
I'm actually on the wall as to whether it's useful to have templates for specific libraries like this, as described in https://github.com/commercialhaskell/stack-templates/issues/55 .  I really don't want to start seeing lens-aeson-hspec-containers templates.
In the meantime I'm happy to merge very library specific templates, but the policy may change.  I don't really want the default library creation process to include "make a stack template"

@mgsloan: That makes a lot of sense. I've renamed it to servant-0.4. (There's already a servant template.)

Oops forgot to merge.  Thanks!


single dispatch for descriptions and netbox

This looks really great! I pushed one commit that sets the steps to match my documentation updates in orch-core, otherwise this looks perfect IMO. Really nicely done.


New Edge Browser automation - create groups/tabs/pins based on configuration file/preset/template

Examples
{
  "tabs": [
    {
      "url": "https://github.com/devynspencer",
      "pinned": true
    },
    {
      "url": "https://www.youtube.com/feed/history",
      "pinned": true
    }
  ],
  "tab_groups": [
    {
      "name": "Projects",
      "color": "Pink",
      "tabs": [
        "https://github.com/users/devynspencer/projects/990",
        "https://github.com/users/devynspencer/projects/991",
        "https://github.com/users/devynspencer/projects/992"
      ]
    },
    {
      "name": "Documentation",
      "color": "Violet",
      "tabs": [
        "https://discord.js.org/docs/packages/discord.js/14.15.3",
        "https://discordx.js.org/docs/discordx/getting-started"
      ]
    },
    {
      "name": "Communication",
      "color": "Purple",
      "tabs": [
        "https://discord.com/channels/123456/123456",
        "https://outlook.live.com/mail/0/inbox",
        "https://outlook.live.com/calendar/0/view/month"
      ]
    },
    {
      "name": "Development Tools",
      "color": "Gray",
      "tabs": [
        "https://www.tablesgenerator.com/markdown_tables#",
        "https://coolors.co/palettes/trending"
      ]
    }
  ]

Tab group colors are apparently static, oops.


I cant install any Extension (error:of central directory record signature not found)

VSCode Version: Code 1.9.1 (f9d0c687ff2ea7aabd85fb9a43129117c0ecf519, 2017-02-09T00:26:45.394Z)
OS Version: Windows_NT ia32 6.1.7601

messageService.ts:126  of central directory record signature not found: Error: end of central directory record signature not found,    at c:\Program Files (x86)\Microsoft VS Code\resources\app\node_modules\yauzl\index.js:98:14,    at c:\Program Files (x86)\Microsoft VS Code\resources\app\node_modules\yauzl\index.js:342:5,    at c:\Program Files (x86)\Microsoft VS Code\resources\app\node_modules\fd-slicer\index.js:32:7,    at FSReqWrap.wrapper [as oncomplete] (fs.js:681:17)e.doShow @ messageService.ts:126

You are behind a proxy, can you provide some help in #18103 ?

I'm getting the same error
end of central directory record signature not found
On version:
Visual Studio Code - Insiders

Version 1.10.0-insider
Commit<PHONE_NUMBER>1592cf5fe64fd911b1efe41b1ea732
Date 2017-02-20T07:13:27.765Z
Shell 1.4.6
Renderer 53.0.2785.143
Node 6.5.0

Even when installing from VSIX.
Extensions are upgrading normally, but if I install https://marketplace.visualstudio.com/items?itemName=AureliaEffect.aurelia from market, or download .vsix and try to install, I get the same message.
This is on Windows 8.1, with proxy set to Fiddler, which is allowing extension manager to do remote searches and download extension updates without any problem.

I have the same extension on another machine (Mac), and it installs just fine, so, I'm assuming it's not extension specific.


Split repo into separate packages for app, tasks and src

I'll leave the description for the team, but ideas from me:

Using npm workspaces to split up the repo
Each area can have its own tests, config and package.json

Replace complex Jest project wildcards with per-package configs
If the review app wants all tests to have Puppeteer, great (won’t affect other packages)

module.exports = {
  projects: [
    '<rootDir>/app/*',
    '<rootDir>/src/*',
    '<rootDir>/tasks/*'
  ]
}

Making the "Review app" a user of govuk-frontend
Standalone package that uses npm install govuk-frontend (symlink via npm workspaces), following the Node.js documentation, rather than mount an already-built /public directory

import 'govuk-frontend-repository'
import 'govuk-frontend-review'
import 'govuk-frontend'

Making smaller "exemplar" how-to packages to help the community
Example setups for using govuk-frontend via Rollup, webpack bundlers etc


M13675-Copy a permalink and paste into another channel

Summary
Added e2e test to validate permalink of any random message in another DM/channel
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/10075

@saturninoabril  could you please review it

@composednitin Can you confirm that this spec is passing locally on you? It's always failing; looks like copying text is not working properly. Could you try it again? thanks

yes added some change

@saturninoabril @thekiiingbob could you please review it again

@saturninoabril @composednitin  Ok, I re-re-read the JIRA ticket again and I think we are dealing with a two tests vs just one.
Original Test:
Steps:

Log into https://community.mattermost.com
Click on the 3-dot menu to the right of any message in any channel
Click on "permalink"
Click on "copy link" then on "close"
Switch to another channel, Direct or Group Message
Paste the copied permalink into the message input box (CTRL+V) and hit "Enter"

Expected:

Permalink should copy and paste properly
Once pasted into a new channel it should render properly
Clicking on the permalink takes you to the conversation

For the first part, we really want to verify that the permalink is correct, and that when we click it, it's copied to the clipboard. Now, problem is that we don't have access to the clipboard easily. BUT, we know what code does this - https://github.com/mattermost/mattermost-webapp/blob/master/components/get_link_modal.jsx#L42
I propose we spy on that function with:
cy.document().then((doc) => {
  cy.spy(doc, 'execCommand').as('execCommandSpy')
})

Then, after you click the copy link button, you can do: cy.get('@execCommandSpy').should('have.been.calledWith', 'copy') to verify it was actually called.
The second test then is that when we paste in a link to another channel (which doesn't necessarily need to be a link we get from this modal) that "does it render properly, and does in navigate to that channel properly?"
You can combine all this into one test, of course,  it just feels like two to me.

This issue has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!
/cc @jasonblais @hanzei

Hey @composednitin - let us know if you have any questions about the feedback from @thekiiingbob above? Thanks for your help thus far!!

Hey @composednitin,
Please note that we will close this PR after another 10 days of inactivity. If you don't object, other contributors might complete this PR for you.

Hey @composednitin,
As per our process for inactive contribution I will close this. Thank your for this PR. :+1: Let me know, if you still want to work on this.


Add SIL and IRGen support for a ConstantStringLiteral instruction

This supports a utf8 or utf16 encoding.
Because we don't support reference count inert instructions in the runtime (yet?) such strings don't go into the read-only section. Instead we initialize the string's strong reference count to two.
rdar://30545013

@swift-ci Please smoke test


Introduce a custom error type to classify errors.

When checking signatures and evaluating policy, we may run into errors
for a variety of reasons unrelated to non-compliance, such as the registry
serving a 500.  This change introduces an error type that we can use to
produce and wrap errors in a way that we can distinguish between incidental
and verification errors in the calling logic.
After this change, we will be able to write:
verr := &cosign.VerificationError{}
if errors.As(err, &verr) {
   // This is a verification error, handle accordingly.
} else {
  // This is something else, handle differently.
}

Related: https://github.com/sigstore/policy-controller/issues/109
Release Note
Documentation

Codecov Report

Merging #2114 (31f7f9b) into main (39fb8d6) will increase coverage by 0.05%.
The diff coverage is 74.07%.

@@            Coverage Diff             @@
##             main    #2114      +/-   ##
==========================================
+ Coverage   26.27%   26.33%   +0.05%
==========================================
  Files         129      130       +1
  Lines        7582     7588       +6
==========================================
+ Hits         1992     1998       +6
  Misses       5333     5333
  Partials      257      257

Impacted Files
Coverage Δ

pkg/cosign/verify.go
32.08% <63.15%> (ø)

pkg/cosign/errors.go
100.00% <100.00%> (ø)

pkg/policy/eval.go
78.94% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us.


Bad index values abort job in OnDemandAccumulatorPyramidIO

val (x, y) = indexScheme.toCartesian(index)

Should be wrapped in a try/catch to handle cases where the index cannot be read/cast, e.g. nulls and other bad data.

Though there was a problem in the accumulator tiler, the problem Rob was seeing was actually in CSVReader - and would affect batch tiling as well as accumulator tiling.


LFO depth for square/pulse doesn't match ARIA

Not sure if you consider this a bug, but I saw that sfizz LFOs for pulse/square are range [-1:1] whereas ARIA appears to be [-0.5:0.5]. Here is a test case
<region>
lfo1_freq=1
lfo1_wave=3
lfo1_pitch=600
sample=*sine

In ARIA this jumps 6 semitones (3 semitones up, 3 semitones down), whereas in sfizz this is a jump of one octave. This only affects square/pulse wave forms (2-5)

If it's not matching to ARIA, it's definitely a bug.
I'll check it ASAP, thanks for reporting.

If it's not matching to ARIA, it's definitely a bug.
I'll check it ASAP, thanks for reporting.

In ARIA this jumps 6 semitones (3 semitones up, 3 semitones down), whereas in sfizz this is a jump of one octave. This only affects square/pulse wave forms (2-5)

The current behavior of sfizz follows that of Cakewalk, but it turns out ARIA acts different.
Anyway we are following ARIA in priority, so I'm going to change it, and also review other shapes while at it.

whereas ARIA appears to be [-0.5:0.5].

In fact it's [0;1]. I have to modify the sine too, which has inverted phase.

Fixed by #613


2076-2 Support

Codecov Report

Merging #115 (2049db3) into master (146670a) will increase coverage by 0.83%.
The diff coverage is 95.61%.

@@            Coverage Diff             @@
##           master     #115      +/-   ##
==========================================
+ Coverage   89.51%   90.35%   +0.83%
==========================================
  Files         115      120       +5
  Lines        5534     5557      +23
==========================================
+ Hits         4954     5021      +67
+ Misses        580      536      -44

Impacted Files
Coverage Δ

...clude/adm/elements/audio_block_format_binaural.hpp
93.75% <ø> (+6.25%)
:arrow_up:

include/adm/elements/audio_block_format_id.hpp
100.00% <ø> (ø)

include/adm/elements/audio_block_format_matrix.hpp
93.75% <ø> (+6.25%)
:arrow_up:

...nclude/adm/elements/audio_block_format_objects.hpp
100.00% <ø> (ø)

include/adm/elements/audio_channel_format.hpp
94.64% <ø> (-0.10%)
:arrow_down:

include/adm/elements/audio_channel_format_id.hpp
100.00% <ø> (ø)

include/adm/elements/audio_content_id.hpp
100.00% <ø> (ø)

include/adm/elements/audio_object.hpp
100.00% <ø> (ø)

include/adm/elements/audio_object_id.hpp
100.00% <ø> (ø)

include/adm/elements/audio_object_interaction.hpp
100.00% <ø> (ø)

... and 138 more

Continue to review full report at Codecov.

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

For when this is merged:
There are a few commits in this branch relating to github actions which are also in master (to avoid rebasing and the branches from this getting out of sync). This should therefore be rebased to remove these before merging.

merged through #131


[1.1.x][KOGITO-4110] - Single namespaced operator reacts on CRs created in d…

…ifferent namespaces
See: https://issues.redhat.com/browse/KOGITO-4110
cherry-pick: https://github.com/kiegroup/kogito-cloud-operator/pull/712
Signed-off-by: Tarun Khandelwal<EMAIL_ADDRESS>Many thanks for submitting your Pull Request :heart:!
Please make sure your PR meets the following requirements:

[x] You have read the contributors' guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains a link to the JIRA issue
[x] Pull Request contains a description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[ ] You've tested the new feature/bug fix in an actual OpenShift cluster
[ ] You've added a RELEASE_NOTES.md entry regarding this change

/jenkins prior test

/jenkins prior test

/jenkins prior test

/jenkins prior test

/jenkins prior test

/jenkins prior test

/jenkins prior test

/jenkins prior test

/jenkins prior test

/jenkins prior test

/jenkins prior test

/jenkins prior test

/jenkins test

/jenkins test

/jenkins test

/jenkins test

/jenkins test

/jenkins test

tests ok. @spolti please review and merge so it can be included in 1.1.x

tests ok. @spolti please review and merge so it can be included in 1.1.x


[shared_preferences] Update macOS for NNBD

macOS federated plugin implementations that contain no Dart code just
need their Dart SDK bumped in order to be considered nullsafe.
Part of https://github.com/flutter/flutter/issues/74876
Pre-launch Checklist

[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide and the C++, Objective-C, Java style guides.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test exempt.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I signed the CLA.
[x] All existing and new tests are passing.

I suspect syncing this branch should fixed the presubmit failures


[BUG] Variable not properly set after loading model instance

Describe the bug
It seems self._init_completed is not set after successfully loading a model
https://github.com/Project-MONAI/monai-deploy-app-sdk/blob/main/monai/deploy/operators/monai_bundle_inference_operator.py#L463
Steps/Code to reproduce bug
Use ai_spleen_seg_app
Expected behavior
self._init_completed is set to True
Environment details (please complete the following information)

OS/Platform: Ubuntu
Python Version: 3.8.10
Method of MONAI Deploy App SDK install: pip
SDK Version: 0.4.0

It looks like lines 458-463 should be:
                with self._lock:
                    if not self._init_completed:
                        self._bundle_path = self._model_network.path
                        self._init_config(self._bundle_config_names.config_names)
                        self._init_completed = True

There is an extra but harmless conditional check currently.

It looks like lines 458-463 should be:
                with self._lock:
                    if not self._init_completed:
                        self._bundle_path = self._model_network.path
                        self._init_config(self._bundle_config_names.config_names)
                        self._init_completed = True

There is an extra but harmless conditional check currently.

Thanks. It is not an issue at runtime, and I thought the style/format checkers should at least give a warning. Anyway, I will add the R- Value there in the code.


Clear pending timeout (if a new transition started before it finished)

https://codepen.io/zaceno/pen/XzOwPd
If you use 1000ms time or so, and you click the button relatively fast, you'll notice glitchy behavior/transitions, due to the fact that timeouts aren't cleared if there are pending ones.
function runTransition (el, attr, before, after, ondone) {
    var easing = attr.easing || 'linear'
    var time = attr.time || 300
    var delay = attr.delay || 0
    setStyle(el, before)
    setTimeout(function () {
        requestAnimationFrame(function () {
            setStyle(el, after)
            el.style.transition = 'all ' + easing + ' ' + time + 'ms'

          // HERE:
          // clear current pending timeout (if any) before setting another timeout

            setTimeout(function () {
                el.style.transition = null
                ondone && ondone()
            }, time)
        })
    }, delay)
}

Thanks @atomiks! It's a good idea, but I don't think just clearing the timeout will be enough to get the behavior you want (Try here: https://codepen.io/zaceno/pen/ELybGW).
The idea is, if you click the same square multiple times, you want it to slowly move towards its intended position from where it is now, without janking, right?
I need to think a bit more about this - but to be honest, I don't think it's a high priority issue ... more "nice to have".

You're right - clearing the timeout fixes the jank due to the old timeout firing before the new transition finishes (so it will instantly move to the new position), but the elements should transition from their current mid-transition position... does getBoundingClientRect() work there?

So I started using this module in a proper project and this is proving to be kind of a dealbreaker.

Rapidly clicking a button that expands an item inside the transitioned list item causes glitchy transitions of the list items (normal user behavior I'd say)
Adding a delay to stagger list items that transition in continually increments the delay each time the list items are filtered since the timeout is not cleared, causing them to not appear after a while

@atomiks While I'm still not sure if I'll be able to fix this, do you think you could provide an example like in a codepen or repo?
Especially an example of "Rapidly clicking a button that expands an item inside the transitioned list item" since I'm having trouble visualizing what's going on.
No promises, but it would be helpful to me (or hopefully someone smarter) in finding a solution.

@atomiks
I'm going to close for now -- I'm still open to looking into this but I need an example to work with. Reopen if you've got one!


Parser: Highlight 'self' and 'Self' correctly when allowed to be an identifier

Could you clarify the motivation for this patch? What's the issue?

The parser allows to declare generic parameters and certain bindings named  self or Self without having to escape them, but was not promoting these to identifier tokens. For example, Self in struct G<Self> is highlighted as a keyword.

I see. But I think this change affects too broad. For example:
struct S {
  func foo() {
    self.foo()
  }
}

This self should be parsed as a keyword but it uses consumeIdentifier().

Also
struct S<Self> {
  func foo() -> Self {
    Self
  }
}

The parser doesn't know these Self in the function mean whether the current type (keyword) or the generic parameter. It's semantic thing. So I feel like, for syntax coloring, we should keep parse them as keywords.

I see. But I think this change affects too broad

Right, I was careless, glad you noticed!

The parser doesn't know these Self in the function mean whether the current type (keyword) or the generic parameter. It's semantic thing. So I feel like, for syntax coloring, we should keep parse them as keywords.

Yes, the intention was to fix syntax coloring only for identifiers that are known to be declarations (just the generic parameter in your example).

After a closer look, the rules on whether something is allowed to be declared self or Self seem rather sketchy. Sometimes the parser lets these identifiers through only to fail the expectations of Sema. I doubt there currently is any value in landing the intended fix, so closing this off.


The model I want to use was added only after transfomer 4.2.1, what should I do in this case?

thank you for your great work！but if i don't use transformer=4.2.1,The code will report an error “cannot import name '_model_unwrap' from 'transformers.trainer'“，could you tell me how to solve this case？thanks a lot~

Hi,
In this case you can upgrade your transformers. Our code should also work for later transformers (unless there is some API change then you need to modify our code accordingly)


Deriving Deserialize fails with cryptic error when struct member is templated on the type (?)

use serde::{Serialize, Deserialize};
use serde_json;
use std::marker::PhantomData;

#[derive(Serialize, Deserialize, Debug)]
pub struct ID<Entity>(PhantomData<Entity>);

#[derive(Serialize, Deserialize, Debug)]
struct Point {
    pub id: ID<Self>,
}

fn main() {
    let point = Point {  id: ID(PhantomData) };

    let serialized = serde_json::to_string(&point).unwrap();
    println!("serialized = {}", serialized);

    let deserialized: Point = serde_json::from_str(&serialized).unwrap();
    println!("deserialized = {:?}", deserialized);
}

error:
error[E0308]: mismatched types
 --> src/main.rs:8:21
  |
8 | #[derive(Serialize, Deserialize, Debug)]
  |                     ^^^^^^^^^^^ expected struct `Point`, found struct `_IMPL_DESERIALIZE_FOR_Point::<impl _IMPL_DESERIALIZE_FOR_ID::_serde::Deserialize<'de> for Point>::deserialize::__Visitor`
  |
  = note: expected type `ID<Point>`
             found type `ID<_IMPL_DESERIALIZE_FOR_Point::<impl _IMPL_DESERIALIZE_FOR_ID::_serde::Deserialize<'de> for Point>::deserialize::__Visitor<'_>>`

Thanks! This is being tracked in #1565. I would accept a PR to fix this.


Implement BlockDriverManager

Preliminary implementation of BlockDriverManager.
Please take a look and see what needs to be changed or improved @kevinputera @ljiazh3ng

Closed in favour of #150 and its future implementation PR


Feature/network

added all functions in network.py but some fucntions are missing

sorry, wrong repo


IRC client gets disconnected by the remote when too much data is received

Bug Report
Code
// repeat 1000+ times
chatClient.join(_channelName).then(success);

Expected behavior
My chat client joins all channels and stays in there.
Actual Behavior
All channels are joined, but as soon as Twitch tries to send me around 0.6Mbps of chat messages, Twitch kills the connection (reason 1006 from what I could gather from the logs). I cannot quickly find the exact limit in the documentation of Twitch, but this behaviour is to be expected according to https://discuss.dev.twitch.tv/t/limits-on-irc-channels-join/16247/2
Maybe I missed it in the documentation of this library, but is it possible to create separate connections that handle a couple of channels each, as apparently this send buffer is limited per IRC connection, unlike many other limits (e.g. 15 JOINs/PARTs per 15 seconds which is per IP address).
Coincidentally while adding error handling to restore from this failure I noticed that chatClient.onDisconnect is not implemented yet, or am I missing something. Right now I added code to abuse the "private" chatClient._connection.onDisconnected but I am not entirely sure if that would trigger either as it hasn't happened since.
Environment

Version: 2.3.5 (from npmjs)
Node version: 11.x (dockerhub node:11-alpine)
Operating system: Alpine (docker)

After discussion about this issue on the discord channel (for anyone finding this at a later time):

Connection pooling is not planned for short-term

onDisconnected from ircv3 is forwarded and available

If you want connection pooling to different IRC connections, handle it yourself for now.

@d-fischer
It's been 2-3 years 😅
On a serious note, I think connection pooling would be a cool addition to this package.

@jabbink are you still using twurple for irc connections? should i use something else?
how are you handling connection pooling yourself? im running into this issue where disconnects are happening way too frequently, and it's creating issues for my users in production. it takes way too long to join over 2k+ users rn

and these all in one day:

@Geczy I stopped the project I needed this for not too long ago.
I pooled connections by making a new chat client every x channels I wanted to join, keeping track which channel mapped to which client; that reduced the unexpected disconnects to basically 0.


feat: build css tailwind too slow

https://github.com/tailwindlabs/tailwindcss/issues/2544

@sansan88 did not  check what/why/when/where and if it has been released but the related tailwind css issue is closed. maybe "just"  an update can solve this now?
https://github.com/tailwindlabs/tailwindcss/issues/2820#event-4114007388

ohh just pushed an update. let's see what happens

haha nothing new..  I have to go through the issue again.. seems there is another problem.

🥺


Error: Cannot find module 'weak'

I build pantomjs2 from source , and  excute npm install at  prerender path, everything is go ok,
but when run  node server.js , display flow error
Error: Cannot find module 'weak'
at Function.Module._resolveFilename (module.js:336:15)
at Function.Module._load (module.js:278:25)
at Module.require (module.js:365:17)
at require (module.js:384:17)
at new D (/root/prerender/node_modules/phantom/node_modules/dnode/index.js:28:20)
at module.exports (/root/prerender/node_modules/phantom/node_modules/dnode/index.js:8:12)
at /root/prerender/node_modules/phantom/phantom.js:159:13
at Server.handler (/root/prerender/node_modules/phantom/node_modules/shoe/index.js:22:9)
at Server.emit (events.js:107:17)
at App.emit (/root/prerender/node_modules/phantom/node_modules/shoe/node_modules/sockjs/lib/sockjs.js:182:27)
2015-04-05T06:46:18.306Z worker 23 died, restarting!
module.js:338
throw err;
^
Error: Cannot find module 'weak'
at Function.Module._resolveFilename (module.js:336:15)
at Function.Module._load (module.js:278:25)
at Module.require (module.js:365:17)
at require (module.js:384:17)
at new D (/root/prerender/node_modules/phantom/node_modules/dnode/index.js:28:20)
at module.exports (/root/prerender/node_modules/phantom/node_modules/dnode/index.js:8:12)
at /root/prerender/node_modules/phantom/phantom.js:159:13
at Server.handler (/root/prerender/node_modules/phantom/node_modules/shoe/index.js:22:9)
at Server.emit (events.js:107:17)
at App.emit (/root/prerender/node_modules/phantom/node_modules/shoe/node_modules/sockjs/lib/sockjs.js:182:27)
2015-04-05T06:46:18.348Z worker 24 died, restarting!
2015-04-05T06:46:18.763Z starting phantom on port [12325]
2015-04-05T06:46:18.799Z Server running on port 3000
2015-04-05T06:46:18.803Z starting phantom on port [12326]
2015-04-05T06:46:18.830Z Server running on port 3000

node is unzip at /usr/local/node-v0.12.2-linux-x64/

That's some kind of configuration issue not related to Prerender. Did npm install install weak?

Installed Python and this problem went away:
https://www.python.org/downloads/

I have installed python 2.6.6

I installed 2.7.9. Closed the command window, deleted the node_modules directory, and did npm install again. Then it just worked.

thanks to oliver-moran
I   excute " yum -y update " , then  did npm install  again. then it worked .

Could add it to the docs that Python is required (and the min version, if known).

Thanks, that's probably a good idea

For anyone who may be trying to install Prerender on a server that has multiple versions of Python installed (such as a WebFaction server) and is getting a version related error when node-gyp rebuild tries to run because the default Python version is wrong, try this:
$ npm config set python /path/to/executable/python2.7

Thanks @graywithanA !


Floating Input with right Icon hangs Simulator on iOS and Android

I have gone through these following points

[]   Check latest documentation: https://docs.nativebase.io/
[]   Check for existing open/closed issues for a possible duplicate before creating a new issue: https://github.com/GeekyAnts/NativeBase/issues
[]   Use the latest NativeBase release: https://github.com/GeekyAnts/NativeBase/releases
[]   Check examples from NativeBase KitchenSink https://github.com/GeekyAnts/NativeBase-KitchenSink
[]   For discussion purpose make use of NativeBase Slack: http://slack.nativebase.io/
[]   For queries related to theme, check Theme Variables from Docs and live NativeBase Theme Editor http://nativebase.io/customizer/

Issue Description
I'm trying to use Floating Input with right icons but it hangs the simulator on iOS as well as on Android.
react-native - v0.60.5,
react - v16.8.6
native-base - v2.13.4
xCode - v10.3.1
Android Studio - v3.4.1
react-native-cli - v2.0.1
Expected behaviour
It should show icon on right.
Actual behaviour
Simulator hangs
Steps to reproduce
import React, { Component } from 'react';
import { Container, Header, Content, Item, Input, Icon } from 'native-base';

export default class SuccessInputTextboxExample extends Component {
  render() {
    return (
      <Container>
        <Header />
        <Content>
          <Item floatingLabel success>
            <Input placeholder='Textbox with Success Input' />
            <Icon name='checkmark-circle' />
          </Item>
        </Content>
      </Container>
    );
  }
}

Thanks

Please try with the latest version, v2.12.5.

@PlabanJr
v2.13.5 doesn't have this issue.
Thanks.


How to get access to raw JSON of response? [question]

Currently I'm using generated Kotlin classes to access a GraphQL API.  At one point, I need to get access to the response type in JSON form.  If I used Jackson to serialize it to JSON, it usually works just fine, unless I'm using fragments.  If I'm using fragments, I get something similar to the following:
{
        "__typename": "Channel",
        "fragments": {
            "commonChannelInfo": {
                "__typename": "Channel",
                "id": "96895296-a3ec-45c3-bc85-2ff229cb5fde",
                "name": "#lips",
                "description": "Crowd favorites sealed with a kiss 💋",
                "facets": [
                    "Makeup"
                ],
                "subscription_status": false,
                "answered_users": [
                    {
                        "__typename": "User",
                        "avatar": "329b29b8-cae9-4c0d-9f01-3c36ad3b4fe2"
                    },
                    {
                        "__typename": "User",
                        "avatar": "329b29b8-cae9-4c0d-9f01-3c36ad3b4fe2"
                    },
                    {
                        "__typename": "User",
                        "avatar": "329b29b8-cae9-4c0d-9f01-3c36ad3b4fe2"
                    },
                    {
                        "__typename": "User",
                        "avatar": "329b29b8-cae9-4c0d-9f01-3c36ad3b4fe2"
                    }
                ],
                "stats": {
                    "__typename": "ChannelStats",
                    "question": 647,
                    "answer": 1525,
                    "subscription": 610
                }
            }
        }
    }

If I were using "normal" GraphQL, the fragments property wouldn't be there.  Is there a way to get access to what the GraphQL is responding with directly?  Or alternatively, is there a known workaround?
Thanks!

If you really need, you can access the raw json from an okhttp interceptor. But apollo-android should give you access to all the data from your queries using a typesafe API, which is usually better than using Jackson or something else to parse json. Is there a specific use case where you need access to the json from some data that is not in your query ?
See also https://github.com/apollographql/apollo-android/issues/1771

@martinbonnin Thanks for the tip, I'll give that a try.  It's not about the data per se, it's about the shape of the data, I have a need to have it represented as JSON (in the same way that it gets returned by GraphQL if you were to access it directly).  Other times it's useful to have it in the type safe API, because I need to apply some kind of transofm
So it looks like I can access the raw response via the interceptor...do you know if I can pass it along to the final result somehow?  Or do I only have access to it within the context of the interceptor?

do you know if I can pass it along to the final result somehow? Or do I only have access to it within the context of the interceptor?

It's mostly available from the context of the interceptor. If you need it somewhere else, you'll have to add support for that manually.


Add ELBv2 config file support

Add support to filter ELBv2 by name in config file.
Test output:
https://gist.github.com/brandonstrohmeyer/f29c2e48a09214d821c79a1293a99647

LGTM as well! I'll kick off a sanity check regression build now and if that passes, we can merge this in! Thanks for your contribution!

Ah looks like there is a conflict after merging in dynamodb :(
I also see a potential conflict wiht the ecs branch as well. Perhaps you should combine the two PRs together so we can merge it in one go?

Opened #266 as a combined PR to resolve merge conflicts.


support for new api objects in external-storage

Recently new objects were added to api, example SecretReference or RBDPersistentVolumeSource etc.. These objects are needed for RBD and CephFS provisioners.  When I bump up the apimachinery version to make these new objects available, the compile process fails with large number of errors.  I would appreciate any guidance here.

@wongma7

Let’s merge https://github.com/kubernetes-incubator/external-storage/pull/392 first, it may make things easier for you

I’m now continuing work in https://github.com/kubernetes-incubator/external-storage/pull/419
K8s repos have tags now so it will be much easier to avoid the “bump and all hell breaks loose” situation by ensuring all the repos have the same tag,


Add tests for gateway in mint

As discussed  in https://github.com/minio/minio/pull/4384 , need to design tests which are specific for gateways with different backends, ex: mint --gateway azure ENDPOINT.

List of GCS related tests to be added, maintaining it here temporarily,

https://github.com/minio/minio/issues/4397
https://github.com/minio/minio/issues/4400
https://github.com/minio/minio/issues/4401
https://github.com/minio/minio/issues/4395
https://github.com/minio/minio/issues/4380

ETA:2017-06-26 12:00

closing this for now as we don't have enough cycles to do this


feat(integration): ai ppt generator agent

Proposed Changes
created a ai ppt generator agent for fetch ai hackathon iitg  by team codex that lets you create agent by just putting topic description and slides number
Linked Issues
[if applicable, add links to issues resolved by this PR]
Types of changes
What type of change does this pull request make (put an x in the boxes that apply)?

[ ] Bug fix (non-breaking change that fixes an issue).
[ ] New feature added (non-breaking change that adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to stop working as expected).
[ ] Documentation update.
[ ] Something else (e.g., tests, scripts, example, deployment, infrastructure).

Checklist
Put an x in the boxes that apply:

[x] I have read the CONTRIBUTING guide
[x] Checks and tests pass locally

If applicable

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that code coverage does not decrease
[ ] I have added/updated the documentation

Further comments
[if this is a relatively large or complex change, kick off a discussion by explaining why you chose the solution you did, what alternatives you considered, etc...]

Closed as this submission does not meet the submission criteria


Docs on runtime requirements

Nice work Machha!
Wanted to know if it works on edge runtimes? Specifically if there's any deps on node API's? Docs on it would be great!

Hi

Nice work Machha!

Thank you 😄

Wanted to know if it works on edge runtimes? Specifically if there's any deps on node API's? Docs on it would be great!

I think it should work with edge runtime. There is no extra dependency on any other package and uses simple fetch under the hood.


plan mission/stage key resources section

to make sure people can find the source code and refer to the thing we're planning to build, a way to display links to the demo/repo would be helpful

In the v2 kick-off we agreed that each mission should have a key resources section.
@jlengstorf original note:

keep links to demo/source prominently listed throughout missions/stages

should we have this be something we add as a sanity input for each stage and/or the mission home page?

In the v2 kick-off we agreed that each mission should have a key resources section.
@jlengstorf original note:

keep links to demo/source prominently listed throughout missions/stages

should we have this be something we add as a sanity input for each stage and/or the mission home page?


fix: apply default gas limit estimations

Logic for updating gas limit if estimation < action default is only applied in generateTxCallback. This adds the correct action to all V2 and V3 methods
Developer Notes:

Skipped updates L2Pool estimates (v3-pool-rollup), need to test to make sure limits are correct
In Pool helper (v3-pool-contract), deposit still exists for backwards-compatibility but ProtocolAction.supply is used for gas estimation ( ProtocolAction.deposit is only used in V2)

Link T-4437


"Enable Statusbar Buttons" config option has no affect.

Toggling the Enable Statusbar Buttons option has no affect. The status bar buttons remain present no matter what.
Expected behavior: Toggling the Enable Statusbar Buttons options off should remove the buttons from the statusbar.
OS: Ubuntu 17.10
Atom v1.26.1
php-debug v0.3.0

Are you talking about the "Allow PHP-Debug to be opened from the Atom status bar" setting? I've tested this on both Windows and Ubuntu and it appears to be working correctly.

I'm talking about the Enable Statusbar Buttons option as seen here:

and how that option has no effect on the statusbar buttons as seen here:

I see, it looks like what's happened is the settings UI is showing settings that no longer exist after the 0.3.0 update. My suggestion would be to remove all the PHP-Debug Settings from config.cson and see if they stop appearing.

My suggestion would be to remove all the PHP-Debug Settings from config.cson and see if they stop appearing.

This worked, thanks.


Removed depreciated object embeds

Both Google and Vimeo have depreciated the use of object embeds in favor of completely self-contained iframes earlier this year.
https://developers.google.com/youtube/player_parameters?hl=en
https://developer.vimeo.com/player/embedding

Going to fix the tests as well. Closing this for now.

thx


updated documents

Updated documents in the sdk/communication/call-automation SDK

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-communication-callautomation


[NXdataViewer] Toggle points/line/line-points action does not work

Nothing happens when clicking on the icon.

For me, this is working for curves but not for scatter plots (i.e., when you give a value to each (x, y) point).
BTW, there is an issue when calling ArrayCurvePlot.setCurveData without 'ylabel' argument.

The reason why a scatter is displayed is that the mr dataset is not monotonically increasing. In fact, it is monotonically decreasing. Should we interpret a monotonically decreasing axis as a curve too?.

Yes. Always decreasing or always increasing can be nicely represented by a line :-)

Sounds right.
Nexus does not provide a description of whether it is a curve or a scatter plot?

Not as far as I know. I always allow the user to show the lines. If that makes sense or not is not my problem then :-)


fix: Android 14 bug fixed (One of RECEIVER_EXPORTED or RECEIVER_NOT_E…

com.android.billingclient:billing-ktx:6.0.1 makes apps using react-native-iap  crash when run on Android 14:
One of RECEIVER_EXPORTED or RECEIVER_NOT_EXPORTED should be specified when a receiver isn't being registered exclusively for system broadcasts

com.android.billingclient:billing-ktx:6.1.0 is safe for use.

@hyochan , Thanks!


[Hexagon]: Add upload function to hexagon session

This PR adds upload to hexagon RPC server running on Hexagon side. This will remove the need of adb command for uploading/executing binary files on hexagon.

Thank you, and the changes made address all the specific changes requested.  I think the last item remaining is how the new design would support persistent artifacts stored on the hexagon device across multiple sessions.


UUIDField with unique re-uses previous value and raises an IntegrityError

Describe the bug
Per Django's documentation snippet around UUIDField https://docs.djangoproject.com/en/3.2/ref/models/fields/#uuidfield it's acceptable to use primary_key=True, default=uuid.uuid4, editable=False. In my particular case, the UUID isn't the model primary key, but it has unique=True.
When duplicating a model with such configuration, an IntegrityError is raised.
To Reproduce
Steps to reproduce the behavior:

Create a model with uuid = models.UUIDField(default=uuid.uuid4, unique=True, editable=False)
Try to duplicate it using django-clone
IntegrityError is raised, duplicate key value violates unique constraint...

Expected behavior
The field is unique, but has a default callable. It should use the value generated by the callable.
Additional context
This might be a more specific instance of #330 - I've created a new issue because I believe this field can be treated differently like it's done with models.DateTimeField, models.DateField here: https://github.com/tj-django/django-clone/blob/52e35707efe193ee15542a7019059285e3a3c495/model_clone/mixins/clone.py#L326
A few lines above when value = getattr(instance, f.attname, f.get_default()) is called, f.get_default() actually invokes uuid.uuid4 and a new UUID is generated, but getattr finds f.attname already there (the uuid), so it re-uses.
I propose another special case for UUIDField, just because Django's documentation explicitly contains a hint that might lead django-clone to fail. Something simple as
if isinstance(f, UUIDField):
    value = f.get_default()

Should suffice, but the conditional can be fancier.
Another approach would be to always identify if the default is callable, and call it, no matter the field type. While less specific, it probably does what the user intended more often, which is to execute the callable when saving the field for the first time.
Either way, I can create a PR if needed. Thoughts on either approach or perhaps another direction I should take?

@inerte Your correct this is a valid use case and a bug that is a result of using the source instance value and falling back to the default specified on the field.
This should be resolved in the latest release.
Thanks


Support for Alfresco 5.x

I updated module.properties with the following change:
from:
module.repo.version.max=4.99
to:
module.repo.version.max=5.99
And rebuild the AMP, and tested with Alfresco Enterprise 5.1.0 (with Share OAuth v2.3.0). Everything (appears to) work as expected. Can the file be updated in the source and and new version (v2.6.1?) be made available?
Thanks

Alas, I can't post a new tweet:
"An error occurred trying to post the message"

Hi Mark, it could be that something has changed on the API side as Twitter do make changes every so often, or it could be an issue with Share OAuth. I'll try and reproduce this myself with 5.1, but in the meantime did you manage to grab any log output from Share that may point to the problem?


Microsoft Store still not working

I took ownership of Windows Apps to look inside.
Then used your script to fix it.  My windows apps seem to run fine with the exception of the Microsoft Store itself.
When I try running it, nothing happens.
In Event Viewer (System), I do see "The server Microsoft.WindowsStore_22402.1401.4.0_x64__8wekyb3d8bbwe!App did not register with DCOM within the required timeout."
I tried add-appxpackage but it made no difference.
Any ideas?

try running "wsreset" in command, probably elevated for fun, wait a few minutes and restart.
Worked for me.


Add julia ffi example

Calling C functions from Julia is quite easy as well. I wrote a simple example for Julia.
https://julialang.org/.

Thank you @r9y9 !


[Patient Intake Sample] Implement multiple workflows for Questionnaire

Implement different workflows based on specific answers (e.g., additional questions for veterans).

I think this is a great idea. A few thoughts:

Can you find some example questions based off of real patient intake form? You should be able to find a PDF by googling around
You can use the Questionnaire enableWhen (https://hl7.org/fhir/R4/questionnaire-definitions.html#Questionnaire.item.enableWhen) field to add conditional questions


Performance issues in the definition of conv_2d,  src/util.py

Hello, I found that in the definition of conv_2d, src/util.py,  kernel_initializer  will be created if no initializer is given explicitly. Moreover, conv_2d   is called repeatedly in down_sampling, here. To avoid recreating rebundant nodes in tf computation graph, I think the kernel_initializer should be created before the loop in down_sampling, and pass it to  conv_2d.
Looking forward to your reply.  Btw, I am very glad to create a PR to fix it if you are too busy.

Hello, I found that in the definition of conv_2d, src/util.py, kernel_initializer will be created if no initializer is given explicitly. Moreover, conv_2d  is called repeatedly in down_sampling, here. To avoid recreating rebundant nodes in tf computation graph, I think the kernel_initializer should be created before the loop in down_sampling, and pass it to  conv_2d.
Looking forward to your reply. Btw, I am very glad to create a PR to fix it if you are too busy.

Hi @JamesCao2048 ,
Sorry for the late reply that I was too busy recently and didn't notice this issue.
For the problem that you mentioned, I think that  down_sampling part is necessary. In HRNet, there are x2, x4, x8 times down-sampling. You will need to write them separately if you don't use the while-loop. What's more, kernel_initializer in each conv_2d layer should be different and that's why I didn't create it and pass it to conv_2d.
That's my personal opinion and not sure whether I am correct.
Thanks again.


Adding GetSrvShard to the vtgate server internal API.

(this is a short term fix to an internal google issue).
@enisoc @guoliang100 @apmichaud

(I'd like to get that merged in before the cut today if possible!)

LGTM

LGTM


Portainer addon starts but dies immediatly when visiting the portainer page

Problem/Motivation
Just installed HassOS 5.11 on a Raspberry Pi 4. Installed the Portainer addon, I can start it but as soon as I visit the Portainer page the addon dies and I am asked to start it again.
Expected behavior
Portainer addon starts
Actual behavior
Portainer addon does not start
Steps to reproduce

Start portainer addon
Navigate to the Portainer menu item in the sidebar

Logs
Feb 21 16:56:19 homeassistant hassos-supervisor[881]: 21-02-21 16:56:19 INFO (SyncWorker_3) [supervisor.docker.interface] Cleaning addon_a0d7b954_portainer application
Feb 21 16:56:21 homeassistant hassos-supervisor[881]: 21-02-21 16:56:21 INFO (SyncWorker_3) [supervisor.docker.addon] Starting Docker add-on ghcr.io/hassio-addons/portainer/aarch64 with version 1.4.0

Can you please post the addon logs

I've now noticed the Log tab on the addon page. There is a clear message that says to disable protection mode to make the portainer addon work. So my fault, I should have RTFM :-)


jgaye/310 fix side panel loading state

Related issue
Solving issue #310
Changes

Display a text and spinner in the PropertyDetailSection while properties are being loaded
Hide the SidePanelControlBar while properties are being loaded

Design based on Slack conversation here.
How to test
Start the server (with backend data) and go to the localhost:3000/map page. While the map and properties are being loaded from the backend, the text and spinner will be displayed on the right hand side.
The text and spinner should be replaced by the buttons of the SidePanelControlBar and the content of the PropertyDetailSection as soon as the data is available.
NB: if you use the React Developer Tools, you can toggle the loading prop to true in the Components to re-display the loading state, as shown in the screenshot attached

This PR looks great for the loading, but I think we also want to show something better than '0 Properties' when you're zoomed out, right?
@thansidwell should we create a new ticket for that?

@brandonfcohen1 I think zooming out the and properties disappearing is a disorienting experience. And it's best solved by...

Somehow making properties visible even when zoomed out. (Ideal)
Or NOT allowing people to zoom out to the point where the properties disappear.

So yeah, I'd advocate for a ticket with either of those options.

@all-contributors please add @jgaye for code


Remove Simulator

Removes the simulator. Plugin simulation will be moved to a separate project.
From this PR onward, Matricks is intended only to run on devices that will drive LEDs.

Confirmed working on a Raspberry Pi 3B+. Had to dramatically increase swap in order to get it compile, but it will compile and work as expected if you follow the new installation instructions in the README.
Raspberry Pi 4 (or other boards with >1GB RAM) should not need to increase swap in order to compile Matricks, although it may take awhile to compile. A note was added to the README suggesting cross-compilation on devices with memory issues, and an issue (#15) was created to add instructions for cross-compilation to the README.


bugfix: Modify docker run error after agent folder move

Relates to:
Docker running error
Risks
Low, The scope of influence is the operation of docker
Background
What does this PR do?
After the author moved the agent folder, docker could not map the agent directory in docker, so docker volumn mapping incorrectly. I re-mapped the agent directory
docker has the following error:
sudo docker compose -f docker-compose-waifu.yaml up
[+] Running 1/0
✔ Container waifu  Created                                                                                0.0 s
Attaching to waifu
waifu  |
waifu  | >  eliza@ start /app
waifu  | >  pnpm --filter "@ai16z/agent" start --isRoot "--characters=characters/waifu.character.json"
waifu  |
waifu  | No projects matched the filters in "/app"

What kind of change is this?
Bug fixes (non-breaking change which fixes an issue)
Documentation changes needed?
My changes do not require a change to the project documentation.
Testing
Where should a reviewer start?
Carry out
. ./scripts/docker.sh build
. ./scripts/docker.sh run

Try to see if the docker log is error free

LGTM - thanks so much for fixing this

This is what I should do, hope that the open source project will get better and better


update ReadMe version

Please update the ReadMe to reflect the version number change you made in the last commit

Done


HSEARCH-2700 Decide how simpleQueryString() should behave with null or empty input

https://hibernate.atlassian.net//browse/HSEARCH-2700

Jenkins, retest this please

merged!


Fixes Issue 467

Fixes #467
Problem Summary

A payment model is created when hitting the Add Payment button on invoices.
This model is invalid, but it's creation sets the paidTotal to 0.
This set's status to 'Paid' for invoices requiring 0 payment.

Fix Summary

request adds additional logic to where the status field is set

keeps track of last unpaid status if it needs to return to it
(e.g. return to either Bill or Draft)

fixes missing checks for invalid payments when calculating remaining
balance
implements cleanup function to destroy payment model placeholder
upon modal exit with invalid model state.

@kaungst Thanks for the PR.  I made some comments in your code changes.

@kaungst, hi. Are you planning to address this PR further or should we close it?

It looks like this issue has been addressed in pr #654. @jkleinsc can you confirm and close if so?

@tangollama yup.  Closing this one.


Bump :ui doom

hlissner/emacs-doom-themes@7307dc6 -> hlissner/emacs-doom-themes@294a432

Thanks @hlissner  I made the change.

I meant in the commit message, not the title of the PR.

Sorry @hlissner I don't know how to change the commit title. I can erase it and do it again.

That's fine. I've bumped the module in cd3c865. Thanks for bringing it to my attention!

You're welcome. I'll try to be more careful in the future.


Experimental localStorage not working with UAC

When UAC is enabled (at the highest level) the experimental localStorage method (possibly also the default localStorage method) is not working.
That's fine, I guess, but I can't seem to find an error in the log anywhere?

Have you considered using this package for localstorage?
https://www.npmjs.com/package/electron-json-storage

I've created a Module for that package:
`/* eslint-disable no-unused-vars */
import moduleJson from './module.json';
import electronStorage from 'electron-json-storage';
/**

Example module.

@param {Object} log         - Winston logger instance

@param {Object} skeletonApp - reference to the skeleton app instance

@param {Object} appSettings - settings.json contents

@param {Object} eventsBus   - event emitter for listening or emitting events

                          shared across skeleton app and every module/plugin

@param {Object} modules     - references to all loaded modules

@param {Object} settings    - module settings

@param {Object} Module      - reference to the Module class

@constructor
/
export default class NodeSocket {
constructor({ log, skeletonApp, appSettings, eventsBus, modules, settings, Module }) {
/*

You can delete unused vars from the param destructuring.
Left them here just to emphasize what is passed. Delete the eslint rule at the top
when done.
You can also just have a one config param and do Object.assign(this, config);
*/
this.module = new Module(moduleJson.name);

// Get the automatically predefined logger instance.
this.log = log;
this.eventsBus = eventsBus;
// Never do time consuming or blocking things directly in the constructor.
// Instead hook to 'beforeDesktopJsLoad, desktopLoadedorafterInitialization` events.
// This will also ensure plugins providing things like splash screens will be able
// to start as quickly as possible.
this.eventsBus.on('desktopLoaded', () => {
this.init();
});
}

init() {
// Do some initialization if necessary.
this.registerApi();
// Lets inform that the module has finished loading.
this.eventsBus.emit(`${moduleJson.name}.loaded`);

}
// Usage
// Desktop.fetch(moduleName, eventName, timeout, ...args).then().catch();
// Desktop.fetch('JSONStorage', 'getAll').then((data) => { console.log(data) }).catch(() => { console.log("FAIL"); });
registerApi() {
this.module.on('getAll', (event, fetchId) => {
electronStorage.get('clientSettings', (error, value) => {
if (error) {
this.module.respond('getAll', fetchId, error);
}
else {
this.module.respond('getAll', fetchId, value);
}
});
});
this.module.on('setItem', (event, fetchId, key, value) => {
  electronStorage.set(key, value, (error) => {
    this.module.respond('setItem', fetchId, error);
  });
});

}
}
`

@wojtkowiak if you want I can extend try to extend this module to become a plugin (also replacing Meteor._localStorage)?

@KoenLav I'm up for testing whatever you implement on a barebones todo-list app on windows, mac, and linux.

@matthova I decided to use another implementation based on electron-settings (it seems to be better for the useCase of storing device settings (which is what we are using this for).
https://pastebin.com/cqVsZiqq
Usage instructions in the comments above the methods.
@wojtkowiak is there an easier way to expose these functionalities from the main process to the webview? Or is using Desktop.fetch the most simple way imaginable.

Hey, sorry for not responding. I had a slight offline period recently ;)

@wojtkowiak is there an easier way to expose these functionalities from the main process to the webview? Or is using Desktop.fetch the most simple way imaginable.

Yes, the Desktop.fetch intent is to ease it out. Probably there is still room to make the IPC more dev friendly by automating declaration of Desktop.on and a few other things but that is not on my radar right now.
I am interested in why it fails. It probably do not have rights to modify the files. I will check it out as I need to know that for the final mechanism for LocalStorage and IndexedDB that will come soon in the next major release.

No problem!
Is there a specific reason why you are not making use of one of the already existing packages like Electron JSON Storage or Electron Settings?

@wojtkowiak we decided to make use of Electron Settings for now.
It seems to work consistently.
For consistency I think we would prefer migrating to LocalStorage/IndexedDB (to use the same methods platform independently), however for iOS we're also making use of a separate storage driver for localforage (cordovaSQLiteDriver).


mesg: ttyname failed: Success

guest:
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu Xenial Xerus (development branch)
Release:	16.04
Codename:	xenial

host:
$ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 14.04.4 LTS
Release:	14.04
Codename:	trusty

root@zebra:~# lxc exec ldap bash
root@ldap:~# sudo -i
mesg: ttyname failed: Success
root@ldap:~#

Is this actually a bug?

Looks like the same issue as #1724
Basically the GNU C library returns an empty string on ttyname() if it can't deref /proc/self/fd/0. It however still returns true when asked if that same path is a tty causing some confusion :)
Closing as duplicate.


[Feature Request] fixed release location for Arch PKGBUILD

I would like to maintain archi in the AUR but there is warning about changing release locations. I would be easier to provide the package for Arch users if there is a fixed release location.

Please download from the downloads page.

Then every user has to do it on its own. I need a link for a script that pulls the release.

Zero effort leeching is not supported.


Ssync: connection timeout = 120s; fixed connection error handling

thanks, for the feedback.
please review the changes here: https://github.com/rancher/sparse-tools/pull/42

rebased


app支持输入框内容的清空并输入

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
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.

补充一点：支持输入框清空 DSL


need to add maven dependency to include tools.jar in ate

jdk javac in tools.jar is used to dynamically compile caserunner. but the user might only install jre which has tools.jar missing.
ate needs to find a way to install tools.jar in it's maven dependency and change caserunnergenerator code to accommodate this.

Maybe we can add tools dependency in POM:

default-tools.jar

java.vendor
Sun Microsystems Inc.

com.sun
tools
1.4.2
system
${java.home}/../lib/tools.jar

2015-02-14 2:24 GMT-04:00 peidong-hu<EMAIL_ADDRESS>
jdk javac in tools.jar is used to dynamically compile caserunner. but the
user might only install jre which has tools.jar missing.
ate needs to find a way to install tools.jar in it's maven dependency and
change caserunnergenerator code to accommodate this.
—
Reply to this email directly or view it on GitHub
https://github.com/bigtester/automation-test-engine/issues/64.

Yes. I have used the similar approach for another lib before. It should
work. But not sure if we need to or how to change caserunnergenerator yet
to work with it. (Some article says that code needs to be changed if use
maven to include the lib.)
On Feb 14, 2015 9:34 AM, "Jun Yang"<EMAIL_ADDRESS>wrote:

Maybe we can add tools dependency in POM:

default-tools.jar

java.vendor
Sun Microsystems Inc.

com.sun
tools
1.4.2
system
${java.home}/../lib/tools.jar

2015-02-14 2:24 GMT-04:00 peidong-hu<EMAIL_ADDRESS>
jdk javac in tools.jar is used to dynamically compile caserunner. but the
user might only install jre which has tools.jar missing.
ate needs to find a way to install tools.jar in it's maven dependency and
change caserunnergenerator code to accommodate this.
—
Reply to this email directly or view it on GitHub
https://github.com/bigtester/automation-test-engine/issues/64.

—
Reply to this email directly or view it on GitHub
https://github.com/bigtester/automation-test-engine/issues/64#issuecomment-74377307
.

a workaround is to check if the system has jdk installed or not, and give user proper message to install jdk if only jre found.


Implement new CULL Card: Goosebumps

1: Deal 3 damage to target 3(4) times.

From what I looked at, the isMultiDamage seems to correspond to damaging multiple enemies rather than multiple damage to the same enemy. Things like flechettes don't set it to true, but Doom and Gloom does.
Also if I try to use multiDamage for the DamageInfo it throws a type error. Happy to update if there is a better way to do this, but from what I see adding multiple attack options is typically how this is handled.
For the other updates, your wording is way better. I will make that change

Ohh, that makes sense about the meaning of isMultiDamage. Sorry for sending you on that side quest. Since this was your initial foray to helping with the project, I wanted to be sure your pull requests got some eyes on them right away, even if they're my rusty ones. It's great to see the diligence you've put in!


Save Function

Save Function Design
Scope
This design is intended to cover only the function that stores data back to DCS in a user branch. It does not cover any UI/X look and feel. It will impliment using the dcs-js package.
Context
During the edit of a Scripture text, the user may wish to save current work at any time. The app will respond to a save request by the user by doing the following:

Inputs:

Type of content: ['usfm' , 'perf']
Content to be saved
BookId to be saved
DCS user name

Actions:

If save, convert PERF to USFM (unless Epitelete does this)
Test for existence of user branch, creating if it doesn't exist
Save content to user branch

Outputs:

Console log showing save details
Boolean return indicating success (or failure)
Message with failure details (to be used in an error popup)

Decomposition of Save Function
Functions to Code
existsUserBranch
This function will test to see if a user branch exists. We need a naming convention for these branches since they will be created by the app if they don't exist.
So how about gt-{bookId}-{username}?
Parameters: bookId and username.
Returns: true or false.
Exception: on any system failures.
createUserBranch
This function will create the user branch above if it does not exist.
Parameters: bookId and username
Returns: true (if created ok) or false (if create fails; also throws exception)
Exception: on any create failure
saveUserBranch
This function saves the supplied content to the user branch.
Parameters:

bookId
username
content type ("perf" or "usfm")
content

Returns: true or false (on failure to save)
Exception: on failure to save an exception will be thrown


utils: Guard LoadLibrary against dll preloading / hijacking attacks

See https://support.microsoft.com/en-us/help/2389418/secure-loading-of-libraries-to-prevent-dll-preloading-attacks for more info

Thanks for your PRs, Mark! :)
Is there any advantage in using LOAD_LIBRARY_SEARCH_SYSTEM32 when one has to implement the alternative code path for LOAD_WITH_ALTERED_SEARCH_PATH anyway?
And does LoadLibraryExW actually have any advantage over LoadLibraryW, where I could also specify a full path to the shcore.dll file?

LOAD_LIBRARY_SEARCH_SYSTEM32

LOAD_LIBRARY_SEARCH_SYSTEM32  is also used for all static dependencies of the target dll, which are almost always specified using a relative path.
So if it is available, it is the safer approach.

@learn-more I've made the coding style consistent with the rest and added some commits. Fine to merge that?

Sure!


MarGo build failed on Window 10

After update to 18.02.17-1 MarGo can't be built on Windows 10.
| GoSublime error: MarGo build failed
|                > This is possibly a bug or miss-configuration of your environment.
|                > For more help, please file an issue with the following build output
|                > at: https://github.com/DisposaBoy/GoSublime/issues/new
|                > or alternatively, you may send an email to<EMAIL_ADDRESS>|                >
|                >
|                > cmd: `['C:\\Go\\bin\\go.exe', 'build', '-tags', 'gosublime', '-v', '-o', 'gosublime.margo_r18.02.17-1_go1.9.2.exe', 'gosublime/cmd/margo']`
|                > stdout: `
|                >
|                > `
|                > stderr: `
|                > ..\..\src\gosublime\gosublime.go:4:2: no Go files in ~\AppData\Roaming\Sublime Text 3\Packages\GoSublime\src\disposa.blue\margo
|                > `
|                > exception: `None`
|   install state: busy
| sublime.version: 3143
| sublime.channel: stable
|       about.ann: a18.02.17-1
|   about.version: r18.02.17-1
|         version: r18.02.17-1
|        platform: windows-x64
|            ~bin: ~\AppData\Roaming\Sublime Text 3\Packages\User\GoSublime\windows-x64\bin
|       margo.exe: ~bin\gosublime.margo_r18.02.17-1_go1.9.2.exe (missing)
|          go.exe: C:\Go\bin\go.exe (ok)
|      go.version: go1.9.2
|          GOROOT: C:\Go\
|          GOPATH: D:\Workspace\gocode\
|           GOBIN: (not set)
|       set.shell: []
|       env.shell:
|       shell.cmd: ['C:\\WINDOWS\\system32\\cmd.exe', '/C', '${CMD}']
|    sh.bootstrap:
|                > 	using shell env GOPATH=D:\Workspace\gocode\
|                > 	using shell env GOROOT=C:\Go\
|                > 	using shell env PATH=C:\Go\bin;D:\Workspace\gocode\bin;C:\Program Files (x86)\Common Files\Intel\Shared Libraries\redist\intel64\compiler;C:\Program Files (x86)\Intel\iCLS Client\;C:\Program Files\Intel\iCLS Client\;C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0\;C:\Program Files (x86)\Intel\Intel(R) Management Engine Components\DAL;C:\Program Files\Intel\Intel(R) Management Engine Components\DAL;C:\Program Files (x86)\Intel\Intel(R) Management Engine Components\IPT;C:\Program Files\Intel\Intel(R) Management Engine Components\IPT;C:\WINDOWS\system32;C:\WINDOWS;C:\WINDOWS\System32\Wbem;C:\WINDOWS\System32\WindowsPowerShell\v1.0\;C:\Program Files (x86)\NVIDIA Corporation\PhysX\Common;C:\Program Files (x86)\Skype\Phone\;C:\Program Files\Intel\WiFi\bin\;C:\Program Files\Common Files\Intel\WirelessCommon\;C:\Program Files (x86)\QuickTime\QTSystem\;~\AppData\Local\Microsoft\WindowsApps;~\AppData\Local\atom\bin;C:\Program Files\Microsoft VS Code\bin;C:\Program Files\Git\bin;~\bin;~\go\bin
|                > 	go version: `go1.9.2` (raw version string `go1.9.2`)
|                > 	shell bootstrap took 0.644s
|                >

I was trying to add the stub for this error but got another one:
| GoSublime error: MarGo build failed
|                > This is possibly a bug or miss-configuration of your environment.
|                > For more help, please file an issue with the following build output
|                > at: https://github.com/DisposaBoy/GoSublime/issues/new
|                > or alternatively, you may send an email to<EMAIL_ADDRESS>|                >
|                >
|                > cmd: `['C:\\Go\\bin\\go.exe', 'build', '-tags', 'gosublime', '-v', '-o', 'gosublime.margo_r18.02.17-1_go1.9.2.exe', 'gosublime/cmd/margo']`
|                > stdout: `
|                >
|                > `
|                > stderr: `
|                > disposa.blue/margo
|                > gosublime/margo
|                > net
|                > gosublime/something-borrowed/types
|                > gosublime
|                > gosublime/margo/meth/importpaths
|                > # gosublime
|                > ..\..\src\gosublime\gosublime.go:8:2: undefined: margo.Configure
|                > ..\..\src\gosublime\gosublime.go:8:26: undefined: margo.Opts
|                > crypto/x509
|                > vendor/golang_org/x/net/lex/httplex
|                > vendor/golang_org/x/net/proxy
|                > net/textproto
|                > mime/multipart
|                > crypto/tls
|                > net/http/httptrace
|                > net/http
|                > net/http/pprof
|                > net/rpc
|                > gosublime/something-borrowed/gocode
|                > `
|                > exception: `None`
|   install state: busy
| sublime.version: 3143
| sublime.channel: stable
|       about.ann: a18.02.17-1
|   about.version: r18.02.17-1
|         version: r18.02.17-1
|        platform: windows-x64
|            ~bin: ~\AppData\Roaming\Sublime Text 3\Packages\User\GoSublime\windows-x64\bin
|       margo.exe: ~bin\gosublime.margo_r18.02.17-1_go1.9.2.exe (missing)
|          go.exe: C:\Go\bin\go.exe (ok)
|      go.version: go1.9.2
|          GOROOT: C:\Go\
|          GOPATH: D:\Workspace\gocode\
|           GOBIN: (not set)
|       set.shell: []
|       env.shell:
|       shell.cmd: ['C:\\WINDOWS\\system32\\cmd.exe', '/C', '${CMD}']
|    sh.bootstrap:
|                > 	using shell env GOPATH=D:\Workspace\gocode\
|                > 	using shell env GOROOT=C:\Go\
|                > 	using shell env PATH=C:\Go\bin;D:\Workspace\gocode\bin;C:\Program Files (x86)\Common Files\Intel\Shared Libraries\redist\intel64\compiler;C:\Program Files (x86)\Intel\iCLS Client\;C:\Program Files\Intel\iCLS Client\;C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0\;C:\Program Files (x86)\Intel\Intel(R) Management Engine Components\DAL;C:\Program Files\Intel\Intel(R) Management Engine Components\DAL;C:\Program Files (x86)\Intel\Intel(R) Management Engine Components\IPT;C:\Program Files\Intel\Intel(R) Management Engine Components\IPT;C:\WINDOWS\system32;C:\WINDOWS;C:\WINDOWS\System32\Wbem;C:\WINDOWS\System32\WindowsPowerShell\v1.0\;C:\Program Files (x86)\NVIDIA Corporation\PhysX\Common;C:\Program Files (x86)\Skype\Phone\;C:\Program Files\Intel\WiFi\bin\;C:\Program Files\Common Files\Intel\WirelessCommon\;C:\Program Files (x86)\QuickTime\QTSystem\;~\AppData\Local\Microsoft\WindowsApps;~\AppData\Local\atom\bin;C:\Program Files\Microsoft VS Code\bin;C:\Program Files\Git\bin;~\bin;~\go\bin
|                > 	go version: `go1.9.2` (raw version string `go1.9.2`)
|                > 	shell bootstrap took 0.638s
|                >

I tried to reinstall GoSublime, but it didn't help. 😞

I think that error must be coming from the file $Packages\User\GoSublime\src\gosublime\gosublime.go.
Replace all package paths that have disposa.blue with gosublime.
I added a note about it about a month ago but it looks like I might've removed it too soon.

Oh, you were right. I removed $Packages\User\GoSublime and reinstall GoSublime package through Package Control, and now all works like a charm. Thank you!


Change repository link

Currently the link of the repository is https://axaguildev.github.io/
Could we change it to https://axaguildev.github.io/design-system/
Because when we are on the toolkit we want to see things related to the toolkit, not other tools.
What do you think @samuel-gomez-axa @youf-olivier @arnaudforaison ?

Totally agree


[Feature Request] Add Kodi Favorites

The movies, tvshows, musics shortcuts on the Navigation Drawer are so great.  But, the Kodi's Favorite Button (the Star Button) is so difficult to be reached by Kore Remote.
Understand that reasons for features such as TV Channel Switching, recording are strong and valid, developers may be hammering how to make a good room in the tiny screens of our mobile.  However, I strongly suggest the team should consider making Kodi Favourite Shortcut available on the  Navigation Drawer just like what they've done to movies and tvshows.
Kodi Favorites can store TV Channels with logos, and it can store a myriads of Kodi addons and programs.  It allows renaming too.  How can Kore not availing this feature in its remote function?
Please, please consider!  Imagine the degree of convenience and how pleasant it will look.

#81

Not wanting to be annoying i support this idea, why was it closed? doubled?

Yes, same as #81


Build process and output

Need to rethink the build process and output.

I had an idea for this that I would be happy to PR!
One of the thoughts I had was the use the Handlebar's templating library to make the template more dynamic. So instead of manually specifying each button, it could render every type of button using just this:
{{#each people}}
    {{this.text}}

{{/each}}

Then you could pass in an array of values that looks something like:
[
    {
        id: "github",
        link: "https://github.com/",
        alt: "GitHub Logo"
    },
    {
        id: "my_custom_website",
        link: "https://somewhere.com",
        alt: "Website Logo",
        imageLink: "https://cdn.com/img.png"
    }
]

My thought is that the express file can compile the handlebars code and output it to index.html just as it is doing now!
Thoughts?

Hey, thank you!  This issues was to address the js build step, but happy to discuss your feature here.  I like this!  I am thinking about making to change to how this works (a v2 really) and will ping you when it's ready so that you can open a PR against that!  Thank you!

Hey, thank you!  This issues was to address the js build step, but happy to discuss your feature here.  I like this!  I am thinking about making to change to how this works (a v2 really) and will ping you when it's ready so that you can open a PR against that!  Thank you!

Sounds good!
There's lots of options depending on how simple/complex you want it to be to be honest.
The same thing can be done in a similar fashion in React which I think I saw you mention?
Either way, happy to help where I can, just drop me a ping 😄


Using "Tune Model Hyperparameters" with "Train Anomaly Detection Model" modules

In the Anomaly Detection: Credit Risk sample experiment, the Tune Model Hyperparameters module is used with Train Anomaly Detection Model, however I couldn't find anything in the documentation about using both together. In what scenarios would I use one over the other? What's the difference between these three approaches?

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

ID: 08ef555f-c571-2dff-2ee2-4e2536c334b7
Version Independent ID: c97c820d-4f16-4348-e6a8-637ad9d9fc2b
Content: What is - Azure Machine Learning Studio
Content Source: articles/machine-learning/studio/what-is-ml-studio.md
Service: machine-learning
GitHub Login: @garyericson
Microsoft Alias: garye

@JeffreyCA
Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.
@garyericson @ericlicoding Hi Gary and Eric, would you please check this issue? Thanks. And also, the link of Doc-feedback under Module References repo is broken. Please check and update.

@YutongTie-MSFT please assign to @xiaoharper

@JeffreyCA @xiaoharper Update: The feedback link is fixed.

@JeffreyCA  Tune Model Hyperparameters  module and Train Anomaly Detection Model module are both training models, they are fitting for different scenarios and typically you only need to use one of them. You can read this article  for more detail.
However, the Anomaly Detection: Credit Risk sample experiment does use two training model. But the two trainer are connected to two different datasets. The Tune Model Hyperparameters   is used to optimize parameters for maximum F1 score.  Then it's connected to Train Anomaly Detection Model  to using the optimized parameters to train the model on another dataset.

Thanks for the clarification.


Bug in rs2_project_point_to_pixel() (rsutil.h)?

Required Info

SDK Version
2.34.0

Issue Description
The same code path is used for "modified Brown Conrady" and "inverse Brown Conrady" models:
static void rs2_project_point_to_pixel(...)
{
  float x = point[0] / point[2], y = point[1] / point[2];

  if ((intrin->model == RS2_DISTORTION_MODIFIED_BROWN_CONRADY) ||
      (intrin->model == RS2_DISTORTION_INVERSE_BROWN_CONRADY))
  {

      float r2  = x*x + y*y;
      float f = 1 + intrin->coeffs[0]*r2 + intrin->coeffs[1]*r2*r2 + intrin->coeffs[4]*r2*r2*r2;
      x *= f;
      y *= f;
      float dx = x + 2*intrin->coeffs[2]*x*y + intrin->coeffs[3]*(r2 + 2*x*x);
      float dy = y + 2*intrin->coeffs[3]*x*y + intrin->coeffs[2]*(r2 + 2*y*y);
      x = dx;
      y = dy;
  }

At the same time the description for the "inverse Brown Conrady" model is:
Equivalent to Brown-Conrady distortion, except undistorts image instead of distorting it
So in the case of RS2_DISTORTION_INVERSE_BROWN_CONRADY the code seems to be using an undistorion model when converting to pixel space, instead of a distortion model, which doesn't make sense.
So is this a bug or are RS2_DISTORTION_INVERSE_BROWN_CONRADY and RS2_DISTORTION_MODIFIED_BROWN_CONRADY just assumed equivalent for sensors that are projected into (e.g. RGB)?

@rafaelspring hello, the enums names sound confusing, partly as they are not meant to describe whether their usage is exclusively distort or un-distort, but to define the model selected for producing the distortion coefficients.
In both cases 3D->2D projection distorts the texture coordinates, i.e. it assumes that the 3D input is correct but the target 2D frame data is intrinsically distorted, as in the RGB sensor case. Note however, that when the 3D->2D projection is applied to undistorted data, as with Aligning Color to (undistorted) Depth/IR, then those coefficients are zeroed no distortion occurs.
In 2D->3D de-projection a reverse distortion is applied to un-distort the pixel coordinates accordingly.

@ev-mp Ok, so can I assume that rs2_project_point_to_pixel() is doing the right thing and that if I get RS2_DISTORTION_INVERSE_BROWN_CONRADY as the model for a sensor that I project into, then I can just use the coefficients and plug it into a normal distortion model?
I guess the failure case is if you have a sensor that can be used both to project "out of" as well as to project "into". Because as far as I see it, there is no consistent model that you could assign to such a sensor that would handle both projection as well as de-projection correctly.

@rafaelspring,

..then I can just use the coefficients and plug it into a normal distortion model

In case by 'normal distortion model' you refer to Brown-Conrady model equations - then probably yes.

there is no consistent model that .. would handle both projection as well as de-projection correctly

The SDK provides clear evidence to the contrary - each calibrated 2D sensor with a well-defined intrinsic (and extrinsic) matrix can and will be used to be "project onto" or "project from".
For instance, when registering Depth to Color then the Color is "projected onto", When you align in the opposite direction the the Color frame is used to "project out of". So the distortion/reverse distortion methods are invoked consistently in a symmetric manner.

@ev-mp
Uhh.. Nope. The math just doesn't work out.
Have a look in the code and try deproject(project(X)) for a nonzero point X and a nonzero Inverse-BC coefficient vector.  Are you getting x back?
For the simplest of cases, try [a 0 0 0 0] as dist. coefficients with a nonzero a.

@ev-mp Just found this page where things are explained reasonably well:
Inverse Brown-Conrady Distortion
An image is distorted, and has been calibrated according to the inverse of the Brown-Conrady Distortion model. This model provides a closed-form formula to map from distorted points to undistored points, while mapping in the other direction requires iteration or lookup tables. Therefore, images with Inverse Brown-Conrady Distortion are being undistorted when calling rs2_deproject_pixel_to_point(...). This model is used by the RealSense SR300's depth and infrared image streams.
Although it is inconvenient that projection and deprojection cannot always be applied to an image, the inconvenience is minimized by the fact that RealSense devices always support deprojection from depth images, and always support projection to color images. Therefore, it is always possible to map a depth image into a set of 3D points (a point cloud), and it is always possible to discover where a 3D object would appear on the color image.


GraphRBAC

Work in progress, creating PR to get automation and CI

Can one of the admins verify this patch?

Automation for azure-sdk-for-python

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-node

Automation for azure-sdk-for-java

Automation for azure-sdk-for-go

@AutorestCI regenerate azure-sdk-for-python

@AutorestCI regenerate azure-sdk-for-python

Automation for azure-sdk-for-js

@AutorestCI regenerate azure-sdk-for-go

@AutorestCI regenerate azure-sdk-for-go

Encountered an unknown error
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/azure_devtools/ci_tools/github_tools.py", line 33, in exception_to_github
    yield context
  File "/usr/local/lib/python3.6/dist-packages/azure_devtools/ci_tools/bot_framework.py", line 105, in manage_comment
    response = getattr(self.handler, orderstr)(webhook_data.issue, *split_text)
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/restapi/restbot.py", line 54, in regenerate
    return self.rebuild(issue, repotag)
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/restapi/restbot.py", line 36, in rebuild
    self.sdk_default_base
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/restapi/github_handler.py", line 151, in rest_pr_management
    sdk_tag=sdk_tag
  File "/usr/local/lib/python3.6/dist-packages/swaggertosdk/SwaggerToSdkNewCLI.py", line 254, in generate_sdk_from_git_object
    with manage_git_folder(gh_token, Path(temp_dir) / Path("rest"), branched_rest_api_id, pr_number=pr_number) as restapi_git_folder, \
  File "/usr/lib/python3.6/contextlib.py", line 81, in __enter__
    return next(self.gen)
  File "/usr/local/lib/python3.6/dist-packages/azure_devtools/ci_tools/github_tools.py", line 272, in manage_git_folder
    clone_to_path(gh_token, temp_dir, split_git_id[0], branch_or_commit=branch, pr_number=pr_number)
  File "/usr/local/lib/python3.6/dist-packages/azure_devtools/ci_tools/github_tools.py", line 200, in clone_to_path
    _git_clone_to_path(https_authenticated_url, folder)
  File "/usr/local/lib/python3.6/dist-packages/azure_devtools/ci_tools/git_tools.py", line 76, in clone_to_path
    repo = Repo.clone_from(https_authenticated_url, str(folder))
  File "/usr/local/lib/python3.6/dist-packages/git/repo/base.py", line 988, in clone_from
    return cls._clone(git, url, to_path, GitCmdObjectDB, progress, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/git/repo/base.py", line 939, in _clone
    finalize_process(proc, stderr=stderr)
  File "/usr/local/lib/python3.6/dist-packages/git/util.py", line 333, in finalize_process
    proc.wait(**kwargs)
  File "/usr/local/lib/python3.6/dist-packages/git/cmd.py", line 415, in wait
    raise GitCommandError(self.args, status, errstr)
git.exc.GitCommandError: Cmd('git') failed due to: exit code(128)
  cmdline: git clone -v<EMAIL_ADDRESS>/tmp/tmpywi9d2s4/rest
  stderr: 'Cloning into '/tmp/tmpywi9d2s4/rest'...
POST git-upload-pack (gzip 8727 to 4314 bytes)
fatal: write error: No space left on device
fatal: index-pack failed
'

Encountered a Subprocess error
Command: dep ensure
Finished with return code 1
and output:
The following issues were found in Gopkg.toml:

  ✗ unable to deduce repository and source type for "golang.org/x/tools": unable to read metadata: unable to fetch raw metadata: failed HTTP request to URL "http://golang.org/x/tools?go-get=1": Get http://golang.org/x/tools?go-get=1: dial tcp: i/o timeout
  ✗ unable to deduce repository and source type for "golang.org/x/crypto": unable to read metadata: unable to fetch raw metadata: failed HTTP request to URL "http://golang.org/x/crypto?go-get=1": Get http://golang.org/x/crypto?go-get=1: dial tcp: i/o timeout

ProjectRoot name validation failed


RIFE-large trained model link??

Where can I find the RIFE-large pretrained model? I can only see RIFE and RIFE-m models in the github repository.

https://drive.google.com/file/d/179f3HQ79vjQdon3O06wIHY_5DhqBWW2l/view?usp=sharing
Hope this works for you. It may have some metrics better than the paper and some worse than the paper report.
I haven't verified it with github's code, just for your reference.

Because IFRNet-Large performs better (similar PSNR and better speed) and the codebase is well organized. The main idea of IFRNet is similar to RIFE. RIFE is dedicated to proposing a real-time model (RIFE), RIFE-Large is just a side experiment.


Incorrect regex substitution (search and replace)

Substituting all instances of a character with an empty string doesn't work properly, when this character comes in multiple consecutive copies.
Here is an example, that can be reproduced in the current sandbox:

Type xxxWORDxx on a line.
Keep the cursor on the line, in normal mode.
Try to remove all x characters in the line with the command :s/x//g.

Observed: the line now contains xWORDx. The last consecutive character is not removed (for each group of x characters).
Expected: the line should contain WORD, instead (as in the original vim and as in the CodeMirror 5 online editor)

Thanks!