Update order of steps in GH

Summary:

Changes the order of steps running according:

Build -> Test -> Code Coverage -> Lint

Issue Reference(s):
Fixes #394
/claim #394
Build & Testing:

[x] I ran cargo test successfully.

Checklist:

[x] I have performed a self-review of my own code.

@tusharmath, confirming if we want the Code Coverage upload after all the steps are executed. That is how it is structured in this PR, cause it didn't make sense to me to not let the linter run in case of code coverage upload failure. But let me know if you think any differently.
Also, can you approve the workflow so I can verify it here again? Updated workflow tested once here - https://github.com/utsav00/tailcall/actions/runs/6369299383/job/17289423954

Hey @utsav00 thank you for contributing to tailcall 🙏

@tusharmath, confirming if we want the Code Coverage upload after all the steps are executed. That is how it is structured in this PR, cause it didn't make sense to me to not let the linter run in case of code coverage upload failure. But let me know if you think any differently.

That makes perfect sense! Thanks!

@utsav00 I posted this on twitter — https://twitter.com/tusharmath/status/1708813317404520529
I think there consensus in doing it in parallel, do you mind making that change? I will update the github issue.

Interesting. Then the idea is to have two jobs, one for testing and one for format and lint checks, and the deploy and release should still be dependent on both of these two jobs, right?

@utsav00 Yes, that would be the best! Thanks :)

Can you review this again @tusharmath please? Tested this here - https://github.com/utsav00/tailcall/actions/runs/6396355226/job/17361951318

Thanks @utsav00! This will truly improve the dev-ex of our contributors 🙏

@utsav00 It seems like nothing is building on main. Can you have a look.

Hey @tusharmath, I just got time to look into this, apologies for delay. Observing the actions, it seems like the workflow is running well. Did you see a recurrence of this exact issue again from the screenshot? I can look into this if this is a repeating behavior

Yes working now! Thanks once again.
On Thu, 5 Oct, 2023, 10:13 pm Utsav, @.***> wrote:

Hey @tusharmath https://github.com/tusharmath, I just got time to look
into this, apologies for delay. Observing the actions, it seems like the
workflow is running well. Did you see a recurrence of this exact issue
again from the screenshot? I can look into this if this is a repeating
behavior
—
Reply to this email directly, view it on GitHub
https://github.com/tailcallhq/tailcall/pull/396#issuecomment-1749291861,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABPPMQHR7GMPKSSSY3AOQTX53PTLAVCNFSM6AAAAAA5OEN4HWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONBZGI4TCOBWGE
.
You are receiving this because you were mentioned.Message ID:
@.***>


NR-174236 - Entity synthesis for AWS - NAT Gateway

Relevant information

Checklist

[x] I've read the guidelines and understand the acceptance criteria.
[x] The value of the attribute marked as identifier will be unique and valid.
[x] I've confirmed that my entity type wasn't already defined. If it is I'm providing an explanation above.

Can you change the name of the folder to match the domain type of the entity defined? INFRA AWSNATGATEWAY... expected infra-awsnatgateway


@NativeCoroutinesState does not work for Interfaces

I have the following use case: I want my common ViewModel to implement a ViewModelInterface, so that I can easily replace it with mock data for SwiftUI/Compose previews. My ViewModelInterface looks something like this:
interface MyViewModelInterface {
    fun onButtonPressed()

    @NativeCoroutinesState
    val uiState: StateFlow<MyUiState>
}

I expected this to turn my uiState from a StateFlow to MyUiState in my Swift code. However, what actually happens is that uiState is not accessible from Swift at all anymore.
I tried to write the code generation from the annotation myself, but the following does not work either:
interface MyViewModelInterface {
    fun onButtonPressed()

    @HiddenFromObjC
    val uiState: StateFlow<MyUiState>
}

@ObjCName(swiftName = "uiState")
val MyViewModelInterface.uiStateValue: MyUiState
    get() = uiState.value

One thing that worked for me was putting the objc getter inside the Interface itself. This is a bit clunky though because the iOS specific code has to live in the common code, where the interface is defined.
interface MyViewModelInterface {
    fun onButtonPressed()

    @HiddenFromObjC
    val uiState: StateFlow<MyUiState>

    @ObjCName(swiftName = "uiState")
    val MyViewModelInterface.uiStateValue: MyUiState
        get() = uiState.value
}

Is there a way to get this to work, or is there an alternative way I could achieve my goals?
edit: I got this to work with an expect/actual interface, but it's still a lot of extra code
expect interface MyViewModelInterface {
    fun onButtonPressed()
    val uiState: StateFlow<MyUiState>
}

// iOS
actual interface MyViewModelInterface {
    actual fun onButtonPressed()

    @HiddenFromObjC
    actual val uiState: StateFlow<MyUiState>

    @ObjCName(swiftName = "uiState")
    val MyViewModelInterface.uiStateValue: MyUiState
        get() = uiState.value
}

//Android
actual interface MyViewModelInterface {
    actual fun onButtonPressed()
    actual val uiState: StateFlow<MyUiState>
}

Hi! This is an unfortunate issue with the Kotlin Objective-C interop.
Extensions properties/functions are exported to ObjC as top level properties/functions (also see the docs).
In Swift you should see an extra "class" MyViewModelInterfaceNative which contains the uiStateValue property/function.
To make this a little better you could define an extension property in Swift (that uses the uiStateValue from this native "class").


Update Agate-sql to version 0.5.8

check the upstream
https://github.com/wireservice/agate-sql/tree/0.5.8

check the pinnings
https://github.com/wireservice/agate-sql/blob/0.5.8/setup.py
https://github.com/wireservice/agate-sql/blob/0.5.8/setup.cfg
https://github.com/wireservice/agate-sql/blob/0.5.8/example.py

corrected the python pinings

check changelogs
https://github.com/wireservice/agate-sql/blob/0.5.8/CHANGELOG.rst

There were only bug fixes mentioned in the change log, for example:

Fix tests for Linux packages.

additional research
https://github.com/conda-forge/agate-sql-feedstock/issues

There are currently no open issues in conda-forge

verify dev_url
verify doc_url
added pip to the test section
verify the test section
additional tests

In order to test the agate-sql package version 0.5.8 the following test was conducted
conda build agate-sql-feedstock --test
The test result was the following:
All tests passed
In additioon to test the scvkit package was used to test agate
conda build csvkit-feedstock --test
the test result was the following:
All tests passed
Based on the research finding and on the test results we can conclude that it is safe to update agate-sql to version 0.5.8

Please add missing packages pip and wheel to the host section. And why pip check is disabled?

Pip and wheel were added to the host section
https://github.com/AnacondaRecipes/agate-sql-feedstock/blob/update_to_0.5.8/recipe/meta.yaml#L26&L27
In addition the pip check was added to the test section.
https://github.com/AnacondaRecipes/agate-sql-feedstock/blob/update_to_0.5.8/recipe/meta.yaml#L36&L41

@varlackc Should we merge or close this PR?


Minified *.properties and *.css are still requested

Hello,
I have a problem, when minifying with openui5_preload that the included properties and css files are still requested seperately, altough they are included in the Component-preload.js.
This is my folder structure:
|   Gruntfile.js
|   package.json
\---webapp
    |   Component.js
    |   index.html
    |   manifest.json
    +---controller
    |       View1.controller.js
    +---css
    |       style.css
    +---i18n
    |       i18n.properties
    +---model
    |       models.js
    \---view
            View1.view.xml

I have the following openui5_preload config:
openui5_preload: {
	component: {
		options: {
			resources: {
				cwd: 'webapp',
				prefix: 'test/minification',
				src: [
					'**/*.js',
					'**/*.fragment.html',
					'**/*.fragment.json',
					'**/*.fragment.xml',
					'**/*.view.html',
					'**/*.view.json',
					'**/*.view.xml',
					'**/*.properties',
					'**/*.json',
					'**/*.css'
				]
			},
			dest: 'dist'
		},
		components: 'test/minification'
	}
}

which generates the following Component-preload.js:
jQuery.sap.registerPreloadedModules({
	"version": "2.0",
	"name": "test/minification/Component-preload",
	"modules": {
		"test/minification/Component.js": "sap.ui.define([\"sap/ui/...",
		"test/minification/controller/View1.controller.js": "sap.ui.define([\"sap...",
		"test/minification/model/models.js": "sap.ui.define([\"sap/ui/model...",
		"test/minification/view/View1.view.xml": "<mvc:View controllerName=\"test...",
		"test/minification/i18n/i18n.properties": "title=Title\nappTitle = App...",
		"test/minification/manifest.json": "{\"_version\":\"1.1.0\",\"sap.app...",
		"test/minification/css/style.css": "/* Enter your..."
	}
});

When  I run the app, then the i18n.properties and style.css file are still requested seperately:

What can I do to solve this issue? The app works when the properties and style files are not included (as you see in the plunk) - but it is still confusing that they are requested.
Here is my example app: (zip upload to issue did not work...)
https://www.dropbox.com/s/6nnieomdtn4eoq2/openui5_preload_example.zip?dl=0
Plunk here:
https://embed.plnkr.co/KWcMn1/

Notwithstanding issues with the grunt component, I know that UI5 repeatedly makes test calls to the host to find a more specific language file. I.e. First it will try and load an i18n_de.properties file, if it can't find that it tried with a less restrictive i18n.properties file and so on.
So even if you packaged up your i18n.properties file into the component preload I'm pretty sure ui5 will still make all those other calls looking for more specific language files, etc.
I wonder if there is a way to overload this process to have ui5 code ONLY look for the current locale language file?!?

Oh well, the style.css must be loaded because it is linked in the html-file. My fault. The language stuff sounds reasonable.

@denisenepraunig Although you can include files like *.css in your Component-preload.js they will NOT be considered when loading they in any way, e.g.:

<link> tag
jQuery.sap.includeStyleSheet
mainfest.json entry sap.ui5/resources/css

The last use case (or even the last two) could be supported in future, but are currently not.
So for now I would recommend to not include *.css files in your preload and load them separately.

@matz3 Altough the i18n.properties file got packaed, SAPUI5 still tried to request it (and the other specific languages). In the SAP Web IDE build of the Component-preload.json the file is not included.
So - should we not include the properites files inside the preload file? Your default settings package them. According to my Plunker https://embed.plnkr.co/KWcMn1/ they are requested, but if they are not there, the translations are taken from the Component-preload.json.
Or should we include them and ignore that SAPUI5 still tries do load them?

Thanks. I will have a look into this. But it doesn't seem to be correct to me. I think there is a difference between using bundleName and bundleUrl for the sap.ui.model.resource.ResourceModel.
Internal incident<PHONE_NUMBER> (and maybe even<PHONE_NUMBER>) could be related.
As this is most probably an issue within openui5 not the grunt tooling I will close this issue in favor of a new one at https://github.com/SAP/openui5 after having a deeper look into the issue.

@denisenepraunig Finally I had a look into this again. The reason why the properties file is not taken from the preload in in your example for the first time (translation of manifest.json) but for the second (i18n model) is that the module name has to be guessed using the URL (see https://github.com/SAP/openui5/blob/4a7a346b40d0fe480f762154d4b05de276f75fe7/src/sap.ui.core/src/jquery.sap.global.js#L4348).
This only works in this case when the URL starts with a dot. Although it might be possible to fix this issue by improving the "guess" logic I don't think it would be worth. At least I've never heard of this issue before, so it should be very rare.


Add driver for ST LSM9DS1/LSM9DS0

Hi,
is it possible to add drivers for mentioned above IMUs in the near future?
Thank you in advance,

Isn’t there already a driver for this? We used this chip on the PXF back in 2014...

I could find only LSM303 in the repository..

Great, thank you very


Fix #24137 Different number of leading zeros on MINGW printf("%lg")

Use _set_output_format() on MINGW platform to force _snprintf_s() to
conform to the C99 standard and match the other platforms.
Bugsquad edit: Fixes #24137.

https://travis-ci.org/godotengine/godot/jobs/607924096?utm_medium=notification&utm_source=github_status <- i feel like this travis failure isn't due to this patch?

Thanks!

This PR breaks MinGW/clang/LLD build using Windows 10 UCRT, we should additionally have !defined(_UCRT) check.


Periodic code review - v1.2

Manual periodic review of all code files to look for any code issues/smells that may have crept in through incremental updates to the code base.

Code reviewed as part of v1.2 in Jan 2021. Some inconsistencies found with Android version of the code leading to improved performance of iOS working with older Android devices, and reduced battery usage. Also reduced CPU usage in determining whether a device has been seen before/receently.


DAT-10549 add support for azure sql db

@jnewton03 I have not yet added the scheduled trigger. I wanted to test this first then add a schedule. But I can do it whenever you wan't pre-merge


Allow to omit content-type header if no param sent in watcher API

I take it #607 is the quick fix/workadound for status.get, which should be undone, when this is fixed properly, correct?

That is correct.

I'd like to try approach with single pipeline by not apply plug to particular action - kind of:
plug(:enforce_json_content) when action not in [...]


add pyvista axes widget examples

@jourdain, some examples to help with #34

Closing as it is now included in https://github.com/Kitware/trame-vtk/pull/34


[gen_l10n] Add scriptCode handling

Description
Adds script code handling to the supportedLocales list, as well as to the message lookup logic for the localizations delegate. This allows Flutter applications using the gen_l10n tool to correctly parse script code. This change will cause the lookup logic to check in the following priority order:

language+script+country code
language+script code
language+country code
language code only

I picked script code preference over country code preference because it is a more broad category (multiple countries could share the same script code). Also, this is the order used by basicLocaleListResolution.
Further Considerations
Eventually, it would be nice to give developers control over the localizations delegate lookup algorithm, as well as deciding which delegates to expose. However, that seems out of scope for this PR and can be future improvements to the tool.
Issues
This fixes https://github.com/flutter/flutter/issues/53840.
However, this does not address the unique case described in https://github.com/flutter/flutter/issues/51645, which will be addressed at a later time
Tests

Integration tests to ensure that script code is properly being read.

Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: Replace with a link to your migration guide

Closing in favor of https://github.com/flutter/flutter/pull/53868, since there's some Cirrus presubmit weirdness


Possibility to inject key-vault values as environment variables with or without k8s secrets

Describe the solution you'd like
Many applications have a native ability to read options/parameters from environment variables like e.g. dot net core. Right now to get that behavior we need to:

Mount the files that won't be used later
Add secrets as k8s secrets that won't be used later
Map k8s secrets to env.variables

For many applications there is not much need for all of these and it would be a nice feature to just allow injection of variables without the need to configure either mounts or secrets.
Anything else you would like to add:
It would be a cool additional feature if injecting as env. variable can update k8s secrets (like what is happening now during the mount if configured) with enabled auto-rotation that will restart pods to inject new environment variables with renewed secrets if the secret has changed.
Or perhaps it's possible to evaluate the injected variables instead of creating a k8s secret to determine if pod should be restarted.
That would allow for completely pain-free secret update, like e.g. database password update.

Are we able to use Deployment env.valueFrom.secretKeyRef along with SecretProviderClass secretObjects already?

Mount the files that won't be used later

Even with the ability to use env.valueFrom.secretKeyRef, there is still a requirement to mount the files, otherwise the synced secrets aren't created. That's a bit unfortunate; it makes it more complicated to build a chart that can use regular secrets OR the CSI driver in different environments. Each deployment needs to be modified to mount the volume from the CSI driver.

Is there anyway to expose vault secret directly as pod's env variables.  My client does not allow you to use k8s secret due to base64 encode, which is still clear text. For e.g some db and sensitive data store credentials really need to be hidden from K8s admins/developers.


List Relationships API improvements

Pull request check list

[x] Proper tests/regressions included?

Affected functionality

Adding the possibility to filter relationships using a time window startDate and endDate
Adding pagination possibilities pageNumber and pageSize in the API, the next PR will add this capability to the database and do the proper unit tests.

Description of change

The old implementations do not support pagination or time filters. To improve clients usage this PR adds these capabilities.

Hi @Victorblsilveira ,
I have some comments on the Pull Request:

Most important thing: This PR should ONLY focus on introducing changes to the APIs to add parameters for pagination, filtering by trust domain ID, filtering by consent status, and ordering by create_at. It's crucial to maintain a clear and focused scope for better code management and readability.

Regarding the renaming of the parameter consentStatus to "status": I believe that consentStatus conveys the intended meaning better. Let's stick with consentStatus to ensure clarity and consistency throughout the codebase.

I noticed some formatting issues in your IDE that are introducing unnecessary changes. It would be great if you could resolve these formatting inconsistencies.

The validation of pageSize and pageNumber being less than or equal to 100 may not make sense in all scenarios. For instance, if there are more than 10k rows in the database, the user won't be able to list them all anyway. I suggest modifying the validation to only enforce that the parameters are greater than 0 instead.

The validate methods seem to have more arguments than necessary and are performing additional tasks, such as logging errors. It's best to keep the responsibilities of these methods focused and concise. They should only handle the necessary validations and not handle logging errors. Let's refactor the validate methods to adhere to the Single Responsibility Principle and keep them as lean as possible.

Please address these points and let me know if you have any questions or need further clarification.


v8.0.0 fails because of missing postinstall script

Are you submitting a bug report or a feature request?
bug report
What is the current behavior?

v8.0.0 does not install because scripts/patch-immutable-flow-error.js is not part of files.
Other information

node_modules/redux-form: Command failed.
Exit code: 1
Command: node scripts/patch-immutable-flow-error.js && opencollective-postinstall

Error: Cannot find module '/path/to/project/node_modules/redux-form/scripts/patch-immutable-flow-error.js

Maybe it's not even needed, previously there were two postinstall properties in package.json. The second one probably overwrote the first one:
https://github.com/erikras/redux-form/commit/e46a2b8566d1bfce4416d00668517c8d929daa15

Fixed in v8.0.1, thanks!


Loaded scripts should have a collapse all action

Refs: #64230
As other debug views that are trees the loaded scripts view should have a Collapse All action in the top of the view.
Here's a code pointer to the action https://github.com/Microsoft/vscode/blob/07202fbfb82bdada89472dd4b1c3c699b0eab0b4/src/vs/workbench/browser/viewlet.ts#L188

I'd like to give this a shot 🎉


Enabling SBOM across repositories

This issue is to capture discussions happening in various SIGs around creating a software bill of materials.

https://github.com/open-telemetry/opentelemetry-java/pull/5948

There seems to be at least 3 SBOM formats. We probably should decide on which one we want to use first.

Just for the record, I used syft and cyclonedx-py to generate SBOMs for the OTel Python opentelemetry-sdk package in SPDX and CycloneDX formats.

SWIDs claim to be an SBOM format but are more of an identifier format, but also not really a great solution there. Realistically the decision really is between SPDX and CycloneDX, though they are mostly interchangeable unless you have a very specific use case. My perspective comes from CNCF TAG Security, work with OpenSSF and work with SPDX community.
If folks are interested I can pull in experts to discuss with your community.

I am interested :v:, which is this use case you mention above, @mlieberman85?

If folks are interested I can pull in experts to discuss with your community.

This would be great @mlieberman85!

SWIDs claim to be an SBOM format but are more of an identifier format, but also not really a great solution there. Realistically the decision really is between SPDX and CycloneDX, though they are mostly interchangeable unless you have a very specific use case. My perspective comes from CNCF TAG Security, work with OpenSSF and work with SPDX community.
If folks are interested I can pull in experts to discuss with your community.

I am interested ✌, which is this use case you mention above, @mlieberman85?

So the use cases where they're not right now are mostly focused on specific fields that might be in one but not the other. For example there's AI/ML model specific bill of materials fields in CycloneDX released already, and a similar SPDX feature is still being developed.
Let me reach out to some of the folks in the community and tag them in this thread.

Adding @puerco who is an expert in the community.

Hello happy to help out! Do you have a tracker of projects that need to build their SBOMs?


[boost-locale] Fix boost-locale failing to build with ICU

Get boost-locale to link with ICU library

fixes #3537 by modifying and apply a patch that already existed (and mentioned in #3537): 0001-Fix-boost-ICU-support.patch

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

Hi @Victor-Bergman
Thanks for this PR.
Could you please also bump the version  as  1.72.0-1 in CONTROL file?

After making the 2 changes from the review (remove include and move PATCHES), the tests on linux and OSX fail (above).  These were passing last night and it makes no sense that these 2 changes should cause them to fail.

/azp run

/azp run

/azp run

/azp run

Need to test feature.

/azp run

Feature has passed on the following triplets:

x86-windows
x64-windows
x64-windows-static
x64-linux

@Victor-Bergman
Could you please help confirm if this can work on static build? Or is there something that needs to be configured separately?

@Victor-Bergman
Could you please resolve the conflicts?

@Victor-Bergman
Is work still being done for this PR?

Can confirm that the PR is working and fixing the icu build for boost-locale. Thanks!

The baseline failures should be addressed by https://github.com/microsoft/vcpkg/pull/12067

/azp run

Done, @BillyONeal  @vicroms, could you please help merge this PR?


Lens 3.6.x seems to lose connection and present errors, requiring manual reconnects

Since moving to the 3.6.x release, whenever I switch back to Lens after a while, I'll have a screen full of errors like this:

The prior version seemed to be able to gracefully handle connection issues.
I can click Reconnect and it will restore whatever I was on before.
To Reproduce
Steps to reproduce the behavior:

Be connected for awhile
Maybe simulate some kind of intermittent connection loss (although I'm not seeing anything specific like Wifi drop or anything like that). Other applications with persistent connections such as SQL tools seem fine.

Expected behavior
Ideally, it would be able to tolerate a small connection blip without user intervention.
Environment (please complete the following information):

Lens Version: 3.6.5
OS: 10.14.6

Additional context
We are using OIDC authentication, which has a key that expires every few hours and must be refreshed. Lens 3.6.0 and later seems to be able to handle that now (the old version did not). It's possible that is contributing to the issues when the key expires, it loses the Watch and fails until a refresh token is sent and the key is updated.

More evidence to support that it fails when the access token expires:

Is this this a problem with 4.1.4?


Display all network folders on the sidebar

What's the Problem?
Network drives folder
Solution/Idea
List of available network drives (PC's) instead of folder with all of them.
Now:

Should be:

Alternatives
.
Priorities

Capability
Priority

This proposal will allow developers to accomplish W
Must

Files Version
No response
Windows Version
No response
Comments
No response

Same as #8287

no it's not. I have access to the drives

from your issue it seemed you couldnt access the network computer shown in file explorer witch is the same as the title of the other one

Network Explorer can't access to other computers

from your issue it seemed you couldnt access the network computer shown in file explorer witch is the same as the title of the other one. although there issue is not very desiptive.

Network Explorer can't access to other computers

Read carefully. Devices from the network are nested in the "Network" folder, and are not immediately displayed in the menu

Asking @yaichenbaum for approval.

Same issue

My report was inappropriately merged with this issue. My Network Drives panel option shows the network folder, but nothing in it. In Windows Explorer, I can see all of the network devices. In Files App, I see nothing but an empty folder. This ticket seems to be complaining that they have to open the Network folder inside the Network item in the panel. That's not my case at all. Whoever is managing these requests and closing them is not addressing the issue here.

The network folder scan for all items on the network but can be slow (There's a loading icon for this in the tab bar). Thats a limitation of Windows/File explorer.
If you see the main issue screenshot you can see this issue asks to show devices on the sidebar as well, which is what you are requesting.

Do you have any update on this feature request

As far as I am concerned. feel free to close this ticket. After multiple
occurrences of my computer literally grinding to a halt while trying to
save or open files, I realized the one common denominator was Files. I
loved how your app looked, but it was impossibly slow in my configuration.
So slow, in fact, that I was able to reboot faster than it would have taken
to get a directory listing in Files from Office or any of the Adobe apps.
I wish you the best of luck, but it's just not there for me at this time.
 I
have about 8 drives and half a dozen Operating Sysems installed so Windows
never knows where anything is. So t's not your fault, it's just not
compatible with my mess.
On Wed, May 10, 2023 at 12:12 PM Lukiluc29 @.***> wrote:

Do you have any update on this feature request
—
Reply to this email directly, view it on GitHub
https://github.com/files-community/Files/issues/8304#issuecomment-1542475009,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFYP4M4TA3C5XMCXFUZHJDXFO45FANCNFSM5OEFLW3A
.
You are receiving this because you commented.Message ID:
@.***>

Do you have any update on this feature request

Thanks for reaching out, I added this to the project board.

I'm working on this.


Minor fix in documentation for uv

Should be:
cd $NOBACKUP
mkdir swell-project
cd swell-project
git clone git@github.com:geos-esm/swell swell-develop

Can be fixed easily with another PR
also add checkout branch option
thanks @mjkagnes123 for testing

Worked! Thanks.


updater: Support MariaDB and Sirenia upgrade properly

Closes #2580

@titanous @benbjohnson we also need to handle updates for Redis as they aren't being done atm either.

LGTM


fix (#1200): pin python requirements.txt versions

closes #1200
Using pip-compile a requirements.txt file was generated which pins all the required versions to execute the e2e tests successfully.
Basics

[x] The PR is rebased with current master
[x] I added a line to changelog.md
[x] Details of what I changed are in the commit messages
[x] References to issues, e.g. close #X, are in the commit messages and changelog
[ ] The buildserver is happy

Checklist

[ ] I fully described what my PR does in the documentation
[ ] I fixed all affected documentation
[ ] I fixed the introduction tour
[ ] I wrote migrations in a way that they are compatible with already present data
[ ] I fixed all affected decisions
[ ] I added automated tests or a manual test protocol
[ ] I added code comments, logging, and assertions as appropriate
[ ] I translated all strings visible to the user
[ ] I mentioned every code or binary not directly written or done by me in reuse syntax
[ ] I created left-over issues for things that are still to be done
[ ] Code is conforming to our Architecture
[ ] Code is conforming to our Guidelines
[ ] Code is consistent to our Design Decisions
[ ] Exceptions to any guidelines are documented

First Time Checklist

[ ] I have installed and I am using pre-commit hooks
[ ] I am using Tailwind CSS Linting

Review

[ ] I've tested the code
[ ] I've read through the whole code
[ ] I've read through the whole documentation
[ ] I've checked conformity to guidelines
[ ] I've checked conformity to requirements
[ ] I've checked that the requirements are tested

I don't have much experience with Python, but LGTM

LGTM. Maybe we don't need e2e/requirements.txt then?

Good idea, I dont know. I feel like everyone is keeping both.


Adding Environnement Variable to setup the path for the temporary encoding

Hello there,
Here's my little contribution to this project.
My output drive is a RAID-5 full of HDD and I would like to setup the temporary encoding to an SSD mounted elsewhere, so I dig down a little and purpose this pull request.
Feels free to comment and share your opinions ! I'd appreciated that.
(P.S : sry for the approximative english ^^)

Related to #320

Thanks for this, but I don't know if it's worth a new setting. There are already a lot of them... I guess the same thing can be achieved this way:

Set the output folder to be on your SSD.
Once the encode is done, move the file to its final location. This can be done by using hooks.


Update Package To Be Side Effects Free

Link to relevant issue
This pull request resolves #253
Description of changes
Updates the way that design tokens and VS Code theme listening/application is accomplished so the toolkit no longer has any side effects.
Also, add sideEffects: false to package.json.

@connor4312 Adding you as an optional PR reviewer since you've been a lot of help in the implementation of this PR and it would be great to get any final feedback from you on this.
But also no pressure if you don't have the time or don't want to.


[question] possibly open call activity directly when receiving new call?

Hello @mamzii thank you for your question. Yes, you can do it. As a possible solution you can create service which will save chat connection (like ChatConnectionManager.kt). Then after receive call push-notification you will start this service and initialize RTCSessionManager. RTCSessionManager starts listen calls and after receive new call will open CallActivity.kt

feel free to open other questions if you have


[Question] within zeno.ipynb, how did you arrive at optimal hyper parameters

I saw that you are passing these hyper parameters for your neural networks . Can you please explain on how you arrived at it and how do you know that it is optimal.
TEXT = data.Field(lower=True, tokenize=list)
bs=64; bptt=8; n_fac=42; n_hidden=128

TEXT3 = data.Field(lower=True, tokenize=list)
bs=64; bptt=8; n_fac=42; n_hidden2=512

Is there any methodology like Bayesian optimization,Grid Search,Genetic algos?

Hi, sure! I'm quite novice so I'm afraid I can't provide much in the way of methodology.

bs=64: this was used during part 7 of the fast.ai course, and seemed to work well enough for Zenobot. I might try bs=1 for prediction as I hear that works well
bptt=8: again, suggested by fast.ai
n_fac=42: same reason
n_hidden=512: once my dataset got larger, I found that increasing the hidden layers improved the model


[Request]: Add shadPS4

Prerequisites

[X] I have searched all issues/PRs to ensure it has not already been reported or fixed.

Criteria

[X] Reasonably well-known and widely used
[X] A game or a game-related tool
[X] English interface (or at least English documentation)
[X] Latest stable or development version
[ ] Full version (i.e. not a trial version)
[ ] Fairly standard install (e.g. uses a version-specific download URL, no elaborate pre/post install scripts)

Name
shadPS4
Description
PS4 emulator for Windows,Linux,MacOS
Homepage
https://shadps4.net/
Download Link(s)
https://github.com/shadps4-emu/shadPS4/releases
Some Indication of Popularity/Repute
10k github stars

Feel free to open a pull request for this 🙂

Hey, pls take a look.
https://github.com/Calinou/scoop-games/pull/1253


Feature request: stop without unlocking the wallet

The "stop" command currently requires the wallet to be unlocked. This is inconvenient when using scripts to start/stop LND, when those scripts don't have knowledge of the wallet passphrase and can't assume the wallet will always be unlocked.
I'd say that a script that has access to the admin macaroon is already sufficiently authorized to stop LND. Why require the wallet passphrase to stop the software?
Maybe this is not so important if LND does a proper shutdown when it receives SIGTERM. In that case, please document this behavior or point to the documentation.

Can this be closed now that SIGTERM is handled?

Not yet, I believe the StopDaemon RPC would also need to be added to the WalletUnlocker service.

closing this issue, as unsure about the need for this feature. If LND is locked, it can be killed without any adverse impact, which the automated scripts should be able to handle in their logic.


Build failing

Hi,
I tried connecting the camera to the USB port of my Linux machine and tried steps to build, but the build failed, and I was unable to create and load the project in the QT creator and see the output image. Please, someone, help me with the detailed steps of connecting this camera and viewing output. I was not able to understand the process from the steps. Are the steps mentioned in the readme correct, or are there any further commands required to build and visualize the output?
While building in the GitHub readme, it is mentioned below:
GetThermal is a QT qmake project. So, all you should have to do now is to fire up QTCreator, open the project file, and hit run.
I have installed qt creator on a Linux machine; which project file do I need to open and run? Can someone be brief about it to get the getthermal for visualizing the thermal camera data?
Can someone help me with this? Thanks in advance.

Hi Sandeep,
the project file used by qmake /  QT Creator is GetThermal.pro. Another issue you might be facing is that the latest Qt version (6.7) is not able to compile this Qt 5.7+ Project as it is configured for 5.7+ Qt version.

After you have the qmake / run project, another error you might also be facing after building is:
: -1 : error: cannot find -lGL: No such file or directory
I found out this happens because the necessary development package for OpenGL is missing, assuming you're on Debian/Ubuntu you can just run

sudo apt-get install libgl1-mesa-dev
to fix it :)


Use template plugin and evade local sudo (v2020)

Re-applying changes from 'lost' PR #28
( More infos about how that happenend: https://github.com/cloudera/cloudera-playbook/issues/68 )

merged - thanks @lhoss


Namespaced Artisan module commands create broken Classes

When you for example run php artisan module:make-controller "Admin\HomeController" ModuleName
The result is a class like this:

class Admin/DashboardController extends Controller
{
}

I'll close this due to inactivity.


Can't get the convos to work

Here's my config, I'm on minecraft 1.21.4 and mod version 1.1
{
  "cloudflareApiKey": "<Key>",
  "cloudflareAccountId": "<Id>",
  "personalities": [],
  "nameCandidates": [],
  "modelName": "@cf/meta/llama-3-8b-instruct",
  "conclusionMessageLimit": 10,
  "villagerChatPrefix": "$",
  "useExistingVillagerName": true,
  "showTimeConsumption": false,
  "apiConfigPlatform": "cloudflare",
  "timeoutSeconds": 10
}

Do not expose your account id and api key to public.
I'll handle it ASAP.

You didn't fill personalities and nameCandidates. Please fill name.
If still not working, please upload game log.

Oh shoot, I didn’t even think about it. Thank you so much for noticing and taking care of it!
I populated the name and personality fields at first, but it threw an error and said “unexpected character” whenever I typed in between the brackets. I’ll try again and see how that goes.

If still errors, upload your config file as an attachment.
Don't forget to remove private credentials :)


Create a new release for the guzzle6 branch

Is there anything left to be done before a tagged release of the guzzle6 branch can be created?

The guzzle6 branch of Guzzle Command need to be merged into master and a release has to be tagged first before Guzzle Service can follow. I am working at this topic.

OK, so waiting on : https://github.com/guzzle/command/issues/35

Done!


Fix for various issues with cache on AppVeyor

This PR makes use of a cache-s3 tool in order to handle stack build artifacts and cache the them on S3 bucket for future builds. Fixes long builds and DEVOPS-565
Here is what needs to be done in order for the tool to work properly:

Set up IAM user with no permissions and only API access (i.e. with key and secret) through the AWS console
S3 bucket created with above user having full access to that bucket (terraform can be used in order to get that done quickly: terraform snippet
Environment variables set in AppVeyor:

S3_BUCKET
AWS_REGION
AWS_ACCESS_KEY_ID
AWS_SECRET_ACCESS_KEY
CACHE_S3_VERSION=v0.1.0

Flag in AppVeyor general setting checked: Enable secure variables in Pull Requests from the same repository only

Example AppVeyor builds: https://ci.appveyor.com/project/lehins/cardano-sl/history
@domenkozar Please, let me know if you need any help with setting up any of the above resource. Also I'll be glad to answer any questions regarding the tool itself. There is a pretty decent cache-s3/README and also each command has a full featured --help flag, so you can try using it locally before deploying it on CI.

Superseeded by https://github.com/input-output-hk/cardano-sl/pull/2166


Support PostgreSQL datastore

Pull Request Checklist

Description

Add support for PostgreSQL datastore.
Related Issue

Fixes #1135
Motivation and Context

Sensu Go 5.10 added support for PostgreSQL datastore that has to be setup with sensuctl. This adds support for that as well as optionally manages the actual postgresql database.

This should also have a manifest for vagrant so it can easily be consumed there.

We can't use Vagrant because postgresql support requires a license, so for someone to use it in Vagrant would require they supply a license file.

Do you want me to make it so an environment variable can define the path to license file or something? We don't currently have any enterprise licensed support in the vagrant images.


v2 API Proposal Document

Hello 👋
I am Aryan, a Computer Science student and currently an intern at Apple on the Swift on Server team. As part of my internship, I have been working with @fabianfett to propose a new v2 API for the swift-aws-lambda-runtime library.
swift-aws-lambda-runtime is an important library for the Swift on Server ecosystem. The initial API was written before async/await was introduced to Swift. When async/await was introduced, shims were added to bridge between the underlying SwiftNIO EventLoop interfaces and async/await.
However, just like gRPC-swift and postgres-nio, we now want to shift to solely using async/await constructs instead of EventLoop interfaces. For this, large parts of the current API have to be reconsidered. This also provides a good opportunity to add support for new AWS Lambda features such as response streaming.
We have written a document that explains the current limitations of the library and proposes a detailed design for the v2 API.
Please read the proposal and voice your opinions either here or on the forum post. We are looking forward to your feedback!
cc: @sebsto @tachyonics @FranzBusch @Lukasa @tomerd @weissi @adam-fowler @ktoso @0xTim

Thank you @aryan-25 ! I'll have a look this morning

This is not related to this doc, but should we move the Documentation.docc directory just underneath Sources ?
@fabianfett wdyt ?
Current struct :
Sources/AWSLambdaRuntimeCore
└── Documentation.docc
    ├── Resources
    │   ├── code
    │   └── tutorials
    └── tutorials

Proposed struct
Sources/Documentation.docc
├── Resources
│   ├── code
│   └── tutorials
└── tutorials

I don't have further input other than what was already mentioned

@aryan-25 Thanks so much for continuing to push this forward! 🙏🚀

The discussion here has settled. @aryan-25 addressed most of the feedback. From my point the only open issue concerns the use of DispatchWallTime in LambdaContext. I've created an extra ticket for this #384.
Last call to all participants: I intend to merge this PR on Monday. Is there any other unaddressed feedback besides #384, that we should create an extra ticket for?


Update tools/stresser

#335 Updated the packet API, but tools/stresser has not yet been updated in response.
While we're at it, the stresser is missing some features. The PR that added the tool was merged a bit too hastily and some essential functionality was left out. In particular, the fake clients just stand at the spawn point without moving.
Ideally, the tool should behave almost the same as rust-mc-bot with clients running around and flailing their arms. See this.

I think this should be split into 2 separate issues.

I think this should be split into 2 separate issues.

The first issue could be called like "Upstream stresser tool", and the second like "Stresser bots random actions".

I think this should be split into 2 separate issues.

Done.


Expose CheckClusterIngressHealthy to the pr_test.go

Description
This PR exposes CheckClusterIngressHealthy to the pr_test.go to know the ingress state of the cluster.
https://github.ibm.com/GoldenEye/issues/issues/10545

Release required?

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content
Addition of getClusterIngress method to expose the ingress state of the cluster

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

/run pipeline

TestRunRoksPattern is getting skipped and PostApplyHook doesn't initiate

I ran TestRunRoksPattern in local after commenting out enableSchematicsTests and I found that this example creates more than two cluster in logs.
TestRunRoksPattern 2024-09-16T13:54:54+05:30 logger.go:66:   "cluster_names": {
TestRunRoksPattern 2024-09-16T13:54:54+05:30 logger.go:66:     "sensitive": false,
TestRunRoksPattern 2024-09-16T13:54:54+05:30 logger.go:66:     "type": [
TestRunRoksPattern 2024-09-16T13:54:54+05:30 logger.go:66:       "tuple",
TestRunRoksPattern 2024-09-16T13:54:54+05:30 logger.go:66:       [
TestRunRoksPattern 2024-09-16T13:54:54+05:30 logger.go:66:         "string",
TestRunRoksPattern 2024-09-16T13:54:54+05:30 logger.go:66:         "string"
TestRunRoksPattern 2024-09-16T13:54:54+05:30 logger.go:66:       ]
TestRunRoksPattern 2024-09-16T13:54:54+05:30 logger.go:66:     ],
TestRunRoksPattern 2024-09-16T13:54:54+05:30 logger.go:66:     "value": [
TestRunRoksPattern 2024-09-16T13:54:54+05:30 logger.go:66:       "ocp-ioblal-management-cluster",
TestRunRoksPattern 2024-09-16T13:54:54+05:30 logger.go:66:       "ocp-ioblal-workload-cluster"
TestRunRoksPattern 2024-09-16T13:54:54+05:30 logger.go:66:     ]
TestRunRoksPattern 2024-09-16T13:54:54+05:30 logger.go:66:   },

This module utilizes schematics to run the tests and support for post apply hook is not supported in schematics. I will create an issue to track the addition of support of post apply hook in schematics, after which CheckClusterIngressHealthy can be exposed in the landing zone

Closing this PR as postapply hook feature is yet to be supported in schematics workspace. I will be creating an issue to support postapply hook in schematics post which support for CheckClusterIngressHealthy can be added.

Issue to support postapply hook in schematics: https://github.ibm.com/GoldenEye/issues/issues/11287


downloadFile error: Cannot read property 'toString' of null

Describe the bug
Hi! First off, love the work ya'll put into this library, don't know what to do without it :).
I've had some issues with the 'downloadFile' function, more like inconsistencies, as it works great for some files but not for others. For instance, I keep receiving:
UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'toString' of null
every time I try to download CSGO's pak01_4.vpk or pak01_127.vpk. Note, the download starts successfully, it's only after a little while that the error is thrown. And it works perfectly for any other .vpk file. Might be an issue on my end, but I made a simple scenario where it keeps failing after around 10-15% of the total download. I'll attach an excerpt of code that throws:
        async update() {
		const manifestId = await this.getLatestManifestId();
		const [manifest] = await this.user.getManifestAsync(730, 731, manifestId);
		const manifestFiles = manifest.files;
		const fileName = 'pak01_127.vpk';
		const file = manifestFiles.find((f) => f.filename.endsWith(fileName));
		const filePath = `${this.config.dataDirectory}/${fileName}`;

		await this.user.downloadFile(730, 731, file, filePath,
			// Had problems with destructering the second argument as it randomly returned undefined
			// FIXME: SteamCrypto.symmetricDecrypt or unzip returns null which causes the error
			(err, event) => {
				if (event && event.type === 'progress') {
					console.log('progress...');
				}
			});
	}

this.user is a steam-user instance. this.config is a simple config object.
I tried looking into underlying libraries such as stdlib\components\hashing\basic_hash.js and stdlib\components\hashing.js, however, I'm not that familiar with JavaScript and this library.
Versions
What version of the module and what version of Node.js are you using?
steam-user: 4.17.1
Node.js: v.12.18.3
npm: 6.14.6
Screenshots and Error Logs
If applicable, add screenshots and error logs to help explain your problem.

Thanks for the report. This is fixed in v4.17.2.


Broken link in blogpost: 2024/05/22/swe-bench-lite.html

Issue
FYI, noticed that this blog: https://aider.chat/2024/05/22/swe-bench-lite.html
Has a broken link in the Acknowledgements: https://aider.chat/2024/05/22/SWE-bench-docker
Version and model info
N/A

Thanks!

Np :)


Remove read timeout form http endpoint since config sanitization fails

if read_timeout is set benthos wont start. Unrecognized option.
INFO Running main config from specified file       @service=benthos path=./config.yaml
ERRO Config lint error                             @service=benthos lint="./config.yaml(5,1) field read_timeout not recognised"
ERRO Shutting down due to linter errors, to prevent shutdown run Benthos with --chilled  @service=benthos

Can we get this PR merged: @difabion
This helm chart is unusable for now because if we disable HTTP, livnessprobe/readnessprobe will fail, and if enable it read_timeout config will fail.

Hey, sorry for the misconfiguration and thanks for the PR.  I removed it in 0.7.2.  Please try it out and let me know if this works for you.

Yeah, this works. Thanks.