Must needed Banner or Hero component

You must needed add banner or hero component

Hi @Atikur5500,
Thank you for your suggestion regarding the addition of a banner or hero component to Keep React. We appreciate your input and are always excited to explore ways to enhance our component library based on the community's needs.
A banner or hero component is a fantastic idea, and it can serve multiple purposes in web applications. It can help in drawing user attention, making announcements, promoting key features, or simply creating an attractive top section for a webpage. We believe that such a component would be a valuable addition to our library.
We're thrilled to inform you that the development team is already considering this feature as part of our roadmap. We understand its importance and the positive impact it can have on user experience and web design. We'll work diligently to create a banner or hero component that's versatile, customizable, and easy to integrate into web projects.
We're committed to providing developers with a rich set of components that simplify web development, and your feedback plays a crucial role in shaping the future of Keep React.
If you have any more specific ideas, requirements, or feedback related to this feature or any other aspect of our library, please feel free to share them with us. We value your insights and look forward to making Keep React even more powerful and user-friendly.
Once again, thank you for your contribution and for helping us make Keep React better with each suggestion.
Best regards,
Md Ariful Islam,
The Keep React Team


fixes #66 - removing minimatch dependency

(it's not a direct dependency)
happy #hacktoberfest :)

Thanks! 😊


Lock grunt-contrib-connect to the last version that supports npm 0.10

Version 1.0.1 has a breaking change for npm 0.10, but 1.0.0 works.

Abandoning.


Support Policy Expiration

Transactions will now be invalid if the block height is greater than the transaction policy expiration field. For more context see https://github.com/FuelLabs/fuel-specs/issues/593
This is being added to the VM

Blocked by:

https://github.com/FuelLabs/fuel-core/pull/2447
https://github.com/FuelLabs/fuel-vm/pull/871
https://github.com/FuelLabs/fuel-specs/pull/616


Measuring performance during tests

Probably a plug-in for jslitmus, bomerang, dromaeo or something like that and then capture the result.

Closing as this can be solved in a plugin now.


hypercli v:latest (new cask)

Where's the download link on the homepage?
I only see a "private beta"

@adidalal Thanks for your review.
The download link is in the page.

@carmark Are you in charge of that page? Because brew upgrade is absolutely unnecessary, there.

@vitorgalvao
Thanks for your point this, I have updated the source of page, but it may talk a while to show the modification.

For uninstall, shouldn't opt/bin/static and all subdirectories also get removed? Please add that / change uninstall accordingly

Also, for s complete uninstall, I’m guessing we could at least get a pkgutil:. We have scripts and documentation to make as complete as possible.

Package ID: sh.hyper.hyper

Thank you for the contribution. It needed some fixes, so they were made in 5dbd1136ebaa4ab9a5c2902f3fae0bbf9760771c. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.


Include File max side for file labeling, as well as max number of labels

We need to include that the max size for a file is 50MB here (it states there is a max size, but doesn't say what it is), as well as the max number of applies for an application is 100 per day.  this is crucial information that should be added here.

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

ID: 9a3dc78f-a44c-de54-2a66-3f830345a90d
Version Independent ID: fdc60175-126c-a9a0-acd3-1f8e8c3b3263
Content: Troubleshooting content inspection errors - Cloud App Security
Content Source: CloudAppSecurityDocs/troubleshooting-content-inspection.md
Service: cloud-app-security
Product: ****
Technology: ****
GitHub Login: @shsagir
Microsoft Alias: shsagir

@gesmith-Ms Thanks for contacting Microsoft with feedback about our product documentation. Depending on the complexity, it could take from a few business days to a few weeks to look further into your question, issue, or suggestion. We'll contact you if we need further information, and you'll receive an email message each time we "@mention" you in a comment.
Thanks again for submitting your feedback. Your suggestions help improve our documentation.

@gesmith-Ms Thanks for your feedback. We've submitted a documentation change based on your suggestion, and the updated article should be live by tomorrow. Since labels are not related to this troubleshooting guide, it was not added but can be seen in the following topic: Azure Information Protection integration.
Please let us know if there's anything more we can do for you. To follow up on this issue, leave us a comment with an @mention.


undefined function used?

Call to undefined function CanGelis\PDF\snake_case()

fixed by #7

thanks


[Refactor] Paloma message attestation router

As an engineer, I want to define message attestation in a more modular way so that extending attestation handlers with hooks allows me to work on advanced protocol features without bloating technical dept.

DoD

[ ] Extract existing logic into modular handlers
[ ] Allow for dynamic assignment of handler to message type
[ ] Introduce hooks to run before and after the handler

Other
⏰ Estimate
~ 3 days

@byte-bandit i need a description of this ticket for this week's Weekly Wings ser. Will you please provide the full requirements and changes?

@taariq Done. These changes will come in parallel with https://github.com/VolumeFi/paloma/issues/875 as I've started the refactoring while working on the ERC20 token transfer and realised one simple message wasn't going to cut it any longer in order to support n tokens per target chain.


d3plus grouped barchart hides some lines on the x-axis

d3plus grouped barchart hides some lines on the x-axis..The beginning and the ending lines on the barchart are hidden.How can I solve this

Thanks for reporting this @deveid, this was one we've been trying to hunt down. Should be fixed now!


Use native npm commands

fixes: #1 #2 #3

Hey @FDiskas,
Thanks for your PR. But using shell command to get package data is not a stable solution as you might face problem to identify errors e.g. package not found or any internal errors, always prefer to use programmatical API instead of CLI command.
As an instance, do you prefer to use MySQL CLI command to fetch data instead of mysql programming API in your application?

reinventing the wheel

How you going to solve feature issues like: Private modules, custom registry like bitsrc provides.
And about MYSQL - if you need just single stuff from mysql I wil use mysql cli instead of some big and useless wrapper. It's a good example in your repo lodash and babel. Why it is used if it's not used? :)

Hey @FDiskas, you pointed out some unnecessary deps which needs to be removed.
I am not thinking about private modules, custom registry.

@rousan Should I resolve conflicts or delete this pull request?

Closing it.


Flaky (not reproducable) Validation

Environment

Nuget Package: JsonSchema.Net
Nuget Version: 3.2.1
OS: Kubernetes / Docker / dotnet:6.0-alpine
.Net Target: .NET 6.0

Describe the bug
The Validate(JsonNode? root, ValidationOptions? options = null) returns under unknown condition that "Expected value to match one of the values specified by the enum" even though, the field value is valid and a immediate retry succeeds.
To Reproduce
We tried to reproduce the error by consuming the same message from kafka/filesystem/encoded/decoded/brute-force-parallel and failed.
Expected behavior
Enums should always be detected correctly with the same value
Additional context
We consume messages (~30/second) from Kafka and validate them against a (cached) JsonSchema:
Our method:
public JsonMessageValidationResult Validate(JsonDocument json, SchemaOptions schemaOptions)
{
	JsonSchemaInformation schemaInformation = _schemaLoader.Load(schemaOptions.SchemaFileName, schemaOptions.SchemaType);
	JsonSchema jsonSchema = schemaInformation.JsonSchema;
	ValidationResults validationResults = jsonSchema.Validate(json.RootElement, ValidationOptions);

	if (!validationResults.IsValid)
	{
		validationResults = jsonSchema.Validate(json.RootElement, ValidationOptions);

		if (validationResults.IsValid)
		{
			_logger.JsonValidationErrorFlaking();
		}
		else
		{
			_logger.JsonValidationErrorsDetected(validationResults.NestedResults.Count, schemaOptions.SchemaType, schemaInformation.ApiVersion);
			_logger.JsonValidationTrace(schemaOptions.SchemaType, schemaInformation.ApiVersion, AsString(json));
		}
	}

	ValidationResult validationResult = new(validationResults.IsValid, GetValidationErrors(validationResults), schemaInformation.ApiVersion);
	return new JsonMessageValidationResult(validationResult);
}

In the code you see that we do a first validation, which might fail and then we do the same validation, with the  same JsonDocument.RootElement and the same ValidationOptions and it will NOT fail, e.g. reach "_logger.JsonValidationErrorFlaking()". We are stumped by that and can't really think on what we are doing wrong.
The experienced error rate is 1/2_000_000.
Schema:
{
  "type": "object",
  "properties": {
    "mailpieceId": {
      "$ref": "#/definitions/MailpieceId"
    },
    "timestamp": {
      "$ref": "#/definitions/DateTime",
      "description": "The timestamp of this state"
    },
    "state": {
      "$ref": "#/definitions/State",
      "description": "The clearance state of the parcel"
    }
  },
  "required": [
    "mailpieceId",
    "timestamp",
    "state"
  ],
  "additionalProperties": true,
  "$id": "#ClearanceEvent",
  "$schema": "http://json-schema.org/draft-07/schema",
  "definitions": {
    "MailpieceId": {
      "type": "string",
      "description": "18 digits swiss barcode (alt Sendungsnummer, Identcode, shipment number)",
      "pattern": "(98|99)[0-9]{16}",
      "$id": "#MailpieceId"
    },
    "DateTime": {
      "type": "string",
      "description": "String @format date-time,\nas defined by RFC 3339, section 5.6,\nfor example: 2014-05-22T10:42:42.542+00:00.",
      "format": "date-time"
    },
    "State": {
      "type": "string",
      "enum": [
        "DENIED",
        "GRANTED"
      ],
      "$id": "#State"
    }
  }
}

Event:
{
        "mailpieceId": "996013622604113322",
        "timestamp": "2019-10-10T22:26:20.489571+02:00",
        "state": "GRANTED"
    }

Thanks for the report.

They are run in parallel. I tried to reproduce it in the tests too.

That gives me a clue, thanks. I should be able to look into it this week.

@seilc1 are you always using the schema you posted, or does the schema vary?  If you're using the same schema, is it the same .Net instance, or are you deserializing each time?

The implementation of the schemaLoader holds a ConcurrentDictionary. The schema is loaded only once and stays the same during the application lifetime (it is embedded in the assembly as a resource). The SchemaLoader is registered as Singleton at the container.
public class JsonSchemaLoader : IJsonSchemaLoader
{
    private readonly ConcurrentDictionary<Type, JsonSchemaInformation> _resolvedSchemaInformation = new();

    public JsonSchemaInformation Load(string schemaFileName, Type schemaType)
    {
        if (_resolvedSchemaInformation.TryGetValue(schemaType, out JsonSchemaInformation? resolvedSchemaInformation))
        {
            return resolvedSchemaInformation;
        }

        Assembly schemaAssembly = schemaType.Assembly;
        string? version = schemaAssembly.GetName().Version?.ToString();
        Stream? schemaStream = schemaAssembly.GetManifestResourceStream(schemaFileName);

        if (string.IsNullOrEmpty(version) || schemaStream == null)
        {
            throw new ArgumentException($"No schema found for schema file name: {schemaFileName} and type: {nameof(schemaType)}");
        }

        using StreamReader reader = new(schemaStream);

        // JsonSchema only supports draft version 06+
        JsonSchema jsonSchema = JsonSchema.FromText(AllowAdditionalProperties(reader.ReadToEnd()));
        JsonSchemaInformation schemaInformation = new(jsonSchema, version);

        _resolvedSchemaInformation[schemaType] = schemaInformation;

        return schemaInformation;
    }

    private static string AllowAdditionalProperties(string schemaJson)
    {
        return schemaJson.Replace("\"additionalProperties\": false", "\"additionalProperties\": true", StringComparison.InvariantCulture);
    }
}

Well, I was able to replicate the issue.  Still not sure what the problem is, though.  In one test run, I saw an error on the regex.
I updated the error message for enum as well so that it includes the values.  (I didn't before because there could be a lot of values, and they're not always strings.)  This returned the error message Expected value "GRANTED"to match one of the values specified by the enum["DENIED","GRANTED"]`` which is obviously incorrect.

What?!

The only thing I can think of is that context.LocalInstance is somehow changing between the evaluation of the if condition and the next statement.  But the context is self-contained.  It's not shared outside of the thread.

I don't think it matters because I see no reason and I also think the compiler will optimize this away.
However, you access Values rather than _values.  Not sure if it's by intention.

This test shows that the issue is actually in the comparer.
[Test]
public void Issue337_ParallelComparisons()
{
	string? failed = null;

	try
	{
		var arrayText = "[\"DENIED\",\"GRANTED\"]";
		var valueText = "\"GRANTED\"";

		var array = JsonNode.Parse(arrayText)!.AsArray();
		var value = JsonNode.Parse(valueText);

		Parallel.ForEach(Enumerable.Range(1, 1000000).ToList().AsParallel(), i =>
		{
			var array0 = array[0];
			var array1 = array[1];
			var array0Hash = JsonNodeEqualityComparer.Instance.GetHashCode(array0);
			var array1Hash = JsonNodeEqualityComparer.Instance.GetHashCode(array1);
			var valueHash = JsonNodeEqualityComparer.Instance.GetHashCode(value);

			if (array0Hash != valueHash && array1Hash != valueHash)
			{
				failed ??= $@"Hashcode failed on iteration {i}

value: {valueHash}
array[0]: {array0Hash} - {array0.ToJsonString()}
array[1]: {array1Hash} - {array1.ToJsonString()}";
				return;
			}

			if (!JsonNodeEqualityComparer.Instance.Equals(array0, value) &&
				!JsonNodeEqualityComparer.Instance.Equals(array1, value))
			{
				failed ??= "Equals failed";
			}

			if (!array.Contains(value, JsonNodeEqualityComparer.Instance))
			{
				failed ??= "Contains failed";
			}
		});
	}
	finally
	{
		if (failed != null)
		{
			Console.WriteLine(failed);
			Assert.Fail();
		}
	}
}

It looks like while my equality comparison takes into account the underlying value of a JsonNode being a JsonElement, the hash code calculation doesn't.

Ultimately, it looks like JsonNode isn't threadsafe.  If I change the test to
[Test]
public void Issue337_ParallelComparisons()
{
	string? failed = null;

	try
	{
		Parallel.ForEach(Enumerable.Range(1, 1000000).ToList().AsParallel(), i =>
		{
			var arrayText = "[\"DENIED\",\"GRANTED\"]";
			var valueText = "\"GRANTED\"";

			var array = JsonNode.Parse(arrayText);
			var value = JsonNode.Parse(valueText);

			var array0 = array[0];
			var array1 = array[1];
			var array0Hash = JsonNodeEqualityComparer.Instance.GetHashCode(array0);
			var array1Hash = JsonNodeEqualityComparer.Instance.GetHashCode(array1);
			var valueHash = JsonNodeEqualityComparer.Instance.GetHashCode(value);

			if (array0Hash != valueHash && array1Hash != valueHash)
			{
				failed ??= $@"Hashcode failed on iteration {i}

value: {valueHash} - {value.AsJsonString()}
array[0]: {array0Hash} - {array0.AsJsonString()}
array[1]: {array1Hash} - {array1.AsJsonString()}";
				return;
			}

			//if (!JsonNodeEqualityComparer.Instance.Equals(array0, value) &&
			//	!JsonNodeEqualityComparer.Instance.Equals(array1, value))
			//{
			//	failed ??= "Equals failed";
			//}

			//if (!array.Contains(value, JsonNodeEqualityComparer.Instance))
			//{
			//	failed ??= "Contains failed";
			//}
		});
	}
	finally
	{
		if (failed != null)
		{
			Console.WriteLine(failed);
			Assert.Fail();
		}
	}
}

(moving the parsing inside the Parallel.ForEach(), it works fine.
I don't know that I can do anything about this.

tldr; of the linked issue ☝: JsonElement.GetString() isn't thread safe.  I use this in my equality checking.  There's no other option, so I can't really do anything about this until the .Net team fixes that.
I suggest implementing a retry.  The odds are low that it'll happen on successive validations of the same message.  Happily, validation is really quick.  Unhappily, there's not really any indicator other than a failed validation (which could happen anyway) that a concurrency error happened.

It's been added to the .Net 8 milestone.

I'm going to close this since there's not else I can really do about this.

Thank you very much for your super and fast support! Our team and I were amazed :-)


Fix Got devices failed

issue #1
Possibly due to a problem with the catlink server, the following error may occur when logging in.
Login 000000000 failed: [{'returnCode': 2001, 'toast': False, 'msg': '用户不存在，请先注册', 'data': {}, 'timestamp': '1716092971910', 'success': False}, {'platform': 'ANDROID', 'internationalCode': '00', 'mobile': '000000000', 'password': 'password', 'noncestr':<PHONE_NUMBER>792, 'sign': 'sign'}]

When the login fails, Account.token will be set to an empty string. Due to the lack of Account.token, the response from the device is {'msg': 'Device offline or other faults','returnCode': 1000'}. However, the system Will not Login again because returnCode != 1002. This resulted in ongoing issues getting the devices.

Hi there
does this PR relates to https://github.com/hasscc/catlink/issues/13 ? The integration stopped working from my side and I think it may relates to the above PR.

works like a charm, thanks a lot!


doc: update roadmap

What type of PR is this?
/kind documentation

What this PR does / why we need it:
update roadmap
Which issue(s) this PR fixes:

Fixes #
Special notes for reviewers:

Does this PR introduced a user-facing change?

Additional documentation, usage docs, etc.:

/cc @x893675


Backblaze B2 storage driver

W.I.P. version of the Backblaze B2 storage driver.
Please keep in mind that the PR is still under development and not yet ready for a review :)
TODO:

[x] Code refactor
[x] Tests
[x] Documentation

Added some tests over weekend. Will go ahead and merge this initial implementation.


value should not be set when using ratioMetrics

When using ratioMetrics, it doesn't make sense to include or require objectives.value, and should be removed

What does value mean exactly? To me it reads the same as target

@ian-bartholomew I am looking to pick this up but I am not entirely sure if value is needed for thresholdMetric or not needed at all.
I made a Draft PR to get some feedback here https://github.com/OpenSLO/OpenSLO/pull/56


WIP Add Fedora 23 support

With this PR I'm fixing defaults for:

CentOS6
CentOS7
Fedora 23

ability to override is available (as before)

Hmm. Something wrong with group_vars.

Thanks for you PR.


function deployment fails in kubernetes version >=1.19

When installing kubeless in Kubernetes version <=1.18, function deploying and ui work perfect.
However, for a cluster in  Kubernetes version 1.19 and 1.20 (tried both), the install seams to work.
Buy when a function is deployed, gives error:
NAME    NAMESPACE       HANDLER         RUNTIME         DEPENDENCIES    STATUS
hello   default         test.hello      python3.6                       MISSING: Check controller logs
The UI also does not work in version 1.19 and 1.20.
Can you please replicate this, and see if this can be fixed.
Thanks,
J.

I notice Python 2.7 is no longer supported in the later version of Kubeless.  You may want to try specifying Python 3.6 as a runtime.
Also, any failures you observe with sample code could possibly be due to syntax difference between Python versions.  eg., print event vs. print (event).  You may want to check on the pod log file to observe the details of failure.


Add instructions for BASH completion installation

Git BASH completion always has to be installed along with Elegant Git
completion in order to have the last completion working.
The contribution:

[x] updates all affected documentation
[x] provides commits messages which comply with the CONTRIBUTING.md > Committing the changes rules
[x] updates the completion scripts if requires
[x] complies with all requirements from README.md > Hands-on development notes

@bees-hive/elegant-git-maintainers, please review.
@alexbeatnik please test instructions and let me know if they work on your environments.

@alexbeatnik thank you for the testing.


Bug: Allow typescript-eslint/utils@7?

Describe the bug
Right now, this package's package.json lists an explicit dependency on @typescript-eslint/utils@6:
https://github.com/azat-io/eslint-plugin-perfectionist/blob/ca69069bd3f44e7d603d429a8cda2630aacc3e7b/package.json#L78
@typescript-eslint/utils@7 should be listed as compatible. The major version just dropped support for older ESLint, Node, and TypeScript versions.
-    "@typescript-eslint/utils": "^6.13.0",
+    "@typescript-eslint/utils": "^6.13.0 || ^7.0.0",

Code example
n/a
ESLint version
n/a
ESLint Plugin Perfectionist version
2.9.0
Additional comments
I wasn't sure which issue template to use. Sorry if this was the wrong one. 😅
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

@azat-io I went ahead and sent #135. Cheers!


#40 Add missing vtkPointData.h include statement

Closes #40.

For trivial fixes like this, could please just submit a PR without an accompanying issue. It just adds unnecessary noise to the reviewing process. Just add the description of the problem to the comment area of the PR submission. Thanks.


Chocolatey

Hi!
Please, consider add git-secrets to Chocolatey.
https://chocolatey.org/courses/creating-chocolatey-packages
Thank you.

As an alternative, git-secrets is available via https://pre-commit.com/ which can set it up on a per-repository basis for you.


Addition of notes endpoint and object, addition of properties to obswell object

Changes required for development of SAGeoData -> SWIMS interface. Mostly additions to the obswell object, and a new object and endpoint for obswell notes.
Note - considered using wcr-coordinatedata object instead of easting, northing, zone etc. in obswell object, but wcr-coordinatedata has some properties that are possibly not relevant in the obswell context (such as survey method). Suggest possibly create a more generic 'coordinatedata' object which wcr-coordinatedata could inherit from?

considered using wcr-coordinatedata object instead of easting, northing, zone etc. in obswell object, but wcr-coordinatedata has some properties that are possibly not relevant in the obswell context (such as survey method). Suggest possibly create a more generic 'coordinatedata' object which wcr-coordinatedata could inherit from?

Yes, wcr-coordinatedata is in line with what goes into the well completion report.
We could create an entity that holds only the location details like lat, long, datum, easting, northing and zone.
Please add me as a collaborator on your project in apicurio and I'll start making those changes

@n8-coder I've made the changes to coordinate data in the obswell object as requested, and added example values for monitoringNetworks property.


Added resource config to the PnP Controls sample

Q
A

New feature?
yes

What's in this Pull Request?
Added the PnP controls resource file configuration to the PnP Controls sample web part.

thx Elio, appreciated.


Running kuksaval-server unittests in GH actions

We should run the kuksa.val server unittests in GH actions, then we could disable Jenkins, as the setup is not very stable (mainly due to missing disk space, but also sometimes due to the numerous SSH tunnels around the world....)
(Coverage in GH actions is a bonus, but the unit test shall be there)
It might also free some capactiy to throw more runners to GH actions

Would also close #144

Actually we have an action https://github.com/eclipse/kuksa.val/blob/master/.github/workflows/kuksa_val_unittest.yml
We need to check, it does what we want, and the triggers are not correct: Should build any time val-server is updated

Builds on (release) tags and works. No need to invest further effort here for now


Indicator for whether a story has been translated already

There should be an indicator of some sort to let people know whether a story has been translated into a particular language already, so that they don't accidentally duplicate work that has already been done.
The easiest way to do this might be to import the index of translated stories from the website into a json file and show a list of completed languages at the top of the page for each story.
It probably won't be possible to actually do the comparison between the current language and already translated languages without the language names being standardized. Right now people might fill in the ISO language code, the English name of the language, the local name, or something else. There could be a dropdown menu with a list of languages, but we want to avoid limiting possible target languages to a pre-determined list, so this isn't a good solution.

Moved from @espensj's comment here:
This is my suggestion for a new GUI:
You start here:

You type your language. For 99% of languages we are likely to ever encounter it's simply a matter of matching English and the name of the language in the language itself, such as "Danish" and "dansk". If either of these two words are entered as the name of the language, you get to this page:

If the language is not recognized (there are no stories translated yet) you get the same page, except the link. If you click on the link "Click here for a full list", you get this:

Note that I've added sorting function and "recommended" (not the best word choice, perhaps). The idea is that some stories are quality assured – by ASP, or the Global ASP team, since there are quite few non-edited stories out there.

Here is a list of about 350 languages with English and native names: https://en.wikipedia.org/wiki/List_of_language_names
If that's not enough (and sure it's not!) then this list of ISO-639 has English names and some native names (I think they will often be the same, since it's not like English has it's own name for every tiny language).
https://en.wikipedia.org/wiki/ISO_639:a
I say we start with the first, and if we get a language within a year that's not on that list I'll eat my hat (as we customarily do in Norway in these situations).

Interesting ideas. I like the sort functionality and we might think about adding something like this in future, though I don't know how practical it would be at the moment.
I've also been thinking about this issue, and I think there is a workable solution that addresses the most immediate problem (we need to know whether the story we are translating is already available in the target language).
I think you hit on the key to the problem earlier. The most straightforward solution is to realize that we don't need to match the story against every possible language -- we just need to match the languages that Global-ASP already has. That is a very different (and much more tractable) problem: all it requires is to set a variable containing all the Global-ASP languages and story codes, something like this:
var gasp = [{"no":"0253,0123,0321"},{"sv":"0023,0231,0123"},{"zh":"0065,0331,0123"}]

Then, it's a very easy matter to simply match the input target language and the index number of the next/previous story. If the story has been translated already we could either skip it or gray it out until a new language was selected.
I was going to say we should skip it as you suggested, but then I realized that some translators (ahem) are going to want to translate into more than one language, and would miss that opportunity if the app skips stories. So I'm going to suggest that the translation area be greyed out, and a message appear saying "this story has already been translated into xx language; please choose another language or skip to the next story" or something similar.
This allows us to circumvent the entire list of languages issue completely -- we don't need to suggest any languages other than the ones that are currently available (and they show up as suggestions underneath the normal textbox). If you're translating into a new language, you can call it whatever you want and we'll sort it out later. The app certainly doesn't need to know that much detail :smile:

Sounds good. But I'm falling asleep. You should too!
E
On 16 October 2015 at 22:45, dohliam<EMAIL_ADDRESS>wrote:

Interesting ideas. I like the sort functionality and we might think about
adding something like this in future, though I don't know how practical it
would be at the moment.
I've also been thinking about this issue, and I think there is a workable
solution that addresses the most immediate problem (we need to know whether
the story we are translating is already available in the target language).
I think you hit on the key to the problem earlier. The most
straightforward solution is to realize that we don't need to match the
story against every possible language -- we just need to match the
languages that Global-ASP already has. That is a very different (and much
more tractable) problem: all it requires is to set a variable containing
all the Global-ASP languages and story codes, something like this:
var gasp = [{"no":"0253,0123,0321"},{"sv":"0023,0231,0123"},{"zh":"0065,0331,0123"}]
Then, it's a very easy matter to simply match the input target language
and the index number of the next/previous story. If the story has been
translated already we could either skip it or gray it out until a new
language was selected.
I was going to say we should skip it as you suggested, but then I realized
that some translators (ahem) are going to want to translate into more than
one language, and would miss that opportunity if the app skips stories. So
I'm going to suggest that the translation area be greyed out, and a message
appear saying "this story has already been translated into xx language;
please choose another language or skip to the next story" or something
similar.
This allows us to circumvent the entire list of languages issue completely
-- we don't need to suggest any languages other than the ones that are
currently available (and they show up as suggestions underneath the normal
textbox). If you're translating into a new language, you can call it
whatever you want and we'll sort it out later. The app certainly doesn't
need to know that much detail [image: :smile:]
—
Reply to this email directly or view it on GitHub
https://github.com/dohliam/gasp-translator/issues/4#issuecomment-148887526
.

I say we start with the first, and if we get a language within a year that's not on that list I'll eat my hat (as we customarily do in Norway in these situations).

I would be very cautious about this. The danger with "comprehensive" language lists is that they never are. For example, I can think of more than half a dozen languages that we are working on or targetting right now (let alone within a year) that aren't on the list you linked: Taishanese (also known as Toisanese, Hoisanva), Jamaican (also known as Creole/Patois), Suzhou dialect, Musqueam, Sto:lo, Kansai dialect (or any other dialect of Japanese), Quechua (!)
None of the Italian languages are listed (e.g. Calabrian). The only Arabic dialect listed is Egyptian. There isn't even anything about Quebecois French! You don't have to eat your hat, but I hope you see my point... :sweat_smile:

I would love to see Quechua and Thaishanese, but by point is, will we get then? Within a year (or a reasonable time span?) For now I think 350 languages will cover 99% of our needs. And when someone adds Thaishanese (which Melanie speaks, by the way), which I'm sure you can conjure up, then it's just a matter of doing it manually. I don't think we will be overwhelmed by manually adding these languages.

Yes, the languages I listed above are translations we are working on right now.

@espensj This feature has now been added to the Norwegian translator -- please test it out. If it works for you, we can add it to the main gasp-translator application.

Something is very wrong here...

Is it also possible to make  in the target language box leave the box? Right now it gives a new line, which makes no sense.

Right now if you press the  key in the target language box you get a new line. It should leave the text box.

Strange, I don't see that at all...:

Did you remember to reload the page?

Actually, I see the problem -- at some point you must have inadvertently entered a linebreak (most likely by pressing "enter"), and this was saved in your local storage... which then screwed everything up. We can fix this by stripping all linebreaks in the language box.

This should now be fixed... let me know if it works for you.


Update .net SDK based on latest official swagger spec for sql 2017-03-01-preview

Description

Update .net SDK based on latest official swagger spec for Sql 2017-03-01-preview, here's the link for that committed PR:
https://github.com/Azure/azure-rest-api-specs/commit/b3f4823378bab4812792ad0b90da6e8a8385a2f1

include newly added testcase TestCancelDatabaseOperation for list and cancel database operation API, and new test passed

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] I have read the contribution guidelines.
[x] The pull request does not introduce breaking changes.

General Guidelines

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[x] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[x] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
[x] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

@jiawang003,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

You can use this instead of checking for record mode
https://github.com/Azure/azure-sdk-for-net/blob/psSdkJson6/src/SdkCommon/TestFramework/ClientRuntime.Azure.TestFramework/Utilities/TestUtilities.cs#L84

updated to use TestUtilities.Wait instead of checking for record mode

resolved all review comments so far

@shahabhijeet the last published version was 1.6 ( https://www.nuget.org/packages/Microsoft.Azure.Management.Sql ) so 1.7 in assemblyversion is already correct.


Feature ask for prompt develop

Would be great to propose an idea and let it improve "a certain amount of time" and choose the best, or even better, choose to reiterate the prompt suggestions base on it, and ask for modification, that way it would be much easier to generate a prompt, as of for now you should use another AI or write a good prompt yourself

https://github.com/mshumer/gpt-author/pull/26 I've got a similar idea in mind, feel free to add more suggestions on how we can improve the prompt generation as well 👍


Visualize workflow

Using cwl functionality

Already covered by a different project:
https://view.commonwl.org/
https://github.com/common-workflow-language/cwlviewer


Complete: Instruction Tasks 41-48 (PAWS-X and PEC)

I have created a total of 50 new instruction tasks that were generated from two datasets:

PAWS-X
PEC

Within the set of instruction tasks, they are organized into three unique tasks (41, 42, and 43) as well as variations of said tasks that include differing languages from the seven total supported languages.
If a fourth unique instruction is required per the homework requirements, please let me know.

@swarooprm I have updated my instruction task with the following changes:

For translation tasks, the fields Input_language and Output_language has been added
For classification tasks, the definition is more explicit on the two types of classifications
For all tasks, the explanations have been modified to use more natural language

Please let me know if this looks good. Thank you!

Perfect. Thank you!
I will be making the formal submission to the repository today. I see the deadline has been extended to October 15th, so that is good news! :tada:

@jwande18  Just wanted to update that, if you plan to submit these tasks to the main Github repo and join our research project, Monday (27th Sept) is the internal deadline. It should not take more than a couple of minutes to submit.


Interest types mapping

This PR closes #97 with the following changes:

Add support for interests categories rather than just interests types
Add text labels even when no share %, either 'Owns' or 'Controls'
Fix unknown interests dashed lines & improve support for unknown interests
Fix edge stroke widths based on share %

For this example (https://github.com/openownership/data-standard/blob/main/examples/bods-package-annotations.json) I am getting the following error:
Uncaught runtime errors:
ERROR
e is undefined
./src/model/edges/edges.js/v/o</C<@http://localhost:9000/main.js:2:19671
./src/model/edges/edges.js/v/o<@http://localhost:9000/main.js:2:19859
v@http://localhost:9000/main.js:2:18938
_@http://localhost:9000/main.js:2:21074
./src/index.js/a<@http://localhost:9000/main.js:2:15492
h@http://localhost:9000/main.js:2:10686
./demo/index.js/e/<@http://localhost:9000/main.js:2:10509
c@http://localhost:9000/main.js:2:719599
./node_modules/regenerator-runtime/runtime.js/a/s._invoke@http://localhost:9000/main.js:2:719353
./node_modules/regenerator-runtime/runtime.js/_/</<@http://localhost:9000/main.js:2:719958
t@http://localhost:9000/main.js:2:524
u@http://localhost:9000/main.js:2:736

There are two entities and a relationship connecting them, so doesn't appear related to missing statements.

Thanks @radix0000, the example file didn't have any interests listed which wasn't handled. It should now produce a dashed line as per the spec.
Question for @kd-ods: the spec seems to imply no unknown text nodes are ever rendered. Is this correct? The current behaviour is that unknown text is always rendered, except when the interestType is not shareholding or votingRights, so wanted to check before stripping that code out entirely.

the spec seems to imply no unknown text nodes are ever rendered. Is this correct?

Do you mean by 'unknown text nodes': 'unknown' text labels for edges? If so, yes, that is the implication. For the record (and for you to sense-check!), my rationale is:

'unknown' is unhelpful in that it could mean any number of things:

'we don't know if there is a relationship here or not'
'unknown type of interest'
'unknown magnitude of interest'
'unknown directness - are there intermediaries here'

the risk is that in interpreting the visualisation, the viewer picks only one of those meanings, when - actually - all of (ii) - (iv) apply, and (i) does not apply.
the risk exists whether or not the 'unknown' label is there, but I think the label prompts the viewer to read more specificity or meaning into interpretation of the edge visual.
in any case, we now have the wonderful attribute-viewing facility, so that people can select the edge and see for themselves what is and isn't known.

Maybe I'm off the mark, though.


Print content of columns for gtest

What problem does this PR solve?
Issue Number: close #5203
Problem Summary:
What is changed and how it works?
Results can be printed as the following:
printColumns({expected_col, actual_col});

col4: (0: NULL, 1: Int64_-234, 2: Int64_-123, 3: Int64_0, 4: Int64_5, 5: Int64_9999, 6: Int64_24353)
__QB_1___QB_2_col4: (0: NULL, 1: Int64_-234, 2: Int64_-123, 3: Int64_0, 4: Int64_5, 5: Int64_9999, 6: Int64_24353)

printColumns({expected_col, actual_col}, 1, 4);

col4: (1: Int64_-234, 2: Int64_-123, 3: Int64_0, 4: Int64_5)
__QB_1___QB_2_col4: (1: Int64_-234, 2: Int64_-123, 3: Int64_0, 4: Int64_5)

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/cc @ywqzzy  /cc @SeaRise

/run-all-tests

/merge

/run-all-tests


Typo in getting started with prisma migrate

Inside the main function, add the folowing query to read all User records from the database and print the result:

https://www.prisma.io/docs/getting-started/setup-prisma/start-from-scratch-prisma-migrate-typescript-postgres
change: 'folowing' to 'following'

Thanks for pointing this out! 💚


Error when using tunnel agent with proxy, and credentials are not valid

I wrote this code for tunnel agent
this.tunnelingAgent = tunnel.httpsOverHttp({         proxy: {             host: parseProxy.hostname,             port: parseProxy.port,             proxyAuth: parseProxy.auth //incorrect         },         headers: {             'Connection': 'Keep-Alive'         },         rejectUnauthorized: false     });
and this for needle
var options = {  ... agent: this.tunnelingAgent }
If proxy auth credentials are incorrect then error throws on line 374 in the needle.js (core)
request.socket.removeListener('end', on_socket_end);
and write

Cannot read property 'removeListener' of null

If proxy auth credentials are correct then all works fine.
Maybe it need some checks if proxy connection was failed and socket not exist..
Needle v1.0.0

Oh, good point. This means that request.socket does not exist, right? Seems like an easy fix.

How could we check whether the proxy connection failed?
Can you share a snippet of code so I can reproduce the error?

Ping @cloudever

Hi. Sorry, I can't just share it, currently I'm using local fork of your needle lib

There was a fix for the socket listeners a while ago, so I'm closing this one. Feel free to reopen it if you find anything!


Update scamlinks.txt

Added:
discordgift.app
discordsteams.com
steamdlscords.com

It's scams.txt fwiw, not that it matters though since the typo was only in the commit name.

scamlinks is the one i imported from vortex, ordinarily the ones added by us should be in scams. but realistically it doesnt matter much

Yep, I didn't realize you created a new file for that. My mistake. Shouldn't have changed the commit name when merging the PR


tests: Skip EDITOR=cat when not a terminal

Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

This should fix the lxd-test-kernel daily failures. I've tracked it down to our EDITOR=cat logic failing when stdin is /dev/null.

@stgraber why are the lxd-test-kernel tests different (in terms of  stdin being null) compared to the normal pull request test runs?

Normal pull request is run from the Jenkins agent which provides a pts device. The daily tests are run through MAAS and SSH and seem to have a closed/null stdin.


Haven't received my MYX coin

HI.I have sent 3500 MYX coins to my mac-os wallet but still havent received it and 8 confirmations are already done,I'm starting to get a bit nervous about that, address was correct,i'm sure.
Transaction id :     eeedfb0eadd3b136aa3990cf0e916a03f640826915ae6c37f82f7593825019cb
Wallet has 0 balance still

THANKS
HELP,PLS

3500 ISNT A LARGE AMOUNT,why still no coins

my username: aid00123    if needed

@aid0123 it shows that your wallet is not synced. Did you download the latest wallet version?
Can you post here the content of ~/Library/Application Support/Monyx/debug.log file?

pls,explain how to do that

I dont actually remember what version of wallet I have downloaded .Does it mean that ive lost funds?I just downloaded a first wallet I so maybe 7 days ago.Am I stupid or not

https://docs.google.com/document/d/1kQf4rc-WYmAvKGfINmUVSNWPe0oI3X3qP7a1VfcURgM/edit?usp=sharing

please solve my issue

PLEASE

@aid0123 try to delete everything EXCEPT the 'wallet.dat' file in ~/Library/Application Support/Monyx folder, download the new wallet and run it again?

Ok,i will try

If this is going to be solved u gonna be named the best lending platform ,at least by me.I will spread how kind and reliable your support is,thnx)

@aid0123 can't promise anything, but I suppose the private keys must be still in the wallet file, so theoretically you should have access to the address once new wallet is synced.

Can I get a refund if goes wrong.Can't afford to lose this...I went all in believing in you..

NEXT time i;m gonna be 10 times more accurate

maan, I swear currently u are the best lending platform, at least because of perfect support.REALLY nice work, continue it, deserves attention of other people.Thnx,I will try to spread it

Big respect,thumbs up


movie-list CLI: can't delete a movie if the title ends with something in parenthesis

Expected behaviour:
Be able to copy the movie title from the output of flexget movie-list list [LIST_NAME] and remove it from the list with the command flexget movie-list del [list name] <movie title>.
Actual behaviour:
If the movie title ends with something in parenthesis, e.g. The Human Centipede III (Final Sequence) (please don't ask how this movie ended up on my list), the movie-list plugin is not able to find it in the list. It seems that it treats the part in parenthesis as the year of the movie.
Steps to reproduce:

Add a movie with parentheses in the title to a list, e.g. flexget movie-list add [LIST_NAME] "The Human Centipede III (Final Sequence)" -i imdb_id=tt1883367
Output the movie list: flexget movie-list list [LIST_NAME]
Copy the movie title: The Human Centipede III (Final Sequence)
Remove it from the list: flexget movie-list del [LIST_NAME] "The Human Centipede III (Final Sequence)"
Get an error: "Could not find movie with title The Human Centipede III (Final Sequence) in list [LIST_NAME]"

Config:
Irrelevant.
Log:
No entries in the log that seem relevant.
Additional information:

Flexget Version: 2.8.4
Python Version: 2.7.9
Installation method: pip
OS and version: OSMC: Open Source Media Center

Workaround
Add the year manually:
flexget movie-list del [LIST_NAME] "The Human Centipede III (Final Sequence) (2015)"
Removing movie The Human Centipede III (Final Sequence) (2015) from list [LIST_NAME]
It would also be nice if there was a way of removing a movie by one of the IDs, e.g. flexget movie-list del [LIST_NAME] -i imdb_id=tt1883367.

I added the ability to delete by movie ID with https://github.com/Flexget/Flexget/commit/8bce8453e1abdf7dd7400a87cf11297759ac5068
It's not a full fix, but will do until I change the title matching logic (it has some effects on movie list in general, need to test it thoroughly)

I think this can be closed actually.


fix: modify how RegExp matchers are identified

This is an edge case I found while playing with https://github.com/qawolf/qawolf and https://github.com/hoverinc/playwright-testing-library.
In my particular case, I was adding playwright-testing-library queries to the repl context.
import * as pwt from 'playwright-testing-library';

// ....
export const repl = (context?: Record<string, unknown>) =>
  qawolf.repl({
    ...pwt.queries,
  });

That would cause RegExp created inside repl to no longer be identified as such here.

I modified the implementation to verify the constructor as this seems to be enough.
I added a test to reproduce the issue with repl.

Hey @halissonvit, thanks for the PR! Sorry it took me so long to respond. This makes sense to me, however do you think you could update the tests to be more generic and remove the dependency on repl?

Also, the build should be fixed if you just rebase as I recently merged support for Playwright 1.4.x.

Feel free to reopen this but I'm going to close it unless I run into a more general need for the RegExp serialization change.


compat: Allow Flux 0.13

This allows flux v0.13.4 to be used.  Getting the very latest would seem to need a newer version of Functors.

This seems fine. Thanks! I will take a look at the problem with Functors 0.3


Disable pin tooltip

Hi,
I'd like to request a setting to disable the 'pin' tooltip that you see when hovering over c# code. I personally find this pretty annoying and would really prefer not to see it.

Additionally, I have no idea what this actually does. I merely sets a close/X button at the end of the line but does nothing else, what is the purpose of it?

it's supposed to be where type/variable info appears. (This example is from TypeScript)

Ah ok, thanks. I think it would be a useful improvement then to only show the dialog when it has content.


Update README.md

typo fix

failures are because of the modflow-devtools dependency. it will be fixed in another PR.

I'm adding this change to #257


Map integration

Value Proposition
As a user, I want to have a map in order to know where I am and where I can go.
Description

Acception Criteria

[ ] display map
[ ] display LocationMarker of current location

Tasks

[ ] install necessary packages
[ ] implement leaflet
[ ] implement LocationMarker of current location
[ ] implement navigation between nestingbox-list and map

Hi Gesa,
I would rewrite the VP a little bit (e.g.): As a user, I want to see on a map where I am so that I can navigate to the nestingBoxes around me.
Further the "navigation" task could be quite complicated (as Thomas said) because you have no route-api in leaflet. Maybe it is smart to leave that point for now?

Thank you @PhilippWeiss85,
I updated the VP, but I left out the nesting boxes as they are not shown on the map yet.
Thank you for the hint with the navigation! I have left this task out.


Too many bugs in composition mode

Do you want to request a feature or report a bug?
bug
What is the current behavior?

Input some chars use IME, then command + z to undo the chars, it will not work
Input two line text, select them all, use IME to input some chars, the second line would not be removed

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem. You can use this jsfiddle to get started: https://jsfiddle.net/gmertk/e61z7nfa/.
Just go to the official site to try: https://draftjs.org/
What is the expected behavior?

undo should work when input use IME
use IME should replace all selected lines

Which versions of Draft.js, and which browser / OS are affected by this issue? Did this work in previous versions of Draft.js?
version >= 0.11.0 all have these problems, I notice that you have changed the composition handler, that is the cause of these bugs.
like this: https://github.com/facebook/draft-js/blob/master/src/component/handlers/composition/DraftEditorCompositionHandler.js#L256, the author must be
drunk when he write the code, EditorState.push has push the same contentState, this line of code just useless.

Testing this briefly, it looks like it’s the first change to the editor that can’t be un-done:

I couldn’t reproduce the issue with replacing text, do you have more detailed steps @szrenwei?

@thibaudcolas

In fact, all your input use IME can't be undo, you can undo because you start a new block. ( the content add by IME just not been add to undo stack )

As blew, select all text and try to replace:

Thank you, that seems pretty clear-cut. Trying this again, I also managed to get the editor to crash (and the whole page, I imagine the demo doesn’t have any error handling).

see this https://github.com/pomelovico/keep/issues/30

see this pomelovico/keep#30

How/where would I implement this to get it to work? I'm still fairly new/learning but I feel like this might be just what I need!

see this pomelovico/keep#30

How/where would I implement this to get it to work? I'm still fairly new/learning but I feel like this might be just what I need!

see this pomelovico/keep#30

How/where would I implement this to get it to work? I'm still fairly new/learning but I feel like this might be just what I need!

You just need to listen to comositionstart event when your editor mounted (like componentDidMount)

see this pomelovico/keep#30

How/where would I implement this to get it to work? I'm still fairly new/learning but I feel like this might be just what I need!

You just need to listen to comositionstart event when your editor mounted (like componentDidMount)

see this pomelovico/keep#30

How/where would I implement this to get it to work? I'm still fairly new/learning but I feel like this might be just what I need!

You just need to listen to comositionstart event when your editor mounted (like componentDidMount)

Thank you!

see this pomelovico/keep#30

How/where would I implement this to get it to work? I'm still fairly new/learning but I feel like this might be just what I need!

You just need to listen to comositionstart event when your editor mounted (like componentDidMount)

Thank you!

#3177


1.18.1 Error when loading [Bug Report]

Issue
When server starting we get some errors and we can't open any crate.
Versions (please complete the following information):

Minecraft Server Version: Paper 1.18.1
Plugin Version: Dev build 117

Pastebin any errors in console
Link: https://pastebin.com/

Will be patched

Update to the latest 1.18 build on jenkins https://jenkins.badbones69.com/job/Crazy-Crates-Dev and let me know


Cannot use an Enum as part of a composed ID.

Hello,
I want to build a relation table containing three foreign keys ID's and another column containing an ENUM. As I need each row to be unique, I added a @@id containing all three ID's and the ENUM column.
When I try to start my app after generating the binaries, it gives me the following error:

After looking a bit into it, I found out that types.py contains a syntax error:

Is this behaviour expected? Can I use all four information as ID's?
Sincerely. Thanks,

Is there any way to not use the ENUM as part of the ID and make the row unique while this fix is not released?

@danielweil I've created a fix for this issue, can you please try it out and let me know if it fixes your issue?
pip install -U git+https://github.com/RobertCraigie/prisma-client-py@fix/190

Sure!

It worked perfectly. Can you make a new merge with the transaction branch for a wip for me?

Yeah of course, I've just updated the wip/transactions branch.
Thanks for the bug report!

Excelent work!


#2607 Tag/Catalog search suggestions and autocomplete

Description
Tag

P.H: Search by tag name
Popular tag recommendation list (top 10 tags based on number of times registered in metadata)
Tag search autocomplete

Catalog

P.H: Search by catalog name
Popular catalog recommendation list (top 10 catalogs based on number of times registered in metadata)
Catalog search autocomplete

Related Issue : #2607

How Has This Been Tested?

Make sure that the popularity item is displayed when the input bar is in focus.
When you enter a search term, check that the item corresponding to the search term is displayed.

Need additional checks?
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

run build
deploy to 2

@eltriny

Different layer design for tags and catalogs

If there is no search result, mark as "No result".

run build
deploy to 2


Z-ordering on Android not working as expected

Bug Report
Current behavior
Z-index does not fully work on Android, its either on top or on bottom but overlapping multiple views won't work
Steps to reproduce

Create Publisher with z-index of -1
Create Subscriber that is overlapping the Publisher with a z-index of -2
Steps 1 and 2 with z-index reversed
See the difference

What is the current bug behavior?
The plugin uses(by default)  SurfaceViews, this particular View does not work as any normal View(More info). Because of that the setZ or setTranslationZ won't do much expect putting it in front or behind the WebView.
What is the expected correct behavior?
That the given z-index puts the right view "in front".
To do so we could use TextureViews(tokbox reference). I am not entirely sure if there is a reason for not using TextureViews, maybe @msach22 knows something about that.
Both TextureViews and SurfaceViews have their pros and cons, but in general you should try to keep the total count of SurfaceViews down.
I have writen an implementation that makes use of the TextureView and  Matrix transforms. The implementation makes use of a custom ViewGroup that will be used instead of the TextureView(mPublisher.getView(), mSubscriber.getView()). The ViewGroup  adds the TextureView as a child and overides the dispatchDraw method. We then position and scale the TextureView using Matrix transforms and in the dispatchDraw  we use Path to clip the TextureView within the given bounding rectangle.
IMPORTANT
This implementation has been written and used in combination with PR49 and PR59.
I can make a PR with this implementation if required, I do belief that this is a needed feature if people want to make properly use of z-ordering on Android.

@wolfenrain Can you add a PR without the usage of PR59

@wolfenrain Are you still working on the PR without #59 ?

If it helps anyone this fork solved the issue for me:
https://github.com/talkspiritlab/cordova-plugin-opentok

@msach22 sorry for the late response, I havent worked on it yet, but I can make some time to work on it if needed

@wolfenrain No worries, do you think you will have the capacity to create a PR?

I also have the same problem
trying to add html button with absolute position on top of the video and I can't see the element

This is a problem also in IOS
Please update the issue title

@Shtibel -
I have had a partial success in android by making doing the following:

Set #publisher css with z-index of -1
Set #subscriber css with z-index of -2
Set background:transparent for ion-app
After publishing remove any content inside the publisher container
     this.session.publish(this.publisher);      if (OT.updateViews) {        OT.updateViews();        document.getElementById("publisher").innerHTML = "";      }
Do the same on streamCreated also
this.session.on({    streamCreated: event => {      this.subscribed = true;      this.session.subscribe(event.stream, "subscriber", {        insertMode: "append",        width: "100%",        height: "100%",        style: {          buttonDisplayMode: "on",          audioLevelDisplayMode: "on"        }      });      if (OT.updateViews) {        OT.updateViews();        document.getElementById("subscriber").innerHTML = "";      }    })
Have had success with this on Android 7, but not Android 8

You may also want to look at this file and see if you can do some changes there.
cordova-plugin-opentok/src/android/OpenTokAndroidPlugin.java
Try looking at the lines where addView and setTranslationZ functions are used.

Is this still going on?

It might help https://github.com/damianrejman/cordova-plugin-opentok/pull/1/commits/db1692b0e9859f8d3d5f95d51a05d2fe77fb1bc1


Empty menu cache when deleting blogconfig instances

Delete django / menu cache when a blogconfig is deleted

Coverage increased (+0.003%) to 95.841% when pulling f9a0ae277e5cca2fb61b3eb31645071c8c56ee1c on hotfix/issue-495-clear-menu-cache into 77da018b2f5dc8f81b301fb770b1b4c80d9b1937 on release/0.9.x.


List routes kind URLs in odo list

/kind bug
Once https://github.com/openshift/odo/pull/2885 is in, we should list route based URLs in odo list.
blocked on https://github.com/openshift/odo/pull/2885

Are there any plans to handle this in sprint 184?

/assign

Blocked by: #3160
/status blocked


Hello There ! What’s new ?

Hi,
Glad to see an official project from gce around for this topics.
Any news to share ?
Thanks,

Hello,
Currently working slowly on it, need help if it's possible :slightly_smiling_face:

Hi,
Sorry i’m not that good enough :) . Godspeed to you ! 🔥


Urobiome package term suggestion menstrual_status

New term details
Term name - menstrual status
Structured comment name - menstrual_status
Definition - If applicable (studies on female urobiome), indicate either: Pre-menarche (optional, specify Tanner stage 1-5), Menstruating,  Pregnant, Postpartum (up to 1 year after delivery) ,Post-menopausal , Unknown,
Expected value - hormone status
Value syntax - {text};{termLabel} {[termID]}
Example - Post-memopausal
Preferred unit -
Package - Urobiome

Additional context
Suggested as part of the Urobiome package.

Menopause is a similar existing term that is "the date of onset of menopause", which is very difficult to specify. Menstrual status would be a more general term.
Make the definition more general (studies of females, rather than studies of female urbiome). Move the CV out of the current definition and into a list field.
@lakarstens will work on the definition wording.

Proposed new definition: Menstrual or estrous status of host or organism, if applicable (e.g studies of female sex).
Still identifying appropriate controlled vocabulary.


Modal scrolling on mobile devices - what is the current state?

Fiddle: http://jsfiddle.net/stefek99/t6oq2gze/
It was also here:

https://github.com/twbs/bootstrap/issues/7501
https://github.com/twbs/bootstrap/issues/3361

Some StackOverflow too:

http://stackoverflow.com/questions/19060301/how-to-prevent-background-scrolling-when-bootstrap-3-modal-open-on-mobile-browse
http://stackoverflow.com/questions/9538868/prevent-body-from-scrolling-when-a-modal-is-opened

I think it's a common issue and should be fixed, my solution (after doing some research):
body.modal-open {
    position: fixed;
}

Hi @stefek99!
You appear to have posted a live example (http://fiddle.jshell.net/stefek99/t6oq2gze/show/light/), which is always a good first step. However, according to the HTML5 validator, your example has some validation errors, which might potentially be causing your issue:

line 75, column 7: Stray end tag body.

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

We explicitly mention this in the documentation:

Modals, navbars, and virtual keyboards
Overflow and scrolling
Support for overflow: hidden on the <body> element is quite limited in iOS and Android. To that end, when you scroll past the top or bottom of a modal in either of those devices' browsers, the <body> content will begin to scroll.

my solution (after doing some research):
body.modal-open { position: fixed; }

Note that this has the side-effect of "scrolling" to the top of the page while the modal is open.

Closing then.

Hello Bootstrap team,
As mentioned in the issue #7501, this bug should be fixed in the version 3.
I'm currently using version 3.3.6 and unfortunately the issue can be still be reproduced, similar to the animated picture above. Also, if you use an iPad with iOS 8 by browsing this page https://nakupanda.github.io/bootstrap3-dialog/, you'll be able to reproduce when opening the modal samples showed in the page. Please note that the samples in the page are using Bootstraop 3.3.5.
Despite that this effect is mentioned in the documentation, I wonder if there will be chances to fix the issue since it's really annoying when the modal has its own scrolling.
Thanks so much in advance for any feedback you can provide.

Why was this closed?

@vanflymen It's a bug in the browsers themselves. We can't do anything about it on our end. It's already mentioned in our documentation as a known issue.

If you want to follow its progress, the browser bug is https://bugs.webkit.org/show_bug.cgi?id=153852

Just run into this bug, Here is my fix:
$('body').on('touchmove', function(e){
 if($('.scroll-disable').has($(e.target)).length) e.preventDefault();
});
$('body').on('shown.bs.modal', function(){
 $(this).addClass('scroll-disable');
});
$('body').on('hidden.bs.modal', function(){
 $(this).removeClass('scroll-disable');
});


wrong version of mtls-ambassador in staging

fails the test, hence it's not deploying.

it will pass, but it really should be 3.2 right?

Point is, hosted is not deploying cause of this

Approving, but as mentioned on slack I believe the version mender-staging is wrong and we need to fix it.

agreed. Just want to deploy the site.


Is a way to get manifest which are no more related to a tags via API

That's my need, To prepare GC, we need to delete manifest which are no more related to a tag.
I've searched in API but I cannot found a way to get this list of manifests.
So my solution is to get the list from the registry filesystem.
So I look for a way by API to get these data. Could you create an extension API ?

Is there anyone working on this feature?

I hacked a script together that deletes untagged manifests. I need some help to make it work better as my knowledge of the internals of the registry is very limited. In it's current form is seems to work ok but it isn't perfect. Hope it helps!
https://github.com/Richie765/docker-tools

I have a script purge_registry.sh that doesn't use the API but works on the whole registry directory structure instead.   The only thing missing is deleting unreferenced blobs.  I plan to work on this feature before I implement this script in Python instead.

Thanks, that looks great. I also found this one:
https://gist.github.com/mortensteenrasmussen/512f0566dbc3ef1cc4a2c47dd9cdb973


Expand the watch area

Implement changes to ignore Springfield.
Brief description of changes
Write a simple description of what changes you've made to contribute to this project

Expand the watch area
Implement changes to ignore Springfield.

Testing
What did you do to test your changes? can you post screenshots?
Updated watch area

SGF Ignored area

A local test will be performed with the 00Z launch before merging will be allowed.
Pull Request Checklist
These tasks must be completed before a merge will be done

[x] CI Tests pass.
[ ] Manual testing evidence provided.
[ ] Program runs when a launch is detected (if core is modified)

yarn run v1.22.19
$ node src/core.js
[Mongo] Connected.
[MQTT] Connected
[Discord] Connected.
[Balloon] Sonde DFM 21046373 from SGF is being ignored!
[Balloon] Sonde DFM 21046373 from SGF is being ignored!
[Balloon] Sonde DFM 21046373 from SGF is being ignored!
[Balloon] Sonde DFM 21046373 from SGF is being ignored!
[Balloon] Sonde DFM 21046373 from SGF is being ignored!
[Balloon] Sonde DFM 21046373 from SGF is being ignored!
[Balloon] Sonde DFM 21046373 from SGF is being ignored!
[Balloon] Sonde DFM 21046373 from SGF is being ignored!
[Balloon] Sonde DFM 21046373 from SGF is being ignored!
[Balloon] Sonde DFM 21046373 from SGF is being ignored!
[Balloon] Sonde DFM 21046373 from SGF is being ignored!
[Balloon] Sonde DFM 21046373 from SGF is being ignored!
[Balloon] Sonde DFM 21046373 from SGF is being ignored!
[Balloon] New sonde detected! Lockheed Martin LMS6-403 8189935
Sonde 8189935 is ready for release

There's a bug with the message being triggered multiple times. This should be addressed before merging.

Been a bit tied up on other projects/work. This will be fixed and ready to use soon.


Phase is not properly updated ....

... in case of multiplayer running on Local.
Deal = {
    name: "deal",
    turn: { moveLimit: 1 },
    setup: () => ({
      deal: deal(),
      bidding: [],
    }),
    phases,
  };

phases = {
  [PHASE_BIDDING]: {
    moves: { bid },
    endIf: (G) => {
      return  G.bidding.lenght === 10;
    },
    start: true,
    next: PHASE_DECLARE,
  },
  [PHASE_DECLARE]: { moves: playCard },
};

When G.bidding.length achieve length 10 then phase is finished ( i see that ctx.phase says 'PHASE_DECLARE' however playCard move is not visible in moves and i see  Uncaught TypeError: moves.playCard is not a function. When i added playCard to the list of allowed moves of PHASE_BIDDING then i see ERROR: disallowed move: playCard ERROR: move not processed - canPlayerMakeMove=false, playerID=[0] Maybe it is because im running multiplayer game witj four client in one browser window, but up to documentation it is allowed.

Thanks for the report. Do you think this could be for the same reason I outlined in my answer to #670?
If not, would you be able to share a CodeSandbox or other reproduction for us to test?

Closing this due to lack of reproduction for now. Feel free to open with more details!


Fixed event delegation

Refactored delegate to fixed a bug which prevents that a delegation on a form element will not trigger events of child inputs. Or in other words... this will work now:

      handleEvents.delegate(document.querySelector('body'), 'keyup', 'form', e => console.log("hello world"));

Did it similar like the delegate lib: https://github.com/zenorocha/delegate/blob/master/src/delegate.js

Changed in package.json the module path to main src file for better IDE (e.g. Webstorm) and build support
jsDoc: flagged useCapture as optional and use the correct syntax for optional parameters
Renamed prototype to polyfill
Added package-lock.json (see https://stackoverflow.com/questions/44206782/do-i-commit-the-package-lock-json-file-created-by-npm-5)

Thank you very much for contributing, I will be checking and approving this PR as soon as posible.

I recognized that you use a yarn lock file... so I think you can delete my package-lock :)

No problem, I tested with both yarn and npm and both files can coexist together.


Add service tag filter support

A small change to add service filtering by tag, supported by consul v1 api.

Cool. Thank you! Will push a new version with this now.

:D

Done! Available in npm and bower


Internal load balancer diagram is needed

Hi, it would be really nice to have am Azure diagram similar to one in the doc, but for an internal setup.
So, you start with the Internet -> Internal Azure Firewall -> Internal load balancer -> VM group, and back again.
Many thanks.
Mark-Allen Perry

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

ID: 07b33cf5-3747-59b6-29b4-9aa076a6d271
Version Independent ID: ddd15c52-275e-68e6-513e-07d42292b059
Content: Integrate Azure Firewall with Azure Standard Load Balancer
Content Source: articles/firewall/integrate-lb.md
Service: firewall
GitHub Login: @vhorne
Microsoft Alias: victorh

Many thanks.
The Azure diagrams on microsoft.com are very helpful and can easily explain what is needed and how it is connected.  In addition, many companies require architecture plans and these can assist a lot of people to produce those.
I thank you again and wish you a great day.
Mark-Allen

Created an workitem in the backlog for this.
#please-close


Array destructuring shorthand syntax wrong fix

Bug Description
The array destructuring shorthand syntax gets fixed as array() instead of list(), which creates syntactically invalid code.
Minimal Code Snippet
Original code
$arr              = [ 'foo' => 'bar' ];
[ 'foo' => $foo ] = $arr;

Code after running phpcbf
$arr                   = array( 'foo' => 'bar' );
array( 'foo' => $foo ) = $arr;

Expected code
$arr                  = array( 'foo' => 'bar' );
list( 'foo' => $foo ) = $arr;

Error Code
It only complains about the bracket syntax Generic.Arrays.DisallowShortArraySyntax.Found.
Environment

Question
Answer

PHP version
7.3.11

PHP_CodeSniffer version
3.5.5

WPCS version
2.3.0

WPCS install type
Composer global

Tested Against develop branch?

[ ] I have verified the issue still exists in the develop branch of WPCS.

Oops... Sorry, I didn't find that issue. :neutral_face:


Optimize where exists sub-queries into aggregate and join

Which issue does this PR close?
Closes #160.
Rationale for this change
In order to evaluate DataFusion as a candidate query engine, users need to be able to run industry standard benchmarks like TPC-H. Query 4 is a good initial candidate, because it is being blocked only by a relatively simple optimization rule to turn exists subqueries into joins.
This PR includes the minimum necessary changes to get Query 4 passing, but I believe this is a generalizable approach that will work for the remaining queries in the TPC-H suite being blocked by subquery-related issues.
I wanted to PR early to start the conversation, but I intend to either submit subsequent PRs generalizing this approach, or extend this PR until we have all the TPC-H subquery cases covered.
What changes are included in this PR?
An optimization rule for decorelating a narrowly defined set of queries. Those not explicitly covered will remain unaltered.
Are there any user-facing changes?
Any single-column join where exists correlated subquery should now work.

Codecov Report

Merging #2813 (858b284) into master (839a618) will decrease coverage by 0.01%.
The diff coverage is 72.68%.

@@            Coverage Diff             @@
##           master    #2813      +/-   ##
==========================================
- Coverage   85.20%   85.19%   -0.02%
==========================================
  Files         274      276       +2
  Lines       48666    48848     +182
==========================================
+ Hits        41468    41616     +148
- Misses       7198     7232      +34

Impacted Files
Coverage Δ

datafusion/common/src/scalar.rs
74.94% <ø> (+0.11%)
:arrow_up:

datafusion/core/tests/sql/mod.rs
93.25% <0.00%> (-4.39%)
:arrow_down:

...tafusion/physical-expr/src/expressions/datetime.rs
59.21% <64.40%> (+26.55%)
:arrow_up:

datafusion/optimizer/src/subquery_decorrelate.rs
82.85% <82.85%> (ø)

datafusion/core/tests/sql/subqueries.rs
88.23% <88.23%> (ø)

datafusion/core/src/execution/context.rs
79.02% <100.00%> (+0.02%)
:arrow_up:

datafusion/core/tests/sql/timestamp.rs
100.00% <100.00%> (ø)

datafusion/optimizer/src/simplify_expressions.rs
82.04% <100.00%> (+0.02%)
:arrow_up:

datafusion/core/src/physical_plan/metrics/value.rs
86.93% <0.00%> (-0.51%)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

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

I double checked with cargo run --release --bin tpch -- benchmark datafusion --iterations 3 --path ./data --format tbl --query 4 --batch-size 4096 and observed that this didn't resolve "the real" query 4, just my stripped down minimal failing test case.
I made some adjustments, and I was able to run query 4 with the presently committed code:
+-----------------+-------------+
| o_orderpriority | order_count |
+-----------------+-------------+
| 1-URGENT        | 10594       |
| 2-HIGH          | 10476       |
| 3-MEDIUM        | 10410       |
| 4-NOT SPECIFIED | 10556       |
| 5-LOW           | 10487       |
+-----------------+-------------+
Query 4 iteration 2 took 43617.9 ms and returned 5 rows
Query 4 avg time: 45785.94 ms

This is slow, but matches my postgres results:
+---------------+-----------+
|o_orderpriority|order_count|
+---------------+-----------+
|1-URGENT       |10594      |
|2-HIGH         |10476      |
|3-MEDIUM       |10410      |
|4-NOT SPECIFIED|10556      |
|5-LOW          |10487      |
+---------------+-----------+

The remaining failing queries seem to fall into two categories:

Ones that fail because we only handle where exists with this PR (not where x < (subquery) expressions)
Ones that fail due to multiple subqueries in the same filter expression (which means we have to run this iteratively or something)

Probably duplicated work with #2421

Closed in favor of #2885


Fixes documentation for radiobutton

Fixes
<label for="radiodemo" class="checkable">Or me</span>

which is not correct.

Not sure whatsup with that newline in the diff :/


setup SCTM modular sklearn pipeline

[x] Add factory functionality for STCM, pack it into pipeline
[x] Cover factory functionality for STCM by test cases
[x] Pack SCTM node into sklearn pipeline
[x] Cover SCTM sklearn pipeline by test cases

Resolved under #25


Snippets BroadcastEmitter optimization

Details:

item1
...

Tickets:

ticket-id

@gdmitt, rebase was done. Please, merge


MySql: included package tag for java, added java readme

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Can one of the admins verify this patch?

Automation for azure-sdk-for-go

Automation for azure-sdk-for-java


Setting Fullscreen

Hi, is there a way to set the player to fullscreen? I know you can call .SetFullscreen(1) from the Unity Instance. Is there a way to do that here? Thanks.

I actually just created this. I'm about to create a pull request.

Awesome, thanks!


add replicasets and daemonsets to cohabitating resources

Signed-off-by: Steve Kriss<EMAIL_ADDRESS>ReplicaSets and DaemonSets also moved from extensions to apps. Add them to cohabitating resources so they don't get backed up twice.

LGTM


ServiceBus won't deadletter messages when processing raises generic exceptions

Liquid will deadletter the message only when:

The method returns a Task
ReceiveModeisPeekLock (deleteAfterRead = false`).
The exception thrown has a Inner Exception (a first level exception won't do).
The inner exception is InvalidInputException or a BusinessValidationException

as can be seen below:
https://github.com/Avanade/Liquid-Application-Framework/blob/00dd8d3af8dc5437ffd6f8a94e95be3c94193efa/src/Liquid.OnAzure/MessageBuses/ServiceBus.cs#L272-L299
This means that on most cases, the message isn't deadlettered. I still need to investigate the default behavior of Service Bus in  this case.

So, as can be confirmed on docs,

Queues and subscriptions each have a QueueDescription.MaxDeliveryCount and SubscriptionDescription.MaxDeliveryCount property respectively; the default value is 10. Whenever a message has been delivered under a lock (ReceiveMode.PeekLock), but has been either explicitly abandoned or the lock has expired, the message BrokeredMessage.DeliveryCount is incremented. When DeliveryCount exceeds MaxDeliveryCount, the message is moved to the DLQ, specifying the MaxDeliveryCountExceeded reason code

So, as expected, the Service Bus will dead letter a message automatically after a few (10, by default) attempts. Therefore, it seems that the reasoning behind the current implementation in Liquid is that, if the error is a InvalidInputException or a BusinessValidationException, then we know that the message won't ever be accepted and should be dead-lettered directly.
Other cases will result in retries until a certain number of attempts.


Add Documentation for Saving all annotations to Server

In Real world case it would be better to set initial annotations property and also save annotation data to server api

OK, I will try to carry out a better documentation after I have finished my final exams.

Hi,
I have read these code today and I think that saving all annotations can be done by:

Implement a StoreAdapter that connects to your server, or
Use LocalStoreAdapter to store annotations first in localstorage, and then save these annotations to server regularly.

And I have to remind that even with some bugs fixed compared with upstream (Submitty) version, the code quality is still not production-ready and it's recommended to use it only for prototype.

Hi , I wish my work can help you.
I have figured out a solution of pdf.js annotation, use only pdf.js dist, and it is very easy to import into your project.
Demo and source code: https://demos.libertynlp.com/#/pdfjs-annotation
Demo video:https://www.youtube.com/watch?v=IYD7WKzx-yc&t=3s
functions:
① highlight : highlight selected text;
② underline : underline selected text;
③ edit: edit annotation ( copy/delete/add comment/navigate );
④ output annotations : output annotations for importing;
⑤ import annotations : import annotations for current file;
⑥ download annotations : download annotations and comments text;
⑦ undo annotations : undo annotations in current page;
⑧ help : open help file;


The state of checkbox's configurations is not saved

Doesn't save previous state, it's reverting to "checked" state each time.

@Spectralgo this is by design.  You can change the state under options.  This is just to allow you to do a one time override of the settings.  On opening the form it will revert back to what you save under settings.


Optimize with webpack (Extension causes high cpu load)

Issue Type: Performance
Extension Name: azurerm-vscode-tools
Extension Version: 0.4.2
OS Version: Darwin x64 17.7.0
VSCode version: 1.30.0

:warning: Make sure to attach this file from your home-directory: /Users/skotkar/msazurermtools.azurerm-vscode-tools-unresponsive.cpuprofile.txt :warning:
Find more details here: https://github.com/Microsoft/vscode/wiki/Explain:-extension-causes-high-cpu-load

Fixed by #164


Need to implement authentication

Need to implement authentication @prathameshkhade

@sidhantnaik , Added Authentication via Pull Request  #2


make global environment variables available for pipeline substitution

This makes global environment variables (as specified by WOODPECKER_ENVIRONMENT) available for pipeline substitution.
Notably, it prepends GLOBAL_ to each to ensure existing variables are not overwritten.
I should probably add a commit that documents this behavior (I'll do that soon).
The variables would still be available without a prefix within the pipeline task.
The use case I created this for is for specifying things like images, common to many steps, that should be updated together, or specifying a common registry that many pipelines will push to. Things that aren't really "secrets", but don't belong statically in the pipeline yaml.
As in:
pipeline:
  build-and-publish:
    image: woodpeckerci/plugin-docker-buildx
    group: docker
    settings:
      dockerfile: Dockerfile
      repo: ${GLOBAL_REGISTRY}/${GLOBAL_USER}/woodpecker-plugin

TODO:

[x] support cli exec --env
[x] tests
[x] documentation

yes some tests would be nice

The use case I created this for is for specifying things like images, common to many steps, that should be updated together, or specifying a common registry that many pipelines will push to. Things that aren't really "secrets", but don't belong statically in the pipeline yaml.

I was thinking about some similar use-case as well and had the idea to at some point rename secrets to variables and allow the user to mask them (in case of a secret).

I think this should also apply to cli exec

The use case I created this for is for specifying things like images, common to many steps, that should be updated together, or specifying a common registry that many pipelines will push to. Things that aren't really "secrets", but don't belong statically in the pipeline yaml.

I was thinking about some similar use-case as well and had the idea to at some point rename secrets to variables and allow the user to mask them (in case of a secret).

That'd be useful, as long as there's a capability for viewing and changing existing values and there's an option for them to be available across projects. I think I recall another ticket for that… #362

I think this should also apply to cli exec

Gotcha, so anything passed to the --env flag? I'll take a look.

I added cli exec support and a procBuilder test.
I still need a cli test, documentation, and consensus on the naming behavior regarding the GLOBAL_ prefix. And I suppose a consensus on whether variables should be allowed to overwrite existing variables. Allowing this would include variables like CI_COMMIT_SHA, so I'm weary of that idea.
I think I lean towards dropping the prefix and not allowing overwriting.

The use case I created this for is for specifying things like images, common to many steps, that should be updated together, or specifying a common registry that many pipelines will push to. Things that aren't really "secrets", but don't belong statically in the pipeline yaml.

I was thinking about some similar use-case as well and had the idea to at some point rename secrets to variables and allow the user to mask them (in case of a secret).

well I guess we can introduce GLOBAL_ and set vars should then have no prefix and do overwrite stuff ...
for the ui part I'm fine with re-using the secrets stuff, but in backend we should difference strongly between them, for security reasons

OK, so this drops the GLOBAL_ prefix, but still avoids overwriting existing variables. Another reason to avoid the prefix is that running from cli exec meant the variable was prefixed for the yaml substitution, but not prefixed inside the container. Leaving off the prefix, but not overwriting, seems like a safe alternative without having to understand multiple variable names in different contexts.
I added documentation and examples (let me know if anything is missing, unclear, or needs to be expanded).
I have a procBuilder test, but I'm struggling to put together something for cli/exec. Specifically, I think I'd like the test entry point to be run(…) and I'd ideally mock the input file, but there are multiple places that read from that input file. Even entering at the method that I modified (execWithAxis(…)) would require reading from the file. I did find some tips on mocking a file system, but that would require making additional changes to how the file is read in the exec methods. Any thoughts on a better way to create a test?

Just wanted to check in and see if there's anything else requested for this? My only outstanding thought is the cli test, which I think would require changes to the operational parts of the cli code. I'd actually be more inclined to put that in a separate PR if it's desired.
Anyway, let me know if anything is missing or needs work.

I dont understand why you droped GLOBAL_

I dont understand why you droped GLOBAL_

As I thought more about the prefix, it seemed to just be confusing.
You add a variable name for the environment, but then to use it in the pipeline configuration you have to add a prefix to it. You can also use the variable inside the pipeline container, but there it doesn't have a prefix.
The only thing positive about the prefix is that it identifies the source of that variable, but only in the pipeline.
I suppose the alternative would be to add the prefix when the variables are injected to the container environment as well, but that'd be a breaking change to existing behavior. I figured it'd be safer to modify the behavior I was introducing, rather than the current state.
If you disagree I can modify the PR, including updating the documentation.
Options:

Leave as is with no prefix
Restore prefix when processing the pipeline. Leave the unprefixed version within the container.
Restore the prefix and update the container environment to also include a prefix.
Restore the prefix and update the container to include both a prefixed and unprefixed version. And mark the unprefixed version as deprecated.

I personally prefer the first option for simplicity, but I understand there might be other considerations or future interactions to consider.

+1 for the current option.

Just checking in regarding if there's any further requests on this PR, or if I can "ignore it" until you're ready to merge. Thanks.


Cryptocurrency prices

Would be awesome if it could do cryptocurrency prices out to more than 2 decimals instead of rounding up to $.01 or just showing at $.00. Thanks.

This might be caused by the fact that I have to use a hack with the scrolling library I use. what’s the price you see vs the price you expect to see?

Thanks for quick reply.  Some examples, LYL displays as $0.01, when actual value is $.006288. So its calculating it at $.01 which is way overestimating the value.  Some other examples, CV shows as $0.00, when should be $0.002488, which is causing it to show as estimate value of $0.  So really its only the cryptos that are under $.01/share that are most affected it seems.

Oof.. yeah, that's a pretty big bug! The fix should be simple, I just need to calculate using the actual value rather than the truncated display value.
I'll try to get the fix out soon. Thanks for pointing it out.

No worries.  I love your website, very handy.  I'm not sure what is happening, but sometimes I have to re-enter all my holdings.  I'm assuming it's some type of browser history clear that does it.  But if there was a way to manually save a "copy" of your holdings in a file and then reload that would be cool.  Probably a lot of work, but something to think about.

@neuralgia https://github.com/pastudan/altfolio-live/commit/ecad4569c062f83e934b6fe9fc5fe2ef50dd9da4 fixes the price calculation issue. The fix has been deployed so you should see it live on the site.

As for the holdings, clearing your browser history certainly does clear your holdings, since those are stored in localStorage. To avoid that issue, I'd need to build a login system and store all your holdings server-side, which I'm not ready to do. If someone builds that functionality though, I'd be happy to merge it in.


RAM code and COPYSECTION

This is a proposed solution to #272, but it's different enough that I'm making a separate issue.
I've implemented something that makes RAM code easy before, for swapdump.
Basically, I implemented two new rgbds features:

A -r option to rgbasm which lets you put code in RAM (https://github.com/Sanqui/rgbds/commit/4f7b98fd383064fe749b92321c62ac7f0cb098d8)
A COPYSECTION directive which creates a binary copy of an already-written section (https://github.com/Sanqui/rgbds/commit/eac57476c1b5acf54cfb05c138412e4e72058825)

Usage then follows:
SECTION "ram code", WRAM0[$C000]

RAMCode:
    jp RAMCode

This will not emit any bytes, but the section will be compiled, the space will be reserved and the labels will be calculated (resulting in a proper jp $C000)
COPYSECTION "ram code", ROMX[$4000], BANK[$1]

This will take the section output as "ram code" and create a new section with exactly the same data.  Since the WRAM section would never have been output, this is the way to actually place the generated opcodes in ROM, to be copied to RAM later.
Now, I don't know if this approach is sensible and I haven't tested it extensively, but should you like it, I can rebase and make a pull request.

I think it's sensible, but let's wait if there are other opinions.

The ability to have a "data" section with the run/load addresses is something I've been missing from cc65 as well.

Is it ready to rebase?


Enhancements to redacted mode

Make redacted mode apply to all personally identifying things in iMessage profile specially in the starts chat using section
Change wording of alias, to say something like redacted phone number or redacted email instead of just alias
Make redacted mode also apply to my Mac / my device section (side note maybe change it to say my device instead of my Mac in unredacted mode)

@TaeHagen fufilled in https://github.com/TaeHagen/bluebubbles-app/commit/78f70b1bbb53289b46cfe6b74014fc940f61d005 right

If show chat name as placeholder is enabled, then this shows in redacted mode at least for group chats
I suggest this option be turned off whenever redacted mode is toggled


Typo in the example code, file: ArduinoENC28J60

Typo in file: ArduinoENC28J60, line 9. No quotation marks. Looks like this:
ThingerENC28J60 thing(USERNAME, DEVICE_ID, DEVICE_CREDENTIAL);
instead of this:
ThingerENC28J60 thing("USERNAME", "DEVICE_ID", "DEVICE_CREDENTIAL");
It is in the documentation as well.

Hi! looking to the code, the example seems to be correct. Notice that the USERNAME, DEVICE_ID, and DEVICE_CREDENTIAL is defined in the previous lines, so the example should be able to compile.
Bests.


Configure proxy during init

RancherOS Version: 1.1.0
Where are you running RancherOS: VMware
We currently use packer to create a vmware image for rancheros 1.0.4. Packer performs this command to enable the vmware-tools:
ros service enable open-vm-tools
This command is run in a shell which contains the correct http proxy environment.
In Rancheros 1.1.0, the open-vm-tools service is automatically enabled during boot. However, this fails since the proxy configuration is not available.
How can I configure the proxy during the boot process?

I found out how to configure this in cloud-config.yml. This issue can be closed.


Best process to rename several Grav project GitHub repos?

I want to rename several of my Grav projects to better align them (theme and skeleton), and was wondering if there are any best practices to do so? If I recall correctly, when a project GitHub repo is renamed it can temporarily cause issues with the GPM. Thanks very much!

just create a ticket and give me the breakdown of the old and new names.  I can coordinate with you on slack to ensure we get the timing right.
This is one of those scenarios that will totally be handled by our new GPM logic that we're working on.

Will do, thanks very much Andy... and 👍👍👍 on that new GPM logic you are working on!


Feat serverless response stream

Information

Type
Breaking change

Feature
No

Add streaming response support for AWS Lambda. Improve also the documentation related to Platform Serverless.
Todos

[x] Tests
[x] Coverage
[x] Example
[x] Documentation

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

v7.78.0
GitHub release

Your semantic-release bot :package::rocket:


Adding Ryujinx

in steam buddy can you add ryujinx ?
it's a switch emulator which is very user friendly

To add it to Steam Buddy, the emulator would need to support plug and play controller setup. Do you know if it does?
yuzu is already included in GamerOS, but is not hooked up to Steam Buddy. You have to set it up manually and create your own shortcuts.


ARCSI added and processing section reordered

ARCSI atmospheric correction software added; "Processing" section reordered; EO-me description extended

Thank you for the PR @neteler and please excuse my late reply. Merged into the list.


Patching CBMC task proofs.

Description
Continuing the xPendedTicks fix following this https://github.com/aws/amazon-freertos/pull/1730.
Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This PR is raised solely to test out the fix, as it runs faster in your infra. If you see CBMC failures, please don't panic. That's exactly what I plan to fix.

/bot run checks

/bot run checks

This PR is no longer valid. https://github.com/aws/amazon-freertos/pull/1758


[ExcludeFromCodeCoverage] attribute does not work on methods

Hi,
I've been struggling with adding [ExcludeFromCodeCoverage]  attribute . For some reason, coverlet ignores it when I decorate any method for exclusion. However, it seems to be working fine when I add it to a class. I've created a project on github that reproduces this issue.
https://github.com/ssztangierski/coverlet-exclude-attribute
command when I run test from "WeatherControllerTests" folder:
dotnet test WeatherController.Tests.csproj /p:CollectCoverage=true /p:CoverletOutputFormat=cobertura
I'd like to exclude CreateHostBuilder method from Program.cs
Is this intended or am I doing something wrong? There is also   coverage.cobertura.xml file in the repo.
  <class name="CoverageTest.Program/<>c" filename="Program.cs" line-rate="0" branch-rate="1" complexity="1">
          <methods>
            <method name="<CreateHostBuilder>b__1_0" signature="(Microsoft.AspNetCore.Hosting.IWebHostBuilder)" line-rate="0" branch-rate="1">
              <lines>
                <line number="25" hits="0" branch="False" />
                <line number="26" hits="0" branch="False" />
                <line number="27" hits="0" branch="False" />
              </lines>
            </method>
          </methods>
          <lines>
            <line number="25" hits="0" branch="False" />
            <line number="26" hits="0" branch="False" />
            <line number="27" hits="0" branch="False" />
          </lines>
        </class>

Hi @ssztangierski can you try with nightly build?We fixed some similar bug recently and maybe we've the soluton ready on master https://github.com/tonerdo/coverlet/blob/master/Documentation/ConsumeNightlyBuild.md

cc: @matteoerigozzi

Hi @MarcoRossignoli, it does work like it should with nightly build. Could you please tell me when can I expect nightlybuild to be officially released?

1 April more or less https://github.com/tonerdo/coverlet/blob/master/Documentation/ReleasePlan.md

Feel free to close this issue.

Thanks a lot!

FYI current changelog https://github.com/tonerdo/coverlet/blob/master/Documentation/Changelog.md


Galactic Scale

Found an interesting issue - apparently moons can spawn too close to a gas giant to land on them.  The game loads the transition to the gas giant instead of the moon you're trying to get to, making it impossible to land on the moon.

Here's a copy of the save.
wierdness.zip

list ' o mods

Thanks for that I'm hoping to get some time to sort this today

Fixed in <IP_ADDRESS>


Can't start bipython

After installing all the components, when typing bipython into the terminal I get:
Traceback (most recent call last):
  File "/bin/bipython", line 9, in <module>
    load_entry_point('bipython==0.1.2', 'console_scripts', 'bipython')()
  File "/usr/lib/python2.7/site-packages/bipython/__init__.py", line 1769, in main
    myrepl = URWIDRepl(event_loop, palette, interpreter, config)
  File "/usr/lib/python2.7/site-packages/bipython/__init__.py", line 651, in __init__
    self.ipython = self.connect_ipython_kernel()
  File "/usr/lib/python2.7/site-packages/bipython/__init__.py", line 708, in connect_ipython_kernel
    self.echod(":IPython " + s + " failed", "Info")
TypeError: echod() takes exactly 2 arguments (3 given)

any idea?

I am having the same error having installed bipython through pip.
Actually I was seeing a different error yet before having installed a few more packages that also prevented me from starting the new ipython notebook so it may be a dependency issue ...
Now I am on this error too and still curious to try bipython.
greetings!

Any luck solving this?
Just installed bipython from pip on OSX (python2.7) and see same error.