Cognitive Services API missing

Are the Azure Cognitive Services APIs going to be added to this repo?
I see on the API sites (eg https://dev.projectoxford.ai/docs/services/56f91f2d778daf23d8ec6739/operations/56f91f2e778daf14a499e1fa ) that a Swagger definition is available.
If I were to contribute, would I simply need to gather these misc swagger files and add them to a forked repo and then PR? Or is this repo a lot more than that?
Thanks
Ken

There already is a swagger for cognitive services, but it look quite different than the one @kpfaulkner is linking. Maybe someone in Cognitive Services team knows a bit more?
cc @daniel-ms

Hi
I thought the existing swagger was for management of the Cognitive services as opposed to actually "using" them (so to speak).
Actually nearly every swagger definition I've looked at in this repo seems to be about managing as opposed to using, or maybe I'm just looking at the wrong ones?

On 17 Nov. 2016, at 05:42, mcardosos<EMAIL_ADDRESS>wrote:
There already is a swagger for cognitive services, but it look quite different than the one @kpfaulkner is linking. Maybe someone in Cognitive Services team knows a bit more?
cc @daniel-ms
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

No, you are right. Most of this swaggers are for managing. The ones for using are the ones outside an arm folder.

In that case, doesn't that explain why the swagger is different? The existing one is for managing but from what I see each section of CSvcs (Face, Vision, Speech etc) all have their own Swagger definitions for the actual using of the APIs and not just managing?

Then my question for @daniel-ms would be more on the side of why that swagger doesn't live in this repo as a dataplane swagger.

Pinging @daniel-ms

This swagger file is for managing Cognitive Services "accounts", that is, create/update/delete accounts. For using each individual Cognitive Services API (like Face, Vision etc.), you may want to use their own SDK in GitHub. And so far as I know they don't have swagger files.
Hope this clarifies the situation and can we get this issue closed?

@felixwa
The checked-in swagger if for managing accounts, yes. About the individual APIs there is a swagger available in the link @kpfaulkner provided (there is an API definition button in the corner that includes the swagger).
We were wondering if there is a plan to merge that swagger here in this repo.

@mcardosos , I see but no, we don't have plan to merge them here. In fact those swagger files are automatically generated by Azure API Management, not maintain by us. I don't think it provides much value we put them into this repo.


feat: make jupyter a dev dependency

Description
This PR helps to make jupyter a dev depenency. To do this:

jupyter has been removed from requirements.txt.
'jupyter' has been shifted to "dev" from "install_requires" in setup.py.

Fixes #2457
Type of change
Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s. If necessary, also add to the list of breaking changes.

[x] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist:

[x] No style issues: $ pre-commit run (see CONTRIBUTING.md for how to set this up to run automatically when committing locally, in just two lines of code)
[x] All tests pass: $ python run-tests.py --all
[x] The documentation builds: $ python run-tests.py --doctest

You can run unit and doctests together at once, using $ python run-tests.py --quick.
Further checks:

[x] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works

@deepeshaburse any progress with this?

Being worked upon in #2846


Need to be able to extract a thumbnail of recorded videos to display in UI

There is currently no easy way to get a thumbnail for a recorded video.
It is currently only possible by creating a custom platform-specific service which then uses the AvAssetImageGenerator for iOS or the MediaMetadataRetriever on Android to generate a Jpeg image to be used as a thumbnail.
It would be nice if this was built into the MediaPlugin

FYI I'm working on a PR for this.


Node Clustering

Add the ability to group/cluser nodes based on a data attribute. This will likely be isolated to only some layouts.

https://bl.ocks.org/mbostock/7882658
https://bl.ocks.org/mbostock/7881887
https://github.com/ericsoco/d3-force-cluster/blob/master/lib/forceCluster.js

WIP: https://github.com/reaviz/reagraph/tree/clusters

Reference: https://github.com/vasturiano/d3-force-cluster-3d/issues/4

A first cut of this was checked in, still a few TODOs.


New feature request - workflow diagnostics

We are running into a situation where task execution occasionally stops. When this happens, would be great to run a diagnostic check and report information on a workflow, including number or registered executors and their state to help isolate the root cause.

Please test the code I just pushed. Have a look at the test class WorkflowManagerStateSampler for an example of a sampling state checker. You can copy that class or write something based on it.

Will the newly added 'WorkflowAdmin.getWorkflowManagerState' work with prior versions? Or, is the state a new attribute requiring the scheduling and execution to also use the updated code?

Yeah it works with old versions. It only analyzes the manager it's called from.
====================
Jordan Zimmerman

On Apr 1, 2016, at 1:56 PM, Jim Bugwadia<EMAIL_ADDRESS>wrote:
Will the newly added 'WorkflowAdmin.getWorkflowManagerState' work with prior versions? Or, is the state a new attribute requiring the scheduling and execution to also use the updated code?
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub

Thanks! This works and I am able to retrieve the state. Here is what I got back:
WorkflowManagerState{isConnectedToZooKeeper=true, schedulerState=LATENT, consumersState=[]}
Can you please explain the schedule and consumer states?

That means that that manager is not the scheduler. More importantly it means it has no task executors. Any idea why?
====================
Jordan Zimmerman

On Apr 1, 2016, at 2:41 PM, Jim Bugwadia<EMAIL_ADDRESS>wrote:
Thanks! This works and I am able to retrieve the state. Here is what I got back:
WorkflowManagerState{isConnectedToZooKeeper=true, schedulerState=LATENT, consumersState=[]}
Can you please explain the schedule and consumer states?
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub

I connected a new WorkflowManager, from a test program solely to query the
state of the workflow.
Is there a way we can print the state of all other WorkflowManager
instances from this?
Thanks,
Jim
On Fri, Apr 1, 2016 at 1:04 PM, Jordan Zimmerman<EMAIL_ADDRESS>wrote:

That means that that manager is not the scheduler. More importantly it
means it has no task executors. Any idea why?
====================
Jordan Zimmerman

On Apr 1, 2016, at 2:41 PM, Jim Bugwadia<EMAIL_ADDRESS>wrote:
Thanks! This works and I am able to retrieve the state. Here is what I
got back:
WorkflowManagerState{isConnectedToZooKeeper=true, schedulerState=LATENT,
consumersState=[]}
Can you please explain the schedule and consumer states?

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/NirmataOSS/workflow/issues/25#issuecomment-204547652

No. That would require major work
====================
Jordan Zimmerman

On Apr 1, 2016, at 3:29 PM, Jim Bugwadia<EMAIL_ADDRESS>wrote:
I connected a new WorkflowManager, from a test program solely to query the
state of the workflow.
Is there a way we can print the state of all other WorkflowManager
instances from this?
Thanks,
Jim
On Fri, Apr 1, 2016 at 1:04 PM, Jordan Zimmerman<EMAIL_ADDRESS>wrote:

That means that that manager is not the scheduler. More importantly it
means it has no task executors. Any idea why?
====================
Jordan Zimmerman

On Apr 1, 2016, at 2:41 PM, Jim Bugwadia<EMAIL_ADDRESS>wrote:
Thanks! This works and I am able to retrieve the state. Here is what I
got back:
WorkflowManagerState{isConnectedToZooKeeper=true, schedulerState=LATENT,
consumersState=[]}
Can you please explain the schedule and consumer states?

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/NirmataOSS/workflow/issues/25#issuecomment-204547652

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub

OK if I close this?

Yes, thanks!
On Tue, May 10, 2016 at 9:45 AM, Jordan Zimmerman<EMAIL_ADDRESS>wrote:

OK if I close this?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/NirmataOSS/workflow/issues/25#issuecomment-218218301


stem in title not parsed

Title of asciidoc:
=== stem:[Set] product

downdoc output markdown title:
### stem:[Set] product

Only attributes are expanded in section titles and discrete headings. No other substitutions/translations are applied. So in order for this to work, substitutions would need to be applied to the heading text.

adoc:
···
[subs="normal+"]
== stem:[a/b]
···
md:
···
stem:[a/b]
···
seems no effect in heading...

substitutions would need to be applied to the heading text.

I meant they need to be applied in the downdoc code, not in the source AsciiDoc.


[Android] Add to F-Droid

Similarly to https://github.com/spesmilo/electrum/issues/1700 and https://gitlab.com/fdroid/rfp/issues/188 , there is also a request for packaging this fork for Dash of Electrum for F-Droid, see https://gitlab.com/fdroid/rfp/issues/817

Hi, the current Electrum Dash fork is maintained here: https://github.com/akhavr/electrum-dash
Please use that repo and report there if you have any issues.


Issue with create_network_from_weights

Right now, the args loaded from sdxl_train_network.py uses .lora_network_weights variable, but in train_network.py, it tries to read .network_weights instead, and thus throwing NoneType error.

The links do point to the specific lines


Replacing omahaproxy.appspot due to deprecation

Thanks @rany2.

The only consideration might be how we select the fraction. I don't have a clue how that could change in the future, but it's possible that ratio might vary more significantly. We'll just have to see what happens.
The results can be ordered, so if that ends up being a problem we could maybe add &order_by=fraction%20desc to the URL and select the first entry: https://versionhistory.googleapis.com/v1/chrome/platforms/win/channels/stable/versions/all/releases?filter=endtime=none&order_by=fraction desc

We all know by now that Google plays on its own rules, so all we can do is just observe and adapt :)

On a side note, is there any reason for having the check run every six hours? Would it cause problems to have it run more frequently?

The value was taken randomly IIRC. I was thinking that if we are to be notified of a new release with the maximum delay of 6 hours, that would be acceptable. Also I had no idea which rate limits are there on omahaproxy, so this value was taken almost randomly. We can try increasing the frequency of the check. Another limiting factor I can think of could be GitHub rate-limiting since there is some logic in issue-creation that pulls GitHub API. Every hour then maybe?

The value was taken randomly IIRC

The value was not taken randomly. According to Google's documentation, fraction is the ratio of users who were pushed the update. So if it's 0.9 then 90% of users on the stable channel were sent the update by Google which I think is a good enough metric for it to be "stable."

@rany2 The value I was referring to was 6 hours interval for a check, not the fraction.

@PF4Public Ah sorry, I wasn't following very well :sweat_smile:

Would it cause problems to have it run more frequently?

IMO an interval of one hour isn't too abusive.

P.S. I liked @Ahrotahn's proposal. Not sure if it's too late to switch this to: curl -so- 'https://versionhistory.googleapis.com/v1/chrome/platforms/win/channels/stable/versions/all/releases?filter=endtime=none&order_by=fraction%20desc'  | jq -rc '.releases | first | .version'

@rany2 Why not :) It does not matter, who is filtering :)

I wonder if there is a desire to make all commits signed? @networkException Was last merged commit a hint or just a random trial?

I mainly wanted to see if and how it is possible with our workflow. Improved verifiability is definitely something I'd like to advance in the project though

Something I'm quite a fan of is the new ssh-based commit signing. Previously I wasn't too invested in signing git commits because I hate GPG but having it use ssh keys instead is very convenient!

If I recall correctly, using the squash and merge option in the github web interface will have the commit signed by github's key.

If I recall correctly, using the squash and merge option in the github web interface will have the commit signed by github's key.

… partially signed :)

I didn't expect the mac releases to be rolled out so slowly.  Would it be a good idea to order by the version number instead?

order by the version number instead

What do you mean?

He means like this:
curl -s 'https://versionhistory.googleapis.com/v1/chrome/platforms/win/channels/stable/versions/all/releases?filter=endtime=none'  | jq -rc '.releases | .[] | .version' | sort -Vr | head -1

Will it bring betas in the list?

No

I could do it later today.

Just keep in mind -V is a GNU extra, on FreeBSD it behaves differently

Could jq do the sorting then?

Sorry I should have mentioned that the api can do it!  All you would have to do is change order_by=fraction%20desc to order_by=version%20desc in the URL.

Sorry I should have mentioned that the api can do it! All you would have to do is change order_by=fraction%20desc to order_by=version%20desc in the URL.

Are you sure it's natural sort?

Answering my own question: yes, it is. @PF4Public Just replace fraction with version and it works just fine!


Application translation Russian and Ukrainian

Please add translation.
I translated it into Russian and Ukrainian languages.
AntiSplit-M.v1.6.3.zip

Thanks I have added it


Serialization interface

[ ] Serialization interface with pure virtual Load and Store functions
[ ] Serialization paths (conversion from schema.inter -> code) must be well segregated from the target module

#192 Serialization Interface


Add memory swap @open sesame 7/26 15:55

Initial implementaion of cache pool.
It inherites from memory pool to utilize optimized memory information.
And, swap device class is introduce. It operates with cache memory and keeps data permantly in the device.
The storage size is fixed when the device opens, sparse file is created with garbage data. And, it's removed when the device close,
Signed-off-by: Jiho Chu<EMAIL_ADDRESS>Issue: #1966
Self evaluation:

Build test:	 [X]Passed [ ]Failed [ ]Skipped
Run test:	 [X]Passed [ ]Failed [ ]Skipped

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1965. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1965-202207192125160.92761492729187-b386e4b2f97ac4a112589d6245968d0cf35700f8/.

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1965-202207201120020.43601894378662-b4ea4e84217e4bbf431711e0d1a2b273980f92ad/.

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1965-202207201719290.063426971435547-cf94acb476bf155644303a1af57d7b8c228e813d/.

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1965-202207261555000.47095704078674-cf94acb476bf155644303a1af57d7b8c228e813d/.

Please add cache_pool.cpp and swap_device.cpp at jni/Android.mk to resolve android ci issue

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1965-202208101152440.26695609092712-38f89adb6157e1030057989d36329091331ec8de/.

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1965-202208171601300.87476491928101-c538bd60f22bce369b0ffce39de31f80b5e72189/.

:octocat: cibot: @jihochu, test/unittest/unittest_nntrainer_profiler.cpp does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

It depends on #1987 . Please ignore build error before merging it.

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1965-202208251456040.096474885940552-7e2aad1867e37a8704492116ca38d424e73ba0ae/.

:octocat: cibot: @jihochu, test/unittest/unittest_nntrainer_profiler.cpp does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

:octocat: cibot: @jihochu, test/unittest/unittest_nntrainer_profiler.cpp does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

:octocat: cibot: @jihochu, test/unittest/unittest_nntrainer_profiler.cpp does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1965-202208251714590.60714101791382-2c739b9c351f9c5b77776771783f246a9da683e7/.

Added "RELEASE CRITICAL": As long as this is an experimental feature that is turned off by default and can be turned on with users' explicit configuration, this is not going to affect the behaviors.
For further testing and verification, let's merge this as long as it does not affect the behaviors with the default configuration.

:octocat: cibot: @jihochu, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1965-202210061349420.022874116897583-597e51058bfe582d4e4529b66d974cc244584cb1/.


fix(state): Compare subtrees in non-finalized chains when evaluating chain equality

Motivation
Close #7447.
Solution

Compare the subtrees in eq_internal_state.
This fix turned out to be trivial since none of our tests create subtrees in non-finalized chains.
I thought of moving the chain comparison to Eq, but we already use Eq for another purpose, so I left the comparison as it is.

Reviewer Checklist

[ ] Will the PR name make sense to users?

[ ] Does it need extra CHANGELOG info? (new features, breaking changes, large changes)

[ ] Are the PR labels correct?
[ ] Does the code do what the ticket and PR says?

[ ] Does it change concurrent code, unsafe code, or consensus rules?

[ ] How do you know it works? Does it have tests?

I think the ideal fix would be to put the fields in eq_internal_state() in an inner struct, where we derive PartialEq. Then we could just call inner.eq() from there.

This seems quick and beneficial because we won't forget new fields as we already did, so I'm going to do that.


e2e: rename test files for clarity

rename files something.go to something_test.go
to emphasize they actually hold test code.
No changes in behaviour.
Signed-off-by: Francesco Romani<EMAIL_ADDRESS>

/lgtm
/approve


heroku deploy

Try the following:

[ ] push docker image with front (https://devcenter.heroku.com/categories/deploying-with-docker)
[ ] try create database (https://devcenter.heroku.com/articles/heroku-postgresql)
[ ] try roll up migrations via tl-migrate
[ ] try deploy server service and connect it with front
[ ] research "deploy with heroku button" (https://devcenter.heroku.com/articles/heroku-button)

Well... That was interesting idea. But heroku is not as simple as I think. So, if I decided to migrate task-list from home server to any cloud, I get VPS on DigitalOcean or similar.


Add more interesting windows to the example

Codecov Report

Merging #26 (7c2dd5a) into main (6546a9e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #26   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         5
  Lines          121       121
  Branches        16        16
=========================================
  Hits           121       121

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 6546a9e...7c2dd5a. Read the comment docs.


Is there a way to disable dynamic backgrounds?

I would prefer just [one] or [no] background image instead of it changing with the cover.
Also is there a way to make it less laggy while scrolling through playlists?
The theme is absolutely beautiful.

yes i will add feature to make the background solid color
and scrolling through the playlist consumes cpu thats why its laggy its not the problem of the theme i think its something to do from spotify side
checkout #26
https://github.com/sanoojes/better-bloom/issues/26#issuecomment-2149698163

That would be amazing thanks!
Noted @ scrolling. That's unfortunate.

check the update ! you can now select between background styles


Ceedling Fails to Compile If Module Is Named "Exception"

Ceedling ignores #include "Exception.h" in test file and does not compile the module.
I use Ceedling version 0.30.0 with :use_exceptions: TRUE.  Ceedling correctly compiles CException, however.
The following way of includes will fail to compile Exception module:
#include <stdio.h>
#include "Exception.h"
#include "CException.h"

However, it will compile if arranged this way:
#include "Exception.h"
#include <stdio.h>
#include "CException.h"

I can't reproduce this issue on Ceedling 0.30.0 and for me module is compiled fine, but my Exception module is really simple. What kind of error do you have and what your Exception module contains? Do you use Windows or Linux?

ceedling version

   Ceedling:: 0.30.0
      Unity:: 2.5.1
      CMock:: 2.5.2
 CException:: 1.3.2

My files:

src/Exception.c

#include "Exception.h"

int myException(void)
{
  return -1;
}

src/Exception.h

#ifndef EXCEPTION_H
#define EXCEPTION_H

int myException(void);

#endif // EXCEPTION_H

test/test_Exception.c

#include "unity.h"
#include <stdio.h>
#include "Exception.h"
#include "CException.h"

void setUp(void)
{
}

void tearDown(void)
{
}

void test_myExceptionReturnMinusOneEverytime(void)
{
  TEST_ASSERT_EQUAL(-1, myException());
}

Compilation result:
Test 'test_Exception.c'
-----------------------
Generating runner for test_Exception.c...
Compiling test_Exception_runner.c...
Compiling test_Exception.c...
Compiling unity.c...
Compiling Exception.c...
Compiling CException.c...
Compiling cmock.c...
Linking test_Exception.out...
Running test_Exception.out...

--------------------
OVERALL TEST SUMMARY
--------------------
TESTED:  1
PASSED:  1
FAILED:  0
IGNORED: 0

Sorry for the delay. I forgot to follow on this issue. There was typo in my earlier post. I meant to write:
#include <stdio.h>
#include "CException.h"
#include "Exception.h"

The #include "Exception.h" comes after #include "CException.h" to reproduce the bug.
I run under Windows 10. I have the same versions for Unity, CMock, CException as you do.

After I changed headers sequence I didn't notice this even under Windows 10. Have you tried my example to confirm the problem? Maybe this issue is related to the headers dependency in your project and not to the ceedling. What kind of error do you have? Can you show the output log?


Update uploadable.md

minor fixes in code example
defined $name property, return $path property instead of undefined var

thanks ;)


Make R2 optional

In some cases, we may just have R1

quite true, esp. long reads

See #13, lots of goodies there.

Fixed in #13


Investigate isSettled implementation

Emberjs implements a method isSettled and that checks some metrics to defined a "settled" state (page loaded, no ajax requests pending, no events etc...
Investigate if implementing a similar utility in wendigo is possible and useful
https://github.com/emberjs/ember-test-helpers/blob/master/API.md#settled

https://github.com/jtassin/pending-xhr-puppeteer


add universidad de aysen

Domain of emails for university students and teachers. (Univeridad de Aysén)
Here is the institution's website:
https://www.uaysen.cl

@cvargas-uaysen First stage of verification passed, official domain is uaysen.cl as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@cvargas-uaysen Pull request merged. Thank you.


Support Delaying when max retries was exceeded

It's a very popular scenario when max retries is exceeded and user should be waiting, for example: 5min, before next retry. didMaxRetries is quite complex to handle that scenario. it's will be great if you can support that scenario out of the box, I appreciate that.

@bigbang489
Thanks for using my package!
If the number of retries is exceeded, would you prefer a method that disables the operation and counts down to unlock?

If the max retries (e.g. 5) is exceeded, I would like to force user to wait, for example 5min, after the user can retry another 5 times, and wait 5min, and so on.. This is to eliminate the ability to retry to many times to find the passcode, for example my passcode has only 4 digits,if we don't force the user to wait, any user can find my passcode by trying a few thoundsands inputs

Added an option to temporarily disable the input when the experimental trial limit is reached.
Since we haven't published it to pub.dev yet, please edit pubspec.yaml as follows.
dependencies:
  flutter_screen_lock:
    git:
      url: https://github.com/naoki0719/flutter_screen_lock.git
      ref: naoki0719/issue39

Example code
screenLock<void>(
    context: context,
    correctString: '1234',
    maxRetries: 2,
    retryDelay: Duration(seconds: 3),
    delayChild: Center(
      child: Text(
        'Cannot be entered temporarily because it failed the specified number of times.',
        softWrap: true,
      ),
    ),
  );

Thank you a lot, I appreciate that


[JBWS-4106] surefire plugin update, sync with jboss-parent-pom

https://issues.jboss.org/browse/JBWS-4106
surefire plugin update (JDK 10/11 needs this), sync with jboss-parent-pom

FYI - https://github.com/rsvoboda/jbossws-cxf/commit/d45d7c51152f60a75dc4fc4341f4d6a2b33f158f this is the WIP work for Java 10/11


jssor slider using angularjs - almost!

Is there a working sample of the jssor slider using angularjs for the "Image Gallery" -the sample with the thumbnails below the main image -  that I can use for a reference?   I have managed to create a HTML page with this scenario and I have it working except for these issues:

Using a six image deck, image number 4 is blank when it is its turn to appear and the styling that is applied for that pic is "display = none".
Using a seven image deck, image numbers 4 and 5 are blank when it is its turn to appear and the style for that pic is "display = none".
Using a deck of five images or less all images appear in the slider correctly.
None of the thumbnails show up on the bottom as they are supposed to however clicking on each thumbnail takes you to the correct slider picture.

Theses tests were run using jssor.slider.mini.js.  When I use jssor.slider.min.js there is a different behavior, in short, on a seven image slider deck image numbers 3, 4 and 5 appear blank.
I am not able to paste my HTML page here as the code gets all fouled up and unreadable.  And since I am not allowed to attach an HTML page to this posting I have instead attached a txt version of my HTML file for those who would like to have a look at what I have done.  It can be downloaded and entered into any IDE for perusal I assume.
Can the jssor folks please please provide some guidance?  I have been trouble shooting this for over a day so far.  Thank you for your time.
jssorTest.txt

Hope this will help you ..
https://github.com/adebisi-fa/dang-jssor

Not sure if you use jssor slider 22.1.5 or lower. If so, please try to use the latest version.

Adding slides dynamically has been implemented.
check $AppendSlide, $RemoveSlides, $ReloadSlides api.
https://github.com/jssor/slider/blob/master/examples/api-play-pause-playto-goto-next-prev-autoplay-append-remove-reload-properties-destroy.html
See https://github.com/jssor/slider/issues/217


Changing the font size for a list

Describe the bug
Changing the font size for a list does not change check box size.
Steps to reproduce
p {
font-size: 60px;
}
li {
font-size: 30px;
text-align: right;
}
The version of Memos you're using
latest
Screenshots or additional context
No response

Then can the size be changed…?


Enabled assertions in build.gradle

Codecov Report
Base: 70.78% // Head: 70.78% // No change to project coverage :thumbsup:

Coverage data is based on head (755c9b7) compared to base (cef4cdf).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #108   +/-   ##
=========================================
  Coverage     70.78%   70.78%
  Complexity      579      579
=========================================
  Files            96       96
  Lines          1814     1814
  Branches        217      217
=========================================
  Hits           1284     1284
  Misses          467      467
  Partials         63       63

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Remove QR code width check from eventRegistration.js

This PR removes the QR width check in the Cypress eventRegistration.js test. Instead only the presence of the QR code is checked.
The test has proven to be flaky and has failed several times due to changes in the QR code generation. Although the code is generated correctly, the test fails.
This change in the Cypress test makes the test less sensitive to changes in the QR code generation.
It resolves the recurring issue of the test failing with the following output:
expected '<canvas#eventqrcode.eventqr-preview>' to have attribute 'width' with the value '827', but the value was '1654'.
See also:

https://github.com/corona-warn-app/cwa-website/issues/2271

The latest failure was caused by changes made in:

PR https://github.com/corona-warn-app/cwa-website/pull/2730

@MikeMcC399 Thanks.


Fixed addGNSS_6 function prototype

Changed the firmware version format into one compatible with the https://semver.org/ org standard. Changed the prototype of the addGNSS_6 method to allow for negative values being passed in.
Signed-off-by: Sidd Gupta<EMAIL_ADDRESS>

HI @delhiryder
Thanks for the merge request.
The changes in the CayenneLPP code are a real bug-fix, in my Arduino version of the code this was already fixed.
For the version name change, I prefer to keep it as is, it makes it easier to see in the debug output what type of firmware is running.
I will fix the bug and commit the changes, but I will keep the version name. But feel free to change it in your own copy of the code.


Unsupported "!" in sql ?

JSqlParser vesion: 3.0
select * from task_info where id=1 and !(id=1 and id=2)

Excetion:
net.sf.jsqlparser.parser.ParseException: Encountered unexpected token: "!" "!"
at line 1, column 40.
Does not support this grammar? Thank you

This is not yet supported, only for primary expressions. I will look into it.

The ! variant of not was already implemented, but it was missing at some relation parts of the grammar. You could use the new version, if you compile it yourself. I could not yet deploy it.

666  Thank's


Remove a \

It throws an error

What version of PHP are you using? The backward slash is the base PHP namespace.

wow, an old one i wasn't aware of... 5.2.7 ! With Namespaces being 5.3+
I switch it to 5.4, it's all good now

Thought so :P Also 5.4 is no longer maintained and even 5.5 is only getting security fixes now. Highly recommended you switch to 5.6 for LTS.


TableColumn: Add scoped footer slot

Please make sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow Element's contributing guide (中文 | English | Español | Français).
[x] Make sure you are merging your commits to dev branch.
[x] Add some descriptions and refer relative issues for you PR.

Hey guys,
we have created a scoped slot for the TableColumn footer so that we are able change the rendering of it. For this we used the same approach as for the scoped Header slot.

Please provide specific usage scenarios.


Disallow mutually exclusive flags 611

Self explanatory
Closes https://github.com/criblio/appscope/issues/611
QA Instructions https://github.com/criblio/appscope/issues/611

Looks great!  Merging!


setup release-it with lerna-changelog integration

Replaces ember-cli-release with release-it. It integrates lerna-changelog through a plugin in release-it workflow. The setup is based on the setup script from rwjblue: https://github.com/rwjblue/create-rwjblue-release-it-setup
It does not touch docs app. That means it only partly addresses #1034.

Rebased to fix merge conflicts (yarn.lock only). Will merge after tests have passed.

Thanks @jelhan!


stream_select behaviour

In SelectLoop.php's call to stream_select the 'read' and 'write' arrays are populated such that the array keys are the integer cast of the resource id and the values are the resource itself. Usually the output from stream_select preserves the array keys and this is passed to pollManager->handle.
However I have found the array keys are not always preserved. In my testing with a single socket the key becomes 0 instead of the integer cast of the resource id. This breaks pollManager->handle, causing it to not find the socket. I cannot explain why it only happens sometimes.
The following debugging code added to SelectLoop.php's dispatch function can detect the problem in my application:
if ($count) {
/// begin insert debugging code ///
foreach ($read as $id => $resource) {
if ($id != (int) $resource) {
die("stream_select mismatch $id != $resource");
}
}
// end insert debugging code ////
}
The fix is to modify the 'handle' function in SelectIoManager.php.
Replacing:
foreach ($active as $id => $resource) {
...
With:
foreach ($active as $resource) {
$id = (int) $resource;
...

What operating system are you using? I've never seen this behavior... maybe it's something I can patch in stream_select() itself. Regardless, since the fix was so simple in Icicle I did what you suggested and cast $resource to an int instead of using the array key.

Using Windows OS.


refactor: optimize Docker build with caching (using cargo chef) + fix build issues

This PR fixes some Docker build issues and allows us to build the dependencies separated from the main application. This allows the Docker layer compiling dependencies to be cached and reused in later builds, greatly reducing the build time when there isn't any dependency changes. It uses cargo-chef (the source code is pretty simple and straightforward).
I've done the same for BP Node, LNP Node, Storm Node and Store Service. I've been building the whole stack with that this morning, it is all working well so far. As the changes for the other repos Dockerfiles are substantially identical, I'm first opening this one PR to review.

Sidenote: I only tested this against the latest releases (rc, alpha), and not against master due to frequent cargo / build issues with master.

Sidenote: I only tested this against the latest releases (rc, alpha), and not against master due to frequent cargo / build issues with master.

The recipe.json is created by the chef prepare command. The command basically goes through all the directories and subdirectories to find dependencies' Cargo.toml, in order to extract all the dependencies of the project. AFAIU the file contains all the Cargo.toml files contents.
Then, in another build stage, we import that file. If the file has changed since the last build (which means dependencies has changed), it will detect the change and rerun the chef cook command (which is basically an alias for cargo build for building dependencies). If it hasn't changed, the next steps will use the cached layer from the previous build.

You can check out the project repo for more details: https://github.com/LukeMathWalker/cargo-chef

@dr-orlovsky Opening a bunch of PR on every repo for the Dockerfiles. For the instructions, I only added the build ones (without usage), I plan to open a PR to LNPBP/nodes with full docker compose instructions (if that seems ok to you)


Move macpaw-gemini into cask-versions (1.5.17,1461238766)

[x] Checked there aren’t open pull requests for the same cask.
[x] Checked there aren’t closed issues where that cask was already refused.
[x] Checked the cask follows the requirements of acceptable casks.
[x] When naming the cask, followed the token reference.
[x] Commit message includes cask’s name.
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.

This should be called gemini-classic now
Homepage is http://macpaw.com/gemini-classic

OK - I've renamed it - and updated the home page, but I left the url/appcast stanzas the same, because that is where they actually are downloaded from.  https://dl.devmate.com/com.macpaw.site.GeminiClassic/GeminiClassic.dmg is just a shortcut to whatever the "latest" version is.


The Web UI is incomplete in offline environment

I installed reposilite in offline environment, but found that the Web UI display is incomplete. I read the relevant source code and found online ".css" and ".js" files are referenced in "index.html". This problem is solved temporarily by downloading these files and modifying "ReposiliteHttpServer.java", but I don't think it's a good solution.

Hi, the frontend was never meant to be used in fully offline mode, but I can take care of it during some of the changes related to the #138


VC Cue List step fader not in sync on existing projects

Reported in the forum.
Project attached here: http://www.qlcplus.org/forum/viewtopic.php?f=2&p=39729#p39718
Really weird.

3 fixes are possible:
https://github.com/plugz/qlcplus/commit/27349ae09efa55551b071fb359cd530027057875

use qRound instead of qFloor in slotSlider1ValueChanged. I haven't tested this with a 255-sized cuelist but it should work.
in slotCurrentStepChanged, block signals of m_slider1 before changing its value so we don't enter slotSlider1ValueChanged
do both of those

@plugz thanks.

If you have time, please feel free to fix this issue.
It's very easy to reproduce with Santiago's project.

@mcallegari
Yes, I have tested the fixes with Santiago's project, but want to test corner cases before pushing it. I'm short on time too.

Fixed by b031094263f9c4680fa56ea17d06b1b1894d1eee ?

Yes

I just compiled qlcplus 4.10.1 and the issue seems to be solved. I tested with the test workspace that I published in the forum and with an old workspace with a 28 steps cuelist changing the cuelist properties to use step fader.
Good job!! ;-)


release-metadata/2.2/releases.json does not contain 2.2.401 SDK

I'm unable to install the 2.2.401 SDK using the DevOps "UseDotNet" task. Not sure if that task is using the wrong releases-index.json (related to #3114 ?) or if this is a bug in the json files here.
The DevOps UseDotNet task gets the release info from https://raw.githubusercontent.com/dotnet/core/master/release-notes/releases-index.json.
That file contains the following block for the 2.2 channel - pointing to https://dotnetcli.blob.core.windows.net/dotnet/release-metadata/2.2/releases.json

However, that file does NOT contain the latest 2.2.401 SDK.

Sorry about that. https://dotnetcli.blob.core.windows.net/dotnet/release-metadata/2.2/releases.json has been updated to include 2.2.401.
cc @vivmishra

Thank you very much @leecow for the quick fix!


Web framework

please. anything so you can template the header and navbar. refactor everything

Use a static site generator! Try https://github.com/11ty/eleventy/

Use a static site generator! Try https://github.com/11ty/eleventy/


Deleting cache directory as a troubleshooting effort

Might be worth mentioning somewhere that one possible way to debug is to delete the .contentlayer directory and restart the server. But ideally users would also log issues along the way.

We may be better served to have a troubleshooting section including known issues.
This should also be worked in conjunction with #32

If that's the path, let's address #33 first.


rename requirements of extension tests from txt to sh

Type of Change
Bug fixed.
Description
rename requirements of extension tests from txt to sh
Expected Behavior & Potential Risk
N/A
Dependency Change?
N/A

https://inteltf-jenk.sh.intel.com/job/neural_speed_extension/99/

https://inteltf-jenk.sh.intel.com/job/neural_speed_extension/100/


Feature Request: Encrypted Setup

This isn't an issue, just a request. I'd really like to see an encrypted partition install, having a system unencrypted is a major concern on mobile devices. It's possible to create an encrypted BTRFS partition from my research.

Hi i have created a encryption function with luks in my fork of the script, https://github.com/EnigmaDR/ArchTitus/blob/main/0-preinstall-luks.sh and i added configuring mkinitcpio in 1-setup.sh.

Make a pull request?

Added


Include a link to this repo on npmjs.com

Here's a reference: https://docs.npmjs.com/files/package.json#repository

Superseded by #13.


FEATURE - Allow for metadata fields to be included in staging models

Are you a Fivetran customer?

Original request identified by @sachams
Is your feature request related to a problem? Please describe.

No, this is to add the metadata fields to staging models.
Describe the solution you'd like

Ideally a customer would be able to add the metadata field and possible parse the field out as well.
Current PR Work
PR #18

@sachams I wanted to loop back up here to let you know our team has spent most of this week working on dbt 20 compatibility updates. We are going to focus on upgrading this package as well as our other packages and then we will be able to tackle this request once we have completed all our upgrades.

Just wanted to add that I'm interested in using the stripe connector, but would also require the metadata fields to perform the kind of analysis / reconciliation that I'm planning.

Thanks for commenting @mnahkies! Would you be able to specify which metadata field for which object you would be interested in? I am still trying to scope out how our team will approach this task and it would be helpful to get more insight into which metadata fields you are most interested in and for which table(s).
Thanks!

We are interested in the subscription metadata specifically.
Given that the contents are custom key->value pairs, I would be satisfied with either a target column metadata JSON / TEXT column containing a serialized JSON object.
Alternatively the proposal of putting the content into columns like metadata_my_key sounded fine too.
Since writing my comment, I have realized that we should be able to fulfill our use case by using the subscription_id to then reconstruct the metadata from our own records that are loaded from another source, so perhaps it isn't as important for us as I initially thought.

Hi @fivetran-joemarkiewicz how's the work going?
By the way, a colleague recently set up the Mixpanel DBT package and was able to use the event properties pivot to extract custom fields from JSON into columns. Could this approach be used for Stripe fields?

Hi @sachams I was able to get some more information from @mnahkies and the approach you pointed out for Mixpanel was exactly the route I was considering going!
Our team has still been pretty busy the last few weeks and we have not been able to tackle this feature at the moment. With this in mind I will add this ticket to our upcoming sprint (starting tomorrow) and will keep you both updated for when a working branch is ready to be tested out.
Thanks as well @sachams for the PR #18 as this will be extremely helpful!

@mnahkies and @sachams I have been able to add the functionality for the charge, invoice, payment_intent, payment_method, payout, plan, refund, and subscription staging models to pivot out defined metadata fields within PR #28. Huge thanks to @sachams for the bulk of the PR work 🎉
It would be great if you would be able to read through the updates and give the new functionality a try and let me know if it all looks good on your end. You can do so by using the below config in your packages.yml and be sure to update your variables configuration based off the documentation I added in the PR for the metadata pivot.
packages:
    - git: https://github.com/fivetran/dbt_stripe_source.git
      revision: customer/sachams-metadata-add
      warn-unpinned: false

Hi All,
Just wanted to provide an update that the PR #28 has been merged and was included in the v0.4.3 release of the stripe_source package! This should be live on the dbt hub at the top of the hour. Thanks again for all of your help with integrating this into the package.
As this has been included in the latest release I will close this issue. Please feel free to open a new issue if you have any other issues or feature requests. Thanks again!


Update GitHub Actions to avoid set-output deprecation

Starting 1st June 2023 (planned) workflows, ::set-output name...:: syntax cannot be used. You can see the example warnings here: https://github.com/tensorflow/tensorflow/actions/runs/3752114734
ref. GitHub Actions: Deprecating save-state and set-output commands | GitHub Changelog
This PR fixes the issue by using the GITHUB_OUTPUT environment variable as instructed in the above article.

Here's the trimmed version of the GitHub Action test run for .github/workflows/sigbuild-docker-presubmit.yml: https://github.com/shuuji3/tensorflow/actions/runs/3792737075/jobs/6449190102
It checks only the DATE output but confirmed the output is stored as expected. Other outputs would be the same.

Thank you for reviewing and additional improvements! 🙂


chore: enable avalanche support

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


lunatask: 1.7.8 -> 2.0.0

Description of changes

Update lunatask to 2.0.0
This release brings massive changes to the navigation, UX and others, while introducing the much-awaited Relationship Tracking
Full changelog: https://lunatask.app/releases/2.0.0
Note: the update to 2.0.0 is also required to use the app, as now when logging in with versions older than 2.0.0, the service gives an error to update the app first.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Done in #308596.


Are there any suggestions for installing Google's sparse hash map project, if I am not a root user.

Are there any suggestions for installing Google's sparse hash map project, if I am not a root user.
When I was trying to compile and install it locally, I still encountered permission issues. Can you give me some suggestions?

make  install-am
make[1]: Entering directory '/home/lijiale/envs_openpcdet2/sparsehash'
make[2]: Entering directory '/home/lijiale/envs_openpcdet2/sparsehash'
test -z "/usr/local/lib" || /bin/mkdir -p "/usr/local/lib"
test -z "/usr/local/share/doc/sparsehash-2.0.2" || /bin/mkdir -p "/usr/local/share/doc/sparsehash-2.0.2"
/bin/mkdir: cannot create directory ‘/usr/local/share/doc/sparsehash-2.0.2’: Permission denied
Makefile:692: recipe for target 'install-dist_docDATA' failed
make[2]: *** [install-dist_docDATA] Error 1
make[2]: Leaving directory '/home/lijiale/envs_openpcdet2/sparsehash'
Makefile:1144: recipe for target 'install-am' failed
make[1]: *** [install-am] Error 2
make[1]: Leaving directory '/home/lijiale/envs_openpcdet2/sparsehash'
Makefile:1138: recipe for target 'install' failed
make: *** [install] Error 2

Thanks.

Hi, you can install it into a local lib (e.g., ~/.local/lib) following https://stackoverflow.com/questions/3239343/make-install-but-not-to-default-directories.

Thanks for your kind suggestions.  I successfully installed sparsehash and torchsparse.

Hi, after make install, how do you set in your ~/.bashrc


feat: update footer links and search indices

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

The translated bits that I seen in this PR were transled in-house or via Google?

The translated bits that I seen in this PR were transled in-house or via Google?

Seem to be ported from the news local.
https://github.com/freeCodeCamp/news-theme/tree/master/locales
So @RafaelDavisH and @miyaliu666 translated those.

Seem to be ported from the news local.
https://github.com/freeCodeCamp/news-theme/tree/master/locales
So @RafaelDavisH and @miyaliu666 translated those.

footer links were translated in-house on https://github.com/freeCodeCamp/news-theme/tree/master/locales/es.json
Some links were not translated because there are no articles for those yet and some other links because by convention it's best to use them in English.

footer links were translated in-house on https://github.com/freeCodeCamp/news-theme/tree/master/locales/es.json
Some links were not translated because there are no articles for those yet and some other links because by convention it's best to use them in English.


[Feature request] Dark mode

It would be really nice to have a dark mode, which could possibly also include inverting (at least the black and white) comics.!

I wouldn't recommend inverting the comics if we pursue this. I think it quite changes any given drawing, as well as making Black Hat and White Hat switch roles, as funny as that could be.


Use more run exports

Additional comment:
@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/cairo-feedstock#65.


Cli wrappers enhancement

Adresses/Fixes #1044, #1045, #1047
Implements

Bugfixes of the cli's CMD and shell wrappers
Support of cli calls outside of devon environment

@devonfw/cobigen

Hi @markusschuh,
from my point of view this already addresses the issues you mentioned. But please check it just in case-


Use single threads for testing

I actually couldn't find a reference to CARGO_TEST_THREADS in the
current cargo codebase, but RUST_TEST_THREADS is definitely correct.
This should reduce some spurious failures on some platforms.

The reason for Cross.toml is that Cross runs everything (but not on Mac) in a docker instance, so it uses a whitelist to passthrough environment variables.

I'm actually just realizing that I don't think that trust has released since that feature was added, so this probably still doesn't do anything on non-Mac platforms.

Weird.  It looks like @posborne set that variable correctly on his branch, but somehow it didn't get merged correctly.
Compare https://github.com/posborne/rust-nix/commit/0e3ef36c7406f92483bfc75a37a382a1509fdb54
to a8cf6cbb9de5c4a4f65687c424933220289e5333
bors r+


Add Optimizely Integration to iOS Tracker

It would be awesome if we could build something similar to the Optimizely integration offered by the JS tracker for iOS. The integration could surface the same fields that are used in the optimizelySummary context (link).

Yes, this would be cool. I thought we had a ticket for this already, but can't find it...

Apologies if there already is one - I couldn't find it!
I'm working on getting this live on iOS with Lifecake next week. I'll see if there's anything we can use that Filippo builds!


Develop upstream sync 230213

https://github.com/ROCmSoftwarePlatform/frameworks-internal/issues/3667

retest Ubuntu-GPU-multi please

retest cpu-non-pip please

retest cpu-non-pip please

retest cpu-non-pip please

retest Ubuntu-GPU-multi please
retest cpu-non-pip please

retest Ubuntu-GPU-multi please

retest Ubuntu-GPU-multi please
retest cpu-non-pip please


openCV with GPU

I was looking to use OpenCV with Cuda8. I have Cuda8, but have not been successful with getting openCV working with it. Is there any guidelines for that?

You have to install Cuda 8 first (which I assume you have) and compile OpenCV with at least these flags
-D WITH_CUDA=ON
-D ENABLE_FAST_MATH=
-D CUDA_FAST_MATH=1
-D WITH_CUBLAS=1
After you run cmake, you should get in the ouput that NVIDIA CUDA is enabled.

Thanks nachomartinr. I tried the above steps and had some issues with OpenCV, thus raised the issue. I was wondering if you have version that you have have tried out.

I have a GPU enabled OpenCV 3.1 installed in my computer, but compiled from sources. I had problems compiling it, but they weren't related to CUDA, but to the GCC version (OpenCV does not support GCC 6 yet). I don't know of any binary release with enabled GPU acceleration.

Closed as out of scope for this repository.


make PTR record lookups in UDP functionality optional

This change maintains default behavior and backwards compatibility but allows for avoiding DNS PTR lookups per UDP packet.

I think I'd prefer to just have it never look it up, and expose the raw InetSocketAddress object as an additional key.

Were you thinking something like this?


Enabling self service will remove account lock option

Describe the issue:
When user enable self registration option, account lock option get removed from console user section. Also once disable the self registration option will not revert this. Hence need to check this on new pack.
https://github.com/wso2/product-is/assets/39077751/ac72e4e7-f263-45a7-a972-9961fa7238e4
How to reproduce:

Login to console
Create user
Check wether account lock option is available
Navigate to login & registration
Select self registration
Enable self registration  (This should not be enable before)
Check user lock option is avaible.

Expected behavior:
User lock option should be there even though the self registration get enable. Disable self registration will not display back the account lock option.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: IS 7.0.0-beta
OS: [e.g., Windows, Linux, Mac]
Database: H2
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

Not reproducible in the latest build.
https://github.com/wso2/product-is/assets/41533942/2aaa4b33-ee2e-48f8-89f7-f7d5d9adaf9b

Analysis
This is reproducible in the latests build.
According to [1], the account lock option will be shown when value of SelfRegistration.LockOnCreation is set to true. When Activate account immediately option is checked[2] under self registration, this particular property is set to false. Hence, the account lock option is not visible under the edit user UI. Then if the self registration option is disabled[3], the false value remains the same because, only the value of SelfRegistration.Enable will be changed. Ideally, when self registration is disabled, all properties relevant to the self registration should be brought back to default values.
[1] https://github.com/wso2/identity-apps/blob/a6e7fd7ae7be4ae70393e3e30f820384511cb7bf/apps/console/src/features/users/components/user-profile.tsx#L1156
[2]

[3]

https://github.com/wso2/product-is/assets/50801227/5fce2c54-c7e0-456c-9aa9-40bc68147778


Support Sphere Pop Up

Please increase the width of the pop-up so the Receive Karmic text becomes a one-liner
Expected:

Result:

Can we also implement a new text to the button so when a  metamask transaction is pending it displays:
"Sending ETH..."


go-bindata: use fetchFromGitHub

Cleaned up expression as well. Verified sha256 is reproducible and unchanged.

Motivation for this change
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 89138 1

1 package marked as broken and skipped:
- docker-machine-xhyve

18 packages built:
- browsh
- docker-machine-kvm2
- documize-community
- go-bindata
- go2nix
- hydron
- kops
- kops_1_15
- kubectl
- kubectx
- kubernetes
- magnetico
- minikube
- minishift
- morph
- ngrok-1
- openshift
- traefik


[FEATURE] Replace actor start/stop events with actor_scaled event

Is your feature request related to a problem? Please describe.
There are currently 4 events related to scaling an actor:

actor_started
actor_stopped
actors_started
actors_stopped

Describe the solution you'd like
I propose we use a single event to capture all of these: actor_scaled
Describe alternatives you've considered
N/A
Additional context
I'm not sure how this impacts wadm, so would appreciate input from @brooksmtownsend @thomastaylor312

Hello @connorsmith256 I would like to work on this issue can you please assign it to me I have a question should we have to replace all the actors_started and actors_stopped to actors_scaled

w.r.t. wadm, it would be nice if we introduced the new actor_scaled event and deprecate the others so that we can work forwards with backwards compatibility for a release or two.
Ideally:

Add actor_scaled to the list of events, emitted whenever an actor is started, stopped, or scaled. If there's a cloudevents way to mark an event deprecated (TBD) we should do that too.
Note in that wasmCloud release that the other events are deprecated.
Update wadm and wash to expect the actor_scaled event.
Remove old events

Question: How do we treat the idea of scaling an actor with no maximum concurrency? Does an actor_scaled event with a max of 0 mean infinite, or that it stopped?
Strawman answer: Scaling to 0 means that the actor is no longer running. Actually scaling with a maximum level of concurrency should be supported with a big number, e.g. if the scale is a u16 we should represent the max as 65536.

#1107 will remove actor_started and actor_stopped and actor_start_failed in favor of actor_scaled, but leaves the plural events for now so we can make the compatible changes in wadm, the lattice observer, wash, etc.

Closed in #1107


update document for v1.3

1, update README.md
2, format doc part11, part12
3, add a python script to scan markdown files to generate SUMMARY.md with anchor.

looks good


Support for structural variants with precise position in HGVS

We want to support variants in ClinVar that are lacking chr_pos_ref_alt information in the XML, but do have precise position information in HGVS identifiers, for example NC_000011.8:g.67967551_67974756del.
To get consequences for these we can parse the HGVS expression and query VEP, see some examples of how to do this here.  We will explode evidence along target genes as usual and apply a threshold for number of target genes (to be determined).
For variant_id, we can use RCV unless there are duplicates, otherwise investigate using VCV.  Other fields can be reported as we do for other variants.
We also need to add HGVS expression to the evidence string schema for all variant types, including repeat expansions and simple variants.

Quick summary of internal discussion we had on the above:

Moving ahead with the prioritisation & association approach outlined above makes sense, VCV isn't suitable as it's not aggregated across submitters as far as we know and we're not very close to a unified variant representation.
The discrepancy in locations (and hence functional consequences) is probably a red herring that seems to have been fixed in more recent versions of ClinVar, but it brought up an idea to find other examples where we have both HGVS and sequence location, get consequences via each pipeline, and compare them as a way to validate our approach.

Remaining questions:

Confirm whether we should try to incorporate these for the upcoming submission - I think it's possible but tight, and we would need to confirm some things like

new JSON schema (I think it's just a field for something like variantHgvs), and
gene threshold to filter very large variants.

Any preference on which HGVS to use for consequences and/or report in the evidence string?  These are present in ClinVar with varying frequency.  My current approach is to have a "preferred current HGVS" that can stably be associated with the variant, which prioritises the top-level genomic RefSeq HGVS but will fall back on others.  Theoretically this could decrease coverage (my previous counts were using all HGVS present in the record), but honestly I doubt by much.
Should we try to report HGVS for records that have coordinates but not HGVS expressions?  Generating a stable identifier from coordinates will be much easier than parsing, but not sure it will be useful for Open Targets.

If you want a sneak preview of consequences from the new pipeline you can find them here.

It looks like the VEP output is less precise with HGVS than with coordinate.
But one thing that confuses me is the fact that we're using the same VEP endpoint for both don't we ?
So maybe the issue is that VEP does not normalise the variant or worse makes assumption about how a variant have been normalised.
Maybe it would be valuable to look into normalisation of the HGVS variant so that they are left aligned.
Then we don't need compare consequences but we could compare the variant coordinates directly.

@apriltuesday I was worried at first when I read that 13,643 differ in consequence terms, and 125 differ in target genes (31% of discrepancies does not sound good). But your follow-up explanation is reassuring and highlights not an issue with the HGVS-VEP route, but rather a possible issue with the consequence filter we impose.
I see both POVs being valid: reporting too many consequences of a variant may be too much information to handle and hide the key consequences; but reporting only one, based on a one dimensional ranking, seems restrictive to me.
Since I have not attended to nor read discussions about this topic I would love to have a bit more info on this issue in the future, as to why we chose to report one single consequence for each variant.
Once again, thanks for the explanations of your investigations, @tcezard & @apriltuesday!
The essays are welcome

Closing this as done, further extensions/fixes/investigations can be opened as new issues.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-28-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4179

Questions ?
Please refer to the Backport tool documentation


[not_landed] Missing needinfo comment

As reported by @gijsk, the tool added needinfo flags without an explanatory comment:  https://bugzilla.mozilla.org/show_bug.cgi?id=1772940#a2984306_575867

This happens when a needinfo is added without the user nickname: https://github.com/mozilla/bugbot/blob/92d32813d9b54252557299177aee65ae385ccbfe/bugbot/bzcleaner.py#L526-L536
This seems regressed by #1064

Another example reported by @Archaeopteryx: https://bugzilla.mozilla.org/show_bug.cgi?id=1881037#c26


appplatform release

https://github.com/Azure/sdk-release-request/issues/3946

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


Add Busbar Chart

Add busbar (https://github.com/busbar-io/busbar-server) Chart to allow installation through Helm.

I'm now leaving this for @unguiculus to come back around to. Want to make sure his issues are addressed in a way that he's good with.

Let me ask you again if you would consider adding a dependency to the Docker registry chart instead of installing it yourself. As mentioned it can now handle S3 as well. Also, the way you set up the registry is AWS-specific which doesn't seem ideal.

Hi @unguiculus,
I would rather not use the docker-registry dependency for now. Currently Busbar is AWS centric and we do have plans to make it GCE compatible in the near future so I would prefer to add the Docker Registry dependency once we roll out the GCE compatibility (what would also give us some more time to test/adjust the charts accordingly).


only a simple n00b question

We aim for an implementation both clean and elegant, but the API is king. We sacrifice a great deal of implementation elegance for even a slightly cleaner API.

What is the difference between elegance and cleanliness in that context ... can you give me an example so I can understand I am a little confused I am sorry ...

I think "clean and elegant" is mostly a rhetorical flourish.  While they are not quite synonyms, there is a great deal of overlap here, especially when talking about the implementations.
But the point is that if we can make a better API for the user, we do so, even at the cost of making the underlying code uglier.  Conversely, if a beautiful simplification of the code would lead to even a slightly less comfortable API, we wouldn't do so.  In practice, of course, this is tempered by other real-world considerations such as performance and even our own patience.  But it's an important goal we mostly stick to.
Here's an example:  We could write map as a pure function in an extremely clean manner:
const map = curry(fn, xs) => xs.map(fn))

Instead we have three separate files involving a number of dependencies and these blocks of code:
var map = _curry2(_dispatchable(['fantasy-land/map', 'map'], _xmap, function map(fn, functor) {
  switch (Object.prototype.toString.call(functor)) {
    case '[object Function]':
      return curryN(functor.length, function() {
        return fn.call(this, functor.apply(this, arguments));
      });
    case '[object Object]':
      return _reduce(function(acc, key) {
        acc[key] = fn(functor[key]);
        return acc;
      }, {}, keys(functor));
    default:
      return _map(fn, functor);
  }
}));
export default map;

export default function _map(fn, functor) {
  var idx = 0;
  var len = functor.length;
  var result = Array(len);
  while (idx < len) {
    result[idx] = fn(functor[idx]);
    idx += 1;
  }
  return result;
}

function XMap(f, xf) {
  this.xf = xf;
  this.f = f;
}
XMap.prototype['@@transducer/init'] = _xfBase.init;
XMap.prototype['@@transducer/result'] = _xfBase.result;
XMap.prototype['@@transducer/step'] = function(result, input) {
  return this.xf['@@transducer/step'](result, this.f(input));
};

var _xmap = _curry2(function _xmap(f, xf) { return new XMap(f, xf); });
export default _xmap;

Why do we do this?  Because it's a better user experience to be able to write these:
map (square, [1, 2, 3, 4, 5]) //=> [1, 4, 9, 16, 25]
map (square, {a: 1, b: 2, c: 3}) //=> {a: 1, b: 4, c: 9}
map (square, increment) (4) ==>  25
map (square, Just (7)) //=> Just (49)
map (square, Nothing()) //=> Nothing
// ...

than to have separate mapArray, mapObject, mapFunction, mapMaybe, etc,, functions.  This version of map supports transducers, avoids the map(parseInt) pitfall, and works on arbitrary Functors.
Here we went a long way from the cleanest, most elegant implementation into a much uglier one in order to make our API more elegant.  And that's the point.

My God I love that answer I was unable to understand the difference between the two terms and asked the question naively without knowing what to expect as an answer but I feel that what ever my expectations could have been this answer goes beyond my theoretical expectations and make me understand that your work in this library is taking functional programming in JavaScript at an artistic level (a metaphorical way of describing the perfection of your technical knowledge)

@Lucxium:
I suspect that the library is much more workmanlike than artistic.  But thank you for your kind words.


Add podcast to Amazon music

Can you add your podcast to Amazon music app?

+1

Looks like its approved already https://music.amazon.com/podcasts/3f16a46b-6281-4fc7-99de-380fbeb6d970/syntax---tasty-web-development-treats

ADded! https://github.com/syntaxfm/website/pull/1409


compose: Fix bug where pasted content would only go to the compose box.

This commits ensures that we pass in the correct context (compose box or a message being edited) when pasting content.
Fixes: Issue described in this CZO thread

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @N-Shar-ma!
Definitely a thing that TypeScript would have caught.


adding hyper-stylesheet

Checklist for submitting an awesome plugin, theme, or resource:

[x] The title for my package or theme uses its npm title (hyper-plugin-example)
[x] The link for my package or theme uses the npmjs.com link (https://www.npmjs.com/package/hyper-plugin-example)
[x] There is a visual representation of what my plugin or theme does in the repo. (Plugin: screenshot or gif || Theme: screenshot of the theme applied to Hyper)
[x] Put your awesome item at the BOTTOM of the correct (plugin, theme, or resource) list.
[x] VERY IMPORTANT: I've written a short (one sentence) description for my package or theme as the description of the awesome plugin, theme, or resource in the README.md file I'm submitting in the PR.

https://github.com/chrisdothtml/hyper-stylesheet

Cool ☺


chore: Refactoring action execution flows to clean up files

This task is simply dividing action execution related flows into separate files for clarity. It will allow us to pull this module out in the future.

/ok-to-test

/ok-to-test

/ok-to-test

/build-deploy-preview skip-tests=true


Add support for using JsBarcode in implementations that don't implement measureText

We use an HTML to PDF converter which returns undefined for ctx.measureText(). In this PR we return 0 instead of crashing the JavaScript code.
I was not able to run node_modules/gulp/bin/gulp.js lint as it complains about ' Unexpected top-level property "predef".' in .eslintrc.
If I need to do something else, please let me know

Hi and thanks for the contribution 🙂


Initial features

Must have
For internal viability:

[x] Handle & allow expression of the whole TS type system

[x] Primitives (bool, num, string)
[x] Top (any), Bottom (never), Unit (void) and null (null, undefined)
[x] Arrays
[x] Indexed Interfaces
[x] Interfaces
[x] Union
[x] Intersections
[x] Tuples
[x] Enums
[x] Object
[x] Type aliases (currently modeled as a first-class type similar to interfaces, not 100% sure that is correct)
[x] Literal types (string, number/integer, boolean?)

[x] Serialize TS types
[x] Discover all top-level types (interfaces, aliases) necessary to emit interfaces
E.g. if you want to publish IFoo { member: IBar } IBar will also need to be emitted
[x] Small DSL to define interfaces and/or modify generated interfaces
[x] Macro that generates case classes

Should have
For internal and public viability (i.e. if these are checked we could definitely do a public release)

[x] Write to files/streams
[x] Basic readme and documentation including example project
[ ] Scala 2.11 & 2.12 cross-compilation
[x] Publishing to a public maven repository
[ ] Emit in a namespaces/module/class, handle subpackaging. Including overwriting by user?
[ ] Solid test suite
[x] SBT Plugin to generate the TS files
[x] Lots of predefined mappings, e.g. java8 dates, uuid etc.

[x] Generics for collections (List[T] etc)

[ ] Handle or give an error on recursive definitions
[ ] Mark all subtypes of a specific type as having the same mapping
[ ] Provide a default parameter for implicit TSTypes, a macro which understands built-in types and case classes so implicits aren't mandatory the be defined

Could have
Useful features which aren't the top priority

[ ] Generics? Not sure how they are useful in a serialization <> deserialization context like this is used for
[ ] Style options? i.e. 2-4 spaces indentation, whether to emit (T | undefined) as member: (T | undefined) or member?: T etc.
[ ] Create an enum from a sealed trait

Won't have
Currently not in scope

[ ] Generating Scala case classes from typescript interfaces
So much potential here, basically the dual. Useful to make sure your back-end can parse the frontend JSON. Auto-generate JSON deserialises based on TS interfaces

The project has a lot of the basics, a separate issue has been creates for all missing features


Preview site with invalid certificate fails

I confirm (by marking "x" in the [ ] below: [x]):

[x] This is not a troubleshooting question. Troubleshooting questions go here: http://www.mattermost.org/troubleshoot/.
[x] This doesn't reproduce on web browsers (such as in Chrome). If it does, issue reports go to the Mattermost Server repository.
[x] I have read contributing guidelines.

Summary
Preview links with an invalid certificate returns an error on channels.
Environment

Operating System: Ubuntu 20.10 / MacOS
Mattermost Desktop App version: 4.6.2 (ubuntu), 4.6.1 (MacOS)
Mattermost Server version: 5.31.0

Steps to reproduce

On a channel, share a link with an invalid certificate (in my case https://dasprogramm.co.uk/ )
Go out of the channel and go back

Expected behavior
The certificate is invalid. The preview should fail gracefully.
Observed behavior

Possible fixes

if you hit cancel, does it fail gracefully?
we added the option to handle invalid certificates, as there are situations when there is the need to bypass the certificate validation

It does fail gracefully.
But every time we go back on the channel we get the popup.

Ok, I reopened this ticket to discuss again with the team.

Closing as this is the same issue solved here: https://github.com/mattermost/desktop/pull/1733


Misleading error in rule validation when failing to load external resources

The request and response validation rule added in #1259 is sensitive to errors while reading external resources. In this case it shows
{"title":"Internal Server Error","status":500,"detail":"check invocation failed:
  id=Z001 title=Path parameters validation checkName=checkSchemaOrContentProperty
  reason=java.lang.IllegalStateException: parameter.name must not be null"}

on an example API as the following:
openapi: 3.0.1
info:
  x-audience: company-internal
  x-api-id: anything
  description: Test
  title: Test
  version: 0.0.1
  contact:
    name: Test
    email<EMAIL_ADDRESS>    url: https://test.de/test

paths:
  /tests:
    parameters:
      - $ref: 'https://opensource.zalando.com/restful-api-guidelines/request-headers-1.0.0.yaml#/X-Flow-ID'

    get:
      responses:
        default:
          content:
            application/problem+json:
              schema:
                $ref: 'https://zalando.github.io/problem/schema.yaml#/Problem'
      security:
        - BearerAuth:
          - test.read

components:
  securitySchemes:
    BearerAuth:
      type: http
      scheme: bearer
      bearerFormat: JWT

Note: This is behavior may repeat in any other general rule.

I think this is a parser problem. Zally uses Swagger parser with a full resolution option. That means if the external resource is incorrect or missing, the parser should fail. In this case it will never reach rules.

After updating the parser there is no such error anymore. I have added an additional test to verify this problem in #1283 .


Create sample app to test Cocoapods integration

Current sample app is manually setup.

[x] iOS
[ ] tvOS
[ ] macOS

PR https://github.com/linkedin/LayoutKit/pull/122

tvOS and macOS test project https://github.com/linkedin/LayoutKit/pull/132


Column does not exist when using query alias inside subquery

Version
1.17.2
What happened?
It seems like sqlc is unable to generate the code for the query below even though it runs fine on Postgres. Might be a parsing issue.
Relevant log output
query.sql:7:37: column "height" does not exist

Database schema
CREATE TABLE public."user" (
    id uuid DEFAULT public.uuid_generate_v4() NOT NULL,
    username text NOT NULL,
    email text NOT NULL,
    created_at timestamp without time zone DEFAULT CURRENT_TIMESTAMP NOT NULL,
    bio text
);

CREATE TABLE public.user_measurements (
    user_id uuid DEFAULT public.uuid_generate_v4() NOT NULL,
    height real,
    ptp real,
    waist real,
    hip real,
    usual_uk_size smallint
);

SQL queries
SELECT u.id, u.username, u.email, u.bio,
(SELECT row_to_json(r) FROM (SELECT m.height, m.ptp, m.waist, m.hip, m.usual_uk_size WHERE m.user_id=u.id) AS r) AS measurements
FROM "user" AS u
LEFT JOIN user_measurements AS m ON m.user_id=u.id
GROUP BY u.id, m.user_id;

Configuration
version: "2"
sql:
  - engine: "postgresql"
    queries: "query.sql"
    schema: "schema.sql"
    gen:
      go:
        sql_package: "pgx/v5"
        package: "sqlc"
        out: "../../db/sqlc"
        emit_pointers_for_null_types: true

Playground URL
https://play.sqlc.dev/p/9379c132cfe6b9cecdc75cd72b2078ca562e3fd880c2e9e3a7c3530afb1dc855
What operating system are you using?
macOS
What database engines are you using?
PostgreSQL
What type of code are you generating?
Go

I'm not able to reproduce this issue. Instead, I'm getting the following syntax error for your query:
# package db
query.sql:1:1: syntax error at or near "FROM"

Alright, I've hit another roadblock. Trying to reproduce your query using PostgreSQL 15.2 results in the following error:
query.sql:1:1: ERROR: column "u.username" must appear in the GROUP BY clause or be used in an aggregate function (SQLSTATE 42803)

Can you double check your query? Once it's working I can figure out why sqlc is having trouble with it.

I've been unable to get this query to run inside PostgreSQL. We just launched a database-backed SQL analyzer which should fix this issue. If you think it's still broken in v1.23.0, please open a new issue with a working schema and query.


Invalid behavior in the Visual Studio debugger for F# when using DebuggerTypeProxy attribute with certain types

Version: Visual Studio 2015, .NET Framework 4.5.2, F# 4.0
The issue does not appear in Visual Studio 2013.
Consuming the following C# class from F# causes the F# debugger to behave strangely.
[DebuggerTypeProxy(typeof(Test<>.Proxy))]
public class Test<T> {

    public Test(T value) {
        ShouldBeHidden = value;
    }

    class Proxy {
        Proxy(Test<T> t) {}

        public int ShouldBeVisible {
            get;
        } = -1;
    }

    public T ShouldBeHidden { get; }
}

In the C# debugger, viewing an instance of the type works as expected.
Code: var t = new Test<int>(456)

However, in the F# debugger, this occurs instead:
Code: let sd = ConsoleApplication1.Test(456)

Note that the number the debugger displays has nothing to do with the actual number.

I can't reproduce this on Roslyn master with the latest F# nightlies:

Can close as fixed. @cartermp

Yup, and this should be live in VS 2017 Update 3.  Thanks @saul!


Release v2.3.3

Summary
First release to the Marketplace
Ticket Link
Part of https://github.com/mattermost/mattermost-plugin-servicenow/issues/135

Codecov Report
Base: 66.24% // Head: 66.24% // No change to project coverage :thumbsup:

Coverage data is based on head (47e1512) compared to base (3343003).
 Consider uploading reports for the commit 6ca40b3 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #138   +/-   ##
=======================================
  Coverage   66.24%   66.24%
=======================================
  Files          16       16
  Lines        1573     1573
=======================================
  Hits         1042     1042
  Misses        493      493
  Partials       38       38

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

/update-branch


Fix SDK version in global.json file

The version number currently used doesn't match with an actual SDK version. It should be 8.0.xxx. I've updated it to use the latest SDK version.

@andrueastman Yes, the install-dotnet scripts that are popularly used to install dependencies will fail because the version format is incorrect (it should be 8.0.xxx). The rollForward doesn't do anything in this case since the underlying version format is malformed.


deployment system

Our buildsystem doesn't include the QT DLLs (or any future libraries we may need) next to the executable. We need a cross-platform system to get the app ready for deployment/distribution. Or we could just copy the DLLs manually like the old times.

@ItchyTrack I remember that the old CMake build system had this section that seemed to be for this:
install(TARGETS ${PROJECT_NAME}
    BUNDLE  DESTINATION .
    RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR}
    LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR}
)

qt_generate_deploy_app_script(
    TARGET ${PROJECT_NAME}
    OUTPUT_SCRIPT deploy_script
    NO_UNSUPPORTED_PLATFORM_ERROR
)

install(SCRIPT ${deploy_script})

It's a good starting point. I think qt_generate_deploy_app_script is a good choice for getting all the QT libraries working. I haven't been able to get it to work yet. I'm not sure what the install section does, I think that is it using hallucinated variables.

While we're improving the build system, a precompiled header would be nice...

while we're doing all this, we should probably get rid of all the unnecesary headers
and see if there are MAJOR compilation improvements to be made by a little forward declaring

hey.. while we are doing all of this, it might be nice to create some CMakePresets files


Terms and conditions

[x] Create terms and conditions service
[x] Make Auth0 lock configuration customizable via config.json
[ ] Enable signup terms & enforce them for all new users.
[ ] Test redirect for all accounts that have not accepted tos OR that have accepted old version (remember to store tos version in the first place)
[ ] Create profile page (user email - read only, company - editable, terms and conditions - checkbox, save button)
[ ] Enable redirect

Useful links:

https://community.auth0.com/questions/1631/require-accept-terms-on-social-connections
https://auth0.com/docs/libraries/lock/v10/new-features

Out of scope:

terms of service for passport strategies.

As usually, the reality was not that simple. Auth0 happen to not store any info about the licence, and the signup 'agree to terms' works only for login/password method and can be easily workarounded for social login providers.
Therefore ALL users which have not accepted terms and conditions will be redirected to a page where they can do that.


In voice-start and voice-end events, user.name is null

Maybe this isn't always true, but for our server it is. The code in question is:
https://github.com/EvolveLabs/electron-mumble/blob/master/lib/MumbleConnection.js#L715
if( packet.frame && !user.talking ) {
    user.talking = true;
    this.emit( 'voice-start', { session: user.session, name: user.name, talking: true } );
}

The packet shape is correct, it seems like mumble just doesn't send the user name anymore. Instead it gives you the session id and expects you to correlate it with a user I think. This could possibly be corrected by setting the name based on the users collection:
var name = user.name || this.users[ user.session ].name;
this.emit( 'voice-start', { session: user.session, name: name, talking: true } );

Our workaround is to essentially do this in the handler:
connection.on('voice-start', function (user) {
  user.name = connection.sessions[user.session].name;
  ...
});

I'll try to send a PR soon-ish if you don't do it sooner.

Hm. That's weird if the user.name || this.users[ user.session ].name works given the user in the packet is set from this.users[ user.session ] around line 800
Mumble never sent the name in voice packet, only the session ID. All the convenience bits, such as user name, are purely from node-mumble. Not sure when I've got time to check this - I've got the Node 4.0.0 compatibility stuff to check for lots of dependent modules still as well and a small hiking trip planned for the weekend. Might find time next week unless you come up with a PR before that. :)

I'm probably swamped until after next week. But I'm not blocked, just something interesting there.


tesseract prints "Estimating resolution as " (takes >1 second to print this line) even if I pass SetVariable("user_defined_api", "70")

I'm calling tesseract with
SetVariable("user_defined_api", "70")
It supressed
Warning: Invalid resolution 0 dpi. Using 70 instead.
however I still see
Estimating resolution as 304
which takes more than 1 second to print, so I need to get this out

Please take a look on how "Estimating resolution" is killing performance
https://photos.app.goo.gl/UKKTKSiAnQT4mB876

Read your post. Again and again and again until you find YOUR mistake. If it does not help, try to replicate problem with tesseract executable ;-)

@zdenop I think you are talking about my typo in api. Unfortunately it's not in my code, just here.
Somebody told me to use
tess.SetVariable("user_defined_dpi", "71");
and it worked. Don't know why. Posting here to help someone.

Yes, but there are more problems:

finding reason / relevant part of tesseract code for your issue is easy
instead of understanding your problem you write issue, you made screen cast, you made stackoverflow post...
you ignored guidance for issue tracker: for support you have to use tesseract user forum
your approach is wrong: you did not try to understanding suggestion from SO, you just used it. If somebody will suggest you to run rm -fr / will you do it? This approach is dangerous not only in programming but also in real life.


Why is there a sync CLI?

--sync <on> <off>                 Turn sync on or off

The CLI is already quite cluttered, under what circumstances would someone use this over the UI? Plus this would trigger a quick pick in the UI anyway?

Related request - https://github.com/microsoft/vscode/issues/92516

@bpasero was just telling me about how we should be keeping CLI options to a bare minimum in https://github.com/microsoft/vscode/issues/93241#issuecomment-602716329, and this seems very niche.

Yes, +1 for not having it. If this is for testing reasons, consider to at least hide it from the help output.

This is not for testing reasons and it is as per user request here - #92516.
Another use case which also makes sense for this is that - if I want to start VS Code by disabling sync because if sync is on there is no way to start VS Code without triggering sync. By the time window comes up and I disable sync VS Code would have already started syncing. It is very similar to --disable-extensions.
@Tyriar I closed this in favour of original request and to continue discussion there.

@sandy081 I understand the need but should it move under the troubleshoot section? this seems like a edge case to me.


Helm Variables

Add missing variables which were used on v1 by a user.
Description

Makes several options configurable through helm.
Motivation and Context

Regression

How Has This Been Tested?

Types of changes

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

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added unit tests to cover my changes.

Add missing variables which were used on v1 by a user.

What does missing mean here? Did we at some point deliberately remove these variables and now re-adding them?

Added to the raw yaml, but now that we use helm one can't simply edit the yaml anymore...
Perhaps we should consider adding custom env variables..

bors merge


fix for #889

What this PR does / why we need it:
This fixes #889
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #889
Special notes for your reviewer:
Release note:

- this PR adds code to decrypt in 5 MB chunks (just like the encrypt for multipart is done in chunks of 5 MB). So,

- download the whole file from cloud backend
- decrypt in 5 MB chunks
- re-assemble and send to browser

Have tested with .exe files and .txt files, lesser than 5MB and larger than 10 MB, did a byte comparison of original file and downloaded file. Screenshots shared below

![image](https://user-images.githubusercontent.com/48434725/74078957-c1f62b80-4a56-11ea-8929-25cae654a120.png)

![image](https://user-images.githubusercontent.com/48434725/74078965-dfc39080-4a56-11ea-987e-2560fb30fcaa.png)

![image](https://user-images.githubusercontent.com/48434725/74078970-efdb7000-4a56-11ea-96fa-bcad4d8ec8ce.png)

@sfzeng @kumarashit please review, thanks

the code has gone out of sync with current code, been over a month, closing this


Merge prTesting_201907081205 branch to master branch

Docs Build status updates of commit 1037ae9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


1275-removing-token-sets-sometimes-fails

Sometimes when using Git storage removing a token set fails. It seems to be connected to the name of the set, as it's not happening all the time.
https://www.loom.com/share/a383c5d6b04e42b1aede02bd60f91846

In this commit I only deleted the file Colors/Archive.json - but Base/Colors/Archive.json was deleted as well.
@six7 In my computer it works properly. Could you check this video and again in your computer.
https://www.loom.com/share/a383c5d6b04e42b1aede02bd60f91846

Great, works now! Merging in


Cannot pre-fill a text area field

Trying to use d-form-textarea and d-textarea with a loaded value by default with v-model:
<d-form-textarea       name="feDescription"       rows="5"       v-modal="test"       :value="test"     ></d-form-textarea>
But the textarea is blank even though there is a default value for the "test" property.

OK I have resolved this. Turns out I needed to add:
:value.prop="test"