Inform the user about BSD and LGPL libs

As you can see here: minimax.h
And on many other places around the code, Godot uses BSD and LGPL software without informing the users.
I'm not a lawyer but I think while on source version might not be an issue, is totally not fair to distribute without at least a LICENSES file that lists those libraries.
I'd put this issue to a high priority since those people who made the libraries must have their credits given.
It might sound stupid, but tomorrow, the same thing could happen to me and you.

Given that minimax.h comes directly without modifications from the first commit to the repo, and has copyright of 1999-2004, I guess that was way before anyone was thinking of GPL and similar licenses...

Surely it's been a long time, and wasn't on purpose, but that's what it is.
The average user won't give a shout about the licenses and someone must credit those programmers.

We shouldn't be using LGPL anywhere, because we don't distribute any shared
objects. This file seems to be the only one that's LGPL in that library
(everything else is some sort of BSD), can we replace it? It's a bit silly,
it's all macros that probably already exist somewhere else in our code, and
they're pretty common anyway, but we don't want LGPL stuff. I assume we
looked up the license of Musepack back then and it came up as BSD and we
didn't see that file. Otherwise just remove it, with Opus it might not be
necessary any more.
I agree that we should maintain one big file with everything that needs to
be included for distribution, maybe put it in the exports, or somewhere
inside the binaries if it complies with all the licenses.
On 29 December 2016 at 05:59, Alex<EMAIL_ADDRESS>wrote:

Surely it's been a long time, and wasn't on purpose, but that's what it is.
The average user won't give a shout about the licenses and someone must
credit those programmers.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/7388#issuecomment-269600237,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVmPcAW_Dh75hrvTwQgCYBYv9tJaVI8ks5rM3ZfgaJpZM4LXbHT
.

Its a little bit confusing. The website of musepack shows the license of libmpcdec as BSD, but when you download the tarball you will find precisely those header under the LGPL license. The advertised successor is licensed LGPL. I'm not sure what the authors intended. If the library is seen as keep-worthy, best would be probably to contact the author, and ask about the license, maybe to get written agreement that the files can be used under BSD as well, otherwise the library should just be removed.

I agree about contacting the authors of libmpcdec.
About the other libs I've made a quick(and dirty) script to find licences: http://pastebin.com/Rh1EUgvK
This is what I've came up so far:
http://pastebin.com/2CuCtFXM
It's far from perfect but it's something.
(Note the !!ISSUE marked libs are some I didn't understood the license)

Appirater should be removed, I don't think it was ever enabled..
On 2 January 2017 at 05:28, Alex<EMAIL_ADDRESS>wrote:

I agree about contacting the authors of libmpcdec.
About the other libs I've made a quick(and dirty) script to find licences:
http://pastebin.com/Rh1EUgvK
This is what I've came up so far:
http://pastebin.com/2CuCtFXM
It's far from perfect but it's something.
(Note the !!ISSUE marked libs are some I didn't understood the license)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/7388#issuecomment-269944222,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGVmPbsShflb67bAWrHX5DWCBRKn_00Tks5rOLUogaJpZM4LXbHT
.

For the reference: https://github.com/godotengine/godot/issues/6745#issuecomment-256144607
I don't think we should document the sublicenses of hidden headers of thirdparty libraries - we should give for each library the license(s) that upstream projects claim to have (what I've started in thirdparty/README.md), and if they did not do the legal legwork themselves to give credits where due (in the case of BSD or MIT licensed components), it's their problem.
Copylefted code hidden in a BSD licensed library is another issue though as it impacts us, so the file should be removed or the library altogether.
Then we also have some code that is not in thirdparty/ and that was taken from thirdparty sources, especially some header-only libraries or code snippets from other projects that were adapted to Godot (SDL2, Bullet, minizip). I had started referencing them here: https://github.com/godotengine/godot/issues/5248

All right, starting to work on this for real now that most thirdparty code has been properly moved to the thirdparty/ directory.
I'll use @Alex-doc's work as well as the nice licensecheck tool to try to get something exhaustive.

For the reference re: the above discussions, Appirater was dropped from master and 2.1, and libmpcdec was dropped from master.

Here's the first part: copyright.txt
It lists the copyright and licensing info for all Godot files minus (for now, will be added later) the thirdparty/ directory, using Debian's machine readable debian/copyright standard format: https://www.debian.org/doc/packaging-manuals/copyright-format/1.0/#copyright-field
For thirdparty libraries I'll use a similar syntax, but I won't go into the details of every single file of the thirdparty code. For libraries, I'll stick to the "main" license and copyright advertised by the upstream project - it's their job to go further down.

Here's an updated version with all thirdparty content documented:
copyright.txt
It's not the prettiest or the most readable output, but for such a boring job as listing licensing details for every single file in the code repo, it does the job :)
We'll see if we can provide a human-readable version of it that could be used e.g. to go with distributed games, or an HTML page on our website that game developers could link to.


VSCode Integrated Terminal - ZSH Tab completion bug

System information

VSCode Version: 1.47.1
OS: Linux x64 5.4.52-1-MANJARO

Issue Summary
Since I started using zsh as the default shell in VSCode, Tab completion started behaving weirdly. Whenever I hit Tab, it screws up the prompt.
Steps to Reproduce:

Using VSCode Terminal (zsh), type an incomplete line of code then hit Tab for suggestion
Error is observed.

Please see some screenshots of this issue below. On the right hand side you can see zsh behaving normally in my system's terminal emulator Konsole.

Tab Completion for Filename

Tab Completion for command-line arguments

Does this issue occur when all extensions are disabled?: Yes

Unconfirmed, my guess is that zsh is doing something that the terminal doesn't support, maybe there is a zsh config that will disable whatever that is.

Hi @Tyriar
I found out that VSCode Terminal doesnt like this symbol: ▶ on my zsh prompt, so I replaced it with '>' and it's working fine.
I'm not sure the root cause of this though.. since ▶ works fine with bash as the Default Shell.
Regards,

For me, it seemed the problem was special UTF8 characters. But i added that to the LANG variable to my run commands and the tab completion seems to work fine now.
# You may need to manually set your language environment
export LANG=en_GB.UTF-8

This situation has since improved with the introduction of unicode 11 support which defines how wide the characters are expected to be. Things that you can try to work around issues like this:

Tweak terminal.integrated.unicodeVersion
Tweak terminal.integrated.detectLocale


Hello this day and i played 1 wingman and enemy they wrote mee im sliding and my body is sideways some helps? i dont have on any features and it doing me :(

Config->Reset Config-> Whole

Make sure you have "Moonwalk" option disabled (in "Misc").


feat(testlib): add graphql-toolkit-parser test vectors crate

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 86.79%. Comparing base (309cb33) to head (770999f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #44   +/-   ##
=======================================
  Coverage   86.79%   86.79%
=======================================
  Files           5        5
  Lines         780      780
=======================================
  Hits          677      677
  Misses        103      103

Flag
Coverage Δ

integration
87.80% <ø> (ø)

unit
0.00% <ø> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[stable/kube-state-metrics] Bump to v2.0.0-alpha

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Bumps version to https://github.com/kubernetes/kube-state-metrics/releases/tag/v2.0.0-alpha
I fixed some changed variables.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @mrueg

Thanks for your contribution. I would prefer not to add any alpha releases but wait for the proper 1.1.0


bug: song stats not working

🔍 Is there already an issue for your problem?

[X] I have checked older issues, open and closed

ℹ Environment / Computer Info
- Spotify version: <IP_ADDRESS>2
- Spicetify version: 2.38.5

📦 Extension name
song stats
📝 Description
I downloaded everything correctly but when I play a song and click on "View Song Stats" the popup window is just blank.
📸 Screenshots

Works just fine on my end. Reinstall or download it locally.


Clicking on songs gives MediaPlayer error(-38,0) on occasions

Clicking on songs sometimes doesnt play and on debugging it shows error (-38,0) and a message Attempt to call getDuration without a valid mediaplayer
Clicking on it again starts playing. This happens only when I am clicking on songs and not when the player is playing one song after another
Here I found a similar problem. Apparently the start() method is being called before prepareAsync completes its task.
http://stackoverflow.com/questions/18482018/mediaplayer-error-38-0

I've run into this before, but I haven't been able to consistently reproduce it. It definitely needs to get fixed before the next stable release, but it'll probably take a while to figure out what's causing this since it sounds like a race condition.
Let me know if you have more information about how to reproduce this, otherwise I'll try to look into it when I get a chance.

For some reasons frequency has increased. Almost after every 5-7 clicks this is appearing.
I encountered another exception that is dead object exception in the stub proxy.

Jockey has switched to ExoPlayer for media playback as of #146, so this error is no longer possible.


Error parsing jail list

The list of jails is not parsed correctly as there's a  instead of a  in front of the first jail.
Tested with Fail2Ban v0.11.1.
The fix this issue i would suggest to replace the space in line 100 at the last sed-command with \s as this will remove all whitespace characters.

I never encountered this issue as it's parsed correctly, even though there is a tab in front of it. However, removing all whitespaces is generally a better idea. Thanks for the report, added in latest release


ENH: Add controlled-H and controlled-Y gate

Add controlled H gate and controlled Y gate and test for them.

close #55


[Bug] cmd_texture doesn't release renderer reservation

Describe the bug
After using cmd_texture, the renderer queue is left full until a bot restart.
This was fixed in the temp-xmas branch here: https://github.com/GOF2BountyBot/GOF2BountyBot/commit/40af47cea93be6cf27dcb309be91405da5c84eb0
Steps to reproduce
Steps to reproduce the bug. E.g:

use $tex with valad arguments
wait for composited textures
use $showme ship with valid arguments
observe error

Message link
https://discord.com/channels/226528541368385536/456218562428796929/946609017692778496

fixed by not making a reservation


Add PoW for browser parallelized with Web Workers

This method should take two params:

difficulty (number of zeros in hash)
and buffer of data.

The data will be concatenated with a nonce and hash will be calculated. The hash should contain the difficulty number of 0 in the beginning.
Use https://www.npmjs.com/package/worker-task-runner for parallelizing the task.

PoW PR: https://github.com/ilyavf/tx-builder/pull/7
No web workers yet.


install problem

when i install, this error comes up. can anyone help?

problem solved. os version is ubuntu16, need higher version of cmake, g++ and gcc. installation completed.


Update emscriptenPackages.xmlmirror

emscriptenPackages.xmlmirror is out of date.
It's hard to see because it don't has a version number.
I add that in another PR.
There are some new commits since then: https://gitlab.com/odfplugfest/xmlmirror/commits/master
cc @qknight

it has a version now, but can get updated again


Support for Nvidia GPUs

After installation using pip of the latest version of panpipes, when running integration step, the scvi falls back on CPU even if running on CPU thread, because of lack of jax. Running pip install --upgrade "jax[cuda11_pip]" -f https://storage.googleapis.com/jax-releases/jax_cuda_releases.html as suggested here solved the problem.

I did clone and install yesterday and the issue was still there, in the ran_scvi.log:
INFO - raw counts found in layer
INFO - Unable to initialize backend 'cuda':
INFO - Unable to initialize backend 'rocm': module 'jaxlib.xla_extension' has no attribute 'GpuAllocatorConfig'
INFO - Unable to initialize backend 'tpu': INTERNAL: Failed to open libtpu.so: libtpu.so: cannot open shared object file: No such file or directory
WARNING - An NVIDIA GPU may be present on this machine, but a CUDA-enabled jaxlib is not installed. Falling back to cpu.
I did see in scvi-tools documentation they suggest running
conda install pytorch torchvision torchaudio pytorch-cuda=11.7 -c pytorch -c nvidia
conda install jax jaxlib -c conda-forge
in addition to installing scvi, so it may be an alternative to my original solution.

@Zethson could you please pass this on to the scvi team?

we have reinstated jax as the first installed dependency, however i can't guarantee that the order of installation will effectively ensure that the GPU is recognised. we'll do further testing, and add info in the installation to warn the users of this behaviour (which is not due to panpipe), which can be solved with the additional installation step as you mentioned. thanks for reporting @wlason !


clean up front (semantic-ui)

the front gui got a little messy now with tiny icons, move some stuff to the settings menu maybe...  I moved the recycle name to the field but I still feel like it should be cleaner.  I think open project should only be on the dialog prompt for new / update not on the main gui....  I also think re-scan addons could be in the settings menu.   will move some stuff around now.

Sounds good, I agree it messes it up a little.

Only the BrowseProjectFiles icon remains for now in update mode. Others appear when needed.


Not all stage-specific parameter initialized

cli_params_def.py contains stage-specific parameter definitions that are set up in the corresponding classes. However, when used in the examples, the following options are not set up; they are missing from the help prompt and cannot be passed via CLI:
- learning_rate and pred_col_name_suffix (examples/train_examples.py),
- y_data_preds_suffix, json_scores_suffix, and pred_col_name_suffix (examples/infer_examples.py)
I suspect some inconsistencies between setting up parameters by passing a dict of options and by specifying them in the config file

The issue seems to be resolved. Please reopen in case of reoccurrence.


Is it safe to run OpenDevin?

OpenDevin is an agent and I am super worried whether it is safe to run OpenDevin or not.
For example, it can steal sensitive information, password etc.
How do you guarantee it is safe to run it in a local machine?

Don't send sensitive information to it and you won't have cause to worry. The same goes for ChatGPT, Gemini, or any other model.

You can use a local llama server

I am not worrying about ChatGPT, Gemini. These are established company.
I am more about about the OpenDevin code itself.

well you can check code by your self... I am checking it to extend this or help with the project... seems to be clean... nothing weird for now... :smile:

I am not worrying about ChatGPT, Gemini. These are established company.
I am more about about the OpenDevin code itself.

Please don't post sensitive information to any model. There is absolutely no good reason to do so.

@code2graph there is most likely a risk since the model is capable of creating and executing its own code. It all really depends if you point the model in a direction that is a problem. For example if you gave it the workspace of sys32 that would be a really bad idea. It is most likely safe to run if you don't give it files like @kjenney said and you don't try to make it do something malicious. With LLM's there is always a risk that they hallucinate something and go off the rails so if your computer has really important information you should look into using a VM or something similar to mitigate risks.

We should add more information about the safety measures that are in place with OpenDevin (docker sandbox, etc.) and why those are still not perfect, to the README.

I am not worrying about ChatGPT, Gemini. These are established company.

Yes, but you actually don't know what they do.

I am more about about the OpenDevin code itself.
For example, it can steal sensitive information, password etc.

All code are opensource, so you don't need to worry it will steal sensitive information on purpose. You can chcek the code logic.

there is most likely a risk since the model is capable of creating and executing its own code

Agree with that, and that is one of the reason why we run it in a sandbox/docker. Still more improvement and welcome any suggestions.


sleep() before interactive() causes freeze

The code below run as expected, printing the git diff help, then result StillAlive, then exiting.
fn main() {
    let mut cmd = std::process::Command::new("git");
    cmd.args(["diff"]);
    cmd.current_dir("/tmp");
    let mut session = expectrl::Session::spawn(cmd).expect("Can't spawn a session");
    let mut opts = expectrl::interact::InteractOptions::terminal().unwrap();
    let res = opts.interact(&mut session).unwrap();
    println!("result {:?}", res);
}

This code freezes, printing nothing and not exiting:
fn main() {
    let mut cmd = std::process::Command::new("git");
    cmd.args(["diff"]);
    cmd.current_dir("/tmp");
    let mut session = expectrl::Session::spawn(cmd).expect("Can't spawn a session");
    std::thread::sleep(std::time::Duration::from_millis(300));
    let mut opts = expectrl::interact::InteractOptions::terminal().unwrap();
    let res = opts.interact(&mut session).unwrap();
    println!("result {:?}", res);
}

Hitting return causes the program to continue as before. It seems like the delay is causing expectrl to enter the interactive mode in a blocking way, but that's not quite right because interactive mode should require ^] to exit, not just a return. Also, there's no input command here awaiting a return.
The expected behavior would be that the output is exactly as before but delayed 300ms.

So it was a while :)
But I guess I figure something out :smile:
The issue was not why it doesn't work on freebsd and macos.
But why it works on linux :smiley:
Have no clue yet why.
But essentially when process is exists.
We can't no longer read anything.
On all platforms.
on linux we get a error in such case but on macos and freebsd we get nothing just an empty stream of data.
I would say we can't do anyting with this at least I don't yet see.
I'll update the doc of interact function accordingly.
But overall that's it.
No surprises :disappointed:
PS: My comment about not stopped macos processes though is interesting. It may need an investigation cause I don't remember what it was :smiling_face_with_tear:


NUTCH-1870 XSL parse filter

apply patch contributed by @albinscode
load configuration files from classpath and address thread-safety

Note: not ready yet:

TODOs in code
unit tests fail (with DOM built by tagsoup parser)
see also open points in NUTCH-1870

Hi @sebastian-nagel, was going through this. Out of curiosity why hasn't this still merged?
I see in the discussions everyone is ok with the code. And it doesnt have any merge conflicts.

See the TODOs in the comment here and Jira.


feat: custom return routes management

Fixes #3129

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 162 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 22.374%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/stores/base.js
0
4
0.0%

lib/ZwaveClient.ts
0
158
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
3834

Relevant Lines:
18116

💛  - Coveralls

Further TODOs:

[ ] In the return routes list, put the priority route always on top and do not make it draggable
[ ] Table headers for the return routes (if it looks somewhat ok?)
[ ] in the map, show a star on the priority return route too
[ ] do not show "route failed between undefined --> undefined, undefined --> undefined" in the priority route editor
[ ] add toggles to turn off rendering priority and return routes (can get chaotic)


Update node-updates-kured.md

line 70 fails in Linux shell:
the dot in "kubernetes.io/os" needs to be escaped.  Modified command:
helm install my-release kubereboot/kured --namespace kured --set nodeSelector."kubernetes.io/os"=linux
Tested in Ubuntu 20.04

@MGoedtel

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

I've tested and confirmed the command fails and the proposed formatting change enables it to complete successfully. #sign-off


x/pkgsite: remove github.com/manuelcoppotelli/flatten

What is the URL of the page with the issue?
https://pkg.go.dev/github.com/manuelcoppotelli/flatten
What did you expect to see?
As per instructions at https://go.dev/about for removing a package: please remove the whole https://pkg.go.dev/github.com/manuelcoppotelli/flatten module.
I am the creator of this module.
It is an old module and neither the git repo nor the module exist anymore.
Thank you

Done.


Add additional options for generating imports

Including imports is not enabled by default in the Buf CLI. As a result, users have been getting tripped up on generating code when they have dependencies on protovalidate.
This adds some additional guidance in our generating code docs to make sure they include imports since these should essentially always be included when generating with Protobuf-ES.

@timostamm I added include_imports to all snippets of buf.gen.yaml just to reinforce that it should be there. But if you'd rather it be listed only inRegenerating code, lmk.


Prepare release version

Code polish

[x] Remove imgui
[x] Disable mouse cursor
[x] Application to find relative asset folder
[x] Disable camera control debug
[x] Add splashscreen
[x] Disable keyboard controls
[x] Remove networking stuff (to avoid the firewall warning)
[x] Control console presence (on Windows)

Packaging

[x] Add app icon

@Fferric If you want to take a look at the resulting app, if there is anything else to disable?


Build optimization: this will configure to use the more secure SHA-256 based passw…

…ord authentication used in MySQL 8+ instead of the insecure mysql_native_password plugin.
To use caching_sha2_password with docker compose, you need to do the following steps:
1,Open your docker-compose.yml file and locate the service definition for your MySQL server. Add or modify the following line under the command key:
command: mysqld --default-authentication-plugin=caching_sha2_password
2,Restart your MySQL server container to apply the changes.
3,Connect to your MySQL server using the root account and execute the following SQL command to change the authentication plugin for the root user:
ALTER USER 'root'@'localhost' IDENTIFIED WITH caching_sha2_password BY 'xagent';

Close this pr because of wrong commit format. You should not put everything in commit title and should be careful about commit body. Please kindly refer to here.

描述字段必须直接跟在 <类型>(范围) 前缀的冒号和空格之后。 描述指的是对代码变更的简短总结，例如： fix: array parsing issue when multiple spaces were contained in string 。
在简短描述之后，可以编写较长的提交正文，为代码变更提供额外的上下文信息。正文必须起始于描述字段结束的一个空行后。


RKE BYOC test

What this PR does / why we need it:
Which issue(s) this PR fixes (optional)
Closes #
Special notes for your reviewer:

Closing as new PR is raised


Move some more common enum diagnostics into the decl checker

In preparation for checkEnumRawValues being turned into a request, move the common diagnostics to the decl checker so they're always emitted at the right time.

@swift-ci please smoke test

Sorry, what's the actual change here?

checkEnumRawValues diagnoses its parent enum in a few places.  This pulls those diagnostics into the decl checker.  This code path is about to become lazy, and those diagnostics need to be emitted for every enum decl in the primary.

Isn't checkEnumRawValues already called from the decl checker, though?

It is.  I wanted to move these someplace conceptually cleaner since these diagnostics don't have much to do with the result of the work the request is going to do - and also I think this'll let me drop the call to getInterfaceType() later.

Thanks
⛵

Okay, I guess it does seem reasonable to have the diagnosing be separate from the value computation.


🚀 Feature: Add database cleaner templates

Appwrite Functions allow you to extend and customize your Appwrite server functionality by executing your custom code. 🤩 You can learn more at our official Appwrite Functions docs.
What you’re building
Your task is to implement the database cleaner template in Node.js.
Database cleaner integrates with Appwrite Databases and removes old documents from the collections older than X number of days, as configured by the RETENTION_PERIOD_DAYS environment variable. The Database cleaner should be scheduled using CRON, defaulting to running at 1:00 AM every day.
This can be useful when you have apps that receive a lot of write traffic but don’t need super long retention, such as live location updates of IoT devices or chat logs in a video game.
Your function’s implementation should be similar to other Node.js function templates in code format and style. Make sure to follow our contributing guide.
Tasks summary:

[ ]  Fork & clone Appwrite's templates repository
[ ]  Create a branch feat-implement-database-cleaner-node
[ ]  Write code for the function in folder node/database-cleaner/.
[ ]  Test the function using Appwrite, provide a video.
[ ]  Write a README.md explaining what the function does, how to use it, and customize it. Here is an example.
[ ]  Submit a pull request to our templates repository

If you need any help, contact us on our Discord server.
Are you ready to work on this issue? 🤔 Let us know, and we will assign it to you 😊
Happy Appwriting!

I'd like to work on this issue. :)

I'd like to work on this issue. :)

Hi @dishwasher-detergent  have assigned the issue to you! Thank you for showing interest in contributing to Appwrite! Happy Hacktoberfest 🎃
Notes:
Please update us with your progress every 3 days, so that we know that you are working on it.
Join us on Discord - https://appwrite.io/discord to chat about Hacktoberfest and Appwrite!

Should this be taking in the collection you are wanting to clean via the environment variables or should this just clean all collections in a project?

Are we wanting to explicitly tell it what collection to clean based on an environment variable, or are we wanting this to just clean all collections in a project?

As the description says, we want it to clean the collections

hi @Haimantika , i would like to work on this in case @dishwasher-detergent  drops.

@Haimantika I would to contribute for this feature.

@Haimantika I would to contribute for this feature.

There is already an approved PR out for this.


[bug] WebSocket HMR not working as expected with Nuxt

Describe the bug
The methods described in this document  does not mark HMR work on mobile devices.
Because Nuxt does not use vite's HMR config. Nuxt rewrite vite config, and uses ws://$host:$port/_nuxt/ to provide HMR:
09-27 18:01:57.473  9019  9019 E Tauri/Console: File: http://tauri.localhost/_nuxt/@vite/client - Line 535 - Msg: WebSocket connection to 'ws://tauri.localhost/_nuxt/' failed: Error in connection establishment: net::ERR_CONNECTION_REFUSED
09-27 18:01:57.473  9019  9019 E Tauri/Console: File: http://tauri.localhost/_nuxt/@vite/client - Line 535 - Msg: Uncaught (in promise) SyntaxError: Failed to construct 'WebSocket': The URL 'ws://localhost:undefined/_nuxt/' is invalid.

It tries to connect to localhost:undefined and tauri.localhost, but this is incorrect.
All endpoints have been tried and there is no WebSocket support. This may require a direct connection to the host.
Reproduction
From official document: https://v2.tauri.app/start/frontend/nuxt/
adb devices # connect a Android devices ...

pnpm tauri android dev

If this is not detailed enough, please call me for reproduction.
Expected behavior
No error.
Full tauri info output
[✔] Environment
    - OS: Windows 10.0.22631 x86_64 (X64)
    ✔ WebView2: 129.0.2792.52
    ✔ MSVC: Visual Studio Community 2022
    ✔ rustc: 1.81.0 (eeb90cda1 2024-09-04)
    ✔ cargo: 1.81.0 (2dbb1af80 2024-08-20)
    ✔ rustup: 1.27.1 (54dd3d00f 2024-04-24)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (default)
    - node: 20.14.0
    - pnpm: 9.11.0
    - npm: 10.7.0
    - bun: 1.1.28

[-] Packages
    - tauri 🦀: 2.0.0-rc.15
    - tauri-build 🦀: 2.0.0-rc.12
    - wry 🦀: 0.43.1
    - tao 🦀: 0.30.1
    - @tauri-apps/api : 2.0.0-rc.5
    - @tauri-apps/cli : 2.0.0-rc.16

[-] Plugins
    - tauri-plugin-log 🦀: 2.0.0-rc.2
    - @tauri-apps/plugin-log : not installed!

[-] App
    - build-type: bundle
    - CSP: connect-src ws://*
    - frontendDist: ../dist
    - devUrl: http://localhost:3000/
    - framework: Vue.js (Nuxt)
    - bundler: Webpack

Stack trace
09-27 18:02:41.207  9019  9019 E Tauri/Console: File: http://tauri.localhost/_nuxt/@vite/client - Line 535 - Msg: WebSocket connection to 'ws://tauri.localhost/_nuxt/' failed: Error in connection establishment: net::ERR_CONNECTION_REFUSED
09-27 18:02:41.209  9019  9019 E Tauri/Console: File: http://tauri.localhost/_nuxt/@vite/client - Line 535 - Msg: Uncaught (in promise) SyntaxError: Failed to construct 'WebSocket': The URL 'ws://localhost:undefined/_nuxt/' is invalid.
09-27 18:04:34.822  9019  9019 I HwViewRootImpl: removeInvalidNode jank list is null
09-27 18:04:37.867  9019  9019 I HwViewRootImpl: removeInvalidNode jank list is null
09-27 18:04:40.996  9019  9019 E Tauri/Console: File: http://tauri.localhost/__nuxt_devtools__/client/_nuxt/l4ouzdbv.js - Line 8 - Msg: Uncaught (in promise) Error: [birpc] timeout on calling "getOptions"
09-27 18:04:41.307  9019  9019 E Tauri/Console: File: http://tauri.localhost/__nuxt_devtools__/client/_nuxt/l4ouzdbv.js - Line 8 - Msg: Uncaught (in promise) Error: [birpc] timeout on calling "getModuleOptions"
09-27 18:04:41.309  9019  9019 E Tauri/Console: File: http://tauri.localhost/__nuxt_devtools__/client/_nuxt/l4ouzdbv.js - Line 8 - Msg: Uncaught (in promise) Error: [birpc] timeout on calling "getOptions"
09-27 18:04:41.312  9019  9019 E Tauri/Console: File: http://tauri.localhost/__nuxt_devtools__/client/_nuxt/l4ouzdbv.js - Line 8 - Msg: Uncaught (in promise) Error: [birpc] timeout on calling "telemetryEvent"
09-27 18:04:41.314  9019  9019 E Tauri/Console: File: http://tauri.localhost/__nuxt_devtools__/client/_nuxt/l4ouzdbv.js - Line 8 - Msg: Uncaught (in promise) Error: [birpc] timeout on calling "getOptions"

Additional context
No response

I found a workaround:
{
  devServer: {
    host: isMobile ? '<IP_ADDRESS>' : undefined,
  },
  hooks: {
    'vite:extend': function ({ config }) {
      if (config.server && config.server.hmr && config.server.hmr !== true) {
        config.server.hmr.protocol = 'ws'
        config.server.hmr.host = '192.168.XXX.XXX'
        config.server.hmr.port = 3000
      }
    },
  },
  vite: {
    clearScreen: false,
    envPrefix: ['VITE_', 'TAURI_'],
    server: {
      strictPort: true,
      watch: {
        ignored: ['**/src-tauri/**'],
      },
    },
  },
}

But I think undefined appearing in URLs is a bug.

Hey @Sun-ZhenXing can you share a working example, as I am getting a white screen creating a nuxt app on mobile

I dig a little bit further and saw those errors as the culprit of the white screen
11-18 01:04:14.193 15907 15907 E Tauri/Console: File: - Line 138 - Msg: Uncaught TypeError: Cannot redefine property: postMessage
11-18 01:04:14.194 15907 15907 E Tauri/Console: File: - Line 2 - Msg: Uncaught TypeError: Cannot redefine property: metadata
11-18 01:04:14.194 15907 15907 E Tauri/Console: File: - Line 25 - Msg: Uncaught TypeError: Cannot redefine property: TAURI_PATTERN
11-18 01:04:14.194 15907 15907 E Tauri/Console: File: - Line 5 - Msg: Uncaught TypeError: Cannot redefine property: path

@gkkirilov for some reason, when running npx tauri android dev after nuxi generate both folders dist and .output wipe with no reason apparently... Ive taken some snapshots while npx tauri android dev --verbose was running and I got this 5 different both folders state:
FIRST STATE:

SECOND STATE:

THIRD STATE:

FOURTH STATE:

FIFTH STATE:

The wipe of both folders occurs exactly at this part of the logging:

If I remove ssr: false  the blank screen dissapears and content appears as usual but no HMR.
In both cases the initial loading page of Nuxt appears as usual.
Any clues?


Will this plugin be in the addon manager at some point?

Will this plugin be in the addon manager at some point?

I don't own addon manager repository and I am not responsible for its content. Sorry.


Daedalus loses sync with a wallet when sending from a different computer

Platforms: Mac OS High Sierra, Windows 10
Release: Daedalus 1.0.3619
Bug
If I have a wallet with the same seed on two different computers and I do a send transaction from one computer, the other one does not refresh the wallet balance before Daedalus is restarted. If the wallet receives funds subsequently, the first computer will not see them before the application is not restarted. Doing a "soft refresh" with Cmd+R does not solve the problem: Daedalus must be closed and started again.
This is not expected behavior as Daedalus says "Blocks synced 100%" in the top upper corner.

I noticed this as well on the old version (1.0.3619).   Still present on <IP_ADDRESS>8?


Revert "RPM package and systemd service file fixes (for CentOS 8)"

Reverts SoftEtherVPN/SoftEtherVPN#1402

sorry, @stevemuskiewicz , I have to revert. "vpnserver" refuses to start.
I will address it later


Update README.md

Add a user widget info for PointClustering

Thank you Sir. this cmv is really a good solution in terms of js map.
I learned a lot from this community.
Sent from my iPhone

On Jul 22, 2015, at 3:26 PM, Tim McGee<EMAIL_ADDRESS>wrote:
Thanks again @ERS-Long!
—
Reply to this email directly or view it on GitHub.


v1.9 backports 2020-12-02

#14193 -- bpf: optimize dsr and add ipip support for lb-only (@borkmann)
#14217 -- bpf: revert changes to metrics directions contants (@jibi)
#14214 -- docs: Correct typo in upgrade notes (@sayboras)
#14201 -- node: Handle arpinging when remote node is in different L2 (@brb)
#14054 -- kpr: ensure DirectRoutingDevice is in devices (@kkourt)
#14222 -- vendor: Fix cilium/arping goroutine leak (@jrajahalme)
#14187 -- ci/helpers: Delete CRDs in CleanupCiliumComponents (@gandro)
#14232 --  bpf: Fix program size issue with host firewall in IPv4-only mode (@pchaigno)

Once this PR is merged, you can update the PR labels via:
$ for pr in 14193 14217 14214 14201 14054 14222 14187 14232; do contrib/backporting/set-labels.py $pr done 1.9; done

test-backport-1.9

This is your periodic reminder that reusing the start-backport, submit-backport scripts per the backporting process will avoid submitting against the wrong branch, reviewer requesting type issues :-) https://docs.cilium.io/en/v1.9/contributing/release/backports/#creating-the-backports-branch

test-backport-1.9

4.19 job nodeport BPF tests now failing, seems like some verifier issue:
https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Kernel/4053/testReport/junit/Suite-k8s-1/18/K8sServicesTest_Checks_service_across_nodes_Tests_NodePort_BPF_Tests_with_vxlan_With_host_policy_Tests_NodePort/
Top 5 errors/warnings:

 R0=map_value(id=0,off=0,ks=20,vs=48,imm=0) R3=inv(id=0,umax_value=255,var_off=(0x0; 0xff)) R4=map_ptr(id=0,off=0,ks=14,vs=56) R6=ctx(id=0,off=0,imm=0) R7=inv0 R8=inv1 R9=inv4294967162 R10=fp0,call_-1 fp-144=map_value fp-152=0 fp-160=map_value fp-200=fp
 R0=map_value(id=0,off=0,ks=14,vs=56,imm=0) R6=ctx(id=0,off=0,imm=0) R7=map_value(id=0,off=0,ks=14,vs=56,imm=0) R8=inv1 R9=inv1 R10=fp0,call_-1 fp-136=0 fp-144=map_value fp-152=map_ptr fp-160=map_value fp-200=fp
Command execution failed
Error while rewriting endpoint BPF program

4.19 is failing with a complexity error. It's likely related to the host firewall (2 tests failing have the host firewall enabled).
The regression could have been introduced by either #14193 or #14217, the two backported PRs that touch bpf/. I don't think #14193 affects the program size when DSR and LB-only are disabled, so this is likely coming from #14217. I'm not sure how to proceed given we need that PR to fix an upgrade bug. /cc @jibi

Link to the two 4.19 failure: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Kernel/4053/
Temporary revert of #14193 to pinpoint the cause to either #14193 or #14217.

test-backport-1.9

test-backport-1.9

@aanm I've updated the PR labels as per the command in the original post.


Add a migration to fix and represent specialist documents

Migration takes 5 minutes to run.

There are no cases where items have a parent link, so there is no risk of duplicate links.
SELECT links.* FROM editions INNER JOIN documents ON documents.id = editions.document_id INNER JOIN link_sets ON link_sets.content_id = documents.content_id INNER JOIN links ON links.link_set_id = link_sets.id WHERE publishing_app = 'specialist-publisher' AND link_type = 'parent' AND links.created_at < '2017-03-06';


Strange time behaviour after restart (IDFGH-11996)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
I have a fleet of around 250k devices (esp32/esp32s3) connected to the internet and synchronized via NTP. Last week I made an update that updated around 40k of them. In this update, I changed esp-idf from 4.4.3 and added a check to assume that the device is synchronized after restart if it stores the time on the device is larger than 01.01.2024. After some time I noticed that for around 20% of the devices(around 8k), timestamps are messed up by up to +/- 8 hours.
When I tested it with a few dev boards + and a few devices I never observed this issue.
What is the expected behaviour of the system clock when you restart the esp. When restarting esp it is powered on all the time.
I get time using this code:
int64_t getCurrentSystemClockTimestampMs()
{
    struct timeval currentTime = {};
    gettimeofday(&currentTime, nullptr);
    int64_t millis = static_cast<int64_t>(currentTime.tv_sec) * 1000 + (currentTime.tv_usec / 1000);
    return millis;
}

@damian-kurek-wizzdev
Do you mean it works with esp-idf-4.3.3 but has issue with newer esp-idf version? Which version (git describe --tags output)  is used when the issue happened?
@KonstantinKondrashov Any update?

@AxelLin
I forget to add this 4.4.3 to 5.1.1. I am not sure if it happened on the earlier idf version.

@damian-kurek-wizzdev
Just want to clarify if the +/- 8 hours is related to timezone.
What is your current timezone setting?

@AxelLin sorry I did not have time to look into it again, but It wasn't 8 hours flat it was something like 8:23. I asked my devops to get it from the database.  Additionally, sometimes it was 8:xx and sometimes 4:yz

-54 years was lack of synchronization for some reason (I did a bit later). But there you can see the time difference.
"0f026bbc-958c-4eb7-afb3-3901ad527483"	"-01:11:47.875"
"16c199b2-f9e7-40e4-99d5-f96ea9983eac"	"02:23:47.6"
"17b5e75d-6c56-43ce-bb93-808746163e81"	"-54 years -15 days -05:43:08.2"
"22fe0009-22d3-4338-a387-60424421eab8"	"-01:02:57.9375"
"281bf913-a910-4a5f-81a2-0ec3ab6b3a7c"	"-02:10:41.75"
"39fe62d2-836b-4de7-a5d4-a80cd8db771c"	"-01:07:19.483333"
"42a5cc04-abfe-448c-b7f7-cbcbe25ae3b7"	"-54 years -15 days -04:37:37.863014"
"6f81463e-0f9d-41c4-bce4-6c5e0afeff02"	"-01:08:02.05814"
"a36c32d5-13a6-45e1-a2fb-caf1b4cd97d6"	"-01:44:36.837209"
"b4198da8-8eb1-418d-b153-e48da429eca1"	"-01:04:21.074074"
"ba514624-bf60-4a24-a9ee-63059e93a9b9"	"-54 years -15 days -07:24:28"
"ba55394d-66c6-4cf1-933a-6397caba9a2f"	"-54 years -15 days -04:16:59"
"ccfb39ae-e9bb-42cb-a5aa-94194156fe79"	"03:16:32.678571"
"ce06f6ce-40fc-4eb7-bf64-6ba81af6d980"	"-54 years -15 days -05:49:07.892857"

@KonstantinKondrashov Any feedback about the symptom reported by https://github.com/espressif/esp-idf/issues/13066#issuecomment-1985258949 ?


datetime.datetime is not recognized as the equivalent python type to types.TimestampType()

Actually, this isn't an issue. I was looking for a workaround for this comment, wrote the code below and got ValueError.
import databricks.koalas as ks
from datetime import datetime, timedelta

def tomorrow(date) -> datetime:
    return date + timedelta(days=1)

test_df = ks.DataFrame([(datetime(2019, 10, 24))], columns=['date'])
test_df['date'] = test_df['date'].map(tomorrow)

>>> test_df
...
ValueError: Invalid returnType: returnType can not be None

The cause is that datetime.datetime is not recognized as the equivalent python type to types.TimestampType().
_base = {
    types.StringType(): [str, 'str', 'string'],
    types.BinaryType(): [bytes],
    types.ByteType(): [np.int8, 'int8', 'byte'],
    types.ShortType(): [np.int16, 'int16', 'short'],
    types.IntegerType(): [int, 'int', np.int, np.int32],
    types.LongType(): [np.int64, 'int64', 'long', 'bigint'],
    types.FloatType(): [float, 'float', np.float],
    types.DoubleType(): [np.float64, 'float64', 'double'],

    types.TimestampType(): [np.datetime64],  # <- should be included here.

    types.DateType(): [datetime.date],
    types.BooleanType(): [bool, 'boolean', 'bool', np.bool],
    types.ArrayType(types.StringType()): []
}

https://github.com/databricks/koalas/blob/master/databricks/koalas/typedef.py#L112
I think datetime.datetime should be included in types.TimestampType().
@ueshin @HyukjinKwon What do you think?

Oh, yes. looks like we should add. It works in pandas right?

Let me check if this code works when using pandas instead of koalas.

It worked with pandas. I'll create a PR.


Time units handled incorrectly

Description
When using the Online IDE with Calcpad, time units (s, min, h, Hz) are not handled correctly. This makes calculations such as Rotations Per Minute quite prone to error.
A specific example is 1 / 1min = 1 h^-1 in calcpad. As well, the default units of multiplications of 1 min * 1 ms are h.
Screenshots

To Reproduce
I used the following to show unexpected bahaviour
t = 1min
1/t
t * 1 Hz
t_2 = 1 ms
1/t_2
t * t_2

OK, It is ready!
You can try it now on calcpad.eu or download the updated desktop version:
https://github.com/Proektsoftbg/Calcpad/releases/tag/5.6.6

Thanks for looking into this! I found another issue with time units when playing around:

I would expect this to give y=60 min^-1, rather than 60 s^-1
x = 1/s
y = x * 60 s / min

Thank you!
This one was fixed too. I amended the previous release.

Great to see the tool improving. Thanks!


Add env var configuration samples

Add samples for functionality from #262

There's now ample configurations explained in readme and docs. Plus, some samples mention env vars.


Removed mongo 2.6 and 3.0 builds

Mongo 2.6 and 3.0 are no longer officially supported by SlamData.  Translation: we no longer CI them.

MongoDB's support policy can be found at https://www.mongodb.com/support-policy.
MongoDB 2.6 was reached EOL in October 2016.
MongoDB 3.0 will reach EOL in February 2018.
We have agreed to officially support MongoDB 3.2+ in a couple of slack posts:

https://slamdata.slack.com/archives/C0H2S6TLM/p1505179125000157
https://slamdata.slack.com/archives/C5XP2RQG1/p1505932611000211

It doesn't make much sense to build for platforms we're not supporting.


Can I use this with Amazon Aurora?

Hi, I tried to use this with Amazon Aurora for port forwarding but ran into this error:
aws ec2-instance-connect open-tunnel \
  --private-ip-address <IP_ADDRESS> \
  --instance-connect-endpoint-id eice-0c5db94cacd12345 \
  --remote-port 5432 \
  --local-port 5432
Listening for connections on port 5432.
[1] Accepted new tcp connection, opening websocket tunnel.
2023-06-28 11:50:56,333 - awscli.customizations.ec2instanceconnect.websocket - ERROR - [1] Encountered error with websocket: (9, 'Bad file descriptor')
[1] Closing tcp connection.
[2] Accepted new tcp connection, opening websocket tunnel.
2023-06-28 11:51:00,000 - awscli.customizations.ec2instanceconnect.websocket - ERROR - [2] Encountered error with websocket: (9, 'Bad file descriptor')
[2] Closing tcp connection.
[3] Accepted new tcp connection, opening websocket tunnel.
2023-06-28 11:51:03,750 - awscli.customizations.ec2instanceconnect.websocket - ERROR - [3] Encountered error with websocket: (9, 'Bad file descriptor')
[3] Closing tcp connection.
[4] Accepted new tcp connection, opening websocket tunnel.
2023-06-28 11:51:07,363 - awscli.customizations.ec2instanceconnect.websocket - ERROR - [4] Encountered error with websocket: (9, 'Bad file descriptor')
[4] Closing tcp connection.
[5] Accepted new tcp connection, opening websocket tunnel.
2023-06-28 11:51:11,025 - awscli.customizations.ec2instanceconnect.websocket - ERROR - [5] Encountered error with websocket: (9, 'Bad file descriptor')
[5] Closing tcp connection.
[6] Accepted new tcp connection, opening websocket tunnel.

Any ideas?

Unfortunately this is no longer possible, it seems ~yesterday the service added extra validation to only allow ports 22 and 3389. I'll close this issue as a duplicate of #5


Fixed a bug with duplicated SessionStarted event

Fixes #5964
Proposed changes:

Custom metadata was restarting the session
Not the situation is handled properly

Status (please check what you already did):

[x] added some tests for the functionality
[x] updated the documentation
[x] updated the changelog (please check changelog for instructions)
[x] reformat files using black (please check Readme for instructions)

Cherry-picked and closed in #5767


Update Rook Helm chart and container image to v1.9.11

The renovate PRs look a bit broken, cf. #71 and #74 (version mismatch between commit/title and PR body)
Checklist

[x] PR contains a single logical change (to build a better changelog).
[x] Categorize the PR by setting a good title and adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog.
[x] Link this PR to related issues or PRs.

Check for missing runbooks for alertrules (e.g. CephPoolGrowthWarning)

https://github.com/rook/rook/pull/10968 doesn't impact us on Exoscale, since we're not directly trying to use a partition, but instead use the partition through a Kubernetes PV.


I2S-driver: apll_predefine was ignored

In i2s.c, there are predefined values for audio PLL configuration in the apll_predefine array. In i2s_apll_calculate, the sampling rate supplied was compared to the bits_per_sample value in the array. This is a fix of this comparison.

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

Never mind, this seems to break the i2c driver.


ENH: Improve test coverage of ImageMoments

Add masking from SpatialObjects for
computing the ImageMoments.

@dzenanz The error was the whole point of this patch set.  It demonstrates that this code fails where previously it had succeeded (see #663 for the same code passing).
I've been discussing with @aylward and refactored to provide a more clear path to working code. in subsequent patches.

@aylward it would be great if you could look at these changes.
@N-Dekker This is related to the conversation you and @aylward were having on a different PR.  I went ahead and implement part of it because the current implementation was breaking my codebase.


qView: init at 2.0

A fast, unobtrusive Qt-based image viewer.

Motivation for this change
Things done

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

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

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

@GrahamcOfBorg build qview

@acowley Thank you for your contribution to nixpkgs! :tada:

Thank you for the fast and helpful review, @aanderse!


pageserver: near overload benchmark fails to generate /metrics

My best guess for cause is the remaining with_label_values calls because missed metrics only happen during the busiest times but are fine before and after.
Requests to /metrics fail because of collection time is high on this line: https://github.com/neondatabase/neon/blob/1d3ae57f18462e150a74d044c4f59c34a6c27e69/libs/utils/src/http/endpoint.rs#L275
Such cases like look like this for example:
2024-03-04T20:06:10.836918Z  WARN request{method=GET path=/metrics request_id=edba3aef-656c-415a-82d4-16576f98952c}:blocking: failed to write out /metrics response: Io(Kind(BrokenPipe)) after_bytes=0 total_ms=32828 spawning_ms=85 collection_ms=32742 encoding_ms=0

Possible culprits are can be found with rg with_label_values. What we should do instead is to store the value with_label_values call created somewhere and only do this query once.

Discussion from planning meeting: this should be a higher priority fix.

afaik we haven't seen this in a while


Implement a specific authenticator for Yandex federated authentication

Is your feature request related to a problem? Please describe.
Since Yandex is no longer supporting for OpenIDConnect.[1] It cannot be integrated using the default OIDC connector.
Describe the solution you would prefer
A specific connector for Yandex extending OAuth2 Generic Authenticator.
Additional context
[1] - https://yandex.com/dev/passport/doc/dg/concepts/openid-migrate.html

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Can the active Link have a ref?

So, it'd be really awesome if I could easily access my active Link via a ref easily. Something like this:
render: function(){
    return (
        <Link to="inbox"> Inbox </Link>
        <Link to="account"> Account </Link>
    )
}

Then, say if I clicked inbox, It would set a ref to something like this.refs.activeLink so then I could easily in another method reference that dom node. I can even do the work for this if it is something you guys are interested in.

If you want something like this, you could wrap our Links in your own component, and do some checking if its active.


Automate Tagging and Release of aws/karpenter and aws/karpenter-core

Tell us about your request
Currently, releasing a newer version of Karpenter, requires that we first tag the same version that we are releasing with in karpenter-core, then bump the version of karpenter-core in the karpenter go.mod to be the tagged version, then tag the version that we are releasing in karpenter to complete the release process.
We should automate this process so that kicking off a release involves starting a GH workflow for a release version which:

Tags the version in karpenter-core
Bumps the version of karpenter-core to the tagged version in the karpenter go.mod
Tags the version in karpenter
Create release notes that contains a link to the karpenter-core release notes for the same version.

Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
N/A
Are you currently working around this issue?
N/A
Additional Context
No response
Attachments
No response
Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

what is the difference between karpenter and karpenter-core?

Karpenter is the AWS-specific version of Karpenter and has the AWS cloudprovider inside of it. Karpenter-core is meant for any common packages and common controllers that we expect to run across all cloudprovider versions of karpenter.


[IMP] Be more specific in sorting of model's private attributes

Private attributes for models are usually placed as follows:

Model declaration:

Model name (_name)
Inheritances (_inherit and _inherits)

Human readable name (_description)
Other optional parameters (order, displayed name field, model-specific modifiers, etc)

However, current documentation could give the wrong impression the attribute _inherit goes after _description, which is not what is usually done.
This commit provides a recommended order for private attributes without explicitly requesting it.

Hi @yajo, could you review this, please?

Hi @sbidoul,
Could you merge, please?
This has several approvals already.
Regards,


limit log collection by age

optionally limit log collection by days old
--only-collect-logs-newer-than 5d
--only-collect-logs-newer-than 2h
This should probably be based on file modified time and has a dependency on #9 being done

perhaps log-age or log-window ?

--log-age 5d
--log-age 12h
like that?


[Iconosquare]  TypeError: the JSON object must be str, not 'NoneType'

[x] I've verified and I assure that I'm running youtube-dl 2016.08.12

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

$ youtube-dl -v 'http://statigr.am/p/522207370455279102_24101272'
[debug] System config: []
[debug] User config: ['-4', '-f', 'best', '--prefer-free-formats', '--no-cache-dir', '--no-mtime', '--youtube-skip-dash-manifest']
[debug] Command-line args: ['-v', 'http://statigr.am/p/522207370455279102_24101272']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.08.12
[debug] Python version 3.5.2+ - Linux-4.7.0-rc7-686-i686-with-debian-stretch-sid
[debug] exe versions: ffmpeg 3.1.2, ffprobe 3.1.2, rtmpdump 2.4
[debug] Proxy map: {}
[Iconosquare]<PHONE_NUMBER>55279102_24101272: Downloading webpage
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl", line 9, in <module>
    load_entry_point('youtube-dl==2016.8.12', 'console_scripts', 'youtube-dl')()
  File "/usr/local/lib/python3.5/dist-packages/youtube_dl/__init__.py", line 433, in main
    _real_main(argv)
  File "/usr/local/lib/python3.5/dist-packages/youtube_dl/__init__.py", line 423, in _real_main
    retcode = ydl.download(all_urls)
  File "/usr/local/lib/python3.5/dist-packages/youtube_dl/YoutubeDL.py", line 1786, in download
    url, force_generic_extractor=self.params.get('force_generic_extractor', False))
  File "/usr/local/lib/python3.5/dist-packages/youtube_dl/YoutubeDL.py", line 691, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/lib/python3.5/dist-packages/youtube_dl/extractor/common.py", line 347, in extract
    return self._real_extract(url)
  File "/usr/local/lib/python3.5/dist-packages/youtube_dl/extractor/iconosquare.py", line 37, in _real_extract
    video_id)
  File "/usr/local/lib/python3.5/dist-packages/youtube_dl/extractor/common.py", line 549, in _parse_json
    return json.loads(json_string)
  File "/usr/lib/python3.5/json/__init__.py", line 312, in loads
    s.__class__.__name__))
TypeError: the JSON object must be str, not 'NoneType'

http://statigr.am/p/522207370455279102_24101272

Redirects to https://pro.iconosquare.com/.


Add Google OAuth Provider

Implement login/signup with google provider

Hey!
I would love but this will require me to have a validated Google OAuth app which is very complicated.
I need to write Terms and Privacy but I don't have time to do it.
Feel free to open an PR with Terms and Privacy!
But adding the Google OAuth souldn't be more complicated than the GitHub or Twitch one!

Something like this should work.

Privacy Policy
Effective Date: [Insert Date]

Introduction

Welcome to [Your Company Name] ("we," "our," or "us"). We are committed to protecting your privacy and ensuring you have a positive experience on our website and in using our products and services.
This Privacy Policy explains how we collect, use, disclose, and safeguard your information when you use Google OAuth to access our services. Please read this policy carefully. If you do not agree with the terms of this Privacy Policy, please do not access our services.

Information We Collect

When you use Google OAuth to sign in to our service, we collect the following information from your Google account, depending on the permissions you grant:
Basic Profile Information: This includes your name, email address, profile picture, and language preference.
Google User ID: A unique identifier provided by Google to manage your account.
Other Information: Any other information you choose to share with us through your Google account.
3. How We Use Your Information
We use the information we collect through Google OAuth for the following purposes:
Authentication: To verify your identity and allow you to access our services.
Personalization: To provide a more personalized experience based on your profile information.
Communication: To send you updates, notifications, and other relevant information about our services.
Service Improvement: To analyze how users interact with our services and improve them.
4. Sharing Your Information
We do not sell, trade, or otherwise transfer your personal information to outside parties, except as described in this Privacy Policy. We may share your information in the following situations:
Service Providers: We may share your information with third-party vendors and service providers who perform services on our behalf, such as hosting, data analysis, and customer service.
Legal Requirements: We may disclose your information if required to do so by law or in response to valid requests by public authorities (e.g., a court or government agency).
Business Transfers: In the event of a merger, acquisition, or sale of all or a portion of our assets, your information may be transferred as part of that transaction.
5. Data Security
We use administrative, technical, and physical security measures to protect your personal information. While we strive to protect your information, we cannot guarantee its absolute security.

Your Choices

Access and Update: You can access and update your personal information through your account settings.
Revoke Access: You can revoke our access to your Google account at any time through your Google account settings.
Data Deletion: If you wish to delete your account and the associated data, please contact us at [Contact Information].
7. Third-Party Links
Our service may contain links to third-party websites or services. We are not responsible for the privacy practices or content of these third parties.

Changes to This Privacy Policy

We may update this Privacy Policy from time to time. We will notify you of any changes by updating the "Effective Date" at the top of this policy. We encourage you to review this Privacy Policy periodically to stay informed about how we are protecting your information.

Contact Us

If you have any questions about this Privacy Policy or our practices, please contact us at:
[Your Company Name]
[Your Address]
[Your Email Address]
[Your Phone Number]

@Barbapapazes is the above something we can work with?
I prepared the necessary code additions already in this branche here; GreenmeisterDavid/gavarnie-fix-github-login@add-google-oauth , but haven't added a privacy policy or terms yet.

Hey @GreenmeisterDavid 👋,
Thanks! I will add some terms and privacy to Gavarnie as I had to make some for https://soubiran.dev. Then, I will request Google to approve the app. In the mean time, feel free to open a draft PR!


[JAVA-4816] perf: improving performance of ObjectId parseHexString()

JIRA Ticket: https://jira.mongodb.org/browse/JAVA-4816
The current implementation of ObjectId parseHexString is somewhat slow, because it makes a total of 12 calls to Integer.parseInt, which is not the fastest to begin with. This imposes some overhead to all ObjectId(string) calls, which can really add up at scale, for example when deserializing a large amount of ObjectId data via Jackson.
Replacing this implementation with a hand-rolled version which converts each part of the hex string to a byte. This in my rough testing is about 5x faster than the current implementation of parseHexString.
Tests (on my M1 MacBook Pro):
    public static void main(String[] args) {
        long startTime = System.currentTimeMillis();
        for (int i = 10000000; i < 100000000; i++) {
            new ObjectId("1234567890abcdef" + i);
        }
        long endTime = System.currentTimeMillis();

        System.out.println("Time elapsed: " + (endTime - startTime));
    }

On master:
Time elapsed: 19460

On branch:
Time elapsed: 3617

Hi @eonwhite thanks for the contribution. Can you do a couple of things to move this forward:

Open an issue in the JAVA project of MongoDB's Jira, and then reference the issue number in the PR description (see the CONTRIBUTING guide for details).  Please attach your performance tests as well.
Run ./gradlew bson:check to make sure all static and runtime checks are passing

Thanks @jyemin -- have resolved issues and verified that tests pass.

Thanks again for the contribution @eonwhite


Integration stats are now sorted correctly, and by default, by integr…

…ation time (descending).
Resolves #57

Looks good!


Add src to dictionary

This pull request adds "src" to the dictionary

src shouldn't be in the dictionary. Why did this need to be added?

Reverting this PR.


hightlight: correctly highlight loose comments

close #1907

This was a surprisingly easy fix, :shipit:


UML-751 As a user I want my code to be accepted if I entered the C- or V-

Purpose
As a user (actor)
I want my valid actor code to be accepted even if I have included the 'C-' at the start
So that I can add my LPA to my account
As a user (viewer)
I want my valid viewer code to be accepted even if I have included the ‘V-’ at the start
So that I can view the LPA summary
Fixes UML-751
Approach
Adjusted RegEx & applied additional filtering to remove C- or V-
Learning
Any tips and tricks, blog posts or tools which helped you. Plus anything notable you've discovered about the Refunds service
Checklist

[x] I have performed a self-review of my own code
[ ] I have added relevant logging with appropriate levels to my code
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] The product team have tested these changes

Coverage increased (+0.04%) to 81.169% when pulling 6fafa5520ae222f5875d93c0da352eb4f049cff0 on UML-751_C-V-acceptance into c3cb3de3f3ca3d126f6e2e212b66d322bbd57fab on master.


Feature: Adding contributors section to the README.md file.

There is no Contributors section in readme file .
As we know Contributions are what make the open-source community such an amazing place to learn, inspire, and create.
The Contributors section in a README.md file is important as it acknowledges and gives credit to those who have contributed to a project, fosters community and collaboration, adds transparency and accountability, and helps document the project's history for current and future maintainers. It also serves as a form of recognition, motivating contributors to continue their efforts.

@zxlzr Kindly assign me this issue, I want to work on it. Thank You!!

Thanks. I have assigned the issue.


Use TestAppDelegate to prevent normal app launch during testing

When I was going through the Eidolon code looking for examples of Moya usage, I noticed the work around for preventing normal app launch while testing. I thought I would share the approach that I have used before. It's more code and more files, but I like that it gets any reference to XCTest out of AppDelegate and prevents AppDelegate from ever loading at all--so less risk of unintended side effects during testing. For these reasons, I think it is slightly more elegant. Unfortunately I was coding blind on this one because of Xcode 8 and a problem with cocoapods-keys, so I'm not 100% sure it even compiles 😬. But the idea is there if you want to use it at all. If not, no worries, the current solution is certainly more concise. Cheers!

Ah yeah! We use this technique in Eigen and Energy - I'm totally cool wit this PR if we get a human to verify on our side 👍
Thanks @scottrhoyt - nice contribution

No problem!

Looks good, I'll verify locally on Monday. Thanks!

Looks like eidolon actually uses quite a bit of the app delegate, even under testing, so we actually need to keep it as-is. I think our existing workaround (to isolate side effects from tests) is okay, not great, but it works :smile: Thanks again for the PR!


Incremental anti-join issue

I had trouble with a block Josh provided for automatically binding root elements to external root. According to an email exchange Josh says:

this is definitely a runtime issue. Incremental antijoin is unfortunately pretty complicated, particularly in the context of partial joins.

Josh goes on to say:

In the interim, you can work around the issue by replacing the middle block with:

  widget-root = [#tw-widget-root]
  elem = [#mhtml/element]
  elem != widget-root
  not(
    parent = [#mhtml/element children: elem]
    parent != [#tw-widget-root]
  )

bind
  widget-root.children += elem

which just looks for the widget root separately in the not.

Here is the original code which illustrates the issue:
Use #mhtml/element as a stand-in for #html/element
Create the pretend #tw-widget-root external root
Create parent->child and childless elements
commit
  [#tw-widget-root #mhtml/element text: "#tw-widget-root"]
  [#mhtml/element text: "parent" children: [#mhtml/element text: "child"]]
  [#mhtml/element text: "childless"]

Automatically add parentless elements to #tw-widget-root
search
  widget-root = [#tw-widget-root]
  elem = [#mhtml/element]
  elem != widget-root
  not(
    parent = [#mhtml/element children: elem]
    parent != widget-root
  )

bind
  widget-root.children += elem

Display the children of #tw-widget-root. Expect to see only
'parent' and 'childless', but am also seeing 'child' here
search
  [#tw-widget-root children: child]
  child = [#mhtml/element]
bind
  [#ui/div text: child.text]

I've tracked the issue down.  The contents of the antijoin (not) are executed as a separate join node. Joining the results of that join node with the results of the main query happens after that join node has run, and the join node has no knowledge that it's being used as part of an antijoin. Unfortunately, in the case of expressions like !=, this sort of post filtering isn't statically guaranteed to be correct (in fact, != is one of the few cases that could theoretically be special-cased, but doing so isn't fixing the real issue).
The takeaway: This is an architectural issue with 0.3 that I can't think of a trivial solution to. Due to a generally more flexible architecture in 0.4, we've accidentally avoided this issue entirely there. Depending on the 0.4 timeline, it may be better to table this issue and not publicly release 0.3 outside of these previews. If it looks like 0.4 is further off than expected, I'll need to get my hands dirty expanding the regression test suite and then reworking the way antijoin is written.
Unfortunately, this bug (and any real fix) will also impact chooses, since they're implemented using antijoin.


Prefer nvidia channel for conda builds

Fixes https://github.com/pytorch/vision/issues/5635

@malfet The issue persists on the cmake_linux_gpu job:
ImportError: libcupti.so.11.3: cannot open shared object file: No such file or directory
The binary_linux_conda_*_cu115 jobs are failing because they use ffmpeg-5.0.0 which deprecates some functionality that we currently use. There is another PR that attempts to fix this at #5644.
I think we should merge this after the cmake job is also fixed and then resolve the rest of the jobs on the other PR.


Special handling of dates

Added special handling of dates instead of converting them to regular objects.
This is the result of the discussion in issue #25 that was created due to issue #24.
Also made sure we don't try to deep merge dates.
This fixes #24.

Looks great! Nice and straightforward.

Looks like there are some cross-browser test failures: https://travis-ci.org/#L600
Possibly new Date(dateObject) doesn't pass through time zone in all browsers?

@rtfeldman: Seems like it! Replacing the .toISOString() comparisons with .getTime() comparisons should solve the problem. You want me to add it to the current PR, a new one or do you want to do it yourself?

If you don't mind, a new PR would be great. :smiley:

Seems it's not just .toISOString()! IE now reports different .getTime() numbers: https://travis-ci.org/rtfeldman/seamless-immutable#L616
Maybe using a different constructor is in order?

I just got access to an IE11 and tested it (and in a Firefox) and saw the problem with using the constructor:
new Date(originalDate).
But the following works in both:
new Date(originalDate.getTime()).
I have seen a lot of examples doing it the first way and as far as I understood it should then use the .valueOf method on the date which generates the same output as .getTime in all browsers. So not sure why the first way fails, but I will change it to the second way.
Thanks for the suggestion @rtfeldman. A PR is coming, sorry for the mess.

The constructor new Date(originalDate) throws away the millseconds in some browsers (IE, FF) but not other (Chrome, others?).
Using new Date(originalDate.getTime()) preserves the milliseconds. This is fixed in PR #33.


No distributable available

This may or may not be an issue with gittio:
Take a look at this module: http://gitt.io/component/ti.cloudpush
When running the command gittio install ti.cloudpush I get:
[INFO]  ti.cloudpush searching...
[ERROR] ti.cloudpush:commonjs no distributable available
[ERROR] ti.cloudpush:ios no distributable available
[ERROR] ti.cloudpush:android no distributable available

It confused me because the gitt.io website seems to pick up the module correctly.
Please close this issue if the ti.cloudpush module itself is to blame.

Hi @Bertuzb, when you see the download column saying N/A you'll run into this. The CLI's install only works when gitTio has found a download to install.


adding standalone etl classes with tests in the same directory

Here are my 3 ETL objects that I presented on in the Geospatial Presentation Exchange (FileDownloader, DataFrameToTable, DataFrameToDatabase). Each object is shipped with the tests for it as well as any useful ancillary files. FileDownloader and DataFrameToTable are the tested objects, whereas DataFrameToDatabase will have automated tests shortly.

@jlucas-esri  thanks so much! can you add a simple readme.md to the root of the folder you are adding that gives a quick description of what the scripts do? something like this: https://github.com/Esri/Geospatial-Center-Code/blob/master/python/api/add-attribute-index-hosted-feature-layer/README.md
thanks!

@apfister Just committed those READMEs and a quick update to one of the objects

niiiiice thank you!


pkgsStatic.brotli: fix patch

Motivation for this change
Fixing #102957
Things done

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

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

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

By the way, perhaps some of you would like to be in brotli.meta.maintainers?


Disallow functions without body completely

Background
We used to support this kind of function body in syntax:
fnBody : IMPLIES expr
       | (BAR clause)* ;

This implicitly allowed this kind of function def a : Nat and we cannot reject it by simply checking whether the body.isLeft() or the body is non-empty right since there's at least one legal case I can tell:
def y (f : Fin zero) : Nat

The Fin zero does not have any constructor thus no matching is needed, so this decl should tyck.
Suggested solution
Proposing changing the syntax to: a function must explicitly have a body, either an expression or a pattern matching
fnBody : IMPLIES expr
-       | (BAR clause)* ;
+       | (BAR clause)+ ;

and for legal cases, users must explicitly match the param with impossible, like:
def y (f : Fin zero) : Nat
  | ()

If a function has both empty telescope and empty body, then we reject it. Does this work? Because the coverage checker will correctly reject it if it has parameters

If a function has both empty telescope and empty body, then we reject it. Does this work? Because the coverage checker will correctly reject it if it has parameters

tql. this solution looks much better


Fix tooltip free fuel

Changed the description of the empty fuel tool tip to more accurately describe what the button does.

If you update the tooltip to your last suggestion, I'll approve and we can get it merged!

.deploy to development

.deploy to production


ISearchCommentsResponse matches property type interface

Hi 🙂, thanks for the great lib!
It seems that I get an array of ItemConversation instead of ItemComment on a search comments response.
https://github.com/baileyherbert/envato.js/blob/master/src/endpoints/catalog.ts#L190

Hi, thanks for reporting this! I hope my little mistake didn't cause you too much trouble... 😅
I just released a fix as version 3.0.5. Let me know if you have any issues & cheers!

@baileyherbert my pleasure 🙂
Thank you for fixing it so fast, cheers!

@baileyherbert I just see that ItemComment id should be a number https://github.com/baileyherbert/envato.js/blob/0cd7ea5f182d6f3b1f8d8c191e9283c1f90bccb3/src/types/api.ts#L164

Aaahh!!! Another one? How embarrassing, don't look! 😱😱😱
Let me go through those types in detail real quick and verify everything, and then I will release version 3.0.6 haha

@baileyherbert found one more 🙂
https://github.com/baileyherbert/envato.js/blob/0cd7ea5f182d6f3b1f8d8c191e9283c1f90bccb3/src/types/options.ts#L199

Alright, I fixed a lot of little mistakes and 3.0.6 is now published. 👍
With respect to the item_id property, I'm going to keep it as string | number for now (to ensure backwards compatibility) and will remove the string in the next major update.

@baileyherbert thank you 🙂


[QUESTION] Switch off Axon.Loop.run metric stdout

When I am in testing or in production, I may not want the metrics used during developed printed. Metrics such as:
Epoch: 0, Batch: 1850, Accuracy: 0.8927606 loss: 0.3565226
Epoch: 1, Batch: 1850, Accuracy: 0.9309664 loss: 0.2938470
Epoch: 2, Batch: 1850, Accuracy: 0.9429026 loss: 0.2605216
Epoch: 3, Batch: 1850, Accuracy: 0.9509048 loss: 0.2371558
Epoch: 4, Batch: 1850, Accuracy: 0.9524919 loss: 0.2207826

Or
Batch: 1874, Accuracy: 0.9613333

that is printed during training or testing on calling Axon.Loop.run.
Is there some way to switch this off in the dev and prod environments, preferably in my config.exs?

I believe you can turn it on and off using the loop API. then if necessary, you can add the relevant config, read it yourself, and pass as argument to the loop API.

Oops, to be clear, I am assuming there is an Axon.Loop API but @seanmor5 would be the best person to tell. My point is just that reading from config should be your responsibility, not Axon’s. :)

reading from config should be your responsibility, not Axon’s. :)

That's fine, but I see no way of turning it off in the docs for run/4.

It's not well documented, but when you declare trainer you can specify log: -1 (or any negative number) and it will turn off logging. If you want to open a PR expanding the docs, that would be great!

when you declare trainer you can specify log: -1 (or 0 or any negative number)

Looking through the src, the same is not true for Loop.evaluator/1. Should I send a PR for this as well?

Definitely!


Update CONTRIBUTING.md

feat: CONTRIBUTING.md: update readme to Add an Extension

@mcred any chance you can review this please?

@jnewton03 @mcred :are we good to merge this PR?


Decide what to do with ETL.validate in a partially refactored world

Description
Right now def validate in the ExtractTransformLoad base class only runs if self.GEO_LEVEL is not None. This appears to be None for all data sources. Do we want to bring some of this validation back somehow?

Turns out this was set at the class level, not instance level. Lots of data sources are using this check.


Unable to build latest version of pprof

Describe the bug
TL;DR: Attempting to update the pprof pkg to support latest changes/bug fixes. Cannot build package as there is no cgo/SWIG to build c files when testing for symbolization on Mach-O binaries (see error below). However, when building using go get -u github.com/google/pprof in a nix-shell (see build instructions on main google/pprof repo), no such errors occur.
Here is the file which successfully builds latest HEAD of pprof within a nix-shell environment.
# shell.nix; building pprof using go within nix-shell
with import <nixpkgs> {};

mkShell {
  name = "profiling";
  buildInputs = [
    gperftools
    go
    llvmPackages_11.llvm # llvm-symbolizer
    llvmPackages_11.clang # to link core libs
  ];
  # Automate process of extracting correct paths to binaries. This approach
  # works despite the fact that there might be multiple different versions of
  # the pkg of interest, as each version has its own unique hash.
  shellHook = ''
    clang_dir=$(ls -l /nix/store/ | grep ${llvmPackages_11.clang.pname + "-" +
      llvmPackages_11.clang.version} | grep '^d' | awk '/ / {print $NF}')
    export PATH="/nix/store/$clang_dir/bin:$PATH"

    # Use latest pprof (from github)
    # NOTE: delete go pkgs using `go get pprof@none`, then sudo rm -rf $HOME/go
    export GOPATH="$HOME/go/"
    go get -u github.com/google/pprof
    export PATH="$GOPATH/bin:$PATH"
  '';
}

Full details
Main goal/motivation: Profile code to identify areas which could benefit from optimization. To that end, we install gperftools (version 2.8), which includes a version of pprof to pretty print the results of the CPU profile. However, it does not properly symbolize the addresses to its respective function/line in the code. See example output below:
[user@system:~]$ pprof -text ./main main.prof
Using local file ./main.
 Normally, this is all we need to do: specify the standalone version of pprof, add that binary to the path (so that we are using the standalone version and not the version included with gperftools), and add either binutils or llvm for addr2line or llvm-symbolizer, respectively. Although this produces (human-readable) symbolizations, there are still some bugs/errors in the output (notice how this version cannot properly symbolize Mach-O binaries).
[user@system:~]$ PPROF_BINARY_PATH=./main
[user@system:~]$ pprof -text -trim=false main.prof
Local symbolization failed for libdyld.dylib: stat /usr/lib/system/libdyld.dylib: no such file or directory
Local symbolization failed for libsystem_platform.dylib: fat Mach-O archives are currently unsupported
Local symbolization failed for libsystem_kernel.dylib: fat Mach-O archives are currently unsupported
Local symbolization failed for libsystem_c.dylib: stat /usr/lib/system/libsystem_c.dylib: no such file or directory
Some binary filenames not available. Symbolization may be incomplete.
Try setting PPROF_BINARY_PATH to the search path for local binaries.
File: libc++.1.dylib
Type: cpu
Showing nodes accounting for 8.09s, 100% of 8.09s total
      flat  flat%   sum%        cum   cum%
     8.02s 99.13% 99.13%      8.02s 99.13%  k_prime
     0.03s  0.37% 99.51%      8.02s 99.13%  primes
     0.03s  0.37% 99.88%      0.03s  0.37%  std::__1::locale::use_facet
     0.01s  0.12%   100%      0.01s  0.12%  [libsystem_kernel.dylib]
         0     0%   100%      8.06s 99.63%  <unknown>
         0     0%   100%      8.06s 99.63%  [libdyld.dylib]
         0     0%   100%      0.01s  0.12%  [libsystem_c.dylib]
         0     0%   100%      4.88s 60.32%  [libsystem_platform.dylib]
         0     0%   100%      8.06s 99.63%  main
         0     0%   100%      0.01s  0.12%  std::__1::__stdoutbuf::overflow
         0     0%   100%      0.04s  0.49%  std::__1::basic_ostream::operator<<
         0     0%   100%      0.01s  0.12%  std::__1::basic_ostream::put
         0     0%   100%      0.01s  0.12%  std::__1::endl

Given that recent versions of pprof address (some of) these bugs and add additional features, it is worthwhile to use a more updated version of pprof to allow for better profile reporting. Attempting to update to a recent version by changing the rev and associated sha256 in the default.nix file (similarly for the dependencies), and then build the file results in the following error below: It is unable to build the executable because it is not able to compile the hello.c or lib.c files without cgo/SWIG.
building '/nix/store/i8k11azazldg6xb1kfshbx4p4g44i7zf-pprof-unstable-2021-08-27.drv'...
unpacking sources
unpacking source archive /nix/store/ny13myl1rp35lca820k2c11jjpqy877b-pprof-02619b8
source root is pprof-02619b8
patching sources
configuring
unpacking source archive /nix/store/ngm69vin9qs4m4q291kzbi5fyxk0kr59-readline-2972be2
unpacking source archive /nix/store/sds05435lbg694gb4vd4r9qa85nf01fz-demangle-28f6c0f
building
internal/unsafeheader
internal/race
unicode/utf8
container/list
math/bits
runtime/internal/sys
internal/nettrace
unicode
sync/atomic
runtime/internal/atomic
runtime/cgo
internal/cpu
encoding
unicode/utf16
crypto/internal/subtle
runtime/internal/math
crypto/subtle
vendor/golang.org/x/crypto/cryptobyte/asn1
vendor/golang.org/x/crypto/internal/subtle
github.com/google/pprof/third_party/d3
github.com/google/pprof/third_party/d3flamegraph
github.com/google/pprof/third_party/svgpan
internal/bytealg
math
runtime
internal/reflectlite
sync
internal/singleflight
internal/testlog
math/rand
errors
sort
internal/oserror
path
io
vendor/golang.org/x/net/dns/dnsmessage
strconv
syscall
crypto/internal/randutil
text/tabwriter
hash
strings
bytes
hash/adler32
crypto/hmac
hash/crc32
crypto
reflect
crypto/rc4
vendor/golang.org/x/crypto/hkdf
vendor/golang.org/x/text/transform
bufio
html
regexp/syntax
internal/syscall/execenv
internal/syscall/unix
time
regexp
context
io/fs
internal/poll
internal/fmtsort
encoding/binary
crypto/x509/internal/macos
os
encoding/base64
crypto/sha1
vendor/golang.org/x/crypto/poly1305
crypto/cipher
crypto/sha256
crypto/sha512
crypto/md5
crypto/ed25519/internal/edwards25519
encoding/pem
vendor/golang.org/x/crypto/chacha20
crypto/des
crypto/aes
os/signal
vendor/golang.org/x/sys/cpu
vendor/golang.org/x/net/route
path/filepath
fmt
vendor/golang.org/x/crypto/chacha20poly1305
io/ioutil
os/exec
net
encoding/hex
flag
log
net/url
vendor/golang.org/x/crypto/curve25519
encoding/json
compress/flate
debug/dwarf
vendor/golang.org/x/text/unicode/norm
math/big
vendor/golang.org/x/net/http2/hpack
mime
mime/quotedprintable
net/http/internal
vendor/golang.org/x/text/unicode/bidi
github.com/ianlancetaylor/demangle
text/template/parse
compress/gzip
compress/zlib
vendor/golang.org/x/text/secure/bidirule
github.com/google/pprof/profile
vendor/golang.org/x/net/idna
text/template
debug/macho
debug/pe
debug/elf
github.com/google/pprof/internal/measurement
github.com/google/pprof/internal/graph
crypto/rand
crypto/dsa
encoding/asn1
crypto/elliptic
github.com/google/pprof/internal/elfexec
crypto/ed25519
crypto/rsa
html/template
crypto/x509/pkix
vendor/golang.org/x/crypto/cryptobyte
crypto/ecdsa
github.com/chzyer/readline
vendor/golang.org/x/net/http/httpproxy
net/textproto
crypto/x509
vendor/golang.org/x/net/http/httpguts
mime/multipart
crypto/tls
net/http/httptrace
net/http
github.com/google/pprof/internal/plugin
github.com/google/pprof/internal/symbolz
github.com/google/pprof/internal/transport
github.com/google/pprof/internal/binutils
github.com/google/pprof/internal/report
github.com/google/pprof/internal/symbolizer
github.com/google/pprof/internal/driver
github.com/google/pprof/driver
github.com/google/pprof
github.com/google/pprof/fuzz
package github.com/google/pprof/internal/binutils/testdata: C source files not allowed when not using cgo or SWIG: hello.c lib.c
builder for '/nix/store/i8k11azazldg6xb1kfshbx4p4g44i7zf-pprof-unstable-2021-08-27.drv' failed with exit code 1
error: build of '/nix/store/i8k11azazldg6xb1kfshbx4p4g44i7zf-pprof-unstable-2021-08-27.drv' failed

A short-term fix can be applied by updating to a commit which does not attempt to compile these files (as they are unavailable within the main pprof repo). From some initial tests, this compiles and generates plausible profiling reports. However, it means that we cannot update the package beyond this revision. This will be problematic in the future as new fixes/features are introduced; essentially we are side-stepping the main issue (see above error for details).
A workaround to get the latest version of pprof working is to install go separately, set the GOPATH and install pprof (see build commands here), which should add that location of the binary in the path. Then, we can run pprof without any error and with all the latest features.
[user@system:~]$ PPROF_BINARY_PATH=./main
[user@system:~]$ pprof -text -trim=false main.prof
Local symbolization failed for libdyld.dylib: stat /usr/lib/system/libdyld.dylib: no such file or directory
Local symbolization failed for libsystem_c.dylib: stat /usr/lib/system/libsystem_c.dylib: no such file or directory
Some binary filenames not available. Symbolization may be incomplete.
Try setting PPROF_BINARY_PATH to the search path for local binaries.
File: libc++.1.dylib
Type: cpu
Showing nodes accounting for 8.19s, 100% of 8.19s total
      flat  flat%   sum%        cum   cum%
     8.13s 99.27% 99.27%      8.13s 99.27%  k_prime
     0.02s  0.24% 99.51%      0.02s  0.24%  [libsystem_kernel.dylib]
     0.02s  0.24% 99.76%      0.02s  0.24%  std::__1::__compressed_pair::first
     0.01s  0.12% 99.88%      7.68s 93.77%  primes
     0.01s  0.12%   100%      0.03s  0.37%  std::__1::__list_imp::__sz
         0     0%   100%      7.70s 94.02%  <unknown>
         0     0%   100%      7.70s 94.02%  [libdyld.dylib]
         0     0%   100%      0.02s  0.24%  [libsystem_c.dylib]
         0     0%   100%      0.11s  1.34%  [libsystem_platform.dylib]
         0     0%   100%      7.70s 94.02%  main
         0     0%   100%      0.02s  0.24%  std::__1::__stdoutbuf::overflow
         0     0%   100%      0.02s  0.24%  std::__1::basic_ostream::operator<<
         0     0%   100%      0.02s  0.24%  std::__1::basic_ostream::put
         0     0%   100%      0.02s  0.24%  std::__1::endl
         0     0%   100%      0.02s  0.24%  std::__1::list::size

Note that we can safely ignore the warnings that some of the *.dylib files cannot be symbolized as on MacOS Big Sur, Apple made changes such that dynamically linked libraries can no longer be searched as files within the system. Even if we were able to symbolize those libraries, this won't really matter as we will likely not be optimizing them anyways.
However, this approach is very cumbersome from a UX perspective, as it doesn't really build a derivation, but rather installs the binary in a user-defined path. As such, any changes would require the user to manually input commands to modify/delete the binaries. Instead, we should focus on making a working pprof derivation out of the box, such that all nix features work seamlessly.
So, to summarize, there are three approaches to solving this error:

Update revision to a "working", but old build (since it doesn't run test cases on c files)
Install go separately, build head commit of pprof (this works without error, albeit with a very cumbersome build UX)
Determine cause of why we are getting the SWIG/cgo errors when building in a nix-only environment (on builds that attempt run tests on C files). For future maintainability, this is the most robust approach to properly fix the issue (in my opinion).

Steps To Reproduce
Steps to reproduce the behavior:

Update pprof package (and its dependencies) to latest revision(s):

# pprof/deps.nix
[
  {
    goPackagePath = "github.com/chzyer/readline";
    fetch = {
      type = "git";
      url = "https://github.com/chzyer/readline";
      rev = "2972be24d48e78746da79ba8e24e8b488c9880de";
      sha256 = "104q8dazj8yf6b089jjr82fy9h1g80zyyzvp3g8b44a7d8ngjj6r";
    };
  }
  {
    goPackagePath = "github.com/ianlancetaylor/demangle";
    fetch = {
      type = "git";
      url = "https://github.com/ianlancetaylor/demangle";
      rev = "28f6c0f3b63983aaa99575ca3b693afff7996387";
      sha256 = "0rsq3622gd40f1x1l7caidsxrmzg1993ich2higwd94fqbxs1r83";
    };
  }
]

# pprof/default.nix
{ lib, buildGoPackage, fetchgit }:

buildGoPackage rec {
  pname = "pprof-unstable";
  version = "2021-08-27";
  rev = "02619b876842e0d0afb5e5580d3a374dad740edb";

  goPackagePath = "github.com/google/pprof";

  src = fetchgit {
    inherit rev;
    url = "git://github.com/google/pprof";
    sha256 = "0j9y1a8hcbmd6rk0lmlqlxyzbnn9a0qnfh3g78cyip3wk9z47p0d";
  };

  goDeps = ./godeps.nix;

  meta = with lib; {
    description = "A tool for visualization and analysis of profiling data";
    homepage = "https://github.com/google/pprof";
    license = licenses.asl20;
    longDescription = ''
      pprof reads a collection of profiling samples in profile.proto format and
      generates reports to visualize and help analyze the data. It can generate
      both text and graphical reports (through the use of the dot visualization
      package).

      profile.proto is a protocol buffer that describes a set of callstacks and
      symbolization information. A common usage is to represent a set of sampled
      callstacks from statistical profiling. The format is described on the
      proto/profile.proto file. For details on protocol buffers, see
      https://developers.google.com/protocol-buffers

      Profiles can be read from a local file, or over http. Multiple profiles of
      the same type can be aggregated or compared.

      If the profile samples contain machine addresses, pprof can symbolize them
      through the use of the native binutils tools (addr2line and nm).

      This is not an official Google product.
    '';
  };
}

In a nix-shell, attempt to build the package

with import <nixpkgs> {};

mkShell {
  name = "profiling";
  buildInputs = [
    gperftools
    pprof
    llvmPackages_11.llvm # llvm-symbolizer
    llvmPackages_11.clang # to link core libs
  ];
  # Automate process of extracting correct paths to binaries. This approach
  # works despite the fact that there might be multiple different versions of
  # the pkg of interest, as each version has its own unique hash.
  shellHook = ''
    clang_dir=$(ls -l /nix/store/ | grep ${llvmPackages_11.clang.pname + "-" +
      llvmPackages_11.clang.version} | grep '^d' | awk '/ / {print $NF}')
    export PATH="/nix/store/$clang_dir/bin:$PATH"

    pprof_dir=$(ls -l /nix/store/ | grep ${pprof.pname + "-" + pprof.version} \
      | grep '^d' | awk '/ / {print $NF}')
    export PATH="/nix/store/$pprof_dir/bin:$PATH"
  '';
}

Expected behavior

The pprof package builds properly and we are able to see correctly symbolized, human-readable profiling reports.
Screenshots

Not necessary, but can attach if it would help.
Additional context

N.A.
Notify maintainers

Since you seem to have made major changes to these files:
@vdemeester
@vcunat
Metadata

[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-darwin"`
 - host os: `Darwin 20.5.0, macOS 10.16`
 - multi-user?: `no`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.3.8`
 - channels(ayushkarnawat): `"darwin-20.09pre246307.9b8530e3721, nixpkgs-21.03pre250093.0da76dab4c2"`
 - nixpkgs: `/Users/ayushkarnawat/.nix-defexpr/channels/nixpkgs`

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

I'm not good with darwin or go, so I most likely won't be of much help.  The last time I just kept using the "old" version of pprof from gperftools (on linux).

Thanks @vcunat, I suspect that it has something to do with certain build instructions preventing cgo/SWIG from being detected when building the package. In fact, I tried compiling with both (and associated flags) to see if that would resolve the issue, but it didn't work (and yes, I checked that they were included in the PATH).
Digging deeper, I was curious as to how buildGoPackage is building packages, and thanks to documentation, found that it is no longer the recommended method for building go packages. It is now deprecated in favor of buildGoModule. Rather than specifying the dependencies and associated sha256 in a deps.nix file, the buildGoModule has an intermediate step which automatically fetches and builds an (intermediate) derivation of all the dependencies the main package requires, and uses that to build the final binary. Simply using this function resolved the issue.
I still do not understand why the original error is occurring when using buildGoPackage, but that probably does not matter now. I will open a PR with the fix (reproduced below for completeness):
# pprof/default.nix
{ lib, buildGoModule, fetchgit }:

buildGoModule rec {
  pname = "pprof-unstable";
  version = "2021-08-27";
  rev = "02619b876842e0d0afb5e5580d3a374dad740edb";

  src = fetchgit {
    inherit rev;
    url = "git://github.com/google/pprof";
    sha256 = "0j9y1a8hcbmd6rk0lmlqlxyzbnn9a0qnfh3g78cyip3wk9z47p0d";
  };

  # intermediate fetcher derivation
  vendorSha256 = "0yl8y3m2ia3cwxhmg1km8358a0225khimv6hcvras8r2glm69h3f";
  runVend = false;

  meta = with lib; {
    description = "A tool for visualization and analysis of profiling data";
    homepage = "https://github.com/google/pprof";
    license = licenses.asl20;
    longDescription = ''
      pprof reads a collection of profiling samples in profile.proto format
      and generates reports to visualize and help analyze the data. It can
      generate both text and graphical reports (through the use of the dot
      visualization package).

      profile.proto is a protocol buffer that describes a set of callstacks
      and symbolization information. A common usage is to represent a set of
      sampled callstacks from statistical profiling. The format is described
      on the proto/profile.proto file. For details on protocol buffers, see
      https://developers.google.com/protocol-buffers

      Profiles can be read from a local file, or over http. Multiple profiles
      of the same type can be aggregated or compared.

      If the profile samples contain machine addresses, pprof can symbolize
      them through the use of the native binutils tools (addr2line and nm).

      This is not an official Google product.
    '';
  };
}


[Optimizers][Dont-Merge] Fix for intermittent NaNs.

Fix intermittent nans coming from FusedAdam.
CC : @jithunnair-amd

Looks like details of failing checks are not available somehow: "Problem accessing http://ml-ci.amd.com:21096/job/pytorch/job/apex-rocm/30/    Reason: Not Found"
VPN on/off doesn't change this. What is the correct address to see the build logs?

Closing this PR as this change affects other things.


M3.6: Ensure that the data browser recovers from downtime responsively (to meet 95% uptime objective, and to avoid more than 24 hours constant downtime)

@tburdett commented on Tue Dec 04 2018
┆Issue Number: HCAB-373

I'm closing this epic tickets.  Availability is something you architect for up front, then measure in operation and correct as necessary.