Flag disable some debug functionality by default

Disable ctrlz, mem registry, some of pprof, and some handler write ability by default through --debug flag.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ostromart
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: vadimeisenbergibm
If they are not already assigned, you can assign the PR to them by writing /assign @vadimeisenbergibm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ostromart: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
8210875063592b021cd5b58511dd599cc1e9be60
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
8210875063592b021cd5b58511dd599cc1e9be60
link
/test istio-presubmit

prow/istio-unit-tests.sh
8e19763008402dc1bb72f074bf30c85aa688b7da
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ostromart
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
If they are not already assigned, you can assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS
tests/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ostromart: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
002173df8edd4478cb191693fc46310adbc1feb0
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
002173df8edd4478cb191693fc46310adbc1feb0
link
/test istio-unit-tests

prow/istio-presubmit.sh
002173df8edd4478cb191693fc46310adbc1feb0
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
002173df8edd4478cb191693fc46310adbc1feb0
link
/test istio-unit-tests

prow/istio-presubmit.sh
7a8bc53563201b6b8a39e5bb4121c160cf7bce46
link
/test istio-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ostromart: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
7a8bc53563201b6b8a39e5bb4121c160cf7bce46
link
/test istio-presubmit

prow/istio-unit-tests.sh
7a8bc53563201b6b8a39e5bb4121c160cf7bce46
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@sdake I've tried to be selective in what is being disabled exactly for that reason.
afaics istioctl is only using /debug/syncz and config_dump. If we find anything else in there, I think we should remove it from istioctl.

Moved to https://github.com/istio/istio/pull/11543


The missing file aug_eq_model

As shown in the title, the file aug_eq_model.py is missing. Would you like to upload it? Thanks

Yeah what did happen to that file, any ideas @g-benton?

I have an old version
import torch
import torch.nn as nn
import torch.nn.functional as F
from augerino.utils import expm

class DiffEqAug(nn.Module):
    """
    Differetiable rotations and translations
    """
    def __init__(self, padding=0):
        super().__init__()
        self.Sigma = nn.Parameter(torch.eye(3))
        self.Mu = nn.Parameter(torch.zeros(3))
        self.aug = True
        self.pad = nn.ConstantPad2d(padding, 0.)
        self.crop = lambda x: x[:, :, padding:-padding, padding:-padding]

    def sample_z(self, x):
        bs, _, w, h = x.size()
        z = torch.randn(bs,3,device=x.device,dtype=x.dtype) @ self.Sigma.T + self.Mu
        self.z = z

    @staticmethod
    def transform(img, z):
        bs, c, h, w = img.size()
        # Build affine matrices for random translation of each image
        affineMatrices = torch.zeros(bs, 2, 3, device=img.device)
        affineMatrices[:,0,0] = z[:,2].cos()
        affineMatrices[:,0,1] = -z[:,2].sin()
        affineMatrices[:,1,0] = z[:,2].sin()
        affineMatrices[:,1,1] = z[:,2].cos()
        affineMatrices[:,:2,2] = z[:,:2]/(.5*w+.5*h)
        flowgrid = F.affine_grid(affineMatrices, size=img.size(), align_corners=True)
        return F.grid_sample(img, flowgrid, align_corners=True)

    def forward(self, x):
        x = self.pad(x)
        self.sample_z(x)
        return self.transform(x, self.z)

    def inverse(self, x):
        x = self.transform(x, -self.z)
        return self.crop(x)

class UniformEqAug(nn.Module):
    """
    Differetiable rotations and translations.
    """
    def __init__(self, gen_scale=10., trans_scale=0.1, padding=0):
        super(UniformEqAug, self).__init__()

        self.trans_scale = trans_scale

        self.lower = nn.Parameter(torch.zeros(6))
        self.upper = nn.Parameter(torch.ones(6))
        self.g0 = None
        self.pad = nn.ConstantPad2d(padding, 0.)
        self.crop = lambda x: x[:, :, padding:-padding, padding:-padding]
        self.weights = None

    def sample_weights(self, x):
        bs, _, w, h = x.size()
        weights = torch.rand(bs, 6)
        weights = weights.to(x.device, x.dtype)
        self.weights = weights * (self.upper - self.lower) + self.lower

    def transform(self, x, weights):
        generators = self.generate(weights)

        ## exponential map
        affine_matrices = expm(generators)
        flowgrid = F.affine_grid(affine_matrices[:, :2, :], size = x.size(),
                                 align_corners=True)
        x_out = F.grid_sample(x, flowgrid,align_corners=True)

        return x_out

    def forward(self, x):
        x = self.pad(x)
        self.sample_weights(x)
        return self.transform(x, self.weights)

    def inverse(self, x):
        x = self.transform(x, -self.weights)
        return self.crop(x)

    def generate(self, weights):
        """
        return the sum of the scaled generator matrices
        """
        bs = weights.shape[0]

        if self.g0 is None:
            ## tx
            self.g0 = torch.zeros(3, 3, device=weights.device)
            self.g0[0, 2] = 1. * self.trans_scale

            ## ty
            self.g1 = torch.zeros(3, 3, device=weights.device)
            self.g1[1, 2] = 1. * self.trans_scale

            self.g2 = torch.zeros(3, 3, device=weights.device)
            self.g2[0, 1] = -1.
            self.g2[1, 0] = 1.

            self.g3 = torch.zeros(3, 3, device=weights.device)
            self.g3[0, 0] = 1.
            self.g3[1, 1] = 1.

            self.g4 = torch.zeros(3, 3, device=weights.device)
            self.g4[0, 0] = 1.
            self.g4[1, 1] = -1.

            self.g5 = torch.zeros(3, 3, device=weights.device)
            self.g5[0, 1] = 1.
            self.g5[1, 0] = 1.

        out_mat = weights[:, 0] * self.g0.unsqueeze(-1).expand(3,3, bs)
        out_mat += weights[:, 1] * self.g1.unsqueeze(-1).expand(3,3, bs)
        out_mat += weights[:, 2] * self.g2.unsqueeze(-1).expand(3,3, bs)
        out_mat += weights[:, 3] * self.g3.unsqueeze(-1).expand(3,3, bs)
        out_mat += weights[:, 4] * self.g4.unsqueeze(-1).expand(3,3, bs)
        out_mat += weights[:, 5] * self.g5.unsqueeze(-1).expand(3,3, bs)

        # transposes just to get everything right
        return out_mat.transpose(0, 2).transpose(2, 1)

class AugEqModel(nn.Module):
    def __init__(self,model, aug, traincopies=1, testcopies=8):
        super().__init__()
        self.aug = aug
        self.model = model
        self.traincopies = traincopies
        self.testcopies = testcopies

    def forward(self,x):
        # ToDo: go to log softmax
        copies = self.traincopies if self.training else self.testcopies
        outputs = []
        for i in range(copies):
            pred = self.aug(x)
            pred = self.model(pred)
            pred = self.aug.inverse(pred)
            outputs.append(pred)
        return sum(outputs) / copies

I think its in models/aug_modules.py now, but we should fix the broken import.

Thanks for your clarification.


Handle filename mismatch when hash is known

Since #63 there is a risk to serve a wrong filename as Content-Disposition, because someone else uploaded the same file but under a different filename.
Find a way to avoid that.

634a00becec1f4db6a6fe5f660502c66c128f873


App will not launch on iPhone simulator due to errors with iOS app files

Steps to Reproduce

Run flutter create bug.
Update the files as follows: ...
I zipped the bug folder and tried to upload it but Github said it exceeded 10MB so not sure what to do.
...
Run the Flutter app on an iPhone simulator or iPhone device. I'm attempting to launch it on an iPhone simulator on a Mac and it's giving me errors and will not launch. The issue is not with the simulator because the simulator is working with a the Flutter default test app.

Expected results:
The app launching and functioning on the iPhone simulator.
Actual results:
I saw errors: Please take a look at the "errors" file I attached. I didn't paste it in because it's too long and it would exceed the character limit of this textbox.
errors.txt

Logs
_No logs appeared when I ran the app._

For flutter run --verbose please check the "run_verbose" file I attached. I didn't paste it in because it's too long and it would exceed the character limit of this textbox.
run_verbose.txt

Analyzing AutoSentinelApp...

   info • Unused import: 'dart:ui' • lib/detector_painters.dart:1:8 • unused_import
   info • Unused import: 'dart:math' • lib/detector_painters.dart:2:8 • unused_import
   info • Unused import: 'colors.dart' • lib/drowsy.dart:3:8 • unused_import
   info • Unused import: 'dart:ui' • lib/face_detection_camera.dart:10:8 • unused_import
   info • The value of the field '_isStart' isn't used • lib/face_detection_camera.dart:70:8 • unused_field
   info • The declaration '_resetButtonPressed' isn't referenced • lib/face_detection_camera.dart:101:8 • unused_element
   info • Unused import: 'signin.dart' • lib/home.dart:6:8 • unused_import
   info • Unused import: 'drowsy.dart' • lib/home.dart:11:8 • unused_import
   info • Unused import: 'package:flutter_session/flutter_session.dart' • lib/home.dart:13:8 • unused_import
   info • Unused import: 'home.dart' • lib/intro_page.dart:4:8 • unused_import
   info • Unused import: 'package:flutter_local_notifications/flutter_local_notifications.dart' • lib/localnotification.dart:2:8 • unused_import
   info • Name types using UpperCamelCase • lib/localnotification.dart:4:7 • camel_case_types
   info • Name types using UpperCamelCase • lib/localnotification.dart:9:7 • camel_case_types
   info • Unused import: 'package:flutter_local_notifications/flutter_local_notifications.dart' • lib/main.dart:8:8 • unused_import
warning • The parameter 'email' is required • lib/main.dart:35:39 • missing_required_param
warning • The parameter 'email' is required • lib/main.dart:48:44 • missing_required_param
warning • The parameter 'email' is required • lib/main.dart:82:62 • missing_required_param
   info • Unused import: 'signin.dart' • lib/new.dart:5:8 • unused_import
   info • Unused import: 'package:geolocator/geolocator.dart' • lib/new.dart:6:8 • unused_import
   info • Unused import: 'package:url_launcher/url_launcher.dart' • lib/new.dart:9:8 • unused_import
   info • Unused import: 'drowsy.dart' • lib/new.dart:10:8 • unused_import
   info • Unused import: './settings.dart' • lib/settings/tile.dart:3:8 • unused_import
warning • The parameter 'email' is required • lib/sidemenu.dart:74:67 • missing_required_param
   info • Unused import: 'package:http/http.dart' • lib/signin.dart:2:8 • unused_import
   info • Unused import: 'dart:convert' • lib/signin.dart:3:8 • unused_import
   info • Unused import: 'componets/socal_card.dart' • lib/signin.dart:8:8 • unused_import
   info • Unused import: 'package:flutter_session/flutter_session.dart' • lib/signin.dart:12:8 • unused_import
   info • Unused import: 'dart:convert' • lib/signup.dart:1:8 • unused_import
warning • The parameter 'email' is required • lib/signup.dart:78:51 • missing_required_param
warning • The parameter 'email' is required • lib/signup.dart:213:69 • missing_required_param
   info • This function has a return type of 'String', but doesn't end with a return statement • lib/up.dart:109:38 • missing_return
   info • This function has a return type of 'String', but doesn't end with a return statement • lib/up.dart:134:38 • missing_return
   info • This function has a return type of 'String', but doesn't end with a return statement • lib/up.dart:159:38 • missing_return
   info • This function has a return type of 'String', but doesn't end with a return statement • lib/up.dart:212:38 • missing_return
warning • The parameter 'email' is required • lib/up.dart:270:63 • missing_required_param

35 issues found. (ran in 4.6s)

[✓] Flutter (Channel stable, 1.22.6, on macOS 11.1 20C69 darwin-x64, locale en-US)
    • Flutter version 1.22.6 at /Users/ismile2/Documents/flutter
    • Framework revision 9b2d32b605 (4 weeks ago), 2021-01-22 14:36:39 -0800
    • Engine revision 2f0af37152
    • Dart version 2.10.5

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/ismile2/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] Xcode - develop for iOS and macOS (Xcode 12.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.1

[!] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] Connected device (1 available)
    • iPhone 12 Pro Max (mobile) • 4F851663-3368-498A-AF31-1D0EA2FEA172 • ios • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)

! Doctor found issues in 2 categories.

@ahmedjaafar6
from your logs seems that the error is caused by audioplayers
which is a 3rd party plugin, please open an issue in the dedicated github
or provide your flutter doctor -v, your flutter run -d chrome -v
and a minimal reproducible code sample without 3rd party plugins
**Launching lib/main.dart on iPhone 12 Pro Max in debug mode...
Running pod install...
Running Xcode build...
Xcode build done.                                           64.4s
Failed to build iOS app
Error output from Xcode build:
↳
    2021-02-22 10:42:46.257 xcodebuild[4098:9838698] warning:  The file reference for "../../../../../../../../../flutter/.pub-cache/hosted/pub.dartlang.org/audioplayers-0.16.2/darwin/Classes/AudioplayersPlugin.m" is a member of multiple groups ("Classes" and "Classes"); this indicates a malformed project.  Only the membership in one of the groups will be preserved (but membership in targets will be unaffected).  If you want a reference to the same file in more than one group, please add another reference to the same path.
    2021-02-22 10:42:46.273 xcodebuild[4098:9838698] warning:  The file reference for "Firebase.debug.xcconfig" is a member of multiple groups ("Support Files" and "Support Files"); this indicates a malformed project.  Only the membership in one of the groups will be preserved (but membership in targets will be unaffected).  If you want a reference to the same file in more than one group, please add another reference to the same path.
    2021-02-22 10:42:46.274 xcodebuild[4098:9838698] warning:  The file reference for "Firebase.release.xcconfig" is a member of multiple groups ("Support Files" and "Support Files"); this indicates a malformed project.  Only the membership in one of the groups will be preserved (but membership in targets will be unaffected).  If you want a reference to the same file in more than one group, please add another reference to the same path.
    2021-02-22 10:42:46.280 xcodebuild[4098:9838698] warning:  The file reference for "GoogleUtilities.debug.xcconfig" is a member of multiple groups ("Support Files" and "Support Files"); this indicates a malformed project.  Only the membership in one of the groups will be preserved (but membership in targets will be unaffected).  If you want a reference to the same file in more than one group, please add another reference to the same path.
    2021-02-22 10:42:46.280 xcodebuild[4098:9838698] warning:  The file reference for "GoogleUtilities.release.xcconfig" is a member of multiple groups ("Support Files" and "Support Files"); this indicates a malformed project.  Only the membership in one of the groups will be preserved (but membership in targets will be unaffected).  If you want a reference to the same file in more than one group, please add another reference to the same path.
    ** BUILD FAILED **

Hi @ahmedjaafar6
First of all, change your Target iOS to 9
This looks like an issue with Cocoapods, please try the following

flutter clean
delete Podfile rm iOS/Podfile
Execute flutter pub get
flutter run

If the problem persists, please provide your flutter doctor -v, your flutter run -v,  your pubspec.yaml
a minimal reproducible code sample that does not use 3rd party plugins,
and I will reopen it.
Thank you

Hi, thank you both for replying.
@iapicca I don't think the audioplayer is the issue because I tried removing it from my pubspec.yaml and doing flutter pub get and the problem persists.
@TahaTesser Where do I change the target? In the podfile I uncommented the line at the top that says "platform :ios, '9.0'". and then followed your other steps but the problem persists. Please take a look at the attached image

Here are the things you guys asked for:
Pubspec.yaml:
pubspec.yaml.zip
flutter doctor -v:
[✓] Flutter (Channel stable, 2.0.1, on macOS 11.1 20C69 darwin-x64, locale
    en-US)
    • Flutter version 2.0.1 at /Users/ismile2/Documents/flutter
    • Framework revision c5a4b4029c (22 hours ago), 2021-03-04 09:47:48 -0800
    • Engine revision 40441def69
    • Dart version 2.12.0

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/ismile2/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.3, Build version 12C33
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[✓] Connected device (2 available)
    • iPhone 12 Pro Max (mobile) • 4F851663-3368-498A-AF31-1D0EA2FEA172 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-14-3 (simulator)
    • Chrome (web)               • chrome                               •
      web-javascript • Google Chrome 88.0.4324.192

• No issues found!

flutter run -v:
It's too long so I put it in a txt file
flutterdoctorv.txt
flutter run -d chrome -v:
It's too long so I put it in a txt file
flutterrundchromev.txt
I do agree with @TahaTesser that it might be an issue with the CocoaPods and the target iOS because that's what I read in some of the errors. With that being said, I don't see how sending in the entire app will help, or even a minimal reproducible code, especially because I'm not even sure what to cut out of it to make it minimally reproducible. The best thing I think I can send is the "ios" folder of the app since that has the Pod stuff and the target iOS stuff which seem to be the main issue, and I guess that could even maybe count as "minimally reproducible". Here it is:
ios.zip
Please let me know if you guys need anything else specific from me and I'll try to provide. Thank you again!

@ahmedjaafar6
from your pubspec
seems that there are plenty of 3rd party plugins
dependencies:
  flutter:
    sdk: flutter
  image_picker: ^0.6.7+21
  camera: ^0.7.0
#  audioplayers: ^0.16.2
  flutter_svg: ^0.19.1
  http: ^0.12.2
  rflutter_alert: ^1.1.0
  geolocator: ^5.3.1
  geocoder: ^0.2.1
  weather: ^1.2.4
  url_launcher: ^5.7.10
  fluttertoast: ^7.1.6
  shared_preferences: ^0.5.12+4
  flutter_session: ^0.1.1
  alt_http: ^0.3.0
  flutter_local_notifications: ^3.0.3
  #changed the two below to lower version
  firebase_core: 0.3.1+1
  firebase_ml_vision: ^0.6.0+1

as mentioned above

[...] please provide your flutter doctor -v, your flutter run -v, your pubspec.yaml
a minimal reproducible code sample that does not use 3rd party plugins,
and I will reopen it.
Thank you

Hi @ahmedjaafar6
Make sure you also target iOS 9 in Runner>General
You wanna make sure none of the packages causing the issue and isolate the cause, if you want some packages causes the problem, please file issue on the package's repository
For issues relate to Cocoapods, please file an issue on their dedicated GitHub repository
If you can reproduce the issue without third-party packages, please provude a minimal reproducible code sample that does not use 3rd party plugins (not just ios folder)
Thank you

Hi,
@TahaTesser I didn't find the "General" folder inside Runner. Here's all there is:

@TahaTesser @iapicca I'm fine with opening an issue with Cocoapods and/or the third party plugins but the thing is is that I don't even know where exactly the problem is stemming from and what it is for me to open an issue with them. Sometimes from the errors it seems like it's Cocoapods and sometimes it seems like it's Firebase related and sometimes other things, so it's total chaos. I'm just trying to get a sense of direction. Like I mentioned last time, I have no idea what to cut out to make it minimally reproducible, since I'm not sure where the problem is stemming from and since the problem might be linked with the third party plugins (Again, I'm not sure if it actually is and which one exactly) removing the third part plugins might not lead us to a solution. If you can give me more specific details and help me narrow it down that way even if it is something that's third party, I would at least know how to approach it and who exactly to open an issue with, that would be appreciated.
Thank You

Hi @ahmedjaafar6

@TahaTesser I didn't find the "General" folder inside Runner. Here's all there is:

It's inside Xcode
It seems you are asking for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this, you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


Add a Changelog.md file, and fill GitHub releases

This is needed to know what has changed from release 0.1 and 0.2, and upcoming releases.
Thank you!

To express changes per version, we've so far relied on git features like tags, rather than GitHub-specific features like Releases. At the moment, the releases page will show the changes for each version if you reveal the tag comment ("…"). (I should add details to 0.1's comment; it's empty at the moment.)
Upcoming versions would be a good reason to use Releases, but in this project's case, upcoming versions are not planned so much as cut when we're satisfied with the state of develop.
But of course I'm not visiting this project as a newcomer. Can you say more about the benefit you would get from our maintaining a changelog someplace other than in tags?

The changelog files is GitHub independent and allows to glance at changes,
and makes them searchable.
On Mon, Aug 12, 2019, 15:25 Kevin Conner<EMAIL_ADDRESS>wrote:

To express changes per version, we've so far relied on git features like
tags, rather than GitHub-specific features like Releases. At the moment,
the releases page
https://github.com/willowtreeapps/sign-in-with-apple-button-android/releases
will show the changes for each version if you reveal the tag comment ("…").
(I should add details to 0.1's comment; it's empty at the moment.)
Upcoming versions would be a good reason to use Releases, but in this
project's case, upcoming versions are not planned so much as cut when we're
satisfied with the state of develop.
But of course I'm not visiting this project as a newcomer. Can you say
more about the benefit you would get from our maintaining a changelog
someplace other than in tags?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/willowtreeapps/sign-in-with-apple-button-android/issues/47?email_source=notifications&email_token=ABVG6BJGV7BBZ35BHKAE4RTQEFQDLA5CNFSM4IK7GY2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4CQPZY#issuecomment-520423399,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABVG6BNNYPY2OCAYKRSV7HTQEFQDLANCNFSM4IK7GY2A
.

I do like ease for the reader, as in glanceability. OK, I think we'll add that file.

Am a fan of https://keepachangelog.com/en/1.0.0/, using that format on assertk


merge from template-node-app

package.json

chore(scripts): add ava --timeout 4m tests/-ava.mjs tests/-ava-node.mjs && ava --timeout 2m tests/-ava.mjs tests/-ava-node.mjs (scripts.test:ava)
chore(scripts): add c8 -x 'tests/**/' --temp-directory build/tmp ava --timeout 4m tests/-ava.mjs tests/*-ava-node.mjs && c8 report -r lcov -o build/coverage --temp-directory build/tmp (scripts.cover)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 58.883%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
109

Relevant Lines:
182

💛  - Coveralls


Backend endpoint to create a room

The room will be used for the Planning Poker game

The room entity will be like as follows.
Entity Name : Room

RoomId - GUID - PK
RoomName - String(150) - Name of the room
CreatedBy - GUID
IsActive

Should we consider adding a CreatedAt or CreatedDate field, @mahedee bhaiya?

Should we consider adding a CreatedAt or CreatedDate field, @mahedee bhaiya?

Yes I can. But, can you please tell me what would be the purpose of that field @MiliDas

The room will be used for the Planning Poker game

Do you mean the entity name will be Game instead of Room? @ratanparai

I believe that since these rooms are used briefly and only once, it is necessary to clean them periodically. By incorporating a CreatedAt or CreatedDate field, we can efficiently remove older data when needed. @mahedee bhaiya

I believe that since these rooms are used briefly and only once, it is necessary to clean them periodically. By incorporating a CreatedAt or CreatedDate field, we can efficiently remove older data when needed. @mahedee bhaiya

Ok. I think, room will not be that much data to archive but yes, it can be used for sorting. Like if a user create multiple room now he want to order it by date. Then it is required. Ok. I am going to add this field. Thanks for the suggestion.


[material-ui][Autocomplete] Fix error in Google Maps demo

When using the demo code for autocomplete with Google Maps, there is an error thrown which seems to be discussed in other issues with respect to other broken examples, but I don't see a solution in place yet.
This addresses the Google Maps example only.  There are probably other examples that should be updated too, but getting this merged seems like a positive step forward even if there are other positive steps forward that can follow.
Examples of issues closed as incomplete without a solution that seem to be reporting a similar error, especially with NextJS and React 18.3+:

https://github.com/mui/material-ui/issues/41486
https://github.com/mui/material-ui/issues/41035
https://github.com/mui/material-ui/issues/40905
https://github.com/mui/material-ui/issues/39966
https://github.com/mui/material-ui/issues/39474
https://github.com/mui/material-ui/issues/38272

This is still open:

https://github.com/mui/material-ui/issues/39833

The issue appears to have been partially addressed in

https://github.com/mui/material-ui/pull/39795
https://github.com/mui/material-ui/pull/40968

but this particular issue is still in the latest, so here's a step to fix it.
This borrows from https://github.com/mui/material-ui/issues/39833#issuecomment-2164181730 in place of a slightly less elegant earlier draft.

It looks like I can't add a label on the PR as is required by test, and the GitHub editor doesn't apply prettier settings.

It looks like that requires setting up a special billing account, getting an API key, and possibly breaking pseudonymity in open-source contributions more broadly just to re-demonstrate an issue that has several existing reports already.  The existing example code does not need to be modified to demonstrate the issue; it appears that using it as-is in a project with React 18.3+ and your own API key would likely suffice to show it, assuming that NONE of the other issue reports appear credible.
For quick setup close to the context where I saw it, I recommend running npx create-next-app@latest in an empty directory, with default options.  Add the original and/or modified file affected by this PR into /components, paste in your API key on line 13, and then import/add it onto the default page.tsx that you get with the new Next app. Make sure your key allows requests from http://localhost:3000 or has no referrer restriction.

The demo in https://mui.com/material-ui/react-autocomplete/#globally-customized-options also has the same problem with the key prop. I was playing with passing a custom renderOption to Autocomplete and saw the error.

I saw other demos were updated in #42691. Maybe we need to update this one and this one as well? I'll leave this PR to be reviewed by @aarongarciah and @DiegoAndai since they have more context.

We'll hold this PR until https://github.com/mui/material-ui/pull/42689 is merged so we get the correct types.
Note that React 18.3.0 started to throw warnings when spreading key. It's not Material UI specific (see https://github.com/facebook/react/releases/tag/v18.3.0). We're fixing the issues as we find them.
This error, and the one in GloballyCustomizedOptions.tsx mentioned by @m4theushw, were harder to catch than the ones fixed in https://github.com/mui/material-ui/pull/42691 because these warnings only happen at runtime and the Google Maps demo didn't throw the warning until there are results, which is not the case if you don't have an API key 😅
I see that there are more warnings that we didn't catch because they are fired when the autocomplete is open and renderOption is called. I'll push the fixes to this PR so we merge them all at once.

@wbt when a demo is modified (the .tsx file), you need to run pnpm docs:typescript to generate the corresponding .js file.

Just pushed a commit that should solve all React warnings related to spreading key in Autocomplete demos.
I recommend reviewing this PR with the "Hide whitespace" option enabled.

See how to enable it

I just rebased to get the latest typing fixes.


VS2017: completion icon issue for items in non opened namespaces

When listing items which are not in scope, it seems the icon used (and impacts filtering as well) doesn't follow the type of symbols, at least for symbols defined in C# assembly:

ClassLibrary1.zip

Fixed in https://github.com/Microsoft/visualfsharp/pull/2896

Thanks!

@cartermp Can we close it now?


Shape viewer outputs 404 error on inputs the game can accept.

The game will accept this: https://community-vortex.shapez2.com/shape?identifier=P-P---P-%3Acrcr--P-%3AP-cr--P-%3AP-cr--P-
Another example: https://shapez.soren.codes/shape?identifier=P-------
Both shapes are accepted by the Signal Producer, are displayed by the Display, and can be assembled (but that's irrelevant).
Is this a bug or some kind of validity check?   If the game accepts the code then the viewer should accept it.

Thank you for reaching out!
This is definetly an validity check issue on my end. The game is always right :+1:

@Avernar
I found the issue: The problem was caused by the fact that the viewer supports the short-hand version of shapes like P-. This allows a shape identifier to be a valid quad or hex shape identifier, which was determined "valid" before.


[TILT-2] Tilt::Template#render instrumentation

Overview
This instrumentation creates new metrics with the style of: View/#{class_name}/#{file_path}/Rendering
An example metric would look like View/Tilt::HamlTemplate/blog/layout.html/Rendering
Traces and spans related to view rendering within Tilt should be nested within the action that causes the template to render.
If a template is rendered within the block of another template rendering call, it will appear as nested within the outer template.
Example
require 'tilt'
require 'haml'

def index
  Tilt.new('layout.haml').render {
    Tilt.new('body.haml')
  }
end

Should result in a nested span that looks like this:

Branch Name/Background
This project began with the intention of instrumenting Tilt::Template#prepare, to discover this is a protected method. I decided to pivot to instrument #initialize instead, for the time being.
With initial testing in the UI, I was uncertain whether anything of additional value will be captured through #initialize that cannot be captured using #render.
After talking with @tannalynn, we decided to opt-out of instrumenting #initialize and instrument #render instead so that it is on-par with our ActionView instrumentation.
Related Github Issue
Closes #836
Closes #835
Testing
Multiverse tests added to verify functionality for prepend, chain, disabled recording of metrics
Tested with Rails calling a Haml view, and did not see a duplicated metric for Tilt, just the existing ActionView metric.
Reviewer Checklist

[ ] Perform code review
[ ] Add performance label
[ ] Perform appropriate level of performance testing
[ ] Confirm all checks passed
[ ] Add version label prior to acceptance

Oh also, we should the changelog entry for this! Almost forgot about that.


kube-state-metrics cant get node label

kube-state-metrics cant get node label
when i use kube-prometheus, i cant get node label from metrics: kube_node_labels{job="kube-state-metrics"}
imgae
image: k8s.gcr.io/kube-state-metrics/kube-state-metrics:v2.6.0

args
args:
  - --host=<IP_ADDRESS>
  - --port=8081
  - --telemetry-host=<IP_ADDRESS>
  - --telemetry-port=8082

clusterRole
...
rules:
- apiGroups:
  - ""
  resources:
  - configmaps
  - secrets
  - nodes
  - pods
  - services
  - serviceaccounts
  - resourcequotas
  - replicationcontrollers
  - limitranges
  - persistentvolumeclaims
  - persistentvolumes
  - namespaces
  - endpoints
  verbs:
  - list
  - watch
...

prometheus sql
kube_node_labels{job="kube-state-metrics"}

i got
...
kube_node_labels{node="ip-10-162-123-166.ap-northeast-1.compute.internal"} 1
...

I also encountered the same problem. I could not get indicators such as label_alpha_eksctl_io_cluster_name. However, when I looked at the label of node in the cluster, all of them were available. For node, it seems that only node IP, instance, job, and container information can be obtained

The current version is 0.11, which uses the default configuration of the project and only changes the node selector in namespace and env.
Does anyone know the reason for this?   I've been doing some research, kubernetes mentions Flags --metric-blacklist and --metric-whitelist were renamed to --metric-denylist and --metric-allowlist.--metric-labels-allowlist allows you to specify a list of Kubernetes labels that get turned into the dimensions of  the kube_<   resource-name>   _labels metrics.    By default, the metric contains only name and namespace labels.
i have try the setting, but kube_node_labels still has the same label.
Pls help

according to  - https://github.com/kubernetes/kube-state-metrics/issues/1716
this is the correct value which seem to work
  kube-state-metrics.extraArgs:
   - --metric-labels-allowlist=nodes=[*]


apt-get install scrcpy always installs scrcpy 1.12.1 and not the latest

How can I install the latest on ubuntu?

Thanks for that.
HTTP request sent, awaiting response... 302 Found
Location: https://objects.githubusercontent.com/github-production-release-asset-2e65be/111583593/fd21c402-c927-425e-bd21-448e449a147d?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20220309%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20220309T125407Z&X-Amz-Expires=300&X-Amz-Signature=97cc2255ccc8f66620b6470b81580f2a533bf09360ac3386a06e7ec1fc491aa0&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=111583593&response-content-disposition=attachment%3B%20filename%3Dscrcpy-server-v1.23&response-content-type=application%2Foctet-stream [following]
--2022-03-09 12:54:08--  https://objects.githubusercontent.com/github-production-release-asset-2e65be/111583593/fd21c402-c927-425e-bd21-448e449a147d?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20220309%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20220309T125407Z&X-Amz-Expires=300&X-Amz-Signature=97cc2255ccc8f66620b6470b81580f2a533bf09360ac3386a06e7ec1fc491aa0&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=111583593&response-content-disposition=attachment%3B%20filename%3Dscrcpy-server-v1.23&response-content-type=application%2Foctet-stream
Resolving objects.githubusercontent.com (objects.githubusercontent.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to objects.githubusercontent.com (objects.githubusercontent.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 41123 (40K) [application/octet-stream]
Saving to: ‘scrcpy-server’

scrcpy-server                                        100%[======================================================================================================================>]  40.16K  --.-KB/s    in 0.08s

2022-03-09 12:54:08 (515 KB/s) - ‘scrcpy-server’ saved [41123/41123]

[scrcpy] Verifying prebuilt server...
scrcpy-server: OK
[scrcpy] Building client...
The Meson build system
Version: 0.53.2
Source dir: /home/m/scrcpy
Build dir: /home/m/scrcpy/build-auto
Build type: native build
Project name: scrcpy
Project version: 1.23
C compiler for the host machine: cc (gcc 9.4.0 "cc (Ubuntu 9.4.0-1ubuntu1~20.04) 9.4.0")
C linker for the host machine: cc ld.bfd 2.34
Host machine cpu family: x86_64
Host machine cpu: x86_64
Found pkg-config: /usr/bin/pkg-config (0.29.1)
Run-time dependency libavformat found: YES 58.29.100
Run-time dependency libavcodec found: YES 58.54.100
Run-time dependency libavutil found: YES 56.31.100
Run-time dependency sdl2 found: YES 2.0.10
Run-time dependency libavdevice found: YES 58.8.100
Run-time dependency libusb-1.0 found: YES 1.0.23
Checking for function "strdup" : YES
Checking for function "asprintf" : YES
Checking for function "vasprintf" : YES
Header <sys/socket.h> has symbol "SOCK_CLOEXEC" : YES
Configuring config.h using configuration
Build targets in project: 3

Found ninja-1.10.0 at /usr/bin/ninja
[54/54] Linking target app/scrcpy.
[scrcpy] Installing (sudo)...
[sudo] password for m:
[0/1] Installing files.
Installing app/scrcpy to /usr/local/bin
Stripping target 'app/scrcpy' using strip.
Installing server/scrcpy-server to /usr/local/share/scrcpy
Installing /home/m/scrcpy/app/scrcpy.1 to /usr/local/share/man/man1
Installing /home/m/scrcpy/app/data/icon.png to /usr/local/share/icons/hicolor/256x256/apps
Installing /home/m/scrcpy/app/data/zsh-completion/_scrcpy to /usr/local/share/zsh/site-functions
Installing /home/m/scrcpy/app/data/bash-completion/scrcpy to /usr/local/share/bash-completion/completions

m:~/scrcpy$ scrcpy
bash: /usr/bin/scrcpy: No such file or directory
m:~/scrcpy$ cd
m:~$ scrcpy
bash: /usr/bin/scrcpy: No such file or directory
m:~$

Is /usr/local/bin in your PATH? (echo $PATH)

Yes it is
m:~$ $PATH
bash: /home/m/anaconda3/condabin:/home/m/Android/Sdk/platform-tools:/home/m/.local/bin:/usr/local/sbin**:/usr/local/bin**:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin: No such file or directory

What is the result of type scrcpy? (did you set an alias?)
Did you install the scrcpy snap package? (if so, uninstall it)
What if you call /usr/local/bin/scrcpy?

It looks like there was no alias for it so after I added an alias of "scrcpy" for /usr/local/bin/scrcpy then it worked.
thanks so much for you help


Feature request: allow add, files_write, block_put, … to take AsyncRead or similar

The title actually says it all, but:
Various getters return asynchronous streams (async_ftp::FtpSream::retr in my case, but so does e.g. cat — seems unfair to return something the library itself doesn't accept), and those cannot be easily passed to functions like add. Buffering in memory or temporary files may be expensive or even infeasible, depending on the size.
So it would be nice if some form of async stream could be passed to add and friends.
(This requires changes to rust-multipart-rfc7578, I think. I hope not to hyper, too?)

I currently buffer big video files in my app. I would definitely appreciate more asynchrony.
+1 from me!

https://github.com/cetra3/mpart-async/
Would it be useful? I'm not familiar with that kind of stuff.


Android Chrome Toasts don't go away after repeatedly launching

Reported by vince#8203 on the Svelte Discord. Cannot currently reproduce, requested details. Bizarre because number of toasts is largely independent of toast removal logic.

Not able to reproduce


Feedback component broken

If you use the feedback component with custom content, it throws a react hydration error. Simply try to set a custom content to the feedback in theme.config.tsx and the error should show.
If not reproducible, please let me know, I'll show more details. I think another solution would just be allowing us to update the text on the feedback button.

sharing the workaround:
  feedback: {
    content: (
      <button
        onClick={() => {
          const win = window.open(
            "https://forms.gle/9JWjWtjbWytZZcNRA",
            "_blank",
            "noopener,noreferrer"
          );
          if (win) win.opener = null;
        }}
      >
        Leave feedback ↗
      </button>
    ),
  },


IndexError: list index out of range

Hello, I have found this issue, when I tried to see the email arrived clicking the view message button, I just got the IndexError: list index out of range error, would you be so kind to take a look ?
thanks in advance.

chknet is work؟

This repository is for Tempmail bot dude, so you are out of topic. Search some info about chknet, and thats it.

Hello, I have found this issue, when I tried to see the email arrived clicking the view message button, I just got the IndexError: list index out of range error, would you be so kind to take a look ?
thanks in advance.

Fixed view message button


Realm SDK Native crash (SIGSEGV, librealmc.so, invoke_core_notify_callback)

How frequently does the bug occur?
Sometimes
Description
We have recently migrated some of our local databases from the java SDK to the kotlin SDK, we have noted a native crash on our Crashlytics reports and it points to the realm kotlin SDK, at least from what I can tell. Find the stacktrace from Crashlytics and logcat below.
Note that we do use atlas sync service in our app but that database is not migrated to kotlin SDK yet and since the trace points to kotlin then I'm assuming this is related to one of the kotlin SDK databases.

No
Reproduction Steps
We are working on trying to come up with repro steps.
Version
1.12.0
What Atlas App Services are you using?
Local Database only
Are you using encryption?
No
Platform OS and version(s)
Android 10
Build environment
Android Studio version: Android Studio Iguana | 2023.2.1 Canary 10
Android Build Tools version: 33.0.2
Gradle version: 7.6.2

This looks like a duplicate of https://github.com/realm/realm-kotlin/issues/1543, which is fixed in 1.13.0. I will close as a duplicate for now. Please get back if you observe anything similar with newer releases.

This issue doesn't look like #1543, as I mentioned in the description my issue doesn't relate to atlas sync (at least from the logs we were able to gather), #1543 is directly related to it. Do you mind taking another look? 🙏 @rorbech

Hi @jhonnyinstacart. The referenced issue is not specifically related to sync. It was just appearing in our sync integration tests due to timing of these use cases.
The referenced issue a "crash when notifying on a released scheduler" which is exactly what your the invoke_core_notify_callback entry from your stack trace indicates.


RequestAuthorizationCodeAsync just hangs in beta5

Hi,
I'm calling RequestAuthorizationCodeAsync like so:
tokenResponse = await client.RequestAuthorizationCodeAsync(
                    tokenExchangeInput.Code,
                    tokenExchangeInput.RedirectUri);

I don't get any errors or anything. It just seems to hang. Any help is appreciated.
-J

It seems like this has something to do with the HttpClientHandler UseProxy option. When I set UseProxy = false it appears to fix the issue.
-J


[stable/traefik] Refactor annotations for yaml and consistency

What this PR does / why we need it:
There were some inconsistencies with how annotations where handled between the ingress, service and deployment yamls.  Also in cases where no annotation were defined the "annotation" key was still present in the rendered yaml with no values.  This was causing issues when validating the yaml with tools such as kubeval
Which issue this PR fixes
Inconsistent yaml generation regarding annotations
Special notes for your reviewer:
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

Sorry for the duplicate PR, github and I had some "disagreements" about the DCO and editing commits.  Also wanted to move this to my company repo/fork..
Thanks!


[WIP] Include app context

Let's not merge this as this has a lot of stuff that may be unnecessary for the middleware!


Two folders with same name

git has two folders with the same name:
HPCCSystemsGraphViewControl
hpccsystemsgraphviewcontrol

Only a Windows user would think those were the same name ...

Only a Linux user will be affected!


Support for multi-select

aarond [7:33 PM]
I’m trying to use multiple select with the following <%= select f, :challenge_ids, ["foo": "1","bar": "2","baz": "3"], class: "form-control", multiple: ""  %> but only the id of the last selected item gets sent to the server in the params
aarond [7:33 PM]
how can I get it so submit a collection?
asonge [8:14 PM]
@wsmoak: https://github.com/phoenixframework/phoenix_html/blob/master/lib/phoenix_html/form.ex#L525-L558
asonge [8:17 PM]
i wouldn’t try a :foo[] kind of functionality, i’d probably add a multi option to select…or test for values being a list.

See comments in https://github.com/wsmoak/phoenix_multiple_select/blob/master/web/templates/post/form.html.eex

https://github.com/phoenixframework/phoenix_html/pull/17


Feature: Split Markdown view in editor.

Hello,
Leaps looks awesome. Have you given thought to having a markdown preview option something similar in the editor to the following:
+------+----------------+----------+
|      |                |          |
| File | Input          | Comments |
| List |                |          |
|      |                |          |
|      +----------------+          |
|      |                |          |
|      |                |          |
|      | Preview        |          |
|      |                |          |
+------+----------------+----------+

I suppose it could be more generalised to being a command output pane but markdown preview would be nice as a first kick of the tires. :)
Kind Regards,
Nathan

Hey @nfisher, I've wanted to add this in for a while but my front-end skills are my most lacking. To be honest the entire UI could do with some TLC from a more qualified dev/designer.

@Jeffail you could rewrite the GUI in GIOUI.
Golang code on both side lowers the work and maintenance.
And opens up lots of new functionality possibilities.
The current text editor is pretty capable. cut and paste and tabs work also.
Demo:https://gioui.org//files/wasm/kitchen/index.html
Code: https://github.com/gioui/gio-example/blob/main/kitchen
You get complete control over the Ops of the Text Editor. See here: https://github.com/gioui/gio-example/blob/main/kitchen/kitchen.go#L237
This is because every key stroke by the user can be acted on in the control loop.
The websockets lib you use would need to be changed to https://pkg.go.dev/nhooyr.io/websocket
Here is a good example: https://git.sr.ht/~whereswaldon/pointstar
there is a gio chat that works currently too.
https://git.sr.ht/~gioverse/chat
https://github.com/gioverse/chat
run https://github.com/gioverse/chat/tree/main/example/kitchen to see it in action.


Add a warning page that 2fa is going to be mandatory soon.

Currently non-superadmins can skip setting up 2fa but that is
going to disappear soon. This commit adds a warning page that this
is happening and a link where users can get help.

Trello: https://trello.com/c/jCZwXfk6/1149-change-admin-site-to-warn-of-two-factor-authentication-changes

thanks @koetsier can you explain the controller logic a bit so we're all clear on it?

thanks @koetsier can you explain the controller logic a bit so we're all clear on it?

There's a new link on the 2FA page that only shows up if you're not a super admin (otherwise 2fa would already be mandatory) that takes you to a new controller two_factor_authentication_unable_controller.rb, which handles showing the 2FA warning screen (2nd screenshot). If the user decides to skip 2fa, the update method on that controller is called and 2FA is ignored for the session`. If not, one link takes the user back and the other sends the email


Fault injection with replace operation with no-op

Like https://github.com/stewartsmith/libeatmydata
replace operation with no-op for those operations where it possible
Implementation https://github.com/stewartsmith/libeatmydata/blob/master/libeatmydata/libeatmydata.c

3.1. Disk drives that do not honor sync requests
Unfortunately, most consumer-grade mass storage devices lie about syncing. Disk drives will report that content is safely on persistent media as soon as it reaches the track buffer and before actually being written to oxide. This makes the disk drives seem to operate faster (which is vitally important to the manufacturer so that they can show good benchmark numbers in trade magazines). And in fairness, the lie normally causes no harm, as long as there is no power loss or hard reset prior to the track buffer actually being written to oxide. But if a power loss or hard reset does occur, and if that results in content that was written after a sync reaching oxide while content written before the sync is still in a track buffer, then database corruption can occur.

USB flash memory sticks seem to be especially pernicious liars regarding sync requests. One can easily see this by committing a large transaction to an SQLite database on a USB memory stick. The COMMIT command will return relatively quickly, indicating that the memory stick has told the operating system and the operating system has told SQLite that all content is safely in persistent storage, and yet the LED on the end of the memory stick will continue flashing for several more seconds. Pulling out the memory stick while the LED is still flashing will frequently result in database corruption.

Note that SQLite must believe whatever the operating system and hardware tell it about the status of sync requests. There is no way for SQLite to detect that either is lying and that writes might be occurring out-of-order. However, SQLite in WAL mode is far more forgiving of out-of-order writes than in the default rollback journal modes. In WAL mode, the only time that a failed sync operation can cause database corruption is during a checkpoint operation. A sync failure during a COMMIT might result in loss of durability but not in a corrupt database file. Hence, one line of defense against database corruption due to failed sync operations is to use SQLite in WAL mode and to checkpoint as infrequently as possible.

https://www.sqlite.org/howtocorrupt.html


fix sudo flag

in buildroot image restarting mycroft service fails because it is a user service
this PR adds a flag for sudo, the system PHAL plugin should set this as needed

I will do some research because you can also restart systemd services over a dbus call which already takes care of the system or user stuff as those dbus addresses are already splitted up.


[Suggestion] Add healthcheck

Docker supports a healthcheck command, which can indicate whether or not the container is doing fine or is having problems.
Once strapi 3.0.0 stable has been released, a healthcheck for the Docker image should be available. This can  help DevOp-teams find dead or problematic strapi instances and restart them.
Right now a simple check for any response from the API / WebServer should be enough. In the long run a proper healthcheck API or call to a monitoring API in strapi should be used.
The support for a healthcheck in strapi has been suggested in https://github.com/strapi/strapi/issues/755.

I agree with you. In any case, Strapi should have a dedicated API to store/retrieve the health status.

I already tried to add a very simple wget-healthcheck but ran into unexpected problems. Maybe I can look into a small node / js script that can do a HTTP check for now, which we can later extend to use the API once that is ready.
The idea for the node / js script comes from here: https://blog.sixeyed.com/docker-healthchecks-why-not-to-use-curl-or-iwr/
That makes the code more portable as it only depends on node and not OS tools (even thought we probably won't switch to another image; Alpine is amazing so far) and as far as extending the healthcheck is concerned, a script like that is also much more flexible.

FYI, every Strapi app has already a route to make sure the instance is doing fine HEAD /_health https://github.com/strapi/strapi/blob/master/packages/strapi/lib/Strapi.js#L162
Btw, I really like the idea. However, I'm not sure to understand why do we need to store the health status?

For the moment we should only report the health status to docker container.
Thanks @Aurelsicoko for the info!

@Aurelsicoko I thought that storing the health status might be useful to not fail on the first failed request, but only after a certain failure rate.
Using that route we can do that in the Docker healthcheck itself based on the result of the last X checks, but we will have to find a way to store the last X results (I think the Docker healthcheck cannot save its state, but maybe we can use a small file-based counter).

Unless anyone else is very eager to work on this, I will try to get something up and running next week.

@msgeissler Where do you think you will be store the health status of each server? We have the Core Store API (https://github.com/strapi/strapi/blob/master/packages/strapi/lib/core/store.js) which allows storing any kind of data easily in the database. It could be useful in this case...

@Aurelsicoko for now (I guess it will stay that way, that's the docker way of things) each strapi container will run its own and only one instance of strapi. Since the docker healthcheck is running inside the container as a separate script, it has to deal with only one server.
I am also still quite inexperienced with server side javascript: can I just use the store from outside the context of strapi? Since the docker healthcheck script will be run by docker every x seconds in a separate node instance it might not share the runtime context of strapi.
I will probably create a PoC first that only queries the API without storing the data. That means that the container will be flagged as unhealthy on the first failed check of the strapi healthcheck route. It could then be extended to support some threshold over the last x checks (to allow one failed check in 10 checks or something). But that is then all optional and maybe also needs to be discussed. After thinking about it a bit, I am not sure myself at what point the container should be flagged as unhealthy.

You're right, let's create a POC to make sure you can do what you're talking about and we will solve the storage problem later...

PR #8 already implements a PoC healthcheck. :)
The build currently fails for another reason, but there is already a PR #9 that fixes that.

Closed by #8. Thank you both 👍

Glad I can help.
(Btw it is awesome that the changes are already on Docker Hub. The automated builds are great. :D )


feat: Try all nodes and allow backup nodes

High Level Overview of Change
s1 and s2 stopped responding. This will allow the explorer to try multiple entry points at once and choose the winner.
Type of Change

[x] New feature (non-breaking change which adds functionality)
[ ] Tests (You added tests for code that already exists, or your new feature included in this PR)

Still need to add some tests.


PanoramaSky broken

While the galaxy texture in the demo looks normal in the editor, upon running it on the oculus quest, the texture is squished into the middle beyond recognition.

This is an issue with this repo as an export to a normal android phone has a normal galaxy texture.

I have no idea how this bug is even occurring within the limited code that exists in this repo. The only file that seems to have something to do with rendering is src/framebuffer.cpp so I'm going to assume that there's a mistake somewhere in there.

This seems to have been fixed on the actual sdk of the oculus quest with the new firmware update.

For some reason this seems to be broken again.

I tried to look at bit deeper into the issue.  I have not yet understood what the actual issue is but here are my findings so far:
It seems to be related to the eye projection matrices. The projection matrices used are asymetrical which is a special path in the panorama sky shader (#define USE_ASYM_PANO):
It is checked here to set the proper shader parameters:
https://github.com/godotengine/godot/blob/de8ce3e625e74833aec6a5d165e7e82100a1dbf3/drivers/gles3/rasterizer_scene_gles3.cpp#L2564
and then executes this code in the shader:
https://github.com/godotengine/godot/blob/de8ce3e625e74833aec6a5d165e7e82100a1dbf3/drivers/gles3/shaders/copy.glsl#L171
The problem is now maybe related to the large constant chosen for z:
cube_normal.z = -1000000.0;
From what I understand so far of this algorithm, this should be fine and actually it should not matter a lot what number is chosen there.
But if I set this to cube_normal.z = -1.0; the sky appears undistorted;
setting it to -1000.0 the sky starts to get distorted a bit later as seen in this screenshot here:

Not sure what is happening here; a first guess is maybe the mobile shader execution has some numerical precision problems with the high numbers (and then normalizing the very long vector which probably will square the large values again)... or one of the input parameters is not as expected...
the projection matrices handed over from the ovr plugin look fine to me; here is a debug print of the values set to the shader parameter asym_proj in the function
_draw_sky(...) (https://github.com/godotengine/godot/blob/de8ce3e625e74833aec6a5d165e7e82100a1dbf3/drivers/gles3/rasterizer_scene_gles3.cpp#L2566):
Left eye:
camera.matrix[2][0]=-0.174072, camera.matrix[0][0]=0.917286, camera.matrix[2][1]=-0.106141, camera.matrix[1][1]=0.833537
Right eye:
camera.matrix[2][0]=0.174072, camera.matrix[0][0]=0.917286, camera.matrix[2][1]=-0.106141, camera.matrix[1][1]=0.833537
Looking at the screenshot for z = -1000.0 it is maybe also a hint that the lookup seems to be wrong only in up/down direction (the full texture seems to be squished into the middle and then just the border extended up/down)

Setting the constant here https://github.com/godotengine/godot/blob/de8ce3e625e74833aec6a5d165e7e82100a1dbf3/drivers/gles3/shaders/copy.glsl#L172 to
cube_normal.z = -1.0;
instead of -1000000.0 solves the issue.
Also I just realized that the actual shader code:
	vec3 cube_normal;
	cube_normal.z = -1000000.0;
	cube_normal.x = (cube_normal.z * (-uv_interp.x - asym_proj.x)) / asym_proj.y;
	cube_normal.y = (cube_normal.z * (-uv_interp.y - asym_proj.z)) / asym_proj.a;
	cube_normal = mat3(sky_transform) * mat3(pano_transform) * cube_normal;
	cube_normal.z = -cube_normal.z;

	vec4 color = texturePanorama(normalize(cube_normal.xyz), source);

from a mathematical (not numerical) perspective should produce an identical result independent of what the magnitude of cube_normal.z is at the beginning as it is just a linear scale to the whole cube_normal vector that is then linearly transformed and finally normalized which cancels out the scale.
Still I'm puzzled what is happening on the oculus quest. I checked the shader and it sets
precision highp float;
also calling
glGetShaderPrecisionFormat(GL_FRAGMENT_SHADER, GL_HIGH_FLOAT, range, &precision);
returns

[127 127] 23
on the oculus quest which is full 32bit float precision...

The fix was merged to godot master so I think the issue can now be closed.

This issue exists still in the gles2 renderer (which is the renderer used in the demo scene); I created a PR to fix it in gles2:  https://github.com/godotengine/godot/pull/32259
Should this be a new issue here or this issue reopened?

Let's reopen this issue.

Fixed by PR #32259


fix: foxy max delayed withdraw

Description

Notice

[ ] Have you followed the guidelines in our Contributing guide?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

closes #
Risk

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #4108   👈

This comment was auto-generated by Graphite.


⚠ ClamAV has degraded performance

In 0ab40af, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 84 ms

Resolved: ClamAV performance has improved in 13ab8e0 after 7 minutes.


(Viral**Video)@Camilla Araujo Leaked Video Online HD

In the realm of celebrity privacy and consent, the Camilla.Araujo Nude Leaks leaked video controversy has ignited a firestorm of discussion. This incident, involving the unauthorized release of explicit videos and photos of the renowned actress, has sent shockwaves through the entertainment industry and beyond. Join Vninvestment as we delve into the intricate details of this case, exploring its impact on Camilla.Araujo Nude Leaks’ career, the legal ramifications, and the broader societal implications surrounding the non-consensual sharing of private content in the digital age.
⬇👇⬇👇 Watch/Download ⬇👇⬇👇
Watch 🟢 ➤ ➤ ➤ 🌐 Click Here To link (Full Viral Video Link)
🔴 ➤►DOWNLOAD👉👉 (Full Viral Video Link)
“Camilla.Araujo Nude Leak leaked nudes”: How this ‘trending topic’ on X is spreading malware
In a shocking turn of events, explicit videos, personal photos, and private messages involving Camilla.Araujo Nude Leaks were leaked online without her consent. The unauthorized release of this private content sent shockwaves through the entertainment industry and sparked a heated public debate.
Camilla Araujo Nude Leak Responds
Watch 🟢 ➤ ➤ ➤ 🌐 Click Here To link (Full Viral Video Link)
🔴 ➤►DOWNLOAD👉👉 (Full Viral Video Link)
In the face of the controversy, Camilla.Araujo Nude Leaks released a powerful statement condemning the unauthorized release of her private content. She expressed her deep distress and disappointment, emphasizing the violation of her privacy and the potential legal consequences for those responsible.

Camilla Araujo Leaked Video Online HD
Watch 🟢 ➤ ➤ ➤ 🌐 Click Here To link (Full Viral Video Link)
🔴 ➤►DOWNLOAD👉👉 (Full Viral Video Link)


Update Tika to version 1.7

I noticed that Tika 1.7 was released so time to upgrade!

Closed by #22. Thank you @jrotello!

Posted about this release


add code style "chromium", "microsoft" to file annotation

Fix #14
also, adjust the order a bit to make it look consistent with the order of the code style output from clang-format --help for easy comparison.
 --style=<string>               - Coding style, currently supports:
                                     LLVM, GNU, Google, Chromium, Microsoft, Mozilla, WebKit.

Codecov Report

Merging #15 (415079c) into main (d0149f6) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #15   +/-   ##
=======================================
  Coverage   80.65%   80.65%
=======================================
  Files           6        6
  Lines         646      646
=======================================
  Hits          521      521
  Misses        125      125

Impacted Files
Coverage Δ

cpp_linter/clang_format_xml.py
100.00% <100.00%> (ø)

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

run pre-commit run --all-files successfully from my environment, not sure why it failed in the workflow.

I think the pylint settings just need to be updated. I see this in the CI log:

************* Module /home/runner/work/cpp-linter/cpp-linter/pyproject.toml
pyproject.toml:1:0: W0012: Unknown option value for '--disable', expected a valid pylint message and got 'missing-timeout' (unknown-option-value)

This message usually indicates that pylint was updated and the update brought changes that encountered an unexpected config option.


fix(nmariana): Fix nmariana judge/author field

Minor fix for naming convention in NMI.

Left a comment about "judge" and "author"
Unrelated, on line 21 year = str(date.today().year)[-2:] would read better as year = date.today.strftime("y")

I think date.today().strftime("%Y")[-2:]


Implement file upload without name test

Adds a test for files.upload (without custom name).
Adds nock as a devDependency for testing.

I'd enjoy any feedback on the approach I took and would also be happy to
add coverage to some of the other functions if we're happy with the
general direction here.
Conversation started in
https://github.com/Skycatch/node-box/pull/3#issuecomment-136026675

:+1:

:+1:

:+1:

Ping @lyaunzbe, curious about your thoughts.

LGTM. Thanks! You are more than welcome to add additional coverage for other functions, it is much appreciated!

Great, thanks! @lyaunzbe what would you think about setting up CI (maybe using http://circleci.com/) for this repo?


Gestures that aren't "horizontal" enough are blocked by horizontalOnly

While I was playing with some of the examples, I noticed that certain gestures were not resulting in any movement. For example, a SwipeableCard, which has horizontalOnly specified would not pick up on certain gestures I was making (e.g. an arc that starts fairly vertical and eventually moves horizontally).
In the iOS code, I see that gestureRecognizerShouldBegin uses the heuristic of "more horizontal than vertical" to determine whether it should respond to a gesture. I'm curious whether this was done for performance reasons, or something else? It may be worth adding an additional property that controls this behavior, while leaving verticalOnly and horizontalOnly to control movement axes.

thanks for the comment, we will check this whether this is a bug or not

@wsun have you tested this on Android ?

I haven't tested on Android -- only iOS. Not sure this is a bug, since it behaves exactly as you would expect after having read the code, but seems more like a design decision to bundle together 1/ movement axis and 2/ gestures to respond to into a single boolean. I can think of situations where one would prefer these unbundled (e.g. if you were making a carousel, you might only want items to move horizontally, but would be fine responding to gestures that start off vertical).

I am closing the issue as a part of an effort to cleanup and revive the project. If this issue persists after v1.0.0, you are more than welcome to reintroduce it.
Thanks for your contribution.


SyntaxError: Support for the experimental syntax 'jsx' isn't currently enabled (154:9)

Have anyone encountered the same problem while importing this package?
`./node_modules/react-native-switch-selector/index.js
 SyntaxError: C:\Users\HP\Desktop\MSU_Work\Tool2_CPT_JS\Refactored_CPT\frontend\node_modules\react-native-switch-selector\index.js: Support for the experimental syntax 'jsx' isn't currently enabled (154:9):

   152 |
   153 |       return (
       |         ^
   155 |           key={index}
   156 |           disabled={disabled}
   157 |           style={[

 Add @babel/preset-react (https://git.io/JfeDR) to the 'presets' section of your Babel config to enable transformation.
 If you want to leave it as-is, add @babel/plugin-syntax-jsx (https://git.io/vb4yA) to the 'plugins' section to enable parsing.`

I had these Babel plugins installed in my local node modules but it didn't work, so is it a problem coming along with the package?

Having the same issue


Bugfix/Not all content is rendered after redirecting from /my-account endpoint. #3325

Related issues

closes #3325
Short description and why it's useful

Moved router guard to core MyAccount page mounted hook. Now Server and Client have the same code.
Screenshots of visual changes before/after (if there are any)

Which environment this relates to
Check your case. In case of any doubts please read about Release Cycle

[x] Test version (https://test.storefrontcloud.io) - this is a new feature or improvement for Vue Storefront. I've created branch from develop branch and want to merge it back to develop
[ ] RC version (https://next.storefrontcloud.io) - this is a stabilisation fix for Release Candidate of Vue Storefront. I've created branch from release branch and want to merge it back to release
[ ] Stable version (https://demo.storefrontcloud.io) - this is an important fix for current stable version. I've created branch from hotfix or master branch and want to merge it back to hotfix

Upgrade Notes and Changelog

[ ] No upgrade steps required (100% backward compatibility and no breaking changes)
[x] I've updated the Upgrade notes and Changelog on how to port existing VS sites with this new feature

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change
Contribution and currently important rules acceptance

[x] I read and followed contribution rules
[x] I read the TypeScript Action Plan and adjusted my PR according to it
[x] I read about Vue Storefront Modules and I am aware that every new feature should be a module

@przspa could you remove/fix these errors from dev console?

@alinadivante fixed

@przspa by some reason the CI is failing can you please check it?

@pkarw now everything is ok


Avoid using overly general CSS selectors last term in CSS

While profiling rendering HTML elements in Zulip, which uses KaTeX for its math rendering, I determined that the KaTeX CSS uses a few inner selectors that end up slowing down style calculations for any site that import the KaTeX CSS, whether or not any .katex elements actually appear on the page. I did my investigation using the built-in Chrome profiling tool detailed in https://css-tricks.com/the-truth-about-css-selector-performance/.
The mechanism is that the browser's internal implementation does bottom-up matching, so a rule of the form .katex * will end up doing work for every DOM element whose styles are being (re)calculated. The .katex * rule in katex.css alone, in particular, consumes about 2.3% of the time spent by the browser to calculate styles when rendering a Zulip message feed with a hundred messages in it; basically because it is checked for each of the 9452 DOM elements that get rendered when doing so.
The fix for this issue is to just to have the inner selector in any CSS rules be as specific as possible; in particular, very broad final selectors like *, div, and span are expensive. I know this is counterintuitive. The CSS model is that a selector like .katex * safely only applies to katex blocks. But apparently browser implementations do their work bottom-up, rather than top-down, so very broad inner selectors end up burning a lot of electricity.
See this Zulip thread and linked threads for more context.
Here's the katex.css code for that single rule:
    * {
        // Prevent background resetting on elements in Windows's high-contrast
        // mode, while still allowing background/foreground setting on root .katex
        -ms-high-contrast-adjust: none !important;

        // Insulate fraction bars and rules from CSS that sets border-color.
        border-color: currentColor;
    }

I'm not sure what that second block does, but the first one seems to be a deprecated rule (https://blogs.windows.com/msedgedev/2024/04/29/deprecating-ms-high-contrast/) and possibly not necessary.
I think the complete list of CSS rules with issues of this form in KaTeX are the following:

Bare svg and path and img rules. These are not significant in my profile; presumably only an app using a very large number of images would care. So probably these are only worth fixing if it's easy:

svg {
    display: block;
    position: absolute; // absolute relative to parent
    width: 100%;
    height: inherit;

    // We want to inherit colors from our environment
    fill: currentColor;
    stroke: currentColor;

    // But path elements should not have an outline by default
    // that would make them bigger than we expect.
    path {
        stroke: none;
    }

    // And we don't want to inherit any other style properties
    // that could affect SVG rendering without affecting font
    // rendering. So we reset these properties to their default
    // values for every <svg> element.
    // See https://www.w3.org/TR/SVG/painting.html
    fill-rule: nonzero;
    fill-opacity: 1;
    stroke-width: 1;
    stroke-linecap: butt;
    stroke-linejoin: miter;
    stroke-miterlimit: 4;
    stroke-dasharray: none;
    stroke-dashoffset: 0;
    stroke-opacity: 1;
}
img {
    border-style: none;
    min-width: 0;
    min-height: 0;
    max-width: none;
    max-height: none;
}

The * rule used as an example above.
A half-dozen rules that have a class-free span at the end, and thus end up doing matching work for every span in the app. These in total consume about 3% of the rendering time in my profile of rendering 100 Zulip messages; presumably their impact will be more or less than that of the * rule, depending on how what portion of DOM elements in a given project using KaTeX are spans. Rather than including all of them, here's a grep that shows how to find them all:

$ grep ' span {' node_modules/katex/src/styles/katex.scss
        > span {
            > span {
        > span > span {
    .clap > .inner > span {
            .delim-size1 > span {
            .delim-size4 > span {

I hope you'll consider fixing this! Since it's only 5% of our processing time in aggregate, it's not going to have big impact on Zulip in particular, but given how many sites integrate KaTeX, it might save a significant amount of wasted electricity/latency to improve these expensive inner selectors.

Thanks Tim! This is very helpful analysis. I was thinking recently about how CSS matching rules could be implemented by a fast algorithm, and bottom-up is I think necessary, so this makes a lot of sense, but I hadn't quite realized this implication.
I wonder whether the two * rules could be applied to the .katex element, and the values would be inherited down the subtree. (The second one is a limited form of reset CSS, I believe, but I'd have to check old issues/PRs to remember what exactly it protects against.) Probably need to test.
Any chance you'd be willing to pursue this to the point of a PR?

Normally I'd be happy to put up a PR, but I think this is the kind of refactoring project where verifying the changes don't break anything is almost all of the actual effort, and I'm not sure I know the KaTeX system well enough to be able to generate the various elements that would be being modified.
(For example, I have no idea how to create a delim-size4 element, or how to check what possible spans might be inside it that the CSS rule might be intended to apply to).


[VL] Add support for ORC decimal IsNotNull filter

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
(Fixes: #ISSUE-ID)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

close


Invalid response: 413 Request Entity Too Large when using interactive widgets in Orchest

Describe the bug
When using interactive widgets inside Orchest (see #1235) I frequently get errors like these, usually right after creating the first widget:

"File Save Error for Experiments.ipynb
Invalid response: 413 Request Entity Too Large"
Since autosave is on, this modal dialog will keep popping up even if I dismiss it. The solution is usually to clear the corresponding output.
Expected behavior
I can use interactive widgets without errors.
To Reproduce
Steps to reproduce the behavior:

Install ipywidgets and ipympl on Jupyter
Create a new project, and install matplotlib inside the environment
Enable the ipympl matplotlib backend by running %matplotlib ipympl inside the notebook
Make plot with lots of elements
Observe error happening

Screenshots

Environment

OS: Linux Mint
Browser: Firefox
Orchest's version: 2022.08.8

Just to be absolutely sure, you get this issue only in Orchest? And when running the same code in JupyterLab outside of Orchest this doesn't happen?

Just confirmed that I cannot reproduce this with JupyterLab 3.4.2 outside of Orchest, the save process seems to work normally.

@astrojuanlu Could you verify whether this issue is not fixed by https://github.com/orchest/orchest/pull/1242? Maybe the version upgrade worked its magic? :crossed_fingers:


Set speed

Overview
If you first reduce the animation speed setSpeed(0.1), and then (after some time) increase setSpeed(1), then the animation jumps forward (as if the animation speed was always equal to "1").
You can observe this behavior in any linear animation, like
https://lottiefiles.com/animations/progress-bar-fast-4mRIN4LXSY
This code will break the animation
/* BAD CODE */
player.setSpeed(0.1)
setTimeout(() => {
    player.setSpeed(1)
}, 1000)

Corrects the behavior by calling pause() before setting the new speed.
player.setSpeed(0.1)
setTimeout(() => {
    player.pause()
    player.setSpeed(1)
    player.play()
}, 1000)

Consuming repo
dotlottie-web
Labels

[ ] Add the Type: Bug label to this issue.

@rismirnov this issue is resolved in<EMAIL_ADDRESS>


Asset added to cart is gone from cart when moving from one page to another page

Asset added to cart is gone from cart when moving from one page to another page
With the version 1.4.0 on AEM 6.4, we are seeing issue with the asset added to the cart missing after going to a different page.  This issue does not occurs with version 1.4.0 on AEM 6.3 SP1.

@ayfengl I know this error is reproducible when ASC is deployed to a fresh 6.4 instance. Do you see the same behavior when upgrading an AEM instance on 6.3.1 + ASC -> 6.4 ?

@godanny86 we actually noticed this in an upgraded instance from AEM 6.3.1 + ASC 1.1.4 to AEM 6.4 and we apply the latest ASC 1.4.0 to take benefit of ASC fixes since then.

Ok thanks

The PR #158 fixes the persistent issue but still noticing that the Remove from cart label is not consistent after subsequent page refreshes. So there is still a little more work to be done before closing this out.

Fixed


A wrong test case in "Confirm the Ending"  exercise

Challenge Confirm the Ending has an issue.
User Agent is: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.110 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
function confirmEnding(str, target) {
  // "Never give up and good luck will find you."
  // -- Falcor
  ans = "";
  arr = str.split(" ");
  if(arr.length == 1) {
    if(target == arr[0].substring(arr[0].length-1)) {
      ans = true;
    }
    else {
      return false;
    }
  }
  else if(arr.length > 1) {
    if(arr[arr.length-1] == target) {
      ans = true;
    }
    else {
      ans = false;
    }
  }
  return ans;
}

confirmEnding("He has to give me a new name", "me");

This code if perfectly fine, but there is a wrong test case stating that

confirmEnding("He has to give me a new name", "me") should return true.

it should be false.

The string does end with "me", "He has to give me a new name".

my code:
function confirmEnding(str, target) {
return str.substr(-target.length)==target;
}
confirmEnding("He has to give me a new name", "me");

@JBuji your solution is very smart, congratulations =D

@BrunoFDeveloper Thank you. "helpful link" was very useful.


feat: apply bridge client changes to helm chart

Issue
This contains a BC still is discussion. Although avoidable it is for the better that it should change: unsecured TLS by default it not great right.
https://gravitee.atlassian.net/browse/APIM-338 (close but this a trailing part of it for APIM Gateway)

A separate commit contains a reformat of values.yaml
The rest are the actual changes + doc

Description

Simplify configmap setup for Gateway
Change values.yaml with new defaults and parameters (as comments)
Note that when cloud.enabled = true many of these values will be enforced internally in an Hybrid Gateway, so this PR is a general one for all other use cases.

Additional context

🏗 Your changes can be tested here and will be available soon:
Console: https://pr.team-apim.gravitee.dev/7748/console
Portal: https://pr.team-apim.gravitee.dev/7748/portal
Management-api: https://pr.team-apim.gravitee.dev/7748/api/management
Gateway v4: https://pr.team-apim.gravitee.dev/7748
Gateway v3: https://pr.gateway-v3.team-apim.gravitee.dev/7748

📚  View the storybook of this branch here

LGTM, @phiz71 , what is the process when you do a breaking change? you have documentation to update?

For a minor release, we have to see with @anthonyjgatti  and @gravitee-io/tech-writers to include this BC in the future release note


Remove incorrect semicolon in CS0501, fix spelling

Summary
This PR removes an unnecessary/incorrect semicolon in one of the examples for resolving compiler error CS0501.
A semicolon after a curly-braced method body will instead trigger compiler error CS1597.
Some spelling corrections are included as well.

@dotnet-policy-service agree


Combined parser

Introducing the delicious MultiHarParser

Coverage decreased (-1.57%) to 98.43% when pulling 17ae18ce08b283f580431f445ca4a6c96d012aa3 on combined-parser into 497d0837d8a707b8199b1ca402e6b837ba465a1d on master.

Coverage decreased (-1.57%) to 98.43% when pulling d118f4c47c6f8a5d82e1ee1785b112c7380f30e4 on combined-parser into 497d0837d8a707b8199b1ca402e6b837ba465a1d on master.

Coverage decreased (-1.65%) to 98.35% when pulling 90bf0133fa7d65a5af3819466e0be64178e9f5a0 on combined-parser into 497d0837d8a707b8199b1ca402e6b837ba465a1d on master.


feat: Introduce web bindings and orb NPM package

This change proposes a milestone feature for Noosphere: a working NPM package that vends a TypeScript-based library that exposes Noosphere to JavaScript via a Wasm kernel.
In addition to the NPM package, this change includes wasm-bindgen-based FFI on the Rust side and a minimum viable Lit + Redux viewer application that is designed to render sphere content when loaded from a commodity IPFS Kubo gateway.
In support of the loading-out-of-a-gateway use case, the change includes a feature-gated KuboStorage implementation that can wrap any Storage implementation. KuboStorage will "fall back" to looking for blocks from an IPFS gateway if a requested block is not found in local storage. This makes KuboStorage suitable for read-only Noosphere use cases where an IPFS gateway API may be available.
I chose https://github.com/google/wireit to orchestrate builds across Rust and TypeScript packages. Largely thanks to Wireit, we don't need to be aware of yet-another-tool to run correct builds that span multiple source trees. Running npm run build from the typescript folder is sufficient to build everything (including Rust dependencies) in the intended order. The same goes for running tests: npm run test from the typescript folder will ensure all dependencies are built in time for the tests to run.
@aomarks has offered to lend us his scrutiny to make sure we get the most out of wireit.
Our test suite has also been expanded to include TypeScript tests (which are Mocha-formatted tests that run in headless Chrome). This should get us coverage from the JavaScript side of the web FFI.
Fixes #165

I'm going to go ahead and merge since the last approval was dismissed by an unambiguously good (and small) patch. Thanks so much for the feedback @jsantell @aomarks and @SebastianZimmer !


VRChat doesn't receive data from VRCFaceTracking

VRCFaceTracking receives data (screenshot), but VRChat doesn't receive anything - OSC debug console is empty.
Different program (VSeeFace) does reach VRChat (but they use different parameters so no existing avatar with face tracking works)

I might need a little more context about how you are using VRCFT: Are you using an avatar that is setup for VRCFaceTracking? OSC data will only be sent once an avatar that supports VRCFT is loaded.

I repeated it today with a different avatar to make sure. I was in VRC in the avatar that (from description and another person's claims) supports face tracking, with OSC turned on when I started the VRCFT. The tracking is activated on the avatar.
I used the latest release (v5.0.0-pre-2) from VRCFT with included custom libs (the one from here wouldn't initialize for some reason)


Move and Resize to percentage of desktop size (enhancement)

We would like to run the same command on desktops of different sizes and resize a window to 25% of desktop and move to the top right corner say 5% from the top and 5% from the right.

Here is a sample batch file I created to do this:
@echo off
for /f "tokens=2,10-11" %%a in ('cmdow /p') do (
if "%%a"=="0" set "WIDTH=%%b" & set "HEIGHT=%%c"
)
set /a "W=(%WIDTH%*100)/233
set /a "H=(%HEIGHT%*1000)/1176
set /a "R=(%WIDTH%-%W%)-20
set /a "T=(%HEIGHT%-%H%)/5
cmdow.exe Execution /res
cmdow.exe Execution /siz %W% %H% /mov %R% %T%


Update README.md

Hi Buddy, I have made a change in the Readme Title ...It will help you to get better reach ..and will be helpful for everyone in Open source community .
Thanks..

This library works on other platforms besides Forms.


[Data Error] Taxon of FECA_ECOLI_Tsuboyama_2023_2D1U is wrong!

The source organism of FECA_ECOLI_Tsuboyama_2023_2D1U is Escherichia coli, a Prokaryote.

Fix in https://github.com/OATML-Markslab/ProteinGym/pull/40

Hi @ginnm -- good catch and thank you for the PR! Just merged to main.
Best,
Pascal


question for the implementation of CRF

Hi , can I use the pystruct to implement the model in the picture ?
Thanks a lot,
Ben

Depends on what the states of the h are. But probably yes.

h has 7 states，and x is 1 *4096 vector.
Btw is a example for fully connected crf.
Thanks

12 nodes  shall be enough.

that should work. Though pystruct is only doing categorical nodes.

Is there an example for this model？

Do you know how to implement that? I'm also doing the same work like you, and I don't know how to use this CRF. Can you tell me that?

https://pystruct.github.io/auto_examples/plot_latent_node.html#sphx-glr-auto-examples-plot-latent-node-py

Hi,
Can GraphCRF use two types of templates? That is two different set of nodes with different number of classes that share weights in each set.
For example, In the image posted by qdinfish is it possible to build a model to find the best assignment for (h1,...,hn, y) with GraphCRF or other methods in pystruct? here h is individual human action and may have k classes and y is collective activity that can have m classes. So there are two different type of potential functions between nodes.
I see that you wrote this in user guide "The main assumption in these models in PyStruct is that nodes are homogeneous, that is they all have the same meaning. That means that each node has the same number of classes, and these classes mean the same thing." and I think it's not possible but I want to know whether it is what I understood or not? If we can't have two different type of nodes, do you think it is possible to add that kind of functionality?
Thanks


Trigger Deploy on Branch /v\d+.\d+$/

When we are developing on top of the master branch, we still need to make bug fixes for the stable branch.
Like we are developing v0.13.x now, and we are maintaining the version of v0.12.x, which is a branch for the stable version, named v0.12.

TravisCI for publishing NPM
CircleCI for rebuilding Docker Image

See:

Build Stages: Deploying to npm - https://docs.travis-ci.com/user/build-stages/deploy-npm/

done.
stages:
  - test
  - pack
  - name: deploy
    if: branch = /^(master|v\d+\.\d+)/


Batch june rest2

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

[X] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[X] I adhere to the Command Guidelines.

@marstr Is there anywhere else we need to edit to allow the image based tests to pass? Seems to be a problem since there is a major version dependency conflict.

@bgklein says:
@marstr Is there anywhere else we need to edit to allow the image based tests to pass? Seems to be a problem since there is a major version dependency conflict.

First thing to update is the requirements.txt files in src/azure-cli/. Once that's happened, I'll help investigate remaining failures to make sure their not systemic on our end. :)

@xingwu1 please approve so CLI team can merge

@marstr / @tjprescott if either of you want to take a look at this (specifically the README/setup/requirements files since those are recently reformatted) please review by EOD Friday and feel free to merge if you do. Otherwise I will merge after some traveling Friday.


Fix Code Scanning Alert tool field

The tool field on the API response of listing code scanning alerts is a struct instead of a string. This PR fixes the alert tool field to a struct.
Github API: https://docs.github.com/en/rest/reference/code-scanning#list-code-scanning-alerts-for-a-repository
Deprecation notice from Github: The tool_name field is deprecated and will, in future, not be included in the response for this endpoint. The example response reflects this change. The tool name can now be found inside the tool field.
https://docs.github.com/en/rest/reference/code-scanning#list-code-scanning-analyses-for-a-repository

@googlebot I signed it!

I apologize for the PR name changes. I see now that this is a "fix" and not an addition... and is therefore a breaking API change.

Friendly ping @wesleimp

@gmlewis @wesleimp
gentle bump

@gmlewis @wesleimp
Can we get one more review for the PR please? We are not able to use go-github because of the issue in Alert tool field.

@gmlewis @wesleimp
Can we get one more review for the PR please? We are not able to use go-github because of the issue in Alert tool field.

Please feel free to have any contributor to this repo provide a code review and approval, and we will get this merged.  Thanks.


udpate k8s ingress controller route policy

fix #633

Codecov Report

Merging #651 (57a0e65) into main (d98176f) will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #651      +/-   ##
==========================================
+ Coverage   80.00%   80.03%   +0.03%
==========================================
  Files          95       95
  Lines       10986    10986
==========================================
+ Hits         8789     8793       +4
+ Misses       1715     1712       -3
+ Partials      482      481       -1

Impacted Files
Coverage Δ

pkg/util/timetool/distributedtimer.go
95.45% <0.00%> (-4.55%)
:arrow_down:

pkg/object/mqttproxy/client.go
79.63% <0.00%> (-0.91%)
:arrow_down:

pkg/filter/validator/basicauth.go
79.35% <0.00%> (+0.45%)
:arrow_up:

pkg/cluster/cluster.go
59.93% <0.00%> (+0.93%)
:arrow_up:

Continue to review full report at Codecov.

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


How to use custom color for step label string

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
I wanna use Stepper in a black background. Accordingly, need to change the step label string from the primary color (grey) to a light one (white).
I applied a custom style to StepLabel as below:
const ColorlibStepLabel = styled(StepLabel)(() => ({
  [`&.${stepLabelClasses.active}`]: {
    [`& .${stepLabelClasses.label}`]: {
      color: '#fff',
    },
  },
  [`&.${stepLabelClasses.completed}`]: {
    [`& .${stepLabelClasses.label}`]: {
      color: '#fff',
    },
  },
  [`& .${stepLabelClasses.label}`]: {
    color: 'rgb(255, 255, 255, 0.3)',
  },
}));

After this, I was able to get white color for disabled state but not for completed and active state.

I'd like to know the correct way how to change the color for completed and active step label string.

Please provide a CodeSandbox (https://material-ui.com/r/issue-template-latest), so that we can reproduce and share the fix with you. Thanks!

Oh, that's a good idea.
https://codesandbox.io/s/customizedsteppers-material-demo-forked-o169v
Hope this works for you. Thank you!

https://codesandbox.io/s/headless-field-nbbcy
This is also the same one.

hey @euj1n0ng, i think this will work:
 [& .${stepLabelClasses.label}]: {     [&.${stepLabelClasses.completed}]: {       color: "rgb(255, 255, 255, 0.3)"     },     [&.${stepLabelClasses.active}`]: {
color: "rgb(255, 255, 255, 0.9)"
},
color: "rgb(255, 0, 64)	"

}`
working here: https://codesandbox.io/s/busy-mccarthy-x403u?file=/src/Demo.tsx

hey @euj1n0ng, i think this will work:
 [`& .${stepLabelClasses.label}`]: {
    [`&.${stepLabelClasses.completed}`]: {
      color: "rgb(255, 255, 255, 0.3)"
    },
    [`&.${stepLabelClasses.active}`]: {
      color: "rgb(255, 255, 255, 0.9)"
    },

    color: "rgb(255, 0, 64)	"
  }

working here: https://codesandbox.io/s/busy-mccarthy-x403u?file=/src/Demo.tsx:1494-1740

@amanzrx4 , yes, it works like a charm! Thank you so much. 😍


[android] UIBittorrentDownload::checkSequentialDownload with a setting

Add a torrent download setting that brings back the behavior which downloads the first 10Mb or 30% of a file sequentially.
If this setting is on, the checkbock for sequential download on torrent transfer details must become disabled.

Done
https://github.com/frostwire/frostwire/commit/c9722c6754e26fc7738f59e4dcb72d031af1e5f7