Enable support of  scope ID for IPv6 link local addresses

Try to resolve issue https://github.com/alexcrichton/curl-rust/issues/568

Thanks! Is it possible to add a test for this?

Thanks! Is it possible to add a test for this?

Unfortunately, I have no idea how it could be tested with unit tests: from rust point of view, it just on more define in build script for a C library, from the other side, the effect of this define could be observed deep in the libcurl library itself. It is possible, to test it in an integration test within specific environment, but it would require much effort to set up. I could just say, that this change is working for my project.

Does connecting with ipv6 not work at all without this? Or could a dummy server be set up with a different link-local ID and then curl tries to connect to it and it would fail if this isn't supported?

Does connecting with ipv6 not work at all without this? Or could a dummy server be set up with a different link-local ID and then curl tries to connect to it and it would fail if this isn't supported?

The issue is related to the IPv6 link-local addresses (all addresses within fe80::/10 range) only.
Just one example: let suppose you have a local web server running with python3 -m http.server -b "::" 8123. Your eth0 interface has a link local IPv6 address fe80::c4:d164:150d:282c. Then you make standard curl calls like this:
curl http://[fe80::c4:d164:150d:282c%eth0]:8123  # works fine, with scope id eth0
curl http://[fe80::c4:d164:150d:282c]:8123       # will not work without scope id

Without the flag (HAVE_SOCKADDR_IN6_SIN6_SCOPE_ID) enabled, both variants would fail.
In the libcurl itself, the HAVE_SOCKADDR_IN6_SIN6_SCOPE_ID flag is defined whenever the system's struct sockaddr_in6 has the field sin6_scope_id (no idea, where it could be missing, maybe some very old systems). See curl's original CMakeLists.txt: https://github.com/curl/curl/blob/curl-8_9_0/CMakeLists.txt#L291


removedUserFromGroup fails with (assuming) no permissions

As you can see from the error below, I have had no luck in using api.removeUserFromGroup(userID, threadID). No matter who I try to kick from group conversations, it doesn't seem to kick them at all, instead saying that I don't have access to the page. This could be due to a fault of my own, but I'm using perfectly valid IDs (such as<PHONE_NUMBER>60303), which are obtained from something like api.getThreadList, or message.author.
ERR! ERROR in removeUserFromGroup -->  { __ar: 1, ERR! ERROR in removeUserFromGroup -->    error: 1357031, ERR! ERROR in removeUserFromGroup -->    errorSummary: 'This content is no longer available', ERR! ERROR in removeUserFromGroup -->    errorDescription: 'The content you requested cannot be displayed at the moment. It may be temporarily unavailable, the link you clicked on may have expired or you may not have permission to view this page.', ERR! ERROR in removeUserFromGroup -->    payload: null, ERR! ERROR in removeUserFromGroup -->    bootloadable: {}, ERR! ERROR in removeUserFromGroup -->    ixData: {}, ERR! ERROR in removeUserFromGroup -->    lid: 'redacted' }

Ok, I've just realised this is my fault. Apologies
Looks like I caught the exception

Awesome.  In the future, here are some things to check (I had already typed them when I saw your response so I figured why not).
Does addUser/sendMessage work using the same threadID?  Can you verify that the user was not, in fact, actually removed from the group even though the call failed?  We've had issues in the past with facebook returning error when the call was successful.


nvim_win_get_position/ nvim_open_win maybe problematic

see https://github.com/Yggdroot/LeaderF/issues/501

nvim_win_get_position returns ['0', '0'].
That is why the heigth is set to -1, resulting in
{'relative': 'editor', 'anchor': 'SW', 'height': -1, 'width': 70, 'row': -1, 'col': 0}
as a paramter to open_win.

Hi
Thanks for this report.
I'm not sure, but is the problem that the echo nvim_win_get_position(win_getid('%')) result is always [0, 0] on the float window? Indeed, I have confirmed that the above result is [0, 0] on the float window displayed by Yggdroot/LeaderF. Also, neovim-qt may return a normal position such as [14, 26]. I think this problem may seems to be a problem on nvim core.
I will continue to investigate.

I reported an issue: https://github.com/neovim/neovim/issues/11935

So, what is the verdict? is leaderF implementation should change?

Yes, according to neovim/neovim#11935, leaderF implementation will need to change.
According to https://github.com/neovim/neovim/issues/11935, I think the following line needs to be changed to nvim_win_get_config():
https://github.com/Yggdroot/LeaderF/blob/d6a6b63eedf2b91c2f5c620b6ea1a5090fc7fee7/autoload/leaderf/python/leaderf/manager.py#L429-L430

this issue fixed with https://github.com/Yggdroot/LeaderF/commit/eb91151972c55f4a6b695387c0bee3fb6d3a7698 and https://github.com/akiyosi/goneovim/commit/8cad126573b3503d42d0b7a69680a8fd13fec08b

This has been fixed in v0.4.5.
please confirm.

This is working up to wrong position. But their bug anyway. Thanks.

Is it still displayed in the wrong position?


runtime: buildmode=c-shared segfaults on linux/arm (ARMv7)

What version of Go are you using (go version)?
% go version
go version go1.5.1 linux/arm

What operating system and processor architecture are you using?
% echo $GOOS/$GOARCH $GOARM
linux/arm 7

My machine is Rasberry Pi 2.
% uname -a
Linux raspberrypi 4.1.7-v7+ #817 SMP PREEMPT Sat Sep 19 15:32:00 BST 2015 armv7l GNU/Linux
% gcc --version | head -n 1
gcc (Raspbian 4.8.4-1) 4.8.4

What did you do?

I tried to build a c-shared object on Raspberry Pi 2, based on the articles below.

Building Go 1.5 on the Raspberry Pi _ Dave Cheney
Building Python modules with Go 1.5 - Filippo.io

Here is Go code exports Sum function.
sum.go
sum.go

package main

import "C"

//export Sum
func Sum(a, b int) int {
		return a + b
}

func main() {}

And the following C code calls Sum function.
main.c
#include <stdio.h>
#include "sum.h"

int main() {
  printf("1 + 2 = %d\n", Sum(1, 2));
  return 0;
}

To build a shared object, I execute the commands below.
% go build -buildmode=c-shared -o ./sum.so ./sum.go
% gcc -c ./main.c
% gcc ./main.o ./sum.so

Then I run ./a.out, it segfaults.
% ./a.out
zsh: segmentation fault  ./a.out

What did I something wrong? Thanks.

FYI, buildmode=c-archive also occurs an error.
% go build -buildmode=c-archive -o ./sum.a ./sum.go
% gcc ./main.o ./sum.a
./sum.a(000001.o): In function `x_cgo_sys_thread_create':
/home/pi/tmp/go1.5/src/runtime/cgo/gcc_libinit.c:20: undefined reference to `pthread_create'
./sum.a(000001.o): In function `_cgo_sys_thread_start':
/home/pi/tmp/go1.5/src/runtime/cgo/gcc_linux_arm.c:25: undefined reference to `pthread_sigmask'
/home/pi/tmp/go1.5/src/runtime/cgo/gcc_linux_arm.c:33: undefined reference to `pthread_attr_getstacksize'
/home/pi/tmp/go1.5/src/runtime/cgo/gcc_linux_arm.c:36: undefined reference to `pthread_create'
/home/pi/tmp/go1.5/src/runtime/cgo/gcc_linux_arm.c:38: undefined reference to `pthread_sigmask'
./sum.a(000001.o): In function `x_cgo_init':
/home/pi/tmp/go1.5/src/runtime/cgo/gcc_linux_arm.c:66: undefined reference to `pthread_attr_getstacksize'
collect2: error: ld returned 1 exit status

So ./sum.a works fine when I passed gcc -pthread option.
% gcc -pthread ./main.o ./sum.a
% ./a.out
1 + 2 = 3

However, ./sum.so segfaults even though I passed gcc -pthread.
% gcc -pthread ./main.o ./sum.so
% ./a.out
zsh: segmentation fault  ./a.out

CC @mwhudson

Can you try this on tip?

@mwhudson What do you mean?

I mean with a version of go built from current git, not the 1.5.1 release. I've changed some things around how c-shared works on ARM since the release and it would be interesting to know if they help.

Oh I'm sorry, I forgot to switch the branch to Go 1.5. Should I close this issue and re-post?

Are you saying your code succeeds with Go 1.5.1 but fails with Go top? Or the other way around? Or something else.
No need for a new issue.

Are you saying your code succeeds with Go 1.5.1 but fails with Go top?

The code does not succeeds with Go 1.5.1. I just tried the code on Go 1.5.1 because it is current release version. Now I test the code by using the latest master branch, the shared object sum.so works fine.
% go version
go version devel +0a486b8 Tue Oct 6 01:11:30 2015 +0000 linux/arm
% go build -buildmode=c-shared -o sum.so sum.go
% gcc ./main.o ./sum.so
% ./a.out
1 + 2 = 3

Thank you.

I've run into the same issue with Go 1.5.1 on linux/arm, and my solution was to add:
-installsuffix=
to my "go build" command.  Why that is required is a mystery to me, but it is required.  (It's not required for linux/amd64 nor darwin/amd64, which we use extensively.)
I bet iant@ knows why this is required as he added these flags to tests in "misc/cgo/testcshared", which is how I found out the solution to my woes.  (It fails in "save_g" otherwise, trying to access the G pointer from TLS.)

Using -installsuffix will force code to be recompiled.  We may be a little confused about when it is needed with other options.  If you still need to use it on tip, please open a different issue about that.


UI Router 1.0.0 Fix

I am noticing that unauthenticatedRedirectPath no longer seems to trigger with the ui router transition code. If I force legacy state events unauthenticated redirect works with unauthenticatedRedirectPath and authManager.redirectWhenUnauthenticated().
https://ui-router.github.io/ng1/docs/latest/modules/ng1_state_events.html
Stepping through, it looks like the unauthenticated redirect is relying on loginPath and loginPath requires app state as opposed to a path.

The library needs to be updated to support UI Router 1.0. It's in the backlog but no ETA yet unfortunately.

Hi @chenkie Actually I submitted pull request for this some time ago (#155 )
I've been succesfully using my fork in production since then with angular-ui-router version ^1.0.3. It would be nice if you could merge / review my PR :)

please fix


N/A professional designation does not show up properly in dropdown or diffs [6]

Describe the Bug:
When a user selects "N/A" as a professional designation, the dropdown goes blank instead of displaying N/A. The same thing happens in the diffs--instead of N/A we see nothing. Question: the summary page shows "Not added"--do we want to that, or N/A?
Probability (how likely the bug is to happen, scored from 1-5): 4
(For example, probability of 5 is something like "it happens to all users every time they log in."
Probability of 1 "only happens to certain users when a really specific and unlikely path is followed.")
Effect (how bad the bug is when it does happen, scored from 1-5): 2
(For example, effect of 5 is "the entire app crashes and makes it unusable for all users" or "the bug causes the wrong data to be saved, with critical information (e.g. payment) being affected."
Effect of 1 is "It makes some styling look a little bit weird.")
Steps to reproduce the behaviour:
Screenshots:

https://www.loom.com/share/3eb126a500e44e94832f89fa4f2d1de6
Additional information:

OS: [e.g. iOS]
Browser: [e.g. chrome, safari]
Version: [e.g. 22]
Additional context

@nanyangpro , when professional designation is set to "N/A", do we want to see "N/A" in the summary and diffs, or do we literally want to see "N/A"? It looks like we went with "Not added" in this issue: https://app.zenhub.com/workspaces/climate-action-secretariat-60ca4121764d710011481ca2/issues/gh/bcgov/cas-cif/1352

Also Project type field.

Thanks @BCerki! As discussed with @Sepehr-Sobhani, for both fields

"N/A should be an actual option for users to select, and once selected, "N/A" should  be displayed as the value of the field as well as on the diff section
However, since both fields are "Optional", we should still allow users to empty them. We can do so by having a blank option for selection or an "x" icon for clearing the field.


fix lost text color on button on set as primary email

This will change the button color to blue to fix #1521. After the screenshot:

LGTM

LGTM


⚠ Bofrost.de has degraded performance

In ea9ebba, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1475 ms

Resolved: Bofrost.de performance has improved in 6db810b after 6 minutes.


Replace uvicorn server with gunicorn

🚀 Feature
Is there a particular reason why LiTServe has not considered moving the app server to gunicorn instead of uvicorn?
Motivation
Uvicorn doesn't do process management and is not recommended to be used for production-grade systems (as mentioned in the uvicorn documentation itself).
Pitch
Replace the uvicorn server in server.py with a gunicorn server
Alternatives
Additional context

gunicorn is a process manager when used with uvicorn. We don't need it here as LitServer acts like a process manager here. Also, uvicorn has its own process manager now so we will keep it lean.
https://www.uvicorn.org/deployment/#using-a-process-manager


Tracker implementation/documentation incorrectly refers to width and height settings as "max"

Cleanup from early mistakes

The two constructors will be updated to reflect the correct parameter names and description
/**
 * @brief creates a new, uniquely named KTL Tracker object
 * @param[in] name unique name for the new Tracker
 * @param[in] width output frame width for GIEs and TISs to work on
 * @param[in] height output frame height for GIEs and TIS to work on
 * @return DSL_RESULT_SUCCESS on success, DSL_RESULT_TRACKER_RESULT otherwise
 */
DslReturnType dsl_tracker_ktl_new(const wchar_t* name, uint width, uint height);

/**
 * @brief creates a new, uniquely named IOU Tracker object
 * @param[in] name unique name for the new Tracker
 * @param[in] config_file fully qualified pathspec to the IOU Lib config text file
 * @param[in] width output frame width for GIEs and TISs to work on
 * @param[in] height output frame height for GIEs and TIS to work on
 * @return DSL_RESULT_SUCCESS on success, DSL_RESULT_TRACKER_RESULT otherwise
 */
DslReturnType dsl_tracker_iou_new(const wchar_t* name,
    const wchar_t* config_file, uint width, uint height);

The dsl_tracker_max_dimensions_get and dsl_tracker_max_dimensions_get will be updated to the following.
/**
 * @brief returns the current maximum frame width and height settings for the named IOU Tracker object
 * @param[in] name unique name of the Tracker to query
 * @param[out] width output frame width for GIEs and TISs to work on
 * @param[out] height output frame height for GIEs and TIS to work on
 * @return DSL_RESULT_SUCCESS on success, DSL_RESULT_TRACKER_RESULT otherwise
 */
DslReturnType dsl_tracker_dimensions_get(const wchar_t* name, uint* width, uint* height);

/**
 * @brief sets the maximum frame width and height settings for the named IOU Tracker object
 * @param[in] name unique name of the Tracker to update
 * @param[in] width output frame width for GIEs and TISs to work on
 * @param[in] height output frame height for GIEs and TIS to work on
 * @return DSL_RESULT_SUCCESS on success, DSL_RESULT_TRACKER_RESULT otherwise
 */
DslReturnType dsl_tracker_dimensions_set(const wchar_t* name, uint width, uint height);

Merged into v0.16.alpha


[NPM] Remove uses of npm build tag from linux bpf code

What does this PR do?
Removes the use of the npm build tag.
Motivation
This requires the use of the npm build tag when running tests locally something that was not required before these changes were made. There needs to be a wider change where there is a clean separation between linux only and linux with eBPF requirements code; some of the files (like epbfless_tracer.go) should not be required to have the linux_bpf tag.
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Ssahu-Noc376-AddNocCodes2021Table

Created a new JobBoard.NocCodes2021 table with the same schema as NocCodes, except that the Code model is a 5-digit string.

The changes are for sub-task https://hive.aved.gov.bc.ca/jira/browse/NOC-376

Closing this PR in favor of #128


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-source-version-target/pull/7 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-source-version-target.git

Docs Build status updates of commit baf5993:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

versioningArchiveSourceFolder/TOC.yml
:white_check_mark:Succeeded
View (sql-server-2014 || azuresqldb-current || azuresqldb-mi-current || azure-sqldw-latest || >= aps-pdw-2016 || >= sql-server-2016 || >= sql-server-linux-2017 || sql-server-previousversions || sqlallproducts-allversions)

versioningArchiveSourceFolder/docfx.json
:white_check_mark:Succeeded

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


Support for activerecord/activesupport 6.0

Summary
Hi! First of all, thanks for these great framework!
I would like suggest some alternative to support use Jets with activerecord/activesupport bellow 6.1.0. For exemple: 6.0.0
Motivation
There is a lot of rubygems that normally depends from activesupport, activerecord and do not support the last versions from that gems. Eg:  I need to use the redshift-client gem, but it require to use activerecord 6.0.0 or bellow.
Drawbacks
I understand that support really old versions will cost. But, at least, 2 latest versions maybe is a good investment in order to allow more people to use Jets.

Wondering which gem examples required 6.0 and below.
That being said, think it's ok to loosen the activesupport and the like to 6.0+ instead of 6.1+ in the jets.gemspec. Though, remembered there were some interfaces that changed from 6.0 to 6.1. Here's one example https://github.com/boltops-tools/jets/pull/539 There may be more.
So would need some respond_to? to check for what interface to use. Hence the overhead. Think it's worth investigating. Will review and consider PRs.

Hi @tongueroo! Thanks  for the update!
Actually, there is no explicit dependency.
But in the "activerecord6-redshift-adapter" gem for example, the class active_record/connection_adapters/determine_if_preparable_visitor.rb is used in order to extend a module.
And that class do not exists in the ActiveRecord 6.1 version.
Is there some code to change in Jets in order to support 6.0+ ? If so, is there some path?

Hi @tongueroo
Is there some code to change in Jets in order to support 6.0+ ? If so, is there some path?

@rinaldifonseca I am curious to hear out what exact issue you have with activesupport 6.0 ?
Have you tried using jets v2.3.19 ? The requirement to 6.1 was done with 3.0 jets release
or you could also fork jet and relax the requirement back to 6.0


publish 2.0.0

See: https://github.com/dart-lang/lints/pull/72

v2.0 is live on pub: https://pub.dev/packages/lints/versions/2.0.0.


No "xxx already exist" validation message pops up when creating a containerized function app with the same name again

OS: All
Build Version: vscode-azurefunctions-1.13.5-alpha.0-4
Regression: Not a Regression
Repro Steps:

Open a containerized function project in VS Code.
Create a containerized function app named "cindytest".
Right click on the Function App tree item and click on the command "Create Function App in Azure…" -> Click "Container Image" -> Enter the same name as the above "cindytest".
Check whether an "xxx already exist" validation message pops up.

Expect:
An "xxx already exist" validation message pops up.

Actual:
No "xxx already exist" validation message pops up.
More Info:

Pop up an error after creating the second “cindytest” containerized function app with the same name.

An "xxx already exist" validation message pops up when entering the containerized function app name same as regular function app name.

This issue doesn't reproduce when creating a regular function app.

Hi! Thanks for filing this issue. At the moment the portal also does not validate containrized function apps with the same name because of this we cannot validate on our end without knowing if it's within the subscription we need to validate or globally. I will reach out to the functions team and see if we can get some more information.


Nested Link nodes do not stop propagating events

Problem
When two Link nodes are nested in each other, clicking on the topmost node sends event to both nodes, instead of just the top node. This results in the browser navigating to two different links.
Steps To Reproduce
Steps to reproduce the behavior:

Page setup:

#[derive(Clone, Routable, Debug, PartialEq)]
enum Route {
    #[route("/")]
    Home {},
    #[route("/login")]
    Login {},
    #[route("/signup")]
    SignUp {},
}

fn App() -> Element {
    rsx! {
        Router::<Route> {}
    }
}

#[component]
fn Home() -> Element {
    rsx! {
        Link { class: "flex bg-frame p-10", to: Route::Login {},
            p { "Login" }
            Link { class: "flex bg-primary w-full h-full p-10", to: Route::SignUp {},
                p { "Signup" }
            }
        }
    }
}

#[component]
fn Login() -> Element {
    rsx! {
        p { "login" }
    }
}

#[component]
fn SignUp() -> Element {
    rsx! {
        p { "signup" }
    }
}

On "/" page click on "Signup" button
Dioxus navigates to "/signup" and then "/login". Clicking back button returns to "/signup", and clicking again returns to "/"

Expected behavior
Navigates to the "/signup" page.
Screenshots
https://github.com/DioxusLabs/dioxus/assets/71188597/bb18175f-6436-4331-a297-678000485854
Environment:

Dioxus version: v0.5.1
Rust version: 1.78
OS info: Linux
App platform: web

Questionnaire

[ ] I'm interested in fixing this myself but don't know where to start
[ ] I would like to fix and I have a solution
[ ] I don't have time to fix this right now, but maybe later

Hi there,
How are you?
I faced same issue.
Let us contact

Thanks to the help from discord members, I can confirm that you can work around this issue by adding the following to the inner Link node.
Link {
    // add this
    onclick: |ev: MouseEvent| e.stop_propagation(),
    ...
}

Thanks for your kindness.
If you don't mind, we can contact in discord.
This is my invitation link.
https://discord.gg/vbXnqCC6
Hope we meet in discord.
Best regards

Hello,
According to the HTML specification an a element mustn't be nested inside another a element.
Is the expected behaviour really to navigate to the inner link?
Maybe a warning/error would be more appropriate.
In which case would you want to nest two links?

Links are not supposed to be nested - this is not an issue on the Dioxus side of things. The stop_propagation workaround does fix this for this particular case, however, you shouldn't be nesting links in the first place. We could warn if a link is nested within another link, but I think the browser will also warn you.


New Privileges Unlocked Without an Increase in Honor.

Hello,
I logged on to codewars today and I had discovered that all abilities up to 3,000 honor had been unlocked while still having just over 1,000 honor. This is definitely a bug. Below is a screenshot of my notifications in codewars:

Thanks,
D.Dawson

Hi,
The same thing happened to me as well. I solved a beta kata and I instantly gained almost 600 honor points and reached somewhere in 800's. When I logged in today, the honor points that I got were removed.
Screen shot of my notifications in codewars :

Thanks.

Hi,
That's strange - abilities were unlocked on my account - I never had a jump in horour. It is exactly the same as yesterday.
Thanks,
D. Dawson

This was due to an accidental update to the honor system, which will finally be rolled out properly this week. For now just enjoy the new privileges :)


i can train your model(ap 49.2)successfully on my own datasets,but it doesn't output eval info per epoch

it's my eval setting

your implementation is the best models in mmdetection which i can get,other good models in mmdetection need mask label,
but your model performs not good on my own datasets with pafpn

Hi, the eval info per epoch should be given if you correctly give the configuration and you can also evaluate the performance through the offline manner.
The first thing to determine is whether the base model is appropriate for your training set. If your dataset is completely different from COCO, the default configuration of anchor maybe not appropriate for your training set.
If the base model can perform a satisfactory performance on your own dataset, TSD can steadily further improve performance.
If there still exists the problem for TSD, you can provide more experimental details.

./tools/dist_train.sh  configs/faster_rcnn_x101_64x4d_fpn_TSD.py  8 --validate


Get AccessToken

Hello
I try to get the access token but it's always null:
Request (LaunchRequest sent via Alexa Tester)
{
  "version": "1.0",
  "session": {
    "new": true,
    "sessionId": "amzn1.echo-api.session.abeee1a7-aee0-41e6-8192-e6faaed9f5ef",
    "application": {
      "applicationId": "amzn1.ask.skill.f20763c7-00a0-4cd5-b2c3-db4fca81e949"
    },
    "attributes": {},
    "user": {
      "userId": "amzn1.account.AM3B227HF3FAM1B261HK7FFM3A2",
      "accessToken" : "anyAccessToken"
    }
  },
  "request": {
    "type": "LaunchRequest",
    "requestId": "amzn1.echo-api.request.9cdaa4db-f20e-4c58-8d01-c75322d6c423",
    "timestamp": "2015-05-13T12:34:56Z"
  }
}

Code to get the access token where accessToken is always null
app.launch(function (req, res) {
    var accessToken = req.sessionDetails.accessToken;
    if (accessToken === null) {
        //no token! display card and let user know they need to sign in
        res.card({
            type: "LinkAccount"
        }).say("Please link your account");
    } else {
        //has a token
        res.say("Welcome!").shouldEndSession(false).send();
    }
});

Is that an issue or am I doing anything wrong?
Thanks!

Could be a bug. Try to write a test for it to demonstrate?

I wrote a simple test and figured out that the accessToken actually gets delivered as expected.
But it does not work when using the "Alexa Tester" user interface. So I guess I have to hunt the bug there....

So does this mean this issue here can be closed and the problem is that the Alexa Tester (on the Amazon website?) isn't sending it?

No. Im talking about the Alexa Tester that is part of Alexa-App. The UI
that allows you to build and send requests.
On Sun, 15 Jan 2017, 22:34 Daniel Doubrovkine (dB.) @dblockdotorg, <
<EMAIL_ADDRESS>wrote:

So does this mean this issue here can be closed and the problem is that
the Alexa Tester (on the Amazon website?) isn't sending it?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/alexa-js/alexa-app/issues/116#issuecomment-272725570,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF3DpvigsXf5fgC2XEqje6ULac392uN2ks5rSpDogaJpZM4Lfzyl
.

Got it. Leaving this as is. You should dig deeper :)

I figured out how to set the accessToken:
when set in server.js's function preRequest via req.body.session.user.accessToken it is available in the app.
But now i'm struggeling with the applicationId. Amazon recommends to check the applicationId in the skill which I would like to do in the app.pre function that gets called on each request.
But how can I set it on server.js so I can use the Alexa Tester?
I tried various version like req.applicationId or req.body.applicationId but non of them made it availabe in req.applicationId where it is when invoked via Lambda.
Thanks for your help!

I figured out how to set the accessToken:
when set in server.js's function preRequest via req.body.session.user.accessToken it is available in the app.
That works for me!
Thanks for your help.


Web Worker compatibility

Over at jsdom, our web worker test fail if we update our dependencies. Here is the error that I got:
ReferenceError: window is not defined at https://github.com/visionmedia/debug/blob/17d0e0bb49b46f53bd97aa6d7b56bf9990a8cb62/browser.js#L13
This line is missing a typeof window !== 'undefined' check. This check is present in other locations in this file, so this issue looks like a small oversight.
After #378 is resolved, you might be interested in https://www.npmjs.com/package/karma-mocha-webworker @yamikuronue

This broke with 1c625d4, which landed in yesterday's version 2.4.5 release.

Crap!  Yup, this is totally on me.  Will submit an update shortly

@thebigredgeek I find you have a commit six hours ago, do you fix it?
if (typeof window !== 'undefined') {
  exports.storage = 'undefined' != typeof window.chrome
               && 'undefined' != typeof window.chrome.storage
                  ? window.chrome.storage.local
                  : localstorage();
}
else {
  exports.storage = localstorage();
}

This should be fixed for 2.5.x  Can you confirm?

Also @Joris-van-der-Wel we'd love to integrate that into our workflow.  I tried earlier but I kept getting missing framework errors :(

Also @Joris-van-der-Wel we'd love to integrate that into our workflow. I tried earlier but I kept getting missing framework errors :(

Odd, it is working properly for the jsdom repo. This is our configuration file for jsdom: karma-webworker.conf.js, however it is a bit extra complicated because it has to deal with browserify. Here is a travis build that uses it.
You can find more example configurations in the test directory of karma-mocha-webworker

This should be fixed for 2.5.x Can you confirm?

Almost ;), the issue still occurs on line 185:
/** Attach to Window*/
if (window) {
  window.debug = exports;
}

This line should be if (typeof window !== 'undefined') {
Alternatively, you could replace all instances of window with self. This works within a worker, but also outside of a worker.

I noticed that a lot of other modules pin debug to specific versions instead of using a version range, I end up with 13 different copies of debug in jsdom. Luckily the only one that ends up in my browserify bundle is pinned at ^2.0.0.

@thebigredgeek any traction on this? We're almost there. Need to close issue on jsdom and currently blocked. /cc @Joris-van-der-Wel

On it right now.  Merry Christmas everyone!


Integration of Meter Mainnet with Trust Wallet

Integration of Meter Mainnet with Trust Wallet
Prize Bounty
$3000 USD (paid in MTRG on Meter mainnet completion of Bounty)
Challenge Description
Trust wallet does not natively support Meter Network. Integrate the Meter chain with Trust wallet (https://trustwallet.com/) to enable users of Trust Wallet hold Meter Mainnet tokens. Future maintenance is not required after the final approval of your work.
Submission Requirements

Full implementation of all checklist items defined in the Trust wallet New Blockchain Support instructions (https://developer.trustwallet.com/wallet-core/newblockchain)
All submissions, including all code, must be open source for future use and reference by the community, and links to external documents must be provided in the Github repo submission
PR Accepted by the Trust Wallet
Demo and unit tests of the submission

Judging Criteria

Meter Network supported on Trust Wallet

Winner Announcement Date
As soon as the pull request is ready.
Resources

Trust Wallet New Blockchain Integration - https://developer.trustwallet.com/wallet-core/newblockchain
Everything you need to start building on Meter - https://docs.meter.io/developer-documentation/introduction
Remix - https://docs.meter.io/developer-documentation/interacting-with-meter-with-remix
Web3 API - https://docs.meter.io/developer-documentation/meterify-api
dApp tutorials - https://docs.meter.io/developer-documentation/meter-dapp-tutorials
Github - https://github.com/meterio
Community tutorials - https://github.com/meterio/meter_docs/tree/main/tutorials
Interoperability guide - http://github.com/meterio/token-list
Any questions, need additional resources or technical support
a.	Telegram - https://t.me/Meter_IO
b.	Discord - https://discordapp.com/invite/WPjTpMG

I implemented the Trust wallet integration but Trust wallet decided not to support the Meter chain right now
See: https://github.com/trustwallet/wallet-core/pull/2185

PR merged by trustwallet team


Error converting Object to Byte Array

Here is my code:
public static byte[] getBytesFromObject(Object obj) {
    String string = JSON.stringify((JSObject)obj);
    byte[] data = string.getBytes();
}

public static Object getObjectFromBytes(byte[] data) {
    String string = new String(data);
    Object obj = JSON.parse(string);
    return obj;
}

and this is the output in the console:
Uncaught TypeError: Converting circular structure to JSON
    --> starting at object with constructor 'Object'
    |     property '$array' -> object with constructor 'Object'
    |     property '$meta' -> object with constructor 'Object'
    --- property 'item' closes the circle
    at JSON.stringify (<anonymous>)

Am I doing this right? If not then how would I go about converting an Object to a byte array and the byte array back into an Object?

You should avoid direct casting of non-JS objects to JSObject.
If you want to stringify JS, please, pass JS objects to JSON.stringify, e.g. something like this:
JSMapLike<JSObject> json = JSObjects.create();
json.set("foo", JSNumber.valueOf(23));
json.set("bar", JSString.valueOf("hello"));
System.out.println(JSON.stringify(json));

You should avoid direct casting of non-JS objects to JSObject.
If you want to stringify JS, please, pass JS objects to JSON.stringify, e.g. something like this:
JSMapLike<JSObject> json = JSObjects.create();
json.set("foo", JSNumber.valueOf(23));
json.set("bar", JSString.valueOf("hello"));
System.out.println(JSON.stringify(json));

Okay, thank you! How would I go about converting a Java class into a Byte Array?
For example:
TestClass test = new TestClass();
Object obj = (Object)test;

public class TestClass {
    //...
}

How would I go about converting a Java class into a Byte Array?

What do you mean?

How would I go about converting a Java class into a Byte Array?

What do you mean?

Like this https://www.baeldung.com/object-to-byte-array
On this tutorial they have a class called "User" and they convert it to a byte array using ObjectOutputStream

You should avoid direct casting of non-JS objects to JSObject.
If you want to stringify JS, please, pass JS objects to JSON.stringify, e.g. something like this:
JSMapLike<JSObject> json = JSObjects.create();
json.set("foo", JSNumber.valueOf(23));
json.set("bar", JSString.valueOf("hello"));
System.out.println(JSON.stringify(json));

How would I parse and retrieve objects from JSON?

@JessePinkman27 there's no way to serialize and deserialize Java objects to/from JSON. Jackson won't work with TeaVM. You should either construct JS objects via JS interaction layer or write a library that acts like Jackson, but relies on code generation (perhaps, annotation processors) rather than reflection.


Add docs for network isolation

Also adds docs for using ExtraMounts to customize the ansible-runner
image.
Signed-off-by: James Slagle<EMAIL_ADDRESS>

/retest

/retest


Add Emphasizer for Buttons in Drawing

Is your feature request related to a problem? Please describe.
Hi, in drawing section it's hard to know if pen, eraser or highlighter is selected.
Describe the solution you'd like
If you can keep the shape around them(when we click pen, eraser or highlighter buttons an orange shape spawns around them) it will let user to know which button is selected.
Describe alternatives you've considered
Pen, eraser, highlighter images can be added for mouse cursor. When we press buttons cursor will change

I'll work on adding a border or changing the color of the selected tool. This will be updated in the next version, which, due to these bugs, will be released within a week.

This issue has been fixed with commit 634842680b.

Right now I only have emphasizer for eraser.
Have you done this on purpose or is this a bug ?
https://github.com/user-attachments/assets/7a268bf2-f379-479f-ac62-248d11a67525

I’m so sorry! This issue was supposed to be fixed before it entered production. I’ll make sure it gets patched in the next version. I hope this doesn’t cause any problems and that you’ll continue to enjoy using Beaver Notes.

I’m so sorry! This issue was supposed to be fixed before it entered production. I’ll make sure it gets patched in the next version. I hope this doesn’t cause any problems and that you’ll continue to enjoy using Beaver Notes.

It's not a big deal at all, I just wanted to inform you. 👍


Downloading models always fails

Anjok07, hello, thank you for your excellent work. I always fail to download the model. When the download reaches 6%, it prompts failure. Please guide me, thank you!

This web problem only appears in China, So I answer in zh-cn.
您应该挂个梯子上网，并且设置为一直开启，这样应该就不会有问题了🤣
问题解决了记得关闭Issue


feat: Filter overflow group functionality

Final PR for initial d2l-filter-overflow-group component. Adds documentation and actual overflow functionality.

Opening a fresh PR for this


Reading local file from disk

Hey,
first of all, thank you for electron-vue. It's a really nice combo to work with. 👍
I recently decided to create a little side-project of mine and while working on features I ran into one issue. I am trying to read a file from disk (I am writing an app, that should save/load user data from and to disk). My code looks like this:
const fs = require('fs')
...
methods: {
  fetchLocalData () {
    fs.readFile('./cl-data.json', 'utf8', function (err, data) {
      if (err) {
        return err
      } else {
        this.healthCheck = 'boo'
      }
    })
  },
...
data () {
  return {
    healthCheck: ''

When I try to run it, I get this error:
Uncaught TypeError: Cannot set property 'healthCheck' of undefined
    at eval (eval at <anonymous> (renderer.js:1288), <anonymous>:22:28)
    at tryToString (fs.js:426)
    at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:413)

It seems to me (however improbable it sounds) like the inner scope of readFile doesn't have access to Vue object's variables. At this point I am not sure what am I doing wrong. Or am I trying to do something, that isn't even possible?
Notes:
Bug can be reproduced by attempting to read local file and assign it's content (or assign any global variable) from within readFile method.

Node version: v7.7.3
NPM version: 4.2.1
Operating System: Win10 64bit

@vlad-saling
The context of this isn't inherited in the code you posted. Using an arrow function will solve this issue, as it provides lexical binding so the context of this is shared.
fs.readFile('./cl-data.json', 'utf8', (err, data) => {
  if (err) {
    return err
  } else {
    this.healthCheck = 'boo'
  }
})

Oh my, indeed... You rock! Thank you :) You can close this issue.

No problem sir. Glad you got this sorted out.


Feat/prod cpc 1036 product ratings

JIRA: https://champlainsaintlambert.atlassian.net/browse/CPC-1036
Context:
This adds a Rating associated to a Product & Customer (Auth User) Entity in the backend.
Changes

Separated Product and Rating inside the source package
Added Rating (Rating Entity, Rating Repository, Rating Service, Rating Controller)
Created Rating (RatingServiceClient & RatingController) inside of API-Gateway

Dev notes (Optional)

Other tests & UI functionality will be added inside different PRs.

Linked Pull Requests (Optional)
https://github.com/cgerard321/champlain_petclinic/pull/641 (Branch was somewhat broken, this PR attempts to resolve this)

The implementation looks good, and the tests are well done.