Subscription Client API changed

I was trying to use GraphiQL Subscriptions Fetcher and I realised that the API for the Subscriptions Transport WS had changed.
So now they deprecated the subscribe function and instead give a request one that returns an observable. My solution is this:
subscription = subscriptionsClient.request({
  query: graphQLParams.query,
  variables: graphQLParams.variables,
});

observable = subscription.subscribe((result, err) => {
  if (err) {
    observer.error(err);
  } else {
    observer.next(result);
  }
});

And to unsubscribe you just do observable.unsubscribe().
Currently I can not open a PR, but feel free to use my code or even help me improve it.

when will be fixed?


Color picker in code editor is broken

Godot version
4.0 4ac993ffe2
System information
Mint 10
Issue description
Color picker in editor is broken. It replaces whole line with color params.
Steps to reproduce

Have piece of code with color, in my case:

func _draw() -> void:
	draw_circle(_vertex, _radius, Color.RED)

Right click on color, choose color picker from context menu
Change color
Instead of replacing color, whole line is replaced with color params:

func _draw() -> void:
(0.7421875, 0.7421875, 0.7421875)

As a bonus you can't revert this operation
Minimal reproduction project
No response

btw it works fine if color is already defined not via constant but via args, for instance using color picker with this piece of code :
	draw_circle(_vertex, _radius, Color(0.30042162537575, 0.82421875, 0.44441643357277, 0.69999998807907))

will work fine


refine a new api in redis

This PR is updating a new API added in https://github.com/Azure/azure-sdk-for-net/pull/44847

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.Redis


Frontier Silicon entity id unreliable

Home Assistant release with the issue: 0.82.0

Last working Home Assistant release (if known): -
Operating environment (Hass.io/Docker/Windows/etc.): Docker

Component/platform: media_player.frontier_silicon

Description of problem:
My Frontier Silicon (a Revo Heritage) gets one of the device ids media_player.unnamed_device or media_player.revoheritage on restart seemingly at random.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
media_player:
  - platform: frontier_silicon
    host: <IP_ADDRESS>
    port: 2244

Issue persists on 0.89.1. The only thing that changed is that frontier_silicon components can't have names since 0.88.
Config snippet:
media_player:
  - platform: frontier_silicon
    host: <IP_ADDRESS>
    password: ...

In "Unused Entities in Lovelace":

After restarting ten times or so, it finally shows up correctly:

with the (automatically assigned) name media_player.konspirative_kuechenkonzerte


Web - Payment - An error when trying to mark the withdrawal account as the default

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go URL https://staging.new.expensify.com/
Login with Expensifail account
Go to Settings->Workspace->Connect Bank account->Connect with Plaid
Choose Fidelity bank
Enter your credentials and company information
After added new bank account go to Settings->Payment
Try to mark the withdrawal account as the default

Expected Result:
The payment method should be marked as the default
Actual Result:
An error when trying to mark the withdrawal account as the default
Workaround:
Unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: N/A
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/197309667-c22aad82-d57a-4c56-8f51-34cf4554f3fb.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Error messages come from the API so I think this should be Internal

Given that this is a bug, I think we should have a BZ team member assigned. Adding the auto-assigner.

Looks like that didn't work. Going with triage instead.

Looks good - adding the engineering label since we're handling internally.

Ah, apologies - it's been added!

To be clear then @JmillsExpensify, given this discussion, should I add the demolition label?

Yes, exactly!

This one was just assigned to Monil and so I'm sure he'll be able to get some movement in coming days! Will leave this to @miljakljajic @MonilBhavsar to help drive forward 👍

I'll take a look

@MonilBhavsar did you find anything?

Was able to reproduce the issue
The error for this particular use case is thrown from here. Logs
This error is thrown when we account is not in OPEN state or of type PERSONAL. In this case it is an OPEN account, but is of type BUSINESS.
Chatting with @MariaHCD to figure it out since she has more context about Bank accounts and wallet.

When user adds VBBA by going to Workspace > Connect Bank account.
We display that bank account in Payments Page (Profile > Payments) and on tapping that account, an option to make it a default payment method. But we have a backend validation where we can't set business bank account as default method.
We can fix this by not displaying "Make default payment method" in Payments page for Business Bank account. Thoughts?

^ Worth a discussion in #product?

Yes, I think that's a good discussion for #product. I also like the solution, but let's get more conversation going to confirm we all agree that's the best page. For instance, we also might think about adding "badges" to the bank accounts so that it's more clear why some bank accounts (e.g. biz) are treated differently than others (e.g. personal).

For instance, we also might think about adding "badges" to the bank accounts so that it's more clear why some bank accounts (e.g. biz) are treated differently than others (e.g. personal).

Oooh nice idea! I'll stay tuned for that #product discussion 🤓

@michaelhaxhiu Please join https://expensify.slack.com/archives/C03U7DCU4/p1668596349084729

PR is merged!

I think we can close this, or we have to complete the checklist?


plot filter labels

New default, e.g., pg.filters.SimpleTight(G, 4).plot().

Without labels (as before) with pg.filters.SimpleTight(G, 4).plot(labels=False).

Coverage increased (+0.3%) to 88.53% when pulling 171a1389ff39351fe6b1eeb85a9632fee4eb863f on plot-labels into cb06eaf578938bb5d097db9797cf165a572eeeb3 on master.


drop this because we do have timezone in all data$reportMetadata$time…

…zone now

HA - thanks for catching that! I completely forgot to remove that

no prob, was looking at the timezone on dvhydro and searching for "timezone" to see all instances of its use in dvhydro, and this just happened to come up on top of the find in files list in R Studio :D


How to deploy multi-nodes training without hostfile?

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
I cant get the IP for each node before I started the training. The GPUs are allocated when you are starting a task, so I can't set the hostfile before. Can I use the multi-nodes training just like torch did? Just pass the master_addr and master_port?

You can do one of two things:

Generate the hostfile using information from your job scheduler (e.g. slurm) that can be parsed by your deepspeed multinode-runner of choice (i.e. one of the following: https://github.com/microsoft/DeepSpeed/blob/master/deepspeed/launcher/constants.py#L6-L9). An example of this for Slurm and openMPI would be:

#!/bin/bash
hostfile=/fsx/$(whoami)/hostfiles/hosts_$SLURM_JOBID
rm $hostfile &> /dev/null # for consecutive calls to this script in interactive jobs

# Iterate over the nodes allocated to your job
for i in `scontrol show hostnames $SLURM_NODELIST`
do
    # For each node, add num_gpus (8 here) as hosts
    echo $i slots=8 >>$hostfile
done

Alternatively, if your job scheduler provides a launcher (e.g. Slurm has srun), you can add that as a DeepSpeed multinode runner. We have already added a multinode runner with slurm/srun at https://github.com/microsoft/DeepSpeed/pull/2404 which you're encouraged to use.

Hi @Quentin-Anthony @StellaAthena ,
Can't find any documentation on how to use "SLURM Multinode Runner". Can you please provide more information about


[Bug - Edge Addons] Unable to add language on store listing to have description on more than 1 language

Describe the bug
I'm unable to add language on Store Listing to have more than 1 language although 6 languages are detected on package.
To Reproduce
Steps to reproduce the behavior:

Go to "Store Listings"
Try to click on "Add a language"
It's disabled

Expected behavior
Being able to add language on store listing to have description on 6 languages.
Screenshots

Desktop (please complete the following information):

OS: Mac OS X
Browser chrome and firefox
Version firefox 127 / chrome 126

Hi @mailinblack-app ,
Thank you for reporting this. We are looking into this and will let you know as soon as we have an update.

Hi @sjoerdspindle, thank you for your patience.
Our engineering team has confirmed that when a package contains multiple languages, the Listing page will automatically populate those languages. The “Add a language” button is intentionally disabled to prevent unnecessary additions. It will only become active if a language is deleted from the UI and needs to be added back.
Please let us know if you have any further questions or need additional assistance.

Thank you for getting back to me!
I'm afraid my issue/bug report was misunderstood.
I'm unable to change/update the extension name and description. We want to translate and localize the description for each language/country.
Please see the attached screenshots below. On the Packages page it mentions the languages as expected ("Languages in package: German, English, Spanish, French, Dutch, Portuguese (Portugal)"), but on the Listings page I am unable to change the description, nor does the dropdown menu to Add a language become available.
And yes, we did set a default_locale in the manifest.
I hope you can help out.

Hi @sjoerdspindle, thank you for your detailed response and the screenshots. I apologize for any misunderstanding.
To clarify, you are trying to update the extension name and description for each language/country on the Listings page, but you are unable to do so. Although the languages are correctly listed on the Packages page and you have set a default_locale in the manifest, the dropdown menu to add a language is not available, and you cannot change the description.
I will relay this information to our engineering team to investigate further. We appreciate your patience and will get back to you as soon as we have more information.
Thank you for your understanding.

Indeed, your summary is totally correct. A couple weeks ago it did show the default_locale (which is Dutch in our case) and the description, but the other languages were not available or selectable. Now even Dutch, our default_locale, is not even visible.
We do use the Publish API to update packages, but even when I add the package as a .zip-file manually they are still not selectable unfortunately.

@sjoerdspindle, could you please provide us with your product ID? Additionally, if possible, could you share the package you are experiencing issues with?
Thank you for your cooperation.

@sjoerdspindle from my experience, they will only show up when messages used in the manifest.json file show up in the specific messages.json files. They don't have to be different from your default_locale. However they need to be specified to be picked up by the Microsoft Edge Store.

@ManikanthMSFT Does Microsoft want to reconsider their current approach? Especially for webExtensions with partially translated languages (which is what most of us deal with), the current setup is problematic.
For some of the webExtensions, some messages from message.json have to be removed to prevent the Edge store from picking up the locale and force to include a full store description for that specific locale. Dropping this requirement and allow a fallback to the default_locale would be very welcome. As would be the edge store actually looking for the list of locales in _locale files instead of just relying on the manifest.json.


Some videos not working in safari on ios 15 version.

Be sure to search for your issue before opening a new one.
Current Behavior
Some videos not working on ios 15 safari. Not all videos are not working.
It doesn't work only in certain locations. However, pure HTML5 video tags work.
(The control bar moves!)
Exactly the same tag I am using. The only difference is the video name.
<ReactPlayer
  url={require("public/videos/video.mp4")}
  loop
  playing
  muted
  playsinline
  config={{
    file: {
      attributes: {
        preload: "auto",
      },
    },
  }}
/>

Expected Behavior
It should work fine on ios safari with no video stuttering or screen freezes.
Steps to Reproduce

That's what I posted above.

Environment
ios 15.1, 15.2 mobile safari

Hi!
I've had a very similar problem since the iOS 15 update.
My problem is quite different, I have to process base64 (representing mp4) and display it in a player :

First I transform the base64 in an array of bytes and return it from the function in Blob
Then I create a local URL with the Blob and I use it as the url of React-Player

base64 to Blob function :
const b64toBlob = (b64Data: string, contentType = "", sliceSize = 512) => {
  const byteCharacters = atob(b64Data);
  const byteArrays = [];

  for (let offset = 0; offset < byteCharacters.length; offset += sliceSize) {
    const slice = byteCharacters.slice(offset, offset + sliceSize);

    const byteNumbers = new Array(slice.length);
    for (let i = 0; i < slice.length; i++) {
      byteNumbers[i] = slice.charCodeAt(i);
    }

    const byteArray = new Uint8Array(byteNumbers);
    byteArrays.push(byteArray);
  }

  return new Blob(byteArrays, { type: contentType });
};

Usage :
 const blob = b64toBlob("MYBASE64...", "video/mp4");
 const blobUrl = URL.createObjectURL(blob); // React-Player url

Maybe you can try to turn your local file into a Blob and create a local URL like I do with base64 ; A lot of extra operations but this is the only solution I found at the moment 👀
I hope this will be useful for you!


432 oracle

Add a more user-friendly message when unable to connect to a target database.
Add object status to extract.
Exclude view columns from the column data type extractions.
Force environment defaults to override any sites-specific settings in login.sql or glogin.sql.


Recommend cdnjs instead of jsdelivr ( or both are provided ) in document ?

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
My website has layout problem because of SSL certificate error occurred on jsDelivr.
They said ( on twitter , ref. https://twitter.com/jsDelivr/status/1472870623051456522 ) :

Unfortunately today jsDelivr unexpectedly lost its ICP license in China. As effect the regional CDN disabled our account.
This resulted in the extended outage we had in mainland China and Taiwan.
Other regions were unaffected.

Then I found resources on cdnjs may help.
Ref. https://cdnjs.com/libraries/bootstrap
May you please provide this info. on document ?
Thanks anyway.
Reduced test cases
None
What operating system(s) are you seeing the problem on?
Windows
What browser(s) are you seeing the problem on?
Chrome
What version of Bootstrap are you using?
v4.6.1

The issue with jsDelivr was limited to mainland China and was quickly fixed by routing all Chinese traffic to a different CDN. If you still have issues you need to purge your DNS cache.

The issue with jsDelivr was limited to mainland China and was quickly fixed by routing all Chinese traffic to a different CDN. If you still have issues you need to purge your DNS cache.

Yes, the issue was resolved.
Thanks for your contribution and works.
I meant the document on Bootstrap official site.
Reference to the document on jQuery official site , they provide more than one CDN.
( https://jquery.com/download/#using-jquery-with-a-cdn )
BTW, Chinese traffic to jsDelivr now routed to Fastly, but ICMP RTT was not so good ( as before ).

We don't have any plans on offering any other CDN at this point. I strongly believe @jimaek can solve any issues that might arise :)


fix: title is missing in the sidebar

PR Fixes:

1 sidebar content title

Resolves #1621

Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I assure there is no similar/duplicate pull request regarding same issue

closing in favour of https://github.com/code100x/cms/pull/1632

that guy was new to open source so promoting him : )

That's awesome!


Custom BubbleSizeCalculator - Question/Potential bug

I've created a custom bubble size calculator to change the size of some messages by extending JSQMessagesBubblesSizeCalculator and overriding the messageBubbleSizeForMessageData function.
All works well and I get a custom view size for some of my bubbles but I have noticed that if I trigger reloadData the size of the very first and very last bubble seems to go wrong. Commenting out the custom bubble size and there isn't this anomaly.
See screenshot attached and code below. Am I changing the bubble sizes in the right place?
@implementation JSQARXBubbleSizeCalculator
- (CGSize)messageBubbleSizeForMessageData:(id<JSQMessageData>)messageData
                              atIndexPath:(NSIndexPath *)indexPath
                               withLayout:(JSQMessagesCollectionViewFlowLayout *)layout
{
    ChatMessage *msg = (ChatMessage *)messageData;
    CGSize size = [super messageBubbleSizeForMessageData:messageData atIndexPath:indexPath withLayout:layout];

   // return size; //this would fix the problem but I lose my custom sizes
    if (msg.useCustomBubble) {
         size.height = 108.0f;
    }
    return size;
}

The bubble in question below has useCustomBubble as false.

how do you use your custom class? post an example!

Simply by setting it in viewDidLoad;
self.collectionView.collectionViewLayout.bubbleSizeCalculator = [JSQARXBubbleSizeCalculator new];

Thanks @markdelf ! :smile:
Please see the contributing guidelines and provide more information here.
Also, if you could fork and reproduce this in the demo project -- that would be great :+1:

@jessesquires I've found the problem, I don't think it was not really related to the BubbleSizeCalculator.
Basically I had some logic in heightForCellTopLabelAtIndexPath which shows a date just once per day. The logic was quite flawed and caused chaos (was not consistently returning the correct value). The side effect was what we're seeing in the screenshot.
Hope I didn't waste any of your time with my question!

Thanks for the update @markdelf ! No worries! :smile:

how do I implement this on swift? I'm having trouble fixing the size of my messageBubbles. :|

subclass the JSQMessagesCollectionViewFlowLayout, and override messageBubbleSizeForItemAtIndexPath(indexPath: NSIndexPath!) method there you return the size for each index path.
And assigned it in chat view controller viewDidLoad:
self.collectionView.collectionViewLayout = ()

oh thanks. I hope it works


testing solutions for issue #279

Hi ! May I ask what would be the expected timescale for this patch to be released ? The incompatibility with numpy 1.23 is causing a serious instability in yt's CI (two levels downstream, via pyart). I'd just like to know whether it'd be worth working on a temporary workaround. Thanks !

@jswhit just to add another data point, conda-forge ppc builds were also failing with a segfault and this PR solved it too. See https://github.com/conda-forge/cftime-feedstock/pull/47

I will merge this today and then being the process of creating a 1.6.1 release.

BTW, PR #282 should make it easier to issue releases. I'll finish it as soon as this one is merged.


In functionTest case /start/global.php is loaded after service providers

In functionTest case /start/global.php is loaded after service providers

Laravel Version: 4.2.0
PHP Version: 7.1.8
Database Driver & Version: MySQL 5.6

We are facing a problem in Laravel 4.2 where our /start/global.php is loaded/executed after some ServiceProvider. In our ServiceProvider public function register() and public function boot() are called before /start/global.php loaded/executed. This only happends while running our application tests via phpUnit. If we run our applicatoin via Browser on our Server the sequence of loading is correct.
TestCase.php:
<?php

    namespace app\tests\base;

    class TestCase extends \Illuminate\Foundation\Testing\TestCase {

    	/**
    	 * Creates the application.
    	 *
    	 * @return \Symfony\Component\HttpKernel\HttpKernelInterface
    	 */
    	public function createApplication()
    	{
    		$unitTesting = true;
    		$testEnvironment = 'testing';

    		return require __DIR__ . '/../../../bootstrap/start.php';
    	}
    }

phpunit.xml
<?xml version="1.0" encoding="UTF-8"?>
<phpunit backupGlobals="false"
         backupStaticAttributes="false"
         bootstrap="bootstrap/autoload.php"
         colors="true"
         convertErrorsToExceptions="true"
         convertNoticesToExceptions="true"
         convertWarningsToExceptions="true"
         processIsolation="false"
         stopOnFailure="false"
         syntaxCheck="false">
    <testsuites>
        <testsuite name="Application Test Suite">
            <directory>./app/tests/base/</directory>
        </testsuite>
    </testsuites>
</phpunit>

We looked deeper into the codes and recognized that $app->booted() in vendor/laravel/framework/src/Illuminate/Foundation/start.php is executed after public function register() and public function boot() (ServiceProvider).
How could we force the testing environment to load our application in the correct way? - Also refer:

https://laravel.com/docs/4.2/lifecycle
https://stackoverflow.com/questions/52073643/laravel-4-2-in-functiontest-case-start-global-php-is-loaded-after-service-prov

Full codes for reproduction can be downloaded here: https://github.com/linslin/laravel-42-test-boot-sequence

Hi,
Unfortunately Laravel 4.2 is no longer supported.
I will be closing your ticket here. Instead please try asking your question on one of the many great community support areas that will likely give you a better answer more quickly:

Laravel Slack (https://larachat.co/)
Laravel.io Forum (https://laravel.io/forum)
Laracasts Forum (https://laracasts.com/discuss)
StackOverflow (http://stackoverflow.com/questions/tagged/laravel)

Thanks in advance.


vim 8.2 --version plays hide and seek

vim --version finicky about when it gives any output:
  $  vim=./.GVim-v8.2.2488.glibc2.15-x86_64.AppImage
  0$  $vim --clean --version >/tmp/v82 2>&1              #this command captures nothing!
  0$  $vim --clean --version >/tmp/v82o 2>/tmp/v82e      #also nothing!
  0$  $vim --clean --version 2>&1 >/tmp/v82y             #this finally coaxes --version from 8.2
  0$  ls -ld /tmp/v82*
-rw-rw-r-- 1 g greg    0 Feb 10 16:04 /tmp/v82       #nothing!
-rw-rw-r-- 1 g greg    0 Feb 10 16:04 /tmp/v82e
-rw-rw-r-- 1 g greg    0 Feb 10 16:04 /tmp/v82o
-rw-rw-r-- 1 g greg 4081 Feb 10 16:04 /tmp/v82y     #got it here finally

By contrast the 8.1 behavior exhibited below is the behavior i expect.
  $  dpkg-query -W vim
vim     2:8.1.2269-1ubuntu5                              #(8.1 as shipped with ubuntu focal)
  0$  vim --clean --version >/tmp/v81 2>&1
  0$  vim --clean --version >/tmp/v81o 2>/tmp/v81e
  0$  vim --clean --version 2>&1 >/tmp/v81y
  0$  ls -ld /tmp/v81*
-rw-rw-r-- 1 g greg 3106 Feb 10 17:57 /tmp/v81     #8.1 plays fair
-rw-rw-r-- 1 g greg    0 Feb 10 17:57 /tmp/v81e
-rw-rw-r-- 1 g greg 3106 Feb 10 17:57 /tmp/v81o
-rw-rw-r-- 1 g greg 3106 Feb 10 17:57 /tmp/v81y

  $  vim81ver=$(vim --version 2>&1)                                                    #8.1 plays fair
  $  vim82ver=$($vim --version 2>&1)                                                   #8.2 plays hide&seek
  $  declare -p vim82ver vim81ver
declare -- vim82ver=""                                                                    #8.2: nothing here!
declare -- vim81ver="VIM - Vi IMproved 8.1 (2018 May 18, compiled Apr 15 2020 06:40:31)   #8.1 is fine
Included patches: 1-2269
Modified by<EMAIL_ADDRESS>Compiled by<EMAIL_ADDRESS>Huge version without GUI.
VIM - Vi IMproved 8.2 (2019 Dec 12, compiled Feb  9 2021 08:24:52)
Included patches: 1-2488
Compiled by travis@travis-job-4e22b497-0cc0-4353-bd05-c5bdb4eb329c
Huge version with GTK2 GUI.  Features included (+) or not (-):
+acl               -farsi             +mouse_sgr         +tag_binary
+arabic            +file_in_path      -mouse_sysmouse    -tag_old_static
+autocmd           +find_in_path      +mouse_urxvt       -tag_any_white
+autochdir         +float             +mouse_xterm       +tcl
-autoservername    +folding           +multi_byte        +termguicolors
+balloon_eval      -footer            +multi_lang        +terminal
+balloon_eval_term +fork()            -mzscheme          +terminfo
+browse            +gettext           +netbeans_intg     +termresponse
++builtin_terms    -hangul_input      +num64             +textobjects
+byte_offset       +iconv             +packages          +textprop
+channel           +insert_expand     +path_extra        +timers
+cindent           +ipv6              +perl              +title
+clientserver      +job               +persistent_undo   +toolbar
+clipboard         +jumplist          +popupwin          +user_commands
+cmdline_compl     +keymap            +postscript        +vartabs
+cmdline_hist      +lambda            +printer           +vertsplit
+cmdline_info      +langmap           +profile           +virtualedit
+comments          +libcall           +python/dyn        +visual
+conceal           +linebreak         +python3/dyn       +visualextra
+cryptv            +lispindent        +quickfix          +viminfo
+cscope            +listcmds          +reltime           +vreplace
+cursorbind        +localmap          +rightleft         +wildignore
+cursorshape       +lua               +ruby              +wildmenu
+dialog_con_gui    +menu              +scrollbind        +windows
+diff              +mksession         +signs             +writebackup
+digraphs          +modify_fname      +smartindent       +X11
+dnd               +mouse             +sound             -xfontset
-ebcdic            +mouseshape        +spell             +xim
+emacs_tags        +mouse_dec         +startuptime       -xpm
+eval              -mouse_gpm         +statusline        +xsmp_interact
+ex_extra          -mouse_jsbterm     -sun_workshop      +xterm_clipboard
+extra_search      +mouse_netterm     +syntax            -xterm_save
   system vimrc file: "$VIM/vimrc"
     user vimrc file: "$HOME/.vimrc"
 2nd user vimrc file: "~/.vim/vimrc"
      user exrc file: "$HOME/.exrc"
  system gvimrc file: "$VIM/gvimrc"
    user gvimrc file: "$HOME/.gvimrc"
2nd user gvimrc file: "~/.vim/gvimrc"
       defaults file: "$VIMRUNTIME/defaults.vim"
    system menu file: "$VIMRUNTIME/menu.vim"
  fall-back for $VIM: "././/share/vim"
Compilation: gcc -c -I. -Iproto -DHAVE_CONFIG_H -DFEAT_GUI_GTK -pthread -I/usr/include/gtk-2.0 -I././/lib/x86_64-linux-gnu/gtk-2.0/include -I/usr/include/gio-unix-2.0/ -I/usr/include/cairo -I/usr/include/pango-1.0 -I/usr/include/atk-1.0 -I/usr/include/cairo -I/usr/include/pixman-1 -I/usr/include/gdk-pixbuf-2.0 -I/usr/include/libpng16 -I/usr/include/pango-1.0 -I/usr/include/harfbuzz -I/usr/include/pango-1.0 -I/usr/include/glib-2.0 -I././/lib/x86_64-linux-gnu/glib-2.0/include -I/usr/include/freetype2 -I/usr/include/libpng16 -I/usr/include/freetype2 -I/usr/include/libpng16 -g -O2 -D_REENTRANT -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
Linking: gcc -L. -Wl,-Bsymbolic-functions -Wl,-z,relro -Wl,-z,now -fstack-protector -rdynamic -Wl,-export-dynamic -Wl,-E -L/usr/local/lib -Wl,--as-needed -o vim -lgtk-x11-2.0 -lgdk-x11-2.0 -lpangocairo-1.0 -latk-1.0 -lcairo -lgdk_pixbuf-2.0 -lgio-2.0 -lpangoft2-1.0 -lpango-1.0 -lgobject-2.0 -lglib-2.0 -lfontconfig -lfreetype -lSM -lICE -lXt -lX11 -lXdmcp -lSM -lICE -lm -ltinfo -lelf -lcanberra -ldl -L/usr/lib -llua5.3 -Wl,-E -fstack-protector-strong -L/usr/local/lib -L././/lib/x86_64-linux-gnu/perl/5.26/CORE -lperl -ldl -lm -lpthread -lcrypt -L././/lib/x86_64-linux-gnu -ltcl8.6 -ldl -lz -lpthread -lm -lruby-2.5 -lpthread -lgmp -ldl -lcrypt -lm -L/usr/lib
  $
  $  echo $TERM
screen
  $  tmux -V
tmux 3.1b
  $  uname -a
Linux e540 5.4.0-60-generic #67-Ubuntu SMP Tue Jan 5 18:31:36 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
  $  cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=20.04
DISTRIB_CODENAME=focal
DISTRIB_DESCRIPTION="Ubuntu 20.04.1 LTS"
  $

I can't reproduce the described behaviour with my own-built Vim 8.2.2496 (Big) on openSUSE Linux 5.2
linux-tuxedo:~ # vim --version >v8.2 2>&1
linux-tuxedo:~ # vim --version >v8.2o 2>v8.2e
linux-tuxedo:~ # vim --version >v8.2y
linux-tuxedo:~ # ll v8.2*
-rw-r--r-- 1 root root 3842 Feb 11 01:32 v8.2
-rw-r--r-- 1 root root    0 Feb 11 01:33 v8.2e
-rw-r--r-- 1 root root 3842 Feb 11 01:33 v8.2o
-rw-r--r-- 1 root root 3842 Feb 11 01:33 v8.2y
This is how I configure Vim:
export CONF_OPT_GUI='--enable-gui=gtk3'
export CONF_OPT_MULTIBYTE='--enable-multibyte'
export CONF_OPT_AUTOSERVE='--enable-autoservername'
export CONF_OPT_FEAT='--with-features=big'
#export CONF_ARGS2='--with-vim-name=vim-big'
export<EMAIL_ADDRESS>and the resulting --version output is as follows:
VIM - Vi IMproved 8.2 (2019 Dec 12, compiled Feb 10 2021 21:49:04)
Included patches: 1-2496
Compiled by<EMAIL_ADDRESS>Big version with GTK3 GUI.
Best regards,
Tony.

P.S. My shell is bash version 4.4.23(1)-release (x86_64-suse-linux-gnu)

P.P.S. No change if I add --clean on the command line (i.e. still behaves as expected, v8.2e is empty and the other three are still the same length as before).
Best regards,
Tony.

This is what I see:
$ vim --clean --version >/tmp/v82 2>&1
$ vim --clean --version >/tmp/v82o 2>/tmp/v82
$ vim --clean --version 2>&1 >/tmp/v82y
$ ls -l /tmp/v82*
-rw-rw-r-- 1 pel pel    0 feb 11 08:13 /tmp/v82
-rw-rw-r-- 1 pel pel 4289 feb 11 08:13 /tmp/v82o
-rw-rw-r-- 1 pel pel 4289 feb 11 08:13 /tmp/v82y

$ gvim -f -g --clean --version >/tmp/gv82 2>&1
$ gvim -f -g --clean --version >/tmp/gv82o 2>/tmp/gv82
$ gvim -f -g --clean --version 2>&1 >/tmp/gv82y
$ ls -l /tmp/gv82*
-rw-rw-r-- 1 pel pel    0 feb 11 08:17 /tmp/gv82
-rw-rw-r-- 1 pel pel    0 feb 11 08:17 /tmp/gv82o
-rw-rw-r-- 1 pel pel 4289 feb 11 08:17 /tmp/gv82y

I use vim-8.2.2497 (huge), GTK2 GUI on Linux x86_64 xubuntu-18.04.5, using bash-4.4.20 shell.

I think this is long-standing expected behavior.  The mch_msg() function shown below is responsible for vim --version output:
https://github.com/vim/vim/blob/v8.2.2488/src/message.c#L3184-L3211
    void
mch_msg(char *str)
{
#if (defined(UNIX) || defined(FEAT_GUI)) && !defined(ALWAYS_USE_GUI) && !defined(VIMDLL)
    // On Unix use stdout if we have a tty.  This allows "vim -h | more" and
    // uses mch_errmsg() when started from the desktop.
    // When not going to start the GUI also use stdout.
    // On Mac, when started from Finder, stderr is the console.
    if (
# ifdef UNIX
#  ifdef MACOS_X
	    (isatty(2) && strcmp("/dev/console", ttyname(2)) != 0)
#  else
	    isatty(2)
#  endif
#  ifdef FEAT_GUI
	    ||
#  endif
# endif
# ifdef FEAT_GUI
	    !(gui.in_use || gui.starting)
# endif
	    )
    {
	printf("%s", str);
	return;
    }
#endif

The output will be written if at least one of these two conditions is met:

stderr is a tty
No GUI is in use (or will be in use)

Note that though the version output is destined for stdout, the decision is based on whether stderr is a tty.
When stderr is redirected to a file or a pipe, the first condition above fails; but the version information will still be written if Vim is started without a GUI.  In the event that Vim intends to start a GUI and stderr is not a tty, then the output will be suppressed.
Vim starts a GUI when the -g argument is passed, or when the name of the executable file starts with g; that's why gvim invokes the GUI by default whereas vim does not:
https://github.com/vim/vim/blob/v8.2.2488/src/main.c#L1805-L1816
/*
 * Check for: [r][e][g][vi|vim|view][diff][ex[im]]  (sort of)
 * If the executable name starts with "r" we disable shell commands.
 * If the next character is "e" we run in Easy mode.
 * If the next character is "g" we run the GUI version.
 * If the next characters are "view" we start in readonly mode.
 * If the next characters are "diff" or "vimdiff" we start in diff mode.
 * If the next characters are "ex" we start in Ex mode.  If it's followed
 * by "im" use improved Ex mode.
 */
    static void
parse_command_name(mparm_T *parmp)

For some Vim ports, the GUI may always be started.
I suspect this new build of Vim starts the GUI, unlike the previous builds.  Since stderr is also being redirected, no version output is generated.
  In the first example, stdout is redirected to /tmp/v82, and then stderr is redirected to be the same as the current (already redirected) stdout, making both point to /tmp/v82.  The second example uses independent files for stdout and stderr, so the order doesn't matter.  But the third case first redirects stderr to the original location for stdout (the terminal), and then redirects stdout to /tmp/v82y, leaving stderr still pointing to a tty; this triggers the "stderr is a tty" condition for the mch_msg() function, causing the version output to be displayed.
As far as I can tell, there are only two ways to force Vim to display the version output on stdout: either ensure stderr is a terminal, or ensure the GUI will not be started.  Looking through the source, there is this section of the command_line_scan() function for parsing command-line arguments:
https://github.com/vim/vim/blob/v8.2.2488/src/main.c#L2278-L2282
	    case 'v':		// "-v"  Vi-mode (as if called "vi")
		exmode_active = 0;
#if defined(FEAT_GUI) && !defined(VIMDLL)
		gui.starting = FALSE;	// don't start GUI
#endif

The switch -v is intended to force Vi-compatible mode; but it has the side effect of setting gui-starting to FALSE, e.g.:
$ vim --version 2>/dev/null | wc -c
4100

$ gvim --version 2>/dev/null | wc -c
0

$ gvim -v --version 2>/dev/null | wc -c
4100

So perhaps adding a -v switch will make the version check work as expected.
Michael Henry

my 2¢:
either the manual should clarify that if you want to capture --version into a bash variable you need 'vim -vV'
or vim -V should understand it won't be starting any gui and behave accordingly.

hopefully the manual will clarify that if you want to capture --version into stdout, eg a bash variable, you need '[g]vim -v --version'.  (the gvim appimage is the only appimage available.)

vim --help|&less is also mute!  This is terrible behavior in my opinion.
vim --help|less leaves standard error hidden outside of less and thus invisible to the user.  Not good.
yes vim -v --help|&less works but it's certainly not intuitive.


Bridge UI - Metis transactions keeping hitting error

Background
Transactions to bridge, deposit pool liquidity, or add router liquidity keep hitting an "underlying network changed" error. This only happens for Metis.
Eg.

Definition Of Done
Metis transactions work.

This has occurred before when adding router liquidity on other networks. Most often, it happens when you try to add router liquidity and need to switch networks. After switching, this error will pop up. Refreshing the page after you've connected to the right chain works in those cases.
However, for Metis transactions even refreshing while connected to Metis doesn't help.


The last class of a subject is cut off and not included in the semester list.

For example, CSE 5027-07 and ARTS 2127 for Fall 2024.

works now, except for whatever this is
['Off-campus', '', '', 'SLF', 'Sandia Labs Off Campus', '', '', 'SLO', 'Sandia Labs On Campus', '', '', 'V', 'Video Lectures-Live & Recorded', '']
I'm not fixing this one class


Update READMME.md, supprot swift guide.

I like the idea of adding Swift to the README, but it would be good if the Obj-C and Swift sections did not have duplicate text.
Also, you should be able to use bit flag enums in Swift without rawValue; e.g. let foo: Flag = [.Flag1, .Flag2]


ApiException No message

Having issues with this and Laravel 5.7 module. Both return same issue with no real diagnosis.
Any api call returns API Exception: No Message

@mpclarkson Any update on this?

Hi Greg, the author of this repository, Matthew,  is no longer with this organisation and I doubt he is wishing to maintain it in the future.
Regards,
Julian.
Sent from my iPhone

On 6 Apr 2019, at 4:40 am, Greg Avola<EMAIL_ADDRESS>wrote:
@mpclarkson Any update on this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


[Question] Would you mind making Encoding/Decoding interfaces (and implementations) public?

Context
We, at AsyncAPI are developing an Event Gateway. Our first POC will cover only a Message validation proxy for produced Kafka messages.
That task includes decoding/encoding requests and responses, as some of those have to be filtered, skipped (sent directly to the broker), and even modified (such as FindCoordinator and Metadata responses).
As you can imagine, there is a lot to do if starting from scratch (you already know more than I do actually) so we would love to reuse your models (for requests and responses) and most of the logic around encoding/decoding.
Problem Description
Most of the Encoding/Decoding interfaces and function helpers are private, implementations as well. For illustration, taking one example:

Decoder interface: https://github.com/Shopify/sarama/blob/master/encoder_decoder.go#L48-L52
Real decoder (Decoder and PacketDecoder implementation): https://github.com/Shopify/sarama/blob/master/real_decoder.go

If we could reuse all of those, we would not only save a lot of time but have a stronger implementation, considering the maturity of Sarama. Also worth mentioning that we would become maintainers in the long term.
Conclusion
This issue is not a feature request. It is only a question. We are happy to make all the needed changes and making a compromise on not breaking compatibility (so changes would increase a minor version).
Thanks for all the work you folks do here :)

@smoya thanks for raising this. We had previously discussed doing something like this as part of the Sarama v2.0.0 roadmap. The intention was to have pure kafka protocol types and the decoder/encoder in its own package, following the spec closely. Then the aim was to update the core Sarama client to never accept or return those protocol-specific types, but to instead have its own user-friendly shims that abstract internals away, satisfy the error, stringer interfaces etc.

I wrote some comments on https://github.com/Shopify/sarama/issues/1732

I wrote some comments on https://github.com/Shopify/sarama/issues/1732


Feature/152 arrow key navigation

Fixes #152

This change is

Pull Request Test Coverage Report for Build 674

0 of 10 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.01%) to 57.5%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/IMAPClient-UI.package/ICFolderDialog.class/instance/buildWith..st
0
2
0.0%

packages/IMAPClient-UI.package/ICFolderDialog.class/instance/emailListKeyPress..st
0
8
0.0%

Totals

Change from base Build 671:
-0.01%

Covered Lines:
2323

Relevant Lines:
4040

💛  - Coveralls

More info about how it works in comments of #152


arm64 support?

What would be required for supporting 64-bit ARM?  I want to run Rebol on my laptop (Acer R13 -- MediaTek  MT8173(c) 64-bit ARMv8) but currently I get this error:
./rebol-view/rebol: error while loading shared libraries: libX11.so.6: cannot open shared object file: No such file or directory
Meanwhile I do have X11 installed and other X11 applications seem to work.
Here's the version of rebol I'm using:
Downloads/rebol-view/rebol: ELF 32-bit LSB executable, ARM, EABI5 version 1 (SYSV), dynamically linked, interpreter /lib/ld-linux-armhf.so.3, for GNU/Linux 2.6.32, BuildID[sha1]=13961069bc477d1233c6edd300fee274f8f798e7, stripped```

And this is my system:

```$ uname -a
Linux localhost 3.18.0-14875-g438cb8ab27c6 #1 SMP PREEMPT Sat Sep 16 09:49:59 PDT 2017 aarch64 aarch64 aarch64 GNU/Linux```

Also I have libc6 already installed:
[sudo] password for lx:
Reading package lists... Done
Building dependency tree
Reading state information... Done
libx11-6 is already the newest version (2:1.6.3-1ubuntu2).
libx11-6 set to manually installed.
0 upgraded, 0 newly installed, 0 to remove and 18 not upgraded.```

It's probably just a filename issue. What's the name of your libX11 file and where is it? When you find it, just make sure the Rebol file is updated with that location.

This repository is for Rebol 3 (R3-alpha) which has no view version. It looks you were just trying to run Rebol2 on your ARM64 laptop.


[Aura-Safe][Safe Management][Multisig Tx] Update text "AURA" and format amount XX.XXXXXX

Description
Display incorrect text "Aura" and format amount

Update text: "Aura"
Format amount: XX.XX

Environment

Browser: Chrome
Wallet: Keplr
Safe: Aura
Environment: serenity

testnet

Steps to reproduce

Go to Assets/Coins page: https://safe.serenity.aura.network/59:aura1t7d0avw96ryedtv79ahehzm80thptndducjmdg/balances
Go to Transactions page: QUEUE and HISTORY tabs: https://safe.serenity.aura.network/59:aura1t7d0avw96ryedtv79ahehzm80thptndducjmdg/transactions/history
Check text "AURA" and format amount

Expected result

[x] Update text: "Aura" -> "AURA" at screens below:
ASSETS/COINS
Send Funds pop up (steps 1 of 2)
Transactios/QUEUE tab
Transactios/HISTORY tab
[x] Format amount: XX.XX -> "XX.XXXXXX" at screens below:
ASSETS/COINS
Send Funds pop up (steps 1 of 2)
Send Funds pop up (steps 2 of 2)
Transactios/QUEUE tab
Transactios/HISTORY tab

Obtained result
Display incorrect text "Aura" and format amount

Update text: "Aura"
Format amount: XX.XX

Screenshots

ko thể làm table giống 100%

Verfied PASS on serenity (04-Aug-2022)


[DOC-1430] Scalar NestJS Integration with Helmet doesnt work

Expected Behavior
it won't run alongside Helmet https://docs.nestjs.com/security/helmet.
Actual Behavior
it should work with helmet
Reproduction steps
install nestjs
install scalar nestjs integration
install helmet
wont run

Link to site / OpenAPI Spec
No response
Version
latest
What browsers are you seeing the problem on?
No response
DOC-1430

hey @emmanuelvr, i wasn't able to reproduce this. are you able to share a repo or show console log errors of helmet not working with Scalar & NestJS? :)

Hi @marclave! Thanks a lot for looking into this.
In my case, whenever I enable helmet and try to access the /reference endpoint, I see this error in my browser's console:
Refused to load the script 'https://cdn.jsdelivr.net/npm/@scalar/api-reference' because it violates the following Content Security Policy directive: "script-src 'self'". Note that 'script-src-elem' was not explicitly set, so 'script-src' is used as a fallback.
And then the screen just remains white. I thought that having CORS enabled might have to do with the issue, but only Helmet seems to affect the website.

@emmanuelvr are you using it with express or fastify?
https://docs.nestjs.com/security/helmet
It looks like its related to the CSP issue in this guide, you will probably need to add the following :)
await app.register(fastifyHelmet, {
   contentSecurityPolicy: {
     directives: {
       defaultSrc: [`'self'`, 'unpkg.com'],
       styleSrc: [
         `'self'`,
         `'unsafe-inline'`,
         'cdn.jsdelivr.net',
         'fonts.googleapis.com',
         'unpkg.com',
       ],
       fontSrc: [`'self'`, 'fonts.gstatic.com', 'data:'],
       imgSrc: [`'self'`, 'data:', 'cdn.jsdelivr.net'],
       scriptSrc: [
         `'self'`,
         `https: 'unsafe-inline'`,
         `cdn.jsdelivr.net`,
         `'unsafe-eval'`,
       ],
     },
   },
 });

let me know if that works ✨

That solved the issue!! Thanks a lot @marclave. I'm using Express, so I had to combine your suggestion and the Express step. It looks like this:
app.use(
    helmet({
      crossOriginEmbedderPolicy: false,
      contentSecurityPolicy: {
        directives: {
          defaultSrc: [`'self'`, 'unpkg.com'],
          styleSrc: [
            `'self'`,
            `'unsafe-inline'`,
            'cdn.jsdelivr.net',
            'fonts.googleapis.com',
            'unpkg.com',
          ],
          fontSrc: [`'self'`, 'fonts.gstatic.com', 'data:'],
          imgSrc: [`'self'`, 'data:', 'cdn.jsdelivr.net'],
          scriptSrc: [
            `'self'`,
            `https: 'unsafe-inline'`,
            `cdn.jsdelivr.net`,
            `'unsafe-eval'`,
          ],
        },
      },
    }),
  );

Amazing! Glad to hear it all works @emmanuelvr ✨

For those who facing issue with fonts loading and CSR, in the solution above, replace fonts.gstatic.com to fonts.scalar.com.


model location

Thanks for your amazing work！
Can you explain in detail how to place the corresponding model?

You need to create a "chekpoints" folder then a "lgm" folder in "ComfyUI-3D-Pack" and put the model file that seems to be : https://huggingface.co/ashawkey/LGM/tree/main

Different models will be placed under checkpoints/[model name]:
For example:

checkpoints/lgm
checkpoint/tgs

Still, this is a small inconvenient for running model for the first time, ok, I'll fix this issue in the next update so it will automatically download the model if you don't have it


Redux-React Basic Tutorial issues

Not quiet sure where else to put this.
For the Basic Tutorial https://react-redux.js.org/introduction/basic-tutorial there are 3 issues.
Issue 1: The Sandbox Code: https://codesandbox.io/s/6vwyqrpqk3 uses react: 16.4.2, but needs at least 16.8.4 because of .memo()
Issue 2: The tutorial implements the connection for the VisibilityFilters, but it omits setting the filters
const VisibilityFilters = ({ activeFilter, setFilter }) => {
<span
key={`visibility-filter-${currentFilter}`}
className={cx(
  "filter",
  currentFilter === activeFilter && "filter--active"
)}
onClick={() => {} /** waiting for setFilter handler*/} <------this should be mentioned in the Tutorial, or filled in
>
{currentFilter}
</span>

Issue 3: Similar to issue 2, the tutorial doesnt implement the toggleTodo() in Todo.js.
Thanks!

The tutorial really needs to be rewritten anyway, to show use of hooks instead of class components and connect.
Right now the best tutorial is actually in the Redux core docs "Fundamentals" tutorial:
https://redux.js.org/tutorials/fundamentals/part-5-ui-react

The tutorial really needs to be rewritten anyway, to show use of hooks instead of class components and connect.
Right now the best tutorial is actually in the Redux core docs "Fundamentals" tutorial:
https://redux.js.org/tutorials/fundamentals/part-5-ui-react


Add NESC

https://nesc.edu.tt/

@KemyToro First stage of verification passed, official domain for NESC Technical Institute is nesc.edu.tt as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@KemyToro Thank you for your patience while waiting for a review and processing your request. Pull request merged.


scale.nice() returns values outside domain for values less than 1

In playing around with percentages in scales, I discovered if I have domainValues < 1, scale.nice() produces a domain with a max value outside of the original domain. Whereas for domainValues >=1, the scale.nice() would be restricted to the original domain.
Example 1:
scale.domain([0,0.28]).nice() => [0,.3]
Example 2:
scale.domain([0, 28]).nice() => [0, 28]
The behavior for the first example was actually what I needed for my visualization, and I think it could be done for example 2 (returning [0,30]) to create nicer niced scales.

This is because 0.28 does not exactly equal 28 / 100 in IEEE 754 binary floating point. It’s slightly bigger.
(0.28).toFixed(32) // "0.28000000000000002664535259100376"

So a more precise comparison would be:
d3.scaleLinear().domain([0, 28.000000000000002664535259100376]).nice().domain()

Which returns a domain of [0, 30], similar to how the smaller domain returns [0, 0.3].
Nicing strictly increases the size of the domain, so if the upper bound of the domain is greater than the exact value of 0.28 (which 0.28 in IEEE 754 is), then any nicing will result in a bigger domain. Unfortunately, there’s not really a way for our API to distinguish between you setting the upper bound of the domain to 0.28 exactly and setting it to 0.28000000000000003.


update README example to use webpack 2

I know this is kind of a duplicated issue with #73, but it would be very helpful to folks new to webpack.
Also, configuring standard would also be changed into options in loaders.
Huge thanks to your work!

@yaodingyd can you send a pull request?

@timoxley Gladly!

PR created #77.
Thanks!


Staging bug fixes maclist

Subject:Resolving radios turn-off problem when supplying the maclist for blacklist and whitelist feature

merged into trunk, Thanks !


Remove wunderlist

Hey @yannbertrand
just removing a dead link - Wunderlist has been aquired by Microsoft and integrated into their own product. The API is gone too afaik.
Cheers,
Peter

Thank you

No worries :+1:


fix clear_database that can't find a populated PuppetDBExtensions.config

use the options hash instead.
also fix john's previous fix to run_agents_with_new_site_pp.
this allows tests/anonymize/anonymize_profile.rb to run assuming other tests are able to teardown properly.

@johnduarte not sure if you want PRs here...
don't want to lose track of these fixes as i run around chasing squirrels


another stab at getting barcodes into the mix

I noticed that the current PR #22 for a barcode visualisation is failing, so I thought I'd try to fix up whatever the problem is, and in the process decided make the thing a little more to my tastes. Main change is that the H0 and H1 barcodes are combined in a single plot with finer lines than previous (since this is the most common scenario to examine) and individual plots for each dimension are produced if dim != 2.
As well, the optional export_png keyword argument in the plot_bacode() method will cause a PNG version of the barcode to be returned as an io.BytesIO() object.
Example use is included in the docstring for the Barcode class:
n = 300
t = np.linspace(0, 2 * np.pi, n)
noise = np.random.normal(0, 0.1, size=n)

data = np.vstack([((3+d) * np.cos(t[i]+d), (3+d) * np.sin(t[i]+d)) for i, d in enumerate(noise)])
diagrams = ripser(data)

bc = Barcode(diagrams['dgms'])
bc.plot_barcode()

Codecov Report

Merging #24 into master will decrease coverage by 7.79%.
The diff coverage is 10.44%.

@@            Coverage Diff            @@
##           master      #24     +/-   ##
=========================================
- Coverage   80.03%   72.24%   -7.8%
=========================================
  Files          10       10
  Lines         531      598     +67
  Branches      105      118     +13
=========================================
+ Hits          425      432      +7
- Misses         78      138     +60
  Partials       28       28

Impacted Files
Coverage Δ

persim/visuals.py
40.11% <10.44%> (-18.94%)
:arrow_down:

Continue to review full report at Codecov.

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

I see, failure due to lack of tests written. Any ideas how I can address this?

Thanks for this PR!!
I wouldn't worry too much about the codecov results.  Testing plotting code can be pretty nebulous.
Could you post come screenshots of the output this produces into a comment?

Sure, easy! Here is the output from the sample code provided:

Awesome! Thanks again for picking this up!
Does it look the same if you plot above $H_1$?
Would it be easy to make the x-axes the same for all of the sets? I could see it being helpful for comparisons, but also annoying if each have different scales.  What do you think?
In the $H_1$ barcode, the bars are so thin it was hard to see. How difficult is it to make the width of each bar determined by the number of bars shown?

Awesome! Thanks again for picking this up!

No problem, I was using the feature after all

Does it look the same if you plot above $H_1$?

No, if dim != 2 then individual plots are produced for each dimension. I can extend this up to $H_2$ trivially, but beyond that things get a bit large, so I figured that combining $H_0$ and $H_1$ was a good idea, everything else left on and individual basis.

Would it be easy to make the x-axes the same for all of the sets? I could see it being helpful for comparisons, but also annoying if each have different scales. What do you think?

Very easy, but the question arises about the number of bars for small $\epsilon$ values. If you want to share the axis and still have it readable, I should also make it possible for the user to specify the number of bars they want to to see so that the low-$\epsilon$ junk is truncated off, optionally of course.

In the $H_1$ barcode, the bars are so thin it was hard to see. How difficult is it to make the width of each bar determined by the number of bars shown?

The linewidth is already controllable via the linewidth keyword argument, but making that a function of n_bars would be a bit tricky. See above comment for controlling what is displayed; if that is added then the user could specify the linewidth as well in order to get something visually appealing.
Would you like me to do that?

Sorry for abandoning this, saw it today and thought I should fix it up. Better late than never!


SE3 not multiplying properly?

When multiplying SE2 elements, the translation gets properly transformed. With SE3, the translations are not properly transformed when multiplying:

As you can see, in the second part, the translation stays constant which it shouldn't

Oops, my bad... On X rotation doesn't change X...


feat: switch bls to napi rebuild

Motivation
Swaps out SWIG/Herumi implementation of blst with Napi implementation.
Currently deployed on feat1 group and is developing metrics.  Will post below as they progress.
~DO NOT MERGE until the package version for bls is added.  Needs to be merged and published first though~
Description

Remove references to @chainsafe/blst
Add new version of @chainsafe/bls that uses the updated @chainsafe/blst under the hood
~Add export UV_THREADPOOL_SIZE=$(nproc) to ./lodestar entrance script and benchmark commands to ensure adequate performance~ Set process.env.UV_THREADPOOL_SIZE = os.availableParallelism().toString() in cli/src/cmds/beacon/beaconHandler.ts
Move G2_POINT_AT_INFINITY is_inf test to @chainsafe/blst
Add a hidden chain.blsAddVerificationRandomness CLI option for applying mult randomness to same message verification.  Defaults to true unless explicitly set as false via the CLI
Add implementation of mult randomness for same message verification
Flatten beacon-node/src/chain/bls directory structure by removing multithread folder
Add addVerificationRandomness to verification options of queued jobs.  Not available via the external function signature and is set via the CLI options and merged with options passed to the call to IBls.verifySignatureSets
Remove support in BlsMultiThreadWorkerPool for Herumi implementation. (see notes below)
Remove bls Worker support and code (see notes below)
Remove serialization of keys and sigs and pass as deserialized objects because no worker boundary exists anymore
Renamed "./maybeBatch.js" to "./verifySets.js"
Renamed "./worker.js" to "./runBlsWorkReq.js"
Combined beacon-node/src/chain/bls/multithread/utils.ts with beacon-node/src/chain/bls/utils.ts
Fixed one unit test because deep equal of Vitest failed because Uint8Array and Buffer are non-compatible so wrapped the buffer in Uint8Arry.from

NOTES

Before this can be merged @chainsafe/blst and @chainsafe/bls need to both be published and the package version needs to be swapped in the package.json files in here.  For now a sha is being used to target the head of the feature branch of @chainsafe/bls
Potentially need to tune how canAcceptWork() functions to optimize performance.  Possibly should remove queue functionality and only use queue on the gossipsub side or vice versa and only use queue in bls so that there are not two queues introducing throttling
The functionality to test is_inf for G2_POINT_AT_INFINITY is not longer exported from the deps so no way to test from the Lodestar side.  Potentially should export that constant from @chainsafe/bls and move the test to there so that the value and unit test for it are in the same place. Unfortunately the is_inf in not available from Herumi so it really should be in @chainsafe/blst however if that is where the test and value live it will no longer be a peerDependency which is not great  Need to import @chainsafe/blst as a dependency to lodestar now and we are maintaining it as a peerDependency in @chainsafe/bls, so its possible to use the value from the blst repo now.
Removed workers and Herumi entirely.  It was not possible to even access herumi after digging extensively through the code.  Would require a major refactor of all imports of @chainsafe/bls to even allow for using Herumi. Removing inaccessible, unused and unnecessary option greatly simplified class structure of BlsMultiThreadWorkerPool because workers should no longer be used realistically.  The only case were workers would have been beneficial to retain would be to run Herumi on-thread in a worker.  It would not be ideal to run napi on-thread in workers and is better to run it via libuv scheduling
Follow-up PR could potentially improve the metrics but leaving as-is for now so we can see apples-to-apples comparisons with existing branches.  Can come back as a second pass to add/remove/update now that workers are no longer present
randomBytesNonZero is exported by @chainsafe/blst now but not via @chainsafe/bls because re-exporting would cause the same peerDependency issue as re-exporting G2_POINT_AT_INFINITY.  This need to be resolved but highlighting here to get the teams' feelings on the best way to handle this

It seems like this branch will likely get moved to unstable.  Moving over some important comments from the other contender branches that should be persisted.
Setting threadpool size from PR #6660
https://github.com/ChainSafe/lodestar/pull/6660#discussion_r1563993760
https://github.com/ChainSafe/lodestar/pull/6660#discussion_r1565308955
https://github.com/ChainSafe/lodestar/pull/6660#discussion_r1565450094
https://github.com/ChainSafe/lodestar/pull/6660#discussion_r1567171450
Other branches/PR's that preceded this PR
#6362
#6516

Culmination of:
18,127 additions
34,555 deletions
(not including any intermediate branches or PR's, abandoned attempts or unused implementations)

@matthewkeil I am curious about the use of switchable for light-client. I previously removed it here

Will def need to be included because there are several environments that need to get supported.

Does this PR introduce new environments that require such changes? All tests were passing AFAIK?
Also looks like this change defaults the light-client to herumi if user doesn't specify otherwise, which would be a change in behavior.

Culmination of:
18,127 additions
34,555 deletions

Do you have any numbers on LoC in terms of src code changes? Would assume the napi one is leaner which from the looks of deleted vs. added it seems like it but noticed there are generated files like prebuild/blst_wrap.cpp in SWIG version.
I would be really interested in learning a bit more about the new vs. previous implementation, maybe we can have a session for that as well?

@jeluard and @nflaig got the cruft removed from the blst import and things work as they did before.  Ready for review again.
The PR in bls Chainsafe/bls#179 needs to get merged and published first and I will replace the last commit with the final bls version 8.1.0

Culmination of:
18,127 additions
34,555 deletions

Do you have any numbers on LoC in terms of src code changes? Would assume the napi one is leaner which from the looks of deleted vs. added it seems like it but noticed there are generated files like prebuild/blst_wrap.cpp in SWIG version.
I would be really interested in learning a bit more about the new vs. previous implementation, maybe we can have a session for that as well?

I was looking at this but I think it will be hard to apples to apples.  There are lots of LOC that are in the node-api and node-addon-api that do part of what the blst_wrap.cpp did.  They are not all used as its a full API.  And in the blst_wrap, that was a generated file so its not overly terse as much was SWIG boilerplate stuff that gets autogenerated so LOTS of copy/paste with small changes per class.

@tuyennhv and @nflaig I caught that I was changing the UV_THREADPOOL_SIZE after the beacon args got initialized, which has FS and other calls.  I moved that logic above the init but logger is not available at that point so left the actual log statements below and just did the check and update of the value before that function just in case fs happens in there at some point.

@matthewkeil  sim tests failed consistently, looks good to me other than that

@matthewkeil sim tests failed consistently, looks good to me other than that

Finding lots and lots of tcp errors in the sim test logs....
[node-1-beacon-lodestar] [60618]: 2024-04-24T09:00:53.776Z libp2p:tcp:socket:error encrypting inbound connection to Error: Error occurred during XX handshake: read ECONNRESET failed
[node-1-beacon-lodestar] [60618]: 2024-04-24T09:00:53.776Z libp2p:tcp:socket:error failed to upgrade inbound connection CodeError: Error occurred during XX handshake: read ECONNRESET
    at DefaultUpgrader._encryptInbound (file:///Users/matthewkeil/Documents/dev/chainsafe/lodestar/node_modules/libp2p/dist/src/upgrader.js:492:19)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async DefaultUpgrader.upgradeInbound (file:///Users/matthewkeil/Documents/dev/chainsafe/lodestar/node_modules/libp2p/dist/src/upgrader.js:113:25) {
  code: 'ERR_ENCRYPTION_FAILED',
  props: {}
}

@nflaig your wish is my command.   Moved the logging of libuv size back to the worker.  Wasnt getting logged during tests and that needs to happen so its back "home" now

@matthewkeil sim tests failed consistently, looks good to me other than that

I merged in unstable again to get the old version of libp2p-tcp and set the UV_THREADPOOL_SIZE=4 for sim tests.
Fixed the issues I was seeing in the logs

Culmination of:
18,394 additions
34,709 deletions
(not including any intermediate branches or PR's, abandoned attempts or unused implementations)

:tada: This PR is included in v1.19.0 :tada:


fix(escaping): Escape key/value properties on scope

Previously, we weren't escaping HTML entities in scope key/vals.  This opens up a whole host of issues.
Instead, escape things before we drop them into our extension panel.
Closes #135

LGTM

:+1:

Landed in d8054968b56cec5be53aaff06d3e4b883ddf0bb1

Great. Thank you!


Install DB migrations on driver startup

Install DB migrations on driver startup. We can just open our own database connection like the cli script does and arrange for alembic output to get to the neutron logs.

[ ] Install DB migrations on driver startup
[ ] Add alembic to requirements.txt so that we're sure it's available on all the neutron servers
[ ] Plumb alembic output to the neutron logging

I'd rather have an install utility #101

Not doing this, an install utility is way more useful and predictable.


Fresh Sift build (focal) on qubes

Hello all,
I decided to try building sift from scratch on a fresh focal system.  Not built on top Remnux.
I followed the github guide for installing.
Attached is the saltstack.log.
saltstack.log

Hi @TheWanderer1983 , after looking at all of the issues you've had, you seem to be getting the exact same error message each time, related to the 'systemd-run' error code 100. Can you provide the output of the following:
sudo apt-get update
sudo apt-get install -f
sudo apt search wireless-tools
ls -la /etc/apt/sources.list.d/

Hello @digitalsleuth,
Here are the requested outputs.
$ sudo apt-get update
Hit:1 https://download.docker.com/linux/ubuntu focal InRelease
Hit:2 https://repo.saltproject.io/py3/ubuntu/20.04/amd64/3004 focal InRelease
Get:3 http://ppa.launchpad.net/gift/stable/ubuntu focal InRelease [18.0 kB]
Hit:4 http://archive.canonical.com/ubuntu focal InRelease
Hit:5 http://archive.ubuntu.com/ubuntu focal InRelease
Hit:6 http://ppa.launchpad.net/openjdk-r/ppa/ubuntu focal InRelease
Get:7 http://archive.ubuntu.com/ubuntu focal-security InRelease [114 kB]
Hit:8 http://ppa.launchpad.net/sift/stable/ubuntu focal InRelease
Get:9 http://archive.ubuntu.com/ubuntu focal-updates InRelease [114 kB]
Get:10 http://archive.ubuntu.com/ubuntu focal-updates/main amd64 Packages [2,000 kB]
Get:11 http://archive.ubuntu.com/ubuntu focal-updates/universe amd64 Packages [927 kB]
Fetched 3,172 kB in 28s (113 kB/s)
Reading package lists... Done
sudo apt-get install -f
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following packages were automatically installed and are no longer required:
libnorm1 libpgm-5.2-0 libzmq5 python3-colorama
Use 'sudo apt autoremove' to remove them.
0 upgraded, 0 newly installed, 0 to remove and 2 not upgraded.
sudo apt search wireless-tools
Sorting... Done
Full Text Search... Done
broadcom-sta-dkms/focal-security,focal-updates <IP_ADDRESS>1-12ubuntu0.1 all
dkms source for the Broadcom STA Wireless driver
broadcom-sta-source/focal-security,focal-updates <IP_ADDRESS>1-12ubuntu0.1 all
Source for the Broadcom STA Wireless driver
iw/now 5.4-1 amd64 [installed,local]
tool for configuring Linux wireless devices
ls -la /etc/apt/sources.list.d/
total 44
drwxr-xr-x 2 root root 4096 Aug  2 08:31 .
drwxr-xr-x 7 root root 4096 Aug  2 08:29 ..
-rw-r--r-- 1 root root   71 Aug  2 08:31 docker.list
-rw-r--r-- 1 root root   71 Aug  2 08:31 docker.list.save
-rw-r--r-- 1 root root  124 Aug  2 08:31 gift-ubuntu-stable-focal.list
-rw-r--r-- 1 root root  124 Aug  2 08:31 gift-ubuntu-stable-focal.list.save
-rw-r--r-- 1 root root  128 Aug  2 08:31 openjdk-r-ubuntu-ppa-focal.list
-rw-r--r-- 1 root root   84 Aug  2 08:31 saltstack.list
-rw-r--r-- 1 root root   84 Aug  2 08:31 saltstack.list.save
-rw-r--r-- 1 root root  124 Aug  2 08:31 sift-ubuntu-stable-focal.list
-rw-r--r-- 1 root root  124 Aug  2 08:31 sift-ubuntu-stable-focal.list.save
date
Tue 02 Aug 2022 10:10:10 PM AEST

Can you run the following as well and paste the output?
cat /etc/apt/sources.list | grep -v "#"

cat /etc/apt/sources.list | grep -v "#"
deb http://archive.ubuntu.com/ubuntu focal universe
deb http://archive.canonical.com/ubuntu focal partner
deb http://archive.ubuntu.com/ubuntu focal-security multiverse
deb http://archive.ubuntu.com/ubuntu focal-updates main universe multiverse restricted
deb http://archive.ubuntu.com/ubuntu focal multiverse

Thanks @TheWanderer1983 , it looks like you have all the repos you need enabled, so I'm not quite sure why you are unable to install a few of the tools found in those repos. Can you tell me what steps you took to set up the OS?

Sure, This is a Qubes System. So I cloned a Focal template. Specifically this one https://qubes.3isec.org/Templates_4.1/
Then I followed the Option 2A: SIFT Easy Installation on Native Ubuntu System as detailed here. https://www.sans.org/tools/sift-workstation/
The only variation is I didn't create that user account. I'm using the default account for this VM which has sudo.

Sounds good, I'll give that process a shot and see if I come across the same issue.

Thanks for the help. Appreciated.


Request pending forever if content type is application/json

Problem/Feature Request Summary
Not able to parse request body when the header is application/json
Environment Information

koa-body: 4.1.0
Koa: 2.7.0
Node.js: 12.4.0
koa-router: 7.4.0

Current Behavior
When request Content-Type header is application/json the request stay pending forever resolving. ,
Steps to Reproduce
const Router = require("koa-router");
const koaBody = require("koa-body")
Expected Behavior
application/json content parsed
Context (Environment)
For urlencoded or text request its working fine.
I tried to debug it and I can see the options passed to the co-body json https://github.com/dlau/koa-body/blob/ed81445ccd6d2e3b5e4605ce502b60c75386f8f5/index.js#L74 are:
{
   encoding: 'utf-8',
   limit: '1mb',
   strict: true,
   returnRawBody: false
}

Can you confirm that the koa-router example works for you? We have the same basic setup.
https://github.com/dlau/koa-body/blob/master/examples/koa-router.js

Yes, I'm using it in other routes with other content types and it works it also work on that one if I change the content type.
I just add the koa-router.js example to my project and it also works fine

I don't think we'll be able to troubleshoot this without an example repository.

I can confirm we're experiencing this as well, but I should note that this only happens when we deploy to Google Cloud Functions, running locally works fine.
Reproducing this is pretty simple:
const koa = require("koa");
const koaBody = require("koa-body");
const KoaRouter = require("koa-router");

const app = new koa();

app.use(koaBody({ multipart: true }));

const router = new KoaRouter();

router.post("/", ctx => {
  ctx.body = ctx.request.body;
});

app.use(router.routes());

if (process.env.NODE_ENV !== "production") {
  server.listen(port);
}

exports.callback = app.callback();

In postman if you enable the header Content-Type: application/json it hangs forever, but if you take it out then it responds immediately.

Decided to spend some time and dig into this and was able to replicate it on the functions emulator.
Looks to be an issue with the ctx.req stream already being "read" and  the raw-body dep doesn't handle that, see here:
https://github.com/stream-utils/raw-body/issues/57
https://github.com/stream-utils/raw-body/pull/58
Not sure what the solution is here though to actually get koa-body access to that data though...

Same issue

Same issue.
It happens with HTTP2 connections?


Feature: Go syntax highlighting

Similarly to https://github.com/onivim/oni2/issues/473, would it be possible to port over a Go syntax extension?

Should be resolved now with @NinoFurger 's PR #636


[SMALLFIX]fix a spell in UnderFileSystemFileOutStream line 79

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

Can one of the admins verify this patch?

ok to test

@wkxydcg Thanks for your contributions!
It looks like your commit is not associated with your github account. You can either add your email address to github (https://help.github.com/articles/adding-an-email-address-to-your-github-account/) or change the email address of your commits (http://stackoverflow.com/a/1320317/401884). Thanks!

Merged build finished. Test PASSed.

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

update with my own email!

Merged build finished. Test PASSed.

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


only one window tiles on secondary montitor since 0.3.6

Since updating to the latest release I sometimes have the issue that my secondary monitor (builtin macbook display) will only allow one window, if I drag a second window onto the montor, it will switch places with the window currently on that monitor.

Today i noticed an additional odd behaviour. Sometimes when moving a window from one screen to another it also moves other windows as well. See this new video

Do you think you could see if this also happens in v0.3.5? That's the version that introduced the commit i mentioned above.

Actually I told garbage in the first post, the issue was on 0.3.5, I thought I had 0.3.6 installed. Now I'm on 0.3.6 and as of now the issue didn't resurface. If it happens again I will post here, but for now I think it is resolved with 0.3.6.

okay it seems that the bug wasn't gone, at the moment im experiencing the same issue as in the video with a Safari window and a textedit window. Im currently on 0.3.8

It even happens when I move some windows around with the warp command
see this video where a wanted to warp the top left window -> east

Pretty sure it is the commit I mentioned earlier. I'll remove those changes.

I've removed the code I mentioned, on the latest master. Hopefully this fixes the issue you've been seeing.

Let me know if you are still having this problem and I will reopen the issue. It should be fixed though.


Procedure to install agents still uses old RPM repository

The documentation on how to install/configure a manually installed agents, still uses the pkg.inmanta.com RPM repository (See: https://docs.inmanta.com/community/dev/install/2-configure-agents.html#step-1-installing-the-required-inmanta-packages).

This seems to be fixed: https://docs.inmanta.com/community/dev/install/3-configure-agents.html#step-1-installing-the-required-inmanta-packages


Fetch pull request model for reviews in open state

Verify Review status to fetch only pull request models for open reviews to avoid having duplicate links on GitHub comment.
Bot finds multiple entries in pull request model table but only one of them is linked to open reviews and rest all are linked to closed reviews.
Unfortunately cosmos DB doesn't allow join across two containers.

Fixes #3862

Fixes #4035


Text to ASL Convertor

Is your feature request related to a problem? Please describe.
ASL is important. It is the only way to include specially abled people.
Describe the solution you'd like
I would like to design a Chrome extension for a text-to-ASL converter.

Why are other extensions labelled as level 2 and this as level 1? Just curious.


Automatic upgrade to Camel Quarkus 3

Quarkus has just published a doc page about the topic:
https://quarkus.io/guides/update-quarkus
We should check how well it works for CQ projects now and whether something needs to be fixed.

I started testing in some of our apps.
this is one issue to follow https://github.com/quarkusio/quarkus/issues/33805

We've had this since 3.2.x as part of the quarkus update tooling.


Downtarget Kestrel to NETStandard 1.3

Kestrel doesn't need any functionality beyond NS1.3 so down targeting helps more scenarios, including moving more aspnet projects to be down targeted.

Hi @mjsabby, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

cc @davidfowl

@pranavkm was also working on this. Can you also take a look?

I was waiting for the build to go through (We had a bad CI day). This looks :shipit:

Thanks for the PR!


Condense Enlarge effects

This merges the normal (level 2) and heightened (level 4) effects for the spell Enlarge. It also changes the flat modifier on dex-based checks to actually grant the clumsy condition. Partially addresses #4144.

This shouldn't have a choice set except for level 6+, and it doesn't warrant using a badge.


Package installers don't close on Debian 9 after tray and p2p successfully installed.

I can't close them, please help!!
Screenshot from 2018-01-23 12-17-14.png

Didn't ever face this error on Debian.


Adding Icons

Is there an easier way to add icons instead of using svg files? I have tried to add some custom icons and it doesn't seem to work properly (maybe I am formatting incorrectly).
Can the Solana icon inside the QR code be changed as well?
Thanks

All of the code is open source licensed, so it can be forked and modified. Changing the styles or replacing the Solana logo is your call, but I'd encourage you to consider how these changes might affect user recognition and their experience at the point of sale.
https://github.com/solana-labs/solana-pay/blob/master/core/src/createQR.ts is what contains the options to generate the QR. You could implement your own QR code styles by calling createQROptions and overriding what you want, or by providing your own options from scratch.

I just wanted to change the inside QR logo to USDC for example so the user knew they were paying with USDC not to hide the solana aspect that's all! Thank you


ComBat parametric adjustment returning NANs in Matlab

Dear Experts,
I am running ComBat in Matlab for Mac version <IP_ADDRESS>0202 (R2019b).
When I run the batch correction using parametric adjustments, I get all NANs. This has been a recurring issue for me, even in other datasets.
There are no NANs in my data and, in the past, the non-parametric adjustment has worked.
Any idea what might be causing this?
Thank you,
Leonardo Tozzi

@leotozzi88 Hi Leonardo, are there features that have a constant values across images in your data?

Indeed, I think this was probably the issue: I tried running the analysis in R and I got the warning saying that there were constant features. After removing those, the analysis ran.

@leotozzi88 Note to myself: add warning message for the Matlab version for such scenarios


Problem with fetchPage in softDeleted elements

Hi,
I'm trying to get soft deleted elements with the fetchPage function (added with the pagination plugin).
With the param withDeleted : true I get correctly my element, but, the count result not include the soft deleted elements :/
This is my request :
Model
    .forge()
    .where((qb) => {
        qb.whereNot('deletedAt', null);
    })
    .fetchPage({
        withDeleted : true
    });

And this is the result :
{
    models : [
        { ...mymodel },
    ],
    pagination : {
        page : 1,
        pageSize : 20,
        rowCount : 0,
    },
}

EDIT :
After research, the fetchPage method make two requests, one with elements (using limit/offset), one for only count. It's appear, the count request ignore the withDeleted attribute :
select `Routes`.* from `Routes` where `deletedAt` is not null limit ?

select count(distinct `Routes`.`id`) from `Routes` where `deletedAt` is not null and `Routes`.`deletedAt` is null

Have you a solution with this problem ?

Ok, I open an issue and create a PR directly in Bookshelf to fix this bug. I close this issue when Bookshelf team release the fix.
Issue : https://github.com/bookshelf/bookshelf/issues/1913
PR : https://github.com/bookshelf/bookshelf/pull/1914


How to show navigationbar

So by adding an event(swipe left) i am able to hide the navigation bar. but now i wish to show it again on swiperight. How do i do this? My code is :
$(renderer.doc).on("swipeleft",function(){
var autoHideNavigationBar =true;
window.navigationbar.setUp(autoHideNavigationBar);
window.navigationbar.hideNavigationBar();
});
$(renderer.doc).on("swiperight",function(){
autoHideNavigationBar =false;
window.navigationbar.setUp(autoHideNavigationBar);
});

MaheshSinghSawant, did you find a solution for showing the nav bar back? I'm struggling with the same issue.

any news on this? It could be really usefull to be able to show hidden bar. Thanks


improve error handling for unknown URLs; add tests

Right now, if you call BridgePF with an unrecognized URL, Play Framework throws a 400 with an HTML response body. This causes the JavaSDK ErrorResponseInterceptor to fail to parse the JSON and return a BridgeSDKException w/ status 400 instead of a BadRequestException.
This change makes it so the exception selection code is still called even if JSON parsing fails. This also cleans up some of the error handling logic and adds a suite of tests.
Testing done:

mvn verify (unit tests and other validation)
Manual Test: Changed URLs in local server. Pulled in changes to Integ Tests and ran UploadTest against local server. Verified in test output that a BadRequestException was thrown instead of a BridgeSDKException.

Looks good, I'm merging. It's surprising that Play throws a 400 for an URL that doesn't exist, it should throw 404. But even if it did, we would want to throw BadRequestException with that error code, not EntityNotFoundException (REST + HTTP status codes is ambiguous here, in one case it's 404 because they've used the wrong URL, definitely an error, in the other case some entity is missing, this is an error can happen at runtime). So a 400 BadRequestException is close enough.


☂ Linter: eslint-plugin-react-perf

https://github.com/cvazac/eslint-plugin-react-perf
Getting Started
just new-react-perf-rule <RULE NAME>

This will  create a <rule name>.rs file in crates/oxc_linter/src/rules/react-perf. Then, register the rule in crates/oxc_linter/src/rules.rs by adding the name to the react-perf module and in declare_all_lint_rules at the bottom.

NOTE: this task will be available once this pr has merged

Rules

[x] #2042
[x] #2043
[x] #2044
[x] #2045

Last task:

[ ] add a cli flag --react-perf-plugin and change all the rules to correctness.

How do these rules for extensions work? I'm using the oxlintrc.json configuration file.

How do these rules for Visual Studio Code extensions work? I'm using the oxlintrc.json configuration file.


Use secure versions of memset and strncpy in libtrie and libcpuid

What problem does this PR solve?
Issue Number: close #9280
Problem Summary: see #9280.
What is changed and how it works?

Replace the usages of `memset` and `strncpy` with their secure versions.

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[x] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

Replace the usages of `memset` and `strncpy` with their secure versions.

Closing as these are false-positives and the fix don't work for non-Windows platform at all.


Enchanted book not found

Hi, I'm using the latest Fabric 1.21.1 version of the mod. In fact, it doesn't work at all, I can't find this enchantment. But I can see this mod in the modmenu.

Hi, Lymno. Thank you for your report. I'll address this issue by the end of the day.

I've replaced fabric 3.0.0 mod-file with what resolves the issue. Hope you like it!


[Fix] Fix PPL bug

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Please describe the motivation of this PR and the goal you want to achieve through this PR.
Modification
Fix injected noise.
Whose help do you want? @ them here!
BC-breaking (Optional)
Does the modification introduce changes that break the backward-compatibility of the downstream repositories?
If so, please describe how it breaks the compatibility and how the downstream projects should modify their code to keep compatibility with this PR.
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist
Before PR:

[ ] I have read and followed the workflow indicated in the CONTRIBUTING.md to create this PR.
[ ] Pre-commit or linting tools indicated in CONTRIBUTING.md are used to fix the potential lint issues.
[ ] Bug fixes are covered by unit tests, the case that causes the bug should be added in the unit tests.
[ ] New functionalities are covered by complete unit tests. If not, please add more unit test to ensure the correctness.
[ ] The documentation has been modified accordingly, including docstring or example tutorials.

After PR:

[ ] If the modification has potential influence on downstream or other related projects, this PR should be tested with some of those projects.
[ ] CLA has been signed and all committers have signed the CLA in this PR.

Tested on stylegan2_c2_8xb4-800kiters_ffhq-256x256, score 353.


Image filter does not filter

Expected behavior
When filtering the image list from docker
Actual behavior
Loading images......spinner shows continuously
Information about the Issue
ver 0.17.5

I have this issue on macOS 10.12.6, see also #4255


Overhaul and fixes

Description
App overhaul
Issues Resolved

All the code was rewritten in Kotlin
Many memory leaks was fixed
App now compile for the latest SDK version, which means that the app will work on all devices
Behavior of most code was not touched, this is further work. Only fixes and replaces of some restricted legacy

Important
I need to check the full functionality of the application, assistance is welcome.

Wow! Thank you very much! I will have a look this Friday!

Unfortunately I won't be able to do this today, but I'll try to have a look on Sunday.

All seems to be running fine! Thank you!!

Thank you so much for your awesome work @berhram , this greatly simplified further development on the Android app!


[Q] Is there a way to reset / unpipe?

In my application I'd like to be able to disconnect / reconnect the underlying TCP connection w/o having to destroy and recreate the modbus client object. Is it safe to just call client.pipe(newSocket) (after already doing this with another connection/socket object) or will the object still have some references to the old one?

Yes, it should be if there's nothing pending around. Still, I wouldn't recommend.

OK, that's good enough for me for now. Thanks.


Gocd cicd actions broke in the sqlite submodule

Issue description
&2|00:30:47.177 STDERR: fatal: No url found for submodule path 'sqlite/thirdparty/mvsqlite' in .gitmodules
&2|00:30:47.177 STDERR: fatal: run_command returned non-zero status while recursing in the nested submodules of sqlite

log.txt
Steps to reproduce
Go to https://ci.v-sekai.cloud/go/pipelines#!/ and see it is red.
Minimal reproduction project (For SDK or engine issues)
None.
Using a released version?
No
What OS? Any other relevant information? (V-Sekai version, Godot revision, System information)
Windows 11, V-Sekai Godot Game, Vulkan Nvidia 3000

Resolved https://ci.v-sekai.cloud/go/tab/build/detail/godot-groups-editor/228/defaultStage/1/linuxJob


Added timeout methods to hikari.guilds.Member

Added timeout and untimeout methods to hikari.guilds.Member
Summary

Since ban and kick have their own methods, I thought adding timeout methods to the Member class would be useful.
Checklist

[ ] I have run nox and all the pipelines have passed.
[ ] I have made unittests according to the code I have added/modified/deleted.

Related issues

This is just an alias of Member.edit, which I see as unnecessary and just obfuscates things.
I -1 this

I can see how this'd make sense from a UX perspective, my assumption is that most of the time you're not going to be editing
any of the other fields on a member when you're timing them out and having a named moderation action method for it should make it generally easier to find how to time someone out from a doc search or from glossing over code.
But i'm honestly mostly indifferent to this idea tbh also "untimeout" just sounds weird

I think, if this were to be implemented at all, it is sufficient to pass None to communication_disabled_until in timeout. There is no need for untimeout. With that said I think Member.edit() is already more than enough for this.

I really appreciate the PR, but I dont think it is needed.
I agree it does make sense in a UX perspective, which is the idea behind this PR. But such a small alias of something that already exists in this object I don't see as necessarily helpful and just adds more code to maintain.
Again, thanks for the PR, but will be closing it


feat: switched out external secrets editing for populate to new library

Hi @chrismellard. Thanks for your PR.
I'm waiting for a jenkins-x or todo member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository.

Hi @chrismellard. Thanks for your PR.
I'm waiting for a jenkins-x or todo member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign rawlingsj
You can assign the PR to them by writing /assign @rawlingsj in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign rawlingsj
You can assign the PR to them by writing /assign @rawlingsj in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@chrismellard any chance you could rebase with the latest dependencies to get the new go mod stuff? I've been going through getting everything onto 0.20.2 with consistent dependencies

@chrismellard any chance you could rebase with the latest dependencies to get the new go mod stuff? I've been going through getting everything onto 0.20.2 with consistent dependencies

/ok-to-test
/lgtm

/ok-to-test
/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jstrachan
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jstrachan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jstrachan
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jstrachan]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


⚠ Bofrost.fr has degraded performance

In bc4dc6c, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 899 ms

Resolved: Bofrost.fr performance has improved in 994a88e after 6 minutes.


Way to get "pretty" y-axis number ticks

I have a bar chart with the following auto-generated y-axis ticks.
Is there a way to make it more visually appealing including:

nice rounded number ticks: e,g. 10,000,000, 20,000,000, 30,000,000 ... 120,000,000, 140,000,000
right aligned numbers so the commas line up

For the first item, I'm looking into a way to auto-generate those ticks myself, but was wondering if a solution was already available.

I now have a initial implementation here:
https://godoc.org/github.com/grokify/gotilla/math/mathutil#PrettyTicks
This gives me the following:

I also have an abbreviation function now for the following:
https://godoc.org/github.com/grokify/gotilla/strconv/strconvutil#Int64Abbreviation

this is great

I use this to get nice ticks:
import (
	"golang.org/x/text/language"
	"golang.org/x/text/message"
)

YAxis: chart.YAxis{
	ValueFormatter: func(v interface{}) string {
		if vf, isFloat := v.(float64); isFloat {
			p := message.NewPrinter(language.English)
			vi := int64(vf)
			return p.Sprintf("%d", vi)
		}
		return ""
	},
}


Pre-Trained Public Weights Not Downloadable

The weights are currently stored at the following locations, seem to be no longer accessible:
WEIGHT_NAME_TO_CKPT = {
"detr": [
"https://storage.googleapis.com/visualbehavior-publicweights/detr/checkpoint",
"https://storage.googleapis.com/visualbehavior-publicweights/detr/detr.ckpt.data-00000-of-00001",
"https://storage.googleapis.com/visualbehavior-publicweights/detr/detr.ckpt.index"
],
"deformable-detr": [
"https://storage.googleapis.com/visualbehavior-publicweights/deformable-detr/checkpoint",
"https://storage.googleapis.com/visualbehavior-publicweights/deformable-detr/deformable-detr.ckpt.data-00000-of-00001",
"https://storage.googleapis.com/visualbehavior-publicweights/deformable-detr/deformable-detr.ckpt.index"
],
"deformable-detr-refine_bbox": [
"https://storage.googleapis.com/visualbehavior-publicweights/deformable-detr-refine_bbox/checkpoint",
"https://storage.googleapis.com/visualbehavior-publicweights/deformable-detr-refine_bbox/deformable-detr-refine_bbox.ckpt.data-00000-of-00001",
"https://storage.googleapis.com/visualbehavior-publicweights/deformable-detr-refine_bbox/deformable-detr-refine_bbox.ckpt.index"
]
}
Do you have them hosted somewhere else?

same issue.. did you get these files? any new link?


"unable to get local issuer certificate" errors at the point of Firebase creation

Every time I try to set BlueBubbles up on my machine, I get an error after auth'ing to Google:
[GCP] Failed to create project: unable to get local issuer certificate
I have a pair of projects (BlueBubbles, BlueBubblesApp) in my account, and there's a Cloudflare proxy to connect to once I complete setup. When I try to log in via the Linux app, there are error messages/warnings about not having Firebase being required, and I can't log in.

Every time I try to set BlueBubbles up on my machine, I get an error after auth'ing to Google:
[GCP] Failed to create project: unable to get local issuer certificate
I have a pair of projects (BlueBubbles, BlueBubblesApp) in my account, and there's a Cloudflare proxy to connect to once I complete setup. When I try to log in via the Linux app, there are error messages/warnings about not having Firebase being required, and I can't log in.

For the issue you are having with the Google setup on the server, it looks like your mac might need to be updated? Not to the next version, but maybe a security update? Seems like it's unable to verify/get the firebase.google.com certificate, which is unusual. Do you have a proxy or network security setup through your router? Definitely not usual.
And as a result, it may also be causing the issues you are seeing with Firebase on the Linux App

My apologies for not responding, and just to close the loop: It looks like an aggressive blocklist on NextDNS was causing the issue: Firebase turns up on a few blocklists, including one I was using. Once I allow-listed it, everything worked perfectly.