add cgroup function that can limit cpu share of each worker

add cgroup function that can limit cpu share of each worker, and it really help indeed

AFAIK, if some feature is non-trivial, it requires backing JIRA ticket for it

@caishize please file a JIRA here https://issues.apache.org/jira/browse/STORM . I haven't taken a close look at the changes but does any of these changes interfere with running on windows?

@caishize Any update on this?  I really would like to have cgroup support, and this should work really well with the resource aware scheduling we are working on, on another JIRA.

Yes,there's an update haven't been  submitted, but it still only works on an Linux operating system which has the  cgroup installed..
Sent from my iPhone

On Oct 3, 2015, at 11:46 PM, Robert (Bobby) Evans<EMAIL_ADDRESS>wrote:
@caishize Any update on this? I really would like to have cgroup support, and this should work really well with the resource aware scheduling we are working on, on another JIRA.
—
Reply to this email directly or view it on GitHub.

any updates on this PR?

it is great that storm begin to support cgroup function just like jstorm

@revans2 @hustfxj @jerrypeng @caishize
Should we add support for cgroups with the jstorm merge, or should we try to add support for it now?
@caishize It would be good to create a Jira issue for this change and resolve the conflicts with the master branch.

I agree with @d2r, or would at least answer that, yes, we should hold off on cgroups support until the jstorm merge. I believe that @revans2 might have already covered this in JIRA. If not, please open a new issue.

@ptgoetz  Yes, @revans2  has created one. Following is the JIRA for jstorm merge.
https://issues.apache.org/jira/browse/STORM-1336

JStorm's cgroups merged with #1053. Closing this PR.


Store csv file in S3 bucket

User story

As a software developer
I want to provide a backend service that stores a csv file in the S3 bucket
So that it can be retrieved at a later point

Acceptance criteria

[x] Backend provides a api endpoint that receives a csv file
[x] Backend then stores the csv file in a S3 bucket
[ ] Backend saves the location in the s3 bucket (a link) of the csv file in the database

DoD general criteria

Feature has been fully implemented
Feature has been tested
Feature has been merged into the main branch
Important parts have been documented in the technology wiki
All acceptance criteria were met
Product owner approved features
Developers agreed to release

Create an API enpoint for a file upload, e.g. file servie -> as generic as possible.
s3bucket service that handles the file and saves it.

send file directly to s3bucket ??
send file metadata to backend to save it into mongodb

the file metadata is saved in the mongodb database as own filemetaData entity

File Data entity
id
location: s3bucekt download link
fileName:
fileSize:

@Keldami @ingunnaf Hey whats the plan for uploading file
I was thinking it is better to upload directly from frontend and send a http call request with the S3 bucket link where exactly the file is stored, as it saves overhead of sending file to backend again

Hello! I don't get notifications about comments here so text me on Slack if you want to reach me faster:) I didn't see you comment until now so I added it as a backend endpoint, and opened a MR it now

I think it would be better to have all the logic centralized in the backend. Also keep in mind that in this case the frontend has to do more tasks, such as sending the raw data to the data pipeline. This means that there are already two calls that the frontend would have to make. We should make sure that the frontend has to perform as few tasks as possible in order to ensure high performance.


Categories are difficult to configure

Categories cannot be configured easily. E.g. setting accept_unofficial_submits is difficult. append_content in conf.py can be used but it is a bit cumbersome.

append_content example:
File conf.py:
append_content = [
    '_config_aplus/categories.yaml',
]

File _config_aplus/categories.yaml:
categories:
  submit:
    accept_unofficial_submits: True
    status: nototal
  feedback:
    accept_unofficial_submits: True
    confirm_the_level: True


Add unit tests for _download_url?

Currently _download_git, _download_svn and _download_hg are all not getting hit by the test suite.
It will probably be useful to add unit tests that hit (via the public interface) those code paths.
I think the easiest way to get a useful version of this working is to mock out os.system and check that the correct system calls are made.
We can and possibly should also make integration tests that actually make the relevant system calls, but we can defer those later because they have some more dev-ops-y implications.

Or just deprecate them?

@benoit-pierre Any change to this code is going to give us negative diff coverage because it's changing untested code, which makes it hard to evaluate whether such changes (including changes that raise DeprecationWarnings) are safe to merge.
If the eventual goal is to deprecate/remove this functionality, then we can skip the integration tests.

So these calls are soft-deprecated in the sense that easy_install is deprecated. The issue remains that setup_requires still relies on easy_install and eggs. I'm okay leaving these untested as I consider these use cases largely unsupported and the workaround is to use pip and pyproject.toml to prevent setup_requires from being activated.

@jaraco I think we should do the basic smoke tests (i.e. mock out the system calls), if only so that the CI won't complain about coverage issues when we make changes like #1499.
Also, when we do deprecate them, we'll want tests to make sure the deprecation warnings are raised, and having some existing "mock tests" will make that easier (since we can just inject the "pytest.warns" assertion directly into those tests).

I'd like to work on this

Looking into it...

working on this with @vitoace and @serjx86


Fix memory allocation and GUI update outside of main thread (macOS compatibility)

Fix macOS compatibility by:

Allocating properly large array (stack allocation not permitted for array holding large response like image)
Use default browser to open internet link on macOS
Move button toggles to main thread as MacOS only allows GUI update from main thread

All the screenshot plugins are now leaking in case of error but you don't need to fix those. I can take care of them. If you fix the stuff I mentioned we are good to merge :)

I fixed the double free however for indent I used the GNU indent command line tool, which does not seem to have the same indent rules as your files. Which IDE or tool do you use for indentation ?

Oh no, please do not run it through an indent tool! :)
They rarely do a good job - they will almost always ruin the intended look of the author. Many of these tools are especially bad at breaking up long lined function prototypes and other stuff. I think your output proves my point :)
I use vim to automatically indent my functions as I go and fix up whatever mistakes it makes. For lxi-tools I'm using 2 space indentation, no tabs - same as the GTK project. I used to use 3 space notation and I plan to fix all the files but now is not the time to do so.


release web-speedy

Needs fix: Please also exscute integration tests on travis
index_m is current? feels like dummy files?!
makes it really sens to always do require('speedtest-net')()... ?! This always creates a new function instance, or ?! That feels very strange
https://github.com/iobroker-community-adapters/ioBroker.web-speedy/blob/master/main.js#L177 ... admin will most like not catch up new state Lists without an object reload
timeouts not cleared ;-) gggrrrrrrrr ...
When such a test is running )considering it needs some time to measure). Is there any way to cancel this? If yes please also do in unload. Else is there any maximum time because elsoe you might ned to consider it as stopTimeout in io-package (normally the controller expect the proecess to be finished after 500ms ... is this really always the case here?
Please make sure to check compact mode usage. Especially all relevant stop/retsart cases

Needs fix: Please also exscute integration tests on travis

added,  please note its not mandatory in adapter creation templates anymore

index_m is current? feels like dummy files?!

removed html_m as I don't use a. config page

makes it really sens to always do require('speedtest-net')()... ?! This always creates a new function instance, or ?! That feels very strange

No, it does not fixed :)

https://github.com/iobroker-community-adapters/ioBroker.web-speedy/blob/master/main.js#L177 ... admin will most like not catch up new state Lists without an object reload

Hmm, I don't see another way handling this creating states will result in same behavior.
If you have any other suggestions than accepting the reload I am open for it :)

timeouts not cleared ;-) gggrrrrrrrr ...

Shame one me my friend, fixed !

When such a test is running )considering it needs some time to measure). Is there any way to cancel this? If yes please also do in unload. Else is there any maximum time because elsoe you might ned to consider it as stopTimeout in io-package (normally the controller expect the process to be finished after 500ms ... is this really always the case here?

Please make sure to check compact mode usage. Especially all relevant stop/retsart cases

compact mode disabled, this adapter us much to much memory intensive to run in compact mode on low hardware so should be avoided.....


Fix UnsafeByteBufUtil#setBytes() cause JVM crash (#10791)

Motivation:
Passing a null value of byte[] to the Unsafe.copyMemory(xxx) would cause the JVM crash
Modification:
Add null checking before calling PlatformDependent.copyMemory(src,  xxx)
Result:
Fixes #10791 .

@Ech0Fan can you add a unit test as well and can you also sign our icla:
https://netty.io/s/icla

@Ech0Fan can you add a unit test as well and can you also sign our icla:
https://netty.io/s/icla

Yes I wanted to add a unit test, but seems I didn't see any unit test for UnsafeByteBufUtil, so... should I add a unit test named UnsafeByteBufUtilTest even if it is A non-public class

Thanks @Ech0Fan. Couple of comments:

Probably should move the null check outside of the nonzero length check block
I think we also need to do bounds check on src relative to srcIndex and length

Thanks for comments.
I think that would be nice.

@Ech0Fan yes please just add a unit test for the class... its package private so it will work :)

@Ech0Fan yes please just add a unit test for the class... its package private so it will work :)

I've added the unit test,  verify it please

@netty-bot test this please

@Ech0Fan thanks a lot!


Add memory profiler ?

It is based on line_profiler, works with the same decorator @profile and has the same kind of output : https://github.com/fabianp/memory_profiler

I would VERY much like to see this. Please? Pretty please? :)

Well, PR welcomed ! :)
I never actually used a memory profiler, but I might start if it is implemented in spyder...

I'm in the same boat, never used, because it wasn't implemented in Spyder, but I'd like to. :)
As for a PR, I have no experience coding user interface stuff and really didn't understand most of the spyder_line_profiler code. Is there anything in particular I can help with?


Kernel Verifier broken on platform.dill

Unhandled exception:
patched_sdk/lib/async/stream.dart:93:29: Verification error: StaticGet of 'dart.async::_EmptyStream::' without getter.

I need a command line that reproduces this problem.

Here's the patch:
diff --git a/pkg/kernel/bin/transform.dart b/pkg/kernel/bin/transform.dart
index 544e7aae7b..977ed7174a 100755
--- a/pkg/kernel/bin/transform.dart
+++ b/pkg/kernel/bin/transform.dart
@@ -18,7 +18,7 @@ import 'package:kernel/transformations/empty.dart' as empty;
 import 'package:kernel/transformations/method_call.dart' as method_call;
 import 'package:kernel/transformations/mixin_full_resolution.dart' as mix;
 import 'package:kernel/transformations/treeshaker.dart' as treeshaker;
-// import 'package:kernel/verifier.dart';
+import 'package:kernel/verifier.dart';
 import 'package:kernel/transformations/coq.dart' as coq;

 import 'util.dart';
@@ -111,7 +111,7 @@ Future<CompilerOutcome> runTransformation(List<String> arguments) async {
   // TODO(30631): Fix the verifier so we can check that the transform produced
   // valid output.
   //
-  // verifyProgram(program);
+  verifyProgram(program);

   if (format == 'text') {
     writeProgramToText(program, path: output);

Here are the reproduction commands:
DART_CONFIGURATION=ReleaseX64 ./pkg/front_end/tool/fasta compile pkg/kernel/lib/ast.dart
out/ReleaseX64/dart -c ./pkg/kernel/bin/transform.dart -t empty -o /dev/null pkg/kernel/lib/ast.dart.dill

This issue is due to the way we handle redirecting constructors, for example:
static factory •<T extends core::Object>() → link::LinkBuilder<link::LinkBuilder::•::T>
   let
     dynamic #redirecting_factory = lin::LinkBuilderImplementation::•
   in
     invalid-expression;

This has been a known issue for a while. If my memory serves, then Peter and Kevin were discussing some possible solutions to this problem.
Right now we have the kernel verifier disabled, which I think is a big red flag - it catches bugs early on and we should have it enabled.
Precisely because of this, I wanted to land cl/3002053002, which changes verifier.dart to allow this pattern pass verification until the redirecting factory issue has been solved!
@peter-ahe-google  / @kmillikin Can we make the verifier.dart change?

Blocked on #29841

Fixed now.


Issue with setting up AR scene in landscape

I create an AR scene with objects positioned off real world coordinates.
I get the objects position and translate them into Viro AR coorindates.
My app is locked in Portrait mode, and when I load my AR scene in portrait mode everything seems to be positioned fine. But when I have my device in landscape, and load my AR scene they seem to be positioned incorrectly, like there off by 90 degrees of so.
How does the scene get setup, should I make adjustments based off the orientation of the device when setting up the scene?

Hi @Coolspot91 ,
We'd be glad to help. We'd perhaps need a little more info from you to be able to look at what issues you are having.

Are you building on iOS or Android? Would help in identifying the root cause?
Are you building a stand alone app or is this through our test bed app?
Can you share screenshots of exactly what you see?

Not entirely sure of what problem you are facing, but we achieve the same behavior (locking app to portrait mode in our Figment AR app. Check it out on App Store and Play Store, if you haven't yet.
Copy pasting below portrait orientation related code snippets from our Android app for headstart:
These lines in AndroidManifest.xml
    <uses-feature android:name="android.hardware.camera.ar" android:required="true"/>
    <uses-feature android:name="android.hardware.screen.portrait" />

and activity clause as follows
<activity
        android:name=".MainActivity"
        android:label="@string/app_name_figment"
        android:configChanges="keyboard|keyboardHidden|orientation|screenSize"
        android:windowSoftInputMode="adjustResize"
        android:screenOrientation="portrait">

Closing this issue due to inactivity. Please reopen if you are still having problem with additional comments and we can investigate further. Thanks


RL Training not Seeing Gains from More Waypoints

Test to see if the correct waypoints are being given to observations

Closing since this is quite old and not specific enough. A number of waypoint improvements have gone in recently.


use pigz instead of gzip

Apparently this could make backups much faster.

Hi @taf2, thanks for the suggestion. gzip is more common to use, but you can use whatever compressor you like. You can configure a custom compressor as described here: http://backup.github.io/backup/v4/compressor-custom/


SAM Inference fails: Client Error: Not Found for url: http://<IP_ADDRESS>:8080/data/upload/1/aa6a641e-OIP.jpeg"

Hello,
I am trying to use the SAM ML Backend in Label Studio. I have followed the guide from Getting Started Using Segment Anything with Label Studio. I did everything as instructed, but every time I try to label using Auto-Detect, it fails to create any segmentation labels.
Here is the log:
[2024-01-09 07:36:11,470] [DEBUG] [label_studio_ml.api::log_response_info::143] Response body: b'{"detail":"HTTPError: 404 Client Error: Not Found for URL: http://<IP_ADDRESS>:8080/data/upload/1/aa6a641e-OIP.jpeg","request":{},"result":{"traceback":"Traceback (most recent call last):\n File \"/usr/local/lib/python3.8/site-packages/label_studio_ml/exceptions.py\", line 39, in exception_f\n return f(*args, **kwargs)\n File \"/usr/local/lib/python3.8/site-packages/label_studio_ml/api.py\", line 61, in _predict\n predictions = model.predict(tasks, context=context, **params)\n File \"/app/model.py\", line 48, in predict\n predictor_results = PREDICTOR.predict(\n File \"/app/sam_predictor.py\", line 195, in predict\n return self.predict_sam(img_path, point_coords, point_labels, input_box)\n File \"/app/sam_predictor.py\", line 167, in predict_sam\n self.set_image(img_path, calculate_embeddings=False)\n File \"/app/sam_predictor.py\", line 83, in set_image\n image_path = get_image_local_path(\n File \"/usr/local/lib/python3.8/site-packages/label_studio_ml/utils.py\", line 66, in get_image_local_path\n image_local_path = get_local_path(\n File \"/usr/local/lib/python3.8/site-packages/label_studio_tools/core/utils/io.py\", line 105, in get_local_path\n r.raise_for_status()\n File \"/usr/local/lib/python3.8/site-packages/requests/models.py\", line 1021, in raise_for_status\n raise HTTPError(http_error_msg, response=self)\nrequests.exceptions.HTTPError: 404 Client Error: Not Found for URL: http://<IP_ADDRESS>:8080/data/upload/1/aa6a641e-OIP.jpeg\n"},"status":500}\n'
Here is my docker-compose.yml:
version: "3.8"

services:
  server:
    container_name: server
    image: humansignal/sam:v0
    build:
      context: .
      shm_size: "4gb"
    deploy:
      resources:
        limits:
          memory: 16G
        reservations:
          memory: 8G
    environment:
      # Change this to your model name
      - SAM_CHOICE=MobileSAM
      - LOG_LEVEL=DEBUG
      # Add these variables if you want to access the images stored in Label Studio
      - LABEL_STUDIO_HOST=http://<IP_ADDRESS>:8080
      - LABEL_STUDIO_ACCESS_TOKEN=8681fe6e75faaf151c8f13588c6ee977586dfba9
    ports:
      - 9090:9090
    volumes:
      - "./data/server:/data"

I tried to open the image I upload in http://<IP_ADDRESS>:8080/data/upload/1/aa6a641e-OIP.jpeg, but it can't open the image and Access Error: 404 -- Not Found
Any suggestions and help would be greatly appreciated. Thank you!

I have the same problem

This problem is fixed in the latest version of ml backend and label-studio-sdk (>1.0.5).


DOWNLOAD FOLDER DOESNT APPEAR

Describe the bug
The app does not detect the new ones that it downloaded in "Downloads"
To Reproduce
Steps to reproduce the behavior:

Rename "Download" folder to anything else, and then rename it as it was.
Open aves
Search "Download" folder
does not appear

Expected behavior
Let Aves detect the "Downloads" folder again and the new images appear.
Screenshots
If applicable, add screenshots or screen recordings to help explain your problem. If they are too private for this public space, feel free to send them by email.
System information and logs:
Package: deckers.thibault.aves
Installer: com.android.vending
Aves version: 1.11.9-play, build 128
Flutter: stable 3.24.0
Android version: 9, API 28
Android build: PPPS29.55-35-18-7
Device: motorola moto g(6) play
Support: dynamic colors=false, geocoder=true, HDR=false
Mobile services: ready
Connectivity: wifi
System locales: es_AR
Storage volumes: /storage/FF07-1E20/, /storage/emulated/0/
Storage grants:
Error reporting: true
Additional context
I would like there to be an option to search for albums in the system whenever I want.
aves-logs-20240810_222410.txt

FIXED

Remember I made a backup of the download folder, deleted the folder and then downloaded a new image from the browser, birds detected Download now and then I put the files back in the download folder


Get " net/http: invalid header field name ""

hi,
got this issue while retrieving my csv
Get "http://<IP_ADDRESS>:12345/test.csv": net/http: invalid header field name ""
my csv looks like :
Time,1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22,23,24,25,chaud Z1,chaud Z2,chaud Z3,froid Z1,froid Z2,froid Z3,regul Z1,regul Z2,regul Z3 2020-12-03 10:30:30,15.32,15.42,14.31,15.12,15.62,15.62,14.51,14.71,16.02,14.84,15.22,14.71,14.51,15.42,15.01,16.02,14.71,15.42,15.42,14.54,14.94,15.42,15.52,15.09,15.04,250.37,249.62,250.65,249.89,250.68,250.98,250.10,250.01,250.06 2020-12-03 10:30:35,15.32,15.42,14.31,15.12,15.62,15.62,14.61,14.71,16.02,14.84,15.22,14.71,14.51,15.42,15.01,16.02,14.71,15.42,15.42,14.54,14.84,15.42,15.52,15.09,14.94,250.37,249.62,250.66,249.89,250.67,250.97,250.10,250.02,250.05 2020-12-03 10:30:40,15.32,15.42,14.31,15.12,15.62,15.62,14.61,14.61,16.02,14.84,15.22,14.71,14.51,15.42,15.01,16.02,14.71,15.42,15.42,14.54,14.84,15.42,15.52,15.09,14.94,250.37,249.62,250.66,249.88,250.67,250.97,250.07,250.02,250.07 2020-12-03 10:30:45,15.32,15.42,14.31,15.12,15.62,15.62,14.61,14.61,16.02,14.84,15.32,14.71,14.51,15.42,15.01,16.02,14.71,15.42,15.42,14.54,14.84,15.42,15.52,15.09,14.94,250.37,249.62,250.66,249.88,250.67,250.98,250.04,250.02,250.07
meaning the dates are not in colums but in lines (as most of the CSV logging somethign does.
is your plugin able to withstand this kind of presentation ?

Your CSV looks fine to me and should be supported. The error is coming from net/http, which leads me to believe it's a connectivity issue. Have you added any custom headers in your data source config?

hi,
i re did the job on my local computer , i didnt add any custom header and use simple httpserver module from python .
python -m SimpleHTTPServer 12345
Serving HTTP on <IP_ADDRESS> port 12345 ...
<IP_ADDRESS> - - [30/Dec/2020 10:24:14] "GET /test.csv HTTP/1.1" 200 -
<IP_ADDRESS> - - [30/Dec/2020 10:24:25] "GET /test.csv HTTP/1.1" 200 -
still get the error.
can it be ... file encoding (CR/LF windows/unix style ? )
or the python http module is not sending correctly some expected header maybe ?
i ll give a try with another http server. (i definitely upvote for local csv upload in order to ease the use of grafana for "one shot users that are not admin or powerusers or database gurus"

I created a GitHub Gist with your CSV content which seems to work.

The plugin uses a fairly naïve approach when it comes to detecting timestamps. It basically tries to parse a list of common formats, among which yours is present. One thing to note here is that since your data doesn't contain time zone information, it will assume UTC.
Local CSV is a heavily requested feature, and I will start working on it during this week.

Here's a schema that seems to work for your data:

i ll give a try next week, I tried to do exactly that but couldnt get a graph, I ll get on it again.
the final apparoach is to be able to make grafana eating back is own CSV's :-) (this csv was exported bay the inspect menu of grafana.

i ll give a try next week, I tried to do exactly that but couldnt get a graph, I ll get on it again.
the final apparoach is to be able to make grafana eating back is own CSV's :-) (this csv was exported bay the inspect menu of grafana.

managed to graph.
this means Grafana is able to eat its own food since this CSV was created by grafana !!! (in the inspect menu you have options to export data as a CSV.
this means also with the sole installation of grafana you are able to dig inside all the PLC/industrial softwares that only knows the excel/csv export formats.
and this is to my mind very important since you can quickly export a dataset and share it to another guy with grafana without the need of linking to the database (this problematic is very common on industries , ppl in offices just need something like a viewer but do not necessary have a realtime direct access to the data produced by the process/ machines.
@marcusolsson the next big showstopper for the CSV's may be to let the user choose the date format (in order not to miss the impossible to guess automatically : month and day permutation on some locales ( MM/DD/YYYY vs DD/MM/YYYY).
do you want me to raise a special issue for this point in order to trace it better ?

managed to graph.
this means Grafana is able to eat its own food since this CSV was created by grafana !!! (in the inspect menu you have options to export data as a CSV.
this means also with the sole installation of grafana you are able to dig inside all the PLC/industrial softwares that only knows the excel/csv export formats.
and this is to my mind very important since you can quickly export a dataset and share it to another guy with grafana without the need of linking to the database (this problematic is very common on industries , ppl in offices just need something like a viewer but do not necessary have a realtime direct access to the data produced by the process/ machines.
@marcusolsson the next big showstopper for the CSV's may be to let the user choose the date format (in order not to miss the impossible to guess automatically : month and day permutation on some locales ( MM/DD/YYYY vs DD/MM/YYYY).
do you want me to raise a special issue for this point in order to trace it better ?

Currently, the plugin detects date format by simply attempting to parse different formats.
https://github.com/marcusolsson/grafana-csv-datasource/blob/c0461b120353106d93f588b5fbef4f58b00572e5/pkg/csv.go#L185-L229
Hopefully, most date formats should already be covered, and I'd be happy to consider adding more formats. But yes, for more obscure date formats you'd have to provide an explicit format string. I was hoping to avoid it for UX purposes, but if there's enough people asking for this, I'd be willing to add it. If this would be useful to you, please create another issue where we can discuss this further.
I'm closing this issue as it seems like you got it working in the end. Let me know if the problem reappears and I'll reopen it!

Currently, the plugin detects date format by simply attempting to parse different formats.
https://github.com/marcusolsson/grafana-csv-datasource/blob/c0461b120353106d93f588b5fbef4f58b00572e5/pkg/csv.go#L185-L229
Hopefully, most date formats should already be covered, and I'd be happy to consider adding more formats. But yes, for more obscure date formats you'd have to provide an explicit format string. I was hoping to avoid it for UX purposes, but if there's enough people asking for this, I'd be willing to add it. If this would be useful to you, please create another issue where we can discuss this further.
I'm closing this issue as it seems like you got it working in the end. Let me know if the problem reappears and I'll reopen it!


Add ability to connect to bookkeeper using tls

Is your feature request related to a problem? Please describe.
I have a bookkeeper cluster that has TLS enabled and need pulsar clients to communicate to my cluster using tls
Describe the solution you'd like
The ability to specify tls settings for my broker to connect to bookkeeper
Describe alternatives you've considered
Using stunnel + iptables to redirect

This functionality would be very useful for running a secure cluster and allowing a separate Pulsar functions cluster to also communicate with BK. Super interested in having that capability.

But for TLS, It need these configs:
http://bookkeeper.apache.org/docs/latest/security/tls/#configuring-clients
We need add and support these config in BookKeeperClientFactoryImpl, which is similar to bk client config in it.

Any update on this issue?  Is there an ETA for this?

@david-streamlio
I think we have added broker to bookie mTLS support with this PR: https://github.com/apache/pulsar/pull/5042
https://github.com/apache/pulsar/blob/master/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java#L792-L818
Is that you are looking for?

Yes, thank you!!


next-less is using<EMAIL_ADDRESS>

With the latest version of less (3.10.0) my builds are breaking.
I found this related issue at the less repo: https://github.com/less/less.js/issues/3414
I had to fix less to version 3.9.0 for it to keep building.

Yes 3.10.0 is breaking update. you loader must be upgrade 5.0.0.

With the latest version of less (3.10.0) my builds are breaking.
I found this related issue at the less repo: less/less.js#3414
I had to fix less to version 3.9.0 for it to keep building.

Yes 3.10.0 is breaking update. you loader must be upgrade 5.0.0.


Use awesome_bot for link validation.

Cool :)

CommonMark plugin can be removed from the Markup section.
I wonder if we should remove that section alltogether and move the Geshi one to Templating section.

Can we --allow-dupe for now?

404 http://www.meetup.com/nyc-cakephp/ other than that it looks good.

http://www.meetup.com/nyc-cakephp/  is accessible for me.

Silly me it's a 404 page.

Victory!

What do you think about removing the Markup section in favor of Templating?

Do it.

@dereuromark I think the guy who "owned" it ended up not renewing?


Unregistering metric reporting for closed shards

Description of changes:
Unregister metric reports for closed/complete shards. Fixes a bug where confusing aggregated metrics are shown after resharding.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks @elphastori . Looks good to me, merging


YYSpriteSheetImage has memory issues?

Hi, I use YYLabel to display text with emoji in a message app. In the text parser the the label, I returned a YYSpriteSheetImage (with is also a UIImage) attachment.
When the message shows, the memory uses grows even I don't do anything. And after 4 or 5 times of memory warning, the app crashes.
This problem should be reproduced easily when there are about 50 emojis showing at the same time, the memory grows fast.
Also, when I check this with the Allocation profiler, I saw a massive CG Raster Data memory not being deallocated.
Please help me about where I was doing wrong, I am despairing

I'v tested add 10 sprite image in a same view, the memory usage is normal but the cpu usage is very high.
If you use sprite image in a single view, it will decode all images into memory and it may cause memory warning. I recommend to use the frame based image or animated image (GIF/APNG/WebP) to reduce memory and CPU cost.


error : cannot call the function

Hi, would you mind giving more details as to what's wrong, please?


Added codecov support.

Issue #: https://github.com/bcgov/entity/issues/5241
Description of changes:

Add codecov configuration yaml file;
Add "--coverage" parameter into unit test command;
Add upload coverage report action into CI;
Add codecov badge in Readme;

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-business-create-ui license (Apache 2.0).

Wow, looks good! Whats the coverage at now, i'm scared to find out?

https://codecov.io/gh/bcgov/bcrs-business-create-ui


Is it possible to remove default Titlebar and other window chrome?

I've quite liked my (admittedly super limited) experience with this so far, and was wondering if there was a way to hide the default chrome on windows (not just on Windows) and then create my own titlebar and controls and such
I'm unsure whether this is something that hasn't been implemented or if I've just missed something, any help/advice/etc would be appreciated :D
✌

Hm. Not sure if you are asking about the window decoration for the parts created with Nuklear, or the system window decorations. For the first - I haven't tried that, frankly speaking, but think it is possible, and just depends on how you declare those. For the second - the answer severely depends on the Nuklear backend you use. For instance, for nuklear-backend-gfx, while using glium as a window context provider this may help. Same for winit. For GDI you need to look for the same window context setup in winapi, fist in the native API, then whather this API is supported in winapi crate. Hope this helps.


fix(security): upgrade semver and fast-xml-parser

Solves #1078
https://github.com/advisories/GHSA-c2qf-rxjj-qqgw
https://github.com/advisories/GHSA-gpv5-7x3g-ghjv

Pushed a new commit as meanwhile semver got a patch update to 7.5.3

Thanks @devinea. Update version-overrides and also enhanced overrides entry. Ready for re-review.


Add Dependabot for GitHub Actions

This PR adds Dependabot checking for GitHub Actions
As the current checkout action is out of date, it is expected that Dependabot will open an update PR if and when this is merged.

Cool. Thank you!


Insert - Should the variable be pasted?

Should the variable be pasted after clicking insert into text field?

@dawid-g fixed

verified as fixed.


missing SDK support for premium disk v2

Bug Report

import path of package in question, e.g. .../services/compute/mgmt/2018-06-01/compute
SDK version : latest

Specify the exact commit if possible; one way to get this is the REVISION
column output by go list -m <module>, for example go list -m github.com/Azure/azure-sdk-for-go/sdk/azcore.

output of go version

- What happened?
Trying to create new premimum disk V2, could not find this type.
https://github.com/Azure/azure-sdk-for-go/blob/sdk/resourcemanager/compute/armcompute/v1.0.0/sdk/resourcemanager/compute/armcompute/zz_generated_constants.go#L527
const (
// DiskStorageAccountTypesPremiumLRS - Premium SSD locally redundant storage. Best for production and performance sensitive
// workloads.
DiskStorageAccountTypesPremiumLRS DiskStorageAccountTypes = "Premium_LRS"
// DiskStorageAccountTypesPremiumZRS - Premium SSD zone redundant storage. Best for the production workloads that need storage
// resiliency against zone failures.
DiskStorageAccountTypesPremiumZRS DiskStorageAccountTypes = "Premium_ZRS"
// DiskStorageAccountTypesStandardLRS - Standard HDD locally redundant storage. Best for backup, non-critical, and infrequent
// access.
DiskStorageAccountTypesStandardLRS DiskStorageAccountTypes = "Standard_LRS"
// DiskStorageAccountTypesStandardSSDLRS - Standard SSD locally redundant storage. Best for web servers, lightly used enterprise
// applications and dev/test.
DiskStorageAccountTypesStandardSSDLRS DiskStorageAccountTypes = "StandardSSD_LRS"
// DiskStorageAccountTypesStandardSSDZRS - Standard SSD zone redundant storage. Best for web servers, lightly used enterprise
// applications and dev/test that need storage resiliency against zone failures.
DiskStorageAccountTypesStandardSSDZRS DiskStorageAccountTypes = "StandardSSD_ZRS"
// DiskStorageAccountTypesUltraSSDLRS - Ultra SSD locally redundant storage. Best for IO-intensive workloads such as SAP HANA,
// top tier databases (for example, SQL, Oracle), and other transaction-heavy workloads.
DiskStorageAccountTypesUltraSSDLRS DiskStorageAccountTypes = "UltraSSD_LRS"
)

What did you expect or want to happen?
Add new support for premimum disk v2

How can we reproduce it?

Anything we should know about your environment.
latest go version

I see we have
DiskStorageAccountTypesPremiumV2LRS DiskStorageAccountTypes = "PremiumV2_LRS"
in our github,  info on 3rd party side need update
https://pkg.go.dev/github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/compute/armcompute#DiskStorageAccountTypes


Cannot update membership tier descriptions

Issue Summary
I can't seem to add/update the description for any membership tiers. Entering a description and then saving clears out whatever value was entered. The database also does not get the updated value either.
https://user-images.githubusercontent.com/13506/199175848-c10bb422-68ff-4622-86c2-1b9ece1ae716.mov
I've tested this with fresh installs of Ghost both locally and on a DO droplet where I host several other Ghost instances. This functionality appears to be working fine on another instance of Ghost running v5.11 but not on the latest version at the time of this report v5.22.1.
Steps to Reproduce

Install a new Ghost instance (v5.22.1)
Connect Ghost to Stripe account
Add a description to any membership tier
Description does not save and no error message is displayed

Ghost Version
5.22.1
Node.js Version
14.17.0
How did you install Ghost?
Locally / Ubuntu 18.04
Database type
SQLite3
Browser & OS version
No response
Relevant log / error output
No output in error logs

Code of Conduct

[X] I agree to be friendly and polite to people in this repository

This has been fixed and released. Please update to 5.22.2 :)


Implement autoformatter for flatbuffers files

Flatbuffers doesn't have any code formatter nor LSP server available as far as I can tell.
We can easily implement one though, since we already do all the semantic parsing necessary as part of project hope.

Nice, thanks @kristoff3r! I'll try it out as soon as I get a minute.


[Bug]: The rest endpoint is in the background odata. But it doesn't present an OData MetadataEndpoint

What happened?
The /rest/... of the genereated API contains at the end technically an ODATA endpoint. But the problem is for many API tools is, that the generated part doesn't make the $metadata endpoint accessible. This endpoint is for example used in PowerBi to be connected to an OData API and get the schema and all valid endpoints from it.
So my expactation would be that I could call:
rest/$metadata and get the XML schema of the OData feed back.
Currently I get:
{"error":{"code":"EntityNotFound","message":"Invalid Entity path: $metadata.","status":404}}

Version
Azure Static Web Apps CLI (1.1.3)
What database are you using?
Azure SQL
What hosting model are you using?
Static Web Apps (SWA)
Which API approach are you accessing DAB through?
REST
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi @paule96,
Thank you for raising this. The REST endpoint support we enabled so far tries to adhere to the Microsoft REST API guidelines here. And we indeed added OData style $filter clauses. However, we haven't completely added the OData support. This would be a new feature request and not considered as a bug.
In the meantime, if you are interested in getting the metadata of the exposed REST endpoints, you could retrieve the OpenAPI document which the latest version of Data API builder - 0.7.6 - generates.
This OpenAPI description document (also referred to as a schema file) is generated using the developer provided runtime config file and the database object metadata for each REST enabled entity defined in the runtime config file. The schema file is generated using functionality provided by the OpenAPI.NET SDK. Currently, the schema file is generated in adherence to OpenAPI Specification v3.0.1 formatted as JSON.
The OpenAPI description document can be fetched from Data API builder from the path:
GET /{rest-path}/openapi
In Development mode, Data API builder enables viewing the generated OpenAPI description document from a dedicated endpoint:
GET /swagger

Just to have the correct informations, if I want to implement the metadata document the following things are a todo:

parse and implement relations in the EdmModelBuilder
add in the RestController or inside the RestService logic to responde to the $metadata path

This work would enable also the implementation of $expand inside the RequestParser, because the relations are required for the metadata endpoint, and can be accessed then by the rest of the framework. The important thing for the implementation of the $expand stuff is to respect the permissions of the user.


 I've changed the meaning of this ocean depth to be 'height above -1000m' instead of what it used to be which was something like 'water depth plus 100'. some more information from the commit:
We need '0' to mean deep water. This is required so that when lods are faded in/out, it can fade to 0 (deep). It also means that outside the biggest lod the value can be 0 and not appear to be shallow. This fix changes the meaning of the depths - it is now the height of the sea floor above some deep value (currently -1000m).
the reason for this change: i discovered that the outer skirt of geometry was being shaded as shallow water. so there was a bright blue region near the horizon. it wasn't noticeable for some views but would become worse with less LODs or for high viewpoints, so im changing the meaning of depth so it works in all scenarios.
if you have any thoughts etc let me know

Looks fine to me, we ended up using your depth offset anyway.
we do have a heavily modified ocean.shader because we have about 12 ocean
params that we change per pixel (we store this info in textures a bit like
the way you cache the ocean depth, except its at a constant very low res,
rather than a different res per lod)
On Sun, Sep 23, 2018 at 8:07 PM, Huw Bowles<EMAIL_ADDRESS>wrote:

FYI @dizzy2003 https://github.com/dizzy2003 - i'm aware you guys have
your own ocean depth shaders? I've changed the meaning of this ocean depth
to be 'height above -1000m' instead of what it used to be which was
something like 'water depth plus 100'. some more information from the
commit:
We need '0' to mean deep water. This is required so that when lods are
faded in/out, it can fade to 0 (deep). It also means that outside the
biggest lod the value can be 0 and not appear to be shallow. This fix
changes the meaning of the depths - it is now the height of the sea floor
above some deep value (currently -1000m).
the reason for this change: i discovered that the outer skirt of geometry
was being shaded as shallow water. so there was a bright blue region near
the horizon. it wasn't noticeable for some views but would become worse
with less LODs or for high viewpoints, so im changing the meaning of depth
so it works in all scenarios.
if you have any thoughts etc let me know
You can view, comment on, or merge this pull request online at:
https://github.com/huwb/crest-oceanrender/pull/76
Commit Summary

Gracefully handle case where no ocean present
Wow - ?? operator doesnt work with serialised props
Fix for ocean depth baseline

File Changes

M src/unity/Assets/Crest-Examples/Scripts/OceanDebugGUI.cs
https://github.com/huwb/crest-oceanrender/pull/76/files#diff-0 (6)
M src/unity/Assets/Crest/Scripts/Helpers/RenderAlphaOnSurface.cs
https://github.com/huwb/crest-oceanrender/pull/76/files#diff-1 (6)
M src/unity/Assets/Crest/Scripts/LodData/LodDataSeaFloorDepth.cs
https://github.com/huwb/crest-oceanrender/pull/76/files#diff-2 (4)
M src/unity/Assets/Crest/Scripts/OceanDepthCache.cs
https://github.com/huwb/crest-oceanrender/pull/76/files#diff-3 (9)
M src/unity/Assets/Crest/Scripts/RenderOceanDepth.cs
https://github.com/huwb/crest-oceanrender/pull/76/files#diff-4 (2)
M src/unity/Assets/Crest/Scripts/Shapes/ShapeGerstnerBatched.cs
https://github.com/huwb/crest-oceanrender/pull/76/files#diff-5 (8)
M src/unity/Assets/Crest/Shaders/Ocean.shader
https://github.com/huwb/crest-oceanrender/pull/76/files#diff-6 (7)
M src/unity/Assets/Crest/Shaders/OceanEmission.hlsl
https://github.com/huwb/crest-oceanrender/pull/76/files#diff-7 (4)
M src/unity/Assets/Crest/Shaders/OceanLODData.hlsl
https://github.com/huwb/crest-oceanrender/pull/76/files#diff-8 (6)
M src/unity/Assets/Crest/Shaders/Resources/OceanDepths.shader
https://github.com/huwb/crest-oceanrender/pull/76/files#diff-9 (15)
M src/unity/Assets/Crest/Shaders/Resources/OceanDepthsCache.shader
https://github.com/huwb/crest-oceanrender/pull/76/files#diff-10 (4)
M src/unity/Assets/Crest/Shaders/Shapes/ShapeGerstnerBatch.shader
https://github.com/huwb/crest-oceanrender/pull/76/files#diff-11 (2)
M src/unity/Assets/Crest/Shaders/Simulation/Resources/
ShapeSimFoam.shader
https://github.com/huwb/crest-oceanrender/pull/76/files#diff-12 (2)
M src/unity/Assets/Crest/Shaders/Simulation/Resources/
ShapeSimWave.shader
https://github.com/huwb/crest-oceanrender/pull/76/files#diff-13 (2)

Patch Links:

https://github.com/huwb/crest-oceanrender/pull/76.patch
https://github.com/huwb/crest-oceanrender/pull/76.diff

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/huwb/crest-oceanrender/pull/76, or mute the thread
https://github.com/notifications/unsubscribe-auth/AOU9GPMLaKyJdQ_6rHtzqTgTfse6TyMzks5ud11XgaJpZM4W1o60
.

Interesting. Thanks for the note. If there's anything we could simulate in loddata's instead of baking out to textures, let me know, but i can imagine these are artist authored and not replaceable with anything automatic.
Merging this PR.


Bug in decision tree example

Line 47: let entropy_pred_y = gini_model.predict(&test);
should be let entropy_pred_y = entropy_model.predict(&test);

that slipped through review, thanks! I pushed a fix in e50a291b


Makefile cleanup

cleanup make vet target duplicates
enforce no leftovers from make bundle instead of just make manifests

/ok-to-test

/test precommit-check

Somehow the make bundle in pre-commit is using a bad version for the operator version.  It looks like it's using the Golang version somehow:
/go/src/github.com/openstack-k8s-operators/operator/bin/kustomize build config/manifests | operator-sdk generate bundle -q --overwrite --version 1.19
Error: invalid command options: 1.19 is not a valid semantic version: No Major.Minor.Patch elements found


New optional categorical field (channel quality) in .ome-tiff.channels.csv

Select the target metadata schema name (can be multiple), or choose 'Other' if not listed.
Other
If 'Other' is selected, please enter the schema name below.
.ome-tiff.channels.csv
Describe the change request (use a numbered list if many).
Field name: channel quality
Field type: categorical
Definition: This is a rough metric of the quality of the individual channel within the image. If any channels are denoted as "poor" quality, then the reason for including the poor channel(s) must be noted in the abstract for this dataset.
Options: Optional
Values: good, fair, poor

Added in HRAVS v2.6.0


__scan and __rand should not have to take an argument

Currently, the __scan and __rand intrinsics take an identifier as an argument, to determine where to store the random or fetched value. This should not be necessary; the implementation can just return a temporary. This will simplify the stdlib implementations of rand() and scanc() somewhat. E.g.

function c = scanc()
{
/*
    let c;
    __scan(c);
*/
    let c = __scan();
}

Fixed


Feature fenebondshift

I have made  changes to the BondEvaluatorFene.h and associated files like module.cc, test_bond_fene.py and bond.py , like you have asked, in order to change the FENE bond potential to V = -0.5kR0R0ln(1-((r-delta)/R0)^2)

For some reason, I can't reply to your questions in the threads (probably since GitHub labels the lines "outdated" now that you addressed the comments), but:

We chose to error out when epsilon == 0 because FENE almost always needs to be used with a repulsive core since its minimum is 0. If this will cause an issue for you, we can remove that restriction. But, it's good practice to keep PRs small and focused on a specific set of changes, so we would want to handle that separately.
You found a mistake in the tests! Could you please apply the same correction in test_potential? We want to test the azplugins.bond.fene class and not hoomd.md.bond.fene. Probably the original developer used both versions to implement the test, and just forgot to switch to the right one. (And we missed it in code review).

There is a failing test (see output above) related to the energy in your new potential energy code. Could you please investigate? Note that you are able to run these tests locally as well using ctest if you want to check on your workstation before comitting.

Dear Mike,
Would you tell me the command to run these unit-tests. I think I found the mistake, but will run this before I commit.
Regards,
Jude

From: Michael Howard @.***>
Sent: Tuesday, March 1, 2022 3:45:45 PM
To: mphowardlab/azplugins
Cc: Vishnu, Jude Ann; Author
Subject: Re: [mphowardlab/azplugins] Feature fenebondshift (PR #61)
There is a failing test (see output above) related to the energy in your new potential energy code. Could you please investigate? Note that you are able to run these tests locally as well using ctest if you want to check on your workstation before comitting.
—
Reply to this email directly, view it on GitHubhttps://github.com/mphowardlab/azplugins/pull/61#issuecomment-1055520750, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AX5W2YTPWGGT7RHD6BKXPILU5YURTANCNFSM5PQWPGUQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID: @.***>

How to run tests is described in the README, but in the build directory:
make test

will run all tests, or you can invoke ctest with options to run only certain tests like:
ctest -R azplugins-test_bond_fene-*

will get you all the tests whose name matches this pattern


Trouble understanding results

I seem to be having some issues with business_time.  I'm not sure if I'm not using it properly or not understanding the results properly.  Can you explain how this result occurs?
2.3.1 :031 > d1 = DateTime.parse('2017-01-07 19:55:29.842345')
 => Sat, 07 Jan 2017 19:55:29 +0000
2.3.1 :032 > 3.business_days.after(d1)
 => Fri, 13 Jan 2017 09:00:00 +0000

I would expect 3 business days to be January 11th.  I have no idea how this results in the 13th.  Even more confusing, a later starting date (2 days later, Jan 9th 00:00), considers 3 business days from that date to be earlier than the previous date (Jan 7th).
2.3.1 :039 > d2 = DateTime.parse('2017-01-09 00:00:00.000000')
 => Mon, 09 Jan 2017 00:00:00 +0000
2.3.1 :040 > 3.business_days.after(d2)
 => Thu, 12 Jan 2017 09:00:00 +0000

I'm guessing this has something to do with the time.  Seems like a bug, or am I missing something?

@ryancrisp It's a known 'issue' which was discussed some time ago, see: https://github.com/bokmann/business_time/pull/116. As you can see, numerous issues have been opened to address this rather unintuitive behaviour but it doesn't seem like it's going to be changed anytime soon.

this is different than #116 and seems like a real bug.  both of your test cases should result in the Thursday date.
will investigate.


Include Dependencies in setup.py

Need to include sqlmodel and cryptography

I always thought that this was managed by find_packages(), but I don't have much experience with packaging.
I'll assign this to the first-release milestone.

find_packages() is for finding packages in your project while install_requires is for dependencies https://packaging.python.org/en/latest/guides/distributing-packages-using-setuptools/#packages


Scoped 'getConfiguration' is causing unexpected behaviour

VS Code Version: 1.53.2
OS Version: macOS Catalina (10.15.7)

Steps to Reproduce:

Generate a showWorkspaceFolderPick
Pass returned folders uri as a scope property in getConfiguration

Example:
    const quickPickWorkspace =  async () => {
        const folder = await vscode.window.showWorkspaceFolderPick({ placeHolder: 'Pick the workplace folder...' });
        if (folder) {
	        console.log(folder.uri); // Always correct / different
	        const configuration = vscode.workspace.getConfiguration('', folder.uri);
	        const currentValue = configuration.get('boxed-components.useTemplates');
	        console.log('Reading:', currentValue); // Always the same settings (incorrect)
        }
    }

The expected behaviour here is that getConfiguration should return the value from /.vscode/settings.json.
I believe this value keeps getting cached, I'm able to swap out which value keeps getting returned. The incorrect value is returned each time even if the workspace folder with that setting is removed from the explorer

Does this issue occur when all extensions are disabled?: Yes

You are changing your boxed-components.useTemplates setting in between calls to quickPickWorkspace() I assume?  But the updated setting is not being read correctly?
When I call quickPickWorkspace() (using your code) via a custom command I can't reproduce this behaviour.  How are you calling quickPickWorkspace()?
I am on W10 though, so can't replicate macOS.

@ArturoDent not exactly, I simply have two folders in my Workspace (two roots), both have .vscode/settings.json files, and both have a different value set in this configuration.
Here's the repo to this extension, it's very basic. In extension.js on line 88 you'll see the snippet above that I mentioned. Any eyes on this would be greatly appreciated! the ReadMe explains the data shape that the extension expects
https://github.com/kylewetton/vscode-boxed-components

I see you have not defined any scope to your setting 'boxed-components.useTemplates'. By default it will be scope to window and values from folders will be ignored while reading in a MR workspace.
You have to define your setting to be resource scoped so that it can be applied per folder in a MR folder workspace.
See the sample for reference - https://github.com/microsoft/vscode-extension-samples/blob/main/configuration-sample/package.json


CLI's starter template is incorrect

This is feedback for the firebase-tools repo, adding it here just because it's related to the Python EAP.
Running firebase init functions results in invalid functions code.

The import is incorrect. In the template, it is from firebase_functions import https, but it should be from firebase_functions import https_fn.
~firebase_admin is missing from requirements.txt~ edit: I'm not sure if this is needed
https should be renamed to https_fn in the function code

Created a PR: https://github.com/firebase/firebase-tools/pull/5569

PR is merged, fix will be available after the next CLI release

Fixed in<EMAIL_ADDRESS>


[FEATURE] Add Kilometers/Hour to Miles/Hour conversion and reverse

Description
Add a conversion utility snippet for Kph to Mph and one for Mph to Kph
PR Type

[x] Snippets, Tests & Tags (new snippets, updated snippets, re-tagging of snippets, added/updated tests)
[ ] Scripts & Website & Meta (anything related to files in the scripts folder, how the repository's automated procedures work and the website)
[ ] Glossary & Secondary Features (anything related to the glossary, such as new or updated terms or other secondary features)
[ ] General, Typos, Misc. & Meta (everything else, typos, general stuff and meta files in the repository - e.g. the issue template)

Guidelines

[x] I have read the guidelines in the CONTRIBUTING document.
[x] My PR doesn't include any testlog changes.

I totally agree. The archive would likely be the best place for this snippet.

Like I've already said in #764, snippets like these have real-world applications, but are limited to specific scenarios. I see the benefit of categorizing as Math, but I also think these are a good fit for the archive.
Adding to Math

PRO: More visibility - people who need these snippets will find them more easily.
PRO: The Math category will stop feeling kind of empty.
CON: They will clutter the npm package, which is not ideal.
CON: They are essentially just mathematical formulas applied and don't showcase much in terms of the language, so their teaching capability is quite limited.

Archiving

PRO: Less clutter in the main list and npm package.
CON: Harder to find, if someone is looking for them.

I'm leaning more towards archiving, but I think the archive itself needs a revamp and we should start listing it as a normal category in the website (contents/search etc.) and give it its own tags, so these snippets are more visible and do not clutter the main list and package. If we can manage to do that in the coming months, then I vote archive.

+1 on your opinions, would you be kind enough to move your snippets to snippet_archive/ ? 🙂


DateTime conversions yield inconsistent results because of UTC

When using DateTime fields, input DateTime are saved in the yyyy-MM-ddTHH:mm:ssZ format, but if the DateTime isn't converted to UTC before saving, the value of the same object returned by a SOQL query will yield a different date. This is because the expected input and output of Salesforce is UTC, and a conversion to UTC doesn't take place automatically on the input, but is automatically converted to local time on the output.
The workaround, of course, would be to convert all dates to UTC before saving, and expect local times on the return.
It would be nice to have the expected input match the expected output from Salesforce. Thus, if I send in a local time, the conversion would automatically set the correct UTC date, and then a subsequent SOQL query would return the same local time.
Simple test would be to:

Create an SObject with a date
Set the date field to a local date
Save the SObject
Query the SObject by ID to get a copy of the object
Compare dates

The test will fail if there isn't an explicit conversion after step 4 to convert the queried date back to UTC.
This does beg a larger look at a larger design feature. The automatic conversion at the back end is somewhat problematic once you start working with dates outside of your local time. Thus, if you serialize an object from another system that has a different timezone, then the date has to be set to an offset of your local time, and the test above would fail again. The work around for this, of course, would be to convert the dates to prior to a save and after a query, but a similar problem of tedium exists.

This is an excellent point. Perhaps the flow needs to be revisited.

for reference - https://developer.salesforce.com/docs/atlas.en-us.soql_sosl.meta/soql_sosl/sforce_api_calls_soql_select_dateformats.htm

If it's a DateTime - it would be nice to assume that all DateTime objects are UTC as you do today, and use the UTC format as you do today: YYYY-MM-DDThh:mm:ssZ. That leaves the timezone issues up to the user. It would be dangerous if a user had a DateTime in UTC but accidentally had the DateTimeKind set to local. Seems safer to assume that it's UTC as most users would probably only send a DateTime in as UTC knowing that they are working with a database that could be accessed from any time zone, and also knowing that DateTime does not have time zone data.
DateTimeOffset, however is a different situation. If its set to a timezone, we can use the offset format YYYY-MM-DDThh:mm:ss+hh:mm or YYYY-MM-DDThh:mm:ss-hh:mm, and if it is UTC, then YYYY-MM-DDThh:mm:ssZ.


[YUNIKORN-2936] Access K8s dashboard by port-forwarding

What is this PR for?
According to this reported issue, kubectl proxy strips the authorization header from the request, causing every login attempt to the K8s dashboard to result in a 401 Unauthorized error.
The recommended way to access the dashboard is by using port-forwarding.
What type of PR is it?

[ ] - Bug Fix
[x] - Improvement
[ ] - Feature
[ ] - Documentation
[ ] - Hot Fix
[ ] - Refactoring

What is the Jira issue?
https://issues.apache.org/jira/browse/YUNIKORN-2936

Thanks for reviewing!
1.6.0 versioned page has been updated, please check.


Remove the WASM WAVM ast2wasm Emscript support module and add tests.  (STACKED on wasm globals support PR)

Remove the WASM WAVM ast2wasm Emscript support module and add tests.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM if you could explain the goal of test case as we talked offline.


Hourlies and Pathchanging support

Feel free to use http://arn.ericchu.net/ to get hourlies and pathchanging support.
For example:

Path
Version
Binary url

arm9loaderhax.bin
release
http://arn.ericchu.net/release/arm9loaderhax.bin

arm9loaderhax.bin
hourlies
http://arn.ericchu.net/latest/arm9loaderhax.bin

a9lh/payload.bin
release
http://arn.ericchu.net/release/a9lh/payload.bin

Thanks for the offer!
I kinda wanted to do my own thing as a programming exercise and your service was the "if it doesn't work out I'll just use this guy's service".
I hope you won't feel bad but I'd like to avoid doing it unless I need to.
However if you ever implement a functionality that might be impractical to do on a 3DS I'll be happy to reconsider!
PS: Looking at the source code of your service, I see you do HTML scraping of the release page. Since you're using Node.js and therefore have JSON support builtin, you might want to consider using this api instead, that'll cover you from possible layout changes (and/or avoid Github being angry at you if you get a pretty big traffic surge).

lol ok. Cutting down on dependencies has its benefits :)
Thanks for the api suggestion. I was in a rush to get something working, but I'll definitely look into it.

I would like hour-lies and Path changing support too! Hope it's somehow implemented in the future with this app!

Hourlies are in!


Add Finder Service to add XMP data

Add the locationator write-xmp feature to a Finder Service.

Hi,
How is this coming along... any updates?
Thanks...

Haven't started. Will get done when it gets done 😀

Taking a slight detour from this...I figured out how to read the EXIF/IPTC/XMP metadata using native calls so exiftool not needed which will help with the service. Now working on being able to write this data natively. I think I can make this work without exiftool at all.

Taking a slight detour from this...I figured out how to read the EXIF/IPTC/XMP metadata using native calls so exiftool not needed which will help with the service. Now working on being able to write this data natively. I think I can make this work without exiftool at all.

Wow.... that is quite something...
Although on a separate note...
EXIFtool is used extensively by a lot of people, its read/write methods have been tested extensively, and support for new camera and metadata is updated very regularly.
Do you run the risk of having to update your code more frequently, as and when file formats change or some new versions of XMPdata get released?
Would it be better to support both (native and Exiftool) - since you have already written the EXIFtool wrapper?

Do you run the risk of having to update your code more frequently,

I'm using Apple's macOS APIs for reading/writing metadata so that's Apple's problem, not mine.

Would it be better to support both (native and Exiftool)

Nope, this is a side project for fun and adding more features means adding more maintenance and I only need a working solution, not a configurable solution.

I see another advantage that people would be find it easier to modify the EXIF calls in your code

They could do that with the native solution too -- it's python code just like the exiftool interface would be (If I do use exiftool, I'll use my own ExifTool python wrapper).
I haven't tried it yet but I suspect that calling exiftool from the app is going to run into problems just like the CLI installer did (#10), namely the subprocess won't be able to access files because it will run in a protected environment. Services work by passing a pasteboard with an NSURL to the serving app and the app has permission to the files in the urls because it was user initiated, thus getting around the system integrity permissions. If I pass the file to an external process, I'm betting it won't work...but I'll test it.
I really want to do this natively for another project so there's an added benefit to implementing it this way.

I would much appreciate, if you could support both methods and make it user selectable in the menu item.

I don't plan to do that at this time but the code is open source so you are welcome to add this. Again, I need to test this first to see which solution works best but the goal of this project is to do one specific thing, not be a general purpose exiftool wrapper.

Understood... all valid reasoning.

Initial Service is working. This one performs the reverse geocoding and copies result to clipboard. Will work on the "write to XMP" service next.

Initial Service is working. This one performs the reverse geocoding and copies result to clipboard. Will work on the "write to XMP" service next.

Small suggestion...
Could you make the sub-menu item text (Get reverse geocoding with Locationator) shorter.... e.g. maybe something short like Reverse Geocode ?
... or maybe follow another method, where there is a main sub-menu called 'Locationator' and then sub-sub-menus under it.
Locationator>

Copy geocoding
Reverse Geocode
[future]

Can't do nested menus in Services

This is what I've got for Services at the moment.

As I suspected, exiftool cannot be called by the app due to sandboxing.  I will figure out how to do this using the native APIs.

It might be possible to package exiftool inside the App bundle but exiftool for macOS isn't a standalone binary and requires several libraries to also be installed.  Not worth my time to figure that out for "it might work" solution which will likely still run into sandbox issues.

The "write XMP" Service appears to be working. It does not work with RAW though. exiftool can write the metadata fields to the RAW file but the CoreGraphics / ImageIO routines don't appear to be working for writing to RAW. They read the metadata from the RAW files just fine.

Services implemented in v0.1.0.  The write XMP service doesn't work with RAW files but that's a problem for someone else to solve.


Throw exception if _convert_for_display called on non Number

In trying to come up for some reason behind issue #6365 (which only
happens on some platforms) the best guess is that some components are
managing to get a string value all the way up to the Polymer UI for
temperature, which then an increment of +0.5 is treating as a string
concat operation instead of addition. So 20 + 0.5 becomes 200.5 hits
the max thermostat value.
This will throw an exception if the climate temp value isn't a
number. That's going to turn a soft fail into a hard fail on
potentially a number of platforms. Mysensors is one of the platforms
that was reported as having the issue. So put some explicit float
casts where that might be coming from as well.
Fixes #6365

I'm thinking it's better to do the type conversion in the update and set methods. For the update method it should be done conditionally. One reason for the move is that some of these properties will/can return None, so you'd have to check for that before converting to float.

@MartinHjelmare we already convert to float in the service schema's, so a set method is never called with a string. However, we don't validate each property that gets back. It's important that we are consistent in the typing so raising if it's not a number seems like the right way to go.
Yes, this will break some platforms, but eventually we will fix them all and everyone will have an improved experience

@balloob I was referring to the mysensors module only. The changed properties are designed to return None in some cases, so this has now introduced a bug.

Oh snap, I'm sorry @MartinHjelmare, I didn't realize that.

It isn't as bad as I first thought, but will still lead to error in some cases. It's only current_temperature that will blow up if the user hasn't defined a V_TEMP. We could just add a check before the conversion in that property.

Created #8193 to address the bug.


Nadro/3079/screenshot improvements

closes https://github.com/KittyCAD/modeling-app/issues/3079
Documentation https://www.electronjs.org/docs/latest/api/desktop-capturer
Issue
html2canvas or html2canvas-pro work but are fragile and not robust enough. Overtime with libraries being updated or new components we create the "screenshot" feature with this library will break.
Proof of breaking
html2canvas-pro is broken in  electron, chrome browser, and edge browser while using linux OS.

Solution
If users are using the browser based application the screenshot tool will default to taking a snapshot from the video element.
If users are using the desktop application in electron it will default to the native desktop solution in electron to take a screenshot. If the electron desktop screenshot fails, it will fall back to video element stream.
I added an extra comment to the github issue to indicate that browser screenshots are limited.
I removed the html2canvas-pro package.
Example from browser

Example from electron

I read that MacOS can prevent screenshots by default. I can get access to see if it is enabled or not.
I cannot ask for one time permissions for a screenshot though since the ask API does not allow screen.
My new plan:
Table based on the documentation of operating systems when asking for media

System
Native Screenshot
Video Canvas Stream screenshot

Browsers
:x:
:heavy_check_mark:

Linux
:heavy_check_mark:
fallback if native fails

Windows
:heavy_check_mark:
fallback if native fails

MacOS
needs to enable in system preferences
default or fallback if native fails

We can always come back to how to improve the MacOS workflow. If a user wants to enable it manually they will get native screenshot support otherwise it will default to the video canvas stream.
I've updated the github bug ticket to have an HTML comment that says
<!--
  If you are capturing from a browser there is limited support for screenshots, only captures the modeling scene.
  If you are on MacOS native screenshots may be disabled by default. To enable native screenshots add Zoo Modeling App to System Settings -> Screen & SystemAudio Recording for native screenshots.
-->


Vision semi-GAPIC

This commit removes the entire Vision manual layer and replaces it with a thin wrapper around the GAPIC.
It makes several other changes:

The GAPIC and Protos are packaged alongside the library, rather than being separate packages on PyPI. (I plan to move toward this generally, but not all at once.)
The docs are updated.
The system tests are dramatically thinned down; more should be ported from the old tests before this goes anywhere.

Tagging this "do not merge" for now while other questions about this are worked out.

@dhermes @jonparrott I think we might actually be ready to close on this. Please give it another review.
Notes:

There are documentation modifications in other directories, generally made necessary by the addition of sphinx.ext.napoleon. In particular, every place where @tseaver wrote See: {url}, it now reads See {url}. (Sphinx only cared about the ones with a line break, but I edited them all for consistency.)
Similarly, every ReST directive that said :type: type has changed to :type: :class:\type``.

Luke, is there any way we can move the doc changes to another PR?
On Fri, Jun 2, 2017, 8:36 AM Luke Sneeringer<EMAIL_ADDRESS>wrote:

@dhermes https://github.com/dhermes @jonparrott
https://github.com/jonparrott I think we might actually be ready to
close on this. Please give it another review.
Notes:

There are documentation modifications in other directories,
generally made necessary by the addition of sphinx.ext.napoleon. In
particular, every place where @tseaver https://github.com/tseaver
wrote See: {url}, it now reads See {url}. (Sphinx only cared about the
ones with a line break, but I edited them all for consistency.)
Similarly, every ReST directive that said :type: type has changed to :type:
:class:\type``.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-python/pull/3373#issuecomment-305823593,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUcxtdC2QybQ1AcTrBGiPuPdS2dd0wks5sACv-gaJpZM4NRNA3
.

Not if I want either build to pass. :-(

That's fine. Small diffs are better. If you're really concerned you can
target a non-master branch.
On Fri, Jun 2, 2017, 9:23 AM Luke Sneeringer<EMAIL_ADDRESS>wrote:

Not if I want either build to pass. :-(
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/google-cloud-python/pull/3373#issuecomment-305837575,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPUc_e_gui5GZhJK0HivED00q3aFAsKks5sADcOgaJpZM4NRNA3
.

I will make a side PR.

@jonparrott gave the go-ahead for merge in Google Hangouts.

Note: We are not releasing this yet; we want to write new samples first.


Starting m2m data exchange for one equipment with asset id only

What is missing?
The initial situration is, that someone just has a physical device with an asset id (, which should be many devices in the future due to IEC61406-01 or -02). The use case could be to make the Digital Product Passport as AAS accessible or other data like available e. g. in incoming goods inspection.
Here the user has a software which can download the AAS, but now he has only the asset Id and no discovery API. One approach could be to get a (a few) general repositories. This could be solved by the approach suggested in https://github.com/admin-shell-io/aas-specs-api/issues/330).
We assume that the data is made available public or a security negotiation is included optionally.
How should it be fixed?
But additionally the way could just lead to the repositories which are relevant for the asset. That is, the response is a list of ass and submodel endpoint for the asset id, e. g. a list of shell descriptors.
In general, and also for the DPP, it might be necessary to open a human-readable web page. Here, we suggest to introduce a new mime-type for the GET when calling the asset id: application/shell-descriptors
[
{
    "assetKind": "Instance",
    "endpoints": [
        {
            "interface": "AAS-1.0",
            "protocolInformation": {
                "href": "https://aasx-server-p2.dev.pxcio.net/shells/aHR0cHM6Ly9pNGQuZGUvVC8xMDQ0MDI0L2Fhcw"
            }
        }
    ],
    "globalAssetId": "https://i4d.de/T/1044024",
    "idShort": "aasT_1044024_FL_SWITCH_2208_PN",
    "id": "https://i4d.de/T/1044024/aas",
    "specificAssetIds": [
        {
            "name": "assetKind",
            "value": "Type",
            "externalSubjectId": {
                "type": "ExternalReference",
                "keys": [
                    {
                        "type": "GlobalReference",
                        "value": "assetKind"
                    }
                ]
            }
        }
    ],
    "submodelDescriptors": [
        {
            "endpoints": [
                {
                    "interface": "SUBMODEL-1.0",
                    "protocolInformation": {
                        "href": "https://aasx-server-p2.dev.pxcio.net/shells/aHR0cHM6Ly9pNGQuZGUvVC8xMDQ0MDI0L2Fhcw/submodels/aHR0cHM6Ly9pNGQuZGUvVC8xMDQ0MDI0L3N1Ym1vZGVsL05hbWVwbGF0ZQ"
                    }
                }
            ],
            "idShort": "Nameplate",
            "id": "https://i4d.de/T/1044024/submodel/Nameplate",
            "semanticId": {
                "type": "ExternalReference",
                "keys": [
                    {
                        "type": "GlobalReference",
                        "value": "https://admin-shell.io/zvei/nameplate/2/0/Nameplate"
                    }
                ]
            }
        },

}
]

[ ] I have signed the required Developer Certificate of Origin (DCO) already.

The json example looks like a normal json payload: what exactly is different? Where exactly do you want the new "mime-type" to be added?
If I understand correctly you look for the following interaction:
https://admin-shell-io.github.io/aas-specs-antora/IDTA-01002/v3.1/http-rest-api/interactions.html#fig:seq-sm-endpoints

Hello Birgit,
The mime-type has to be added to the HTTP GET of the Asset Id, because this is the only information, the client application has in this particular moment.
So the endpoint of the discovery service from https://admin-shell-io.github.io/aas-specs-antora/IDTA-01002/v3.1/http-rest-api/interactions.html#fig:seq-sm-endpoints is unknown to the client application and the interaction does not work. So, I suggest to weaken the following assumption, in some scenarios (asset id is a URL):

Endpoints for infrastructure interfaces like AAS registries are known at design time of the client application or configured manually before start-up.

As the server knows all repositories, he can do the above mentioned interaction also on serverside and return the minimum setup of the dataspace for this particular asset.
Regards
Björn

I have the feeling that a few things are mixed together in the proposal:

Allowing the declaration of different mime-types in the endpoint declaration
Knowledge and access (in particular authentication patterns) to infrastructure services, e.g., Discovery Service and AAS Registry
Principle knowledge and access (in particular authentication patterns) to AAS endpoints and/or the AAS Repositories.

If we can assume that all endpoints are publicly available, I think the following situation occurs:

The client access one (or many) Discovery Services -> result: List of Shell Identifiers
The client access one (or many) Registries -> List of ShellDescriptors
The client wants to open a website for a specific AAS from that list -> How to find the right endpoint that does not provide only application/json but, e.g., text/html?

We could expose the non-AAS endpoints maybe like this (note the second item in endpoints):
[
{
    "assetKind": "Instance",
    "endpoints": [
        {
            "interface": "AAS-3.0",
            "protocolInformation": {
                "href": "https://aasx-server-p2.dev.pxcio.net/shells/aHR0cHM6Ly9pNGQuZGUvVC8xMDQ0MDI0L2Fhcw"
            }
        },
        {
            "interface": "text/html",
            "protocolInformation": {
                "href": "https://aasx-server-p2.dev.pxcio.net/shell-website/aHR0cHM6Ly9pNGQuZGUvVC8xMDQ0MDI0L2Fhcw"
            }
        }
    ],
    "globalAssetId": "https://i4d.de/T/1044024",
    "idShort": "aasT_1044024_FL_SWITCH_2208_PN",
    "id": "https://i4d.de/T/1044024/aas",
    "specificAssetIds": [
        {
            "name": "assetKind",
            "value": "Type",
            "externalSubjectId": {
                "type": "ExternalReference",
                "keys": [
                    {
                        "type": "GlobalReference",
                        "value": "assetKind"
                    }
                ]
            }
        }
    ],
    "submodelDescriptors": [...]
}
]

What I don't understand right now from the problem description is why the cited assumption needs to be changed. It might be possible to do it also at the runtime of the client application but this would overload the chapter, therefore we put it out of scope...

Schema-wise this would be ok, as Endpoint/interface is only defined as a string. However, I do remember that there were some limitations on what can be used as a value, need to look again.'

yes, see table https://admin-shell-io.github.io/aas-specs-antora/IDTA-01002/v3.1/specification/interfaces-payload.html
only SUBMODEL-3.0 etc. are allowed so far
The interface attribute is not for mime types.


The VB Code is not Showed

The vb code in this page is not showed. Only C# code is showed. These code should be below:

Replace the Sheet1_Startup event handler with the following code. When Sheet1 is opened, this code adds a second line of text to the worksheet.

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

ID: a12b2f91-753b-4771-85c2-75808137088a
Version Independent ID: a047d08b-9306-8f1e-2d69-a932417f97b2
Content: Create your first document-level customization for Excel
Content Source: docs/vsto/walkthrough-creating-your-first-document-level-customization-for-excel.md
Product: visual-studio-windows
Technology: office-development
GitHub Login: @John-Hart
Microsoft Alias: johnhart

@Zhang-Yuqi -- thank you for your feedback.  Please consider these resources:

Visual Studio on Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

@John-Hart -- please look into this section.

@Zhang-Yuqi There is a control at the top of the page where you can switch between C# and VB (Visual Basic), as follows:

If you select VB, you'll see the Visual Basic code.
@John-Hart @WilliamAntonRohm FYI :)

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.


Rich notifications (iOS 10)

In iOS 10, remote notification payloads can be mutated on the client before presentation to the user, allowing for rich content to be displayed (images, video).
The key to this is a mutable-content boolean value in the payload that must be set to 1.
Further information in the WWDC presentation (page 16): http://devstreaming.apple.com/videos/wwdc/2016/708tbh8wnspsg01hxwx/708/708_advanced_notifications.pdf

I'm not sure why loadCredentials is failing, as I don't believe I've touched it in this PR.

Coverage decreased (-0.1%) to 61.759% when pulling 7704ef0e02155d92f422f0f398af80228444d2c5 on alexanderedge:master into eccd565975adbe5c40208f7763c9cf897f1fef73 on argon:master.

https://github.com/argon/node-apn/pull/413/files#diff-bb4752dc574e5f03dd2d86526e4ec90eL271 there's the problem

Coverage decreased (-0.1%) to 61.759% when pulling 32e45c8cf10bdcd941dd10ead38f5ec56abaf95c on alexanderedge:master into eccd565975adbe5c40208f7763c9cf897f1fef73 on argon:master.


Add animations with react-flip-move

I use react-flip-move as a drop in replacement for my tables to create easy animations. It's easy enough to modify existing code that already builds tables in react by replacing the  element with

  <FlipMove typeName='tbody'>
    {this.renderRows()}
  </FlipMove>

I believe the easiest to add this functionality would be the ability to pass in a wrapper object that react-bootstrap-table would use to replace the tbody element that react-bootstrap-table creates, but the API doesn't currently allow this.
Does this approach seem correct?

@jehartzog, thanks for your feedback, I'm not very familiar with animation, but I maybe a wrapper object is a well design. Anyway, I prefer to fixing some bugs and forward to v3.0.0 as production release, then I'll try to support this feature, so PR is welcome.

I have started the PR and have basic animations working. The insert animation actually looks great, the remove one animations but not correctly. I also created a demo page to try them out.
I'll need to cleanup the way I pass props so you can actually configure the animations, but I'd like to do the proof of concept to make sure FlipMove will able to easily add great animations with minimal API changes.
Let me know if you are interested in getting this PR more polished or see a better way to proceed!

@jehartzog, awesome, how about show me your demo page or your demo code, Because I'm pretty care the usage of this component(or your implement function), it is supposed to be very easy and simple to understand or configure for user, In addition, I always doing some clean or tuning for each PR, so just focus on the implementation for functionality and let me know if you got any trouble 👍
Thanks your contribution and you did a great job

I tried to get a version of this up and running on my own remote server but I mainly work with Meteor so am slow with webpack, sorry! If you find yourself wanting to play with you on your own machine you can grab the branch.
So far this is all the code I modified (besides passing in the prop) in TableBody.js started at line 167:
    // TODO Refactor this to allow passing in more animations controls to FlipMove
    let tableBody = null;
    if (this.props.animate) {
      console.log('here');
      tableBody = <FlipMove typeName='tbody' ref='tbody'>{ tableRows }</FlipMove>;
    } else {
      tableBody =
{ tableRows }
;
    }

    return (

          { React.cloneElement(tableHeader, { ref: 'header' }) }
          { tableBody }

    );

As you can see, it's as easy as swapping out the
 component with <FlipMove>, passing some props, and then fixing all the issues that pop up :D.
Here is an example, the add row actually works well, but the delete row has some css issues.

I'm still trying to figure out if this is a bug in FlipMove or just how it works with react-bootstrap-table.

That's cool, thank you :)


[4.2] SILGen: When transforming to AnyHashable materialize the value in a temporary

emitAnyHashable expects an address value.
Cherry-pick from #17928.
Explanation: When transforming values to AnyHashable we sometimes (e.g
tuples of strings) omit converting the value to an address type by
storing it to the stack leading to a compiler crash.
The fix is to store the value to the stack if it is not already
available as an in memory value.
Scope: Affects only conversions to AnyHashable. The code seems to stem
from 2016.
Risk: Low. Code is added that checks that the value expected by
emitAnyHashable is already in memory if it is not we store it to memory.
Testing: A Swift CI test was added.
rdar://40583597

@swift-ci Please test

@jckarter Can you review this?

@swift-ci Please nominate


Problems compiling MMseqs2

Hi
I am trying to compile the MPI version of MMseqs2 and I am getting the following error
Expected Behavior
Should compile without errors
Current Behavior
Fails with:
src/CMakeFiles/mmseqs-framework.dir/build.make:1598: recipe for target 'src/CMakeFiles/mmseqs-framework.dir/linclust/kmerindexdb.cpp.o' failed
make[2]: *** [src/CMakeFiles/mmseqs-framework.dir/linclust/kmerindexdb.cpp.o] Error 1
CMakeFiles/Makefile2:459: recipe for target 'src/CMakeFiles/mmseqs-framework.dir/all' failed
make[1]: *** [src/CMakeFiles/mmseqs-framework.dir/all] Error 2
Makefile:127: recipe for target 'all' failed
make: *** [all] Error 2

Steps to Reproduce (for bugs)
cmake -DHAVE_MPI=1 -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=. ..
make

MMseqs Output (for bugs)
Full error log here
Context
Providing context helps us come up with a solution and improve our documentation for the future.
Your Environment
Include as many relevant details about the environment you experienced the bug in.

Git commit used (The string after "MMseqs Version:" when you execute MMseqs without any parameters): fe2b746e63519424f565b4fbeb4ca5dba905adc7
Which MMseqs version was used (Statically-compiled, self-compiled, Homebrew, etc.): Self-compiled
For self-compiled and Homebrew: Compiler and Cmake versions used and their invocation:
GCC: gcc version 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.10)
OPENMPI: (OpenRTE) 1.10.2
CMAKE: 3.5.1
Server specifications (especially CPU support for AVX2/SSE and amount of system memory): Intel Xeon Processor (Skylake, IBRS) with 256GB ()
Operating system and version: Ubuntu 5.4.0-6ubuntu1~16.04.10

Looks good Milot. Thank you!


Which is loading event

Issue Type (delete the irrelevant ones)

[x] Bug

Specs
What version are you using?
2.17.3
What browser?
Chrome
Expected Behavior
When reset datatable, there should be a loader
Actual Behavior
There is no loader

Still exists

@SevymolSebastian, could you explain what "reset" here means? Not sure what exactly you're asking for.


Consider tie handling in Q(B)

Currently we assume the treatment rules $\hat \tau(\cdot)$ are continuous. If ties are present, we currently only reflect them in $\pi_B$ through an optional tie.breaker argument. With many treatment arms, ties can arise between both units and arms.  With a single arm, if at $B$  sample $i$ and $j$ are tied, then $i$ enter the policy if sample $i$'s tie.breaker (by default equal to the sample index) is lower.
We could also take ties into account in $Q(B) = E[\langle \pi_B(X_i), \tau(X_i) \rangle]$.

Closing as not very relevant. The intended use-case is to obtain some targeting function $\hat \tau(\cdot)$ and to evaluate it with maq. Most fit $\hat \tau(\cdot)$ functions will deliver continuous-valued predictions. If you are in a setting where you want to evaluate a treatment rule that can have many ties you should probably look at RATE.


Please add Freetube

Freetube is a private youtube client.
Its deb-file is always available at https://freetubeapp.io/#download.
Thank you

Sorry, we won't be accepting any daily/nightly, beta or pre-release software. Please see:

Add guidelines for what packages are acceptable for inclusion