feat: Implement SHOW CREATE FLOW

I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
close: #3939
What's changed and what's your intention?
Implement SHOW CREATE FLOW
Checklist

[ ] I have written the necessary rustdoc comments.
[ ] I have added the necessary unit tests and integration tests.
[ ] This PR requires documentation updates.

Would you like to add some sqlness tests for SHOW FLOW?
BTW, It seems we need to add sqlness tests for creating flow and dropping flow cc @discord9 @waynexia

Would you like to add some sqlness tests for SHOW FLOW? BTW, It seems we need to add sqlness tests for creating flow and dropping flow cc @discord9 @waynexia
Sure, i'll add tests after some review feedback, because the output format of SHOW CREATE FLOW might change.

Please add some sqlness tests following https://github.com/GreptimeTeam/greptimedb/blob/main/tests/README.md

It seems that the create flow statement randomly fails.

Sorry but currently flow only implement for standalone mode, could you move the testcase out of common folder for now? I will put it back when we have distributed support for flow service?

Please add some sqlness tests following https://github.com/GreptimeTeam/greptimedb/blob/main/tests/README.md

It seems that the create flow statement randomly fails.

@irenjj Thanks for your contribution! Would you share your email address to me (by email to tison@greptime.com)?
I'd like to send you a letter for further engagement :D

@irenjj Thanks for your contribution! Would you share your email address to me (by email to tison@greptime.com)?
I'd like to send you a letter for further engagement :D

Sure! I've already sent the email.


Specify SPDX tag for semver

The semver utility is not widely packaged by distributions. As such projects may wish to incorporate the script directly as a helper to support packaging and versioning of a project.
In such cases, it is important to ensure that the licence of the semver utility is maintained.
Provide an Apache-2.0 SPDX tag [0][1] within the semver script to ensure the licencing is clear.
[0] https://spdx.dev/ids/
[1] https://spdx.org/licenses/Apache-2.0.html
Signed-off-by: Kieran Bingham<EMAIL_ADDRESS>

Thanks for your contribution @kbingham

Thank you!
I'll use this latest version to support my release management for the libcamera project. This semver script is very useful to support the process. Thanks!


Unrecoverable error on lists

module Test

unfold let clamp_mask  = Seq.createL [ 255uy; 255uy; 255uy; 15uy;
                                                        252uy; 255uy; 255uy; 15uy;
                                                        252uy; 255uy; 255uy; 15uy;
                                                        252uy; 255uy; 255uy; 15uy ]

returns
not an lid: Cons: Prims_list uint8_t
Fatal error: exception Warnings.Fatal("Unrecoverable error")

Interestingly, the following
module Test

unfold let clamp_mask  = Seq.createL [ 255uy; 255uy; 255uy; 15uy;
                                                        252uy; 255uy; 255uy; 15uy;
                                                        252uy; 255uy; 255uy; 15uy;
                                                        252uy; 255uy; 255uy; 15uy ]

unfold let mask_list = [ 255uy; 255uy; 255uy; 15uy; 252uy; 255uy; 255uy; 15uy;
                         252uy; 255uy; 255uy; 15uy; 252uy; 255uy; 255uy; 15uy ]

now goes further, and drops the entire file with
[Checker.lookup_type] internal error Prims.list not found
Warning 4: <toplevel>: Malformed input:
Warnings.Fatal("Unrecoverable error")
Dropping Test (at checking time)

⚠ Dropped some files while checking

Yes, will fix. The solution is to just drop a declaration, not the entire file.


[FEATURE] Need FieldType which allows special chars for regexp

Hi,
within the Quagga plugin I want to build AS-Path lists for BGP.
In the configuration file they look like this:
ip as-path access-list 1 permit ^$
ip as-path access-list 2 permit _12340$
ip as-path access-list 3 permit _8767$

EDIT: They could also look like
ip as-path access-list  1 ^6[4-5][4-6][0-9][0-9]$
When I use FieldType Text I can insert the value, but when creating a ModelRelation field pointing to it, it wont get recognized. If I don't use the special characters it works.
Perhaps there's a way how to avoid creating a new FieldType?
Thanks!

Hi Michael,
Can you explain with a short example what your model looks like for this piece?
The relation field is meant to model a normal foreign key type of relation, a fuzzy match might not be the best solution here, but I need some more context to help you out.
I don't have a lot of time available, so please excuse me for not reading the current codebase.
Best regards,
Ad

Hi,
thanks for taking the time!
I have also an issue open at the plugins Project:
https://github.com/opnsense/plugins/issues/150
There you can see what I want to do. Please don't take care about the other problem within this issue because I'll restructure the whole process, but AS Path lists are regexp in config.
Here is a sample config from a Quagga BGP daemon:
router bgp 65550
 network <IP_ADDRESS>/24
 neighbor <IP_ADDRESS> remote-as 8767
 neighbor <IP_ADDRESS> update-source eth0
 neighbor <IP_ADDRESS> route-map SET_IN_FILTER in
!
ip as-path access-list 3 permit _8767$
!
route-map SET_MNET_IN_FILTER permit 10
 match as-path 3
 set local-preference 300
!

This is the config in text file.
So, I created a tab AS-Path (like in screenshot in plugins/ #150), then I create a tab Route-Map and there is the ModelRelation field to the AS-Path list. In the end I have to add the Route-Map with the referenced AS-Path list to a specific neighbor.
Hope this helps :)

@AdSchellevis Sorry, forgot the model:
This is the field for the AS Path input:
https://github.com/mimugmail/plugins/blob/bgp/net/quagga/src/opnsense/mvc/app/models/OPNsense/Quagga/BGP.xml#L105
And the reference in the route-map tab is:
https://github.com/mimugmail/plugins/blob/bgp/net/quagga/src/opnsense/mvc/app/models/OPNsense/Quagga/BGP.xml#L135
When only using number like:
ip as-path access-list 3 permit 8767
I can select the list, but with:
ip as-path access-list 3 permit _8767$
there's only "none"

@mimugmail just to be sure, when your text field contains a regex and you show the name field, it can't find the uuid anymore (no display in the gui)? if that's the case, it sounds like a bug somewhere.
The relation fields use the uuid of the array item, so the actual content shouldn't really matter.

@AdSchellevis Hm, I'm a bit embarressed now. After you said it sounds like a bug I started browser debugging to find some errors but now it works. I don't really know why, but it's getting saved now correctly. Perhaps I had a wrong relation set but I did so many changes since then ... can't say why.
Sorry for wasting your time :(

@mimugmail no problem, things happen, enjoy the rest of your coding journey and feel free to open an issue when you run into trouble.


only identify one SNAP-OG

Dear Jacob
I used orthofinder for gene family clustering of five species (Ab, Ad, Ak, As, Lo). Then I would like to use orthoSNAP with default parameter to classify multi-copy gene families. But there are many gene families whose trees actually have multiple clades. But orthoSNAp can only identify one SNAP-OG. For example, the following gene families.  orthoSNAP identified only one SNAP-OG, including Ab|AB06G03741-RB, Lo|LO08G03008-RA, As|AS05G03226-RB. But why wasn't the other clade identified as SNAP-OG？Can another clade be a SNAP-OG？
((Ab|AB06G03741-RB:0.0993161,(As|AS05G03226-RB:0.0364093,(Lo|LO08G03008-RA:0.0337749,Lo|LO08G03009-RA:0.0289553)0.996:0.0172954)0.992:0.0215731)1:0.0494888,((Lo|LO08G03007-RB:0.0387437,(As|AS05G03228-RB:0.0205169,As|AS05G03236-RA:0.0262063)0.897:0.0062029)1:0.157462,((Ad|AD03G00060-RA:0.0339494,(Ad|AD03G00059-RA:0.0129773,((Ad|AD08G00004-RA:0.0885369,Ad|AD08G00001-RC:5e-09)0:6e-09,Ad|AD08G00003-RA:0.0236448)0.763:0.00254957)0.632:0.00239112)0.912:0.0083526,(Ab|AB06G01323-RA:0.00687747,(Ab|AB06G01309-RA:0.00290853,Ak|AK09G04755-RB:0.00193846)0.734:0.0009175)0.991:0.0153459)1:0.267442)1:0.0494888);

Best regards
Xiaoxu

Hi Xiaoxu,
I hope you are doing well.
You are only getting one SNAP-OG because the genes in the green box do not constitute a proper SNAP-OG. This is in part due to the relationship between the two Ab paralogs and Ak.
Hope this helps!
best,
Jacob

Dear Jacob
Thank you for your reply. Is the clade in the green box not a SNAP-OG because Ab|AB06G01309 and Ak|AK09G04755 are clustered together,not Ab|AB06G01323. But bootstrap value of Ab|AB06G01309 and Ak|AK09G04755 is only 0.734?  Can the clade in the green box become a SNAP-OG by adjusting orthosnap parameters? I did not change the results of orthoSNAP by adjusting the -s and -o.
best regards
Xiaoxu

You can get two SNAP-OGs from that tree.
I would recommend changing the -s parameter from the default, which is 80, to something like 0.7.


Custom downtime & uptime messages.

Updated version of #180

I have read the CLA Document and I hereby sign the CLA


Discuss with the sensor and devices team to see what kind of sensor readings would be generated from the sensors and what format would that be in

The sensor readings provided will be 8 types of different emissions from the devices and then the devices will send data to the database in .json format, but the database will export data in .csv files.


Release controller 0.10.2 and helm chart 0.12.2

Added a note to docs/developer-guide/releasing.md on the pre-1.0 version policy that uses "y" for breaking changes and "z" for features+fixes in "0.y.z" versions.


Portal location starts at top left of IDE then flashes to right location

Fixed with heavy popups in SourceMarker


GPU RTX 3090 - RuntimeError: CUDA error: no kernel image is available for execution on the device

My environment is:
GPU :  RTX 3090
cudatoolkit :  11.0.221
pytorch :  1.8.0.dev20201005
torchvision :  0.8.0.dev20201007
any one has solution?
C:\Users\pc006\.conda\envs\face2\lib\site-packages\torch\nn\functional.py:3105: UserWarning: The default behavior for interpolate/upsample with float scale_factor changed in 1.6.0 to align with other frameworks/libraries, and now uses scale_factor directly, instead of relying on the computed output size. If you wish to restore the old behavior, please set recompute_scale_factor=True. See the documentation of nn.Upsample for details.   warnings.warn("The default behavior for interpolate/upsample with float scale_factor changed "   0%|                                                                                | 0/4 [00:00<?, ?it/s] Exception in thread Thread-2: Traceback (most recent call last):   File "C:\Users\pc006\.conda\envs\face2\lib\threading.py", line 916, in _bootstrap_inner     self.run()   File "C:\Users\pc006\.conda\envs\face2\lib\threading.py", line 864, in run     self._target(*self._args, **self._kwargs)   File "webcam.py", line 100, in make_animation     use_relative_jacobian=relative, adapt_movement_scale=adapt_movement_scale)   File "C:\Users\pc006\Desktop\faceswap\animate.py", line 33, in normalize_kp     jacobian_diff = torch.matmul(kp_driving['jacobian'], torch.inverse(kp_driving_initial['jacobian'])) RuntimeError: CUDA error: no kernel image is available for execution on the device

finally It work!
I comment all about jacobian part
I don't know why lol?

yo how did you fix it im on a 3080

yo how did you fix it im on a 3080

hi, I founded that the pytorch inverse function is weird with 3090, so I translated the matrix to cpu mode before inverse the matrix.
ref:
torch.inverse(matrix.cpu()).cuda()

Could you go into more detail on

yo how did you fix it im on a 3080

hi, I founded that the pytorch inverse function is weird with 3090, so I translated the matrix to cpu mode before inverse the matrix.
ref:
torch.inverse(matrix.cpu()).cuda()

Could you go into more detail on how you went about translating the matrix to cpu mode before inversing the matrix?
Thanks in advance

Could you go into more detail on

yo how did you fix it im on a 3080

hi, I founded that the pytorch inverse function is weird with 3090, so I translated the matrix to cpu mode before inverse the matrix.
ref:
torch.inverse(matrix.cpu()).cuda()

Could you go into more detail on how you went about translating the matrix to cpu mode before inversing the matrix?
Thanks in advance

There are two .py with torch.inverse()
dense_motion.py
#jacobian = torch.matmul(kp_source['jacobian'], torch.inverse(kp_driving['jacobian']))
test_inverse = torch.inverse(kp_driving['jacobian'].cpu())
jacobian = torch.matmul(kp_source['jacobian'], test_inverse.cuda())
animate.py
#jacobian_diff = torch.matmul(kp_driving['jacobian'], torch.inverse(kp_driving_initial['jacobian']))
test_inverse = torch.inverse(kp_driving_initial['jacobian'].cpu())
jacobian_diff = torch.matmul(kp_driving['jacobian'], test_inverse.cuda())
@r06521601 Thank you for saving my day!

Could you go into more detail on

yo how did you fix it im on a 3080

hi, I founded that the pytorch inverse function is weird with 3090, so I translated the matrix to cpu mode before inverse the matrix.
ref:
torch.inverse(matrix.cpu()).cuda()

Could you go into more detail on how you went about translating the matrix to cpu mode before inversing the matrix?
Thanks in advance

There are two .py with torch.inverse()
dense_motion.py
#jacobian = torch.matmul(kp_source['jacobian'], torch.inverse(kp_driving['jacobian']))
test_inverse = torch.inverse(kp_driving['jacobian'].cpu())
jacobian = torch.matmul(kp_source['jacobian'], test_inverse.cuda())
animate.py
#jacobian_diff = torch.matmul(kp_driving['jacobian'], torch.inverse(kp_driving_initial['jacobian']))
test_inverse = torch.inverse(kp_driving_initial['jacobian'].cpu())
jacobian_diff = torch.matmul(kp_driving['jacobian'], test_inverse.cuda())
@r06521601 Thank you for saving my day!

Could you go into more detail on

yo how did you fix it im on a 3080

hi, I founded that the pytorch inverse function is weird with 3090, so I translated the matrix to cpu mode before inverse the matrix.
ref:
torch.inverse(matrix.cpu()).cuda()

Could you go into more detail on how you went about translating the matrix to cpu mode before inversing the matrix?
Thanks in advance

There are two .py with torch.inverse()
dense_motion.py
#jacobian = torch.matmul(kp_source['jacobian'], torch.inverse(kp_driving['jacobian']))
test_inverse = torch.inverse(kp_driving['jacobian'].cpu())
jacobian = torch.matmul(kp_source['jacobian'], test_inverse.cuda())
animate.py
#jacobian_diff = torch.matmul(kp_driving['jacobian'], torch.inverse(kp_driving_initial['jacobian']))
test_inverse = torch.inverse(kp_driving_initial['jacobian'].cpu())
jacobian_diff = torch.matmul(kp_driving['jacobian'], test_inverse.cuda())
@r06521601 Thank you for saving my day!

I just find that model.py also have torch.inverse() .
model.py
#normed_driving = torch.inverse(kp_driving['jacobian'])
normed_driving = torch.inverse(kp_driving['jacobian'].cpu()).cuda()

That exactly happens to me, and I fixed the problem by updating python 3.8.13 from 3.7.x, and installed latest PyTorch 1.12.1.
It was not needed to comment out any code.

@superwhyun unbelievable. the required torch version is 1.0.0, while you use the latest one. there should be huge gap in the apis.

That exactly happens to me, and I fixed the problem by updating python 3.8.13 from 3.7.x, and installed latest PyTorch 1.12.1. It was not needed to comment out any code.

cool!

I got it to work using:
Python 3.10.14, pip install torch==2.0.1+cu118 torchvision==0.15.2--index-url https://download.pytorch.org/whl/cu118
Just make sure you use a torch, torchvision that is compatible with your Cuda version


New design for No Access

⚠ Warning
This PR is branched off from #2870, so make sure that this branch is merged before reviewing.
📒 Description
Introduce new design for Overlay Controller (Missing Workspace)
🕶 Types of changes
New feature (non-breaking change which adds functionality)
🤯 List of changes

[x] Refactor the logic of OverlayViewController
[x] Wrap all TOS and Workspace views to new separated container, so it doesn't mess each other (We don't have TOS design yet)
[x] Implement new design.

👫 Relationships
Closes #2849
🔎 Review hints
Put following code to [MainWindowController windowDidLoad] to present the overlay for testing.
	dispatch_after(dispatch_time(DISPATCH_TIME_NOW, (int64_t)(3.0 * NSEC_PER_SEC)), dispatch_get_main_queue(), ^{
		[[NSNotificationCenter defaultCenter] postNotificationOnMainThread:kDisplayOverlay
																	object:[NSNumber numberWithLong:0]];
	});

Same logic and design -> 💯

Rebased successfully 🥇

Rebased successfully 🥇 It's ready to review after #2870

It's ready for reviewing @IndrekV 👍


btn-block style is not applied on the component after latest update

After updating to version 1.2.0 'btn-block' style class is not applied after using it in settings.
My setting config is:
multiSelectSettings: IMultiSelectSettings = {
checkedStyle: 'fontawesome',
buttonClasses: 'btn btn-default btn-block',
showCheckAll: true,
showUncheckAll: true,
dynamicTitleMaxItems: 3,
displayAllSelectedText: true
};
Have a look at the below images:
Version 1.1.0

Version 1.2.0

This solved a similar issue for me
multiSelectSettings: IMultiSelectSettings = {
checkedStyle: 'fontawesome',
buttonClasses: 'btn btn-default btn-block',
showCheckAll: true,
showUncheckAll: true,
dynamicTitleMaxItems: 3,
displayAllSelectedText: true,
containerClasses: 'full-width'
};
where
.full-width{     width: 100%; }

This is due to a change to display the control as inline-block by default as too many users were reporting that the control didn't close when clicking to the sides (as this was filled by the container).
Simply overriding containerClasses in config with anything should be sufficient to prevent the default dropdown-inline class from being applied.

Thanks @chris--jones @SherjeelSikander


Image path points to wrong directory

I'm having trouble including an image in my custom 404 page. If I add an image as docs/docsite/rst/images/foo.png and a custom 404 file docs/docsite/rst/404.rst that looks like this:
.. _page_not_found:

Oops!
=====

The page you're looking for doesn't exist in this version of the documentation.

.. image:: images/foo.png
   :alt: Custom 404

Return to the :ref:`index page for this version <index_anchor_name>`.

then build the docsite with config settings in the conf.py that look like this:
notfound_default_language = "foo"
notfound_default_version = "1.0"
notfound_no_urls_prefix = False
# no value set for notfound_template
# no value set for notfound_context

The 404 page gets built, the stylesheets and all toctree links load correctly for pages at any level of the filetree (hooray!).
However, in the HTML output, Sphinx places the image in a folder called _static/images/ (and places some other images in a folder called _images), but the path in the HTML output for the 404 page points to a folder called images:
<img alt="Custom 404" src="/foo/1.0/images/foo.png" />

Instead of the image, the alternate text appears in the HTML of the 404 page. Any pointers for me?

Can you share your full conf.py file?

@stsewd thanks for the quick reply! The conf.py on this branch is at https://github.com/acozine/ansible/blob/custom-404/docs/docsite/rst/conf.py.

Also, can you try installing the extension from master? (I think there was a fix for this)

Hm. when I do pip install --force-reinstall --user git+git://github.com/rtfd/sphinx-notfound-page@master it re-installs Sphinx 2.x again, and if I try to build with that version, I get:
Sphinx parallel build error:
RecursionError: maximum recursion depth exceeded

With Sphinx 1.7.9 I see the behavior described above. The src tag for our logo gets the correct path /foo/1.0/_static/images but the src for the 404 image gets /foo/1.0/images/.

If you disable the extension, what's the src resulting in the output page? Does the image load properly?

Oh, interesting. If I disable the extension, the image is placed in _images and the src points to it there.  The other images under the docs/docsite/rst tree are there as well.
We also have images in the _static/images folder - those come from our sphinx theme (docs/docsite/_themes/sphinx_rtd_theme/static/images).

I'm not sure to follow you.
Can you paste what's the HTML tag generated for the image with the extension enabled and what's with it disabled?

I was able to reproduce the issue by myself. I created a PR with the example not working at #46 and I will continue the work for the fix there. Thanks for reporting it!


[EditableText] value should be selectable via double click on web in edit mode

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

run sample on web platform ( I did not try other platforms)
click "hello to enter edit"
try selection with double click

Expected results
text selection
Actual results
no text selection
Code sample
import 'package:flutter/material.dart';

const Color darkBlue = Color.fromARGB(255, 18, 32, 47);

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

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData.dark().copyWith(
        scaffoldBackgroundColor: darkBlue,
      ),
      debugShowCheckedModeBanner: false,
      home: Scaffold(
        body: Center(
          child: MyWidget(),
        ),
      ),
    );
  }
}

final controller = TextEditingController(text: 'hello world');
final focusNode = FocusNode();

class MyWidget extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return EditableText(
      controller: controller,
      focusNode: focusNode,
      backgroundCursorColor: Colors.yellow,
      cursorColor: Colors.red,
      style: const TextStyle()
    );
  }
}

Screenshots or Video
No response
Logs
No response
Flutter Doctor output
tried in dart pad master and stable channels, respectively 3.16.0 and 3.17.0

Thanks for filing the issue. Reproduced this issue on the latest Flutter channels (stable/beta/master) and previous stable release 3.13.9. Verified this on Web/MacOS/Android platforms. The text is also not selectable when I click and drag the cursor as usual behavior.

flutter doctor -v (stable, beta, master)
[✓] Flutter (Channel stable, 3.16.0, on macOS 14.1 23B74 darwin-x64, locale en-VN)
    • Flutter version 3.16.0 on channel stable at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision db7ef5bf9f (8 hours ago), 2023-11-15 11:25:44 -0800
    • Engine revision 74d16627b9
    • Dart version 3.2.0
    • DevTools version 2.28.2

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A507
    • CocoaPods version 1.13.0

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

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio Giraffe.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[!] Android Studio (version unknown)
    • Android Studio at /Applications/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build 17.0.8+0-17.0.8b1000.22-10799086)

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = /Applications/Android Studio.app/
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

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

[✓] Connected device (3 available)
    • RMX2001 (mobile) • EUYTFEUSQSRGDA6D • android-arm64  • Android 11 (API 30)
    • macOS (desktop)  • macos            • darwin-x64     • macOS 14.1 23B74 darwin-x64
    • Chrome (web)     • chrome           • web-javascript • Google Chrome 119.0.6045.123

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[!] Flutter (Channel master, 3.17.0-15.0.pre.28, on macOS 14.1 23B74 darwin-x64, locale en-VN)
    • Flutter version 3.17.0-15.0.pre.28 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    ! Warning: `flutter` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 2949638e82 (2 hours ago), 2023-11-26 19:35:25 -0500
    • Engine revision ebebb25aaa
    • Dart version 3.3.0 (build 3.3.0-152.0.dev)
    • DevTools version 2.30.0-dev.4
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A507
    • CocoaPods version 1.13.0

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

[!] Android Studio (version unknown)
    • Android Studio at /Applications/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build 17.0.8+0-17.0.8b1000.22-10799086)

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio Giraffe Patch 3.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = /Applications/Android Studio.app/
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

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

[✓] Connected device (3 available)
    • RMX2001 (mobile) • EUYTFEUSQSRGDA6D • android-arm64  • Android 11 (API 30)
    • macOS (desktop)  • macos            • darwin-x64     • macOS 14.1 23B74 darwin-x64
    • Chrome (web)     • chrome           • web-javascript • Google Chrome 119.0.6045.159
    ! Error: iPhone has recently restarted. Xcode will continue when iPhone is unlocked. (code -14)

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

[!] Flutter (Channel beta, 3.17.0-0.0.pre, on macOS 14.1 23B74 darwin-x64, locale en-VN)
    • Flutter version 3.17.0-0.0.pre on channel beta at /Users/huynq/Documents/GitHub/flutter_beta
    ! Warning: `flutter` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_beta. Consider adding /Users/huynq/Documents/GitHub/flutter_beta/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_beta. Consider adding /Users/huynq/Documents/GitHub/flutter_beta/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 0b591f2c82 (3 weeks ago), 2023-11-02 22:01:10 -0400
    • Engine revision dff66925dc
    • Dart version 3.3.0 (build 3.3.0-91.0.dev)
    • DevTools version 2.29.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.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A507
    • CocoaPods version 1.13.0

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

[!] Android Studio (version unknown)
    • Android Studio at /Applications/Android Studio Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build 17.0.8+0-17.0.8b1000.22-10799086)

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio Giraffe Patch 3.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] Android Studio (version 2022.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • android-studio-dir = /Applications/Android Studio.app/
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

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

[✓] Connected device (3 available)
    • RMX2001 (mobile) • EUYTFEUSQSRGDA6D • android-arm64  • Android 11 (API 30)
    • macOS (desktop)  • macos            • darwin-x64     • macOS 14.1 23B74 darwin-x64
    • Chrome (web)     • chrome           • web-javascript • Google Chrome 119.0.6045.159

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

cc @LongCatIsLooong @Renzo-Olivares

Hi @cedvdb, thanks for reporting this issue. Currently EditableText does not support the full-suite of text selection gestures on any platform, it only supports tap and long press gestures. Input/Text widgets that support the full-suite of text selection are TextField/CupertinoTextField/SelectableText. To give EditableText gesture support you need to use TextSelectionGestureDetectorBuilder which builds a TextSelectionGestureDetector that you can wrap your EditableText with. Let me know if that solves your issue.

@Renzo-Olivares I used Textfield instead.

Thanks for the fyi - noting this as another issue that https://github.com/flutter/flutter/issues/120613 would resolve

Double-Tap Text Selection Solution
After some exploration, I found a workaround to achieve text selection on double-tap in the EditableText widget. This solution involves using the GestureDetector to detect double-tap events and then programmatically selecting all text
and property of  selectionColor and showSelectionHandles
Here's the updated code:
`import 'package:flutter/material.dart';
const Color darkBlue = Color.fromARGB(255, 18, 32, 47);
void main() {
runApp(MyApp());
}
class MyApp extends StatelessWidget {
@override
Widget build(BuildContext context) {
return MaterialApp(
theme: ThemeData.light().copyWith(),
debugShowCheckedModeBanner: false,
home: Scaffold(
body: Center(
child: MyWidget(),
),
),
);
}
}
final controller = TextEditingController(text: 'hello world');
final focusNode = FocusNode();
class MyWidget extends StatelessWidget {
@override
Widget build(BuildContext context) {
return GestureDetector(
onDoubleTap: () {
// Select all text when double-tapped
final newText = controller.text;
controller.value = controller.value.copyWith(
selection: TextSelection(
baseOffset: 0,
extentOffset: newText.length,
),
);
focusNode.requestFocus();
},
child: EditableText(
selectionColor: Colors.black38,
controller: controller,
focusNode: focusNode,
backgroundCursorColor: Colors.yellow,
showSelectionHandles: true,
cursorColor: Colors.red,
style: const TextStyle(),
),
);
}
}
`


Set OTel schema url

Contributes to https://github.com/Azure/azure-sdk-for-net/issues/31650

API change check
API changes are not detected in this pull request.


improve the scroll experience by learning from stackedit.io/

Hi, there.
I found the scroll sync experience of stackedit is really great, Is it possible to implement the scroll sync algorithm? Thanks

It could be done by implementing all of stackedit in Atom.  stackedit is
open source so it is possible.  It would be a lot of work.

Sorry for my unclear expression, I mean implement the same scroll sync algorithm on top of existing atom editor and markdown preview components.
发自我的 iPhone

在 2016年6月22日，02:33，mark-hahn<EMAIL_ADDRESS>写道：
It could be done by implementing all of stackedit in Atom. stackedit is
open source so it is possible. It would be a lot of work.
―
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Sorry for my unclear expression.

No it was clear.  I meant that putting all of stackedit in atom would be
the right way to do it.  I doubt the algorithm alone could be cloned
because it is from a totally different environment.


Added try, attempt and except tags processing

Resolves #23
Other minor improvements:

fixed various clippy warnings
added with_escaped() config method
updated to latest quick_xml and env_logger crate (redundant to the latest @kailan update)
Re-exported Reader and Writer from quick_xml and updated existing examples
Made a fn out of the most of the callback closure to improve readability and reduce nested indentation
added test for nested try tags
XML tags are first queued now upon the receipt
Added esi_try_example
worked on .cargo/config.toml and Cargo.toml (plural) so cargo build and cargo test now work

This should be to your liking now. :) Feel free to squash commits when merging. Thanks!

Yay! Thanks so much for your collaboration @vagetman.


skip step 4c

Step 4c should be skipped if you want to be able to start ArcGIS Enterprise via https://myapplications.microsoft.com/. If you don't skip that step, you will land on the wrong page, when you start ArcGIS Enterprise via https://myapplications.microsoft.com.

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

ID: 6c288067-e317-42e5-cde1-9799d6b903d5
Version Independent ID: ac2fcafc-d77e-efac-7c89-9593b0c80d8b
Content: Tutorial: Azure AD SSO integration with ArcGIS Enterprise - Microsoft Entra
Content Source: articles/active-directory/saas-apps/arcgisenterprise-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@laurenskusse
Thanks for your feedback! We will investigate and update as appropriate.

@laurenskusse
Thank you for the additional information. You are correct that step 4c should be skipped if you want to start ArcGIS Enterprise via https://myapplications.microsoft.com/. Skipping this step will ensure that you land on the correct page when starting ArcGIS Enterprise via https://myapplications.microsoft.com/.


Add id3 tags to mp3 file

Fixes #11

@jcalazan is this something you would like to add?

Hey Mattias,
Sorry for the delay, this is great!  Thanks!

Nice, no problem! when will it be available at http://www.youtubeadl.com?

I actually already pushed, but I'm currently fixing my rabbitmq server, be back up soon hopefully.

:+1:

Back up! Your change is there, looks good!


WIP: Issue #270: add atomic implementations based on atomic updaters

See issue #270 for details.

Current coverage is 87.18% (diff: 100%)

Merging #272 into master will increase coverage by 0.10%

@@             master       #272   diff @@
==========================================
  Files           251        251
  Lines          7579       7580     +1
  Methods        6048       6048
  Messages          0          0
  Branches       1529       1529
==========================================
+ Hits           6600       6609     +9
+ Misses          979        971     -8
  Partials          0          0

Powered by Codecov. Last update 2ec4a11...9d1a7dc


Non string Audience claims and VerifyAudience with required set to false

JWT tokens that have an array of strings in the audience claim can sneak through the VerifyAudience call when required is set to false
VerifyAudience tries to cast the aud claim to a string, and does not mind if it fails. When m["aud"] is not a string, aud will be set to "", and VerifyAudience will think there was no Audience claim in the token, and since req is set to false, it will return true.
https://github.com/dgrijalva/jwt-go/blob/06ea1031745cb8b3dab3f6a236daf2b0aa468b7e/map_claims.go#L13-L18
https://github.com/dgrijalva/jwt-go/blob/06ea1031745cb8b3dab3f6a236daf2b0aa468b7e/claims.go#L93-L102
This most likely affects the other VerifyXXXX methods too.
I don't know enough just about the spec to know what to do in this case, I just know that Auth0 is returning these multiple audience tokens, and that's how I ran into this issue.
Here's a sample payload:
{
  "iss": "https://xxxxx.eu.auth0.com/",
  "sub": "github|xxxxx",
  "aud": [
    "xxxx",
    "https://xxxx.eu.auth0.com/userinfo"
  ],
  "iat":<PHONE_NUMBER>,
  "exp":<PHONE_NUMBER>,
  "azp": "xxxx",
  "scope": "openid profile email"
}

I believe this here is the relevant part in the spec: https://tools.ietf.org/html/rfc7519#section-4.1.3
4.1.3.  "aud" (Audience) Claim

   The "aud" (audience) claim identifies the recipients that the JWT is
   intended for.  Each principal intended to process the JWT MUST
   identify itself with a value in the audience claim.  If the principal
   processing the claim does not identify itself with a value in the
   "aud" claim when this claim is present, then the JWT MUST be
   rejected.  In the general case, the "aud" value is an array of case-
   sensitive strings, each containing a StringOrURI value.  In the
   special case when the JWT has one audience, the "aud" value MAY be a
   single case-sensitive string containing a StringOrURI value.  The
   interpretation of audience values is generally application specific.
   Use of this claim is OPTIONAL.

I can confirm this issue.

Running into the same scenario. Isn't the problem here that VerifyAudience doesn't correctly parse the "aud" claim when is it an array of strings (vs. a single string)? jwt-go just turns the array of audience tokens into the empty string ""
@oponder did you find a way to get around the issue? I'm also using Auth0 and I don't know how to solve it since it seems like Auth0 only returns multiple audience tokens, which jwt-go doesn't seem to support

I got around it by using this ugly thing:
claims := token.Claims.(jwt.MapClaims)
parsedAud := ""
switch aud := claims["aud"].(type) {
case []interface{}:
	var auds []string
	for _, v := range aud {
		auds = append(auds, v.(string))
	}
	parsedAud = fmt.Sprintf("%v", auds)
case interface{}:
	parsedAud = aud.(string)
}
if subtle.ConstantTimeCompare([]byte(parsedAud), []byte(expectedAud)) != 1 {
	return errors.Errorf("invalid audience (expected %v vs actual %v)", expectedAud, parsedAud)
}

Note that the slice case needs to be first in the type switch, otherwise it matches interface{}. Also, I'm using subtle.ConstantTimeCompare like in the library, but I'm not sure why it's necessary. I guess that's why the package is called subtle…

@dgrijalva would you merge a PR solving this issue in the above way?

This should be fixed in the upcoming v4. There's no way to fix it for StandardClaims without breaking the API. I'm on the fence about backporting the MapClaims version of the fix to v3 as quietly changes behavior.

Any news about this? Just came across the same issue.


Add stats page

In prep for a place to put the progress chart (#200), having a page displaying hunt statistics would be nice.
Things to do:

[x] Make the page and link it to the frontpage of a hunt (#318)
[x] Add basic stats (solves, open puzzles, unsolved, meta solves) (#318)

[x] Integrate basic stats with main hunt (puzzle) view

[x] Add time-based stats (solves/hr, mins/solve) (also #318)
[ ] Make page update periodically (e.g., every 10 seconds like the puzzles page does)

Adding the progress chart should be the ultimate goal.

Marking this as resolved and merging the updating into the update issue (#390).

Marking this as resolved and merging the updating into the update issue (#390).


euc-kr is a superset of iso-8859-1

Description
Fix encoding issues when importing CSV files in Excel 2007
When importing CSV files into Excel 2007, there were encoding issues. While converting from ISO-8859-1 to UTF-8 worked, encoding errors persisted.
After trying various approaches, it was discovered that EUC-KR is a subset of ISO-8859-1. By adding code to handle EUC-KR encoding, the issue was resolved.
This commit addresses the encoding problems by incorporating the necessary changes to handle EUC-KR encoding when importing CSV files into Excel 2007.
Visual changes
Functional changes

[x] Code style has been fixed by running the composer cs command.
[ ] Changes have been tested to not break existing functionality.
[ ] Documentation is up-to-date.

Thanks


Tp  isn't used in WaveTools

@adimako I can't see where Tp is used after input to RandomWaves. It  looks like there is some missing logic on what to  do if Tp or  fp is provided.

Looks like this has been fixed


Replace getType with HivenObjectConstructur Class

Prior Issue:
The getType Class was prior used for getting objects and wrapping them, but the functionality lacks good implementation and extension. This results in many issues while using it and very low performance and little benefit of using it in the current version.
Describe the solution you'd like
Rewrite of the getType Class and replacement with the HivenObjectConstructur Class, which will implement async methods for "constructing" Hiven objects(=> constructing here means fetching data using async and passing them to the classes instead of forcing that data to be generated in the __init__, resulting in rather over-complicated object construction. Classes should in this case only be generated with pre-set data and not necessarily need to implement own solutions to fetch the needed data). The HivenObjectConstructur would then also then be able to implement better exception handling, logging and caching, which can help speed up performance by reusing constructed objects and also data.
Describe alternatives you've considered
Possibly construction methods inside of the types classes as @classmethods => easier fetching and usability while needing more implementation in every class, but could be cleaner
Additional context
Current getType Class

Will reopen since it will be built in with #46

Replaced with an in-class @classmethod constructur as in #46


Remove top level domain settings for ext link

Summary
This PR removes top level domains for external link module.
Metadata

Question
Answer

Did you use a meaningful pull request title?
yes

Did you apply meaningful labels to the pull request?
yes

Did you perform a self review first?
yes

CHANGELOG reflects changes?
yes

Did you perform browser testing?
yes

Risk level
Low

Relevant links
https://thinkoomph.jira.com/browse/RIG-246

/approved


[Question] How to connect to 5GC

Hellow @baleeiro17 ~
i want to use my5G-RANTester (VM 2)to connect my open5gs core (VM1). I have configured my config.yaml as following. I run ./ app gnb first. Then i run ./ app ue ,but it shows some error message that is "Error inaddress already in use". How should I do to fix it?
this is my config.yaml    <P.S. <IP_ADDRESS> is my open5gs core's ip>

./app gnb

./app ue

Hi @Evanlin1224 ,thank you for your interest in project and sorry for delay answer.
The comand ./app ue will simulate a UE and an GNB at the same time. It is not necessary execute ./app gnb in separate. The purpose of this command is simulate a gnb without UE. In your case I think you have two process(gnb) running with the same ip and port.
Other importante information is about IMSI, this must have 13 digits in config.yaml, you can use<PHONE_NUMBER>001.

@baleeiro17  Thanks for your help. I tried modify my config.yaml as following. I used command ./app ue this time, but it shown "AMF is inactivate". Then I look open5gs log message, it shows "Cannot find S_NSSAI. Check amf.plmn_support.s_nssai". So I tried to change sst: "000001" in the config.yaml. And it shows strange message.

change sst: "000001"

Hi @Evanlin1224, thank you for report those errors. In tester side had some wrong choices. But I resolved those issues in branch open5gs. Can please you testing again? In my environment worked fine. Before testing UE registration, can please verify if in /tmp has file named gnb.sock and if yes, please, remove. This is cause of the error in the first image.
About configurations that you have to insert in web ui, default values(imsi<PHONE_NUMBER>00001) in config.yaml will work fine but you can change if you prefer.
Thank you for help the project, Lucas.

Hello @baleeiro17, I have more or less the same issue. I copy-paste what I get when I run "sudo ./app ue":


fix(block): address warning for pullFrame method

With this PR, I am trying to address the warning that is currently reported in the above-mentioned pullFrame method. Do you think this logic adjustment would make sense, @tanim-ics?

I guess this PR is also related to #24.

Superseded by #36.


Documentation?

From reading the documentation it is not clear to me if:

this package is intended to read CAN bus log files, and if yes, which formats are supported
or if it is intended to talk with a CAN interface directly, and if yes, how

The plan is and was that this package can be used to decode and encode data from and maybe into CAN bus log files. The second point you have mentioned is also possible but I do not plan to add it to this repository. Do you know more about implementing a Julia wrapper around C code?

Yes, I can implement Julia wrappers around C code. But if the first goal is to read CAN bus log files, which data format do you support or do you want to support? I am currently using .csv files from the kvaser memorator light.

To be honest, currently this package has functionality for analyzing arbitrary CAN bus data using signals and messages, derived from the implementation of PEAK sym files. However, as of now this package does not support any data format. I want to support the candump format because I found it to be very flexible and simple to implement. Are you bound to use Julia for your data analysis?


Added support for different Malibu icon sets

Hey @idan, this is awesome!
We've added support for marketing icons in Malibu, so here's a PR to add support for it to react-malibu.
To load different icon sets, I added a set prop to <MalibuSprites />:
<MalibuSprites set='marketing' />
Happy to discuss other options for implementation, too.

Added tests, too. 👍

@superhighfives published 3.2.0

Aw, you're the best, @idan! 🙌


Coffeescript Language Support For Steps

First off, thank you kindly for this preprocessor; it's streamlining a whole lot of our acceptance testing and seems simple enough to get across the board. There is a feature I would like to request, however; the ability to write steps in the Coffeescript language.
Let's take an example scenerio:
Scenario: Test
    Given I log in

With a step in Coffeescript:
Given "I log in", ->
    cy.url
        .should 'not.be', 'Failed'
        .and 'be', 'Success'

Currently, it finds the step but then once we attempt to run it in the Test Runner, the test fails as it cannot find the step.
Granted, my Coffee is a little shonky. But the idea of minimizing the amount of code and keeping it so simple a designer could implement it would be absolutely delightful.

Due to personal reasons, the previous maintainers of this package are stepping down and handing the reigns over to me, a long-time contributor to the project and a user of it myself. This is a responsibility I'm very excited about. Furthermore, I'd like to thank @lgandecki ++ for all the work that they've done so far.
Read more about the transfer of ownership here.
The repository has however moved and all outstanding issues are being closed. This is not a reflection of the perceived importance of your reported issue. However, if after upgrading to the new version, you still find there to be an issue, feel free to open up another ticket or comment below. Please make sure to read CONTRIBUTING.md before doing so.


[WIP] Remove -enable-experimental-nested-generic-types flag

Fantastic!

-1 ABI FIXMEs in the standard library is a good day

Almost works! Hitting a weird generic metadata cache bug now, interesting...

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - c71652b1492b93142c98710d0848399897f7f39d
Test requested by - @slavapestov

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - c71652b1492b93142c98710d0848399897f7f39d
Test requested by - @slavapestov

@swift-ci Please test

@swift-ci Please test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - c71652b1492b93142c98710d0848399897f7f39d
Test requested by - @slavapestov

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - b4f55f71feae319a4ec20df4a944c5de3fbff075
Test requested by - @slavapestov

@swift-ci Please test

@swift-ci Please test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - b4f55f71feae319a4ec20df4a944c5de3fbff075
Test requested by - @slavapestov

FYI, I dealt with the compiler_crashers failure separately. The objc_readClassPair failure is genuine, though.

@swift-ci Please test

@swift-ci Please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 7ba5617a9de9eafd51814bee50e246147277438d
Test requested by - @slavapestov

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 7ba5617a9de9eafd51814bee50e246147277438d
Test requested by - @slavapestov

@swift-ci Please test

@swift-ci Please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 914c0bcda563495414a599be5738ab0c08b582f2
Test requested by - @slavapestov

@swift-ci Please test

@swift-ci Please test

w00t!

Looks like this means we can also tackle FIXME #23 (ClosedRangeIndex ➡ClosedRange.Index). Then again, we're going to refactor the ranges when we get the new generics features so maybe not worth it rn.


patch release v0.16.2

Release Summary

Version change: v0.16.1 → v0.16.2
Merge: releases/v0.16.2 → master
Compare the changes between the versions

:warning: When merging this pull request, you need to "Squash and merge" and make sure the title starts with patch release v0.16.2.
After that, a commit patch release v0.16.2 will be added and shipjs trigger will be able to trigger the release based on the commit.
Fore more information, please refer to the mergeStrategy section of the guide.

This is going to be published by the following command:
yarn publish --no-git-tag-version --non-interactive --tag latest

This pull request is automatically generated by Ship.js

Codecov Report

Merging #88 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #88   +/-   ##
=======================================
  Coverage   88.74%   88.74%
=======================================
  Files          16       16
  Lines         755      755
  Branches      109      109
=======================================
  Hits          670      670
  Misses         41       41
  Partials       44       44

Continue to review full report at Codecov.

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


Don't show the "Sign and return your framework agreement" link if countersigned framework agreement exists

Fix if condition to display links properly
Even after judiciously refactoring the /frameworks/dashboard page, it
turned out that there was a stray if condition checking for a
template variable that was no longer being passed in.
This meant that, after uploading a signed framework agreement and
receiving a countersigned one back, that it would still be possible
to upload a new signed framework agreement -- anarchy, basically.
This bugfix removes that possibility.

:+1:


base-coverage-file does not seem to be working

Describe a bug

Despite providing a file that exists it does not appear to be used.
Expected behavior

I see a diff with the base report.
Details

Action version: rc-6
OS, where your action is running (windows, linux): linux

action.yml file

on: push
jobs:
  report:
    runs-on: ubuntu-latest
    steps:
      - uses<EMAIL_ADDRESS>        with:
          github-token: ${{ secrets.GITHUB_TOKEN }}
          threshold: 1
          skip-step: all
          annotations: failed-tests
          coverage-file: ./coverage/report.json
          base-coverage-file: ./coverage/base.json

Additional context

Running using node v14.18.2
Begin initialization stage...
Initialization stage ended
Begin head coverage collection...
Begin installing dependencies...
Installing dependencies skipped
Installing dependencies ended
Begin running tests...
Running tests skipped
Running tests ended
Begin collecting coverage...
Loading code coverage from file: ./coverage/report.json
Collecting coverage ended
Begin parsing coverage...
Parsing coverage ended
Head coverage collection ended
Begin switching to base branch...
Switching to base branch skipped
Switching to base branch ended
Begin base coverage collection...
Base coverage collection skipped
Base coverage collection ended
Begin generating report...
Generating report ended
Begin report publish...
Report publish ended
Begin failed tests' annotations publication...
Failed tests' annotations publication skipped
Failed tests' annotations publication ended
Begin coverage annotations publication...
Coverage annotations publication skipped
Coverage annotations publication ended

Looks like it's probably this logic
https://github.com/ArtiomTr/jest-coverage-report-action/blob/9b54202076053f448ba42e4b81f49da7cb5d3f90/src/run.ts#L84-L86
Even though I provide my own base report file it's skipping the collection if it is a) not in a pull request context and b) has not switched to the base branch.
I believe the correct logic should be to simply not skip if the user has explicitly provided a base-coverage-file input value.


HandBrakeCLI-git is not installed

Hello :)
I keep having this error despite having installed handbrake and handbrake-cli.
The machine is under debian 8 server.
Do you have any idea how to fix it ?
Thank's ! :)

It's my fault, I always seem to do this. If you look under the options towards the top of h265ize, you'll see a handbrakecli variable. It will have HandBrakeCLI-git in the variable, just change it to HandBrakeCLI. I'll fix this once I get home.

Thank you :)


Add a method for random url/form hash prefixes

Add a method to the SecurityService.cfc that generate unique prefixes for hashed URL and FORM keys ( to replace the static 'v' and 'f' values currently in use) - the method should accept the total chars for the prefixes and default to a-z lowecase chars only.
Add a call to Application.cfc to call this function and cache the results for X amount of time (day(s)) and store them in an application scoped variable
Modify the BaseBean getUidHash() functions to utilize the application scoped variable for each type (url and form) in place of the static 'v' and 'f' prefixes used now
A minimum cacheTime for these application scoped variables should be set and noted in comments (1 day?) and a recommendation for a scheduled task to update these at slow times to prevent adverse actions happening to actual users during a transition between hash values should be added to both the code and the README.

Purpose of this change is to possible introduce additional variability into the hashed keys to both prevent deep linking (as using the date does now) and to make predicting the patterns used in these obfuscated keys more difficult to attack programatically.

This also opens the door to users simply manually assigning their preferred prefixes for uidHash values directly in Application.cfc


Missing detail in bullet-points

Issue in ./help/c-integrating-target-with-mac/a4t/analytics-tracking-server.md
There is detail missing from each of the three bullet-points under point #1.

Rewrote topic to include the newer Adobe Experience Platform Debugger and Developer Tools.
https://docs.adobe.com/content/help/en/target/using/integrate/a4t/analytics-tracking-server.html
Note: duplicate image is being corrected and will be available shortly.
Thanks for the input!


CORS error when loading schema from GraphQL Code Generator

I kept getting CORS errors when GraphQL Code Generator tries to load the schema. In the end I solved the issue by adding Origin: 'http://localhost:3000' to the schema request headers so my codegen.yml file looks like this:
overwrite: true
schema:
  - 'http://localhost:4000/graphql':
      headers:
        Origin: 'http://localhost:3000'
documents: 'src/graphql/**/*.graphql'
generates:
  src/generated/graphql.tsx:
    plugins:
      - 'typescript'
      - 'typescript-operations'
      - 'typescript-urql'

Does anyone know a better solution?

In the end I just enabled all cors requests on development environment.


Fix "current" channel version check

Node no longer uses even/odd numbers for current/stable. The effective "channels" are now current/LTS with no modern equivalent of "unstable."
Also removed hardcoded path in in iss script.

Hi @rainyday
Your PR has been fixed by other changes, hence I will close this PR. :)


datetime-local and time input fields get the standard form field styling

Even though materialize doesn't present the fancy popup with datetime-local and time fields, they should still get the nice styling in forms.

I agree


API for checking if App is available

After seeing #9, it is apparent that the developer will need to ask if the app is available or not. Something like:
if AvailableApps.Twitter.installed {
    // Twitter is installed
}

Can it be?
if Applications.installed.Twitter{
// Twitter is installed
}


Add understanding mainnet transaction fees

Store this in our docs folder

This is very clear, concise and well written. The author deserves praise; not everyone can write like this. In my experience, technical writing is often sorted out by editors, and sometimes in translation. This is practically flawless.


Ensure Target Directories Exists During Upgrade's Image-Preload Phase

This PR updates the curl calls used during the upgrade's prepare job to ensure any missing target folders are created.
IMPORTANT: Please do not create a Pull Request without creating an issue first.
Problem:

There are reported cases where upgrade failed during the "prepare" job phase because the curl command wasn't able to download the image list files to the missing target directory.
Solution:

Update the curl call in the package/upgrade/lib.sh script to use the built-in --create-dirs option to automatically create the necessary local directory hierarchy.
From the man doc:
--create-dirs

When used in conjunction with the [-o, --output](https://curl.se/docs/manpage.html#-o) option,
curl creates the necessary local directory hierarchy as needed. This option creates the directories
mentioned with the [-o, --output](https://curl.se/docs/manpage.html#-o) option combined with
the path possibly set with [--output-dir](https://curl.se/docs/manpage.html#--output-dir). If
the combined output filename uses no directory, or if the directories it mentions already exist,
no directories are created.

Related Issue:
#6963
Test plan:
i. Using v1.3.1, build a local ISO to include changes in this PR
i. Install and set up Harvester using the local ISO
i. Perform an upgrade following the upgrade documentation
i. Ensure that the upgrade completed successfully
i. Ensure that the cattle-system/apply-hvst-upgrade-4hgz2-prepare-on-* pod completed successfully
Additional verification:

Launch the upgrade container with a bind mount setting using Docker. Confirm that the in-container curl commands could successfully create the missing folder on the host

e.g., call download_file from within the container even though /var/lib/rancher/agent/images/ doesn't exist on the host:
# call download_file within the container
2d5431e7ceee:/ # source /usr/local/bin/lib.sh
2d5431e7ceee:/ # download_file https://releases.rancher.com/harvester/v1.4.0-rc5/version.yaml /host/var/lib/rancher/agent/images/version.yaml

on the host, the /var/lib/rancher/agent/images/ would have been created and the downloaded file written, as long as the container uid has rw permissions to the target path:
$ sudo ls -al /var/lib/rancher/agent/images/
total 12
drwxr-x--- 2 root root 4096 Nov 12 14:32 .
drwxr-x--- 3 root root 4096 Nov 12 14:32 ..
-rw-r--r-- 1 root root  378 Nov 12 14:32 version.yaml

@mergify backport v1.3 v1.4


fix: should use domain-qualified finalizer name

Pull Request

[ ] Make sure you have followed Kube-OVN Code Style.

What type of this PR
Examples of user facing changes:

Bug fixes

Which issue(s) this PR fixes
Fixes #(issue-number)

We should migrate the finalizer field during initialization.

We should migrate the finalizer field during initialization.

ok，


browser-extensions: github-review-filter

Chrome extension to filter files in GitHub code review using glob

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md

I am:

[x] Suggesting edits to an existing topic or collection
[ ] Curating a new topic or collection

EDITING AN EXISTING TOPIC OR COLLECTION*
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288)
[x] Content (and my changes are in index.md)

Please explain why these changes are necessary:
CURATING A NEW TOPIC

[ ] I've formatted my changes as a new folder directory, named for the topic as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME])
[ ] My folder contains a *.png image (if applicable) and index.md
[ ] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Please explain why you think this topic should be curated:
CURATING A NEW COLLECTION

[ ] I've formatted my changes as a new folder directory, named for the collection as it appears in the URL on GitHub (e.g. https://github.com/collections/[NAME])
[ ] My folder contains a *.png image (if applicable) and index.md
[ ] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Please explain why you think this collection should be curated:
Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Thanks for the PR but we frown on authors submitting their own work. Sorry!


DumpsterDiver

This is a really great tool: https://github.com/securing/DumpsterDiver. Not sure which category would be best for it though. Offensive probably?

Added, thanks @SpenGietz :)


OAuth2 class seems to have some bugs with service account JWT flow.

(1) The DEFAULT_EXPIRY_MINUTES is actually used as seconds, so it should be renamed to _SECONDS and updated to 3600:
https://github.com/google/google-auth-library-php/blob/master/src/OAuth2.php#L38
(2) According to this doc:
https://developers.google.com/identity/protocols/OAuth2ServiceAccount
"prn" is no longer used and "scope" is required. So some of the logic here:
https://github.com/google/google-auth-library-php/blob/master/src/OAuth2.php#L366
needs to be updated.
Can you confirm this is the case. I can submit a pull request for this. Thanks.

Regarding the required scope, I guess application default credentials don't deal with scope in the same way, so the logic for dealing with scope as optional in the toJwt() method is correct.

This has been fixed and released in the v0.7 release.


Submit Transaction: Simulate and Save transaction buttons

Simulate transaction button action on the Submit Transaction page
Cleanup items:

Updated "Payments for Transaction" info link on Endpoints
Replaced all setTimeout functions with a delayedAction() helper
Created XDR_TYPE_TRANSACTION_ENVELOPE const for "TransactionEnvelope" XDR type
Sign Transaction page: "View in XDR" and "Wrap with Fee Bump" button actions

Preview is available here:http://laboratory-pr943.previews.kube001.services.stellar-ops.com/

Preview is available here:http://laboratory-pr943.previews.kube001.services.stellar-ops.com/

Preview is available here:http://laboratory-pr943.previews.kube001.services.stellar-ops.com/

Preview is available here:http://laboratory-pr943.previews.kube001.services.stellar-ops.com/


Script access issues to composer cache...

This is happening. I think the script needs to perform a chown on the .composer directory and files.
Package operations: 70 installs, 0 updates, 1 removal

Removing pkeogan/cloud9-install-laravel-5.5 (1.0.3)
Installing symfony/thanks (v1.0.3): Downloading (100%)    Failed to download symfony/thanks from dist: /home/ubuntu/.composer/cache/files/symfony/thanks does not exist and could not be created.
Now trying to download from source
Installing symfony/thanks (v1.0.3): Cloning 8c12bbe4ba from cache
Installing doctrine/inflector (v1.2.0): Downloading (100%)
Installing doctrine/lexer (v1.0.1): Downloading (100%)
Installing erusev/parsedown (1.6.4): Downloading (100%)
Installing vlucas/phpdotenv (v2.4.0): Downloading (100%)
Installing symfony/css-selector (v3.4.3): Downloading (100%)    Failed to download symfony/css-selector from dist: /home/ubuntu/.composer/cache/files/symfony/css-selector does not exist and could not be created.
Now trying to download from source
Installing symfony/css-selector (v3.4.3): Cloning e66394bc76 from cache
Installing tijsverkoyen/css-to-inline-styles (2.2.1): Downloading (100%)
Installing symfony/polyfill-mbstring (v1.6.0): Loading from cache
Installing symfony/var-dumper (v3.4.3): Downloading (100%)    Failed to download symfony/var-dumper from dist: /home/ubuntu/.composer/cache/files/symfony/var-dumper does not exist and could not be created.
Now trying to download from source
Installing symfony/var-dumper (v3.4.3): Cloning 545be7e78c from cache
Installing symfony/routing (v3.4.3): Downloading (100%)    Failed to download symfony/routing from dist: /home/ubuntu/.composer/cache/files/symfony/routing does not exist and could not be created.
Now trying to download from source
Installing symfony/routing (v3.4.3): Cloning e2b6d6fe7b from cache
Installing symfony/process (v3.4.3): Downloading (100%)

[ErrorException]
copy(/home/ubuntu/.composer/cache/files/symfony/process/409c7dcf445b2cdcae173a2977f56da7767a7858.zip): failed to open stream: Permission denied

x2


Kafka 4060 docs update

Updated the docs with changes in KAFKA-4060.

Why does this change code? Shouldn't it only touch docs/upgrade.html ?

@hjafarpour Could you use git pull to merge all previous commits? Also there are code changes which I think should not be included..

I'm rebasing the branch and making minor changes based on @ijuma comment.

Merged to trunk and 0.10.2.


Increase code coverage on JSON FieldReader and FieldWriter

Add unit test coverage of JSON FieldReader and FieldWriter to cover:

Happy path: All primitive types within their normal operating range (short, int, long, double, etc.)

Double: Double.MAX_VALUE, Double.MIN_VALUE
Integer:  Integer.MAX_VALUE, Integer.MIN_VALUE
Long:  Long.MAX_VALUE, Long.MIN_VALUE
Short:  Short.MAX_VALUE, Short.MIN_VALUE

Error path(s):

All primitive types with values outside their range and using deci
Test with numbers that have decimal precision going into whole numbers types
{ integerFieldPrecisionError: 13412.4345 }

Syntax error: Put a "string" value in an integer field
{ booleanFieldSyntaxError: abcd }

@igarashitm please review before assignment for development

@igarashitm please review before assignment for development

+1 looks like "must have" for the tests!


Investigate using reservedWordsMapping to control acronym casing

This SDK was recently updated to use openapi-generator v6.6.0.  That version adds support for reservedWordsMapping in the Go generator: https://github.com/OpenAPITools/openapi-generator/pull/15083
We may be able to use reservedWordsMapping to standardize how certain acronyms (Bgp/BGP, Vrf/VRF, etc.) are capitalized in generated code.
Try adding a reserved words mapping to this SDK to see if that feature will enable us to standardize acronym capitalization without introducing unintended side effects.

Id=ID seems to not be too aggressive for this project. It may be unnecessary based on upstream changes to follow Go initialisms recommendations. Nonetheless, the diff still included ById methods.

Closing; while the acronym casing might not match our ideal, it nonetheless works.  In our limited experimentation with openapi-generator options, none of them really provided the ideal output.  If the Metal team wants to put more effort into changing acronym casing, they should do that in equinix/equinix-sdk-go.


Flicker on removing and adding notifications from eww

The notification-frame window will flicker on every update from end. This is because end calls eww open even when the window is already open, which (oddly) causes the window to reload.

I've this in the eww issues before. Apparently this only happens on some WMs. With bspwm it works fine. Interestingly, there's the issue elkowar/eww#260 which was fixed. Apparently there has been a regression.
The question is how to fix this on our end (lol). Is there a command that gives you the state of the eww window? Just keeping track of it with a boolean flag seems too fragile since you can also close the window manually.

Okay, it seems that eww debug contains a field called open_windows that we can use.

The question is how to fix this on our end (lol). Is there a command that gives you the state of the eww window? Just keeping track of it with a boolean flag seems too fragile since you can also close the window manually.

In a shell script I was writing, I just did this hacky little thing
if [ "$(eww active-windows | grep 'indicator-frame')" == "" ]
then
    eww open indicator-frame
fi

Oh, I'm blind, I did not see that command on the help page. Even better.


adding typings for react-measure

@chrisdrackett react-measure provides types in release candidate.

@TrySound thats true, but the RC is a full rewrite, I would assume it will have a different version number and api.

@chrisdrackett Are you still looking to get this merged in?

@GAntoine yes, I just haven't had a chance to get back and update based on the feedback!


One import fix needed for Python 2.7 compatibility. 2.7 seems indifferent to the bytes v. str.

All should be ready for submission to PyPI, so installation would be as easy as "pip install humod" or "sudo !!".

Merged - many thanks!


grid.set is ok.. but, how to hide any widgets?

Hello guys!
Thank you for this amazing project. I love it. I have a question for you.
I am working with a grid, and it is ok. But i need to change my widgets because i am using a dynamic json to draw the widgets. I mean, i need to draw "x" number of bar widgets from a REST-API. If that json is sending me 3 nodes, then I need to draw three bar-widgets with the data. If that REST-API sends me twelve nodes, then I need to draw twelve.
I can use grid.set to draw that widgets with a "for" or something similar but... if my new request has a number of nodes  lower than before one then i need to hide the widgets  painted in the screen (or something similar) because screen.render() doesn't clear the buffer with the painted widgets i had before.
So, is there any method to clear the screen buffer or to hide widgets in a grid?
Thx

Check out how wopr does something similar:
https://github.com/yaronn/wopr/blob/master/lib/document-viewer.js#L52

Good trick!! working for me. Thx my lord. :+1:

cool :)


Increase font size

Please increase the font size in all plots. Right now, the labels and legends cannot be read in our paper.

Font size increased, it's possible to set font-size (and other properties) for all mpl-plots created within SpySMAC via the spysmac/plot/mpl_style-file.


Develop to Main release

Title.

:tada: This PR is included in version 0.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Async on_transition()

This is work by @wvhulle and me to get on_transition() working with async.

Converted back to a draft because we have lifetime issues using it in our codebase.


Change .path to .fsPath to support Windows

Fixes #91. Could you take a look? Thanks!

I have tested on Windows, but not on other platforms.

Thank you for your contribution
I updated nodejs workflow https://github.com/oshikiri/vscode-generic-annotator/commit/dbd95fe7d678f4d7343522448a6e0c19477d5ba0 to invoke CI for this branch. Could you rebase this branch onto the latest master?

Done!


Logging

Right now, there is not that many info for debugging if something goes wrong. Apart from logging in requests, the app should print config params that are being used when it starts.

I would say this was closed in #32 if not then please reopen this issue.


Some questions regarding VecNormalize

According to the docs, when creating custom environments, we should always normalize the observation space. For this, you have the VecNormalize wrapper, which creates a moving average and then normalizes the obs.
Let´s say I have 2 observations: height (m) and weight (kg) of a person. My observation space would be something like a Box with low = [0, 0] and high = [2.5, 300]. But since I'm using a VecNormalize, this isn't correct anymore, right?
So should I instead change it to low = [-10, -10] and high = [10, 10]? (10 being the default clipping value for VecNormalize)
Another question: when should we normalize the rewards as well? (in the mujoco example shown in the docs you chose to only normalize the observations - why?)
Finally, what's the purpose of the discount factor? Should it be the same as the discount factor of whatever algorithm we're using?

So should I instead change it to

the boundaries in the observation space does not really matter (for everything that is not images), we usually set them to [-inf, inf].

Another question: when should we normalize the rewards as well?
Finally, what's the purpose of the discount factor? Should it be the same as the discount factor of whatever algorithm we're using?

Good question, the answer is there: https://github.com/openai/baselines/issues/538 and https://github.com/openai/baselines/issues/629
additional resource:  https://github.com/hill-a/stable-baselines/issues/234

Should it be the same as the discount factor of whatever algorithm we're using?

yes

in the mujoco example shown in the docs you chose to only normalize the observations - why?

We should change that (we would appreciate a PR for that), it is an old example, no real reason for not normalizing the reward too.

Thank you!
One more question: is VecNormalize compatible with GoalEnv? How would I go about using VecNormalize with HER?

is VecNormalize compatible with GoalEnv?

For now, it is not.

How would I go about using VecNormalize with HER?

I would advise to use DDPG and its built-in normalization in that case, or to create a gym.Wrapper (instead of a VecEnvWrapper) that will do the job.

Ok, thanks!

I would advise to use DDPG and its built-in normalization in that case

Oh, so not all algorithms require us to normalize the observations? Are there other that have this built-in besides DDPG?

Oh, so not all algorithms require us to normalize the observations?

No, as mentioned in the doc, it is usually crucial for PPO and A2C, but not for SAC and TD3 for instance. DDPG is the only one, for legacy reason.

I wanted to follow up on this topic to ensure I am implementing VecNormalize properly.  I am primarily looking at how to continue to training while keeping moving average from the previous training.  I have attached some code below
    env_learn = SubprocVecEnv(env_list)
    env_learn = VecCheckNan(env_learn , raise_exception=True)
    env_learn = VecNormalize(env_learn , training=True, norm_obs=True, norm_reward=True)

    env_render = Simulator()     # custom environment used to rending
    env_render.render_environment = True

    model = PPO2(policy ='CustomPolicy', env = env_learn , verbose = 1,
                 vf_coef = VF_COEFF,
                 noptepochs = EPOCHS,
                 ent_coef = ENT_COEFF,
                 learning_rate = LEARNING_RATE,
                 tensorboard_log = tensorboard_log_location,
                 n_steps = NSTEPS,
                 nminibatches = MINIBATCHES)

    model.save(results_folder + run_name)

    # Training the model
    for i in range(number_training_steps):
        logname = run_name + '_' + str(i)
        model.learn(total_timesteps = int((total_timesteps/number_training_steps)),
                    reset_num_timesteps = False,
                    tb_log_name = logname)

        env_learn.close()

        path = results_folder + logname
        model.save(path)

        # testing the performance of the model
        for j in range(3):
            obs = env_render.reset()
            done = False
            rewards = 0.0
            while done != True:
                action, _states = model.predict(obs)
                obs, reward, done, info = env_render.step(action)
                rewards += reward
                env_render.render(mode='file')
            print(f'For Training step {i}, and test number {j} the reward was: {rewards}')

        if i < number_training_steps:
            env_learn = SubprocVecEnv(env_list)
            env_learn = VecCheckNan(env_learn, raise_exception=True)
            env_learn = VecNormalize(env_learn, training=True, norm_obs=True, norm_reward=True)
            model.load(load_path=path)
            model.set_env(env_learn)

Curious if anyone has done an implementation like this in the past and if I am doing it correctly.  I am concerned with the way I currently have it implemented, the moving averages won't be retained for the next set of training the way I have it setup

Does the layer_norm kwarg have anything to do with this? Or is that a different kind of normalization?

Or is that a different kind of normalization?

Layer normalization is quite different, see associated paper: https://arxiv.org/abs/1607.06450
It is there mostly because of the parameter noise exploration for DDPG (cf doc).

I wanted to follow up on this topic to ensure I am implementing VecNormalize properly.

@cevans3098 I can only recommend you to take a look at the rl zoo, you forgot to save and load the VecNormalize in your case.
Closing this issue as the original question was answered.


Using Chef with Private Gem Server

Ohai Chefs!  I apologize in advance as this is probably more of a question than an issue that I cannot find the answer to.  I am currently using Chef behind a firewall and run a private gem server.  I would like to be able to use community cookbooks from the Supermarket without modification but the issue I am having is that I cannot figure out how to configure the Chef Client to pull gems from my private gem server and not rubygems.org when community cookbooks install gems via the chef_gem resource.
Any help is greatly appreciated as I have searched high and low but could not find an answer.  Thanks!

@st-isidore-de-seville how is your private gem server designed can you please provide more input on this.

Thanks for filing a GitHub Issue.
At Chef, we use GitHub Issues to track bugs and feature requests. Questions like "How do I .... with Chef?" are better answered by our awesome community on the Chef mailing lists, or on StackOverflow. (Please pick only one place to ask your question, however, as it's considered impolite to post the same question to multiple forums.)
You can find information about joining and asking questions on our mailing lists here. On StackOverflow, please tag your question with the "chef" label so that it shows up in this list.
We will be closing your issue. Please get in touch with the community via the above mechanisms in order to find a solution to your problem.


run()

Ability to run an injected function with a specific container instead of the default one

Won't do, it doesn't make any sense now that the container is internal only.


Implement Peerconnection::localDescription

Done on top of #12.
This PR adds Peerconnection::localDescription(). It is implemented with a non-ideal trick of calling two javascript functions--rtcLocalDescriptionSdp and rtcLocalDescriptionType--which introduces some redundancy. Perhaps this can be improved by returning a string array or with use of multi-value return from javascript to c++, which I currently do not know how to implement...

All the above feedback applied!


Feature nrtest

This pull request addresses Issue #97. It configures Travis CI to run epanet-nrtestsuite.

Looks like my approach to load dependencies for my python test tooling is failing on Travis.

If anyone has any suggestions on how to fix this. I would appreciate you sharing them.

Abandoning this pull request. I will continue to work on configuring Travis.


Do not render Tooltip and Menu elements until needed

Part of #2243
Before this PR the DOM was flooded with html elements produces by each Menu and Tooltip components visible on the screen. Every table scroll caused bunch of these elements removed and created in a crazy speed.
Before:
https://user-images.githubusercontent.com/9607060/161263049-980de65e-f743-4dac-9094-31c12ea43fef.mov
After:
https://user-images.githubusercontent.com/9607060/161263081-fb70ee99-8425-41ec-bc99-dd2e53871b87.mov

Should we also late bind the 's too so that we don't call apiManager.lookupLink() so many times unless needed?

Good catch. Probably this might be related https://github.com/lensapp/lens/issues/4512. Let's do this in separate PR.


added createSecurityOrder

Coverage decreased (-0.2%) to 7.899% when pulling 23f26713f943ee8161082151c06457946e3f5068 on Tr0nYx:master into 8be10cad8fa4acac9ea3c76b951d1c7f58202c73 on Alpha-Trader:master.

Coverage increased (+14.06%) to 22.131% when pulling 953a1077a3d56b6cacd73f1d6bf0d99b67c91a20 on Tr0nYx:master into 8be10cad8fa4acac9ea3c76b951d1c7f58202c73 on Alpha-Trader:master.

Coverage increased (+14.07%) to 22.138% when pulling 7a4004eb6551b25e79588e3f224d3e736b06ccfa on Tr0nYx:master into 8be10cad8fa4acac9ea3c76b951d1c7f58202c73 on Alpha-Trader:master.

Coverage increased (+14.07%) to 22.138% when pulling 7a4004eb6551b25e79588e3f224d3e736b06ccfa on Tr0nYx:master into 8be10cad8fa4acac9ea3c76b951d1c7f58202c73 on Alpha-Trader:master.

Coverage increased (+16.5%) to 24.544% when pulling 3c27fbd0fb2575542deb654b8a3c8829703fca81 on Tr0nYx:master into 8be10cad8fa4acac9ea3c76b951d1c7f58202c73 on Alpha-Trader:master.

Coverage increased (+16.5%) to 24.544% when pulling 3c27fbd0fb2575542deb654b8a3c8829703fca81 on Tr0nYx:master into 8be10cad8fa4acac9ea3c76b951d1c7f58202c73 on Alpha-Trader:master.

Coverage increased (+16.5%) to 24.544% when pulling 3c27fbd0fb2575542deb654b8a3c8829703fca81 on Tr0nYx:master into 8be10cad8fa4acac9ea3c76b951d1c7f58202c73 on Alpha-Trader:master.


fix superflous space and make error lowercased

Hi there,
this PR removes a superflous space and makes the start of the sentence lowercased.

It seems this has been fixed in another fashion. Feel free to reopen if you see anything off :)


47 auto update release model info

Fixes #47 .
Description
This PR has the following updates:

for update-model-info workflow, change to trigger only on workflow_dispatch. This change is used to avoid raising 403 error when a PR from forked repo is merged. In the future, when a PR is merged, our admin can run this workflow, and then a changed bundle will be updated into the release.
auto add a pull request which contains the changed model_info.json.
remove mypy check, it is because each bundle in this repo is independent, do mypy check will consider them together, which may raise unnecessary errors.

Status
Ready
Please ensure all the checkboxes:

[x] Codeformat tests passed locally by running ./runtests.sh --codeformat.
[ ] In-line docstrings updated.
[ ] Update version and changelog in metadata.json.
[ ] Ensure versions of packages such as monai, pytorch and numpy are correct in metadata.json.
[ ] Descriptions should be consistent with the content, such as eval_metrics of the provided weights and TorchScript modules.
[ ] Files larger than 25MB are excluded and replaced by providing download links in large_file.yml.

I tested this PR with my forked repo:
https://github.com/yiheng-wang-nv/model-zoo/runs/7739260622?check_suite_focus=true#step:3:94

It could also be an automated daily job on the Dev branch to check for model updates? In case the admin forget to trigger the workflow.

Hi @wyli , thanks for the suggestion. Yes, we can also set a daily job, and check if the bundle (with version) is existing in release. The missing bundles will be uploaded, with a PR that contains the changed model_Info.json. Hi @Nic-Ma , what do you think?

If we changed many bundles in many PRs within one day, please ensure that no conflicts or errors in the model_info.json.
Otherwise, I feel fine about the idea.
Thanks.

Update: now the workflow is scheduled everyday at 10:00 am (UTC time), and can be manually triggered.


allow listing files and file downloads for any service

new commands added are file list and file download

Looks pretty good. Built locally and exercised commands

The last two commits 099b8d10ff4f282d9c404078cd63762b3a3fc151 and 9f650425f0172752204d61e5f55de4d7845d324a fix issue #18. The git commands on association no longer run with sh -c "..." and instead use the git commands directly. Also the username and password prompt has been fixed for the default Command Prompt only.
Fixes #18

Fixes #17

Fixes #19


data and constructors required by tests: Should we provide them in stubs?

See in https://github.com/exercism/xhaskell/pull/464#discussion_r95073688 how I argued:

The test is expecting the four constructors
If we don't provide it, the stub solution doesn't compile with the tests.
If we don't provide it, the student has to read the test file, find all the possible constructors, and make the data type exactly as we want it anyway.

So, I argued to provide it to the student.
We can see that this is distinct from the case where the test demands an abstract type (such as a Robot) and wishes to perform some operations on it. In this case:

the test doesn't care about the implementation of the type
the student can choose the representation
therefore we make a data Robot = Dummy in the stub file

In this issue I'll discuss a few exercises of the first type:

allergies: Allergen
meetup: Weekday and Schedule
space-age: Planet

So, should we give these three exercises the same treatment, and fully define the above-mentioned data types, with constructors?
(If doing so, we can remove the .meta/DONT-TEST-STUB file for each of these exercises)

I think that this is the way to go. We have to sacrifice a little flexibility in designing exercises, but we gain a more coherent user-experience and automatic testing of stubs. 👍


Add a flag to test configuration file

I think it would be useful to add a flag to test validity of the configuration JSON file.
Something similar to "-t" flag in nginx.

@kyessenov this seems useful. Right now, we use mocks to verify configurations in the test code:
https://github.com/lyft/envoy/blob/master/test/example_configs_test.cc
We do not want to depend on gmock in the prod code. Will need to figure out how to do a verification pass without using mocks which might involve some targeted changes throughout various portions of the code.

@rlazarus is taking this one.

Related: https://github.com/lyft/envoy/issues/538

FYI, we need something like this pretty soon for our internal tooling. @rlazarus where are you on this? Can @junr03 help out? Another short term option is to just build a separate binary similar to what @hennna did using the full mocks.

It would be nice to verify RDS and CDS configs as well since the base config can be fairly small and most of it is fetched dynamically.

@kyessenov +1 one we need that also. cc @junr03

Going to close this out as the initial impl is done. We can open new issues with new requests.


v-repeat and v-if on same element not working

I have a
 I need to repeat and conditionally show. I found that using a v-repeat and v-show on the same element does not work, so previously I have been using a <template> for repeating and then a v-if on the actual repeating element. However in Internet Explorer 10+ this is causing issues, because it does not allow a
 to live inside of a <template>.
I am still on version 0.11.8. Will upgrading solve the issues, i.e. allow me to combine v-repeat with v-show?
Or is there another way to work around this?
FYI, this current markup is not working, it does repeat but the conditionality is not working as expected:

    ....

Could you try using filterBy instead of v-if?

    ....

Excellent, I got this working now. Using this syntax throughout now. Many thanks for the fast reply!


Added New Animated Button - 2

Developed my 2nd new animated button using HTML and CSS

Thank you @MrCuber
Looking forward to more contributions from you! 🧑💻
Thank you so much for your contribution.🌟
Make sure to ⭐ and follow me.

I would love to connect with you all ❤

https://www.linkedin.com/in/om-gawande/


Updating data visualization spanish translation

Translated Cheatsheet Information
Cheatsheet Name:  data-visualization
Language:  Spanish
Your Name (as you want to be credited on the website):  David Díaz Rodríguez
Date created/updated (Month YYYY format): 06/2024
Check that you have done the following:

[X] Added the language abbreviation to the file name (ex. data-transformation_es.pdf).
[X] Included a PDF file of your translated cheatsheet to the translation folders.
[X] Included a Keynote or PowerPoint file for your translated cheatsheet to the translation folder.
[X] Not duplicated an existing translation. If an existing translation exists, I have edited it.

Thank you! Could you let me know if the updates are those that match the most recent updates to the English versions?

Yes @mine-cetinkaya-rundel as I commented in #346 I first update the pptx original version so it matches the keynotes and the did the translations

Thank you!


Issue solved

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Checklist:

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

Hey @kashish0604, you are not supposed to delete the existing file and create a new one, you have to solve the bugs within the mentioned file
Also, I have noticed that you have solved two issues In the same PR, you will have to make separate PR for both issues, with a detailed description of what you have done in the PR.
NOTE: Please look into the description when you make the PR, and fill in the required details.
If you like you can submit a new PR with the requested changes. :)


Whatsapp is not connecting

I made the integration with WAHA API but I`m getting able to connect WhatsApp, then I also tried to connect Whatsapp using Swagger it shows the following error:

I got an error right after scanning the QR code since today:
{
"statusCode": 422,
"message": "The session under FAILED status. Please try to restart it.",
"error": "Unprocessable Entity"
}

Hi! Please make sure you're using the latest version, 2023.08.10 - you can get it with /api/version request

https://github.com/devlikeapro/whatsapp-http-api/issues/164
https://github.com/devlikeapro/whatsapp-http-api/issues/162
Kindly provide dockef logs if the issue persists after updating to the latest version.


Bug: Popup window to link account doesn't bring up sign in page

Describe The Bug
The "Link Account" button in Homebridge UI fails to bring up the sign in page for Honeywell, instead giving an error code 400.
To Reproduce
Click on the Link Account Button in Homebridge UI
Expected behavior
The sign-in page for Honeywell Home should load
Relevant log output
This is what the browser spat out in the popup window when I clicked on the Link Account button:

code	400
message	"The redirect URL provided does not match the redirect URL registered for the app."

Config for homebridge-honeywell-home
N/A

Screenshots

Device & Model
TH6220WF2006
Node.js Version
v16.11.1
NPM Version
v8.0.0
Homebridge Version
v1.3.6
Homebridge Honeywell Home Plugin Version
v11.1.0
Homebridge Config UI X Plugin Version
v4.41.2
Operating System
Raspberry OS (Homebridge); Windows (the browser)

Looks like your relink url is the same as what you are currently try to relink with.
What do you have in the relink hurl in the Honeywell developer account?

The callback url in Honeywell developer is http://<IP_ADDRESS>:8581/

It should be port 8585

That seemed to have been the issue! The copy button was being a little wonky and I believe I pasted homebridge ui's url instead.


Incorrect release binary architecture

disk_usage_exporter_linux_amd64: Mach-O 64-bit arm64 executable, flags:<|DYLDLINK|PIE>

Up. I confirm the same.

Fixed now, sorry for the mistake.


Trash ICON!!

nice one.
this could make it.
At least we need an obvious icon for people like @cookiengineer
J/K ;)


register missing reflect types

Objective

While generating https://github.com/jakobhellermann/bevy_reflect_ts_type_export/blob/main/generated/types.ts, I noticed that some types that implement Reflect did not register themselves
Viewport isn't reflect but can be (there's a TODO)

Solution

register all reflected types
derive Reflect for Viewport

Changelog

more types are not registered in the type registry
remove Serialize, Deserialize impls from Viewport

I also decided to remove the Serialize, Deserialize from the Viewport, since they were (AFAIK) only used for reflection, which now is done without serde. So this is technically a breaking change for people who relied on that impl directly.
Personally I don't think that every bevy type should implement Serialize, Deserialize, as that would lead to a ton of code generation that mostly isn't necessary because we can do the same with Reflect, but if this is deemed controversial I can remove it from this PR.
Migration Guide

KeyCode now implements Reflect not as reflect_value, but with proper struct reflection. The Serialize and Deserialize impls were removed, now that they are no longer required for scene serialization.

bors r+