Docker image node:2.0 fails to build on azure pipelines

I get this error:
The command '/bin/sh -c apt-get install build-essential libcairo2-dev libpango1.0-dev libjpeg-dev libgif-dev librsvg2-dev' returned a non-zero code: 1
##[error]The command '/bin/sh -c apt-get install build-essential libcairo2-dev libpango1.0-dev libjpeg-dev libgif-dev librsvg2-dev' returned a non-zero code: 1
##[error]/usr/bin/docker failed with return code: 1

The azure pipelines actually add more steps than I get when building normally with docker, this is the pipeline step:
steps:
- task: Docker@2
  displayName: build
  inputs:
    containerRegistry: 'rabobank-docker'
    repository: 'pdf-functions'
    command: build
    Dockerfile: Dockerfile
  env:
    AzureFunctionsJobHost__Logging__Console__IsEnabled: true

And this is the dockerfile:
FROM mcr.microsoft.com/azure-functions/node:2.0

ENV AzureWebJobsScriptRoot=/home/site/wwwroot

COPY . /home/site/wwwroot
WORKDIR /home/site/wwwroot

# install dependencies and build
RUN npm install -g yarn
RUN yarn install
RUN yarn build

That's a really odd error. I'm not sure why it would be trying to install these packages. Are these installed during your yarn build step? or does this happen before

I'm also seeing a problem building a custom Python worker containing FFMPEG. Perhaps there's an issue with the functions-base image? I've tried this build and a previous build installing bedtools (which used to work).
The command '/bin/sh -c apt-get update &&     apt-get install -y wget &&     echo 'debconf debconf/frontend select Noninteractive' | debconf-set-selections &&     apt-get update &&     apt-get install -y apt-transport-https curl gnupg &&     curl https://packages.microsoft.com/keys/microsoft.asc | apt-key add - &&     curl https://packages.microsoft.com/config/debian/9/prod.list > /etc/apt/sources.list.d/mssql-release.list &&     echo 'deb http://security.debian.org/debian-security jessie/updates main' >> /etc/apt/sources.list &&     apt-get update && apt-get install -y locales &&     echo 'en_US.UTF-8 UTF-8' > /etc/locale.gen &&     locale-gen &&     apt-get update &&     apt-get install -y debconf libcurl3 libc6 openssl libstdc++6 libkrb5-3 unixodbc msodbcsql17 mssql-tools libssl1.0.0     git make build-essential libssl-dev zlib1g-dev libbz2-dev      libreadline-dev libsqlite3-dev wget curl llvm libncurses5-dev libncursesw5-dev     xz-utils tk-dev libpq-dev python3-dev libevent-dev unixodbc-dev &&     curl -L https://github.com/pyenv/pyenv-installer/raw/master/bin/pyenv-installer | bash &&     apt-get install ffmpeg &&     PYTHON_CONFIGURE_OPTS="--enable-shared" pyenv install $PYTHON_VERSION &&     pyenv global $PYTHON_VERSION &&     pip install pip==$PYTHON_PIP_VERSION && 	    export WORKER_TAG=1.0.0b4 &&     export AZURE_FUNCTIONS_PACKAGE_VERSION=1.0.0b3 &&    wget --quiet https://github.com/Azure/azure-functions-python-worker/archive/$WORKER_TAG.tar.gz &&     tar xvzf $WORKER_TAG.tar.gz &&     mv azure-functions-python-worker-* azure-functions-python-worker &&     mv /azure-functions-python-worker/python /python &&     rm -rf $WORKER_TAG.tar.gz /azure-functions-python-worker &&     pip install azure-functions==$AZURE_FUNCTIONS_PACKAGE_VERSION azure-functions-worker==$WORKER_TAG' returned a non-zero code: 1


[DH-542] Add investment project modified-since endpoint for MI

Implemented as per spec in https://github.com/uktrade/data-hub-api-spec/.

Codecov Report

Merging #457 into develop will increase coverage by 0.02%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #457      +/-   ##
===========================================
+ Coverage    96.26%   96.28%   +0.02%
===========================================
  Files          107      107
  Lines         3239     3257      +18
  Branches       289      289
===========================================
+ Hits          3118     3136      +18
  Misses          84       84
  Partials        37       37

Impacted Files
Coverage Δ

datahub/investment/urls.py
100% <100%> (ø)
:arrow_up:

datahub/search/investment/views.py
94.17% <0%> (+1.15%)
:arrow_up:

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 1d14b55...a1b3170. Read the comment docs.


Some kind of issue with chunk loading

I have a .schem file with width 127 and length 113 (according to nbt.getShort("Width") and nbt.getShort("Length") in Schematic.toWorld() ).
With the default spawn point set in server.properties (world-spawn=world;20.5;17;22.5;-90;0), some parts of the map are never sent to the client, which means that they fall out of my map in those areas.
When I make no other changes except set the spawn point in server.properties to world-spawn=world;62.5;21;36.5;-45;0, then the whole map appears to be correctly transmitted to the client.  (I have barrier blocks around the outer edge of my map, and have verified that everything I can get to seems to be correctly loaded as they're all showing up.).
As near as I can tell, it's some sort of chunk loading issue, but I haven't been able to figure out exactly what's wrong.  I instrumented PlayerInteractManager.update() and don't see the missing chunks get loaded when the player moves, so I suspect there is an issue there somewhere, but I haven't been able to identify the root cause yet.
I'm testing with the latest master commit (e88656b1bf1eb4ff1f940a63da65ef7eb7e889a7) at the time of writing.
default world spawn (world-spawn=world;20.5;17;22.5;-90;0)

custom world spawn (world-spawn=world;62.5;21;36.5;-45;0)
Note that this is exactly the same place the previous screen shot was taken from.  The only change is the world-spawn coordinates in server.properties.

I believe if a chunk is right next to the edge of the world (chunks outside of the schem) they don't render properly. You can try to see if this is the case by adding an extra empty chunk on each side of your schem.


Add simple basic auth support and allow custom guzzle client.

I saw #20 and didn't fully like that implementation as it uses another method to add basic auth support which seems a bit odd...
So I added 2 additional and optional constructor arguments along with a setter to inject your custom guzzle client so you can do anything you like with it, including adding middleware etc...

Closed in favor of #22


i18n support concensus

how can i help with internationalization with russian, spanish and greek?
i only see the deustland languaje supoported and of course english

hey hey, calm u'r horses!!! i see a po file and a mo file, must i genretae the mo file with gettex , if so why then the php file?

you just need to add a new php file for each locale, copy the de.php for each locally ;)

added storqagechars2 es and ru texts with  064a80f9013c9419979b425bbbc9609d3523579f and started a wiki how to with two ways, the simple as added those files and another way such using translation web utility program interface or other..

thanks for the support

ignore the email from jenkins. I currently restructuring the tests!

made stup of wiki, work in progress https://github.com/hypery2k/owncloud/wiki/i18n-translations-internationalization-support-how-to
spanish and russian for roundcube and revealjs are in progress

due wiki are started i closed here, each i18n file will be a independent issue

Mmmh. Actually the getext template and intermediate translation .po-files are rather used to automatically add new text to the collection of strings or messages that should be translated. This gettext stuff is rather a method to keep thing in sync with the source-code, which is why I do not quite understand the point of removing the template files with commit  4e424d6f66db0a5361728bd4d4102fcf06cd2af3
Just a question. Just keeping the php-files and nothing else means that each new to-be-translated message has to be added manually to each translation file.
In this respect it should also be noted that in OC8 the php-files are no longer used: translations are converted into json format. There is a Perl-script in owncloud/l10n/l10n.pl (here "owncloud/" refers to the base-directory of the Owncloud installation, not to Martins OC-applications) which can be used to scan the source-code for messages and calls xgettext in order to update .pot and .po files.
One of the files removed by 4e424d6f66db0a5361728bd4d4102fcf06cd2af3 was a slightly tweaked version of that script, originating from the OC7 distribution.
Best,
Claus

umm that what's i hate when a enterprise leade and open source proyect! sure when i learn how to produce and pushed i18n files oc developer will change the api again!

Still this does not mean that gettext is useless; gettext is not an OC thing. Also, at least the i18n.pl script which converts the .po files to js and json is delivered together with the OC releases.
Regardless of the API issues, personally, I do not think that is a good idea to maintain hand-made translation files. I think that something like gettext pays off in the long run. Further, w.r.t. the API changes: if the po-files are used as the primary source for the translated messages, then API changes like this do not hurt so much.
Cheers,
Claus

i agree about mantain hand-made thinks, but its a system too complicated for few strings...

i changed the issue tittle, now here we have to discuss how we doit with i18n support and how we made that.. now post ideas and REMENBER easy to mantain and generated, we must take care about those apps must be easy to colaborate to others... and for author

OTOH, the rest of Owncloud uses the GNU gettext thing, which is also otherwise widely in use, at least in the open-source sector. Reinventing the wheel always bears the danger of finally ending up with something not less complicated and confusing as the original thing. Also, some authors work on more then just one App. So even if there are only few strings in Roundcube and StorageCharts, it might still be a good a idea to use a standard tool, or tools which are also used by other apps.

gettext as firts proposed.. ok...
note: i not part of mantain manually do it! but if are any more esyle rather that gettext i'll take it!

see https://github.com/hypery2k/owncloud/wiki/Localization for more details

je je very funny great script @hypery2k i can see that uses gettex so the system are then gettex, so the po file must be in sync with the sources, that part are by u'r own..
users part are only run the script, so the script i think (due i can see) generates or updates files using the po template?

hi @hypery2k i uploaded a firts translation file, this time for spanish ony, russian will waith for my (very bad) professor (@nerv55) to revise if i commit well the strings jeje
but i got and error wehen goes to write the php file... got this:
intranet@sysmail00:~/Devel/owncloud/roundcube/src/main/php/l10n$ ../../../../../scripts/l10n.pl roundcube write
Can't locate Locale/PO.pm in @INC (@INC contains: /etc/perl /usr/local/lib/perl/5.10.1 /usr/local/share/perl/5.10.1 /usr/lib/perl5 /usr/share/perl5 /usr/lib/perl/5.10 /usr/share/perl/5.10 /usr/local/lib/site_perl .) at ../../../../../scripts/l10n.pl line 3.
BEGIN failed--compilation aborted at ../../../../../scripts/l10n.pl line 3.

i soved installing in debian squeeze/lenny liblocale-po-perl package so i might document this in wiki, but  this work only for debian based, for any other distro each users must be by own

edited and added requeriments if got errors:
https://github.com/hypery2k/owncloud/wiki/Localization#requeriments

DOES storagechars2 app will stay as is or also a mechanism for language support will be generated? (i need what will be for close this issue)


Add all documented result codes

Coverage remained the same at 100.0% when pulling 8e9dc62f01f4b6ccaab819d21d92f3d6c53e3e1c on more-constants into 8d82fc665d619965a73c895416d8b91bfc4d1f5d on master.


ENYO-5994: Forward key events in VirtualList

Checklist

[x] I have read and understand the contribution guide

[ ] A CHANGELOG entry is included

[ ] At least one test case is included for this feature or bug fix

[x] Documentation was added or is not needed

[ ] This is an API breaking change

Issue Resolved / Feature Added
Applications cannot respond to onKeyDown events when passed to a VirtualList, since we blindly stop event propagation when capturing it in the list.
Resolution
We can use the react dispatcher to turn our key events into synthetic events. This will allow us to properly forward events and allow them to be acted on by the application. Also, I've added a change to only stop event propagation when focus actually moves within the VirtualList.
Additional Considerations
I've created this as a draft PR, since I have real performance concerns about this change. Converting the key events to synthetic events means that we're introducing the overhead of the react dispatcher for every key event.
It's fairly critical that key events in VirtualList be as lightweight as possible, this helps scrolling performance when navigating via 5way - especially during a hold, where key events occur rapidly. Attaching an event listener directly to the dom allows us to immediately respond to the event - whereas using synthetic events adds time to the amount of work that needs to be done per event.
When using synthetic events, the time it takes to perform a single keydown event increases by 20ms. When performing a 5way hold where the speed of focus change is near its maximum, focus change takes place in the scroll event, so less time is required in the key events themselves. That being said, synthetic events increase the time of each keydown event by 3x the amount (from ~3ms to ~9ms).
Measurements are using chrome 6x slow.
This does not seem ideal to me, but would like thoughts or alternative solutions.

@germboy

What is the usability impact of the change on device?
Do we have a "safe" threshold under which we should stay for handling key events to prevent jank?
Would it be possible to only support the synthetic event callback on the first keydown and then swap to native listeners for subsequent events? This might be super weird but perhaps worth considering.

We may have to consider an alternative impl for the use case than forwardWithPrevent if the perf cost is too great. Maybe handling keydown in the capture phase for this scenario would be sufficient since the app could stop it then and avoid key handling by VL.

To address your first 2 points:

The impact appears minor - however, it would easily negate the minor gains we made in our last round of optimizing key handlers for VL. The minor gains we previously made were what allowed us to restore animations to VL - and even then, @viodragon2 shared concerns that it may not be improved enough.
I don't think we have identified a "safe" threshold. I think we're still bordering on not optimized enough, so adding any additional time to key events concerns me.
Supporting an initial synthetic event then passing off to native listeners is indeed weird ... it has potential to introduce errors, but theoretically could be an option. I'll investigate it.
Thanks for the input - I'll investigate the options suggested 👍


Cape and Elytra mappings for PlayerEntity

Most of the fields mapped are to do with controlling the trailing position and angle of the cape when displayed.
Also filled in some missing method params/method names.

@Juuxel Branch updated


[Feature Proposal] Pin full

This is similar to #370 but I don't necessarily agree that pin sizes are needed, hence the new issue.
(I think the pin-r mr-2 workaround suggested in that issue is solid.)
What I was hoping for were options for pinning 100%.
Maybe something like:
pin-l-full pin-r-full pin-t-full pin-b-full
Submenu use-case:

I do not want to comment class names, but I often use 50% and 100%, so I'm definitely for any solution.

I was looking for something like this.
I'd like to extend this proposal even further to add a way to define custom values like for paddings:
pin: {
    'half': '50%',
    'full': '100%',
    'custom': '127.36%',
},

Resulting in:
.pin-l-half {
    left: 50%;
}
.pin-l-full {
    left: 100%;
}
.pin-l-custom {
    left: 127.36%;
}
.pin-t-half {
    top: 50%;
}
/* And so on */

@adamwathan, if you're interested in this, I can put together a pull request.

I think this would be better as a plugin using the top-, left-, bottom-, and right- prefixes instead of trying to put it under pin, because pin doesn't make sense as a name anymore if the element isn't actually pinned to the edge.
I'm not entirely opposed to making this change in core at some point but I definitely don't want to add new values under the pin naming convention.

@adamwathan Given that Tailwind 1.0 changed the naming for position utilities, top-full bottom-full left-full right-full seem like they would fit in well now. Can we revisit the possibility of these?

@chasegiunta If you're not using the CDN version of tailwind, you can add those utilities using the inset plugin in your tailwind config file:
// tailwind.config.js

module.exports = {
  theme: {
    extend: {
      inset: {
        'full': '100%'
      }
    }
  }
}

@AlexVipond Ah! Totally missed that somehow in the other issues. Thanks! That'll do for now. I guess the question kinda remains as to whether it's usage is prevalent enough to warrant being added to the default.

@AlexVipond Actually, inset doesn't have directional modifiers, which means doing something like bottom: 100%; requires two classes, inset-y-full top-auto so it's still a little hacky.

@chasegiunta Adding full to inset will also generate classes such as bottom-full.

@benface Major facepalm. Everyone forget this happened.


Debugging dotnet core app on Raspberry Pi & Linux?

Nope.
grumble I see the problem, there does not look to be a linux-arm debug package at present (at least where I would expect it).
Over SSH the remote device accesses https://aka.ms/getvsdbgsh and runs the script, on the Ubuntu machine, it looks something like:
GetVsDbg.sh -r linux-x64 -v vs2017u1
which results in a download of https://vsdebugger.azureedge.net/vsdbg-15-1-10630-1/vsdbg-linux-x64.zip
when run on the pi, the RID is different:
GetVsDbg.sh -r linux-arm -v vs2017u1
And then an HTTP request to https://vsdebugger.azureedge.net/vsdbg-15-1-10630-1/vsdbg-linux-arm.zip fails as there is nothing there.
Aside from the rather poor experience of having this silently fail without any information available in VS, it is a shame that there is not a debug package for linux-arm.
Looking at the x64 package, I'm not quite sure where those binaries are coming from (source/repo wise), but I guess some more hunting is required.

Original issue:
I've got a .NET core (2.0) app which works perfectly on Windows and on a Raspberry Pi 2 (and 3) running Rasbian Jessie when run interactively. Wanting it to run headless and full time, so set it up as a systemd service and found it to be crashing ~30 seconds after startup, then leading to another instance being run. This of course is bad.
This made me look into the remote debugging abilities for .net core apps in VS2017 (15.3.4), which seems to be supported over SSH. I even validated this by deploying my app to an x64 Ubuntu 16.04 LTS VM. I could not only run the app, but remote debug via VS2017 over SSH.
For some reason, I cannot do the same on either a Pi 2 or Pi 3 with which I've tried on this app.
Visual Studio can connect and list processes, selecting the process I see and opting to debug with the 'Managed (.NET Core for Unix)' debugger, will initially see a modal dialog saying "Loading debug adapter..." for a period of time, then will be dropped back to VS with no further information. Further attempts to attach fail more quickly.
I can attach with "Native (GDB) code", however that doesn't give me the info I am looking for.
The .net core sdk does not today work on Raspberry Pi, but AFAIK that shouldn't matter here when it wasn't installed to the Ubuntu machine either. In all cases I am simply running the self-contained executable, so am not sure what else might need to be on the system
App is being built & published with:
dotnet publish -c debug -r linux-arm
or
dotnet publish -c debug -r ubuntu.16.04-x64
Are there any logs on the Linux or VS Windows side which I should be looking at to understand what might be failing?

@sywhang @gregg-miskelly for investigation.

This issue tracks supporting ARM debugging for the vsdbg side: https://github.com/OmniSharp/omnisharp-vscode/issues/1562
The .NET Runtime likely also has work, but I don't know if they have an issue tracking it.

After my second post I did eventually find that OmniSharp ticket.
At the very least, the owners of GetVsDbg.sh & Visual Studio need to have a little chat and allow one to tell the other "No compatible debugger packages could be located." Better yet, also adding "In the mean time, please try debugging against a supported platform, here is a list..."
In the end, I deployed my app to the Ubuntu VM and was able to debug it and solve my issues there, which then unblocked my initial problem on the Pi... however there must be a better experience in this, as clearly I am not the only person to spend quite a bit of time trying to figure out why something (which currently isn't supported (which is ok)) doesn't work.

@gregg-miskelly is there a place to file an issue for this comment?

At the very least, the owners of GetVsDbg.sh & Visual Studio need to have a little chat and allow one to tell the other "No compatible debugger packages could be located." Better yet, also adding "In the mean time, please try debugging against a supported platform, here is a list..."

For now I'm going to close this issue because it's tracked in a larger theme of enabling first class arm32 support.

You can try this now: https://github.com/OmniSharp/omnisharp-vscode/issues/1562#issuecomment-348349242

How about ./getvsdbgshbeta -v latest -r linux-arm -l ~/vsdbg?
It's from https://vsdebugger.azureedge.net/vsdbg-15-1-11120-1/vsdbg-linux-arm.zip.
I could not get it going on arm64 with .NET Core 2.0.

@IvanFarkas arm is now supported, but NOT arm64. I opened https://github.com/OmniSharp/omnisharp-vscode/issues/2105 to track arm64 support.

I need to be able to run and debug .NET Core 2.1 C# console app on arm64v8.
DragonBoard 820c is a powerful single board computer to be used as an Azure IoT Edge hub.
We need to be able to run and debug .NET Core 2.1 C# console app on arm64v8 running in a Docker container.
Let's hope that this will be achieved by .NET Core 2.1 RTM!

I opened ticket #1337

How Microsoft expect .NET Core 2.1 to be popular for any IoT app if it does not run on arm64v8 architecture?
The success of Azure IoT Edge depends on this!
We MUST be able to Run and Remote Debug an Azure IoT Edge module that is a .NET Core 2.1 C# Docker app on an arm64v8 architecture like the DragonBoard 410c or DragonBoard 820c.
Is this the next Microsoft Bob?


parallelization issues with prepOrbidata.r

According to https://github.com/rietho/IPO/issues/44 and  https://github.com/rietho/IPO/issues/49 there are new updates to xcms that change the way how multicore processing calls are handled. The result is, while using the latest xcms package, warnings are thrown out and the script defaults to single-threaded processing, effectively slowing it down. Helen reported having a 12-hour IPO optimization run that didnt finish as a result of this. I'm trying to test a solution which is tricky since IPO and xcms now use different calls for multithreading.

Another thing to note, this change completely breaks multithreading on MSWindows as Biocparallel does not support multicore processes that OS.


Modbus coil support

Add coil support for Modbus protocol

Looks like flake8 and pylint both returned some messages. You'll need to address these before this can be pulled. You can run these checks locally with the check_style script in the scripts directory.
Also, please change this pull request to be against the dev branch rather than the master.

Your updates look good, just a few style fixes that were caught by the style checker.
 The properties state_attributes in both the sensor and switch modbus.py files are completely superfluous. Could you remove those? Also, in the sensor, the update function is missing a docstring. In the switch, the turn_on, turn_off, str, and update functions are missing docstrings. Would you mind adding some? For the life of me, I have no idea why that wasn't flagged by the style checker. Thanks.
Other than that, everything looks pretty good to me.

Don't forget to switch your pull request to be against the dev branch of Home Assistant. This is the branch you will want to sync with.

To  switch it you will have to close this pull request and start a new one.
On Sun, Aug 2, 2015, 21:04 Ryan Kraus<EMAIL_ADDRESS>wrote:

Don't forget to switch your pull request to be against the dev branch of
Home Assistant. This is the branch you will want to sync with.
—
Reply to this email directly or view it on GitHub
https://github.com/balloob/home-assistant/pull/233#issuecomment-127055699
.


Update compatible tedious versions

Tedious v17.0.0 was recently released. It dropped node 16 and 19 support

Great - merged into 0.14.0!


Update ios klarna mobile sdk

Update from 2.2.2 to 2.5.1

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: alexandra.damaschin:x: agrapinealexandra.damaschin 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.


SegmentContextExecutor doesn't have reference to currentSubsegment

Using aws-xray-recorder-sdk-core:2.8.0 for manual instrumentation, experienced/recreated issue locally on Java 8 inside a JUnit Test.
The issue I'm experiencing is the SegmentContextExecutor propagates the correct Segment to a CompletableFuture, but when I run AWSXRay.beginSubsegment() inside the entity.run() runnable, its parent is the current Segment, not the current Subsegment. Similarly, when I run AWSXRay.getCurrentSubsegment(), I receive an exception.
When using a local debugger to troubleshoot I noticed the SegmentContextExecutor provides the current Trace Entity to my async function, but loses reference to the currentSubsegment, even executing inside the Entity Runnable: AWSXray.getTraceEntity().run(...).
I can tell the segment is made available, since AWSXray.getTraceEntity() returns the correct Segment, however, if there's an ongoing Subsegment, I can't get a reference to it using AWSXray.getCurrentSubsegment(). I receive a "com.amazonaws.xray.exceptions.SubsegmentNotFoundException: No subsegment in progress." exception. However, when examining the Entity/segment in the debugger, I can see it has a Subsegment marked as in progress - both through inspection and AWSXRay.getTraceEntity().getSubsegments().get(0).isInProgress().
The example below prints the exception I mentioned above.
    @Test
    void testNativeCompletableFuture() {
        AWSXRay.beginSegment("Test");
        AWSXRay.beginSubsegment("Test Subsegment");
        try {
            CompletableFuture.runAsync(() -> AWSXRay.getTraceEntity().run(() -> AWSXRay.getCurrentSubsegment()), SegmentContextExecutors.newSegmentContextExecutor()).get();
        } catch (InterruptedException e) {
            e.printStackTrace();
        } catch (ExecutionException e) {
            e.printStackTrace();
        }
        AWSXRay.endSubsegment();
        AWSXRay.endSegment();
    }

I would expect the currentSubsegment to be available when the referenced recorder is propagated to the executor, so when AWSXRay.beginSubsegment() is called inside the entity.run() runnable, its parent is the current subsegment.

Hi @dmccorm2 - yeah it seems like SegmentContextExecutors could have been just based on the current Entity instead of Segment which is causing your issue. I think the current workaround would be to treat the entity as an executor using a lambda
    @Test
    void testNativeCompletableFuture() {
        AWSXRay.beginSegment("Test");
        AWSXRay.beginSubsegment("Test Subsegment");
        Entity entity = AWSXray.getTraceEntity();
        try {
            CompletableFuture.runAsync(() -> AWSXRay.getTraceEntity().run(() -> AWSXRay.getCurrentSubsegment()), entity::run).get();
        } catch (InterruptedException e) {
            e.printStackTrace();
        } catch (ExecutionException e) {
            e.printStackTrace();
        }
        AWSXRay.endSubsegment();
        AWSXRay.endSegment();
    }

Is this a reasonable workaround for now? Going forward, we need to overhaul our context APIs as they're pretty confusing right now.


[Bug]: When copy pasting Modal Widget - Not able to see the newly pasted widget focused

Is there an existing issue for this?

[X] I have searched the existing issues

Description
When we D&D a Modal widget, copy it and paste in the same page, we are able to see the widget gets pasted but not focused.
Steps To Reproduce

Drag and drop and Modal Widget
Copy the Modal Widget 1, Paste it in the same page (Modal Widget1 Copy)
We are able to see Modal Widget 1 copy in the side bar also in property pane but not in the canvas focused.

https://www.loom.com/share/03ebffdac89142ecaba31600186585dd
Public Sample App
No response
Version
Release

Tested and found to be working fine.


PHP backend example

Why the PHP example was removed?
I have found some dead links pointing to:
https://github.com/bleenco/ngx-uploader#backend-example-using-plain-php
How the PHP backend should look like?
Could you show some JSON responses at least?
Is this a streamed version or multipart? How the memory limits are managed for big files (eg. 1GB)?

Hi @taat, I'm not a php guy at all and I'm since 4.x onboard. So I can't answer the question, why the example was removed. But I think, it's version related.
ngx-uploader uploads the files as multipart upload.
For the php part, look at W3Schools or ask your question on Stack Overflow


ICAL generator not handling color array

PHP Fatal error:  Uncaught TypeError: LitMessages::ParseColorString(): Argument #1 ($string) must be of type string, array given, called in /var/www/vhosts/johnromanodorazio.com/httpdocs/LiturgicalCalendar/api/dev/includes/LitCalAPI.php on line 2348

fixed in commit 2bce5e3


Any example how to send "Simple Select", "Mutli Select" values for the product?

Any example how to send "Simple Select", "Mutli Select" values for the product and how to send null scope?

resolved


Postgres connection timeout

📑 I have found these related issues/pull requests
Tried to setup PostgreSQL with custom port but getting connection timout
postgresql://root:root@<IP_ADDRESS>:4743/postgres
postgres://root:root@<IP_ADDRESS>:4743/postgres

🛡 Security Policy

[X] I agree to have read this project Security Policy

Description
No response
👟 Reproduction steps
Add PostgreSQL
👀 Expected behavior
Successful ping
😓 Actual Behavior
connect ETIMEDOUT <IP_ADDRESS>:4743
🐻 Uptime-Kuma Version
latest docker
💻 Operating System and Arch
debian
🌐 Browser
firefox
🖥 Deployment Environment

Runtime: docker
Database: -
Filesystem used to store the database on:  -
number of monitors:  -

📝 Relevant log output
No response

Could you add details about your environment and what you are trying to monitor?
You forgot to fill out most of the form which makes it hard to provide actionable help.
In particular: what docker-network are you using? Where is the posters container in relation to uptime Kuma?

I try to monitor postgresql database using these DSN's (redacted host) with custom postgresql port. But gitea can't reach the database and throws connect ETIMEDOUT <ip>:4743 error
postgresql://root:root@<ip>:4743/postgres
postgres://root:root@<ip>:4743/postgres

But gitea can't reach the database and throws connect ETIMEDOUT <ip>:4743 error

This is Uptime Kuma, not gitea.
Also, you did not answer the questions I asked of you:

Could you add details about your environment and what you are trying to monitor?
You forgot to fill out most of the form which makes it hard to provide actionable help.
In particular:

what docker-network are you using?
Where is the postgres container in relation to uptime Kuma?

[!TIP]
In case you did not know: docker has more than one network type with only some of them allowing access to the local network and some not even allowing access to remote networks

what docker-network are you using?

On the same host, all the docker commands are above. MariaDB works with Uptime Kuma mysql://root:root@example.com:4742 but postgres doesn't postgresql://root:root@example.com:4743/postgres

Not sure if this is the cause but it may have to do with the listening address you have configured for the postgres instance.
https://stackoverflow.com/questions/48341164/how-to-make-postgres-docker-accessible-for-any-ip-remotely

I have tried to reproduce your example from above and Uptime Kuma and MariaDB cannot communicate.
=> please provide further details about your environment and how to create it reliably (including what you put at example.com in the connection strings)
I think this is because of your docker-network configuration and that the -p.
When I put them into the same network, everything works as expected:
version: '3.8'

services:
  uptime-kuma:
    image: louislam/uptime-kuma:latest
    ports:
      - "<IP_ADDRESS>:3001:3001"
  mariadb:
    image: mariadb:latest
    ports:
      - "4742:3306"
    environment:
      - MARIADB_ROOT_PASSWORD=root
  postgres:
    image: postgres
    ports:
      - "4743:5432"
    environment:
      - POSTGRES_USER=root
      - POSTGRES_PASSWORD=root

I found out that ufw was the problem. Postgres port wasn't allowed in firewall. Kuma and postgres are running on the same host. But I used a domain name in dsn for postgresql. However, for some reason, I can connect to postgres in dbeaver from by home.


Force the usage of wait for completion in the test suite

Pull Request
Related issue
Fixes #<issue_number>
What does this PR do?

...

PR checklist
Please check if your PR fulfills the following requirements:

[ ] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
[ ] Have you read the contributing guidelines?
[ ] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

Closing as non planned


i got error provider.

Error: [$injector:unpr] Unknown provider: eProvider <- e <- bingMapDirective
please help me.

Sorry for the late response.  For example, are you using any sort of minification / uglification / obfuscation tools in your build process? You are the second person to report this issue and I am having a hard time reproducing it.

This should hopefully be addressed with dc33a1063.
Closing issue for now until someone reports otherwise. This change will be available in any version after 0.2.1


Document the language

The syntax, now that it's pretty nailed down, needs some form of documentation.

I want to wait with this until we have a more stable language. If there's one thing I dislike more than writing documentation - it's updating documentation.

Alternatives discussed:

https://asciidoctor.org/
https://github.com/rust-lang/mdBook

Also, we (at least me) are starting to feel more confident in the language stability.


Window.setTimeout and Window.clearTimeout methods are missing

Maybe I am not up-to-speed with the specs but I am missing the setTimeout and clearTimeout methods in elemental. Are they left out with a specific reason ? Should I use something else ?

The method we use are elemental2.dom.DomGlobal.setTimeout(...) and elemental2.dom.DomGlobal.clearTimeout(...) and they are available in 1.0.0-RC1 version at least.
HTH

Ah thanks, I was looking in the wrong place!


draw_texture() scale calculation contains incorrect behaviour.

When drawing a texture using DrawHandle::draw_texture(), the scale provided in DrawTextureParams is multiplied by the source texture (tex).
params.scale.x * tex.width() as f32,
params.scale.y * tex.height() as f32,

(src/drawing.rs:388)
Since scale is the ratio between the source texture and the destination texture, this behavior is incorrect. For example, my use-case is drawing a single tile from a tile atlas. The total size of the tile atlas should not have a bearing on the sprite's size on the destination texture.
This behavior should indicate the ratio between the source texture and the destination texture — scale = destination /  source.
For example, for a 16x16px area from the source texture being transformed onto a 72x72px area, scale should be 4.5x. This is far more intuitive than the current implementation.

I see the problem. I imagine it was just some sloppy work on my part when I was adding DrawTextureParams, because this is definitely not intended behavior.
Posting the fix here for now. I'll see if there are any more problems with how I use DrawTextureParams and then upload a new version.
let source = params.source.unwrap_or(Rectangle::new(0., 0., tex.width() as _, tex.height() as _));

unsafe {
    ffi::DrawTexturePro(
        tex.raw.clone(),
        source.into(),
        Rectangle::new(
            position.x,
            position.y,
            params.scale.x * source.width as f32,
            params.scale.y * source.height as f32,
        ).into(),
        params.origin.into(),
        params.rotation,
        params.tint.into(),
    )
}

Thanks so much! I made my own fix in the meantime, so no rush.


SQL query statement about case ?

OrientDB Version: 2.2.20
Java Version: jdk8
OS: centos7
SQL statement in about
select count(case when c1 = 1 then 1 else NULL end) as id1, count(case when c2 = 2 then 1 else NULL end) as id2 from table
In OrientDB, How to handle the 'case when' statement ?
Thanks.

Hi @shushengxing
There is no equivalent in OrientDB, but you can do something similar with an IF statement, eg.
SELECT sum(id1) as id1, sum(id2) as  id2 FROM (
    SELECT if(eval(" c1 = 1 "), 1 , 0) as id1, if(eval(" c2 = 2 "), 1, 0)) as id2
    FROM table
)

I hope it helps
Thanks
Luigi

Oh, I see. I will try.
Thank you, @luigidellaquila .

Hi @shushengxing
I'm closing this issue, please feel free to re-open if you need further info
Thanks
Luigi


advi should output samples at last iteration after being interrupted (ctrl-c)

john reid asks

Also would it be possible to have cmdstan output samples from the best iteration so far if it is interrupted (Ctrl-C)?

Yes, it is possible.
There is an interrupt callback that is supposed to be called at every iteration. For CmdStan, this is currently a noop, but that could change.
But, the advi code needs to be cleaned up and brought up to the rest of Stan's standards in order for this to work.
Once the issue Dustin created is fixed, this should be a lot easier to do.

On Nov 3, 2015, at 6:54 AM, Alp Kucukelbir<EMAIL_ADDRESS>wrote:
john reid asks
Also would it be possible to have cmdstan output samples from the best iteration so far if it is interrupted (Ctrl-C)?
—
Reply to this email directly or view it on GitHub.

roger that. this would be super useful.

You mean not stop when you interrupt it, but continue to work?  That
seems counter to the way interrupts are supposed to work.  If it's just
a super-fast cleanup or dump that's OK, but if you output samples and there
are lots of parameters, that can be a lot of work.

Bob

On Nov 3, 2015, at 6:54 AM, Alp Kucukelbir<EMAIL_ADDRESS>wrote:
john reid asks
Also would it be possible to have cmdstan output samples from the best iteration so far if it is interrupted (Ctrl-C)?
—
Reply to this email directly or view it on GitHub.

just a super-fast dump of the mean parameters of q (the variational
approximation).
typically, these parameters are more than enough to diagnose "what's going
wrong"; which is a whole lot better than what ctrl-c does right now (no
output).
On Tue, Nov 3, 2015 at 2:31 PM, Bob Carpenter<EMAIL_ADDRESS>wrote:

You mean not stop when you interrupt it, but continue to work? That
seems counter to the way interrupts are supposed to work. If it's just
a super-fast cleanup or dump that's OK, but if you output samples and there
are lots of parameters, that can be a lot of work.

Bob

On Nov 3, 2015, at 6:54 AM, Alp Kucukelbir<EMAIL_ADDRESS>wrote:
john reid asks
Also would it be possible to have cmdstan output samples from the best
iteration so far if it is interrupted (Ctrl-C)?
—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/stan-dev/stan/issues/1663#issuecomment-153463155.

OK --- I was just taking "sample" literally.   A parameter
dump should be fine even for lots of params.

Bob

On Nov 3, 2015, at 4:04 PM, Alp Kucukelbir<EMAIL_ADDRESS>wrote:
just a super-fast dump of the mean parameters of q (the variational
approximation).
typically, these parameters are more than enough to diagnose "what's going
wrong"; which is a whole lot better than what ctrl-c does right now (no
output).
On Tue, Nov 3, 2015 at 2:31 PM, Bob Carpenter<EMAIL_ADDRESS>wrote:

You mean not stop when you interrupt it, but continue to work? That
seems counter to the way interrupts are supposed to work. If it's just
a super-fast cleanup or dump that's OK, but if you output samples and there
are lots of parameters, that can be a lot of work.

Bob

On Nov 3, 2015, at 6:54 AM, Alp Kucukelbir<EMAIL_ADDRESS>wrote:
john reid asks
Also would it be possible to have cmdstan output samples from the best
iteration so far if it is interrupted (Ctrl-C)?
—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/stan-dev/stan/issues/1663#issuecomment-153463155.

—
Reply to this email directly or view it on GitHub.

@dustinvtran, if you're going to work on this, branch from feature/issue-1751-service-methods until it gets merged.


Performance after 1.0 release on OS X 10.10.3

Since I installed the 1.0 release of Atom the performance has been absolutely atrocious.  I don't mean "a bit slower than before" but tremendous slow-down/lag when typing, accessing menus, etc.  I tried re-installing Atom (yes, I cleaned out my .atom directory) but it did not fix the issue.
Has anyone else experienced similar problems?

Sorry, this had to do with updates.  They were just taking forever.


Question: which version of Python was used?

Hi,
I would love to do some experiments with this fascinating work, but I'm having trouble getting the code to run.
At first I tried on a MacBook Pro M1, using Python 3.9, which I expected to be a challenge. It went better than expected, but ran into trouble with the open3d and python-opencv packages (no compatible package found).
Ultimately I switched to the Rosetta (i386) emulation on the same computer, using Python 3.8, but still no luck with Open3D. Then I read somewhere that using Python 3.7.7 should work, but this resulted in a humongous log of compiler errors and warnings. Since I'm pretty new to Python, can you say which version you used at the time of creation? And which operating system and version?

Seems to be mostly about Open3D with M1, I suggest you raise an issue with Open3D.
I used/tested it with a MacBook with an Intel i7, macOS Catalina.

That's a valid point, thanks.
In the meantime -good news- I've gotten the code to work on Windows 10, using the Windows Linux Subsystem feature https://seanthegeek.net/234/graphical-linux-applications-bash-ubuntu-windows/.  I wasn't able to get the CUDA driver to work according to this instruction https://docs.nvidia.com/cuda/wsl-user-guide/index.html#installing-nvidia-drivers, but did manage to convince pytorch to only use the CPU.
To others who may be interested: after a lot of thrashing about, there's a good chance that the only lines that really need to be changed are these two at the top of config.py, just change "cuda" to "cpu":
train_device = "cuda:0"
eval_device = "cuda:0"

So my next step is to try it with my own SfM datasets.
And before I forget: thanks for making this available, it is a really very impressive work!
This issue can be closed for what I'm concerned.

I installed python==3.7 torch 1.7.0 and corresponde version of torch-scatter and so on.


test pr

[One line description of your change]
Motivation:
[Explain here the context, and why you're making that change. What is the problem you're trying to solve.]
Modifications:
[Describe the modifications you've done.]
Result:
[After your change, what will change.]

@swift-nio-bot test api please

@swift-nio-bot test api please

@swift-nio-bot test this please

@swift-nio-bot test this please

@swift-nio-bot test this please

@tomer you're the best. This is awesome!!
03:15:42 Checking _NIO1APIShims.json... OK
03:15:42 Checking NIOConcurrencyHelpers.json... OK
03:15:42 Checking NIOFoundationCompat.json... OK
03:15:42 Checking NIOHTTP1.json... OK
03:15:42 Checking NIO.json... OK
03:15:43 Checking NIOTestUtils.json... OK
03:15:43 Checking NIOTLS.json... OK
03:15:43 Checking NIOWebSocket.json... ERROR
03:15:43 ==============================
03:15:43 ERROR: public API change in NIOWebSocket.json
03:15:43 ==============================
03:15:43
03:15:43 /* Generic Signature Changes */
03:15:43
03:15:43 /* RawRepresentable Changes */
03:15:43
03:15:43 /* Removed Decls */
03:15:43 EnumElement WebSocketErrorCode.unexpectedServerError has been removed
03:15:43
03:15:43 /* Moved Decls */
03:15:43
03:15:43 /* Renamed Decls */
03:15:43
03:15:43 /* Type Changes */
03:15:43
03:15:43 /* Decl Attribute changes */
03:15:43
03:15:43 /* Fixed-layout Type Changes */
03:15:43
03:15:43 /* Protocol Conformance Change */
03:15:43
03:15:43 /* Protocol Requirement Change */
03:15:43
03:15:43 /* Class Inheritance Change */
03:15:43 done

it totally noticed the API breakage :)
Also CC @xwu, this is really cool


Skip leading empty ranges in roundRobin

Previously, roundRobin would always attempt to fetch its first element from the first range passed to it, even if that range was empty.
Fixes bugzilla issue 24384.

Assuming this is ok to merge since it's functionally identical to the approved version and the tests pass.


feat(help): Improve output of the help text to show nested scripts

What:
The output for the help command has been updated to show nested format

Why:
To make it clear to users which scripts are children of other scripts

How:

Summary: Render out the script tasks as a tree structure, uses lib 'ascii-tree'
The existing function to getAvailableScripts was updated to include a parameter to indicate the indent level of a task, as the function recurses the level is incremented.
Using 'ascii-tree' library we use a special character '#' to indicate indent level.
Closes #73

Current coverage is 100% (diff: 100%)

Merging #75 into master will not change coverage

@@           master   #75   diff @@
===================================
  Files          11    11
  Lines         261   265     +4
  Methods         0     0
  Messages        0     0
  Branches        0     0
===================================
+ Hits          261   265     +4
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update 0cb76a9...f519b84

@kentcdodds I have quite a few scripts, with 2 or 3 subscripts. I can see the benefit of having them indented, to quickly see the "main" scripts and their subscripts. But at the same time, we are repeating the full command (for clarity) so I don't know if it give a huge benefit after all.
Would it be possible to have an example in a terminal? I can fetch the PR and test against one of my project too if you don't have any test project @gunnx

@tleunen Yes please, my project doesn't have much more then the p-s project itself.

I am ambivalent here. It may be easier to see how commands relate at the cost of potentially soft-wrapping.

@kentcdodds I should have said as well this is my first PR as I know you are involved in support that.

Let me try this out on my work project that has a lot of scripts and see how it looks.

Alrighty! I've published a beta version of your changes and people can try it out by installing p-s@beta. Looks pretty good, but I think I've found a bug:

You'll see that the watch scripts almost look like they're coming out of the flow script. But in reality there's just not a watch script or a watch.test script for that matter. So we'd want to de-indent those I think.

@kentcdodds Are you able to post the package-scripts file too? Just wondering why you'd have sub-sub tasks without any parent tasks?

I'm experiencing the same thing that @kentcdodds reported.

It seems it happens when tasks don't have any script @gunnx

@tleunen Hi what does that look like in the package-scripts file?

Like this:

The result of the getCopyImgCmd is the command executed.

@tleunen Yes thanks, I can see that resolveScriptObjectToString will return undefined if it can't find a script. Is this expected behaviour? Should script be made to be required if you define it as an object

Should script be made to be required if you define it as an object

No, I think that @tleunen's usage is totally valid and we should probably make that look good in this output.

@kentcdodds Ok I think I have a way, so are you saying if there is a nested script but it has no parent script then it should not be indented.

Hey @gunnx, I've been using this for a little while, and I'm not convinced that it improves the readability of the output... Especially when wrapping the text. I don't think that I'm going to merge this one. I'm really sorry about that. If you want to give your hand in another PR on the repo, I'll definitely accept a PR that fixes this but you reported: https://github.com/kentcdodds/p-s/issues/74

Ok sure, I'll take a look at the other issue

Again, I'm sorry your first pull request didn't work out.  This was a great pull request! You can do great things in open source 😀

No problem, it was good to go through the process :)
On 3 November 2016 at 13:59, Kent C. Dodds<EMAIL_ADDRESS>wrote:

Again, I'm sorry your first pull request didn't work out.  This was a great pull request! You can do great things in open source
😀
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kentcdodds/p-s/pull/75#issuecomment-258149785, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AB4Pj-_Pi7u6eVY40B9cUDsyS3BdAQQVks5q6ei_gaJpZM4Kiy6Z
.


[FR] generate sdist that wont need cython

projects like borgbacup that for various reasons can't ship manylinux wheels (like the need for libs outside of manylinux)
it would be neat if it was possible to create a sdist that does not need cython to build
this may need upstream work in hatch for enablement

projects like borgbacup that for various reasons can't ship manylinux wheels (like the need for libs outside of manylinux)
it would be neat if it was possible to create a sdist that does not need cython to build
this may need upstream work in hatch for enablement

Good idea, this wouldn't be too difficult IMO - I think the plug-in could be adapted for sdist to compile (cython) & package the c/cpp headers. I took a quick look at the setup.py for borg & that appears to be the approach used.
Will add as test feature in next release

Hey @RonnyPfannschmidt, a few thoughts about this -
I've gone ahead and done some testing & in theory, the c/cpp files can be generated & e.g. separated within sublevels with a bootstrap setup.py. Where I think this gets messy is that this plugin supports multiple platform & arch specific filters on e.g. flags, templated kwargs, etc. So, since pip install xxxlib already installs hatch-cython if it is defined in build-dependencies, & thus Cython. As such, during sdist builds it seems redundant to add this in when the arguments are already passed both to Cython & the c/++ compiler. I would say it is more ideal to leave the raw source files in the sdist, and allow the target platform to follow the platform & arch disposition through this plugin. A similar feature I may add instead, is the ability to perform platform & arch specific file ignores. Thus, at build time the writer may optionally compile e.g. windows vs linux specific files. From my understanding of the project used as an example, this would have a more consistent effect than playing with the sdist. Let me know your thoughts, I'll keep this open for a bit in case someone stumbles on this & has differing opinions

The primary details for this idea is to reduce the build dependency closure for packagers
It's indeed a bit contrary to build time conditions


Add documentation to VisibleEntities and related

Objective
Add missing docs
Solution
Add documentation to the VisibleEntities component, its related
check_visibility() system, and that system's label.
See Discord discussion here : https://discord.com/channels/691052431525675048/866787577687310356/990432663921901678

I've deleted the link. The clippy issue was the missing : between the link and its target (so the link was effectively malformed and ignored). But after that the link becomes broken. I think I had already issues like this in the past, when trying to link to another crate. I can't remember the details, but I think I had pinged the rustdoc folks and there was no good solution. Part of the issue is if I remember correctly that the sprite crate is bevy_sprite or bevy::sprite depending on context, and that confuses rustdoc. I can't remember all the details. Anyway, that works now.
Ping @alice-i-cecile for merge, pretty please 😉

bors r+


[sui-core] Infra to restart and stop active services

Initial infrastructure that allows starting, restarting and exiting active components, for two key purposes:

clean shutdown.
reloading the config / committee upon epoch change.

Many thanks for the review @bmwill ! And for keeping me honest on handling edge cases and errors.


[no ticket] fix: withdraw registration via pending_withdrawal

Never set withdrawn directly, let the backend handle it.

Purpose
Front-end follow-up to PLAT-1333

Summary of Changes
When the user triggers a withdrawal, set pending_withdrawal: true instead of withdrawn: true, since the withdrawal has to be confirmed before it's actually withdrawn.

Side Effects
n/a

Feature Flags
ember_registries_detail_page

QA Notes
Withdrawal should work.

Ticket

n/a
Reviewer Checklist

[ ] meets requirements
[ ] easy to understand
[ ] DRY
[ ] testable and includes test(s)
[ ] changes described in CHANGELOG.md

😱 type error!


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

@microsoft-github-policy-service agree [company="{your company}"]

This pipeline configuration builds the application and produces a build for the Release configuration.


dataset_description required for derivatives

Looks like #168 requires a dataset_description.json file to be required for all folders passed to BIDSLayout even those that may not be full BIDS datasets. e.g.:
layout = BIDSLayout(['/path/to/bids/', '/path/to/more/event/files')

This seems like unintentional behavior to me, as roots for derivatives should not require such a description, correct?

Good point. We should probably add a requires_description flag to the config files that's False by default and True only for the BIDS config. This would all go in grabbids rather than grabbit. Feel free to open a PR, otherwise I'll get to it probably tomorrow.

I got it


behnaz/gifs-affiliate

there are 14 gif files, marketing team asked to share with affiliates

two pr's mixed, so closed


sync versions between environments

Is it expected to have v3.0 also onprem?
https://github.com/opendevstack/ods-project-quickstarters/blob/b73bb4a1b9afc6974ca7b055fe1cc7e496a9a741/ocp-templates/templates/rshiny-app.yml#L144

Yes .. See ods-core / shared Images / ocp config/ bc.yml


sciond: Handle ISD-local AS-wildcard path requests

Fixes #2282

This change is

go/sciond/internal/fetcher/fetcher.go, line 523 at r2 (raw file):
Previously, scrye (Sergiu Costea) wrote…
Now that the code below is reasonably complex, it might be useful to make dsts be initialized only once (right now it's initialized above for the unindented code path, and re-initialized below with make inside the if, which is a bit confusing).
Or maybe something like:
var dsts map[addr.IA]struct{}
if !isWildcardDestination {
    dsts = chooseDestinations(listOfCores, isLocalISD)
} else {
    dsts = reg.Dst.IA()
}

Even some micro-functions might help with readability a bit.

The branches above should be the other way around, btw.


homekit controller in 0.68.0 fails on hass.io

with homekit-controler enabled per the docs you get this error on statup.
Unable to install package homekit==0.6: Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-0_5lntts/cffi/

Additional information:

+1 same problem. Hassio on Raspberry Pi 3

I’m having the same issue.  0.68.1 did not I fix the issue.

I also experienced the same issue as this on Hassio 0.68 and after upgrading to 0.68.1 as per our discussion here:  Issues with HomeKit
(SyncWorker_6) [homeassistant.util.package] Unable to install package homekit==0.6: Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-kedmi3ai/cffi/ ERROR (MainThread) [homeassistant.requirements] Not initializing homekit_controller because could not install requirement homekit==0.6 ERROR (MainThread) [homeassistant.setup] Setup failed for homekit_controller: Could not install all requirements.
I see the following on the dashboard:
`Invalid Config
The following components and platforms could not be set up:
homekit-controller
Please check your config.`

Fix with next Home-Assistant Update

https://github.com/home-assistant/hassio-build/commit/76a13dbf373bd04c1bceefe8a27ad8997f47226a


Process unstructured data sources

These contain valuable data nuggets among an ocean of junk and we need to be able to find the good things there.
Some sources are:

mailing lists such as #100 #104 #108
changelogs #233
reflogs of commit (see also the commits from vulncodedb and SAP/Eclipse steady KB)
bug and issue trackers (such as Django, etc)

We can either automate it all, but that's going to be super difficult, or rather start to craft a curation queue and parse as much as we can to make it easy to curate by humans (see #218 ) and progressively also improve some mini AI and classification to help further automate the work.

A reference: https://hal.science/hal-03430826/document

Interested in the Project Idea...

Interested in the Project Idea...
I think, processing of changelogs, reflogs of commits and mailing list data can be a automated

Please also check: https://github.com/cve-search/git-vuln-finder

https://github.com/pyupio/changelogs

I guess the process of change logs of Apache mailing list can be automated using OpenAI' API, where we scrape the data using Selenium, feed into LLM, get the output as json format and then update the database accordingly. What is your view on that @pombredanne . #218  Can also be implemented.

Automating the extraction of valuable information from Apache mailing list changelogs using OpenAI’s API and other tools is a great initiative and I think for the unstructured data we can focus primarily onto the Dataset for feature Engineering and classified into diverse group
Model Training: Fine-tune the selected model on a prepared dataset of CVEs in code. This will help the model learn to identify vulnerabilities in the unstructured data..... As well as we can use LoRA for the model to train
Vulnerability Detection: Use the trained model to parse through the unstructured data and identify potential vulnerabilities. This could involve using NLP techniques to understand the vulnerability descriptions and infer the vulnerable package name and versions.
Most Important Parameter to be checked is this
Text Classification: This involves categorizing text into predefined groups. [In vulnerability detection, this could be used to classify descriptions as either indicating a vulnerability or not
Information Extraction: This is the process of automatically extracting structured information from unstructured text data.


[QUESTION] How to select a document by its Dictionary<string, string> key and value?

I have the following Foo class with a dictionary property List:
public class Foo
{
public string Name { get; set; }
public Dictionary<string, string> List { get; set; }
}
My question is how to select (retrieve) all (or single) Foo whose List (dictionary) contains the key='k1' and its value is 'v1'?
In DB I have a record of Foo with List='{"k1":"v1","someOtherKey":"1_1"}'
Tried this one but FindOne always returns NULL:
var doc = db.GetCollection();
var result = doc.FindOne("List[*].k1 any in @0", "1");
Also, tried anything possible with no luck. My guess is that the dictionary is the same class's object so filtering by its known key should work.

Morning cup of coffee brought a new crazy idea that works ))
Solution: doc.FindOne("List.k1 = '1'");
I.e. think about that stored dictionary like a complex object with keys as properties.


Updates libs and build tools to the last stable versions

This fix resolves an incompatibility with API and build tools v28.
There were switched support libraries to androidx.* namespace, updated the gradle plugin and build tools versions.

Thanks for your contribution!

Thanks a lot for your contribution.


New-TenantAllowBlockListItems: A parameter cannot be found that matches parameter name 'Allow'

A prior issue #8258 was opened for problems with the -Allow parameter. It was suggested to use -Allow instead of -Action Allow.
-Allow does not work either.
This is a pretty critical cmdlet to have working because of the Secure By Default changes that are rolling out to tenants now. With this command not working, this creates a much higher amount of manual work.

Hi @beercity, thank you for your feedback.
Allow parameter worked for me:

But please have in mind that this cmdlet is still in preview, please check the note at the begging of the article.
Thank you again!

@dariomws Thank you very much for the contribution and sharing this explanation. @beercity Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Questions about Constructing kittival Datasets and Modifying Parameters in Cross-Validation

Hello. Thank you for sharing your work. It is a very advanced method and I am interested in your work.
I was trying to run the code and have two questions about cross-validation. I am a beginner, so I apologize for being off the mark with my question.
•Do I need to make the kittival dataset by myself? Or does your program include a step to divide the training data of kitti into 5 parts? I understand how you divide the images from another closed issue. However, I would like to know whether this step is included in the program.
•I managed to get the results, but I don't seem to have been able to modify the parameters properly in the validation step.
What I did:
Step1.  Set the training state as True and the dataset_name to "kittival", and the data_path to /xxx/data/kittival/exp1. Run LFDRoadseg_main.py.
Step2. Change the training state to False. Set the validating state as True and the dataset_name to "kittival", and the data_path to /xxx/data/kittival/exp1. Run LFDRoadseg_main.py.
Step3. Repeat step1 and step2 for exp2 to 5.
Step4. Change the validating state to False. Set the testing state as True and set the dataset_name to "kitti", and the data_path to /xxx/data/kitti. Run LFDRoadseg_main.py.
If there is a wrong process,  could you please tell me?
I apologize for asking many questions, and I would greatly appreciate your answers. Again, thank you for sharing your great work.

I am sorry for my late response.
There are no further questions at this time.
Thank you for your support.

I have a question here.
How ckpt_path and vis_path are set in the outputs of your configs file as you repeat steps 1-3. Because when I validating an exp1, I go on to complete exp2 training and validating. I found that this error occurs when validating.


nodejs

If this is a request for a label, then I'd say this is a no as labels should not be related directly to a language


Integration Test - Release - 2019/02/21 08:46:22

Auto create for integration test. IssueCommandFlow

#assign PRMergerTest1


Allow to pass an account to script functions

We should allow to pass an account to scripts functions, that will be used to run those functions. A design doc should be first made to propose an interface.

Hi, i would love to work on it, but i need more clarity.

Hi @vibenedict , thanks for your interest in the task. This one, however, is probably a bit too complex for a newcomer, I'm afraid (it requires a design document first, and generally is not clearly defined atm). I know you were interested on other issues as well, so I think we will fid something more suitable :)


Twig extension coreextension set escape deprecated

see https://github.com/timber/timber/pull/2062
This adds a check to see if the new class exists in Twig

wrong base branch, my bad :/


[English] Remote Procedure Call(RPC)

Add Remote Procedure Call(RPC) to the glossary

I would like to work on it.

@Alt-Shivam, you can only be assigned one term at a time. Please let me know which one I should assign to you. Also, not sure if I mentioned it before, but please make sure to read the updated Style Guide. We added a lot more info.

Thanks for this info @CathPag
I will take the TLS one.

@Alt-Shivam do you still want this  issue? Happy to assign it.

Would love to
But I am on Chaos Engineering #336

Hey Jason can I work on this issue?

Here you go, @Poobalan1210. Please make sure to read how to contribute and the style guide before getting started.

Thanks for the assign Cath.Yep i will see the guidelines

@Poobalan1210, I'll open this issue up again since it's been assigned to you over a month and we haven't seen any PR yet. If you have been working on something and were about to submit, please let me know and I'll reassign it to you.

The maintainers have reviewed this term and determined that it doesn't meet the criteria to be included in the Glossary. I'll go ahead and close this issue.


Multiple ingresses using the same voyager service and ELB

Hi All,
I am quite new to Kubernetes so please excuse my ignorance. I was reading throught this article https://itnext.io/save-on-your-aws-bill-with-kubernetes-ingress-148214a79dcb which explains how to setup nginx using multiple ingress objects but a single ELB. I was wondering if this is something that voyager supports as well?
From my testing, for each ingress object voyager will create a coresponding service (along with the ELB). Would it be possible to have a multiple ingress objects using the same service/ELB?
Thank you

Hi,
I have same need.  I have been looking around and voyager ingress dose not support this future, I hope is just for the moment. How I have been able to do it is to keep a single ingress yaml file and now I am thinking to a processes to maintain that file.
All I hope is this future will become implemented. A cluster with 800-1000 applications an all exposed is not so ok with 1000 ELB on aws.

This is possible. You create your deployments/services as usual then create an ingress resource where you declare the hosts and the backends serving those hosts. You can do this even across namespaces, if voyager is installed without namespace restriction. This means one ELB/NLB and as many hosts as you want.
The service which "serves" your deployment shouldn't be type: Loadbalancer, it should be the default, meaning type: ClusterIP. The ingress resource will request the load balancer, not the service.

@ssro Thank you, I will try it!

I have been testing the deploy of an application using igress.yaml. The ingress is created but when I am deploying the second service to use same ingress is falling:
Error: release xxxxxx failed: ingresses.voyager.appscode.com "xxxx-ingress-voyager" already exists
The idea is to use same chart for deploying multiple services using same ingress/ELB
If am changing the template to a dynamic name like  {{.Release.Name}} this will create a new ingress and implicit a new ELB
With k8s objects dose work but I am trying I want to use helm ingress template.


It would be good if .as_u64() could parse BitStrings into numbers, too

Hello,
I think that it would be useful if .as_u64() could parse BIT STRING values into numbers, too, as BitStrings turns out to be sometimes used as enumerated integer values in ASN.1 definitions.
I will maybe make a pull request for that if the maintainer is OK.

That looks great for the cases the bitstring is indeed the representation of a small integer. I applied #29, thanks (so closing issue)
For other cases, I'm wondering if the bitvec crate would be a good crate to use to abstract values.


Fix original astropy tests

We should make sure that all the original io.fits tests pass

Done in #45 and #47


Critical vulnerability with json-schema-deref > mpath

currently the<EMAIL_ADDRESS>package has a critical vulnerability, they plan to update it?
│| Critical      │| Type confusion                                               │|
│ Package       │ mpath                                                        │
│ Patched in    │ >=0.8.4                                                      │
│ Dependency of │ oas-tools                                                    │
│ Path          │ oas-tools > json-schema-deref > mpath                        │
│ More info     │ https://npmjs.com/advisories/1785   |

Hi @raffrearaUS, any update on this? Thank you.

Hi, this vulnerability has been patched for the released 2.2.0 version. Check #304 .


Provide a hook in init-tools for a bootstrapped cli.

https://github.com/dotnet/source-build/pull/227#discussion_r144298297
We need not only a way to override the entire tools directory but also a way to just override the cli we download.

Closing in favor of https://github.com/dotnet/source-build/issues/334


[EFR32] Add documentation and enable release build

Problem
Lack of documentation with build option
Change overview
Add documentation
Testing
Local testing of the doc

Fast tracking since Platform specific changes


Make spy() output nicer plots by default

A small rewrite of the spy() function to get better defaults. The differences compared to the current one are

the axes now are now compactly around the matrix, whereas previously there
was quite a lot of empty space, depending on the dimensions of the matrix
the aspect ratio of the axes is 1:1 by default, since squares look better
zeroes now get drawn, but NaNs get filtered out (currently zeros leave holes and NaNs make it crash)

Everything can still be overridden by passing a custom Coord or Scale, so I do not think that this forces the user into anything, but these are better defaults if you just want to take a quick look at some arbitrary matrix.
I.e. we go from this

to this

Plots produced by this simple script: https://gist.github.com/mortenpi/24730ed5f1b6de8976a4

Looks great!  I'll merge it in a day or two if no objections are raised.

:+1:


ERROR: Failed to send data!Releasing device interface...

Trying to flash recovery in gt-i9090 and get that error

same error for a samsung S10, anyone found a solution ?

same error


Merge live branch to master

Coverage decreased (-1.6%) to 86.001% when pulling 7c728407759d4867f215bf156f7a4a0c0495b6a0 on tibkiss:merge_live_20170602 into 1fc9f860847c7b6ff412ab437fa2acfeda0fe2c3 on zipline-live:master.


Update index.rst

Updated index at docsrc root to point to main pages for each section

Updated to point to releaseNotes index

Updated index to point to correct files and changed base to master


[Question] How many devices can be managed at most  through yurt-iot-dock in a k8s cluster?

What happened:
What you expected to happen:
Is there any performance report about yurt-iot-dock and k8s api-server when the number of managed devices increases?
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

OpenYurt version:
Kubernetes version (use kubectl version):
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

others
/kind question

@LavenderQAQ PTAL

@zhang943 I'm sorry, we haven't done the performance test yet. The number of devices to access should depend on the access capability of edgex, you can check the documentation of edgex. OpenYurt deploys edgex in a distributed manner, and the increase in access capacity is more than offset by the loss of performance on the management side (you can deploy multiple EdgeXs in multiple nodepools). Moreover, cluster scale and data generated by each device are also different, so it is difficult to directly give a definite value.

@LavenderQAQ Thanks.


Using multiple attributes of a dataset as a single argument

Example plot heading which includes model and variable name,
i.e --title="%model% %experiment%"   ->   --title="ACCESS1-0 rcp45"

This has been implemented as an addition to the ProcessUnit API - the syntax is kw_string="--title ${model} ${experiment}"
See commit 4bec77fac1f7b3d8646e4d6b49f2bf3df6098af4


Request for azurerm_kubernetes_fleet_member resource

What resource do you need?
Terraform Resource Name: azurerm_kubernetes_fleet_member

What is your use case?

We want to use Azure Fleet and the crossplane provider already supports creating a fleet manager, but without the fleet member resource, no clusters can be added to the fleet making it not useful.

Hi @vrabbi,
Thank you for bringing up this. We would appreciate it if you would like to contribute. We are preparing a guide on adding new resources. You can try to add the resource yourself by trying the steps here. The guide is currently under review and if you have any questions, feel free to ask.


Compiler diagnostics only contain the file name

Consider the following diagnostic:
ERROR [core.bal:(394:21,394:48)] invalid list binding pattern: expected an array or a tuple, but found 'wso2/nballerina.types:0.1.0:UniformSubtype'

Note how only the file name is provided in the message. If we are working on a large project with multiple modules, this is not exactly helpful in narrowing down where the error occurred since you don't know which module contains the mentioned file. You have to basically hunt down in each module for that file.

Also see https://github.com/ballerina-platform/ballerina-lang/issues/33909.


refactor: use index appropriately

Description
Moves quite a bit of files in favor of exporting from an index.ts.

Honestly, within a project, I'm not sure if I like the usage of index.ts files. Usually a pattern I only observe in libraries.
I was actually thinking a PR of removing all index.ts files that only consist of export * from './file';

Honestly, within a project, I'm not sure if I like this usage of index.ts files. Usually a pattern I only observe in libraries.
I was actually thinking a PR of removing all index.ts files that only consist of export * from './file';

Not a bad idea. I'm not a TS expert by any means. @andrewmackrodt, have any thoughts?

Closing for now, but will keep for discussion.


How to call another micro-service on GatewayFilterFactory ?

Hello, Can you tell me how to call another micro-service on a GatewayFilterFactory?
I want to call the authentication micro-service when a request come from user.
1st, I tried to use feign client. but I can not get the ServletRequestAttribute on the RequestInterceptor because spring cloud gateway use webflux instead of web.
Here is the code :
public class FeignAuthRequestInterceptor implements RequestInterceptor {

    private final Logger logger = LoggerFactory.getLogger(FeignAuthRequestInterceptor.class);
    private static final String HEADER_TOKEN = "token";
    private static final String HEADER_USERNAME = "username";

    @Override
    public void apply(RequestTemplate requestTemplate) {
        ServletRequestAttributes attributes = (ServletRequestAttributes) RequestContextHolder.getRequestAttributes();
//TODO the attribute is always null so I can not get the original request header
        logger.warn("the request attribute is {}", attributes);
        Map<String, Collection<String>> queries = requestTemplate.queries();
        if (requestTemplate.queries().get(HEADER_TOKEN) != null) {
            requestTemplate.header(HEADER_TOKEN, queries.get(HEADER_TOKEN).iterator().next());
        }
        if (requestTemplate.queries().get(HEADER_USERNAME) != null) {
            requestTemplate.header(HEADER_USERNAME, queries.get(HEADER_USERNAME).iterator().next());
        }
        requestTemplate.queries(queries);
        /**
         HttpServletRequest request = attributes.getRequest();
         Enumeration<String> headerNames = request.getHeaderNames();
         if (headerNames != null) {
         while (headerNames.hasMoreElements()) {
         String name = headerNames.nextElement();
         String values = request.getHeader(name);
         requestTemplate.header(name, values);
         }
         }
         **/
    }
}

Then I tried to use webClient of spring5 to make a request to the authentication micro-service but I can not get the response
Mono<ResponseResult> res = webClient.post().uri(auth.getAuthUri())
                    .headers((httpHeaders -> httpHeaders.putAll(exchange.getRequest().getHeaders())))
                    .contentType(MediaType.APPLICATION_JSON_UTF8)
                    .body(BodyInserters.fromObject(JSONObject.toJSONString(params)))
                    .accept(MediaType.APPLICATION_JSON_UTF8)
                    .retrieve()
//                    .bodyToMono(ResponseResult.class);
                    .bodyToMono(ResponseResult.class)
                    ;

When I use the
res.block();

method to get the response , there is an exception
block()/blockFirst()/blockLast() are blocking, which is not supported in thread reactor-http-nio-2"

So. I want to know how to call another micro-service on spring cloud gateway environment.
Thanks a lot.
best wishes.

Trying to authenticate in a filter is probably not a good idea, for a variety of reasons.  Maybe you need to rethink your architectural decision?

Feign is not going to work as it is not reactive.
This is also why you cannot call .block.
Unfortunately it is hard to say what is wrong without a way to reproduce the problem.  Using WebClient should be fine.

Feign is not going to work as it is not reactive.
This is also why you cannot call .block.
Unfortunately it is hard to say what is wrong without a way to reproduce the problem. Using WebClient should be fine.

I tried to use WebClient ，But I do not know how to get the response from webclient call.  Usually, .block() works. But it does not work well in webflux environment. So would you mind tell me how to get the response when using WebClient ? Thanks a lot .
Best wishes to you .
@ryanjbaxter

See https://github.com/spring-cloud/spring-cloud-security/blob/master/spring-cloud-security/src/main/java/org/springframework/cloud/security/oauth2/gateway/TokenRelayGatewayFilterFactory.java
For an example

Thanks a lot.
I take a  cursory look. But I am sorry, I don't think that's what I need. I deployed an authentication service used shiro in another project.  So I need a way to call that. I want to use WebClient as @ryanjbaxter  said . But I can not get the webclient response  without .block method .
Sorry for the inconvenience.

Take more than a cursory look. The class in the filter could be replaced with webclient. In fact I think if you dig into it, it uses web client.
https://docs.spring.io/spring-boot/docs/current/reference/html/boot-features-webclient.html

Thanks .
And Thank you for your contribution to spring cloud development.

hello spencergibb,
I can not find the source code about the ServerOAuth2AuthorizedClientRepository class.

https://github.com/spring-projects/spring-security/blob/master/oauth2/oauth2-client/src/main/java/org/springframework/security/oauth2/client/web/server/ServerOAuth2AuthorizedClientRepository.java
But it doesn't really matter. This issue isn't to provide a lesson on reactive programming with reactor.

See https://github.com/spring-cloud/spring-cloud-gateway/blob/master/spring-cloud-gateway-core/src/main/java/org/springframework/cloud/gateway/filter/WebClientHttpRoutingFilter.java
And
https://github.com/spring-cloud/spring-cloud-gateway/blob/master/spring-cloud-gateway-core/src/main/java/org/springframework/cloud/gateway/filter/WebClientWriteResponseFilter.java

@vxray
I have the same problem as you
@Override
public GatewayFilter apply(Config config) {
    return ((exchange, chain) -> {
        WebClient.create().post().uri("http://<IP_ADDRESS>:9001/user/query").retrieve().bodyToMono
                (String.class).flatMap(s -> {
            System.out.println(s);
            return Mono.just(s);
        });
        return chain.filter(exchange);
    });

how to get WebClient response

the webclient all returns a Mono and needs to be part of the filter chain. So instead of return Mono.just(s).
@Override
public GatewayFilter apply(Config config) {
    return ((exchange, chain) -> {
        return WebClient.create().post().uri("http://<IP_ADDRESS>:9001/user/query").retrieve().bodyToMono
                (String.class).flatMap(s -> {
            System.out.println(s);
            return chain.filter(exchange);
        });
    });

@spencergibb

the webclient all returns a Mono and needs to be part of the filter chain. So instead of return Mono.just(s).
@Override
public GatewayFilter apply(Config config) {
    return ((exchange, chain) -> {
        return WebClient.create().post().uri("http://<IP_ADDRESS>:9001/user/query").retrieve().bodyToMono
                (String.class).flatMap(s -> {
            System.out.println(s);
            return chain.filter(exchange);
        });
    });

Can I pass service name in URI, because port will be dynamic

I also had the same kind of problem and got resolved as mentioned in this answer
https://stackoverflow.com/a/77624403/1057291

I also had the same kind of problem and got resolved as mentioned in this answer
https://stackoverflow.com/a/77624403/1057291
You can check the demo project here
https://github.com/namalfernandolk/cloud-gateway-demo.git

@namalfernandolk Is webClient need set timeout or not ?  code:
webClient.post()                 .uri(uri)                 .contentType(MediaType.APPLICATION_JSON)                 .bodyValue(vo)                 .retrieve()                 .bodyToMono(String.class)                 .timeout(Duration.ofSeconds(5))


Debugging Issue

- Do you want to request a feature or report a bug?
- What is the current behavior?
- If the current behavior is a bug, please provide the steps to reproduce.
- What is the expected behavior?
- Please mention your node.js, NPM, Hugo and operating system version.

This is bug issue


after setup GO111MODULE=on, how to download a denpendency package  in any directory

What version of Go are you using (go version)?

$ go version
go version go1.11 darwin/amd64

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
$ go env

What did you do?
i setup export GO111MODULE=off, i can download a denpendency package by go get xxx in anywhere
i setup export GO111MODULE=on, i can't download a denpendency package by go get xxx in anywhere
after create a tmp directory , and run go mod init tmp,   and go get xxxx, it can download the xxx package
when  i setup export GO111MODULE=on, i wish download a denpendency package by go get xxx in anywhere, can you tell me how to do ?

before not use go modules, you can download a library in any directory by running go get xxxx
after using go modules, you must entry a directory which have a go.mod file, than you can run go get xxx download library to $GOPATH/pkg/mod, it's hard than not use go modules.
i want to know that when i setup GO111MODULE=on, how to download a library in any directory without a go.mod file

Please clarify what you mean by “download”. What do you want to do with the package once you've downloaded it?

Use go1.12beta1. Both go get and go mod download should work outside of a module.


build: remove standard version

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (main@df612ab). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2561   +/-   ##
=======================================
  Coverage        ?   93.22%
=======================================
  Files           ?      308
  Lines           ?    25185
  Branches        ?     2044
=======================================
  Hits            ?    23479
  Misses          ?     1675
  Partials        ?       31

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Adding libcereal-dev for non-boost serialization needs

We use serialization for some storage of large data types, but are trying to move away from Boost depends.  Cereal is a substitute for boost serialization that is lightweight and beats boost in benchmarks.
https://packages.ubuntu.com/xenial/libcereal-dev
https://packages.debian.org/stretch/libcereal-dev
https://apps.fedoraproject.org/packages/cereal-devel

CONTRIBUTING doesn't specific WHERE to add such links.  Presumably not in
the yaml.
On Fri, Jun 28, 2019 at 8:29 AM Chris Lalancette<EMAIL_ADDRESS>wrote:

@clalancette requested changes on this pull request.
Can you please add in the links to the packages as requested in
CONTRIBUTING
https://github.com/ros/rosdistro/blob/master/CONTRIBUTING.md? This just
makes it easier for us to track the packages in the future. Thanks.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ros/rosdistro/pull/21639?email_source=notifications&email_token=AA5N7RXNC22VTMAISAG5BMLP4YG2ZA5CNFSM4H4BFQ52YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB47LQCQ#pullrequestreview-255768586,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA5N7RURHJ2QNX7CPHQNI7DP4YG2ZANCNFSM4H4BFQ5Q
.

https://packages.ubuntu.com/xenial/libcereal-dev
https://packages.debian.org/stretch/libcereal-dev
https://apps.fedoraproject.org/packages/cereal-devel

CONTRIBUTING doesn't specific WHERE to add such links. Presumably not in the yaml.

Yeah, just in the description of the PR is where we expect it.  An update to CONTRIBUTING to specify where to add the links would be appreciated.

Just checking to see if there was anything else you were waiting on from me before this can get merged.  Thanks!

Just checking to see if there was anything else you were waiting on from me before this can get merged. Thanks!

Nope, just waiting for a second approval, then we can merge.

Not trying to be a jerk, but bumping this.  I have multiple entities waiting on a new feature, but I can't push to them until their install procedure automatically will install this system dep.

I appreciate the help.  Thanks


Bugfix octree factory

this change let the errors  in the factory to be seen in the backtrace and not silently removed by the call of ondiskoctree.

PTAL @nnmm

duplicate


Flags in mixed C/C++ compilation

I have a setup where a mix of C (.c) and C++ (.cc) files have to be compiled into one static linked library.  For the C files the -std=c99 flag is necessary:
let mut library = gcc::Config::new();
library.include("library/src/include").flag("-std=c99");

for src in &library_src {
    libraryl.file(src);
}

library.compile("liblibrary.a");

However, the compilation fails on the first C++ file, since clang complains about the invalid flag:
cargo:warning=error: invalid argument '-std=c99' not allowed with 'C++/ObjC++'
ExitStatus(ExitStatus(256))

command did not execute successfully, got: exit code: 1

Other build tools solve this by allowing to define separate flags for C and C++ files (e.g. flag_c and flag_cc, while flag is used for all).
Is a setup like this already possible or can support be added?

Ah yeah for now you'll have to build two separate .a files as gcc-rs isn't good at picking up which goes to which file. Does that work for you?

That should probably work. Allowing to set different flags for different compilers would however be a convenient feature.
The problem in my case right now is that the include path seems to be ignored or headers files from the system are used instead for the so far single C++ file, so I can't report back with an definitive answer.

Sounds bad! If that's the case it's definitely a bug! Let me know what you find though

I'm going to close this for now since I think that using two builds is workable enough for now to solve this.


Allow PR checks to be run against any base branch

Description of change
I've been working in a fork and found that CI wasn't running in the PRs against my feature branch. This meant the code wasn't actually buildable since I'd missed including a file into the PR.
This change will cause all PRs to run integration tests and benchmarks.
We may want to allow a way to disable running actions in the future, but for now we just expand this to cover all base branches.
I'm also leaving push events out of scope, although we could consider a similar change. This may have more of an impact on community forks though so I'll leave it as is for now.
Does this change impact existing behavior?
It impacts CI only. Pull requests not against main as the base branch will now run benchmarks and integration tests.
Does this change need a changelog entry in any of the crates?
No.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and I agree to the terms of the Developer Certificate of Origin (DCO).

Thoughts on only extending to feature branches? e.g. feature/**

Thoughts on only extending to feature branches? e.g. feature/**

I don't really have an opinion here, although I'd be inclined to cover additional branch prefixes like private-feature/**.


⚠ dev-deces.matchid.io search API has degraded performance

In 0cd4ba9, dev-deces.matchid.io search API (https://dev-deces.matchid.io/deces/api/v1/search?q=pompidou) experienced degraded performance:

HTTP code: 200
Response time: 4772 ms

Resolved: dev-deces.matchid.io search API performance has improved in 9589bee after 6 minutes.