Android back button doesn't work correctly in CupertinoTabBar apps

The Android back button does not work correctly, if a widget is pushed from within a CupertinoTabBar.

Start the app on an Android Device
Tap the button "Tap me"
Try to go back with the Android back button.

Expected: The app shows the previous widget with the "Tap me" button again
Actual: The app is closed
import 'package:flutter/cupertino.dart';

void main() => runApp(MaterialApp(home: CupertinoNavigationDemo()));

class CupertinoNavigationDemo extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return CupertinoTabScaffold(
      tabBar: CupertinoTabBar(
        items: const <BottomNavigationBarItem>[
          BottomNavigationBarItem(
            icon: Icon(CupertinoIcons.home),
            title: Text('Home'),
          ),
          BottomNavigationBarItem(
            icon: Icon(CupertinoIcons.home),
            title: Text('Home'),
          )
        ],
      ),
      tabBuilder: (BuildContext context, int index) {
        return CupertinoTabView(
          builder: (BuildContext context) {
            return CupertinoDemoTab();
          },
        );
      },
    );
  }
}

class CupertinoDemoTab extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Container(
      color: Colors.white,
      child: FlatButton(
        onPressed: () {
          Navigator.of(context).push(CupertinoPageRoute<void>(
            builder: (BuildContext context) => DetailPage(),
          ));
        },
        child: Text("Tap me"),
      ),
    );
  }
}

class DetailPage extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return CupertinoPageScaffold(
        navigationBar: const CupertinoNavigationBar(),
        child: Container(
          color: Colors.green,
        ));
  }
}

Looking for it too

Deduping into #14083

#14083

#14083 is different, without CupertinoTabBar

There is a solution for that.
Like said many times before the problem is that you have multiple navigators coming with your CupertinoTabBar.
So what you want is a gobal navigator key for each tab. Chances are you probably need that anyway at some point in your app. So in my App I've created as much global keys as I have tabs in my tabbar.
  final GlobalKey<NavigatorState> firstTabNavKey = GlobalKey<NavigatorState>();
  final GlobalKey<NavigatorState> secondTabNavKey = GlobalKey<NavigatorState>();
  ...

then set the according navigator key to the CupertinoTabView():
CupertinoTabView(
   navigatorKey: firstTabNavKey,
   builder: (BuildContext context) {
      return MyWidget();
 );

So far so good. Now lets handle the android back button correctly.
Wrap your CupertinoTabScaffold in a WillPopScope() widget just like that:
WillPopScope(
  onWillPop: () async {
    return !await currentNavigatorKey().currentState.maybePop();
  },
  child: CupertinoTabScaffold(
    ...
  )
)

a little helper method ensures that it'll always call maybePop() on the current visible navigator by switching over the current tab index.
GlobalKey<NavigatorState> currentNavigatorKey() {
    switch (_tabController.index) {
      case 0:
        return firstTabNavKey;
        break;
      case 1:
        return secondTabNavKey;
        break;
      case 2:
        return thirdTabNavKey;
        break;
      case 3:
        return fourthTabNavKey;
        break;
      case 4:
        return fifthTabNavKey;
        break;
    }

    return null;
  }

Thats it. Works like charm 😊

Thank you @robin7331 , this fixed the problem for me. It's noteworthy, that _tabController is a member variable of the tyoe CupertinoTabController, which has to be passed to the CupertinoTabScaffold for the controller property.
However, this introduced another problem for me, whenever I change code triggering a hot reload, my App completely crashes with the following error screen. Needless to say, this hinders development immensely, because we have to completely reload the app everytime we make a simple change. Did anybody experience the same issue?

I ran into the same issue. Fixed by changing from StatelessWidget to StatefulWidget.

Thank you @robin7331 , this fixed the problem for me. It's noteworthy, that _tabController is a member variable of the tyoe CupertinoTabController, which has to be passed to the CupertinoTabScaffold for the controller property.
However, this introduced another problem for me, whenever I change code triggering a hot reload, my App completely crashes with the following error screen. Needless to say, this hinders development immensely, because we have to completely reload the app everytime we make a simple change. Did anybody experience the same issue?

You are the absolute MVP @hannukuk , thank you very much! A super simple fix that also explains why it worked for a lot of people, they were probably using a stateful widget in the first place.

@robin7331
I suggest using List<GlobalKey<NavigatorState>> instead of creating individual keys.
final tabNavKeys = List<GlobalKey<NavigatorState>>[
  GlobalKey<NavigatorState>();
  GlobalKey<NavigatorState>();
  ....
];

Then in your CupertinoTabView:
CupertinoTabView(
   navigatorKey: tabNavKeys[0],
   builder: (BuildContext context) {
      return MyWidget();
 );

And finally, in WillPopScope:
WillPopScope(
  onWillPop: () async {
    return !await tabNavKeys[_tabController.index].currentState.maybePop();
  },
  child: CupertinoTabScaffold(
    ...
  )
)

You absolutely don't need this monstrous currentNavigatorKey() method.

thanks @robin7331 , @hannukuk  , @j-holub , @REKURDennis ...after a lot of searching I find this page, It surely works like a charm. Thanks a ton again .

Even after implementing this, my Android phone and simulator is crashing, or leaves the flutter app, will using the backbutton. Is there more i must do?

Even after implementing this, my Android phone and simulator is crashing, or leaves the flutter app, will using the backbutton. Is there more i must do?

@ASemeniuk

final tabNavKeys = List<GlobalKey>[
GlobalKey();
GlobalKey();
....
];
You absolutely don't need this monstrous currentNavigatorKey() method.

Adding to the succinctness of this, you can go a bit further and use List.generate() instead. For ex:
final List<GlobalKey<NavigatorState>> tabNavKeys = List.generate(num, (_) => GlobalKey<NavigatorState>());

Even after implementing this, my Android phone and simulator is crashing, or leaves the flutter app, will using the backbutton. Is there more i must do?
Away from the CupertinoTabScaffold the back button works fine:
On the CupertinoPageScaffold which is open, i navigate to a new screen via
onTap: () {
Navigator.of(context, rootNavigator: true).push(
       CupertinoPageRoute(
              builder: (BuildContext context) => NewPage()),
}

and on the NewPage()
i have a button with:
 Navigator.of(context).pop();

And on that screen (NewPage()) the back button is working fine.

I am facing the same issue,did you find any solution ?


AES secrets encryption

PR enables AES secrets encryption.
It consist of #1475, #1544. Part of #1541 and #1814.

Made decorator over SecretService instead of SecretStore
Temporary removed Tink encryption
Temporary removed migration
Removed unnecessary code
Fixed documentation

Config:
  WOODPECKER_ENCRYPTION_KEY: GjVHT007c4x3N+YPbsZld+hifba1enXkOzIb/0h6oW8=

Test pipeline:
steps:
  test:
    image: alpine
    commands:
      - echo "$SOME_SECRET" | sed -e 's/\(.\)/\1\ /g'
    secrets:
      - some_secret

New secret creation

Logs:
{"level":"debug","id":0,"name":"some_secret","time":"2023-08-20T19:37:42Z","caller":"/woodpecker/server/plugins/secrets/encrypted.go:219","message":"encryption"}
{"level":"debug","id":9,"name":"some_secret","time":"2023-08-20T19:37:42Z","caller":"/woodpecker/server/plugins/secrets/encrypted.go:230","message":"decryption"}

DB:
woodpecker=# select secret_id, secret_name, secret_value from secrets;
 secret_id | secret_name |                          secret_value
-----------+-------------+----------------------------------------------------------------
         9 | some_secret | PUattjAz6EOP28sbJOEaDSZyXRDrPxGQv9EyQHQPimrWLQELr59WYp83DUNQ6w
(1 row)

Test run:
+ echo "$SOME_SECRET" | sed -e 's/\(.\)/\1\n/g'
s u p e r - s e c r e t - v a l u e

Changed secret value

DB:
woodpecker=# select secret_id, secret_name, secret_value from secrets;
 secret_id | secret_name |                           secret_value
-----------+-------------+------------------------------------------------------------------
         9 | some_secret | ttXvtoOY3ixRMeU/niFNG/O6jcO2VCuCK1MZjFr68OzlytBD5d6q8ylUsvRfXs/M
(1 row)

Test run:
+ echo "$SOME_SECRET" | sed -e 's/\(.\)/\1\ /g'
c h a n g e d - s e c r e t - v a l u e

well before we merge we have to make sure migration of non encrypted and back and mix in-between works properly ... we had to deactivate it by code last time :/

Tests.
 secret_id | secret_name | secret_value
-----------+-------------+--------------
         1 | s-name      | s-value

WOODPECKER_ENCRYPTION_KEY: GjVHT007c4x3N+YPbsZld+hifba1enXkOzIb/0h6oW8=
WOODPECKER_SECRETS_ENCRYPTION_MODE: EnabledAndEncrypt

{"time":"2023-09-24T10:47:26Z","caller":"/woodpecker/cmd/common/logger.go:99","message":"LogLevel = debug"}
{"level":"debug","mode":"EnabledAndEncrypt","time":"2023-09-24T10:47:28Z","caller":"/woodpecker/server/plugins/secrets/setup.go:39","message":"setting up secrets service"}
{"level":"fatal","error":"secret and encryption services cannot be nil","time":"2023-09-24T10:47:28Z","caller":"/woodpecker/cmd/server/setup.go:127","message":"failed to set up secrets service"}

WOODPECKER_ENCRYPTION_AES_KEY: GjVHT007c4x3N+YPbsZld+hifba1enXkOzIb/0h6oW8=
WOODPECKER_SECRETS_ENCRYPTION_MODE: EnabledAndEncrypt

{"time":"2023-09-24T10:49:20Z","caller":"/woodpecker/cmd/common/logger.go:99","message":"LogLevel = debug"}
{"level":"debug","time":"2023-09-24T10:49:21Z","caller":"/woodpecker/server/plugins/encryption/aes.go:41","message":"initializing AES encryption service"}
{"level":"debug","time":"2023-09-24T10:49:21Z","caller":"/woodpecker/server/plugins/encryption/aes.go:68","message":"AES encryption service has been initialized"}
{"level":"debug","mode":"EnabledAndEncrypt","time":"2023-09-24T10:49:21Z","caller":"/woodpecker/server/plugins/secrets/setup.go:39","message":"setting up secrets service"}
{"level":"debug","time":"2023-09-24T10:49:21Z","caller":"/woodpecker/server/plugins/secrets/setup.go:74","message":"encrypting secrets"}
{"level":"debug","id":1,"name":"s-name","time":"2023-09-24T10:49:21Z","caller":"/woodpecker/server/plugins/secrets/encrypted.go:48","message":"encryption"}
[GIN-debug] [WARNING] Running in "debug" mode. Switch to "release" mode in production.

 secret_id | secret_name |                     secret_value
-----------+-------------+------------------------------------------------------
         1 | s-name      | _aes_D9Nimg9+pyyQQaaLcuGOm2GbkBMQcIvp0qAY7NE9oQcctGM

 secret_id | secret_name |                     secret_value
-----------+-------------+-------------------------------------------------------
         1 | s-name      | _aes_D9Nimg9+pyyQQaaLcuGOm2GbkBMQcIvp0qAY7NE9oQcctGM
         2 | second      | _aes_gD0ybRavPstWzDUH0guQz3myx53Z0r6dVBZgGmPs5W36fT7T

WOODPECKER_ENCRYPTION_AES_KEY: GjVHT007c4x3N+YPbsZld+hifba1enXkOzIb/0h6oW8=
WOODPECKER_SECRETS_ENCRYPTION_MODE: DisabledAndDecrypt

{"time":"2023-09-24T10:52:07Z","caller":"/woodpecker/cmd/common/logger.go:99","message":"LogLevel = debug"}
{"level":"debug","time":"2023-09-24T10:52:08Z","caller":"/woodpecker/server/plugins/encryption/aes.go:41","message":"initializing AES encryption service"}
{"level":"debug","time":"2023-09-24T10:52:09Z","caller":"/woodpecker/server/plugins/encryption/aes.go:68","message":"AES encryption service has been initialized"}
{"level":"debug","mode":"DisabledAndDecrypt","time":"2023-09-24T10:52:09Z","caller":"/woodpecker/server/plugins/secrets/setup.go:39","message":"setting up secrets service"}
{"level":"debug","time":"2023-09-24T10:52:09Z","caller":"/woodpecker/server/plugins/secrets/setup.go:94","message":"decrypting secrets"}
{"level":"debug","id":1,"name":"s-name","time":"2023-09-24T10:52:09Z","caller":"/woodpecker/server/plugins/secrets/encrypted.go:64","message":"decryption"}
{"level":"debug","id":2,"name":"second","time":"2023-09-24T10:52:09Z","caller":"/woodpecker/server/plugins/secrets/encrypted.go:64","message":"decryption"}
[GIN-debug] [WARNING] Running in "debug" mode. Switch to "release" mode in production.

 secret_id | secret_name | secret_value
-----------+-------------+--------------
         1 | s-name      | s-value
         2 | second      | second-v

Considering that the Encryption Documentation was complete remove do we have to assume that this feature is killed? We really like to see encryption. Is there something we can do to can help?

In regard to this PR, I would rework "migration" from env vars hack to CLI command and/or UI button <- requires API.
For plain->encrypted direction the flow can be

Load application with plain secrets service.
Issue encrypt command or click button.
Lock secrets functionality.
Load encrypted service/decorator.
Encrypt all secrets.
Unlock secrets service.

But there were thoughts about external secrets providers and addons story => this might not suit "core".

Didn't know, that deleting a branch closes PR... Anyway, code is here, someone can continue the work.


Get Set for our Back End Development Projects - create simpler version for API Projects

I think Clementine is a wonderful starting place for the Full Stack Apps but many users are finding it to be unnecessary and unhelpful for the API projects.  They are getting tripped up on things like authentication and MVC patterns, which aren't necessary for the API Projects.
Therefore, I'd like to suggest that the Clementine generator instructions are moved to a new step preceding the Full Stack projects.  For the API Projects, either create a different boilerplate or provide none at all.

Maybe something like this

Create a directory named for whatever your project is (e.g. time-server-api)
Run npm init in the console and follow the prompts to set up your package.json file (most things you can leave as default)
Run npm install express --save to insall express.js and save it as a dependency for your project.
Add a .gitignore file to your directory which ignores the node_modules folder
Run git init to initialize this as a git repository
Create a file in your directory called server.js
For the API projects you can write all your code in this file.  For a starting place I recommend this as a simple example of node/express set up: http://expressjs.com/en/starter/hello-world.html

I was very confused on how to start after the Clementine tutorial, so jondcoleman's suggestion helped out a lot, so I think it would be useful if you included something like this before Clementine. Thumbs up :+1:

+1
Additionally, I didn't find Clementine.js a good starting point for the dynamic web projects. Its directory structure is heavily based on Angular. As we are supposed to develop the frontend component with React, a React specific version would be much more useful.

I agree with this as well. I got very confused when first encountering Clementine for API projects due to unknown components (Passport and Mongoose). I ended up stripping pretty much everything and working from server.js

+1 I was trying to figure out where I was going wrong before I was pointed in the direction of "modularization" which hadn't yet been really covered. Once I realized I could run the program in the server file, I was squared away. Not necessary to use Clementine.

cc/ @FreeCodeCamp/issue-moderators

I am in favor. But lets hear it from others.

I agree with @raisedadead, I am also in favor :+1:

I completely agree. By that point in the curriculum students should be familiar with Express and that is the only thing needed to start cracking on those projects.

I'm going to give this a go.

Bumping up for comments.
/to @BerkeleyTrue @QuincyLarson and /cc @FreeCodeCamp/issue-moderators

@raisedadead, I think it's a good idea to move... However, HyperDev is still limited in options, where as c9 is a full linux box.

IMO c9 makes more sense to teach with but I see the advantage of HyperDev.

For the API challenges, we could definitely consider HyperDev. It is much, much easier to get started with, and replaces both c9.io (to a large extent) and Heroku (which requires a credit card and excludes a LOT of campers).
If we are going to move away from using Clementine.js anyway (which I agree is too heavy for the API challenges), I vote we move to HyperDev.
We can link to wiki articles that show campers how to use c9.io/Heroku, or dev locally. But HyperDev seems to be a sane default for all of the back end challenges.
@jondcoleman thank you for taking the initiative to submit the PR. What do you think about hyperdev?

Thanks @QuincyLarson
I feel torn about HyperDev - the abstraction certainly makes it easier.  There are a few disadvantages to HyperDev in my mind:

Loss of true git workflow.  You can export your project to github but there's no individual commits or standard workflow.
Non-standard ways of adding libraries.  For example, users might get tripped up by the fact that there is no npm install x --save when most guides out there will describe that as the way to install a package.
Lack of ability to use task runners (gulp, webpack, etc).  This wouldn't be a big deal for the API projects but for the dynamic web apps this would be a pain.
Lack of ability to use generators or boilerplates not specifically designed for HyperDev.  Because of the forced project structure this seems be an issue.
HyperDev is still in Beta so who knows what issues campers might run into.

My opinion would be to stay with c9 for the dynamic apps.  For API projects I would lean towards sticking with c9 but think it's less of an obstacle there.

@jondcoleman thanks a lot for this research, and you are absolutely right, we should not be moving to something, that needs no standard ways of doing stuff. This would confuse beginners, which ironically is the opening post in this thread.
Let's just stick to removing the ClementineJS part and simplify the instructions as suited best with C9.
Thanks a lot, once again.

Pleas i need someones help, i dont know how to link my git hub with my FCC

@steelman80 please consult the Help Chat Room.


filtering for an alb resource won't import its underlying resources

Problem:
When you do:
terraformer import aws --resources=alb --region=
It imports all underlying resources, i.e.
aws_lb
aws_lb_listener
aws_lb_listener_rule
aws_lb_listener_certificate
aws_lb_target_group
aws_lb_target_group_attachment
However, if you filter the alb you want by filtering it via id (arn), it does not import all its underlying resources, but simply the lonely aws_lb.
terraformer import aws --resources=alb --filter="Name=id;Value='arn:aws.......' --regions=
This is inconsistent behavior and makes importing a single ALB and its underlying resources a complete chore.

even filter option is not working with tags
terraformer import aws --resources=alb filter="Name=tags:test;Value=import"


ProgressColor is not showing properly

I use the color code #1577F0 (blue) but it seems to be drawing with  #36303F (tuna), screenshot and code below
Screenshot => http://i.imgur.com/jf1c8PD.png
Code
    productionDetailProgressBar.setProgressColor(R.color.colorPrimary);
    productionDetailProgressBar.setMax(100);

And in xml
    <com.akexorcist.roundcornerprogressbar.RoundCornerProgressBar
            android:id="@+id/progress_1"
            android:layout_width="match_parent"
            android:layout_height="30dp"
            android:padding="8dp" />

Nvm, got it figured out. Have to use Color.parseColor instead of passing color resource id


"luna send" popup whenever settiing the screen brightness

If I have the TV brightness as a separate switch, whenever I change it's percentage values I get an alert on the TV that says "luna send" with an OK button that never disappears.

Hi,
can you post a debug log from the point when that happens?

As stated in the README the deep debug log property has only an effect if you start homebridge in debug log. So you need to start homebridge with the -D flag in debug mode like that: homebridge -D. Then you need to replicate the issue and watch the log for information which pops there.

This is when attempting to "flick the switch" for changing picture mode to "Game":
I have a LG 65B7V or something, it has firmware version 06.00.05, and webOS version 3.9.0-62705 (dreadlocks2-dudhwa). Running Homebridge from a raspberry pi.
[28.6.2021, 22:17:32] [webostv] [StiTV] system.notifications service - createAlert requested. Title: lunaSend, Message: lunaSend, Type: confirm, Modal: true
[28.6.2021, 22:17:32] [webostv] [StiTV] Message from TV

[28.6.2021, 22:17:32] [webostv] [StiTV]  {
  type: 'utf8',
  utf8Data: '{"type":"response","id":"814d1cda000f","payload":{"returnValue":true,"alertId":"com.webos.service.apiadapter-1624911453362"}}'
}
[28.6.2021, 22:17:32] [webostv] [StiTV] {
  "returnValue": true,
  "alertId": "com.webos.service.apiadapter-1624911453362"
}
[28.6.2021, 22:17:32] [webostv] [StiTV] system.notifications service - closeAlert requested. Alert id: com.webos.service.apiadapter-1624911453362
[28.6.2021, 22:17:32] [webostv] [StiTV] Message from TV

[28.6.2021, 22:17:32] [webostv] [StiTV]  {
  type: 'utf8',
  utf8Data: '{"type":"error","id":"814d1cda0010","error":"404 no such service or method","payload":{}}'
}
[28.6.2021, 22:17:32] [webostv] [StiTV] Unknown error while requesting closeAlert from service system.notifications

The log looks fine, it works as expected, but it seems like older webOS TVs do not have the closeAlert service...
Does the control center remote work for you? If so then i will fallback to the ENTER button on older webOS TVs.

Yes, I can use Enter, all directions, and back. But the action doesn't actually work. It's not changing the picture mode.

The picture mode should change after you press the ok button, if that is not the case then your TV probably does not support the specified picture modes.

Oh wait, if I push the button twice in a row it seems to work:

Be on "Standard" picture mode
Activate/swipe "Game" picture mode
Popup appears, but picture mode is not changed
Activate/swipe "Game" again
Picture mode is changed
close popup

If I close the popup after the first swipe, it just shows the popup again without changing the picture mode.

I will provide a hotfix update in a moment which should fix the popup issue.

@sfla please try version 2.2.1, the popup should now not appear anymore.

The popup now flashes for a frame before disappearing, which is completely fine. Sadly, it didn't fix the issue regarding not actually changing the picture mode. And since the message is now dismissed immediately, I'm no longer able to call the action twice to force it. Strange! Race condition? Or maybe it somehow needs to be called twice before you dismiss it?

Nothing needs to be called twice, it either works or not, If it does not change the picture mode after it disappears once then your tv most probably does not support the specified picture mode.

As I stated in previous comment, it did work if I called the action twice. Once to show the popup, then once again WHILE the popup was visible. It seems like the action needs to be called while this popup is visible, and that the current implementation somehow tries to change the picture mode before the popup has appeared.

You can try to replace this code lines:
https://github.com/merdok/homebridge-webos-tv/blob/9ccfb407392986a3a2ebfea1e490da43c42b9dd0/lib/LgTvController.js#L1113-L1119
with this code:
let result = this.openAlert('lunaSend', 'lunaSend', true, buttons, onClose, onFail, 'confirm', true).then((res) => {
      if (res && res.alertId && this.getWebOsVersion() >= 4) {
        this.closeAlert(res.alertId); // seems that the close alert service is only available on webos4 or higher
      } else {
        let result = this.openAlert('lunaSend', 'lunaSend', true, buttons, onClose, onFail, 'confirm', true).then((res) => {
          this.sendRemoteInputSocketCommand('ENTER');
        })
      }
    })

And let me know if that works for you.

It took me an hour to wrap my head around building stuff for Homebridge, but it worked like a charm! The popup occasionally blinks twice now (when going from Game to Standard), but that's no problem. It now successfully changes the picture mode every time. Thanks, man! And excellent response time.
Another question: I literally ONLY need the ability to change between Game and Standard picture mode through the Home-app. Power and inputs etc are all controlled by the connected devices.
The layout for this TV in the app is pretty darn hideous. Is it possible to customize the layout?
This is what I'm looking at:

I don't need the power-button, and the modes don't have to be huge switches, they can just be two small buttons. Is that possible?

Good to hear that it works now :) Seems like that code is required for TVs with webOS lower then 4.0, will update that.
As for you questions, you should be able to hide the power button with the hideTvService property. After that you can ungroup the two mode tiles in the home app which should give you two separate buttons. Or just simply straight ungroup the tiles.
As for the code, this is not supported by Home app, adding this characteristic does not change anything in the Home app UI, so Apple first needs to implement it... but they taking themselves time since this is already since a long time but no support in the Home app still...

Perfect! Enabling "Hide the TV Services" did the trick. The only side-effect this had (obviously) is that the icon for the TV changed from a "TV icon" to a "Switch icon". In addition, the "state" of the icon is always "off" regardless of the TV's power status. This makes complete sense, though. Still, it's now a lot easier to use, thanks!
In a perfect world I'd prefer to keep the TV-icon for the group, and that the group mimics the power state of the actual TV (but preferably without having the power-button inside). I can understand that this might be literally impossible to implement though.
Thanks for all your help.
I didn't create this issue, but I'm pretty certain it can be closed.

Good to hear that it works👍
Unfortunately we do not live in an ideal world😅 If you want to see the status of the tv then you need to have that power button enabled which is part of the tv service, this is also the switch which is used for the group tile to show the status. If you hide the tv service then you will lose all that information.


JWT Authentication for Secure User Access using Node js Express Js and MongoDB [ Backend ]

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
This feature implements backend authentication using JSON Web Tokens (JWT) to securely authenticate users in a Node.js application. Users will be able to sign up, log in, and receive a JWT upon successful authentication, which they can use to access protected routes in the application. JWTs are generated on the backend and validated with a secret key.
Use Case
User Registration: When a new user signs up, their credentials (username, password, etc.) are stored in the database with a hashed password.
User Login: Users log in by providing their credentials, which are verified against the database. If successful, a JWT is issued.
Protected Routes: Certain routes (like user profile or settings) are accessible only when a valid JWT token is provided in the request header. These routes check the token for authentication and authorization.
Benefits
Stateless Authentication: JWT allows for stateless authentication, meaning the server doesn’t need to store session data. All authentication info is included in the token itself.
Scalability: Since there’s no need to manage session data, it’s easier to scale the application horizontally.
Security: JWT tokens are signed and can be verified using a secret key, ensuring the token’s integrity. The token includes the user’s ID or other relevant data, making authentication efficient and secure.
Efficiency: JWT can be easily used in single-page applications (SPA) or mobile apps where maintaining a session on the backend might be challenging.
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24EXTD contributor
[X] I have starred the repository

@Abhishek-TG18 go ahead


Text Such as "\alpha" in latex is ignored while building pdf due to ""

The text such as \alpha and \beta (not in math environment) is not reflected in pdf because it is starting with a "". For example the highlited line in both the below figures

@AakashGfude Would you mind to help me with this (as John suggested)

sure @AnjuJoon , I will have a look

Thanks, @AakashGfude. Also, this is an issue with Julia lectures. In 3-4 lectures, there are symbols like \$ or \\$ in-between text for the demonstration, which are wrongly interpreted by latex of course because they symbolize the beginning and ending of math environment. This will also give an error while building the global pdf and individual pdfs and stop the process.

Hi @jstac @AnjuJoon @mmcky , so regarding the issue of insering greek letters, the pdf in julia is working (you can see the \alpha being rendered ) :-

whereas in python case, \alpha is not being rendered.

The difference is the way rst is written  :-
julia - ``\alpha`` in  rst, `\\alpha` in ipynb, and \texttt{\textbackslash{}alpha} in tex
python - `\\alpha` in rst, \\alpha in ipynb, and \alpha in tex
so, my question is, is second also the correct way? should the python one be handled? or if handled it may break something else?

Hi @AakashGfude why should it be different for Julia and python lectures since the normal text(other than code) works the same way for Julia or Python and the conversion to ipynb is same too


No Google Cardboard or Daydream support, even as Polyfill?

When I try to enter VR on a Pixel 2 XL (set to use Daydream) or a Samung Galaxy S8 (set to use Cardboard), in both cases it 'enters' VR but all I see is a black screen. Confirmed on both phones that other WebXR experiences, such as aframe, load fine.

interested in a solution for this as well, discovered it works very well with Samsung Internet on android LG and Samsung phones, would be great to get it to render in Chrome as well.

noticed that it does register if using Firefox too, but yeah Chrome just gives a black screen (A-Frame does work with Chrome). Even in Firefox though, hitting the 'gear' button doesn't take me to the Cardboard set-up the way other WebVR sites do.

thanks for that, did not check with FireFox on Android before, it works (feels like it's slightly loosing frame rate in comparison with Samsung Internet), tested it on Magic Leap as well in Helio 0.98, unfortunately gets stuck at "Made with Unity" stage and never starts, other VR examples created with immersive-web libs are working however with very low frame rate, loosing spatial tracking.

Vive Cosmos getting no gyro and no controllers, renders the world in headset and shows split screen in browser but no looking around or interaction possible. (tested on https://mixedreality.mozilla.org/unity-webxr-export/Build/)


None class for some objects

Hi,
I am using your demo notebook for getting annotation but, for some objects label is None. How to resolve this ? Do i need to provide certain class names only which are predefined ?

@ERYAGNIK003 Can you share the code with which you are working? What base and target models are you using?

Yes, sure.
I have used tutorial from roboflow and have defined classes as: CLASSES = ['car', 'bike', 'person', 'bus','van'].
roboflow notebook tutorial: https://blog.roboflow.com/enhance-image-annotation-with-grounding-dino-and-sam/
GroundDino model : https://github.com/IDEA-Research/GroundingDINO.git
SAM model : https://github.com/facebookresearch/segment-anything.git
Specifically, for bikes it is showing None type objects.

Hello! That tutorial is not using Autodistill. If you want to use Autodistill with those models, you can use:
from autodistill_grounded_sam import GroundedSAM

# define an ontology to map class names to our GroundingDINO prompt
# the ontology dictionary has the format {caption: class}
# where caption is the prompt sent to the base model, and class is the label that will
# be saved for that caption in the generated annotations
# then, load the model
base_model = GroundedSAM(ontology=CaptionOntology({"shipping container": "container"}))

# get predictions for a single image
results = base_model.predict("image.png")
print(results)

# label all images in a folder called `context_images`
base_model.label("./context_images", extension=".jpeg")

Since there has been no activity on this issue in a year, I am going to close it. If the code above did not fix the problem, let us know.


bug

config = {'algorithm': 'C4.5'} #Set algorithm to ID3, C4.5, CART, CHAID or Regression
model = chef.fit(df, config)
this code does not work with the newest pandas library. exception occurs. with pandas 1.5.3 everything good.

try with chefboost's new 18 version


Dauntless Shield is not a defense modifier

It just raises Defense stage by 1, like using Defense Curl. It's certainly doesn't always boost it in Grassy Terrain, hahaha

Thanks!


[Grid] Add responsive direction and spacing props

[x] I have followed (at least) the PR section of the contributing guide.

The aim of this PR is to have responsive direction and spacing props for Grid component.
Fixes: #6140
Preview: https://deploy-preview-26590--material-ui.netlify.app/components/grid/#responsive-values

@oliviertassinari I found a flaw in local fn resolveBreakpointValues, so I simplify it with getBreakpointValue and put it in system package (breakpoints.js because it needs the type there). With this change, the complexity is reduced and this is now possible.
<Grid container columns={{ xs: 8, md: 12 }}>
  <Grid item xs={4} />
</Grid>

in md, the item will have 33.33% width.

@siriwatknp Great to have taken the initiative. If we could add a test case for the bug we fix, it would be even better

@oliviertassinari I reverted the change I made and resolve conflict from #26559. I think it is good to go (also fix the type columnSpacing: number; rowSpacing: number;  to responsive

@siriwatknp Looks great 👍

@siriwatknp We could have updated this list with rowSpacing and columnSpacing


Paper tape punch output does not scroll properly in Google Chrome

Paper tape output on the Control Panel should scroll from right to left as the punch operates. This works correctly in Firefox, but not in Chrome -- Once the textbox is full, scrolling ceases, and you can no longer see the latest punch output.

Resolved, not sure how. There haven't been any source changes that should affect this. Perhaps it was a Chrome bug that got fixed by a recent update.


Better code example series

I don't think azure-voting-app-redis is a best example.
https://github.com/Azure-Samples/azure-voting-app-redis
Ideally, can we document how to create an app that has multiple images based on each projects, ideally via visual studio?

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

ID: 65f634bf-2280-a093-5897-724310dafa2d
Version Independent ID: f11c0af7-bfaf-6d64-2c75-9015b0eb1f30
Content: Kubernetes on Azure tutorial  - Prepare an application
Content Source: articles/aks/tutorial-kubernetes-prepare-app.md
Service: container-service
GitHub Login: @mlearned
Microsoft Alias: mlearned

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@PingPongSet we appreciate the feedback. At this time we have decided to leave the doc as is. However your comment will remain on the doc so if others feel the same they can let us know and we can always reconsider in the future.

I agree with @PingPongSet. This sample doesn't give an idea of how to start from scratch from VS.


Adds attributes to XML for international labels.

This commit adds to the XML used to purchase postage, so international
labels can be purchased through the gem. To use it one must include a
hash with a customs key. Within this hash needs to be certain
attributes. I have included a sample.
{
   ContentsType: 'Merchandise', #This is the type of contents.
   CustomsCertify: TRUE, #This is either TRUE or FALSE.
   CustomsSigner: 'Jim Johnson', #This is the name of the certifier.
   ToCountryCode: "GB", #Two character country code.
   CustomsItems: [{
     Description: 'Electric Parts Tubes', #Description of the items being sent.
     Quantity: 6, #Quantity of items.
     Weight: 2.5, #Weight in ounces
     Value: 13.86, #Value of items being sent.
     CountryOfOrigin: 'US' #Country items were made.
   }]
}

The CustomsItems is expected to be an array even if its length is one.
as to any other questions about the particular attributes one can check
the Endicia Api Docs,
and search the attributes, because the naming used corresponds with the
XML attributes, that are needed, within the documentation.

@dpritchett I just realized I opened this pull request in the wrong place. Sorry, for any confusion this may have caused. I currently work at TubeDepot, and I was trying to update our fork of the gem. I just wasn't paying attention when I opened the pull request. I am going to go ahead and close this pull request.

Very cool! Hope it works out on the other repo 😀


com_google_fonts_check_035: adding exception for python3

This pull request addresses the problems described at issue #1936
I tested this fix on MacOS 10.11 and it works.

Travis does not like it...
On python 3.6:
************* Module fontbakery.specifications.general
W: 96,17: Using deprecated method readPlistFromBytes() (deprecated-method)

According to the documentation, plistlib.readPlistFromBytes is "Deprecated since version 3.4: Use loads() instead."

I'll resume work on this at PR #1947


Perf: fix extraneous commits

Summary:

calendar is now directly derived from viewing thanks to useMemo instead of computed after effect. This spares on extraneous commit!
callbacks which are referentially stable (e.g., they don't depend on variables in useLilius scope) have been moved outside of useLilius (inRange, clearTime).
remaining callbacks have been wrapped in a useCallback hook to maintain referential equality and avoid potential extraneous re-renders in consumer components and hooks. More on referential equality in this nice article.
enable react-hooks eslint plugin which was installed but not used.
suppress rules of hooks infringements in examples/; you might want to take a look and fix those but I didn't want the scope of this PR to be too large.

@jsamr massive thanks!


Fixed unquote in GET and POST requests for alien characters

Fixed unquote in GET and POST requests for German umlauts 'äöüÄÖÜ' and other alien characters.
If characters like  'äöüÄÖÜ' are typed in a form input the browser converts FIRST to utf-8 and SECOND escapes the sequence.
At the server the string has to be unquoted (%xx -> X) first and than convertid from utf-8 to unicode

:)


[FEATURE][dashboard] Remove the function of a lost shuffle server.

What changes were proposed in this pull request?
If some shuffle servers are lost, they need to go offline and the Coordinator has been in the lost state. You need to add the delete function to delete these servers in the memory.

Why are the changes needed?
Fix: #1401
Does this PR introduce any user-facing change?
Yes.
How was this patch tested?
Add UT.

@xianjingfeng @rickyma @jerqi @maobaolong PATL

Codecov Report
Attention: Patch coverage is 0% with 13 lines in your changes missing coverage. Please review.

Project coverage is 53.47%. Comparing base (dddcced) to head (7283732).
Report is 36 commits behind head on master.

Files
Patch %
Lines

...ache/uniffle/coordinator/SimpleClusterManager.java
0.00%
9 Missing :warning:

...iffle/coordinator/web/resource/ServerResource.java
0.00%
4 Missing :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1870      +/-   ##
============================================
- Coverage     53.53%   53.47%   -0.07%
- Complexity     2356     2404      +48
============================================
  Files           368      372       +4
  Lines         16852    17198     +346
  Branches       1540     1579      +39
============================================
+ Hits           9022     9196     +174
- Misses         7303     7462     +159
- Partials        527      540      +13

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

ping @xianjingfeng ~

Merged to master. Thanks @yl09099 @xianjingfeng


display confirmation message after submitting newsletter form

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
to display confirmation message after submitting newsletter form
Use Case
its important user gets a positive response that there response has been recorded
Benefits
No response
Priority
High
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

I would like to work on this issue pls assign me this @mansiruhil13


Tiles 7238 and  8796 are 3-sigma outliers in new Lya QSOs

Inspecting tile-qa from 20230326 I've found that tiles 7238 and  8796 are 3-sigma outliers in new Lya QSOs. I've marked those two tiles as unsure, waiting for more feedback from the SurveyOps team.

The tile QA does not look suspicious, except for the bright star on tile 8796.

@ashleyjross commented on slack " the tile qa looks fine for these, they are almost definitely ok. (Anything with n_overlapping > 5 is usually fine.)". Closing this issue.

those are the "usual" case for tiles with high previous coverage, hence few new lyas, hence more scatter.
the general rule is that, as long as the error bar is overlapping the 3sigma red shaded region, it s good.
confirming Ashley s comment, here are the newlya spectra for those two tiles, which look good:
https://data.desi.lbl.gov/desi/users/raichoor/tmpdir/tmp-daily-newlya-7238-thru20230326.pdf
https://data.desi.lbl.gov/desi/users/raichoor/tmpdir/tmp-daily-newlya-8796-thru20230326.pdf
ps: I really need to push that script to desisurveyops (or desispec); sorry!


Refactor Mesh Request Handler

Motivation
Closes #4389
MeshHashReq contains redundant information that can potentially be abused by an attacker to crash a node by forcing the node to allocate more memory than what is available on the system.
Changes

Removed Steps from request and renamed Delta to By
Calculate Steps from the other 3 parameters
Limit the number of hashes that can be requested in a single request to MaxHashesPerReq (defaults to 100)
Do not echo back the layers that were requested
Instead of using a string builder to query layers from DB use SQL statements

Test Plan

Added tests for changed functionality

TODO

[x] Explain motivation or link existing issue(s)
[x] Test changes and document test plan
[x] Update documentation as needed

DevOps Notes

[x] This PR does not require configuration changes (e.g., environment variables, GitHub secrets, VM resources)
[x] This PR does not affect public APIs
[x] This PR does not rely on a new version of external services (PoET, elasticsearch, etc.)
[x] This PR does not make changes to log messages (which monitoring infrastructure may rely on)

bors try

bors try

bors try

@piersy @countvonzero
Going through the review makes me realise that maybe the current implementation can be improved further. The fork finder sets up a boundary based on the latest layer two nodes agree on (From) and the oldest two nodes disagree on (To). This boundary is then used to request all hashes between those two layers (limited by MaxHashesInReq - currently hard coded to 100 on both the client and server side).
This means that if a fork is detected a request is made that can request up to 100 hashes, but if From and To differ by more than 100 at least another request (for at least 3 hashes) has to be made to find the layer where the divergence happened.
What if we change that logic to instead do a binary search and update the boundaries based on if the hash in the middle between From and To matches or not? This would simplify the code further by using a recursive search: If the hash matches update From, if it differs update To -> call recursively until From and To differ by no more than 1 layer -> To is the layer where the fork occurred.
So instead of 1 request with 100 hashes, we would do 5-6 requests for one hash each (assuming From and To differ by 100).
This would also remove the need for GetAggHashes, the need for MaxHashesInReq, some of the validation currently needed to check for overflows, etc.

Sounds good to me, I'm just wondering why that approach wasn't taken initially, is it because the overhead of each request would be high?
A side note, I'm still seeing syncer.Config.MaxHashesInReq existing in this PR

@piersy: I updated the ForkFinder also to use a const for MaxHashesInReq instead of having it a configurable parameter

New Issue for refactoring (after genesis): https://github.com/spacemeshos/go-spacemesh/issues/4517

bors merge

bors merge


Fix arrow-avro compilation without default features

Which issue does this PR close?
none
Rationale for this change
Before the change, using arrow-avro without default features would fail to compile. This can be seen with
cargo check -p arrow-avro --no-default-features

What changes are included in this PR?
make code compile with default features disabled
Are there any user-facing changes?
no

( Found when working on https://github.com/apache/arrow-rs/issues/6796 )


Report invalid params keys

Invalid configuration options should error to make it easier to detect user errors

Published types should help with this already. Now that the config is simplified this is less of an issue.  Closing for now but we can revisit if it causes issues


imagepullsecret missing in helm chart

We wanted to deploy sloth with helm chart but we couldnt deploy successfully because we wanted to use our private image repository, it seems like imagepullsecret is missing sloth helm deployment yaml.

https://github.com/slok/sloth/pull/400 pr merged


[test]Add feature for CRI extensions

Signed-off-by: Hamza El-Saawy<EMAIL_ADDRESS>

Can you please use the common form test: Add feature... for the commit/PR description?
What is this new feature flag intended to do?

I just saw this comment, sorry
And it prevents running the CRI restart API tests (until we move those tests out of this repo to the ADO AzCRI repo)


Implements add users directly inline from empty group view

resolves #5654

Solved by having a parent view “on top” which manages the edge-cases of transitioning from/to empty group (i.e. no group users).
Also had to modify the group users collection to do a fetch on the batch operations to reflect the state (thus the promise resolve is delayed slightly).
@javierarce can you review please?
cc @xavijam @juanignaciosl

Staging?

@xavijam :rocket: rui1/2 (hold your :horse: for a few minutes)

:+1: Code looks good!

@xavijam …deployed on rui1+2 now, have fun! :roller_coaster:

Looks great!


Added redirect for DNS naming topic

What are your changes?
Added redirect for DNS naming topic
What is the urgency level of this change?

[ ] Blocker
[ ] High
[ x] Medium

I have completed these items:

[ x] I have built locally and tested for broken links and formatting.
[ x] If appropriate, I have added redirects.

If you included a comment with your commit, it appears here:

Related: https://github.com/dcos/dcos-docs/pull/651

retest please


[BUG] Delta table out of order after caused by mergeSchema

Bug
Delta table out of order after caused by mergeSchema
Describe the problem
Append rows with the first schema (Delta streaming)
Create aggregate (aggregate is correct)
Append rows using a new schema (Delta streaming)
Create aggregate (aggregate is wrong)
Steps to reproduce
Append rows with the first schema (Delta streaming)
Create aggregate (aggregate is correct)
Append rows using a new schema (Delta streaming)
Create aggregate (aggregate is wrong)
Observed results
The aggregate with the new schema is inconsistent (wrong)
Expected results
The aggregate results should be the same as with the previous schema
Further details
Environment information

Delta Lake version: 1.0.0
Spark version: 3.1.3
Scala version: 2.12

Willingness to contribute
The Delta Lake Community encourages bug fix contributions. Would you or another member of your organization be willing to contribute a fix for this bug to the Delta Lake code base?

[ ] Yes. I can contribute a fix for this bug independently.
[ ] Yes. I would be willing to contribute a fix for this bug with guidance from the Delta Lake community.
[ ] No. I cannot contribute a bug fix at this time.

Do you have a concrete example to show the issue? It's hard to get the problem you raised right now.

Yes I have a concrete example, I will also edit the description to explain further the inconsistency.


nixos/sabnzbd: add package option

Motivation for this change
Add a package option to the sabnzbd service.
Things done

Tested with this fix because sabnzbd is broken on master: https://github.com/NixOS/nixpkgs/pull/138800

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I think you're looking for overlays. This is exactly what overlays are for.

I think you're looking for overlays. This is exactly what overlays are for.

There are many other services with this same option: https://search.nixos.org/options?channel=21.05&from=0&size=50&sort=relevance&type=packages&query=package
Is there a style guide for when overlays should be used vs adding a package option to the module?

It's fine to add the option of you really want to. I always mention overlays to make sure everyone knows that overlays exist and what they are for - then they can decide what the right answer is.

Makes sense, thanks!  In this case I would still like to add it.


Rework arguments & remove --random (for now)

@Natim the argument for --random didn't work for me.
Also, I cleaned the code a tiny bit, because I thought it would be nicer to have things more separated.

I took you comments in https://github.com/caesarlib/caesarlib.rs/pull/15/


/en-US/docs/Web/API/Element/innerHTML: spelling error "elment's".

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details

In the section "Usage Notes" -> "Reading the HTML contents of an element" there is a spelling error:
In the sentence:

Reading innerHTML causes the user agent to serialize the HTML or XML fragment comprised of the elment's descendants.

... the word elment's should be element's.

Fixed; thanks for reporting!


data order be changed after deleteObject

Hello,
i have a question for deleteObject behavior.
i addObject to realm like
 1
 2
 3
 4
 5
 6
 7

than delete the 3 object use deleteObject, i think the data should change to
 1
 2
 4
 5
 6
 7

but the data change to
 1
 2
 7
 4
 5
 6

why the latest object fill to the third position? is this a normal behavior?
if my english is not clear enough, i can create a repository to reproduce what i say.
thanks.

Hello! First of all, this only happens for lists of objects of a certain class that you can get from e,g. +allObjects (and not for RLMArrays) In these cases there is no guarantee on the ordering of objects, Realm can change it whenever it wants. In this case the behaviour you have experienced avoids having to 'move' all the objects that 'come after' the one you deleted!
So in short, if you need an ordered list of objects, put it in an RLMArray property!

Hello! First of all, this only happens for lists of objects of a certain class that you can get from e,g. +allObjects (and not for RLMArrays) In these cases there is no guarantee on the ordering of objects, Realm can change it whenever it wants. In this case the behaviour you have experienced avoids having to 'move' all the objects that 'come after' the one you deleted!
So in short, if you need an ordered list of objects, put it in an RLMArray property!

Hello @GreatApe,
thanks, i got it.
close this issue.

Hello @GreatApe,
thanks, i got it.
close this issue.


Add condition to replace onDoubleClick with onClick to make it work for mobile.

Any chance that it can be approved? It would be nice to replace onDblClick with onClick for mobile devices.

Hey @sherbakov1, just quickly looked over this.
First I haven't built this for mobile as there are lot of edge cases. However this solution seems to forward the onDoubleClick (meant to start edit) into a onClick (meant to start a selection).
If your goal is to only start edit on touch, can I suggest trying ontouchstart to see if that works? This would also remove the helper code to detect mobile

Just did a quick test: onTouchEnd={onDoubleClick}  is sufficient for what you're trying to achieve. Feel free to just add that and I'll merge.

@nadbm thank you for the fast response and your help. I just pushed these changes.

Coverage decreased (-0.3%) to 93.385% when pulling d0323c9129d1d2b6f533bfc1f5024a6cbda973fc on sherbakov1:master into 894a4ea7386f80c7224259d7892ca4f03123b5a3 on nadbm:master.


training issue

Hi!
Have a question about training.
After 16 hours of training, I still get average reward 0.
Will be happy if you can explain what can be wrong?
Maybe it's a problem with default setup parameters?

25%|███▊           | 6374997/25000000 [16:18:15<48:41:33, 106.25it/s]INFO:deep_trading_agent:avg_r: 0.0000, avg_l: 0.000135, avg_q: -0.001807, avg_ep_r: 0.0000, max_ep_r: 0.0911, min_ep_r: -0.0984, # game: 5000
26%|███▊           | 6399993/25000000 [16:22:11<48:18:45, 106.94it/s]INFO:deep_trading_agent:avg_r: 0.0000, avg_l: 0.000135, avg_q: -0.001469, avg_ep_r: 0.0000, max_ep_r: 0.0985, min_ep_r: -0.0641, # game: 5000
26%|███▊           | 6424989/25000000 [16:26:07<48:06:53, 107.24it/s]INFO:deep_trading_agent:avg_r: 0.0000, avg_l: 0.000138, avg_q: -0.001775, avg_ep_r: 0.0001, max_ep_r: 0.1445, min_ep_r: -0.0460, # game: 5000
26%|███▊           | 6449993/25000000 [16:30:03<48:41:25, 105.83it/s]INFO:deep_trading_agent:avg_r: -0.0000, avg_l: 0.000134, avg_q: -0.001525, avg_ep_r: -0.0000, max_ep_r: 0.0223, min_ep_r: -0.0371, # game: 5000
26%|███▉           | 6477033/25000000 [16:34:16<47:10:48, 109.06it/s]INFO:deep_trading_agent:avg_r: -0.0000, avg_l: 0.000138, avg_q: -0.002763, avg_ep_r: -0.0000, max_ep_r: 0.0302, min_ep_r: -0.0762, # game: 5000
26%|███▉           | 6499197/25000000 [16:37:41<47:10:50, 108.92it/s]INFO:deep_trading_agent:avg_r: 0.0000, avg_l: 0.000142, avg_q: -0.003163, avg_ep_r: 0.0000, max_ep_r: 0.0352, min_ep_r: -0.0225, # game: 5000
26%|███▉           | 6526765/25000000 [16:41:56<47:30:54, 108.00it/s]INFO:deep_trading_agent:avg_r: -0.0000, avg_l: 0.000135, avg_q: -0.003114, avg_ep_r: -0.0000, max_ep_r: 0.0253, min_ep_r: -0.1445, # game: 5000
26%|███▉           | 6551381/25000000 [16:45:43<47:47:03, 107.25it/s]INFO:deep_trading_agent:avg_r: 0.0000, avg_l: 0.000131, avg_q: -0.002506, avg_ep_r: 0.0000, max_ep_r: 0.0643, min_ep_r: -0.0199, # game: 5000
26%|███▉           | 6577145/25000000 [16:49:41<47:26:52, 107.85it/s]INFO:deep_trading_agent:avg_r: -0.0000, avg_l: 0.000137, avg_q: -0.001795, avg_ep_r: -0.0000, max_ep_r: 0.0300, min_ep_r: -0.1185, # game: 5000
26%|███▉           | 6599989/25000000 [16:53:14<46:38:00, 109.60it/s]INFO:deep_trading_agent:avg_r: -0.0000, avg_l: 0.000137, avg_q: -0.002334, avg_ep_r: -0.0000, max_ep_r: 0.0495, min_ep_r: -0.1122, # game: 5000

I believe that you are using code from the main branch in which the model contains earlier implementation of both the batch normalization and the dropout layers. There was an issue with the earlier implementation which I had then corrected in the dev branch. Recently I have merged the dev and master branches.
Please use the recent code from the main branch and re-run the training process.
If you wish to remove dropout from the model, set keep_prob for each layer to 1.0 in the configuration file.

Thanks, I just checked out latest master branch, and run the code with default config.
The problem still exists - after one day of learning it learns nothing :(
 47%|██████▌       | 11724993/25000000 [22:58:13<26:00:25, 141.79it/s]INFO:deep_trading_agent:avg_r: -0.0000, avg_l: 0.000079, avg_q: 0.027890, avg_ep_r: -0.0001, max_ep_r: 0.2953, min_ep_r: -0.2274, # game: 5000
 47%|██████▌       | 11749993/25000000 [23:01:09<25:57:28, 141.79it/s]INFO:deep_trading_agent:avg_r: -0.0000, avg_l: 0.000077, avg_q: 0.027767, avg_ep_r: -0.0002, max_ep_r: 0.2545, min_ep_r: -0.2279, # game: 5000
 47%|██████▌       | 11774985/25000000 [23:04:05<25:54:32, 141.79it/s]INFO:deep_trading_agent:avg_r: -0.0000, avg_l: 0.000088, avg_q: 0.026840, avg_ep_r: -0.0001, max_ep_r: 0.3711, min_ep_r: -0.6593, # game: 5000
 47%|██████▌       | 11799997/25000000 [23:07:01<25:51:34, 141.79it/s]INFO:deep_trading_agent:avg_r: 0.0000, avg_l: 0.000079, avg_q: 0.026633, avg_ep_r: 0.0000, max_ep_r: 0.2962, min_ep_r: -0.2784, # game: 5000

I haven't taken a deep look at the training scheme, but intuitively, something seems wrong when I graph the following Loss and Q-Value
Loss minimization seems good, which perhaps implies that the networks are learning, but the fact that the Q-Value reward is also becoming minimized seems alarming:

@joseph-zhong I have been making corrections in the code, the latest changes are reflected in the dev branch where I have integrated this repo with my gym-cryptotrading environment for ease of use.
With these bugs removals, I am also targeting to remove this specific issue but currently the btc_sim.average.q curve that I am getting is quite different from what you have shown (this curve might result from an earlier commit) but still converges to 0 towards the end of training.
Any help upon this matter is greatly appreciated.


style not working ?

pls check

@matamune94 If you are using nuxt simply import the file within nuxt.config.js.
  /*
  ** Global CSS
  */
  css: [
    'vue-awesome-notifications/dist/styles/style.css'
  ]

Vue integration section in docs was updated. CSS import guide was added: https://f3oall.github.io/awesome-notifications/docs/integrations/vue


Adding dropdown in the Contribution Guide

There is an alignment issue on the right hand-side navigation bar for the contribution page as seen in the screenshot below.

Kindly assign this issue to me so that i can rectify this problem.

I just realized the above (Caveats for code snippets) is a sub part of the Contributing Code snippets so adding a drop down would make it easier to recognize.

hi, this is consistent with the rest of the site. every page with a right menu displays it this way


Using react-navigation with state managers

I have worked on a project that used the version 2.3 of the library, and on this project we integrated the redux store with our navigation.
After sometime i started a new project, with the version 3. of the react-navigation. Reading through the docs, i find that the is no problem if you dont need to attach the navigation to your store, and this is not 'supported' anymore.
Would be a problem, create my store, my reducers, actions etc and my navigation component. And then, to call another screen use something like:
export const redirectAuthenticateAccount = email => async () => {   NavigationService.navigate(routes.AuthenticateAccount, { email }); };
My doubt is that it would cause some problems if a navigate through screen using this approach, and if needed call a dispatch to my actions?

hello there! this should not cause any problems as far as i can tell, but i didn't quite understand the last sentence, could you explain what you mean by "if needed call a dispatch to my actions?" and why this is making you doubt?

If i  use something like:
export const requestList = () => async (dispatch) => {
// Here i dispatch my action to perform some changes on the store
dispatch(requestFaqInfo());
//Here i do a request to my amazing api
const { payload } = await fetchContent();
//Here, based on this change i call a navigate to some screen
NavigationService.navigate(routes.AuthenticateAccount, { payload }); };
};
By doing this aproach, existe a possibilidade de ocorrer algum erro com minha store?
is there any possibility of an error with my store? Because by performing a dispatch(CallMyNavigateAction())  for some reason, my store ends up being reset.

It'd be nice for this to demonstrate how to use Redux, Mobx, and React Context in a react-navigation project

@brentvatne If so, I can link to a project of mine that uses Redux and React-Navigation separately. Although the approach I chose is simple, it turns out to be a bit complex to try to use both libraries on the same project.

sure, what's the link to the project?

@brentvatne I was on vacation and I could not send the project link before, if it is still possible to link it, I would be happy to contribute. =)
Repo link: https://github.com/MateusAndrade/react-navigation-redux-example

we have a section on mobx state tree and on using redux (not to store navigation state but just for your app state):
https://reactnavigation.org/docs/en/redux-integration.html
https://reactnavigation.org/docs/en/MST-integration.html


allow extraVolumeMounts without enabling default persistentVolume

Originally posted by @genofire in https://github.com/woodpecker-ci/helm/pull/94#discussion_r1358218931

Might be a false-positive

This should be open again, still not fixed.

just try this broken-values.yaml
server:
  resources:
    requests:
      cpu: "500m"
      memory: "200m"
  persistentVolume:
    enabled: false
  extraVolumes:
    - name: docker-config
      configMap:
        name: docker-config
  extraVolumeMounts:
    - name: docker-config
      mountPath: /etc/docker


Broker cleanup 2

Part 2 of Issue #22
@travisjeffery Made the changes suggested in PR #25
Change set: Added wrapper over serf.MemberStatus and raft.FSM to cleanly hide hashicorp raft and serf from jocko behind interfaces
Next steps: add/edit tests
Will open a new issue for this

Don't make a separate issue, the PR is basically an issue with code.

Looks pretty good!

Cool. Please merge PR.

ok merged it


Hamburger menu

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
This is an interesting style of drop down menu. Can be used in any website to add on to its UI.
I want to do this task so please assign me.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Steps Followed✅
Check the box with [x] If you followed all the steps.

[ ] Is your feature request related to a problem? Please describe.
[ ] Describe the solution you'd like
[ ] Describe alternatives you've considered
[ ] Additional context

Code of Conduct✅

[ ] I agree to follow this project's Code of Conduct.
[ ] I checked the current issues for duplicate problems.
[ ] I have gone through project README.

This is an interesting style of drop down menu. Can be used in any website to add on to its UI.
I want to do this task so please assign me.

@ankita-104
Are you willing to work on this issue, if not I would like to close this?


Chart range filter with other data

Hi there I am trying to recreate the ChartRangeFilter chart but I having some problems. If anyone can help, please. The image is the reference.

The thing is that i need to show the info for hours on the filter and the info minute to minute on the core chart, if anyone has done something like that I will be so happy/


tc start should show n + 1 issues at log-level=INFO

N + 1 issues are displayed only in the check command at the moment. We propose that the users should be able to see the total number of N + 1 issues that exist at the time of tc start.
The N + 1 information is only shown in case the log level is <= INFO.

/bounty 80$

Can I get assigned?

Hi @tusharmath, Can I work on this?

Hi @neo773, I just opened the feed to ask if I can work on this to gain some more experience(and hunt the bounty ngl) with the codebase. I see that the code coverage is not done in your PR yet. Can I add the tests for this? We could split the bounty if you like but I am fine with you having fully too. I just want to gain some experience around the project.

@A-N-uraag
I think that should be done in a separate PR as it's not in the scope of this issue.
And as far I recall there's partial coverage via unit tests.

Hi @neo773 and @tusharmath. I have submitted a PR since the earlier one was different that what was expected. Hope that's fine.


feat(payment): PAYPAL-2729 trigger customer execution method after creation account in create account form for BT AXO

What?
Trigger customer execution method after creation account in create account form for BT AXO
Why?
To be able to trigger PayPal Connect authentication flow right after customer creates an account
Testing / Proof
Unit tests
Manual tests
https://github.com/bigcommerce/checkout-js/assets/25133454/5113eb42-8b07-44ec-8106-e2bdce079a6f

Thanks @animesh1987 for the review. I fixed all comments in separate PR:
https://github.com/bigcommerce/checkout-js/pull/1470


database or collection per app option...?

It looks like the database storage is sorted for app by collection prefix which defaults to the appID, is this true?
Is it possible to allow an option upfront in the config to setup a new db for each app created?  Like in config.json an option we can set to true that will create a new mongodb://localhost:27107/<appId>.  This will allow easier portability backup and recovery on a per app basis...ya think?
If you want to take it a step further for security concerns you can generate a mongo username/password for new app also:  mongodb://username:password@localhost:27107/<appId> and create the user in mongo.  If you offer this feature then you will need to create the mongo admin username:password along with the dashboard admin on first app creation.

The original concept is that the multiple parse server apps only need one mongo db link because some DaaS (like mLab) is not available to set multiple database link in shared/sandbox cluster.
now you can set "useCollectionPrefix":false in config.json, the database link would be mongodb://localhost:27107/. In such case, the database would be easier to migrate.


BUG: Deleting drives in GUI after file upload prevents loading of the same file

As title states, found by @gtdang

ah. Seems like this bug just happened because of what we were doing with testing the FileUpload counter. I had commented out the resetting of the widget to show the counter incrementing to 1. We decided to continue to zero it instead. This actually resolves this. Good to know for if we ever want to restore the counter functionality, but I think this can be closed.


Check cross-nested-stack references

Check if the generated CloudFormation files contain the imported files as they should be.

They are good when exported/imported ES6-style


Always generate limit top in SqlDataProvider

I have code similar like this
$sql = "select field1,field2,ect from stuff";
$dataProvider = new SqlDataProvider([
    'sql' => $sql
]);

I look into my log file then i got one field added , maybe generate by SqlDataProvider class this is part of generated code
 SELECT TOP 20 * FROM (SELECT rowNum = ROW_NUMBER() over (ORDER BY (SELECT NULL))

I dont want  this code , because this code will be issued when i try to union 2 table , of course the field will be different 1 table at first got this generate code the second table is not.
how i can disable this feature ? so not show again select top .. ect

hi i got the solution
$sql = "select field1,field2,ect from stuff";
$dataProvider = new SqlDataProvider([
    'sql' => $sql,
    'pagination' => false
 ]);

pagination => false to set no use pagination that was remove select top blalbalbal ...
about issue union in my code i use one bind variable to multiple place and that was not support with my driver so i just create another bind variable and fix now.
http://www.yiiframework.com/forum/index.php/topic/24466-mssql-query-yii-problem/


fix ts warning

since 6.0.8 release in react native typescript complains about the types

Thank you will review today


Improvements .gitignore

Objective
I tried to improve gitignore. I am not a programming expert yet, so I don't know all the junk files. If someone provides me with a list of other files, can you point me to it below? I also tried to sort them logically, hope it works.

This is also my first PR in the history of this engine. and I'm very happy 😄

😄

I quite like the reorganization and comments :) I've left a couple of suggestions to try and improve clarity.

I quite like the reorganization and comments :) I've left a couple of suggestions to try and improve clarity.

Do you know any other junk files?

sorry i made a mistake


 It's run into the recalled battery issues for that model, so I've replaced it with a MacBook Pro (13-inch, M1, 2020) while I wait for it to be repaired.
Issue is however that electron-notarize appears to completely crash the M1 (full shutdown and restart) if it is interrupted.
The computer going to sleep interrupts it, causing the crash. Disabling sleep seems to indicate that notarize simply hangs forever sometimes, and never resolves (had it sitting for over an hour, which I'd never seen on the intel macbook). Trying to ctrl-C out of the process also interrupts notarize, causing the crash.
Of note, notarize doesn't seem to hang every time, as I've had one build that did eventually resolve, but all previous and subsequent ones seem to get stuck indefinitely.
electron-notarize 1.2.1
node v14.16.1
electron-builder 23.0.2
notarize script as follows:
require('dotenv').config();
const { notarize } = require('electron-notarize');

exports.default = async function notarizing(context) {
  console.log(`starting notarizing`, context)

  const { electronPlatformName, appOutDir } = context;

  if (electronPlatformName !== 'darwin') {
    return;
  }

  const appName = context.packager.appInfo.productFilename;

  console.log(`doing notarize`)
  const notarizeResult = await notarize({
    appBundleId: context.packager.info._configuration.appId,
    appPath: `${appOutDir}/${appName}.app`,
    appleId: process.env.APPLEID,
    appleIdPassword: process.env.APPLEIDPASS,
  });

  console.log(`notarize complete`)

  return notarizeResult
};

upon restart, all I see in terminal is the following from the restored terminal window:
[...assorted building/packaging logs]
starting notarizing {
  [...snip]
}
doing notarize

[Restored Mar 29, 2022 at 4:06:17 PM]

Mac crash report as follows:
panic(cpu 1 caller 0xfffffe002c5febf8): Sleep transition timed out after 35 seconds while calling power state change callbacks. Suspected bundle: com.apple.iokit.IO80211Family. Thread 0x40afe.
Debugger message: panic
Memory ID: 0x6
OS release type: User
OS version: 21D62
Kernel version: Darwin Kernel Version 21.3.0: Wed Jan  5 21:37:58 PST 2022; root:xnu-8019.80.24~20/RELEASE_ARM64_T8101
Fileset Kernelcache UUID: 7C53FB49FDE6F710235278E472070819
Kernel UUID: F5893147-A9D2-3D60-B5A4-E4AE9BA0A628
iBoot version: iBoot-7429.81.3
secure boot?: YES
Paniclog version: 13
KernelCache slide: 0x00000000244c4000
KernelCache base:  0xfffffe002b4c8000
Kernel slide:      0x0000000024cf4000
Kernel text base:  0xfffffe002bcf8000
Kernel text exec slide: 0x0000000024ddc000
Kernel text exec base:  0xfffffe002bde0000
mach_absolute_time: 0x18a277486ea
Epoch Time:        sec       usec
  Boot    : 0x6242530f 0x00042514
  Sleep   : 0x00000000 0x00000000
  Wake    : 0x00000000 0x00000000
  Calendar: 0x62436691 0x00076ffe

Zone info:
Foreign   : 0xfffffe0033510000 - 0xfffffe0033524000
Native    : 0xfffffe100061c000 - 0xfffffe300061c000
Readonly  : 0xfffffe14cd2e8000 - 0xfffffe1666c7c000
Metadata  : 0xfffffe8305c1c000 - 0xfffffe8311b54000
Bitmaps   : 0xfffffe8305c20000 - 0xfffffe8308828000
CORE 0 recently retired instr at 0xfffffe002bf6b90c
CORE 1 recently retired instr at 0xfffffe002bf6a320
CORE 2 recently retired instr at 0xfffffe002bf6b90c
CORE 3 recently retired instr at 0xfffffe002bf6b90c
CORE 4 recently retired instr at 0xfffffe002bf6b910
CORE 5 recently retired instr at 0xfffffe002bf6b910
CORE 6 recently retired instr at 0xfffffe002bf6b910
CORE 7 recently retired instr at 0xfffffe002bf6b910
CORE 0 PVH locks held: None
CORE 1 PVH locks held: None
CORE 2 PVH locks held: None
CORE 3 PVH locks held: None
CORE 4 PVH locks held: None
CORE 5 PVH locks held: None
CORE 6 PVH locks held: None
CORE 7 PVH locks held: None
CORE 0: PC=0xfffffe002be66e30, LR=0xfffffe002be66e30, FP=0xfffffe6f762cbef0
CORE 1 is the one that panicked. Check the full backtrace for details.
 I'm also keeping an eye to keeping the macbook process active during building/notarizing, and it seems to be helping with not crashing, though it's not 100%.
Of note, on my machine, interrupting the process from command line(ctrl-c) while notarize was awaiting completion would immediately and reliably cause the crash/restart. (the few times I tested it)

I have the same problem too.

I've been having this issue for months now too, it seems to work every now and then and sometimes just hang all the time.

I believe this issue is specific to the altool legacy notarization workflow. I was able to work around it and get notarization consistent by:

getting my team ID by manually running altool

xcrun altool —list-providers -u<EMAIL_ADDRESS>-p “appp-assw-ordh-ere”
Look for the WWDRTeamID.

edit your JS configuration for electron-notarize to add two new properties:
{
appBundleId:  …
…
appleIDPassword: …
teamId:  “ID_FROM_PREVIOUS_STEP”,
tool: “notarytool”
}

This change needs to be made by 2023 by everyone anyway, and notarytool is faster, so there’s very little downside to this workaround.
Would still love to get to the bottom of why shelling out to altool has such dire consequences on the M1, but this at least gets things up and running again.

Thanks, I've tried this method only once and it worked first go, hopefully this fixes it. It's more like an upgrade than a workaround. I've done a bit of digging recently and found that it could be related to [different] methods used to upload to Apple's notarization servers. And your solution using notarytool to fix this makes me think that might have been the case.
Source: https://stackoverflow.com/a/55868047/16608863

Fixed by using notarytool instead. Closing per #141


bug(mat-menu): close mat-menu on mouse leave - it starts flickering

Description
I am using mat-menu, I am displaying a menu when the mouse enters the event for a specific button.it works fine.
But I want to close mat-menu when the mouse leaves the button. it is not working.
Please fin the sample in below:
https://stackblitz.com/edit/example-angular-material-toolbar-menu-vrobtz?file=app%2Fapp.component.html
Reproduce steps

open the above sample
hover home icon, you can see menu popup gets opened
move away before entering menu, you can see menu popup not get closed
sample - https://stackblitz.com/edit/example-angular-material-toolbar-menu-vrobtz?file=app%2Fapp.component.html

Note - I have tried to use mouse-leave for that button but it start flickering
sample - https://stackblitz.com/edit/example-angular-material-toolbar-menu-uxlbba?file=app%2Fapp.component.html

Closing since this isn't something that we officially support.


Error flushing statements

Describe the bug
Every time I update the table, I do a full update (delete first and then insert)
However, when executing insert statements recently, Error flushing statements often appear.
Code example

Error log
Error flushing statements.  Cause: ru.yandex.clickhouse.except.ClickHouseUnknownException: ClickHouse exception, code: 1002,  failed to respond
ClickHouse server

ClickHouse Server version:  <IP_ADDRESS>
(https://github.com/ClickHouse/ClickHouse/blob/master/programs/obfuscator/Obfuscator.cpp#L42-L80) if necessary

Good day, @Evre-wuwy!
What clickhouse-java client version do you use? Have you tried with latest?
Would you please add more information:

exception stack trace
logs showing the problem

Thanks in advance!


add new status ROLLBACK_FAILED

when we attempt a rollback in case of deployment failures and when the rollback also fails, we must set the status to ROLLBACK_FAILED.
Currently we set DESTROY_FAILED which is misleading.

@baixinsui i had thought about it and decided not to have ROLLBACK_SUCCESS. Because in the happy case, the rollback happens without the knowledge of end user. So status in this case must be still 'DEPLOYMENT_FAILED' with the actual error of the failure.
Good point. If we are setting it to DESTROY_SUCCESS, then it must be fixed. We can fix that as part of this user story or another one.

OK.


[semver:patch] Move bash to scripts

Move to script like any other command.

Appears this change has been made already in a previously applied update. Thank you for your contribution


The router is not destroying a page when it gets unloaded via a step back in history

Hi, I put together a mini app to try to clear completely, from the memory/history, a page when the user navigate back, using settings: lazyDestroy false and destroyOnHistoryBack true.
I have a router with 2 pages. The default one is Home and the second one is Article. The article page has a dynamic route: article/:id.
From Home I go in the Article page and show its id, then I go back in Home with Router.step(-1) and then I go in Article again but this time I am passing a different id on the route. I am expecting to see shown only the second id but the first one is still there.
What am I doing wrong?
Thanks in advance.
Here is my settings.json
{   "appSettings": {     "stage": {       "clearColor": "0x00000000",       "useImageWorker": true     },     "debug": false   },   "platformSettings": {     "path": "./static",     "log": true,     "showVersion": true,     "showFps": false,     "numberNavigation": false,     "router": {       "destroyOnHistoryBack": true,       "lazyDestroy": false,       "gcOnUnload": true     }   } }
Here is my App.js
`import { Lightning, Utils, Router } from 'wpe-lightning-sdk'
import routes from './routes';
export default class App extends Lightning.Component {
static getFonts() {
return [{ family: 'Regular', url: Utils.asset('fonts/Roboto-Regular.ttf') }]
}
static _template() {
return {
Background: {
w: 1920,
h: 1080,
color: 0xfffbb03b,
src: Utils.asset('images/background.png'),
},
Pages: {
w: 1920,
h: 1080,
forceZIndexContext: true,
},
}
}
_captureLeft() {
Router.step(-1);
}
_setup() {
Router.startRouter(routes, this);
}
}
`
Here is my routes.js
import Home from './Home'; import Article from './Article'; export default {     root: 'home',     routes: [         {             path: 'home',             component: Home,         },         {             path: 'article/:id',             component: Article,         },     ], };
Here is my Article.js
`import {Lightning} from 'wpe-lightning-sdk';
export default class Article extends Lightning.Component {
static _template() {
return {
rect: true,
color: 0xff000000,
w: 1920,
h: 1080,
Debug: {
text: {
text: '',
},
},
};
}
set id(v) {
    this.tag('Debug').text.text = this.tag('Debug').text.text + v;
}

}
`
Here is a screenshot of the first time in the article (with url path)

Here is a screenshot of the second time in the article (with url path)

With the new sdk version 3.0.0 this problem is not longer reproducible.


[6.3.0] Friday automation testing. Please ignore

Forked from #203

Cherry-pick was attempted, but failed to push. Please check if the branch, cp203-6.3.0, exists


remove code, update README to point to uproxy

As per https://github.com/uProxy/uproxy-lib/pull/419 and https://github.com/uProxy/uproxy-networking/pull/260.

👍


Deprecate "requesting changes" (non-harbormaster) flow

Harbormaster is the future, and everyone should be using it. Let's remove some code.

See also: #120  -- this should make that one easier


Break dependencies on ITextBufferProjectService and remove the service

ITextBufferProjectService is somewhat of an odd service that's left over from the VS for Mac support. Essentially, it provided APIs to retrieve the project file path for a given ITextBuffer. However, it would also allow the user to acquire objects to pass between the APIs that were untyped, because the objects were specific to VS or VS for Mac. Since VS for Mac, is no longer present, we can just get rid of the ITextBufferProjectService abstraction and use VS-specific helper methods to perform the same function.
While doing this I made a couple of other changes:

I've added a file in the VS layer to configure the vs-threading analyzers to include our JoinableTaskContext.AssertUIThread() method in the analysis.
I've added package references to the VS shell in MS.VS.Editor.Razor. Now that this project isn't shared with VS for Mac, we can go ahead and use VS shell types there.

@dotnet/razor-compiler: You don't need to review. I suspect you were included because of the changes to Versions.props, but the changes there are just VS packages.

I suspect you were included because of the changes to Versions.props

Yup, everyone gets pinged for changes to "eng" files, since we don't really have a roslyn-infrastructure equivalent for razor.


Added a check to prevent the same parameters being queried repeatedly

This addition makes sure that the query functionallity is only run when there is a different set of parameters. It prevents users from spamming the ajax calls

Wouldn't this check only on last query? Like:
if (this.last_query === q) return;

What do you mean? This should mean that when you run the query function with the same parameters, it will return before it calls ajax. If you then run it with different parameters it would go past the `if

Yeah, I agree with you but the only parameter that has to be checked is the input query, right?
So query method would be like:
  query(q) {
    let this_ = this,
        // ....

    if (this.last_query === q) return;
    this.last_query = q;
    this.clearResults();

    // ....

Thank you @ojsdude


Update monospaced fonts used in Editor and REPL components

Update monospaced fonts used in Editor components
Summary: Changes font settings for all code editor components and the REPL input.
Changelog

Remove non-monospaced fallback Droid Sans font from all editor components

This fixes a bug where some Linux distributions fail to load the monospaced fonts and fallback incorrectly to Droid Sans instead of the system-mapped monospace font

Set the Inconsolata font used in to the SICP web edition as the default font for all editor components, REPL input, REPL output and substituter

Consolas then monospace are set as fallback fonts
Addresses https://github.com/source-academy/cadet-frontend/pull/767#issuecomment-515885192

Increase font size for the above edited components to counteract the smaller point size of the Inconsolata font

Mocks and Tests

Update snapshots for the Editor component

Screenshots
Broken non-monospaced fallback font

Before-after font comparison
TBD.
Last updated 8 Sept 2019, 11:45PM

Pull Request Test Coverage Report for Build 3435

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 36.957%

Totals

Change from base Build 3432:
0.0%

Covered Lines:
2690

Relevant Lines:
6515

💛  - Coveralls

Hi Prof @TobiasWrigstad, as an update, I've checked the web edition of SICP and it does indeed use Inconsolata as the default font for programs in the text. I have spoken to @jiayushe and we agreed that the previous style was in error.


Upgrade to wildmatch version that doesn't require Python 3.10

And instead requires Python 3.9.

https://github.com/bjd2385/wildmatch/issues/41


inject labels

Put the logic of injecting labels into span in the trace_proto.go because

The semantic of injecting labels is on the level of building spans, not implementing trace exporter.
func protoFromSpanData would be used by both func ExportSpan() and func ExportSpans() in trace.go. It would have  duplicate callings if we put the code in trace.go or cloudtrace.go.

Tested it by client.go on GCE and got non-duplicate results

Ready for final review

@YANYZP Could you please send a separate PR with unit tests for this code?

Could you please send a separate PR with unit tests for this code?

Sure, will do.


NIFI-13602 Correct Logout Supported Status for Clustered User

Summary
NIFI-13602 Corrects the behavior of the logoutSupported property in the Current User REST API Response status when NiFi is running in a cluster.
Following changes for the new user interface in NiFi 2.0.0-M4, the Current User response includes a logoutSupported property that controls the visibility of the logout link. This is necessary as the REST API determines whether the client is presenting an Application Bearer Token or an X.509 Client Certificate for authentication.
When running in a cluster, a node replicates requests for the current user to all nodes, instead of using the local response status. With request replication using X.509 Client Certificates for cluster node requests, the logoutSupported property always returned false, regardless of the browser client authentication method.
To resolve the behavior, changes include checking the local Current User status and using that value of logoutSupported instead of the replicated and merged status.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[X] Apache NiFi Jira issue created

Pull Request Tracking

[X] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[X] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[X] Pull Request based on current revision of the main branch
[X] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[X] Build completed using mvn clean install -P contrib-check

[X] JDK 21

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Will review...


typo: Pacakage -> Package

@Dansimp @AndreaBarr - PR is ready for final review. This is just a typo issue. Thanks.


added link to book  Art of Programming Contest

Is this link really a book?

yeah this link corresponds to author website of the book.This website contains a link to the book.

I took another look and still couldn't find the book. Could you please give me a direct link to it or help me find it?

Here is the link http://www.comp.nus.edu.sg/~stevenha/database/Art_of_Programming_Contest_SE_for_uva.pdf

I'm sorry but I still cannot find the book on the link you provide in this PR.
I went there 3 times now and couldn't find this PDF. I suspect most readers of this repo will not be able to find it either.


Cert manager

[ ] Install cert manager and integrate with Istio.
https://cert-manager.io/docs/

[ ] We also need to have some mechanism to monitor certificate expiration and have some way to warn devops:

[ ] HTTPS certificates
[ ] kubeadmin client certificates

this is done


Status Dashboard

https://github.com/twilio/stashboard
Looks interesting. May be useful in future.

Will be handled differently in v4.


User Story refactors

Moved System specific messages on SystemBasedUserStoryTest to extensions.
EventNotificationSystem test converted to SystemBasedUserStoryTest

Coverage increased (+0.1%) to 99.728% when pulling f9702436885b2acfdccd69f4ce284612fff2d1ef on userstory-refactors into 1cddc415f09c1a49b2573d5edcb18537cd7dcacd on master.


Bump TypeScript to 4.9.5, fix types and build

Currently universal-router breaks projects that use TypeScript 4.8 and greater. This PR fixes these issues.

@koistya thanks for letting this run, I fixed package-lock.json and it should work fine now :) Hope we can merge this soon :)

Hi, any chance for release to NPM? :) Thanks!

@mastahdocent done, thank you!


[stable/airflow] Adding a secretMap instead of adding multiple secrets

What this PR does / why we need it:
This will allow you to specify a kubernetes Secret kind which will load all it's values in the map instead of having to specify each secret individually. You can use the secrets: [] and can switch to secretsMap: when ready.
Which issue this PR fixes
N/A
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign gsemet

@gsemet @thesuperzapper @maver1ck Just the ok from you before it can merge.

ok for me ! great thanks !
/gltm

/assign @gsemet

/assign @thesuperzapper

@gsemet could you add the lgtm label again.  Unfortunately we can't make a new PR without bumping up the version so I am going to make an issue to delete that file. If there was a reason for the file let me know.

@alexbegg  It was being used for testing locally. But i'll have it removed. I'll be pushing a PR in a few moments.


Make latest tag available for use

This simplifies things in Containerfile so there is no need to edit this whenever new Fedora version gets released.
main-39 = main-latest
This one benefits from it:
https://github.com/ublue-os/startingpoint/pull/212

Maybe not needed after all, I will close this issue


Add an Example for InlineKeyboardButton.SwitchInlineQuery Property

Please add an example for InlineKeyboardButton.SwitchInlineQuery Property.
TODO (optional): Describe a specific scenario you would like to see addressed.
Help Topic: https://mrroundrobin.github.io/telegram.bot/html/P_Telegram_Bot_Types_InlineKeyboardButton_SwitchInlineQuery.htm

hi
I want when click button open contacts list and when open one of contacts, type my username bot in message field

Still no examples with switch_inline_query property?
I noticed only that the value of this property will not be sent with chat switching (in fact just a description, if I'm right).


Storing PNG string in database

Background
as part of the layout manager feature we have been taking screenshots of the layouts and we are storing them as strings which are very long. We want to investigate what the best way of handling/storing PNGs is.
example length for the PNG string is 13186 characters
Time-boxed: 1 day
Outcome

shortlist of options with pros and cons for dealing with screenshots
meeting for discussing

Questions

format? (i.e. String, Binary)
compression?
changes to DTO?
same format in client side/ local storage?

The simple fix is to add the @Lob annotation to the screenshot field on the server side, keeping it as a String for transferal and storage. This has been implemented in #57.
A new ticket has been created off the back of a discussion about possible alternative data formats for generating, transferring and storing the screenshot. It would require too much work to explore those data formats as part of this spike, but could possibly be worth exploring in the future.


Use Fb-Flo with grunt

Hi! How can i use fb-flo with grunt task like grunt-compass or grunt-uglify ?
Thanks.

there are a few grunt plugins for flo now on npm


Webview sometimes hitting invalid token error on focus

Version: 1.47.0-insider
Commit: 7923112cdd20575406ec547b55a99c289dd7d7e2
Date: 2020-06-29T05:43:37.799Z
Electron: 8.3.3
Chrome: 80.0.3987.165
Node.js: 12.13.0
V8: 8.0.426.27-electron.0
OS: Darwin x64 19.5.0
I originally brought this up in #98746 as seeming related. I've only seen it happen through the Git Graph extension, but the author had investigated in https://github.com/mhutchie/vscode-git-graph/issues/301 and felt that this was a bug in VS Code. I should note that as of a month or two ago, I can trigger this every time from a certain repo, but others do not expose this behavior. So it's possible that's it's something about the extension itself, but mhutchie's investigation seemed thorough. Wherever the issue resides, it seems likely to be a rare confluence of events that I'm just lucky enough to experience.
Steps to Reproduce:

Open a Git Graph tab, which I understand to be a Webview
Change focus to another tab
Come back to Git Graph. Contents are now gone.

As seen in the log, the first error is Invalid or unexpected token though the view contents are seemingly fine before getting back the focus.

Does this issue occur when all extensions are disabled?: I've not yet come across any way to trigger this apart from Git Graph, so no.

Developer Tools log:
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): init
log.ts:179 DEBUG WebviewResourceRequestManager(d744cc61-f95a-4326-9d46-e02a52634442): init
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): will set html
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): will update content
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): webview ready
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): will post message on 'styles'
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): will post message on 'content'
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): will post message on 'focus'
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): dom-ready
log.ts:179 DEBUG WebviewResourceRequestManager(d744cc61-f95a-4326-9d46-e02a52634442): did-start-loading
log.ts:179 DEBUG WebviewResourceRequestManager(d744cc61-f95a-4326-9d46-e02a52634442): did register
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): did post message on 'styles'
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): did post message on 'content'
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): did post message on 'focus'
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): will post message on 'focus'
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): did post message on 'focus'
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): did-set-content
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): will post message on 'message'
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): did post message on 'message'
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): will post message on 'message'
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): did post message on 'message'
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): will set html
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): will update content
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): will post message on 'content'
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): did post message on 'content'
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): will post message on 'focus'
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): did post message on 'focus'
webviewElement.ts:149 [Embedded Page] Uncaught SyntaxError: Invalid or unexpected token
webviewElement.ts:149 [Embedded Page] The Content Security Policy 'default-src 'none'; style-src vscode-webview-resource: 'unsafe-inline'; script-src 'nonce-ZQqKrJxFs0rCyI4HiJlKH9To4iGH4oho'; img-src data:;' was delivered via a <meta> element outside the document's <head>, which is disallowed. The policy has been ignored.
webviewElement.ts:149 [Embedded Page] Uncaught ReferenceError: acquireVsCodeApi is not defined
log.ts:179 DEBUG Webview(d744cc61-f95a-4326-9d46-e02a52634442): did-set-content

When this happens again, can you run: Developer: Open webview developer tools. In these dev tools, look for where the Uncaught SyntaxError: Invalid or unexpected token error is happening. This should help us track down what is going wrong

Ah, that's helpful. Thanks for letting me know about those separate Webview developer tools. So, I was able to pin down specifically what's triggering it and create a dummy repo with a single commit message Test commit with </script> tag. The error triggers as follows (JSON string truncated as indicated):
const acquireVsCodeApi = (function() {
				const originalPostMessage = window.parent.postMessage.bind(window.parent);
				const targetOrigin = '*';
				let acquired = false;

				let state = JSON.parse("{\"currentRepo\":\"/Users/jim/temp\", ...TRUNCATED... ,\"date\":1593476449,\"message\":\"Test commit with

Clearly the presence of </script> in that string is cutting off the script parsing prematurely.

Thanks for the investigation. The </script> tag definitely explains the issue


Define infrastructure and cloud deployment

Still undecided what to do here.
Vagrant?
Kubernetes or DC/OS.
Use Terraform?

Closes #5 due to sub issues


How to write into a particular cell of a csv file?

As in my business case scenario, I want to add loop on csv file and add data into last column of same csv file so that at the time of running same bot again it will start from new row which is not processed. how it can be done?

Are you familiar with JavaScript? There is Excel integration which you can read rows and range of data, and make modifications. However, some JavaScript is needed to loop through the data and process. See examples - https://tagui.readthedocs.io/en/latest/reference.html#excel


New Crowdin updates

:tada: This PR is included in version 1.11.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket: