[camera] [android] Front camera not flashing screen when flash is turned on.

Description
Using the front camera on iOS with flash works as expected - the screen flashes white color.
The same approach on android with the package just takes a photo without flashing the screen. At the same time using the device's camera flashes the screen as on iOS.
Steps to Reproduce

I noticed this in a private project, but it is easy to replicate in the camera example.
Setup:

Flutter 3.3.3
camera: 0.10.0+1

Select front camera
Turn the flash to always on
Take a picture

Expected results:
The screen should flash white
Actual results:
No screen flash
Tested on iPhone 8(iOS 15.4), Google Pixel 5 (android 13), Samsung Galaxy S9+ (android 10)

Logs

flutter doctor -v                      14:29:14
[✓] Flutter (Channel stable, 3.3.3, on macOS 12.6
    21G115 darwin-x64, locale uk-UA)
    • Flutter version 3.3.3 on channel stable at
      /Users/zaliskyi/flutter
    • Upstream repository
      https://github.com/flutter/flutter.git
    • Framework revision 18a827f393 (7 days ago),
      2022-09-28 10:03:14 -0700
    • Engine revision 5c984c26eb
    • Dart version 2.18.2
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android
    devices (Android SDK version 33.0.0-rc3)
    • Android SDK at
      /Users/zaliskyi/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0-rc3
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/Contents/Home/bin/ja
      va
    • Java version OpenJDK Runtime Environment
      (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode
    14.0.1)
    • Xcode at
      /Applications/Xcode.app/Contents/Developer
    • Build 14A400
    • CocoaPods version 1.11.3

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

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android
      Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨
      https://plugins.jetbrains.com/plugin/9212-fl
      utter
    • Dart plugin can be installed from:
      🔨
      https://plugins.jetbrains.com/plugin/6351-da
      rt
    • Java version OpenJDK Runtime Environment
      (build 11.0.12+0-b1504.28-7817840)

[✓] VS Code (version 1.71.2)
    • VS Code at /Applications/Visual Studio
      Code.app/Contents
    • Flutter extension version 3.50.0

[✓] Connected device (5 available)
    • Pixel 5 (mobile)  • 0A201FDD4005N1
      • android-arm64  • Android 13 (API 33)
    • ART L29 (mobile)  • DUJNW20526002155
      • android-arm64  • Android 10 (API 29)
    • iPhone 8 (mobile) •
      44CA9E5A-AF87-439E-8065-865706CAA84B • ios
      •
      com.apple.CoreSimulator.SimRuntime.iOS-16-0
      (simulator)
    • macOS (desktop)   • macos
      • darwin-x64     • macOS 12.6 21G115
      darwin-x64
    • Chrome (web)      • chrome
      • web-javascript • Google Chrome
      106.0.5249.91
    ! Error: iPad Oleksandr is busy: Waiting to
      connect and unlock the device. Xcode will
      continue when iPad Oleksandr is finished.
      (code -10)

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Hello @ozalisky. I do not have a physical iOS device to verify the behavior of the camera. Could you share a screen recording of said behavior so that I may verify the issue?

Hi @exaby73
Here are the videos:

Camera package on iOS
https://user-images.githubusercontent.com/40992789/194253542-eb6a46af-99b4-486a-a42c-816232326675.mp4

Camera package on samsung
https://user-images.githubusercontent.com/40992789/194256239-9201e232-e188-4034-b80d-9f97c0f569ed.mp4

Camera app on samsung
https://user-images.githubusercontent.com/40992789/194256845-23140238-7c0b-47ac-a468-914838429e60.mp4

@ozalisky thanks, I also can reproduce the issue on my device (Realme 6, Android 11) which supports flashlight.
The issue is reproducible on the latest stable and master Flutter channels.

Back camera: ✅
Front camera: ❌

✅: No Issue ❌: Issue reproduced

Plugin version: camera: ^0.10.0+3
Sample code: https://pub.dev/packages/camera/example

flutter doctor -v (stable and master)
[✓] Flutter (Channel stable, 3.3.4, on macOS 12.6 21G115 darwin-x64, locale en-VN)
    • Flutter version 3.3.4 on channel stable at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision eb6d86ee27 (23 hours ago), 2022-10-04 22:31:45 -0700
    • Engine revision c08d7d5efc
    • Dart version 2.18.2
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 31.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.0.1)
    • Xcode at /Applications/Xcode14.0.1.app/Contents/Developer
    • Build 14A400
    • CocoaPods version 1.11.3

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

[✓] Android Studio (version 2021.2)
    • 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 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2022.2.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 70.0.5
    • Dart plugin version 222.4167.21

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/221.5591.52/IntelliJ IDEA CE.app
    • 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

[✓] VS Code (version 1.71.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.48.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.6 21G115 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 105.0.5195.125

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[✓] Flutter (Channel master, 3.5.0-2.0.pre.14, on macOS 12.6 21G115 darwin-x64, locale en-VN)
    • Flutter version 3.5.0-2.0.pre.14 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 59d68e03ee (15 minutes ago), 2022-10-05 23:53:34 -0400
    • Engine revision 5f2daf05e0
    • Dart version 2.19.0 (build 2.19.0-273.0.dev)
    • DevTools version 2.18.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 31.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.0.1)
    • Xcode at /Applications/Xcode14.0.1.app/Contents/Developer
    • Build 14A400
    • CocoaPods version 1.11.3

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

[✓] Android Studio (version 2021.2)
    • 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 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2022.2.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 70.0.5
    • Dart plugin version 222.4167.21

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/221.5591.52/IntelliJ IDEA CE.app
    • 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

[✓] VS Code (version 1.71.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.48.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.6 21G115 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 105.0.5195.125

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

@lore-co Different issues need to be reported as new issues, rather than comments in unrelated issues.

Facing the same issue
camera: ^0.10.0+4
flutter doctor -v
[✓] Flutter (Channel stable, 3.3.7, on macOS 13.0 22A380 darwin-x64, locale en-IL)
    • Flutter version 3.3.7 on channel stable at /Users/danielgah/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision e99c9c7cd9 (6 days ago), 2022-11-01 16:59:00 -0700
    • Engine revision 857bd6b74c
    • Dart version 2.18.4
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/danielgah/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 14.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14B47b
    • CocoaPods version 1.11.3

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

[✓] Android Studio (version 2021.2)
    • 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 11.0.12+0-b1504.28-7817840)

[✓] VS Code (version 1.70.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (3 available)
    • Mi A3 (mobile)  • 5d0ddb114eaa • android-arm64  • Android 11 (API 30)
    • macOS (desktop) • macos        • darwin-x64     • macOS 13.0 22A380 darwin-x64
    • Chrome (web)    • chrome       • web-javascript • Google Chrome 107.0.5304.87

[✓] HTTP Host Availability
    • All required HTTP hosts are available

Any news or advices to resolve this issue ?
Tyvm


HTTP Delete and Patch

Hi Timothy,
As far as I understand, the 'delete' and 'patch' (edit: and 'put' too it seems) HTTP methods do not work properly.
@hug.delete('/delete_me')
def delete_me():
    return

@hug.delete('/delete_me2')
def delete_me2():
    return {'msg': 'deleted'}

In both cases (and with 'patch' and 'put' methods too), I get the following error:
Traceback (most recent call last):
File "hug/api.py", line 381, in hug.api.ModuleSingleton.__call__.api_auto_instantiate (hug/api.c:10677)
File "[...]/versions/3.5.2/lib/python3.5/site-packages/falcon/api.py", line 189, in __call__
responder(req, resp, **params)
File "hug/api.py", line 280, in hug.api.HTTPInterfaceAPI.version_router (hug/api.c:8023)
File "hug/interface.py", line 581, in hug.interface.HTTP.__call__ (hug/interface.c:15515)
File "hug/interface.py", line 447, in hug.interface.HTTP.gather_parameters (hug/interface.c:11449)
File "hug/format.py", line 35, in hug.format.parse_content_type (hug/format.c:915)
TypeError: argument of type 'NoneType' is not iterable

I might must be doing something wrong though: if you could point me in the right direction, that'd be awesome.
I've also read on the web that a 'delete' HTTP method could send a response body.
Best.

HI @mhavel,
Agreed that this should work! I've tried copying and pasting exactly what's listed above and testing it with
curl -X DELETE http://localhost:8000/delete_me2 both on the server and client side everything worked flawlessly and I recieved the expected response {"msg": "deleted"}. Any more information you can provide? Any chance there is more code enough that is calling the failure?
Thank you!
~Timothy Crosley

Hi @timothycrosley,
Thanks for testing on your side. Yes there is some extra code, but I don't see what could cause the issue.
Actually, a closer code snippet to what I'm actually using is:
import hug
import numpy as np

auth = hug.http(requires=hug.authentication.basic(hug.authentication.verify('guest', 'letmein')))

@hug.local()
@auth.delete('/delete')
def delete(receipt:hug.types.smart_boolean=True, versions=1, examples='receipt=True'):
    """Delete method"""
    # processing stuff
    arr = np.random.sample(size=34)
    arr *= arr
    if receipt:
        return {'msg': 'deleted', 'arr_sq_mean': arr.mean()}
    else:
        return

When I test that locally on my computer, it works as expected indeed (both using hug -f and uwsgi). Not sure what is causing my issue in my production app, which API is not much more complex than the snippet above.
Anyway thanks for your input, if I understand the issue I'll let you know (if that's relevant to you).
Best,
Mathieu


system can not run successful

are you still on github?

a

Hey! Yes, I'm still alive. Can I help you?

thank you.
I have run the system successful by modifying the nmakefile in lots of module and linking the previous version fuzzy library.
Have you ever measure the performace of the adaptive-module?
Such as comparing the video quality between the adaptive-gaminganywhere and native-gaminganywhere when setting different network conditions include bandwidth、delay and loss?

I have run the system successful by modifying the nmakefile in lots of module and linking the previous version fuzzy library.

Ah i see, I have never modified nmakefile because I was only using visual studio's solution back then. I'm sorry 🤣

Have you ever measure the performace of the adaptive-module?
Such as comparing the video quality between the adaptive-gaminganywhere and native-gaminganywhere when setting different network conditions include bandwidth、delay and loss?

Uhm, I have never compared the performance of this adaptive version with vanilla gaminganywhere, but I did run it while I was tweaking network condition and captured it in action. I have linux box acted as router in the server side, so I can apply traffic control on them but not very specific and detailed like setting delay and loss individually. Actually, there's no loss at all, only limited bandwidth:
https://drive.google.com/file/d/0B_-gKYbRt3KbQVktVnZ4QWo0b0E/view?usp=sharing. (somehow the video is sped up because of misconfiguration of capturing software)
Well, it kinda worked in terms of adaptivity but the game's unplayable. There's also bug, when the network conditions recovered, the stream quality didn't restore back to where it should be.

Oh I see.
I run the ga-client and ga-server in two machine installed windows 10 OS , and using https://github.com/WPO-Foundation/win-shaper as network condition parameter tuning tool which can change the loss、bandwidth、delay on the network card.
And I compare the PSNR and SSIM between the YUV file generated by adaptive-gaminganywhere and native-gaminganywhere,
the results seem to show that adaptive-gaminganywhere did not get better performance than native-gaminganywhere.

Well, it kinda worked in terms of adaptivity but the game's unplayable. There's also bug, when the network conditions recovered, the stream quality didn't restore back to where it should be.

emmmm，i also found this problem, was it caused by fuzzy rule setting？I will try to modify the fuzzy rule to see if it can solve this problem and get better peformance than native-gaminganywhere.
Glad to receive your replay:smiley:


Feature Request: Text cell with optional value selection by dropdown

Hi, first of all thank you for this great plugin, I used it a lot as jQuery plugin and appreciate the port as vanilla js.
What I'm not happy with is the way how dropdowns are used. Currently, the user will open the dropdown, choose one of the options and therefore set the cell's value to the selected option. But if the dropdown misses an option, the user has no way to set the cell to his desired value.
To solve this problem, I modified the behaviour of a dropdown when text is entered. Instead of opening the dropdown, openEditor and later on closeEditor are called with an extra argument, which triggers the default text edition state. After closing, I update the source and remove duplicates.
Is there a reason why dropdowns are implemented as "One out of N values" ? If yes, I would love to hear about the reasoning. If not, I could open a PR after merging to v3 and some polishing.

I am going to add a feature in the roadmap of Jsuites. The possibility to add a new value in your dropdown.

This has been implemented, you can allow the user to enter new options, remoteSearch, and many other options.


Help pls

Describe your problem here.

Its my frist nod, i need help
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No response

From what I see in the error, There is a problem in the instrumental. You may need to change the Instrumental with a newer version of the song.


Form validation Input type email regex

bootstrap 4.0 -> validation
https://getbootstrap.com/docs/4.0/components/forms/#validation
Input -> type -> email -> Value = x@x  is valid -> form submit -> show div class -> valid-feedback
regex used in type=”email” from W3C:
  /^[a-zA-Z0-9.!#$%&’*+/=?^_{|}~-]+@[a-zA-Z0-9-]+(?:.[a-zA-Z0-9-]+)*$/ `

Operating system  (Windows 10 x64)
Browser  (Chrome Versión 64.0.3282.167 (Build oficial) (64 bits) )

so what's the point you're trying to make? the validation section doesn't include an email example (and validation itself uses the browser's built-in validation, so it's up to the browser to run appropriate regexp behind the scenes to then set the correct valid/invalid pseudoclass that bootstrap then styles.

Value = x@x is valid

you are correct

regex used in type=”email” from W3C:
/^[a-zA-Z0-9.!#$%&’*+/=?^_{|}~-]+@[a-zA-Z0-9-]+(?:.[a-zA-Z0-9-]+)*$/

regex matches 'x@x'
Official w3c email regex tester:
https://www.regextester.com/97767
https://en.wikipedia.org/wiki/Email_address#Examples
valid:   'user@localserver'


Identical names of Datasets from different databases running the same application

Hello,
I'm very new to Superset, so please forgive me if it's a newbie question or has been asked a lot.
I'm testing Superset with a couple of databases that all have the same type of application writing data to them. So all the databases obviously have the same tables in them.
Now after I added two databases and tables as datasets, I get Datasets with identical names, and I can't see a way to differentiate between them. When I want to create a new chart and want to choose a dataset, they're indistinguishable:

Is there any way to give the Datasets a specific name or similar, to differentiate them?
Thank you for your insights!

Thanks for suggesting!
@betodealmeida Hey Beto, I think this request is valid, can we improve? Quick solution could be prompting user to edit dataset name when same name is detected when dataset is added. wdyt?

@Retrospector we have in our roadmap to improve the dataset experience, and the first thing is to allow people to name their datasets, instead of using the table name.

Hey @junlincc and @betodealmeida !
I appreciate your feedback.
I'll keep a close eye on the roadmap then :)
Thank you for your work on Superset!


Call it "Cleanarr"!

That's it. Not really an issue, but just a thought.  ;)
Either way - great work, I'm gonna give this a go now. :D

bump

Haha I love it! I only worry that if I name it in-line with the bigger players in the OSS home media game (the 'arr' family) I don't have the capacity to keep up with the demand for (the admittedly few) feature requests from the community. E.g. There's already a few feature requests and issues, but I've been a pretty busy with life stuff for the last few months so unfortunately I haven't had the time to action any of them.
Anyway, I love the name suggestion! I'm going to change it accordingly when I get a minute to create a new dockerhub project and update the buildscripts etc.

Haha I love it! I only worry that if I name it in-line with the bigger players in the OSS home media game (the 'arr' family) I don't have the capacity to keep up with the demand for (the admittedly few) feature requests from the community. E.g. There's already a few feature requests and issues, but I've been a pretty busy with life stuff for the last few months so unfortunately I haven't had the time to action any of them.
Anyway, I love the name suggestion! I'm going to change it accordingly when I get a minute to create a new dockerhub project and update the buildscripts etc.

Awesome! Can't wait! :D
As far as implementing requests - I'm currently also a bit tied up with some other projects, but once those are completed, I'd be happy to lend my knowledge of Plex and Python to this project to help where I can.

D8ahazard aint you the guy that was making Flex.tv?
Also great app. im making a templete atm for the unraid CA store. If you want to make a official one ill gladly take it down :)

Yes sir, that's me. Unfortunately, some API changes mean the app doesn't
currently work. Rewriting the whole thing in c# is another item on my list.
Currently finishing a project called "Glimmr", and then I'll probably start
working on Flex TV again. ;)
On Sat, Aug 7, 2021 at 5:24 PM Alphacosmos @.***> wrote:

D8ahazard aint you the guy that was making Flex.tv?
Also great app. im making a templete atm for the unraid CA store. If you
want to make a official one ill gladly take it down :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/se1exin/Plex-Library-Cleaner/issues/30#issuecomment-894713191,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMO4NFYFIZ4QMSC52ZMDMTT3WXCXANCNFSM5BWSXYPA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

@d8ahazard It's done, the project is now named 'Cleanarr'. Thank's for the great name 😄
The new docker project can be found here: https://hub.docker.com/r/selexin/cleanarr
I've also ensured that the old plex-library-cleaner repo continues to receive updates for users that don't see this update message

i personally dont agree with the rename, i don't really wanna be a gatekeeper for the "arr" family but this repo seems to specifically only work with plex (which isn't even a part of the "arr" family) and doesn't integrate at all with sonarr/radarr/etc.
If that is a plan for the future then i am of course hopeful 🙂


maybe need to modify session garbage collector section. (FileHandler)

Actually, I saw similar another issue.
https://github.com/codeigniter4/CodeIgniter4/issues/942
I tested it.
I use php 7.1.
and below is some config value I use.
gc_maxlifetime = 7200  (2 hour)
gc_probability = 50   // (50%)
gc_divisor = 100
sid_bits_per_character = 5  (php auto config)

I think that maybe need to modify session garbage collector section.
Session/Handlers/FileHandler.php
public function gc();
original code is ...
$pattern = sprintf(
'/^%s[0-9a-f]{%d}$/', preg_quote($this->cookieName, '/'), ($this->matchIP === true ? 72 : 40)
);
but, why 40? I can't understand this.
for example, one of my session file name is a_sessionba2o0ekqqv8ogvqonrl2u0uqg86so8fm
a_session is cookiename.
so, session id is "ba2o0ekqqv8ogvqonrl2u0uqg86so8fm". the length of this string is 32.
So, actually, in my case 32 is correct value.
and I found this code part.
in Session/Session.php
protected function configureSidLength();
after be called this function, sidRegexp value is, "[0-9a-v]{32}" (in my case)
So, I tried to modify gc part code.
Here are some code. (Of course, this is incomplete code. I hope CI developer modify it properly)
PHP Code:
   $sid_regexp = '';
    $bits_per_character = (int) (ini_get('session.sid_bits_per_character') !== false ? ini_get('session.sid_bits_per_character') : 4);
    $sid_length = (int) (ini_get('session.sid_length') !== false ? ini_get('session.sid_length') : 40);
    switch ($bits_per_character) {
        case 4: $sid_regexp = '[0-9a-f]'; break;
        case 5: $sid_regexp = '[0-9a-v]'; break;
        case 6: $sid_regexp = '[0-9a-zA-Z,-]'; break;
    }
    $sid_regexp .= '{' . $sid_length . '}';
    $pattern = sprintf( '/^%s%s$/', preg_quote($this->cookieName, '/'), $sid_regexp );

    /* // Original Code
    $pattern = sprintf(
            '/^%s[0-9a-f]{%d}$/', preg_quote($this->cookieName, '/'), ($this->matchIP === true ? 72 : 40)
);*/

Anyway, after I modified it, Finally My php Garbage Collector could delete garbage session files.
Actually there is no problem in CI 3 FileHandle gc section.
I think CI3 already used code that similar to above code.
So, please consider it, check this point.

Your solution seems pretty thorough. I'm all for including this fix. Care to submit a PR for it?

@lonnieezell Thank you! but I'm just a beginner developer. Actually I don't know how to PR or test a part of code. besides I don't understand that matchIP part.
So, I'm so sorry to bother you. but could you please consider it?

Who said I understood the matchIP part? lol That's all code from CI3 that I didn't touch.
That's no problem, though, I'm not sure how long it will be until it gets fixed up. Will take a bit of a deep dive into your solution to ensure I understand all of the details, then figure out how to test it - which might be a bit tricky....

@lonnieezell  I agree. it might be not so easy. Anyway I can wait for your work. So, Cheer up!!


Parsing multiple statements

I'm looking through the code, and also looking through the version in OCaml on the LLVM wiki, and I can't see where you parse multiple program statements. Is it possible to explain how you did it, as I'm trying to parse multiple statements at the moment, and am totally stuck.

Each evaluate call parses a single statement with self.parser.parse_toplevel


Backport pytest per-broker overrides to 2.0

Closing without merging since this commit was merged as part of https://github.com/apache/kafka/pull/8347


Desktop view missing button

Looks fine on mobile and tablet. But when opened with Firefox and MS Edge using Gmail the link shows up as plain text:

Thanks @tkwon. Trying to reproduce. Can you show me the full HTML you're sending? And to confirm, using latest Firefox in Gmail?

I cut and paste directly from your email.html. I'm attaching a copy for your review.
I used Firefox 48.02 as well as Microsoft Edge 25.10586.
Unfortunately I can't get Chrome to run on my computer due to other issues.
I did verify the email works fine on desktop using GoDaddy email. So it seems to be related to Gmail.
email_confirm.txt

FYI, I also found out the button doesn't appear when using mail from zoho.com

Are you inlining the CSS before sending? You either need to use the template that is inlined email-inlined.html or your ESP needs to inline your CSS for you before sending.
https://github.com/leemunroe/responsive-html-email-template#sending-emails-using-a-marketing-service-like-campaign-monitor-or-mailchimp

Ok thats most likely the problem. I'm using django fyi and sending the email myself.

Ah gotcha. I'm not familiar with django but try this.


[0.7.3 W7x64] Sometimes my Date Modified is NULL/empty

In very rare cases some of my downloaded files do not have a Date Modified property. It is null or empty and subliminal crashes on it.
'Traceback (most recent call last):
File "C:\Python27\Scripts\subliminal-script.py", line 9, in
load_entry_point('subliminal==0.7.3', 'console_scripts', 'subliminal')()
File "build\bdist.win-amd64\egg\subliminal\cli.py", line 146, in subliminal
File "build\bdist.win-amd64\egg\subliminal\video.py", line 325, in scan_videos
ValueError: timestamp out of range for platform localtime()/gmtime() function'

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/20201455-0-7-3-w7x64-sometimes-my-date-modified-is-null-empty?utm_campaign=plugin&utm_content=tracker%2F321252&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F321252&utm_medium=issues&utm_source=github).

Seems its back?
ERROR:subliminal.cli:Unexpected error while collecting directory path /media/dns323/TVSERIES
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/subliminal/cli.py", line 334, in download
    scanned_videos = scan_videos(p, age=age, archives=archives)
  File "/usr/local/lib/python2.7/dist-packages/subliminal/core.py", line 495, in scan_videos
    if age and datetime.utcnow() - datetime.utcfromtimestamp(os.path.getmtime(filepath)) > age:
ValueError: year is out of range

This is on a file with date modified = NULL

What version are you using?

Issue spotted here: https://github.com/Diaoul/subliminal/blob/master/subliminal/core.py#L494-L497

Version 2.0.3


Kubernetes User Guide shows RCs but not Deployments

This is a...

[ X] Bug Report

Problem:
Kubernetes User Guide shows RCs but not Deployments.
Proposed Solution:
Update the user guide to show deployments.
Page to Update:
http://kubernetes.io/docs/user-guide/

Closing because this issue is not pointing us to a specific section of the website to fix. After a quick 'grep' over the website, I'm unable to identify sections where we are using RC while Deployment is more appropriate. Please feel free to reopen this if there is a case we are missing.


Invalid parsing syntax in HTML tag

I have following text:
<div class="alert alert-warning">
**bold**

# h1
## h2

> quote1
> quote2

* uno
* due

1. one
2. two

[foo.bar](http://foo.bar)
</div>

And parer won't parse syntax into div wrapper.

Expected. See other reports.

If Parsedown is aiming to be CommonMark compliant then this is a bug.
Though a single blank line should follow the initial <div> in order for the first **bold** to be parsed correctly.


CB-10652 Added the instruction on how to prepare a slave for Appium t…

…ests
https://issues.apache.org/jira/browse/CB-10652

@dblotsky please take a look

Will the iOS steps always be necessary, or will they become unneeded later? For Android, do you have a link to the bug that when fixed will obsolete this Android setup?

@dblotsky iOS steps will always be necessary only if you want to run the tests on real device.
You don't need to run these steps to run the tests on the simulator.

Also, added the links to the Appium issues describing the problem and the workaround for the chromedriver issue.


[Gradient Buttons] -> [Online Design Tools]

Gradient Buttons
Ready to use copy/paste gradient buttons
Link: https://gradientbuttons.colorion.co/
Checklist:

[x] I have performed a self-review of submitted resource and its follows the guidelines of the project.

Hey @ckissi, I think it would fit better in the colors section. All the links on the site are also related to colors and gradients too.


ci: add GitHub token permissions for workflows

This PR adds minimum token permissions for the GITHUB_TOKEN in GitHub Actions workflows using https://github.com/step-security/secure-workflows.
GitHub recommends defining minimum GITHUB_TOKEN permissions for securing GitHub Actions workflows

https://github.blog/changelog/2021-04-20-github-actions-control-permissions-for-github_token/
https://docs.github.com/en/actions/security-guides/automatic-token-authentication#modifying-the-permissions-for-the-github_token
The Open Source Security Foundation (OpenSSF) Scorecards treats not setting token permissions as a high-risk issue

This project is part of the top 100 critical projects as per OpenSSF (https://github.com/ossf/wg-securing-critical-projects), so fixing the token permissions to improve security.
Before the change:
GITHUB_TOKEN has write permissions for multiple scopes, e.g.
https://github.com/rubocop/rubocop/runs/7939389109?check_suite_focus=true#step:1:19
After the change:
GITHUB_TOKEN will have minimum permissions needed for the jobs.
Signed-off-by: Varun Sharma<EMAIL_ADDRESS>
Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[ ] Squashed related commits together.
[ ] Added tests.
[ ] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[ ] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

@rubocop/rubocop-core Anyone familiar with these token permissions to provide a review?

There seem to be some similar PRs opened and I found a discussion at Node.js repository.
https://github.com/nodejs/node/pull/43743
It seems that there are pros and cons, and it is difficult for me to judge whether or not to merge this. And I have no curiosity and opinion about this yet.

@varunsh-coder ping :-)

Sorry for the delay. I will do this today.

@varunsh-coder ping :-)

Sorry for the delay. I will do this today.

Done.

Thanks!


The TemporalUpscaler did not appear

The game is RE4 Remake. After placing the latest versions of the dinput8.dll file from the pd-Upscaler branch and PDPerfPlugin.dll file into the game's root directory, why doesn't REFramework display TemporalUpscaler in the game? Can someone tell me the correct installation method? Thank you very much.

I think the Ada DLC completely broke the temporal upscaler functionality and it's been like that since then. I never got it to work either with the latest version of the game.

Use the pd-upscaler builds from the Actions page, they are compiled automatically.

Has something been broken about this again? I have dinput8.dll, PDPerfPlugin.dll and nvngx_dlss.dll in the same folder where re4.exe is and the TemporalUpscaler option does not show up. I used the latest nightly build of REFramework (https://github.com/praydog/REFramework-nightly/releases/tag/nightly-998-f2254d2cf45574f1385c189a559644c5c8d59a89) and I'm playing the Steam version of the game.


Paragraph text box

How can I get a text field box that looks like a paragraph text field box, so users can fit more text into the box?

@ebenezer103 do you mean a TextView?

yes that exactly please

@ebenezer103  Use addTextView() instead of addTextField()

Worked perfectly, thank you for your help. I know you are going to close the issue, but how would I go about changing the overall colour from the default colour to another colour.

From SCLAlertViewExample/ViewController.swift:179 - 182
`let icon = UIImage(named:"custom_icon.png")
let color = UIColor.orange
    _ = alert.showCustom("Custom Color", subTitle: "Custom color", color: color, circleIconImage: icon!)`

Is this what you're after?

            let appearance = SCLAlertView.SCLAppearance(
                kTitleFont: UIFont(name: "HelveticaNeue", size: 0)!,
                kTextFont: UIFont(name: "HelveticaNeue", size: 12)!,
                kButtonFont: UIFont(name: "HelveticaNeue-Bold", size: 14)!,
                showCloseButton: false
            )

            let alert = SCLAlertView(appearance: appearance)

            // Add a text field
            //let alert = SCLAlertView()
            let txt = alert.addTextView()
            alert.addButton("Done") {
                print("Text value: \(txt.text!)")
            }
            alert.showEdit("", subTitle: "What can the organisation improve on?")

This is my code so far when I attempt to implement it and place the code you gave me into it, it doesn't work.

            let appearance = SCLAlertView.SCLAppearance(
                kTitleFont: UIFont(name: "HelveticaNeue", size: 0)!,
                kTextFont: UIFont(name: "HelveticaNeue", size: 12)!,
                kButtonFont: UIFont(name: "HelveticaNeue-Bold", size: 14)!,
                showCloseButton: false
            )

            let alert = SCLAlertView(appearance: appearance)

            // Add a text field
            //let alert = SCLAlertView()
            let txt = alert.addTextView()
            alert.addButton("Done") {
                print("Text value: \(txt.text!)")
            }
            alert.showEdit("", subTitle: "What can the organisation improve on?")

This is my code so far when I attempt to implement it and place the code you gave me into it, it doesn't work.

You're trying to use showEdit which will give a default colour, and a default image.  If you want to preserve the edit icon you can use this instead, with the extension below.
let customColor = UIColor.green
 _ = alert.showEdit("", subTitle: "What can the organisation improve on?", colorStyle: customColor.colorAsUInt)

// This extension was extracted from SCLAlertView.swift:689
extension UIColor {
    var colorAsUInt: UInt {
        var red: CGFloat = 0, green: CGFloat = 0, blue: CGFloat = 0, alpha: CGFloat = 0

        getRed(&red, green: &green, blue: &blue, alpha: &alpha)

        var colorAsUInt32 : UInt32 = 0
        colorAsUInt32 += UInt32(red * 255.0) << 16
        colorAsUInt32 += UInt32(green * 255.0) << 8
        colorAsUInt32 += UInt32(blue * 255.0)

        return UInt(colorAsUInt32)
    }
}

Thank you very much, this works!


Helm chart not uninstalling custom resources

I've been installing the operator using terraform to install using your helm chart and I'm finding some problems running a terraform destroy - it eventually times out because it can't delete the namespace. This seems to be due to the fluentbits custom resource finalizer - if I patch the resource to remove the finalizer, the deletion proceeds as expected.
To take terraform out of the equation, I created the namespace by hand and then ran a helm install, using the same values, followed by helm uninstall - this deleted the operator pod but not the daemonset, nor the fluentbits custom resource. The helm release is deleted, though, so cleanup is to delete the namespace, then patch the fluentbits resource again.

Helm does not remove resources such as CRD

There is no support at this time for upgrading or deleting CRDs using Helm. This was an explicit decision after much community discussion due to the danger for unintentional data loss. Furthermore, there is currently no community consensus around how to handle CRDs and their lifecycle. As this evolves, Helm will add support for those use cases.
https://helm.sh/docs/chart_best_practices/custom_resource_definitions/

And fluentbit is not deployed by helm, it is generated by Fluentbit-Operator.

Aaaah, OK, of course - d'oh! Thanks @wenchajun...


#197 Updated the Home view to look like the vue version.

Signed-off-by: Ed Snodgrass<EMAIL_ADDRESS>Contributes to # (add issue number here. i.e. #302)
What did you do?
Added a home view with a hero image with text overlayed.
Why did you do it?
Last piece of React Admin.
How have you tested it?
Loaded the page
Were docs updated if needed?

[ ] No
[ ] Yes
[ ] N/A

Type of change

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

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] My changes generate no new console logs
[ ] Fixes entire issue
[ ] Partial fix for issue

Hi @ed-snodgrass,  You probably noticed already.  But FYI... failed tests.

Hi @ed-snodgrass, You probably noticed already. But FYI... failed tests.

I had not noticed. Thank you @markstur and I apologize for that. I've re-added the testid and we should be good to go.


Feat: EyeTracker class

Description
Added an EyeTracker class that can keep parsed eyetracker related data.
Fixes issue #597
Implemented changes

[x] Create class EyeTracker
[x] Use class in Experiment
[x] Add short-access to Experiment.eye_tracker.sampling_rate from Experiment

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change is or requires a documentation update

How Has This Been Tested?

[x] Test class tests.unit.gaze.eyetracker_test.py

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[ ] I have checked my code and corrected any misspellings

#618 fixes the docs problems


Ask for advice for att2in2 under scst training

Hi. I trained att2in2 model with the default settings, but I got a lower score than https://github.com/ruotianluo/ImageCaptioning.pytorch/issues/10。
Here is my result:
Bleu_1: 0.796 Bleu_2: 0.622 Bleu_3: 0.471 Bleu_4: 0.351 ROUGE_L: 0.561 CIDEr: 1.118
and result in https://github.com/ruotianluo/ImageCaptioning.pytorch/issues/10:
Bleu_1: 0.777 Bleu_2: 0.613 Bleu_3: 0.465 Bleu_4: 0.347 ROUGE_L: 0.560 CIDEr: 1.156
And this how I trained the model:
(1) pretrained att2in2 for 25 epochs with the same settings (the same spatial feature of image, the same batch size, schedule sampling strategy from 0, the same learning rate decay, and so on), and I obtained comparable results with yours.
(2) then I trained it with scst for another 35 epochs. Learning rate was fixed to 5e-5. The cache for computing CIDEr is coco-train-idxs.
Compared with your result, the CIDEr is worse, but others metrics are better. The result bothers me a little bit, which makes me doubt about my experiment settings.
Is there anything trivial details I missed?
I wonder the schedule-sampling used in pretrained model will affect the exploration of the RL, but I have not had it a try. Any advice will be appreciated. Thanks a lot.

I did find if you start scst at different times, the performance  will be different in such way. (Higher on other metrics and lower on cider)
I actually forgot what exact setting I use. Try to start scst at 30 epochs or 35 epochs?

Ok. Thanks a lot. I will keep trying. It really takes too much time to train the model with RL. Sigh.


How to install?

Sorry guys I am not an expert in working with github a lot.
Ubuntu 24.04 would be my OS of choice

I haven't tried with v24, but it works for me on v22.

Clone the repository.
Create a config.jsonc file in the root directory. Copy all data from config-sample-v0.7.dev.jsonc and replace it with your values:

data_folder
telegram_chat_id
telegram_bot_token
api_secret
api_key

Install all required packages.
Run the following commands:

python -m scripts.download_binance -c config.jsonc
python -m scripts.merge -c config.jsonc
python -m scripts.features -c config.jsonc
python -m scripts.labels -c config.jsonc
python -m scripts.train -c config.jsonc
python -m scripts.signals -c config.jsonc
python -m scripts.train_signals -c config.jsonc

Start the bot with python -m service.server -c config.jsonc

P.S. It's important to understand how the code works. During installation, you may encounter some errors, so it’s best to investigate how each script functions.


page 737 equations 22.8 &22.9

I think u_u and v_i  next to e_{ui} are swapped between equations 22.8 and 22.9
I believe they should read
u_u = u_u + \rho(e_{ui} v_i - \lambda u_u)
and
v_i = v_i + \rho(e_{ui} u_u - \lambda v_i)

fixed, thanks

you're most welcome. love the books, thx for writing them.

you're most welcome. love the books, thx for writing them.
On Fri, Apr 29, 2022 at 11:32 PM Kevin P Murphy @.***>
wrote:

fixed, thanks
—
Reply to this email directly, view it on GitHub
https://github.com/probml/pml-book/issues/336#issuecomment-1113903499,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXJPETUYMR6FLCGXPXVUAGDVHSST3ANCNFSM5UWDI35Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Rob Haslinger
Director, Data Science
@.*** |<PHONE_NUMBER>
InMarket http://inmarket.com | Linkedin
https://www.linkedin.com/company/inmarket_1/ | Twitter
https://twitter.com/inmarket_media
Maximize the Moments That Matter
https://inmarket.com/press/


improve typings

Codecov Report

Merging #76 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #76   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          16     16
  Lines         281    281
  Branches       54     54
=====================================
  Hits          281    281

Impacted Files
Coverage Δ

src/client/client.ts
100% <100%> (ø)
:arrow_up:

src/cache/cache.ts
100% <100%> (ø)
: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 b85a323...dd36d42. Read the comment docs.

Codecov Report

Merging #76 into master will not change coverage by %.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master       #76   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           16        16
  Lines          281       281
  Branches        54        54
=========================================
  Hits           281       281

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 b85a323...dd36d42. Read the comment docs.


Document usage with wp.com wp/v2 API endpoints

This issue was raised in Gitter chat:

Out of the box, if I got a Wordpress hosted at the "official" wordpress_com site, is it's API level ok with node-wpapi ?

The built-in WordPress REST API is supported by WordPress.com through their public-api.wordpress.com site; the base URL for endpoints for a given site is e.g.
https://public-api.wordpress.com/wp/v2/sites/your_site

where your_site is a string like "kadamwhite.wordpress.com". Then you can request collections and resources as normal:
https://public-api.wordpress.com/wp/v2/sites/kadamwhite.wordpress.com/posts
https://public-api.wordpress.com/wp/v2/sites/kadamwhite.wordpress.com/tags

Because of the structure of this URL, we have to do a bit of adjustment to get our client to generate the proper URL format. The simplest way is to chain a .namespace() call onto our requests:
import WPAPI from 'wpapi';
const wpcom = WPAPI.site('https://public-api.wordpress.com');
wpcom.posts().namespace('wp/v2/sites/kadamwhite.wordpress.com').get()....

this is a bit verbose, so you could make a helper method to reduce the duplication:
function withSite( site ) {
  return function( request ) {
    return request.namespace( site );
  };
}
const withMySite = withSite( 'kadamwhite.wordpress.com' );

// Now wrap requests in `withMySite` before using .get() or .then():
withMySite( wpcom.posts() ).get()....

Alternatively, you should be able to download the API root for your site and use that to bootstrap a client specific to your own .com site.
Note that authentication with the WordPress.com-hosted version of the WordPress REST API is untested, and probably not feasible at this time, since WordPress.com sites provide no plugins for authentication and arbitrary custom plugins cannot run on WordPress.com.
This issue tracks moving this documentation into the README.

FWIW - first time wpapi user here and google brought me here. It would be helpful to include a working snippet.  It's obvious now, but at first I didn't realize what I needed to do.
Example of how to list posts at wordpress.com blog using basic auth:
const WPAPI = require('wpapi');

const wpcom = new WPAPI({
  endpoint: 'https://public-api.wordpress.com', // not your blog domain
  username: 'your-blog-username',
  password: 'your-user-password'
});

wpcom.posts().namespace('wp/v2/sites/mywordpresscomblog.wordpress.com')
  .get()
  .then(console.log)
  .catch(console.error);


Repo sync for protected CLA branch

The pull request is created from main637879034638103791 to main to fix git push error for protected CLA branch

@opbld16 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Revert "chore: upgrade test should use VERSION by default (#4393)"

Revert this for now to release v1.2.0-rc.1
https://github.com/envoyproxy/gateway/pull/4476#issuecomment-2436776156

why revert?

#4393 fix other problem, you'd better send another one:

revert the change in kube.mk
chagne lastVersionTag to "v1.1.2"

we are going to always hit this issue when releasing, we can't revert each time, I would instead hardcode this in the test, and add it as a step post releasing the bits

we are going to always hit this issue when releasing, we can't revert each time, I would instead hardcode this in the test, and add it as a step post releasing the bits

we should change the logic if version == "" || strings.Contains("rc")

oh, same when releasing 1.2.0.

Try to fix it with https://github.com/envoyproxy/gateway/pull/4524

Try to fix it with #4524

This only works for the main branch. If we try to release a minor release like v1.1.3 after v1.2.0, it will use v1.2.0 as for upgrade test.
I'll update the version in upgrade test after releasing v1.2.0 and use that PR as an example to add a step in the release note.


Tact 2.0 RFC

This is a very incomplete draft proposal for the next major version of Tact. It's basically a couple of grammar bullet points so far. Comments are welcome.
Grammar

Do not allow using struct, message, contract, init, bounced, external, primitive, map, and receive, get (or whatever we change those to) as identifiers and make those keywords, e.g. let struct: Int = 0 will become illegal;
Commas (,) should separate fields in in struct/message declarations to make it consistent with struct/message definitions;
Compact Int type ascriptions for contract for storage variables: e.g. x : int32 instead of x : Int as int32;
Getter definitions should make it clear that getters are not accessible on-chain (see https://github.com/tact-lang/tact/issues/249#issuecomment-2053648197);
Internal and external message receivers syntax should be more consistent (see https://github.com/tact-lang/tact/issues/9#issuecomment-1999887650);
Writing to storage variables should look differently compared to analogous operations on temporary (stack) variables, e.g. storage_var <- expression;
Contract storage variables should be accessed using a keyword other than self, for example, storage.var (we might even want to hint at it syntactically with storage { var : Type ... } declaration inside a contract);
More concise syntax for map expressions and operations: map literals ({1: "foo", 2: "bar"}), map access (map[key]), map updates (map[key] = val and map[key] += inc, etc.);
Capitalize map type identifier: Map<K, V>;
Equality comparisons via hashing should be explicit. We compare cells and slices via hashing but use == / != which hides this fact. Switching to something like ==# / !=# makes it more obvious;
Syntactic support for sending messages, e.g. instead of send(SendParameters{value: amount, to: self.owner, mode: mode}) we can have something like send {value: amount, to: self.owner, mode: mode};

Semantical changes

Disallow initializing maps with emptyMap by default, in other words m: map<Int, Int>; without the corresponding entry in init should produce a compilation error;
Disallow map comparisons with == and !=: it should be evident that these are not trivial operations in terms of gas consumption;
Send mode should get its own type, not Int;
Fix SendPayGasSeparately naming, see #149;

Tooling

Tool to help migrate Tact v1 projects to Tact v2

Couple of ideas:

https://github.com/tact-lang/tact/issues/9#issuecomment-1999887650
Promote get attribute to be it's own function type to make it clear that they're special:

contract Example {
    field: Int;

    // Like this (removed `fun` keyword here):
    get field(): Int {
        return self.field;
    }

    // Or maybe even like this, to make it clear that they're off-chain
    // (and would therefore be called "offchain functions" and not "getter functions"):
    off getField(): Int {
        return self.field;
    }
    offchain getField(): Int {
        return self.field;
    }
}

Commas (,) should separate fields in in struct/message declarations to make it consistent with struct/message definitions;

Not sure here, as this will also influence fields (persistent state variables) in contracts and traits — they use semicolons in their declarations as of now

Not sure here, as this will also influence fields (persistent state variables) in contracts and traits — they use semicolons in their declarations as of now

It's fine, though. Syntactically it's included between a pair of curly braces, so there won't be any grammar conflicts. And there is a nice principle saying that declarations and definitions should resemble each other. And it makes it resemble TS more, which is what we target in terms of syntax

Added a couple more bullet points:

Writing to storage variables should look differently compared to analogous operations on temporary (stack) variables, e.g. storage_var <- expression;
Contract storage variables should be accessed using a keyword other than self, for example, storage.var (we might even want to hint at it syntactically with storage { var : Type ... } declaration inside a contract);

More concise syntax for map expressions and operations: map literals ({1: "foo", 2: bar}), map access (map[key]), map updates (map[key] = val and map[key] += inc, etc.)

Syntactic support for sending messages, e.g. instead of send(SendParameters{value: amount, to: self.owner, mode: mode}) we can have something like send {value: amount, to: self.owner, mode: mode};

Disallow initializing maps with emptyMap by default, in other words m: map<Int, Int>; without the corresponding entry in init should produce a compilation error;

Tool to help migrate Tact v1 projects to Tact v2

We can use ast-grep, which uses Tree-sitter to perform AST-based search and replace. Additionally, it can lint code with the visually same error reports like Rust does, provided that we specify linting rules — this can be a starting point for our linters.
Apart from CLI, it also has a Node.js binding @ast-grep/napi with jQuery-like utility methods to traverse and manipulate syntax tree nodes

@novusnota I was thinking we can have a tool that combines Tact v1 parser and Tact v2 code formatter: this way we can migrate contracts if we only have local syntactic changes. Plus we can do some more massaging of the original contract, like explicit map initializations.

@anton-trunov Oh, so you want for code formatter to not only format, but also perform lint-like fixes?

Do not allow using struct, message, contract, init, bounced, external, primitive, map, and receive, get (or whatever we change those to) as identifiers and make those keywords, e.g. let struct: Int = 0 will become illegal;

Oh, so you want for code formatter to not only format, but also perform lint-like fixes?

@novusnota Not at all, I was talking about creating a tool like tact-migrate-v1-to-v2 which would combine the parser for Tact v1 and the future code formatter for Tact v2 + plus some more automatic migrations that are possibly not purely syntactical

Disallow initializing maps with emptyMap by default, in other words m: map<Int, Int>; without the corresponding entry in init should produce a compilation error;

Why? I have multiple use cases where I need to declare an empty map without adding any entries in init (eg. RBAC).

Why? I have multiple use cases where I need to declare an empty map without adding any entries in init (eg. RBAC).

Semantically nothing will be changed here, the user will have to just explicitly state the initial value for map. This part should be migrated automatically

Any plans to revive this? https://github.com/tact-lang/docs-obsolete/blob/main/tact-design.md#interfaces
I'd love to see impl MyTrait for MyStruct { }

Yep, we definitely need adhoc polymorphism in Tact

Not sure here, as this will also influence fields (persistent state variables) in contracts and traits — they use semicolons in their declarations as of now

It's fine, though. Syntactically it's included between a pair of curly braces, so there won't be any grammar conflicts. And there is a nice principle saying that declarations and definitions should resemble each other. And it makes it resemble TS more, which is what we target in terms of syntax

Well, as struct and message are essentially declaring a new type and we're trying to target TS, we should also consider that in TS type declarations use semicolons (like we do) and not commas, see:

Albeit it's just a suggestion — do we need a bounced<> type wrapper going forward? Maybe, having the bounced() receiver is clear enough?

... as struct and message are essentially declaring a new type ...

Regarning struct vs. message — users of Tact may have a confusion between message as type declarations and messages as means of contract interaction on the blockchain.
I'm not sure if that's a valid issue (need to gather community feedback here), but it may be beneficial to merge struct and message declarations under the same name, say, struct or even tuple, to later make it closer to an advanced successor of TVMs tuples.
That way we could clarify things and make it simpler to read and write Tact code. And that would also alleviate the need to capitalize Messages/Structs in the docs to refer to the types and not to the communication itself :)

I think the current definition of struct and message is clear enough and they offer clear separation of usage scenarios.
If we were to merge the two then what would become message(0x1234) M {}? Does struct(0x1234) M {} even make sense?

Suggestion:
Get rid of multi-line /**/ comments. Instead, use variations of single-line comments //: //! as the top-level comments describing the current file (may be omitted, I guess) and /// (or regular //, for super-simplification!) as the documentation comments for the things under them (as it's done in Zig, Rust and, to some extent, Dart).
Motivation:

Current multiline doc-comments are, in fact, written as if they're a multiple single-line comments, because every line is prefixed by *. It's very redundant fact of JSDoc and JavaDoc, which removes the whole point (IMO) of having

It's much easier for users to type and maintain single-line // comments and series of them, especially without the advanced help from the editor. More comments, more documentation, more clarity!

It's much easier for us to NOT have multi-line comments, as they're the only token in Tact which can span multiple lines and completely change the list of tokens, and parsed AST of the rest of the file as soon as the beginning /* appears. Not having any multi-line tokens would significantly ease the lexing phase from the perspective of incremental updates (we can now lex and re-lex by lines, essentially making way for lexing in-parallel, like Zig does), but this would also improve the performance of lexing and parsing phases! Blazing-fast AST generation for compiler and for tooling.

Great suggestion! Looks like it's about time to move this issue to its own repo and continue discussions in separate issues there. Wdyt?

Well, yeah, this can be moved to tact-lang/roadmap or to some more RFC-specific repo and then referenced from this issue

We may try to add direct TVM assembly function wrappers, similar to how it can be done in FunC: RETALT example. This will remove the need to write them in FunC, and then import and wrap in a native function in Tact — we would just write those assembly things ourselves.
Perhaps, it's best to add another attribute to native functions in Tact, so that we have @asm("...") in addition to the existing @name(func_name_here):
@asm("RETALT")
native returnAlt();

// or even drop the string literal and allow users to write multi-line assembly right between the perentheses (or even replace those with braces `{}`):

@asm(
   RETALT
) native returnAlt();

@asm{
   RETALT
} native returnAlt();

A nitpick, but void → Unit rename for Tact could be nice. As a bonus, it would allow explicitly specifying such return type in functions. But, perhaps, we don't need to introduce return Unit;, just empty return;.
Kotlin, the language we name in the first commandment of Tact, has it:
https://kotlinlang.org/docs/functions.html#unit-returning-functions.

We can also consider removing the primitive type declarations entirely, since they provide no value to end-users and are rather useless to / replaceable in the compiler.

The discussion has been moved elsewhere for the time being.


fixes newlines and quoting in clusters crd. Fixes #124

This should be solved in cloudnative-pg if we want to, as tags are present in structs' descriptions themselves: https://github.com/cloudnative-pg/cloudnative-pg/blob/684e8eedeb8faa27dcaaf81723dc7138b7a98769/api/v1/cluster_types.go#L952
My guess is that it's to format it properly in the docs when we generate them with https://github.com/EnterpriseDB/k8s-api-docgen, which we are evaluating swapping out in favor of https://github.com/ahmetb/gen-crd-api-reference-docs


BUG

ValueError: Index Date_Time invalid

Hey @qq583 can you please provide more information on how to reproduce that bug?

When I run run_forecasting.py ,it prompt “Traceback (most recent call last):
File "D:/untitled2/forecasting/run_forecasting.py", line 54, in
df = pd.read_csv("data/AirQualityUCI.csv", index_col=config["index_col"])”，and “ValueError: Index Date_Time invalid”.
please tell me how to solve this problem. thank you !!!!

@qq583 if I go to the root folder and run:
python3 -m examples.forecasting.run_forecasting --do-train=True

Can you try to do that?

sorry i can‘t . The trouble is "df = pd.read_csv("data/AirQualityUCI.csv", index_col=config["index_col"])" .
and "ValueError: Index Date_Time invalid"

Have you downloaded the dataset?

Yes,but i don't know the title of the data. my data title is "

Date
Time
CO(GT)
PT08.S1(CO)
NMHC(GT)
C6H6(GT)
PT08.S2(NMHC)
NOx(GT)
PT08.S3(NOx)
NO2(GT)
PT08.S4(NO2)
PT08.S5(O3)
T
RH
AH

",is it true?

My bad the .csv wasn't the right one. Try again with the data from here

sorry , i can't download this data,could you put this data into this package ?
Thank you very much!!!!!!

There's a download zip button on the top right.

Thank you , my bro!!!


🚀 Feature Request: Making implement in readme.md file

🎤 Tell us your idea
I want to add information regading how to install the code zip file in the vs code .

hello @Piyushsahu99
we can guide the installing of zip code in vs code as:
download the zip file and extract the contents to a folder
open VS Code, and select File > open oolder to open the extracted folder OR we can directly go to folder and use Open with VSCode to open it in editor
if it's a Node.js project, run npm install or yarn install in the terminal to install dependencies
and then check package.json to check the run method either it is npm run dev or npm start and launch it
or we can add any additional instructions in the project's README to assist

Sorry, For the inconvenience but this is my first time 😔

hey @Piyushsahu99 it's okay there is first time for everyone , don't be sad , it is good to ask questions and solve doubts!! stay motivated


ipad上支持吗

是否已有关于该错误的issue或讨论？ | Is there an existing issue / discussion for this?

[ ] 我已经搜索过已有的issues和讨论 | I have searched the existing issues / discussions

该问题是否在FAQ中有解答？ | Is there an existing answer for this in FAQ?

[ ] 我已经搜索过FAQ | I have searched FAQ

当前行为 | Current Behavior
No response
期望行为 | Expected Behavior
No response
复现方法 | Steps To Reproduce
No response
运行环境 | Environment
- OS:
- Python:
- Transformers:
- PyTorch:
- CUDA (`python -c 'import torch; print(torch.version.cuda)'`):

备注 | Anything else?
No response

On IOS or ipadOS, the cli way is working correctly with the model using llamacpp, but we don't have the manpower to wrap it into swift, if you want to help support us we would appreciate it.


Feature paginator sum

A new sum-up behavior is added into paginator.  It will be triggered by a new setting in paginators-1.json named sum_keys (new naming suggestions are welcomed!).

I will work on a major refactor soon.  No review is needed for this one for now.

Replaced by PR #689


Removed fbprophet and pystan as dependency

Removed GlobalModel
Removed prohet from STLF
Removed from fbprophet and pystan in pyproject and requirements.txt
Removed test cases which are no longer needed

@sankarsiva123 Please review

Removed neuralprophet from pyproject.toml and requirements.txt


feat(ocppj): add logging of raw json messages

add logging for incoming and outgoing JSON messages to allow easier debugging.
All incoming JSON messages are logged as
<<< [JSON]
and all outgoing as
>>> [JSON]

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 87.293%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
6279

Relevant Lines:
7193

💛  - Coveralls


CAMEL-19375: Upgrade to angus-mail 2.0.1, test camel-mail against GreenMail instead of mock_javamail

Description

Upgrade to angus-mail 2.0.1
Test camel-mail against GreenMail instead of mock_javamail

Tracking

[x] https://issues.apache.org/jira/browse/CAMEL-19375

[x] I formatted the code using mvn -Pformat,fastinstall install && mvn -Psourcecheck

faf3f1e:

Reduced the public API of org.apache.camel.component.mail.Mailbox by not inheriting from ArrayList
Made Mailbox a bit more resistant against concurrency issues.

The angus-mail upgrade does not have any impact on camel-jms and when trying to reproduce locally, I see between 0 and 3 tests failing in camel-jms. Are the JMS tests known to be flaky?

A little bit as they run concurrently - @otavioprado is improving the tests

they run concurrently

Interesting, I did not know some tests were executed in parallel. BTW, with the current changes, it is doable to make the tests in camel-mail ready for concurrent execution too. The key change in the current PR is that each mailbox user and mailbox is explicitly present in the test class as a field. What is missing is to change the creation of the users and mailboxes to random usernames, email addresses and passwords so that they cannot clash with other tests executed parallely. Plus the users and mailboxes should be removed after the test execution. This is doable e.g. by implementing a Junit5 extension.

86e5586: Fixed formatting
I wonder why -Pformat is not active by default?

they run concurrently

Interesting, I did not know some tests were executed in parallel. BTW, with the current changes, it is doable to make the tests in camel-mail ready for concurrent execution too. The key change in the current PR is that each mailbox user and mailbox is explicitly present in the test class as a field. What is missing is to change the creation of the users and mailboxes to random usernames, email addresses and passwords so that they cannot clash with other tests executed parallely. Plus the users and mailboxes should be removed after the test execution. This is doable e.g. by implementing a Junit5 extension.

Yeah. We have been tweaking the JMS tests to reduce their flakiness. If you want to give it a try with the camel-mail ones, you need need to set the following properties on the pom:
<camel.surefire.reuseForks>true</camel.surefire.reuseForks>
<camel.surefire.forkCount>1</camel.surefire.forkCount>
<camel.surefire.forkTimeout>1200</camel.surefire.forkTimeout>
<camel.surefire.parallel>true</camel.surefire.parallel>
<camel.surefire.parallel.factor>0.5</camel.surefire.parallel.factor>

And then adjust the code to avoid using non-shareable resources when possible (or isolate the non-concurrent ones to a different execution).
Ideally, converting the code to use the JUnit 5 extension from camel-test-infra-core  instead of using CamelTestSupport from camel-test-junit5 can also help making them simpler and easier to maintain.


Bumped production development number for 1.27-2

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold

/lgtm


Golang bootstrap checks out to agent-%n directory

If you use the %n pattern in an agent name, the new golang bootstrap checkout gear checks out the folder %n instead of substituting in the correct number.

This is fixed in master


Not found all videos

Good afternoon, first congratulations for the work, an excellent lib.
What happens is that I don't know if there was any change in the way of scrapping the library, but all the videos now come back as "not found"

Vou dar uma olhada

I verified with a quick test via postman, that the axios URL is returning a page with a "Download Please Wait" button...
Maybe should make the axios

I did a quick test, with node-fetch, copying the request data via Developer Tools and apparently it worked

My source:
import cheerio from "cheerio";
import fetch from "node-fetch";

const twitterGetUrl = (url_media) => {
  return new Promise((resolve, reject) => {
    let twitter_url_array = url_media.split("/");
    twitter_url_array[5] = twitter_url_array[5].split("?")[0];
    url_media = twitter_url_array.join("/");
    var url = url_media.replace("twitter", "ssstwitter");

    fetch(url, {
      headers: {
        accept: "*/*",
        "accept-language": "pt-BR,pt;q=0.9",
        "content-type": "application/x-www-form-urlencoded; charset=UTF-8",
        "hx-current-url": url,
        "hx-request": "true",
        "hx-target": "target",
        "sec-fetch-dest": "empty",
        "sec-fetch-mode": "cors",
        "sec-fetch-site": "same-origin",
        "sec-gpc": "1",
      },

      body: `id=${encodeURIComponent(
        url
      )}&locale=en&tt=ba51541c4db79bbc91d4432631e0dea2&ts=1665807283`,
      method: "POST",
    })
      .then((result) => result.text())
      .then((result) => {
        let $ = cheerio.load(result),
          videoUrl = [],
          imageUrl = null,
          response = {},
          type = null;
        //CHECK FILE TYPE
        console.log($("div.result_overlay > img").length);
        if ($("div.result_overlay > img").length != 0) {
          type = $("div.result_overlay")
            .text()
            .includes("Oops! It seems that this tweet doesn't have a video!")
            ? "image"
            : "video/gif";
        }

        if (type == "video/gif") {
          $("div.result_overlay > a").each((i, element) => {
            let cheerioElement = $(element);
            let videoDimensions = cheerioElement
              .text()
              .split(" ")[1]
              .includes("x")
              ? cheerioElement.text().split(" ")[1]
              : null;
            let videoWidth =
              videoDimensions != undefined
                ? videoDimensions.split("x")[0]
                : null;
            let videoHeight =
              videoDimensions != undefined
                ? videoDimensions.split("x")[1]
                : null;
            videoUrl.push({
              width: videoWidth,
              height: videoHeight,
              dimension: videoDimensions,
              url: cheerioElement
                .attr("href")
                .includes("https://video.twimg.com")
                ? cheerioElement.attr("href")
                : `${cheerioElement.attr("href")}`,
            });
          });
          response = {
            found: true,
            type: "video/gif",
            dimensionsAvailable: videoUrl.length,
            download: videoUrl,
          };
        } else if (type == "image") {
          $("div.result_overlay > img").each((i, element) => {
            let cheerioElement = $(element);
            imageUrl =
              cheerioElement.attr("src") != "/images/no_thumb.png"
                ? cheerioElement.attr("src")
                : null;
          });
          response = {
            found: true,
            type: "image",
            download: imageUrl,
          };
        } else {
          response = {
            found: false,
          };
        }
        resolve(response);
      })
      .catch((err) => {
        reject(err);
      });
  });
};

(async () => {
  let response = await twitterGetUrl(
    "https://twitter.com/TweetsOfCats/status/1547778899534172160"
  );
  console.log(response);
})();

I checked yesterday and it looks like the site has put a limit of 1 request every 10 seconds so maybe I'll redo it with another site or do it directly on twitter.

I got another site, already did the update should be working again :) ty


I have several tests that create files and check the permissions of the created files. On UNIX systems, the permissions of the created files depend on the value of umask, which is a process-global variable managed by the OS, the same way that environment variables are process-global variables managed by the OS.
I expect to be able to change umask, run go test, and have go test re-run my tests with the new umask. However, after changing umask, go test incorrectly re-uses its cached test results:
$ go version
go version go1.23.1 darwin/arm64

$ umask 002                               # <--- set umask to 002

$ go test .
ok      tmp     0.252s                    # <--- tests pass when umask is 002

$ umask 022                               # <--- change umask to 022

$ go test .                               # <--- re-run tests
ok      tmp     (cached)                  # <--- BUG: test results re-used from cache

$ go test . -count=1                      # <--- disabling cache gives correct test result
--- FAIL: TestUmask (0.00s)
    umask_test.go:34: got 644, want 664
FAIL
FAIL    tmp     0.249s
FAIL

The example test is:
package main

import (
	"io/fs"
	"os"
	"path/filepath"
	"testing"
)

func TestUmask(t *testing.T) {
	// This tests passes when the umask is 002 (e.g. on Debian-based systems),
	// but fails when the umask is 022 (e.g. on RedHat-based systems and macOS).
	filename := filepath.Join(t.TempDir(), "file")
	if err := os.WriteFile(filename, nil, 0o664); err != nil {
		t.Fatal(err)
	}
	fileInfo, err := os.Stat(filename)
	if err != nil {
		t.Fatal(err)
	}
	// Ensure that the file's permissions are 0o666 &^ 0o002 == 0o664.
	if got, want := fileInfo.Mode().Perm(), fs.FileMode(0o664); got != want {
		t.Fatalf("got %03o, want %03o", got, want)
	}
}

Note that Go creates a test input ID that includes the values of environment variables and the mtimes and sizes of files accessed by the test. This test input ID should also include the initial umask value. I will create a CL that fixes this.
What did you see happen?
go test re-used a cached test result, when it should not have.
What did you expect to see?
go test should not re-use cached test results when the umask is changed.

Related Issues and Documentation

cmd/go: 'go test' fails to cache test result due to modified `.git` directory #52573 (closed)
os: TestMkdirStickyUmask failed in github code space #62684 (closed)
testing: test failes if leaving files without write permission in T.TempDir #40853
testing: calling os.Getenv in TestMain does not factor into test cache #59849 (closed)
 cmd/go: TestGoBuildUmask fails when source tree has a POSIX ACL set #17909 (closed)

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

CL: https://go-review.googlesource.com/c/go/+/614035

see thread https://groups.google.com/g/golang-nuts/c/x7-8Pk9ZmYU/m/ouJ3DLM0AwAJ

Copying in my comment from the mailing list thread:
We do not want to record all aspects of the external environment that could possibly affect the test. That would be too much. So we need some way to know what should be recorded and what should not. What should that be? Our current approach is simple: we record environment variables that the test reads and we record files opened within the module but not files opened outside of the module. Note that if a test opens some file outside of the module, we cache the test results even if that file changes.
What approach should we use for test caching that will tell us that umask should be cached?

Firstly, thank you Ian for your patience and open mind in discussing this!

We do not want to record all aspects of the external environment that could possibly affect the test. That would be too much.

Agreed.

So we need some way to know what should be recorded and what should not. What should that be?

Given that, as there is so much external state, it is impossible in practice for the go tooling to know what should be recorded.
As a crazy idea, how about enabling the tests themselves to say what should be recorded?

Our current approach is simple: we record environment variables that the test reads and we record files opened within the module but not files opened outside of the module.

This is fair but also a slight simplification. Currently, the go test cache only checks these files sizes and modification times.
For sure, we all agree that reading the full state that the cache depends on is prohibitively expensive, and that cache invalidation is a hard problem.

What approach should we use for test caching that will tell us that umask should be cached?

So, radical idea. What if tests could explicitly declare what they depend on?
Say I could write, for my tests that depend on umask, something like:
func TestThatDependsOnUmask(t *testing.T) {
    t.DependsOn("umask " + getUmask().String())
    // ...
}

where testing.T.DependsOn(s string) adds s to the test input IDs (i.e. modifies the test cache key).
This totally frees the Go authors from having to decide what to include or exclude from the test input ID. Instead, the decision is deferred to users, who are the people who know their test cache requirements best.
Thoughts?

It's an interesting idea but I think it would require cmd/go/internal/test/test.go to recognize the "umask" command.  So DependsOn seems too flexible: it would be too easy to add a dependency that would not be recognized.
As a separate concern, more knobs is harder to use.  It's already straightforward to test differing umask values within a test.  And if someone wants to set umask outside of the test binary, it's easy to disable caching by passing -test.count=1.  So I don't yet see the compelling argument for this approach.


Add black and pylint for python lambdas

We're already using cfn-lint, but it would be nice to have black and pylint integrated to the CI (they're already used locally by me, but are not enforced).

It's just a single function though, and there shouldn't be more of them... closing. :'(


⚠ Bofrost.be has degraded performance

In 039cc6c, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1364 ms

Resolved: Bofrost.be performance has improved in 8b17269.


Optimize some code of the dubbo metadata module

Optimize some code of the dubbo metadata module

Codecov Report

Merging #8783 (96c2e1e) into 3.0 (8376a6d) will decrease coverage by 0.12%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                3.0    #8783      +/-   ##
============================================
- Coverage     63.87%   63.74%   -0.13%
  Complexity      313      313
============================================
  Files          1151     1151
  Lines         48365    48334      -31
  Branches       7293     7259      -34
============================================
- Hits          30891    30809      -82
- Misses        14099    14138      +39
- Partials       3375     3387      +12

Impacted Files
Coverage Δ

...pache/dubbo/metadata/report/support/Constants.java
100.00% <ø> (ø)

...etadata/report/support/AbstractMetadataReport.java
62.67% <100.00%> (ø)

...ng/transport/dispatcher/all/AllChannelHandler.java
68.96% <0.00%> (-20.69%)
:arrow_down:

...ng/transport/dispatcher/WrappedChannelHandler.java
47.82% <0.00%> (-15.22%)
:arrow_down:

...ache/dubbo/remoting/transport/AbstractChannel.java
75.00% <0.00%> (-12.50%)
:arrow_down:

...dubbo/common/status/support/LoadStatusChecker.java
50.00% <0.00%> (-11.54%)
:arrow_down:

...moting/transport/netty4/NettyEventLoopFactory.java
22.22% <0.00%> (-7.78%)
:arrow_down:

...mmon/threadpool/support/fixed/FixedThreadPool.java
80.00% <0.00%> (-7.50%)
:arrow_down:

...ache/dubbo/remoting/api/NettyEventLoopFactory.java
30.00% <0.00%> (-6.37%)
:arrow_down:

...dubbo/remoting/exchange/support/DefaultFuture.java
87.85% <0.00%> (-5.85%)
:arrow_down:

... and 48 more

Continue to review full report at Codecov.

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


Not getting Notifications if app is closed via swipe

I have oneplus 3 ,I am getting notifications when app is open or is in background but when i swipe off it does not get fcm notifications.
Is it my device or what?

It is because of DOZE mode and battery optimization,you just have to turn off battery optimization for all apps or particular app.
Settings>> Battery >> battery Optimization >> find your app >> select >> if optimized click on don't optimize >> try pushing notification..
Hope this helps..
Enjoy :) :)

@rajeshjakhar1092 there are a number of existing issues on this topic, most notably #4 and #89.  Please consult those threads.


Add more examples for python finder.

Added a few examples for Python so people without much Flutter development experience won't have to spend time digging it like me.
I would appreciate if there are examples for by_ancestor and by_descendant. But I still don't know how to use that as of today. 😄

@hex0cter  Thanks for providing more example about Python APIs, are you able to try switchContext API with Python?
self.driver.switchContext('NATIVE_APP')
I am getting AttributeError: 'WebDriver' object has no attribute 'switchContext' .Do you know if this is an api only for JS client?

@whynotSQA please try
driver.switch_to.context("NATIVE_APP")

😄


Small cleanup

Poked my head around a bit and found a few things I considered low hanging fruit. Not entirely sure if you're interested in this type of PR since I'm not actually adding/fixing anything but figured there's no harm in opening it and finding out :)

I removed 298e0cf69a2c3d54a7750f02808aac373f1286bd ( filterIsInstance although I'm hoping for some more clarification) and the ObserverAdapters.

Pulled the filterIsInstance change back in, everything should be good to go 👍

Thanks!


eqmac freezes after wake from hibernation/sleep

eqmac freezes after wake from hibernation/sleep, problem is solved only by restarting the mac, tested on big sur 11.4, macbook pro 13 2015 with NVME

Duplicate of #433
Also didn't follow Issue Template.
Closing this issue.


Added some functions

Added getNote / getAlbum / sendGif.
Not clean code but maybe works fine.
With them, added some "not working functions".
(Would you please fix them?)

Awesome! Do you have a working updateProfileCover? In my version, i just success for uploading image to OBS, /myhome/c/upload.nhn and that doesn't change cover image. that's a trouble
Tomorrow i will update these func and implement your requests
Thanks


Could not find reflection lib or JSON parser

I'm trying out websocket client example here https://github.com/advantageous/qbit/wiki/[Doc]-Using-QBit-microservice-lib's-WebSocket-support
First of all the package org.boon where is it from? is there a maven artifact for it?
anyway i've used boon dependency which could cover this using io.advantageous.boon..
now it compiles but when i run it i get this:
Factory was null
Exception in thread "main" java.lang.IllegalStateException: Could not find reflection lib or JSON parser
at io.advantageous.qbit.QBit.registerReflectionAndJsonParser(QBit.java:64)
at io.advantageous.qbit.QBit.doGetFactory(QBit.java:51)
at io.advantageous.qbit.QBit.factory(QBit.java:39)
at io.advantageous.qbit.http.server.HttpServerBuilder.build(HttpServerBuilder.java:193)
at com.jarvis.sandbox.qbit.EchoWebSocket.main(EchoWebSocket.java:19)

These are dependencies i have now
    <dependency>
        <groupId>io.advantageous.qbit</groupId>
        <artifactId>boon</artifactId>
        <version>0.7.2</version>
    </dependency>

    <dependency>
        <groupId>io.advantageous.qbit</groupId>
        <artifactId>qbit-core</artifactId>
        <version>0.7.2</version>
    </dependency>

io.advantageous.qbit
boon
0.7.2

io.advantageous.qbit
qbit-boon
0.7.2

 io.advantageous.qbit
qbit-core 0.7.2
Fadi has a bunch of example starter projects.
Of course they are all using gradle..
On Sun, Mar 29, 2015 at 3:20 AM, Vachagan Balayan<EMAIL_ADDRESS>wrote:

These are dependencies i have now
<dependency>
    <groupId>io.advantageous.qbit</groupId>
    <artifactId>boon</artifactId>
    <version>0.7.2</version>
</dependency>

<dependency>
    <groupId>io.advantageous.qbit</groupId>
    <artifactId>qbit-core</artifactId>
    <version>0.7.2</version>
</dependency>

—
Reply to this email directly or view it on GitHub
https://github.com/advantageous/qbit/issues/228#issuecomment-87387874.

--
Rick Hightower
(415) 968-9037
Profile http://www.google.com/profiles/RichardHightower

Hi Vach,
In the link bellow you can find the example you're interested in, fully working. thanks for your interest.
https://github.com/fadihub/echo-websocket.git
https://github.com/advantageous/qbit/wiki/[Detailed-Tutorial]-Building-a-HTTP-server-with-WebSocket-support-using-QBit--(very-easy)


Quick chat skill guide option

Is your feature request related to a problem? Please describe.
As someone who isn't necessarily fond of using chat commands due to a slight disability of my left arm, I would find it extremely beneficial to be able to say my levels in public chat without having to type out !lvl thieving.
Adding this toggleable feature to the skill guide would also allow for a quicker way to say said level.
Describe the solution you'd like
A right-click quick chat TOGGLEABLE feature added to the skill guide to display level to public chat.
Additional context

This would be a macro, which is not allowed by Jagex.


FixedResistanceDpM

As discussed in https://github.com/iea-annex60/modelica-annex60/pull/607 the model Fluid.FixedResistances.FixedResistanceDpM will be refactored soon, it will be split up in two simpler models. We are using FixedResistanceDpM in two models:

https://github.com/UdK-VPT/BuildingSystems/blob/master/BuildingSystems/Technologies/Chillers/CompressionChiller.mo
https://github.com/UdK-VPT/BuildingSystems/blob/master/BuildingSystems/Technologies/SolarThermal/ThermalCollector.mo

These two models will have to be updated, possibly using the conversion script supplied in the A60 Pull Request.

The changes in Annex60 have been merged, so when integrating the latests Annex60 developments, the two models in BuildingSystems have to be updated.


fix requirements again

Hi! your requirements.txt does not have all necessary dependencies to run your tests (e.g. scikit-image), is this intended? I saw the closed issue #47 and the fix included scikit-image, which is now gone.

Hi
install package in dev mode to run tests (it will install pytest and scikit-image)
pip install segmentation-models[tests]

thanks!


changes to guide

Hi, I saw that there were important changes to the guide. Some things have improved, but I also see missing parts. For example, the part that explains how to delete the container and the corresponding disk image is missing, while only mention is made of the elimination of the container while preserving the data. Then another important thing is missing, namely the command to restart the container automatically when the server is restarted (docker update --restart = unless-stopped dnscrypt-server). So I think it's partly to be reviewed, but it can't be changed directly for other users on GitHub like it is for dnscrypt-proxy.

Hi!
Yes, using mounted volumes should makes things easier, especially updates.
Is docker update --restart=unless-stopped required if the --restart=unless-stopped option was already provided when the container was created?
I thought it was redundant, but maybe not! I don't use Docker that much.
A section on how to delete both the image and the container will be added.
You should have commit access to the repository, if you want to make changes directly :)

Is docker update --restart=unless-stopped required if the --restart=unless-stopped option was already provided when the container was created?

Thank you for giving me the opportunity to contribute to the guide. In fact the above command does just what I was looking for. If I saw something to change I will let you know.


[DOC] add query_by_tokens option in Neural Sparse Search

What do you want to do?

[x] Request a change to existing documentation
[ ] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
In 2.14 we'll add query by tokens option to Neural Sparse Search. We want to modify docs of Neural Sparse part to inform this change.
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.
https://github.com/opensearch-project/neural-search/issues/608
https://github.com/opensearch-project/neural-search/pull/693

Thanks for creating the issue, @zhichao-aws ! What is the ETA for the doc PR? We need to have it open and in review by 4/30 to meet entrance criteria for 2.14. Thanks.

Thanks for creating the issue, @zhichao-aws ! What is the ETA for the doc PR? We need to have it open and in review by 4/30 to meet entrance criteria for 2.14. Thanks.

Hi @hdhalter, I'll cut the doc PR and finish its content by 4/29

PR created. https://github.com/opensearch-project/documentation-website/pull/7040


Scheduler tasks stuck

Once there was a problem in running task (some fatal error maybe, or server issues), field 'started_at' not clear and task will stuck and never run again.

I think that was there intentionally so that a "running" task wouldn't run again.  If a task does not end normally and is left in the running state, "usually" it's better to not simply try to rerun the task, but to investigate the cause of the problem.
There used to be a facility to manually force a task to re-run - which could be done after the problem has been rectified.
Maybe take the above into consideration when you refactor.

To not re-run task it is great idea, sure, but if my task stops for a few days while I didn't see that it is not running, it is a real problem. It's useful to implement some mechanism to avoid that. Or at least show in web interface that task is stacked too long time and some button to re-run it manually.
Thank you.

Yes, if you can think of a good way to do that, it would be great.  Maybe run the idea by Byron. Scheduler is a very useful component and any improvements will be welcomed!

@timur560 I agree the current way this is handled is not correct, a shutdown handler should update the state of the task before exiting, this way we can see the task is failing.
I think its fine to let it keep failing, as the failure could be caused by an external issue, such as a 3rd party service outage; which could remedy itself.

@timur560 this is has been addressed in release 0.0.12
You will need to update your console config file with the following:
    'components' => [
        'errorHandler' => [
            'class' => 'webtoolsnz\scheduler\ErrorHandler'
        ]
    ]

Very useful fast update. Thank you, @badams .


[BUG] Error public app Microsoft Store

:information_source: Info

Version: e.g. v0.7.5
2.8.18
:speech_balloon: Description

Error public store
Enter a description of your problem here.
Any logs, error and assets you have that can help to find this bug.
invalid identity name ("identity_name"): "55371JMDSOFTWARESRL.ContaCloudJD_25xda2h4j032t". need to be a string between 3 and 50 characters in length that consists of alpha-numeric, period, and dash characters.
pub finished with exit code -1
I'm given it to me by the microsoft store
You must upload at least one package. If you are using market groups, then each market group must have at least one package.
You must fix all package validation errors before submitting.
contacloundjd.msix16.7 MB
Invalid package family name: 55371JMDSOFTWARESRL.ContaCloudJD_fxkeb4dgdm144 (expected: 55371JMDSOFTWARESRL.ContaCloudJD_25xda2h4j032t)
Invalid package publisher name: CN=Msix Testing, O=Msix Testing Corporation, S=Some-State, C=US (expected: CN=71C7788D-8364-4DE1-ACE0-0CE3103A6C0D)
:scroll: Pubspec.yaml

My Config
msix_config:
identity_name: 55371JMDSOFTWARESRL.ContaCloudJD_25xda2h4j032t #ContaCloudJD #ContaCloudJD.ContaCloudJD.MyApp
publisher: CN=71C7788D-8364-4DE1-ACE0-0CE3103A6C0D
publisher_display_name: JMDSOFTWARE SRL
display_name: ContaCloudJD
msix_version: <IP_ADDRESS>
capabilities: "internetClient"
logo_path: assets/logo.png #C:<PathToIcon><Logo.png>
store: true
We ask that you include your pubspec.yaml file as a common problem we have seen has been the pubspec.yaml file being incorrect

Hello @JoseDiaz2211
"_" is invalid in the 'identity_name' : 55371JMDSOFTWARESRL.ContaCloudJD_25xda2h4j032t
You can find your publisher_display_name, identity_name and publisher in Microsoft Store dashboard, looks something like this:


appstream: 1.0.2 -> 1.0.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/appstream/versions.
meta.description for appstream is: Software metadata handling library
meta.homepage for appstream is: https://www.freedesktop.org/wiki/Distributions/AppStream/
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.0.3 with grep in /nix/store/j7fd78a695cyz2p21161qfgd932ypnzz-appstream-1.0.3
found 1.0.3 in filename of file in /nix/store/j7fd78a695cyz2p21161qfgd932ypnzz-appstream-1.0.3

Rebuild report (if merged into master) (click to expand)

626 total rebuild path(s)

626 package rebuild(s)

First fifty rebuilds by attrpath

Sylk
adapta-gtk-theme
adwsteamgtk
ags
ajour
alfis
altair
alvr
amberol
anytype
appimage-run
appimage-run-tests
appstream
apx-gui
arduino-ide
ascii-draw
ashpd-demo
audio-sharing
authenticator
autokey
bada-bib
badlion-client
banking
baobab
bazecor
beekeeper-studio
beeper
bespokesynth
bespokesynth-with-vst2
betterdiscord-installer
blackbox-terminal
blanket
bloomrpc
blueberry
blueprint-compiler
boatswain
bookworm
bootstrap-studio
bottles
bottles-unwrapped
boxbuddy
breath-theme
budgie.budgie-control-center
budgie.budgie-desktop
budgie.budgie-desktop-with-plugins
budgie.budgie-gsettings-overrides
budgie.magpie
budgiePlugins.budgie-analogue-clock-applet
budgiePlugins.budgie-user-indicator-redux

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/j7fd78a695cyz2p21161qfgd932ypnzz-appstream-1.0.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A appstream https://github.com/r-ryantm/nixpkgs/archive/691ae1799766379bfa1f140280fde697a0790b31.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/691ae1799766379bfa1f140280fde697a0790b31#appstream

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/j7fd78a695cyz2p21161qfgd932ypnzz-appstream-1.0.3
ls -la /nix/store/j7fd78a695cyz2p21161qfgd932ypnzz-appstream-1.0.3/bin

Pre-merge build results
NixPkgs review skipped

Maintainer pings

This version bump changes some detail in the build system (https://github.com/ximion/appstream/pull/620) and breaks downstream users of appstream-qt on staging-next (#328673): https://hydra.nixos.org/build/267318864/nixlog/1/tail
I'm not familiar enough with nix and meson to propose a fix though.

I don't have time to look into it right now. If the derivation is multi-output, does removing dev from outputs help?

No, it doesn't. Looking into it.


⚠ Cafe7 has degraded performance

In d6a36a8, Cafe7 (https://cafe7.com.au) experienced degraded performance:

HTTP code: 200
Response time: 19583 ms

Resolved: Cafe7 performance has improved in 42b37f0.