CI: Setting up VM on scality.cloud

Support scality.cloud
Applying changes to support both rackspace and scality.cloud providers

iptables-services should not prevent the merge I believe, but It's a concern to be addressed in the product itself.

iptables-services should not prevent the merge I believe, but It's a concern to be addressed in the product itself.

I somewhat disagree. This PR apparently changes the environment in which tests are executed, so it should fix the tests to work in this new environment. Admittedly it does so right now, but using a work-around rather than a proper fix.

Made the requested changes.
I'm waiting for the test results on a VM running on scality.cloud, do not approve/merge until this build pass: https://eve-dev.devsca.com/github/scality/metal-k8s-thomas/#/builders/5/builds/14

Ahh not understanding well what Thomas said, I was on the test suite not MetalK8s that we disabled iptables. Yes agree to remove iptables at all, it was a work around for rackspace.

The build passed on scality.cloud as well so if there's no comment/request for changes we can merge it.

@tcarmet In general we let the author of the PR merge after approvals etc :)

Oh. In that case...


Datatype QASMM8 is taking more time than F32 in ACL 19.05

Output of 'strings libarm_compute.so | grep arm_compute_version':
arm_compute_version=v19.05 Build options: {'embed_kernels': '1', 'gles_compute': '1', 'logging': '1', 'arch': 'arm64-v8a', 'opencl': '1', 'neon': '1', 'asserts': '1', 'debug': '0', 'os': 'android', 'Werror': '1'} Git hash=4ba87dbdc3b22220eba4a792c1f5c87e7a88c7af
Platform:
Operating System:
Android 8.1
I am trying to move my graph to QASSM8 datatype from F32 and assuming that it should be faster with slight impact on accuracy. Contrary to this QASSM8 is taking more time than F32.
I have also tried with the example graph_mobilenet_v2 provided with ACL. I have calculated elapsed time for one run() and below are the results.
./graph_mobilenet_v2 --layout=NCHW --model-id=0 --type=F32 --target=CL --tuner-file=/data/cnn/tuner.file --enable-tuner --tuner-mode=Exhaustive
64 ms
./graph_mobilenet_v2 --layout=NCHW --model-id=0 --type=QASYMM8 --target=CL --tuner-file=/data/cnn/tuner.file --enable-tuner --tuner-mode=Exhaustive
120ms
Note
I have excluded the time for setup and initial 5 runs. I will execute run() for 100 times and then taking an average. Below is my modified do_run() function of graph_mobilenet_v2:
void do_run() override
{
struct timespec start, stop, start1;
	graph.run();
	graph.run();
	graph.run();
	graph.run();
	graph.run();

	clock_gettime(CLOCK_MONOTONIC, &start1);
	for(int i=0; i < 100; i++) {
		graph.run();
        }
	clock_gettime(CLOCK_MONOTONIC, &stop);

	std::cout << "run time for 100 iteration: " << (stop.tv_sec - start1.tv_sec) * 1e3 + (stop.tv_nsec - start1.tv_nsec) / 1e6 << " ms\n" << std::endl;
}

Questions

Why QASSYM8 is taking more time than F32? Is this expected? Am I missing something?

Let me know if you want more details.

Hi @krgor,
thanks for using ACL! :)
I had a look at the command used and I noticed the data layout passed to the example is not the recommended one. Rather than using NCHW, can you try to use NHWC?
Also, could you tell us what GPU are you running on? For Mali-G71 and Mali-G76, I would expect better performance for QASYMM8 since we can use a dedicated instruction for this computation. In case of other GPUs, this instruction may not be available so the uplift could be significant reduced or/and similar to F32. The reason behind this reduced uplift is behind the GEMMLowp implementation would require to store the partial accumulations on int32 variables.
If you are profiling ACL, I can also suggest to have a look at our latest webinar on performance analysis. Below you can find the link:
https://pages.arm.com/webinar-performance-analysis-for-embedded-deep-learning-recording.html?_ga=2.201001129.646105822.1561922123-1818525636.1557823069
The webinar presents also the experimental benchmark test suite for ACL on the Linaro repo which can be used to better profile your networks.
Hope this can help.
Don not hesitate to write if you require any further information or you have feedback for our experimental benchmark test suite.
Thanks,
Gian Marco

@gmiodice ,
Thanks for you suggestions.
The Board has Mali-T860 MP4 GPU.
Below are the results with various combinations of data type and layout.
./graph_mobilenet_v2 --layout=NCHW --model-id=0 --type=F32 --target=CL --tuner-file=/data/cnn/tuner.file --enable-tuner --tuner-mode=Exhaustive
64 ms
./graph_mobilenet_v2 --layout=NCHW --model-id=0 --type=QASYMM8 --target=CL --tuner-file=/data/cnn/tuner.file --enable-tuner --tuner-mode=Exhaustive
120ms
./graph_mobilenet_v2 --layout=NHWC --model-id=0 --type=F32 --target=CL --tuner-file=/data/cnn/tuner.file --enable-tuner --tuner-mode=Exhaustive
57ms
./graph_mobilenet_v2 --layout=NHWC --model-id=0 --type=QASYMM8 --target=CL --tuner-file=/data/cnn/tuner.file --enable-tuner --tuner-mode=Exhaustive
97ms
NHWC seems to be better than NCHW but QASYMM8 is worse than F32. My understanding is that using QASYMM8 may take similar time compared to F32 if some of the operations are not supported by GPU but in any case it should not be taking more time than F32. Am I missing something?
Thanks for sharing webinar link. I will go through it and let you know if I will any more doubts.

@gmiodice ,
I have gone through the seminar and as instructed, I have tried with benchmarking examples. Below is the output which explains my concern. This clearly shows that qasymm8 has very bad performance than f32.
QASYMM8 data-type
Version = arm_compute_version=v0.0-unreleased Build options: {'embed_kernels': '1', 'arch': 'arm64-v8a', 'opencl': '1', 'neon': '1', 'examples': '1', 'debug': '0', 'os': 'android', 'Werror': '0'} Git hash=d5462ff012b3398addb1140d5c52c65ea16a27a5
CommandLine = ./benchmark_graph_mobilenet --example_args=--target=cl,--type=qasymm8,--enable-tuner,--tuner-mode=Exhaustive,--tuner-file=/data/cnn/benchmark_tuner.file --iterations=10
CL_DEVICE_VERSION = OpenCL 1.2 v1.r18p0-01rel0.022056fe7b169c7e0d8547fe2747032d
Iterations = 10
Running [0] 'Examples/benchmark_graph_mobilenet'
Threads : 1
Target : CL
Data type : QASYMM8
Data layout : NHWC
Tuner enabled? : true
Tuner mode : Exhaustive
Tuner file : /data/cnn/benchmark_tuner.file
Fast math enabled? : false
Wall clock/Wall clock time:    AVG=142448.4444 us, STDDEV=1.83 %, MIN=139293.0000 us, MAX=145891.0000 us, MEDIAN=141632.0000 us
Executed 1 test(s) (1 passed, 0 expected failures, 0 failed, 0 crashed, 0 disabled) in 10 second(s)
F32 data-type
Version = arm_compute_version=v0.0-unreleased Build options: {'embed_kernels': '1', 'arch': 'arm64-v8a', 'opencl': '1', 'neon': '1', 'examples': '1', 'debug': '0', 'os': 'android', 'Werror': '0'} Git hash=d5462ff012b3398addb1140d5c52c65ea16a27a5
CommandLine = ./benchmark_graph_mobilenet --example_args=--target=cl,--enable-tuner,--tuner-mode=Exhaustive,--tuner-file=/data/cnn/benchmark_tuner.file --iterations=10
CL_DEVICE_VERSION = OpenCL 1.2 v1.r18p0-01rel0.022056fe7b169c7e0d8547fe2747032d
Iterations = 10
Running [0] 'Examples/benchmark_graph_mobilenet'
Threads : 1
Target : CL
Data type : F32
Data layout : NHWC
Tuner enabled? : true
Tuner mode : Exhaustive
Tuner file : /data/cnn/benchmark_tuner.file
Fast math enabled? : false
Wall clock/Wall clock time:    AVG=71083.0000 us, STDDEV=3.88 %, MIN=65563.0000 us, MAX=74186.0000 us, MEDIAN=70181.0000 us
Executed 1 test(s) (1 passed, 0 expected failures, 0 failed, 0 crashed, 0 disabled) in 5 second(s)

Hi @krgor
This is expected because the GPU you are running the network has no support for the dot8 instruction. This instruction is present in mali-g71, mali-g76 and mali-g77.
https://developer.arm.com/ip-products/graphics-and-multimedia/mali-gpus/mali-t860-and-mali-t880-gpus
https://www.arm.com/products/silicon-ip-multimedia/gpu/mali-g76


Twitter authorization not working

In the logic apps designer, when I try to use "When a new tweet is posted", I click Sign In, a popup window authorizes me, and then the "This logic app will connect to: Twitter" window gets a red exclamation point, and mouseover says "Failed with error: 'The browser is closed.'. Please sign in again."

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

ID: ae1a4055-33b7-eead-6eb4-0092b7112567
Version Independent ID: 7b866c4b-46d2-d420-278e-f49ec05e5d0d
Content: Create a function that integrates with Azure Logic Apps
Content Source: articles/azure-functions/functions-twitter-email.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

To be clear, on my twitter account, it shows "Microsoft Azure Logic Apps by Microsoft" being authorized. So the logic app/azure side is not remembering this authorization.

Hi @gwalters69
Thank you for your feedback! We will review and update as appropriate.

@gwalters69 Just following up to see if you had any additional details that you're able to share for us to repro the issue you saw, otherwise, we'll proceed to close this issue.

I keep "Authorizing" and "stay logged in" to my Twitter account.
I even made a different twitter account without 2 factor authentication, and I still see the same problems.
[cid:0f564e18-db00-48d2-b073-43072d493e5e]
Each of the "gwalters69 (twitter-x)" ... is another time I am authorizing logic apps to have permissions to talk to my twitter account.
Definitely getting tired of this.
Happy to do a screen share.
--George
Cell:<PHONE_NUMBER>

From: Mike Urnun<EMAIL_ADDRESS>Sent: Wednesday, December 12, 2018 10:58 PM
To: MicrosoftDocs/azure-docs
Cc: George Walters; Mention
Subject: Re: [MicrosoftDocs/azure-docs] Twitter authorization not working (#20342)
@gwalters69https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgwalters69&data=02|01||5cd874f00ecd46a12acd08d660af42c2|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636802703180066069&sdata=sSoHdWgf8qwpsNopd49kiEFcdB4RGa1nNtDQR0kX2vY%3D&reserved=0 Just following up to see if you had any additional details that you're able to share for us to repro the issue you saw, otherwise, we'll proceed to close this issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F20342%23issuecomment-446834693&data=02|01||5cd874f00ecd46a12acd08d660af42c2|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636802703180066069&sdata=%2BwQ%2F7smaaX%2FGjytIVhBiELRITnth%2BJJ6%2FCb%2BO6PgGqc%3D&reserved=0, or mute the threadhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKFsTLQSXAmsh_sAXAypkWKeJ_XxJA3wks5u4dBrgaJpZM4ZLsFL&data=02|01||5cd874f00ecd46a12acd08d660af42c2|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636802703180066069&sdata=dLx2lMYfBixV2JWema7YQCKcwqJSUwQq%2Bb%2Bmp2XcF3M%3D&reserved=0.

Hi @gwalters69
We'd like to work more closely with you to address the particular issue that you're encountering. Could you reach out to us via email at<EMAIL_ADDRESS>with your subscription ID as well as the URL of this issue included in your email?
I've gone through the steps of this doc again just to double check every step, and didn't run into any issues and the solution worked as expected for me:

Because we didn't identify any faulty step in this documentation and since this channel is for driving improvements towards MS Docs, we will now proceed to close this issue for now. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


[FeatureRequest] Run script upon start of application (for type status bar)

Please add the following Feature: Run the script when the application is launched (probably needed only for app type status bar).
When I start an application made with Platypus with the "status bar" UI, it does not call the script bound to the app. The script gets called after the user clicked on the status bar icon. I would like to let the script get executed already when the app gets started by the user.
For this purpose Platypus should have a new option "execute script upon start", which calls the script with an argument "start-up" or similar or maybe a configurable keyword as soon as the app created by Platypus gets executed.
(edit: removed app type droplet from request, it does not apply)

> When I start an application made with Platypus with the "status bar" or
"droplet" UI, it does not call the script bound to the app. The script gets
called after the user clicked on the status bar icon or dropped a file to
the droplet. I would like to let the script get executed already when the
app gets started by the user.
I don't understand how that would work with a droplet.  When the file is
dropped on the icon, the Platypus wrapper app is launched, and the filespec
of the dropped file is passed to your script as a command line parameter.
When your script starts, it reads its command line parameters and acts on
that information.  So the droplet wrapper app itself is not running until a
file is dropped on it.  It is not continuously running in the background
waiting for a file to be dropped on it.
...BC
--
@.***
Just another geek in Northern New England, USA.

It does already work with a droplet: The script gets called when the starts up. Thank you for pointing it out.
Do you see a possibility for a status bar app, did I miss something there as well?

> Do you see a possibility for a status bar app, did I miss something
there as well?
I have not used that particular feature of Platypus, but let's take a look
at the documentation.  For a status bar app, the Platypus wrapper app does
the following:

Creates a Status Item in the menu bar when the app is launched.
Every time the status item is clicked, the script is executed and its
text output shown line for line in a menu.
If a menu item is selected, the script is executed again with the
title of the selected item passed as an argument to the script.

So in step #2, your app is executed with no command line arguments
which is the signal to your app that it needs to output a set of one or
more menu lines.  It sounds like what you want is a new step, let's call it
"1a".  What is it that you want your script to do when it is called during
the initial menu setup phase?
...BC
--
@.***
Just another geek in Northern New England, USA.

My app shows predefined entries to mount a remote file system under a status bar icon. Clicking on one of them connects, resp. disconnects, when it is already connected. I would like to add a new feature to connect an entry upon start of the app. For this to work my script should be called when the app starts.

> I would like to add a new feature to connect an entry upon start of the
app. For this to work my script should be called when the app starts.
So to generalize, it sounds like the new feature for the "Status Menu"
interface would be adding a new start-up script configuration field.  When
the Platypus wrapper app launches a Status Bar Item, it instantiates the
status bar icon and service as before, but it would also have the option of
calling a start-up script to perform some initialization task, such as
starting up a daemon, before exiting.
I think providing a new Status Menu Interface configuration field for the
start-up script would be cleaner than requiring support for an additional
calling method to invoke the current menu-handling script.  For one thing
the new start-up field eliminates the possibility of breaking any existing
apps using the original interface for apps of this type.  Another
consideration is that the logic for the start-ip may be very different from
the menu handling logic, so the ability to have two different scripts
allows for a better separation of concerns.  But at the same time, there is
nothing to prevent the developer from using the same script for both
the start-up and menu-handling.  The developer is free to use whatever
command line options or parameter values to invoke the special start-up
mode, using that exclusively within the start-up configuration field.
Sounds like a plausible and potentially worthwhile use case.  But being
unfamiliar with the Platypus code base, I have no idea how complicated it
would be to add such a feature.  In particular I don't know if having to
now support two scripts for an app, instead of just one, would be
problematic for the package building phase.
...BC
--
@.***
Just another geek in Northern New England, USA.

I'm in the same boat as @phloggu: I'd like to start a service when the menubar item is launched.
Maybe I'll have to create a regular application as a startup helper, which, when executed, would run the service and start the menubar item.


Polkadot/qr-code scanner has lots of issues

When I tried to add an address from Nova wallet, this happened!

Its scanning capabilities are limited.
We need to delve into how popular wallets handle QR code information transmission.

@AMIRKHANEF  isn't it fixed?

yes, I guess all related issues has been fixed!


feat: Onboard Chicago Crime dataset

Description
Based on #
Note: It's recommended to open an issue first for context and discussion.
Checklist
Note: Delete items below that aren't applicable to your pull request.

[x] Please merge this PR for me once it is approved.
[x] If this PR adds or edits a dataset or pipeline, it was reviewed and approved by the Google Cloud Public Datasets team beforehand.
[x] If this PR adds or edits a dataset or pipeline, I put all my code inside  datasets/chicago_crime and nothing outside of that directory.
[x] This PR is appropriately labeled.

@adlersantos - closing this one as this branch is facing some issues . I have created a new PR #199 for the same dataset.


Help with customization and deployment to production

Hello, we want to understand who to contact to get help with deployment. The documentation is really low and we need some guidance to join the testing and dev. Proces as we want to be one of first to be running on type 1.
Thanks to pointing us to right direction

Hey. If you have specific questions feel free to ask them here and the relevant folks can answer.


Move benchui to snabb-lwaftr

benchui has been replaced by snabb-lwaftr benchui.

This PR solves issue https://github.com/Igalia/snabbswitch/issues/60


org.osmdroid.library.R cannot be imported

Issue Type
[ ] Bug
[ ] Build system related
Description and/or steps/code to reproduce the problem
Hello everyone,
I've got the following issue:
java.lang.NullPointerException: Attempt to invoke virtual method 'org.osmdroid.views.MapViewRepository org.osmdroid.views.MapView.getRepository()' on a null object reference         at org.osmdroid.views.overlay.infowindow.InfoWindow.<init>(InfoWindow.java:53)         at org.osmdroid.views.overlay.infowindow.BasicInfoWindow.<init>(BasicInfoWindow.java:54)         at org.osmdroid.views.overlay.infowindow.MarkerInfoWindow.<init>(MarkerInfoWindow.java:42)         at org.osmdroid.views.MapViewRepository.getDefaultMarkerInfoWindow(MapViewRepository.java:54)         at org.osmdroid.views.overlay.Marker.<init>(Marker.java:115)         at org.osmdroid.views.overlay.Marker.<init>(Marker.java:92)         at com.thumbUp.ui.map.ChildFragmentOsmMap.displayMarkers(ChildFragmentOsmMap.java:197)
in my procedure which is aim to display markers on the map:
`
private void displayMarkers(ListMapPointDTO mapPoints) {
List markers = poiMarkers.getItems();
if (mapPoints.getMapPointDTOList() != null) {
for (MapPointDTO mapPointDTO : mapPoints.getMapPointDTOList()) {
GeoPoint location = new GeoPoint(
mapPointDTO.getMapPointLocationModel().getLatitude(),
mapPointDTO.getMapPointLocationModel().getLongitude());
            boolean exist = false;
            for (Marker existingMarker : markers) {
                if (existingMarker.getPosition().equals(location)) {
                    exist = true;
                    break;
                }
            }

            if (!exist) {
                Marker marker = new Marker(osmMap);

                marker.setPosition(location);
                marker.setOnMarkerClickListener((marker1, mapView) -> {
                    Bundle bundle = new Bundle();
                    bundle.putString("id", mapPointDTO.getMapPointLocationModel().getMapPointId());
                    Navigation.findNavController(getActivity(), R.id.nav_host_fragment).navigate(R.id.nav_point_details, bundle);
                    return true;
                });
                marker.setAnchor(Marker.ANCHOR_CENTER, Marker.ANCHOR_BOTTOM);
                poiMarkers.add(marker);
            }
        }
    }

    poiMarkers.invalidate();
}`

After my inspection it appeared that Android Studio cannot resolved R in org.osmdroid.library.R in class MapViewRepository.java at org.osmdroid.views.MapViewRepository.getDefaultMarkerInfoWindow(MapViewRepository.java:54)
Environment
If it's a bug, version(s) of android this affects:
All
Version of osmdroid the issue relates to:
6.1.5
Could you give me any advice how to resolve it?
Thank you in advance

Do you always get it?
For me it is an intermittent (rare) error on which I spent puzzling quite some time very recently... I blamed my order of creation (in onCreate of my activity) of mapview and its overlay (with the markers).
But the error reappeared in my testing.  It is occurring usually on switching out off my activity.

It rather looks like you create infowindows with a null MapView:
java.lang.NullPointerException: Attempt to invoke virtual method 'org.osmdroid.views.MapViewRepository org.osmdroid.views.MapView.getRepository()' on a null object reference

null MapView

at org.osmdroid.views.overlay.infowindow.InfoWindow.<init>(InfoWindow.java:53)
mMapView.getRepository().add(this);

in the constructor of InfoWindow

Why does this happen?...

Well, finally I've found the reason. The layout which implies also the fragment class connected to it were changed before creating markes on the map. So it caused NullPointerException due to MapView didn't exist in a new View. Just a mistake in a program's logic.
Thank you for your help.


Stop vendoring libraries

we have other dependencies, and I'm quite certain this is a historical
artifact.
It's also less code to maintain.

We need those for bspacking

@anmonteiro I thought we made some modifications to Easy_format but if the tests pass then that must not have been the case.
@chenglou do we actually need these two for bspacks? I thought these ones would be the most straightforward ones to bspack but I could be wrong.
@jaredly's diff for using the esy sources during bspacks should cover this. Let me know if I'm not considering something.
One nice thing about vendoring cmdliner though is that it ensures you never get a version conflict when someone else depends on a different version range of cmdliner (since it's such a ubiquitous package - that's one nice perk of vendoring).

I'll leave this open for now because I think it'd be a nice changeset – less maintenance burden. Feel free to close if you think it's not that useful.

Yeah, I'm torn about this one @anmonteiro - I think it shouldn't be a burden for bs-packs with @jaredly's changes (we could confirm) but on the other hand vendoring very commonly used libraries can help avoid package manager conflicts.


[WFLY-4974] XAResource of ActiveMQ does not implement org.jboss.tm.XA…

…ResourceWrapper
fix typo in the service file name so that the service is properly loaded
to provide the correct implementation.
JIRA: https://issues.jboss.org/browse/WFLY-4974

Linux Build 7113 is now running using a merge of d0fcddea91fcb13ea4696fb4020d3db6e8107775

Windows Build 2165 is now running using a merge of d0fcddea91fcb13ea4696fb4020d3db6e8107775

Linux with security manager Build 1986 is now running using a merge of d0fcddea91fcb13ea4696fb4020d3db6e8107775

Linux with security manager Build 1986 outcome was SUCCESS using a merge of d0fcddea91fcb13ea4696fb4020d3db6e8107775
Summary: Tests passed: 877, ignored: 284 Build time: 0:06:54

Linux Build 7113 outcome was FAILURE using a merge of d0fcddea91fcb13ea4696fb4020d3db6e8107775
Summary: Tests failed: 3 (3 new), passed: 2952, ignored: 355 Build time: 0:46:24
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ejb.mdb.cdi.MDBRAScopeCdiIntegrationTestCase.testMe: <no details avaliable>

org.jboss.as.test.integration.ejb.security.callerprincipal.GetCallerPrincipalTestCase.testStatelessLifecycle: <no details avaliable>

org.jboss.as.test.integration.messaging.mgmt.JMSQueueManagementTestCase.testMessageRemoval: <no details avaliable>

Windows Build 2165 outcome was FAILURE using a merge of d0fcddea91fcb13ea4696fb4020d3db6e8107775
Summary: Tests failed: 2, passed: 2953, ignored: 355 Build time: 0:46:40
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.clustering.xsite.XSiteSimpleTestCase.testPutRelayedToBackups: <no details avaliable>

org.jboss.as.test.manualmode.ejb.client.cluster.EJBClientClusterConfigurationTestCase.testServerToServerClusterFormation: <no details avaliable>


Support the --create-dirs option

It would be nice if Cake.Curl exposed the --create-dirs option when downloading a file from a remote host.
From curl's documentation:

When used in conjunction with the -o, --output option, curl will create the necessary local directory hierarchy as needed. [...] If the --output file name uses no dir or if the dirs it mentions already exist, no dir will be created.

Since this option only applies to downloading a file, it would make sense to add it to the CurlDownloadSettings type.

Fixed in f09f146.


The floor seems to be ignored by OpenGL and recast, also cannot generate navmesh for the floor plane or house floor, only for top structures (high points)
But the floor can be seen if looking from below to upwards

I though it was some problem with the .obj file, but blender could render it without any problems

If anybody can help me, below is the level.obj file
level.zip

It seems taht the cell size is a little bit large.

As was already stated, if you're getting things rendering by disabling backface culling, that implies that your normals or winding order is wrong for the floor.  Also keep in mind that RecastDemo is just a demo.  It doesn't have the most robust obj loading code, so it's possible there's a bug somewhere there as well.


improve add limit check for lpad/rpad function input big value of length

if the user input big value of length, maybe cause some memory problem or
exceeded size limit of thrift, so add this check to report error for user firstly.
Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

run performance


attempted to fix the issue which page layout broken on some Android tablets like Galaxy Tab 10 or Nexus 7

moved the loading part of readium-shared-js library to EpubServer, so that library, EPUB xhtml files and resources are all handled the same way to avoid missing some JavaScript files.

Closing as stale


Added progressbar stories

It seems that the font is not loading correctly (Clearly evident in the helper-text example). I checked other component stories and it looks like (at least) is rendering a sans-serif. Is something we need to check?

Looks like helperText has a wrong token in the code. It is using "fontFamily" token (which doesn't exist in this component) instead of "helperTextFontFamily" token.
I will create an issue to fix this.

Tests without value will change all the times, so we could add value in all tests to avoid this.

Tests without value will change all the times, so we could add value in all tests to avoid this.

Added values to examples.


Use correct attribution in footer of documentation pages

Is your feature request related to a problem or challenge?
To be inline with ASF policy, we should update the text in the footer used in the docs (e.g. https://arrow.apache.org/datafusion/) to match the footer used in the main website (https://arrow.apache.org/). i.e. add the trademark attribution test.
This applies to docs for DataFusion, the Python bindings, and Ballista
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

I will take a shot at doing this

I filed https://github.com/apache/arrow-datafusion/pull/8760 for DataFusion and I filed seprate tickets for the subprojects (and linked them back to the main epic https://github.com/apache/arrow/issues/39462)

[ ] https://github.com/apache/arrow-datafusion-python/issues/553 (DataFusion Python)
[ ] https://github.com/apache/arrow-ballista/issues/950 (Ballista)
[ ] https://github.com/apache/arrow-ballista-python/issues/64 (Ballista Python)


Add include guards to the header files

Some header files does not have the "classic" include guard:
#ifndef HEADER_FOO
#define HEADER_FOO
...
#endif /* HEADER_FOO */

Even though in some cases that is not technically mandatory, as a design/stylistic suggestion I propose to add the include guards to every header file as a de-facto coding rule.

For what it's worth, it's my opinion that this is not worth spending time (yours, nor the team's) at this point in the release cycle.  We have a big backlog of PR's and coverity things to worry about.

Understood. Actually I'm studying the internals especially of the libcrypto and if I spot something that smells bed I'll try at least to improve it... I hope this is the spirit of the project

Absolutely and completely that's the spirit!  It might be more useful to open issues for these kinds of things, and we can get back to them after the 1.1.1 release.  Or keep doing what you're doing.  Every contribution to improve the code helps!

All the public headers have the guards. Some of the internal ones do not have them. I do not think this is a "feature" worth working on. If somebody creates a PR adding the guards, it will be considered. Closing.


[New Core]: The Speed Rumbler

Core Author username
jotego
Core .yml snippet
display_name: The Speed Rumbler
repository: jtbin
path: pocket/zips/jotego.jtrumble.zip

Closed as per #793. Sorry for not catching this sooner.


Add scrollleft / scrollright to x11 backend.

Add scroll left, scroll right to x11 backend.
Mappings taken from here
https://bbs.archlinux.org/viewtopic.php?id=194776
And tested with xev

:+1:

Thanks :D


Bulk delete results

This doesn't support arena results. There are only tests for models and controllers, not javascript/views. There are some improvements that could be made like being able to select across pages (a la gmail).

Hmm, I also need to verify that a user can't pass result IDs that don't belong to them. Not sure how to cancel the pull request.

Re-opening pull request so others can tackle this if they so desire.

@stevschmid I forked your repo and pushed my branch there. Created a pull request from that. This probably shouldn't be merged into master until someone can verify a user can't pass result IDs that don't belong to them by fudging the form post or something to that nature.

Thanks. I will looking at it as soon as I have some time to spare.

Upcoming version will include this feature.


DNS not resolving with lastest version

Since my HA updated the dns plugin to version 2022.04.1, most of integrations (that require Internet) are failing due to names not being resolved.
As example, HomeConnect integration logs:
Logger: homeassistant.config_entries
Source: helpers/config_entry_oauth2_flow.py:197
First occurred: 6:21:08 PM (1 occurrences)
Last logged: 6:21:08 PM

Error setting up entry Home Connect for home_connect
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/site-packages/aiohttp/connector.py", line 1154, in _create_direct_connection
    hosts = await asyncio.shield(host_resolved)
  File "/usr/local/lib/python3.9/site-packages/aiohttp/connector.py", line 880, in _resolve_host
    addrs = await self._resolver.resolve(host, port, family=self._family)
  File "/usr/local/lib/python3.9/site-packages/aiohttp/resolver.py", line 33, in resolve
    infos = await self._loop.getaddrinfo(
  File "/usr/local/lib/python3.9/asyncio/base_events.py", line 861, in getaddrinfo
    return await self.run_in_executor(
  File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.9/socket.py", line 954, in getaddrinfo
    for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno -2] Name does not resolve

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 339, in async_setup
    result = await component.async_setup_entry(hass, self)
  File "/usr/src/homeassistant/homeassistant/components/home_connect/__init__.py", line 83, in async_setup_entry
    await update_all_devices(hass, entry)
  File "/usr/src/homeassistant/homeassistant/components/home_connect/__init__.py", line 105, in update_all_devices
    await hass.async_add_executor_job(hc_api.get_devices)
  File "/usr/local/lib/python3.9/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/src/homeassistant/homeassistant/components/home_connect/api.py", line 70, in get_devices
    appl = self.get_appliances()
  File "/usr/local/lib/python3.9/site-packages/homeconnect/api.py", line 136, in get_appliances
    data = self.get(ENDPOINT_APPLIANCES)
  File "/usr/local/lib/python3.9/site-packages/homeconnect/api.py", line 83, in get
    res = self.request("get", endpoint)
  File "/usr/local/lib/python3.9/site-packages/homeconnect/api.py", line 77, in request
    self._oauth.token = self.refresh_tokens()
  File "/usr/src/homeassistant/homeassistant/components/home_connect/api.py", line 62, in refresh_tokens
    run_coroutine_threadsafe(
  File "/usr/local/lib/python3.9/concurrent/futures/_base.py", line 446, in result
    return self.__get_result()
  File "/usr/local/lib/python3.9/concurrent/futures/_base.py", line 391, in __get_result
    raise self._exception
  File "/usr/src/homeassistant/homeassistant/helpers/config_entry_oauth2_flow.py", line 472, in async_ensure_token_valid
    new_token = await self.implementation.async_refresh_token(self.token)
  File "/usr/src/homeassistant/homeassistant/helpers/config_entry_oauth2_flow.py", line 89, in async_refresh_token
    new_token = await self._async_refresh_token(token)
  File "/usr/src/homeassistant/homeassistant/helpers/config_entry_oauth2_flow.py", line 179, in _async_refresh_token
    new_token = await self._token_request(
  File "/usr/src/homeassistant/homeassistant/helpers/config_entry_oauth2_flow.py", line 197, in _token_request
    resp = await session.post(self.token_url, data=data)
  File "/usr/local/lib/python3.9/site-packages/aiohttp/client.py", line 535, in _request
    conn = await self._connector.connect(
  File "/usr/local/lib/python3.9/site-packages/aiohttp/connector.py", line 542, in connect
    proto = await self._create_connection(req, traces, timeout)
  File "/usr/local/lib/python3.9/site-packages/aiohttp/connector.py", line 907, in _create_connection
    _, proto = await self._create_direct_connection(req, traces, timeout)
  File "/usr/local/lib/python3.9/site-packages/aiohttp/connector.py", line 1166, in _create_direct_connection
    raise ClientConnectorError(req.connection_key, exc) from exc
aiohttp.client_exceptions.ClientConnectorError: Cannot connect to host api.home-connect.com:443 ssl:default [Name does not resolve]

The issue is resolved simply by downgrading to previous version:
ha dns downgrade --version 2022.2.0
But after a few hours, HA detects that there is a new version available, it is upgraded to 2022.04.1 and then it starts to fail again.
This is my current dns config (everything is standard):
➜  ~ ha dns info
fallback: false
host: <IP_ADDRESS>
llmnr: true
locals:
- dns://<IP_ADDRESS>
- dns://<IP_ADDRESS>
mdns: true
servers: []
update_available: true
version: 2022.02.0
version_latest: 2022.04.1

DNS logs doesn't show anything prompting the error.
The only strange thing that I can observe, is that when DNS are not resolved, nslookup shows strange characters in the response, like a bad encoding (I don't have any example right now, since it is working fine now, will fail again in a few hours).

I had the same issue today.
Found it suspicious that the servers list is empty.
The workaround for me was to run ha dns options --servers dns://<IP_ADDRESS> (replace with your DNS provider of choice).
Now there is an entry in the servers list and resolving works again.


⚠ Mobitopup has degraded performance

In 2f3bb07, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 4792 ms

Resolved: Mobitopup performance has improved in effc3f5 after 422 days, 7 hours, 1 minute.


dotnet-sdk_7.0: 7.0.101 -> 7.0.102

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 210153 run on x86_64-linux 1

1 package failed to build:

BeatSaberModManager

6 packages built:

ArchiSteamFarm
dotnet-aspnetcore_7
dotnet-runtime_7
dotnet-sdk_7
pinta
ryujinx


minecraf

hello guys welcome back to my Minecraft lets play today we're gonna find some diamonds

uh oh
didn't proccess

apparently the apostrophe broke it
also now that i think about it, you could probably inject code to run on this repo with that

i think i fixed it. you can try it again if you'd like

apparently the apostrophe broke it
also now that i think about it, you could probably inject code to run on this repo with that

yeah I was kinda thinking the same thing


Add config for glove size

Currently running KerbalVR with Moardv's avionics systems (mas). Unfortunately, the gloves are far too big to interact with switches with any reasonable accuracy. To be fair, the control panels are absolutely cluttered with the IVA's that come with MAS. Maybe just a number input, similar to world scale? Terrific mod, by the way

How do I add a

Sorry, havent used github before. How do I mark this as a feature request?

I've put the appropriate label on this issue. Thanks for the suggestion, I'll work on it :)

Same issue here. I'm trying to play in VR in an A.L.C.O.R. IVA, and it's effectively rather impractical because I either miss the button or can't use any of the more advanced UI thingies (like the power  throttle or any of the "spin" buttons, like the thrust limiter).
The biggest handicap is the set of square buttons all around the in-game screens in the cockpit When you have buttons A,B and C next to each other and are trying to click B, it's excruciating : It takes 10 attempts to click neither A nor C.
Even without changing the size of the glove, maybe just adding a ray pointer at the finger tip to see exactly at what the finger is pointing?

Is there some sort of debug mode of your mod where we can see the collision areas of in-game props? I'm curious why the issue is mostly with the buttons around the smaller type of RasterMonitor, and not the buttons of the larger RasterMonitor monitor. The buttons look like they have the same size in-game, so it might be an issue with a careless collision box.

Hey wait a minute it seens to HAVE been implemented, in the "hands size" parameter"??? @Vivero , any feedback? Is this available or not?

The solution seems to lie in _manipulatorSize and fingertipCollider.

I have reduced the fingertips size in this pull request : https://github.com/Vivero/Kerbal-VR/pull/88
Now the creator of KerbalVR needs to accept it.
I don't know if it fixes all issues related to gloves being too big, but at least you can now click the buttons around the RasterPropMonitors.


As a User I want to place a bet using Discord-Bot-Commands

Discord-Command i.e. !bet Amanda x Peter,

command interaction created. Data needs to be stored in db.

blocked by db innit


feat: remove NIP-09

I think there is no real deletion in a distributed system.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 5 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 95.451%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
868

Relevant Lines:
915

💛  - Coveralls


Use upstream git source

Until https://github.com/terraform-google-modules/terraform-google-cloud-storage/pull/338 is merged

Related to #1136


Support for Withdraw()

Should be possible to call a method Withdraw(from_address, to_address) which will create a transaction that requests an asset withdrawal or token withdrawal.

Is this different from Transfer/SendAsset, or has this functionality been supported already?

Not implemented yet.

Implemented, but needs proper testing.


I got a error

When I tried to add new server info (url and mac) I got this error
(node:2927) UnhandledPromiseRejectionWarning: FetchError: invalid json response body at http://<server url>/server/load.php?type=stb&action=handshake reason: Unexpected token < in JSON at position 0     at /home/dys/PiTV/out/PiTV-linux-x64/resources/app/.webpack/main/index.js:24260:15     at processTicksAndRejections (internal/process/task_queues.js:97:5)     at async /home/dys/PiTV/out/PiTV-linux-x64/resources/app/.webpack/main/index.js:24990:21 (node:2927) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict(see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 62) (node:2927) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code. (node:2927) UnhandledPromiseRejectionWarning: FetchError: invalid json response body at http://nextv.bossna-caffe.com:25461/c/server/load.php?type=stb&action=handshake reason: Unexpected token < in JSON at position 0     at /home/dys/PiTV/out/PiTV-linux-x64/resources/app/.webpack/main/index.js:24260:15     at processTicksAndRejections (internal/process/task_queues.js:97:5)     at async /home/dys/PiTV/out/PiTV-linux-x64/resources/app/.webpack/main/index.js:24990:21 (node:2927) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag--unhandled-rejections=strict(see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 64)

Thank you for posting the issue, I will try to reproduce that error as soon as possible. I think it is a server-specific error, but when I try, I will write you back.

Thanks for quick answer, I don't think it's a server-specific error, every server I tested works on android.

I understand, while I was testing I haven’t encountered any errors like this and my friends also tested on different servers. But you might be right, since it’s an early-stage program, I will check back and try to fix it. Thank you.

Any way, good luck, I'm a linux user over 10 years, me and a lot of users would like to have app like this, alternative to SF-VIP Player (for Windows), and STB emu (Android).

@Dysfunctional89 sorry for the late response but I found out that you put server address as http://<server_addr:port>/c/, which you should just write the part http://example.com:1234, nothing else. No "/c/" is needed

Sorry for the late response, I was busy a bit.
I think this error happens due to the server response, since it resolves to  and you were able to play the first channel you clicked. Have you tried to do a , it renews the tokens to the server. Probably you have figured it out after so many days (again, sorry for the late response 😅 )

I am closing this issue due to inactivity, if the problem still persists, reopen it.


Add AWSElasticBeanstalkMulticontainerDocker role for ec2 instance profile

Fixes https://github.com/cloudposse/terraform-aws-elastic-beanstalk-environment/issues/79

@aknysh Sorry, that was me in auto pilot mode. Applied all suggestions.
Thanks for maintaining all the modules.


[SPARK-22472][SQL] add null check for top-level primitive values

What changes were proposed in this pull request?
One powerful feature of Dataset is, we can easily map SQL rows to Scala/Java objects and do runtime null check automatically.
For example, let's say we have a parquet file with schema <a: int, b: string>, and we have a case class Data(a: Int, b: String). Users can easily read this parquet file into Data objects, and Spark will throw NPE if column a has null values.
However the null checking is left behind for top-level primitive values. For example, let's say we have a parquet file with schema <a: Int>, and we read it into Scala Int. If column a has null values, we will get some weird results.
scala> val ds = spark.read.parquet(...).as[Int]

scala> ds.show()
+----+
|v   |
+----+
|null|
|1   |
+----+

scala> ds.collect
res0: Array[Long] = Array(0, 1)

scala> ds.map(_ * 2).show
+-----+
|value|
+-----+
|-2   |
|2    |
+-----+

This is because internally Spark use some special default values for primitive types, but never expect users to see/operate these default value directly.
This PR adds null check for top-level primitive values
How was this patch tested?
new test

cc @gatorsmile @kiszk @srowen

Test build #83644 has started for PR 19707 at commit dad5080.

LGTM except one minor comment

Test build #83644 has finished for PR 19707 at commit dad5080.

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

Merged build finished. Test PASSed.

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

Test build #83654 has started for PR 19707 at commit a59c399.

Test build #83654 has finished for PR 19707 at commit a59c399.

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

Merged build finished. Test PASSed.

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

LGTM

Thanks! Merged to master/2.2


Back-patch to 7.x to allow analyzer v0.40.x

The built_value_generator package at version 7.1.0 depends on analyzer: '>=0.39.3 <0.40.0', which prevents consumers from resolving newer versions of certain build packages. However, built_value_generator is completely compatible with analyzer v0.40.x as-is. So, to allow consumption of built_value* 7.x with newer build packages without also having to upgrade to Dart 2.12+, this commit widens the analyzer range to include v0.40.x.
I've verified locally that on both Dart 2.7.2 and 2.13.4:

pub get resolves
./tool/presubmit builds all packages successfully
pub run test passes on built_value_generator

Some context: Workiva is about to upgrade from Dart 2.7 to 2.13 (for a few reasons, we were unable to upgrade to versions prior to 2.12). So far, we've been able to upgrade all of our individual packages to run on 2.13 successfully while still supporting dependency ranges that resolve on both 2.7 and 2.13. This buys us the ability to build our main application on both versions and switch which version is deployed with a flag.
One of the critical dependency versions that we need to resolve to on 2.13 is build_web_compilers v2.12.0, because prior to this version, code is incorrectly opted-in to NNBD even though the package's minimum Dart SDK is less than 2.12. Unfortunately, built_value_generator v7.1.0 prevents us from resolving to that version because of its analyzer dependency. Because of that, we had been using a >=7.1.0 <9.0.0 range which we've now learned is unsafe because code generated with built_value_generator v8 (reasonably) references code that does not exist in built_value v7.
If this is an acceptable back-patch, we would be able to depend on built_value and built_value_generator ^7.1.1 and resolve build_web_compilers 2.12.0 on Dart 2.13, which means we'd be able to complete this upgrade smoothly.
@davidmorgan is this something you'd be open to?

@kevmoo Also tagging you here since I just noticed David's location and that it's already evening for him.

Seems reasonable, but I don't have keys to build_value 😢

er, built_value

@kevmoo No problem, we can wait until tomorrow! Do you by chance have push access to the repo? Just wondering if we could get a branch off of the 7.1.0 tag created so that I can open a PR not against my own fork.

@jonasfj sorry for the ping - I saw your name listed on the google.dev publisher page, are you a maintainer of this package?


[Bug] S3 Plugin: Add encrypted tag to the secret key

Description
It is not apparent that the AWS Secret key is encrypted when filling out the datasource form. Refer to MySQL & PostgreSQL plugins for the UI.
Important Details

Version: [Cloud / Self-Hosted vx.x]
OS: [e.g.MacOSX]
Browser [e.g. chrome, safari]

Fixed - PR #2979


Compatibility with the Tagger -module

Hello! I think it would be awesome, if there was a way for this module to work with the Tagger -module. It would help a lot with how targets are recognized with the motion tracker. It would also help with all filtering problems as well as allowing an easy way for the GMs to decide what the players will see when using the tracker.

Hi @Waltrobus is the tagger module already available in foundry ? I'll take a look :)

Sorry it's been a while, I will take a look soon (been quite off this lately)

Hello, I took a look Tagger seems to have his own system adding metadata to Foundry information.
While the idea might seem good, am afraid it will add more support on my side that I have a hard time to give, by experience, adding dependencies creates a lot of complexity.
For the moment I won't add support for the Tagger module, but I might think about something that can make the bridge without adding this dependency


NO-JIRA: Sync OSGi broker config with latest changes from ARTEMIS-1628

Is there no option to just copy (or build) this file from the source so manual syncing isn't necessary?

Well, actually the broker.xml is build from several templates and snippets. OSGi needs a complete file to trigger broker deployment. Therefore a a file need to be constructed during build process. Meight be tricky.

Any further objection to not merge this?


Remote "origin" does not support the Git LFS locking API

I followed your instructions but came up with a problem. I would like to seek your help. Do you know any with this problem?
Remote "origin" does not support the Git LFS locking API. Consider disabling it with:
  $ git config lfs.<unknown>.locksverify false
Uploading LFS objects:   0% (0/1), 0 B | 0 B/s, done.
batch request: missing protocol: "<unknown>"
error: failed to push some refs to 'github.com:Wsine/test.git'

I managed to solve the problem by encoding the <SECRET_ACCESS_KEY> via https://www.urlencoder.org/.
The key contains / characters. Maybe some hints on the Readme file can help to ease the use.
I am keeping the issue open for your review before closing it.

Also, I configured with git config lfs.locksverify false to disable the locking.

I'll add that to the README. By the way, what S3 provider were you using?

I am using the Backblaze B2 backend


Full width

Hi,
Is it possible set full width region container on the page eg.

Yes, there is a page level template option to remove the page container
margin.
On Mon., Oct. 5, 2020, 03:13 aspcsharp9<EMAIL_ADDRESS>wrote:

Hi,
Is it possible set full width of page eg.
[image: full_w]
https://user-images.githubusercontent.com/15157982/95049893-ee983b00-06ea-11eb-9afd-4f4ca5d1a4f0.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/vincentmorneau/material-apex/issues/381, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJEHO2IKIORXZVA5XGJB33SJFWYBANCNFSM4SEJWRMA
.

you can use css as below to do so
By default these will be set to smaller value.
.container, .main-container .ma-main-container, .ma-region.container{
width: 100% !important;
}
.container, .main-container .ma-main-container, .ma-region.container{
max-width: 100% !important;
}

you can use css as below to do so
By default these will be set to smaller value.
.container, .main-container .ma-main-container, .ma-region.container{
width: 100% !important;
}
.container, .main-container .ma-main-container, .ma-region.container{
max-width: 100% !important;
}


bug champion select

bug champion select, runes and post game, champions without image

I adjusted the images, the language is in fr_fr, I wanted to put it in pt_br, but because in fr_fr I couldn't find the images folder, so I selected it in the panel's config tab

with pt_br it doesn't work so I left it in the fr_fr option without being the first option and pulled the images, since the post game I'm still configuring it but I wanted to translate it, how can I? where is the text for me to translate, editing the fonts model is not working either

Ok I see, the folder with the images must have a problem.
I'll try to find out why it's not working.
For the translation I forgot to put a custom field, I'll have a patch made to fix that.

The file are fixed and custom win lose add in v1.1.7


Remove KomputeWorkgroup class in favour of std::array<uint32_t, 3>

Currently the kompute workgroup class doesn't add much, so we'll remove the class.
There could also be a typedef to make it more intuitive but may not even be necessary.

Closed via #151


Don't try to build with pypy

We rely on SWIG to build Python wrappers, and SWIG is not compatible with PyPy.
Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Relates regro/cf-scripts#1386

No longer needed, as conda-forge is sunsetting pypy anyway.


Changed lambda layer to allow xception for custom inputs, does not ef…

…fect anything else otherwise. Noticed this when attempting to use xception backbone for an image with a custom input shape. Tested on images in the repository and attained the same results as the demo, so it does not negatively affect anything else.

Prior to this update x = Concatenate()([x, dec_skip1]) would throw a ValueError as the shapes would not line up. This update remedies this.

Thx for the PR!

Getting an issue with this PR for saving models.
When using version before this PR:
git checkout 00a6c4956b171aef7380d44236ac666faff5c5c9
model.save('model.h5') -> no issue
model.to_json() -> no issue
When using repo after this PR:
git checkout 1b742d6617b7dd2bce082e0c62b7141248aa84c9
model.save('model.h5') -> TypeError: can't pickle _thread.RLock objects
model.to_json() -> TypeError: can't pickle _thread.RLock objects

To avoid the saving model issue above, use these lines instead.
size_before2 = tf.keras.backend.int_shape(skip1)
x = Lambda(lambda xx: tf.compat.v1.image.resize(xx,
                                                        size_before2[1:3],
                                                        align_corners=True))(x)


support patterns or hatching

Reported by karl smith on 16 Feb 2012 18:14 UTC
Support Ferret patterns; e.g. FILL /PATTERN=
Brush patterns are supported in the graphics delegate and engines, but the Ferret pattern names are not known.  See $FER_DIR/ppl/*.pat files.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1919

Comment by karl.smith on 13 Mar 2013 21:44 UTC
There was some discussion at the Feb 2013 GFDL meeting regarding patterns versus hatching.  The indication was that hatching would be quite useful and would remove the need for patterns.  (The ticket summary and description updated to reflect this).  Adding Andrew to this ticket and initially setting to GFDL Low Priority.
'''Patterns:'''
Qt uses bitmaps to indicate foreground versus background color, and the bitmap is tiled.  There are a number of provided brush "styles"; see
http://qt-project.org/doc/qt-4.8/qbrush.html
One could define a custom pattern as a texture with the TexturePattern brush "style".  Thus one could support all the *.pat patterns.  It is possible to do solid color gradients across the filled area, but that is another topic, and I suspect not of interest in Ferret.
In Cairo the the cairo_pattern_t is "the paint with which cairo draws".  The focus is on solid colors and color gradients.  There does not appear to be any default patterns as in Qt.  So whatever patterns are supported would need to be created (by drawing the design to the pattern surface).  Not a trivial change.  I have not investigated if there are outside resources available to help with this.
I think this all falls under patterns where the source is a predefined bitmap mask.  However, the Qt vertical, horizontal, cross, forwards diagonal, backwards diagonal, and diagonal cross patterns would also be instances of some predefined hatching.
'''Hatching:'''
For simple hatching support, I would see the user specifying type (parallel lines or perpendicular crossing lines), line angle relative to horizontal, line width, and between-line spacing.
Implementing in Cairo would be no different than for patterns (actually, easier as the user provides the details): use the specifications to create the pattern to draw with by drawing lines to the pattern surface.
Implementing in Qt would be the essentially the same as with Cairo - a custom drawing of lines to create a bitmap to be used as a TexturePattern for the brush.
'''Conclusion:'''
Put the effort into support for hatching, and forget about patterns in PyFerret.  Because running "no-display" (Cairo) needs to be able to produce (nearly) identical results as when running high-quality (Qt), the predefined Qt patterns does not give us much, since the effort still needs to be put in to reproduce the patterns in Cairo.  And hatching is really what is wanted.

Modified by karl.smith on 24 May 2013 23:38 UTC

Comment by karl.smith on 17 Dec 2013 17:10 UTC
Have received a couple of e-mail (Patrick Brockmann; Isabelle Gouttevin, who  indicated there were a lot of folks wanting it) requesting support for patterns in pyferret.  I suspect hatching would suffice, but need to check.  Elevated the priority for this "enhancement" (actually a return of functionality available to traditional ferret).

Comment by andrew.wittenberg on 17 Dec 2013 17:31 UTC
I think that hatching would suffice for us at GFDL.  We'd greatly prefer this to be implemented as vector line-drawing, rather than a bitmap.  Controllable parameters would be color, angle, spacing, and line thickness.  (Cross-hatching could be achieved easily by the user if needed -- by simply overlaying a second hatch at 90deg.)
A second, related useful item would be a dot-grid -- with its key parameters being color, angle, spacing, and dot-diameter.


Add documentation of coding standard

What would you like to be added
We should extend the contribution guide (CONTRIBUTING.md) with documentation about coding standards in this project.
We can alternatively create a HACKING.md file and put it there.
The documentation should include information about:

Linting:

imports
literals
linting tools used in this project (CI/precommit hook)

Testing:

Unit test standards
Integration tests (we should start making those)

Architecture:

Data pipeline (event system)
Code location
Principles to add new code

Why is this needed

Make code reviews shorter and save commenting about such topics
Welcome new developers
Preserve and share knowledge

@iYehuda I can work on this. Should I start? Also, I might need some help along the way. Thanks.


Add TokenManagement solution to WMAgents

Impact of the new feature
In order to start switching to token based authentication we need to decide and setup token management solution.
Is your feature request related to a problem? Please describe.
Currently there are multiple solutions exists:

oidc-agent a standard tool which can be installed via RPMs
curl based approach (used in CMSWEB) based on IAM credentials, see token.sh and associated Dockerfile
token manager from auth-proxy-serve (APS), see here which is similar to curl based solution but provides daemon (similar to token.sh) capabilities

Describe the solution you'd like
Decide which tool to use and adopt it in cronjob for WMAgent. For that we need:

new spec file
new RPM or any other deployment solution
optionally obtain IAM credentials (if we'll rely on them)
deploy to WMAgents
adopt in WMCore workflows, via TokenManager class in all HTTP requests, see #10939

Describe alternatives you've considered
Additional context
#10118 , #10939

Alan, I doubt it is a good idea. The token management should be independent from WMA framework/tools. I don't see any benefits of re-inventing the wheel. I pointed out to three solutions which are independent from WMA/WMCore tools and I don't see any benefits to incorporate yet another solution to WMA/WMCore stack.

As discussed in today's WMCore team meeting, we decided to promote this issue to High priority this quarter, while https://github.com/dmwm/WMCore/issues/11728 is getting demoted to Medium priority (this one had been originally considered for Q4).

Just a brief update on this issue. I have been working on this with Stephan L. and HyunWoo from FNAL, running a few tests in submit1 and making a few changes here and there. We can see the ScitokensFile variable in the grid runtime environment, but we still need to ensure that the token:
a) is continuously updated on the schedd node
b) gets transferred to the grid runtime
b) gets continuously updated in the grid runtime

Short update: we are still failing to get the kerberos token in auto-pilot (based on the keytab). I also took this opportunity to update the text file mentioned in my previous comment.

Another update: the keytab has been created - it needs to be updated whenever there is a password change - and that seem to be working properly. However, we are still figuring out where the token is transferred to in the grid job and whether it's properly refreshed. That investigation depends on running workflows (jobs) in the grid and communicating with some experts at FNAL. Given the slow progress on that, I am moving this ticket to Waiting.
In addition, I have also transferred the content of the token.txt file above over to WMCore in GitLab:
https://gitlab.cern.ch/dmwm/wmcore-docs/-/merge_requests/6

Instead of closing this issue out, I see now that I actually misunderstood this GH issue. This ticket seems to be asking for a solution to manage tokens within the agent allowing it to communicate with external services (central services, MonIT, CRIC, Rucio, etc). While I have been working on setting up a token on the agent side and propagate it to the production grid jobs.
New ticket has been created https://github.com/dmwm/WMCore/issues/11968, which was just added to the project board under Q2/2024. I am now demoting/removing this issue from the current quarter.

well, we need both. And AFAIK @mapellidario has been waiting for you to lead on both. Which seemed fine in the spirit "WMA has this almost done, let's see what they have before we dive into it". But if this new effort is only starting now, please feel free to talk with him and find out if he can help. I'd like to see tokens in use in CRAB before Dario leaves at the end of August :-)

note: Brian B. was very clear about "this issue was already solved by e.g. FIFE people at FNAL" and IIRC Farrukh should know everything. IIUC the solution does not require running an OIDC agent. Sorry for noise. LIkely you, Stephan, Valentin already know more/better.

@belforte yes, Farrukh was helping with the condor setup on the FNAL schedd side, but as mentioned above, our focus was solely on letting HTCondor manage the token for us and make sure that an up-to-date token would be kept in the grid job and loaded by CMSSW. Unfortunately we are not able to work on any of the other Token related issues for the moment, as there are other higher priority projects taking the team's effort.
I have this documented here: https://cms-wmcore.docs.cern.ch/wmcore/Tokens-in-WMAgent/#next-steps, or here for a better markdown experience.
Lastly, I think another discussion involving the Fermilab team and Brian is going to happen in May, so we might have a clearer roadmap on how to proceed with token integration as well. Nonetheless, we are happy to talk to Dario if he decides to get started on this.

After talking with @stlammel , we agreed that this issue will be addressed once CMSWEB is tokenized, likely Q1/2025


Codeql analysis

https://codeql.github.com/docs/

thanks for adding this; I'm guessing there is no analysis because this hasn't run properly on main yet?

Yes because it hasn't yet run on main.


Use Exposed's JSONB support

Please see the commit messages for details.

Merged as test failures are unrelated.


add link to Natural Scenes T&C on HF dataset card

via https://huggingface.co/datasets/pscotti/mindeyev2

Using these files requires that you have already agreed to the Natural Scenes Dataset's Terms and Conditions.

I don't know what this dataset is off hand and the name is generic enough that I'm not confident that I would find the same dataset referenced if I googled it. Would be helpful to provide a URL to eliminate this ambiguity.

This is the Natural Scenes Dataset but curated specifically for MindEyeV2. We'll contextualize everything once our entire codebase is public. 2 weeks.

Yes we still need to finish MindEye2 before we work on cleaning up this repo and better explaining how others can use it. Appreciate that you are interested in this project though, please feel free to let us know any issues after we release the preprint for this project (which should be next month)


run image_quality_prediction.py shape erro

thanks for your work. it is very cool. I test jpg image with size 1919 × 1440. it will show me that:
tensorflow.python.framework.errors_impl.InvalidArgumentError:  Incompatible shapes: [1,661,32] vs. [1,193,32]
	 [[node model/tri_q_image_quality_transformer/add_1 (defined at /data2/zhx3/triq/src/models/transformer_iqa.py:198) ]] [Op:__inference_predict_function_9663]

Errors may have originated from an input operation.
Input Source operations connected to node model/tri_q_image_quality_transformer/add_1:
 model/tri_q_image_quality_transformer/concat (defined at /data2/zhx3/triq/src/models/transformer_iqa.py:194)

Function call stack:
predict_function

This is because the input image has very large resolution, much lager than the imageset that the model has been trained on. If you look at the method create_triq_model in triq_model.py, maximum_position_encoding=193, and the position encoding of your image is 661. A quick and dirty way is to increase this argument to 661, for example. But I am not sure if it will work, as the model has not been trained with such big images. Give a try and please let me know what you get.

I think it can't work:
Traceback (most recent call last):
  File "examples/image_quality_prediction.py", line 24, in <module>
    predict_mos = predict_image_quality(model_weights_path, image_path)
  File "examples/image_quality_prediction.py", line 13, in predict_image_quality
    model.load_weights(model_weights_path)
  File "/data2/zhx3/env_python3.7_pytorch1.5/lib/python3.7/site-packages/tensorflow/python/keras/engine/training.py", line 250, in load_weights
    return super(Model, self).load_weights(filepath, by_name, skip_mismatch)
  File "/data2/zhx3/env_python3.7_pytorch1.5/lib/python3.7/site-packages/tensorflow/python/keras/engine/network.py", line 1266, in load_weights
    hdf5_format.load_weights_from_hdf5_group(f, self.layers)
  File "/data2/zhx3/env_python3.7_pytorch1.5/lib/python3.7/site-packages/tensorflow/python/keras/saving/hdf5_format.py", line 707, in load_weights_from_hdf5_group
    K.batch_set_value(weight_value_tuples)
  File "/data2/zhx3/env_python3.7_pytorch1.5/lib/python3.7/site-packages/tensorflow/python/keras/backend.py", line 3384, in batch_set_value
    x.assign(np.asarray(value, dtype=dtype(x)))
  File "/data2/zhx3/env_python3.7_pytorch1.5/lib/python3.7/site-packages/tensorflow/python/ops/resource_variable_ops.py", line 846, in assign
    self._shape.assert_is_compatible_with(value_tensor.shape)
  File "/data2/zhx3/env_python3.7_pytorch1.5/lib/python3.7/site-packages/tensorflow/python/framework/tensor_shape.py", line 1117, in assert_is_compatible_with
    raise ValueError("Shapes %s and %s are incompatible" % (self, other))
ValueError: Shapes (1, 661, 32) and (1, 193, 32) are incompatible

The max size may be  :1024 × 768. ok I will random crop image to 1024 × 768 and get the mean mos value.

I think it can't work:
Traceback (most recent call last):
  File "examples/image_quality_prediction.py", line 24, in <module>
    predict_mos = predict_image_quality(model_weights_path, image_path)
  File "examples/image_quality_prediction.py", line 13, in predict_image_quality
    model.load_weights(model_weights_path)
  File "/data2/zhx3/env_python3.7_pytorch1.5/lib/python3.7/site-packages/tensorflow/python/keras/engine/training.py", line 250, in load_weights
    return super(Model, self).load_weights(filepath, by_name, skip_mismatch)
  File "/data2/zhx3/env_python3.7_pytorch1.5/lib/python3.7/site-packages/tensorflow/python/keras/engine/network.py", line 1266, in load_weights
    hdf5_format.load_weights_from_hdf5_group(f, self.layers)
  File "/data2/zhx3/env_python3.7_pytorch1.5/lib/python3.7/site-packages/tensorflow/python/keras/saving/hdf5_format.py", line 707, in load_weights_from_hdf5_group
    K.batch_set_value(weight_value_tuples)
  File "/data2/zhx3/env_python3.7_pytorch1.5/lib/python3.7/site-packages/tensorflow/python/keras/backend.py", line 3384, in batch_set_value
    x.assign(np.asarray(value, dtype=dtype(x)))
  File "/data2/zhx3/env_python3.7_pytorch1.5/lib/python3.7/site-packages/tensorflow/python/ops/resource_variable_ops.py", line 846, in assign
    self._shape.assert_is_compatible_with(value_tensor.shape)
  File "/data2/zhx3/env_python3.7_pytorch1.5/lib/python3.7/site-packages/tensorflow/python/framework/tensor_shape.py", line 1117, in assert_is_compatible_with
    raise ValueError("Shapes %s and %s are incompatible" % (self, other))
ValueError: Shapes (1, 661, 32) and (1, 193, 32) are incompatible

The max size may be :1024 × 768. ok I will random crop image to 1024 × 768 and get the mean mos value.

Then it might be the only way for now, even though image patching and average is the approach that I want to avoid. The model can be trained by setting maximum_position_encoding to larger value when using small resolution images. However, I am not sure if that will work because the positional weights beyond 193 will probably not be trained well.

There is another approach to solve the problem. You can change line 146 in transformer_iqa.py by increasing the pooling size. For example, you can change to self.pooling_small = MaxPool2D(pool_size=(4, 4)) or even larger.

ok. I will try. thanks for your reply.

ok. I will try. thanks for your reply.

If possible, please also let me know your results. I am very curious. Thanks.


Add JSDoc documentation for repository

Summary of Changes
This PR adds JSDoc documentation to the entire repository for improved code readability and documentation.
Summary of Modified Files

/home/runner/work/eliza/eliza/packages/_examples/plugin/src/actions/sampleAction.ts
/home/runner/work/eliza/eliza/packages/_examples/plugin/src/evaluators/sampleEvalutor.ts
/home/runner/work/eliza/eliza/packages/_examples/plugin/src/index.ts
...

Instructions for Reviewers
Please review the changes in all modified files to ensure that the JSDoc documentation accurately reflects the code functionality and provides useful information for developers. Thank you!

Putting **/ instead of */ in some instances causing the source code not to parse. We should close this, then get that bug fixed then try again

why are you adding jsdocs to the /packages file?


2D regridding

Spatial interpolation (x, y, z) is quite common, and we should support it. It's almost trivial to expand regridding to a "structured" 3d model or a layered ugrid2d model (think GeoTOP).
However for layers with differing z bounds (think REGIS), slightly more work is required. A nice parallel is the VTK "explicit" structured grid.
https://www.kitware.com/introducing-explicit-structured-grids-in-vtk-and-paraview/
Hence I'd propose naming these StructuredGrid3d, LayeredUnstructuredGrid2d, ExplicitStructuredGrid3d, and ExplicitLayeredUnstructuredGrid2d. The current regridding utilities are expecting in the form of find_overlap1d_nd which allows "stacking" any additional dimension, then iterating over it (i.e. stack (y, x), then check at every xy all z overlaps).

Closed in favor of #73.


[newrelic-infrastructure] Add Windows support

Is this a new chart?
No
What this PR does / why we need it:
Adds Windows support for New Relic Infrastructure on Windows nodes and sets sensible default values
Which issue this PR fixes:

Fixes #41

Special notes for your reviewer:
Hi @alejandrodnm, here's what I've included in the PR:

New Windows DaemonSet added (daemonset-windows.yaml)
New values added and documented: enableLinux, enableWindows, windowsNodeSelector and image.windowsTag
Added .gitignore from official Helm Charts repo
The new nrStaging value was throwing an error from _helpers.tpl when global variables were not applied. I added some additional checking to fix this error.

Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

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

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

Hi @mr-smithers-excellent thanks for the PR, I'll try to review this today, or tomorrow.
Also, thanks for spotting the nrStaging bug, in all my tests I had global sets and didn't spot it. At first I was trying to do something like what you did but in some cases it wasn't working so I did the check with eq. In your code, if you set any of the variables to false the template returns '     false' and it takes that as true in the if. I added a fix, we just needed to add the - to trim the values, it's already on master so you have to rebase.

@alejandrodnm sounds good, I'll resolve that conflict. Looks like there's a conflict with the Chart version as well. What version would you like me to set here?

I didn't have time to review it today, I'll do it tomorrow.
As for the version you can increase to 0.15.0, I'll talk with the team to latter move this to 1.0.0

No worries. Handled the rebasing and Chart version bump. All ready for review now.

@alejandrodnm changes have been applied and CI is passing after another chart version bump.

@mr-smithers-excellent Thanks again for the PR :)


How to build the Project?

💬 Question
I just grab the latest master branch i was stumble with this error.. any ideas?
UI Kitten and Eva version

Package
Version

@eva-design/eva

@ui-kitten/components

This code is not from master branch.


only bind my wsl network

Hi, Thanks for the contributions.
I just follow the Sample Code to start olric by the embedded way,
However, I find it listens to the 3320 port, only binding to my WSL network address, while not binding to my LAN.
Here is my ipconfig. Sorry for the Chinese. I assumed you could roughly understand what it means.
   Synced with 0 initial nodes => discovery.go:59
2021/11/24 09:57:20 [INFO] Routing table has been pushed by <IP_ADDRESS>:3320 => operations.go:87
2021/11/24 09:57:20 [INFO] The cluster coordinator has been bootstrapped => discovery.go:42
2021/11/24 09:57:20 [INFO] Memberlist bindAddr: <IP_ADDRESS>, bindPort: 3322 => routingtable.go:399
2021/11/24 09:57:20 [INFO] Cluster coordinator: <IP_ADDRESS>:3320 => routingtable.go:400
2021/11/24 09:57:20 [INFO] Node name in the cluster: <IP_ADDRESS>:3320 => olric.go:390
2021/11/24 09:57:20 [INFO] Olric bindAddr: <IP_ADDRESS>, bindPort: 3320 => olric.go:396
2021/11/24 09:57:20 [INFO] Replication count is 1 => olric.go:398
2021/11/24 09:57:20 [INFO] Olric is ready to accept connections

My question is:
How can I bind it to all networks?
BTW, if I use the embedded way to start my application on different machines, can I use these embedded olric services as a cluster, sharing the same memory?

I am closing this issue due to a lack of activity.


Rename this repository to substrate-faucet

As this faucet now supports token drips from web request, we should rename it substrate-matrix-faucet -> substrate-faucet.

@mutantcornholio will paritytech/gitspiegel handle this change?
I believe that is our only blocker.

@alvicsam ^^

@mutantcornholio will paritytech/gitspiegel handle this change?
I believe that is our only blocker.

It should. gitspiegel is configured using node ids, which should not change with renaming.

@mutantcornholio @mordamax  paritytech/security#9 has been merged. Can we now rename the repo to polkadot-testnet-faucet?

yep, perfect name imo

Project has been renamed.


Code completion in get_node or $ doesn't work for sibling or higher nodes

Godot 3.1 alpha 3
Windows 10 x64
When using the get_node function, the code editor will offer autocomplete for all the nodes descending from the node that possesses the script in the scene. This is nice, but no code completion is offered for sibling nodes or further up the ancestry.
For example
get_node("../siblingNode")
won't offer autocomplete for "siblingNode" after "../" has been written.
Is this because there are possibly many ancestors, siblings, and so on, and so the list of strings would often be very long right out the gate with get_node(? Can instead the list of siblings appear only once this much has been written: get_node("../ , or would that require too much rework of the code completion system?
It would be a nice QOL improvement to be able to more quickly reference these nodes with get_node and $, not worry about spelling them incorrectly, and not feel like I had already misspelled something just because the autocomplete turned off.

Since any node in the tree can be an instance of a self-contained scene that doesn't have any further parent nodes while edited standalone, I wouldn't want the editor to encourage this coding style. Reference in (down the tree), signal out (up the tree / sideways by proxy of the parent) should always be preferred in this case, because it leads to less tight coupling and better testability.

While I agree this isn't a very good practice, I don't see a technical reason that should prevent listing siblings / other child nodes of the parent. This works fine:

Agree with @PetePete1984 that it's preferable for the editor to not provide suggestions that would lead to bad practices, at least in most use cases.
Perhaps an optional configuration flag could be added to "unlock" that behavior, for developers who know what they are doing and would benefit from it?

Perhaps an optional configuration flag could be added to "unlock" that behavior, for developers who know what they are doing and would benefit from it?

Since this behavior is widely frowned upon, I don't think adding an option for it makes sense. Doing this will bite you in the long run 🙂


Redesign the footer section and the footer links.

Please assign me for this.

some are working on it


Migrate backend to implement StateGraph instead of MessageGraph

Summary
Currently, the OpenGPTs backend only implements MessageGraph. Implementing StateGraph in the backend demonstrates the ability to support arbitrary graphs within OpenGPTs. Migrating the current usage of MessageGraph to StateGraph is the first step.
Changes in this PR are adopted from this PR.
Implementation

Rename internal variables and update docstrings: messages --> values.
Add support for Dict[str, Any] agent state type: Sequence[AnyMessage] --> Union[Sequence[AnyMessage], Dict[str, Any]].
Breaking API changes to rename interfaces to be agnostic to a agent state type.

To Do

[ ] ~Migrate get_chatbot_executor() to use StateGraph~
[ ] ~Migrate get_retrieval_executor() to use StateGraph~
[ ] ~Migrate get_google_agent_executor() to use StateGraph~
[ ] ~Migrate get_openai_agent_executor() to use StateGraph~
[ ] ~Migrate get_xml_agent_executor() to use StateGraph~

Note: This PR can be merged without completing these to-do's.

Update: Chatted with @nfcampos offline and the items in the To Do list will be implemented in a separate PR. get_chatbot_executor() and get_retrieval_executor() will be updated, but the rest will be left alone.


Call storage service copy api

Use annotation to inject storage service, and call the copy() method. I have not tested it yet. Please let me know if anything wrong.

LGTM.


Add extra migration triggered by updated django / parler

Fix #494

Coverage increased (+0.07%) to 95.568% when pulling f6a3e0263e11e072a8126d7d53facfc6b033a3a1 on hotfix/migration into dce5eac582794f19c6741ce0903000a2d1be4f8d on develop.


SlickGrid Dissapearing Text in Frozen Column on Horizontal Scroll

I have used SlickGrid in my project. Grid contains 30 columns. Initially grid works fine. it has horizontal as well as vertical scroll. I have frozen first column.
Issue is that when i scroll it horizontally and reached at right most and then after vertical scrolling frozen column data doesn't display. any suggestion please help.

You are using branch 2.0-frozenRowsAndColumns, right? This is because bug in SlickGrid: in line 1947 slick.grid.js file missing five parameter. Because of that SlickGrid not rerendered frozen cells.
Quick Fix
Add d as five parameter to appendCellHtml in line 1947.
        } else if (options.frozenColumn > -1 && i <= options.frozenColumn) {
          appendCellHtml(stringArrayL, row, i, colspan, d);
        }

Why this error appears?
Consider the render function. First it takes rendered rows, and then deletes all rows from the cache, except the specified range. Because rendered rows takes with reserve in three rows, an error occurs only after vertical scroll for more than three rows, when starts rendering missing rows in renderRows function. This function call appendRowHtml for each rendered row, and that function check position right border every cell in row. If that position in visible range, cell rendered properly. But for frozen columns, values in columnPosRight array after horizontal scroll become smaller left border of viewport. And the execution goes to second branch of if condition. In that branch missing five parameter d in call appendCellHtml. Because of that cell do not rendered.

Hi,
Thank you so much. I have incorporated your comment into my solution and issue has been fixed.
Thanks once again for your quick response. :)


tcp: add uv_tcp_timeout

refs: https://github.com/libuv/libuv/issues/3651

Hi ! Does anyone help review this ? Thanks !

I would potentially argue that most setsockopt calls should be dealt with by the user calling into the C library directly

Do you mean we should call this in caller(such as Node.js) instead of Libuv?

A more fundamental issue: TCP_USER_TIMEOUT doesn't do anything for non-blocking sockets, it won't wake up epoll_wait(). There's not much point in adding it to libuv.


Grouped Updates: rails-i18n

I really like your grouped update feature for rails. Would it be possible to include rails-i18n if the project uses the gem? Because those Depfu upgrades seem to cause mutual conflicts as of now:

Hi @leoarnold, good idea! While it's not technically the same group since there is not always a corresponding version for each Rails release. I've added it now and it should help conflicts on Rails major version upgrades.

:clap: 33 minutes from initial report to fix deployed is beyond any SLA


Dcproj is not being automatically created

With an existing project, I am able to generate the Dockerfile with Add Docker Support. However, the dcproj does not get generated.
Is there a way to get VS to manually create it?

@aluong -- Andy, thank you for your question. It looks like you're having an issue with the product itself, rather than an issue with the documentation. Here are a few options where you might consider asking your question:

https://social.msdn.microsoft.com/Forums/vstudio
https://www.stackoverflow.com
https://www.reddit.com/r/VisualStudio/

As well, you might also consider using the Report a Problem tool to report the issue.


pure virtual method called

I build the project with the follwing command on a raspberry2 Model B:
scons Werror=1 -j8 debug=1 asserts=1 arch=armv7a os=linux build=native opencl=0 neon=1

Then,
cd build
LD_LIBRARY_PATH=. ./neon_convolution

But got output like this:
terminate called without an active exception
Aborted

I found that it crashed in conv3x3.run().
gcc/g++ version:
gcc version 4.8.4 (Raspbian 4.8.4-1)

What's wrong?
Thank you.

OK, in src/runtime/NEON/functions/NEConvolution.cpp, there is no NEConvolution3x3::run().

Hi,
Unfortunately I don't have a Raspberry Pi to reproduce this issue.
A few things I would suggest you try:

Could you please try to run the binaries we provided here: https://github.com/ARM-software/ComputeLibrary/releases

Cross compile instead of building natively  (In case it is a problem with the Raspberry Pi toolchain)

Try to run the example in GDB to see where it crashes.

We've successfully run this example on a variety of platforms so I'm curious to have more information about the issue you're seeing.

I only just noticed you were running Raspbian: isn't this OS compiled for armv6 ? (i.e no NEON support)
Have you tried using an armv7 hard float image instead ?
Should be faster too.

Hi @AnthonyARM ,
I found that it crashed in std::thread. And it can be solved by removing -march=armv7-a -mthumb in the compiler flags.
Details of my Raspbian:
uname -a
Linux raspberrypi 4.4.50-v7+ #970 SMP Mon Feb 20 19:18:29 GMT 2017 armv7l GNU/Linux

I really want to know what is armv7l and I didn't find it in the -march lists.
Maybe, it is a bug with the toolchain.

Yes, like mentioned above: raspbian is compiled for armv6 which is why it works when you remove armv7-a and thumb.
Could you try with a Linux armv7 OS ? ((Like Arch Linux or Ubuntu))

Hi @AnthonyARM ,
Thank you.
I will try Arch Linux and tell you the result later.

Yes, it worked on Arch Linux.
uname -a
Linux alarmpi 4.9.19-1-ARCH #1 SMP Thu Mar 30 19:23:11 MDT 2017 armv7l GNU/Linux

Cool, thanks for confirming.
Feel free to re-open the issue or create a new one if you run into more problems


On Robotoff question, display the image that triggered insight creation

Steps to reproduce the behavior

Go to this product<PHONE_NUMBER>990
Click on Robotoff question link
The displayed image (weight insight) is not the one from which the insight was created from. It makes it difficult to know whether the insight is true or not.

Expected behavior
We should display (as first image at least) the image where the information comes from. This data is available in Robotoff API response.
Screenshot/Mockup/Before-After

Smartphone model

Device: OnePlus 7t
OS: Android
App Version: 3.20.0+737

Hi @raphael0202!
I'm not familiar with Robotoff, but that is what we get as a robotoff question:
class RobotoffQuestion extends JsonObject {
  final String? barcode;
  final String? type;
  final String? value;
  final String? question;
  @JsonKey(name: 'insight_id')
  final String? insightId;
  @JsonKey(name: 'insight_type', unknownEnumValue: InsightType.UNKNOWN)
  final InsightType? insightType;
  @JsonKey(name: 'source_image_url')
  final String? imageUrl;

What you suggest is to display imageUrl first in the image list if it is not null, am I right?
Or even to display only imageUrl, or all the images if null?
For the moment we don't use imageUrl and always display the same "image carousel" (cf. QuestionPage):
ProductImageCarousel(
            widget.product,
            height: screenSize.height / 6,
          ),

Sorry for my late answer, my notifications were muted for this project 😅
Indeed source_image_url when available is the image from which the insight was extracted from. See https://robotoff.openfoodfacts.org/api/v1/insights/detail/d8ae9556-c075-429c-a2f0-1c82d2220539
So in that case displaying it first in the carousel would make sense.

The images are stored here: https://static.openfoodfacts.org/images/products/227/176/000/5965/1.jpg
.org for prod and .net for preprod

Thank you @raphael0202 for your answer! Actually we already use "those" roots:
  ///Url base for images in prod: needs to match the domain of uriProdHost
  static String imageProdUrlBase =
      'https://static.openfoodfacts.org/images/products/';

  ///Url base for images in test: needs to match the domain of uriTestHost
  static String imageTestUrlBase =
      'https://static.openfoodfacts.net/images/products/';

@WildOrangutan Looks interesting for you too, something like:
  static String? buildFullSizeUrl(
    final String? barcode,
    final ProductImage image, {
    final QueryType? queryType,
  }) =>
      barcode == null
          ? null
          : '${getProductImageRootUrl(barcode, queryType: queryType)}/${image.rev}.jpg';

@raphael0202 @WildOrangutan I'm not sure things are that easy:

I found front images without the corresponding expected full image
=> if the image urls are correct for Robotoff, I'm ok with that
not very common, but we have potentially crop parameters (area + angle), and I don't know how this would behave
=> we'll see

In https://static.openfoodfacts.net/images/products/111/111/111/1111/front_de.51.full.jpg, 51 is actually the revision number of the image. If you want to get the base image you should  look at front_de in the images field. There is the revision number and the image ID.
But why do you mention this? You want a resized version of the image for faster loading I guess?

@raphael0202

I assumed that for each barcode, there's a list of images, starting with id (or rev_id?) 1 increasing
I assumed that front_de.51.full.jpg meant that image 51 has been linked to the front/de display (in all sizes)
and I hoped that somehow just knowing 51 was enough to say "the whole image file of id 51 is in https.../51.jpg.
Probably some confusion on my side between image id and image REV id.
The thing is that it could have been useful for @WildOrangutan that needs to switch from a 400 picture to the full version (not a faster loading, the other way around: a fast load and a slower load when asking for details). A direct access like "from front_de.51.full.jpg just go to 51.jpg could have been useful, but it does not seems to work this way and anyway we have another solution

Regarding crop parameters, I don't know how often it is used but it's possible to upload a picture with crop parameters + angle:
  ProductImage({
    required this.field,
    this.size,
    this.language,
    this.url,
    this.rev,
    this.imgid,
    this.angle,
    this.coordinatesImageSize,
    this.x1,
    this.y1,
    this.x2,
    this.y2,
  });

In that case I assume that there's the full picture uploaded on the server, and cropped/rotated versions as well.
Perhaps it's not your problem at Robotoff: you may only deal with pictures visible for users, not the uploaded pictures the crops were computed from.

Perhaps it's not your problem at Robotoff: you may only deal with pictures visible for users, not the uploaded pictures the crops were computed from.

Yes in Robotoff we only extract insights from "raw" pictures (pictures with digits as names such as 1.jpg or 2.jpg).

and I hoped that somehow just knowing 51 was enough to say "the whole image file of id 51 is in https.../51.jpg.

Indeed it does not work this way, you have to query the API, fetch the images field, look at the front_de key to get the image_id and get the full image. But as Robotoff deals with raw images, there is no resized (100, 200, 400) versions of the image, unless the image has been selected (front, nutrition....) without any cropping.
Can you confirm this @stephanegigandet?

@raphael0202 As far as I'm concerned you've answered my question: it's not as straightforward as I imagined, period.
Which is not a big deal as I have a solution to do what I wanted to in a first place.
So @stephanegigandet, unless @raphael0202 insists for specific reasons, I don't need any confirmation.

Well I'm interested in having a confirmation from Stéphane, as what I've explained earlier is just based on my own observations, we could update documentation accordingly if confirmed as this question often arises.

The raw uploaded images are stored in sequence: 1.jpg, 2.jpg, 3.jpg
We also generate resized versions of those, at 100 and 400 pixels (max of width or height, whichever is larger).