Curious why drop netstandard

I am evaluating paths forward for data and serialization layers, and Marten is among my choices, but I am curious, why did you _drop netstandard circa 5.0.0, since 6.0.0 it seems.
As a subscriber, consumer of such packages, I do not generally like to up-lift such components from API surface areas to specific runtimes until the very last possibly moment when it should become more opinionated, such as toward the more user facing layers, UI, web, and so on.
I consider netstandard2.0 to be the bare minimum these days, depending on the API required. Or if slightly more advanced features are desperately needed, then perhaps even netstandard2.1. Although sadly admitedly, that is end of life (a mistake I think, but whatever).

.NET Standard was dropped as Marten already didn't run on .NET Framework in v5.0 so there was no benefit in supporting it (.NET Framework support was dropped with v4). There's no point spamming the codebase with preprocessor directives and an extra build/test matrix to "support" .NET Standard if the library doesn't run on anything apart from modern .NET targets.
In the same way, you as a user should not be writing libraries that target .NET Standard if they're not going to be shared across .NET Framework/Core, it's shooting yourself in the foot for no benefit.
Our support policy is whatever version of (Modern) .NET Microsoft supports and the target framework enforces that. This is also in alignment with Microsoft's own advice: If you don't need to support .NET Framework, you could target .NET Standard 2.1 or .NET 8. We recommend you skip .NET Standard 2.1 and go straight to .NET 8.


Rethink repo structure for easier organization

Question from @jilaga on branding repo structure:

Do I go ahead and upload the Nebula flyers to the prints-and-digital/flyer folder on the repo or create a Nebula folder to house all the assets?
In case you are wondering why I have this dilemma, It is a constant battle with keeping the repo organized and making things accessible.

Not urgent, but good to get input:

from @NPDebs on best practices for repo structure
from @doaamkader as she'll be using existing assets & prob. planning on adding new ones

@jilaga I think create a folder for Nebula is better to house all the assets


Link to official website

Presented in README.md link is outdated:

http://azsky2.html.xdomain.jp/linux/azpainter/index.html

Change it to current!

http://azsky2.html.xdomain.jp/linux/azpainter.html

Thank you. Updated the README.


Filter by param to return single object instead of an array?

I am creating a mock version of a real API my app will interact with.
For my real API, I have the following routes:
[API_ENDPOINT]/api/v1/items/:userId
[API_ENDPOINT]/api/v1//item/:itemId

For the real API, [API_ENDPOINT]/api/v1//item/dbff01e3-f9e0-424e-84d6-999999999999 returns a single object:
{
    "userId": "99999999-9999-9999-9999-999999999999",
    "itemId": "dbff01e3-f9e0-424e-84d6-999999999999",
    "value": "foo"
}

And [API_ENDPOINT]/api/v1//items/99999999-9999-9999-9999-999999999999 returns an array of obejcts:
[{
    "userId": "99999999-9999-9999-9999-999999999999",
    "itemId": "dbff01e3-f9e0-424e-84d6-888888888888",
    "value": "foo"
},
{
    "userId": "99999999-9999-9999-9999-999999999999",
    "itemId": "dbff01e3-f9e0-424e-84d6-777777777777",
    "value": "bar"
},{
    "userId": "99999999-9999-9999-9999-999999999999",
    "itemId": "dbff01e3-f9e0-424e-84d6-666666666666",
    "value": "baz"
}]

These routes get data from the same source, which is what I'm emulating in my mockdata.json. Here's what my data and routes look like:
mockapi.json
{
    "items": [
        {
            "userId": "11111111-2222-3333-4444-555555555555",
            "itemId": "dbff01e3-f9e0-424e-84d6-111111111111",
            "value": "bar"
        },
        {
            "userId": "11111111-2222-3333-4444-555555555555",
            "itemId": "dbff01e3-f9e0-424e-84d6-222222222222",
            "value": "baz"
        },
        {
            "userId": "11111111-2222-3333-4444-666666666666",
            "itemId": "dbff01e3-f9e0-424e-84d6-333333333333",
            "value": "bax"
        },
        {
            "userId": "11111111-2222-3333-4444-777777777777",
            "itemId": "dbff01e3-f9e0-424e-84d6-444444444444",
            "value": "bay"
        }
    ]
}

routes.json
{
    "/api/v1/items/:userId": "/items?userId=:userId",
    "/api/v1/item/:itemId": "/items?itemId=:itemId"
}

My issue is I need /api/v1/item/:itemId to return a single object and the underlying json-server filter returns an array of one single object.
I need the real and mock API output to match for these routes. Any ideas?

By adding a custom route that uses a plural 'items' it's trying to match all objects that have that id.
I was going to suggest just not using that second custom route, but I realized the source of the problem here is probably that you're using itemId instead of id ...

@BrennerSpear - yeah, I'm running into the same issue as #544. I want to be able to filter my "items" set by "userId" and "itemId", in their respective routes. I know I can create two collections of "items" and use an "id" property, but I would like my mock API to be as close to the real API data as possible.

Okay so in your real API, the function on the server side is a custom function that selects only the first item it finds with that itemId and returns it as an object, rather than a collection? if you were to do a sql query of SELECT * FROM items WHERE itemId = 1; I think it would still return a collection, yes?

Actually, in my API, a query of itemId = 1 returns a single object and a query of userId =1 returns a collection.

@fillippeyton Try this and add the query singular=1.
singular.js
'use strict'

module.exports = (req, res, next) => {
  const _send = res.send
  res.send = function (body) {
    if (require('url').parse(req.originalUrl, true).query['singular']) {
      try {
        const json = JSON.parse(body)
        if (Array.isArray(json) && json.length === 1) {
          return _send.call(this, JSON.stringify(json[0]))
        }
      } catch (e) {}
    }
    return _send.call(this, body)
  }
  next()
}

> json-server --watch db.json --middlewares singular.js
> curl http://localhost:3000/users?user_id=100
[{"id":1,"user_id":"100","name":"test"}]
> curl http://localhost:3000/users?user_id=100&singular=1
{"id":1,"user_id":"100","name":"test"}

@narugami that looks very promising! Thanks! I'll try it out!.

Hi, I am facing the same problem.
The middleware indeed works, but only when accessing through direct url, not a custom route.
routes.json
{
    "/v1/users/:id": "/users?user_id=:id&singular=1",
    "/v1/*" : "/$1"
}

 curl "http://localhost:3000/v1/users?user_id=1fc15400-747d-4d92-bccd-0c64098ac35f&singular=1"

{"user_name":"admin","user_id":"1fc15400-747d-4d92-bccd-0c64098ac35f"}

 curl "http://localhost:3000/v1/users/1fc15400-747d-4d92-bccd-0c64098ac35f
[
  {
    "user_name": "admin",
    "user_id": "1fc15400-747d-4d92-bccd-0c64098ac35f"
  }
]

But aren't you now modifying your rest request just to match your mock api?

Or am I missing something about how this works?

I have the same issue -  in my real API, I return arrays of items for GetAll and when I return a single item it is the entity itself, not an array of 1.
http://localhost:3000/items/
http://localhost:3000/items/1   (does not return Item with ItemID of 1.)
http://localhost:3000/items?itemId=1  (returns an array of 1 which is not what I want)


Ignore some links

Ignore links used for navigation, etc.
Much easier to ignore entire files, like those used for indexing.

yes, provide an option to ignore some folders will be better.

@linfz94,
I've added a mechanism to review only notes with certain tags.
This is available in the settings of the latest release. Do check it out.


Changed the cursor color and added dynamic background.

@Akshatchaube01 #163 Solved.

Changed the color of cursor as earlier it was not visible in white background and also I have added the dynamic background to the aboutus, demosection and models section.
Preview:

Can you please pull the latest changes and commit

@Akshatchaube01 Pulled the changes and my branch is up to date with main branch.

The changes made in the last PR are not visible in your work

@Akshatchaube01 But I have successfully raised the PR and also pulled recent all changes.

There was a change just an hour ago, pull that too and then commit...

@Akshatchaube01 Kindly Check.


Allow passing httpc opts

Hey, I needed finer control over request timeouts. Let me know please if that can be merged upstream. Thanks.

@chrismccord ping, should I rebase the PR?

please rebase and I will merge

@chrismccord done

Please doc the new config key and we are all set

:+1: PR updated

<3 <3 <3


[build][cpp] Add cmake option to skip HTTP downloads

Fixes #30385 for C++/cmake.
Add a gRPC_DOWNLOAD_ARCHIVES switch that is on by default but can be disabled for environments building without Internet connectivity.
Passes build tests:
$ python tools/run_tests/run_tests.py -l c++ --build_only
PASSED: tools/run_tests/helper_scripts/build_cxx.sh [time=7116.3sec, retries=0:0SUCCESS: All tests passed

=== run_tests.py DEBUG INFO ===
command: "tools/run_tests/run_tests.py -l c++ --build_only"
===============================
$

And tested in a Docker container with eth0 down to ensure we can now build when -DgRPC_DOWNLOAD_ARCHIVES="OFF" is passed to cmake.

The Mergeable check fails b/c the PR doesn't have labels, but I don't think I have permissions to add labels to PRs for this project.

Can you try to use gRPC_BUILD_TESTS instead of gRPC_DOWNLOAD_ARCHIVES which you added? I assume that those downloaded artifacts are only for testing so we can achieve this without adding a new flag.
And you need to modify its corresponding template file as well. (Please check out the sanity checks)

Can you try to use gRPC_BUILD_TESTS instead of gRPC_DOWNLOAD_ARCHIVES which you added? I assume that those downloaded artifacts are only for testing so we can achieve this without adding a new flag.

I looked into this and as best I can tell these artifacts are completely separate from the tests and are only used in relation to gRPC's xDS features (e.g, in tools/distrib/python/xds_protos/build.py).
I was able to successfully build with gRPC_BUILD_TESTS on and gRPC_DOWNLOAD_ARCHIVES off, so it seems like building the tests is orthogonal to downloading these artifacts, that downloading the artifacts is not a dependency for building the tests.

And you need to modify its corresponding template file as well. (Please check out the sanity checks)

I'll add a commit that updates templates/CMakeLists.txt.template.
I poked around the documentation some but am not sure if I also need to generate artifacts from that template for commit or if the generated artifacts are just ephemeral as part of a build. If I need to generate artifacts for commit, please let me know.
Also not sure if I can/need to run any sanity checks related to the template change locally or just wait for the CI to do the sanity checks. I'll add the commit but if there are any local sanity checks I should run, please let me know.

Any updates on this? The review comment was addressed nearly a month ago and this should be a relatively small change to review.

I was able to successfully build with gRPC_BUILD_TESTS on and gRPC_DOWNLOAD_ARCHIVES off, so it seems like building the tests is orthogonal to downloading these artifacts, that downloading the artifacts is not a dependency for building the tests.

I thought that those artifacts are required for the test then it sounds a bit strange to me. It seems that nothing requires those downloaded achieves? Did you see the cases where it fails to build with gRPC_DOWNLOAD_ARCHIVES off?

I poked around the documentation some but am not sure if I also need to generate artifacts from that template for commit or if the generated artifacts are just ephemeral as part of a build. If I need to generate artifacts for commit, please let me know.

You can just run tools/distrib/sanitize.sh to generate those files from templates.

Any updates on this? The review comment was addressed nearly a month ago and this should be a relatively small change to review.

Sorry about that. I happened to miss this.

I thought that those artifacts are required for the test then it sounds a bit strange to me. It seems that nothing requires those downloaded achieves? Did you see the cases where it fails to build with gRPC_DOWNLOAD_ARCHIVES off?

I don't see a path for the cmake build to fail with gRPC_DOWNLOAD_ARCHIVES off.
The only place I see these downloaded files used is in tools/distrib/python/xds_protos. Maybe if one runs its build_validate_upload.sh script, then build.py won't find these directories. Why it's currently structured that cmake then downloads those instead of the Python scripts for xds_protos, I don't know. Maybe that could be a separate effort by someone more familiar with the xds_protos Python packaging to move this logic out of cmake to the scripts in there.

You can just run tools/distrib/sanitize.sh to generate those files from templates.

This runs successfully. I don't see any generated files added that git is tracking, so I'm assuming nothing needs committed for this.
Let me know if there's anything else that needs done for this PR. It looks like all the tests now pass on the 2nd run.

@veblush @yashykt @eugeneo
Are we waiting on anything else to close this PR? I think it's a relatively simple cmake change but it's been pending for a few months now. Thanks.

@yashykt One more review, please.


Streaming progress of nested graph

As currently implemented, breadboard-server will stream progress of graph traversal. Unfortunately, it will not do that for included or slotted graphs, since they have their own run cycle.
This is likely related to #63 and has to be resolved there.

Something to discuss when we meet next time:
Consider introducing something like closures in the runner, in lieu of nested instances. Semantics stay the same otherwise, but things like streaming progress gets easier, binding inputs to slotted graphs much easier, etc.
I had to do something analogous for integrity (as I have to track flows up and down through parallel includes), so it might simplify that as well.


Captured Images/Videos not shown up in Gallery.

@patoigz The issue seems to be fixed now.

Unfortunately, it doesn't work...

Can you describe your environment.

Are you sure that you tested with latest version.

@jaiselrahman yes... I've tested the last version... But nothing happens. Im using an emulator. Pixel XL API 27, Android 8.1.0

The issue is with the LoaderManager reusing existing Loader. Using LoaderManager.restartLoader() seems to work fine. Can you double check it.


Move react-searchkit integration / theming to invenio-search-ui

We have (or will have at time of writing) merged search integration and theming code here, but that code would be helpful to wider community by being in invenio-search-ui. It would also reduce the amount of custom code in this repository.

A lot of work has already been done here, and I'm closing the ticket. If there's pending work to do, it should be opened as specific issues.


ART-1210: Auto run multiarch pre-release job for 4.3 and 4.4

Will run pre-prelease job in parallel for multiple versions and arches.
Test run with multiarch support:
https://saml.buildvm.openshift.eng.bos.redhat.com:8888/job/hack/job/yuxzhu/job/poll-payload/31

Updated for multiarch.

aside from the conflict, this will need some rework for the separate arch-specific release-controllers.

OK.

Finished. Please do another review. Thanks.
@thiagoalessio @sosiouxme

@vfreex LGTM except I'm mildly worried about these complaints: https://saml.buildvm.openshift.eng.bos.redhat.com:8888/job/hack/job/yuxzhu/job/poll-payload/57/console
Invoking action for 4.3.0-0.nightly
expected to call org.kohsuke.groovy.sandbox.impl.SandboxedMethodClosure.call but wound up catching WorkflowScript.startPreReleaseJob; see: https://jenkins.io/redirect/pipeline-cps-method-mismatches/
[Pipeline] echo (hide)
Invoking action for 4.4.0-0.nightly
expected to call org.kohsuke.groovy.sandbox.impl.SandboxedMethodClosure.call but wound up catching WorkflowScript.startPreReleaseJob; see: https://jenkins.io/redirect/pipeline-cps-method-mismatches/

Not digging too deep but I think this is what happens when you use iterator methods; I'm not sure exactly what effect it has, other than apparently raising an error that makes the build come out UNSTABLE.

I noticed this warning during development. I believe it is a false positive warning when calling a method via a pointer, similar to https://issues.jenkins-ci.org/browse/JENKINS-59492. Those functions are called correctly without any issues.
The UNSTABLE result is not set by the warning. https://github.com/openshift/aos-cd-jobs/pull/2053/files#diff-b89f72b7074288d1245c7ece87537d33R99 and https://github.com/openshift/aos-cd-jobs/pull/2053/files#diff-b89f72b7074288d1245c7ece87537d33R114 set the build result to UNSTABLE when it fails to call the release controller API or downstream pre-release job fails. Currently many non-x86_64 API endpoints return 404, such as https://openshift-release-ppc64le.svc.ci.openshift.org/api/v1/releasestream/4.3.0-0.nightly-ppc64le/latest because no accepted payload exists.

Thanks for the explanation. Hm, I'd like it to be OK to have missing endpoints (just ignore them) and report success but it doesn't seem like a reason to hold this up.


Profiling .NET MAUI Android Apps

Android framework version
net8.0-android
Affected platform version
Android 11
Description

Steps to Reproduce
Created the Sample Maui App.
Trying to implement the profiling in android app.
Following this article "https://github.com/dotnet/android/blob/main/Documentation/guides/tracing.md?plain=1"
After executing the "dotnet-trace collect -p 38604 --format speedscope" command facing the below issue I had also attached the screenshot of the same
.
System.IO.EndOfStreamException: Unable to read beyond the end of the stream.
at System.IO.BinaryReader.InternalRead(Int32 numBytes)
Thanks for the help !!
Did you find any workaround?
No response
Relevant log output
No response

I have a YouTube video on dotnet-trace, etc. I would guess there is probably a step you've missed:

https://youtu.be/hIYwz5dsZ6M

Did the YouTube video help?


Folder Traits are Not Supported for Use as a Conversion Trait

Issue in help/using/features/audience-lab/audience-lab-manage-test-groups.md
In Item #5 on the page, it discusses Conversion Traits. A bullet point should be added to indicate that using a Folder Trait as a Conversion Trait is not supported. It will result in 0 Aggregate and Trend reporting being displayed within the test.
Jira References:
AAM-38962
AAM-38961
AAM-38684

Thanks for tracking this down and creating the ticket, @JefferyTaylor0204 !
Please see the updated page https://docs.adobe.com/content/help/en/audience-manager/user-guide/features/audience-lab/audience-lab-manage-test-groups.html.


Serialize to different data format?

I am working with an existing CouchDB dataset where records are structured like this:
{
  "_id": "...",
  "_rev": "...",
  "my_attr": "value",
  "my_other_attr": "value"
}

Ember pouch seems to want my data in this format:
{
  "_id": "...",
  "_rev": "...",
  "data": {
    "my_attr": "value",
    "my_other_attr": "value"
  }
}

Whilst I like this in principle it's not practical for me to migrate my CouchDB data as there are other clients which depend on the existing format. My question is, do you know of a good way which I can serialize and deserialize data between the two formats?
I'm wondering if there's somewhere I can get inbetween the replication from PouchDB to CouchDB and transform the data? Any help would be greatly appreciated!

As is always the way, I've figured it out but only after asking the question. Thanks for writing transform-pouch @nolanlawson! :heart:

Haha, glad it worked out! :)


Illegal Instruction when using dotnet commands on 2.1.300-rc1 SDK with Ubuntu 18.04 ARM64 on WSL

Illegal Instruction when using dotnet commands on 2.1.300-rc1 SDK with Ubuntu 18.04 ARM64 on WSL
Binary Information:
Downloaded Ubuntu 18.04 from here https://www.microsoft.com/store/productId/9N9TNGVNDL3Q
Downloaded Linux ARM (glibc) 64-bit from https://github.com/dotnet/core/blob/master/release-notes/download-archives/2.1.0-rc1-download.md
System Information:
msinfo32 output:

OS Name	Microsoft Windows 10 Pro Insider Preview
Version	10.0.17666 Build 17666
Other OS Description 	Not Available
OS Manufacturer	Microsoft Corporation
System Manufacturer	ASUSTeK COMPUTER INC.
System Model	NovaGo TP370QL
System Type	ARM64-based PC
System SKU	6
Processor	Snapdragon(TM) 835, 2208 Mhz, 8 Core(s), 8 Logical Processor(s)
BIOS Version/Date	Qualcomm Inc. TP370QL.253, 3/30/2018
SMBIOS Version	3.0
Embedded Controller Version	255.255
BIOS Mode	UEFI
BaseBoard Manufacturer	ASUSTeK COMPUTER INC.
BaseBoard Product	NovaGo TP370QL
BaseBoard Version	1.0

lsb_release output from Ubuntu 18.04:

Architecture:        aarch64
Byte Order:          Little Endian
CPU(s):              8
On-line CPU(s) list: 0-7
Thread(s) per core:  1
Core(s) per socket:  8
Socket(s):           1
Vendor ID:           ARM
Model:               0
Model name:          Cortex-A57
Stepping:            r1p0
CPU max MHz:         2208.0000
BogoMIPS:            1900
Hypervisor vendor:   Windows Subsystem for Linux
Virtualization type: container
Flags:               fp asimd evtstrm aes pmull sha1 sha2 crc32

The following commands throw this error:
dotnet new
dotnet --info
dotnet --version
Output:
snickler@WOATARGET:~/dotnet-2.1-rc1$ ./dotnet new
Illegal instruction (core dumped)
snickler@WOATARGET:~/dotnet-2.1-rc1$ ./dotnet --info
Illegal instruction (core dumped)
snickler@WOATARGET:~/dotnet-2.1-rc1$ ./dotnet --version
Illegal instruction (core dumped)

Hey @BenHillis @terrajobst - any thoughts on what's causing these Illegal Instruction errors?

I'm not sure, I know @Brian-Perkins has been looking at some arm64 issues recently.

One thing I didn't add - the --list-runtimes and --list-sdks switches are the only ones that don't return Illegal Instruction
snickler@WOATARGET:~/dotnet-2.1-rc1$ ./dotnet --list-sdks
2.1.300-rc1-008673 [/home/snickler/dotnet-2.1-rc1/sdk]
snickler@WOATARGET:~/dotnet-2.1-rc1$ ./dotnet --list-runtimes
Microsoft.NETCore.App 2.1.0-rc1 [/home/snickler/dotnet-2.1-rc1/shared/Microsoft.NETCore.App]

I attempted to install from the instructions linked in the OP and it doesn't appear ARM64 versions are present. If someone can provide a set of commands I can run from a clean Ubuntu 18 install in order to repro this, it would be very helpful.

Thank you. I thought I had tried that link, but on attempt number two it looks good. I ended up here which wasn't as useful.

@Brian-Perkins - Have you been able to reproduce?

I was able to reproduce. It is complaining about a cache flush instruction -- I am looking into it.

Great, Thanks!

From: Brian-Perkins<EMAIL_ADDRESS>Sent: Thursday, May 17, 2018 11:53:36 AM
To: dotnet/core
Cc: Jeremy Sinclair; Author
Subject: Re: [dotnet/core] Illegal Instruction when using dotnet commands on 2.1.300-rc1 SDK with Ubuntu 18.04 ARM64 on WSL (#1561)
I was able to reproduce. It is complaining about a cache flush instruction -- I am looking into it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fcore%2Fissues%2F1561%23issuecomment-389915978&data=02|01||1fa56466078f4e2b073c08d5bc0e5a16|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636621692175750660&sdata=ISXqGhoWhw8P%2FcyWMKIkjguygoWpEX%2FudmcSIkH2saA%3D&reserved=0, or mute the threadhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD1IpWE4VD6VLs934lRIqrVT2pZ9cggkks5tzZ0AgaJpZM4T-yJf&data=02|01||1fa56466078f4e2b073c08d5bc0e5a16|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636621692175750660&sdata=wb69Ng4%2BfWgOveUgqwjRXwAcWlx%2FF4RyixGKu7mEzdA%3D&reserved=0.

We are working on a WSL fix for this. I will add another update when it makes it out in an Insiders Build.

This is great to hear. Thanks so much @bitcrazed @Brian-Perkins

This should be resolved in Insider Build 17692 - WSL specific release notes here if you are on the Insider track. There is no fix yet for the RS4 release -- I will update when that changes.

I know what I'm trying out when I get home tonight. Thanks for digging into this. I'll let you know if I happen to run into anything.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Brian-Perkins<EMAIL_ADDRESS>Sent: Thursday, June 14, 2018 5:23:33 PM
To: dotnet/core
Cc: Jeremy Sinclair; Author
Subject: Re: [dotnet/core] Illegal Instruction when using dotnet commands on 2.1.300-rc1 SDK with Ubuntu 18.04 ARM64 on WSL (#1561)
This should be resolved in Insider Build 17692https://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fblogs.windows.com%2Fwindowsexperience%2F2018%2F06%2F14%2Fannouncing-windows-10-insider-preview-build-17692&data=02|01||48399c27f6ae4bd1bce808d5d23d159d|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636646082146039126&sdata=lzXWkVlVK5erxwEJZJT3CaALgsNXmzjCuQKeYMVr7MA%3D&reserved=0 - WSL specific release notes herehttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FWSL%2Freleases%2Ftag%2F17692&data=02|01||48399c27f6ae4bd1bce808d5d23d159d|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636646082146039126&sdata=6ilnSbsM7WOoUrrsQn8d37OuAb1P7cfcAhGnzoeYymI%3D&reserved=0 if you are on the Insider track. There is no fix yet for the RS4 release -- I will update when that changes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fcore%2Fissues%2F1561%23issuecomment-397442559&data=02|01||48399c27f6ae4bd1bce808d5d23d159d|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636646082146039126&sdata=85osk8JCx%2FkieAwx%2FW7fH0smu%2BBIDVLvddddTYW21x8%3D&reserved=0, or mute the threadhttps://nam03.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD1IpUxeNWTk_C2B6DdlyY4itwL4RBbsks5t8tRVgaJpZM4T-yJf&data=02|01||48399c27f6ae4bd1bce808d5d23d159d|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636646082146039126&sdata=VOGOggi0mGlTpW3qC4WlMEjxJheP8bLCV4r0M%2BHcoec%3D&reserved=0.

I will mark this issue as fixed, but I'm opening another issue that I've discovered while trying to use it.


Add Tetum language

Reproduction

Describe the bug
Adds initial translations for the "Tetum" language.
Tetum is one of the languages spoken in East Timor (Timor Leste) and will be the primary language for our site "Partisipa" https://partisipa-staging.catalpa.build
tet.ts.txt
Environment

@sashamilenkovic Is there any further action required on our part for this?

@joshbrooks Sorry, I haven't had a chance to look into this yet. Your txt file looks good, I'll add this to my queue.


[BUG] Model * GPU size memory required for tensor parallel inference

Describe the bug
If we set num_gpus to N, deepspeed will start N thread to load model, and each thread will take the full size of model memory. For example, if the model takes 80Gb memory, then if we set num_gpus to 8, the total memory needed is 80*8 = 640Gb. This is not a accessible volume even for most ML servers. Can I check if there's any workaround?
To Reproduce
deepspeed --num_gpus 8 DeepSpeedExamples/inference/huggingface/text-generation/inference-test.py --name decapoda-research/llama-65b-hf --batch_size 4 --test_performance --ds_inference

Expected behavior
Each thread shouldn't consume memory of full model size.

encounter the same issue, expecting the workaround

Hi @zhch12121, what version of DeepSpeed are you using?

encounter the same issue, I think this maybe work, but i don't konw which api can do:
loading the model on one thread, predshard the model and save.

encounter the same issue, expecting the workaround

Hi @liding1992 and all, you should be able to run using --use_meta_tensor flag and using checkpoints. I tested this on my side. Let know if that works for you.

Hi @liding1992 and all, you should be able to run using --use_meta_tensor flag and using checkpoints. I tested this on my side. Let know if that works for you.

I tested LLAMA-7B and LLAMA-13B and DeepSpeed worked.
here is my gpu info

here is my ds_report info

here is my package info
centos 7.9
python 3.10.9
cuda 11.7
transformers 4.28.1
torch 1.13.1+cu117
here is my code
import sys
import time
import os
# New deepspeed path
sys.path.insert(0, '/Code/DeepSpeed')
import torch
import deepspeed
from transformers import LlamaForCausalLM, LlamaTokenizer
import argparse
from utils import Performance
from deepspeed.runtime.utils import see_memory_usage
from arguments import parser

args = parser.parse_args()
# here
deepspeed.init_distributed()

local_rank = int(os.environ.get('LOCAL_RANK', 0))
world_size = int(os.environ.get('WORLD_SIZE', 1))

print(f'local_rank: {local_rank}, world_size: {world_size}')

if args.local_rank == 0:
    see_memory_usage("before init", True)

t0 = time.time()
tokenizer = LlamaTokenizer.from_pretrained(args.model)
model = LlamaForCausalLM.from_pretrained(args.model)
if args.local_rank == 0:
    print(f"initialization time: {(time.time()-t0) * 1000}ms")
    see_memory_usage("after init", True)

model = deepspeed.init_inference(
    model,
    mp_size=world_size,
    dtype=torch.half,
    replace_with_kernel_inject=True
)
if args.local_rank == 0:
    see_memory_usage("after init_inference", True)

batch = tokenizer(
    #"The primary use of LLaMA is research on large language models, including",
    "DeepSpeed is a machine learning framework",
    return_tensors="pt",
    add_special_tokens=False
)
# here
batch = {k: v.cuda(local_rank) for k, v in batch.items()}
generated = model.generate(batch["input_ids"], max_length=args.max_new_tokens)

if args.local_rank == 0:
    print(f'in:{batch["input_ids"]}\nout:{tokenizer.decode(generated[0])}')

1*A10
after init_inference
MA 12.81 GB         Max_MA 12.81 GB         CA 13.45 GB         Max_CA 13 GB
CPU Virtual Memory:  used = 8.83 GB, percent = 2.4%

2*A10
after init_inference
MA 6.65 GB         Max_MA 6.65 GB         CA 7.71 GB         Max_CA 8 GB
CPU Virtual Memory:  used = 11.04 GB, percent = 3.0%

Thanks @liding1992 for confirming. Closing this issue for now.


[Feature Request] Pressing 'C' or 'CTRL + L' to clear the terminal after fvm flutter run

Before creating a feature request make sure the suggestion fit within our principles
Is your feature request related to a problem? Please describe.
After running fvm flutter clean, there is no easy way to clear the terminal using a keyboard shortcut.
Describe the solution you'd like
Either accepting pressing 'c' key likeflutter run command does or respecting 'CTRL + L' keyboard shortcut to clear the terminal.
Describe alternatives you've considered
I cannot find any alternative solutions right now. Maybe hitting enter key serveral times to shift everything up is an option.

Here are two videos running flutter's example app. I am running this on M1 macbook air.

As you can see, the fvm terminal just adds a few lines when 'c' is pressed while flutter terminal clears the entire terminal when 'c' is pressed.
https://user-images.githubusercontent.com/30948010/182015655-354166e5-9052-4c11-afff-3958217d229c.mov
https://user-images.githubusercontent.com/30948010/182015659-4f6216c0-baec-49ac-ac72-0b63f2d7be89.mov

@leoafarias  can you please check the above videos ? Are you still unable to replicate this on a mac?


BUG[Improving UI of Rate US PAGE] #298

Title and Issue number
BUG[Improving UI of Rate US PAGE] #298
Title :BUG[Improving UI of Rate US PAGE]
Issue No. : #298
Code Stack : HTML, CSS
Close #298
Video/Screenshots (mandatory)

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have gone through the  contributing.md file before contributing

Are you contributing under any Open-source programme?
Yes, JWOC'24 phase 2

@apu52 please could you review PR


Changes to support retrieval of operations from translog based on specified range

Description
Changes to support retrieval of operations from translog based on specified range
Issues Resolved
https://github.com/opensearch-project/OpenSearch/issues/1100
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

start gradle check

start gradle run

start gradle check


Improve worker.js reliability

While it doesn't happen right now in this particular example, lastPtr can be potentially overridden several times before the module is fully initialised.
Rather than having a boolean and a storage for one last argument, await a promise returned from wasm_bindgen itself in the new onmessage handler before executing actual command.
This way all the potential tasks will queue up naturally, wait for the initialisation, and then execute in a correct order.

Nice!

@alexcrichton Thanks for merging!
By the way, I found this published crate that pretty much extracted code from this example in a reusable fashion: https://crates.io/crates/web_worker.
I believe it's slightly out of sync and misses e.g. this worker.js, but otherwise it seems like quite a good idea to have a ready-to-use adapter for other people (like me) playing with multithreading in Rust + Wasm on their own.
I wonder if you'd want to coordinate with author and maybe help to make sure that their code is up-to-date, or make your own crate, and then switch this example to use it as a dependency?

Sure yeah, refactorings here seem reasonable to me! When I made this example afaik it was the only thing that could work, so it was the minimal thing I got working quickly. If there's more official ways to do it nowadays seems good!

@alexcrichton Oh, I don't think there are more official ways - as I said, that crate is basically an extracted version of your code here.
What I'm asking is if you'd want to make that crate the official way and sync your code to it, or publish your own crate for others to use.

I think we probably don't want to farm this out just yet, I don't have the time to maintain a separate crate and the threads spec is volatile enough I think we want to retain the ability to make changes here if necessary.

Fair enough. I think I'll publish some small wrapper myself then, but for now am hitting some Chromium bugs that need to be fixed before it's released.


chore: Migrate dataproc synth.py to bazel

The changes in grpc stubs are caused by the gRPC upgrade from 1.10 (more than a year old) to 1.27 (same version which is used as runtime dependency)

@chingor13 PTAL


bottomNavigationBar doesn't work well.

      bottomNavigationBar: Container(
        decoration: BoxDecoration(
          gradient: LinearGradient(
            // Where the linear gradient begins and ends
            begin: Alignment(0.7, 1.0),
            end: Alignment(-0.7, -1.0),
            // Add one stop for each color. Stops should increase from 0 to 1
            // stops: [0.6, 0.7],
            colors: [
              // Colors are easy thanks to Flutter's Colors class.
              Color.fromRGBO(245, 243, 106, 1.0),
              Color.fromRGBO(177, 238, 111, 1.0),
            ],
          ),
        ),
        child:  FlatButton(
          child: Text(
            "TEST",
            style: TextStyle(color:Colors.white, fontSize: 18, fontWeight: FontWeight.bold),
          ),
          onPressed: () {
            Navigator.pop(context);
            _tabBloc.changeBottomTab(2);
          },
        )
      )

I've used bottomNavigationBar in Scaffold.
However, like the picture, iPhoneX's bottom bar's overlapped my bottomNavigationBar.
I want to it never overlap mine.
Can I do automatically?

the code is
bottomNavigationBar: BottomNavigationBar(...)

And, other usage do work well...`

Please consider asking support questions in one of the other channels listed at http://flutter.io/support .

You can add the Scaffold inside a SafeArea

You can add the Scaffold inside a SafeArea

I did add SafeArea()... but haha :0

Thanks @Kartik1607 !

@inthewalter Perhaps you can add Padding bottom to your Container with the value of MediaQuery.of(context).padding.bottom


Show an error message when a Librarian wants to delete a document which has dependancies

Is your feature request related to a problem? Please describe.
When we delete a document, there is no alert if there is an e-item, an acquisition order or a borrowing request linked to it. We should get an alert, as there should be an alert in case of ANY dependencies with the document: items, e-items, acquisitions, borrowing request, relation, series...)
Describe the solution you'd like
Show a message for any dependancy linked to a document., when we try to delete it.

note for future discussions: series will not be added to the notification, as they are considered as a parent record for the document, therefore they are not blocking for the document deletion (also series should not be blocking the deletion because they can have other document attached)


Simple plugin mechanism

Allow compiling application code with the OL3-Cesium library by simply
putting plugin javascript code in the src/plugins directory.
User automatically gets error checking, styling and documentation.
This extension mechanism is intended for people not familiar with
Closure. More advanced use-cases should be handled with a custom build.

@elemoine, do you have other comments?

Rebased on master.

Anymore review?

Looks good to me. Thanks.

Thank you for the review @elemoine.


:sparkles: add last seen user in top bar

Closes https://github.com/Zenika/skillZ/issues/319
Actually, the last_seen field in Auth0 is not available.
What I'm doing is the following :

When the user is connected, I check every 60 seconds the difference in minute between last_seen and current date.
If difference > 10, update the last_seen value for the current user

Very good code ! But i wonder why such a big complexity just for a last seen. In my opinion last seen = last login, shouldn't it be only updatable when the user log in ? Et once you have it when he logs, just on the front you compare this date to the current date.
Maybe i don't have all the context, but still great code with useContext :)

Hello !
In the issue, you spoke about the attribute last_login. And in this PR, you speak about last_seen 🤔  The last_login is not usable ?
When the user is connected, I check every 60 seconds the difference in minute between last_seen and current date. It's not a little hard to do that ? If a script run frequently, could we have less execution ? In this case I think we can increase delay

Hello ! In the issue, you spoke about the attribute last_login. And in this PR, you speak about last_seen 🤔 The last_login is not usable ?
When the user is connected, I check every 60 seconds the difference in minute between last_seen and current date. It's not a little hard to do that ? If a script run frequently, could we have less execution ? In this case I think we can increase delay

I think it should be better if we

To test :
npm hasura metadata apply && npm generate:local

Render :

It can be interesting to add your rules in doc/rg.md
And personnally I like this feature :+1:
With @StephanieHP, we imagined a message with the date of last connection which was displayed only if it had been a while since we had connected. We didn't define a time limit. Maybe we should wait for her return on this POC to reconfirm a larger delay with her?

ℹ My last commit (https://github.com/Zenika/skillZ/pull/328/commits/f6d014051a07ec384c6255c0081bf1169fd9db0e) changes the way to save user's last login in database.
It is now working like the following :

At each login, two properties are updated in database :

current_login = Date.now()
last_login = previous value of current_login

Then, only last_login is displayed in top bar

Hello @MailyLehoux and @benjaminbrgs :)
Can you maybe use the same Typo that is used up for the name of the consultant "Mai-Ly LEHOUX" ? I think it's going to be better.

Hello @MailyLehoux and @benjaminbrgs :) Can you maybe use the same Typo that is used up for the name of the consultant "Mai-Ly LEHOUX" ? I think it's going to be better.

Hi @StephanieHP :)
Actually, it is the same typo as the consultant's name but in a different size
Name : 16px
Agency : 14px
Last login : 12px

@MailyLehoux It seems to work on staging, I let you test as well 👍🏼

nice it works indeed :+1:


Super slow performance when I enable Linter ( with cow power ) latest version

Linter is at 1.11.23, this issue has been plaguing my works for months, each return on a 500 lines HTML takes 4 seconds to respond, all is good as soon has I disable the Linter, is there a config I'm missing ?? It's now in default mode all the way through , it 's the most downloaded package here !

This is something that you'll have to report to the linter package. The Atom FAQ has information on how to contact the maintainers of any Atom community package or theme.


Server Definitions

Related to #9 .

Server runtimes should be declared outside of the protocol module.
A service runtime implementation is based on synthetic handlers generated by the protocol module as described in #9. Service runtimes may be implemented like in the example below:

implicit val GreeterHandler: Greeter.Handler[Future] = new GreeterHandler[Future] {
   def sayHello(msg: MessageRequest): Future[MessageReply] = ???
   def lotsOfReplies(msg: MessageRequest, observer: StreamObserver[MessageReply]): Future[Unit] = ???
   def lotsOfGreetings(msg: StreamObserver[MessageReply]): Future[StreamObserver[MessageRequest]] = ???
   def bidiHello(msg: StreamObserver[MessageReply]): Future[StreamObserver[MessageRequest]] = ???
}

These implicit handlers will be considered in the start and other combinators in the @server definition that will require the runtime information needed to start the server.

For a given list of protocols such as @service trait Greeter { ... }, @service trait Runner ... a server runtime impl may be generated aggregating all the services which would run in such a server.

@server[Greeter, Runner, ...] trait Server

Annotating a trait as a @serverautomatically creates an algebra that includes common operations such as start(), stop(), ... and lifecycle operations related to the server as well as exposing all the aggregated services. Server handlers are auto-implemented although they follow the same freestyle principles of letting the user override their operation via implicits. Server handlers require implicit evidence of a ServerConfig which provides runtime information related to ports/transport and other config values needed to expose the server over a network.

Fixed partially by https://github.com/frees-io/freestyle-rpc/issues/13
It'll be completed by #29


Gradio App is not working

Hi @abhi1nandy2 ,
unfortunately your gradio web-app is not working anymore. Could you provide a working sample of your model?

Please refer to https://github.com/abhi1nandy2/EMNLP-2021-Findings/blob/main/gradio/demo.py


releaserc updated

fix(IBM Cloud SCC): first release

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Increase size of max header length to accomodate token authentication

Some tokens are larger than the current max header value length allows for.

No, aside from increasing the size of the http_t structure 8-fold, this change would break binary compatibility.
What header do you need larger for authentication? We already store the Authorization header in an allocated string, and for CUPS 2.3.x and larger for WWW-Authenticate as well.

We need to use HTTP_FIELD_AUTHENTICATION. Should we be using HTTP_FIELD_WWW_AUTHENTICATE instead?

HTTP_FIELD_AUTHENTICATION (Authentication: ...) is what you send from the client to the server. That one has been allocated for a very long time (since Kerberos was added) so the 256 byte value limit does not apply.
HTTP_FIELD_WWW_AUTHENTICATE (WWW-Authenticate: ...) is what you send from the server to the client in a challenge (401) response. That field was recently just changed to be allocated for the forthcoming CUPS 2.3 release.
What sort of problems are you having? Are the Authentication: headers sent via libcups getting truncated or ???

There is some confusion on my team about the changes we needed to make. I'm going to close this.
Sorry about that! Thanks for your help!


Feature: API to update pre/post ci/cd using app id and env id

🔖 Feature description
API to update pre/post ci/cd using app id and env id
🎤 Pitch
Right now we have to get workflow id, ci pipeline id manually to update any pre/post ci using API but that doesn't work if we want to perform the operations in loop
✌ Solution
Having an api that does it automatically using app id and env id can be helpful
🔄 Alternative
No response
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

AB#4048

Please include branch change option also in this API


⚠ Bofrost.es has degraded performance

In c4b11cb, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1501 ms

Resolved: Bofrost.es performance has improved in 5e37ce2 after 9 minutes.


Collapsed navbar margins

A collapsed navbar does not have the correct margins.  A simple example is the display of login/join buttons which are not centered.  This may be due to the .navbar .btn-toolbar margin-bottom of -13px.  If that is set to 0, the buttons display correctly when collapsed but not when uncollapsed.
Likely another override is needed for the collapsed toolbar.
Also the -13px and similar should be computed in the less file from the navbar height variable.

This issue should be moved to bedrock-angular-navbar along with any relevant CSS so this is a modularized problem+solution.


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/opstest2/docs-archive-test-source.git

Docs Build status updates of commit ca0e80e:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Published posts not display nor in user feed nor in global feed.

I published a post but I can't see it anywhere. Neither on my feed nor if the global feed.
But when using the direct link I am able to see the post, even without being logged in, so it does appear to be published.
I've also noticed that my draft posts are also nowhere to be seen.
The link to the post: https://ldjam.com/events/ludum-dare/55/$390641/the-beginning-of-somethings-1

Should be there now. New users need to be manually approved.

I have the same issue https://ldjam.com/events/ludum-dare/55/demons-on-demand/$394081/


32 bit flag field

I have a 32 bit flag field (4 bytes), however if I read 4 bytes and begin parsing the flags it seems the bit counters are based on 8 bits?
However, if I read 16 bits I do get a ".15-0" notation.
Also, I think I'm getting confused with the endianness, when reading the 16 bits it seems to have taken the right 16, while individual bits start from the left??
What would be the recommended approach for interpreting a 4-byte (32-bit) flag field?
Or, is there any first-class support for enumerations planned?
    read(4);
    addRow('Flags', getHexValue(), 'Flags');

    addDetails(() => {
        readBits(16);
        addRow('reserved', getHexValue());
        readBits(1);
        addRow('flag-15', getHexValue());
        readBits(1);
        addRow('flag-14', getHexValue());
        readBits(1);
        addRow('flag-13', getHexValue());
        readBits(1);
        addRow('flag-12', getHexValue());
        readBits(1);
        addRow('flag-11', getHexValue());
        readBits(1);
        addRow('flag-10', getHexValue());
        readBits(1);
        addRow('flag-9', getHexValue());
        readBits(1);
        addRow('flag-8', getHexValue());
        readBits(1);
        addRow('flag-7', getHexValue());
        readBits(1);
        addRow('flag-6', getHexValue());
        readBits(1);
        addRow('flag-5', getHexValue());
        readBits(1);
        addRow('flag-4', getHexValue());
        readBits(1);
        addRow('flag-3', getHexValue());
        readBits(1);
        addRow('flag-2', getHexValue());
        readBits(1);
        addRow('flag-1', getHexValue());
        readBits(1);
        addRow('flag-0', getHexValue());

-John

If a file is read the bytes are read from the beginning of the file to the end.
If you read a few bytes e.g. read(4) the bytes are read and afterwards interpreted according to the endianness.
You can as well read bits. This also starts at beginning of the file or wherever the read pointer is at the moment. For reading bits there is no endianness. I.e. it always starts at bit 0, goes to bit 7 and then starts with bit 0 of the next byte.
Easiest thing in your case would be to change the text and start with "flag-0".
Alternatively you could do another readBits(16) (instead of the readBits(1)) and use getBitsValue().
This will return a string like "00010010_00001111".
You could use that directly and explain the flags/bit positions.
Or you could probably assign getBitsValue() to a string (e.g. myFlags) and just add the string at a certain index to the addRow, e.g. addRow("flag-15", myFlags[16]). 16 because of the "_" in the middle.

Thank you maziac.
Taking in what you've said I'll make it work as-is, but a couple things remain unintuitive to me.

If I ask it to read(4) then parse the bits, I would think of the LSB as the right-most bit i.e. bit-0, not 32. However if I was reading a bit at a time (from the beginning) then there is no real concept of a bit number. I have to interpret the read(4) as doing nothing for me except making an expandable grouping in the UI.
Given my example above, the bit numbers recycle after 8 bits. This seems arbitrary at a byte boundary; why not a word or a dword or qword, etc. Especially when I have 4 bytes in the buffer.

I guess I was hoping to take advantage of the bit counting feature, I just won't be able to do that.

The read(4) is not used only to group bits.
You could e.g. group 2 dwords and do a
read(2)
read(2)

in the details.
If someone mixes this approach e.g.
read(2)
readBit(1)
readBit(3)
readBit(2)
readBit(2)
read(1)

what should be returned for the bits?
Or in other words: when reading the first bit it is not clear yet what is read afterwards.
What size should be assumed?
I'm open to any suggestions.

Yes, I understand. If there are only bits in the details.
But what about my example.

That's what I was mentioning could be real simple, if a read() is executed just reset the counter back to zero.
So the read(n) will establish the bit counter from 0=>(n*8-1).
If another read(n) is performed at any time it advances to the next byte boundary and resets the counter.
Now I'm still developing my parser so I'm not completely versed in the behavior enough to know if this meshes with the current design, or breaks something.

I elaborate the example/your suggestion and use read(size) to illustrate the behavior:
    read(5);
    addRow('Flags', getHexValue(), 'Flags');

    addDetails(() => {
        readBits(1);
        addRow('.1a', getHexValue());
        readBits(1);
        addRow('.1b', getHexValue());
        ...
        readBits(1);
        addRow('.1p', getHexValue());
        read(1);
        readBits(1);
        addRow('.2a', getHexValue());
        ...
        readBits(1);
        addRow('.2g', getHexValue());
        readBits(1);
        addRow('.3a', getHexValue());
    }

would result in (name, size, value):
.1a .0  0
.1b .1  0
...
.1p .15  0
.2a .0 0
.2b .1 0
...
.2g .7 0
.3a .8 0
Assuming all values are 0.
This works somehow, but it does not make much sense to me.
Why, in this case, the size for the bits should be 40 at all.
There is no relationship between the read bytes (read(5)) and the bit-size for the bits.

To comment I'm a little weak on the relationship between read() and readBits().
If I read(1) then readBits(1) will that bit be the first bit of stream, or will it be the 9th bit? In other words, does readBit fetch more data or does it interpret the existing buffer from the last read()?


We need to either fix the "invite a new user" experience or get rid of it.

I've done some involuntary hallway testing with friends & family, trying to get them on Matrix (specifically on matrix.org). Some of them are technologically proficient, others newbies.
This included:

the "share this room" link;
the "invite-by-email" feature;
both the web client and the Android client.

From 6 people, only two managed to get invites to work, and these are not the proficient users. The others left convinced that Element is broken, "can't replace WhatsApp", etc.
We really need to fix this experience.

I think it might be more of a UX issue (hence normally filed under element-web in the first instance, even though it spans multiple clients) than a spec issue, to be honest. It's not obvious from the report what would be changed at the spec (or server) level.

I feel this is largely a duplicate of https://github.com/matrix-org/matrix-doc/issues/2971


Tiled rendering

@guidow, you ready for a review of this?

Yes, this is ready for review now.

Seems generally ok and much simpler than I guess I was expecting.  I thought it might be beneficial to offer two modes of tiling, one where the jobtype on the agent determines how to tile the image and another where the user can.  But...this setup will probably work fine and it's not like we couldn't include something more customizable later.

I've rebase this branch to master to make it mergable again.

Any more objections to mergeing this?

Nope none, sorry for the delay got caught up on another project.  Merge when ready.


Cannot modify an existing file

Preflight Checklist

[x ] I agree to follow the Code of Conduct that this project adheres to.
[x ] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Describe the bug
Error saving file
An operation that depends on state cached in an interface object was made but the state had changed since it was read from disk.
The only thing I can do is to save the file as new file with different name
To Reproduce
It started in my computer lately and now I see it consistent just by open saved file and modify it, than try save it
Expected behavior
When modify existing file it should save the changes on the same file
draw.io version (In the Help->About menu of the draw.io editor):

draw.io version [e.g. 21.3.5]

Desktop (please complete the following information):

OS: windows
Browser chrome
Version 113.0.5672.93

Additional context
Add any other context about the problem here.

Is this in a synchronized local folder (Google Drive, iCloud etc)?

What is the path of the folder you're writing to?

C:\Users\ashiebe\Documents\MyFiles\TaProj

Unfortunately, without being able to recreate, there isn't anything we can do.


Drawing lines with vertexes always closes the shape in PIXI 3

In PIXI 2, I could draw a line with a vertex like so:
var renderer = PIXI.autoDetectRenderer(800, 600, { antialias: true });
document.body.appendChild(renderer.view);
var stage = new PIXI.Stage();
var graphics = new PIXI.Graphics();
stage.addChild(graphics);

// draw line
graphics.lineStyle(5, 0xffffff);
graphics.moveTo(100, 200);
graphics.lineTo(200, 200);
graphics.lineTo(200, 100);
graphics.endFill();
renderer.render(stage);

Result using PIXI 2.2.9 works as expected.

http://jsfiddle.net/vzgoawzy/

When I do the same thing in PIXI 3.0.7, an extra unwanted line is drawn back to the starting point. This occurs with both  Canvas and WebGL renderers:
var renderer = PIXI.autoDetectRenderer(800, 600, { antialias: true });
document.body.appendChild(renderer.view);
var stage = new PIXI.Container();
var graphics = new PIXI.Graphics();
stage.addChild(graphics);

// draw line
graphics.lineStyle(5, 0xffffff);
graphics.moveTo(100, 200);
graphics.lineTo(200, 200);
graphics.lineTo(200, 100);
graphics.endFill();
renderer.render(stage);

Result has unexpected behaviour. An extra unwanted line is drawn back to the start.

http://jsfiddle.net/wanh44t4/

Is there a way to get the top result using PIXI 3?

It seems I can achieve the desired result by doing this:
graphics.currentPath.shape.closed = false;
But is there a better, more modular way to do this?

Duplicate of #1892 I believe.


[CSM-504] Add test on size of getFakeChangeAddress

5000 local runs passed ok

Rewrote this stuff. Is it now what you expected to see here, @gromakovsky?

@Martoon-00 fix hlint and stack2nix.

Oh uzh etot ci


Allow patching games running on LOVE2D 12, or otherwise using luaL_loadbufferx

I have a whole 5 hours saga behind this. Generally, the entire system has been ported to use luaL_loadbufferx instead of luaL_loadbuffer. on the Balatro side, this should change nothing. the game already used it internally.  Games on newer versions of love like love 12 should now work properly. tested with no issues on Mac and Windows, with both Balatro (Love 11.5) and the Beatblock demo(Love 12.0)

Closes #69

And a general heads up that this will likely be merged after PR #68, so you might need to rebase and resolve some merge conflicts.

when will this be merged?


tabs directive does not work when tab content is loaded dynamically

When you try to use ng-include to load tab content you can run into a strange bug. If ng-include loads the content later than the tabs have been initialized then multiple tab content can be visible for the first time when the page appears. It can happen with any other dynamic content loading.
I investigated a bit the possible solutions. I suggest to add a 'reload' property to the directive which can be triggered from outside. So that it is possible to reload the tabs again when the sub-pages are ready.
Ps.: maybe it would be a better solution to make available the jQuery's tabs API, but the guys from materialize have overridden the original function, so it has limited possibilities.

Fixed by #163, right?

yep. :) can you make a new release?

Done.

thanks


feat(html/minifier): compress doctype

Description:
Lowercase to better gzip + fix bug in parser
BREAKING CHANGE:
No
Related issue (if exists):
No

@kdy1 Rebased :+1:


Setup and Configure Event Hub namespace in CDC subscription

Setup and Configure Event Hub namespace in CDC subscription to be used by FHIR Service event subscription for buffering FHIR resource created events.

Aaron is the only team member that currently has access to the CDC subscription. The team did a FHIR build-out hackathon to get the core infrastructure for our FHIR solution created, including the Event Hub namespace and Event Hub resources to be used for post-processing of FHIR data. The team documented the process in preparation for Terraform automation.

An Event Subscription was added to the HDS Workspace (FHIR Service) to enable messages for "FHIR Resource Created" events which flow to an Event Grid System Topic whose endpoint is this Event Hub.


Using NodeGeocoder with RxJs

Hi, first of all thanks for the library, it's quite cool.
I am now trying to bind the geocode callback with RxJs, but I can't seem to make it work.
The error that I get is the beloved 'TypeError: Cannot call method 'geocode' of undefined'
The library without Rx is working fine, and I am also using Rx to run similar methods in other dependencies, as illustrated, and they also work just fine.
This code runs well until I try to geocode inside the Rx chain.
This is my simple code snippet:
var NodeGeocoder = require('node-geocoder')
var otherDependencie = require('other')

var options = {
    // My Google API setup
};
var geocoder = NodeGeocoder(options)

var executeRx = rx.Observable.fromNodeCallback(otherDependencie.execute)
var geocodeRx = rx.Observable.fromNodeCallback(geocoder.geocode)

executeRx("query")
    .flatMap(function (res) {
        return rx.Observable.fromArray(res)
    })
    .flatMap(function (item) {
        return geocodeRx(item.address)
    })
// And so on...

The only difference that I find between the functions I am calling is that execute is declared inside module.exports, while in this library prototype is being used.
I am new to Rx in Javascript (used it just in Java) and there seems not to be much documentation about this fromNodeCallback method (the given example is really simple):
I know that it also receives context as a parameter, so it might be lacking that, but everything I tried didn't work.
This is the Readme file of it: https://github.com/Reactive-Extensions/RxJS/blob/master/doc/api/core/operators/fromnodecallback.md
Any help is highly appreciated, hope I am not the first one integrating Rx with this library.
Thanks!

Hey, finally managed to do it by wrapping the function myself like this:
var geocodeRx = rx.Observable.fromNodeCallback(geocode)

function geocode(geocoder, address, callback){
    geocoder.geocode(address, function (err, res) {
        callback(err, res)
    })
}

Not sure if there is a better way that uses directly the NodeGeocoder, but I don't see much difference.
I will close it as it is solved to me, and there it is the 'solution' if anybody else needs it.

Hi thanks for the solution 👍 I am in a near futur going to add more doc and sample


Cleanup SPEC.md.

Added a linked table of contents to the top.
Distinguished between information necessary for writing an executable
(now under "Engine execution") and higher-level requirements that
aren't directly related to the executable ("Engine specification
file", "Packaging", "Naming convention").
Made the title capitalization consistent because I'm OCD like that.

@brynary I've been told you're the one who should look at this. The intent here is not to change the meaning of the spec, merely to clean up the presentation in preparation for other changes.

@wfleming Bryan has deputized you to 👍  this particular PR if you think it's ready.

@wfleming Actually looks like you have to merge for me.


[FEATURE]: Support for Static Page generation

Description
I would love to be able to maintain a campaign site for my adventures and love the formatting you are maintaining here.  When I found this tool I was looking for ways to do static page generation and hosting on github and/or gitlab.  Your markdown support and pdf generation is amazing and is exactly what I'm looking for.
Could you add support for generating static sites/pages for such campaign sites?  i.e. Hugo or Jekyll

Being able to generate that static page from a git repository would have a lot of potential for community development.
For instance, if I have different chapters of my homebrewery spread across markdown files in a git repository, it would allow people to submit pull requests to suggest edits, add polish, what have you. Having an option to use homebrewery as a build tool to generate the resulting page/pdf would be very powerful.

This may actually be similar functionality to #802

Closing, as this is covered by two other issues.

Downloadable HTML + CSS bundle: #661
Auto-building from CLI: #802


Clean up SMPP transport tests

This will make #965 easier and is also worth doing on its own.

Most easily reviewed one commit at a time.

:+1: from me. Perhaps someone with a little more knowledge of the code could have a look too but it looks like a pretty clean and simple refactoring to me.

:+1:


fix the imageoverride without imageregistry

Signed-off-by: Zhiwei Yin<EMAIL_ADDRESS>

/cherry-pick release-2.5

/lgtm

related PR：https://github.com/stolostron/backlog/issues/22623

/hold


Better descriptions

I like your API. Planning on enhancing descriptions? @senamakel

We'll have them updated :) It'll be on it's way


PNG file resize fail

It can not resize PNG files anymore. Please investigate

Hi @clickmeplz,
Unfortunately I cannot reproduce any issues with PNG resizing.
Can you reproduce this issue on the demo?
e.g. with the following large PNG image, it should be rescaled on client-side to a smaller version (both for the preview and the file to be uploaded):
https://upload.wikimedia.org/wikipedia/commons/3/35/Fond_d'écran_de_Debian.png

Sure... Here it is

There's no guarantee that the resized file will be smaller in bytes.
However your issue mentioned that you're unable to resize at all and it seems this is not the case on the demo?

Sorry! but i don't understand your mean?
I downloaded your link to client. It is 748K (2560 * 1600).
Then i uploaded at demo. It will be 922K (172 * 1080).
If i changed it to .jpg file everything is OK. My local jpg will be 243 K but when i uploaded at demo it changed to 101K, which means demo works fine with jpg.

If you want to make sure resized images are smaller in file size, you can provide the imageType and imageQuality options:
$('#fileupload').fileupload({
    imageType: 'image/jpeg',
    imageQuality: 0.5
});

See also:
https://github.com/blueimp/jQuery-File-Upload/wiki/Options#imagetype
https://github.com/blueimp/jQuery-File-Upload/wiki/Options#imagequality
https://developer.mozilla.org/en-US/docs/Web/API/HTMLCanvasElement/toBlob
You can even apply these options to the demo, by executing the following in the browser console:
$('#fileupload').fileupload('option', {
    imageType: 'image/jpeg',
    imageQuality: 0.5
});

Since saving canvas images as PNGs does not support the quality option, you have to use JPEG (or WEBP) for file size savings.
For PNGs, the browser uses default options which might create bigger file sizes, despite smaller dimensions.

Thank you to explain completely. Perhaps i have to restrict users only to only JPEG and GIF, until i find a solution, because i am loosing my web storage very fast, because of PNG files. Thank you.

You're welcome.
If you wanna protect your web storage limits, I would recommend you to limit the max file size and max number of files to upload on the server-side.

I appreciate your recommend.

Alright, sounds like we can close this issue?

Sure, thank you so much


Improve messaging when there is no downloaded binary

In #338 I neglected to add the v to my version when specifying --tag 0.16.2.
The messaging should be improved; in other words, if there is no tag, then we should not see the message that I saw:
Moving /tmp/gruntwork-script-modules/cloud-nuke_darwin_amd64 to /usr/local/bin/cloud-nuke and setting execute permissions
This is misleading, because moving insinuates that the binary actually exists.
Can we detect the existence of the tag, and show something like:
There is no tag 0.16.2?

I think this is more an issue for gruntwork-install (and probably fetch?), so we should move it there!

Ah, yes, that's right. I'll move it.

Tracked here


Justify why we prefer watts over kilowatts in the UK

See this discussion in UnitConverterDataLoader.cpp. We currently prefer to use "Watts" instead of "Kilowatts" on devices in the UK region. We don't have any remaining documentation on why this choice was made (it's been around since 2013).
We should understand why watts might be preferred over kilowatts in this region and add a comment to the code, or make the logic work like the rest of the world if we don't have a good justification.

as a brit, certainly i'd expect anything less than 1kW to be in watts, not sure if this flag is relevant in that case or not,  - e.g. a 100W light bulb, not a 0.1kW one. but for larger numbers, not sure I have an opinion.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

@mcfedr: That tendency applies in the US as well, and clearly does not explain how the Calculator treats the UK differently from the US.
I think in theory the question is roughly "when you talk about energy in general do you talk about <>?". I envision talking about kW (en-US/en-CA).
Note: when talking about power plants/municipalities, I would scale to MW/GW.

Presumably because Watt is the SI derived unit for power, which most of the world has adopted. Seems like it may be worth swapping this around so the USA is special-cased?
Either way, I prefer it starting from the Watt here in the United Kingdom. 👍

W is an SI unit. kW is a derivate. So I guess starting with W would be a logical choice. The only exception is to start with kg instead of g.

It occurs to me that one reason the US is using kW is that the conversion to HP makes more sense (similar scale) vs W to HP.

In fact, the only thing that's the same scale as Watts is Foot-pounds/minute. @mcfedr , @jamierocks, @MovGP0: Have any of you needed to use that unit (including in primary school...)?

The only exception is to start with kg instead of g

Slightly offtopic, but kilogram is the actual base unit so that's not really an exception.

@jsoref Never used Foot-pounds/minute or BTUs/minute - nor do I have an idea what pounds or BTUs are (I believe feet were the length of a kings foot?).
I was only ever taught in SI units, though occasionally I've seen maths questions that use miles-per-hour - though that kinda makes sense, as our roads still use mph.

It might or might not be obvious, but unit conversions are only useful if users have occasion to use a pair of units. A calculator is not needed to convert kW to W.
If people live in a purely SI world then a large portion of the calculator's unit conversions:

Volume (m³)
Length (m)
Weight and Mass (kg)
Temperature (C)
Energy (J) -- you can find BTU there
Area (m²) -- albeit converting 1km² to 1e6 m² might be...
Speed (m/s) -- albeit km/h ~ m/s
Power (W)
Data (...)
Pressure (Pa)
Angle (rad) -- albeit deg

... are useless.
Thus the question in some respect is: "Which non-SI units are users likely to being using?"
Slightly offtopic: I can't convert from angstrom to lightyear (this is sad) -- the closest I can get more or less is meters to nanometers.

Absolutely, kinda missed the point on that comment. Common non-SI units in the UK are:

Miles (roads)
Pint (milk)
Stone (mass)
Feet + Inches (height)

Kelvin is the SI for temperature - I'm guessing calculator already handles that (I'm not at a windows machine to check I'm afraid).
Though having ones commonly used in the USA is totally useful as well - for example recipes, which are like deciphering ancient Egyptian texts.

Yes, it handles K. For reference, here's a way to see the units it more or less handles: https://github.com/Microsoft/calculator/blob/master/src/CalcViewModel/DataLoaders/UnitConverterDataConstants.h

The only exception is to start with kg instead of g

Slightly offtopic, but kilogram is the actual base unit so that's not really an exception.

I know. Except you use the cgs system 😅 But in that case the cm is the exception to the rule of not having a prefix to the base unit.

Temperature (K) -- albeit C
Data (...)
Angle (rad) -- albeit deg

The temperature unit is not C (Coloumb) and also not ° (angular-degree), but °C (degrees celsius). The degrees prefix (°) is an indicator that it's a relative unit; temperature units with a ° should be seen as a postfix function that converts to K (kelvins):
{ °C(r) ↦ (r + 273.15) ⋅ K | r ∈ ℝ }

This means that when subtracting two thermodynamic temperatures, which are given in °C, you always get the difference in K, since °C does not make sense here mathematically/physically.
Most people don't understand that, so the SI defined the exception that people can write °C, but you need to think about it as K, which I think makes it even more confusing.

The physical dimension for data is the entropy S, which has the SI unit kg⋅m²⋅s⁻²⋅K⁻¹.  But as a non-physicist, you can also call it 'bit' instead.

Note that gon (400 gon = 2π rad) is a totally different unit than ° (356° = 2π rad). There is also the military definition of a ' (3600' = 2π rad). I am unsure why the Windows calculator writes 'grad' instead of 'gon', since people tend to confuse 'grad' with '°'. Also would prefer to have the scientific calculator in rad-mode per default; especially since there is already a π-symbol on it.

There is great conversation here about decisions around unit converters. Closing this as a duplicate of #515, where we are exploring a more comprehensive review of our strategy in this feature area.


Classrooms: support final balancing phase with column selections and highlights

Who is this PR for?
K1-6 educator teams, part of https://github.com/studentinsights/studentinsights/pull/1683.
What problem does this PR fix?
In initial pilot runs, after folks create their classroom lists using their own process, they have a hard time figuring out how to revise class lists if they notice imbalances between classrooms (eg, more students with IEP in one classroom, higher STAR Math scores in one particular classroom).  This helps support them in that final part of the process.
This also avoids concerns with putting markers on student cards that would stay with them always - now we only "label" kids in these ways when teachers want to see this in the final phase of the process.  They still approach planning and do the initial placement process starting with student strengths or whatever else they choose (with all these various markers hidden and not influencing the process until the end).
What does this PR do?
Adds new interactions to the tables at top, where clicking on a column selects that column and updates the student cards below to call out students in relation to that column.  For some attributes this highlights students (eg, if they have an IEP or 504 plan) while for others it colors the student's card (eg, for gender identity).
This also adds http://gka.github.io/chroma.js/ for color math.
Screenshot (if adding a client-side feature)
hovering

hovering enough for tooltip

click to select

story showing different card stylings:

Checklists

[x] Author checked latest in IE - Classlists
[x] Author improved specs for code in need of better test coverage
[x] Author included specs for new code

selfie


Snippet Issue: Right Side Cover Art

Seems like you're working on the expanded flyout to the right side for the album art, similar to this snippet....
Nevermind then

#92


Make PBXFileElement a superclass

This PR makes PBXFileElement a superclass for PBXFileReference, PBXGroup, and PBXVariantGroup.
According to http://www.monobjc.net/xcode-project-file-format.html PBXFileElement is an abstract superclass, so it shouldn't occur as itself. Empirically this is supported by the fact that I grepped through all project.pbxproj files on my disk and there is none that uses this string.
(find ~/src -name 'project.pbxproj' -exec rg PBXFileElement \{\} \;)
The benefit of using the common parent is that the three shared fields (name, path, sourceTree) have to be dealt with only in one class.
Important:
I did this without thinking a lot, just change & break, then fix until all tests pass.
So please go through it with a fine comb and try to break things. Thanks.

This change is

Great catch @gubikmic. Looks good to me. Make sure you update the CHANGELOG.md and if everything passes it should be ready to be merged. Nice job 🎉

Glad I saw this. I was about to submit a PR to add a path property to PBXVariantGroup because my project has one.


[Device Support Request] Bticino L4027C

I have a connection problem with the Bticino L4027C, it is found and can be used but the notification LED on the button remains flashing as if it were still in discovery status. In this way, when the power is cut off, the device is no longer found by the home assistant because it is offline, in fact the color of the LED on the button returns to being red as if it were not connected to anything.
After a lot of resetting the roller shutter button on only one i managed to get it to work and get the green led to go off. At home I also have the Bticino L4411C and these have never given me any problems, they work first time!
I've tried them all but there's no way to get it recognized. I have a Sonoff Zigbee v3 ZBDongle-E stick.
Thanks!

Same problem with L4027C, after pairing the green led still flashing. Shutter work but if I remove the power supply the pair is lost and led is a fix red.
Thanks

@davix10 Please fill out the issue template in the future. Now, just post the device signature and diagnostic information for it here.
@myrkoo What coordinator are you using?

@TheJulianJES sorry but I am beginner, where can I find this information?

I'm using sonoff ZBdongle-P ZNP = Texas Instruments Z-Stack ZNP protocol: CC253x, CC26x2, CC13x2 da ZHA
After a lot of attempts my devices was paired. During the pairing I pushed the connection switch many times and led goes purple and after few minutes off. Very strange!
Thanks

I'm using sonoff ZBdongle-P ZNP = Texas Instruments Z-Stack ZNP protocol: CC253x, CC26x2, CC13x2 da ZHA After a lot of attempts my devices was paired. During the pairing I pushed the connection switch many times and led goes purple and after few minutes off. Very strange! Thanks

Hi, im facing the same issue, could you please give me some advice on the proedure you followed to pair the shutters? At the moment i paired two of those triggering opening wiht the physical button before put them in paring mode, but for the others seems to not work.


Question - how does security work with setting the amount in createOrder()

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem? Please describe.
Apologies, this is not really a feature request, but I wonder; is it secure that this package is basically all run on the client?
For example, order details are created on the client side and passed into the actions.order.create function. Even if the order details were not created on the client, the function to generate the checkout would be run on the client side. For example:
import { PayPalScriptProvider, PayPalButtons } from '@paypal/react-paypal-js'

const orderDetails = {
  purchase_units: [
    {
      description: 'A description',
      amount: {
        currency_code: 'USD',
        value: 5
      }
    }
  ],
  application_context: {
    shipping_preference: 'NO_SHIPPING'
  }
}

  const createOrder = (data, actions) => {
    return actions.order.create(orderDetails).then((orderID) => {
      return orderID
    })
  }

      <PayPalScriptProvider
        options={{
          'client-id': process.env.PAYPAL_CLIENT_ID
        }}>
        <PayPalButtons
          fundingSource={'paypal'}
          createOrder={createOrder}
        />
      </PayPalScriptProvider>

As this is all done on the client, I don't see what stops someone altering any of the potential values passed into actions.order.create - eg lower the value ? I have not done this because I don't actually know how, but this seems to go against everything I have learnt about generating these sorts of values and processing them on the client?
Describe the solution you'd like
Either an update to the docs explaining why this is ok to be done on the client, or an update to the package if this is in fact insecure.
Describe alternatives you've considered
N/A
➕ Anything else?
No response

Hi @alex-r89, PayPal recommend doing a server-side integration with the createOrder() and onApprove() callbacks. That way you can validate the final amount in the onApprove() callback before approving the payment. This question has come up before too. Here is my response in a similar issue: https://github.com/paypal/react-paypal-js/issues/197#issuecomment-945961288
This seems like a great thing for the team to document in the react-paypal-js readme.

Thanks for the info. Yeah, the multiple official paypal docs for this all do everything on the client:

https://www.npmjs.com/package/@paypal/react-paypal-js
https://paypal.github.io/react-paypal-js/?path=/docs/example-paypalbuttons--default
https://developer.paypal.com/sdk/js/reference/

There is also no real mention that this is insecure. Even on the server side docs (https://developer.paypal.com/demo/checkout/#/pattern/server) it doesn't mention that this method is secure and the others should be avoided.
I dont really understand how the server side integration of createOrder() and onApprove() in the above example actually works though. I see it makes a fetch to a server endpoint I assume, but what would we actually be validating here? What is it that we should be doing on the server - it doesnt seem like there is any paypal server side node package to run to generate or verify the request we would make? There doesnt seem to be a paypal server side node package which I can use to generate a valid orderId or confirm anything we post to our endpoint. I dont see what the request to something server side is doing.
Is there anyway you can explain this at all?
Thanks

Here's the video series:
https://www.youtube.com/playlist?list=PLJa2J9JcsT9WRoNhETHRZkXZ-NrF0YBWU
There's a doc update coming in another week or two that will add these examples to the docs.
Currently, the video and accompanying examples only showcases the most basic server-side order creation/capture, but more advanced verification and fulfillment as well as a more thorough explanation of the tradeoffs between the approaches are something we hope to add in the future.
I don't know if that quite closes the issue, but hopefully gets us closer!


Using_The_Physics_Engine.md miss-pelllings

some mispellings:
Introdution -> Introduction (missing R)
Bablon.js -> Babylon.js
phyisics -> physics
handlig -> handling
neccessary -> necessary
sphers -> spheres
Solide -> Solid ?!?
applys -> applies
pyhscis -> physics
phyysics -> physics
pyhscis -> physics (a second time)
bwtween -> between
inlustrations -> illustrations
that's all folks!
PS Nevertheslss I didn't understand the whole paper yet ;-)
source: https://github.com/BabylonJS/Documentation/blob/50e389eba101a55ab169ae1339d05d755477572d/content/tutorials/03_Advanced/Using_The_Physics_Engine.md

Thanks!
I'll fix those asap.


fix IOU - Name is not visible in the IOU request page #2996

Details

returning back  param option.text to src/pages/home/sidebar/OptionRow.js
creating a fulltitle string constant (firstNames) and
assigned to the text property if hasMultipleParticipants is true
in src/libs/OptionsListUtils.js
Fixed Issues

Fixes #2996
Tests

Log in to e.cash with a expensifail account
Navigate to a conversation
Click on the + button
Click on request money
Set an amount
Click continue
Making sure that the Username is visible

QA Steps

Tested On

[x] Web
[ ] Mobile Web
[ ] Desktop
[ ] iOS
[x] Android

Screenshots

Web

Mobile Web

Desktop

iOS

Android

👋 @Viacheslav80, can you let us know why you're unable to test?

I tested it right now

I've already tested it


Add Mastermind URL

Description
Add Mastermind external URL to be used in web VEP e103.
Views affected
None
Possible complications
None
Merge conflicts
None
Related JIRA Issues (EBI developers only)

This PR needs to be merged together with https://github.com/Ensembl/public-plugins/pull/396


Not building with current pytorch master

Which pytorch 0.4 version specifically does your warp-ctc pytorch binding work with?

Thank you for raising the issue and apologies for not noticing earlier. Ryan has just fixed the build with the official release.


Convert | Implement 'toJson()' and 'toString()'

I would like to add this into the repo.
We are using excel primarily for importing purposes, so I had always convert into json format and I would like to have this in this library itself.
Additionally I added toString() to Excel and Sheet, so a print will show more than Instance of '#'
Question:
When I cloned "_isContain()" was not defined?
Remove this line after/before merge.

@KevinBLT Thanks Kevin,
I'll look at the code and will push an update as soon as possible.


Pull these graphical changes please

I think this is how it works?

That it how it works. Normally you would describe your changes here but it's not neccessary this time.


Maven publish should retry publication on transient remote errors

Hi @ljacomet
This is for https://github.com/gradle/gradle/issues/8992
Still unsure how to do this for Ivy. Would this be part of `ExternalResourceResolver?
Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[x] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[x] Ensure that tests pass locally: ./gradlew <changed-subproject>:check

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

Thanks for merging @ljacomet ,
Is https://github.com/gradle/gradle/blob/master/subprojects/dependency-management/src/main/java/org/gradle/api/internal/artifacts/repositories/resolver/ExternalResourceResolver.java the right place to do something for Ivy ? or where would that be? any direction/guidance would be phenomenal


modelcluster. ValueError : invalid literal for int() with base 10: 'THU' on locations pages in the admin preview

On a fresh setup of the bakerydemo with Wagtail 5.0.1
I notice an error on the edit page for the "Location page" types in the preview section
e.g.
Request Method: | GET
http://<IP_ADDRESS>:8000/locations/akranes/
exception type: ValueError
invalid literal for int() with base 10: 'THU'
Exception Location:  /Users/leon/envs/demos/wagtail/venv/lib/python3.11/site-packages/modelcluster/queryset.py, line 211, in test_day

Thanks for reporting! I believe this is an instance of #wagtail/django-modelcluster#171, so I'm closing this as it's an issue with modelcluster rather than bakerydemo.


Add first version of the application form

Here's a new version without placeholders. Can you use this one instead?
ExpectedBehavior_Templates_JobApp (1).pdf


+1 for publishing to HACS

Just wanted to +1 my support for this being published into HACS, now that the original repo for this plugin has been abandoned.

I second that! 👍

Thanks!
I have to fix some issues, as the repository as it is is failing the validations from HACS.
I will let you know as soon I have some progress.

@alexruffell, I really liked the idea, but just to make our life easier, create a new issue with that idea so we have a place specifically to handle the discussions related to this.
Thanks!!

Just wanted to +1 my support for this being published into HACS, now that the original repo for this plugin has been abandoned.

I've updated the instructions for adding this as a Custom Repository in HACS (you can do this right now) and created a PR to add it as a default repository in HACS store, so hopefully we will soon be back there.

Hi @alexruffell
Can you share how you have done that?

Once the card was removed, this is what I went with:

@taduo This is the yaml for the card:
type: entities
entities:
  - entity: sun.sun
    type: custom:multiple-entity-row
    name: Sun
    toggle: false
    show_state: false
    state_color: false
    entities:
      - type: attribute
        entity: sun.sun
        name: Dawn
        format: time
        attribute: next_dawn
      - type: attribute
        entity: sun.sun
        name: Sunrise
        attribute: next_rising
        format: time
      - type: attribute
        entity: sun.sun
        name: Noon
        attribute: next_noon
        format: time
      - type: attribute
        entity: sun.sun
        name: Sunset
        attribute: next_setting
        format: time
      - type: attribute
        entity: sun.sun
        name: Dusk
        attribute: next_dusk
        format: time
  - entity: sensor.moon_phase

@alexruffell, I really liked the idea, but just to make our life easier, create a new issue with that idea so we have a place specifically to handle the discussions related to this.

Done, thanks!
https://github.com/edwardtfn/home-assistant-sun-card/issues/8

Thanks @alexruffell. This is what I needed!

@taduo This is the yaml for the card:

nice!
we can now do this btw in HA 2023.4
        entities:
          - entity: sensor.sun_next_dawn
            name: Dawn
            format: time
          - entity: sensor.sun_next_rising
            name: Sunrise
            format: time
          - entity: sensor.sun_next_noon
            name: Noon
            format: time
          - entity: sensor.sun_next_setting
            name: Sunset
            format: time
          - entity: sensor.sun_next_dusk
            name: Dusk
            format: time
          - entity: sensor.sun_next_midnight
            name: Night
            format: time

Please take a look at the notice about the future of this repository: https://github.com/edwardtfn/home-assistant-sun-card#notice-about-the-future-of-this-repository


Apollo 6.0 SanFrancisco MAP ERROR

Describe the bug
I am trying to test Apollo 6.0 with LGSVL. But I find that the SanFrancisco Map is loaded in a wrong direction. And I do not know how to correct it.
To Reproduce
Steps to reproduce the behavior:

Download SanFrancisco base_map.bin from LGSVL or Use the bin file of Apollo 5.0. Save at modules/map/data/san_francisco
execute following instructions:

dir_name=modules/map/data/san_francisco
bazel-bin/modules/map/tools/sim_map_generator --map_dir=${dir_name} --output_dir=${dir_name}
bash scripts/generate_routing_topo_graph.sh --map_dir ${dir_name}

restart apollo and test it with a random case by LGSVL

Additional context
Add any other context about the problem here.

Could you please tell me how to correct it? Thank you!

Have you made any other changes, if the map is correct, then the car's localization is wrong

Have you made any other changes, if the map is correct, then the car's localization is wrong

I did not change anything. I use the .bin file provided by the LGSVL and use Apollo's scripts to convert it.

so I suggest you need to see if the localization is correct.

@daohu527 Hi, I had the same problem. I found similar issues on the SVL repo:
https://github.com/lgsvl/simulator/issues/1909
https://github.com/lgsvl/simulator/issues/1754
There seems to be a mismatch between the map in the simulator and the HD map for Apollo 6.0.
In the release 2021.3, they said they "Fixed SF, Shalun, Highway101 map origin rotation issues", but the issue on SanFrancisco map still exists.
Could you tell me configs you use, like OS, simulator version, map ID/website or even more details, if you run in SanFrancisco successfully?

Sorry, LGSVL is out of maintenance, Since they don't have a hint about the error, I might be able to fix this in two weeks.

Sorry, LGSVL is out of maintenance, Since they don't have a hint about the error, I might be able to fix this in two weeks.

Thanks!
By the way, I have tried on different versions of LGSVL simulator (2021.1-2021.3) using official SanFrancisco map. I found that positions with the same GPS are invariable in the dreamview, but not in the simulators. Besides, the GPSs of the pre-defined spawn points are different in different versions of simulator.

Sorry, LGSVL is out of maintenance, Since they don't have a hint about the error, I might be able to fix this in two weeks.

Hi, maybe I have fixed this problem, I have uploaded this map into the LGSVL map library named "SanFrancisco_correct". I hope it could save your precious time.

Sorry, LGSVL is out of maintenance, Since they don't have a hint about the error, I might be able to fix this in two weeks.

Hi, maybe I have fixed this problem, I have uploaded this map into the LGSVL map library named "SanFrancisco_correct". I hope it could save your precious time.

Thanks! The maps match!

@JasonBourne1998 Thank you for a great job!

@JasonBourne1998 nice job

Sorry, LGSVL is out of maintenance, Since they don't have a hint about the error, I might be able to fix this in two weeks.

Hi, maybe I have fixed this problem, I have uploaded this map into the LGSVL map library named "SanFrancisco_correct". I hope it could save your precious time.

Nice job @JasonBourne1998 !
I have tested your SanFrancisco map and see that it works on Ubuntu version of LGSVL 2021.3, but issue happened when simulate it on Windows 10 version. Particularly, from the beginning of simulation (or even after click Start), LGSVL on Windows only showed EV with gradient background, and without any objects such as building, road. I have tried to adjust other settings on the scenario (e.g. day time, weather), but nothing has changed.
If you have time, please take a look on this issue. Thank you very much.

@tuanngokien Have you solved the problem? I have the same problem as you. Could you tell me how to solve the problem?

Sorry, LGSVL is out of maintenance, Since they don't have a hint about the error, I might be able to fix this in two weeks.

Hi, maybe I have fixed this problem, I have uploaded this map into the LGSVL map library named "SanFrancisco_correct". I hope it could save your precious time.

Nice job @JasonBourne1998 ! I have tested your SanFrancisco map and see that it works on Ubuntu version of LGSVL 2021.3, but issue happened when simulate it on Windows 10 version. Particularly, from the beginning of simulation (or even after clicking Start), LGSVL on Windows only showed EV with gradient background, and without any objects such as building, road. I have tried to adjust other settings on the scenario (e.g. day time, weather), but nothing has changed. It is so weird because with original SanFracisco map released by SVL team, it works (but having wrong direction problem as mentioned on the main post).
If you have time, please take a look on this issue. Thank you very much.

@tuanngokien Have you solved the problem? I have the same problem as you. Could you tell me how to solve the problem?


Merge generic device helper functions into gpu set.

Remove the use of old NDevices* functions.

@RAMitchell Some planed changes from #3608 extracted here to let that PR less messy.  To complete that PR, some functions need to be generalized for both CPU and GPU.  Since I can't test it on multi-gpu environment, I think it's best to split this as an independent PR to ease reviewing.

@RAMitchell , @canonizer I think I need to rebase it on #3446 .  Otherwise this should be all of it once tests passed.  Please help reviewing, I can't test it on multi-gpu environment.  Thanks in advance. :)

Codecov Report

Merging #3626 into master will increase coverage by 0.24%.
The diff coverage is 97.91%.

@@             Coverage Diff              @@
##             master    #3626      +/-   ##
============================================
+ Coverage     50.39%   50.63%   +0.24%
  Complexity      188      188
============================================
  Files           172      174       +2
  Lines         13986    14031      +45
  Branches        457      457
============================================
+ Hits           7048     7105      +57
+ Misses         6713     6701      -12
  Partials        225      225

Impacted Files
Coverage Δ
Complexity Δ

src/common/timer.h
56.52% <ø> (ø)
0 <0> (ø)
:arrow_down:

src/common/host_device_vector.h
100% <ø> (ø)
0 <0> (ø)
:arrow_down:

tests/cpp/common/test_gpuset.cc
100% <100%> (ø)
0 <0> (?)

src/common/common.h
100% <100%> (ø)
0 <0> (ø)
:arrow_down:

src/common/gpu_set.h
95.45% <95.45%> (ø)
0 <0> (?)

src/data/sparse_page_dmatrix.cc
94.67% <0%> (+0.59%)
0% <0%> (ø)
:arrow_down:

src/data/sparse_page_raw_format.cc
64.15% <0%> (+22.64%)
0% <0%> (ø)
:arrow_down:

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 cb4de52...4f1a5e6. Read the comment docs.

LGTM


HeaderName in .header is automatically converted to lower-case

Hello!
While using Request builder .header is ignoring case in HeaderName and automatically converted to lower-case.
I created issue https://github.com/hyperium/http/issues/535. and hoped to solve it through using .header method, but unfortunately - it is automatically ignoring my case and converting to lower-case header name.
I hope you make changes and at least willl stop forcing custom user headers to lower-case.

This has been discussed in #402 and #924.

Not exactly up for discussion


Grammar fix

Proposed changes

Unreleased project version (optional)

Related issues (optional)

This is actually on purpose; we are an anti-gerund crowd around here when it comes to headings. :)


Add New User form doesn't seem to work more than once

After generating a new user, subsequent users get the same ID

This seems to be the result of the user either not being active yet, or not having an email account configured. If the email is required, we should pass username and email when generating the username.


Fix: properly treat null in nix 2.19 path-info

As mentioned in https://github.com/utdemir/nix-tree/pull/68#issuecomment-1830125958

❤ Thanks :)


Failure of importing nif file

I have installed the blender_niftools_addon-master addon and are getting the below error message when I try and import a premade .nif file. I have googled the term ' asset folder' and all the hits were for 'asset library' which were not much help.
bpy.context.space_data.system_bookmarks_active = 1
Setup your assets folder before importing!
Any ideas on what could caused this?

Context: trying to import .nif from starfield

Hi there,
There's a simple guide showing how to install and use the plugin: https://starfieldgeometrybridge.github.io/docs/user-manual/hello-world/
Also, grab the latest release from here (instead of Nexus page): https://github.com/SesamePaste233/StarfieldMeshConverter/tree/master/dist/tool_export_mesh
Please go through the installation steps. Thanks!