[OpenSearch Playground] New Domain for ML

Is your feature request related to a problem?
Generally ML models need much resources like memory/CPU. We need dedicated ML nodes in cluster to run ML models. It will be good to have a dedicated ML OpenSearch playground. So we can add dedicated ML nodes and scale easily.
domain name:  ml.playground.opensearch.org/
permission: same with current OpenSearch playground, support readonly anonymous user; better to provide some admin account to ML team, so they can ingest data/manage model/cluster etc.
storage: 100GB

@ylwu-amzn Thanks for the request, we will take a look and start work on it


Change name to area_name

Changing the attribute name in the region codelist files nuts1.yaml, nuts2.yaml and nuts3.yaml to area_name.
An attribute named name causes issues with the latest nomenclature release.
Running the tests, it seems we have another issue that will need to be fixed on the nomenclature side concerning multiple units.

Looks like there's a few gremlins left. I'll investigate further.

Looks like there's a few last issues to fix. I'm taking a look.

Looks like it's all good now. @danielhuppmann, I'll go ahead with the merge to restore the service.


demo: cannot resolve module redux-router

This is a duplicate of #91 and #111, but both of those are closed and the solution offered in #91 doesn't seem to resolve.
Running npm start throws the error below. I tried a few configurations to see if I was missing something obvious, but the bug appears to be over my head.
ERROR in ./index.js
Module not found: Error: Cannot resolve module 'redux-router' in ...
 @ ./index.js 25:19-42

I had to update from whatever version npm installed to v1.0.0-beta3. That seemed to fix this problem.

Worked perfectly - thank you!


Fix Docker-compose.yml default postgres user

The user was set as "root" with no password.
When running the tests, it stated that "postgres" user was not available.
This happened because postgres image was being built with the user "root"

Related Issues: #1971

Introduction
Fix docker-compose user to use the postgres default one, so the tests can run.
Fixes #1971.
Motivation
You should be able to run the tests with only the following commands:
$ npm i
$ docker-compose up -d
$ npm test

Proposed solution
Pretty simple, just remove the defined user, and let it fallback to the postgres image default user.
Alternatives considered
Well, maybe we could change the user being set for the tests, and set it as root. But that's adding code, whereas removing code is more elegant IMO.

This was released in version 0.15.0.


Issue726

We're making more use of the OLCStringsAsUTF8 capability marker.
Resolves #726
In the process, I discovered that the coutwildrnp.shp lacked a valid code page.

Coverage decreased (-0.6%) to 82.549% when pulling 13f2007563dcca59209049ce33218c4206df23be on issue726 into 5a675083a99e277f9d984b8d6f9ea6c386be921f on master.


Header horizontal character style customization

Hello,

I would like to create Pandoc Markdown compliant grid tables. For this I just
need to be able to change the header horizontal character to '=' while leaving
the table borders and column separators at '-'

I think this small change would be an excellent addition. Thanks very much for
your great library.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 25 Sep 2013 at 6:57

Binance Airdrop Guide: Claim Your $1500 in BNB Now!
🎉 Congratulations! You've won a share of the $1.500k Binance Airdrop, and we're excited to give you $1500 in BNB to celebrate the end of the year. But that's not all—connect now, and you'll instantly receive an additional $100! Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. 🌐 Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. 🔄 Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

💰 Don't miss out on your $1.500k in BNB! Plus, enjoy an instant $100 upon connection. Act now to secure your tokens. And hey, seize the chance to win up to $20,000—it's your golden opportunity!
Winners: @


Model enhancement bugfix

That PR fixes issues after introducing new merging algorithm.
#3014, #3243, #2345, #3211, #3082, #563.
The main issue is when we faced model with nodes (properties) that we already merged, we just skip that model instead of looking for the same models of that type.

@MaksimOrlov thank you!


Encourage form submitters to link to the recipient's own instructions

Recently I learned about Seattle Children's request for donations from a local nurse. I know they've edited that post at least two times, slightly revising what they are willing to accept and drop-off location options. It would be great if this URL was linked under Seattle Children's so everyone could verify the latest information with minimal communication overhead.
More generally, encouraging form submissions to include links like this could help keep all the information current. The field for

Drop off instructions, eg curbside procedure or mailing address ATTN: instructions:

could include something like this:

(recommended: link to their public request where information is kept updated)

By the way, Seattle Children's isn't currently accepting any kind of N95s. Their post currently says

Update: We are currently accepting basic isolation masks and surgical masks. At this time, we kindly decline any other masks. We currently do not have a need for anything other than masks (e.g. latex gloves) at this time.

They clarified for me what they mean by "basic isolation masks".

Could you submit a new entry for Seattle children's through the form that has the updated info and the link? from there we can publish and remove the older entry.
I can't change the form field titles, because it breaks our pipeline (which I learned the hard way earlier today!) If it becomes worthwhile to update this to add the things you've requested, we'll need to have a brief downtime to do so. Right now I don't think it's worth it.

I submitted a new entry for Seattle Children's and a separate one for the Bellevue clinic mentioned in the FB post. Added new instructions, link, and removed N95's.

Hello! Right now the text on the form says "Drop off instructions, eg curbside procedure or mailing address. If you want donors to email or call you, please include contact info that can be made public in this field:"
What if we changed it to: "Write drop off instructions below or paste a link to your organization's own page containing instructions. For written instructions, please include details such as curbside procedure, mailing address, email address, and/or phone number. Please note all information entered here will be made public." (Slightly copy-edited for clarity)

See the 2nd paragraph of @r-pop's reply. Unless the technical barriers have been remedied this may not be feasible.

Yes, we now have separated the human-readable label from the code-readable labels in the sheet. Big relief. I like Tina's edit and will make it on the US-en form now.

Fixed!


Simple way to disable builtin method/prop highlighting

I prefer not to highlight all the known methods and props of built-in objects. Many of them are too generic and conflict with unrelated code, creating pretty things like this:

(Not to mention that sometimes a keyword is highlighted differently based on context, as you can see with parent: in one case it is part of javascriptGlobals, in another it is a javascriptBOMWindowProp, but really it should not be highlighted at all).
I haven't found a setting to disable them. I could comment out all the /syntax/yajs/* includes, but that also removes useful things like the global functions themselves (I would like to keep Map, but not Map.get).

Perhaps a more flexible approach is to avoid linking these highlight groups directly in the syntax files, but instead cluster them into a small number of categories that are easy to link in syntax/javascript.vim or override in .vimrc? Ideally, I would like to have:

javascriptBuiltinGlobal: top-level objects and functions, e.g. Map, Promise, parseInt.
javascriptBuiltinStatic: static properties/methods of the above, e.g. Symbol.iterator, Object.create, Number.EPSILON, restricted to follow the relevant global as they do now.
javascriptBuiltinProp: instance properties and methods of the above, e.g. Array.push, Object.toString.

Then specific implementation synrax files could add their own things to these (for node.js, require goes into javascriptBuiltinGlobal and require.resolve into javascriptBuiltinStatic).
Personally, I could then comment out all syntax/yajs/* includes that are not relevant to me (I'll keep es6 and node.js, for example) and also do something like this in my own environment:
hi link javascriptBuiltinGlobal Structure
hi link javascriptBuiltinStatic Identifier
hi link javascriptBuiltinProp   Normal

I am going to refactor this part.
But not very soon.(will be very busy until end of year).
Some info for you.
https://github.com/othree/webidl-to-vim-syntax will be the base.
Not sure how it will be included now.

Awesome, for now I'll work around it.

Since the performance issue on neovim.
The webidl.vim will not come in as default.
So this will then be implement later.
But my idea is beside your suggestion.
Give an option to not include syntax/yajs/*.
I think will have a little better performance.


username error

there is no username is set on my localhost just I am on the root directory .but its shows username error :(

root directory!!! Please!!! Use MySQL database with their 'username' and 'password' :-|


chore: removes chimoney for limited walletconnect v2 support

Description
Removes ChiSpend - Super App for limited WalletConnect v2 support. Currently connecting appears to support v2, but all other dapp action requests fail.

@uchibeke Do you have plans to add support for WalletConnect v2?


[BUG ?] Holding SHIFT triggers the Plugin Command

Hello @Mom0aut !!!!
Just to let you know that if we hold the SHIFT button for 2 seconds or more, triggers the command the same way that if we hit shift 2 times.

Version 1.0.7
Windows 7
Obsidian 0.13.23

Many times I use SHIFT+Letter to capitalize a letter in for example New York or USA for example.
I don’t know if is possible to avoid this while holding SHIFT or to increase the time until triggers when holding the button.  :)
Thanks for reading this!
Have a great day!!!!!!

thank you for your report, this bug should be fixed with the new version 1.0.9

Thank you very much @Mom0aut for your reply and for the quick fix !!!!!
Working perfect on version 1.0.9 - Obsidian 0.13.23
Have a great day!!!!!


📝 updated shard key singlestore documentation link

The original link doesn't seem to work for me, so I'm suggesting to change to this link.

Ah good catch! Are the rest of the changes some sort of auto formatter? Looks like it caught a few double spaces.

It looks like it! I've got my VS Code setup to be pretty opinionated... 😅 I can revert, update the link and leave the rest of the formatting?

Yeah maybe just update the link? I dunno if it really matters, but it's probably cleaner that way!

Ok, let me know if that's better - I reverted back to the original document, updated the link and saved without executing my formatter.

Merged. Thanks @DanielFerguson!


Nutch 2465

Please verify the request carefully.  I think about possible pitfalls.

Hi @okedoki, thanks! Could you please first

remove all files from the pull request which shouldn't be there: *~, .classpath, .project, etc. If you add a new file there must be a clear reason for it.
choose meaningful commit messages which state what has been changed. It's good to reference the NUTCH-XXX issue but an additional description (e.g., "moved classes into package o.a.n.protocol.interactiveselenium.handlers following directory structure") will help to review and also makes it easier to browse the history later on
the author of a commit is tracked by git, no need to repeat it in the message

The classpath issues need to be fixed in build.xml ("target: eclipse").

Thanks for your comment.
Unfortunately, I found out some bugs with this solution. I'm closing this pull request.
Feel free to propose a (working) solution.


How to change dot bullet point with image ?

I am using version 4.6.0  in the project.
I want to replace the dot bullet point with the image in the list. I do not want to replace the number-based bullet point.
Please specify expected/actual behavior
Test number list\n1. Number1\n2. Number2\n3. Number3\n4. Number4\n\nTest bullet list\n- Bullet1\n- Bullet2\n- Bullet3\n- Bullet4\n"
So please guide me on how to do it.

@noties  please help me

Hello @chaurasiadilip ,
you can use custom span for bullet lists - ChangeBulletSpanSample. How image is used depends on your needs. But most likely you will want to look at existing LeadingMarginSpan implementations, like:

BulletListItemSpan
TaskListSpan

@noties  Thanks for guiding me. I got success for me.


test: migrate from tape to tap

alternative to #1171
Needs to be tested against CI. If we're not using tap output in CI I'd like to remove -Rtap.

@cclauss I probably should have run your Python changes through CI because there are some problems on Windows which may be coming from it? See https://ci.nodejs.org/job/nodegyp-test-commit/nodes=win2016-vs2017/678/console
15:29:52     # Subtest: build simple addon in path with non-ascii characters
15:29:52         1..1
15:29:52 Traceback (most recent call last):
15:29:52   File "fixtures/test-charmap.py", line 23, in <module>
15:29:52     print(main())
15:29:52   File "fixtures/test-charmap.py", line 19, in main
15:29:52     print(textmap[encoding])
15:29:52   File "C:\Python27\lib\encodings\cp1252.py", line 12, in encode
15:29:52     return codecs.charmap_encode(input,errors,encoding_table)
15:29:52 UnicodeEncodeError: 'charmap' codec can't encode character u'\u012b' in position 3: character maps to <undefined>
15:29:52         ok 1 - python console app can't encode non-ascii character. # SKIP
15:29:52         # skip: 1
15:29:52     ok 3 - build simple addon in path with non-ascii characters # time=67.458ms
15:29:52

Could that be coming from abef93ded which did:
-  if textmap.has_key(encoding):
-    print textmap[encoding]
+  if encoding in textmap:
+    print(textmap[encoding])

CI for this: https://ci.nodejs.org/view/All/job/nodegyp-test-pull-request/133/
Failing on Windows with the python problem above, but not failing on Node 12, I'm not sure what that's about. Some funky unicode difference that's coming down in from the test suite?
CI isn't using tap output at all for anything automatic, but it is handy for text output. So what I've done is introduce a "test-ci" script that will be run on Travis (and I'll update Jenkins to use it too), otherwise it'll do the normal prettified tap output.

Our current Travis CI runs on Linux but we can easily add Windows and/or macOS if that would be useful.

very useful, so many of our problems are OS specific here.

A bunch more minor updates to make the standard npm test output nice and clean, so turning off logging output. If you have your loglevel set to something custom it's going to make the tests noisier, but there's also some stuff in there that will log.warn and some console.log too.
There's a new deprecation in Node 12 that spits out to stderr because we're using a self-signed cert with an IP address, so there's a commit in here that replaces that with a self-signed for localhost and the warning goes away.
Test output:

npm run test-ci is the verbose nested TAP output (but without the stderr mess) plus the coverage at the bottom as well.

CI isn't using tap output at all for anything automatic, but it is handy for text output. So what I've done is introduce a "test-ci" script that will be run on Travis (and I'll update Jenkins to use it too), otherwise it'll do the normal prettified tap output.

Travis is probably better off with the prettified output. As it is it doesn't display all of it (output is too long):

For the CI the only reason I can think of to keep the tap output is if we want to use tap2junit on it (which we don't currently).
Since switching to tap would by default enable coverage we could consider enabling codecov (which we use elsewhere in the org, e.g. citgm https://codecov.io/gh/nodejs/citgm, node-core-utils https://codecov.io/gh/nodejs/node-core-utils) for this repository.

:thumbsup: you're right, it looks pretty good in Travis: https://travis-ci.com/nodejs/node-gyp/jobs/210631751

landed in 395f843

This broke Node v6. CI failed because of the python issues, and that probably caused the real failures in v6 to go unnoticed.
What should we do here? I don't think we should support unsupported Node versions at all, but for that we need to update the engines field and exclude from CI. Or should we revert this and keep supporting Node v6? It would probably be nicer to drop support on a semver-major version, though I'm not sure we strictly need to.
cc @nodejs/node-gyp

CI failed because of the python issues

Do you have a url to these failures?

https://ci.nodejs.org/job/nodegyp-test-commit/676/ and https://ci.nodejs.org/job/nodegyp-test-commit/677/
There are two failures. On CI, every run of v6.2.1 and v6.14.4 failed while running npm install:
06:31:49 npm ERR! enoent ENOENT: no such file or directory, rename '/home/iojs/build/workspace/nodegyp-test-commit/nodes/ubuntu1604-64/node_modules/.staging/@types/prop-types-5c4fdf37' -> '/home/iojs/build/workspace/nodegyp-test-commit/nodes/ubuntu1604-64/node_modules/tap/node_modules/@types/prop-types'
06:31:49 npm ERR! enoent ENOENT: no such file or directory, rename '/home/iojs/build/workspace/nodegyp-test-commit/nodes/ubuntu1604-64/node_modules/.staging/@types/prop-types-5c4fdf37' -> '/home/iojs/build/workspace/nodegyp-test-commit/nodes/ubuntu1604-64/node_modules/tap/node_modules/@types/prop-types'

Locally, if I use a newer version of npm, this does not happen. This is probably an issue with npm or the module being installed, I don't know if it is related to this PR. If we're to support Node v6 we should work around this somehow if possible.
However, after using a new npm to install, this error happens when running npm test:
C:\Users\Administrator\Desktop\node-gyp\node_modules\tap\bin\run.js:125
const main = async options => {
                   ^^^^^^^

SyntaxError: Unexpected identifier
    at createScript (vm.js:56:10)
    at Object.runInThisContext (vm.js:97:10)
    at Module._compile (module.js:549:28)
    at Object.Module._extensions..js (module.js:586:10)
    at Module.load (module.js:494:32)
    at tryModuleLoad (module.js:453:12)
    at Function.Module._load (module.js:445:3)
    at Module.runMain (module.js:611:10)
    at run (bootstrap_node.js:394:7)
    at startup (bootstrap_node.js:160:9)
npm ERR! Test failed.  See above for more details.

addressing it in #1808, but maybe we need to extend our travis matrix to include the major Node versions, probably just on Linux (I don't know which version of Python, maybe it doesn't matter).


Add DESKTOP tag for the leapp report

can be used by KDE/GNOME actor instead of UPGRADE:
https://github.com/oamg/leapp-repository/pull/310

Can one of the admins verify this patch?


Security Blog

I intend to build a Blog website using React, which I'm currently in the process of learning. Also I'm specializing in cyber security, hence the content of the blog will be my research articles written in the process of my preparation for CEH certification.

I intend to build a Blog website using React, which I'm currently in the process of learning. Also I'm specializing in cyber security, hence the content of the blog will be my research articles written in the process of my preparation for CEH certification.

Sorry for this but we are not accepting React Projects


API - access roles

Allow roles to be configured for every API endpoint. If the developer does not define a role then the system defaults are applied automatically.

Done in commit 12cf6c699176f343ef77aae1fdbdc951bcb13643


Add Revved up by Develocity badge

This pull request adds a badge to the project's README to indicate that it uses the Develocity instance hosted at https://ge.spring.io/.
Other Spring projects, such as Spring Boot and Spring Framework, already have the badge present in their README.

Thanks for the contribution @erichaagdev ! Much appreciated.


Create and publish a docker image

It would be great if we could have a docker image of this project published on docker hub to easily integrate it to our pipeline.

Also, the possibility to throw the logs to other databases would be great. My specific scenario is that I want to push postgres logs to an immutable storage (like immudb).


Add more precise telemetry for signature verification

We should add more fine-grained telemetry/statuses so we can more precisely diagnose failure reasons and solutions.
Assigning to February so we ideally look into this next week.
Owner: @dtivel

This has been done. Closing as completed.


updated DVM integration

LGTM. Are you sure that jenkins jobs will have the same env as jenkins JVM process?

Yes I've spun it up locally and been able to use docker from within a jenkins job.

LGTM ... can we tag for a release as well


floating action button position in rtl direction

in rtl direction, floating action button Placed on right of page while it should be palaced on left of page. please fix this problem.

I think it's better to be optional and developer decides it goes to left or right side of the app.

Fixed in v2

Floating button got hidden in Android
I am using floating action button in page.
But its not working in android device.
When scroll the page sometimes only shows the button.
Device : Lenovo model:: Vibe k5
OS: Android 5.1.1- lollipop

Other device screens


lua: Patch CVE-2022-28805

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[ ] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[ ] Ready to merge

Summary

Patch CVE-2022-28805 in the lua package. Patch taken from upstream.
Change Log

lua: Patch CVE-2022-28805

Does this affect the toolchain?

YES
Links to CVEs

https://nvd.nist.gov/vuln/detail/CVE-2022-28805

Test Methodology

Build ongoing

Verified locally the package builds. Merging PR to be ready for the branch cut.


185298829 Add Simulator Tile

PT Stories:
https://www.pivotaltracker.com/story/show/185298829
https://www.pivotaltracker.com/story/show/185298834
https://www.pivotaltracker.com/story/show/185298864
This PR features the first round of work for the Simulator Tile. It adds a basic implementation of the tile, hooks it up to a document's sharedVariables, sets up, updates, and displays a couple of example built-in variables, gives the tile a title, and sets up an example curriculum with a simulation tile that can be dragged to a student's working document.

@bacalj I just hadn't merged that work yet. The simulator tile is now using it :)


README.md: fix typo

I think you meant 'Luckily' not "Hopefully".

Thank you ! "Luckily" you figured out what i meant !
Thanks
Arnaud


Adding a quick note partially disables notion-enhancer

What is happening?
After I press the Quick Note Button on the bottom of the page a new item is created for the database that I just added. This results in multiple unwanted behaviour of the window:

The Theme breaks
The Notion Enhanced Button disappears
Multiple minor tweaks are not working anymore (See screenshot, i.e. Help button again, Share as Text etc.)

What should be happening?
IMO The Page should open in a new Tab and this tab should be immediately visible. Without any noticeable difference in appearance
Screenshots (if applicable):
Before:

After:

Operating system/browser name & version:
Arch Linux 5.15.12-arch1-1
used https://aur.archlinux.org/packages/notion-app-enhanced/ to install the app
notion-enhancer and/or notion-repackaged version:
Notion 2.0.18-1 (notion-enhancer 0.11.0)
notion-enhancer configuration (enabled/disabled mods and their settings):
{
  "_mods": {
    "a5658d03-21c6-4088-bade-fa4780459133": true,
    "e1692c29-475e-437b-b7ff-3eee872e1a42": true,
    "87e077cc-5402-451c-ac70-27cc4ae65546": true,
    "0a958f5a-17c5-48b5-8713-16190cae1959": true,
    "35815b3b-3916-4dc6-8769-c9c2448f8b57": true,
    "e0700ce3-a9ae-45f5-92e5-610ded0e348d": true,
    "d61dc8a7-b195-465b-935f-53eea9efe74e": true,
    "1794c0bd-7b96-46ad-aa0b-fc4bd76fc7fb": true,
    "74856af4-6970-455d-bd86-0a385a402dd1": true,
    "5174a483-c88d-4bf8-a95f-35cd330b76e2": true,
    "d64ad391-1494-4112-80ae-0a3b6f4b0c3f": true,
    "5d7c1677-6f6d-4fb5-8d6f-5067bcd0e24c": true
  },
  "e1692c29-475e-437b-b7ff-3eee872e1a42": {
    "remember_last_open": false,
    "last_open_tabs_cache": [
      {
        "notionUrl": "https://www.notion.so/quick-note-1-2-2022-1-37-04-AM-575fe3ffe8dc4bd381582bf69d4d76bf",
        "icon": "url(\"notion://www.notion.so/images/emoji/twitter-emoji-spritesheet-64.d3a69865.png\") 18.6441% 94.9153% / 6000% 6000%",
        "title": "Pfadi"
      }
    ]
  },
  "e0700ce3-a9ae-45f5-92e5-610ded0e348d": {
    "share": true
  },
  "36a2ffc9-27ff-480e-84a7-c7700a7d232d": {
    "panel.width": 240
  },
  "5174a483-c88d-4bf8-a95f-35cd330b76e2": {
    "tweak.hide_help": true,
    "tweak.hide_slash_for_commands": true,
    "tweak.spaced_lines": true,
    "tweak.condensed_bullets": true
  },
  "74856af4-6970-455d-bd86-0a385a402dd1": {
    "topbar_copy_text": false
  },
  "5d7c1677-6f6d-4fb5-8d6f-5067bcd0e24c": {
    "hotkey": "Control+Shift+Q",
    "target_db": "9937c82569454bf487030ca5e2923ad5"
  }
}

further feedback
First I wanted to say that I very much appreciate the work that went into this project and the result is overall very impressive to me :). I do have some feedback that could get incorporated into the application. As a Software Developer myself, through tinkering I was able to determine the id of the database through the following way:

Copy link to block of database.
Analyze url and test the two values. Figured the first big value is the correct one.

I don't thin this process is in any way intuitive. I think there should be some sort of Description that'll guide you through the process.
If more information is needed HMU!

I have this same bug, although Notion goes back to normal after closing it.
If there's more than one window open it only affects the one that created the quick note (at least with my config). I'm using that in the meantime to create notes without breaking the whole thing (maybe knowing this is useful to someone with the same issue).

I believe I figured out the issue, on line 25 of the quick note mod we're doing a location.assign to the page of the newly created note.
The issue arises from the fact that the webview "refreshes" rather than using Notion view's React Router (or whatever kind of frontend routing mechanism they're using), so the styles/scripts that Notion Enhancer loads aren't loaded at that time.
Not familiar at all with the project (first time looking at its code) but I think that a fix would be to re-inject styles/scripts (if that's even possible), hope that's useful somehow @dragonwocky :)


Fix pruning mask in function invert

As discussed in issue #32, the computation of the pruning mask seems to be not correct. As an effect, the first occurrence of a non-linear variable was neither renamed nor pruned.
This PR closes this issue by proceeding as proposed by @AndrasKovacs: We first compute the partial renaming, returning a set of non-linear variables and the forced spine. A second function mask then creates the pruning mask where all occurrences of a non-linear variable are pruned.
The changes affect package 05-pruning and 06-first-class-poly, where only the function invert has been rewritten to:
invert :: Lvl -> Spine -> IO (PartialRenaming, Maybe Pruning)
invert gamma sp = do
  let go :: Spine -> IO (Lvl, IS.IntSet, IM.IntMap Lvl, IS.IntSet, Spine)
      go []                                        = pure (0, mempty, mempty, mempty, [])
      go (sp :> (force -> ft@(VVar (Lvl x)), i)) = do
        (dom, domvars, ren, nlvars, fsp) <- go sp
        case IS.member x domvars of
          True  -> pure (dom + 1, domvars,             IM.delete x ren,     IS.insert x nlvars, fsp :> (ft, i))
          False -> pure (dom + 1, IS.insert x domvars, IM.insert x dom ren, nlvars,             fsp :> (ft, i))
      go _                                         = throwIO UnifyError

  (dom, domvars, ren, nlvars, fsp) <- go sp

  let mask :: Spine -> Pruning
      mask []                         = []
      mask (fsp :> (VVar (Lvl x), i))
        | IS.member x nlvars          = Nothing : mask fsp
        | otherwise                   = Just i : mask fsp
      mask _                          = impossible

  pure (PRen Nothing dom gamma ren, mask fsp <$ guard (not $ IS.null nlvars))

Thanks! Comments:

I think the domvars is now unnecessary to build because we can check for nonlinearity by looking up from the partial renaming instead.
In mask, it would be better to get rid of the impossible case by takine [Lvl] as argument instead of Spine.

Thanks for your feedback! I updated the PR accordingly.


fix: use fetchApi instead of explicit identityApi token in remaining plugins

Hey, I just made a Pull Request!
fixes: https://github.com/backstage/backstage/issues/23015#issuecomment-1950966308

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

I wonder why CatalogImportClient and KubernetesBackendClient  were exported. Are they supposed to be used outside of their plugins? 🤔
@vinzscam  no idea, this is how I found them.


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_4eicbtu2bz/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_4eicbtu2bz/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_4eicbtu2bz/clone/recipe/meta.yaml:38: invalid_url: https://github.com/Azure/azure-sdk-for-python/tree/main/sdk/table/azure-table : Not reachable: 404
ERROR: /tmp/abs_4eicbtu2bz/clone/recipe/meta.yaml:42: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_4eicbtu2bz/clone/recipe/meta.yaml:42: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_4eicbtu2bz/clone/recipe/meta.yaml:42: missing_license_family: The recipe is missing the about/license_family key.
WARNING: /tmp/abs_4eicbtu2bz/clone/recipe/meta.yaml:42: missing_description: The recipe is missing a description
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_4caj5brcli/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_4caj5brcli/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_4caj5brcli/clone/recipe/meta.yaml:38: invalid_url: https://github.com/Azure/azure-sdk-for-python/tree/main/sdk/table/azure-table : Not reachable: 404
ERROR: /tmp/abs_4caj5brcli/clone/recipe/meta.yaml:42: missing_license_family: The recipe is missing the about/license_family key.
WARNING: /tmp/abs_4caj5brcli/clone/recipe/meta.yaml:42: missing_description: The recipe is missing a description
ERROR: /tmp/abs_4caj5brcli/clone/recipe/meta.yaml:42: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_4caj5brcli/clone/recipe/meta.yaml:42: missing_dev_url: The recipe is missing a dev_url
Errors were found


change dev app name #133

We need to update Dev app name before making Prod app
I could push the code to Dev app manually.

Codecov Report

Merging #134 (c560f08) into main (c42b4ac) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #134   +/-   ##
=======================================
  Coverage   95.89%   95.89%
=======================================
  Files           5        5
  Lines         268      268
=======================================
  Hits          257      257
  Misses         11       11

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 c42b4ac...c560f08. Read the comment docs.

IBM Cloud toolchain: Delivery Pipeline deployed promotions to prod, including commit c7fcc1fbc3cba13ba0ff89d30f0149dfa78f6c13

IBM Cloud toolchain: Delivery Pipeline deployed promotions to dev, including commit c7fcc1fbc3cba13ba0ff89d30f0149dfa78f6c13


Entry doesn't belong in directory "" (contains subdir) - ignoring

Hello,
I am newby with rclone but I test with many Object Storage and I have the same error when I do a ls or a copy :

Entry doesn't belong in directory "" (contains subdir) - ignoring

I use rclone version 1.37 on Debian 8 Amd64 with swift OVH Object Storage :
# rclone ls -vv test-rclone:/
2017/07/27 10:04:50 DEBUG : rclone: Version "v1.37" starting with parameters ["rclone" "ls" "-vv" "test-rclone:/"]
2017/07/27 10:04:51 INFO  : Swift container : Modify window is 1ns
2017/07/27 10:04:51 ERROR : Apache/Apache.pdf: Entry doesn't belong in directory "" (contains subdir) - ignoring
2017/07/27 10:04:51 ERROR : Apache/Formation_Apache.odp: Entry doesn't belong in directory "" (contains subdir) - ignoring
2017/07/27 10:04:51 ERROR : Apache/Formation_Apache.pdf: Entry doesn't belong in directory "" (contains subdir) - ignoring
2017/07/27 10:04:51 ERROR : Apache/htaccess.pdf: Entry doesn't belong in directory "" (contains subdir) - ignoring
2017/07/27 10:04:51 DEBUG : Go routines at exit 4
2017/07/27 10:04:51 DEBUG : rclone: Version "v1.37" finishing with parameters ["rclone" "ls" "-vv" "test-rclone:/"]

remote config :
--------------------
[test-rclone]
type = swift
user = *******
key = **********
auth = https://auth.cloud.ovh.net/v2.0
domain =
tenant = **************
tenant_domain =
region = GRA1
storage_url = https://storage.gra1.cloud.ovh.net/v1/AUTH_****************/test-rclone
auth_version =
--------------------

I done a mistake somewhere ?

I done a mistake somewhere ?

Take the container off the end of the storage_url and add it to the path you give rclone - I think that is confusing rclone.
So
storage_url = https://storage.gra1.cloud.ovh.net/v1/AUTH_****************/

And
rclone ls test-rclone:test-rclone

Why are you specifying the storage URL BTW? You should specify the auth_url instead

Got same issue with S3 storage and rclone version 1.37
rclone.conf:
[Storage-Name]
type = s3
env_auth = false
access_key_id = id
secret_access_key = key
region = us-east-1
endpoint =
location_constraint =
acl = private
server_side_encryption =
storage_class =
command example:
./rclone copy --delete-before --copy-links Storage-Name:bucket-name/dirnamel/ /home/user/dest-dir/

@ogerasymenko please make a new issue on github, and follow the instructions there by trying the latest beta and including a log with -vv - thanks!

I'm going to close this issue as I think the original question is answered.


Add refund attribute

I propose to add the attribute is_refund to the class ContractResultItem.
This would be useful to differentiate eGLD transfers from gas refund when inspecting the smart contract results.
I looked into the previous PRs of this repo and concluded that I should not change the package version for this change. Please correct me if I'm wrong.
For the context, I'm developping a devops package for MultiversX contract and this feature would be extremely useful. (At the moment, I see no way of differentiating refunds from egld transfers)

Hello @EtienneWallet ! The suggestion looks good, we've approved it. The only issue is that we require signed commits in order to merge the PR. To be faster I've reproduced your PR here. I have also increased the package version so when the PR is merged a new release will be out.

Thanks @popenta ! This will be very useful 😁
My commits will be signed next time 😉


Register extension factory registries.

Description:
Create a mechanism for registering extension factory registries in
yet another registry. This registry maps a category name to an
extension factory registry so that for each category, we can generate
a list of registered factory names.
Factory registries have to opt in (once) to this mechanism, in order
to support the use of factory registries that are not user-visible
at either build time or run time.
Subsequently, we can use this new factory to emit the named of
registered extension factories to the startup log and to the config
dump endpoint.
Risk Level: Low
Testing: Bazel test
Docs Changes: N/A
Release Notes: N/A
Fixes: #8070

/cc @moderation

On Nov 5, 2019, at 4:41 AM, Matt Klein<EMAIL_ADDRESS>wrote:
@mattklein123 requested changes on this pull request.

The way I had been imagining this to work is:
• Every factory type has some type of built-in name (so built into Server::Configuration::AccessLogInstanceFactory)
• During the constructor/registration of an individual factory, it would register itself into the factory-wide map.
I think if we do that we don't need any extra registration code like this? WDYT?

The reason I didn’t try to make this approach work is because I believed that there were things using registries that wouldn’t really be regarded as extensions (from the perspective of it being an optional feature). Though it seems to me now that changes in a factory registry could also be seen as fine-grained version information, so I guess it’s works just publishing it all :)

Yup +1 let's just auto publish all.

@jpeach friendly request to never force push. It makes reviews much more difficult. Please only merge master and add commits. Thank you!

Very cool and thanks @jpeach. This will help us trim down the base for Contour :smile:


Quagga/FRR Migration failed.

Describe the bug
Upgrading of templates for quagga/frr fails since atleast 20.1. I got this error on the major upgrade and every other upgrade of the package. Even on reinstallation.
Keep version OPNsense\Quagga\General (1.0.1)
Keep version OPNsense\Quagga\OSPF (1.0.4)
*** OPNsense\Quagga\OSPF6 Migration failed, check log for details
*** OPNsense\Quagga\RIP Migration failed, check log for details
*** OPNsense\Quagga\BGP Migration failed, check log for details
Reloading plugin configuration
Configuring system logging...done.
Reloading template OPNsense/Quagga: OK

To Reproduce
Steps to reproduce the behavior:

Upgrade or Uninstall/Install FRR Routing Package

Expected behavior
Templates upgrade and reload.
Relevant log files

`2020-09-06T23:10:29
syslog-ng[98957]
Configuration reload finished;

2020-09-06T23:10:29
syslog-ng[98957]
Configuration reload request received, reloading configuration;

2020-09-06T23:10:27
config[30163]
#2 {main} )

2020-09-06T23:10:27
config[30163]
#1 /usr/local/opnsense/mvc/script/run_migrations.php(57): OPNsense\Base\BaseModel->runMigrations()

2020-09-06T23:10:27
config[30163]
#0 /usr/local/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php(678): OPNsense\Base\BaseModel->serializeToConfig()

2020-09-06T23:10:27
config[30163]
Stack trace:

2020-09-06T23:10:27
config[30163]
in /usr/local/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php:567

2020-09-06T23:10:27
config[30163]
Model OPNsense\Quagga\BGP can't be saved, skip ( Phalcon\Validation\Exception: [OPNsense\Quagga\BGP:asnumber] invalid integer value

2020-09-06T23:10:27
config[30163]
[OPNsense\Quagga\BGP:asnumber] invalid integer value

2020-09-06T23:10:26
config[30163]
#2 {main} )

2020-09-06T23:10:26
config[30163]
#1 /usr/local/opnsense/mvc/script/run_migrations.php(57): OPNsense\Base\BaseModel->runMigrations()

2020-09-06T23:10:26
config[30163]
#0 /usr/local/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php(678): OPNsense\Base\BaseModel->serializeToConfig()

2020-09-06T23:10:26
config[30163]
Stack trace:

2020-09-06T23:10:26
config[30163]
in /usr/local/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php:567

2020-09-06T23:10:26
config[30163]
Model OPNsense\Quagga\RIP can't be saved, skip ( Phalcon\Validation\Exception: [OPNsense\Quagga\RIP:networks] list validation error

2020-09-06T23:10:26
config[30163]
[OPNsense\Quagga\RIP:networks] list validation error

2020-09-06T23:10:26
config[30163]
#2 {main} )

2020-09-06T23:10:26
config[30163]
#1 /usr/local/opnsense/mvc/script/run_migrations.php(57): OPNsense\Base\BaseModel->runMigrations()

2020-09-06T23:10:26
config[30163]
#0 /usr/local/opnsense/mvc/app/models/OPNsen

`

Environment
Software version used and hardware type if relevant.
e.g.:
OPNsense 21.1.a_69-amd64FreeBSD 12.1-RELEASE-p8-HBSDLibreSSL 3.1.4
Intel® Core™ i3-5020U 2.2GHz
Network Intel® I211-AT

I get that when installing the FRR plugin in opnsense 20.7.2.
FreeBSD psfwl01.lan 12.1-RELEASE-p7-HBSD FreeBSD 12.1-RELEASE-p7-HBSD #0  427d53bc125(stable/20.7)-dirty: Sun Jul 26 05:51:42 CEST 2020     root@sensey64:/usr/obj/usr/src/amd64.amd64/sys/SMP  amd64

CPU: Intel(R) Atom(TM) CPU  E3845  @ 1.91GHz (1916.71-MHz K8-class CPU)
real memory  =<PHONE_NUMBER> (4096 MB)
avail memory =<PHONE_NUMBER> (3779 MB)
FreeBSD/SMP: Multiprocessor System Detected: 4 CPUs

Still fails with 21.1.3 - Development Branch.

@fabianfrz Can you take care of this? It looks like the default settings are invalid, hence the migration will fail on new installs.

Maybe this helps:
2021-03-22T16:29:57 | configd.py[43133] | [f4248bc3-d943-4e5c-8808-29173ea1ec07] FRR diagnostics "show ipv6 ospf6 route json" |
-- | -- | -- | --
2021-03-22T16:29:57 | configd.py[43133] | [0b1adeb3-f19b-47e0-9de9-a863bd110341] Script action failed with Command '/usr/local/opnsense/scripts/frr/legacy-diagnostics.py --ospfv3-database' returned non-zero exit status 1. at Traceback (most recent call last): File "/usr/local/opnsense/service/modules/processhandler.py", line 479, in execute stdout=output_stream, stderr=error_stream) File "/usr/local/lib/python3.7/subprocess.py", line 363, in check_call raise CalledProcessError(retcode, cmd) subprocess.CalledProcessError: Command '/usr/local/opnsense/scripts/frr/legacy-diagnostics.py --ospfv3-database' returned non-zero exit status 1. |
2021-03-22T16:29:56 | configd.py[43133] | [0b1adeb3-f19b-47e0-9de9-a863bd110341] FRR diagnostics "show ipv6 ospf6 database json" |
2021-03-22T16:29:56 | configd.py[43133] | [5a5911e4-9641-4e47-934b-7e235a2caeca] request quagga
After reinstall:
2021-03-22T16:32:52 | config[36247] | #2 {main} ) |
-- | -- | -- | --
2021-03-22T16:32:52 | config[36247] | #1 /usr/local/opnsense/mvc/script/run_migrations.php(55): OPNsense\Base\BaseModel->runMigrations() |
2021-03-22T16:32:52 | config[36247] | #0 /usr/local/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php(686): OPNsense\Base\BaseModel->serializeToConfig() |
2021-03-22T16:32:52 | config[36247] | Stack trace: |
2021-03-22T16:32:52 | config[36247] | in /usr/local/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php:575 |
2021-03-22T16:32:52 | config[36247] | Model OPNsense\Quagga\BGP can't be saved, skip ( Phalcon\Validation\Exception: [OPNsense\Quagga\BGP:asnumber] invalid integer value |
2021-03-22T16:32:52 | config[36247] | [OPNsense\Quagga\BGP:asnumber] invalid integer value |
2021-03-22T16:32:52 | config[36247] | #2 {main} ) |
2021-03-22T16:32:52 | config[36247] | #1 /usr/local/opnsense/mvc/script/run_migrations.php(55): OPNsense\Base\BaseModel->runMigrations() |
2021-03-22T16:32:52 | config[36247] | #0 /usr/local/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php(686): OPNsense\Base\BaseModel->serializeToConfig() |
2021-03-22T16:32:52 | config[36247] | Stack trace: |
2021-03-22T16:32:52 | config[36247] | in /usr/local/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php:575 |
2021-03-22T16:32:52 | config[36247] | Model OPNsense\Quagga\RIP can't be saved, skip ( Phalcon\Validation\Exception: [OPNsense\Quagga\RIP:networks] list validation error |
2021-03-22T16:32:52 | config[36247] | [OPNsense\Quagga\RIP:networks] list validation error |
2021-03-22T16:32:52 | config[36247] | #2 {main} ) |
2021-03-22T16:32:52 | config[36247] | #1 /usr/local/opnsense/mvc/script/run_migrations.php(55): OPNsense\Base\BaseModel->runMigrations() |
2021-03-22T16:32:52 | config[36247] | #0 /usr/local/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php(686): OPNsense\Base\BaseModel->serializeToConfig() |
2021-03-22T16:32:52 | config[36247] | Stack trace: |
2021-03-22T16:32:52 | config[36247] | in /usr/local/opnsense/mvc/app/models/OPNsense/Base/BaseModel.php:575 |
2021-03-22T16:32:52 | config[36247] | Model OPNsense\Quagga\OSPF6 can't be saved, skip ( Phalcon\Validation\Exception: [OPNsense\Quagga\OSPF6:routerid] text validation error |
2021-03-22T16:32:52 | config[36247] | [OPNsense\Quagga\OSPF6:routerid] text validation error

The issue i still present in 21.1.6

New packages to be INSTALLED:

frr7: 7.4_6
libssh: 0.9.5
libyang: 1.0.184
net-snmp: 5.9_3,1
os-frr: 1.21_3
rtrlib: 0.6.3

Number of packages to be installed: 6

The process will require 30 MiB more space.
[1/6] Installing libssh-0.9.5...
[1/6] Extracting libssh-0.9.5: .......... done
[2/6] Installing libyang-1.0.184...
[2/6] Extracting libyang-1.0.184: .......... done
[3/6] Installing rtrlib-0.6.3...
[3/6] Extracting rtrlib-0.6.3: .......... done
[4/6] Installing net-snmp-5.9_3,1...
[4/6] Extracting net-snmp-5.9_3,1: .......... done
[5/6] Installing frr7-7.4_6...
===> Creating groups.
Creating group 'frr' with gid '168'.
Creating group 'frrvty' with gid '152'.
===> Creating users
Creating user 'frr' with uid '168'.
Adding user 'frr' to group 'frrvty'.
[5/6] Extracting frr7-7.4_6: .......... done
[6/6] Installing os-frr-1.21_3...
[6/6] Extracting os-frr-1.21_3: .......... done
Stopping configd...done
Starting configd.
Migrated OPNsense\Quagga\General from 0.0.0 to 1.0.2
Migrated OPNsense\Quagga\OSPF from 0.0.0 to 1.0.4
*** OPNsense\Quagga\OSPF6 Migration failed, check log for details
*** OPNsense\Quagga\RIP Migration failed, check log for details
*** OPNsense\Quagga\BGP Migration failed, check log for details
Reloading plugin configuration
Configuring system logging...done.
Reloading template OPNsense/Quagga: OK
Reloading template OPNsense/Syslog: OK

There is not a real problem, quite sure there are some fields marked as required but dont have a default

I think this is fixed, as I havent seen any migration errors anymore.

https://github.com/opnsense/plugins/pull/2761


Ugrade CLI parameter handling

Use https://github.com/CLIUtils/CLI11 instead of the current ict/command.

Nah, let's leave it for now. CLI11 takes too long to compile.


Hide donation button when user has no balance

Idea

Remove the option to donate when user clicks send if the user has a balance below the minimum donation amount

Closing due to wrongful termination from the Litecoin Foundation.


C# NativeAOT export fails to load resources with ResourceLoader.LoadThreadedRequest() or any async resource loading, but runs fine in non-NativeAOT exports

Tested versions

Reproductible in 4.2.0.stable.mono

System information
Godot v4.2.stable.mono - Windows 11 - Vulkan (Mobile) - dedicated NVIDIA GeForce RTX 3050 Laptop GPU (NVIDIA; <IP_ADDRESS>04) - 11th Gen Intel(R) Core(TM) i5-11320H @ 3.20GHz (8 Threads)
Issue description
When launching the game from Godot, everything works fine, it also works with normal export (without NativeAOT), but for some reason the ResourceLoader.LoadThreadedGet(scenePath) will hang the application (if called from non-async function, if the function is async then it will just wait forever). I've also tried this code:
PackedScene? scene = null;
await Task.Run(() =>
{
    scene = ResourceLoader.Load<PackedScene>(scenePath);
});

which works just fine when launching from Godot or non-NativeAOT deployment, but for some reason will not work with NativeAOT. I have to mention that calling ResourceLoader.Load() works ok, but it just seems weird that async code won't work correctly. Is there a problem with async/await in NativeAOT?
I know that NativeAOT needs special configuration in Godot, but I already did that:
<PublishAOT>true</PublishAOT>
...
<ItemGroup>
  <TrimmerRootAssembly Include="GodotSharp" />
  <TrimmerRootAssembly Include="$(TargetName)" />
</ItemGroup>

Also, there are a lot of warnings after the AOT build finishes (further detailed here), but from my understanding it shouldn't be such a big problem with it.
Steps to reproduce
See MRP below. The code will fail if the export is NativeAOT. To enable NativeAOT, follow this Godot blog post.
Minimal reproduction project (MRP)
private async Task<PackedScene?> GetSceneDataAsync(string scenePath)
{
    PackedScene? scene = null;
    await Task.Run(() =>
    {
        LoadScene(scenePath, out scene);
    });
    GD.Print("Scene loaded!");

    return scene;
}

private void LoadScene(string scenePath, out PackedScene? scene)
{
    GD.Print("Started loading scene...");
    scene = null;
    ResourceLoader.LoadThreadedRequest(scenePath);

    ResourceLoader.ThreadLoadStatus status = ResourceLoader.LoadThreadedGetStatus(scenePath);
    if (status == ResourceLoader.ThreadLoadStatus.Failed)
    {
        GD.Print("Loading failed.");
        throw new System.Exception();
    }
    else if (status == ResourceLoader.ThreadLoadStatus.InvalidResource)
    {
        GD.Print("Loading failed.");
        throw new System.Exception();
    }
    else
    {
        GD.Print("Getting scene...");
        scene = (PackedScene)ResourceLoader.LoadThreadedGet(scenePath);
        GD.Print("Scene obtained.");
    }
}

Here, "Getting scene" will be written (sometimes also "Scene obtained"), but "Scene loaded" never will.

Duplicate of https://github.com/godotengine/godot/issues/87596


Analyzer should warn when a constructor has unused parameters

The following code gives no warnings:
class Foo {
  Foo(bool test);
}
void main() {
  new Foo(true);
}

I think this is a dupe of https://github.com/dart-lang/sdk/issues/18946


Receive the following errors when trying to include

HTTPRequest.hpp: In member function 'http::Response http::Request::send(const string&, const string&, const std::vector<std::__cxx11::basic_string >&)':
HTTPRequest.hpp:398:72: error: 'getaddrinfo' was not declared in this scope
if (getaddrinfo(domain.c_str(), port.c_str(), &hints, &info) != 0)
^
HTTPRequest.hpp:401:49: error: 'freeaddrinfo' was not declared in this scope
std::unique_ptr<addrinfo, decltype(&freeaddrinfo)> addressInfo(info, freeaddrinfo);
^~~~~~~~~~~~
HTTPRequest.hpp:401:62: error: template argument 2 is invalid
std::unique_ptr<addrinfo, decltype(&freeaddrinfo)> addressInfo(info, freeaddrinfo);
^
HTTPRequest.hpp:401:94: error: expression list treated as compound expression in initializer [-fpermissive]
std::unique_ptr<addrinfo, decltype(&freeaddrinfo)> addressInfo(info, freeaddrinfo);
^
HTTPRequest.hpp:406:46: error: base operand of '->' is not a pointer
if (::connect(socket, addressInfo->ai_addr, static_cast<socklen_t>(addressInfo->ai_addrlen)) < 0)
^~
HTTPRequest.hpp:406:91: error: base operand of '->' is not a pointer
if (::connect(socket, addressInfo->ai_addr, static_cast<socklen_t>(addressInfo->ai_addrlen)) < 0)

Same error over here.

HTTPRequest.hpp: In member function 'http::Response http::Request::send(const string&, const string&, const std::vector<std::__cxx11::basic_string<char> >&)':
HTTPRequest.hpp:398:72: error: 'getaddrinfo' was not declared in this scope
             if (getaddrinfo(domain.c_str(), port.c_str(), &hints, &info) != 0)
                                                                        ^
HTTPRequest.hpp:401:49: error: 'freeaddrinfo' was not declared in this scope
             std::unique_ptr<addrinfo, decltype(&freeaddrinfo)> addressInfo(info, freeaddrinfo);
                                                 ^~~~~~~~~~~~
HTTPRequest.hpp:401:62: error: template argument 2 is invalid
             std::unique_ptr<addrinfo, decltype(&freeaddrinfo)> addressInfo(info, freeaddrinfo);
                                                              ^
HTTPRequest.hpp:401:94: error: expression list treated as compound expression in initializer [-fpermissive]
             std::unique_ptr<addrinfo, decltype(&freeaddrinfo)> addressInfo(info, freeaddrinfo);
                                                                                ^
HTTPRequest.hpp:406:46: error: base operand of '->' is not a pointer
             if (::connect(socket, addressInfo->ai_addr, static_cast<socklen_t>(addressInfo->ai_addrlen)) < 0)
                                              ^~
HTTPRequest.hpp:406:91: error: base operand of '->' is not a pointer
             if (::connect(socket, addressInfo->ai_addr, static_cast<socklen_t>(addressInfo->ai_addrlen)) < 0)
                                                                                ^~

What compiler/OS/WindowsSDK/C++ version you use?

@elnormous I'm using GCC, C++11.

I can not reproduce this with any version of gcc. What version of gcc are you using? What operating system?

GCC Version is gcc version 6.3.0, Operating System Windows.
Compiled : g++ main.cpp

Are you using cygwin on mingw?

Ok, this is a problem with MinGW. Fixed it in https://github.com/elnormous/HTTPRequest/commit/14b3beb633f879253e063452dd4fe723d59e1210


Remove eviction warnings

The reality is that eviction warnings do not help. The problem is not that they might point to a problem but that, in practice, every project takes advantage of evictions to solve the dependency problem, so evictions are expected in a project of a certain size. Any real problem will be dominated by a multitude of false positives.
And false positives are in there by design, because

eviction is the main feature of the dependency resolution engine
compatibility rules are different for different projects (so sbt falls back on the most pessimistic interpretation of "every two distinct versions might be incompatible")
even if rules like semver exist, the opposite of "guaranteed backward compatibility" is not "incompatibility" but "maybe a potential probability of incompatibility in particular parts but most likely everything is fine"

The intention might have been good, but the reality makes those useless and annoying.

My first thought was to reduce the level of eviction warnings to debug but debug logging is almost useless for most users because it is so verbose. If they were reduced to info, which sort of makes sense, it doesn't help anything since the default logging level is info.
This reveals a broader issue that I have with the sbt logging system. In my opinion, the tagged logging is a bit overused. Right now almost all of the sbt logging that is displayed to the user is at the info level with the [info] tag. This makes the sbt output very verbose by default for many tasks. Often the majority of the output isn't useful. Running at the warn level though actually breaks the ux of certain features like ~, where the instructions are logged at the info level (which was a change I made because at the time of the change, using println could cause the instructions to be printed before the previous task had finished logging). If you run sbt -warn, you also wont see any output at all until the eviction warning is printed and the shell prompt appears.
This may be a complete nonstarter for legacy reasons, but the change I would make if it were solely up to me is that I would change logger.out and logger.err to just print to stdout and stderr without adding a tag. There are a number of messages that are logged at the info level, like the welcome banner and the ~ instructions that I would change from logger.info to logger.out. I would also change the default log level warn. I think that would make the overall ux much more pleasant for the majority of use cases while still allowing users to turn on info logging if they need more information. Basically I think that if there are no warnings or errors, a user shouldn't see any tagged output. This would make the sbt output more similar to what you see from other tools like gradle and mill which are generally significantly less verbose than sbt.
I know this isn't entirely germane to the actual issue but this has been kicking in my head for a little while but didn't feel quite worthy of making a standalone issue.

IMO laissez-faire JAR-swapping is one of the largest threat to JVM ecosystem. What's the point of carefully crafting type system, if we're all going to swap it out with a mystery JAR right when we're about to deploy the service? I'd say precisely because the evictions are unavoidable once the classpath becomes large, we should highlight the potential problematic evictions that human may miss.
I'd be the first to admit that the current implementation leaves much to be desired, but given that many projects fit into certain mold, I personally think it's not impossible to reduce the false positives for the majority of the users. Instead of removing seat belts from the cars, I think we need to improve on the safety by making it detect the dangers at better precision.

The more I think about it, it's kind of a damned if you do, damned if you don't situation. Right now, the eviction warnings feel like a broken check engine light. It is impossible to tell from the warning whether or not there is actually a problem so it feels annoying rather than helpful. I am not sure that I believe this, but I could make an argument that the warnings are actually harmful because the high false positive rate cause it to be ignored. Let's say a user's build is working great but there are some harmless evictions and then the user adds a new library that causes an eviction that actually breaks the application. Because they were being warned before and the application was working, they probably won't even think to check if there is a problem with the new library. That's why I find the false positives problematic.
On the other hand, you make a great point about the type system. You're actually kind of lucky though if the reason for the eviction is type related. The problem that you see would then be likely to be a runtime crash. What is more insidious is a library that uses semantic versioning but introduces a regression in an implementation in a newer version. I don't know how any dependency management system could guard against that so that is an argument for displaying eviction warnings.
Given the high rate of false positives, maybe this can be made to be more like the unused setting lint rules? I'm thinking of something like expanding EvictionWarningOptions to allows the user to whitelist certain artifacts from warnings? What is frustrating about the current state of affairs is that there is no way to turn off harmless warnings besides disabling the warning entirely which could then mask a real problem when a new library is introduced.

Hi! Just chanced upon this discussion and will add my 2c

I am not sure that I believe this, but I could make an argument that the warnings are actually harmful because the high false positive rate cause it to be ignored

That's exactly what happened in my case. I thought this was such a great idea when it first came out. Now I have no faith in it at all to provide me with any value and I just want to disable it to stop it spamming me. I'm always in the sbt shell; it's my primary interface and to me personally, useless log lines has a high impact and I really strive to remove them. When the evicted feature first came out, I used to run evicted and look at what it's reporting but everything it reported was a completely acceptable (and desirable) eviction. Now I've come to subconsciously just treat this feature as noise.

JAR-swapping is one of the largest threats to JVM ecosystem

Personally I have this problem no more frequently than once every 5-10 years.

Instead of removing seat belts from the cars, I think we need to improve on the safety by making it detect the dangers at better precision

I agree with the attitude, it would be great if this 1) became practically useful, and 2) stopped annoying users, but we're not there yet. It's a good argument for wanting to spend time in the future to improve the feature, but it's a not a good argument for keeping the status quo, knowing that it's having a primarily negative impact (at least IMO). So if your seatbelts don't work and you're hoping to fix them in a year's time, you should remove the seatbelts until they're fixed. If you don't, we drivers will and we probably won't put them back in our metaphoric cars once seatbelts v2 arrive because we've learned to just write-off the potential usefulness of the feature.

Here's my proposal to remove eviction warning in 1.5.0, and replace it with something better, which instead of warning it would actually fail the update task, but only do so when it's 100% sure, and remove the guessing completely:
https://eed3si9n.com/enforcing-semver-with-sbt-strict-update


Update Vault to 0.11

https://www.hashicorp.com/blog/vault-0-11

@bonifaido Please hold on for Vault 0.11.1. There is a problem with policy provisioning which is fixed and expected to be released in 0.11.1 and it is not available yet.
https://github.com/hashicorp/vault/issues/5219

Hi @gowthamsubbu, it was updated to Vault 0.11 already (https://github.com/banzaicloud/bank-vaults/releases/tag/0.2.19), but you can always set the Vault image tag in each of our projects (operator, helm chart)


How to install Multiwallet and use

Hi i am new to golang can any one provide me steps to run this.
Can i access it through rpc or rest api?

@HafeezRai at the moment, multiwallet RPC is not functioning. There are a few commands you can run, but you'll need to setup your go environment to build it first. You can follow these instructions first to setup go: https://golang.org/doc/install
Then, you should be able execute the following commands:
go get -u github.com/OpenBazaar/multiwallet
cd $GOPATH/src/github.com/OpenBazaar/multiwallet

go run cmd/multiwallet/main.go -h

That last command will give you some subcommands you can then add to the end (in place of the -h):
Usage:
  main [OPTIONS] <command>

Help Options:
  -h, --help  Show this help message

Available commands:
  balance         get the wallet's balances
  chaintip        return the height of the chain
  currentaddress  get the current bitcoin address
  dumptables      print out the database tables
  newaddress      get a new bitcoin address
  spend           send bitcoins
  start           start the wallet
  stop            stop the wallet
  version         print the version number

@placer14 When I try following your directions, errors are thrown for Mocktxnstore. It all looks like simple type errors. Is there anyway around this?


FFI crashes if native functions call into the API

Dart:
void testGCInNative() {
  UnaryOp negateAndGC = ffiTestFunctions
      .lookupFunction<NativeUnaryOp, UnaryOp>("NegateAndGC");
  Expect.equals(0x8000000000000000, negateAndGC(0x8000000000000000));
}

C:
DART_EXPORT int64_t NegateAndGC(int64_t in) {
  for (intptr_t i = 0; i < 10000000; ++i) {
    Dart_NewInteger(0x8000000000000000);
  }
  return -in;
}

Due to this issue, it's currently impossible to test the behavior of FFI closures when a GC occurs within the native call. This should be done when this issue is fixed.

Note that NegateAndGC is obligated to test the return value of Dart_NewInteger for an unwind error and to propagate it.


Closed Below Mentioned Issue

Closes #28
Closes #26

Merging as Tested On Local Device , revert if things Break .


Don't require .NET 6

Describe the bug
Make it backwards compatible with .NET 5, which is shipped with every modern OS. Don't require to install a huge framework, especially since on developer machines it might very well interface with application being built on .NET 5.
Expected behavior
Works with .NET 5 and 6, whichever version is installed.
Desktop (please complete the following information):

OS: Windows 10
Version 2.35.4 currently, out of neccesity

Additional context
Embedding .NET 6 in the portable version is crude and makes it absolutely massive. Don't do that. Actually fix the problem please. .NET is not as safely upgradable as Java is.

.NET 5 has been EOL since more than 1 years ago.
Also many latest C# features require latest .NET runtime or https://github.com/Sergio0694/PolySharp.
.NET forces us to update every LTS versions.

.NET 4 is still in regular use, so I dunno. The point is, don't require users to install (or require yourself to bundle) a framework, when another super heavy bulky version is already installed on the system. Just use whatever the OS comes with, I'm sure it perfectly capable as proven by 2.35.4.

@thany There's no .NET 4. Do you refer to .NET Framework 4.x? They are too stale like zombies and Java 8. They are different from .NET Core 1-3 and .NET 5+. We can't use C# 8+ features out of the box including null-safe reference types (it's disabled by default in this project though). Also they lack some convenient APIs.
I'm preparing a pull request that shouldn't work in them.
From the order of project templates in the latest Visual Studio, Microsoft doesn't seem to recommend .NET Framework for new projects anymore.
As for the bundling of runtimes, I'll wait for an author's reply. I haven't ever been asked to install the desktop runtime of .NET 6 though.

I don't know, I don't keep tracking of the 17 billion versions of .NET (Core, Framework, or whatever) and what is a zombie and what isn't.
Again, the point is, don't require users to install an additional framework, when a perfectly good one is already installed (and indeed shipped with Windows). Only on an older version of Windows, a .NET install might be justifyable.
I frankly don't care about the reasoning behind it. As a user, I'm annoyed when I download a few megabytes of a program, and then am required to install hundreds of megabytes for, for a framework that (again) might interfere with whatever else I might be running. I'm just simply not going to do that. And yes, bundling the framework is a solution, but a terrible one at that.

https://github.com/dotnet/core/issues/6411#issuecomment-868813991
It seems MS won't bundle .NET 6+ by default.

https://learn.microsoft.com/en-us/dotnet/framework/whats-new/
but for new development, use .NET 6 or later.

In your words, don't require developers to downgrade to the stale framework not recommended these days by MS HQ.

am required to install hundreds of megabytes more,

I measured the size in the folder C:\Program Files\ScreenToGif (installer version). You said a few MB but actually it was just 167MB. This is smaller than Git (370MB), GIMP (1.78GB), iTunes (412MB), Wireshark (202MB), Firefox (220MB), and draw.io desktop app (414MB).

a perfectly good one is already installed (and indeed shipped with Windows)

I will never say this is "perfectly good". This is pre-installed just for backward-compatibility like the ancient VB6 runtime (the support for its development tools has already been expired). No new features except for the native support for ARM are added since 2019. Only security and bug fixes are provided.
https://github.com/NickeManarin/ScreenToGif/releases/tag/2.38.1
@NickeManarin Does Light.Portable not contain .NET 6 runtime and require us to install it?

You said hundreds of MB but actually it was just 167MB

For the installed entire framework. When bundled, it's """only""" 167MB for a program that can be under 14MB.
And it doesn't matter if it's smaller than other popular programs. That's whataboutism. Compare it to STG previous version: you'll agree that the portable version has ballooned to well over 10x its size, because .NET needs to be bundled.
Firefox is 220MB for very good reasons. It can do SO MUCH MORE, but the same rule would apply: if Firefox would blow up to 2.2GB, some folks might accept it, but certainly no one would appreciate it.
GIMP and Wireshark are big because they have to include or embed the Qt framework. At least for Qt there's no preinstalled version available on Windows, so I accept that. Git is big because it includes mingw64, which shouldn't be neccesary and I think it can be left out. But it's there. At least the executable isn't 370MB, right?
Other big applications might be that way because they are web-based, like VScode or Postman. Sure, they could use the preinstalled Edge, but they don't. I don't quite agree with that either, but at least in those cases, the programs themselves don't make the decisions directly to include a whole Chromium runtime - that's how Electron works.
In the case of STG, it uses .NET directly - not through some library/framework. STG's authors have made the conscious and deliberate decision to go for a framework that has to be priorly installed (or bundled) pretty much everywhere. And THAT is the problem. It was a deliberate decision to go this way. And what does the user gain from it? Nothing. STG 2.35 (14MB) works perfectly. Going over to a new .NET version is purely developer experience. And for the supposed convenience of the developers, we now have to deal with either installing yet another bulky framework that, again, might interfere with other stuff running on whatever the current .NET framework is, or we have to swallow a humongous bundled version.

Don't count on the .NET Framework, as dim as old guys. The reason Microsoft keeps building it into Windows is a similar reason you don't kill your old parents.
Versions of it vary from computer to computer. Some install 4.8 but some 4.6 or earlier due to compatibility concerns (4.8 should work though).
If ScreenToGif targeted at 4.8? Those that install 4.6 can't run ScreenToGif. How unsound it is.
I don' agree with your reason why you discriminate .NET from Qt and Chromium.
Don't be picky with a capacity that's only several thousandths of SSD. You should be the first to complain to Microsoft, not authors of individual software. There're so many apps much larger than ScreenToGif.
ScreentToGif itself without .NET is 21MB according to Light.Portable. The combination of it and https://dotnet.microsoft.com/download/dotnet/6.0 is the best for you. And of course, you'd choose the builds of other .NET-powered software that doesn't ship with a runtime.

The backward compatibility of Windows is phenomenal, but we should not be complacent about it.

I don't count on it. Microsoft is both famous for its phenomenal backwards compatibility in certain areas, and infamous for the lack thereof in other areas. Installing a newer version of .NET always leads to problems with other stuffing depending on it. Always.

If ScreenToGif targeted at 4.8? Those that install 4.6 can't run ScreenToGif. How unsound it is.

4.6 then, what do I care? Whatever is shipped with Windows 10 and up. And AGAIN, it worked fine in STG 2.35, so why even change over to a new version in the first place?

I don' agree with your reason why you discriminate .NET from Qt and Chromium.

We'll agree to disagree then. If you don't mind installing a whole framework for each and every application, good for you. But it's not normal to be required to do so.

Don't be picky with a capacity that's only several thousandths of SSD.

Here we go again. Sure ONE application being a little on the larger side is fine. But I don't have one application, and every application author thinks it's fine, because everyone has hundreds of terabytes anyway, right?
Similarly, throwing ONE plastic bottle in the ocean is fine, but there are millions, because everyone thinks "just one more" is acceptable. It's not.
To summarise: don't just grab all the storage you think you can get away with, just because it's available. Don't waste resources.

ScreentToGif itself without .NET is 21MB according to Light.Portable. In the first place, Windows, unlike Unix, doesn't have much of a habit of sharing libraries with multiple pieces of software.

Oh? .NET is our prime example. DirectX, OpenGL, everything behind Modern Apps? You know very well .NET is shared, don't kid yourself. All you have to do is use it, and not require a different version just because you wanna do it differently for whatever reason.
In the end, the light  portable version doesn't work, and the heavy portable one is too big. That's all. Why, how, when, where, what, I don't actually care about any of that as long as I'm wearing the proverbial hat of a user. The current situation is not acceptable, and I'm forced to stick with the previous release. That's all. You can talk about it long and short, but the fact remains that the newest version is not usable.

@NickeManarin Sorry for messing here up.

You installed NET framework 4.8, 4.7 or what ever version before, as it came with Windows or by manually installing.

I didn't, not on purpose anyway.

Do the same with NET 6 and then just download the light version and you should be fine.

I will not repeat myself again if you not only fail, but straight up refuse to understand my point.

Remember that this is a free and open source software. Peace out.

That is not really an excuse for anything. I'm trying to be reasonable towards making the software better for everyone, not just the developer. If you cannot fathom the issue at hand, then 2.36 and up is considered broken and unusable. The latest working version is 2.35.4, for mostly everyone except those who have yielded.
If you think the discussion is pointless, that says more about you than it does about me. Cheers!


EOL warning is loud for non-EOL ruby versions

Printing this at every startup is a pretty invasive warning for a ruby version which is not EOL for another 9 months:
WARNING: You are running Ruby 2.3.7, which is nearing end-of-life.
The Google Cloud API clients work best on supported versions of Ruby. Consider upgrading to Ruby 2.4 or later.
See https://www.ruby-lang.org/en/downloads/branches/ for more info on the Ruby maintenance schedule.
To suppress this message, set the GOOGLE_CLOUD_SUPPRESS_RUBY_WARNINGS environment variable.

Thanks for the feedback. I agree that it's loud. We're taking a stricter approach towards ruby version support than we have in the past and we feel it's important to let the users know well in advance. Do you have any suggestions for making the warnings less visible once a user understands the change in version support?

I would consider putting it in the gem post-install message for ruby versions that are still maintained.
As it is, I'm going to have to look at this message 30 times a day unless I define your custom environment variable across all hosts.  Gets kind of old, especially since I don't agree that indirect gem dependencies should take responsibility upon themselves for nagging me about ruby versions.  Couldn't every gem make the same argument that it should nag me?

Fair enough. Moving the warning to a post-install message on signet https://github.com/google/signet/pull/110


Details (optional)
Please can I contribute by adding this term to glossary ?

It would be appropriate for other maintainer(s) to assign this issue. 😊

HI @akshatcoder-hash, we decided that this term shouldn't be part of the glossary as it's not a widely known one. I'll go ahead and close this issue. If you want to work on a term, feel free to check out the list of open issue. There are a ton of them.


ci: Add Dependabot configuration for GitHub Actions

Pull Request
Description
This change introduces a Dependabot configuration file (.github/dependabot.yml) to automate dependency updates for GitHub Actions. The configuration sets up daily checks for updates, targets the main branch, and groups all GitHub Actions updates together. Commit messages for these updates will be prefixed with "deps(github-actions)".
fixes #27

Merge activity

Oct 16, 3:08 AM EDT: A user merged this pull request with Graphite.


Missing 3.x branch on repo

Tested versions
N/A
System information
N/A
Issue description
Might be a big problem for people who don't wanna use the official Godot source code yet want to keep using 3.x instead of 2.x/4.x
Steps to reproduce
N/A
Minimal reproduction project (MRP)
N/A

I think the main branch should befocused instead. Maintaining one version will be hard enough.

I believe that 2.x is just small mistake. 3.x is legacy and I don't see any point in working on it. It would mostly be duplicates of what upstream does.

I don't think 3.x is worth maintaining here. As a fork of the engine, there should be single point of divergence, from which development continues.

I do wonder if we should have the latest version of 3.x and figure out a way to allow user to move from 3.x to Redot 4.x, cause I know some users aren't ready to move to 4 yet.

I also think latest versions should be prioritized. 3.x/2.x are legacy and thus useless.

I agree with anyone above, but it would be more meaningful to backport some Godot 3 features like Rooms&Portals occlusion culling system.

Yeah some of the lighter weight features from godot 3 like the room and portal system could be helpful for supporting lower spec pcs which imo is one of the main reasons to use 3.x, the other main one obviously being if you already have a godot 3 project that's mostly done already and it wouldn't make sense to upgrade at this point

Main reason to use 3.x is support for wider spectrum of hardware available; It has quite the following. 4.x gets all other features including quality of life ones, which can be backported to 3.x. 3.x is quite lively and 3.7 will be quite nice.

OpenGLES2? Working on cheaper Mali chipsets? Backporting 4.x features for 3.x code? So this is just a Vulkan only remix then. Not everybody is a 3D fanatic.


Linux unit tests fail

Context
After #1208 , tests under ubuntu fail due to compiler bug in Swift.
This issue is created to provide visibility.
Related to: #1198

👋🏻  Update:

swift compiler also cannot use NSCoder with typealias ... = String - it melts down. (Read here for the context)

I'll create an MR soon to resolve this ⏳


Typo: Update azure-ad-integration-cli.md

Typo: two Azure AD application are configured -> two Azure AD applications are configured

@mocofound : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Typo fix verified.


Using RST for documentation generation

TL;DR

Using protoc gen doc RST support to make it simpler to generate docs and correctly link them internally
This avoids the post build script to fix the html links which was proposed and implemented here
https://github.com/flyteorg/flyteidl/pull/168

Fixed the documentation warning that arose of the usage of RST which required fixing the comments in the proto files

As part of this did clean up the proto files to remove the commented out validation code which was using protoc-gen-validate . Once we officially support then the code can revived from older version instead of keeping commented out code in the repo

Also removed unused imports from the proto files which would generate warnings during the doc generation process

Fixed the referenced links to google.protobuf.[timestamp|Duration|struct] by generating there documentation as part of the core.rst . This doc is generated at the tail end of the core documentation and helps with not having dead links to these where they are referenced in our documentation

Using a template for all RST file generation except for core. This was done to exclude the Scalar value doc to be generated in each module of the documentation
https://github.com/pseudomuto/protoc-gen-doc/blob/ae63e7aba9ef91669b2bc76634b33feead0e4f1a/resources/markdown.tmpl#L98

The new template removes the Scalar value and is only generated during core module generation and hence other modules can refer to the scalar values from the core.

Updated the contribution doc to use the protoc-gen-doc branch until RST support is officially available

https://docs.flyte.org/projects/flyteidl/en/docs-staging/developing.html#docs-generation
Rest of the docs can be seen here
https://docs.flyte.org/projects/flyteidl/en/docs-staging/protos/index.html
Type

[ ] Bug Fix
[ ] Feature
[ ] Plugin

Are all requirements met?

[ ] Code completed
[ ] Smoke tested
[ ] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
How did you fix the bug, make the feature etc. Link to any design docs etc
Tracking Issue
https://github.com/lyft/flyte/issues/
Follow-up issue
NA
OR
https://github.com/lyft/flyte/issues/

@kumare3 we need those additional steps for doc generation
We can separate steps for doc generation completely from make generate .


Fix dart pub publish

Top level pub has been removed in the latest stable sdk, and there is one more usage (the most important one) that was missed on previous changes.

Recent dart-sass release was failing due to this: https://github.com/sass/dart-sass/runs/6516857405?check_suite_focus=true


Discord invitation Link doesn´t work

The discord invitation link doesn´t work for me. I don´t know if it's only a personal issue or if others have the same. Could you have a look at it? If it´s only me, can you send me a new link? Thanks!

Hi @CodingFactoryT, the Discord link in the project page is a fully public invite that's used by dozens of people every day to join the community server. I believe it's on your end, but I don't have any details (such as a username) that I can use to investigate why you're unable to use it.


Pressure test cause a fatal error.

Describe the bug
A clear and concise description of what the bug is.

The memory is increased dramatically.

An crash error happened which point at the line below in MongoCore>Sessions.swift file.:

sessionManager?.releaseSession(serverSession)
And throw an error message below:
Fatal error: Can't form Range with upperBound < lowerBound: file
To Reproduce
Steps to reproduce the behavior.
I use Jmeter pressure tool to simulate 1000 users to do the infinite request on http://localhost:8080/v1/roles.
I use Vapor 4 to create routes and setup repositories, controllers, migrations,...
Here snip code is:
try v1.register(collection: RoleController())

try app.databases.use(.mongo(connectionString: dbURI), as: .mongo)

app.migrations.add(RoleMigration(), to: .mongo)

Expected behavior

The memory usage should be stable.
Do not crash.

Server-side Swift:

OS: [macOS]
Swift Version: 5.2
MongoKitten  Version [6.5.0] (fluent-mongo-driver from 1.0.0)

Additional context
I changed to fluent-postgres-driver package. Everything is going to quite well.
The memory was suppressed at the level nearly 23.4 MB. In additionally, the crash didn't happened.
In contrast, fluent-mongo-driver caused the memory over 200MB, and crashed unexpectedly.

Hey @qi-shun-wang,
I've got a few questions to help me investigate the issue, since it's occurring in quite a special place:

Are you using a transaction in that route?
Are you connected to a cluster?
Did you see what kinds of objects were lingering in memory?

Based on the large memory usage and the location of the crash, I'm assuming MongoKitten one of the following:

MongoKitten is making a lot of connections, something that really mustn't occur beyond 1 connection per-server.
A large amount of transactions happening don't happen on the same thread, and session storage is using memory for the large amount of sessions
MongoKitten keeps disconnecting and reconnecting for some reason, and data lingers (probably sessions)

Also, are you able to submit a reproduction sample?

Are you using a transaction in that route? No. Just simple query on an route that is all I have done.
Are you connected to a cluster? No. An MongoDB instance run as container in the docker environment.
Did you see what kinds of objects were lingering in memory?
I push the current project as a reproduction sample.
Sample
You should slightly change these files before you reproduce the problem.

Sources/App/Setup/databases.swift

import Vapor
import FluentPostgresDriver
/// Register your application's databases here.
public func databases(_ app: Application) throws {
    // Configure a PostgreSQL database
    let dbURI =  Environment.get(AppEnvironment.MONGO_DB_CONNECTION_URI.value) ?? "mongodb://localhost:27017/test"
   try app.databases.use(.mongo(connectionString: dbURI), as: .mongo)
   //try app.databases.use(.postgres(url: "postgresql://optima:123456@localhost:5446/postgres"), as: .psql)

}

Sources/App/Setup/migrate.swift
.mongo as migration target

  app.migrations.add(PolicyMigration(), to: .mongo)

d, and session storage is using memory for the large amount of sessions
MongoKitten keeps disconnecting and reconnecting for some reason, and data lingers (probably sessions)

Is that creation of the session and destroy process improperly run on different thread than cause this problem unexpected ?

@qi-shun-wang after a long search without results I just noticed you're on MongoKitten 6.5.0 😅
There were a few important patches made since then:
https://github.com/OpenKitten/MongoKitten/releases/tag/6.5.1
https://github.com/OpenKitten/MongoKitten/releases/tag/6.5.2

Currently, fluent-mongo-driver depends on 6.5.0 version. I will patch the package and upgrade to 6.5.2 for testing the situation. If there is no problem happened, I will leave a message here. 👍

I pulled the fluent-mongo-driver and changed the version 6.6.3.
Let the sample's dependency refer to local fluent-mongo-driver in Package.swift.
.package(path: "../fluent-mongo-driver"),
Sadly, the problem still exist.

I've found your issue! It was quite a bit simpler and different than what I was looking for. The implcit session was released to the session pool. So the session pool kept filling with references to this implicit (global) session ID. Let me know if 6.6.4 fixed your issue! And thanks for bringing this to my attention.

Brilliant! The problem have solved.


KDEMouse

Description of the new feature / enhancement
You guys should insource this open source KDEMouse project.
https://www.codeproject.com/Articles/15320/KDE-Mouse-Emulator
This is a small application which emulates the behaviour of the mouse in KDE. It uses global window hooks and WTL.
In KDE (a desktop environment for UNIX operating systems), Windows can be moved and resized without having to click on Window edges or going through menus. To move a Window, simply press ALT and press left mouse button and reposition the Window by moving the mouse. To resize a Window, press the right mouse button instead, and adjust the position of the bottom right corner by moving the mouse.
I chose not to write a lengthy article about this tool mainly because the techniques I have used have already been covered on Code Project and elsewhere. The main technique I utilize is the hooking mechanism provided by Windows. A search for SetWindowsHookEx() will give you plenty of material on that topic. When you start the application, a tray icon will appear. The icon is an image of a typical basic Microsoft mouse with a scroll wheel. You can't miss it. Right click it for Enable, Disable, About and Quit. The provided zip file contains the source code as well as precompiled binaries. The binaries have been compiled in release mode.
The application will only manipulate top level Windows, which have the WS_THICKFRAME style set. Your child Windows and unresizable dialogs are thus safe.
Scenario when this would be used?
moving windows which happens all the time
Supporting information
No response

duplicate of https://github.com/microsoft/PowerToys/issues/269

/dup https://github.com/microsoft/PowerToys/issues/269


Streaming Assistant Not returning result after submitting the tool result

Confirm this is an issue with the Python library and not an underlying OpenAI API

[X] This is an issue with the Python library

Describe the bug
I am trying to use streaming assistant with function calling and whenever the tool output is submitted, stream ends , it is not getting the final message or response from assistant.  Please let me know , whether I am doing something wrong
from openai import OpenAI
import time
import json
import time

from openai.types.beta import AssistantStreamEvent

client = OpenAI()

def get_current_weather(location, unit="fahrenheit"):
    """Get the current weather in a given location"""
    if "tokyo" in location.lower():
        return json.dumps({"location": location, "temperature": "10", "unit": "celsius"})
    elif "san francisco" in location.lower():
        return json.dumps({"location": location, "temperature": "72", "unit": "fahrenheit"})
    else:
        return json.dumps({"location": location, "temperature": "22", "unit": "celsius"})

def getNickname(location: str) -> str:
    return location + "God"

from typing_extensions import override
from openai import AssistantEventHandler

function_map = {
    "getCurrentWeather": get_current_weather,
    "getNickname": getNickname
}

from dataclasses import dataclass

@dataclass
class tool_output:
    tool_call_id: str
    output: str

from openai.types.beta.threads import Message, MessageDelta
from openai.types.beta.threads.runs import ToolCall, RunStep

class EventHandler(AssistantEventHandler):
    def __init__(self, thread_id, assistant_id):
        super().__init__()
        self.output = None
        self.tool_id = None
        self.thread_id = thread_id
        self.assistant_id = assistant_id

    @override
    def on_text_created(self, text) -> None:
        print(f"\nassistant > ", end="", flush=True)

    @override
    def on_text_delta(self, delta, snapshot):
        print(delta.value, end="", flush=True)

    @override
    def on_end(self, ):
        print(f"\n end assistant > ",self.current_run_step_snapshot, end="", flush=True)

    @override
    def on_exception(self, exception: Exception) -> None:
        """Fired whenever an exception happens during streaming"""
        print(f"\nassistant > {exception}\n", end="", flush=True)

    @override
    def on_message_created(self, message: Message) -> None:
        print(f"\nassistant > {message}\n", end="", flush=True)

    @override
    def on_message_done(self, message: Message) -> None:
        print(f"\nassistant > {message}\n", end="", flush=True)

    @override
    def on_message_delta(self, delta: MessageDelta, snapshot: Message) -> None:
        print(f"\nassistant > {delta}\n", end="", flush=True)

    @override
    def on_tool_call_created(self, tool_call):
        print("The type of tool call is ", type(tool_call), flush=True)

        # print(f"\n tool call created assistant > {tool_call.type}\n, >>>> {tool_call.function.json}", flush=True)
        # print("function name is ",tool_call.function.name,flush=True)
        # print("function argument is ", tool_call.function.arguments, flush=True)
        # function_name=tool_call.function.name
        # print(tool_call.function.arguments)
        # function_args=json.loads(tool_call.function.arguments)
        # fn = function_map.get(function_name)
        # output=fn(function_args.get("location"))
        # print(output)
        # tool_output(tool_call.id,output)

    @override
    def on_tool_call_done(self, tool_call: ToolCall) -> None:
        print(" toolcall val in on_tool_call_done is The type of tool call is ", tool_call, flush=True)

        tool_id = tool_call.id
        f = tool_call.function.name
        arg = json.loads(tool_call.function.arguments)
        fn = function_map.get(f)
        output = fn(arg.get("location"))
        run_id = self.current_run.id
        print("The output of function call is ",output,flush=True)
        self.tool_id=tool_id
        self.output=output
        client.beta.threads.runs.submit_tool_outputs(
            thread_id=thread.id,
            run_id= self.current_run.id,
            tool_outputs=[{
                "tool_call_id": self.tool_id,
                "output": self.output,
            }],

            stream=True
        )

    @override
    def on_run_step_created(self, run_step: RunStep) -> None:
        print("The type of run step is ", type(run_step), flush=True)
        print(f"\n run step created assistant > {run_step}\n", flush=True)

    @override
    def on_run_step_done(self, run_step: RunStep) -> None:
        print(f"\n run step done assistant > {run_step}\n", flush=True)

    def on_tool_call_delta(self, delta, snapshot):
        # print(f"\n tool call delta assistant > {delta.type}\n", flush=True)

        if delta.type == 'code_interpreter':
            if delta.code_interpreter.input:
                print(delta.code_interpreter.input, end="", flush=True)
            if delta.code_interpreter.outputs:
                print(f"\n\n tool call delta output >", flush=True)
                for output in delta.code_interpreter.outputs:
                    if output.type == "logs":
                        print(f"\n{output.logs}", flush=True)

    @override
    def on_event(self, event: AssistantStreamEvent) -> None:
        print("In on_event of event is ", event.event, flush=True)

thread = client.beta.threads.create()
print(thread.id)

message = client.beta.threads.messages.create(
    thread_id=thread.id,
    role="user",
    content="Get me weather of San Francisco ",
)

with client.beta.threads.runs.create_and_stream(
        thread_id=thread.id,
        assistant_id="asst_x",
        event_handler=EventHandler(thread.id, "asst_x"),

) as stream:

    stream.until_done()
    print("\n")
    print(stream.current_run_step_snapshot)

# stream = client.beta.threads.runs.create(
#     thread_id=thread.id,
#     assistant_id="asst_x",
#     instructions="Please address the user as Jane Doe. The user has a premium account.",
#     stream=True,
# )

# for event in stream:
#     print(event.model_dump_json(indent=2, exclude_unset=True))

    # print(stream.get_final_messages())

The last line of output prints that thread.run.step is in progress but stream is completed and program exits

 end assistant >  RunStep(id='step_x', assistant_id='asst_a', cancelled_at=None, completed_at=None, created_at=1710513033, expired_at=None, failed_at=None, last_error=None, metadata=None, object='thread.run.step', run_id='run_x', status='in_progress', step_details=ToolCallsStepDetails(tool_calls=[FunctionToolCall(id='call_x', function=Function(arguments='{"location":"San Francisco, CA","unit":"f"}', name='getCurrentWeather', output=None), type='function', index=0)], type='tool_calls'), thread_id='thread_x', type='tool_calls', usage=None, expires_at=1710513631)

RunStep(id='step_x', assistant_id='asst_x', cancelled_at=None, completed_at=None, created_at=1710513033, expired_at=None, failed_at=None, last_error=None, metadata=None, object='thread.run.step', run_id='run_x', status='in_progress', step_details=ToolCallsStepDetails(tool_calls=[FunctionToolCall(id='call_x', function=Function(arguments='{"location":"San Francisco, CA","unit":"f"}', name='getCurrentWeather', output=None), type='function', index=0)], type='tool_calls'), thread_id='thread_x', type='tool_calls', usage=None, expires_at=1710513631)

Assistant creation

assistant = client.beta.assistants.create(
  instructions="You are a weather bot. Use the provided functions to answer questions.",
  model="gpt-4-1106-preview",
  tools=[{
      "type": "function",
    "function": {
      "name": "getCurrentWeather",
      "description": "Get the weather in location",
      "parameters": {
        "type": "object",
        "properties": {
          "location": {"type": "string", "description": "The city and state e.g. San Francisco, CA"},
          "unit": {"type": "string", "enum": ["c", "f"]}
        },
        "required": ["location"]
      }
    }
  }, {
    "type": "function",
    "function": {
      "name": "getNickname",
      "description": "Get the nickname of a city",
      "parameters": {
        "type": "object",
        "properties": {
          "location": {"type": "string", "description": "The city and state e.g. San Francisco, CA"},
        },
        "required": ["location"]
      }
    }
  }]
)

To Reproduce
Please use the above code to reproduce
Code snippets
No response
OS
windows
Python version
3.11
Library version
1.14.0

When you encounter a tools interaction the current stream does end. When you submit the tool responses you start a new stream that you have to handle. There is a helper for this as well so you can submit your response using this instead:
 with client.beta.threads.runs.submit_tool_outputs_stream(..):    ...
This also takes a handler class (you can reuse the one you already have) and helps you to process the subsequent stream resulting your tool outputs.

Thanks for reporting, we'll take a look next week!

I hit this same thing but got it working. It seems to be working as designed. Arguably it’s not super intuitive if you’re coming from the old non-streaming API (I kept looking for action_required when the SDK uses on_tool_call_done).
@pstern-sl’s right that the stream ends and you’re expected to get a new stream using submit_tool_output_stream from on_tool_call_done

you can reuse the one you already have

Yeah though note you can reuse the class but not the object. You need a new instance of your handler.
Here’s a working example for reference: https://github.com/datastax/astra-assistants-api/blob/main/examples/python/function_calling/basic.py#L88-L98

Hey Seb! Thanks so much – that's a great script. Is it okay if we use/reference it elsewhere?