Samples Consolidation

Fixes #2212
Proposed Changes
We should refine and consolidate this list to the following reduced set of samples that streamline the covered concepts and is representative of typical customer workflow needs. Sample 49 is all feature sample. It is renamed as 11 now. Sample 48 70 are excluded .no

Testing

@vishwacsena please suggest.

We have documentation that refers to sample 11 as it is.
https://docs.microsoft.com/en-us/azure/bot-service/bot-builder-howto-qna?view=azure-bot-service-4.0&tabs=cs
We'd need to update that alone with any changes we make here. So don't merge for now.

@vishwacsena  Please suggest.

@Kaiqb what changes if any are required to the current doc? https://docs.microsoft.com/en-us/azure/bot-service/bot-builder-howto-qna?view=azure-bot-service-4.0&tabs=cs
@johnataylor Apart from the basics of getting set up with QnA Maker KB, I do not see much in the doc which should not bear any impact due to this sample. Can you clarify what changes do we need to make to this PR to address any concerns?

Sorry accidently closed. We do want this PR in principle. We do feel the QnAMaker samples could be consolidated and that would make the feature more approachable and easier to learn.
However, the docs currently refer to code verbatium, for exampe the code here that shows code that would be deleted by these changes as far as I understand it.
Specifically the docs show adding a GetAnswersAsync call inline in the QnABot OnMessageActivityAsync code - this is a reference to the current sample 11. However the new code replaces this with a call to run the dialogs.
So this PR would make the docs and the sample out of sync.
There are a bunch of alternatives here: for example update the docs. Or we could decide that we think the super basic sample 11 is good and we like the docs and so we consolidate only the more advanced samples into a single sample. Ending out with two samples: basic and advanced.

@vipeketi Can you resolve the conflicts in this PR and also update to leave sample 11 unchanged, just consolidating samples 48 / 70 into 49.
Thanks

Abandoning this PR for changes moved to simple delete task,  fresh PR https://github.com/microsoft/BotBuilder-Samples/pull/2494.


Fix broken links after exporting + importing a Zulip organization with a new domain

Reproducer:

Mention a user in a message.
Do a manage.py export
Do a manage manage.py import on the result.
Click on the mention.

Some issues of this form will be fundamentally hard to fix (e.g. links to specific stream/user IDs), but if you export an organization using manage.py export and then import it via manage.py import, we will renumber all the message/user/stream IDs (as part of avoiding collisions on import).  This will result in instances of our mention syntax on the server being broken.
The right fix for this involves modifying fix_message_rendered_content to do the following:

[x] Edit the rendered_content HTML for rendered messages, changing the data-user-id/data-stream-id values to the appropriate ones post-import.  This is the piece that's high priority.
[ ] Potentially, edit the @**Tim Abbott|12345** style mentions in any message raw content, again replacing the user ID with the new value of that user's ID post-import.  This would potentially allow us to re-render the message much later without issues.
[ ] Potentially, add code to edit any links inside messages to narrows, re-mapping those to use the updated user/stream IDs.  This is likely a huge pain and not worth doing, but I'm writing it down because it would be useful if we did it, and it may be worth thinking about when we decide what parser approach we're going to use.

@zulipbot claim

Is it possible to dump the mapping between original and renumbered stream/user/message id while importing? So we can try to manually replace all old link with new one.


Server create example

Can you provide an example PHP script to create a server ?

Now the Readme contains example of server creation


Got Docker containers working

:tada: This PR is included in version 1.5.1 :tada:
Your semantic-release bot :package::rocket:


Add swift_version to spec

Specifies the swift version that will be used for the pod regardless of the swift version the app target uses.

Note: This feature was added on CocoaPods 1.4.0 (http://blog.cocoapods.org/CocoaPods-1.4.0/)

looks good, just waiting for CI to pass

hm CI failed at pod lib lint --quick

@hkellaway CI was using an older version of pods. Also, we could delete the gem install cocoapods in .travis.yml since bundler is already installing the version specified in the Gemfile


Add cleanup for tests on galaxy

Some schemas aren't cleaned up after running tests on galaxy. It seems to be related to test_store_failures and test_custom_schema tests.
Add appropriate cleanup of these schemas.

The reason for this issue is that dbt test suite is only cleaning up default schema while running test suite. There is already a GitHub issue in dbt-core repo for this: https://github.com/dbt-labs/dbt-core/issues/7030
For now, we could add some custom cleanup on our side, or maybe just manually clean up these residues on Galaxy, say, once a month, until root cause is fixed on dbt-core side.

It's addressed.


Support netty io_uring

related https://github.com/sofastack/sofa-rpc/issues/1349

Summary by CodeRabbit

New Features

Introduced an option to utilize io_uring for improved transport performance.

Refactor

Enhanced transport layer to dynamically select channel implementation for better flexibility and efficiency.

Tests

Added tests to ensure the reliability of the new dynamic channel selection mechanism.

看一下dubbo的issue Introduce I/O URing to netty4 remoting
就不引用了


[analyzer] Fix skipping report from plist

Lets suppose that we are running the CodeChecker analyze command by using
multi cores and we are also using a skip file.
If the compilation database contains two translation units which
different only in the -o parameter, it will produce the same plist file.
It is possible that we try to modify this file from 2 threads
to skip reports from header files in a non-thread safe way. This way
the plist files may be corrupted.
In this patch we will use portalocker to lock the file before touching
it to make sure that only 1 process is writing to this file in the same time.

@gyorb I have the following compilation database:
[
	{
		"directory": "/home/csordasmarton/projects/test",
		"command": "g++ main.cpp -o main1.o",
		"file": "main.cpp"
	},
        {
		"directory": "/home/csordasmarton/projects/test",
		"command": "g++ main.cpp -o main2.o",
		"file": "main.cpp"
	},
        {
		"directory": "/home/csordasmarton/projects/test",
		"command": "g++ main.cpp -o main3.o",
		"file": "main.cpp"
	}
]

So I compile the same source file but the output option is different.
When we generate plist files from it the output parameter will be removed from the build command before we hash it:
https://github.com/Ericsson/codechecker/blob/e010bb904cd0469034483df215d5890ea47fc500/analyzer/codechecker_analyzer/analyzers/result_handler_base.py#L118-L123
So the plist file name will be the same in all case.
By the way this problem was reported by a user.

I understand the problem I just do not see why the compile command uniqueing does not filters out the compile commands here, so in the example you mentioned main.cpp should be analyzed only once not three times. This is just extra analysis work on the same file.


Is this project 64 bit ready?

More a question than an issue:
Is this project ready for Google's new 64 bit requirement?
https://android-developers.googleblog.com/2019/01/get-your-apps-ready-for-64-bit.html

As long as your project supports it, it will automatically generate 64-bit so
abiFilters 'armeabi-v7a','arm64-v8a','x86','x86_64'


Test/Group id should stay stable across hot-restarts - because in "isolation mode" and with #130, we use id to identify tests across hot restarts

solution: #133


Help shows wrong location at first use

Tested versions
Doesn't work as expected:

v4.2.2.stable.official [15073afe3]
v4.3.dev3.official [36e943b6b]
v4.3.dev5.mono.official [89f70e98d]

Works:

v4.0.3.stable.official [5222a99f5]

System information
Godot v4.3.dev5.mono - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1060 6GB (NVIDIA; <IP_ADDRESS>12) - Intel(R) Core(TM) i5-3570K CPU @ 3.40GHz (4 Threads)
Issue description
When searching for help, the editor view doesn't show the correct location if the help page is not already open. Instead the bottom of the help page is shown.
Steps to reproduce

Make sure help view is not open (or present in script list next to editor view)
Press F1 to open "Search Help" window
Type "generate_random_bytes"
Click Open button

-> Crypto's help view is opened, but the view is focused to the bottom of the help page.

Press F1 again and search "generate_random_bytes" again

-> Now the view is focused to generate_random_bytes function.
Minimal reproduction project (MRP)
N/A

Thank you for reporting, this has already been solved:

https://github.com/godotengine/godot/issues/88281

Though a 4.2 cherry pick would be needed, marked it for cherry picking and closing this


Issues with production build & start

I am not able to deploy the app. So far I have set and tested the smtp and database url.
The db was successfully created and after some tsconfig problems I was able to run the prisma/seeds.ts to fill thedb with dummy data.
Locally I have the problem that login does not work. The seeds admin credentials have been changed though.I am receiving the OTP but im not redirected to the next page.
Is it because the JWT Secret is missing?
When I was trying to deploy the app to vercel, I have been getting too many errors with eslint and module import errors. I have fixed the eslint errors quick and dirty by temporary putting eslint off.
I want to know if there are common tips you may give to notice when building for production. It is all good when I run the development environment.
Is the current app limited in functionality or are the full features working already? (Because the demo app shows limited functionality aswell)
What is the idea behind the admin and storefront directories? I could not really read it somewhere.
Thank you in advance!

I've forgotten to mention the root directory step in the README.md file. There are 2 standalone Next.js apps in this project, therefore you need to deploy 2 apps in Vercel, one for each app in the apps directory.
There is a Root Directory option in your project's settings in the Vercel Dashboard where you can specify what directory Vercel should use to build your app.

I have been successful with the root directory tip. I could login to the admin panel after putting the jwt_secret key but the login in the storefront was not working anyway. Another thing I have noticed was that deleted products in the db haven‘t been updated in the ui straight after deletion.
Are these points fixed in the stable versions? Probably I have cloned the development version. Thank you in advance.

I have been successful with the root directory tip. I could login to the admin panel after putting the jwt_secret key but the login in the storefront was not working anyway. Another thing I have noticed was that deleted products in the db haven‘t been updated in the ui straight after deletion. Are these points fixed in the stable versions? Probably I have cloned the development version. Thank you in advance.

The project has only 1 branch, there is no development version. There are no problems with the login process and no problems with the CRUD operations on the admin dashboard. There is no caching implemented so if the document is deleted on database it should be instantly removed from the admin UI and the storefront, but I'll try and take a second look at it and fix it if there are any issues.

Nice thank you, I will also take a look too because I have deleted the vercel instances. Could you define strict steps to follow for a complete setup of the admin and storefront instances please, so I would check if I have forgotten something.


initial_commit

Somehow, something unknown broke the the CI/CD broke with the first PR version. Here a new PR:
First version to add the Raman Open Database File reader.
Tasks

[x] assign all fields via config file
[x] generalize reader to work with all present example (witec .yaml+.txt  AND .rod file)
[x] figure out fields, which can not be directly assigned - take a look into ROD documentation | Some information can be found in the README file for the .rod example.
[x] Make Issue to remove the assignment of "single-channel" for the "detector_channel_type" in NXoptical_spectroscopy avoid pytest warnings. This requirement will be relaxed, as this will be relaxed later in the nexus definitions

I want to update later the .rod file parser, to include all available meta data - with a seperate PR.

I have now included the config files as well wihtin the tests. Would be better if they are only at one location.
Maybe I can remove then the config folder in source? As this config file is spcified for the respective example, it may be better to place it there?

I have now included the config files as well wihtin the tests. Would be better if they are only at one location.
Maybe I can remove then the config folder in source? As this config file is spcified for the respective example, it may be better to place it there?

The point of having it in src/ is that you don't need to always use it as an input file to the reader. If you remove it from there, you e.g. always have to drop it to NOMAD. I thought the rod data has always the same structure? Then it makes sense to have a config as default. You can always drop another config file that takes priority over the default.
By the way, when I remove the config file in tests/data/rod, the tests still work. But not so for removing it from tests/data/witec

@lukaspie
"The point of having it in src/ is that you don't need to always use it as an input file to the reader. If you remove it from there, you e.g. always have to drop it to NOMAD. I thought the rod data has always the same structure? Then it makes sense to have a config as default. You can always drop another config file that takes priority over the default."
Ah alrigtht. Then I want to keep it that way. Sounds legit.
"By the way, when I remove the config file in tests/data/rod, the tests still work. But not so for removing it from tests/data/witec"
Hm okay? Ill try to take a look into it. Weird.


Zephyr upmerge 06032023

Zephyr upmerge based on upstream commit https://github.com/zephyrproject-rtos/zephyr/commit/6d9adf2e8af476819fce802d326a02226d98ed0c

I see the following which have been reverted and not reapplied:

e498ffb1e3 Revert "[nrf noup] samples: psa_crypto: update documentation"

sample is removed upstream by commit c63fb217601fb35903536db1c258571b8cd81b9b ("samples: tfm_integration: Remove psa_crypto")

bc6b19d20b Revert "[nrf noup] boards: arm: thingy53: Disable USB CDC added by MCUBoot"

Thanks, fixed


DSD-1462: Updating Storybook documentation for Template component

Fixes JIRA ticket DSD-1462
This PR does the following:

Updates Storybook documentation for all the Template components.
Compare with prod: https://nypl.github.io/nypl-design-system/reservoir/v1/?path=/docs/components-page-layout-template--template-app-container-component

How has this been tested?
Locally.
Accessibility concerns or updates

N/A

Checklist:

[ ] I have updated the Storybook documentation accordingly.
[ ] I have added relevant accessibility documentation for this pull request.
[ ] All new and existing tests passed.

Front End Review:

[ ] Review the Vercel preview deployment once it is ready.

I'll update the sidebar prop to be configurable. That's a good point about copying the code example. Similarly, I think it's because you can't update props on the Docs page anymore.


tools: enhance Ubuntu setup script

Now handles $DIR with spaces. Also fixes UV_BIN variable.
Description
Ubuntu setup script failed when running in a directory whose parent has spaces. This PR fixes the problem.
Verification
Script tested in a clean Ubuntu 24.04 install.

PR cancelled, will resubmit after more testing

PR re-opened, I had forgotten to activate Python's virtual environment


Windows 11 is not detected

platform.version = "Windows 10 Pro" 10.0 (Build 22621)

@JCKodel thats came from io.Platform.operatingSystemVersion
And linked issues:
https://github.com/dart-lang/sdk/issues/47334
https://learn.microsoft.com/en-us/answers/questions/586619/windows-11-build-ver-is-still-10-0-22000-194
You can parse it by yourself and based on build version exactly detect operating system version.


Fixing a minor typo

This was a small typo (it had a ";" instead of a ":" ) in my previous PR(#10) added as part of #9

lgtm!


Create initial Helm chart for Registration API

Initial helm chart committed:
https://github.com/EOEPCA/helm-charts-dev/commit/de17d9407f072bab08a08ce27c0b9826625aeb7f
https://github.com/EOEPCA/helm-charts-dev/releases/tag/registration-api-2.0.0-dev1

Deployment to develop cluster is blocked until we publish images on DockerHub

Unblocked now, going to rename the current registration-api to registration-api-v1x in argocd

Working helm chart:
https://github.com/EOEPCA/helm-charts-dev/releases/tag/registration-api-2.0.0-dev4
First deployment:
https://registration-api.develop.eoepca.org/

New upstream version of pygeoapi 0.18.0 has been added in helm chart and deployed on develop cluster
https://github.com/EOEPCA/helm-charts-dev/releases/tag/registration-api-2.0.0-dev5


Add localized strings with SwiftGen.

Fixes #25

Not sure why but there were odd merge conflicts. Attempting a new PR to see if that will resolve it.


Feat: Custom plugin management in Konnect

Description
Documentation for managing custom plugins through Konnect, using either the UI or the Konnect custom plugins API.
https://konghq.atlassian.net/browse/DOCU-2596
Testing instructions
Preview links:
https://deploy-preview-6164--kongdocs.netlify.app/konnect/gateway-manager/plugins/
https://deploy-preview-6164--kongdocs.netlify.app/konnect/gateway-manager/plugins/add-custom-plugin/
https://deploy-preview-6164--kongdocs.netlify.app/konnect/gateway-manager/plugins/update-custom-plugin/
https://deploy-preview-6164--kongdocs.netlify.app/gateway/latest/plugin-development/distribution/#install-the-plugin
Checklist

[x] Review label added
[x] PR pointed to correct branch (main for immediate publishing, or a release branch: e.g. release/gateway-3.2, release/deck-1.17)

The Custom plugin schema endpoints link on "Updating Custom Plugins" page renders a blank page with "Error Network Error" message - https://deploy-preview-6164--kongdocs.netlify.app/konnect/api/control-plane-configuration/latest/#/Custom Plugin Schemas

That's expected. Dev portal doesn't render in Netlify previews, unfortunately. You can see that link working on prod:
http://docs.konghq.com/konnect/api/control-plane-configuration/latest/#/Custom Plugin Schemas


add command still uses l/date|entry

This issue seems to already be fixed.

This issue seems to already be fixed.

Correct me if im wrong, I believe it has only been fix for addlog command(referring to #121), but not for add command

can we remove it entirely?

can we remove it entirely?
I think can is can, Im gna go ahead and remove it


Enhancement: Update phpunit/phpunit

This PR

[x] updates phpunit/phpunit

💁♂ For reference, see https://github.com/sebastianbergmann/phpunit/compare/6.5.8...7.4.4.

Codecov Report

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

@@           Coverage Diff           @@
##             master    #18   +/-   ##
=======================================
  Coverage       100%   100%
  Complexity        5      5
=======================================
  Files             2      2
  Lines            10     10
=======================================
  Hits             10     10

Continue to review full report at Codecov.

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


No error for setter with self-referential, implicit any argument

Bug Report
A setter can have an argument with a self-referential annotation, and this argument will implicitly have type any, with TS producing no errors for it.
🔎 Search Terms
setter, implicit any
🕗 Version & Regression Information
As far back as 4.3 (when we first let setters/getters in type literals).
⏯ Playground Link
Playground link with relevant code
💻 Code

declare var y: {
    set prop2(arg: typeof y.prop2);
};

🙁 Actual behavior
We don't get any errors, even with noImplicitAny set.
But arg has implicitly type any, because its type annotation refers to the type of the setter itself, and there are no getters.
🙂 Expected behavior
An error on prop2 or arg, at least under noImplicitAny.

Actually, an error is reported, but the error message is misleading and is reported as a global error because the logic attempts to report on the get accessor, but there isn't one. So, several things are wrong here.


⚠ Samsung has degraded performance

In a5e73fa, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2504 ms

Resolved: Samsung performance has improved in 6395af1.


docs clarification: token "leaks" into class?

regarding this section:
https://github.com/JoePenna/Dreambooth-Stable-Diffusion#-using-the-generated-model
when you use Natalie Portman person it is clearly showing a lot of your sks subject, without using sks in the prompt. is that because the training images actually "leak" into the class person?
thanks.
sks person

Natalie Portman person

Noted this is actually explained in the docs. Closing

Noted this is actually explained in the docs. Leaking does occur throughout the class. Closing


feat: remove delete_intermediary_models field from the compute_plan view

Description

Remove the delete_intermediary_models field from the compute_plan view.
Motivation and Context

This field should not be used anymore to control the cleanup of intermediary assets. Instead users should use the transient bool on task outputs.  It is already not sent by all the clients so we can safely remove it.
How Has This Been Tested?

e2e tests.

/e2e --help

Usage: /e2e [options] [help]

/e2e may appear anywhere as long as it is on its own line

Options:
  --refs <value>                                   Extra refs (branch or tag) with format REPO=GIT_REF,REPO=GIT_REF.
  Supported repositories: substra-hlf-k8s, orchestrator, substra-backend, substra-frontend, substra-tools, substrafl, substra, substra-tests, connect-ci.
  Example: /e2e --refs substra-backend=some_branch,orchestrator=some_tag (default: {})
  --tests-to-run, --tests <tests-to-run>           Comma-separated list of tests to run. Valid options: sdk,workflow,substrafl,frontend or NONE. (default: "sdk")
  --orchestrator-mode, --mode <orchestrator-mode>  Comma-separated list of orchestrator modes to run tests for. Valid options: standalone,distributed (default: "standalone,distributed")
  -h, --help                                       display help for command

End to end tests: :hourglass_flowing_sand: STARTING

--mode=standalone --tests=sdk,frontend

/e2e --mode=standalone --tests=sdk,frontend

End to end tests: :hourglass_flowing_sand: STARTING


The whole Model builder experience feels broken...

System Information (please complete the following information):

Model Builder or CLI Version: Model Builder <IP_ADDRESS>1402
Visual Studio Version (if applicable):

Visual Studio Professional 17.5.4+33530.505
Visual Studio Community 17.5.4+33530.505

Describe the bug
Well, this Model Builder tool seems to be very poorly written or maintained, it has several issues. The most easiest to see if those you get by just running the Machine Learning Model and follow the steps outlined in the getting started guide.
The two most easiest to find is that the scenario page is missing the images. The second is that after the initial training I always get following error (I get this both when I run the getting started guide and other scenarios).
Missing images:

Error:
Updating nuget dependencies... (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-25 08:32:52.5050 INFO Starting update NuGet dependencies async. (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-25 08:32:52.5361 INFO Project Miscellaneous Files cannot be accessed. It may be unloaded. (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-25 08:32:52.5361 INFO Couldn't find project named C:\Development\ML.NET\src\myMLApp\myMLApp.csproj (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-25 08:32:52.5361 INFO Installing nuget package, package ID: Microsoft.ML, package Version: 2.0.0 (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-25 08:32:52.5361 DEBUG Value cannot be null.
Parameter name: project
   at NuGet.VisualStudio.Implementation.Extensibility.VsPackageInstallerServices.GetInstalledPackages(Project project)
   at Microsoft.ML.ModelBuilder.Utils.Utilities.<InstallNugetPackageAsync>d__20.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.ML.ModelBuilder.ViewModels.TrainViewModel.<UpdateNugetDependenciesAsync>d__101.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
   at Microsoft.ML.ModelBuilder.ViewModels.TrainViewModel.<GenerateCodeBehindFilesAsync>d__100.MoveNext() (Microsoft.ML.ModelBuilder.Utils.Logger.Debug)

Expected behavior
Well, the model builder should work is at least what we can expect.

Can you share the target framework of myMLApp? There might be some edge cases in retrieving project name in some target frameworks.

I have tried both .NET 6 and .NET 7 with the same result. Interesting enough, I created a VM on azure with the preconfigured VS2022 professional installation, and there it works, but not on any of my personal computers.. I have verified the versions of both studio and the model builder, and they are both the same on all environments.
One difference though (if we can trust the logs) is that on my local computer it tries to add the 2.0.0 of the Microsoft.ML NuGet package, while on the VM it adds 1.7.0...
Here is the complete log from a simple run on my computer (with .NET 7):
2023-04-26 08:55:22.9684 INFO Set log file path to C:\Users\<user>\AppData\Local\Temp\MLVSTools\logs\MLModel1-6HLURX.txt (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:49.0951 DEBUG Start AutoMLService pid: 58756 (Microsoft.ML.ModelBuilder.Utils.Logger.Debug)
2023-04-26 08:55:49.3848 INFO start multiclass classification (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:49.3848 INFO Evaluate Metric: MacroAccuracy (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:49.3848 INFO Available Trainers: LGBM,FASTFOREST,FASTTREE,LBFGS,SDCA (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:49.3848 INFO Training time in second: 10 (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:52.4468 INFO |      Trainer                             MacroAccuracy Duration    | (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:52.4468 INFO |--------------------------------------------------------------------| (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:52.4468 INFO |0     FastTreeOva                         0,6643     2,5940         | (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:52.4754 DEBUG found best trial - trial id: 0 (Microsoft.ML.ModelBuilder.Utils.Logger.Debug)
2023-04-26 08:55:55.4973 INFO |1     FastForestOva                       0,7180     3,0120         | (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:55.4973 DEBUG found best trial - trial id: 1 (Microsoft.ML.ModelBuilder.Utils.Logger.Debug)
2023-04-26 08:55:57.7503 INFO |2     FastTreeOva                         0,7293     2,2410         | (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:57.7503 DEBUG found best trial - trial id: 2 (Microsoft.ML.ModelBuilder.Utils.Logger.Debug)
2023-04-26 08:55:58.8085 INFO |3     SdcaMaximumEntropyMulti             0,5000     1,0520         | (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.1734 INFO [Source=AutoMLExperiment, Kind=Info] cancel training because cancellation token is invoked... (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2923 INFO |--------------------------------------------------------------------| (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2923 INFO |                          Experiment Results                        | (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2923 INFO |--------------------------------------------------------------------| (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2923 INFO |                               Summary                              | (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2923 INFO |--------------------------------------------------------------------| (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2923 INFO |ML Task: multiclass classification                                  | (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2923 INFO |Dataset: C:\Development\ML.NET\data\yelp_labelled.txt               | (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2923 INFO |Label : col1                                                        | (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2923 INFO |Total experiment time :     9,0000 Secs                             | (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2923 INFO |Total number of models explored: 5                                  | (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2923 INFO |--------------------------------------------------------------------| (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2923 INFO |                        Top 4 models explored                       | (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2998 INFO |--------------------------------------------------------------------| (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2998 INFO |      Trainer                             MacroAccuracy Duration    | (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2998 INFO |--------------------------------------------------------------------| (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2998 INFO |2     FastTreeOva                         0,7293     2,2410         | (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2998 INFO |1     FastForestOva                       0,7180     3,0120         | (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2998 INFO |0     FastTreeOva                         0,6643     2,5940         | (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2998 INFO |3     SdcaMaximumEntropyMulti             0,5000     1,0520         | (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.2998 INFO |--------------------------------------------------------------------| (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:55:59.4861 INFO [Source=AutoMLExperiment, Kind=Info] cancel training because cancellation token is invoked... (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:56:00.8010 INFO Generate code behind files (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:56:05.6012 INFO

Copying generated code to project... (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:56:05.6152 INFO Copying MLModel1.consumption.cs to folder: C:\Development\ML.NET\src\Test2 (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:56:05.6152 INFO Copying MLModel1.training.cs to folder: C:\Development\ML.NET\src\Test2 (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:56:05.6152 INFO Copying MLModel1.evaluate.cs to folder: C:\Development\ML.NET\src\Test2 (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:56:05.6152 INFO COMPLETED (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:56:05.6152 INFO

Updating nuget dependencies... (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:56:05.6152 INFO Starting update NuGet dependencies async. (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:56:05.6501 INFO Project Miscellaneous Files cannot be accessed. It may be unloaded. (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:56:05.6501 INFO Couldn't find project named C:\Development\ML.NET\src\Test2\Test2.csproj (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:56:05.6501 INFO Installing nuget package, package ID: Microsoft.ML, package Version: 2.0.0 (Microsoft.ML.ModelBuilder.Utils.Logger.Info)
2023-04-26 08:56:05.6760 DEBUG Value cannot be null.
Parameter name: project
   at NuGet.VisualStudio.Implementation.Extensibility.VsPackageInstallerServices.GetInstalledPackages(Project project)
   at Microsoft.ML.ModelBuilder.Utils.Utilities.<InstallNugetPackageAsync>d__20.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.ML.ModelBuilder.ViewModels.TrainViewModel.<UpdateNugetDependenciesAsync>d__101.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
   at Microsoft.ML.ModelBuilder.ViewModels.TrainViewModel.<GenerateCodeBehindFilesAsync>d__100.MoveNext() (Microsoft.ML.ModelBuilder.Utils.Logger.Debug)

@dozer75 Can you share with us your VS version on local compute && azure? There might be a change in VS SDK which prevents model builder from getting project name

It is in the initial post, but here it is again:

System Information (please complete the following information):

Model Builder or CLI Version: Model Builder <IP_ADDRESS>1402
Visual Studio Version (if applicable):

Visual Studio Professional 17.5.4+33530.505
Visual Studio Community 17.5.4+33530.505

Yes, as said, everything has the same version on all my instances both local and Azure based.

Strange thing, I started a new test project today on my Azure VM and all of the sudden I get the same error as described there as well (but with 1.7.0 ML packages, which is still strange all compared)...


feat: add min-height in .rspress-doc

Summary

copilot:summary
Details

copilot:walkthrough
Related Issue

Checklist

[ ] I have added changeset via pnpm run change.
[ ] I have updated the documentation.
[ ] I have added tests to cover my changes.

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


feat(rust): hostname port on tcp inlet outlet

Current behavior
Some inputs accept ip addresses and ports but should be able to support hostname and ports. This PR enables hostname and ports for the tcp outlet command. Note: some command access inputs that don't support hostnames and ports, i.e. when specifying a bind address and port.

Proposed changes

Change tcp outlet to parameter to be String type
Change tcp inlet from parameter to be String type
Update usage of those parameters to enable String type
Created the HostPort struct that uses the Host struct within the url crate
Created a host_parser util function that validates input for port, ip:port, and hostname:port generating a HostPort struct
Implemented some conversions to reuse the host_parser function with the socket_addr_parser

Fixes #5563

Checks
Tests pass

[x] All commits in this Pull Request are signed and Verified by Github.
[x] All commits in this Pull Request follow the Ockam commit message convention.
[x] I accept the Ockam Community Code of Conduct.
[x] I have accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam-contributors repository. The easiest way to do this is to edit the CONTRIBUTORS.csv file in the github web UI and create a PR, this will mark the commit as verified.

@alcolmenar looks like there is a clippy warning that we must fix to pass the CI checks. We can merge the PR after that is fixed!

Looks like there's an error that popped up during these bats tests also. I'm working through it atm


Generics: Unable to expand a slice of functions to variadic parameter using a named generic type

What version of Go are you using (go version)?
go1.18beta2 - Reproducible on Go Playground using Go Tip
Does this issue reproduce with the latest release?
Yes
What did you do?
https://go.dev/play/p/G8gP91GzVKr?v=gotip
What did you expect to see?
I can expand a slice to be used for a variadic parameter. The parameter specifies a generic function type
What did you see instead?
✅  Passing in a single function (to a variadic generic function parameter) works
❌  Trying to expand a slice fails
type Option[T any] func(T) error

./prog.go:40:6: cannot use options (variable of type []func(opts *SayOptions) error) as type []Option[*SayOptions] in argument to Say

@ghostsquad This is because Go does not support covariance nor contravariance. https://github.com/golang/go/issues/7512


support pcm_s16le

in go2rtc, currently supported is pcm_s16be, can support for pcm_s16le included as well?  this would help compatability with certain wyze cameras which use pcm_s16le

Can you provide more info how you pass this data to go2rtc?
I'm not sure that pcm_s16be was right choose. And maybe pcm_s16le more common standard.

we are capturing the audio from alsa which is in pcm_s16le format, then we:
exec:ffmpeg -hide_banner -f alsa -ac 1 -i hw:0,0 -c:a copy f rtsp {output} to go2rtc`
...obviously s16le doesn't work at the moment but this is how we would do it.

Can you enable trace logs for rtsp module. I need to check SDP part from FFmpeg.

FFmpeg + RTSP + pcm_s16le has a problem. FFmpeg doesn't pass info about codec name and params...
I can see you set only one channel for your camera. Please check what the default sample rate:
ffmpeg -hide_banner -f alsa -ac 1 -i hw:0,0 -c:a copy -f null -

@AlexxIT
[root@WCV3:~]# ffmpeg -hide_banner -f alsa -ac 1 -i hw:0,0 -c:a copy -f null -
Guessed Channel Layout for Input Stream #0.0 : mono
Input #0, alsa, from 'hw:0,0':
  Duration: N/A, start:<PHONE_NUMBER>.800710, bitrate: 256 kb/s
  Stream #0:0: Audio: pcm_s16le, 16000 Hz, mono, s16, 256 kb/s
[NULL @ 0x775fc040] Requested output format 'null' is not a suitable output format
pipe:: Invalid argument
[root@WCV3:~]#

OK. Looks like I can detect codec based on bitrate. It always constant for specific sample rate + channels settings.

OK. You can try latest master version. Do you need binary or you can build it yourself?

If you can provide mipsle binary I can test

go2rtc_linux_mipsel.zip

@AlexxIT confirmed working!

https://github.com/AlexxIT/go2rtc/releases/tag/v1.6.1


how to get a response like 'Clean: true, ..' after script executed

greeter.py
#!/usr/bin/env python
# encoding: utf-8
from sys import stdin, stdout

# For each line FOO received on STDIN, respond with "Hello FOO!".
while True:
  line = stdin.readline().strip()
  if line == 'bye':
     print('bye')
     stdout.flush()
     break
  print('Hello %s!' % line)
  stdout.flush() # Remember to flush

start command
./websocketd -port 8889 --loglevel debug --ssl --sslcert=cert/server.crt --sslkey=cert/server.key --dir=script --devconsole
in this case , browser side got a response like 'Clean: false..'
i want to get a response with 'Clean: true,.., what's wrong with my script above, help pls

According to reference [0], closing cleanly requires that one end send the
bytes 0xff, 0x00
In python3, I guess that means: sys.stdout.buffer.write(bytes([0xff, 0x00]))
I'm not sure off the top of my head if websocketd handles this as expected,
as it does split frames on \n bytes, so it doesn't exactly treat the
subproc's outgoing bytes as raw bytes.
[0]
https://stackoverflow.com/questions/4812686/closing-websocket-correctly-html5-javascript#:~:text=To close the connection cleanly%2C a frame consisting of just a 0xFF byte followed by a 0x00
On Wed, Apr 19, 2023 at 10:52 AM standzhou2 @.***>
wrote:

greeter.py
#!/usr/bin/env python
encoding: utf-8
from sys import stdin, stdout
For each line FOO received on STDIN, respond with "Hello FOO!".
while True:
line = stdin.readline().strip()
if line == 'bye':
print('bye')
stdout.flush()
break
print('Hello %s!' % line)
stdout.flush() # Remember to flush
start command
./websocketd -port 8889 --loglevel debug --ssl --sslcert=cert/server.crt
--sslkey=cert/server.key --dir=script --devconsole
i want to get response like 'Clean: true..', but i got 'Clean: false',
what should i do to get response like 'Clean: true...h'
[image: image]
https://user-images.githubusercontent.com/119867339/233130640-4645150b-0fb9-4903-8c60-c632decab9be.png
—
Reply to this email directly, view it on GitHub
https://github.com/joewalnes/websocketd/issues/440, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHHDTKWSUVNPTA2J6QGGBTXCAC2TANCNFSM6AAAAAAXEIXV5U
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

thank you so much
i just have a try, it seems not work.

According to reference [0], closing cleanly requires that one end send the bytes 0xff, 0x00 In python3, I guess that means: sys.stdout.buffer.write(bytes([0xff, 0x00])) I'm not sure off the top of my head if websocketd handles this as expected, as it does split frames on \n bytes, so it doesn't exactly treat the subproc's outgoing bytes as raw bytes. [0] https://stackoverflow.com/questions/4812686/closing-websocket-correctly-html5-javascript#:~:text=To close the connection cleanly%2C a frame consisting of just a 0xFF byte followed by a 0x00
…
On Wed, Apr 19, 2023 at 10:52 AM standzhou2 @.> wrote: greeter.py #!/usr/bin/env python # encoding: utf-8 from sys import stdin, stdout # For each line FOO received on STDIN, respond with "Hello FOO!". while True: line = stdin.readline().strip() if line == 'bye': print('bye') stdout.flush() break print('Hello %s!' % line) stdout.flush() # Remember to flush start command ./websocketd -port 8889 --loglevel debug --ssl --sslcert=cert/server.crt --sslkey=cert/server.key --dir=script --devconsole i want to get response like 'Clean: true..', but i got 'Clean: false', what should i do to get response like 'Clean: true...h' [image: image] https://user-images.githubusercontent.com/119867339/233130640-4645150b-0fb9-4903-8c60-c632decab9be.png — Reply to this email directly, view it on GitHub <#440>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAHHDTKWSUVNPTA2J6QGGBTXCAC2TANCNFSM6AAAAAAXEIXV5U . You are receiving this because you are subscribed to this thread.Message ID: @.>

I looked a little more closely.
The gorilla/websocket library that websocketd uses allows sending a CloseMessage and other control messages, but websocketd doesn't expose an API to use that.
If you attempt to write 0xff, 0x00 or other control codes in the websocket script, websocketd ends up treating that as plain data, and wrapping that in a data packet when sending to the client. This includes the case of passing --binary to websocketd.
Maybe you can use some app-specific indicator, a normal data packet, for the server to communicate clean closure?

If you're under linux, you can try to use https://github.com/ctn-malone/websocketd-controller which can be used as a wrapper around your script and will forward stdout, stderr & exit event (which contain the exit code of the wrapped script)

What is difference that clean:true would make?

It’s just an easy way to get json events everytime something is outputted to stdout, stderr or if the program exits. I like the standard way of communicating it provides but I might be a bit biased 😅


chassis-server requires truststore.p12

When running the egeria-server-chassis-spring.jar file according to the README, the truststore file is required.  If it isn't in the local path, the server won't start.  Including truststore.p12 from the egeria repo fixes this problem, but it is not documented yet.  May be related to https://github.com/odpi/egeria/issues/3705

Will update docs (README) asap to reflect this (ie. when running the server-chassis from somewhere other than a local build, in the root directory of Egeria core repo where the truststore.p12 file exists, then we need to ensure there is a truststore.p12 file present -- eg. copy the one from Egeria core as a placeholder if none other to be used).


[python] add flag of displaying train loss for lgb.cv()

Adding flag on lgb.cv() for displaying train loss during training iteration.

Closed #1105.
Glad to see how very old issues are closing! 😃

I began to doubt in new parameter name. Seems that I was wrong when suggested to rename disp_* -> show_*. Both these words describe some appearance. There are even the following remark in show_stdv description:

Results are not affected by this parameter, and always contain std.

In contrast, new parameter affects performance because requires additional metric computation for train dataset.
I think we need a better word. What about calc(ulate), comp(ute), eval(uate)? I personally prefer eval.
BTW, loss should be replaced with metric for the clarity too.

I'm sorry that I didn't have much time because of moving.
I'll check today.


Counter value SSR

When using SSR the counter value is cached when it was first requested. Afterwards even the counter value is increased refresh of the page will provide old value since it comes from the cache. Use fetch policy none in the query to avoid this and test if that resolves the issue.
Initial SSR request

Incrased the counter from fronted

And refresh the page

Fixes "Invariant Violation: loadable: SSR requires @loadable/babel-plugin, please install it #466"
https://github.com/srinisoundar/loadable-server-side-rendering/blob/main/webpack.config.babel.js#L49C27-L49C46__


Screen generation is generating saga instead of style

Hi,
I just began a project with Ignite, but when I came to generate a screen called 'Home', ignite created me a saga in the HomeScreenStyle instead of a style.

By the way, in the screen generation I had some {{ props.name }} expressions instead of just HomeScreen. I couldn't launch my screen until I change all the {{ props.name}} for HomeScreen. (something related to the template engine I guess)

Thank you,
Elie

Hey Elie!  It looks like we fixed this a while ago, but there wasn't a new/released version of the boilerplate!   We're gonna circle up to make sure this doesn't happen again.  Thanks so much for bringing this to our attention!   Sorry for the silly issue.

@GantMan when can we get the fix for above? Copy/pasting screens and styles is annoying. Thanks

should be fixed in latest boilerplate.  Please test
ignite add ir-boilerplate-2016
let me know if that works.

We try to fix issues ASAP, but weekends always slow us down.   🎉  🎈

I got 'import RoundedButton' was not found in file. while running ignite add ir-boilerplate-2016. Was it completed? What was it suppose to return on success?

Just tested generating a screen and it looks like this has been fixed.

@valinaga Could you file a new issue over at https://github.com/infinitered/ignite-ir-boilerplate-2016 if you're still having this issue?


Migrate to Android 11 API for insets and UI visibility

Flutter currently uses a set of hacks and workarounds to determine visibility of onscreen keyboard and navbar when in fullscreen mode: https://github.com/flutter/engine/pull/18193
As of Android 11 (API 30), there now exists native ways of properly determining this behavior. We should migrate to targeting the latest SDK and implement the new API to avoid hacks in the future.

Need to do https://github.com/flutter/flutter/issues/56597 first

Would this help with https://github.com/flutter/flutter/issues/34678 and https://github.com/flutter/flutter/issues/40974?

https://github.com/flutter/engine/pull/18339
The above PR is WIP and still using the preview 4 version of API 30. We should wait until the full android API 30 before considering landing it.


Approach: Build on top of gzdeflate()/gzinflate()

Refs #1.

I've posted this ticket mostly for the reference only.
Both gzdeflate() and gzinflate() operate on a complete string buffer. This means you can not process individual chunks of this buffer. As such, they're not an option for a streaming (de-)compressor.
For the reference: The overhead of processing individual chunks may be overkill if your maximum buffer size is limited. Some protocols actually emit (smaller) individual frames which use a DEFLATE compression. In this case, buffering the whole frame and then using the complete buffer is probably okay.

Php7 supports incremental gzip

Thanks @staabm! Are we talking about #4 here or do you have anything other in mind? :)

Exactly, https://github.com/amphp/aerys/blob/master/lib/functions.php#L538. Glad you already found it yourself :-)

Thanks for sharing the link though! Having an example use case will certainly help implementing #4.


send timestamp to firebase

Hi!
I'm trying to update fields of a document through a json via node-red. This field in Firebase is timestamp format, but when sending a json:

{"date":1623966732}

, the field in Firebase becomes number with the value sent.
Is there a way to send a timestamp and Firebase recognize it as a timestamp?
Thanks!

Hi,
Can you give us a sample of the JSON data?
When I need a Timestamp I send a Javascript Date to the Firebase node and it is interpreted as a timestamp in Firebase.


Error occur when docker-compose up -d

➜  compose git:(master) docker -v
Docker version 17.06.2-ce, build cec0b72
➜  compose git:(master) docker-compose -v
docker-compose version 1.19.0dev, build 520fad3
➜  compose git:(master)

➜  compose git:(master) ✗ docker-compose up -d
Creating compose_kong-migration_1 ... done
Creating compose_consul_1         ...
Creating compose_kong_1           ... done
Creating compose_kong_1           ...

ERROR: for nginx-lb  Container "1c2f12be9311" is unhealthy.
ERROR: Encountered errors while bringing up the project.

i think must have something wrong with consul ,so consul log is ：

==> WARNING: Bootstrap mode enabled! Do not enable unless necessary
==> WARNING: It is highly recommended to set GOMAXPROCS higher than 1
==> Starting raft data migration...
==> Starting Consul agent...
==> Starting Consul agent RPC...
==> Consul agent running!
         Node name: '1c2f12be9311'
        Datacenter: 'dc1'
            Server: true (bootstrap: true)
       Client Addr: <IP_ADDRESS> (HTTP: 8500, HTTPS: -1, DNS: 53, RPC: 8400)
      Cluster Addr: <IP_ADDRESS> (LAN: 8301, WAN: 8302)
    Gossip encrypt: false, RPC-TLS: false, TLS-Incoming: false
             Atlas: <disabled>
2018-01-26T06:11:35.590465115Z
==> Log data will now stream in as it occurs:
2018-01-26T06:11:35.590473171Z
    2018/01/26 06:11:35 [INFO] serf: EventMemberJoin: 1c2f12be9311 <IP_ADDRESS>
    2018/01/26 06:11:35 [INFO] serf: EventMemberJoin: 1c2f12be9311.dc1 <IP_ADDRESS>
    2018/01/26 06:11:35 [INFO] raft: Node at <IP_ADDRESS>:8300 [Follower] entering Follower state
    2018/01/26 06:11:35 [INFO] consul: adding server 1c2f12be9311 (Addr: <IP_ADDRESS>:8300) (DC: dc1)
    2018/01/26 06:11:35 [INFO] consul: adding server 1c2f12be9311.dc1 (Addr: <IP_ADDRESS>:8300) (DC: dc1)
    2018/01/26 06:11:35 [ERR] agent: failed to sync remote state: No cluster leader
    2018/01/26 06:11:36 [WARN] raft: Heartbeat timeout reached, starting election
    2018/01/26 06:11:36 [INFO] raft: Node at <IP_ADDRESS>:8300 [Candidate] entering Candidate state
    2018/01/26 06:11:36 [INFO] raft: Election won. Tally: 1
    2018/01/26 06:11:36 [INFO] raft: Node at <IP_ADDRESS>:8300 [Leader] entering Leader state
    2018/01/26 06:11:36 [INFO] consul: cluster leadership acquired
    2018/01/26 06:11:36 [INFO] consul: New leader elected: 1c2f12be9311
    2018/01/26 06:11:36 [INFO] raft: Disabling EnableSingleNode (bootstrap)
    2018/01/26 06:11:36 [INFO] consul: member '1c2f12be9311' joined, marking health alive
    2018/01/26 06:11:39 [INFO] agent: Synced service 'consul'
    2018/01/26 06:13:15 [ERR] dns: recurse failed: read udp <IP_ADDRESS>:53: i/o timeout
    2018/01/26 06:13:15 [ERR] dns: all resolvers failed for {docker.for.mac.localhost. 28 1}
    2018/01/26 06:13:17 [ERR] dns: recurse failed: read udp <IP_ADDRESS>:53: i/o timeout
    2018/01/26 06:13:17 [ERR] dns: all resolvers failed for {docker.for.mac.localhost. 28 1}
    2018/01/26 06:13:22 [ERR] dns: recurse failed: read udp <IP_ADDRESS>:53: i/o timeout
    2018/01/26 06:13:22 [ERR] dns: all resolvers failed for {docker.for.mac.localhost. 1 1}
    2018/01/26 06:13:30 [ERR] dns: recurse failed: read udp <IP_ADDRESS>:53: i/o timeout
    2018/01/26 06:13:30 [ERR] dns: all resolvers failed for {docker.for.mac.localhost. 28 1}
    2018/01/26 06:13:32 [ERR] dns: recurse failed: read udp <IP_ADDRESS>:53: i/o timeout
    2018/01/26 06:13:32 [ERR] dns: all resolvers failed for {docker.for.mac.localhost. 28 1}
    2018/01/26 06:13:32 [ERR] dns: recurse failed: read udp <IP_ADDRESS>:53: i/o timeout
    2018/01/26 06:13:32 [ERR] dns: all resolvers failed for {docker.for.mac.localhost. 28 1}
    2018/01/26 06:13:34 [ERR] dns: recurse failed: read udp <IP_ADDRESS>:53: i/o timeout

New version consul healthy check failed ! how should i do can make it work again.

This looks like an issue in consul not in Kong. Can you paste your docker-compose.yml file?

Closing WRT lack of feedback


Upgrade to latest from upstream

lgtm as much as the massive merges can -- I suppose we should test it by just throwing this branch into a testing account?

lgtm as much as the massive merges can -- I suppose we should test it by just throwing this branch into a testing account?

Can we just test it by using this branch for the new testing vpc we are about to create?

For sure! I don't see why not.


When in dark mode the short keys dialog text is black instead of white or gray

Describe the bug
When in dark mode the table contents are black
To Reproduce
Steps to reproduce the behavior:

Start graphiql on localhost:4000
Open settings and switch to dark mode
Open Short Keys dialog
View table

Expected behavior
The table contents should be light gray like rest of the text in Dark mode
Screenshots

Versions:

PostgreSQL: [e.g. 14.1]
pg_graphql commit ref: ea2ab60

Additional context
Should get fixed with the latest version of graphiql

The IDE from your screenshot is from https://github.com/graphql/graphiql so that would be the appropriate place to open this issue
pg_graphql is the GraphQL execution engine that GraphIQL communicates with. It doesn't have a UI component

sorry I may have misunderstood. Are you asking for the GraphIQL version to be updated in the pg_graphql docker-compose setup?

Yes, that is correct, I don't see that issue in the live demo of graphiql,
that's why I meant maybe updating it would work.
I can do that with a PR
On Sat, 17 Dec, 2022, 2:04 am Oliver Rice, @.***> wrote:

sorry I may have misunderstood. Are you asking for the GraphIQL version to
be updated in the pg_graphql docker-compose setup?
—
Reply to this email directly, view it on GitHub
https://github.com/supabase/pg_graphql/issues/291#issuecomment-1355565389,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIKDLTVGK5ACRK6FLXGFMCDWNTG4BANCNFSM6AAAAAATBMUPNQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Got it! a PR would be great, thanks 🙌

resolved by #292


Update type hints in admin/ui/components.py

the contexts are mutable, therefore a MutableMapping makes more sense than a Mapping type
PyCharm is complaining that I can't just do ctx.update or ctx['foo'] = "bar" on a Mapping[str,Any]-type. I think rightfully so. If I understand the typings correctly, we want a MutableMapping here.
Please check the following:

[x] Do the tests still pass?[^1]

[x] Does the code comply with the style guide?

[x] Run make lint from the Wagtail root.

[x] For Python changes: Have you added tests to cover the new/fixed behaviour?
I feel that, to have tests for this, we'd need mypy first, which is not in the project yet, right?

[^1]: Development Testing
[^2]: Browser and device support
[^3]: Accessibility Target

thanks @lb-


[SPARK-29676][SQL] ALTER TABLE (RENAME PARTITION) should look up catalog/table like v2 commands

What changes were proposed in this pull request?
Add AlterTableRenamePartitionStatement and make ALTER TABLE ... RENAME TO PARTITION go through the same catalog/table resolution framework of v2 commands.
Why are the changes needed?
It's important to make all the commands have the same table resolution behavior, to avoid confusing end-users. e.g.
USE my_catalog
DESC t // success and describe the table t from my_catalog
ALTER TABLE t PARTITION (id=1) RENAME TO PARTITION (id=2) // report table not found as there is no table t in the session catalog

Does this PR introduce any user-facing change?
Yes. When running ALTER TABLE ... RENAME TO PARTITION, Spark fails the command if the current catalog is set to a v2 catalog, or the table name specified a v2 catalog.
How was this patch tested?
Unit tests.

Test build #113038 has started for PR 26350 at commit c4b96d6.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/17960/
Test PASSed.

@viirya @cloud-fan @imback82
could you please review?

Test build #113038 has finished for PR 26350 at commit c4b96d6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class AlterTableRenamePartitionStatement(

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/113038/
Test FAILed.

retest this please

Test build #113047 has started for PR 26350 at commit c4b96d6.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/17968/
Test PASSed.

Test build #113048 has started for PR 26350 at commit ef4af4f.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/17969/
Test PASSed.

Test build #113054 has started for PR 26350 at commit 851ee70.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/17975/
Test PASSed.

Test build #113047 has finished for PR 26350 at commit c4b96d6.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class AlterTableRenamePartitionStatement(

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/113047/
Test PASSed.

Test build #113048 has finished for PR 26350 at commit ef4af4f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/113048/
Test PASSed.

Test build #113054 has finished for PR 26350 at commit 851ee70.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/113054/
Test PASSed.

Thanks! Merging to master.

Thanks! @viirya @imback82 @dongjoon-hyun


Adding a bit more documentation regarding how to install language servers

Would appreciate it if you can explain  what exactly the cmd and  cmd is executable expect and where do they expect it in order to start properly.
Adding screenshots from java and tsserver

This is documented in vim.lsp.start_client()

well the servers are for sure installed, then the question would be how to set the path correctly.

If this was installed via lspinstall, as your logs indicate, you'll have to get support via their issue tracker. I'm not familiar with the inner workings of how that plugin overrides cmd to point at its install location. Otherwise, you can ask on our discourse https://neovim.discourse.group/

I did ask there first, this is i've got as a response
kabouzeid commented 8 hours ago
This is not part of lspinstall but part of lspconfig :)

Right, so I can tell you 100% your issue is that your language server is not installed correctly. LspInfo will tell you that. This repository doesn't install/manage language servers for you.
If you want to install the language server without lspinstall, you can follow CONFIG.md here or, better yet, for java, you should probably be using nvim-jdtls which includes installation instructions in the readme.

To elaborate, cmd is an array of the form { bin, arg1, arg2, ..., argN }. Where bin is either an absolute path or just the name of the binary if available in $PATH. So for tsserver it would be something like this:
cmd = {"typescript-language-server", "--stdio"}

Or
cmd = {"/usr/local/bin/typescript-language-server", "--stdio"}

i take it back it was actually the plugin's fault

now java should be a bit harder than an npm install :D


locker withdraw

Hope to help you with this little modfication !
Another question : you don't check if the token is the good token ?

It's a copy (not in storage), the reason it's written in the order it's written is to prevent reentrancy attacks (setting to 0 before transfer). I could add a claimed flag to the lock struct (adding storage) instead of making the copy, but practically it won't make much of a difference. Overall gas would be about the same.

ok. just afraid to lose amount if action transfer fail.


update tests to refactored code

nothing new. just updates based off of earlier refactoring work

@webdjoe please review and merge when ready


No custom hover for menu-only layout control

re https://github.com/microsoft/vscode/issues/150788
"workbench.layoutControl.enabled": true,
"workbench.layoutControl.type": "menu",

use the config above
hover over the layout control
:bug: native hover
(any other value for .type uses the custom hover)

This is actually because the SubmenuEntryActionViewItem extends BaseActionViewItem whereas the MenuEntryActionViewItem extends ActionViewItem.
ActionViewItem is what provides the custom hover capability.
Here is a quick summary of the ActionViewItem structure:
MenuEntryActionViewItem
ActionViewItem (handles hover delegate)
BaseActionViewItem (IActionViewItem)
Disposable

SubmenuEntryActionViewItem
DropdownMenuActionViewItem
BaseActionViewItem (IActionViewItem)
Disposable

**extends ActionViewItem**
ActionWithDropdownActionViewItem
MenuSeparatorActionViewItem
MenuEntryActionViewItem
ReactionActionViewItem
DropDownMenuActionViewItem
ToggleMultilineActionViewItem
QuickFixActionViewItem
NotebooKernelActionViewItem
StatusBarActionViewItem

**extends BaseActionViewItem**
ActionViewItem
SelectActionViewItem
DropdownMenuActionViewItem
BaseMenuActionViewItem
ToggleActionViewItem
DropdownWithPrimaryActionViewItem
DropdownWithDefaultActionViewItem
ActivityActionViewItem
StartDebugActionViewItem
ReplFilterActionViewItem
MarkersFilterActionViewItem
FolderSettingsActionViewItem
TestingExplorerFilter

There is no clear guidance on how to use these aside from the very recommended approach of using createActionViewItem when using a toolbar/actionbar. Ideally we could move to only one "BaseActionViewItem", then several reusable widgets that extend that.

ActionViewItem is what provides the custom hover capability.

I have changed that (which closed this issue) but I am also in for making this all simpler. Something tells me that we should have a higher level (workbench, platform) "toolbar controller" that you just create a for a menu (id). Internally it would use the lower level widgets but it would do all the lifting and glue that today various util-functions do

I still see native hover. I assume this is not expected.


[HELP] How to personal sign, make transaction with Coinbase ?

Describe the bug
I was able to connect coinbase, but cannot request a message signed
The error keeps saying "No matching key. session topic doesn't exist:
And I found out that cause we are connecting to coinbase data which does not contain topic
I'm currently using v.3.1.2
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Hello @mthinh, can I see your implementation?

`
Future<String?> personalSign({
required String message,
required String wallet,
String? chainId,
}) async {
await _w3mService.launchConnectedWallet();
if (_w3mService.session?.sessionService.isCoinbase == true) {
  final data = await _w3mService.cbRequest(
    chainId: chainId ?? _currentWalletChainId ?? ETHEREUM.chainId,
    request: SessionRequestParams(
      method: 'personal_sign',
      params: [message, wallet],
    ),
  );
  if (data is String) return data;
  return null;
}

final data = await _app!.request(
  topic: _w3mService.session?.topic ?? '',
  chainId: chainId ?? _currentWalletChainId ?? ETHEREUM.chainId,
  request: SessionRequestParams(
    method: 'personal_sign',
    params: [message, wallet],
  ),
);
if (data is String) return data;

return null;

}
`
Thanks @quetool , I just found out that we always have cbRequest method to handle coinbase case

cbRequest shouldn't be used and it will be deprecated soon. Let me take a look at your code.

This is what you need to do. I added some explanatory comments.
Future<String?> personalSign({
  required String message,
  required String wallet,
}) async {
  _w3mService.launchConnectedWallet(); // you don't have to await this method

  // not sure why you used `_app!.request()` here, what is _app?
  final data = await _w3mService!.request(
    topic: _w3mService.session?.topic ?? '',
    // chainId chas to be always the current selected chain, otherwise request might not work on several wallets if the chain was not approved.
    chainId: _w3mService.selectedChain!.chainId,
    request: SessionRequestParams(
      method: 'personal_sign',
      params: [message, wallet],
    ),
  );
  if (data is String) return data;

  return null;
}

cool, thanks, let me try to
@quetool

This is something on Coinbase side. Anyway this is not a personal_sign, right? Did the personal_sign example work? Any chance you can share the code for this transaction issue? I'm pretty sure it might be something wrong in the transaction object that Coinbase can't part properly.

@quetool , this is the code to make a transaction request, it works with other wallet (metamask, etc,..) but coinbase.
Btw, sign transaction is working now. Thanks

Wonderful! Would be interesting if could share the transaction object so I could debug and report to Coinbase if the issue is on their side

Hi @quetool , this is how transaction object looks like

And is this not working? The only thing I see is the missing 0x in the hex representation of the value but that shouldn't be an issue. It is weird anyway that is not added if you are using .toJson() method from package:walletconnect_flutter_v2/apis/utils/extensions.dart

Please remove the copyWith(data: transaction.data ?? '') from transaction and try again.

If I remove it, will face this error, cause data is required as string

btw, Im using eth sepolia to test , also try data = 0x0 but no luck

Seems like you are using and old version of our package. Could you please update to web3modal_flutter: ^3.2.2?

@quetool yes. currently using 3.1.2 cause I dont see any differences between 3.1.2 and 3.2.2 in term of coinbase stuff

But there are :)
You can check coinbase_service here https://github.com/WalletConnect/Web3ModalFlutter/blob/master/lib/services/coinbase_service/coinbase_service.dart#L222
Several other bug fixes and new features were made in following versions https://pub.dev/packages/web3modal_flutter/changelog
I would totally update if I were you :)

@quetool sure ah, thanks, let me update and retry

Thanks @quetool , I can see transaction confirmation popup now. Thanks a lot for your great help 🙏

Wonderful! I'll close the issue then! Thanks!


Add MacOS Support?

Feature request
MacOS is the mainstream development operating system.
Why?
I use Linux, Windows and MacOS at the same time, according to my different application scenarios.
Especially looking forward to adding support for MacOS. thanks.
Alternatives
No response
Additional context
No response

It supports macOS.
I developed tsx with it.
Does it not work for you?

A typescript file include top-level await an ES module.
In this case, it may break and crash, which is irregular.
The same code works fine in ubuntu、kali and windows.

But the ts code after build runs fine in MacOS.
node --no-warnings --experimental-vm-modules --experimental-specifier-resolution=node --experimental-wasm-modules dist/main.js"

Sounds like you're trying to report a bug.
Please follow the Bug report template and provide the necessary info for others to help you.


[WIP] Improved versioning part1 #277

Version stored in one place (version.mk). Probably still something wrong with the jenkinsfile + need to make sure everything gets cleaned properly when necessary.

@mmalohlava nope, I just commit it in sometimes by accident (bad habit of using -a when doing git commit), thanks for pointing it out.
Still some fixes are necessary (think we're not cleaning everything properly) but so far so good 👍

Ah also not sure how to link now from README to the latest bleeding-edge whl

s3up umi udelat latest link

@mmalohlava think this should pass and work as intended. Any obvious suggestions I should fix?
Two things:

in other projects I noticed the Jenkinsfile/gradle version is subbing (with tr) all + signs - not sure why that was done so I left it as + everywhere for now

still not sure how to point from the README to the latest version? Yes VERSION.latest bucket is getting updated but the file contains the build number - different every time. Know any way to go around this? I tried setting the whole S3 bucket to public so we could point users there but that doesn't seem to work? Other thing I'm considering is to remove the version number from the wheel file name?

Looks like what you are doing heavily conflicts with what I've done in the other PR.  Any idea how to proceed?  Are all of your changes real, or did you do alot of reformatting?

@mdymczyk regarding 1. question - not necessary, i think i did that just to avoid having + in version name to avoid any later collisions
regarding 2.question - it depends what you would like to achieve? Do you want to have link to the latest nightly/latest release? My idea is to always upload to private space (artifacts.h2o.ai) but then have additional step for promotion - for nightly it could be automatic, for releases manual click. The promotion will just take private artifacts and put them into public space and updates all links.

@mmalohlava in the readme we have a link to the bleeding edge build, which is a static link and worked until now because each build generated an artifact with the same name, i.e. s3://bucket/0.0.4/h2o4gpu-0.0.4.py (which was also put in 0.0.latest). Now the problem is we'll put into the 0.1.0 and 0.1.latest buckets files with the whole version, i.e. X.Y.Z+local_version. Not sure how to link to that from our main readme.
I.e. I will run a build and the artifact will be s3://bucket/0.1.latest/h2o4gpu-0.1.0+master.1.py but then after the next build it will be s3://bucket/0.1.latest/h2o4gpu-0.1.0+master.2.py. I was hoping I could make a symlink in S3 like s3://bucket/0.1.latest/h2o4gpu-latest.py which I could modify after s3up to point to the newly uploaded artifact (then I could just hard code in the readme that as the URL to download) but doesn't seem S3 supports that...

Implemented in another PR.


Make a BYOND -> OD savefile converter

A lot of SS13 servers rely on BYOND savefiles for all playerdata, so it would be nice to help servers migrate if we could steal look at one of the existing external savefile parsers and see if we could natively convert BYOND savefiles to OD savefiles inside an entire SS13 repo.

Here's a rust example but I'm pretty sure other savefile parsers exist too: https://github.com/tgstation/dm-savefile/

Note: IIRC, 515 is changing the savefile format a ton.
Mothblocks had the idea that it might be better to just write a tool in BYOND DM that can read the savefile and export it in the OD JSON format.

I think the only other savefile parser is a Python thing that doesn't work with lists =(

Here's a rust example but I'm pretty sure other savefile parsers exist too: https://github.com/tgstation/dm-savefile/

Off topic but I wish this existed when I was doing the bee savefile memes.
I have a basic converter made in python that could read basic info that made up pAI saves (https://github.com/ParadiseSS13/Paradise/blob/master/tools/pai_savefile_converter/pai_savefile_converter.py) but its not complete for the savefile spec, though its a good jumping off point.
https://github.com/20kdc/byond-data-docs/blob/master/formats/SAV.md is also invaluable info


New Contributor

Team: [TEAM NAME]
{
"title": [Content Writer],
"isLead": [true],
"payPerDay": 2800,
"createdDate": "2020-07-08T02:14:59.307535Z",
"contributor": {
"contributorId": [CONTRIBUTOR ID],
"displayName": [Chibuzor Eze-nwosu],
"githubUsername": [GITHUB USERNAME],
"profileImage": [PROFILE IMAGE URL (Slack or GitHub URL)],
"slackUsername": [Chibuzor]
}
}

@kristykjlee different details in Payment Processor of this member can you check and let me know the correct details. Check issue id 132 in Payment Processor. Thanks

@kristykjlee different details in Payment Processor of this member can you check and let me know the correct details. Check issue id 132 in Payment Processor. Thanks

@kristykjlee Is this issue the same as #1197 ??

@kristykjlee Is this issue the same as #1197 ??


Decimal separator not working properly

I need to be able to change '.' for ',' as my default decimal separator when copying query results from dbeaver, but although the option is present in the "Data Format" preference tab, it doesn't seem to have an effect on the data copied to the clipboard.
Is there a way to achieve this?
Thank you!

You can use Advanced Copy (ctrl+shift+c or from context menu). There you can choose target format (Display format is what you need).

It looks like this question was solved. It will be closed.

@IuVI could you clarify this a bit.
What is the actual problem?
Why copy/paste isn't working in your case? Please provide some samples.
If you need to change decimal separator char then you can change locale on Data format pref page.
If you need to use ANSI separator - just use simple copy or copy in database native format.

See the difference?
Same happens when you try to export table from Oracle to CSV using DBeaver. It keeps display formats so that data from the exported CSV become unreadable for other applications.

Looks like problem with grouping. In Russian locale thousands are divided by "space"(it is actually a special unicode character). You can disable grouping in Data Format prefs (for  numbers).

Well. yeah. It's not actually spaces, it's special characters. And yeah I can solve the problem with "spaces"
by changing Data Format prefs. Still the problem with ',' and '.' as fractional part separator remains during copypaste and it can't be solved in the DBmanager.
There's a lot of other issues relating to the difference in formatting comparing to default DBmanagers, but it's not related directly to this topic.
I see you have many issues to resolve next year.
Thanks for help! Merry Christmas and happy NY to the DBeaver devteam!

Thank you!
Please feel free to create new tickets for other formatting issues in DBeaver. If some of them can be fixed easily then this fix will be included in the nearest version.
Most of our existing open tickets are relatively complex - that's why we have so many of them.


android notifs: Fix regression where notifs dropped when avatar not fetched

IconCompat.createWithBitmap [1] expects a non-null Bitmap object.
Since abd605d0e, we've been directly passing the value returned from
our own fetchBitmap function, which sometimes returns null. It
seems like the symptom would be a failure to deliver a notification
when the value is null.
So, go back to not calling IconCompat.createWithBitmap unless we
get a non-null value from fetchBitmap.
See discussion at
https://chat.zulip.org/#narrow/stream/243-mobile-team/topic/sentry.20IllegalArgumentException.3A.20Bitmap.20must.20not.20be.20null.2E/near/1283842.
[1] https://developer.android.com/reference/androidx/core/graphics/drawable/IconCompat#createWithBitmap(android.graphics.Bitmap)

Thanks for debugging and fixing this! Looks good -- merged, with a small formatting tweak on top.

Thanks for the review and the formatting tweak!


preloadingStrategy  not working

🐞 Bug report
Command (mark with an x)

[ ] new
[ ] build
[x] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] xi18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?

 Yes, the previous version in which this bug was not present was: ....
Description
I used preloadingStrategy in app-routes.like this way
imports: [RouterModule.forRoot(routes, { preloadingStrategy: AppCustomPreloader })]
but it not works.
🔬 Minimal Reproduction
add preloadingStrategy in app routes under imports
🔥 Exception or Error
No modules get preloaded.
🌍 Your Environment
Angular CLI: 7.2.0
Node: 10.21.0
OS: win32 x64
Angular: 7.1.4

Hi, @akshata456,
it’s unclear what AppCustomPreloader consists off.
Can you setup a minimal repro please? You can read here why this is needed. A good way to make a minimal repro is to create a new app via ng new repro-app and adding the minimum possible code to show the problem. Then you can push this repository to github and link it here.
This might be related to your directory structure so its really important to get an accurate repro to diagnose this.
Note: Angular 7 is no longer supported see: https://angular.io/guide/releases#support-policy-and-schedule

Thanks for reporting this issue. However, you didn't provide sufficient information for us to understand and reproduce the problem. Please check out our submission guidelines to understand why we can't act on issues that are lacking important information.
If the problem persists, please file a new issue and ensure you provide all of the required information when filling out the issue template.


does not work with this manifest

manifest:
https://cmesk-ott-vod-prep-prot.ssl.cdn.cra.cz/0199/3211/cze-dash-s-axinom-sd1-sd2-sd3-sd4-hd1-hd2-P1BCeo6o.ism/.mpd

i'm on windows 10
this is how i am running it:
dash-mpd-cli.exe https://cmesk-ott-vod-prep-prot.ssl.cdn.cra.cz/0199/3211/cze-dash-s-axinom-sd1-sd2-sd3-sd4-hd1-hd2-P1BCeo6o.ism/.mpd --cookies-from-browser Firefox --quality best --ffmpeg-location "D:\coding\yt-dlp\ffmpeg-master-latest-win64-gpl-shared\bin\ffmpeg.exe" --mp4decrypt-location "D:\coding\yt-dlp\Bento4-SDK-1-6-0-640.x86_64-microsoft-win32\bin\mp4decrypt.exe" --mkvmerge-location "D:\coding\yt-dlp\mkvtoolnix\mkvmerge.exe" -o test.mkv

versions:
ffmpeg version N-111859-g3c397a1d46-20230830
mkvtoolnix version 80.0 Roundabout
mp4decrypt version in file path

it produces an uplayable mkv file, i cannot play it on VLC with error "VLC could not decode the format " " (No description for this codec)" and the mp4 encoding fails after a while and is corrupted:

this is the info about the mkv file:

i am also attaching the manifest file:
manifest.mpd.txt
let me know if you need additonal info, thanks.

what format do the description keys look like and how can i pass them to the downloader? i can get them from the network tab of the service i used to get this manifest, or from somewhere else if you tell me.

This is not the right place to discuss workarounds for DRM. You might be interested in the forums at videohelp.com.

okay, thanks.


Repack failure for nets with multiple destinations within a complex block.

Which part of OpenFPGA is buggy

[ X ] FPGA-Bitstream / Repack

Describe the bug
During repack, nets which start at the output of a primitive inside of a complex block (e.g. a LUT output) but have multiple destinations (e.g. the signal is both routed to the input of the flipflop and the output of the CLB) are added to the routed nets during repacking twice. This results in a failure, since the repacker thinks a Pin is overused. See following snippet from the log:
.
.
.
Pb route for Net d:
Source node:
	clb[0]/fle[3]/ble4[0]/ff[0].Q[0] -> clb[0]/fle[3]/ble4[0]/ff[0].Q[0]
Sink nodes:
	clb[0].O[6]
Pb route for Net c:
Source node:
	clb[0]/fle[3]/ble4[0]/lut4[0].out[0] -> clb[0]/fle[3]/ble4[0]/lut4[0].out[0]
Sink nodes:
	clb[0]/fle[3]/ble4[0]/ff[0].D[0]
	clb[0].O[7]
Pb route for Net c:
Source node:
	clb[0]/fle[3]/ble4[0]/lut4[0].out[0] -> clb[0]/fle[3]/ble4[0]/lut4[0].out[0]
Sink nodes:
	clb[0]/fle[3]/ble4[0]/ff[0].D[0]
	clb[0].O[7]
Added 6 nets to be routed.
Route failed due to overuse pin 'clb[0].O[7]': occupancy '2' > capacity '1'!
Route failed due to overuse pin 'clb[0].O[7]': occupancy '2' > capacity '1'!
Route failed due to overuse pin 'clb[0].O[7]': occupancy '2' > capacity '1'!
Route failed due to overuse pin 'clb[0].O[7]': occupancy '2' > capacity '1'!
Route failed due to overuse pin 'clb[0].O[7]': occupancy '2' > capacity '1'!
.
.
.

As you can see, the net c is added to be routed twice. This results in the overuse pin error.
I have prepared a zip file (repack_bug.zip) with an example task to reproduce the error. In the zip file, you also find an edited repack.cpp file, which fixes this error. Since I'm not very familiar with the code base this fix may be a bit "hacky". You can implement it as is or employ a better fix.
In my tests, this bug only occurs for nets, of which the sourcepin is within the complex block. If the sourcepin is an input pin of the block, repack seems to work fine.
To Reproduce
Steps to reproduce the behavior:

Clone OpenFPGA repository and checkout latest / any fairly recent commit
Execute OpenFPGA task provided in the zip-file above
See error

Expected behavior
Repack should succeed and generate a valid bitstream.
Enviornment:

OS:

[ X ] Ubuntu 20.04 (Running on a VM) --->

Compiler:

[ X ] gcc-9 (9.4.0)

Version:

[ X ] Current master

@Crossbomber It makes sense to me, feel free to create a PR.


WinAppDriver NuGet Package

Hey,
According to the release notes for the current stable release v1.2.1 it suggests the use of the WinAppDriver nuget package:

As with the previous WinAppDriver v1.1 release, it is recommended that you use the WinAppDriver NuGet package to take full advantage of the advanced input with the Actions API.

Is this still the case? According to the package information this package is still in preview mode and the version does not match the current WinAppDriver versions (v1.2.1 vs v1.0.1-preview).

Hey thanks for the reply. I can only see the preview one. Even in nuget manger within VS I can only find the package when I check Include prerelease option and it matches what I'm seeing over at nuget.org.
Where can I get the stable version that you are referring to?

I use Appium from nudget package for my tests. The stable version is the WAD server release.

Right I see thats what we're doing now. Thank you for clearing that up for me. That documentation is very misleading.


Remove the database support code

We won't be integrating the app with any database since contests keep getting postponed all the time, and it is not practical to maintain a database of all the upcoming contest images. So, we need to remove all the code added for using a database.

hi bro I hope you're doing great. after deleting all the code that uses the database, from where do you want to get your images?

Hi, Thanks for your interest in contributing!
We will get the images by first calling the GET /contests and immediately sending the data retrieved to POST /download. We're not storing them anywhere. But there are some redundant imports and initializations in init.py and router.py, which we planned to use for the database. We would like to remove that code now.
Feel free to reply, if you have more questions.


Blob Name

Currently, the player is given a random name from a list. This should be changed back to the default behaviour of allowing the player to enter their own name.

And no confederate flags...

We should probably re-enable all the options (no skins, no names, show mass, dark theme, and whatever the other one is).

Yeah.
CMDER IS AWESOME FOR WINDOWSSSSS
but iTerm2 is better.... :(

wut

Did you just accidentally paste something from somewhere else or what?

I've been developing on a Hackintosh lately and iTerm2 is amazing.  My little Windows laptop needs a good console but the only thing I found was Cmder.

And that has to do with the visibility of settings checkboxes because...?

no reason

Does what I just did count as internet censorship? I deleted something @Firedrake969 said and the conversation that followed.
I probably just did something illegal. xD

what conversation?  i can't remember xD

oops

Nearly done with name...

Name setting added: 1bc40c843b1f9e0e05519513ae9d8483764275f2
Other settings may be added later. If anyone else (@ScratchAgarioBots/modders) wants to help with those, feel free.


nologo inconsistency

The synopsis says the switch is --nologo.
The arguments say it's only available since .NET Core 3.0, but -nologo (note single dash) works using .NET Core 2.2.6.
The arguments say it's --no-logo (note extra dash), but that doesn't work in any version (tested on 2.2.6 and 3.0.0 preview 8).

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

ID: 910cd365-d63a-00dc-417c-db017f83bf6f
Version Independent ID: 5e495ddd-140e-de5c-a793-d4a9f88be51e
Content: dotnet build command - .NET Core CLI
Content Source: docs/core/tools/dotnet-build.md
Product: dotnet-core
Technology: dotnet-cli
GitHub Login: @mairaw
Microsoft Alias: mairaw

--no-logo should be a typo.
--nologo is indeed added in 3.0: https://github.com/dotnet/cli/commit/afcf7604ffb32e6c50bb0d91463f6f2550d772e2#diff-b61ae302364b0e662a9b728d70221acc
The reason -nologo works is because msbuild uses it.

IMO, --no-logo makes sense in the context of --no-build and --no-restore.

You need go to the cli repo opening a issue for that

@imba-tjd https://github.com/dotnet/cli/issues/12557


RFC: notation for invocation-wide core.shm names?

I've been heavily using the new core.shm facility to easily share FFI-based objects and there's a couple of improvements I'd really like, but want to discuss them first.
In the usual single-threaded architecture, the 'pid-local' syntaxes, like /freelists/struct_packet_# are quite handy to get a global structure that doesn't conflict with other snabb instances running at the same time, and also are easy to cleanup afterwards (something like for d in /var/run/snabb/*; do if [! -e /proc/$(basename $d) ]; then rm -r $d; fi; done).
When extending to a multiprocessing architecture, two new needs appear: "foreign" and "invocation-wide" objects.
Foreign objects
Many objects are process-specific,(i.e, "one packet freelist per process" => /freelists/struct_packet_# => /var/run/snabb/<pid>/freelists/struct_packet_#); but some could be "one per process, but sometimes read by another one".
For example, a simple statistics counter: there's one for each process, so it's fast to update with each processed packet. With a name like /stats/packets, resolving to /var/run/snabb/<pid>/stats/packets. But to get the aggregate statistics, the 'main' process has to read the counter of the 'working' processes.  For that, I've added an optional id argument to shm.resove() and shm.map() that defaults to the current pid.  When omitted, the behaviour is the same as usual: shm.map('/stats/packets', 'struct stats') maps an object for the current process, but makes it easy for the main process to ask shm.map('/stats/packets', 'struct stats', true, pid) for a readonly map of the stats object of the given pid.
Invocation-wide objects
There are some objects that are truly global (i.e. the DMA pool) or that don't belong to a single process, but are specified by name (i.e. an interprocess link) unfortunately, the "fully qualified" variant is not enough, since it would be the same for all SnabbSwitch invocations.  Either from one run to the next (making for very weird and non-funny failures by inheriting the 'just before crash' state of the previous test) or between simultaneous but independent runs.
The easiest way to identify program invocations is the process group id, which is set by the shell when executing jobs, and can be read with S.getpgid() from ljsyscall.
Currently, I'm just using the optional id argument discussed above, like shm.map('/dma_pool', heap_t, false, S.getpgid()), but it would be much nicer to have a name syntax for that and simply use shm.map('#/dma_pool', heap_t) or something like that.
The biggest obstacle right now is settling in a passable syntax.  So far, the only one I've been able to come up looks like:
foo/bar => /var/run/snabb/<pid>/<path>/foo/bar
/foo/bar => /var/run/snabb/<pid>/foo/bar
#/foo/bar => /var/run/snabb/<pgid>/foo/bar
//foo/bar => /var/run/snabb/foo/bar

Am I wrong in thinking it's too ugly?  Using :/foo/bar looks a little better but can be harder to spot with some fonts.
What about ditching the idea of "scope defining prefixes" and just replacing # with <pid> and $ with <pgid> ? then:
foo/bar => /var/run/snabb/<pid>/<path>/foo/bar
#/foo/bar => /var/run/snabb/<pid>/foo/bar
$/foo/bar => /var/run/snabb/<pgid>/foo/bar
//foo/bar => /var/run/snabb/foo/bar

... but also ...
foo/#/bar => /var/run/snabb/<path>/foo/<pid>/bar
foo/$-bar => /var/run/snabb/foo/<pid>-bar

is that too much useless flexibility?

So I suppose that we have separate questions of syntax and semantics here, i.e. what different kinds of scope should exist and how should we specify the right one.
For syntax one idea would be to specify the scope with separate functions map_global(), map_process(), map_local(), etc. That would be an alternative to encoding scope into the object name.
For semantics it seems worthwhile to support only a minimum number of simple kinds of objects. Do we really want to have global objects that are not owned by any one process? How are we going to synchronize access to them? Specifically, would it be simpler for each process to allocate memory from a private DMA pool instead of sharing that?
How would we solve these problems if Snabb processes were each started separately as peers instead of by forking children from a common parent? (Could we support both models i.e. forked children and links between separate processes? Could we make our implementation simpler in the process e.g. by minimizing shared state that will have to be synchronized?)

@javierguerragiraldez You can use snabb gc to cleanup stale runtime files: #558
Personally I feel adding new special syntax is overkill for this situation. We now have:
foo/bar => /var/run/snabb/<pid>/<path>/foo/bar  (for ourself)
/foo/bar => /var/run/snabb/<pid>/foo/bar
//foo/bar => /var/run/snabb/foo/bar             (for outside ourself)

We can use "//"..pid.."/path" to reference another processes resource, and depending on layout, define a function that builds a Group PID path for us, without having to support that specific layout in core.shm.

I am closing this because there is currently no activity. Please reopen it when necessary.


Upgrade hadolint to 2.11

Upgrade hadolint to 2.11 to support copying from images, not only stages and other improvements.
I didn't upgrade to 2.12 as it doesn't add any features. Let me know if you think it is worth upgrading to the latest.

I'd prefer latest version, even more as the change is simple. :+1:

Thanks!


Add Heatmap to the list of wizards

Thumnail designed at wizardsTumbnails.fw.png.

Wizard content (in order):

Marker size
Threshold (marker-fill-opacity)
Animated? If true:

Duration
Steps
Trails
Resolution

New position of Torque, Torque Category and Heatmap wizard buttons, after Bubble

The wizard has now been added with a feature flag


No valid logs found

I followed the instructions on the app after building it. I get this after running it:

No valid logs found

I'm running Xcode 11.4.1

Nuked derived data from the command line with Xcode closed and it seems to have resolved the issue.


explorer is not working for kylin parachain in localnet

Overview
explorer is not working for kylin parachain in localnet
Steps to Reproduce
Steps to reproduce the behavior:

run dive chain polkadot -p kylin --explorer --metrics
Add the endpoint in explorer

Expected Behavior
EXplorer to show block details

verified and closing this issue as fixed


Add planes

As an Airline Manager, I can add planes to our fleet so that I can assign them to flights.

Implemented


Fix colors not loading for events with offseason team numbers

Fixes a bug where some events included teams with very large numbers for their offseason robots, which the FRC Colors API does not accept as valid.
ex. team 1234B is represented as team 123400001, but should be mapped to be just 1234
Without this PR, the request to FRC Colors fails, and no colors are returned

Thanks for working on this! Is it also possible to handle the case where the bubble is completely white (maybe in the context provider so it is centralized?)

Sure, how do you think that should be implemented? Check if the luminance of the primary color is below a threshold (so, poor contrast against a white background), and then exclude it from the colors Map? (which would cause it to render as just the default blue color)

I think there should be some visual difference between having a blue color and having a light color. Also not sure what the ideal solution is. Maybe note which teams are light in the context and draw a thin gray/black outline in the figure?

I think ideally if we are drawing an outline for each point, it would use the secondary color for teams instead of just black/gray.
This is what the 581 scouting app does currently:
Basically every team with a white/very light primary color will have a secondary color that is much darker & easier to see on a white background.

I tried seeing if either the bubble or scatter plot allow setting the border color of the points, and neither seemed like they had an easy API to do so. Most likely it would need to be implemented in a fairly hands-on way, maybe styling the individual points on the DOM themselves by setting inline styles on them?

I'm planning on switching all figures to https://airbnb.io/visx soon, so maybe we can table this until then.

That works too. Should I make a PR (or update this one) to just set the points to be the default blue if the contrast is poor? As a temporary improvement.

Yeah that works for now. You can just add it to this PR I think

@avgupta456 are you able to review this please?

Instead of depending on color-luminance, can you just define and use one of these functions?
//Uses Rec. 709 (HDTV) coefficients
//R * 0.2126 + G * 0.7152 + B * 0.0722
var y1 = luminance.rec709(r, g, b);

//Uses Rec. 601 (PAL/NTSC) coefficients
//R * 0.299 + G * 0.587 + B * 0.114
var y2 = luminance.rec601(r, g, b);

And then just for simplicity, can you not use a for ... of loop so we don't need to bump ES5 yet. I'll do that alongside some more structural changes in a couple months.

@avgupta456 updated the PR


v7 - upgrade to angular 5 compiler

@leo6104 good idea - going to do that. Thanks.

waiting for angular5 support

@tigredonorte v6.6.0-1 should support Angular 5, although I don't have any large applications upgraded yet to make use of it. It still uses the v4 compiler/etc under the hood - but works with v5
This current branch is published under v7.0.0-3 - which uses the updated compiler (although not backwards compatible with v4), and includes changes like using lettable operators.
I'm a little pressed for time, but if anyone is able to give it a test - would be great. There was nothing specifically breaking in v4 to v5 - and was mostly just a matter of updating peer dependencies in the v6 branch.

I upgraded an existing app to angular 5, changed rxjs syntax and upgraded to angular-store v7.0.0-3.  No adverse effects that I can see.


[markdown] List indentation with tab

Hi,
I've started the Laverna note-taking webapp, which uses CodeMirror as its markdown editor. I noticed a behavior that seems to come from CodeMirror since I can reproduce it in the markdown demo.
Basically, when I'm writing a bullet list, pressing <enter> at the end of an item generates a new item below, which also starts with a bullet. Eg:
* my first item<enter>
* <cursor>

Currently, pressing <tab> only pushed the cursor further, without touching the bullet item:
* my first item
*    <cursor>

Now, it would be great if pressing <tab> could indent this new bullet item so that it actually becomes a sub-item:
* my first item
    * <cursor>

And conversely, pressing <shift-tab> would unindent the bullet item.
Thanks!

The continuelist addon should be a good starting point for an implementation of the requested behavior.

Indeed, you can add this as optional behavior to continuelist if you want, and submit a pull request, but I'm not taking feature requests, so I'm closing this issue.