Message column duplicates if Attributes contains Message.

During migration from the old code base, we are going to switch for OTEL. We are trying to use Aspire Dashboard for migration steps as a great toll for visualizing our Logs, Metrics, and traces.
We are goiung to keep our logs in the same format we currently have to keep backward compatibility. Everything works well except logging page. We faced the issue that Message column duplicates as a result filtering stopped to work as well as vilualization.
Repro steps:
using System.Diagnostics.Tracing;
using Microsoft.AspNetCore.Mvc;
using OpenTelemetry.Logs;

var builder = WebApplication.CreateBuilder(args);
builder.Logging.AddOpenTelemetry(logging =>
{
    logging.IncludeFormattedMessage = true;
    logging.IncludeScopes = true;
});
builder.Services.Configure<OpenTelemetryLoggerOptions>(logging => logging.AddOtlpExporter());

var app = builder.Build();

app.MapGet("/sample", ([FromServices] ILogger<Program> logger) =>
{
    const string message = "Hello World";
    IReadOnlyList<KeyValuePair<string, object?>> eventData = new List<KeyValuePair<string, object?>>()
    {
        new KeyValuePair<string, object?>("Message", message),
        new KeyValuePair<string, object?>("ActivityId", 123),
        new KeyValuePair<string, object?>("Level", (int) EventLevel.Informational),
        new KeyValuePair<string, object?>("Tid", Thread.CurrentThread.ManagedThreadId),
        new KeyValuePair<string, object?>("Pid", Environment.ProcessId),
    };

    logger.Log(LogLevel.Information, 0, eventData, null, formatter: (_, _) => null);

    return message;
});

Actual result:

Expected result:
Only one Message column exist. Message from the Attributes can be renamed or added new Column Body.

Hi
I took a look at this issue, and I don't agree with the recommended fix. It's valid to have an attribute in a log message called Message. It shouldn't override the log body message.
For example, log.LogInformation("Error occurred: {Message}", "Test message"); should result in the UI showing Error occurred: Test message. It shouldn't just be Test message.
You're seeing this log message:
System.Collections.Generic.List`1[System.Collections.Generic.KeyValuePair`2[System.String,System.Object]]

because Microsoft.Extensions.Logging calls ToString() on the state if there is no formatter provided. You need to change the call to log to include the value you want as the log message body: logger.Log(LogLevel.Information, 0, eventData, null, formatter: (_, _) => message);. Or the value passed as state can return it in ToString(). This has nothing to do with Aspire dashboard. It's just displaying the information sent to it.
I think there are some improvements here. Filtering for "built in" columns should include attributes with the same name. e.g. Message uses the body message and a message attribute. Also, the dashboard shouldn't show duplicate names in the dropdown.

@JamesNK, yes, we already handled adding formatters. It works, but as a result we have two Message columns.


Trouble using beatExtraction() from command line

I was able to call stFeatureExtraction and it extracted 4 files, 2 csv's and 2 npy's. Do I need to pass one of these to beatextraction somehow? I can't decipher how to use this function. Thanks for any help
Here is the command I used:
python audioAnalysis.py beatExtraction -i black-widow.wav
Traceback (most recent call last):
File "audioAnalysis.py", line 440, in
beatExtractionWrapper(args.input, args.plot)
File "audioAnalysis.py", line 51, in beatExtractionWrapper
F = aF.stFeatureExtraction(x, Fs, 0.050 * Fs, 0.050 * Fs)
File "/home/luis/Documents/Mood/pyAudioAnalysis_beats/audioFeatureExtraction.py", line 572, in stFeatureExtraction
curFV[2] = stEnergyEntropy(x)                    # short-term entropy of energy
File "/home/luis/Documents/Mood/pyAudioAnalysis_beats/audioFeatureExtraction.py", line 48, in stEnergyEntropy
subWindows = frame.reshape(subWinLength, numOfShortBlocks, order='F').copy()
ValueError: cannot reshape array of size 4400 into shape (220,10)

I had the same problem. Then I discovered that the audio that I was using was stereo, and we required a mono audio file.

Convert your stereo signal to mono signal.
signal = audioBasicIO.stereo2mono(signal)

Convert your stereo signal to mono signal.
signal = audioBasicIO.stereo2mono(signal)

Just in case anyone is seeing this now, the syntax has been updated to: signal = audioBasicIO.stereo_to_mono(signal)


v4: Forms cleanup

This branch aims to overhaul a decent amount of forms styles and documentation with the hopes of getting it near final. It's still WIP and has a few more todos (noted below).
Quick screenshot

Changes

#17498, #18130: Removes horizontal padding on .form-control-label so it avoids overriding grid-based form layouts and renders correctly in stacked forms.

#18515: Add .form-control-{sm/lg} classes to match small and large input classes, namely for vertical alignment of label and input in horizontal form groups.

#18702: Updates form validation docs to include basic examples for situations in which you'd want to use each state.

#18703: Drops .text-help for .form-control-feedback, designed specifically to only display temporary form validation feedback text in addition to normal help text.

#18704: Updates form validation docs to include examples of validation feedback text alongside always-present form field help text.

Todos

[x] #17248, #17266, #19277: Fieldsets vs divs in form groups
[x] #18390: Consider renaming .radio and .checkbox
[x] #18573: Consider renaming .form-control-label for it's more contextual use cases (e.g., only horizontal/grid forms and not necessary with radios/checkboxes as shown in #18389).
#19004: Color contrast and color blindness on validation colors Punting on this for another time
[x] #19139: Validation states on custom forms (needs select menus and file inputs)—maybe cherry-pick that PR into this one?
[x] #19771: Form control sizing not applying to <select>s.

There's also this (again) https://github.com/twbs/bootstrap/issues/19004 but that would require revisiting all the $brand-colors

@petetnt I'll add it to the todo list and see what I can do :).

Note https://github.com/twbs/bootstrap/pull/19277 which supersedes https://github.com/twbs/bootstrap/pull/17266

Decent update to this today:

Renamed .radio and .checkbox to .form-check.
Renamed .radio-inline and .checkbox-inline to .form-check-inline. Thought about trying to connect those classes more, but they're too different.
Fixed #19771 so that form control sizing classes work on <select>s again.

Have some more fiddling to do yet, but getting there.

Got the bulk of this done, but still have more changes to do as follow-up.

I see a lot of stuff is commented out, not sure where the issue is regarding this?
Screenshot:


Bug-Fix: No error message is shown on selecting a question

Fixes
This PR resolves the following issues

When a suggestion gets selected on clicking Get Answer no longer show the error message instead the application works as expected
The big error in console for unique key is also resolved

Changes

Modified autocomplete
Generated suggestions based on example questions with unique id
Tackled unique key error on console

Issue
closes #86
Screenshot
As the application first need to load tweet because of private_api_key I'm unable to demonstrate the result. But In production it will resolves those problems.

Good find Sagar and nice fix

Thanks 🥰


.Net: Remove Memory property from SKContext

Motivation and Context
Different skills and functions require different types of memory. Some skills might just need local, high-access stores while others need remote, authenticated access. Some are powered by embedding (vectors) indexed in the cloud, while others may be generating lightweight embeddings locally, on the fly. We cannot assume that memory is "one size fits all", or that even all the skills/functions in a single Plan use the same memory provider.
Description
Instead of surfacing one single 'Memory' instance to all skills, we're peeling this back and simplifying somewhat. In this first pass, the Memory property of SKContext is removed. The updated guidance will be to instantiate Skill classes with the memory provider you'd like them to have access to. This also opens the door for the TextMemorySkill to be registered several times in the kernel (with different aliases), granting access to different memory stores.
BREAKING CHANGE: though this is not expected to have a large impact, updates will be required for anyone:

using SKContext.Memory: will need to pass it into the function via an alternate route; likely Skill constructor injection
or those using the new SKContext() constructor and explicity passing the optional ISemanticTextMemory: creating a new SKContext with the new constructor is not advised, as this is typically done by a call through the Kernel. The new SKContext constructor is planned to be hidden in a future release.

Contribution Checklist

[X] The code builds clean without any errors or warnings
[X] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[X] All unit tests pass, and I have added new tests where possible
[ ] I didn't break anyone :smile:

Closing, reopening as merge from feature/ branch in sk repo

Food for thought.
I saw that the suggested implementation in this change for the MemorySkill was injecting the Memory when the skill was constructed not on usage/context basis.
Currently in our implementations there's no way to access our available AIServices based in the context of execution of a skill and IMHO we should think in a scenario where we should be able to get a service from within a Semantic Function depending on the context it is being executed.
i.e: Memory Skill might use different memories/text completion services when executed from different IKernels using the same ISkillCollection...

Food for thought.
I saw that the suggested implementation in this change for the MemorySkill was injecting the Memory when the skill was constructed not on usage/context basis.
Currently in our implementations there's no way to access our available AIServices based in the context of execution of a skill and IMHO we should think in a scenario where we should be able to get a service from within a Semantic Function depending on the context it is being executed.
i.e: Memory Skill might use different memories/text completion services when executed from different IKernels using the same ISkillCollection...

Yeah good call. To do this currently, you would create several TextMemorySkill instances and name them with different skill names (aliases).
The single memory provider is the core problem we're (starting to) fix with this PR. This change is one of likely several (to keep it "focused").
Additional changes ahead:

Moving memory skill into its own package, so it can evolve and grow faster than SK.Core
Consider having a collection of memory services registerable in Kernel, instead of just one (may or may not be needed)


Investigate Upcoming Conferences to Attend

[ ] 2016 IEEE International Conference on Big Data (Big Data 2016): http://cci.drexel.edu/bigdata/bigdata2016/CallPapers.html
[ ] Twelfth International Conference on Open Repositories, OR2017: http://or2017.net/ (@rgrp keynoted and ran a workshop at the last conference and would be interesting to see how that went and whether it is worth going again).  Location: Australia. Deadline for contributions: 20 November 2016
[ ] Research Data Network workshop: 30 November https://research-data-network.readme.io/docs/3rd-research-data-network-st-andrews-university-december-2016

@jobarratt

RDA 9th Plenary: https://rd-alliance.org/plenaries/rda-ninth-plenary-meeting-barcelona

Force2017: https://www.force11.org/meetings/force2017

IDCC2017: http://www.dcc.ac.uk/events/idcc17/submissions

PASIG NYC: http://www.pasignyc.org/

@jobarratt I'll move these to Hubspot

@jobarratt moved to Hubspot


Add interface to create a patch rather than going through git

This would be equivalent to git commit. In fact it should just be an alias. The suggestions we have received around this is that adding it would make the workflow actions fully complete and cohesive within git-ps.
I see where this stance comes from but it is defiintely lower priority in my mind.

This has been integrated into mainline therefore I am closing this issue.


[AsyncStorage] Using undefined type for key is not supported.

Current behavior
I'm using AsyncStorage to store some strings. I'm connected with my iPad and getting this warning in the Metro Bundler console.
[AsyncStorage] Using undefined type for key is not supported. This can lead to unexpected behavior/errors. Use string instead.
Key passed: undefined
Wierdly I'm only getting those on my iPad, not on my iOS simulator.
Expected behavior
Everything seems to work. Just want to understand where it comes from?
Environment

Async Storage version: 1.8.1
React-Native version: 61.5
Platform tested: iOS

I'm also getting this error alongside, but only on iPad again. I felt like it was about some entries that were in my manifest.json prior. so i wiped it with .clear() but the problem persits.
I found this thread: https://github.com/facebook/react-native/issues/9743
which describes the same issue with AsyncStorage. They provided a solution but it didnt help me to fix my problem.

Hi @farbexot ,
Can you provide some repro steps?

Hi Krizzu,
yes I found the problem. I am using items from a FlatList and have a function that fills up the rows in order to maintain proper layout. The renderItem calls AsyncStorage.getItem() and here the name of the items were undefined as they came from the pesudo-items.
I just added if(item !== undefined) before getItem()
Thanks for your help!


[Languages] Using the AES algorithm, encrypt in JavaScript and decrypt in C#

Topic Suggestion
Using the AES algorithm, encrypt in JavaScript and decrypt in C#
Proposed article introduction
An asymmetric encryption algorithm, the Advanced Encryption Standard (AES), is used. Joan Daemen and Vincent Rijmen, two Belgian cryptographers, created the algorithm. AES offers a block length of 128 bits and key lengths of 128, 192, and 256 bits, and was meant to be efficient in both hardware and software. AES Crypt is also entirely free and open-source software. Because it is open-source, a number of people have contributed to the program and evaluated the source code to guarantee that it secures data securely.
Web-based apps are often utilized in today's society, and we are subject to a variety of threats. To prevent these, we can encrypt data at the client level, and then decode it at the server level when the user submits the information. In this article, I will look into details on how we can use the AES algorithm concept to  encrypt in JavaScript and decrypt in C#
Key takeaways

Creating solution.
Adding AES JavaScript file.
Adding controls on Forms.
Writing JavaScript for Encryption of fields value.
Adding AESEncrytDecry code for decrypting.
Finally decrypting on button click event and getting plain text value from it.

Article quality
This will be a unique tutorial that will have all the information needed to understand then concept of encryption and decryption in C-sharp language. I will use the simplest language for anyone to understand without any hardship.
References
N/A

Good afternoon and thank you for submitting your topic suggestion.
Your topic form has been entered into our queue and should be reviewed (for approval) as soon as a content moderator is finished reviewing the ones in the queue before it.

Sounds like a helpful topic - let's please be sure it adds value beyond what is in any official docs and/or what is covered in other blog sites. (the articles should go beyond a basic explanation - and it is always best to reference any EngEd article and build upon it). @amarawrite
Please be attentive to grammar/readability and make sure that you put your article through a thorough editing review prior to submitting it for final approval. (There are some great free tools that we reference in EngEd resources.)
ANY ARTICLE SUBMITTED WITH GLARING ERRORS WILL BE IMMEDIATELY CLOSED.
Please be sure to double-check that it does not overlap with any existing EngEd articles, articles on other blog sites, or any incoming EngEd topic suggestions (if you haven't already) to avoid any potential article closure, please reference any relevant EngEd articles in yours. - Approved


onRestart() is not called after rotation change

How does the activity respond when the user rotates the screen?
When the screen is rotated, the current instance of activity is destroyed a new instance of the Activity is created in the new orientation. The onRestart() method is invoked first when a screen is rotated. The other lifecycle methods get invoked in the similar flow as they were when the activity was first created.

When rotation changes;
onPause()->OnStop()->OnDestroy->OnCreate()->onStart()->OnResume()

onStart()->onRestoreInstanceState()->onResume()


Support Verilog AMS

This is probably an epic, but still.
I guess that two primary uses for OpenROAD are education (possibly self-education like in my case) and open source hardware.
If it is so, this flow misses support for analog-mixed-signal cores which are critical for powering actual devices and interacting with many peripherals.
Asking this here because currently there are no open source Verilog/VHDL AMS supporting flows.

I agree with the sentiment but this is too far outside the bounds of anything we are planning to work.  The closest project I know of is https://fasoc.engin.umich.edu/

Thanks, this seems nice.


[kiali] use kiali operator to install and manage kiali

to be paired with https://github.com/istio/installer/pull/556

@jmazzitelli: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

mandiff_operator
d0d4ab64a5e18520c5ac905715f47f4214e04774
link
/test mandiff_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

mandiff_operator
d0d4ab64a5e18520c5ac905715f47f4214e04774
link
/test mandiff_operator

gencheck_operator
d0d4ab64a5e18520c5ac905715f47f4214e04774
link
/test gencheck_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

mandiff_operator
d0d4ab64a5e18520c5ac905715f47f4214e04774
link
/test mandiff_operator

gencheck_operator
d0d4ab64a5e18520c5ac905715f47f4214e04774
link
/test gencheck_operator

e2e_operator
d0d4ab64a5e18520c5ac905715f47f4214e04774
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_operator
d0d4ab64a5e18520c5ac905715f47f4214e04774
link
/test gencheck_operator

e2e_operator
d0d4ab64a5e18520c5ac905715f47f4214e04774
link
/test e2e_operator

mandiff_operator
78d2a802292ecfcad37a2db74a95e03717a958cc
link
/test mandiff_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e_operator
d0d4ab64a5e18520c5ac905715f47f4214e04774
link
/test e2e_operator

mandiff_operator
78d2a802292ecfcad37a2db74a95e03717a958cc
link
/test mandiff_operator

gencheck_operator
78d2a802292ecfcad37a2db74a95e03717a958cc
link
/test gencheck_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

mandiff_operator
78d2a802292ecfcad37a2db74a95e03717a958cc
link
/test mandiff_operator

gencheck_operator
78d2a802292ecfcad37a2db74a95e03717a958cc
link
/test gencheck_operator

e2e_operator
78d2a802292ecfcad37a2db74a95e03717a958cc
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Please run...

In the CONTRIBUTING.adoc (or somewhere here - README maybe) you should explain to new developers what their GO env var should be. GOPATH / GOROOT.  Mine are set for my own projects, but if I am to run BUILD_WITH_CONTAINER=0 make update-goldens I need to set my GO env vars to something else. I tried this but it isn't working:
$ export GOROOT=$(pwd)
$ export GOPATH=$(pwd)
$ BUILD_WITH_CONTAINER=0 make update-goldens
Building with your local toolchain.
REFRESH_GOLDEN=true go test ./cmd/mesh/...
warning: GOPATH set to GOROOT (/home/jmazzite/source/istio.io/operator) has no effect
cmd/mesh/manifest-migrate.go:18:2: cannot find package "encoding/json" in any of:
	/home/jmazzite/source/istio.io/operator/src/encoding/json (from $GOROOT)
	($GOPATH not set. For more details see: 'go help gopath')
... and more errors...

What should be my GOPATH/GOROOT?

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_operator
78d2a802292ecfcad37a2db74a95e03717a958cc
link
/test gencheck_operator

e2e_operator
78d2a802292ecfcad37a2db74a95e03717a958cc
link
/test e2e_operator

mandiff_operator
0bc837dfe1ca9a5de61a002d7a62f2a93277b43f
link
/test mandiff_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e_operator
78d2a802292ecfcad37a2db74a95e03717a958cc
link
/test e2e_operator

mandiff_operator
0bc837dfe1ca9a5de61a002d7a62f2a93277b43f
link
/test mandiff_operator

gencheck_operator
0bc837dfe1ca9a5de61a002d7a62f2a93277b43f
link
/test gencheck_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

mandiff_operator
0bc837dfe1ca9a5de61a002d7a62f2a93277b43f
link
/test mandiff_operator

gencheck_operator
0bc837dfe1ca9a5de61a002d7a62f2a93277b43f
link
/test gencheck_operator

e2e_operator
0bc837dfe1ca9a5de61a002d7a62f2a93277b43f
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_operator
0bc837dfe1ca9a5de61a002d7a62f2a93277b43f
link
/test gencheck_operator

e2e_operator
0bc837dfe1ca9a5de61a002d7a62f2a93277b43f
link
/test e2e_operator

mandiff_operator
d9b6551f5317a931c143dd1df2e45d747a22f65e
link
/test mandiff_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e_operator
0bc837dfe1ca9a5de61a002d7a62f2a93277b43f
link
/test e2e_operator

mandiff_operator
d9b6551f5317a931c143dd1df2e45d747a22f65e
link
/test mandiff_operator

gencheck_operator
d9b6551f5317a931c143dd1df2e45d747a22f65e
link
/test gencheck_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

mandiff_operator
d9b6551f5317a931c143dd1df2e45d747a22f65e
link
/test mandiff_operator

gencheck_operator
d9b6551f5317a931c143dd1df2e45d747a22f65e
link
/test gencheck_operator

e2e_operator
d9b6551f5317a931c143dd1df2e45d747a22f65e
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

BUILD_WITH_CONTAINER=0 make update-goldens
make clean gen

I think you need BUILD_WITH_CONTAINER=0 when running make clean gen, too.. e.g.:
export BUILD_WITH_CONTAINER=0
make update-goldens
make clean gen

At least when I do that, make is doing something.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_operator
d9b6551f5317a931c143dd1df2e45d747a22f65e
link
/test gencheck_operator

e2e_operator
d9b6551f5317a931c143dd1df2e45d747a22f65e
link
/test e2e_operator

mandiff_operator
d5f1bdf08675629e4ad768091781026b9d7253df
link
/test mandiff_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_operator
d9b6551f5317a931c143dd1df2e45d747a22f65e
link
/test gencheck_operator

e2e_operator
d9b6551f5317a931c143dd1df2e45d747a22f65e
link
/test e2e_operator

mandiff_operator
d5f1bdf08675629e4ad768091781026b9d7253df
link
/test mandiff_operator

test_operator
d5f1bdf08675629e4ad768091781026b9d7253df
link
/test test_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

mandiff_operator
d5f1bdf08675629e4ad768091781026b9d7253df
link
/test mandiff_operator

test_operator
d5f1bdf08675629e4ad768091781026b9d7253df
link
/test test_operator

e2e_operator
d5f1bdf08675629e4ad768091781026b9d7253df
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

test_operator
d5f1bdf08675629e4ad768091781026b9d7253df
link
/test test_operator

e2e_operator
d5f1bdf08675629e4ad768091781026b9d7253df
link
/test e2e_operator

mandiff_operator
7f2580285f6daaa7de9d877a0103f0d7799a5b91
link
/test mandiff_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e_operator
d5f1bdf08675629e4ad768091781026b9d7253df
link
/test e2e_operator

mandiff_operator
7f2580285f6daaa7de9d877a0103f0d7799a5b91
link
/test mandiff_operator

test_operator
7f2580285f6daaa7de9d877a0103f0d7799a5b91
link
/test test_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

I have no idea what mandiff is telling me nor what is wrong. default profile has diffs ?

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

mandiff_operator
7f2580285f6daaa7de9d877a0103f0d7799a5b91
link
/test mandiff_operator

test_operator
7f2580285f6daaa7de9d877a0103f0d7799a5b91
link
/test test_operator

e2e_operator
7f2580285f6daaa7de9d877a0103f0d7799a5b91
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

I ran "make update-charts" and "make generate-types generate-values" just to see if there is anything else I need to do (I'm just guessing - looking at the Makefiles to see what targets are available to run). They changed nothing. So, without documentation directing me on how to develop in this operator repo, I'm going to have to ask that someone "in the know" either tell me what else needs to be done or try to complete the changes required. Right now I'm stumped.

Please run
BUILD_WITH_CONTAINER=0 make update-goldens
make clean gen

to make the tests pass.

@morvencao why must BUILD_WITH_CONTAINER=0 need be set?  If the concern is over HUB/TAG overrides, a) I believe @howardjohn has fixed that b) if he hasn't I will fix it as soon as I have a test case where it is broken c) unset HUB;unset TAG 😀
If there is some other concern, please let me know.  This repo is on the BUILD_WITH_CONTAINER plan, and as such, there should be no good reason to turn off BUILD_WITH_CONTAINER - unless as a developer you want to develop with your own dependencies.
Cheers
-steve

@jmazzitelli my environment on Linux is:
sdake@beast-01:~$ env
TAG=sdake
XDG_SESSION_ID=865
TERM=xterm-256color
SHELL=/bin/bash
ISTIO=/home/sdake/go/src/istio.io
SSH_CLIENT=<IP_ADDRESS> 56633 22
SSH_TTY=/dev/pts/0
USER=sdake
LS_COLORS=rs=0:di=01;34:ln=01;36:mh=00:pi=40;33:so=01;35:do=01;35:bd=40;33;01:cd=40;33;01:or=40;31;01:mi=00:su=37;41:sg=30;43:ca=30;41:tw=30;42:ow=34;42:st=37;44:ex=01;32:*.tar=01;31:*.tgz=01;31:*.arc=01;31:*.arj=01;31:*.taz=01;31:*.lha=01;31:*.lz4=01;31:*.lzh=01;31:*.lzma=01;31:*.tlz=01;31:*.txz=01;31:*.tzo=01;31:*.t7z=01;31:*.zip=01;31:*.z=01;31:*.Z=01;31:*.dz=01;31:*.gz=01;31:*.lrz=01;31:*.lz=01;31:*.lzo=01;31:*.xz=01;31:*.bz2=01;31:*.bz=01;31:*.tbz=01;31:*.tbz2=01;31:*.tz=01;31:*.deb=01;31:*.rpm=01;31:*.jar=01;31:*.war=01;31:*.ear=01;31:*.sar=01;31:*.rar=01;31:*.alz=01;31:*.ace=01;31:*.zoo=01;31:*.cpio=01;31:*.7z=01;31:*.rz=01;31:*.cab=01;31:*.jpg=01;35:*.jpeg=01;35:*.gif=01;35:*.bmp=01;35:*.pbm=01;35:*.pgm=01;35:*.ppm=01;35:*.tga=01;35:*.xbm=01;35:*.xpm=01;35:*.tif=01;35:*.tiff=01;35:*.png=01;35:*.svg=01;35:*.svgz=01;35:*.mng=01;35:*.pcx=01;35:*.mov=01;35:*.mpg=01;35:*.mpeg=01;35:*.m2v=01;35:*.mkv=01;35:*.webm=01;35:*.ogm=01;35:*.mp4=01;35:*.m4v=01;35:*.mp4v=01;35:*.vob=01;35:*.qt=01;35:*.nuv=01;35:*.wmv=01;35:*.asf=01;35:*.rm=01;35:*.rmvb=01;35:*.flc=01;35:*.avi=01;35:*.fli=01;35:*.flv=01;35:*.gl=01;35:*.dl=01;35:*.xcf=01;35:*.xwd=01;35:*.yuv=01;35:*.cgm=01;35:*.emf=01;35:*.ogv=01;35:*.ogx=01;35:*.aac=00;36:*.au=00;36:*.flac=00;36:*.m4a=00;36:*.mid=00;36:*.midi=00;36:*.mka=00;36:*.mp3=00;36:*.mpc=00;36:*.ogg=00;36:*.ra=00;36:*.wav=00;36:*.oga=00;36:*.opus=00;36:*.spx=00;36:*.xspf=00;36:
KUBECONFIG=/home/sdake/.kube/config
MAIL=/var/mail/sdake
PATH=/home/sdake/bin:/home/sdake/.local/bin:/home/sdake/google-cloud-sdk/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin:/usr/local/go/bin:/usr/local/kubebuilder_1.0.8_linux_amd64/bin:/home/sdake/go/bin
PWD=/home/sdake
LANG=en_US.UTF-8
SHLVL=1
HOME=/home/sdake
GITHUB_USER=sdake
LOGNAME=sdake
HUB=docker.io/sdake
XDG_DATA_DIRS=/usr/local/share:/usr/share:/var/lib/snapd/desktop
SSH_CONNECTION=<IP_ADDRESS> 56633 <IP_ADDRESS> 22
GOPATH=/home/sdake/go
LESSOPEN=| /usr/bin/lesspipe %s
XDG_RUNTIME_DIR=/run/user/1000
LESSCLOSE=/usr/bin/lesspipe %s %s
_=/usr/bin/env

TLDR;
unset GOROOT
export GOPATH=$HOME/go
mkdir -p $HOME/go/src/istio.io
cd $HOME/go/src/istio.io
git clone https://jmazzitelli/operator
cd operator
make update-goldens

That - coupled with other comments in this review, should get you going.
Please have a read of this (very dated) wiki that explains how to install the various toolchains:

I have no idea what mandiff is telling me nor what is wrong. default profile has diffs ?

I don't know either and it is very frustrating. When @ostromart says why - and you understand it, would you be kind enough to document it in our developer wiki:
https://github.com/istio/istio/wiki

I ran "make update-charts" and "make generate-types generate-values" just to see if there is anything else I need to do (I'm just guessing - looking at the Makefiles to see what targets are available to run). They changed nothing. So, without documentation directing me on how to develop in this operator repo, I'm going to have to ask that someone "in the know" either tell me what else needs to be done or try to complete the changes required. Right now I'm stumped.

@jmazzitelli all of Istio's repos have a make gen target which is in theory meant to call all code-generators and generate all code.  If that isn't working correctly because of BUILD_WITH_CONTAINER bugs, can someone do me a solid and file a bug with a reproducer that is intelligible, assign to me, and I'll fix?
Cheers
-steve

Please have a read of this (very dated) wiki that explains how to install the various toolchains:

@sdake I assume you meant to include a link here?
I just ran "make update-goldens" and got this:
$ make update-goldens
Building with the build container: gcr.io/istio-testing/build-tools:master-2019-11-08T17-21-18.
Using docker credential directory /home/jmazzite/.docker.
REFRESH_GOLDEN=true go test ./cmd/mesh/...
exit status 255
FAIL	istio.io/operator/cmd/mesh	1.249s
FAIL
Makefile.core.mk:88: recipe for target 'update-goldens' failed
make: *** [update-goldens] Error 1
make: *** [update-goldens] Error 2

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

test_operator
7f2580285f6daaa7de9d877a0103f0d7799a5b91
link
/test test_operator

e2e_operator
7f2580285f6daaa7de9d877a0103f0d7799a5b91
link
/test e2e_operator

mandiff_operator
d6e35ba1bbf956a075893cb6393f8d46881593b3
link
/test mandiff_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

e2e_operator
7f2580285f6daaa7de9d877a0103f0d7799a5b91
link
/test e2e_operator

mandiff_operator
d6e35ba1bbf956a075893cb6393f8d46881593b3
link
/test mandiff_operator

test_operator
d6e35ba1bbf956a075893cb6393f8d46881593b3
link
/test test_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

mandiff_operator
d6e35ba1bbf956a075893cb6393f8d46881593b3
link
/test mandiff_operator

test_operator
d6e35ba1bbf956a075893cb6393f8d46881593b3
link
/test test_operator

e2e_operator
d6e35ba1bbf956a075893cb6393f8d46881593b3
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

There is something wrong in my branch to cause the failure, but I don't know what the error is

The error is because the "manifest generate" command fails with a exit code of 255. It is probably because I didn't update-charts. But I can't update charts because my other PR in istio/installer isn't merged yet, but of course I can't merge it until I can confirm THAT PR works, and I can only confirm THAT PR works if I can run and test THIS PR to confirm that it works. It's a catch-22. I can't merge the istio/installer PR until I confirm this istio/operator PR works, but I can't build/test this istio/operator PR until I merge istio/installer PR.
So, in short, the way these two installer/operator repos work together make it pretty much impossible for a contributor to contribute to them. This needs to be fixed. As it is now, working in these two repos requires the person to have intimate knowledge of the make targets and the build scripts in both repos (none of this is documented) to get this to build and test locally to confirm everything works and get the tests to pass.

@jmazzitelli I, and I think everyone else working on this, completely agrees its a huge mess right now. We are working right now to make this a lot simpler hopefully, see https://github.com/istio/istio/issues/19146. Sorry that its in the current state right now

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

test_operator
d6e35ba1bbf956a075893cb6393f8d46881593b3
link
/test test_operator

e2e_operator
d6e35ba1bbf956a075893cb6393f8d46881593b3
link
/test e2e_operator

mandiff_operator
f3441e94a5f6ba21e0d0386f7547a84b00907a5b
link
/test mandiff_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

test_operator
d6e35ba1bbf956a075893cb6393f8d46881593b3
link
/test test_operator

e2e_operator
d6e35ba1bbf956a075893cb6393f8d46881593b3
link
/test e2e_operator

mandiff_operator
f3441e94a5f6ba21e0d0386f7547a84b00907a5b
link
/test mandiff_operator

gencheck_operator
f3441e94a5f6ba21e0d0386f7547a84b00907a5b
link
/test gencheck_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jmazzitelli: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

mandiff_operator
f3441e94a5f6ba21e0d0386f7547a84b00907a5b
link
/test mandiff_operator

gencheck_operator
f3441e94a5f6ba21e0d0386f7547a84b00907a5b
link
/test gencheck_operator

e2e_operator
f3441e94a5f6ba21e0d0386f7547a84b00907a5b
link
/test e2e_operator

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Implement Vmware vCloud NetworkManager

Purpose or Intent
This patch adds basic NetworkManager functionality for the Vmware Cloud. Since FOG support is quite limited atm, most of the data is left empty. Network structure is fetched so that user can navigate thru cloud_networks, cloud_subnets, network_ports and eventually reach VMs.
NOTE: Vmware Cloud does not support cloud_subnets therefore dummy values are used to comply with structure
NOTE: Vmware Cloud does not have separate network manager so we share credentials with CloudManager.

/cc @Ladas @agrare

@miha-plesko please review the test failures.  Follow up with @agrare if you need help tracking down some of these failures.

looks good to me, nice job :+1:
just fix the 'red !', except the ones with get_ prefix and it should be good to go for me.
for the next PR, you should also add UI specs, usually just add the factories and list the vmware to here: https://github.com/Ladas/manageiq/blob/39d26600c06a3d8d66a056dd5e7398553d7c4b7d/spec/views/layouts/listnav/_cloud_subnet.html.haml_spec.rb#L11-L11
and here: https://github.com/Ladas/manageiq/blob/5e91d1d05e36fd0c8b298b7faf6803dc31538de4/spec/controllers/cloud_subnet_controller_spec.rb#L10-L10
for all network models
I am preparing a test helpers so we can test also relations and nested lists in UI,

Looks good @miha-plesko  :+1:

@Ladas, @agrare, @miha-plesko
I feel like we should wait to merge this until vapps are supported in Fog.  Currently this is using fake data for the vapps details.
Are there any timelines on getting the fog support added?
/cc @Fryguy

@blomquisg since I don't know timelines regarding vapp networks, I'll throw them out of this PR so only VDC networks will remain.

@blomquisg hm, seems to me it's better to fake the fog model, rather than not showing that type of networks at all?

@blomquisg @Ladas We discussed this internally as well. The problem here is most probably the naming of the function fake_vapp_networks and the comment in the code giving the impression that the Fog will eventually return vApp network configurations. We are not actually faking anything there - we are just providing information that we can get from vCloud (more or less just the name :-)). Even ID we have to build because vApp network names can be the same in different vApps.
ATM this is not going to be added to Fog as there is no reason for this. vApp network is internal to vApps (orchestration stacks). They are not to be used by other vApps (for example, during the provisioning of a new vApp, it is not possible to connect to another's vApp network.
However it is beneficial if we can actually list them and display them in the topology. If we omit them, only main (VDC) networks will be shown there.
To further demonstrate this, look at this topology diagram of our small testbed. If we omit vApp networks, we will see only subnet-vdc-net-miha as the network and only TTY-Linux-1-mm would be connected to it - there wouldn't be any other connection in the diagram.
Anyway, alternative to removing this is to simply renaming fake_vapp_networks to get_vapp_networks and remove the comment about Fog.
We can go either way, so let us know which one you prefer.

@gberginc make sense to me. I would incline to renaming fake_vapp_networks to get_vapp_networks.
For the Fog exposure, it depends if vapp networks can have any actions placed on them? We tend to move all Remote Model definition to Fog. So Fog would have vapp_networks.all, that would return you what fake_vapp_networks does now.
But if vapp_networks is not really a model worth modeling, I guess it can stay as it is. :-)

@Ladas I see your point with respect to Fog. We'll keep this in mind while we continue working on the networking for vCloud. My assumption was that since the vCloud API itself does not provide direct access to vApp networks, this should not be changed in Fog, but in case we find ways to modify them, we should do it in Fog.

Based on the discussion above I've renamed fake_vapp_networks to get_vapp_networks. I must admit that 'fake' was actually a misleading adjective for what the function was doing. While one would think that we're faking the data, we were actually faking the Struct where the data got stored, and not the data itself.
Since this is the very first PR regarding VMware Cloud networking, I've tried to make it as simple as possible (but still it's 28 files large). Later, when we dig deeper into network integration, I believe we will need to parse vApp networks in its own function anyway (as opposed to putting both VCD and vApp network parsing into get_networks)

The problem here is most probably the naming of the function\

@gberginc Makes sense, thanks for clarifying


[Github] treeItem is more clear what it's about

https://github.com/shopware/platform/pull/2578

Imported from Github. Please see: https://github.com/shopware/shopware/pull/2937

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-35043.
[created from NEXT-35043, comment 486509]


To fix processing repeat data in sign up

done
https://github.com/AlexeyOs/GoodCourses/commit/84a830fcff40c7065caa591ae5608a4533638858


Protect endpoints with guards

[x] determine which endpoints should only be accessible as authenticated user
[ ] determine which endpoints should not be accessible as authenticated user
[ ] determine which endpoints should be removed (/are not needed)

Everything is protected, but now all the tests are fucked hihi


Replacing parameters dynamically in language lines in Laravel 5.1

I have language file called resources/lang/en/message.php in laravel 5.1. Now I want to define a custom message line like this:
'deletemsg'=> 'The :name deleted successfully.',

and use in page.blade.php with following format:
{{trans('message.deletemsg')}}

In Laravel Documentation said to replace parameter it should be written:
{{ trans('messages.welcome', ['name' => 'Dayle']); }}

But I want the :name parameter dynamically load in all language and I don't type statically.In Laravel has a validation.php that has a attributes array to get variable of attribute. I like to do. I think we can do so again

Still not solved

Hi @essivision
I need exactly you asked in your post. Did you find a solution for it?
Thank you.
Raul

just add change like this ['name' => $name];

Hi @masterpieceicang
Thank you for your answer but that doesn't work.
controller.php
return Redirect::route('auth.reset', ['token'=>$token])
                    ->with('error-message', trans('validation.password.no_update'));

validation.php (validation language lines)
'password'             => [
        'no_update' => 'Su :attribute no ha sido actualizada. Intentelo de nuevo.',
 ],

'attributes' => ['password' => 'contraseña'],

I want the :attribute parameter dynamically change with the attribute I have set.

I have the same problem, any solution?


remove marks and axis

I noticed the Mark's remove method was not called, this fixes that, and I think the same for axis. As discussed with @jasongrout

Should we also call remove on the interaction and the scales?

there is also interaction_view, I'm not sure I know bqplot good enough to cover all of these cases.

I thought we had this before. Don't know where it went! Thanks :)

@ssunkara1 will you handle any other views that need deleting?

@ssunkara1 will you handle any other views that need deleting?

+1. If this looks good, perhaps we should merge and iterate.

@maartenbreddels Thanks. This looks good. Merging.

Great :+1:

Thanks @maartenbreddels


Fix missing wifi device data

Getting wifi iface names from 'uci show wireless' is no longer an option since these do not always get saved to config- and routers that were missing hard coded ifname config were not populating wifi devices to report to ops. Instead, 'iw dev' lists all interfaces for wifi and those ifnames can be more reliably accessed from there.

Can you add a unit test for the parsing logic? Just break out the parsing from the calling of the command.


Add Cat Toys Page & Content

Add a page called "Cat Toys" with some content. Please provide references as well.
Feel free to make any adjustments you may like.
VitePress Docs: https://vitepress.dev/

hey @SanditZZ  assign me this issue


Data lifecycle form fields only shows at most 2 digits when values may have 3 or more

Describe the bug
Form fields for the data lifecycle settings are too short and only show 2 digits at most. Some fields could reasonably have 3 digits like the compliance reports settings.
For example, if the number number of days to keep compliance reports is set to 120 you will only see 12 in the form field unless you click in the field and use the arrow keys to scroll over to the right.
Where this becomes more of a problem is when the user enters the values incorrectly and types in something like 900 or 1200 instead of the desired 90 or 120 days and it won't be immediately obvious that it has been entered incorrectly.
To Reproduce
Steps to reproduce the behavior:

Login as Admin
Browser to Settings -> Data Lifecycle
Go down to the compliance lifecycle settings and type in 900
See that the field only shows 90 without clicking in it and scrolling the field to the right.

Expected behavior
A description of what you expected to happen.
The form field should auto-resize to show the entire value of the input.
Screenshots
If applicable, add screenshots to help explain your problem.
Versions (please complete the following information):

Browser: Chrome
Automate Build Number:<PHONE_NUMBER>5626

Additional context

thanks @teknofire!
@vjeffrey - While working on this, Do you know if there is a max number we'd like to set for something like this?

@SEAjamieD is there a reson to have a max? can you grow and shrink that box to adjust to the number supplied?

i don't think there's really a need for a max. we could do a nice thing where if they enter something large like 900 we warn them and tell them to go look at docs re: space reqs but also can just let them enter what they enter

@susanev - was asking about max since i'm unfamiliar with the use case.  I'm not sure about growing and shrinking the box as the numbers are supplied, though I have seen text shrink as the number gets larger.  I'll play around with it and provide some options.

something like this would be rad
https://css-tricks.com/auto-growing-inputs-textareas/


AppendExisting is not available anymore on ExtentHtmlReporter

AppendExisting is not available anymore in v4 and nothing is indicated in the breaking changes, nor how to migrate from v3 to v4.
Could you please give some indications?

As far as I know, this feature was never implemented in v3.  This is an open issue in the Java repo as well, and you will notice it is marked as slow-burner because of the complexity and level of effort required.
@anshooarora  this can be marked an enhancement for a future release.

Hello. Any update on whether this will be implemented any time soon? Thanks.

Hi @anshooarora. We are looking to append test results into one HTML file using C# v4. Is this going to be implemented anytime soon?

This will be implemented but not in version 4.  This feature requires quite a bit of re-engineering as version 4 has a ton of reporters, each with own nuances.  Working on the version 5 which will include this feature..  no ETA as of now though.

This was recently added in the Java binding and will soon be available in the .NET implementation.

Hi @anshooarora !  I hope all is well.  I just wanted to check in and see if this functionality has been added to the latest .Net version or possibly the estimate time range to fix.  I downloaded version 4.0.3 for the .net bindings but I don't see the AppendExisting method or unless it's called something else.  Thanks in advance for your help.

Hi @anshooarora !  I just wanted to check in on this item to see if there is an update?

Sorry for the delay.  This was recently reintroduced in Java version 5 pre-release, and will be included in the coming few weeks for .NET as well.

Hi, Is it added to .Net?

Hi @anshooarora Any Update on the v5.0, I am eagerly waiting for AppendExisting Html file

Hi, this feature is available starting version 5.  Docs here:  https://www.extentreports.com/docs/versions/5/net/index.html#combine-multiple-reports

Hello, is this feature available in version 5 for .NET? Where can I find the documentation? Thank you.

Hello, is this feature available in version 5 for .NET? Where can I find the documentation? Thank you.

@lijinxinKing This feature is available using the JsonFormatter:
var spark = new ExtentSparkReporter("spark.html");
var json = new ExtentJsonFormatter("extent.json");
var extent = new ExtentReports();
extent.CreateDomainFromJsonArchive("extent.json");
extent.AttachReporter(json, spark);


Use i18n language keys for all strings.

Every language string should be making use of Rails i18n features in order to provide a solid baseline for the addition of languages.

For me the first version of any app should be monolingual, look for ex, hey and basecamp both monolingual, also i18n add some complexity that I don’t want at this moment.


Add 6 Aerodrome CLM pools on base

@ying13 Internally we're currently developing vaults to go over Aero and Velo CLMs, so they've mostly been deployed already but not yet released which is why I believe this PR hasn't been touched. You're welcome to deploy CLMs for the other platforms that don't currently require the reward pools like Uniswap and Camelot


An error happened in 'go get installation'

There's probably not qiscus-sdk-api repository in github. Please address to this issue.
$ go get github.com/gotsunami/go-cloudinary/cloudinary

# cd .; git clone https://github.com/qiscus/qiscus-sdk-api /Users/ikedat/.gvm/pkgsets/go1.8/global/src/github.com/qiscus/qiscus-sdk-api
Cloning into 'github.com/qiscus/qiscus-sdk-api'...
remote: Repository not found.
fatal: repository 'https://github.com/qiscus/qiscus-sdk-api/' not found
package github.com/qiscus/qiscus-sdk-api/api/admin/v1: exit status 128

sorry

never mind

@ikeikeikeike I'm having the same issue could you please tell me how you solved this?

@ikeikeikeike I'm having the same issue could you please tell me how you solved this?


HELP-Med show questions

Making the questions appear as soon as the instructions have been read. This uses a Boolean that seems to be misnamed. For some reason, whether or not the questions appear is controlled by isGenerativeContext. To avoid breaking anything else, I have kept the name as it is, but this seems like something that should be refactored eventually.

Good point. The other changes aren't useful without line 334, so I'll just cancel this PR. It would be nice to be able to show the questions right away because some users keep trying to put their answers in the chatbots, but changing the "Submit" button seems like a bad idea right now, so I'll keep it as is.


No such file or directory on android ffmpeg in api level 28

I use ffmpeg and its ok when i run it on android device with api level 25 but when i run it on device with api level 27 show "No such file or directory" error in my logcat.
My ffmpeg library: bravobit.nl.ffmpegandroid
Please help me.
My full logcat:
E/FFmpeg: Exception while trying to run: [/data/user/0/com.myapplication/files/ffmpeg, -i, /external_files/org_94594219219624692.mp3, -af, adelay=0|0, -preset, ultrafast, /external_files/vc1_94594219219624692.mp3]
    java.io.IOException: Cannot run program "/data/user/0/com.myapplication/files/ffmpeg": error=2, No such file or directory
        at java.lang.ProcessBuilder.start(ProcessBuilder.java:1048)
        at nl.bravobit.ffmpeg.ShellCommand.run(ShellCommand.java:15)
        at nl.bravobit.ffmpeg.FFcommandExecuteAsyncTask.doInBackground(FFcommandExecuteAsyncTask.java:43)
        at nl.bravobit.ffmpeg.FFcommandExecuteAsyncTask.doInBackground(FFcommandExecuteAsyncTask.java:12)
        at android.os.AsyncTask$2.call(AsyncTask.java:333)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1162)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:636)
        at java.lang.Thread.run(Thread.java:764)
     Caused by: java.io.IOException: error=2, No such file or directory
        at java.lang.UNIXProcess.forkAndExec(Native Method)
        at java.lang.UNIXProcess.<init>(UNIXProcess.java:133)
        at java.lang.ProcessImpl.start(ProcessImpl.java:128)

I fixed it by adding the correct path to the ffmpeg binary like this:
`
val params = "-version"
val ffmpeg = FFmpeg.getInstance(context)
val command = "%s/ffmpeg %s".format(context.filesDir.path, params)
ffmpeg.execute(params, cb)
`
And yes, it works with API below 28 too.

Have you tried the function to check to see if it's available first yet?
if (FFmpeg.getInstance(this).isSupported()) {
  // ffmpeg is supported
} else {
  // ffmpeg is not supported
}

When looking at the code, this seems to ensure that the ffmpeg executable is pulled from the resources. I wasn't expecting that one myself. I'm glad I looked.

The same problem. I am trying to use ffmpeg on Pixel 3a with Android 10 and compile version 28.

This may be duplicato of https://github.com/bravobit/FFmpeg-Android/issues/126


ENYO-5426: Fix GridListImageItem image, caption, and subcaption styles.

Checklist

[X] I have read and understand the contribution guide

[ ] A CHANGELOG entry is included

[ ] At least one test case is included for this feature or bug fix

[X] Documentation was added or is not needed

[ ] This is an API breaking change

Issue Resolved / Feature Added

moonstone caption and subcaption styles did not override ui ones.
If using only a caption or a subCaption, there is the gap between its text and the GridListImageItem border.

Resolution

Add the comments in the ui caption and subcaption classes so that moonstone ones could override.
Adjust the GridListImageItem's height depending on the caption and subcaption height.

Additional Considerations
Links
ENYO-5426
Comments
The target branch for this PR is not develop but feature/ENYO-5426-same-height. So I did not update CHANGELOG.md and no test case is needed.

Hi @Djspaceg
I fixed the issues that you mentioned.
The Travis CI build failed. I seems to be related with not my PR but our Enact build.

I've made a few changes:

Switched from Layout to Column (trivial, it's just an alias to Layout orientation="vertical")
Removed the (now) unnecessary LESS variables. With the captions being Cell shrink, they'll automatically size to the dimensions of their content. (36 and 39 px respectively)
BONUS: Updated the VL stories to use 100% height instead of a pixel value since now Storybook allows content to fill the content space reliably.

@ybsung, if these changes are ok with you, you can go ahead and merge this into the other ticket. 👍

@LucieRoy - Updating the height of the VirtualList stories might impact some test cases.

Oh we could remove the unnecessary LESS variables. Cool!!!. I'll verify the PR tomorrow.

Hi @Djspaceg

Switched from Layout to Column (trivial, it's just an alias to Layout orientation="vertical")

: I verified all samples worked well. Thanks.

Removed the (now) unnecessary LESS variables. With the captions being Cell shrink, they'll automatically size to the dimensions of their content. (36 and 39 px respectively)

: By doing it, the code in my PR is  much better now.
I found that if removing line-height styles in the caption and the subcaption, it would be much flexible. If app developers override the font-size of the caption or the sub-caption, the caption or the subcaption could be clipped because of the line-height. By removing it, we could prevent clipping it. Please let me know your opinion about my commit.

BONUS: Updated the VL stories to use 100% height instead of a pixel value since now Storybook allows content to fill the content space reliably.

: 100% height is the one that we expected to use when creating VirtualList samplers. But as @ryanjduffy mentioned, we need to check its TCs. At the first time, I tried to add height: 100%; min-height: 552px(or 549px); styles not to break any VirtualList and VirtualGrdiList TCs. But I found that I needed to update the Scroller samples, too. So it would be better to separate it with this PR. I created PLAT-64761 to apply 100% height to all VirtualList, VirtualGridList, and Scroller samplers.
CC: @LucieRoy

Separating the height change into its own PR makes sense for trackability/history.
Resetting the line-height to initial means that we lose the ability to control how tall-glyphs are handled in locales that need that feature. Additionally, we don't want any developer to effortlessly override the UX defined font-size for these components; that would remove consistency (perceived visual quality) and limit our ability to make framework-level design updates in the future. If a developer is going to undertake such an action, they need to approach it with the level of care and consideration that it involves, e.g. consideration for RTL, Marquee, dynamic Item sizing, tall-glyphs in every locale, which fonts are used, sub-pixel font rendering, font hinting sizes, and multiple screen sizes/scales. While it does make it easier for the styles to be overridden, by an external user, it's my opinion that that isn't the function of this component.
As an alternate perspective, and reminder, if the Moonstone defined styles are not meeting the needs of the app developers using it, we need to update those internally. Requiring every app to correctly define all of their own custom rules consistently along with every other app not only wastes their time but defeats the purpose of a well designed, well maintained, framework. We have to change if they need us to be different.
I'm going to recommend moving forward with this PR and reverting the line-height change. What I'll do is I'll revert that change, merge this into the other PR, and if we change our mind about the line-height addition, we can cherrypick that back into the other PR before it's merged.

@Djspaceg
Thanks for your explanation. I agree with your opinion. I had to think about it more deeply.


Fix fontMetric baseline

The 'TEST' is a text, and the other is a svg. The 'TEST' of canvas is a little bit on the down side.
The dom

The canvas

The campare

@niklasvh Hello, could you please look at this PR?


Not compatable with SWHttpTrafficRecorder

I wanted to point out that this pod isn't compatible with the SWHttpTrafficRecorder as it points out on the readme. Both this pod and that pod try to register their own versions of NSURLProtocol which means one of them is going to lose, so either you don't record the file or you don't stub the response. I spent a good chunk of time trying to figure out why they weren't working together and thought this should be noted on your readme, as it makes it look like it's been tested and it should work when in reality it doesn't.

Thanks for the issue.
It has been tested… but separately, as you usually don't both stub and record what you stub, but rather record first and stub later.
But you're right in that it should be mentioned in the README that you should not use both at the same time but rather one after the other, so the README should be more clear about that.

So the reason I have a two in one is that I try to automate everything. I attempt to record, if the file name for the request doesn't exist then I do record the request, otherwise if it does exist I stub the request based off of the file. This requires me to register both.

I see.
I think you should still be able to register OHHTTPStubs manually (I don't remember of SWHttpTrafficRecorder offers an API to register the NSURLProtocol manually but OHHTTPStubs has setEnabled: and setEnabled: forSession: methods that should allow you to add OHHTTPStubs' internal NSURLProtocol manually even after SWHttpTrafficRecorder did set itself up and added is own, making them both intercept the requests (I indeed haven't tested this use case of supporting them both at the same time but it's worth a try as it should theoretically work if you do it this way)

I thought that only one NSURLProtocol can be registered at a time? I don't think I understand your proposed solution. Can you show me what you mean? This would save me a lot of heartache, I was considering merging the two pods into one so I can merge their protocols and use them at the same time.

No you can register as many as you want. Let me find a Mac in a minute so I can give you an example

Ok, so, basically my proposed solution is for you to call OHHTTPStubs.sharedInstance.setEnabled(true, forSessionConfiguration: sessionConfig) (assuming you use a custom NSURLSession build from an NSURLSessionConfiguration, and not NSURLConnection nor the shared NSURLSession) before creating your NSURLSession with it.
You can find this method's implementation here. As you can see, this method doesn't replace the supported NSURLProtocols of an NSURLSession, but instead just inserts OHHTTPStubs's own NSURLProtocol at the top of the sessionConfig.protocolClasses array.
This insertion is automatically done on the two NSURLSessionConfiguration's defaultSessionConfiguration and ephemeralSessionConfiguration presets (thanks to some swizzling).
If you use SWHttpTrafficRecorder, it also inserts its own NSURLProtocol in this array (see here). But for this to work you'll have to:

Ensure you create your custom NSURLSessionConfiguration and set it up both for OHHTTPStubs AND for SWHttpTrafficRecorder before you create your NSURLSession(configuration: …) initializer, because when you create your NSURLSession from a configuration, the configuration is copied, so any further changes on the NSURLSessionConfiguration you used after creating the NSURLSession won't affect that NSURLSession retroactively (this is explained in Apple's doc on NSURLSession)
Ensure that both OHHTTPStubs an SWHttpTrafficRecorder did set up their NSURLProtocol in the right order and on the same NSURLSessionConfiguration

So, now that I've had time to re-read the implementation of both pods, I don't see why this wouldn't work:

Ensure that you have the pod 'OHHTTPStubs' pod in your Podfile. If you only have pod 'OHHTTPStubs/Swift' for example, this subspec doesn't contain the NSURLSession-supporting subspec by default (this might change in the future, but for now the Swift subspec only contains the Swift helpers, not the other subspecs. See the dedicated section in the README about this
Use this kind of code (I wrote it in Swift but should work the same in ObjC):

import Foundation

// Start with the default configuration
let config = URLSessionConfiguration.default

// If you indeed have the `OHHTTPStubs/NSURLSession` subspec listed in your `Podfile.lock`
// then OHHTTPStubs should have automatically  injected its own `NSURLProtocol` at this point
// You can check this using some NSLog:
NSLog("protocol classes after OHHTTPStubs: \(config.protocolClasses)") // this should contain `OHHTTPStubsProtocol` among others

do {
  // not sure about the translation of SWHttpTrafficRecorder API in Swift here, I'll let you adapt
  let started = try SWHttpTrafficRecorder.sharedRecorder().startRecording(path:nil, forSessionConfiguration:config)
  // At this point, SWHttpTrafficRecord should itself have added its own NSURLProtocol to the
  // protocolClasses array, intercepting the traffic first, BEFORE anybody else (including OHHTTPStubs)
  NSLog("Started: \(started)")
  NSLog("protocolClasses after SWHTR: \(config.protocolClasses)")
} catch {
  NSLog("Error while trying to start SWHttpTrafficRecorder")
}

// At this point, SWHttpTrafficRecorder is listed first in the protocolClasses array
// then OHHTTPStubs is next in line. This means that SWHTR will intercept requests first, then
// OHHTTPStubs will be given the ability to stub them next, then if the request passed thru both
// the request will finally hit the real world.

// If you want to make `OHHTTPStubs` intercept the requests BEFORE SWHttpTrafficRecorder
// Then you have to ask `OHHTTPStubs` to remove itself then re-insert itself at index 0
OHHTTPStubs.setEnabled(false, forSessionConfiguration: config) // remove first
OHHTTPStubs.setEnabled(true, forSessionConfiguration: config) // then reinsert, at index 0

// And ONLY then, create your NSURLSession, because if you create it before
// setting up the NSURLSessionConfiguration, changes in the configuration won't do anything
let session = URLSession(configuration: config)
let task = session.dataTask(with: yourURLHere)
task.resume()

What's the difference between adding the protocol like so:
[NSURLProtocol registerClass:OHHTTPStubsProtocol.class];
and like so:
        NSMutableArray * urlProtocolClasses = [NSMutableArray arrayWithArray:sessionConfig.protocolClasses];         Class protoCls = OHHTTPStubsProtocol.class;         if (enable && ![urlProtocolClasses containsObject:protoCls])         {             [urlProtocolClasses insertObject:protoCls atIndex:0];         }         else if (!enable && [urlProtocolClasses containsObject:protoCls])         {             [urlProtocolClasses removeObject:protoCls];         }         sessionConfig.protocolClasses = urlProtocolClasses;?
If I register the class for NSURLProtocol does it register it for the default protocol, so any call to default.configuration after adding a protocol class will include the new protocol class?

The difference is that one applies to global sessions/connections, the other to sessions built with a given session configuration.

[NSURLProtocol registerClass:…] only applies to requests sent via NSURLConnection or [NSURLSession sharedSession], and does not apply to requests sent via an NSURLSession built from an NSURLSessionConfiguration
[NSURLSessionConfiguration setProtocolClasses:] is used by requests sent via an NSURLSession that has been created from that given NSURLSessionConfiguration, and doesn't affect the NSURLSession instances created by other NSURLSessionConfigurations nor requests sent via NSURLSession

For more info, see:

here in the README
here in OHHTTPStubs's API documentation of those two methods, which explain the difference as well.

For in-depth understanding, you can also find detailed information in Apple's API documentation on NSURLSession / NSURLSessionConfiguration and the URL Loading System Programming Guide

Alright maybe I'm confused here but something doesn't seem right. I was assuming that the setEnabled method is what was registering the class under the default session configuration, which is what AFNetworking was using. But after some testing I don't know if this is the case, how are you making it so that you stub AFNetworking requests? The reason I don't think it's the registerClass function is because SWHttpTrafficRecorder uses that to register its own protocol class but when I do:
print("before: ", URLSessionConfiguration.default.protocolClasses)
print("did register AFRecordingProtocol.self: ", URLProtocol.registerClass(AFRecordingProtocol.self))    // This is my custom recording protocol looks exactly like SW's
print("after: ", URLSessionConfiguration.default.protocolClasses)

the before and after are the same:
before:  Optional([OHHTTPStubsProtocol, _NSURLHTTPProtocol, _NSURLDataProtocol, _NSURLFTPProtocol, _NSURLFileProtocol, NSAboutURLProtocol])
did register AFRecordingProtocol.self:  true
Optional([OHHTTPStubsProtocol, _NSURLHTTPProtocol, _NSURLDataProtocol, _NSURLFTPProtocol, _NSURLFileProtocol, NSAboutURLProtocol])
As you can see, OHHTTPStubsProtocol is being registered but my custom protocol isn't listed. How is OHHTTPStubs making it so that the class is being registered with AFNetworking? The session manager I'm using is an AFHTTPSessionManager and I pass it the default configuration:
[NSURLSessionConfiguration defaultConfiguration];
I make sure to setup the session configuration after I've added the protocol classes, so that isn't the issue either.
I'm using swift and objc interchangeably because my project has both and the code snippets come from different files.

Alright so I took a look at OHHTTPStubs+NSURLSessionConfiguration and see where the method swizzling is happening. Now I understand my error. I was thinking that the URLProtocol RegisterClass was what was adding OHHTTPStubs to the session configuration but this isn't the case, it's being 'swizzled' in. Thanks for pointing that out in your comment, it took me a couple of reads through to understand exactly what you meant and I also had to read through the implementation and on method swizzling.

Yup that's it.
Again, it's explained explicitly here in the README as pointed out before.

Yea I read through that a bunch of times actually but since I'm a bit of a noob I didn't understand exactly what you meant by swizzling. I thought that the registerclass was what was swizzling the method because I didn't understand the concept of swizzling. I think all the new terminology threw me off too so I'd just get confused when reading through the readme.
Again I really appreciate the help!

Ah right. Maybe we should put that NSHipster's link next to the "swizzling" mention in the README then.

Maybe that and mentioning where you're swizzling the sessions, what really did it for me is actually seeing where you guys swizzled the config.


Add CloudWatch alarm for failure of crypto autobuy state machine

Added this functionality, tested and confirmed to work.


feature: rename IMBS to transport space

Feature Description
I suggest rename imbs to transport space.
imbs is the acronym for inventory + mailbox space.
in our terms, mailbox+inventory is exactly the transport abstraction. so we could just call it transport space, which makes it clearer.
Proposed Solution
rename it
Want to contribute?

[ ] I would like to work on this issue.

Additional Context
No response

/cc @ezrasilvera
/cc @pdettori
/cc @MikeSpreitzer
/cc @clubanderson

@nirrozenbaum just to clarify, you are suggesting that imbs1 becomes transport1 in all docs/examples/ arch description ?

yes

another point to mention - once we add the cluster inventory abstraction, we might want to host the abstract inventory CRs in a space. this would probably be called inventory space (the inventory of KS, not of the transport specific).
therefore, IMBS doesn't make much sense at this point.
so potentially we can have -
WDS, Transport Space, Inventory Space.
and inventory space can run on the same space as WDS or transport to minimize number of spaces.

I have no strong opinion on that.  Transport seems like not accurate to me, but I don't have a good suggestion :-)

@nirrozenbaum is it urgent ?   Can we do a renaming PR later on ?  (i.e., during phase1 but after we stabilize our docs a little ) ?


Added global command functions

Noticed there were bindings for global commands so I simply just copy pasted the guild command functions and changed some stuff. Everything seems to work.

Approving because I no longer want to maintain this


app-colcon.meta vs colcon.meta

This is not really an issue, but more to document what I found out after spending a lot of time having compilation errors.
A lot of the online documentation mentions the need for a file named colcon.meta, eg to use a different transport mechanism.  Not much of the documentation mentions a file named app-colcon.meta.
Renaming my colcon.meta to app-colcon.meta solved my compilation problem relating to RMW_UXRCE_TRANSPORT_CUSTOM not being defined.

Both shall be used as you can see here:
https://github.com/micro-ROS/micro_ros_espidf_component/blob/dd5108214f747780ac401ab50a2b6ae97f1480fc/libmicroros.mk#L97
In that sense, colcon.meta is intended to configure the library, and app-colcon.meta is a sort of override for application-specific configurations.
Make sure you clean the library completely before modifying it and changing colcon meta parameters.

Yes, of course.  My point is that there is documentation out there that specifies colcon.meta in an application, not in the library.  I initially had a colcon.meta, inherited from a previous developer, that worked with the galactic version.  When I changed to humble, it was complaining about the transport not being configured.  Changing the file to app-colcon.meta fixed it.
Yes, systematically was doing clean-microros, clean and fullclean.

AFAIK the only documentation that this package has is its own README.
And you are right, app-colcon.meta is not documented there.
Contributions are welcomed, please feel free to open a PR to include a paragraph mentioning it.


How to train on custom scenes?

Hello, I found that in the code, we need to load a checkpoint to initialize the original scene. I am wondering if I have a custom scene, which means I need to train its NeRF first (instead of directly loading a pretrained checkpoint), how can I modify the configuration file to achieve this?
Thank you!

Hi, Thank you for your interest in our work.
In order to modify a custom scene using our framework you would need to specify the following fields in the configuration file:

base_weights - the path to your trained model representing your custom scene. (see base_weights folder).
datadir - Path to the input data, depends on its type.(I suggest you that you run download_data.sh script to see how it should look).
3.dataset_type -  the dataset_type - blender/llff/llff and is360Scene=True(real 3D scenes). Currently we support only these 3 types of scenes.
box_points_path - After you have all of these you can run the according notebook for your scene to localize a 3 box which will be your region of interest. The path to the box point array should be specify in the field .
base_dir - path where all your run logs will be saved.
expname - name of the dir which will be saved inside base_dir and will contain the logs for the current run.

Notes:

Generally, the documentation of all parameters for the configuration file is located in config_parser function in run_BlenderNeRF.py.
Our code is based on NeRF-Pytorch so I suggest you will train your scene using this repository.

Hi, Thank you for your interest in our work. In order to modify a custom scene using our framework you would need to specify the following fields in the configuration file:

base_weights - the path to your trained model representing your custom scene. (see base_weights folder).
datadir - Path to the input data, depends on its type.(I suggest you that you run download_data.sh script to see how it should look).
3.dataset_type -  the dataset_type - blender/llff/llff and is360Scene=True(real 3D scenes). Currently we support only these 3 types of scenes.
box_points_path - After you have all of these you can run the according notebook for your scene to localize a 3 box which will be your region of interest. The path to the box point array should be specify in the field .
base_dir - path where all your run logs will be saved.
expname - name of the dir which will be saved inside base_dir and will contain the logs for the current run.

Notes:

Generally, the documentation of all parameters for the configuration file is located in config_parser function in run_BlenderNeRF.py.
You can use the examples provided in the repository to see some use cases.
Our code is based on NeRF-Pytorch so I suggest you will train your scene using this repository.

Thank you for your response. I'm wondering if I can train a NeRF model, whether it's feed-forward or 360, directly using nerf-pytorch and then load it into your codebase?

Yes you can :).
We are using the same architecture as in nerf-pytorch so it should work.


🐛 [ Bug ]:  激光迷宫&导弹猎场，简单模式有bug。

您使用的版本？
3.0.9
系统版本？
Windows 10 22H2 19045.3208
您遇到了什么问题？

激光迷宫简单模式无法使用。
导弹猎场简单模式无法使用。

请提供错误截图？
一些模式需要的数据
# "type\":4,\"id\":0,\"subId\":0,"    金币小
# "type\":4,\"id\":0,\"subId\":1,"    金币中
# "type\":4,\"id\":0,\"subId\":2,"    金币大
# "type\":4,\"id\":1,\"subId\":0,"    魔方小
# "type\":4,\"id\":1,\"subId\":1,"    魔方中
# "type\":4,\"id\":1,\"subId\":2,"    魔方大
# "type\":4,\"id\":2,\"subId\":0,"    绿材料
# "type\":4,\"id\":2,\"subId\":1,"    蓝材料
# "type\":4,\"id\":2,\"subId\":2,"    紫材料

1.激光迷宫简单模式发送数据包。
{"head":{"cmdDataSplitLength":0,"cmdId":48,"cmdLength":0,"cmdSequence":"{cmdSequence}","cmdVersion":4,"headVersion":0,"timestamp":"{timestamp}","crcVerify":0,"platform":0,"reconnect":false,"sid":"{sid}","uid":"{uid}"},"type":4,"id":1,"subId":0,"targets":[],"md5s":[],"fuid":"0","pvpAreaId":-1}

激光迷宫结算数据包，这是以前的数据包，不确定最新的数据是否有改动，可用。"totalWave":60
{"head":{"cmdDataSplitLength":0,"cmdId":13,"cmdLength":0,"cmdSequence":"{cmdSequence}","cmdVersion":4,"headVersion":0,"timestamp":"{timestamp}","crcVerify":0,"platform":0,"reconnect":false,"sid":"{sid}","uid":"{uid}"},"aidTime":0,"ashTime":0,"totalDashTime":0,"boomATK":8,"bossCount":1,"bossLarge":0,"bossLargeScore":0,"bossReward":0,"bossSmall":1,"bossSmallScore":500,"comboNum":[0,0,0,0,0,0,0],"comboScore":0,"comboSign":[0,0,0,0,0,0],"comboSignScore":0,"dsArr":[],"endlessLoopCount":0,"enemiesLarge":0,"enemiesLargeScore":0,"enemiesLargeStorm":0,"enemiesMid":0,"enemiesMidScore":0,"enemiesMidStorm":0,"enemiesPasserby":0,"enemiesPasserbyScore":0,"enemiesPasserbyStorm":0,"enemiesSmall":22,"enemiesSmallScore":44,"enemiesSmallStorm":0,"engineDmgMax":1122,"enterWormholeNum":0,"flameTime":0,"hp":28161.75,"hpMax":28161.75,"improperDs":false,"isRandomBossKill":0,"m1":0,"m1Large":66,"m1LargeScore":3300,"m1Mid":233,"m1MidScore":2330,"m1Small":246,"m1SmallScore":1230,"props":[{"type":4,"sd_id":0,"quality":3,"level":0,"num":1},{"type":4,"sd_id":1,"quality":3,"level":0,"num":0},{"type":4,"sd_id":2,"quality":3,"level":0,"num":0}],"score":180000,"scoreShieldSecond":0,"secondCnt":74,"shieldSecond":0,"stormSecond":0,"stormTime":0,"success":true,"targets":[],"totalWave":60,"weaponDmgMax":2150,"wingmanDmgMax":1032}

2.导弹猎场简单模式发送数据包。
{"head":{"cmdDataSplitLength":0,"cmdId":48,"cmdLength":0,"cmdSequence":"{cmdSequence}","cmdVersion":4,"headVersion":0,"timestamp":"{timestamp}","crcVerify":0,"platform":0,"reconnect":false,"sid":"{sid}","uid":"{uid}"},"type":4,"id":2,"subId":0,"targets":[],"md5s":[],"fuid":"0","pvpAreaId":-1}

导弹猎场结束数据包，这是以前的数据包，不确定最新的数据是否有改动，可用。"totalWave":10
{"head":{"cmdDataSplitLength":0,"cmdId":13,"cmdLength":0,"cmdSequence":"{cmdSequence}","cmdVersion":4,"headVersion":0,"timestamp":"{timestamp}","crcVerify":0,"platform":0,"reconnect":false,"sid":"{sid}","uid":"{uid}"},"aidTime":0,"ashTime":0,"totalDashTime":0,"boomATK":8,"bossCount":1,"bossLarge":0,"bossLargeScore":0,"bossReward":0,"bossSmall":1,"bossSmallScore":500,"comboNum":[0,0,0,0,0,0,0],"comboScore":0,"comboSign":[0,0,0,0,0,0],"comboSignScore":0,"dsArr":[],"endlessLoopCount":0,"enemiesLarge":0,"enemiesLargeScore":0,"enemiesLargeStorm":0,"enemiesMid":0,"enemiesMidScore":0,"enemiesMidStorm":0,"enemiesPasserby":0,"enemiesPasserbyScore":0,"enemiesPasserbyStorm":0,"enemiesSmall":22,"enemiesSmallScore":44,"enemiesSmallStorm":0,"engineDmgMax":1122,"enterWormholeNum":0,"flameTime":0,"hp":28161.75,"hpMax":28161.75,"improperDs":false,"isRandomBossKill":0,"m1":0,"m1Large":66,"m1LargeScore":3300,"m1Mid":233,"m1MidScore":2330,"m1Small":246,"m1SmallScore":1230,"props":[{"type":4,"sd_id":0,"quality":3,"level":0,"num":1},{"type":4,"sd_id":1,"quality":3,"level":0,"num":0},{"type":4,"sd_id":2,"quality":3,"level":0,"num":0}],"score":180000,"scoreShieldSecond":0,"secondCnt":74,"shieldSecond":0,"stormSecond":0,"stormTime":0,"success":true,"targets":[],"totalWave":10,"weaponDmgMax":2150,"wingmanDmgMax":1032}

for bug:  其他无异常。

已修复激光普通关卡，导弹简单没问题，出现丽莎是体力或次数不足。
导弹关卡测试打了两次困难两次普通后简单关卡次数变为0，不知道是什么原因


Question: how does flubber approach morphing paths with different #s of shapes?

This isn't a bug report... more just a question. Hope you don't mind. :)
I'm the author of ShapeShifter, an SVG icon animation app. You can see an example video showing how to animate a play icon into a pause icon here: https://www.youtube.com/watch?v=2aq3ljlnQdI
Shape Shifter provides a feature called "auto-fix" which gives "best guess estimates" for path morphing animations, similar to your library. It uses an application of the Needleman-Wunsch algorithm. Right now I only have it working well with single-shape SVGs though.
Flubber looks like it handles multi-shape path morphs pretty well, so I am kind of curious how you approached creating "best guess" animations for paths with different #s of shapes? Thanks in advance if you have any tips. :)

@alexjlockwood I have too multi-shape handling (published, then removed repo), i split paths, tweening each and combine while tweening.

Just wondering... how do you determine where to split the paths?

Split by moveto command. But flubber does same thing plus triangulation matches where to split. I tried implement flubber way, works with full flubber library, but slower

ah-ha! triangulation sounds super useful. in fact, I think I may have found a demo you wrote in the past that might explain it. https://bl.ocks.org/veltman/218a162265c772f86bc26c1bc91fe58b
thanks for the tip, i'm definitely going to explore this further. :)

@alexjlockwood I looked to your code, looks amazing, but i never understand Angular (learned React, Vue, etc...) but not understand Angular. Is you have basic/es6 js version?

The general approach is as follows, to morph a shape into N shapes:

Triangulate the polygon using earcut.
Put those triangles into a TopoJSON topology.
Merge the smallest triangle into one of its neighbors.
Repeat step 3 until only N shapes are left.

@veltman I've using your app to improve visually morphing, but very slow comparing to my tools (visually best is your). And earcut+topojson very high size. Maybe there we reduce size, improve performance together?

Oops, submitted too early.

Match the resulting pieces with the destination shapes by finding the permutation that minimizes the distance between the centroid of each piece and the centroid of its destination shape. (Flubber skips this optimization if there are too many shapes, it would take forever otherwise)
Morph each pair.

@veltman Triangulation is bit slow and has some cons, it creates unexcepted line as you see in some case

@veltman For step 3:

Merge the smallest triangle into one of its neighbors.

How do you determine which neighbor to merge with? My best guess is that you would pick the neighbor with the smallest total area?
I am also wondering if flubber does any special handling for shapes that contain holes?

@dalisoft Most of the math for ShapeShifter is contained in these two directories:
https://github.com/alexjlockwood/ShapeShifter/blob/master/src/app/scripts/algorithms/AutoAwesome.ts (Needleman-Wunsch algorithm for aligning SVGs)
https://github.com/alexjlockwood/ShapeShifter/tree/master/src/app/model/paths (a bunch of Path-related math for splitting/reversing/shifting/winding/etc. SVG paths)

@alexjlockwood it currently just merges into the first neighbor by arc order so it's rather arbitrary (but deterministic). I did experiment with some alternative merging strategies but didn't see any real improvement in the look of the results. I may try to modify the triangulation to produce more regular triangles, since the current approach produces a lot of very long skinny ones (#1).

@veltman I try your solution triangulation but slow, also i am tried to decurve my points and then after triangulation recurve, not works. Can you have solution curves? Without getPointAtLength solution, because it is slower


Start supporting more specific window title detection

This should help fix some of the windows in #73. On Win32, it handles every program I have. More detail and structure can continue to be added to support other conventions on other platforms.

Thanks for submitting this, just one minor comment. Otherwise LGTM!


⚠ Fetchers has degraded performance

In d59bfef, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 311 ms

Resolved: Fetchers performance has improved in 8f9e247.


Delete entire words with ctrl + backspace

When editing text, it would be nice to be able to delete entire words by hitting ctrl + backspace. This is a common keyboard shortcut in many application. I imagine many others in addition to myself use it frequently, and are trying to use it in Excalidraw.
In any case, thank you for this useful application!

I just noticed this. It's a recent regression. We'll fix asap!


The converter is not robust against markup errors within the MediaWiki dump

When processing a dump that contains a markup error such as [[Category: Something] instead of [[Category: Something]] the script gets stuck.

Would you be happy with the problem line being removed before calling pandoc? If so, adding a try/except somewhere here should be the right place (inside function cleanup_mediawiki):
https://github.com/peterjc/mediawiki_to_git_md/blob/1ad6868c21e8f10a72574326682345495b6a899b/convert.py#L177

I would expect to either "auto-correct" such an error (which would be kind of hard) or simply skipping but without causing the overall process to fail. Together with something like an error message in the summery at the end indicating that something could not be processed because of markup errors, so I can go back and potentially fix them.

Yes. A message to stderr would seem sensible. Do you want to work on this?

Let me look into it this weekend, but no promises :)

duplicate of #4
ideally pandoc should render the exact same result as mediawiki. if pandoc cannot do that, we should use something else... for example, run the original PHP renderer in a python subprocess, then feed the rendered HTML to pandoc to generate markdown.
possible solutions:
https://github.com/wikimedia/mediawiki/blob/master/includes/parser/Parser.php - the old parser of mediawiki, written in PHP
https://github.com/wikimedia/mediawiki-services-parsoid - bidirectional parser between wikitext and HTML5, the new parser of mediawiki, written in JavaScript and PHP
https://github.com/5j9/wikitextparser - A Python library to parse MediaWiki WikiText
https://github.com/TrueBrain/wikitexthtml - Library to render WikiText to HTML, written in Python
https://github.com/tatuylonen/wikitextprocessor - Python package for WikiMedia dump processing (Wiktionary, Wikipedia etc). Wikitext parsing, template expansion, Lua module execution.
https://github.com/tatuylonen/wiktextract - Wiktionary dump file parser and multilingual data extractor, "this tool expands templates and Lua macros in Wiktionary. That enables much more accurate rendering"

I wonder if any of those tools would be significantly more robust to MediaWiki markup errors?

I think I found another gotcha with MediaWiki errors - when the file can't be converted to Markdown by pandoc, I currently skip the revision. Usually the original author immediately follows up with another revision to fix things, so this is fine. However, if not, the two sets of changes get mixed under the second author. I suspect this is complicating the user blocking functionality too.
I'm wondering now if it would be simpler to do this:

Parse the XML dump,

Save every revision as a MediaWiki file and commit that in git

Convert the final revision from MediaWiki to Markdown with pandoc as one large commit

This captures the page history more closely, and should also be significantly faster - far fewer calls to pandoc, and no git history cleanup to drop empty commits (originally done for the corner case where the MediaWiki changed but not the Markdown).


Custom small length dataset

Hi !
Thank you for this foundation model, I've been trying to finetune it on a dataset of 10K timeseries but each timeseries only has 100  values (1min frequency and values ranging from 100 to -100). The timeseries are not following each other, so I can't append them.
Using your finetuning notebook, results are not good at all currently. What I'm interested in is predicting the last 20 values for each timeseries I feed the model so the context size should always be 80 + 20.
If I'm not mistaken, currently the finetuning script takes a random context inside each timeseries of the dataset right ?
Do you think my dataset is something the model can be finetuned on or should I build a model from scratch for this task ?
Thank you very much for any insights :)

Hi, thanks for the detailed comment.
Note that the finetuning scripts are not out yet (apologies for the delay: I was on vacation and just got back, planning to releasing them in 2 weeks - hope that's OK).  I can help debug and obtain better predictions once the finetuning script is out.
Regarding your dataset, can you please explain what you mean by "the time series are not following each other"? Do you mean they are irregularly spaced?
Since you have very less data in your task, I think the foundation model should perform better than a from-scratch model, but this could differ depending on the data. Nevertheless, I suggest obtaining metrics on the from-scratch model of your choice meanwhile.

Hi @scarbain Is this resolved?

Hi @ashok-arjun, unfortunately I haven't had time to get back on this. I'll let you know if I do :)


PackageId:CPython3.Exe.x64;PackageAction:DownloadPackage;ReturnCode:0x800706ba;

This issue has been moved from a ticket on Developer Community.

Describe your issue here
===DO NOT EDIT BELOW THIS LINE===
PackageId:CPython3.Exe.x64; PackageAction:DownloadPackage; ReturnCode:0x800706ba;

Original Comments
Feedback Bot on 4/22/2023, 06:30 PM:
(private comment, text removed)

Original Solutions
(no solutions)

It seems the problem occurs when the CPython installer fails. This could be due to network timeouts, corporate firewall policy, VPN configuration, anti-virus, or other problems. See if a retry helps, and if not, see if you can download the Python runtime directly from https://python.org/ and install it manually.
If that doesn't solve your issue, we could use more detailed logs to see what's really going on. Can you please collect more logs by running http://aka.ms/vscollect.exe and uploading %temp%\vslogs.zip and reopen the ticket if it persists? Thank you!😊


Bug: swc incorrectly un-strings object keys with periods in them

Try running swc on the following js:
const x = {
  'foo.bar': true
}

It will output
'use strict';
var x = {
        foo.bar: true
};

The foo.bar is not correct syntax. swc needs to keep string object keys that contain js special characters as strings. It's probably easier to just keep all string object keys as strings.

Thank you for the reporting an issue.
I fixed this with<EMAIL_ADDRESS>


Stops plants from growing when they do not have enough resources

About the PR

Plants were growing without the need for water or nutrients.
Same as the closed pull request https://github.com/space-wizards/space-station-14/pull/21502 but with the branch separate from master.
Why / Balance

This issue was brought up in https://github.com/space-wizards/space-station-14/issues/21494 .
Plants should not grow without the resources needed to build their cell structures.
Technical details

Checks if a plant is missing nutrients or water every update instead of waiting for _random.Prob(0.35f) to return true.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

fix: You now need water and nutrients to grow plants in hydroponic trays.

A nitpick about the CL entry:
"You now need water and nutrients to grow plants in hydroponic trays." is not ideal because this has always been the intention and doesn't explain what was fixed. Something like "Plants can no longer grow without sufficient water and nutrients." better explains what was fixed.

Thank you for your contribution. You were missing a period at the end of your Changelog. Please review the Changelog guidelines.


Napi setup

Agreed project structure at last meeting.
Project structure includes:

placeholder datastax driver code
datastax examples, that are added to CI
datastax unit tests, that are added to CI
datastax integration tests, that are currently NOT added to CI
structure for napi project
clippy and js-formatter in CI
Datastax copyright notice moved to separate file.

Shouldn't we close this pull request?


Keep track of k8s api server usage

@yaron2 can you provide some info on this issue please?

Updated the description.

We should set limits on the containers as well.

@zroubalik yes that's a must. a potential memory leak or CPU hog in KEDA should not drain the node running it.

Great, thanks for the update!


test: added tests for isJsxElement in ast-js-utils

Summary

part of  #1023
added tests for isJSXElement function in js-ast-utils

Test Plan

./rome test --all tests passed

Thank you! We actually have:
import {template} from "@internal/js-ast-utils";

template.statement`foobar;`;
template.expression`bar()`;

So you wouldn't need to have helper and would simplify this a lot. I'll merge this anyway though since it's still valuable. Do you want to submit a follow up PR with the helper changes?


Provide a way to disable app data cloud backup

Problem description
Currently, on Android, one can disable cloud backup by setting AndroidManifest.xml application's  attribute android:allowBackup to "false" (see https://developer.android.com/guide/topics/data/autobackup#EnablingAutoBackup) by including the following in config.xml:
  <widget ... xmlns:android="http://schemas.android.com/apk/res/android" />
  <edit-config file="AndroidManifest.xml" target="/manifest/application" mode="merge">
    <application android:allowBackup="false" />
  </edit-config>

On iOS, the localStorage implementation is based on a Documents file, which is automatically backed up (see https://developer.apple.com/icloud/documentation/data-storage/index.html) and this behavior cannot be configured.
Expected behavior
Tabris.js should provide API to disable backup for app storages on iOS. See native API.

One way to approach this is to give developer an API that would allow him to set NSURLIsExcludedFromBackupKey flag. The other way would be to use Cache directory which is never backed up. This would also mean that we need to give the developer ability to choose between storing files in Documents or Cache directory. I think we could do both in long run but as an immediate solution I would propose the first one.

BackupWebStorage preference?