Dracula theme for tmux

Dracula theme for tmux
tmux-airline-dracula
https://github.com/sei40kr/tmux-airline-dracula

That's awesome dude, here is mine, hope you like it

Any of you would like to thansfer your theme to the organization so its easier for interested users to find it?

Why not, but why do you mean by "transfer your theme to the organization" ?

@pichardoJ since OP's repo has been archived, do you have a repo with your version of the theme? If not would you mind creating one using the template if @mamyn0va doesn't have one either? You could also use @sei40kr's if the instructions are still correct as long as we update the screenshot to show a little more.

My tmux dracula theme is forked from another tmux conf (not dracula), so I guess I need to ask the author if I can transfer my theme to your organization, under MIT LICENSE.
@gpakosz What do you think?

any news on this?

I would be happy to provide my theme but I still need the approval from @gpakosz.
In the meanwhile, you can find it here: https://github.com/mamyn0va/tmux

https://github.com/danerwilliams/tmux-dracula
Only tested my theme on mac os so far. I need to look in to how my unix pipelines for getting battery information and network SSID might change from mac to linux.

Install with tpm by adding set -g @plugin 'danerwilliams/tmux-dracula' to .tmux.conf if anyone wants to try it out :)
Feedback is welcome!

It took a while, but now it's official:

https://draculatheme.com/tmux
https://github.com/dracula/tmux

I would love to hear your thoughts on this @mamyn0va @pichardoJ @sei40kr

nice!
i just wish there was an easier way to have tmux themes with customization
on the  bottom bar. For example the one dark tmux theme has those powerline
symbols and so on and I always wanted to know how to have that more custom
but with the color scheme maintaining and it seems the two are very tightly
coupled right?
On Wed, Apr 8, 2020 at 1:38 AM Zeno Rocha<EMAIL_ADDRESS>wrote:

It took a while, but now it's official:

https://draculatheme.com/tmux
https://github.com/dracula/tmux

I would love to hear your thoughts on this @mamyn0va
https://github.com/mamyn0va @pichardoJ https://github.com/pichardoJ
@sei40kr https://github.com/sei40kr
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dracula/dracula-theme/issues/240#issuecomment-610688005,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD7WGRY7WK3ASOMYLNYIRDLRLPBOZANCNFSM4DV4PEXQ
.

@dagadbm I think that it's possible to set it up so that powerline symbols would be an optional setting. It looks like  nord theme tmux does this.
if [ "$status_content" != "0" ]; then
if [ "$no_patched_font" != "1" ]; then
tmux source-file "$_current_dir/$NORD_TMUX_STATUS_CONTENT_FILE"
else
tmux source-file "$_current_dir/$NORD_TMUX_STATUS_CONTENT_NO_PATCHED_FONT_FILE"
fi
fi
Maybe this would be a good issue / feature request to add?

Not sure if this is the appropriate place, but it's about Dracula and tmux, so I guess it fits.
After checking this recent release I realized that my tmux needed a revamp to match my whole Dracula setup. However, I'm already using the great Oh My Tmux configuration file. Inspired by this release and Dracula's Vim Airline theme, I decided setting up a minimal "theme" for Oh My Tmux.
So that's what I did!:
https://github.com/atgmello/dracula.omt

I'd note that @mamyn0va 's effort is appreciated. However I'd like to keep my version up to date with @gpakosz 's repo and also keep my other configuration intact. I guess that what I came up with solves both issues.
I would also like to take this opportunity to thank the whole Dracula Team! You're doing an amazing job porting such an awesome theme for so many apps! :)

Great job!
I admit that my version is not so synced with @gpakosz's repo ;)

Is great, good job guys.


Using namespace literals

In C++14 new user defined literals are, by convention, contained in a namespace literals:
#include <iostream>
#include <chrono>
int main()
{
    using namespace std::chrono_literals;
    auto day = 24h;
    auto halfhour = 0.5h;
}

We should allow these types of using declarations.

See https://github.com/tkruse/cpplint/tree/using-namespace-literals

The C++ style guide's ban on using namespace has no exception for literal namespaces, so cpplint shouldn't either.


Align ListItemIcon with ListItemText's primary

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
According to the Material-Design-Specs the ListItemIcon should be aligned with the ListItemText's primary, but it's vertically centered instead.

Expected Behavior 🤔

I can fix this one :) will put up a PR this afternoon

This issue is outdated (MD2 related). We aim to have a fresh look for Material UI in the near future, so I’m closing this.


🐛 Bug Report: Default password shown in plain text in template creation review page

📜 Description
While creating template in RHDH, if password value is not provided , review page shows default password in plain text instead of asterisks.
👍 Expected behavior
It should not show the password in plain text
👎 Actual Behavior with Screenshots

👟 Reproduction steps

Create a Software Template with this input field eg -: https://github.com/sudhishmk/rhdhcatlogexample/blob/main/defaultpswdtemplate.yml
Create a resource using this template
Leave the Repository password field untouched. The default value defghj will be used
Click Next until the Review page is shown
The value defghj will be visible in plain text
Repeat the steps above and fill in a random password
The password is masked with asterisks on the Review page

📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

@jesuino I can work on this

Does it make a difference to remove the default value?

Hello,
Good news! I can't reproduce this with Upstream backstage! I will just track the fix to link it here. Any hints about where I should look is welcome. Thanks!

I tracked the fix to this PR - I can confirm it indirectly fixed this issue because before it secret fields were not handled in 'ReviewState.tsx'.
https://github.com/backstage/backstage/pull/26110
I am closing this issue, thank you all!


Modifications to make JVM Agent work on IBM JVM 8.

Mods to make JVM Agent work on the IBM JVM 8. Hopefully this is useful for those that happen to run IBM's JDK and want to use the Jolokia JVM Agent.

Thanks a lot ! looks good on a first sight. Unfortunately I'm a bit busy these days, but I'm going to review and integrate this PR asap. Please ping me, if I don't come back to this within the next two weeks.
thanks again ...

seems to be a minor merge conflict. would be also awesome if you could resolve this by rebasing to the current master.
Thanks!

Sure, will fix tonight.

Rebased, fixed conflict, squashed commits, pushed. Hope this is useful for those few of us that from time to time are on J9.

Thanks a lot ! 'hope I can get a version out soon, but very likely not this week anymore.


BUG: Unknown CMake command "ExternalData_add_test" when building SpatialObjectsModule

CMake Error at /home/chris/Slicer/CMake/SlicerMacroSimpleTest.cmake:44 (ExternalData_add_test):
  Unknown CMake command "ExternalData_add_test".
Call Stack (most recent call first):
  SlicerModules/SpatialObjectsModule/Testing/Cxx/CMakeLists.txt:44 (SIMPLE_TEST)

-- Configuring incomplete, errors occurred!
make[2]: *** [Stamp/TubeTK/TubeTK-configure] Error 1
make[1]: *** [CMake/Superbuild/CMakeFiles/TubeTK.dir/all] Error 2
make: *** [all] Error 2

Commenting out this line [1], it appears to configure and build properly.
[1] https://github.com/TubeTK/TubeTK/blob/master/SlicerModules/TubeTKModules.cmake#L27

Slicer no longer supported.


Accessing Model on HuggingFace

I'm having trouble accessing the model for the inference and evaluation.
"OSError: CARDS_RoBERTa_Classifier is not a local folder and is not a valid model identifier listed on 'https://huggingface.co/models'"

If you want to access the pre-trained model, you'll need to download the weights in the README (https://drive.google.com/uc?export=download&id=1cbASuoLNY-kJcm7hUFLTGYzblZFzxaVo) and edit the notebook to point to the relevant directory. I didn't upload the model to huggingface and don't maintain this implementation, so can't help you there.


Update MRN Infrastructure

Reverse proxy needs to be set up for authentication
This will allow everything to be encrypted

Finished a while ago


Fix a test in rollup target alias change

Signed-off-by: bowenlan-amzn<EMAIL_ADDRESS>Issue #, if available:
Description of changes:
CheckList:

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Thank you for making this change, @bowenlan-amzn


Error when transforming wrapped parser

A picture is worth a thousand words:

Context: I want to take some ZodType<T> (which I have aliased as Parser<T>) and wrap it inside a { data: T } parser, which I then want to destructure using transform to extract the actual data of type T.
I don't really know what I'm doing wrong, but it seems like in the simple case (line 5 in the screenshot) it just works.
What am I doing wrong here?
Version: 3.0.0-alpha.32
TypeScript Version: 4.2.2

Using transform inside a generic function generally doesn't work. The .transform method relies on knowing the exact inferred type of the schema, so it can infer the output type of the transformation function. For instance
z.string().transform(val => val.length);

Here, TypeScript knows val is string, and can infer that the output of the transform is number.
But inside a generic function like wrap, the schema type is any because you could pass any Zod schema into it.
You can achieve what you want, but you have to explicitly type the wrap function:
const wrap = <T extends z.ZodTypeAny>(
  inner: T
): ZodEffects<ZodObject<{ data: T }>, z.infer<T>> => {
  return z.object({ data: inner }).transform((val: any) => {
    return val.data;
  });
};

const schema = wrap(z.string());
schema.parse({ data: "asdfasdf" }); // => "asdfasdf"

Check out #355 for an explanation of how this works.


Timeouts during Upsert

I have a job where I'm upserting ~60GB+/2billion rows into delta table in S3.  This timeout is controlled by spark.executor.heartbeatInterval
	at org.apache.spark.rpc.RpcTimeout.org$apache$spark$rpc$RpcTimeout$$createRpcTimeoutException(RpcTimeout.scala:47)
	at org.apache.spark.rpc.RpcTimeout$$anonfun$addMessageIfTimeout$1.applyOrElse(RpcTimeout.scala:62)
	at org.apache.spark.rpc.RpcTimeout$$anonfun$addMessageIfTimeout$1.applyOrElse(RpcTimeout.scala:58)
	at scala.runtime.AbstractPartialFunction.apply(AbstractPartialFunction.scala:38)
	at org.apache.spark.rpc.RpcTimeout.awaitResult(RpcTimeout.scala:76)
	at org.apache.spark.rpc.RpcEndpointRef.askSync(RpcEndpointRef.scala:103)
	at org.apache.spark.executor.Executor.reportHeartBeat(Executor.scala:996)
	at org.apache.spark.executor.Executor.$anonfun$heartbeater$1(Executor.scala:212)
	at scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.java:23)
	at org.apache.spark.util.Utils$.logUncaughtExceptions(Utils.scala:1996)
	at org.apache.spark.Heartbeater$$anon$1.run(Heartbeater.scala:46)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.runAndReset(FutureTask.java:308)
	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$301(ScheduledThreadPoolExecutor.java:180)
	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:294)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
Caused by: java.util.concurrent.TimeoutException: Futures timed out after [10000 milliseconds]
	at scala.concurrent.impl.Promise$DefaultPromise.ready(Promise.scala:259)
	at scala.concurrent.impl.Promise$DefaultPromise.result(Promise.scala:263)
	at org.apache.spark.util.ThreadUtils$.awaitResult(ThreadUtils.scala:293)
	at org.apache.spark.rpc.RpcTimeout.awaitResult(RpcTimeout.scala:75)

This appears to be happening at the stage where Delta is joining the existing table to the incoming data:

I've tried all manner of things:

Changed the partitions (current partitioning scheme leads to parquet files between 1-3mb)
Altering the timeouts (spark.network.timeout set to 1000s)
Adding more memory (spark.[driver/executor].memory set to 38912m)
Removing driver memory limits (currently set spark.driver.maxResultSize to 20g, 36g available)

These steps seem to have solved various memory issues that proceeded the timeout error, but I cant find anything that would help here. Any advice on how to move forward?
Spark: 3.1.1
Delta: 1.0.0

Solved the problem by:

repartitioning my data according to the partitions used in Delta before the write
picking partitions that resulted in ~200MB files
Increasing my shuffle partitions


Cancel Button for Create Account Dialogue Not Working

The Cancel Button in the Create Account Dialogue Window doesn't appear to work for me. I'm on Chrome.

This functionality seems to be working on firefox and chrome for me.
It would help if you could retest this and if it's not working provide a screenshot of the developer console ( right click -> inspect -> click on console tab).


Error occurred during dotnet publish: 1

Has anyone found a solution for this?
It used to work with CLI 0.0.9. But stopped working after upgrade to 2.1.
I have tired all posted solutions here with no luck. I have upgraded and downgraded the versions but no luck.

@duncanawoods , thank you for the reply. The dotnet build does not show any errors. There are warnings but it is common. I tried it with a new vanilla app and still get the same error regardless of .net core version. Electron CLI is installed globally. Could it be my Windows development environment?

As I say, post the output of the tool otherwise I can't help. The warnings may be relevant and errors can appear before warnings which is what tripped me up before with the same question.

@duncanawoods
Here is the output:
C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB>electronize start
Start Electron Desktop Application...
Microsoft Windows [Version 10.0.17134.523]
(c) 2018 Microsoft Corporation. All rights reserved.
C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB>dotnet publish -r win-x64 --output "C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\obj\Host\bin"
Microsoft (R) Build Engine version 15.9.20+g88f5fadfbe for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.
  Restoring packages for C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.BLL\MyElectronApp.BLL.csproj...
  Restoring packages for C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\MyElectronApp.WEB.csproj...
  Restoring packages for C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.Entities\MyElectronApp.Models.csproj...
  Restoring packages for C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.DAL\MyElectronApp.DAL.csproj...
  Restore completed in 377.52 ms for C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.Entities\MyElectronApp.Models.csproj.
  Restore completed in 565.84 ms for C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.DAL\MyElectronApp.DAL.csproj.
  Restore completed in 579.79 ms for C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.BLL\MyElectronApp.BLL.csproj.
  Generating MSBuild file C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\obj\MyElectronApp.WEB.csproj.nuget.g.props.
  Generating MSBuild file C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\obj\MyElectronApp.WEB.csproj.nuget.g.targets.
  Restore completed in 1.79 sec for C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\MyElectronApp.WEB.csproj.
  MyElectronApp.Models -> C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.Entities\bin\Debug\netcoreapp2.0\MyElectronApp.Models.dll
  MyElectronApp.DAL -> C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.DAL\bin\Debug\netcoreapp2.0\MyElectronApp.DAL.dll
  MyElectronApp.BLL -> C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.BLL\bin\Debug\netcoreapp2.0\MyElectronApp.BLL.dll
Infrastructure\AseEncryptor.cs(76,30): warning CS0168: The variable 'ex' is declared but never used [C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\MyElectronApp.WEB.csproj]
Infrastructure\AseEncryptor.cs(114,30): warning CS0168: The variable 'ex' is declared but never used [C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\MyElectronApp.WEB.csproj]
Controllers\QuestionController.cs(88,45): warning CS1998: This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\MyElectronApp.WEB.csproj]
Controllers\CoursesController.cs(38,42): warning CS1998: This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\MyElectronApp.WEB.csproj]
Controllers\CoursesController.cs(198,42): warning CS1998: This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\MyElectronApp.WEB.csproj]
Controllers\CoursesController.cs(213,42): warning CS1998: This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\MyElectronApp.WEB.csproj]
Controllers\HomeController.cs(52,46): warning CS1998: This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\MyElectronApp.WEB.csproj]
Controllers\HomeController.cs(59,46): warning CS1998: This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread. [C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\MyElectronApp.WEB.csproj]
Infrastructure\LicenseFilter.cs(23,24): warning CS0169: The field 'LicenseFilter.publicKey' is never used [C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\MyElectronApp.WEB.csproj]
Infrastructure\LicenseFilter.cs(17,24): warning CS0414: The field 'LicenseFilter.licensePath' is assigned but its value is never used [C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\MyElectronApp.WEB.csproj]
Infrastructure\LicenseFilter.cs(22,33): warning CS0414: The field 'LicenseFilter.licenseServicePublicUrl' is assigned but its value is never used [C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\MyElectronApp.WEB.csproj]
Infrastructure\LicenseFilter.cs(21,33): warning CS0414: The field 'LicenseFilter.licenseServiceUrl' is assigned but its value is never used [C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\MyElectronApp.WEB.csproj]
Infrastructure\LicenseFilter.cs(18,24): warning CS0414: The field 'LicenseFilter.publicKyePath' is assigned but its value is never used [C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\MyElectronApp.WEB.csproj]
Infrastructure\LicenseFilter.cs(19,24): warning CS0414: The field 'LicenseFilter.error' is assigned but its value is never used [C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\MyElectronApp.WEB.csproj]
  Trimmed 209 out of 449 files for a saving of 33.95 MB
  Final app size is 73.03 MB
  MyElectronApp.WEB -> C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\bin\Debug\netcoreapp2.2\win-x64\Xngine.dll
  MyElectronApp.WEB -> C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\bin\Debug\netcoreapp2.2\win-x64\Xngine.Views.dll
  Trimmed 209 out of 449 files for a saving of 33.95 MB
  Final app size is 73.03 MB
  MyElectronApp.WEB -> C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\obj\Host\bin\
C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB>
Error occurred during dotnet publish: 1

Thanks.
The thing that stands out is that there are two version of .net core involved:
There is 2.0

MyElectronApp.BLL -> C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.BLL\bin\Debug\netcoreapp2.0\MyElectronApp.BLL.dll

and 2.2

MyElectronApp.WEB -> C:\Users\Admin\OneDrive\TransferLand\dashboard\MyElectronApp\MyElectronApp.WEB\bin\Debug\netcoreapp2.2\win-x64\Xngine.dll

I struggle to keep track with how sdk versions are set at build, run and publish but it sounds like there is mismatch happening between e.g.

what is set in .csproj
what is set in global.json
what publish will default to

Take a look at:
https://docs.microsoft.com/en-us/dotnet/core/versions/selection

Do you have any yellow triangles over your dependencies in visual studio?
Can you update your project to use 2.2?
What output do you get for a normal dotnet publsh?

I managed to get rid of the warnings and update all projects to core 2.2.
Now the Electornize start command does not give the error, it starts the web server but the Electron Window does not open. Looking at the output folder, I noticed that it actually does NOT generate the Electron related folders/files.
I will share the output and my .csproj content in a bit.

0.0.11 currently works with 2.1. You should try 2.1.9 (2.1.505 SDK)

Thank you GFS, I have it all setup and working.

@srameh I know this issue has been closed but can you share your solution for how you got it to work? Maybe it's only your .csproj file?

I Installed the CLI globally. Here is the .csproj entry:

$(IncludeRazorContentInPack)

$(IncludeRazorContentInPack)

$(IncludeRazorContentInPack)

PreserveNewest


chore(main): release 0.3.2

:robot: I have created a release beep boop
0.3.2 (2024-09-28)
Bug Fixes

add the --service flag in the kardinal flow telepresence intercept command (#259) (5d22282)
fix broken website CSS by refactoring styled-components SSR logic (#257) (505e885)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/kurtosis-tech/kardinal/releases/tag/0.3.2 :sunflower:


github_packages: odie on missing skopeo.

If HOMEBREW_FORCE_HOMEBREW_ON_LINUX is set then we don't want to try to brew install or it will try and install from source (which takes too long).

Review period skipped due to critical label.


create test run does not populate teststatistics field

2018-07-12 06:55:54.32: root        : DEBUG    Route template: {project}/_apis/test/{resource}/{runId}
2018-07-12 06:55:54.36: root        : DEBUG    Api version '4.1'
2018-07-12 06:55:54.38: root        : DEBUG    POST https://REDACTED/THEPROJECT/_apis/test/Runs
2018-07-12 06:55:54.40: root        : DEBUG    Request content: {'runStatistics': [{'count': 1, 'state': 'Completed', 'outcome': 'Failed'}], 'isAutomated': True, 'name': 'MyTest'}
2018-07-12 06:55:54.43: msrest.pipeline: DEBUG    Configuring request: timeout=100, verify=True, cert=None
2018-07-12 06:55:54.45: msrest.pipeline: DEBUG    Configuring proxies: ''
2018-07-12 06:55:54.46: msrest.pipeline: DEBUG    Evaluate proxies against ENV settings: True
2018-07-12 06:55:54.49: msrest.pipeline: DEBUG    Configuring redirects: allow=True, max=30
2018-07-12 06:55:54.49: msrest.pipeline: DEBUG    Configuring retry: max_retries=3, backoff_factor=0.8, max_backoff=90
2018-07-12 06:55:54.59: urllib3.connectionpool: DEBUG    Starting new HTTPS connection (1): REDACTED
2018-07-12 06:55:54.361: urllib3.connectionpool: DEBUG    https://REDACTED:443 "POST /THEPROJECT/_apis/test/Runs HTTP/1.1" 200 443
2018-07-12 06:55:54.378: root        : DEBUG    Response content: {"id":74094,"name":"MyTest","url":"https://REDACTED/THEPROJECT/_apis/test/Runs/74094","isAutomated":true,"iteration":"THEPROJECT","owner":{"displayName":null,"id":"00000000-0000-0000-0000-000000000000"},"project":{"id":"493b23e6-f63f-4c56-9c2a-4d09eec371e8","name":"THEPROJECT"},"state":"Unspecified","totalTests":0,"incompleteTests":0,"notApplicableTests":0,"passedTests":0,"unanalyzedTests":0,"revision":2,"webAccessUrl":"https://REDACTED/THEPROJECT/_TestManagement/Runs#runId=74094&_a=runCharts"}
{'comment': None, 'substate': None, 'release_environment_uri': None, 'last_updated_date': None, 'owner': <vsts.test.v4_1.models.identity_ref.IdentityRef object at 0x0516C110>, 'unanalyzed_tests': 0, 'post_process_state': None, 'id': 74094, 'custom_fields': None, 'project': <vsts.test.v4_1.models.shallow_reference.ShallowReference object at 0x0516C310>, 'passed_tests': 0, 'is_automated': True, 'build_configuration': None, 'state': u'Unspecified', 'total_tests': 0, 'additional_properties': {}, 'build': None, 'dtl_environment_creation_details': None, 'due_date': None, 'release_uri': None, 'completed_date': None, 'test_environment': None, 'incomplete_tests': 0, 'last_updated_by': None, 'plan': None, 'phase': None, 'started_date': None, 'run_statistics': None, 'test_message_log_id': None, 'name': u'MyTest', 'drop_location': None, 'dtl_environment': None, 'dtl_aut_environment': None, 'url': u'https://REDACTED/THEPROJECT/_apis/test/Runs/74094', 'test_settings': None, 'error_message': None, 'iteration': u'THEPROJECT', 'controller': None, 'web_access_url': u'https://REDACTED/THEPROJECT/_TestManagement/Runs#runId=74094&_a=runCharts', 'filter': None, 'not_applicable_tests': 0, 'created_date': None, 'release': None, 'revision': 2}

Was using the API to create a test run with minimal information (name field, is_automated, and the run_statistics), it appears the test run is being created but on the rest for the particular runid the runStatistics field is [] empty.
One thing I did note on the vsts api page is there is no 4.1 documentation, the 4.0 exists but not for test case management api calls being documented. There is now a 5.0 which does list these api calls as available? Is that perhaps the issue? Or is there more information I need to provide to creating the test run?
My goal is to have a tool generate the test runs and results because the test management framework doesn't appear to allow arbitrary scripts to be considered 'automation' so I am wanting to populate the results myself after series of scripts run...

Found was using wrong api calls, think I got it figured out now...


Is it possible to use it with vim-plug?

Hello!
First of all, thanks for your outstanding work. I've been using multiple of your projects for some time already and they're amazing!
So, back to the topic. I have a trouble with some slow plugins which make my startup time around 700ms (me being on 11 years old HDD doesn't help XD). It's not a big deal, but it'd be nice to reduce it mb to 300. So I decided to try out this plugin.
Thing is, if I invoke setup it right after the end of vim-plug:
vim.call('plug#end')
require('impatient')

it won't affect my startup time.
But if I do it before, it will fail to find this module.
Any way I can fix that?
(and btw, I don't have a huge preference for vim-plug, so I probably don't mind moving to something newer, it just will take some time to migrate for my 40-50 plugins setup to ensure no errors and all, you know. so if impatient isn't working on vim-plug, I'll perhaps migrate)

Can you link to your config and I can make some suggestions?

Sure! here
here is how I load the plugins
(vim.g.use_ide() thing there is to load only on "ide" mode activated. That's the one I'd like to boost)

Firstly, impatient only boosts the performance of loading lua files/modules. If a plugin contains no lua code, then it won't benefit at all.
Secondly vim-plug doesn't really provide anything we can use to load impatient earlier so we will need to load it manually.
Vim-plug installs all its plugins to vim.fn.stdpath('data') .. '/plugged' so we need to do something like:

vim.o.rtp = vim.fn.stdpath('data') .. '/plugged/impatient.nvim' ..','.. vim.o.rtp

require'impatient'

You'll probably want to wrap this in pcall in case impatient isn't installed.

Added the first line of your snippet to the very beginning of init.lua and require'impatient', didn't help the startup time.
Although looking at :LuaCacheLog, looks like it loaded a lot of caches (but not the one for python provider :sweat_smile: , which is what introduces some overhead)

Loading the module of the python provider won't be causing the slowness, it'll be running the module, i.e. executing the Lua code.
I'd avoid using any rpc plugins if you can, they're a little bit antiquated in the age of fast Lua plugins that run in-process.

Okay, thanks


Feedback on specifications, submitted by Rachel Lawson TESTING

Rachel Lawson TESTING (cmxhd3NvbkBkaWdpdGFsaW1wYWN0YWxsaWFuY2Uub3Jn) submitted the following feedback: This is just Rachel testing out a form. It's far from perfect but lets us test the concept of what we need to do to make something permanent.

okay to close


convert_to_openai_function not generating properly with nested BaseModels

Discussed in https://github.com/langchain-ai/langchain/discussions/17481

Originally posted by mimichelow February 13, 2024
Checked other resources

[X] I added a very descriptive title to this question.
[x] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from langchain_core.utils.function_calling import convert_to_openai_function
from enum import Enum
from pydantic import BaseModel, Field
from typing import List

class Tool(Enum):
    WEB_SEARCH = "web-search"
    RSS_FEED_SCRAPER = "rss-feed-scraper"
    USER_INPUT = "user-input"

class Task(BaseModel):
    """Tasks based on the objective"""
    id: int = Field(description="Create an ID and make sure all task IDs are in chronological order")
    task: str = Field(description="Task description should be detailed. TASK Example: 'Look up AI news from today (May 27, 2023) and write a poem'")
    tool: Tool = Field(description="Current tool options are [text-completion] [web-search] [rss-feed-scraper]")

class TaskList(BaseModel):
    """List of tasks"""
    task_list: List[Task] = Field(description="List of tasks")

extraction_functions = [convert_to_openai_function(TaskList)]

Description
I'm trying to migrate from the deprecated convert_pydantic_to_openai_function with the previous Pydantic class, but the new function is not going beyond the first level. The produced output is
[
  {
    "name": "TaskList",
    "description": "List of tasks",
    "parameters": {
      "type": "object",
      "properties": {},
      "required": ["task_list"]
    }
  }
]

Is this a known bug, do i have to manually produce the full schema to pass as a dictionary now? I can't find anything in how to generate the full openai function using the new function.
Thanks
System Info
System Information

OS:  Windows
OS Version:  10.0.22621
Python Version:  3.11.6 (tags/v3.11.6:8b6ee5b, Oct  2 2023, 14:57:12) [MSC v.1935 64 bit (AMD64)]
Package Information

langchain_core: 0.1.22
langchain: 0.1.6
langchain_community: 0.0.19
langsmith: 0.0.87
langchain_openai: 0.0.5
Packages not installed (Not Necessarily a Problem)

The following packages were not found:

langgraph
langserve

Hi @eyurtsev, @mimichelow!
Upon investigating the error, it appears that the issue lies with the import of pydantic. The nested function seems to only work correctly when imported as from langchain_core.pydantic_v1 import BaseModel, Field instead of from pydantic import Field, BaseModel.
Within the convert_to_openai_function, when dealing with a standard Pydantic BaseModel, it's reaching the elif callable(function) and it calls  the convert_python_function_to_openai_function(function) method part instead of the section where convert_pydantic_to_openai_function (which handles nested versions) is called.
The chunk in the code: (from line 281 langchain_core/utils/function_calling.py)
if isinstance(function, dict):
        return function
elif isinstance(function, type) and issubclass(function, BaseModel):
        return cast(Dict, convert_pydantic_to_openai_function(function))
elif isinstance(function, BaseTool):
        return format_tool_to_openai_function(function)
elif callable(function):
        return convert_python_function_to_openai_function(function)

Ya it also silently will drop some special keys - looking forward to upgrading to pydantic v2 in the future

Updating the notebooks from the course on "Functions, Tools and Agents with LangChain" at DeepLearning.ai by Harrison Chase has similar issue.
class Information(BaseModel):
"""Information to extract."""
people: List[Person] = Field(description="List of info about people")
convert_to_openai_function(Information)
{'name': 'Information',
'description': 'Information to extract.',
'parameters': {'type': 'object', 'properties': {}, 'required': ['people']}}
from langchain.utils.openai_functions import convert_pydantic_to_openai_function
convert_pydantic_to_openai_function(Information)
{'name': 'Information',
'description': 'Information to extract.',
'parameters': {'$defs': {'Person': {'description': 'Information about a person.',
'properties': {'name': {'description': "person's name", 'type': 'string'},
'age': {'anyOf': [{'type': 'integer'}, {'type': 'null'}],
'description': "person's age"}},
'required': ['name', 'age'],
'type': 'object'}},
'properties': {'people': {'description': 'List of info about people',
'items': {'description': 'Information about a person.',
'properties': {'name': {'description': "person's name", 'type': 'string'},
'age': {'anyOf': [{'type': 'integer'}, {'type': 'null'}],
'description': "person's age"}},
'required': ['name', 'age'],
'type': 'object'},
'type': 'array'}},
'required': ['people'],
'type': 'object'}}
Using:
from langchain_core.pydantic_v1 import BaseModel, Field
solves the problem.

It saved my day


some firmware packages timestamped with 1980 in iso_minimal

I tried to reproduce building nixos-minimal-21.05.596.3a2e0c36e79-x86_64-linux.iso .
When I mount the nix-store.squashfs inside the iso, the contents appear to be the same (nix-hash 99d5f8add79590e77015c51bb7101dac), but the nix-store.squashfs files themselves differ. The first difference seems to be near the start of the file:
3,6c3,6
< 00000020  91 16 7e 05 0c 00 00 00  ca e0 b9 26 00 00 00 00  |..~........&....|
< 00000030  c2 e0 b9 26 00 00 00 00  ff ff ff ff ff ff ff ff  |...&............|
< 00000040  ba b8 9d 26 00 00 00 00  b6 c4 a9 26 00 00 00 00  |...&.......&....|
< 00000050  9e 07 b8 26 00 00 00 00  74 dd b9 26 00 00 00 00  |...&....t..&....|
---
> 00000020  91 16 da 05 0c 00 00 00  f2 e0 b9 26 00 00 00 00  |...........&....|
> 00000030  ea e0 b9 26 00 00 00 00  ff ff ff ff ff ff ff ff  |...&............|
> 00000040  ba b8 9d 26 00 00 00 00  12 c5 a9 26 00 00 00 00  |...&.......&....|
> 00000050  12 08 b8 26 00 00 00 00  9c dd b9 26 00 00 00 00  |...&.......&....|

I think this is using mksquashfs from https://github.com/plougher/squashfs-tools as invoked from nixos/lib/make-squashfs.nix, so it should have the fix from #114454, but I haven't double-checked yet.
/cc @plougher

Looking at https://dr-emann.github.io/squashfs/ the mismatches in the header are:

root_inode_ref | u64(InodeRef) | A reference to the inode of the root directory of the archive
bytes_used | u64 | The number of bytes used by the  archive. Because squashfs archives are often padded to 4KiB, this can  often be less than the file size
id_table_start | u64 | The byte offset at which the id table starts
xattr_id_table_start | u64 | The byte offset at which the xattr id table starts

< 00000020  91 16 7e 05 0c 00 00 00  ca e0 b9 26 00 00 00 00  |..~........&....|
            | root_inode          |  | bytes_used          |
< 00000030  c2 e0 b9 26 00 00 00 00  ff ff ff ff ff ff ff ff  |...&............|
            | id_table_start      |  |xattr_id_table_start |
< 00000040  ba b8 9d 26 00 00 00 00  b6 c4 a9 26 00 00 00 00  |...&.......&....|
            | inode_table_start   |  |directory_table_start|
< 00000050  9e 07 b8 26 00 00 00 00  74 dd b9 26 00 00 00 00  |...&....t..&....|
            | fragment_table_start|  | export_table_start  |
---
> 00000020  91 16 da 05 0c 00 00 00  f2 e0 b9 26 00 00 00 00  |...........&....|
> 00000030  ea e0 b9 26 00 00 00 00  ff ff ff ff ff ff ff ff  |...&............|
> 00000040  ba b8 9d 26 00 00 00 00  12 c5 a9 26 00 00 00 00  |...&.......&....|
> 00000050  12 08 b8 26 00 00 00 00  9c dd b9 26 00 00 00 00  |...&.......&....|

Am I reading this wrong?
Would be missing:

inode_table_start | u64 | The byte offset at which the inode table starts
directory_table_start | u64 | The byte offset at which the directory table starts
fragment_table_start | u64 | The byte offset at which the fragment table starts
export_table_start | u64 | The byte offset at which the export table starts

So we have changes on:

root_inode
bytes used
id_table_start
directory_table_start
fragment_table_start
export_table_start

< 00000020  91 16 7e 05 0c 00 00 00  ca e0 b9 26 00 00 00 00  |..~........&....|
            | root_inode          |  | bytes_used          |
< 00000030  c2 e0 b9 26 00 00 00 00  ff ff ff ff ff ff ff ff  |...&............|
            | id_table_start      |  |xattr_id_table_start |
< 00000040  ba b8 9d 26 00 00 00 00  b6 c4 a9 26 00 00 00 00  |...&.......&....|
            | inode_table_start   |  |directory_table_start|
< 00000050  9e 07 b8 26 00 00 00 00  74 dd b9 26 00 00 00 00  |...&....t..&....|
            | fragment_table_start|  | export_table_start  |
---
> 00000020  91 16 da 05 0c 00 00 00  f2 e0 b9 26 00 00 00 00  |...........&....|
> 00000030  ea e0 b9 26 00 00 00 00  ff ff ff ff ff ff ff ff  |...&............|
> 00000040  ba b8 9d 26 00 00 00 00  12 c5 a9 26 00 00 00 00  |...&.......&....|
> 00000050  12 08 b8 26 00 00 00 00  9c dd b9 26 00 00 00 00  |...&.......&....|

Am I reading this wrong?
Would be missing:

inode_table_start | u64 | The byte offset at which the inode table starts
directory_table_start | u64 | The byte offset at which the directory table starts
fragment_table_start | u64 | The byte offset at which the fragment table starts
export_table_start | u64 | The byte offset at which the export table starts

So we have changes on:

root_inode
bytes used
id_table_start
directory_table_start
fragment_table_start
export_table_start

Most of the above changes in value are probably irrelevant, and bogus.  This is because Squashfs is a compressed metadata filesystem where tables are packed on a byte basis.
Basically, when even only one thing changes, it changes the positions of everything else from that point on.  That in turn will cause differences in compression ratio, which will cause differences again, in a cascading knock-on effect.
The things you can tell from the hexdump are this:

The inode table start values are the same.  This means the data compressed to exactly the same size (you can tell this because the inode table is placed immediately after the data).   This means it is likely there is no data differences.
The directory_table_start in the second image is 92 bytes larger than in the first image.  This means the inode table compressed differently.

Everything from the fragment table start onwards will be different simply because of the inode table size difference.
The obvious question now is, why is the inode table size different?  There are lots of possible reasons.
The most likely reason is the inode table compressed differently because the inode contents are different.  Assuming the data hasn't changed, then anything related to where data is stored/size will be the same.   So it is probably file attributes which has changed.  Here it helps to think about what hasn't changed, because that hasn't changed in the headers

The number of inodes are the same.  This means the one filesystem hasn't hardlinked files, and the other stored them as duplicates.  Because then the inode count would be different.
The id (uid/gid) count is the same.  You can probably then rule out ownership changes.

This leaves mode and timestamps for regular files, plus symlink contents changing, device major/minor changes etc.
Basically, if the input to Mksquashfs changes, then it won't be reproducible.

For reference, I compared the iso from https://channels.nixos.org/nixos-21.05/latest-nixos-minimal-x86_64-linux.iso (which was 21.05.596.3a2e0c36e79) to a self-built one from the same commit.

This leaves mode and timestamps for regular files, plus symlink contents changing, device major/minor changes etc.

Indeed looking at the mounted squashfs it seems there are inconsistent timestamps in there. Weird that I didn't see them in meld earlier - perhaps they got lost in the sea of dangling symlink warnings.
All those timestamps seem to be in /nix/store/qcpdgqki4y0mi015dkh4ahgsrfbsf3hs-firmware-linux-nonfree-2021-05-11, /nix/store/srjp3lv3qfand8pwsj8np32yw1q4f7ms-intel2200BGFirmware-3.1 and /nix/store/xmp5w9p9jb4r2nq8sa1wyc33s6ymq198-zd1211-firmware-1.5, see diffoscope.txt.
These corresponds to firmwareLinuxNonfree, intel2200BGFirmware and zd1211-firmware respectively, though nix-build --check doesn't indicate any problems. I'll dig more later today, unless somebody beats me to it ;)

So, the iso from hydra has Modified timestamps at 1980-01-01 00:00:00 +0000, while my locally built iso has them at 1970-01-01 00:00:01 +0000.
You would expect 1970, i.e. as per the last bullet of https://nixos.org/manual/nix/stable/index.html#ssec-derivation). When I look at the filesystem on the machine I built my iso, the timestamp is also 1970-01-01 01:00:01 +0000, just like on my regular machine (where I strongly suspect it came from the hydra cache).
So the question now is: where did hydra get that 1980 date from for these 3 paths? (it indeed seems unlikely that this was introduced by mksquashfs)

@tilpner spotted 1980 happens to be the default SOURCE_DATE_EPOCH in stdenv since bc4927a5263aef5f418e81dc97c23bbf57f23d81 - though that doesn't explain yet how it's ending up as the modified date on those files.

zd1211 is timestamped 1980 for the nixos-unstable iso as well, but is correctly 1970 in nixos-20.09 iso's - even recently-built ones :thinking:

zd1211 is timestamped 1980 for the nixos-unstable iso as well, but is correctly 1970 in nixos-20.09 iso's - even recently-built ones thinking

Hmm:
20.09.4244.47e580e291f has 1970, https://hydra.nixos.org/build/126794772 cached from https://hydra.nixos.org/build/79087510
20.09.4351.1fbbf2eda31 has 1980, https://hydra.nixos.org/build/143505001 cached from https://hydra.nixos.org/build/79087510
21.05.1205.b72bde7c4a1 has 1980, https://hydra.nixos.org/build/126794772 cached from https://hydra.nixos.org/build/79087510
so.. cached from the same build, but different date?

So, one theory I had was that this might be a problem related to the hydra machine building the squashfs. That doesn't seem to be the case, though:

for 20.09 on 17a455c, 25fe709c.packethost.net produced 1980
for 20.09 on 6867200, 25fe709c.packethost.net produced 1970
for 20.09 on 1fbbf2e, a1034eb9.packethost.net produced 1980
for 20.09 on 59b8d9c, ike.ewi.tudelft.nl produced 1970

Even for the GNOME ISO (1205) it's specifically these 3 fixed-output derivations (firmwareLinuxNonfree, intel2200BGFirmware and zd1211fw) that are stuck on 1980 :exploding_head:

This hasn't been fully explained, but got fixed due to other changes to those derivations.

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/nixos-reproducible-builds-minimal-installation-iso-successfully-independently-rebuilt/34756/1

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/contribute-to-the-nix-2023-recap/37484/10


Accept a user-specified transform when rendering a tree.

I'm not sure if there's a better way of doing this (maybe by inserting a Group node and adding a transform?). I'm using it to translate a document which has content that goes beyond (0, 0) so that nothing is clipped by the origin, and to apply different scale factors to the document (targeting various display densities).

I'm not sure about this one. transform should be provided to both render and render_node. We don't need a separate method just for this.

Ok, I've updated the change so that transform is provided to render and render_node. We could also do a RenderOpts struct, then we could add other fields to it over time and provide a Default implementation...

Yes, a separate struct might me useful in the feature.


UI: Snapshot tree disappeared from time to time

Looks like a bug from https://github.com/rancher/longhorn-ui/pull/90

Fixed.


Error crashes app when cachedInfo is null

Figure out where this could happen and fix it - are we asserting its existence where we should not?

I set up an error boundary for now, will re open if this remains an issue.


A note an a feature without the feature gate?

On common/events/mod.rs:EventTypeProperties
/// NOTE: to use EventProperties on wrapper types, like Event, you must enable the "enum-dispatch" feature.
I think this is once meant to be but no longer valid, right?

Correct.


fix: update policy description

Issue
https://gravitee.atlassian.net/browse/APIM-2187
Description
A small description of what you did in that PR.
Additional context

Gravitee.io Automatic Deployment
🚀 A prerelease version of this package has been published on Gravitee's private artifactory, you can:

use it directly by updating your project with version: 1.3.3-apim2187-SNAPSHOT
download it from Artifactory here

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

1.3.3
GitHub release

Your semantic-release bot :package::rocket:


Throws when passing an empty array into an attribute

Hi there! I've been trying to write a PR for this but haven't quite figured out where the issues lays, so I thought I should write an issue while it is fresh and hopefully I can rubber duck myself into an answer.
datasource db {
  provider = "postgres"
  url      = env("DATABASE_URL")
}

generator client {
  provider      = "prisma-client-js"
  binaryTargets = "native"
}

model UserProfile {
  userID String @id @unique

  /// A one liner
  bio String?

  /// Hrefs which show under the user
  links String[] @default([])
}

Will throw due to   links String[] @default([]). It throws in:
  this.OPTION(() => {
      this.CONSUME(lexer.LRound);
      this.MANY_SEP({
        SEP: lexer.Comma,
        DEF: () => {
          this.SUBRULE(this.attributeArg);
        },
      });
      this.CONSUME(lexer.RRound);
    });

Which feels a bit odd because this.attributeArg eventually includes this.array which should be able to handle the empty array judging by the JSON parser in the playground which has the same definition code.
Converting it to links String[] @default([""]) passes, but would have different runtime behaviours.
Things I've tried:

Defining an 'empty array' parse rule for it
Adding array specifically to the attributeArg fn

Thanks for the report 👍  I'll take a look at this when I get a chance

@orta this is fixed in v0.4.2. Turns out I had an Array symbol in the lexer, which was '[]', intended to be used in the field parsing logic (e.g. for columns such as String[]). Anyway, the lexer was picking up @default([]) and assigning the Array symbol to it, which caused it to skip over the this.array rule.
I removed the Array symbol and switched the field rule to consume the '[' and ']' symbols individually.

Thanks @MrLeebo! (sorry I didn't get a PR though)


Attendance add dropdown to select gradebook category, fixes #110

This PR adds a Gradebook Category dropdown to Attendance's Grading settings. It's a feature we added recently at University Of Dayton, and we're sharing it with the community now. It depends on a couple of changes in Gradebook, which we've opened a Sakaiproject Jira to track: https://sakaiproject.atlassian.net/browse/SAK-47743
This PR's Attendance ticket is https://github.com/sakaicontrib/attendance/issues/110.

Can we merge this in?


Add bitcoin-core version 0.17.0.

Pull request to include version 0.17.0

Thank you very much @alingenhag!

Docker Hub already setup as well.

You are welcome.


Slapper + PiggyAuth

[03:02:23] [Server thread/CRITICAL]: "Could not pass event 'pocketmine\event\player\PlayerExhaustEvent' to 'PiggyAuth v2.0.1': Argument 1 passed to PiggyAuth\Main::isAuthenticated() must be an instance of pocketmine\Player, instance of slapper\entities\SlapperHuman given, called in phar:///home/gmplus/Server/plugins/PiggyAuth_v2.0.1 (3).phar/src/PiggyAuth/EventListener.php on line 192 on PiggyAuth\EventListener
[03:02:23] [Server thread/CRITICAL]: TypeError: "Argument 1 passed to PiggyAuth\Main::isAuthenticated() must be an instance of pocketmine\Player, instance of slapper\entities\SlapperHuman given, called in phar:///home/gmplus/Server/plugins/PiggyAuth_v2.0.1 (3).phar/src/PiggyAuth/EventListener.php on line 192" (EXCEPTION) in "/PiggyAuth_v2.0.1 (3).phar/src/PiggyAuth/Main" at line 149

Fixed.


copy of coords for Cuboid if no barycenter

self.barycenter and self.coords were the same object on this class when barycenter was not passed as an argument.

Thanks!


css attribute doesn't work without bind or interpolation

Is this a bug?
https://gist.run/?id=28368d0649a8af90180046525b698b2e
I would would have expected both to work.

I suppose it is a bug...though it doesn't make any sense to do that.
@jdanyow Would this be easy to fix?

Only reason why I noticed it was because in my grid I have a option where you pass in the style you want to add in a col-css attribute for the "simple html" column option.
So when someone tried to use just regular styling like I do in the box that fails, it didnt work, and they told me I had a bug :smile:
Was kinda unsure if it was a bug or just me, the hack works fine and they where happy, so isnt really a problem, just wanted to report it in case you guys didnt know.

It's "technically" a bug :) We will fix it. If it isn't a quick fix we may prioritize other issues higher for now though.

Np, focus on the important stuff, if anyone get a issue they can use the hack I used :-)
Keep up the good work, Aurelia made web programming fun  :+1:

could be: won't fix

Yeah. I don't think we're planning to fix this since the css attribute is really only for when you need to bind styles. It technically only exists because of a bug in IE (and maybe Edge too, can't remember anymore) that prevents using interpolation directly within the style attribute.


In error: "AssertionError: LivPrb is not a list or time varying distribution"

parameters = {'cycles': 0,
 'aNrmInitStd': 0.0,
 'aNrmInitMean': 6,
 'LivPrb': 0.9949620563926881,
 'PermGroFac': 1.0,
 'pLvlInitMean': 0.0,
 'pLvlInitStd': 0.0,
 'Rfree': 1.0,
 'TranShkStd': [0],
 'PermShkStd': [0],
 'CRRA': 3,
 'DiscFac': 0.99,
 'ex_post': None,
 'AgentCount': 1,
 'num_per_type': 1000,
 'RiskyAvg': 1.0916031021270372,
 'RiskyStd': 0.09296520824951145,
 'T_sim': 3000,
 'UnempPrb': 1.0,
 'IncUnemp': 0.0}

a1 = SequentialPortfolioConsumerType(**parameters)

a1.solve()

Gets me this error:
AssertionError: LivPrb is not a list or time varying distribution, but should be because it is in time_vary

This doesn't seem right.

This is correct. LivPrb is survival probability, and so it should be a
list. It depends on age, if we're doing a lifecycle model.
On Tue, Jul 18, 2023 at 2:34 PM Sebastian Benthall @.***>
wrote:

Assigned #1305 https://github.com/econ-ark/HARK/issues/1305 to @mnwhite
https://github.com/mnwhite.
—
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/issues/1305#event-9854075140, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADKRAFIN6QJUXFALLA6LBLTXQ3JMVANCNFSM6AAAAAA2OZMB4Y
.
You are receiving this because you were assigned.Message ID:
@.***>

Ok, glad this is fixed with the new parameters object.
@mnwhite I don't follow the logic there. It's not a lifecycle model in this
case.
On Tue, Jul 18, 2023, 4:48 PM Alan Lujan @.***> wrote:

In the new Parameters dictionary, If LivPrb is a float it should be
recognized as time invariant, if it's a list it should be length T_cycle
and recognized as time varying.
—
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/issues/1305#issuecomment-1640971659,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAQZEBXQFVW52EWNWR5JW3XQ3ZBJANCNFSM6AAAAAA2OZMB4Y
.
You are receiving this because you authored the thread.Message ID:
@.***>

It's not a lifecycle model, but LivPrb is a "lifecycle" attribute, so, it needs to be generated as LivPrb = [ 0.9949620563926881 ] because it's an infinitely repeated "lifecycle" of length one.

Ok. I'm following now.
It sounds like this is something that's hard-coded in the model which probably ideally is not hard-coded in the model, since we run into cases all the time where we want something that wasn't originally thought of as time-varying to be time-varying, etc.
I suppose https://github.com/econ-ark/HARK/pull/1240 is the fix for this.
Looking forward to that PR!

Right. For something like ConsIndShock or the portfolio model, most
parameters can be "harmlessly" moved between time_vary and time_inv on the
solution side. Because the simulation method is hard-coded with the base
assumptions about what's time-varying, additional work needs to be done on
that side. But with the simulation overhaul to be based on the "model
equations representation", whether a particular object has a t-subscript is
something that can/should be handled at runtime.
In ConsIndShock, the big exception is CRRA. As written, the solution code
assumes that CRRA is constant across periods, and uses this fact in
approximating the upper extrapolation. This should be fixed along side the
change to the timing of expectations, so that it checks whether next
period's (marginal) value function identifies itself as coming from the
standard consumption-saving problem. Or we just have a flag that can tell
it not to attempt clever extrapolation.
On Wed, Jul 19, 2023 at 10:21 AM Sebastian Benthall <
@.***> wrote:

Ok. I'm following now.
It sounds like this is something that's hard-coded in the model which
probably ideally is not hard-coded in the model, since we run into cases
all the time where we want something that wasn't originally thought of as
time-varying to be time-varying, etc.
I suppose #1240 https://github.com/econ-ark/HARK/pull/1240 is the fix
for this.
Looking forward to that PR!
—
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/issues/1305#issuecomment-1642180889,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADKRAFJKR7GEEOTEBMJDWV3XQ7UM3ANCNFSM6AAAAAA2OZMB4Y
.
You are receiving this because you were mentioned.Message ID:
@.***>


set WriterConfig.MaxRowsPerRowGroup not work

Hello
I set MaxRowsPerRowGroup=10 and then generated write, but it did not work. MaxRowsPerRowGroup is still the default value.
conf := parquet.DefaultWriterConfig()
conf.MaxRowsPerRowGroup = 10
write := parquet.NewGenericWriter(f, conf)

I found that there should be an error in the initialization configuration.
func (c *WriterConfig) ConfigureWriter(config *WriterConfig) {
	keyValueMetadata := config.KeyValueMetadata
	if len(c.KeyValueMetadata) > 0 {
		if keyValueMetadata == nil {
			keyValueMetadata = make(map[string]string, len(c.KeyValueMetadata))
		}
		for k, v := range c.KeyValueMetadata {
			keyValueMetadata[k] = v
		}
	}

	*config = WriterConfig{
		CreatedBy:            coalesceString(c.CreatedBy, config.CreatedBy),
		ColumnPageBuffers:    coalesceBufferPool(c.ColumnPageBuffers, config.ColumnPageBuffers),
		ColumnIndexSizeLimit: coalesceInt(c.ColumnIndexSizeLimit, config.ColumnIndexSizeLimit),
		PageBufferSize:       coalesceInt(c.PageBufferSize, config.PageBufferSize),
		WriteBufferSize:      coalesceInt(c.WriteBufferSize, config.WriteBufferSize),
		DataPageVersion:      coalesceInt(c.DataPageVersion, config.DataPageVersion),
		DataPageStatistics:   config.DataPageStatistics,
		MaxRowsPerRowGroup:   config.MaxRowsPerRowGroup,
		KeyValueMetadata:     keyValueMetadata,
		Schema:               coalesceSchema(c.Schema, config.Schema),
		BloomFilters:         coalesceBloomFilters(c.BloomFilters, config.BloomFilters),
		Compression:          coalesceCompression(c.Compression, config.Compression),
		Sorting:              coalesceSortingConfig(c.Sorting, config.Sorting),
	}
}

MaxRowsPerRowGroup should be ?
MaxRowsPerRowGroup:   coalesceInt64(c.MaxRowsPerRowGroup, config.MaxRowsPerRowGroup)

After I modified the code like this, I re-executed the code and obtained the expected rowGroup.
Does DataPageStatistics have the same problem?

Hello @Shinena1998, thanks for reporting the issue!
I believe you are correct, could you send a pull request to address the issue?


Rename pony-stable to tack

Folks are confusing stable with thinking its the stable release of Pony. We initially discussed renaming to stable but @jemc pointed out that could be confusing as well.
We would need to rename the repo, and any packages such as brew etc as well as check all documentation.
Please submit your thoughts on names below and we can vote on them. If you like a particular name please use the 👍 reaction.

I suggested barn.  Or maybe pony-barn?

silo

hay

remuda [ref]

tack

rodeo! 🐴

(Note that you should feel free to vote for more than one name, as you wish)

corral

saddlebag

trailer
Because it is used to transport ponies from some other place into your
project.
On Fri, Jul 13, 2018 at 6:02 PM, John Mumm<EMAIL_ADDRESS>wrote:

saddlebag
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ponylang/pony-stable/issues/75#issuecomment-404965253,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArmYXrKEWGSis_8nbiI4p3sugt5bQXRks5uGRkLgaJpZM4VLw3X
.

saddle

HorC for like horsie + c for compiler

The shorter the name, the better. @mfelsche's "tack" is short, related to horses and is also a verb that basically means "attach". Pretty smart, if you ask me.

ponyhof
German "cute" word for stable. There's also a saying "das Leben ist kein Ponyhof", roughly translating to "life is not a party". Yup, that's how we Germans roll. lässt das Mikrofon fallen

I think herd works well.

jockey

So far tack and rodeo are in the lead with six :+1: votes each at the time of this comment.
I did a quick search for name collisions:

Rodeo, A Native Python IDE for Data Science

Note that this software package does indeed have a executable named rodeo, which may clash in a user's PATH.

I couldn't find a name collision for tack that seemed like it might cause a problem.
As a result, I'm going to remove my vote for rodeo and keep my vote for tack.

This is a sad day for rodeo

Tack is the winner.

@dipinhora what is involved with the release process in changing the name that you know of?

While we're renaming stable, may I suggest renaming bundle.json to tack.json? We can temporarily support both (with a warning?) for a bit.

@plietar the bundle.json renaming sounds like a good idea.

Agreed.

I've created a branch to start working on this.
tack-json.
There's a PR of keeping track of progress.
https://github.com/ponylang/pony-stable/pull/91

any news?

@Akiyamka I am not aware of any news on tack. Are you aware that stable will eventually be superseded by corral?

@rhagenson I did not know this, thank you very much for the link


Support underscores in patterns

Like in the match thing? I think that'd be doable...

I haven't tried underscores in match. I meant something like lam _. blah.

@favonia Ah! yes.

I just checked and this works now, I must have fixed it at some point.


Threshold decryption plugin implementation for OCR2.

Can we merge this PR, please? @pszal are you OK to approve or still expecting some changes? Have we agreed to add a special "ErrNotFound" error type?

@pszal are you OK to approve or still expecting some changes?

I'm ok with merging and refining it later.


feature: use pytest to collect and execute test fillers

This PR (work in progress) ports the test-filling process implemented in the custom tf framework to the pytest framework. Pytest becomes responsible for collecting fillers (test functions) and managing their execution to generate fixtures from the spec defined within them.
This is achieved by:

Defining a test spec in a test function (as before),
Storing them from within the test function in a test-function-scoped helper pyest fixture, and,
Executing the stored TestSpec/filler (and writing the fixtures) in pytest's pytest_runtest_call() hook.

Pytest is a widely used Python framework and offers several advantages out-of-the-box:

Separation of test filler collection and execution; "pytest --collect-only" can be used to explore which fillers are available. Use "pytest --collect-only -v" for a list with docstrings.
Pytest will continue execution if a filler fails ("tf" stops upon error).
Parametrization of tests (the ethereum fork is also parametrized) which gives more context to test filler errors.
Easier filtering and specification of a subset of test fillers during development (e.g. "pytest -k warm_coinbase" or "pytest -k Shanghai").
Easily drop into the debugger via "pytest --pdb" upon test filler error.
pytest-xdist can be used to execute test fillers concurrently.

Some demos and screenshots of pytest with execution-spec-tests can be found here (but note that these were generated from a different and now very out-of-date branch).
Quick start for working with this branch:
git clone git@github.com:danceratopz/execution-spec-tests.git
git checkout feature/use-pytest-to-collect-and-hydrate-test-fillers
python -m venv ./venv
source venv/bin/activate
pip install -e .
pytest --collect-only
pytest -v
# or run a newly ported filler explicitly
pytest -v fillers/eips/eip3651.py

To do list for framework related tasks:

[ ] Port ReferenceSpec version checking (to reference_spec() fixture in conftest.py?)
[ ] Remove unused tf-related framework code (e.g., src/ethereum_test_filling_tool/filler.py)
[ ] Update README and docs
[ ] ...

To do list for porting test fillers to pytest format:

[x] fillers/eips/eip3651.py
[x] fillers/eips/eip3855.py
[ ] fillers/eips/eip3860.py
[ ] fillers/eips/eip4844/datahash_opcode.py
[ ] fillers/eips/eip4844/excess_data_gas.py
[x] fillers/example/acl_example.py
[x] fillers/example/yul_example.py
[ ] fillers/security/selfdestruct_balance_bug.py
[x] fillers/vm/chain_id.py
[ ] fillers/vm/dup.py
[ ] fillers/withdrawals/withdrawals.py

I think this is a great idea! My only request is to try and keep the core dev who doesn't write python on a regular basis in the back of your mind as you develop the filler interface. One problem with the retesteth fillers IMO is that they became a DSL that only those who were regularly writing them could use / understand. One goal of this project should be for core devs to i) come in and fairly easily understand what is and isn't being tested and ii) write tests for a proposal if your team is busy with other things or their proposal isn't yet a priority.


Incorrect behaviour of type guards?

Bug Report
🔎 Search Terms
type guard
🕗 Version & Regression Information
4.3.5
⏯ Playground Link
https://www.typescriptlang.org/play?#code/IYIwzgLgTsDGEAJYBthjAgEhCAHAolFAPZQICmAHhOQHYAmGhJZA3gFAIKiQzwIBzchGx4AwsXrkAFAEoAXAloBXALYhyUANzsAvu3Yo0GAPIBrUQSKkK1Ooyw4rLBBy5CRTiVLmKV6zVcEKGFlKFoEACYABmitBH19Q1R0BAA5YggAMWJlBktmGyoaBgwC6zZOQWFLbxkFJTUNNmDQ8IQAFmiO+MSDI1SAEWJVYABLWkKyYvsmCqD9AYwJVVGGDOzchlsSh2HRiamF-uJaSAQxsBW1+g2cvPoEAF4EaTBlEAArcnhFKYb3l8fohLghrsB1pl7tsngA+BCA778CaQCGwcjEABmYJGNzuW3oJzOiFGuH241oABViOUWAB9NImSl0gDqJgASgBpACSaQA4s9XpoWIpyYcKg1aTY4a4qlwxtjpJdwZDNg9pMLSLJZLKuHq9SEIGEIrRyAB3dJQglSqAaioAOlU5HQwCEsh0+v0+rlrSN7VNFvMNrtLEdzrArvI7r0RPOpLFVJpTimrI5PP5gpDpFFIwp-0UNue8LceoVQvmKIgaIx2JVtytDx1Jf1XENxqU5stavyyYqWagYZdbo9eq9ep9bf9naDvZY-cHEeHeiAA
💻 Code

abstract class HttpError extends Error {
  abstract getHttpCode(): number;
}

class OkHttpError extends HttpError {
  getHttpCode(): number { return 200; }
}

class NotFoundHttpError extends HttpError {
  getHttpCode(): number { return 404; }
}

class DomainError extends Error { }
class CommandNotFound extends DomainError { }

const isCommandNotFound = (subject: Error): subject is CommandNotFound => subject instanceof CommandNotFound

const mapDomainToHttpError_NOT_WORKING = (error: DomainError): HttpError => {
    if (isCommandNotFound(error)) {
        return new NotFoundHttpError(error.message);
    }

    return new OkHttpError(error.message); <-------- Property message does not exists on type `never`
}

const mapDomainToHttpError_WORKING = (error: DomainError): HttpError => {
    if (error instanceof CommandNotFound) {
        return new NotFoundHttpError(error.message);
    }

    return new OkHttpError(error.message);
}

🙁 Actual behavior
simple type guard and its plain body pasted in if statement do not behave equivalently
🙂 Expected behavior
simple type guard and its plain body pasted in if statement should behave equivalently

This is likely due to #17344 and intentional.
In general having "empty" classes is code smell, unless you make them nominally typed (have a private or protected member). There's no difference between Error and DomainError, they're essentially the same. And instanceof has a gotcha in TypeScript: Because classes with only public members are structurally typed, you can assign a matching object literal. instanceof will fail, but TypeScripts type-system will assume it will always match.
Also you extend Error wrong. see: https://github.com/Microsoft/TypeScript/wiki/FAQ#why-doesnt-extending-built-ins-like-error-array-and-map-work


"noindex" URL are listed in sitemap

My sitemap contains a lot of URL that have a Meta "noindex".
So webmastertools send me an alert.

Hi,
This is a sample to explain my issue :
You can see on my sitemap : https://buen-polvo.es/sitemap.xml some URLs like "https://buen-polvo.es/miembro_motera1_2408242.html".
But, if you open source code of this URL, you can see a Meta for bots with "noindex" :

This is inconsistent for Google because of we ask to Google Bot to index this URL (on sitemap), but when Google Bot try to analyse this URL, it see a noindex, so it can't index this URL.
I hope it's more clear now.
Regards

Thanks, this is very helpful. I'm really short on time lately and I'm unlikely to be able to address this until late April.
Hopefully you'll manage until then.

Thanks. I'll try to manage it waiting your update.
Regards.

Hi,
In my side, I created a script which is cleaning bad url, but it's very slowly, so I never update my sitemap.
Did you have time to look at this problem of noindex ?
Best regards.

I'll get on it in a few days.

Aaaand I'm done with final exams.
Expect the patch this week :sunglasses:

Greaaaat 👍
I look forward to testing it :)

I'm sorry, but I can't find your patch.

It's a work in progress. I thought this would be easier. A major problem is that with links I only need to match a single attribute (href), with meta tags, I need to match both the name and content. It's tricky to get right.

A cheap that you can apply yourself is to simply check if the meta tag string is present in the html but hard-coding the check here: https://github.com/knyzorg/Sitemap-Generator-Crawler/blob/0b89cd5f53b02472d33131a2ebb62396003bf8df/sitemap.functions.php#L367
But my regular expression skills are somewhat rusty and regular expressions were never meant to parse html.
The entire project was written back for when PHP installations had finicky support for parsing HTML natively, and should have become unnecessary with the release of PHP7... yet here we are. I will eventually re-write as a binary with a proper HTML parser and deprecate the project.

@wcmohler is working on it in #83.

@knyzorg
pull request #83 works as expected. but it should follow the links from noindex pages and add them to sitemap.xml file
example:
page A have "noindex" meta
Page A links to page B and Page C
Page B and  Page C don't have meta "noindex"
Result: Page A should be omitted but Page B and C should be added to the sitemap.xml file.


[ffigen] Fix some C warnings

Fixes https://github.com/dart-lang/native/issues/1822

coverage: 87.965%. remained the same
when pulling 22dca9b828af5b3273b8dc6fb2e129fed42ffec0 on cwarn
into 8c16b6c68da9589de2c57794fb861536c4f9279b on main.


Do not log the secret

Should avoid ever logging the user's secret -- causes trouble for debugging and makes it harder to get client run logs in benchmarks :/

Client secrets should now be hidden in Config and PingRequest structs so no more worries about that. Out of curiosity, do you think token_key fields from the server also be hidden?

the token key is fine since it's transient 👍


Make it possible to create mirror volumes

Hi,
It would be appreciated to be able to create mirror volumes thanks to your "mapr_volume" library.
Is this feature in your backlog already ?

@nladuguie Waiting for feedback from comment in pull request.

Merged to master


How to present conditional and looping with flowy?

Hi, I would like to create a flowchart tool with flowy to describe the program process. Then I got 2 problems about how to initiate arrows between blocks:

Conditional statement: suppose there was an if-else statement and 2 branches were created, then the subsequent blocks were not able to be merged into one since one block with 2 parents is not allowed.
Looping: similar problem. Is it possible to represent a loop with Flowy?

Maybe it was hard to have a clear description only with words. For example, is Flowy able to describe the following flowchart?

Currently Flowy doesn't have such features as it's meant to be minimal. Could implement those if I could think of a simple user behaviour to make such connections - would the blocks automatically come with that sort of logic (e.g. the Validate block loop, or in the case of if/else)? Are there only specific scenarios where this would happen or should it be possible to generate connections between blocks as needed?

I discovered this amazing project just 5 minutes ago, but I'm already considering replacing my Elsa Workflow designer (which is based on jsPlumb) with Flowy.
However, as @kafai97 describes, I'm looking for the ability to allow arbitrary connections be made between any two blocks. This enables modeling flowcharts to loop back, fork/join and multiple incoming connections to a given block as depicted here:

From what I've seen so far, Flowy automatically creates a connection when dropping a block from the toolbox onto another block on the canvas.
Perhaps the following UX could help:

User drops a block A onto the canvas.
User drops another block B onto the canvas (not on block A, but below/besides/arbitrary location).
Now the user wants to connect the two blocks manually. They do so by clicking/hovering the source block, after which available endoints (the little blue circles) appear. User grabs one of these endpoints and drags it onto the target block. A new connection is created after releasing the mouse.

It would work a little bit like this demonstration.
In the case of the designer requirements for Elsa Workflows, I would need the designer to allow each block to have an arbitrary number of handles/endpoints from which the user can drag connections to other blocks.
I guess the tricky part might be the automatic layout of blocks that you have currently in place. I noticed that as soon as I drop a block onto another block, a connection is automatically created and the block is automatically given a location on the canvas.  When you were to allow users to drop blocks without having a connection initially, it would probably be difficult to determine where the block should be located. On the other hand, perhaps it makes sense for disconnected blocks to not be auto-positioned at all, and just let them "float" - until the user connects this block to another one, after which Flowy would perform the automatic layout.
Is this something you would consider implementing?

I discovered this amazing project just 5 minutes ago, and I'm already considering replacing my Elsa Workflow designer (which is based on jsPlumb) with Flowy.
However, as @kafai97 describes, I'm looking for the ability to allow arbitrary connections be made between any two blocks. This enables modeling flowcharts to loop back, fork/join and multiple incoming connections to a given block as depicted here:

From what I've seen so far, Flowy automatically creates a connection when dropping a block from the toolbox onto another block on the canvas.
Perhaps the following UX could help:

User drops a block A onto the canvas.
User drops another block B onto the canvas (not on block A, but below/besides/arbitrary location).
Now the user wants to connect the two blocks manually. They do so by clicking/hovering the source block, after which available endoints (the little blue circles) appear. User grabs one of these endpoints and drags it onto the target block. A new connection is created after releasing the mouse.

It would work a little bit like this demonstration.
In the case of the designer requirements for Elsa Workflows, I would need the designer to allow each block to have an arbitrary number of handles/endpoints from which the user can drag connections to other blocks.
I guess the tricky part might be the automatic layout of blocks that you have currently in place. I noticed that as soon as I drop a block onto another block, a connection is automatically created and the block is automatically given a location on the canvas.  When you were to allow users to drop blocks without having a connection initially, it would probably be difficult to determine where the block should be located. On the other hand, perhaps it makes sense for disconnected blocks to not be auto-positioned at all, and just let them "float" - until the user connects this block to another one, after which Flowy would perform the automatic layout.
Is this something you would consider implementing?

I understand the need for this sort of functionality in some scenarios, but I just feel like this is not what this library is really about. I created this because all the other flowchart libraries were unnecessarily complicated, made it hard to place blocks which you then had to connect manually, and often required other dependencies like jQuery.
Flowy is just a simple library that allows people to quickly implement flowchart functionality in their projects with basic features, with very few lines of code. People are free to add more functionality to the library depending on their needs, but I like the base being as simple as possible, with only the most important features coming together with it.
What you are proposing is basically just turning this into a complex flowchart library, and having the same functionality that many other Javascript libraries already have - nothing unique about that. I don't really plan to do so, and in any case, it would be something separate from Flowy, most certainly.

I'm closing this issue as it deviates too much from the current library, and I can't think of any possible implementations that would work with it in the current state.


[bug 1202288] Update translation-related reqs

This updates translation-related requirements to recent, supported
versions.

Babel: 1.3 -> 2.0
dennis: 0.4.3 -> 0.6.1
polib: 1.0.3 -> 1.0.7
pytz: 2013.9 -> 2015.4
translate-toolkit: 1.11.0 -> 1.13.0

Also, I moved the following from vendor/packages/ to vendor/src/:

Babel
click
translate-toolkit

These are all translation-related requirements that needed updating. To test them, I did the following:

ran the unit tests
ran the smoke tests
ran ./manage.py extract and ./manage.py merge
ran ./manage.py lint --errorsonly locale/
create a fresh vagrant vm and run all the above in the vm except smoketests

Pretty sure that covers all the bases here. There were no code changes--just version updates. The diff is mostly me removing stuff from vendor/packages/.
I'm going to wait for Travis to greenlight this, then merge it and push it.

Green! Merging.


The extension does not change the JSON config, log seems to be ok

Hi!
I've added the JsonFile directive to the project, and it seems that it is replaced, however the installed file is unchanged. Some details:
I've added DeployAppSettings attribute by transforming harvested results. In obj's _AppComponents_dir.wxs:
        <DirectoryRef Id="INSTALLFOLDER">
            <Component Id="cmpN1YDbNWtqH.nEoZvn.aEtfLXHi8" Guid="*">
                <File Id="DeployAppSettings" KeyPath="yes" Source="$(var.AppPublishDir)\appSettings.json" />
            </Component>

added a few instructions to Product:
    <ComponentGroup Id="ConfigComponentGroup" Directory="INSTALLFOLDER">
      <Component Id="ConfigComponent" Guid="7732c259-55ad-45dc-89f1-81c6c6906f4f">
        <Json:JsonFile Id="test1" File="[#DeployAppSettings]" ElementPath="$.test1" Action="deleteValue" />
        <Json:JsonFile Id="test2" File="[#DeployAppSettings]" ElementPath="$.test2" Value="1" />

the appSettings.json in project's folder:
{
  "test1": 0,
  "test2": 0,
  "System": {

According to the log, the values are processed:
Wix4JsonFile:  Entered Wix4JsonFile CA
Wix4JsonFile:  Created PMSIHANDLE hView
Wix4JsonFile:  Created PMSIHANDLE hRec
Wix4JsonFile:  MSIHANDLE's created for Wix4JsonFile CA
Wix4JsonFile:  Getting Wix4JsonFile Id.
Wix4JsonFile:  Getting Wix4JsonFile File for Id:test1
Wix4JsonFile:  Getting Wix4JsonFile ElementPath for Id:test1
Wix4JsonFile:  Getting Wix4JsonFile Value for Id:test1
Wix4JsonFile:  Getting Wix4JsonFile Flags for Id:test1
Wix4JsonFile:  Getting Wix4JsonFile Component for Id:test1
Wix4JsonFile:  Updating Wix4JsonFile for Id:test1
Wix4JsonFile:  Created wifstream, C:\Program Files\MyApp\appSettings.json
Wix4JsonFile:  Using the following flags, 1, false
Wix4JsonFile:  Found ElementPath as $.test1
Wix4JsonFile:  Updated ElementPath [ to $.test1
Wix4JsonFile:  Updated ElementPath ] to $.test1
Wix4JsonFile:  Found create set to false
Wix4JsonFile:  About to delete value: |$.test1| with value(s) Wix4JsonFile:  About to delete value: |$.test1| with value(s) [0]
Wix4JsonFile:  Deleted the json $.test1
Wix4JsonFile:  created output stream
Wix4JsonFile:  dumped output stream
Wix4JsonFile:  closed output stream
Wix4JsonFile:  Getting Wix4JsonFile Id.
Wix4JsonFile:  Getting Wix4JsonFile File for Id:test2
Wix4JsonFile:  Getting Wix4JsonFile ElementPath for Id:test2
Wix4JsonFile:  Getting Wix4JsonFile Value for Id:test2
Wix4JsonFile:  Getting Wix4JsonFile Flags for Id:test2
Wix4JsonFile:  Getting Wix4JsonFile Component for Id:test2
Wix4JsonFile:  Updating Wix4JsonFile for Id:test2
Wix4JsonFile:  Created wifstream, C:\Program Files\MyApp\appSettings.json
Wix4JsonFile:  Using the following flags, 2, true
Wix4JsonFile:  Found ElementPath as $.test2
Wix4JsonFile:  Updated ElementPath [ to $.test2
Wix4JsonFile:  Updated ElementPath ] to $.test2
Wix4JsonFile:  Found create set to false
Wix4JsonFile:  Found 1 elements
Wix4JsonFile:  About to replace value: |$.test2| {1}
Wix4JsonFile:  Updating the json $.test2 with values 1.
Wix4JsonFile:  created output stream
Wix4JsonFile:  dumped output stream
Wix4JsonFile:  closed output stream
Wix4JsonFile:  Closed PMSIHANDLE hView
Action ended 16:05:08: Wix4JsonFile_X64. Return value 1.
Action ended 16:05:08: INSTALL. Return value 1.
Property(S): ALLUSERS = 1

However, after the installation, the file C:\Program Files\MyApp\appSettings.json is not modified  (it has both test1 and test2, set to 0).
Any ideas what is wrong?
Update:
I found that the replacement is working, but only when the current user is a read admin, i.e. the following MSI properties are set:
Property(S): AdminUser = 1
Property(S): MsiTrueAdminUser = 1

Assuming that there is a PS1 file that runs the installer, e.g. Install.ps1:
path_to_msi\Installer.msi /L*v Install.log

These steps result with modified JSON file:

Open the Windows Terminal with PowerShell as Administrator.
Run Install.ps1

These steps result with unmodified JSON file:

Open the Windows Terminal with PowerShell
Run Install.ps1
Click  Yes in UAC prompt (current user is in Adminstrator's group)

what version of the nuget package are you using? can you share the nuget link?

In the installed I have the following references:
  <ItemGroup>
      <PackageReference Include="Wix4JsonFileExtension" Version="<IP_ADDRESS>" />
      <PackageReference Include="WixToolset.Heat" Version="4.0.3" />
  </ItemGroup>

These are latest versions of these packages.
Also I found when Wix4JsonFileExtension does work: if the installer is called from terminal opened as Administrator, JSON file is updated in the installed folder. When the installer starts under the current user and opens UAC prompt, JSON file is not updated. Could it be that the Wix4JsonFileExtension's action is started under the user who called the installer and does not inherit admin's permissions?

Can you try this with the latest package now and see if it errors out when privs are missing?

The last fix is not helping, unfortunately.
I think this may help:

I've tried to test the fix, be the project does not build with an error:

The WiX Toolset v3.11 (or newer) build tools must be installed to build this project. To download the WiX Toolset, see http://wixtoolset.org/releases/

Hi,
Yer I noticed that after I released it, there is a new version that sorts it, please try again
Regards
Ben
On 16 Jan 2024 at 17:39 +0000, Alex Netkachov @.***>, wrote:

The last fix is not helping, unfortunately.
I think this may help:
image.png (view on web)
I've tried to test the fix, be the project does not build with an error:

The WiX Toolset v3.11 (or newer) build tools must be installed to build this project. To download the WiX Toolset, see http://wixtoolset.org/releases/
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

I've tested with <IP_ADDRESS>. It is a latest available at the Nuget at the moment.

Can you test with the latest version and confirm if it fixes the issue?


service mesh control plane

Short description:
Naming mistake on previous PR
More details:
What this PR does / why we need it:
Which issue(s) this PR fixes:
Special notes for reviewer:
Bug:

/approve

/verified


Set voltageChart min y-value to 3V

After #23 was merged, spend a couple weeks trying the new graph ranges and felt it could be better.
Quick data analysis supports increasing min voltage to 3v; fewer than 0.1% of nodes have a voltage less than this.
Now set the minimum voltage to 3V.

I think I'll leave this as-is for now. I like the idea of being able to see under 3v, even if it's not common. Will likely implement larger graphs in the future. Maybe full width and to see further back in time. Thanks though :)


walletconnect_dart: ^0.0.11 i am not able connect to metamask

SocketException: Failed host lookup: 'c.bridge.walletconnect.org' (OS Error: No address associated with hostname, errno = 7)

You need to update to a package that uses WalletConnect V2

I am facing same issue, Can u please elaborate more

You need to update to a package that uses WalletConnect V2

What package can i use please


rosdep: bad rule for python-pymodbus in python.yaml

What's going on here (debian: precise): https://github.com/ros/rosdistro/blob/master/rosdep/python.yaml#L951
Added in: https://github.com/ros/rosdistro/commit/453b138b5bc1da7a6972dd398a75998d0cb09e05
python-pymodbus:
  debian:
    precise: [python-pymodbus]
  fedora: [pymodbus]
  ubuntu:
    precise: [python-pymodbus]
    raring: [python-pymodbus]
    saucy: [python-pymodbus]
    trusty: [python-pymodbus]

Looks like a typo, that would never resolve for Debian.

Yeah, it's from when paul backfilled a lot of debian. It looks like he was
a little too quick on the copy and paste:
https://github.com/ros/rosdistro/commit/453b138b5bc1da7a6972dd398a75998d0cb09e05#diff-701c2f61a9f7ac8fc393ccc6630c8fa5R202


Move docs into the root directory

Move the PyTorch/ORT docs sources in the top level folder so that all of the libraries can be documented using this system

What's the motivation behind this change?

I added the motivation to the description


Adding the navbar in Home page

Hi,I am contributor of GSSOC'23, I just checked that in home page navbar is not working on mobile view . I want to added this feature in this project that's will be user friendly and attractive. As a front-end developer geek I can fixed this issue.it will be my nice starter in open source contribution, So Please assign me this issue. @akshitagupta15june
Thank you

some are working on it

ok, thank you @akshitagupta15june  for replying to me.
I'll try to figure out another issue or add new features and rise my request later.


convex_hull of points gives non-convex shape

Expected behavior and actual behavior.
I'm trying to get convex hull for a set of points created by projection to a plane. Many different cases occur throughout the code execution so I have no control on what will be the input for convex_hull method.  The problem occurs with these set of points:

Which looks like this:

A convex_hull from the multipoint object with the data above looks like this:

If order of the points is changed like this the result becomes a rectangle as expected:

Correct shape:

Steps to reproduce the problem.
My problem originated in pygeos, though it happens the same with shapely as well.
import pygeos
from shapely.geometry import MultiPoint
import matplotlib.pyplot as plt

# Original data, gives non-convex shape
obj_1 = [[-2.00000000e-01, -1.00000000e-01],
       [ 1.38777878e-17, -1.00000000e-01],
       [ 2.00000000e-01, -1.00000000e-01],
       [-1.38777878e-17, -1.00000000e-01],
       [-2.00000000e-01,  1.00000000e-01],
       [ 1.38777878e-17,  1.00000000e-01],
       [ 2.00000000e-01,  1.00000000e-01],
       [-1.38777878e-17,  1.00000000e-01]]

# Rearranged data, gives convex shape
# obj_1 = [[-2.00000000e-01, -1.00000000e-01],
#         [-1.38777878e-17, -1.00000000e-01],
#        [ 1.38777878e-17, -1.00000000e-01],
#        [ 2.00000000e-01, -1.00000000e-01],
#        [-2.00000000e-01,  1.00000000e-01],
#        [ 1.38777878e-17,  1.00000000e-01],
#        [ 2.00000000e-01,  1.00000000e-01],
#        [-1.38777878e-17,  1.00000000e-01]]

x1, y1 = zip(*obj_1)
plt.figure(1)
plt.scatter(x1, y1)

# Try with pygeos
obj_2 = pygeos.get_coordinates(pygeos.convex_hull(pygeos.multipoints(obj_1)))
x2, y2 = zip(*obj_2)
plt.figure(2)
plt.plot(x2, y2)
plt.scatter(x2, y2)

# Try with shapely
obj_3 = MultiPoint(obj_1)
x3, y3 = [z.tolist() for z in obj_3.convex_hull.exterior.coords.xy]
plt.figure(3)
plt.plot(x3, y3)
plt.scatter(x3, y3)

plt.show()

Operating system
Windows 10
Shapely version and provenance
Shapely = 1.8.5.post1, pygeos = 0.10

Same error with shapely 2.0 too. This is an upstream bug with GEOS, e.g.:
$ geosop -a "MULTIPOINT (-0.2 -0.1, 1.38777878e-17 -0.1, 0.2 -0.1, -1.38777878e-17 -0.1, -0.2 0.1, 1.38777878e-17 0.1, 0.2 0.1, -1.38777878e-17 0.1)" convexHull -f wkt
POLYGON ((-0.2 -0.1, -0.2 0.1, 0.2 0.1, 0 -0.1, 0 -0.1, 0.2 -0.1, -0.2 -0.1))

a workaround is to round off a few digits, e.g.:
from shapely import wkt

g = wkt.loads(wkt.dumps(obj_3))
assert g.convex_hull.is_valid

Thanks for the report @fmertilten, this robustness bug has been fixed upstream by @dr-jts in both GEOS and JTS. We can keep this issue open until the next shapely release with the next version of GEOS.

Happy to see there was an easy fix. The workaround suggested also works for my code, thanks for that @mwtoews .


Add support for custom buttons with attached dialogs

There is currently an issue where when a required field is not filled in or given, the API call still returns a 200 success, and so the SSUI returns back to the service with a message saying "Success! X field is required".
While testing this, an issue was also discovered where while ordering a service through a service template custom dialogs do not honor the required fields and allow submissions with "incomplete" forms. See git issue: #14
I'm also having a lot of issues with specs, so I figured I'd throw this WIP out there to see if travis works any better.

Tested PR with @eclarizio and found a few issues:

Button group names are appended with " | ServiceTemplate | <service_template_id>"
Button groups are hidden by the UI if the screen width is decreased.  (Exacerbated by item 1)
When the service is loaded the Custom Buttons are fully resolved and any dynamic dialog fields are processed at that time.  The dialogs should only be resolved when the custom button is about to be displayed and the dynamic fields should be resolved at that time.

Item 1 example

Item 2 example

Depends on: https://github.com/ManageIQ/manageiq/pull/7117

@gmcculloug I've fixed the issues mentioned above, but on Friday when I rebased I ran into issues actually getting the UI up and running so I couldn't actually test. Some of the specs are also missing tests due to that setback, but I figured I'd throw up what I have if you can test it.

@gmcculloug Alright, this should be good to go, just updated with the latest. Still trying to get my environment up and working but the code is ready.

Further testing found that refreshing a dynamic field for a custom button fails because it currently references the dialog assigned to the "provision" actions for the service's template.  The previous issues appear to be working at this point.

Currently waiting for https://github.com/ManageIQ/manageiq/pull/7147 to be un-wip'd and merged, since this now uses an API call introduced in that PR that was meant to fix the issue described above.

Reference https://github.com/ManageIQ/manageiq/pull/7330 above is an additional feature that will include the custom buttons that have been assigned via the "Automate/Customization" interface.

@dclarizio Verified in SSUI. LGTM :+1:

@dclarizio Looks good.


Add query_text to SQLServer integration failure to obfuscate debug log

What does this PR do?
When the SQL query obfuscation fails, we log the obfuscation error as a debug log but, with SQLServer, we don't include the query text like we do for postgres and mysql and it makes it very hard to identify and fix bugs in the obfuscator.
Motivation
Saw many obfuscation debug logs which aren't enough to help us fix the obfuscator:
2021-11-24 21:51:22 UTC | CORE | DEBUG | (pkg/collector/python/datadog_agent.go:128 in LogMessage) | sqlserver:6fc456edead9257c | (statements.py:169) | Failed to obfuscate query: at position <line>: bind variables should start with letters, got "

Additional Notes

Review checklist (to be filled by reviewers)

[x] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[x] PR title must be written as a CHANGELOG entry (see why)
[x] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[x] PR must have changelog/ and integration/ labels attached

Closing since this was discussed before and the query text could include PII/sensitive information that we can't log.


Request to add semikong-8b to ollama

I would like to deploy the semikong-8b model locally. Semikong is a large model fine-tuned from Llama focused on the semiconductor domain. You can find the model on Hugging Face at this link: “https://huggingface.co/pentagoniac/SEMIKONG-8b-GPTQ and the GitHub repository here: “https://github.com/aitomatic/semikong. The model is in the .safetensors format. I attempted to convert it using llama.cpp, but encountered the following error: “ValueError: Can not map tensor 'model.layers.0.mlp.down_proj.g_idx'”. I have not found a suitable solution for this issue. I would love to see this model added to Ollama. Thanks for the developers' hard work.
My English is not very good, and these sentences were generated using a translation tool.

Unfortunately llama.cpp doesn't suppot GPTQ quantization.  In order for the model be imported, it would have to be quantized from the original source, which doesn't appear to be on huggingface.  The 70B unquantized model is available.


Cannot execute because a node is missing the class_type property.: Node ID '#36'

I've tried to install the node via ComfyUI Manager, and it failed,
And I try to directly download the zip file and unzip in /custom_node, then when I try to run the sample workflow, it pop out the error says that "can't find the node",
Since I'm also using 秋叶整合包，and I try to download the node via 整合包， it still shows the error "node can't find"

I've already pip install the torch 2.3.0+cu121/cu118, torchvision 0.18.0+cu121/cu118 and deleted the newer version, still error.
To be honest, it's quite frustrated.

I try to create a brand new comfyUI, and only with the 3d pack,
using:
python311,
torch=2.3.0+cu121-cp311-cp311-win_amd64,
torchvision=0.18.0+cu121-cp311-cp311-win_amd64
torchaudiot=2.3.0+cu121-cp311-cp311-win_amd64
And set the system path: D:\StableDiffusion\comfyui_3D\ComfyUI_windows_portable\python_embeded
And add:
set currentDir=%~dp0
set pathToAdd=%currentDir%python_embeded\Scripts
set PATH=%PATH%;%pathToAdd%
to the run_nvidia_gpu.bat,
still failed

我尝试通过 ComfyUI Manager 安装节点，但失败了，我尝试直接下载 zip 文件并解压到 /custom_node 中，然后当我尝试运行示例工作流时，它弹出错误说“can't find the node”，由于我也在使用秋叶整合包，并且我尝试通过整合包下载节点，它仍然显示错误“node can't find” 我已经 pip 安装了 torch 2.3.0+cu121/cu118、torchvision 0.18.0+cu121/cu118 并删除了较新的版本，仍然错误。老实说，这很沮丧。

Me too

我尝试通过 ComfyUI Manager 安装节点，但失败了，我尝试直接下载 zip 文件并解压到 /custom_node 中，然后当我尝试运行示例工作流时，它弹出错误说“can't find the node”，由于我也在使用秋叶整合包，并且我尝试通过整合包下载节点，它仍然显示错误“node can't find” 我已经 pip 安装了 torch 2.3.0+cu121/cu118、torchvision 0.18.0+cu121/cu118 并删除了较新的版本，仍然错误。老实说，这很沮丧。

help me plz

我尝试通过 ComfyUI Manager 安装节点，但失败了，我尝试直接下载 zip 文件并解压到 /custom_node 中，然后当我尝试运行示例工作流时，它弹出错误说“can't find the node”，由于我也在使用秋叶整合包，并且我尝试通过整合包下载节点，它仍然显示错误“node can't find” 我已经 pip 安装了 torch 2.3.0+cu121/cu118、torchvision 0.18.0+cu121/cu118 并删除了较新的版本，仍然错误。老实说，这很沮丧。

help me plz

I create a complete new comfyui-native, and just install the 3D pack, ensure to install all the necessary requirements and the right version,  then you may still need to install the requirement one by one or solve the issue one by one and check the other issues, then you may get a runnable comfyui with the 3D pack,
hope it helps.. :)

Cannot execute because a node is missing the class_type property.: Node ID '#20'是什么原因


Do not include log4j in compile dependencies

Currently, the dependency tree is as follows:
+--- org.slf4j:slf4j-api:1.7.21 -> 1.7.25
|    +--- org.slf4j:slf4j-log4j12:1.7.21
|    |    +--- org.slf4j:slf4j-api:1.7.21 -> 1.7.25
|    |    --- log4j:log4j:1.2.17
Please don't include slf4j-log4j as a compile time dependency. Place it as a "runtime" dependency. Scoping details at https://maven.apache.org/guides/introduction/introduction-to-dependency-mechanism.html
Basically add runtime to you pom for org.slf4j:slf4j-log4j12. This is messing with the ability to use log4j 2.x in a project. API is correct to leave as compile scope.

commit 8efb28e10dbd1c56e1bbef0376083b8d43ba26f1
Note:  I will be switching to logback over the next few commits.


Add ability to automatically create a zip file when a folder is given

This would save one zip command :)

I'm working on this :)