Cannot deserialize consecutive CDATA elements

There is an issue deserializing consecutive CDATA elements. I stumbled across this after serializing some text containing CDATA tags. While serializing closing ']]>' is correctly replaced with ']]]]><![CDATA[>' which were not treated properly in the deserialization process.
I could fix the bug in the XmlReader's Content() method as of below.
public string Content()
{
	switch (_reader.NodeType)
	{

		case XmlNodeType.Attribute:
			return _reader.Value;
		default:
			if (_reader.IsEmptyElement && _reader.CanReadValueChunk)
			{
				Set();
				return string.Empty;
			}

			var isNull = IsSatisfiedBy(NullValueIdentity.Default);

			if (!isNull)
			{
				_reader.Read();
			}

			var result = isNull ? null : _reader.Value;

			if (_reader.NodeType == XmlNodeType.CDATA)
			{
				var sb = new StringBuilder(result);
				do
				{
					_reader.Read();
					Set();
					sb.Append(_reader.Value);

				} while (_reader.NodeType == XmlNodeType.CDATA);

				result = sb.ToString();
			}
			else if (!string.IsNullOrEmpty(result))
			{
				_reader.Read();
				Set();
			}

			return result;
	}
}

And here is the test:
public sealed class Issue447Tests
{
	[Fact]
	public void Verify()
	{
		const string content = @"<?xml version=""1.0"" encoding=""utf-8""?><Issue447Tests-Subject xmlns=""clr-namespace:ExtendedXmlSerializer.Tests.ReportedIssues;assembly=ExtendedXmlSerializer.Tests.ReportedIssues""><Message><![CDATA[Hello ]]><![CDATA[world!]]></Message><Message2><![CDATA[Not ]]><![CDATA[this.]]></Message2></Issue447Tests-Subject>";

		var serializer = new ConfigurationContainer().Create()
													 .ForTesting();

		serializer.Deserialize<Subject>(content)
			.Message.Should().Be("Hello world!");

		serializer.Deserialize<Subject>(content)
			.Message2.Should().Be("Not this.");
	}

	sealed class Subject
	{
		[Verbatim]
		public string Message { get; set; }

		[Verbatim]
		public string Message2 { get; set; }
	}
}

All other tests passed successfully, too :wink:.
Edit: Full post of test class.

Hah!  Good catch, @ms0815user.  Thank you for reporting it.  I didn't even know this was possible. 😁  I will take a look at this and see if I can get it in for today's release. 👍

Alright @ms0815user you can try out this build here:
https://github.com/ExtendedXmlSerializer/home/pull/448#issuecomment-692705093
I mostly went with your implementation, and did another check before creating the StringBuilder to avoid a  hotpath allocation unless necessary.  I did look into more performant solutions but they were either incomplete and/or required unsafe code.  Since this is a bit of an edge case I'm OK with the allocation if necessary.
FWIW, don't be afraid to submit a PR as well going forward.  Doing so will get your name listed as a PR author in our README/release notes upon release.  If you're into that sort of thing, that is. 😊

Alright, this has been posted to NuGet:
https://www.nuget.org/packages/ExtendedXmlSerializer/
Please let me know of any further questions/issues you encounter regarding this issue and I will reopen to take a look.  Closing for now.


LESS relative paths for CSS url() change when requiring gulp tasks externally

When I run gulp build from Project-Root/Semantic-UI, my LESS from /site-theme/globals/site.overrides...
background: url(/static/img/crossword.png);

...yields the following CSS:
background: url("../../../static/img/crossword.png");

However, when I run gulp build from Project-Root where I have my own gulpfile...
var
  gulp  = require('gulp'),
  watch = require('./Semantic-UI/tasks/watch'),
  build = require('./Semantic-UI/tasks/build')
;

gulp.task('watch', watch);
gulp.task('build', build);

... my LESS yields this CSS:
background: url("../../static/img/crossword.png");

*Note: there are only 2 "../" rather than the correct 3.
Is this expected behavior?
My directory structure for reference:
Project-Root
|
|gulpfile.js     <-------My custom gulpfile
|/Semantic-UI
| |
| |gulpfile.js   <-------Provided by SUI
| |/tasks
| | |
| | |watch.js
| | |build.js
|
|/public
| |
| |/themes/semantic/packaged/
| | |
| | |semantic.css
| |
| |/static/img/
| | |
| | |crossword.png
| |
|
|/site-theme
| |
| |/globals
| | |
| | |site.overrides    <------ Location of the LESS in question
| | |

Since I don't have much time to debug this right now I recommend experimenting yourself.
This is how css paths are resolved


Updated website and tutorials for v4

Some tutorials now have animation in them to give a clearer explanation of how things work. Added a short tutorial about creating Atoms with the new Search function. Page about installation has also changed slightly.

Great work adding the gif:s! Approved


Potential Licensing Concerns in 'RenpyThief' Software

Dear Maintainers,
I am writing to bring to your attention potential licensing concerns regarding the 'RenpyThief' software, which is actively promoted on Bilibili (author's Bilibili page: https://space.bilibili.com/314844/dynamic).
This software appears to be built heavily on the integration and modification of several open-source projects, which may be used in a way that is inconsistent with their respective licensing terms. Notably, several projects under the GPL v3 license, which require derivative works to be open-sourced under the same license, seem to be central to its core functionalities.
The following are some of the potentially impacted projects:
RPG Maker MV Cheat Menu Plugin: https://github.com/emerladCoder/RPG-Maker-MV-Cheat-Menu-Plugin
RPG Maker MV Speed Plugin Clipboard Hook: https://ulmf.org/threads/rpg-maker-mv-speed-plugin-clipboard-hook.8976
MisakaHookFinder: https://github.com/hanmin0822/MisakaHookFinder
Textractor: https://github.com/Artikash/Textractor
BepInEx: https://github.com/BepInEx/BepInEx
XUnity.AutoTranslator: https://github.com/bbepis/XUnity.AutoTranslator

My intention is solely to inform you of these observations and the potential licensing conflicts arising from the use of open-source projects in 'RenpyThief.' Determining the implications and any necessary actions falls within your purview.
However, I believe it was important to highlight this potential issue due to the reliance on open-source projects as the foundation for the software's core features.
Thank you for your time and understanding. Please let me know if I can provide any further details or clarification.
Best regards

I don’t understand what bepinex and XUnity.AutoTranslator have to do with it There are applications written in C# and have nothing to do with Renpy, which is written in Python. RenpyThief Chinese is interested in translating something using the Renpy engine XUnity.AutoTranslator tool for translating anything using the Unity engine bepinex is a bootloader for unity, net framework, net, net core I'm interested in clarification, write them to me please


comparing variable

Is it possible to compare two variables?
{
"RuleName": "CompareTwoVaribles",
"ErrorMessage": "Variables could not be compared.",
"ErrorType": "Error",
"RuleExpressionType": "LambdaExpression",
"Expression": "input1.field1 == input.field2"
}

you can compare variables the same way you would do in c# lang. If the types are different, you may need a typecasting before comparing


Adding create-file worker

This worker creates a file with the given path / filename and content
@JakobMiksch @hblitza please review

Thanks for review!


Long lag after exceptions are handled

After "done handling exceptions" is printed, there is still a long lag. We should profile and add logging statements so the user doesn't think the program is hanging.

No longer an issue, reopen if needed.


feat: Add Cheatcodes::AddAccount

Give an overview of the tasks completed
Arbiter currently assumes all accounts are initialized before the execution. However sometimes the user cannot anticipate all the involved accounts. With the Cheatcodes::AddAccount cheat code, the user can inject a new account mid execution and then update the balance with Cheatcodes::Deal cheat code.
The PR is based on top of #977.
REMINDER! Please check that you have done the following prior to submitting this PR:

[x] Checked that the relevant version(s) have been incremented if necessary.
[x] Ran both and made any changes necessary for:

cargo +nightly fmt --all
cargo clippy --all

Reviewing next week

Overall like the changes - just need to make sure everything merges in clean

At first I wanted to expose the whole Instruction struct and add Cheatcodes(Instruction) but then the struct is cyclic.
Maybe a more sound way would be to have an apply_instruction(Instruction) function or modify apply_cheatcode(Cheatcodes) to apply_cheatcode(Instruction) however this would be a breaking change.

Actually - maybe we can just keep the existing apply_cheatcode and expose a new method apply_instruction(Instruction), @kinrezC wdyt?


Fix #268: 'warning: no files found matching X under directory Y

These warnings were due to absent filetypes being looked for based on the contents of MANIFEST.in. This PR fixes those warnings both at the time of creating the package and at the time of installing.

Coverage remained the same at 86.223% when pulling 0b9ce0c66769639489b696726ea420048e8d3c55 on mfcovington:develop into 076c473726a47478e4b626fc2a1c0dd91777f53c on dyve:develop.

Thanks!


CCDB: Update dependencies

Changes

Updates ccdb dependencies: core-js, d3, sinon, moment, whatwg-fetch, and webpack in the main project package.json.
Set fetch call on the window, per instructions in whatwg-fetch.
Add import 'regenerator-runtime/runtime'; in tests to satisfy babel.

How to test this PR

Run ./frontend.sh
Run a local server and check tools on http://localhost:8000/data-research/consumer-complaints/

Screenshots

This fixes the map for me locally (non-Docker)


[Contribution wanted] Readme file in another language

Hi! I really love this template and wanna make some contribution. Since there are many current and potential users that might have trouble reading the long readme file, I would like to translate the Readme.md into Chinese if you like. This would attract more users and reduce their time of fighting with language. Hope I can help. 😄

Sure, go ahead! :+1:
My mandarin is quite limited (so are my other non-coding language skill :smile: ), so I would delegate possible translations to the "user base". It's probably best to create separate files, e.g. Readme.zh-CN.md etc. and link all of them in the main Readme like
Forty Seconds CV
================

[German] [French] [Chinese] ...

## Description

and link-back the main document in the respective translations at the same position. What do you think?

Great! 👍  That's quite a good way, I have seen other projects using this method too. It's clear and easy to find.
I would make a PR later when I finished, . I'm really not an expert at coding, this is what I can help. :v:

If we're going to do different languages, I don't mind taking care of the Dutch version. Although most of us Dutchies understand English quite well, it does make the class look more professional. I'll wait for the PR to be merged, and I'll add Dutch to it.

Great! Then I will take care of the German version.

I've completed the Dutch translation here: #31.
Cheers!

Great! Thanks!
Sorry for the long delay...life with a newborn is more time-limiting than expected :sweat_smile:

Thank you!
And congrats! I can imagine! I hope you're also enjoying it, though :wink:


Draw shapes to avoid/not leave

Having an icon-tool to shape lines, circles or rectangles that would be useful to delimit a zone where whe want to draw a track inside or outside, these independent objects could be also saved in GPX format as independant segments to be reloaded along classical tracks (we already can simulate this using the "as the crow flies" functionality), these objects would also indicate their length or surface. You can use my very humble code here : http://mapa4.free.fr/Click&Cercle.htm

This is interesting because GraphHopper Routing API allows to specify shapes to avoid (circles or polygons), see here.

Thanks to have taken this into account for a next enhancement !

Unfortunately the block_area parameter is now deprecated as announced here. I don't think this feature could be implemented without being supported by the routing API itself.

Too bad...


Path as token

Question!
I have read the path-to-regexp library. And i saw a method that i need.
pathToRegexp.parse()
I could not find in sources. Does vue-router exposes this or something similar?

No it doesn't expose it.

Hi, thanks for your interest but Github issues are for bug reports and feature requests only. You can ask questions on the forum, the Discord server or StackOverflow.


Defer fd.Close() (fixes #37)

Should be self explanatory, fixes #37

What happens if we're already returning another error though - say, if the write fails. There's a risk we overwrite it with a nil error from the close?

@st-jenkins add to whitelist

@st-jenkins add to whitelist

The function isn't propagating the error back to the caller, it only prints the error, so overwriting shouldn't cause a problem. With that said, there would be a problem if the function is changed later to return the error, I suppose it's better to keep the err variable within the deferred function's scope.

Indeed, I only read the diff. That function is a bit of a WTF as it should return an error to the caller instead, but what the hell.


Issue #745: changes in http_client example

Changing http_client examples following questions in #745

Codecov Report

Merging #746 into master will decrease coverage by 0.96%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #746      +/-   ##
==========================================
- Coverage   67.67%   66.70%   -0.97%
==========================================
  Files          53       53
  Lines        7446     6407    -1039
==========================================
- Hits         5039     4274     -765
+ Misses       2407     2133     -274

Impacted Files
Coverage Δ

include/pistache/router.h
92.00% <0.00%> (-8.00%)
:arrow_down:

include/pistache/tcp.h
75.00% <0.00%> (-5.00%)
:arrow_down:

include/pistache/net.h
86.66% <0.00%> (-3.34%)
:arrow_down:

include/pistache/flags.h
85.71% <0.00%> (-3.18%)
:arrow_down:

include/pistache/view.h
86.36% <0.00%> (-3.12%)
:arrow_down:

include/pistache/description.h
79.16% <0.00%> (-2.98%)
:arrow_down:

include/pistache/http_headers.h
92.85% <0.00%> (-2.98%)
:arrow_down:

src/common/description.cc
17.07% <0.00%> (-2.78%)
:arrow_down:

include/pistache/optional.h
90.62% <0.00%> (-1.97%)
:arrow_down:

include/pistache/date.h
34.12% <0.00%> (-1.78%)
:arrow_down:

... and 39 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 a90fecd...1e0a6dc. Read the comment docs.


Avoid RangeError: Maximum call stack size exceeded

Let's say I have 150 schemas that I need to use for input validation of my application.
I did try with asyncCompile but it doesn't work.
addSchema method doesn't compile the added schemas.
There is a way to precompile all my 150 schemas before validating and avoid Maximum call stack error?

I did try with asyncCompile but it doesn't work.

You may file a separate issue on that, not sure why it didn't work, but some details would help.

addSchema method doesn't compile the added schemas.

They are compiled when you call compile or getSchema method. Not compiling when addSchema is called allow to add them in any order.

There is a way to precompile all my 150 schemas before validating and avoid Maximum call stack error?

I doubt the Maximum call stack error is caused by the number of schemas, more likely it is caused by infinite recursion in schema definition for the same location in data instance: either you have schema that is similar to {$ref: '#'} or to schemas referencing each other at the same location in data.
Do they all use each other so they cannot be compiled separately? Please make a minimal example to reproduce the issue.
When you file an issue, can you please use the template?


POST /user/ms-username

Endpoint to link MS usernames to freeCodeCamp account

I would like to work on this issue


Setup renovate

Setup Renovate to automatically manage dependencies.
You'll need to install the Renovate app for this to work.

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (fa9ee9c) compared to base (f4c9faa).
Patch has no changes to coverable lines.

:exclamation: Current head fa9ee9c differs from pull request most recent head 3018425. Consider uploading reports for the commit 3018425 to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #127   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           40        40
  Lines         1240      1240
  Branches       260       260
=========================================
  Hits          1240      1240

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[Storage] Fix az storage entity merge: Stop automatically casting DisplayVersion to float

Description

Bug fix for RE: Bug regarding types in az storage entity
When updating a table entity, a number string is automatically casted to an int or float so "2021" would become an int and "2021.4" would become a float
We are stopping this automatic conversion for
Testing Guide

az storage entity merge (…) --entity PartitionKey=”fisken” RowKey=”” DisplayVersion=”2021.4”
-> Success, data type is Double
az storage entity merge (…) --entity PartitionKey=”fisken” RowKey=”” DisplayVersion=”2021.4”<EMAIL_ADDRESS>-> Success, data type is String
History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

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

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Storage


feat: add move stack

migrates https://dwm.suckless.org/patches/movestack/
https://dwm.suckless.org/patches/movestack/dwm-movestack-6.1.diff

Tested, very cool :)
I think you should also include in the PR the needed change in config.def.h, so it will work out of the box:
    { MODKEY|MOD_SHIFT,             'J',       movestack,           {.i = +1 } },
    { MODKEY|MOD_SHIFT,             'K',       movestack,           {.i = -1 } },

+1 would be good to have config by default.

@prabirshrestha  @nir9  Defaults are added
I gave multi monitor a shot but this highly exceeds my C skills :D
I think this would be the next big feature :D

@PascalSenn you can start by porting some of the code from https://github.com/prabirshrestha/dwm-win32-6. I personally only use one monitor and dwm-win32-6 is quite buggy so stopped using it and went back to this one.
I have been actually thinking of adding lua support for dwm-win32 instead. I started a bit of work with duktape js but didn't like it much as it was old javascript. https://github.com/prabirshrestha/dwmjs. Some how tools I use these days seems to be using lua more such as wezterm. Also I have been sending patches to vim to improve lua support. In linux for the past few months I have been using awesomewm which is quite powerfull but didn't like much the lua api and the gpl license doesn't help here. I also want to make the bar more customizable hence first need to port lemonbar to win32 which I mentioned it at https://github.com/prabirshrestha/dwm-win32/issues/5.
I have also been very impressed with lua editor here https://github.com/rxi/lite.
So my ideal goal is to actually have the simplicity of dwm and lemobar with the power of lua that is similar to awesomewm and lite. I have also been thinking of might be using rust instead of c since learning rust haven't been fan of manual memory management in c. Let me know if any of you have preference here or want to stick with just c.

Merged. Thanks.


SAK-49440 msgcntr modify the default behavior for 'mark as read'

https://sakaiproject.atlassian.net/browse/SAK-49440

Looks good just needs a couple of minor changes, so I will approve

There is still a lot of work to be done here


spring-cloud-gcp-storage  can not be used with FakeStorage or local emulator

If I use the spring-cloud-gcp-storage with
<dependency>
    <groupId>com.google.cloud</groupId>
    <artifactId>google-cloud-nio</artifactId>
    <version>0.34.0-alpha</version>
</dependency>

I will get the following error, b/c of missing implementation:
Caused by: java.lang.UnsupportedOperationException: null
	at com.google.cloud.storage.contrib.nio.testing.FakeStorageRpc.create(FakeStorageRpc.java:98) ~[google-cloud-nio-0.34.0-alpha.jar:0.34.0-alpha]
	at com.google.cloud.storage.StorageImpl$2.call(StorageImpl.java:111) ~[google-cloud-storage-1.15.0.jar:1.15.0]
	at com.google.cloud.storage.StorageImpl$2.call(StorageImpl.java:108) ~[google-cloud-storage-1.15.0.jar:1.15.0]
	at com.google.api.gax.retrying.DirectRetryingExecutor.submit(DirectRetryingExecutor.java:89) ~[gax-1.16.0.jar:1.16.0]
	at com.google.cloud.RetryHelper.run(RetryHelper.java:74) ~[google-cloud-core-1.15.0.jar:1.15.0]
	at com.google.cloud.RetryHelper.runWithRetries(RetryHelper.java:51) ~[google-cloud-core-1.15.0.jar:1.15.0]
	at com.google.cloud.storage.StorageImpl.create(StorageImpl.java:107) ~[google-cloud-storage-1.15.0.jar:1.15.0]
	... 107 common frames omitted

If I try to use it with a local running emulator configured by:
@Bean
public static Storage storage(GcpProjectIdProvider projectIdProvider) {
	return StorageOptions.newBuilder()
			.setCredentials(NoCredentials.getInstance())
			.setHeaderProvider(new UsageTrackingHeaderProvider(GcpStorageAutoConfiguration.class))
			.setProjectId(projectIdProvider.getProjectId())
			.setHost("http://localhost:8081")
			.build().getService();
}

I will get the following error:
Caused by: com.google.api.client.googleapis.json.GoogleJsonResponseException: 404 Not Found
Not Found

	at com.google.api.client.googleapis.json.GoogleJsonResponseException.from(GoogleJsonResponseException.java:146) ~[google-api-client-1.23.0.jar:1.23.0]
	at com.google.api.client.googleapis.services.json.AbstractGoogleJsonClientRequest.newExceptionOnError(AbstractGoogleJsonClientRequest.java:113) ~[google-api-client-1.23.0.jar:1.23.0]
	at com.google.api.client.googleapis.services.json.AbstractGoogleJsonClientRequest.newExceptionOnError(AbstractGoogleJsonClientRequest.java:40) ~[google-api-client-1.23.0.jar:1.23.0]
	at com.google.api.client.googleapis.services.AbstractGoogleClientRequest$1.interceptResponse(AbstractGoogleClientRequest.java:321) ~[google-api-client-1.23.0.jar:1.23.0]
	at com.google.api.client.http.HttpRequest.execute(HttpRequest.java:1065) ~[google-http-client-1.23.0.jar:1.23.0]
	at com.google.api.client.googleapis.services.AbstractGoogleClientRequest.executeUnparsed(AbstractGoogleClientRequest.java:419) ~[google-api-client-1.23.0.jar:1.23.0]
	at com.google.api.client.googleapis.services.AbstractGoogleClientRequest.executeUnparsed(AbstractGoogleClientRequest.java:352) ~[google-api-client-1.23.0.jar:1.23.0]
	at com.google.api.client.googleapis.services.AbstractGoogleClientRequest.execute(AbstractGoogleClientRequest.java:469) ~[google-api-client-1.23.0.jar:1.23.0]
	at com.google.cloud.storage.spi.v1.HttpStorageRpc.create(HttpStorageRpc.java:219) ~[google-cloud-storage-1.15.0.jar:1.15.0]
	... 113 common frames omitted

The running emulator will show the following:
[datastore] Feb 19, 2018 12:10:52 PM io.gapi.emulators.netty.NotFoundHandler handleRequest
[datastore] INFO: Unknown request URI: /storage/v1/b?project=projectXYZ&projection=full

How to use this resource binder in local development setup without running against the real cloud?

Using the google-cloud-nio, are you overwriting the auto-configured Storage bean with something like:
  @Bean
  public Storage storage() {
    return LocalStorageHelper.getOptions().getService();
  }

As for using the "emulator",  you're referring to the Datastore emulator, not Storage emulator. AFAIK there is no Storage emulator in gcloud.

Using the google-cloud-nio, are you overwriting the auto-configured Storage bean with something like:

yes exactly

As for using the "emulator", you're referring to the Datastore emulator, not Storage emulator. AFAIK there is no Storage emulator in gcloud.

OK good to know that there is a difference

It looks like you're trying to create a bucket using google-cloud-nio.
I don't think it's necessary for the emulator. You can just create the blobs directly.
Can you share a bit more of how you're using Storage?

This is how I use it:
@Autowired
private ResourceLoader resourceLoader;

@PostConstruct
public void getResource() throws IOException {
	WritableResource wr = (WritableResource) resourceLoader.getResource("gs://test-bucket/foo.txt");
	try (OutputStream out = wr.getOutputStream()) {
		out.write("123".getBytes());
	}
}

even if i don't use any bucket name like resourceLoader.getResource("gs://foo.txt") I get an exception:
Caused by: java.lang.UnsupportedOperationException: Cannot get output stream for a storage bucket.
	at org.springframework.cloud.gcp.storage.GoogleStorageResourceBucket.getOutputStream(GoogleStorageResourceBucket.java:172)

That's because gs://foo.txt is the URI for a bucket, not a file. You must add the bucket to the URI.
I'm going to assume that using the Datastore emulator was an oversight, since it appears you want to test your storage code locally.
In that case, it seems like FakeStorageRpc could be the way to go. However, make sure you read the class's Javadoc:
 *
This class is not thread-safe. It's also (currently) limited in the following ways:
 *

 *
Supported
 *

 *
object create
 *
object get
 *
object delete
 *
list the contents of a bucket
 *
generations
 *

 *
Unsupported
 *

 *
bucket create
 *
bucket get
 *
bucket delete
 *
list all buckets
 *
file attributes
 *
patch
 *
continueRewrite
 *
createBatch
 *
checksums, etags
 *
IAM operations

 *

 *

It doesn't look like it supports creating buckets.

@joaoandremartins

That's because gs://foo.txt is the URI for a bucket,

I know. As you can see in my comment above I tried both: gs://foo.txtand gs://test-bucket/foo.txt

@Andy2003 Can you try against master? I think your issue is related to bucket auto-creation that we had in there before and removed with #433. Thanks!

#433 solved this issue


Update installModels to go through the v2 API endpoints

https://github.com/RelationalAI/rai-sdk-javascript/blob/fdbf90dd33cd3de52f47aa1251d0c60b597e1551/src/api/model/modelApi.ts#L26-L41
It's currently using the v1 actions API. It should instead be issuing an exec() query that loads the models, similar to this implementation of load_json in the julia SDK:
https://github.com/RelationalAI/rai-sdk-julia/blob/6c9ddf55fe791d82b15c8a9e9f80c0bcdec04064/src/api.jl#L650-L655
Ideally it should construct a query string + inputs, that would look something like either of these options:
query = `def insert:rel:catalog:model = model_insertions`;
inputs = {
    name: "model_insertions",
    value: models.map((m, index) => {
        [m.name, m.value],
    })
};

or like this (copied from the current Console code):
  models.forEach((m, index) => {
    const inputRelation = `__model_value__${index}__`;

    queryStrings.push(
      `def delete:rel:catalog:model["${m.name}"] = rel:catalog:model["${m.name}"]`,
      `def insert:rel:catalog:model["${m.name}"] = ${inputRelation}`,
    );
    queryInputs.push({
      name: inputRelation,
      value: m.value,
    });
  });

https://github.com/RelationalAI/rai-ux/blob/2ed5627959f793b19f5f968d8113509c8610e548/console/src/utils/sdkUtils.ts#L310-L330

@denisgursky I've assigned this one to you, and put it at medium priority. It would be great to have this converted so that we can see a dent in our v1 vs v2 numbers 😭

Note: This issue has been migrated to https://relationalai.atlassian.net/browse/RAI-6246.
This link is only accessible to employees of RelationalAI.


Adjust tolerances of F.einsum tests

#3986

jenkins, test this please.

Jenkins CI test (for commit 8164f3586aacebd853658e6a91a832c4134e6795, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

jenkins, test this please.

jenkins, test this please.

Jenkins CI test (for commit 8164f3586aacebd853658e6a91a832c4134e6795, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

Jenkins CI test (for commit 22c46dc56d2ed048029ffe3da0ea10affad898d8, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

jenkins, test this please.

Jenkins CI test (for commit 22c46dc56d2ed048029ffe3da0ea10affad898d8, target branch master) failed with status FAILURE.
(For contributors, please wait until the reviewer confirms the details of the error.)

CI failures seem unrelated. LGTM!


V 97.6 Many issues with carousel

Links are not working on ios devices (Iphone, ...) with Safari Browser.
On Firefox the Mouse pointer is not released for "swiping" - on Chrome it works fine.

Fixed in 5d68686


Triggered Kotsadm version update 2021.11.4-2-nightly

Automated changes by the auto-kotsadm-update GitHub action

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


Lazy-load files and dependencies

Calling require 'json_schemer' (or added the gem to Gemfile) eager-loads a lot of files and dependencies which adds to startup time and memory usage. The performance impact is tiny, but with many gems installed, it all adds up.
I suggest using Ruby's autoload feature to allow lazy-loading files and depencies. With this PR the number of files loaded is decreased from 98 to 12.
I used the following script to check how many files are included.
module Kernel
  alias_method :old_require, :require

  def require(file)
    loaded = old_require(file)
    puts file if loaded
    loaded
  end
end

$LOAD_PATH.unshift "#{__dir__}/lib"
require 'json_schemer'

Closing this as out-of-date. Feel free to reopen if you'd like to continue working on it.


Force alignment of structures

Unaligned structures can incur significant performance penalties during memory operations.
Fixes #145

@mkannwischer Added alignment constraints in poly.c and kem.c.


Is any solution to add Comments for translation in attribute?

for this situation :
<td data-title="{{'some Title' | translate}}"> </td>

1.5 years later, I run into the exact same issue.
One idea comes to mind:
Maybe the filter could look for a comment NEXT to it in the source, something like:
<!-- /// Something something, hush hush! -->
{{ 'This is just an example. Foo. Bar!' | translate }}

Similar to https://api.jquery.com/prev/


test flake: mutating web hook failure

https://cloudnative-k8sci.southcentralus.cloudapp.azure.com/view/pmem-csi/job/pmem-csi/job/devel/330/testReport/junit/fedora-1_17.direct-production/E2E/Testing___1_17____Testpattern__Dynamic_PV__filesystem_volmode___volumeMode_should_fail_to_use_a_volume_in_a_pod_with_mismatched_mode__Slow_/
Testing / 1.17 / fedora-1_17.direct-production.E2E.[Testpattern: Dynamic PV (filesystem volmode)] volumeMode should fail to use a volume in a pod with mismatched mode [Slow] (from fedora-1_17)

/mnt/workspace/gopath/pkg/mod/k8s.io/kubernetes@v1.20.2/test/e2e/storage/testsuites/volumemode.go:292
Feb 25 03:22:38.246: Failed to create pod
Unexpected error:
    <*errors.StatusError | 0xc00a8b35e0>: {
        ErrStatus: {
            TypeMeta: {Kind: "", APIVersion: ""},
            ListMeta: {
                SelfLink: "",
                ResourceVersion: "",
                Continue: "",
                RemainingItemCount: nil,
            },
            Status: "Failure",
            Message: "Internal error occurred: failed calling webhook \"pod-hook.pmem-csi.intel.com\": Post https://pmem-csi-intel-com-scheduler.default.svc:443/pod/mutate?timeout=10s: context deadline exceeded",
            Reason: "InternalError",
            Details: {
                Name: "",
                Group: "",
                Kind: "",
                UID: "",
                Causes: [
                    {
                        Type: "",
                        Message: "failed calling webhook \"pod-hook.pmem-csi.intel.com\": Post https://pmem-csi-intel-com-scheduler.default.svc:443/pod/mutate?timeout=10s: context deadline exceeded",
                        Field: "",
                    },
                ],
                RetryAfterSeconds: 0,
            },
            Code: 500,
        },
    }
    Internal error occurred: failed calling webhook "pod-hook.pmem-csi.intel.com": Post https://pmem-csi-intel-com-scheduler.default.svc:443/pod/mutate?timeout=10s: context deadline exceeded
occurred
/mnt/workspace/gopath/pkg/mod/k8s.io/kubernetes@v1.20.2/test/e2e/storage/testsuites/volumemode.go:315

There are not log entries for the webhook call in the controller, so it could be that this somehow timed out already in the kube-scheduler (DNS lookup issue?).
We should relax the configuration for the mutating webhook when testing and allow it to fail: the testing system doesn't retry pod creation in many cases, so even temporary failures with the webhook are leading to test flakes.

Resolved by allowing the webhook to fail also during testing: PR #907


Segformer swi signed

Following Segformer SWI #1292 PR but now signed.

LGTM overall. Please fix minor docstring issues and we can merge. Great work!


Needs media-split block for blocks under Purpose Build Apps

https://www.esri.com/en-us/capabilities/field-operations/overview#:~:text=behind your firewall.-,Plan,-Get the office
Doc just lists them as content with no structure.

https://www.esri.com/en-us/capabilities/field-operations/overview#:~:text=behind your firewall.-,Plan,-Get the office

Import done, the block will need further styling


ci: align Python version of tox and Github action

Just like https://github.com/Qiskit/qiskit-addon-mpf/pull/44.
See also #45.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>5:
0.0%

Covered Lines:
431

Relevant Lines:
431

💛  - Coveralls


Update release notes

Signed-off-by: Kunal Kotwani<EMAIL_ADDRESS>
Description

Fixes release notes for v2.5

Issues Resolved

N/A

Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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

Skipping gradle check since only release notes were added.


Segfault on load DB

How frequently does the bug occur?
Sometimes
Description
Our app crashed for some users (including me) at launch, when loading the database. And this only happens once per user.
We suspect it's due to a migration ...
This migration: https://github.com/Infomaniak/android-kMail/blob/1.0.9/app/src/main/java/com/infomaniak/mail/data/cache/RealmDatabase.kt#L175
Stacktrace & log output
OS Version: Android 13 (TQ3A.230805.001)
Report Version: 104

Exception Type: Unknown (SIGSEGV)

Application Specific Information:
Segfault

Thread 0 Crashed:
0   libart.so                       0x6d179e5f6c        <unknown> +<PHONE_NUMBER>48
1   split_config.arm64_v8a.apk      0x6c91704f48        register_results_notification_cb
2   split_config.arm64_v8a.apk      0x6c91702744        Java_io_realm_kotlin_internal_interop_realmcJNI_register_1results_1notification_1cb
3   base.odex                       0x6c9a7a4744        <unknown> +<PHONE_NUMBER>68

https://sentry-mobile.infomaniak.com/share/issue/1dd3e2bd23154ad0982af11835ad33d3/
Can you reproduce the bug?
No
Reproduction Steps
No response
Version
1.10.2
What Atlas App Services are you using?
Local Database only
Are you using encryption?
No
Platform OS and version(s)
Android 12 and 13
Build environment
https://github.com/Infomaniak/android-kMail/blob/1.0.9/app/build.gradle
https://github.com/Infomaniak/android-kMail/blob/1.0.9/gradle/libs.versions.toml#L17

This always happens with "1.11.1".
And given the breadcrump, I don't think it's migration.
It's getting to be a lot 😬

https://sentry-mobile.infomaniak.com/share/issue/1dd3e2bd23154ad0982af11835ad33d3/
https://sentry-mobile.infomaniak.com/share/issue/293c388ca27f4c82bf24efdc41aebf16/

All events affect only android 12 and 13 devices
I checked, we already had 1.10.2 realm on our application in 1.0.8.
But what's changed on our side is that we've gone from compileSdk 33 to 34
https://github.com/Infomaniak/android-kMail/compare/1.0.8...1.0.9
Is Android 14 the problem?

Hi @JorisBodin
I don't think Android 14 is the problem. My guess is you are trying to register a change listener to a type deleted by  the migration, but I'm not sure how that is possible.
Following your migration logic it looks like you ended up calling newRealm.deleteAll() which should be fine and at least I could not reproduce the error using the same method.
I'm still investigating it.

Hello @cmelchior,
The problem occurred between 1.0.8 and 1.0.9
The newRealm.deleteAll() was already present before 1.0.8, so that's not the problem.
Thank you

The stacktrace looks like it could be related to #1598.  The callback class reference did not create a global reference so the class could potentially be unloaded behind the scene of the static initalized jclass.
I can't explain how this would be related to migration though, but maybe the issue is otherwise related to two simultaneous instances of the app during an upgrade.

@JorisBodin The above fix will be in 1.14.0. Is currently available in 1.14.0-SNAPSHOT (using snapshots if you want to see if it changes anything already.

Not reproducing the bug, we'll see if the error disappears from sentry after 1.14.0 update 🤞

@JorisBodin This popped up from another user on 1.14.1. Are you still seeing this and have you been able to gather more insights on exactly when it is happening?

@tevincent explained here https://github.com/realm/realm-kotlin/issues/1726#issuecomment-2084500711
I thought that was enough
Is it possible to reopen the PR?
Thanks

Need more info?


Typescript prop validation from .attrs

Not sure if I'm using styled-system incorrectly or if this isn't possible, but I'm currently unable to infer the theme's props when calling styled-system props from the .attrs function
I've constructed an example in code-sandbox here
import { color, ColorProps } from "styled-system";
import { styled } from "./styled-components";
import { theme } from "./theme";

const Box = styled("div")<ColorProps<typeof theme>>`
  ${color}
`;

// theme's colors are "primary" and "secondary"
// Is there a way to ensure type-safety when declaring color
// from the ".attrs" function?
const PrimaryBox = styled(Box).attrs({ backgroundColor: "primary" })``;

// This should throw a ts error since backgroundColor is not a key of theme colors
const TypescriptErrorBox = styled(Box).attrs({ backgroundColor: "primaryy" })``;

I'm wondering if this is more an issue with styled-components than styled-system, but in all cases, I'd love to inject some knowledge into .attrs.


BUG: Error when trying to delete a user

Steps to reproduce:

Install package in a Neos system with several user accounts
go to User Management
click on the delete icon of any user and confirm you want to delete

Error Message:
An exception occurred while executing 'DELETE FROM neos_contentrepository_domain_model_workspace WHERE name = ?' with params ["user-redakteur"]

SQLSTATE[23000]: Integrity constraint violation: 1451 Cannot delete or update a parent row: a foreign key constraint fails (`<db-name>`.`shel_neos_workspacemodule_domain_model_workspacedetails`, CONSTRAINT `FK_923CCEA8D940019` FOREIGN KEY (`workspace`) REFERENCES `neos_contentrepository_domain_model_workspace` (`)

Additional Information:

Neos: ~8.3.0
PHP: ^8.2
shel/neos-workspace-module: 1.0.0

Should be fine now with release 1.0.1

Btw. the "delete user" keywords were the relevant one here. For some reason, deleting a workspace worked fine. I don't actually know what the difference is as the deletion of the user triggers the deletion of the workspace 🤷🏻♂
So thanks for a good description!


Extend e2e command to work with heptio-extended e2e tests

The first pass of e2e is only solving for the common case of conformance e2e tests.
We would ideally like to reuse this command to fit any e2e-esque set of tests (such as heptio's extended e2e tests).

I think the answer is to generalize and rebrand the command in general.
I'm going to be writing up a design doc for this but it seems like a really good place for improvement this cycle.
Right now my thinking is along the lines of wanting to get out status codes and 1-2 different types of results parsing (e.g. junit). This would probably be done via some extra data the plugin can drop to help sonobuoy know where a certain piece of information is. TBD.


Out-of-memory issues with large tsv-file

Check list

[x] I have read through the README
[x] I have searched through the existing issues

Environment info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Others:

Version
csview 0.3.6

Problem / Steps to reproduce
First of all, thanks for this very nice and useful tool.
I'd like to view a large TSV-type file but it dies because of out-of-memory.
Steps to reproduce:

wget https://storage.googleapis.com/gcp-public-data--gnomad/release/2.1.1/vcf/exomes/gnomad.exomes.r2.1.1.sites.1.vcf.bgz
zcat "gnomad.exomes.r2.1.1.sites.1.vcf.bgz" | grep -v "^##" | csview -t | less -S

Would it be possible to limit csview's memory usage and make it play nicely with less?

Hi @Hoeze, how big is the file ? csview needs to load the entire file into memory to compute alignment. Fow now the easiest way to avoid oom on too large files is to filter the data you focus on then feed them into csview:
preview first 10,000 lines:
zcat "gnomad.exomes.r2.1.1.sites.1.vcf.bgz" | grep -v "^##" | head -n 10000 | csview -t | less -S

preview lines between 100 and 200:
zcat "gnomad.exomes.r2.1.1.sites.1.vcf.bgz" | grep -v "^##" | sed -n '100,200p;201q' | csview -t | less -S

I see, thanks for the explanation @wfxr.
Yes, the files can be up to 1TB of compressed size.
One solution could be to calculate alignment on the first 10,000 lines.
Another useful feature would be to split cells which are too large into multiple lines:
+------------------+
| too large column |
+------------------+
| some cell with v |
| ery long text    |
+------------------+

@Hoeze Unfortunately the crate prettytable-rs which csview depends on does not support streaming rendering. But your advice will be considered when csview implements its own rendering someday.

Hi @Hoeze, streaming rendering has been implemented by https://github.com/wfxr/csview/pull/27. Thanks for your contribution! You can try the latest release. Please let me know if there are any problems.


How to rotate image

thanks to provide library.
but one things is remaining to rotate image after crop

Make sure your image doesn't contain an EXIF rotation tag. If it does you should handle the rotation yourself before passing the image to android-cropimage since this library doesn't handle EXIF metadata.


[firebase_database] Support the v2 Android embedder plugins API

@amirh
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/315. Any further collaboration will be done there.

To avoid confusing with project https://github.com/flutter/flutter/projects/59 this issue will remain open here till resolved.


enable the test non_auth

once fixed,
Scenario: I can't access build image page with norole user
  Given I am authorized as "norole" with password "norole"
  And I navigate to "rhn/manager/cm/imageprofiles/edit/2" page
  And I should not see a "Internal Server Error" text

https://github.com/SUSE/spacewalk/pull/1172


Allow shell escapes for cargo.

OS: Debian, mixed version.
VSCode version: 1.14.1
Extension version: 1.5.0
Toolchain version: rust 1.44.0-nightly
Build target: nightly-x86_64-unknown-linux-gnu
Python version: 3.8.3rc1
I wanted to use ${relativeFileDirname} to select the current example to debug.  I was unable to use shell escapes to run perl, but that failed.
There is a part that confuses me, how does lldb know what cargo built?
Debug log
        {
            "type": "lldb",
            "request": "launch",
            "name": "Debug example",
            "cargo": {
                "args": [
                    "run",
                    "--example=$(path=\"${relativeFileDirname}\" perl -e 'my$p=$ENV{path};if($p=~%examples/([^/]*)%){$1}else{print\"sandbox\"}')",
                ],
            },
            "args": [],
            "cwd": "${workspaceFolder}"
        },

Unfortunately, cargo needs to be run before VSCode expands $-variables.  I can't expand them myself either, because there isn't an API to query these values in VSCode.   I suggest you use a task to build the current example.  Expansion of variables will work in other launch config keys and values, e.g. in "program".

There is a part that confuses me, how does lldb know what cargo built?

It runs cargo with --message-format=json, then parses its output to get runnable compilation artifacts.


Error passing weights

Hi.
First of all, thanks for your great repo.
When I'm passing weights like this:
Poly1FocalLoss(num_classes=num_classes,
                      reduction='mean',
                      weight=torch.tensor([0.1, 0.9]))

I get this error:
RuntimeError: The size of tensor a (256) must match the size of tensor b (2) at non-singleton dimension 3
B, C, H, W = 32, 2, 256, 256
Label is not one_hoted.

Hello @farhadinima75 and you are welcome,
The weight tensor shape should be broadcastable to [B, num_classes, H, W], both when labels are one-hot and when they hold class ids. In your case you could do:
weight = torch.tensor([0.1, 0.9]).reshape([1, 2, 1, 1])
...
Poly1FocalLoss(num_classes=num_classes,
                  reduction='mean',
                  pos_weight=weight)

Note that I have just updated the code, now Poly1FocalLoss takes 2 weight parameters: weight and pos_weight because the implementation of the  underlying binary_cross_entropy loss also takes 2 types of weight tensors. I have updated README. Check out the links there for further information on how to set pos_weight and weight.

Worked. TNX.


Standardizing data prior to use

The first question I have is, would it be worth standardizing my datasets before using CTAB-GAN on them?
I haven't done this so far as the CTAB-GAN tutorial didn't include standardization, but realized after running cWGAN code (which does use standardizing) that it could potentially speed up the time it takes to run CTAB-GAN on each dataset, among other benefits.
Or would doing so mess up the running of CTAB-GAN?

No, you don't need to normalize your data to use CTAB-GAN. It has already implemented mode-specific-normalization for continuous variables and one-hot encoding for categorical variables.

Thanks!


docs: fix incorrect feature flag name in migration build doc.

The migration guide currently lists COMPILER_FILTERS as a feature flag but to disable compiler support for filters, you must specify COMPILER_FILTER: false (not plural COMPILER_FILTERS) in compatConfig. Although the enum key for the feature is COMPILER_FILTERS (plural), the value string is singular so it would be more helpful to list the singular form.

Hmm, I think it's better to correct the enum value to match the key and the docs (the missing S was a mistake)

I think it's better to correct the enum value to match the key and the docs
I think that would be more ideal. I will fix it by the weekend.

Updated the PR to fix enum value.


Add Option for Archive Action in Notification

Fixes https://github.com/k9mail/k-9/issues/3403
Adds a new notification option allowing the user to chose to have an 'Archive' button on new message notifications instead of the 'Delete' button.

No worries!  As I was implementing this it struck me that the existing approach to notification actions was lacking.  However, I did not want to try and just totally refactor it without some input on a go-forward vision.  I figured I would submit what I had and either it would be accepted, or it would spark a conversation about a better direction to go.
So, thank you for the vision in #3530!


Update 'ros' to 1.14.6-p0

I've followed this guide step by step carefully. [Yes]

I've tested this package remotely and have excluded all broken builds.
Here is the links to the Travis/AppVeyor with status "All passed":

https://ci.appveyor.com/project/kgeorgiev93/hunter/builds/25028221
https://travis-ci.com/kgeorgiev93/hunter/builds/114242211

Testing:

https://ci.appveyor.com/project/ingenue/hunter/builds/25029710
https://travis-ci.org/ingenue/hunter/builds/541201779?utm_source=github_status&utm_medium=notification

Released: https://github.com/ruslo/hunter/releases/tag/v0.23.187


Suppress "Flow is initializing"

It would be great if there was a config option to remove this message. Its rather disruptive to the build process.

What build system are you using? You could ignore stderr I suppose, but it would be nice if either flow or the build system handled this more gracefully

Why would this ever show up more than once?  Why would this ever show up more than once?  Why would this ever show up more than once?  Why would this ever show up more than once?

@gabelevi We're using Webpack, and we only seem to get this spam once when it starts up. Not the end of the world, but we usually want to keep everything as clean as possible :).

Oh actually this isn't a Windows issue, this happens for everyone. Flow is trying to do a \r sort of thing to show a spinner while flow status waits for the Flow server to finish initializing or rechecking.
At the very least, if we should check Unix.isatty before clearing the line. And we probably shouldn't print status messages at all.

@gabelevi https://github.com/facebook/flow/blob/1f1c44269ee2a7473e68531d207154d067a742f0/hack/utils/tty.ml#L110
Perhaps webpack is doing something that makes fd 1 still look like a tty instead of a redirection, which would trick Unix.isatty.

@mroch - print_clear_line checks isatty, the logspew in the picture above is just coming from regular Utils.prerr_endlinef.


[MinFrequency] regex variable not computed in constructor

Hi!
For information, I don't have time for now to send a pull request but:
I build a MinFrequency by its constructor but when the GenericRecrawlableResolver .getMatchingMinFrequency () method is called then the MinFrequency throws the message "Value or pattern missing in minimum frequency" due to regex property is null.
The MinFrequency.setPattern () method  correctly builds the regex property.

I just made a new snapshot release with a fix for this.  Please confirm.  Thanks for reporting it!

It's ok.


idarename: auto apply type on instances when available

Only implemented in IDA right now. If you run CExporter/the types are in your IDB, it will set instances to their respective types.

Working on getting this applying types to vfuncs and funcs too - is it a safe assumption that all funcs in data.yml's a1 will be type*? I'll push what I have soon - it checks a1 for being a pointer/int64 - but I want to make sure.

Proper type definitions in data.yml is planned in The Future:tm: with this - maybe we can add support for typing functions in it with this? Right now, at least for MJIManager, @KazWolfe did the following:
Client::Game::MJI::MJIManager:  # Island Sanctuary
  instances:
    - ea: 0x14218B020
      pointer: True
  funcs:
    0x140B6C400: Initialize # static
    0x140B6C450: Destroy    # static
    0x140B6C490: GetSingleton  # static
    0x140B6F8B0: ctor
    0x140B6FB30: dtor
    0x140B6E570: LoadDemandResearchPacket
    0x140B6EAF0: IsRecipeUnlocked # (this, recipeId)
    0x140B6EB20: IsPouchItemLocked # (this, pouchItemId)
    0x140B6CDF0: IsFunctionUnlocked # (this, functionId)
    0x140B6D020: HandleActorControlPacket
    0x140B6DB50: HandleInboundPacket
    0x140B6EFA0: GetVisibleMinimapIcons
    0x140B6EB30: IsItemLocked
    0x140B6ECF0: GetPastureSlotCount
    0x140B6EE80: GetFarmSlotCount

I propose either parsing the comments or allowing you do something like 0x140B6CDF0: IsFunctionUnlocked(Client::Game::MJI::MJIManager* this, uint8 functionId) and it'll parse the function arguments. If this is a design we want, I can look into implementing it.

I really like the premise of this idea, but it raises a couple concerns:

This would only be useful so long as our offsets/structs/everything are correct. A naïve readthrough of an IDB that has had this applied to could cause notable confusion or annoyance, especially during post-update periods or with systems that we don't regularly verify for offset correctness. A dev exploring those systems with invalid type hints can very easily be led astray.
IDA is not particularly good at determining how many arguments something has. It generally gets things right, but it also definitely can get confused. How would this interact with cases where arguments aren't solidified? Or, worse, arguments change?

I'm not particularly enthusiastic about the idea of using comments for things, as there's no structure for comments nor do I think anyone bothers checking them for correctness. If we want to add a system like this, we should give it a proper set of fields/similar unless we don't really have a choice in the manner.

I agree with your concerns, don't have much commentary to add to them, but one more problem I've noticed: my IDB has tripled (!!!) in size, from 500MB to 1.5GB.

^ That latest commit adds a proof of concept system for declaring argument types in data.yml. You can do this:
0x140B6EAF0: IsRecipeUnlocked(Client::Game::MJI::MJIManager* this, _ recipeId) -> bool
0x140B6EB20: IsPouchItemLocked(Client::Game::MJI::MJIManager* this, _ pouchItemId) -> bool
0x140B6CDF0: IsFunctionUnlocked(Client::Game::MJI::MJIManager* this, _ functionId) -> bool

The arguments (parenthesis and its contents) and the return type (arrow and its following content) are optional. Supply _ as a type to not replace it and use the current value. When replacing a type bigger than its original value, it is ignored to not cause Decompiler Chaos.
I would like to have a consistent list of types (byte, short, int, long, float, double, unsigned variants) so they can work between Ghidra and IDA, and I also want to deduplicate the code for return type and argument type (notably type parsing and pointer calculation).

Great idea in concept, but does not fit the structure of data.yml and could lead to issues if functions parameter calls changes


.top() in android sdk does not  work

50 is default top value.however if I put .top(500) , it does not work and query is still fetched for 50 from android native client.

Addressed in #101


Weather Parameters Typo in Documentation

Carla Version 0.9.6 (release)
OS: Ubuntu 18.04
Bug: Documentation claims you can set the cloud cover using the cloudiness member variable in the carla.WeatherParameters class, the actual API requires cloudyness.

I can confirm this. The PythonAPI documentation refers to cloudiness, which incidentally is the correct way to spell it.
https://carla.readthedocs.io/en/latest/python_api/#carla.WeatherParameters
So both
https://carla.readthedocs.io/en/latest/python_api_tutorial/#changing-the-weather
and the API itself should be changed.


#2498 try different entry points method

Description
Fixes #2498
See https://github.com/click-contrib/click-plugins/issues/31#issuecomment-748349583
Type of change
Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s. If necessary, also add to the list of breaking changes.

[ ] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[x] Bug fix (non-breaking change which fixes an issue)

Key checklist:

[ ] No style issues: $ flake8
[ ] All tests pass: $ python run-tests.py --unit
[ ] The documentation builds: $ cd docs and then $ make clean; make html

You can run all three at once, using $ python run-tests.py --quick.
Further checks:

[ ] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works

How are test passing when this doesn’t add entrypoints as a dependency?


[stable/grafana] Namespace override

What this PR does / why we need it:
It adds a namespaceOverride parameter, which allows this chart to be used in cases where templating is used across namespaces. By providing this parameter, the resulting templating will output to the specified namespace.
Special notes for your reviewer:
This change is needed for implementing https://github.com/helm/charts/issues/18837 and inspired by https://github.com/helm/charts/pull/15202
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

Mentioning the fine people taking care of this chart: @zanhsieh @rtluckie @maorfr

/assign @zanhsieh

@arruzk
Would you mind to fix lint-charts error before we step further?
[ERROR] templates/: render error in "grafana/templates/deployment.yaml": template: grafana/templates/deployment.yaml:39:44: executing "grafana/templates/deployment.yaml" at <include (print $.Template.BasePath "/dashboards-json-configmap.yaml") .>: error calling include: template: grafana/templates/_helpers.tpl:57:16: executing "grafana.namespace" at <.Values.namespaceOverride>: nil pointer evaluating interface {}.namespaceOverride

@zanhsieh I fixed the problem. Thank you!

/ok-to-test

/retest

/test pull-charts-e2e

@arruzk
Could you rebase the chart version to latest please? Thank you.

/test pull-charts-e2e

/lgtm


Switched AzureStaticWebApp to new swa client api and added custom build variable functionality

Task name: AzureStaticWebAppV0
Description: Switched to the new Static Web Apps client api and added custom build variable functionality
Documentation changes required: N
Added unit tests: N
Attached related issue: Y Azure/static-web-apps#392
Checklist:

[x] Task version was bumped
[x] Checked that applied changes work as expected

@anatolybolshakov can you please help with the stuck PR checks

@bishal-pdMSFT @anatolybolshakov Do we have any updates, please?
CC: @azooinmyluggage @kmkumaran


Enhancements to docs and test

This adds some docs a test and updates the readme.

Thx for the review. I fixed the noted things.


Add App Search as manageable resource

Overview
Create terraform resource/s to replicate the App Search abstraction
Context
The App Search abstraction is very effective in getting new users building powerful autoscaling search functionality. The console driven approach is great when prototyping but is has its dangers as replicating resources is a manual process.
We are building a web app and want to be able to innovate with search as the technology develops, particularly the web crawler functionality. As we progress through dev, test, stage, prod we want to be able to reproduce immutable resources accurately and rollback instantly.
We could probably do this (?) if we build our own cluster from available terraform resources but this implies a learning curve starting from zero and loses the benefit of the App Search abstraction for us, and presumably for elastic.co in terms of making it easy for new users to adopt the technology?
We need an IaaC approach that offers the power and the speed of the abstraction with the ability to reproduce the exact tuning of the resource. I am specifically thinking of synonyms, weights, bias etc. Autoscaling is a big attraction when starting out.
Environment
Currently we are building out stacks with Terraform through a GitHub Action workflow and want the ability to control all of our app resources in the same workflow.

hi @Chanonry,
AppSearch as an application has been superseded by Enterprise Search, which we fully support in the Terraform provider, I'm not sure I understand if there are any blockers to using the Terraform provider as is with an enterprise_search resource set.

@Chanonry , If  I understand you correctly, what you're looking for is a terraform provider that configures appsearch / ent. search itself (like crawlers, etc), as code.
Is such a depiction correct?

Yes, spot on

On 5 May 2021, at 12:48, Omer Kushmaro @.***> wrote:
@Chanonry https://github.com/Chanonry , If I understand you correctly, what you're looking for is a terraform provider that configures appsearch / ent. search itself (like crawlers, etc), as code.
Is such a depiction correct?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/elastic/terraform-provider-ec/issues/302#issuecomment-832624759, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE7I4HKEKEJPQDNXW6SZLW3TMEWCFANCNFSM43MDTTCQ.

Superseded ? Didn’t I watch a webinar a couple of weeks ago on the “new” service ?? Very good it was too.
I raised a query in the forum on whether I could create an AppSearch stack at that level of abstraction with IaaC plus use IaaC to capture synonyms, web crawling, weights etc etc. The suggestion was that there this was possible in part not full. so I should raise a feature request.  I am not sure what the position is, I like the product and want to get going as fast and as productively as possible. Struck me that would be a common view.
What I want to avoid is having to regress to a much lower level of abstraction to build a stack akin to what I can create with AppSearch through the console, ie I want to get going without having to learn how to expertly configure the stack etc.
If that is currently possible, can you point me at the resources which help me set it up?  Maybe that’s all it needs, dunno.
As I said the AppSearch abstraction is attractive as it takes away much of the learning curve for new customers like me and gets us into your eco system. If we want to get clever later on we can, but the initial barrier to entry has gone benefiting us both.

On 5 May 2021, at 05:38, Marc Lopez Rubio @.***> wrote:
hi @Chanonry https://github.com/Chanonry,
AppSearch as an application has been superseded by Enterprise Search, which we fully support in the Terraform provider, I'm not sure I understand if there are any blockers to using the Terraform provider as is with an enterprise_search resource set.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/elastic/terraform-provider-ec/issues/302#issuecomment-832405673, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE7I4HKKYYI4VUOFM53SEK3TMDDSXANCNFSM43MDTTCQ.


Fix extension registration order.

Fixes https://github.com/godotengine/godot/issues/50031. This is the second patch, the first one broke GDExtensions and had to be reverted.
I have tested this with the demo extension we have under godot-cpp.

adf80b4 works fine for me: I can load GDExtensions.

Might be good to check how #58350 might affect this.

#58350 changes a small issue with index usage at https://github.com/godotengine/godot/blob/91a57b5b6a567ff10e30f4c0f1385c78e39ca284/core/extension/native_extension_manager.cpp#L49-L51 and https://github.com/godotengine/godot/blob/91a57b5b6a567ff10e30f4c0f1385c78e39ca284/core/extension/native_extension_manager.cpp#L73-L75
Shouldn't conflict.

#58350 changes a small issue with index usage at
https://github.com/godotengine/godot/blob/91a57b5b6a567ff10e30f4c0f1385c78e39ca284/core/extension/native_extension_manager.cpp#L49-L51
and
https://github.com/godotengine/godot/blob/91a57b5b6a567ff10e30f4c0f1385c78e39ca284/core/extension/native_extension_manager.cpp#L73-L75
Shouldn't conflict.

I also tested the demo with both PR's merged and it works.

Tested with the godot-cpp test project (with headers updated to latest upstream master) and it seems to work fine (I did get a GDScript crash on first import but that's not related to the changes in this PR).
CC @mhilbrunner @groud would be good if you can test on your GDExtension project too.

I can confirm this fixes the issue and does not seem to impact any of our GDExtension projects negatively. Found no issues. :)

I can confirm the fix works on the VideoStream extension I'm working on.

Thanks!


ask for advice

Hello, I'm very interested in your work. What a wonderful display !!
Personally, Fourier seems to imitate the process of human painting, and all the examples you show can be drawn with one stroke.
I would like to ask if Fourier supports multiple strokes.
Suppose there are two butterflies in the original PNG image, or there are multiple paths in my SVG file,
I want to draw two butterflies with two strokes, because if one stroke is used, there will be a big jump between the two targets，
And even if there is only one target or path, I also want to use multiple strokes in order to simplify and not cause a lot of jumping details,
So in general, my question is how to control Fourier drawing so that it can support user-defined stroke number？
I'm looking forward to your answer，Thank you very much。

Thak you very much for your comment.
Answering your question, the code I wrote is based on this video and there just one path (function) is approximated. If you have multiple paths in your SVG, what you must done is approximate each of the paths as a separate function defined between 0-1 local values. The tricky part may appear when drawing the full path, so the approximated functions may be organized previously.
In case, you want to separate a target path, you have two options: 1) manually divide the path with a SVG editor; 2) properly define the integration limits when obtaining the coefficients of the approximated functions.
I hope my comment was useful, let me know if I can help you.

Thak you very much for your comment.
Answering your question, the code I wrote is based on this video and there just one path (function) is approximated. If you have multiple paths in your SVG, what you must done is approximate each of the paths as a separate function defined between 0-1 local values. The tricky part may appear when drawing the full path, so the approximated functions may be organized previously.
In case, you want to separate a target path, you have two options: 1) manually divide the path with a SVG editor; 2) properly define the integration limits when obtaining the coefficients of the approximated functions.
I hope my comment was useful, let me know if I can help you.

Thak you very much for your comment.
Answering your question, the code I wrote is based on this video and there just one path (function) is approximated. If you have multiple paths in your SVG, what you must done is approximate each of the paths as a separate function defined between 0-1 local values. The tricky part may appear when drawing the full path, so the approximated functions may be organized previously.
In case, you want to separate a target path, you have two options: 1) manually divide the path with a SVG editor; 2) properly define the integration limits when obtaining the coefficients of the approximated functions.
I hope my comment was useful, let me know if I can help you.

Thank you for your timely and useful reply。
If my SVG has several paths, I will calculate the Fourier series for each path separately, and then draw each path separately on the same canvas. In this way, I can draw all paths, and there is no redundant connection between the paths.
What I want to do is to use Canny edge detection algorithm to simplify the edge of the original PNG or JPG image, and then get the coordinate points of each path, calculate each Fourier series separately, and draw it on the same canvas, as just mentioned

Thak you very much for your comment.
Answering your question, the code I wrote is based on this video and there just one path (function) is approximated. If you have multiple paths in your SVG, what you must done is approximate each of the paths as a separate function defined between 0-1 local values. The tricky part may appear when drawing the full path, so the approximated functions may be organized previously.
In case, you want to separate a target path, you have two options: 1) manually divide the path with a SVG editor; 2) properly define the integration limits when obtaining the coefficients of the approximated functions.
I hope my comment was useful, let me know if I can help you.

Thank you for your timely and useful reply。
If my SVG has several paths, I will calculate the Fourier series for each path separately, and then draw each path separately on the same canvas. In this way, I can draw all paths, and there is no redundant connection between the paths.
What I want to do is to use Canny edge detection algorithm to simplify the edge of the original PNG or JPG image, and then get the coordinate points of each path, calculate each Fourier series separately, and draw it on the same canvas, as just mentioned

Thak you very much for your comment.
Answering your question, the code I wrote is based on this video and there just one path (function) is approximated. If you have multiple paths in your SVG, what you must done is approximate each of the paths as a separate function defined between 0-1 local values. The tricky part may appear when drawing the full path, so the approximated functions may be organized previously.
In case, you want to separate a target path, you have two options: 1) manually divide the path with a SVG editor; 2) properly define the integration limits when obtaining the coefficients of the approximated functions.
I hope my comment was useful, let me know if I can help you.

Thank you for your timely and useful reply。
If my SVG has several paths, I will calculate the Fourier series for each path separately, and then draw each path separately on the same canvas. In this way, can I draw all paths, and there is no redundant connection between the paths.
What I want to do is to use Canny edge detection algorithm to simplify the edge of the original PNG or JPG image, and then get the coordinate points of each path, calculate each Fourier series separately, and draw it on the same canvas, as just mentioned

Thak you very much for your comment.
Answering your question, the code I wrote is based on this video and there just one path (function) is approximated. If you have multiple paths in your SVG, what you must done is approximate each of the paths as a separate function defined between 0-1 local values. The tricky part may appear when drawing the full path, so the approximated functions may be organized previously.
In case, you want to separate a target path, you have two options: 1) manually divide the path with a SVG editor; 2) properly define the integration limits when obtaining the coefficients of the approximated functions.
I hope my comment was useful, let me know if I can help you.

Thank you for your timely and useful reply。
If my SVG has several paths, I will calculate the Fourier series for each path separately, and then draw each path separately on the same canvas. In this way, can I draw all paths, and there is no redundant connection between the paths.
What I want to do is to use Canny edge detection algorithm to simplify the edge of the original PNG or JPG image, and then get the coordinate points of each path, calculate each Fourier series separately, and draw it on the same canvas, as just mentioned

To accomplish a result similar that I have in the repository, you may need a analytical representation of the path and I am not sure if you can get those with the Canny Edge Detection algorithm.  If the discretization of the edge is very fine, maybe you are able to approximate the function properly and obtained suitable coefficients.
Regards,

To accomplish a result similar that I have in the repository, you may need a analytical representation of the path and I am not sure if you can get those with the Canny Edge Detection algorithm.  If the discretization of the edge is very fine, maybe you are able to approximate the function properly and obtained suitable coefficients.
Regards,


Boxes that dynamically highlights code blocks background colour?

Would it be possible to have boxes that dynamically highlights code blocks background colour?
Something like this perhaps...
Just thought it'd be cool.

https://github.com/Microsoft/vscode/issues/31220


Request - Different icon for folders

Hi Christian,
Can you please change the prefix icon of folders in EmulationStation?
The current star icon for folders is exactly the same as a for favorite game.
In my case, I have the multi-disk Amiga games in each folder, like so (example game Gods)
/Amiga/Gods/Gods.m3u              <= Created by me so that UAE4ARM can swap disks
/Amiga/Gods/Gods(Disk 1 of 2).adf
/Amiga/Gods/Gods(Disk 2 of 2).adf
EmulationStation shows the Gods folder with a star on the left because it's a folder with multiple files. BUT the the star is also used to represent Favorites!
I have a ton of multi-disk amiga games (was common) and all have stars.
I have no idea of those entries which ones are Folders or Favorites.
Thanks

I'll review this and see what's involved to accommodate that change.

This has been accommodated with the update released today.  Folders are now designated with a right arrow.

Thank you!


fix(*) cleanup un-needed marshall_event handlers on schemas

Since the removal of Serf, marshall_event is not used anymore by any
schema (invalidations uses cache_key or custom invalidation handlers
instead).
This just gets rid of unused code.

Maybe chore(*) instead of fix(*)?

chore: Changes related to the build process, the dependencies, or auxiliary tools and libraries such as LuaRocks, Travis-ci, etc...

This is concerning the code itself instead. I agree the current scope is not ideal, but I could not think of a better one except cleanup(*) remove undeeded ..., which we don't really have...

Maybe we could expand the definition to:
chore: Maintenance changes, related to code cleaning that isn't considered part of a
refactor, build process updates, dependency bumps, or auxiliary tools and libraries
changes such as LuaRocks, Travis-ci, etc...

?

Seems like a reasonable adjustment.


Ability to set node from structure

Edit (2018/09/16): This was changed from .fill to .set for version 15.
Basically, this would allow you to set multiple properties at once:
functionDeclaration.set({
    isAsync: true,
    hasExportKeyword: true
});

It would be useful with #45 too:
functionDeclaration2.set(functionDeclaration1.getStructure());

Edit: Mostly all done... still todo:

[x] GetAccessorDeclaration
[x] SetAccessorDeclaration
[ ] DecoratorDeclaration
[ ] ExportDeclaration
[ ] ExportSpecifier
[ ] ImportDeclaration
[ ] ImportSpecifier
[ ] HeritageClause (maybe)
[ ] TypeParameterDeclaration
[x] VariableDeclaration
[x] VariableDeclarationList
[x] VariableStatement

Run ts-node scripts/verification/main ensure-classes-implement-structure-methods to see the remaining work on this issue.


Layer mask blending fix

The PR updates the anim layer mask blending to be consistent with that used by the anim evaluator for animation transitions and blend trees.
I confirm I have read the contributing guidelines and signed the Contributor License Agreement.

can you please replace this.value.fill(0, 0, 3).fill(1, 3); by 4 assignments


sqrt(norm(x))

Taking the sqrt and the norm looks like a bug.
https://github.com/andrenarchy/krypy/blob/master/krypy/utils.py#L238

To me it looks fine. It's \sqrt{||<x,y>||} where the norm could actually be replaced by abs(). I guess I took that shortcut so I don't have to deal with array shapes and always get a scalar?


Customized negative values are ignored

Any custom values in theme/extend config for negative margins (as per https://tailwindcss.com/docs/margin#negative-values) is ignored - for example, using the following config:
theme: {
  extend: {
    margin: {
      foo: '10px',
      '-foo': '-20px'
    }
  }
}

Will generate the following rules (ignoring the positional margins for now):
.m-foo {
  margin: 10px;
}
.-m-foo {
  margin: -10px;
}

Instead of:
.m-foo {
  margin: 10px;
}
.-m-foo {
  margin: -20px;
}

-m-{value} is how you do negative values. In your case, maybe you should try m--foo or something.

This is something I'd like help with too.
In my case, I'm using variables and need custom negative spacing.
theme: {
  extend: {
    spacing: {
      fluid: 'var(--fluid)',
      '-fluid': 'calc(0px - var(--fluid))'
    }
  }
}


Use SyntheticEvent in React event docs.

This is more readable and accurate(i.e. Flow doesn't give a type error) as demonstrated here.

This is no longer necessary per the new docs: https://flow.org/en/docs/react/events/


support the case when no resource type set names are specified in config

We need this when we start supporting inventory metadata config files served by the server.

LGTM assuming we still need all of the remote mgmt support.


variable name missmatch

Github action is expecting :

TEST_APP_ID, does not exist in the doc, and I think this is test_account_client_id in config.yaml ?
TEST_WORKSPACE_APP_ID, the doc says it is WORKSPACE_API_CLIENT_ID in the yaml, but that is not there, must be api_client_id ?
TEST_WORKSPACE_APP_SECRET, the docs says it is WORKSPACE_API_CLIENT_SECRET in the yaml, but it does not exist,  must be api_client_secret ?

Same experience here.
@chboudry Did you attempt a workaround?

@SvenAelterman  If you are using an automation account, the nTEST_APP_ID is redundant. Same for TEST_WORKSPACE_APP_ID and TEST_WORKSPACE_APP_SECRET. We have an open issue to make it a non required params.  For now you can put any dummy value inside (non empty value).

@LizaShak was an issue created for this? It has come up a couple of times recently. Thanks.


write 2 different files (question)

Hello, is it possible that with 1 pump read from the source once, and write to 2 output files??, the first one will be a .csv file and the second one will be an excel...
this way i wouldnt have to read again the source.. if that makes sense..
Thanks!.

Hi, you can do it with three pumps:
{ mixin: { CsvWriterMixin, ExcelWriterMixin }, Pump } = require 'datapumps'
Promise = require 'bluebird'

(sourceReader = new Pump())
  .buffers
    csvReport: sourceReader.createBuffer()
    xlsxReport: sourceReader.createBuffer()
  .process (item) ->
    Promise.all [
      @buffer('csvReport').writeAsync item
      @buffer('xlsxReport').writeAsync item
    ]

(csvWriter = new Pump())
  .from sourceReader.buffer 'csvReport'
  ...

(xlsxWriter = new Pump())
  .from sourceReader.buffer 'xlsxReport'
  ...

Promise.all [ sourceReader.run(), csvWriter.run(), xlsxWriter.run() ]

In the sourceReader pump, I've created two output buffers, and wrote the same data to both of them in process. Then I just created two other pumps reading from those buffers.

Ohhh, so I'm guessing that in sourceReader pump I specify  the "from" to let's say a mysql db, correct?
Thanks for the clarification

Yes, that's right.

Hey, so ive made my example :).
http://pastebin.com/vDVnV9EP
but when i run it some items goes to one buffer while others go to the other..
so i end up with some items in the csv file and the rest in the xlsx file :/
is there something im missing there?

Hi, I was debugging the code for an hour after I've found out, that both csvWriter and xlsxWriter reads from csvReport buffer :). Just change csvReport to xlsxReport in line 44 and it should be ok.

Wow thats embarrasing, sorry for wasting your time..
but thank you!

No problem, datapumps should have reported this.
Your welcome.

Hello, me again.. so im trying to filter data inside the process method using a single pump
http://pastebin.com/U9H2SxqP
like that, but im getting
Error in pump (root): Error: .process() did not return a Promise
isnt pump.writeRow overrides the need of a promise?

No it does not. In fact, pump.writeRow() returns a promise, which can be returned in process.
My recommendation is to make orderDtmfs return a promise (its uses a cb now, I assume), and then you could do something like this:
.process(function(account) {
  return orderDtmfs(account)
    .then(function(dtmfs) {
      return pump.writeRow([ account.cdr_id, account.account_id, dtmfs ]);
    });


Get transactionReceipt for iOS old receipt validation service

Hi,
In our app, the back-end validates the transaction receipt for iOS. The back-end unfortunately wasn't updated for several years, so it can only handle the transactionReceipt from SKPaymentTransaction. Is there a way to get that property from the PurchaseAsync method?
Thanks in advance!
regards,
Annelies

When you implement IInAppBillingVerifyPurchase  and pass it in, you will get
var receiptUrl = NSData.FromUrl(NSBundle.MainBundle.AppStoreReceiptUrl);
string receipt = receiptUrl.GetBase64EncodedString(NSDataBase64EncodingOptions.None);
        return verifyPurchase.VerifyPurchase(receipt, string.Empty, productId, transactionId);

Hi James,
Thanks for your response. I added the Verify purchase method in my code and see that it gives me a receipt with all transactions ever done.
I will need the deprecated transactionReceipt of skpaymenttransaction. https://developer.apple.com/reference/storekit/skpaymenttransaction/1617722-transactionreceipt?language=objc
Is there any way to get that one? Thanks in advance!
regards,
Annelies

You should probably fork the repo and customize it to your liking as it is deprecated now so I will not add it in.

Ok, thanks. Now i know what to do.


Update WebP connection to support CocoaPods+Frameworks

Love this framework! This makes it possible to use WebP with PINRemoteImage with the use_frameworks! option. We have to use a preprocessor definition because at compile time we won't have libwebp headers accessible.
Let me know if there's anything that needs tweaking. I tested that this worked in the use_frameworks! case.

Thanks for doing this! And sorry for the long delay in looking at it. Can you take a look at why the webp tests are failing? I assume it has to do with the test projects spec not having webp?

use_frameworks! doesn't work with iOS 7

Sorry for the delay here, I'm totally swamped and I let this slip through the cracks. I'm cutting a release right now, but I'll investigate the failing tests this week.

Hey @Adlai-Holler any chance you can look at this soon?

The party's moved to #129


docs: fix API ref md links downgrading api-documenter

Issue or need
Some API reference links in documentation were not working. They displayed the markdown syntax [link text](link URL), but were not processsed as links.
After reviewing the generated Markdown files and latest releases from api-documenter and mkdocs-material, seems that api-documenter recently started to use HTML tables instead of Markdown tables. And seems mkdocs-material doesn't handle that well.
Proposed changes
Downgrade api-documenter to a previous version where files were pure Markdown. Will see how to solve it better.

Quick reminders

🤝 I will follow Code of Conduct
✅ No existing pull request already does almost same changes
👁 Contributing docs are something I've taken a look at
📝 Commit messages convention has been followed
💬 TSDoc comments have been added or updated indicating API visibility if API surface has changed.
🧪 Tests have been added if needed. For instance, if adding new features or fixing a bug. Or removed if removing features.
⚙ API Report has been updated if API surface is altered.

#485  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @davidlj95 and the rest of your teammates on  Graphite

Seems that HTML tables generated from API Documenter is being discusses though:

https://github.com/microsoft/rushstack/pull/4578
https://github.com/microsoft/rushstack/issues/4586


Duplicate attachments displayed in the reporter when multiple formats are specified

Environment (please complete the following information):

protractor-multiple-cucumber-html-reporter: 1.8.1
Node.js version: 8.11.1
NPM version: 5.10.0
Platform name and version: Windows 7
protractor version: 5.3.1
Cucucmber version: 5.1.0
protractor-cucumber-framework version: 6.1.2

Config of protractor + protractor-multiple-cucumber-html-reporter
const cucumObj = {
    frameworkPath: require.resolve('protractor-cucumber-framework'),
    cucumberOpts: Object.assign({
        require: _require,
        profile: false,
        'no-source': true,
        format: [
            "./logger/protractor-html-screenshot-reporter/CucumberFormatter.js", // Custom formatter
            `json:${TEST_SUMMARY_DIR}/results/results.json`,
        ],
        print: () => { }
    }, config.protractorConfig.cucumberOpts),
    plugins: [{
            package: 'protractor-multiple-cucumber-html-reporter-plugin',
            options: {
                automaticallyGenerateReport: true,
                removeExistingJsonReportFile: true
            }
        }]
};

Describe the bug
Duplicate attachments displayed in the reporter. Here is what I am trying to attach,
this.attach(JSON.stringify({"url": "www.google.com"}), 'application/json');
And I'm expecting the output should be (in the reporter),
[
  "\"{\\\"url\\\":\\\"www.google.com\\\"}\""
]

But I'm getting
[
  "\"{\\\"url\\\":\\\"www.google.com\\\"}\"",
  "\"{\\\"url\\\":\\\"www.google.com\\\"}\""
]

Too much of " and \ are annoying anyway. But I don't care it now
When I remove my custom formatter, I do see the expected result.
[
  "\"{\\\"url\\\":\\\"www.google.com\\\"}\""
]

Apologize. I didn't notice that I was creating two instances of EventDataCollector which was causing this issue. So please close this issue.

Thnx for the feedback


Run notebooks in CI

We'd like to setup CI tests for the example notebooks in ecoscope so that these are tested against any core feature changes.
Can explore using:  nbmake pytest plugin

@davidovichmarcos based on our sync, here's the draft tests we were working on together
import pathlib
from dataclasses import dataclass

import pytest

NB_DIR = pathlib.Path(__file__).parent.parent / "doc" / "source" / "notebooks"

KNOWN_ERRORS_REGEXES = {  # This is basically a GitHub ticket queue
    "EarthRanger_IO.ipynb": "expected Series",
}

@dataclass
class Notebook:
    path: pathlib.Path
    raises: bool = False
    raises_match: str = None

ALL_NOTEBOOKS = [
    Notebook(
        path=p,
        raises=False if not p.name in KNOWN_ERRORS_REGEXES else True,
        raises_match=KNOWN_ERRORS_REGEXES.get(p.name),
    )
    for p in NB_DIR.rglob("*.ipynb")
]

@pytest.mark.parametrize(
    "notebook",
    ALL_NOTEBOOKS,
    ids=[nb.name for nb in ALL_NOTEBOOKS],
)
def test_notebooks(notebook: Notebook):
    if notebook.raises:
        # Option 1: if papermill run in "raise errors" mode
        with pytest.raises(Exception, match=notebook.raises_match):
            # First prove to ourselves that the notebook is still broken
            # papermill.execute_notebook(str(notebook), str(notebook.with_name("output.ipynb")))
            # then xfail when we know it is
            pytest.xfail(f"Notebook {notebook.path} is known to fail with error {notebook.raises_match}")
        # Option 2: if papermill run in "capture errors" mode
        # error = papermill.execute_notebook(str(notebook), str(notebook.with_name("output.ipynb")))
        # assert notebook.expected_error_cell_number == error.cell_number
        # assert notebook_raises_match == error.text
        # pytest.xfail(f"Notebook {notebook.path} is known to fail with error {notebook.raises_match}")
    # papermill.execute_notebook(str(notebook), str(notebook.with_name("output.ipynb")))

@davidovichmarcos here is the blog post we were discussing https://blog.ganssle.io/articles/2021/11/pytest-xfail.html
The idea of linking bug ticket numbers as the xfail reason (discussed there) seems really awesome!


Which Direct2D function can replace the GdipLoadImageFromStream function

Which Direct2D function can replace the GdipLoadImageFromStream function?

This project is for DirectX Tool Kit for DX12. I can't really provide any help with Direct2D beyond pointing you to Microsoft Docs and the original samples.
You should look at other community support sites for help.


Should Three be defined as a peer dependency?

Hi,
I'm testing three-orbitcontrols-ts, and having problems zooming.
The scene is set up with THREE.PerspectiveCamera, and when dollyIn/Out is called this.object instanceof THREE.PerspectiveCamera is false.
When evaluating this.object.constructor it points to a version of Three pulled inn by orbitcontrols, while THREE.PerspectiveCamera points to the main version.
Should Three be a peer-dependecy so that a separate version is not included?

This is basically the same issue as issue #1 and should be fixed if #4 is released.


add pre-push check

Whatever travis is running to validate a PR should be run prior to pushing to the git remote.

@patflynn what do you mean?

a presubmit check using a git hook to validate as much as we can do quickly.

Currently, mvn clean install is what Travis uses to validate the build. We should all have a habit of building before pushing a change, but I'm weary of putting in a git hook for this. A build can take a minute or two, and when I do git push, I don't really want to wait at that point.

I agree. It should be anything we can do that's fast. So possibly only
build and lint. Or build and lint and fast tests.
On Mon, Jul 25, 2016 at 11:11 AM, meltsufin<EMAIL_ADDRESS>wrote:

Currently, mvn clean install is what Travis uses to validate the build.
We should all have a habit of building before pushing a change, but I'm
weary of putting in a git hook for this. A build can take a minute or two,
and when I do git push, I don't really want to wait at that point.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/app-tools-lib-for-java/issues/114#issuecomment-234982531,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHf5HeP5wK-9cARtqNHLRtQRfFg2mMbUks5qZNIJgaJpZM4IyFVo
.


Make web sys version of webgl example

Hi, I ported the webgl example to web-sys. It would be great to get this merged in. Please let me know if you have any suggestions.

Thanks for catching those issues. I hopefully have resolved them now.

Thanks!!


making changes to support multiple templates based on form types

What and why?
How to test?
Trello
https://trello.com/c/A8u90DZF/1344-make-changes-to-case-service-to-pass-form-type-as-classifiers-while-giving-a-call-to-the-notify-gateway

/deploy sinhaa

/deploy sinhaa

/deploy sinhaa


if i use custom wrapper, how to set default form class?

if set custom wrapper && set default form class in custom wrapper
then form class are set default form class in custom wrapper
Current version
_form.html.erb
<%= simple_form_for SOMEPATH, html: { class: "form-horizontal" }, wrapper: :horizontal do |f| %>

then rendered
<form class="simple_form form-horizontal">

currently manual set form class.
how about this?
_form.html.erb
<%= simple_form_for SOMEPATH, wrapper: :horizontal do |f| %>

simple_form configure file
config.wrappers :horizontal, class: "form-group" do |b|
  b.default_form_class "form-horizontal"
end

then rendered
<form class="simple_form form-horizontal">

i look for this feature, but i could not found it.

Unfortunately the form class can only be set globally, not per-wrapper, and that's because wrappers are associated with each input, not with the form itself (the ability to set a wrapper for the entire form is an addition to avoid having to pass it for each input).
You have the global form_class and default_form_class options to set on your Simple Form initializer, they apply to all forms. If you want a custom class to be passed to specific forms, you could come up with a helper of your own, like this:
def horizontal_simple_form_for(path, options = {}, &block)
  options = options.deep_merge(html: { class: 'form-horizontal' }, wrapper: :horizontal)
  simple_form_for(path, options, &block)
end

(totally untested).

aha! i got it.
Thanks @carlosantoniodasilva ;)


docker-versioning ?

maybe helpful to get versions on the docker-images like frigate does?
i now want to pull 1.0, but hard to see the versioning here https://github.com/jakowenko/double-take#docker-compose
maybe there is a dockercompose convention to get specific versions?
??   image: jakowenko/double-take:1.0.0 ?
thats what i have:
services:
  double-take:
    container_name: double-take
    image: jakowenko/double-take
    #image: jakowenko/double-take:0.10.2
    #image: jakowenko/double-take:beta-1.0.0
    #image: jakowenko/double-take:beta

    #image: blakeblackshear/frigate:stable-amd64
    #beta
    #image: blakeblackshear/frigate:0.9.0-beta2-amd64
    #image: blakeblackshear/frigate:0.9.0-rc1-amd64
    # rc
    #image: blakeblackshear/frigate:0.9.0-rc2-amd64
    #image: blakeblackshear/frigate:0.9.0-rc3-amd64
    image: blakeblackshear/frigate:0.9.0-rc4-amd64

image: jakowenko/double-take:latest

Should work

Hey, as ragenhe pointed out, just use jakowenko/double-take:latest and that will also be the most recent stable version. The method you have above is if you'd want to go back to a specific version.

that was my question: what version-number-schema do i have to use?
where to lookup which number works?
does this work? how to proof/lookup?
image: jakowenko/double-take:beta-1.0.0

Those are more than likely images that were being used for testing.
:latest will work until you want to try an available beta, the :beta would be used.

that was my question: what version-number-schema do i have to use?
where to lookup which number works?
does this work? how to proof/lookup?
image: jakowenko/double-take:beta-1.0.0

All the available tags are listed here.

No problem! You shouldn't need an account to see the tags, try this link. It worked for me without being logged in.

All the tags are gone.
wanted to explicitly pull ver 1.2.
but there are not tags anymore.
not on github-landing page nor on docker... 😢

All the tags are gone.
wanted to explicitly pull ver 1.2.
but there are not tags anymore.
not on github-landing page nor on docker... 😢

Looks like an issue with Docker Hub right now.


Ability to specify strategies for agents in EvvoIslandBuilder

Is your feature request related to a problem? Please describe.
There is a way to assign strategies to agents, but right now, there's no way of giving anything other than the default strategy to an agent when constructing islands through the builder.
Describe the solution you'd like
Each of the methods on EvvoIslandBuilder that result in an agent being created (so, addCreator, addModifier, and addDeletor should have an optional strategy parameter.
Describe alternatives you've considered
We could create a single case class that takes a Function and an AgentStrategy and pass those in to the builder, but that would add complexity to the API for making Islands, which is not worth it. Islands can keep track of the data that they need to build the agents they are building.
Additional context
This won't be that helpful until the data passed to the strategy is more comprehensive, but that's a separate PR.

Actually, I've been thinking about this more - maybe strategy should also be a method on Agents, which defaults to  "always run"? @drassaby @rachlin let me know which design choice you think is beteter

Looks like this can be closed now since agents can be assigned strategies when they're created, right @julian-zucker?

Yeah, although the built-ins don't support overriding the strategies. I'll make a new issue for that


trace exporter for elasticsearch

trace exporter for elasticsearch

@bogdandrutu I keep seeing an error in travis-ci about my javadocs can't seem to replicate it locally

@bogdandrutu I missed pushing one of the changes in this particular PR. Let me close it and re submit.  So you help me with the actual error from javadocs. Travis-ci doesn't show the exact error.


services appear twice

Except k3s-service, the other services appear twice in the return value of the rc-update command.
Is this normal?

This is still happening in v0.2.0 and seems weird.  Marking as a bug.

Accidentally discovered that this is caused by the same init scripts existing under /etc/init.d and /usr/etc/init.d. Not sure what would be the best approach for fixing it, but a quick test with not copying the init scripts to /etc and linking from /etc/runlevels/ to /usr/etc/init.d seemed to boot okay.