Feature Request: Save the filter for each PROJECT, BORADS or SPRINT.

Firstly, this product is really what I want, it really let us focus on jobs.
But if there're more projects or boards to be managed, it will be very hard to manage the filters, because the filter will reset every time I switch the projects list.

I suggest to change the Close button to Save filters button would be better, like this: [Clear filters] [Save filters]! 😋

@turnerguo Thank you for the proposition.
We just need to save filters by default(without save button).
We will do it soon.

I'm new to development/contributing to open source. I'd love to take a shot at this if that's okay.

@parmira5 Hi! Thank you for your request, but this feature not quite suitable to start contributing.


feat(umd): Add UMD build

closes #8

Current coverage is 100% (diff: 100%)

Merging #71 into master will not change coverage

@@           master   #71   diff @@
===================================
  Files          10    10
  Lines         173   173
  Methods        37    37
  Messages        0     0
  Branches       18    18
===================================
  Hits          173   173
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 6882daa...ed96652


Sanity-checking our Hellinger Distance Implementation

I have been comparing our implementation of the Hellinger distance for two multivariate normal distributions with the equation defined on Wikipedia, and I am left with two questions:

Is the gamma parameter utilized in our multivariate_hellinger function supposed to be 0.125 (or 1/8) instead of 0.00125?  https://github.com/quinngroup/ornet/blob/e604527847574b23cb94caafe33b4766ef003390/src/measure.py#L59
Are we returning 1 minus the actual distance value?https://github.com/quinngroup/ornet/blob/e604527847574b23cb94caafe33b4766ef003390/src/measure.py#L81-L86

That does indeed seem like a bug. But from what I can tell, it was your Nov 2019 commit that introduced this code--so my question would be, what reference were you using?

1-dist becomes a similarity metric, so my guess is we're using the metric down the road for spectral analysis of an affinity matrix. But it would be good to double-check this is actually happening.

If I am remembering everything correctly, I referred to a code snippet that Mojtaba had written around that time and directed me to since I was still unfamiliar with the metric. I'm glad someone else was able to look over it to make sure I wasn't missing anything (again).
Ok perfect. This lines up with how we have been utilizing it, but I had noticed that a lot of documentation still referred to the metric as the distance, rather than the similarity metric, except in the affinityfunc.py file.


Adding CMS PlaceholderFields for the Catalog model.

These PlaceholderFields have been added because Django CMS Apphooked
views can not make use of the placeholders that appear in inherited
templates. These new PlaceholderFields allow further customization of
each Catalog's detail view.

 Can't create container plugin on catalog page

:ship:it


[3.5beta1 server]ERROR: Script does not inherit a Node: res://

Godot version
v3.5.beta1.mono.official [b9b23d222]
System information
Debian 11
Issue description
I tried to use server and it give me an error:
Godot Engine v3.5.beta1.mono.official.b9b23d222 - https://godotengine.org

Mono: Log file is: '/home/vale/.local/share/godot/app_userdata/Authentication/mono/mono_logs/2022-02-01_13.58.03_1320802.log'
ERROR: Script does not inherit a Node: res://autoload/Signals.cs
   at: start (main/main.cpp:1904)
ERROR: Script does not inherit a Node: res://autoload/Setup.cs
   at: start (main/main.cpp:1904)
ERROR: Script does not inherit a Node: res://autoload/Authenticate.cs
   at: start (main/main.cpp:1904)
ERROR: Script does not inherit a Node: res://autoload/GameServers.cs
   at: start (main/main.cpp:1904)
ERROR: Script does not inherit a Node: res://autoload/DbHelper.cs
   at: start (main/main.cpp:1904)

where , for example, Signals have a Node like any of others:
using Godot;
using System;

namespace DesertEdge.Singleton
{
    public class Signals : Node
    {
        public override void _Ready()
        {
        }
    }
}

If I use headless server it work properly.
I followed the guide line posted here: https://docs.godotengine.org/en/stable/tutorials/export/exporting_for_dedicated_servers.html
So I tried to use .pck file - as stated in docs -  but same error.
I tried: (godot-server.64 is the Godot server donloaded form website -mono version )

~/myproject/godot-server.64
~/myproject/godot-server.64 --main-pack myproject.pck
~/myproject/myproject.64  <-- renamed with same name as my myproject.pck

Still same errors..
Steps to reproduce
see before
Minimal reproduction project
NA

@git2013vb Please upload a minimal reproduction project to make this easier to troubleshoot.

This seems to be a general issue with godot mono, where script files break when the node class is inside of a namespace. Also happens in a non-server godot project in editor (not exported)

Also happens on all platforms (have this on mac and windows as well

@git2013vb Please upload a minimal reproduction project to make this easier to troubleshoot.

Having a project will not help you more that what I described in issue, I think.
But if you insist ill give you two(2) instruction to have a MRP: (less effort than login and download/unpack any file zip from here)

create a new project (the exactly version is already defined in issue)
add a basic class in autoload like this one:

using Godot;
using System;

namespace DesertEdge.Singleton
{
   public class Signals : Node
   {
       public override void _Ready()
       {
       }
   }
}

While I was testing something else I incurred again at this  issue  using v4.0.rc2.official [d2699dc7a]
I think its the same
So I can add a MRP this time :)
TestDictStr.zip


[ISSUE-41] Upgrade JBoss Class File Writer to 1.3.0.Final for JBoss Invocation to work on JDK12 and above

https://github.com/jbossas/jboss-invocation/issues/41

@chengfang please review

Thanks for review @chengfang !


[WIP] Feature/update to hex

Description of the Change
Change bytes in proto to string with hexes.
Benefits
One format for keys and hashes everywhere
Possible Drawbacks
Yet to find by revieweres

For conflict resolution we are waiting for #1914 to be merged into dev, then a PR to be merged from https://github.com/hyperledger/iroha/tree/trunk/block-oneof

Closed in favour https://github.com/hyperledger/iroha/pull/1940


fix: decoding APK with many compact entries and unknown uses-sdk attrs

This fixes 2 new issues with a stock APK sourced from an Android 15 ROM.
https://drive.google.com/file/d/1x9udLN4W5I7chyGp1ZY8Cyfhu1vXezU9/view
(classes files removed to reduce size, they are irrelevant)

mIn.readShort() for size in readEntryData is incorrect and the size < 0 check is not possible.
Entry size is stored by AAPT2 as an unsigned short and thus will never be negative.
Reading it as a signed short will cause negative entry sizes in compactly packed entries in
very large string pools and will result in a lot of "APKTOOL_DUMMYVAL_" values.

sdkInfo isn't stored properly for APKs with unexpected properties in uses-sdk tag.
As far as I can tell, these attributes serve no purpose and can be ignored.
In the given APK, additional "android:versionCode" and "android:versionName" attributes appear
in the uses-sdk tag, purpose unknown and they don't represent the actual version of the app.

   E: uses-sdk (line=26)
     A: http://schemas.android.com/apk/res/android:minSdkVersion(0x0101020c)=35
     A: http://schemas.android.com/apk/res/android:versionCode(0x0101021b)=31
     A: http://schemas.android.com/apk/res/android:versionName(0x0101021c)="3.1"
     A: http://schemas.android.com/apk/res/android:targetSdkVersion(0x01010270)=35

So we could build a sample of compactly packed resources that exceeds a signed short max (~32k) thus to trigger the flaw of reading a signed short vs unsigned

So we could build a sample of compactly packed resources that exceeds a signed short max (~32k) thus to trigger the flaw of reading a signed short vs unsigned

Right, though the main reason we don't hit this issue often is because how many entries are required (+ them being compact) in order to hit the signed short max value of 32767, where the next size (= index) will become -32768.

Wrote the sample app. Could you write the test? Not really great at unit testing.
https://drive.google.com/file/d/1WF979N5qTWzGhQNtTMS8BET1dvPwecQ0/view

Easy source for sample? I like to keep sources for samples https://github.com/iBotPeaches/TestApks

Easy source for sample? I like to keep sources for samples https://github.com/iBotPeaches/TestApks

Well... I didn't make a whole project just to compile a simple resources dir with the --enable-compact-entries flag passed to AAPT2. ^_^''

Well... I didn't make a whole project just to compile a simple resources dir with the --enable-compact-entries flag passed to AAPT2. ^_^''

Fair, I always appreciate the reminder when it comes months later how I generated x. Sometimes I use a simple bash script to perform the actions if Gradle doesn't support it yet - https://github.com/iBotPeaches/TestApks/blob/master/ARSCTest/build.sh


Provide a way to send a JSON object with WebClient

Motivation:
It would also be nice to have APIs that automatically convert a request body into bytestring and converts a response into an object.
Modifications:

Add method ofJson in HttpRequest
Add method aggregateAs in HttpResponse

Result:

Closes #3707
Users can convert a request body into bytestring automatically :

HttpRequest.ofJson(HttpMethod.POST, "/receiveContent", requestContent)

Users can convert a response into an object automatically :

Content content = HttpResponse.of(MediaType.JSON_UTF_8, contentString)
                              .aggregateAs(Content.class)
                              .join();

ErrorMessage error = HttpResponse.of(MediaType.JSON_UTF_8, contentString)
                                 .aggregateAs(HttpStatusClass.SERVER_ERROR, ErrorMessage.class)
                                 .join();

ErrorMessage error = HttpResponse.of(MediaType.JSON_UTF_8, contentString)
                                 .aggregateAs(status -> status != HttpStatus.OK, ErrorMessage.class)
                                 .join();

Wrapper<Content> wrapperContent = HttpResponse.of(MediaType.JSON_UTF_8, contentString)
                                              .aggregateAs(new TypeReference<Wrapper<Content>>(){})
                                              .join();

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

@karellen-kim I think #4021 takes over the features of this PR.
Let me close this PR. 🙇♂ I hope to see the next PR in the near future. 😆

But still we can't send a JSON object as it is, right? #4021 only takes care of the receiving part.

Ah, this PR includes HttpRequest.ofJson() also. I missed that. Let me reopen and continue to develop the sending part.

Gentle ping. @karellen-kim

It seems like this PR has been stale for a while, let me reassign @monorisk to finish this PR


Can't return different class than the annotated one.

I would like to have a method that receives one type of object and returns another type.
for example:
class SomeClass(EndpointsModel):
    name = ndb.StringProperty()

class OtherClass(EndpointsModel):
    otherName = ndb.StringProperty()

@SomeClass.method(path='mymodel', http_method='POST', name='mymodel.insert')
    def MyModelInsert(self, my_model):
        my_model.put()
        otherModel = OtherClass(otherName='someothername')
        return otherModel

I'm getting:
ServerError (Method MyApi.MyModelInsert expected response type <class '.SomeClass'>, sent <class '.OtherClass'>)
Is this a problem with the library , or is this feature just not available ?

Try this
    @SomeClass.method(response_message=OtherClass.ProtoModel(),
                     path='mymodel', http_method='POST', name='mymodel.insert')
    def MyModelInsert(self, my_model):
        my_model.put()
        otherModel = OtherClass(otherName='someothername')
        return otherModel

Also note, your request is not an "issue" with the library and hence does not belong on this issue tracker. In the future please post on Stack Overflow. (Though one might argue a lack of sufficient documentation is an "issue".)

thanks ! I just saw your video on youtube about this library and it was great.
I am new to app engine and I know this isn't related at all , but I'm trying to figure out what is the best approach for structuring the data and how it fits with endpoints. how do you build relations between entities , i completley understand the technical side but I have not found any recommendations as to how to approach modeling some common app structures. what works well and what doesn't.
i was wondering if you could nudge me on the right direction as i have not found much help just searching the web.

Ask a question on StackOverflow and I'll try to answer there.


Ability to manually trigger an update

Is there a way to manually trigger an update instead of having to wait for the next update interval?

PR merged.


DataValidationView (the Build a Submission view) Refactor 1 - Doc string and type hint updates

Summary Change Description
Housekeeping updates only.  This will be the first of a break-up of a large PR (#1015).
Affected Issues/Pull Requests

Partially addresses #829
Merged into: #1013
Next PR: #1020

Review Notes

See comments in-line.
Checklist

This pull request will be merged once the following requirements are met.  The
author and/or reviewers should uncheck any unmet requirements:

Review requirements

Minimum approvals: 1
No changes requested
All blocking issues resolved by reviewers
Specific reviewers: @add_username_here

Review period: 2 days

Associated issue/pull request requirements:

[x] All requirements in affected issues marked "resolved" are satisfied
[x] All required pull requests are merged (or none)

Basic requirements

[x] All linters pass
[x] All tests pass
[x] All conflicts resolved

Overhead requirements

[x] New/changed method tests implemented/updated (or no change)
[ ] Updated Unreleased section of changelog.md (or no change)
[x] Migrations created & committed (or no change)

Merged via #1076


SAC (#34)

Starting work on SAC (#34).

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 1.90476% with 206 lines in your changes missing coverage. Please review.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files
Coverage Δ

src/common/distributions/distribution.rs
100.00% <ø> (ø)

src/common/distributions/normal.rs
94.94% <100.00%> (ø)

src/common/utils/modules.rs
98.18% <100.00%> (ø)

src/simple_sac/agent.rs
0.00% <0.00%> (ø)

... and 2 files with indirect coverage changes


Create catalan translation

Hi @trombonecot and thank you for your contribution!
Did you know you can use Weblate to manage translation? It's a web interface made to facilitate translations through a nice UI instead of editing JSON files: https://weblate.foundryvtt-hub.com/projects/channel-fear/
I will make a new version tonight with your change, or later if you manage to also translate other contents?

I've added all missing translations. If you could merge it today perfect! I won't be using it until next week, but if you merge it I'll test it. Thanks!
I didn't know about the UI of translation, maybe for corrections I'll use it.

New version 0.1.4 with Català done. 👍🏻 😄


Linux Ubuntu 18.04 - Invalid file descriptor to ICU data received

Steps to reproduce
Tell us about your environment:

Puppeteer version: 1.3.0
Platform / OS version: Ubuntu 18.04
URLs (if applicable):
Node.js version: v8.10.0

What steps will reproduce the problem?
Fresh install, and running puppeteer to produce PDFs
What is the expected result?
Puppeteer runs and exports a PDF
What happens instead?
It errors with the following exception.
[0509/163819.806050:ERROR:icu_util.cc(133)] Invalid file descriptor to ICU data received.
[0509/163819.806212:FATAL:content_main_delegate.cc(53)] Check failed: false.
#0 0x56341d486b0c base::debug::StackTrace::StackTrace()
#1 0x56341d49f780 logging::LogMessage::~LogMessage()
#2 0x56341b828d23 content::ContentMainDelegate::TerminateForFatalInitializationError()
#3 0x56341d1c1deb content::ContentMainRunnerImpl::Initialize()
#4 0x56341d1cbb72 service_manager::Main()
#5 0x56341d1c0f14 content::ContentMain()
#6 0x5634212d09b9 headless::(anonymous namespace)::RunContentMain()
#7 0x5634212d0a42 headless::HeadlessBrowserMain()
#8 0x56341d1c8f9d headless::HeadlessShellMain()
#9 0x56341b8271ac ChromeMain
#10 0x7fc2dd77ab97 __libc_start_main
#11 0x56341b82702a _start

I have also resolved the issue - see the next comment.

The issue is permissions of the directories/files.
When puppeteer was installed, the permissions were default owner of nobody:<usergroup> but not all files had group or others bits set (i.e. 600 or 700)
Puppeteer was installed as npm install -g puppeteer and it is run as the local user.
The fix was to chmod the files to set the group and others bits to match the "users".
I left the ownership as what it was - nobody:<usergroup>
755 for directories
755 for executable files
644 for all other files
I am unsure if the root:root ownership is a bad situation.
cd /usr/local/lib/node_modules/puppeteer/.local-chromium
find . -type d | xargs -L1 -Ixx sudo chmod 755 xx
find . -type f -perms /u+x | xargs -L1 -Ixx sudo chmod 755 xx
find . -type f -not -perms /u+x | xargs -L1 -Ixx sudo chmod 644 xx

Just a heads up I had to use -perm on the commands above but this worked! thank you!!!

fixed @someguy9 thanks. (typed from memory)


[Resolves] Support ingesting a digest model report/cache

Closes #2

TODO: add a check for duplicate YAML reports in the multi model analysis tool.


Define default provisioning params on plans

This allows an operator to define a plan and provide a default set of provisioning parameters. I am doing this incrementally so that it's not a giant PR. There will be follow-up PRs to add: svcat create plan --from, defaults on classes, etc. The set of milestones is laid out in the proposal.
TODO:

[x] Add feature gate
[x] Documentation

Doc preview available at https://deploy-preview-2282--svc-cat.netlify.com/docs/service-plan-defaults/

I gave this a spin on my AKS cluster and it looks good!
/lgtm


Expose additional configuration options for S3 storage

Is your feature request related to a problem? Please describe.
The used implementation for S3 has problems with some S3 backends, like this: https://github.com/goharbor/harbor/issues/12317
The proposed mitigation from that ticket does not work with the harbor operator, because the CRD does not allow setting multipartcopythresholdsize. Patching the ConfigMap for the registry pod does resolve the issue, but that's no sustainable solution.
Describe the solution you'd like
I would like to set multipartcopythresholdsize and other S3 settings via the CR.

Change reference: https://github.com/goharbor/harbor-helm/commit/35c746ca292e33433ded0c0cd1aa51c0672638a8


Use File Icon for suggestions doesn't work in .jsx file imports

Create a .jsx file, and write import {asd} from '../'.
No icons are shown for files.
I reported this in #44860 but it was closed without a response.

@jrieken This is just based on the label, correct?
TS intentionally does not insert file extensions in paths (since you would not want to be importing a .ts path in the compiled javascript)

It's label and/or detail, whatever matches best: https://github.com/Microsoft/vscode/blob/9742f05593cd36a28284eaf8ed8c95edc34a7d56/src/vs/editor/contrib/suggest/suggestWidget.ts#L169-L170
I agree that the label should be without suffix but I think the detail is a good place for it


unable to switch to input window when using grabcut.py with very large image

OpenCV version: Trunk
Host OS: Linux (Ubuntu 14.04)

Steps to reproduce the issue

Find a random image that has resolution larger than the current display

Type in the below command in terminal

 python /home/jason/opencv/samples/python/grabcut.py ultrahigh.png

I am unable to bring the input window to front hence cannot experiment grabcut on a larger image. I am expecting being able to bring the input window to front.
Low res images works normally since both the output and input window could be both visible at the same time.

What is UI backend used?

@jasjuang Try to comment line
cv2.moveWindow('input',img.shape[1]+10,90)
in grabcut.py

This sample works as expected. There are no reasons to write complex logic to process cases which are not related to simple demonstration (like resizing showed windows), because it is a "sample".

@sovrasov Thanks your solution works, however it appears that grabcut is not capable to process images at such high resolution anyways. When I try to draw the bounding box it freezes for a few minutes and then crashes.


Investigate imlications of third-party tools with regards to tracking/logging

There is an ongoing discussion regarding the legality of Google Analytics with regards of data transfer to the US.
We should look at all third-party tools we use and check where the servers are located, data is stored, potential cookies/logging.
Further details to be added

This might be covered in individual issues for Mux and Algolia for example, but just adding this issue here so that we have it in the backlog


NIP-44 encrypt/decrypt

Can we add these calls to the implementation? Amber already replies them via NIP-46.

Checkout latest v1.3.3, should work. Btw nsec.app supports nip44 too.

Done


Would it be possible to provide a downloadable link for the rendered images?

Hi, thanks for demonstrating this fantastic work!
I have been following the provided rendering code based on your instructions.
However, as mentioned, the rendering speed is extremely slow, even with 8GPUs.
https://github.com/cvlab-columbia/zero123/issues/4
Do you have an estimation of how long the whole process would take to render the objarvse dataset?
Will it be possible to have a downloadable link, such as Google drive, dropbox, etc for your preprocessed dataset (or even a reasonable size subset)?
Thanks!

Hi @MultiPath , thank you for your interest in our work. We understand the difficulty of rendering the dataset. We are working with huggingface to host our training dataset there for people to download. Stay tuned for the updates the in the next few days!

Hi, that would be awesome!! Looking forward to running on the same settings soon!

Thanks a lot in advance @ruoshiliu for sharing that dataset! Like @MultiPath , I'm also interested in this question:

Do you have an estimation of how long the whole process would take to render the objarvse dataset?

Thanks @ruoshiliu ! Was this using EEVEE or Cycles?

Was this using EEVEE or Cycles?
Nevermind, I see in the paper you used Cycles. Thanks for the great paper and dataset!

Just wondering any progress on this. Really appreciate it in advance. @ruoshiliu
I have tested rendering by myself, but I found the speed is much slower than what you showed above.
Will the number of workers affect the rendering speed (I tried 8)? If the data is available soon then I would be super grateful.
Thanks again

Just to clarify, the problem on my side using CYCLE seems it always showing to be "loading rendering kernel" for each camera view and it is extremely slow.

Still working on the dataset. It could take a few more days. I'll update under this thread when it's ready for download.
The total number of workers was 16 (workers_per_gpu=2) for me and anything beyond that didn't make any difference. Too many workers could put unnecessary load to CPU.

Dataset can be downloaded here:
https://github.com/cvlab-columbia/zero123#objaverse-rendering-preliminary
P.S. This is a temporary solution so the downloading speed can be very slow and can take several days to finish.

Hi all, I'm sorry to let you know that we might need to close downloading of the dataset soon due to higher-than-expected traffic, which is causing some difficulty to our cluster. We are seeking a different solution and will provide updates shortly. It will remain open for the next 24 hours but no guarantee after that. Sorry for the inconvenience.

My downloading will finish soon in 4 hours. Is it possible to turn off the downloading after tonight? @ruoshiliu

I can also think about what we can help after downloading this if needed...

Hi all, I'm sorry to let you know that we might need to close downloading of the dataset soon due to higher-than-expected traffic, which is causing some difficulty to our cluster. We are seeking a different solution and will provide updates shortly. It will remain open for the next 24 hours but no guarantee after that. Sorry for the inconvenience.

Hi @ruoshiliu,
May I ask if it can be postponed for a period of time, I still need about 40 hours to download all the data, and I have downloaded 500G so far, it would be a pity for me to just give up. Please remain open for 48 hours, thank you very much!

Quick update: we are setting up a mini-NAS dedicated to hosting the dataset for download. The link will remain active until then. Likely there will also be an option to download from AWS S3.

@mitming , we'll try to keep the link active until we find an alternative way of hosting.

@mitming , we'll try to keep the link active until we find an alternative way of hosting.

Thanks for your effort!

Hi all, the dataset hosted on AWS S3 can be downloaded from here:
wget https://tri-ml-public.s3.amazonaws.com/datasets/views_release.tar.gz

Please let me know if it works.

Looks like the link has been working as expected -- closing this issue.


Name for analysis page is verb not noun

Where to find the issue
https://www.coronawarn.app/
Describe the issue
All other headings on the website https://www.coronawarn.app/ are English language nouns.
The heading "Analyse" could be an English language verb or it could be a German language noun.

Suggested change
Change the heading from "Analyse" to "Analysis" to make it an English language noun, like the other headings.
(If it was supposed to be an English verb, then "Analyze" would conform to the rest of the website which uses US English, not UK English spelling.)
References
https://dictionary.cambridge.org/dictionary/english/analyse
https://dictionary.cambridge.org/dictionary/english/analysis

Internal Tracking ID: EXPOSUREAPP-9698

@FabioFuschi-SNK FYI

PR https://github.com/corona-warn-app/cwa-website/pull/1777 removes "Analyse" from the menu on https://www.coronawarn.app/en/, however if anybody already bookmarked
https://www.coronawarn.app/en/analyse/ or
https://www.coronawarn.app/de/analyse/
the menu bar still shows "Analyse", as before.
Also, under
https://www.coronawarn.app/en/analysis and
https://www.coronawarn.app/de/analysis/
the heading of the page is still "Analyse".  I'm not sure that that heading is needed, directly above the text "Corona-Warn-App (CWA): Kennzahlen". If it is needed there, then consider whether it should be changed to "Analysis" there as well.

FYI: https://github.com/corona-warn-app/cwa-website/pull/1774 has removed the analysis page from the website again.

Assuming that the analysis page will be published again, leaving this issue open.

@dsarkar / @FabioFuschi-SNK / @mtb77
PR https://github.com/corona-warn-app/cwa-website/pull/1793 has added the navigation name "Analyse" back.
Originally it was "Analyse", then it was changed to "Analysis", now it is "Analyse" again,
Is that what you want it to be?
This issue pointed out the inconsistency of using a verb in that place when everything else in the menu is a noun.

Commented it in #1793

@mtb77

Commented it in #1793

Thanks! I will leave this issue open until the naming is resolved.

@MikeMcC399 @mtb77 @FabioFuschi-SNK FYI PR https://github.com/corona-warn-app/cwa-website/pull/1800

The name has been changed to "Analysis" in PR #1800 which resolves this issue, so now closing.
Thanks very much to @dsarkar ! 👍🏻

I disagree with the decision to name the page "Analysis" in German.
Analysis in German has a different meaning, see https://www.duden.de/rechtschreibung/Analysis.
It makes no sense to use a word which is out of context just for consistency, IHMO.
I opened #2331 to change "Analysis" to "Analyse".

@Ein-Tim

I disagree with the decision to name the page "Analysis" in German.

I understand your point about the discrepancy between the menu item "Analysis" and the page topic text "Analyse" on https://www.coronawarn.app/de/analysis/. I already noted in https://github.com/corona-warn-app/cwa-website/issues/1776#issuecomment-930283228 that I didn't think the page header was needed and I think in fact it would be much better if it were removed, which would need some reformatting of the page.
Since this is really a new (although related) topic could you please open a new issue for this point?

@MikeMcC399
I will see how my PR #2331 is handled and then open an issue if necessary.

@Ein-Tim

I had a new idea, so I opened issue https://github.com/corona-warn-app/cwa-website/issues/2352.

@MikeMcC399
And I like the idea very much, this is the way I'd go!


Update the front end to use Material UI Dahsboard template

See https://github.com/mui-org/material-ui/tree/master/docs/src/pages/getting-started/templates/dashboard
extracted the urls to a variable

Looks sleek! I would think it would be easier to manipulate the views if they were locked to the screen bounds rather than scrolling the whole page. (Views can still contain scrollbars)
Just a thought though!

Yeah @aldebaran30701 I do actually want to remove the scroll bars, one of the reasons I wanted to put this in a PR instead of overriding whats there. I am not sure how to do this at this stage - if you know that would be good to have fixed.

OK going to merge this in so it does not get out of sync with others working on the develop branch but noting the scroll bar issue needs to be fixed.


feat: create EppoPrecomputedClient

labels: mergeable
Fixes: #issue
Motivation and Context
Description
How has this been tested?

@sameerank this PR could really use some fleshing out of the motivation and description. even if after merged, that'd still be helpful and appreciated


Bump django-allauth

Bumping from 0.50.0 to 0.52.0, allows to have some new interesting security features like rate limit on reset password and ACCOUNT_PREVENT_ENUMERATION for signup.
(And thank you @iMerica for your work on this tool :pray: )

LGTM

Released in 2.2.7
https://pypi.org/project/dj-rest-auth/


Sample indexes

Is there any way to access specific sample indexes, e.g., given a minibatch, is it possible to know the unique index of each sample in it?

@vturrisi Yes! Inside a transform, you can set the with_indices flag to get access to image indices (https://docs.ffcv.io/ffcv_examples/transform_with_inds.html). You can also pass an indices= flag to the Loader in order to specify which indices you want to read (e.g., if you want to read indices in a specific order, you can pass a permutation of np.arange(dataset_len) to indices and then use OrderOption.Sequential), lmk if this helps!

Hmm, I think that's not exactly what I need. What I want to do is access the indexes outside of the pipeline, something like
train_loader = Loader(....)
for idx, imgs, label in train_loader:
    ....

I don't need to influence the order the images are read, just need to access this info from outside. Is it doable?

I don't think it should be too hard to add support for this, let me think about whether its possible to do natively. In the meantime, the easiest possible workaround is just writing the beton file with indices (this will add no overhead at all when you don't want indices as you can just set pipelines={'idx': None, ...} when constructing the loader and it won't load the indices at all).
If re-writing the baton file is too expensive (I don't think it should be, since the writing is usually pretty fast even for imagenet), I think there's a way to do this with a custom transform (but will have to think a bit more).

We could have a with_indices on the Loader that would be very easy to add

I'm going to start a new a v0.0.3 branch and start adding all these little features and people can start testing before we release it.

I was adding this when I realized that just having a with_indices it might be problematic as there will be no way to copy them to the GPU or do anything with it. Is it something we think will be useful or should we stay with with_indices @andrewilyas  @vturrisi ?

@GuillaumeLeclerc sorry, I didn't get it. If with_indices is problematic, what's the other option?

@vturrisi I think what Guillaume is saying is that with_indices would just return the indices as a scalar array (compare this to, e.g., an index pipeline that could do augmentations, do asynchronous copying to the GPU, etc). Maybe we could have an index_pipeline argument instead of with_indices that could fulfill this purpose without having to rewrite the baton file?

Is it something we want in the next release ? Anyone down to write the unit tests/documentation for this. I have a bit too much on my plate right now :/

Personally speaking, I don't think it's something urgent. I'm waiting for implementations of common transformations (like color jitter), or for me to have time to implement them myself, to switch to fccv.


Needs more Stars!

This is beautiful, succinct code. Thank you!

Thank you for your kind words @coolaj86 :)


PLT-7567: Integration of Team Icons

Summary
Integration of team icon upload / display mechanics. Errorhandling, refactoring of setting_picture to be context specific, style modifications, tests.
Ticket Link
https://github.com/mattermost/mattermost-server/issues/7616
Reference PR
https://github.com/mattermost/mattermost-server/pull/8284
https://github.com/mattermost/mattermost-redux/pull/403
https://github.com/mattermost/mattermost-api-reference/pull/334
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit and component tests passed
[x] Added or updated unit tests (required for all new features)
[x] Has server changes (please link)
[x] Has redux changes (please link)
[x] Has UI changes
[x] Includes text changes and localization file (.../i18n/en.json) updates
[ ] Touches critical sections of the codebase (auth, posting, etc.)

@chumbalum looking forward to testing this! If you're interested, can I ask your help to also submit a PR to the mobile repo so we have React Native support for this feature?

@esethna I set up my developer machine for mobile development the last days. Still having some issues since some of my modifications in the code doesn't seem to be reflected in the app, although hot reloading seems to be working. I have to take another look at this.
In the app, there currently is no screen for editing a team. Nevertheless, I could try to integrate the team icons in the "select_team" screen.

@chumbalum That would be very helpful!
For mobile app, there isn't a screen to edit a team as you mentioned, and you don't need to worry about adding a setting for it on mobile.
Having the team icon appear on the team sidebar similar to the webapp would be great for consistency though.

@chumbalum if you update mattermost-redux with yarn upgrade mattermost-redux this should be ready to test

@jwilander done. looks good. I did not do this step in my PR for mattermost-mobile (https://github.com/mattermost/mattermost-mobile/pull/1470). So if this should have happened, it must be done seperately because the PR is already merged

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-0dbfa2e731b31943c.spinmint.com
Test Admin Account: Email<EMAIL_ADDRESS>| Password: user-0
Test User Account: Email<EMAIL_ADDRESS>| Password: user-1
Instance ID: i-0dbfa2e731b31943c

@jwilander  Just tested on the test server. There is some problem. Parts of this PR are in conflict with commit https://github.com/mattermost/mattermost-webapp/commit/cb8223f79311480de91477aaf96878c15718ee76
There, the mm_config has been removed, which I was using in my work. So I need to fix that.

Sounds good, let us know if you need any help with that
On Mar 8, 2018 05:13, "Christian Hoff"<EMAIL_ADDRESS>wrote:

@jwilander https://github.com/jwilander Just tested on the test server.
There is some problem. Parts of this PR are in conflict with commit
cb8223f
https://github.com/mattermost/mattermost-webapp/commit/cb8223f79311480de91477aaf96878c15718ee76
There, the mm_config has been removed, which I was using in my work. So I
need to fix that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mattermost/mattermost-webapp/pull/796#issuecomment-371442918,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACjF4u94MKT0hvH3iGtZ5E-ORsCUyOvVks5tcQQrgaJpZM4SEeKx
.

I made a rebase and the necessary changes. I made a redux connected component out of team_general_tab (file was moved into a subfolder) and passed the maxFilesSize via the store into it.

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-09f13ec7326a45056.spinmint.com
Test Admin Account: Email<EMAIL_ADDRESS>| Password: user-0
Test User Account: Email<EMAIL_ADDRESS>| Password: user-1
Instance ID: i-09f13ec7326a45056

@amyblais should be ready for you to test

Hi @chumbalum,
Thank you for this contribution, good work here!
I have started testing and I just had a few notes:

The "Save" button doesn't become disabled after closing the Team Icon modal after not saving a chosen image:

for Team Icon, press Edit
press Select
Choose an image and press "Open"
do not press "Save" but press "Cancel" to close Team Icon modal
press Edit again for the Team Icon
"Save" button is not disabled but I think it should be (it becomes disabled when repeating these steps for uploading a profile picture).

Question: Profile pictures also take BMP and JPEG, is this expected to be different for the Team Icon?

Hi @amyblais

I can take a look at this, but cannot guarantee to finish it until monday since I'm somewhat busy over the weekend

I orientated myself on the profile_picture. In the profile_picture component, it looks like only png and jpegs are allowed (I did not test it, though). Maybe the translation is outdated. (https://github.com/mattermost/mattermost-webapp/blob/master/components/user_settings/general/user_settings_general.jsx#L253)

Hi @chumbalum,

No problem, no rush!
It's possible that it's outdated, I'll ask one of our engineers to take a look.

@chumbalum,
Quick FYI, I got a confirmation that the help text in the profile picture setting is correct and that all BMP, JPG, JPEG and PNG files should work.

Hello @amyblais I just tested it on the pre-release server. You can select bmp files but actually submitting a bmp is not possible. In the code there is a limitation of the mimetypes image/jpeg and image/png

tested with https://us.v-cdn.net/6024342/uploads/editor/wq/uj8v09udsgd3.bmp

@chumbalum,
Sorry about that, looks like you are correct there :)

@jwilander
I did address your suggestions concerning redux actions & await
@amyblais
The issue with save button got fixed, too

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-0ec4f185be5575a8c.spinmint.com
Test Admin Account: Email<EMAIL_ADDRESS>| Password: user-0
Test User Account: Email<EMAIL_ADDRESS>| Password: user-1
Instance ID: i-0ec4f185be5575a8c

Spinmint test server created at: http://i-0840aa5b4abf35e0f.spinmint.com
Test Admin Account: Email<EMAIL_ADDRESS>| Password: user-0
Test User Account: Email<EMAIL_ADDRESS>| Password: user-1
Instance ID: i-0840aa5b4abf35e0f

@chumbalum Looks like I'm getting a JavaScript error now when trying to save the uploaded team icon

Will defer to @amyblais for other testing, was just curious and wanted to try out the feature :)

Hi @jasonblais . I cannot explain that error. On my local machine it is working normally. It seems like the spinmint server is not using the current mattermost-redux version, since it can't find that function. I commited another yarn upgrade mattermost-redux. I made a make clean, make run and tested it positive again on my local setup. Can you initiate another spinmint server?

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-0ba711429803243fe.spinmint.com
Test Admin Account: Email<EMAIL_ADDRESS>| Password: user-0
Test User Account: Email<EMAIL_ADDRESS>| Password: user-1
Instance ID: i-0ba711429803243fe

@chumbalum We're having some trouble with the test servers so we'll have to wait until tomorrow to have a new one generated :)

@chumbalum,
I tested again and noticed 2 new issues that I hadn't seen last time I tested:

Cursor is on the left hand side of text in any Team settings and Account settings that lets you edit text, e.g.

Go to Team settings.
Click "Edit" on Team Name.
Cursor is on the left side of the text but should be on the right side.

Repro steps for another issue in Team settings:

In Team Settings, click "Edit" on any of the setting options to open the edit modal.
Don't edit anything and don't close the edit modal, just click on the channel behind the Team settings window so that the Team settings window closes.
Switch to another team.
Switch back to the same team that you switched out from.
Go to Team settings.
The edit modal is still open but it should be closed.

Hi @amyblais
I can have a look at those points but maybe it makes more sense to open new tickets for those optimization tasks, since they don't really belong to the team icon topic and prevent this PR to be closed/merged which causes more work for rebasing / keeping this PR up to date

FWIW I agree with @chumbalum anything not directly related to the addition of the new feature should be handled by a new ticket and PR

Sounds good all, I created tickets here and here for the issues I saw.
This PR looks good, will approve - @jasonblais this is ready for you to confirm before merging.

One minor modification to consider is to make the team icons in the sidebar square. Currently, the team icon in the webapp is 43*39px. This was done because it was defined in the Issue ( https://github.com/mattermost/mattermost-server/issues/7616 ). The upload dialog itself is square and so is the implementation in mattermost-mobile. In addition it is more intuitive to work with square images because the user doesn't have to think about scaling behaviour

Spinmint test server destroyed

@jwilander Ready for last dev reviwe


Other log types

Hello,
Epic library.
Is it possible to support other log metrics, such as video and 3d plots?

Thank you for your interest in wandb-cpp.
Firstly, supporting 3D plots is challenging . This is because even the main wandb platform doesn't implement comprehensive features for this. However, I have already implemented a feature to output logs compatible with wandb's Object3D. This functionality allows you to plot 3D point clouds and boxes. For more details, please see:
https://github.com/yhisaki/wandb-cpp?tab=readme-ov-file#wandbobject3d
Next, supporting video is possible, but it is not currently implemented. I hope to add this feature soon using https://docs.wandb.ai/ref/python/data-types/video.
As an alternative to these features, you can use https://github.com/yhisaki/wandb-cpp?tab=readme-ov-file#save-file. This allows you to save any files.

We have added support for video upload functionality! For more details, please see here!

so good


How to show number being edited in edit dialog

Although i have seen that Order ID is displayed during editing in sample code.
I am unable to achieve the same in my custom table.
My custom table do not have identity column but an integer column as primary key.
can someone please help me.

Add it to form

ok.
thanks.


Removing a contact from associated list doesn't actually remove the contact

Issue Summary
Remove Contacts from a List returns a 202 response code with job id, but the membership doesn't change.
Steps to Reproduce

Create a list using /v3/marketing/contacts endpoint.
Subscribe a contact to that list by providing the list_ids
Try unsubscribing the contact from the list using the /v3/marketing/lists/{list_id}/contacts?contact_ids={contact_id} endpoint
I receive the 202 successful response with associated job_id
When I go back to the contact's profile the list is still present under Associated Lists.

Is there something I am missing, or is there another endpoint that unsubscribes/removes contact(s) from the list ?
Technical details:

@sendgrid/client version: "6.3.0"
node version: "10.17.0"

Hello @arpitashekhar,
Thanks for submitting a GitHub issue! We are very sorry that you are running into this problem. In order to better serve you, as this does not present itself as a library specific issue, we would like to ask that you reach out to our support team at https://support.sendgrid.com.
Thank you!
SendGrid DX Team


Index Sorting: Introduce Unicode Collation Instead of Asciibetical Sorting

I'm not entirely sure if this issue related to Asciidoctor PDF or to Asciidoctor itself (but since I've encountered the problem using Asciidoctor PDF, I'm reporting it here).
In the Index of a PDF document converted via Asciidoctor PDF, the entries for each letter are sorted Asciibetically, instead of alhpabetically.
Entries in uppercase are listed before entries in lowercase, which leads to differently cased same-entries to be separated by other entries which would normally come after them:
AAA
BBB
CCC
aaa

... where one would expect "aaa" to directly follow "AAA".
This makes looking up entries in the Index troublesome, as the reader is forced to jump up and down the entries list to check differently cased entries are available for a given term — needless to say, it could lead to missing out many entries that are actually indexed.
A quick solution could be that of converting entries to lowecase (internally) before sorting them (but preserve their letter case in the actual output).
But, ideally, a true Unicode sorting algorithm would be better, allowing to properly handle special letters with accents, diacritics, etc (so that, for example, the letters "è" and "È" would be treated as an "e" in sorting).
The latter solution might require Asciidoctor to rely on a Unicode collation algorithm to handle the task properly:

Wikipedia » Unicode collation algorithm
Unicode Collation Algorithm: Unicode Technical Standard #10

This might introduce some complications, as dirrefent locales have different criteria for sorting special letters, but Asciidoctor could rely on the document's language specification (:lang:) to decide how to sort entries.
There are some ready made collation charts that Asciidoctor could use to establish how entries should be sorted, based on the document's locale:

Mimer SQL Unicode Collation Charts
MySQL UCA-based Unicode Collation Charts

Since the Index is an important tool for readers, ebabling them to look up and find topics in very large books, proper sorting of the Index entries according to the document's locale is an important feature, affecting the quality of the final document.

I've started exploring this. It looks like we could delegate to functions provided by the ffi-icu gem to achieve this result.
In the meantime, I think we should spin off a separate issue to make sorting case insensitive.

I've started exploring this. It looks like we could delegate to functions provided by the ffi-icu gem to achieve this result.

Sounds good to rely on a dedicated Gem for this.

In the meantime, I think we should spin off a separate issue to make sorting case insensitive.

Done: #1406.

I was looking at the feature offered by ICU; it looks like it offers tons of cool features beside Unicode collation, including:

Code Page Conversion.
Formatting numbers, dates, times and currency amounts according the conventions of a chosen locale.
Time Calculations beyond the traditional Gregorian calendar.
Bidi support for handling text containing a mixture of left to right (English) and right to left (Arabic or Hebrew) data.

So, possibly the introduction of the ffi-icu gem could also allow more features into Asciidoctor, especially regarding formatting dates and numbers according to document locale, as well as handling non UTF-8 external assets (e.g. asciidoctor/asciidoctor#3248).
Maybe both Asciidoctor PDF and Asciidoctor could benefit from it.

I've been able to successfully integrate the ffi-icu gem. I think it's a viable path forward. There's also the twitter_cdlr gem, but I found it more difficult to use.
Here's an article I found that helped me to understand how to use ICU: https://bartvanraaij.dev/2020-10-17-converting-utf8-strings-to-ascii-using-icu-transliterator/


No input file specified.

I have web host.And getting error. No input file specified. I can' t name database to ci_bootstrap_3.I can only get database name like vhost64869s1 because web host provider won't let to change.Even with xampp getting this error.

change it in https://github.com/waifung0207/ci_bootstrap_3/blob/master/application/config/ci_bootstrap.php

change it in https://github.com/waifung0207/ci_bootstrap_3/blob/master/application/config/database.php


Web is unavailable

Hi! Your website https://www.kunkalabs.com/mixitup/ is unavailable, i need it for licence info.

Thank you - all fixed now.


Issues with filenames that contain dots

Say you have two policy files, one for in-direction, the other for out direction named like this:
vlan1234.foo.bar.baz-in.pol
vlan1234.foo.bar.baz-out.pol
The converter script will ignore everything behind the first dot and create a new file called:
vlan1234.yaml
The resulting file only contains the in-acl, the out-acl is missing.
Tested with node/18.16.1 and pol2yaml 1.2.0.

Thanks @mf84. I just updated pol2yaml on NPM with his fix. pol2yaml 1.2.1 resolves this issue.

Also feel free to join the Aerleon slack server if you haven't already: https://join.slack.com/t/aerleon/shared_invite/zt-1ngckm6oj-cK7yj63A~JgqjixEui2Vhw


Swiping the onboarding not working in iOS 13

I can confirm the same problem.

I made a pull request that fixes this, without forcing presentation in full screen.
https://github.com/Ramotion/paper-onboarding/pull/124#issue-445602674.


Set player error on request failures

Summary
Hi @chemoish, currently We are using this package in our project and it is being really awesome so far. While debugging a concurrency restriction we  have in the backend I was trying to add a custom error message in our player to handle that scenario and noticed that for some http responses that come from our backend  the error is not being set in the player:
this.player_.error({
  code: 0,
  message: errorMessage,
});

By looking at your code I think I found the problem so this PR is an attempt to fix it.
When we
const request = new XMLHttpRequest();

and then
    request.addEventListener('error', this.onCertificateError, false);

the only errors that error listen to are network errors and cross-origin errors, when I http response comes from the backend load handler will be called instead and the request is considered successful.
So, in order to catch all the errors that comes from the backend we have to check the satus code of the request and when is not 200 execute the error logic that will end up setting a custom error in the player instance.
Also I found a dependency error while trying to npm run build for babel-core so I added that package to devDependencies in a different commit. That change is totally unrelated to this PR so let me know if you want to remove that commit from this PR.
Feedback is more than welcome!
Thank you.

@kimurakenshi Thanks for your gracious help!


[FR] Add an alternate "active" animation

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Consider this config:
  - type: custom:fluid-level-background-card
    card:
      type: glance
      entities:
        - entity: person.ildar
    entity: input_number.test_level_1
    fill_entity: input_boolean.test_boolean
    level_color:
      - 244
      - 0
      - 204

Currently a "bubbles" animation is shown if fill_entity = ON.
A "waves" animation is shown always.
I suggest an alternate variant:
-- if fill_entity = ON -> show "bubbles" & "waves";
-- if fill_entity = OFF -> show a "still water".
A type of animation may be defined by some new option like waves_always:
-- true:  the current behaviour (default);
-- false: show "bubbles" & "waves" if fill_entity = ON; if fill_entity = OFF -> do not show "bubbles" & "waves".
A reason behind this is that in some circumstances (people's age/eyes, a screen type, etc) these "bubbles" may be less visible.
Then a "wave" animation could be rather informative that "something is going on".
Describe the solution you'd like
A clear and concise description of what you want to happen.
see above.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
See above.
Additional context
Add any other context or screenshots about the feature request here.

I set this as todo without any schedule so far. I will see after I finish with outstanding bugs and planned features

I suggest adding the possibility to control the bubble size

bubble size

Could be an interesting alternative.

Hello, question: I have under "fill_entity: sensor.iphone_battery_state" with the state charging or not charging.
But no bubbles are forming? If it needs to be on or off, I have to do that with a helper, right?

sensor.iphone_battery_state

which is not on/off
yes, need to have a template binary sensor

Hello, question: I have under "fill_entity: sensor.iphone_battery_state" with the state charging or not charging.
But no bubbles are forming? If it needs to be on or off, I have to do that with a helper, right?

It shoild be bollean value.
Could you provide the state of that entity? I may be able to handle that state.

Hello, question: I have under "fill_entity: sensor.iphone_battery_state" with the state charging or not charging.
But no bubbles are forming? If it needs to be on or off, I have to do that with a helper, right?

It shoild be bollean value.
Could you provide the state of that entity? I may be able to handle that state.


tests/kernel/common/kernel.common fails for native_posix on Ubuntu 16.04

The failure occurs while running sanitycheck when native_posix builds with the host compiler (gcc 5.4.0 20160609) on Ubuntu 16.04 (Ubuntu 5.4.0-6ubuntu1~16.04.9) .
The tests/kernel/common/kernel.common fails as follows:
ccache /usr/bin/gcc -DBUILD_VERSION=v1.12.0-290-g7a7e4f5 -DKERNEL -D_POSIX_C_SOURCE=200809 -D__ZEPHYR__=1 -I/ho
/home/apurboch/zephyr/tests/kernel/common/src/intmath.c:50:1: error: bit-field ‘__error_if_negative’ width not
 NEG_CHECK(int, 0x80000000);
 ^
/home/apurboch/zephyr/tests/kernel/common/src/intmath.c:52:1: error: bit-field ‘__error_if_negative’ width not
 ROLLOVER_CHECK(int,<PHONE_NUMBER>, -2147483648);
 ^
cc1: error: unrecognized command line option ‘-Wno-format-truncation’ [-Werror]
cc1: all warnings being treated as errors
CMakeFiles/app.dir/build.make:140: recipe for target 'CMakeFiles/app.dir/src/intmath.c.obj' failed
make[3]: *** [CMakeFiles/app.dir/src/intmath.c.obj] Error 1```

Indeed..
@andyross , you added those checks (e9174c6ddeddf58d8aebba5c89e50965731899e6), and seems you were trying to fix a similar issue in
718a4cd24597853c9151ff57978a0b3de05c5329 So, would you mind taking it?
The exact error:
In file included from /usr/include/features.h:367:0,
                 from /usr/include/stdint.h:25,
                 from /usr/lib/gcc/x86_64-linux-gnu/5/include/stdint.h:9,
                 from [..]/include/zephyr/types.h:10,
                 from [..]/include/kernel_includes.h:17,
                 from [..]/include/kernel.h:17,
                 from [..]/include/zephyr.h:18,
                 from [..]/tests/ztest/include/ztest.h:47,
                 from [..]/tests/kernel/common/src/intmath.c:8:
[..]/include/toolchain/gcc.h:26:28: error: bit-field ‘__error_if_negative’ width not an integer constant
 #define BUILD_ASSERT(EXPR) _Static_assert(EXPR, "")
                            ^
[..]/tests/kernel/common/src/intmath.c:24:25: note: in expansion of macro ‘BUILD_ASSERT’
 #define NEG_CHECK(T, N) BUILD_ASSERT((-((T)N)) == (~((T)N))+1)
                         ^
[..]/tests/kernel/common/src/intmath.c:50:1: note: in expansion of macro ‘NEG_CHECK’
 NEG_CHECK(int, 0x80000000);
 ^
[..]/include/toolchain/gcc.h:26:28: error: bit-field ‘__error_if_negative’ width not an integer constant
 #define BUILD_ASSERT(EXPR) _Static_assert(EXPR, "")
                            ^
[..]/tests/kernel/common/src/intmath.c:27:37: note: in expansion of macro ‘BUILD_ASSERT’
 #define ROLLOVER_CHECK(T, MAX, MIN) BUILD_ASSERT((T)((T)1 + (T)MAX) == (T)MIN)
                                     ^
[..]/tests/kernel/common/src/intmath.c:52:1: note: in expansion of macro ‘ROLLOVER_CHECK’
 ROLLOVER_CHECK(int,<PHONE_NUMBER>, -2147483648);
 ^

Seems to happen when setting this gcc version to c99.
Setting it to 2011 it works just fine.
diff --git a/CMakeLists.txt b/CMakeLists.txt
index 1c9eb62..7696310 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -120,7 +120,7 @@ zephyr_compile_options(
 )

 zephyr_compile_options(
-  $<$<COMPILE_LANGUAGE:C>:-std=c99>
+  $<$<COMPILE_LANGUAGE:C>:-std=c11>

   $<$<COMPILE_LANGUAGE:CXX>:-std=c++11>
   $<$<COMPILE_LANGUAGE:CXX>:-fcheck-new>

Also it seems that before your fix in 718a4cd24597853c9151ff57978a0b3de05c5329 it was also breaking, just a bit differently:
[..]/tests/kernel/common/src/intmath.c:50:30: error: enumerator value for ‘__build_assert37’ is not an integer constant
[..]/tests/kernel/common/src/intmath.c:52:30: error: enumerator value for ‘__build_assert41’ is not an integer constant
 ROLLOVER_CHECK(int,<PHONE_NUMBER>, -2147483648);
                              ^

Related to #8093

#8520 will most likely fix this as a sideeffect of changing to C 2011

Ugh, this is an ugly one.  At least a proximate cause is a host tooling glitch.  GCC 5.4 supports _Static_assert() just fine, but it's not using it on Ubuntu 16.04.
Instead, there is code in /usr/include/x86_64-linux-gnu/sys/cdefs.h that looks like this:
#if (!defined _Static_assert && !defined __cplusplus \
     && (defined __STDC_VERSION__ ? __STDC_VERSION__ : 0) < 201112 \
     && (!__GNUC_PREREQ (4, 6) || defined __STRICT_ANSI__))
# define _Static_assert(expr, diagnostic) \
    extern int (*__Static_assert_function (void)) \
      [!!sizeof (struct { int __error_if_negative: (expr) ? 2 : -1; })]
#endif

This is a glibc file, pulled in transitively from stdint.h.  And if you follow the logic, it will refuse to use the compiler builtin when we're in STRICT_ANSI and not at a C11+ standard level (which seems wrong to me: shouldn't that be only when -pedantic is passed?  Normally gcc does not disable extensions in -std=... arguments).
And it fails. I actually don't know why it fails.  It works on all but two of hte expressions, and those definitely result in compile-time integer constants.  Maybe there's a limit of expression depth or some kind of other interaction with the 32 bit rollover being tested?  It definitely smells like a compiler bug: gcc 8.1.1 on Fedora hits exactly the same path building native_posix (i.e. it uses the glibc macro and not the builtin) and builds correctly.
So... options:

We can pass -std=c11 in native_posix.  This would be my preference, though I don't know if we have policies about this and it sounds like you tried it and it resulted in other failures somewhere else? (This one intmath.c file builds with with it, at least).

We can add #define _Static_assert _Static_assert before the system includes in this one test file, which trips the first part of that test.  Awful hack, but will change fewer things and involves less risk.

2b. We can add -D_Static_assert=_Static_assert to the command line for native_posix globally.  Because we really don't want that native macro, we want the builtin!  Less of a hack.  Somewhat uglier.  Hard to document cleanly.

We can disable the two relevant expressions test on this one platform combination (Ubuntu + native_posix).  Though we'll probably rediscover the same issue somewhere else.

Back to @aescolar as the fix requires policy decisions internal to native_posix and I don't know what to pick :)

we keep tripping over issues with host GCC version and the native build.
maybe the best solution is to just put a specific version of the host compiler in the SDK?

@andyross : I have been stalling a bit this one, expecting the llvm one would make it in with the c11 switch (which should just work for the native port)
@andrewboie : That file seems to be from libc6, not from gcc, so I guess bundling the compiler would not have helped in this case. Anyhow, the only real compiler issue I recall was due to new warnings (#7610), and those may actually reveal something we want to fix, and if not we could always add a pragma. So, I guess we do not need to give up on supporting whatever gcc version the host has (yet).

@andyross I think it would be best to go for C 2011. _Static_assert being a 2011 feature, even if most compilers support it with older versions, it would seem we cannot rely on them doing it correctly.
The other options you proposed seem to be too specific hacks for this particular compiler+c library version, apart from too hackish.


Roadmap for this repository

This repository was initially a bed where I placed my Trino experiments on the Trino Community Broadcast as I looked into the intersection of core Trino query engine abstractions, overlapping tech in Trino's ecosystem, and new features in Trino.
That said, I am reaching a point where I'd like narrow the focus on maintaining common areas of the code that seem to be the most valuable tutorials, and have a community section where I will keep older examples available for reference and depend on community contributions to keep them up-to-date.
I see the value and do have some time to update this repository, but there's honestly so many examples that require some specific tuning and testing every new version that to me it's better to prioritize few over many, while leaving space to add one-off examples that may or may not be maintained to keep the guarantees clear here.
My initial thoughts are to maintain the following:

[ ] [❗HIGH] Trino + Iceberg + REST/Polaris + MinIO

Tutorial: Run through initial setup and REST internals.

[ ] [⚠MED] Trino + Iceberg/Delta + REST/Unity + MinIO

Tutorial: Using Unity to collaborate bettween Iceberg and Delta formats.

[ ] [❗HIGH] Trino + Iceberg/Hive + Hive Metastore + MinIO

Tutorial: Run through initial setup through Hive.
Tutorial: Cover Hive Metastore internals.
Tutorial: Run migration to Iceberg.

[ ] [❗HIGH] Trino + Postgres

Tutorial: Run through a tutorial on why add a query engine to a database?

[ ] [ℹLOW] Trino + Trino Gateway + Kubernetes

Tutorial: How to handle multiple Trinos for multiple areas of focus
Trino Clusters:

(Small) Trino + Postgres
(Med) Trino + Iceberg
(Big) Trino + Iceberg (Fault Tolerant)

[ ] [❗HIGH] Testing framework and some basic tests.
[ ] [[⚠MED] Automate releases triggered on new Trino versions (Run tests with new version if.

Run tests with the new version, if it passes, auto-update the scripts, otherwise keep current version and add Issue.

[ ] [❗HIGH] Migrate other tutorials to a community folder.
[ ] [ℹ LOW] Change this repository to an awesome list

I'm happy to (continue to) maintain trino-getting-started/hive/trino-b2!
I also just noticed trino-getting-started/iceberg/trino-iceberg-minio - I'll see if I can create a trino-iceberg-b2 variant. Would that live in the new community folder?

I'm happy to (continue to) maintain trino-getting-started/hive/trino-b2!
I also just noticed trino-getting-started/iceberg/trino-iceberg-minio - I'll see if I can create a trino-iceberg-b2 variant. Would that live in the new community folder?

Yeah, that will be community but I almost want a way to indicate the ones that are actually getting love.
Another option would be to migrate that variant if you'd like to fork and I'll be happy to link to your repo from the readme. That way I'm not a bottleneck and it gets the same visibility.
WDYT?

It could live on https://github.com/backblaze-b2-samples, for sure. I'm thinking I could create a trino-getting-started-b2 (or something like that) repository, containing hive/trino-b2 and, at some point in the future, iceberg/trino-iceberg-v2. It seems sensible to keep the same directory structure.
As far as I remember, those samples are self-contained, right - I wouldn't need to drag anything else across?

It could live on https://github.com/backblaze-b2-samples, for sure. I'm thinking I could create a trino-getting-started-b2 (or something like that) repository, containing hive/trino-b2 and, at some point in the future, iceberg/trino-iceberg-v2. It seems sensible to keep the same directory structure.
As far as I remember, those samples are self-contained, right - I wouldn't need to drag anything else across?

That's correct, all self-contained and Docker reliant. Hopefully the Docker reliant part will eventually go away: https://github.com/trinodb/trino/issues/23344

@bitsondatadev - here you go: https://github.com/backblaze-b2-samples/trino-getting-started-b2


[tech-debt] use ethers types for ContractTransaction and ContractReceipt

This built-in types from ethers should correctly type everything as BigNumber instead of the hand-written types here:
https://github.com/KlimaDAO/klimadao/blob/eb04b89aed573ccb9dc4894735023e2ddee294f9/lib/types/offset.ts#L41
When creating a txn object from a Contract method, ethers does not type it. This can be reconciled by typing the txn variable with ContractTransaction
import { ContractTransaction } from "ethers";

const txn: ContractTransaction = await contract.approve(
  addresses["mainnet"][params.spender],
  parsedValue.toString()
);
const receipt = txn.wait(1); // inferred as ContractReceipt ✅
receipt.gasUsed // inferred as BigNumber ✅

[ ] remove hand written types like RetirementReceipt and any others

This should be an easy one and a good reason to get familiar with how we do ethers txns @0xemc


doc: Correct release document

Brief Summary of The Change

Pull Request checklist
I confirm that the following checklist has been completed.

[ ] Add/Change test cases for the changes.
[ ] Add/Change the related documentation, should also change docs/source/config.rst when you change file default_config.yaml.
[ ] (Optional) Add your change to UPDATING.md when it is an incompatible change.

Codecov Report

Merging #67 (20c96cf) into main (778b130) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #67   +/-   ##
=======================================
  Coverage   90.95%   90.95%
=======================================
  Files          61       61
  Lines        2211     2211
=======================================
  Hits         2011     2011
  Misses        200      200

Flag
Coverage Δ

unittests
90.95% <ø> (ø)

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

Impacted Files
Coverage Δ

src/pydolphinscheduler/core/task.py
93.78% <0.00%> (ø)

src/pydolphinscheduler/core/yaml_workflow.py
92.47% <0.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Adding the links in CONTACT FOR TOURS section

Related Issue

Issue No.  84#

Changes you made :

Adding the links in CONTACT FOR TOURS section

Describe the changes :
i added default links of the social media sites.
Check list :

[x] I have followed code style of this project.
[x] I have read the CONTRIBUTING document.
[x] All the aspects mentioned in the issue are covered.
[x] The website is properly working after my changes.
[x] 🌟 ed the repo

Type of change :

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

@imdebamrita Thank you for this PR
But you have to create an separate branch for each PR.
You can't push commits into main branch.
Read the contributing guidelines. 📖
Plz create new PR. 👍


Cli.Parse not working on Mono 5

Cli.Execute works fine, but Cli.Parse throws the following error. Works on Mono 4.8 though...

Hi @JorritPosthuma I'll have a look into this, thanks for the report.
I haven't actively tested EntryPoint on Mono, but would like to get it stable if you're planning on using the two together. Any reports you have would be very welcome.

Notes:

The code is failing on EntryPoint's .GetHelp extension, which tries to get a HelpAttribute on each define option.
Mono should return null if the attribute has not been applied, but this doesn't seem to be happening.
Instead Mono seems to be 'finding' 2 or more attributes 'matching' the HelpAttribute type, and throwing

@JorritPosthuma could you try a couple of things?

Try removing Required from each of your properties, just to get the number of attributes under 2. Do you get the same error?
Try applying a [Help(detail) attribute to each property. Do you still get an error?

I'm a little at a loss as to why Mono is doing this. Could you also check that you're on the latest 5.0 version? (5.0.1)

Hi! Thanks for helping ☺.
I tried 1 and 2, but it makes no difference. Even without any arguments, it's throwing the AmbiguousMatchException. I test it with Mono <IP_ADDRESS>

@JorritPosthuma I think I've fixed this and have put out 1.2.2 for you.
Let me know how it goes!

Yes! It works! Thanks a lot!!!

Great! Please don't hesitate to report any niggles you come up with. I rarely work with Mono, so EntryPoint least battle-hardened, there.
Closed by 6f59c1769cb80c897757b8a3b7b3b76ee0514f44

Will do ☺


Fix hmmlist parameter in nextflow_schema.json

This commit changes the hmmlist parameter from type "array" to type "string". This fix also lists all of the options so that they can be selected from a drop-down menu in the Tower parameter GUI.

Awesome. Thanks!
@robsyme  Can you reply in a comment with I have read the CLA Document and I hereby sign the CLA if you can?  I will try to make that instruction clearer as well

I have read the CLA Document and I hereby sign the CLA

Also not all the GitHub Actions are working yet so don't worry about that

Thanks again

No worries at all. Hope the fix helps!

For @chasemc; Just so I remember later...
We discussed on nf-core slack. This does fix the issue in Tower but only allows selecting a single HMM database input. For now I'll keep the previous implementation (broken Tower) so that multiple DBs can be input. Will revisit alt solutions later. Rebased and put this commit on a new branch "not_pulled_in_fixes_tower_breaks_functionality" to be kept around until decision is made


clp-s: Implement table packing

Description
This PR implements table-packing; we combine small tables together into one compression stream until they reach a certain size threshold in order to avoid having many tiny compression streams. This helps avoid outliers in compression ratio, particularly when we enable features like array-structurization which can create many small table.
On the compression side the key differences are that (1) SchemaWriter now keeps track of the total in-memory size of the table it owns instead of determining it after writing to a compression stream; (2) before compression tables are sorted by that in-memory size, and smaller tables are packed together in sequence until their combined size reaches a certain threshold; and (3) table metadata has been changed to accommodate table packing.
On the decompression side the business of making sure tables are read in the correct order, as well as decompressing the packed streams is implemented in TableReader. Most of the rest of the change is contained in SchemaReader. The logic for reading table metadata is split between TableReader and SchemaReader where TableReader reads metadata about individual compression streams, and SchemaReader reading metadata about how schema tables map to those streams. Also note that schema tables now need to be read in the order they appear in the table metadata, which can be different than schema ID order.
Note: this PR makes the decision to leave uncompressed size of individual schema tables out of the table metadata. This is because uncompressed size can be derived from other metadata we do store, and storing uncompressed size in addition to metadata offsets would actually increase the amount of work we need to do to check an archive isn't corrupt while decompressing it.
Validation performed

Validated that this PR fixes bad compression ratio outliers during array structurization
Validated that performance seems to be within variance compared to before this change

Great work! One thing to note is that we may need to clarify the distinction between schema and table, as now a table refers to something like a merged table.

Right, yeah that will probably be too confusing to anyone new to this code. We could change the terminology to "streams" or "packed streams" or "packed compression streams" to disambiguate from schema tables, and rename TableReader to PackedStreamReader or something? I kind of like PackedStreamReader since it can hold the double meaning of multiple things packed within a stream, and multiple streams packed together in a file.
We should definitely give it some thought and clear up the terminology before merging at any rate.

Great work! One thing to note is that we may need to clarify the distinction between schema and table, as now a table refers to something like a merged table.

Right, yeah that will probably be too confusing to anyone new to this code. We could change the terminology to "streams" or "packed streams" or "packed compression streams" to disambiguate from schema tables, and rename TableReader to PackedStreamReader or something? I kind of like PackedStreamReader since it can hold the double meaning of multiple things packed within a stream, and multiple streams packed together in a file.
We should definitely give it some thought and clear up the terminology before merging at any rate.

Yeah, PackedStreamReader is better. Not sure if @kirkrodrigues has any better ideas.

Yeah, PackedStreamReader is better. Not sure if @kirkrodrigues has any better ideas.

Seems reasonable.

This PR should have everything it needs to get merged, so could you take another look @wraymo? If you don't have bandwidth to do the final review soon I'll probably go ahead and do a few more changes to make the metadata format forward-compatible with future encoding plans.

This PR should have everything it needs to get merged, so could you take another look @wraymo? If you don't have bandwidth to do the final review soon I'll probably go ahead and do a few more changes to make the metadata format forward-compatible with future encoding plans.

I'm going to go ahead and slightly extend the table packing metadata in the way that I explain in that doc that I shared with you.


Remove hacky workaround and parallelize operations

Now that the hacky workaround is removed, there is no need for temp files. Which also means we can do parallel operations without worry.

@slang25

Hopefully the build is fixed now. Had tried to use System.Text.Json but that dependency wasn't found, so I switched to Newtonsoft which is already being used everywhere in this project.


When passing Classes in html!, component props don't need .clone(), but element props do

Problem
You can write prop_name={&value} when passing a prop of type Classes to a component (because Classes implements ImplicitClone), but you can't when passing it to an element, instead requiring you to write prop_name={value.clone()}, which seems inconsistent.
Steps To Reproduce
#[derive(Properties, PartialEq)]
struct Props {
    class: Classes,
}

#[function_component]
fn Foo1(props: &Props) -> Html {
    html! {

    }
}

#[function_component]
fn Foo2(props: &Props) -> Html {
    html! {
        <Foo1 class={&props.class} />
    }
}

Expected behavior
Successful compilation
Screenshots
error[E0277]: the trait bound `yew::Classes: std::convert::From<&yew::Classes>` is not satisfied
  --> .../foo.rs:69:19
   |
68 | /     html! {
69 | |
   | |                   ^^^^^^^^^^^^ the trait `std::convert::From<&yew::Classes>` is not implemented for `yew::Classes`
70 | |     }
   | |_____- required by a bound introduced by this call
   |
   = note: required for `&yew::Classes` to implement `std::convert::Into<yew::Classes>`
help: consider dereferencing here
   |
69 |
   |                   +

Environment:

Yew version: v0.21
Rust version: nightly
Target, if relevant: wasm32-unknown-unknown

Questionnaire

[x] I'm interested in fixing this myself but don't know where to start
[ ] I would like to fix and I have a solution
[ ] I don't have time to fix this right now, but maybe later

This could be fixed by adding an impl From<&Classes> for Classes, but I'm curious why html! is using From instead of IntoPropValue (as in component props) in the first place.


Copy Subnets

Add prompt to allow user to launch in available subnets

Workon on code here: https://github.com/stefhen/migrate-deployments/tree/copy_subnets

Fixed in https://github.com/stefhen/migrate-deployments/pull/5


New subscribers receive no messages after plumtree_metadata_manager crashed

Environment

VerneMQ Version: 1.3.0
OS: Container-Optimized OS on GKE
Erlang/OTP version (if building from source):

Expected behavior

plumtree_metadata_manager should not crash
New subscriber should receive messages if plumtree_metadata_manager has recovered

Actual behaviour

plumtree_metadata_manager crashed
New subscribers does not receive any messages, while existing subscribers do receive messages

Hello guys, I believe I have ran into the same issue as reported in #547. I have a 4-node cluster where some nodes seem randomly stop publishing messages to new subscribers. However, existing subscribers seem working fine.
I have got three questions here

Should the case clause in https://github.com/erlio/plumtree/blob/master/src/plumtree_metadata_manager.erl#L472 handles an empty list?
Or should plumtree_metadata_object:values(Metadata) not return an empty list?
Does plumtree_sup respawn a new plumtree_metadata_manager? If so, why new subscribers fail to receive messages?

2018-05-22 07:10:06.360 [error] <0.239.0> CRASH REPORT Process plumtree_metadata_manager with 0 neighbours exited with reason: no case clause matching [] in plumtree_metadata_manager:store/2 line 472 in gen_server:terminate/7 line 812
2018-05-22 07:10:06.360 [error] <0.220.0> Supervisor plumtree_sup had child plumtree_metadata_manager started with plumtree_metadata_manager:start_link() at <0.239.0> exit with reason no case clause matching [] in plumtree_metadata_manager:store/2 line 472 in context child_terminated
2018-05-22 07:10:06.362 [error] <0.238.0> gen_server plumtree_broadcast terminated with reason: {{{case_clause,[]},[{plumtree_metadata_manager,store,2,[{file,"/opt/vernemq/distdir/1.3.1/_build/default/lib/plumtree/src/plumtree_metadata_manager.erl"},{line,472}]},{plumtree_metadata_manager,read_merge_write,2,[{file,"/opt/vernemq/distdir/1.3.1/_build/default/lib/plumtree/src/plumtree_metadata_manager.erl"},{line,457}]},{plumtree_metadata_manager,handle_call,3,[{file,"/opt/vernemq/distdir/1.3.1/_build/default/lib/plumtree/src/plumtree_metadata_manager.erl"},{line,316}]},{gen_server,try_handle_call,...},...]},...}
2018-05-22 07:10:06.363 [error] <0.238.0> CRASH REPORT Process plumtree_broadcast with 0 neighbours exited with reason: {{{case_clause,[]},[{plumtree_metadata_manager,store,2,[{file,"/opt/vernemq/distdir/1.3.1/_build/default/lib/plumtree/src/plumtree_metadata_manager.erl"},{line,472}]},{plumtree_metadata_manager,read_merge_write,2,[{file,"/opt/vernemq/distdir/1.3.1/_build/default/lib/plumtree/src/plumtree_metadata_manager.erl"},{line,457}]},{plumtree_metadata_manager,handle_call,3,[{file,"/opt/vernemq/distdir/1.3.1/_build/default/lib/plumtree/src/plumtree_metadata_manager.erl"},{line,316}]},{gen_server,try_handle_call,...},...]},...} in gen_server2:terminate/3 line 995
2018-05-22 07:10:06.363 [error] <0.220.0> Supervisor plumtree_sup had child plumtree_broadcast started with plumtree_broadcast:start_link() at <0.238.0> exit with reason {{{case_clause,[]},[{plumtree_metadata_manager,store,2,[{file,"/opt/vernemq/distdir/1.3.1/_build/default/lib/plumtree/src/plumtree_metadata_manager.erl"},{line,472}]},{plumtree_metadata_manager,read_merge_write,2,[{file,"/opt/vernemq/distdir/1.3.1/_build/default/lib/plumtree/src/plumtree_metadata_manager.erl"},{line,457}]},{plumtree_metadata_manager,handle_call,3,[{file,"/opt/vernemq/distdir/1.3.1/_build/default/lib/plumtree/src/plumtree_metadata_manager.erl"},{line,316}]},{gen_server,try_handle_call,...},...]},...} in context child_terminated

@Lance0312 Hi Lance, thanks for reporting. The underlying dvvset is capable of returning an empty list, however we don't expect it to do so in this case... Something has to be wrong, I'll have to dig into the Plumtree Metadata code to see what it is.
Plumtree Sup should restart those children. However, it is actually possible that the Routing Table subscriptions don't survive the restart. Will investigate this as well.

hi @Lance0312 just curious, are you using any (custom) plugins or is it stock VerneMQ?

Hello @larshesel, only a auth_on_register webhook is enabled.

Also, vmq_passwd is enabled as well.

Hello @dergraf @larshesel, after a short period time of observation. It seems that the problem occurs much more frequently on a cluster of 4+ nodes (have tried 4, 5, and 6 nodes). I am now running the cluster on 3 nodes, though one of the nodes get killed by OOM and restart once a while.
Also, I have manually interpret the failed case clause, and found that it should not be resolved as an empty list. Do you have any ideas why it is resolved as an empty list?
Below is a last message which causes the crash
{
    merge,
    {
        {
            vmq,
            subscriber
        },
        {
            [],
            <<"mosq/Xc7Nx2nCRueH[cl^xB">>
        }
    },
    {
        metadata,
        {
            [
                {
<EMAIL_ADDRESS>                    5,
                    []
                },
                {
<EMAIL_ADDRESS>                    6,
                    []
                },
                {
<EMAIL_ADDRESS>                    7,
                    [
                        {
                            [
                                {
<EMAIL_ADDRESS>                                    true,
                                    []
                                }
                            ],
                            {
                                1527,
                                949468,
                                491482
                            }
                        }
                    ]
                },
                {
<EMAIL_ADDRESS>                    2,
                    []
                }
            ],
            []
        }
    }
}

@Lance0312 thanks for details! is it always the same cluster node crashing?
Also, I'm going to mark this as 'bug', as we obviously need a solution for the empty list question.

@ioolkos, no, it is not always the same node crashing. Sometimes, several nodes crashed at the same time with the same last message (same subscriber field).

Hi @Lance0312 I'm trying to investigate this a bit more. Do you know if the client <<"mosq/Xc7Nx2nCRueH[cl^xB">> was using clean_session=false? Are you perchance using allow_multiple_sessions=on?

Hi, we are falling in the same issue.

Falling into the same issue, on a cluster of three nodes, the reproduction steps are still unclear and we will try to provide more information on this topic. The version is VerneMQ 1.3.0. Enclosed the crash logs and the vernemq.conf.
etc_vernemq_vernemq.21.txt
crash.log.0.txt

Is there any more data you would need to help checking into this issue ? :)

Hi @monsieurpug apologies for our delay in getting to this.
Is upgrading an option? If yes, we'd recommend that including switching to the new vmq_swc layer for distribution of metadata. Set metadata_plugin = vmq_swc in the vernemq.conf (unfortunately this will not take over the existing LevelDB databases, but create new ones).

Thanks for the reply, upgrading might be an option, but I see that this feature is available from 1.7.0 but is still considered in Beta so to us that might be too early for production I guess.
Furthermore, what is the impact of not taking over existing LevelDB databases ? Do we lose existing databases ? Is a classic node-per-node upgrade from 1.3.0 in plumtree to 1.7.0 swc possible ?

@monsieurpug I recommend to upgrade to a version >= VerneMQ 1.6. We introduced a new safe-guard that prevents the plumtree_metadata_manager from crashing in this rare case. Whenever VerneMQ would hit such a case we'd log a warning Known issue, please report here https://github.com/erlio/vernemq/issues/715  (which is this issue btw)
Regarding SWC: We currently don't have automatic migrations that transform the plumtree databases to the SWC databases. So you'd start with a fresh state.

Thanks, that is good news. Do you know what are the consequences on the metadata when falling on the issue and not crashing thanks to the workaround ? Are there maybe inconsistencies between the nodes that could have consequences on the overall service like described here : https://github.com/vernemq/docker-vernemq/issues/67 ?

The problems described in the mentioned issues should have been solved since 1.6.0 as the main problem causing the mentioned problems was the crashing of plumtree_metadata_manager.
However, the root cause of course is and was the problematic metadata object, which at that point doesn't hold data anymore, but hasn't been properly deleted (that's our current hypothesis). So, our current approach is to re-delete the object to ensure it holds the proper tombstone value.

Thanks for the clarification.


Add vulnerability class to the report

Description
This commit adds the vulnerability class (os-pkgs or lang-pkgs) to the report so that it becomes possible to identify vulnerabilities based on this.
Related issues
Fixes https://github.com/aquasecurity/trivy-operator/issues/564
Checklist

[x] I've read the guidelines for contributing to this repository.
[ ] I've added tests that prove my fix is effective or that my feature works.
[ ] I've updated the documentation with the relevant information (if needed).
[ ] I've added usage information (if the PR introduces new options)
[ ] I've included a "before" and "after" example to the description (if the PR is a user interface change).

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

@fardin01 if you wish to discuss it in details I'll be happy. you can find me on Slack

@chen-keinan Thanks for your feedback, it's certainly better this way. Please take another look. Once you're okay with the changes I will update the docs too.

@fardin01 look good , couple of comments:

need to update comment on supported values on Helm values yaml file
need to update docs as well

@fardin01 LGTM 🚀  thanks you for the contribution , I'm cutting a release today , I'll merge the code after the release.


Add indexof-to-includes transform

This transforms code like:
foo.indexOf(bar) > -1

to
foo.includes(bar)

That seems unsafe to me if an object implements an indexOf method that's not on an Array or String. I think I'd prefer not having this in this repo.

Ok!

For anyone who stumbles upon this, I just pushed a new version that preserves comments at 074a22d7e8657be86687920398af0f2450615a13

Thanks for making this codemod though. I'm assuming you audited your codebase to make sure there are no data structures defining indexOf that don't also have an includes :D

Yep!


[Improve][Security] Vite Server Options (server.fs.deny) can be bypassed using double forward-slash (//)

Description

InLong Component
Other for not specified component
Are you willing to submit PR?

[ ] Yes, I am willing to submit a PR!

Code of Conduct

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

I would like to work on this issue


Performance

Serializers.data

StoreFront

Serving Image (Client Side Cache)
https://developers.google.com/web/fundamentals/performance/optimizing-content-efficiency/http-caching?hl=en

Ngnix GZIP
https://www.digitalocean.com/community/tutorials/how-to-optimize-nginx-configuration
4)Postgres
5)Gunicorn

Server side caching

https://www.dabapps.com/blog/logging-sql-queries-django-13/

https://github.com/linkedin/naarad

S3 for files.
Minio - https://github.com/minio/minio


Update README.md

Update with correctly link to WASI, for some browsers at least the current URL cannot be opened.

Thank you @adtrevor!


services script removes all containers and volumes

services script calls stoppingContainers function which removes all containers and volumes in the machine where it's run, not only the ones from the tutorial.

labels have been added to the docker-compose file identifying the tutorial components.
The stoppingContainers function is in place to ensure the tutorial starts up  from a clean slate - it now reduces to:
docker ps --filter "label=org.fiware=tutorial" -aq)
docker volume ls -qf dangling=true)
docker network ls  --filter "label=org.fiware=tutorial" -q)

Volumes is still removing dangling volumes, but networks and container are cleaning up nicely. If you still find the volumes command too aggressive, please update your local copy and comment out the lines here


Skip opencensus and opentelemetry to unblock fetch references job

In the past days no new references have been fetched because of opencensus/opencensus failing to fetch because of an error in a proto file.
The failure is:
processing reference opentelemetry/opentelemetry:acc967d52c8485f0ae3338f2c311ee91a100b70c
opentelemetry/proto/trace/v1/trace.proto:289:32:message opentelemetry.proto.trace.v1.Span: fields flags and parent_span_is_remote both have the same tag 16

The offending commit is this one: https://github.com/open-telemetry/opentelemetry-proto/commit/acc967d52c8485f0ae3338f2c311ee91a100b70c#diff-5c407ac9c675a60170f94ab620f5329f25e5388d6790864b7799abce66f79332 (which is reusing the same tag number)
Skipping both opencensus and opentelemetry, since the former is now being forwarded to the latter.

Additional context: the issue was introduced in commit https://github.com/open-telemetry/opentelemetry-proto/commit/acc967d52c8485f0ae3338f2c311ee91a100b70c which caused an error:
processing reference opentelemetry/opentelemetry:acc967d52c8485f0ae3338f2c311ee91a100b70c
opentelemetry/proto/trace/v1/trace.proto:289:32:message opentelemetry.proto.trace.v1.Span: fields flags and parent_span_is_remote both have the same tag 16

And fixed in commit https://github.com/open-telemetry/opentelemetry-proto/commit/123368f9c39983f3ad191c619f19917a0e48e73b where they removed the ParentSpanIsRemote attribute.
So the following commits need to be ignored/skipped:
acc967d52c8485f0ae3338f2c311ee91a100b70c
473303bdf6c79f03eb5cb1b7b210a3ad3ad37b7c
a2a8f8bbe784d66afa9296b29a5950ed81bbd2a0
ef0a3b1b8a2030af3296b93f08787e25c76ac25e
bd30ca35990411fc47922cd3c0e1184974e685dc
e75abec3bc7e0b4c060d4c355c25f22cf0418754