smart routing makes labels out of position

When smart routing is false, labels are sticking with respective links. but when smart routing is true, link positions are not working as expected.
react-flow-chart version: 0.0.13

@MrBlenny  any update on this?

I was able to create a work around to find the longest line for a link and use the center of that.
First you have to import generateSmartPath, generateRightAnglePath and generateCurvePath
import {   ILinkDefaultProps,   IPosition,   LinkDefault,   generateSmartPath,   generateRightAnglePath,   generateCurvePath, } from "@mrblenny/react-flow-chart";
I then created two functions to find the longest line and the center. (Sorry for the screenshots but the code snippets would not format).

In your component you will need to add these properties.
const points = config.smartRouting     ? !!toPort && !!matrix       ? generateSmartPath(matrix, startPos, endPos, fromPort, toPort)       : generateRightAnglePath(startPos, endPos)     : generateCurvePath(startPos, endPos);   console.log(points);   const center = findCenter(points);
Then you will just need to set your labels x and y axis to the 'center' properties positions.
<Label style={{ left: center.x, top: center.y }}>
Full component is attached. correct extension is .tsx
WorkflowLink.txt


Add option to disable timed dependant tests

The disabled tests rely on timing to pass correctly. On a virtualized system under heavy load, these tests randomly fail because they miss a timer or other timing related issues.

Thanks for the PR. This looks like a good idea but I think it should be done with a plain AC_DEFINE(HAVE_xxx) rather than putting something in apr.h which becomes part of the API.

Thanks for the PR. This looks like a good idea but I think it should be done with a plain AC_DEFINE(HAVE_xxx) rather than putting something in apr.h which becomes part of the API.

This has been corrected now.


Fix:Circular loading spinner is oval in safari browser

Fix:#29041
When defining height and aspect-ratio, Safari will calculate the width based on these two values. However, it appears that Safari applies the aspect-ratio based on the total height (which includes border-width), which may cause the total width of the element to be twice the border-width more than the total height.

When defining width and aspect-ratio, it seems that Safari accurately calculates the height based on these two values

But I have not found any related documentation, so please let me know if there are any issues.

This will likely break this rule below which only sets height:
https://github.com/go-gitea/gitea/blob/8d55eaff851f397d4f5356a7a303b0b251f366c7/web_src/css/modules/animations.css#L48
Further below it sets both:
https://github.com/go-gitea/gitea/blob/8d55eaff851f397d4f5356a7a303b0b251f366c7/web_src/css/modules/animations.css#L64-L65
So I think we should make all 3 rules the same, either set only 1 dimension, or both in all 3.
Regarding the Safari issue: Does changing box-sizing fix it? It sounds like a Webkit bug, check https://bugs.webkit.org/.

This will likely break this rule below which only sets height:
https://github.com/go-gitea/gitea/blob/8d55eaff851f397d4f5356a7a303b0b251f366c7/web_src/css/modules/animations.css#L48
Further below, another variant sets both:
https://github.com/go-gitea/gitea/blob/8d55eaff851f397d4f5356a7a303b0b251f366c7/web_src/css/modules/animations.css#L64-L65
So I think we should make all 3 rules the same, either set only 1 dimension, or both in all three rules.
Regarding the Safari issue: Does changing box-sizing fix it? It sounds like a Webkit bug, check https://bugs.webkit.org/.

I tried changing box-sizing but it did not solve it, perhaps it would be more appropriate to set both width and height

I think https://bugs.webkit.org/show_bug.cgi?id=244697 might be the related bug.

I think this might be the related bug, but there are also others.

I might have found the related one in the list you provided：https://codepen.io/legion80/pen/ZEPeLdz
It displays correctly when absolute positioning is not used on ::after（It has been centered using flex layout）

Yes, it's the same bug. It's strange that it only happens with height but not width. The workaround is likely good but we should add a link comment to the webkit bug and adjust the two subsequent selectors I linked to only set width.

updated:3102bd9

Hmm, we need to be really careful with these spinner CSS adjustments, they are used in many places in the UI.

You're right, I've already checked everything I could notice about the spinner's CSS, could there maybe be something I've missed?

You're right, could there possibly be something that I've missed in my adjustments? Perhaps we can also opt for minimal changes, just fixing the positioning issue with ::after, so that even if the height and aspect-ratio are used together, it won't cause a problem on Safari.

Hmm, we need to be really careful with these spinner CSS adjustments, they are used in many places in the UI.

You're right, could there possibly be something that I've missed in my adjustments? Perhaps we can also opt for minimal changes, just fixing the positioning issue with ::after, so that even if the height and aspect-ratio are used together, it won't cause a problem on Safari.

I'd say keep changeset minimal to fix the issue. I think just switching width to height in the .is-loading class and the classes below that extend the selector. There might also be some non-CSS code that adjust height currently, if they exist, they need to switch as well. Also add a comment linking to the webkit bug.

I'd say keep changeset minimal to fix the issue. I think just switching width to height in the .is-loading class and the classes below that extend the selector.
updated

There might also be some non-CSS code that adjust height currently, if they exist, they need to switch as well.
It seems that no results were found through the is-loading search.

Also add a comment linking to the webkit bug.
updated

It breaks the loading UI http://localhost:3000/devtest/gitea-ui
Before:

After:

It breaks the loading UI http://localhost:3000/devtest/gitea-ui
Before:

After:

This seems to be caused by the flexible layout, it doesn't seem to be an effective method to solve

It does reproduce on /devtest, let's see if I find a simpler fix.

Found a better workaround: https://github.com/go-gitea/gitea/pull/29801

Thanks for your effort in this PR @HEREYUA, I think we'll use the alternative proposed in the other PR. This PR was helpful in progressing towards the solution and is very much appreciated :)


[plugins] dart analyze fails in local_auth

As shown here, it seems that local_auth/local_auth has analyzer issues.
Running locally gives the below error.
A PR was raised.
Analyzing local_auth...

   info • example\lib\readme_excerpts.dart:30:7 • Use key in widget
          constructors. • use_key_in_widget_constructors

1 issue found.

The problematic code was reverted (https://github.com/flutter/plugins/pull/5691) and then relanded (https://github.com/flutter/plugins/pull/5692). flutter/main has recovered and is accepting changes normally.
Thanks for the report!


Adding the web app

Introduce a web app that can be run locally within a separate directory.
Default next js app
The next js app is created using https://create.t3.gg/
It build with strong type and tailwind by default
Type file
This PR include a lot of types in src/types
They are generated by chat gpt base on the erc7730 schema
interface ERC7730Schema  is a base interface too later build our Design
Lint on json file
Some json file has been automatically lint by my vscode

This PR was spllited and cleaned into smaller one


Fix deprecation of Connection::executeUpdate in InheritanceUpdaterTrait

1. Why is this change necessary?
Shopware\Core\Framework\Migration\InheritanceUpdaterTrait::updateInheritance calls the deprecated Doctrine\DBAL\Connection::executeUpdate method.
This can safely be resolved because the method is alredy deprecated in version 2.13.8 which is the minimum supported version according to composer.json
Source: https://github.com/doctrine/dbal/blob/2.13.8/lib/Doctrine/DBAL/Connection.php#L1479
2. What does this change do, exactly?
It updates InheritanceUpdaterTrait to call Doctrine\DBAL\Connection::executeStatement instead of the deprecated Doctrine\DBAL\Connection::executeUpdate.
executeUpdate intern just calles executeStatement and triggers a deprecation. Therefore it should be safe to just directly use executeStatement instead.
3. Describe each step to reproduce the issue or behaviour.
Call updateInheritance on Shopware\Core\Framework\Migration\InheritanceUpdaterTrait.
4. Please link to the relevant issues (if any).
Deprecation of Connection::executeUpdate: https://github.com/doctrine/dbal/pull/4163
5. Checklist

[x] I have rebased my changes to remove merge conflicts
[ ] I have written tests and verified that they fail without my change
I'm not sure how to test that a deprecation is (not) triggered. I'd be willing to create a test for this if there is a way.
[x] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-23855
Please use this issue to track the state of your pull request.

@cngJo Thanks for your contribution :tada: We have merged  your commit


grant s3:GetObject permissions for transfer-products lambda

Now that we're transferring files using s3.Bucket.copy(), instead of downloading them via the public URL using requests, we need to grant s3 permissions to read the source data from HyP3.
I'm leaving the permissions generic so transfer-products could potentially read data from any (or all) hyp3 deployments, but I could be talked into adding a SourceBucket cloudformation parameter to make the read permissions specific.
We're using s3.Bucket.copy(), I can't remember if that function tries to copy S3 tags along with the object, so we might end up also needing s3:GetObjectTagging permissions.

This relates to the change made this morning in https://github.com/ASFHyP3/hyp3-flood-monitoring/pull/49 (changing how we transfer products, this IAM change is needed to support that change)


mj-column not working in Outlook 2010 (desktop)

Hi,
mj-column is not rendering width: 50% or 30% for Outlook 2010 (Desktop). My code:
<mj-section>
  <mj-group width="100%">
    <mj-column>
      <mj-image width="137px" height="185px" padding="0" src="https://mjml.io/assets/img/easy-and-quick.png" />
      <mj-text align="center">

Easy and quick

Write less code, save time and code more efficiently with MJML’s semantic syntax.

      </mj-text>
    </mj-column>
    <mj-column>
      <mj-image width="166px" height="185px" padding="0" src="https://mjml.io/assets/img/responsive.png" />
      <mj-text align="center">

Responsive

MJML is responsive by design on most-popular email clients, even Outlook.

      </mj-text>
    </mj-column>
  </mj-group>
  <mj-group width="100%">
    <mj-column>
      <mj-image width="137px" height="185px" padding="0" src="https://mjml.io/assets/img/easy-and-quick.png" />
      <mj-text align="center">

Easy and quick

Write less code, save time and code more efficiently with MJML’s semantic syntax.

      </mj-text>
    </mj-column>
    <mj-column>
      <mj-image width="166px" height="185px" padding="0" src="https://mjml.io/assets/img/responsive.png" />
      <mj-text align="center">

Responsive

MJML is responsive by design on most-popular email clients, even Outlook.

      </mj-text>
    </mj-column>
  </mj-group>
</mj-section>

Hi,
You have more than 100% of content inside a section,
If you want to have 2 group of 100% (note that the width is no longer needed if you have only that inside a section), then you should have 2 section.


Fixes so prototype cardano-node can sync Byron from mainnet using in-mem backing store

Fixes CAD-3904.
These commits include small patches that ultimately enabled a cardano-node prototype (branch nfrisby/CAD-3904-utxohd-inmem-only in that repo) to sync Byron from mainnet. (It failed on the first non-empty Shelley block; CAD-4018 tracks that.)

Everything builds on this commit, but, as expected, some tests still fail; these changes were only driven by getting the cardano-node branch to successfully sync (only Byron, as it turns out) via ./cardano-node.exe run --topology ./configuration/cardano/mainnet-topology.json --database-path ./state --port 3001 --config ./configuration/cardano/mainnet-config.yaml  --socket-path \\.\pipe\cardano-node

bors merge

bors cancel


Every task as a section 🤔

Why do you handle every task as a tableview section instead a section of tasks?!
TaskListViewModel.swift#L51

@LeonardoCardoso, every task is handled as a row in a single section: https://github.com/devxoul/RxTodo/blob/2b189e76e5bc5532aefc0134dd8583e2dd1dc44a/RxTodo/Sources/ViewControllers/TaskListViewModel.swift#L59-L64

Nvm.

Hey, @devxoul! Yes, this is the point! I meant a row in a section. I was just wondering why... I'm asking that because I had a real hard work converting to a RxTableViewSectionedAnimatedDataSource because the insertion and deletion with animation are more intuitive in this case.

@LeonardoCardoso hmm I have no idea what you're talking about :( Did you mean why did I use
Section
  - Task
  - Task
  - Task

instead of
Section
  - Task
Section
  - Task
Section
  - Task

?

Yes.


Quantum shim moment.js in vendor bundle

I am unable to shim moment.js in the quantum vendor bundle.
The shim works well when not using quantum.
When using quantum, the source js is not included.
I would appreciate any guidance.
Best regards,
John
fuse.bundle('vendor')
.shim({
jszip: {
source: "node_modules/jszip/dist/jszip.js",
exports: "jszip"
},
moment: {
source: "node_modules/moment/moment.js",
exports: "moment"
}
})
.instructions(' ~ main.ts ');
// moment/index.js
$fsx.f[1] = function(module,exports){
module.exports = moment;
}

Hi John!
That might be a bug, in the meantime try setting it to the global config, it should work (I think)

I wasn't able to reproduce the issue.
But i think i know what's the problem
const { FuseBox, QuantumPlugin, WebIndexPlugin, HTMLPlugin, UglifyJSPlugin, CopyPlugin } = require("fuse-box");
const isProduction = true;
const fuse = FuseBox.init({
    homeDir: "src",
    output: "dist/$name.js",
    shim : {
        foo : {exports : "window.foo", source : "./foo.js"}
    },
    plugins: [
        WebIndexPlugin(),
        isProduction && QuantumPlugin({
            target: "browser",
            treeshake: true,
        })
    ]
});

fuse.dev();
fuse.bundle("app")
    .instructions(`>index.ts`);

fuse.run();

Quantum creates shims.js file. So if you used WebIndexPlugin you wouldn't have had this problem.


Bin directory from rubocop is excluded

See this: https://github.com/yegor256/pdd/blob/48cd3ba95a258e438ed9a0e5b0a37de71fdde57f/.rubocop.yml#L3
The bin folder must be checked by the Rubocop.

@yegor256/z please, pay attention to this issue

@aivinog1/z this project will fix the problem faster if you donate a few dollars to it; just click here and pay via Stripe, it's very fast, convenient and appreciated; thanks a lot!

@aivinog1 feel free to submit a pull request

@0crat assign @aivinog1

@0crat assign @aivinog1  (here)

@yegor256 The job #119 assigned to @aivinog1/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; @aivinog1/z is not a member of this project yet, but they can request to join, as §1 explains; there will be no monetary reward for this job

Manual assignment of issues is discouraged, see §19: -5 point(s) just awarded to @yegor256/z

It is strongly discouraged to assign jobs to their creators, see §19: -15 point(s) just awarded to @yegor256/z

Bug was reported, see §29: +15 point(s) just awarded to @aivinog1/z

@rultor/z the issue is closed not by @aivinog1/z (its creator); I won't close the order; please, re-open it and ask @aivinog1/z to close it

@yegor256 Can you reopen this ticket that I can close its properly.

@yegor256 ping.

@aivinog1/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@0crat wait for reply from @yegor256

@0crat wait for reply from @yegor256 (here)

@aivinog1 The impediment for #119 was registered successfully by @aivinog1/z

@yegor256/z please, pay attention to this issue

@aivinog1 reopened

@aivinog1/z this project will fix the problem faster if you donate a few dollars to it; just click here and pay via Stripe, it's very fast, convenient and appreciated; thanks a lot!

@yegor256 Ok, I'm closing it.

@0crat status

@0crat status (here)

@aivinog1 This is what I know about this job in C3T46CUJJ, as in §32:

The job #119 is in scope for 6days
The role is DEV
The job is assigned to @aivinog1/z for 6days
There is no monetary reward attached, it's a free job
The job has an impediment
The budget is 30 minutes/points
These users are banned and won't be assigned:

@aivinog1/z: This user reported the ticket

Job footprint (restricted area)

Job #119 is not in the agenda of @aivinog1/z, can't inspect

@dennis00010011b/z please review this job completed by @aivinog1/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

The job #119 is now out of scope


Upgrade eslint and jest

cc @cpojer

Thanks! Can you add "testEnvironment": "node" to the Jest config? It should make Jest's startup instant because we won't have to wait for jsdom.

Sure!


[QA] determine if we should add back LRO status operations

In the swagger readme it appears we are removing all the LRO status operations from the code: https://github.com/Azure/azure-sdk-for-python/blob/main/sdk/cognitivelanguage/azure-ai-language-questionanswering/swagger/README.md?plain=1#L276
Due to arch board feedback, we decided to keep these in CLU. Need to understand if we want to do the same thing for QA.

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Storage:0.2486428,Docs:0.18152589,Azure.Core:0.043674923'

Still working out the best solution/pattern for exposing these methods. We will not expose in this release.


signal: signal handler may cause task's state error

For example, task is blocked by nxsem_wait(sem1), use nxsem_wait(sem2)
in signal handler, and take sem2 successfully, after exit from signal
handler to task, nxsem_wait(sem1) returns OK, but the correct result
should be -EINTR.
Signed-off-by: Zeng Zhaoxiu<EMAIL_ADDRESS>Summary
Impact
Testing

For example, task is blocked by nxsem_wait(sem1), use nxsem_wait(sem2)
in signal handler, and take sem2 successfully, after exit from signal
handler to task, nxsem_wait(sem1) returns OK, but the correct result
should be -EINTR.

@zhaoxiu-zeng
Can you extend the ostest so that we can test if the PR works correctry?

For example, task is blocked by nxsem_wait(sem1), use nxsem_wait(sem2)
in signal handler, and take sem2 successfully, after exit from signal
handler to task, nxsem_wait(sem1) returns OK, but the correct result
should be -EINTR.

@zhaoxiu-zeng
Can you extend the ostest so that we can test if the PR works correctry?

For example, task is blocked by nxsem_wait(sem1), use nxsem_wait(sem2)
in signal handler, and take sem2 successfully, after exit from signal
handler to task, nxsem_wait(sem1) returns OK, but the correct result
should be -EINTR.

@zhaoxiu-zeng Can you extend the ostest so that we can test if the PR works correctry?

https://github.com/apache/incubator-nuttx-apps/pull/955


Changed 'include' to 'include_tasks'

Changed 'include' to 'include_tasks' as 'include' will be deprecated in ansible v2.8
Changes
Outline changes pull request is introducing
Verify
Add any steps on how to verify the changes work as intended

Add any relevant fixes or closes to reference github issues

A few things here:

Looks like these changes introduce failures

Test failures are related to these changes as collectd appears to not have been ran correctly

There isn't any backwards compatibility for versions of ansible that don't know about include_tasks
In the future please follow the contributing templates provided for issues, pull requests, etc.

Open to getting this updated as it will slowly become deprecated. Just need to consider the backwards compatibility as well as making sure the tests pass.


Wildcard in redirect URI seems to be unsupported even for "signInAudience": "AzureADMyOrg"

In the article it says

Wildcard URIs are allowed, however, for apps that are configured to sign in only work or school accounts in an organization's Azure AD tenant. To add redirect URIs with wildcards to app registrations that sign in work or school accounts, use the application manifest editor in App registrations in the Azure portal.

I tried doing this and I would get an error "Error detail: Invalid value specified for property 'replyUrlsWithType' of resource 'Application'. "

I checked and my app registration has 	"signInAudience": "AzureADMyOrg". Am I doing something wrong or is the documentation now outdated? I need to do this for the same reason as this StackOverflow post, so that I can get it to work with my CI/CD pipeline.

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

ID: cc4c5921-4070-a714-44e6-c4309fe5fd53
Version Independent ID: c52c5c94-bb2e-10cb-9673-98aae4aa37d4
Content: Redirect URI (reply URL) restrictions - Microsoft Entra
Content Source: articles/active-directory/develop/reply-url.md
Service: active-directory
Sub-service: develop
GitHub Login: @madansr7
Microsoft Alias: saumadan

@jlian Thanks for your feedback. Could you please specify what value you are passing in redirect URI?

I'm trying to do https://static-string-*.centralus.1.azurestaticapps.net/.auth/login/aad/callback - for Azure Static Web App's pull request preview environment feature, specifically the pull request kind

@jlian Wildcards are allowed in redirect URI to sign in with work or school accounts. A wildcard ‘*’ should represents a valid tokens sequence and jointed by specific flag ‘.’ or ‘/’.
Belows examples are accepted as valid URLs:
https://static-string-/centralus.1.azurestaticapps.net/.auth/login/aad/callback
https://static-string-/./centralus.1.azurestaticapps.net/.auth/login/aad/callback

As this is not documented related issue. So, closing this issue here.

Sorry, I don't understand how this isn't a documentation issue. Shouldn't we make it clear in the docs how the user would add wildcards in the redirect URI?
BTW, after several hours of trial and error I got it to work with one pattern:
https://*.centralus.1.azurestaticapps.net/.auth/login/aad/callback

The first part cannot have any other characters at all. This is all currently undocumented.

@shwetamathur7 please reopen this issue as docs updates are still needed

I am running into this problem as well. The documentation states:
To add redirect URIs with wildcards to app registrations that sign in work or school accounts, use the application manifest editor in App registrations in the Azure portal.

However, I have been unable to get this to work in any capacity.

Having the same issue, already tried a lot of the options here. Interestingly *.tld is accepted but adding a known subdomain with wildcard apex domain seems not to be

@shwetamathur7 Can this be reopened since there is definitely a documentation problem? I've run into this issue 3 times over the past year, and I always end up spending over an hour troubleshooting before I remember the docs are still incorrect.

Same issue here. Wildcard redirect uris are not working. I'm using the https://*.example.com/login/callback structure and even that is not working.
He always seems to return back to the last known redirect URI without a wildcard that was configured, even if that callback URI was deleted. I can save the URIs in the manifest but it seems they are completely ignored and I'm getting returned to an old URI that had no wildcards in it. Very strange.

Sorry, I don't understand how this isn't a documentation issue. Shouldn't we make it clear in the docs how the user would add wildcards in the redirect URI?
BTW, after several hours of trial and error I got it to work with one pattern:
https://*.centralus.1.azurestaticapps.net/.auth/login/aad/callback

The first part cannot have any other characters at all. This is all currently undocumented.

This is the only solution that worked.

@jlian thanks for persevering with this!  I would've been totally stumped had it not been for this issue.  Really ought to be reopened and the docs updated.

This should be re-opened, as it's not possible to add wildcard using portal UI, it is possible only by manual updating the manifest file, and there is NO any information on the MS Documentation related to this.
https://learn.microsoft.com/en-us/azure/active-directory/develop/reply-url
this states:
Wildcard URIs are currently unsupported in app registrations configured to sign in personal Microsoft accounts and work or school accounts.
Wildcard URIs are allowed, however, for apps that are configured to sign in only work or school accounts in an organization's Microsoft Entra tenant.
So those two sentances already contradict each other.
Also, why you don't simple change the validation in the Azure Portal UI to allow adding wildcards for organization's Microsoft Entra tenant?

Still getting valiation error when adding the wild card in manifest.json


Dealing cards

Any way to deal black jack with a simple command?
One face up, the other down.
Opponent has both face up?

Not at the moment.
BTW, looking at https://en.wikipedia.org/wiki/Blackjack I see there is no such thing as a single uniform "black jack", pretty much every place has it's own spin of the system.


Unsafe Bootstrap Command with user defined cluster wide read_only block setting

Is your feature request related to a problem? Please describe.
Sometimes Opensearch cluster faces quorum loss (half or more than half of master eligible nodes are lost). We have opensearch-node unsafe-bootstrap command which allow us to bootstrap cluster (but with some data loss). Many times Cx would not know about this data loss. One way to fix this is by restoring cluster from snapshot.
Describe the solution you'd like
During bootstraping cluster if we apply read only block to whole cluster (cluster.blocks.read_only) and give some kind of notification (quorum loss + unsafe-bootstrap), Cx can take appropriate actions from their side. No write request should execute successfully by any node of cluster after bootstrap to prevent data corruption (Atomicity of bootstrap + application of read only block). Cx can recover data by restoring cluster from snapshot in case any. To apply read only block after unsafe bootstrap is optional (default is false, but true is recommended).
Describe alternatives you've considered
opensearch-node unsfe-bootstrap command already has a warning message and confirmation before running the tool. This is just an extra layer of soft signal we can provide to prevent data loss.
Additional context
Add any other context or screenshots about the feature request here.

Why should we provide read only block as part of unsafe bootstrap command? Can't you apply the setting externally from your orchestration system that sends out the notification as well?

Think about following race condition.
We want that settings to be in cluster immediately after bootstrap so that any node which joins newly formed cluster has that setting applied. Basically we want both operation "to apply read only block" and "bootstrap the cluster" in atomic way. Otherwise assume any node which joins cluster before applying read only block from external orchestration, can allow write request to cluster followed by application of read only block, can lead to data corruption for cx.
So best way to do this is applying settings before bootstrap, hence we should take this as arguments of unsafe-bootstrap command.

@harmishlakhani can you please update the description of the issue with these details as it is not clear why we want to introduce read_only block during unsafe-bootstrap


updata service and node api - [closed]

In GitLab by @luotian-github on Oct 9, 2017, 09:34
Merges apidev -> dev
updata service and node api

In GitLab by @wknet123 on Nov 13, 2019, 10:05
closed


about data's version

the normal expression of 3d voxel file is .binvox.do you konw how to convert the result of .npy to .binvox?

I realize this is a tad old but does this not solve your problem: https://github.com/EdwardSmith1884/3D-IWGAN/blob/3cb004619bd7e37d2b33cf300622154656051615/scripts/binvox_rw.py#L233


Issue #3313 fixed

*BUGS:
Issue: #3313
For things other than bugs:

Describe that thing in very short line, word limit is 200.
Otherwise use issue #id if the issue was filed as feature request.

@bhanu-webkul
issue resolved please let me know if any remains

done

HI @vishal-webkul
KIndly resolve conflicts so that we can merge this branch.


typescript array item type deduce problem

Suppose in TypeScript, there is a variable named 'arr' which is an array of type 'Array', and when accessing 'arr[n]', the value may be undefined. However, TypeScript recognizes the type of 'arr[n]' as 'T'. Will ts-reset fix this issue?

No, but you can use noUncheckedIndexedAccess to help.


Fix the XX:MaxDirectMemorySize exception

Hi Guys,
I fixed the error by letting the forked peon process only index with a single thread to constraint the memory usage and therefore the XX:MaxDirectMemorySize error will be thrown. This might make indexing a bit slower since it only will use one thread, but since it is a single machine, performance will not be one of the goals I would guess.
Cheers, Fokko

A small update:
8) Not enough direct memory.  Please adjust -XX:MaxDirectMemorySize, druid.processing.buffer.sizeBytes, druid.processing.numThreads, or druid.processing.numMergeBuffers: maxDirectMemory[2,147,483,648], memoryNeeded[10,737,418,240] = druid.processing.buffer.sizeBytes[1,073,741,824] * (druid.processing.numMergeBuffers[2] + druid.processing.numThreads[7] + 1)

By default, the druid.processing.numThreads property is determined dynamically. Therefore the XX:MaxDirectMemorySize should also be dynamic, but we can also set it hard so we are sure that everyone can run a indexing job, but might give a performance penalty because there are potentially more resources available; for example, only using 1 core instead of 4. But this docker image, in its current state is not suitable for production.

i am not against increasing the amount of direct memory, but it might be better to lower the buffer processing size druid.processing.buffer.sizeBytes and set num of threads to 2.

Fair point, let me change this.

I just decreased the druid.processing.buffer.sizeBytes to 512m. Therefore we can keep the direct memory at 2g. One processing, two mergebuffers, and one additional thread makes 2g in total.
I just build and tested the image locally.

👍


We don't know what the message::vision::Goal::MeasurementType enum means

As mentioned in #315 here, we don't know what the goal measurement types actually mean/do. We need to figure out what they do, then document that field in Goal.proto, and make sure that the use of this message is consistent with that finding.

This was resolved by #713. We now only have two types of measurement, and we have documented what they mean. Closing


Can't set empty header

Code:
$curl = new Curl();
$curl->setHeader('test', '');
$curl->setHeader('test2;', '');
$curl->setHeader('test3;');

there is no any way to set empty headers

Hi.
To remove an internal header, use Curl->removeHeader():
$curl = new Curl();
$curl->removeHeader('User-Agent');
$curl->get('https://www.example.com/');

To set a header with no value, use Curl->setOpt():
$curl = new Curl();
$curl->setOpt(CURLOPT_HTTPHEADER, ['X-Custom-Header;']);
$curl->get('https://www.example.com/');

Use $curl->verbose(); to verify what is being used in the request.


chore: added typegen test for type with native asset

Closes #3073

Summary

Incoming from the forum.
Sway type and generated TypeScript types

Checklist

[X] All changes are covered by tests (or not applicable)
[X] All changes are documented (or not applicable)
[X] I reviewed the entire PR myself (preferably, on GH UI)
[X] I described all Breaking Changes (or there's none)

Closed as this appears to be an issue in Sway:

https://github.com/FuelLabs/sway/issues/6488


Amazon Minimum now $1.00?

I started getting timeout errors, so I turned off headless mode to see what was happening, and was presented with this:

I don't know if this is affecting everyone, or just me, but figured I would post here with a heads up.

Bummer... Me too...

Set amount_min: 100 in config.txt. Example here: https://jakehilborn.github.io/debbit/#configtxtExplanation


Build and publish docker image to Azure

labels such as enhancement are meaningless IMO.
Provide a description, a better title that talks about what we're trying to achieve and put it up on the DevSecOps project.


Add clear form button after answering

The form now clears automatically after submitting, so this is no longer needed.


Tracking Screen Views

Is there a way to track screen view particular to a user as we do in google analytics ?

Just log your screen
Something like this..
import Analytics from 'react-native-firebase-analytics';

    // somewhere in your code, for example componentDidMount
    // Firebase Analytics
    Analytics.logEvent('view_my_screenX', {
      'item_id': product._id,
      'in_stock': product.inStock,
      'some_nice': "other_value"
    });

Will this work ? for tracking screen view ?
Analytics.logEvent("screen_view",{
"name":"my_screen_name"
})

Analytics.setScreenName('my_screen_name')?

@evollu I think Analytics.setScreenName('Screen Name') makes more sense, however one needs to be careful to tag all screens otherwise screen names will be incorrectly  assigned to the last tagged screen

I think he wants to track screen with custom dimensions
https://github.com/idehub/react-native-google-analytics-bridge/blob/master/src/GoogleAnalyticsTracker.js#L82

@sibelius I don’t think firebase SDK supports that


When use Pluck method with a struct slice, gorm can not fill the value into slice, but the slice struct element implemented sql.Scan

Your Question

When use Pluck method with a struct slice, gorm can not fill the value into slice, but the slice struct element implemented sql.Scan

The document you expected this should be explained
Can you support this operation in later upgrades?

https://gorm.io/docs/advanced_query.html#Pluck

When use Pluck method with a struct slice, gorm can not fill the value into slice, but the slice struct element implemented sql.Scan

Pluck need to scan each row to a struct and append it to the slice, it can't works with a scanner.


Test 1

Current dependencies on/for this PR:

main

PR #1   👈

PR #2

This stack of pull requests is managed by Graphite.


fix: completed tls credential context typing

:tada: This PR is included in version 2.5.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Support object literal arguments

Are there plans to support these?  Your library is my favorite GraphQL library for Java/Kotlin, but not being able to send objects to my mutations makes it a deal-breaker for many of my projects.

I was curious as well, and it might be because of the rules for input objects listed in the spec?
http://facebook.github.io/graphql/October2016/#sec-Input-Object-type-validation
Moshi Json custom type adapters do this pretty cleanly and something similar might work here, though. Explicitly declaring schema types is already required in kgraphql, why not add a input object declaration DSL in the same place? @pgutkowski thoughts?

Hi @malteseduck, thank you for your report.
I'd love to introduce support for object literals, but unfortunately lately I don't have much time due to my career and personal responsibilities. Additionally, responding to @prestongarno question, this feature is not there mainly because request parsing module is not designed well and it should be refactored before adding features.
I hope you know that you still can pass objects as arguments in variables, like in this test case
Anyway, with subscriptions and pagination it will be my main concern for future development.


Collapsable Project Namespace "Projectless" View

What kind of request is this (question/bug/enhancement/feature request):
Enhancement
Steps to reproduce (least amount of steps as possible):
As a global admin launch a cluster.
After the cluster is ready add a namespace at the cluster level. Do not select a project for the namespace from the drop down.
Navigate to the default project, then namespaces.
Result:
The project-less name space is visible but should be collapsed.

Close in favor of https://github.com/rancher/rancher/issues/22986


Mebo 2.0 Support

Support for the Mebo 2.0, without a Raspberry pi, since it already requires a phone to use

I can start working on this now, but cannot test until at least January 14

Waiting until 0.6 is released


Fails to download zips

[Program.cs_Main(110) @ 23:27 - Info] Exception downloading BeastSaberFeed (2) - System.AggregateException-One or more errors occurred.
at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
at SyncSaberService.SyncSaber.DownloadSongs(Dictionary`2 queuedSongs)
at SyncSaberService.SyncSaber.DownloadSongsFromFeed(String feedType, IFeedSettings _settings)
at SyncSaberService.Program.Main(String[] args)
Temp directory has 1KB corrupted zips so everything fails

Also getting this error.
[SyncSaberService @ 04:02 - Info] Playlist "SyncSaber Playlist" found in old playlist format.
[SyncSaberService @ 04:02 - Info] Playlist "BeatSaver Newest" found in old playlist format.
[SyncSaberService @ 04:02 - Info] Playlist "BeastSaber Curator Recommended" found in old playlist format.
[SyncSaberService @ 04:02 - Info] Playlist "BeastSaber Followings" found in old playlist format.
[SyncSaberService @ 04:02 - Info] Playlist "BeastSaber Bookmarks" found in old playlist format.
[SyncSaberService @ 04:02 - Info] Playlist "ScoreSaber Top PP" found in old playlist format.
[SyncSaberService @ 04:02 - Warning] Skipping FavoriteMappers.ini feed, no authors found in C:\Program Files (x86)\Steam\steamapps\common\Beat Saber\UserData\FavoriteMappers.ini
[SyncSaberService @ 04:02 - Info] Downloading songs from bookmarks feed...
[SyncSaberService @ 04:02 - Info] Finished checking pages, found 43 songs
[Program.cs_Main(142) @ 04:02 - Info] Exception downloading BeastSaberFeed (1) - System.AggregateException-One or more errors occurred.
at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
at System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
at SyncSaberService.SyncSaber.DownloadSongs(Dictionary`2 queuedSongs)
at SyncSaberService.SyncSaber.DownloadSongsFromFeed(String feedType, IFeedSettings _settings)
at SyncSaberService.Program.Main(String[] args)
[SyncSaberService @ 04:02 - Info] Finished downloading songs in 0 min 1 sec

This is due to BeatSaver's new song ID format (single hex number instead of 1234-1234). An update should be coming sometime this week to fix it.

Fixed in 0.1.0


[COMMUNITY] David Riazati -> Committer

Please join us to welcome @driazati as a new committer to TVM. He has contributes lots of improvements to our CI, build and general automation that are instrumental for us to serve our growing community.

Commits History
Code Review
Community Forum Summary

Congratulations @driazati!

Thanks for the work on CI especially @driazati B), it's a whole lot better now!


Export all methods for the GQLType class

Hello!
I noticed that release 0.15.0 says

GQLType exposes typeOptions to modify labels: typeOptions :: f a -> GQLTypeOptions

But I could not use it as it does not seem to be exposed. At least not when importing Data.Morpheus.Types. The haddock seems to confirm this as it only shows the methods description and implements: http://hackage.haskell.org/package/morpheus-graphql-0.15.0/docs/Data-Morpheus-Types.html
This PR should export all the methods, but perhaps it's overkill and more than desired. Happy to make any changes.
Thank you!

hi @carlosdagos,  thank for your PR!
we can expose: KIND, description, implements, getDescriptions, typeOptions, getDirectives. other functions should be  hiddden.

Thank you @nalchevanidze! Done in latest commit :+1:


Update and rename datepicker-syles-and-templates.md to datepicker-sty…

Rename datepicker-syles-and-templates.md to datepicker-styles-and-templates.md

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

I think the TOC also needs to be updated.

Because this PR changes a file name, we need to add a redirect node here, or anywhere in this file.
The new node should be:
        {
            "source_path": "docs/framework/wpf/controls/datepicker-syles-and-templates.md",
            "redirect_url": "/dotnet/framework/wpf/controls/datepicker-styles-and-templates",
            "redirect_document_id": true
        }

We also need to update incoming links to that topic:

@acidshards including redirects and internal links, this is now looking like a larger PR.
If you want to make those changes, we'd be thrilled to take them. If not, let us know and we'll accept these improvements, and add a new PR to fix the redirections and internal links.
Thanks again for spotting this and fixing it.

Thanks again for the contribution @acidshards I just :shipit:.  It will appear on the live site in a day or so.

@BillWagner @mairaw I'll knock out the links real quick 🏃. Also, the redirect file is starting to get messy again, so I'll fix that up, too.


Add a deprecation warning on the 'Available for hire' docs page

What article on docs.github.com is affected?
https://docs.github.com/en/github/setting-up-and-managing-your-github-user-account/what-does-the-available-for-hire-checkbox-do
What part(s) of the article would you like to see updated?
The page gives a great explanation of what the checkbox does, great! But what if the user doesn't head to jobs.github.com to see that the entire point of the checkbox and the docs' explanation didn't matter as It's shutting down? Now that's a problem.
It would be ideal for the page to be reflected with information stating it's currently deprecated, and again when its fully shut down.
Additional information
EDIT: After further research, another two locations which may benefit from a similar change could be https://docs.github.com/en/github/getting-started-with-github/github-glossary#github-jobs and https://docs.github.com/en/github/finding-talent-with-github-jobs


GIMP-from-git snap (WIP)

Add changes to snappy-playpen/README.md and add gimp-git/ project directory

Thanks so much for the contribution, you rock!
Looking at the continuous integration output, it seems that snapcraft fails. Could you add any comments about the failure and where you'd need help?

As dholbach as mentioned on gitter, the snap doesn't yet have glib or gtk-pixbuf as parts with deps yet.
However my biggest hangup is I don't really have a clear idea of how to issue build environment variables into the part building process, and once parts are staged how to get the later parts to see/use them as dependencies.

I get very consistent errors which come from not providing ACLOCAL with what it wants:
libtool: install: /usr/bin/install -c .libs/libgio-2.0.lai /root/parts/glib/install/lib/libgio-2.0.la
libtool: warning: remember to run 'libtool --finish /lib'   <-- Every component in a built part complains about not being in '/lib' when it's finished compiling
I think something like libgio is a file or fileset kind of include, where the system's shipped version is fine, and maybe needs to be included(?), but don't have a clear idea of how to do that either.

Holy moley, got all the way through to building gimp itself on a snapcraft cleanbuild just now.
Here's the error output from that:
== Missing mandatory dependencies ==

GIMP configuration failed.
The following mandatory dependencies are missing:
  - gegl-0.3 >= 0.3.6
  - gegl
      *** Could not find gegl in your PATH.
  - gtk-update-icon-cache
      *** Could not find gtk-update-icon-cache in your PATH.
  - libmypaint >= 1.3.0
  - Python 2 headers
      *** Please install them, or skip building the python scripting extension by
      *** passing --disable-python to configure (but then you will not be able
      *** to use scripts for GIMP that are written in Python).

It won't solve your problem, but for the missing python 2 headers, you can probably for now do as the comment says at the end of the config step and pass --disable-python to the build

For gegl, it seems the configure script is looking for the $GEGL env variable https://github.com/GNOME/gimp/blob/master/configure.ac#L655

And continuing on gegl, I think the issue might be that you're pulling from master instead of the gegl-03 branch. Or at least this seems to indicate a similar issue

Pulling gegl master is a big part of gimp-git, as that's where nearly all the pixel buffer manipulation horsepower is, so I really want the build to include that for sure (as opposed to the documentation or i18n) while getting it to start working.
Regarding the environment variable action, can all of those definitions just get included as 'literals' in the configflags: property, per part, or is there some other way? I remember seeing serguisens on an old snappy-clinic doing some things with $SNAP variables or something, but the examples were fast and I couldn't figure out how they were working

Indeed, sorry for the noise on gegl. I think regarding build variables you can define them as -D<var> directly, but you might be better off asking @sergiusens or @kyrofa on gitter or #snappy on IRC.

Yes thank you :)
Trying to finish some edits on the above post as progress report/notes to post into the gitter channel before I go off to do things afk this afternoon

Closed the request so I can keep updating the repo without blowing up the travis build history with broken crap.


RxJS 5.5.6: pipeable switchMap operator does not fire any event

Bug Report
Current Behavior
We want to upgrade to RxJS 6 and therefore migrated to the pipeable operators as described in RxJS v5.x to v6 Update Guide.
We faced twice the issue that the switchMap operator does not work correctly and does not fire an event. As a solution we had to add tap() operator before to make the switchMap operator to fire an event.
And yes, we always subscribe to the observable ;-)
Reproduction
Simple StackBlitz example with the same dependencies but there I cannot reproduce the issue:
https://stackblitz.com/edit/angular-946rlj
Code snippets from our project:
    return this.userService.user$.pipe(
      // if this tap is removed, no events are fired from the switchMap operator below
      tap(() => {
        return;
      }),
      switchMap(user => this.getZoneInformationOfUser(user.id, productRoles))
    );

Observable.forkJoin(
  this.translateService.get('Translation1'),
  this.translateService.get('Translation2')
)
  .pipe(
    // if this tap is removed, no events are fired from the switchMap operator below
    tap(() => {
      return;
    }),
    switchMap(([title, text]) => {
      const ref = this.modal.open(ConfirmDialogComponent, {
        data: {
          title,
          text
        }
      });
      return ref.afterClosed;
    })
  )
  .subscribe(result => {
    if (result === 'confirm') {
      this.confirmUpdate();
    }
  });

Expected behavior
I would expect that the switchMap operator does work as before without the pipeable operators in RxJS 5.5.6
Environment

Runtime: Chrome v68.0.3440.84
RxJS version: 5.5.6
Angular 5.2.7 without CLI

Can you provide a reproduction that does not involve your project-specific services, etc.? That is, can you provide a stand-alone reproduction?

@cartant: Sure. Unfortunately it is not reproducible in a simple standalone reproduction:
https://stackblitz.com/edit/angular-946rlj

@cartant That makes sense. So I will close this issue and we have to investigate on our side. Thanks for your time!

No worries. Hopefully, you'll be able to either figure out what's going on or create a reproduction. If you're able to reproduce the behaviour, I'll be happy to investigate it.

We did solve our issue.
The root cause for the issue was that the rxjs library was provided twice in our project. Somewhere (I suspect this line https://github.com/ReactiveX/rxjs/blob/5.5.6/src/util/toSubscriber.ts#L11) there is an instanceof check. Because the Subscriber is from the other rxjs instance the check fails and we end up in another code branch.
By removing one of the rxjs instances we where able to solve the issue.


Add note for policy ingestion task

Ingest policy script uses API for PBP. If this feature is not available, the feature will not work well.
https://github.com/treasure-data/treasure-boxes/blob/master/scenarios/monitoring/basic_monitoring/scripts/ingest_policy.py
Added comments to ~README and~ initial_ingest_policy.dig.

@o-mura @wchen4 Could you take a look at this change when you get a chance?

LGTM!

Thank you!


changes in .main-content

changed text and style of class main-content
commented text that was in main-content in index.html

resolve your conflicts first


fix: upgrade to version 3.1.345

from: https://github.com/jenkins-x/jx-cli

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

Needs approval from an approver in each of these files:

OWNERS

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

@jenkins-x-bot-test: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
f1f88373651418162d62c3c4b2193030840dc898
link
/test vault

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

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
f1f88373651418162d62c3c4b2193030840dc898
link
/test vault

gsm
f1f88373651418162d62c3c4b2193030840dc898
link
/test gsm

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

/retest

@jenkins-x-bot-test: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

vault
f1f88373651418162d62c3c4b2193030840dc898
link
/test vault

gsm
f1f88373651418162d62c3c4b2193030840dc898
link
/test gsm

gsm-kapp
f1f88373651418162d62c3c4b2193030840dc898
link
/test gsm-kapp

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


Member variables in ES6

relates to #1708
/cc @arv @caitp
TODO:

[x] generated ctor if super class - currently parseStatement `super(...arguments)`  fails,
[x] the coma expression (class expression) is not valid: let and ";",
[x] add tests

I've just added some code for class expression.
@arv do you have some time this week for a quick chat about this.
The current (invalid) transform:
// source
// Options: --member-variables --output-language=es6 --script --source-maps

class C {
  c1 = 1;
  c2;
  static c3 = 3;
}

var t = class {
  t1 = 1;
  t2;
  static t3 = 3;
}

// es6 output
class C {
  constructor() {
    this.c1 = 1;
  }
}
Object.defineProperty(C, "c3", {value: 3});
var t = (let $__0 = class {
  constructor() {
    this.t1 = 1;
  }
};, Object.defineProperty($__0, "t3", {value: 3});, "$__0";);
//# sourceMappingURL=traceured.js.map

The latest commit use an IIFE:
class C {
  constructor() {
    this.c1 = 1;
  }
}
Object.defineProperty(C, "c3", {value: 3});
var t = (function() {
  let $__0 = class {
    constructor() {
      this.t1 = 1;
    }
  };
  Object.defineProperty($__0, "t3", {value: 3});
  return $__0;
})();

I've integrated your feedback + add superclass transformation

@arv the last commit add test however I don't understand why the code is parsed as an expression, the output is:
      + expected - actual

      +(function() {
      +  let $__0 = class C {
      +    constructor() {
      +      this.c1 = 1, this.c2 = 2;
      +    }
      +  };
      +  Object.defineProperty($__0, "c0", {
      +    value: 0,
      +    enumerable: true,
      +    configurable: true,
      +    writable: true
      +  });
      +  return $__0;
      +})()
      -...

Make sure that you are creating valid trees. You can test this with options.validate = true.

The test was using parseExpression(), fixed

Travis as one error with annotations: https://travis-ci.org/google/traceur-compiler/builds/53180488#L210
I don't know how to fix this.
I have two more errors locally:
  2) SourceMapMapping columnIndexGreaterOrEqual:
     Error: timeout of 2000ms exceeded
      at [object Object].<anonymous> (/home/victor/dart/traceur-compiler/node_modules/mocha/lib/runnable.js:139:19)
      at Timer.listOnTimeout [as ontimeout] (timers.js:110:15)

  3) SourceMapMapping testNextPosition:
     Error: timeout of 2000ms exceeded
      at [object Object].<anonymous> (/home/victor/dart/traceur-compiler/node_modules/mocha/lib/runnable.js:139:19)
      at Timer.listOnTimeout [as ontimeout] (timers.js:110:15)

I think this looks good. However, isn't there a lot of code being duplicated between ClassTransformer and ES6ClassTransformer? Also, maybe we could always run ES6ClassTransformer (and rename it) before ClassTransformer? Then ClassTransformer does not need to know about member variables. Maybe that is problematic because we lose context needed for annotations?

However, isn't there a lot of code being duplicated between ClassTransformer and ES6ClassTransformer?

I thought about that but:

the code is slighty different,
the ClassTransformer is already a huge beast, adding ESx switches would make the logic hard to follow.

Also, maybe we could always run ES6ClassTransformer (and rename it) before ClassTransformer?

May be. I have to think about it.

The Travis error looks real. You probably forgot to git add the file.

Oops !

LGTM. Can you squash?


Exclude static properties and methods differently when formatting objects.

Assuming the existing intended behaviour remains desired, then this resolves #416

@davedevelopment Any comment?

I'm a :+1:. I'd really like to replace a lot of this code with sebastianbergmann/exporter et al in the not too distant future.

@davedevelopment I have no objections to that.

@nick-jones Merged and thanks :P


Switch from HashMap to BTreeMap for deterministic output

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 96.96970% with 1 line in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

src/process.rs
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines
Coverage Δ

src/compiler.rs
84.31% <100.00%> (ø)

src/runtime.rs
91.46% <100.00%> (ø)

src/process.rs
58.12% <0.00%> (ø)


Create Java Badge

Hello, there is no Java Badge, is it possible to create a Java Badge?

Unfortunately this wont be possible since this project uses Simple Icons and as mentioned in their issue: https://github.com/simple-icons/simple-icons/issues/7374 the Java logo was removed for legal reasons.
If the logo is ever added to Simple Icons this repo should update the changes automatically.


Closes #340. Update gradle-tooling for all exercises for the template.

Updating gradle tooling for all exercises.
Providing script to do it automatically.

These changes seems quite trivial and, however we can roll-out them gradually.

Seems that reference checking tests are not playing nicely with it. :weary:
Will check 'em first.

It must be prohibited by some convension of human rights to review each of 336 files in a PR :)
Let me summarize my findings on a brief view:

per excercise: Gradle build script rewritten to Kotlin DSL
per excercise: gradlew instructions updated in favour of the latest version of Gradle
clean is invoced before test running

All these changes look ok.
Didn't I miss something?

It must be prohibited by some convension of human rights to review each of 336 files in a PR :)

Oops, sorry, forgot to put appropriate "not ready" label for this PR. :sweat_smile:

According to the build failure, bin/journey-test.sh script is still expecting a test folder to contain groovy-scripted gradle.build.

I think problem is a bit deeper. Somewhere in compileStarterSourceKotlin task (kotlin stdlib not resolved there). I'm gonna check/rewrite testing tools first.


Initial pass on log severity reduction

With the updated definition on logging levels (see: #837), there are a few events that are potentially issuing logs at too high of a level, alerting the DevOps team for things that aren't actionable, or at too low of a level.
While some instances are obvious adjustments, I've opened this as a draft for feedback on the changes being made & also to ask questions on non-obvious adjustments.

Some initial questions on downgrading from warning to info:
https://github.com/python-discord/bot/blob/ee7cfbfca1b23408d7cb3f603498347fcef00c86/bot/cogs/sync/cog.py#L37-L44
I'm assuming this is still something we want to be aware of, in case the race condition becomes a recurring issue?
https://github.com/python-discord/bot/blob/ee7cfbfca1b23408d7cb3f603498347fcef00c86/bot/cogs/sync/syncers.py#L133-L138
Since I believe this is being done as part of an interactive session with a staff member, does it need to separately emit a warning level log?
https://github.com/python-discord/bot/blob/ee7cfbfca1b23408d7cb3f603498347fcef00c86/bot/utils/messages.py#L94-L95
I'm assuming this is fine as an info log. It appears that we'd only get to this condition from the antispam cog, where we're intentionally skipping the upload to prevent duplicating the spam.

I'm assuming this is still something we want to be aware of, in case the race condition becomes a recurring issue?

I don't quite get this warning. A 404 would mean the user is not found in the database. The only way I can see that happening is if a user is edited the moment they join the server, and the PATCH for the edit goes through before the user is POSTed to the database. If that's the case, then, if left as a warning, it would remind us to come up with a better solution. On the other hand, it may be safe to ignore since a future sync will eventually update the user, though I'm not sure what implications being temporarily out of sync has for other features which rely on user info from the database. my opinion is to leave it a warning.

Since I believe this is being done as part of an interactive session with a staff member, does it need to separately emit a warning level log?

It could either be from a manual invocation of the sync command or from an automatic sync when the bot starts. Semantically it makes sense to be a warning, but it's not something we'd want to be notified about in Sentry. Of most concern would be when the confirmation of an automatic sync times out; manual invocations could be a lower log level.

I'm assuming this is fine as an info log. It appears that we'd only get to this condition from the antispam cog, where we're intentionally skipping the upload to prevent duplicating the spam.

This is also used by Big Brother. Semantically, I think a warning makes sense but it's not something we need to be notified of in Sentry since there's simply nothing we can do about a file being too large.

I suppose it all has to be an info level to avoid sentry bugging us?

I'm not particularly concerned about Sentry being verbose as long as it's telling us about things that require action. In the case of the syncer reaction loop, staff has already been pinged in the same context that the warning would be issued, so unless I'm missing something this is essentially pinging twice.

This is also used by Big Brother.
I meant the else clause specifically. From what I can see, other than the antispam call of this method, all other current calls to this method utilize the default link_large = True. which is handled by the previous elif condition.

so unless I'm missing something this is essentially pinging twice.

Yes, it would. For that reason, it should be moved down to info. If it weren't for Sentry then I'd have kept this as a warning.

In the case of the antispam call, reaching the else clause is intentional and not something that we need to be reminded of.

OK, makes sense. I'm fine with an info for it then.


Code Quality: Introduce unit tests

📄 Description
Add a tests project to FluentHub to test features automatically
This change will require research about integration tests and knowledge on how to implement them properly.
⚡ Required changes

Add unit tests

✅ Tasks
📋 Checklist:

[ ] Search about integration tests in WinUI
[ ] Add a project for integration tests
[ ] Run the project locally
[ ] Add a task to build and run the project in CI

Unit tests are much easier for class libraries. You wanna implement UI tests as well like in Files?

No I mean unit tests for class libraries such as the core.

No I mean unit tests for class libraries such as the core.

Oh! Let's do this in #486


[Bug]: wrong title

Description
for gallery page we can change the title name events to gallery
Expected Behavior
we can change the title
Show us the trouble with screenshots

Which browsers are affected?
Chrome
Which OS are affected?
Windows
Enter the version of your web browser
Version 115.0.5790.102 (Official Build) (64-bit)
Device?
Desktop
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

please assign me this issue @Vaishnavi-Patil2211

Hey there @khadar1020,
I have already created an Issue #1164 for this problem.
Please close your issue and pr, because it duplicate
Hope you understood, Thank you!

Additionally you are not permitted to open a PR without even getting assigned to this ISSUE.
Make sure to follow contributing guidelines, otherwise you want face problems in future from GSSOC Team.


Automate creating html from tsv file

Task: #3
When merged, this PR will:

Add parser for TSV files generating HTML strings from them

Many changes have been introduced thanks to Alex' comments and pair coding, please re-review if anything has been missed.
I'm changing the PR status from draft to normal as it's a working solution (althouth still with its flows and compromises)

For now it seems that idea with TsvParser is dead since we will get the data from API in a form of json, not tsv. Useful stuff will be cheery picked from this branch into 22-create-json-parser


feat: make Login and Groceries modules lazy loaded

@tjvantoll , this updates the routing configuration (app/app.routing.ts). Should the tutorial be updated?

Hey @sis0k0,
The tutorials do not use the master branch, so these changes won’t break anything. I try to keep this repo showing an example that’s as real-world as possible, and also shows our best practices.
With that in mind, is lazy loading something we want to recommend as the default approach? My only concern here is this adds a little extra complexity that might be hard for newer users to follow, but I think that’s fine if we feel there’s a big benefit.

Hey @tjvantoll,
I think it will be nice to recommend lazy loading as the default approach since it improves the start-up time of the app. In my opinion, we should also promote that in our tutorial.
@hdeshev, @valentinstoychev what do you think about that?

I like lazy loading too, but I don't think it's "getting started" material. I suggest we merge this to master (since it isn't a part of the tutorial), and leave the tutorial as is. IMHO lazy loading should be documented separately.

Ok, let's merge these into master and leave it that way :)


Rotating around point

Hi there
I enjoyed reading your tutorial, as the help for the scripting found online is very sparse. Anyway, I was wondering how I can rotate an object around a defined point (x/y). Until so far I only managed to rotate it with the given parameters:
Transformation:
Transformation.DOCUMENTORIGIN
Transformation.TOPLEFT
Transformation.LEFT
Transformation.BOTTOMLEFT
Transformation.TOP
Transformation.CENTER
Transformation.BOTTOM
Transformation.TOPRIGHT
Transformation.RIGHT
Transformation.BOTTOMRIGHT

Any help on that?
Thank you in advance.

No, there is not.
But a workaround would be to use DOCUMENTORIGIN.

Move the object with the point you want to rotate to the document origin
Rotate it around the document origin: groups[j].rotate(60 * j, true, false, false, false, Transformation.DOCUMENTORIGIN);

You can have a look at my script https://github.com/ThibaultJanBeyer/kaleidoscope-pattern-now.js where I did exactly this!
Make sure to show me your script when it is finished.

Hm. That sounds promising as well. I found another work around with some math:
I have this function which lets me calculate the x and y position of an object rotating with a specific radius around a center point.
function calculatePosition(angleInDegrees, radius, centerPoint){

    // Convert from degrees to radians via multiplication by PI/180
    var x = (radius * Math.cos(angleInDegrees * Math.PI / 180)) + centerPoint.x;
    var y = (radius * Math.sin(angleInDegrees * Math.PI / 180)) + centerPoint.y;

    return {x: x, y: y};
}

And yes you are truly right about the pain in the ass. I had to open the script everytime in illustrator to see what happens and I don't have a logging function. And somehow the extendToolKit seems to crash when I run the code, but in Illustrator it's all good...


Implement Java Web Start interface to allow application be executed via the network.

Implement Java Web Start interface to allow application be executed via the network.
See https://en.wikipedia.org/wiki/Java_Web_Start

See also https://github.com/Wbondar/harsh-crystal/issues/4


Doc Feedback: Unnecessary character in API Visibility title

Location : https://apim.docs.wso2.com/en/latest/GettingStarted/key-concepts/

Closing since it's fixed.


[PORT]Use Number.Epsilon for equal check in expression

Fixes #3266
Description
Use Number.EPSILON for equal check in expression

Pull Request Test Coverage Report for Build 536429007

0 of 0   changed or added relevant lines in 0 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.07%) to 84.69%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

libraries/adaptive-expressions/src/functionUtils.internal.ts
5
96.22%

Totals

Change from base Build 535919117:
0.07%

Covered Lines:
18216

Relevant Lines:
20522

💛  - Coveralls


python3Packages.pysmappee: init at 0.2.17

Motivation for this change
Python Library for the Smappee dev API
https://github.com/smappee/pysmappee
This is a Home Assistant dependency.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Depends on #112854 for Python 3.9.

Result of nixpkgs-review pr 114155 at 46f1ea9d run on aarch64-linux 1

1 package marked as broken and skipped:

python39Packages.pysmappee

2 packages built:

python37Packages.pysmappee
python38Packages.pysmappee


haskell package sets: add profiled versions

I think this is a much cleaner end-user interface

@peti

It may be cleaner for the special case of providing profiled libraries, but this interface is way less powerful than the overriding one. For example, how do we cope with people who want to build Haskell packages without running the test suite? Without shared libraries? Without object splitting? The list of build options goes on and on, and so far overriding mkDerivation is the only mechanism that can solve all these use cases.
There is an ongoing discussion about improvements of the user interface at http://lists.science.uu.nl/pipermail/nix-dev/2015-October/018379.html, and the subject of providing special attributes has come up. I like the idea, basically, but if we go for that approach then I want a solution that solves more than just the profiling case.

@peti sounds good, thanks for the pointer to that discussion. I'll close this in favor of an eventual more unified solution


HTML5 build auto disconnect when call socket_send

Hi there,
I have cloned this and run the example.
(To build HTML5, I removed codegen folder. and no change anything)
When build and run HTML5, I got Disconnected when call socket_send
nakama.socket_send(socket, channel_join_message)

You can see log here: https://anotepad.com/note/read/94xi4wqj
Could you please take a look for this?

We have replaced the old Lua based WebSocket implementation from the initial release of Nakama-Defold with a new native WebSocket extension (see #20). This issue is resolved through the upgrade to the native WebSocket solution.

We have replaced the old Lua based WebSocket implementation from the initial release of Nakama-Defold with a new native WebSocket extension (see #20). This issue is resolved through the upgrade to the native WebSocket solution.


Error when using formControlName instead of ngModel

Btw, is there support for formControl ?

Yes.

How can I use formControlName instead of ngModel? Can you guys put and code sample in the wiki? I say that because ngModel will be removed in Angular 7


Only space owner can create a child area.

partial fix for #https://github.com/almighty/almighty-core/issues/1299
tests updated/ added as needed.

[test]

Codecov Report

Merging #1361 into master will increase coverage by 0.04%.
The diff coverage is 81.81%.

@@            Coverage Diff             @@
##           master    #1361      +/-   ##
==========================================
+ Coverage   59.91%   59.96%   +0.04%
==========================================
  Files         119      119
  Lines       10855    10865      +10
==========================================
+ Hits         6504     6515      +11
  Misses       3713     3713
+ Partials      638      637       -1

Impacted Files
Coverage Δ

controller/area.go
88.05% <81.81%> (-0.55%)
:arrow_down:

controller/workitem.go
82.54% <0%> (+0.7%)
:arrow_up:

Continue to review full report at Codecov.

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


Add Port Redirect Functionality

This takes the guesswork out of finding ports, particularly important for users trying to connect to client-based servers.
The server will only advertise over 19132 but will not accept game connections to it (rather reporting that the server is full).
This will rewrite the port in the options, allowing the user to easily connect to the server. This also allows the safe assumption of port 19132 always being the default, as using that we can then find the correct port, simplifying end-user configuration.
Reference: https://wiki.vg/Raknet_Protocol#Unconnected_Pong
Output: Connecting to <IP_ADDRESS>:45388 BlahBlah (My World), version <IP_ADDRESS>  (as 1.19.30)

Thanks!


[Enhancement Test] support range join predicate

Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[x] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.2
[ ] 3.1
[ ] 3.0
[ ] 2.5

[FE Incremental Coverage Report]
:disappointed: fail : 90 / 117 (76.92%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/SessionVariable.java
2
4
50.00%
[1073, 1074]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/transformation/DeriveRangeJoinPredicateRule.java
77
102
75.49%
[56, 57, 75, 86, 93, 94, 95, 113, 128, 138, 155, 161, 168, 175, 176, 177, 178, 179, 185, 186, 187, 188, 189, 192, 197]

:large_blue_circle:
com/starrocks/sql/optimizer/Optimizer.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/statistics/ColumnStatistic.java
9
9
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/rule/RuleType.java
1
1
100.00%
[]

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)


fix: accommodate slow provider readiness

Hi @DanielSinclair, are you planning to merge this soon ? I'm also facing similar issue due to slow injector. If this can be merged sooner. It will be very helpful

@punithbm Which wallet are you seeing this issue with?

@punithbm Which wallet are you seeing this issue with?

Frontier Wallet https://chrome.google.com/webstore/detail/frontier-wallet/kppfdiipphfccemcignhifpjkapfbihd


make-demo-unique appends instead of replacing

The make-demo-unique appends the project name instead of replacing it, causing the project name to become longer each time the command is run.
For example, here's the command that I'm currently running:
ansible-playbook ci-playbook.yaml -i inventory/ -e demo_projectname=ryan-test -e scm_ref=HEAD

This results in a project called labs-ci-cd-ryan-test. I'd expect the project to be called ryan-test.
I encountered an unrelated error related to postgresql. I ran the same playbook again, causing my project name to become ryan-test-ryan-test. In params/jenkins/deploy:
NAMESPACE=labs-ci-cd-ryan-test-ryan-test

I'd expect to see this:
NAMESPACE=ryan-test

Running the playbook a few times will cause the project name to exceed the 63-character limit, causing further runs to fail. A short-term workaround is to reset the playbook to its initial state by running git checkout params. (You will lose any config changes that you have made.)

the way we have this written with regex, its a bit of a challenge to do it right. I'm going to be exposing the regexp / replace pairs for end users to modify without changing the role. so
I think the shorter answer is simply have the playbook git checkout params as a clean up step and include a warning to users that they need to save local changes first

@rdebeasi ok - I've gotten the regex to the point where it is idempotent, so the playbook can be run many times. it still does a post fix instead of a replace, but this is just the nature of the beast as far as I see it. PR shortly


No ObservationSpec in the init.py file

Description
Error
File [~/NERCST/nercst/pointings/fivepoint.py:14](https://file+.vscode-resource.vscode-cdn.net/Users/r.yamada/NERCST/playground/~/NERCST/nercst/pointings/fivepoint.py:14)
     11 from ..utils import figureof, stamp, text
     13 # from neclib import get_logger
---> 14 from neclib.coordinates import ObservationSpec
     15 from scipy.optimize import OptimizeWarning, curve_fit
     17 from ..core import io, chopper_wheel, baselinefitting

ImportError: cannot import name 'ObservationSpec' from 'neclib.coordinates' ([/Users/r.yamada/NERCST/.venv/lib/python3.9/site-packages/neclib/coordinates/__init__.py](https://file+.vscode-resource.vscode-cdn.net/Users/r.yamada/NERCST/.venv/lib/python3.9/site-packages/neclib/coordinates/__init__.py)

Expected behavior
from neclib.coordinates import ObservationSpec

@r-yamada1998
Could you clarify the intention?
If you expect ObservationSpec to auto determine observation types, it's not implemented yet, so please use the dedicated spec parsers instead.
If you want to use ObservationSpec in type annotations, it seems valid to make the import alias.

Closing this issue, the reason is the same as #450.


How to differentiate between header text vs paragraph text?

Hi! I absolutely love this project.  Quick question though.
After processing a document and printing the result, is there a way to see what is header text vs what is just a paragraph?
Right now all text, including headers, are being printed with the paragraph text.
Thanks!

This should be easy to do with small updates to the code. If you look at the code, text from header, main document and footer are appended to a single 'text' variable. You can collect these text in different variables and print as per your need.


[BUG] changeCaptureIntervalSeconds does nothing for SqlServerStream streams

Description
normally, the changeCaptureIntervalSeconds defines the polling interval for our streams.
This is especially interesting for SqlServerStream streams, since they are used to full load tables from the source.
Right now the changeCaptureIntervalSeconds will just be ignored, causing Arcane to reload the tables as fast as possible.
Steps to reproduce the issue

create a stream with SqlServerStream type
set high value for changeCaptureIntervalSeconds
Start stream, and watch it go

Describe the results you expected
We should be able to configure how often SqlServerStream streams will try to load tables from the source
System information
No response

@george-zubrienko @s-vitaliy As discussed for the CRM streams

Discussed with @jeppe742 that I will fix this bug and test it with unit tests. We will try to run integration tests when the stream is actually needed.


chore(main): release 0.13.1

:robot: I have created a release beep boop
0.13.1 (2024-07-11)
Bug Fixes

lambda Builder creating resources for wrong parent (a7d94e9)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.13.1
:sunflower:


Ways to 'freeze' browser ?

Hi,
It would be nice to add browser.freeze() / browse.unfreeze()  methods, to completely freeze the actions
Eg:
page.on('pageerror', (e) => {
browser.freeze()
log(e)
// take screenshots
// do bunch of api calls
// more ...
browser.unfreeze()
})

await page.click('.clickthis') // all good
await page.click('.clickthisthrowanerror') // pageerror event handler will freeze the browser
await page.click('.clickthis') // will wait until the browser is unfreezed to execute

Typically we would need to lock this._connection.send() in Connection.js when the browser is 'freezed' to avoid more actions to be done in the 'frozen' state.
If this is something that you would think it's worth to add but you maintainers are too busy with other tasks, I'll be happy to have a try on working on this.

Typically the second await page.click('.clickthis') will wait (or timeout) whenever browser.unfreeze() is called to execute.
My scenario is as follow:

I have a set of actions,  If an action throws an error I want to freeze the browser (= not do next actions) to do necessary api checks

page.on('pageerror', (e) => {
browser.freeze() Calling this method will lock action 4
log(e)
await myApiCheck();
await someFailureChecks();
browser.unfreeze() // calling this method will unlock Action 4
})

await page.click('.clickthis') // Action 1
await page.click('.clickthis') // Action 2
await page.click('.clickthisthrowanerror') // Action 3
await page.click('.clickthis') // Action 4

I hope my use case is clear ^^

Instead of browser.freeze, it looks like you'd want await page.click to throw an error if clicking caused an error on the page.
It isn't that pretty, but you can replace your async function with a generator, and all await with yield. Then repeatedly await generator.next().value, but stop if an error occurs in between.

Ah didn't think of generators, that would probably fulfill my use case.


Failed to load API definition @hapi/joi 16

i am upgrade to Joi 16 but get this error message when i load swagger ui:

Environment
ProductName:	Mac OS X
ProductVersion:	10.14.6
BuildVersion:	18G95

Binaries:
Node: 10.16.3 - /usr/local/opt/node@10/bin/node
Yarn: 1.17.3 - /usr/local/bin/yarn
npm: 6.11.3 - /usr/local/bin/npm

npm Packages:
"@hapi/hapi": "^18.4.0",
"@hapi/hoek": "^8.2.4",
"@hapi/inert": "^5.2.2",
"@hapi/joi": "^16.1.2",
"hapi-swagger": "^10.1.2",

Error trace
Debug: handler, error
    TypeError: Cannot read property 'properties' of undefined
    at Object.parameters.fromProperties (/Users/project/node_modules/hapi-swagger/lib/parameters.js:50:17)
    at module.exports.internals.paths.internals.paths.getSwaggerStructures (/Users/project/node_modules/hapi-swagger/lib/paths.js:420:32)
    at routes.forEach.route (/Users/project/node_modules/hapi-swagger/lib/paths.js:194:32)
    at Array.forEach (<anonymous>)
    at module.exports.internals.paths.internals.paths.buildRoutes (/Users/project/node_modules/hapi-swagger/lib/paths.js:161:10)

Joi v16 is currently not supported. Please see https://github.com/glennjones/hapi-swagger/issues/614


known limitations of graph view

I'm loving this plugin. I'm planning to use it for a big academic project, and the graph view is very important for me in that context (the graph view of this plugin is way better than the other plugins or obsidian to HTML programs I've seen before). So, I'd like to know if there are any known limitations on the graph view feature of this plugin. like, how many nodes is too much or how many edges is too much, or any other varibles.

It should be able to handle a lot of notes, but it will use CPU more heavily than obsidian's graph. I would recommend testing it if you are worried, but I have tested it with about 1000 notes with no problem. To be honest I could not find any value larger than that to test it on so I was unable to do any further testing.

on my machine "generating HTML" process works fine, it's the "saving" process that stops. and the stopping line is somewhere between 15k to 17k notes, as per my heuristic tests.
Here I post some info if that could be of any help:
$uname -rmsv
Linux 5.15.0-73-generic #80-Ubuntu SMP Mon May 15 15:18:26 UTC 2023 x86_64

$lscpu
Architecture:            x86_64
  CPU op-mode(s):        32-bit, 64-bit
  Address sizes:         39 bits physical, 48 bits virtual
  Byte Order:            Little Endian
CPU(s):                  8
  On-line CPU(s) list:   0-7
Vendor ID:               GenuineIntel
  Model name:            Intel(R) Core(TM) i5-8265U CPU @ 1.60GHz
    CPU family:          6
    Model:               142
    Thread(s) per core:  2
    Core(s) per socket:  4
    Socket(s):           1
    Stepping:            12
    CPU max MHz:         3900.0000
    CPU min MHz:         400.0000
    BogoMIPS:            3600.00
    Flags:               fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mc
                         a cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss
                         ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc art
                          arch_perfmon pebs bts rep_good nopl xtopology nonstop_
                         tsc cpuid aperfmperf pni pclmulqdq dtes64 monitor ds_cp
                         l vmx est tm2 ssse3 sdbg fma cx16 xtpr pdcm pcid sse4_1
                          sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsav
                         e avx f16c rdrand lahf_lm abm 3dnowprefetch cpuid_fault
                          epb invpcid_single ssbd ibrs ibpb stibp ibrs_enhanced
                         tpr_shadow vnmi flexpriority ept vpid ept_ad fsgsbase t
                         sc_adjust bmi1 avx2 smep bmi2 erms invpcid mpx rdseed a
                         dx smap clflushopt intel_pt xsaveopt xsavec xgetbv1 xsa
                         ves dtherm ida arat pln pts hwp hwp_notify hwp_act_wind
                         ow hwp_epp md_clear flush_l1d arch_capabilities
Virtualization features:
  Virtualization:        VT-x
Caches (sum of all):
  L1d:                   128 KiB (4 instances)
  L1i:                   128 KiB (4 instances)
  L2:                    1 MiB (4 instances)
  L3:                    6 MiB (1 instance)
NUMA:
  NUMA node(s):          1
  NUMA node0 CPU(s):     0-7
Vulnerabilities:
  Itlb multihit:         KVM: Mitigation: VMX disabled
  L1tf:                  Not affected
  Mds:                   Not affected
  Meltdown:              Not affected
  Mmio stale data:       Mitigation; Clear CPU buffers; SMT vulnerable
  Retbleed:              Mitigation; Enhanced IBRS
  Spec store bypass:     Mitigation; Speculative Store Bypass disabled via prctl
                          and seccomp
  Spectre v1:            Mitigation; usercopy/swapgs barriers and __user pointer
                          sanitization
  Spectre v2:            Mitigation; Enhanced IBRS, IBPB conditional, RSB fillin
                         g, PBRSB-eIBRS SW sequence
  Srbds:                 Mitigation; Microcode
  Tsx async abort:       Not affected

$sudo dmidecode -t memory
[sudo] password for xinyaanzyoy:
# dmidecode 3.3
Getting SMBIOS data from sysfs.
SMBIOS 3.0.1 present.

Handle 0x0017, DMI type 16, 23 bytes
Physical Memory Array
	Location: System Board Or Motherboard
	Use: System Memory
	Error Correction Type: None
	Maximum Capacity: 16 GB
	Error Information Handle: No Error
	Number Of Devices: 1

Handle 0x0018, DMI type 17, 40 bytes
Memory Device
	Array Handle: 0x0017
	Error Information Handle: No Error
	Total Width: 64 bits
	Data Width: 64 bits
	Size: 8 GB
	Form Factor: SODIMM
	Set: None
	Locator: Bottom
	Bank Locator: BANK 2
	Type: DDR4
	Type Detail: Synchronous
	Speed: 2667 MT/s
	Manufacturer: Samsung
	Serial Number: 334AE5AD
	Asset Tag:<PHONE_NUMBER>
	Part Number: M471A1K43CB1-CTD
	Rank: 1
	Configured Memory Speed: 2400 MT/s
	Minimum Voltage: 1.35 V
	Maximum Voltage: 1.5 V
	Configured Voltage: 1.2 V

$sudo dmidecode -t processor
# dmidecode 3.3
Getting SMBIOS data from sysfs.
SMBIOS 3.0.1 present.

Handle 0x0004, DMI type 4, 48 bytes
Processor Information
	Socket Designation: U3E1
	Type: Central Processor
	Family: Core i5
	Manufacturer: Intel(R) Corporation
	ID: EC 06 08 00 FF FB EB BF
	Signature: Type 0, Family 6, Model 142, Stepping 12
	Flags:
		FPU (Floating-point unit on-chip)
		VME (Virtual mode extension)
		DE (Debugging extension)
		PSE (Page size extension)
		TSC (Time stamp counter)
		MSR (Model specific registers)
		PAE (Physical address extension)
		MCE (Machine check exception)
		CX8 (CMPXCHG8 instruction supported)
		APIC (On-chip APIC hardware supported)
		SEP (Fast system call)
		MTRR (Memory type range registers)
		PGE (Page global enable)
		MCA (Machine check architecture)
		CMOV (Conditional move instruction supported)
		PAT (Page attribute table)
		PSE-36 (36-bit page size extension)
		CLFSH (CLFLUSH instruction supported)
		DS (Debug store)
		ACPI (ACPI supported)
		MMX (MMX technology supported)
		FXSR (FXSAVE and FXSTOR instructions supported)
		SSE (Streaming SIMD extensions)
		SSE2 (Streaming SIMD extensions 2)
		SS (Self-snoop)
		HTT (Multi-threading)
		TM (Thermal monitor supported)
		PBE (Pending break enabled)
	Version: Intel(R) Core(TM) i5-8265U CPU @ 1.60GHz
	Voltage: 0.7 V
	External Clock: 100 MHz
	Max Speed: 8300 MHz
	Current Speed: 1485 MHz
	Status: Populated, Enabled
	Upgrade: Socket BGA1528
	L1 Cache Handle: 0x0005
	L2 Cache Handle: 0x0006
	L3 Cache Handle: 0x0007
	Serial Number: To Be Filled By O.E.M.
	Asset Tag: To Be Filled By O.E.M.
	Part Number: To Be Filled By O.E.M.
	Core Count: 4
	Core Enabled: 4
	Thread Count: 8
	Characteristics:
		64-bit capable
		Multi-Core
		Hardware Thread
		Execute Protection
		Enhanced Virtualization
		Power/Performance Control

on my machine "generating HTML" process works fine, it's the "saving" process that stops. and the stopping line is somewhere between 15k to 17k notes, as per my heuristic tests.
Here I post some info if that could be of any help:
$uname -rmsv
Linux 5.15.0-73-generic #80-Ubuntu SMP Mon May 15 15:18:26 UTC 2023 x86_64

$lscpu
Architecture:            x86_64
  CPU op-mode(s):        32-bit, 64-bit
  Address sizes:         39 bits physical, 48 bits virtual
  Byte Order:            Little Endian
CPU(s):                  8
  On-line CPU(s) list:   0-7
Vendor ID:               GenuineIntel
  Model name:            Intel(R) Core(TM) i5-8265U CPU @ 1.60GHz
    CPU family:          6
    Model:               142
    Thread(s) per core:  2
    Core(s) per socket:  4
    Socket(s):           1
    Stepping:            12
    CPU max MHz:         3900.0000
    CPU min MHz:         400.0000
    BogoMIPS:            3600.00
    Flags:               fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mc
                         a cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss
                         ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc art
                          arch_perfmon pebs bts rep_good nopl xtopology nonstop_
                         tsc cpuid aperfmperf pni pclmulqdq dtes64 monitor ds_cp
                         l vmx est tm2 ssse3 sdbg fma cx16 xtpr pdcm pcid sse4_1
                          sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsav
                         e avx f16c rdrand lahf_lm abm 3dnowprefetch cpuid_fault
                          epb invpcid_single ssbd ibrs ibpb stibp ibrs_enhanced
                         tpr_shadow vnmi flexpriority ept vpid ept_ad fsgsbase t
                         sc_adjust bmi1 avx2 smep bmi2 erms invpcid mpx rdseed a
                         dx smap clflushopt intel_pt xsaveopt xsavec xgetbv1 xsa
                         ves dtherm ida arat pln pts hwp hwp_notify hwp_act_wind
                         ow hwp_epp md_clear flush_l1d arch_capabilities
Virtualization features:
  Virtualization:        VT-x
Caches (sum of all):
  L1d:                   128 KiB (4 instances)
  L1i:                   128 KiB (4 instances)
  L2:                    1 MiB (4 instances)
  L3:                    6 MiB (1 instance)
NUMA:
  NUMA node(s):          1
  NUMA node0 CPU(s):     0-7
Vulnerabilities:
  Itlb multihit:         KVM: Mitigation: VMX disabled
  L1tf:                  Not affected
  Mds:                   Not affected
  Meltdown:              Not affected
  Mmio stale data:       Mitigation; Clear CPU buffers; SMT vulnerable
  Retbleed:              Mitigation; Enhanced IBRS
  Spec store bypass:     Mitigation; Speculative Store Bypass disabled via prctl
                          and seccomp
  Spectre v1:            Mitigation; usercopy/swapgs barriers and __user pointer
                          sanitization
  Spectre v2:            Mitigation; Enhanced IBRS, IBPB conditional, RSB fillin
                         g, PBRSB-eIBRS SW sequence
  Srbds:                 Mitigation; Microcode
  Tsx async abort:       Not affected

$sudo dmidecode -t memory
[sudo] password for xinyaanzyoy:
# dmidecode 3.3
Getting SMBIOS data from sysfs.
SMBIOS 3.0.1 present.

Handle 0x0017, DMI type 16, 23 bytes
Physical Memory Array
	Location: System Board Or Motherboard
	Use: System Memory
	Error Correction Type: None
	Maximum Capacity: 16 GB
	Error Information Handle: No Error
	Number Of Devices: 1

Handle 0x0018, DMI type 17, 40 bytes
Memory Device
	Array Handle: 0x0017
	Error Information Handle: No Error
	Total Width: 64 bits
	Data Width: 64 bits
	Size: 8 GB
	Form Factor: SODIMM
	Set: None
	Locator: Bottom
	Bank Locator: BANK 2
	Type: DDR4
	Type Detail: Synchronous
	Speed: 2667 MT/s
	Manufacturer: Samsung
	Serial Number: 334AE5AD
	Asset Tag:<PHONE_NUMBER>
	Part Number: M471A1K43CB1-CTD
	Rank: 1
	Configured Memory Speed: 2400 MT/s
	Minimum Voltage: 1.35 V
	Maximum Voltage: 1.5 V
	Configured Voltage: 1.2 V

$sudo dmidecode -t processor
# dmidecode 3.3
Getting SMBIOS data from sysfs.
SMBIOS 3.0.1 present.

Handle 0x0004, DMI type 4, 48 bytes
Processor Information
	Socket Designation: U3E1
	Type: Central Processor
	Family: Core i5
	Manufacturer: Intel(R) Corporation
	ID: EC 06 08 00 FF FB EB BF
	Signature: Type 0, Family 6, Model 142, Stepping 12
	Flags:
		FPU (Floating-point unit on-chip)
		VME (Virtual mode extension)
		DE (Debugging extension)
		PSE (Page size extension)
		TSC (Time stamp counter)
		MSR (Model specific registers)
		PAE (Physical address extension)
		MCE (Machine check exception)
		CX8 (CMPXCHG8 instruction supported)
		APIC (On-chip APIC hardware supported)
		SEP (Fast system call)
		MTRR (Memory type range registers)
		PGE (Page global enable)
		MCA (Machine check architecture)
		CMOV (Conditional move instruction supported)
		PAT (Page attribute table)
		PSE-36 (36-bit page size extension)
		CLFSH (CLFLUSH instruction supported)
		DS (Debug store)
		ACPI (ACPI supported)
		MMX (MMX technology supported)
		FXSR (FXSAVE and FXSTOR instructions supported)
		SSE (Streaming SIMD extensions)
		SSE2 (Streaming SIMD extensions 2)
		SS (Self-snoop)
		HTT (Multi-threading)
		TM (Thermal monitor supported)
		PBE (Pending break enabled)
	Version: Intel(R) Core(TM) i5-8265U CPU @ 1.60GHz
	Voltage: 0.7 V
	External Clock: 100 MHz
	Max Speed: 8300 MHz
	Current Speed: 1485 MHz
	Status: Populated, Enabled
	Upgrade: Socket BGA1528
	L1 Cache Handle: 0x0005
	L2 Cache Handle: 0x0006
	L3 Cache Handle: 0x0007
	Serial Number: To Be Filled By O.E.M.
	Asset Tag: To Be Filled By O.E.M.
	Part Number: To Be Filled By O.E.M.
	Core Count: 4
	Core Enabled: 4
	Thread Count: 8
	Characteristics:
		64-bit capable
		Multi-Core
		Hardware Thread
		Execute Protection
		Enhanced Virtualization
		Power/Performance Control

Are there any errors in the developer log? (Ctrl + Shift + J)

okay, so I reran the same heuristic test (at ~19k notes), which actually worked this time! I assume then it was a resource issue (but I still doubt that for several reasons I won't go into). I'll try again at a higher order of magnitude of a number of notes.
that aside, FYI, I noticed something: the vault containing 1887 items used 10.5GB space, and the generated HTML folder containing 1860(I guess we see this number change because of .obsidian/ and lib/) items used 1.2GB!!!
btw, it was Ctrl + Shift + i for me.

Make sure you select Multi-file export from the presets. If you do not do this it will package all of the code and resources into every single html file and use a LOT of space, resources, and time! If you have already done this then I will look into this a bit more.
In chrome you can press Ctrl + Shift + J to go directly to the console (in case there was a different tab open in the dev tools), but I guess electron doesn't support that.