Out of memory error on some URL's

I get the following error from lambda when trying to make multiple screenshots with a device pixel ratio of 2 or more:
[0916/083542.361633:FATAL:memory.cc(22)] Out of memory. size=524288
Sept 16 08:35:53 Lambda_2663
If I take just one screenshot it’s fine but I’m trying to stitch multiple screenshots together to make a full height screenshot of a website, and get this error after multiple screenshots have been taken.
I would appreciate any help or advice on how to debug this. Thank you!

I should add that this error occurs on 3gb lambda.

Hello, I'm experimenting with the same problem.
Just to confirm, can you share the URL you're trying to screenshot?

@cjroebuck How high is "high"? Without more info (URL, code, etc...) there's little I can do to help.
On your initial comment you shared this:

[0916/083542.361633:FATAL:memory.cc(22)] Out of memory. size=524288

524288 bytes is exactly 512MiB - are you sure your Lambda function really has 3GB of RAM allocated? Another thing to be aware is that the theoretical maximum resolution of a screenshot that is able to fit in 3GB of memory is roughly 28377x28377 pixels (excluding overheads and actual encoding):
3 GiB = sqrt(3221225472 / 4) = 28377.9
512 MiB = sqrt(536870912 / 4) = 11585.2

Please share more details.

Sure, I will get back with more details ASAP.
On Fri, 20 Sep 2019 at 01:03, Alix Axel<EMAIL_ADDRESS>wrote:

@cjroebuck https://github.com/cjroebuck How high is "high"? Without
more info (URL, code, etc...) there's little I can do to help.
On your initial comment you shared this:
[0916/083542.361633:FATAL:memory.cc(22)] Out of memory. size=524288
524288 bytes is exactly 512MiB - are you sure your Lambda function really
has 3GB of RAM allocated? Another thing to be aware is that the theoretical
maximum resolution of a screenshot that is able to fit in 3GB of memory is
roughly 28377x28377 pixels (excluding overheads and actual encoding):
3 GiB = sqrt(3221225472 / 4) = 28377.9
512 MiB = sqrt(536870912 / 4) = 11585.2
Please share more details.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alixaxel/chrome-aws-lambda/issues/61?email_source=notifications&email_token=AAAC7MCTT3QE24BAY7LMW7LQKPHXDA5CNFSM4IW7MCLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7ESILA#issuecomment-533275692,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAC7MGXMFLGCS57Q6G4NZLQKPHXDANCNFSM4IW7MCLA
.

As an update, it seems that the issue was caused when running with the flag: —force-device-scale-factor=2.
Removing this flag resolves the issue.

So the out of memory issue is back and I can re-create it on a certain URL 100% of the time.
The url is http://wallpapersafari.com/w/UrvL4y/ and the reason it triggers this issue is most likely because the url loads up a huge image.
Running this same URL on a normal puppeteer in a VM on digital ocean, there are no such issues. I suspect the problem is to do with the --single-process flag again.
It looks like some kind of shared memory resource is being used up because running on a 3gb lambda the max memory usage is reporting as 978MB:
    REPORT RequestId: 17a19e62-329f-403c-88ab-de55029ad31e	Duration: 11138.35 ms	Billed Duration: 11200 ms	Memory Size: 3008 MB	Max Memory Used: 978 MB	Init Duration: 1149.85 ms

Here is the code to re-create it:
  const chromium = require("chrome-aws-lambda");

  let args = [
    ...chromium.args,
    // below flags are some things i've tried to see if I can figure anything out before coming back here!
    // "--single-process",
    // "--disable-dev-shm-usage",
    // "--disable-webgl",
    // "--media-cache-size=5242880000",
    // "--disk-cache-size=5242880000",
    // "--unlimited-storage",
    // "--no-sandbox",
    // "--use-gl=swiftshader",
    // `--js-flags="--runtime_call_stats --max-old-space-size=1024
    // --abort-on-uncaught-exception --trace_gc_object_stats
    // --track_gc_object_stats"`,
  ];

  let browserOptions = {
    args,
    executablePath: await chromium.executablePath,
    headless: true,
    dumpio: true,
  };

  console.log("run with options", browserOptions);

  const browser = await chromium.puppeteer.launch(browserOptions);

  const context = await browser.defaultBrowserContext();

  page = await context.newPage();

  await page.goto("http://wallpapersafari.com/w/UrvL4y/", {
    waitUntil: "load",
  });

  console.log("taking screenshot..");

  let buffer = await page.screenshot();
  console.log("took screenshot ok");

Above code fails with log:
Oct 01 22:03:16 Lambda_266316613429_us-east-1     chrome-lambda-test [1001/<PHONE_NUMBER>04:ERROR:validation_errors.cc(76)]     Invalid message:     VALIDATION_ERROR_UNEXPECTED_NULL_POINTER (null     field 1)
Oct 01 22:03:16 Lambda_266316613429_us-east-1     chrome-lambda-test [1001/<PHONE_NUMBER>04:WARNING:network_service.cc(193)]     Mojo error in NetworkService:Validation failed     for DiscardableSharedMemoryManager     ResponseValidator     [VALIDATION_ERROR_UNEXPECTED_NULL_POINTER (null     field 1)]
Oct 01 22:03:16 Lambda_266316613429_us-east-1     chrome-lambda-test [1001/<PHONE_NUMBER>96:FATAL:memory.cc(22)] Out of     memory. size=576000000
Oct 01 22:03:16 Lambda_266316613429_us-east-1     chrome-lambda-test #0 0x55877ffccc89 <unknown>
...
Oct 01 22:03:16 Lambda_266316613429_us-east-1     chrome-lambda-test #17 0x7f871d6d9f1d __clone
Oct 01 22:03:16 Lambda_266316613429_us-east-1     chrome-lambda-test -
Oct 01 22:03:16 Lambda_266316613429_us-east-1     chrome-lambda-test     Received signal 6
Oct 01 22:03:16 Lambda_266316613429_us-east-1     chrome-lambda-test #0 0x55877ffccc89 <unknown>
Oct 01 22:03:16 Lambda_266316613429_us-east-1     chrome-lambda-test #1 0x55877ff36e83 <unknown>
....
Oct 01 22:03:16 Lambda_266316613429_us-east-1     chrome-lambda-test [end of stack trace]
Oct 01 22:03:16 Lambda_266316613429_us-east-1     chrome-lambda-test Calling _exit(1). Core file     will not be generated.
Oct 01 22:03:16 Lambda_266316613429_us-east-1     chrome-lambda-test 2019-10-01T22:03:11.486Z	    17a19e62-329f-403c-88ab-de55029ad31e	got error     Error: Navigation failed because browser has     disconnected!

Instead of write into a buffer, pass path to the puppeteer to save it into a file.
That worked fine to me

I should have said that this crashes during navigation.
It does not get as far as the page.screenshot line so I don’t think it has anything to do with the actual screenshot.

@Kikobeats I notice that the URL fails on microlink - just sent you a report via email!

The error under the hood is Navigation failed because browser has disconnected! 🤔

I think that is the top-level error from puppeteer. The 'under the hood' error is due to the chromium process crashing due to being out of memory, hence the browser has 'disconnected'!

@cjroebuck That does indeed seem to be the case, if you skip that huge image it doesn't crash:
const chromium = require('chrome-aws-lambda');

exports.handler = async (event, context) => {
  let result = null;
  let browser = null;

  try {
    browser = await chromium.puppeteer.launch({
      args: chromium.args,
      defaultViewport: chromium.defaultViewport,
      executablePath: await chromium.executablePath,
      headless: chromium.headless,
    });

    let page = await browser.newPage();

    await page.setRequestInterception(true);

    page.on('request', (interceptedRequest) => {
      if (interceptedRequest.url() === 'https://cdn.wallpapersafari.com/72/72/UrvL4y.jpg') {
        interceptedRequest.abort();
      } else {
        interceptedRequest.continue();
      }
    });

    await page.goto("http://wallpapersafari.com/w/UrvL4y/");

    result = await page.title();
  } catch (error) {
    return context.fail(error);
  } finally {
    if (browser !== null) {
      await browser.close();
    }
  }

  return context.succeed(result);
};

Furthermore, only requesting the specific image doesn't seem to crash either:
const chromium = require('chrome-aws-lambda');

exports.handler = async (event, context) => {
  let result = null;
  let browser = null;

  try {
    browser = await chromium.puppeteer.launch({
      args: chromium.args,
      defaultViewport: chromium.defaultViewport,
      executablePath: await chromium.executablePath,
      headless: chromium.headless,
    });

    let page = await browser.newPage();

    await page.goto('https://cdn.wallpapersafari.com/72/72/UrvL4y.jpg');

    result = await page.title();
  } catch (error) {
    return context.fail(error);
  } finally {
    if (browser !== null) {
      await browser.close();
    }
  }

  return context.succeed(result);
};

I'm not sure there's an easy fix for this tho, as it's a inherit limitation of the Lambda specs. :/

This is a good hint tho:
Oct 01 22:03:16 Lambda_266316613429_us-east-1     chrome-lambda-test [1001/<PHONE_NUMBER>96:FATAL:memory.cc(22)] Out of     memory. size=576000000

For a 12000x12000 image, 4 bytes per pixel, it's exactly 576000000 bytes.
I need to look into Chromium code to see if there's any way to overwrite the maximum buffer size.

@alixaxel Thanks for chiming in here. Yep 576mb would be much more than the 512mb /tmp space allowed by the Lambda env - assuming it's writing this to disk somewhere. If it is storing it in RAM, then there should be enough space with 3gb.

@cjroebuck Yeah - unlike GCF, the /tmp space is not memory mapped, so my guess is that there's a hardcoded threshold that the browser will not exceed under certain (unknown so far) conditions.
I did try to remove the --memory-pressure-off flag, but didn't change a thing.
I need more time to look into Chromium source code / flags to see if anything can be done.

It looks like chromium is writing these large temporary files to /tmp directory. I managed to get the same code working with chrome-aws-lambda on a VM and could see the following files were being written when navigating to the URL, this is after 5 separate navigations, hence the 5 s:
$> lsof -s | awk '$5 == "REG"' | sort -n -r -k 7,7 | head -n 50
chromium  24912                   root   79u      REG              252,1 576004096      14608 /tmp/.org.chromium.Chromium.HHssLU (deleted)
chromium  19215                   root   79u      REG              252,1 576004096      14622 /tmp/.org.chromium.Chromium.91YDuK (deleted)
chromium  19011                   root   72u      REG              252,1 576004096       1718 /tmp/.org.chromium.Chromium.yZV48t (deleted)
chromium  14160                   root   76u      REG              252,1 576004096      16467 /tmp/.org.chromium.Chromium.5zCkX3 (deleted)
chromium  10412                   root   77u      REG              252,1 576004096       6897 /tmp/.org.chromium.Chromium.Hflbp1 (deleted)

Notice that they are all 576004096 bytes large.

@alixaxel why the cache size is 32MB?
Could be a aria le value based on machine specs?

What I don’t understand is

why a 14mb image is generating a 576mb cache entry.

Why requesting the image directly i.e. the cdn link does not cause the same out of memory issue.

I’ve tried disabling cache in puppeteer and using low values for the disk-cache-size and media-cache-size flags and still get this error. I think you’re right that I should now open an issue at pptr or chromium.
Thank you for your help on this.

@Kikobeats This package being specific to AWS Lambda, and Lambda having always 512MB of space on /tmp I found out that 32MB is more than enough to satisfy the usual caching needs of a short-lived Lambda and still leave enough room to unpack Chromium and anything else the user might want to.
If it was based on machine specs, we would have to infer the amount of free space in the system, which is not a particularly fast operation. The flag is there, and can be omitted or overridden, but I found it's a good sensible default that provides some speed up in the majority of cases.

@cjroebuck For the first question I have a theory: the 14MB image is actually JPEG-encoded, but to render it the browser has to first "decode" the JPEG into pixels (aka a bitmap) - a bitmap in browserland is usually compressed of four channels (red, green, blue and alpha), each one of those taking a value between 0 and 255, or 8 bits (or 1 byte). In this case, since it's a 12000x12000 pixels image:
(12000 * 12000) * 4 = 576000000 bytes

The extra 4096 (4KB) you are seeing can likely be explained by some header/padding from Chromium.
As for the second question, I'm not sure either... My guess would be that when you request the image indirectly (thru that page), Chromium considers it an asset and tries to cache it (failing, obviously - why it's not honoring the *-cache-size flags is another mystery), but when you are requesting it directly it's no longer consider an asset. That's my best guess at this point.

@cjroebuck If you end up opening an issue with PPTR or Chromium, could you please link it here?

@cjroebuck
I'm moving from AWS Lambda to Kubernetes and looks this URL is now working:
https://api.microlink.io/?url=http%3A%2F%2Fwallpapersafari.com%2Fw%2FUrvL4y%2F&screenshot=
so probably is a thing just related with AWS Lambda or how this library is provisioned 🤔

Has anyone else ran into this? We're also seeing the same issues, but are not in a spot to be able to move from Lambda to container-based infrastructure. Is the current suggestion to still open up a ticket with Puppeteer or Chromium?
On an unrelated note, has anyone been able to reproduce this locally in a Docker container? We tried to reproduce locally by limiting the memory and /tmp disk space, but neither seemed to reproduce the error.

I had a similar issue on AWS EC2 instance. and it was fixed after freeing up some disk space on the instance.

I have the same issue while trying to create PDFs from the lambdas :/

Has anyone tried mounting AWS' EFS on a lambda and configured chromium to use that as its /tmp? I gave it a shot here and it didn't work, but I'm not sure if I configured it 100% correctly.

were getting:

stderr: [0717/163455.681939:FATAL:memory.cc(38)] Out of memory. size=262144

--single-process resolved this issue for me

--max-decoded-image-size-mb can potentially solve this problem with a caveat. Have in mind that the more aggressive you go with this setting, the more likely your function will succeed without Chromium running out of memory.
So something like --max-decoded-image-size-mb=1 could be enough to fix the problem, but here is the caveat. --max-decoded-image-size-mb is not a compression algorithm. With this flag is the image will get dropped completely if it goes above a threshold.
I'm not entirely sure what's determines the threshold. It doesn't seem to be the inflated size since I tested this with an aggressive value of 0.2 and many pages seemed to load the images normally without dropping it. But the larger the actual sizes were, the more likely the image would get dropped.
Leaving this as a potential solution here since I know not everyone is screenshotting or screenshotting without a particular strict need for accuracy.

Has anyone tried mounting AWS' EFS on a lambda and configure chromium to use that as its /tmp? I gave it a shot here and it didn't work, but I'm not sure if I configured it 100% correctly.

As someone who did exactly this and ran it in production for a few months, my advice is don't do it.
Chromium writes to the /tmp folder when loading images on a page and when taking screenshots. The bigger the images/screenshot the more it writes.
What I found is that EFS's throughput wasn't enough to deal with this much I/O operations when trying to run at scale (you're sharing a single EFS with all of your lambdas).
In the end we ended just placing the chromium binary in EFS and letting chromium write to /tmp, this somewhat worked and allows us to run slightly larger images/screenshots as we've freed 130MB of space, but it's expensive (you need to run EFS in provisioned throughput mode) and it's still limited.

Should be fixed with this https://aws.amazon.com/blogs/aws/aws-lambda-now-supports-up-to-10-gb-ephemeral-storage/


coprocessor/dag/expr: add un-pushed builtin UDFs(Log1Arg, Log2Args)

What have you changed?
This PR implements the scalar function Log1Arg and Log2Args
What are the type of the changes?
Improvement
How has this PR been tested?
unit test
Does this PR affect documentation (docs/docs-cn) update?
no
Does this PR affect tidb-ansible update?
no
Refer to a related PR or issue link
Benchmark result if necessary (optional)
Add a few positive/negative examples (optional)

@breeswish PTAL


Create a field in add command (Skill Sets)

Create a new field for adding skills. The field name will be 'Skills'.
Context
From the recruiter's and HR professionals' point of view, there is a need to view the candidates' skill sets. Hence, it is important to have them as a field in the contact list.
Close #44

LGTM


Bump Dependencies #1506601691

Bumps dependencies for product-is. Link : https://github.com/wso2/product-is/actions/runs/1506601691

 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.jenkins-is-staging seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


build: repalce megacheck with staticcheck

Codecov Report

Merging #116 into master will not change coverage.
The diff coverage is n/a.


Replaced using the build SHA for versioning with an explicit version number.

Fixes issue #16 (kind of).


Smurthi/status bug

Description
Please explain the changes you made here.
Release notes
The title of the pull request will be used as the default release notes description. If more detail is needed to communicate to partners the scope of the PR's changes, use this release notes section.
Checklist
General
Read the full guidelines for PRs here

[ ] Added the correct label: < feature | enhancement | bug | under-development | dependencies | infrastructure | ignore-for-release | database >
[ ] Assigned to a specific person, civiform/developers, or a more specific round-robin list
[ ] Added an additional reviewer from outside your organization as FYI (if the primary reviewer is in the same organization as you)
[ ] Created unit and/or browser tests which fail without the change (if possible)
[ ] Performed manual testing (Chrome and Firefox if it includes front-end changes)
[ ] Extended the README / documentation, if necessary

Database evolutions
Read the guidelines here

[ ] Assigned two reviewers
[ ] Guarded against already existing resources using IF NOT EXISTS and IF EXISTS
[ ] Downs created to undo changes in Ups

User visible changes

[ ] Followed steps to internationalize new strings

[ ] Added context strings to new messages
[ ] Didn't use a message in applicant facing code that isn't translated yet (unless behind a flag)

[ ] Wrote browser tests using the validateAccessibility method
[ ] Tested on mobile view. See mobile device mode
[ ] Manually tested at 200% size
[ ] Manually evaluated tab order

New Features

[ ] Add new FeatureFlag env vars to server/conf/helper/feature-flags.conf
[ ] Conditioned new functionality on a FeatureFlag
[ ] Wrote browser tests with the feature flag off and on, etc.

Instructions for manual testing
If instructions are needed for manual testing by reviewers, include them here.
Issue(s) this completes
Fixes #<issue_number>; Fixes #<issue_number>...

I think another issue might be if we end up deleting statuses, or changing their names (can we do that? I forget). Applications would end up with something in latest_status that doesn't correspond to a status in the latest program rev. By ensuring an application can only have a status that is present in the program rev it was applied for, we know this can never happen. We can maybe hack around that problem, but we're kind of creating bandaids on bandaids then. --> Today in a program, we change, ""Pending" status to "In-queue", the pending status is not completely deleted from an application. So with this fix, we are setting a status which is not initially associated with the application, but the current program version, we are not adding to any issue. Applications will still be marked "Pending". Going forwards, all new applications can be marked ""In-queue". With bulk status update, all "Pending" can be easily set to "In-queue"


Added Optional Redirect URI for OAuth Clients that require specific Redirect URIs

Jumped the gun too soon, I will have more changes.

Thanks for the PR 👍
Before looking at this could you please check the parameter name again?
The readme says redirectUri but the code uses redirect_uri. Would prefer redirectUri for the option.
And could you please stash the commits into one?
Thanks 😄

Hey! Fixed the issue and rebased. Not sure why there is a conflict though.

Thanks 😄
Just landed and published as 2.2.0

Awesome! :smile:


how to check all the sup-nodes when I check the sub-node?

I just found Checkall function.
But, I need to check one sub-node then all the sup-nodes in the node also can be checked.
how to achieve it?
Thanks.

This has been answered before: https://github.com/jonmiles/bootstrap-treeview/issues/107


Bug about health bar losses

Please check for duplicates, similar issues or closed issues with the same problem, as well performing simple troubleshooting steps (such as deleting the export folder if it's an issue with compiling, reinstalling Kade Engine) before submitting an issue.
nope
If you are playing a downloaded version of the game, what operating system are you using?
Windows (x86), Windows (x86_64), Linux, or macOS? Specify below.
macOS
What version of the game are you using? Look in the bottom left corner of the main menu.
1.3
Have you identified any steps to reproduce the bug? If so, please describe them below. Use images if possible.
No
Please describe your issue. Provide extensive detail and images if possible.
It makes be lose everytime I lost 5 notes at once, I noticed you gain very little but when you miss a note you get pushed back VERY hard

why are you playing on 1.3, also this was probably fixed by now

Probably because it's the last version that supports Old Input System? (though idfk why, since recent Kade Engine versions added "Ghost Tapping" and you can turn it off which is basically Old Input System)

ghost tapping
Ghost Tapping is a feature that most people use to allow you to keep a rhythm in a game by tapping notes that don't exist, this is just a way that some people keep a metronome in their mental mind.

Old Input was a bad way of describing it, and it has been changed to ghost tapping. Changing ghost tapping on or off in newer versions doesn't change the inputs, it changes how you can input notes. But otherwise nothing else.
To your question, the health drain was intentional. But has been moved down in recent builds.
A more in-depth explanation of how inputs are handled:
Inputs are handled based on the exact timing of your note presses. There exists an entire hit window of 166 milliseconds to hit something.

Or based on each note (complex accuracy, another option that factors in your difference between hitting the notes. For example, you hit a note at 36ms, that would be a sick since the default sick window is 45ms. it would also factor into your accuracy using the Wife3 function described here)
Generally, the idea of the input is to reward higher skill plays to allow people to overachieve and have a fair gameplay loop with the ability to customize it depending on their skill. In the options, you can change the hit windows to make them stricter or easier, depending on your play style or map choice.
I hope this is a good enough explanation for anyone who is lurking on this or anyone who wants to know anything about KE at all. I'm sure OP doesn't care in the slightest lol.
But this is more of a summary of what we do.

"Mental mind" :trollface:

"Mental mind" :trollface:

imagine a mind in a mind :trollface:

OK


[main] Upgrade to latest dependencies

Cron. -knative-prow-robot
/cc knative-sandbox/api-core-wg-leads
/assign knative-sandbox/api-core-wg-leads
Produced via:
./hack/update-deps.sh --upgrade --release v0.23
./hack/update-codegen.sh

Details:
bumping knative.dev/pkg 01dccc5...3a2ae6d:
  > 3a2ae6d resolve cluster domain using env CLUSTER_DOMAIN as a fallback (# 2092)
  > 4302f33 Update knative/community files (# 2091)
  > 728bc4a Update OWNERS_ALIASES to match autogen in community (# 2078)

/lgtm
/approve


This updates traffic stats to use the new github repo for influxdb (github.com/influxdata)

The spec file will now build with the master branch of InfluxDb, because I could not getting working with a specific branch due to the github repo moving.  I suspect that once a new tag is created by influxdb we can make this work again.

Someone should tell them to push tags too...


Unexpected behavior of center_object when zoom not provided

Environment Information
Python 3.12.7 (main, Oct  1 2024, 02:05:46) [Clang 15.0.0 (clang-15<IP_ADDRESS>)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import geemap
>>> geemap.Report()

--------------------------------------------------------------------------------
  Date: Mon Dec 16 12:43:01 2024 -03

                OS : Darwin (macOS 15.2)
            CPU(s) : 11
           Machine : arm64
      Architecture : 64bit
               RAM : 18.0 GiB
       Environment : Python
       File system : apfs

  Python 3.12.7 (main, Oct  1 2024, 02:05:46) [Clang 15.0.0
  (clang-15<IP_ADDRESS>)]

            geemap : 0.35.1
                ee : 1.4.2
        ipyleaflet : 0.19.2
            folium : 0.18.0
        jupyterlab : Module not found
          notebook : Module not found
         ipyevents : 2.0.2
         geopandas : 1.0.1
--------------------------------------------------------------------------------
>>>

I
Description
I was running the following code in an interactive python environment with no issues.
import ee

ee.Authenticate()
ee.Initialize(project="unicef-geospatial")

def get_uruguay_map():
    """Displays a map of Uruguay"""
    country_boundries = ee.FeatureCollection("USDOS/LSIB_SIMPLE/2017")
    uruguay_boundries = country_boundries.filter(ee.Filter.eq("country_na", "Uruguay"))

    country_map = geemap.Map()
    country_map.center_object(uruguay_boundries)
    country_map.add_layer(uruguay_boundries, {}, "Uruguay Boundaries")
    return country_map

map = get_uruguay_map()
display(map)

The map was properly displayed and centered correctly, as can be shown in the photo.

Then, I've changed to saving it as a html file to display it in a browser, so I changed the
last line of code to:
map.save("uruguay_map.html")

and started running directly with python and not in an interactive environment. For some reason, the map stopped being centered correctly, as can be show in the next photo.

When I added a zoom level to the map, it worked as expected, but I need for the zoom level to be dynamic based on the bounds of the country.

I've also tried calculating the boundries and calling the zoom_to_bounds method, but it didn't work.
This issue is related to this issue

This is an ipyleaflet issue rather than a geemap issue. When zoom level is not specified, it uses Map.fit_bounds. If zoom is specify, it updates Map.center. The ipyleaflet Map.fit_bounds does not work when you convert the Map to HTML.
https://github.com/gee-community/geemap/blob/fa956dc1019e25793200a6360333504b4364a1e2/geemap/core.py#L829-L851

The ipyleaflet Map.fit_bounds does not work when you convert the Map to HTML

Interesting. Is there any workaround to center an object in HTML without specifying a zoom level? Or is there a way to calculate which zoom level will fit a given object?

Yep, perfect.
Thanks!


Story - Various non-functional, organizational, and CI improvements

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@d8fa3f4). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #12   +/-   ##
=========================================
  Coverage          ?   79.57%
=========================================
  Files             ?       30
  Lines             ?     1141
  Branches          ?      186
=========================================
  Hits              ?      908
  Misses            ?      186
  Partials          ?       47

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


UI - List functions and test cases

When a test case is selected, the operation and args should be altered to the corresponding test case operation and args

Merged


Update NPMJS package references

Q
A

content fix?
yes

New article?
no

Related issues?
n/a

What's in this Pull Request?
updated package references for consistency so:

first reference is linked to the NPM package page on https://www.npmjs.org
all references are bold & have the correct @microsoft/ scope prefix where appropriate

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/spfx/extensions/guidance/using-custom-dialogs-with-spfx.md
:white_check_mark:Succeeded

docs/spfx/office-ui-fabric-integration.md
:white_check_mark:Succeeded

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

thx AC, appreciated.


15.1 Reverse Integer(cpp): have not check integer overflow....

Did you notice that the reversed integer might overflow? Assume the input is a 32-bit integer, then the reverse of<PHONE_NUMBER> overflows. How should you handle such cases?
yeah?

Thanks for pointing this out, I'll check it later.

 signed long long int r=0;

 for(;x;x/=10)
        r = r*10+x%10;

 if(abs(r)>(pow(2,31)-1)) return 0;
 else return (int) r;

@billlipeng
Thanks!
#include <match>

class Solution {
public:
    int reverse(int x) {
        signed long long int result = 0;
        for (; x; x/=10)
        {
            result = result * 10 + x % 10;
        }
        if (abs(result) > (pow(2, 31)-1)) {
            return 0;
        } else {
            return result;
        }
    }
};

+1. The solution is wrong.

谢谢，我周末抽时间检查一下

This issue was solved. Plz check my comment on Feb 26, update the code and close it.  Thanks a lot!

同发现问题。我觉得这个解很好。
https://leetcode.com/discuss/51595/short-accepted-c-solution-using-int-without-long-or-string


Added script to easily launch the app

Just a small script to easily launch your app in case you want to use it.

Sorry, to disturb you, and it's so dumb of me, I can't make out where the file is :(, please help!
and thank you so much in advance for helping others (including me) who are hardcore non-technical.


Add support for component aws-s3

http://camel.apache.org/aws-s3.html

Found 10 matching posts for subject:aws-s3 in Camel - Users within 700 days


Guest avatar is missing from the show card on the guest page

Loving the new site!
I'm not sure if this is intentional but the guest avatar is missing from the show cards which appear on the guest page.
I thought I'd let you know.
Example
In the episode with David East, his avatar is missing:
URL: https://syntax.fm/guest/david-east

I wasn't sure from the query below whether it retrieves the guest or not:
https://github.com/syntaxfm/website/blob/main/src/routes/(site)/guest/[name_slug]/%2Bpage.server.ts
However, from the schema it looks like a show should have all of the guests in it.
https://github.com/syntaxfm/website/blob/main/prisma/schema.prisma#L117C18-L117C18
Happy to PR if you point me in the right direction but it also might be intentional so didn't want to assume.
Thanks,
Marc

Waiting for #1302 before tackling

fixed!

Nice! Thanks, Wes! 🎉


webpack can't find module vscode, 'vscode-debugadapter', and 'vscode-debugprotocol'

The issue is not so much with vscode, but more with its documentation, more specifically here: https://code.visualstudio.com/api/working-with-extensions/bundling-extension
The example given in the following link fails when we add other modules:
https://github.com/Microsoft/vscode-extension-samples/tree/master/webpack-sample
Here's a patch that can be applied to the webpack-sample to reproduce the issue:
https://pastebin.com/peqvJ1Wt
I also asked on stackoverflow:
https://stackoverflow.com/questions/60024640/
Any ideas on how to fix this?
Thanks

Maybe debugger people like @isidorn and @roblourens know? This seems very web pack specific and you should consult their resources

I am not a webpack expert but the node-debug extension is webpacked and uses both of those modules. Maybe you can take some inspiration from that configuration: https://github.com/Microsoft/vscode-node-debug

Thanks for your help. That solution is much more complex than the solution proposed in this sample. In the end it seems that commenting out the line
//  "module": "es6" // override tsconfig.json so that TypeScript emits ...
does the trick. Apparently it might be an issue with ts-loader. Independently the current sample documentation doesn't work as is described. I guess I'll check if everything still works without that line, and if not I'll reach out to the ts-loader people. But then again, why was that line added?
It does say:
// override tsconfig.json so that TypeScript emits native JavaScript modules.
But even without that line it still seems to generate js.


Instrumentation: Refactor billing logic to improve auditability and log all major steps

Currently, a lot of billing logic is hidden inside a black box.
We don't have enough logs to audit any abnormal behavior in contract billing.
Also, Things like overflowed calculations when encountered are handled implicitly without any logs.
The billing code is crucial and must be properly instrumented.
Benefits:

Easier identification and diagnosis of issues.
Gaining visibility into the application's behavior in production environments.

Additional context
https://github.com/threefoldtech/home/issues/1560

Update:
Almost done


Course content fetch

The user can browse available tasks and other content in a specific course.

The test server should probably have at least one invalid course id for testing.


New release?

I'm trying to build @mastahyeti 's U2F example, but it looks like the latest release pre-dates the example by nearly a year. It might be nice to post a new release at some point so that the U2F example works out-of-the-box.
Also, for future people that are looking to build that example note that you need another compilation flag:

gcc u2f.c -o virtual_u2f -framework IOKit -framework CoreFoundation

See https://github.com/unbit/foohid/pull/9#issuecomment-281926981.


Refactor Int type to Long type in MC stats library

This change is

Current dependencies on/for this PR:

main

PR #1226

PR #1233
PR #1184

PR #1247

PR #1252

PR #1256   👈

PR #1253

This comment was auto-generated by Graphite.


feat!: Change default mappings for CLR type decimal

@jskeet Review only the last commit, all the others are from #12557


Use docker cp instead of mounting host volume

@PhilipGough I'm hoping you can tell me if this changes how it was supposed to work. I suspect, that I would have needed to give write permissions to the volume I was sharing (/tmp/artifact) in order for it to work.

➜  ansible-service-broker git:(master) ✗ make deploy
docker run --rm -v "/home/dzager/go/src/github.com/openshift/ansible-service-broker/build":/tmp/artifact docker.io/philipgough/dlv:centos cp /go/bin/dlv /tmp/artifact
cp: cannot stat '/tmp/artifact/dlv': Permission denied
make: *** [Makefile:80: build-dev] Error 1

Ok, will test this change tomorrow to ensure it works on osx. It looks fine

@djzager change works fine on osx. I've created a pr to your branch https://github.com/djzager/ansible-service-broker/pull/1 which fixes a minor issue that arose when testing this.
Thanks


container is not destroyed after test

container is not destroyed after running tests

Weird...
https://github.com/geowarin/docker-junit-rule/blob/8e5373c15ceed67fe4161cd64e5fbe6c4cd63ef2/src/main/java/com/github/geowarin/junit/DockerRule.java#L78-L87

I'm closing it as I don't know what else the lib can do to destroy the container...
Please reopen if you are still experiencing the issue


Some small updates to start contributing

Added a README that explains what the project is about. This is based on our discussions on Twitter. It also contains some things I ran into and might be good to know to start.
I also added some missing assets to the StereoKitTest/Assets folder that were available in StereoKitCTest/Assets.

This is awesome! Thanks a bunch =D
I was been putting off documentation just because of how fast I expect things to change, this will totally be helpful regardless of changes!


Logic APP "PaloAlto-PAN-OS-BlockIP" fails - Error "Could not establish trust relationship for the SSL/TLS secure channel."

Describe the bug
The Logic App "PaloAlto-PAN-OS-BlockIP" fails in the Step 'List Address Objects' wit the Error - The underlying connection was closed: Could not establish trust relationship for the SSL/TLS secure channel."
My Palo Alto in Azure has an Self Signed Certificate. how can the http request be influenced with regard to the certificate error so that the connector works?
Expected behavior
Connections should be established to the palo firewall. There is no hint in the connector to deal with this problem.
Screenshots

Hi @secuit, thanks for flagging this issue, we will soon get back to you on this. Thanks!

Is there any update? Do you know when you can check this issue? Thanks. Sebastian

Hi @secuit,
As your Palo Alto has a Self-Signed Certificate. You need to configure logic app for Standard logic app workflows and also have to complete a few extra steps for this authentication type. Otherwise, the call fails. Please refer the below screenshots and documentation.
https://learn.microsoft.com/en-us/azure/logic-apps/logic-apps-securing-a-logic-app?tabs=azure-portal#access-for-outbound-calls-to-other-services-and-systems
TLS/SSL certificate authentication -
https://learn.microsoft.com/en-us/azure/connectors/connectors-native-http#tlsssl-certificate-authentication

While deploying the Standard logic app workflows, the following steps must be taken:
Step 1. you need to configure the - Integration Service Environment in same resource group.
Step 2. the checkbox for Associate with integration service environment should be enabled while deploying the logic app.
note: - The ISE will be visible in the dropdown option after step 1 completed.

Kindly let us know whether the above steps worked for you.
Thanks!

Hi @secuit, hope you are doing well. could you please have a look at above comment. Thanks!

Hi @secuit, Gentle Reminder: We are waiting for your response on this issue. If you still need to keep this issue active, please respond to it in the next 2 days. If we don't receive a response by 03-09-2023 date, we will be closing this issue.
Thanks!

Hi @secuit, since we have not received a response in the last 5 days, we are closing your issue #9034 as per our standard operating procedures. If you still need support for this issue, feel free to re-open at any time. Thank you for your co-operation.


Added Swedish language (sv-se)

Here is a stab at adding support for Swedish. Hoping it's correct :)

Please add the locale to master

@adamwdraper Any change you could give this a try?

Would be awesome if this got merged in anytime soon...pull request opened in 2015 😄


CI Test Failures due to Dask

Many of the PRs are stuck due to the test_combine_output test which fails due to a to_netcdf call that internally calls compute on the dask array portion of the dataset.

If I install ensembleperturbation locally using only pip I get the same crash issue. This seems to be an issue related to netcdf/hdf5 library:
Thread 69 "python" received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x7fff6626c640 (LWP 5089)]
0x00007fff554d336e in H5SL_search () from /home/smani/miniconda3/envs/netcdfcrash/lib/python3.10/site-packages/netCDF4/../netCDF4.libs/libhdf5-7b49ac63.so.310.2.0
#0  0x00007fff554d336e in H5SL_search () from /home/smani/miniconda3/envs/netcdfcrash/lib/python3.10/site-packages/netCDF4/../netCDF4.libs/libhdf5-7b49ac63.so.310.2.0
#1  0x00007fff5548657f in H5P_set () from /home/smani/miniconda3/envs/netcdfcrash/lib/python3.10/site-packages/netCDF4/../netCDF4.libs/libhdf5-7b49ac63.so.310.2.0
#2  0x00007fff5531d6cc in H5CX_pop () from /home/smani/miniconda3/envs/netcdfcrash/lib/python3.10/site-packages/netCDF4/../netCDF4.libs/libhdf5-7b49ac63.so.310.2.0
#3  0x00007fff553203a9 in H5Dget_space ()                                                                                from /home/smani/miniconda3/envs/netcdfcrash/lib/python3.10/site-packages/netCDF4/../netCDF4.libs/libhdf5-7b49ac63.so.310.2.0                                       #4  0x00007fff557962de in nc4_find_dim_len ()
   from /home/smani/miniconda3/envs/netcdfcrash/lib/python3.10/site-packages/netCDF4/../netCDF4.libs/libnetcdf-6ebc343d.so.19                                          #5  0x00007fff557956d8 in HDF5_inq_dim () from /home/smani/miniconda3/envs/netcdfcrash/lib/python3.10/site-packages/netCDF4/../netCDF4.libs/libnetcdf-6ebc343d.so.19
#6  0x00007fff5575fb5a in nc_inq_dimlen () from /home/smani/miniconda3/envs/netcdfcrash/lib/python3.10/site-packages/netCDF4/../netCDF4.libs/libnetcdf-6ebc343d.so.19
#7  0x00007fff55a6a258 in __pyx_pf_7netCDF4_8_netCDF4_9Dimension_8__len__ (__pyx_v_self=0x7fff55b3efc0) at src/netCDF4/_netCDF4.c:46964
#8  __pyx_pw_7netCDF4_8_netCDF4_9Dimension_9__len__ (__pyx_v_self=<netCDF4._netCDF4.Dimension at remote 0x7fff55b3efc0>) at src/netCDF4/_netCDF4.c:46919
#9  0x00007fff55a6bfff in __pyx_pf_7netCDF4_8_netCDF4_8Variable_5shape___get__ (__pyx_v_self=0x7fff55b368e0) at src/netCDF4/_netCDF4.c:57974
#10 __pyx_pw_7netCDF4_8_netCDF4_8Variable_5shape_1__get__ (__pyx_v_self=<netCDF4._netCDF4.Variable at remote 0x7fff55b368e0>) at src/netCDF4/_netCDF4.c:57810
#11 __pyx_getprop_7netCDF4_8_netCDF4_8Variable_shape (o=<netCDF4._netCDF4.Variable at remote 0x7fff55b368e0>, x=<optimized out>) at src/netCDF4/_netCDF4.c:108391
#12 0x0000555555696c82 in _PyObject_GenericGetAttrWithDict (obj=<optimized out>, name=<optimized out>, dict=<optimized out>, suppress=<optimized out>)
   at /usr/local/src/conda/python-3.10.14/Objects/object.c:1254
#13 0x00007fff55a37470 in __pyx_tp_getattro_7netCDF4_8_netCDF4_Variable (o=<netCDF4._netCDF4.Variable at remote 0x7fff55b368e0>, n='shape')
   at src/netCDF4/_netCDF4.c:108355
#14 0x00007fff55a10eac in __Pyx_PyObject_GetAttrStr (attr_name=<optimized out>, obj=<netCDF4._netCDF4.Variable at remote 0x7fff55b368e0>)
   at src/netCDF4/_netCDF4.c:120721
#15 __pyx_pf_7netCDF4_8_netCDF4_8Variable_48__getitem__ (__pyx_v_self=0x7fff55b368e0,                                                                                __pyx_v_elem=(<slice at remote 0x7fff54bea700>, <slice at remote 0x7fff54beacc0>)) at src/netCDF4/_netCDF4.c:64177
#16 0x0000555555698bf3 in cfunction_vectorcall_FASTCALL (func=<built-in method getitem of module object at remote 0x7ffff7b0ee30>, args=0x7fff54b89b10,
    nargsf=<optimized out>, kwnames=<optimized out>) at /usr/local/src/conda/python-3.10.14/Objects/methodobject.c:430
#17 0x0000555555688850 in _PyObject_VectorcallTstate (kwnames=0x0, nargsf=<optimized out>, args=0x7fff54b89b10,                                                            callable=<built-in method getitem of module object at remote 0x7ffff7b0ee30>, tstate=0x5555592fcd70)
   at /usr/local/src/conda/python-3.10.14/Include/cpython/abstract.h:114
#18 PyObject_Vectorcall (kwnames=0x0, nargsf=<optimized out>, args=0x7fff54b89b10, callable=<built-in method getitem of module object at remote 0x7ffff7b0ee30>)
    at /usr/local/src/conda/python-3.10.14/Include/cpython/abstract.h:123                                                                                #19 call_function (kwnames=0x0, oparg=<optimized out>, pp_stack=<synthetic pointer>, trace_info=0x7fff66269790, tstate=<optimized out>)
    at /usr/local/src/conda/python-3.10.14/Python/ceval.c:5893
#20 _PyEval_EvalFrameDefault (tstate=<optimized out>,
    f=Frame 0x7fff54b89970, for file /home/smani/miniconda3/envs/netcdfcrash/lib/python3.10/site-packages/xarray/backends/netCDF4_.py, line 113, in _getitem (self=<NetCDF4ArrayWrapper at remote 0x7fff55b66e00>, key=(<slice at remote 0x7fff54bea700>, <slice at remote 0x7fff54beacc0>), getitem=<built-in method getitem of module object at remote 0x7ffff7b0ee30>, original_array=<netCDF4._netCDF4.Variable at remote 0x7fff55b368e0>), throwflag=<optimized out>)                                               at /usr/local/src/conda/python-3.10.14/Python/ceval.c:4213


scipy.stats.binom_test / binom.sf return incorrect values for large x and n

scipy.stats.binom.sf and scipy.stats.binom_test returns incorrect values for large inputs.  The bounds on the inputs is not documented, and similar functions in other libraries (e.g. R's binom.test) do not have this problem.
Reproducing code example:
>>> np.seterr(all='warn')
{'divide': 'warn', 'over': 'warn', 'under': 'ignore', 'invalid': 'warn'}
>>> scipy.stats.binom_test(1e7, 2e7, 0.5, 'greater')
0.5094024848633918
>>> scipy.stats.binom_test(1e8, 2e8, 0.5, 'greater')
0.6860874231168742
>>> scipy.stats.binom_test(1e9, 2e9, 0.5, 'greater')
0.8854076875623922
>>> scipy.stats.binom_test(1e10, 2e10, 0.5, 'greater')
nan

Error message:
None.
Expected behaviour:
All calls in the repro example should return a value near 0.5.  For example, in R:
> binom.test(1e7, 2e7, 0.5, 'greater')$p.value
[1] 0.5000892
> binom.test(1e8, 2e8, 0.5, 'greater')$p.value
[1] 0.5000282
> binom.test(1e9, 2e9, 0.5, 'greater')$p.value
[1] 0.5000089
> binom.test(1e10, 2e10, 0.5, 'greater')$p.value
[1] 0.5000028

Notes:
Internally, for the "greater" alternative, scipy calls binom.sf(x - 1, n, p), which then calls scipy.special.bdtrc(floor(x), n, p).  This forwards to Cephes' bdtrc, according to scipy's docs.
We can verify that the bug lies within bdtrc:
>>> scipy.special.bdtrc(1e10, 2e10 - 1e10 + 1, 0.5)
nan

Non-broken implementations of the regularized incomplete beta function (such as TensorFlow's tf.math.betainc) will return the expected value (0.5).
This also affects anything else that calls bdtrc, including binom.sf.  There may be a case that bdtrc is Working as Intended (because the documentation is explicit that it's just a wrapper for Cephes' buggy implementation), but I think for binom.sf and binom_test it's clear that there is at least a documentation bug if not an opportunity to make things better.
I think there are 4 viable solutions, in what I think is most-preferred to least-preferred:

File a bug with Cephes and wait for them to fix it, documenting the breakage in the meantime.
Use a normal approximation in binom.sf when "k" and "n" are large (perhaps checking that "p" isn't too biased).  Fortunately the circumstances when bdtrc breaks usually coincide with when this approximation is good. :)
Implement a regularized incomplete beta function within scipy that isn't broken, instead of using Cephes.
Claim this is Working as Intended, but document the limitation.

I'm happy to send Pull Requests given some direction about which of the choices above to go with.
Scipy/Numpy/Python version information:
>>> import sys, scipy, numpy; print(scipy.__version__, numpy.__version__, sys.version_info)
1.5.4 1.19.4 sys.version_info(major=3, minor=6, micro=8, releaselevel='final', serial=0)

thanks for reporting it, this is bug which also impact the distribution stats.binom
stats.binom.cdf(1e9, 2e9, 0.5) # 0.885...

Sounds like a potential job for boostinator!
@mckib2 Could you add this to the demo?

@mdhaber Done.  The boost implementation of binomial survival function appears to do the right thing here.

I think this is the same issue as this one:
https://github.com/scipy/scipy/issues/5503
-Lucas Roberts

On Nov 15, 2020, at 2:32 PM, Christoph Baumgarten<EMAIL_ADDRESS>wrote:

thanks for reporting it, this is bug which also impacts the distribution stats.binom
stats.binom.cdf(1e9, 2e9, 0.5) # 0.885...
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I agree with @rlucas7 that this is essentially a duplicate of gh-5503, but I'll add a note there to look for additional test cases here.


Fails to Launch. Win Error 127.

I've ensured that I have the ^4 Bit version of TS4 and downloaded the latest packs of your program, as well as the C++ redistributable.
I have Steam and Steamam VR open and running when launching, but the game refuses to acknowledge the VR add in. I've included my debug log below.
2023-03-30 14:38:54: module loading initiated
2023-03-30 14:38:54: Reading memmory
2023-03-30 14:38:54: Done reading memmory
2023-03-30 14:38:54: Scale_patch_address1: 0x1410f5318 Scale_patch_address2: 0x1410f5305 code_injection_base_address: 0x14028c50e
2023-03-30 14:38:54: patchlocation: 0(0x1410f5318) = 0f119180000000410f28d20f14d7f30f59f4410f28e3f30f59c5f3410f5ee10f
2023-03-30 14:38:54: patchlocation: 1(0x1410f5305) = 0f1151700f57d20f14c80f28c40f14d10f57c90f119180000000410f28d20f14
2023-03-30 14:38:54: patchlocation: 2(0x14028c50e) = 0f1141300f104a400f1149400f1042500f1141500f104a600f1149600f104270
2023-03-30 14:38:54: loading openvr_api
2023-03-30 14:38:54: loading vrdll: C:\Users*User*\Documents\Electronic Arts\The Sims 4\Mods\convexvr_sims4-vr\s4vrlib.dll
2023-03-30 14:39:32: could not load vrdll: [WinError 127] The specified procedure could not be found
2023-03-30 15:09:58: module loading initiated
2023-03-30 15:09:58: Reading memmory
2023-03-30 15:09:58: Done reading memmory
2023-03-30 15:09:58: Scale_patch_address1: 0x1410f5318 Scale_patch_address2: 0x1410f5305 code_injection_base_address: 0x14028c50e
2023-03-30 15:09:58: patchlocation: 0(0x1410f5318) = 0f119180000000410f28d20f14d7f30f59f4410f28e3f30f59c5f3410f5ee10f
2023-03-30 15:09:58: patchlocation: 1(0x1410f5305) = 0f1151700f57d20f14c80f28c40f14d10f57c90f119180000000410f28d20f14
2023-03-30 15:09:58: patchlocation: 2(0x14028c50e) = 0f1141300f104a400f1149400f1042500f1141500f104a600f1149600f104270
2023-03-30 15:09:58: loading openvr_api
2023-03-30 15:09:58: loading vrdll: C:\Users*User*\Documents\Electronic Arts\The Sims 4\Mods\convexvr_sims4-vr\s4vrlib.dll
2023-03-30 15:09:58: could not load vrdll: [WinError 127] The specified procedure could not be found

The error message is coming from this line:
https://github.com/convexvr/sims4-vr/blob/434ef30f0bca37aa55909593ce3a06df6ac2f48b/src/sims4-vr/src/main.py#L361
Windows is saying that it won’t load the s4vrlib.dll file, windows if not very informative as to why that is.
That it is not loading is either cause the file s4vrlib.dll does not exits or it means that some other dependency that s4vrlib.dll needs can not be found.
s4vrlib.dll  has the following dependencies, make sure that you have them installed.
d3d9.dll
d3dx9_43.dll
openvr_api.dll
KERNEL32.dll
USER32.dll
d3d11.dll
dxgi.dll
VCRUNTIME140_1.dll
VCRUNTIME140.dll
api-ms-win-crt-time-l1-1-0.dll
api-ms-win-crt-heap-l1-1-0.dll
api-ms-win-crt-stdio-l1-1-0.dll
api-ms-win-crt-math-l1-1-0.dll
api-ms-win-crt-runtime-l1-1-0.dll
api-ms-win-crt-string-l1-1-0.dll


[cxx-interop] Do not pass the C++ standard if C++ interop is not enabled

This fixes the Swift PR testing failures.
Motivation:
If C++ interop is not enabled, SwiftPM should not pass -Xcc -std=c++17 flags.
Result:
Only pass these flags if C++ interop is enabled.

@swift-ci please test


RGE no tested

My RGE class has passed all 5 tests.

RGE with control variate has been integrated into ExclusiveKL class. New test functions for different hessian calculation methods have been added to test_objective.py.

Codecov Report
Patch coverage: 98.96% and project coverage change: +0.59 :tada:

Comparison is base (f80031b) 88.42% compared to head (e8e5ea7) 89.01%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #52      +/-   ##
==========================================
+ Coverage   88.42%   89.01%   +0.59%
==========================================
  Files          12       12
  Lines        1520     1611      +91
==========================================
+ Hits         1344     1434      +90
- Misses        176      177       +1

Impacted Files
Coverage Δ

viabel/objectives.py
89.91% <98.76%> (+4.29%)
:arrow_up:

viabel/tests/test_objectives.py
100.00% <100.00%> (ø)

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

validation of hessian method added to constructor of ExclusiveKL

@Kwuin Please also update .travis.yml to test on Python 3.9. Something wonky is going on with Travis CI on 3.7 and 3.9 is the minimum version for bridgestan


Sengoku Musou 2 with Moushouden & Empires HD Version [PCSG00233]

Game summary

Game name: Sengoku Musou 2 with Moushouden & Empires HD Version
Game serial: PCSG00233
Game version: 1.00

Vita3K summary

Version: v0.1
Build number: 1075
Commit hash: https://github.com/vita3k/vita3k/commit/6142543

Test environment summary

Tested by: Zangetsu38
OS: Windows 10
CPU: AMD Ryzen 2700X
GPU: NVIDIA GTX 1080 Ti
RAM: 16 GB DDR4 3600 mhz

Issues

Shader news error

[shader::usse::USSETranslatorVisitor::vpck]: Source not loaded

crash

Log
vita3k.log

Vita3K v0.1.5 2721-150ced29
goes to play the intro video
graphics are messed up
then it freezes up with music playing
vita3k.log


feat(workflow)/e2e

Codecov Report

Merging #111 (acc0c27) into main (498bea7) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #111   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           15        15
  Lines          198       198
  Branches        45        45
=========================================
  Hits           198       198

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


package-verifier ran automated testing even though it wasn't fully provisioned for testing yet

REF: https://groups.google.com/forum/#!topic/chocolatey/LJCDP_gThpo;noredirect=true
PROBLEM: package-verifier attempted to test solarwinds-take-control-teamviewer v13.0.6447 but failed because it was not finished provisioning vagrant.
Please see this gist:
https://gist.github.com/choco-bot/3137078013ef0144c1ad9338140f4181

@bcurran3 @AdmiringWorm Have you seen anthing like this lately?

@mkevenaar
no, I have not.

No. I have not seen repeats of this recently.

I think this issue can be closed.
We have changed some settings in the Vagrantfile on the server.
/cc @gep13 @ferventcoder


Null safety feedback: Warn on unnecessary import silence

In order to support the migration, packages can disable warnings:
// ignore: import_of_legacy_library_into_null_safe
import 'package:shared_preferences/shared_preferences.dart';
However, when packages migrate to null safety, these ignores are no longer needed.  Since Dart can detect when this is necessary, I'm hoping that it can detect when it's not necessary and warn on these (as is now the case with the latest update to shared_preferences).  This warning would be a good candidate for pedantic.

We have support written for that, but have some internal clean-up that needs to happen before we can enable it.
The current plan is to enable the support always, not via lint rule, as we can't see any value for any user in having unnecessary ignore comments left in their code.

Yeah duplicate of https://github.com/dart-lang/sdk/issues/35234 or https://github.com/dart-lang/sdk/issues/25917


Add mock lease & Fix test

The old version has wrong logic about lease. This pr implemented it.
The old version has some wrong test logic. This pr use real etcd to fix the test logic.

The tests are somewhat simple. I didn't see any tests regarding lease expiration, it seems more like testing lock and unlock rather than the lease itself.

Added simple test


Support for Portenta board

First of all, I must say that it is a joy to use your library.
Please add support of your library to Arduino portenta C33.

I'm not familiar with that board, but this library isn't board-specific. It should work with anything that implements an Arduino Stream class.

Thanks Sara, but I'm still having problems with CRC uncorrect response. I was wondering if the Renesas architetture is not compatible with this library.
Paolo


0.10.0 broke assume-role

SharedConfigAssumeRoleError: failed to load assume role for arn:aws:iam::xxxxx:role/RoleName, source profile has no shared credentials

0.9.0 works fine.

I'm not able to repro this on linux or mac.  Can you give some more info on how you installed (build from source, github release, brew release, etc)?
This is especially odd considering nothing changed w.r.t assume role (only change between 0.9 and 0.10 was handling more okta factor types).

I installed from the binary release uploaded to GitHub.  Running macOS 10.13.3 (High Sierra).

This problem only occurs with packer.
We discovered that unsetting the AWS_PROFILE environment variable addresses the issue.

This has been fixed in v0.11.0


Display crate version on timings graph

What does this PR try to resolve?
Fixes #7384
The solution here is the simplest one: since duplicate crates with different versions can cause confusion, we always output the version.
How should we test and review this PR?
Build any create using the --timings option:
cargo build --timings

and verify the resulting HTML file.
Additional information
The formatting is consistent with how the crates are displayed in the terminal output for Cargo and also on the table below the graph. Initially, this comment suggested said formatting for ease of searching, but I do believe the browser can't search in the graph itself.
Sample console output:
   Compiling lazycell v1.3.0
   Compiling unicode-xid v0.2.4
   Compiling unicode-width v0.1.10
   Compiling glob v0.3.1
   Compiling curl-sys v0.4.65+curl-8.2.1
   Compiling curl v0.4.44
   Compiling git2 v0.17.2

Sample rendered HTML output:

Possible issues and future work
A possible issue in this solution is that the output becomes too noisy. Other possible solutions:

Only display the version if there are two "units" with the same name. One possible implementation is to count the names, aggregate them in a simple map and look it up during rendering.
Create a small selection to select the disambiguate method between "Never", "Always" and "Only duplicates". This may be overkill.

@bors r+

:pushpin: Commit da0fab31c6f47868742974619111a7d453f44188 has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit da0fab31c6f47868742974619111a7d453f44188 with merge 2da47eb48cc8b76013e55d4dfaf614888d2217e4...

Thought you've changed the format but apparently not.
Will wait for you replying back :)
@bors r-

:sunny: Try build successful - checks-actions
Build commit: 2da47eb48cc8b76013e55d4dfaf614888d2217e4 (2da47eb48cc8b76013e55d4dfaf614888d2217e4)

@weihanglo I've adjusted the format to number 2, only showing the version when there are duplicate crates and taking epage's suggestion to add a : separator and put the version between (). Here's a report so you can have a look yourself if you want.
cargo-timing-20230801T120113Z.tar.gz

BTW, could you help updates screenshots in that pages? Some of them could be in a better resolution.

I'm sorry, I don't know which screenshots you mean, but I'm happy to help if you point me in the correct direction.

Was talking about https://doc.rust-lang.org/nightly/cargo/reference/timings.html, though it is not necessary for this pull request. Anyway, thanks for the contribution!
@bors r+

:hourglass: Testing commit 28674b90eb2f24693f58076231ab81bd0852b202 with merge 63ec5e2e838ea0d9635509c3dcaa05ee49456d29...

But I'm still getting some duplicate behavior, apparently we are compiling the same crate twice? Or it's showing on the graph twice and I'm not sure why.

Anything different in the Unit can contribute to the duplicates. For examples, build script dependencies have different profile settings than normal runtime dependencies. And different "CompileMode" would also make it looks like a duplicate.
https://github.com/rust-lang/cargo/blob/c91a693e7977e33a1064b63a5daf5fb689f01651/src/cargo/core/compiler/unit.rs#L37-L85

Thanks for working with us on this!

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing 63ec5e2e838ea0d9635509c3dcaa05ee49456d29 to master...

No problem! This was quite an easy process. Thanks for the reviews and iteration on the PR! I'll contribute again when I can.


Publish

Docs Build status updates of commit 089aacd:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/develop/create-sso-office-add-ins-aspnet.md
:white_check_mark:Succeeded
View

docs/develop/create-sso-office-add-ins-nodejs.md
:white_check_mark:Succeeded
View

docs/excel/make-custom-functions-compatible-with-xll-udf.md
:white_check_mark:Succeeded
View

docs/outlook/append-on-send.md
:white_check_mark:Succeeded
View

docs/outlook/autolaunch.md
:white_check_mark:Succeeded
View

docs/quickstarts/outlook-quickstart.md
:white_check_mark:Succeeded
View

docs/reference/javascript-api-for-office-error-codes.md
:white_check_mark:Succeeded
View

docs/tutorials/outlook-tutorial.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Function idea: subscriptions through push notifications

I just want to put this out here in case anyone else is also considering the same approach. The idea is that you would set up a SSS, and send the data as a Push notification.
So it would basically behave as a client side subscription, but instead of getting the results over an active wss connection, you would get the results as push notification, for when your app is not running.
Based on: https://developers.google.com/web/fundamentals/getting-started/codelabs/push-notifications/ (client-side) and https://github.com/web-push-libs/web-push (SSS-side) for browser, and similar platforms for mobile push notifications (for example: AWS SNS).

Just to clarify for anyone else who's wondering (as it can be confusing with the terms that are almost similar: web, push & notifications), this is different from a web sockets approach like here: https://github.com/graphcool/graphcool/tree/master/examples/push-notifications


[MacOS] - Provide error message when accessing String.fromEnvironment as non-const

Recently I added some --dart-define values to a Mac OS build. I quickly coded it like this:
final myValue = String.fromEnvironment("MY_VARIABLE");

This resulted in empty values for my various build variables. I was confused why this was happening. There weren't any error messages, just empty values. I decided to build the same project for web and see what happens. When I ran the web version, I got an error complaining about needing to use a const constructor. That's when I realized that apparently I must access these environment variables as consts:
const myValue = String.fromEnvironment("MY_VARIABLE");

That fixed my issue.
Flutter should provide appropriate error messages on Mac OS and any other platform when these variables are accessed in illegal ways. It's difficult to notice the issue if a developer isn't already aware of it.
Flutter version: Flutter 2.8.0 • channel stable

Hi @matthew-carroll, I see a similar issue in https://github.com/flutter/flutter/issues/55870.
Kindly follow up there for more information.
Closing from here as duplicate.
If you disagree, kindly leave a comment below and I will reopen.
Thank you


Torque doesn't fallback to default renderers if there's no img list

When there are image assets in Carto, but in a particular point no image is used, it errors and the point is skipped.

Fixed in PRs above.


Usage with NPM?

Hello!
I was interested in using your plugin with my project, however it doesn't look like it plays nicely with NPM / Node based projects.  I was wondering if you had any documentation on setting it up for this use case?  Any time I import it into my project it simply errors out.  If you need any additional information or a code example then please let me know.
Thanks!

Hello,
You're right, it could do with being modularised.
The fork at https://github.com/pbellesa/videojs-abloop has this done simply for node.js / common js modules and might work
I'll try to get it changed to use JS modules as per the videojs project soon. Do you have an example of a plugin that does this well I could refer to?

...a code sample would be great if you could send one.

Thanks for sharing the fork.  The documentation that's over there (as far as setup and configuration) would likely be a better code example than what I could provide.  I wouldn't imagine the fork being worth using though considering how old it is.
If you still need a code example from me though, my plan is to include the functionality here within my project.
The setup would likely be similar to their getting started documentation:
import videojs from 'video.js'
import 'abLoopPlugin'

const player = videojs(elementId)

// Some plugins either auto-detect the videoJS instance or have the instance passed as a parameter
abLoopPlugin(videojs, { configOptions })

If you need anything else from me, then please let me know.

Thanks, I'll try to get this done soon

I've updated the code (v1.1.0) on github to work in node etc, as an ES6 module (not yet published to npm)
Could you please try using the file at https://github.com/phhu/videojs-abloop/blob/master/videojs-abloop.js and let me know if it works ok?

I think this is fixed - please let me know if there are any issues.


WIP: Use AnatomicalOrientation in  OrientImageFilter

Preserve compatibility with SpatialOrientation enums with implicit converstion. Additionally both the PositiveEnum and NegativeEnum are supported.

PR Checklist

[ ] No API changes were made (or the changes have been approved)
[ ] No major design changes were made (or the changes have been approved)
[ ] Added test (or behavior not changed)
[ ] Updated API documentation (or API not changed)
[ ] Added license to new files (if any)
[ ] Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
[ ] Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

The current failing test in related to a new tests added based on tests on the DICOMOrientedImage filter that tests streaming. This filter has a number of pipelining issues that also need to be addressed. I suppose that should be done in another PR.
Additionally the default behavior is to ignore the input image's direction cosine matrix, which is not the expected behavior of an ITK filter. Not sure what the "compatible" way to update this behavior is. The proposed approach of just a new filter with new/correct behavior did not seem to be liked by the community.

Is the rest of the class design in accordance with migration guide?

I am not certain what you are asking about. The OrientImageFilter does not have enums itself.

I meant the anatomical orientation enum classes, are they in accordance with ITK5 style? I totally forgot about that before, and your wrapping commit reminded me of that.

I meant the anatomical orientation enum classes, are they in accordance with ITK5 style? I totally forgot about that before, and your wrapping commit reminded me of that.

The AnatomicalOrientation class contains the enum and are not in the ITK namespace. This was done because scoping in the class space: itk::AnatomicalOrientation::PositiveEnum seemed more sensible than a single long Enum name e.g. itk::AnatomicalOrientationPositiveEnum.
However, the wrapping does not seems to be working for these enums. @thewtex  Can you look at the wrapping of these enums? Currently, I can't build ITK wrapping out of the box due to clang-xml binaries not working due to external library dependencies ( I create an issue some where for this).

Additionally the default behavior is to ignore the input image's direction cosine matrix, which is not the expected behavior of an ITK filter. Not sure what the "compatible" way to update this behavior is. The proposed approach of just a new filter with new/correct behavior did not seem to be liked by the community.

I am not sure where there was discussion or opposition to this approach, but providing a new filter with the correct behaviors sounds right to me.

However, the wrapping does not seems to be working for these enums. @thewtex Can you look at the wrapping of these enums? Currently, I can't build ITK wrapping out of the box due to clang-xml binaries not working due to external library dependencies ( I create an issue some where for this).

From the build error:

Wrapping/castxml_inputs/itkAnatomicalOrientation.cxx:17:10: fatal error: 'itkPositiveEnum.h' file not found

I suspect itk_wrap_include("itkPositiveEnum.h"), etc. is needed.

If I had to do this again I would not include the deprecation/legacy remove in the first PR, and add that in a separate PR. Doing it all had once require 3X the work, and problems testing and building.

I am not sure where there was discussion or opposition to this approach, but providing a new filter with the correct behaviors sounds right to me.

The pipeline and default behaviors where not mentioned but the general approach to replace the filter was discussed here:
https://github.com/InsightSoftwareConsortium/ITK/issues/4788

/azp run ITK.Windows


⚠ AGEG website (prod) has degraded performance

In d68810c, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 2105 ms

Resolved: AGEG website (prod) performance has improved in 644442c.


Incorrect event names in API doc for $firebaseArray.$watch

The API doc for $firebaseArray.$watch (https://github.com/firebase/angularfire/blob/master/src/FirebaseArray.js#L245) is incorrect.
Wrong: { type: 'added|updated|moved|removed', key: 'key_of_item_affected'}
Right: { event: 'child_added|child_updated|child_moved|child_removed', key: 'key_of_item_affected'}

If you find yourself depending on those event names (i.e. listening to server events), it's likely that you've misunderstood the purpose of the AngularFire classes and there's probably a better way to accomplish your goals. I'm surprised to see, in fact, that we pass anything into the $watch callback. That should have been removed before the 1.0 release.
In general, using $extend to modify the $$added, $$updated, and $$removed would be more appropriate for manipulating data and intercepting server events.  Anything that can't be done with these should probably use a custom service and tap directly into the Firebase SDK.

BTW: the $watch method is also documented here: https://www.firebase.com/docs/web/libraries/angular/api.html#angularfire-firebasearray-watchcb-context


React Native 0.60+ Autolinking feature is required

Sorry for reposting. Please re-open #397 again, because it had not been solved yet which had been closed by a slate bot.

Idiot bot

I think wix don't know that GitHub have a tab called "Issues" for reporting.


MapView's onPress-event is not fired when clicking on an Overlay-image (Android only)

Is this a bug report?
yes
Have you read the Installation Instructions?
yes
Environment
"react": "16.3.1"
"react-native": "0.55.3",
"react-native-maps": "0.21.0",
Steps to Reproduce
Render the MyMapView component posted below.
This should show a 300x300px MapView with ocean in the background, an Overlay showing Octocat, and a Marker in the middle:

Click/press on the MapView, either on the Octocat-image, or on the ocean in the background.
Expected Behavior
The onPress event of the MapView should be fired, and a marker be added at the position that was clicked.
Actual Behavior
On iOS, the marker is always added as expected.
On Android, the marker is added as expected when clicking on the blue background. However, it is not added when clicking on the Overlay-image. I verified in the debugger that the event is not being fired at all when clicking on octocat.
Reproducible Demo
import * as React from 'react';
import MapView from 'react-native-maps';

export default class MyMapView extends React.Component {
  constructor(props) {
    super(props);
    this.state = {
      markers: [
        { latitude: 0.001, longitude: 0.0 },
      ],
    };
  }

  addMarker(coordinates) {
    this.setState({
      markers: [...this.state.markers, { latitude: coordinates.latitude, longitude: coordinates.longitude }]
    });
  }

  render() {
    return (
      <MapView
        initialRegion={{
          latitude: 0,
          longitude: 0,
          latitudeDelta: 0.03,
          longitudeDelta: 0.03,
        }}
        width={300}
        height={300}
        onPress={event => this.addMarker(event.nativeEvent.coordinate)}
      >
        <MapView.Overlay
          image={'https://assets-cdn.github.com/images/modules/logos_page/GitHub-Mark.png'}
          bounds={[[0.01, -0.01], [-0.01, 0.01]]}
        />
        {this.state.markers.map(marker =>
          (<MapView.Marker
            key={marker.index}
            coordinate={{ latitude: marker.latitude, longitude: marker.longitude }}
          />)
        )}
      </MapView>
    );
  }
}

Hi, I am having the same issue.
Anyone have a way to mitigate this?

I am also struggling with this. Is there anyone with a workaround for this behavior?

According to Google Documentation the underlying GroundOverlay must have set the "Clickability" property to false.
AirMapOverlay.java sets this hardcoded to true.

Still no update on this?


Add option to NOT generate or import SSH key

what

Added import_ssh_key variable.
Add conditional check for if import and generate are both used it gives error message.
Fixed errors that happen when import and generate values are both false

why

Generate added option to use module without generating or importing a key

@Jamie-BitFlight
Thanks again for testing and submitting the PR.
See the inline comments.
I think I don't really understand the problem this PR is trying to solve.
The module can already either import an existing key
https://github.com/cloudposse/terraform-aws-key-pair/blob/master/main.tf#L16,
or generate a key and then import it into AWS
https://github.com/cloudposse/terraform-aws-key-pair/blob/master/main.tf#L27
Please give more info on what you are trying to achieve.

@aknysh The problem is, it can't not do both.
So if you use the terraform-aws-key-pair module in a module that isn't enabled.
Then you can't also disable terraform-aws-key-pair.
If the terraform-aws-ec2-instance module, sets generate_ssh_key_pair to false then terraform-aws-key-pair tries to load a .pub file. Which doesn't exist.
I made this change so that, if generate_ssh_key is false, it doesn't generate a key.
And added the import_ssh_key to allow for the importing.

@Jamie-BitFlight  will close it for now since it's very old.
Please reopen if needed


Pathfinder v0.64

iterative macro fixes
html css fixes

THIS INCLUDES THE .63 pull request so you can ignore .63 and go right to .64
https://github.com/Roll20/roll20-character-sheets/pull/1839


Output OSC 7 "File URI" sequence where appropriate

Description of the new feature/enhancement
In the hopefully not too far future, we'll see https://github.com/microsoft/terminal/pull/8921 and follow up PRs land. Additionally there are other terminals that support this sequence right now such as Mintty. Winget can take advantage of this sequence when displaying a list of installed applications including when an application is first installed for example. Users can click on that "link" and open explorer.exe at the path of the installed application. This can be useful when needing to troubleshoot installations of an application or uninstallation where winget may be having issues (or an underlying dependency within Windows such as the "Uninstall or Change a Program" in Programs and Features within the Control Panel.) The use of the sequence will be dependent on the ability to list installed applications including eventually the built-in features.

Proposed technical implementation details (optional)

This is not what OSC 7 is for. You want OSC 8 to emit hyperlinks. OSC 7 communicates the working directory from a process to a terminal.


[SPARK-7315][STREAMING][TEST] Fix flaky WALBackedBlockRDDSuite

FileUtils.getTempDirectoryPath() path may or may not exist. We want to make sure that it does not exist.

Merged build triggered.

Merged build started.

Test build #31629 has started for   PR 5853 at commit b08d4f1.

Merged build triggered.

Merged build started.

Test build #31629 has finished for   PR 5853 at commit b08d4f1.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
     "public class " + className + extendsText + " implements java.io.Serializable
class DataFrameStatFunctions(object):

Test build #31636 has started for   PR 5853 at commit 141afd5.

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

Merged build finished. Test PASSed.

Test build #31636 has finished for   PR 5853 at commit 141afd5.

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/31636/
Test PASSed.


fix(Dropdown): handle text as a content in renderLabel

Fixes #4046.

This PR fixes a regression from #4003 (v1.1.0).
That change affected renderLabel prop in Dropdown (will be used to render labels with multiple). In #4003 I decided to avoid breaking changes and keep the existing behavior via this condition:
if (React.isValidElement(text) || _.isFunction(text)) {

However, I can't consider the previous behavior as valid as it will break the layout once text will be a JSX element (see behaviors below).
test snippet
<Grid columns={2}>
  <Grid.Column>
    <Dropdown
      placeholder="Select Friend"
      fluid
      selection
      multiple
      options={[
        {
          key: "Jenny Hess",
          text: Jenny Hess,
          value: "Jenny Hess"
        },
        {
          key: "Elliot Fu",
          text: "Elliot Fu",
          value: "Elliot Fu"
        }
      ]}
    />
  </Grid.Column>
  <Grid.Column>
    <Dropdown
      placeholder="Select Friend"
      fluid
      selection
      options={[
        {
          key: "Jenny Hess",
          text: Jenny Hess,
          value: "Jenny Hess"
        },
        {
          key: "Elliot Fu",
          text: "Elliot Fu",
          value: "Elliot Fu"
        }
      ]}
    />
  </Grid.Column>
</Grid>

Mention that in the first case text is a JSX element.
pre v1.1.0 behavior

https://codesandbox.io/s/busy-hooks-5p5pq
v1.1.0 behavior

https://codesandbox.io/s/distracted-dawn-xg28c
fix in this PR

To keep previous behavior use:
<Dropdown renderLabel={({ text }) => text} />

Codecov Report

Merging #4047 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #4047   +/-   ##
=======================================
  Coverage   99.84%   99.84%
=======================================
  Files         186      183    -3
  Lines        3255     3280   +25
=======================================
+ Hits         3250     3275   +25
  Misses          5        5

Impacted Files
Coverage Δ

src/addons/Portal/Portal.js
100.00% <ø> (ø)

src/addons/Portal/PortalInner.js
100.00% <ø> (ø)

src/addons/TextArea/TextArea.js
100.00% <ø> (ø)

src/behaviors/Visibility/Visibility.js
100.00% <ø> (ø)

src/collections/Form/FormField.js
100.00% <ø> (ø)

src/elements/Button/Button.js
95.55% <ø> (ø)

src/elements/Input/Input.js
100.00% <ø> (ø)

src/elements/Placeholder/Placeholder.js
100.00% <ø> (ø)

src/modules/Checkbox/Checkbox.js
100.00% <ø> (ø)

src/modules/Dimmer/DimmerInner.js
100.00% <ø> (ø)

... and 13 more

Continue to review full report at Codecov.

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

Released in<EMAIL_ADDRESS>🚢


Do we need to add some operator like rxjs？

delay
merge
race
all
cancel
fork
call
put
take
takeEvery
takeLatest
takeLeading

just those frequently used


Quick Start Guides - Incorrect Error Message

TypeScript Version:  2.2.1 / nightly (2.2.0-dev.201xxxxx)
Code
In section Type annotations you are shown that trying to use [0, 1, 2] as a string gives an error.
While running this myself I got different behavior from the TS transpiler/compiler.
// A *self-contained* demonstration of the problem follows...
function greeter(person: string) {
    return "Hello, " + person;
}

var user = [0, 1, 2];

document.body.innerHTML = greeter(user);

Expected behavior:
greeter.ts(7,35): error TS2345: Argument of type 'number[]' is not assignable to parameter of type 'string'

Actual behavior:
greeter.ts(7,26): Supplied parameters do not match any signature of call target

A PR to the handbook (https://github.com/Microsoft/TypeScript-Handbook) would be appreciated.

I will create a pull request soon. I read on the readme that I shouldn't make pull requests but instead open issues 😀

Where did you see that? In doc/README.md, we say to only open issues on the spec. We redirect handbook users to the handbook repository.

Ok maybe that's why I was mistaken thanks
On Apr 21, 2017 10:01 PM, "Daniel Rosenwasser"<EMAIL_ADDRESS>wrote:

Where did you see that? In doc/README.md
https://github.com/Microsoft/TypeScript/blob/a1a2006074a01ca0b75b08294377cc52302e3e71/doc/README.md,
we say to only open issues on the spec. We redirect handbook users to the
handbook repository.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/TypeScript/issues/15302#issuecomment-296278389,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAbmZx-8Fcq-8asjWTzFIJ-DHAyFsFSdks5ryPz9gaJpZM4ND3jH
.

The old error message is more helpful and descriptive than the new one. Is the best resolution to fix the documentation? Looks like a regression to me.

@gcnew But the new error message is the current error message that is thrown when using this code.

@Alonski Yes, I understand that. I'm trying to start a discussion, because helpful messages are one of the major selling points of TypeScript. This is the simplest of function invokations - no overloads, no generics, no unions. My understandig is that the new error message is a serious degradation and maybe the problem should be solved in the compoler by bringing the old behaviour back.

@gcnew has a good point - the content was written that way to make it easier for users to understand what's going on. The real problem is that the compiler is gives bad error messages.

This works better now


[Task] Add integration test for more dubbo api

Description
Hi community,
In integrated test of alibaba-dubbo or apache-dubbo, there is only /findById is covered. But we should test all dubbo api in examples. So we need to do the following task.
If you want to participate, please leave a message under issue.
Task List

[ ] add integrated test for DubboTestService in shenyu-examples-apache-dubbo-service
[ ] add integrated test for DubboMultiParamService in shenyu-examples-apache-dubbo-service
[ ] add integrated test for DubboTestService in shenyu-examples-alibaba-dubbo-service
[ ] add integrated test for DubboMultiParamService in shenyu-examples-alibaba-dubbo-service

The integration test is expected to add in shenyu-integration-test-alibaba-dubbo or shenyu-integration-test-apache-dubbo, and you can new a test class while doing integrated test.

please assign these to me

add integrated test for DubboTestService in shenyu-examples-apache-dubbo-service
add integrated test for DubboTestService in shenyu-examples-alibaba-dubbo-service
the other two have some problem, I am writing this


Compatiablity with .net core applications

We are in the process of migrating our libraries to .net 2.0 and this caused an issue in our test project as it no longer functions in our unit tests because we changed the test project to a .net core(2.0) console application. Right now, the settings plugin will throw a not implemented exception when run in our .net core console app. I am not sure if we are doing something wrong or if it isn't currently compatible.
Our approach is to see if we can fix the plugin or we will migrate the test project back to net framework project with a target of 4.7.x
Let me know if you need more info.
Thanks

Install the 3.2.0 or 4.0.0 beta and will work out nicely for you :)

Worked great, thanks for the quick response.


async/await for twisted ApplicationSession

From the version  16.4 twisted support async/await (py3.5+) https://twistedmatrix.com/documents/current/core/howto/defer-intro.html#coroutines-with-async-await
but it doesn't work in crossbar (crossbar-0.15, autobahn-0.16.0, py3.5, twisted-16.4.1, txaio-2.5.1):
from autobahn.twisted.wamp import ApplicationSession
from autobahn.wamp.exception import ApplicationError

class AuthenticatorSession(ApplicationSession):
    async def onJoin(self, details):
        async def authenticate(realm, authid, details):
            ticket = details['ticket']
            ....
            return data['role']

        try:
            await self.register(authenticate, 'com.app.authenticate')
            self.log.info("Dynamic authenticator registered")
        except Exception as e:
            raise e

in logs i have warning: ...python3.5/site-packages/twisted/internet/defer.py:587: builtins.RuntimeWarning: coroutine 'onJoin' was never awaited
so, will be fine support this or if this is my fail - add example (re #653)

I looked into this a little; it revealed a couple bugs:

Twisted had a bug in Deferred support for this (https://twistedmatrix.com/trac/ticket/8861)
txaio will needs a couple minor changes

...but the good news is that with those changes, it does in fact work. The first step will be to wait for the Twisted 16.5.0 release, though (which does contain the .send fixes).

16.6 is out which has all the fixes. As far as I know, it should be solid now.

Hello,
Does it work now ?
And more importantly, can I use it for the websocket part with twisted ?
I'm currently working on some writing material for Tobias, and he adviced:
"Could you post your findings to above issue, so we can take  @meejah in the loop, because he might know better" :)
I tried something like this:
class EchoClientProtocol(WebSocketClientProtocol):
    ...
    async def onOpen(self):
       await do_async_stuff()
    ...

It works with asyncio of course, but not with twisted, where I get the "coroutine 'onOpen' was never awaited" mentioned previously.
Did I get something wrong or the support is not in yet ? I don't need the feature, I just need to know what's the best code style I can use for the writing.

>>> import twisted
>>> twisted.__version__
'17.5.0'
>>> import autobahn
>>> autobahn.__version__
'17.8.1'

I believe we'd need to enhance either Autobahn (or txaio) so that async def methods for the Autobahn/WebSocket callbacks work properly -- basically, you need to wrap them in an ensureDeferred call for Twisted to turn the async-def method into a Deferred-returning method...

That said, pure "user" code should be able to take advantage of Python3 features right now like so (untested):
class Component(ApplicationSession):
    def onJoin(self, details):
        return ensureDeferred(self._on_join(details))

    async def _on_join(self, details):
        print("joined {}".format(details))
        await self._other_async()

@meejah this sounds like we could get away with adding overrides for onJoin etc to autobahn.twisted.wamp.ApplicationSession the the bodies simply returning the base result wrapped in ensureDeferred ?

Hmm, maybe? I've not played with ensureDeferred enough to know if that'll play nicely with Py2 code using "not async-def".
I think we might need to check if calling self.onJoin (et al) returned a co-routine or not -- which might make it a good thing to just put in txaio.ensure_future instead (i.e. that can then "just work" no matter if the function returns a Deferred directly, uses @inlineCallbacks or async def)

@ksamuel rgd the AB chapter, I guess even if we get it working easily and quickly (as above seems to indicate), essentially all code out there uses AB/Twisted in an inlineCallback/Deferred kind of style, and hence one could argue the text should use that style as well. On the other hand, the new style is just .. nicer=) a little syntax sugar goes a long way. Opinions?
@meejah yeah, sounds good! txaio is a better place for this mangling of return values into one type. I am wondering: do you mean a new function (txaio.ensure_future) or the existing txaio.as_future:
(cpy362_1) oberstet@thinkpad-t430s:~/scm/crossbario/crossbar-fabric-public$ find /home/oberstet/scm/crossbario/autobahn-python/autobahn -name "*.py" -exec grep -Hi "as_future" {} \;
/home/oberstet/scm/crossbario/autobahn-python/autobahn/util.py:            future = txaio.as_future(handler, *args, **kwargs)
/home/oberstet/scm/crossbario/autobahn-python/autobahn/wamp/component.py:                    d = txaio.as_future(self._entry, reactor, session)
/home/oberstet/scm/crossbario/autobahn-python/autobahn/wamp/component.py:        d = txaio.as_future(comp.start, reactor)
/home/oberstet/scm/crossbario/autobahn-python/autobahn/wamp/protocol.py:            lambda _: txaio.as_future(self.onConnect),
/home/oberstet/scm/crossbario/autobahn-python/autobahn/wamp/protocol.py:                    lambda _: txaio.as_future(self.onJoin, details),
/home/oberstet/scm/crossbario/autobahn-python/autobahn/wamp/protocol.py:                d = txaio.as_future(self.onLeave, details)
/home/oberstet/scm/crossbario/autobahn-python/autobahn/wamp/protocol.py:                d = txaio.as_future(self.onChallenge, challenge)
/home/oberstet/scm/crossbario/autobahn-python/autobahn/wamp/protocol.py:                    d = txaio.as_future(self.onLeave, details)
/home/oberstet/scm/crossbario/autobahn-python/autobahn/wamp/protocol.py:                d = txaio.as_future(self.onLeave, details)
/home/oberstet/scm/crossbario/autobahn-python/autobahn/wamp/protocol.py:                        future = txaio.as_future(handler.fn, *invoke_args, **invoke_kwargs)
/home/oberstet/scm/crossbario/autobahn-python/autobahn/wamp/protocol.py:                            on_reply = txaio.as_future(endpoint.fn, *invoke_args, **invoke_kwargs)
/home/oberstet/scm/crossbario/autobahn-python/autobahn/wamp/protocol.py:            d = txaio.as_future(self.onLeave, details)
/home/oberstet/scm/crossbario/autobahn-python/autobahn/wamp/protocol.py:        d = txaio.as_future(self.onDisconnect)
/home/oberstet/scm/crossbario/autobahn-python/autobahn/websocket/protocol.py:                f = txaio.as_future(self.onConnect, request)

Oh, sorry yes I meant existing as_future

Ah, right;) Ok, then as far as I see, there is actually 2 issues here:

txaio: add the ensureDeferred support to txaio.as_future
AB: actually use it (also in WebSocket code) ;) as above grep shows, it is only used in onConnect right now, but it should be used in the other hooks as well ..

Does that make sense?

@ksamuel rgd the AB chapter, I guess even if we get it working easily and quickly (as above seems to indicate), essentially all code out there uses AB/Twisted in an inlineCallback/Deferred kind of style, and hence one could argue the text should use that style as well. On the other hand, the new style is just .. nicer=) a little syntax sugar goes a long way. Opinions?

Opinions?

@oberstet: I will start writing all code using the inlineCallback/Deferred syntax. If we get a release allowing async / await before I submit my final draft, I'll add a note with an example using the new syntax and a mention AB supports it. But most of the code will use the inlineCallback/Deferred syntax in order to be congruent with the rest of the book.

@ksamuel I think that makes the most sense; most Twisted users will likely "expect" to see "straight Deferreds" or "@inlineCallbacks" style code.

async/await should be fully usable on Py3/Twisted now with Autobahn