Questions about custom blots

At first, I discovered the problem when I used blots/embed custom components in my project. At first, I thought it was a problem with my code. But then I looked at the more famous plug-in quill-mention and found that it also had the same problem.
Steps to reproduce quill-mention: (Website: https://quill-mention.com/)

Enter @ in the editor and select an option.
Manually delete the space after @ and adjust the input method to Chinese.
Enter, before hitting the space to confirm the input content, you will find that the temporary English in the input box will cause the style to be disordered.
After tapping the space to confirm the input content, the cursor moves to the front of the input content.

At the same time, I also noticed that blots/embed will generate two ""; in the element. I tried to delete this character, and the input problem was solved, but there was a problem with cursor positioning. Below is my screen recording, please help me take a look at it, thank you
https://github.com/user-attachments/assets/bbf8509e-6136-4c6c-a381-302ebd048b21

You can see the source code of quill-memtion on github. The code for the custom emoji expression in my project is as follows:
import { Emoji } from<EMAIL_ADDRESS>import './index.css';

const Embed:any = Quill.import('blots/embed');

// 定义 EmojiBlot 类，继承自 Embed
class EmojiBlot extends Embed {
  // 构造函数
  constructor(scroll: any, node: HTMLElement) {
    super(scroll, node);
    this.clickHandler = this.clickHandler.bind(this);
  }

  // 静态方法，创建 emoji 节点
  static create(data: Emoji): HTMLElement {
    const node = super.create();
    node.classList.add('emoji-wrap');
    this.createEmojiNode(node, data);
    return node;
  }

  // 静态方法，创建 emoji 的图片节点
  static createEmojiNode(domNode: HTMLElement, data: Emoji): HTMLImageElement {
    const { path, name } = data;
    const emojiDom = document.createElement('img');
    emojiDom.className = 'emoji';
    emojiDom.src = path;
    this.setDataValues(domNode, { name, path });
    domNode.appendChild(emojiDom);
    return emojiDom;
  }

  // 静态方法，设置数据属性
  static setDataValues(element: HTMLElement, data: { [key: string]: string }): void {
    const domNode = element;
    Object.keys(data).forEach((key) => {
      domNode.dataset[key] = data[key];
    });
  }

  // 静态方法，移除数据属性
  static removeDataValues(element: HTMLElement, datas: string[]): void {
    const domNode = element;
    datas.forEach((key) => {
      delete domNode.dataset[key];
    });
  }

  // 空的 clickHandler 方法
  clickHandler(): void {}

  // 静态方法，获取节点的数据属性
  static value(domNode: HTMLElement): DOMStringMap {
    return domNode.dataset;
  }
}

// 设置静态属性
EmojiBlot.blotName = 'emoji';
EmojiBlot.tagName = 'span';

// 注册 EmojiBlot
Quill.register(EmojiBlot);


Thread dead lock when changing the imagesource very fast

I observed a randomg freeze of my App. After some research I found that it seems that two threads inside ffimageloading deadlock.

in my case I have assigned an IMagesource in Xaml and replace it imediately after the Page appears.

Really, This error occurs.
I tested on Android, Xiaomi Redmi 4  (Android 6.0) - Heavy List example
So, app will freeze, if user starts to scroll fastly.

It should be fixed with the newest prerelease nuget. Could you check it?

I have this problem also. is it already fixed? which version has the fix? I am using version 2.2.9

@EmilAlipiev
You should use the latest PRErelease package.
You need to enable pre-release pachages channel.

Hi I tried latest pre Release and it still occurs for me. I dont know if it is deadlock but when i scroll fast within ListView,  list gets frozen and i get message that app is not responding. Do you have any Suggestion please?

Should be fixed in the newest prerelease. Please reopen if it still occurs.


Hotfix syntax img

Update images for the GUI of arx, application, apikeys.
Fix format of hyperlink regarding to the suggestion of Xander
Fix some broken paragaphs due to syntax error

The quickstart section and contact us button will be updated in the next PR

@mattmahdieh I fixed the docs by replacing old encryption primitives and changing from GUI/graphical interface -> dashboard. Plz approve if it make senses.


Missing ActivityFeedBinding

There is import here https://github.com/yigit/dev-summit-architecture-demo/blob/b5c6bd562d7a4fc7eebf19bd03573d7819ad3140/client/app/src/main/java/com/android/example/devsummit/archdemo/view/activity/FeedActivity.java of class which does not exist in the project:
import com.android.example.devsummit.archdemo.databinding.ActivityFeedBinding;

That was generated by the Data Binding Library automatically. More information on that here: https://developer.android.com/tools/data-binding/guide.html#generated_binding

@farmazon3000
As @savekirk already said, it is generated. Please close the issue if it is no longer a problem for you - thanks ;)


Need demo

That's a great work. If possible, could you also provide a test demo to show this model how effective in predicting. Because, it's not easy to using a such large model immediately.

Thank you for your interest in our work!
If you're looking to view the training results, we have provided the fine-tuned dataset and the source code, which you can directly use for testing. You can find the source data in ./data/paired_time_series_news_training_data/ and the fine-tuning code in finetune-lls4ts.sh. This should help you run tests and explore the model's effectiveness more easily.
If you’re looking for a user-friendly demo of the model, we are currently working on it. We’ll make an announcement when it's ready for public use.

Thanks! I think a quick-start demo may help your project get more attention.


Update CI to LLVM 17

Update clang-tidy and clang-format in CI to LLVM 17

Also add misc-include-cleaner to clang tidy checks.


Ender Book Crash

Minecraft version & Mod Version:
1.12.2
1.12.2-1.15.10
Single player or Server:
SP
Describe problem (what you were doing / what happened):
One of my users used the Ender Book, and crashed to desktop.
Original report: https://github.com/NillerMedDild/Enigmatica2Expert/issues/105
Crash log (if any):
https://hastebin.com/akawefegiv.sql
(If this is a suggestion and not a bug report, delete all this and just write your suggestion!)

I put a comment on the other issue.
I loaded version 1.28 of the expert back, havent replicated it yet so still investigating

Looks like i do have some chunk loading force action going in
https://github.com/PrinceOfAmber/Cyclic/blob/c7edb436d41868a015254d219c8909c45e5be6ea/src/main/java/com/lothrazar/cyclicmagic/item/enderbook/ItemEnderBook.java#L164
... seems like it has been there for a while, since at least 2017-07-11 according to git blame, but i cant think of what else would cause corruption like this

Ender wing does similar teleportation, investigating overlap https://github.com/PrinceOfAmber/Cyclic/issues/850

Just had a similar problem: http://tcpst.net/nuto
Was playing in multiplayer and just testing the book to see what it does. The other player, who was standing right next to me (and also next to my target location, as it was only a couple blocks away) heard the noise the book makes and then also crashed.

Yep Xandaros crash log also mentions playing sound "net.minecraft.client.audio.SoundManager.func_148605_d(SoundManager.java:252)"

Aha, i deleted all my custom sound effect calls and still heard a sound.  So vanilla was trying ot hit up the enderman built in sound, and mine was going on top (plus i was playing them twice and doing weird server/client stuff)
Cleaned that all out for next release so 1.15.13 will be more stable

Not sure if this will be a 100% fix but i cleaned up what i can let me know if it dies again https://minecraft.curseforge.com/projects/cyclic/files/2575589

Sounds good, thank you :)


code-settings-sync  Shan  2.4.2  hava a problem

VSCode Version: Code 1.8.1 (ee428b0eead68bf0fb99ab5fdc4439be227b6281, 2016-12-19T14:49:23.350Z)
OS Version: Windows_NT ia32 6.1.7601
Extensions:

Extension
Author
Version

html-snippets
abusaidm
0.0.18

project-manager
alefragnani
0.12.2

jquerysnippets
donjayamanne
0.0.1

vscode-great-icons
emmanuelbeziat
1.1.37

vscode-angular2-emmet
jakethashi
1.0.9

code-settings-sync
Shan
2.4.2

Steps to Reproduce:
1.code-settings-sync Shan 2.4.2
if i Disable this extension.
vscode will not show this problem.

Please try opening an issue against the extension: https://github.com/shanalikhan/code-settings-sync


Adding checks for enable and disable buttons

Adding checks for enable and disable buttons

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


veux store feature requests

ability to have multiple vuex stores, with options to disable some of them.

What would the disable feature do? Modify the existing store?

You shouldn't be using multiple stores. You can use modules if you want to split up your app logic

Agreed, that was 5 months ago!


Fix string interpolator error in HeartbeatReceiver

Error log before fixed
15/03/29 10:07:25 ERROR YarnScheduler: Lost an executor 24 (already removed): Executor heartbeat timed out after ${now - lastSeenMs} ms

Can one of the admins verify this patch?

Jenkins this is ok to test

Test build #29376 has started for   PR 5255 at commit c93f2b7.

Test build #29376 has finished for   PR 5255 at commit c93f2b7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29376/
Test PASSed.

Thanks. Merging in master & branch-1.3.

Actually never mind - only master. This doesn't exist in 1.3.


feat: reduce no. of worker creation for vb

Details(context, link the issue, how was the bug fixed, what does the new feature do)

Implementation note, gotchas, related work and Future TODOs (optional)

What are the steps to check the difference @raviteja83?


New recipes: Threema Desktop and Threema Work

Download Source:
https://threema.ch/de/work/download
https://releases.threema.ch/web-electron/v1/release/Threema-Work-Latest.dmg

Also a request from @maxschlapfer for Threema Desktop (consumer version).
https://threema.ch/en/download

Recipe for consumer version now done.


Language

I was wondering how you translate, the language
in my case I will translate to portuguese brazil

I have just created an example here
https://luca-vercelli.github.io/DataTable-AltEditor/example/09_translations/example9.html
If you do some translation, you could pull it here :
https://github.com/KasperOlesen/DataTable-AltEditor/tree/master/translations

very good!


build jdk on mac and ubuntu hit ##[error]The process '/bin/bash' failed with exit code 1

build jdk on mac and ubuntu hit ##[error]The process '/bin/bash' failed with exit code 1:
./mk-cacerts.sh: line 79: '/home/runner/work/build-jdk/build-jdk/jdk/boot'/bin/keytool: No such file or directory
5110
##[error]The process '/bin/bash' failed with exit code 1
https://github.com/AdoptOpenJDK/build-jdk/runs/1114107682?check_suite_focus=true

mk-cacerts.sh was added 5 days ago https://github.com/AdoptOpenJDK/openjdk-build/pull/2033/files

The reason is keytool is not available.


Unable to replicate performance on MABrax

Hi there :wave:
I've been trying to match the performance that you show in the paper, where you get around 5000 return for ant_4x2, however when I run your code locally I get around -600 after 10m timesteps with the config that you share on this repo (in your paper you get ~3500 at 10m timesteps).
If you could let me know any config differences or provide a script that is able to match the performance that would be great :+1:
Another question, when plotting your results in the paper do you sum all agent returns to get 5000 or are you taking the mean return for each agent?
Your paper results

Our local run of your repo

Thanks for the reply @gardarjuto! Sorry I'm going to re-open this issue as I don't get the results you are claiming and I think it might be useful for future people who look into this.
I set total timesteps to 3e8 and I'm only seeing ~750 return and it doesn't seem to learn passed 1e8 steps:

As a point of comparison Mava's feed forward IPPO gets ~1300 return, but again nowhere near 5000. I checked the brax version that installs with JaxMarl and it's now well passed 0.0.16, it's at 0.9.3, so I assume this is the issue. The question is why are you then requiring brax>=0.9? I don't think pinning to such an old version of brax is at all ideal (in fact I couldn't get it running), so are there any plans to update your benchmark - not necessarily in the paper, but possibly as a note in this repo so that people have a point of comparison for the current version of brax?


fix(swiperAction): 删除alipay-style依赖,修复swiperAction样式引入问题

删除alipay-style依赖,修复swiperAction样式引入问题

Codecov Report
Base: 4.90% // Head: 4.93% // Increases project coverage by +0.03% :tada:

Coverage data is based on head (90797b9) compared to base (596afee).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##           master    #556      +/-   ##
=========================================
+ Coverage    4.90%   4.93%   +0.03%
=========================================
  Files         155     153       -2
  Lines        3158    3138      -20
  Branches      802     801       -1
=========================================
  Hits          155     155
+ Misses       2619    2600      -19
+ Partials      384     383       -1

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


composer require fails with v2

Hello,
I've updated my app to L6 and it works well. PHP 7.3.
But I can't install laravel-stats:
→ composer require "wnx/laravel-stats" --dev
Using version ^2.0 for wnx/laravel-stats
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for symfony/console (locked at v4.4.4) -> satisfiable by symfony/console[v4.4.4].
    - Conclusion: don't install phploc/phploc 4.0.0
    - Installation request for wnx/laravel-stats ^2.0 -> satisfiable by wnx/laravel-stats[v2.0.0].
    - Conclusion: don't install sebastian/version 3.0.0|install phploc/phploc 4.0.0
    - Conclusion: remove sebastian/version 3.0.0|install phploc/phploc 4.0.0
    - wnx/laravel-stats v2.0.0 requires phploc/phploc ~4.0|~5.0 -> satisfiable by phploc/phploc[4.0.0, 4.0.1, 5.0.0].
    - phploc/phploc 4.0.1 requires sebastian/version ^2.0 -> satisfiable by sebastian/version[2.0.0, 2.0.1].
    - phploc/phploc 5.0.0 requires sebastian/version ^2.0 -> satisfiable by sebastian/version[2.0.0, 2.0.1].
    - Can only install one of: sebastian/version[2.0.1, 3.0.0].
    - Can only install one of: sebastian/version[2.0.0, 3.0.0].
    - Installation request for sebastian/version (locked at 3.0.0) -> satisfiable by sebastian/version[3.0.0].

Installation failed, reverting ./composer.json to its original content.

Do you have an idea?
Bests

Hi 👋
The output of composer require says:

Conclusion: don't install phploc/phploc 4.0.0

However, laravel-stats requires phploc/phploc:~5.0.
https://github.com/stefanzweifel/laravel-stats/blob/ac14fbade26b210ab2398336a79cd4fddef9beeb/composer.json#L19
Also sebastian/version is mentioned multiple times. Did you require any of these dependencies in your projects main composer.json?
Maybe you have to upgrade phpunit/phpunit? Or some other dependencies? Maybe you can run composer outdated and update some of your dependencies?

According to our test suite, the package can be installed with Laravel 6 on php 7.3

https://github.com/stefanzweifel/laravel-stats/runs/446872047?check_suite_focus=true#step:4:10
https://github.com/stefanzweifel/laravel-stats/runs/446872037?check_suite_focus=true#step:4:10

Thanks for your answer. The thing is I don't require these packages, they are all deep.
Everything is up to date, an extract of my composer.json:
{
    "require": {
        "php": ">=7.2",
        "adaojunior/passport-social-grant": "4.0.*",
        "aws/aws-sdk-php": "3.133.*",
        "barryvdh/laravel-snappy": "0.4.*",
        "caouecs/laravel-lang": "4.0.*",
        "cloudflare/sdk": "1.1.*",
        "doctrine/dbal": "2.10.*",
        "egulias/email-validator": "2.1.*",
        "fruitcake/laravel-cors": "1.0.*",
        "fzaninotto/faker": "1.9.*",
        "laravel/framework": "6.15.*",
        "laravel/passport": "8.4.*",
        "laravel/socialite": "4.3.*",
        "league/csv": "9.5.*",
        "league/flysystem-aws-s3-v3": "1.0.*",
        "league/fractal": "0.19.*",
        "mlocati/ip-lib": "1.9.*",
        "phpoffice/phpspreadsheet": "1.10.*",
        "predis/predis": "1.1.*",
        "sendinblue/api-v3-sdk": "6.1.*",
        "sentry/sentry-laravel": "1.6.*",
        "spatie/laravel-backup": "6.7.*",
        "spatie/laravel-image-optimizer": "1.5.*",
        "spatie/laravel-medialibrary": "7.18.*",
        "stripe/stripe-php": "7.25.*",
        "therobfonz/laravel-mandrill-driver": "1.0.*",
        "zanysoft/laravel-zip": "1.0.*"
    },
    "require-dev": {
        "barryvdh/laravel-ide-helper": "2.6.*",
        "beyondcode/laravel-dump-server": "1.3.*",
        "brianium/paratest": "4.0.*",
        "fabpot/goutte": "4.0.*",
        "lanin/laravel-api-debugger": "3.5.*",
        "laravel/tinker": "2.2.*",
        "mockery/mockery": "^1.3",
        "phpunit/phpunit": "9.0.*",
        "riimu/kit-phpencoder": "2.4.*"
    }
}

I'm a bit stuck :(

Thanks for providing your composer.json! This makes debugging such issues so much easier.
By using composer why phploc/phploc I could figure out, that the problem lies in the incompatibility of sebastian/version, phpunit/phpunit:9.* and phploc/phploc.
There's already an open issue in the phploc repository: https://github.com/sebastianbergmann/phploc/issues/204. However, the proposed solution of just using the phar, will not work for us 😬.
I will have to do some further testing/investigation, on how we could resolve this issue quickly, as I assume a lot of people will upgrade their projects to use phpunit v9 soon.

In the meantime I've released v2.0.1 which adds support for phpunit v9.

→ composer why phploc/phploc

  [InvalidArgumentException]
  Could not find package "phploc/phploc" in your project

Thanks for doing investigations! :)

Hello again,
The upstream issue in phploc/phploc has been closed and a new major version of phploc has been released.
I've upped the version constraints in this package in #167 and released a new patch version v2.0.2.
However, I couldn't reproduce your error anymore today on my machine (even before releasing v2.0.2).
I'm closing this issue for now. If you still can't install the package with the same error message, please just reopen this issue.

Great job @stefanzweifel it's OK now!
Have a nice day!


change id in example to string

TreeDataItem.id is a string, but example code was setting to a number.
interface TreeDataItem {
    id: string

Thanks for the fix.

Thanks for the fix.

Thanks the component 👌


[EventHub Schema Registry] REST API Review

Swagger PR: 15855

API Stewardship Review Board Meeting 7-Sept-21

There are a number of questions around how a user would discover/get the collection of schemas.
Individual schema IDs are passed with the event--then the GET can be used.
Why doesn't the swagger version match the folder version?  The folder has -preview.
What happens if someone creates a group called schemas with a schema called schemas? How do you resolve your routes
Why is everything coming back headers?
Is your POST an Upsert?  I can't understand why the doc comment is saying "Get ID for existing schema?
Where is the API for the version? We do not want to have "hidden APIs." This should be added.

C# Examples

Re: Changing Schema/Serialization Type to Content Type in headers:
One thing we can do that wasn't explicitly brought up in the meeting was to have our Content-Type return a known type, such as application/json but include an optional parameter to specify the exact type of schema that is being used. For example, an Avro schema is constructed with JSON, so the returned Content-Type would be something like application/json;serialization=avro.
Is this an option that makes sense to implement while keeping AutoREST happy? I believe @johanste and @mikekistler were looking into AutoREST compatibility.


Authelia Integration

I self host a bunch of services including pigallery2, and utilise Authelia for SSO. Is there any intention of integrating with something like Authelia? Authelia provides authentication services for services, and only after authentication will it allow requests through to back end web services. This stops the 'double login' when a user has to login pass Authelia, then a login for pigallery2. I know you can turn off authentication to pigallery2 but you need this for album sharing.
Would  a possible solution be to allow Authelia to pass the user via a header of 'Remote-User' - if this matches a registered pigallery2 user, assume this is 'authenticated'? This will possible also allow other similar services such as Authelia to integrate, but I only know of Authelia as that is what I am using.

Unfortunately this is not possible at the moment and I do not see a simple way to add it.  The app is not prepared for SSO.
Sharing link also creates a guest user with restricted access to the gallery. I think it would be challenging to set that part up.

Hi - its been a while and I finally taken the plunge and trying to implement this feature. Note I have zero previous Typescript2 experience so bear with me.
So far added 3 new configuration items

externalAuthenticationEnabled
externalAuthenticationHeader
externalAuthencationLogoutURL

I think I have this working mostly, where if its enabled, then it will check the configured header for the user and does a lookup in the PG2 users for a matching user (a pre-requisite to have the user configured in pg2). Anyway,
I am struggling with the logout -  when a user clicks Logout from the menu, that calls the backend logout and I can see you delete the user object from the session. I would also like to (if configured) redirect the whole browser window to the configured logout URL. I have tried res.redirect(URL) but this does not work as the menu item does not expect the return and redirect the window.location.href.
Do you have any tips or pointers how I can redirect the whole frame to the external URL to logout after calling PG2 logout?

A q first: Would you like to push this change back to the main codebase or you are setting this up for your own use-case?
If you want to make it available for others, I would rather wait for the extension support to be available. (in 1-2 months) and it through that. We can make it sure that this can be achieved through extension. Do not worry that if you already have this at place, moving it to an extension should be mostly copy-paste.
For the implementation question:
I would use only a single source of through for being logged in. In this case authelia. So I would make sure that the logout button logs out the user from there (i.e.: replacing the logout button to log users out there).
Then the app will need to be able to detect if the user is still logged in there.
Disclaimer: I do not know authelia and never implemented SSO.

I am doing this to fit my use case and I expect this same feature would be useful for other people who self host and obtain a Single Sign On solution. So was hoping to push this back via a pull request and see if you would review integrate if possible if you wish as its your app, your branch and your choice (one of the great things about open source).
I am happy to wait until this new extension support before completing.
As to the implementation

I was aiming to call logout on Authelia via the browser/app but am failing to do this. I only kept the existing functionality of deleting the user from the PG Session manager for completeness (and the aim of being lightweight and being able to run on a Pi) even though if serves not true purpose otherwise. A shame I can't logout - but not the end of the world here.

With respect to your extension changes - does this include 'standardising' or simplifying the URL's of resources for PG. I ask as for adding an external authentication mechanism you would need to specify URL's which do not need protection and URL's that do. For example for sharing an album in PG2, you would want the authentication mechanism to ignore authentication for those resources, otherwise any attempt to access any other album/resource would trigger the authenticator to check for permissions before either blocking or allowing.

On the extension question: yes I plan to add some simplified way to add REST endpoints with authentication.
About the orginal request:
I was playing around with selfhosted stuff and read a bit (very little) about authelia.
It seems to me that most revers proxies can use it. In that case I would just configure a reverse proxy with authelia and disable the authentication within the app.
Its good:

You do not have to implement it
Hopefully people with security experiences set it up.

I started using Nginx proxy manager front of my pigallery2 and authelia supports it:
https://www.authelia.com/integration/proxies/nginx-proxy-manager/

One of the great features I love in PG2 is the ability to share an album. Quite often I have galleries of events for external people I like to share with. A simple obfuscated link and password in PG2 works great for this. Turning off authentication in PG2 disables this brilliant feature.
I use traefik (takes more fiddling to set up but handles SSL Lets Encrypt certs simply when set up and I don't need to manage it ongoing).
The reasons I was trying to integrate a external header for the user id to pass through are
1 - Keep the album sharing capability
2 - Can map users to internal PG2 users and have PG2 authorisations (users vs admin vs guest).
Unfortunately 2.0 of PG2 has broken album sharing with a password (see https://github.com/bpatrik/pigallery2/issues/744).
Its not the end of the world if it isn't supported as I can work around this by simply keeping the main personal PG2 gallery as is and spin up another instance of PG2 on a different domain with a single album and  login managed by authelia or PG2.

Turning off authentication in PG2 disables this brilliant feature.
Ahh true. The assumption is if there is no password protection, why do obfuscation or anything, the whole gallery is public.

The reasons I was trying to integrate a external header for the user id to pass through are
1 - Keep the album sharing capability
2 - Can map users to internal PG2 users and have PG2 authorisations (users vs admin vs guest).

Good point its indeed good reason.


Extra methods

Adds extra methods - vibrant for getting the vibrant color of an image, average for getting the average color of an image.

So can I get the status of this PR?

Closed due to no direction


(Do not merge) Enable generators

This bundles the regenerator runtime when you use the generator syntax.
It would fix #125 and be the first step towards #51.
One issue I’m seeing right now is that minified bundle increases by 5kB min+gzip even if you don’t use generators because transform-runtime currently externalizes Babel helpers whether you want it or not, and they bring in some stuff from core-js including Symbol.
@kittens said he’s open to fixing it on Babel side by adding helpers: [true] to transform-runtime so we can turn them off.

@kittens said he’s open to fixing it on Babel side by adding helpers: [true] to transform-runtime so we can turn them off.

helpers: false babel/babel#3603

@gaearon @kittens
That doesn't seem like the right fix. Wouldn't it make more sense to either compile babel-runtime without the runtime transform, or else not pull in polyfills in helpers when polyfilling is disabled?
Those helpers shouldn't be pulling in polyfills via sniffing for their existence.
All else being the same, just externalizing the helpers ought to decrease bundle size.

It seems like a bug that using babel-runtime makes https://github.com/babel/babel/blob/v6.11.6/packages/babel-helpers/src/helpers.js#L8-L12 pull in a Symbol polyfill.

Released<EMAIL_ADDRESS>with the PR ^


api: Upgrade go-swagger version to v0.30.5

Also to add the renovate configuration for auto update version later. Just a note we might still need to run make generate-api manually till the work with self-hosted renovate with post-hook is done.
https://github.com/cilium/cilium/pull/30185

/test

/test

/test


Teletan matched string with [characters]{7} and thus also matched 8 c…

…har strings. It also used a strange character definition syntax with M-N-P-Z form, which would allow - char as well.
Generator was also a bit inefficient and was replaced by a simpler version.

Many thanks for your helpful work. Some of your proposals seem to be very elegant. The team will manage your work on Monday. BTW: Why did you expand in your regex the range notation in the original by a complete list? Is there anything wrong with the range notation 0-9 etc. ?

Hi @kreincke , I expanded to a list just that it's easier to generate TANs with a simple random number generator. There are libraries to generate random strings given a regex, but I didn't want to include an additional dependency to your project.
Further, I found the list of characters easier to be read than the regex pattern with too many skipped characters in the set.

HI @theodiefenthal Many thanks for this excellent explanation.

Oh dear, I just realized that I replaced generateTan instead of generateTeletan. Too much tan confusion at 2am :)
@dfischer-tech thankfully fixes this already in https://github.com/corona-warn-app/cwa-verification-server/pull/51 , so I won't add another commit+PR+proper unit test for valid tan and teletan here anymore.


Move to middleware v2

Middleware v2 is a big update to how we handle middleware. Instead of pointing Hoverfly to a script, we now provide Hoverfly with the script. Hoverfly will now manage the script itself.
The benefit of doing this is to

Avoid any issues with paths when starting Hoverfly in one directory and then changing directory to set the middleware
Allow middleware to be set remotely

This update also tidies up the middleware part of Hoverfly as well as the web interface for setting both local binary middlewares and middlewares over HTTP.

LGTM!


Support Bi boost

BigQuery
Syntax and data
Metric tables:

[x]  parser: make sure the statement includes metric only. https://github.com/Canner/canner-metric-layer/pull/214
[x]  sql rewrite: rewrite statement and query duckdb with pre-aggregation table https://github.com/Canner/canner-metric-layer/pull/214
[x] 🔥check return data format or types
[x] exclude unsupported syntax which can be executed in duckdb,e.g explain, show, describe, insert, create. https://github.com/Canner/canner-metric-layer/pull/214
[ ] provide a workaround for https://github.com/duckdb/duckdb/issues/5673

Storage:

[x] export data to parquet: https://github.com/Canner/canner-metric-layer/pull/206
[x] import data to duckdb: https://github.com/Canner/canner-metric-layer/pull/206
refresh:
[x] 🔥fix rate
[ ] expression

Canner
pre-condition
can used in canner
Storage:

[x] *🔥canner provide export parquet api: provide APIs to export parquets to cloud storage and duckdb support s3 and gcs only.
[x] *🔥cleaner: canner can clear temp parquets or provide APIs which can be used by accio
[ ] *🔥 sync result: wire protocol used
[x] *🔥 async result: web team sql explorer?
[x] 🔥 adjust refreshing codes

🔥 implementation of export and import data from canner

[x] 🔥export data from canner
[x] 🔥import data to duckdb: file base; ~s3, azure blob,~ local (no gcs in canner)

note: * implemente in canner  ; 🔥 critical
Others

[ ] memory management
[x] 🔥extract module for bi boost

https://github.com/duckdb/duckdb/pull/8302 has been merge
cc @liun03 @wwwy3y3 @brandboat

https://github.com/duckdb/duckdb/pull/8302 has been merge

Many thanks !!!


` character counted as space before closing parens?

Using the new ES6 syntax, I have several template strings passed as arguments to a function or method, formatted like this:
dothing(`url${variable}`)

Standard reports an error at the end of the line. The error text is:
Illegal space before closing round bracket

Confirmed.

Thanks for reporting! Fixed in 2.6.2.

:+1:


Sign in page

The same as log-in page, but with sign-up page form

completed the improvements, but  i was not able to add the svg images

saw your changes, and found the error. Re-opening PR to help you identify the changes that are required


Protocol version 30 is not supported.

Whenever I try to make a query through the php api, I get the error message "Protocol version 30 is not supported." I just got an enterprise version 2.1.0 edition. I noticed it only happens to the 2.1 version, does not have the problem with the 2.0 version.

Are you using version 1.2.0 of the driver?
https://github.com/Ostico/PhpOrient/releases/tag/v1.2.0
Scott

no, I am using 1.1.9 just checked it.

Thanks smolinari, v1.2.0 fixed the problem


[Bug] Invalid out_path in VisualizerHooks in some cases

Prerequisite

[X] I have searched Issues and Discussions but cannot get the expected help.
[X] I have read the FAQ documentation but cannot get the expected help.
[X] The bug has not been fixed in the latest version (dev) or latest version (1.x).

Task
I'm using the official example scripts/configs for the officially supported tasks/models/datasets.
Branch
1.1x branch https://github.com/open-mmlab/mmdetection3d/tree/1.1
Environment

Reproduces the problem - code sample

Reproduces the problem - command or script

Reproduces the problem - error message

Additional information
This line will ha ve undefined img_path for datasets without images.

Fixed in this PR: https://github.com/open-mmlab/mmdetection3d/pull/2342/files

Got it. Will close


[BugFix] Memmap-like for lazy stacked tds

Description
Allow for memmap-like with heterogeneous tds.

Codecov Report
Base: 87.97% // Head: 87.95% // Decreases project coverage by -0.03% :warning:

Coverage data is based on head (28fe729) compared to base (1a5d743).
Patch coverage: 71.42% of modified lines in pull request are covered.

:exclamation: Current head 28fe729 differs from pull request most recent head 2fb1e3a. Consider uploading reports for the commit 2fb1e3a to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #246      +/-   ##
==========================================
- Coverage   87.97%   87.95%   -0.03%
==========================================
  Files          28       28
  Lines        9572     9586      +14
==========================================
+ Hits         8421     8431      +10
- Misses       1151     1155       +4

Flag
Coverage Δ

linux-cpu
85.97% <71.42%> (-0.03%)
:arrow_down:

linux-gpu
87.88% <71.42%> (-0.03%)
:arrow_down:

linux-macos-cpu
?

linux-stable-cpu
86.01% <71.42%> (-0.03%)
:arrow_down:

linux-stable-gpu
87.88% <71.42%> (-0.03%)
:arrow_down:

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

Impacted Files
Coverage Δ

tensordict/tensordict.py
80.96% <71.42%> (-0.06%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Bare minimal Installation

Is there any way that I can do just install for applying pre-trained models to predict? FreeSurfer build for SynthSR does not work on Windows nor ARM based CPU (basically all new OSX).

Yes sure, you can install the code distributed here as well as the required libraries listed in requirements.txt and the trained models. Everything is explained in the README.
If you need additional guidelines for the installation, you can follow steps 1,2 and 4 of the installation section for SynthSeg (https://github.com/BBillot/SynthSeg), which has the same requirements.


reuse port option?

running multiple instances with reuse port option. is it possible?

Yes, once we let you set SO_REUSEPORT on the listening socket.
I also wonder what is the use case?

@eaufavor is there an eta on this?

What is the use case? We need to understand the use case to prioritize it. Thanks.

to have "worker mode" on some of the existing integration with program


implement wasi-sql interface in SpiderLightning

Describe the solution you'd like
To be mergeable, this should also come w/ at least one implementation.
Additional context
n/a

~done


Ruckus

Submission Template

Ruckus
Virtual SmartZone (vSZ)
5.1 to 6.0
Status:  Affected
Update Available:  No
Notes: Also affects SmartZone, Unleashed Multi-Site Manager (UMM) and FlexMaster products
https://support.ruckuswireless.com/security_bulletins/313
https://support.ruckuswireless.com/security_bulletins_downloads/313?type=pdf
https://support.ruckuswireless.com/security_bulletins_downloads/313?type=txt
Last Updated:  2021 12 13

Thank you for providing this information and we will update the list.


Tree View - Checkbox Tree "Flickering"

Describe the bug
When the checkbox on the top node is clicked multiple times (quite quickly), the checkboxes of the child nodes start to "flicker".
How to reproduce
Stackblitz Example
Steps to reproduce the behavior:

Click checkbox on node "Group A" twice very quickly (like a double click)
Again, click checkbox on node "Group A" twice very quickly
Note that the checkboxes of the child nodes start to flicker endlessly

Expected behavior
No "flickering" should occur.
Versions
App

Angular: 7.2.0
Node: ?
Clarity: 1.0.4

Device:

Type: HP zBook
OS: Windows 10
Browser Chrome
Version 71.0.3578.98 (64-Bit)

Additional notes
The problem seems to get worse the more components are present on the page. As it might be difficult to reproduce the issue on another environment, I'm attaching a screen capture from my machine.
Tree View Flickering Video Capture.zip

This issue is also causing me grief. Here's an example: Stackblitz example

I spent some time reproducing this on our local dev app and investigating. As I suspected, making the (clrSelectedChange) EventEmitter synchronous solves the flickering problem. It also introduces Expression Has Changed errors which is the reason we made it asynchronous in the first place.
What that tells me is that the flickering is due to Angular’s asynchronous event emitters (which we do not control) not being able to fire all the events in a single change detection cycle, thus triggering multiple change detection cycles with a partial state updated for each.
The problem with the solutions you suggested above is that:

They seem to assume the problem is Clarity not being to update all checkboxes in time before the second click. That’s actually not the case, our update is instantaneous for trees of that size. As I mentioned, what’s taking time is the event each child fires when the parent is clicked, that one is asynchronous and delayed by Angular. At that point the two-way binding re-triggers an input with Angular, and we get these weird loops where children of a node can't agree.
As soon as we tell Angular async event emitters to fire the event, it’s out of our hands. There is no way for us to detect if they’re all done firing or not. Note that delaying the processing of a parent being toggled doesn’t change anything: if Angular only has time to fire at most 20 event emitters between two change detection cycles, no matter how much you delay the processing this 20-events window will still cause the flickering.

If based on my explanation above anyone has an idea for a solution, I’ll be happy to take a look at it and help you contribute. Or I also might have just missed some aspect of the issue and there is indeed an obvious solution. That would be awesome.
In the meantime, this will be prioritized like any other issue we have, because the investigation itself of a potential fix is long enough that we have to schedule time for it during our plannings. (That's what our "Needs investigation" label means, by the way.)

We are also hitting the issue, I've implemented a work-around for the moment.

Buggy behavior with clr-tree-node provided checkbox: https://stackblitz.com/edit/clr-tree-native-selection
Non-buggy behavior with custom checkbox inside clr-tree-node: https://stackblitz.com/edit/clr-tree-custom-selection

Hi all, I am working on a fix and am validating it against the various reproductions here. PR is up at #3227
@gssjr You are also affected by this same bug, but also are not setting the selection correctly for all of the child nodes on your own. The second example (without mutating the tree) is better, but the expression changed errors are on your side.


Remove entrance-related presets

Hi,
I've noted that every door interface give me building=entrance tag value combination which is supposed to be deprecated (according to the wiki https://wiki.openstreetmap.org/wiki/Tag:building%3Dentrance).
There is entrance=main in the autocomplete field, but not entrance key with the value yes. I guess it's kind of paradoxal to not use the proper tag for entrance for an app called every door 😛😛 (joke, but I seriously think your app has a good potential)
Also it would be nice for those two tags to have some input to add addr=* in combination with the entrance.

You are supposed to add entrances via the dedicated entrances mode, not via the [+] button. Removed these presets from 0.7.


.babelrc support?

I want to add support for dynamic import syntax using this ,babelrc:
{
  "plugins": ["syntax-dynamic-import"]
}

However, it seems to be ignored.
buble is used with webpack (roots.io sage9 theme).
// ...
      {
        test: /\.js$/,
        exclude: [/node_modules(?![/|\\](bootstrap|foundation-sites))/],
        use: [
          { loader: 'cache' },
          { loader: 'buble', options: { objectAssign: 'Object.assign' } },
        ],
      },
// ...

This issue may be related: https://github.com/babel/babel-loader/issues/552
{ loader: 'buble', options: { objectAssign: 'Object.assign', babelrc: true } },

doesn't work either.

I'm not sure what you're trying to do. While Babel and Buble are both ESNext-to-ESNow transpilers, they are totally different implementations. Buble doesn't consume Babel configuration or plugins at all.
If you meant imlementing an rc file for Buble (i.e. have it scan for a .bublerc if no config is passed), that would totally be possible. However, it will still be incompatible with .babelrc for the said reason above.

@fskreuz: Thanks. So for this particular project I would have to switch to Babel for supporting this feature (syntax-dynamic-import).


Footer page content update

Let's add some more substance to the Footer page with the following items:

What it is
Why we need it
What are the policies around it

Can people customize it? (NO)
When to use the minimal footer

Just added a Box note with suggested content updates here
Also created a PR based on the above content update suggestions.
https://github.com/carbon-design-system/ibm-dotcom-library-website/pull/249

Closing it since the udpates have been made!


kernel: add a work queue that runs on the main thread.

Add an optional work queue that runs on the main thread after the
application's main() function returns, to allow for the reuse of stack
memory allocated for initialization.
Resolves #21322.
Signed-off-by: Josh Gao<EMAIL_ADDRESS>

Ping: is this approach acceptable for #21322? I think it's the most elegant solution, but I can investigate alternatives if we don't want to take this patch.

Not giving this a -1, but note there was another workq patch recently that had to mutate the API a bit to allow finer control over where the queue ran.

@andyross Can you link it please?

Not giving this a -1, but note there was another workq patch recently that had to mutate the API a bit to allow finer control over where the queue ran.
My suggestion there, and here, is that what we really want is an API that exposes the work queue main loop to the user as a C function, e.g.
/**
  * Enters the main loop for the specified queue in the current thread, and does not return.
  */
FUNC_NORETURN void k_work_queue_exec(struct k_work_q *queue);

Obviously this would have to be documented and used carefully, but it avoids a lot of the churn we're seeing where people don't like our default work queue thread scheme.

Yeah, that was my initial thought, but the problem of how to initialize the queue would make this a highly error-prone API (e.g. if you're appending items to the work queue in response to a callback that happens at different initialization levels depending on what board you're targeting. There's a macro to statically initialize a k_queue, and I could add one for k_work_q as well, but it's still a rather sharp-edged API surface for something that seems (to me, at least) to be useful for only this one use case.
(More specifically: why would you want to spawn a thread into a work queue, instead of just creating a work queue and appending your thread main onto it? In the case of the main thread, it's because you can't. My understanding is that aren't (shouldn't be?) any other threads with reclaimable resources that a user would want to reclaim in this way.)
I can understand the desire for having a more general k_work_queue_{exec, exit, etc.} interface, but that seems to be way more of a rats nest (e.g. do you allow reentrancy? what about work queues execing each other?). I'm not saying that I'm unwilling to implement this, but I think it needs consideration of whether it's actually worth implementing, and if so, what needs to be implemented.

Rebased onto master


Support for "Auto-Checkout On Paste"

Is it possible to support for "Auto-Checkout On Paste" ? I have enabled "Auto-Checkout On Edit", but it won't be affected on ctrl+v. Typing is working well.
thx.

Hi. I will try to look into it when I've got time. It's currently a known limitation/bug with that feature.
Thanks.

Thanks.


Query limit disappears when overriding scope to unscoped

Hi,
somewhere in the wiki I found a way to force RailsAdmin to use unscoped scope, instead of the default_scope, like this:
list do
  scopes [:unscoped]
end

It does work, but unfortunately it also gets rid of query limit. Instead of setting the limit to 20, it sets it to 0 and tries to load my whole DB.
Is this supposed to happen or is this a bug? Is there a way to use unscoped with correct limit?
Thanks!

If I'm not mistaken, the scope limiting is happening in config.fields.association.rb#41:
register_instance_option :associated_collection_scope do
      associated_collection_scope_limit = (associated_collection_cache_all ? nil : 30)
      proc do |scope|
        scope.limit(associated_collection_scope_limit)
      end
end

Seems to me like the easiest thing for you to do is simply re-scope the limit after you do a full unscope.
list do
  scopes([:unscope]).limit(associated_collection_scope_limit)
end

I haven't been able to test the above solution (writing this from my phone) so let me know if things go awry.

Hey @CollinGraves, thanks for the reply. Unfortunately, your idea doesn't work because scopes([:unscope]) returns just an array of symbols ([:unscoped]), so I can't set the limit in there. Any other ideas?

I'm closing this as I don't see how this is possible without changing rails admin code. Calling unscoped on a Mongoid::Criteria object removes previously applied limit. If any other scope is used (other than unscoped), this works as expected.


fixing IE remove() bug

fix for the issue #18
;)

lgtm! thank you. I'll do the build and version bump right away


[feat] add files property, as counterpart to .apexignore

It is safer to decide what to include in the zip, rather than blacklist what not to include.
NPM for instance uses a "files" property in the package.json (which actually refers to paths to include, not individual files). A similar functionality would be great in apex.

My functions only ever have what I want to deploy (go binaries in my case) but happy to take a pull-request if anyone wants this, could be nice to have!


Fix Patch 1

Hi, fixed following:

syntax error due to line break inside conditional
verification of content-type during registration / login

Ad 2 - I think you accidentally committed without the piece of code that you referred to in your comments (https://github.com/jcs/bitwarden-ruby/issues/13#issuecomment-348612985), I also fixed it so it would properly handle data coming from the login form.

Reviewed and tested !
Thanks ! Hope it will be merged into JCS (official) repo.


Redrawing Shell Prompt

Bug when only token is a newline character, redraws shell prompt as many time as newline char was only input

Seems like that was fixed by you man! :+1:


Map style updates

Vegetation color clashes with teal clusters

Aapeli will help Jesse set up maputnik, etc

@aapeliv was this one completed by 34834? If so can we close this one?

It wasn't, but I'm not sure if this is a priority. Closing and @jesseallhands can reopen if he thinks it's important. Jesse I can show you how to edit the map layer!


Fix spec for summarization and translation outputs

Output key must be summary_text instead of generated_text. See https://huggingface.co/docs/api-inference/detailed_parameters#summarization-task.
EDIT: same for translation. See https://huggingface.co/docs/api-inference/detailed_parameters#translation-task.

Thanks for the quick reviews :)


can not evaluate with CAPTURE metric with multiprocessing samples

Does anyone run into the following problem?

This error is caused by wrongly formatted input predictions and ground truths.  Please refer to this repo's homepage for expected input formats. CAPTURE requires just the same input format as other metrics in pycocoevalcap.


Push to Nuvola Player project

Seems to work ... nothing fancy

Hello @fastcat. Could you port your script to the Nuvola SDK? sudo pip3 install -U nuvolasdk; nuvolasdk convert-project should do the trick. If not, please create a bug report.

Done -- 2d65986ec852bf19d1061157031c8df9e9d906b6
I should note that rendering of Google Keep in WebKitGTK is currently broken -- this is not a nuvolaplayer3 issue, as I verified it is also a problem in the WebKit MiniBrowser.  Something seems to be wrong with float positioning I'm guessing, causing all the notes (and all the list items inside a checkboxes-enabled note) to render atop one another.
I've also noticed that the logic to detect links that should open in an external app, inherited from the Google Calendar integration, is broken in both.  I have a suspicion this is related to click tracking -- links that functionally open in new windows don't have the initial new window flag set that the integration JavaScript uses to detect "external" links.

Great :-) Could you transfer the repository to the Nuvola Maintainers team? You will be still the main maintainer.

I should note that rendering of Google Keep in WebKitGTK is currently broken -- this is not a nuvolaplayer3 issue, as I verified it is also a problem in the WebKit MiniBrowser. Something seems to be wrong with float positioning I'm guessing, causing all the notes (and all the list items inside a checkboxes-enabled note) to render atop one another.

It occurs also in the latest unstable WebKitGTK 2.15.91. How experienced with HTML, CSS and JavaScript are you? Are you able to find a hot fix? Or could you create a bug report at WebKitGtk?

I've also noticed that the logic to detect links that should open in an external app, inherited from the Google Calendar integration, is broken in both. I have a suspicion this is related to click tracking -- links that functionally open in new windows don't have the initial new window flag set that the integration JavaScript uses to detect "external" links.

Well, this is a bug introduced in tiliado/nuvolaplayer@b8716717. So I've just fixed it in the master branch. The fix is also available in the master branch of flatpak builds and will be later propagated to other builds as well.
Could you test the flatpak build? DEB/RPM builds of new scripts are not going to be provided.
apt install flatpak xdg-desktop-portal-gtk
reboot # to add Flatpak directories to various system paths
flatpak remote-add --from gnome https://sdk.gnome.org/gnome.flatpakrepo
flatpak remote-add --from nuvola https://dl.tiliado.eu/flatpak/nuvola.flatpakrepo
flatpak install gnome org.gnome.Platform//3.22
flatpak install nuvola eu.tiliado.Nuvola//master eu.tiliado.NuvolaAppGoogleKeep//master

Also, your Tiliado account has been upgraded to a developer account. (3.1 Rolling Releases are currently available only to developers and patrons.)

The decision to go pure-flatpak begets the obligatory: https://xkcd.com/927/
Adding a new package management system and installing duplicate copies of system libraries using tools that don't integrate with the distro system & security update tools ... just for one app ... not high on my list of things I'm likely to do.
I love nuvolaplayer, and I understand why something like flatpak is attractive from a developer point of view.  But only supporting that ends up being pretty hostile from a user point of view.
☹

Thanks for sharing your concerns about flatpak builds. Every feedback is important in this phase, especially the negative one. I believe Flatpak builds of Nuvola are a huge step forward because Flatpak finally, after years of struggling, allows Nuvola project to pursue its goals and users will benefit from it. (I have been writing an article on this topic.)

The decision to go pure-flatpak begets the obligatory: https://xkcd.com/927/

Well, there are now only two competing standards - Flatpak and Snap - and I think there is healthy competition between them.

Adding a new package management system and installing duplicate copies of system libraries using tools that don't integrate with the distro system & security update tools ... just for one app ... not high on my list of things I'm likely to do.

This is a pretty valid point. I'm afraid Nuvola will lose some users because of that. On the other hand, it may get new users from distributions where Nuvola packages have never been available or are poorly maintained.

I love nuvolaplayer, and I understand why something like flatpak is attractive from a developer point of view. But only supporting that ends up being pretty hostile from a user point of view.

The decision to support only Flatpak is simply driven by the lack of manpower as the Nuvola project has only a single core developer. It is still possible to provide DEB/RPM packages if volunteers are found to take over this task. Are you interested?

Well, there are now only two competing standards - Flatpak and Snap - and I think there is healthy competition between them.

And deb and rpm and Arch's thing, and Slackware's thing, and ...
My point here is mainly that Flatpak and Snap are not the "native" package format of any major distribution (AFAIK).  Getting one to be the native/primary package manager for a major distro might be as hard as e.g. getting Red Hat to switch to DEBs.  Or maybe they get lucky and some distros ship native support for them in their core package manager -- that seems more likely, but still all speculation.

It is still possible to provide DEB/RPM packages if volunteers are found to take over this task. Are you interested?

To be honest, at the moment I'm giving Google Play Music Desktop Player a whirl.  Thus far it has upsides and downsides compared to NuvolaPlayer (packaging and otherwise).  GPM itself seems to run smoother in it for whatever reason (presumably down to the different browser core), and flash-less playback works out of the box, but of course it doesn't have the easy extensibility of NuvolaPlayer to other/custom web apps.
If I come back to NuvolaPlayer, I would be interested in contributing to the packaging, at least for Debian/Ubuntu packages.  I think I have the know-how (I've done some basic dpkg work before), and I definitely have the resources (hardware with sufficient disk space and cpu/memory to run VMs/containers to do clean builds).

Well, there are now only two competing standards - Flatpak and Snap - and I think there is healthy competition between them.

And deb and rpm and Arch's thing, and Slackware's thing, and ...

These are not competitors of Flatpak and Snap at all. The mission of Flatpak and Snap is to provide cross-distribution framework for distribution of sandboxed (desktop) apps. They are designed to coexist with traditional package managers, not to replace them.

My point here is mainly that Flatpak and Snap are not the "native" package format of any major distribution (AFAIK). Getting one to be the native/primary package manager for a major distro might be as hard as e.g. getting Red Hat to switch to DEBs.

That's not the goal of Flatpak nor Snap.

Or maybe they get lucky and some distros ship native support for them in their core package manager -- that seems more likely, but still all speculation.

There is no need for that in low-level core package managers. in case of high-level package managers, GNOME Software 3.22 already supports Flatpak.

If I come back to NuvolaPlayer, I would be interested in contributing to the packaging, at least for Debian/Ubuntu packages. I think I have the know-how (I've done some basic dpkg work before), and I definitely have the resources (hardware with sufficient disk space and cpu/memory to run VMs/containers to do clean builds).

Looking forward to that :-)

These are not competitors of Flatpak and Snap at all. The mission of Flatpak and Snap is to provide cross-distribution framework for distribution of sandboxed (desktop) apps. They are designed to coexist with traditional package managers, not to replace them.
...
There is no need for that in low-level core package managers. in case of high-level package managers, GNOME Software 3.22 already supports Flatpak.

Aah, and now I understand the disconnect.  I was coming at it from the context of "I mostly live on the command line and need to run apt/yum update/upgrade on 30+ machines, Flatpak/Snap don't play nice with that."
And if GNOME's package management GUI can process Flatpak & Snap, I'm sure some creative person could eventually update apt & yum to process them too, even if I'm not going to hold my breath there.


Thing tile persistance

Find some way to persistently save the information as to where the tiles are located. When a user moves a tile's location somewhere else, they shouldn't have to do it again when they reload the website.
This could be stored in pds ruleset or an ent in manifold.

The Manifold SPA now dispatches to the Manifold Pico, sending position information of every thing tile that changed per drag or resize. The Manifold pico stores this information in a map, keyed by the thing names. On page load, this data is sent to the SPA.
Changes are shown in this commit: https://github.com/Picolab/Manifold/commit/7832c383c813b2373ed0175e23d9458b7a95a8e8

Yay!!

On Nov 13, 2017, at 3:00 PM, Nicholas Angell<EMAIL_ADDRESS>wrote:
The Manifold SPA now dispatches to the Manifold Pico, sending position information of every thing tile that changed per drag or resize. The Manifold pico stores this information in a map, keyed by the thing names. On page load, this data is sent to the SPA.
Changes are shown in this commit: 7832c38 https://github.com/Picolab/Manifold/commit/7832c383c813b2373ed0175e23d9458b7a95a8e8
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/Picolab/Manifold/issues/10#issuecomment-344073137, or mute the thread https://github.com/notifications/unsubscribe-auth/AAQyh6hl9GDuZu8hMGKjYv85U1GryGkhks5s2LvsgaJpZM4O39Fj.

Current bug: While making changes to the layout of the Things page saves positions in the Manifold Pico, switching to an alternate page in the sidebar, such as Communities, and back to Things will revert any changes made if the page is not refreshed or reloaded.
The mapStateToProps function uses the initial state obtained at page load to fill in the grid any time the Things page is opened, such as when switching to Communities to Things. A change should be made such that the data is removed after initial load, or updated to the page state.

I am experiencing two bugs right now. 1) When deleting a pico, that pico's position data isn't cleared from the entity variable in the manifold pico. 2) On an initial creation, I get the whole "cannot read property of x" error. If I refresh, it works fine.

Deleting a Thing from the Manifold SPA with the manifold/remove_thing event will clear the position data from the Manifold pico. Errors in which the positions of the Things cannot be read are now dealt with by giving default values to the Thing tiles.
Note that the Github version of the manifold_pico.krl ruleset was updated. I have not run into an error with Thing creation and the "cannot read property x of undefined" error.


✨ Add HomeLive for database connection animation

This pull request introduces a new LiveView module, SequinWeb.HomeLive, which creates an animated welcome page for the Sequin application. The page transitions through three states: welcome, connecting, and connected.
Key features:

Animated transitions between states with fade-in and fade-out effects.
Responsive design with dark mode support.
Database connection test and information display.
Error handling for failed database connections.

The module includes:

Mount function to initialize the LiveView.
Render function with conditional rendering based on the current step.
Component functions for each step (welcome, connecting, connected).
Handle_info callbacks to manage state transitions and animations.
Helper functions for determining the next step and retrieving database information.

This addition enhances the user experience by providing visual feedback during the application startup and database connection process.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#139  👈
#138
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @RTLS and the rest of your teammates on  Graphite


Add button animations

Fixed in https://github.com/hiluw/to-doot-list/commit/48bb8f4a86fc91264abe98766bead90198d8cd50

I've only added it to one button. 🤔

Fixed in https://github.com/hiluw/to-doot-list/commit/536f8b1185aabc31316eb69b986f44308c744da7


Added gMSA accounts support

Currently gMSA accounts are not supported by agent.
Recovered changes made previously.
Issue: https://github.com/microsoft/azure-pipelines-agent/issues/1294
Testing: tested with gmsa account on a local machine with Active Directory account, prepared unit tests for NativeWindowsServiceHelper (IsManagedServiceAccount method)

@mmrazik could you please take a look?

/azp run


rp-radio cannot find dependacy mumble-voip

Error listed in title is stoping me from using radio, mumble voip works fine, I tried pma voice w the radio for that and it gave me the same error

It's because the resource is called "mumble-voip-master", not "mumble-voip".
And you never should download the code file, instead download the latest released file.


new lang tag, chinese + russian lang generators

I thought it would be nice to check string encoding, if you have possibility to generate strings in  different languages.

Hi @musinit,
Thanks for the PR.
Would you mind do a linter checking first?
In your local, just run this commands
# Downloading the linter
$ make lint-prepare

# Apply the linter
$ make lint

Sure!

@bxcodec how is this PR for you? If all in all it's ok as an idea and realisation, I will cover it with more tests to pass all checks.

Overall looks good.
So basically, you just change the character, isn't it?
I'm not sure yet about the correctness, but I'm good. Please add more coverage if possible 🙏

Don't understand, what means codecov/patch — 91.83% of diff hit (target 94.92%)..hope someone will help me to understand what should I change (more test coverage?)

Don't understand, what means codecov/patch — 91.83% of diff hit (target 94.92%)..hope someone will help me to understand what should I change (more test coverage?)

It's okay, I think it affected by previous code.
The coverage for the whole project is already surpass the previous one, so it's okay


Move the IngestionService to it's own package (as it's not "meta")

Codecov Report

:exclamation: No coverage uploaded for pull request base (ingestion_config@f22341e). Click here to learn what that means.
The diff coverage is 100%.

@@                 Coverage Diff                 @@
##             ingestion_config    #6160   +/-   ##
===================================================
  Coverage                    ?   83.28%
  Complexity                  ?    12694
===================================================
  Files                       ?     1470
  Lines                       ?    47918
  Branches                    ?     5875
===================================================
  Hits                        ?    39907
  Misses                      ?     5289
  Partials                    ?     2722

Impacted Files
Coverage Δ
Complexity Δ

...main/java/io/crate/ingestion/IngestionService.java
100% <ø> (ø)
15 <0> (?)

...rc/main/java/io/crate/metadata/MetaDataModule.java
100% <ø> (ø)
5 <0> (?)

...te/ingestion/IngestionDDLClusterStateModifier.java
86.66% <ø> (ø)
5 <0> (?)

sql/src/main/java/io/crate/plugin/SQLPlugin.java
100% <ø> (ø)
16 <0> (?)

...rate/metadata/rule/ingest/IngestRulesMetaData.java
90.16% <ø> (ø)
43 <0> (?)

sql/src/main/java/io/crate/plugin/SQLModule.java
100% <100%> (ø)
2 <0> (?)

Continue to review full report at Codecov.

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


feat: Add priority field to workflow steps and DAG tasks. Fixes #1092.

Fixes #1092

I think it'd be good to add an example to the examples folder

I think it'd be good to add an example to the examples folder

Will do that tomorrow as we as add an example to this PR.

Still needs changes.

This PR is 2 months old. Is it abandoned?

This PR is 2 months old. Is it abandoned?

No, should be ready to merge.


METRON-1134: Allow parser command line options to be specified in the zookeeper parser config.

Contributor Comments
It would be useful to maintain sensible defaults for various properties in the construction of parser topologies within the sensor parser config in zookeeper. This, along with a UI, would allow people to manage defaults per sensor in a single place. CLI should take precedent and override anything stored in zookeeper.
Manual testing via fulldev should focus on ensuring the following conditions hold for the properties able to be specified in the config:

Specify the property in the config.
Start parser and ensure that the property is preserved.
Specify the property via the CLI param
Start parser and ensure that the CLI param is honored

The properties able to be specified in the config are as follows:

spoutParallelism : The kafka spout parallelism (default to 1).  This can be overridden on the command line.
spoutNumTasks : The number of tasks for the spout (default to 1). This can be overridden on the command line.
parserParallelism : The parser bolt parallelism (default to 1). This can be overridden on the command line.
parserNumTasks : The number of tasks for the parser bolt (default to 1). This can be overridden on the command line.
errorWriterParallelism : The error writer bolt parallelism (default to 1). This can be overridden on the command line.
errorWriterNumTasks : The number of tasks for the error writer bolt (default to 1). This can be overridden on the command line.
spoutConfig : A map representing a custom spout config. This can be overridden on the command line.
securityProtocol : The security protocol to use for reading from kafka.  This can be overridden on the command line and also specified in the spout config via the security.protocol key.  If both are specified, then they are merged and the CLI will take precedence.
stormConfig : The storm config to use.  This can be overridden on the command line.  If both are specified, they are merged with CLI properties taking precedence.

Pull Request Checklist
Thank you for submitting a contribution to Apache Metron.
Please refer to our Development Guidelines for the complete guide to follow for contributions.
Please refer also to our Build Verification Guidelines for complete smoke testing guides.
In order to streamline the review of the contribution we ask you follow these guidelines and ask you to double check the following:
For all changes:

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes:

[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?

[x] Have you included steps or a guide to how the change may be verified and tested manually?

[x] Have you ensured that the full suite of tests and checks have been executed in the root metron folder via:
mvn -q clean integration-test install && build_utils/verify_licenses.sh

[x] Have you written or updated unit tests and or integration tests to verify your changes?

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

[x] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via site-book/target/site/index.html:
cd site-book
mvn site

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
It is also recommended that travis-ci is set up for your personal repository such that your branches are built there before submitting a pull request.

#post-777

Testing Instructions beyond the normal smoke test (i.e. letting data
flow through to the indices and checking them).
Preliminaries
Set an environment variable to indicate METRON_HOME:

export METRON_HOME=/usr/metron/0.4.1

Deploy the dummy parser

Edit $METRON_HOME/config/zookeeper/parsers/dummy.json:

{
  "parserClassName":"org.apache.metron.parsers.json.JSONMapParser",
  "sensorTopic":"dummy"
}

Create the dummy kafka topic:
/usr/hdp/current/kafka-broker/bin/kafka-topics.sh --zookeeper node1:2181 --create --topic dummy --partitions 1 --replication-factor 1
Persist config changes: $METRON_HOME/bin/zk_load_configs.sh -m PUSH -i $METRON_HOME/config/zookeeper -z node1:2181
Start via $METRON_HOME/bin/start_parser_topology.sh -k node1:6667 -z node1:2181 -s dummy

Send dummy data through

Edit ~/msg.json with the following content:

{ "guid" : "guid0", "sensor.type" : "dummy", "timestamp" : 100 }

Send msg.json through to kafka via cat ~/msg.json | /usr/hdp/current/kafka-broker/bin/kafka-console-producer.sh --broker-list node1:6667 --topic dummy

Test Case: Update via patch

Patch the message in ES and create a new field 'project' by executing
the following:

curl -u user:password -X PATCH --header 'Content-Type: application/json' --header 'Accept: */*' -d '{
  "uuid" : "guid0",
    "sensorType" : "dummy",
    "patch" : [
    {
      "op": "add"
        , "path": "/project"
        , "value": "metron"
    }
  ]
}' 'http://node1:8082/api/v1/update/patch'

Validate that the message has a field 'project':

curl -XPOST 'http://localhost:9200/dummy*/_search?pretty' -d '
{
  "_source" : [ "project" ]
}
'

Test Case: Update via replace

Replace the message in ES and create a couple of modifications:

new field new_field == "brand new"
modified timestamp == 7
Execute the following:

curl -u user:password -X POST --header 'Content-Type: application/json' --header 'Accept: */*' -d '{
     "uuid" : "guid0",
     "sensorType" : "dummy",
     "replacement" : {
       "source:type": "dummy",
       "guid" : "guid0",
       "new_field" : "brand new",
       "timestamp" : 7
      }
   }' 'http://node1:8082/api/v1/update/replace'

Validate that the message has a field 'project':

curl -XPOST 'http://localhost:9200/dummy*/_search?pretty' -d '
{
  "_source" : [ "new_field", "timestamp" ]
}
'

Test Case: Validate that the changes persist to HBase

Execute echo "scan 'metron_update'" | hbase shell
Ensure that

There is one row: guid0
There are 2 columns
The values associated with the columns correspond to the two modifications done in the tests above

Does the UI not support this then?

Or Rest?

The UI and REST do allow you to override (they construct CLI params for starting parsers), but there is no way to save these params between starts.  This would allow people to specify the configs in zookeeper so they would persist between parser starts.

So we need a ui jira for this?

Yeah, we would need a UI JIRA to have people set this for the sensors.  It provides some value by itself and it should not be a regression, but in order to change it in the UI, it'd need a hook.

Also, to my knowledge, the other value it provides is being able to set these configs in such a way that ambari starting parsers actually is configurable (just not configurable directly from the ambari config).  Essentially this allows users to specify their own defaults rather than take the defaults in code.

We will need to integrate this in with 1136, maybe we should make it 'part' of it.

I need to think of how this need to integrate with the archetype and extension installer

So, correct me if I'm wrong, but it should be independent of the archetype and extensions.  They define the operation of the parser, this would be entirely specifying the configs for the storm topology.  It should interact with the archetype and extensions the same way that the CLI options interact with it.

Maybe I'm miss understanding.  Does this introduce per parser configuration?

Per parser instance config.  It's not defaulting config for all instances of a parser (i.e. not defaults for all Grok parsers, but you can specify defaults for the squid parser in the parser config in zookeeper).

To my knowledge, even in a post-777 world, you're still specifying this via the CLI (which is the only way to specify it now in Metron).

maybe a little better explanation of 'how' we are doing this will help unfog me?

Ok, I think maybe a better explanation is in order then.  Currently, when you start a topology, you can specify certain properties of the topology via the CLI.  These are things like the spout parallelism, number of executors, etc.  Originally these were added to the CLI because they were the things that you needed to adjust to scale a given parser.  What this PR is about is giving you the alternative to specify these topological or structural properties in the parser config in zookeeper.  This enables the admin to specify a config setup for a given instance of a parser in zookeeper without having to specify the CLI options.
Now, it bears mentioning that this is also backwards compatible.  For these configuration options mentioned in this PR's description, the order of precedence is:

CLI
Zookeeper
which is to say that properties specified in the CLI will be honored first.

In a post-777 world, you would still have these CLI parameters for the basic topological properties (likely) and the config is still stored in zookeeper, so there really should be no conflict between this and what's going on in 777, I think.
Now, as to the 'how', well, when starting the parser, in the CLI code, we pull the zookeeper config and figure out the parameters to ParserTopologyBuilder.build(...) which gets invoked to construct the parser topology, like it did prior to this PR.

Yeah, we definitely need this.

I am +1 on this.  Though I think it should be documented in the tuning guide as well at some point.

I tested this in full dev and everything worked as advertised.  The only major issue I see is that the num workers property is missing from SensorParserConfig.  I feel like this is one of the most important settings for a topology.  You could set it with "topology.workers" in the "stormConfig" property but it's not obvious and I think this setting should be a first class citizen like the others.
For me, the use of the ValueSupplier functional interface made it more difficult to understand when reviewing.  The code seemed more verbose and complex and I wasn't able to work out the benefit of using it in this case.  It's is a very minor style issue and is just my (probably worthless) opinion.  I would not hold up the PR over something like this, just some honest feedback.
Awesome work.  If we can promote num workers to a getter method in SensorParserConfig then I'm confident giving a +1.

@merrimanr Thanks for mentioning num workers.  I just missed it when going through; I intended to add all relevant things to override, so I'm going to add num workers and num ackers too.  I'll add them now.
The intent of the use of suppliers here is to keep the logic for parameter overrides in the CLI without needing the CLI to necessarily have to know how to pull in the sensor parser configs (we currently keep that functionality in the ParserTopology class.  I believe this separation of concerns has some value, so I wanted to provide a way where we could keep the logic in the CLI without having to interact with zookeeper and move logic around, thus moving from "here's the value" to "here's a way to get the value" type semantics.  I could see how that might be confusing since it's creating some indirection where there was no indirection before.  Perhaps I can add some comments to that effect justifying the reasons and it'll be clearer?

Alright, I've added both a comment about the supplier abstraction and support for num workers and num ackers.

Thanks for the writeup on ValueSupplier.  I think you went above and beyond documenting it and I appreciate that.  it makes sense now that I have my head around it :)
I noticed the numWorkers and numAckers were missing in a couple places.  The test is probably not necessary but I think we want it in the README.

Ok, added the workers and ackers in the README.  Take a look and let me know if you see anything else.

Looks good +1


Failed to run start command

Describe the bug
failed to run start command
https://stackblitz.com/run?file=src%2Fapp%2Fproduct-list%2Fproduct-list.component.html
Link to the blitz that caused the error
https://stackblitz.com/run?file=src%2Fapp%2Fproduct-list%2Fproduct-list.component.html
Steps to reproduce
while running the angular-practice-project that is provided in angular official documentation I got the error "filed to run the commad"
Expected behavior
the code should run properly
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots
No response
Platform
Browser name  = Chrome
Full version  = 103.0.5060.53
Major version = 103
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/103.0.5060.53 Safari/537.36 Edg/103.0.1264.37
performance.memory = {
  "totalJSHeapSize": 67442897,
  "usedJSHeapSize": 63701401,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Hash = c4393a56cbb8e2fec13bc31575603ce9568f347d

Additional context
No response

I believe this is a duplicate of #724. We are aware of the issue and trying to resolve it.


[Bug]: Particles still appearing without control

Contact Details
<EMAIL_ADDRESS>What happened?
I was looking for a solution, but i give up, i start my number of particles in 5, but, when i just move the scroll or i just get afk, the particles keep increasing.
This image is when i open the page,

But, after a minutes, this happens:

I really don't know what i'm doing wrong, thanks for your time.
tsParticles Version
@tsparticles/react: ^3.0.0  @tsparticles/slim: ^3.5.0
tsParticles Configuration
const options: ISourceOptions = useMemo(
    () => ({
      background: {
        color: {
          value: "#111",
        },
      },
      fullScreen: false,
      fpsLimit: 120,
      interactivity: {
        events: {
          onClick: {
            enable: true,
            mode: "push",
          },
          onHover: {
            enable: true,
            mode: "repulse",
          },
        },
        modes: {
          push: {
            quantity: 3,
          },
          repulse: {
            distance: 100,
            duration: 0.4,
          },
        },
      },
      particles: {
        color: {
          value: "#ffffff",
        },
        links: {
          color: "#ffffff",
          distance: 200,
          enable: true,
          opacity: 0.5,
          width: 1,
        },
        move: {
          direction: MoveDirection.none,
          enable: true,
          outModes: {
            default: OutMode.out,
          },
          random: false,
          speed: 1.2,
          straight: false,
        },
        number: {
          density: {
            enable: false,
          },
          value: 5,
        },
        opacity: {
          value: 0.5,
        },
        shape: {
          type: "circle",
        },
        size: {
          value: { min: 1, max: 5 },
        },
      }
    }),
    [],
  );

What browsers are you seeing the problem on?
Firefox, Microsoft Edge
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Can you update @tsparticles/slim to 3.6.0 and let me know if the problem is fixed? Otherwise please provide a full sample with the issue

I had some problems updating but i did it, and still the problem,  but i found the reason, i know why particles appear just using the scroll. It's because the component detect the clicks out of its canvas box. So, when i click in some place of the page, appear more particles. Do u need that i upload my page to reply the problem? Give me some time i didnt uploaded my page to any host. Thanks for your reply

I had some problems updating but i did it, and still the problem, but i found the reason, i know why particles appear just using the scroll. It's because the component detect the clicks out of its canvas box. So, when i click in some place of the page, appear more particles. Do u need that i upload my page to reply the problem? Give me some time i didnt uploaded my page to any host. Thanks for your reply

No if you already found the issue no need to send a sample. The default value for interactivity.detectsOn is window because it has the best behavior, but since you don't need it, you can use canvas instead, this detects clicks only inside the canvas and not in the whole window.

thank you so much, sorry i didn't know


502.5 error - VS 2017 15.7.1; .net core 2.0.8 mvc

Hello - I am encountering the same issue posted earlier #681 with VS2017 15.7.1 Enterprise. I have created a .net core 2.0 (AspNetCore 2.0.8) mvc project. made some cosmetic changes to the default template and published to azure app service. the publish was successful but got 502.5 error when viewing it.
i have tried the wwwroot wipe, and it was not successful.
In the event log, i got the following:
8000808c.100010Keywords1129163281ApplicationRD00155DFE9F35Application 'MACHINE/WEBROOT/APPHOST/XYZCOM' with physical root 'D:\home\site\wwwroot' failed to start process with commandline 'dotnet .\xyz.com.dll', ErrorCode = '0x80004005 : 8000808c.100420Keywords1129293625ApplicationRD00155DFE9F35Warning: Could not create stdoutLogFile ?\D:\home\site\wwwroot\logs\stdout_8344_201851116448.log, ErrorCode = -2147024893.100010Keywords1129293921ApplicationRD00155DFE9F35Application 'MACHINE/WEBROOT/APPHOST/XYZCOM' with physical root 'D:\home\site\wwwroot' failed to start process with commandline 'dotnet .\xyz.com.dll', ErrorCode = '0x80004005 : 8000808c.
Is there any other information i can post here so that we can have better idea?
Thank you,
Henry

2.0.8 hasn't been made available on Azure yet. Expect an announcement next week.

@Tratcher, thank you for the information.  I will try to downgrade it and redeploy.

Downgraded to 2.0.3;
closed my VS 2017;
cleaned the solution;
redeployed and it was successful.

I ran into this problem multiple times in the past. Where can we find out the current versions supported on Azure? I am yet to find documentations on the versions supported.

https://github.com/aspnet/IISIntegration/issues/476


The Vite Config does not process ESBuild define option

Describe the bug
The Vite configuration does not process ESBuild define option.
Reproduction
Refer README of https://github.com/trivikr/vite-esbuild-define-test
System Info
System:
    OS: macOS 11.4
    CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
    Memory: 189.07 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 14.17.4 - /var/folders/1v/nghjtx0s3hg_s7xz_73cjjg47fny27/T/fnm_multishells/74000_1628473634393/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 6.14.14 - /var/folders/1v/nghjtx0s3hg_s7xz_73cjjg47fny27/T/fnm_multishells/74000_1628473634393/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 92.0.4515.131
    Firefox: 78.10.1
    Safari: 14.1.1
  npmPackages:
    esbuild: 0.12.19 => 0.12.19
    vite: 2.4.4 => 2.4.4

Used Package Manager
yarn
Logs
yarn run v1.22.10
$ vite -c vite.esbuild.config.js --debug
  vite:config bundled config file loaded in 48ms +0ms
  vite:config using resolved config: {
  vite:config   esbuild: { define: { global: 'window' } },
  vite:config   configFile: '/Users/trivikr/workspace/vite-esbuild-define-test/vite.esbuild.config.js',
  vite:config   server: { fs: { strict: undefined, allow: [Array] } },
  vite:config   configFileDependencies: [ 'vite.esbuild.config.js' ],
  vite:config   inlineConfig: {
  vite:config     root: undefined,
  vite:config     base: undefined,
  vite:config     mode: undefined,
  vite:config     configFile: 'vite.esbuild.config.js',
  vite:config     logLevel: undefined,
  vite:config     clearScreen: undefined,
  vite:config     server: { fs: [Object] }
  vite:config   },
  vite:config   root: '/Users/trivikr/workspace/vite-esbuild-define-test',
  vite:config   base: '/',
  vite:config   resolve: { dedupe: undefined, alias: [ [Object] ] },
  vite:config   publicDir: '/Users/trivikr/workspace/vite-esbuild-define-test/public',
  vite:config   cacheDir: '/Users/trivikr/workspace/vite-esbuild-define-test/node_modules/.vite',
  vite:config   command: 'serve',
  vite:config   mode: 'development',
  vite:config   isProduction: false,
  vite:config   plugins: [
  vite:config     'vite:pre-alias',
  vite:config     'alias',
  vite:config     'vite:dynamic-import-polyfill',
  vite:config     'vite:resolve',
  vite:config     'vite:html',
  vite:config     'vite:css',
  vite:config     'vite:esbuild',
  vite:config     'vite:json',
  vite:config     'vite:wasm',
  vite:config     'vite:worker',
  vite:config     'vite:asset',
  vite:config     'vite:define',
  vite:config     'vite:css-post',
  vite:config     'vite:client-inject',
  vite:config     'vite:import-analysis'
  vite:config   ],
  vite:config   build: {
  vite:config     target: [ 'es2019', 'edge88', 'firefox78', 'chrome87', 'safari13.1' ],
  vite:config     polyfillDynamicImport: false,
  vite:config     outDir: 'dist',
  vite:config     assetsDir: 'assets',
  vite:config     assetsInlineLimit: 4096,
  vite:config     cssCodeSplit: true,
  vite:config     sourcemap: false,
  vite:config     rollupOptions: {},
  vite:config     commonjsOptions: { include: [Array], extensions: [Array] },
  vite:config     dynamicImportVarsOptions: { warnOnError: true, exclude: [Array] },
  vite:config     minify: 'terser',
  vite:config     terserOptions: {},
  vite:config     cleanCssOptions: {},
  vite:config     write: true,
  vite:config     emptyOutDir: null,
  vite:config     manifest: false,
  vite:config     lib: false,
  vite:config     ssr: false,
  vite:config     ssrManifest: false,
  vite:config     brotliSize: true,
  vite:config     chunkSizeWarningLimit: 500,
  vite:config     watch: null
  vite:config   },
  vite:config   env: { BASE_URL: '/', MODE: 'development', DEV: true, PROD: false },
  vite:config   assetsInclude: [Function: assetsInclude],
  vite:config   logger: {
  vite:config     hasWarned: false,
  vite:config     info: [Function: info],
  vite:config     warn: [Function: warn],
  vite:config     warnOnce: [Function: warnOnce],
  vite:config     error: [Function: error],
  vite:config     clearScreen: [Function: clearScreen]
  vite:config   },
  vite:config   createResolver: [Function: createResolver],
  vite:config   optimizeDeps: { esbuildOptions: { keepNames: undefined } }
  vite:config } +5ms
  vite:deps Hash is consistent. Skipping. Use --force to override. +0ms

  vite v2.4.4 dev server running at:

  > Local: http://localhost:3000/
  > Network: use `--host` to expose

  ready in 273ms.

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

By default, ESBuild is applied to ts, jsx and tsx files in vite.

By default, ESBuild is applied to ts, jsx and tsx files in vite.

Thank you @dejour for your comment.
I noticed that it's mentioned in https://vitejs.dev/config/#esbuild
I've updated my repro repo to include .js files as follows:
 // Refs: https://vitejs.dev/config/
 export default defineConfig({
   esbuild: {
+    include: [".js"],
     define: {
       global: "window",
     }

The issue is still reproducible https://github.com/trivikr/vite-esbuild-define-test

try using this way
esbuild: {
    include: [/\.js$/],
    exclude: [/\.jsx$/],
    define: {
      global: "window",
    },
  }

Verified that passing [/\.js$/] instead of [".js"] fixes the issue.
 // Refs: https://vitejs.dev/config/
 export default defineConfig({
   esbuild: {
-    include: [".js"],
-    exclude: [],
+    include: [/\.js$/],
+    exclude: [/\.jsx$/],
     define: {
       global: "window",
     },

Should this be fixed in vite, as it accepts string as well as Regex in include/exclude params?


Support for regions in supported languages

The library now supports region names in all six supported (as of 16th of May 2024) languages


MNE Epochs notch_filter feature request (re: #12092)

Describe the new feature or enhancement
Currently, notch_filter can be applied to raw objects but does not have the functionality to filter epochs, as brought up previously by user hyphenzhao in request #12092. With the way I set up my pipeline, I epoch the data early on to apply other cleaning steps and am intentionally applying the notch filter at a later step. It would be very useful to be able to apply notch_filter to epoched data, in my case to avoid converting epochs back into a raw object in order to apply the notch filter.
Describe your proposed implementation
User hyphenzhao described this nicely in their post. Briefly, my understanding is that each epoch could be treated as a separate record, and the width of the notch filter could be adjusted partly based on the epoch length.
Describe possible alternatives
The alternative is to reconstruct a raw object from the epoched data by concatenating the epochs into a continuous numpy array, then using mne.io.RawArray to create a new raw object. This is what I am doing now in order to use the notch filter, but I'm worried it may introduce edge artifacts. How would you recommend handling possible edge artifacts introduced by the conversion back into a raw object?
Additional context
No response

@ikarabinas in principle I think this should be doable by moving notch_filter from this class:
https://github.com/mne-tools/mne-python/blob/195a2cc8009160fd125e355b0280e903a941c874/mne/io/base.py#L1147
to this class
https://github.com/mne-tools/mne-python/blob/195a2cc8009160fd125e355b0280e903a941c874/mne/filter.py#L2459
Would you be up for trying a PR to make this change?

@larsoner Great! I hope this works. Let me know if the edits I made in my PR reflect the changes you were referring to.


Loading of the CSS file

Hi
I think for the description in the README.md there is a little mistake.
Loading of the CSS should be done like this:
<link rel="stylesheet" href="dist/attribution.css">
and not with loading the plugin.js file, isn't it?
Thanks for the cool plugin

Hi @moikorg,
Well spotted, yes, the plugin's css-stylesheet should be loaded here, of course.


update lalrpop

lalrpop was pretty out of date which can spread to crates bringing in lots of conflicts of old dependencies, inflating compile times,
crate sizes, etc. this updates lalrpop to the latest minor version. it added a new token "UnexpectedEOF" however I saw unexpected tokens message was about "unexpected eof" so i coped most of the implementations for diagnostics from them , assuming the implementation would be correct. I'm happy to change it though, I admittedly didn't look to deep since i seemed to find something that worked.

Happy New year! Sorry for the delay reviewing this change.
Thanks for this!

Happy New year! Sorry for the delay reviewing this change.
Thanks for this!

No worries at all! Hope you had a lovely new year!

No worries at all! Hope you had a lovely new year!


Feature/113/add readme toc

Pull Request
Description:
Tried out that recommended npm package - actually works quite well. I've placed the comments in the README.md to tell the code where to keep the table of contents, and configured the update-readme script to run the command updater and the TOC updater both.
Related Issue:

Closes #113

I noticed the command script didn't update the README, might have to look into it


Add compiled binaries to releases

I think either something like this, or adding a Makefile, for people who don't have/don't want to install Gradle.

I think either something like this, or adding a Makefile, for people who don't have/don't want to install Gradle.

Any Makefile would still need Gradle unless you made your own Java build system. Anyways gradlew downloads it for you, just run ./gradlew build (make sure Java is installed).

I mean you could do something like
JCFLAGS:=# ...

all:
    javac $(JCFLAGS) ./path/to/files*/*.java
    # ...
    jar -cfm Manifest.txt ./path/to/files*/*.class
    # ...

Since Gradle is chonky, but it's up to you anyways.

I'll just add install instructions.


fix missing batch header partial files

Workaround for incorrectly created partial files, where entries exist without a BatchHeader record.

Codecov Report

Merging #1296 (61ddac7) into master (a88b247) will increase coverage by 0.00%.
The diff coverage is 90.00%.

:exclamation: Current head 61ddac7 differs from pull request most recent head f656fc7. Consider uploading reports for the commit f656fc7 to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1296   +/-   ##
=======================================
  Coverage   88.58%   88.58%
=======================================
  Files          73       73
  Lines        7080     7099   +19
=======================================
+ Hits         6272     6289   +17
- Misses        476      477    +1
- Partials      332      333    +1