Comparison format

Signed-off-by: Raul Sevilla<EMAIL_ADDRESS>Description
In some cases we don't need to generate a CSV/google spreadsheet and just output the fancy benchmark-comparison output

/rerun all

Results for e2e-benchmarking CI Tests

Workload
Test
Result
Runtime

router-perf-v2
./ingress-performance.sh
FAIL
00:05:34

etcd-perf
./run_etcd_tests_fromgit.sh
PASS
00:02:25

scale-perf
./run_scale_fromgit.sh
PASS
00:13:32

network-perf
./run.sh
PASS
00:08:05

kube-burner
WORKLOAD=cluster-density ./run.sh
PASS
00:05:14

kube-burner
WORKLOAD=node-density ./run.sh
PASS
00:05:31

kube-burner
WORKLOAD=node-density-heavy ./run.sh
PASS
00:05:59

kube-burner
WORKLOAD=pod-density-heavy ./run.sh
PASS
00:06:01

kube-burner
WORKLOAD=max-services ./run.sh
PASS
00:05:04

kube-burner
WORKLOAD=max-namespaces ./run.sh
FAIL
00:04:06

upgrade-perf
./run_upgrade_fromgit.sh
PASS
00:01:01

/rerun all

Results for e2e-benchmarking CI Tests

Workload
Test
Result
Runtime

router-perf-v2
./ingress-performance.sh
PASS
00:05:39

etcd-perf
./run_etcd_tests_fromgit.sh
PASS
00:02:25

scale-perf
./run_scale_fromgit.sh
PASS
00:14:08

network-perf
./run.sh
PASS
00:07:19

kube-burner
WORKLOAD=cluster-density ./run.sh
PASS
00:05:10

kube-burner
WORKLOAD=node-density ./run.sh
FAIL
00:27:31

kube-burner
WORKLOAD=node-density-heavy ./run.sh
PASS
01:04:48

kube-burner
WORKLOAD=pod-density-heavy ./run.sh
PASS
00:05:51

kube-burner
WORKLOAD=max-services ./run.sh
PASS
00:05:03

kube-burner
WORKLOAD=max-namespaces ./run.sh
PASS
00:06:20

upgrade-perf
./run_upgrade_fromgit.sh
PASS
00:01:00

/rerun all

Results for e2e-benchmarking CI Tests

Workload
Test
Result
Runtime

router-perf-v2
./ingress-performance.sh
PASS
00:06:01

etcd-perf
./run_etcd_tests_fromgit.sh
PASS
00:02:30

scale-perf
./run_scale_fromgit.sh
PASS
00:14:57

network-perf
COMPARISON_FORMAT="" ./run.sh
PASS
00:06:46

kube-burner
WORKLOAD=cluster-density ./run.sh
PASS
00:04:49

kube-burner
WORKLOAD=node-density ./run.sh
PASS
00:04:55

kube-burner
WORKLOAD=node-density-heavy ./run.sh
PASS
00:06:01

kube-burner
WORKLOAD=pod-density-heavy ./run.sh
PASS
00:06:01

kube-burner
WORKLOAD=max-services ./run.sh
PASS
00:04:39

kube-burner
WORKLOAD=max-namespaces ./run.sh
PASS
00:05:39

upgrade-perf
./run_upgrade_fromgit.sh
PASS
00:01:00


Add drf-sendables

Project Information

Project Name: drf-sendables

Project URL: https://github.com/amikrop/drf-sendables

Description: User messages for Django REST Framework

Criteria
Please answer the following questions about the project you are submitting. This will help us evaluate if the project should be included in the Awesome Django list.

Is the project new?

[x] Yes
[ ] No

How long has the project been maintained? 2 months

How many releases has it had if it's a library or package? https://pypi.org/project/drf-sendables/#history

Are you the author or are you submitting the project on behalf of a company?

[x] I am the author
[ ] I am submitting on behalf of a company
[ ] Other (please specify)

What makes it awesome? Its simplicity, yet high configurability. Also, how you can install several different instances of the app under different URLs, each having their own models and configuration.

Thank you for submitting drf-sendables to us. The project looks very promising. Overall, the project is too new and doesn't quite have a long enough release record for us to feel comfortable including it. We are happy to re-evaluate it at a later date.


feat(Keys): Constrain the encryption in broadcast

The individual encryption components #5867, #5899 and #5900 is to be tied together such that they can be used like the oracle for the fix of #5901. Should be possible to simply "plug-in" when those 4 are merged.

This is a duplicate of #1139, with a bit more context on components.


Add better error message when WebGL is not enabled

When I go to geojson.io, the message say my browser is out of date, which is weird given I run Firefox 105.0.2, and while 106.0 is out since 3 days, the real issue is that I disabled WebGL a long time ago. So I propose to change the error message to give a clue in case someone did like me (either disabled due to issue with driver stability, or disabled to avoid fingerprinting as it was done on some version of the tor browser)

Thank you for this contribution!


add deprecation warning when argument passed to --with contains hyphens (-)

in #164 we no longer allow hyphens passed to --with, instead, users should use underscores.
to give them time to update their code, we should throw a FutureWarning whenever they pass an argument that contains hyphens and tell them to use underscores instead

cc @tonykploomber

Is this for all places where the with parameter is used? In SqlPlotMagic, RenderMagic, and SqlMagic.

they all use the same underlying mechanism so it's better to add it there https://github.com/ploomber/jupysql/blob/master/src/sql/store.py

@Palashio
For the Futurewarning, you can refer how we did to the var expand
Code: https://github.com/ploomber/jupysql/pull/137/files#diff-769097f5e36be57ddec10c78df0900db8c1e254f5485bc94367aa7fcb07991c5R114
Test: https://github.com/ploomber/jupysql/pull/137/files#diff-9aed7fd5fd841447ef03fafe06253743be39cb1363c953da27c5e04350191794R173

@edublancas
Gotcha, since we are giving user some time to update their code.
I guess the timeline will be like this:
Merge #245 -> Wait for 1 month -> Merge #164 ?
Also, in #164, we should show some error message when the argument passed to --with contains hyphens (-)

Merge https://github.com/ploomber/jupysql/issues/245 -> Wait for 1 month -> Merge https://github.com/ploomber/jupysql/pull/164 ?

as soon as #245 if fixed, I'll make a release. Then we merge #164 but wait for two weeks to make a release

Also, in https://github.com/ploomber/jupysql/pull/164, we should show some error message when the argument passed to --with contains hyphens (-)

yes, please update it so we throw an error, ensure that the error suggests how to fix it (by replacing - with _)


Fixed #32131 -- Also check in debug mode if static file exists

The behaviour of ManifestStaticFilesStorage differs between debug and
non-debug mode when a referenced static file does not actually exist (or an
absolute path is used instead of a relative one).
The following code demonstrates the issue:
from django.contrib.staticfiles.storage import ManifestStaticFilesStorage
>>> ManifestStaticFilesStorage().url("path/to/a/file.png")
'/static/path/to/a/file.png'
>>> ManifestStaticFilesStorage().url("/path/to/a/file.png")
'/static/path/to/a/file.png'
>>> ManifestStaticFilesStorage().url("path/to/a/file.png", force=True)
'/static/path/to/a/file.f208556b8835.png'
>>> ManifestStaticFilesStorage().url("/path/to/a/file.png", force=True)
Traceback (most recent call last):
  File "<input>", line 1, in <module>
    ManifestStaticFilesStorage().url('/path/to/a/file.png', force=True)
  File "/app/.pyenv/versions/3.7.2/lib/python3.7/site-packages/django/contrib/staticfiles/storage.py", line 153, in url
    return self._url(self.stored_name, name, force)
  File "/app/.pyenv/versions/3.7.2/lib/python3.7/site-packages/django/contrib/staticfiles/storage.py", line 132, in _url
    hashed_name = hashed_name_func(*args)
  File "/app/.pyenv/versions/3.7.2/lib/python3.7/site-packages/django/contrib/staticfiles/storage.py", line 420, in stored_name
    raise ValueError("Missing staticfiles manifest entry for '%s'" % clean_name)
ValueError: Missing staticfiles manifest entry for '/path/to/a/file.png'

The root cause for the issue is that when the staticfiles are hashed Django puts
them into the manifest file ("staticfiles.json") with a relative path (i.e.
without a leading slash) and when trying to find the file with leading slash
it fails because it does not exist. When getting the URL in debug mode it just
joins the path to the STATIC_URL (in our case /static) and we get the valid
url /static/path/to/a/file.png so no error is being raised.
This can cause trouble when all test systems are using debug = True and an
incorrect static file path is only detected after deployment to the production
instance that uses debug = False.
In order to fix the problem we should also check in debug mode whether the
referenced file really exists.
https://code.djangoproject.com/ticket/32131#ticket

Closing as per ticket. (Would need tests and docs.)


CRUD Operations to Cart Module

Modules

[x] #2
[x] #3
[ ] #4

Dynamic Modules with Database

[ ] Add Product to Cart
[ ] Remove Products From the Cart
[ ] Update Product Quantity in the Cart


Ensure viper binds to env vars in mindev

Summary
The explicit viper binding for environment variables was missing.
Change Type
Mark the type of change your PR introduces:

[x] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 39.348%. remained the same
when pulling 48c4277a5bada2526a501f9e419a9a89bc39c102 on `lint-auth-token
into 009bc5654d336957e3cfd6c4857823169769799d on main.


DTSPO-13832 - add check for java 17 in dbMigrate

dbMigrate stage uses default java installation on agent which is still version 11. This is causing an issue on draft-store as some recently updated dependencies require java 17.

Is this more like it? There's not an exact parallel with the camunda pipeline in withPipeline so took the cue from sectionBuildAndTest


question about attention layer

HI！
Acccording to paper,Attention only calculated between node's neighboor node.I am condused when i see code about "attention_layer", it is calculated on all nodes not only neighboor node?

Hello,
Thank you for the issue and your interest in GAT!
You are correct that the coefficients are calculated for every node. However, before applying the softmax function, we add the bias_mat, setting all non-edge pairs to "negative infinity" (-1e9). This forces the softmax to assign them a weight of zero, effectively discarding them.
Hope this helps!
Thanks,
Petar

Thank you!


Making the navbar and testimonials dark mode compatible

@abhay-raj19 Can you assign the issue under GSSoC'24

Hello, I am a full stack developer with the required tech stack, kindly assign me with this issue  so that I can provide my insights and solutions.

Hello, I am a full stack developer with the required tech stack, kindly assign me with this issue so that I can provide my insights and solutions.

This issue have been completed @Manaswini123456 you can think of a new one 🙂.

Will be happy to assign You @Manaswini123456


HVM doesn't use normal order reduction.

Consider the HVM program below. What do you expect the output to be?
(Foo Bar) = A
(Foo B) = B
Bar = B

Main = (Foo Bar)

I expected the output to be A, but HVM outputs B.
This is a problem because it means HVM isn't using normal order reduction.
That means that there exist programs that can terminate but that will fail to terminate on HVM.
For example, if we add the rule B = Bar to the program above it will never terminate on HVM:
(Foo Bar) = A
(Foo B) = B
Bar = B
B = Bar

Main = (Foo Bar)

If possible, HVM should be revised to use normal order reduction.
In normal order reduction the outermost redex (reducible expression) is always reduced first.
In this case that's (Foo Bar), which readily reduces to A.

That's not right! HVM does use normal order reduction. When you pattern-match an argument, HVM must reduce strict arguments to weak-head normal form in order to pattern-match, just like in Haskell. The issue is that, in your program, Bar isn't a constructor, it is a definition that evaluates to B, which itself evaluates to Bar, forming a loop. As such, if your program depends on reading the value of Bar, which is the case for Foo Bar, it will fail to halt, rightfully so. The same issue would happen on Haskell:
data Foo = A deriving Show

foo :: Foo -> Int
foo A = 0

a :: Foo
a = b

b :: Foo
b = a

main :: IO ()
main = do
  print (foo a)

Thanks for the reply! Please consider the below HVM program:
(Foo (Bar 1)) = (A 1)
(Foo (B 1)) = (B 1)
(Bar 1) = (B 1)

Main = (Foo (Bar 1))

It's a constructor-only version of the original program I posted. As far as I can tell it has the same problem. HVM reduces (Foo (Bar 1)) to (B 1) but in normal order reduction it should reduce to (A 1), no? What am I missing?

Normal order evaluation doesn't mean you don't reduce anything; it means you reduce something when it is needed, which is the case here, since Foo pattern-matches on its first argument. As such, (Foo (Bar 1)) will trigger a reduction of (Bar 1) to weak-head normal form, so it turns into (Foo (B 1)), which evaluates to (A 1). Note that while this looks like a "constructor-only" version, it isn't, because you use Bar as both a constructor (in (Foo (Bar 1)) = (A 1)) and as a function ((Bar 1) = (B 1)). This isn't legal in Haskell (you can't make constructors compute), which is why we couldn't replicate it there, but the situation is exactly the same. HVM allows it because there is no need to forbid, but it does have the weird behavior you're seeing.


Degraded web3torrent performance

I think that the incentive algorithm is broken, leading to wires dropping, too many channels, and slow download speeds.

Another theory: the wallet hits a wall making some payment, the buffer runs out, an the wire gets dropped.


[research] Add error logging to the frontend

first for smart contract interactions
then for next api endpoints
then for other core user journeys
maybe a top level catch-all for the whole site / app (if it's not too noisy)

I am aware of bugsnag, sentry.io... what is the industry standard?

must be lightweight, perhaps even wrapped in a next /api/ endpoint so that we don't add anything to our js bundle.
how does pricing work?

Little market study.

A quick read of the tools documentation show that they all offer the basic features we would need:

Javascript error logging (catch unhandled exceptions)
Performance monitoring

Most of them generate Breadcrumbs (Pieces of information that are added to the error report - eg. user interractions prior to the crash -) and enable developpers to add custom breadcrumbs
Some of them have a feature called session replay. This works by logging timestamped snapshots of the DOM to the server when it changes. Those snapshots can be replayed later. (eg. https://sentry.io/for/session-replay/ )
I Highlighted 3 platforms which provide all these features: Sentry, LogRocket and DataLog.
In regard with the KlimaDAO project, Sentry has an edge because it offers a native NextJS SDK and support for Vercel which can enable end to end performance profiling.


Uplift Onset Picker form inputs/selects, fix crash (#256, #396)

Summary
These changes aim to update the Onset Picker to use the new Terra Form components (#256) and address the crash found in #396. The name prop values have also been condensed into a single prop that is used as a base value to create unique ids and names for child components. These changes will necessitate a major version bump.
Additional Details
The new form components support blanks, and resetting the date was the requested behavior in #396, so a number of changes were done to handle undefined/blank values. New translations were added but are currently all English. I will hopefully have correct translations by the end of the week.

How easy would it be to separate out the crash fix from the rest of this work?  I'm hesitant to tie them together.

@ryanthemanuel It should be fairly simple to separate the crash fix. Those changes were fairly minor.

@doogthemushroom We will need an Upgrade Guide since this is a MVB

I misspoke about the ease of separating the crash from this update. I'm going to leave them together. It comes down to the deprecated Select's not supported blank values. This results in an inconsistency between the onsetDate getting reset when clearing the date in the Date Picker, and then the incorrect displayed values in the Onset Picker Select's.

@emilyrohrbough Added an Upgrade guide to the docs folder.

Translations have been updated.


Add integration tests

Currenltly jubakit integration test is only done by example scripts, which does not cover very well, especially the error cases (Jubatus process invocation failure etc.)  We need to add such tests.

Partially fixed via #45. Still need some work.


Duplicated calls inside a while without a yield return null

Hi, I have a weird little problem running a Coroutine that has a loop and a yield return new WaitForSeconds inside.
For some reason, if I don't include a yield return null the code inside the loop will be executed twice.

                // Adding "yield return null"  a loop.

            }
        }
}

Output:

TestTwice Loop: start
TestTwice Loop: Time 0 C: 0
TestTwice Loop: Time 1.007233 C: 1
TestTwice Loop: Time 1.007233 C: 2
...

StackTrace of those repeated logs/calls.:

Same Time Counter Value = 1

TestTwice Loop: Time 1.007233C: 1
UnityEngine.Debug:Log(Object)
Presentation.Temporal.FirstLoad.c__Iterator1:MoveNext() (at Assets/Scripts/Presentation/Temporal/FirstLoad/FirstLoadController.cs:88)
Svelto.Tasks.SerialTaskCollection:RunTasks() (at Assets/Plugins/Svelto/TaskRunner/SerialTaskCollection.cs:63)
Svelto.Tasks.SerialTaskCollection:MoveNext() (at Assets/Plugins/Svelto/TaskRunner/SerialTaskCollection.cs:42)
Svelto.Tasks.Internal.CoroutineEx:MoveNext() (at Assets/Plugins/Svelto/TaskRunner/CoroutineEx.cs:33)
Svelto.Tasks.Internal.PausableTask:MoveNext() (at Assets/Plugins/Svelto/TaskRunner/PausableTask.cs:121)
Svelto.Tasks.Internal.c__Iterator0:MoveNext() (at Assets/Plugins/Svelto/TaskRunner/Runners/UnityCoroutineRunner.cs:225)
UnityEngine.SetupCoroutine:InvokeMoveNext(IEnumerator, IntPtr)

Same Time Counter Value = 2

TestTwice Loop: Time 1.007233C: 2
UnityEngine.Debug:Log(Object)
Presentation.Temporal.FirstLoad.c__Iterator1:MoveNext() (at Assets/Scripts/Presentation/Temporal/FirstLoad/FirstLoadController.cs:88)
Svelto.Tasks.SerialTaskCollection:RunTasks() (at Assets/Plugins/Svelto/TaskRunner/SerialTaskCollection.cs:63)
Svelto.Tasks.SerialTaskCollection:MoveNext() (at Assets/Plugins/Svelto/TaskRunner/SerialTaskCollection.cs:42)
Svelto.Tasks.Internal.CoroutineEx:MoveNext() (at Assets/Plugins/Svelto/TaskRunner/CoroutineEx.cs:33)
Svelto.Tasks.Internal.PausableTask:MoveNext() (at Assets/Plugins/Svelto/TaskRunner/PausableTask.cs:121)
Svelto.Tasks.MonoRunner:ExecuteFirstTaskStep(IPausableTask) (at Assets/Plugins/Svelto/TaskRunner/Runners/MonoRunner.cs:55)
Svelto.Tasks.MonoRunner:StartCoroutine(IPausableTask) (at Assets/Plugins/Svelto/TaskRunner/Runners/MonoRunner.cs:44)
Svelto.Tasks.Internal.c__Iterator0:MoveNext() (at Assets/Plugins/Svelto/TaskRunner/Runners/UnityCoroutineRunner.cs:226)
UnityEngine.SetupCoroutine:InvokeMoveNext(IEnumerator, IntPtr)

Thanks !

hey thanks a lot for reporting a bug! I will check it ASAP!

Sorry I had to take so long to test this. My current test is like this:
while (counter < 2)
{
counter++;
            yield return new WaitForSeconds(1f);

            Debug.Log(counter);
        }

and it correctly prints 1 and 2 only. Does it mean that the new version I am going to commit soon is fixed now?

I tested it with the latest version and I cannot reproduce it, let me know if it still happens to you

Thanks for the fix! :)
If I see an issue I will let you know !!


feat(notifier): add support for basic auth

Hi,
I add support for basic auth for notifications if your apprise-api instance is protected.
I also took the opportunity to add loggers to facilitate diagnostics.
Furthermore, I'm new to Node.js, feel free to make any observations.
Regards,

have you tested this locally?

Thank you, that sounds good. I'll merge this in


Hotfix/mobileFixes

Purpose
These fixes close #2781, #2773, and #2260.  Prior to these fixes, certain elements were not centered in the navbar and the navbar was also wider than it should be on the "Profile" page when on mobile.  Also, the name header on the "Profile page" for mobile was getting obscured/overlapping links previously and this addresses that.  This PR cleans up these misaligned UI elements to create a smoother user experience.
Changes

Adjusted line height of the text next to the logo/gravatar to make sure that it was centered with the image
Adjusted the padding/margins on the images so that they vertically center
Changed the size of the font and the margins of the name header on the "Profile" page so that the name fits better.
Removed a class from the table on the "Profile" page that does not seem to do anything other than make the table too wide

Side Effects
Many of the changes could be made without any performance issues between the full site and the mobile site.   Anything that needed to be different on the two was done using @media tags in the CSS so it would not have any side effects on the other.  Tested on Chrome, Firefox, and Safari.

Nice @ticklemepierce :+1:
@caneruguz If you haven't, can you please review these commits? But also, @ticklemepierce -- This should probably not be a "[Hotfix]" rather merge request it to "develop" without that tag -- These are good, but not critical, can should get an opportunity to go through QA combined into a full release.

@ticklemepierce there is something wrong with this PR, you are probably not following the pull/push logic of the change. Ideally in a PR only your commits should be listed, otherwise that means you haven't synced your local changes properly.

@ticklemepierce Your pr is based on develop which in this case can only be merged into develop instead of master. Again hotfix based on master, feature based on develop.

Yes so ping us in the new PR that you make so we can take a look at your changes. Thanks!

@ticklemepierce Could you close this pr since you already reopen the other one?


Supplying Single point energy not in Gaussian format

The excellent disk management feature of ORCA makes it a very attractive tool for computing single point energy. For small molecules, the gaussian postHF computation can take several TB of disk space while ORCA only takes several hundred GB. So I wonder if it is possible to supply ORCA file or input manually the single point energy file with the Gaussian opt/freq output file?

Are you trying to apply an ORCA single point energy calculation to a gaussian opt/freq? If so, GoodVibes will recognize single point calculations in the same directory by their filename and the option specified.  For example, you could use python -m goodvibes file.log --spc DLPNO where file.log is your gaussian file and file_DLPNO.out is your ORCA calculation in the same directory.

@luchini18 Exactly what I want. Thank you.


I've changed some default parameters

Some default parameters had not been defined.

Hello, sorry I'll not revert the spaces. My Editor did this by default (Visual Studio Code) and they are not needed. My changes to the optional parameters are all correct (I've checked the doku for every change).

Thank you for your contribution.
Unfortunately we're not accepting pull requests with unrelated changes - we're trying to keep git history consistent to make it easier to support this project.
That is, I'm closing this PR. If you want to contribute your changes, please create PR according to our CONTRIBUTING.md. We will be happy to review it.


Deployment Package vs. Software Update Package

It's not very logically that "Deployment" package means software update packages.
Pretty much everything ConfigMgr does is deployment - not just software updates.
And it's called Software Update Packages in the console.
Same for Operating System Images and System Installer.
They are called Images and Upgrade Packages in the console.

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

ID: 8fcba1a4-9a0b-c34c-1c55-4d9c4fe52128
Version Independent ID: d3a8950d-f0aa-3dfd-8cd9-b656c9617283
Content: Publish-CMPrestageContent - Configuration Manager
Content Source: sccm-ps/ConfigurationManager/Publish-CMPrestageContent.md
Product: configuration-manager
Technology: configmgr-other
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

Thanks @heinejeppesen for taking the time to provide feedback on Configuration Manager! I want to make sure it gets directly into the hands of the engineering team. Fortunately, we've got dedicated channels for that very purpose. :) See the Product feedback section in our docs. I think it’s important that you submit it directly, because that allows you to receive notifications and more closely monitor the progress. I'm closing this issue as there's nothing actionable for docs at this time.


Are you working on a spam open boxes for Dino Box, please let me you :D

We are working on that right now


Interfaces and Connectors Cmdlet

[x] CREATE Interface

[x] READ Interface

[x] UPDATE Interface

[x] DELETE Interface

[x] CREATE Connector

[x] READ Connector

[x] UPDATE Connector

[x] DELETE Connector

[x] Remove-Connector -Test

[x] Get-Interfaces -Id 42 -Providers

[x] Get-Interfaces -Id 42 -Consumers

[x] Get-EntityKind -Id 42 -Connectors

[x] Get-EntityKind -Id 42 -Interfaces

[x] Get-EntityKind -ProvidesInterfaces 1337

[x] Get-EntityKind -RequiresInterfaces 1337

[x] Get-Node -ProvidesInterfaces 1337

[x] Get-Node -RequiresInterfaces 1337

[x] Test-Connectors -Provider EkId1 -Consumer EkId2

[x] Test-Connectors -NodeId 1 -Consumer EkId2

$continuation = $true;
$response = $svc.Core.EntityKinds.Execute();
do
{
	foreach($ek in $response)
	{
		if([biz.dfch.CS.Appclusive.Public.Constants+EntityKindId]::IntrinsicEnd.value__ -ge $ek.Id)
		{
			continue;
		}
		if($ek.Name -eq $ek.Version)
		{
			Log-Warn $fn ("{0} [{1}] has identical Version. Skipping ..." -f $ek.Name, $ek.Id);
			continue;
		}

		$eks.Add($ek);
	}
	$continuation = $response.GetContinuation();
	if(!$continuation)
	{
		break;
	}
	$response = $svc.Core.Execute($continuation);
}
while($continuation)

@seilc1 can we close this issue as soon as the pull request gets merged or is there still something open?

As discussed with @seilc1 we can close this issue.


fix(charts): Change lvm-controller's Kind from StatefulSet to Deployment (#196)

Pull Request template
Why is this PR required? What issue does it fix?:
This PR address #196 issue and tries to avoid lvm-controller's disruption if underlaying node(s) crashed. Deployment's controller will restart failed pod(s) on healthy node, unlike StatefulSet's controller.
What this PR does?:
It change lvm-controller's Kind from StatefuleSet to Deployment.
Does this PR require any upgrade changes?:
No, as far as I know.
If the changes in this PR are manually verified, list down the scenarios covered::

Helm release migration from sts to deployment (LE, 2 replicas).

Checklist:

[x] Fixes #196
[x] PR Title follows the convention of  <type>(<scope>): <subject>
[x] Has the change log section been updated?
[ ] Commit has unit tests
[ ] Commit has integration tests
[ ] (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
[ ] (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

Closing since it's cherry-picked in #284


Misleading Github title: Symfony2

"Adds CORS headers support in your Symfony2 application"
From previous commits, I judge that at least Symfony 4 is supported or even required.
Maybe you could show in the README which versions of this bundle support which Symfony versions?

Fixed github description, thanks

Thanks!


Nonce is too large

Bitmex has begun rejecting requests sent using Bitmex.NET with the following error:
System.AggregateException: One or more errors occurred. (Nonce must be a valid number and below 2^53. You sent:<PHONE_NUMBER>710664., Code:HTTPError) ---> Bitmex.NET.BitmexApiException: Nonce must be a valid number and below 2^53. You sent:<PHONE_NUMBER>710664., Code:HTTPError
at Bitmex.NET.BitmexApiProxy.d__7.MoveNext()
Evidently there is a nonce upper limit of 2^53. Using the formula in NonceProvider.GetNonce():
new DateTime(1990, 1, 1).AddTicks((long)Math.Pow(2, 53)) = 17/7/2018 11:58:45pm
i.e. as of just before midnight last night, generated nonce values are too large.

Closing as it was fixed in commit 4e0c4b343791a06ca7253970510e2b02bf1f673f.


how to set  multi graphics device id when run some train example like dnn_semantic_segmentation_train_ex

MY PC  multi graphics， i need  2rd  device to   train, how to set or choose?

got it，tks
发自我的iPhone
------------------ Original ------------------
From: Davis E. King @.>
Date: Wed,Mar 31,2021 7:20 PM
To: davisking/dlib @.>
Cc: zouwen198317 @.>, Author @.>
Subject: Re: [davisking/dlib] how to set  multi graphics device id when run some train example like dnn_semantic_segmentation_train_ex (#2339)
Call set_device() or use the CUDA_VISIBLE_DEVICES environment variable.
On Wed, Mar 31, 2021 at 4:06 AM zouwen198317 @.***>
wrote:
> MY PC multi graphics， i need 2rd device to train, how to set or choose?
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/davisking/dlib/issues/2339>, or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/ABPYFR7DQ27LUALC2R46M23TGLJZBANCNFSM42EAG63Q>
> .
>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Handle a user cancelling a purchase

Overview
Starting to work on gracefully handling when a user wants to cancel a purchase. This happens when they hit "Cancel" on the alert before buying.
Rusty with my Obj C, would love some pointers / guidance. The callback does log appropriately:
Current state of PR
When you tap "Cancel' the error below occurs. The callback is logged correctly:
2017-07-14 15:22:13.997 [info][tid:com.facebook.React.JavaScript] user_cancelled

Circling back here, thanks for the early feedback. Its been years since I worked within Obj C so I may not be able to get this finished, but I am willing to push it as far as I can.

And in case it is related to these changes. Below is what I am getting when my products attempt to load:
Error Domain=NSURLErrorDomain Code=-999 "Cannot connect to iTunes Store" UserInfo={NSErrorFailingURLStringKey=https://sandbox.itunes.apple.com/WebObjects/MZInit.woa/wa/initiateSession?ix=6&os=10&locale=en_US, NSLocalizedDescription=Cannot connect to iTunes Store, NSErrorFailingURLKey=https://sandbox.itunes.apple.com/WebObjects/MZInit.woa/wa/initiateSession?ix=6&os=10&locale=en_US}

Building to my device and have removed the app and signed out of the App Store several times.

My products are fetching now and my latest logs are:
2017-07-18 21:10:33.074 [warn][tid:main][InAppUtils.m:61] No callback registered for transaction with state purchased.
2017-07-18 21:10:33.075 [warn][tid:main][InAppUtils.m:61] No callback registered for transaction with state purchased.
2017-07-18 21:10:33.075 [warn][tid:main][InAppUtils.m:61] No callback registered for transaction with state purchased.
2017-07-18 21:10:33.075 [warn][tid:main][InAppUtils.m:61] No callback registered for transaction with state purchased.

Will try and patch up those missing links

@chirag04 Would you be interested in taking a look at this? Not very strong with Obj C and would love to get this resolved to ship this app.


Throw Exception when adding @AutomapConstructor to multiple constructors

Fixes https://github.com/mybatis/mybatis-3/issues/2626
Provides helpful error messages when adding @AutomapConstructor to multiple constructors.
thank you.

@harawata Thanks for the review!
I understood and moved the test code package. also edited the PR title and content.

Coverage increased (+0.004%) to 87.325% when pulling bc3c37c2838307537cefddd1f00afe4aa5240d70 on sogoagain:arg-name-based-constructor-automapping-only-one-automap-constructor into 034cacd1db3e3db5e154d72b30ed0f20b803aaa2 on mybatis:master.


fix: VueJS infinite query parameters reactivity when hasInfiniteQueryParam

Status

READY
Description
Fix https://github.com/anymaniax/orval/issues/1378 for detailed issue description
Todos

[ ] Tests
[ ] Documentation
[ ] Changelog Entry (unreleased)

Steps to Test or Reproduce

Assigned @Maxim-Mazurok he is the Vue expert.

Thanks, after confirmation of fix I can try to add a test for that

Hey @Maxim-Mazurok, is there anything I can do to speed things up on this one? I checked the existing tests and didn't find any for the useInfiniteQuery implementation. That's probably because this bug wasn't spotted for a long time

LGTM I will merge. If @Maxim-Mazurok had and issue we can always fix again.

Hey guys, don't have much time for OSS lately, so probably do not assign me to reviews for now... We do update orval periodically, and I check if generated files changed, and we have pretty good test coverage, so if there will be any bugs - I'll report back to the repo.
As for infinite query, very likely that we missed it, as we don't use infinite query feature. So this probably makes sense. I would suggest creating a functional test in the future, I remember adding a few for the vue query sample. Hope this helps, cheers!


Rate limit Introduce2 circuits or force-close circuits with long build times?

Current unpreventable attacks are possible due to lots of Introduce2 cells being sent with the client not needing to verify, which results in extremely long build times which cannot be closed fast enough because of graceful closure. Is it possible to add within vanguards a way of rate limiting Introduce2 cells or simply allowing the force close of circuits with build times that exceed X (value configured within vanguards config)?
This would be perfect and an instant fix right now if something like this would be possible to introduce into this project.

The circuit building causes the Tor process CPU to be pinned at 100%, rendering it unresponsive and seems to be from the crypto libraries used for circuit building.

Unfortunately due to how Introduce requests work, rate limiting them (presumably by closing the circuit -- that's the only lever I have at the control port level) would just cause whatever fraction of requests to take longer or fail. I am not sure it would actually improve your situation.
However, have you seen https://trac.torproject.org/projects/tor/ticket/30291 and its children? It looks like the upcoming 0.4.1 alpha release will have performance improvements for this case. The 0.4.1 series will have its first alpha shortly after May 15th, and will stabilize around August: https://trac.torproject.org/projects/tor/wiki/org/teams/NetworkTeam/CoreTorReleases#Calendar:
You can also built current Tor master and try that out, or try to backport those child tickets yourself...

Also relevant: https://trac.torproject.org/projects/tor/ticket/15516 (needs child tickets to be completed first before it is useful).
Roger's point on that ticket is interesting: if the attacker's client is well-behaved, then closing over-used INTRODUCE circuits in vanguards might slow it down. But services would also still suffer from availability loss while under attack. So trying to solve this in vanguards alone seems like it would not satisfy https://en.wikipedia.org/wiki/Kerckhoffs's_principle -- it might work for a while, but the adversary could adapt and still find ways to keep services down.
If someone wants to try it anyway to conduct some experiments, the implementation would be very similar to the MAX_HSDESC_KILOBYTES option, except instead of checking for circuits with purpose HS_SERVICE_HSDIR, the check would be for purpose HS_SERVICE_INTRO:
https://github.com/mikeperry-tor/vanguards/blob/12f1dcc9b9584edcc45cba86b3e0a00a0d829c1a/src/vanguards/bandguards.py#L238
In fact, a one-line hack that changes HS_SERVICE_HSDIR to HS_SERVICE_INTRO there would cause the MAX_HSDESC_KILOBYTES option to apply to introduce2 cells, causing the client to close introduce circuits after so many kilobytes.
Again, I think that's not likely to work well, so I'm not going to bother with trying that experiment myself, but I've been surprised before.

Thanks for that Mike, it seemingly works when the Tor process is restarted, closing a lot of the attacking circuits, however it is still overwhelmed pretty quickly to 100% CPU.
Is there anything else you could recommend which could make it more "aggressive" as such?

The Tor Control Port is limited in what it can do. The only thing it can do in this case is close the circuits more often, which will cause you to randomly drop more intro requests and not build as many rend circuits. If you lower that MAX_HSDESC_KILOBYTES config setting after editing that line, it will close the circuit and log at that lower limit.
Out of curiosity, when you test this, do you see any NOTICE messages in vanguards log about relay overuse? This is relevant to Issue #38 and might also shed light on this problem.

I am also curious if the vanguards process itself exhibits high CPU load under these conditions.

I'll update you with the outputs shortly, when I lowered the MAX_HSDESC_KILOBYTES setting it didn't seem to take effect, not sure if it isn't following the config file for some reason.
Vanguards CPU is often pretty high around 120-140, but I attributed that mostly to the fact I wasn't using pypy.

Vanguards itself might be overloaded and not closing the circuits fast enough. If it is hitting 100% CPU itself you should definitely consider pypy. 100% CPU means that it will be delayed in responding to any limit overages, which may impact your results.
But it should still close circuits eventually... The setting is actually named circ_max_hsdesc_kilobytes in the vanguards.conf file, under the [Bandguards] section. Are you sure you're loading the config?

I'm prepping to launch a tor mirror of one of my large communities and I've been doing testing with an HS_SERVICE_INTRO addition, it actually does help keep the service reachable far longer than normal.

When you tested this, did you see any NOTICE log messages in vanguards log about any relays being used "above its consensus weight" (direct quote from log)? This is relevant to Issue #38 and might also shed light on this problem.

I personally have not

Ok this option is now provided and documented in the conf file, README_TECHNICAL.md, and changelog.


Fixed wrong code example in ArrayField documentation

merged in a44dc200d0cbd651f66a2082aa9909ff8f51a659, thanks!

:+1: :cake:


[CI] Notify owners on job/unit test failure

What does this PR do?

Notify team that owns Gitlab CI job when it fails on a Gitlab master/deploy pipeline
Notify team that owns Go unit test when it fails on a Gitlab master/deploy pipeline
Improve internal messages for CI failures

Motivation

Make it easier for teams to notice their tests are failing on a master/deploy pipeline

Additional notes

This PR does not cover CircleCI/AppVeyor failures.

I ran
CI_PIPELINE_ID=4198290 inv -e pipeline.notify-failure --print-to-stdout

It worked as expected, so I am merging 🚀


Improve README

Could have a "Before" and "After" section annotation what it actually does.

That's a great suggestion Orta! Will work on that.

Yeah, that's exactly what I would like to see


standardize UPA format

In order to ensure consistency, we need to standardize the formats of upa and genome_upa in workspace_downloader.py to match the format 'a/b/c'.

Context: https://github.com/kbase/collections/pull/327#discussion_r1230107466


Adding Call to Action molecule

Adding Call to Action molecule
Additions

Adding Call to Action molecule.

Testing

Visit  /landing-page-1/.

Review

@jimmynotjim
@sebworks
@KimberlyMunoz
@Scotchester

Screenshots

:+1: Looks good

:+1:

Added two minor comments, but :+1: again.


Cleaning up ChatMessage ?

We were having some debate about whether / how to clean up the ChatMessage class in schema/ which is a thin wrapper around LangChain BaseMessage, which is starting to add more complexity than value.
A few options:

Do nothing, keep ChatMessage. Maybe it's a useful abstraction. However it adds a lot of complexity with the BackgroundTask idea built out in !42
Remove ChatMessage, just return LangChain BaseMessages. Pass on any complexity like the Anthropic format to the consumer
Use the BaseMessage but make a best effort to clean up and always return the "simple" type, specifically having content=str or empty str if there's a tool use in an AIMessage. This would be similar to the behavior of ChatMessage today (in fact the JSON version might end up equivalent or very close)

If we go with 2 or 3 there are a couple of other considerations:

Dealing with run_id - This is added to ChatMessage today and needs to also be added to a new baseMessage. Maybe in additional_kwargs?
Best way to serialize LC BaseMessage - LC provides a message_to_dict / messages_from_dict functions which are used a bit in schema.py today. However the serialization format is kind of ugly. Maybe we can just use .model_dump()?

Without trying to implement it, I'd be in favor of (3) to maintain a similar pattern to today but simplify the schema and extra abstractions in this template. Also welcome thoughts and other ideas from users!

cc @peterkeppert @antonioalegria @gbaian10 @3x10RaiseTo8 what do you think?

I tend to prefer (1) because encapsulating all of that complexity in a single place makes the client and presentation layer much simpler and easier to adapt.
Having said that, there's probably the need to support richer kinds of outputs and tasks. To me that suggests an expansion of the ChatMessage to have a list of Tasks and perhaps an extensible support for "rich content" -- i.e. not just text. In this case you wouldn't need to support it here but leave the extension point for the implementer.
Regarding (3), note that there are AIMessages with tool calls and a message.

I also prefer option 1, but I hope to change the original to retain only the response_metadata part (which model was used and how many tokens were spent). The other content seems almost entirely repetitive.
Additionally, the current API only returns the last message, and the type always appears to be "ai".

Have you considered returning the entire messages, or adding a parameter to decide whether to return only the last one or all of them?

My preference is 2) and if not, then I think 1) is second best, as this way we don't venture into improving Langchain's message abstraction, which is not this toolkit's core functionality. I think it's possible we will see some improvement in the future on Langchain's side and we might be able to use it. 1) adds some complexity to ChatMessage, however I think it is a reasonable amount. Let's see if it keeps growing as more functionality comes in and if so, it might give us a reason to revisit the topic later.

Sounds good - let's stay with (1) for now - when I have a bit of time will take a look in some closer detail at how to simplify the existing code a bit and make it more extendable for something like BackgroundTask.
if anyone else visits this and has more feedback or input it's welcome!

I spent some more time thinking about this and how to make it more extendable with requirements like what's in https://github.com/JoshuaC215/agent-service-toolkit/pull/42. Here's an updated proposal:

Add a new type "custom" to ChatMessage, and an optional field custom_data: dict[str, Any] (or similar)
Add a utility function send_custom_message(data: dict[str, Any]) for agents. This function would send a LangChain message like ChatMessage(role="custom", content=[data]), or maybe use the additional_kwargs field. This could be sent using the adispatch_custom_event function, similar to the approach used here
Note: This custom type could also be returned by nodes directly in other scenarios!
On receiving a langchain.ChatMessage with role="custom" from the agent stream, the service can convert it to a schema.ChatMessage(type="custom", custom_data=content[0]) or similar.
The author would be responsible for basic serialization / deserialization of their custom type e.g. calling .model_dump() / .model_validate() on either end if it's a Pydantic model.

I think this approach would require a minimal change to the core service / schema and be extendable to various use cases like the one @peterkeppert has been working on. I could also imagine something like an agent returning some multimedia or non-text format could make use of this (I think?)
Would love to hear any thoughts! I think I can work on making this change.

I like this idea a lot. In my case, it removes the need for a custom Langchain message type (TaskMessage). I updated #42 along the lines of this suggestion. If we have a helper function for sending custom messages, I will update the pull request to use it.
I didn't see a need to change anything in the service. This point "On receiving a langchain.ChatMessage with role="custom" from the agent stream, the service can convert it to a schema.ChatMessage(type="custom", custom_data=content[0]) or similar." can (and I think should) be handled by ChatMessage.from_langchain() function.

This work is completed with #72 and #74


go build fails for plugin building / dependency cannot be found

Hi,
I cannot build the plugin i am working on because of a dependency that cannot be fetched:
github.com/cloudfoundry/cli/cf/resources
../../go/src/github.com/cloudfoundry/cli/cf/i18n/i18n.go:10:2: cannot find package "github.com/cloudfoundry/cli/cf/resources" in any of:

https://github.com/cloudfoundry/cli/search?utf8=✓&q=AssetNames
You can see no such package or func signature exist.
@simonleung8 @krishicks
Thanks!

Hi @emirozer,
That dependency isn't fetched; it's generated. You'll need to run bin/generate-language-resources in github.com/cloudfoundry/cli.

Ah okay, probably docs aren't up to date on this, will open a pr. Thanks!

I've updated the plugin README: https://github.com/cloudfoundry/cli/commit/3d0c935657e138c6090c77d667479fab6e0fdd6b


hiddify-next: init at 1.1.1

init with appimage version
fix #285375, related #287770
also add wenjinnn to maintainer-list
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

When you're done, make sure that you have 2 separate commits in this order:

maintainers: add wenjinnn
hiddify-next: init at 1.1.1

When you're done, make sure that you have 2 separate commits in this order:

maintainers: add wenjinnn
hiddify-next: init at 1.1.1

All done! thanks for the suggestion and guide.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3855

Seems like after 24.05 release, appimageTools.wrapType2 behavior changed. after 24.05, this command will fail:
    mv $out/bin/hiddify-next-1.1.1 $out/bin/hiddify

now the location $out/bin/hiddify-next-1.1.1 change to $out/bin/hiddify-next, so I adapted this behavior.

Since hidiffy violates the license used by sing-box I thought we shouldn't package it.

Well, I'm ok with that reason. should we close this PR now or wait for confirmation from others?

Clost this since #326367 pack it from source.


Extracting UV textures in the video for all the frames

Describe the bug
Extracting UV textures for multiple frames in the video gives black image as output after the 1st frame. The output becomes greyish after the 1st frame and then eventually becomes black.
To Reproduce
add the line
uv_tex(img, ver_lst, tddfa.tri, show_flag=args.show_flag, wfp=wfp)
at line 131 in the demo_video_smooth.py below
elif args.opt == '3d'
Expected behavior
To produce the UV textures for all the images(frames) in the video

how to generate the 3D render picture with color?

@Zaverichintanoxolo
Did you solved that? i saw this exact issue

I don't remember correctly but it was due to tri index
Try to replace tri by tri - 1


Update gpu_hist algorithm

Put new GPU histogram algorithm with loss guided mode into production. Remove old algorithm.

Codecov Report

Merging #2901 into master will increase coverage by 0.01%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #2901      +/-   ##
============================================
+ Coverage     43.41%   43.43%   +0.01%
  Complexity      207      207
============================================
  Files           154      154
  Lines         11639    11632       -7
  Branches       1167     1164       -3
============================================
- Hits           5053     5052       -1
+ Misses         6255     6249       -6
  Partials        331      331

Impacted Files
Coverage Δ
Complexity Δ

src/tree/tree_updater.cc
61.53% <ø> (ø)
0 <0> (ø)
:arrow_down:

src/learner.cc
25.66% <0%> (+0.25%)
0 <0> (ø)
:arrow_down:

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 24f527a...e6929c2. Read the comment docs.


[QBD] Offline mode for RequireQuickBooksDesktopModal

Details
Add Offline mode for RequireQuickBooksDesktopModal
Fixed Issues

$ https://github.com/Expensify/App/issues/50548
PROPOSAL:
Tests

Prerequisites:

Turn on Beta quickbooksDesktopOnNewDot on your account

Sign in on an account above
Go to any Workspace -> Accounting
Click on Connect under QuickBooks Desktop connection
While RHP is loading, quickly disconnect network.
Verify that after this the offline screen is displayed (The offline screen is displayed until the link is received)

[x] Verify that no errors appear in the JS console

Offline tests

Prerequisites:

Turn on Beta quickbooksDesktopOnNewDot on your account

Sign in on an account above
Go to any Workspace -> Accounting
Click on Connect under QuickBooks Desktop connection
While RHP is loading, quickly disconnect network.
Verify that after this the offline screen is displayed (The offline screen is displayed until the link is received)

QA Steps

Prerequisites:

Turn on Beta quickbooksDesktopOnNewDot on your account

Sign in on an account above
Go to any Workspace -> Accounting
Click on Connect under QuickBooks Desktop connection
While RHP is loading, quickly disconnect network.
Verify that after this the offline screen is displayed (The offline screen is displayed until the link is received)

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
not relevant

Android: mWeb Chrome
not relevant

iOS: Native
not relevant

iOS: mWeb Safari
not relevant

MacOS: Chrome / Safari
https://github.com/user-attachments/assets/dfa64e80-898d-4bcb-bcd4-4691a5c5936a

MacOS: Desktop
https://github.com/user-attachments/assets/c43732d0-9d35-46ce-89c3-17bfe3c414ed

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.


Prepare release 0.8.0

Closes #101

Coverage remained the same at 100.0% when pulling 7825c371d1a774ee5021f779fed4ca6a531367df on rodjek:release-0.8.0 into a96b65b2f8b3e00a3b699d343cbb972299d38910 on camptocamp:master.


bug: qdrant using poetry

trigger, install qdrant using poetry
poetry add qdrant_client
error:
Note: This error originates from the build backend, and is likely not a problem with poetry but with grpcio-tools (1.48.2) not supporting PEP 517 builds. You can verify this by running 'pip wheel --no-cache-dir --use-pep517 "grpcio-tools (==1.48.2)"'.

hi @vikyw89
could you please provide more details on your environment and the commands you ran to install qdrant-client?
I've tried this commands:
pyenv local 3.10 && python3 -m venv venv && source venv/bin/activate && pip3 install -U pip && poetry init
poetry add qdrant-client
it worked and installed grpcio-tools==1.64.1

I'm on ubuntu 24
The command was:
poetry add qdrant-client

it normally should not crush with this error, maybe you have some other dependencies which led to this error
in order to investigate it, we could start with the python version you're using and the list of dependencies (the other requirements in your pyproject.toml)

I'm on ubuntu 24
The command was: poetry add qdrant-client
I managed to install it by specifying the grpcio-tools version
Also why are you using pyenv and venv and poetry all at the same time ?

that's just the matter of taste

Will you get a nested env with those long command line ?
I'll send it my .toml tomorrow,
Dependency management in python is so problematic

no, it results in a regular env

I run into the same issue whether installing qdrant_client or langchain-qdrant.


Bitwise operators

Added bitwise operators the the package

Nice, but what is reason to add such? And how casting will work? Example, Int16 & uint32?

Reason: I use expr mainly to parse Modbus data, typical expression I would like to use are:
273.15 + (registers[0] << 16 | registers[1]) / 1000.0
(registers[0] & 8) == 8
The first one is now solved with:
273.15 + (registers[0] * 65536 + registers[1]) / 1000.0
but I don't know an easy way to solve the second one. Besides that the first expressions are easier to understand.
Casting: don't really now how to solve that in a clean way. Now you can only use unsigned integer types. Technically it wouldn't matter to expect signed integers as well but I think it gives unexpected behaviors for the binary not operator.

I see. Let me test your solution. Also lolike ci failed.

Hi @munnik
Adding this snippet,
case OpLeftShift:
	code("OpLeftShift")

case OpRightShift:
	code("OpRightShift")

case OpBitwiseAnd:
	code("OpBitwiseAnd")

case OpBitwiseXor:
	code("OpBitwiseXor")

case OpBitwiseOr:
	code("OpBitwiseOr")

case OpBitwiseNot:
	code("OpBitwiseNot")

after,
https://github.com/antonmedv/expr/blob/2c1881a9909453f9f1047886d9be0b94e0ba5c48/vm/program.go#L145-L147
should pass the tests. It wasn't able to resolve the opcodes for disassembly of bitwise operators.

Hi @aaditya-panik,
Thank you for the suggestion, it fixed the tests.

Hi @munnik, @antonmedv
I noticed that using numeric literals for bitwise operations seems to throw an error because they are parsed as int and the operations check for uint type. I have another branch which I had used to fix this (https://github.com/aaditya-panik/expr/commit/d24c08d94a400a11db1660531ab28fee3d4c1c23), let me know what you think. (Yet to add some tests)

Is it still actial? Not sure we want to add bitwise ops here.


How do I temporarily disable the auto-alignment, in Insert mode?

I'm continually fighting with the auto-aligner, and would like to temporarily disable it. Can I do that?

I know what you mean, sometimes the indenter gets hyperactive.
Internally the config uses neovimhaskell/haskell-vim for indentation.
To turn it off, see https://github.com/neovimhaskell/haskell-vim#indentation
It would be cool to bind that to a key command. I'd take a pull request for that if you're feeling ambitious.

I've found that putting this in my vimrc.local works:
nnoremap <silent> <F9> :setl noai nocin nosi inde=<CR>
Although, it's a "one shot" technique, whereas I'd prefer a toggle.


Awesome jQuery

This patch adds awesome-jquery list. This time I've gone through the awesome manifesto and followed all the practices and requirements mentioned there. Thank you for considering merging it to the awesome repository also.

Looks good. Thanks for reading the contributing guidelines (almost nobody does) :D


Repeatedly getting Only one tag found for the given repository. Usually this is the case for the initial release.

The GitHub action repeatedly generates a "no changes" changelog even when there were labeled pull requests merged in between the versions. I think the problem is found somewhere in the tag resolution for this repository.
To produce the following output I did the following:

On branch staging with tag v0.0.8
Create a new branch with changes, create a PR onto staging and add a label fix to this PR
Merge that PR and update my local staging branch with git pull origin staging
Increase the version with npm version patch -> v0.0.9
Push the tag with git push origin staging --follow-tags

createReleaseChangelog.yml:
name: Create Release and Create Changelog
on: [push]

jobs:
  create-release:
    runs-on: ubuntu-latest
    if: startsWith(github.ref, 'refs/tags/')
    steps:
      - uses: actions/checkout@v3
        with:
          fetch-depth: 0

      - name: Build Changelog
        id: github_release
        uses: mikepenz/release-changelog-builder-action@v3
        with:
          configuration: changelog-configuration.json
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

      - name: Create Release
        uses<EMAIL_ADDRESS>        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
        with:
          body: ${{steps.github_release.outputs.changelog}}

changelog-configuration.json:
{
    "categories": [
      {
        "title": "## 🚀 Features",
        "labels": ["feature"]
      },
      {
        "title": "## 🐛 Fixes",
        "labels": ["fix"]
      },
      {
        "title": "## 🧪 Tests",
        "labels": ["test"]
      },
      {
        "title": "## ✨ Magic",
        "labels": ["magic"]
      }
    ],
    "ignore_labels": [
      "ignore"
    ],
    "sort": {
      "order": "ASC",
      "on_property": "mergedAt"
    },
    "template": "${{CHANGELOG}}\n\n\nUncategorized\n\n${{UNCATEGORIZED}}\n",
    "pr_template": "- ${{TITLE}}\n   - PR: #${{NUMBER}}",
    "empty_template": "- no changes",
    "label_extractor": [
      {
        "pattern": "(.) (.+)",
        "target": "$1",
        "flags": "gu"
      },
      {
        "pattern": "\\[Issue\\]",
        "on_property": "title",
        "method": "match"
      }
    ],
    "duplicate_filter": {
      "pattern": "\\[ABC-....\\]",
      "on_property": "title",
      "method": "match"
    },
    "transformers": [
      {
        "pattern": "[\\-\\*] (\\[(...|TEST|CI|SKIP)\\])( )?(.+?)\n(.+?[\\-\\*] )(.+)",
        "target": "- $4\n  - $6"
      }
    ],
    "max_tags_to_fetch": 1000,
    "max_pull_requests": 1000,
    "max_back_track_time_days": 365,
    "exclude_merge_branches": [
      "Owner/qa"
    ],
    "tag_resolver": {
      "method": "semver",
      "filter": {
        "pattern": "api-(.+)",
        "flags": "gu"
      }
    },
    "base_branches": [
      "dev"
    ]
}

Log:
Run mikepenz/release-changelog-builder-action@v3
  with:
    configuration: changelog-configuration.json
    includeOpen: false
    ignorePreReleases: false
    failOnError: false
    fetchReviewers: false
    commitMode: false
  env:
    GITHUB_TOKEN: ***

📘 Reading input values
  ℹ Configuration successfully loaded.

🔖 Resolve tags
  ℹ Found 1[2](https://github.com/<private>/runs/7088282055?check_suite_focus=true#step:3:2) (fetching max: 1000) tags from the GitHub API for <private>
  🔖 Resolved current tag (v0.0.9) from the 'github.context.ref'
  ℹ Only one tag found for the given repository. Usually this is the case for the initial release.
  /usr/bin/git rev-list --max-parents=0 HEAD
  5c9e17757804d8f06ec45e6fa45[3](https://github.com/<private>/runs/7088282055?check_suite_focus=true#step:3:3)071baddfdf0f
  🔖 Resolved initial commit (5c9e1775780[4](https://github.com/<private>/runs/7088282055?check_suite_focus=true#step:3:4)d8f06ec4[5](https://github.com/<private>/runs/7088282055?check_suite_focus=true#step:3:5)e[6](https://github.com/<private>/runs/7088282055?check_suite_focus=true#step:3:6)fa4530[7](https://github.com/<private>/runs/7088282055?check_suite_focus=true#step:3:7)1baddfdf0f) from 'git rev-list --max-parents=0 HEAD'
  🔖 Resolved previous tag (5c9e17757[8](https://github.com/<private>/runs/7088282055?check_suite_focus=true#step:3:8)04d8f06ec45e6fa453071baddfdf0f) from the tags git API

🚀 Load pull requests
  ℹ Comparing <private> - '5c[9](https://github.com/<private>/runs/7088282055?check_suite_focus=true#step:3:9)e17757804d8f06ec45e6fa453071baddfdf0f...v0.0.9'
  ℹ Found 1859 commits from the GitHub API for <private>
  ℹ Fetching PRs between dates 2021-[11](https://github.com/<private>/runs/7088282055?check_suite_focus=true#step:3:11)-04T09:36:46.000Z to 2022-06-28T08:00:54.000Z for <private>
  ℹ Retrieved 374 merged PRs for <private>
  ℹ Retrieved [18](https://github.com/<private>/runs/7088282055?check_suite_focus=true#step:3:19)59 release commits for <private>

Warning: ⚠ No pull requests found

Do you have any leads on what might be causing this?

It's interesting. Given the code here: https://github.com/mikepenz/release-changelog-builder-action/blob/develop/src/tags.ts#L32-L61 it looks like your repository has no other tags specified?
Can you double check what the GitHub API returns for your project?
Additionally you'll filter out more tags given the pattern:
    "tag_resolver": {
      "method": "semver",
      "filter": {
        "pattern": "api-(.+)",
        "flags": "gu"
      }
    },
    ```

But it looks your tag is not following the naming e.g.: `v0.0.9`

You are right npm is doing their version tags in this weird vX.X.X way so I changed the regex in the configuration:
  "tag_resolver": {
      "method": "semver",
      "filter": {
        "pattern": "v(.+)",
        "flags": "gu"
      }
    },

However I still get no pull requests found while running the action even when there was a labeled PR merged in between the tag's date. This is the updated log:
📘 Reading input values
  ℹ Configuration successfully loaded.

🔖 Resolve tags
  ℹ Found [14](https://github.com/<private>/runs/7089843881?check_suite_focus=true#step:3:15) (fetching max: 1000) tags from the GitHub API for <private>
  🔖 Resolved current tag (v0.0.11) from the 'github.context.ref'
  🔖 Resolved previous tag (v0.0.10) from the tags git API

🚀 Load pull requests
  ℹ Comparing <private> - 'v0.0.10...v0.0.11'
  ℹ Found 3 commits from the GitHub API for <private>
  ℹ Fetching PRs between dates 2022-06-28T09:29:39.000Z to 2022-06-28T09:36:22.000Z for <private>
  ℹ Retrieved [19](https://github.com/<private>/runs/7089843881?check_suite_focus=true#step:3:21)5 merged PRs for <private>
  ℹ Retrieved 3 release commits for <private>

Warning: ⚠ No pull requests found

The log notes that it found some merged PRs between the date range.
Based on your comment above you have a different base branch in your environment?
The action filters out PRs not merged into defined base branches
https://github.com/mikepenz/release-changelog-builder-action/blob/d246f23caf649bf18333f005883e50fdeb78019d/src/releaseNotes.ts#L167-L171

Thanks that solved the issue I didn't think about this base branch field :)


Yank to zsh commandline is not working with vi-mode

I'm using zsh with vi keybindings for my prompt.
When I use Y to copy text and paste it to the current prompt it does not work as expected.
When I'm in insert mode everything is fine.
When I'm in normal mode the text gets butchered (because tmux-yank does not enter insert mode)
I can work around this issue but it would be nice if tmux-yank would check if the prompt is in normal mode and enter insert mode.

I've been having a similar issue, but only intermittently. For example, it'll work as expected for a while... then my computer will go to sleep, and then it'll stop working. Very strange behavior. I haven't been able to find a root cause or workaround.

I've been having a similar issue, but only intermittently. For example, it'll work as expected for a while... then my computer will go to sleep, and then it'll stop working. Very strange behavior. I haven't been able to find a root cause or workaround.

I think this is a related issue: https://github.com/tmux-plugins/tmux-yank/issues/107


chore(ui): Add icons for panel generation feature

Adds 3 icons

@connieelee this code has been merged as https://github.com/wandb/weave/pull/1943 so closing this PR. Sorry for all of the CI troubles.
Related internal Slack threads:
https://weightsandbiases.slack.com/archives/C03BSTEBD7F/p1721064750272799
https://weightsandbiases.slack.com/archives/C03BSTEBD7F/p1721072010553639

@connieelee this code has been merged as #1943 so closing this PR. Sorry for all of the CI troubles. Related internal Slack threads: https://weightsandbiases.slack.com/archives/C03BSTEBD7F/p1721064750272799 https://weightsandbiases.slack.com/archives/C03BSTEBD7F/p1721072010553639

sounds good. tysm!!


Prefer gateways that support private payments

I don't know if this added complexity really changes anything.

I am now convinced


Install provisioning profile not working

Hi,
the installation via the fastlane aciton "update_project_provisioning" is not working.
The profile was not installed before manually.
How is it possible to install a provisioning profile at a path?
With my implemented gym command i got the following error:

Code signing is required for product type 'Application' in SDK 'iOS 10.3'

We don't recommend using the update_project_provisioning action any more, check out our code signing guide linked by the bot 👍

@KrauseFx why should we avoid using update_project_provisioning?
I checked the code signing guide and I'm already using match, but can't figure out how to update app identifier and project provisioning with Fastlane.

Hey @grifotv, could you provide a bit more information around what you are trying to do, the more context we have the better! 😄 !


"Initializing TS/JS language features" freezes app for too long on every restart.

Version: 1.30.0
Commit: c6e592b2b5770e40a98cb9c2715a8ef89aec3d74
Date: 2018-12-11T22:21:33.585Z
Electron: 2.0.12
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
OS: Linux x64 4.13.0-43-generic
Steps to Reproduce:

Download VS Code
Open a JavaScript file

Looks like someone is running around closing issues EXACTLY describing this issue by claiming they don't have enough info.  I would rather the entire app be a basic text editor than have that extension slow my machine down one iota.

I even tried some other solutions posted elsewhere such as changing my jsconfig.json to:
{
"compilerOptions": {
"module": "es2015",
"target": "ES6",
"lib": ["es6"]
},
"exclude": [
"node_modules",
"**/node_modules/*"
]
}

All I want to do is change the CSS of an element in the editor, (which is it's own goddamn nightmare) and every SINGLE time I restart, the whole machine hangs until it's done and I just want it to stop.  I already hate TS as much as a person can.

I believe this was caused by some theme I had installed. While the loading time is still long, it has reduced enough where I believe it's nominal on my super slow old hardware.

Same issue here. No themes installed.
Version: 1.30.0 (user setup)
Commit: c6e592b2b5770e40a98cb9c2715a8ef89aec3d74
Date: 2018-12-11T22:29:11.253Z
Electron: 2.0.12
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
OS: Windows_NT x64 10.0.17134

Doesn't that also disable the JS/TS Language features?

I think the important take-away here is that something(?) is affecting performance and it is not easy or clear to find the culprit. You can selectively allow extensions while disabling the rest, narrowing it down, but what would be better is something that automatically warned you there might be a problem with X-extension because it's taking a long time to finish doing whatever.

Same issue.  Reverted to 1.25.1.

After a very brief glance at the logs, my next guess is that the TS server launching is slowing some people's machines way down. I loathe TypeScript. It makes everything it touches take more time and only helps you if you're error prone.


#8 [3/4]: Implement model fitting with mgcv

[!IMPORTANT]
This PR a trial for using a stacked PR workflow. Do not merge.

Extends #18 to fit a GAM using the supplied formula and data. There are two implemented backends, mgcv::gam() and mgcv::bam(). Dispatch to the appropriate backend is handled via an internal S3 class -- the methods are exported but the generic is private so the methods are still internal to the package. Decent default arguments are supplied, with optional user-supplied arguments passed via a list.
There's also a new warning for legal but unwise parameterization.

Having everything based on main makes it quite a pain. Are you just doing this to get the CI? Why not just update the CI to trigger on PRs against branches that aren't main.

Yep this is just for CI. I'll switch the base before opening the PR. I think before updating CI rules we want to check in on whether the whole stacked PR thing is working.

My only concern here is about how opinionated this is getting - i.e. very. We might want that and in that case its fine but I do think we should be taking choices here that still allow people to use it for other things even if the default behaviour is very opinionated.

Agreed -- I think the push and pull here is really helpful. I don't want to be overly rigid with my view of the target user and target use-case, but also I want to avoid stepping into the territory of EpiNow2/Epinowcast/EpiAware. Those are closer to custom model foundries than I think this package should be. That doesn't we should prevent any/all customization (and the current implementation is likely too constraining), but I do want to hear a convincing use-case for e.g. custom model formula and arguments. What would the user flow be? Who is this user? Does this package provide enough benefit over handrolling in mgcv or jumping right to EpiNow2 for us to optimize for their use-case?

My other concern is there are a few notes about implementing other backends etc but what is coded here doesn't really seem set up for success there (unless success is lots of internal character string based switches which I assume it isn't). Is there an issue scoping out how extensions would be implemented?

The current flow is here is that RtGam() fits the model and is the class constructor. I'd strongly prefer we avoid the public data constructor -> public model constructor flow, but perhaps an internal S3 class that switches would be possible? Issue to discuss is https://github.com/CDCgov/cfa-gam-rt/issues/21

I think before updating CI rules we want to check in on whether the whole stacked PR thing is working.

Agree but from a review perspective the current setup of having to wade through all the old commits is very bad which makes it hard to know if stacked PRs are working

Sure, but this PR isn't open for review yet as I note at the top?

Sure, but this PR isn't open for review yet as I note at the top?

Fair enough. @seabbs out for now then!

... vs a list + does this allow actual default args to fit_model vs an internal list?

Addressing here

Having the class check be in fit_model.default vs when preparing the class. Doing so allows for user extension at the cost of control and potentially some opaque errors (though not at the moment).

I think the counterpoint to consider is what to do if/when we need to move to S3 for other steps (I'm thinking predict and maybe formula would be needed). Centralizing the check for backends into one place adds clarity I'd think, at the cost of extensibility as you noted.

(I'm thinking predict and maybe formula would be needed)

If you did do this and backends didn't have methods these would by default throw an informative error (but not a incredibly pretty one) if you just had no default method


Create dependabot.yml

I was testing a dependabot terraform github action but I don't think we need it so I'm going to close this.


Overlapping access requests prematurely revoke access

Describe the bug
Access is removed when a session ends even if that same access is granted by another active access session.
To Reproduce
Steps to reproduce the behavior:

Create and approve a request
When the first request is nearing expiry, create and approve another request for the same access (ie. to effectively extend the duration)
Wait for the first access session to end, and observe the role get revoked

Expected behavior
When an access session ends, TEAM will not revoke the access if it has been granted by another ongoing access session.
TEAM Version: v1.1.2

still an issue

Still an issue

This would definitely be helpful


Cannot generate any icon

ImageMagick installed
Android Platform
Cordova config.xml file exist

But i generating nothing with the cordova-icon command,
Is someone get the same problem?
 Checking Project & Icon

  ✓  platforms found: android
  ✓  icon.png exists
  ✓  config.xml exists

 Generating Icons for android

getting same issue??


Added Weather_App by Ukesh-dev

Weather App

This is the Weather App used for forecasting the weather of the city/county.
Website Link Weather App

Task Checklist
Put x in between the square brackets without any spaces to tick your Task.

[x ] You have created a new branch for contribution.
[x ] Added README.md for your project in a specified folder.
[x ] Add Demo in GIF or Screenshot form.
[x ] You are Registered Officially for HacktoberFest 2021 as Partcipant.
[x ] Followed Contribution Guidelines.

This Project is Already Assigned to someone else ! And We won't accept PR without issue assignments if the same project is assigned to another person ! Plz Create PR only after you get assigned.


Added indices on journal_entries date fields

Changes
Adds indices on journal_entries.created_at and journal_entries.updated_at.

How to test these changes?
n/a

Related issues

@bugout-dev check

[ ] Run migrations on production DB

Mypy broken, locust broken. Will create issues to fix these things.


Add /auction2 route

This is an empty duplication commit as I cloned /auction to /auction2; wanting to avoid all green in the diff when I start writing code.

Good foresight! :)


Have 1 popver for multiple divs

Hello, i have a table with 30 items each one of them should have 1 popover oper on click div but only 1 popver can be opeone at any given time...
Can u have 1 popover for all of them or u have to render all 30 and keep logic to close all when 1 is open ?

Here's a codesandbox I whipped up showing how I'd approach this.
I'll close this for now! Thanks!


GTT-1075: Added page that explains the user statuses

Description
Added a page that explains the user statuses and added a link to this page from the users page.
Testing
Verified in my local environment that it looks as expected.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I know is just a help page but is worth adding a unit test at least to exercise mounting the component and verify it renders.

Sure. done!


[GraphqlDemo-1] Select Docker image for the project

First, we need to determine what operating system will be sufficient for the application's needs. The first requirement will be to base the application on the Linux operating system. Ideally, the image will not be particularly elaborate. We will expand the image ourselves with the necessary packages.
Please provide in the comments examples of 4 Docker images that could be useful for this project.

For example images of Ubuntu, Alpine, Debian and Arch Linux

Could you list the advantages and disadvantages of these systems?

Ubuntu is derived from Debian, so it is available for many architectures, while Arch Linux supports only x86 architecture. Also, Debian has over 50000 packages, although Arch is more stable than Debian distros and basically faster. Alpine Linux is commonly used for managing routers and firewalls.

It has been settled. Ubuntu will be the main OS to run this application. Good choice!


Fix for queue once lock key not being deleted properly

Problems: redis lacks flush
When running QueueOnce subclass task, I noticed the lock key never got deleted, event though QueueOnce.after_return was called and issued self.once_backend.clear_lock(key).
After examining it turned out that I had to add self.redis.flushall() to clear_lock(key) for key to be cleared properly.
Here are my libs versions:
In [1]: import redis, celery, celery_once

In [2]: redis.__version__
Out[2]: '3.2.0'

In [3]: celery.__version__
Out[3]: '4.2.1'

In [4]: celery_once.__version__
Out[4]: '2.1.0'

In this pull request I added redis.flushall() after lock delete request, which solves the issue.

Hi @kuba-lilz flushall deletes all keys of all databases, I don't think it is the solution here.
This sounds like it could be related to #93 (the keys aren't generating correctly).
As I mentioned there, could you include the code snippet of the task and how it is called?

Bad implementation that solves problem that didn't exist, see discussion in #93


[BUG] Sorting should be global not per page

Describe the bug
Currently, if you have more than one page and sort some of the columns, only the results in that particular page are sorted - like there are no other items. This is not the typical UI behavior.
To Reproduce
Steps to reproduce the behavior:

Go to a grid, containing a lot of element (multiple pages)
Sort by some column and look at the result

Expected behavior
The sorting must be applied for all items, not only for this particular page.
Screenshots
None.
Desktop (please complete the following information):

OS: Windows 10 Pro x64
Browser: Chrome
Version: 80.0.3987.116 (Official Build) (64-bit)

Additional context
None.

@vladimirpetrov This is not possible, would require backend change. The sorting behaviour is same as in old UI. I've marked it as an enhancement.

@vladimirpetrov I'm closing this as won't fix as Primate cannot fix this without the backend API allowing sorting of all items. The issue needs to be raised/fixing at https://github.com/apache/cloudstack thanks.


Investigate work required for merging Fladle

Prepare a document containing a list of actions required for merging Fladle into Flank monorepo.
Describe each action briefly if needed.
Attach document as a comment.
This list is important for further work in the scope of epic.

Steps to merge Fladle
IDEA 1: Merge Fladle and Flank repositories
Get latest Fladle version
From Fladle repository take module fladle-plugin and move it into Flank repository
Update dependencies
For know dependencies replace its name and version with coresponding names and version from buildSrc durectory. For new dependencies
Think about naming
If we would like to change plugin name, please update all fladle occurence with new name
Add test project
Add test flank project and use it as our test tool for fladle changes. The test project could be taken from  Fladle repository or written from scratch.
Remove Flank dependencies
Replace all flank external dependencies with flank test_runner module
Setup new publications
Setup new plugin publication with credentials managed by Flank team
Move Fladle documentation
Move documentation from fladle repo and place it as subheader in Flank documentation
IDEA 2: Use composite builds
Based on @bootstraponline and @no input
instead of merging Fladle and Flank repositories together we could achive same behaviour using Gradle composite builds

What are (if there are any) the pros and cons of each idea?

My proposal would be the following:

Add an integration test for the fladle plugin in the flank project. This is a good idea whether or not we are going to integrate both projects to prevent regressions.
Use a composite build to substitute the flank dependency for the in development version in this integration test.

Thank you @runningcode for your input. You have the biggest knowledge about Fladle, so I think that we should follow your proposition
@jan-gogo could we treat steps proposed by @runningcode as the next tasks in this epic?

Here are some example integration tests: https://github.com/runningcode/fladle/blob/master/fladle-plugin/src/test/java/com/osacky/flank/gradle/integration/FlankGradlePluginIntegrationTest.kt

IDEA 2 sounds reasonable since we have a lot of work around Corellium and refactor. Fladle integration test will protect us from losing compatibility with small effort. I will vote for leaving IDEA 1 for the future, now we should focus only on IDEA 2.

Investigation done, follow up ticket created https://github.com/Flank/flank/issues/1804


Library cannot be found (debug version)

The tool says that "Qt6Widgets" is missing, but it is in the folder. Sometimes "Qt6Guid" is missing instead.

Use the release version, I do not provide the debug version of QT binaries.

Okay, but the release version can't find Steam anymore: "Steam must be running to use this tool (SteamAPI_Init() failed)."
Do you know, how to solve the problem? (I started Steam before I opened the tool of course)

it CAN fail for multiple reasons, 99% it's because Steam isn't running. the other 1% are edgecases.

The thing is that, for me the map uploader works fine. If restarting your computer doesn't work then the problem lays way deeper than I thought. Debugging the application on Windiows is quite difficult for me, because I don't own a Windows system, at least, not one that I haven't tested already (the tool works on it). so... I'll see what I can do.

I've had 2 separate Windows users confirm that the tool works, it's likely something wrong with your machine. Make sure steam is running, update steam and ensure Portal 2 is installed.

Okay that's really weird, I'll try it on another machine. The last Steam update was on the 7th March 2024, Portal 2 is installed and Steam is running ofc.

The thing is that, The Steam API does NOT update, ever, if it were to, a lot of legacy applications Valve and other Steam users have made in the past would break, with no one to maintain those tools it's Valve's number 1 priority is to never change the legacy Steam API, which this tool relies on because Portal 2 relies on it. besides just setting the application theming and env variables the first piece of code that runs is:
	if ( !SteamAPI_Init() )
	{
	...
	}

It may be that your computer does not allow applications to set env variables, though, I doubt that. However, if it is indeed the case. You need to put this file at the same location as the .exe
steam_appid.txt
I cannot guarenteee it'd fix it, but it'd be worth a shot.

I tried that, but unfortuntately, this also doesn't work.

Do I have to put the folder in a special directory? Like bin?

I tried reinstalling Steam but that also didn't work :(

This might be a redundant question, but do you have Portal 2 installed? And if so, is it on your main drive or a secondary drive?

I tried installing Portal 2 on both drives, but this also didn't work.

What happens if you run the map uploader from the command prompt?
Does that show anything?

No, running it from the admin command prompt doesn't work either.

Fixed the problem now. But it was so stupid, I don't want to talk about it... :|


Sync Tabs Groups stopped working after Firefox update

Hello,
The Sync Tabs Groups stopped working after Firefox update. How to fix?
Thank you!

Any response? Could be fixed?

Hello,
Please read: https://github.com/Morikko/sync-tab-groups#the-project-is-no-more-maintained
Alternative: https://addons.mozilla.org/en-US/firefox/addon/simple-tab-groups/


Add prefix and suffix args to prelaunch

Possible workaround for #35

Broken when prefix and suffix are unspecified

Broken when prefix and suffix are unspecified


compile failed

I just run make then I got
(.text+0xafb): undefined reference to `munmap'
collect2: error: ld returned 1 exit status
Makefile:11: recipe for target 'bootgo.bin' failed

You need to compile a cross compiler.
https://wiki.osdev.org/GCC_Cross-Compiler
https://wiki.osdev.org/Why_do_I_need_a_Cross_Compiler%3F
When you compile gcc then do not forget enable go language:
--enable-languages=c,c++,go


Sample tests are flaking

looking at https://github.com/googleapis/nodejs-monitoring/pull/358, there's no reason the sample tests should be failing.

These appear to be outright failing now:
23 passing (42s)
  1 failing

  1) metrics
       should read time series data aggregated:
     Command failed: node metrics.js read-aggregate
metrics.js read-aggregate [projectId]

Aggregates time series data that matches 'compute.googleapis.com/instance/cpu/utilization'.

Options:
  --version        Show version number                                                                         [boolean]
  --projectId, -p                                                                    [string] [default: "long-door-651"]
  --help           Show help                                                                                [boolean]

TypeError: Cannot read property 'value' of undefined
    at timeSeries.forEach.data (/tmpfs/src/github/nodejs-monitoring/samples/metrics.js:328:49)
    at Array.forEach (<anonymous>)
    at readTimeSeriesAggregate (/tmpfs/src/github/nodejs-monitoring/samples/metrics.js:325:14)
    at process._tickCallback (internal/process/next_tick.js:68:7)

  Error: Command failed: node metrics.js read-aggregate
  metrics.js read-aggregate [projectId]

  Aggregates time series data that matches 'compute.googleapis.com/instance/cpu/utilization'.

  Options:
    --version        Show version number                                                                         [boolean]
    --projectId, -p                                                                    [string] [default: "long-door-651"]
    --help           Show help                                                                                [boolean]

  TypeError: Cannot read property 'value' of undefined
      at timeSeries.forEach.data (metrics.js:328:49)
      at Array.forEach (<anonymous>)
      at readTimeSeriesAggregate (metrics.js:325:14)
      at process._tickCallback (internal/process/next_tick.js:68:7)

      at checkExecSyncError (child_process.js:629:11)
      at Object.execSync (child_process.js:666:13)
      at execSync (test/metrics.test.js:23:28)
      at Context.it (test/metrics.test.js:154:20)
      at process._tickCallback (internal/process/next_tick.js:68:7)

npm ERR! Test failed.  See above for more details.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>samples-test: `cd samples/ && npm link ../ && npm install && npm test && cd ../`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>samples-test script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/node/.npm/_logs/2020-03-06T00_51_30_952Z-debug.log
cleanup

closing in favor of https://github.com/googleapis/nodejs-monitoring/issues/382, we are actually tracking flaky tests now and we will treat each as a p1.


blazepose assumes that every bone is on input

yes, that's how blazepose works - if it were not like that, i'd use it as a default as otherwise its quite good. but as it is, i keep it as optional model. if you want to report to devs that trained it, go to mediapipe's page.


AWS Lambda with Node 10/12

I had this working well with the 3.7.2 version in AWS Lambda using node 8.
AWS is removing the access to node 8 in AWS Lambda next month.
I went to move to node 10 or 12 but need to upgrade to the 4.0.0 version to support node 10.
Mapnik itself loads fine in node 10 for version 4.x. When it registers plugins, on the AWS Lambda environment, that I need I get this error though

Problem loading plugin library: /var/task/node_modules/mapnik/lib/binding/lib/mapnik/input/pgraster.input (dlopen failed - plugin likely has an unsatisfied dependency or incompatible ABI)

The bindings are being pulled from "https://mapbox-node-binary.s3.amazonaws.com/mapnik/v4.3.1/node-v64-linux-x64-Release.tar.gz"
Using node 12 with 4.3.2 I get a different error completely.

/var/task/node_modules/mapnik/lib/binding/mapnik.node: undefined symbol: _ZN2v82V813DisposeGlobalEPPNS_8internal6ObjectE

I've have tried to using docker images of lambci/lambda and amazon/lambda-build-node10.x to recreate the issue using npm install and a simple index.js file but mapnik plugins seem to load fine.
I'm not sure what the next step should be with node 8 support being dropped next month.

This appears to be a problem with the node version that nodejs12x and nodejs10x uses.
If I uses a custom execution layer from
https://github.com/lambci/node-custom-lambda
It works fine.