Implement VA.gov content changes for 526EZ notification

Value Statement:
As a Veteran
I want to see relevant content on VA.gov for the 526EZ notification
So that I am acquainted with the notification email
Acceptance Criteria

GIVEN I fill out a disability compensation application on VA.gov
WHEN I read the updated content of the application and confirmation screen
THEN I understand I will receive an email after successful submission

GIVEN I fill out a disability compensation application on VA.gov
WHEN I get routed to the experience that has the 526EZ conformation email turned on
THEN I can view the updated content

GIVEN I fill out a disability compensation application on VA.gov
WHEN I get routed to the experience that does not have the 526EZ conformation email turned on
THEN I can view the old content

Checklist:

[ ] Update the first paragraph of step 4 of 5 (Additional information) to, "This is the contact information we have on file for you. Please review it to make sure the information below is correct. We’ll send any important information about your disability claim to your mailing address. After you submit your claim, we’ll send you a confirmation email to your email address."
[ ] Update the last paragraph of step 4 of 5 (Additional information) to, "Any updates you make here to your contact information will only apply to this application. If you want to update your contact information for all of your VA accounts, please go to your profile page.
[ ] Update the first line of the confirmation screen to, "We'll send you an email to confirm that we received your claim. You can also print this page for your records."

Assumptions:

Additional Info/Resources:

See attached screenshots  and

Out of Scope:

Open Questions:

Content must be added to the feature toggle for the 526EZ email (but not required for dark launch) - decision was made to add this content to the feature toggle. AC 2 and 3 were added

Is this dependent on releasing the confirmation email feature to 100% of users? Or should this be behind the same toggle?

Update:

Managed to run vets-website and vets-api locally. I was running into an issue with logging in, which turned out to be a problem with me overwriting the saml_ssoe section in setting.local.yml. This should be removed/commented out so that the idp_metadata_file in settings.yml is config/ssoe_idp_int_metadata_isam.xml.
Once I was able to log in with a user, I tried several different accounts, namely 228, 246, 248, 150, and 1. At least 228 definitely has an intent to file, as I tried on dev, but when running it locally, it says there is not enough info on the account to fill out this application. Also tried mocking the data with the yarn mock-api command in the vets-website readme and no luck.
Decided to instead try testing by running the cypress tests and stopping them at the Additional information page. Even though my toggle is potentially turned on, I don't think I'm passing it in to the component correctly. Going to push that change so it's on the branch. (Branch on both repos, starting with notification-api#22)

PRs:

vets-website: https://github.com/department-of-veterans-affairs/vets-website/pull/14935
vets-api: https://github.com/department-of-veterans-affairs/vets-api/pull/5220

Status:

Add all three copies are added with conditional rendering based on feature toggles value
Pushed up to working vets-website branch
Note comment in description box regarding disabling linting error for in test file for contactInformation

To do:

need test for connected component of ConfirmationPage (had quickly tested the unwrapped component)
3 integrated tests are broken that consume ConfirmationPage (yarn test:unit should return those failing tests)
please refactor!

Rant:

Linter is great, but it takes forever to run and so compounds dev time :(

Hi! I'm from the Benefits and Memorial 1 team. We're responsible for this form. I was wondering if these content changes have been reviewed by any content editors? Our go to person is Peggy Gannon. I'll send her a link to this ticket.

Hi @Mottie ! Yes, these changes have been reviewed. @toddstanich has been working with Peggy to approve these copy changes.
cc @miabecker

@toddstanich @miabecker
Just to confirm, do you want the same copy displayed on the confirmation page regardless of submission status? With the way the copy was already set up, it said to "Please print this page for your records." whether it was successful or not.

@marisahoenig does the email still send eventually in this case? And prior to our changes, did the "Please print this page.." text still display in this scenario?

@toddstanich Yes x2, it still sends eventually and always sent the "Please print this page for your records."

@marisahoenig I think we're good as is then. Thanks!

@toddstanich We are looking further at the different error cases that occur, and when an alert says "We’re sorry. Something went wrong when we tried to submit your claim", the email does not get sent because an error occurred but the old copy is rendered regardless of the error status. Do we still want the new copy displayed in this case?

@marisahoenig Good catch. No, we should not display the content mentioning an email if the email will not be sent.

Desk checked & tested/reviewed with @jsmithVA ! Moving to sign off


[api] Add regex path parameter matching

Refer to https://github.com/tipsy/javalin/pull/894

Fixed with https://github.com/avaje/avaje-jex/commit/f2e86b32214bacacff3a1e49a4fb019d7d19d57a

Nice!


[Item] Blazemail

All game item. Used on heroes that like to trade hits and fight/tank a lot (think of Axe, Bristleback, LC).
Crafted using Radiance and a Blademail. Upgraded using same stuff as a normal Radiance.
Passive Effect: Any attacking unit has a 15% chance to be blinded for 1 second.
Active Effect:Warflame:This is slightly complex- there are 2 parts. 1- The user basically obtains a burning aura, like a Radiance but dealing more damage. However, the miss chance component only applies to units that are not attacking the user.. 2-The user reflects 100% of the damage dealt to him. This ability has a 30 second cooldown  and costs 25 mana to cast.
Stats:Provides some good armour and damage. Nothing else.
Can be disassembled.

Neat idea, @Kolkonut . To be honest,  I'd enjoy having such an item as an upgrade to blademail.
But care to explain what you meant by " However, the miss chance component only applies to units that are not attacking the user.."? So you mean that the radiance's miss chance doesn't apply when the item is active?

30s cooldown,  25 mana... OP as F.

@Wujekklawy , I believe those numbers are just for show, so the basic mechanic is not affected. So it's a more or less  neat tank/initiator item

@VoidsKeeper Consider three units, A,B and C. Let A have a Blazemail. A activates the Blazemail.
Now consider B and C. Both of them are within the radius of the Blazemail effect. B targets C and attacks. These attacks have a X% miss chance.
Now let B attack A. These attacks will not miss.
This is to ensure that the effects of evasion and damage return do not negate each other.

@Kolkonut or you just do it like centaur return,  no matter if you miss or not you will take damage

@Wujekklawy why not both? You could also do it like this:
On active, Blazemail gives X% chance to miss all allied targets in the radius, but the player himself does not have a miss chance.

The passive is hard to follow. Does it get muted the the active is engaged?
I don't like the way the active blind works. Blademail punishes people for attacking you. They have counterplay, the item has a clear purpose. This just punishes people. It says "buy mkb or you lose"
Also, what is the blind miss %? Just a ballpark. Is it high like KotL or Tinker, or low like vanilla Radiance?

@Vafnar that, good sir, is an excellent idea.

@Haganeko around 15-20% perhaps.

@All I must admit I prefer the combination of @Haganeko and @Wujekklawy suggestions. They feel better than my own idea.

I say combining a blademail effect for oneself with miss chance for attacking allies is... not on theme, I guess, is what I'm trying to say. I like the passive, and I like it getting muted on activation (because one wants people to hit them when blademail is active), but I am very indisposed towards it giving allies what is essentially evasion.

@Haganeko , those could be two separate items , now that I think about it.

@VoidsKeeper that might be better. This should probably only have the burn. Alternatively, perhaps crafting it with different variants of the Radiance change the active to Return+That Radiance?
Like the Lucience Blazemail heals allies, the Radiance Blazemail burns enemies etc.

Is it alright if I rewrite the suggestion to be simpler to understand and integrate some of these great ideas?
@Haganeko @VoidsKeeper ?

Of course. Just remember Martyr's Mail is already a thing.

Yeah, go ahead! :)

Let's get really complex... let's make the burn % amount, be damage dealt from the last use of the item divided by 10. For example, if the last time you used the active, you dealt 300 damage, then the next time you activate the effect, you deal 30 damage per second, creating a compound effect that is lost when the timing of the active is used wrong, punishing misplays harshly. This could only work against heroes though as I can see people abusing this against creeps. When the active is off however it goes back to baseline damage to that of normal Blazemail.
Or this can be totally ignored and maybe used for a different item.

@Vafnar then you end up with item that deal 0 damage and there no way to change that.

@Vafnar that would reward good play and stuff ,however, it would be rather finicky to use. It's definitely an interesting idea, perhaps suggest it as a separate item?

@Wujekklawy to counter this, have the starting damage be 10, if you hit all 5 heroes and 7 creeps, that is 120 damage and the next time you use it it will have 12 dps, if you hit all 5 heroes and 2 creeps, that is 84 dps, which means next time you deal 8.4 dps, and the cycle continues until the sweet sweet release of death that i yearn for at every waking moment

Still looks way too complicated. Closed.


Add HTable method to HBase Plugin

Distinguishing HBase read and write requests are helpful to analyze the usage of APP behavior.
So I record the HTable method to realize the function.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.jimolonely seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Where can I find icinga-php-library v0.11.0?

Hello Icinga Community,
I want to add the certificate monitoring (x509) module to my Icinga 2 installation.
The official Icinga website says that Icinga PHP Library (>= 0.11.0) is required.
On my Icinga 2 installation I have the ipl v0.10.1 installed. But where can I find the ipl v0.11.0 or above?
On github it says that ipl v.0.10.1 is the latest version.
For me that makes no sense.
Can anyone help me out?
Thanks a lot and have a nice day.
Best regards.

Hallo @zusa777,
The master branch of the X509 module is requiring an IPL version, which is not yet released, but will soon be. See https://github.com/Icinga/icingaweb2-module-x509/issues/139#issuecomment-1324826550 to workaround this issue temporarily. Have a nice weekend!

Hello @yhabteab
Thanks for your fast reply. Do you more or less know when ipl v0.11.0 is out?
Have also a nice weekend!

It will be out this week.


Implement Encode/Decode for std::time::Duration

Hello, thanks a lot for this library.
Would it be possible to implement Encode/Decode for std::time::Duration?. My use case is for a wallpaper daemon (swww) that play animations like gifs. I need to serialize the frame's duration, since for some image formats (like gif), they may not be the same for all frames in the image.
Right now, I am probably just going to go with a u128 representing nanoseconds, but it would be nice if I could just continue to use Duration directly, like I did when using rkyv.

https://github.com/SoftbearStudios/bitcode/commit/5f25a59be3e66deef721e7eb2369deb1aa32d263


#95 No such file or directory solution

So the workaround is to use absolute path when checking against ruby. I am not able to come with better solution, like changing to the current working directory before issuing the !lmake.

After some googling I have found vim-rails changing QuickFixCmdPre variable:
./bundle/vim-rails/autoload/rails.vim:  autocmd QuickFixCmdPre  make call s:push_chdir()
https://github.com/tpope/vim-rails/blob/master/autoload/rails.vim#L4543
Would it be possible to reset the value prior starting make and then setting it back? What you think?

With the latest and greatest versions of both plugins, it works.


LCFS - Error saving edits to Allocation agreement data

Describe the Bug:
Edits made to rows in the Allocation Agreement input form are not saving. Attempting to save changes results in an error, and the data remains unchanged.
Expected Behaviour:
Edits to rows in the Allocation Agreement input form should be saved successfully without any errors.
Actual Behaviour:

When attempting to save changes to a row, an error occurs.
The edits are not saved, and the original data remains unchanged.

Implications:

Users are unable to update Allocation Agreement data, potentially leading to incomplete or inaccurate compliance reports.

Steps To Reproduce:

User/Role: BCeID user
Navigate to the Allocation Agreement input form.
Make edits to any row.
Observe the error and verify that the changes are not saved.

Additional Context:

It is normal to get 422s while the row is invalid or incomplete, but it should be showing an error message for what is required. Was the label stuck saying Updating Row?

Yes the row was complete/valid but it hung on Updating row


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Panda Error

I am a newby to python - can you please explain how to resolve the issue below please.

c:\Crypto\Experiment\pycryptobot-main -Bitcoin\pycryptobot-main>python pycryptobot.py --live 1
Traceback (most recent call last):
File "C:\Users\Administrator\AppData\Local\Programs\Python\Python39\lib\site-packages\pandas\core\indexes\base.py", line 5845, in _raise_if_missing
raise KeyError(f"{not_found} not in index")
KeyError: "['created_at', 'market', 'action', 'type', 'size', 'filled', 'fees', 'price', 'status'] not in index"
c:\Crypto\Experiment\pycryptobot-main -Bitcoin\pycryptobot-main>

A new major version was released this year. Please upgrade to v7.0.34 or later and if you are still having a problem, raise an issue.
Make sure all your packages are up to date:
python3 -m pip install -r requirements.txt -U


Serial test fail

When I run any serial test in Linux the following error appears:
Starting ERPC server...
test_lists_server_serial_test: /erpc-master/erpc_c/infra/erpc_framed_transport.cpp:67: virtual erpc_status_t erpc::FramedTransport::receive(erpc::MessageBuffer*): Assertion `m_crcImpl && "Uninitialized Crc16 object."' failed.
Aborted

I also couldn't successfully test serial port based eRPC, I used virtual serial port created by "socat", maybe it doesn't support virtual serial port. I want to know if there is any problem with my code.
Do we have the eRPC demo based on serial port？

Hi, unfortunately there is no example showing the serial transport usage. It deals with not-maintained transport.


Unity.Interception support for UWP

Are there any plans to add support for the Unity.Interception library for UWP apps?  When I add it to my UWP application via NUGet and it fails.

As far as I know this isn't possible due to missing API - namely missing Reflection.Emit(). IMHO the only way to support interception would be to make it compile time - I'm thinking some xml configuration and T4.

FWIW, there is also PostSharp.  If you run an open source project you get a free professional license: https://www.postsharp.net/

@Mike-EEE
Could you explain how post sharp could help running interception in UWP?


cppapi :InterceptPlugin fix

This change unbreaks using cppapi ServerIntercepts
(Ran into a hanging FastCGI plugin while trying address comments in
https://github.com/apache/trafficserver/pull/3749)

@oschaaf Looks good. Can you squash the commits?

@maskit Sure. Squashed and force-pushed.

Nice work!

Cherry-picked to 8.0.x


No audio?

After the video was created, there is no audio, the source has audio though.

Hi, the audio is removed if the image_limit and the starting_frame are not setted to 0. Did you do one of those things?
(i need to write this in the readme )

Updated README


Concave hull

http://www.it.uu.se/edu/course/homepage/projektTDB/ht13/project10/Project-10-report.pdf
https://github.com/Liagson/ConcaveHullGenerator
https://www.codeproject.com/Articles/1201438/The-Concave-Hull-of-a-Set-of-Points

After some resources, here are some solutions:

Use the implementation Grasshopper has.
Alpha concave, which is a generalisation of the Concave Hull.
K-nearest
Or from here, we have the Graham Scan or the Jervis March algo

@cesarecaoduro @d3ssy Any preferences?

@sonomirco I think you're confusing CONVEX hull and CONCAVE hull.
There is no concave hull in GH, but there is an old third-party implementation of alpha-shape .

Yep, true.
The other code mentioned is from here


chore: single snp coloc case

no longer filtering out single snp studyLocusOverlaps.
when: f.col("sumlogsum") == f.col("logsum12") == 0 :
This causes problems when we have to log the result.
Instead of 0, replace with a very small number.
Works when one causal snp is overlapping, returns high h4 value very close to 1.

What is the status here?

Added additional tests for other edge cases where the overlapping SNPs are not causal/ only causal in one study.  The fix returns sensible results in both cases(high h0 or high h1).
This doesn't happen in practice as we take the union of the overlaps between credible sets, in Finngen r10 for example there are 3.4k/39k studyLocusOverlaps which are single SNP, all of which are causal (min.PP = ~0.95, avg.PP=0.99).


when i use tween animation  in app bar flexibleSpace,  ios swipe navigation that time image overlape

Steps to reproduce
When utilizing tween animation within the flexibleSpace of the AppBar and performing swipe navigation on iOS, there is an issue where the image experiences overlap.
Expected results

Actual results

Code sample
import 'package:campus/ui/themes/theme.dart';
import 'package:flutter/material.dart';

void main() {
  runApp(MaterialApp(home: FirstScreen()));
}

class FirstScreen extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Zoom Animation'),
      ),
      body: Center(
        child: ElevatedButton(
          onPressed: () {
            Navigator.push(
              context,
              MaterialPageRoute(builder: (_) => SecondScreen()),
            );
          },
          child: const Text("click"),
        ),
      ),
    );
  }
}

class SecondScreen extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      backgroundColor: Colors.red,
      appBar: AppBar(
        flexibleSpace: CommunityBannerInAppBarWidget(),
      ),
      body: const Center(child: Text('second Screen')),
    );
  }
}

class CommunityBannerInAppBarWidget extends StatefulWidget {
  CommunityBannerInAppBarWidget({
    Key? key,
  }) : super(key: key);
  @override
  State<CommunityBannerInAppBarWidget> createState() =>
      _CommunityBannerInAppBarWidgetState();
}

class _CommunityBannerInAppBarWidgetState
    extends State<CommunityBannerInAppBarWidget> {
  @override
  void initState() {
    super.initState();
  }

  @override
  Widget build(BuildContext context) {
    var defaultImage = Image.asset(
      "assets/images/drawer/profile_insitute_banner.png",
      width: CustomAppTheme.fullWidth(context),
      //height: 180,
      height: AppBar().preferredSize.height +
          CustomAppTheme.statusBarHeight(context) +
          60,
      fit: BoxFit.cover,
    );
    return TweenAnimationBuilder<double>(
      tween: Tween(begin: 1.0, end: 1.5),
      duration: const Duration(milliseconds: 500),
      builder: (context, progress, child) {
        return Transform.scale(scale: progress, child: defaultImage);
      },
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
nothing happen in console only ui issue

Hi @vijaymsc
Your sample code is incomplete as it's using external classes/resources. I updated it as below but don't see the issue. You can try this sample code (you need to replace the image asset) and confirm if the issue persists.

Demo
https://github.com/flutter/flutter/assets/104349824/bd73a74b-cd8b-493f-9494-4f1a157e20a3

Sample code
import 'package:flutter/material.dart';

void main() {
  runApp(MaterialApp(home: FirstScreen()));
}

class FirstScreen extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Zoom Animation'),
      ),
      body: Center(
        child: ElevatedButton(
          onPressed: () {
            Navigator.push(
              context,
              MaterialPageRoute(builder: (_) => SecondScreen()),
            );
          },
          child: const Text("click"),
        ),
      ),
    );
  }
}

class SecondScreen extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      backgroundColor: Colors.red,
      appBar: AppBar(
        flexibleSpace: CommunityBannerInAppBarWidget(),
      ),
      body: const Center(child: Text('second Screen')),
    );
  }
}

class CommunityBannerInAppBarWidget extends StatefulWidget {
  CommunityBannerInAppBarWidget({
    Key? key,
  }) : super(key: key);
  @override
  State<CommunityBannerInAppBarWidget> createState() =>
      _CommunityBannerInAppBarWidgetState();
}

class _CommunityBannerInAppBarWidgetState
    extends State<CommunityBannerInAppBarWidget> {
  @override
  void initState() {
    super.initState();
  }

  @override
  Widget build(BuildContext context) {
    return TweenAnimationBuilder<double>(
      tween: Tween(begin: 1.0, end: 1.5),
      duration: const Duration(milliseconds: 500),
      builder: (context, progress, child) {
        return Transform.scale(scale: progress, child: Image.asset('assets/house.png'));
      },
    );
  }
}

I'm checking this on Flutter stable 3.16.8. Please provide the output of flutter doctor -v.

yes issue persists , you replace Transform.scale child widget Container(color:Colors.yellow) in your sample code now check  issue is thare

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.7.3, on macOS 14.2.1 23C71 darwin-arm64, locale en-IN)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] Connected device (3 available)
[✓] HTTP Host Availability
• No issues found!

issue is resolved when use ClipRect as a parent class
class SecondScreen extends StatelessWidget {
@override
Widget build(BuildContext context) {
return ClipRect( // add this widget
child: Scaffold(
backgroundColor: Colors.red,
appBar: AppBar(
flexibleSpace: CommunityBannerInAppBarWidget(),
),
body: const Center(child: Text('Second Screen')),
),
);
}
}

Thanks for the update. I can reproduce this issue on the latest stable/master channels as well. Adding ClipRect to 2nd page can work around this.

Demo
https://github.com/flutter/flutter/assets/104349824/44ecf809-20a9-4347-bba6-01409998cd8c

Complete sample code
import 'package:flutter/material.dart';

void main() {
  runApp(MaterialApp(home: FirstScreen()));
}

class FirstScreen extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Zoom Animation'),
      ),
      body: Center(
        child: ElevatedButton(
          onPressed: () {
            Navigator.push(
              context,
              MaterialPageRoute(builder: (_) => SecondScreen()),
            );
          },
          child: const Text("click"),
        ),
      ),
    );
  }
}

class SecondScreen extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      backgroundColor: Colors.red,
      appBar: AppBar(
        flexibleSpace: CommunityBannerInAppBarWidget(),
      ),
      body: const Center(child: Text('second Screen')),
    );
  }
}

class CommunityBannerInAppBarWidget extends StatefulWidget {
  CommunityBannerInAppBarWidget({
    Key? key,
  }) : super(key: key);
  @override
  State<CommunityBannerInAppBarWidget> createState() => _CommunityBannerInAppBarWidgetState();
}

class _CommunityBannerInAppBarWidgetState extends State<CommunityBannerInAppBarWidget> {
  @override
  void initState() {
    super.initState();
  }

  @override
  Widget build(BuildContext context) {
    return TweenAnimationBuilder<double>(
      tween: Tween(begin: 1.0, end: 1.5),
      duration: const Duration(milliseconds: 500),
      builder: (context, progress, child) {
        return Transform.scale(
          scale: progress,
          child: Container(
            color: Colors.yellowAccent,
          ),
        );
      },
    );
  }
}

flutter doctor -v (stable and master)
[✓] Flutter (Channel stable, 3.16.9, on macOS 14.1 23B74 darwin-x64, locale en-VN)
    • Flutter version 3.16.9 on channel stable at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 41456452f2 (14 hours ago), 2024-01-25 10:06:23 -0800
    • Engine revision f40e976bed
    • Dart version 3.2.6
    • DevTools version 2.28.5

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

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

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

[✓] Android Studio (version 2023.1)
    • Android Studio at /Applications/Android Studio Hedgehog.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

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

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

[✓] Connected device (4 available)
    • RMX2001 (mobile) • EUYTFEUSQSRGDA6D                         • android-arm64  • Android 11 (API 30)
    • iPhone (mobile)  • d9a94afe2b649fef56ba0bfeb052f0f2a7dae95e • ios            • iOS 15.8 19H370
    • macOS (desktop)  • macos                                    • darwin-x64     • macOS 14.1 23B74 darwin-x64
    • Chrome (web)     • chrome                                   • web-javascript • Google Chrome 120.0.6099.234

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

• No issues found!

[!] Flutter (Channel master, 3.19.0-9.0.pre.204, on macOS 14.1 23B74 darwin-x64, locale en-VN)
    • Flutter version 3.19.0-9.0.pre.204 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    ! Warning: `flutter` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/huynq/Documents/GitHub/flutter/bin/dart, which is not inside your current Flutter SDK checkout at /Users/huynq/Documents/GitHub/flutter_master. Consider adding /Users/huynq/Documents/GitHub/flutter_master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision efb1346767 (32 hours ago), 2024-01-27 14:07:24 -0500
    • Engine revision f3d48be766
    • Dart version 3.4.0 (build 3.4.0-81.0.dev)
    • DevTools version 2.31.0
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

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

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

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

[✓] Android Studio (version 2023.1)
    • Android Studio at /Applications/Android Studio Hedgehog.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

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

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

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

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

! Doctor found issues in 1 category.

Workaround in https://github.com/flutter/flutter/issues/142211#issuecomment-1913959351


Is Travis CI still being used?

This commit suggested that Travis is no longer in use however the Travis files are still in the repo. Should they be removed?

I would take a PR to remove them.

Followup question: is coveralls still being used? .coveralls.yml appears to have been added at the same time as Travis.

Yeah we could remove that too.


Allow --no-require-sha and Brewfile require_sha:false

Cross-issue on homebrew-cask

Provide a detailed description of the proposed feature
Allow to activate require-sha by default/globally via HOMEBREW_CASK_OPTS="--require-sha" and to deactivate when necessary using --no-require-sha or in a Brewfile cask "name", args: { require_sha: false }.
What is the motivation for the feature?
I want to require all casks to have a checksum by default, globally.
But as some formulae doesn't have one, I need a way to selectively deactivate it.
And of course I don't want to do the opposite; i.e. manually requiring only when available.
Example use case
HOMEBREW_CASK_OPTS="--require-sha"

brew install --cask --no-require-sha font-roboto

Brewfile:
cask "font-roboto", args: { require_sha: false }

Let's leave the discussion in the linked issue until we have an outcome there as this repository will be dependant on that.

I've also just realised, this is already an example in the README :/


Already has error "invalid_grant" on ResourceOwnerPasswordValidationContext result before authentication

Issue / Steps to reproduce the problem
I am using ResourceOwnerPassword grant as I want users to log in directly from Angular SPA client rather having to redirect to another log in page. In my IResourceOwnerPasswordValidator implementation, not sure why ResourceOwnerPasswordValidationContext.Result already has errored with "invalid_grant" even before any credenticals check logic begins:

In the log, it just says "User authentication failed: Invalid username or password" which I don't think is the real error; invalid_grant is definitely a more serious error.
Below is the code that is relevant:
The client:
 private IEnumerable<Client> GetClients(ICollection<string> allowedCorsOrigins)
        {
            return new List<Client>
            {
                new Client
                {
                    ClientId = "web-badders-rank",
                    ClientName = "Badders Rank Web Client",
                    ClientSecrets = {new Secret("secret".Sha256())},
                    //AccessTokenType = AccessTokenType.Reference,
                    AllowedGrantTypes = GrantTypes.ResourceOwnerPassword,
                    AllowAccessTokensViaBrowser = true,
                    AccessTokenLifetime = 3600 * 24 * 10, // 10 days
                    IdentityTokenLifetime = 3600 * 24 * 10, // 10 days
                    // AllowedCorsOrigins = allowedCorsOrigins,
                    //AlwaysIncludeUserClaimsInIdToken = true,
                    //RequireConsent = false,
                    AllowedScopes =
                    {
                        IdentityServerConstants.StandardScopes.OpenId,
                        IdentityServerConstants.StandardScopes.Profile,
                        "badders-rank-api"
                    }
                }
            };
        }

public void ConfigureServices(IServiceCollection services)
{
services.AddOptions();
services.AddMemoryCache();
services.Configure(Configuration.GetSection("DataBase"));
        services.AddMvcCore()
                .AddJsonFormatters();
         services.AddScoped(typeof(IPlayerRepository), typeof(PlayerRepository));
        services.AddScoped(typeof(IGroupRepository), typeof(GroupRepository));

        services.AddTransient<IResourceOwnerPasswordValidator, ResourceOwnerPasswordValidator>();
        services.AddIdentityServer()
            .AddDeveloperSigningCredential()
            //.AddInMemoryPersistedGrants()
            .AddInMemoryApiResources(GetApiResources())
            //.AddInMemoryIdentityResources(GetIdentityResources())
            .AddInMemoryClients(GetClients(new[] { "http://localhost:4200" }))
            //.AddTestUsers(GetTestUsers())
            .AddResourceOwnerValidator<ResourceOwnerPasswordValidator>();

        services.AddAuthentication(JwtBearerDefaults.AuthenticationScheme)
            .AddIdentityServerAuthentication(options =>
            {
                options.Authority = _authority;
                options.ApiName = "badders-rank-api";
                options.RequireHttpsMetadata = false;
                options.ApiSecret = "secret";
                options.SupportedTokens = SupportedTokens.Both;
            });

        services.AddCors(option => option.AddPolicy("MyPolicy", builder =>
        {
            builder.AllowAnyOrigin();
        }));
    }

Relevant parts of the log file
2017-12-27 17:36:57.698 +00:00 [WRN] CorsPolicyService did not allow origin: http://evil.com/
2017-12-27 17:36:57.700 +00:00 [DBG] Request path /connect/token matched to endpoint type Token
2017-12-27 17:36:57.700 +00:00 [DBG] Endpoint enabled: Token, successfully created handler: IdentityServer4.Endpoints.TokenEndpoint
2017-12-27 17:36:57.701 +00:00 [INF] Invoking IdentityServer endpoint: IdentityServer4.Endpoints.TokenEndpoint for /connect/token
2017-12-27 17:36:57.702 +00:00 [VRB] Processing token request.
2017-12-27 17:36:57.702 +00:00 [DBG] Start token request.
2017-12-27 17:36:57.702 +00:00 [DBG] Start client validation
2017-12-27 17:36:57.702 +00:00 [DBG] Start parsing Basic Authentication secret
2017-12-27 17:36:57.702 +00:00 [DBG] Parser found secret: BasicAuthenticationSecretParser
2017-12-27 17:36:57.702 +00:00 [DBG] Secret id found: web-badders-rank
2017-12-27 17:36:57.702 +00:00 [DBG] Secret validator success: HashedSharedSecretValidator
2017-12-27 17:36:57.702 +00:00 [DBG] Client validation success
2017-12-27 17:36:57.702 +00:00 [VRB] Calling into token request validator: IdentityServer4.Validation.TokenRequestValidator
2017-12-27 17:36:57.702 +00:00 [DBG] Start token request validation
2017-12-27 17:36:57.702 +00:00 [DBG] Start resource owner password token request validation
2017-12-27 17:38:52.566 +00:00 [ERR] User authentication failed: Invalid username or password
2017-12-27 17:38:52.568 +00:00 [ERR] {
  "ClientId": "web-badders-rank",
  "ClientName": "Badders Rank Web Client",
  "GrantType": "password",
  "Scopes": "badders-rank-api",
  "UserName"<EMAIL_ADDRESS>  "Raw": {
    "grant_type": "password",
    "username"<EMAIL_ADDRESS>    "password": "***REDACTED***",
    "scope": "badders-rank-api"
  }
}
2017-12-27 17:38:52.569 +00:00 [VRB] Invoking result: IdentityServer4.Endpoints.Results.TokenErrorResult

I just checked IdentityServer4 source code and got even more confused why this has happened because in the soruce code below from  private async Task<TokenRequestValidationResult> ValidateResourceOwnerCredentialRequestAsync(NameValueCollection parameters) in the /src/IdentityServer4/Validation/TokenRequestValidator.cs, clearly resourceOwnerContext  is just created and should have no error before it gets passed into ValidateAsync, and still in my case resourceOwnerContext is already errored before any attempt of authentication.
            /////////////////////////////////////////////
            // authenticate user
            /////////////////////////////////////////////
            var resourceOwnerContext = new ResourceOwnerPasswordValidationContext
            {
                UserName = userName,
                Password = password,
                Request = _validatedRequest
            };
            await _resourceOwnerValidator.ValidateAsync(resourceOwnerContext);
          // more source code

@stt106 How are you? Did you manage to fix this issue?


[Logging] Add Fluentd and Fluentbit configs

Q
A

Bug fix?
yes

New feature?
no

API breaks?
no

Deprecations?
no

License
Apache 2.0

What's in this PR?

Add Fluentd and Fluentbit configs
Add Fluentd and Fluentbit versions to the output

Checklist

[x] Implementation tested (with at least one cloud provider)
[x] Error handling code meets the guideline
[x] Logging code meets the guideline (TODO)

nice catch, thx @pepov


Alternative way to import the library

Hi, great library, would like to know if it's possible to npm install the library and import from node modules (e.g. import { JSTable } from 'jstable-psl') instead of using the cdn or cloning the library.
As through npm install it is easier to update the library version from time to time.
Thanks!

Hey @nathanielongyiitak,
JSTable is currently not published to npm, but I will look into it.
Please feel free to submit a pull request for importing JSTable as module.

Noted, thanks =)

JSTable is now published to NPM: https://www.npmjs.com/package/@jstable/jstable


Perl syntax highlighting doesn't recognize regex delimiters other than /regex/

monaco-editor version: 0.14.3
Browser: Chromium 51
OS: Ubuntu 15.10
Steps or JS usage snippet reproducing the issue:
The Perl syntax highlighter doesn't recognize regex delimiters other than /regex/ for regular expressions. Perl allows the use of alternative delimiters such as m!regex!. This throws off the highlighting when using a character such an apostrophe (like m!can't!). The syntax highlighter thinks the apostrophe is the beginning of a single quote and highlights everything that comes after it as a string:

I have submitted a pull request.

Thank you, fixed via https://github.com/Microsoft/monaco-languages/pull/50


How can I Customize size? ❓

as the tittle says, how can i customize the size of the resulted svg ?
thanks in advance.

Hi @joseph-sx
Just sent a new version 0.3.2 in which I added height and width option properties to QRCodeSVG
Check this demo for more info

Thanks! We'll check it out


Fix OSX keymap

This adds an OSX-specific keymap, since by default people save with super+s on OSX.

Before I pull, I wanna confirm the situation with ctrl+, is that correct? I don't have a Mac but I've always heard mention of cmd instead of ctrl.
Also, as is customary, I like to add anyone who submits changes to the AUTHORS.md file, if you don't object to being added, the license is MIT

What did you want to confirm about the "ctrl+"? If you're asking about whether people use cmd instead of ctrl (as in Windows, Linux) for lots of commands, the answer is yes. So save, cut, copy, paste are cmd+(s,x,c,v) respectively.
Similarly, Sublime's default OSX key bindings for stuff like ctrl+p (file searcher), ctrl+shift p(command palette) are  ctrl->cmd as well.
However, I did leave the ctrl+space, since cmd+space is mapped to the search/launcher or Quicksilver(popular launcher) on OSX by default. I personally have mapped the capslock->ctrl, since ctrl is this tiny button on a mac. Not sure how many people have done such.
Sure feel free to add me in. I definitely don't mind the license of MIT.

Yeah I was referring to the ctrl+space, but ST2's default keymaps uses ctrl+space for OS X as well so I guess that's ok.


Change the fault members identifying way

When same host having more than one instance and if one instance goes
down fault member map contain the host name. It is a bug. It must be
unique for each instance. Change identifier to mamber_id witch unique
one in conf file

@sajithab Thanks for the pull request. I have now merged this master branch.
Notes:
I may not call this a bug because in Stratos each member is considered to run on a dedicated host. That's why we have used the member hostname as the key in this map. However since you found this problem while running multiple servers on the same host this could be considered as an improvement.


Issue/21/pandas

The only real changes needed to get pandas working was making the spec-z data explicitly an array and putting in the .loc statements to not change a slice of an array.  I also re-did how we put in the non-detect values, as the old was was inefficient and bad.  A few linting changes as well.  Things work for me with both pandas and OrderedDict inputs, and all tests pass on my local machine, so I think this covers everything.

yeah, I'll create an issue now to note that we should think about removing pandas dependence if avoidable, depending on future changes to parquet reading.


crypt.h missing from src/kml/base/contrib/minizip

This requires having another minizip installation to provide this header.

on archlinux, crypt.h isn't included in the minizip installation in the repo (as opposed to some other distros).
In order to build this, I had to take the crypt.h file from https://github.com/rashadkm/minizip, adding it to src/kml/base/contrib/minizip, then edit unzip.h in the same folder to change <minizip/crypt.h> to <crypt.h> on line 183

It seems it's supposed to be an internal header--
https://github.com/madler/zlib/pull/229
https://bugs.archlinux.org/task/58711
https://gitlab.archlinux.org/archlinux/packaging/packages/zlib/-/commit/ca317b2fda17ab67d40ff113606274a753cf4365


Add Nix flake (WIP)

This flake exposes a package and a devShell to develop on Cashu and build the binaries.
With flakes enabled, run nix build from project root.
Tested only on Linux.
currently the cryptography package throws an error on "BN_Add" symbol.

The error you've showed was related to the bitcoin library in Cashu which is used for P2SH. I could imagine that we can remove that functionality and with it the entire library from the code base.
Should I PR that in a separate branch and you can merge it into this branch?

That works yes, thanks!

That works yes, thanks!

done!

I also moved the nostr client directly into the repo: https://github.com/cashubtc/nutshell/pull/346

OK, updated to use the new nostr setup. Rebased and squashed.

Not sure if the poetry.lock updates here are useful are not, they were automatically generated.
This builds the cashu binaries successfully on my machine. Could some mac folk (@chrisguida maybe?) test it as well?
Starting the mint fails because it's looking for a tor PID in the directory, I'll look at this next.

I'll get to this in a week or so! Unfortunately traveling without my mac currently

Starting the mint fails because it's looking for a tor PID in the directory, I'll look at this next.

Any idea where this comes from? Nothing in the repo knows about PID files afaik....

just attempted nix build on my mac. seems a hash wasn't specified somewhere?
> nix build
trace: warning: Unknown cryptography version: '41.0.3'. Please update getCargoHash.
error: hash mismatch in fixed-output derivation '/nix/store/q5zdnqdvji51gldarqrnmvr76q50z8gk-cryptography-41.0.3-vendor.tar.gz.drv':
         specified: sha256-AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA=
            got:    sha256-LQu7waympGUs+CZun2yDQd2gUUAgyisKBG5mddrfSo0=
error: 1 dependencies of derivation '/nix/store/0mqas2hvyns7ipciy0c1mfcs9g4jkskm-python3.10-cryptography-41.0.3.drv' failed to build
error: 1 dependencies of derivation '/nix/store/wxr9qh49r05vhk6mkzm06nb85jqhnqm1-python3.10-cashu-0.14.0.drv' failed to build

Any idea where this comes from?

not sure but it disappeared after rebase so all good

seems a hash wasn't specified somewhere?

We're avoiding specifying a hash by overriding cryptography with preferWheels = true so it doesn't try to do a fixed-output derivation. I can reproduce if i don't override the cryptography lib. It's basically this error. Maybe there's an extra mac thing to do?

Hmm ok, what's the workaround? Should I copy this hash somewhere?

idk tbh. This works on linux, not sure why it fails for mac.

Would love this to work on mac as I'm also one of those people!

Could you guys add a git workflow that checks whether the flake works as the repo might get updated?


How Do I Contact Kraken Support Phone Number?

Contact Kraken Support Number + 1(415)-787-9212 by Phone, simply go with This is the official support number where you can get assistance for any issues or questions related to Kraken Account.

Contact Kraken Support Number + 1(415)-787-9212 by Phone, simply go with This is the official support number where you can get assistance for any issues or questions related to Kraken Account.


[Closes SOV-608] Fixed Bug: Security - When a potential validator contacts pool a json is created for it before adding it to the pool

[Closes SOV-608] Fixed Bug: Security - When a potential validator contacts pool a json is created for it before adding it to the pool

I really like the integration with tests in GoCD. Nice work.

@keenondrums see comments. Need to resolve these before accepting the pull request. Also, we should have plenum-PR tests passing.

OK, this is approved. Thanks Andrey.


Styles don't exist in many of the index.css files for the examples

Describe the bug
Many of the React examples don't contain the used styles in the index.css file.
For example the full width resizable table example
none of these styles exist in the index.css file so the example aren't displaying correctly.

Your minimal, reproducible example
https://github.com/TanStack/table/blob/main/examples/react/full-width-table/src/index.css
Steps to reproduce
Open the  React full width resizable table or the full width example. Check the index.css file.
Expected behavior
The table should be full width in the full width examples
How often does this bug happen?
Every time
Screenshots or Videos

Platform
Windows 10, Chrome Version 103.0.5060.134 (Official Build) (64-bit)
react-table version
8.5.2
TypeScript version
No response
Additional context
No response
Terms & Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

Hi, @MA-MacDonald!
Did you manage to make this example correct ? I faced the similar problem and I'm looking for a solution.
Thanks in advance!

@ytnytn1 Yeah if you look in the index.html it looks like they are pulling in tailwindcss which is where those "missing" classes are coming from.


Simplify function to improve performance for long text.

No need to split input into lines when specifically removing lines beforehand. No need to lowercase for each n-gram iteration, either.
This is my first contact with Kotlin so please excuse any obvious mistakes and comment if you have questions/advice.

Codecov Report

Merging #29 into master will decrease coverage by 0.17%.
The diff coverage is 83.33%.

@@             Coverage Diff              @@
##             master      #29      +/-   ##
============================================
- Coverage     59.42%   59.25%   -0.18%
  Complexity      123      123
============================================
  Files            13       13
  Lines           700      697       -3
  Branches        158      157       -1
============================================
- Hits            416      413       -3
  Misses          229      229
  Partials         55       55

Impacted Files
Coverage Δ
Complexity Δ

...om/github/pemistahl/lingua/api/LanguageDetector.kt
72.94% <100%> (-0.14%)
62 <0> (ø)

...pemistahl/lingua/internal/TestDataLanguageModel.kt
72.72% <80%> (-4.2%)
2 <0> (ø)

Continue to review full report at Codecov.

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

@rotzbouw Sorry for the late response. Thank you for your pull request. You are absolutely right, the changes you suggest make perfect sense. I just made the modifications manually because there would be merge conflicts with your pull request. Closed.


take into account only first 3 bits on integer when inverting serializers

I noticed the issue after adding Oj serializer and trying to use Marshal. Instead of original object it would return a string. Turns out serializer couldn't be detected. That's because it would do 1 & 4 in rassoc which is 0. Original implementation works for 3 = "11", and 4="100" works for 4 but cuts off everything below.  Instead we should extract first 3 bytes (7 = "111") from passed integer regardless of serializers array length.
Thanks @epilgrim for helping me out with this one.

Coverage remained the same at 100.0% when pulling 4e8f9d09ba8c366f24fb37d7b8387ba97c233bf0 on kagux:fix/serializer_decompose into 553d45273d179ece53c723cb9b4a827a0c378d5b on sorentwo:master.

Thanks again. Merged locally with 4aaa51a. I'm going to pull in the next PR and do a minor version release.


Add environment option to migration apply command

The Entity Framework migration command currently does not support applying migrations to an alternative connection string. This PR will add an "environment" option to the apply command as shown below:
Usage: ef migration apply [arguments] [options]

Arguments:
  [migration]  The migration to apply. Use '0' to unapply all migrations

Options:
  -c|--context <context>                The context class to use
  -s|--startupProject <startupProject>  The start-up project to use
  -e|--environment <environment>        The environment to use
  -h|--help                             Show help information

The idea being your model uses either "StartupEnvironment" classes or makes use of "config.Environment.json" files, as per http://docs.asp.net/en/latest/fundamentals/environments.html.
You can then run a command like dnx . ef migration apply -e Staging which will pass the environment option to WebHostBuilder and call your project startup with the correct IHostingEnvironment.
Please let me know if any chances are required for a merge, or if the team already has a different direction for this.

Hi @ph1ll, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla2.dotnetfoundation.org.

TTYL, DNFBOT;

@ph1ll, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

Hey,
We are planning to change how all this is implemented because the concept of environments is very ASP.NET specific. Here is the item tracking this work https://github.com/aspnet/EntityFramework/issues/2357. We will revisit environments once we have completed this redesign.
~Rowan


forEach Error on startup

Using newest stable version of discord / betterdiscord

This is the repo for BDv2, please go over to the BD support server for stuff like this.


Any tool to convert Latex code to typst?

Any tool to convert Latex code to typst?If no,I am considering create one.

Pandoc.
https://github.com/jgm/pandoc/issues/8713

Pandoc is the way to go.

Just wanted to chime in here and say that the web app now integrates Pandoc (with a few tweaks). You can drop your LaTeX source when creating a new project have it converted the best we can.

Is there a way to convert more than one .tex file...?

Is there a way to convert more than one .tex file...?

Command line scripting with pandoc

Sorry, should have been clearer: Yes, in the web app
Something like "import & convert .tex file"

Sorry, should have been clearer: Yes, in the web app Something like "import & convert .tex file"

This is likely an issue you should file in https://github.com/typst/webapp-issues
https://github.com/typst/webapp-issues/issues/238#issuecomment-1852003322 There, somebody also mentioned that restriction.


PIF Indicator Problem?

My CC is showing a yellow highlight even though the available is matching the account balance.  (2462.88).  When I fresh load YNAB it is shown correctly with no highlights.  However, after that I adjusted a few goals & WAM'ed a few budgets (not touching the CC category or doing anything that would change the Available number), and that is when it turned yellow.  The only way to fix it is to add a $ to the CC category, making it incorrect, then undoing it, making it correct again.
Windows 10, Chrome

@Flyou10 I'm having some trouble reproducing this one...
Do you have any steps that we can follow to reproduce this consistently? Are you still seeing the issue?

No I am not having this problem anymore.  Thank you for looking into it.

I see this is an older post, but I'm having a similar issue with this. I'm not sure if it's just my account or what, but I have one credit card that we're trying to pay off. It is no longer highlighted in yellow--it's green just like all of our other cards, even though the current CC payment amount is less than the balance on the card. I tried refreshing but it remains green. I should note, it is green in the list of all of the credit card payments, but when you click on the budget line under CC payments, the right hand column that details the CC information does say "available" and the amount in yellow. It's just green in the actual budget.
I'm not as tech savvy as everyone else on here, so I am not sure if I am reporting this correctly. I'd just love for it to go back to yellow so we don't think it's paid in full! The setting is "on" in the toolkit. Thanks for your help!

@Flyou10 @marley412 can y'all verify this is resolved and close the Issue if it is? Thanks.

This remains true for me.

@marley412 can you provide a screenshot showing what you're seeing? Thanks for helping diagnose this.

Here is a screenshot from my computer.  As you can see, next to my "Bank of America" it shows I have 23 charges that are not categorized.  Since they are not categorized, my "Pacing" numbers along the right are gone.  Also, the goal shading is back to normal YNAB behavior.  Example, the "Tithe" category is shaded yellow, instead of blue for the goal not being met.

@Flyou10 I don't think the lack of values in the Pacing column is related to having uncategorized transactions. After looking at the code I can't see any reason why the Pacing values are missing. Did you perchance open this budget after previously viewing another budget? I know some of the features have had issues with that but I don't see the same issues with the Pacing feature. I'm stumped at this point.

I'm not totally comfortable with putting a screenshot of my whole budget on a public forum, but I'm happy to send y'all one privately if that's a possibility. Here's a small picture though. I have overspending on my Citi Double Cash credit card in two categories. One of those categories is medical expenses. When you look at credit card payments, everything looks good because it's green. You have to actually click on it to see the right hand column where it says the available is yellow. It's super weird because it works correctly for our USAA credit card, but it doesn't for the citi or the discover. We don't use the Home Depot card often, so I'm not sure how it works for that one. We put everything on our credit cards, and we are now paid in full for all of them (I just haven't covered our overspending on those 2 categories yet), so it's not as big of a deal now as it used to be. I just wanted to let y'all know. I appreciate all you do for the toolkit--it makes YNAB a million times better!

@marley412 What is the balance of the Citi Double Cash card? I'm wondering if the Available is orange because it is less than the CC balance. Where does the $823.03 in the Available column come from? Isn't that supposed to represent budgeted dollars that have not been spent? That doesn't really make sense though since you've only got .90 budgeted unless there is unspent money from a previous month?
I don't use credit cards so am not real familiar with them.

The 823.03 is budgeted spending. But, because I've overspent on the card, the amount should not be green. There was a 90 cent return on the card, so that's why the -.9 is there. The actual amount on the card is 856.24. That's why $823.03 should be yellow, because it's not the same as the actual amount. I'm not sure if I'm explaining this well, but hopefully that helps!!

I know this is a pretty old bug report, but I've been working my way through the oldest items. At least as far as I can tell, I haven't been able to reproduce this issue. Uncategorized transactions or not, I haven't been able to find a situation where the Available amount is green yet the Available amount doesn't match up properly with the card balance.
I somewhat suspect that this may actually be related to the missing Pacing column -- perhaps there was a Javascript error on the page, potentially unrelated to the PIF indicator feature, that prevented the PIF feature from working as expected.
Due to the age of this report and the lack of updates I'm going to close this bug report. But if you're still having issues with this, please feel free to open a new bug report with your details, and we can look into it in more detail. Thanks!


doc: add 0.44.0 release link to download.md

part of: https://github.com/apache/incubator-opendal/issues/3818

cdn seems not update with source


VecNormalize gamma parameter

I agree that it makes very much sense to apply the same gamma for the reward discounting as well as the decay of the running averages of the VecNormalize:
https://github.com/ikostrikov/pytorch-a2c-ppo-acktr/blob/34b46eac0abd61e7fb20ff78bb2066eecf5ea7d5/main.py#L66
However, if someone uses the code for an undiscounted (gamma = 1.0) reward accumulation (as it is sometimes the case for Monte-Carlo style updates) this becomes problematic. While it makes sense to summarise all undiscounted rewards over the course of one episode, the same is not true for the running mean which accumulates the rewards of all previous steps and episoded.
I suggest an appropriate warning if gamma=1.0 to save other people from debugging this problem

Yes, a warning sounds reasonable.

Due to a change in the VecNormalize code ( https://github.com/openai/baselines/blob/4dc697e670aecc5b9ef540ef95c175258f5da59e/baselines/common/vec_env/vec_normalize.py#L29) this warning is not necessary anymore because the accumulated returns for normalisation are reset after an environment is done. So I think we can close this issue.


WIP Rek orders panel display view

Description
This makes an editable-panel of the orders panel for the office page. It includes loading entitlements as well as info from service member. It should be used to model loading multiple schemas (i.e. orders and service member) in the future.
Special attention

The link to the orders doesn't work and I don't know why. Pointers?
How do I get commas in the lbs?

Code Review Verification Steps

[x] All tests pass.
[x] There are no aXe warnings for UI.
[x] This works in IE.
[x] Request review from a member of a different team.
[x] (TRIAL) Have the Pivotal acceptance criteria been met for this change?

References

Pivotal story for this change

Screenshots

@kilbergr To add commas to the pounds, we should probably add a helper function that can live in Office/helpers.jsx like I did in this PR. That way we can write it once and use it everywhere we need to format big numbers. (This can probably be a bug that we fix later).


[CSL-1663] Use patched version of ekg-stats to fix infrequent updates

See https://github.com/input-output-hk/cardano-sl/pull/1577 for the full context.
This PR switch to be using a patched version of ekg-statsd where we removed the diff optimisation .

@dcoutts can we include this to 1.0 asap? :)

Hopefully the build won't fail like it did for the previous patch, where @jmitchell had to bump the cabal revision for the memory package. If that's the case, I'm gonna force-push so we keep this as an atomic commit.

Merged manually.


Handle Feed refreshing whenever opened through Nav Events

https://reactnavigation.org/docs/en/navigation-events.html

dunzo


⚠ Samsung has degraded performance

In d20be3f, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 1971 ms

Resolved: Samsung performance has improved in f8cf9b4.


Generate local map for msf localization

We appreciate you go through Apollo documentations and search previous issues before creating an new one. If neither of the sources helped you with your issues, please report the issue using the following form. Please note missing info can delay the response time.
System information

OS Platform and Distribution (e.g., Linux Ubuntu 18.04):Linux Ubuntu 20.04
Apollo installed from (source or binary): source
Apollo version (3.5, 5.0, 5.5, 6.0): 8.00

我在进行apollo和carla的联合仿真。为了启动msf定位，我使用脚本msf_simple_map_creator.sh尝试生成local_map。我使用cyber_recorder录制了record文件，然后使用命令/apollo/scripts/msf_simple_map_creator.sh /apollo/record_file/0226/  /apollo/modules/localization/msf/params/velodyne_params/velodyne128_novatel_extrinsics.yaml 10 /apollo/records/map/生成定位需要的地图。
命令运行到最后出错：

在输出目录/local_map/image/000/north/10下，只有这一个文件夹：

里面只有一张图片：

之后我启动了msf定位，但是在仿真的时候没有作用。我使用cyber_monitor发现msf_status, msf_gnss, msf_lidar这些主题是红色的。我想知道错在哪里。
另外还有一个脚本是msf_local_map_creator.sh，这个好像也可以生成local_map，但是运行脚本需要的参数pcd folder, pose file和zone id我不知道要怎么获得。

I have same issue. Did you solve the problem?

No, I haven't solved the problem.
| |
@.***
|
|
@.***
|
---- Replied Message ----
| From | HyeonSeo @.> |
| Date | 02/28/2024 13:10 |
| To | @.> |
| Cc |<EMAIL_ADDRESS>|
| Subject | Re: [ApolloAuto/apollo] Generate local map for msf localization (Issue #15307) |
I have same issue. Did you solve the problem?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

No, I haven't solved the problem. | | @.*** | | @.*** |
…
---- Replied Message ---- | From | HyeonSeo @.> | | Date | 02/28/2024 13:10 | | To | @.> | | Cc |<EMAIL_ADDRESS>| | Subject | Re: [ApolloAuto/apollo] Generate local map for msf localization (Issue #15307) | I have same issue. Did you solve the problem? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>

I used the script msf_local_map_creator.sh with pcd files and pose.txt . I could make the pcd files from .record file using /apollo/bazel-bin/modules/localization/msf/local_tool/data_extraction/cyber_record_parser . You can refer to https://github.com/ApolloAuto/apollo/issues/10158

No, I haven't solved the problem. | | @.*** | | @.*** |
…
---- Replied Message ---- | From | HyeonSeo @.> | | Date | 02/28/2024 13:10 | | To |<EMAIL_ADDRESS>| | Cc | _<EMAIL_ADDRESS>| | Subject | Re: [ApolloAuto/apollo] Generate local map for msf localization (Issue #15307) | I have same issue. Did you solve the problem? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
I used the script msf_local_map_creator.sh with pcd files and pose.txt . I could make the pcd files from .record file using /apollo/bazel-bin/modules/localization/msf/local_tool/data_extraction/cyber_record_parser . You can refer to https://github.com/ApolloAuto/apollo/issues/10158

很抱歉再次打扰你。我想问一下另外两个参数pose.txt与zone id是怎么获得呢？
你使用这个脚本成功生成了msf定位需要的地图了吗？

No, I haven't solved the problem. | | @.*** | | @.*** |
…
---- Replied Message ---- | From | HyeonSeo @.> | | Date | 02/28/2024 13:10 | | To |<EMAIL_ADDRESS>| | Cc | _<EMAIL_ADDRESS>| | Subject | Re: [ApolloAuto/apollo] Generate local map for msf localization (Issue #15307) | I have same issue. Did you solve the problem? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>
I used the script msf_local_map_creator.sh with pcd files and pose.txt . I could make the pcd files from .record file using /apollo/bazel-bin/modules/localization/msf/local_tool/data_extraction/cyber_record_parser . You can refer to https://github.com/ApolloAuto/apollo/issues/10158

很抱歉再次打扰你。我想问一下另外两个参数pose.txt与zone id是怎么获得呢？ 你使用这个脚本成功生成了msf定位需要的地图了吗？

pose.txt is consist of extrinsic parameter between Lidar and ego. Then you have to write your own pose.txt file. In the pose.txt, you need to give numbers of each pcd file's name.

当我使用脚本msf_simple_map_creator.sh生成local_map时出现了这种错误：
在调用MapNodeIndex::get_map_node_index函数时，传入了不合法的n与m参数


Submitted new blog post: 2020-05-11-tutorial-backing-up-kubernetes-clusters-with-k8up.md

This PR submits a new blog post for review: "Tutorial: Backing up Kubernetes Clusters with K8up" with a proposed publication date of Monday, May 11th, 2020.

/assign @kbarnard10

Any news here? Is it possible to get it out soon(ish)?

Any updates @kbarnard10 @bobbytables @onlydole ?

Hi @akosma - Thanks for your submission to the Kubernetes blog! It looks like the content is the same as the K8up tutorial which we consider a repost and unfortunately cannot accept. If you'd like to do an original blog post on K8up we'd be happy to review again. Let me know if you have any questions, but I'm going to go ahead and close this out.


Higher Order Tap - Method Not Found

Larastan Version: 2.9.2
--level used: 6
Pull request with failing test:

Description
When using higher order tapping, I get this errror:
 Call to an undefined method Illuminate\Support\HigherOrderTapProxy<App\Models\Category|null>::update()

Laravel code where the issue was found
$category = tap($category)->update([
    'name' => $name,
]);

Thank you for your report @fotrino!
Could you make sure in the code that $category cannot be null before calling tap?

HI @szepeviktor, just checked & by removing the null option on the public property it does not throw the error.
class EditCategoryForm extends Form
{
-  public ?Category $category;
+  public Category $category;

    public function set(Category $category): void
    {
        $this->category = $category;
    }

    public function store(): Category
    {
        $this->validate();

        return tap($this->category)->update([
            'name' => $this->name,
        ]);
    }
}

Looks like problem solved? Make sure value is non-null before trying to tap it


Maximum value not triggered on dxNumberBox, when copy paste number with length more than 15

Bug Report

Package versions:
devexteme version: 18.2.7
devextreme-angular version: 18.2.7

Steps to reproduce:

You have number from excel file with format accounting like this:  $800,000,000,000,000,000.00
You set minimum value: 1
You set maximum value:<PHONE_NUMBER>99
and then copy-paste the number into numberbox, but the maximum value not triggered.
Current behavior:

the maximum value can't trigger when user change the value on dxNumberBox
Expected behavior:

the maximum value can trigger when user change the value on dxNumberBox
Screenshots:

here for demonstrate the bug: https://gph.is/g/E06NQ0V
here the link codesanbox: https://codesandbox.io/s/m07ue
Thanks

is there any answer?


Changed 'export =' to 'export default' to support typescript with target ES6

When using the jquery typescript definition file and compiling on 1.5.0-alpha to target ES6 it throws the following error:
error TS1203: Export assignment cannot be used when targeting ECMAScript 6 or higher. Consider using 'export default' instead.
This change will fix that. Hope I don't break anything, first time doing a pull request.

note https://github.com/Microsoft/TypeScript/issues/2929


put link to ToS below the 'Connect Github' button in Github Flow

Many repos don't get to the ToS because they are at the end of the flow. We should add them below the button so everyone sees them. add: Please read or Terms of Service here

Fix is live https://opencollective.com/opensource/apply


Add a script that scrapes the resonance repo and creates figure directories for the assignment deliver

I changed the directory setup so this isn't as much of an issue.


feat(mantine): add mantine

Please provide enough information so that others can review your pull request:
We will merge this PR, but we need to make a few updates before the release. Thanks for your hard work @alicanerdurmaz !!
Self Check before Merge
Please check all items below before review.

[ ] Corresponding issues are created/updated or not needed
[ ] Docs are updated/provided or not needed
[ ] Examples are updated/provided or not needed
[ ] TypeScript definitions are updated/provided or not needed
[ ] Tests are updated/provided or not needed
[ ] Changesets are provided or not needed

I really think that we should not ship without refine-mantine-form or at least without a release plan for refine-mantine-form which wraps @mantine/form. React-hook-form is good to go but still @mantine/form will be a missing piece.
I was going to say the same thing for the datagrid/table component but with this comment here https://github.com/mantinedev/mantine/discussions/1057#discussioncomment-3163669 So, maybe we can just use a custom table or use mantine-data-grid which uses react-table-v8 and at first glance looks like it's going to be compatible with out react-table package and combining these two will be a breeze.


virtualbox: check state before attempting start

This checks the state of the machine before attempting a start (and only starts if necessary).
Fixes #593

/cc @nathanleclaire

LGTM

@nathanleclaire assuming you mean start a running instance in DO, i agree -- we will need to put this in the Driver spec and rely on driver maintainers to make sure it's uniform.  thx!


Patch out Web Environment Integrity

Is your feature request related to a problem? Please describe.

So Google has added the highly-controversial Web Environment Integrity into Chromium sources. You can get a rough guess about exactly how bad it is looking at the Issues of the repo. It is already in there, and it has made its way into Kiwi as well. Do it right now, open Dev Tools and try to call navigator.getEnvironmentIntegrity. If it's not null undefined, we're in trouble.
Why exactly we're in trouble, you may ask? Well. One of the core features of Kiwi is:

Block invasive ads with our powerful Ad Blocker. Enable and manage from options ( ⋮ ) -> Settings -> Ads. Browse without distractions with our Pop Up Blocker!

... an ad blocker. Google is literally killing that by making sure websites can now look at your browser and see if anything EVEN TRIES to modify the website. Devtools? Stylus? Consent-o-matic? GONE. If we keep WEI anywhere close, this might be the end of usable internet.
Describe the solution you'd like

Patch out the WEI. Vivaldi does not have it - tho that's most likely because its a month old, but still. Maybe at some point we can just fake it, as most people do with user agents and whatnot. And as some people do with SafetyNet (which shares the same concept, by the way).
Describe alternatives you've considered

I've tried to use Firefox instead. Not having desktop-like tabs even on a 15 inch screen is horrible. Having to manually add every remotely interesting extension to a collection is boring and painful. And don't even get me started on the design.
Additional context

One thing I found when looking for the sources is the fact that the integrity stuff apparently actually does calls to SafetyNet. Yeah, this might be a hu-u-uge issue for people who want to use adblocker with any amounts of money. At all.
At the moment of me writing this, Android is the only supported platform, too.
The integrity stuff itself is stored right here in the sources.

Keep yourself alive and your environments free!

Hey,
I'm aware of WEI.
It's an horrible idea, like other recent ideas from non-engineering teams at Google, eg. Privacy Sandbox.
The plan, at least for now, (depending how it evolves on the Google-side) is to make Kiwi to appear trusted to websites, even if it involves artificially claiming to the website it has not been altered.
For example, by re-sharing attestation tokens, or by providing an alternative mirror-DOM to the JS-side, if an adblocker is running (e.g. run the page fully, and then hide elements during rendering / non-visible phase).
Google already did a DRM-like approach with Widevine and it's horrible for 3rd party browsers (and the Widevine team is not helpful at all).
In general, I don't think WEI will get popular, because websites won't adopt it as they would loose audience.
So I feel this is a non-issue, at least in the short-term, but I want to keep an eye on it.
I think you could reach to https://open-web-advocacy.org/ if you want to be active against WEI in general - these people have been incredibly helpful for all regulations questions :)
They are the one who managed to "convince" Apple to open to new web browser engines.
Arnaud.

Well, there are websites that just deny your presence on them with WEI. Seek no further than one of my own - with a simple one-liner you can just deny a browser that has WEI from accessing the website. Guess if we're faking it, maybe a toggle (per-website, but a global one would be fine at the very least) that changes states between "We are trusted" and "We have no WEI" would be a good idea?

Obviously this is a very good candidate for chrome://flags but I still think that the main point is to make WEI adoption as painful as possible to adopters, so they swallow their own poison.
If we can render WEI useless, I think this will be a much stronger signal than staying on the side.
I am saying this because in bug reports, this is ultimately the browser's fault otherwise. "It works on Chrome, but not on Kiwi, this is Kiwi's fault" and I want to avoid such case.
I have the intuition that if Firefox and Safari don't adopt WEI (and I'm a bit worried they will, because they are completely under influence from Google, but it's not that sure) that WEI is going become a useless standard.
In such case, if WEI is requested, why not reload such sites pretending to be Firefox or Safari right :) ?

by providing an alternative mirror-DOM to the JS-side, if an adblocker is running (e.g. run the page fully, and then hide elements during rendering / non-visible phase).

This defeats the purpose of running an ad blocker. I don't care about advertising; I don't want to be tracked around the web, by Google or otherwise. I think you'll find a lot of people feel the same way.


Azure UI is only showing dots on the 'feature manager' pane

Something seems to wrong in the Azure UI.
We're only using AppConfig for feature management, and I have 44 keys.
My keys are still there but I cannot see them properly.
Under "feature manager", it lists the keys but without any columns showing values:

Under 'config explorer' it's empty:

Thanks!

Thanks @RichardChen820 , it's back working now.


"bad" files caused project export to fail

If input files could not be processed for some reason, they should just be ignored. Project file creation should still work

Fixed in release <IP_ADDRESS>


events: Add test for removing custom profile field data.

Follow-up noted in #22103, see this comment for particular context.
Adds a test for removing the value set for a user's custom profile setting without adding another value.
When adding the test coverage, found two bugs in the events logic for the "realm_user" case in apply_event:

First, the updates to these custom profile fields, as well as other fields in the event's "person" object, were being done in a loop over the fields in the state's "raw_user" object, which was unnecessarily making the same update multiple times.

So the first commit unnests those updates so that they are only updated once.
The commit that introduced the nesting was  https://github.com/zulip/zulip/commit/649fccde6ba438a213c6c43688462e3506563019

Second, in the case that the field value was being removed, the user object's "profile_data" was being updated to have None as the value, instead of removing the field ID from the "profile_data".

So the second commit, fixes that logic to remove the field ID from the "profile_data" if present. And adds the test coverage for these events.
A similar bug in the events logic was fixed in commit https://github.com/zulip/zulip/commit/96c61a1a41ecc911fc442513a6164a911b15f700 for when a custom profile field is removed from a realm.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Looks good to me. CI failure seemed to be caused by a network problem, so re-ran it.
@timabbott ready for your review.

Merged, thanks @laurynmm and @sahil839!


RSpec requirement ?

Hello
I'm fairly new to Rails and system tests so the following might be totally obvious :)
Does magic_test only work for Rspec tests?
From reviwing https://github.com/bullet-train-co/magic_test/blob/main/lib/magic_test/engine.rb it seems that if Rspec is not defined magic_test wont configure correctly?
The somewhat longer request:
I got Capybara running for system tests in docker  using mini_test and to my best of knowledge solving all issues mentioned in #21 . I can write system mini_tests and run them headless in the docker image screenshots in tmp and all.
When adding in magic_test my mini_tests runs as before,  but i get some errors from running magic_test in the docker image:
`Frame number: 0/27
From: /bundle/vendor/ruby/3.0.0/gems/magic_test-0.0.9/lib/magic_test/support.rb:84 MagicTest::Support#magic_test:
78: def magic_test
79:   return unless ENV["MAGIC_TEST"].present?
80:   empty_cache
81:   @test_lines_written = 0
82:   begin
83:     magic_test_pry_hook
=> 84:     binding.pry
85:   rescue
86:     retry
87:   end
88: end
Frame number: 1/27
Frame type: block
From: /app/test/system/basics_test.rb:7 BasicsTest#test_getting_started:
4: test "getting started" do
5:   visit root_url
6:
=> 7:   magic_test
8: end

`
Any chance of a qualified guess what causes this issue?

Im closing this issue to open an other as i have solved everything here.
No RSpec requirement, that was just me not looing in the gem.spec.
The pry binding issue is solved in docker by:
tty: true
stdin_open: true
So no i have a docker image that binds on pry for magic_test.


Check Bashism Unit test isn't working

Describe the bug
The bashism is checking random files, look at #699 and its unit test to see it checking a different sh file.
Screenshots

Checklist

[x] I checked for duplicate issues.
[x] I checked already existing discussions.
[x] This issue is not included in the roadmap.
[x] This issue is present on both stable and development branches.

AGAIN?? COME ON I REMEMBER IT WORKING AFTER THE LAST PR

/label 'github actions'

Fixed.

I can feel my mental state level dividing by 5 every time this workflow doesn't work.


Question about Mobx and Redux

Hi Jean-Yves,
I've recently started to work on React using Next.js and Mobx. I'm currently working on a prototype for a dashboard so your project is a great help.
I was thinking that Mobx is fulfilling the same purpose as Redux but in a different way. I'm confused then, any clarification is welcome :)
Regards

Redux is a kind of flux pattern while Mobx is a state management library. So if you like flux pattern, you can use Redux. Remux is another example of a flux pattern, but using Mobx to manage state. At the end, I found that using a flux pattern for a very complex software is a good think because all the events are flowing in one direction. It is more easy to maintain.
For small application, you can use Mobx without any flux pattern. It will be faster to develop.


Replace error serialization for the journal

Originally raised: - https://github.com/NomicFoundation/ignition/pull/279#discussion_r1244294359
We are not capturing the stack, hence we are tricking ourselves. We are really just storing the error message.
We should have a set of more specific error states rather than storing the Error.
TODO

[ ] Encapsulate thrown errors from onchain-dispatcher
[ ] Add everything error for each onchain dispatcher function
[ ] Document in types each known error and deal with it
[ ] Which can be simulated in integration tests

If there is an exception we should kill the process. If it is an expected error condition we return it into a value and return it.
We should work with return values, with exceptions as not expected.
Ideally we would have a list of known coded error states (not just the generic Error object).

Start with: revert and hash out more iteratively.

This has already been implemented in the development branch


nd_item get_group_linear_id() not working

Bug summary
Using nd_range and nd_item with the get_group_linear() on GPUs seems to be not computing anything.
To Reproduce
#include <sycl/sycl.hpp>
#include <vector>
#include <iostream>
using namespace sycl;

int main()
{
    // Running on CPU works, NVidia/AMD both don't seem to work
    // sycl::queue Q = sycl::queue(sycl::gpu_selector());
    sycl::queue Q = sycl::queue(sycl::cpu_selector());

    std::cout << "Running on: " << Q.get_device().get_info<info::device::name>()
              << std::endl;

    // Unimportant setup of variables and vector<int>
    // Declare size and dimensions
    constexpr size_t size = 5;
    constexpr size_t dimX = 64, dimY = 64, dimZ = 64;
    std::vector<int> values(size * dimX * dimY * dimZ, 0);

    {
        // size of local range
        constexpr size_t B = 64;
        // Create buffer associated with input
        buffer<int, 3> values_buff = make_async_writeback_view(values.data(), range<3>{size * dimX, dimY, dimZ}, Q);

        // Submit the kernel to the queue
        Q.submit([&](handler &cgh)
                 {
            accessor acc_values{values_buff, cgh, write_only};

            // declare global and local ranges
            range global{size * dimX, dimY, dimZ};
            range local{B, B, B};

            // Each block of 64*64*64 should be set to it's work group id
            cgh.parallel_for(nd_range{global, local}, [=](nd_item<3> it)
                            {
                                acc_values[it.get_global_id()] = it.get_group_linear_id();
                            }); });
    }
    // Wait for writebacks
    Q.wait();

    // Should display the work group in ascending order
    for (int index = 0; index < size; ++index)
    {
        std::cout << "index " << index * dimX * dimY * dimZ << " with value " << values[index * dimX * dimY * dimZ] << std::endl;
    }
}

Expected behavior
As it can we seen when running on a CPU, it should output first item of each 64^3 block, which holds the value of the new workgroup id in linear ascending order, e.g.:
index 0 with value 0
index 262144 with value 1
index 524288 with value 2
index 786432 with value 3
index 1048576 with value 4

Describe your setup
Currently using hipsycl 0.9.1.
Running on Ubuntu 18.04.1, cuda 11.8.89, llvm 13.
hipSYCL was built with clang13 and I compiled the provided examples with an adapted version of your CMakeLists.txt example
cmake .. -DhipSYCL_DIR=/opt/hipSYCL-llvm13/lib/cmake/hipSYCL/ -DHIPSYCL_TARGETS="omp;cuda:sm_75"
Kind regards

In sub_group.hpp I found the get_group_linear_id() definition uses __ffs which is a cuda function, may this be part of the issue?
  HIPSYCL_KERNEL_TARGET
  linear_id_type get_group_linear_id() const {
    __hipsycl_if_target_hiplike(
      // Assumes __hipsycl_warp_size is power of two
      return local_tid() >> (__ffs(__hipsycl_warp_size) - 1);
    );
    __hipsycl_if_target_spirv(
      return __spirv_BuiltInSubgroupId;
    );
    __hipsycl_if_target_host(
      // TODO: Is this correct?
      return 0;
    );
  }

Most likely the issue is that you launch a kernel with a massive group size of 646464, which is almost certainly not supported by your hardware. CUDA devices typically support local sizes of 1024, maximum.
There is currently a deficit in hipSYCL where it does not report kernel launch errors due to group sizes exceeding the hardware limits in some compilation flows. In that case, it just fails silently.
So I believe from the hipSYCL perspective this is a duplicate of #828. To solve the issue for you, it should be sufficient to choose a local size that is supported by the hardware :)

Thanks, that makes sense., although a bit unfortunate it fails silently.
In my case I kind of require a size of 64^3.
I've read a bit more about that topic and found your slides/post on scoped parallelism in hipsycl.
As far as I've understood, scoped parallelism can be used in a similar way as nd_range. So I could use that to break down my problem into logical work sizes that exceed physical work sizes.
hipSYCL would automatically split my logical size into fitting physical sizes, is that correct?
Kind regards

While it is possible to implement scoped parallelism in that way, I think the current implementation assumes 1 physical work item per 1 logical work item on GPU for performance reasons (on CPU, it is different). So it would probably not help you.
Why do you need to use 64^3 items per work group? Do you really need synchronization across this many items? Otherwise, there is little reason to use nd_range parallel for.
Note that you can easily implement the behavior you describe yourself (example in 1D for clarity):
parallel_for(nd_range{global, physical_local}, [=](nd_item<1> it){
  for(int i = 0; i < num_logical_items_per_physical; ++i){
    int logical_local_id = it.get_local_id(0) + i * it.get_local_size(0);
  }
});


File not found when using Azure direct access

File not found when using Azure direct access. Anyone get it working?

var AzureStorageAdapter = require('parse-server-azure-storage').AzureStorageAdapter;
var filesAdapter = new AzureStorageAdapter(
           process.env.AS_STORAGENAME,
           process.env.AS_CONTAINERNAME,
           options);
var api = new ParseServer({
    databaseURI: databaseUri || 'mongodb://localhost:27017/dev',
    appId: process.env.APP_ID || 'myAppId',
    masterKey: process.env.MASTER_KEY, //Add your master key here. Keep it secret!
    serverURL: serverURL,
    fileKey: process.env.FILE_KEY,
    filesAdapter: filesAdapter,

This package only handles un-secure files. Stop

Where is options variable?


Editing user added local songs causes the local song's file to be deleted

in NongAddPopup::addLocalSong the code attempts to copy songPath into destination, so it fails when they are pointing to the same file. They are pointing to the same file when editing a local song.
The code fails because:

It attempts to delete destination and copy songPath into it. But when they are the same file then songPath is also deleted.
The copy function doesn't allow both inputs to point to the same file

The fix is checking with std::filesystem::path::compare if they are the same path. If it is the same path, then skip copying.
Also, I replaced the "exists then remove" with the copy option: std::filesystem::copy_options::overwrite_existing.
And removed the std::string error = ""; line. I think it's useless

I think the logic here was meant to handle the case where you want to change the song file too. Not sure I ever tested it though.

oh wait copy_file can overwrite the destination, never knew that


Deploying to Heroku

Hey.  Thanks for this wonderful system.  I'm having some issues deploying to Heroku.  Following the instructions https://github.com/jedireza/drywall/wiki/Deploying-to-Heroku.
Everything works fine if I run it locally
node app.js

But I get an Application Error when deployed to Heroku (https://evening-shore-8405.herokuapp.com/).
Looking at the logs, this is what I get http://pastebin.com/TgN0gRQ6
Any help would be much appreciated!

Thanks for this wonderful system.

Thank you! :wink:

I'm having some issues deploying to Heroku. Following the instructions https://github.com/jedireza/drywall/wiki/Deploying-to-Heroku

Woof, we really need to update that guide.

It looks like you're missing a module possibly:
Error: Cannot find module 'iconv-lite'

This could happen if your dependencies are in devDependencies since by default on Heroku, npm only installs production assets. Here are some links that may help.

https://github.com/jedireza/drywall/wiki/Dependency-%26-Asset-Check-ins
https://devcenter.heroku.com/articles/nodejs-support#customizing-the-build-process

Also, I recently updated the guide for Aqua about deploying to Heroku, although it's a different project, the concepts are still relevant.

https://github.com/jedireza/aqua/wiki/Deploying-to-Heroku

I hope this helps. Keep us posted.

Thanks for the prompt reply!  This was due to my ignorance on how to use Heroku -- 1st time using it).  I knew it was related to iconv-lite but I didn't know how to go about solving it exactly.  I had followed the instructions on https://github.com/jedireza/drywall/wiki/Dependency-%26-Asset-Check-ins even before posting this issue to no avail...
For those who run into the error for deploying to Heroku.  Inside the drywall directory do a:
npm install iconv-lite

Which will install the module inside the node_modules sub-directory.  Then:
git add node_modules

It'll add everything new inside of the node_modules sub-directory.  And finally:
git commit -m "adding additional dependencies"
git push heroku master

And you should be good to go!

This is not a great solution. I would rather move that package from devDependencies to dependencies in package.json!

Hi @benmarten :wave:
Which solution are you referring to? There are actually many ways to go about this that I mention here:
https://github.com/jedireza/aqua/wiki/Deploying-to-Heroku#dependencies-and-build-files
Is there something else you think we should mention?


[Bug]:

Your current environment

The output of `python collect_env.py`
Your output of `python collect_env.py` here

Model Input Dumps
No response
🐛 Describe the bug

When initializing the ModelConfig, the get_sentence_transformer_tokenizer_config method is executed. However, this method cannot directly or indirectly pass the token variable, making it impossible for me to provide a token to retrieve the config

The token parameter was declared, but no argument was passed.
Before submitting a new issue...

[x] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

I think this argument is not necessary as the recommended way is to set the HF_TOKEN environment variable instead. cc @flaviabeo

@DarkLight1337 can you assign this BUG to me? I can work on this


native_minecraft_version and supported_minecraft_versions return null.

Hey! While experimenting more with the API (and using it in something I'm developing) I realised that some resources return null on native_minecraft_version and supported_minecraft_versions while others return an empty string on native_minecraft_version and an empty array on supported_minecraft_versions (expected behaviour).
Of course, this only happens on resources that don't have this information.
Such as these 2 resources:
Error: https://www.spigotmc.org/resources/269/ (https://api.spigotmc.org/simple/0.2/index.php?action=getResource&id=269)
{
  "id": "269",
  "title": "BungeePlayerCounter",
  "tag": "Show player's count of each server in a scoreboard",
  "current_version": "1.5.2 beta",
  "native_minecraft_version": null,
  "supported_minecraft_versions": null,
  "icon_link": "https://www.spigotmc.org/styles/default/xenresource/resource_icon.png",
  "author": {
    "id": "7517",
    "username": "AlphartDev"
  },
  "premium": {
    "price": "0.00",
    "currency": ""
  },
  "stats": {
    "downloads": "22408",
    "updates": "11",
    "reviews": {
      "unique": "42",
      "total": "43"
    },
    "rating": "4.45238"
  },
  "description": "(...)"
}

Expected behaviour: https://www.spigotmc.org/resources/313/ (https://api.spigotmc.org/simple/0.2/index.php?action=getResource&id=313)
{
  "id": "313",
  "title": "BungeeTabListPlus",
  "tag": "Customize your tab list",
  "current_version": "3.4.4",
  "native_minecraft_version": "",
  "supported_minecraft_versions": [],
  "icon_link": "https://www.spigotmc.org/data/resource_icons/0/313.jpg?1455645578",
  "author": {
    "id": "9041",
    "username": "CodeCrafter47"
  },
  "premium": {
    "price": "0.00",
    "currency": ""
  },
  "stats": {
    "downloads": "106209",
    "updates": "114",
    "reviews": {
      "unique": "311",
      "total": "356"
    },
    "rating": "4.63987"
  },
  "description": "(...)"
}

I'm not sure what is the cause.

Ohhhh, okey! Thanks for the clarification.

This could be improved to be made more consistent, I will look into it.

@simpleauthority you may want to always return empty string and empty array (or null) if they're not set and add a property "set_before_versions" of some kind or sth.

I feel like that returning an empty String is.... bad.
The native_minecraft_version should imo always return null when never set or unset and supported_minecraft_versions should always return an array, either empty or filled.
This inconsistent behaviour with null and non-null values is only confusing and causes us to have extra work.


get member function includes disabled AWS accounts

The get_member function should skip disabled AWS accounts as those are no longer available.
https://github.com/aws-samples/aws-security-hub-cross-account-controls-disabler/blob/fd04bd2780a96e53cb350a5812db62f4ab0c2371/UpdateMembers/src/GetMembers/index.py#L102-L103

fixed!


Browser throws an error when runing pxt serve for pxt-microbit

Following the instructions in the pxt-microbit README, after running pxt serve, chrome browser is opened but it throw some error on the browser's console as follow:
GET http://localhost:3232/blb/target.js 404
Uncaught TypeError: Cannot read property 'pxtCrowdinBranch' of undefined
    at http://localhost:3232/blb/main.js:1925:122
    at <anonymous>
From previous event:
    at HTMLDocument.<anonymous> (http://localhost:3232/blb/main.js:1914:10)
    at i (http://localhost:3232/blb/jquery.js:2:27151)
    at Object.fireWith [as resolveWith] (http://localhost:3232/blb/jquery.js:2:27914)
    at Function.ready (http://localhost:3232/blb/jquery.js:2:29707)
    at HTMLDocument.J (http://localhost:3232/blb/jquery.js:2:29892)

Make sure you're using the v0 branch of pxt when running pxt serve for the microbit target.


Fix pgsql connection error on circle ci

Added PGHOST: localhost
Used direct URL for DB instead of -h, --user etc.

Manually merged into master, works like a charm!


fix(deps): bump braces and micromatch to avoid vulnerabilities

References:

https://github.com/micromatch/braces/pull/40
https://github.com/micromatch/micromatch/issues/249

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket: