signal link progressbar

trying to do a NotifySignal from a float to ProgressBar.value but it doesn't seem to work, placing the event somewhere else does assign de value properly but just setting ProgressBar.value=model.Percentage does nothing

Hi,
die you get a QML error that Percentage is no property?
QML.Net converts the property names to camelCase because this is the QML standard.
So maybe just using modwl.percentage solves your problem?

i'm not 100% sure what was it, i did use proper casing, however it is working now, thanks


Add a new endpoint to get a stream by id.

Also added a commit to update the API_FEATURE_LEVEL which was missed in dfd7902.
Fixes: #22082

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Merged with these changes:
diff --git a/templates/zerver/api/changelog.md b/templates/zerver/api/changelog.md
index af95588e89..d061c10825 100644
--- a/templates/zerver/api/changelog.md
+++ b/templates/zerver/api/changelog.md
@@ -23,7 +23,7 @@ format used by the Zulip server that they are interacting with.
 **Feature level 132**

 * [`GET /streams/{stream_id}`](/api/get-stream-by-id):
-  Added new endpoint to get a stream by id.
+  Added new endpoint to get a stream by ID.

 **Feature level 131**

diff --git a/zerver/openapi/zulip.yaml b/zerver/openapi/zulip.yaml
index cc50699b31..862fb7a40e 100644
--- a/zerver/openapi/zulip.yaml
+++ b/zerver/openapi/zulip.yaml
@@ -13488,7 +13488,7 @@ paths:
                         "result": "error",
                       }
                     description: |
-                      An example JSON response for when the stream id is not valid.
+                      An example JSON response for when the stream ID is not valid.
     delete:
       operationId: archive-stream
       summary: Archive a stream

We generally have a pattern of capitalizing ID; git grep 'Invalid.*id' suggests the "invalid stream id" error message is one of the few exceptions here. Probably worth changing the capitalization on that error message if it's quick.

Merged! Thanks for catching the API feature level bug -- we should try to avoid introducing that class of mistake when we can, especially for breaking API changes, since it will mean the mobile apps (for example) may incorrectly handle server versions from Git commits in between.


Problem adding via SPM

Describe the bug

I am getting this problem when I add a Package to a Project via SPM. How can I fix this?
Steps To Reproduce
Problem adding via SPM

Expected behavior
Problem adding via SPM
Swift Liveness Version
1.3.3
Xcode version
15
Relevant log output
No response
Is this a regression?
No
Regression additional context
No response
OS Version
macOS 14.6.1
Device
iPhone 15 Pro Max
Specific to simulators
No response
Additional context
No response

Hi @behzod-rajabov, can you please check if your git configuration is set to force signatures for annotated tags? You can check that by running git config --get tag.forcesignannotated - if that returns true, you can disable it using git config --global tag.forcesignannotated false.
This setting requires annotated tags to be signed with GPG, and the tags for this repository are not signed.

Hi @mattcreaser This setting is returning me the "false". But the problem is still there.

@behzod-rajabov I was unable to reproduce this, so it seems like a local issue with your SPM cache.
I'd advice you to check if either of the following fixes the issue:

Delete the DerivedData folder
On Xcode, go to File -> Packages -> Reset Package Caches
Delete your SPM cacherm -rf ~/Library/Caches/org.swift.swiftpm
rm -rf ~/Library/org.swift.swiftpm

Hi @ruisebas I have done all the options you gave, but unfortunately the problem still occurs.

In the screenshot you shared, you can see it explicitly says it's attempting to fetch a cached version.
Maybe try removing the package from the list of Recenty Used?

Yes I tried all possible methods. I tried adding it to other projects as well. But the problem is the same.

@ruisebas Yes I tried all possible methods. I tried adding it to other projects as well. But the problem is the same.

I wiped everything on my end as well to be sure, but I still can't reproduce it.
Are you perhaps using SourceTree? There's a recent bug that causes issues with SPM because it adds the following to the ~/.gitconfig file:
[safe]
    bareRepository = explicit

So if you have those entries in your config file, please remove them and try again.

If that still doesn't work (or you don't have those entries), one last thing you could try is to run the proper swift package commands to clean up:
swift package purge-cache
swift package reset

Also make sure your Xcode is closed when you run them.

Yes @ruisebas , the problem is with SourceTree. Thanks for your help 👍🏼


About shadow quality

The following differences exist when the same model is displayed.
The code in this book displays the following

SaschaWillems' sample shows the following display.
https://github.com/SaschaWillems/Vulkan/tree/master/examples/shadowmappingcascade

This model was downloaded from.
https://vulkan.gpuinfo.org/downloads/vulkan_asset_pack_gltf.zip
The images were arranged as follows.

This gltf model contains textures, so I've added minimal changes to make it loadable.
ModelLoader.java
189Line

  ...
  ModelData.Material material = processMaterial(aiMaterial, texturesDir, flags);
  ...

331Line:

 private static ModelData.Material processMaterial(AIMaterial aiMaterial, String texturesDir, int flags) {
    try (MemoryStack stack = MemoryStack.stackPush()) {
        AIColor4D colour = AIColor4D.create();

        Vector4f diffuse = ModelData.Material.DEFAULT_COLOR;
        int result = aiGetMaterialColor(aiMaterial, AI_MATKEY_COLOR_DIFFUSE, aiTextureType_NONE, 0,
                colour);
        if (result == aiReturn_SUCCESS) {
            diffuse = new Vector4f(colour.r(), colour.g(), colour.b(), colour.a());
        }
        AIString aiTexturePath = AIString.calloc(stack);
        aiGetMaterialTexture(aiMaterial, aiTextureType_DIFFUSE, 0, aiTexturePath, (IntBuffer) null,
                null, null, null, null, null);
        String texturePath = aiTexturePath.dataString();
        if (texturePath != null && texturePath.length() > 0) {
            texturePath = texturesDir + File.separator + new File(texturePath).getName();
			if( texturePath.indexOf("*") != -1) {
				texturePath += "@@" + flags ;
			}
            diffuse = new Vector4f(0.0f, 0.0f, 0.0f, 0.0f);
        }

        AIString aiNormalMapPath = AIString.calloc(stack);
        Assimp.aiGetMaterialTexture(aiMaterial, aiTextureType_NORMALS, 0, aiNormalMapPath, (IntBuffer) null,
                null, null, null, null, null);
        String normalMapPath = aiNormalMapPath.dataString();
        if (normalMapPath != null && normalMapPath.length() > 0) {
            normalMapPath = texturesDir + File.separator + new File(normalMapPath).getName();
			if( normalMapPath.indexOf("*") != -1) {
				normalMapPath += "@@" + flags ;
			}
        }

        AIString aiMetallicRoughnessPath = AIString.calloc(stack);
        Assimp.aiGetMaterialTexture(aiMaterial, AI_MATKEY_GLTF_PBRMETALLICROUGHNESS_METALLICROUGHNESS_TEXTURE, 0, aiMetallicRoughnessPath, (IntBuffer) null,
                null, null, null, null, null);
        String metallicRoughnessPath = aiMetallicRoughnessPath.dataString();
        if (metallicRoughnessPath != null && metallicRoughnessPath.length() > 0) {
            metallicRoughnessPath = texturesDir + File.separator + new File(metallicRoughnessPath).getName();
			if( metallicRoughnessPath.indexOf("*") != -1) {
				metallicRoughnessPath += "@@" + flags ;
			}
        }
        ...

Texture.java

public Texture(Device device, String fileName, int imageFormat) {
	Logger.debug("Creating texture [{}]", fileName);
	recordedTransition = false;
	this.fileName = fileName;
	ByteBuffer buf;
	try (MemoryStack stack = MemoryStack.stackPush()) {
		IntBuffer w = stack.mallocInt(1);
		IntBuffer h = stack.mallocInt(1);
		IntBuffer channels = stack.mallocInt(1);
		int importMark = fileName.indexOf("*") ;
		if( importMark != -1) {
			String[] split = fileName.substring( importMark + 1, fileName.length()).split("@@") ;
			int	textureIndex = Integer.parseInt( split[0]) ;
			int	flags = Integer.parseInt( split[1]) ;
			fileName = new File( fileName).getParentFile().listFiles()[0].getAbsolutePath().replace( "" + File.separatorChar + "", "/") ;
			AIScene		aiScene = null ;
			try {
				aiScene = aiImportFile( fileName, flags) ;
				if( aiScene == null) {
					throw new RuntimeException("Error loading model [modelPath: " + fileName + "]");
				}
				PointerBuffer	mTextures = aiScene.mTextures() ;
				AITexture		aiTexture = AITexture.create( mTextures.get( textureIndex)) ;
				buf = stbi_load_from_memory( aiTexture.pcDataCompressed(), w, h, channels, 4) ;
				if( buf == null) {
					throw new RuntimeException("Image file [" + fileName + "] not loaded: " + stbi_failure_reason());
				}
			} finally {
				if( aiScene != null) {
					aiReleaseImport( aiScene) ;
				}
			}
		}
		else {
			buf = stbi_load( fileName, w, h, channels, 4) ;
			if( buf == null) {
				throw new RuntimeException("Image file [" + fileName + "] not loaded: " + stbi_failure_reason());
			}
		}
		setHasTransparencies(buf);
		width = w.get();
		height = h.get();
		mipLevels = (int) Math.floor(log2(Math.min(width, height))) + 1;

		createTextureResources(device, buf, imageFormat);
	}

	stbi_image_free(buf);
}

My guess is that this is because in the shadow map I do not add support for transparencies. Leafs may be just tetxures inside quads, which produce this lack of resilution. To fix that you should add a fragment shader to shadow render and pass material information. I will review this and probably add support for that in the future.

Sorry for the delay in contacting you.
In order to investigate the cause of this, I tried to create a shadow with forward rendering, in which case it appears correctly.
So, I stopped destroying fragments in the fragment shader when creating with forward rendering, and I got the same shadows as created with the geometry shaders.
I feel that this is because the geometry shader cannot discard fragments.
Here is the fragment shader when the shadows are created by forward rendering.
#version 450

layout (constant_id = 0) const int SHADOW_MAP_CASCADE_COUNT = 4 ;
layout (constant_id = 1) const int MAX_TEXTURES = 100 ;

layout(location = 0) in vec2 inTextCoords;
layout(location = 1) flat in uint inMatIdx;

struct Material {
    vec4 diffuseColor;
    int textureIdx;
    int normalMapIdx;
    int metalRoughMapIdx;
    float roughnessFactor;
    float metallicFactor;
};

layout (std430, set=1, binding=0) readonly buffer srcBuf {
    Material data[];
} materialsBuf;
layout(set = 2, binding = 0) uniform sampler2D textSampler[MAX_TEXTURES];

void main()
{
    Material material = materialsBuf.data[inMatIdx] ;

    float alpha = texture( textSampler[ material.textureIdx], inTextCoords).a;
    if (alpha < 0.5) {
        discard;
    }
}

If we do the following, the shadows will be the same as those created by the geometry shader.
    ...
    float alpha = texture( textSampler[ material.textureIdx], inTextCoords).a;
//    if (alpha < 0.5) {
//        discard;
//    }

I'ts been a long time, but I'm currently working on adding support for transparencies in shadows. The approach is what I mentioned, passing texture information while generating depth maps and discard fragments which are below a certain alpha threshold.


typedef struct hipFuncAttributes now C compatible.

Fixes #591.
Fixes #694.

@jeffdaily. Thanks for the patch. HIP development flow happens in the following way. PR targets master branch -> PR merged into master branch -> commits cherry-picked into release branch.
I tried changing the base branch for this PR, but it seems to mess things up and adds a lot of unnecessary clutter. So I have taken the liberty to resubmit you patch as a new pull request #744 which targets the right branch.
So closing this PR.


Azure Devops Syntax

The Azure DevOps wiki engine expects mermaid to be marked with the syntax of
::: mermaid
:::
instead of the markdown syntax of
Include the mermaid diagram

Is there any plan to support preview of this syntax?

Thanks! Added to backlog.

Would be awesome! (also see this thread: https://developercommunity.visualstudio.com/idea/826044/mermaid-use-the-code-blog-identifier-istead-of-a-c.html?childToView=947222#comment-947222)

Btw, the findDiagram function already seems to be looking for :::mermaid, but I think the syntax is ::: mermaid (with a space in between, see docs).

Thanks for the tip with the discussion!
The space in between looks very weird and does not fit any use pattern that I've heard of in any markdown flavours... but this is the situation with Azure DevOps 😷

fixed in v1.5.1


Incorrect Display of an Error Message

Environment

Qiskit Terra version:  0.20.2
Python version:  3.9.12
Operating system: macOS Monterey 12.4

What is happening?
An error message related to the unitary simulator gets printed out incorrectly. It doesn't show the proper strings.
How can we reproduce the issue?

qc = QuantumCircuit(2, 2)
qc.h(0)
qc.h(1)
qc.cry(0.64, 0, 1)
qc.measure_all()

backend = BasicAer.get_backend('unitary_simulator')
result = backend.run(transpile(qc, backend), shots = 100).result()

result.get_counts(qc)

This shows the following error :
BasicAerError: 'Unsupported "%s" instruction "%s" in circuit "%s"  unitary_simulator measure circuit-122'

What should happen?
Instead of the error being displayed as it is, it should be like this :
BasicAerError: 'Unsupported "unitary_simulator" instruction "measure" in circuit "circuit-122"'

Any suggestions?
A simple change in the syntax of the associated code should fix the issue.

This is a very simple fix. I'd like to start to contribute to Qiskit, and would love to get this fixed. Shall I work on it?

Hi @r-taeja: yes, please do!  I'll assign you (no pressure, it's just a tracking tool so others can see you're on it).  Feel free to ask questions if you need a hand anywhere.


⚠ Bofrost.de has degraded performance

In 8b955ce, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1112 ms

Resolved: Bofrost.de performance has improved in 7786716 after 8 minutes.


how to set bounce address

hi, i want to make all the bounce mail to a specified email address! how can i set the bounce address!
is there anyone can help me? thanks

Bounce address is defined with the from address of the SMTP envelope. By default the envelope is composed automatically from the from, to, cc and bcc mail option fields but you can override it with the envelope option, see example here.
Bear in mind that if you define your own envelope, you also need to include all recipient addresses, not just the sender address as the defaults are not used in this case.

i have tried this way, but the mail still return to my sender address, even through i set the from address of the SMTP envelope to be<EMAIL_ADDRESS>


html preview corrupt since 1.47

since the latest update to version 1.47 any html preview panes don't work. I experience this in atom-html-preview as well as in the plot pane of julia-client.
(Windows 10)

cc #20604  #20754

Thanks for taking the time to contribute!
We noticed that this is a duplicate of #20604  #20754. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Use DROP INDEX IF EXISTS to avoid errors

in v20.2 indexes are not automatically created for referencing columns
in FK constraints, so some DROP INDEX statements were failing in 20.2.

LGTM


Web Bluetooth App

This is a web app that does things like show real-time data, chart historical data, and configure the Aether.
This website should be presentation only. Its functionality should be provided over the Control API.
This should be a PWA-capable static website, hosted on Github. It should use the experimental Web Bluetooth feature to talk to the Aether and thus not require any backend.
Ideally, it should be designed to be flexible for a number of app scenarios:

Something like Blazor Desktop or React Native.
Being hosted directly on the Aether via ASP.NET Core and have the control API accessed over a web service, pulling data via WiFi rather than Bluetooth.

@GabrielGamy is working on this.


Allow expression evaluation to fail

Instead of panicking, let's just return a failure. One example

Done in fcbbcc2c1f0d46256617ee45968a95f15a50c519


Add and improve license attribution for SSAO and SSIL shader code

Found out that I had missed this will discussing license attribution with @akien-mga

Thanks!


Model a DiMES experiment with GITRm

@LC-GA I;m using this issue to open a thread for GITR-M:
References scripts: https://github.com/jguterl/pyGITRM/tree/dev_for_luca
What is required for GITR-M:
produce a mesh (see https://github.com/jguterl/pyGITRM/tree/dev_for_luca)
produce a background plasma (see wiki on PyGITRm: add link once found)
produce a source of particles (see https://github.com/jguterl/pyGITRM/tree/dev_for_luca)
setup inputconfig file


[BUG] Cannot install recompiled apk - there was a problem parsing the package

Information

Apktool Version (apktool -version) - 2.7.0
Operating System (Mac, Linux, Windows) - Linux
APK From? (Playstore, ROM, Other) - F-Droid

Steps to Reproduce

apktool d orig.apk
apktool b orig
Install the apk

APK
https://f-droid.org/repo/us.spotco.fennec_dos_21081020.apk
Questions to ask before submission

Have you tried apktool d, apktool b without changing anything? Yes
If you are trying to install a modified apk, did you resign it? Yes - same error message is shown before jarsign, after jarsign, and even after zipalign
Are you using the latest apktool version? Yes

Could you run adb logcat during the install so you can see the actual error?

Using APKToolGUI on Windows works fine when recompiling this app - that means that I am (still) doing something wrong on my end in Linux. I'm closing the issue since it seems it's not a problem with apktool.


Is it possible to do pagination

Would it be possible to do pagination? And will this be ok for large data sets? Also, what would be the correct direction?

You could pass skip and limit parameters to your publication and use them in your query. As far a large datasets, you will have to experiment and see what works with your application and hardware. I don't recommend using it with very large datasets with many child publications, but your mileage may vary.


Add support for JS localeCompare operation

We don't want to try and provide collation support ourselves, it's very large, but might reasonably provide a call-through to the JS String.localeCompare operation. Some questions about where to put it, and what to do in a non-JS context. Would it make sense to provide it in dart:html or similar library instead?

+1
👍 , I REALLY needed this last week... it would be nice to simply call ".localeCompare" than the crazy lowercasing and compareTo..


How are you using gcloud?

As discussed, BigQuery and more example applications using this library were recent priority. Now that BigQuery has launched (YAY!), I'd like to draw some more attention to the latter. Currently we have an Examples section in the README that has only 1 item in it  :neutral_face:. I'd like to suggest we add a couple more and vow to maintain them. Simple is better.
stephenplusplus/gcloud-keystore seems promising. So, @stephenplusplus, can we add this to the list?
Please list ideas for more examples we can implement or other example repos!
Included so far:

gitnpm
gcloud-keystore

Hello there! Still unannounced (will be in a few days), but online, my project uses gcloud-node on the server side. Here's the url: https://wav.hya.io

@janesconference very cool application! Thanks for the addition. Once you launch, feel free to send a pull request to add your link to the list of sites using it if you'd like to include it! :)

Thanks for the feedback! This issue is pretty hard to find for newcomers now, so I'll just continue snooping on https://www.npmjs.com/browse/depended/gcloud to see who uses us :)


Remove stale services from registrator

This PR removes stale services from the registrator's memory. A stale service is a service that is present in the memory but its corresponding container is not actually running on the host (or has exited).
This issue has been observed when registrator at times misses out on capturing some docker (die) events. Hence this could lead to stale services which also get propagated into the registry backends.
My solution here is to add logic during cleanup to check whether the services in memory actually have their corresponding container running on the host. If not, then remove it.
I have tested this on my end and it seems to work just fine!
Relates to: https://github.com/gliderlabs/registrator/issues/325

Looks good. Not sure why this wasn't added sooner. Registrator is reconciling state between two other parties, so it makes sense Sync should happen in two directions. Or rather, Docker state is authoritative, so I'm surprised Sync didn't originally first consult the Docker API.
One other review and thumbs up?

Thanks @dsouzajude for looking into this, the code change is pretty good overall, though the Fatal call you copied from another place doesn't seem right to me to exit if there's an error here.
This does seem like it might be necessary to address the issue, though I would be interested to understand how it missed the events when these containers exited. It might indicate a race condition between handling "start" and "die" when the container exits too quickly. Or did Docker change their events again, and sends a different event when the container exits on startup?
If this is still needed, it should also be compatible with the --deregister=on-success flag. With that enabled we need to check the exit status of the container so that we don't immediately deregister containers that exited with an error (this was requested by some people that wanted the services to remain in Consul to signal a healthcheck error). In that case, those containers should remain registered even though they're not in Docker.

Thanks @progrium @mgood for the review!
@mgood I'm not really sure why it looks like we missed out on some "die" events. As mentioned in the issue i opened up some days ago, it feels like probably it wasn't able to scale enough to capture all the events. I say this because we are using amazon ecs for orchestration - this repeatedly launches new containers until it reaches a desired number of containers that should be run. So there was an endless loop of containers starting and exiting quickly, so i'm not sure registrator could handle it.
Atleast in most cases, its reproducible in the sense if you use aws ecs as an orchestration tool and set the service scale to maybe 5 and have a buggy image that is run in the container, you would probably notice this.
But this fix does not only solve this problem, it ensures reliability which I think is good to have that always keeps registrator services in sync with what we have running on the host.

Any update/ feedback on this? Should we go forward with the change? @mgood

I had the same problem on my coreos cluster. Playing around with ansible and messing with systemd units led me to stale/duplicated services.
I had to stop consul, fortunately wasn't a server, and wipe out checks and services folder.

+1 from me for this PR
i will build it on my own while waiting for merge

+1 from me for this PR

+1 from me for this PR

+1

+1

This looks like what would fix my issue. After VM crash in my dev box (stupid Windows 10 restarting on updates) I have stale entries in Consul.


Fix: Sanity check that bitcoind is running on the specified network

Make sure that bitcoind is running on the same network teosd is running on. And error if not.
Fixes #113

This one requires some manual testing because we can't depend on bitcoind for unit tests.


Covariant IRequest Handler

Hi, I would like to have a single IRequestHandler that take cares of all my IRequest commands / queries. I was able to achieve this with INotificationHandler but the very same example with IRequest instead of  INotification fails, so I assume there is a technical reason for that? I would need to make it work with IRequest because I need to return a value of a specific type depending on the IRequest sent.

Am I correct to assume that it is not possible to do?
If yes to previous question, I'd be curious to know what is the reason?

Also, I've found that I was able to achieve what I was looking to do using Behavior, here is an example of what I did:
using MediatR;

namespace MediatRDebug;

public interface ICommand<TResponse> : IRequest<TResponse>
{
    public Task<TResponse> Execute();
}

public class MyPipeline<TRequest, TResponse> : IPipelineBehavior<TRequest, TResponse>
    where TRequest : ICommand<TResponse>
{

    public async Task<TResponse> Handle(TRequest request, RequestHandlerDelegate<TResponse> next, CancellationToken cancellationToken)
    {
        return await request.Execute();
    }
}

public class MyCommand : ICommand<string>
{
    public Task<string> Execute()
    {
        return Task.FromResult("This is a test");
    }
}

Now, it works as I would expect, but it feels kind of weird using the pipeline like that. Am I doing something wrong here or this would be the recommended way to achieve what I'm trying to do?
Thanks.

Probably my best answer here is "don't use MediatR". It was not designed for such a scenario. You're probably better off building your own mediator class and handling the generic funny business yourself.


Update create-pool-extensions.md to include new Azure Monitor agent extension

@Padmalathas

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Add libintrafont

This one comes from the pspdev/libintrafont repo. It seems to work okay on PPSSPP.

Sounds like a great idea to include this! Thanks!


Video (YouTube) embed block - cookie handling and sustainability

Link to Ticket

Note
The ticket is for both FE and BE. While this PR handles both, the focus was primarily on the BE. Additional work required on the FE to ensure consistency with overall site design and technical standards.

Description of Changes Made
This makes changes to how YouTube video embeds are rendered, as follows

using the the lite-youtube-embed npm package
using a modified OEmbed finder that uses the above. In order for existing YouTube videos to use this modified OEmbed finder, a management command has been created -- ./manage.py update_youtube_embeds
adding a cookie consent overlay banner on YouTube videos

The result is that a YouTube video does not load when a Page loads, unless the user decides to load it. We ask them for consent, but they can decide that they don't want us to ask them every time, in which case the cookie consent overlay banner won't be displayed -- but the video still won't load unless they click on the play button.
How to Test

After building the images and spinning up the containers, run ./manage.py update_youtube_embeds in the web container, then run honcho start
Open a Page that has a YouTube embed (e.g. http://<IP_ADDRESS>:8000/blog/charity-seo-what-you-need-to-know-right-now/), or add a YouTube embed to any existing Page's body StreamField
Interact with the video on the Page, with the ACs in mind. You can use your browsers' dev tools to check for cookies, storage, network, etc.

Screenshots

Expand to see more

MR Checklist

[x] Add a description of your pull request and instructions for the reviewer to verify your work.
[x] If your pull request is for a specific ticket, link to it in the description.
[x] Stay on point and keep it small so the merge request can be easily reviewed.
[x] Tests and linting passes.

Unit tests

[ ] Added
[x] Not required

Documentation

[ ] Updated build docs
[ ] Updated editor guidelines (https://docs.google.com/document/d/1PAWccdQ4tfaZsrEWmpDhvP3GH5RRmBOARFVp4b-kje8/edit?usp=sharing)
[x] Not required

Browser testing

[x] I have tested in the following browsers and environments (edit the list as required)

Latest version of Chrome on Ubuntu
Latest version of Firefox on Ubuntu

[ ] Not required

Data protection

[x] Not relevant
[ ] This adds new sources of PII and documents it and modifies Birdbath processors accordingly

Accessibility

[ ] Automated WCAG 2.1 tests pass
[ ] HTML validation passes
[ ] Manual WCAG 2.1 tests completed
[ ] I have tested in a screen reader
[ ] I have tested in high-contrast mode
[ ] Any animations removed for prefers-reduced-motion
[ ] Not required

Sustainability

[ ] Images are optimised and lazy-loading used where appropriate
[ ] SVGs have been optimised
[x] Perfomance and transfer of data considered
[ ] If JavaScript is needed alternatives have been considered
[ ] Not required

Pattern library

[ ] The pattern library component for this template displays correctly, and does not break parent templates
[ ] The styleguide is updated if relevant
[ ] Changes are not relevant the pattern library

I'm getting some eslitnt errors when running npm run build after installing the lite-yt-embed package but again I'm happy to resolve these.
/Users/helen/repos/torchbox/torchbox.com/tbx/static_src/javascript/main.js
  15:8  error  Unable to resolve path to module 'lite-youtube-embed/src/lite-yt-embed.css'  import/no-unresolved
  16:8  error  Unable to resolve path to module 'lite-youtube-embed/src/lite-yt-embed'      import/no-unresolved```


pull request for e2e test at 01/27/2017 23:56:17

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Is this possible to do progress-bar based on synchronous file upload?

Could you explain in more detail what do you mean? Do you mean like a server-side progress bar in the promote phase?

So, as I understood, you would like to have a client-side progress when uploading, but with using remote: true. Unfortunately, this isn't possible, because the progress bar is only applicable to uploading itself, once you're in your controller the uploading is already finished. If you want a progress bar, you need to use a JS library like jQuery-File-Upload or Dropzone.js instead of remote: true, along with the direct_upload plugin.
I will close this as I prefer using GitHub issues only for bug reports, but if you have any more questions or want to continue the discussion, a much better place to do that is on the Shrine Google group.


New bridge: FilterMore

this bridge allows the user to have a filtered version of a feed.
it is quite advanced: offers regexp filtering, sorting, limiting...
I used it when I wanted to follow a websites including events. They had no way to filter based on city, but they were specifying that in the title, so I could use this bridge to get what I wanted.
The has_media field is particularly useful for podcasts: sometimes I want to just play the first audio in a feed. With this, I can :

filter has_media
limit=1
which gives me exactly what I wanted.

we already have FilterBridge which i hope you have taken a look at.
need more time and community feedback on whether to include this bridge.

we already have FilterBridge which i hope you have taken a look at.

yes, I know that and have used it. I think that FilterMore is far more advanced, but I can see how having duplicates is not great for the project.

yes FilterMore I like the name and its features.
if okay i would like wait for community feedback from other users.


Thf 446 published date for articles UI

Publication date field added to article content type.
How to test:

Related PR -> THF-446-published-date-for-articles-drupal
Run drupal site branch with introductions  THF-446-published-date-for-articles-drupal
Create new article page on Drupal https://drupal-tyollisyyspalvelut-helfi.docker.so/node/add
Check that article has date under page title.
Create new article https://drupal-tyollisyyspalvelut-helfi.docker.so/node/add and scheduled publication date for next day.
See next day that the date underneath title has changed to the publication date.

Developer:
Add console.log('node') NodeArticlePage.tsx file row 13 and open the created article page. You should see published_at field in the console.

Lets test how the publication date behaves on the test environment.


HSClient.get_template_edit_url doesn't take a test_mode parameter

When using the function as suggested in the documentation, I get permission errors that suggest using test_mode=1, but the function in the python sdk won't take it as a parameter. Is there another way I should be using the test mode?

Hi,
In this SDK, you can enable test mode using an optional parameter called test_mode. For example:
# Send a signature request with remote files
signature_request = client.send_signature_request(
                                test_mode=True,
                                files=None,
                                file_urls=["http://www.example.com/download/sample.pdf"],
                                title="NDA with Acme Co.",
                                subject="The NDA we talked about",
                                message="Please sign this NDA and then we can discuss more. Let me know if you have any questions.",
                                signing_redirect_url=None,
                                signers=signers,
                                cc_email_addresses=cc_email_addresses)

Cheers!

Hi,
I've seen that on other functions. The problem is that get_template_edit_url doesn't accept the parameter test_mode. As in
client.get_template_edit_url(template_id, test_mode=True)
throws
get_template_edit_url() got an unexpected keyword argument 'test_mode'
I've gotten the code to work by doing
client.get_template_edit_url(template_id + "?test_mode=1")
but I imagine you want your sdk to offer a better solution.

This is actually something we need to figure out on the back-end. This endpoint should be available despite test_mode, since it's simply retrieving an existing template for embedded editing. I'll keep you posted.
p.s., Thanks for the PR!


Sound resolution not adjustable

[IMPORTANT: Please remove or replace everything between two square brackets ( [ ] ) before posting your issue.]
Description
[The Spotify plugin doesn't work after troubleshooting, although that is already a noted issue. My problem is that the sound resolution can't be adjusted above 1024]
Versions
[2.02]

Rainmeter Version: [4.2.0 r3111 (64 Bit)]
OS Version: [Windows 10 Home latest update]
Skin Version: [Monstercat Visualizer 2.02]
Spotify Version: [Plugin 2.15, most recent desktop client]
Audio Device [Behringer U-Phoria 202HD, 24bit, 192k]

Seems like an issue with the config file, so if youre comfortable to delete your current settings go to Documents/Rainmeter/Skins/Monstercat-Visualizer/@Resources and remove the variables.ini file and replace it with the default one from here: https://github.com/MarcoPixel/Monstercat-Visualizer/blob/master/%40Resources/variables.ini.
After that, restart Rainmeter and go to the settings. Click on the box with the sound resolution and then enter your value and when you're done press ENTER to confirm it.
If it's still not working, please send in a screenshot of the Rainmeter log after you've tried to set the value.


Now it is possible to use msgpack instead of JSON.

To do this call postRobot.enableMsgpack(window) from both sides. You need to check version of post-robot by yourself

For unknown reasons npm didn't added msgpack5 in package.json...

Hi @evgkul -- thanks so much for the PR!
I'm wondering -- would it make sense to add generic support for serializers/deserializers? Then people could plug in whatever serializer they like, including msgpack.
Something like:
postRobot.registerSerializer({
    serialize(obj) {
        return msgpack5.encode(obj).toString('hex');
    },
    deserialize(str) {
        return msgpack5.decode(str);
    }
});

Then we avoid pulling in extra dependencies into the core post-robot bundle, and anyone would be able to plug-in their own favorite serializer.

By the way, congrats on the first PR! Happy to offer any help to get this ready to merge in. :)

I think that with generic support of serializers/deserializers it will be too hard to synchronize them.

Can you elaborate? How would it be hard to synchronize them, and is that not a problem using msgpack also?

Task of post-robot is to make communication between cross-domain iframes simpler. If host page will have serializer and iframe will not then they will communicate by JSON or simply will not work.

Makes total sense, but how do we avoid that problem even for this PR?

For that is needed mechanism to send through JSON to other side a marker(unused property) and wait for response marker and then activate msgpack

Isn't that a problem with the current PR? If I sent a message but the other window has not done enableMsgpack() then it won't be able to correctly read the message, right?

If it has support of msgpack it will be able to read
if (typeof message === 'string') {              parsedMessage = jsonParse(message);          } else {              parsedMessage = msgpack5.decode(message);          }

Ah -- I think that's going to be a problem. We need to support IE9, which I believe only supports string messages.

When I will have some free time I will add base64 encoding if IE is detected

I'd prefer to go down the pluggable-serializer path. I just checked and the minified msgpack5 is still 120kb. That's 3 times the size of post-robot.min.js, and I don't feel like it makes sense to bundle it in, given that for most usecases JSON serialization is enough.
Totally get your concern about synchronizing the serializers. Perhaps we could do something like:
postRobot.registerSerializer('msgpack5', {
    serialize(obj) {
        return msgpack5.encode(obj).toString('hex');
    },
    deserialize(str) {
        return msgpack5.decode(str);
    }
});

Then pass msgpack5 in each message, and error out if that deserializer has not been registered on the receiving end. That way people will be able to easily figure out in which pages they haven't added the serializer/deserializer.
Bear in mind -- postMessaging is just in-memory messaging, so the size of the message shouldn't be a bottleneck most of the time. But I do see value in being able to register custom serializer in case people want to pass across more elaborate data types.

Hmm... I will try to add marker with array of serializers/deserializers

... or will just replace msgpack 5 with https://github.com/kawanet/msgpack-lite (50kb) :)

Well, for now I will close this pull request

Moved this PR's commit into "msgpacksupport" branch


Uncaught Parse error in .ftpconfig JSON : SyntaxError: Unexpected token

[Enter steps to reproduce below:]

...As soon as I added second project folder to Atom this error started to happen.

Atom Version: 1.0.2
System: Microsoft Windows 8.1 Pro
Thrown From: Remote-FTP package, v0.7.1
Stack Trace
Uncaught Parse error in .ftpconfig JSON : SyntaxError: Unexpected token
At C:\Users\Marko\.atom\packages\Remote-FTP\lib\client.js:78

undefined

Commands
     -2:34.5.0 remote-ftp:toggle (ol.tree-view.full-menu.list-tree.has-collapsable-children.focusable-panel)
  2x -2:29.4.0 remote-ftp:connect (ol.tree-view.full-menu.list-tree.has-collapsable-children.focusable-panel)

Config
{
  "core": {}
}

Installed Packages
# User
Remote-FTP, v0.7.1

# Dev
No dev packages

Sorry to bother with this, when something like that happens, just check your config file and see if you have " " marks open somewhere.
I simply checked my config and saw that I was missing " at the end of the host line.


makeRequest doesn't return XHR object

We upgraded from 1.1.0 to 1.7.0, and now we can't establish a WebSocket connection. After the FastBoot compatibility PR #1035, makeRequest() no longer returns the jQuery XHR object, but a Promise instead. We were using that XHR object to pull out some header information (CSRF token specifically) and saving it off for other requests to use. Is there a new way to get the header info from makeRequest()? If not, the request.headers property should also be returned from the fetch() call.

It would be helpful if https://github.com/simplabs/ember-simple-auth/blob/master/addon/authenticators/oauth2-password-grant.js#L362-L364 returned the response object with a responseJSON property like it does in the if and the catch.
            } else {
              response.responseJSON = json;
              resolve(response);
            }

We can't change the value that makeRequest resolves with now as that would be a breaking change actually. I do agree we should do that though as currently the API is a bit inconsistent. I think the way to go here would be to add a resolveWithResponse property similar to rejectWithResponse that would allow us to introduce this in a non-breaking way.

cleaning up old issues

@marcoow, Any timeline on if/when your alternate suggestion would be implemented?

No - if you'd like to contribute a PR implementing just that, that would be ❤


Add Responsive Text in voter registration Help page

Fixes #408

You can text the mobile view in this website http://mattkersley.com/responsive/
Describe the changes you've made
I add responsive text style in help.html, remove the class of input-row
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
NA
Deploy your changes to github page or netlify and mention deployment link:
NA
Additional context (OPTIONAL)
Add any other context or screenshots about the feature request here.
Test plan (OPTIONAL)
A good test plan should give instructions that someone else can easily follow.
How someone can test your code?
Checklist

[x] I have mentioned related issue number in PR description.
[x] I have added screenshots and related demo links of my changes..
[x] The title of my pull request is a short description of the requested changes.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] Open Source Program names(OPTIONAL: If you participated in any open-source program then please mention the program name here)

Fetch upstream your repo is 11 commits behind.

i done git pull upstream master
then continue steps 7 until 11?


Include or explude namespaces based on labels

Hello,
I think that botkube should be able to adapt its selective filter according to labels on the namespace, i.e I would like to be able to either include or exclude namespaces depending on some labels that the user would choose (production for example).
I know that a wildcard filter exists for excluding but I would rather my logic not to be dictated by the namespace name but rather by the labels attached to it.
As far as I understood, annotating the namespace with botkube.io/disable: true would only disable changes of the namespace itself and not its content and thus it doesn't fill the gap (and it would be too opinionated of a label in my sense anyway).
I hope I haven't missed any already existing feature, and would like to hear what you think.
Thanks!

This sounds like a good feature to have

Sounds like a good feature, @mumrau you are thinking about a customizable label, attached to a namespace that excludes all the resources in the namespace?
Like for example, configure the custom label in the yaml file or something like that?

I don't think about exclusion but configuration and customization.
I want botkube to react in harmony with other applications based on a common (and thus customizable) label.
Let's say I have ns-a and ns-b:

ns-a has criticality: major
ns-b has criticality: minor

This way, all my monitoring and observability tools know whether they should act or no based on that label, I don't have to have a mapping of major whith is botkube.io=yes and prometheus.io=activated or such.
I hope I make sense.


What does seq_len means?

Thank you for sharing this wonderful project. It works quite well for my number recg problem.
But I have some confusion.
In feed_dict, self.__seq_len: [self.__max_char_count] * self.__data_manager.batch_size,
and
max_char_count = reshaped_cnn_output.get_shape().as_list()[1] .
I'm not quite understand of that. Should seq_len be the width of img after cnn part of shape(batchsize, ?)
Sorry for my poor English.

After the CNN part of the model, the image will become a tensor of size 24x512 (if you use a width of 100px). The 24 represents the maximum number of characters that we could "read" and therefore the number of timesteps that the RNN should process.
I would recommend reading the paper as the author explains it quite well.

3q


Allow camera use different up directions

Although Vec::Y is good default, in many other cases where we need to change the camera up direction.

An example of where this is a problem is when trying to do a top down view.
In this case the Up-vector (Vec3::Y) is co-planar with the forward vector which is down the negative Y-axis. This makes the cross product operation fail when trying to compute the view transform.
Could we get this fix merged?

Yes, thank you!


Netbox not working after upgrade

i have just install the last version and i get an error :
Traceback (most recent call last):
File "/usr/local/lib/python3.5/dist-packages/gunicorn/workers/sync.py", line 135, in handle
self.handle_request(listener, req, client, addr)
File "/usr/local/lib/python3.5/dist-packages/gunicorn/workers/sync.py", line 176, in handle_request
respiter = self.wsgi(environ, resp.start_response)
File "/usr/local/lib/python3.5/dist-packages/django/core/handlers/wsgi.py", line 170, in call
response = self.get_response(request)
File "/usr/local/lib/python3.5/dist-packages/django/core/handlers/base.py", line 124, in get_response
response = self._middleware_chain(request)
File "/usr/local/lib/python3.5/dist-packages/django/core/handlers/exception.py", line 41, in inner
response = response_for_exception(request, exc)
File "/usr/local/lib/python3.5/dist-packages/django/core/handlers/exception.py", line 86, in response_for_exception
response = handle_uncaught_exception(request, get_resolver(get_urlconf()), sys.exc_info())
File "/usr/local/lib/python3.5/dist-packages/django/core/handlers/exception.py", line 131, in handle_uncaught_exception
if resolver.urlconf_module is None:
File "/usr/local/lib/python3.5/dist-packages/django/utils/functional.py", line 35, in get
res = instance.dict[self.name] = self.func(instance)
File "/usr/local/lib/python3.5/dist-packages/django/urls/resolvers.py", line 306, in urlconf_module
return import_module(self.urlconf_name)
File "/usr/lib/python3.5/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 986, in _gcd_import
File "", line 969, in _find_and_load
File "", line 958, in _find_and_load_unlocked
File "", line 673, in _load_unlocked
File "", line 665, in exec_module
File "", line 222, in _call_with_frames_removed
File "/opt/netbox/netbox/netbox/urls.py", line 22, in
url(r'^dcim/', include('dcim.urls', namespace='dcim')),
File "/usr/local/lib/python3.5/dist-packages/django/conf/urls/init.py", line 50, in include
urlconf_module = import_module(urlconf_module)
File "/usr/lib/python3.5/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 986, in _gcd_import
File "", line 969, in _find_and_load
File "", line 958, in _find_and_load_unlocked
File "", line 673, in _load_unlocked
File "", line 665, in exec_module
File "", line 222, in _call_with_frames_removed
File "/opt/netbox/netbox/dcim/urls.py", line 6, in
from . import views
File "/opt/netbox/netbox/dcim/views.py", line 24, in
from . import filters, forms, tables
File "/opt/netbox/netbox/dcim/forms.py", line 16, in
from formfields import MACAddressFormField
ImportError: No module named 'formfields'
any idea on what's the problem ?

This appears to be an installation issue. Please post to the mailing list if you require assistance.


Better handling of iptables logs

This changes a few things for better iptables log handling  :

Only parse kernel message containing IN= to prevent lots of unparsed lines in metrics
Skip line if iptables decision is ACCEPT
OUT= is not always present
remove unused begining of regex, which can prevent parsing

Here are a few example of log lines which weren't parsed correctly and are now OK
From a Proxmox VE server, when we log accepted packets
2021-03-02T18:40:09.984Z pve-bkp kernel: 101 6 tap101i0-IN 02/Mar/2021:19:40:07 +0100 ACCEPT: IN=fwbr101i0 OUT=fwbr101i0 PHYSIN=fwln101i0 PHYSOUT=tap101i0 MAC=76:c8:d3:82:b5:d3:b6:d2:88:c3:15:e6:08:00 SRC=<IP_ADDRESS> DST=<IP_ADDRESS> LEN=60 TOS=0x00 PREC=0x00 TTL=56 ID=54553 PROTO=TCP SPT=36494 DPT=10051 SEQ=271997875 ACK=0 WINDOW=29200 SYN

Also from PVE, standard DROP. Note the absence of OUT=
2021-03-02T18:40:27.254Z cvn1 kernel: 0 6 PVEFW-HOST-IN 02/Mar/2021:19:40:22 +0100 policy DROP: IN=enp1s0f0 MAC=a8:1e:84:96:92:20:a0:93:51:b7:50:73:08:00 SRC=<IP_ADDRESS> DST=<IP_ADDRESS> LEN=40 TOS=0x00 PREC=0x00 TTL=241 ID=54321 PROTO=TCP SPT=46363 DPT=8123 SEQ=3807223242 ACK=0 WINDOW=65535 SYN

Linux host, using a custom log prefix (-j LOG --log-prefix 'Firewall: ')
2021-03-02T19:00:36.192Z stor2 kernel: Firewall: IN=em1 OUT= MAC=24:6e:96:82:7c:28:a0:93:51:b7:55:ef:08:00 SRC=<IP_ADDRESS> DST=<IP_ADDRESS> LEN=40 TOS=0x00 PREC=0x00 TTL=245 ID=43340 PROTO=TCP SPT=41041 DPT=34035 WINDOW=1024 RES=0x00 SYN URGP=0

With the above change, most of my logs are now parsed, and I do not trigger false positives with accepted-but-logged packets

Great, let me review this and add the tests !
Thanks again for all those MR :love_you_gesture:


Ensure single $ZPLUG_HOME/bin in path

Further ensure for uniqueness.  Updated fix for #212

Merged! Thanks!


Fix a number of resource leaks when using SourceKit-LSP in tests or as a library

Shutdown toolchain connections on exit. This also fixes SR-11309.
Fix reference cycle for build system manager delegate.

@swift-ci please test

@swift-ci please test macOS

@swift-ci please test macOS


detect fonts required by Beaver Builder

ie, user needed to add the following to Additional URLs:
/wp-content/plugins/bb-plugin/fonts/fontawesome/webfonts/fa-solid-900.woff
/wp-content/plugins/bb-plugin/fonts/fontawesome/webfonts/fa-solid-900.woff2
/wp-content/plugins/bb-plugin/fonts/fontawesome/webfonts/fa-solid-900.ttf
/wp-content/plugins/bb-plugin/fonts/fontawesome/webfonts/fa-solid-900.svg
/wp-includes/js/mediaelement/mejs-controls.svg

This is also an issue with elementor.


[Bug] Querying both the Time field and the t1 field triggers a crash bug.

Search before asking

[X] I searched in the issues and found nothing similar.

Version
1.3.2
Describe the bug and provide the minimal reproduce step
DROP DATABASE root.db0
CREATE DATABASE root.db0

INSERT INTO root.db0(timestamp,t1) VALUES(1724497540136, true)

# query 1 success
SELECT time FROM root.db0
SELECT t1 FROM root.db0

# query 2 error
SELECT time, t1 FROM root.db0

What did you expect to see?
Query 1.1 returns "No Results," while Query 1.2 can retrieve all the data. Query 2 should correctly return all the data.
What did you see instead?
Query 2 reports the following error:
4668710 [main] WARN  o.a.t.transport.TIOStreamTransport - Error closing output stream.
java.net.SocketException: Socket closed
        at java.net.SocketOutputStream.socketWrite(SocketOutputStream.java:118)
        at java.net.SocketOutputStream.write(SocketOutputStream.java:155)
        at java.io.BufferedOutputStream.flushBuffer(BufferedOutputStream.java:82)
        at java.io.BufferedOutputStream.flush(BufferedOutputStream.java:140)
        at java.io.FilterOutputStream.close(FilterOutputStream.java:158)
        at org.apache.thrift.transport.TIOStreamTransport.close(TIOStreamTransport.java:156)
        at org.apache.thrift.transport.TSocket.close(TSocket.java:252)
        at org.apache.iotdb.rpc.TElasticFramedTransport.close(TElasticFramedTransport.java:106)
        at org.apache.iotdb.jdbc.IoTDBConnection.reconnect(IoTDBConnection.java:545)
        at org.apache.iotdb.jdbc.IoTDBStatement.reConnect(IoTDBStatement.java:705)
        at org.apache.iotdb.jdbc.IoTDBStatement.requestStmtId(IoTDBStatement.java:690)
        at org.apache.iotdb.jdbc.IoTDBStatement.<init>(IoTDBStatement.java:93)
        at org.apache.iotdb.jdbc.IoTDBStatement.<init>(IoTDBStatement.java:127)
        at org.apache.iotdb.jdbc.IoTDBConnection.createStatement(IoTDBConnection.java:202)
        at org.apache.iotdb.cli.AbstractCli.executeQuery(AbstractCli.java:549)
        at org.apache.iotdb.cli.AbstractCli.handleInputCmd(AbstractCli.java:435)
        at org.apache.iotdb.cli.AbstractCli.processCommand(AbstractCli.java:811)
        at org.apache.iotdb.cli.Cli.readerReadLine(Cli.java:216)
        at org.apache.iotdb.cli.Cli.receiveCommands(Cli.java:199)
        at org.apache.iotdb.cli.Cli.serve(Cli.java:163)
        at org.apache.iotdb.cli.Cli.runCli(Cli.java:101)
        at org.apache.iotdb.cli.Cli.main(Cli.java:60)
Msg: java.sql.SQLException: org.apache.thrift.transport.TTransportException: Socket is closed by peer.

Anything else?
Dear IoTDB team, I believe this should be classified as a crash bug. If it is a bug, please mark this issue as "bug". Thank you!
Are you willing to submit a PR?

[x] I'm willing to submit a PR!

The Time column is always included in IoTDB's regular queries and does not need to be manually specified. Additionally, this error occurred during the runtime when the connection between the client and server was interrupted


node fails to get any peers, restarts and fails again

While running node through systemd, I found it fails to get peers and shuts down, systemd brings it up again and then it dies again. Removing coda-config fixed the peers issue, node has peers now and is bootstrapping.
At least one other user (@inv0ke) reported the same, i think others had it too.
Coda-config folder too big to attach here, even after archiving - let me know if you want it and I'll send it in another way, I have it saved.
Running daemon with these options:
DAEMON_OPTIONS="-discovery-port 8303 -propose-key /home/nik/keys/my-wallet -run-snark-worker 4vsRCVxL7t27LsPU54SYwfWBjA8UrgdRH2QVyXQZCZJdWMXr5kFJ4yTDbfaXjqphVPKUXWZvzougae6YX6icbngH2DLUvfAFvkp8NzWPoW2GMqZCJ3aZKEZDjB5tG9bnZmM7AuhZN6L6tDZZ"
DAEMON_PEERS="-peer /dns4/seed-one.genesis.o1test.net/tcp/10002/ipfs/12D3KooWP7fTKbyiUcYJGajQDpCFo2rDexgTHFJTxCH8jvcL1eAH -peer /dns4/seed-two.genesis.o1test.net/tcp/10002/ipfs/12D3KooWL9ywbiXNfMBqnUKHSB1Q1BaHFNUzppu6JLMVn9TTPFSA"

Closing this issue as it is stale for a long time.


Not working on windows phone

Any ideas?

it's kind of problem to check it out on WP. Would you like to fix it?


rush: 1.8 -> 1.9

Motivation for this change
http://lists.gnu.org/archive/html/info-gnu/2019-04/msg00007.html
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Works for me, non-nixos linux x86_64.


pdfViewer.currentPageNumber fails to change page

Configuration:

Web browser and its version: Chrome 85
Operating system and its version: Windows 10
PDF.js version: 2.5.207
Is a browser extension: No

Steps to reproduce the problem:

https://jsfiddle.net/chris_h/eq7fd94z/
Note how the view remains on page 1 even though currentPageNumber is called after the pagesinit has fired.

What is the expected behavior?
After calling pdfViewer.currentPageNumber = 2 I would expect the view to change to showing page 2.
What went wrong?
The view remains on page 1

The viewerContainer needs to be absolutely positioned, note https://github.com/mozilla/pdf.js/blob/e51e9d1f33bd6e404bfa795799f1cc035537091e/examples/components/simpleviewer.html#L24-L36
Hence this issue is, for all intents and purposes, a duplicate of #11626.

Ah, thank you. That does indeed fix the issue, I did not realise that property was required. 👍


fix(TestFrameworkOrchestrator) Temporarily ignore bithound checks

Making Bithound issues worse while trying to fix them, great! 💪


fix  is undefined

I found a problem that initial value of v-model of  is treated as undefined if contains .
This fix,  is treated as default value.

Sorry, I merged it without realizing it's on master. Can you re-submit the PR against dev branch instead?


Subscribe to all messages

In my usecase I want to forward any message that I receive.
The subscribe method needs a certain type. But it will be a bit too redundant if I have to list each type.
Is this a limitation? or can we extend it?

ping @wucke13

Sadly this project seems abandoned, what a shame 😢

I don't feel no shame :smile:
So, to implement this I see three options:

one either has to name every variant of the MavMessage enum for a message set, and subscribe to it. This is extremely tedious.
alter the source code of the original mavlink crate to include an iterator over all variants of said MavMessage enum. That way one does not have to type all the combinations, instead one can benefit from code generation already included in bespoke crate.
Just go through the integer IDs for the subscribing, ignoring errors for unknown IDs.

I would definitely rule out option 1. Option 2. seems to be the technically best solution, but back when I pursued an active interest in this it seemed almost impossible to agree on something with the mavlink maintainers, probably like me they were and/or are short on time. Option three is the quick'n dirty approach.


I have a problem with gen_protos on aarch64 platform.

OS Platform and Distribution
Ubuntu20.04 LTS
Compiler version
gcc/gcc+8
Programming Language and version
Python3.8
Installed using virtualenv? pip? Conda?(if python)
No response
MediaPipe version
0.10.0
Bazel version
5.1
XCode and Tulsi versions (if iOS)
No response
Android SDK and NDK versions (if android)
No response
Android AAR (if android)
None
OpenCV version (if running on desktop)
No response
Describe the problem
I have successfully built MediaPipe <IP_ADDRESS> on aarch64 platform, created whl file and run it on Raspberry Pi4.

However, when building with MediaPipe 0.10.0 using the same procedure, the following error is output.

How can i solve this problem?

Complete Logs
root@ubuntu:/mp/mediapipe# python3 setup.py gen_protos
running gen_protos
generating proto file: build/lib.linux-aarch64-3.8/mediapipe/tasks/cc/components/processors/proto/detection_postprocessing_graph_options_pb2.py
mediapipe/tasks/cc/components/processors/proto/detection_postprocessing_graph_options.proto:30:12: Explicit 'optional' labels are disallowed in the Proto3 syntax. To define 'optional' fields in Proto3, simply remove the 'optional' label, as fields are 'optional' by default.
mediapipe/tasks/cc/components/processors/proto/detection_postprocessing_graph_options.proto:34:12: Explicit 'optional' labels are disallowed in the Proto3 syntax. To define 'optional' fields in Proto3, simply remove the 'optional' label, as fields are 'optional' by default.
mediapipe/tasks/cc/components/processors/proto/detection_postprocessing_graph_options.proto:39:12: Explicit 'optional' labels are disallowed in the Proto3 syntax. To define 'optional' fields in Proto3, simply remove the 'optional' label, as fields are 'optional' by default.
mediapipe/tasks/cc/components/processors/proto/detection_postprocessing_graph_options.proto:44:12: Explicit 'optional' labels are disallowed in the Proto3 syntax. To define 'optional' fields in Proto3, simply remove the 'optional' label, as fields are 'optional' by default.
mediapipe/tasks/cc/components/processors/proto/detection_postprocessing_graph_options.proto:47:12: Explicit 'optional' labels are disallowed in the Proto3 syntax. To define 'optional' fields in Proto3, simply remove the 'optional' label, as fields are 'optional' by default.
root@ubuntu:/mp/mediapipe#

Hello @Heartful-echo,
Could you please elaborate your query with complete details and steps followed to reproduce the issue from our end.

Hi @kuaashish
Thank you for your comment.
My steps are below.
$ git clone -b v0.10.0 https://github.com/google/mediapipe
$ sed -i -e "/"imgcodecs"/d;/"calib3d"/d;/"features2d"/d;/"highgui"/d;/"video"/d;/"videoio"/d" third_party/BUILD
$ sed -i -e "/-ljpeg/d;/-lpng/d;/-ltiff/d;/-lImath/d;/-lIlmImf/d;/-lHalf/d;/-lIex/d;/-lIlmThread/d;/-lrt/d;/-ldc1394/d;/-lavcodec/d;/-lavformat/d;/-lavutil/d;/-lswscale/d;/-lavresample/d" third_party/BUILD
$ sed -i -e "/^        # Optimization flags/i \        "ENABLE_NEON": "OFF"," third_party/BUILD
$ sed -i -e "/^        # Optimization flags/i \        "WITH_TENGINE": "OFF"," third_party/BUILD
$ sudo python3 setup.py gen_protos
Next, I want to run the following command to build, but I get an error with python3 setup.py gen_protos.
sudo python3 setup.py bdist_wheel
I see a file called detection_*** added in 0.10.0.
This file has syntax = "proto3"; Until now it was syntax = "proto2";
I'm assuming this has an effect.
I tried changing the syntax="proto3" in the detection_*** files to syntax="proto2" and building.
The whl file is created but when I write the code using this whl file I get the below error.
root@ubuntu:/mp# python3 object.py
Traceback (most recent call last):
File "object.py", line 3, in
import mediapipe as mp
File "/usr/local/lib/python3.8/site-packages/mediapipe/init.py", line 30, in
import mediapipe.tasks.python as tasks
ImportError: cannot import name 'python' from 'mediapipe.tasks.python' (/usr/local/lib/python3.8/site-packages/mediapipe/tasks/python/init.py)

@Heartful-echo
I could not reproduce the issue from our end. Would you please let us know the complete steps followed from the mediapipe documentation or Any link? Which you can point here that will be helpful to reproduce the issue.

@kuaashish
Can you provide the whl file of mediapipe0.10.0 for aarch64 in python3.8 environment?
My aim is to use mediapipe0.10.0 on Raspberry Pi4 64bit.
Thank you

@Heartful-echo,
We do not have official wheels for aarch64 in python now. However, Support of Raspberry Pi is in our roadmap and will be available soon.

@kuaashish
This is very good information.
I hope one day MediaPipe will be available on Raspberry Pi and CPUs with the same aarch64 architecture.
This case will be closed.
I appreciate your support.

@kuaashish
This is very good information.
I hope one day MediaPipe will be available on Raspberry Pi and CPUs with the same aarch64 architecture.
This case will be closed.
I appreciate your support.


ATS_PROBE: add SystemTap ATS probe autest

Add an autest to verify ATS_PROBE SystemTap functionality. Sadly, this requires sudo, so this cannot run in CI. Thus this test will be skipped by default but can be unskipped by developers who want to use it to test ATS_PROBES.

Cherry-picked to v10.0.x


Issue 217/consolidate inbox outbox

This PR is mainly about consolidating Inbox and Outbox into a single Messages Page. With this, Inbox and Outbox would be re-routing to Messages with and Inbox and Outbox being used as toggles for displaying their respective components as oppose to sending the user to another page.
https://github.com/codeforpdx/PASS/assets/14917816/b5335da9-14ad-4a00-9728-7ee54a2ef629
Considering how similar the two message component are they're now refactored as the new MessageFolder component, which changes based on folderType. For an easy transition to Material UI from styled-components, I've moved all relevant stylings into MessageStyles.js

Since you are already redoing Navbar Links to have Messages, want to rename Forms to My Documents for now?

However, the more I look at messages, the less it looks like email and the more it looks like a chat app where you have basically one conversation with each person. IMO it would be good to touch base with UI/UX about which system would be more useful.

I'd be really interested to hear what the caseworkers think about this. Maybe they really don't need anything more robust than a chat system. Although if we do opt for a more standard email system, we don't necessarily have to include every feature.

I'd be really interested to hear what the caseworkers think about this. Maybe they really don't need anything more robust than a chat system. Although if we do opt for a more standard email system, we don't necessarily have to include every feature.

Building out a fully featured email system is going to be a really big ask. Easiest way would probably be to bolt on an email server onto the solid server and just use email as is. But that's not a small ask either. I'd be hesitant to sell that as a core feature if we can avoid it.

Alright, I think I've touched most if not all the comments in this PR. Think it's ready for merging.


Container View JSQMessagesViewContoller Not Showing

New issue checklist

[ ] I have read the README, documentation, and FAQ.
[ ] Contributing guidelines confirmation: ____
[ ] I have searched existing issues and this is not a duplicate.

General information

JSQMessagesViewController version:
iOS version:
Devices/Simulators:
Reproducible in the demo project? (Yes/No):
Any related issues:

What happened?

http://i.stack.imgur.com/7OOPV.png

Container View JSQMessagesViewContoller Input tool bar Not Showing

It looks like the development version has this issue.

Hello everyone!
I'm sorry to inform the community that I'm officially deprecating this project. 😢 Please read my blog post for details:
http://www.jessesquires.com/blog/officially-deprecating-jsqmessagesviewcontroller/
Thus, I'm closing all issues and pull requests and making the necessary updates to formally deprecate the library. I'm sorry if this is unexpected or disappointing. Please know that this was an extremely difficult decision to make. I'd like to thank everyone here for contributing and making this project so great. It was a fun 4 years. 😊
Thanks for understanding,
— jsq


SLURM won't replace $JOB_ID placeholder

Hi, I'm using SLURM 17.02.3.
In my environment, I noticed filenames of stdout and stderr are wrong (e.g., worker.$JOB_ID.err/out)
https://github.com/dask/dask-drmaa/blob/master/dask_drmaa/core.py#L42-L43
'outputPath': ':' + worker_out_path_template % dict(jid='$JOB_ID.$drmaa_incr_ph$', kind='out'),
'errorPath': ':' + worker_out_path_template % dict(jid='$JOB_ID.$drmaa_incr_ph$', kind='err'),

Solution
Instead of $JOB_ID, SLURM uses %j placeholder, please see https://slurm.schedmd.com/sbatch.html
jid='$JOB_ID.$drmaa_incr_ph$' should be jid='%j.$drmaa_incr_ph$' in slurm.
I confirmed that when I hard-cord it in slurm but I don't know how to determine slurm or any other grid engines.

Handling Job ID on LSF is a bit different than say SGE or SLURM ( https://github.com/dask/dask-drmaa/issues/32 ). So also needs some hand holding to get the right filename.

Generally thinking we need some way to configure part of the filename for stdout or stderr. Starting with worker or better yet dask-worker. is fine and ending with .out or .err respectively is also fine. What would be good is to make the middle chunk (logfile_infix ?) configurable so one can include any %j, %J, or $JOB_ID or whatever else the user might want to add there (e.g. proj-v2.3.2). This should handle the different clusters using DRMAA with their conventions better while providing better functionality to the user. We can also leave the $JOB_ID part as default to avoid breaking existing use cases.

Trying to address this in PR ( https://github.com/dask/dask-drmaa/pull/56 ). Feedback and helping testing would be appreciated.


Help interpreting error on "python udocker pull [image]"

Hi there 👋
I am running this script on AWS Lambda:
export HOME=/tmp
export UDOCKER_DIR=/tmp
export UDOCKER_BIN=/tmp
export UDOCKER_LIB=/tmp
export UDOCKER_CONTAINERS=/tmp

cd /tmp

curl https://raw.githubusercontent.com/indigo-dc/udocker/devel/udocker.py >udocker
chmod +x ./udocker

python udocker version # prints -> version: 1.1.1
# also prints -> tarball: https://owncloud.indigo-datacloud.eu/index.php/s/AFImjw8ii0X72xf/download https://cernbox.cern.ch/index.php/s/VC7GuVWA7mYRAiy/download http://repo.indigo-datacloud.eu/repository/indigo/2/centos7/x86_64/tgz/udocker-1.1.1.tar.gz

# Pull image
python udocker pull quay.io/skilbjo/iris:x86 # failure occurs here
# failure occurs also with a more mainstream image like ubuntu:17.10

and here is my stack trace
File "udocker", line 6725, in <module>
sys.exit(Main().start())
File "udocker", line 6638, in __init__
Config().user_init(self.cmdp.get("--config=", "GEN_OPT"))
File "udocker", line 322, in user_init
if self._read_config(config_file):
File "udocker", line 298, in _read_config
data = cfile.getdata()
File "udocker", line 911, in getdata
buf = filep.read()
OSError: [Errno 14] Bad address

I'm having trouble interpreting it, does this look familiar to anyone?

I noticed udocker help and udocker --help also generate the same error so seems likely there is a step I'm missing on setup; any ideas?

Please try to remove all your exports, keep only UDOCKER_DIR
export UDOCKER_DIR=/tmp

Hi, @skilbjo. If you are looking for an easy use of udocker in AWS Lambda, please consider taking a look at SCAR: https://github.com/grycap/scar
Regards,
Germán.

Thanks for the help @jorge-lip and @gmolto  !!!!
I was actually able to get it working. Well, almost- I still need to be able to run the container I download, seems like I need to also get the udocker-1.1.0-RC2.tar.gz from https://github.com/grycap/scar/tree/master/lambda. The current issue I have is
export UDOCKER_DIR=/tmp

cd /tmp

curl https://raw.githubusercontent.com/grycap/scar/master/lambda/udocker >/tmp/udocker
chmod +x udocker

udocker pull quay.io/skilbjo/iris:x86
udocker create --name=iris quay.io/skilbjo/iris:x86
udocker setup --execmode=F1 iris
udocker run --nosysdirs iris cat /etc/os-release

Error: no libfakechroot found for this host os

It seems the current indigo-dc/udocker/master and indigo-dc/udocker/devel branches are not working with AWS Lambda at the moment (the failure, always with the stack trace I posted above). The way I was able to get it working was with this command:
curl https://raw.githubusercontent.com/grycap/scar/master/lambda/udocker >/tmp/bin/udocker

udocker not working on AWS Lambda might not be an issue for you guys though, if so feel free to close/resolve this issue.
I will post a working script (in comparison with the original script I opened the issue with) however once I get it working. It may help any newcomers who saw the blog post on HackerNoon but was not able to get it working (since that blog post is now out of date).

@jorge-lip and @gmolto might you have any idea what this Error: patchelf executable not found error means?
I seem to be making it quite far with this script, except for this particular error:
Warning: running as uid 0 is not supported by this engine
Warning: non-existing user will be created
Error: patchelf executable not found

here is my script
export UDOCKER_DIR=/tmp
export UDOCKER_BIN=/tmp/bin
export UDOCKER_LIB=/tmp/udocker_dir/lib

cd /tmp

# binary download from https://raw.githubusercontent.com/grycap/scar/master/lambda/udocker
ln -s /var/task/udocker /tmp/bin/udocker || :

# binary download from https://github.com/grycap/scar/blob/master/lambda/udocker-1.1.0-RC2.tar.gz
tar --directory='/tmp' -xf /var/task/udocker-1.1.0-RC2.tar.gz

udocker install

udocker pull busybox # quay.io/skilbjo/iris:x86
udocker create --name=iris busybox # quay.io/skilbjo/iris:x86
udocker setup --execmode=F1 iris

udocker run -v /dev -v /proc -v /etc/hosts --nosysdirs \
  iris ls /etc

Just a quick message. So it looks like it can work without issue (no patchelf error) with the grycap/cowsay image referred to in the example in https://github.com/grycap/scar. I noticed that grycap/cowsay is based on ubuntu:16.04 rather than busybox/alpine linux.
This may be an issue related to the problems with busybox alluded to in @gmolto 's excellent write up here: https://github.com/indigo-dc/udocker/issues/76 and https://github.com/grycap/scar/issues/28
However !!
I also noticed this bit of code
(from https://github.com/grycap/scar/blob/master/lambda/udocker )
    def select_patchelf(self):
        """Set patchelf executable"""
        conf = Config()
        arch = conf.arch()
        if arch == "amd64":
            image_list = ["patchelf-x86_64", "patchelf"]
        elif arch == "i386":
            image_list = ["patchelf-x86", "patchelf"]
        elif arch == "arm64":
            image_list = ["patchelf-arm64", "patchelf"]
        elif arch == "arm":
            image_list = ["patchelf-arm", "patchelf"]

which calls
(from https://github.com/grycap/scar/blob/master/lambda/udocker#L1134 )
    def arch(self):
        """Get the host system architecture"""
        arch = ""
        try:
            machine = platform.machine()
            bits = platform.architecture()[0]
            if machine == "x86_64":
                if bits == "32bit":
                    arch = "i386"
                else:
                    arch = "amd64"
            elif machine in ("i386", "i486", "i586", "i686"):
                arch = "i386"
            elif machine.startswith("arm"):
                if bits == "32bit":
                    arch = "arm"
                else:
                    arch = "arm64"

somehow, that is not returning amd64, for which surely the AWS machines running lambda are.
for reference, $(uname -a) in AWS Lambda returns this
Linux ip-10-34-75-211 4.9.81-35.56.amzn1.x86_64 #1 SMP Fri Feb 16 00:18:48 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
So at least the patchelf error could be resolved by modifying the udocker code to force-jam the lambda machines to return amd64 when run on lambda.

Considering I was able to get this to work with ubuntu, and there is already an open issue related to Alpine Linux, I'm gonna close this.
Also, I'm gonna take a look at using https://github.com/bitnami/minideb as an alternative base image that will probably work with udocker + AWS Lambda

Also, I was actually able to get past the patchelf error on the busybox/alpine image by just changing my dockerfile slightly: https://github.com/skilbjo/iris/blob/master/deploy/default/Dockerfile.arm
FROM resin/armhf-alpine:3.5

ENV QEMU_EXECVE=1

COPY deploy/qemu/qemu-arm-static /usr/bin/

RUN ["qemu-arm-static","/sbin/apk","-U","add","patchelf"]

But alas, I am now getting the Error: command not found or has no execute bit set: ['usr/local/deploy/bin/run-job'] error referred to in the issues linked to this ticket; so I'd recommend minideb as an alternative to alpine linux

Hi @skilbjo, we had the same issues when running alpine images.
Minideb is a good option to make small images, but if you want to go one step further you can check this project: https://github.com/grycap/minicon.
It allows to reduce even more the container image by removing all the libraries not used by the process inside the container.
Also, in the following weeks this functionality and more are going to be included in the scar framework: https://github.com/grycap/scar


docs: storybook: remove vue-specificity & add addons

Works only in conjunction with (i.e. contains) #114; please look at that first.
Gives us usable a11y & knobs in docs which work on the stories from vue-components (see chromatic).
I added a temporary dummy commit to show the build results to verify that dc9602e is truly needed (see absence of stories.json) - I don't understand why storybook does not automatically do that for us.
Bug: https://phabricator.wikimedia.org/T257824

Rebased.


The plugin upi_pay uses a deprecated version of the Android embedding.

The plugin upi_pay uses a deprecated version of the Android embedding.
The plugin upi_pay uses a deprecated version of the Android embedding.
To avoid unexpected runtime failures, or future build failures, try to see if this plugin supports the Android V2 embedding. Otherwise, consider removing it since a future release of Flutter will remove these deprecated APIs.
If you are plugin author, take a look at the docs for migrating the plugin to the V2 embedding:

Duplicate of #44


Cache sample sets

This PR caches the conversions between Symbolic/PointSet distributions to sample set distributions. This has two effects, this is now possible and makes sense:

I think it also increases performance by about 50% on my larger models.

[ ] Remove accidentally checked in profile
[ ] Test to see how much this increases performance based on the malaria consortium.

I also include a performance test I'm working with. What's the protocol with these tests? Should I keep this one as a test case?

Codecov Report

Merging #1441 (d85838e) into develop (5b68c48) will increase coverage by 6.39%.
The diff coverage is n/a.

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

@@             Coverage Diff             @@
##           develop    #1441      +/-   ##
===========================================
+ Coverage    58.76%   65.16%   +6.39%
===========================================
  Files           89       89
  Lines         4921     4934      +13
  Branches       840      842       +2
===========================================
+ Hits          2892     3215     +323
+ Misses        1978     1682     -296
+ Partials        51       37      -14

Impacted Files
Coverage Δ

src/library/registry/helpers.ts
93.97% <0.00%> (+1.20%)
:arrow_up:

src/public/SqProject/index.ts
86.95% <0.00%> (+3.10%)
:arrow_up:

src/dist/SampleSetDist/SampleSetDist.ts
67.53% <0.00%> (+5.44%)
:arrow_up:

src/dist/DistOperations/index.ts
69.53% <0.00%> (+5.46%)
:arrow_up:

src/PointSet/Continuous.ts
53.76% <0.00%> (+9.04%)
:arrow_up:

src/XYShape.ts
57.72% <0.00%> (+11.07%)
:arrow_up:

src/fr/genericDist.ts
59.70% <0.00%> (+11.94%)
:arrow_up:

src/dist/BaseDist.ts
42.85% <0.00%> (+14.28%)
:arrow_up:

src/dist/SymbolicDist.ts
51.32% <0.00%> (+15.85%)
:arrow_up:

src/dist/PointSetDist.ts
65.06% <0.00%> (+20.90%)
:arrow_up:

... and 4 more

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

This PR seems to mildly increase performance actually! Not enough cache hits or something? Worked that out by looking at the profiling artifacts provided by GitHub actions.


Misc syntax improvements with respect to the usage of backends

Old syntax is still working, but with deprecation warnings.

Fixes isse #77

Codecov Report
Base: 91.68% // Head: 90.61% // Decreases project coverage by -1.07% :warning:

Coverage data is based on head (4d5c389) compared to base (2646a3a).
Patch coverage: 15.62% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #83      +/-   ##
==========================================
- Coverage   91.68%   90.61%   -1.08%
==========================================
  Files           7        8       +1
  Lines        2358     2386      +28
==========================================
  Hits         2162     2162
- Misses        196      224      +28

Impacted Files
Coverage Δ

src/PartitionedArrays.jl
0.00% <0.00%> (ø)

src/SequentialBackend.jl
88.59% <12.50%> (-9.93%)
:arrow_down:

src/MPIBackend.jl
88.05% <16.66%> (-4.58%)
:arrow_down:

src/Interfaces.jl
91.00% <100.00%> (ø)

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.


Release/1.4.X To Release/1.5.X

This PR is meant to merge in changes from 1.4.x into 1.5.x

Replaced by a merge PR https://github.com/corona-warn-app/cwa-app-android/pull/1218


User CRUD tests "before all" hook

Hello all, am using Node Version: 6.12.2
When I tried to run grunt test  it didn't run.
As I was required to update the mock-fs version from 3.4.0 to 3.12.1 , reason it do not support node version: 6.12.2.
After updating I tried to run grunt test I got  error:
User CRUD tests "before all" hook:
     TypeError: Cannot read property 'isDirectory' of undefined
      at resolveSync (/home/amitsin6h/meanjs/MeanjsApp/node_modules/serve-favicon/index.js:174:11)
      at favicon (/home/amitsin6h/meanjs/MeanjsApp/node_modules/serve-favicon/index.js:61:12)
      at Object.module.exports.initMiddleware (/home/amitsin6h/meanjs/MeanjsApp/config/lib/express.js:68:11)
      at Object.module.exports.init (/home/amitsin6h/meanjs/MeanjsApp/config/lib/express.js:227:8)
      at Context.<anonymous> (/home/amitsin6h/meanjs/MeanjsApp/modules/users/tests/server/user.server.routes.tests.js:30:19)
      at Hook.Runnable.run (/home/amitsin6h/meanjs/MeanjsApp/node_modules/mocha/lib/runnable.js:196:15)
      at next (/home/amitsin6h/meanjs/MeanjsApp/node_modules/mocha/lib/runner.js:258:10)
      at Immediate.<anonymous> (/home/amitsin6h/meanjs/MeanjsApp/node_modules/mocha/lib/runner.js:275:5)
      at runCallback (timers.js:672:20)
      at tryOnImmediate (timers.js:645:5)
      at processImmediate [as _immediateCallback] (timers.js:617:5)

Why not use node LTS or Latest version ?


Parse app image when deleting an app (#1744)

Checklist

[x] The title of this PR would make a good line in Acorn's Release Note's Changelog
[x] The title of this PR ends with a link to the main issue being address in parentheses, like: This is a title (#1216). Here's an example
[x] All relevant issues are referenced in the PR description. NOTE: don't use GitHub keywords that auto-close issues
[x] Commits follow contributing guidance
[ ] Automated tests added to cover the changes. If tests couldn't be added, an explanation is provided in the Verification and Testing section
[x] Changes to user-facing functionality, API, CLI, and upgrade impacts are clearly called out in PR description
[x] PR has at least two approvals before merging (or a reasonable exception, like it's just a docs change)

Issue: https://github.com/acorn-io/acorn/issues/1744

This is actually a problem with the CRD generation. I didn't understand how the recursion stopped. That is, a job has a container has a sidecar, but a sidecar is a container which can have a sidecar. This infinite recursion wasn't handled by the CRD generation.
This proposed fix is only a work around. I will try implementing the fix to CRD generation so this will not be necessary.


Update to 1.21.3, I think?

Updated to 1.21.3.
Added Recipes to datagen, because I didn't want to deal with it.
Fell in love with cicada.
Switched to official maven for cooldown coordinator rather than enjarai's mirror
Added and then removed ModItemTagGenerator.

Oops, I don't know how to fix this PR. Going to open a new one.


Clear output console (Button + clear on Play)

This PR resolves Feature #1825

Added a "Clear" button above output console
Added a Setting option to automatize clearing when Play is started

Thanks.

Cool. Thank you.


Add runtime-class option to remainder of benchmarks

Add runtimeClassSupport (stable as of Kubernetes 1.20) to the remaining benchmarks that do not support this option.
Closes https://github.com/cloud-bulldozer/benchmark-operator/issues/501

Can one of the admins verify this patch?

/rerun all

Results for ec2_jjb

Test
Result
Retries
Duration (HH:MM:SS)

test_backpack
Pass
0
00:04:17

test_byowl
Pass
0
00:01:08

test_fiod
Pass
0
00:20:34

test_fs_drift
Pass
0
00:04:41

test_hammerdb
Pass
0
00:03:57

test_iperf3
Pass
0
00:06:46

test_kubeburner
Pass
0
00:18:54

test_pgbench
Pass
0
00:03:13

test_smallfile
Pass
0
00:05:12

test_stressng
Pass
0
00:02:03

test_sysbench
Pass
0
00:01:14

test_uperf
Pass
0
00:24:06

test_vegeta
Pass
0
00:04:14

test_ycsb
Pass
0
00:01:57

test_scale_openshift
Pass
0
00:09:20

@RobertKrawitz minor nit but can you standardize the optional argument explanation.
i.e.:
The option **runtime_class** can be set to specify an optional
runtime_class to the podSpec runtimeClassName.  This is primarily
intended for Kata containers.

is worded slightly differently in some of the docs. Standardizing on one explanation of the option would be preferable. Otherwise it looks good!

@RobertKrawitz minor nit but can you standardize the optional argument explanation.
i.e.:
The option **runtime_class** can be set to specify an optional
runtime_class to the podSpec runtimeClassName.  This is primarily
intended for Kata containers.

is worded slightly differently in some of the docs. Standardizing on one explanation of the option would be preferable. Otherwise it looks good!

I agree; the problem is that the arguments are structured differently in different benchmarks.


AGS UI/UX Overhaul

Our AdvancedGameScreen (AGS) is - as are many aspects of Terasology - optimized for being used by our developers to test changes. It is less suitable for (even advanced) players. For them, the amount of modules can be overwhelming and it's not clear enough, which modules are incompatible, what the colors tell you, and so on.
Also, for instance the search filters are easy to overlook and are probably (and sadly) not used very much.
Hence, the AGS needs a general UI/UX overhaul ( pinging our UI/UX expert @stefaniamak for some ideas and maybe sketches 🎨 ).
At the moment, @DarkWeird is working towards cleaning up the game start stages, which could affect the AGS and change its contents or how it's currently used. This is why, I propose to discuss ideas for a UI/UX overhaul here and align it with the stage changes in the near-future as they come up.
In the meantime, though, maybe we can identify and pick some low-hanging fruit, though, to improve the situation and guide the player a bit better in the AGS as it currently is.
(Potentially) Related Issues:
#4309
#4129
#4068

I dreaming about using TWL there.
Map objects to widgets via TypeWidgetHandlers :3
Decompose code, widget parts


Can't SSH into boxcutter/ubuntu1604

Anyone else having problems with this? Seen on two host computers (and I'm pretty sure an older version of the box used to work on one of the computers).
Firstly, VM boot is really slow - around 5 mins - so Vagrant times out trying to connect (yes, I know there is a setting to increase the time out).
But then, once booted,
 ssh -p 2222 vagrant@localhost

just times out.
On one of the machines, I was able to open a direct terminal to the box, shell in with user/pass vagrant/vagrant and run the following fine
ssh vagrant@localhost

Which suggests SSHD is running fine but it's a networking problem. The networking settings for port forwarding in VirtualBox seem fine to me. So I'm a bit stuck .... Thanks!

See https://github.com/chef/bento/issues/701

So, that's saying VirtualBox 5.1.8 is broken and I should go back to 5.1.6?
I'm on V4.1.44 and V5.0.16 :-D

5.1.6 is the golden version that works. Check if your NIC has the cable connected.

I don't think it's my VirtualBox. Both my host boxes can run boxcutter's debian box fine, so it's something special about this ubuntu box.

Let's try and narrow this down - can you confirm that the latest version of boxcutter/ubuntu1604 works for you?

When a box doesn't start, the first thing you should try is enable GUI and check the console.

Obviously broken box.
Ping @annawake

I did open a console, as said in my first post. I saw the same kind of message, but for me if I leave it for 5 mins it clears and the box boots anyway. Well, it boots fine for logging in at the console, but I can't log in over SSH which is the problem.
I suspect it's a broken box to, hence this issue, but when it's volunteers doing open source (Thanks!) I like to really check things out before reporting so as not to be a pain.

Sorry, I might have replied a bit too quickly, felt like the hundred issue about "can't connect to SSH with VirtualBox" (reported on https://github.com/chef/bento).

Ah haa! Know I said this used to work on my laptop? Fortunately, it turns out I haven't cleared out my vagrant boxes. So I could just look and see what version I had before.  By adding:
config.vm.box_version = "2.0.18"

I can make this box work - on both my host machines. So it's something since this version that has broken.

I get this issue to, running VMware Fusion, and on VirtualBox (running Mac OS X).

I'm seeing this issue as well. Virtualbox on OS X.
Ubuntu 14 and CentOS work fine.

Just tried the latest version of ubuntu1604-desktop and I see this:

That command times out, and SSH fails. This ties in with the symptoms I saw of a very long boot and then no external network interfaces once booted.
Again,
	config.vm.box = "boxcutter/ubuntu1604-desktop"
	config.vm.box_version = "2.0.18"

boots fine.

I've fixed this by adding the following to the VagrantFile:
# Bugfix for "Cable connected: off"
config.vm.provider :virtualbox do |vm|
    vm.customize [
        "modifyvm", :id,
        "--cableconnected1", "on",
    ]
end

Solution found at: https://lists.debian.org/debian-cloud/2016/09/msg00051.html

It looks like VirtualBox 5.1.10 includes a fix for this issue, give it a go and let everyone know how it works for you.

FWIW, I was using VirtualBox 5.0.30 with box-cutter/ubuntu1204 and had the same lack of networking issue, for which the "cable connected" fix worked like a charm. After upgrading VB to 5.1.10, the config fix was thankfully no longer needed.

Upgrading at least one of my VirtualBoxes would be a right pain, as it comes from Linux packages. Also, as I'm trying to write stuff for others I really don't want to have to add a big warning saying "Must be 5.1.10!".
It seems like there must be something else going on here. Older versions of the same Boxcutter boxes work fine on my machines with older VirtualBox versions. That implies something else is going on - could it be to do with the version of the software used to build the box?

Yes it depends on the version used for building.
On Nov 30, 2016 22:13, "James"<EMAIL_ADDRESS>wrote:

Upgrading at least one of my VirtualBoxes would be a right pain, as it
comes from Linux packages. Also, as I'm trying to write stuff for others I
really don't want to have to add a big warning saying "Must be 5.1.10!".
It seems like there must be something else going on here. Older versions
of the same Boxcutter boxes work fine on my machines with older VirtualBox
versions. That implies something else is going on - could it be to do with
the version of the software used to build the box?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/boxcutter/ubuntu/issues/82#issuecomment-263996980,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAiCgzTOBWZglbvy-tFSqSdVufI-tX-Eks5rDebugaJpZM4KlHTO
.

In that case, when can we expect new versions of the Boxcutter boxes built using the latest version of Virtualbox? (I am unfamiliar with Boxcutter's release schedule.) I can test them on my older versions of VB and if that worked, that would be much better - the BC boxes would work on all recent versions of VirtualBox. Thanks,

There was a box release yesterday with the 5.1.10 guest additions.
I don't know why, but so far what I've heard is that people can't use these boxes with VirtualBox 5.0.x - there is the problem you mention about ssh. The issue could be with VirtualBox or vagrant or maybe even packer, that's currently unclear.
Atlas does not support multiple versions of providers per box, so boxcutter will only build boxes with the latest 5.1.x guest additions, and we hope that there is some solution that comes about that allows people using VirtualBox 5.0.x to use the boxes.

I tested V2.0.25 of boxcutter/ubuntu1604:

Mac - VirtualBox 5.0.16 - broken.
Linux - VirtualBox 4.1.44 - broken.

I tried adding the "--cableconnected1", "on", fix:

Mac - VirtualBox 5.0.16 - works.
Linux - VirtualBox 4.1.44 - Works.

So, at least there is a workaround.
But it's worth noting:

people using VirtualBox 5.0.x to use the boxes

It's not just 5.0.x, it seems to be all pre 5.1.10. Which given all the Linux packaged versions, must be a lot. All Ubuntu versions are broken, for instance: http://packages.ubuntu.com/xenial/virtualbox

so boxcutter will only build boxes with the latest 5.1.x guest additions

Not an expert on this, but Is there scope here to deliberately build boxes with an older version of VirtualBox? If we seem to be saying VB 5.1.6 was fine and then VB introduced a bug, wouldn't building against older versions make sense until the bug is fixed? 2.0.18 of the box works fine on my machines, so it's something about the builders version of VB not the users version. (Presumably parallels and vmware_desktop won't care what version of VB is used, so Atlas only allowing one version shouldn't be a problem.)
Anyway, thanks - I appreciate how frustrating it must be having a platform bug like this pop up :-(

I can confirm that my fix (https://github.com/boxcutter/ubuntu/issues/82#issuecomment-260902424) is no longer needed when working with virtualbox v5.1.10 & vagrant 1.9.0.

@annawake, this is still an issue for me with VMware Fusion. Can we reopen this?

Can I just say, my personal view is that forcing users to upgrade to the latest VirtualBox isn't really a solution in my eyes. Some users won't want to or won't be able to. But if that's the line the project is taking I understand, thanks for discussing this.

Both vagrant ssh and ssh -p 2211 vagrant@localhost works fine for me with Fusion Pro 8.5.3, Vagrant 1.9.1 and boxcutter/ubuntu16.04 2.0.26. 👍


changelog updates for v2.5.0-rc1

UBAN needs a manual update

UBAN needs a manual update

We can publish the UTF8ONLY spec once this release goes out and is on the testnet.

We can publish the UTF8ONLY spec once this release goes out and is on the testnet.

suresure, sounds good

suresure, sounds good


Create a way to either check parallel worked, or warn if it was specified and it failed

This ticket https://github.com/numba/numba/issues/2460 has a valid sample that doesn't have much performance change under the parallel target likely due to lack of work. It raises the question more generally of:

If a user has specified parallel=True how does the user know if the function was able to be transformed and executed in parallel, given fall back to njit scalar execution is silent.

Options to resolve this:

Provide a function to check if a function will execute using the parallel scheduler.
Add a warning if parallel=True is requested but the function could not be transformed for parallel execution.
Both of the above.

I like both, since option 1 is handy for unit tests, but option 2 is better for users

#2493 adds in a warning mechanism. Checking a function will execute with the parallel scheduler at present requires inspection of the IR for calls to @do_scheduling. Locally I've put in a CPUDispatcher.is_parallel property such that users could do:
import numpy as np
from numba import njit
@njit(parallel=True)
def foo(A):
  return A + 1

B = np.ones((10,))
res = foo(B)
assert foo.is_parallel

but need to convince myself that a false positive can't appear.


Fix 263new

Pull Request Checklist:

[ ] This PR addresses an already opened issue (for bug fixes / features)
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] HISTORY.rst has been updated

After merging to master and before closing the branch:

[ ] bumpversion (minor / major / patch) has been called on master branch
[ ] Tags have been pushed

What kind of change does this PR introduce?

Does this PR introduce a breaking change?

Other information:

This PR fixes #xyz

Note:This is just a test to see if builds pass will push to official 'fix_263' if all is ok

The new xarray v0.14.0 is breaking utils.percentile_doy()'s  grouby,reduce() call - bug fix coming : see https://github.com/pydata/xarray/issues/3402.
The new version also seems to break ensembles.create_ensemble() but this I have managed to fix it (should work in 0.13.0 as well)
We could possibly pin the xarray version to 0.13.0 while waiting for the bug fix to appear on pypi?Opinions? @Zeitsperre @huard

I can go forward with this and tag the release as 0.11.

I've got a few more changes to push ....

Pull Request Test Coverage Report for Build 1137

81 of 129 (62.79%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.09%) to 90.355%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

xclim/ensembles.py
81
129
62.79%

Totals

Change from base Build 1124:
-2.09%

Covered Lines:
1452

Relevant Lines:
1607

💛  - Coveralls

@huard can you confirm that this addresses your comments in https://github.com/Ouranosinc/xclim/pull/264?  Sorry for the hassle but the original fix_263 branch is really messed up and I seem unable to push these additions/mods without wasting a bunch of time

Do you need this soon ? I can only take a cursory look before Nov 11.
I spotted a couple of syntax issues in the docstrings.
My main issue were the graphics, which is now fixed. I suggest you don't wait for my review and go ahead with the merge. An upcoming task anyway is to create an "Ensemble" class, and I expect that clustering could be a method from that class, which will give us a chance to do some further refactoring if needed.