feat: Support structured JSON logging for all Argo commands

This option makes the log easier to parse and aggregate by services like Stackdriver, Splunk, and Logstash.
Signed-off-by: terrytangyuan<EMAIL_ADDRESS>Checklist:

[x] My organization is added to USERS.md.

Can this be added to the server as well? Maybe even the CLI too?

I can see this can be useful for the server (just added to the server as well) but probably not for the CLI since there doesn't seem to be any use cases to parse logs from the CLI.

I can see this will be useful for the server (just added to the server as well) but probably not for the CLI since there doesn't seem to be any use case to parse logs from the CLI.

The CLI can be used in an automated/scripted pipeline such as Jenkins. Getting logs from those pipelines is certainly a useful use case.

I can see this will be useful for the server (just added to the server as well) but probably not for the CLI since there doesn't seem to be any use case to parse logs from the CLI.

The CLI can be used in an automated/scripted pipeline such as Jenkins. Getting logs from those pipelines is certainly a useful use case.

Sounds good. I'll refactor and add this to other commands as well.

Comments addressed. Please take another look when you get a chance.

@alexec @simster7 Would you like to take another look at this when you get a chance? It would be great to get this in before next rc release.

I’m not sure about this.  Most CLI commands do not do any logging, so I’m not clear of what problem we’re trying to solve?

I’m not sure about this. Most CLI commands do not do any logging, so I’m not clear of what problem we’re trying to solve?

@simster7 What do you think? Shall we just add this flag for controller & executor & server?
@alexec Are you okay with adding this for these three commands though?

I think this would be useful for the three commands you mention.

Thanks for confirming! I just narrowed down the scope of this PR to those three commands. Let me know if I missed anything else.

The failing test seems to be related to master branch and unrelated to this PR.


Support a transparent annotation

Something like #[automergeable(transparent)] should be possible to use for things like wrapper structs to prevent an unnecessary overhead.

This is improved now since tuple structs with one item are treated as transparent.


Reduce panning lag

Moving the dots in the background causes an extreme lag while panning, an alternative needs to be found

temporarily fixed by removing dots from background


ASSETS-31406 add HMAC param to support service account authN

Description
Followup to https://github.com/adobe/asset-compute-commons/pull/91
For https://jira.corp.adobe.com/browse/ASSETS-31406

Add HMAC signature as header to webhook payload
privateKey will only be available for Asset Compute actions not for custom-workers.
Add unit tests
Related PR will have will have access to keys. see https://git.corp.adobe.com/nui/worker-libvips/pull/92

Fixes # (issue)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

LGTM apart from a few nitpicks

add a test when private key is not there and establish the expected behavior
verify on hmac-signature.js check if it needs to be moved to test libs but its fine where it is

Thanks, to be consistent, changed logic to throw error if private key doesn't exist. Added test for it.
In two minds about verify in hmac-signature.js, its currently only used in test, but will leave it as is for readability and completion.


Readme update

Updated the readme to fix a typo and to add a link to the map of menu strings to menu locations I made earlier that helped me with the Turkish translation.

Closing pull request as I sent an updated one that also fixes typos and has some proofreading changes.


mcumgr seems to lock up when it receives command for group that does not exist

Describe the bug
It seems that mcumgr is locking the smp_svr sample application up when it receives a command for which it does not have a handler for, noticed whilst testing fs checksum additions. Test command: \06\09AAwAAAACAEYBAL//2nk=\0a
No further looking for the cause or how this happens has been investigated as of yet. This is with the shell mcumgr transport, no other transport has been tested.
To Reproduce
Steps to reproduce the behaviour:
Build smp_svr sample with it set to include fs and use shell for mcumgr transport, send above test command, observe as UART will no longer respond and no output is emitted.
Expected behavior
For an error to be emitted and for the application to not lock up
Impact
Showstopper if all it takes to DoS a device is a simple invalid mcumgr command
Environment (please complete the following information):

OS: Linux
Toolchain: Zephyr SDK 0.13.1
Commit SHA or Version used: 2ef47509c37163a0848b72b6ab37b7dbf831661e

Additional context
Will update bug once I investigate more

I have tried to verify that on e186ebc3143e642f69c9379abdccccf4f7fbfaf8 with following steps:

I have built smp_svr with OS group only, with taskstat command enabled only, for BT and uploaded it to board;
I have issued command "taskstat", with mcumgr-cli, got error 2, "no mem", which is correct with default conf as there is not enough space to collect taskstat;
I have issued command "echo<PHONE_NUMBER>" and got rc=8, "not supported"; mcumgr-cli does not report error correctly but this is problem with mcumgr-cli;
I have issued command "image list" and got "Error: 8" (rc=8), because entire image group has not been compiled in.

The smp_svr seems to correctly response with rc=8 on non-existing commands.

If you use AAwAAAACAEYBAL//2nk= does it cause the issue? This is using a 16-bit group where the upper byte is non-zero, so might be caused by that, I'll have a look at it later and see if I can find at what point it's freezing

I have tried to decode your input and got this: 00 0c 00 00 00 02 00 46  01 00 bf ff da 79
I think that console frame length is not encoded in big endian (https://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/mgmt/mcumgr/lib/transport/smp-console.md), so the smp decoder probably waits for more 04 14 frames to fill in the missing bytes.

Doesn't seem to be the issue. If I send a valid command, I get a response, and it's all being done by the same underlying code. This is Qt-based:
void MainWindow::SendMessage(QByteArray message)
{
	QByteArray msg;
	uint16_t size = message.length();
	size += 2;
	msg.append((uint8_t)((size & 0xff00) >> 8));
	msg.append((uint8_t)(size & 0xff));
	msg.append(message);
	uint16_t crc = crc16(&message, message.length(), 0x1021, 0, true);
	msg.append((uint8_t)((crc & 0xff00) >> 8));
	msg.append((uint8_t)(crc & 0xff));

	QByteArray b64msg;
	b64msg.append((uint8_t)0x06);
	b64msg.append((uint8_t)0x09);
	b64msg.append(msg.toBase64());
	b64msg.append((uint8_t)0x0a);

	gspSerialPort.write(b64msg);
}

void MainWindow::on_btn_board_clicked()
{
    QByteArray message;
    QByteArray cbor;

    message.append((uint8_t)0x00); //Read
    message.append((uint8_t)0x00); //Flags
    message.append((uint8_t)0x00); //Length A
    message.append((uint8_t)0x00); //Length B
    message.append((uint8_t)0x00); //Group A
    message.append((uint8_t)0x46); //Group B
    message.append((uint8_t)0x01); //Sequence
    message.append((uint8_t)0x00); //Message ID

    QCborStreamWriter datamap(&cbor);
    datamap.startMap();
    datamap.endMap();

    message[3] = (uint8_t)cbor.length();
    message.append(cbor);

    SendMessage(message);
}

Indeed seems to be a bug, it's stuck on finding the group but will not get past 70:
mgmt_find_group (command_id=0, group_id=70) at /tmp/aa/zephyr/subsys/mgmt/mcumgr/lib/mgmt/src/mgmt.c:100
100             for (group = mgmt_group_list; group != NULL; group = group->mg_next) {
(gdb) bt
#0  mgmt_find_group (command_id=0, group_id=70) at /tmp/aa/zephyr/subsys/mgmt/mcumgr/lib/mgmt/src/mgmt.c:100
#1  mgmt_find_handler (group_id=<optimized out>, command_id=0) at /tmp/aa/zephyr/subsys/mgmt/mcumgr/lib/mgmt/src/mgmt.c:134
#2  0x00015bca in smp_handle_single_payload (handler_found=<synthetic pointer>, req_hdr=0x20004dbc <sys_work_q_stack+2076>,
    cbuf=0x20004ddc <sys_work_q_stack+2108>) at /tmp/aa/zephyr/subsys/mgmt/mcumgr/lib/smp/src/smp.c:213
#3  smp_handle_single_req (handler_found=<synthetic pointer>, req_hdr=0x20004dbc <sys_work_q_stack+2076>,
    streamer=0x20004e34 <sys_work_q_stack+2196>) at /tmp/aa/zephyr/subsys/mgmt/mcumgr/lib/smp/src/smp.c:213
#4  smp_process_request_packet (streamer=streamer@entry=0x20004e34 <sys_work_q_stack+2196>, req=0x20006518 <_net_buf_pkt_pool>)
    at /tmp/aa/zephyr/subsys/mgmt/mcumgr/lib/smp/src/smp.c:327
#5  0x00006ca4 in zephyr_smp_process_packet (nb=<optimized out>, zst=0x20001680 <smp_shell_transport>)
    at /tmp/aa/zephyr/subsys/mgmt/mcumgr/smp.c:249
#6  zephyr_smp_handle_reqs (work=0x20001680 <smp_shell_transport>) at /tmp/aa/zephyr/subsys/mgmt/mcumgr/smp.c:265
#7  0x00010d88 in work_queue_main (workq_ptr=0x20000fe0 <k_sys_work_q>, p2=<optimized out>, p3=<optimized out>)
    at /tmp/aa/zephyr/kernel/work.c:649
#8  0x00012f6e in z_thread_entry (entry=0x10d05 <work_queue_main>, p1=<optimized out>, p2=<optimized out>, p3=<optimized out>)
    at /tmp/aa/zephyr/lib/os/thread_entry.c:36
#9  0x00012f6e in z_thread_entry (entry=0x10d05 <work_queue_main>, p1=<optimized out>, p2=<optimized out>, p3=<optimized out>)
    at /tmp/aa/zephyr/lib/os/thread_entry.c:36
#10 0x2aa640a6 in ?? ()
Backtrace stopped: previous frame identical to this frame (corrupt stack?)
(gdb)
#0  mgmt_find_group (command_id=0, group_id=70) at /tmp/aa/zephyr/subsys/mgmt/mcumgr/lib/mgmt/src/mgmt.c:100
#1  mgmt_find_handler (group_id=<optimized out>, command_id=0) at /tmp/aa/zephyr/subsys/mgmt/mcumgr/lib/mgmt/src/mgmt.c:134
#2  0x00015bca in smp_handle_single_payload (handler_found=<synthetic pointer>, req_hdr=0x20004dbc <sys_work_q_stack+2076>,
    cbuf=0x20004ddc <sys_work_q_stack+2108>) at /tmp/aa/zephyr/subsys/mgmt/mcumgr/lib/smp/src/smp.c:213
#3  smp_handle_single_req (handler_found=<synthetic pointer>, req_hdr=0x20004dbc <sys_work_q_stack+2076>,
    streamer=0x20004e34 <sys_work_q_stack+2196>) at /tmp/aa/zephyr/subsys/mgmt/mcumgr/lib/smp/src/smp.c:213
#4  smp_process_request_packet (streamer=streamer@entry=0x20004e34 <sys_work_q_stack+2196>, req=0x20006518 <_net_buf_pkt_pool>)
    at /tmp/aa/zephyr/subsys/mgmt/mcumgr/lib/smp/src/smp.c:327
#5  0x00006ca4 in zephyr_smp_process_packet (nb=<optimized out>, zst=0x20001680 <smp_shell_transport>)
    at /tmp/aa/zephyr/subsys/mgmt/mcumgr/smp.c:249
#6  zephyr_smp_handle_reqs (work=0x20001680 <smp_shell_transport>) at /tmp/aa/zephyr/subsys/mgmt/mcumgr/smp.c:265
#7  0x00010d88 in work_queue_main (workq_ptr=0x20000fe0 <k_sys_work_q>, p2=<optimized out>, p3=<optimized out>)
    at /tmp/aa/zephyr/kernel/work.c:649
#8  0x00012f6e in z_thread_entry (entry=0x10d05 <work_queue_main>, p1=<optimized out>, p2=<optimized out>, p3=<optimized out>)
    at /tmp/aa/zephyr/lib/os/thread_entry.c:36
#9  0x00012f6e in z_thread_entry (entry=0x10d05 <work_queue_main>, p1=<optimized out>, p2=<optimized out>, p3=<optimized out>)
    at /tmp/aa/zephyr/lib/os/thread_entry.c:36
#10 0x2aa640a6 in ?? ()
Backtrace stopped: previous frame identical to this frame (corrupt stack?)
(gdb) c


Group dependabot updates

Recently, the upload-artifact and download-artifact actions were updated, and dependabot opened a PR #3557. For repositories that use both the upload and download actions, this update caused CI breakages when the two packages were updated in separate PRs. Whilst we don't yet have any CI that uses download-artifact, it would be prudent in my view to take advantage of grouping all CI updates into a single PR.

@agoose77 wanna hit the merge button? :D

I'm gonna merge this so that it's in place for the next round of dependabot updates


Build fails when space in path

Hello, is this on Windows? #488

Ubuntu 14.04

Ah, I've just reproduced this. I wonder if something has changed in gyp/node-gyp recently? It looks like https://github.com/nodejs/node-gyp/issues/65 might still be the underlying problem as module_root_dir doesn't appear to be correctly escaped. Perhaps we can work around this - leave it with me.

Commit da0dc28 on the pencil branch appears to have solved this:
mkdir "/tmp/hello world" && cd "/tmp/hello world" && npm install lovell/sharp#pencil
```

More importantly it hasn't broken anything else on [Linux](https://travis-ci.org/lovell/sharp/jobs/148866282), [OS X](https://travis-ci.org/lovell/sharp/jobs/148866283) and [Windows](https://ci.appveyor.com/project/lovell/sharp/build/597/job/3tam4cqywlxwfos2).

Thanks for reporting this.

This should be fixed in v0.16.0, now available via npm.


Question about cell truncation and answer truncation

Hi, thank you for the nicely written repo and the nice work!
I am reading through the part on table truncation and have a question about how cell and answer truncations are performed for long cells. Specifically, you have:

https://github.com/microsoft/Table-Pretraining/blob/da8bbdf98857a21335379eedc74836809c22c9b9/tapex/processor/table_processor.py#L18 which truncates cells up to a certain number of tokens, and
https://github.com/microsoft/Table-Pretraining/blob/da8bbdf98857a21335379eedc74836809c22c9b9/tapex/processor/table_processor.py#L31 which processes the answers without any explicit truncation.

My question is that for an answer that may come from a very long cell, if I understand correctly, the cell might already be truncated, yet the answer still remains the original length, right? This may not be an issue for testing/inference, but may be an issue during training, because in this case we are asking the model to generate things that do not exist in the input (because some tokens were truncated).
Is this a known limitation, or am I missing something? If it is a known limitation, is there an easy way I can remedy this using this codebase? Thanks for your time.

Hi @yuhaozhang , thanks for your interest on our work! The intention of cell-level truncation is to truncate some too long cell values to avoid it occupying too many tokens in the input sequence (e.g., a description about a book). And for the output side, I think the following code has applies the same truncation on answers:
https://github.com/microsoft/Table-Pretraining/blob/da8bbdf98857a21335379eedc74836809c22c9b9/tapex/processor/table_truncate.py#L62
The only problem may be recovery. If you’d like to recover the original cell with post-processing, I would recommend to maintain a cell mapping dictionary by yourself.
Best,
Qian


Prevent platform_feature creation

Hi,
I was wondering why there is a need for the platform_feature.
Isn't it possible to have the features created with their plugin dependencies populated directly? Now all the dependencies are populated at the level of the platform_feature. I could be mistaken, but doesn't that kind of break the feature definitions?
Cheers,
Dann

Hold on, this seems to be a side effect of the Maven dependency I was trying this on.

feature(id: 'org.apache.jena', name: 'Apache Jena', version: '3.1.0') {

    plugin 'org.apache.jena:apache-jena-libs:3.1.0'
}

This seems to happen if the artifact is a .pom. Perhaps I am making the wrong assumption, here.
With a similar top-level .pom for GWT, nothing gets included and the feature is empty.

feature(id: 'com.google.gwt', name: 'GWT', version: '2.8.0') {

    plugin 'com.google.gwt:gwt:2.8.0'
}

If I remember correctly the platform feature is needed because it describes what is to be content of the p2 repository / update site. If you don't give the same ID to the platform feature as to one of the others you shouldn't have a problem.
I'm not sure, but it might cause problems when you have feature IDs that are equal to bundle symbolic names. So you could try something like com.google.gwt.feature your feature's ID.
I'm not sure what you mean with pom dependency - bundles can only be created for Jars, so it might be that it is skipped if it's only artifact is a pom. Might also be that is ignored in the dependency mechanism because there is no Jar.
Does your dependency get properly resolved by Gradle? What is listed for gradle dependencies?

The .pom is a container, a convience construct to get all the jars of GWT. Of course, if it gets ignored, its dependencies will be ignored, as well. Which explains the empty feature.

Well, I just say it might. I have no idea how Gradle deals with this kind of dependency. Does it work for you in other Gradle projects?
If your dependency is a so-called bill-of-materials (bom) which is supported in Maven, you might need a third-party plugin to be able to resolve it. See for instance here and here.

Assuming this is solved, as there was no further feedback. Please reopen if that's not the case.


Exception Descriptions vs Messages is ambiguous

The guide describes the importance of good descriptions that will be used for internationalizing, but also describes grammatically correct messages.  The Exception class has only a message, and it is generally formatted to be a full error message.  Using the message also as a key for internationalizing seems like a bad idea - are they supposed to be two separate members, an internationalization key "description" and a user-friendly "message"?  Or are we supposed to double-dip on the "message"?  If so, then the "description" terminology is misleading.

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

ID: b1ea31f6-516f-100d-046e-1a0445d92824
Version Independent ID: a713e8c9-8210-761d-9ae7-238793825ff3
Content: Best Practices for Exceptions
Content Source: docs/standard/exceptions/best-practices-for-exceptions.md
Product: dotnet
GitHub Login: @mairaw
Microsoft Alias: mairaw

Thanks, @Pxtl, for submitting another feedback item.
The term "description" appears suddenly in this article with no explanation. It is in fact the Message property, which is intended to be a localized string. Typically, localized exception messages are stored as values corresponding to a named key in a resource file. For information on using resources in .NET applications, see  Resources in Desktop Apps and ResourceManager Class.
I've opened PR #6597 to clarify the confusion created by the abrupt introduction of the term "description". It aims at providing a quick fix, though. Localizing exception messages is an important topic, but one that's not addressed in the documentation. I'll also open an issue and add it to our backlog to create a new topic about creating localized exception messages.
Again, thanks for raising this issue.


Tapping this token in Wallet tab seem to hang the app somewhat reliably

Tapping this token in Wallet tab seem to freeze the app somewhat reliably

This is with: 0x0d590124d2faabbbdfa5561ccbf778914a50bcca
Final release candidate v3.53.3 available on 20220402

Found the cause, will attempt to fix


Allow extended config on cron settings

Workaround odd behaviour in go-ini whereby an embedded field does not get the values from the same section.
Fix #12934
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

Codecov Report

Merging #12939 into master will increase coverage by 0.00%.
The diff coverage is 63.63%.

@@           Coverage Diff           @@
##           master   #12939   +/-   ##
=======================================
  Coverage   43.09%   43.10%
=======================================
  Files         658      658
  Lines       72455    72464    +9
=======================================
+ Hits        31226    31232    +6
+ Misses      36175    36174    -1
- Partials     5054     5058    +4

Impacted Files
Coverage Δ

modules/setting/cron.go
66.66% <63.63%> (-33.34%)
:arrow_down:

modules/indexer/stats/db.go
43.47% <0.00%> (-17.40%)
:arrow_down:

modules/charset/charset.go
68.53% <0.00%> (-4.50%)
:arrow_down:

modules/git/utils.go
73.77% <0.00%> (-3.28%)
:arrow_down:

models/unit.go
46.57% <0.00%> (-2.74%)
:arrow_down:

modules/process/manager.go
72.50% <0.00%> (-2.50%)
:arrow_down:

routers/repo/view.go
37.47% <0.00%> (-0.65%)
:arrow_down:

modules/git/repo.go
46.19% <0.00%> (-0.51%)
:arrow_down:

modules/queue/workerpool.go
60.00% <0.00%> (+1.22%)
:arrow_up:

services/mirror/mirror.go
19.93% <0.00%> (+1.35%)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

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


Get self reference modules working

I was partially confused due to a potential spec bug
in package.exports that didn't take into account top-level
application (without a . main)
This now works and has a passing test.
Refs: https://github.com/nodejs/node/issues/30633

@MylesBorins, I cloned your branch and ran the test locally. It is failing on my machine. Could this have to do with the ~/package.json hack?

That is exactly why it is failing. Have not figured out why that breaks it
yet... Definitely a weirdo bug
On Mon, Nov 25, 2019, 3:20 AM Derek Lewis<EMAIL_ADDRESS>wrote:

@MylesBorins https://github.com/MylesBorins, I cloned your branch and
ran the test locally. It is failing on my machine. Could this have to do
with the ~/package.json hack?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DerekNonGeneric/lorem-demo/pull/1?email_source=notifications&email_token=AADZYV5MQ6CWNAXODSG4BZLQVODFDA5CNFSM4JREVJUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFBRAXY#issuecomment-558043231,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADZYV3V2HZ57TI6VF3EDADQVODFDANCNFSM4JREVJUA
.

I will take it from here and investigate further, thanks!

Hi @MylesBorins and @DerekNonGeneric
I ended up here while following the trails of LOAD_SELF_REFERENCE from here https://nodejs.org/dist/latest-v13.x/docs/api/modules.html and https://github.com/nodejs/node/pull/31009
I was looking into the implementation to get help on my quest to get 'self reference" working from here https://github.com/MylesBorins/lorem-demo/tree/fc5ed2558670028e3c5bb02f2ae098044d63212a
The implementation uses --no-esm and test fails in absence of it. Does it mean that self resolution is non-ESM feature?

Hello @fusionstrings, glad you found your way here!

Does it mean that self resolution is non-ESM feature?

Nope, it definitely works with import too!
Here's a brief explanation that might help clarify what's being described above.
The --no-esm flag disables '-r esm' behavior by switching off the --esm flag, which is on by default. This '-r esm' behavior would preload the esm module at startup. The esm, module, which is a userland loader to polyfill versions of Node.js without native support for ES Modules, is no longer necessary in v13.2+. This is because the majority of the ES module implementation is unflagged in the 13.x release line.
I would recommend checking out the master branch of this repository (the one attached to this thread) to see a currently functional implementation. Just make sure you're on v13.2+. It should be working! I know how confusing this can be, so I've dropped some links below that I hope will clarify what I've written above. Ultimately, the cause of it not working on my end was that the integrated VSCode terminal, which I was using to test this behavior, was using the wrong version.
Let me know if there's anything else. :)

--no-esm §
-r, --require module §
esm, module §
v13.2 Unflagging announcement §

Note: the esm module does not put your code in module context. It is still in commonjs context, and I would advise against using it if you are interested in testing the new ES module implementation.

Thanks a lot @DerekNonGeneric for explaining in detail especially about significance of --no-esm.
I got it working now 👍

One thing to add --no-esm is a tap specific flag... Not for the nodejs
runtime
On Mon, Feb 24, 2020, 7:04 AM Dilip Kr. Shukla<EMAIL_ADDRESS>wrote:

Thanks a lot @DerekNonGeneric https://github.com/DerekNonGeneric for
explaining in detail especially about significance of --no-esm.
I got it working now 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DerekNonGeneric/loader339/pull/1?email_source=notifications&email_token=AADZYV3KAKQ74KVDIHEWY43REOZVPA5CNFSM4JREVJUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMXRQYA#issuecomment-590288992,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADZYVYPQTXNIRUDPNBNNN3REOZVPANCNFSM4JREVJUA
.

@MylesBorins Thanks for clarifying.
The issue with me was extremely weird. I matched with the code in master branch of https://github.com/DerekNonGeneric/loader339 and my code was using same convention.
As @DerekNonGeneric mentioned that he was using VSCode integrated terminal, so was I.
I use nvm and ensured correct version was activated.
I still can't figure out what was wrong 😢but a restart seems to have fixed it for me.
Assurance that master branch has working code ensured that something is wrong from my side.


Fuzzy font in integrated terminal after resize

When resizing the main window, the font in the integrated terminal ends up fuzzy, seemingly dependent on the exact pixel size of the window. If adjusting by a few more pixels, it sometimes returns to normal rendering.

VSCode Version: 1.43
OS Version: Windows 10 1909

Steps to Reproduce:

Open integrated terminal
Resize main window by a few pixels

Used font: Fira Code, Font size: 11.5/12. Also occurs with Consolas font.
Here's the same window, resized in between each screensnip:

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

Do you have window.zoomLevel in your settings?

Do you have window.zoomLevel in your settings?

Yes I did (value: 1). Problem gone after removing.
Didn't seem to make editor text fuzzy in the same way.

@larsch the editor is drawn using the DOM, the terminal uses canvas (unless you set it to the slower dom renderer). AFAIK there isn't a way to fix this one off rounding error that causes the blurriness when you scale the browser window. Closing as designed for now, I encourage setting fontSize settings instead of zoomLevel to avoid this.


Bike-shedding: switchFirst and switchMapFirst names

Given that we've migrated to strategy and strategyMap names, do the names switchFirst and switchMapFirst make sense?  Should it be switchFirstMap?
Furthermore is the strategy name switchFirst the best choice?

I'd also try to find a name other than switchFirst. I actually couldn't figure out what it does, and similarity with switch didn't help.
I'll just brainstorm some names to get us kickstarted:

proceed and proceedMap
finish and finishMap
thorough and thoroughMap
deal and dealMap
entire and entireMap
switchEntire and switchEntireMap
ignoreUntilComplete and ignoreUntilCompleteMap
persist and persistMap
drag and dragMap

...

deal

Haha... just "deal"

BUMP: I think we still need to come up with something here.

Wait a minute, how is switchFirst different to mergeAll with concurrency=1?

Wait a minute, how is switchFirst different to mergeAll with concurrency=1?

it drops all incoming observables until the first one it receives completes.

And I just noticed we don't have tests for the concurrency parameter given to mergeAll. But you see what I mean right? With concurrency=1, mergeAll ignores all incoming observable until the active(s) one(s) complete.

Alright, I noticed the difference. mergeAll(1) doesn't drop incoming observables, switchFirst does.
mergeAll(1)
    var x = hot(   '--------a---b---c--|               ');
    var y = hot(   '--------------d--e---f---|      ');
    var z = hot(   '-----------------------g--h---i---|');
    var e1 = hot(  '------x----y--------z-------------|', { x: x, y: y, z: z });
    var expected = '--------a---b---c----f----h---i---|';

    expectObservable(e1.mergeAll(1)).toBe(expected);

switchFirst
    var x = hot(   '--------a---b---c--|               ');
    var y = hot(   '--------------d--e---f---|      ');
    var z = hot(   '-----------------------g--h---i---|');
    var e1 = hot(  '------x----y--------z-------------|', { x: x, y: y, z: z });
    var expected = '--------a---b---c------g--h---i---|';

    expectObservable(e1.switchFirst()).toBe(expected);


Can the wheels from this package be Python 3 generic (not specifc to each Python version)?

I would assume none of the C code from this package is using the Python C API, is that correct?
If so, I think maybe the wheels can be declared abi3 compatible (the reduced Python3 API) and that'd reduce the total number of wheels to 1 per platform, instead of having one per Python version and per platform.
This would also have the advantage that existing wheels marked with abi3 would be inmediately installable in newer Python versions (at least until a Python release updates the ABI to abi4, maybe the future no-gil), so the package would not have to be updated just to support the latest Python release, like in https://github.com/pyocd/libusb-package/issues/16.
I've not yet had to create a Python wheel with C extensions, so not my area of expertise and I could be wrong here, but worth exploring?

From the cibuildwheel docs I've found this entry about the "ABI3 wheels (Limited API)":
https://cibuildwheel.readthedocs.io/en/stable/faq/#abi3
And it links to this repo as an example: https://github.com/joerick/python-abi3-package-sample
But I've also noticed this in the v2.8.0 changelog, which looks promising as well:
https://cibuildwheel.readthedocs.io/en/stable/changelog/#v280

Adds support for building py3-none-{platform} wheels. This works the same as ABI3 - wheels won't be rebuilt, but tests will still be run across all selected versions of Python.
These wheels contain native extension code, but don't use the Python APIs. Typically, they're bridged to Python using a FFI module like ctypes or cffi. Because they don't use Python ABI, the wheels are more compatible - they work across many Python versions.
Check out this example ctypes project to see an example of how it works. (#1151)

https://github.com/pypa/cibuildwheel/pull/1151

Attempt PR:

https://github.com/pyocd/libusb-package/pull/20


[Bug]: Cannot select Main campaign

BG3 Mod Manager Version
<IP_ADDRESS>
BG3 Game Version
Full Release Hotifx 4
Bug Summary
As the title suggests I cannot select "Main" campaign even though I set the game data path correctly.  There is only a selection for "gustavDev". Not sure if it is related but I noticed that since recently, no matter which mod I add, when I start a new game I am loaded to some dev scene instead of character creation.
Links
https://imgur.com/a/ldgV0tE

GustavDev is the main campaign, though it should only display that as the name if you're in developer mode (Settings -> Preferences -> Enable Developer Mode).
Do you have the Mod Fixer mod? Sounds like you're missing it, leading the game to compile an empty story.div.osi, which breaks scripts / character creation.

Thanks for the hint! Will check that out.

GustavDev is the main campaign, though it should only display that as the name if you're in developer mode (Settings -> Preferences -> Enable Developer Mode).
Yep, switched out of dev mode now it's "Main" again.

GustavDev is the main campaign, though it should only display that as the name if you're in developer mode (Settings -> Preferences -> Enable Developer Mode).
I now installed it and was able to go to character select, but did get some warnings that a story couldnt be found and it is loading the default. Probably still have to figure out how to set up the mod correctly.

Can close this issue, thanks for your advice.


Gradle fail while deploying the App in local device

Summary:
After solving the absolute compile error, when the developer wants to deploy the app to the local device, the gradle fails with the following log message ->
Default interface methods are only supported starting with Android N (--min-api 24): java.util.Collection com.google.common.collect.BiMap.values() Message{kind=ERROR, text=Default interface methods are only supported starting with Android N (--min-api 24): java.util.Collection com.google.common.collect.BiMap.values(), sources=[Unknown source file], tool name=Optional.of(D8)}
This can be easily solved by click File > Project Structure in Android Studio, and change the Source and Target Compatibility to 1.8.
We should add this in the official documentation of the app in this repo. (under build section), so that other developers do not have to waste time fixing this issue.

I need a bit more information here - how did you run into this error? I didn't think we had any places in the code where we used default interface methods, but I could easily be wrong. Can you point me at the offending code?


Added ability to pass options to dropdown divider

Q
A

Is bugfix?
❌

New feature?
✔

Breaks BC?
❌

Fixed issues
-

👍


LWMS ID 11022: Fixed worpress db user error in Lab 4

Add here your WMS ID
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[ ] All filenames are lower case (including folders, images, files, etc.)
[ ] Filenames are descriptive
[ ] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[ ] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[ ] Image references in markdown contain an alternate text

Hi Anoosha,
I am trying to fix a problem for the WordPress workshop. A user wrote to me and complained about it. Which new files are you talking about? I only want to deploy the LWMS ID 11022: in the folder plforacle/database/tree/main/mds-wordpress.
Thank you,
Perside Lafrance Foster | MySQL Principal Partners Solution Engineer
email<EMAIL_ADDRESS>phone: 1.786.877.9063
Oracle  Florida
From: Anoosha Pilli @.>
Sent: Friday, May 12, 2023 5:43 PM
To: oracle-livelabs/database @.>
Cc: Perside Foster @.>; Mention @.>
Subject: [External] : Re: [oracle-livelabs/database] LWMS ID 11022: Fixed worpress db user error in Lab 4 (PR #257)
@anooshapilli commented on this pull request.
@plforaclehttps://urldefense.com/v3/__https:/github.com/plforacle__;!!ACWV5N9M2RV99hQ!JDggYimQUkJUdK3IpMudutu25vEGMBT2lkIUfaTnDaThMztrnNkIWtVc8bBHIav98DUfUBMTXrrqFA5ljYmONBMhigtx$ - There are new files created and updated in the folders other than WMS ID 11022 workshop folder. Could you share more details about these changes and the ID of those workshops?
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/oracle-livelabs/database/pull/257*pullrequestreview-1425187814__;Iw!!ACWV5N9M2RV99hQ!JDggYimQUkJUdK3IpMudutu25vEGMBT2lkIUfaTnDaThMztrnNkIWtVc8bBHIav98DUfUBMTXrrqFA5ljYmONHkFyRGT$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/AMYQBSTTQFTWGAM3MMCLTILXF2VGVANCNFSM6AAAAAAX77YQUE__;!!ACWV5N9M2RV99hQ!JDggYimQUkJUdK3IpMudutu25vEGMBT2lkIUfaTnDaThMztrnNkIWtVc8bBHIav98DUfUBMTXrrqFA5ljYmONBHOCpZP$.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

Several of the changes are for Oracle cloud world ( I am updating 4 different workshops for that event) .... I don't want to publish them yet. They are not part of the WordPress folder (LIveLab Workshop). I will create a new PR for the WordPress LiveLab.


exec opt numbers table func performance

Proposed changes
before：
[tpch]>select count(e1) from (select 1 as k1) as t lateral view explode_numbers(1000000000) tmp1 as e1;
+------------+
| count(e1)  |
+------------+
|<PHONE_NUMBER> |
+------------+
1 row in set (1.60 sec)

after：
[tpch]>select count(e1) from (select 1 as k1) as t lateral view explode_numbers(1000000000) tmp1 as e1;
+------------+
| count(e1)  |
+------------+
|<PHONE_NUMBER> |
+------------+
1 row in set (0.60 sec)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Allow Long as ID type in Entity for Cosmos DB

Related issue:
https://github.com/Azure/azure-sdk-for-java/issues/12712

/azp run java - cosmos - ci

/azp run java - cosmos - tests

/check-enforcer override


When installing modules, they should be copied to ejabberdctl module folder

@who i'm just trying to understand the mechanism that loads modules from the "outside", and i see that you did the original pull request, so here goes;
https://github.com/rroemhild/docker-ejabberd/blob/master/scripts/post/11_ejabberd_install_modules.sh#L14
Shouldn't the script that installs "sideloaded" modules copy them to ejabberdctl's $CONTRIB_MODULES_PATH, rather than
${EJABBERD_HOME}/.ejabberd-modules/sources/${module_name}

Furthermore, i'm not sure ${module_name} should be a part of the path.
When i do
docker run -v ./ejabberd/module_source/:/opt/ejabberd/module_source/ ...

where the source contains a module, mod_offline_push, the structure will end up as
${EJABBERD_HOME}/.ejabberd-modules/sources/mod_offline_push/mod_offline_push

and the installation will fail miserably;
ejabberd_1  | Error: [{missing,"mod_offline_push.spec"},
ejabberd_1  |         {missing,"COPYING"},
ejabberd_1  |         {missing,"README.txt"},
ejabberd_1  |         {missing,"src (Erlang) or lib (Elixir) sources directory"},
ejabberd_1  |         {invalid_spec,"no such file or directory"}]

Hi, i merged #116. Does this merge fix your issue?

@hhravn Did @rroemhild's change fix the issue for you?

Hi there,
I am having a bit of a problem to add a module!  for some reason when I try to build the container I get that log below, via docker-compose
EJABBERD_SOURCE_MODULES=mod_offline_http_post
ejabberd_1  | Updating module specs... ok
ejabberd_1  | Analyzing module mod_offline_http_post for installation
ejabberd_1  | Copying module to ejabberd folder /opt/ejabberd/.ejabberd-modules/sources
ejabberd_1  | Running module_check on mod_offline_http_post
ejabberd_1  | Error: not_available
Ejabberd 17.04


Comments bug

Type: Bug
Runs selected commments in the program
VS Code version: Code 1.74.3 (97dec172d3256f8ca4bfb2143f3f76b503ca0534, 2023-01-09T16:59:02.252Z)
OS version: Windows_NT x64 10.0.19044
Modes:
Sandboxed: No
Extensions (14)

Extension
Author (truncated)
Version

blackbox
Bla
0.10.12

vsc-material-theme
Equ
33.6.0

vsc-material-theme-icons
equ
2.5.0

code-runner
for
0.12.0

autolua-debugger
liz
0.0.8

isort
ms-
2022.8.0

python
ms-
2022.20.2

vscode-pylance
ms-
2023.1.40

jupyter
ms-
2022.11.1003412109

jupyter-keymap
ms-
1.0.0

jupyter-renderers
ms-
1.0.14

vscode-jupyter-cell-tags
ms-
0.1.6

vscode-jupyter-slideshow
ms-
0.1.5

material-icon-theme
PKi
4.23.1

(2 theme extensions excluded)

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
azure-dev_surveyone:30548225
pyindex848cf:30577861
nodejswelcome1:30587005
3biah626:30602489
89544117:30613380
pythonsymbol12cf:30652349
2i9eh265:30646982

Can you be a lot more specific about the steps you are trying on what code?  Are you trying to run code runner extension on code in comments?

/needsMoreInfo

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Update virtual-machines-common-classic-manage-extensions.md

Fixed typos.

Hi @alexandair, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;

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

@tysonn this is good to merge.
Thanks @alexandair .


php artisan koel:sync -v throws exception

Ubuntu 16.04
Koel: 3e3004a9045f16eda15735a4d9a4856beaa28f62 (aka Nov 8th)
Running as a virtual host in Apache2
Music is stored on Google Drive and accessed via google-drive-ocamlfuse
It is failing reliably in the same spot. I wish I could identify where but I don't understand how it is traversing the directory. I thought it was a bad cover file but I've moved that out and no difference. So it appears to be moving on to a new directory but that directory is not identified. It is correct, /var/www/html/koel/storage/framework/cache/data/eb doesn't exist, let alone the subdirectory/file.
Error output:
  368/2412 [▓▓▓▓░░░░░░░░░░░░░░░░░░░░░░░░]  15% 2 secs18 Clash of Iron.mp3 has no changes – ignoring

  [ErrorException]
  file_put_contents(/var/www/html/koel/storage/framework/cache/data/eb/29/eb294c927370eb27bdb92407dcdd5f83281ffca3): failed to open stream: No such file or directory

Exception trace:
 () at /var/www/html/koel/vendor/laravel/framework/src/Illuminate/Filesystem/Filesystem.php:122
 Illuminate\Foundation\Bootstrap\HandleExceptions->handleError() at n/a:n/a
 file_put_contents() at /var/www/html/koel/vendor/laravel/framework/src/Illuminate/Filesystem/Filesystem.php:122
 Illuminate\Filesystem\Filesystem->put() at /var/www/html/koel/vendor/laravel/framework/src/Illuminate/Cache/FileStore.php:66
 Illuminate\Cache\FileStore->put() at /var/www/html/koel/vendor/laravel/framework/src/Illuminate/Cache/Repository.php:175
 Illuminate\Cache\Repository->put() at /var/www/html/koel/vendor/laravel/framework/src/Illuminate/Cache/Repository.php:291
 Illuminate\Cache\Repository->remember() at /var/www/html/koel/vendor/laravel/framework/src/Illuminate/Cache/CacheManager.php:305
 Illuminate\Cache\CacheManager->__call() at /var/www/html/koel/vendor/laravel/framework/src/Illuminate/Support/Facades/Facade.php:221
 Illuminate\Support\Facades\Facade::__callStatic() at /var/www/html/koel/app/Models/File.php:389
 App\Models\File->getCoverFileUnderSameDirectory() at /var/www/html/koel/app/Models/File.php:289
 App\Models\File->generateAlbumCover() at /var/www/html/koel/app/Models/File.php:257
 App\Models\File->sync() at /var/www/html/koel/app/Services/Media.php:80
 App\Services\Media->sync() at /var/www/html/koel/vendor/laravel/framework/src/Illuminate/Support/Facades/Facade.php:221
 Illuminate\Support\Facades\Facade::__callStatic() at /var/www/html/koel/app/Console/Commands/SyncMedia.php:75
 App\Console\Commands\SyncMedia->syncAll() at /var/www/html/koel/app/Console/Commands/SyncMedia.php:55
 App\Console\Commands\SyncMedia->handle() at n/a:n/a
 call_user_func_array() at /var/www/html/koel/vendor/laravel/framework/src/Illuminate/Container/BoundMethod.php:29
 Illuminate\Container\BoundMethod::Illuminate\Container\{closure}() at /var/www/html/koel/vendor/laravel/framework/src/Illuminate/Container/BoundMethod.php:87
 Illuminate\Container\BoundMethod::callBoundMethod() at /var/www/html/koel/vendor/laravel/framework/src/Illuminate/Container/BoundMethod.php:31
 Illuminate\Container\BoundMethod::call() at /var/www/html/koel/vendor/laravel/framework/src/Illuminate/Container/Container.php:539
 Illuminate\Container\Container->call() at /var/www/html/koel/vendor/laravel/framework/src/Illuminate/Console/Command.php:182
 Illuminate\Console\Command->execute() at /var/www/html/koel/vendor/symfony/console/Command/Command.php:264
 Symfony\Component\Console\Command\Command->run() at /var/www/html/koel/vendor/laravel/framework/src/Illuminate/Console/Command.php:167
 Illuminate\Console\Command->run() at /var/www/html/koel/vendor/symfony/console/Application.php:888
 Symfony\Component\Console\Application->doRunCommand() at /var/www/html/koel/vendor/symfony/console/Application.php:224
 Symfony\Component\Console\Application->doRun() at /var/www/html/koel/vendor/symfony/console/Application.php:125
 Symfony\Component\Console\Application->run() at /var/www/html/koel/vendor/laravel/framework/src/Illuminate/Foundation/Console/Kernel.php:122
 Illuminate\Foundation\Console\Kernel->handle() at /var/www/html/koel/artisan:35

Looks like your cache folder doesn't exist?

The cache folder exists but the eb cache folder does not. i.e. it's good all the way up to /var/www/html/koel/storage/framework/cache/data
This may not be pertinent.... My audio files are split between audiobooks and music. I started with the music folder, received this error and switched to the parent folder. It scanned the whole audiobooks folder, returned to the music folder and hit the error again.

Any ideas or suggestions? Debug output they might help?

Make sure all folders belong to the web server user and have sufficient rights plus enough free space on the drive (obviously).


[feature] Notifiarr integration

Version and OS
v0.45.5 on docker
Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
Could you please add integration with Notifiarr?
Additional context

Already supported, just read the Apprise information https://github.com/caronc/apprise/wiki/Notify_notifiarr


Empty request in custom store-api route

Description
I registered a custom route for the store-api (like mentioned here in the documentation: https://docs.shopware.com/en/shopware-platform-dev-en/developer-guide/controller), but sadly the request object keeps empty.

Thats the generated request-object, even if i set a json body or append query params.

Did You used JSON.parse on response object?

why json.parse? The snippets above are written in PHP. And the request-object is generated by shopware, so i think i dont need to parse anythink, or do i?

You need to pass as header Content-Type application/json otherwise shopware does not parse it


[Gekidou - MM-47656] Members List UI Changes

Summary
This PR modifies the User List UI

[x] size of avatar changed to 40px
[x] prioritize Full Name over username
[x] if Name is not set, show username in the same style as Full Name centered in the cell
[x] Username is smaller & greyed out in cases where name and username are available
[x] truncate long names to the available space
[x] check circles are moved to the right
[x] size change and style change to check circle (only change made was a shift to the right)

[x] default state
[x] selected state

[x] section size and left margin changes
[x] fix disabled users when reach the max

[x] (8 in constants, but in reality this is not working and only 7 are allowed)
[x] when no more user allowed, set the opacity

Changes are from this Figma Comment

Ticket Link
https://mattermost.atlassian.net/browse/MM-47656
Checklist

[ ] Added or updated unit tests (required for all new features)
[ ] Has UI changes
[ ] Includes text changes and localization file updates
[ ] Have tested against the 5 core themes to ensure consistency between them.

Device Information
This PR was tested on:
Screenshots

 Change
	 Before
	 After

 disabled view (disabled style added to selectable icon)

if Name is not set, show username in the same style as Full Name centered in the cell

Release Note

My guess is that haveing having a Guest and Deactivated tag on the same line would be too much? We could argue that it doesn't matter that a deactivated user was a guest, but that is up for debate.

There are other elements, but it sounds like we'll work on the chips and the bottom panel for when users are selected in other PRs, so I'll wait for follow-ups to comment on those things.

This is correct. I have broken the changes in to smaller ticket in hopes of reducing the amount of PR growth experienced in the search screen :)

@matthewbirtch, I fixed your issue 1).
For 2, 3, and 4, I suggest we create tickets for now.
I also fixed a but when a user has selected the max number of users. Now the opacity of the radio button changes to opacity of 0.16. This was in code but not getting used.

@matthewbirtch, I fixed your issue 1).
For 2, 3, and 4, I suggest we create tickets for now.

Sure, that's fine by me.

I also fixed a but when a user has selected the max number of users. Now the opacity of the radio button changes to opacity of 0.16. This was in code but not getting used.

@jfrerich for this state, I think we should grey out the entire row - just like in the design

@matthewbirtch, I made the change to add opacitcy of 0.32 to the entire row if it is disabled.


Link component ignore the basepath of LocationProvider

Could it be that you forgot to contemplate it? or maybe I'm missing something to make it work? I tried writing the basepath in all the ways
LocationProvider
<LocationProvider basepath="/subfolder/">
  <App />
</LocationProvider>

Navbar
<nav>
  <Link to="/">Home</Link>
  <Link to="/about-us">About Us</Link>
</nav>

When I click on a link, it goes to the root. The only way to make it work is to write the basepath in the "to" too, but it's not very practical.
Navbar Working
<nav>
  <Link to="/subfolder/">Home</Link>
  <Link to="/subfolder/about-us">About Us</Link>
</nav>

Thanks

This is definitely a big bug. That's supposed to work as you've
illustrated. I'll dig in and see what I can find.
On Mon, Feb 25, 2019 at 7:11 PM Nicolas Bocassi<EMAIL_ADDRESS>wrote:

Could it be that you forgot to contemplate it? or maybe I'm missing
something to make it work? I tried writing the basepath in all the ways
LocationProvider

Navbar

Home
  About Us

When I click on a link, it goes to the root. The only way to make it work
is to write the basepath in the "to" too, but it's not very practical.
Navbar Working

Home
  About Us

Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tannerlinsley/react-location/issues/3, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AFUmCU00DlfvEkaePguVp-rXO7saD8UCks5vRKXcgaJpZM4bRQad
.

Should be fixed in latest @next


About adding supported list templates ("Page" library)

In a SharePoint on-premise environment, the "Page" library (template id 850) is used by many users.
This template can't be migrated by SPMT tool.

The list template 850 is not supported

Could you clearly mention that "Page" library is not supported in this page?
https://docs.microsoft.com/en-us/sharepointmigration/sharepoint-migration-supported-list-templates

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

ID: 390c33f7-7e70-6741-2a61-9538b87bec0a
Version Independent ID: c3be3fb4-c28b-5513-651a-fe8b174a0a33
Content: SharePoint Migration Tool supported SharePoint List templates - Migrate to Microsoft 365
Content Source: migration/sharepoint-migration-supported-list-templates.md
Service: sharepoint-online
GitHub Login: @JoanneHendrickson
Microsoft Alias: jhendr

Hello @temiyaha Tetsuo,
How are you? Hope you are doing good, the article lists those templates that are supported, so everything that is not explicitly mentioned = not supported.

Hell @get-itips
Thanks for the reply. I'm doing well. Hope you are doing well too.
Some list templates are clearly listed as unsupported. (Supported by SPMT is "No")
The "Page library" (850) is used by many customers, so it's very helpful to have it clearly stated in this page.
Is it difficult to list?
Or could you mention that "everything that is not explicitly mentioned is not supported." at the top of this page?

@temiyaha Tetsuo, you are right, I'm sorry there is indeed a supported column in the table.
Will propose an improvement to the article and ask for a review.

Thank you in advance for your consideration.
Let me know if there's anything I can do to help.

Hello @get-itips
Do you have any update on this issue?

Hello @get-itips
Do you have any update on this issue?

Hello! PR is awaiting approval

Thank you for your support!


Use github actions instead of travis

:pencil: Description
Replacing travis with github actions. I'll open a second pr after this one to try to automate the publishing of docker images.

Added travis back just so the github checks pass.
Once it's merged we can replace them and I'll delete travis again.


Data Binding in ListBox-Template only working one way

I would expect clicking the button to update the UI, setting the text to "B", but the value is only changed internally but never updated in the UI.
Binding to TextBox.Text, CheckBox.IsChecked etc to set values in the model works just fine.
Hoping for a quick fix/workaround as it'll be hard to get around this in my current project...
Code to replicate:
MainWindow.xaml.cs (Usings/Namespaces omitted):
public class MainWindow : Window
{
    public TestModel[] Models { get; set; } = new[] { new TestModel() };

    public void ButtonCommand() =>
        Models[0].Prop = "B";

    public MainWindow() => InitializeComponent();
    private void InitializeComponent()
    {
        AvaloniaXamlLoader.Load(this);
        DataContext = this;
    }
}

public class TestModel : AvaloniaObject
{
    public AvaloniaProperty<string> propProp = AvaloniaProperty.Register<MainWindow, string>(nameof(Prop), "A");
    public string Prop { get => GetValue(propProp); set => SetValue(propProp, value); }
}

MainWindow.Xaml (Root element omitted):
<StackPanel>
  <ListBox Items="{Binding Models}">
    <ListBox.ItemTemplate>
      <DataTemplate>
        <TextBlock Text="{Binding Prop, Mode=TwoWay}"/>
      </DataTemplate>
    </ListBox.ItemTemplate>
  </ListBox>
  <Button Content="Switch"/>
</StackPanel>

The replication code was garbage but that's besidet the point. I fixed it somehow. Don't ask me how, all I did was that I changed AvaloniaProperty.Register<Operation, string> to AvaloniaProperty.Register<MainWindow, string>, and that works. I'm confused but also glad that maybe this wasn't a bug after all.


DOCK-2383: changed User auth to optional

Description
This PR changes the User authentication in the users/checkUser endpoint to Optional.
Before fix:

After fix:

Review Instructions
Log out and go to https://qa.dockstore.org/users/hyunnaye and confirm that the credentials error is not shown.
Issue
https://github.com/dockstore/dockstore/issues/5480
Security
If there are any concerns that require extra attention from the security team, highlight them here.
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that you pass the basic style checks and unit tests by running mvn clean install
[x] Ensure that the PR targets the correct branch. Check the milestone or fix version of the ticket.
[x] Follow the existing JPA patterns for queries, using named parameters, to avoid SQL injection
[x] If you are changing dependencies, check the Snyk status check or the dashboard to ensure you are not introducing new high/critical vulnerabilities
[x] Assume that inputs to the API can be malicious, and sanitize and/or check for Denial of Service type values, e.g., massive sizes
[x] Do not serve user-uploaded binary images through the Dockstore API
[x] Ensure that endpoints that only allow privileged access enforce that with the @RolesAllowed annotation
[x] Do not create cookies, although this may change in the future
[x] If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.

Will fix this in UI instead with https://github.com/dockstore/dockstore-ui2/pull/1772


cannot create .jar

Hi!  can you post or send me the .jar resultant file? I can't achieve it! :(   if I try to create the .jar with "mvn package" I obtain this error:
"Error: Could not find or load main class src.main.java.syno.fileduplicateremover.SynoFileDuplicateRemover"
Some idea? thanks! :)

Hi, I am using Maven v.3.3.1. No issues. Have you tried from the root of the project?
synofileduplicateremover-0.0.1-SNAPSHOT.zip
Just change the extension back to .jar


fix: collapse frame under modal for screen reader support

GitHub Issue: #

Proposed Changes

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

What is the current behavior?

When using a screen reader (Narrator, TalkBack, VoiceOver, ...), if the user opens a modal, the screen reader has access to the elements of the previous frame (the one under the modal).
What is the new behavior?

When using a screen reader (Narrator, TalkBack, VoiceOver, ...), if the user opens a modal, the screen reader does not have access to the elements of the previous frame (the one under the modal).
Impact on version

[ ] Major (Public API was modified.)

Public constructs (class, struct, delegate, enum, etc.) were removed or renamed.
Public members were removed or renamed.
Public method signatures were changed or renamed.

[ ] Minor (Public API was extended.)

Public constructs, members, or overloads were added.

[X] Patch (Public API was unchanged.)

A bug in behavior was fixed.
Documentation was changed.

[ ] None (The library is unchanged.)

Only code under the build folder was changed.
Only code under the .github folder was changed.

Checklist
Please check that your PR fulfills the following requirements:

[ ] Documentation has been added/updated.
[ ] Automated Unit / Integration tests for the changes have been added/updated.
[ ] Updated BREAKING_CHANGES.md (if you introduced a breaking change).
[X] Your conventional commits are aligned with the Impact on version section.

Other information

/azp run


Update -> 0.31.0

The extension lacks v0.31.0 min, extension should be updated to version v0.31.0.

Yes yes I know! Gotta remove some highlighting for some of the old sigils, add lambda etc.
I actually do most of my min coding on my iPhone and iPad these days, while for the extension I need to try it out on a laptop, so... 😬

Yes yes I know! Gotta remove some highlighting for some of the old sigils, add lambda etc.
I actually do most of my min coding on my iPhone and iPad these days, while for the extension I need to try it out on a laptop, so... 😬

Sorry 😁


Error: Could not get remote files

When running the actual-ai Docker container, I get the following error:
><EMAIL_ADDRESS>dev-docker
> ../node_modules/.bin/nodemon --exec "npm run lint && node ./app.js"

[nodemon] 2.0.22
[nodemon] to restart at any time, enter `rs`
[nodemon] watching path(s): *.*
[nodemon] watching extensions: js,mjs,json
[nodemon] starting `npm run lint && node ./app.js`

><EMAIL_ADDRESS>lint
> eslint --fix **/*.js

Application started
Starting classification process
An error occurred: Error: Could not get remote files
    at handlers.api/download-budget (/opt/node_app/node_modules/@actual-app/api/dist/app/bundle.api.js:29444:31)
node:internal/process/promises:288
            triggerUncaughtException(err, true /* fromPromise */);
            ^

Error: Could not get remote files
    at handlers.api/download-budget (/opt/node_app/node_modules/@actual-app/api/dist/app/bundle.api.js:29444:31)

Node.js v18.20.4
[nodemon] app crashed - waiting for file changes before starting...

I am self-hosting the actual-server on my Synology NAS. As a result, I am using a self-signed SSL certificate. I believe the is the cause of my issue. I received the same error with actual-helpers, and the developer of that project was able to fix it by adding a config option to permit the actual-api to allow self-signed SSL certs.
https://github.com/psybers/actual-helpers/issues/2
These are the changes I see via GitHub, but I have asked the project owner for further details on the fix, and will provide updates as I get them.

Changed files for Self-Signed Certifiates

Dockerfile
Add:
# Allow self-signed SSL certs
ENV NODE_TLS_REJECT_UNAUTHORIZED=0

.env
Add:
# allow self-signed SSL certs
NODE_TLS_REJECT_UNAUTHORIZED=0

Thanks in advance for the help!

Could you try add that environment variable to the container?
Like this:
actual-ai:
    image: docker.io/sakowicz/actual-ai:latest
    restart: unless-stopped
    environment:
      - ACTUAL_SERVER_URL=http://actual_server:5006
      - ACTUAL_PASSWORD=your_actual_password
      - ACTUAL_BUDGET_ID=your_actual_budget_id # This is the ID from Settings → Show advanced settings → Sync ID
      - CLASSIFICATION_SCHEDULE_CRON=0 */4 * * * # How often to run classification.
      - CLASSIFY_ON_STARTUP=true # Whether to classify transactions on startup (don't wait for cron schedule)
      - SYNC_ACCOUNTS_BEFORE_CLASSIFY=false # Whether to sync accounts before classification
      - OPENAI_API_KEY=your_openai_api_key
#      - OPENAI_MODEL= # optional. required if you want to use a specific model, default is "gpt-3.5-turbo-instruct"
#      - OPENAI_BASE_URL= # optional. required if you don't want to use the OpenAI API but OpenAI compatible API
#      - ACTUAL_E2E_PASSWORD= # optional. required if you have E2E encryption
      - NODE_TLS_REJECT_UNAUTHORIZED=0

And let me know if it helped.

Same error

Update: My Docker skills need some work. Adding the environment variable to the docker-compose.yml did solve the specified problem.
Now I'm getting a new error further along in the process. I'll troubleshoot and open a new issue, if needed.
Thanks!

For me this keeps occurring as well with the following actualai.env config mapped to the Docker container:
ACTUAL_SERVER_URL="http://<IP_ADDRESS>:5004"
ACTUAL_PASSWORD="..."
ACTUAL_BUDGET_ID="e3e2625b-..."
CLASSIFICATION_SCHEDULE_CRON="0 */4 * * *"
CLASSIFY_ON_STARTUP=true
SYNC_ACCOUNTS_BEFORE_CLASSIFY=false

LLM_PROVIDER=ollama
OPENAI_API_KEY=x
OLLAMA_BASE_URL=http://<IP_ADDRESS>:11434
OLLAMA_MODEL=llama3.2

NODE_TLS_REJECT_UNAUTHORIZED=0

And docker compose file:
 actual_server:
    image: docker.io/actualbudget/actual-server
    container_name: actual_server
    env_file: actualbudget.env
    ports:
      - 5004:5006
    volumes:
      - ./data:/data
      - ./certs:/certs
    restart: unless-stopped

  actual_ai:
    image: docker.io/sakowicz/actual-ai
    container_name: actual_ai
    depends_on: [actual_server]
    env_file: actualai.env
    restart: unless-stopped

It's this the exact docker-compose file? Are these containers in the same network?
IMO, you should use ACTUAL_SERVER_URL="http://actual_server:5004"
You can try to get into the actual_ai container and check via curl if the connection is OK.

Yep this is within the same compose.yaml. The IP I'm using here is the internal Docker IP which should always just route to the correct container in the engine.
Changing to the container name or service name from like you suggest has the same result. I can not docker exec into the container to open a shell because it keeps quitting/restarting/exiting 1


Different behaviour Questionnaire/$extract and Questionnaire/id/$extract

The second one does not validate context

#17 we should remove context validation everywhere


VBA 64 bits reading error

If I declare a dictionary from another module and use it in a function of another module then when trying to read a key using myDic("myKey") while in the debugger the keys are shown, the value returned is "Empty"
It works as expected on 32 bits version of excel but not on 64 bits
If I remove the dictionary module to use the default windows dictionary it works fine again

Related to https://github.com/VBA-tools/VBA-JSON/issues/111

I'm also noticing this error with the dictionary only on 64 bit versions.  The read seems to work ok the first time but fails on subsequent reads.  This is also quite variable as to whether it fails or not - doesn't seem to be any discernible pattern.  My temp fix is to read the value only once.

Hi,
On 64 bits excel when i try to read a value like that  LastName = Item("subscriber")("lastName")
I received a message : Automation Error

same problem here.

Allawys use 32 bit if you want your macros or VBA to work properly , the 64
bit is a problem , personally I stoped entertaining it anymore. I used to
have problems with Ms Access 2016 , I had to switch back to 32 BIT because
it even allows third party products to run side by side
Managing Partner
Ca Premier Accounting Package Developer - Zambia
Nector Prime Accounting Solutions
Findeco House 7th Floor Room 8
Cairo Road
Lusaka, Zambia
Nector Prime Accounting Solution is a legally independent firm of
Chartered Accountants and does not accept any responsibility or liability
for the actions or inaction on the part of any other individual
unaffiliated firm or firms.
P  Please do not print this e-mail unless absolutely necessary. Save our
ENVIRONMENT.
This e-mail may contain privileged and confidential information. If you are
not the intended recipient, you must not copy or distribute it. If you have
received this e-mail in error, please notify the sender and delete the
e-mail immediately. You should ensure that this e-mail and attachment is
virus free. E-mail is not a 100% secure communication medium. We advise you
understand this lack of security and take any necessary measures when
emailing us. We reserve the right to read any email or attachment entering
or leaving our systems from any source without prior notice. Information
about Nector Prime Accounting Solutions can be found at our offices at
Findeco House 7th Floor Room 8 or Call us on Cell No. +260977352401
On Mon, 28 Oct 2019 at 22:47, Felix-N<EMAIL_ADDRESS>wrote:

same problem here.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/VBA-tools/VBA-Dictionary/issues/21?email_source=notifications&email_token=ALEEAHLYQPR3UQDTWFYW6KTQQ5FVBA5CNFSM4HGVALVKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECOKZWY#issuecomment-547138779,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALEEAHMUMOF3R76CO3FX4DTQQ5FVBANCNFSM4HGVALVA
.


Fixed CSV file upload does not correctly handle 'UTF-8 with BOM' encoding

Closes #5812
What's Changed

Used a fresh .csv file and the failing .csv file in the issue description to test this fix and they both succeeded.
Screenshots

N/A
TODO

[x] Explanation of Change(s)
[x] Added change log fragment(s) (for more information see the documentation)
[ ] Attached Screenshots, Payload Example
[ ] Unit, Integration Tests
[ ] Documentation Updates (when adding/changing features)
[ ] Example App Updates (when adding/changing features)
[ ] Outline Remaining Work, Constraints from Design

Let's confirm that this also fixes #5985 as well.


Opening accordion from controller

I'm using ng-repeat to show accordions and I want to track which accordion is selected so that I can open it from the controller. So is there any way to achieve this.

Any feedback for above issue? I am trying same. @HarishSha  have you got any luck to do that?


[SYSTEMML-1391] Update project compilation to Java 8

Update maven-compiler-plugin source and target to 1.8.
Minor updates to MLBlock for Java 8 in Eclipse (Neon) on OS X.
Add JavaPairRDD casts in FrameConverterTest for Java 8.

All tests passed here: https://sparktc.ibmcloud.com/jenkins/job/SystemML-PullRequestBuilder/1219/

LGTM - thanks @deroneriksson

Thanks for reviewing @mboehm7!


Discovery index for private and withdrawn items

This PR is limited to discovery configuration and tests.
It creates a new discovery configuration discoverableAndUndiscoverableItems which can be used for admins to find withdrawn and private items. It includes facets to verify the withdrawn and private state.
The discovery configuration can be used by anyone, but the SolrServiceResourceRestrictionPlugin will ensure only admins can find withdrawn and private items. This is also displayed in the tests

Thanks for the update, @benbosman .  Merging as this is at +2.


Fix typos in documentation

I noticed some typos while reading through the markdown files.

Brilliant, thanks!


[dask] xgb.dask.train() fails on dask-kubernetes cluster

I tried tonight to test the recent xgboost.dask changes on a dask-kubernetes cluster on EKS (per https://github.com/dmlc/xgboost/pull/6343#issuecomment-722494918).
Unfortunately, I ran into this error:

AttributeError: /opt/conda/envs/saturn/lib/libxgboost.so: undefined symbol: XGDMatrixSetDenseInfo

Reproduction Information
training code
I omitted the code I used to create my client (...[CLIENT CODE]...) because it uses a dask-kubernetes cluster provisioned with a commercial product. I can see that work is getting scheduled onto that cluster when the DaskDMatrix is set up and when training starts, so I'm confident that that isn't the issue.
import os
import time

import dask.array as da
import xgboost as xgb

from dask.distributed import Client, wait
from dask_ml.metrics import mean_absolute_error
from dask_saturn import SaturnCluster

.....[CLIENT CODE].....

num_obs = 1e5
num_features = 50

X = da.random.random(
    size=(num_obs, num_features),
    chunks=(1000, num_features)
)
y = da.random.random(
    size=(num_obs, 1),
    chunks=(1000, 1)
)

X = X.persist()
_ = wait(X)

y = y.persist()
_ = wait(y)

dtrain = xgb.dask.DaskDMatrix(
    client=client,
    data=X,
    label=y
)

bst = xgb.dask.train(
    client=client,
    params={
        "verbosity": 2,
        "tree_method": "hist",
        "objective": "reg:squarederror"
    },
    dtrain=dtrain,
    num_boost_round=10,
)

I installed xgboost by cloning from latest master (https://github.com/dmlc/xgboost/tree/fcfeb4959c6e361f2fd1cd18c3b61b598dc205ae).
sudo apt update
sudo apt-get install -y cmake build-essential
git clone https://github.com/dmlc/xgboost.git /tmp/xgboost
pushd /tmp/xgboost/python-package
    git submodule init
    git submodule update
    python setup.py install
popd

full stacktrace
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-5-462f078606cf> in <module>
      8     dtrain=dtrain,
      9     num_boost_round=10,
---> 10     evals=[(dtrain, 'train')]
     11 )

/srv/conda/envs/saturn/lib/python3.7/site-packages/xgboost/dask.py in train(client, params, dtrain, evals, early_stopping_rounds, *args, **kwargs)
    742     return client.sync(
    743         _train_async, client, params, dtrain=dtrain, *args, evals=evals,
--> 744         early_stopping_rounds=early_stopping_rounds, **kwargs)
    745
    746

/srv/conda/envs/saturn/lib/python3.7/site-packages/distributed/client.py in sync(self, func, asynchronous, callback_timeout, *args, **kwargs)
    831         else:
    832             return sync(
--> 833                 self.loop, func, *args, callback_timeout=callback_timeout, **kwargs
    834             )
    835

/srv/conda/envs/saturn/lib/python3.7/site-packages/distributed/utils.py in sync(loop, func, callback_timeout, *args, **kwargs)
    338     if error[0]:
    339         typ, exc, tb = error[0]
--> 340         raise exc.with_traceback(tb)
    341     else:
    342         return result[0]

/srv/conda/envs/saturn/lib/python3.7/site-packages/distributed/utils.py in f()
    322             if callback_timeout is not None:
    323                 future = asyncio.wait_for(future, callback_timeout)
--> 324             result[0] = yield future
    325         except Exception as exc:
    326             error[0] = sys.exc_info()

/srv/conda/envs/saturn/lib/python3.7/site-packages/tornado/gen.py in run(self)
    733
    734                     try:
--> 735                         value = future.result()
    736                     except Exception:
    737                         exc_info = sys.exc_info()

/srv/conda/envs/saturn/lib/python3.7/site-packages/xgboost/dask.py in _train_async(client, params, dtrain, evals, early_stopping_rounds, *args, **kwargs)
    705         futures.append(f)
    706
--> 707     results = await client.gather(futures)
    708     return list(filter(lambda ret: ret is not None, results))[0]
    709

/srv/conda/envs/saturn/lib/python3.7/site-packages/distributed/client.py in _gather(self, futures, errors, direct, local_worker)
   1849                             exc = CancelledError(key)
   1850                         else:
-> 1851                             raise exception.with_traceback(traceback)
   1852                         raise exc
   1853                     if errors == "skip":

/srv/conda/envs/saturn/lib/python3.7/site-packages/xgboost/dask.py in dispatched_train()
    654         worker = distributed.get_worker()
    655         with RabitContext(rabit_args):
--> 656             local_dtrain = _dmatrix_from_list_of_parts(**dtrain_ref)
    657             local_evals = []
    658             if evals_ref:

/opt/conda/envs/saturn/lib/python3.7/site-packages/xgboost/dask.py in _dmatrix_from_list_of_parts()
    605     if is_quantile:
    606         return _create_device_quantile_dmatrix(**kwargs)
--> 607     return _create_dmatrix(**kwargs)
    608
    609

/opt/conda/envs/saturn/lib/python3.7/site-packages/xgboost/dask.py in _create_dmatrix()
    595                       feature_names=feature_names,
    596                       feature_types=feature_types,
--> 597                       nthread=worker.nthreads)
    598     dmatrix.set_info(base_margin=base_margin, weight=weights,
    599                      label_lower_bound=label_lower_bound,

/opt/conda/envs/saturn/lib/python3.7/site-packages/xgboost/core.py in __init__()
    506         self.handle = handle
    507
--> 508         self.set_info(label=label, weight=weight, base_margin=base_margin)
    509
    510         self.feature_names = feature_names

/opt/conda/envs/saturn/lib/python3.7/site-packages/xgboost/core.py in inner_f()
    419         for k, arg in zip(sig.parameters, args):
    420             kwargs[k] = arg
--> 421         return f(**kwargs)
    422
    423     return inner_f

/opt/conda/envs/saturn/lib/python3.7/site-packages/xgboost/core.py in set_info()
    527         '''Set meta info for DMatrix.'''
    528         if label is not None:
--> 529             self.set_label(label)
    530         if weight is not None:
    531             self.set_weight(weight)

/opt/conda/envs/saturn/lib/python3.7/site-packages/xgboost/core.py in set_label()
    656         """
    657         from .data import dispatch_meta_backend
--> 658         dispatch_meta_backend(self, label, 'label', 'float')
    659
    660     def set_weight(self, weight):

/opt/conda/envs/saturn/lib/python3.7/site-packages/xgboost/data.py in dispatch_meta_backend()
    663         return
    664     if _is_numpy_array(data):
--> 665         _meta_from_numpy(data, name, dtype, handle)
    666         return
    667     if _is_pandas_df(data):

/opt/conda/envs/saturn/lib/python3.7/site-packages/xgboost/data.py in _meta_from_numpy()
    597     ptr = interface['data'][0]
    598     ptr = ctypes.c_void_p(ptr)
--> 599     _check_call(_LIB.XGDMatrixSetDenseInfo(
    600         handle,
    601         c_str(field),

/opt/conda/envs/saturn/lib/python3.7/ctypes/__init__.py in __getattr__()
    375         if name.startswith('__') and name.endswith('__'):
    376             raise AttributeError(name)
--> 377         func = self.__getitem__(name)
    378         setattr(self, name, func)
    379         return func

/opt/conda/envs/saturn/lib/python3.7/ctypes/__init__.py in __getitem__()
    380
    381     def __getitem__(self, name_or_ordinal):
--> 382         func = self._FuncPtr((name_or_ordinal, self))
    383         if not isinstance(name_or_ordinal, int):
    384             func.__name__ = name_or_ordinal

AttributeError: /opt/conda/envs/saturn/lib/libxgboost.so: undefined symbol: XGDMatrixSetDenseInfo

output of conda info`
     active environment : saturn
    active env location : /opt/conda/envs/saturn
            shell level : 2
       user config file : /home/jovyan/.condarc
 populated config files : /opt/conda/.condarc
          conda version : 4.8.2
    conda-build version : not installed
         python version : 3.7.7.final.0
       virtual packages : __glibc=2.28
       base environment : /opt/conda  (writable)
           channel URLs : https://conda.saturncloud.io/pkgs/linux-64
                          https://conda.saturncloud.io/pkgs/noarch
                          https://conda.anaconda.org/conda-forge/linux-64
                          https://conda.anaconda.org/conda-forge/noarch
                          https://repo.anaconda.com/pkgs/main/linux-64
                          https://repo.anaconda.com/pkgs/main/noarch
                          https://repo.anaconda.com/pkgs/r/linux-64
                          https://repo.anaconda.com/pkgs/r/noarch
          package cache : /opt/conda/pkgs
                          /home/jovyan/.conda/pkgs
       envs directories : /opt/conda/envs
                          /home/jovyan/.conda/envs
               platform : linux-64
             user-agent : conda/4.8.2 requests/2.22.0 CPython/3.7.7 Linux/4.14.193-149.317.amzn2.x86_64 debian/10 glibc/2.28
                UID:GID : 1000:100
             netrc file : None
           offline mode : False

Other Notes
I'll try to come up with a reproducible example using dask-cloudprovider so that' it's 100% reproducible (no redacted code).

I think you have an outdated libxgboost.so.  Could you please check your image?

Yep you're right, I ran find / -name "libxgboost.so"and found that even this doesn't remove some old libxgboost.so that I have on PATH
conda uninstall -y xgboost
pip uninstall -y xgboost

I'll remove the old library from my image and try again, thanks.

Thanks for testing!  Feel free to let me know if there's anything I can help.  I will close this one now as this specific issue is resolved.

@trivialfis I'm very happy to tell you that after I was able to clear out my old libxgboost.sos, training worked on EKS! (using the code snippet I shared above).
Thanks for all the great work!!! :tada:


fix(es/lexer): Fix lexing of numbers in e notation with large exponent (#3060)

Description:
This fixes lexer panicking on numbers in scientific (e) notation with large exponents. See #3060 for the more detailed description of the problem and why the panic happens.
The proposed fix is to first check if exponent became INFINITY after parsing and, it that case, just return a fixed value instead of trying to form the string which the whole number and parse it. That fixed value is determined as follows:

it is INFINITY if exponent was positive and mantissa was nonzero
it is 0.0 otherwise (is exponent was negative or mantissa was zero)

This behaviour matches what browsers (tested Chrome and FF) and Babel do (at least if I did not miss something).
Also, a couple of tests is added to crates/swc_ecma_parser/src/lexer/number.rs.
Related issue (if exists):
Closes #3060

 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.

Cool! ^_^
You're welcome -)
Thank you for making SWC!


Cancel flow actions?

In my (MST) app I have encountered the need to be able to cancel flow actions. Does mobx-keystone support this or could it easily support it?
There's a longstanding MST PR with discussion about it here https://github.com/mobxjs/mobx-state-tree/pull/691. I am currently running a fork of MST with flow cancellation and it works well.
Specifically the use case for this is long running flows that are yielding awaiting some condition that may or may not happen before the user navigates away or takes some other action that invalidates them. Without the ability to cancel flows these yields would never continue, resulting in a memory leak.

You may use a cancellation token pattern to cancel flows/promises/anything async as shown here:
https://codesandbox.io/s/cancellation-token-kepth?file=/src/index.ts

@xaviergonz thanks for that example. I don't think that pattern would work for the case I'm envisioning. Imagine a flow action modeling a sequence of steps a user takes in a UI or a series of animations that need to play one after another:
runSequenceOfSteps =  _async(function* (this: MyModel) {
        for(const step of this.steps){
              //this promise might never resolve depending on what the user does
              //for example if they navigate away from the screen before the condition for the step is satisfied
             yield* _await(conditionsForStepPromise())
              //imagine these steps are complex, needing to call and await many other things
        }
}

//elsewhere in app
myModelInstance.runSequenceOfSteps()
//now it is possible for runSequenceOfSteps to yield forever because a promise for a step
//might never get resolved or rejected!

It would be beneficial to be able to cancel a flow like this that could yield a promise that might not ever get resolved. Especially in more complex user interaction cases like a series of steps outlined above. I think it's probably fairly common at least in UI for async actions to become "invalid" or no longer relevant while they await some condition.
The only alternative I can see to being able to cancel flows is to painstakingly ensure that every promise yielded in a flow will resolve/reject in every possible case in which the flow might need to cancel. By doing that I think it would be possible to use the cancellation token pattern you suggested. I think however it would be quite easy to miss a case and end up with a flow "stuck" on a promise that will never resolve and cause a memory leak or unexpected behavior.
I know promise cancellation is not a standard thing, but so far I can't think of another way to handle this sort of situation gracefully.
This is how I am using a patched version of MST with flow cancellation:
    .actions((self) => {
        let pendingPromises: CancellablePromise<FlowReturn<any>>[] = []

        /**
         * Based on https://github.com/mobxjs/mobx-state-tree/issues/681
         * Added a .cancel() function to flows so they can be stopped if they're waiting for a yield
         *
         * This wraps a flow and keeps a reference to its promise so it can be cancelled
         * @param generator
         */
        function trackedFlow<R, Args extends any[]>(
            generator: (...args: Args) => Generator<Promise<any>, R, any>
        ): (...args: Args) => Promise<FlowReturn<R>> {
            //Make flow function
            const theFlow = flow(generator)
            //Return a wrapper function that keeps track of when it's called and cleans up after it
            return function wrappedFlow(...args: Args): Promise<FlowReturn<R>> {
                //Call the flow
                const p = theFlow(...args)
                //Record the promise
                pendingPromises.push(p)
                //When promise is resolved or rejected remove the flow from tracked
                p.finally(function removeCompletedFlow() {
                    _.remove(pendingPromises, (pending) => pending === p)
                })
                //Return the promise to caller
                return p
            }
        }

      const myAsyncAction = trackedFlow(function* myAsyncAction( ) {
            .....
            yield when(() => self.somethingThatMightOrMightNotHappenBeforeDestroy)
            .....
      })

     function beforeDestroy() {
            //Cancel all outstanding flows when the model is torn down
            pendingPromises.forEach((p) => p.cancel())
            pendingPromises = []
        }

    return {
        beforeDestroy,
       myAsyncAction
     }
})

Essentially there are conditions in which running flows no longer matter to the app. They just needs to be stopped and cleaned up right away but they could be awaiting promises that now cannot be resolved. A cancel method for flows makes that case easy and clean to handle.
I could also be missing a cleaner and easier way to handle this case however!

What about using a lib that supports cancellable promises (e.g. bluebird http://bluebirdjs.com/docs/api/cancellation.html) and do state mutation via sync actions inside that pormise?

I think that would probably work but might have a few downsides, primarily

Requires another lib
Requires writing cancellable flows in a "special" way different from the standard mobx-keystone flows
Requires defining sync actions for all mutations even if the only place they're used is in the cancellable flow

I'll have to give this approach a try when I get some time to see how it pans out in practice however.

Would be nice to support this without using cancellation tokens


Broken help link

In the Extensions tab of a user's profile (file accounts/templates/profile/extensions.html), the help link points to {{HELP_URL}}extensions.html but it should point to {{HELP_URL}}extensions/index.html.
Thanks for the great work!

True, Reproducible on the editor hosted at https://numbas.mathcentre.ac.uk which redirects to https://docs.numbas.org.uk/en/latest/extensions.html instead of https://docs.numbas.org.uk/en/latest/extensions/. I am fixing this and sending a PR 😄


Ninja eclipse generator uses make

When using the Eclipse CDT4 generator and specifying the ninja build system
catkin_make -G"Eclipse CDT4 - Ninja"

catkin_make tries to invoke make
####
#### Running command: "make -j4 -l4" in {removed}
####
make: *** No targets specified and no makefile found.  Stop.
Invoking "make -j4 -l4" failed

When attempting to add the --use-ninja argument
catkin_make -G"Eclipse CDT4 - Ninja" --use-ninja

the user will receive the following error message.
Error: either specify a generator using '-G...' or '--use-ninja' but not both

As a work around, the following lines were commented out in the catkin_make script
    elif args.use_ninja:
        return fmt("@{rf}Error: either specify a generator using '-G...' or '--use-ninja' but not both")

Just passing the CMake argument -G... is not enough. Instead pass the option --use-ninja and catkin_make will invoke the right tool.

If I use both, I get an error. I don't want to only use ninja; I'd also like to generate eclipse project files at the same time. There's currently no way of doing that without applying the hacky fix I specified above.

Currently --use-ninja implies -G Ninja. For your case it would be possible to extend the code to accept a custom -G argument and use that while also using --use-ninja to invoke the right command afterwards.
I will reopen the ticket. If you are interested in this feature please consider providing a pull request for it.

Closing due to inactivity.


Add workflow to regenerate code

Add workflow to regenerate code.
User experience:

Go to github actions from user's fork repo.
Click workflow "Regenerate all code".
Choose the branch you are working on, and click "Run workflow".

Result: It helps you run eng\Generate.ps1 and push a new commit to your branch like:

Sample workflow link: https://github.com/pshao25/autorest.csharp/actions/workflows/main.yml
Sample PR link: https://github.com/pshao25/autorest.csharp/pull/1

Please run eng/DownloadSharedSource.ps1 to update the shared code.


Instance segmentation on CoCo

Hi Kaiming, will you share your code or config of transferring to instance segmentation on CoCo.

In https://github.com/facebookresearch/moco/tree/master/detection/configs


support pub struct IDb(pub Opaque<Surreal<Db>>)

I'm not sure if that mode is supported. Or how to support it
pub struct IDb(pub Opaque<Surreal<Db>>);

@sealed
class SurrealDb extends FrbOpaque {
  final Db bridge;
  SurrealDb.fromRaw(int ptr, int size, this.bridge) : super.unsafe(ptr, size);
  @override
  DropFnType get dropFn => bridge.dropOpaqueSurrealDb;  //  error

  @override
  ShareFnType get shareFn => bridge.shareOpaqueSurrealDb;

  @override
  OpaqueTypeFinalizer get staticFinalizer => bridge.SurrealDbFinalizer;
}

class IDb {
  final SurrealDb field0;

  IDb({
    required this.field0,
  });
}

error

The getter 'shareOpaqueSurrealDb' isn't defined for the type 'Db'.
Try importing the library that defines 'shareOpaqueSurrealDb', correcting the name to the name of an existing getter, or defining a getter or field named 'shareOpaqueSurrealDb'.

Is it related to 773?

Hmm looks like a bug.
/cc @rogurotus who implemented the wonderful Opaque feature - a naive guess, maybe we forget to trigger code generation for this case?

I was unable to reproduce the error. Can I take a look at the generated Db (frb api class)

https://github.com/huang12zheng/flutter_rust_bridge_template/tree/reproduce_863
you can find object from native/src/surreal.rs
To be honest, I just saw Opaque today.
I probably don't understand the code generation properly
@rogurotus

flutter_rust_bridge/frb_codegen/src/generator/rust/mod.rs  line - 139
Yes, this is indeed a bug. related functions are generated for arguments, not for return type.
        lines += distinct_input_types (should be distinct_output_types)
            .iter()
            .map(|f| self.generate_related_funcs(f, ir_file))
            .collect();

until fixed, you can create an empty function:
pub fn touch(opauqe: Opaque<Surreal<Db>>) {}


attributes and hash escaping

I think I've founding an interesting corner case with attributes and escaping though it may just be ignorance of the proper escaping rules (in which case please show me where to RTFM)
I have two styles:
.red {
   color: #ff0000
}
.blue {
   color: #0000ff
}

I wish to render some text as follows:
#red
#blue
normal

Now in an asciidoc sandbox.adoc I put:
[red]##I am red# +
[blue]##I should be blue but I'm not# +
I am normal

The first coloured line containing a leading hash comes out red
The second coloured line containing a leading hash does not come out blue.
In the html I get:

I am red#

[blue]I should be blue but I’m not#

I am normal

This also happens if I try using ## to obtain a hash as in:
[red]###I am red# +
[blue]###I should be blue but I'm not# +
I am normal

If I use # in the input I get # in the output.
Another interesting case is:
[red]#\\## I am red# +
[blue]#\\## I am blue# +
I am normal

Which yields one red hash on the first line and two blue hashes on the second line.
I as workaround I do:
:hash: #

[red]#{hash} I am red# +
[blue]#{hash} I am blue# +
I am normal

Note that http://asciidoctor.org/docs/user-manual/ suggests I should use:
[rolename]`monospace text`

But the backtick has an effect of its own.
[red]`hello world`

comes out as:
hello world

There are a few ways to go about this. First, if your content includes a formatting character, then you definitely need to be using the unconstrained formatting at the very least.
[red]###I am red## +
[blue]###I am blue## +
I am normal

However, visually parsing that is tough. That's why you probably want to either switch to using a character reference for the content part:
[red]##I am red# +
[blue]##I am blue# +
I am normal

or by using an attribute reference:
:hash: #

[red]#{hash}I am red# +
[blue]#{hash}I am blue# +
I am normal

(You could also set the value of hash to # to be safe).
A more brute force approach is to use an inline passthrough, but that should really only be done as a last resort.
[red]#++#++I am red# +
[blue]#++#++I am blue# +
I am normal

Attributes are very powerful because they allow you to inject content without being interpreted as formatting marks.

But the backtick has an effect of its own.

Yes, backtick is for when you want to make the text monospaced. The hash is for unformatted text (role only).


Update package imports to "apache" (github) relative

As we have moved to Apache org. under Github,  we should change package import statements to reflect this change...
for example:
import (
"github.com/openwhisk/openwhisk-client-go/whisk"
"github.com/openwhisk/openwhisk-wskdeploy/config"
}
should be:
import (
"github.com/apache/incubator-openwhisk-client-go/whisk"
"github.com/apache/incubator-openwhisk-wskdeploy/config"
}

close as merged


Spacy3.x not supported (tag_map no longer exists)

Hi -- I'm trying to make use of PyMagnitude in a Docker image that has Spacy 3.0.6 installed. When trying to load the downloaded vectors I receive the following error:
---------------------------------------------------------------------------
ImportError                               Traceback (most recent call last)
<ipython-input-37-54185549ba8b> in <module>
      1 import spacy
----> 2 from spacy.lang.en import tag_map

ImportError: cannot import name 'tag_map' from 'spacy.lang.en' (/opt/conda/envs/nlp/lib/python3.8/site-packages/spacy/lang/en/__init__.py)

A bit of Googling turned up:
The TAG_MAP and MORPH_RULES in the language data have been replaced by the more flexible AttributeRuler

See further info on Backwards Incompatibilities.

Same issue here.


Allow text wrapping on Action buttons when layout is vertical

Hello, we are receiving customer reports that they would like text to wrap on Action buttons in WebChat
https://github.com/Microsoft/BotFramework-WebChat/issues/616
Seems like this is a reasonable request if button layout is in vertical mode.

Seems reasonable to me too.

The question is, should it be a host config option? Will some host want to prevent wrapping?

It's possible...
At the very least we should provide a tooltip, in case text is truncated with ellipses.

Any update on this?

I would like to at least control the action button layout. The WebChat defaults to horizontal whereas Skype defaults to vertical. I would like to be able to specify vertical in the WebChat for the action button layout for times where a choice of buttons (like the Bot Framework choice prompt) is needed, but without forcing users to deal with a horizontal carousel scroll.

I would also like the text on buttons to wrap. In some cases I am asking the user which question they meant, and some questions are longer and get truncated. As danmarshall said above, at least provide a tooltip if the text is truncated.

This is a good suggestion, but not something that's on our roadmap at this time. We may reconsider it in the future, or if you would like to make a contribution please see our contribution guidelines here.

We have a new request for this same feature: https://github.com/microsoft/AdaptiveCards/issues/6198


[terra-application-utility] Add theme context

Summary
Add theme context to terra-application-utility.
Deployment Link

https://terra-framework-deployed-pr-#.herokuapp.com/
Testing

Additional Details

Thank you for contributing to Terra.
@cerner/terra

I see you have two updated screenshots. Why did they change?

I see you have two updated screenshots. Why did they change?

Some unusual behaviour, was facing similar issue in clinical theme PR , Fixed now 758ba11


ReactDOM - findDOMNode

Use findDOMNode from ReactDOM.

duplicate of #195, #190, #164, #145, #148, #128, #114, #110

@chentsulin Sorry for making you type all those numbers. :) Thank you for your comment.

It's doesn't matter. Welcome for any contribution to OSS projects ：）


Improve position of outgoing/incoming demand button

This is the button that appears on PT info view when Advanced Parking AI is active: https://github.com/CitiesSkylinesMods/TMPE/wiki/Public-Transport-Info-View
Allystrya 12 hours ago:

Is there a way to move the outgoing/incoming demand button? I have tried different mouse and keyboard w/mouse combinations but nothing seems to work and I didn't see anything in the Wiki.
I didn't even know where it was until here recently when I was sent a screenshot but because of the resolution I have to use it sits right behind the transport window and I can barely see the top of it. I can tell I can click it because of the buildings that change however i am not sure if I am on outgoing on incoming because I can't see the rest :)
If there isn't a way to move it, then is there any way to tell which demand channel I am on?

Btw, I think there is no info in wiki about Parking space demand view when you select Traffic info view while TM:PE menu is visible

The what view?
These are what I currently have on the info views:

https://github.com/CitiesSkylinesMods/TMPE/wiki/Public-Transport-Info-View
https://github.com/CitiesSkylinesMods/TMPE/wiki/Traffic-Info-View
https://github.com/CitiesSkylinesMods/TMPE/wiki/Traffic-Routes-Info-View

Regarding traffic info view, I thought it always showed where it's difficult to find parking regardless of TMPE menu state?
I will test next time in-game to see if having TMPE toolbar open changes anything.

I'm blind, lol...
Menu state doesn't do anything besides showing actual demand numbers when you turn on building debug info overlay

Just tested, couldn't see any difference between TMPE menu being open or not.
However, I also didn't notice any building colors being applied so maybe something is broken? Or maybe I provided enough parking? I'll see if it changes as my city grows.

Yeah, public transport view seem to be working, but parking demand not at all.
Values of parking demand (debug view) doesn't seem to be changing, even if I disable parking roads in whole/half city...
It looks like they walk by foot instead of trying to use cars xD

From vague memory of a discussion with Victor some years ago, the parking thing is based on Parking AI reaching destination and then having to try find somewhere to park - each subsequent attempt will bump the value for parking issues on the building the cim is trying to get to.

Yup, exactly that's in the code, but I have to figure out how to force cims to fail while they're searching for parking space 😄