Create Release action

Create an action which will generate a release for us so no more building anything locally and is now just the click of a button😄
Also:

Switch the current action over to use linux so that it works quicker
Get the current action building for both linux and windows.
Have the current action only build debug versions as they are all we really need from it

that's my fault XD

that's my fault XD

You gave me a really good basis to make the Release action so we can push releases to here https://github.com/stayintarkov/SIT.Manager.Avalonia/releases whenever we want without having to build anything ourselves locally etc.
Then the other changes I made to yours were just tweaks that I found when making the release action like realizing it'd be quicker running on Linux (saves minutes as we only have so many).


Missing  exception handling while loading user settings

This method raises error but there is no handling when calling.
Here is the link to the code.
def import_user_settings(args):
    ...
    if not Path(search_path, 'settings.py').exists():
    raise FileNotFoundError('`settings.py` is not found in {}'.format(search_path))
    ...

stg = import_user_settings(args)

Could you tell me if you have any troubles with this issue or if you have an idea of potential bugs that may arise in the future?

Sure.  It is actually duplicated stuff of #210.
If you are not planning to error handle, there is no need to use raise since the error message is not human friendly.
Example
Traceback (most recent call last):
  File "/Users/keisukeyamashita/.local/share/virtualenvs/HDNNP-D8nCws6U/bin/hdnnpy", line 11, in <module>
    load_entry_point('hdnnpy', 'console_scripts', 'hdnnpy')()
  File "/Users/keisukeyamashita/.local/share/virtualenvs/HDNNP-D8nCws6U/lib/python3.6/site-packages/pkg_resources/__init__.py", line 487, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/Users/keisukeyamashita/.local/share/virtualenvs/HDNNP-D8nCws6U/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2728, in load_entry_point
    return ep.load()
  File "/Users/keisukeyamashita/.local/share/virtualenvs/HDNNP-D8nCws6U/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2346, in load
    return self.resolve()
  File "/Users/keisukeyamashita/.local/share/virtualenvs/HDNNP-D8nCws6U/lib/python3.6/site-packages/pkg_resources/__init__.py", line 2352, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "/Users/keisukeyamashita/Dropbox/go/src/github.com/ogura-edu/HDNNP/hdnnpy/hdnnp.py", line 3, in <module>
    from . import settings as stg
  File "/Users/keisukeyamashita/Dropbox/go/src/github.com/ogura-edu/HDNNP/hdnnpy/settings.py", line 69, in <module>
    stg = import_user_settings(args)
  File "/Users/keisukeyamashita/Dropbox/go/src/github.com/ogura-edu/HDNNP/hdnnpy/settings.py", line 51, in import_user_settings
    raise FileNotFoundError('`settings.py` is not found in {}'.format(search_path))
FileNotFoundError: `settings.py` is not found in /Users/keisukeyamashita/Dropbox/go/src/github.com/ogura-edu/HDNNP

raise is not a good way to give user feedback.

Duplicated with .xyz parsing.
parsing data/GaN.xyz ... Traceback (most recent call last):
  File "/Users/keisukeyamashita/.local/share/virtualenvs/HDNNP-D8nCws6U/bin/hdnnpy", line 11, in <module>
    load_entry_point('hdnnpy', 'console_scripts', 'hdnnpy')()
  File "/Users/keisukeyamashita/Dropbox/go/src/github.com/ogura-edu/HDNNP/hdnnpy/hdnnp.py", line 37, in main
    generator = DataGenerator(stg.dataset.xyz_file, 'xyz')
  File "/Users/keisukeyamashita/Dropbox/go/src/github.com/ogura-edu/HDNNP/hdnnpy/data.py", line 397, in __init__
    self._construct_training_datasets(data_file, format)
  File "/Users/keisukeyamashita/Dropbox/go/src/github.com/ogura-edu/HDNNP/hdnnpy/data.py", line 434, in _construct_training_datasets
    parse_xyzfile(original_xyz)
  File "/Users/keisukeyamashita/Dropbox/go/src/github.com/ogura-edu/HDNNP/hdnnpy/data.py", line 479, in parse_xyzfile
    for atoms in ase.io.iread(str(xyz_file), index=':', format='xyz', parallel=False):
  File "/Users/keisukeyamashita/.local/share/virtualenvs/HDNNP-D8nCws6U/lib/python3.6/site-packages/ase/io/formats.py", line 503, in iread
    **kwargs):
  File "/Users/keisukeyamashita/.local/share/virtualenvs/HDNNP-D8nCws6U/lib/python3.6/site-packages/ase/io/formats.py", line 527, in _iread
    fd = open_with_compression(filename, mode)
  File "/Users/keisukeyamashita/.local/share/virtualenvs/HDNNP-D8nCws6U/lib/python3.6/site-packages/ase/io/formats.py", line 313, in open_with_compression
    return open(filename, mode)
FileNotFoundError: [Errno 2] No such file or directory: 'data/GaN.xyz'

See this one for great infos.
https://stackoverflow.com/questions/16138232/is-it-a-good-practice-to-use-try-except-else-in-python


Getting no credentials provided

Question

Trying to setup github action for my private repo and here is the step for setting up gcloud
- name: Setup Gcloud
      uses: GoogleCloudPlatform/github-actions/setup-gcloud@master
      with:
        version: '275.0.0'
        service_account_key: ${{ secrets.GCLOUD_KEY }}
        export_default_credentials: true

I always get the following output

Seems like it never recognizes service_account_key is provided. Any insights or help is appreciated.

Hi @jainkunal
Can you confirm that a secret called GCLOUD_KEY has been set? I believe if GCLOUD_KEY secret does not exist, this is the expected behavior and service_account_key input will not show in the run logs.

Thanks @bharathkkb for the quick response, I had a small typo in the secret key name and didn't realize it. Wish github provided better error message.


Game State Refactor

Fixes #505
Hoo boy.
Removes the usage of bsdiff for game states, instead using a proper dirty flag system. Performance is much better now although bandwidth usage is slightly higher.
Helps a lot with #558 but it's not fixed yet. I suspect it's just Lidgren trying to ack messages now.
TODO:

Deletions of entities and components Done.
Optimize bandwidth usage more? Not doing this.
Optimize player states. They're not using a dirty system yet. Not doing this.

@Acruid this is mostly done. I haven't removed player state syncing though because the client needs it to connect for some reason. If that's fine go ahead and merge it.


Mask not working as expected

I have the following json
{
  "tom": 3,
  "data1": {
    "first": "Hello",
    "second": {"third": "salude", "fourth": "nvm"}
   },
  "data2": {
    "first": "Bye",
    "second": {"third": "Cheers", "fourth": "nvm"}
   }
}

And I'm applying the follwing mask on it.
"data1(first,second/third),data2(first,second/third)"
I expected to see the,
{
  "data1": {
    "first": "Hello",
    "second": {"third": "salude"}
   },
  "data2": {
    "first": "Bye",
    "second": {"third": "Cheers"}
   }
}

but I'm only getting
{
  "data1": {
    "first": "Hello",
    "second": {"third": "salude"}
   }
}

Can you help me out?

@MathiasHaudgaard thanks for letting me know! That appears to be a bug. I can reproduce it.

Ok, please let me know when it's fixed. Thanks! :smiley:

@nemtsov It seems like the error comes from the buildTree functions. It exits the scope in the wrong order. When it comes to ")" it pops "/" and now it cant exit the last scope because of
if (peek && peek.tag === "/") doesn't match with "("

I'm bumping into the same error right now. Are there any updates here?

any updates?

Is the library abandonware?

Hey folks, although I don’t have a ton of time to work on this library myself, I’m more than happy to review PRs and help guide people interested in making contributions to it.

This the only bug i could find so far. Can you fix it?

What i found is, if / is not at the end of the query, what is next is going to be ignored.
Not working:
data1(first,second/third),data2(first,second/third)

Working:
data1(first,second),data2(first,second/third)

This is the only library that supports partial responses in plan JS. It is a shame just to abandon it.

This is resolved in v1.0.4 thanks for reporting it @MathiasHaudgaard


minimum required RBAC permissions

Proposal: [Maybe it should be in the documentation]
Could somebody share all permissions to use with ServiceAccount role ?
Or maybe does  this application need some additional checks ?
I've tried with (sorry for mess)
- apiGroups: ["apiextensions.k8s.io"]
  resources: ["customresourcedefinitions"]
  verbs: ["list", "get", "update", "delete"]
- apiGroups: ["networking.k8s.io"]
  resources: ["networkpolicies"]
  verbs: ["list"]
- apiGroups: ["*"]
  resources: ["configmaps"]
  verbs: ["list"]
- apiGroups: ["apps"]
  resources: ["statefulsets"]
  verbs: ["list"]
- apiGroups: ["autoscaling"]
  resources: ["horizontalpodautoscalers"]
  verbs: ["list"]
- apiGroups: ["policy"]
  resources: ["poddisruptionbudgets"]
  verbs: ["list"]
- apiGroups: [""]
  resources: ["namespaces"]
  verbs: ["list", "get"]
- apiGroups: [""]
  resources: ["services"]
  verbs: ["list", "delete", "update"]
- apiGroups: [""]
  resources: ["resourcequotas"]
  verbs: ["list", "update"]
- apiGroups: [""]
  resources: ["secrets"]
  verbs: ["list"]
- apiGroups: ["extensions"]
  resources: ["daemonsets"]
  verbs: ["list"]
- apiGroups: ["extensions"]
  resources: ["ingresses"]
  verbs: ["list","delete"]
- apiGroups: ["batch"]
  resources: ["jobs","cronjobs"]
  verbs: ["list"]
- apiGroups: [""]
  resources: ["pods"]
  verbs: ["create","delete","get","list","patch","update","watch"]
- apiGroups: ["extensions"]
  resources: ["deployments", "deployments/rollback"]
  verbs: ["create","delete","get","list","patch","update","watch"]
- apiGroups: [""]
  resources: ["pods/exec"]
  verbs: ["create","delete","get","list","patch","update","watch"]
- apiGroups: [""]
  resources: ["pods/log"]
  verbs: ["get","list","watch"]
- apiGroups: [""]
  resources: ["secrets"]
  verbs: ["get"]

but it seems that it doesn't work
i received error
[ERROR] Deployment/my-frontend rollout timed out after 438.0s
when I try with admin permission  all are fine.

Hi @dzirg44,
Adding a list of RBAC permissions to the README sounds like a good idea. It might be tricky since kubernetes-deploy will create anything you ask it to, but at least having the needed permissions for the resources we explicitly support would be good.
I suspect you're missing permissions on on replicasets in this case..


vscode-icons breaks VSCode 1.4.0 for some users

We are receiving numerous bug reports from users over the past days where either VS Code does not show anything or parts of the UI are broken. I am raising the issue here and ask if you are aware of these issues:

https://github.com/Microsoft/vscode/issues/11576
https://github.com/Microsoft/vscode/issues/11578
https://github.com/Microsoft/vscode/issues/11265
https://github.com/Microsoft/vscode/issues/10566

@bpasero I'm aware of these issues but it would be nice if these users could be redirected to this repo and report here so we could try to know how exactly they messed up with the files. They explain that they cannot see something but they don't say what they did before. Did they update the extension? Did they execute some of the vscode-icons commands? In what order? I mean, we need this issues to be reported here so we can get proper feedback.
Anyway, we'll try to reproduce this and see if this can be diminished or eradicated. Hopefully, with the next API integration these issues will not raise so often.
By the way, in the repository's readme we have a FAQ and a Disclaimer section explaining that there's always a possibility to make VSCode crash and that the solution is to reinstall it 99% of the times. They could use the generated .bak files but it's usually faster and safer the "reinstalling option".

Imho vscode-icons should detect that the replacement of JS files failed and restore the bak automatically (e.g. by doing a checksum of the resulting JS file and seeing if the operation suceeded or not).

I've added a guard in the injection process in order to avoid double injection which is somehow how all things were getting messed. It will be released in a minute.
Btw, I've seen that you have released the new 1.5.0 stable version. I was somehow under the impression that this was going to happen on Sept 12th (as per @aeschli email conversations) and has caught me by surprise. :disappointed:  I'm still working on it as this is a busy week for me and not have much time.
Related #11288


Adding or removing video elements will cause other platform views to flicker on Safari

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
1, open https://flutter-webrtc.github.io/flt-video-element-flicker-demo/ in Safari (iOS/macOS)
2, Click the + button and access the local camera to render the video stream on the video element
3, Clicking the + button multiple times will reproduce the error, and the displayed video stream will flicker. On the contrary, clicking the - button will also flickering.
Expected results
Correctly render videos on Safari without flickering, no matter the video element add or remove or changes video elements size
Actual results
UI flickering occurs in Safari ios/macOS
Code sample
A git repo that can be used to reproduce the bug
https://github.com/flutter-webrtc/flt-video-element-flicker-demo
And the deployed online test page
https://flutter-webrtc.github.io/flt-video-element-flicker-demo/
Screenshots or Video

Screenshots / Video demonstration
Chrome m118 on the left, Safari 17.1 on the right
https://github.com/flutter/flutter/assets/17733746/7660f5f2-774e-49c5-82f1-1a7af9da2ca1

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
flt-video-element-flicker-demo % flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.13.8, on macOS 14.1 23B74 darwin-arm64, locale en-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] VS Code (version 1.83.1)
[✓] Connected device (4 available)
[✓] Network resources

• No issues found!

Hi @cloudwebrtc
When running your demo, I notice this warning from browser console on both Chrome and Safari:
Flutter was unable to create enough overlay surfaces. This is usually caused by too many platform views being displayed at once. You may experience incorrect rendering.

Could you simplify the sample code by using a pure HtmlElementView instead of RTCVideoView widget to see if the issue still persists?

hey @huycozy, I have removed RTCVideoView and used HtmlElementView to wrap html.VideoElement for rendering,
https://github.com/flutter-webrtc/flt-video-element-flicker-demo/blob/main/lib/main.dart
but the issue seems to still exist.
This is the latest deployment
https://flutter-webrtc.github.io/flt-video-element-flicker-demo/

Also, I noticed that deleting flt-platform-view in the debug console causes flickering

But if you delete the video element node, it will not flickering.

Thank you for checking that. I think this is close to https://github.com/flutter/flutter/issues/71884 which points out a performance issue when adding too many platform views.
Per a comment here, maybe you can try embedding all video elements to a platform view to see if it can be improved or not.
Labeling the issue for other's input. Reproduced this issue on Flutter master channel as well.

flutter doctor -v (stable and master)
[✓] Flutter (Channel stable, 3.13.9, on macOS 14.0 23A344 darwin-x64, locale en-VN)
    • Flutter version 3.13.9 on channel stable at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d211f42860 (7 hours ago), 2023-10-25 13:42:25 -0700
    • Engine revision 0545f8705d
    • Dart version 3.1.5
    • DevTools version 2.25.0

[✓] 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 Giraffe.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A240d
    • 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 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
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 14.0 23A344 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 118.0.5993.117

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

• No issues found!

[!] Flutter (Channel master, 3.16.0-18.0.pre.55, on macOS 14.0 23A344 darwin-x64, locale en-VN)
    • Flutter version 3.16.0-18.0.pre.55 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 9e04246c48 (2 hours ago), 2023-10-27 06:39:03 +0530
    • Engine revision bedc49efc8
    • Dart version 3.3.0 (build 3.3.0-56.0.dev)
    • DevTools version 2.28.1
    • 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 Giraffe.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A240d
    • 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 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
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)

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

[✓] Connected device (3 available)
    • iPad (mobile)   • 00008103-000A1464346A201E • ios            • iOS 17.0.3 21A360
    • macOS (desktop) • macos                     • darwin-x64     • macOS 14.0 23A344 darwin-x64
    • Chrome (web)    • chrome                    • web-javascript • Google Chrome 118.0.5993.117

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

! Doctor found issues in 1 category.

Confirmed that this happens in Safari, but not in Chrome and Firefox. Does this happen without Flutter, using plain HTML, if you just add canvas tags connected to the camera normally?

hey, @yjbanov Sorry for forgetting to reply, I added a new test repo and used dart:html to render local video. It seems that adding/removing html.VideoElement under a div does not flicker on safari. What you said is on canvas. Does rendering mean using Context2D to update video images? (webrtc directly sets the videoStream to a videoElement)
repo https://github.com/flutter-webrtc/dart-video-element-flicker-demo
online page:
https://flutter-webrtc.github.io/dart-video-element-flicker-demo/

Thanks for confirming


Add support for  nuxt-content

Summary by CodeRabbit

New Features

Introduced a new composable function for fetching localized page content.
Added comprehensive legal information pages (contact, imprint, and privacy) in both German and English.
Implemented dynamic, localized navigation in the footer.

Bug Fixes

Improved SEO metadata across new legal pages.

Enhancements

Enhanced localization support with dynamic paths for legal pages in the footer.
Updated styling for markdown content for better readability.
Added client-side only rendering for the FAQ module on the homepage.

Chores

Included @nuxt/content module in the project dependencies.
Updated TailwindCSS configuration to include Markdown content for purging unused styles.

@coderabbitai resolve all

Currently build fails, most likely because of https://github.com/fi3ework/vite-plugin-checker/pull/327 and https://github.com/vuejs/language-tools/issues/4484
With the upgrade to vue-tsc 2.x and Typescript 5.5.x, pipeline should be fixed.

@coderabbitai full review. Annalise also why the build fails and provide suggestions/

Awaits the release of Nuxt 3.12.3 - https://github.com/nuxt/nuxt/pull/27648


[Register] Unknown register errors

Describe your question
register
Wrong answer for RegisterUserReq(password=8db5e609749c4957, qq=998970494167868265018424, wechat=null, name=aaaafde0c, sex=FEMALE, birthday=7月30日, sign=7ccf8514-a89a-45): expected < 0, got<PHONE_NUMBER>477864
Why is this invalid?
birthday format is correct
have name,password,sex
no same qq in the database

You are right. That's my problem


CAMEL-11373: Add camel-twilio Karaf feature and update doc

https://issues.apache.org/jira/browse/CAMEL-11373

@davsclaus Done. I hope it's fine now.

Thanks for the PR it has been merged. Do you mind closing this?

Thank you!


Documentation text width could be larger

Documentation for packages seem to takes only about half the screen width. This makes reading some struct definitions quite frustrating. See for example https://godoc.org/github.com/Shopify/sarama#Config
Would it be possible / a good idea to make the text area larger so we don't have to scroll horizontally?

Past a point, increasing line length decreases reading speed. I suggest fixing the package instead of making everything on godoc slower to read.

As a point of reference, the source code seems to be about equally cut off when viewing in GitHub as well:

While I think godoc.org could be a little wider by default (maybe 10-20% more than it is now), in this case this package sarema package is primarily to blame for low readability. It should stay around 100~ or fewer characters wide, but it exceeds 120-140+ in many cases.

I filed https://github.com/Shopify/sarama/issues/540
We should leave godoc.org as-is.


Depth config for X-Forwareded-For header.

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
I'm using Traefik and found that there's a nice config for IPWhitelist which is depth. So it would be nice if the depth config is applied globally without using IPWhitelist middleware since services staying behind Traefik will get the filtered X-forward-for header for better security.

Hello,
I'm sorry, but we do not understand what you mean (and we're afraid you might be confused about the meaning of Depth there).
Could you give us a concrete example, with how you think traefik behaves at the moment, and how you wish traefik would behave instead?


Compile error on Raspbian

Some error occurs while running the building script on Raspbian.

In file included from /home/huzi/caffe2/caffe2/contrib/gloo/allreduce_ops.cc:5:0:
/home/huzi/caffe2/third_party/gloo/gloo/allreduce_ring_chunked.h:38:68: note:   mismatched types ‘std::initializer_list<Tp>’ and ‘long unsigned int’s
chunkSize = std::max(minSize, (count_ + chunks_ - 1) / chunks_);
^
In file included from /usr/include/c++/4.9/algorithm:62:0,
from /home/huzi/caffe2/caffe2/contrib/gloo/allreduce_ops.h:3,
from /home/huzi/caffe2/caffe2/contrib/gloo/allreduce_ops.cc:1:
/usr/include/c++/4.9/bits/stl_algo.h:3454:5: note: template<class _Tp, class _Compare> _Tp std::max(std::initializer_list<_Tp>, _$ompare)
max(initializer_list<_Tp> __l, Compare comp)
^
/usr/include/c++/4.9/bits/stl_algo.h:3454:5: note:   template argument deduction/substitution failed:
In file included from /home/huzi/caffe2/caffe2/contrib/gloo/allreduce_ops.cc:5:0:
/home/huzi/caffe2/third_party/gloo/gloo/allreduce_ring_chunked.h:38:68: note:   mismatched types ‘std::initializer_list<Tp>’ and ‘long unsigned int’
chunkSize = std::max(minSize, (count + chunks - 1) / chunks);
^
caffe2/CMakeFiles/Caffe2_CPU.dir/build.make:65: recipe for target 'caffe2/CMakeFiles/Caffe2_CPU.dir/contrib/gloo/allreduce_ops.cc$o' failed

I suspect it to be a compiler relevant error but I've tried to upgrade to the latest version of compiling tools. Please check.

Wrong repo, sorry.


(Russian) Cannot create new Trezor wallet due to UI

The Russian translations push the buttons off the screen for the Trezor create new wallet operation. You cannot complete the wizard.
Screenshot:

Suggest we also check the regular create wallet in russian as it might have a similar problem

Looks the same as the German one recently (see #773). We need to add a proper wrapper as in other views.

@jim618 can you run through the new FEST tests. You'll see that the Trezor and KeepKey devices have had their UI's translated in error. They are supposed to be partitioned away to avoid being included.

I've adjusted the Trezor and KeepKey components to use HTML wrapping which should allow for correct display over a variety of languages:

Unfortunately, I cannot get the title to show in Russian without a small amount of clipping which is annoying.
Please run this up with different languages and explore various screens with lengthy titles to ensure that there is no regression.
Ready for review and close.

I have created in Russian a Trezor wallet and a soft wallet.
All screens looked fine apart from this one:

I could still navigate off the screen so it's not fatal.

Sorry - meant a KeepKey wallet

Just created a Trezor wallet in German and it all looked good.

This also applied to the Trezor screen (they use the same underlying panel view). I've increased the use of HTML wrapping in labels, especially in status labels so they should wrap much more predictably now across a range of languages (including RTL/LTR).
Here's an example of what to expect:

Ready for review and close.
PS I also hardened the handling of a failed device after switching languages and wallet types. Kind of a corner case but the UX was no response to attach/detach of device in the Unlock screen.

Worked through creating a new Trezor wallet in Russian.
All the screens look good.
Closing


When fewer Items are used it basicly uses that item to a insane amount

When fewer Items are used it basicly uses that item to a insane amount, It needs to be adjusted to an better amount amount the times it spawns should be mutch less as default. Allso why are you rewarded when not completing a full note?

Currently, every note longer than 200 milliseconds has a 25% chance of spawning an item. The spawn rate of items is based on the following table:

In the future, I might make the overall spawn rate, which is currently set at 25%, configurable in the options.

Perhaps I could convert this Excel sheet into a .csv file, enabling it to be read at runtime. This way, people without development skills could edit the probabilities.
By the way, I'm still working on the balancing. I haven't had a chance yet to test it with friends. 😄

Allso why are you rewarded when not completing a full note?

Made a new issue #8

@askusbloodfist
Thanks for your feedback :D
I added these options to manage spawn probabilities:


Styling issue

Hello
great work with this component!
I incorporated SuperSelectField into my project and noticed few quirks:

blue border, font not matching the rest of MUI components:

still selectable when disabled:

To make sure this has nothing to do with my setup i spun up an empty create-react-app project and created above recordings. Am I missing something obvious?

Hello
Add a "css reset" in your main .css file:
*:focus { outline: none; }

@Sharlaan thanks for a quick reply!
So the outline is gone, what about the other issues - any ideas? Why does it not inherit styles from main material package? Something to do with create-react-app?

This is what I'm looking at (disabled prop is set to true but i can still select the field, font is not Roboto, underline is different than on regular SelectField):

Hi, Please help me  have implemented multi select in my web APP,  facing some styling issue like , It works fine but when I am start typing in filter text box it appends Vertical and Horizontal scroll-bar,

@ nitupune please don't pollute tkostus thread with a different issue, open your own.
@tkostus : i just commited a version fixing underLine misalignment and shouldnot get focused color when disabled=true. it is in lib/ folder here in this repo, please tell me if it fixes your issues.

@Sharlaan many thanks - I have actually figured this out since and made some other tweaks to make it practically undistinguishable from the original SelectField:

I've forked your repo and made the tweaks there, mostly in SelectionPresenter - feel free to review and include in your code. It's not very scientific so I'm not proposing a PR :)


feat: Add depth as store SMT argument

Prior to this commit, MerkleStore allowed the creation of Sparse Merkle tree only with the maximum depth of 63. However, this doesn't fit the Tiered Sparse Merkle tree requirements, as it will contain trees of depth 16.
This commit adds the depth argument to the MerkleStore methods that will create Sparse Merkle trees.

@bobbinth I also had to increase the depth limit to 64 to conform with the maximum depth of the tiered sparse merkle tree


batch zkapp txn tool [develop]

Develop version of the following PRs

#11988
#11989

Checklist:

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

Closes #0000

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me


Custom Domain Integration for AWS

As a developer, I want to have the option to set up a custom domain which gets used as an alias for the CDN in AWS

implemented by #14


xml for all endpoints

Hi Hans, may I ask what the plan is here? Will JSON stay an option for all/some endpoints? I assume for the schemas, JSON(-schema) would be the simplest way to use.


added regex to add css files to generated file

This PR looks at the users _app.{js,tsx} file and adds all css imports that where in it to the generate file.

Big if true


placemarkFromPosition(location) keeps working and not working

🔙 Regression

Description
location = geolocator.getCurrentPosition(
desiredAccuracy: LocationAccuracy.best);
placemark = geolocator.placemarkFromPosition(location);
Both methods to getCurrentPosition and placemarkFromPosition work on android emulator at home.
However, getCurrentPosition method works, but placemarkFromPosition does not work in other places.
###EDIT
I noticed that the location that was derived from getCurrentPosition in other city was same as my home's location. Even though I ran the app after I had wiped the emulator's data. I wonder why I wasn't able to get placemark.
Reproduction steps

On app's initState, call the method below.import 'package:geolocator/geolocator.dart';

Future<Country> getCountryInstance() async {
  Geolocator geolocator = new Geolocator();
  Position currentLocation;
  List<Placemark> placemark;
  String isoCountryCode;

  try {
    currentLocation = await geolocator.getCurrentPosition(
    desiredAccuracy: LocationAccuracy.best);
   placemark = await geolocator.placemarkFromPosition(currentLocation);
 } catch (e) {
   print(e);
 }
 print("currentLocation: $currentLocation");        // Lat: 77.4219983, Long: -142.084
 print("placemark: $placemark");                          //  null
 if(placemark != null) {
    isoCountryCode = placemark[0].isoCountryCode;
    print("user iso country code: $isoCountryCode");

 }
   return isoCountryCode;
 }

     `
### Configuration

**Version:** 5.1.1+1

**Platform:**
- [ ] :iphone: iOS
- [✔] :robot: Android

I'm using version ^5.1.4 and I've same issue.
✓] Flutter (Channel stable, v1.9.1+hotfix.4, on Mac OS X 10.14.6 18G103, locale en-AR)
    • Flutter version 1.9.1+hotfix.4 at /Users/user/Documents/Flutter/flutter
    • Framework revision cc949a8e8b (9 days ago), 2019-09-27 15:04:59 -0700
    • Engine revision b863200c37
    • Dart version 2.5.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/user/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.0, Build version 11A420a
    • CocoaPods version 1.7.4

[✓] Android Studio (version 3.4)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 35.3.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

[✓] IntelliJ IDEA Community Edition (version 2019.1.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 35.3.3
    • Dart plugin version 191.7141.49

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

[✓] Connected device (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)

• No issues found!

Since version 6.0.0 the geocoding features are no longer part of the geolocator plugin and moved to their own plugin which can be found here.


Annual Review - EVerest

@yarille request from Marco, our CEO. He would want present to review himself, but he is in Japan. He request to be the first project review during the meeting on the 19th.

@RobertDeLeeuw sure. No problem. I can schedule this right after the general updates.

Moving out to 1/9 per Marco's request.

Renewed for another year on 1/9


Race condition with setTimeout

I have two elements on a page that each have an on-click-outsite set (two date pickers), say dp1 and dp2. I am consistently able to produce a race condition where i open dp1, open dp2 (which closes dp1), open dp1 again (closing dp2), and open dp2 again, but dp1 one does not close. The problem is that the setTimeout runs in the next tick, which allows the deregistration of a handler to destroy the dom listener, such that the handler that is then registered is never called.
The simple (naive?) fix is to just remove the setTimeout and run the handler registration immediately. (I'm not sure of the use case that the setTimeout addresses.)
If the setTimeout is required, the fix is to move the code that creates the dom listener inside the setTimeout callback.
Both of these fixes are trivial, and if there is a preference i can provide a PR.

Bump. Got burned by the same issue, although I had to investigate it myself to recognize what it was.
To speculate on the answer to your question @mlohbihler, I think the settimeout might be there as a precaution in case something isn't fully loaded on the DOM when the bind hook is executed. I don't know exactly what that would be though.
If it helps at all, this is what the function would look like using fix 2:
function bind(el, binding) {
  var callback = binding.value,
      modifiers = binding.modifiers;

  // unbind any existing listeners first

  unbind(el);

  setTimeout(function () {
    if (!domListener) {
      domListener = on(document.documentElement, 'click', function (event) {
        registeredHandlers.forEach(function (handler) {
          return handler.check(event);
        });
      });
    }

    registeredHandlers.push(modifiers.static ? staticStrategy(el, callback) : dynamicStrategy(el, callback));
  }, 0);
}


Unable to access swift-build on Azure Devops

https://github.com/compnerd/swift-build/blob/master/docs/GettingStartedWindows.md links to https://compnerd.visualstudio.com/compnerd/swift-build (Section Installation, Step 2) which gives me a 401 Unauthorized. I could've sworn that I was able to access that URL a few weeks ago.

Should it link to https://compnerd.visualstudio.com/swift-build instead?

Weird, they should be the same.  If the other URL works for you, a PR for that would be wonderful!

Never mind, I cannot read.  You are absolutely correct with the second URL.  There’s an extra user name in the first one and is incorrect.


feat: pragma randomness

Closes #554

This change is

@okhaimie-dev  great thanks! will review later today

is this Pragma VRF? It should be named as such. It's confusing, I don't know what this refers to when I read this contract. Your contract is also named randomness

it is the pragma randomness contract address. I will make the variable names more descriptive in my next commit using the new example you suggested

Hey @okhaimie-dev are you going to revise this? Do you need any help?

@enitrat
I'll complete it this week. I'll reach out if I need further assistance MAX by Wednesday

I am still working on the Dice game example.
https://github.com/okhaimie-dev/dice-game
I will used this to replace the initial example I've got in a bit

Closing this, you can reopen a new one when you're ready


fixed the processing io.EOF on grpc streams

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

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

Project coverage is 44.26%. Comparing base (41c4ab0) to head (233938b).

Files
Patch %
Lines

internal/conn/grpc_client_stream.go
0.00%
5 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1325      +/-   ##
==========================================
+ Coverage   42.00%   44.26%   +2.26%
==========================================
  Files         321      258      -63
  Lines       34130    29134    -4996
==========================================
- Hits        14335    12896    -1439
+ Misses      19325    15793    -3532
+ Partials      470      445      -25

Flag
Coverage Δ

go-1.20.x
44.26% <0.00%> (-0.07%)
:arrow_down:

go-1.21.x
?

go-1.22.x
?

macOS
?

ubuntu
?

unit
44.26% <0.00%> (+2.26%)
:arrow_up:

windows
44.26% <0.00%> (+2.29%)
:arrow_up:

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

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


Eliminate lost file botches in uploading and deleting large numbers of files.

@mseul @Lizabrazil @cuahsimarko @dtarb @engineSound
There are several ways in which a long-running Django request can be interrupted. These include pressing the "back" button and Django timeouts. At present, file ingestion and deletion is subject to these interruptions, leading to file integrity failures in which Django and iRODS disagree on the presence of files. This is most likely with an upload request that uploads large numbers of small files, or a deletion request for a large number of files.
Symptoms are different for the two kinds of interruptions:

File upload interruptions leads to files that are present in iRODS but not in Django. This is because sending them to iRODS is nearly instant but ingesting them into Django can take considerable time.

File deletion interruptions lead to files that are present in Django but not in iRODS. This is likewise because iRODS deletion is nearly instant but deleting files from Django can take considerable time.

This is unrelated to uploading and downloading of "large files". This instead affects requests that act on large numbers of (typically small) files.
The user symptoms of this are that files (or even whole resources) that are deleted are still present in Django after they disappear from iRODS, while files that are uploaded disappear from the UI but are invisibly still present in iRODS. The user perceives an upload botch as a failed upload, but the upload consumes quota. The user perceives the deletion of a resource as unsuccessful, but the resource doesn't contain any data.
The proposed solution for this is to make uploads and deletions of large numbers of files uninterruptible, by backgrounding the process of ingesting the files. This means -- to the user -- that the resource is unavailable while an upload is in process.
At present, the only way to deal with such integrity problems is via the management commands check_resource and repair_resource. These commands indicate when there are problems and try to repair them. However, it would be considerably better if these problems could not occur.

In some cases the symptoms of this are serious enough to completely break the UI for the resource.

@alvacouch thanks for sorting this out.  I was a bit unclear what the meaning of "the resource is unavailable while an upload is in progress" is, or how it would manifest.  What if a user (even separate user) hits the resource landing page during this operation (from a separate browser tab or window or computer).  What happens.  What about if the landing page is already open, while actions are happening from another browser.

@cuahsimarko See https://docs.google.com/document/d/1KvtdBfZcQ6kaftnupali-5_Osi7NtA3DyVtxK6pxB0s/edit?usp=sharing for detailed notes on where problems occur in the code.

@dtarb @Lizabrazil It will look like the resource is still uploading files, which it actually is. The spinner will continue. However, the user can do other things while this is happening, including working with other resources. If you browse back to the resource while it is -- e.g. -- still uploading 1000 files, the spinner will return to tell you that an operation is still in progress. This spinner will also appear during other time consuming operations, including, e.g., unzip zipfile into resource. It will persist as long as the operation is running, and will disappear when the operation completes. Since these operations can take considerably longer than a Django request -- which is the root of all evil -- the spinner will persist through multiple user commands until the operation is done. During this time, the file interface will remain unavailable, but the user can do other things.

I'm removing myself for now as we were unable to reproduce and there are still some unknowns as to root cause.

@alvacouch tagging you as it part of large file handling and async

@alvacouch tagging you as it part of large file handling and async

Addressed via asynchronous operations.


Release successfully created, but build step failed.

Hi,
I'm using the gitea release module to push a release and its content to our gitea.
The module seems to successfully push the content to gitea, as you can see here:

gitea_release log seems to be indicate that the release was ok, but as you can see here, the build step fails:

The version of gitea used is 1.5.1, and the attachement mime types have been configured in the app.ini to allow tar.gz mime types.

Closing because of lacking details


[xamarin.ios] update quickstart to Unified (64-bit)

based on a diff after updating a template downloaded from manage.windowsazure.com - hard to test though.
I would like to do a subsequent PR with updated resources for iPhone 6/Plus.
resubmission of this PR on the "old" repo.

This looks good and I tested it on my side and it worked OK with one minor tweak. For this to go into the master branch, it should not reference the 2.0.0-beta version of the MobileServices SDK (I made a comment against the commit).

Fixed in 8ca5981f174b7ab74c575c61893223355bb3a988 (now references 1.3.1)


CLOUDP-117936: [MongoCLI] We should remove the deprecate message when generating doc pages

Proposed changes

Jira ticket: CLOUDP-117936

Checklist

[x] I have signed the MongoDB CLA
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added any necessary documentation in document requirements section listed in CONTRIBUTING.md (if appropriate)
[x] I have addressed the @mongodb/docs-cloud-team comments (if appropriate)
[x] I have updated e2e/E2E-TESTS.md (if an e2e test has been added)
[x] I have run make fmt and formatted my code

Further comments

Issue:
https://www.mongodb.com/docs/mongocli/stable/command/mongocli-atlas/ shows two deprecate messages

LGTM


events.js:48 throw arguments[1]; // Unhandled 'error' event

I have written below code in streamreader.js file.
var fs = require("fs");
var readStream;
try
{
readStream=fs.createReadStream("hello.txt");
readStream.on('data',function(data){
console.log(data);
});
readStream.on('end',function(){
console.load('file ended');
});
}
catch(err)
{
readStream.on('error',function()
{
console.log(err);
});
}
but when i try to run this file using >node streamreader.js
I am getting below error.
events.js:48
throw arguments[1]; // Unhandled 'error' even
^
Error: ENOENT, The operation completed successfully.
'hello.txt'
Can you help me to resolve this issue?

This is not in any way related to Socket.IO, if you have issues post them at the node.js mailing list not here.
<3 Thanks.


Implement JSON object

JerryScript-DCO-1.0-Signed-off-by: Zoltan Herczeg<EMAIL_ADDRESS>JerryScript-DCO-1.0-Signed-off-by: Roland Takacs<EMAIL_ADDRESS>

Since the project went on public, this is not updated automatically anymore.

New PR created: #307


Cannot render html: 'int' object has no attribute 'text'

Prework

[X] Read and agree to the code of conduct and contributing guidelines.
[X] If there is already a relevant issue, whether open or closed, comment on the existing thread instead of posting a new issue.

Didn't see this issue already but let me know if it already exists!
Description
It appears that data frames with integer column names error when attempting to convert to html, and don't display nicely in notebook output
Reproducible example
In a Jupyter notebook, run the following and then try to display the objects generated by GT
import pandas as pd
import numpy as np

from great_tables import GT

df1 = pd.DataFrame(np.random.randint(0,100, (3, 3)))
x = GT(df1)

df2 = pd.DataFrame(np.random.randn(3, 3))
y = GT(df2)

df3 = pd.DataFrame({"x": [1,2,3]})
z = GT(df3)

df1_colnames = df1 = pd.DataFrame(np.random.randint(0,100, (3, 3)), columns = ['x', 'y', 'z'])
x_colnames = GT(df1_colnames)

z displays as expected, as an html table. x and y give the following error, and do not render in html
If I rename the columns, then tables that previously failed are now able to render and display successfully
File ~/.pyenv/versions/3.10.11/lib/python3.10/site-packages/great_tables/_text.py:52, in _process_text(x)
     50     type = "plaintext"
     51 else:
---> 52     text = x.text
     53     type = x.type
     55 if type == "from_markdown":

AttributeError: 'int' object has no attribute 'text'

Expected result
All of the above should be able to display as nice html tables in a notebook.
Development environment

Operating System: Mac OS
Python 3.10.11
great_tables 0.1.5

Hmm... I suspect the error occurs when a DataFrame has non-string names for columns (this is possible in pandas, but not in polars).
It seems like we should either...

explicitly raise an error, when someone tries to use GT() on a frame with non-string names, OR
allow non-strings as names, but coerce them to strings, and give a warning.

Also, just as a side note, you could coerce the labels to strings using the most simple/straightforward method, e.g. df.columns.astype(str) which is probably sufficient for the most common case of integer labels but may not always match the coercion to string that pandas does, or if you wanted to more closely match the pandas behavior (since you mention this is probably not an issue for polars dataframes anyway), you could use the pandas string formatter e.g. pd.io.formats.format.GenericArrayFormatter(df1.columns).get_result()

it also seems odd to me as a user that for example, non-string row labels render perfectly fine, but non-string column labels would be a problem

That's fair--we need to coerce the names to strings, so that selection behavior is unambiguous (e.g. whether 0 a position or a name is ambiguous in pandas, and this has had a big effect on its API), but once they're strings it should be okay!

Now fixed by https://github.com/posit-dev/great-tables/pull/127. Thanks @machow !


sanitycheck: modified to upload directory with multiple files

Updated class SanityCheck to support uploading of the multiple files.
For the reference I used class TCF, and realization of the methods for
uploading of multiple files.
Now using SanityCheck we can upload a directory with .xml files in it.
I deleted a -P setup, now is not necessary to use a Plan configuration.
Also deleted -c configuration setup, because config board name is
detecting by the script from the file name.
To run script please for the reference use that command:
python3 report.py --runner sanitycheck -j folder_name -V commit_name    -p 5 -s 23 -m 9
Where folder_name is the folder name with .xml files and commit_name is the name of the commit.
Signed-off-by: Maksim Masalski<EMAIL_ADDRESS>

@nashif
Please review my changes, it is what we talked with you about in Shanghai.

@nashif I resolved all errors, please take a look.

@xuhao8210 Please review it too.

@nashif please review my changes, and leave comments.

Updating the testsuite should not be part of uploading results. Out testsuite will become really messy if we update the suite whenever someone try to upload new results. Testsuite management need to be driven using another tool dedicated for test management.

@nashif

I've separated commits.

My opinion about creating separate tools: one for uploading and one for updating test suite.
I think your suggestion can be implemented, but my vision how I understand your idea, it can be not convenient to use that tools together.
That how I see it (maybe we have different vision) on that implementation.
If we will have 2 tools. We start the process from running of the uploading tool. It uploads test cases already stored in the TestRail, then writes into the log file information about missed test cases and new sections to add (that tool is my commit #1). After we need to run second tool to update test suite and add only new test cases into TestRail, using previous log file information. Then need to run first tool again, to receive a proper Test Run in the TestRail.

We can't update TestRun once it was created in the TestRail, that why I decided to modify my script. Because when I press button "Rerun" in the TestRun window, nothing happens.
Of course may be a problem that user can be not experienced and can use script not carefully, but that problem can be solve by giving TestRail access to the right people, who understand what he is doing.

2\. I think your suggestion can be implemented, but my vision how I understand your idea, it can be not convenient to use that tools together.
    That how I see it (maybe we have different vision) on that implementation.

This really need to be a different tool, not everyone uploading test results will have access to change the testsuite.
It also leads to a mess in the testsuite when someone had local tests that nobody else has, all of the sudden we will end up with a testsuite that is not manageable and has lots of dead entries and things that are not part of zephyr.

We can't update TestRun once it was created in the TestRail, that why I decided to modify my script. Because when I press button "Rerun" in the TestRun window, nothing happens.
Of course may be a problem that user can be not experienced and can use script not carefully, but that problem can be solve by giving TestRail access to the right people.

The process should be for the testcases to be updated and maintained separately from the test runs. This means it is not tied to a testrun and will have to be managed on its own.
Basically the process will be:

run a script that verifies that the testsuite in testrail is in sync with what we have in the git tree. If not in sync, make sure to sync them. (This step can be simplified and managed using output from sanitycheck)
Run tests and generate results
Upload results, the tests in the run should all be in Testrail.

btw, look at https://github.com/nashif/zephyr/tree/testsuite_sc and run
sanitycheck --test-tree
after install the anytree python module (pip3 install --user anytree)
This will give you an idea of where I am going with this, the output represents the testsuite and how we can sync it with testrail, basically the tree should represent what we have in testrail now and should be global and not based on test runs and results which only gives you what you actually have tested, not all possible tests that could be run.

The above now is a PR in https://github.com/zephyrproject-rtos/zephyr/pull/21097

The above now is a PR in zephyrproject-rtos/zephyr#21097

Thanks for your comments, I will start to implement it.
BTW, do you know why sometimes Sanitycheck generates corrupted results for some test cases in the test report .xml files?
Like that one below. My current script uploads that corrupted test cases too. But in fact that test cases are one test case called sys.mutex.nouser. There are more examples like that one in the TestRail.
|   | C74142 | sys.mutex.nouser.thread_06@119. Failed to take mutex 0x0012e088 |   |   |
-- | -- | -- | -- | -- | -- | --
  |   | C73985 | sys.mutex.nouser |   |   |
  |   | C73829 | sys.mutex.nouser.thread_06@119. Failed to take mutex 0x20002288

Thanks for your comments, I will start to implement it.

implement what?

BTW, @nashif do you know why sometimes Sanitycheck generates corrupted results > for some test cases in the test report .xml files?

Not sure what the issue is, please open a bug with more details.

Thanks for your comments, I will start to implement it.

implement what?

BTW, @nashif do you know why sometimes Sanitycheck generates corrupted results > for some test cases in the test report .xml files?

Not sure what the issue is, please open a bug with more details.

I mean to change my current script according to your comments. Am I right?

@nashif Hello. I decided to move 3rd commit what I had there to another PR, it will be another script. That PR now just have an updated report.py script to upload multiple files and nothing more. Could you please merge that PR to let me move forward with the development of the second script? Second script will update a test suite. Updated report.py from the current PR is vital for my team members to build a new open-source feature to make it possible automatically add defects to the test cases. That feature can let community see defects in the TestRail for the test cases associated with the PR on github for zephyrproject. Thanks.


IGAPP-1032: Add finnish language

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword IGAPP.

I hope, I did it the right way. I was not sure whether it was needed in Testumgebung or Testumgebung Ende-zu-Ende, so I just created a few pages in both.
There should be 4 pages in Finnish:
Willkomen
-Beratung und Hillfe
--Test Inhalte
--Helferkreis

I hope, I did it the right way. I was not sure whether it was needed in Testumgebung or Testumgebung Ende-zu-Ende, so I just created a few pages in both. There should be 4 pages in Finnish: Willkomen -Beratung und Hillfe --Test Inhalte --Helferkreis

Nice, thank you. It is enough to make changes like this in Testumgebung. Works fine on web and android.

Tested on firefox and android. Had to uncomment lines 34 to 44 in loadEvents.ts and replace events with [] in line 45 and 46 though since events endpoint is currently not working, see https://chat.tuerantuer.org/digitalfabrik/pl/iq7n7jmizfb3pxzy5t53krnope.
Will app constans be translated as well?

Anything I need to do here or is it fine to merge?

Fine to merge :) We can do it in a separate PR.


Navbar Improvements and Footer redesign

The search bar is way too small.
The footer needs to be at the bottom of the page, not the top.

I would like to work on this as part of hacktoberfest.

Go ahead! @TheInspiredConjurer

Don't forget to star the repository!

Done!

@TheInspiredConjurer updates?

@TheInspiredConjurer as the deadline for solving the issue is 3 days after it was opened, this issue will be closed at the end of today if you don't show any activity.

@lovishprabhakar okay, I will submit it by today


Deny video sharing

Motivation

ℹ In my application i would like to show unlisted videos from youtube. I would like to prevent users from clicking on the title of the video in order to open it on youtube and be able to share it, or click directly on the share button from the player. Now I can do this disabling user interaction but than the user can't use the video controls.

Solution

ℹ Set the code in order that if I set isUserInteractionEnabled: false and showControls: true the controls are shown but the user can't click in the 3 dots and share the link or click in the video title and open YouTube.

The YouTubePlayerKit.Configuration includes the playInline parameter which can be set to false in order to automatically start the video in fullscreen after the video has been loaded.
YouTubePlayer(
    source: .url("https://youtube.com/...."),
    configuration: .init(
        playInline: false
    )
)

Reference: https://developers.google.com/youtube/player_parameters


enchance the text box colors

I want to change the text box colors to somewhat glossy as to make it trendy and new.
Please assign this task to me so that I start working on it. @DhairyaBahl Please assign it to me.
Thanks
Himika Mishra

@Himika-Mishra Assigned ! All the best 🔥

Thanx a lott!!


Questions about median scaling

Hi, as shown in the code, the last activation function is tanh,  the depth image used for surpervised is scaled to 0-1 in training, and the output of network will scale to 0~80m (kitti) in testing.
# training
if self.dataset.upper() == 'KITTI' or self.dataset.upper() == 'VKITTI':
	#print("Using outdoor scene transform.")
	arr_depth = np.array(depth, dtype=np.float32)
	arr_depth[arr_depth>8000.0]=8000.0
	arr_depth /= 8000.0   # cm -> m
	arr_depth[arr_depth<0.0] = 0.0

# testing
pred_depth_np += 1.0
pred_depth_np /= 2.0
pred_depth_np *= 80.0

I know it based on previous work, but i am curious why scaling instead of directly regressing to the real depth value? Will it bring performance improvements?
Thanks in advance!

This code is not median scaling. During the training process, we linearly map the ground truth depth (0~80m) to (-1~1). The purpose of this operation is to help the training process to be more stable.

Does this have any effect on accuracy？

By the way, could you explain what is the median scaling and why needs the operation? Thanks!

Since the depth predicted by our method is defined up to a scale factor, for evaluation we multiply the predicted
depth maps by a scalar that matches the median with the ground truth.

Ok, i got it, thanks!


Feature 26

Solves: https://github.com/MechanicalManiacs/UltimateGoal/issues/26

@KeshavVarad Made an accidental PR again. Watch out, Git can be tricky sometimes.


react-bootstrap's Checkbox component

I'm trying to alter my state with a checkbox, using createFieldClass with controls.checkbox on the Checkbox component from react-bootstrap.
However, when checking the component, the state gets set to "on" instead of true or false, and once I check the checkbox, I cannot uncheck it. What would be the right way to do this?
Using an <input type="checkbox" /> inside a  <Field> works fine, but I'd prefer to use the checkbox component react-bootstrap provides
It can be reproduced easily via this gist (should work fine if you paste it on esnextbin)

That should be working:
const CheckboxField = createFieldClass({
  'Checkbox': props => ({
    onChange: e => props.onChange(e.target.checked)
  })
})

@cbioley that worked perfectly, thanks!
Off topic question, what would be a good channel in discord to ask stuff related to the project? I hesitated to ask this on the redux-form channel

@rkrdo You're welcome ^^
I don't know.
You should ask @davidkpiano. He's really a nice guy.

Huge thanks @cbioley! Anyway, @rkrdo I'm (for the most part) always available on the Reactiflux discord chat under davidkpiano. I'll see if we can reuse redux-form or create react-redux-form.

Sounds good, thanks! @davidkpiano @cbioley

You can also fix it little bit cleaner like this:
const CheckboxField = createFieldClass({Checkbox: controls.checkbox})
Checkbox.defaultProps.type = 'checkbox'

@davidkpiano I tracked down the issue to this line, there is no type prop available in react-bootstrap's Checkbox.

Thanks, @sakarit! You can also add it manually:
<Field ...>
  <Checkbox type="checkbox" />
</Field>


Non-vision senses have no effect when using 'Token inset corners and center (5 points)' option for Range Points and Vision 5e is enabled

When Vision 5e is enabled and the ATV Range Points setting is 'Token inset corners and center (5 points)', tokens lose the ability to use any non-vision senses (hearing, tremorsense, etc.). When ATV Range Points is set to any other option then senses work as expected.
Verified this occurs with only ATV and dependencies and Vision 5e enabled. No errors in console.
Foundry v11.313, dnd5e 2.1.3, ATV v0.5.8, socketlib v1.0.13, libWrapper v<IP_ADDRESS>, Vision 5e v1.10.4, hosting: The Forge
Steps to recreate:

In ATV, select the 'Token inset corners and center (5 points)' option for Range Points.
Enable the Vision 5e module (if not already enabled).
Add 2 Starter Heroes to the scene from the SRD Starter Heroes compendium.
Add the Blind condition to one of the tokens and then select the Blind token.

Closed as I think this is fixed in v0.6 series.


Duplicate titles appear in navigation pane

Duplicate titles displaying in the navigation pane (see image). This doesn't affect the actual metadata but can be confusing.

Brays is no longer being developed and has been deprecated. You can follow future work with Mason


Change Hue entertainment brightness

Feature request
Is it possible to change the brightness of hue entertainment lamps?

What problem does this feature solve?
What does the proposed API look like?
How should this be implemented in your opinion?
Are you willing to work on this yourself?

Change the brightness at the color adjustments, this will reduce the overall.


REQUIRED process [ros_calib_init_optimizer-2] has died!

运行roslaunch linkalibr ros_calib_init.launch后报错如下：
[ INFO] [1652091019.397516439]: Frame: 547 / 550
[ INFO] [1652091019.652656943]: Frame: 548 / 550
[ INFO] [1652091020.011336717]: Frame: 549 / 550
[ INFO] [1652091020.301443733]: Frame: 550 / 550
Rot3:
0.999763   0.0107452   0.0189149
0.0106649   -0.999934  0.00433886
0.0189603 -0.00413611   -0.999812
Euler Angles: 0.248644  178.916  179.384
Marginal Covariance
0.596503 -0.00196232 -0.00136384
-0.00196232    0.587782   0.0279263
-0.00136384   0.0279263    0.594626
================================================================================REQUIRED process [ros_calib_init_optimizer-2] has died!
process has finished cleanly
log file: /home/hqc/.ros/log/3c887e46-cf4f-11ec-aca0-9822ef00c7d8/ros_calib_init_optimizer-2*.log
Initiating shutdown!
[ros_calib_init_optimizer-2] killing on exit
[ros_calib_init-1] killing on exit
shutting down processing monitor...
... shutting down processing monitor complete
done

@SubMishMar I didn't solve it, but I don't think this error will have much impact on the result, so I haven't considered this error

The process died on https://github.com/unmannedlab/imu_lidar_calibration/blob/main/linkalibr/src/ros_calib_init_optimizer.cpp#L173 just after processing the last frame.
[ INFO] [1692115069.770867042]: Frame: 550 / 550
================================================================================
REQUIRED process [ros_calib_init_optimizer-2] has died!
process has died [pid 1091109, exit code -11, cmd /home/alwyn/catkin_ws2/devel/lib/linkalibr/ros_calib_init_optimizer __name:=ros_calib_init_optimizer __log:=/home/alwyn/.ros/log/b643b8ce-36d0-11ee-90ab-2fb04df3755f/ros_calib_init_optimizer-2.log].
log file: /home/alwyn/.ros/log/b643b8ce-36d0-11ee-90ab-2fb04df3755f/ros_calib_init_optimizer-2*.log
Initiating shutdown!
================================================================================
[ros_calib_init_optimizer-2] killing on exit
[ros_calib_init-1] killing on exit
shutting down processing monitor...
... shutting down processing monitor complete


Coveralls coverage decreased 0.0% without any changes

I've enabled Coveralls again: https://github.com/mongomapper/mongomapper/pull/701.
It worked fine first, but doesn't now. The coverage decreased to 0.0%.
https://coveralls.io/github/mongomapper/mongomapper

Disabled for now: https://github.com/mongomapper/mongomapper/pull/704
I want to investigate it and re-enable.


fix: set language property in default config

When generating the default config file, set the language property.
Fixes #689 .

Thanks! How is VSCode treating multi-module projects with mixed languages, e.g. with a Java backend and a JS frontend module? The appmap.yml is only created when the "Run with AppMap" button is clicked, so Java should be fine as a default. I just don't know what will happen with AppMaps in a submodule with JS, for example.

VS Code relies on the installer to create appmap.yml. If it can't determine the project type, it asks the user. This seems equivalent to deciding that the project is a Java project when the user chooses "Start with AppMap" in IntelliJ.

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

v0.66.0
GitHub release

Your semantic-release bot :package::rocket:


Small Misspelling in options screen

"Check fo updates" Should be replaced with "Check for updates"

Heh... ops :D

:tada:


When will the weight and inference code be open sourced?

Your work is very meaningful and can bring more playability to the existing ecosystem based on sd1.5 and sd xl. When will it be open sourced? and hopefully SD XL will be open sourced too

still waiting..

+1

Thank you for your patience and here comes our voice! ELLA-SD1.5 is now released. Welcome to try! 🥰

Yeah; please let us know if the SDXL version will ever be released. I know you guys mentioned it requires "extensive review", but a definitive answer would be good

SDXL version will not be released, thanks.

So, useless. Back to waiting for SD3 I guess.

SDXL version will not be released, thanks.

Thank you for clarifying it :)
Could you at least provide the training code you used ? That would be absolutely amazing!

SDXL version will not be released, thanks.

Thank you for clarifying it @TencentQQGY @melohux :)
Could you at least provide the training code you used ? That would be absolutely amazing!

SDXL version will not be released, thanks.

Guys if you don't release it this will make very little to almost no impact. The community will move to alternate solutions soon with SD3 or RPG Master. This is your chance to make Tencent a leader in image generation, preempt SD3 before it comes out.

Leader in image creation?! Tencent is known for releasing half-working or straight up broken things.
I was very hyped for ELLA's release, but after about a month of waiting they release... this. And there's no training data, no finetuning scripts, paper skips over the most important implementation details etc. - all too typical of Tencent project qualities.
And now this "announcement" that SD XL adapter will not be released - this is Tencent's spit into community's face. Abhorrent.
As it stands right now, ELLA has serious quality problem. See https://github.com/kijai/ComfyUI-ELLA-wrapper/issues/3

M A S T E R P I E C E 👌
And to be frank with you - I do not care if they ever fix this: I'm moving on.
There are two alternative projects that achieve more or less same thing:

https://github.com/ShihaoZhaoZSH/LaVi-Bridge - currently has just SD1.4 adapter, but they're working on SD XL one.
https://github.com/litwellchi/M2Chat - impressive paper, but code and checkpoints are yet to be released.

Next time, when Tencent folks put out a paper promising SOTA in next 2 weeks - be wary, maybe.

Leader in image creation?! Tencent is known for releasing half-working or straight up broken things. I was very hyped for ELLA's release, but after about a month of waiting they release... this. And there's no training data, no finetuning scripts, paper skips over the most important implementation details etc. - all too typical of Tencent project qualities. And now this "announcement" that SD XL adapter will not be released - this is Tencent's spit into community's face. Abhorrent.
As it stands right now, ELLA has serious quality problem. See kijai/ComfyUI-ELLA-wrapper#3

M A S T E R P I E C E 👌
And to be frank with you - I do not care if they ever fix this: I'm moving on.
There are two alternative projects that achieve more or less same thing:

https://github.com/ShihaoZhaoZSH/LaVi-Bridge - currently has just SD1.4 adapter, but they're working on SD XL one.
https://github.com/litwellchi/M2Chat - impressive paper, but code and checkpoints are yet to be released.

Next time, when Tencent folks put out a paper promising SOTA in next 2 weeks - be wary, maybe.

I think this is being unnecessarily unfair. The weights literally only just released, and there is a separate implementation to the one in that repository you linked which is making progress with improving quality. Furthermore the prompt adherence in ELLA for 1.5 is astounding, and can be used to create an initial image to then process through img2img to improve quality.
For a model that's only been out for 2 days I think it needs more time for the community to see what they can do with it.

So disappointed. I had high hopes for SDXL since SD3 might never be released

SDXL version will not be released, thanks.

Guys if you don't release it this will make very little to almost no impact. The community will move to alternate solutions soon with SD3 or RPG Master. This is your chance to make Tencent a leader in image generation, preempt SD3 before it comes out.

RPG Master???

SDXL version will not be released, thanks.

Guys if you don't release it this will make very little to almost no impact. The community will move to alternate solutions soon with SD3 or RPG Master. This is your chance to make Tencent a leader in image generation, preempt SD3 before it comes out.

RPG Master???

@JelloWizard https://github.com/YangLing0818/RPG-DiffusionMaster


raspberrypi-eeprom: 2023.01.11-138c0 -> 2023.10.30-2712

Description of changes
https://github.com/raspberrypi/rpi-eeprom/releases/tag/v2023.10.18-2712
https://github.com/raspberrypi/rpi-eeprom/releases/tag/v2023.10.30-2712
This release of rpi-eeprom adds support for the Raspberry Pi 5, requiring some changes.
I have only tested this on a Raspberry Pi 4.
Also add myself as maintainer.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2806

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3050

Result of nixpkgs-review pr 262105 run on x86_64-linux 1

1 package built:

raspberrypi-eeprom

LGTM, thanks!


Added @Oracel Smith Microcontainer Builder

Smith
[Explain what the project is about and why it should be listed]
A tool by Oracle which creates Microcontainers. In essence it allows you to squash images after they have been built using the same principle as multi-stage builds. What is also interesting is it maintains dependencies.
By submitting this pull request I confirm I've read and complied with the below requirements.
Yup
Please read it multiple times. I spent a lot of time on these guidelines and most people miss a lot.
I did

I have read and understood the contribution guidelines

The project submitted conforms to the quality standards outlined in the contribution guidelines

Go to the contribution guidelines and read it again.

RULES (because you probably didn't read the guidelines):
Links will be tested with awesome_bot. Please make sure your URL to be listed is unique (do some search queries), otherwise put a link at the end of README.md file.

FORMAT

Use the following format: [Name](Link) - Description.
[Name] should be the title of the project
(Link) should be pointing to a GitHub repository, not to a website
Description. should end with a full stop/period
For GitHub repositories, add the name of the author with a link to its GitHub account after the description, like By [@author](GitHub account)

ORDER: The entries are in lexical order (English letter order)
LABEL (Optional): Add one of the following icons next to the project name like [Name](Link) :icon: - Description.

:heavy_dollar_sign: indicates if one needs a paying subscription to use the project.
:construction: indicates that the project is a work in progress/beta release.
:skull: indicates that the project is obsolete, but still worth mentioning it.

ping @vegasbrianc

Sorry somehow I missed your feedback @veggiemonk I've update the PR according to your feedback. Thanks and hope all is well!!


\color affects following \right, put array cells in their own groups

Fix #1844 by giving leftright nodes a rightColor attribute for how to color the right bracket.  Now \color sets the macro \current@color in the current environment (in particular, resetting after \right), just like color.sty does in LaTeX.  This is used to specially pass the current color into any following \right and then into the leftright parse node.
To confirm LaTeX behavior, here's how it renders the new ColorImplicit test:

(I tried texcmp, but it doesn't work very well with colored text; suffice to say the letters are also aligned with our test.)

Failing screenshot tests show that there's currently environment leakage from one cell of an array to the next, unlike in LaTeX.  I'll need to fix this before this can be reviewed/merged.

Codecov Report

Merging #1845 into master will decrease coverage by 0.01%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master    #1845      +/-   ##
==========================================
- Coverage   94.63%   94.62%   -0.02%
==========================================
  Files          80       80
  Lines        4664     4673       +9
  Branches      811      814       +3
==========================================
+ Hits         4414     4422       +8
- Misses        225      226       +1
  Partials       25       25

Flag
Coverage Δ

#screenshotter
88.86% <88.88%> (ø)
:arrow_up:

#test
86.68% <77.77%> (-0.02%)
:arrow_down:

Impacted Files
Coverage Δ

src/parseNode.js
84.21% <ø> (ø)
:arrow_up:

src/environments/array.js
98.31% <100%> (+0.01%)
:arrow_up:

src/functions/color.js
100% <100%> (ø)
:arrow_up:

src/functions/delimsizing.js
96.7% <83.33%> (-0.95%)
:arrow_down:

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 7f778d1...b3c02f4. Read the comment docs.

Fixed! (Turned out to be really easy.) So this PR also fixes an issue with \defs incorrectly spanning multiple cells of an array.

OK, group isolation wasn't quite as easy as I thought, because of the messyness that is nextToken.  I fixed most cases, but the very first character after \begin{matrix} is not in a group like it should be. Fixing that will be painful... and this PR is definitely an improvement (currently there is no isolation!), so probably worth merging for now.

May I ask what is holding up this pull request? I need to be able to colour one side of a bracket without knowing the size of the contents, so I can't use \big, etc...

Sorry for the delay; I hadn't realized this was blocked on me. I just implemented @kevinbarabash's comments, so it's ready for (re)review.


Reduces files to the built-in makefile type for pre-commit

As nudged here, but this won't be fully effective until https://github.com/pre-commit/identify/pull/353 is merged.
I think this is safe to merge before 353 because so few people use *.make... I think I've seen it maybe a few times in 20 years. Also, this hook is so new to checkmake that probably not that many people are using it yet.
Checklist
Not all of these might apply to your change but the more you are able to check
the easier it will be to get your contribution merged.

[X] CI passes
[X] Description of proposed change
[X] Documentation (README, docs/, man pages) is updated
[X] Existing issue is referenced if there is one
[ ] Unit tests for the proposed change

All of the upstream stuff is merged, so this can go in, @mrtazz

@mrtazz this is safe to merge now

@mrtazz Any chance for a merge?

sorry for the delay. Thanks for taking the time to contribute

Thank you! Could you perhaps do a release so that folks using the pre-commit hook can point to a tag instead of a hash?

@mrtazz Any chance for a tag?


pandoc vs markdown-it

I'm considering using another markdown implementation than pandoc, which will ease adding features like:

youtube videos
hide/show toggable content (ref #128)
bootstrap tabs without using full HTML syntax

markdown-it has a plugin system and seems to be the goto solution nowadays.
Thoughts?

I have no opinion about which solution is used in the backend. If markdown-it is simpler and more powerful I suppose it makes sense to switch.
However, when doing these more "advanced" features make sure there are decent fallbacks for the PDF, so that the projects still make sense when printed on paper.
For the examples above I am thinking printing a link (that can be clickable in the electronic pdf) to the Youtube video, and always showing the toggable content (ok for hints as discussed in #128 at least) should be enough.

Good points! Showing content in print can easily be fulfilled with @media print CSS.
This is semi urgent, as pandoc is having some troubles on windows. But I think this will have to wait a bit, as I would need to write a header section plugin(pandoc --section-divs). Rather fix the windows trouble first.

Plugins are now written:

markdown-it-header-sections
markdown-it-attrs

So it should be relative easy to make the transition over to markdown-it.

+1 to moving away from pandoc!

Last piece: markdown-it-implicit-figures.


Error evaluating filter expressions

Hello,
My cvs file contains the items below:
"Nom","Localité"
"Zoé","Paris"
"Léa","Lyon"

In the source file, I wrote:
csvtable
columns:
	- Nom
	- Localité
source: contact/export.csv
filter: Localité == "Paris"

If the column name to be filtered contains accents, the following error is encountered:
Error evaluating filter expressions: Lexical error on line 1. Unrecognized text.
Localité == "Paris"
-------^.

I have tried with single and double quotes, but it does not work.
Could not parse CSV table spec: Unexpected scalar at node end at line 6, column 10:

filter: "Localité" == "Paris"
         ^

The accents are not a problem in the values, it matches correctly:
filter:  Nom == "Zoé"

For the column sorting, I have the same error :
Error evaluating filter expressions: Lexical error on line 1. Unrecognized text.
Société
----^.

with this code
columns:
	- Nom de famille
	- Prénom
	- Société
	- Emails
	- Téléphones
	- Localité
	- Région
	- URL
	- Notes
source: contact/export-contacts.csv
sortBy:
	- Société
	- Nom de famille
	- Prénom

Unfortunately, the library we're using for handling filter expressions and sorting (filtrex) doesn't support non-ascii column names in much the same way that a programming language isn't likely to support them for variable names -- and that's because under the hood, those column names are variable names.  There's good news, though: you can easily set a variable name for your columns by using the extended syntax discussed here: https://github.com/coddingtonbear/obsidian-csv-table#user-content-selecting-particular-columns
Unfortunately, this does mean that your CSV file's header line should contain only ascii characters given that those header entries are used like variable names throughout the filtering and sorting system, but you can easily change the displayed header using the instructions above.  E.g.
```
columns:
- expression: nom_de_famille
  name: Nom de famille
- expression: prenom
  name: Prenóm
...
```

I know this isn't the answer that you want to hear, and I'm sorry about that!  If it helps: this limitation applies even for English headers, it's just a little less likely to pinch in English given that we'll only run into the problem if we have a header having a space in its name.
Good luck!


Add cssnano with the safe flag to the list of minifiers.

With the safe flag enabled, cssnano passes the suite. I'm going to overhaul this behaviour for version 4 (safe by default), but it would be nice to show that it does pass when you turn off the more risky optimisations.

Well, it seems the tool is becoming useful!


datepicker.css not found

datepicker.css is not there assests !

It is not on your vendor/bower/bootstrap-datepicker/dist? where is the bower folder setup?

A:\xampp_win32_5.6.12\htdocs\pm2\vendor\bower\bootstrap-datepicker\dist\css

@sjnlabs2013 thank you... we have changed: https://github.com/2amigos/yii2-date-picker-widget/blob/master/src/DatePickerAsset.php#L21 do you mind double checking again?

Have added new version tag. Simply do composer update. Hope it works.


Add optional safeTxGas feature

To match the new feature in the chains endpoint the FEATURES enum was extended to add SAFE_TX_GAS_OPTIONAL.

Why is this then alongside the same details that are to be returned from the endpoint?

Reopened because we still need to define this feature in the enum.


update to AWS SDK PHP 3.x & add profile/region support

This update adds the following features:

authentication through profile instead of key/secret pair
region support with auto-detection from environment or profile configuration if available
fix bug if no key is provided

This update also provides unit tests:

see https://continuousphp.com/git-hub/fdewinne/composer-aws/build/f4223295-2754-4ef4-96d1-21e956f3594a/output for execution details

I just restored the composer.lock file... even if it's not necessary in this case.
If you look at the final note in the composer.lock documentation, we are in the case of a library in a composer plugin. Then composer ignores this file on install command.

@fdewinne Yes but while the file is not needed for installing the library or plugin it has advantages to keep a lock file for libraries regardless: The use of composer install in CI is faster and deterministic (without the lock composer runs an update which it shouldn't do on CI), and on bug reports that contain test failures you can verify which version of dependencies were actually used in the run if a lock file is present.

By the way thanks for the PR! Just got distracted yesterday when I started the review so only left you with that one silly comment on the first file ;-)

Looks great apart from what I mentioned :-)

Tagged this as https://github.com/naderman/composer-aws/releases/tag/1.0.0


EditForm OnValidSubmit event is raised when validation failed using MustAsync() method.

Hi @ryanelian. I'm having an issue doing async validation using MustAsync() method. Even though the validation is performed fine, so I get validation errors, the EditForm's OnValidSubmit event callback i raised. This situation only occurs when getting on page and submitting for the first time. I reproduced the issue using your EmailCheckerService.
The model class:
public class TestModel
    {
        public string Id { get; set; }
    }

The validator:
public class TestModelValidator : AbstractValidator<TestModel>
    {
        private readonly EmailCheckerService _emailCheckerService;

        public TestModelValidator(EmailCheckerService emailCheckerService)
        {
            _emailCheckerService = emailCheckerService;
            RuleFor(Q => Q.Id).NotEmpty().MustAsync(CheckEmail);
        }

        private async Task<bool> CheckEmail(string emailId, CancellationToken cancellationToken = default)
        {
            return await _emailCheckerService.IsAvailableAsync(emailId);
        }
    }

The view:
<EditForm OnValidSubmit="@ValidSubmit" Model="@_testModel">
    <FluentValidator></FluentValidator>
    <InputText<EMAIL_ADDRESS>    <ValidationMessage For="() => _testModel.Id"></ValidationMessage>
    <button type="submit">Save</button>
</EditForm>

protected override void OnInitialized()
    {
        _testModel = new TestModel();
    }

    private async Task ValidSubmit()
    {
        await Task.CompletedTask;
    }

    private TestModel _testModel;

The project:
<Project Sdk="Microsoft.NET.Sdk.Web">

  <PropertyGroup>
    <TargetFramework>netcoreapp3.1</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Accelist.FluentValidation.Blazor" Version="3.0.0" />
    <PackageReference Include="FluentValidation" Version="9.2.2" />
    <PackageReference Include="FluentValidation.DependencyInjectionExtensions" Version="9.2.0" />
  </ItemGroup>

</Project>

Registration:
 services.AddScoped<EmailCheckerService>();
services.AddValidatorsFromAssemblyContaining<Program>();

To reproduce:

Go into index page.
Submit the empty form.
Put a breakpoint inside the ValidSubmit() method.
See that it is called when the async method from the service hasn't finished yet.

What is the .NET version?

Sorry, I'm not seeing the issue that you spoke of.
I modified the demo project to have an OnValidSubmit event:
<EditForm OnValidSubmit="ValidSubmit" Model="Form2">

    async Task ValidSubmit()
    {
        await Task.CompletedTask;
    }

Then I put a breakpoint into the ValidSubmit method. It's not hitting when there's an error.
Note that the Form2 is a NestedForm class which has MustAsync validation.
So I failed to reproduce your issue. Closing.

I am encountering the same problem, if you make the delay 5 secs you will notice it more easily. I dont think we can do anything about it bc the blazor framework is not awaiting the task and just continues.

I am encountering the same problem, if you make the delay 5 secs you will notice it more easily. I dont think we can do anything about it bc the blazor framework is not awaiting the task and just continues.

The Blazor framework can't await the task, because ValidateModel is async void.  That's a big no-no for exactly this sort of reason.  Until Blazor introduces an actually-async validation event, the only correct method is to synchronously wait for validation to complete before the event returns.  This has to be done very carefully to avoid deadlocking.


MenuItem not getting active class when using vue-router unless clicked on twice.

ElementUI version
1.1.2
OS/Browers version
Version 54.0.2840.98 (64-bit)
Vue version
2.1.0
Steps to reproduce
Use Menu component with vue-router. I set prop to router="true" and pass in a route object to the MenuItem component. Eg route="{ path: '/users' }".
What is Expected?
When clicking on a menu entry it should navigate to that route and also mark that route as active.
What is actually happening?
Clicking on a menu entry successfully navigates to the new route via vue-router but it does not mark the route as active unless I click on the menu entry a 2nd time. On the 2nd click it will then mark the route as active and apply the active class.

i don't know whether u use the default-active prop.


Parsing failure

This parses ok by the original EO interpreter, but fails to parse by this project.
+package org.eolang

[unused] > simplest-class
  unused' > @

@dours thank you for reporting! Can you share a link that explains this part of the syntax? The parser hasn't been updated in a long time, so there may be a lot of missing features like this one.

@nikololiahim  It is section 2.11 here https://arxiv.org/pdf/2111.13384.pdf


fix: Python identifier mapping in request Router

The sitemap.xml routing in several ViUR projects is broken. This re-enables support of parts containing dots within module/method routing.

I removed this for reasons in #865 ...

Then please talk to @icubes and tell him how to provide a sitemap.xml generated from a template, which worked in viur-core prior 3.6.

OK the routing will be done manually in this case, so the code won't be changed.
Furthermore, there will be a follow-up on this PR to entirely remove any replacements, to fix #1063.


Update nx 14.1

Codecov Report

Merging #102 (5bc1fc3) into master (9642dd0) will decrease coverage by 0.07%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #102      +/-   ##
==========================================
- Coverage   20.58%   20.51%   -0.08%
==========================================
  Files          62       63       +1
  Lines         272      273       +1
  Branches       36       36
==========================================
  Hits           56       56
- Misses        216      217       +1

Impacted Files
Coverage Δ

libs/eslint-config/jest.config.ts
0.00% <0.00%> (ø)

libs/nx-remotecache-s3/jest.config.ts
0.00% <0.00%> (ø)

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 9642dd0...5bc1fc3. Read the comment docs.


Native Go SQLite Driver

Have you considered using https://gitlab.com/cznic/sqlite instead of github.com/mattn/go-sqlite3 ?
A native SQLite driver would make it easier to compile for multi-arch/cross-compiled apps

Hi Jason,
Initially I was using a native driver but the performance looked a bit lackluster, but might have been a problem with my initial implementation.
I'll try again with a native driver and do some proper benchmarks.

I know a bit late, but I've finally added support for cgo/nocgo builds in #5 .
Non-cgo versions can be chosen by setting the nocgo tag. -tags nocgo. Hopefully it helps!


Automated cherry pick of #1290: fix: misc bugfxi for quotas

Cherry pick of #1290 on release/2.10.0.
#1290: fix: misc bugfxi for quotas

/lgtm
/approve

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.10.0@c99869a). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff                @@
##             release/2.10.0   #1291   +/-   ##
================================================
  Coverage                  ?   7.89%
================================================
  Files                     ?     369
  Lines                     ?   57635
  Branches                  ?       0
================================================
  Hits                      ?    4551
  Misses                    ?   52685
  Partials                  ?     399

Flag
Coverage Δ

#aFlag
7.89% <0%> (?)

Impacted Files
Coverage Δ

pkg/cloudcommon/policy/resources.go
0% <ø> (ø)

pkg/compute/models/quotas.go
3.17% <0%> (ø)

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 c99869a...52aa709. Read the comment docs.


BLU10 Exercise Notebook Q2 - repeated question

Describe the bug
In Q2, the second task in this question asks us to complete the function get_sparsity_representation_savings, however, the hash in the assert cell below  is identical to the one from the first question of this Exercise. I believe the expected answer for the second question is supposed to be different from what the first question is asking.
Also, the question should be asking us to round our result up to a certain decimal so that the assert doesn't go wrong for different people
Screenshots

This issue should be closed with commit https://github.com/LDSSA/batch3-students/commit/82ed23603911cb963a62fc9bb4d8008f3965ed90.
Please review @snovaisg :)


Prevent content from showing up below drawers

Such as here where the home screen balances show up below the accounts selector drawer

I'd add that, while we still have @stacks/ui and have to replace for Panda, it's quite likely we'll introduce visual errors like this. Would be better to get rid of first, then fix all UI issues, that patching midway.

Linking that work here to track as a blocker: https://github.com/leather-wallet/extension/issues/4379


Add xs:any Meta element to support extension/decoration of events

We need a way to allow the events or objects described in the events to be extended or decorated using content/structures described outside of the event-logging schema.
For example #30 potentially requires events to be decorated with some concept of the veracity of an event.  The veracity could be described by its own XML schema (or some other data format) and the content included in a Meta element within the event.

The only comment on this PR and #34 and #32 would be to enumerate the Crown Copyright years, perhaps change to 2016, 2017, 2018 or some such sequence.