Compiled Regex Text Clarification

Hi,

These codes are then converted to MSIL. When a method is called, the JIT compiler executes the MSIL

In the above text, it is the JIT compiler that executes the MSIL? I thought it would be the CLR instead.
Thanks.

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

ID: 14698b87-a995-290f-bdd7-9c2d6867704c
Version Independent ID: f199cd3f-8dce-ad4e-9712-3545e35557e7
Content: Best Practices for Regular Expressions in .NET
Content Source: docs/standard/base-types/best-practices.md
Product: dotnet-fundamentals
GitHub Login: @adegeo
Microsoft Alias: adegeo

@wildcatgerry By "executes", I think the author means "converts to machine code".


fix(cli): fix cts support

Summary
currently rspack-cli will throw error when .cts in folder which contains type:module in package.json

Require Documentation?

[x] No
[ ] Yes, the corresponding rspack-website PR is __

!canary


Error occuring while Log files sending to elastic search

please check logstash configuration once and check the installed java version.


SID creation: force aircraft to cross fix before turning?

Hey guys new to github and new to contributing, I was playing around with KJFK trying to code some SIDs and came across an issue with the DEEZZ4 departure specifically.  Departing 31L/R the sid calls for aircraft to fly to SKORR, CESID, YNKEE, then vectors to DEEZZ.  But as you can see below, aircraft cut corners and don't make it all the way to the assigned fixes.  Is there anyway to program the SID to stop this behavior?

hi @headphase, and welcome... two things I'm seeing here:

But as you can see below, aircraft cut corners
this might actually be desired (I'm not sure). Some fixes are "fly-by" (the pictured behavior), and others are "fly-over", where they must pass the fix before initiating the turn. That wouldn't be too tough to add, but it's currently not super high on my list of things to get to.

Departing 31L/R the sid calls for aircraft to fly to SKORR, CESID, YNKEE, then vectors to DEEZZ
This is a bigger deal, however. The chart says they should pass through YNKEE on a 173 heading, and then will be vectored to DEEZZ, whereas that airplane clearly has other things in mind. :stuck_out_tongue_winking_eye: Adding the option for a step of a SID to include simply a heading (or also stuff like a heading until reaching an altitude) would be a big improvement.
So to answer your question, no, there currently isn't a way to stop it from doing this, other than maybe adding a hidden fix a few miles past YNKEE (to make it hidden, just start the name with an underscore) as a temporary workaround, but it will not fix the issue of turning northbound. We'll keep this issue open as a record of that, so hopefully it can be fixed.

Aha!  Thanks I wasnt aware of hidden fixes, that will work great for now.  But yes it would be excellent if we could designate fly-over fixes with a symbol, maybe a hashtag or something?  Especially since they are important on these RNAV procedures.

Ok added in a few hidden fixes to get the desired behavior (made visible in the screenshot), but these pilots are still ignoring them and going straight to DEEZZ 😐
In fact they are now skipping the other published fixes completely, haha...

^Both a/c departed 31L. First got caf cvs w 31l, second got w 31l caf cvs.
Short answer: It's not your fault; this is a new bug you've found.
Long answer: Apparently what's happening is that the aircraft have a departure runway in mind when they spawn that is based on the wind, which is currently 180 at 6 (best aligned with the 22s). When he is told caf or assigned a SID with the sid command, he uses the runway he assumes he will be assigned. Once he is told to go to a different runway, he should update his fms to fly the different runway transition, but he doesn't. I'll need to add that in. So, good catch. :+1:
Temporary Workarounds:

assign runway first, then assign sid or use caf
Use runways best aligned with the wind (by not specifying at all, just say w or taxi instead of w 31l or taxi 31l), and then they will end up at the runway they assumed they would be assigned, and their behavior will match the expectation.

Created a new issue for the flyover fixes, to be tracked and worked separately: #427.


add a link to the Node.js API Docs

We could add a link to http://docs.maidsafe.net/safe_app_nodejs/ in the README to make it easier for people to find the documentation 🙂

Coverage remained the same at 83.932% when pulling ea9817ffd00141fc80173601f494b08e9897c612 on frabrunelle:docs into 9ddf8442301f8f3a1d8bc5d7c737241cb57d743c on maidsafe:master.

Coverage remained the same at 83.932% when pulling 939a2641db11c646b41d1e036f5d437b0d49f017 on frabrunelle:docs into 9ddf8442301f8f3a1d8bc5d7c737241cb57d743c on maidsafe:master.

Coverage remained the same at 83.932% when pulling 939a2641db11c646b41d1e036f5d437b0d49f017 on frabrunelle:docs into 9ddf8442301f8f3a1d8bc5d7c737241cb57d743c on maidsafe:master.

Coverage remained the same at 83.932% when pulling ea9817ffd00141fc80173601f494b08e9897c612 on frabrunelle:docs into 9ddf8442301f8f3a1d8bc5d7c737241cb57d743c on maidsafe:master.

Coverage remained the same at 83.932% when pulling 0cf8c760b0d509152471999fff40d5975fbdb025 on frabrunelle:docs into 9ddf8442301f8f3a1d8bc5d7c737241cb57d743c on maidsafe:master.

Coverage remained the same at 83.932% when pulling 0cf8c760b0d509152471999fff40d5975fbdb025 on frabrunelle:docs into 9ddf8442301f8f3a1d8bc5d7c737241cb57d743c on maidsafe:master.

Coverage remained the same at 84.477% when pulling 4b8092d00ce4f4377e43be457f3bfb94d8d73e6d on frabrunelle:docs into 2763a3f8596e29496dee0e688979ed5067c23812 on maidsafe:master.

Coverage remained the same at 84.414% when pulling 398723c72c2029f50415bdf4fa2eb525d6032b10 on frabrunelle:docs into ae62c5e76ade4665fc8069faf36dd8b6a93f467b on maidsafe:master.

Coverage remained the same at 84.414% when pulling 398723c72c2029f50415bdf4fa2eb525d6032b10 on frabrunelle:docs into ae62c5e76ade4665fc8069faf36dd8b6a93f467b on maidsafe:master.

Coverage remained the same at 84.008% when pulling e1ecb7e0e33bd471123eeee904937214360b46d5 on frabrunelle:docs into 8ef42f7754d1ae6391fb4329e8e4e2fa5f20baf8 on maidsafe:master.


chore(node): dependency updates

What

bumped @grpc-js/grpc to latest
bumped alpine docker image
bumped node (local) to 18
replaced jest with vitest
update actions workflow (same node version)

replace dependency A with B:


Sound Shaders

I'm currently unable to run sound shaders in SHADERed, both on desktop (AMD) and laptop (Intel surface pro), OS is win10 pro. A burst of noise is heard during initialise and SHADERed no long outputs sound after that.

Thanks for reporting!
I currently can't reproduce this issue on my machine but I plan to update that part of SHADERed anyway at some point in the near future (I want to switch from SFML's audio library to miniaudio before relase of v1.6) so I might fix it along the way. Currently I have no idea what's causing this issue.

I believe that this should be fixed with one of the recent updates (I switched from SFML to miniaudio).


Cannot use object of type Zxing\Qrcode\Decoder\ECBlocks as array

When I do this:

... I get
Cannot use object of type Zxing\Qrcode\Decoder\ECBlocks as array

... with a reference to  vendor\khanamiryan\qrcode-detector-decoder\lib\qrcode\decoder\Version.php at line 96:
return $this->ecBlocks[$ecLevel->getOrdinal()];

Is there some sort of hard limit for string length around 300-400 chars? Or is it dependent from the size of the QR code? Please explain this somewhere and/or catch the error before khanamiryan/qrcode-detector-decoder receives it.

Hi @ThomasLandauer here the URI is generated and readable without any issues when I insert your code fragment in a template. So I think this is not caused only by the length of the contents. Please create a failing test case so I can see if I can create a workaround.

Sorry, I'm not sure where/how to create an appropriate test :-(
I just found out that it depends on the size setting (in your config.yml). So try this:

At size=100 I get what you (probably) intended: Adjust your parameters to increase readability or disable built-in validation.

At size=200 I get the Zxing\Qrcode\Decoder\ECBlocks message from above.

And at size=1500 I get OutOfMemoryException :-)

Thank you, will check again later.

Hi @ThomasLandauer, here I also get the "Adjust your parameters" message you mentioned for sizes 100 and 600. However, when I generate an 1000 pixel image it validates or when I use my Android barcode scanner for the 600 pixel image I am able to read the result, so here the built-in validator is somewhat stricter than my own.
About the memory consumption: when I generate a 2500 pixel image, it validates, but this takes about 100MB to generate without validation and 350MB with validation. I also checked a 1000 pixel image and here it took 10MB to generate without validation and 100MB with validation, a tenfold increase. So I will add a warning in the readme about this big performance hit.
If you have issues with this strictness or memory consumption, please create an issue in the  https://github.com/khanamiryan/php-qrcode-detector-decoder library so the maintainer can look at it. I don't get an error at size 200 so I am not able to reproduce that issue (maybe it is fixed in the latest version) but this is clearly an issue in the validator library so if it still occurs you can also mention that there.


Handle server schema version 3

Not handled because I don't have the right device:

Add tests for Entry Control notification event (I don't have a device that does Entry Control CC notification events)

eventData in EntryControlNotificationArgsDataType can be a buffer. Should we parse the buffer like we do for values?

damn, that extra line took me out of the patch requirement 🙁  I could switch it back but I think this reads better

We want to test the entry control CC notification anyway, right?

We want to test the entry control CC notification anyway, right?

Ideally yes. I can fake a payload for it but would be great to get one from an actual device

Are we ok with proceeding with this as is and adding the Entry Control CC tests later? I'm not sure that we'll be able to get test data quickly if at all

What devices produce those notifications? I have a lock.

What devices produce those notifications? I have a lock.

I believe it is Secure Keypad Specific Device Class -> https://www.silabs.com/documents/public/miscellaneous/SDS10242-Z-Wave-Device-Class-Specification.pdf page 80

It looks like there are three products that qualify for this:
https://products.z-wavealliance.org/products/2478?selectedFrequencyId=2
https://products.z-wavealliance.org/products/3870?selectedFrequencyId=2
https://products.z-wavealliance.org/products/3575?selectedFrequencyId=-1

Thanks goes to @WizKid for providing the test data for the last test


Add token length limitation to Conversation Memory class

Currently, the Conversation Memory Class can store an arbitrary number of strings with no maximum length limitation.
However, the language model has a fixed prompt length.
Feature Requirements:
Add a parameter for the maximum token length in the Memory Class. If not specified, the Memory Class should default to the current behavior with no maximum string length.
If a string is added to the Memory Class that exceeds this maximum length, the Memory Class should truncate the messages to fit within the maximum length. [remove the oldest messages]

@li2109 Prompt lengths don't seem to be consistent across LLMs

@li2109 Prompt lengths don't seem to be consistent across LLMs

Yes, they are not the same. There will be a parameter you can set to adjust

Yes, they are not the same. There will be a parameter you can set to adjust

To look into whether the different LLMs provide APIs for token lengths, or do they use magic values in each LLM model configuration?

For 1) i don't think there's API specifically for this? Well, the prompt length is fixed given the model.
For 2) Magic number won't work as developer eventually will update it.
I think Neither would work. manual input of prompt length probably is the best option here.
Let's say the prompt length is 100 tokens; some people would prefer leaving 50 tokens for history; some might think 80 is better. Current user input also plays a role here.


Timestamp causes "No link definition found" warning

Thank you for the great plugin.
I found a problem when using the "Date when checked" option. The date is enclosed in square brackets [] which causes VS Code to assume it is the beginning of a link reference, throwing the "No link definition found" warning.
Is it possible to change the brackets to some other characters such as parenthesis () that won't confuse VS Code? If this user configurable, that would be ideal since I could see different linters complaining about different characters; or, just plain different user preferences.

Ditto on "Thank you for the great plugin." ;-)
I ran into this one too. Using parenthesis would work for me too, if adding a configuration is too much work.

I just pushed a PR with a suggestion of a fix for this issue. I hope it helps.


TextEditingController got exception because of invalid text selection

I got an exception after calling clear() in TextEditingController and then click TextField widget immediately and below is the call stack.
══╡ EXCEPTION CAUGHT BY GESTURE ╞════════════════════════════
═══════════════════════════════════════
I/flutter (20797): The following assertion was thrown while handling a gesture:
I/flutter (20797): invalid text selection: TextSelection(baseOffset: 3, extentOf
fset: 3, affinity:
I/flutter (20797): TextAffinity.upstream, isDirectional: false)
I/flutter (20797): When the exception was thrown, this was the stack:
I/flutter (20797): #0      TextEditingController.set:selection (package:flutter/
src/widgets/editable_text.dart:94)
I/flutter (20797): #1      EditableTextState._handleSelectionChanged (package:fl
utter/src/widgets/editable_text.dart:504)
I/flutter (20797): #2      RenderEditable.handleTap (package:flutter/src/renderi
ng/editable.dart:624)
I/flutter (20797): #3      _TextFieldState._handleTap (package:flutter/src/mater
ial/text_field.dart:404)
I/flutter (20797): #4      GestureRecognizer.invokeCallback (package:flutter/src
/gestures/recognizer.dart:102)
I/flutter (20797): #5      TapGestureRecognizer._checkUp (package:flutter/src/ge
stures/tap.dart:161)
I/flutter (20797): #6      TapGestureRecognizer.acceptGesture (package:flutter/s
rc/gestures/tap.dart:123)
I/flutter (20797): #7      GestureArenaManager.sweep (package:flutter/src/gestur
es/arena.dart:156)
I/flutter (20797): #8      _WidgetsFlutterBinding&BindingBase&GestureBinding.han
dleEvent (package:flutter/src/gestures/binding.dart:147)
I/flutter (20797): #9      _WidgetsFlutterBinding&BindingBase&GestureBinding.dis
patchEvent (package:flutter/src/gestures/binding.dart:121)
I/flutter (20797): #10     _WidgetsFlutterBinding&BindingBase&GestureBinding._ha
ndlePointerEvent (package:flutter/src/gestures/binding.dart:101)
I/flutter (20797): #11     _WidgetsFlutterBinding&BindingBase&GestureBinding._fl
ushPointerEventQueue (package:flutter/src/gestures/binding.dart:64)
I/flutter (20797): #12     _WidgetsFlutterBinding&BindingBase&GestureBinding._ha
ndlePointerDataPacket (package:flutter/src/gestures/binding.dart:48)
I/flutter (20797): #13     _invoke1 (dart:ui/hooks.dart:134)
I/flutter (20797): #14     _dispatchPointerDataPacket (dart:ui/hooks.dart:91)
I/flutter (20797): Handler: onTap
I/flutter (20797): Recognizer:
I/flutter (20797):   TapGestureRecognizer#485eb(debugOwner: GestureDetector, sta
te: ready, won arena, finalPosition:
I/flutter (20797):   Offset(240.2, 138.4), sent tap down)
I/flutter (20797): ═════════════════════════════════════════════════════════════
═══════════════════════════════════════

Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel beta, v0.2.8, on Microsoft Windows [Version 6.1.7601], loca
le zh-TW)
[√] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[√] Android Studio (version 3.1)
[!] VS Code, 64-bit edition (version 1.21.1)
[√] Connected devices (1 available)
! Doctor found issues in 1 category.

Please add instructions how to reproduce.

Thanks, Zoechi.
I think this issue easily happened in the low-end devices or its background task was heavy.
I have a TextField with a TextEditingController and a cancel button in TextField and I attached the detail steps as following:
Step1: Type some words on TextField
Step2: Click cancel button [which will call cancel() function of TextEditingController]
Step3: Click TextField immediately before the words on TextField has not being cleared.
And the exception occurred.
I also update the flutter sdk to a newer version but this issue still happened.
[√] Flutter (Channel beta, v0.3.2, on Microsoft Windows [Version 6.1.7601], loca
le zh-TW)
[√] Android toolchain - develop for Android devices (Android SDK 27.0.3)
[√] Android Studio (version 3.1)
[!] VS Code, 64-bit edition (version 1.21.1)
[√] Connected devices (1 available)

Some code that shows how you initialize TextEditingController and TextField` would be helpful to diagnose the problem.

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.

This happens to me when I trigger an onTap event to clear a text controller, while the text field dose not have focus.
I am trying to have a way to click outside of the textfield to drop the keyboard and then have an option to clear the textfield as an after thought.
I want this because the text in the field results in showing conditional filter results in a listview, the keyboard is in the way. I want to be able to filter, select items in the results with the keyboard removed then clear the text which will clear the filter results. I hope that makes sense?
I am using 'FocusScope.of(context).requestFocus(FocusNode());' to pull fucus from the texfield and close the keyboard. calling textController.clear(); or text.controller.text = '' causes the error when the textfield does not have focus.
If I have the textfield request focus before I clear the text I am back to where I started with the keyboard open but if the user presses the clear button at this point there is little chance they would want to enter more text in the field. I want to clear the text without interacting with the keyboard or textfield in anyway.
The following works but you do see the keyboard briefly...
onTap: () {
                  filterNode.requestFocus();
                  Future.delayed(Duration(milliseconds: 50), () {
                    textController.clear();

                    FocusScope.of(context).requestFocus(FocusNode());
                  });
                },

Thank you.

@zoechi Could you reopen?
We have a simple search field like the following, and when we click the clear button, we get the above error.
class SearchField extends StatefulWidget {
  SearchField({
    Key key,
    this.onTextChanged,
  }) : super(key: key);

  final void Function(String) onTextChanged;

  @override
  _SearchFieldState createState() => _SearchFieldState();
}

class _SearchFieldState extends State<SearchField> {
  final _searchController = TextEditingController();

  @override
  void initState() {
    super.initState();
    _searchController.addListener(() => widget.onTextChanged != null ? widget.onTextChanged(_searchController.text) : null);
  }

  @override
  Widget build(BuildContext context) {
    return Row(
      children: <Widget>[
        Expanded(
          child: TextField(
            controller: _searchController,
            decoration: InputDecoration(
              contentPadding: EdgeInsets.symmetric(vertical: 20.0),
              prefixIcon: Icon(
                Icons.search,
                color: Colors.grey[400],
              ),
              hintText: 'Search',
              suffixIcon: IconButton(
                icon: Icon(Icons.clear, color: Colors.red),
                onPressed: () {
                  _searchController.text = '';
                  // _searchController.clear(); // Both approaches throw
                },
              ),
            ),
          ),
        ),
      ],
    );
  }

  @override
  void dispose() {
    _searchController.dispose();
    super.dispose();
  }
}

Can confirm the above workaround fixes this issue, but leads to unexpected consequences using the library datetime_picker_formfield which results in multiple datetime pickers opening. Although I appreciate this is likely a bug with their plugin.
[√] Flutter (Channel beta, v1.7.8+hotfix.3, on Microsoft Windows [Version 10.0.17763.615], locale en-GB)
[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[√] Android Studio (version 3.4)
[√] VS Code, 64-bit edition (version 1.35.1)
[√] Connected device (1 available)

I can confirm that the work around is solving this issue. So far ok in plain vanilla TextFields.
If anyone is interested, snippet below which will enable text clear icon only when there is a text entry. It will hide the clear icon when the text field is empty. It also hides the annoying text underline as well.
TextField(
                maxLines: 1,
                controller: searchTextController,
                decoration: InputDecoration(
                  hintText: 'Search places...',
                  hintStyle: TextStyle(color: Colors.white70),
                  suffixIcon: searchTextController.text.isNotEmpty
                      ? GestureDetector(
                          onTap: () {
                            // This is not working. Exception - invalid text selection: TextSelection(baseOffset: 2, extentOffset: 2, affinity: TextAffinity.upstream, isDirectional: false)
                            //searchTextController.clear();
                            WidgetsBinding.instance.addPostFrameCallback(
                                (_) => searchTextController.clear());
                          },
                          child: Icon(
                            Icons.clear,
                            color: Colors.white,
                          ),
                        )
                      : null,
                  border: OutlineInputBorder(
                    borderSide: BorderSide.none,
                  ),
                ),
                style: TextStyle(color: Colors.white),
                autocorrect: false,
              ),

bump this. I also met this issue after upgrading the flutter SDK to the latest.

I'm also having this issue.
Thanks for the workaround!

+1

@NickCullen The issue with datetime_picker_formfield is fixed. It had to add the workaround described here.

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.

Sloppy

EDIT - Valid workaround:
WidgetsBinding.instance.addPostFrameCallback((_) => _searchController.clear());
See below for example.
@zoechi Could you reopen?
We have a simple search field like the following, and when we click the clear button, we get the above error.
class SearchField extends StatefulWidget {
  SearchField({
    Key key,
    this.onTextChanged,
  }) : super(key: key);

  final void Function(String) onTextChanged;

  @override
  _SearchFieldState createState() => _SearchFieldState();
}

class _SearchFieldState extends State<SearchField> {
  final _searchController = TextEditingController();

  @override
  void initState() {
    super.initState();
    _searchController.addListener(() => widget.onTextChanged != null ? widget.onTextChanged(_searchController.text) : null);
  }

  @override
  Widget build(BuildContext context) {
    return Row(
      children: <Widget>[
        Expanded(
          child: TextField(
            controller: _searchController,
            decoration: InputDecoration(
              contentPadding: EdgeInsets.symmetric(vertical: 20.0),
              prefixIcon: Icon(
                Icons.search,
                color: Colors.grey[400],
              ),
              hintText: 'Search',
              suffixIcon: IconButton(
                icon: Icon(Icons.clear, color: Colors.red),
                onPressed: () {
                  // _searchController.text = '';
                  // _searchController.clear(); // Both approaches throw
                  // Workaround
                  WidgetsBinding.instance.addPostFrameCallback((_) => _searchController.clear());
                },
              ),
            ),
          ),
        ),
      ],
    );
  }

  @override
  void dispose() {
    _searchController.dispose();
    super.dispose();
  }
}

flutter doctor
[✓] Flutter (Channel stable, v1.7.8+hotfix.3, on Mac OS X 10.13.6 17G7024, locale en-DK)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 10.1)
[✓] iOS tools - develop for iOS devices
[✓] Android Studio (version 3.4)
[✓] VS Code (version 1.36.1)
[✓] Connected device (1 available)
• No issues found!

This worked fine for me

This issue is still prevalent  and the above reproduction should still work.

solution:
() async {
await Future.delayed(Duration(seconds: 0));
_controller.clear();
},


Manually typed date and min/max dates

I have found an issue with saving of a manually typed date value.
Set minDate $scope.datePickerOptions = { minDate: new Date() };
Type the date in past (for example: 27/10/2015). The input value is reset to previous value, but in ngModel remain invalid value.
So I think need add handler for dp.error event:
                    if (ngModelCtrl) {
                        $timeout(function () {
                            ngModelCtrl.$setViewValue(e.target.value);
                        });
                    }
                }).datetimepicker(options);```
What do you think?

Should set 'useStrict' to true in option first, otherwise dp.error won't be fired


Dependency updates, message cleanup, and expanding commands

Upgrade Gradle
Update dependencies to latest versions so it works on Spigot & Paper 1.19.4
Cleanup and normalize all messages to use MiniMessage formatting

Get rid of translateAlternateColorCodes
Add more (most?) messages to messages.yml

Remove sponge support (testing is out of scope for me)
Add new commands & subcommands so parties & elections can be managed via "/" commands

Hello and thanks for making a pull request. I love when someone from the community decided to contribute to my projects and this is why I always make my projects opensource.
The code overall looks really good, I’ll test it as soon as I’ll get home. In the meantime are you able to fix the issues I asked in the review above?

Hello and thanks for making a pull request. I love when someone from the community decided to contribute to my projects and this is why I always make my projects opensource.

Cool!  I needed this for some voting one of our customers wants to do for an building competition they're having, but it didn't work.  I'm more than happy to contribute the updates / new features back for the community.

The code overall looks really good, I’ll test it as soon as I’ll get home. In the meantime are you able to fix the issues I asked in the review above?

Yep, definitely.  I'm also not sure if some of this (in particular the cmdline arg processing to handle spaces, and how to open the elections gui when a player hits a block) since I'm neither experienced with Java nor Bukkit plugin development.

Hi Lorenzo, following up on whether you plan to merge this work or if you need to see other changes?
Additionally, on my mcplaydates-1.0 branch, I've added several new features:

Soft Integration with Holographic Displays so to display up-to-date Elections status in holograms
Soft Integration with GriefPrevention to enable ability to create parties and add them to open elections to vote on when GP claims are transferred to players (I use it for ease of administration of voting on building competitions)
ability to create "voting blocks" that when hit open the voting menu.

Along the way I found several challenges with the asynchronous nature of the way the cache and database worked, so I've also refactored all of the cache and database logic.  All queries now come directly from the cache, which has been updated with a "write-through" caching model.  The cache is now responsible for persisting dirty entries.
LMK if you'd like any of these changes contributed back, otherwise I'm happy to continue maintaining my fork.

Hello, I'm sorry for the delay. Are you still interested in this? @tadhunt

Hi @Lorenzo0111, I'd love to be able to roll our changes back into the project so we don't need to maintain a separate fork, but I don't have time in the forseeable future for any development work, so not sure when I could get back to it.

Closes #125

#126


Consumers getting stuck with "Reached max fetcher queue size (10), sleeping 1s" warning

Version of Ruby: 2.6
Version of Kafka: 1.1
Version of ruby-kafka: 0.7.10

I have a problem for large topics where after a while, consumers (from Karafka gem 1.3.0) get stuck with "Reached max fetcher queue size (10), sleeping 1s" warning. And this is quite random depending on the application (or rather how topics/partitions are assigned to consumers). So some workers don't have any problems at all and some of them are stuck, sometimes all of them are stuck and scaling up/down workers doesn't help.
What would be the way to unstuck the consumers and why does it happen? The topics are indeed large but after a day or two, the consumers would be able to catch up with the offset lag and get to the beginning of the topic so it's not the case that more messages are published that the consumers would be ever able to consume. Would changing a config for max fetcher queue size help and if yes, what would be the recommended value? And is it actually a feature or a bug that the consumers can get stuck with just a warning and not an exception for example?
If it somehow helps, the topics are log-compacted, there are 5 topics and each of them has 10 partitions. We tried various numbers of workers between 2 and 30.

I tried different values for max fetcher queue size 2 and 100 and seems like it's the same case for all of them.

So, the "Reached max fetcher queue size" is nothing abnormal if you have a topic with a huge lag.
Also, please define what "for large topics" mean.Huge messages or huge amount of messages.
Keep in mind, that if you subscribe to a topic that has a lot of messages (and new being produced) and you end up consuming slower than you produce, weird things may happen if you hit the retency policy application.
Also: can you provide logs? I mean debug logs (you can set both ruby-kafka and karafka at the same time).

Would changing a config for max fetcher queue size help and if yes, what would be the recommended value?

Recommended here depends on size of your machine and it's power. It depends.

is it actually a feature or a bug that the consumers can get stuck with just a warning and not an exception for example?

We don't know yet whether it is a bug or a side-effect of you doing something wrong ;)

@mensfeld Thanks for the response.
The retention policy is set to millions of years, so that shouldn't cause any issue, despite the topic being huge in terms of number of messages. Seems like one of the consumer group has caught up eventually with all the messages, so maybe "getting stuck" was just temporary. I'll also try to use Sidekiq backend for Karafka as this will surely help consume faster.
If that won't help, I'll provide some logs, don't want to make too much noise here if there are still some options to try.

This issue can be closed, I've managed to solve the problem. Apparently, it was fully on the application's side and literally it was about missing indexes in one app that were present in the other one and that was the source of the major difference in behavior between the apps 🤦♂ One topic had some heavy queries performed from time to time and this is when the processing was really slow (3-4s per message or even more) + it was causing a massive DB load due to sequential parallel scans which was also causing other topics to be slow. So in the end, consumers were getting kicked out from Kafka all the time (I guess after 30s which was the limit from the config) and that's why they appeared to be stuck.
Thanks for help, and hope it helps someone else when they encounter this kind of problem, I also found that decreasing max fetcher queue size helps a bit with getting unstuck, but certainly, that is not a solution here :P.


Error with sentiment_fts table

I found a error during running twitter_stream.py and dev_server.py like this:
pandas.io.sql.DatabaseError: Execution failed on sql 'SELECT sentiment.* FROM sentiment_fts fts LEFT JOIN sentiment ON fts.rowid = sentiment.id WHERE fts.sentiment_fts MATCH ? ORDER BY fts.rowid DESC LIMIT 10': no such table: sentiment_fts
I supposed the cause of this error is in twitter_stream.py when it was creating sentiment_fts table, but I cannot find anything wrong with it. Could anyone please help me with it?
fts5 was already enabled.

What operating system are you using?
You need newest Python 3.6 on Windows, and for Linux you have to update sqlite module (there is info about that in readme)

I hade the same error running on a Mac with Python 3.8.
The solution was to add the current path to the code.
`import os
import sys
set chdir to current dir
sys.path.insert(0, os.path.realpath(os.path.dirname(file)))
os.chdir(os.path.realpath(os.path.dirname(file)))`


Use of CPL1.0 license in ModelicaStrings.c without proper declaration

Reported by andreas.hofmann7 at 2017-05-09T09:12:34+01:00
Hello everyone,
I currently discovered, figuring out what open source software is used within the Modelica C-Sources (for contribution of FMUs to customers), that there is a cross reference to the CPL1.0 license within "ModelicaStrings.c" starting at line 532.
To my understanding the link to the CPL1.0 license here is not sufficient, since the license (https://opensource.org/licenses/cpl1.0.php) states that:
"When the Program is made available in source code form:

    a) it must be made available under this Agreement; and

    b) a copy of this Agreement must be included with each copy of the Program."

Am I wrong here, when I think there should be a license file available in the C-Sources directory with the CPL1.0 license or that the CPL1.0 license should be available in the source code?
About using a compiled version of the ModelicaString.c I am uncertain, since I do not know if the BSD that is used for the "ModelicaStrings.c" complies with the CPL1.0.
Maybe using a newer version of the General Purpose Hash Function Algorithms Library can help here, since it is licensed under MIT license.
Furthermore, I think it would be a good idea to discuss the topic of open source software with the tool vendors, since using any file from the Modelica/Resources/C-Sources directory within a FMU or a compiled executable, requires to provide a license to the individual file (they have a wide range of copyright holders) as the 2-Clause BSD license requires this (https://opensource.org/licenses/BSD-2-Clause):
"2. Redistributions in binary form must reproduce the above copyright notice, this list of conditions and the following disclaimer in the documentation and/or other materials provided with the distribution."

Best regards,
Andreas

Migrated-From: https://trac.modelica.org/Modelica/ticket/2188

CPL1.0 code was added in 61763832d4c6ce46403919c11a5cd5ff396d6735 for inclusion of Noise/ModelicaRandom.c in MSL v3.2.2 (#1662).

@MartinOtter I have two alternative proposals

Since Modelica.Utilities.Strings.hashString is only referenced from ModelicaTest, but not the MSL itself, it can be removed. We would lose backward-compatibility though).

Since uthash (BSD-style and already part of the MSL) also contains have a dozen hash algorithms I propose to use one of them. Should be easy to fix then.

Which one do you prefer?

I am a bit confused:
In File ModelicaStrings.c line 489 a reference to the Common Public License is given. However, the URL of the author points to http://www.partow.net/programming/hashfunctions/index.html and here it is stated:
General Hash Function License
Free use of the General Hash Functions Algorithm Library available on this site is permitted under the guidelines and in accordance with the MIT License.
I also searched on this web site, but did not find the word "Common Public License".
So, it seems a simple solution is to just remove the wrong statement to the Common Public License in the ModelicaStrings.c file and replace it with the MIT License. The advantage is that no user program is influenced.
If there are good technical reasons to use the newly proposed hash function, then it seems fine to use the new proposal. However, it the only reason is the license, then it seems better to just replace the wrong license statement by the MIT license.

Confusing, indeed: I downloaded GeneralHashFunctions_-_C.zip at Oct. 22 2015 and it said CPL. I downloaded it again today and it now says MIT, though the file date of GeneralHashFunctions.c was kept at Dec. 12 1999.
Thus, basically this is a non-issue, I will take care of by removing the confusing license comment.

Closed via #2262.


Fix next.js getting started doc

fix https://github.com/shadcn-ui/ui/issues/545

@timqian I just merged a fix for this in #1062. Sorry I did not see your PR first. Thanks for helping. Appreciate it!


webview_flutter not load this page

webview_flutter not load this https://www.xiachufang.com/juno/ec-tab/tabs page。
When I use webview_flutter to load this page, it cannot be loaded normally and there is no error message in the console.
The info.plist configuration is as follows：
<dict>
          <key>NSAllowsArbitraryLoads</key>
          <true/>
</dict>
<key>io.flutter.embedded_views_preview</key>
<true/>

But I use iOS native WKWebView and it can be loaded normally。

Hi @wj610671226,
I tried the official webview_flutter example on latest stable (1.20.3) on iPhone 6s and able to see the link load.

Can you try the same and see if it works ?
If issue persists, please provide flutter doctor -v and minimal complete reproducible code sample.
thanks.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


bug: ion-list in modal view height is wrong

Type: bug
Platform: ios 8 webview
I have ion-list that I want it to cover partial amount of modal view which I have set it like below.

but it would not let me and ion-list will just go over bottom part of modal view.

Greetings @fkkcloud!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Hey @fkkcloud, videos can sometimes have problematic layering issues. Could you provide an example we can look at?

Greetings @fkkcloud!
My sensors indicate a reply was requested, but as we have not received one yet, I am using my robot powers to close this issue. However, if you are still experiencing this issue, please feel free to reopen it by creating a new issue, and include any examples or necessary information.
Thank you for allowing me to assist you.


[ALLUXIO-2851] Fix null pointer exception in master/info REST endpoint

https://alluxio.atlassian.net/browse/ALLUXIO-2851

Reopened in https://github.com/Alluxio/alluxio/pull/5633

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/15769/
Test PASSed.


[llvm-context, solidity] Add CLI option '-g' to generate debug inform…

…ation.
Add command line option '-g' to generate source level debug information in the output code. This only works with the LLVM-IR code generator.
Add flag 'emit_debug_info' to the llvm-context optimizer/code-gen settings structure, to record the setting of the '-g' CLI option.
Generate source level debug information for the functions defined when YUL is lowered to LLVM-IR. This includes the deploy_code and runtime functions generated by the compiler.
Generate debug-location information for other constructs that may appear in a contract.

Add debug-info to the immutables-load function.

Reworked 'set_debug_location' to make it easier to set debug locations.


Reverb effect on pre-trained model

Hi
I am successfully generating audio with the pre-trained model, but the output has a reverb quality. Are there any recommended steps for removing this effect from audio generated with the pre-trained model?
Ideally, I would like to produce audio comparable to the samples on the WaveGlow demo page: https://nv-adlr.github.io/WaveGlow

We're currently investigating improvements to this: stay tuned!

Any updates?

Hi
I am successfully generating audio with the pre-trained model, but the output has a reverb quality. Are there any recommended steps for removing this effect from audio generated with the pre-trained model?
Ideally, I would like to produce audio comparable to the samples on the WaveGlow demo page: https://nv-adlr.github.io/WaveGlow

have you found why provided pretrained model generating reverb quality. It sounds like a ring sound in the background right?

Reverb is significantly reduced on the new published model. Give it a try!

Reverb is significantly reduced on the new published model. Give it a try!

Thanks for sharing such a great work. I would like to know what the main difference is between these two models? Is there any modification on model architecture or training procedures?

No modification, we just trained for longer.
Closing due to inactivity.


Custom logs not watched on local/dev

From @mbrodala on December 15, 2016 9:40
Here's an excerpt from my boxfile.yml:
# ...
web.site:
  # ...

  log_watch:
    apache[access]: /data/var/log/apache/access.log
    apache[error]: /data/var/log/apache/error.log
    php[error]: /data/var/log/php/php_error.log
    php[fpm]: /data/var/log/php/php_fpm.log
    typo3[typo3]: var/log/typo3.log
    # ...

However, only the apache and php logs are watched and outputted on nanbox run php-server, the typo3 log is ignored. Is there anything I missed?
Copied from original issue: nanobox-io/nanobox#299

From @mbrodala on December 15, 2016 9:42
I can see the following after nanobox run:
/app $ ps aux | grep tail
gonano     136  0.0  0.0   6044   716 ?        S+   09:36   0:00 tail -f -n0 /data/var/log/php/php_error.log /data/var/log/php/php_fpm.log /data/var/log/apache/access.log /data/var/log/apache/error.log

So it looks like my custom log was indeed ignored.

From @danhunsaker on December 15, 2016 11:48
I'd enjoy that as well.  I'll have a peek at the engine repo and see if I can tweak the script any.  The big trouble will be parsing the Boxfile - that isn't generally done outside the build/compile phases.

From @mbrodala on December 15, 2016 11:37
I guess this makes sense; even php-server is invoked manually and no network_dirs are set up (not neccessary).
But still I can see log output on nanobox run php-server; would it be possible to include custom logs here? Just as minor feature request. Of course I can open/tail the app logs on my host system if not. ;-)

From @danhunsaker on December 15, 2016 11:27
This is actually because the web instance isn't being created in the first place - dev mode doesn't spin up separate containers for webs and workers, instead spinning up one container for each app's console.  So any settings in the web.* or worker.* sections aren't applied in development mode; they'll only kick in when deployed to dry-run or production.

From @danhunsaker on December 15, 2016 11:57
Ah.  Well.  php-server is generated from a template.  While the Boxfile is accessible.  That shouldn't be too tricky to work with to give this additional functionality, though I'll probably have to look at it tomorrow or so.


refactor: remove -d for ros2 caret command

Description
remove -d for ros2 caret command
Related links
Jira: https://tier4.atlassian.net/browse/RT2-975

Notes for reviewers

Pre-review checklist for the PR author

[x] I've confirmed the contribution guidelines.
[x] The PR is ready for review.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[x] The PR has been properly tested.
[x] The PR has been reviewed.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[x] There are no open discussions or they are tracked via tickets.
[x] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

But -f option in create_architecture_file should not be removed because it is used for overwriting the output file as optional action.

Oops, it's my bad. I fixed my original comment.

According to the original discussion, any unnecessary -? should be removed. So, I think -w in check_caret_rclcpp ~and -f in create_architecture_file~ should be also removed.
@nabetetsu Could you confirm if we remove them or not?

fixed in https://github.com/tier4/ros2caret/pull/129/commits/bf20e5e38d263b52ab91e37ce5dbe736d68f9506


Ship wasi-sdk.cmake as part of the SDK

Fixes #56

For the debian pacakge maybe we could do that, but for the tar archive we don't have any way to install stuff outside of whereever the developer wants to extract the archive.
I guess we could put in in <archive_root>/share/cmake?   Would that be preferable?  I'm not sure.  Its not going to be found by cmake automatically either way.

I'd rather it be in <archive root>/share/cmake in the tar. I'm happy with changing it to land inside /usr/share/cmake/ in the deb if that gets it found automatically.

Done


Remove unnecessary .container CSS rules

This gets rid of unwanted vertical scrollbar on the home page.

Before:

After:

eh, textarea now isn't taking full height, gotta fix that.

Current HEAD seems to work properly.

Thanks :+1:


EVA-866 Replace Clinvar Java binding classes if they exist

Delete previous version of the Java XML binding classes when generating them for a new Clinvar version

Coverage remained the same at 75.037% when pulling 9c4e79ed40dd1e0639bd8bb4e8d3b5672296ce0e on pabarcgar:updateOldJavaClinvarParser into 5db69b470e5f8c553859206d67cf2be2efee563b on EBIvariation:master.


Feature Request: pass Svelte Component inside InterActiveTab component body

Thank you for sharing the library, just wanted to ask if it this could be added.
I had to base it on the included interactive tab, not really sure if someone else might need it, its not a very pretty implementation but it works.

    {#each tabs as { label, content, id }}
        {#if activeTabValue === id}

                {#if typeof content === 'string'}

                        {content}

                {:else}
                    <svelte:component this={content} />
                {/if}

        {/if}
    {/each}

Can I ask what kind of svelte component you are planning use? If you can give me an example, it will be helpful.

I'm nesting this inside a modal, as a sort of per table item menu.
i.e.
Table:

Rows
(column) Edit Button Modal
XL Modal
Interactive Tab as a tabbed menu of sorts
Individual Svelte Component Forms that hold menu items

I just modified it for the svelte components.
You can see an example here.
Please update npm:
npm i -D flowbite-svelte@latest

Also please change your key name from title to name. After adding TS, I changed some of key names. Sorry.
interface InteractiveTabType {
    name: string;
    id: number;
    content: string | typeof SvelteComponent;
}

Thank you for the quick update, will update versions now.

In the future version, the Interactive tab component change the structure. It will use slot so you can add another component or HTML etc. So it will be a BREAKING CHANGE.
The reason behind this change is to make the component more flexible. By using the slot, uses can add anything in the slot.

I leave the InteractiveTabs component as it is. And I just added InteractiveTabHead and TabContent components. It makes easy to add anything in the tab.
Demo


adding slack and pushbullet

I added slack and pushbullet

Thoughts on the code?

Hey, sry for not responding sooner – I didn't yet find time to go over the whole PR.
At a quick glance there seem to be some issues with it:

How does this handle the fact that existing config files don't have a pushbullet and slack config?
The comments in the config file don't seem correct
What does the title config in the slack section do?
Why did you rename the default config file name?
Won't this crash for everybody who doesn't have the slackclient python package installed?


Handle dictionaries in zstd

This PR modifies all CLI tools to handle zstd compressed dictionaries.
Note 1
Although ruzstd was used in dictgen, this modification unifies the dependency to zstd throughout this repository because confusion can arise when different libraries with similar functions are used. Pure Rust would be preferable, but the advantages of unifying into one dependency would be greater.
Note 2
This PR adds shell hints to code blocks to highlight comment-outs.

@vbkaisetsu The reason why using shell syntax is to highlight comments in code blocks. So, I moved comments outside and reverted them. https://github.com/daac-tools/vibrato/pull/112/commits/48aefebcdd8a70901bd435d296569faabd77a4db


Ensure zxcvbn is loaded before using it

We had some issues where fast filling of the input (aka autofill) was triggering and undefined is not a function error on the usage of zxcvbn. This should avoid this.

Looks good thanks.  Will merge along with a couple other updates tonight/tomorrow.

I added the check manually while fixing a couple other things before compiling, so it should be good to go.  Thanks again.

Will bump version in npm tomorrow.

npm bumped.

Thanks


Add _msgSender() trick

Fixes #132 (If you'd want it!)
A more full description of the motivation is on that issue. Here is a quick pass achieving that proposal.
I did not change msg.sender instances in constructors, since the benefit would never apply in those scenarios.
Enables MetaTransaction facets and Delegatable.

Hi, thanks for the review!

Same concept as GSN in the OZ contracts, correct?

That's right. They should be completely compatible. I'll verify as much and get back here. And there are a few implementations going around that should all be compatible, so we can probably choose the most readable one. I'll gather the options.

Can you describe these steps in more detail?

I stole this code from someone who understood it better, but it is almost identical to GSN's version (formatted differently). Their comments are this:
    function _getRelayedCallSender() private pure returns (address payable result) {
        // We need to read 20 bytes (an address) located at array index msg.data.length - 20. In memory, the array
        // is prefixed with a 32-byte length value, so we first add 32 to get the memory read index. However, doing
        // so would leave the address in the upper 20 bytes of the 32-byte word, which is inconvenient and would
        // require bit shifting. We therefore subtract 12 from the read index so the address lands on the lower 20
        // bytes. This can always be done due to the 32-byte prefix.

        // The final memory read index is msg.data.length - 20 + 32 - 12 = msg.data.length. Using inline assembly is the
        // easiest/most-efficient way to perform this operation.

        // These fields are not accessible from assembly
        bytes memory array = msg.data;
        uint256 index = msg.data.length;

        // solhint-disable-next-line no-inline-assembly
        assembly {
            // Load the 32 bytes word from memory with the address on the lower 20 bytes, and mask those.
            result := and(mload(add(array, index)), 0xffffffffffffffffffffffffffffffffffffffff)
        }
        return result;
    }

I'll add to the remaining files, while moving the implementation to a util soon, thank you! Feel free to close until I make those changes if you'd like to keep the PR queue neat.

Ok, I've made a few changes:

[x] Moved the _msgSender() implementations into a single util file.
[x] Copied the better-documented version of the _msgSender() that I linked to above.

Before merge, needs to be applied to a few more files: SafeOwnableInternal, ERC20ExtendedInternal, ERC20ImplicitApprovalInternal, ERC4626BaseInternal

SafeOwnableInternal imports OwnableInternal, which now includes the change, so it is inherited.
ERC20ExtendedInternal imports ERC20BaseInternal, which now includes the change, so it is inherited.
ERC20ImplicitApprovalInternal imports ERC20BaseInternal, which now includes the change, so it is inherited.
ERC4626BaseInternal imports ERC20BaseInternal, which now includes the change, so it is inherited.

If I were to block merge myself, I might suggest I add at least one test first to demonstrate reasonable usage of this method.

This is similar to OZ's Context.sol but they don't implement _getRelayedCallSender by default since not all contracts would require meta tx. One can override the default with their desired metatx standard (e.g. ERC2771Context or _getRelayedCallSender)
Have not tested this but this could affect existing contracts using multicall function since it would change the sender.

Oh that's interesting, does the multicall functinality rely on similarly appending to tx.data?

I'm looking for cases where msg.sender == address(this) might return true and return an unexpected sender.
If a contract calls itself with address(this).call without appending sender,  _msgSender() will return an invalid sender address.
In the case of Multicall, there won't be an issue since it uses delegatecall (msg.sender doesn't change to the contract address).


Decide on cluster tool for managing container scaling

[ ] choose provider
[ ] choose how to manage our containers (some suggestions are Apache Tomcat and Kubernetes, dockerswarm)

kubernetes will be default unless strong opinions are found in RG etc

Seems like there is a rough consensus that kubernetes is harder than it needs to be. Looking into fly.io as a possible alternate.

Feedback from RaidGuild was that Kybernetes is hard core an alternative could be fly.io (USD 199/ month) + usage. Needs to be compared to AWS cost.

look into fly.io a little bit more, as long as usage is steady should be a viable contender

We decided that we will go with fly.io, setting up the following regions Singapore, Netherlands, Oregon.


Infinite recursion due to lib/modules.nix changes

I have been attempting to build a system using the git repository, and was successful earlier, but had not pulled down the latest changes. After pulling down changes today I'm getting an error with infinite recursion. It seems to be coming from the lib/modules.nix. Below is the error that I was seeing.
Running a git bisect between the version that worked for me 14ea4e7e7 and the current master, 1035b7b6c9, I ended up at 1d62ad474 as the probable culprit.
# nixos-rebuild build --show-trace -I nixpkgs=/root/nixpkgs
building Nix...
error: while evaluating the attribute ‘config’ at "/root/nixpkgs/nixos/lib/eval-config.nix":26:5:
while evaluating anonymous function at "/root/nixpkgs/pkgs/top-level/all-packages.nix":8:1, called from "/root/nixpkgs/nixos/lib/eval-config.nix":55:10:
while evaluating ‘applyGlobalOverrides’ at "/root/nixpkgs/pkgs/top-level/all-packages.nix":107:26, called from "/root/nixpkgs/pkgs/top-level/all-packages.nix":101:10:
while evaluating ‘pkgsFun’ at "/root/nixpkgs/pkgs/top-level/all-packages.nix":125:19, called from "/root/nixpkgs/pkgs/top-level/all-packages.nix":120:14:
while evaluating the attribute ‘config.nixpkgs’ at "/root/nixpkgs/nixos/lib/eval-config.nix":26:5:
while evaluating the attribute ‘config’ at "/root/nixpkgs/lib/modules.nix":56:25:
while evaluating ‘yieldConfig’ at "/root/nixpkgs/lib/modules.nix":43:29, called from "/root/nixpkgs/lib/modules.nix":42:16:
while evaluating ‘mapAttrs’ at "/root/nixpkgs/lib/attrsets.nix":134:17, called from "/root/nixpkgs/lib/modules.nix":44:32:
while evaluating ‘mergeModules’ at "/root/nixpkgs/lib/modules.nix":111:26, called from "/root/nixpkgs/lib/modules.nix":38:17:
while evaluating ‘mergeModules'’ at "/root/nixpkgs/lib/modules.nix":115:36, called from "/root/nixpkgs/lib/modules.nix":112:5:
while evaluating ‘concatMap’ at "/root/nixpkgs/lib/lists.nix":54:18, called from "/root/nixpkgs/lib/modules.nix":151:9:
while evaluating ‘fold’ at "/root/nixpkgs/lib/lists.nix":20:19, called from "/root/nixpkgs/lib/modules.nix":38:38:
while evaluating ‘fold'’ at "/root/nixpkgs/lib/lists.nix":23:15, called from "/root/nixpkgs/lib/lists.nix":27:8:
while evaluating ‘closeModules’ at "/root/nixpkgs/lib/modules.nix":60:27, called from "/root/nixpkgs/lib/modules.nix":34:16:
while evaluating anonymous function at "/root/nixpkgs/lib/modules.nix":62:49, called from "/root/nixpkgs/lib/lists.nix":49:19:
infinite recursion encountered
error: while evaluating the attribute ‘pkgs’ at "/root/nixpkgs/nixos/lib/eval-config.nix":52:3:
while evaluating anonymous function at "/root/nixpkgs/pkgs/top-level/all-packages.nix":8:1, called from "/root/nixpkgs/nixos/lib/eval-config.nix":55:10:
while evaluating ‘applyGlobalOverrides’ at "/root/nixpkgs/pkgs/top-level/all-packages.nix":107:26, called from "/root/nixpkgs/pkgs/top-level/all-packages.nix":101:10:
while evaluating ‘pkgsFun’ at "/root/nixpkgs/pkgs/top-level/all-packages.nix":125:19, called from "/root/nixpkgs/pkgs/top-level/all-packages.nix":120:14:
while evaluating the attribute ‘config.nixpkgs’ at "/root/nixpkgs/nixos/lib/eval-config.nix":26:5:
while evaluating the attribute ‘config’ at "/root/nixpkgs/lib/modules.nix":56:25:
while evaluating ‘yieldConfig’ at "/root/nixpkgs/lib/modules.nix":43:29, called from "/root/nixpkgs/lib/modules.nix":42:16:
while evaluating ‘mapAttrs’ at "/root/nixpkgs/lib/attrsets.nix":134:17, called from "/root/nixpkgs/lib/modules.nix":44:32:
while evaluating ‘mergeModules’ at "/root/nixpkgs/lib/modules.nix":111:26, called from "/root/nixpkgs/lib/modules.nix":38:17:
while evaluating ‘mergeModules'’ at "/root/nixpkgs/lib/modules.nix":115:36, called from "/root/nixpkgs/lib/modules.nix":112:5:
while evaluating ‘concatMap’ at "/root/nixpkgs/lib/lists.nix":54:18, called from "/root/nixpkgs/lib/modules.nix":151:9:
while evaluating ‘fold’ at "/root/nixpkgs/lib/lists.nix":20:19, called from "/root/nixpkgs/lib/modules.nix":38:38:
while evaluating ‘fold'’ at "/root/nixpkgs/lib/lists.nix":23:15, called from "/root/nixpkgs/lib/lists.nix":27:8:
while evaluating ‘closeModules’ at "/root/nixpkgs/lib/modules.nix":60:27, called from "/root/nixpkgs/lib/modules.nix":34:16:
while evaluating anonymous function at "/root/nixpkgs/lib/modules.nix":62:49, called from "/root/nixpkgs/lib/lists.nix":49:19:
infinite recursion encountered
building the system configuration...
error: while evaluating the attribute ‘config.system.build.toplevel’ at "/root/nixpkgs/nixos/lib/eval-config.nix":26:5:
while evaluating anonymous function at "/root/nixpkgs/pkgs/top-level/all-packages.nix":8:1, called from "/root/nixpkgs/nixos/lib/eval-config.nix":55:10:
while evaluating ‘applyGlobalOverrides’ at "/root/nixpkgs/pkgs/top-level/all-packages.nix":107:26, called from "/root/nixpkgs/pkgs/top-level/all-packages.nix":101:10:
while evaluating ‘pkgsFun’ at "/root/nixpkgs/pkgs/top-level/all-packages.nix":125:19, called from "/root/nixpkgs/pkgs/top-level/all-packages.nix":120:14:
while evaluating the attribute ‘config.nixpkgs’ at "/root/nixpkgs/nixos/lib/eval-config.nix":26:5:
while evaluating the attribute ‘config’ at "/root/nixpkgs/lib/modules.nix":56:25:
while evaluating ‘yieldConfig’ at "/root/nixpkgs/lib/modules.nix":43:29, called from "/root/nixpkgs/lib/modules.nix":42:16:
while evaluating ‘mapAttrs’ at "/root/nixpkgs/lib/attrsets.nix":134:17, called from "/root/nixpkgs/lib/modules.nix":44:32:
while evaluating ‘mergeModules’ at "/root/nixpkgs/lib/modules.nix":111:26, called from "/root/nixpkgs/lib/modules.nix":38:17:
while evaluating ‘mergeModules'’ at "/root/nixpkgs/lib/modules.nix":115:36, called from "/root/nixpkgs/lib/modules.nix":112:5:
while evaluating ‘concatMap’ at "/root/nixpkgs/lib/lists.nix":54:18, called from "/root/nixpkgs/lib/modules.nix":151:9:
while evaluating ‘fold’ at "/root/nixpkgs/lib/lists.nix":20:19, called from "/root/nixpkgs/lib/modules.nix":38:38:
while evaluating ‘fold'’ at "/root/nixpkgs/lib/lists.nix":23:15, called from "/root/nixpkgs/lib/lists.nix":27:8:
while evaluating ‘closeModules’ at "/root/nixpkgs/lib/modules.nix":60:27, called from "/root/nixpkgs/lib/modules.nix":34:16:
while evaluating anonymous function at "/root/nixpkgs/lib/modules.nix":62:49, called from "/root/nixpkgs/lib/lists.nix":49:19:
infinite recursion encountered

cc @nbp

@KaiSforza Have you read about the non-backwards compatible changes in #6794 (in the top post)?
I was bit by the same issue but I solved it after I applied the fixes mentioned there (specifically, the fixes mentioned in the section mkOption, mkIf).

I had not really seen that. Will test and see what happens. It seems like it has to do with the amazon ec2 configurations that are included. I'll be looking around there for a bit now.

I have checked that this works in a VM that is not using the amazon linux configurations. Looks like that will need to be updated before this stuff will build.
Moving things, removing the modulePath parts and changing some other stuff gets it to work just fine. Closing this.

@KaiSforza , Thanks for reporting this issue.
The amazon issue should be fix by commit 9647a72.


Can Tween have a const constructor?

Animatable<> has a const constructor, so can we do the same for Tween?

Tween isn't immutable -- you can dynamically change the begin and end points.

Yeah, there's a trade-off: we can give you a const constructor but then we need to make Tweens immutable.  We can look through the code we have to see how it uses mutability.  I think we use it quite a bit to reconfigure Animation objects that we've already given to someone else.

Yeah, I was just looking. There aren't that many callers, but e.g. timePicker creates a tween, uses it to make an Animation, and then later tweaks it.

We could have a ConstTween class that implements Animatable and isn't mutable.

Thanks for the background - I wasn't aware of that use case. Sounds like the cognitive complexity to make the change work isn't worth the gain.

If we want it, it's not particularly hard: https://github.com/flutter/flutter/pull/1427 (ConstTween)
I could also look at changing the callers so that they don't mutate Tween. There aren't many.

I retract my request. In my newer code, most of my Tweens end up being mutated.


Overlapping

A button to hide the ck ui.
That's it.

Only for 1.4 of course.

I don't really plan on making any further releases for 1.4 except for super critical bug fixes.


'Add single comment' and 'Start review' 404 error

Gitea version (or commit ref):
1.12.5

Git version:
2.28.0.windows.1

Operating system:

Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
When reviewing a PR, both the 'Add single comment' and 'Start review' buttons ask me to reauthenticate and subsequently show a 404 error.  The URL format it redirects to is: domain/ORG/REPO/pulls/3/files#ZgotmplZ/files/reviews/comments. I can't find relevant logs with level set to trace.
Might be unrelated; but after writing a comment, the Preview tab shows up as empty.
...
Screenshots

My ROOT_URL was missing a trailing forward slash.
Was:                http://[domain]:3000
Had to be:       http://[domain]:3000/


[FEATURE] Notifications

Would be nice if at any new walllpaper will appear a notification popup with all the main info (name, resolution, discussion link) and then fade out (like the old reddit wallpaper changer style, not the Windows standard notification)

This is actually a very good idea, but why not windows standard notifications? It would ease some computation

This is actually a very good idea, but why not windows standard notifications? It would ease some computation

Well, it's just personal taste, I don't really like Windows notifications. But if it's easier on your work, it's fine. Maybe later in the future you can develop a nicer notification system and allow users to choose between the two

This is actually a very good idea, but why not windows standard notifications? It would ease some computation

Well, it's just personal taste, I don't really like Windows notifications. But if it's easier on your work, it's fine. Maybe later in the future you can develop a nicer notification system and allow users to choose between the two

This feature has been implemented and will be inthe next release.
I did implement it with custum notifications (It turns out it's actually easier!), and I don't know where else I should use this notification system, or what to do when the notification is clicked.
At the current stage a notification is sent each time the wallpaper changes (can be turned off in settings), should I use this also for communicating a change failure?
Should I add an action when the notification is clicked? Like opening the post on Reddit? Current behaviour is that when clicked the notification goes away, and I honestly kinda like this behaviour because it looks less invasive

On click: maybe you can add in the settings the expected behaviour (like a dropdown menu that allows the user to dismiss it or opening the reddit page)
Adding a notification for changing failure may be good, but not the cleanest way I suppose. It actually doesn't matter to me and I don't know if and how you may ask the userbase about that

I will wait more feedbacks on this question, until then, since the current release supports this feature I'm gonna close this Issue


Leverage CodeBeat for automated code reviews

CodeBeat provides a free automated review of code in a wide variety of languages, including JavaScript, Swift, Go, TypeScript Java, and Kotlin.  It is easy to use - just create a CodeBeat account and add your repository and branch.  CodeBeat will then analyze the code on every checkin.
You can be notified in several ways:

email
directly into Slack
none at all (need to navigate to their site and log into your account)

Alexandre has been using it for his Swift work and has the results sent to his Slack channel.  I use it myself for my "hobby" Javascript work (rkwright@github) and find it very useful.  It is not a substitute for real code reviews, but it is very useful nonetheless.
Having it go directly into Slack for all of our projects might be a bit much, but might be useful.
I'll put an agenda item in next week's R2 meeting about this issue and we can discuss it.

I'm not sure what advantages this would have over lint tools like SwiftLint, TSLint, etc. Those can usually be run from the command line or integrated into IDEs, so you can see things right away as you develop, instead of having to send your code to an external service to analyze. They're also more configurable, and can be used with something like TravisCI, so you can see the results on github linked from your commit. That can send email notifications and I'm sure it can be integrated into slack as well. You can have it run automatically on pull requests, which is nice if you want to enforce style rules before merging.
I guess CodeBeat might be easier to set up, and might be useful if you want a general sense of the quality of your code and don't care about configuring or enforcing specific rules.

"code linting" is a useful indicator of code quality indeed, and an important step in the build workflow. The project setup of r2-streamer-js ; just like other R2 projects, I imagine ; enforces pretty strict code styling rules (not just aspects like indentation / formatting, but also "smarter" TypeScript programming conventions). A linting failure is a build failure, which TravisCI catches (resulting in developer/team notifications). This would be configured as a Git commit hook (precondition to code pushes), but the existing workflow is a sufficient deterrent to bad commits :)
I am not familiar with CodeBeat, but this service obviously appear to offer more high-level functionality for managing code updates, and is probably not mutually-exclusive with existing development practices.


[TxPool] Self pruning stuck transactions

Description
This PR addresses the issue of valid "stuck" transactions. These transactions remain in the pool indefinitely due to being considered recoverable by consensus during block building.
Changes include

[x] Bugfix (non-breaking change that solves an issue)
[ ] Hotfix (change that solves an urgent issue, and requires immediate attention)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)

Checklist

[x] I have assigned this PR to myself
[x] I have added at least 1 reviewer
[x] I have added the relevant labels
[ ] I have updated the official documentation
[x] I have added sufficient documentation in code

Testing

[x] I have tested this code with the official test suite
[ ] I have tested this code manually

Additional comments
Fixes EDGE-45

Good to have this feature. 🙌
Just a little confused because of the method name TxPool.Drop().
DropAccountByTx() would make it more clearer?


Update documentation

Hello and high five for this crate, it indeed looks great!
I'm trying it today for the first time and followed the instructions on docs.rs and also from the book (very good initiative).
TBH my audio knowledge is inexistent, but luckily by following your instructions it doesn't look that difficult.
Just opening this issue here because there's some inconsistencies in the examples, e.g.

with load / load_sound where the docs points out to kira-loaders, but indeed doesn't look necessary anymore (AudioManager's load_sound seems the way to go).
there's something regarding cpal pointing at kira-cpal but I personally didn't need it to play a sound (on MacOS M1).
one piece of advice I would have given to newcomers, is that you need to have your program running for as long as the loading of the file + the sound duration, which can easily be achieved by e.g. using tokio sleep for as long as it lasts.
I'm especially interested in updates to the mixer section, which is the one I'm the most interested into.

Thanks :)

To add some more context, what confuses me the most is that I created some manager e.g.:
#[tokio::main]
async fn main() {
    let now = Instant::now();
    let folder = std::env::var("CARGO_MANIFEST_DIR").expect("manifest dir");
    let file = std::path::Path::new(&folder).join("welcome.mp3");
    let mut audio_manager =
        AudioManager::new(AudioManagerSettings::default()).expect("audio manager");
    let mut sound_handle = audio_manager
        .load_sound(file, SoundSettings::default())
        .expect("sound handle");
    sound_handle
        .play(InstanceSettings::default())
        .expect("play sound");
    let duration = sound_handle.duration() as u64;
    let latency = now.elapsed().as_secs_f64() as u64 + 1u64; // add some offset otherwise too short
    sleep(Duration::from_secs(duration + latency)).await; // allows for playing the sound till its end
}

So far I've seen that I can add an Effect (e.g. cutoff) to the MainTrackHandle, but couldn't find how to get a reference to it.
I've looked into adding a send track or a sub track, but so far I'm still confuse on how to use kira properly.

Well my bad, I've actually finally found out main_track and the rest, so I guess it works even if maybe not the best way to do it:
let effect = Filter::new(FilterSettings::new().cutoff(1000.0));
audio_manager
    .main_track()
    .add_effect(
        effect,
        EffectSettings {
            id: None,
            enabled: true,
            mix: 0.9.into(),
        },
    )
    .expect("effect");

Hi! The book is for kira v0.6.0, which is in beta. You can see the corresponding docs page for that version here.

Thanks @tesselode your crate is really awesome, can't wait to see how it evolves. The simple fact that somebody like me who has literally zero knowledge about sound and still I'm able to produce something with a satisfying result is just incredible haha I'm having a blast !


Support Custom Headers

Provide an autocomplete with standard and frequently used headers.
This involves a bit more development on the backend, so it will take a bit longer to finish as I need to learn the ropes around Golang.

https://www.rfc-editor.org/rfc/rfc9110.html

26fc3da #7: Moves url logic to upper request page
78ba58f #7: Removes unused files
f2cc5c5 #7: Move request stuff to a dedicated component that can use contexts
9f9f931  #7: Support custom headers in go client
8c721c0 #7 Testing how to send structured data to Go side
3ea950e #7 Fix test with proper use of testing-lib
48d89d0 #7 Testing out of my machine.
1071f77 #7 Implements header removal reduction
c774644 #7 Refactors header editor to use request context
8e971a3 #7 Implements a Request context with reducer
2e8bd32 #7 documents todo
ca82172 #7 simplest logic to implement table-based editing
bb26deb #7: WIP table-based header editor
01dc8ab #7: Removes dead code


Use correct stores for proxies and fields

Proxies and fields should use the same signature as the form store, an extra option to taint/untaint the field.

This may have to be a v1.0 fix, since it requires some signature changes.

Will be a feature for 1.x, won't be included in 1.0.

All proxies can now use the whole superForm object, with a taint option.


Update windows-analytics-get-started.md

Adding note about installing the latest monthly rollup

@zdvorak Thanks!


Docs makefile doesn't build

Repro:
cd docs
pip install -r requirements.txt
make html

fails with:
Running Sphinx v2.2.0

Extension error:
Could not import extension sphinxcontrib.googleanalytics (exception: cannot import name 'ExtensionError' from 'sphinx.application'
make: *** [html] Error 2

But my real question is, should it work?

Thanks for the report!
Building the docs work for me with with Sphinx v1.7.3, so it might be an incompatibility with a newer version.
Maybe we might need to pin those versions? I'm using
Sphinx                             1.7.3
sphinx-rtd-theme                   0.4.3
sphinxcontrib-googleanalytics      0.1
sphinxcontrib-websupport           1.0.1


Stacks. Updates after initial PR review

What does this PR do?
🛑 Please open an issue first to discuss any significant work and flesh out details/direction - we would hate for your time to be wasted.
Consult the CONTRIBUTING guide for submitting pull-requests.

Motivation

More

[x] Yes, I have tested the PR using my local account setup (Provide any test evidence report under Additional Notes)
[x] Mandatory for new node blueprints. Yes, I have added usage example to the README.md file in my blueprint folder
[x] Mandatory for new node blueprints. Yes, I have implemented automated tests for all stacks in my blueprint and they pass
[ ] Mandatory for new node blueprints. Yes, I have added a reference to my README.md file to website/docs section for this feature
[x] Yes, I have set up and ran all pre-merge quality control tools on my local machine and they don't show warnings.

For Moderators

[x] The tests for all current blueprints successfully complete before merge?
[x] Mandatory for new node blueprints. All pre-merge quality control tools and cdk-nag tools don't show warnings?
[x] Mandatory for new node blueprints. The deployment test works on blank AWS account according to instructions in the README.md before merge?
[x] Mandatory for new node blueprints. The website builds without errors?

Additional Notes

Hi @AshtonStephens . Great news! It took me less than expected to test the deployment. There are a couple of changes I've made to README.md and tsconfig.json (see more in this commit: https://github.com/aws-samples/aws-blockchain-node-runners/pull/51/commits/d2bd300ed87c5f6aa705569dfb60bc3b54a0d92f ).  Let me know if you are Ok with the changes. If all good on your side, I'll go ahead a merge it to main.

Approved in spirit 👍


Implement Splash Screen for the Application

Problem:
The application currently launches directly into the home screen without any splash screen. A splash screen is needed to improve the user experience by showing a loading state while the app initializes and enhancing the overall visual appeal of the app.
Suggested Solution:
Use Flutter's flutter_native_splash package to implement a splash screen that works across both Android and iOS.
Customize the splash screen with the app’s logo, brand colors, and potentially a loading indicator.
Ensure the splash screen is lightweight and transitions smoothly into the main app content.

@prajapatihet  please assign me this issue and provide the app logo


Entities classes should implement Parcelable for better performance

@tzuhanwu any thoughts on using below gradle dependency ?
'com.ryanharter.auto.value:auto-value-parcel:0.2.0'
more here.
http://ryanharter.com/blog/2016/03/22/autovalue/

Hello @mgujare, @jbrick and I discussed about Parcelable a while ago, adding Parcelable will improve the performance but remove the compatibility to the regular JAVA. We have tried to keep the SDK be shareable between JAVA and Android. However, I do feel like we are having more discussion around the packages that only works for Android, and the shareability makes it more difficult to expand the SDK. I think it's a good time to consider making this SDK optimized for Android.
@mittonk, @jbrick, how do you feel about it?

Seems reasonable to me, we never did end up advertising it as a Java SDK.

Since @jbrick mentioned about not advertising this repo as java sdk i think changes may be needed in build.gradle in order to add android specific dependencies for this particular issue. eg: applying android plugin. Not sure if gradle can recognize 'annotationProcessor' and 'provided' dependencies as mentioned in link above without android plugin. Another likely change to move integration tests under src/main/test directory. Of course all this makes sense if yelp wants to advertise this primarily as android sdk..what say ?

Yes, a change to build.gradle for android plugin will be the first step of doing this. We have decided to move forward with this SDK to support Android specifically. And since the incompatibility with the previous version, a major version bump will be required.
I'll start to draft up the change for the build.gradle and related files, if anyone would like to do the honor, feel free to create a pull-request.

👍


update animate dead to take into account Undead Thralls?

Could this be updated to check for Undead Thralls, and if present, update the hp and damage of the zombie and skeleton, please?
let thrallHp = 0
let thrallDamage = 0
let extraThrall = 0
const undeadThralls = tactor.items.find(item => item.name === "Undead Thralls")
if (undeadThralls) {
	thrallHp = actorData.classes.wizard.levels;
	thrallDamage = prof;
        extralThall =+ 1

}

Then just add thrallHp and thrallDamag to the respective keys.

This is for the Necromancer Wizard ability correct?
This is not SRD available material unfortunately, so I won't be including it as is.
I could make the await MidiMacros.createUndead(args) call return the spawned ids and then you could further mutate them to update HPs and damage accordingly.
Or I could offer some extra customization by passing hp and damage parameters.
I will give it some thought for the next release.

The solution I think I will go with is this.
await MidiMacros.animateDead(args,config)

and the config can be something like
const config = { texture:{zombie:"image path",skeleton:"image path"}, hp: "wizard.level"  , extra:1}

with:

texture: which would change the default Zombie or skeleton image to the ones provided,
hp: the extra hp which the spawned creatures will get, either a number or a string like shown to indicate use the level of the caster,
extra: any number for changing the default number of creatures to be spawned.


Pipenv completes on Windows and hangs on Raspberry Pi

I have a problem with pipenv, when building in different platforms (machine). As you can see in the uploaded Pipfile.zip I have specified some packages to be install in the virtual enviroment. The imageio package doesn't support the raspberry pi yet.
I tried to exclude it in Pipfile using:
imageio = {version = "*", platform_machine = "!= 'armv7l'"}
Even with that directive if I try to install to my raspberry pi, the pipenv hangs!!!
Describe your environment

Raspbian: Linux *** 4.9.59-v7+ #1047 SMP Sun Oct 29 12:19:23 GMT 2017 armv7l GNU/Linux
Python version: Python 3.5.3
Pipenv version: pipenv, version 9.0.1

Expected result
I expected to install all the packages except the imageio package.
Actual result
I hangs! Some how it misses some packages and hangs around numpy package! I tried to get the logs here:
pipenv_log.zip
Any Ideas?

it's compiling from source, mostly likely, which will take a long time. You'll have to configure pipenv to set a longer timeout than 60s for package installation.


FrozenAttribute with matching strategy for xUnit2

Ports the FrozenAttribute with matching strategy to the xUnit2 Glue Library.
For example you can freeze a parameter for any interfaces implemented by its type:
[Theory, AutoData]
public void FreezeByImplementedInterface(
    [Frozen(Matching.ImplementedInterfaces)]Foo p1,
    IFoo p2)
{
    Assert.Equal(p1, p2);
}

or freeze a parameter for any properties with the same name (case-insensitive) and a compatible type:
[Theory, AutoData]
public void FreezeByMatchingPropertyTypeAndName(
    [Frozen(Matching.PropertyName)]string bar,
    Foo p)
{
    Assert.Equal(bar, p.Bar);
}

Thank you for your contribution! It's now live as AutoFixture.Xunit2 3.35.1 (there was an error publishing to NuGet, so 3.35.0 was never published, and I had to bump up the patch version to get it out).


Create translit function

PR includes

[x] Feature
[x] DX

Related issues
Resolve #1054

/canary

/canary


python310Packages.gaphas: 3.9.2 -> 3.10.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.gaphas is: GTK+ based diagramming widget
meta.homepage for python310Packages.gaphas is: https://github.com/gaphor/gaphas
meta.changelog for python310Packages.gaphas is: https://github.com/gaphor/gaphas/releases/tag/3.10.3
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.10.3 with grep in /nix/store/ps8vlqj19v19xrkssn3qhx7zrnj8gg6z-python3.10-gaphas-3.10.3
found 3.10.3 in filename of file in /nix/store/ps8vlqj19v19xrkssn3qhx7zrnj8gg6z-python3.10-gaphas-3.10.3

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

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
gaphor
python310Packages.gaphas
python311Packages.gaphas

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ps8vlqj19v19xrkssn3qhx7zrnj8gg6z-python3.10-gaphas-3.10.3
ls -la /nix/store/ps8vlqj19v19xrkssn3qhx7zrnj8gg6z-python3.10-gaphas-3.10.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

gaphor
python310Packages.gaphas
python311Packages.gaphas

Maintainer pings
cc @WolfangAukang for testing.

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

6 packages built:

gaphor
gaphor.dist
python310Packages.gaphas
python310Packages.gaphas.dist
python311Packages.gaphas
python311Packages.gaphas.dist


Update website content with new service information

This updates a lot of the content on our brochure website to reflect our latest communications around 2i2c as an organization and the hub service in general. It has a few major goals:

Reduce wordiness so that people can scan pages more quickly (we can use docs.2i2c.org for wordiness)
Reduce the number of top-level pages, and focus them around core parts of 2i2c
Update the flow of the top level pages so that they are easier to quickly scan and click through
Add in several calls to action - currently, these point people to our 2i2c docs (docs.2i2c.org) but in the future this might also be "sales" actions
Improve some of our CSS infrastructure to be more maintainable
Generally try to clean things up
Tries to take inspiration from a few product pages (coiled.io and stripe.com in particular) as well as well-organized non-profits in our space (ropensci.org)

I'll try to revisit this again tomorrow, but would love feedback, edits, etc from folks! (cc @colliand who has been helping with this effort more broadly)
preview is here! https://deploy-preview-70--cocky-booth-e7ed17.netlify.app/
Suggestions for review
This is a big refactor, so I think the most helpful way to provide feedback is to look at the preview of the built documentation, and give a general indication of whether this is an improvement according to the goals that I showed above. If there's any major pieces we are missing, I am happy to add some extra content / structure in this PR, but the main question is whether this is a good iterative step forward and if the website "feels" like an improvement :-)
This is another part of https://github.com/2i2c-org/meta/issues/78

I like these changes. Nice work @choldgraf!

Just a note that we are planning to release a blog post about the CS&S partnership tomorrow morning (https://github.com/2i2c-org/2i2c-org.github.io/pull/71). I think it'd be nice if this PR were merge in before that post went live, as I expect it'll give a short bump to our traffic. I know this is a bit short notice, but curious if anybody wants to give an enthusiastic 👍 for this idea, given the current state of this PR :-)
If I don't hear a response, or an objection, then I'll hold off for another iteration or two here.

@choldgraf, I think this is a great step forward.
I quickly navigated the whole site in the preview and I do not see any blocker for merging this one.

thanks everybody for the feedback :-) I think that this is a nice iterative improvement, so I'll merge this in, and will happily continue improving our site style/content as new ideas come up! @yuvipanda I've also opened up an issue to discuss the use of the word "DevOps" as it pertains to our service: https://github.com/2i2c-org/team-compass/issues/256


In activities for changing the access tier, a space is missing ("AccessTier")

Storage Explorer Version: 1.1.0 RC ()
Platform: Linux
Architecture: x64
Build Number: 2018501.1
Commit: 754b96f7
Regression From: N/A
Steps to Reproduce:

Start changing the access tier of a Storage Account
Cancel out of the dialog

In the activity saying I cancelled the change, a space is missing between the words "Access" and "Tier".

Also, the wording of that is really weird. "User cancelled"? Who is the activity log talking to? Just say "Cancelled changing the default access tier of "

Out for PR for 1.1.0


Docusaurus failed to build when using redocusaurus > 1.1.2

Hi Rohit,
I am using<EMAIL_ADDRESS>but I can only successfully build the site when using<EMAIL_ADDRESS>and below. If I use higher version of redocusaurus, I get this error. Any help on troubleshooting if possible please?
 Executing task: yarn run build <

yarn run v1.22.17
$ docusaurus build
[INFO] [en] Creating an optimized production build...

✔ Client

✖ Server
  Compiled with some errors in 22.96s

[ERROR] Docusaurus server-side rendering could not render static page with path /api/marketplace/v2.
[ERROR] Docusaurus server-side rendering could not render static page with path /api/platform.

TypeError: this.searchWorker.terminate is not a function
TypeError: this.searchWorker.terminate is not a function
[ERROR] Unable to build website for locale en.
[ERROR] Error: Failed to compile with errors.
    at /Users/haobinliang/capitalise-developer-portal/node_modules/@docusaurus/core/lib/webpack/utils.js:180:24
    at /Users/haobinliang/capitalise-developer-portal/node_modules/webpack/lib/MultiCompiler.js:554:14
    at processQueueWorker (/Users/haobinliang/capitalise-developer-portal/node_modules/webpack/lib/MultiCompiler.js:491:6)
    at processTicksAndRejections (node:internal/process/task_queues:78:11)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
The terminal process "/bin/zsh '-c', 'yarn run build'" terminated with exit code: 1.

docusaurus.config.js
const config = {
  presets: [
    /** ************ Redocusaurus Config *********** */
    [
      "redocusaurus",
      {
        specs: [
          {
            id: "platform-v1",
            spec: "./openapi/platform-v1.yaml",
            route: "/api/platform",
          },
          {
            id: "marketplace-v1",
            spec: "./openapi/marketplace-v1.yaml",
            route: "/api/marketplace/v1",
          },
          {
            id: "marketplace-v2",
            spec: "./openapi/marketplace-v2.yaml",
            route: "/api/marketplace/v2",
          },
        ]
      }
    ]
  ]
}

package.json
{
  "name": "capialise-developer-portal",
  "version": "1.0.0",
  "private": true,
  "scripts": {
    "docusaurus": "docusaurus",
    "start": "docusaurus start",
    "build": "docusaurus build",
    "swizzle": "docusaurus swizzle",
    "deploy": "docusaurus deploy",
    "clear": "docusaurus clear",
    "serve": "docusaurus serve",
    "write-translations": "docusaurus write-translations",
    "write-heading-ids": "docusaurus write-heading-ids",
    "typecheck": "tsc"
  },
  "dependencies": {
    "@cmfcmf/docusaurus-search-local": "^0.11.0",
    "@docusaurus/core": "2.0.0-beta.21",
    "@docusaurus/plugin-ideal-image": "^2.0.0-beta.21",
    "@docusaurus/preset-classic": "2.0.0-beta.21",
    "@docusaurus/theme-live-codeblock": "^2.0.0-beta.21",
    "@mdx-js/react": "^1.6.22",
    "clsx": "^1.1.1",
    "mdx-mermaid": "^1.2.2",
    "mermaid": "^9.0.1",
    "prism-react-renderer": "^1.3.1",
    "react": "^17.0.2",
    "react-dom": "^17.0.2",
    "redocusaurus": "^1.2.1"
  },
  "devDependencies": {
    "@docusaurus/module-type-aliases": "2.0.0-beta.21",
    "@tsconfig/docusaurus": "^1.0.5",
    "autoprefixer": "^10.4.4",
    "postcss": "^8.4.12",
    "typescript": "^4.6.3"
  },
  "browserslist": {
    "production": [
      ">0.5%",
      "not dead",
      "not op_mini all"
    ],
    "development": [
      "last 1 chrome version",
      "last 1 firefox version",
      "last 1 safari version"
    ]
  }
}

Hey, I can't work on this without a minimal reproduction. Since the latest version does work with the petstore yaml. I can't do much from the error log because there is no stack trace for TypeError: this.searchWorker.terminate is not a function (due to this open issue: https://github.com/facebook/docusaurus/issues/7456 )

Can you try this beta build<EMAIL_ADDRESS>?

Thank you @rohit-gohri ! Sorry that I couldn't give any more info in the first go.
But you worked the magic -<EMAIL_ADDRESS>works!!!

Awesome! Then I'll merge the PR and release it

Released in v1.2.2


Migrate Dockerfile from container-dtc-metviewer to the METviewer repository.

Describe the Task

Migrate Dockerfile from container-dtc-metviewer (https://github.com/dtcenter/container-dtc-metviewer/tree/master/METviewer) to the METviewer repository itself.
Once completed, delete the container-dtc-metviewer repository.
Update the DockerHub METviewer settings (https://hub.docker.com/repository/docker/dtcenter/metviewer/builds) to watch the GitHub METviewer repository.
Define automated build rules to automatically create/update images for specific branches and retags, similar to the setup for MET (https://hub.docker.com/repository/docker/dtcenter/met/builds/edit).

This setup works much better because anytime the METviewer repo changes (i.e. new development on the develop branch, bugfixes on the master_vX.Y branches, or new release tags), DockerHub will create or rebuild the associated image.
Time Estimate
2 days?
Sub-Issues
Consider breaking the task down into sub-issues.

[ ] Add a checkbox for each sub-issue here.

Relevant Deadlines
List relevant project deadlines here or state NONE.
Funding Source
Define the source of funding and account keys here or state NONE.
Define the Metadata
Assignee

[ ] Select engineer(s) or no engineer required
[ ] Select scientist(s) or no scientist required

Labels

[ ] Select component(s)
[ ] Select priority
[ ] Select requestor(s)

Projects and Milestone

[ ] Review projects and select relevant Repository and Organization ones
[ ] Select milestone

Define Related Issue(s)
Consider the impact to the other METplus components.

[ ] METplus, MET, METdb, METviewer, METexpress, METcalcpy, METplotpy

Task Checklist
See the METplus Workflow for details.

[ ] Complete the issue definition above.
[ ] Fork this repository or create a branch of develop.
Branch name:  feature_<Issue Number>_<Description>
[ ] Complete the task and verify your changes.
[ ] Add/update unit tests.
[ ] Add/update documentation.
[ ] Push local changes to GitHub.
[ ] Submit a pull request to merge into develop and link the pull request to this issue.
Pull request: feature <Issue Number> <Description>
[ ] Iterate until the reviewer(s) accept and merge your changes.
[ ] Delete your fork or branch.
[ ] Close this issue.

I made a lot of progress on this today with help from Tatiana. All changes are on the branch named feature_166_docker.

Added docker directory containing the Dockerfile to build the METviewer image and docker-compose.yml to bring it up.
Modified the Dockerfile to pull METcalcpy, METplotpy, and METviewer from GitHub instead of a pre-built tarball.
Modified the Dockerfile to build/install METviewer.
Setup DockerHub to watch the Dockerfile in github.com/dtcenter/METviewer and trigger automated builds for the feature_166_docker branch.
Added a docker/hooks/build file to specify the version of METcalcpy, METplotpy, and METviewer to be built.
Add a docker/.env file to specify the version of METviewer to be pulled/run.

Remaining work:

Test that METviewer actually runs well locally.
Consider finding/adding test_load.xml to the repo and stashing it in /METviewer/xml/test_load.xml inside the repo.
Consider cleaning up the /METviewer directory after it's been built (e.g. compare /METviewer contents between dtcenter/METviewer:3.1 (clean) and dtcenter/METviewer:feature_166 (not clean))
Consider updating the Dockerfile to NOT make /usr/bin/python version 3. Instead, install python3 elsewhere and update the Dockefile to point to that version.
Figure out how to easily load user's data into a local METviewer instance.

After getting direction from Tatiana, I renamed master_v3.1 to main_v3.1 and set it as the new default branch.
I also deleted the master branch and replaced master with main in the GitHub issue templates on the main_v3.1 and develop branches.

Configure automated builds on DockerHub for main_v3.1 (latest), develop, main_v* branches, and v* tags.
https://hub.docker.com/repository/docker/dtcenter/metviewer/builds


Email sign-in link says it has expired

Steps to Reproduce

Enter sign-in email, hit enter
Dialog says to look for email
open email and click Sign-in button
dialog says: The sign in link is no longer valid.
It may have been used already or it may have expired.

Expected Result
signed in to dca-cc.com
Actual Result
Unable to sign in
The sign in link is no longer valid.
It may have been used already or it may have expired.
Sign in

Thanks @JimZeeb, I will take a look into this and come back to you. Meanwhile, could you please send your email to us on<EMAIL_ADDRESS>


Why is hover service / hover widget in vs/editor?

I think it should move to platform or base. We use the workbench hover everywhere now outside the editor.
//cc @benibenj

It's there because it needs the markdown renderer. See https://github.com/microsoft/vscode/pull/201035

But we could move that as well, no?

@bpasero it depends on languages, tokenizer etc. I don't think that's so easy.


Remove version: taers232c.GAMADV-XTD3 version 6.70.06

This version is older than what has been set in max-versions-to-keep by the publisher.
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/taers232c/GAMADV-XTD3/releases/download/v6.70.06/gamadv-xtd3-6.70.06-windows-x86_64.msi
OK
(Automated message - build 910)