Screenshots module fails to select (default) slices when voxel size different than 1mm

When using the screenshot option on subjects processed with FreeSurfer's recon-all including the -cm flag the module fails to create the screenshots (with the default views). We were looking into the code and it seems that the problem is that, when using the -cm flag, the voxel size of FreeSurfer outputs in not 1mm, and therefore in this line:
https://github.com/Deep-MI/qatools-python/blob/580530b24f7f29cc1c7ab3f9211998493f49be7c/qatoolspython/createScreenshots.py#L220
the code fails to select volumetric data because there is no slice with those TkReg RAS coordinates.
As a workaround, we have tried to select the closest volumetric slice, but the there is a (slight) mismatch between the plotter surface and volumetric data. Therefore I would be necessary either first check whether the selected "views" are feasible and if not change them to the closest feasible one for both surface and volumetric data, or interpolate volumetric data. We have some modifications to your code that performs the first option.

Thank you @dmartindeblas, the detailed info and your suggestion is very helpful. I will take a look within the next days and try to solve the issue. If you want to, maybe create a pull request so that we can consider your modifications also. Thanks again.


Migrate issue labels

Existing issue labels should be migrated from internal Adobe github.

If you can give me admin access on the internal git.corp repo I can do this.

I'll do that now

done


Allow code highlighting for more languages

Hi!
While trying to write in my blog some racket code I noticed that it is not highlighted by specifying racket as the language, it also did not work for lisp nor for scheme.
I was wondering if this is a matter that concerns highlight.js? If not, how could highlight support could be added to these languages?
I searched for available languages to be highlighted in _libs/highlight/highlight.min.js but the ones I mentioned are not there.
This is my blog where said code is written. It is hosted on my github account so the source code can be seen.

Thanks! My bad, I did not check that specific section of the docs.


Support for Azure Functions v4 (?)

Is there a way to switch to V4 of the Azure Functions? The docs say that you shouldn't have a function.json, but it seems that that is necessary to use startServerAndCreateHandler?
How should you use startServerAndCreateHandler in this no-json V4 model?
Thanks in advance!

Encountering the same issue as well. Seems the package is still using @azure/functions^3.2.0 when it should be using @azure/functions 4.x which changed how function apps are defined in nodejs

This is related to #42.
Azure Functions v4 is a new programming model for JavaScript/TypeScript and it sees quite a few changes to the types that are used, so it's not a direct upgrade. As it's still in public preview I'm holding off supporting Azure Functions v4.

Makes sense, if you do plan on creating a beta version on a separate branch and are looking for community support I believe some of us on this thread may be interested in helping.

I did explore it early on (when we were prototyping v4) to see what the approach might look like for building libraries on top of the new programming model. I really wanted to do:
import { app } from "@azure/functions";

app.graphql(/* stuff here */);

But it's not possible and ended up having some discussion with the Functions team on what could be good options and there's a few to try but... well... time 😅

Has any further progress been made on this issue, please? Happy (and eager!) to contribute but want to avoid duplicating anyone else's work. Thanks.

No I haven't.
I've moved out of the JavaScript space at Microsoft so am rather time poor to tackle some of the stuff here.

No worries, @AaronPowell! I'm happy to start pulling together a PR if you're willing/able to review it?

No worries, @aaronpowell! I'm happy to start pulling together a PR if you're willing/able to review it?

More than happy to review

Thanks! I might take a little while as I have lots of other stuff on my plate right now and I'm only just getting started with Azure Functions, but I'll submit a proof of concept in the next month or so, and we can no doubt all work towards improving it. Appreciate your help.

@aaronpowell I have fixed the current support of V4 in the current library, Am I able to contribute the code??

@aaronpowell I have fixed the current support of V4 in the current library, Am I allowed to contribute the code??

Sure, submit a PR

Hello everyone, can someone help me set up the repository standard code with the solution?

@aaronpowell  @TengoCharlie is there any progress on this please?
Azure functions v4 model is officially live now not in beta anymore.

@douglas-wang , am looking into it. Will raise a PR soon

@aaronpowell I have fixed the current support of V4 in the current library, Am I allowed to contribute the code??

Sure, submit a PR

I have changes the code as per the function V4. but i have a question that are we going to release another package for function v4 or we will provide support for v4 and v3 by same package?

It'd be best to support them both in a single package

@aaronpowell I just opened a PR to address this issue https://github.com/apollo-server-integrations/apollo-server-integration-azure-functions/pull/66

Thanks @barddoo, I have tested your PR and works perfectly with Azure functions v4.1.0 and Apollo server v4.10.0

@trevor-scheer - This repo has gone stale from an Apollo support standpoint (the last update was 3 months ago) - several people need to update to Azure Functions Programming model v4, and this pull request is blocking them - It looks like @aaronpowell has their attention turned toward other Microsoft priorities, is this something you can help move forward in his absence?

As a follow up on this, I just noticed that one cannot access the body in the get context function as one could in V3, as the body can only be parsed once.
https://github.com/apollo-server-integrations/apollo-server-integration-azure-functions/pull/77


[menu] inconsistent scroll strategy between body and cdkScrollable

Bug, feature request, or proposal:
Bug
Current behavior
On material.angular.io:

Mobile: scroll is caught and menu repositions itself
Desktop: backdrop seems to block scrolling events and cannot scroll while menu is open

Expected behavior
Menu trigger dictates reposition strategy. If that's not ideal or expected, then block should be used.
What are the steps to reproduce?
Can be seen on docs site and demo-app. The difference between mobile and desktop is a result of the fixed sidenav, specifically use of cdkScrollable, not a difference in browsers.
cc @crisbeto

Will be sorted out by https://github.com/angular/material2/pull/7779. Also a dupe of https://github.com/angular/material2/issues/6927.


MaterialDesign-Webfont last version not shown

MaterialDesign-Webfont last version not shown

what am I supposed to understand?

what am I supposed to understand?


Add support for plugin path in omnisharp-vscode

Is this a Bug or Feature request?:
Feature
Description of the problem:
In order to properly debug omnisharp-vscode's interaction with the Razor plugin we need to be able to control where it loads the plugin from.

0f635895fef1129f91d147c89ec0a7030c5e7ff8

https://github.com/OmniSharp/omnisharp-vscode/commit/d75cea2b27c88dee4583835dce1ff3b5fc883dd8


Update dhruv.md

my unmerged pr's group 2 (pair of another 3 pr)

@sanjay-kv
in this group i removed 3 unmerged pr from my group 1
now left

https://github.com/Counselllor/Counsellor-Web/pull/1296
https://github.com/pooranjoyb/popShop/pull/320
https://github.com/Rajan-Barnwal/jobproject/pull/360
https://github.com/JiyaGupta-cs/ShopNex/pull/353
https://github.com/omrajsharma/bigohhh.com/pull/87

@sanjay-kv
when you become free please reeview it


Implement linear_fusion for task spec class

optimize_slices is now unused, but not sure how public this is
same for inline_functions

I checked optimize_slices and there is no usage on Github
inline_functions is not as trivial since that's a more common name :(


Audio/video out of sync

Recording results in video/audio out of sync.  This does not happen when actively connected via the web app.
Ive noticed several times the audio (up to a min) is missing from the beginning of the video.
Any info/direction appreciated.

In record.js there is a bit where it attempts to sync the audio/video. After some testing I ended up with 'adelay=delays=250|250', which works pretty well (the default is 1000|1000).

@colohr you can change the values of adelay here https://github.com/aws-samples/amazon-chime-sdk-recording-demo/blob/master/recording/record.js#L52-L53 to see if that helps.

Feel free to re-open if it's still an issue.

In my case Im not getting audio at all


help(mat-dialog): Closing behaviour of Material dialog

I want to have following behaviour on closing dialog

Dialog can be closed by ESC or any event triggered from inside the dialog
Prevent clicking outside the dialog, to close dialog

To achieve 1st, I need to set disableClose property to false. And to achieve 2nd behaviour, I had to set disableClose property to true.
How can I achieve this?
Thanks.

You can use disableClose along with OverlayKeyboardDispatcher to manually close on escape.


docs: link current tasks with repo entry docs page

Preflight Checklist

[X] I have read the Contributing Guidelines for this project.
[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the pull request tracker for a Pull Request (PR) that matches the one I want to submit, without success.

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

[ ] Tests for the changes have been added (for bug fixes / features)
[X] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[X] Build (npm run build) was run locally and any changes were pushed
[X] Lint (npm run lint) has passed locally and any fixes were made for failures

Changes
Changes in this pull request:

Documentation

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[X] Documentation content changes
[ ] Other (please describe):

Does this introduce a breaking change?

[ ] Yes
[X] No

Other information

Mentions

@lyne-design-system/core

Codecov Report

Merging #762 (fe68850) into master (2f37cdb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #762   +/-   ##
=======================================
  Coverage   29.69%   29.69%
=======================================
  Files          11       11
  Lines         532      532
  Branches      124      124
=======================================
  Hits          158      158
  Misses        355      355
  Partials       19       19

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 fffb24e...fe68850. Read the comment docs.


Modify build script

Search further for "Exiting batchmode successfully now!" in the log to prevent false build error

:+1:


drop examples/bulk_upsert + create integration test of bulk_upsert

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[x] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Other information

Codecov Report
Patch coverage has no change and project coverage change: -18.49 :warning:

Comparison is base (80aa6fb) 63.12% compared to head (1437549) 44.64%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #608       +/-   ##
===========================================
- Coverage   63.12%   44.64%   -18.49%
===========================================
  Files         228      153       -75
  Lines       21773    15576     -6197
===========================================
- Hits        13745     6954     -6791
- Misses       7336     8302      +966
+ Partials      692      320      -372

Flag
Coverage Δ

1.16.x
44.25% <ø> (-18.49%)
:arrow_down:

1.19.x
44.70% <ø> (-17.91%)
:arrow_down:

1.20.x
?

insecure
?

integration
?

macOS-latest
?

secure
?

ubuntu-latest
44.64% <ø> (-18.47%)
:arrow_down:

unit
44.64% <ø> (-0.12%)
:arrow_down:

windows-latest
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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

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


Set owners tag in test resource deployments

This sets up user-created resource groups to have the right metadata so that one has the option to remove DeleteAfter
tags to create long-lived resources. The alternative approach would be to update $BaseName generation to be formatted
like $UserName-$ServiceDirectory so we could parse and query the alias, but I think that would cause downstream
resource deployments that use the basename to fail validation (e.g. storage accounts don't support dash characters).

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Python equivalent for pyramid_up

Hi,
I noticed that there isn't a Python equivalent for pyramid_up, which is quite essential for detecting objects in smaller images. Could this be added to future action points, or is a Python binding for the same considered irrelevant?
I don't mind closing this down if it is not the need of the hour, I would like to try the porting myself, would you mind showing some pointers?

There are already a lot of python image processing libraries that provide
equivalent functions.  But if you want to submit a pull request for this
that's fine with me.


convert meta.yaml to recipe.yaml

Checklist

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

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe/recipe.yaml) and found some lint.
Here's what I've got...
For recipe/recipe.yaml:

The top level meta key schema_version is unexpected

Hi! This is the friendly automated conda-forge-webservice.
I tried to rerender for you but ran into some issues. Please check the output logs of the GitHub actions workflow below for more details. You can also ping conda-forge/core for further assistance or you can try rerendering locally.
The following suggestions might help debug any issues:

Is the recipe/{meta.yaml,recipe.yaml} file valid?
If there is a recipe/conda-build-config.yaml file in the feedstock make sure that it is compatible with the current global pinnnings.
Is the fork used for this PR on an organization or user GitHub account? Automated rerendering via the webservices admin bot only works for user GitHub accounts.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/11272120759.

@conda-forge-admin, please rerender

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

Hi! This is the friendly automated conda-forge-webservice.
I tried to rerender for you but ran into some issues. Please check the output logs of the GitHub actions workflow below for more details. You can also ping conda-forge/core for further assistance or you can try rerendering locally.
The following suggestions might help debug any issues:

Is the recipe/{meta.yaml,recipe.yaml} file valid?
If there is a recipe/conda-build-config.yaml file in the feedstock make sure that it is compatible with the current global pinnnings.
Is the fork used for this PR on an organization or user GitHub account? Automated rerendering via the webservices admin bot only works for user GitHub accounts.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/11272180679.

@conda-forge-admin, please restart ci

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to rerender for you but ran into some issues. Please check the output logs of the GitHub actions workflow below for more details. You can also ping conda-forge/core for further assistance or you can try rerendering locally.
The following suggestions might help debug any issues:

Is the recipe/{meta.yaml,recipe.yaml} file valid?
If there is a recipe/conda-build-config.yaml file in the feedstock make sure that it is compatible with the current global pinnnings.
Is the fork used for this PR on an organization or user GitHub account? Automated rerendering via the webservices admin bot only works for user GitHub accounts.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/11272387487.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to rerender for you but ran into some issues. Please check the output logs of the GitHub actions workflow below for more details. You can also ping conda-forge/core for further assistance or you can try rerendering locally.
The following suggestions might help debug any issues:

Is the recipe/{meta.yaml,recipe.yaml} file valid?
If there is a recipe/conda-build-config.yaml file in the feedstock make sure that it is compatible with the current global pinnnings.
Is the fork used for this PR on an organization or user GitHub account? Automated rerendering via the webservices admin bot only works for user GitHub accounts.

This message was generated by GitHub actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/11272671119.


Run npm deprecate on this package

Just a suggestion but even your own readme does state this package is obsolete at this time. While it does seem the tail dropped off pretty hard in your weekly download stats, it might not be a bad idea to push it a bit further and run npm deprecate resolve-typescript-plugin "Use resolve.extensionAlias in webpack.config.js instead"

At the moment I'm still maintaining this for the benefit of people who use older versions of webpack. I'm not sure npm deprecate is the right thing to do for them.

As in for 4.x.x, if so that's almost 3 years old now isn't it? If it's for 5.x.x, hopefully most people would be able to bump their minor/patch versions without little issue? The message could be Update to<EMAIL_ADDRESS>or later and use resolve.extensionAlias in webpack.config.js instead?

I have clients who are still on webpack 4 for one reason or another, and for that matter some of my own projects are still on webpack 4. In some cases it's a lot of work to migrate and not really worth it. Everyone involved knows they're behind the latest webpack release and doesn't really need an extra deprecation warning to push them off this library when it will be maintained for the foreseeable future.
I will maybe print a deprecation notice during the build for webpack 5 users though. It'd be good to get people doing things the right way where possible.


Possible to override default JSON property naming?

I would like all properties to be kebab-case by default, e.g. MyProperty becomes my-property. This is dead simple with the help of Humanizer's Underscore() and Hyphenate() string extensions by defining a simple contract resolver (shown in F# below):
type KebabCaseContractResolver() =
  inherit DefaultContractResolver()

  override __.ResolvePropertyName(propName: string) =
    propName.Underscore().Hyphenate()

However, JsonApiSerializer comes with its own contract resolver, so I can't use the one I created above. I have also tried subclassing your contract resolver and overriding ResolvePropertyName as above, but that had no effect and ResolvePropertyName wasn't even called.
I currently use JsonPropertyAttribute for all properties to explicitly specify the JSON property names manually, but that's far from ideal since the transformation is trivially automatable as shown above.
Is there a way I can enforce my own JSON property naming convention while using JsonApiSerializer?

Figured it out - this is a job for NamingStrategy:
type KebabCaseNamingStrategy() =
  inherit Newtonsoft.Json.Serialization.NamingStrategy()
  override __.ResolvePropertyName propName =
    propName.Underscore().Hyphenate()

let jsonSerializerSettings =
  JsonApiSerializer.JsonApiSerializerSettings(
    ContractResolver =
      JsonApiSerializer.ContractResolvers.JsonApiContractResolver(
        NamingStrategy = KebabCaseNamingStrategy()
      )
  )

// use jsonSerializerSettings as normal


[WIP] Thesis/workflow-modeler

Modelling BPMN4TOSCA Workflows by combining Managementtasks and defining the Interfaces, Operations and Inputs of Node Templates
Signed-off-by: Björn Müller<EMAIL_ADDRESS>

[x] Ensure that you followed http://eclipse.github.io/winery/dev/ToolChain#github---prepare-pull-request. Especially, we require a single commit
[x] Ensure that the commit message is a good commit message
[x] Ensure to use auto format in all files
[x] Ensure that you appear in NOTICE at Copyright Holders
[ ] Tests created for changes
[ ] Documentation updated (if needed)
[ ] Screenshots added (for UI changes)

Follow up at: https://github.com/eclipse/winery/pull/451


Token Signature could not be verified.

I created in a postgres db, after logging in and trying to access the modules (categories, products) the system closes with the following api message
BadRequestHttpException in BaseMiddleware.php line 51:
Token not provided

in BaseMiddleware.php line 51
at BaseMiddleware->checkForToken(object(Request)) in BaseMiddleware.php line 66
at BaseMiddleware->authenticate(object(Request)) in Authenticate.php line 30
at Authenticate->handle(object(Request), object(Closure)) in Pipeline.php line 137
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 33
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in HandleCors.php line 34
at HandleCors->handle(object(Request), object(Closure)) in Pipeline.php line 137
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 33
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in SubstituteBindings.php line 41
at SubstituteBindings->handle(object(Request), object(Closure)) in Pipeline.php line 137
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 33
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in ThrottleRequests.php line 49
at ThrottleRequests->handle(object(Request), object(Closure), '60', '1') in Pipeline.php line 137
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 33
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Pipeline.php line 104
at Pipeline->then(object(Closure)) in Router.php line 655
at Router->runRouteWithinStack(object(Route), object(Request)) in Router.php line 629
at Router->dispatchToRoute(object(Request)) in Router.php line 607
at Router->dispatch(object(Request)) in Kernel.php line 268
at Kernel->Illuminate\Foundation\Http\{closure}(object(Request)) in Pipeline.php line 53
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in HandleCors.php line 34
at HandleCors->handle(object(Request), object(Closure)) in Pipeline.php line 137
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 33
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in CheckForMaintenanceMode.php line 46
at CheckForMaintenanceMode->handle(object(Request), object(Closure)) in Pipeline.php line 137
at Pipeline->Illuminate\Pipeline\{closure}(object(Request)) in Pipeline.php line 33
at Pipeline->Illuminate\Routing\{closure}(object(Request)) in Pipeline.php line 104
at Pipeline->then(object(Closure)) in Kernel.php line 150
at Kernel->sendRequestThroughRouter(object(Request)) in Kernel.php line 117
at Kernel->handle(object(Request)) in index.php line 53
at require_once('/home/prodap/Workspace/tmp/spa-starter-kit/webservice/public/index.php') in server.php line 21

I see the same problem here!
@flyingluscas if possible, could you please take a look? Apparently other users are facing the same problem: a valid token is passed (checked with http://jwt.io) but the app returns Token Signature could not be verified.
Thanks!

@vedovelli Yes sir, just a moment :)

@vedovelli @carvalhoviniciusluiz Can you guys tell me in what branch is this happening, master or develop?


Schema const enums

What is the problem this feature would solve?
Zod supports these fixed set enums which are like arrays declared with the const keyword. I was hoping Effect's Schema could also.
What is the feature you are proposing to solve the problem?
Support for const arrays.
What alternatives have you considered?
I have tried Schema.Enums which doesn't work.

I believe you want
Schema.Literal("a", "b")

@jessekelly881 Thanks. The Schema.Literal(...arr) works. Sorry for the inconvenience. I suppose an example might be good, but IDK up to you!


Incorrect //= require in application.js

Under "Installation using the jquery-rails gem" is the following code for the application.js manifest file:
//= require jquery
//= require jquery_ujs
Bottom line should be //=require jquery-ujs

No, the docs are correct.


Fix typos in README

Minor changes for consistency and readability

Issue: #

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.

[x]  issue was opened to discuss proposed changes before starting implementation.
[x]  during development, node version specified in package.json was used.
[x]  package versions and package-lock.json were not changed (npm install --no-save).
[x]  app version number was not changed.
[x]  all new code has tests to ensure against regressions.
[x] npm run lint reports no offenses.
[x] npm run test is error-free.
[x]  README and CHANGELOG were updated accordingly.

Description of the Change

Verification Process

Other information

I can check the boxes, but I don't want to get into the habit of doing so mindlessly and the only one that really applies is the last one(?)

Yeah, with text updates it does not matters ;] Many thanks!


Make this available on PyPI

Currently this package is only available via GitHub. Installing from GitHub will download the whole package including all datafiles. Having an installable package on PyPI will prevent downloading the whole data until it's finally requested.

Nevermind, I've just found that this is uploaded to PyPI. :face_with_spiral_eyes:


No completions are provided for field access experssions

Description:
Consider the following source snippet and no completions are provided at the cursor position.
string header = "+ ".'join('join(" + ", headers)).j<cursor> Completions are not provided
Affected Versions:
sl-alpha5 at least

Related discussion https://github.com/ballerina-platform/ballerina-lang/pull/30778#discussion_r641318484

The same error occurs for this.
public function main() {
    string[] hello = ["a", "b"];
    int[] mapResult = hello.'map(function (string a) returns int {
        return 12;
    }).<cursor>
}


Send the welcome message to "Current Batches" instead of "397 Bridge"

This should be a single string change, but it'll greatly improve the alignment between who needs to be introduced to Pairing Bot and who receives the welcome message!

And here's some free scope creep! If we base this on the configured environment, we can start sending the dev bot's messages to #test-bot > 🍐🤖 instead of just printing them to the log!
Relevant code links
https://github.com/recursecenter/pairing-bot/blob/87c300d9a3120837170f6d7040802996d732d1ec/pairing_bot.go#L328-L334
https://github.com/recursecenter/pairing-bot/blob/87c300d9a3120837170f6d7040802996d732d1ec/pairing_bot.go#L372-L374
https://github.com/recursecenter/pairing-bot/blob/87c300d9a3120837170f6d7040802996d732d1ec/zulip/client.go#L53-L57


Fix Merge Logo Size

Logos are broken on mobile

Fixed: https://github.com/weweswap/app-page/pull/193


Issue about shortcut.

In your resnet model, you did a Relu activation before shortcut which after bn(conv2). I saw there are some other people just shortcut to the output of bn without Relu. Which one is better, did you try them separately before?
However, your work is great for new pytorcher like me ^_^. Hope to see wide resnet and deeplab in the future.
Best wishes.

You are right. Shouldn't be a relu here.
I think this is a typo, I checked this is no relu in the history version of this file. I'll remove it later.
Thanks.


Update documentdb-create-dbaccount.md

Updated link for [Consistency levels in DocumentDB]. - I tried to use the ../ path, I believe I got it correct but I might have an error there.  Trying to link to https://azure.microsoft.com/en-us/documentation/articles/documentdb-consistency-levels/.
It was linking to ../articles/azure-portal/resource-group-portal.md (https://azure.microsoft.com/en-us/documentation/articles/resource-group-portal/).

Hi @oedenfield, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@oedenfield : Thanks for your contribution to the Azure documentation! The author, @azdocprs, has been notified to review your proposed change.

@AndrewHoh Hi Andrew. Please review the proposed changes. Thanks.

Hi @oedenfield, Thank you for finding and fixing this! Your change is almost correct, you just need to include the documentdb folder in the relative link, so it should look like this(../articles/documentdb/documentdb-consistency-levels.md). Can you make that change and we'll accept the PR? Thanks!

@mimig1 Thank you for your help.  I believe I've updated the change in the PR.  Please let me know if I've made an error here.  Cheers!

@oedenfield Looks good, thank you!
@jomolnar I #sign-off, but this will likely cause an error as @andrewhoh is listed as the author, but he is OOF this week. Please merge. Thanks!

Only the author of the article can use the sign off comment to add the ready-to-merge label.


UHF-7650: Fix neuvonta chat compatibility with disable-genesys-chat

UHF-7650
Make chat open button available for the helsinki-info page (again).
What was done
Added logic to check if disable-genesys-button has disabled the chat button and replace it with the open chat button.
How to install

Boot up strategia instance (https://github.com/City-of-Helsinki/drupal-helfi-strategia/)
Make sure your instance is up and running on latest dev branch.

git pull origin dev
make fresh

Update the Helfi Platform config

composer require drupal/helfi_platform_config:dev-UHF-7650-helsinki-info-button

Update HDBT

composer require drupal/hdbt:dev-UHF-7650-helsinki-info-button

Run make drush-cr

How to test

If you ran make fresh, you should be able to navigate to this page: https://strategia.docker.so/fi/paatoksenteko-ja-hallinto/ota-yhteytta/helsinki-info
Edit the page, specifically the first 50/50 paragraph containing a text paragraph
Edit the html to include an element with an id of openChat (as in something like Avaa chat)
Load the page. You should see a button element where you added the element.
Clicking on the button should pop the neuvonta chat open.

Other PRs

HDBT: https://github.com/City-of-Helsinki/drupal-hdbt/pull/751

Codecov Report

Merging #568 (24c3cb1) into main (a44de27) will not change coverage.
Report is 4 commits behind head on main.
 Consider uploading reports for the commit fe10bd7 to get more accurate results

@@            Coverage Diff            @@
##               main     #568   +/-   ##
=========================================
  Coverage     12.74%   12.74%
  Complexity      236      236
=========================================
  Files            30       30
  Lines           902      902
=========================================
  Hits            115      115
  Misses          787      787

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Skip flaky tests

Unblocking patch build

Might hit more after this assembly. Is there a way to disable all *.SqlServer.* test assemblies from running?


dont print the entire text when data type is text

🐛 Describe the bug
When we are adding a data type "text", the entire text gets printed due to the below statement.
we can either skip printing text altogether or print part of it.
print(f"All data from {src} already exists in the database.")

Could this be assigned to me?

Hey @dtee1 welcome to embedchain.
Yeah sure. Just assigned.


⚠ Bofrost.fr has degraded performance

In 9dc108e, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 2055 ms

Resolved: Bofrost.fr performance has improved in a161205 after 6 minutes.


Don't default to sqlite3, or provide option to disable?

I'm using SQL Server in my app. When I build a Dockerfile without specifying a database, it assumes I wish to use Sqlite3.
My Dockerfile is generated with the following additional lines:
RUN mkdir /data
# ...
ENV DATABASE_URL="sqlite3:///data/production.sqlite3"
# ...
VOLUME /data

A few notes:

The scanner.rb correctly does not set @sqlite3 = true
However dockerfile_generator.rb sets a default to sqlite3
The Readme seems to imply that if this tool cannot infer the database correctly, you will need to specify it, such as specifying --sqlite3 from the Add a Database section.

I can appreciate that adding sqlite3 as a default is probably convenient or likely, however, if it is being used, it will be in the gemfile, or the database config, and as such, will be detected and should not be assumed unless specified. Because it is, I am manually editing the Dockerfile, even though it's otherwise entirely autogenerated from my config/dockerfile.yml config file.
Would you consider removing the default value for #deploy_database, and require checking as you do for pg and mysql?

Would supporting sqlserver address your needs?
https://github.com/fly-apps/dockerfile-rails/commit/40a1f9ba39682f1c6763fff9983acd03fbfcff30

That would be even better! Cheers!


typos

"ouputs" -> "outputs"

Codecov Report

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

@@          Coverage Diff           @@
##           master    #847   +/-   ##
======================================
  Coverage    78.9%   78.9%
======================================
  Files          34      34
  Lines        6192    6192
======================================
  Hits         4886    4886
  Misses       1306    1306

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 a615499...76be189. Read the comment docs.

Indeed!


Proposal testing

Description
Adds a secret dev environment only function to populate test data into the create proposal fields.
Testing
In the Proposal Creation screen, click the "Create Proposal" title header to populate proposals.

Not sure how I borked this branch.. going to try another PR

Oh, I see now I was merging into the wrong branch....


Drag a texture image directly into the viewport

Drag a texture image directly into the viewport
Until now, to add a texture to a sprite renderer component, the image has to be dragged onto the "Texture" button in the scene hierarchy panel. Now, in addition, one can drag a texture image directly onto a component in the viewport.

May I suggest that you change it to something like this:
`std::filesystem::path filePath = std::filesystem::path(path);
			if (filePath.extension().string() == ".hazel" || filePath.extension().string() == ".ncz")
			{
				OpenScene(std::filesystem::path(g_AssetPath) / path);
			}
			else if (filePath.extension().string() == ".png")
			{
				std::filesystem::path texturePath = std::filesystem::path(g_AssetPath) / path;
				Ref<Texture2D> texture = Texture2D::Create(texturePath.string());
				if (texture->IsLoaded())
				{
					if (m_HoveredEntity && m_HoveredEntity.HasComponent<SpriteRendererComponent>())
						m_HoveredEntity.GetComponent<SpriteRendererComponent>().Texture = texture;
				}
				else
				{
					HZ_WARN("Could not load texture {0}", texturePath.filename().string());
				}
			}`

and keep the rest you had? Because the code you provided will crash if the user tries to drag a texture into the scene and there is no highlighted entity! this fixed that problem and also i personally do not like the style of using the folder structure from 'scenes' and 'textures'... what i mean is, what if somebody has a scene or texture outside of those folders? therefore i made this and you can freely copy it and update your pull request :) Please let me know if this is fine by you :)

May I suggest that you change it to something like this:
			std::filesystem::path filePath = std::filesystem::path(path);

			if (filePath.extension().string() == ".hazel")
			{
				OpenScene(std::filesystem::path(g_AssetPath) / path);
			}
			else if (filePath.extension().string() == ".png")
			{
				std::filesystem::path texturePath = std::filesystem::path(g_AssetPath) / path;
				Ref<Texture2D> texture = Texture2D::Create(texturePath.string());
				if (texture->IsLoaded())
				{
					if (m_HoveredEntity && m_HoveredEntity.HasComponent<SpriteRendererComponent>())
						m_HoveredEntity.GetComponent<SpriteRendererComponent>().Texture = texture;
				}
				else
				{
					HZ_WARN("Could not load texture {0}", texturePath.filename().string());
				}
			}

and keep the rest you had? Because the code you provided will crash if the user tries to drag a texture into the scene and there is no highlighted entity! this fixes that problem and also i personally do not like the style of using the folder structure from 'scenes' and 'textures'... what i mean is, what if somebody has a scene or texture outside of those folders? therefore i made this and you can freely copy it and update your pull request :) Please let me know if this is fine by you :)

Thanks for pointing this out. Initially, I used the extension to identify the action. Then I thought that a scene may not have the ".hazel" extension or a texture may not necessarily be a ".png" file always. Therefore, I changed it to directories.
Regarding the crash, I think, a condition can be added to prevent that.
Bottom line is that it's a design issue and I have no idea whether Cherno will maintain the directory structure or the scene format or the texture format. It will be great if he takes a call on this. In the meantime, I will work on the bug you have pointed out.
Thanks again for your feedback.

@VagueLobster  I have added your code in the new commit. I have also fixed the bug you pointed out. Please have a look at the updated version.

I have just looked at the new commit, and it works now, but why keep the old code? Is there a reason for that?

As I mentioned in my first response that I have no idea whether Cherno will maintain the directory structure or the scene format or the texture format.  Please remove comments, remove imgui.ini from commit, make sure it still merges with no conflicts, and I'll merge it in. 👍

@MajisTcd I think @TheCherno wanted you to remove the comments, not re-use the old code that you commented out! 🙂

@VagueLobster I was about to respond to you, I was just trying to figure out what exactly I need to do. In my machine, the code works fine. If you think you can still find bugs, I am changing it to your code. However, I am not sure about "delete the comments". If you have got it, can you please tell me which comments I should delete?

@VagueLobster I was about to respond to you, I was just trying to figure out what exactly I need to do. In my machine, the code works fine. If you think you can still find bugs, I am changing it to your code. However, I am not sure about "delete the comments". If you have got it, can you please tell me which comments I should delete?

The bug, i thought i could reproduce is not a bug it seems! However, maybe it's just me, but i don't like to be forced to have to use the textures folder to be able to import textures.
I think that the user should have a choice  as to where they want they're textures to be 🙂
Also the users shouldn't be required either to have their scenes inside scenes folder, again it should be their free choice!
That was why i changed your code back in September last year! 🙂

@MajisTcd Actually, there is a minor bug in your code, i just found:
Because of this line of code else if (parentPath == "textures" && name != "None")
the user cannot add textures to sprites that is named None! Again the user should have the freedom to choose
whatever they want imo! Of course maybe not in every instance, but this, i don't see any harm in having the freedom 🙂

@VagueLobster Thanks for your time and suggestions. This is my first time contributing to a public repository. So I am having some trouble using git. I have kept your code but removed the comment mentioning that it's your code. Please let me know how to acknowledge your contribution and I will be happy to do so.

No problem 🙂 You don't have to mention me or that I have changed your code, i'm just happy to help 😄
But you should change your repository to have the imgui.ini from Cherno's most recent pull request and NOT changing it any further, otherwise github registers it as a change that must be commited! Also you need to remove the ColoredCube.hazel from this repository when commiting, because it's not a file that Cherno has made! And it's not necessary 😄 Otherwise it looks good 👍


Add support for full retrieval of SecurityGroup data

In order to allow for idempotent support of passing ingress/egress rules with security groups for the Exoscale terraform plugin, I needed to modify the egoscale library to return more data than just the Security Group Id based off the name.

Thanks @cab105 !

yw


addressing issue 650 integration with core-metadata

Signed-off-by: xmlviking<EMAIL_ADDRESS>Addressing issue with scheduler and core-metadata integration with schedules,scheduleevents, and addressables.  Scheduler will now use core-metadata as the source of truth for schedules and only use it's local configuration scheduler,events when not included in the core-metadata.  Additionally new schedules, events, and addressable unique id's will be generated from core-metadata.
We currently do not have reactive feedback to the support-scheduler from the metadata client.
For now any changes made to core-metadata which require support-scheduler notification should be accomplished via the /api/v1/flush router call on the support-scheduler.  This will flush the current schedules and reload the entire schedules, events, and addressables "including any changes made in the support-scheduler config".
Work to be completed in a future will include:

[ ] - Mocks and Unit Tests around the new enhancement features for the core-metadata
[ ] - Mocks ScheduleClient, ScheduleEventClient, and AddressableClient

recheck

recheck


Error while iterating over chunk blocks

Hello, I'm trying to manually iterate over chunk blocks using the following example:
let complete_chunk = complete::Chunk::from_bytes(&data).unwrap();

for chunk_x in 0..16 {
    for chunk_z in 0..16 {
        for chunk_y in complete_chunk.y_range() {
            let block = complete_chunk.block(chunk_x, chunk_y, chunk_z);

            match block {
                Some(block) => {}
                None => {}
            }
        }
    }
}

But at certain points it fails with the following error:
thread 'main' panicked at /home/jotalanusse/.cargo/registry/src/index.crates.io-6f17d22bba15001f/fastanvil-0.30.0/src/complete/section.rs:26:52:
called `Option::unwrap()` on a `None` value
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Am I doing something wrong? Iterating over the blocks with complete_chunk.iter_blocks() seems to work just fine, but I couldn't find how it is implemented.
You can find the .mca file im using here.

This does look like a bug in fastanvil, thanks for the mca file. I will have to investigate at some point.

Just as an update, I've not had much time to look into this. The MCA file you provided has some chunks in it that I don't have any test coverage for. Some of your chunks have a 'Y=-5' section. A section is a 16x16x16 collection of blocks, a tower of sections make up a chunk.
You have a section for the -80 to -65 Y region for some reason, and that section contains no information at all. Usually there's a block palette at least.
I need to investigate more to know the best way to handle it, and it's definitely surfaced a very real bug in fastanvil, so thanks. :)


I am facing some weird issues on select component.

The following is what I have used the select component on my project.
<Select name="proof" placeholder="Basic" multiple bind:value={currentProofs}>
	<Select.Options slot="options">
		{#each proofOptions as option}
			<Select.Options.Option {option} selected="false" />
		{/each}
	</Select.Options>
</Select>

Once I have changed the currentProofs(in case that I want to edit one item of list), the selected items were not reflected on dropdown menu.

Please provide a reproduction. I am not following what your issue is with the description laid out.

Closing this issue as no reproduction has been provided and I cannot reproduce myself.


False Positives on adding Connext Network Bridge URL as a phishing site

Connext Network is a legit bridge with a creditable company profile. Connext team had been operating as a interoperability protocol for years, and recently they just rebranded from xpollinate to Connext Bridge. This causes their URL to changed to bridge.connext.network. I think this is a misunderstanding to flag Connext domain as a phishing website.
The PR that mark connext as a phishing website was #3670. I strongly believe this was a mistake.
Additional comment here, what are the criteria for merging the PR? It seems that you have no clear explanation on why you add any specific reason why you blacklist a specific site as a phishing. In the PR #3670, the screenshot evidence was just a window showing that user can use subscan to connect to the site without any explanation why this is consider phishing. Furthermore, the Polkadot.js site doesn't provide a choice for users to access the website like Talisman does.
My concern to the above comment is that without a proper checking on why a site is consider scam, it can lead to many false positives when detecting a phishing site.

The screenshots in the PR aren't for proof that something is a scam but for our own administration they are targetting the polkadot ecosystem.
As for proper checking, we report hundreds of sites each month, sometimes, very rarelly a legit sites comes in. The connext site in specifc was reported by some other third party reported I also check and it honestly did look very scammy to me on first sight and tbh it still does but I did some double checking and it apears to be legit indeed so I removed this one now.
Btw the arkham one really is a scam. The real one is arkhamintelligence.com. As for your last note in over a year of reporting thousands of scam sites this is just the second times a legit site has been added to the repo by my knowledge. I consider that pretty good.

Just to add, since this is also an open-source list, anyone is more than welcome to contribute to report for real scams and even flag false positives (like what you just did).
We appreciate your report and honestly our response time was quite fast. In other anti-scam filters / wallets, it'd probably take several working days to resolve this situation (vs. what we did in several hours, on the weekend).


xrepo install brew::llvm fails on Linux

Xmake Version
2.8.2
Operating System Version and Architecture
Ubuntu 22.04.3 LTS
Describe Bug
Invoking xrepo install brew::llvm on Linux fails. The command actually succeeds in installing the package through brew (see below) but the check by xrepo for success afterwards fails for some reason.
However, the same command did work when I tested it on my macOS machine.
Expected Behavior
The command should succeed (on all platforms where Homebrew is supported).
Project Configuration
No project configuration.
Additional Information and Error Logs
xrepo install --verbose --diagnosis brew::llvm

xmake f -c --require=n -vD
checking for platform ... linux
checking for architecture ... x86_64
checking for gcc ... /usr/bin/gcc
checkinfo: cannot runv(zig version), No such file or directory
checking for zig ... no
checkinfo: cannot runv(zig version), No such file or directory
checking for zig ... no
configure
{
    arch = x86_64
    mode = release
    buildir = build
    ccache = true
    kind = static
    plat = linux
    clean = true
    ndk_stdcxx = true
    host = linux
}
xmake require -v -D -j 34 --extra={system=false} brew::llvm
checking for unzip ... /home/linuxbrew/.linuxbrew/bin/unzip
checking for git ... /usr/bin/git
checking for gzip ... /usr/bin/gzip
checking for tar ... /usr/bin/tar
updating repositories ..
pulling repository(build-artifacts): https://gitlab.com/xmake-mirror/build-artifacts.git to /home/silvanshade/.xmake/repositories/build-artifacts ..
/usr/bin/git -c core.fsmonitor=false pull origin main
From https://gitlab.com/xmake-mirror/build-artifacts
 * branch            main       -> FETCH_HEAD
Already up to date.
pulling repository(xmake-repo): https://gitlab.com/tboox/xmake-repo.git to /home/silvanshade/.xmake/repositories/xmake-repo ..
/usr/bin/git -c core.fsmonitor=false pull origin master
From https://gitlab.com/tboox/xmake-repo
 * branch              master     -> FETCH_HEAD
Already up to date.
ok
finding llvm from brew ..
checking for brew ... /home/linuxbrew/.linuxbrew/bin/brew
checking for brew::llvm ... no
note: install or modify (m) these packages (pass -y to skip confirm)?
in brew:
  -> brew::llvm latest
please input: y (y/n/m)
y
installing llvm from brew ..
/home/linuxbrew/.linuxbrew/bin/brew install llvm --verbose
==> Downloading https://ghcr.io/v2/homebrew/core/llvm/manifests/16.0.6
Already downloaded: /home/silvanshade/.cache/Homebrew/downloads/cb3d2cfe2abb4c6576f80b85db0bfc78e816451626f7c81b9fbd5747bd5de66c--llvm-16.0.6.bottle_manifest.json
==> Fetching llvm
==> Downloading https://ghcr.io/v2/homebrew/core/llvm/blobs/sha256:eac29c15c4e506aa90ab2b95d2845fea812aee43a957e3b0ce807d30cad8a254
Already downloaded: /home/silvanshade/.cache/Homebrew/downloads/bd4783f43a9c343893db91f44bb8399e7135cb0efe985816fc463d441fed2292--llvm--16.0.6.x86_64_linux.bottle.tar.gz
==> Verifying checksum for 'bd4783f43a9c343893db91f44bb8399e7135cb0efe985816fc463d441fed2292--llvm--16.0.6.x86_64_linux.bottle.tar.gz'
==> Pouring llvm--16.0.6.x86_64_linux.bottle.tar.gz
/usr/bin/env tar --extract --no-same-owner --file /home/silvanshade/.cache/Homebrew/downloads/bd4783f43a9c343893db91f44bb8399e7135cb0efe985816fc463d441fed2292--llvm--16.0.6.x86_64_linux.bottle.tar.gz --directory /tmp/d20230908-31228-ro2mop
/usr/bin/env cp -pR /tmp/d20230908-31228-ro2mop/llvm/. /home/linuxbrew/.linuxbrew/Cellar/llvm
==> Finishing up
[omitted]
==> Caveats
Emacs Lisp files have been installed to:
  /home/linuxbrew/.linuxbrew/share/emacs/site-lisp/llvm
==> Summary
🍺  /home/linuxbrew/.linuxbrew/Cellar/llvm/16.0.6: 6,881 files, 2GB
==> Running `brew cleanup llvm`...
Disable this behaviour by setting HOMEBREW_NO_INSTALL_CLEANUP.
Hide these hints with HOMEBREW_NO_ENV_HINTS (see `man brew`).
finding llvm from brew ..
checking for brew::llvm ... no

error: .../modules/private/action/require/impl/actions/install.lua:393: fetch brew::llvm-latest failed!
stack traceback:
    [C]: in function 'error'
    [@programdir/core/base/os.lua:921]: in function 'raiselevel'
    [@programdir/core/sandbox/modules/utils.lua:149]: in function 'assert'
    [.../modules/private/action/require/impl/actions/install.lua:393]:

  => install brew::llvm latest .. failed

.../modules/private/action/require/impl/actions/install.lua:393: fetch brew::llvm-latest failed!
stack traceback:
    [C]: in function 'error'
    [@programdir/core/base/os.lua:921]: in function 'raiselevel'
    [@programdir/core/sandbox/modules/utils.lua:149]: in function 'assert'
    [.../modules/private/action/require/impl/actions/install.lua:393]:
if you want to get more verbose errors, please see:
  -> /tmp/.xmake1000/230908/xrepo/working/build/.packages/b/brew_llvm/latest/cache/installdir.failed/logs/install.txt
error: @programdir/modules/private/async/runjobs.lua:256: .../modules/private/action/require/impl/actions/install.lua:469: install failed!
stack traceback:
    [C]: in function 'error'
    [@programdir/core/base/os.lua:921]:
    [.../modules/private/action/require/impl/actions/install.lua:469]: in function 'catch'
    [@programdir/core/sandbox/modules/try.lua:123]: in function 'try'
    [.../modules/private/action/require/impl/actions/install.lua:328]:
    [...modules/private/action/require/impl/install_packages.lua:479]: in function 'jobfunc'
    [@programdir/modules/private/async/runjobs.lua:232]:

stack traceback:
        [C]: in function 'error'
        @programdir/core/base/os.lua:921: in function 'base/os.raiselevel'
        (...tail calls...)
        @programdir/modules/private/async/runjobs.lua:256: in field 'catch'
        @programdir/core/sandbox/modules/try.lua:123: in global 'try'
        @programdir/modules/private/async/runjobs.lua:218: in upvalue 'cotask'
        @programdir/core/base/scheduler.lua:404: in function <@programdir/core/base/scheduler.lua:397>
error: @programdir/core/main.lua:300: @programdir/core/sandbox/modules/os.lua:378: execv(xmake require -v -D -j 34 --extra={system=false} brew::llvm) failed(255)
stack traceback:
    [C]: in function 'error'
    [@programdir/core/base/os.lua:921]:
    [@programdir/core/sandbox/modules/os.lua:378]:
    [@programdir/modules/private/xrepo/action/install.lua:300]: in function '_install_packages'
    [@programdir/modules/private/xrepo/action/install.lua:307]:
    [@programdir/modules/private/xrepo/main.lua:196]:
    [@programdir/plugins/lua/main.lua:123]:
    [C]: in function 'xpcall'
    [@programdir/core/base/utils.lua:280]:
    [@programdir/core/base/task.lua:501]: in function 'run'
    [@programdir/core/main.lua:298]: in function 'cotask'
    [@programdir/core/base/scheduler.lua:404]:

stack traceback:
        [C]: in function 'error'
        @programdir/core/base/os.lua:921: in function 'base/os.raiselevel'
        (...tail calls...)
        @programdir/core/main.lua:300: in upvalue 'cotask'
        @programdir/core/base/scheduler.lua:404: in function <@programdir/core/base/scheduler.lua:397>

why not use add_requires("llvm")?

Yeah, your suggestion might be a better solution for that package.
I was just trying to understand  why brew:: packages didn't work on Linux because I was planning to use some custom packages I had already written formulas for Hombrew.

checking for brew::llvm ... no

find brew::llvm failed after it has been installed.
It will find llvm libraries from brew::llvm pkg-config.
But brew/llvm can have a lot of pc files, and xmake has no way of knowing which libraries to look for, or whether to look for binary or libraries.
Therefore, if you install llvm from the xmake-repo repository, it will be better handled.

It will find llvm libraries from brew::llvm pkg-config.
But brew/llvm can have a lot of pc files, and xmake has no way of knowing which libraries to look for, or whether to look for binary or libraries.

I understand.
It is strange that it works on macOS but doesn't work on Linux.
I did try to debug the problem and was able to trace back to this line, where pkgdir was null:
        local pkgdir = find_path("lib", path.join(brew_pkg_rootdir, nameinfo[1], "*"))

For some reason, the call to find_path there fails on Linux.
From there, I was able to trace back further to this line:
    return os.find(rootdir, pattern, recursion, mode, excludes, callback)

I think for some reason this call is not working in this case on Linux, but don't know why.
I couldn't trace back any further because I wasn't sure where the definition for os.find was located. Maybe this comes from the C code?

local pkgdir = find_path("lib", path.join(brew_pkg_rootdir, nameinfo[1], "*"))

you can print(path.join(brew_pkg_rootdir, nameinfo[1])) and find lib directory in it.

I couldn't trace back any further because I wasn't sure where the definition for os.find was located. Maybe this comes from the C code?

right, it is from C code.

I have fixed it. xmake update -s dev

Awesome, thanks!


Add "blog" section

Separate to release page, a blog page for updates on new features, "in the news" etc

@SchrodingersGat I made a mockup for a bigger website some months ago - maybe useful for a long term goal?
https://matmair.github.io/home.invenhost.com/

Yeah that's nice. It would be great for inventree.org to display that

Cool concept, would make it a legit project :+1:

I think we should discuss what to but on there but would love to do the site itseld.
The pages are done in yaml / markdown just like mkdocs. I will give you access to the source code, we should transfer it to the inventree-org when you deem it ready.

Sounds like a great thing to look at after the 0.6.0 release!

For sure!

Closing out this issue, makes much more sense for this to be on the website rather than the docs pages


Fix usage of OpenGL SVG methods that no longer exist

Remove imports from the (now nonexistent)
manim/mobject/svg/opengl_svg_mobject
manim/mobject/svg/opengl_svg_path
Instead use SVGMobject and SVGPathMobject.

Overview: What does this pull request change?

Motivation and Explanation: Why and how do your changes improve the library?

Links to added or changed documentation pages

https://manimce--2400.org.readthedocs.build/en/2400/
Further Information and Comments

Reviewer Checklist

[ ] The PR title is descriptive enough for the changelog, and the PR is labeled correctly
[ ] If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
[ ] If applicable: newly added functions and classes are tested

Yep, I also think that opengl_text_mobject.py and opengl_tex_mobject.py can be removed. They are not even imported when running from manim.opengl import *, see manim/opengl/__init__.py.

Sounds good, I'll make a new PR for that then. 🧹


Caching promises doesn't work as expected

Using Angular 1.5.8 and angular-cache 4.6
In my service:
    if (!CacheFactory.get('projectCache')) {
      CacheFactory.createCache('projectCache', {
        deleteOnExpire: 'aggressive',
        recycleFreq: 60000
      });
    }
    projectCache = CacheFactory.get('projectCache');

    return {
      getProject: function(id, slug) {
        return $q(function(resolve, reject) {
          $http.get(apiCall, { cache: projectCache })
          .success(function(data) {
            resolve(data);
          }).error(function(data) {
            reject("Server didn't send the correct data");
          });
        });
      }

In my localStorage the value is stored like this:
cachedResp: Array[4]
    0: 200
    1: "Stringified JSON object"
    2: Object
    3: "OK"

It's basically the same issue here https://github.com/jmdobry/angular-cache/issues/115 and here https://github.com/jmdobry/angular-cache/issues/143 but nothing fix it

I think it might be that Angular.js 1.x changed the format of the $http response data that it saves to the cache. Ugh, angular-cache would have to change its behavior depending on which version of Angular is being used.

I tried searching Angular docs for any documentation for that update but I didn't find any.

Same here. Yep seems that response format was changed...

Same here. Yep seems that response format was changed...

Note that starting from 1.6, $http deprecated callback methods: success()/error() were removed. I am not sure how that would affect promises

For our i'm just added wrapper around CacheFactory, to drop token headers from the cache.
Currently sometimes $http uses cache.put with Promise and sometimes with plain Array.

Facing the same issues here, using angular 1.6.7
1 - $http now does not have sucess() / error() methods anymore.
2 - If the value for an URL inside the cache is an array, angular will use the cached array value arbitrarily.
//extract from the $http service file of angular v1.6.7
if (cache) {
        cachedResp = cache.get(url);
        if (isDefined(cachedResp)) {
          if (isPromiseLike(cachedResp)) {
            // cached request has already been sent, but there is no response yet
            cachedResp.then(resolvePromiseWithResult, resolvePromiseWithResult);
          } else {
            // serving from cache
            if (isArray(cachedResp)) { //<---- this will be executed case cached value is an array
            //notice the usage of fixed indexes here....
              resolvePromise(cachedResp[1], cachedResp[0], shallowCopy(cachedResp[2]), cachedResp[3], cachedResp[4]);
            } else {
              resolvePromise(cachedResp, 200, {}, 'OK', 'complete');
            }
          }
        } else {
          // put the promise for the non-transformed response into cache as a placeholder
          cache.put(url, promise);
        }
      }

If you replace the CacheFactory for the angular $cacheFactory, and cache an array, the same problem does not happen.

Investigating further, the reason it works with $cacheFactory for caching an array is simply the way angular creates the cached value.

$http creates a Promise of the http call.
$http will checks the cache (if a cache is available).

If the cache do not have a cached value:
2) $http will put the promise object in the cache (resolved or not)
3) $http wil return the promise.
Notice that eventually that promise will resolve, and the cache will have it resolved for the next call.
If the cache has a cached value:

If the cached value is a Promise, chain the Promise (resolved or not) with the method resolvePromiseWithResult()
If the cached value is an Array, $http will (arbitrarily) use the cached value to resolve an Promise object with the cached array value. (ERROR HERE)
If the cached value is anything else,  $http will resolve an Promise pass the cached value as the response content.

    function sendReq(config, reqData) {
  ....
if (cache) {
        cachedResp = cache.get(url);
        if (isDefined(cachedResp)) {
          if (isPromiseLike(cachedResp)) {
            // cached request has already been sent, but there is no response yet
            cachedResp.then(resolvePromiseWithResult, resolvePromiseWithResult);
          } else {
            // serving from cache
            if (isArray(cachedResp)) {
            //notice the usage of fixed indexes here....
              resolvePromise(cachedResp[1], cachedResp[0], shallowCopy(cachedResp[2]), cachedResp[3], cachedResp[4]);
            } else {
              resolvePromise(cachedResp, 200, {}, 'OK', 'complete');
            }
          }
        } else {
          // put the promise for the non-transformed response into cache as a placeholder
          cache.put(url, promise);
        }
      }
  ...
}

     function resolvePromiseWithResult(result) {
        resolvePromise(result.data, result.status, shallowCopy(result.headers()), result.statusText, result.xhrStatus);
      }

      /**
       * Resolves the raw $http promise.
       */
      function resolvePromise(response, status, headers, statusText, xhrStatus) {
        //status: HTTP response status code, 0, -1 (aborted by timeout / promise)
        status = status >= -1 ? status : 0;

        (isSuccess(status) ? deferred.resolve : deferred.reject)({
          data: response,
          status: status,
          headers: headersGetter(headers),
          config: config,
          statusText: statusText,
          xhrStatus: xhrStatus
        });
      }

I've solved the issue passing the full Promise Object generated by the $http service to the cache.
Angular will check fot the success of the Promise when resolving it on the resolvePromise() method of the $http service. (check code in the my comment above)
angular.module('app', ['angular-cache']).config(function (CacheFactoryProvider) {
angular.extend(CacheFactoryProvider.defaults, {
  maxAge: 3600000,
  deleteOnExpire: 'aggressive',
  onExpire: function (key, value) {
    var _this = this; // "this" is the cache in which the item expired
    _this.put(
      key, //the cache key (URL)
      angular.injector(['ng']).get('$http'.get(key)) //an Promise object of the http call
    );
  }
});
});
}


On User Page, Add "Membership Type" dropdown

When on a /users/<#>/edit page, beneath "Role", add a new dropdown selection with the user's currently assigned membership_type (@pollygee - did I word this in a helpful way?).
The header for this should be "Membership Type
The dropdown list should populate all membership types that can be selected in /membership_types and allow the Volunteer/Admin to select any single option.

I did this as part of displaying it on the dropdown for checkout.  I haven't done anything with the seeds though, I'll try to add that too as part of this.


Feature Request: Run a "sanity-check" program

Hello!
This is a very great project! I have been looking for a much smaller but still capable process supervisor compared to PM2 and this looks like it is exactly what i need :)
I would like to request/suggest a feature: Allow the user to specify a "sanity check". For instance, imagine this shell script:
#!/bin/bash

check() {
  command -v $1 >/dev/null
  if [ $? != 0 ]; then
    echo "$1 was not found!"
    exit 1
  fi
}

check go
check node

If either of these fail, the script exits with exit code 1.
What I would like to do is to run this before any of the processes are invoked as a way to introduce a script that verifies the environment if everything is exactly where it should be. :)
Again thanks a lot for this project, been playing around with it today a lot and it's great!
Kind regards,
Ingwie

Good morning!

Globally. It should just make sure all things are where and how they should be.
No, it should run before all processes. If it returns true, assume that everything is fine and proceed to start the processes.
This could be considered - but I think it'd be better if a per-process check failed, it should only mean that this process can not be started and should only affect dependees.
Depends on if the user supplied arguments or not. Like if they only want to run one process out of many, I think the global check should still run. You could consider passing a PROCESS_COMPOSE_PROCS environment variable to tell the check if it should limit its checks or not.

Feel free to ask! I am not very good with Go, so there is not that much I can contribute. More of a C and Nodejs guy. ^^;

Good afternoon.
I think I have a clearer picture of what you want to achieve.
I think that even today this is almost possible if you define the first process to be this check and all other processes are dependent on it. Or at least the first process in the chain is dependent on it.
The only thing that is missing is the ability to exit on process failure.
What do you think about the following:

I will add support to configure an exit strategy if the process exit code != 0. This will be optional. If not configured the behavior will be as today.
If configured and exit code != 0 the process-compose will gracefully terminate all the other running processes (in case of the preflight check there shouldn't be any) and exit with the same exit code as the check process.
As today, you will be able to disable/enable the check process in configuration.
As a separate feature, I will add the support to run specific processes by providing a CLI argument to process compose. This way you will be able to run all the checks individually and validate their correctness before running all the other processes.
There is a downside to this approach - the check process(es) will be visible in the TUI, I am not sure if this can be a deal breaker for someone.

What do you think?
You are contributing a lot by using the project and thinking about new use cases 😊

Hey!

I think that even today this is almost possible if you define the first process to be this check and all other processes are dependent on it

Good point, didn't think of that. So basically:
sanitycheck:
  command: ./scripts/run-checks.sh

other_proc:
  command: ...
  depends_on:
    sanitycheck:
      condition: process_completed_successfuly

Like so?

What do you think about the following:

I like 2 a lot. 5 is not a big problem really - at least I don't think so.

You are contributing a lot by using the project and thinking about new use cases 😊

Haha, thanks! Most welcome as well. :)

Hey @IngwiePhoenix
Yes, exactly! Your code snippet sums up the idea.
BTW, even today the processes won't run if all of them are process_completed_successfuly dependent on sanitycheck.
I will implement 1-4.

Thanks for the great issue @IngwiePhoenix!
My 2 cents on this as user is that instead of trying to treat the symptoms of an incorrect environment in process-compose, the problem should be solved externally, by providing the correct environment reproducibly and proactively.
Great tools for this purpose are Docker/Nix/Guix plus a bajillion other tools I am sure.
This way process-compose does not have to get bloated with a feature-set that does not really solve the problem.
Example:

what if the go binary is present on the PATH, but is actually not the proper version?
What if node was compiled with an older libv8 than needed?
What if the MacOS/Linux execution environment distinction is important for OS-specific feature reasons and require separate binaries?

Handling this with bespoke shell scripts gets tiring/error-prone really quick, and process-compose can only short-circuit a faulty start.
Even so a simple bash function can accomplish this today:
# put this into direnv/virtualenv/whatever
export PRJ_ROOT=$(pwd)
pcompose() { $PRJ_ROOT/bin/check.sh && process-compose "$@" }; }

$ pcompose completion --help
CHECK: All good!
Generate the autocompletion script for process-compose for the specified shell.
See each sub-command's help for details on how to use the generated script.

Leaving you with the option to easily opt-out of the check by just invoking process-compose directly.

Hi @thenonameguy,
I agree with your approach for a local, single developer environment. It is clear and keeps the process-compose lighter and its configuration simpler.
However, for the multi developers environment, you would want to share your compose.yaml file and expect it to function in a similar manner, even if they run it without the wrapper script.
This can be achieved by adding the relevant checks and dependencies to the processes execution chain and, if needed, termination of process-compose (new functionality).
Something similar to what I proposed to @IngwiePhoenix :
sanitycheck:
  command: "which go"
  availability:
    restart: "exit_on_failure" #new functionality

other_proc:
  command: ...
  depends_on:
    sanitycheck:
      condition: process_completed_successfuly

Hey @thenonameguy !

instead of trying to treat the symptoms of an incorrect environment in process-compose, the problem should be solved externally

Not wrong, I agree! It's an idea I had to bring down the external programms needed mainly. I go between platforms a lot; my laptop does not have WSL or Msys - only Docker. My Desktop has WSL with Nix and Docker and my server is a pure Debian box with only Docker, no Nix. So, for "between platform goers", it might be easier to have that feature, hence why I suggested it.
Hey @F1bonacc1 ! Just noticed you linked a PR and subsequent commits; I'll goa nd check out the new additions!
Sorry for my late response, got a little side-tracked IRL... :)

Hi @IngwiePhoenix,
I added the ability to exit if the process fails exit_on_failure. When this is configured, process-compose will exit with the same exit code as the failed process.
This functionality is part of v0.21.0 release.
Please let me know if it works as expected for you.


Fix typos in security overview

typo (sets -> set)
typo (trainings -> training) [training is an uncountable noun]

Well spotted, thanks very much!


Add whatsnew entry for #8974

Responding to call-out at https://github.com/pydata/xarray/pull/9021 !

@dcherian one issue with the CI change is that these are required checks! So the PRs won't merge without them...
We had to do some galaxy-brain stuff at PRQL to get around this: https://github.com/PRQL/prql/blob/d29f102fafe2498a88aebfa90989c638843b8f99/.github/workflows/tests.yaml#L523...

Perhaps it's ok to manually click merge on these docs only PRs?
IDK


404 Not Found

I'm sure I just havent properly configured asterisk but for some reason I am not able to call any of the API's.
Testing the examples on the documentation all return 404. The resources.json url returns a 404.
wscat -c "ws://<IP_ADDRESS>:8088/ari/events?api_key=asterisk:asterisk&app=hello-world"

error: Unexpected server response: 404

I followed the instructions at this website for enabling ari.
https://wiki.asterisk.org/wiki/display/AST/Getting+Started+with+ARI#GettingStartedwithARI-ConfiguringAsterisk

Enabled http.conf and binding
Enabled ari.conf and created a user
Setup a dialplan
Restarted asterisk

I do get responses on port 8088 but none of the URL's I try to hit work.

Hi, I just managed to solve a similar issue: it turned out that asterisk wasn't loading the ari module. I had to modify modules.conf to fix this.

Hi @jeremywhittington , where's the ari.conf file located? can you mention the directory name, after running npm install wscat.


Provide a way to download *.proto files from DocService

providing *.proto files used in a armeria server would be helpful for cases with no common support/policy for proto file management(ex: stub packaging, etc) established.
example api:
DocService.builder()
    .provideProtos(paths = listOf("foodir/**")) //path regex list
    .build()

provide protocol buffer file directories(or maybe packages?) when building the DocService, then Armeria filters & compresses all *.proto files in these paths and serve it on a 'download protocol buffer files' button click inside docs page or via static path.

That's a great idea, @j-min5u. I wonder if the content of the .proto file is already available somewhere inside the generated class though. If that's the case, we probably don't need to add the files explicitly, although allow adding them manually can sometimes be useful.

@trustin That would be really nice if possible, but the provided proto files would also have to maintain their file structure in addition to their content to be fully compilable considering local imports. It seems the only way to do this is through manual directory/file scan.


Using SqueezeNet on ES6 fails

I'm very interested in this library and image recognition possibilities of deeplearning.
Using Vue.js2 and ES6, I'm trying to reproduce the squeezenet cat-demo but fail to do so.
Here is my component code: this.$refs.toAnalyse point to the cat picture on the DOM =>
        import { NDArrayMathGPU } from 'deeplearn'
        import { SqueezeNet } from 'deeplearn-squeezenet'

        export default {
          name: 'HomePage',
          methods: {
            executeAnalyse () {
              const imgToAnalyse = this.$refs.toAnalyse

              const math = new NDArrayMathGPU()
              const squeezeNet = new SqueezeNet(math) // Line throwing the error
            }
          }
        }

This results in the following error : Uncaught TypeError: deeplearn_1.initializeGPU is not a function
In (unrelated?) news, it seems like the live demo linked earlier on the deeplearn.js website is not working anymore (also throws a TypeError: TypeError: Cannot read property 'texShape' of undefined). It used to work on my computer when I first tried it about a month ago.
Thank you very much for any help.

Btw I digged the issue quite some time, I can't find any information except that it probably comes from the deeplearn-squeezenet library :
Object.defineProperty(exports, "__esModule", { value: true });
var deeplearn_1 = require("deeplearn");
// code omitted
var SqueezeNet = (function () {
    function SqueezeNet(math) {
        this.math = math;
        this.preprocessOffset = deeplearn_1.Array1D.new([103.939, 116.779, 123.68]);
        deeplearn_1.initializeGPU(this.math.getGPGPUContext(), this.math.getTextureManager()); // this line might be the culprit
    }
// lots of code omitted

Seems like the same problem as #445. From what I saw there are a lot of changes going on generalising the math backends at the moment.
I would recommend to checkout version 0.13.2 if you want to play around a bit before this gets fixed

@chaosmail Thank you for the answer. Do you mean version 0.3.12 ?

Yeah exactly, sorry about that!

Was able to make it work indeed with version 0.3.12, thanks!


Create a docker image for rust-teos

We haven't ported the docker image from python-teos after migrating. I think it may be worth doing.
The image can borrow extensively from the one we built for python.
Related to #155
Resources
Dockerfile: https://github.com/talaia-labs/python-teos/blob/master/docker/Dockerfile
entrypoint: https://github.com/talaia-labs/python-teos/blob/master/docker/entrypoint.sh
docs: https://github.com/talaia-labs/python-teos#running-teos-in-a-docker-container

Fixed via https://github.com/talaia-labs/rust-teos/pull/227


kernel-uvm: Update to <IP_ADDRESS>

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

Update kernel-uvm to newest <IP_ADDRESS> kernel source. In addition to this being a newer LTS release, it also includes changes to fix a 5 second delay caused by VMBUS when no ACPI devices are present, which occurs in the case of the UVM.
Signed-off-by: Chris Co<EMAIL_ADDRESS>Change Log

Does this affect the toolchain?

NO
Associated issues

41340169

Links to CVEs
Test Methodology

BuddyBuild - 250739

Have we done a functional test (ideally want to test for both hyperV and cloud)? I see only a 'build' test in the description. Thanks


修复LayerNorm中文文档

PR types
Bug fixes
PR changes
Docs
Describe
修改了参数weight_attr ，bias_attr 的描述，使其与英文描述一致
修改g为增益参数、b为偏置参数
根据代码执行结果增加return内容

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


readme: add table of contents

As the readme is rather long, I suggest to add a table of contents right after the Introduction section.
Makes navigating the documentation quite a bit more convenient.
At the moment this was added manually (using ekalinin/github-markdown-toc), but it could be automated.

See this for the result.

Way. I am a bit speechless.
Regards Henrik
On 1 Sep 2019, at 12.35, G.A. vd. Hoorn <notifications<EMAIL_ADDRESS>wrote:
See thishttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frosin-project%2Frxros%2Fblob%2F02e598f66b4484b0183b4b0829cba458a4ec754f%2FREADME.md&data=02|01||1846e994961d4f59122008d72ec82004|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637029309368510960&sdata=OERPNJsGnfwrIBvMzC%2F1PjTWNQQaro2JnDMvhdZHcBw%3D&reserved=0 for the result.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frosin-project%2Frxros%2Fpull%2F21%3Femail_source%3Dnotifications%26email_token%3DAJYMVX3P6ECNAQUCIAHRSPDQHOLHPA5CNFSM4ISWLLJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5T7MIA%23issuecomment-526906912&data=02|01||1846e994961d4f59122008d72ec82004|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637029309368520965&sdata=X%2F1fFvkuO86blAZGlUA9aBKm642MD24WNGH5XOEylIY%3D&reserved=0, or mute the threadhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJYMVX34ZQYDOB3MAFV5M23QHOLHPANCNFSM4ISWLLJA&data=02|01||1846e994961d4f59122008d72ec82004|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637029309368530970&sdata=%2BjUvW6TN%2BEQjNO0oP7Ak%2B5AIZ3RLjNhpRn%2Bt%2FRWvB%2FU%3D&reserved=0.

If you're OK with this addition @henrik7264, please approve the PR.


graphql voyager does not have select root type

Hello! My graphql voyager does not displaying select root type
I check the schema in a docker container and the schema is generated correctly, but I cant select mutation on graphql voyager
dockerfile
FROM node:alpine as builder
RUN npm install -g graphql-schema-utilities
ADD graphql-schema/ ./graphql-schema/
RUN graphql-schema-utilities -s "{./graphql-schema/**/*.graphql,./graphql-schema/users/*.graphql}" -o "/app/schema.graphql"

FROM graphql/voyager
COPY --from=builder /app/schema.graphql /app/schema.graphql


COS-40: Add cosmetics worker service

Sorry this branch is a bit messy, it's been merged in and out a fair amount.
It sets up the cosmetics-worker service, and makes loading the Cosmetics website send a job to the worker through Redis that logs some text to the console.

Sorry about the mess this was before, I've cherry-picked everything on top of master so it should be a lot clearer now hopefully.

Wasn't intentional to change file permissions, should be fixed now

Pull Request Test Coverage Report for Build 1995

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at ?%

Files with Coverage Reduction
New Missed Lines
%

web/app/mailers/notify_mailer.rb
3
33.33%

Totals

Change from base Build 1993:
0.0%

Covered Lines:

Relevant Lines:
0

💛  - Coveralls


Example to Parse HTTP Response

Can you please put an an example/usage to parse the HTTP Respnonse.
It will be much helpful.

expecting as well

Did any one of you have the sample code?

the test ?

I think benchmark could be more useful: https://github.com/nodejs/http-parser/blob/master/bench.c


Tentatively fix unmarshal error

Proposed changes
This PR tentatively fixes unmarshal error in types command for Orion-LD.
$ ngsi list --host orion-ld types
typesListLd004 json: cannot unmarshal array into Go value of type ngsicmd.entityTypeList Field: (1) []

NGSI-LD spec specifies that GET /ngsi-ld/v1/types returns a EntityTypeList type, unless details=true is specified.
But it returns an empty JSONarray when EntityTypeList is empty.
$ curl orion-ld:1026/ngsi-ld/v1/types
[]

Types of changes
What types of changes does your code introduce to the project: Put an x in the boxes that apply

[X] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Update only documentation, not any source code.

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of
them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before
merging your code.

[X] I have read the CONTRIBUTING doc
[X] I have signed the CLA
[X] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
N/A

Coverage remained the same at 100.0% when pulling 64f4a72e4c49fa7e70882b55aef10428f92b307d on fisuda:fix/types into a144421a5fb11a070b4da57d39202f1907e369b1 on lets-fiware:main.


dependecy problem reportlab - allthough installed...

# ./OCRmyPDF.sh /home/ao/Leerungstermine189973.PDF /home/ao/test.pdf
Please install the python library reportlab.
Debian 6 squeeze

OCRmyPDF v3.0-rc2 needs reportlab >= 3.0

Actual problem with

Opensuse Tumbleweed
python-reportlab is installed
OCRmyPDF still says "Please install the python library reportlab. Exiting...

Having the problem and reading this issue I checked the Tumbleweed reportlab version (2.7-3.3) and
suggest
to change the OCRmyPDF dependency-checker test to "Please install the python library reportlab version >= 3.0". i.e. to notify the user the they must install a correct version.

@jbarlow83 pls. can you explain, where the option  --pdf-renderer tesseract is to be added ? It does not work on a command line of OCRmyPDF

Only the new version (a pre-release) supports it:
https://github.com/fritz-hh/OCRmyPDF/releases  version v3.0-rc2.
Or download the source latest from the "master" branch.
On Tue, 28 Jul 2015 at 10:50 Wikinaut<EMAIL_ADDRESS>wrote:

@jbarlow83 https://github.com/jbarlow83 pls. can you explain, where the
option --pdf-renderer tesseract is to be added ? It does not work on a
command line of OCRmyPDF
—
Reply to this email directly or view it on GitHub
https://github.com/fritz-hh/OCRmyPDF/issues/99#issuecomment-125692663.

hmm, one problem is solved (when checking out here, the default branch is v2.x. i changed this now to master) ...
but now I get
# sh ./OCRmyPDF.sh -h
Traceback (most recent call last):
  File "/usr/lib64/python3.4/runpy.py", line 170, in _run_module_as_main
    "__main__", mod_spec)
  File "/usr/lib64/python3.4/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/usr/local/src/OCRmyPDF/ocrmypdf/main.py", line 16, in <module>
    import PyPDF2 as pypdf
ImportError: No module named 'PyPDF2'

When I run
./OCRmyPDF.sh -h
bash: ./OCRmyPDF.sh: Keine Berechtigung

All files and subdirectories belong to the current user.

It's a Python 3 package now. Run the installer in the current directory: pip3 install -e .

oops, while you wrote your answer, I read the readme and did the pip3, but:
  FileNotFoundError: [Errno 2] No such file or directory: 'mutool'

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /usr/local/src/OCRmyPDF

So I have to install this, too. (should it be added to the dependency checks ???)

uh, not in Tumbleweed. Have to look for it. (Even Tesseract is easier to install)

It's mupdf-tools. If it's a pain to get, how is qpdf? Both do the same thing.
Hmm, it looks like some of the dependency checks aren't running in the -rc2 release.

It is part of mupdf in Opensuse

Now this is gone, but I have
During handling of the above exception, another exception occurred:

    Traceback (most recent call last):
      File "<string>", line 20, in <module>
      File "/usr/local/src/OCRmyPDF/setup.py", line 167, in <module>
        package='mupdf-tools'
      File "/usr/local/src/OCRmyPDF/setup.py", line 119, in check_external_program
        error_unknown_version(program, package, optional, minimum_version)
    TypeError: error_unknown_version() takes 3 positional arguments but 4 were given

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /usr/local/src/OCRmyPDF

(post above updated with the complete output)

Added a possible fix - do a git pull.

I was (I am) already on 6e6f918630bba7077ba9a50d75a138767422bce7 . This gives the above error.

Apologies, I pushed it to wrong repo. commit 6901550 should now be available.

6901550065523e575d7d1af609add653b5d2fb91

Different error output:
pip3 install -e .
Obtaining file:///usr/local/src/OCRmyPDF
    Complete output from command python setup.py egg_info:
    Checking for tesseract >= 3.02.02...
    Found tesseract 3.04
    Checking for gs >= 9.14...
    Found gs 9.16
    Checking for unpaper >= 6.1...
    Found unpaper 6.2
    Checking for pdfseparate >= 0.29.0...
    Found pdfseparate 0.33.0
    Checking for java >= 1.5.0...
    Found java 1.8.0
    Checking for mutool >= 1.7a...
    Traceback (most recent call last):
      File "/usr/local/src/OCRmyPDF/setup.py", line 117, in check_external_program
        version = version_scrape_regex.search(result).group(1)
    AttributeError: 'NoneType' object has no attribute 'group'

    During handling of the above exception, another exception occurred:

    Traceback (most recent call last):
      File "<string>", line 20, in <module>
      File "/usr/local/src/OCRmyPDF/setup.py", line 167, in <module>
        package='mupdf-tools'
      File "/usr/local/src/OCRmyPDF/setup.py", line 119, in check_external_program
        error_unknown_version(program, package, optional)
      File "/usr/local/src/OCRmyPDF/setup.py", line 83, in error_unknown_version
        print(unknown_version.format(**locals()), file=sys.stderr)
    KeyError: 'need_version'

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /usr/local/src/OCRmyPDF

Thanks for your patience. Please pull again and it give another shot.

uh, now I have
pip3 install -e .
Obtaining file:///usr/local/src/OCRmyPDF
    Complete output from command python setup.py egg_info:
    Checking for tesseract >= 3.02.02...
    Found tesseract 3.04
    Checking for gs >= 9.14...
    Found gs 9.16
    Checking for unpaper >= 6.1...
    Found unpaper 6.2
    Checking for pdfseparate >= 0.29.0...
    Found pdfseparate 0.33.0
    Checking for java >= 1.5.0...
    Found java 1.8.0
    Checking for mutool >= 1.7a...

    OCRmyPDF requires 'mutool' 1.7a or higher.  Your system has
    'mutool' but we cannot tell what version is installed.  Contact the
    package maintainer.

    This program is REQUIRED for OCRmyPDF to work.  Installation will abort.

    On systems with the aptitude package manager (Debian, Ubuntu), try these
    commands:
        sudo apt-get update
        sudo apt-get install mupdf-tools

    On RPM-based systems (Red Hat, Fedora), search for instructions on
    installing the RPM for mupdf-tools.

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /usr/local/src/OCRmyPDF

mupdf 1.7-1.3

OCRmyPDF is a beast

I don't know what to think when Linux distributions make up arbitrary version numbers that don't follow the package's own conventions, as in this case.
I dropped the version requirement to mupdf 1.7.

better, but still buggy:
   writing manifest file 'ruffus.egg-info/SOURCES.txt'
    running install_lib
    creating /usr/lib/python3.4/site-packages/ruffus
    error: could not create '/usr/lib/python3.4/site-packages/ruffus': Permission denied

    ----------------------------------------
Command "/usr/bin/python3 -c "import setuptools, tokenize;__file__='/tmp/pip-build-1ekvbx92/ruffus/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /tmp/pip-8nnv8idr-record/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /tmp/pip-build-1ekvbx92/ruffus

You need to install as sudo or create a virtual environment with pyvenv and
install to that environment. As the error message says it doesn't have
permission to write /usr/local.
On Tue, Jul 28, 2015 at 13:47 Wikinaut<EMAIL_ADDRESS>wrote:

better, but still buggy:
writing manifest file 'ruffus.egg-info/SOURCES.txt'
running install_lib
creating /usr/lib/python3.4/site-packages/ruffus
error: could not create '/usr/lib/python3.4/site-packages/ruffus': Permission denied
----------------------------------------

Command "/usr/bin/python3 -c "import setuptools, tokenize;file='/tmp/pip-build-1ekvbx92/ruffus/setup.py';exec(compile(getattr(tokenize, 'open', open)(file).read().replace('\r\n', '\n'), file, 'exec'))" install --record /tmp/pip-8nnv8idr-record/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /tmp/pip-build-1ekvbx92/ruffus
—
Reply to this email directly or view it on GitHub
https://github.com/fritz-hh/OCRmyPDF/issues/99#issuecomment-125750003.

With a virtual environment:
pyvenv venv
source venv/bin/activate
pip3 install -e .


[Suggestion] Techstacks

Suggestion
Just leaving a little suggestion, or at least open the discussion about it.
Somehow displaying each company's tech stacks. I believe this would be a useful feature, but I'm afraid it might cluster the simplicity of the list.
Stackshare
Recently I've come across with Stackshare, which might be a way of linking each company's tech stack, without cluttering the interface too much. Adding a symbol such as 💻,🔧 or 🏢 might be a way of representing the link with iconography, not unlike 🚀 already in use.

Solved by #209


Example issue

Example issue: There's a ghost in the code, everything is broken!!!!!

Example comment: Not again!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!


Add table type improvements

Purpose

Removed unused index information from table
Replace any type input to add operation with anydata

Fixes #
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

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

@@           Coverage Diff           @@
##           master   #18358   +/-   ##
=======================================
  Coverage   14.86%   14.86%
=======================================
  Files          48       48
  Lines        1265     1265
  Branches      201      201
=======================================
  Hits          188      188
  Misses       1063     1063
  Partials       14       14

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 18b2804...fdb3cfa. Read the comment docs.


Merged tables should run parallel, not sequential

CREATE TEMP TABLE sample1 AS (SELECT * FROM sample('id1'));
CREATE TEMP TABLE sample2 AS (SELECT * FROM sample('id2'));
SELECT * FROM sample1
MERGE
SELECT * FROM sample2
OUTPUT AS samples;
sample1 and sample2 should run in parallel.

If I remove https://github.com/yahoo/yql-plus/blob/master/yqlplus_engine/src/main/java/com/yahoo/yqlplus/engine/internal/tasks/GraphPlanner.java#L89-L121 merge will run in parallel for each data source.


Discord invite in Readme is invalid

I would like to follow along in Discord, however the invite links in the ReadMe and issue #8 are invalid.
Could you kindly create a new one?
(The expiration time for invite links is set to 7 days as standard and can be changed when creating the invite.)

https://discord.gg/wJqUzYxYXH
Try this. Have changed the invite settings

That worked, thank you very much!


Carlos Cárcamo: new volunteer via Mozilla Science Lab Collaborate

Hello, I found this project very interesting and I want to help with its development, I have already fork the project and added some lines to it :)This issue was created by @carloscarcamo via Mozilla Science Lab Collaborate

Hi @carloscarcamo feel free to send a pull request if you manage to do some good changes. Also it would be nice to see another person participating in the development. I have to say that this project will have a change in technology soon since we want to use React and Redux eventually for the front-end

@carloscarcamo @JaredHawkins sorry guys I cannot meet this week since it is my bday + lots of changes at my current workplace. I will try to schedule something for the next week Jan 25-31

No problem and happy Birthday @anvk

Happy birthday @anvk!

On Jan 21, 2016, at 11:53 AM, Carlos Cárcamo<EMAIL_ADDRESS>wrote:
No problem and happy Birthday @anvk https://github.com/anvk
—
Reply to this email directly or view it on GitHub https://github.com/JaredHawkins/TweetGeoViz/issues/40#issuecomment-173632383.