wrong return value while create a new ticket

from openssl online document: https://www.openssl.org/docs/manmaster/ssl/SSL_CTX_set_tlsext_ticket_key_cb.html
The return value of the cb function is used by OpenSSL to determine what further processing will occur. The following return values have meaning:
2
This indicates that the ctx and hctx have been set and the session can continue on those parameters. Additionally it indicates that the session ticket is in a renewal period and should be replaced. The OpenSSL library will call cb again with an enc argument of 1 to set the new ticket (see RFC5077 3.3 paragraph 2).
1
This indicates that the ctx and hctx have been set and the session can continue on those parameters.
0
This indicates that it was not possible to set/retrieve a session ticket and the SSL/TLS session will continue by by negotiating a set of cryptographic parameters or using the alternate SSL/TLS resumption mechanism, session ids.
If called with enc equal to 0 the library will call the cb again to get a new set of parameters.
less than 0
This indicates an error.

parent link to https://github.com/apache/trafficserver/pull/400

Commited in https://svn.apache.org/r1736186
Please note that this change has no real effect with the current OpenSSL (and forks) usage, they check only for >= 0 return value when requesting for a new ticket (parameter enc is 1).
See:
https://github.com/openssl/openssl/blob/master/ssl/statem/statem_srvr.c#L3017
https://github.com/libressl-portable/openbsd/blob/master/src/lib/libssl/src/ssl/s3_srvr.c#L2520
https://boringssl.googlesource.com/boringssl/+/master/ssl/s3_srvr.c#2046
Thanks for the report, returning 1 is probably the right thing to do should the SSL libs later change their check on the return value.

you are welcome.
I'm a ATS code reader, just follow the OpenSSL API document to understand
the SSLNETVConnection, and found the `bug' in ATS code.
bryancall ( a committer of ATS ) tell me the code same as the apache httpd
project.
Thanks for your analysis in detail.
2016-03-22 21:45 GMT+08:00 ylavic<EMAIL_ADDRESS>
Commited in https://svn.apache.org/r1736186
Please note that this change has no real effect with the current OpenSSL
(and forks) usage, they check only for >= 0 return value when requesting
for a new ticket (parameter enc is 1).
See:
https://github.com/openssl/openssl/blob/master/ssl/statem/statem_srvr.c#L3017
https://github.com/libressl-portable/openbsd/blob/master/src/lib/libssl/src/ssl/s3_srvr.c#L2520
https://boringssl.googlesource.com/boringssl/+/master/ssl/s3_srvr.c#2046
Thanks for the report, returning 1 is probably the right thing to do
should the SSL libs later change their check on the return value.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/apache/httpd/pull/18#issuecomment-199822745

Please note that after https://github.com/openssl/openssl/commit/5c753de668322bf9903a49ba713b2cbc62667571 got merged into OpenSSL, it no longer just checks for >=0, but the ==0 is actually enforced as documented.
So any OpenSSL with that patch (1.1.0 and higher) will just not inject a ticket key without this patch.

Thanks for the note, the patch is currently being proposed for backport to the next 2.4.x, one more vote and it's in.

Backported in https://svn.apache.org/r1811742, will be part of 2.4.29.


What is the file CompanyList?

What is the file for?
What is the extension?
Naming is consistent with other files

Hmm, I think you are right to leave it out of there before there was anything in there, now it's a piece of rubbish lurking there.
I'm thinking about putting real coding challenges there sorted by company, the ones they send out to juniors to test their coding skills. It's very hard to measure your skills in a useful way, in that way juniors can see what's expected of them.
It could be framework sorted as well, or language sorted, or It could be timed challenges with a limit ("you should be able to code this in about 1h") type questions- or just simple apps with normal user stories or other functionality.

you should be able to code this in about 1h

I found giving someone a test that is not achievable in the time is best, because then can see where they prioritise their efforts 🤓 - just my 2 cents

Yes, I think that is fine, it makes sense from the recruiting side.
It should be fine to state in the test specification for instance, "hey: it's not about finishing this test in the time given, the whole point is to showcase where you prioritize your efforts in that amount of time".
There are not that many companies willing to give juniors a chance, it might be that after months of efforts of applying only one or two might be willing to give them a shot. If they are then given a timed test with a workload way beyond and they don't know this way of thinking they might just hit the ground running and start coding and not showcasing their real potential. If done a bad job, they won't be able to go back and apply in a while if a new position comes out.
But if they know these sort of tests exist they might be able to stay calm and then focus on the architecture or write it in pseudo code, and lastly start to implement the functions of there is time left.
I truly do not like tests that have a very short time limit, it's stressful and stressful thinking is not a good representation. There's also an element of time stealing when the tech does not work, or the internet connection drops, or the in-browser IDE is slow, etc.
I would rather have an open time limit, and then discuss with the recruiter why I took the time I did. For instance, this was totally new to me and I had to learn my way through, or similar.
TLDR: Let's get everybody on the same level of understanding and take some stress away, it's stressful enough to apply and get rejected for x months straight.

I'm revamping the site so this is not relevant anymore.


refactor: Remove special case for host mode

Player object in host mode does not need special case.  Follow the same code path as client/server mode.  This reduces a lot of complexity and makes host mode more consistent with client/server mode.

does this change anything usage wise?
does this change anything in our projects?

No,  all code removed was internal,  usage is exactly the same.
And no,  our projects remain the same,  see ummorpg test passed without modification

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


syz-manager: Confusion about numFuzzing and fuzzingTime

https://github.com/google/syzkaller/blob/8916f5e15133e4de1640fcf6c06f375e35837b3a/syz-manager/manager.go#L219-L220
numFuzzing equals VMs, why ? What if type is "isolated" ?
I get VMs: 0 and fuzzing 0s since fuzzingTime is calculated according to VMs

numFuzzing equals VMs, why ?

VM number show how many VMs is fuzzing.

What if type is "isolated" ?

I don't understand the question. What should be different if type is isolated?
Please use the mailing list for questions in future.


chore: ESLintとprettierの設定

Why
ESLintとしてlinterを設定しておくと、コードが汚くにくい。
コード書くたびにチェックするようにできるので複数人の開発に使えるため
また、prettierを設定しておくとコードが綺麗になる。vscode用の設定も追加するとlocalで編集されてもフォーマットを統一できる。
What

ESLintの設定
prettierの設定

参考リンク
https://blog.monorevo.jp/post-2104

Great.
Can we ignore the  @typescript-eslint/no-unused-vars rule for now and format later (as the app header is the main source of violation and we would refactor them in a while), or we should not overlook them?

@psmuler
Either method is no problem. It depends on which you prefer.
I recommend the way you remove unused code from now because you can  read and write code easily.
You can edit this request freely.

@psmuler
Because I cannot guarantee that it will work, this request doesn't need merge.

The latest updates on your projects. Learn more about Vercel for Git ↗
Name	Status	Preview	Comments	Updated (UTC)
react-yjs	❌ Failed (Inspect)			Dec 24, 2023 4:01am

@psmuler  By the way, did you confirm this error?
probably, it doesn't deploy to Vercel now

Removed all the unused import and added the rule to
The error remaining :
Error: Require statement not part of import statement.  @typescript-eslint/no-var-requires
You can get rid of it by installing latest y-websocket library v1.5.1


Ensure ALSA is installed

On Buster Lite images ALSA is not preinstalled. But the install script should install all dependencies.
Thanks to @mattelacchiato
https://github.com/MiczFlor/RPi-Jukebox-RFID/discussions/1208#discussioncomment-255616

@s-martin you probably meant alsa-utils instead of alsa-tools. at least that's what works for me with buster lite.


Fundamental wave documentation not in sync with the model

Reported by dietmarw on 19 Feb 2016 18:02 UTC
In

Modelica.Magnetic.FundamentalWave.BasicMachines.Components.SymmetricMultiPhaseWinding it says:

The symmetrical multi phase winding consists of a symmetrical winding resistor, a zero and stray inductor as well as a symmetrical multi phase electromagnetic coupling and a core loss model including heat port.

where "zero" links to Modelica.Electrical.MultiPhase.Basic.MutualInductor instead of Modelica.Electrical.MultiPhase.Basic.ZeroInductor. In addition the actual model only contains the ZeroInductor but not a stray inductor. That is actually external to the SymmetricMultiPhaseWinding model.

Modelica.Magnetic.FundamentalWave.BasicMachines.Components.SinglePhaseWinding
also the StrayInductor is mentioned as part of this component and again is actually not part of it.

Migrated-From: https://trac.modelica.org/Modelica/ticket/1916

Comment by christiankral on 22 Feb 2016 20:31 UTC

The link to the zero inductor is now replaced by Modelica.Electrical.MultiPhase.Basic.ZeroInductor

The reference to the stray inductor is now replaced by a link to a stray reluctance -- which is the way how the stray field (parameter stray inductance) is considered in the model. The documentation update applies to Modelica.Magnetic.FundamentalWave.BasicMachines.Components.SymmetricMultiPhaseWinding and Modelica.Magnetic.FundamentalWave.BasicMachines.Components.SinglePhaseWinding.

Changes are applied in c09c9a4bd43943ca761d4089d4a070d1bc71bd12, c522415b9eeb01ba706397e09c45e4f25872def8 in trunk and maintenance.

Changelog modified by christiankral on 22 Feb 2016 20:31 UTC
Fix of documentation of FundamentalWave package

Modified by beutlich on 22 Feb 2016 22:29 UTC

Changelog modified by beutlich on 22 Feb 2016 22:29 UTC
Fixed documentation of FundamentalWave package


Right-Clicking armor doesn't equip

When you right-click an armor piece from your hotbar, the enchantment doesn't apply. You have to manually put it on in the inventory.

Bonus info; CTRL + Click to storage


Feature/dog schema test

this was to add jest as dependency and script to be accessible to all team members


typo: sevice -> service

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Get-SPServiceApplicationProxyGroup.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


⚠ Bofrost.at has degraded performance

In 3657a48, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 3537 ms

Resolved: Bofrost.at performance has improved in e4f8d62.


[Bug]: API code crashing from time to time

Contact Details
No response
What happened?

Steps to Reproduce
this just happens from time to time since last release in production
What browsers are you seeing the problem on?
Chrome
Relevant log output
No response
Screenshots
No response
Code of Conduct

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

August 16, 2023 at 19:58 (UTC+8:00) | Clearing sync timeouts before exiting... | libreChat
August 16, 2023 at 19:58 (UTC+8:00) | There was an uncaught error: | libreChat
August 16, 2023 at 19:58 (UTC+8:00) | Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client | libreChat
August 16, 2023 at 19:58 (UTC+8:00) | at new NodeError (node:internal/errors:399:5) | libreChat
August 16, 2023 at 19:58 (UTC+8:00) | at ServerResponse.setHeader (node:_http_outgoing:644:11) | libreChat
August 16, 2023 at 19:58 (UTC+8:00) | at ServerResponse.header (/app/node_modules/express/lib/response.js:794:10) | libreChat
August 16, 2023 at 19:58 (UTC+8:00) | at ServerResponse.send (/app/node_modules/express/lib/response.js:174:12) | libreChat
August 16, 2023 at 19:58 (UTC+8:00) | at abortMessage (/app/api/utils/abortMessage.js:5:28) | libreChat
August 16, 2023 at 19:58 (UTC+8:00) | at ask (/app/api/server/routes/ask/openAI.js:196:20) | libreChat
August 16, 2023 at 19:58 (UTC+8:00) | at process.processTicksAndRejections (node:internal/process/task_queues:95:5) | libreChat
August 16, 2023 at 19:58 (UTC+8:00) | at async /app/api/server/routes/ask/openAI.js:45:10 { | libreChat
August 16, 2023 at 19:58 (UTC+8:00) | code: 'ERR_HTTP_HEADERS_SENT' | libreChat
August 16, 2023 at 19:58 (UTC+8:00) | } | libreChat
August 16, 2023 at 19:58 (UTC+8:00) | Error: null pointer passed to rust | libreChat
August 16, 2023 at 19:58 (UTC+8:00) | at module.exports.__wbindgen_throw (/app/node_modules/@dqbd/tiktoken/tiktoken_bg.cjs:421:11) | libreChat
August 16, 2023 at 19:58 (UTC+8:00) | at wasm://wasm/00b5f812:wasm-function[783]:0x712c6 | libreChat
August 16, 2023 at 19:58 (UTC+8:00) | at wasm://wasm/00b5f812:wasm-function[781]:0x712ac | libreChat
August 16, 2023 at 19:58 (UTC+8:00) | at wasm://wasm/00b5f812:wasm-function[646]:0x6c064

might be related to https://github.com/danny-avila/LibreChat/pull/689

seem to have been fixed in the release  https://github.com/aitok-ai/LibreChat/releases/tag/2023-08-28a


Laravel composer update give error

I just update the compose and i found below error.
php artisan clear-compiled PHP Fatal error: Call to undefined method Illuminate\View\Compilers\BladeCompiler::directive() in /var/www/html/innomaid/vendor/laravel/framework/src/Illuminate/Support/Facades/Facade.php on line 213

[Symfony\Component\Debug\Exception\FatalErrorException]
Call to undefined method Illuminate\View\Compilers\BladeCompiler::directive()

Script php artisan clear-compiled handling the post-update-cmd event returned with an error
[RuntimeException]
Error Output: PHP Fatal error: Call to undefined method Illuminate\View\Compilers\BladeCompiler::directive() in /var/www/html/innomaid/ve
ndor/laravel/framework/src/Illuminate/Support/Facades/Facade.php on line 213

Here is my composer code
{
"name": "laravel/laravel",
"description": "The Laravel Framework.",
"keywords": ["framework", "laravel"],
"license": "MIT",
"type": "project",
"require": {
"laravel/framework": "5.0.",
"illuminate/html": "5.",
"barryvdh/laravel-dompdf": "0.6.",
"tymon/jwt-auth": "0.5.",
"gloudemans/shoppingcart": "~1.3",
"intervention/image": "^2.3"
},
"require-dev": {
    "phpunit/phpunit": "~4.0",
    "phpspec/phpspec": "~2.1",
    "zizaco/entrust": "dev-laravel-5"
},
"autoload": {
    "classmap": [
        "database"
    ],
    "psr-4": {
        "App\\": "app/"
    }
},
"autoload-dev": {
    "classmap": [
        "tests/TestCase.php"
    ]
},
"scripts": {
    "post-install-cmd": [
        "php artisan clear-compiled",
        "php artisan optimize"
    ],
    "post-update-cmd": [
        "php artisan clear-compiled",
        "php artisan optimize"
    ],
    "post-create-project-cmd": [
        "php -r \"copy('.env.example', '.env');\"",
        "php artisan key:generate"
    ]
},
"config": {
    "preferred-install": "dist"
}

}

try this command
$ composer update --no-scripts

and check the config/app.php providers array that each provider's package exist

We don't support Laravel 5.0.x.

@GrahamCampbell  Then were i go

Its not give the error during composer update but in my application
FatalErrorException in Facade.php line 213: Call to undefined method Illuminate\View\Compilers\BladeCompiler::directive()
error come

update "zizaco/entrust": "dev-laravel-5" to "zizaco/entrust": "1.4.1"
Solve my problem.


Write a Ruby program to find smallest number in an array

Description
Write a Ruby program to find smallest number in an array
Input  : [1, 2, 3, 4, 5]
Output : 1

How to contribute

Save the solution in program/FindSmallestNumberInAnArray.rb file
Add FindSmallestNumberInAnArray.rb file in find-smallest-number-in-an-array folder

!assign


[Bug]: Vscode opens an empty commit message file

Opencommit Version
1.1.22
Node Version
19.8.1
NPM Version
9.6.2
What OS are you seeing the problem on?
Mac
What happened?
A bug happened!

Possible Solution
I dont know.
Steps to Reproduce
No response
Relevant log output
No response

reopening, will take a look at this

@maxbec does it persist still? what git hooks do you have set?


problem to run YCB_Video_toolbox/evaluate_poses_ keyframe.m

Hi, when I use YCB_Video_toolbox to run the matlab code "YCB_Video_toolbox/evaluate_poses_
keyframe.m", I meet the problem! Who can help me ? Thanks a lot in advance!
Error using load
Unable to read file 'Densefusion_iterative_result/0026.mat'. No such file or directory.

Error in evaluate_poses_keyframe (line 50)
    result_my = load(filename);

Hi, please make sure whether "0026.mat" is generated. I guess you might have changed the eval code which probably skips some results when the segmentation from PoseCNN fails to find the target object.

@j96w However, I did not change the eval code? I followed your tips and found only "0025.mat". I could not find the "0026.mat". Can you help me?

Hi, it's working well on my side. It would be better to debug from your side. For example, remove the try-except code and see what's going on with index==26.


License of the repository

Can you please update the license of the repository. Thanks.

I updated the license, thank you.


Fix agg limit contains null values may cause error result

Proposed changes
before：
mysql [test_query_db]>set parallel_pipeline_task_num = 1;                                                                                Query OK, 0 rows affected (0.00 sec)

mysql [test_query_db]>select count(), cast(k12 as int) as t from baseall group by t limit 1;
+----------+------+
| count(*) | t    |
+----------+------+
|       16 | NULL |
+----------+------+
1 row in set (0.01 sec)

mysql [test_query_db]>set parallel_pipeline_task_num = 2;
Query OK, 0 rows affected (0.01 sec)

mysql [test_query_db]>select count(), cast(k12 as int) as t from baseall group by t limit 1;
+----------+------+
| count(*) | t    |
+----------+------+
|        7 | NULL |
+----------+------+
1 row in set (0.01 sec)

after:
mysql [test_query_db]>set parallel_pipeline_task_num = 2;
Query OK, 0 rows affected (0.01 sec)

mysql [test_query_db]>select count(), cast(k12 as int) as t from baseall group by t limit 1;
+----------+------+
| count(*) | t    |
+----------+------+
|        16 | NULL |
+----------+------+
1 row in set (0.01 sec)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Fix extractType type in TypeScript 3.9.6

extractType is currently displays the following error with TypeScript 3.9.x:

error TS2456: Type alias 'extractType' circularly references itself.

From the TypeScript team ^1:

Certain things can be deferred if there's a different declaration to "pause" the circularity at. I don't want to commit any specific circular behavior to bug/not bug; it depends on a lot of different factors and is sensitive to exactly where TS tries to resolve certain conditionals.

So basically, this PR keeps the same behavior as before, but uses a workaround to prevent the TypeScript 3.9.6 compiler from getting into a circularly references error state.
Fixes https://github.com/TCMiranda/joi-extract-type/issues/33

Works for us!

Guys @omairvaiyani and @bvallee-thefork I pushed it to master, but I got some errors on one of my projects. I'll try to understand what happened there before I publish this, ok?
Also, autocomplete and inspections on vscode seem worst, did you try to make it better?
Before:

After:


Chrome Buggy Behavior (potential defect) - Tab Groups being lost

I just installed on Chrome Win10 X64 : Version 64.0.3282.167 (Official Build) (64-bit)
I am experiencing tab groups being lost.
Example Flow

Existing Window with buncha tabs
move buncha tabs to new window (new window contains all tabs for proposed new tab group)
enter name of new group in window where tabs have been moved to
Close Window
Return to "main" or remaining chrome browser window
newly created Tab Group (before closing it's window) is now Missing, and tabs are lost.

Chrome just updated to Version 64.0.3282.186 (as opposed to .167), I haven't tried reproducing with this slightly updated version yet.  Will let you know, perhaps others can try and confirm.

Good catch!

Let me know if it is well corrected in next release.

yep confirmed the fix in chrome. Thank you!

Ack!  The original issue is fixed, but now, the parent window gets stuck with the name of the child window's group!  It's like both windows now are joined at the hip by the new tab group.  In this new use flow, I opened that second chrome window, gave it a name.  Then I simply navigated back to my main window, and clicked on the sync tab groups icon, and it showed up as the child window! (make visible was clicked) when my intention was to give them main window a different name.  If I then click / unclick group visible, the child group that I created previously gets cleared, and now both the parent and the child end up with the same default group name.

I don't understand what really happens. Can you explain it differently ?
What do you mean with child window ?

sorry I've been busy for a while.  Yes I can explain it differently, also I have input on #91 as well, I will look through all the tickets and provide repro instructions for ones I have seen in both Chrome and FF sometime this week.

What do you mean with child window ?

What I meant was, if you open Google Chrome you have an initial Single Window, let's call this MAIN, in the behavior I described above:
bq.  the parent window gets stuck with the name of the child window's group! It's like both windows now are joined at the hip by the new tab group.
Basically, I open Chrome and get my "main" window, then I open a new window, either by dragging a tab or just opening a new window manually.  I then go and name this new window in tab groups with some name.  When I navigate back to the MAIN window, it appears as though the name I gave the child or second window in the tab groups dialog is now the name of the Main or original window.

This problem doesn't seem readily reproducible anymore with latest google chrome and latest tab groups 0.6.6, I will circle back if I am able to observe it again.


add HyperLogLog support (PFADD, PFCOUNT, PFMERGE)

I've made the changes as I originally meant in cacc505. At some point I'll have to go through the whole Redis module and fix the formatting + add references to the redis.io docs.


how to use wechaty-bro.js

how to use wechaty-bro.js
i have try to inject the javascript (wechaty-bro.js) into https://wx.qq.com/ ，but some error has occurred;
like:

If I understand you right: you want to build a new UI for Wechat powered by Wechaty totally from scratch?
Then I'd like to suggest you just run Wechaty from the standalone mode(which is the default, using puppeteer headless mode), then you will be able to use all the Wechaty API to do whatever you want.
To answer your question: you can not inject the wechaty-bro.js to your browser directly, because it is highly coupled with the puppeteer in the current Wechaty code base.
Please let me know if you have any other question.
Good luck!

yes, so i just need to npm install wechaty to use wechaty in  electron ?

Yes, I think so.
Please feel free to let me know if you have any other questions.

thanks

Could you please close this issue if your issue is solved?

I'll close this issue due to inactivity for more than one month.


Switch to NET Standard 1.6

I'm looking to use this with a Xamarin based client and I wasn't able to due to the projects not supporting those platforms.
I accidentally included a version bump for Newtonsoft.Json so let me know if you want me to remove that and I'll fix that up.

Great thing on updating this, please remove the update on Newtonsoft as I haven't had a chance to read what changes it makes.
Thanks!

Done.


Package fails to install due to missing yarn dependency

When trying to add this package to my project with pnpm, I am getting a yarn not found exception. This is because the prepare script is being ran on my machine after install
 ERR_PNPM_PREPARE_PKG_FAILURE  not found: yarn

https://github.com/swan-io/boxed/blob/e951a0239f2034d9ee7ff416f10df7abd12d5620/package.json#L38
According to npm docs, the npm prepare script gets ran before publish but also after installing the package. If the purpose of the prepare script is to test & build the source files, maybe they can be added to prepublishOnly  instead?
Reference: https://docs.npmjs.com/cli/v8/using-npm/scripts#npm-install

Unfortunately, yarn doesn't support prepublishOnly lifecycle script.

Ah you're right, they don't. For yarn it looks like instead they recommed to use the prepack lifecycle script for building/bundling.
https://yarnpkg.com/advanced/rulebook#packages-should-use-the-prepack-script-to-generate-dist-files-before-publishing

Packages should use the prepack script to generate dist files before publishing

Why? The original npm supported many different scripts. So much, in fact, that it became very difficult to know which script one would want to use in which context. In particular, the very subtle differences between the prepack, prepare, prepublish, and prepublish-only scripts led many to use the wrong script in the wrong context. For this reason, Yarn 2 deprecated most of the scripts and consolidated them around a restricted set of portable scripts.

Solution: Always use the prepack script if you wish to generate dist artifacts before publishing your package. It will get called before calling yarn pack (which itself is called before calling yarn npm publish), when cloning your git repository as a git dependency, and any time you will run yarn prepack. As for prepublish, never use it with side effects - its only use should be to run tests before the publish step.

Maybe that will help? Would love to install this package for work but requiring yarn is a blocker


[BUG]  KeyDB never releases space from the Flash Storage

Describe the bug
KeyDB never releases space from the Flash storage. Despite manually deleting keys, the amount of used storage does not significantly decrease.
To reproduce

Run KeyDB with Flash storage enabled.
Insert a large number of keys into the database.
Manually delete a portion of the keys.
Check the storage usage; the space occupied on the disk does not decrease as expected.

Expected behavior
After deleting keys, the amount of storage space used by KeyDB should decrease significantly, reflecting the removal of the data.
Additional information
The storage engine is configured to use Flash.
I’ve ensured that the keys were deleted correctly and are no longer accessible.
However, the storage space used does not change, even after multiple manual deletions.
Version of KeyDB being used: 6.3.4
OS: Ubuntu 20.04

Thank you for the response. I had running keyDB during a month, and every key stored in the SST files are never compacted.
I guess that may there some command like COMPACTDB in order to force manually the compact but I no found nothing.
Aditionally to this, I cant found in the complete code where the Compact is called. During this month obviously the storage never down, and I can reach every KEY that shouldn't be founded, removed manually and by TTL.
Some aditional data that I can provide just ask me please, would great if I can contribute in some way

No, its no there. BUT if I scan manually the SST files, I can found the key there. Its very strange. I will attach here the test

The first time that I set "TEST"
Once time deleted still there and never is removed/compacted (I have 1 month of keys by 192 GB of storage lol)

Thanks!

keydb-server --version
KeyDB server v=6.3.4 sha=00000000:0 malloc=jemalloc-5.2.1 bits=64 build=b532cd0401cb0da4
Im not sure how I have that cpp file given that I installed keydb using apt. How I can check it?


Fixes BackgroundComponent's border to match background alpha.

All in the title.

Yes, it then do not matches with in-game xp tracker interface.


Betting View Changes

Old ticket description
![user-images.githubusercontent.com/11491365/49554163-d1587280-f8af-11e8-8fbf-ad915ebaf5af.png](https://)
Replace trade w/ bet and change the price to odds. Allow people to switch between American, fractional, and decimal odds.

Quantity should go to amount, price should be odds.

There should be no order book or depth chart, just a price history line chart + the order book as we see below with bet + lay, that's what bettors expect to see.
Buy/sell is instead back / lay. The betting ticket can be used for the initial liquidity provisioning page for markets too
My open orders, transactions page, and positions page I've thought less about but transaction pages from betfair, open orders, and positions pages from betfair are below:

https://www.youtube.com/watch?v=n36TrNC3yqs also good video if people aren't familiar with betting to learn.

Ancient


data and pretrained model

First i want to say that your work is great!!!
I have two questions:

When are you planning to publish your the model for Emotion Recognition?
Can you also publish the UGC annotated dataset (sentiment and emotion)?

Thank you so much for this comment! :)
We are currently working on a paper that describes the model and the data.
We will unfortunately be able to publish the ER model and the data only after the paper is published.
We will keep you posted!

Thank you so much for this comment! :)
We are currently working on a paper that describes the model and the data.
We will unfortunately be able to publish the ER model and the data only after the paper is published.
We will keep you posted!


Reader

[x] Slides
[ ] Slides Review (Raúl)
[ ] Language Review - Script (Maureen)

Please, it's necessary to have these first steeps ready before recording. Also a friendly reminder: please practice your English before.

[ ] Recording (04/03/2018)
[ ] Editing
[ ] Youtube
[ ] Language Review (Maureen)
[ ] Publishing (04/12/2018)

Slides #39

Hey @FPerezP! I need to plan next videos and recordings. Had you the opportunity to move one with this? Thanks!

I'm working on addressing Raul's comments. I plan to have them ready for a new review tomorrow.

Great thanks!


Updating IAM service account snippets

I'm working on the following two docs:
https://cloud.google.com/iam/docs/creating-managing-service-accounts
https://cloud.google.com/iam/docs/creating-managing-service-account-keys
The Java code snippets for these docs don't always capture the imports and variables necessary to run the code. To fix this, I added region tags around the imports and variables necessary for each snippet.

mdejong3@ - We have a new format for samples that includes solutions for issues like missing imports. Are you interested in updating these samples to the newer format? I'm happy to answer questions or provide more info on it.

Looks like I can't merge until @lesv looks it over again


docs: fixed the contributing documentation link

For some reason the GitHub wasn't able to process the AsciiDoc syntax to include code from another.
Updated the link of the env.examples  to be referenced as hyperlink.
Reference: https://docs.asciidoctor.org/asciidoc/latest/directives/include/#include-nonasciidoc

Hey @alecharp
The earlier syntax wasn't rendered. It printed the code as it is: include::.env.example[]. As a reader, this did not make sense to me.
My current syntax will create a hyperlink to the file. In this case, the user can view the file themselves

The earlier syntax wasn't rendered.

In GitHub? That is normal, GitHub is not rendering AsciiDoc file correctly.

This is how it is rendered normally.

My current syntax will create a hyperlink to the file. In this case, the user can view the file themselves

No, your current syntax is not correct and is not creating the link your are expecting.

In GitHub? That is normal, GitHub is not rendering AsciiDoc file correctly.

For me the file is rendered this way:

The syntax is rendered in Intellij but not in GitHub.
Hence the PR.

Please install the AsciiDoc plugin on your IDE to see what you are proposing.

Do you still want me to continue with the PR?

The syntax is rendered in Intellij but not in GitHub.

Because GitHub is not rendering AsciiDoc files correctly. But it doesn't mean the syntax of the file is broken.

Do you still want me to continue with the PR?

You can, but the content of the PR is not correct so I won't merge it in this state.
You can add a few words to the original content so say "you can find an example of the variables in .env.example" and keep the include for example.
But as is, the content is not correct and I won't accept it.

The code is hosted on GitHub, people will look at the README there most of the time. I find it unfortunate having a broken rendering by default, especially for a section needed for running the project locally.

But for the contributing guide, shouldn't it be read from the IDE? Even more for details about how to run the application locally? As MD has no include equivalent, it would mean to duplicate the element which I find source of issues when you change them (like when I did implement #159)

@Jagrutiti

Do you still want me to continue with the PR?

No, the documentation seems to be misleading to you, so it might to others. Fixing it is the way to go. I just don't feel you proposed the correct solution so far. Please see https://github.com/jenkins-infra/plugin-health-scoring/pull/199#issuecomment-1408849997 as I suggest something to move forward.

Hey @alecharp
What do you think of adding a GitHub link?
It is easier than searching for a file in project directory. I have made this change in my recent commit.
If not I will follow your last comment.

@Jagrutiti please don't include your .idea/ files, they should not be in the git tree.
I would like you keep the include block and use a proper link to the file. You also included a typo in your change.

@Jagrutiti Please revert the deletion of the include and revert the addition of the files from .idea. Then I'll merge this PR.

Hey @alecharp
I have made the change just like you wanted.

Please see that #203 is targeting a larger refactoring of the documentation. I included your suggestion there directly.
I'd appreciate if you could review that pull request, to make sure that the new documentation is clear.


Proposed alternative to eval $(rbenv init -) (faster shell startup)

I've been using rbenv for years and I absolutely love it.
In recent months, I've also discovered spinoffs for various other language ecosystems, like pyenv, nodenv, and goenv. Even better!
The Trouble: Slow shell startup
Then I added all the following lines to my .bashrc:
for util in rbenv pyenv nodenv goenv; do
  PATH="$HOME/.$util/bin:$PATH"
  eval "$($util init -)"
done

and discovered that my shell startup time grew to ~500ms (presumably what's most expensive is sourcing completion scripts and defining the xxenv() helper function for each utility). It might not seem like a lot, but it makes launching the terminal emulator (or opening new tmux panes) feel really sluggish.
The Answer: Defer rbenv init until it's actually used
Now I do this:
for util in rbenv pyenv nodenv goenv; do
  # 1. Also add `shims` dir to PATH
  PATH="$HOME/.$util/bin:$HOME/.$util/shims:$PATH"

  # 2. Define an alias that undefines itself and runs `rbenv init` on-demand
  alias $util="unalias $util; eval \"\$(command $util init -)\"; $util"
done

and my shell startup is super snappy again! The first time I run rbenv (or any binary in the shims directory) in a given shell session, it takes an extra ~100ms to load—which is almost unnoticeable compared to the 500ms I had to wait any time I opened a new terminal window.
The proposal
Is there any way that this approach (or something like it) might be officially adopted by rbenv? I don't think it would be very elegant to put this in the README (i.e., under the official installation instructions). But maybe the output of rbenv init - could be modified somehow to make it work like this under the hood?
What do you think?

That's a pretty cool solution; thanks for sharing!
What do you think is the shell support for the unalias ... eval solution that you've proposed? Does it work across shells like bash, zsh, tch, ksh, fish?

2. when invoking any rbenv shim

Why would we want the custom alias to resolve in this case? Shims are separate from shell integrations

Maybe I am off base here; I will admit I don't fully understand rbenv internals.
My thinking is, I'd like to lazy-load rbenv whenever I run any ruby command it has a shim for, and not just rbenv itself. So, I cd into a project I'm working on and run bundle, but rbenv hasn't been initialized yet. The bundle shim then invokes rbenv, which invokes my funky alias, which then runs the init script and voilà.
Is that not how it works?

One downside I had not considered yesterday is that the completion script doesn't load until rbenv or one of its shims has been executed. :\

The bundle shim then invokes rbenv, which invokes my funky alias, which then runs the init script and voilà.
Is that not how it works?

Close, but the shim does not have the ability to invoke your shell alias. The shim runs in a subprocess of your shell, which does invoke rbenv, but does not trigger your shell aliases because the subprocess is a separate instance of bash. Additionally, shims require no rbenv shell integrations to work—they just want to be present in PATH.
Let's take a few steps back and go over how rbenv shell integrations work. First of all, it's absolutely possible to use most rbenv functionality without ever enabling its shell integration. This is all that's needed:
export PATH=~/.rbenv/bin:~/.rbenv/shims:"$PATH"

Calling most rbenv commands, switching between versions, and executing shims will all work with this approach. Rbenv was intentionally designed to not need shell integrations (unlike RVM).
However, rbenv offers some integrations, and they are enabled by doing eval "$(rbenv init ...)" in your shell. What that does is:

Ensures that shims are in PATH (although, you can also do this manually)
Sets up rbenv completions
Sets up the rbenv shell function - this enables commands that need to change the state of your running shell, and the only command that needs this is rbenv shell
Automatically performs a rbenv rehash to ensure all shims are up to date (potentially slow—you can disable this by using rbenv init - --no-rehash)

So if you do have rbenv shell integrations enabled and you call rbenv in your shell, the first thing that executes is the rbenv function. You could theoretically make this lazy-loading by defining a rbenv alias. But, when shims execute rbenv, they are executing ~/.rbenv/bin/rbenv and they will always skip any "rbenv" functions or aliases that you might have. That's why shims are unaffected by shell trickery.

One downside I had not considered yesterday is that the completion script doesn't load until rbenv or one of its shims has been executed.

That's true :( It might be a dealbreaker. Some shells like zsh lazy-load completions, and I know that there are hacks for bash to lazy-load completions too, but I wonder whether all this is starting to be more trouble than it's worth.

Wow, was not expecting such a comprehensive treatment. 🙇🙇🙇 Learned a lot from you, big thanks!
(As a point of clarification, this whole misunderstanding appears to stem from me somehow getting all mixed up about my trial-and-error in the shell—I could have sworn that when I came up with my little hack two days ago, running shims would cause rbenv to get redefined as its shell helper function... but now that I try it again, that's clearly not the case.)

I wonder whether all this is starting to be more trouble than it's worth.

100%. Getting lost down yak-shavey rabbit holes is definitely a trademark weakness of mine. Many, many thanks for your patience with me, and letting me lead you down the garden path! 😅


CDK-nag actions from external forks do not have write permissions

Describe the bug
As happens in this PR, the cdk-nag action requires write permissions, but the maximum permissions allowed for a fork in a public repo are read permissions.

alternatives

use ok-to-test --> requires authentication, preferred is GitHub App (not possible for data.all),  other options: personal access token or OAuth app token
run cdk-nag action only on minor release branches (v2m2m0 to main) branch
run cdk-nag on pull_request_target after the PR is merged
run cdk-nag on schedule
Use other than OIDC
running without credentials --> maybe mocking the cdk.context.json, not adding fake values but mocking it

Advice:
6 woudl solve the issue, so preferred if possible. 2 and 3 can also be added.
How to Reproduce
Open a PR from a fork.
Expected behavior
No response
Your project
No response
Screenshots
No response
OS
n/a
Python version
n/a
AWS data.all version
7.0.0
Additional context
No response

Hi @dlpzx This is marked as high priority and in progress, if you are working on it id like to add it to v2.1.0 milestone to track it.

Completed in #767


Plan for how to manage versioning in Curator and in components that Curator depends on

Curator specifies particular versions of components that it depends on that have been tested (e.g. OpenShift API, postgresql etc.)  Make a plan for how we expect updates to Curator or to any of the component parts to be integrated and tested so that potential users will know which versions have been show as compatible.  (Others may also be compatible, but we can't test them all.)  DoD:  A design document listing the alternatives and the associated development, testing and documentation work shoudl be completed and reviewed by the Curator team.

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
/close

@sesheta: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
/close

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


Fix serialization of primitives

Summary
Fix #9033
Details and comments

@adekusar-drl Can you try this PR with your code?

@t-imamichi  Looks good, thanks!


Implement a simple synchronous local HAL scheduling (sibling to task_*)

Systems that have a single processor don't need the fancy task system - they just need a way to run through simple commands kept in a command buffer. Ideally really small systems would compile away all of the HAL but it's a much smaller investment (and keeps code portable) to have a lightweight local-execution HAL backend that just runs synchronously.
This would be similar in nature to what existed before with the async_command_queue (which was async from the host perspective but synchronous from the device perspective) but much much simpler. It's intended for when the host and device are in the same binary/process/bare - in cases where there's a host/device split (no shared memory/independent versioning/different processes/etc) something more involved is required.

The old C++ command buffer stuff lives here:
https://github.com/google/iree/blob/2d534397009fb2bc8189c540799be1084c15499b/iree/hal/host/inproc_command_buffer.h#L205
https://github.com/google/iree/blob/2d534397009fb2bc8189c540799be1084c15499b/iree/hal/host/inproc_command_buffer.cc#L125-L138
This could be made much more efficient; a new version should use iree/hal/local/arena.h for storage and not record any recording-only state (push constants/descriptor sets/etc). The iree/hal/local/task_command_buffer.c does this and only actually commands are recorded. Even that could be made more memory efficient by slicing out the state tracking storage from the block pool and returning it after the iree_hal_command_buffer_end such that only the recorded storage persists.

Some of the plumbing and the starting point is being done as part of #5508. The big work here (and what is mentioned above) is to add support for command buffer recording and explore pre-compiled command buffers (where we encode a binary structure in the compiler and load it at runtime without needing to issue the VM/HAL calls).


SkeletonShaderGLES3 vertex shader compilation error in GET_BONE_MATRIX expanded define [Intel HD 4000 on Windows]

Godot version
4.0.0stable, 4.0.1stable, 4.0.2stable
System information
Windows 11, GLES3, Intel HD 4000
Issue description
I am getting a series of errors trying to load 3D meshes (GLB/GLTF), i tried with and without animations and the errors still showing.
Running on the the Compatibility mode (Due to Intel HD 4000 not supported for Vulkan).
On loading the project, the errors are:

Loading the mesh, the errors are:

Steps to reproduce

Run Windows 11

Open Godot 4.0.2

Apply any GLB/GLTF 3D Mesh to project folder

Attempt to Apply the mesh into the Scene or New Inherited Scene, and the errors will appear

Minimal reproduction project
N/A

Can you please show the full error, it looks like there's more error code above indicating where the fault is in the shader code (hard to localise as the skeleton shader doesn't have that many lines so would be helpful if you showered the full error)

sorry for the late reply, here is all the errors that show.

@Miottolas Can you please copy the full error printout and paste it here? I can see in your screenshot that you have disabled part of the error print. Its important that we get the full error as it prints the entire shader and points us to the line where the error is coming from.

Sure thing, my bad.
I will upload an txt file, hope that's help out,
File: GD4Error.txt

Thank you. That is perfect

This issue still exists in Godot 4.1, thanks

The code it's failing on looks like this:
#ifdef USE_SKELETON
317:
318: #define TEX(m) texelFetch(skeleton_texture, ivec2(m % 256u, m / 256u), 0)
319: #define GET_BONE_MATRIX(a, b, c, w) mat4(TEX(a), TEX(b), TEX(c), vec4(0.0, 0.0, 0.0, 1.0)) * w
320:
321: 	uvec4 bones = in_bone_attrib * uvec4(3);
322: 	uvec4 bones_a = bones + uvec4(1);
323: 	uvec4 bones_b = bones + uvec4(2);
324:
325: 	highp mat4 m;
326: 	m = GET_BONE_MATRIX(bones.x, bones_a.x, bones_b.x, in_weight_attrib.x);    // Error here
327: 	m += GET_BONE_MATRIX(bones.y, bones_a.y, bones_b.y, in_weight_attrib.y);
328: 	m += GET_BONE_MATRIX(bones.z, bones_a.z, bones_b.z, in_weight_attrib.z);
329: 	m += GET_BONE_MATRIX(bones.w, bones_a.w, bones_b.w, in_weight_attrib.w);

I don't see any obvious syntax error, so I expect the problem might be one of:

Not supporting texelFetch
Not supporting % or u
Not supporting expanding nested macros

Hi @akien-mga
Thanks for looking into this!
Here are my logs:
Godot4.1_Error.txt

Please let me know if you need anything. Happy to work with you on this to find root cause. Thanks

Any solution?
I'm also having this same problem in Godot 4.1.2 (in fact, in any version of 4.x).
In Godot 3.x it works normally, but I want to be able to work in 4.x
My specifications:

Intel® HD Graphics 4000 (Version <IP_ADDRESS>61)
Intel® Core™ i3-3217U CPU @ 1.80GHz

Thank you for your attention.

@SamuelEds Did you try upgrading to the drivers linked by Akien above?
Alternatively in Godot 4.2, you will be able to use the ANGLE backend, which will most likely resolve this issue as well

@clayjohn thanks for answer!! The driver's version <IP_ADDRESS>61 is the last, but descontinued.
I try the Godot 4.2 dev 6 with ANGLE like you metioned and worked great!!
This is the first time I've heard about this ANGLE, but anyway, thank you very much!!

Based on the discussion above I conclude that support for older Intel iGPUs is no longer provided by Intel themselves (no driver updates), and on the engine side we can only offer a workaround via ANGLE, which we now do. So this issue can be closed.
Please feel free to comment if you think this needs to be reopened for one reason or another.


hi,about MASLayoutConstraint.

In MASViewConstrain, can you make the MASLayoutConstraint property public? Becasule, I want to get the constant .

@wy19901227 , in fact, MASLayoutConstraint is a subclass of NSLayoutConstraint, you can access all the properties.
And as objective-c is a dynamic language, you can access any property even it's private.

yeh, I can get it by KVC ,I just think make it public will be better.

Exposing constant is difficult since under the hood constraints built using MASConstraintMaker are wrapped in either MASViewConstraint or MASCompositeContraint. The later can't return a meaningful constant


Data caching

Maybe should be done on Infoboxer's side. Just two general considerations:

when you call something like countries ten times a day, there's no point to query (rather huge) list of pages again and again;
when you call countries in a several monthes -- it's reasonable to refetch everything: even country list itself may have changed, not speaking about populations, leaders and so on.

Will be rethought.


[Reopened][XLA] Bring XLA Transpose021Tiled up-to-date with reference CUDA implementation.

Reopening this PR.
The commits have been cherry-picked to master branch.
Ping @jpienaar, @jhseu because you were the previous assignee and assigner.

Can one of the admins verify this patch?

Jenkins, test this please

Do you have some performance results to compare the new and current implementations?

Hi, I did a manual test on two sets of inputs. One transposes 4 matrices of size 1024x1024, the other transposes 4 matrices of size 512x512. It takes current implementation 549.36us and 152.84us to finish whilst only  283.75us and 75.842us for the new implementation.
I actually do not know how to invoke these two methods using Python scripts so I used nvprof on test cases I added in xla/tests/copy_test.cc. Also, nvprof automatically aggregates the performance metrics of kernels with the same name so I can only measure two test cases at one run which is quite inconvenient. Would you please tell me what's the best way to benchmark it? It may come in handy in the future as well.
In addition, I am not sure how important this transpose kernel is. If it is performance critical then I can try to bring other specializations done in the reference cuda kernels here as well.

That sounds good thanks! I'll try to add a benchmark for this.
As to performance critical, this is more about wanting to supplement the decision to change the implementation with results. It is always nicer if an implementation is changed as it provides better performance or corrects a bug. Those are more measurable benefits :)
The folks who are the main GPU backend developers are otherwise engaged this week. If this is not blocking you, I'm going to defer until they are able to review it.

Just to make sure, please do not merge yet, hopefully more commits are to come early next week.

Hi, can anyone take a look at this PR ? For now the CUDA reference implementation seems to be inefficient when dealing with narrow matrices and before I go ahead and implement some specializations for dealing with narrow matrices in XLA, I'd like to make sure that the CUDA version of my idea looks okay.

Jenkins, test this please.

Transient failure on Windows. Jenkins, test this please.

Build timeout on GPU?
Jenkins, test this please.

Jenkins, test this please.


How to copy or publish container without mounts

Required information

Distribution: Ubuntu
Distribution version: 18.04
The output of "lxc info" or if that fails:

Kernel version: 4.4.0-116-generic
LXC version: 3.0
LXD version: 3.0
Storage backend in use: ZFS

Issue description
I create containers which run services. Some containers have mounts to host directories for persistent storage. When i try to copy or publish a container that has a mount on it, it tries to copy not just the container but the entire mount as well.
Steps to reproduce

I run this command to copy a container from one host to another host.

lxc copy ftp-container repository:ftp-container

Information to attach

[ ] Any relevant kernel output (dmesg)
[ ] Container log (lxc info NAME --show-log)
[ ] Container configuration (lxc config show NAME --expanded)
[ ] Main daemon log (at /var/log/lxd/lxd.log or /var/snap/lxd/common/lxd/logs/lxd.log)
[ ] Output of the client with --debug
[ ] Output of the daemon with --debug (alternatively output of lxc monitor while reproducing the issue)

Can you show lxc config show ftp-container --expanded

architecture: x86_64
config:
  environment.http_proxy: ""
  image.architecture: amd64
  image.description: Centos 7 amd64 (20180514_02:16)
  image.os: Centos
  image.release: "7"
  image.serial: "20180514_02:16"
  raw.lxc: lxc.mount.auto = cgroup
  security.privileged: "true"
  user.network_mode: ""
  volatile.base_image: 83465d9a9af4a843c6e9f7bde4bcb7170c7352cc555f8d6b637039006f3f49af
  volatile.eth0.hwaddr:
  volatile.eth0.name: eth0
  volatile.idmap.base: "0"
  volatile.idmap.next: '[]'
  volatile.last_state.idmap: '[]'
  volatile.last_state.power: RUNNING
devices:
  eth0:
    nictype: bridged
    parent: br0
    type: nic
  root:
    path: /
    pool: default
    type: disk
ephemeral: false
profiles:
- default
stateful: false
description: ""

Do you have any feedback on this please?

Can you show cat /proc/self/mountinfo?

@elbrinke

Hi @stgraber
Apologies for the delay. Output below:
470 94 0:46 /lxd/SAMBA/config /var/lib/lxd/devices/ftp-container/disk.config.etc-samba- rw master:36 - zfs containers/persistent rw,xattr,noacl
476 94 0:48 / /var/lib/lxd/devices/ftp-container/disk.samba.samba rw master:39 - zfs storage/samba rw,xattr,noacl
482 94 0:46 /lxd/SAMBA/var /var/lib/lxd/devices/ftp-container/disk.var.var-lib-samba- rw master:36 - zfs containers/persistent rw,xattr,noacl
526 94 0:84 / /var/lib/lxd/storage-pools/default/containers/ftp-container rw,relatime shared:200 - zfs default/containers/ftp-container rw,xattr,noacl

Hmm, that issue doesn't make much sense... Those extra mounts are only inside the container and LXD only ever looks outside of the container.
That is, it will publish or copy what's at /var/lib/lxd/storage-pools/default/containers/ftp-container which doesn't have any of your extra disks applied.

@elbrinke can you look into /var/lib/lxd/storage-pools/default/containers/ftp-container/rootfs and see if that filesystem matches what you expect?


Database Storage

Need to save the Snippets via a database (firebase?). Worst-case scenario, we can hook into localStorage or something to manage state. Would be nice to have this persist across devices, which would mean authentication. Will tackle this when the app is more fleshed out.

Might not be ideal. Would most likely be more beneficial to store snippets locally and give the user the ability to save/edit/delete gists from their Github instead.

Deciding not to save via a database. Closing issue.


easier installation debugging

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10 with Java API
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 1.3.0
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: 8 with cudnn64_6.dll
GPU model and memory: 1080
Exact command to reproduce: HelloTF.java

Describe the problem
Default error message from NativeLibrary load() method is not helpful enough.
Simple suggested improvement: please print the contents in the string variable "frameworkResourceName", which is the missing resource, when throwing a new UnsatisfiedLinkError exception.
Source code / logs
Suggested Source Code Improvement for NativeLibrary.java:
if (jniResource == null) {
  throw new UnsatisfiedLinkError(
      String.format(
          "Cannot find TensorFlow native library %s for OS: %s, architecture: %s. See "
              + "https://github.com/tensorflow/tensorflow/tree/master/tensorflow/java/README.md"
              + " for possible solutions (such as building the library from source). Additional"
              + " information on attempts to find the native library can be obtained by adding"
              + " org.tensorflow.NativeLibrary.DEBUG=1 to the system properties of the JVM.",
              frameworkResourceName,
          os(), architecture()));
}

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

create a folder structure like org\tensorflow\native\windows-x86\tensorflow_jni.dll and paste that inside your class path. try running post that you will not get any error.


India: Deploy PV model

Following on from https://github.com/openclimatefix/PVNet/issues/124
Need to deploy the PV model .We should be able to do this in a similar way to the Wind model

This has happened now


Unclear how configure plugin in "docker compose" approach

Hello,
I'm trying to set-up an on-premise FlexMeasures deployment using docker-compose.yml as starting point.
After creating the account and administrator user, I want to add the ENTSOE plugin to fetch energy price data.
I added a copy of the flexmeasures-entsoe plugin code to the flexmeasures-instance folder and configured the entries in flexmeasures.cfg as described. However, it is not clear to me whether the plugin should be running in the server or in the worker docker container. Should I use the /app/instance/ path, so it is only loaded in the server docker, or should I use /usr/var/flexmeasures-instance? The first path is not mounted in the worker, which is throwing errors. But using the second path, the worker is complaining about loading the entsoe-py dependency, as the requirements.txt is not applied in the worker.

Hi @rhpijnacker I believe this is more of a personal choice. (and maybe this is actually more fitting of an issue on the flexmeasures repo)
At Seita, we even run another kind of service (next to web server and queue worker) must for cron jobs - and the entso tasks are each modelled as a cron job.
If you stay within the current docker-compose architecture, the worker node might be conceptually the more fitting choice (it would do everything that isn't serving web requests). It's a full Ubuntu, so it has a crontab.
Installing requirements.txt then only really has to happen in the worker. I see what you mean though, in that pip install -r instance/requirements.txt only happens in the server - there is no reason I can think of that it shouldn't happen in the worker, as well. In our production infrastructure, we install plugins plus dependencies in both.
I would welcome a pull request which upgrades the worker to be able to run plugins (maybe installing extra requirements from pip install -r /usr/var/flexmeasures-instance/requirements.txt ?


Resizing Columns triggers Column Sorting

Describe the bug (required)
When using both useResizeColumns and useSortBy , resizing the column triggers the column sorting feature. This is using all default props other than styled-components styling.
Provide an example via Codesandbox! (required)
Reproduced: react-table useResizeColuns useSortBy
Steps To Reproduce (required)
Steps to reproduce the behavior:

Open codesandbox
Resize a column

Expected behavior
Event bubbling is disabled, so resizing a column does not register the event in the div behind which registers the click for sorting.
Desktop (please complete the following information):

OS: Ubuntu 18.04, Windows 10
Browsers: Chromium/Chrome 80.0.3987.149, Firefox 71.0 & 74.0
Versions: react-table "7.0.0-rc.7", "7.0.0"

The same

Check https://spectrum.chat/react-table/general/i-combined-the-basic-examples-to-enable-column-sorting-and-resizing~45d0177e-36e1-47fa-9464-76f84a142469

Yep, you just have to put the sortByToggleProps on an element that doesn't wrap the resizer. You can put it on a span that wraps the header text or even a div... whatever floats your boat!


Dropdown menus in navigation are not working in Doxygen pages

Up to v0.3.2, API Doxygen pages have a different navigation layout. They are now modified to reflect the navigation in the other pages but, when clicking on a dropdown menu button, the related menu is not displayed, so the related html code is now temporarily disabled in the doxygen/layout/custom_header.html file. The current result can be seen at https://redtide.github.io/sfizz/api/develop/.

The website is now moved on sfztools.github.io.


adding missing dialog attribute

I wish more people added back attributes and properties not currently covered, thanks! 🙏


Exception: undefined is not an object (evaluating 'n.attributes')

Describe the bug

How to reproduce

Expected behavior

YAML configuration

# please paste here your card config (before backticks below; do not remove them)

Entity debug data

Dev console errors

Screenshots

Version

Closing due to missing any details


Publish RuboCop::LSP.enable to enable LSP mode

RuboCop::Cop::Base.enable_lsp_mode will be renamed to RuboCop::LSP.enable.
Before:
RuboCop::Cop::Base.enable_lsp_mode

After:
RuboCop::LSP.enable

This is because whether or not it is LSP should not be set per cop but should be a state controlled for the entire RuboCop process.
Since RuboCop::Cop::Base.enable_lsp_mode was an experimental private API, it will be renamed without a deprecation warning.
Along with this, the module name has been changed from RuboCop::Lsp to RuboCop::LSP. This change is made to align with Ruby's standard API, like RubyVM::YJIT.enable, making it RuboCop::LSP.enable instead. RuboCop::Lsp.enable does not look as pretty name.
With the renaming to RuboCop::LSP.enable, the usage of this API will be documented. This allows libraries like Ruby LSP and Solargraph, which use internal modules other than RuboCop's built-in RuboCop::LSP::Server#start, to easily utilize the features in #12586 and #12657.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Good change!


Feature Request : pinecone or milvus

Hi
its is possible to add pinecone or milvus to the script as AI memory where we can upload our vectored document to the DB .
regards

We are releasing a Plugins endpoint this week which will utilize plugins. There is still work to do for file integration, and a retrieval plugin still needs to be made, but yes this is planned for now.


[Suggestion] Custom Icons for the TRP3 Toolbar

Hey there. At the moment the TRP3 toolbar uses WoW icons:
https://i.imgur.com/e6sMwDn.png
These icons are fairly vague. It would be better if you guys made them custom icons that show what they are doing like an icon that simply reads "OOC" or "AFK" or "QUEST" or something like that. Icons that let you instantly determine what they do, rather than these WoW icons.

Custom icons would take additional space for little gain and would need to be localized in all the languages we support if they contain text, so it's not something we're really looking to do.
There could be work done to improve the status clarity at a glance, but we would probably need more extensive changes, so it's not a priority right now.


local make fails on mac

Following the instructions in the README for local bootstrapping (https://github.com/mapzen/mapzen-docs-generator#rapid-bootstrapping), the make command fails on a mac at:
curl -sL https://github.com/mapzen/vector-datasource/archive/v0.10.2.tar.gz | tar -zxv -C src-vector-tiles --strip-components=2 --exclude=README.md --wildcards '*/docs/'
with the error:
tar: Option --wildcards is not supported

Thanks for testing @meetar, and to @migurski for a fix.


fix(42034): Missing codefix with 'This may be converted to an async function' suggestion diagnostic

Fixes #42034
/cc @andrewbranch

@andrewbranch Thanks


Image loading 'race condition' issue and solution

I wasn't sure how to name this issue. Anyway, onto the main point.
You got an interface called ILoader which has a function called LoadStream.
I was a little suspicious already at this point because it doesn't return anything, it's just a task.
public Task LoadStream(string gltfFilePath)
{
	if (gltfFilePath == null)
	{
		throw new ArgumentNullException("gltfFilePath");
	}

	return LoadFileStream(_rootDirectoryPath, gltfFilePath);
}

private Task LoadFileStream(string rootPath, string fileToLoad)
{
	string pathToLoad = Path.Combine(rootPath, fileToLoad);
	if (!File.Exists(pathToLoad))
	{
		throw new FileNotFoundException("Buffer file not found", fileToLoad);
	}

	return Task.Run(() => { LoadedStream = File.OpenRead(pathToLoad); });
}

The result is that LoadedStream can and very likely will be overwritten eventually, for example in this line later in GLTFSceneImporter.cs:
await _options.ExternalDataLoader.LoadStream(image.Uri);
_assetCache.ImageStreamCache[sourceId] = _options.ExternalDataLoader.LoadedStream;

This is run N amount of times, where N is the amount of textures the material has.
I propose you rewrite ILoader to not have LoadedStream, but instead LoadStream should be of return type Task<Stream>, like this:
public async Task<Stream> LoadStream(string gltfFilePath)
{
	if (gltfFilePath == null)
	{
		throw new ArgumentNullException("gltfFilePath");
	}
	string pathToLoad = Path.Combine(_rootDirectoryPath, gltfFilePath);
	if (!File.Exists(pathToLoad))
	{
		throw new FileNotFoundException("Buffer file not found", gltfFilePath);
	}

	return await Task.Run(() => File.OpenRead(pathToLoad));
}

And then the previously mentioned lines could look something like this:
var stream = await _options.ExternalDataLoader.LoadStream(image.Uri);
_assetCache.ImageStreamCache[sourceId] = stream;

... at least this works for me. Thanks in advance.

Resolved with #532


Section marks (#%%) not working as expected

Issue Report Checklist

[X] Searched the issues page for similar reports
[X] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[X] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[N/A] Could not reproduce inside jupyter qtconsole (if console-related)
[X] Tried basic troubleshooting (if a bug/error)

[X] Restarted Spyder
[X] Reset preferences with spyder --reset
[N/A] Reinstalled the latest version of Anaconda
[X] Tried the other applicable steps from the Troubleshooting Guide

[X] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
I've got a template that I've used for quite a long time for initial content.  This is in %USERNAME%\.spyder-py3\template.py.  In Spyder 4.0.1 I noticed that the section marks weren't working correctly.  On investigation, it seems that an extra percent on the first section mark makes it work, which seems really weird to me.
Compare the two images below.  In each image, the left-hand side shows the template.py in Vim; the right-hand side shows what happens when you then create a new file in Spyder 4.0.1.  The only difference between the bare template files as shown in Vim in the two images is that there's an extra % on line 5: #%% at the start of the line on the first (top) image vs #%%% on the second (bottom) image.  All the other section markers stay as #%%.
In the first image (with #%% on all sections), the sections aren't marked with a horizontal line in Spyder, in the second image (with #%%% on the first section but not the others), all the sections are marked.

What steps reproduce the problem?

Use the attached template as %USERNAME%\.spyder-py3\template.py (the attached file is the one without the extra % sign.  I've uploaded it as template_py.txt as github won't let me upload it to an issue as a python file.
template_py.txt
Create a new file in Spyder.

What is the expected output? What do you see instead?
Should see section marks but don't.
Paste Traceback/Error Below (if applicable)
N/A
Versions

Spyder version: 4.0.1
Python version: 3.7.6 64-bit
Qt version: 5.9.6
PyQt version: 5.9.2
Operating System name/version: Windows 10

Dependencies

atomicwrites >=1.2.0         :  1.3.0 (OK)
chardet >=2.0.0              :  3.0.4 (OK)
cloudpickle >=0.5.0          :  1.2.2 (OK)
diff_match_patch >=20181111  :  20181111 (OK)
intervaltree                 :  None (OK)
IPython >=4.0                :  7.11.1 (OK)
jedi =0.14.1                 :  0.14.1 (OK)
nbconvert >=4.0              :  5.6.1 (OK)
numpydoc >=0.6.0             :  0.9.2 (OK)
pexpect >=4.4.0              :  4.8.0 (OK)
pickleshare >=0.4            :  0.7.5 (OK)
psutil >=0.3                 :  5.6.7 (OK)
pygments >=2.0               :  2.5.2 (OK)
pylint >=0.25                :  2.4.4 (OK)
pyls >=0.31.2;<0.32.0        :  0.31.2 (OK)
zmq >=17                     :  18.1.0 (OK)
qdarkstyle >=2.7             :  2.7 (OK)
qtawesome >=0.5.7            :  0.6.0 (OK)
qtconsole >=4.6.0            :  4.6.0 (OK)
qtpy >=1.5.0                 :  1.9.0 (OK)
rtree >=0.8.3                :  0.8.3 (OK)
sphinx >=0.6.6               :  2.3.1 (OK)
spyder_kernels >=1.8.1;<2.0.0:  1.8.1 (OK)
watchdog                     :  None (OK)
cython >=0.21                :  0.29.14 (OK)
matplotlib >=2.0.0           :  3.1.1 (OK)
numpy >=1.7                  :  1.18.1 (OK)
pandas >=0.13.1              :  0.25.3 (OK)
scipy >=0.17.0               :  1.3.2 (OK)
sympy >=0.7.3                :  1.5.1 (OK)

How about having a section variable to make sections in the template which will be replaced to #%%
example template:
`# -- coding: utf-8 --
"""
Created on %(date)s
@author: %(username)s
"""
#%(section)s Initialization of Libraries
import sys, os, inspect
currentdir = os.path.dirname(os.path.abspath(inspect.getfile(inspect.currentframe())))
parentdir = os.path.dirname(currentdir)
sys.path.insert(0,parentdir)
parentdir_1up = os.path.dirname(parentdir)
sys.path.insert(0,parentdir_1up)
#%(section)s Standalone Run
if name == 'main':
pass`
Output new file:
`# -- coding: utf-8 --
"""
Created on Wed Jan 13 02:41:46 2021
@author: Tahasanul Abraham
"""
#%% Initialization of Libraries
import sys, os, inspect
currentdir = os.path.dirname(os.path.abspath(inspect.getfile(inspect.currentframe())))
parentdir = os.path.dirname(currentdir)
sys.path.insert(0,parentdir)
parentdir_1up = os.path.dirname(parentdir)
sys.path.insert(0,parentdir_1up)
#%% Standalone Run
if name == 'main':
pass`
To that I have just edited the plugin.py file from Anaconda3\Lib\site-packages\spyder\plugins\editor\widgets at line 1825 where the Dictionary Variable VARS will hold 3 keys instead of two.
                VARS = {                     'date': time.ctime(),                     'username': username,                     'section': '%%',                 }
Adding a pull request if the developers like it they can take it in. Thank you

How about having a section variable to make sections in the template which will be replaced to #%%
example template:
`# -- coding: utf-8 --
"""
Created on %(date)s
@author: %(username)s
"""
#%(section)s Initialization of Libraries
import sys, os, inspect
currentdir = os.path.dirname(os.path.abspath(inspect.getfile(inspect.currentframe())))
parentdir = os.path.dirname(currentdir)
sys.path.insert(0,parentdir)
parentdir_1up = os.path.dirname(parentdir)
sys.path.insert(0,parentdir_1up)
#%(section)s Standalone Run
if name == 'main':
pass`
Output new file:
`# -- coding: utf-8 --
"""
Created on Wed Jan 13 02:41:46 2021
@author: Tahasanul Abraham
"""
#%% Initialization of Libraries
import sys, os, inspect
currentdir = os.path.dirname(os.path.abspath(inspect.getfile(inspect.currentframe())))
parentdir = os.path.dirname(currentdir)
sys.path.insert(0,parentdir)
parentdir_1up = os.path.dirname(parentdir)
sys.path.insert(0,parentdir_1up)
#%% Standalone Run
if name == 'main':
pass`
To that I have just edited the plugin.py file from Anaconda3\Lib\site-packages\spyder\plugins\editor\widgets at line 1825 where the Dictionary Variable VARS will hold 3 keys instead of two.
                VARS = {                     'date': time.ctime(),                     'username': username,                     'section': '%%',                 }
Adding a pull request if the developers like it they can take it in. Thank you


not work with css, less and scss

problem with css, less and scss files
v.0.3.0 - work fine
v.0.3.1 - not work

This will be fixed by tonight, adding support for /* */ style comments, just writing some tests. This repo has had some major entropy due to my negligence but I finally have time to get it back up to a really solid place.

resolved


Intermediate value is not a function?

Here is the source code I am running on Windows...
(async function() {
  const instance = await phantom.create();
  const page = await instance.createPage();
  await page.on('onResourceRequested', function(requestData) {
    console.info('Requesting', requestData.url);
  });

  const status = await page.open('https://stackoverflow.com/');
  const content = await page.property('content');
  console.log(content);

  await instance.exit();
})();

Expected behavior
Just want the output to display source code
Actual behavior
(async function() {
^
TypeError: {(intermediate value)(intermediate value)(intermediate value)(intermediate value)} is not a function
at Object. (C:\indospace.io\proxybot\proxybot.js:54:1)
at Module._compile (module.js:624:30)
at Object.Module._extensions..js (module.js:635:10)
at Module.load (module.js:545:32)
at tryModuleLoad (module.js:508:12)
at Function.Module._load (module.js:500:3)
at Function.Module.runMain (module.js:665:10)
at startup (bootstrap_node.js:187:16)
at bootstrap_node.js:608:3
Environment

OS: Windows 10
Node version (node -v): 8.7.0
package version:

I think this is something you are doing as it is complaining about async not being a valid keyword. Check your node or try running another async example first.

Yup! I ended up taking the code and putting it an individual file and it works.  Probably some semicolon or weird character encoding.


all: ordering numeric values, including NaNs, and the consequences thereof

There are thorny issues around ordering of numerical values, and rather than add to existing conversations I thought it worth pulling them into their own conversation.
The questions here touch on math.Compare (issue #56491) as well as the proposed ordered constraint (issue #59488). That said, I feel there are higher level issues around comparison that should be discussed and resolved before making any other changes to the library or language. I also feel that math.Compare should be pulled for now, not released, until these issues are better understood.
Go provides less than and less-than-or-equal operators that apply to strings, integers, and scalar numbers. (There is no ordering for complex values.) The proposal to add an ordered constraint somewhere is a priori reasonable as it allows one to write comparators, finders, and sorters that apply to all such types.
Except they don't, not quite. The problem is IEEE-754 NaNs. It is a peculiarity of IEEE-754 and its implementation in both hardware and most languages that NaNs do not behave like other numbers under comparison. All "less than" and related comparison operations, including "equals", return false when comparing NaNs, even when the bit pattern of the operands is identical. This is specified by IEEE-754 and, although odd, provides an easy way to see if a value is a NaN: if it is neither equal nor unequal to itself, it is not a number.
This behavior is the key reason why the map clear builtin was added: One cannot pull out all the values of a map to clear it as it is never possible to match a NaN key.
Similarly, sorting and such will misbehave in the presence of NaNs. This is why math.Compare was added, to provide a total ordering on floating point. But of course, the sort package does not use it, nor could a generic sorter, at least absent a generic compare, which is possibly what should happen instead.
Mischievously, this total ordering of NaNs is also defined by IEEE-754 but is not the standard behavior of less than, either as a hardware instruction or in a programming language.
Thus NaNs are ordered but not by the comparison operator. The consequences of this must be clear before defining any ordering constraint.
That is, the addition of an ordered constraint to the language, however it is done, bumps up against this conflict. If it is defined, as suggested, to be any type that implements the < less than operator, any function that uses that constraint may fail unpredictably if a NaN arises, as IEEE-754 demands. On the other hand, if it is instead defined using the total ordering of math.Compare, programs will always work reliably but the < operator is then not allowed, which is bizarre for a constraint defining an ordering.
Neither of these paths is satisfactory. So what should we do? Short answer, I don't know, but we certainly should not change the language or libraries until we have thoroughly thought this through.
I do not, at least at the moment, propose a solution. Instead I am isolating the issue in a discussion to make sure we do the right thing with ordered, so we don't end up in a situation like the one for maps where the language had to be changed to make them behave properly.
Discuss.

No answers, just some notes.
Java implements a class java.lang.Double.  This is an object that contains a single value of type double(same as Gofloat64).  Double has anequalmethod that is mostly like using==on adouble, except that calling equalon two NaN values returnstrue, and calling equalon a positive zero and a negative zero returnsfalse.  Similarly, Doublehas acompareTomethod and acompare` function that reliably sort NaN after all other values (including infinity) and sort positive zero after negative zero.
In Rust the trait core::cmp::Ord (roughly equivalent to Go's golang.org/x/exp/constraints.Ordered) does not permit floating-point types.  The Rust trait core::cmp::PartialOrd does permit floating-point types, the difference between that the PartialOrd comparison method is permitted to return None meaning that there is no ordering.  Sorting a vector of floating-point numbers requires specifying an ordering function to use.
C++ and Python programmers are best advised to not sort vectors containing NaN values.
In Go we have sort.Float64s today.  It sorts NaN values before all other values (which is not what the unreleased math.Compare function does).

A conservative approach is to make the Ordered constraint, wherever it winds up, not permit floating-point types.  The effect would be that generic functions like sort.Min (or cmp.Min, whereever it winds up) and slices.Sort would not permit floating-point types.  This might be occasionally annoying, but it would not be hard to provide a math.Less function that could be used with slices.SortFunc.
However, I have a feeling that the effect would be that people who want to sort a slice of floating-point numbers would reach for slices.Sort and, when it fails, shake their heads in exasperation, do a web search, and call slices.SortFunc without thinking about it further.
So there is another approach we can take.  Don't permit floating-point types for Ordered.  Add a new constraint, maybe called PartiallyOrdered although there is probably a better name, that does permit floating-point types.  Define slices.Sort and sort.Min to take a PartiallyOrdered constraint, and write them to handle NaNs correctly (this can be done in a general and efficient manner as described at https://github.com/golang/go/issues/59488#issuecomment-1500913885).
With this approach, the naïve user will use Ordered and the right thing will happen. The sophisticated user will use PartiallyOrdered and, we hope, learn about how to write their comparisons to handle NaNs.

With this approach, the naïve user will use Ordered and the right thing will happen. The sophisticated user will use PartiallyOrdered and, we hope, learn about how to write their comparisons to handle NaNs.

In my experience with Rust, the situation I've seen tends to be more like, "the naïve user will use PartialOrd because they want their code to work with f64, and the sophisticated user will use Ord because they understand why PartialOrd exists." That might be a matter of documentation.
I also note that narrowing a constraint from PartiallyOrdered to Ordered (e.g. to choose < or math.Compare in a generic Less) would be especially awkward. It seems like this would make me want #45380 even more.

@zephyrtronium I'm not sure I entirely understand your comment about narrowing the constraint.  To be clear, you could still use the < operator with the PartiallyOrdered constraint.
@dr2chase A good error message would help, of course, but my guess is that most people who want to sort floating-point numbers simply don't care about NaN values at all.  So it's weird to tell them that the obvious function doesn't work.  Especially since we can choose to make it work.

I always assumed that the use-cases for < and math.Compare were
Sort(slice)
SortFunc(slice, math.Compare)

respectively. It provides an opportunity to opt into a different comparison predicate, while preserving existing behavior. The XFunc variants of generic APIs in general allow for this, which is also why https://github.com/golang/go/issues/58559 is relevant, since you can just write SortFunc(slice, operators.Lt).
That being said, I agree that Ordered as a name for a constraint is kind of a misnomer when some of the values it applies to are not comparable. Heck, slices.Sort even documents that it misbehaves with NaN. PartiallyOrdered is a much better name to encapsulate types that support <. It was even mentioned in the discussion at https://github.com/golang/go/issues/34915 that Less must be a partial order.
Does renaming Ordered to PartiallyOrdered address this issue? Is Ordered even necessary at that point?

I forgot to mention the delights of signed zeros. Not as problematic as NaNs, but again inconsistent with regard to traditional "less than" vs. total ordering:  https://gotipplay.golang.org/p/qd16ohSUz_p
package main

import (
	"fmt"
	"math"
)

func main() {
	pzero := math.Float64frombits(0)
	nzero := math.Float64frombits(1 << 63)
	fmt.Println(pzero, nzero, nzero == pzero, nzero < pzero, math.Compare(nzero, pzero))
}

prints
0 -0 true false -1

@ianlancetaylor

However, I have a feeling that the effect would be that people who want to sort a slice of floating-point numbers would reach for slices.Sort and, when it fails, shake their heads in exasperation, do a web search, and call slices.SortFunc without thinking about it further.

I'm don't understand the problem with this. Wouldn't slices.SortFunc(s, math.Compare) do what they'd (in all likelihood) do what they want? And if the documentation of slices.Sort mentions that that's what they should use for floating point numbers, wouldn't the web search in all likelihood point them to that as well?
I can see the exasperation as a bit unfortunate, but it seems like a decision that's easy enough to defend - there is no canonical answer for how to order floats, ergo you have to be explicit about it.

An update on the current status.
We removed math.Compare (#60519).
We added cmp.Compare and cmp.Less (#59488).  These work for all ordered types including floating-point types.  They document how they handle NaN and negative zero (any NaN equals any other NaN, any NaN is is less than any non-NaN, negative zero equals positive zero).
We added the cmp.Ordered constraint (also #59488).  It permits floating-point types.  We document that people who care about NaN should use cmp.Compare or cmp.Less rather than == or <.
I don't know whether there is anything else to do here.  I think we're an OK place right now, but I'm definitely too close to the problem.