add Kerberos auth support

Have added support for kerberos authentication for go-mssqldb using the GOKRB5 package.
To enable the support for krb the users need to pass the connection string as the following

To use the keytab file

server=server;user id=user;port=1345;realm=domain;trustservercertificate=true;krb5conffile=/etc/krb5.conf;keytabfile=/path/to/keytabFile.keytab

To use kerberos cache file

"server=server;port=1345;realm=domain;trustservercertificate=true;krb5conffile=/etc/krb5.conf;krbcache=/tmp/krb5cc_1000"
krb5conffile        : path to the krb5 configuration file, defaults to "/etc/krb5.conf"
krbcache            : path to the kerberos cache file, required if initkrbwithkeytab = false
keytabfile          : path to the keytab file, required if initkrbwithkeytab = true
realm               : domain name, required when using kerberos cache

Thank you for doing this.
Is there a reason not to use the latest version "v8" with a module import address of "github.com/jcmturner/gokrb5/v8" ?
I have not done a full review yet, and a few things jump out to me, for which I'll send a review for in a bit.
-Daniel

Hi @kardianos
Currently working on migrating the code to v8

Thank you for doing this.
Is there a reason not to use the latest version "v8" with a module import address of "github.com/jcmturner/gokrb5/v8" ?
I have not done a full review yet, and a few things jump out to me, for which I'll send a review for in a bit.
-Daniel

@kardianos  Working on migrating this code to v8. Will push the changes along with other code review comments.

@kardianos could you please review this PR again? have made the changes as per the review comments

Please look at all line item changes, then also investigate why the CI is failing.

As we have gokrb v8 as a dependency which in turn uses go modules for dependency management ,it requires go version 1.13 and above, that's why CI jobs which are using go version less than 1.13 are failing.

Oh, God, this is brilliant! No more Scala/Python just because Kerberos was not supported by this lib. Thank you, guys!

I've tested this branch on a real AD environment, everything works as expected.

@kardianos any updates please?

Please look at all line item changes, then also investigate why the CI is failing.

As we have gokrb v8 as a dependency which in turn uses go modules for dependency management ,it requires go version 1.13 and above, that's why CI jobs which are using go version less than 1.13 are failing. @kardianos

@kardianos, any chance we could get this merged in? It sounds like requirement for v1.13 is the only blocker and according to the Go Developer survey, it's likely that 84%< of teams are already using 1.13 or above.

I've tested this branch on a real AD environment, everything works as expected.

also tested. looks great

Please look at all line item changes, then also investigate why the CI is failing.

As we have gokrb v8 as a dependency which in turn uses go modules for dependency management ,it requires go version 1.13 and above, that's why CI jobs which are using go version less than 1.13 are failing. @kardianos

@kardianos, any chance we could get this merged in? It sounds like requirement for v1.13 is the only blocker and according to the Go Developer survey, it's likely that 84%< of teams are already using 1.13 or above.

Or maybe @denisenkom could help us get this merged?

rb v8 as a dependency which in turn us

@denisenkom - This is a good feature which we are also looking for. Can this be merged asap?

@kardianos any updates please?

@shueybubbles could you please review this PR, as this has been pending for about 2 months now?

@kardianos any updates please?

@shueybubbles could you please review this PR, as this has been pending for about 2 months now?

Please do this!

@chandanjainn I can look but I don't have merge privileges.
FYI I am considering forking this driver over to the Microsoft organization to have a version with some active maintainers and a corporate budget for maintenance. I think it'd be good to the SQL Server team at MSFT to have a Go driver version it backs. Would you and other members of the community welcome such a version? Is there a better forum to present this idea to the community?

You should absolutly do this. I have been waiting few years for this feature. golang-reddit is a good place to have such ideas.

@chandanjainn I can look but I don't have merge privileges. FYI I am considering forking this driver over to the Microsoft organization to have a version with some active maintainers and a corporate budget for maintenance. I think it'd be good to the SQL Server team at MSFT to have a Go driver version it backs. Would you and other members of the community welcome such a version? Is there a better forum to present this idea to the community?

Yes please.

@chandanjainn I can look but I don't have merge privileges. FYI I am considering forking this driver over to the Microsoft organization to have a version with some active maintainers and a corporate budget for maintenance. I think it'd be good to the SQL Server team at MSFT to have a Go driver version it backs. Would you and other members of the community welcome such a version? Is there a better forum to present this idea to the community?

Yes please.

It kinda seems like this repo has become stale / unsupported... @chandanjainn any further thoughts or timeline on forking the repo? I know multiple teams & companies who rely on this so the lack of support is blocking them at the moment. It would be great to get some enterprise-level backing for it.
FYI @kardianos @denisenkom

@chandanjainn I can look but I don't have merge privileges. FYI I am considering forking this driver over to the Microsoft organization to have a version with some active maintainers and a corporate budget for maintenance. I think it'd be good to the SQL Server team at MSFT to have a Go driver version it backs. Would you and other members of the community welcome such a version? Is there a better forum to present this idea to the community?

Yes please.

It kinda seems like this repo has become stale / unsupported... @chandanjainn any further thoughts or timeline on forking the repo? I know multiple teams & companies who rely on this so the lack of support is blocking them at the moment. It would be great to get some enterprise-level backing for it.
FYI @kardianos @denisenkom

@bmanosh-salesforce i believe your comment was directed at @shueybubbles

@chandanjainn I can look but I don't have merge privileges. FYI I am considering forking this driver over to the Microsoft organization to have a version with some active maintainers and a corporate budget for maintenance. I think it'd be good to the SQL Server team at MSFT to have a Go driver version it backs. Would you and other members of the community welcome such a version? Is there a better forum to present this idea to the community?

Yes please.

It kinda seems like this repo has become stale / unsupported... @chandanjainn any further thoughts or timeline on forking the repo? I know multiple teams & companies who rely on this so the lack of support is blocking them at the moment. It would be great to get some enterprise-level backing for it.
FYI @kardianos @denisenkom

@bmanosh-salesforce i believe your comment was directed at @shueybubbles

Oops, yes. Thank you :)

@bmanosh-salesforce I am making the pitch to create this fork this week. There aren't many precedents for such an "adoption" at Microsoft so I don't have a good time estimate. I think if our open source folks agree it's a good idea I'll need to put together some standard stuff like a code of conduct and contribution guidelines. We'll also need to create a set of Github actions or ADO pipelines to replace the appveyor-based test suite for PR validation. We can readily replace some of the SQL VMs with Linux containers for SQL 2017+ but testing against Windows instances of SQL 2012+ will need a fair amount of investment. I'd prefer not supporting Go versions older than 1.13, to reduce the test matrix considerably. We can certainly open up the fork for PRs without having a full test suite in place with the assumption that early adopters are going to rigorously test their scenarios anyway.

Hi.
I am curious. How does Microsoft decide what languages to write drivers for? I can understand for C#, C++,and visual basic but for a language like Go? What is the correct way to get them engaged?

@keith6014 if by "them" you mean "Microsoft", it's my team that manages most in-house driver and client application development for SQL Server.  We have seen steady growth in Go usage with SQL both inside and outside Microsoft, so we started participating in changes for this repo last year. Go looks like a great open source foundation for creating more command line tools for cross platform CI/CD infrastructure so we plan some investment in that area too, starting with github.com/microsoft/go-sqlcmd, which is built using go-mssqldb

@keith6014 if by "them" you mean "Microsoft", it's my team that manages most in-house driver and client application development for SQL Server. We have seen steady growth in Go usage with SQL both inside and outside Microsoft, so we started participating in changes for this repo last year. Go looks like a great open source foundation for creating more command line tools for cross platform CI/CD infrastructure so we plan some investment in that area too, starting with github.com/microsoft/go-sqlcmd, which is built using go-mssqldb

Understood. I hope you get the funding and attention it deserves. 🤞

bump @kardianos @denisenkom could you please merge this?

I opened #729 to solicit input on a Microsoft fork. We very much appreciate your insights!

@dimdin, could you help us get this merged?

the microsoft fork is open now, you can clone the issue and the PR there.
I'd appreciate some more community input on the structure of this PR to merge it there.
github.com/microsoft/go-mssqldb

A modular approach can be applied to load authentication providers (e.g. kerberos)
Each authentication provider should be able to extend the configuration parameters parsing and read the parsed parameter values when getAuth is called.
An authentication provider module import statement is also required to register the module.
Using this approach allows authentication providers to be implemented without modifying go-mssqldb and reduces dependencies imported by these providers when not in use.
@kardianos @denisenkom Do you have any concerns about the modular loading of authentication providers?

the microsoft fork is open now, you can clone the issue and the PR there. I'd appreciate some more community input on the structure of this PR to merge it there. github.com/microsoft/go-mssqldb

@shueybubbles @chandanjainn would we want to move this PR work over to the Microsoft fork?

@bmanosh-sf @keith6014 have moved this PR to the MS repo here https://github.com/microsoft/go-mssqldb/pull/35
Requesting your feedbacks and help with verifying the changes.

Moved to https://github.com/microsoft/go-mssqldb/pull/35


Fix 564 - Crash when stun server is inaccessible

Monika Pull Request (PR)
What feature/issue does this PR add

Closing issue 564
When STUN server is inaccesible, monika crashes

How did you implement / how did you fix it

Add ping package
Before sending request to STUN server,  ping it first.
Bypasses stun checking if connection.alive returns false.

How to test

Run monika normally, ex: monika -c monika.yml
Ensure you get ip and location: monika is running from etc.. etc.
Disable the network and run the same command as in setp 2. Monika should not crash.

It seems a little like monkey patch. let me know if you know of a better solution.
context:
the error seems from here:
node_modules/stun/src/node_modules/net/create-server.js

baaah


Exposed the CURLOPT_UPLOAD_BUFFERSIZE curl option.

See also: https://curl.se/libcurl/c/CURLOPT_UPLOAD_BUFFERSIZE.html

Looks good but I believe there's some CI errors.

CI / Test (system-curl) (pull_request) Failing after 2m — Test (system-curl)
I guess this CI test uses libcurl earlier than 7.62.0...
Perhaps in the test at tests/easy.rs:303 I could check the libcurl version and skip the h.upload_buffer_size(10) call?
Not sure how to go about this :-)

Conditionally testing the version is fine, but it looks like you'll also need to update systest/build.rs with conditionally checking the constant itself.

All checks have passed

Whew :-)


Enhancement/core ehrdata class

Add core ehrdata class, featuring

all fields from anndata
a field .r, that aligns along obs with first axis, var with second axis, and has a 3rd dimension (time)
a field .t, a pandas dataframe for the time
sliceing 1D, 2D, 3D

Relevant enough to be on main, fixes and more things to come
#24


Remove Portal component and replace usage with action

This PR removes usage of the Portal component and replaces it with a more stable and safe action.
Previously, the Portal component would remove/append the portal to the DOM every time it was needed and the creation of one portal did not mean that it would be re-used for another element. Its implementation as a component also obfuscated exactly what was happening in ways that caused it to be used incorrectly (portals within portals), creating multiple undifferentiated instances (see below).

Instead, this PR adds a stable portal element with the ID rill-portal to app.html. The portal action now appends/removes from this single zone by default. It can be placed directly on the element intended to be rendered elsewhere, making it clearer exactly what is happening.
If necessary, additional portal zones can be added to app.html. The portal action  takes an optional parameter of a CSS selector (string) or HTMLElement.

Just noticed one thing that I think is the cause of the failing test. Looking into it.


Payment pending in Phoenix. Don’t see anything in mutiny

tried to send sats to mutiny.  It says it should stay open while you try to receive but need to switch apps to send from Phoenix.
Now transaction is pending in Phoenix.
Anyone knows how long it stays pending before returning to Phoenix ?
lightning payment not on chain. So weird that it’s pending
on latest version of iOS

Yeah it is hard to send from non-custodial to non-custodial on the same device because they both need to sign for the payment to go through.
The payment pending normally means the transcation is just stuck at the LSP, it should eventually fail back to phoenix

Do you know how long that usually takes ?
On Thu, Jul 13, 2023 at 14:23, benthecarman @.***(mailto:On Thu, Jul 13, 2023 at 14:23, benthecarman < wrote:

Yeah it is hard to send from non-custodial to non-custodial on the same device because they both need to sign for the payment to go through.
The payment pending normally means the transcation is just stuck at the LSP, it should eventually fail back to phoenix
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Funds are back thanks for your help ben
On Thu, Jul 13, 2023 at 14:24, fyou @.***(mailto:On Thu, Jul 13, 2023 at 14:24, fyou < wrote:

Do you know how long that usually takes ?
On Thu, Jul 13, 2023 at 14:23, benthecarman @.***(mailto:On Thu, Jul 13, 2023 at 14:23, benthecarman < wrote:

Yeah it is hard to send from non-custodial to non-custodial on the same device because they both need to sign for the payment to go through.
The payment pending normally means the transcation is just stuck at the LSP, it should eventually fail back to phoenix
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Generate Project Documentation file from Documentation objects in the model

Use the new Documentation objects that each FCO contains to generate html/pdf documentation separate from the code documentation that goes into the generated Doxygen documentation.

By generating these into rst files, we can use the documentation from the models on the wiki and the github docs page.


update gax and synth.py

Fixes #40

[x] Tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[x] Appropriate docs were updated (if necessary)

Codecov Report

Merging #43 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master    #43   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           1      1
  Lines           4      4
=====================================
  Hits            4      4

Continue to review full report at Codecov.

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


Dragging an item moves another one (touchscreen only)

Hi, thank you for this component.
I encounter an issue that is easily reproducible, and only in touch mode:
In this example https://mauricius.github.io/vue-draggable-resizable/?path=/story/how-to--drag-multiple-elements, touch somewhere outside of the items to be sure none is active, then touch one of the item to activate it, and then drag the other item (so the one that was not yet active). As you can see the item that was already active moved with the one you dragged (I didn't use the ctrl key, of course).
There is certainly a way to workaround this, but I was wondering if you would easily have a hint on what could cause this issue ;)
Regards

I forgot to mention the following: in order to test I enable the touch device mode in my web browser dev tools

I tried with a real touch screen: the behavior is the same.


Fix Music.Score.Meta

Remove old combinators and make Score.Meta simply reexport all Score.Meta.* modules.
The functionality here should be moved to Music.Time. Eventually, we might change the Score.Meta modules to use other combinators, but the functionality can remain in the time library anyhow.

This repo has been merged into: https://github.com/music-suite/music-suite
Closing :)


Update LICENSE

Rephrased the first sentence for better readability:

Changed "any person obtaining a copy of this software and associated documentation files" to "any individual obtaining a copy of this software and accompanying documentation files."

Simplified the wording in the clause about allowing others to use the software:

Changed "to permit persons to whom the Software is furnished" to "to permit others to whom the Software is provided."

Changed punctuation for clarity:

Added a comma after "AS IS" and also revised the quotation marks around it.

Minor rephrasing for consistency:

-Changed "WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES" to "WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO WARRANTIES" (removing "the" before warranties for smoother flow).

@eltociear @ChenyangSi @xizhedong can u check this


Port projects

The goal is to port all project pages onto the new website with the proper format
Least manual process:

download the list of projects in the onedrive
pick one
use the prompt below on GPT with index.md
create a .yaml file in src/all-yaml/projects with what ChatGPT gives
create a folder in public/imgs/projects as the project name (look at what ChatGPT tries to reference and create a folder with that name)
copy the images found in step 2 into the folder in step 5

Sample GPT prompt:

here is a TEMPLATE yaml:
(PASTE TEMPLATE YAML FILE HERE, found in all-yaml/teams/team1.yaml or all-yaml/projects/project1.yaml)
now i will give you ANOTHER format of markdown. i want you to translate from markdown to yaml, following the template above when appropriate
default team member pfp location is /imgs/team/default.svg
dont write comments in the YAML file
remember to translate any YT video using ###### [![markdown video](http://img.youtube.com/vi/dQw4w9WgXcQ/0.jpg)](https://www.youtube.com/watch?v=dQw4w9WgXcQ "markdown video")
the youtube string is the FULL link to the video, NOT the id. so just the regular youtube.com?watch=id
if a .md has a # team section. AS LONG as you add the team metadata info, you can remove the mention in the actual content
here is the project schema and the team member schema
(PASTE SCHEMAS HERE, found in src/schemas/ProjectMetaData.ts and src/schemas/TeamMetaData.ts)

Will take on all 21-22, 23-24, and 20-21 projects.


[Feature Request] Option to Disable CORS on Filer

--The Issue--
The Filer, by default, will deny all incoming requests from scripts in web applications due to the  CORS policy. Requests sent that do not pass the CORS check(figure 1 on the page referenced) will fail and not display an error code to the client(in this case, the browser). The Filer cannot be used as an API by the browser unless the CORS policy is ignored by the Filer.
--System Setup--
Version:

Filer start command:

Master start command:

Volume start command:

--Current Behavior--
Requests performed in the browser will fail and not display an error code to the javascript. If the page is inspected, the 'network' tab will display the request in red indicating failure. The only way past this is a local Filer or a development proxy server.

--Expected Behavior--
We want HTTP requests performed by JavaScript code in the browser to pass the CORS check and return a valid response. The Filer would be expected to allow cross-origin requests. The option to enable this could be a flag passed to the 'weed filer' command or possibly a variable in the 'filer.toml' or 'security.toml' file.

Can the request have a "Origin" header?
Current code is:
	if r.Header.Get("Origin") != "" {
		w.Header().Set("Access-Control-Allow-Origin", "*")
		w.Header().Set("Access-Control-Allow-Credentials", "true")
	}

I believe POST requests made by the browser will always have an Origin header that contains the request's address. I think PUT and DELETE will as well.

@chrislusf We are still getting this error. Any suggestions?

you can add some debug message around here, and see why it did not work.
https://github.com/seaweedfs/seaweedfs/blob/master/weed/server/filer_server_handlers.go#L48

Thanks. I'm testing a few things with that code.


fix: improvements in tests, build, dependencies, docs and network options

General improvements:

Enabled SO_KEEPALIVE in TCP connections.
Updated dependencies to the latest versions. This also picked up new TLS certs from webpki-roots.
Formatted code cargo fmt.
Fixed linter issues observed by cargo clippy.
Bumped system tests to use the latest QuestDB version 7.3.2
Relegated code generation for tests in https://github.com/questdb/questdb-client-test.git (questdb-rs/src/tests/interop) behind a new (disabled by default) json_tests feature flag.
Fixed a new Windows build linker issue occurring with Visual Studio 2019 and 2022.
In Rust you can now pass chrono::DateTime objects directly to .column_ts and .at methods. For this enable the chrono_timestamp feature.
In C and C++ disabled building tests and examples by default. This works around some issues users have encountered with the EXCLUDE_FROM_ALL CMake setting: https://github.com/questdb/c-questdb-client/pull/39
Upgraded to the newest version of corrosion corrosion CMake library.
Documentation updates.

Closes:

https://github.com/questdb/c-questdb-client/issues/41
https://github.com/questdb/c-questdb-client/issues/40
https://github.com/questdb/c-questdb-client/issues/38

I've simplified the timestamps APIs: They are now explicit in both Rust and C++.


Game reverts to Classic Crafting after dying, can only be deactivated by rejoining world after manually disabling it

Whenever I die, for some reason the classic Crafting option gets turned on indefinitely until I rejoin the world after deactivating it manually once again.
Disabling the option after dying does not change anything, only Rejoining the world after disabling it once again.
This happens after every death.

#321


Same error but with text files

System information
Google Colab
Latest version of python

Describe the problem

UnicodeDecodeError                        Traceback (most recent call last)
 in ()
14   for file in files:
15     with open(os.path.join(root,file)) as p:
---> 16       l.append([p.read() for file in files])
1 frames
/usr/lib/python3.7/codecs.py in decode(self, input, final)
320         # decode input (taking the buffer into account)
321         data = self.buffer + input
--> 322         (result, consumed) = self._buffer_decode(data, self.errors, final)
323         # keep undecoded input until the next call
324         self.buffer = data[consumed:]
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xc9 in position 9: invalid continuation byte

@shashankrustagiiiitdelhi I'm closing this bug because you haven't posted more information. Please comment here if you still need help.


getManufacturerSync()  Android problem

Bug
react-native-device-info = 5.3.0
getManufacturerSync() works perfectly on ios but on android I have next error:

I'm not sure how to fix it fully https://github.com/react-native-community/react-native-device-info/issues/776
If it's all constants-based and default sync as an API, there are startup performance problems.
If it's all async then you have to do gymnastics to use the API during bootstrap in an app
If some are constants, and some are async, there are always some bits of info that people want sync. But then the debugger breaks.


Update logic package

Add in tests, update java docs and
add in comments to improve clarity
closes #265

Hi, whoever last reviews this please help to merge


MONDO:0022575 biliary hypoplasia[Obsolete]

Mondo term (ID and Label)
MONDO:0022575 biliary hypoplasia
Reason for deprecation

Editor note: consider making this a finding

Suggested term to consider
Perhaps HPO needs to make a term for this instead, as a conceptual complement to Biliary _Hyper_plasia (HP:0006560)
Attn @pnrobinson
Your nano-attribution (ORCID)
If you don't have an ORCID, you can sign up for one here

Biliary Hypoplasia is a phenotype, not a disease. Obsolete added to label and definition, along with has obsolesence reason and is obsolete #8340


Use fastcrypto dep for hash digests

I.e., https://github.com/MystenLabs/sui/blob/main/crates/sui-types/src/crypto.rs#L31

I think @jonas-lj 's recent change covered this, reopen if needed


Improved "Quick Open" dialog navigation

I changed the dialog Tree navigation for when the search box is focused, now it will behave like in Godot 1.1, when the multiple selection is enabled and navigating with up, down, page-up and page-down.
Closes #3362.

It works great :)
I did however get a crash, but I think it's related to @reduz's changes and not this PR specifically.
But if you can have a look to confirm before I merge, that would be great.
To reproduce the crash:

start typing some letters in the search box, so that it does not match any item in the tree (the tree is then empty)
try to move up or down with the arroy keys -> segfault

Actually I can't reproduce the segfault without this PR as the arrow keys are not working as per #3362 ;)

@akien-mga sorry my fault i forgot to commit one thing.

It works great, thanks! :D


Timeout is not supported like stated in the docs?

How do you use Sentry?
Self-hosted/on-premise
Version
3
Steps to Reproduce

Have a lot of applications
Log a lot of stuff
Do nothing but let sentry crash randomly (Or shut it off)
Crash all other apps because sentry is not responding anymore.

Expected Result
Not crashing all applications which use the sentry server
Actual Result
Hello,
i have a question about the docs and answers i found here in PRs/Issues.
Its about a timeout the api (should) provide when contacting a sentry server.
https://docs.sentry.io/platforms/php/configuration/options/
The docs define two http_connect_timeout and http_timeout. Which indicate that the client has an option to use a timeout.
But issues like this:
https://github.com/getsentry/sentry-php/pull/967
Tell me that the options isnt used? I couldnt find a line in the code that its really passend by so i am now curios if the SDK still supports a default timeout out of the box, without setting a special HTTPclient. There are multiple issues like that. They describe the problem Sentry has passing around the timeout options to custom clients and so its not implemented anymore (silently???)
So is the timeout option still in use? It is supposed to be documented?

Tell me that the options isnt used? I couldnt find a line in the code that its really passend by

Support for configuring one of the out-of-the-box supported HTTP clients using the http_connect_timeout and http_timeout options have been implemented in #1282. Is this what you are looking for?

Yes thank you. I will look for deployed code on my side but your hint to the PR does REALY help to tell my boss there is a timeout. I will have to confirm that this is deployed to the current packages since i am not saw any "timeout" in the current code. But really thank you showing me the lines.


disabled public network access for KV and SQL

This pull request to deployment/Deploy.ps1 includes changes to enhance the deployment script by updating firewall rules and adding private endpoints for SQL and Key Vault. The most important changes include removing unnecessary firewall rules, updating public network settings, and adding commands to create private endpoints.
Firewall and network updates:

Removed the command to add a network rule to the Key Vault, simplifying the firewall configuration. (deployment/Deploy.ps1)
Updated the firewall update section to exclude SQL, focusing only on WebApps. (deployment/Deploy.ps1)
Disabled public network access for both SQL Server and Key Vault to prepare for the addition of private endpoints. (deployment/Deploy.ps1)

Private endpoint creation:

Added commands to create private endpoints for SQL Server, enhancing security by using private connections. (deployment/Deploy.ps1)
Added commands to create private endpoints for Key Vault, ensuring secure access through private connections. (deployment/Deploy.ps1)

@santhoshb-msft WIP

The current deployment process first creates resources with public access enabled and then disables public access at the end. This approach will be blocked in environments with Azure policies that prevent the creation of resources with public access enabled. To ensure compliance, resources should be deployed with public access disabled from the onset.
pr-774.pdf

@v-adeadepoju thank you for the inputs. I understand your point but the this PR doesnt address this issue. The PR is specifically to disable the earlier firewall setting.

Yes, il update the installation instruction to include dotnet install 8.0.0 but il do that after our SA 8 release is out, untill then please manually add 8
What im looking for here in this PR is

[ ] - Install complete without network for SQL KV
[ ] - Application working E2E along with the Meter scheduler


reddit and twitter fail to verify

Reddit link: https://www.reddit.com/r/KeybaseProofs/comments/1d6y2te/my_keybase_proof_redditjjorstad2_keybasejjorstad2/
Twitter : https://twitter.com/jjorstad2/status/1797518166563885182

Have the same issue with reddit (twitter is ok) since last week.

reddit proof suddenly reported as working again.

yes reddit now works. But twitter still fails to work.


Could not load file or assembly 'System.Runtime, Version=<IP_ADDRESS> at startup

If I add cloudscribe.Web.Navigation nuget package to my project which using net451 framework, I get this error on startup:
System.IO.FileNotFoundException
Could not load file or assembly 'System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified.
In my project.json is:
"dependencies": {
...
"cloudscribe.Web.Navigation": "1.0.1-beta4"
},
"frameworks": {
"net451": {
"imports": [ "netstandard1.5" ]
}
}
It is OK on netcoreapp1.0 framework, but my project must targed Full .NET Framework 4.5.1.

I just published version 1.0.1-beta5 on nuget.org, I added the target for net451 and I also changed the other target to netstandard1.3
can you try it and confirm it solves the problem for you

On beta5 it's working fine.
Thank you

Thanks for the bug report and confirming the fix!


Azure AI.Search is giving SSL: CERTIFICATE_VERIFY_FAILED certificate verify failed:  unable to get local issuer certificate (_ssl.c:997) Error

I am trying to use the examples provided on corporate provided laptop and I think the error is happening because the certificate is not downloadable in the laptop. Is there any workaround or alternate available?
The issue happens at both office network and in personal network. However it doesnt happen in personal laptop (without doing any additional steps). So I request anyway to bypass the SSL request.
Environment used - Visual Studio Code/ Windows 10 Pro/ venv
The error I am getting is:
`  File "c:\00WorkFiles\AzureOpenSearch.venv\lib\site-packages\streamlit\runtime\scriptrunner\script_runner.py", line 600, in _run_script
exec(code, module.dict)
File "C:\00WorkFiles\AzureOpenSearch\azure-try3.py", line 195, in
main()
File "C:\00WorkFiles\AzureOpenSearch\azure-try3.py", line 189, in main
response, citation = perform_search(query)
File "C:\00WorkFiles\AzureOpenSearch\azure-try3.py", line 152, in perform_search
for result in results:
File "c:\00WorkFiles\AzureOpenSearch.venv\lib\site-packages\azure\search\documents_paging.py", line 54, in next
return next(self._page_iterator)
....
error
azure.core.exceptions.ServiceRequestError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed:
unable to get local issuer certificate (_ssl.c:997)
`
The code in question is attached..
`import os
import streamlit as st
from azure.storage.blob import BlobServiceClient
from azure.search.documents.indexes import SearchIndexerClient
from azure.search.documents.indexes.models import (
SearchIndexerDataContainer,
SearchIndexerDataSourceConnection,
SplitSkill,
InputFieldMappingEntry,
OutputFieldMappingEntry,
AzureOpenAIEmbeddingSkill,
SearchIndexerIndexProjections,
SearchIndexerIndexProjectionSelector,
SearchIndexerIndexProjectionsParameters,
IndexProjectionMode,
SearchIndexerSkillset,
SearchIndexer,
FieldMapping
)
from azure.search.documents import SearchClient
from azure.search.documents.models import VectorizableTextQuery, QueryType, QueryCaptionType, QueryAnswerType
from openai import AzureOpenAI
from azure.core.credentials import AzureKeyCredential
from azure.core.exceptions import ResourceExistsError
import ssl
ssl._create_default_https_context = ssl._create_unverified_context
Azure configuration
endpoint = os.environ["AZURE_SEARCH_SERVICE_ENDPOINT"]
credential = os.environ["AZURE_SEARCH_ADMIN_KEY"]
index_name = os.environ["AZURE_SEARCH_INDEX"]
blob_connection_string = os.environ["BLOB_CONNECTION_STRING"]
blob_container_name = os.environ["BLOB_CONTAINER_NAME"]
azure_openai_endpoint = os.environ["AZURE_OPENAI_ENDPOINT"]
azure_openai_key = os.environ["AZURE_OPENAI_KEY"]
azure_openai_embedding_deployment = os.environ["AZURE_OPENAI_EMBEDDING_DEPLOYMENT"]
azure_openai_chatgpt_deployment = os.environ["AZURE_OPENAI_CHATGPT_DEPLOYMENT"]
Set up Azure OpenAI client
openai_client = AzureOpenAI(
api_key=azure_openai_key,
api_version="2023-03-15-preview",
azure_endpoint=azure_openai_endpoint
)
def upload_to_azure(file):
# Connect to Blob Storage
blob_service_client = BlobServiceClient.from_connection_string(blob_connection_string)
container_client = blob_service_client.get_container_client(blob_container_name)
if not container_client.exists():
container_client.create_container()
# Check if the blob already exists
blob_client = container_client.get_blob_client(file.name)
if not blob_client.exists():
    # Upload the PDF to Blob Storage
    blob_client.upload_blob(file)
else:
    st.write(f"The document '{file.name}' already exists in the blob container.")
    return

# Create a data source
indexer_client = SearchIndexerClient(endpoint, AzureKeyCredential(credential))
container = SearchIndexerDataContainer(name=blob_container_name)
data_source_connection = SearchIndexerDataSourceConnection(
    name=f"{index_name}-blob",
    type="azureblob",
    connection_string=blob_connection_string,
    container=container
)
data_source = indexer_client.create_or_update_data_source_connection(data_source_connection)

# Create a skillset
skillset_name = f"{index_name}-skillset"
split_skill = SplitSkill(
    description="Split skill to chunk documents",
    text_split_mode="pages",
    context="/document",
    maximum_page_length=2000,
    page_overlap_length=500,
    inputs=[InputFieldMappingEntry(name="text", source="/document/content")],
    outputs=[OutputFieldMappingEntry(name="textItems", target_name="pages")]
)
embedding_skill = AzureOpenAIEmbeddingSkill(
    description="Skill to generate embeddings via Azure OpenAI",
    context="/document/pages/*",
    resource_uri=azure_openai_endpoint,
    deployment_id=azure_openai_embedding_deployment,
    api_key=azure_openai_key,
    inputs=[InputFieldMappingEntry(name="text", source="/document/pages/*")],
    outputs=[OutputFieldMappingEntry(name="embedding", target_name="vector")]
)
index_projections = SearchIndexerIndexProjections(
    selectors=[
        SearchIndexerIndexProjectionSelector(
            target_index_name=index_name,
            parent_key_field_name="parent_id",
            source_context="/document/pages/*",
            mappings=[
                InputFieldMappingEntry(name="chunk", source="/document/pages/*"),
                InputFieldMappingEntry(name="vector", source="/document/pages/*/vector"),
                InputFieldMappingEntry(name="title", source="/document/metadata_storage_name")
            ]
        )
    ],
    parameters=SearchIndexerIndexProjectionsParameters(
        projection_mode=IndexProjectionMode.SKIP_INDEXING_PARENT_DOCUMENTS
    )
)
skillset = SearchIndexerSkillset(
    name=skillset_name,
    description="Skillset to chunk documents and generating embeddings",
    skills=[split_skill, embedding_skill],
    index_projections=index_projections
)
client = SearchIndexerClient(endpoint, AzureKeyCredential(credential))
client.create_or_update_skillset(skillset)

# Create an indexer
indexer_name = f"{index_name}-indexer"
indexer = SearchIndexer(
    name=indexer_name,
    description="Indexer to index documents and generate embeddings",
    skillset_name=skillset_name,
    target_index_name=index_name,
    data_source_name=data_source.name,
    field_mappings=[FieldMapping(source_field_name="metadata_storage_name", target_field_name="title")]
)
indexer_client = SearchIndexerClient(endpoint, AzureKeyCredential(credential))
indexer_client.create_or_update_indexer(indexer)
indexer_client.run_indexer(indexer_name)

def perform_search(query):
search_client = SearchClient(endpoint, index_name, AzureKeyCredential(credential))
vector_query = VectorizableTextQuery(text=query, k_nearest_neighbors=1, fields="vector", exhaustive=True)
results = search_client.search(
    search_text=query,
    vector_queries=[vector_query],
    select=["parent_id", "chunk_id", "chunk", "title"],
    query_type=QueryType.SEMANTIC,
    semantic_configuration_name='my-semantic-config',
    query_caption=QueryCaptionType.EXTRACTIVE,
    query_answer=QueryAnswerType.EXTRACTIVE,
    top=1
)

# Extract the relevant chunk and title from the search results
relevant_chunk = None
title = None
for result in results:
    relevant_chunk = result["chunk"]
    title = result["title"]
    break

if relevant_chunk:
    # Use Azure OpenAI chat completion for Q&A
    response = openai_client.chat.completions.create(
        model=azure_openai_chatgpt_deployment,
        messages=[
            {"role": "system", "content": f"You are an AI assistant that helps answer questions based on the given context. Here is the relevant context:\n\n{relevant_chunk}"},
            {"role": "user", "content": query}
        ]
    )

    # Return the assistant's response and the citation
    return response.choices[0].message.content, f"Reference: {title}"
else:
    return "Related documents not found.", None

def main():
st.title("RAG Chatbot")
uploaded_file = st.file_uploader("Upload a PDF", type="pdf")

if uploaded_file is not None:
    try:
        upload_to_azure(uploaded_file)
        st.write("PDF uploaded and vectorized successfully!")
    except ResourceExistsError:
        st.write(f"The document '{uploaded_file.name}' already exists in the blob container.")
    except Exception as e:
        st.write(f"An error occurred while uploading and vectorizing the document: {str(e)}")

query = st.text_input("Ask a question about the documents")

if query:
    response, citation = perform_search(query)
    st.write("Assistant:", response)
    if citation:
        st.write("Citation:", citation)

if name == "main":
main()`

Can you please try following the steps here? Azure/azure-sdk-for-python#20883
If this does work - please open an issue on the azure-sdk-for-python repository.

Thanks Matt. This does fix the problem.  I dont understand the second part as to why I should open issue in the azure sdk repo. Can you please give more details.

Can you please try following the steps here? Azure/azure-sdk-for-python#20883
If this does work - please open an issue on the azure-sdk-for-python repository.

Thanks Matt. This does fix the problem. I dont understand the second part as to why I should open issue in the azure sdk repo. Can you please give more details.

sorry typo - "doesn't". Fixed


feature / view list of qr codes added to account

view list of qr codes added to account
select qr code from list and edit name

will add picture preview and might put edit window on a separate page later


Update blog-posts.md

 Issue by tesar-tech
Thursday Jan 31, 2019 at 07:48 GMT
Originally opened as https://github.com/Wyamio/Wyam.Web/pull/80

Adds post about netlify and azure devops

tesar-tech included the following code: https://github.com/Wyamio/Wyam.Web/pull/80/commits

 Comment by daveaglick
Thursday Jan 31, 2019 at 12:57 GMT

Awesome, thanks!


Fleet - Cannot read property 'linkFor' of undefined when creating a GitRepo

Rancher version: master-head (09/23/2020) e9a98a3
RKE DO cluster on HA k8s Rancher

The bug fix is validated in master-head 6319706
the steps are the same, and the error does not show up anymore.


how to run two multiple scripts using tini

i want to run script entrypoint.sh with tini and also run my backup script to just schedule my job ?How can  i do that ?
NTRYPOINT  [ "/opt/opendj/docker-entrypoint.sh" ]
CMD ["start"]
CMD ["chmod 755 /opt/opendj/scripts/schedule-backup.sh" && "exec tini -g -v  --  /opt/opendj/scripts/schedule-backup.sh"]

All Tini does is spawn a single child (Tini is meant to be run in a container), and wait for it to exit all the while reaping zombies and performing signal forwarding.

So, tini will not follow or be involved in the process lifetime of anything started outside of its child (besides reaping the child's zombies).
I'd just use the host cron to run a new container (or does docker exec) to run a periodic backup script (or a "CronJob" if using Kubernetes).

@yosifkit  is spot-on here. Thanks!

I was inspired by tini but I also needed support for multiple programs to run inside a Docker container (like cron jobs because they are often bundled together with the main program), so I made another init system for Docker, dinit, which supports multiple processes and so you can have a cron daemon run in parallel with your main process.


Featrue: 单据详情蓝盾节点可预览

1.蓝盾重做流程已跑通

Codecov Report

Merging #219 (e85cc52) into v2.6.x_develop (becdd48) will increase coverage by 0.30%.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##           v2.6.x_develop     #219      +/-   ##
==================================================
+ Coverage           51.46%   51.76%   +0.30%
==================================================
  Files                 445      445
  Lines               27409    27416       +7
==================================================
+ Hits                14105    14191      +86
+ Misses              13304    13225      -79

Impacted Files
Coverage Δ

itsm/ticket/views/ticket_remark.py
36.11% <0.00%> (-8.72%)
:arrow_down:

itsm/component/drf/permissions.py
57.24% <0.00%> (+1.44%)
:arrow_up:

itsm/iadmin/serializers.py
93.02% <0.00%> (+4.65%)
:arrow_up:

itsm/iadmin/views.py
52.42% <0.00%> (+4.85%)
:arrow_up:

itsm/iadmin/validators.py
40.27% <0.00%> (+6.94%)
:arrow_up:

itsm/auth_iam/utils.py
33.89% <0.00%> (+9.32%)
:arrow_up:

itsm/iadmin/permissions.py
54.38% <0.00%> (+10.52%)
:arrow_up:

itsm/component/utils/basic.py
54.13% <0.00%> (+15.66%)
:arrow_up:

Continue to review full report at Codecov.

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


Create file and nested directories at once

It would be great, if you could create number of nested directories specifying their names separated with slashes in the dialog that opens while creating new file.
For example if you create file entering following string into dialog: /app/some/module/api.ts
It should create folder structure like this:
app/
  some/
    module/
      api.ts

If some folder  already exists, it should just create subfolder module into it and put api.ts file into it.

As usual, there is an extension that does this :)
File Utils has a 2 commands for creating files, if you specify directories in front of the name it creates them recursively for you (and skips the existing ones as well)

I always prefer native functionalities for this kind of features.
Never thought installing external plugin for creating files in any IDE, But thank you for that reference.
If this "improvement" (not actually issue) isn't relevant, someone will close it.

Related: https://github.com/Microsoft/vscode/issues/12893

Thank you @wallverb . I was looking for similar issues, but missed that one probably.


feat(eslint-plugin): prefer-object-syntax support fetchQuery

This pull request updates the prefer-object-syntax eslint rule to add support for the fetchQuery method
I tried to follow the Contributing guide, please let me know if I need to change something.
Related discussion => https://github.com/TanStack/query/discussions/5300
Thank you for the awesome library ❤

Codecov Report
Patch and project coverage have no change.

Comparison is base (058f030) 97.12% compared to head (99f31fa) 97.12%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5301   +/-   ##
=======================================
  Coverage   97.12%   97.12%
=======================================
  Files          11       11
  Lines         383      383
  Branches      140      140
=======================================
  Hits          372      372
  Misses         11       11

Impacted Files
Coverage Δ

...-query-object-syntax/prefer-query-object-syntax.ts
96.19% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


"bash.exe -c " does not exit correctly when launched using CreateProcess() with pipe redirection

Background information:

Windows build number: Microsoft Windows [Version 10.0.16299.492]

What you're doing and what's happening:  When launching a bash.exe process using the Windows CreateProcess function, the exit code of the process is 259 (corresponding to STILL_ACTIVE) even though the output of the bash.exe command has already been read.

For example, a process created to run bash.exe -c "date" only exits after an additional character is written to the stdin pipe of the process. I have attached a minimal C++ code in Main.cpp which reproduces this issue using the Windows APIs: CreateProcess_WSL_repro.zip
This is surprising since most other non-interactive command-line utilities (like net.exe) on Windows will provide exit code 0 after output is generated.

What's wrong / what should be happening instead: The bash.exe -c "date" process is currently providing exit code 259 when used with CreateProcess pipe redirection, but it should instead provide exit code 0 after executing the command.

Strace of the failing command, if applicable:  not applicable; this issue seems to be occurring outside of the Linux userspace (in WSL/CreateProcess pipe interaction)

For WSL launch issues, please collect detailed logs: This doesn't seem to be a WSL launch issue either, but I can try and get these logs if it would help.

Version 10.0.16299.492

If you read the release notes there have been related issues and most probably has been fixed already on April 2018 Update.

@onomatopellan thanks for letting me know about the WSL release notes. I tried the same workflow on a version 10.0.17134.165 system and it worked correctly, so it does seem like this issue is fixed in newer builds.


[BUG] PDF don't download, request fails with status code 504

PDF don't download, request fails with status code 504
I am logged in with my google account. I am trying to download a PDF file of the resume and when I click the download button it fails because of timeout.
Product Flavor

[x] Managed (https://rxresu.me)
[ ] Self Hosted

To Reproduce

Go to a built resume
Click on the download button

See the error

Expected behaviour
I expected to download a PDF file without timeout errors.
Screenshots

Desktop:

OS: Windows 10
Browser: chrome
Version: Versión 101.0.4951.54 (Official build) (64 bits)

@alanshalem It could have been some unexpected downtime on the server. I haven't actually been tracking the health metrics of it. Are you still facing the same issue?

Just wanted to drop a note and mention that I have also been running into this exact issue the last ~24 hours. Most of the time a PDF export fails with 504 errors, but every once in a while it succeeds after going back and trying again every few hours.
Fantastic project by the way. Thank you so much for your hard work on this! It's both a beautiful and impactful site helping a lot of people.

This looks like it's similar to this issue: https://github.com/AmruthPillai/Reactive-Resume/issues/1048 where the server crashes on upload of a non-ZIP file. This unfortunately has repercussions to other users as there's only one server running.
I am looking into some solutions to this, but the issue is still open to external help as I'm unclear how to catch and pacify these errors without letting the server crash.


Configure Jackson to only scan fields and setters of @BeanParam classes

Thank you so much for all the amazing work on swagger-core!
When using swagger-core on a recent project, I came across an NPE where ParameterUtils.isMethodArgumentAnArray was receiving a null paramClass argument. It turns out that this was because a BeanParam class declared a method with a name like getX, where X did not directly correspond to a field name. Because the method name began with "get," Jackson was returning a BeanPropertyDefinition corresponding to it whose field and setter were null.
After looking at the JAX-RS documentation for BeanParam, I wasn't able to think of a case where introspecting on the getter methods of a BeanParam would be useful, so I configured the ObjectMapper to only consider fields and setters. These seem to be the only places where JAX-RS annotations are used on BeanParam classes.
To demonstrate the issue, I modified one of the BeanParamClasses to include a method beginning with "get" that does not directly correspond to a field.

Sorry, need to rethink the approach here. I didn't notice that the mapper is a singleton, so limiting the visibility of that mapper affects other things that use the singleton. I considered creating the mapper with Json.create(), but this also affects the behavior of Json.mapper because it assigns the static mapper in addition to returning a new instance. Back to the drawing board.

@griswold - keep in mind there are two ObjectMappers - one in Json and one in Yaml and they both need to be taken into account.


#4252: Build tt-metal with clang-17 instead of gcc

Needs https://github.com/tenstorrent-metal/tt-metal/pull/6835 merged first.
PR enables building tt-metal with Clang-17

Are we merging this first, or?
We don't need to make any code changes to tt-metal to make this work?

We probably should update the deps stuff, like in .github/actions

@tt-rkim
PR needs https://github.com/tenstorrent-metal/tt-metal/pull/6835 merged first.
Deps are already installed in all CI VMs, and the provisioning already includes clang-17 installation.
soon to be Dev VMs as well.
Just pre-emptively getting the PRs ready. pending https://github.com/tenstorrent-metal/tt-metal/pull/6835

Can you try running the eager package main then, to confirm that compilation and stuff works?

C++ build binaries job would be good here too

Once other one merged in

2024-03-27T19:56:39.1589961Z make[1]: Leaving directory '/tmp/build-via-sdist-wegedgus/metal_libs-0.46.0rc3.dev26+wormhole.b0/tt_metal/hw'
2024-03-27T19:57:52.3684082Z In file included from tt_metal/impl/dispatch/command_queue.cpp:5:
2024-03-27T19:57:52.3685297Z In file included from ./tt_metal/impl/dispatch/command_queue.hpp:15:
2024-03-27T19:57:52.3692205Z ./tt_metal/impl/dispatch/lock_free_queue.hpp:55:38: error: 'iterator<std::forward_iterator_tag, tt::tt_metal::CommandInterface>' is deprecated [-Werror,-Wdeprecated-declarations]
2024-03-27T19:57:52.3694908Z    55 |         class Iterator : public std::iterator<std::forward_iterator_tag, T> {
2024-03-27T19:57:52.3695799Z       |                                      ^
2024-03-27T19:57:52.3698035Z tt_metal/impl/dispatch/command_queue.cpp:1597:26: note: in instantiation of member class 'LockFreeQueue<tt::tt_metal::CommandInterface>::Iterator' requested here
2024-03-27T19:57:52.3699651Z  1597 |     for (const auto& cmd : tq.worker_queue) {
2024-03-27T19:57:52.3700324Z       |                          ^
2024-03-27T19:57:52.3705874Z /usr/bin/../lib/gcc/x86_64-linux-gnu/13/../../../../include/c++/13/bits/stl_iterator_base_types.h:127:12: note: 'iterator<std::forward_iterator_tag, tt::tt_metal::CommandInterface>' has been explicitly marked deprecated here
2024-03-27T19:57:52.3712160Z   127 |     struct _GLIBCXX17_DEPRECATED iterator
2024-03-27T19:57:52.3712831Z       |            ^
2024-03-27T19:57:52.3715805Z /usr/bin/../lib/gcc/x86_64-linux-gnu/13/../../../../include/x86_64-linux-gnu/c++/13/bits/c++config.h:121:34: note: expanded from macro '_GLIBCXX17_DEPRECATED'
2024-03-27T19:57:52.3717397Z   121 | # define _GLIBCXX17_DEPRECATED [[__deprecated__]]
2024-03-27T19:57:52.3718116Z       |                                  ^
2024-03-27T19:57:55.3832002Z 1 error generated.
2024-03-27T19:57:55.4185133Z make: *** [/tmp/build-via-sdist-wegedgus/metal_libs-0.46.0rc3.dev26+wormhole.b0/tt_metal/impl/module.mk:35: /tmp/build-via-sdist-wegedgus/metal_libs-0.46.0rc3.dev26+wormhole.b0/build/obj/tt_metal/impl/dispatch/command_queue.o] Error 1
2024-03-27T19:57:55.4188200Z make: *** Waiting for unfinished jobs....

Iterator is deprecated? Wonder if it's the CONFIG=release here...

Can you try running the eager package main then, to confirm that compilation and stuff works?
C++ build binaries job would be good here too

2024-03-28T22:11:10.9458149Z ./tt_metal/impl/dispatch/lock_free_queue.hpp:55:38: error: 'iterator<std::forward_iterator_tag, tt::tt_metal::CommandInterface>' is deprecated [-Werror,-Wdeprecated-declarations\
it's just because we have -Werror which treats deprecations as errors. the use of std::iterator is deprecated in C++17 and entirely removed in C++20

Closing pull request as addressed in
https://github.com/tenstorrent/tt-metal/pull/9070


Simulations fail sporadically

Logs from CI:
[W0] Spawned simulation with pid 98731 [seed=18931893 stdout=/tmp/sim-logs-1715377034/app-simulation-seed-18931893-date-06-06-2023_160434.stdout stderr=/tmp/sim-logs-1715377034/app-simulation-seed-18931893-date-06-06-2023_160434.stderr]
exit status 1
[W0] Seed 18931893: FAILED
To reproduce run: go test ./app -run TestAppStateDeterminism -Enabled=true -NumBlocks=1 -Genesis= -Verbose=true -Commit=true -Seed=18931893 -Period=1 -ExportParamsPath /tmp/sim-logs-1715377034/sim_params-18931893.json -ExportStatePath /tmp/sim-logs-1715377034/sim_state-18931893.json -v -timeout 24h
ERROR OUTPUT

exit status 1
panic: halting simulations

goroutine 8 [running]:
main.worker(0x0?, 0x0?, 0x0?)
	/home/circleci/go/pkg/mod/github.com/cosmos/tools/cmd/runsim@v1.0.0/main.go:216 +0x629
main.main.func2(0x0?)
	/home/circleci/go/pkg/mod/github.com/cosmos/tools/cmd/runsim@v1.0.0/main.go:160 +0x5c
created by main.main
	/home/circleci/go/pkg/mod/github.com/cosmos/tools/cmd/runsim@v1.0.0/main.go:158 +0xdd0
make: *** [Makefile:153: test-sim-deterministic] Error 2

Exited with code exit status 2

See: https://app.circleci.com/pipelines/github/CosmWasm/wasmd/3651/workflows/6dcb95be-b290-4d90-ae48-dc4df56c5e6d/jobs/15444

See also: https://app.circleci.com/pipelines/github/CosmWasm/wasmd/3653/workflows/9e9153bd-5342-4aa6-aa5b-9725b6ef5e8d/jobs/15456

There is a bug in sdk simulations. See https://github.com/cosmos/cosmos-sdk/issues/16525


[Python] Doesn't handle well poetry dependencies depending on Python version

Package manager/ecosystem
Python Poetry.
https://github.com/alexpdp7/caw/blob/master/pyproject.toml#L14
My project supports Python 3.6, 3.7 and 3.8, hence:
python = ">=3.6, <3.8"

The project uses dataclasses, so on 3.6 I install a backport:
dataclasses = { version = "0.7", python = ">=3.6,<3.7" }

I think this worked OK, but recently Dependabot started spamming me with issues:
https://github.com/alexpdp7/caw/issues/41
, which seem to complain about this issue.

Dependabot has also recently started failing with
[SolverProblemError]
The current project's Python requirement (^3.6) is not compatible with some of the required packages Python requirement:
  - dataclasses requires Python >=3.6, <3.7

on one of my repos too ( https://github.com/chris48s/mark-sideways ), although it is caused by effectively the same condition in one of my transitive dependencies: https://github.com/willmcgugan/rich/blob/a424fafc5654346b5d6e2db28ad19aad10040581/pyproject.toml#L27
I guess this is a manifestation of https://github.com/python-poetry/poetry/issues/2386 as opposed to a bug in dependabot itself.
Strangely, I'm unable to reproduce this locally with python 3.8/poetry 1.0.10 (although the last post in that linked issue says this is still an issue in that build, apparently)
What version of poetry does dependabot use? (I don't think the error log on my repo tells me)

We're currently on Poetry 1.0.10, and I'm in the process of upgrading to to 1.1.1 @chris48s

Hopefully that upgrade will resolve it :crossed_fingers:

I've now had a successful dependabot run on that repo

I've now had a successful dependabot run on that repo

Happy to hear it! Let's keep this open for a bit longer to make sure it's really resolved, please let me know if you spot anything weird

It seems to be also working for me.

It seems to be also working for me.

Awesome! Alright I'm going to close this out for now, feel free to re-open if needed.


Support Bower's main-files feature

Bower packages can provide a list of files that are important to
projects using this package. Some build tools use this list to copy,
concatenate, or munge the contents of the files in some way without
touching the files in bower_components.
This commit adds the Polymer files to the list of files that should be
delivered to a build tool.

:+1:
I was about to create a PR exactly like this myself.  Please accept! :)

+1   This should also happen in any other polymer components.

This makes sense to me. We recently made a major change to the master branch of this repo with the 0.9 release - would you mind re-submitting a PR here rebasing from the latest master? Thanks!


Fix parent directory creation in distcp-ng

@ibuenros can you review?
We rely on fs.mkdirs to create the parent directories of a CopyableFile if not exists. If the directories exist (created by other mappers), fs.mkdirs will return a false which we ignore.

LGTM


CEF key name "deviceExternalID" should be "deviceExternalId"

CEF key name "deviceExternalID" should be "deviceExternalId" to be mapped properly to DeviceExternalID field in CommonSecurityLog.
Could you modify the documentation.

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

ID: 34e19f18-b108-2702-eac9-86ed2f1b0e82
Version Independent ID: b2d206e7-61cf-781b-1d8a-6ff4e6116233
Content: Common Event Format (CEF) key and CommonSecurityLog field mapping
Content Source: articles/sentinel/cef-name-mapping.md
Service: microsoft-sentinel
GitHub Login: @limwainstein
Microsoft Alias: lwainstein

@yoichimu
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


Will Gateway support private key protect?

how gateyway protect private key?
will like to adjust use hsm?

this can probably be done using the EnvoyPatchPolicy feature, closing for now


Is there any documentation to create a custom dataset?

lerobot/examples elaborates on how to load and train using the existing dataset on the hugging face repos.
Rather I'd like to know how to turn self-collected data into the dataset.
So, I'd like to if there is some documentation for that.

Same here. I have my data ready but the datasets class seem rather complex to instantiate, so one (or more depending on the number of camera for examples) examples would be very nice.

same here！I really need an example.

same question!

@TheArtificialOutsider @zwbx @RochMollero Got it! We will address this issue very soon.
Any chance you could provide a very short sample of the datasets in the comment?
In the meantime, a few pointers and ressources:
README:

https://github.com/huggingface/lerobot?tab=readme-ov-file#the-lerobotdataset-format

See how we use from_preloaded:

https://github.com/huggingface/lerobot/blob/01f8cede0b5f1c16330205b35f4391939e11cb3e/lerobot/scripts/control_robot.py#L336-L344 (see dataset.stats = stats to use it directly after)
https://github.com/huggingface/lerobot/blob/main/lerobot/scripts/push_dataset_to_hub.py#L247-L258

See the content of these files to instantiate the hf_dataset, encode the videos, or store frames, etc.

from hdf5: https://github.com/huggingface/lerobot/blob/main/lerobot/common/datasets/push_dataset_to_hub/aloha_hdf5_format.py#L209-L211
from zarr: https://github.com/huggingface/lerobot/blob/main/lerobot/common/datasets/push_dataset_to_hub/pusht_zarr_format.py#L257-L259
from parquet: https://github.com/huggingface/lerobot/blob/main/lerobot/common/datasets/push_dataset_to_hub/dora_parquet_format.py#L213-L215
from pickle: https://github.com/huggingface/lerobot/blob/main/lerobot/common/datasets/push_dataset_to_hub/xarm_pkl_format.py#L176-L178

cc @michel-aractingi for visibility ;)

Hi, thanks for your attention to this matter. I am using RLbench dataset now. I have raw data now, containing image observations, actions. how can I organize them and transfer them to the hf dataset?
Get Outlook for iOShttps://aka.ms/o0ukef

From: Remi @.>
Sent: Thursday, July 11, 2024 7:08:41 AM
To: huggingface/lerobot @.>
Cc: Wenbo Zhang @.>; Mention @.>
Subject: Re: [huggingface/lerobot] Is there any documentation to create a custom dataset? (Issue #304)
CAUTION: External email. Only click on links or open attachments from trusted senders.

cc @michel-aractingihttps://github.com/michel-aractingi for visibility ;)
—
Reply to this email directly, view it on GitHubhttps://github.com/huggingface/lerobot/issues/304#issuecomment-2221560936, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AJFDSL3NMJK2265XNQKPZDDZLWSWDAVCNFSM6AAAAABKL436HWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRRGU3DAOJTGY.
You are receiving this because you were mentioned.Message ID: @.***>

It would be even better if there were tutorials on how to train using custom data from the gym simulation environment.

Hey there, we are still working on a simplification of the dataset class + upload to hub + tutorial! :)
Sorry if it's taking some time!
Unfortunately, the only option now is to get familiar with: https://github.com/huggingface/lerobot/blob/main/lerobot/scripts/push_dataset_to_hub.py
See some example commands in header. You can eventually adapt one of them to your dataset format.
If you have issue understanding the code, reach out to us on discord #help channel

Closing this as we now have a tutorial to easily record and push your own dataset.
Feel free to reopen if need be ;)

As said in the tutorial “If you don't want to push to hub, use --push-to-hub 0.”, where --push-to-hub 0 should be use?  Replace --repo-id ${HF_USER}/koch_test \ with --push-to-hub 0?
Thanks and best regards!

--push-to-hub 0 is an option of the lerobot/scripts/control_robot.py script. This is simply to deactivate uploading your dataset to the hub when using the record function.
As for the tutorial, it teaches you — amongst other things — how to record a LeRobotDataset with the Koch v1.1 arm (although it can be adapted to other robots, we are working on it).

It seems to me that a useful example would be as ACT, where the dataset is saved in a clear way.

Could you elaborate? What other scenario do you have in mind?


Questions about your work

Hi thanks for your contribution. I have some questions on the work as I'm confused by the training details in the paper.

For the L_simple this is a normal L2 reconstruction loss like in paint by example where you only modify the input layer unet to add more channels is this correct? Does this mean you need to train the unet input layer too? I wonder if you tried the palette method of denoising instead.
For the L_vgg for training the diffusion model, do you decode the latents to compute VGG loss? How many steps did you take to decode? I thought this will be very slow for each step.
For both training losses, they are trained together with no two-stage process? What lambda weights did you give to each training loss?
Is I_gt something that is already available, meaning to say you don't need any paired data to train the model? So the training is self-supervised?
Do you train the whole model or just some layers? How long does training take?

Warping model:

Do you use a pretrained warping model, or is this something you train from scratch? If train from scratch, how long did you take and how many data points?
What is the main difference between this warping model and the models from older papers?

Thanks for your work again

Hi @Polaralia, Thank you for your interest in our work! To answer your questions:

For the model architecture, that is, the structure of UNet, in fact what we use is consistent with the paint by example. Also, our training will be performed on the entire UNet.
For the calculation of vgg loss, we will indeed perform it after decoding the latents. In our experiments, we only sampled once.
During the training process, we train separately for the warping model and the diffusion model.
For the virtual try-on task, we have an image of a person and an image of the clothes that preson is wearing, which constitute a training data pair. I_gt represents the person image.
Yes. And it takes ablout 2 days on 2~4 A100 GPUs.
For the warping module, we simply retrained the model based on the previous method.
For more details, please refer to the complete training code we released later.

Hi @Limbor thank you for the explanation! Now i understand the paper better. For the warping method which one do you use? I am not familiar with warping but curious about this. Is there a fixed resolution to output the flow pixels and how long do you need to train it?

@Polaralia In our experiments, we used PF-AFN as the backbone of the warping module and only trained on 256*192 resolution, it took about 10 hours. I hope this clarifies your doubts.

Hi @Limbor thank you. Do you know if your method works for other kinds of garments other than the top garment? for example bottom clothes. It seems that the warping model only deals with the top, do you know if the method works for bottom/other kinds of clothes/dress?

Hi @Polaralia. Actually, in supplementary, we have conducted experiments on DressCode, which contains three categories of clothes (i.e., top, bottom and dresses). According to the experiment results, the warping module is capable to handle different kinds of clothes.

@Limbor Thank you for the clarification. This is a very interesting contribution. If i am correct, the warping module for the PF-AFN only deals with top category, but you have trained it for all categories? I am curious do you know for out of distribution data does the prediction work accurately and what are some bad edge cases you have seen? Will you be releasing the checkpoints/code for this too?
I am interested in building on your work and learn how the warping model works with diffusion. I am not publishing in the try on space but working more generally in the diffusion space and will cite your work where relevant if I manage to publish.  Thank you for your work!

@Polaralia Yes, we trained on other clothing categories using the PF-AFN network without specific modifications. In terms of effect, the prediction of pants may be simpler, but the prediction of the length of skirts (especially dresses) is a more difficult issue. Regarding the training code of DressCode, we will also release it later.

Hi @Limbor, thanks for sharing! I am certainly interested in checking out the checkpoint/prediction example code for the warping models if you have it available as I am curious about the effects of warping, thank you!

Thanks for releasing the code @Limbor. Is the warping model available as well?

Yes, we have released its model weight, named warp_viton.pth, and we will also release the inference and training code for warping modules and the usage of them.

@Limbor when do you think you will be able to provide code for inference ?

@RishiGitH We have released the inference and training code for warping module

@Limbor i have 5 images of a girl and 2 images of cloth... can we generate image from this??

@sudip550 If you want to test this model on your own data, you should first do some preprocessing on your data refereing to https://github.com/sangyun884/HR-VITON/issues/45#issue-1515217009 and then do warping and blending with our model.


SG Ingress & Egress rules management tab

[x] Security groups ingress rules management tabs
[x] Security groups egress rules management tabs

Hi @GabrielBrascher I think this could be a quick one to implement, I'll ping you on few others that you may pick/choose with some examples/boilerplate code and hints so you can be productive quickly. (this week asap)

@RitchieVincent it's similar to other network tab components, after you're done with other please take this one as well.

Fixed


remove usage of git4idea in modules that shouldn't explicitly depend on git

we have a number of places in the UI where we are using git4idea.DialogManager for non git related things.  These should be replaced with something from the basic platform.

also related to #1896


kwk::shape should be aware of its original storage_order

Tasks

[ ] Fix #8 by implementing kwk::storage_order + associated concepts
[ ] Add support for storage order in kwk::shape
[ ] Fix #39
[ ] Fix #33

Depends on

#44

After soem thoghts, storage_order is not somethign to be done this way. New issue will replace this.


#20 Improve case-sensitive path comparison

Hi. Can you check new logic for case-sensitive path comparison. I don't quite understand the logic of the 3 comparer. Can you describe it in more detail?

Thanks! Merged in scope of #99.


Don't send 'next_run=false' to oVirt 3.6

In oVirt virtual machines have two sets of settings: the settings used
by the current running virtual machine, and the settings that will be
used the next time the virtual machine is started. To select which one
to update it is necessary to use the 'next_run' parameter. For example,
to update the memory that will be used the next time is started the
API request should be like this:
PUT /ovirt-engine/api/vms/{vm:id};next_run=true
The value 'false' should tell the server to update the current running
configuration. But in version 3.6 of oVirt it is just ignored, the mere
presence of the 'next_run' parameter is interpreted as a request to
modify the next start settings, regardless of its value.
Currently the oVirt provider uses 'next_run=false' explicitly, and as a
result it fails to update the current settings. This patch changes the
provider so that it doesn't send the 'next_run' parameter when its value
is 'false'.
https://bugzilla.redhat.com/1356468

@masayag, @borod108 please review.

@jhernand tests fails due to the change.

@masayag you are right, I forgot to update the tests. It should be fixed now.

@durandom can you review/merge?


Windows Exception error

Looking forward for reporting a security issue:
Please report security issues by following our security policy: https://contribute.freecodecamp.org/#/security
Describe the bug
When trying to SAVE and you click an empty save slot the following error exception occurs.
To Reproduce
Steps to reproduce the behavior:

Start game
When first presented with how to save your progress click save button in bottom right corner.
Then Save screen appears with empty save slots, click an empty slot.
See error

Expected behavior
Gameplay should  be Saved up to this point in time, but gives an error.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Windows 11
Browser: EDGE
Version: 108.0.1462.54 (Official build) (64-bit)

Additional context
Add any other context about the problem here.
I was using the APP via the downloaded  ZIP file on windows 11, not via a web browser or smartphone.

Same

https://github.com/freeCodeCamp/LearnToCodeRPG/commit/c220e63c220cb5ca6b3169b1f771c8524d7d99c1


sort: be more specific about string sorting order in docs

Simple changes in documentation regarding sorting strings. Instead of "increasing order" it's "lexicographic order"
Fixes #34666

@googlebot I signed it!


STORM-2477: Add generics to Config types

Because this touches so much of the code to avoid having to constantly upmerge I have a script that will transform most of the code automatically. transform.sh . I don't plan on checking it in.  If everyone is OK with the change I will run the script and just check those changes in.  The other changes and small updates to make the result of running the script not produce compile errors.  Overall this drops more than 800 warnings from the code base (according to eclipse).

@srdo I addressed your review comments about empty map.

@revans2 I meant you could get rid of the emptyMap variables entirely.

@srdo can do.   I like good clean code and I am happy to see people call me out on things that should be fixed.  I find that people who read the code closely and nit pick find more serious bugs during the review which saves me a lot of time debugging and fixing them later on.

@ptgoetz @HeartSaVioR if either of you could take a look at this I would appreciate it.

Left a comment to know intention of changing build process. Others look great.

+1 Nice work.

@HeartSaVioR Thanks I'll make the change


fix: add /bigobj to non-VCPKG MSVC CMake flags, fixing ninja build, and fix CMakeSettings.json

Adds /bigobj to Debug MSVC CMake flags for building the library, required to use nlohmann::json with Ninja (see https://github.com/nlohmann/json/issues/1114) (in library/CMakeLists.txt so it shouldn't affect vcpkg)
Fixes the "x64-Debug" profile to actually build in Debug instead of currently Release
Adds "x64-Release", "x64-Debug-Coro", "x64-Release-Coro" CMake profiles to use with Visual Studio

we don't support vs2015 though

It's an error that happens since vs2015, it still happens on recent versions with Ninja


169 | Majority Element | Python

Majority Element -
What will change -
Type of Issue -

Adding New Code

Programming Language

Python

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

Please assign me this issue @SSKale1

hi can i be assigned for this task?


segfault on <IP_ADDRESS>5 (linux)

🧐 Have you fully read the README and viewed similar issues?

[X] I have read the README
[X] I have read the FAQ
[X] I have checked previous issues, open and closed
[X] I have completed all troubleshooting steps and the issue is not present on stock but returns after re-applying SpotX-Bash

Сountry of your account
Poland
What is your Spotify Plan?
Free
Do you have Spicetify installed?
No
ℹ Computer information
- Spotify version: <IP_ADDRESS>5
- OS: Arch Linux
- Shell: zsh

📝 Description

install spotify with aur
run spotify (runs fine)
run bash <(curl -sSL https://spotx-official.github.io/run.sh) -cfhl -P /opt/spotify
run spotify and get

User flags: --enable-features=UseOzonePlatform --ozone-platform=wayland
/opt/spotify/spotify: /usr/lib/libcurl-gnutls.so.4: no version information available (required by /opt/spotify/spotify)
zsh: segmentation fault (core dumped)  /usr/bin/spotify

(same without userflags, .cache/spotify & .config/spotify don't exist)
🖥 Terminal Input/Output
❯ bash <(curl -sSL https://spotx-official.github.io/run.sh) -cfhl -P /opt/spotify

████╗███╗  ███╗ █████╗█╗  █╗  ███╗  ██╗ ████╗█╗ █╗
█╔══╝█╔═█╗█╔══█╗╚═█╔═╝╚█╗█╔╝  █╔═█╗█╔═█╗█╔══╝█║ █║
████╗███╔╝█║  █║  █║   ╚█╔╝██╗███╔╝████║████╗████║
╚══█║█╔═╝ █║  █║  █║   █╔█╗╚═╝█╔═█╗█╔═█║╚══█║█╔═█║
████║█║   ╚███╔╝  █║  █╔╝ █╗  ███╔╝█║ █║████║█║ █║
╚═══╝╚╝    ╚══╝   ╚╝  ╚╝  ╚╝  ╚══╝ ╚╝ ╚╝╚═══╝╚╝ ╚╝

Using Spotify Directory: /opt/spotify

Latest supported version: <IP_ADDRESS>5
Detected Spotify version: <IP_ADDRESS>5

✔ Cleared app cache
✔ Detected & restored backup
✔ Applied free-tier plan patches
✔ Enabled experimental features
✔ Removed non-music categories on home screen
✔ Removed lyrics background color
✔ Finished

📸 Screenshots
No response

Uninstalling (and removing the /opt/spotify directory completely, idk if this was necessary) and then installing it again resolved the issue for me

@bytedream @k1gen
glad the clean install got things going. To be honest, I'm not sure why this happens... I don't use arch regularly enough to trigger the issue.
did either of you happen to test if <IP_ADDRESS>5 worked prior to patching with SpotX-Bash (or run the SpotX-Bash --uninstall option to reverse the changes it made and then test the client)? I'm curious if the update path itself would have caused this somehow and it was unrelated to the patching or if it was indeed triggered by SpotX-Bash.
Not asking either of you to recreate the scene but if something like this happens again or anyone else out there who is reading this... and it's confirmed that the same client version worked prior to patching with SpotX-Bash (or works again after running SpotX-Bash --uninstall option)...
I would be interested to see the /opt/spotify directory when this happens. ls -la output and hashes of all the files to see if something can be narrowed down. Or even better, compressing the directory and uploading it to some 3rd party file host so the contents could be inspected.
It doesn't make much sense that SpotX-Bash could be causing this and yet a clean reinstall + patch works without issue. But... can't rule it out entirely. I do see lots of "segmentation fault" reports from regular Spotify users on arch after upgrading (no SpotX-Bash involved).

yeah, I noticed the script was taking a long time to detect the installation after upgrading from <IP_ADDRESS>7 to <IP_ADDRESS>5. if I clean-upgrade - this disappears, and there is no such problem now

yeah, I noticed the script was taking a long time to detect the installation after upgrading from <IP_ADDRESS>7 to <IP_ADDRESS>5. if I clean-upgrade - this disappears, and there is no such problem now

See my previous comment edit.. I was in process of rewriting it when you commented. Not sure how to best handle this.. but at least it's now noted what's going on and how to workaround it for now.

thank you for your research on this matter and your great work on SpotX-Bash overall, @jetfir3!

I've corrected this issue.
The user should now be able to use SpotX-Bash normally without having to consider Spotify updates or existing backup files.


Long start delay

Hi, first of all, this project is fantastic.
I'm currently using this with my Sonos speakers.
The issue I am having is a long delay (~4-5s) between starting a video (on Youtube) and the sound getting played.
This only happens when there is silence before playing the audio. The delay between video and audio when playing is minimal (~0.5s)

Thanks! But I'm afraid you can't have your cake and eat it: if you're using the "play silence" option to prevent disconnection in the absence of sound, you have to wait until the Sonos has finished playing the "silence" still in its receive buffer. AFAIK there is no way that swyh-rs can clear that buffer on the S)onos side, except by you toggling the button for the Sonos in swyh-rs (and thus sending the "stop play", "clear playlist", "load playlist" and "start play" commands to the Sonos).

I did some tests using WAV 16 bit 44 kHz and silence, but was not able to get a delay above 1 second.
This is the setting I used in my config.toml:
CaptureTimeout = 250
InjectSilence = true

But I don't have Sonos, perhaps that's the difference.

Yesterday I got a delay above 10sec. I just switched between Youtube videos.
I'll try to test it with your settings today.
One thing I thought of is reducing the size of the silence buffer. Could that maybe decrease the delay?

The size of the silence buffer is based on the value of the capture timeout, if you decrease the capture timeout you atomatically get a smaller silence buffer.

It took a bit longer than expected, but I finally looked at the config file.
It looked like this:
CaptureTimeout = 2000
InjectSilence = false

I then changed it to your settings and now the delay is between 500ms and 1s. Much better.
Are the settings I had the default settings? Because I'm sure that I did not change them myself

They are indeed the default settings. Glad to hear that it's better now!

I suppose I can close this now.


Migrate SDK package contents to Core and UX packages

Reality Collective - Reality Toolkit Feature Request
Is your feature request related to a problem? Please describe

Most of what is currently in the SDK should be in core as essential default components.  The only others are UX related and should be migrated to the UX package.
The SDK package should then be deprecated

Resolved via - https://github.com/realitycollective/com.realitytoolkit.core/pull/13


Protection intent changes and bug fixes

Added protection intent API's.
Fixed documentation for a few classes.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-java

Automation for azure-sdk-for-node

Automation for azure-sdk-for-go

Can one of the admins verify this patch?

@ravbhatnagar please review, addition to new apis to stable api-version.

Automation for azure-sdk-for-js


feat: Allow users to configure the internal thread pool (#11)

This PR introduces a new feature that allows users to configure the internal thread pool used in the _raw_indexing_method function of the EarthEngineBackendArray class.
A new optional parameter executor_kwargs has been added to the function. This parameter is a dictionary that can be used to pass configuration options to the ThreadPoolExecutor. If executor_kwargs is not provided, an empty dictionary is used, preserving the current behavior.
This feature provides users with more flexibility and control over the execution of the function, especially in scenarios where specific thread pool configurations are needed.
Changes:

Added executor_kwargs parameter to _raw_indexing_method function in xee/ext.py.

Please review and let me know if any changes are required.

Hi @alxmrs when i run the test file, this error pops up.
Could not import runpy module Traceback (most recent call last):   File "<frozen runpy>", line 15, in <module>   File "<frozen importlib.util>", line 16, in <module>   File "/Users/hridoy/Desktop/open_source/Xee/xee/types.py", line 16, in <module>     from typing import Union, TypedDict   File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/typing.py", line 27, in <module>     import contextlib   File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/contextlib.py", line 7, in <module>     from functools import wraps   File "/Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/functools.py", line 22, in <module>     from types import GenericAlias ImportError: cannot import name 'GenericAlias' from partially initialized module 'types' (most likely due to a circular import) (/Users/hridoy/Desktop/open_source/Xee/xee/types.py)
I renamed the types.py to type.py then I run the test again, it runs but no output is shown in the terminal. any guidance to run the ext_integration_test.py file?

@alxmrs Thanks for your guidance. Would you check if these changes are working properly? if not,  Please let me necessary the changes I need to make. Thanks

Please fix the merge conflicts in this branch.

Hi @alxmrs . Thanks for the feedback. ,I added  a new parameter executor_kwargs has been added to the init method also resolved merge conflict.

Hi @alxmrs , Would you check the new commits address your comments? If there is still any issue  I would be happy to work on that.

It feels like you are using a bot to make this patch. I'll be happy to work with you to add this feature. First, please do a self review before I make any additional reviews.

Thanks! Please give me couple of weeks.


UNetWeaver error: NetworkBehaviour cannot have generic parameters

I am evaluating the demo of Full Inspector 2.6.2 (Unity 5.1 version) on Unity 5.3.5f1.  When scripts are compiled Unity throws two errors:
UNetWeaver error: NetworkBehaviour BaseNetworkBehavior`1 cannot have generic parameters
UnityEngine.Debug:LogError(Object)
Unity.UNetWeaver.Log:Error(String) (at /Users/builduser/buildslave/unity/build/Extensions/Networking/Weaver/Program.cs:20)
Unity.UNetWeaver.NetworkBehaviourProcessor:Process() (at /Users/builduser/buildslave/unity/build/Extensions/Networking/Weaver/UNetBehaviourProcessor.cs:48)
Unity.UNetWeaver.Weaver:ProcessNetworkBehaviourType(TypeDefinition) (at /Users/builduser/buildslave/unity/build/Extensions/Networking/Weaver/UNetWeaver.cs:1064)
Unity.UNetWeaver.Weaver:CheckNetworkBehaviour(TypeDefinition) (at /Users/builduser/buildslave/unity/build/Extensions/Networking/Weaver/UNetWeaver.cs:1549)
Unity.UNetWeaver.Weaver:Weave(String, IEnumerable`1, IAssemblyResolver, String, String, String) (at /Users/builduser/buildslave/unity/build/Extensions/Networking/Weaver/UNetWeaver.cs:1658)
Unity.UNetWeaver.Weaver:WeaveAssemblies(IEnumerable`1, IEnumerable`1, IAssemblyResolver, String, String, String) (at /Users/builduser/buildslave/unity/build/Extensions/Networking/Weaver/UNetWeaver.cs:1748)
Unity.UNetWeaver.Program:Process(String, String, String, String[], String[], IAssemblyResolver, Action`1, Action`1) (at /Users/builduser/buildslave/unity/build/Extensions/Networking/Weaver/Program.cs:34)
UnityEditor.Scripting.Serialization.Weaver:WeaveUnetFromEditor(String, String, String, String, Boolean)

Failure generating network code.
UnityEditor.Scripting.Serialization.Weaver:WeaveUnetFromEditor(String, String, String, String, Boolean)

Then the Full Inspector Serializer Manager window is opened.  These errors prevent other Editor scripts from compiling in the project as well.

Looks like the issue is with a generated file not including the correct define guards:
// WARNING: This file has been automatically generated by Full Inspector, as part of the serializer
//          selection wizard. It will be overwritten if you change your selected serializers.

// Missing UNITY_5_3
#if !(UNITY_4_3 || UNITY_4_5 || UNITY_4_6 || UNITY_5_0 || UNITY_5_3)
namespace FullInspector {
    public abstract class BaseNetworkBehavior : BaseNetworkBehavior<FullInspector.FullSerializerSerializer> {}
}
#endif

I really should update the demo - these issues have been fixed. Let me know if you need a non-trial build for testing. Thanks!


reduce v2 plugin list, list parent plugins download

@tdawson @bparees the origin extended tests and jenkins v2 images test both look good.  I was also able to confirm from the make logs that the plugins removed from the list that we care about were in fact still downloaded (subversion wasn't but I was wondering if we really need that one)
When all is said and down then, I suspect the larger list for v1 jenkins will remain constant for plugins we don't own, while the smaller list of plugins we don't own for v2 can still change.
Also, for @bparees and I, I believe this will shield from the scenario where a plugin changes dependency versions but the plugin itself does not change its version.  Since we are only including the "parent" plugins now.
I won't change the rhel Dockerfile until a) this PR merges, and b) we confirm with @tdawson that we have new RPMs for the remaining plugins (I think some of them have still changed with the recent PRs)
Even if the test passes, I won't merge until @bparees gets back from PTO and has a chance to digest this (and perhaps argues on inclusion of subversion)

@gabemontero my main concern with this is that if a "parent" plugin changes in the future to no longer require a particular dependent plugin, our image will suddenly stop including it, potentially breaking people's configurations.
Obviously we're already exposed to that since we aren't declaring every plugin that ultimately gets installed, but seeing some of the git and SCM plugins being removed from this list does leave me worried.  So I guess i do wonder if there is a more surgical approach we should take regarding which plugins we explicitly provide and which we implicitly pick up via dependencies.
Of course the rhel based image wouldn't get broken since in that case all the plugins are expliciltly installed by rpm.

Especially if we stayed more aggressive with reducing what we explicitly install, I could add to our tests and confirm that certain plugins are still installed.  We would at least be more assured that we would find out quickly that certain plugins were in fact missing.
Otherwise, as far as the churn goes, while we have had to bump the git related plugins on occasion, the workflow/pipeline plugins have been the greater offenders.  Conceivably in the spirit of being more surgical as you say, we could initially limit the implicit pickup to those, and hopefully only have to occasionally deal with the churn for git, etc.

Especially if we stayed more aggressive with reducing what we explicitly install, I could add to our tests and confirm that certain plugins are still installed.

I like that approach.

OK I added subversion/mapdb back in and then added the tests that confirm that non-pipeline plugins that we used to explicitly pull in are still pulled in by pipeline aggregator/parent plugins.
Don't test for workflow/pipeline under the premise that if they change the relevant list for that umbrella, then we defer to them.

lgtm


Add e2e test for hosted cluster behind a proxy

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Ref https://issues.redhat.com/browse/HOSTEDCP-731
blocked by https://issues.redhat.com/browse/OCPBUGS-6651
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/hold

Needs https://github.com/openshift/cluster-network-operator/pull/1694

/test e2e-aws

/test e2e-aws

/test e2e-aws

needs https://github.com/openshift/cluster-storage-operator/pull/336
/test e2e-aws

/test e2e-aws

/test e2e-aws

/hold cancel

/hold

/hold cancel
/lgtm

/cherry-pick release-4.12


Restoring a Backup with VLANs makes system unreachable

I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
I have searched the existing issues and I'm convinced that mine is new.
Describe the bug
Valid Backup from 20.1.9_1, New Box upgraded to 20.7.1.
Restoring Backup with VLANs makes system unreachable
To Reproduce
Old Box running  20.1.9_1, using interfaces "em0" to "em3"
New Box upgraded to 20.7.1, using interfaces "igb0" to "igb3"
Restoring Backup  by GUI with Reboot.
Watching the serial console to assign interfaces.
No dialog could be entered in console.
System does not come back.
Expected behavior
Dialog on the console or GUI to assign Interfaces,
or Migration of Interfaces based on the same order.
Relevant log files
serial console output
Initializing...........done.
igb0: link state changed to UP
igb1: link state changed to UP
igb3: link state changed to UP
Starting device manager...done.
Configuring login behaviour...done.

Default interfaces not found -- Running interface assignment option.

Press any key to start the manual interface assignment: 1

Valid interfaces are:
igb0             f4:90:ea:00:**:03 Intel(R) PRO/1000 PCI-Express Network Driver
igb1             f4:90:ea:00:**:04 Intel(R) PRO/1000 PCI-Express Network Driver
igb2             f4:90:ea:00::**::05 Intel(R) PRO/1000 PCI-Express Network Driver
igb3             f4:90:ea:00::**::06 Intel(R) PRO/1000 PCI-Express Network Driver

You now have the opportunity to configure VLANs.  If you don't require VLANs
for initial connectivity, say no here and use the GUI to configure VLANs later.

Do you want to configure VLANs now? [y/N]: n

VLAN interfaces:

em0_vlan25      VLAN tag 25, parent interface em0

If you do not know the names of your interfaces, you may choose to use
auto-detection. In that case, disconnect all interfaces now before
hitting 'a' to initiate auto detection.

Enter the WAN interface name or 'a' for auto-detection: igb1

Enter the LAN interface name or 'a' for auto-detection
NOTE: this enables full Firewalling/NAT mode.
(or nothing if finished): igb0

Optional interface 1 description found: OPT1
Enter the Optional interface 1 name or 'a' for auto-detection
(or nothing if finished):

The interfaces will be assigned as follows:

WAN  -> igb1
LAN  -> igb0

Do you want to proceed? [y/N]: y

Writing configuration...done.

Additional context
Workaround:
Logging in in serial console
Using the saved backup and converting the inferfaces with sed:
sed -e 's|em0|igb0|' -e 's|em0|igb0|' -e 's|em1|igb1|' -e 's|em2|igb2|'  -e 's|em3|igb3|' /conf/backup/config-1597766805.0764.xml > /conf/config.xml
Environment
Old Box:
OPNsense 20.1.9_1 (amd64, OpenSSL).
AMD GX-416RA SOC (1597.14-MHz K8-class CPU)
interfaces "em0" to "em3"
New Box:
OPNsense  20.7.1 (amd64, OpenSSL).
AMD GX-416RA SOC (1597.14-MHz K8-class CPU)
Intel(R) PRO/1000 Network Connection, Version - 2.5.3-k
interfaces "igb0" to "igb3"

Looks like the rename wasn’t complete as it’s still insisting on em0_vlan25.

I was watching on the serial console assigning interfaces automatically and non interactive.
The Problem:
The user has no way to intercept the auto renaming failing.

We could reject backup import on mismatches if that is in any form or shape more user friendly. In a way, it's not really a backup.. it's using a backup to do a migration and that requires external knowledge in the form of config.xml edits.

Thanks.
with pfsense the Script was interactive to allow migration.
on OPNsense the standard input seems to be redirected to /dev/null.
I though this was an unintended feature regression.

Hi Dirk,
I think it just stalled instead of going into auto-select after a brief timeout like it has been on OPNsense for most versions. For some that is preferable, but we also received a lot of complains for it from the other half of the users who had an opinion on this and it can clash with automatic SSH access for installer.

OPNsense the standard input seems to be redirected to /dev/null

I don't believe that is true. Are you not seeing boot messages from rc system?
Cheers,
Franco


Fix Volume.delete with kwarg label

Fixes a bug in the backwards-compatibility layer of Volume.delete, raised in #1761
Also adds a test for the volume create and volume delete CLI

@prbot approve, new code is mostly tests