Using ref instead of mutable in store record

According to reason-react docs instance variables should be stored in state and be marked with ref instead of mutable to prepare for concurrent react.
I am unsure whether this makes any difference for a global state management solution like reductive, but would be nice to have a better insight into this from someone who has a better understanding of the internals of reductive and reason-react. If it does make a difference, the fix is rather simple but nice to have to be ready for the future changes.
Would appreciate any opinions/thoughts on this :)

the ref type is roughly:
type ref('a) = {
  mutable contents: 'a
};

so that wouldn't change much conceptually 😊

hm... yes, that makes sense. However, the docs I linked show an example in reasonml playground that demonstrates the difference when reading from state from a closure. Then it seems to work incorrectly if using mutable since the data might get stale by the time it is accessed. So it seems like there might be a risk of having the same problem in reductive..
Please correct me if I am misunderstanding something!

For the function api I think preferring useRef where possible makes sense. For the old API yes, you must use a ref inside of state instead of a mutable field. Is there a scenario in the function api where useRef doesn't make sense?


Add support to untar/unzip necessary files

I need this to work....
necessary_files: [level_0/season10_multi_latlon_geno.geojson,
                  level_2/flirIrCamera/FLIR_indv_plant_temp_S10.tar.gz]

Seems to be working in branch 97-environment_association


[13pt] Testing the developed Python code for evaluating the Water Surface Elevation (WSE) inside the channel grid cells.

Part of Agile Story #204

#Calculating the flow profilewith the known boundary condition at downstream (Ex: There
%pylab inline
from scipy.optimize import fsolve, newton

#Calculating the flow profilewith the known boundary condition at downstream (Ex: There
%pylab inline
from scipy.optimize import fsolve, newton


fix: code cleanup and auto update docker image location

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


Errors in NFLlib Installation

I followed the steps to install NFLlib (in Ubuntu 18.04), but I'm running into some problems. Everything is fine until I run make test. Then, I get the following result:
95% tests passed, 6 tests failed out of 122
Total Test time (real) =  11.14 sec
The following tests FAILED:
98 - run_nfllib_demo1048576_124_uint64_t_op (Child aborted)
100 - run_nfllib_demo1048576_124_uint64_t_func (Child aborted)
108 - run_nfl_eq1048576_124_uint64_t (Child aborted)
110 - run_nfl_neq1048576_124_uint64_t (Child aborted)
114 - run_poly_p1048576_124_uint64_t (SEGFAULT)
118 - run_poly_mpz1048576_124_uint64_t (SEGFAULT)
Errors while running CTest
Makefile:97: recipe for target 'test' failed
make: *** [test] Error 8
Does anyone know why these tests are failing or what I can do about it?
I've also tried compiling the file nfllib_demo_main_func.cpp. But I get the following errors:
nfllib_demo_main_func.cpp: In function ‘int main()’:
nfllib_demo_main_func.cpp:122:14: error: ‘CONFIG’ was not declared in this scope
return run();
^
nfllib_demo_main_func.cpp:122:22: error: no matching function for call to ‘run()’
return run();
^
nfllib_demo_main_func.cpp:28:5: note: candidate: template<long unsigned int degree, long unsigned int modulus, class T> int run()
int run()
^
nfllib_demo_main_func.cpp:28:5: note:   template argument deduction/substitution failed:
nfllib_demo_main_func.cpp:122:22: error: template argument 1 is invalid
return run();
If anyone could help with these issues, I would be very appreciative.

Thanks for reproting! I can reproduce the issue with Release build, everything is fine with debug build, investigating...

@lightnesscaster can you confirm that setting ulimit -s unlimited before running the test fixes the issue?

@tlepoint if @lightnesscaster confirms the issue, i'll do it so!

Thanks for looking into this! I ran ulimit -s unlimited, but I'm still getting three errors. Is there something else I need to do? Also, I am still getting the same errors when trying to run nfllib_demo_main_func
98% tests passed, 3 tests failed out of 122
Total Test time (real) =  27.66 sec
The following tests FAILED:
98 - run_nfllib_demo1048576_124_uint64_t_op (Child killed)
108 - run_nfl_eq1048576_124_uint64_t (Child aborted)
110 - run_nfl_neq1048576_124_uint64_t (Child aborted)
Errors while running CTest
Makefile:97: recipe for target 'test' failed
make: *** [test] Error 8

I think the issue is that some tests allocates big objects on the stack, instead of allocating them. I'll try and look into fixing this properly this week!

Has anybody made any progress on this? I would still like to be able to work with this library. Does anyone have any idea what is causing the demo errors?

@aguinet any news on this fix  ? :)

With the current commit, on a cluster on which I cannot perform a ulimit -s unlimited, only 3 tests fail with a segfault :

98% tests passed, 3 tests failed out of 122
Total Test time (real) = 229.97 sec
The following tests FAILED:
98 - run_nfllib_demo1048576_124_uint64_t_op (SEGFAULT)
114 - run_poly_p1048576_124_uint64_t (SEGFAULT)
118 - run_poly_mpz1048576_124_uint64_t (SEGFAULT)

@serge-sans-paille @aguinet  this is not my domain of expertise, can you make a fix in, say, one week ?
If not I'll handle the issue myself but probably won't respect the coding standards you have.

My problem are not segfaults. For both Release and Debug, I'm getting the following output of make test:
50% tests passed, 61 tests failed out of 122
Total Test time (real) = 397.96 sec
The following tests FAILED:
          2 - run_nfllib_demo8_60_uint32_t_op (Failed)
          4 - run_nfllib_demo8_60_uint32_t_func (Failed)
          6 - run_nfl_add8_60_uint32_t (Failed)
          8 - run_nfl_sub8_60_uint32_t (Failed)
         10 - run_nfl_mul8_60_uint32_t (Failed)
         12 - run_nfl_eq8_60_uint32_t (Failed)
         14 - run_nfl_neq8_60_uint32_t (Failed)
         16 - run_nfl_stream8_60_uint32_t (Failed)
         18 - run_poly_p8_60_uint32_t (Failed)
         20 - run_poly_set8_60_uint32_t (Failed)
         22 - run_poly_mpz8_60_uint32_t (Failed)
         24 - run_poly_serialize_manually8_60_uint32_t (Failed)
         26 - run_nfllib_demo128_14_uint16_t_op (Failed)
         28 - run_nfllib_demo128_14_uint16_t_func (Failed)
         30 - run_nfl_add128_14_uint16_t (Failed)
         32 - run_nfl_sub128_14_uint16_t (Failed)
         34 - run_nfl_mul128_14_uint16_t (Failed)
         36 - run_nfl_eq128_14_uint16_t (Failed)
         38 - run_nfl_neq128_14_uint16_t (Failed)
         40 - run_nfl_stream128_14_uint16_t (Failed)
         42 - run_poly_p128_14_uint16_t (Failed)
         44 - run_poly_set128_14_uint16_t (Failed)
         46 - run_poly_mpz128_14_uint16_t (Failed)
         48 - run_poly_serialize_manually128_14_uint16_t (Failed)
         50 - run_nfllib_demo1024_60_uint32_t_op (Failed)
         52 - run_nfllib_demo1024_60_uint32_t_func (Failed)
         54 - run_nfl_add1024_60_uint32_t (Failed)
         56 - run_nfl_sub1024_60_uint32_t (Failed)
         58 - run_nfl_mul1024_60_uint32_t (Failed)
         60 - run_nfl_eq1024_60_uint32_t (Failed)
         62 - run_nfl_neq1024_60_uint32_t (Failed)
         64 - run_nfl_stream1024_60_uint32_t (Failed)
         66 - run_poly_p1024_60_uint32_t (Failed)
         68 - run_poly_set1024_60_uint32_t (Failed)
         70 - run_poly_mpz1024_60_uint32_t (Failed)
         72 - run_poly_serialize_manually1024_60_uint32_t (Failed)
         74 - run_nfllib_demo8192_124_uint64_t_op (Failed)
         76 - run_nfllib_demo8192_124_uint64_t_func (Failed)
         78 - run_nfl_add8192_124_uint64_t (Failed)
         80 - run_nfl_sub8192_124_uint64_t (Failed)
         82 - run_nfl_mul8192_124_uint64_t (Failed)
         84 - run_nfl_eq8192_124_uint64_t (Failed)
         86 - run_nfl_neq8192_124_uint64_t (Failed)
         88 - run_nfl_stream8192_124_uint64_t (Failed)
         90 - run_poly_p8192_124_uint64_t (Failed)
         92 - run_poly_set8192_124_uint64_t (Failed)
         94 - run_poly_mpz8192_124_uint64_t (Failed)
         96 - run_poly_serialize_manually8192_124_uint64_t (Failed)
         98 - run_nfllib_demo1048576_124_uint64_t_op (Failed)
        100 - run_nfllib_demo1048576_124_uint64_t_func (Failed)
        102 - run_nfl_add1048576_124_uint64_t (Failed)
        104 - run_nfl_sub1048576_124_uint64_t (Failed)
        106 - run_nfl_mul1048576_124_uint64_t (Failed)
        108 - run_nfl_eq1048576_124_uint64_t (Failed)
        110 - run_nfl_neq1048576_124_uint64_t (Failed)
        112 - run_nfl_stream1048576_124_uint64_t (Failed)
        114 - run_poly_p1048576_124_uint64_t (Failed)
        116 - run_poly_set1048576_124_uint64_t (Failed)
        118 - run_poly_mpz1048576_124_uint64_t (Failed)
        120 - run_poly_serialize_manually1048576_124_uint64_t (Failed)
        122 - run_prng_demo (Failed)
Errors while running CTest
Makefile:97: recipe for target 'test' failed

Any clue on what causes the failures? make and make install work fine otherwise. If I run cmake . ; make under tests/, I get:
Building CXX object CMakeFiles/ntt_multi.dir/multi0.o []/NFLLib/tests/multi0.cpp:3:10: fatal error: nfl.hpp: No such file or directory
Any help would be highly appreciated :)

98% tests passed, 3 tests failed out of 122
Total Test time (real) = 116.47 sec
The following tests FAILED:
98 - run_nfllib_demo1048576_124_uint64_t_op (SEGFAULT)
114 - run_poly_p1048576_124_uint64_t (SEGFAULT)
118 - run_poly_mpz1048576_124_uint64_t (SEGFAULT)
Errors while running CTest
Makefile:105: recipe for target 'test' failed
make: *** [test] Error 8
Also this issue.


improvement: enable custom http_adapters

Allow users to customize the behavior of the http client used by the oauth-based strategies by configuring a http adapter.

🚀 Thank you for your contribution! 🚀


Error When Starting the game

Not sure what it is, I made sure to followed every installation step. But this error pops up when trying to run the game. Might be worth noting that the game will still run and even playable with this error

Also sometime MO will said "Can't find SkyrimSE" and the game won't run. Is there any fix for this?
I'm still new to using MO2 and Wabbajack.

I am getting exactlt the same error as the above


scGPTHub submit jobs.

Hi! When I am using scGPTHub, the website, I met a problem. I successfully submitted the job, but scGPT analysis failed. So, what is the information should be contained in the H5ad?
This is the structure of H5ad I submitted.
AnnData object with n_obs × n_vars = 29239 × 60646
obs: 'cell_names', 'celltype', 'batch'
var: 'gene_names'
Thanks!

Hi, thank you for the question! Since scGPTHub is itself a separate service, we would need to log into the backend and see the specific logs of the job you run. So currently, we suggest you contact via the support email so that we can know the job and account. You can find the contact email address listed in the email you should have received when the job finished.
Could you please simply copy the above message in the email? Thank you in advance for your understanding. I will close this issue for now, but please let me know if you can not find the contact email or have other questions.


Updates to Get started with multi-host networking

Updates primarily to expand usage of Swarm cluster and make Compose
section more verbose as proposed in issue #17706 plus other minor corrections.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "mhn-updates" git@github.com:davidcurrie/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Thanks @davidcurrie! Some nice catches in there. I left some comments inline
ping @moxiegirl for review

OK - PR updated having addressed all of the comments above.

Thanks! LGTM
ping @jamtur01 @SvenDowideit @moxiegirl for review

Thanks @jamtur01 - all of the above should be addressed in an update to the PR.

Thanks @davidcurrie! I tested the docs, and all looked good
LGTM
ping @moxiegirl for the merge

@davidcurrie thank you for the contribution! LGTM

@davidcurrie Thanks! LGTM post-facto


added norwegian

Please see this on how to add a new language correctly: https://github.com/xPaw/CF-ABetterBrowser#how-to-add-a-language


Added django-cronjobs

If django-cronjobs is now going to be bundled with playdoh, please update http://playdoh.readthedocs.org/en/latest/userguide/libs.html to that effect. Looks like you just need to add an asterisk.

... I'll do it. I just found a typo I wanted to fix anyway.
https://github.com/mozilla/playdoh-docs/commit/64a5898

Uuuh if you're going to add this to all playdoh apps' default settings you also need to add the actual submodule to playdoh-lib :-/
https://github.com/mozilla/playdoh-lib/commit/d205ed8
https://github.com/mozilla/playdoh/commit/d245eb8


prune: Fix calculation of remaining uncompressed data

What does this PR change? What problem does it solve?
Fixes a regression introduced by #3981. For a fully compressed repository, prune would report a very high number of not yet compressed blobs:
$ restic prune --dry-run --max-unused 1%
repository a14e5863 opened (version 2, compression level auto)
loading indexes...
loading all snapshots...
finding data that is still in use for 5 snapshots
[0:00] 100.00%  5 / 5 snapshots
searching used packs...
collecting packs for deletion and repacking

to repack:           665 blobs / 83.056 MiB
this removes:        369 blobs / 52.054 MiB
to delete:            16 blobs / 13.294 MiB
total prune:         385 blobs / 65.348 MiB
not yet compressed:              16777216.000 TiB
remaining:        109302 blobs / 365.122 MiB
unused size after prune: 2.482 MiB (0.68% of remaining size)

Only the repacking of uncompressed packs reduces the amount of uncompressed data. Previously the counter even overflowed for fully compressed repositories.
Was the change previously discussed in an issue or on the forum?
Fixes #3981

Checklist

[x] I have read the contribution guidelines.
[x] I have enabled maintainer edits.
[ ] I have added tests for all code changes.
[ ] I have added documentation for relevant changes (in the manual).
[ ] There's a new file in changelog/unreleased/ that describes the changes for our users (see template).
[x] I have run gofmt on the code in all commits.
[x] All commit messages are formatted in the same style as the other commits in the repo.
[x] I'm done! This pull request is ready for review.

I've moved the mustCompress flag into the packInfoWithID struct. The packInfo itself isn't really the best place to pass the flag into the repack candidate selection.


flutter pub get fails when using pub workspaces with nested Flutter dependency

Steps to reproduce
reproducible repo: https://github.com/pattobrien/workspaces_bugs

add flutter as a dependency in the root workspace pubspec.
add flutter as a dependency to a nested package.
run flutter pub get in the nested package's directory

Expected results
flutter pub get is able to resolve.
Actual results

[nested_a] flutter pub get --no-example
Resolving dependencies in `/Users/pattobrien/dev/bugs/build_runner_workspaces`...
Downloading packages...
  _fe_analyzer_shared 72.0.0 (73.0.0 available)
  analyzer 6.7.0 (6.8.0 available)
  build_runner 2.4.11 (2.4.12 available)
  build_runner_core 7.3.1 (7.3.2 available)
  collection 1.18.0 (1.19.0 available)
  coverage 1.8.0 (1.9.0 available)
  http_parser 4.0.2 (4.1.0 available)
  material_color_utilities 0.11.1 (0.12.0 available)
  shelf 1.4.1 (1.4.2 available)
Got dependencies in `/Users/pattobrien/dev/bugs/build_runner_workspaces`!
9 packages have newer versions incompatible with dependency constraints.
Try `flutter pub outdated` for more information.
LocalDirectory: '/Users/pattobrien/dev/bugs/build_runner_workspaces/nested_a': pub did not create .dart_tools/package_config.json file.
exit code 1

Flutter Doctor output
Doctor output
❯ flutter --version
Flutter 3.24.0 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 80c2e84975 (13 days ago) • 2024-07-30 23:06:49 +0700
Engine • revision b8800d88be
Tools • Dart 3.5.0 • DevTools 2.37.2

❯ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.24.0, on macOS 14.4.1 23E224 darwin-arm64, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0-rc1)
[✓] Xcode - develop for iOS and macOS (Xcode 15.4)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2023.1)
[✓] VS Code (version 1.92.0)

Note: this is a bug that's depending on the following Flutter issue to be resolved, but due to the severity (being unable to use flutter pub get), this should likely be tracked as its own high priority issue.
https://github.com/flutter/flutter/issues/150196

Reproducible using the repo and steps outlined above.

flutter doctor -v
[✓] Flutter (Channel stable, 3.24.0, on macOS 14.5 23F79 darwin-arm64, locale en-GB)
    • Flutter version 3.24.0 on channel stable at /Users/nexus/dev/sdks/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 80c2e84975 (2 weeks ago), 2024-07-30 23:06:49 +0700
    • Engine revision b8800d88be
    • Dart version 3.5.0
    • DevTools version 2.37.2

[!] Flutter (Channel master, 3.24.0-1.0.pre.560, on macOS 14.5 23F79 darwin-arm64, locale en-GB)
    • Flutter version 3.24.0-1.0.pre.560 on channel master at /Users/nexus/dev/sdks/flutters
    ! Warning: `flutter` on your path resolves to /Users/nexus/dev/sdks/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutters. Consider adding /Users/nexus/dev/sdks/flutters/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/nexus/dev/sdks/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutters. Consider adding /Users/nexus/dev/sdks/flutters/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 289ba8dc25 (5 hours ago), 2024-08-13 00:28:25 -0400
    • Engine revision af68e772d2
    • Dart version 3.6.0 (build 3.6.0-142.0.dev)
    • DevTools version 2.38.0
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

Closing as fixed by https://github.com/flutter/flutter/pull/150850 on the master channel. However, I feel like this change is too involved (and thus risky) to be hotfixed into the stable channel. fyi @sigurdm in case they feel differently.


Bug: Python can't find external XPM files in an art directory

Description:

In C++, if the user specifies a .xpm file as an image, it gets #included in the source file. With Python, however, we have no way of knowing what the current directory will be, and therefore no way of knowing where the .xpm file will be relative to whatever the current directory is. This is noticeable in our python_tests where the launching python script is in the root, but the python module that loads the .xpm file is in the python/ folder. In this case, the project file is in the same location as the launching pyton file, but it could just as easily be in the python/ folder, in which case trying to load the file relative to the project file would fail.
I can't think of a good solution that works for both C++ and Python. Worst case is that we embed the XPM file in Python, but that will increase the size by a large amount unless we first convert it to PNG (which means converting a possible Mask into and Alpha channel). Even then, we're not setup to have an XMP file for C++ which must also be a PNG file in Python.

wxAnimateCtrl is going to have the same problem -- wx.Animation is what needs to load the image, and it doesn't accept a data buffer. SVG is also going to be an issue, though we could potentially zip that and embed the zipped version.

PR #1333 included fixes for external XPM files accessed from both wxPython and wxRuby3. SVG and Animation files are embedded for all three languages, so there is no external dependency there.


Need help in solving Logo issue: Magika Token #19447

I'm submitting a change for Adding Magika Token #19447
[ ] bug report
[ ] feature request
[ x] question about the decisions made in the repository
[ x] question about how to use this project

I'm trying to change the picture for the token  I'm new to this and I can use some help
here is the new URL for the token https://raw.githubusercontent.com/Hotel-Six/Mana-Token/main/Logo.png

please leave the PR open so we can see what's wrong

Sorry I just I cant find the button to reopen

automerged: https://github.com/solana-labs/token-list/pull/19508


fish: Unknown command 'fisher'

mona@Monas-MacBook-Pro ~> fisher i joker
fish: Unknown command 'fisher'

Yes, you need to install Fisherman.


RealTime support

Hello and thank you for the code, it works great!.
In the official SDK, there is a "CKT_ReadRealtimeClocking" with return all clocking records of real-time enabled devices. This method doesn't use the device ID, it for all connected devices.
I've not found that method in the Communication Protocol document, do you know how that method works? and how can it be implemented?.
Thank you.

Can you explain in more detail how that mode works? In the AdvancedSettings class there is the field bool RealTimeMode which I have not yet investigated.
After you enable that, what happens? The device continuously sends new records by itself?

Hello John,
Yes I've been investigating the ethernet traffic and when you enable realtime, the device send clocking record continuosly.
I'm trying to wait for commands in the network stream, and yes, they arrive in realtime from the device.
This is the method I'm trying in AnvizDevice.cs:
public void DownloadRealTimeRecords(Action<ulong, Record> newRecord)`
        {
            th = new Thread(new ThreadStart(async () =>
            {
                while (DeviceStream.CanRead)
                {
                        var resp = await Response.FromStream(0xdf, DeviceStream);
                        Record rc = new Record(resp.DATA, 0);
                        newRecord(this.DeviceId, rc);
                }
            }));
            th.Start();
        }

It calls the newRecord delegate as the new records arrive.
The command received is 0xDF with the format that appears in the documentation.

Thanks for sharing your code. At this time, the library isn't thread-safe, so what you did there will conflict when you try to send other commands. Also, the receiving code is highly coupled to the sending mechanism. I have to change the way we receive packets.
I'll be working on this once I'm done with the server side code, which currently I'm not happy at all.

Hi, with the latest code from this repo, you can now read packets asynchronously.
Can you take a look and see if you can get real time support in this mode?
Thanks

Hi @jleal52 did you have any time to work on this? Any help is appreciated.
Thanks

I'm happy to annouce that this feature has been implemented!


Top margin on star-rating-text misaligns it for smaller size stars (and isn't even needed)

An example of what I mean is in the "Lots of Stars" example in the JSFiddle.
I'm assuming the top-margin of 7px was to align the number vertically, but since the parent div.vue-star-rating already has align-items: center, it isn't even needed.
PS. Nice little component. Thanks!

I can create a PR if you prefer.


tidy-checks=-* triggers false-positive

I added this action to a project I'm helping. The project's owner is still learning C++, so I thought this action would help lament best practices...
I tried to disable clang-tidy by specifying
          tidy-checks: '-*'

but clang-tidy threw some errors which triggered an empty comment.
The workflow log reveals that clang-tidy didn't like being told to do nothing. The output from clang-tidy (which in this case was the --help output) was interpreted as though the user should be notified.

Error: no checks enabled.

I'd like to add a check in the run.py (run_clang_tidy()) that allows skipping clang-tidy when all checks are disabled (when tidy-checks=-*).

I just found that only hidden directories (starting with .) at repo root are ignored. Meaning a hidden sub-directory would not be ignored. I'm fixing that while I'm debugging.
Also, I'm revising the get_line_cnt_from_cols() to open files as binary instead of encoded text. This should better support CRLF & LF line endings. The current implementation might lead to an infinite loop.


Flake8 tools

note, this PR doesn't actually change how any of the code runs, it's just for passing flake8 in the tools directory only.
I did delete tools/utils/ec2recovery.py since it's broken at compile time, and it's been this way for 6 months. Which I assume means it's not used/needed anymore? If it's still used I can undo this and try to repair the file to compile/work.

we'll need to resurrect recovery at some point, but given its state removal is fine. thanks for the pr.


Kickoff Meetings/Follow Up

Completed.  Notes are in the GDrive - Sender Automation Hub

8/19/21 - Met with GUC contact. Notes are located here: https://drive.google.com/drive/folders/1kpli85EuoaoY5XZK7Ig954XFV5PtLIF1

8/24/21 - Meeting scheduled to review reporting.


[BUG] Keysign discover instability on mac

Describe the bug
Occasionally it is impossible to start a keysign as the pairing device keeps waiting for the signing process to begin. It roots from Mac when the app stays open for a long time. Could it be a timeout problem? As a restart of the app on the initiating device fixes it.
Expected behavior
Consistent keysign initiation
Screenshots

Version 29

@johnnyluo it happens quite consistent  for me, after having app open for around ca. 5 minutes.

@johnnyluo When looking in console on mac, it looks like it cycles this message. noticed that this also happens when intenting a 4/5 with two androids included...

@johnnyluo I still have those problems on macOS version and need to restart from time to time... Not sure what's causing it

@johnnyluo I just have that now from time to time on my iPad. Not sure whats causing it. Just documenting so we keep track.

Will close it for now , can't actually reproduce it reliably


Refactor Sink Writer to accept blocks inplace of bytes

Looks like we need to fix some tests

Tests are fixed.

Looks like we need to fix some tests

Fixed

There's a few conflicts @tardunge

There's a few conflicts @tardunge

will fix.

@tardunge please address comments in a separate PR(s)

Sure.

This MR introduced an issue that the Azure writer would not initiate. @tardunge

@tardunge hi, here it is https://github.com/talariadb/talaria/issues/103


report ETCD metrics

Currently, Easegress only reports metrics of itself, but for some issues, like https://github.com/megaease/easegress/issues/413, the root cause is on the ETCD side, so it would be helpful for troubleshooting if Easegress can also report ETCD metrics.

Does the community have plans to support external etcd? In this way, we can use Alibaba Cloud's etcd, which can reduce operation and maintenance costs, and they can handle the reporting of some metrics

https://github.com/megaease/easegress/pull/1137 fixed


Predict impact of drafted message front end

Button inside the extension popup somewhere. Disappears once clicked.
When clicked, slide all messages up by one message slot.
Add in new message blob, opacity 50%.
Reset when they close the popup

Moved to #35


Serinadawn patch 1

Dan & Jordan keep or toss my recommendations here.  I went ahead and cut out all the content that was TO DO or TBD  just to make the page easier to read.  As we flush out more content we can add it back in.  My intent here was to make keep complete info out there.

Only one tiny tweak (which I made) - this shouldn't be ticked. I'm about to merge this though - the rest looks great. Much tidier.
- [ ] Current work is related to replacing the hashing algorithm to prevent the redistribution (rebalancing) of repos across the available Gitservers when a Gitserver is added or removed.


SnackBar is not entirely flush with the bottom

Steps to Reproduce
Seeing this in the Hamilton app, on Android O. App is version 1.1.2.

cc @chinmaygarde and @jason-simmons who were looking at this recently.
It basically boils down to a rounding error with floats IIRC. But I thought it was fixed.

There were a couple of fixes for this issue. One of them was a workaround for Skia returning different bounds for rects that were drawn with AA toggled. The other was the raster cache introducing a rounding error when snapshotting the sub-hierarchy. Both fixes have landed recently. What is the version of the framework and the engine that displays this issue?

I tried the following program today with flutter master from today, and it does not have the issue on my O phone.
Collin, is the currently published Hamilton app on an older framework?
import 'package:flutter/material.dart';

void main() {
  runApp(new MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return new MaterialApp(
      title: 'Flutter Demo',
      home: new Scaffold(
        appBar: new AppBar(title: new Text('Demo')),
        body: new Builder(builder: (BuildContext context) {
          return new Center(
            child: new FlatButton(
              child: new Text('Push me'),
              onPressed: () {
                Scaffold
                    .of(context)
                    .showSnackBar(new SnackBar(content: new Text('Hi!')));
              },
            ),
          );
        }),
      ),
    );
  }
}

I'm not 100% sure which version of Flutter the Hamilton 1.2 release used, but my best guess is 47fe630e4e5a8f6595314dbf701fdb9b7be6a331. There should be another release out soon.

I'm pretty sure this was fixed. I'm going to close this. Please reopen if it's not fixed whenever the Hamilton app is next updated.

cc @lukef


naming iterations

I have checked the following:

[X] I've searched existing issues and found nothing related to my issue.

Describe the feature you want to add
I would like to have a way to give a name to each iteration (would be coming from one of the column of a csv file).
The current behavior is "Iteration X" which is making the reading the html report difficult.
Mockups or Images of the feature
n/a

Hi, thanks for reporting, adding this to our development pipeline.


Remove instance which you don't need

Co-authored-by: Philippe Fehr<EMAIL_ADDRESS>

Thank you for your suggestion. I will consider looking into the potentially destructive side effects of this PR once I have a CI up and running.

I guess this isn't needed as I ran rubocop locally and everything passed

Closing pr in favour of #11


[tool_crash] FileSystemException: Creation failed, OS Error: A file cannot be created if it already exists., errno = 183

flutter_01.log

A similar / related issue is: https://github.com/flutter/flutter/issues/74227

Update: after playing around with the Visual Studio packages, the dependencies were finally cleared up (thank you!), but now I'm back to the previous error (just some of the details have changed):

Flutter could not download and/or extract https://storage.googleapis.com/flutter_infra_release/flutter/d5adde01dd672ba4f6819d85218beb01142dca27/flutter-web-sdk-windows-x64.zip. Ensure you have network connectivity and all of the required dependencies listed at flutter.dev/setup.
The original exception was: FileSystemException: Creation failed, path = 'H:\flutter\bin\cache\flutter_web_sdk\flutter_web_sdk/lib/ui/src/ui' (OS Error: Eine Datei kann nicht erstellt werden, wenn sie bereits vorhanden ist.
, errno = 183).

I have tried deleting the 'flutter\bin\cache\flutter_web_sdk\flutter_web_sdk' folder (the last one in the hierarchy), and get various error messages.
Repeating the "run" without any deletions just got me this:

Flutter failed to check for directory existence at "H:\flutter\bin\cache\flutter_web_sdk". The flutter tool cannot access the file or directory.
Please ensure that the SDK and/or project is installed in a location that has read/write permissions for the current user
flutter_06.log
flutter_05.log
.

May 2023, this issue has still not been solved sigh

Leaving information from duplicate issue https://github.com/flutter/flutter/issues/131978 on new Flutter version for better tracking (the stack trace from OP is on a file, so it's not convenient to quickly capture information)
On stable/3.10.6 with command: flutter build windows
FileSystemException: Cannot create link, OS Error: No se puede crear un archivo que ya existe.
, errno = 183

#0      _Link.throwIfError (dart:io/link.dart:254:7)
#1      _Link.createSync (dart:io/link.dart:182:5)
#2      ForwardingLink.createSync (package:file/src/forwarding/forwarding_link.dart:20:16)
#3      ForwardingLink.createSync (package:file/src/forwarding/forwarding_link.dart:20:16)
#4      _createPlatformPluginSymlinks (package:flutter_tools/src/flutter_plugins.dart:1055:12)
#5      createPluginSymlinks (package:flutter_tools/src/flutter_plugins.dart:985:5)
#6      buildWindows (package:flutter_tools/src/windows/build_windows.dart:64:3)
#7      BuildWindowsCommand.runCommand (package:flutter_tools/src/commands/build_windows.dart:56:11)
<asynchronous suspension>
#8      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1336:27)
<asynchronous suspension>
#9      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#10     CommandRunner.runCommand (package:args/command_runner.dart:212:13)
<asynchronous suspension>
#11     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:296:9)
<asynchronous suspension>
#12     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#13     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:244:5)
<asynchronous suspension>
#14     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:91:9)
<asynchronous suspension>
#15     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>

Any temporary workaround for this issue?

Any updates on this? Getting the same exact error with  flutter run -d windows -v

hey, I had the same error today, in android studio => 1. flutter clean 2. fluuter pub get, this may solve the issue if this doesnt help 3. flutter pub upgrade

I had the same error today, in window 10.
I get the error info with flutter run -d windows -v
`[+9009 ms] Error: Building with plugins requires symlink support.
                Please enable Developer Mode in your system settings. Run
                  start ms-settings:developers
                to open settings.

[   +9 ms] "flutter run" took 10,714ms.
[   +4 ms]
#0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
#1      RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:811:9)

#2      FlutterCommand.run. (package:flutter_tools/src/runner/flutter_command.dart:1389:27)

#3      AppContext.run. (package:flutter_tools/src/base/context.dart:153:19)

#4      CommandRunner.runCommand (package:args/command_runner.dart:212:13)

#5      FlutterCommandRunner.runCommand. (package:flutter_tools/src/runner/flutter_command_runner.dart:355:9)

#6      AppContext.run. (package:flutter_tools/src/base/context.dart:153:19)

#7      FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:295:5)

#8      run.. (package:flutter_tools/runner.dart:119:9)

#9      AppContext.run. (package:flutter_tools/src/base/context.dart:153:19)

#10     main (package:flutter_tools/executable.dart:90:3)
`
I solved it by doing this
Go to system settings, click Update and Security, click For Developers ->Developer Mode -> Enable Install apps from any source, including loose files.


Update all-transact-sql.md

adding the ' to escape the '

@baleng : Thanks for your contribution! The author, @douglaslMS, has been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author, @douglaslMS, has been notified to review your proposed change.

Text appears as "cann't" on the live site; changed text to "can't"


Implement chatops deployment

Two files are necessary for this to work. Here's how we would set this up for backend software.
In the first config, the repository value is included to show how we could set up a slash command processor repository. This first config file would be included in the application repository, and the second config file would be placed in the processor repository. If we don't want to set up a separate repo to process the commands then both workflows would need to be included in the same repo.
chatops.yml
name: Slash Command Dispatch
on:
  issue_comment:
    types: [created]
jobs:
  dispatch:
    runs-on: ubuntu-latest
    steps:
      - name: Dispatch deploy slash command
        id: deploy
        uses: peter-evans/slash-command-dispatch@v2
        with:
          token: ${{ secrets.REPO_ACCESS_TOKEN }}
          reaction-token: ${{ secrets.REPO_ACCESS_TOKEN }}
          commands: deploy
          dispatch-type: workflow
          repository: dictybase-playground/github-actions-experiments
          static-args: |
            repository-name=${{ github.event.repository.name }}
            repository-owner=${{ github.repository_owner }}
            issue-number=${{ github.event.issue.number }}
            html-url=${{ github.event.comment.html_url }}
            comment-id=${{ github.event.comment.id }}
      - name: Edit comment with error message
        if: steps.deploy.outputs.error-message
        uses: peter-evans/create-or-update-comment@v1
        with:
          comment-id: ${{ github.event.comment.id }}
          body: |
            > ${{ steps.deploy.outputs.error-message }}
            > For /deploy commands used in GitHub Issues, you need to specify either specify a `commit` or `branch` to deploy.

            > Example: `/deploy cluster=erickube branch=feature/new-command`

            > For /deploy commands used in pull requests, you can optionally specify a `commit`.

deploy-command.yml
name: deploy-command
on:
  workflow_dispatch:
    inputs:
      repository-name:
        description: "The name of the repository from which the slash command was dispatched"
        required: true
      repository-owner:
        description: "The owner of the repository from which the slash command was dispatched"
        required: true
      issue-number:
        description: "The issue number in which the slash command was made"
        required: true
      html-url:
        description: "URL where comment was created"
        required: true
      comment-id:
        description: "ID of comment with deploy command"
        required: true
      cluster:
        description: "The cluster to deploy to"
        required: true
      commit:
        description: "Commit ID to deploy"
        required: false
      branch:
        description: "Branch to deploy"
        required: false
jobs:
  prepare-deploy:
    runs-on: ubuntu-latest
    name: Testing deploy
    steps:
      - name: Checkout
        uses: actions/checkout@v2
        with:
          repository: ${{ github.event.inputs.repository-owner }}/${{ github.event.inputs.repository-name }}
      - name: download and setup path for github actions binary
        run: |
          mkdir -p github-actions/bin
          curl -L -o github-actions/bin/actions https://github.com/dictybase-docker/github-actions/releases/download/v0.4.1/action_0.4.1_Linux_x86_64
          chmod +x github-actions/bin/actions
          echo "::add-path::$GITHUB_WORKSPACE/github-actions/bin"
      - name: parse deploy command and set vars
        id: vars
        run: actions --log-level debug pcd -f ${GITHUB_EVENT_PATH}
      - name: image build and push
        if: ${{ success() }}
        uses: docker/build-push-action@v1
        with:
          username: ${{ secrets.DOCKER_USER }}
          password: ${{ secrets.DOCKER_PASSWORD  }}
          repository: dictybase/${{ github.event.inputs.repository-name }}
          add_git_labels: true
          tag_with_sha: true
          tag_with_ref: true
          tags: ${{ steps.vars.outputs.image_tag }}
          dockerfile: build/package/Dockerfile
      - name: prepare for deploy
        if: ${{ success() }}
        id: prepare_deploy
        uses: dictybase-docker/prepare-deploy@v1
        with:
          cluster-name: ${{ github.event.inputs.cluster }}
          cluster-zone: "us-central1-a"
          namespace: dictybase
          chart-name: ${{ github.event.inputs.repository-name }}
          chart-path: deployments/charts/${{ github.event.inputs.repository-name }}
          token: ${{ secrets.GITHUB_TOKEN }}
          image-tag: ${{ steps.vars.outputs.image_tag }}
          ref: ${{ steps.vars.outputs.ref }}
          artifact: ${{ github.event.inputs.repository-name }}
          environment: development
  deploy:
    runs-on: ubuntu-latest
    needs: prepare-deploy
    steps:
      - name: check out code
        uses: actions/checkout@v2
      - name: download deployment artifact
        uses: actions/download-artifact@v2
        with:
          name: ${{ github.event.inputs.repository-name }}
      - name: download and setup path for github action binary
        run: |
          mkdir -p github-actions/bin
          curl -L -o github-actions/bin/actions https://github.com/dictybase-docker/github-actions/releases/download/v0.4.1/action_0.4.1_Linux_x86_64
          chmod +x github-actions/bin/actions
          echo "::add-path::$GITHUB_WORKSPACE/github-actions/bin"
      - name: extract information from deployment payload
        id: deploy_info
        run: actions --log-level debug sdp -f deployment.json
      - name: set in_progress deployment status
        if: ${{ success() }}
        run: actions --log-level debug -t ${{ secrets.GITHUB_TOKEN }} --owner ${{ github.event.inputs.repository-owner }} -r ${{ github.event.inputs.repository-name }} ds --state in_progress  --deployment_id ${{ steps.deploy_info.outputs.id }} --url ${{ steps.deploy_info.outputs.url }}
      - name: setup google cloud sdk
        uses: GoogleCloudPlatform/github-actions/setup-gcloud@master
        with:
          version: "307.0.0"
          project_id: ${{ secrets.PROJECT_ID }}
          service_account_key: ${{ secrets.SA_KEY }}
          export_default_credentials: true
      - name: set up helm
        uses: azure/setup-helm@v1
        with:
          version: "v2.16.9"
      - name: get gcloud credentials for k8s cluster
        run: gcloud container clusters get-credentials --project ${{ secrets.PROJECT_ID }} --zone ${{ steps.deploy_info.outputs.zone }} ${{ steps.deploy_info.outputs.cluster }}
      - name: deploy chart
        if: ${{ success() }}
        run: actions --log-level debug dc --name ${{ steps.deploy_info.outputs.chart }} --namespace ${{ steps.deploy_info.outputs.namespace }} --image-tag ${{ steps.deploy_info.outputs.image_tag }} --path ${{ steps.deploy_info.outputs.path }}
      - name: set successful deploy status
        if: ${{ success() }}
        run: actions --log-level debug -t ${{ secrets.GITHUB_TOKEN }} --owner ${{ github.event.inputs.repository-owner }} -r ${{ github.event.inputs.repository-name }} ds --state success  --deployment_id ${{ steps.deploy_info.outputs.id }} --url ${{ steps.deploy_info.outputs.url }}
      - name: set unsuccessful deploy status
        if: ${{ failure() }}
        run: actions --log-level debug -t ${{ secrets.GITHUB_TOKEN }} --owner ${{ github.event.inputs.repository-owner }} -r ${{ github.event.inputs.repository-name }} ds --state error  --deployment_id ${{ steps.deploy_info.outputs.id }} --url ${{ steps.deploy_info.outputs.url }}
  add-reaction:
    runs-on: ubuntu-latest
    needs: deploy
    steps:
      - name: Add reaction
        uses: peter-evans/create-or-update-comment@v1
        with:
          token: ${{ secrets.REPO_ACCESS_TOKEN }}
          repository: ${{ github.event.inputs.repository-owner }}/${{ github.event.inputs.repository-name }}
          comment-id: ${{ github.event.inputs.comment-id }}
          reaction-type: hooray

Apps that need to be implemented:

https://github.com/dictyBase/graphql-server
https://github.com/dictyBase/modware-user
https://github.com/dictyBase/modware-identity
https://github.com/dictyBase/modware-content
https://github.com/dictyBase/static-server
https://github.com/dictyBase/modware-auth
https://github.com/dictyBase/modware-import
https://github.com/dictyBase/modware-order
https://github.com/dictyBase/modware-stock
https://github.com/dictyBase/event-messenger
https://github.com/dictyBase/modware-annotation
https://github.com/dictyBase/graphql-authserver

For the deploy command to work with the modware repos, we are either going to need to rename the charts to match the repo names or we need to update the workflow and script(s) to accept a chart name as input.

The following charts need to be updated to get their database info from a configmap and secret:

[ ] https://github.com/dictyBase/modware-identity
[ ] https://github.com/dictyBase/modware-annotation
[ ] https://github.com/dictyBase/modware-order
[ ] https://github.com/dictyBase/modware-stock

Chatops has now been fully implemented on the backend with significant additions to the dictybase-configuration chart.


Error in how an xml-template-expr is handled with a sequence

Description:
$title.
Steps to reproduce:
import ballerina/io;

public function main() {
    xml x1 = xml `<foo>foo</foo><!--bar-->baz`;
    xml x2 = xml `<foo>foo</foo>`;
    io:println(x1);
    io:println(x1);
    io:println(x1 == x2);
}

This currently prints
<foo>foo</foo>
<foo>foo</foo>
true

Affected Versions:
slp4

Seems to be an issue with the xml sequence literal. Does not occure when xml sequence is constructed using xml:concat function.

Seems to be an issue with the xml sequence literal. Does not occure when xml sequence is constructed using xml:concat function.

Same as https://github.com/ballerina-platform/ballerina-lang/issues/26589

Same as https://github.com/ballerina-platform/ballerina-lang/issues/26589


FINERACT-429: Node aware Scheduler

Fineract-429
Executing jobs based on node id

@avikganguly01 kindly review this PR

@wkk91193 thank you very much for your interest in and contribution to Fineract/Mifos!  I'm an (old) volunteer who got reminded of Mifos/Fineract this week-end at the FOSDEM conference, and I'm taking a moment to help the project clean up some of it's old issues and pull requests etc.
We were wondering whether you are still alive, and would be willing to git rebase develop this PR and take any previous review feedback into account?  If we don't hear from you within 2 weeks, we will take the liberty to close this PR without merging it, to avoid confusion for future new contributors looking at this repository - hope you understand and that's fair.
/Cc FYI @apache/fineract-committers

@wkk91193 I'm currently doing a quick run of review on all open PRs, and was just wondering if you would still like to get this very old open one merged? If so, could you rebase it, so that we can see if it actually builds on Travis? (FYI when this PR was raised, this project did not have automated CI on Travis set up; but meanwhile it does.)
This one also has review feedback from feedback @avikganguly01 which needs to be taken into account.
Without hearing back within let's say ... 1 month, I would be inclined to just close this (un-merged).

Let's close this now, just to have less open PRs; you can open a new PR with this when you resolved the feedback and conflicts.


Some simple API key are not detected in configuration file

Describe the bug
Given some XML file (Xcode Info.plist) with a URL inside containing an API key, when the Gitleaks tool is run, then this secret is not detected
To Reproduce
In some Xcode project, or maybe just in an empty folder, create an Info.plist file with the content bellow.
The run gitleaks in this folder.
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict>
	<key>SOME_SERVICE_URL_KEY</key>
	<string>https://some-domain/some-endpoint.php/get?apikey=aAbBcCdDeF&lang=en_US</string>
	<key>CFBundleAllowMixedLocalizations</key>
	<true/>
	<key>CFBundleIdentifier</key>
	<string>$(PRODUCT_BUNDLE_IDENTIFIER)</string>
	<key>CFBundleSpokenName</key>
	<string>Orange Design System</string>
	<key>UIViewControllerBasedStatusBarAppearance</key>
	<true/>
</dict>
</plist>

Expected behavior
I want gitleaks find the secret aAbBcCdDeF
Screenshots
Basic Info (please complete the following information):

OS: macOS 14.2.1
Gitleaks Version: 8.18.1

Additional context
Add any other context about the problem here.
cc @zricethezav

@pylapp aAbBcCdDeF does not pass the threshold of the generic rule's entropy value of 3.5: https://github.com/gitleaks/gitleaks/blob/8b8920d4c3fe5baf37dfe5675b73eef898555010/config/gitleaks.toml#L468-L475
This is why gitleaks does not flag apikey=aAbBcCdDeF as a secret

Anyway we can configure own secret patterns?

@zricethezav So what do you suggest? Reduce the entropy limit value? Increase it instead?

You probably found a solution to your issue by now, but for people wondering on how to get gitleaks to find such a simple password/key/... I would like to propose a solution that worked for me.
First, create a gitleaks.toml somewhere with the following content:
title = "Low entropy passwords"

[extend]
useDefault = true

[[rules]]
id = "low-entropy-api-key"
description = "Detected a Generic API Key, potentially exposing access to various services and sensitive operations."
regex = '''(?i)[\w.-]{0,50}?(?:access|auth|(?-i:[Aa]pi|API)|credential|creds|key|passwd|password|secret|token)(?:[ \t\w.-]{0,20})(?:[\s|']|[\s|"]){0,3}(?:=|>|:{1,3}=|\|\|:|<=|=>|:|\?=)(?:'|\"|\s|=|\x60){0,5}([\w.=-]{5,150})(?:['|\"|\n|\r|\s|\x60|;]|$)'''
entropy = 2.0
keywords = [
    "access",
    "api",
    "auth",
    "key",
    "credential",
    "creds",
    "passwd",
    "password",
    "secret",
    "token",
]

This just copies the default rule for generic API keys and adjusts the entropy (it has to be lowered to find simpler tokens, an entropy of 2.0 worked in my case). This TOML is intended to extend the default configuration, therefore, the rule ID has to be something else than "generic-api-key", otherwise the default rule will be prioritized.
Then, run gitleaks with this additional (extending) configuration:
gitleaks dir -vc ./gitleaks.toml <path/to/directory/to/be/checked>

This should now find your simple token.


transferOwnership() Function

This function transfers the contract to another owner address.

Here is that function in my old contract standard:
    function transferOwnership(address newOwner) public virtual onlyOwner {
        require(
            newOwner != address(0),
            "Ownable: new owner is the zero address"
        );
        _transferOwnership(newOwner);
    }


Dialog: add text commands for pausing text

Is your feature request related to a problem? Please describe.
Currently the only way to add dramatic pauses (or otherwise control the flow of dialog during one text) is to set the text speed to slow after a word, let the space between words play out slowly and then set it back to normal dialog speed.
Describe the solution you'd like
Bloody Lizard suggested this on Discord, but I've been wanting this myself for some time now. Two additional text commands would be great to have:

One to pause the dialog for a set amount of time. There could be different preset wait amounts to choose from (0.25s, 0.5s, 1s, 2s)
One to pause the dialog until A is pushed

Hi @Rebusmind, Good idea!
So the second suggestion was technically possible already with (quite hidden in documentation) GBVM escape sequences https://www.gbstudio.dev/docs/scripting/gbvm/gbvm-operations#escape-sequences
Using the text BEFORE \006\020 AFTER would write before then wait for the A button before continuing, and changing 020 for other values (it's an octal representation of an 8-bit mask) would change which buttons the dialogue would wait for.
But yeah, not very well documented and requires quite a bit more technical knowledge than I'd prefer so I've instead just implemented a new text code typing !wait will let you choose "Wait Until Button Pressed" which then lets you select the buttons

For your first suggestion, we don't have support for waiting for a certain amount of time just yet in GBVM, I'll have a think about that

Hey Chris, that looks like a great way to implement this. Thanks!
I hope you find a way to implement the other suggestion as well, as I would use that quite often (I love dramatic pauses 😆). Timers don't work during dialog, right?

Hi @Rebusmind managed to get waiting for time periods working too :-)
The !wait text code now has two extra options for waiting for an amount of seconds or frames

If you click the wait code a popup opens where you can change the amount of time and switch between seconds/frames

This will be included in the next update

That's perfect, thanks W(1.0s)so much! 😄


Scope of pattern matched variable inside if statements

If expr is true and is is used with an if statement, varname is assigned and has local scope within the if statement only.

Is this still true? While reviewing code I noticed that the scope of the assigned variable goes beyond the if statement:
        internal async Task<bool> TryServeCachedResponseAsync(ResponseCachingContext context, IResponseCacheEntry cacheEntry)
        {
            if (!(cacheEntry is CachedResponse cachedResponse))
            {
                return false;
            }

            context.CachedResponse = cachedResponse;
            ...
        }

The above code compiles and seems to work.

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

ID: 3126aaa9-8f5f-04f3-2d75-96477b6273cd
Version Independent ID: 3f4b3eac-11eb-2dd6-4f27-b24537367288
Content: is - C# Reference
Content Source: docs/csharp/language-reference/keywords/is.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Good catch @JunTaoLuo Your code is correct: cachedResponse is definitely assigned if false after the if block. That means it is definitely assigned in this example (because of the return inside the if.
Scope extends beyond the if, in this example.


Unmaintained project?

Ping. Are you still there?

Seems like you're not. With all your respect I'm creating a new version of your plugin, updated for modern Sublime Text!
Thank you for your initiative in creating the original.


Hacktoberfest command campaign docs

Docs added for:
COMMAND Commands Added:

command create: https://github.com/mattermost/mattermost-server/pull/9569 & https://github.com/mattermost/mattermost-server/pull/9734
command delete:  https://github.com/mattermost/mattermost-server/pull/9553
command team list:  https://github.com/mattermost/mattermost-server/pull/9531

CONFIG Commands Added:

config get:  https://github.com/mattermost/mattermost-server/pull/9558
config set: https://github.com/mattermost/mattermost-server/pull/9582
config show:  https://github.com/mattermost/mattermost-server/pull/9564

TEAM Commands Added:

team archive:  https://github.com/mattermost/mattermost-server/pull/9565
team search: https://github.com/mattermost/mattermost-server/pull/9512

WEBHOOKS Commands Added:

webhook create-incoming:  https://github.com/mattermost/mattermost-server/pull/9566
webhook create-outgoing:  https://github.com/mattermost/mattermost-server/pull/9779
webhook list:  https://github.com/mattermost/mattermost-server/pull/9528
webhook modify-incoming:  https://github.com/mattermost/mattermost-server/pull/9683

Thanks @jasonblais  I have addressed your feedback.


recordBone formatString fix

format-strings in recordBones don't use any prefixes in front of their fields.

ok, maybe there was a version in which this was implemented in this way.

yes I wasn't sure but the projects where I am using don't feature it. Why should it be in a rel, the values of a recordBone stand on their own, without a distinction between dest and rel.


⚠ Bofrost.lu has degraded performance

In 4e3d806, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2080 ms

Resolved: Bofrost.lu performance has improved in b557e4f.


any chance for english version?

Not even sure if this mod is working right as everything is in another language so not sure what if anything it is doing.

Hey, you can view English documentation and English configuration

https://github.com/ApliNi/AutoUpdatePlugins/blob/main/README_EN.md
https://github.com/ApliNi/AutoUpdatePlugins/blob/main/src/main/resources/Locales/config_en.yml


Direct AJAX calls not getting upgraded to https

As per https://groups.google.com/d/msg/zaproxy-users/byxt8JvbkZU/PNd6ASRBBAAJ
This could cause problems with lots of modern web apps, so we should investigate this asap.

Note: we have tested with JuiceShop locally and that seems to work well.
But it could be thats using relative paths, we should test with apps that use absolute paths, including the http:// part ...

Tried a very simple test locally, and when the ajax call specified the http url then the app failed when the HUD was enabled with: Blocked loading mixed active content “http://localhost:8080/bodgeit/tests/ajax_info.txt”

One option would be to replace all of the http:// strings with https:// ones for the domains that we know we have upgraded.
Would a simple global search and replace on the body work in most cases?

We would have to replace in "all" files (e.g. JavaScript). HSTS would be better as we wouldn't need to manually change the schemes (assuming the target is the same).

Oooh, I like that idea :)
I'll give it a try...

That has its problems too, like forcing to HTTPS even if you are no longer proxying through ZAP (it would be fine with launched browsers).

Hi @psiinon
The specific error I et in the browser console is:
"Blocked loading mixed active content..."
The error is happening on the page used for authentication when a Javascript library loaded on the page attempts to make a call to the server as part of the page load.

Thanks @wonboyn - is the http:// server url hqardcoded in the HTML? If so then we have a plan that should fix that...

Hello Simon, I am also noticing this issue. I am running a scala-play microservice locally. We have configured Chrome browser in our selenium tests to proxy through ZAP.  In ZAP 2.8.0, with HUD (0.4.0) enabled, request to localhost is converted to https and fails to load the page with the below message on the browser:
This site can’t provide a secure connection localhost sent an invalid response. ERR_SSL_PROTOCOL_ERROR.
When I disable HUD, the page is loaded (with http being the protocol).
But if I launch the browser from ZAP UI, with HUD enabled, the page is loaded even with https.
I can see a fix has been merged in 0.5.0. Do you think this issue would get resolved with the above fix?
Thanks.

@ejayaraman I would recommend disabling the HUD with your selenium tests. The HUD is designed to help manual testing and could well interfere with automated tests, which is why the HUD is disabled by default when ZAP is run in daemon mode.
If it works when using the browsers are launched by ZAP but not when you use another browser then this fix is unlikely to help. Its more likely that theres a configuration with your browser.

Thanks Simon. That helps.


Issues with console-plus

Before you read, i'm aware that this is somewhat of a duplicate,  but I haven't seen any issues pointing out this problem.
Currently, I started trying to use console-plus.js, as I've been using a new daemon which I can not run commands in. If I do try to run console-plus in node (cd serverdir/; node console-plus.js), it will echo consoleServer is listening on port 1234 and quit instantaneously.
Terminal output: http://i.imgur.com/019ZxCS.png
I'm not sure if i'm doing something wrong, or it's simply a bug, since the web console hasn't been updated in forever.

Yes @chewydev

Nothing here worked, but thank you guys for helping.

@chewydev im having troub;e going to <IP_ADDRESS>:1234 wehn i go its page wont sedn deta and its ruuningon port 1234

@chewydev can you help me with after it  starts on port 1234 icantgo to <IP_ADDRESS>:1234

@chewydev same it quits so i done the page realy fast and i cameto login but then it closes

@chewydev i figured it outr dont run winb  start.bat obnly node console plys it starts serverit self!!

@chewydev orks for me now !!

I'm not using windows for this, I haven't use start.bat

@chewydev starst.sh ?

That's weird, can you try on another server. Works fine on my end.

@MastaCoder Tried on every server on every one of my machines. They're all on Centos 7 x64 with the latest stable release of NodeJS and NPM.

You sure than open firewall ports correctly? all dependencies? correct node library installation?

@pabloalber84 All of that is correct, yes.

One thing @chewydev , you press "CTRL + C"?
You modify the console-plus.js?

I got the same problem on Ubuntu, but when I opened an issue, the only thing I got was a "Use Windows" tip

Which version of nodejs do you use?

@duwayy make sure not to open star linux .sh just sudo and start the console plus

I haven't modified console-plus.js, no.

What version of node do u use on your servers? @chewydev

@FantasyIsBae Sorry for the late reply, I use v6.11.1.

@chewythepig Updating it to the latest version should fix it.

Thanks dad


SRI integrity check for 6.6.3 from CDN is failing

What should happen?
Using the CDN links defined on the website I should be able to import foundation-sites.
Eg from the website:
<!-- Compressed CSS -->
<link rel="stylesheet"<EMAIL_ADDRESS>integrity="sha256-Q2fXBbKhtsyhYrSLa7hXCNV+FdhbQhyrjks8Kic0u/U=" crossorigin="anonymous">

<!-- Compressed JavaScript -->
<script<EMAIL_ADDRESS>integrity="sha256-tdB5sxJ03S1jbwztV7NCvgqvMlVEvtcoJlgf62X49iM=" crossorigin="anonymous"></script>

What happens instead?
I get the following error messages in Chrome:

Failed to find a valid digest in the 'integrity' attribute for resource<EMAIL_ADDRESS>with computed SHA-256 integrity 'ogmFxjqiTMnZhxCqVmcqTvjfe1Y/ec4WaRj/aQPvn+I='. The resource has been blocked.
localhost/:1 Failed to find a valid digest in the 'integrity' attribute for resource<EMAIL_ADDRESS>with computed SHA-256 integrity 'pRF3zifJRA9jXGv++b06qwtSqX1byFQOLjqa2PTEb2o='. The resource has been blocked.

As you can see, the integrity hashes differ.  For example for foundation.min.js: Q2fXBbKhtsyhYrSLa7hXCNV+FdhbQhyrjks8Kic0u/U= vs ogmFxjqiTMnZhxCqVmcqTvjfe1Y/ec4WaRj/aQPvn+I=.
I can confirm that Chrome is reporting the correct hash because when I check in the terminal I get the same hash:
curl<EMAIL_ADDRESS>\
| openssl dgst -sha256 -binary | openssl base64 -A

Yields ogmFxjqiTMnZhxCqVmcqTvjfe1Y/ec4WaRj/aQPvn+I=.

Hi @br3nt,
Thanks for letting us know.
You can simply change your hash value to ogmFxjqiTMnZhxCqVmcqTvjfe1Y/ec4WaRj/aQPvn+I=.
We will update it.
@joeworkman that has to be updated after every release.

We can copy the values from the jsdelivr pages:

The values are updated and the changes were deployed to the documentation website.


Found unknown Version 8827

Got my save (TheIsland.ark) off my Nitrado server and the tool gives that error.
Any update on the horizon, or is it an issue with it being an Xbox One save?

Oh dear - I don't know if that's an issue with an Ark update or the switch to SavegameToolkit causing a problem with Xbox saves.
Was it working with your Xbox save previously or is this a first attempt?

This was a first attempt, so I have no access to any pre-update copies of the save to test.

After the hex edit, it successfully finds all the tames, but only lists about 15 wild creatures.  Is that expected behaviour?  Does it not actually not creatures outside of render?
Or an indication that more than just the version changed?

Sorry for the delay.
The lack of wild creatures in the save is a serious worry. Normally, creatures that leave render range go into hibernation, and are saved in a hibernation area within the save file. This is read correctly on the PC version.
Savegametoolkit (by Flachdachs) is an almost exact translation of the abandoned ark-tools (by Qowyn) to C#. Adding support for changes to the file format may require more expertise than we collectively have. I'll speak with Flachdachs and see if there's anything we can do.

After using it a bit  more with multiple copies of the save, I can report that the tool "mostly" works.  I've seen some weirdness with creatures I know exist not appearing, and ones it reports not being there when I go in-game, but nothing I've properly documented and can't rule out user error completely.
Smart Breeder has definitely had some issues with some sections of the data though, so if they're both using the same framework I'd expect that neither is doing a perfect job.
If you could allow it to open v10 files as-is (possibly with a warning) it would probably be close enough for most people's purposes.

As much as I'd like to support the format, I'm not happy with partial support like this. I would fully expect complaints about creatures not being found, but there would also be a strong chance of exceptions during the loading process.
If we get better support from the savegametoolkit I will of course update.
For you personally I would recommend compiling a custom savegametoolkit DLL with a different version check.


[cleanup][broker] Cleanup finalPosition null-check in asyncFindPosition

Motivation
Cleanup finalPosition null-check in asyncFindPosition method. Because startPosition null-check has been checed at line1785.
Modifications
Cleanup finalPosition null-check in asyncFindPosition method
Verifying this change

[x] Make sure that the change passes the CI checks.

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/AnonHxy/pulsar/pull/28

/pulsarbot run-failure-checks

Codecov Report

Merging #19497 (82b6ff1) into master (459a7a5) will increase coverage by 0.29%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master   #19497      +/-   ##
============================================
+ Coverage     62.03%   62.32%   +0.29%
+ Complexity    26263    25800     -463
============================================
  Files          1820     1839      +19
  Lines        134039   134900     +861
  Branches      14756    14831      +75
============================================
+ Hits          83147    84080     +933
+ Misses        43265    43108     -157
- Partials       7627     7712      +85

Flag
Coverage Δ

inttests
24.62% <0.00%> (-0.03%)
:arrow_down:

systests
25.36% <0.00%> (?)

unittests
59.69% <100.00%> (-1.27%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.


Easy way to get the squad URL for a build?

Currently I'm building a squad URL in my code, but I wondered if there was an easier way to get this?
If not, where do you think would be a good place for this? Data in squad or something in squad-client?
ex:
build_details_url = urljoin(SquadApi.url, '%s/%s/build/%s' % (group.slug, project.slug, build.version))

# https://qa-reports.linaro.org/lkft/linux-next-oe/build/next-20200713

Hey Justin,
You mean url for UI right? I guess this could go to squad-client. I think it could be a method of each object that when called, it would return the url you're asking for. Let me give it a try

Correct, this link would go to the UI. It would be good for some of our reports to link back to squad or lava if the user would like to deep dive on issues.

This is what I've been using in our code to replicate this:
def squad_build_url(squad_url, group, project, build):
    return requests.compat.urljoin(
        SquadApi.url, "/".join([group.slug, project.slug, "build", build.version])
    )


obdc connection

Hey Good Day to all Fluent Nhibernate
I would like to ask if it is possible to used fluent NHibernate for obdc connection bcoz Dentrix SDK (API) only allow obdc connection. If not possible what is the best way/technique
Best regards and God Bless
John Jake

It's not something that's built in, but you can do it - take a look at this SO question

in that example i got error in dialect

or how do i customize dialect to create like obdcMyDialect

What kind of error? What does the error message say?
2015-11-25 17:36 GMT+03:00 johnjake<EMAIL_ADDRESS>
or how do i customize dialect to create like obdcMyDialect
—
Reply to this email directly or view it on GitHub
https://github.com/jagregory/fluent-nhibernate/issues/332#issuecomment-159624899
.

--
Yours faithfully,
Gleb

public static OdbcConfiguration MyDialect // <-- insert any name here
{
get
{
// insert the dialect you want to use
return new OdbcConfiguration().Dialect<NHibernate.Dialect.MyDialect>();
}
}
missing assembly reference MyDialect does not exist

So, what dialect does your data source use?
2015-11-25 18:26 GMT+03:00 johnjake<EMAIL_ADDRESS>
[image: dialect]
https://cloud.githubusercontent.com/assets/4056983/11401102/c4de6168-9345-11e5-8a51-7437b0898106.png
—
Reply to this email directly or view it on GitHub
https://github.com/jagregory/fluent-nhibernate/issues/332#issuecomment-159640468
.

--
Yours faithfully,
Gleb

the c-treeACE SQL database

I'm not familiar with that database
2015-11-25 19:01 GMT+03:00 johnjake<EMAIL_ADDRESS>
the c-treeACE SQL database
—
Reply to this email directly or view it on GitHub
https://github.com/jagregory/fluent-nhibernate/issues/332#issuecomment-159654928
.

--
Yours faithfully,
Gleb

Sir for the obdc is it possible to just used db2 ? can you give me sample Sir obdc using access or any obdc form maybe it will work. Thanks a lot Sir

Frankly, I have no idea
2015-11-27 4:42 GMT+03:00 johnjake<EMAIL_ADDRESS>
Sir for the obdc is it possible to just used db2 ? can you give me sample
Sir obdc using access or any obdc form maybe it will work. Thanks a lot Sir
—
Reply to this email directly or view it on GitHub
https://github.com/jagregory/fluent-nhibernate/issues/332#issuecomment-160017360
.

--
Yours faithfully,
Gleb

Closing as it seems to be not related to Fluent


Problem: all input and output to sputnikvm should be C friendly

Solution: The product of sputnikvm is machine.used_gas() currently returns a utils::gas::Gas type. Let's come up with a single unified return type containing Gas, and stuff like Code (next point of execution to pick up from) from which we can extract nicely.
This will allow us to format it into what ever way we need and send it over the wire/ABI/etc

knowledge disseminated - closing


Split sqllite and sql server migrations

Part of https://github.com/aspnet/templating/issues/309
@seancpeters does this look right?  We already have a UseLocalDb symbol we can use...  I tried this and I don't think the rename/exclusion is working as I have it, since both sets of files are showing up still...Can you check if there something obviously wrong?

@HaoK - I believe that the problem is because template symbols are case-sensitive. In the symbols section, there is UseLocalDB, but in the source modifier conditions it's UseLocalDb.

@seancpeters that was it!  After talking with @ajcvickers I moved things around a bit, so there's now a Data\SqlLite and Data\SqlServer which contain the 3 migration files, and they get renamed to Data\Migrations.
This should be ready for review now, the functional mvc/razor tests are passing

Updated cleaning up the ApplicationDbContextSnapshot if defs, and added extra IndividualLocalAuth checks to the conditions, look good @seancpeters ?  The files on disk are the same either way, so functionally I don't think it makes a difference


Compilation on windows

Hi
I did all the steps but am stuck with make -j4 on windows. Can you tell me the how to execute this command on windows

Hi,
Sorry, I don't have access to a Windows machine and I don't know what is required to support a native Window build.

Also can you clarify if the correct command is
git submodules update --init
or
git submodule update --init

Just to point out i compiled it in ubuntu and the compiled version works in windows.  :)

i might sound silly but can i want some information regarding something. i know its not the right place to ask.
I am trying to use tpm as security device for fido.
https://medium.com/@herrjemand/verifying-fido-tpm2-0-attestation-fc7243847498
But the code defined here is in javascript. I was wondering how to make the code in site above work with emulator. If you can guide me it will be really helpful. Like is there a javascript version of tpm2-tss? How do i make it running

Most libraries here are written in C/C++ (ibmswtpm, tpm2-tss). They're compiled to WebAssembly bytecode, and then executed in the wasm framework in the browser. The javascript part is just the glue that connects the html UI to the wasm executable. I'm not familiar with FIDO, so I'm afraid I can't help you there.


⚠ API - Lens has degraded performance

In 715c800, API - Lens ($LENS_API_URL) experienced degraded performance:

HTTP code: 503
Response time: 346 ms

Resolved: API - Lens performance has improved in 06312dc after 9 minutes.


Fixed unwrap on None when importing embedded assets

Objective

Fixes unwrap on None problem when importing a embedded asset. Just a slight change.

Solution

Use the 0th instead of 1th in macro embedded_path!
Before:
let after_src = file!().split($source_path).nth(1).unwrap();
After:
let after_src = file!().split($source_path).nth(0).unwrap();

The embedded assets can be now imported correctly.

I think this deserves a more comprehensive fix. I'm working on another PR.

I think this deserves a more comprehensive fix. I'm working on another PR.

Yeah, definitely. But this simple and quick fix works well in my project. Hope there's no more people being confused by this weird problem. And wish you a smooth completion of your PR. : )

Going to move forward with #10383 :) Thanks for the submission regardless, this was helpful to understand the problem.


Uuid as a string type and updates without changes in the entity

Description
I've described the problem here: https://sarvendev.com/2024/10/uuid-orm-and-strange-deadlocks/
Steps to reproduce

create an entity with string $id, mapped as uuid_type
save this entity
get the entity from the database and try to save it without changes
UoW treats this entity as changed because original data contains a Uuid object (returned from UuidType::convertToPHPValue), and the actual data in the entity object contains a string

UPDATE table SET id = '2082c2e5-9eb5-43c1-809e-82265da220f0' WHERE id = '2082c2e5-9eb5-43c1-809e-82265da220f0'

Expected behavior
UuidType always returns the same type, and it isn't possible to make such a mistake. UuidType::convertToDatabaseValue should probably only accept UuidInterface as a parameter. I was thinking about implementing that, but it's a breaking change, so I am not sure about that. Perhaps the same problem occurs also with other types. I've just created this issue to open the discussion on how it could be fixed.

Can you provide some code I can use to reproduce this issue? I've read the blog post and your steps here, but I want to make sure I'm able to fully reproduce the issue.
Thanks!

@ramsey I've just prepared a repository with a simple test to showcase this problem: https://github.com/sarven/uuid-bug-example/blob/main/tests/Repository/EntityRepositoryTest.php
UUID in the entity is represented as a string, it works, but during an update without any changes, it generates additional empty update.
UPDATE entities SET id = ? WHERE id = ?


Deploy Grafana before Worker and after Monitoring stack

How to categorize this PR?

/area logging
/kind enhancement
What this PR does / why we need it:
When a shoot creation is stuck at worker creation, the logs from the not joining nodes are not visible in any UI, since Grafana is deployed alongside the monitoring stack, which is deployed after successful reconciliation of the worker. With this PR we deploy the Grafana after deploying the worker but we don't wait for it to be successfully reconciled. This approach sometimes leads to exclusion of the MCM dashboard from operators Grafana because the ConfigMap containing it is deployed after the deployment of the worker. In such a case, the aggregation of all extension dashboards precedes the deployment of the MCM one. The missing dashboard will be added after the next shoot reconciliation. To avoid the absence of the MCM dashboard and all dashboards in the future which will be deployed after the worker the Grafana is deployed a second time after the monitoring stack.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
/invite @istvanballok @wyb1
Release note:

Deploy Grafana before Worker and after Monitoring stack

Maybe we could redeploy Grafana again at the end of the shoot flow to ensure all dashboards are present?

This is good Idea, but as as second time or as third?

but as as second time or as third?

I didn't quite get this, what do you mean? :)

I didn't quite get this, what do you mean? :)

Just joking :). I know that I have to move the second deployment of the Grafana at the end.

BTW, @rfranzke I didn't saw a better place for the second Grafana deployment. It looks like deploying it alongside the Monitoring t is the best place for now.
@vpnachev suggested deploying the first Grafana right after the logging, so I move it.


Add CertManagerExits logic for PackageController Installation

The existence of cert-manager is required to ensure proper operation of the PackageController. We modify the cluster creation workflow and imperative package installation command to add in the CertManagerExits logic, and guide users to handle the case where cert-manager is not present in the cluster.
Issue #, if available: https://github.com/aws/eks-anywhere-packages/issues/278
Description of changes: Modified the cluster creation workflow and imperative package installation command to add in the CertManagerExits logic
Testing (if applicable): Tested in cluster creation workflow and imperative package installation
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


opensuse 15.2 set display single view cause freeze ,see video

Are you using the latest driver?
yes!
Are you using the latest EVDI version?
evdi 1.7.2
If you are using a DisplayLink device, have you checked 'troubleshooting'
on DisplayLink's website?
yes
Is this issue related to evdi/kernel?
maybe

Linux distribution and its version
Linux kernel version
Xorg version (if used)
Desktop environment in use
1.opensuse 15.2
2.kernel 5.3.18
3.xorg version 1.20.3
4.opensuse 15.2 gnome

video file


Problem handling Kafka exceptions

I am still unsure of the underlying cause and whether this problem is endemic to all installations but it is repeatable on my development workstation, an Intel iMac at home.
The symptom is a slew of errors like:
%5|1647498302.003|REQTMOUT|rdkafka#consumer-2| [thrd:sasl_ssl://kb-2.prod.hop.scimma.org:9092/2]: sasl_ssl://kb-2.prod.hop.scimma.org:9092/2: Timed out FetchRequest in flight (after 1799295ms, timeout #0)
<snip>
%4|1647498302.005|REQTMOUT|rdkafka#consumer-2| [thrd:sasl_ssl://kb-0.prod.hop.scimma.org:9092/0]: sasl_ssl://kb-0.prod.hop.scimma.org:9092/0: Timed out 1 in-flight, 0 retry-queued, 0 out-queue, 0 partially-sent requests
%3|1647498302.005|FAIL|rdkafka#consumer-2| [thrd:sasl_ssl://kb-0.prod.hop.scimma.org:9092/0]: sasl_ssl://kb-0.prod.hop.scimma.org:9092/0: 1 request(s) timed out: disconnect (after 115074987ms in state UP)

The Exception is raised in: /Users/areeda/mambaforge/envs/igwn-py39/lib/python3.9/site-packages/adc/errors.py line  12:
My problem is that this kills the main thread of our DQR alert listener without any way to catch the exception.
I have a merge request going in next.

Can you describe a bit further the circumstances under which this happens? I don't think I've come across anyone having quite this problem listening to the Hopskotch brokers before, and we have at least one listener (hopbeat-monitor) that we leave running for quite extended periods. I also do my development on Intel Macs, and while I haven't seen this myself I'd be interrested to try to recreate it to better understand the context of your PR.

Thank you for looking into this.  I wonder if my use of multiprocessing has anything to do with it?
The listener is fairly simple following the outline in the documentation. When an alert is received it is filtered by alert type and FAR. If it passes a JSON file with the payload is written to disk and a dict is added to a Queue. There are multiple worker processes which ingest that Queue and construct a command line to generate and submit a Condor DAG to create a Data Quality report.
What I've been seeing is the Kafka error between 5 and oh 30 hrs of running. During that time we process an mdc_superevent every hour.
The setup has been running on ldas-pcdev4 at CIT for 52 hrs without a problem.
Another possibility I just thought of is that I have DHCP set up on the Mac with both WiFi and wired connections. The DHCP server set the IP for both based on MAC address so it doesn't change  wonder if lease renewal is somehow messing with hop/SASL. I will force a renewal to check.
The code is pre-release but available with LIGO credentials at git@git.ligo.org:joseph-areeda/dqralert.git

One more detail: it has been running in the igwn-py39 environment.

Given that librdkafka uses multithreading internally, using multiprocessing does sound a bit dangerous; it might be worthwhile to test with multiprocessing.set_start_method('forkserver'), which should be safer, to try to rule that out.
I'm not a member of LIGO, so I can't access any of its internal resources.

Thank you I'll try it.
I was going to wait for release but this can be put up on github if you want to look.

Note to self: Changing the start method from spawn to fork interferes with running under the debugger in PyCharm. but not running on the command line.
Running with the "fork" method (on the command line) had many more of those Kafka errors. Not exactly proof but it sure smells like a thread-safety problem.
The good news is that the PR seems to allow the program to recover.
I will redesign my program to remove multiprocessing.

I don't know if this is a good place to ask but how are people responding to alerts that come in at random times with strict latency requirements?
With multiprocessing off the table, and multithreading locking when using os.system() I can think of a couple of ideas:

Eat the latency for the processing job, hoping it will be quick. Generating and submitting a Condor DAG is only a few seconds but if users add something to wait for data to appear or preprocess this could change.
We could open a named pipe and do all the multiprocessing in another program.
We could write the alert payload to a file and have another program looking for it.

I'm curious how others are addressing this.

We removed  the multitasking option from the process but using multiprocessing.run to execute the external program to process the alert.  I can report 75 hours of processing alert on my development Mac.  On the LIGO clusters we have seen 6 days without the error.
Taken in aggregate this is enough circumstantial evidence to conclude that multiprocessing is incompatible with the libraries.
I will leave the closing of this pull request open for the maintainers to decide if Kafka runtime exceptions should be passed to the calling programs.
I do appreciate the help from cnweaver. The change in frequency of the error by switching from spawn to fork was the key piece of evidence for me.

I'm surprised that the 'forkserver' multiprocessing mehtod isn't a safely viable option. Unfortunately, I don't think I have any other particularly good ideas for for doing non-trivial processing from the same process without introducing delays in receiving further messages. I'll discuss with the rest of the SCiMMA team and see if we can think of anything.
I will still review the associated PR, hopefully tomorrow.

It is interesting that the problem seems to only occur on my Mac but not on the Scientific Linux machines at Caltech.
I'm assuming that it is explained by the random nature of thread safety problems. The exact same [source] code is used in both places.

We received a suggestion for how to handle simultaneous alerts.
Kafaka provides the concept of a consumer group that will distribute events across a group of listeners. My application has to wait for the package between us and adc, kafka to expose it but it will allow a very clean way to process multiple simultaneous events by simply running multiple listeners.

Okay, that does sound like a reasonable approach, and adc-streaming should already allow normal use of consumer groups.

After running for 20 days we got one of these errors on my Mac. Still running under Scientific Linux  without seeing it.
I've been using the Merge Request version and our program recovered nicely.
%3|1648594740.001|FAIL|rdkafka#consumer-2| [thrd:sasl_ssl://kb-0.prod.hop.scimma.org:9092/0]: sasl_ssl://kb-0.prod.hop.scimma.org:9092/0: Receive failed: SSL transport error: Operation timed out (after 280131137ms in state UP, 1 identical error(s) suppressed)
03-29 15:59:00 - ERROR: internal kafka error: KafkaError{code=_TRANSPORT,val=-195,str="sasl_ssl://kb-0.prod.hop.scimma.org:9092/0: Receive failed: SSL transport error: Operation timed out (after 280131137ms in state UP, 1 identical error(s) suppressed)"}
%3|1648594742.030|FAIL|rdkafka#consumer-2| [thrd:sasl_ssl://kb-1.prod.hop.scimma.org:9092/1]: sasl_ssl://kb-1.prod.hop.scimma.org:9092/1: Receive failed: SSL transport error: Operation timed out (after 69725729ms in state UP, 1 identical error(s) suppressed)
03-29 15:59:02 - ERROR: internal kafka error: KafkaError{code=_TRANSPORT,val=-195,str="sasl_ssl://kb-1.prod.hop.scimma.org:9092/1: Receive failed: SSL transport error: Operation timed out (after 69725729ms in state UP, 1 identical error(s) suppressed)"}

I am not sure what this means.

I have merged the corresponding PR (sorry it took so long!). I'm not quite sure when we'll get this into a release, as the 2.1.0 release ended up in a slightly odd state due to upstream issues, so I will leave this issue open at least until we get that out.

Hi @cnweaver I am wondering about the status of this issue since we are seeing this -195 error
KafkaException: KafkaError{code=_TRANSPORT,val=-195,str="Failed to get metadata: Local: Broker transport failure"}

still occasionally (with v2.3.0).

Unfortunately, librdkafka produces this error very readily, and with very little transparency to debug its root cause. (In fairness to librdkafka, one of the major causes of these errors is lack of network connectivity, which it can't really do anything about, and which may be a transient problem.) As such, it's very hard to say whether you're seeing something related to the multiprocessing issue mostly discussed in this thread, or not. Any contextual information you can give which might help us narrow the cause down would be valuable; unfortunately I do not myself have any very good ideas for how to obtain such.


chore: remove unnecessary url detection and use nextUrl instead

Change Summary
This PR removes unnecessary URL detection and uses request.nextUrl instead to provide request URL.
Merge Checklist

[x] PR has a Changeset
[ ] PR includes documentation if necessary
[ ] PR updates the boilerplates if necessary

@stephancill I checked the values on vercel and nextUrl is always filled by hostname, etc.


Add language support for Ukrainian

Description
Add language support for Ukrainian 🇺🇦
Other changes
N/A
Tested
Manually
How others should test
The app correctly displays in Ukrainian if you select that as your language.
The app is launched in Ukrainian for a new user with device language uk-UA
Related issues

Fixes #2069

Backwards compatibility
Yes

@mergifyio refresh

Hey @MuckT as instructed to us We are skipping this PR, please let us know once it is integrated and ready for testing
Slack link : link
CC: @kathaypacific
Thank..!


Ignite 2.4.6.b2

PR created for team city test runs. Please don't merge.

PR is not associated with a ticket. Please see how to associate
https://cwiki.apache.org/confluence/display/IGNITE/How+to+Contribute#HowtoContribute-Creation
Closing PR (which seems to be test-only) due to inactivity. Feel free to reopen this PR if it is actual.


Not Working on both IOS/Android

when i start recording on android it automatically stop after sec and video not save on phone memory, and on ios side i got exception when i start recoding "Recording not active".
please help me to fixed this issue

@ahsanparacha02 I also had this problem.

@yutasuzuki I wonder if you were able to make it work in your example app itself. I have tried multiple times with your example and with new project but no luck. In the android it stops just after starting and nothing works afterwards but in iOS it does not work at all.

May I check this?

Is the version of react-native-record-screen 3.0?
Device or emulator?
(The emulator won't work on iOS. Android confirms that it works with Pixel 3a API 30).

The devices we've checked here are listed below.

iPhone Xs (iOS 13.7)
Pixel 4a (Android 10)

I only have an iPhone Xs and a Pixel 4a, so it may be difficult to solve this problem with other devices.

I test this lib on real devices with iOS 14 and it works fine :) But at the same time it doesn't work on simulators with different iOSs , I think it is because of permissions of recording screen that unavailable on simulators.

@Mopiiex20 I know that when I use the simulator, it stops with an error about the camera.
But if you turn off the camera's permission, it won't work as a module.
Therefore, the iOS simulator is not supported
I am not familiar with iOS development.
If you know how to get the camera to work in the simulator, please let me know.

If people want to use it in the iOS simulator, we'll create a new Issue.

@yutasuzuki Earlier I use this lib to record screen , it record screen on simulators, and probably could help you to implement this feature , as addition this lib use scale factor of different devices to record screen more accurately.

@Mopiiex20 Thanks.
I tried ASScreenRecorder myself. With this library, when I left the video or camera playing, that part of the screen recording turned black.
That's why I made a react-native-record-screen using replaykit.

@yutasuzuki what about green lines when save record ?

@Mopiiex20 I've created a new Issue, so let's talk about it there. #5

@yutasuzuki I managed to run this on android with adding permission check for RECORD_AUDIO and by doing same on iOS as well.

@adeshpandey Good!

I close this Issue.
If there is a problem, open this issue again.


path to download rordenlab/dcm2niix arm64 outdated

Hi Jakob,
thank you for all your efforts.
Problem
The links in dicom_io.py (L23) to the arm64 / Mac version of rordenlab/dcm2niix are not working anymore.
Solution
The link should be updated to https://github.com/rordenlab/dcm2niix/releases/latest/download/macos_dcm2niix.pkg. For Reference see here.
Temporary and quick solution:

load dcm2niix manually to environment
comment out L54-58

#    config_dir = get_config_dir()
#    dcm2niix = config_dir / "dcm2niix"

#    if not dcm2niix.exists():
#        download_dcm2niix()

use dcm2niix directly L60:

    subprocess.call(f"dcm2niix -o {output_path.parent} -z y -f {output_path.name[:-7]} {input_path} {verbose_str}", shell=True)

To discuss
I did not fork and PR these changes directly due to the following considerations:

I am not sure, whether the macos_dcm2niix.pkg-pkg works in both, arm64 and x86_64 environments and have only the option to test it at arm64 (working). If it works also for  x86_64, we could get rid of the differentiation in L24.
The loaded files are not *.zip, but *.pkg. Therefore, downloading macos_dcm2niix.pkg would break zipfile.ZipFile in L36.
I am not sure whether you prefer to keep download_dcm2niix()or want to add dcm2niix to the requirements.

Thank you again and all the best,
Felix

Thanks for pointing this out. I changed the code and documentation: on MacOS automatic download of dcm2niix is no longer possible. You have to manually install dcm2niix now.


Missing .stopPropagation() when navigating by keys

I've found a bug in Select2 4.0-beta
Assume, Select2 is placed within a form with a command (submit) button. You can navigate by arrow keys down and up in the dropdown. If you now press the Enter key, the form will be submitted. This is a default behavior in browsers when user hits Enter. Select2 has evt.preventDefault(); for all keys in this.on('keypress', function (evt) {...}, but this is not enough. You have also to add evt.stopPropagation();. I've added evt.stopPropagation(); after all evt.preventDefault(); and the problem was gone. Can you consider the fix for the final 4.0 release please? Thanks.

This is how keypress callback looks like
    this.on('keypress', function (evt) {
      var key = evt.which;

      if (self.isOpen()) {
        if (key === KEYS.ENTER) {
          self.trigger('results:select');

          evt.preventDefault();
          evt.stopPropagation();
        } else if (key === KEYS.UP) {
          self.trigger('results:previous');

          evt.preventDefault();
          evt.stopPropagation();
        } else if (key === KEYS.DOWN) {
          self.trigger('results:next');

          evt.preventDefault();
          evt.stopPropagation();
        } else if (key === KEYS.ESC || key === KEYS.TAB) {
          self.close();

          evt.preventDefault();
          evt.stopPropagation();
        }
      } else {
        if (key === KEYS.ENTER || key === KEYS.SPACE ||
            ((key === KEYS.DOWN || key === KEYS.UP) && evt.altKey)) {
          self.open();

          evt.preventDefault();
          evt.stopPropagation();
        }
      }
    });
  };

By the way, you already had evt.stopPropagation() for other events, but it was missing for keypress.
Could you fix this bug for the upcoming 4.0 final release? Thanks a lot for your effort!

We specifically removed this for most cases with the intention of adding in a module/option later that would allow for the events to be stopped at different levels.
That is still our intention, and the modules will be available when 4.0.0-rc.1 is released.

Ah, good to know. Thanks!
Am 30.01.2015 15:17 schrieb "Kevin Brown"<EMAIL_ADDRESS>
We specifically removed this for most cases with the intention of adding
in a module/option later that would allow for the events to be stopped at
different levels.
That is still our intention, and the modules will be available when
4.0.0-rc.1 is released.
—
Reply to this email directly or view it on GitHub
https://github.com/select2/select2/issues/2974#issuecomment-72207226.

First, thanks for the quick fix. Is it only available in the full version? How to activate this behavior? Automatically or per option?

+1 I am struggling right now while trying to find documentation to actually use this 4.0 feature.  We need to stop propagation but there is no documentation around this feature / change.


IDS: ContractOffer is missing values for provider and consumer

Problem statement
in the ContractValidationServiceImpl there is a check in place that requires that both the provider and consumer fields of a ContractOffer be not-null:
private boolean isMandatoryAttributeMissing(ContractOffer offer) {
    // TODO add contractStart and contractEnd check as soon as the Infomodel serializer is used
    return offer.getProvider() == null || offer.getConsumer() == null;
}

However, those values are never set, which causes all contract negotiations to be declined.
Proposed solution
The provider and consumer fields must be set during the creation of the ContractOffer, which is done in the ContractOfferServiceImpl.
Suggestion:

provider: the offering connector's ID as URN
consumer: the ID of the ParticipantAgent for whom the offer is generated

What is the ID of the ParticipantAgent? The sub of the corresponding JWT token?
What about using the "referringConnector" property if set?
See also https://github.com/International-Data-Spaces-Association/IDS-G-pre/blob/main/Components/IdentityProvider/DAPS/README.md

@MoritzKeppler The DSC uses the connectorId, so the referringConnector as suggested by you. Nevertheless, the IDS in general defines something different (as I understood): Here, consumer and provider are represented by an ids:Participant. The reason why we never implemented this, was that there was no ParIS and no defined participant handling.

Actually, the case where they are not set is an important one (for an offer, not an agreement, an agreement needs both defined): consumer = null -> 'this offer is applicable to any Participant who is interested'
So, is this if clause in the isMandatoryAttributeMissing really needed?


sqrt_qfe for BLS12381TwistCurveFieldElement

sqrt_qfe for BLS12381TwistCurveFieldElement
Description
Implementation of sqrt_qfe for BLS12381TwistCurveFieldElement.
Type of change

[X ] New feature
[ ] Bug fix
[ ] Optimization

Checklist

[ ] Linked to Github Issue
[ ] Unit tests added
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

[ ] This change is an Optimization

[ ] Benchmarks added/run

Codecov Report

Merging #407 (36afd5a) into main (a80c169) will increase coverage by 0.03%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##             main     #407      +/-   ##
==========================================
+ Coverage   93.81%   93.85%   +0.03%
==========================================
  Files          61       62       +1
  Lines        7643     7741      +98
==========================================
+ Hits         7170     7265      +95
- Misses        473      476       +3

Impacted Files
Coverage Δ

..._curve/short_weierstrass/curves/bls12_381/curve.rs
98.68% <ø> (ø)

...c_curve/short_weierstrass/curves/bls12_381/sqrt.rs
88.77% <88.77%> (ø)

math/src/field/element.rs
96.77% <100.00%> (+2.34%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more