demo test file make voice to text error

Describe the bug
I am using azure sample test error occurs when the text to audio file, reference url:https://docs.microsoft.com/en/azure/cognitive-services/speech-service/quickstarts/speech-to-text-from-file? Tabs =linux&pivots=programming-language-cpp, the same parameter is normal for Java testing, the error message is as follows：
CANCELED: Reason=1
CANCELED: ErrorCode= 5
CANCELED: ErrorDetails=Connection failed (no connection to the remote host). Internal error: 1. Error details: 2460. Please check network connection, firewall setting, and the region name used to create speech factory. SessionId: d9e45623199f411b96edaecca499a01f
CANCELED: Did you update the subscription info?
Please press a key to continue.
Expected behavior
Can be recognized normally
Version of the Cognitive Services Speech SDK
SpeechSDK-Linux-1.7.0
Platform, Operating System, and Programming Language

OS: Centos7
Hardware - x86
Programming language: C++

error.log

Hi, unfortunately CentOS 7 x86 is not supported by SpeechSDK 1.7.0. Officially we support RHEL/CentOS 8 x64 from SpeechSDK 1.10.0 onwards (see https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/releasenotes and https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/speech-sdk?tabs=linux). However, there is a way to have it working on RHEL/CentOS 7 x64 as well, we will publish instructions for this when SpeechSDK 1.11.0 is released. RHEL/CentOS target architectures other than x64 have not been tested and are not officially supported at the moment.


Eraser functionality

From @talapadi on July 24, 2018 14:57
Any drawings should get erased. The pattern should be saved into json
Copied from original issue: chimple/maui#639

Given: User should be on the application.
And: Drawing something on the screen.
And: the user required to erase the drawing.
When: User selects the eraser with the desired size.
Then: The drawing should get erased according to his hand moment


Safari browser - double tap gesture not recognized to zoom in

Hi,
Safari 15 on iOS does not recognize the double tap gesture anymore in order to zoom in.
It perfectly works in Chrome.
Is there any way to fix that?
Thanks

Interesting. Perhaps they added something around double tap in version 15 that makes it harder for the webpage to see the event? Someone will have to do some experimentation to see if there's a way to capture the event. I believe the way we do it in OpenSeadragon is to watch touchstart and touchend events and if there are two sets close enough we consider it a double tap. Maybe there's a special double tap event in the new Safari that masks some of these other events or something?
I found this, but it's not necessarily all that helpful:
https://developer.apple.com/forums/thread/691478
@liveman1 thank you for reporting this! Are you up for debugging it? If so, I can point you in the right direction.

Hi,
thanks a lot for your reply and the explanation.
Sorry for the confusion, I think it is caused by a custom script to trigger a context menu on tap. It is a strange behaviour works quite differently on different devices & browsers.
It would be great if there would be a straight forward recognition of a double tap that does not conflict with an on tap event too easily. Unfortunately, I am no programmer and can‘t help here.
Best regards

Just to be precise, double tap does not work in Safari on iPhones since the latest release of iOS 15.
It works in Chrome on iPhones.
There is no issue on tablets (iPad, Windows) no matter which browser is used.
I could not test it for Android devices.

I see. Well, it's possible we're going to have to wait for a fix from Apple. Still, hopefully someone can do some experimentation in the meantime to see if there are any workarounds. Thanks again for the info!

Hi,
iOS 15 introduced new zoom controller settings.
Obviously, there is a dedicated setting to zoom in on double tab.
When selecting to do nothing on a double tab, everything works great!
I wonder if there is any other way than changing the iOS settings.
Best regards

Good to know!

Hi, this works again. Not sure if there was any change on iPhone/Safari side.  :)

Wild... thanks for the update!


parsing google token

Hello!
I've been using this library for a while, everything was working fine, and still is, except for google oauth. Yesterday I noticed I couldn't loggin with google. so i started debbugging the issue and find out that problem is here
          if (token) {
            if (token.split('.').length === 3) {
              var base64Url = token.split('.')[1];
              var base64 = base64Url.replace('-', '+').replace('_', '/');
              var exp = JSON.parse($window.atob(base64)).exp;
              if (exp) {
                return Math.round(new Date().getTime() / 1000) <= exp;
              }
              return true;
            }
            return true;
          }

The issue is that google is now returning a token in the form ya29..vgIJcUovcUTO9l6dCBZx17me-sagnLFfsO8unYBA0pQ3qOKWUUqBlLAW0elxuDBNTg which generates ["ya29", "", "vgIJcUovcUTO9l6dCBZx17me-sagnLFfsO8unYBA0pQ3qOKWUUqBlLAW0elxuDBNTg"] causing var base64Url = token.split('.')[1]; to receive an empty string.
I have no ide why google is now sending this token faulty token, as i haven't change anything and this was working until last week...
Here is my server side code (using node)
app.post('/node/auth/google', function(req, res) {
  var accessTokenUrl = 'https://accounts.google.com/o/oauth2/token';
  var peopleApiUrl = 'https://www.googleapis.com/plus/v1/people/me/openIdConnect';
  var params = {
    code: req.body.code,
    client_id:  req.body.clientId,
    client_secret: config.credentials.google.secret,
    redirect_uri: req.body.redirectUri,
    grant_type: 'authorization_code'
  };

  // Step 1. Exchange authorization code for access token.
  request.post(accessTokenUrl, { json: true, form: params }, function(err, response, token) {

    var accessToken = token.access_token;
    var refreshToken = token.refresh_token;
    var headers = { Authorization: 'Bearer ' + accessToken };

    // Step 2. Retrieve profile information about the current user.
    request.get({ url: peopleApiUrl, headers: headers, json: true }, function(err, response, profile) {
      if (profile.error) {
        return res.status(500).send({message: profile.error.message});
      }
        var clientResponse = {
          profile : profile,
          token:accessToken,
          refresh_token:refreshToken
        };
        res.status(200).send(clientResponse);
    });
  });
});

Hi,
I think the access_token you are referring to is not a JWT (JSON Web Token).
But you may also retrieve from the token URL (personnaly, I use https://www.googleapis.com/oauth2/v4/token) another field called id_token which is a JWT and from which you should be able to extract the exp claim.

@guillaumegarcia13 Well just as it stopped working out of nowhere, it started working again, google is sending me the right token, but i still don't know what happened


How to create naturally spawning mobs

Hello,
The question is basically in the title already. I believe I read somewhere that one can let magic mobs naturally appear but I haven't been able to figure out how to do this. Could you explain it to me?
Thanks,
leveljaap

You need the MagicWorlds plugin:
https://github.com/elBukkit/MagicWorlds/wiki#randomly-spawning-magic-mobs

Alright thanks!


Spellcheck README.md

Hi!   First of all, thank you for this wonderful work!
I just ran a spell checker for README.md.

Thank you very much.


Idle Disconnect

After ~1 minute of inactivity, the connection closes (unexpectedly to the user). See https://stackoverflow.com/questions/49408031/websockets-in-chrome-and-firefox-disconnecting-after-one-minute-of-inactivity

Disconnect the user after a certain amount of time (e.g. 15 minutes of inactivity).

Disconnect the user after a certain amount of time (e.g. 15 minutes of inactivity).

Disconnect the user after a certain amount of time (e.g. 15 minutes of inactivity).

I am not planning on implementing this for now.


xdebug cannot be installed because the base image has been updated to php8.

Describe the bug

When updating /etc/php/7.4/apache2/php.ini in the directory used for xdebug configuration, the /etc/php/7.4 directory can no longer be found. The reason for this is the update of the Gitpod base image to PHP 8.
https://github.com/apolopena/gitpod-laravel-starter/blob/a26165e22035f442b4d09607c246b29272875c61/.gitpod.Dockerfile#L26
Screenshots

Steps to reproduce

After adapting the .gitpod.Dockerfile to your own needs, e.g. updating the root CAs, GitPod cannot build a working Docker image.
If you leave the .gitpod.Dockerfile unchanged, the existing Docker image can be successfully loaded from the GitPod registry.
Expected behavior

Additional information

Thanks for pointing this out.

Until php 7.4 reaches end of life or at the very least the php 8 adoption rate exceeds that of php 7.4, php7.4 must still be supported by this software. Dynamically (vis starter.ini) supporting different version of php introduces complexities I would rather not address however such is the nature of software engineering.
This fix will take some time as I will be adding support for php_version (either gitpod_latest or 7.4.*) in starter.ini.

This issue blocks the entire project as the workspace image fails to build.
This issue depends on feature https://github.com/apolopena/gitpod-laravel-starter/issues/156


Line numbers are not expanding

There seems to be a weird wrapping issue with line numbers in 2.13 that wasn't present before. Any line number longer than 3 digits is wrapping to the next line in both hex and decimal.

Came here because of this issue as well.
It appears the issue is only reproducible with the "Monaco" font selected though. With hexadecimal line numbers, the last digit always breaks onto a new line, no matter what. The issue exists with decimal line numbers as well, but since those are not prefixed with zeroes, it only happens at the bottom of the file, where the line numbers become long enough to span the entire column.

Thanks for reporting. I'm looking into it.

Looks like c4104ce32f93f1fa208485252c9491c0388b34d1 is the culprit.

Should be fixed now. Can you please test out this version? I'll make it official if it looks good.
https://github.com/ridiculousfish/HexFiend/releases/tag/v2.13.1b1

Yes, that build fixes it.
Thanks for the insanely fast handling. :)

I think the difference is the use of CTFontExtractGlyphsForCharacters.
By the way, this is going off a bit of a tangent, but are there any plans to remove the support code for old targets?
I am guessing it's there for framework users and maybe so people can build for older systems if they need, but maybe a stable, rock solid final tagged release could be worked on that includes the code, still maintain it to a certain extent (fixes for critical bugs, security issues if they appear) but clean out the unused target and architecture conditionals/compatibility types in the master, release config builds with minimum OSX 10.10 I believe? Like there is a lot of boilerplate to juggle the HFFloat/CGFloat/float/double/long double/NSUInteger/unsigned long/unsigned long long and far too many more types and their conversions and operations.
At the very least I'm pretty sure the PPC code could go (though I don't know how much of it is actually functional, does anyone have something with a G4 to test?) maybe even i386 (I think all Intel Macs that could run Snow Leopard officially runs x86_64 binaries), and I know I've kept whining about this without doing much, but I think an under-the-hood UI revamp would be hugely welcomed by many, it'd make new features easier to implement.
But then again, apparently Macs are also going arm in a couple years now, so 🤷
Really curious how Rosetta 2 will fare in terms of performance, but I'm pretty sure Universal 2 binaries won't include i386
Don't quote me on any info below, I sort of stopped trying to get 73d5122 working because I thought it might be worth it trying to subclass standard AppKit components and not lose performance
Apparently text scaling changed on .15 for Catalyst related reasons if I understood correctly (they changed scaling to make the same point size font look similar on iPad and Mac) But the related property is in UI/NSTextView and attributed strings, which apparently results in them looking not even close on Mojave Macs and Catalina Macs if you're doing custom glyph rendering.

So much better. Thank you!


InflateException: Binary XML file line #11: Error inflating class com.yalantis.taurus.PullToRefreshView

InflateException: Binary XML file line #11: Error inflating class com.yalantis.taurus.PullToRefreshView
Caused by: android.view.InflateException: Binary XML file line #11: Error inflating class com.yalantis.taurus.PullToRefreshView
do you know why does this exception happens?
thanks in advance.

this is my xml file:
<com.yalantis.taurus.PullToRefreshView
    android:id="@+id/pull_to_refresh"
    android:layout_width="match_parent"
    android:layout_height="match_parent">

<ListView
    android:layout_width="match_parent"
    android:layout_height="match_parent" />

</com.yalantis.taurus.PullToRefreshView>

I think it's totally ok,but why did the exception happens?

Hi there! Please check the updated version of project. Maybe this issue is gone. I've just tested it on my device - everything works fine.


[fix] Tests fix

There's a failure in Dashboard test that kills Protractor as usual.
2017-10-10 08:46:21,703 INFO [root] searching for title Recent Updates
[08:46:21] E/launcher - There must be present a table Top 5 Integrations: expected false to be true
[08:46:21] E/launcher - AssertionError: There must be present a table Top 5 Integrations: expected false to be true

Lets merge this manually as templates are broken now. Due to re-encrypt route issue...


Updating workflows to use shells

Coverage increased (+0.2%) to 80.842% when pulling fa68babd1594f29ad59532c0fb8438fbee2b71eb on workflow-shell into 276510be97380e17606d0328c91ba37a7b84f61a on dev.


Quickstart should have a multi-job section + add forward pointers to TFlow.

Just talked about this one on Skype.  I'd like the quickstart to make two points:

That you probably want to be running DNNs, but let's first talk about running "something".  It's too bad we couldn't have a DNN here.

That you probably want to run on lots of videos.  So let me show you how to run on one.  Then at the end let's talk about running on a whole directory of videos.  (Hypothetically a whole directory... might want to tell them to download a bunch of videos into a directory.)

Done


Multi-checkbox code bug

This line reads:
if(opts.expressionProperties && opts.expressionProperties.required){

This will never evaluate to true because required is an invalid property for the root of the form config. It should read:
if(opts.expressionProperties && opts.expressionProperties['templateOptions.required']){

Sanity check? @ckniffen

Very sane. Great catch.

Released as 4.4.1


DataRefresh extension function throws exception for subclasses of MutableCommonDataEntity

Using the DataRefresh extension function on a direct subclass of CommonDataEntity works just fine. For example, to refresh an immutable email,
var email: Email
email = email.refresh(contactsApi)

An exception?!
However, when trying to refresh a mutable email,
var email: MutableEmail
email = email.refresh(contactsApi)

Will throw an exception,
java.lang.ClassCastException: contacts.core.entities.Email cannot be cast to contacts.core.entities.MutableEmail

Why?
The reason this occurs is because of the function signature,
fun <T : CommonDataEntity> T.refresh(...): T? =

Combined with the following class hierarchy,
interface CommonDataEntity
interface MutableCommonDataEntity : CommonDataEntity

data class Email : CommonDataEntity
data class MutableEmail : MutableCommonDataEntity

Allows for the refresh extension function to be callable for MutableEmail given that it is an indirect subclass of CommonDataEntity. However, the refresh function itself only returns immutable instances. So,
var email: MutableEmail?
email = email?.refresh(contactsApi) // returns an instance of `Email`, which cannot be casted to a `MutableEmail`.

Technically fixed in https://github.com/vestrel00/contacts-android/pull/114/files#r757986465 but there is a remaining TODO

This is a part of v0.1.9 Release Checklist.


Lend Request

User Stories
As a borrower I want to easily request the item I want to lent.
As an owner I want get notified when a borrower sends a lending request.
Dependencies
Depends on:

#33
#14
#11

UI Mockup

Context

later implementation of #28

Acceptance Criteria

[x] Lend request button is functioning
[x] Button press notifies owner
[ ] Button press starts the lend
[x] Lending only possible when item is not in lend

Am I correct that this issue was completed with the merge of #192? If so, please close the issue


💡 Feature Request - RoleAssignments for PolicyAssignments on RG-Scope

Describe the solution you'd like
Currently only MG and Subscription scoped RoleAssignments for Policiy-Assignments are able. I need the same for Resource-Group Scope
Describe alternatives you've considered
Additional context
RG Scope is sometimes needed if you want to have a more specific Role Assignment instead of using the subscription Scope.

Thanks @SeSeicht have assigned to you as we discussed.


requireUser sometimes hangs

Hi,
we experience some resolving/rejecting issue in our application. we call requireUser but the then handlers are never called.
I started todo some logging in the metoer-angular-auth but it looks ok
it autoruns, finds a valid user, enqueues the deferred.resolve int the $$afterFlush, the Tracker.afterFlush gets also called, but then it disappears.
any ideas what the problem might be?

@mxab I tried to test this issue and it all seems fine for me. I tested it using angular-meteor-whatsapp project, I used requireUser instead of awaitUser before resolving the routes (see referene), and I even tried to fulfill the promise in ChatsCtrl initialization (see [reference(https://github.com/DAB0mB/angular-meteor-whatsapp/blob/master/client/scripts/controllers/chats.controller.js#L4)]) and it worked properly as expected. I used<EMAIL_ADDRESS><EMAIL_ADDRESS>and<EMAIL_ADDRESS>Are you sure you didn't miss anything? Please provide me an example so I can analyze your issue.

it is really hard to reproduce it as we experience it randomly. I will try to replace it with awaitUser. Out of curiosity, the afterFlush seems very intentional in the requireUser part. I assume I should also do a Tracker.afterFlush then?

Hey, I referenced a pull request which should fix this issue. Can you please confirm it that the issue has been fixed? It would be great!

The afterFlush() is there so the promise would be fulfilled once the current computation is finished, otherwise subscriptions made in the promise would be stopped. There is no need to call afterFlush(), just use the API as is.

Hmm I swtiched everything to await, but sill I have this random not finishing issue.
This is the peace of code that currently is the problem
In my $stateProvider.state({..}) definition:
 resolve: {
  'currentUser': function ($auth, $log, $timeout) {
    const awaitUser = $auth.awaitUser();
    $log.debug("awaitUser in AUTH");
    $auth.finally(()=> {
      $log.debug('DONE AWAITING USER');
    });
    return awaitUser;
   }
}

Quite often but so far random I see
awaitUser in AUTH
but not
DONE AWAITING USER
which does not make any sense to me

Hmm I can't seem to reproduce your problem, everything works fine for me. Make sure you actually login before invoking awaitUser.

@DAB0mB I'm working on the same project with @mxab
The problem occurs explicitly inside of the resolve of a $stateProvider.state() setup of ui-router
Following code does not work and accidentally fails for no reason - the .finally()of awaitUser will never getting called. I replaced now the $auth.awaitUser() with standard Meteor code and it works. I am pretty sure that the reason belongs in the context of the resolve handler. Inside of a controller the code works without any issues.
        $stateProvider.state('auth', {
            url: '',
            abstract: true,
            resolve: {
                'currentUser': function($auth, $log, $timeout, $q) {
                    var defer = $q.defer();
                    Meteor.autorun(c => {
                        let loggingIn = Meteor.loggingIn();
                        if (!loggingIn) {
                            if (Meteor.userId()) {
                                defer.resolve();
                            } else {
                                defer.reject();
                            }
                            c.stop();
                        }
                    });
                    return defer;
                }
            }
        })

Ok I made a very small change inside of angular-meteor-auth.js in line 118:
I changed:
	    var computation = this.autorun(function (computation) {

to
	    var computation = Meteor.autorun(function (computation) {

And all now the resolve handler works as expected. I'm getting the feeling that this.autorun, this.subscribe is only working in controller context and not inside of any other context.

One important thing how to actually trigger this error, because most of the time on page reloads it works:
Open the link with the resolve handler with CMD+clickand open mutliple times the link in new browser window. Then the resolve handler starting to stuck

@dohomi you mentioned something about subscriptions. We actually had a similar issue which was already fixed, https://github.com/Urigo/angular-meteor-auth/issues/8. Sadly I can't reproduce this problem so I wish I could help you but I can't, I used awaitUser in many of my projects and I never encountered any problems using it (unless you use older version of angular-meteor or angular-meteor-auth). If you can provide a repo which reproduces the problem I would be glad to help you. If you manage to find the source of the problem and a solution to it please share it with us so we can all enjoy it.

Maybe there some sort of a synchronous error thrown in the resolve handler and angular silently ignores it.

I opened this issue where I had to switch back to Meteor.subscribe:
https://github.com/Urigo/angular-meteor/issues/1338

Is there anything against changing this.autorun to Tracker.autorun? For us it would solve the issue for now.

@dohomi @mxab I'm having troubles reproducing the issue. If we can may a call or something it would be much easier and we can go through it together. Feel free to contact me on the email address presented in my Github account and I would be more than happy to listen and try to help you :-)

Sounds good, we are currently trying to isolate the issue without luck, I want to make some more tests. I first got the feeling that the defer.resolve(user) somehow does not trigger the then but currently it looks more like that the autorun is not that reactive on the logginIn() call as it should.

Hi @DAB0mB I did some more debugging. I still don't know why this is only happening in our project but what I found out that something stops the autorun that does checks for Meteor.loggingIn()
This is my own implementation in my ui-router resolve that I think does pretty much the same thing as the awaitUser
var deferred = $q.defer();

var computation = Tracker.autorun(function (c) {

    var loggingIn = Meteor.loggingIn();
    console.log('autorunning...');
    if (!loggingIn) {
        console.log('loggingIn done!');

        Tracker.afterFlush(function () {
            var user = Meteor.user();

            if (user) {

                deferred.resolve(user);
                console.info('resolving user');
            } else {

                deferred.reject('AUTH_REQUIRED');
                console.error('REJECTING');
            }
        });
        console.log('stopping');
        c.stop();
    } else {
        console.log('loggingIn...');
    }
});
computation.onInvalidate(function () {
    console.trace('onInvalidate');
});
computation.onStop(function () {
    console.trace('onStop');
});

deferred.promise.finally(function () {
    console.log("=================DONE=================");
});

return deferred.promise;

Everytime I reload the page the onStop functions gets called but not because of the loggingIn is false but if I look at the stack trace from something outside.
onStop(anonymous function) @ globalStates.js:69
(anonymous function) @ tracker.js:307
Tracker.nonreactive @ tracker.js:589
Tracker.Computation.stop @ tracker.js:306
(anonymous function) @ tracker.js:567
(anonymous function) @ tracker.js:284
Tracker.nonreactive @ tracker.js:589
Tracker.Computation.invalidate @ tracker.js:283
Tracker.Dependency.changed @ tracker.js:417
$$Reactive.$$changed @ angular-meteor.js:2439
boundMixin.(anonymous function) @ angular-meteor.js:1796
(anonymous function) @ angular-meteor.js:2287
Scope.$digest @ modules.js?hash=a85fbee…:54246
$$Core.$$throttledDigest @ angular-meteor.js:1991
boundMixin.(anonymous function) @ angular-meteor.js:1796
(anonymous function) @ angular-meteor.js:1665
Tracker.Computation._compute @ tracker.js:323
Tracker.Computation._recompute @ tracker.js:342
Tracker._runFlush @ tracker.js:481
onGlobalMessage @ setimmediate.js:102

As the autorun is stopped the deferred is never resolve nor rejected, which seems to be our issue.
If I surround my whole implementation with another Tracker.afterFlush it seems to work
var deferred = $q.defer();
Tracker.afterFlush(function () {
 var computation = Tracker.autorun(function (c) {
...

Just some more notes on the issue, it seems that during the setup of the autorun there is an active computation Tracker.active seems to be true, which as as far as I get it stops the autorun on invalidation
//Tracker.autorun
...
  if (Tracker.active)                                                                                // 565
    Tracker.onInvalidate(function () {                                                                                // 566
      c.stop();                                                                                // 567
    });

which then leads the accounts package when it does the _setLoggingIn(false) to activate the invalidate the dependencies and stops the autorun that fulfills/rejects the promise
Unfortunately it seems that the Tracker.active is always true, but the stopping only happens sometimes/randomly

It maybe related to the same issue that is happening here
https://github.com/meteor/react-packages/issues/99#issuecomment-191648907

@mxab I'm not sure which package is causing this problem, but look at the test scenario I wrote:
it('should keep waiting for user once an outer computation has stopped', function(done) {
  login();

  function login() {
    Accounts.login('tempUser').onStart(function() {
      awaitUser();
    }).onEnd(function() {
      scope.$$afterFlush('$$throttledDigest');
    })
  }

  function awaitUser() {
    var c = Tracker.autorun(function(c) {
      Tracker.onInvalidate(function() {
        c.stop();
      });

      scope.$awaitUser().then(done);
    });

    c.invalidate();
  }
});

It seems like this is the case, am I right? So it means it's not an issue related only to angular-meteor-auth but to any peace of code which is using Scope#autorun().
I hope I understood you correctly, and if so I will solve it by making this test pass and then you can test your app with my fix to see if it really works.

Hi @DAB0mB
as we found out in our research that the react package has a similar problem we think its internally a bug of Tracker.autorunin edge cases. We solve it now with wrapping $auth.awaitUser into a small service setting a timeout:
 const deferred = this.$q.defer();
 setTimeout(()=> {
            this.$auth.awaitUser().then(deferred.resolve,deferred.reject);
 });
 return deferred.promise;

This works now as expected and the promise resolves correctly.

@DAB0mB not sure I fully understand what you test does
So the problem as far as I see it is that the $auth.awaitUser is called but there is an active computation and the loggingIn autorun is treated as a nested autorun and is therefore stopped
if(Tracker.active){
 console.error('active computation',Tracker.currentComputation._func);
}

points to a function in the rootScope service, but not sure if this is really the issue

@mxab @dohomi I'm working on a proper fix an it should be ready today I believe

@mxab @dohomi can you test your app against my changes? Note that I changed both angular-meteor and angular-meteor-auth

@DAB0mB ok will do, which commit/branch should i use for the angular-meteor package?

ok tested it with
"angular-meteor": "https://github.com/DAB0mB/angular-meteor.git#8a4c3dc93b940fb147a00470b33ad79cf1380be1",
    "angular-meteor-auth": "https://github.com/DAB0mB/angular-meteor-auth.git#5cb1bcc2030790710063a2cb6ab6833feb73c6f5",

Unfortunately it sill hangs, i put some log outputs in the package to the auths,
if(Tracker.active){
  console.warn("TRACKER ACTIVE", Tracker.currentComputation)
  debugger;
}
var computation = this.autorun(function (computation) {
...

During my resolve call in the ui router there seems to be an active computation

OK they don't seem like the correct branches, try it with https://github.com/DAB0mB/angular-meteor-auth/tree/fix/%2322 and https://github.com/DAB0mB/angular-meteor/tree/feat/auto-promise. Sorry for the late response

Tried this one as well, no difference, I looked a bit more at my stacktrace. The active computation comes from another awaitUser that is happening in a angular run handler. which is really strange. I also tried to isolate this example (having a run that awaits the user and a resolve in the ui router that awaits the user) but with no luck.

I just noticed that this problem is not only affecting the awaitUser. We experiencing several this.subscribe(...) calls are getting stopped before they finish and they do this always if there is a Tracker.active is already true. The problem is that it happens randomly and in the constructor of Controller.
class MyController{
 constructor(){}
}

Hi everyone, what is the status of this bug? I belive i have the same problem but as i see fix is waiting for some time now?

@otporsad it is kind of good to hear that someone else has this issue as well. I found out that this issue is actually not only affecting the auth package but more or less all initial subscriptions as well (randomly).
One thing how I finally mitigated this problem is to remove all direct calls for $auth.awaitUsers in angular.run(...) functions and put them inside a $timeout.
Could you check if this might help in your case as well?

My problem is with resolve in router.
For example:
 resolve: {       currentUser: ['$auth', function($auth) {           return $auth.awaitUser();       }],
Now the app hangs while loading that route. If i comment out return $auth.awaitUser(); route loads as expected.
I have also one more resolve item that calls a service that has tracker autorun and it hangs to. I dont know if that is related.
If i put $auth.awaitUser in $timeout like this:
 resolve: {       currentUser: ['$auth', function($auth, $timeout) {         return $timeout(() => {           return $auth.awaitUser();         }, 0);       }],
but still problem is not solved. I have been keeping an eye on this issue for days hoping someone smarter will figure something out but no luck :(
But worst thing is that sometimes it reloads and everything works as expected. In my experience this happends when browser window is open and not touched. Then following a file edit meteor reloads the page and route sometimes works as expected, but mostly not. If there is anything i can help to test or solve this please let me know.

Could you check the stacktrace in the resolve call to see if and which computation may already be running?
resolve: {
  currentUser: ['$auth', function($auth) {
    if(Tracker.active){ // appears randomly
      console.trace('ALREADY in computation');
    }
    return $auth.awaitUser();
  }
}]}

Also do you use the tmeasday:publish-counts or the meteorhacks:unblock package by any chance?
Tracker.Computation._compute @ tracker.js:323 <---------------------------
Tracker.Computation @ tracker.js:211<---------------------------
Tracker.autorun @ tracker.js:588<---------------------------
$$Core.autorun @ angular-meteor.js:1900<---------------------------
boundMixin.(anonymous function) @ angular-meteor.js:1796<---------------------------
$$Auth @ angular-meteor-auth.js:90<---------------------------
(anonymous function) @ angular-meteor.js:1756
_construct @ angular-meteor.js:1755
(anonymous function) @ angular-meteor.js:1738
mixin @ angular-meteor.js:1737
(anonymous function) @ angular-meteor-auth.js:204
invoke @ modules.js?hash=371aed9…:29137
(anonymous function) @ modules.js?hash=371aed9…:28945
forEach @ modules.js?hash=371aed9…:24749
createInjector @ modules.js?hash=371aed9…:28945
doBootstrap @ modules.js?hash=371aed9…:26179
bootstrap @ modules.js?hash=371aed9…:26200
onReady @ app.js:57
fire @ jquery.js:3187
fireWith @ jquery.js:3317
ready @ jquery.js:3536
completed @ jquery.js:3552

Just to be sure, that is really not resolving, could you extend the resolve:
resolve: {
  currentUser: ['$auth', function($auth) {
    var isActive = Tracker.active;
    if(isActive){ // appears randomly
      console.trace('ALREADY in computation');
    }
    return $auth.awaitUser().finally(function(){
         console.log("WAS COMPUTATION ACTIVE?", isActive)
    });
  }
}]}

The problem should be there if you don't see the `WAS COMPUTATION ACTIVE?``
And one more thing could you show how you start the app (onReady @ app.js:57)

I start my app with:
onReady = function () {     angular.bootstrap(document, ['meteorDentalApp']); };
I do not see WAS COMPUTATION ACTIVE? only: ALREADY in computation.

can you show  me what is at angular-meteor-auth.js:90 and angular-meteor.js:1756 and arround

angular-meteor-auth.js:90
function $$Auth() {
	    var vm = arguments.length <= 0 || arguments[0] === undefined ? this : arguments[0];

	    // Reset auth properties
	 /* 90 */   this.autorun(function () {
	      // Note that we use Meteor and not Accounts since the following methods are
	      // not available in older versions of `accounts-base` meteor package
	      vm.currentUser = Meteor.user();
	      vm.currentUserId = Meteor.userId();
	      vm.isLoggingIn = Meteor.loggingIn();
	    });
	  }

angular-meteor.js:1756
	  // Invoke function mixins with the provided context and arguments
	  this._construct = function (context) {
	    for (var _len = arguments.length, args = Array(_len > 1 ? _len - 1 : 0), _key = 1; _key < _len; _key++) {
	      args[_key - 1] = arguments[_key];
	    }

	    _this._mixins.filter(_underscore2.default.isFunction).forEach(function (mixin) {
	/* 1756 */      mixin.call.apply(mixin, [context].concat(args));
	    });

	    return context;
	  };

hmm @DAB0mB @Urigo I'm not sure but would it make sense to put the initial autorun in the $$Auth constructor in an timeout?

Good morning all, joining the group with the exact same issue.
I am going to investiguate more to find out the root cause of this ...

@anymos You can try to check if it works for you if you start angular after the initial flush
Tracker.afterFlush(function(){
 angular.bootstrap(...)
})

@mxab, that s interesting, I was thinking that using an afterFluch as you was suggesting to startAngular would have worked.
I still have the tracekrActive even so
I will continue to try to find out, it must be somewhere in the code ;-)

ok, this was just an @otporsad I discussed, for me it kind of works to make sure that any potential autorun inside a angular run is scheduled after a $timeout

I agree that there something linked to the $timeout, this is actaully waht I use at the moment to overcome the issue.
But I am not really sure what is the root cause, I have another application that is really similar to the one I have trouble with.
I am still investigating comparing the code line by line to try to find out the root cause of it.
it s a matter or time, but I will find it

ok progressing it s coming, in my case from a node_modules.
Trying to find out which one now

I found the offensive package:
I have changed : "angular-ui-router": "^0.3.1", to "angular-ui-router": "^0.2.18" and the problem disappear.
Absolutely no idea why it is like that. but ti solved it.
such a pain to nail it down.
I believe the next step would to find out what is the difference between those version to get the root cause of the problem
@mxab, could you kindly check on your side if you have the same version and if it s solving the issue ?

0.3.1, I can try to check if it works, but as far as I remember the issue was quite random, and I need to revert all the runs

"angular-ui-router": "^0.3.0" is having the same problem

for me its a 100% coming up at each time, it s why I am taking the time to find out where it s coming from, as from what I read a lot of time it s random and thus impossible to find out

unfortunately not that much time left to investigate until the end to find out the real root cause.
I fail to understand why and how router ui can have a side effect on the meteor computation


Add missing optional peer dependencies

This diff adds the missing optional peer dependencies on rxjs and zen-observable (required because of the transitive dependency on any-observable, cf https://github.com/sindresorhus/any-observable/pull/25). Without that, Yarn will refuse giving any-observable access to the Observable implementations.
Because it only uses peerDependenciesMeta (and not peerDependencies), this diff is entirely compatible with other package managers (it won't cause warning to appear anywhere).

Thanks!


Add GPX export support

Support export data as gpx files. All files ziped as Gpx.zip.
Each Tile consists of 8192*8192 pixels. By utilizing the latitude and longitude of the bounds for each Tile, we can calculate the corresponding latitude and longitude for each pixel within the Tile, proportionally. This allows us to obtain the latitude and longitude points for each Tile using the LngLatToGlobalXY function.
Creating a GPX file from a list of latitude and longitude points is a bit complex since the original image data doesn't include timestamps, making it impossible to reconstruct the GPX path. To address this, I attempt to reconstruct the original path by finding the closest point for each given point using the sortFilledList function.
Furthermore, to prevent abrupt jumps in the GPX point connections or to avoid excessively large GPX files, I split each Tile into multiple GPX files.
Finally, all the GPX files are packaged into Gpx.zip for downloading using the dumpGpx function.

Hey, this is awesome! thank you so much!
However, I'm kinda busy recently and I'll aim to finish reviewing this in a week or two.
In the meantime, could you try to solve some hygiene issues and make it pass our CI?
Some things I noticed:

remove files changed under frontend. I don't think this PR touches things there.
remove package-lock.json, we use yarn instead of npm.
changes to editor/yarn.lock seems related to switching npm.
make sure code is formated correctly, you can do this by running yarn autofix

It seems the issues have been fixed.
I am an Android developer and not very familiar with frontend technologies, if there are any further issues, please leave a message.

Thank you so much! No worries, I can also clean up the PR for you (just need a bit of time)

Hey! Thank you again and sorry for the long wait.
I modified your code a bit, and you can find the modified version here: https://github.com/CaviarChen/fog-machine/tree/pr-101
If you are happy with it, could you merge it to your branch?
Mostly, things I did are:

Code clean up and refactor.
Move most code to GpxExport.ts because for an experimental feature, having all code in a single place feels better.
Drop the timestamp. It is valid for a gpx file to not have timestamp and having wrong information is wrose than not having it.

For an experimental feature, the current state is pretty good, I'm ok with merging it before making it perfect. So I left a lot of TODOs in the code instead of review comments. We could always improve it later.
Among all these TODOs, I believe https://github.com/CaviarChen/fog-machine/blob/pr-101/editor/src/MainMenu.tsx#L205-L208 is the most important one. The current version is not user friendly enough and may freeze the browser.


[layer] Add checks for layer tensor overwrite bug

This patch adds a check to ensure that when layer tensors are created
and overwrites existing tensors. These checks are enabled only in ENABLE_TEST
mode to ensure that they only run in CI mode, and are called after each
operation - forwarding, calcGradient, and calcDerivative.
Signed-off-by: Parichay Kapoor<EMAIL_ADDRESS>

V2: This patch enables debug mode for the CI build for both ubuntu and
tizen. This enables all the debug tests to be done in the CI which were
disabled till now.
Fixes required to enable the DEBUG mode is also added.


thjs-28: add dislike logic

https://i.gyazo.com/c6783149631c8ca1d489fba9ab074f12.png - test
add logic for dislike and show reaction for all users using socket

bad branch


Where to find cxlactivitylogger?

I'm messing around with trying to get Tensorflow-rocm running on Arch Linux, I ran into quite a few wrinkles so far, all resolvable, but now, I ran into one I can't seem to resolve.
Long story short, on your official documentation refers to a package called cxlactivitylogger, which I can't use, so, I'll need to build it from source.
However, I can't find it anywhere. How can I go about building this from source?

I’ve been using this recipe for that purpose on NixOS.

Thanks a lot! I was looking in all the wrong places heh. I was searching for it in RadeonOpenCompute, not GPUOpen-Tools.

If you need compiled Debian, you can get at http://repo.radeon.com/rocm/apt/debian/pool/main/c/cxlactivitylogger/
ROCm Profiler/CXLactivitylogger is an archived and old project as per https://github.com/rocmarchive/ROCm-Profiler
Same has been replaced with https://github.com/GPUOpen-Tools/RCP
If you issue resolved, you can close this issue.


[Event Request] Codeunit 6620 "Copy Document Mgt." - OnCopySalesReturnRcptLinesToDocOnBeforeCopyLines, OnBeforePurchRcptLineCopyItemTrkg, OnBeforeInsertOldPurchLine

Please add the following Events OnCopySalesReturnRcptLinesToDocOnBeforeCopyLines, OnBeforePurchRcptLineCopyItemTrkg, OnBeforeInsertOldPurchLine, add the following parameters ToPurchLine, FromPurchLineBuf, RecalculateLines in Codeunit 6620 "Copy Document Mgt."
   procedure CopySalesReturnRcptLinesToDoc(ToSalesHeader: Record "Sales Header"; var FromReturnRcptLine: Record "Return Receipt Line"; var LinesNotCopied: Integer; var MissingExCostRevLink: Boolean)
<...>
                    end else
                        CopyLine := FromSalesLineBuf.FindSet() and FillExactCostRevLink;

                    UpdateWindow(1, FromLineCounter);
                    //>><< New event
                    OnCopySalesReturnRcptLinesToDocOnBeforeCopyLines(ToSalesHeader, FromReturnRcptLine, FromSalesLineBuf);
                    if CopyLine then begin

    procedure CopyPurchRcptLinesToDoc(ToPurchHeader: Record "Purchase Header"; var FromPurchRcptLine: Record "Purch. Rcpt. Line"; var LinesNotCopied: Integer; var MissingExCostRevLink: Boolean)
<...>
                            then begin
                                //>><< New event
                                OnBeforePurchRcptLineCopyItemTrkg(ToPurchHeader, ToPurchLine, FromPurchLineBuf, RecalculateLines);
                                if CopyItemTrkg then begin

    procedure CopyPurchInvLinesToDoc(ToPurchHeader: Record "Purchase Header"; var FromPurchInvLine: Record "Purch. Inv. Line"; var LinesNotCopied: Integer; var MissingExCostRevLink: Boolean)
    var
<...>
        IsHandled: Boolean;
<...>
                    if IsTimeForUpdate() then
                        UpdateWindow(2, ToLineCounter);
                    //>> New event
                    IsHandled := FALSE;
                    OnBeforeInsertOldPurchLine(ToPurchHeader, FromPurchLineBuf, OldInvDocNo, OldRcptDocNo, NextLineNo, SkipCopyFromDescription, InsertCancellationLine, IsHandled);
                    IF NOT IsHandled THEN BEGIN
                        //<< End
                    ShouldInsertOldPurchDocNoLine := "Receipt No." <> OldInvDocNo;
<...>
                    if ("Document No." <> OldRcptDocNo) and ("Receipt Line No." > 0) then begin
                        OldRcptDocNo := "Document No.";
                        InsertOldPurchCombDocNoLine(ToPurchHeader, OldInvDocNo, OldRcptDocNo, NextLineNo, true);
                    end;
                        //>><< New line
                    END;
<...>
                        FromPurchInvLine.Get("Receipt No.", "Return Shipment Line No.");

                        //>><< New parameters ToPurchLine, FromPurchLineBuf, RecalculateLines
                        OnCopyPurchInvLinesToDocOnBeforeCopyItemCharges(FromPurchInvLine, NextLineNo, ToPurchLine, FromPurchLineBuf, RecalculateLines);
                        if Type = Type::"Charge (Item)" then begin

    [IntegrationEvent(false, false)]
    local procedure OnCopySalesReturnRcptLinesToDocOnBeforeCopyLines(var ToSalesHeader : Record 36;FromReturnRcptLine : Record 6661;var FromSalesLineBuf :  Record 37 temporary);
    begin
    end;

    [IntegrationEvent(false, false)]
    local procedure OnBeforePurchRcptLineCopyItemTrkg(var ToPurchHeader : Record 38;var ToPurchLine : Record 39;FromPurchLineBuf : Record 39;var RecalculateLines : Boolean);
    begin
    end;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeInsertOldPurchLine(var ToPurchHeader : Record 38;var FromPurchLineBuf :  Record 39 temporary;var OldInvDocNo : Code[20];var OldRcptDocNo : Code[20];var NextLineNo : Integer;SkipCopyFromDescription : Boolean;InsertCancellationLine : Boolean;var IsHandled : Boolean);
    begin
    end;

    [IntegrationEvent(false, false)]
     //>><< New parameters ToPurchLine, FromPurchLineBuf, RecalculateLines
    local procedure OnCopyPurchInvLinesToDocOnBeforeCopyItemCharges(FromPurchInvLine: Record "Purch. Inv. Line"; NextLineNo: Integer; var ToPurchLine: Record "Purchase Line"; var FromPurchLineBuf: Record "Purchase Line" temporary; RecalculateLines: Boolean);
    begin
    end;

Hello @AdrijaSipaviciute
where is the END for OnBeforeInsertOldPurchLine(...
You put IF NOT IsHandled THEN BEGIN but did not define where is closed.

Hello @AleksandricMarko
Sorry, my mistake. Edited my comment. Hope it is clear now.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 12349.


Generate auto-incrementing versions using GitVersioning

This adds Nerdbank.GitVersioning to all projects.  This package provides auto-incrementing revisions to the binaries and nupkgs, and the commit hash into the AssemblyInformationalVersion.  It also updates the version in the vsixmanifest file.

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

Unit test failures are a known and separate issue.  Going ahead with merging this PR.


Gradients don't export correctly

While testing this plugin I tried to recreate the Facebook loading animation (having a masked, gradient-filled "highlight" rectangle animate across some UI elements) and noticed that the gradient doesn't export correctly when exporting as a gif. See here: https://www.dropbox.com/s/tsew3buks4ybfxt/animateMate-gradientFail.gif?dl=0

It could be because of GIF compression. Have you tested out with exporting PNG sequence? If PNG looks ok, then it's gif compression. Using gradients in GIF images is not easiest, because there is limited amount of colors in gif... AnimateMate uses Gifsicle library.
Tip: There is great free app to playback PNG sequences http://djv.sourceforge.net/

Thanks, I did end up resorting to using a PNG sequence and re-exporting from Photoshop as a gif.


chore: Stabilize ActiveJob Instrumentation Tests

Prior to this change tests would intermittently fail due to threads executing out of order.
This change ensures that the AsyncAdapter runs in the main thread and blocks execution to
ensure jobs are completed before making assertions on spans.

why set the queue adapter to async before, then reset to inline after?

Since the queue_adapter is a global variable, I reset it to :inline so it does not bleed into other tests that may run after this one completes.

why rescue nil after queue_adapter.shutdown?

I do not want the test suite to halt if concurrent ruby gets an error. It is not significant to me. Also, the method does not exist on the inline adapter so if I switch it for another test this will fail with a NoMethodError.

why rescue nil after RetryJob.perform_later?

Because when running in the main thread the last execution will raise an error. In most queuing systems the exception is handled and thrown into a dead letter queue. When running the ImmediateExecutor it propagates the error to the caller. In this case it is the test case.

Because when running in the main thread the last execution will raise an error. In most queuing systems the exception is handled and thrown into a dead letter queue. When running the ImmediateExecutor it propagates the error to the caller. In this case it is the test case.

Thanks so much for the detailed explanation! This all makes sense to me 😄


Capture code is cleared when connect button is clicked

When using manual connection rather than URL, I enter the connection code, and when I click Connect, the capture code is cleared instead of sending it to the server.
In the log:
[ClientSocket]: Connection code () sent to server.
Looks like it's in the ConnectButton_Click handler
ConnectCodeBox.Clear();

For now I am just commenting that out and building it locally, but something you might want to fix for your next release.

Oops, pretty sure that's a leftover from when we were only using manual, and it detected that it didn't work. But I guess we commented out the lines above it and not that line lol.

Fixed in 5a3c1bc3314037828ed95296ad45cb86ac5d950c.


View logs buttons on bottom cut off

Rancher Version: Master 6/14
Steps to Reproduce:

Create a service
After container starts running, view logs

Results: No matter what size my browser is, the bottom buttons are cut off

Expected: Buttons should fit in page and scroll bar should not be needed.

Version - master 7/8


Teach In Module error message "BidCos-RF teach-in mode could not be activated"

When opening the RaspberryMatic http://homematic-raspi/ and starting the Teach In Module the error message "BidCos-RF teach-in mode could not be activated" (BidCos-RF Anlernmodus konnte nicht aktiviert werden.) is displayed
thanks for any help!

This is no support fora but a bug/issue tracker of KNOWN bugs in RaspberryMatic. If you cannot start the teach in mode this is certainly no general issue of RaspberryMatic but a user problem. So please go to http://homematic-forum.de/ and ask for help there instead. Only post issues here if you actually found a bug in RaspberryMatic or if you have an enhancement request to add a certain feature.


Mike Garrity lambda sprint challenge--user-interface

Completed the sprint challenge

Code reviewed during 1:1 👍


Fix issue where post.excerpt was not being used for Open Graph descri…

This is a bug fix.
Summary
This fixes this issue where Open Graph description (og:description) on posts uses the site description instead of the post.excerpt.
See issue https://github.com/mmistakes/minimal-mistakes/issues/2964

There's already page.excerpt. post has never been an automatic variable in Jekyll so you may be fixing the wrong thing.

@iBug I am new to Jekyll theme updates.
From this page https://jekyllrb.com/docs/posts/#post-excerpts it sounds like post.excerpt is a variable.
I have tested this fix on my personal website and it seemed to have fixed my issue.
I updated the issue to be more descriptive

When linking to a "post" in a system that supports Open Graph such as Slack or Twitter. The site description is used instead of the first paragraph of the "post" as expected.
The first paragraph is used as an Open Graph description (og:description) for pages and collections as expected.
I expected the default Open Graph description of a post to be the first paragraph of the post instead of the site description.

From this page jekyllrb.com/docs/posts/#post-excerpts it sounds like post.excerpt is a variable.

No, post is a variable because there's {% for post in site.posts %} outside. As I said, post is not an "automatic variable" that gets set based on context, unlike page or site.
I don't understand how it's an issue in the first place. Both page.excerpt and page.description should cover the OG metadata already well.

No, post is a variable because there's {% for post in site.posts %} outside. As I said, post is not an "automatic variable" that gets set based on context, unlike page or site.

@iBug okay it sounds like my fix is not good then.
I will close this pull requests and update the issue with more details and descriptions. Thanks for your feedback.


Add storage_mocks library to Google Cloud Storage

I'm not sure why the mock client wasn't already exposed, but I'm adding it here.

This change is

Hey, thanks for the PR, but I don't think we can accept it.

I'm not sure why the mock client wasn't already exposed, but I'm adding it here.

I have an answer, but I do not think you will be thrilled to hear it....
Adding a public storage_mocks library is something we want to do, and that work is tracked in #7142
The tl;dr for why we can't right now is that //google/cloud/storage:storage_client_testing contains both helpers for mocking the storage::Client (which we "support" and want customer to use) and helpers for the google-cloud-cpp library maintainers to test our internal code (which we do not support nor want customers to use).
To make things even more messy, the mocks which we "support" in google::cloud::storage::testing use internal types.... (which we tell customers not to use 😖).
So we will not add a new storage_mocks until we have decided the exact mocking API we want to expose for the long run. (e.g. no internal types).
As for what you should do now if you want to mock the storage library? You can either depend on //google/cloud/storage:storage_client_testing anyway, and follow the storage mocking guide (which you might be doing already). Or you can wait for #7142 to be completed. I don't have an expected date for when the work will be completed. I doubt it will happen in 2022. It is a very difficult problem to solve, that requires rearchitecting the library.

Thanks for the quick response. Will proceed with the former option you suggested.


change(network): Allow custom testnets to make peer connections and configure more parameters

Motivation
We want to support configuring more custom testnet parameters and to allow Zebra to make peer connections on custom testnets.
Depends-On: #8524
PR Author Checklist
Check before marking the PR as ready for review:

[x] Will the PR name make sense to users?
[x] Does the PR have a priority label?
[x] Have you added or updated tests?
[x] Is the documentation up to date?

For significant changes:

[x] Is there a summary in the CHANGELOG?
[x] Can these changes be split into multiple PRs?

If a checkbox isn't relevant to the PR, mark it as done.
Solution

Replace check for is_default_testnet() in initial_peer_hostnames() with !is_regtest()
Add the remaining configurable parameters in testnet::Parameters to config deserialization

Related changes:

Return early with an empty set from initial_peers() when network.is_regtest() (Zebra shouldn't ever add any peers to the peer cache on Regtest, because it has no initial peers, but it the file could be edited outside Zebra)

Testing
Updated the latest stored test config
Review
Anyone can review.
Reviewer Checklist
Check before approving the PR:

[ ] Does the PR scope match the ticket?
[ ] Are there enough tests to make sure it works? Do the tests cover the PR motivation?
[ ] Are all the PR blockers dealt with?
PR blockers can be dealt with in new tickets or PRs.

And check the PR Author checklist is complete.

Using the main branch i cherry picked the relevant commits and force pushed to fix the conflicts this PR had.


px4iofirmware: fixed a bug with override handling

this fixes a race condition between the DMA completion handler
updating registers in px4io and the mixer used for handling the
override state. The register set code could set r_page_servos[]
between the time when pwm_limit_calc() is called and the servos are
actually output.
@kd0aij Could you test this on a fixed wing? In particular the switch between manual and stabilised.

Rebased on master

This bench tests OK in that QGC reports switching between FW manual and stabilized as reported by QGC and aileron servo behaves correctly in both modes.
But I don't know how the bug manifested, or what exposed it.

Should this be closed if it's in master?

This looks correct to me.

Applied.


Second Crash

Hello, by cell phone I updated the mod and presented me with a new crash then

If the photos don't help with the size, as soon as I get home I'll upload a larger photo.
Note: I didn't update the font, is it necessary?

This should be resolved in https://github.com/ZhengPeiRu21/mod-playerbots/commit/d31fc287afd1f67f97e0b86820f9779bc281b4f1. Please notice if you still see any issues!


[RFE] Make the clipboard integration non-default

Reported by dpal on 7 May 2015 22:49 UTC
Add a way to disable clipboard integration.

Closing as outdated, this is already implemented.


Error while starting PHP with php_sapnwrfc

Hi,
I'm affraid I cannot use your module. I installed Apache/2.4.25 (Win64) with PHP 7.0.15 x64 TS as a module on Windows Server 2012.
There is no problem without  php_sapnwrfc and everything works smoothly. However when I add
extension=php_sapnwrfc.dll
to php.ini (as stated in README.md) and restart Apache I get an error:
PHP Warning:  PHP Startup: Unable to load dynamic library 'C:\PrimeSoft\php-7.0\ext\php_sapnwrfc.dll' - The specified module could not be found.\r\n in Unknown on line 0
The file php_sapnwrfc.dll exists. In fact it is "php_sapnwrfc-1.1.1-7.0-vc14-ts-x64.dll" renamed.
Are there any dependencies required? Visual C++ Redistributable for example? I think it already is installed (PHP works)...
Best wishes,
V3sp

Have you tried using an absolute path to the module in your php.ini?
@ThaDafinser any other ideas?

Same result when using absolute path.

@V3sp have you followed the guide here? https://github.com/gkralik/php7-sapnwrfc/blob/master/docs/installing_nwrfcsdk.md#windows
I think the SAP dependencies are missing, or your PATH is wrong for them.


[feature] update project to use .NET 8

.NET version updated to v8
.NET MAUI 8 migration done (implicit package references)
GitHub workflows updated
minor changes to copy/links/year across text assets

Changes as per guide:
https://github.com/dotnet/maui/wiki/Upgrading-.NET-MAUI-from-.NET-7-to-.NET-8
Build with Rider and run on Mac OS (Catalyst)
Thanks!

@Eilon are you planning on ever reviewing any of these pull requests?

@peterblazejewicz thank you so much for the PR!
@HelloooJoe apparently right now 😁 I'm catching up on a bunch of these items now.

And it only took a week or so to get the GitHub Actions working again 😅


Document variable and function identifier syntax

I can't find any place in the documentation where the rules governing valid identifiers (i.e., variable and function names) is documented. Empirical testing suggests that something like this text (from the fish-shell docs) would be appropriate:

A variable name can not be empty and can contain only letters, digits, and underscores. It may begin and end with any of those characters.

Thanks for reporting. Please file future reports against documentations in https://github.com/elves/elvish.io/issues (this one can remain here).


Dashboard: Add input to change publisher name

Feature Description
In the dashboard, there is currently an option to update publisher logo. However, there is no way to update the publisher name. This default's to the site name.  The publisher name could be different site name, so we should give the user the option to change this in the dashboard.
Alternatives Considered

Additional Context

Now that publisher logos are stored a story level. See https://github.com/google/web-stories-wp/pull/8946. There is also a question if like publisher logo, we should have a global publisher name but allow for an override at the story level.
Extra context. https://xwp.slack.com/archives/C013J2EGH8R/p1631703824049800

cc @gajasinski for product decision on this

Not sure we really need a dropdown to choose between alternate publisher names.
I think a simple text box should suffice.

Okay, cool—think my comment was while I was questioning the overall future look of text boxes in the editor. Does this need a mockup? I just envision a text box like the attached that says "Publisher Name" underneath "Author"

Yeah I don't think this necessarily needs a mockup as it's really a simple change.
It's just important that the field is pre-filled with the publisher name configured in the settings.
The big problem with our text boxes right now is that they don't have labels (which is really bad for accessibility!).
So with the publisher name filled in you have no idea what the field is for:

That's why I pinged you on the #4706 ticket, to see how we can fix this accessibility issue.

Ah, just saw this after answering that other ticket—that's why I was almost hoping a dropdown was needed here so we could avoid this issue (lol). Our design system actually has labels for above fields, but previous designers didn't implement those. Miina and I speculated about whether we should get change the design system to reflect that, but I've been concerned about this accessibility issue.
I think it's okay to put this field in with the in-field label for now, since it doesn't auto-populate, but soon I think I'll need to inventory where the accessibility unfriendly text fields are and come up with blanket solution.

I think it's okay to put this field in with the in-field label for now, since it doesn't auto-populate

It does auto-populate with the publisher name from the settings, which defaults to the site title. That is needed because the publisher name is a requirement.

OH, okay. Then this may be a time to just put a label on there that already exists in the Storybook. We could add it to others later, but it's necessary for this if the description is obscured. One less thing we have to worry about being accessible.


can't run yarn command

hi madchiken,
thank you for your kin support,
when I run yarn command, I get this error:
Done in 0.11s.
yarn run v1.22.10
error Couldn't find a package.json file in "/home/pi"
how can I pass that?

you are running yarn command from the home of your user but you need to cd into the project folder before


Failed to build fedora s390x podvm binaries

try to build fedora podvm binaries s390x with command
docker buildx build  -build-arg BUILDER_IMG=fedora-binaries-builder-s390x:latest    -build-arg AA_KBC=offline_fs_kbc --build-arg DEFAULT_AGENT_POLICY_FILE=podvm/files/etc/kata-opa/allow-all.rego    -o type=local,dest="./resources/binaries-tree"  -f podvm/Dockerfile.podvm_binaries.fedora .

But got error:
1214.9 error: failed to run custom build command for `kms v0.1.0 (/guest-components/confidential-data-hub/kms)`
1214.9
1214.9 Caused by:
1214.9   process didn't exit successfully: `/guest-components/target/release/build/kms-4159d8ccba785fac/build-script-build` (exit status: 1)
1214.9   — stdout
1214.9   cargo:rerun-if-changed=./src/plugins/aliyun/client/client_key_client/protobuf/dkms_api.proto
1214.9   cargo:rerun-if-changed=./src/plugins/aliyun/client/client_key_client/protobuf
1214.9
1214.9   — stderr
1214.9   Error: protoc failed: x86_64-binfmt-P: Could not open '/lib64/ld-linux-x86-64.so.2': No such file or directory
1214.9
1214.9 warning: build failed, waiting for other jobs to finish...
1227.4 make[1]: Leaving directory '/guest-components/confidential-data-hub'
1227.4 make[1]: *** [Makefile:88: build] Error 101
1227.4 make: *** [Makefile.inc:176: /src/cloud-api-adaptor/podvm/files/usr/local/bin/confidential-data-hub] Error 2
------
Dockerfile.podvm_binaries.fedora:38
--------------------
  36 |     WORKDIR /src/cloud-api-adaptor/podvm
  37 |
  38 | >>> RUN LIBC=gnu make binaries
  39 |
  40 |     FROM scratch
--------------------
ERROR: failed to solve: process "/bin/sh -c LIBC=gnu make binaries" did not complete successfully: exit code: 2

but it's on a s390x host, so it's strange.

[root@fcb3670afe17 protoc]# wget https://github.com/protocolbuffers/protobuf/releases/download/v${PROTOC_VERSION}/protoc-${PROTOC_VERSION}-linux-${PROTOC_ARCH}.zip
--2024-03-11 07:11:35--  https://github.com/protocolbuffers/protobuf/releases/download/v3.11.4/protoc-3.11.4-linux-s390x_64.zip
Resolving github.com (github.com)... <IP_ADDRESS>
Connecting to github.com (github.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 302 Found
Location: https://objects.githubusercontent.com/github-production-release-asset-2e65be/23357588/43046580-4f24-11ea-8f90-964cf89f2fa5?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAVCODYLSA53PQK4ZA%2F20240311%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20240311T071135Z&X-Amz-Expires=300&X-Amz-Signature=1db3835f17b1c75595e5ebf23d5e28adab87ff6e7e604395f5611929f27a3805&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=23357588&response-content-disposition=attachment%3B%20filename%3Dprotoc-3.11.4-linux-s390x_64.zip&response-content-type=application%2Foctet-stream [following]
--2024-03-11 07:11:35--  https://objects.githubusercontent.com/github-production-release-asset-2e65be/23357588/43046580-4f24-11ea-8f90-964cf89f2fa5?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAVCODYLSA53PQK4ZA%2F20240311%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20240311T071135Z&X-Amz-Expires=300&X-Amz-Signature=1db3835f17b1c75595e5ebf23d5e28adab87ff6e7e604395f5611929f27a3805&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=23357588&response-content-disposition=attachment%3B%20filename%3Dprotoc-3.11.4-linux-s390x_64.zip&response-content-type=application%2Foctet-stream
Resolving objects.githubusercontent.com (objects.githubusercontent.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to objects.githubusercontent.com (objects.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 1540350 (1.5M) [application/octet-stream]
Saving to: ‘protoc-3.11.4-linux-s390x_64.zip’

protoc-3.11.4-linux-s390x_64.zip           100%[=======================================================================================>]   1.47M  --.-KB/s    in 0.02s

2024-03-11 07:11:35 (74.8 MB/s) - ‘protoc-3.11.4-linux-s390x_64.zip’ saved [1540350/1540350]

[root@fcb3670afe17 protoc]# unzip protoc-3.11.4-linux-s390x_64.zip
Archive:  protoc-3.11.4-linux-s390x_64.zip
   creating: include/
   creating: include/google/
   creating: include/google/protobuf/
  inflating: include/google/protobuf/type.proto
  inflating: include/google/protobuf/duration.proto
  inflating: include/google/protobuf/empty.proto
  inflating: include/google/protobuf/wrappers.proto
  inflating: include/google/protobuf/field_mask.proto
   creating: include/google/protobuf/compiler/
  inflating: include/google/protobuf/compiler/plugin.proto
  inflating: include/google/protobuf/struct.proto
  inflating: include/google/protobuf/any.proto
  inflating: include/google/protobuf/descriptor.proto
  inflating: include/google/protobuf/source_context.proto
  inflating: include/google/protobuf/timestamp.proto
  inflating: include/google/protobuf/api.proto
   creating: bin/
  inflating: bin/protoc
  inflating: readme.txt
[root@fcb3670afe17 protoc]# file bin/protoc
bin/protoc: ELF 64-bit LSB executable, x86-64, version 1 (GNU/Linux), dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, for GNU/Linux 2.6.24, BuildID[sha1]=8e110954b7d51e826bed5d274159d37545961996, stripped
[root@fcb3670afe17 protoc]# bin/protoc
x86_64-binfmt-P: Could not open '/lib64/ld-linux-x86-64.so.2': No such file or directory

After go through https://github.com/protocolbuffers/protobuf/releases
https://github.com/protocolbuffers/protobuf/releases/download/v3.15.0/protoc-3.15.0-linux-s390x.zip
is the earliest version contains the correct s390x binary.


Stop playing music when timer reaches 0

Seems an app cannot directly control another app like Music or preform keyboard shortcuts but I think an app can trigger a shortcut, which could control Music with "Play/Pause" action.
Here is a related video:
https://developer.apple.com/videos/play/wwdc2021/10283/


Add method for deleting user's saved albums

Added a method in client.py and created a script in the examples directory that takes in album uris to delete.

Thank you!


Removing psycopg2 from default imports

Coverage remained the same at 100.0% when pulling 40144ba555b14c75645032820baef9e5bc3b0c5c on sqlite3-only-fix into 70fc26050ec9777e01260e4847e0b319085bcf43 on master.

Coverage remained the same at 100.0% when pulling 40144ba555b14c75645032820baef9e5bc3b0c5c on sqlite3-only-fix into 70fc26050ec9777e01260e4847e0b319085bcf43 on master.

Coverage remained the same at 100.0% when pulling 5ea1a327753b2e8b44f658ec89f526bc0a64b11f on sqlite3-only-fix into 70fc26050ec9777e01260e4847e0b319085bcf43 on master.

Coverage remained the same at 100.0% when pulling 0a205d13ee1d9ce6eab5f84e3442b34ec1263718 on sqlite3-only-fix into 70fc26050ec9777e01260e4847e0b319085bcf43 on master.


Add missing discord link in "the_bevy_organization.md"

Objective

Add change that was suggested in https://github.com/bevyengine/bevy/pull/7185#pullrequestreview-1248746032 but missed

Solution

Add the change

bors r+


New db structure and trigger reorg from orchestrator

Motivation
Removing slot key and introducing stepId concept because slot can be missed and it does not consecutive. As well as orchestrator should support multi shard verification, so new db structure is needed.

tests are failing


x/pkgsite: short summary of package on search page unreadable

For certain packages, the high-level summary of the package is unreadable in the search page. For example, searching for "fmt" comes up with the standard library package (which is easy to read) and another package "github.com/astaxie/beego/orm":

It's actually not totally clear to me what is being printed in the search page, since the README of that package appears to be different for me when I actually click into it. It also has an image that doesn't render.
I don't actually know if there is anything you can/want to do about this, since it's likely working as intended. But I wanted to bring it up as a UX issue.
@julieqiu

The issue here is that the package synopsis is missing a period at the end of the first sentence:
https://github.com/astaxie/beego/blob/v1.12.3/orm/orm.go#L17
I think it makes sense for us to truncate the package synopsis after a certain length on the search results page. For example:

/cc @georgehu


⚠ Minecraft has degraded performance

In fd21206, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1201 ms

Resolved: Minecraft performance has improved in f64abeb.


Leases no longer revoked after stopping SecretLeaseContainer

See https://github.com/spring-projects/spring-vault/issues/855

Hello @mp911de! Do you by any chance have an approximate timeline for releasing this fix? I see that the 4.1.2 milestone doesn't have any open issues.


Support mac OS binary release for CI

Description
Summary
This PR adds macOS platform support to our CI. It modifies key files for smooth macOS builds and releases.
Changes Made

.github/workflows/publish.yml:

Added macOS entries to the os matrix.
Adjusted JDK download, caching, and Gradlew steps for macOS.

buildSrc/src/main/kotlin/buildlogic/CiUtils.kt: Updated functions to handle macOS architecture for file naming.
desktop/app/build.gradle.kts: Added code to work with macOS architecture in packaging.

Example

Test locally

But the window button seems conflict.
I'll solve this later.

Thanks for you interest to contributing to this project but unfortunately I can't merge it for now.
i intentionally not added mac target, as I mentioned here I don't have a Mac device to test and debug the app. if I add or modify the structure of the app it may breaks in the mac user systems.
and it is not that I want for now!

Description
Summary
This PR adds macOS platform support to our CI. It modifies key files for smooth macOS builds and releases.
Changes Made

.github/workflows/publish.yml:

Added macOS entries to the os matrix.
Adjusted JDK download, caching, and Gradlew steps for macOS.

buildSrc/src/main/kotlin/buildlogic/CiUtils.kt: Updated functions to handle macOS architecture for file naming.

desktop/app/build.gradle.kts: Added code to work with macOS architecture in packaging.

Example

### Test locally

兄弟，你测试没问题可以fork一份编译好的发出来。不是正式版本，我们也能先体验当测试版本开心下。

Description
Summary
This PR adds macOS platform support to our CI. It modifies key files for smooth macOS builds and releases.
Changes Made

.github/workflows/publish.yml:

Added macOS entries to the os matrix.
Adjusted JDK download, caching, and Gradlew steps for macOS.

buildSrc/src/main/kotlin/buildlogic/CiUtils.kt: Updated functions to handle macOS architecture for file naming.

desktop/app/build.gradle.kts: Added code to work with macOS architecture in packaging.

Example

### Test locally

兄弟，你测试没问题可以fork一份编译好的发出来。不是正式版本，我们也能先体验当测试版本开心下。

你可以在这里尝鲜MacOS版本，应该也都是能用的地步。
窗口什么还没修，不过只是尝鲜哈。
https://github.com/peanut996/ab-download-manager/releases/tag/v1.4.2


Update zio, zio-test, zio-test-sbt to 2.0.0

Updates

dev.zio:zio
dev.zio:zio-test
dev.zio:zio-test-sbt

from 2.0.0-RC6 to 2.0.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, commit-count:1

Superseded by #35.


Rewrite Fair Squares pallet around Litentry NFT pallet, instead of Anmol NFT pallet

On old dependency since the hackathon, that is outdated and not maintained, needs to be removed and dependencies in the code that touch upon the fractionalisation.

Updated the Diagram


Image in game list still fetched from internet.

Move the image to assets

Added internet permission to Android instead


Perf issues when there are a lot data showing in the window.

I have a question regarding how to disable specify window style. Just let it as system default style.
I have a window with long Grid to load the a big data(about 500 lines). Custom style window will have performance issue. The default style window doesn't have. So I wanna disable the custom style only for this window. is there any approach to do this?

When you include Adonis UI in your App.xaml via
<ResourceDictionary Source="pack://application:,,,/AdonisUI.ClassicTheme;component/Resources.xaml"/>

you include all styles that the library provides.
If you don't want to include some of its styles you need to remove this line and replace it by a custom resource dictionary that you provide yourself.
To do so, you can copy the Resources.xaml of Adonis UI and remove the styles you don't like. Then include your file instead.
This has a disadvantage though. When you update Adonis UI in the future you won't receive new styles and your file may break in case some styles are removed or renamed. So you have to update your file on your own.
Regarding the performance, this seems to be related to #150. I will see what I can do here.

Thank you so much~ @benruehl


Adding support for null and blank values to the AttributeLengthValidator

I need a length validator that allows empty or blank values.  PR is done, opening this to follow contribution policy.

Can you please mention this issue number in the PR commit?

Issue https://github.com/javalite/activejdbc/pull/686


Implement PLDM Tool with BIOS command support.

Implementing the following in the PLDM tool with BIOS command support

GetBIOSTable with tableType : attributeTable
GetBIOSTable with tableType : stringTable
GetDateTime

Implemented & merged


server crashes when a player leaves the game

Mod Version:
DynamicSurroundings-core-1.12.2-<IP_ADDRESS>
Forge Version:
<IP_ADDRESS>38
Link to crash log (if applicable):
https://drive.google.com/file/d/1t4jwBp4aT9Cg4RbRyrT4dAHCZd0Vz-EF/view?usp=sharing
Description:
I would be grateful if you help to deal with the problem :)

Does it always happen, or sporadically?

Does it always happen, or sporadically?

It always happens.

EDIT: Also can you post a link to your server log? May need to zip it.

2019-10-13_22.48.46-log.zip

EDIT2: And was this a logoff or disconnect? Routine in question is due to disconnect and I need to confirm.

disconnect

Has this server ever worked, or is it something you are working on?  There are other errors in the log from plugins reporting errors accessing fields and that looks suspicious.  One thing I saw was information related to OptiFine.  Traditionally that would not be installed on a server.  Not sure if that changed recently, but if you have that on the server I would remove that first and see what happens.

Has this server ever worked, or is it something you are working on? There are other errors in the log from plugins reporting errors accessing fields and that looks suspicious. One thing I saw was information related to OptiFine. Traditionally that would not be installed on a server. Not sure if that changed recently, but if you have that on the server I would remove that first and see what happens.

I deleted all the plugins and mods which  don`t create a config file. But now I have a lot more errors and crash when disconnecting remained :(
2019-10-14-1.log.zip

Has this server ever worked, or is it something you are working on?

The server worked before installing the mod. After the first use, I realized that this mod is the best of its kind and should work on my server

This is an interesting issue.  I am going to make some changes - not entirely sure its going to fix this issue.  Also, not sure if you are aware - you can disable the mod on the server, and the client will still be able to make use of it.  There are something you will not get (like speech bubbles) but all the sounds and effects will happen.
I am not sure when a build will be available for you to try.  I have some testing I need to complete before I publish.

@OreCruncher , In any case, thank you so much for your help in this problem. I hope in future versions you fix it. Good luck with your work :)

Pushed v<IP_ADDRESS> to CurseForge.  Let me know if it fixes the issue.  If it doesn't please reopen the issue.

i think it did...


Mailing unit sprite needs fixn

Description

when the mailing unit is flushed it changes from the blue color to the disposal color.
Screenshots

This should be closed since im p sure it got fixed with the recent resprite

turns out it wasnt resolved


Multicurrency Support

Enables support for custom themes and functionalities for every Nano fork (e.g. Banano).

What is missing for multicurrency? How can we help?

@sebrock I wanted to have the linting before the multicurrency support so the coding style stays consistent in every branch from now on. I'm currently laying out the directories and try out the image switching and theming.

@sebrock Why was code review requested from me? The PR is both flagged as draft and has several conflicts

It wasn´t draft when I requested.
@BitDesert how do you want to proceed with this one?

@BitDesert I would love to continue (better: finally start) on the Banano version.
This branch has somewhat dried out.
Please let us know what your plans are.

Closing in favor of #332


feat(app, app-shell-odd): add function to read csv file from usb for rtp phase2

Overview
add a function that loads csv files from an usb-drive, update protocol card & pinned protocol, and add ChooseCsvFile component (display csv files on USB).
close AUTH-460, AUTH-468, and partially AUTH-462 and AUTH-470
This PR doesn't update protocol details screen and Parametes screen. They will be implemented in a following pr since this PR's main part is displaying csv files on USB.
The following yellow background shows up when turning on the feature flag and a protocol name includes RTP.
The rendering condition part will be update in a following pr.

Confirm selection part will be implemented and a function to control radio buttons (AUTH-458)

Test Plan

push this branch to a Flex
select a RTP protocol that has yellow background in Protocol Dashboard screen
do set up

Changelog

Review requests

Risk assessment

[ ] solve conflicts

this will be taken over by #15551


WIP: Refactor Dexie.Syncable

This is an effort to refactor Dexie.Syncable and to split it into multiple smaller modules which can easier be unit tested.
There is still lots to do. Some of the things to do are:

[ ] Add an eslint configuration and eslint script
[ ] Add karma configuration for travis
[ ] Add karma configuration for release
[ ] Try to find a better way to handle globals than the currently added globals module

Any feedback is appreciated. See also https://github.com/dfahlander/Dexie.js/issues/409

👍
Nice work.
Maybe globals module not needed. Instead let those modules that needs to access or modify properties of db,  export a function that takes db as an argument.

I'm not sure what you mean exactly. Adjust the number of parameters of the exported functions or export functions receiving the globals and return a closure which will be the actual function executing the logic? I like the closure idea. We don't need to adjust the params of each function end we won't need a globals module.

I was also thinking of the closure idea 👍

In order to get travis to run Dexie.Syncable tests on push we would have to add it to .travis.yml and also add the build call for Dexie.Syncable and probably also Dexie.Observable there. I'm not sure if the build tools will allow that in their current state (I don't know if the paths will match since we implicitly change the PWD).
The question for me is: Should we try to add Dexie.Syncable to .travis.yml or would it be better/easier to have Dexie.Syncable and possibly Dexie.Observable in their own repository? Those already have an own package.json and in some sense are treated as an own project.

Good question. There's one benefit with keeping it in the same repo and require Travis tests for the addons to succeed - Changes to Dexie that would break the addons would be detected.

You are right about that. I don't know what will be more work. Possibly adjusting the build process or run tests (probably manually) on the syncable and observable repos each time Dexie is released or at least every major release. In the long run probably the latter. I like separate repositories in general but I get your point. In any case I wouldn't split the repos before Dexie v2.0.0 so I think I will try to add it to travis after I'm done refactoring.

Sounds great. The release script (tools/release.sh) runs the unit tests on browserstack with a slightly larger set of target browsers than the one executed from Travis. It could also execute the addons tests. All releases including minor ones needs to pass those tests.

@dfahlander as you can see I made a great deal of changes and I think Dexie.Syncable is now more readable and also a bit more stable. I think this is enough refactoring for now. There are some more functions that could be put in an own module but those are very interconnected and I'm afraid that putting those in their own module will break more things than it will fix.
It would be great if you could review the changes when you get some time and after that (and after the tasks in the first comment are done) we could merge into master if you think that everything looks good.

🥇 Impressive!
All tests runs perfectly and you've created lots of them. This was a huge refactoring. Simply amazing.
Seems all tasks are complete. Can I merge it in now?

Thanks. Regarding the merge: tests-syncable-dexie.html throws 13 exceptions in the browser but those are also there in the beta 7 tag so I don't know if those are relevant and what is missing is the addition of travis.sh to .travis.yml but maybe you can do that as you can probably better monitor the output of the script on travis. Other than that we are good to merge.


chore: update lint

What I did
Upgrade lint and pre-commit since things were failing in the tests.

fixes: #
How I did it
How to verify it
Checklist

[ ] Passes all linting checks (pre-commit and CI jobs)
[ ] New test cases have been added and are passing
[ ] Documentation has been updated
[ ] PR title follows Conventional Commit standard (will be automatically included in the changelog)

mdformat-pyproject has been updated (0.0.2); you can no update mdformat if you want.

@dtdang Don't forget to also run mdformat

The latest releases of taskiq may have broken silverback.


A hook for Redirects

Full disclaimer: I just started learing React and the React Router release candidate for the first time so forgive me if I'm missing something obvious.
I am wondering if there is a reason why there aren't any hooks for Redirects? I understand that onEnter and onLeave feel inaccurate in this situation but how about something like onRedirect?
The use case I currently have is logging a user out. I have a regular "Log Out" Link that links to /logout. I also have a redirect setup that redirects /logout to /. It would be very convenient to hook in here and kill the "session" for my current user.
<Redirect from="/logout" to="/" onRedirect={ logout }>

Without this hook, it seems like the next best thing is to have a LogOut component which just uses the componentDidMount hook to kill the "session" and then manually redirect via this.props.history.pushState. While this isn't terrible, it seems like I'm just recreating Redirect with support for hook.
Admittedly, this hook seems useful in this situation but I'm not entirely sure how useful it will be in general.

What you want to do is have the route like:
<Route path='logout' onEnter={this.handleOnLogout} />

and then in the handleOnLogout() function you can manage clearing your session and after you are done you redirect to /, so your handleOnLogout is going to look like:
handleOnLogout(nextState, replaceState) {
  console.log('Clear your session here...');
  replaceState({ nextPathname: nextState.location.pathname }, '/');
}

so I don't really see the point on creating something like onRedirect

Ohh nice, that's better. Sorry for the noise.
Thanks!


How to build liblept1760.dll instead of  leptonica-1.76.0.dll

I have built tesseract and libleptonica according compiling tesseract and libleptonica (using vcpkg),
and I got leptonica-1.76.0.dll and tesseract40.dll.
How can I get liblept1760.dll and libtesseract400.dll for tesseract4?

I use the instructions for creating SLNs and then manually jigger them to adjust the DLL names.  I wish the C# layer used the native names.

The c# layer uses the default names generated by the respective cmake projects from their official repositories (I haven't changed them) and I would like to keep it this way since it makes building them easier.
You can of course change them by editing CMakeLists.txt file in ~\leptonica\src directory (look for OUTPUT_NAME).

Note I've updated the build instructions for Tesseract 4.1 (https://github.com/charlesw/tesseract/wiki/Compiling-Tesseract-and-Libleptonica-(using-vcpkg))
As your can see we make no changes to their files.

Opps sorry just realized that we weren't using these names, please ignore my earlier comments  (since deleted to avoid further confusion). I'll rebuild the native libraries and update the src in the repository.

The wrapper now uses the native names generated when using the cmake projects as described in https://github.com/charlesw/tesseract/wiki/Compiling-Tesseract-and-Libleptonica-(using-vcpkg).
Closing task but please let me know if you need any further assistance.


Added message functionality in command line

General Checkups

[x] Have you checked that there aren't other open pull requests for the same issue/update/change?
[ ] If your code includes new features and not just bug fixes/copy editing: Did you include new tests?

Description

Related Issue

Example

Hi @madprime , @gedankenstuecke , @mcescalante , I have added message functionality to be directly used through command line. Please suggest some way to solve the codeclimate issue. Thanks :)

Hi @gedankenstuecke , I have made the suggested changes. Please review :)


Fix for #280

Co-authored with @cescoffier thanks for the tip!

@cescoffier no stuck CI this time 😉


Error in host.json provided for logging

The host.json provided for setting up logging shows errors in Visual Studio. It's on the samplingExcludedTypes line and the error is: Object shouldn't contain additional properties.
Googling the error brought me to: https://github.com/MicrosoftDocs/azure-docs/issues/47219
So, consider this update:
{
"version": "2.0",
"logging": {
"applicationInsights": {
"samplingSettings": {
"excludedTypes": "Request",
"isEnabled": true
}
},
"logLevel": {
"MyNamespace.HttpTrigger": "Information"
}
}
}

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

ID: 5786402e-9d88-5de0-607a-1c2a143af006
Version Independent ID: 15368d1c-b22e-7eb2-413f-cc0d41d2b909
Content: Use dependency injection in .NET Azure Functions
Content Source: articles/azure-functions/functions-dotnet-dependency-injection.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@kinetiq Thanks for catching that! We have worked up a PR to address this, which once merged should reflect in a couple of hours.


Exception during phase body

I get this after using /kill breantique
I am currently running...

SpongeVanilla version:

1.12.1-7.0.0-beta 315

Java version:

8

Operating System:

[x] Linux (Ubuntu, Debian, Fedora, etc)

@Faithcaio looks like some inventory parents are null for some reason or another.

@EmanYm please reupload the log to https://gist.github.com hastebin already deleted the log


SF county only example data creation scripts

see issue #71

Coverage remained the same at 97.775% when pulling 6b41ce78a87a5e789990291443ef0882f356f459 on sf-example into ea34a16433e3a552bc1ec77af82ac3fbadb82cbc on master.


[Entering Trick Answers] After an incorrect/trick answer has been entered, the "What do you think the correct answer is?" field should be cleared

July 14 build:
Observed:
Currently, after an incorrect/trick answer has been entered (e.g., "123" in the attached screenshot), "123" still appears in the "What do you think the correct answer is?" field.
Expected:
After an incorrect/trick answer has been entered (e.g., "123" in the attached screenshot), the "What do you think the correct answer is?" field should be cleared.
Note: After the wait time for hints has passed (and students are then able to view the hints and the correct answer in the Hints card), the "What do you think the correct answer is?" field will be different -- there will instead be a message that reminds students what the correct answer is.

Yong Lin and I verified this earlier today -- thanks!


Add "Report an Issue" item in menu

I've had to bother our venerable developer @danielbuechele on Twitter to find this repo, so I'd suggest a "Report an Issue" item in the Goofy menu that leads here or possibly to a Wiki page describing how to do that and how not to ;o) I'm up for writing a draft for that Wikipage if needed.

That would be a great idea :)
On Mon, Aug 17, 2015 at 7:23 PM Jonathan Allard<EMAIL_ADDRESS>wrote:

I've had to bother
https://twitter.com/jonathanallard/status/633087631375466496 our
venerable developer @danielbuechele https://github.com/danielbuechele
on Twitter to find this repo, so I'd suggest a "Report an Issue" item in
the Goofy menu that leads here or possibly to a Wiki page describing
—
Reply to this email directly or view it on GitHub
https://github.com/danielbuechele/goofy/issues/136.

Created a pull request, https://github.com/danielbuechele/goofy/pull/158


Fixed the DQN example and added Base.length to discrete

The DQN example for the CartPole environment and the environment itself has been updated to stop converting the actions (Discrete(2)) to (-1, 1). Function inv_action has been discarded. ~Please make corresponding changes in the model-zoo as well.~ Changes proposed to the model-zoo version with this PR.

@tejank10 Made the requested changes. Please review.

Check

@tejank10 I looked into the internal error I got with the DiffRL.jl example (Present in this gist) So, it all comes down to line 87 of the CartPole environment which is...
done =  !(all(vcat(-env.x_threshold .≤ x_ .≤ env.x_threshold, -env.θ_threshold_radians .≤ θ_ .≤ env.θ_threshold_radians)))

Any clue as to why this doesn't sit right with Zygote?


Slash Commands Embed Empty

Describe the bug
When using ComponentMessageBuilder inside of a SlashCommandInteractionEvent it ignores all data from an embed. If you have no content set you will recieve this error "[2021-06-07 05:33:25.645669] [SEVERE] [Client] ERROR: [Code: 400] [Message: {"message": "Cannot send an empty message", "code": 50006}]". If you do have Content set it will send the content and ignore the embed without error message.
To Reproduce
Put the files accordingly in a new project. Make sure to change the file extensions to ".dart" as GitHub won't accept the file extension, I changed them to text files. When you get in the files, you will need to locate the app.dart file. Add your token at line 9. Add your guild id that you will reproduce the error at line 14.
Expected behavior
When sending /info into the server the bot should respond with an Embed that has the title set to "Which would you like to choose".
Desktop:

OS: [Windows 10 Pro 20H2]
Dart version [2.13.1]
Nyxx version [2.0.0-rc.5]

This file is in the bin folder
app.txt
These files are in the lib folder
docs.txt
info.txt
info-slash.txt
utils.txt
PUBSEC
pubspec.txt

Thanks, will look into more later for now just stick to content. Will post updates here 🙂 It's sending:
{
  // .....
  "embed": {}
}

I believe it should be sending:
{
  // .....
  "embeds": [
    {}
  ]
}

Because it's a webhook.

Should be fixed, @aarongarnerm please test then close if fixed.

Issue fixed and original author doesnt provided further feedback. Closing


QBitTorrent not showing network disk storage

In the side panel of the vue UI it used to show the remaining storage of the network disk but after a reinstall it shows only the remaining disk spaces of the internal SD card even though I have specified the IP address of my network disk.

I'm not sure why, It is a detail that I had never seen before... Did you reference the network drive as download path? I suppose it only shows the space of the target disk... Or don't you have a backup that could be restored of this addon?

I've just tested and it shows my network disk usage. Please double check your referenced download directory.

Is this config correct?

ssl: true
certfile: fullchain.pem
keyfile: privkey.pem
whitelist: localhost,<IP_ADDRESS>,<IP_ADDRESS>/16,<IP_ADDRESS>/16
customUI: vuetorrent
SavePath: //<IP_ADDRESS>/HDD
Username: admin
DNS_server: <IP_ADDRESS>,<IP_ADDRESS>
PUID: '0'
PGID: '0'
TZ: ''
openvpn_password: ''
networkdisks: //<IP_ADDRESS>/HDD
cifsusername: homeassistant
cifspassword: password

I also have an unrelated question; how can I change the default password for the webUI?

I fixxed it. Download location should be /mnt/HDD.

Exact! Does it work now? Enjoy!