Leveraging hydra-zen for auto-generated and validated Hydra configs

Hello! Thanks for creating this repo -- it looks like it will be very useful. Hydra + PyTorch Lightning is definitely a killer one-two punch!
I wanted to bring your attention to hydra-zen, which is a library that adds functionality to Hydra and simplifies the process of writing configs. I think that this could be very handy for recipes.
Rather than hand-write YAML configs, you can auto-generate configs for objects using hydra_zen.builds.  Furthermore, these config-generating functions provide strict runtime and static checking of the configs, making it trivial to validate all of recipes configs during nightly builds.
Another perk of leveraging hydra-zen is that it can make recipes less dependent on the specific directory layout of your directory structure; e.g. you could avoid having users be required to add a file specifically to torchrecipes/launcher/.
I could go on and on, but I'll leave it at that. I would be happy to provide more details, answer questions, etc. I hope that you find this to be useful 😄
Here are our docs: https://mit-ll-responsible-ai.github.io/hydra-zen/
Our code (our project is very well-tested!): https://github.com/mit-ll-responsible-ai/hydra-zen
And a quick example of using hydra-zen + PyTorch lightning: https://mit-ll-responsible-ai.github.io/hydra-zen/how_to/pytorch_lightning.html

Hi @rsokl! Thank so much for taking a look at this repo, and for the feedback you're providing! This is exactly the sort of input we're looking to get from the community as we co-develop our recipes vision with the PyTorch (and associated) communities!
@tangbinh, given you've been spending a bit of time looking into how we can further improve our integration with hydra and our config-system, if you haven't already, it might be worth taking into account hydra-zen and what that might look like if we take this approach.
From a quick glance at what @rsokl shared, it certainly appears to address at least a subset of the concerns we've heard from our Meta users regarding the existing recipes!

Wonderful! I think it is likely that my colleagues and I hit a lot of the same issues that you are alluding to. Using hydra-zen ended up being quite transformational in terms of removing the repetitiveness and hotspots for mistakes that made Hydra taxing for us to use at-scale across PyTorch-based projects.
We are finally mature enough, both in terms the stability of our code and the quality of our docs, to start putting hydra-zen on peoples' radars (you are the first we reached out to!). That being said, we have been collaborating with a core-dev from omegaconf/Hydra for a while now, to help ensure we maintain broad compatibility and synergy with Hydra.


Links under 'Next steps' open the existing article, forming a link loop

The 'Connecting Microsoft Advanced Threat Analytics to Azure Security Center' link points to https://docs.microsoft.com/en-us/azure/security-center/security-center-ata-integration, but it opens the existing article, which is frustrating and redundant.
The 'Connecting Azure Active Directory Identity Protection to Azure Security Center' link points to https://docs.microsoft.com/en-us/azure/security-center/security-center-aadip-integration, but it opens the existing article, which is frustrating and redundant.

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

ID: 3aa2f48e-a8f8-fb03-753b-9e3b76075b96
Version Independent ID: 2554dcd1-27a2-3a32-4097-b17dd261a1ad
Content: Integrate security solutions in Azure Security Center
Content Source: articles/security-center/security-center-partner-integration.md
Service: security-center
GitHub Login: @rkarlin
Microsoft Alias: rkarlin

@steverlabs Thanks for the feedback !  I have assigned this issue to content author to investigate and update the document as appropriate.

@steverlabs Hi Steve, thanks for your feedback! I'm looking into it and trying to untangle the links there and figure out where the original articles disappeared to. I'll update you when I figure it out.
thanks again for letting us know!

We removed the broken links. thanks for pointing it out.

#please-close


Node.js: enable versions 12 and 13 and makes 12 the default

Allow to install Node version 12 to 13 and change the default version from 10 to 12 as it is now the current LTS.

This PR is an improvement

[x] Documentation is written
[x] parameters.yml.dist is updated
[x] playbook.yml.dist is updated
[x] Changelog was updated

@Seeow Thanks!


audio support

Hi everyone I'll be working on this issue in this PR: https://github.com/iejMac/clip-video-encode/pull/26

Hi everyone,  I'll be working on this issue in this PR: https://github.com/iejMac/video2numpy/pull/19, please feel free to contact me if you have any suggestions!

What needs to be done here:

Implement an AudioReader class similar to the current FrameReader class which reads in a list of videos and allows iteration over the numpy arrays representing the audio from the videos.

P.S. Yuchen's PR has a lot of the work already done, in handle_youtube and read_vids_cv2


Add support for column sorting

Should sorting be limited to data already loaded on the client? Or do we need a hook for sorting all data (including on the web server)? Probably the latter.

Implemented by PR #30


feat(vertical-navigation): added showMenuCollapsed attribute

This allows developers to maintain collapsed state, for the hamburger menu, should the page be refreshed.
Also re-ordered inputs alphabetically.

Deploy preview for patternfly-ng ready!
Built with commit f681c8cdd3ccbb8e534e947f2f7c8e24bff8df83
https://deploy-preview-441--patternfly-ng.netlify.com

:tada: This PR is included in version 4.4.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Plans for dotnet 9 support?

I like using mapping tools for sql projection, and Mapster.Tool code gen abilities makes sql projection super easy to create and maintain, as well as blazing fast at runtime. This library seems as easy sell to an organization as an architect, but my concern is that the tool is still at prerelease, with the last version released 9/2023.
Are there plans to move Mapster.Tool past prerelease? Will it be updated to support dotnet 9?
Also, the library also hasn't been updated since 9/2023. I've seen responses in other issues assuring that the library isn't dead, but still nothing new.
I would love to make the recommended mapping solution in my company's reference architecture Mapster, but would I be recommending a maintained library?

Following up - Now that .NET 9 has been out for 2 weeks, are there any plans to release a version of the library and tool that are compatible?
Or is this project abandoned?


[BEAM-7434] [BEAM-5895] and [BEAM-5894] Fix upgrade to rabbit amqp-client 5.x

In https://github.com/apache/beam/pull/9900 I migrated the rabbitmq amqp client from the 4.x line to the 5.x line. This upgrade removed the deprecated QueueingConsumer so I switched to a pull-based rather than push-based api.
It turns out this change had a semantic difference. The channel.basicGet (pull) is a blocking call. If no messages are delivered to the caller for a long period of time (for example, if no messages have been published to rabbit for some time), the Runner may detect that the reader is stuck and fail with:

java.io.IOException: Failed to advance source: ...

So it seems it's imperative to use a non-blocking read.
In hindsight, upgrading libraries and migrating API approaches was too much to tackle at once. I suspect it's possible to appropriately utilize the pull-based API with RPC timeouts while keeping the channel open, but that shouldn't be a blocker to having just upgraded the libraries.
In this PR, I restore the 4.x QueueingConsumer implementation, brought into Beam's code base, and use that just like we did before the library upgrade. The minor differences are documented.
R: @jbonofre

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---

---
---

Java

Python

---

---
---

XLang
---
---
---

---
---
---

Pre-Commit Tests Status (on master branch)

---
Java
Python
Go
Website

Non-portable

Portable
---

---
---

See .test-infra/jenkins/README for trigger phrase, status and link of all Jenkins jobs.

retest this please

Looks like this needs a rebase can you please do it @drobert

@iemejia I need to revisit this PR. I may cancel it altogether. Since I put this up, I've been having some good conversations about how rabbit should behave on the Beam Dev DL. I'm partway through a significant refactor right now, and I need to revisit whether this PR has merit.

Hi @drobert just pinging in case I can help with something.

It seems this needs a rebase now too.

It seems this needs a rebase now too.

Apologies. My organization has ceased working with Beam so I've had a bit less time to work on this recently. I'm hoping to get back to this in the next few weeks.


Add svelte support

This adds a transformer for svelte. (svelte3, latest version)

Please don't ignore eslint rules.

Sorry, I did forget to run eslint.

Thanks for the review! Here it is again. ready

This looks pretty cool, I wasn't really expect it to work with Svelte before. Thanks a lot!

This looks pretty cool, I wasn't really expect it to work with Svelte before. Thanks a lot!

I wasn't too, but when i realised there was no "Vue" in the package name I thought that it might be worth trying;)

Waiting for feedback from https://github.com/sveltejs/vite-plugin-svelte/issues/46.
It seems like the new method works, but the svelte vite plugin does not support the older svelte versions. If it's already not supported by the svelte team, there's no need for it to be supported here.

not sure this is a good idea. using a component in a svelte file without explicit import is likely going to cause issues with language tools eg. vscode extension . cc @dummdidumm

also vite-plugin-svelte uses enforce: pre so I wonder how you're able to transform before it when you add this plugin after sveltes. and last but not least, a svelte preprocessor would be a more svelty way to achieve this

not sure this is a good idea. using a component in a svelte file without explicit import is likely going to cause issues with language tools eg. vscode extension . cc @dummdidumm

Yeah it will. The language tools have no way to know the imports are added somewhere else. If you use JavaScript without type checking it will of course be okay but in other cases it will show errors.
I'm also not entirely sure if this saves that much. You don't have to type the import, but that's all, Svelte does not need to register the components like Vue does. Using the Svelte extension you also should get import auto completion.

You are right. The infrastructure was juste already set, so I though I might as well try it. Meanwhile, I'll have a look at preprocessors


introduce SRC_LOG_SCOPE_LEVEL

SRC_LOG_SCOPE_LEVEL allows operators to override the log level for
specific loggers and there children. It is an comma seperated
environment variable which takes a logger name and level. If a log
message matches the logger name (or a parent of it does) AND the log
level allows it then it will always be logged. A parent is defined by
how we join scoped loggers with a ".".
The motivation for this is in production we log with very low verbosity,
but when testing a feature it can be useful to turn up the verbosity for
specific logs. For example while I was developing hybrid searcher it
would of been great to make just that debug, but avoid debug for
everything else due to the noise.
Test Plan: Added unit tests which test from the envar level.

Codecov Report

Merging #46 (dcb67b6) into main (9df3f0f) will increase coverage by 4.58%.
The diff coverage is 72.72%.

@@            Coverage Diff             @@
##             main      #46      +/-   ##
==========================================
+ Coverage   55.24%   59.83%   +4.58%
==========================================
  Files          15       15
  Lines         581      600      +19
==========================================
+ Hits          321      359      +38
+ Misses        246      220      -26
- Partials       14       21       +7

Impacted Files
Coverage Δ

init.go
0.00% <ø> (ø)

sinks_output.go
49.12% <70.00%> (+49.12%)
:arrow_up:

logtest/logtest.go
78.08% <100.00%> (ø)

levels.go
66.66% <0.00%> (+66.66%)
:arrow_up:

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

btw did some manual testing as well. This made it so only the debug logs for search requests show up in zoekt. If I turn on debug logging globally there are many more logs:
go work init . ../log
SRC_LOG_SCOPE_LEVEL=searcher=debug go run ./cmd/zoekt-webserver


no twitter for google fetch. close #882

Deploy preview for landscape ready!
Built with commit c3d4377558d80d1c2c1e70ddbdf412fdb52fa5d3
https://deploy-preview-885--landscape.netlify.com


[WIP] Support exact amount for account to account transactions

Do not add any "gas buffer" (currently 25% for any transaction) when we realize that the transaction is between 2 accounts, and therefore costs invariably 21'000 Wei
Pass the gas information to the parity_postTransaction RPC to prevent any error from the node. The node knows nothing about the transaction and does not estimate gas (too costly) so it defaults to an amount of gas that is much greater than 21000.
Should fix https://github.com/paritytech/fether/issues/265 and #250

Should be good to go for ETH transactions, albeit dodgy code, let me know what you think.
I was able to send the exact amount of ETH and leave an account to 0, I tested and didn't break sending tokens so far.
I want to add the gas field in the RPC also for Tokens transfer, so that we avoid weird answers from the node. I'll do this once you have a first look at this code.

For future reference, if anyone is reading this PR:
An alternative, cleaner but more complicated solution would be to put gas as a derived form value, using https://github.com/final-form/final-form-calculate, for a future PR.

Let me know what you think about that, I could use final-form-calculate, it works as expected for ETH and tokens, but I can't get rid of a warning message: Warning: Form decorators should not change from one render to the next as new values will be ignored

Thanks a lot, I address the comments and it should look better now. No more error, token transfer also have gas in postTransaction now.

Thanks again, comments addressed :)

No problem, I think it makes a lot of sense and simplify things quite a bit. I've tested, as always, and it looks like it works :)

:+1:


Updates to code not showing in website.

Noticed any changes being made don't get pushed to the website. We're missing a step somewhere I think.

Got it
Commits are made to the master branch, but you gotta update the gh-pages branch for the changes to take effect.
git checkout gh-pages
git rebase master
git push origin gh-pages
git checkout master


Text encroaching into other modules

I've just noticed tonight, and it may be because of the long text below the icons today, but it's encroaching into the modules on the left hand side as well as pulling over to the left the weather forecast above it (the weather forecast keeps to the right hand side absolutely fine when I remove the uk-pollen-forecast config

			 	  {
    module: 'mmm-uk-pollen-forecast',
    position: 'top_right',
    config: {
        updateIntervalHours : 2,    // update every 2 hrs
        region : 'se',              // forecast for London & SE England
        first_display_date_DD_MM: '01-04', // show module from 1st April
        last_display_date_DD_MM: '01-09'  // hide module after 1st September

    }
 },

Ignore me, I'm an idiot !


Outdated configuration option names - Windows Update for Business

Hello:
The configuration information in this document seems to be outdated.
In the Windows 11 v3.0 and other newer ADMX files, the following path does not exist:
Computer Configuration/Administrative Templates/Windows Components/Windows Update/ Windows Update for Business
It appears that it is now part of:
Computer Configuration/Administrative Templates/Windows Components/Windows Update/ Manage Updates Offered From Windows Update
This path is referenced a few times in this article.  I believe that the article needs to be updated to include this change, as it is unclear to people setting up Windows Update for Business when the examples reference a path that no longer exists.

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

ID: 1684b30a-3187-85ce-0cf7-10d1b8298b7f
Version Independent ID: 9b7170fc-2b39-8e46-7898-894a45de8623
Content: Configure Windows Update for Business - Windows Deployment
Content Source: windows/deployment/update/waas-configure-wufb.md
Product: windows-client
Technology: itpro-updates
GitHub Login: @mestew
Microsoft Alias: mstewart

To make it easier for you to submit feedback on articles on learn.microsoft.com, we're transitioning our feedback system from GitHub Issues to a new experience.
As part of the transition, this GitHub Issue will be moved to a private repository. We're moving Issues to another repository so we can continue working on Issues that were open at the time of the transition. When this Issue is moved, you'll no longer be able to access it.
If you want to provide additional information before this Issue is moved, please update this Issue before December 15th, 2023.
With the new experience, you no longer need to sign in to GitHub to enter and submit your feedback. Instead, you can choose directly on each article's page whether the article was helpful. Then you can then choose one or more reasons for your feedback and optionally provide additional context before you select Submit.
Here's what the new experience looks like.
Note: The new experience is being rolled out across learn.microsoft.com in phases. If you don't see the new experience on an article, please check back later.
First, select whether the article was helpful:

Then, choose at least one reason for your feedback and optionally provide additional details about your feedback:

Article was helpful
Article was unhelpful

Finally, select Submit and you're done!


False Positive on various eclipse jars

Lot of false positive reported as CVE-2020-27225
while comming from 4.19 (2021-03) where this is supposed to be fixed
org.eclipse.core.databinding.beans_1.7.200.v20210111-0759.jar reported as cpe:2.3:a:eclipse:platform:1.7.200
org.eclipse.equinox.servletbridge_1.6.0.v20200511-1902.jar reported as cpe:2.3:a:eclipse:platform:1.6.0:::::::*
org.eclipse.osgi_3.16.200.v20210226-1447.jar reported as eclipse:platform:3.16.200:::::::*
org.eclipse.core.commands_3.9.800.v20201021-1339.jar reported as cpe:2.3:a:eclipse:platform:3.9.800:::::::*
org.eclipse.core.contenttype_3.7.900.v20210111-0918.jar reported as cpe:2.3:a:eclipse:platform:3.7.900:::::::*
org.eclipse.core.databinding.observable_1.10.0.v20200730-0848.jar reported as cpe:2.3:a:eclipse:platform:1.10.0:::::::*
org.eclipse.core.databinding.property_1.8.100.v20200619-0651.jar reported as cpe:2.3:a:eclipse:platform:1.8.100:::::::*
org.eclipse.core.databinding_1.10.100.v20200926-1123.jar reported as cpe:2.3:a:eclipse:platform:1.10.100:::::::*
org.eclipse.core.expressions_3.7.100.v20210203-1000.jar reported as cpe:2.3:a:eclipse:platform:3.7.100:::::::*
org.eclipse.core.jobs_3.10.1100.v20210111-0815.jar reported as cpe:2.3:a:eclipse:platform:3.10.1100:::::::*
org.eclipse.core.runtime_3.20.100.v20210111-0815.jar reported as cpe:2.3:a:eclipse:platform:3.20.100:::::::*
org.eclipse.equinox.app_1.5.100.v20210212-1143.jar reported as cpe:2.3:a:eclipse:platform:1.5.100:::::::*
org.eclipse.equinox.common_3.14.100.v20210212-1143.jar reported as cpe:2.3:a:eclipse:platform:3.14.100:::::::*
org.eclipse.equinox.console_1.4.300.v20210211-2058.jar reported as cpe:2.3:a:eclipse:platform:1.4.300:::::::*
org.eclipse.equinox.event_1.6.0.v20210212-1143.jar reported as cpe:2.3:a:eclipse:platform:1.6.0:::::::*
org.eclipse.equinox.http.registry_1.2.0.v20200614-1851.jar reported as cpe:2.3:a:eclipse:platform:1.2.0:::::::*
org.eclipse.equinox.http.servletbridge_1.2.0.v20210120-1213.jar reported as cpe:2.3:a:eclipse:platform:1.2.0:::::::*
org.eclipse.equinox.http.servlet_1.7.0.v20210202-1229.jar reported as cpe:2.3:a:eclipse:platform:1.7.0:::::::*
org.eclipse.equinox.jsp.jasper.registry_1.1.400.v20200422-1833.jar reported as cpe:2.3:a:eclipse:platform:1.1.400:::::::*
org.eclipse.equinox.jsp.jasper_1.1.500.v20200422-1833.jar reported as cpe:2.3:a:eclipse:platform:1.1.500:::::::*
org.eclipse.equinox.preferences_3.8.200.v20210212-1143.jar reported as cpe:2.3:a:eclipse:platform:3.8.200:::::::*
org.eclipse.equinox.registry_3.10.100.v20210212-1143.jar reported as cpe:2.3:a:eclipse:platform:3.10.100:::::::*
org.eclipse.help_3.8.800.v20200525-0755.jar reported as cpe:2.3:a:eclipse:platform:3.8.800:::::::*
org.eclipse.osgi.compatibility.state_1.2.300.v20210212-1137.jar reported as cpe:2.3:a:eclipse:platform:1.2.300:::::::*
org.eclipse.osgi.services_3.10.0.v20210212-1137.jar reported as cpe:2.3:a:eclipse:platform:3.10.0:::::::*
org.eclipse.osgi.util_3.6.0.v20210212-1137.jar reported as cpe:2.3:a:eclipse:platform:3.6.0:::::::*

Please use the false positive template when reporting false positives. You need to provide the coordinates for the reported false positives (i.e. maven or gradle syntax with the group, artifact, and version). I've tried to find a few of these and I am not able to re-produce the FP. Please open a new ticket with the dependency coordinates if you are still seeing an issue using the latest version of ODC.


Fix build badge

This fixes the build badge

Current dependencies on/for this PR:

main

PR #25   👈

This comment was auto-generated by Graphite.


fixed issue #1617 - [closed]

In GitLab by @liyanq528 on Jul 31, 2018, 08:33
Merges 20180628 -> dev

In GitLab by @wknet123 on Nov 13, 2019, 11:18
The test coverage for backend is FAIL  ,The test coverage for frontend is 38.11   , check  consolse log

In GitLab by @wknet123 on Nov 13, 2019, 11:18
The test coverage for backend is FAIL  ,The test coverage for frontend is 38.11   , check  consolse log

In GitLab by @wknet123 on Nov 13, 2019, 11:18
closed


Rmove all noexcept in pipeline model

What problem does this PR solve?
Issue Number: ref #7404
Problem Summary:
What is changed and how it works?

Rmove all noexcept in pipeline model
Move event->onTaskFinish from ~EventTask() to EventTask::finalizeImpl to avoid the noexcept of destructor.
refine TaskQueue and WaitTaskList to make sure that task::finalize will be called after finish() is called.// After finish is called, the submitted task will be finalized directly and will not be taken.
// And the tasks in the queue can still be taken normally.

add TaskHelper::CATCH_AND_TERMINATE to log the error from TaskThreadPool::loop and WaitReactor::loop

Check List
Tests

[x] tsan passed

gtest_filter=*Event*
gtest_filter=*TaskScheduler*
gtest_filter=*Executor*
gtest_filter=*ComputeServerRunner*
gtest_filter=*TestWaitingTaskList*
gtest_filter=*FIFOTestRunner*

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/run-sanitizer-test tsan

/run-sanitizer-test tsan

https://ci.pingcap.net/blue/organizations/jenkins/tiflash-sanitizer-daily/detail/tiflash-sanitizer-daily/1083/pipeline

/assign

/merge


added clarification about mixing installation of Win32 and LOB

https://github.com/MicrosoftDocs/windows-itpro-docs/issues/6537

Dear @JohanFreelancer9,
kindly do the copy editor review for this PR.
Dear @e0i, for your information
Thank you

This isn't quite a good enough explanation, although it comes directly from https://docs.microsoft.com/mem/intune/apps/lob-apps-windows (i.e. if you do it then app install may fail). I think it's assumed something might go wrong but the question is why. In fact, this is just a warning and mostly because if you mix various app installs and get multiple failures, then it can get complex to troubleshoot.  I'm going to edit this a bit further.


.xz file viewer

XZ providers better compression than GZ
Can you update this to support GZ File Viewer also please?

Sorry, took me a while. I added a pre release for xz and zstandard compression. Hope it works. I don't know anything about the parameters for xz though. If the lib I used supports certain settings I can change the settings dialog. For now, I only exposed compr-level, buffer size and mutli-threading.


Add automatic battery power to the client unit so that power doesn't have to be switched off to go between AC and battery power.

Solved. See schematics for PCB design.


Reg\ Register Operation\ Add Facility - Small Aggregate Fields

Given I am on the add facility form, if my facility type is Small Aggregate, then the only fields that appear are Facility Name and Facility Type; all other facility form fields are hidden.
the facility name field is pre-populated by the SWRS import, but is still editable.

attn: @andrea-williams are there any other details required in this ticket, or is the description currently clear?

@patriciarussellCAS tagging you for visibility, I'm not sure if you have seen this ticket yet!

Thanks @zoeyli-46 - I'm curious why we're not showing the BCGHGID on the facility pages, including small aggregate. Do you happen to know the reason behind this? It might be a BA decision I'm not aware of.

Hey @patriciarussellCAS ! thats a great flag, the facility page does have a BCGHGID, its just not there when you are "adding a facility" because external users cant give themselves a BCGHGID. But small aggregates do have BCGHGID so in the view mode the BCGHGID should show up. I'll reflect that on the wireframe


[sw,testing] Fix clkmgr clock counts for CDC

CDC synchronizers add an extra cycle of uncertainty to clock counts.
Fixes #17145
Signed-off-by: Guillermo Maturana<EMAIL_ADDRESS>

Unless someone sees a high urgency in merging this change, let's hold it while we increase the pass rate of TLTs. Triaging should keep this in mind, though. CC @johngt

Sounds good, thanks @matutem.
(My concern back in January was mainly that we were focusing on increasing the TLT pass rate and thus temporarily wanted to pause TLT changes that didn't come directly from debugging & fixing TLTs.)


Update the example project with the new json config.

I spent some 3 hours trying to figure out why the particle system wasn't working... until I finally went and noticed the examples code https://pixijs.io/particle-emitter/examples/fountain.html and the editor code both have the json in a different format from the one in the Readme https://github.com/pixijs/particle-emitter/blob/master/README.md
An alternative would be include both styles.

The examples do have the JSON in the latest format (with some older boilerplate to handle loading and playback), only the editor is out of date.

I've updated the readme with a note about the editor output needing an extra step to be usable.


Binary_Search_Iter_Recur.cpp

Contains both Iterative and Recursive method for Binary Search. This solves #65

Thanks for contributing!


Fix language issue #37

#37 pull request

Did you check this locally? We just added something in https://github.com/okestonia/koroonakaart/commit/597b93a9fe5720d09894ca3535759ed176c18f02and https://github.com/okestonia/koroonakaart/commit/3e5e5f9514b63eafd8940528e1aa9f1a608f5335

Well, it turns out sleep is important. Thanks for the PR. I have implemented it and reverted most of the changes by @Kypsis  and seems to be working now.

Sorry, missed out on @Kypsis commits. Now pulling again seemed break it as you said. Thanks!

It's okay, I made some small changes so master should build properly now.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Ivar Avi<EMAIL_ADDRESS>Sent: Monday, March 23, 2020 11:00:24 PM
To: okestonia/koroonakaart<EMAIL_ADDRESS>Cc: keeganmcbride<EMAIL_ADDRESS>State change<EMAIL_ADDRESS>Subject: Re: [okestonia/koroonakaart] Fix language issue #37 (#42)
Sorry, missed out on @Kypsishttps://github.com/Kypsis commits. Now pulling again seemed break it as you said. Thanks!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/okestonia/koroonakaart/pull/42#issuecomment-602853444, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AE7MKR3YZZIYERB4JOEZFBDRI7EWRANCNFSM4LSESSHA.


Nested dependent name does not get a semantic highlighting

In the following code:
template <typename T>
struct waldo {
  using location = typename T::resolver::location;  // <--
};

on the indicated line, resolver does not get a semantic highlighting.

Isn't this appropriate?
A dependent name is an identifier with unknown meaning. The editor should highlight it as a generic identifier. This is what happens if clangd provides no style.

We have a DependentName highlighting kind for dependent names. For example, that's what location in the above example gets.

Err, sorry, we have DependentType for dependent types and DependentName for dependent non-types. location is a DependentType, resolver should be as well.
That provides more information than leaving them uncolored, since we distinguish between types and non-types.

Having semantic colors for dependent names also helps distinguish "this is confirmed to be a dependent name" vs. "this is a place where we have an incomplete AST / a clangd bug / etc.".

Proposed fix


refactor(Server): Updated MongoDB variable for Railway

Description
Updated server.js MongoDB variable to connect with Mongodb plugin in railway
Acceptance Criteria

[x] Create MongoDB service in railway
[x] Apply variable provided from plugin into server file
[x] Test user sign up

Type of Changes
Use a check for  ✓ the following type of changes:

Type

:bug: Bug fix

:sparkles: New feature

✓
:hammer: Refactoring

:100: Add tests

:link: Update dependencies

:scroll: Docs

Updates
Before
N/A visual changes as just change in environment variable for MongoDB website
After

Testing Steps / QA Criteria
Postman signup Testing provide email and user password

Vercel not being used for project


Move some APIs to frontend to admin

Move GetReplicationMessagesResponse / GetDomainReplicationMessagesResponse / ReapplyEvents to admin endpoint

Coverage increased (+0.06%) to 67.83% when pulling ad433ec098268e7b099a0883aae461bebe3f4e3f on move-idl into 6c962c9d4996634c0224170dff81e7bbf4a0856b on master.


Publishing the Unit test results fails randomly

Hello.,
This is a scenario, we have observed., where the task Publishing Unit test results fails random on random basis., and re running them will success the job.
we have multiple microservices. and during the unit test cases job flow., some of the services fails randomly without any pattern and rerunning them would result in success run.
its failing with the error:  Request GET /search/issues?q=type%3Apr+repo%3A%22tr%2Fhighq_collaborate%22+6b180f9b738992922e67a8e90ed0fcc03b0df216&per_page=100 failed: Forbidden especially while its trying to create a comment in the github actions (either it could be a PR or a push event).
errors observed are:
Warning: Request GET /search/issues?q=type%3Apr+repo%3A%22tr%2Fhighq_collaborate%22+6b180f9b738992922e67a8e90ed0fcc03b0df216&per_page=100 failed with 403: Forbidden
Warning: Request GET /search/issues?q=type%3Apr+repo%3A%22tr%2Fhighq_collaborate%22+6b180f9b738992922e67a8e90ed0fcc03b0df216&per_page=100 failed with 403: Forbidden
Warning: Request GET /search/issues?q=type%3Apr+repo%3A%22tr%2Fhighq_collaborate%22+6b180f9b738992922e67a8e90ed0fcc03b0df216&per_page=100 failed with 403: Forbidden
Warning: Request GET /search/issues?q=type%3Apr+repo%3A%22tr%2Fhighq_collaborate%22+6b180f9b738992922e67a8e90ed0fcc03b0df216&per_page=100 failed with 403: Forbidden
Warning: Request GET /search/issues?q=type%3Apr+repo%3A%22tr%2Fhighq_collaborate%22+6b180f9b738992922e67a8e90ed0fcc03b0df216&per_page=100 failed with 403: Forbidden

Traceback (most recent call last):
File "/usr/local/lib/python3.8/site-packages/requests/adapters.py", line 440, in send
resp = conn.urlopen(
File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 876, in urlopen
return self.urlopen(
File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 876, in urlopen
return self.urlopen(
File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 876, in urlopen
return self.urlopen(
[Previous line repeated 7 more times]
File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 866, in urlopen
retries = retries.increment(method, url, response=response, _pool=self)
File "/action/publish/retry.py", line 78, in increment
retry = super().increment(method, url, response, error, _pool, _stacktrace)
File "/usr/local/lib/python3.8/site-packages/urllib3/util/retry.py", line 592, in increment
raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='api.github.com', port=443): Max retries exceeded with url: /search/issues?q=type%3Apr+repo%3A%22tr%2Fhighq_collaborate%22+6b180f9b738992922e67a8e90ed0fcc03b0df216&per_page=100 (Caused by ResponseError('too many 403 error responses'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/action/publish_test_results.py", line 387, in
main(settings, gha)
File "/action/publish_test_results.py", line 135, in main
Publisher(settings, gh, gha).publish(stats, results.case_results, conclusion)
File "/action/publish/publisher.py", line 150, in publish
pulls = self.get_pulls(self._settings.commit)
File "/action/publish/publisher.py", line 168, in get_pulls
issues = list(self._gh.search_issues(f'type:pr repo:"{self._settings.repo}" {commit}'))
File "/usr/local/lib/python3.8/site-packages/github/PaginatedList.py", line 56, in iter
newElements = self._grow()
File "/usr/local/lib/python3.8/site-packages/github/PaginatedList.py", line 67, in _grow
newElements = self._fetchNextPage()
File "/usr/local/lib/python3.8/site-packages/github/PaginatedList.py", line 199, in _fetchNextPage
headers, data = self.__requester.requestJsonAndCheck(
File "/usr/local/lib/python3.8/site-packages/github/Requester.py", line 354, in requestJsonAndCheck
*self.requestJson(
File "/usr/local/lib/python3.8/site-packages/github/Requester.py", line 454, in requestJson
return self.__requestEncode(cnx, verb, url, parameters, headers, input, encode)
File "/usr/local/lib/python3.8/site-packages/github/Requester.py", line 528, in __requestEncode
status, responseHeaders, output = self.__requestRaw(
File "/action/publish_test_results.py", line 157, in throttled_gh_request_raw
return gh_request_raw(cnx, verb, url, requestHeaders, input)
File "/usr/local/lib/python3.8/site-packages/github/Requester.py", line 555, in __requestRaw
response = cnx.getresponse()
File "/usr/local/lib/python3.8/site-packages/github/Requester.py", line 127, in getresponse
r = verb(
File "/usr/local/lib/python3.8/site-packages/requests/sessions.py", line 542, in get
return self.request('GET', url, **kwargs)
File "/usr/local/lib/python3.8/site-packages/requests/sessions.py", line 529, in request
resp = self.send(prep, **send_kwargs)
File "/usr/local/lib/python3.8/site-packages/requests/sessions.py", line 645, in send
r = adapter.send(request, **kwargs)
File "/usr/local/lib/python3.8/site-packages/requests/adapters.py", line 510, in send
raise RetryError(e, request=request)
requests.exceptions.RetryError: HTTPSConnectionPool(host='api.github.com', port=443): Max retries exceeded with url: /search/issues?q=type%3Apr+repo%3A%22tr%2Fhighq_collaborate%22+6b[180](https://github.com/tr/highq_collaborate/actions/runs/4988318327/jobs/8931533853#step:8:181)f9b738992922e67a8e90ed0fcc03b0df216&per_page=100Traceback (most recent call last):
File "/usr/local/lib/python3.8/site-packages/requests/adapters.py", line 440, in send
resp = conn.urlopen(
File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 876, in urlopen
return self.urlopen(
File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 876, in urlopen
return self.urlopen(
File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 876, in urlopen
return self.urlopen(
[Previous line repeated 7 more times]
File "/usr/local/lib/python3.8/site-packages/urllib3/connectionpool.py", line 866, in urlopen
retries = retries.increment(method, url, response=response, _pool=self)
File "/action/publish/retry.py", line 78, in increment
retry = super().increment(method, url, response, error, _pool, _stacktrace)
File "/usr/local/lib/python3.8/site-packages/urllib3/util/retry.py", line 592, in increment
raise MaxRetryError(_pool, url, error or ResponseError(cause))
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='api.github.com', port=443): Max retries exceeded with url: /search/issues?q=type%3Apr+repo%3A%22tr%2Fhighq_collaborate%22+6b180f9b738992922e67a8e90ed0fcc03b0df216&per_page=100 (Caused by ResponseError('too many 403 error responses'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/action/publish_test_results.py", line 387, in
main(settings, gha)
File "/action/publish_test_results.py", line 135, in main
Publisher(settings, gh, gha).publish(stats, results.case_results, conclusion)
File "/action/publish/publisher.py", line 150, in publish
pulls = self.get_pulls(self._settings.commit)
File "/action/publish/publisher.py", line 168, in get_pulls
issues = list(self._gh.search_issues(f'type:pr repo:"{self._settings.repo}" {commit}'))
File "/usr/local/lib/python3.8/site-packages/github/PaginatedList.py", line 56, in iter
newElements = self._grow()
File "/usr/local/lib/python3.8/site-packages/github/PaginatedList.py", line 67, in _grow
newElements = self._fetchNextPage()
File "/usr/local/lib/python3.8/site-packages/github/PaginatedList.py", line 199, in _fetchNextPage
headers, data = self.__requester.requestJsonAndCheck(
File "/usr/local/lib/python3.8/site-packages/github/Requester.py", line 354, in requestJsonAndCheck
*self.requestJson(
File "/usr/local/lib/python3.8/site-packages/github/Requester.py", line 454, in requestJson
return self.__requestEncode(cnx, verb, url, parameters, headers, input, encode)
File "/usr/local/lib/python3.8/site-packages/github/Requester.py", line 528, in __requestEncode
status, responseHeaders, output = self.__requestRaw(
File "/action/publish_test_results.py", line 157, in throttled_gh_request_raw
return gh_request_raw(cnx, verb, url, requestHeaders, input)
File "/usr/local/lib/python3.8/site-packages/github/Requester.py", line 555, in __requestRaw
response = cnx.getresponse()
File "/usr/local/lib/python3.8/site-packages/github/Requester.py", line 127, in getresponse
r = verb(
File "/usr/local/lib/python3.8/site-packages/requests/sessions.py", line 542, in get
return self.request('GET', url, **kwargs)
File "/usr/local/lib/python3.8/site-packages/requests/sessions.py", line 529, in request
resp = self.send(prep, **send_kwargs)
File "/usr/local/lib/python3.8/site-packages/requests/sessions.py", line 645, in send
r = adapter.send(request, **kwargs)
File "/usr/local/lib/python3.8/site-packages/requests/adapters.py", line 510, in send
raise RetryError(e, request=request)
requests.exceptions.RetryError: HTTPSConnectionPool(host='api.github.com', port=443): Max retries exceeded with url: /search/issues?q=type%3Apr+repo%3A%22tr%2Fhighq_collaborate%22+6b180f9b738992922e67a8e90ed0fcc03b0df216&per_page=100 (Caused by ResponseError('too many 403 error responses'))
##[debug]Docker Action run completed with exit code 1
##[debug]Finishing: Publish Unit Test Results
##[debug]Docker Action run completed with exit code 1
##[debug]Finishing: Publish Unit Test Results
Yaml script for this job is given as below: (note: all the env variables are been passed correctly)

name: Publish Unit Test Results
uses: EnricoMi/publish-unit-test-result-action@v1
if: always()
with:
files: '**/test-results/jUnitTest/*.xml'
check_name: "${{ matrix.junitsuite }}-Unit Test Results-${{ github.event.repository.name }}

This has been fixed in version v2:
uses: EnricoMi/publish-unit-test-result-action@v2


Ophyd-async devices for HFM and VFM

Required for #377
We need to implement an HFM and VFM device exposed the fields required for I22 nexus files (see linked issue).
Acceptance Criteria

HFM available
VFM available
Fields specified in #377 are exposed

We could also look into making the resultant devices compatible sub-devices of XBPMFeedback.
Would probably have to find a way to marry the two nearly-identical XBPMFeedback classes back together while we're at it. @DominicOram do you have any thoughts?

We could also look into making the resultant devices compatible sub-devices of XBPMFeedback.
Would probably have to find a way to marry the two nearly-identical XBPMFeedback classes back together while we're at it. @DominicOram do you have any thoughts?

That sounds like scope creep to me. I'm not sure if I agree/disagree if we should do it but I don't think it adds anything immediately

i03 doesn't use it's HFM but we do use the VFM. If we wanted to make a generic FocussingMirror we would have the following issues:

i03 has a fine_pitch that i22 doesn't have. This isn't actually used on i03 so we can remove
i03 has a VERT and LAT, I think these correlate to Y and X in i22 but we would need to discuss with controls/scientists

https://github.com/DiamondLightSource/dodal/issues/377 specifies that you want to read incident angle into the nexus file. Presumably this is calculated from the other axes? @Tom-Willemsen or @callumforrester do we have an idea of how it's derived?

^ I've asked one of the i22 scientists how incident_angle gets derived. Will update when I get a reply.

On i22, incident angle is just "pitch", for both the horizontal and vertical focusing mirrors.


Unity complex terrain mesh misbehave

Hi guys,
I'm doing some simulations in Unity+Mujoco and find out,  that for some reason complex terrain meshes misbehave. The MJ body moves away in space, and the object (ball in my case) is rolling on some invisible terrain.
I'll attach video to demonstrate the issue.
Operating system. Tried on MacOS 14.1.1 and Windows 10
MuJoCo version 3.0.1
Thank you in advance!
https://github.com/google-deepmind/mujoco/assets/81800286/d5f6ec02-38cc-4a0b-a436-4437a953a4a4

Is your terrain mesh a child of the root gameObject (and therefore a child of worldbody, and therefore static) or is it a child of some MuJoCo body?  If you grab the XML that's exported and load it in the standalone simulate app, does it suffer from the same problem?  If it's fine in simulate, then there may be a bug in synchronizing the transforms between Unity and MuJoCo -- if you can find it, we'd be grateful for a Pull Request!

Thanks for quick response.
So, the terrain mesh is a child of an empty object with Mj_body component on it.
I've just tried to export XML scene and found out that the terrain mesh looks pretty different. It looks like convex mesh around the original one.

Here is the original one:

The mesh itself procedure-generated in Unity and then added to Mj_geom mesh and meshfilter components.

By default, MuJoCo will only allow collisions with the convex hull of a mesh.  If you're using the mesh as a terrain, you could consider using heightfields.

Are heightfields currently supported by the plugin? As far as I can tell no, but I might have missed something. I think we could reopen this issue or a new one to track implementing it. I'm interested at trying to add support for it.

It would be awesome. And yes, so far there is no native support of height fields.

#1253 Now added basic support for heightfields with the Unity plugin. If you give it a go but run into any issues, let me know!


[core] Behavior PreemptQueuejobs thread

Describe the Bug
Preemptqueuejobs run every 60 seconds. for an AW that is slow in starting pods due to a large image pull time, this may cause premature AW preemption.
Codeflare Stack Component Versions
Please specify the component versions in which you have encountered this bug.
Codeflare SDK:
MCAD: v1.33 and above
Steps to Reproduce the Bug
NA
What Have You Already Tried to Debug the Issue?
NA
Expected Behavior
MCAD should give AW enough time to pull very large images before preemption.
Screenshots, Console Output, Logs, etc.
NA
Affected Releases
R1.33 and above
Additional Context
NA
Add any other information you think might be useful here.

closing, this issue can be fixed by adding requeuing stanza in schedulingSpec stanza


Json parse error

Hello, I am experiencing a JSON parse error, and I would like to fix/debug it myself, and I kindly ask for some initial tips on how to debug the app. Thank you so much for your support!

Found a way to debug the issue, problem lies in Serde so I am closing this. Sorry to bother you!

Sorry to bother you!

You're not bothering anyone!
Did the issue stem from you manually editing the file or did glim serialize it wrongly? If it's the latter I'd like to fix it in glim.

Glim serialized it incorrectly, the error is found at src/client.rs:263

Thanks!

supposedly fixed by #5, closing this for the time being

Hey I just wanted to let you know that this is now working as expected. Thank you so much for your support!


Add a new method Insert<TEntity, TResult> and InlineInsert<TEntity, TResult>

This is now checked-in. Will be available in the version > v1.8.4.


RHBPMS-75: Add latest Zanata translations to complete Japanese locali…

…sation of simulation.

jenkins retest this

+1


Added logs

Helps https://github.com/OpenDevin/OpenDevin/issues/2150#issuecomment-2141140899, https://github.com/OpenDevin/OpenDevin/issues/2150#issuecomment-2141164177

Here @isavita?
https://github.com/OpenDevin/OpenDevin/blob/f188abd7a3ddf0b77751184e943078123524dbd4/opendevin/core/config.py#L446-L447

Here @isavita?
https://github.com/OpenDevin/OpenDevin/blob/f188abd7a3ddf0b77751184e943078123524dbd4/opendevin/core/config.py#L446-L447

cc: @li-boxuan

Info sounds better, the warning under was when there is a parsing error.


/usr/bin/which: no ack-grep in (PATH)

I just updated and I'm getting this error when logging into my centOS box.

Here's a PR to fix this

Caused by #3165, looks like.


CLRProfilingAnalyzer - Identify if the below exception is fatal or not and if it can be fixed

System.IO.DirectoryNotFoundException : Could not find a part of the path 'D:\local\Temp\Reports\<REMOVING SITE NAME>\18-03-05\180305_1207268897\CLRProfilingAnalyzer\RD0003FF44E26F\reportdata\requests.json'.      at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)    at System.IO.FileStream.Init(String path, FileMode mode, FileAccess access, Int32 rights, Boolean useRights, FileShare share, Int32 bufferSize, FileOptions options, SECURITY_ATTRIBUTES secAttrs, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)    at System.IO.FileStream..ctor(String path, FileMode mode, FileAccess access, FileShare share, Int32 bufferSize, FileOptions options, String msgPath, Boolean bFromProxy, Boolean useLongPath, Boolean checkHost)    at System.IO.StreamWriter.CreateFile(String path, Boolean append, Boolean checkHost)    at System.IO.StreamWriter..ctor(String path, Boolean append, Encoding encoding, Int32 bufferSize, Boolean checkHost)    at System.IO.StreamWriter..ctor(String path, Boolean append)    at System.IO.File.CreateText(String path)    at ClrProfilingAnalyzer.Program.DumpSlowRequests(Dictionary2 iisRequests, ClrProfilingAnalyzerStats stats)
at ClrProfilingAnalyzer.Program.Main(String[] args)
`

Researched on it a bit and found that this is happening when an customer is choosing both "CLR Profiler" and "CLR Profiler with Thread Stacks" option. This is not possible in supportportalv2 so they are doing this possibly in the Support Portal or directly browsing to /DAAS.
Have a fix in mind for this from the back end side but we should just get rid of the Support Portal UI in the mean time.


Fixes for redhat yaml extension

fix for https://github.com/TypeFox/monaco-languageclient/issues/803

Thank you. LGTM 🎉

Btw, I've noticed that the build process is not deterministic... the generated common packages can change between builds. In addition to the missing common package dependencies on the main package, I hope that it won't break anything. I've started working on a refactor of the build process, but I didn't have the time to move forward on it for a few weeks now... I'll maybe find some time in the next week


Mark conformances as "used" when the GenericSignatureBuilder needs them.

When the type checker forms a GenericSignatureBuilder to process
requirements, the GSB may look up concrete conformances. Route such
requests through TypeChecker::conformsToProtocol() so they can be
marked as used. This fixes multi-file scenarios where conformances
from another file might be used only in a generic signature in the
primary file, e.g., rdar://problem/31759258.

@swift-ci please smoke test and merge


Is there 2.x backward compatibility layer?

I were preparing update from 2.12 to 3.0 and tried to compile flamethrower with it. It does not compile anymore because a lot of renamed classes. Is there any backward compatible layer, which would allow supporting old code without changes? Or some way, which might allow modified code to support both 3.0 and 2.12 version?

The main problem is that v3 represents a huge rewrite of the whole library with some changes to the API too in order to drastically reduce compilation times. It's not that easy to write a migration layer unfortunately.

I have made surprisingly lot of conversions by crude sed script:
s/uvw::TimerHandle/uvw::timer_handle/g
s/uvw::TimerEvent/uvw::timer_event/g
s/uvw::timer_handle::Time/uvw::timer_handle::time/g
s/uvw::TCPHandle/uvw::tcp_handle/g
s/uvw::UDPHandle/uvw::udp_handle/g
s/uvw::UDPDataEvent/uvw::udp_data_event/g
s/uvw::Loop/uvw::loop/g
s/uvw::loop::Time/uvw::loop::time/g
s/uvw::ErrorEvent/uvw::error_event/g
s/uvw::ConnectEvent/uvw::connect_event/g
s/uvw::CloseEvent/uvw::close_event/g
s/uvw::ShutdownEvent/uvw::shutdown_event/g
s/uvw::EndEvent/uvw::end_event/g
s/uvw::DataEvent/uvw::data_event/g
s/uvw::WriteEvent/uvw::write_event/g
s/uvw::IPv6/uvw::ipv6/g
s/uvw::IPv4/uvw::ipv4/g
s/uvw::udp_handle::Bind/uvw::udp_handle::udp_flags/
s/uvw::tcp_handle::Bind/uvw::tcp_handle::tcp_flags/

It helped in lot of cases. Remaining cases usually were using binduvw::IPv6(std::string, port) style. Is there possible way to use address in std::string and still be able specify family, whether that should use IPv4 or IPv6? Do I have to use sockaddr if I want it specified manually?

I have made surprisingly lot of conversions by crude sed script

Fair enough. Makes sense.

Is there possible way to use address in std::string and still be able specify family, whether that should use IPv4 or IPv6? Do I have to use sockaddr if I want it specified manually?

Yeah, I don't remember all the internals by heart but we mimic the libuv api 90% of the time, so I'd tell you that if you can't do it there then you can't do it in uvw and vice versa.

I looked into util.cpp, details::ip_addr() in new version. It seems suspicious to me. Does it work correctly for IPV6 addresses? struct sockaddr_in6 has size of 28, but normal struct sockaddr has size just of 16. There is special struct sockaddr_storage, which has size of 128, that should be able to contain any address. Since the function returns just structure of size 16, does it contain full IPv6 address when used?

DNS software I have seen often uses union { sockaddr sa, sockaddr_in in, sockaddr_in6 in6 } to store multiple families in single compact structure. But those use just C language. Is there any C++ standard behaviour, which ensures the current construction will always work? Even for different compilers like MSCV, gcc or clang? I haven't tested it yet, but it seems to me there is no obvious guarantee that should work reliably.

Fair enough. Makes sense.

It seems to me backward compatible namespace uvw { typedef UDPHandle udp_handle; }; might provide backward compatibility to types, which just changed type name. That won't work for bind() calls, which have changed from template to normal function.

Yeah, I don't remember all the internals by heart but we mimic the libuv api 90% of the time, so I'd tell you that if you can't do it there then you can't do it in uvw and vice versa.

It seems to me not in this case. I think the problem is with returning sockaddr by value, not by pointer or reference.  Conversion note from libuv 0.10 mentions example how to do that. Since libuv call change just the pointer, it should be safe. But because uvw returns copy of sockaddr, I think that copy would be incomplete for sockaddr_in6. Change of pointer type is safe if it points to data long enough. It seems to me this is not the case in current implementation, where it returns the reference. But since the function returns just sockaddr, I doubt that will work correctly. Returning sockaddr_storage would be safe, just initializing structure by pointer or reference as well.
It seems it also hides possible format error in address text, it initializes just zeroed structure. Which might not be the same. Which is not done by original libuv.

It would be nice if any header had some defines with major (and optionally minor version). That would allow to support somehow also older version from single code, with help of preprocessor's #if UVW_VERSION_MAJOR >= 3. It does not seem possible now, which makes backward compatibility almost undoable.

The previous define could be used by the project using uvw, in my case flamethrower.
No, I haven't tried cmake compat yet. But to make conditional change to the code, even that cmake would have to define some variable or define, on which conditional preprocessor could choose old or the new format. Wouldn't it be easier if the uvw defined such values itself? CMake might then provide compat definition for previous versions.

Yeah, it would be easier if uvw exported a version file to use in your own software. The cmake solution can be a working one for older versions though.
The problem is that I can add this number to version ie 2.12.0 but I can't change its commit. Therefore, the best that I can do is to create version 2.12.1 with this information. To me, it looks even worse then. You've the version if using 2.x.y only if y isn't 0, otherwise I'm sorry but the version isn't available. It sounds really odd, doesn't it?
So, from my point of view, the cleanest approach is to add a version file upstream and that's is. For older versions, one can use the cmake trick or any other trick because we can't really fix it without breaking something else in the meantime (like, the version commit).
Am I missing something?


Disable weak cipher suites

According to SSL Labs test Chromium 70.0.3538.77 (linux_portable) supports weak cipher suites. Currently I use --cipher-suite-blacklist=0x000a,0x009c,0x009d,0x002f,0x0035 command line option to disable them. Perhaps it would be good to have some kind of flag (UI) for that or at least to suggest the option in the docs as a recommendation for improved security.

what about site that only use those ciphers ? you switch back to http ?

I'm thinking that it may be helpful to have info like this in the ungoogled-chromium-wiki. It'll be nice for users who are informing themselves, and happened to hit ungoogled-chromium early in their learning process.
Maybe we could setup a page in the Wiki for extensions and settings to consider?

what about site that only use those ciphers you switch back to http with this settings ?

Actually I don't know if it works like this (pseudo code):
if (cipher is disabled) { use_plain_HTTP() }

or like this:
if (cipher is disabled) { use_strong_cipher() }
if (strong cypher is unavailable) { drop_connection_with_error_message() }

Can you provide some info about it?
My suggestion is with the assumption of the second (and I may be wrong). Personally I use HTTPS Everywhere with "Block all unencrypted requests" enabled 99.99% of the time. So if some site attempts to switch to HTTP I would be notified quite well.

(using https with those ciphers is better than http...)

Well... maybe, maybe not. As you surely know a false sense of security is worse than being aware of insecurity.

Maybe we could setup a page in the Wiki for extensions and settings to consider?

Yes.

Okay, the site just don't work when blocking ciphers giving back this error (tested with uc v70)
This site can’t provide a secure connection intika.be uses an unsupported protocol.
ERR_SSL_VERSION_OR_CIPHER_MISMATCH

You can reproduce the test with --cipher-suite-blacklist=0xC02F,0xC030,0xC013,0xC014,0x009C,0x009D,0x002F,0x003C,0x0035,0x003D
and navigating to https://intika.be
You have a nice setup but i personally don't like "HTTPS Everywhere" at all because of :

The code quality is very poor (make the extension huge resources eater)
How web extensions works to monitor traffic is in itself a resources eater method, and when the code is not good... (try browsing an hour or two without it you will see a huge difference in speed)
Last time i checked the extensions does not always work
I wrote my own https redirector some times ago (i did not publish it...) and the main js code was about 5kb and it did works exactly as expected link-1 link-2
I did found some glitch about privacy in the code that i did not like back when i reviewed the extension
Having an extension that is sized 1.7 Mo (compressed) that have access to all what i do ?! no thanks... any simple script that does exactly the same thing would never go over 50kb (compressed)

Any way here is what i am using right now :
Firefox :

"security.mixed_content..." settings
"security.insecure_connection" settings
And NoHttp and or Http...

UC :

#enable-potentially-annoying-security-features flag
#disallow-unsafe-http-downloads
#enable-mark-http-as

I think in a near future browsers themselves will start blocking http, it would be nice to have a big warning when surfing http as the extension for firefox do...
Any way would be cool to have a flag for "--cipher-suite-blacklist=0x000a,0x009c,0x009d,0x002f,0x0035"

Thanks for confirming that the setting doesn't result in switching to plain HTTP.
Thanks also for the info about HTTPS Everywhere. I have emailed you as I have some off-topic questions about all this.

Unfortunately your server returned:
<postmaster@ [...] 550 sorry, user over quota [mail153] (#5.1.1) (in reply to RCPT TO command)

my site is not maintained any more and the mail there is not used any more... you can contact me at intikadev at gmail

My site is no longer maintained... you can contact me at intikadev at gmail; but its better to just post here https://github.com/ungoogled-software/ungoogled-chromium-wiki/issues/2 to make the information available to anyone

My site is no longer maintained... my mail is intikadev at gmail; but its better to just post here https://github.com/ungoogled-software/ungoogled-chromium-wiki/issues/2 to make the information available to anyone

My site is no longer maintained... my mail is intikadev at gmail

Sent. Thanks!

Thanks. Just a note you may want to add: the IDs of the cypher suites must be in the exact format 0x0000. My tests show that any other (e.g. 0x00 or 0x000000) has no effect.


Update GitHub scraping example in GUIDE.md

Was trying out the example, seems the span is required to get to the anchor now.
Without:
[2024-01-11 22:32:22.571] [info]  []

With:
[2024-01-11 22:33:35.211] [info]  [{"link":"https://github.com/johnlindquist/kit","title":"kit"},{"link":"https://github.com/johnlindquist/angularjs-plugin","title":"angularjs-plugin"},{"link":"https://github.com/johnlindquist/react-streams","title":"react-streams"},{"link":"https://github.com/johnlindquist/angular-resolve","title":"angular-resolve"},{"link":"https://github.com/johnlindquist/rx-handler","title":"rx-handler"},{"link":"https://github.com/johnlindquist/intro-to-angular2","title":"intro-to-angular2"}]

It's approved. Merge?


fix: noopener typo for rel attribute of <a>

ref https://developer.mozilla.org/en-US/docs/Web/HTML/Link_types/noopener

another good catch, thanks!


Replace query-string with URLSearchParams

Hey @loteoo,
Really excited about this project and would love to contribute. I noticed hyperstatic currently has a dependency on the query-string package, but as far as I know it's functionality could be replaced by URLSearchParams.
Would you accept a PR making this swap? I would be willing to write it if so

Hello @SkipKayhil, that's right, using URLSearchParams would be better! I think I simply didn't think about it very much and added in an extra depencency for nothing.
Feel free to create a PR for the swap!
Thank you for the interest in the project as well. I haven't really added much in the past couple months, but definitively plan to work on hyperstatic more in the future!
Feel free to contribute, ask questions and suggest changes, thanks!


(AWS S3 FS) Fix performance issue of S3 filesystem when reading large files

Fix performance issue of S3 filesystem when reading large files (fixes #25991)
Draining the S3 input stream is a bad idea as we might not read the entire file. Particularly for large files that get split (and [processed in parallel) this quickly explodes the amount of data that needs to be transferred.
As long as the remaining unread data fits into the TCP buffer this will continue draining the stream.
This is beneficial as the HTTP connection can be reused.
Otherwise the stream / connection is aborted, which involves some significant overhead.
This must be done when closing, but also when moving the channel to a new position.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @aromanenko-dev
cc @vatanrathi

There is a integration test for S3 FS. But this issue is really only noticeable with very large files which isn't feasible for testing.

Well, I'd say that this is a good case for exactly IT since they are not running all the time.
Anyway, it's not a blocker for now but I hope it fixes the underneath problem. @vatanrathi would you have a chance to test this fix on your side?

But what do you want to assert on @aromanenko-dev? This would be notoriously unreliable....
You could only record metrics and detect regressions that way.

It was fairly easy to reproduce the issue of @vatanrathi locally and verify this patch.

I will be able to run more extensive tests if a test package can be made available to me.

@mosche

But what do you want to assert on @aromanenko-dev? This would be notoriously unreliable....
You could only record metrics and detect regressions that way.

Well, at least to make sure that it reads a quite large file from S3 with affordable amount of time by running a Beam pipeline  (since it was a cause of initial problem).
@vatanrathi Thanks for testing!


[Req] Ability to open in new tab

Hello,
What would be great is to open i2ssh shells in a new tab.
(I never remember how to switch window ^^)
Really not a high priority request, just a suggestion.
XL.

:+1: +1

+1 Would like to see this implemented. Would result in a cleaner workspace.

👍 would love to see this as well

👍 +1


remove geopandas dependency from pynsee

Rewrite or remove function(s) using geopandas
Not really a bug, but it can be if impossible to install package geopandas on some Insee laptops

geopandas is no longer used as a dependency. the function get_map is no longer available by default.


Use koji.pathinfo.signed() to get signed SRPM path

OSBS-8321

The gist of the change is that the SRPM filename is no longer taken from RPM headers, it is instead constructed from the NVR using koji.pathinfo.signed() (or koji.pathinfo.rpm() if there is no key).
The 2nd commit should be squashed to the 1st one if my assumption (see the commit message) is correct, otherwise dropped.
Maintainers will complete the following section:

[ ] Commit messages are descriptive enough
[ ] "Signed-off-by:" line is present in each commit
[ ] Code coverage from testing does not decrease and new code is covered
[ ] JSON/YAML configuration changes are updated in the relevant schema
[ ] Changes to metadata also update the documentation for the metadata
[ ] Pull request includes link to an osbs-docs PR for user documentation updates
[ ] New feature can be disabled from a configuration file

Yes, for real builds there will be 'name'/'version'/'release' keys as well as 'nvr'.

@twaugh thanks, squashed commits

The listRPMs call will return all the binary RPMs, but we need to map those to their SRPMs. Multiple installed RPMs could come from a single SRPM.
An example: listRPMs might return:
[
    "build_id": 745979,
    "name": "libacl",
    "extra": null,
    "arch": "x86_64",
    "buildtime":<PHONE_NUMBER>,
    "id": 6156897,
    "epoch": null,
    "version": "2.2.53",
    "metadata_only": false,
    "external_repo_id": 0,
    "release": "1",
    "payloadhash": "523a222cdd868826a0cee07f1d169aa2",
    "size": 34536,
    "buildroot_id": 4369557,
    "external_repo_name": "INTERNAL",
    "nvr": "libacl-2.2.53-1"
  },
  ...
]

but the getRPMHeaders call will tell us what the SRPM name is:
{
  "SOURCERPM": "acl-2.2.53-1.el8.src.rpm"
}

So we want to use 'acl', not 'libacl', when constructing the 'src' path.

@twaugh, I see. Is the SOURCERPM header the only way to map an RPM to its SRPM? The pathinfo.signed() method requires a dict with 'name', 'version', 'release', 'arch' keys, this way I would have to parse the NVR back out of the  filename.

I don't know another way. There is {{koji.parse_NVRA}} for parsing the filename.

Would be nice if the tests had a different name for the RPM than the SRPM, it would have picked this up.

Reworked the PR, the change is now overall a lot simpler. Also added the test @twaugh suggested.


Feat/support laravel 6

This PR would allow a version of laravel-mongodb support installs on laravel 6.0.
It could be a different release tag or branch as it seems to have been done before.
Or possibly left until laravel 6.0 is more stable and the majority of user base has adopted and migrated to an updated codebase?
for #1821
(apologies for the first brief pr message..., I pressed enter to return the line.)

@jenssegers, @ReeceM, maybe delete changes  for tests in this PR and create in another PR?

https://github.com/jenssegers/laravel-mongodb/pull/1823 this PR with changes only composer.json

Supported added in 3.6.0


Redundant and Confusing Redirection of "Get Started" Button on About Page

Describe the bug
There is a usability issue with the "Get Started" button on the About page. Currently, when clicked, the button redirects users to the home page, which already contains a "Get Started" button that further redirects users to the sign-in page. This redundancy and confusion in the user flow need to be addressed.

Problem Description:
The current redirection of the "Get Started" button on the About page creates a redundant user experience. Users are directed to the home page, where they encounter another "Get Started" button that leads to the sign-in page. This unnecessary redirection adds confusion and may deter users from efficiently accessing the sign-in page.
Proposed Solution:
To improve the user experience and eliminate redundancy:

Directly redirect users to the sign-in page: The code associated with the "Get Started" button on the About page can be modified to directly redirect users to the sign-in page. This approach eliminates the extra step of navigating to the home page and subsequently clicking the "Get Started" button again.

Renaming the button for clarity: If the "Get Started" button on the About page is intended to serve a different purpose than the one on the home page, consider renaming it to reflect its specific function. For example, it could be renamed to "Learn More" or "Explore Features" to provide a clearer indication of the action users will take when clicking the button.

Screen recording of the problem
https://github.com/MonalikaPatnaik/UMatter/assets/62384655/163e42c1-bbfa-4116-ae19-0938750c1f73

okay then redirect it to the signin page!


Adding Helwan university fci.helwan.edu.eg domain

Adding Helwan university

@abdoolly I am sorry, the domain fci.helwan.edu.eg is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Adding option to set the page title

I added an option that allows for choosing and configuring a page title for pages. (As in )

Q
A

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
no

License
MIT

Thanks a lot!


Postgres failing for integers?

With SQLite:
{:ok, _} = Adbc.Connection.query(conn, "select 123;")
{:ok, %Adbc.Result{num_rows: nil, data: %{"123" => ~c"{"}}}

With Postgres:
{:ok, _} = Adbc.Connection.query(conn, "select 123;")
{:ok, %Adbc.Result{num_rows: nil, data: %{"?column?" => ~c"{"}}}

But strings at least seem fine:
{:ok, %Adbc.Result{num_rows: nil, data: %{"?column?" => ["abc"]}}}

~c"" is a charlist, so you are indeed getting [123] back, but shown as a charlist. :)

@josevalim Maybe there's an issue with Explorer then? Because {:ok, _} = DataFrame.from_query(conn, "select 123;") raises:
** (MatchError) no match of right hand side value: {:error, "Generic Error: cannot cast to dtype: i32"}
    (explorer 0.7.2) lib/explorer/polars_backend/shared.ex:101: Explorer.PolarsBackend.Shared.df_dtypes/1
    (explorer 0.7.2) lib/explorer/polars_backend/shared.ex:87: Explorer.PolarsBackend.Shared.create_dataframe/1
    (explorer 0.7.2) lib/explorer/polars_backend/data_frame.ex:34: Explorer.PolarsBackend.DataFrame.from_query/3
    #cell:qihbhy2uwgl4ljpx:1: (file)
    ```

    Let me check if this is also happening on main.

That's fixed in main because we support i32 and all other types now!

(Explorer main)

Thanks :) I might need to cut a release then.


Add Schema descriptions to the UI

The schema may contain descriptions for each of the element groups and types. If a description exists, it should be shown in the UI.
For example on the Table view, if you hover over the column heading for 'hllp' it could show you the 'hllp' type description (this may involve adding some more descriptions to the schema).
The descriptions for the groups could be added to the Table view group sections and also in the Graph view. They could also be included in the Query builder page when you are configuring a View.
The vertex type description could also be shown in the Add Seed dialog.

Merged into develop


Release/v2022.08.9

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.6 out of 7 committers have signed the CLA.:white_check_mark: nhaghighat:white_check_mark: ricklamers:white_check_mark: ncspost:white_check_mark: iannbing:white_check_mark: astrojuanlu:white_check_mark: yannickperrenet:x: dependabot[bot]You have signed the CLA already but the status is still pending? Let us recheck it.


Add function for operation complete status check to VisaInstrument

This PR proposes a function in VisaInstrument class that checks the Operation Complete Query (OPC) bit and ensures that the operation is complete, i.e., the bit is set to 0. The suggested function repeatedly returns the content of the Event Status Register (ESR) and clears it.
Subsumes and closes #1168.
@jenshnielsen
@astafan8

I wonder what happens if this is used after a short command that has already finished by the time this is executed.  In my understanding that would mean that the opc bit has already been set and is cleared by *ESR? and the instrument will then probably deadlock?
It seems to me that the correct way would be.
self.ask_raw("*ESR?")
self.write("startpotentiallyslowcmd")
while int(self.ask_raw("*ESR?").strip())%2==0:
     time.sleep(sleep_time)

This cannot be done easily since it's across multiple function calls in qcodes but perhaps the call to self.ask_raw("*ESR?") should simply not be here?

@astafan8 This still needs testing on real instruments so i am bumping this to 0.10

It seems like if this should be done it should be done using the pyvisa event handling https://pyvisa.readthedocs.io/en/latest/introduction/event_handling.html

Hi guys! Thanks for finishing this. I am having problem to sign in to my account (Gatebuilder). Let me know if I need to do anything!


Default set of templates for LU, LG, dialog snippets available at Composer GA

In order for people to get started successfully w Composer, by public preview we need to include examples/out-of-box content such as:
Standard Intents - Marieke/Riham/Yochay to talk

Greeting
Quit
Etc.
Predefined domains such as cities, states, Time/date etc.

LG - Ben/Sang/Yochay to talk

Confirmation | VA?
Disambiguation | Marieke to ask Lisa Stifelman
Adaptive card  | hero-card samples?

This has come up also in discussions with PVA. They have the following set, which looks like a good start:
Age
Age of a person, place, or thing, extracted as a number
Prebuilt
Boolean
Positive or negative responses, extracted as a Boolean
Prebuilt
City
City names, extracted as a string
Prebuilt
Color
Primary colors and hues on the color spectrum, extracted as a string
Prebuilt
Continent
Continent names, extracted as a string
Prebuilt
Country or region
Country and region names, extracted as a string
Prebuilt
Date and time
Dates, times, days of the week, and months relative to a point in time, extracted as a string
Prebuilt
Duration
Lengths of time, extracted as a string, in standard TimeSpan format
Prebuilt
Email
Email addresses, extracted as a string
Prebuilt
Event
Event names, extracted as a string
Prebuilt
Language
Language names, extracted as a string
Prebuilt
Money
Monetary amounts, extracted as a number
Prebuilt
Number
Cardinal numbers in numeric or text form, extracted as a number
Prebuilt
Ordinal
Ordinal numbers in numeric or text form, extracted as a number
Prebuilt
Organization
Names of organizations, associations, and corporations, extracted as a string
Prebuilt
Percentage
Percentages in numerical or text form, extracted as a number
Prebuilt
Person name
A person's partial or full name, extracted as a string
Prebuilt
Phone number
Phone numbers in the standard US format, extracted as strings
Prebuilt
Point of interest
Points of interest, extracted as a string
Prebuilt
Speed
Speed, extracted as a number
Prebuilt
State
Names and abbreviations for the United States, extracted as a string
Prebuilt
Street address
Numbered addresses, streets or roads, city, state, zip or postal code in the standard US format, extracted as a string
Prebuilt
Temperature
Temperature, extracted as a number
Prebuilt
URL
Website URLs and links, extracted as a string
Prebuilt
Weight
Weight, extracted as a number
Prebuilt
Zip code

@vishwacsena @cwhitten  - where are we with this?

@mareekuh @cwhitten I'd like to make progress on this in R10 or close it one way or another. Can you clarify if you believe this is an R10 issue for composer?
For prebuilt LU, we can totally have the LUIS pre-built domain intents and entities available as LU files. We would simply need to coordinate with LUIS team to figure out the source for those and have them be pulled in by composer so we do not duplicate the actual models.
For LG, we will need to build content from scratch - we could reach out to Lisa to see what we can leverage from her team's work.
For Dialogs, I'm not sure we have any but we could start building dialogs over prebuilt domain models and make them available which would actually offer an E2E solution than having to start from scratch. Part of what VA skills offer is basically this - they start from LUIS prebuilt domains and then add dialog and LG on top. Another route would be to just offer the skills as prebuilt dialog templates.
Thoughts?

Prebuilt -> Use LUIS domains : Yes
LG -> ask Lisa : Yes. (We can also ask the Marvel team who does the personality chats) - I'll ask and maybe we get both
Dialog snippets: The Samples include have pretty good building blocks. Right now they're full bots, but for

asking questions
interruptions
Controlling conversation flow has a lot of them
..

Composer has gone GA. Closing this issue as obsolete.

Having default content that includes predefined LU, LG and dialog assets remains an important need/requirement.
Please connect this to the 'Asset Library' to not lose track.

This should be part of the core + assistant work (VA). The sample assets should be the same assets that are added on top of the core bot in order to create the componentized, adaptive version of the VA bot.

Closing this, this work is being tracked as part of the component model.


Format Date in Organizer Container

Hi, I want to display format date DD-MM-YYYY in Organizer Container, as for now the format is MM-DD-YYYY.
Can you add this feature when initializing a new organizer object?
Thank you very much :)

Solved, I manually edited script in calendarorganizer.js
from
document.getElementById(this.id + "-date").innerHTML = this.calendar.months[this.calendar.date.getMonth()] + " " + this.calendar.date.getDate() + ", " + this.calendar.date.getFullYear();
to
document.getElementById(this.id + "-date").innerHTML = this.calendar.date.getDate() + " " + this.calendar.months[this.calendar.date.getMonth()] + " " + this.calendar.date.getFullYear();

@ilmaisme I'm sorry for the really really really late reply!
I'm working on a new version for the calendar which will support different Locales.
I'll keep this issue open, but it should be resolved in the next week or two.
I'm sorry again 😔


MUSTS

Menu with at least two buttons (play and quit)
Escape to pause the game
Menu pause (two btns)
Fighting system
Inventaire && status

Moved to a milestone.


CI runs are taking almost 14 minutes each

Since #135 and/or #126 we seem to have increased the running time in CI by several times.

main runs seem to be averaging around 14 min
branch runs vary, and may take less, but probably on repeated runs that exploit the cache.

https://github.com/ekumenlabs/beluga/actions?query=branch%3Amain
This issue is to raise this issue and to trigger checking our minute count for the month.

We may open this up again in the future if we find a good solution for static analysis caching.


Kernel too old

Hello,
Using the pre-compiled binary of version 0.1.2 I get the following error:
FATAL: kernel too old
Aborted (core dumped)

lsb_release -a output:
LSB Version:	:base-4.0-amd64:base-4.0-noarch:core-4.0-amd64:core-4.0-noarch
Distributor ID:	RedHatEnterpriseServer
Description:	Red Hat Enterprise Linux Server release 6.7 (Santiago)
Release:	6.7
Codename:	Santiago

yeah, you'll probably have to build yourself with cargo, or wait for me to get this into bioconda. What does:
grep -m4 model /proc/cpuinfo

show?

Thanks for the quick reply. I will try to build, although I have little experience.
I'm on an HPC. cpuinfo is:
model		: 26
model name	: Intel(R) Xeon(R) CPU           X5570  @ 2.93GHz
model		: 26
model name	: Intel(R) Xeon(R) CPU           X5570  @ 2.93GHz

Managed to build with cargo.  Thanks!


fix: null/undefined check for global tool history checks

:tada: This PR is included in version 3.15.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Annotated graphic for "best practices" section of static text

This graphic goes with the "best practices for making projections" section of text. See this doc for all the text, and where this graphic should go.

closed in #520


Optimize hudmatchstatus

CriticalFlaw and I have been trying to track down why our HUD benchmarking numbers were different between our huds. CriticalFlaw discovered that it may be due to custom hudmatchstatus using color schema calls instead of simple RGBA values.
CriticalFlaw speculates this is due to a combination of using multiple schema references combined by animation refreshes happening for each player's hp status in-game.

From what i can tell, this sadly does not seem to have an impact. It would appear the majority of FPS impact comes from tf_use_match 0 vs 1 (0 being better for FPS by ~16-20 FPS)


💡[Feature] : Adding vechical classification model

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
The Vehicle Classification Model will classify vehicles into various categories such as cars, trucks, motorcycles, buses, etc., using machine learning or deep learning models. This model will leverage computer vision techniques and can be implemented using convolutional neural networks (CNN) or pre-trained models like YOLO (You Only Look Once) for object detection and classification.
Use Case
This model can be used in traffic management systems, automated toll collection, smart city infrastructure, and parking systems where identifying the type of vehicle is necessary.
Specific Use Case

Traffic Monitoring: Classifying vehicles on the road to better manage traffic flows.
Parking Systems: Allowing specific parking zones for different types of vehicles (e.g., trucks, cars, bikes).
Toll Management: Charging toll fees based on the type of vehicle, ensuring fair pricing.
Surveillance: Identifying and tracking different types of vehicles for security purposes.

Benefits

Automation in Traffic Management: Real-time classification can help in optimizing traffic signal timings and reducing congestion.
Scalability: The model can be integrated into various systems like surveillance, toll booths, and smart parking lots, improving overall efficiency.
Cost-effective: Automated classification reduces the need for manual monitoring and management of vehicles.
Security and Law Enforcement: The system can be combined with a number plate recognition system to enhance the ability to track and monitor suspicious vehicles.

Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

its already there


comment link has a wrong direction.

On my site, it redirects to a strange address as (page's address+post's address/#disqus_thread).

Could this be related to  #8 ?


Embrassing

So embrassing!

i just tested your DEX and here what i got :

there's no any problem.
it was smooth af from i got the faucet until i found the vote display.
there's a lot of menu that user can easily use this facility.
hahaha i'm so happy to run this testnet. so wen this will be huge?

@Txtdarisaham Thank you for your testing and we hope to continue delivering high quality features and improvements!


Compress readcache.json as it can get quite big.

Being text heavy json it'll compress really well.
But we have some problems...
If we're compressing it I'd rather change the name of the file to include .gz , but that's a breaking change. I could migrate filenames, but the explicit -r flag is problematic as I should keep the same filename.
How about...

If -r was used and the filename has no .gz then don't compress.
If the default was used and a readcache.json exists then migrate to readcache.json.gz
Default the name to readcache.json.gz when not specified.

?

Can be mitigated by fixing #119


Fix eslint for wps express

Fix https://github.com/Azure/azure-sdk-for-js/issues/17300

@vicancy Looks like the build is failing due to missing formatting changes. Can you run rushx format in the package folder and commit the resulting changes?

Thank you @vicancy!


Improving the C++ nullable implementation

[x] 1. When BOND_NO_CXX11_RVALUE_REFERENCES  is not defined, and arg1 is a forwarding-reference (instead of a const l-value reference) then construction of T should use std::forward<Arg1>(arg1) instead of arg1.

For example:
template<typename Arg1>
pointer new_value(detail::no_allocator&, Arg1&& arg1)
{
    return new T(std::forward<Arg1>(arg1));
}

[x] 2. In destroy() when the C++11 allocator model is available (e.g. BOOST_NO_CXX11_ALLOCATOR is not defined) then do not use alloc.destroy(_value) but instead use std::allocator_traits::destroy because a valid C++11 allocator is not required to have a destroy member.

[x] 3. Either make new_value() use std::allocator_traits::construct (in C++11 - or the allocator's construct in C++03) or make destroy() invoke the destructor instead of calling destroy. i.e. Be consistent: If placement new is used, then manually invoke the destuctor. If allocator's construct is used, then use allocator's destroy.

[x] 4. Currently nullable is confined to custom allocators whose pointer type is a raw pointer type compatible with T*. In the C++11 allocator model, this is not required to be true: i.e. you can have valid custom allocators whose pointer type is a kind of smart pointer. So the pointer member type of nullable has to be inferred from the allocator's pointer member type (via std::allocator_traits::pointer in C++11).

[ ] 5. It would be better for the first specialization of nullable to store an object of type std::aligned_storage_t<sizeof(value_type), alignof(value_type)> (instead of storing an object of type value_type) to have a better make_value() implementation. i.e. To not be reliant on compiler optimizations like RVO to elide any potential copy or moves and construct that value_type in-place with the desired arguments. (Of course boost::aligned_storage and boost::alignment_of can be used instead for compatibility with older compilers).

[x] 6. In both specializations of nullable, leverage the EBO (empty-base optimization) for the allocator. Derive privately from Allocator instead of storing it as a member so that for empty allocators (e.g. std::allocator) you're not increasing the sizeof(nullable<T>) by an extra alignof(value_type) bytes or alignof(pointer) bytes. You can do this derivation conditionally if you want (e.g. if the Allocator type is not final and is an empty type).

Edit: Edited formatting and added two new points (5 and 6).

Approval came through; I should be able to submit a pull request for this sometime next week.

There's nothing coming up that would address any of these concerns, so please go ahead. These look like they're all relatively independent, so can you try to keep the individual commits small. Individual pull requests are fine if you want to go that route as well.

Thanks for #174. I've merged it. I also edited the issue to convert things over to check boxes so we can  see what's been done so far.

The way that number 5 is optimally (and minimally) implemented depends on two things:

Your plans for backward compatibility for older C++ language and Boost library versions. For example, any plans on supporting only C++11 and higher (with only VC15 and higher)?
Whether you still want current semantics where value_type is always constructed even when hasvalue is false instead of constructed only in cases where hasvalue becomes true.

We can't drop Visual C++ 2010 (VC11) or Boost 1.59 support yet.
I'm fine with value_type only being constructed if we have something sensible to put there. This makes even more sense if we're only reserving space for it inside the nullable.
I looked through the history of the project--including the original Microsoft-internal implementation--to see whether I could suss out why nullable works this way. It looks like it works this way because the initial implementation was succinct, and it was never radically changed even as new features (e.g., allocators) were added:
template<typename T, /* ... */>
class nullable
{
public:
    typedef T           value_type;
    ...

    nullable()
        : m_value(),
          m_hasvalue(false)
    {}

    explicit
    nullable(const value_type& value)
        : m_value(value),
          m_hasvalue(true)
    {}

    ...

private:
    value_type  m_value;
    bool        m_hasvalue;
};

At is evolved, it was never changed to avoid constructing value_type.


Leftover: ./examples/web-components

Remove until we get back to web components?

@christian-bromann looks like we forgot to remove this dir?


v0.12.0 Docker image

When will this be released to docker hub?

We don't publish to Docker Hub anymore. You can get the image from quay.io/argoprojlabs/argocd-image-updater


Swiper 3.2.7 issue

Hello,
I have a swiper slideshow:
var slideshowSwiper = new Swiper('#slideshow .screen', {
slidesPerView: 'auto',
spaceBetween: 0,
freeMode: true,
autoplay: 1,
speed: 70333,
autoplayDisableOnInteraction: false,
loop: true,
preloadImages: false,
lazyLoading: true,
loopAdditionalSlides: 922299
});
This slideshow consists of only and only 1 swiper-slide which is a very big image in width and therefore the speed of swiper  70333 to keep the swiper moving horizantally (loop is true).
However, problems occur when I want to pause the swiper.... stopAutoplay and startAutoplay arent helping in my case because the swiper will pause at the end of the swiper-slide motion which is 70333 long and restart after it (cz it is only 1 swiper-slide i am forced to structure it like that.....)
I used the following to instantly pause the swiper:
var old_position = 0;
$('#slideshow .plus').mouseover(function () {   //pause slideshow
old_position = slideshowSwiper.getWrapperTranslate();
$(this).parents('#slideshow').find('.swiper-wrapper').css({ 'transform': 'translate3d(' + old_position + 'px,0px,0px)' });
});
This is fine however on mouseleave i want to directly resume the swiper from the instatntly paused position.... I cant figure out a way to do that on mouseleave....
any help?

Swiper and its autoplay is not intended for such use case, stoping autoplay won't stop transition. You probably need to use some custom JS for your transition/animation. You can try something like Velocity.js


ERTFlix extractor not working, and messed subtitles come as mp4 file

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2<IP_ADDRESS> (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
World
Description
While there is a built-in ERTFlix extractor, it's not working without linking directly to the mpd file. Also, the subs come only as mp4 with messed timings and double or triple lines.
Hopefully, it can be fixed.
Verbose log
Extractor failing:
yt-dlp.exe -vU https://www.ertflix.gr/en/vod/vod.127652-ta-kalytera-mas-chronia-ep1-mia-volta-sto-feggari
[generic] vod: Requesting header
WARNING: [generic] Falling back on generic information extractor.
[generic] vod: Downloading webpage
[generic] vod: Extracting information
ERROR: Unsupported URL: https://www.ertflix.gr/en/vod/vod.127652-ta-kalytera-mas-chronia-ep1-mia-volta-sto-feggari

yt-dlp.exe -vU https://mediaserve.ert.gr/bpk-vod/vodext/default/ta-kalytera-mas-hronia-s01-ep01-subs/ta-kalytera-mas-hronia-s01-ep01-subs/index.mpd
[generic] index: Requesting header
[redirect] Following redirect to https://ert-vod2.siliconweb.com/bpk-vod/vodext/default/ta-kalytera-mas-hronia-s01-ep01-subs/ta-kalytera-mas-hronia-s01-ep01-subs/index.mpd
[generic] index: Requesting header
WARNING: [generic] Falling back on generic information extractor.

Subtitle issue:
yt-dlp.exe -vU https://mediaserve.ert.gr/bpk-vod/vodext/default/ta-kalytera-mas-hronia-s01-ep01-subs/ta-kalytera-mas-hronia-s01-ep01-subs/index.mpd

While there is a built-in ERTFlix extractor, it's not working without linking directly to the mpd file.

Provide verbose log!

Also, the subs come only as mp4 with messed timings and double or triple lines.

See https://github.com/yt-dlp/yt-dlp/pull/247#issuecomment-827628350 https://github.com/yt-dlp/yt-dlp/issues/631#issuecomment-893329480

Extractor failing:
yt-dlp.exe --verbose https://www.ertflix.gr/en/vod/vod.127652-ta-kalytera-mas-chronia-ep1-mia-volta-sto-feggari
Subtitle issue:
yt-dlp.exe --verbose https://mediaserve.ert.gr/bpk-vod/vodext/default/ta-kalytera-mas-hronia-s01-ep01-subs/ta-kalytera-mas-hronia-s01-ep01-subs/index.mpd

What pukkandan trying to say is you must run the command and post the result not just post what command you type

Is this one? Sorry, I am new to this.
WARNING: [generic] Falling back on generic information extractor.
I've edited the main Verbose log, too.