[Feature Request] Mention existance of the fields property in the Form component

Existing Component
Yes
Component Name
Form
Description
Suppose I want to do something to a form-item in a form. I know only its prop name but don't have a ref to it. If I have a ref to the form I could access its form-item like this:
this.$refs.form.fields[prop]
Than I could invoke clearValidate for instance for a single field. But this is somewhat wrong because existence of the fields property in form is not documented. Maybe add some getter for form-items to form or mention fields property in docs?

I know only its prop name but don't have a ref to it

Could you give an example why you can't add a ref to a certain <el-form-item>?
form.fields is not meant to be exposed to users.

Suppose I have a lot of <el-form-item>s. Then some of the fields change externally and I want to manipulate a subset of those related <el-form-item>s. I would have to write ref for every field and then I would have to come up with a way to convert a field to a ref name. In short I want to manipulate these items massively using loops and not individually, but also not all at once.


feat(metas.ts): Simplify Twitter Card Tags

Description
This commit optimizes the  plugin by simplifying the Twitter Card tags (twitter:*) when OpenGraph tags (og:*) are present, eliminating the need for redundant tags.
Changes Made:

Removed twitter:title tag when og:title is present.
Removed twitter:image tag when og:image is present.
Removed twitter:description tag when og:description is present.
Simplified Twitter Card tags to include only  and  when  tags are present.

Related Issues
Related to #487
Check List

[x] Have you read the
CODE OF CONDUCT
[x] Have you read the document
CONTRIBUTING

[x] One pull request per feature. If you want to do more than one thing,
send multiple pull request.
[x] Write tests.
[x] Run deno fmt to fix the code format before commit.
[ ] Document any change in the CHANGELOG.md.

Thank you!
Can you update the tests, please? (run npm task test:update)

Thank you!


CI failure:Failed to execute test case D00002/D00003, IPv6 gateway is not valid

Output of the failure test
• [FAILED] [10.766 seconds]
test ippool CR [ippoolCR]
/home/runner/work/spiderpool/spiderpool/test/e2e/ippoolcr/ippoolcr_test.go:22
  [It] add a route with `routes` and `gateway` fields in the ippool spec [D00002, D00003, smoke, A00011]
  /home/runner/work/spiderpool/spiderpool/test/e2e/ippoolcr/ippoolcr_test.go:164

  Begin Captured GinkgoWriter Output >>
    create namespace ns3348-536159128
    Succeeded to create ippool v4pool-3348-570615199
    Succeeded to create ippool v6pool-3348-610654030
    STEP: update v4 pool: invalid `gateway` and valid `route` 10/11/22 10:33:48.644
    STEP: update v4 pool: valid `gateway` and invalid `route` 10/11/22 10:33:48.685
    STEP: update v4 pool: valid `gateway` and `route` 10/11/22 10:33:48.714
    STEP: update v6 pool: invalid `gateway` and valid `route` 10/11/22 10:33:48.743
    STEP: update v6 pool: valid `gateway` and invalid `route` 10/11/22 10:33:48.773
    STEP: update v6 pool: valid `gateway` and `route` 10/11/22 10:33:48.8
    create pod ns3348-536159128/pod3348-644363592
    wait for pod ns3348-536159128/pod3348-644363592 ready
    pod ns3348-536159128/pod3348-644363592 ADDED event

    pod ns3348-536159128/pod3348-644363592 status=Pending

    pod ns3348-536159128/pod3348-644363592 MODIFIED event

    pod ns3348-536159128/pod3348-644363592 status=Pending

    pod ns3348-536159128/pod3348-644363592 MODIFIED event

    pod ns3348-536159128/pod3348-644363592 status=Pending

    pod ns3348-536159128/pod3348-644363592 MODIFIED event

    pod ns3348-536159128/pod3348-644363592 status=Pending

    pod ns3348-536159128/pod3348-644363592 MODIFIED event

    pod ns3348-536159128/pod3348-644363592 status=Pending

    pod ns3348-536159128/pod3348-644363592 MODIFIED event

    pod ns3348-536159128/pod3348-644363592 status=Running

    pod: ns3348-536159128/pod3348-644363592, ips: [{IP:<IP_ADDRESS>} {IP:fd00:127::3}]
    succeeded to check pod ipv4 ip
    succeeded to check pod ipv6 ip
    create pod ns3348-536159128/pod3348-644380792
    wait for pod ns3348-536159128/pod3348-644380792 ready
    pod ns3348-536159128/pod3348-644380792 ADDED event

    pod ns3348-536159128/pod3348-644380792 status=Pending

    pod ns3348-536159128/pod3348-644380792 MODIFIED event

    pod ns3348-536159128/pod3348-644380792 status=Pending

    pod ns3348-536159128/pod3348-644380792 MODIFIED event

    pod ns3348-536159128/pod3348-644380792 status=Pending

    pod ns3348-536159128/pod3348-644380792 MODIFIED event

    pod ns3348-536159128/pod3348-644380792 status=Pending

    pod ns3348-536159128/pod3348-644380792 MODIFIED event

    pod ns3348-536159128/pod3348-644380792 status=Pending

    pod ns3348-536159128/pod3348-644380792 MODIFIED event

    pod ns3348-536159128/pod3348-644380792 status=Running

    pod: ns3348-536159128/pod3348-644380792, ips: [{IP:<IP_ADDRESS>} {IP:fd00:127::6}]
    succeeded to check pod ipv4 ip
    succeeded to check pod ipv6 ip
    check whether the route information is effective
    delete namespace ns3348-536159128
    Delete IPv4 pools v4pool-3348-570615199 and IPv6 pools v6pool-3348-610654030
  << End Captured GinkgoWriter Output

  Expected
      <[]uint8 | len:0, cap:512>:
  not to be empty
  In [It] at: /home/runner/work/spiderpool/spiderpool/test/e2e/ippoolcr/ippoolcr_test.go:324

Log file
e2edebugLog.txt
Link of the CI build
https://github.com/spidernet-io/spiderpool/actions/runs/3226021852/jobs/5279105064
Additional
This morning, the CI in one of my PRs reported that the IPv6 gateway did not take effect in the container. I think it's strange, because this PR only adds a test to New function of Limiter, and nothing else has been changed, so I don't think my PR have introduced any new errors.
But here is an incorrect assertion:

The codes are as follows:

That is, executing ip -6 r | grep 'default via {IPV6_GATEWAY}' in the container does not retrieve anything.
I checked the relevant code, but found no problem. Then I found the IPPool that set the gateway through the log I downloaded:

And, the call result of IPAM CNI of a Pod that allocated IP addresses through the IPPool contains the default  IPv6 route:

Therefore, I think the result returned by IPAM CNI to Main CNI should be correct, and it is doubtful whether Main CNI takes effect on this default route. So I created an identical IPv6 IPPool, and then asked a Pod to allocate IP addresses through it. As a result, I saw both general and gateway generated routes in the container:

Back to the beginning, why did this assertion fail?

The reason is known and I will fix it as soon as possible.
The reasons for this are as follows:

For ipv6 scenarios, the format of the PodRouteValue when added is as follows:

fd00:015d::1

When the application is created, it actually takes effect as:

fd00:15d::1

The check command in the case is:

ip -6 r | grep 'default via fd00:015d::1

In this case, the content cannot be retrieved using the above command, so the os/exec execution returns an error

fix #893


Fixed building extensions on Ruby 64 bit

Resolves https://github.com/Spooner/ashton/issues/37
The history is a little messy so let me know if you want me to re-do as a single commit

I'm not too bothered by the dodgy commit history, but, on further reflection, I think I sent you on a wild goose chase (sorry!). typedef is not related to #define so I think that all we've achieved is to disable the typedef, regardless of platform. Thus, I suspect this will be broken on 32 bit ruby now! I did a quick Google and I think this looks like a better solution:
#if !__x86_64__
// do the typedefs
#endif

Sound good to me ill do it in a new PR

OK. Thanks. Sorry again for my initial brain fart!


Add simple examples

Hi,
A lot of my everyday uses are not in the documentation, ie:

how to add a product/customer/category attribute?
how to get the current customer?

In summary, it would be nice to have some "real life" code snippets.

Thank you for the great suggestions. It helps us know what's really useful for the readers.
We'll try to add those along the way.

Some other suggestions:

How to change the favicon?
How to set the images placeholder?


docs: add React.Fragment info

What: Add info to readme about React.Fragment and its interaction with container.firstChild
Why: This is a gotcha many people might run into.
Checklist:

[x] Documentation
[ ] Tests N/A
[x] Ready to be merged
[ ] Added myself to contributors table

When rendering a component that has a React.Fragment as the root element.
container.firstChild will only grab the first child of the fragment, NOT the fragment JSX-tag

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


os: readConsole don't fill buffer on Windows

What version of Go are you using (go version)?
go version devel +5af7553 Wed Nov 9 00:21:04 2016 +0000 windows/amd64
What operating system and processor architecture are you using (go env)?
Windows 7 64bit
What did you do?
package main

import (
    "fmt"
    "os"
)

func main() {
    p := make([]byte, 100)
    fmt.Printf("> ")
    for {
        n, err := os.Stdin.Read(p)
        fmt.Printf("[%d %q %v]\n> ", n, p[:n], err)
    }
}

https://github.com/golang/go/issues/17427#issuecomment-254703215
go run and type hello\n on the prompt >.
What did you expect to see?
output  [7 "hello\r\n" <nil>]
What did you see instead?
[1 "h" <nil>]
[1 "e" <nil>]
[1 "l" <nil>]
[1 "l" <nil>]
[1 "o" <nil>]
[1 "\r" <nil>]
[1 "\n" <nil>]

When 100 bytes buffer is given, readConsole should be filled with hello\r\n on console. Because GetConsoleMode() is ENABLE_LINE_INPUT and if call ReadFile with enough buffer, it fill as hello\r\n.
https://go-review.googlesource.com/#/c/31114/ will fix this.

This doesn't seem like a bug to me; Read is not guaranteed to read all the available bytes. I'll mark it as a possible fix for the future since it would help with optimization, but you should always remember that Read is free to not fill the buffer whenever it wants.

But it should call ReadFile with enough bytes because ReadFile break half of multi-byte unread.
https://github.com/golang/go/issues/17427#issuecomment-255524483
When the leading byte is read, trailing byte never be read. This is clearly with C code.
https://github.com/golang/go/issues/17427#issuecomment-255541084
So this should be fixed until next release.

The second example looks bug to me and it is reproducible.
I used '%q' instead of '%x' and saw following results:
> ああ<enter>
[3 efbd82 nil]
[1 0d nil]
[1 0a nil]

Note that 'あ' = [e3 81 82], '\r' = [0d], '\n' = '[0a]'.
Although io.Reader don't prohibit partial read, it should return:
> ああ<enter>
[3 e38182 nil]
[3 e38182 nil]
[1 0d nil]
[1 0a nil]

Also here are results in go1.7.3:
> ああ<enter>
[8 e38182e381820d0a nil]

So this is a regression. https://github.com/golang/go/commit/1af769da8260b277bd5aa92b5074b3400b1f8d9d
Even worse, my input was swallowed when:
> あ<enter> // os.Stdin.Read never return in this case

Indeed, that looks like a bug to me. Apologies for misinterpreting the report before; I thought it was only that reads were being broken up into multiple calls. If we're dropping bytes, that's very bad and could be a release blocker. Thanks for the sample output.

The second example looks bug to me and it is reproducible.

@hirochachacha please add your example to TestReadStdin in os package to see what is wrong, and report here. Thank you.
Alex

The second example looks bug to me and it is reproducible.

@hirochachacha please add your example to TestReadStdin in os package, to see what is going on, and report here. Thank you.
Alex

@alexbrainman I tried, but it's not reproducible there.
But I understand what problem is.
diff --git a/src/os/file_windows.go b/src/os/file_windows.go
index 8f2d4d3..cb806bd 100644
--- a/src/os/file_windows.go
+++ b/src/os/file_windows.go
@@ -232,6 +232,7 @@ func (f *File) readOneUTF16FromConsole() (uint16, error) {
        for {
                var nmb uint32
                err := readFile(f.fd, buf[:], &nmb, nil)
+               println("nmb:", nmb, "byte:", buf[0])
                if err != nil {
                        return 0, err
                }

then,
> あ
nmb: 2 byte: 130   // 0x82
nmb: 1 byte: 13    // 0x0d
nmb: 1 byte: 10    // 0x0a

Note that 'あ' = [82 a0] in CP932 encoding.
Current code uses var buf [1]byte, but nmb is 2, a remaining byte (0xa0) is truncated.
I don't know https://msdn.microsoft.com/en-us/library/windows/desktop/aa365467(v=vs.85).aspx
should guarantee that nNumberOfBytesToRead >= lpNumberOfBytesRead or not. At least we expected that.

I understand what problem is.

Yes. It looks weird. We ask for 1 byte, but ReadFile says that it returned returned 2 bytes. I wonder if ReadFile actually have written correct bytes into the buffer (actually second byte past the end of our buffer). Also, please, tell us if ReadFile returns error. Change your code to:
diff --git a/src/os/file_windows.go b/src/os/file_windows.go
index 8f2d4d3..5b0e0c8 100644
--- a/src/os/file_windows.go
+++ b/src/os/file_windows.go
@@ -226,12 +226,13 @@ func (f *File) copyReadConsoleBuffer(buf []byte) (n int, err error) {
 // readOneUTF16FromConsole reads single character from console,
 // converts it into utf16 and return it to the caller.
 func (f *File) readOneUTF16FromConsole() (uint16, error) {
-	var buf [1]byte
+	var buf [2]byte
 	mbytes := make([]byte, 0, 4)
 	cp := getCP()
 	for {
 		var nmb uint32
-		err := readFile(f.fd, buf[:], &nmb, nil)
+		err := readFile(f.fd, buf[:1], &nmb, nil)
+		println("nmb:", nmb, " err=", err, "byte1:", buf[0], "byte2:", buf[1])
 		if err != nil {
 			return 0, err
 		}

and tell us what it prints. Thank you.
Alex

Here you are.
> あ<enter>
nmb: 2 err= (0x0, 0x0) byte1: 130 byte2: 0
nmb: 1 err= (0x0, 0x0) byte1: 13 byte2: 0
nmb: 1 err= (0x0, 0x0) byte1: 10 byte2:0

Unfortunately, readFile doesn't return error even if truncation occurred.
I cannot find the way to recover from truncation. So I think mattn's approach is right.

Here you are.

Thank you. I will think about this.
Alex


Ordered lists are rendered incorrectly

After a recent update the ordered lists are rendered incorrectly in comments and descriptions: the left padding is applied, but the numbers are not shown anymore.
Here's how a comment looks when editing:

And here's how it looks after publishing:

Fixed. Thanks for reporting.


Update README.md

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

Project coverage is 71.83%. Comparing base (9acdf2b) to head (be65182).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main       #5      +/-   ##
==========================================
- Coverage   72.09%   71.83%   -0.26%
==========================================
  Files          14       14
  Lines        1598     1598
==========================================
- Hits         1152     1148       -4
- Misses        446      450       +4

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


Avoid creation of some temporary objects

I consider the impact minimal, but reasoning is sound. Merging.


compile with -parameters

We should compile the library with -parameters, to avoid seeing things like arg0 in code generated from the built-in repository supertypes.
I don't use Maven, and so I don't know how to make this work, but we should do something like it.

@KyleAure @otaviojava Do you guys how to do this so it works?

What is missing is that the api jar also needs to be compiled as tck tests extend from jakarta.data.repository.BasicRepository for example and this is where the missing parameter names are coming from.

OK, I figured it out. I need to set this in a new section in api/pom.xml.
It does not work to set it in the root pom.
Anyway I've tested it and it appears to be working perfectly now.
@KyleAure Could you please review this?

Great, thanks.


Added Kaggle API integration and Titanic dataset

Introduced a mixin for kaggle workflow for download and authentication
Added a unit test that tests the workflow using mocks

Thank you so much @skanjila . It would be great if you could add a couple words about this in the user guide ;)


gtree: 1.8.2 -> 1.8.3

Diff: https://github.com/ddddddO/gtree/compare/v1.8.2...v1.8.3
Changelog: https://github.com/ddddddO/gtree/releases/tag/v1.8.3
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 240775 run on x86_64-linux 1

1 package built:

gtree


feature/update-pytest-asyncio-to-0.6.0

Coverage remained the same at 94.098% when pulling 28fb5b36c5319b5ceea8ce066e305d718dbb7ecf on feature/update-pytest-asyncio-to-0.6.0 into 5f8ee7bc70516c9ec2aa6fc795750dfea304f59f on develop.


Hack removal - jsoncpp

Soft-link for irregular include paths?
Can we remove "homogenization" of outputs?

Fixed in last few commits at https://github.com/doublethinkco/cpp-ethereum-cross/commits/72a11d7d29bbd570a7d6081c4f9f627cf0738bdb


Possible Kivy Garden Module

Hi there,
Created a widget for displaying Numpy Matrices as images using a color map. Using kivy at the moment for creating an App for microscopy image analysis, found it super easy, and definitely something allot of academics could use, a few simple widgets like this and more plotting especially interactive plotting widgets could definitely uptake. Would been keen to get involved but not the greatest coder ever. Would anyone be able to check out the code in my repository and see if it's up to scratch/something you'd want in garden?
Best,
Sam
https://github.com/samocooper/numpy_image

I'm sorry that no one got in touch with you sooner. Personally I just haven't noticed the opened issue properly.
While the description sounds nice, I haven't found any visual representation of what does it do, nor when trying locally the dependencies are resolved (missing skimage, currently known as scikit-image for start). And other less important issues such as proper pep8 formatting, documentation for the functions and proper package name (garden.<package>). On the other hand, you don't need to use that to output an image as you can just load the image and display it in the Kivy Image widget.
If you still feel that you want to include it to the garden, please fix the issues mentioned above and let us know.


Automatically build a new docker image when a new ansible version is out

This should be done after #2
It would be very nice to have such a feature so I wouldn't have to commit and push a commit and keep this up to date with the upstream. We need set it and forget it solutions. This project deserves this.
There are build triggers in docker hub:

And there are webhooks: https://docs.docker.com/docker-hub/webhooks/
Related article:
https://medium.com/mobileforgood/patterns-for-continuous-integration-with-docker-on-travis-ci-71857fff14c5
Related tools (maybe?)

https://requires.io/
https://pyup.io/
https://greenkeeper.io/

I've subscribed to requires.io pull-requests, should be fine like this for now


change position to bottom_left

Hey guys,
I am really a newbie using rasperry, MMMs, html or css.
I like to configure this module to show my photos in the bottom left corner f.e..
If set position: "bottom_left", all pictures are shown in "middle" top left over all other modules. See attached picture

I dont understand why and hope you can help out here

Okay, got a solution which makes me satisfied for now:
As described in Usage I setted showWidth & ShowHeight to make the module smaller at all. Depending on image ratio it fits good now.
For all other newbies, here´s my code:
{  			module: "MMM-GooglePhotos",   			position: "bottom_center", 			config: { 			albums: ["xxxxxxx"], // Set your album name. like ["My wedding", "family share", "Travle to Paris"] 			updateInterval: 1000 * 120, // minimum 10 seconds. 			sort: "random", // "old", "random" 			uploadAlbum: null, // Only album created by create_uploadable_album.js. 			condition: { 				fromDate: null, // Or "2018-03", RFC ... format available 				toDate: null, // Or "2019-12-25", 				minWidth: null, // Or 400 				maxWidth: null, // Or 8000 				minHeight: null, // Or 400 				maxHeight: null, // Or 8000 				minWHRatio: 1, 				maxWHRatio: 5, 				// WHRatio = Width/Height ratio ( ==1 : Squared Photo,   < 1 : Portraited Photo, > 1 : Landscaped Photo) 								}, 			showWidth: 512, // These values will be used for quality of downloaded photos to show. real size to show in your MagicMirror region is recommended. 			showHeight: 288, 			timeFormat: "YYYY/MM/DD HH:mm", // Or relative can be used. 			/*debug: true,*/   			} 		},
If there is a better solution: tell me :-)


Vue support ?

What I gather from this issue https://github.com/prettier/prettier/issues/651, is that vue support should be implemented in something like the editor.
prettier-atom had this PR https://github.com/prettier/prettier-atom/pull/44
Wondering if something similar can be done for vscode ?

It should be possible to do something. (maybe ugly, It seems vscode doesn't have embedded language support)
I don't use vue myself. Has vscode a language associated with it?
PR welcome ! :fire:

I don't use vue myself. Has vscode a language associated with it?

It has something called single file component (SFC). In a single file, you have html, script and styles sections.
So the idea is to single out the script (javascript/typescript) section and send it to prettier for formatting.
I'll try to take a look at the code and see how this can be done.

@jbrodriguez  I'll be waiting for your good news.

+1

+1

+1

Will support it once it lands in prettier. And it seems it will.

Prettier has support for Vue components now (for the <script> and <style> parts): https://github.com/prettier/prettier/issues/2097

So does this extension. It's just disabled by default. disabledLanguages


feat(PL-2466): add joy release validate command

Result of running the new command against a local copy of the catalog with all releases using the generic chart with the values.cue schema:
go run ./cmd/joy --skip-version-check rel validate

Error: validating releases:
  - external-brokerage/qa: api.cronJobs.importer.pod.imageSufix: field not allowed
  - external-brokerage/staging: api.cronJobs.importer.pod.imageSufix: field not allowed
  - servicing-console/demo: frontend.container.targetPort: field not allowed
  - servicing-console/production: frontend.container.targetPort: field not allowed
  - servicing-console/qa: frontend.container.targetPort: field not allowed
  - servicing-console/staging: frontend.container.targetPort: field not allowed
  - tenants/demo: api.deployment.priorityClassName: field not allowed
  - tenants/production: api.deployment.priorityClassName: field not allowed
  - tenants/qa: api.deployment.priorityClassName: field not allowed
  - tenants/staging: api.deployment.priorityClassName: field not allowed

external-brokerage/qa: api.cronJobs.importer.pod.imageSufix: field not allowed

Not in this PR, but I think the cue spec is incorrect. All those jobs would fail without it.
https://github.com/nestoca/catalog/blob/e7e0d9f5d3b407a3336bb78f47ce99e496e13b3a/environments/production/releases/origination/accounts.yaml#L35-L73
Same thing with the targetPort.

Another note:
You are right, targetPort in this context does nothing. I got mixed up with https://github.com/nestoca/catalog/blob/2893e91ff7be0e483b391d4590b65724999d9ae6/environments/production/releases/servicing/servicing-console.yaml#L43-L44


Replacement terminology for "ARG v1" (big) vs "ARG v2" (small)

We currently have 2 main SARS-CoV-2 "ARG"s, the larger one which only goes up to mid 2021, and the smaller one which is subsampled and goes up to mid 2022. We switched from "big" vs "small" to "v1" versus "v2" to refer to these tree sequences, but there's probably a nicer description to use. We can throw around some ideas in this issue.

Here are some ideas:

"dense" (or "unreduced") vs "subsampled"
"wide" vs "long"
"time-limited" vs "sample-limited"

I kind of prefer "wide" versus "long".

Yes, there's something nice and simple about that. We can get Sally's opinion tomorrow too.

"Wide" vs "long" is good, if we can explain the thinking clearly early on.

We don't actually deal with this here (this is where we make the method), so going to close this issue.

Everyone likes it. So good to close anyway.


Line chart: manually limit x-axis tick labels

Hello,
I'm building a line chart with a high number of data points (for the sake of example 10) and I would like to draw lines based on all 10 data points, but I would like to limit the number of x-axis tick labels to exactly 5 (every second data point). Here's the simplified version of what I'm trying to do:
var lineChart = new Chart(cnvLineChart, {
    type: 'line',
    data: {
        labels: [1, 2, 3, 4, 5, 6, 7, 8, 9, 10],
        datasets: [
            {
                label: "Data set 1",
                data: [11, 22, 33, 44, 55, 66, 77, 88, 99, 100]
            },
            {
                label: "Data set 2",
                data: [111, 122, 133, 144, 155, 166, 177, 188, 199, 1100]
            }
        ]
    },
    options: {
        scales: {
            xAxes: [{
                ticks: {
                    stepSize: 2,
                    autoSkip: false
                },
                type: 'linear',
                position: 'bottom'
            }]
        }
    }
});

By default line chart uses categorical scale on the x-axis. But as far as I can understand I need a linear scale to get access to the ticks.stepSize option. However, when i set the type to linear as shown above, my script breaks and gets into an infinite rendering loop.
I searched through issues and found https://github.com/chartjs/Chart.js/issues/2228 where someone suggests to make a tick callback and return an empty string for every second tick. This works, but then I still end up with a really finely granulated grid, which I would prefer followed the tick labels.
Any idea how I can accomplish this?

@nielssj in the callback if you return null or undefined the grid line will also be hidden

Perfect, that works, thank you :)

If you set a tick to be undefined or null using the tick callback, then you lose the ability to format the ticks as they show up when you hover over a point. Is there a way of dealing with this?


Update tapir-core, tapir-http4s-server, ... to 1.6.4

About this PR
📦 Updates

com.softwaremill.sttp.tapir:tapir-core
com.softwaremill.sttp.tapir:tapir-http4s-server
com.softwaremill.sttp.tapir:tapir-json-circe
com.softwaremill.sttp.tapir:tapir-newtype
com.softwaremill.sttp.tapir:tapir-openapi-docs
com.softwaremill.sttp.tapir:tapir-swagger-ui

from 1.6.0 to 1.6.4
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.sttp.tapir" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.softwaremill.sttp.tapir" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, version-scheme:early-semver, commit-count:1

Resolved in main branch


Use updated @FrankDocGroup

If you could update the version to 3.1-SNAPSHOT, we can merge this one 😄


ENG-842: Fetch data at height concurrently

A small change in fetch_data to query the two event logs concurrently, which might bring down the ~2.5s query time to something closer to 1s.

Merged as part of https://github.com/consensus-shipyard/ipc/pull/871


Tweak dark mode colors

We are implementing a Backstage plugin with dark mode support and noticed that some of the colors are the same as in the light theme.
I updated these following Google's guidelines on tonal variants for dark themes (and their color tool).
ProgressCard before/after:

Status before/after:

:heavy_check_mark: Checklist

[x] All tests are passing yarn test
[x] Screenshots attached (for UI changes)
[ ] Relevant documentation updated
[x] Prettier run on changed files
[ ] Tests added for new functionality
[ ] Regression tests added for bug fixes

Any before/after shots? :)

@freben Updated the PR description with some from the Storybook 👍

on a personal note (am not designer), oof those sure have low contrast in the screen shot (along with pending and warning being super close to each other in color + lightness)

Some adjustments per @freben's feedback

@timbonicus this is great, thank you!!


Writing to MOL error

Describe the bug
When trying to write molecules to MOL files with RDKit I get an error despite this being one of the supported file formats listed by the wrapper.
To Reproduce
Steps to reproduce the behavior. A minimal reproducing set of python commands is ideal.
from openforcefield.topology import Molecule
from openforcefield.utils.toolkits import RDKitToolkitWrapper

mol = Molecule.from_smiles('CC')
mol.generate_conformers(n_conformers=1)
# try and write to mol file with rdkit
mol.to_file('test.mol', 'mol', RDKitToolkitWrapper())

KeyError: 'MOL'

I think the writer dictionaries here and here just need to be updated. It may also be better to store the reference to these objects in one variable to save repetition of code and make updating writers easier for example
from rdkit import Chem
_toolkit_file_write_formats = {'SDF':Chem.SDWriter , 'MOL':Chem.SDWriter , 'SMI':Chem.SmilesWriter , 'PDB':Chem.PDBWriter, 'TDT': Chem.TDTWriter }

the function to write the files could then be something like
def to_file(self, molecule, file_path, file_format):
        """
        Writes an OpenFF Molecule to a file-like object
        Parameters
        ----------
        molecule : an OpenFF Molecule
            The molecule to write
        file_path
            The file path to write to
        file_format
            The format for writing the molecule data
        Returns
        ------
        """
        from rdkit import Chem
        file_format = file_format.upper()
        with open(file_path, 'w') as file_obj:
            rdmol = self.to_rdkit(molecule)
            try:
                writer = self._toolkit_file_write_formats[file_format](file_obj)
                writer.write(rdmol)
                writer.close()
            except KeyError:
                rasie InvalidFileTypeError(f'The file type {file_format} is not currently supported to be wrote to by RDKit.')

the readers could also be stored this way
from rdkit import Chem
_toolkit_file_read_formats = {'SDF': Chem.SupplierFromFilename, 'MOL': Chem.SupplierFromFilename, 'SMI': Chem.SmilesMolSupplier, 'TDT':Chem.TDTMolSupplier}

Not sure how useful this second option is however as depending on the input different actions are performed this would only be simpler if the functions could be unified in some way.
Computing environment (please complete the following information):

Operating system linux

It may also be better to store the reference to these objects in one variable to save repetition of code and make updating writers easier for example

I love the idea of defining the reader/writer dict at the class level, rather than twice in the to_file and to_file_obj functions. Part of the reason we never caught the absence of a MOL writer is because of code cleanliness/duplication issues like this.
I'm not sure that I like the try/except logic for finding a writer. For the same level of indentation, we could do if file_format in _toolkit_file_write_formats, and not have to deal with an exception at all.

The try/except construct will save a key hashing and dict lookup, and most of the time the key will exist. Pythonistas usually prefer EAFP than LBYL. Not that it matters in practice if this is not a tight loop.

Glad you like the idea I think it would definitely help keep the functions clean and make adding more in future very easy as you would only have to extend the dictionary.
I would also vote for the try/except block here as asking for permission in most use cases for this function will be a waste of time.

Wow, I just read a little into LBYL and EAFP and this is a serious debate. Even Guido chimed in.
I don't think a 2x speedup is of serious concern in this specific situation. Between now and the heat death of the universe, using LBYL throughout the OFFTK ecosystem may cumulatively require several additional cpu-seconds.
On the other hand, using try/excepts liberally throughout OFFTK runs the risk of diverting developer attention from ToolkitWrapper try/excepts (which will become complex and nasty as we find more failure modes). Developer time is much more expensive than CPU seconds, and if misinterpreting a try/except causes an OFF developer to take 15 more minutes to debug a problem, that's a net loss for the universe.
Anyway, that's just my two cents. I see the points of view on the other side, and would be happy to take a vote among OFF devs and agree to use one convention or another throughout the ecosystem.

Yeah, this is quite a big thing in python letting the flow beeing controlled with exceptions since they are designed to be cheap.  My personal preference is to use exceptions but like you say this can be an issue for bugs if not used precisely, happy to do what the general consensus is, however.  For now, shall I close this with my PR for XYZ file support, ill only change the writers for now as the readers are a little more complicated due to the extra functions called.


Misspelling

In the Next Steps section, the bullet point "Rand the Emulator on Docker for Windows" should be "Ran the Emulator on Docker for Windows" instead.

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

ID: 109ed3f8-faf1-92b9-bf50-4be52e9099d4
Version Independent ID: ebc5326a-46da-6bc1-1f47-01a8e16d11fb
Content: Develop locally with the Azure Cosmos DB Emulator
Content Source: articles/cosmos-db/local-emulator.md
Service: cosmos-db
GitHub Login: @David-Noble-at-work
Microsoft Alias: danoble

Thanks for the feedback! We are currently investigating and will update you shortly.

@devjourney Thanks for reporting! I am working to fix it and will be done soon.


INFO: Created TensorFlow Lite XNNPACK delegate for CPU.

Problem Description

System Information
Mitmproxy: 6.0.2
Python:    3.10.12
OpenSSL:   OpenSSL 3.0.2 15 Mar 2022
Platform:  Linux-6.2.0-33-generic-x86_64-with-glibc2.35

You are using an outdated mitmproxy version, please upgrade.
The error you are seeing is likely an artifact from mitmproxy importing some Python packages. That's specific to your environment and not a mitmproxy bug. If you install mitmproxy in a clean venv, you should be fine.

@mhils


-[FIRTranslateModelLoader modelDownloader:didCompleteDownloadWithModelPath:modelInfo:error:completion:]: unrecognized selector sent to instance 0x...

Hi @firebase,
Started from here:
https://firebase.google.com/docs/ml-kit/ios/translate-text
I'm getting this error(bellow) when I'm running your code example:
import UIKit
import Firebase

final class FirModelViewController: UIViewController {

  override func viewDidLoad() {
    super.viewDidLoad()

    let done = UIBarButtonItem(barButtonSystemItem: .done, target: self, action: #selector(doneTapped(_:)))
    navigationItem.setLeftBarButton(done, animated: true)

    NotificationCenter.default.addObserver(
        forName: .firebaseMLModelDownloadDidSucceed,
        object: nil,
        queue: nil
    ) { [weak self] notification in
        guard let strongSelf = self,
            let userInfo = notification.userInfo,
            let model = userInfo[ModelDownloadUserInfoKey.remoteModel.rawValue]
                as? TranslateRemoteModel else { return }
        // The model was downloaded and is available on the device
    }

    NotificationCenter.default.addObserver(
        forName: .firebaseMLModelDownloadDidFail,
        object: nil,
        queue: nil
    ) { [weak self] notification in
        guard let strongSelf = self,
            let userInfo = notification.userInfo,
            let model = userInfo[ModelDownloadUserInfoKey.remoteModel.rawValue]
                as? TranslateRemoteModel
            else { return }
        let error = userInfo[ModelDownloadUserInfoKey.error.rawValue]
        // ...
    }
  }

  @objc private func doneTapped(_ sender: UIBarButtonItem) {
    // Download the French model.
    let frModel = TranslateRemoteModel.translateRemoteModel(language: .fr)

    // Keep a reference to the download progress so you can check that the model
    // is available before you use it.
    let progress = ModelManager.modelManager().download(
        frModel,
        conditions: ModelDownloadConditions(
            allowsCellularAccess: false,
            allowsBackgroundDownloading: true
        )
    )
  }
  }

The error is:
2020-06-24 13:19:14.870567+0300 SwiftTranslation[4413:87899] 6.25.0 - [Firebase/Analytics][I-ACS800023] No pending snapshot to activate. SDK name: app_measurement
2020-06-24 13:19:14.949302+0300 SwiftTranslation[4413:87899] 6.25.0 - [Firebase/Analytics][I-ACS800023] No pending snapshot to activate. SDK name: app_measurement
2020-06-24 13:19:14.956355+0300 SwiftTranslation[4413:87876] 6.25.0 - [Firebase/Analytics][I-ACS023012] Analytics collection enabled
2020-06-24 13:19:16.652538+0300 SwiftTranslation[4413:87899] -[FIRTranslateModelLoader modelDownloader:didCompleteDownloadWithModelPath:modelInfo:error:completion:]: unrecognized selector sent to instance 0x600001a65fc0
2020-06-24 13:19:16.666737+0300 SwiftTranslation[4413:87899] *** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[FIRTranslateModelLoader modelDownloader:didCompleteDownloadWithModelPath:modelInfo:error:completion:]: unrecognized selector sent to instance 0x600001a65fc0'
*** First throw call stack:
(
	0   CoreFoundation                      0x00007fff23e3cf0e __exceptionPreprocess + 350
	1   libobjc.A.dylib                     0x00007fff50ba89b2 objc_exception_throw + 48
	2   CoreFoundation                      0x00007fff23e5dc34 -[NSObject(NSObject) doesNotRecognizeSelector:] + 132
	3   CoreFoundation                      0x00007fff23e4190c ___forwarding___ + 1436
	4   CoreFoundation                      0x00007fff23e43bf8 _CF_forwarding_prep_0 + 120
	5   SwiftTranslation                  0x00000001032e7185 -[FIRModelDownloader notifyModelDownloadCompletedWithModelPath:modelInfo:error:] + 209
	6   SwiftTranslation                  0x00000001032e5b83 -[FIRModelDownloader downloadModelIfNeededWithConditions:modelInfo:] + 272
	7   SwiftTranslation                  0x00000001032e4817 __56-[FIRModelDownloader downloadRemoteModelWithConditions:]_block_invoke_2 + 426
	8   libdispatch.dylib                   0x0000000105c68f11 _dispatch_call_block_and_release + 12
	9   libdispatch.dylib                   0x0000000105c69e8e _dispatch_client_callout + 8
	10  libdispatch.dylib                   0x0000000105c706fd _dispatch_lane_serial_drain + 788
	11  libdispatch.dylib                   0x0000000105c7128f _dispatch_lane_invoke + 422
	12  libdispatch.dylib                   0x0000000105c7cb65 _dispatch_workloop_worker_thread + 719
	13  libsystem_pthread.dylib             0x00007fff51c08a3d _pthread_wqthread + 290
	14  libsystem_pthread.dylib             0x00007fff51c07b77 start_wqthread + 15
)
libc++abi.dylib: terminating with uncaught exception of type NSException

Please help to understand what I'm doing wrong here.

How are you installing Firebase? Have you added the -ObjC linker flag to your build settings?

@morganchen12, cocoapods, yes, that flag exists.

Can you share your project?

@morganchen12, works. Thank you

Hi @morganchen12 I am struggling with the same issue, how did you solve it ? thanks !

Hi @AlexandreHosek, I disintegrated many times the dependencies and after it worked.

Hello @mihailsalari can you explain in detail that you disintegrated many times the dependencies


policy

Hello
How can i define policy for each portname?
Thanks

You mean different policy for each port? If yes, unfortunately it is not supported at the moment.
If you describe your usecase in more details, maybe I will suggest some solution. What do you want to achieve?

thanks for your reply
i wanna limit connections for each portname
thanks

number of connections? Or tls domains?
more details you provide, more chance I will be able to help.

number of connections
thanks

Well, if you need to limit only number of connections to a port and don't need to limit by client's IP etc, you can run this:
/opt/mtp_proxy/bin eval 'ranch:set_max_connections(port_name, 12345)'

replace port_name with port you need.
But it will be reset after restart.
Unfortunately, there is no better solution right now.


Bootstrapping layouts of windows

One of the key parts of term tool for me is ability to quickly bootstrap project environment.
Eg:

send object w/ params to the scaffolder script
create N windows
set cwd of each window
run custom command in each window

Here is the demo what I do w/ iTerm and its Apple Script API: https://cl.ly/foOf
Looks like currently w/ HyperTerm API I can accomplish only #2 — create N windows. Eg via shortcut:
const { globalShortcut } = require('electron');

exports.onApp = (app) => {
  [1, 2, 3, 4].forEach(n => {
    globalShortcut.register(`Cmd+Control+Alt+${n}`, () => {
      for (let i = 0; i < n; i++) app.createWindow();
    });
  });
};

Here is API sketch.
// Via some channel sending signal
// w/ serializable params to Hyperterm API
exports.onSomeSignal = (app, params) => {
  const { termOne, ...otherTerms } = params;

  app.createWindow(win => {
    win.setCwd(termOne.cwd);
    win.sendCmd(termOne.cmd);
  });

  // etc...
};

Have no idea if it's doable, or maybe there're other plans for such automation tooling for the future releases. Thoughts / directions please :)

@alexfedoseev Nice background I have the same ;) I would see this as a plugin that interpret your JSON file and process to create the environment once the app is started. Could you edit your post since #2 point to a closed Issue and not suitable for this concern?

@ppot Oops, done :)

once the app is started.

I don't think that it's a good idea, since most of devs have multiple projects to start. Here's the general usecase for me: I start my main project, after sometime I have to change something in my oss module -> I switch to next desktop -> scaffold module's project, etc. So it should be something on demand, rather than on app start, I guess.

@alexfedoseev So somme kind of hterm-cli  that execute your script?
Something like hterm  exec ~/yourprojectpath/env.json ?

@ppot Here are my thoughts on this take. From this line:
hterm exec ~/yourprojectpath/env.json
I assume that hterm-cli is already a layout engine, which takes json config and bootstrap layouts. In the end it sounds fine (except I wouldn't put that in the core, but stand alone plugin), but as a first step I think Hyperterm should expose some low level API, around which such engine can be built. As an example I can point to the same old iTerm and itermocil. iTerm provides unopinionated low-level AppleScript API which gives ability to spawn the window, pass command to this window and so on. Around this API was built opinionated itermocil tool — w/ cli, its format of config etc. This decoupling of low level API and higher level abstractions is more flexible, and it will work for the cases when some devs will want to do something very special, which is not possible w/ higher level tool or even build their own engine (for example I don't use itermocil and use AppleScript API directly, b/c of some limitations/issues of this tool).

@alexfedoseev hterm-cli is just theoretical. But I understand the need to pass execution settings interpreted as action to the terminal. Having a low level API would definitely be great. It definitely need an update to let plugins pass command script directly to the term. This require some work but can be done yes.

ps: What is your shell theme?

@ppot I don't remember where it came from, but here's hyperterm colors:
colors: {
  black: '#383d43',
  red: '#c66363',
  green: '#c0c86d',
  yellow: '#eac171',
  blue: '#80a2be',
  magenta: '#b193ba',
  cyan: '#90c9c1',
  white: '#c2c5c3',
  lightBlack: '#636363',
  lightRed: '#a04041',
  lightGreen: '#8b9440',
  lightYellow: '#ec9c62',
  lightBlue: '#5d7f9a',
  lightMagenta: '#82658c',
  lightCyan: '#5e8d87',
  lightWhite: '#6d757d'
}

@alexfedoseev, @ppot I stumbled across hyperterm while looking for some kind of easy automation support for bootstraping development project environments. Is this sort of development still on the radar?
@alexfedoseev, are you able to share you iTerm script that you used to get that working?

@alexfedoseev, have you come across https://www.npmjs.com/package/hyperlayout?

@alexfedoseev, have you come across https://www.npmjs.com/package/hyperlayout?

@smebberson Yep, I've seen this one, but I miss tmuxinator API, it's the best scaffolding tool I used so far. I might just copy it some other day.
You can find my scaffolding scripts for iTerm here: https://github.com/alexfedoseev/dotfiles
But I'm afraid it won't be obvious how things work together as it's a crazy mix of iTerm, Alfred & kwm scripts.

@alexfedoseev yeah, I agree, it would nice to have similar capabilities. I did discover your dotfiles and yeah, it was a crazy mix. I think I mostly understood, but I think I'll opt for something simple for now and go with hyper and hyperlayout. It's in between what I'm doing now and the full thing you can get iTerm/tmuxinator.
I'm really loving hyper so far!


Vagrant cachier uses default sync folder type, which is not available

When running vagrant up with vagrant cachier installed, I get this error:

Vagrant was unable to mount VirtualBox shared folders. This is usually
because the filesystem "vboxsf" is not available. This filesystem is
made available via the VirtualBox Guest Additions and kernel module.
Please verify that these guest additions are properly installed in the
guest. This is not a bug in Vagrant and is usually caused by a faulty
Vagrant box. For context, the command attempted was:
mount -t vboxsf -o uid=900,gid=900 vagrant-cache /tmp/vagrant-cache
The error output from the command was:
/sbin/mount.vboxsf: mounting failed with the error: No such device

I am using geerlingguy/ubuntu1404 box and apparently it does not have VirtualBox guest extensions installed (I get a message saying that earlier as well). I can live with that if cachier can use a different sharing type. Pull Request coming up with changes for that.

@hussainweb - I'm guessing you have the 1.0.3 (I think? maybe 1.0.4) version of that box, which has outdated guest additions. You should also make sure to install the vbguest plugin too: vagrant plugin install vagrant-vbguest, so the guest additions are always up to date.
I just published a new version of the box today, with Guest Additions for 5.1.8.
Closing this issue, will track the PR to still allow the customization to be set for cachier (see my comment there).

I have the latest version of that box (1.1.3). I have used boxes with outdated guest extensions before and sharing still works. Here, it says the guest extensions are entirely missing. Maybe it was an older version of your box itself because the error just appeared recently, after I updated the boxes.
Thanks for the tip on plugin. I'll check it out.

I checked the plugin and see that it installs VB Guest additions if not present. I guess you removed Guest additions from your boxes and relied on the plugin to install it. I remember not seeing this error earlier (but that could also be as I was not using cachier earlier, which means everything was on NFS).
Was this plugin required to use DrupalVM? I didn't see any documentation suggesting this.

I guess you removed Guest additions from your boxes and relied on the plugin to install it.

In this case, the boxes were just a couple months out of date, and had 5.0.24 guest additions—because of a myriad of problems with Vagrant 1.8.4/5 and VirtualBox 5.1.x, I had held off updating the boxes until things were stale.
So the additions are/were present, they were just out of date. And in that case, Vagrant and VirtualBox don't always get along unless you have the vbguest plugin automatically installing the correct version of additions :)

For the record, VB Guest additions are present on geerlingguy/ubuntu1604 box but not on geerlingguy/ubuntu1404. It is 5.0.24 guest additions (on geerlingguy/ubuntu1604) but it works fine. I remember it worked with guest additions 4.x as well (some other box). I remember seeing the warning but it worked. That is why I am so sure that the box is missing the guest additions somehow because I get an error (unless I set it to NFS). I have never faced issues with mismatched VBox and guest addition versions, even differences in major versions.
I haven't tried the vbguest plugin yet, will do soon. In any case, the PR will help in enabling NFS which is useful regardless of default sharing.

@hussainweb - Weird... I must've pushed a bad box build last time. Can you also try running vagrant box update and see if the new version that's available has the additions? I just checked yesterday and it seems they were present.

I can confirm that 1604 does. I will try to create a new VM off the new version of image to check on 1404.
Also, I made the changes you requested on the PR. It is ready for you review. :)

@hussainweb - For some strange reason, the PR has disappeared... it's still in the list here (https://github.com/geerlingguy/drupal-vm/pulls), but I get a 404 when I visit https://github.com/geerlingguy/drupal-vm/pull/951 — did you maybe delete the branch?

That's weird. I think it is a Github issue. All PRs are giving a 500 right now. The branch is there at https://github.com/hussainweb/drupal-vm but gives a 500 when I try to switch to the branch in Github.

@geerlingguy, the PR is visible now.

Merged!


How to debug connection issues

Hi,
I am making a client using the following code
    client = TrainerClient(
        "actor_env",
        FLAGS.ip,
        make_trainer_config(),
        data_store,
        wait_for_server=True,
    )

But I am stuck waiting for server
W0314 20:27:43.566129<PHONE_NUMBER>65952 trainer.py:178] Failed to connect to server, retrying...
W0314 20:27:46.370609<PHONE_NUMBER>65952 req_rep.py:130] Failed to send message: Resource temporarily unavailable

How can I debug this problem?

I don't have enough information to debug this.
Did you run the server? Are both Server and Client sharing the same TrainerConfig? Is the ip address used by the server ping-able by the client terminal?

I am running SERL, but modified it to work for XArm robot. I am posting in agentlace and not SERL repo because I though it was more relevant here.
I am trying to replicate the peg insertion task. The learner (policy) is a TrainerServer and the actor is a TrainerClient.
Both are using the same trainer config builder [1, 2], which is making config here. My IP is set to 'localhost'. I am able to ping from actor (TrainerClient) terminal to learner (TrainerServer) by running ping localhost.

Thanks for trying out SERL. Would check if the port is working too
Also just a sanity check, i would run the script below to make sure the trainer and client is working fine.
from agentlace.trainer import TrainerServer, TrainerClient, TrainerConfig
from agentlace.data.data_store import QueuedDataStore

config = TrainerConfig(port_number=5488,broadcast_port=5489)

# start server
server = TrainerServer(config)
server.start(threaded=True)
server.register_data_store(
        "actor_env", QueuedDataStore(capacity=5))

# start client
client = TrainerClient(
        "actor_env",
        "localhost",
        config,
        QueuedDataStore(capacity=5),
        wait_for_server=True,
)

print("all passed")

To check the port, I ran netstat -tuln. Two of its output rows are as follows:
Proto Recv-Q Send-Q Local Address           Foreign Address         State
TCP             0           0     <IP_ADDRESS>:5489                         <IP_ADDRESS>:*        LISTEN
TCP             0           0     <IP_ADDRESS>:5488                         <IP_ADDRESS>:*        LISTEN

It looks like the ports are working.
Also, the script that you sent works, it printed "all passed".

Hmm, then this seems to be a different issue not related to agentlace. Feel free to open an issue on https://github.com/rail-berkeley/serl and add the printout logs from the learner and actor processes. Thanks


My test_net got all 0 as result

❓ Questions and Help
I used the coco_2017_val dataset from coco website, and I also changed the param in paths_catalog.py.
But when I run python tools/test_net.py  --config-file "configs/e2e_mask_rcnn_R_50_FPN_1x.yaml",
the reult AP for bbox and seg are all 0.
Do you have any idea?

I checked my inference/bbox.json and segm.json, and find the scores predicted for bounding boxes are very low, like around 0.05.
But I cannot see what's wrong in this process, do I need to change  the test_net.py?

I made a really silly mistake. The config-file should be "configs/caffe2/e2e_mask_rcnn_R_50_FPN_1x_caffe2.yaml"  instead. I will close issue.

Great that you found the issue!
Indeed, the configs/e2e_mask_rcnn_R_50_FPN_1x.yaml file has not been pre-trained for detection on COCO, and is meant to be used for training.


Update UserGuide summary to include blacklist, ban and un…

…ban commands

Coverage remained the same at 92.679% when pulling 000780a19ccf063bee7253bc1ee0886cf1e31ada on jaivigneshvenugopal:update_user_guide_summary into c84ac1125afc3ed8c1045c44b058543d65b244c1 on CS2103AUG2017-T17-B1:master.


Remove lights from Anymal robot

I don't really get the rationale, but #8 removed the lights from ur and was accepted, so I guess we should also accept this here.

The idea is to remove lights that are not welcome when one's perform rendering of the scene
There is no reason to have lights in DAE files, except the ones coming from the internal illumination of the scene.

How does the visualization change in GVr?
Could you share a pic?
If I remember correctly, it was badly displayed in GV without the lights.

For reference, there is a gui.removeLightSources in GV that remove all lights from a node.

How does the visualization change in GV?
Could you share a pic?
If I remember correctly, it was badly displayed in GV without the lights.

If you want a better illumination in GV, you need to add an omnidirectional light on top of the scene. But the lights should not be embedded inside the meshes. Otherwise, when the robot moves, the lighting changes too.

For reference, there is a gui.removeLightSources in GV that removes all lights from a node.

Thanks for the hint.


fix: enhance category list resource performances

Introduce a projection filter in API repository in order to minimize
data transfered over the network and improve performances.
see https://github.com/gravitee-io/issues/issues/6255
Issue
https://github.com/gravitee-io/issues/issues/XXXXX
Description
A small description of what you did in that PR.
Additional context

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
Congrats, all the CI steps were 🟢 . The Bot was able to deploy the build of this PR, so you can now try it directly here! 🚀
Notes: The deployed app is linked to the management API of the Element Zero team's environment.

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
Congrats, all the CI steps were 🟢 . The Bot was able to deploy the build of this PR, so you can now try it directly here! 🚀
Notes: The deployed app is linked to the management API of the Element Zero team's environment.


md-sidenav containing md-list and md-tab height issue

Bug, feature request, or proposal:
If in a md-sidenav you put a md-list with an item containing a md-tab-group, when the content of the tab is too long the tabs are moved up above other components.
What is the expected behavior?
The tabs should stay in place and their content should grow downards.
What is the current behavior?
The tabs are pushed up, above other components. The longer the content, the higher they go.
What are the steps to reproduce?
open this plunker
https://plnkr.co/edit/Uo9B6E1ccVpVBT6vB2Nv?p=preview
Click on open to open the sidenav then click on tab1 and tab2 to see how the different content of the tabs pushes up the tab-group. The tabs should always be under the heart icon.
What is the use-case or motivation for changing an existing behavior?
Which versions of Angular, Material, OS, browsers are affected?
@angular 2.2.4
@angular/material 2.0.0-alpha.11-3

Chrome 55
Is there anything else we should know?

List items really aren't meant to contain tabs. This issue is probably obsolete now, but I'd recommend structuring your content differently.


docs.rs

I am requesting transfer to another maintainer, please read and comment here: https://github.com/dylanede/rusttype/issues/55


[Plugin] Metadata Toolkit

Hey! :)
I'm working on a plugin to use the simplify and enhance Metadata use.
Briefly, the plugin does as follow:

In the Kanboard settings page there is a view to create Metadata types. You can specify a friendly name, if it's required, the entity to attach it and the type (text, integer, boolean...)
When a user creates a new entity (project, user or task), he will see  the additional metadata created by the admin as new form field. (The code then validates the user input based on the metadata type declared)
This new data will be shown in the project/user/task base as additional info.

https://github.com/Ryuk87/kanboard-metadata-toolkit/tree/develop
Currently I'm working in the "develop" tree so master is pretty empty :)
I think I will be able to release a first "beta" version during the week.
If you think you could need additional feature, feel free to open an issue directly in the repo.
Every idea is really appreciated :)

Hmmm thats what I wanted with https://github.com/BlueTeck/kanboard_plugin_metadata too. Maybe we should work on one plugin instead of two ;)

@BlueTeck sorry, I did not saw your plugin :) Sure, I can integrate my logic in you code if that's ok!

@Ryuk87 sounds good, the modell controller etc could coexists at the moment, only the Plugin.php has to be merged.

@BlueTeck Thanks! :)

Merging into #2032


Automagic flush when take()ing a MemorySinkStorage

Fixes #2627
Should supersede #2630
Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[x] I've included a screenshot or gif (if applicable)
[x] I have tested demo.rerun.io (if applicable)

PR Build Summary
Docs preview
Examples preview

The added logging in https://github.com/rerun-io/rerun/pull/2630 still makes sense to me, as well as any logging that would catch bugs like this earlier in the future. Dropping user data should be considered very bad.


fix incorrect link

I can't seem to build the website anymore, the rouge highlighting breaks.

I may have broken something in my razetime/rouge fork. It no longer needs to depend on that repository, and the website builds for me with base rouge, ruby 3.1.4 after bundle install && bundle update.
A few sassc deprecation warnings will appear, which should be fixable by using it as a "theme gem".


Link not opening when tapping on title in specific feed

App Store Connect report:
“Source link not opening for articles when tap on title. Tested with: http://www.eldiadevalladolid.com/rss/DVlocal.xml”

This feed isn't a valid RSS feed. Presumably most pertinently, though the validator doens't point this out, the contents of the <link> tags are wrapped in <![CDATA[ markup—which is only intended for use in <description> content, I believe?
If so, this is probably NNW displaying works-as-intended behavior, as the feed doesn't contain valid URLs?
It would be possible, of course, to strip the CDATA markup from around links... would that actually be desirable?

There is no url being stored for each article in the database for this feed when it's a local account. (When it's coming via Feedbin it seems ok.)

Right. It is probably a parser bug.


adding a completion block for removing a single cached item

Thoughts on doing something like this?
Main reason is that busting the cache for a single item was making my KIF tests fail sporadically
because sometimes the cache wouldn't bust in time for the next web request.
Can write a test if interested.

Looks good, I'll have a look around the API if there are other places a completion closure would be nice to have. Did you spot any?

I am closing this in favor of [3.0: Sync by Default].(https://github.com/aschuch/AwesomeCache/releases/tag/3.0)


Buildings overlap roads

Mirroring a comment from the OSM mailing list below.

From C. H.
A few observations from looking over the map there:

There are quite a few cases where gaps between buildings are either
very small (significantly less than a meter) or buildings touch at a
single node.  Both are usually artefacts (probably primarily due to
roof based rather than footprint based outlines).
There seem to be a lot of cases where garages and sheds are tagged as
building=house or building=residential - like here:

https://www.openstreetmap.org/way/409191451

In some cases boundaries of pre-existing landuse polygons (like
landuse=residential/landuse=industrial) and roads now intersect
buildings indicating that mapping is inaccurate - like here:

http://www.openstreetmap.org/#map=17/34.07461/-118.48376
All of these problems could probably be turned into maproulette tasks or
be otherwise worked into QA tools - although at least the first two
will require on-the-ground assessment for proper correction.

My comments are that the first issue with the sheds is something that is probably an artifact of the conflation (#67) but I haven't been following closely enough to say.
For the second bit, the overlapping roads: I think that when bad data is identified by an import is justification enough to even do the import! In this case building footprints have identified wonky TIGER streets. However, from a workflow perspective, I would think this might be caught in the validation phase (in JOSM), but the JOSM validator doesn't always catch everything.

@talllguy
Great comments! I'll look into this a bit more.
We've created a testing MapRoulette challenge to address unreviewed roads in a small section north of L.A. city. http://maproulette.org/map/237
But we need to address those other things as well.

Some roads in the north county have different names than what county has them identified

There are quite a few cases where gaps between buildings are either very small (significantly less than a meter) or buildings touch at a single node.

These can be identified. What should be done with these? Should they be morphed so they don't touch?

roads now intersect buildings indicating that mapping is inaccurate

These too can be identified. The part of the building that touches the road can be shrinked away from
the road. Would this solve the issue?

@wsdookadr could you provide some examples?
A lot of buildings in the county do touch, so it's fine if they share a node. Overlapping buildings isn't preferred though.
And some of the road data is still quite bad for the county. So it may be that the road is still based on the 2007 Tiger Line file import. We've got a MapRoulette challenge for roads in a corner of the county.
But if you have some specific examples we can figure out where the problem is. Thanks!


docs: Add ksqlDB guide

Add a Helm chart setup guide for connecting ksqlDB to Kafbat UI, as the initial setup lacks a ksqlDB connection guide for operators.

@Haarolean Hi. Thanks for the detailed review. I've made all the changes you asked for.


raidboss: More r1s triggers/fixes

Clean up map tiles to only the ones needed
Fix issues with headmarkers and stack/spread callouts (sorry)
Add triggers for Leaping  mechanics
Fix initial middle sync to have a window to combat early pullers
Add Shockwave knockback trigger

Nice! I was working on a PR to fix the headmarkers for nailchipper, but this looks like it will fix it for me. I'll test this out when I get a chance.


SST CLI: add option to pass custom sst.json as parameter

I am currently trying to deploy the same project multiple times automatically using a workflow with different parameters.
These deployments would need to be under different names but the cli does not allow to change the name used.
My approach was to run a script before executing sst build to swap out the sst.json file, however, it would be much cleaner to just pass a custom sst.json when running sst build and sst deploy.

Can you explain the use case a bit more? Typically people deploy to multiple stages and that handles this.

In my case I deploy multiple staging and production environments whith different application names.
The proposed solution to add the project name to the stage would work obviously but it does seem a bit strange to me to combine these two together.
The catch with this approach is if i want to access the application name, for examle, to define domains, etc in a stack I would always have to access the stage instead of the name.
There would be multiple approaches to solve this.
Either allow a --name parameter.
Eg:
sst deploy --stage staging --name projectA
sst deploy --stage production --name projectA

sst deploy --stage staging --name projectB
sst deploy --stage production --name projectB

Or allow to specify a custom sst.json in which name, region, etc... can be specified.
Eg: sst deploy --json custom-sst.json
Currently i solved the problem by simply swapping out the sst.json file on deployments using mv custom-sst.json sst.json before building and deploying, but it would be nice if sst could simply handle passing a custom sst.json out of the box.

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


Update Get-DtcClusterTMMapping.md

Description has been updated now.

Docs Build status updates of commit 6d7be87:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2012r2-ps/msdtc/Get-DtcClusterTMMapping.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Replacement of the PR #1378. No further review necessary.


allow URL object on c.redirect()

The author should do the following, if applicable

[x] Add tests
[x] Run tests
[x] bun run format:fix && bun run lint:fix to format the code
[ ] Add TSDoc/JSDoc to document the code

Format check is not complete.
Do you have different versions of prettier installed globally?

Hi @cometkim
I think this is a good improvement. Looks good to me. Thanks!


Multiplicative Inverse using Extended Euclidean Algorithm

Description
This will output Multiplicative Inverse of a number let say p on plane Zq. Here the main function of code will accept p and q and input.
Important Checklist

[x] I have read the contribution guidelines before making this issue.
[x] I have verified this issue program is not already implemented
[x] I have not pushed this code anywhere else (even my repo)

Languages and Assignees:

[ ] Python: @
[ ] Java: @
[ ] C: @
[ ] C++: @

@Addhyay Which language would you prefer? You can choose only one

@kaustubhgupta  I'll do it in python

I can do this in Java


Shaky foundations for generalized recursion schemes

Hi! I wanted to let you know about a subtle but pretty fundamental bug we've found in Haskell's recursion-schemes library. Since Matryoshka uses the same "recursion-schemes from comonads" approach as we do, and exposes the same gcata, distZygoT, and histHisto building blocks as we do, I'm pretty sure the bug affects Matryoshka as well.
Now, the bug is pretty subtle. Only zygomorphic histomorphisms and their generalizations (such as the infamous zygo-histomorphic prepromorphism) seem to be affected, and even then, they only compute an incorrect answer when the input tree's depth is at least 3. Another symptom is that histomorphisms do more work when implemented via gcata and distHisto than when implemented by hand. That's the symptom I noticed first, but be sure to read further down, it really is a correctness issue, not just a performance issue.
In our Haskell implementation, we have decided to move away from comonads, but that's a pretty big change, so you might consider the following slightly less drastic but still painful change: remove gzygo and distZygoT from the library. zygo and distZygo are still fine. Manually reimplementing histo might be a good idea too. Keep in mind, however, that these are merely the symptoms I have noticed; there might be more.
Anyway, I wrote a lot more about the problem in the upstream issue, but since the problem is pretty subtle, let me know if you have any question.

Speaking of bugs in the Haskell library which also affect Matryoshka: your ghisto implementation seems to have the same bug as ours did: https://github.com/ekmett/recursion-schemes/issues/48
Thankfully, fixing that bug is much easier than the foundational issues above :)


ndef-util: handle UTF-8 encoded special characters

Before, characters like "à" couldn't be parsed and they were falling back to "a" (another example: §)

Copied my old commit :)

@NicholasBertazzon Thanks!


feat: basic ws chat example

The code LGTM and I don't see any unexpected re-rendering. I've left some comments about possible causes for the re-rendering, but your current solution seems fine.


Error when using with pdf.js

I am trying to create a canvas with the contents of a pdf file page using pdf.js with the code below:
   const { createCanvas } = require('@napi-rs/canvas')
   var PDFJS = require("pdfjs-dist/legacy/build/pdf.js");
  const r = await getObject("nmm-imgpro-pdfs", `automark/${taskid}/${filename}`);
    const typedArray = new Uint8Array(r.Body);
    const pdf = await PDFJS.getDocument({ data: typedArray }).promise;
    const dataUrl = await new Promise((resolve, reject) => {
        pdf.getPage(page).then((page) => {
            const viewport = page.getViewport({ scale: 1.5 });
            const canvas = createCanvas(viewport.width, viewport.height);
            const canvasContext = canvas.getContext("2d");
            page.render({ canvasContext, viewport }).promise.then(() => {
                const dataUrl = canvas.toDataURL("image/jpg", 1);
                return resolve(dataUrl);
            });
        });
    })

The page.render({ canvasContext, viewport }).promise seems to be causing the problem producing the error below:
error - unhandledRejection: TypeError: Cannot read property 'width' of undefined
    at CanvasGraphics.beginDrawing (/Users/brianhenderson/Documents/Documents/Repositories/automark/node_modules/pdfjs-dist/legacy/build/pdf.js:14365:37)
    at InternalRenderTask.initializeGraphics (/Users/brianhenderson/Documents/Documents/Repositories/automark/node_modules/pdfjs-dist/legacy/build/pdf.js:12547:18)
    at /Users/brianhenderson/Documents/Documents/Repositories/automark/node_modules/pdfjs-dist/legacy/build/pdf.js:10705:28
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)

I am using NextJS with this code being in a server side function.  Any help would be greatly appreciated.

At its root, this seems to be because the context object returned by canvas.getContext("2d") has an undefined value for canvas on it, despite typings suggesting it should exist.
I tried forcing it like so, before passing it to pdf.js:
    // @ts-ignore
    context.canvas = canvas;

...but then I get a new error:
Error processing lesson 1964 (golden-age-of-china-3b08d85d40c8): TypeError: ctx.getTransform(...).invertSelf is not a function
    at getCurrentTransformInverse (webpack://pdfjs-dist/build/pdf/src/display/display_utils.js:620:51)
    at CanvasGraphics.paintInlineImageXObject (webpack://pdfjs-dist/build/pdf/src/display/canvas.js:3053:7)
    at CanvasGraphics.apply [as paintImageXObject] (webpack://pdfjs-dist/build/pdf/src/display/canvas.js:2993:10)
    at CanvasGraphics.executeOperatorList (webpack://pdfjs-dist/build/pdf/src/display/canvas.js:1231:20)
    at InternalRenderTask._next (webpack://pdfjs-dist/build/pdf/src/display/api.js:3423:37)
    at processTicksAndRejections (internal/process/task_queues.js:95:5)

This new error may be particular to the PDF I'm testing with, and a matter of not all functions being supported yet.

Error: Value is non of these types CanvasElement, SVGCanvas, Image,
https://github.com/mozilla/pdf.js/blob/529906c74f1a53198200be7f61e63f8f94d1a371/src/display/canvas.js#L261


make jfeedly Android compatible

I need to do something about BrowserFrame.java which is probably the only thing which makes this library "desktop only".

@kasoki Is this project still ongoing?

Nope


Failed to resolve component: RouterLink

Any usage of NuxtLink lead to an error
[Vue warn]: Failed to resolve component: RouterLink
If this is a native custom element, make sure to exclude it from component resolution via compilerOptions.isCustomElement.
  at <NuxtLink to= { name: 'index', params: { userId: '123' } } class="text-neutral-501" onClick=fn<onLinkClick> >
  at <UserLink userId="123" ref="VTU_COMPONENT" >
  at <VTUROOT>

Reproduction

I've having the same issue

Hey :wave:
This isn't working because you're using mount from @nuxt/test-utils. This is a completely separate package and you should not mix them like this. What you want to be using is mountSuspended from nuxt-vitest/utils :+1:
So if we adjust your test it would look something like:
import { mountSuspended } from 'nuxt-vitest/utils';
import { describe, it, expect } from 'vitest';
import UserLink from<EMAIL_ADDRESS>
describe('UserLink', () => {
  it('Component should render', async () => {
    const wrapper = await mountSuspended(UserLink, {
      props: { userId: '123' },
    });

    console.log(wrapper.html());

    expect(wrapper.element.tagName).toBe('A');
  });
});

Hope that helps :smile:

Sorry I forgot to mention that my tests based on @testing-library/vue, I updated Reproduction, as you can see from console output Test component was rendered which means nuxt environment bootstrapped and works but RouterLink still not working correctly.

Using anything else other than mountSuspended is not going to work. To use testing-library you need a custom wrapper.
If this is something you want please upvote this issue that I made 😄


Create ODIM profiles to be exposed to NB users

create profiles for ODIM
As Schema definitions for the Redfish Scalable Platforms Management API ("Redfish") are designed to
provide significant flexibility, and allow conforming implementations on a wide variety of products, very
few properties within the Schemas are required by the Redfish specification. But consumers and software
developers need a more rigidly defined set of required properties (features) in order to accomplish
management tasks. This set allows users to compare implementations, specify needs to vendors, and
allows software to rely on the availability of data. To provide that "common ground", a Redfish
Interoperability Profile allows the definition of a set of schemas and property requirements, which meet
the needs of a particular class of product or service.

duplicate of https://github.com/ODIM-Project/ODIM/issues/249


Render Targets

[x] Multiple render targets
[x] Off screen [multiple] framebuffers
[ ] Render target manager
[ ] Framebuffer manager

By starting now, texture units are managed by library and they are being hidden from public api.
Maybe to supporting this idea and make it more robust we could create GkTexture or something like GkTextureProxy for render targets to continue hide texture unit management from public.


[NEW]: German ID Wallet app stopped after 1 day of public availability

Date
2021-09-24
Description
Shortly before the federal election, the German government launched the app "ID Wallet". It was supposed to store driver's licenses and other identification documents, and allow them to be shared with authorized parties (like the police, or during hotel check-ins). Because the distributed ledger back-end met neither basic EU security standards, nor handled more than a few thousand users (in total, not per second), the launch failed and private data stored in the app would have been exposed to identity theft. FOIA requests revealed, that the project developers had known about the shortcomings of their design months in advance.
Links
https://marketresearchtelecast.com/id-wallet-the-german-government-had-long-known-about-it-security-vulnerabilities/190919/
https://netzpolitik.org/2021/interview-zu-id-wallet-konzeptionell-kaputt-und-ein-riesiger-rueckschritt/
Agreement

[X] I irrevocably agree to release my contribution under the CC-BY 3.0 License

Attribution name
No response
Personal link
No response

Added, thank you!


ARTEMIS-4382 Long Time to process export / import

I'm inducing errors on the CI to validate the changes. hence a draft.


can i create BOT for whatsapp ?

I saw on YouTube that the baileys repository can also be used to make BOT, is it possible to make BOT in this repo?

can BOT implementation be combined with this repo?

Yes, it's open source, just fork it and edit it, or use as intended consuming the API provided.

Yes, it's open source, just fork it and edit it, or use as intended consuming the API provided.

how?

okay, thanks everyone for helping. I've found how to make BOT.
simply by changing the contents of the wa.ts file then rebuild.


Dropping attachments in FilePond from Thunderbird (Firefox)

I'm trying to drop attachments from Thunderbird but in Firefox it doesn't work.
With Google Chrome it works well.
Chrome 81
Firefox 76
Thunderbird 68
Example.zip

Hi, this is not the correct repository, drag dropping from other applications isn't very well supported by browsers at this point.


2.4 CRUD routes for server.js

Need establish basic crud routes for server.js
To include:
GET all posts
GET all comments
GET my comments
GET post by id
GET comments by post_id
POST post
POST comment
POST drafted post
PUT drafted post (?)
PUT accepted comments (?)
(additional ones TBD or will create a new issue)

@drcox23,
Just to clarify, was this meant to be posted to the projects board?
If so, I can update the projects board to represent this enhancement and track it from there- mainly to avoid ubiquity.
Chaz

Need establish basic crud routes for server.js
To include:

[ ] GET all posts
[ ] GET all comments
[ ] GET my comments
[ ] GET post by id
[ ] GET comments by post_id
[ ] POST post
[ ] POST comment
[ ] POST drafted post
[ ] PUT drafted post (?)
[ ] PUT accepted comments (?)
(additional ones TBD or will create a new issue)

Update: We will be working with issue cards moving to the projects board, and track using milestones and labels. Thanks @drcox23

Thanks @drcox23 - this issue is now closed

[ ] Initial post save
[ ] Initial comment save


Hide Google API key

You've already included local.properties to the .gitignore file; simply add the api key in your local.properties.
Your API key should always be kept private to prevent misuse.
Anyone who clones this repository should use their own personal API key.

Well yes it makes sense to generally hide API keys, but problem is there isn't any good way (or maybe any way at all to hide static API keys inside the Android application), cause even if i use build config to hide it, one can just decompile the app and see the API key if they wanted anyway, even if you enable proguard/R8 it won't change the API key string or encrypt it in in any form whatsoever. Besides i kept API key public cause i had plans to publish the app on F-droid (which is now done btw) and as you may know F-droid builds the app from source instead of using apk from GitHub releases, so without making API key it wasn't really possible :)
I also had this exact same discussion previously on reddit if you're interested you can check that out here.
Besides you can actually limit who can makes calls using your API key by adding your signature which you use to sign the apk file in "allowed requesters"; (dunno better word for that) in Google cloud console. And since nowadays fdroid supports reproducible builds i.e. the fdroid version of app will have same signature as apk released by me, it's a win win situation cause the said API key will work for both while still being public.

Anyway thanks for taking your time to contribute :)

No matter what we do, if the key is in the app, it can still be extracted. yet it wasn't the intended purpose; it was simply for making it available on GitHub which will make it vulnerable to bots or scrapers.
I was unaware that you would post it on F-Droid, however you could at least obscure it by using any encryption technique or use NDK/JNI, However, since Google Console already provides SHA-1 verification, you don't actually need all that :)
thx


chore: test-python-jenkinsx to 0.0.3

chore: Promote test-python-jenkinsx to version 0.0.3

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Connect to local Model

Hi,
Will I be able to connect to my local Model?
Did you release the plugin to the marketplace? I can't find it.
Thanks,
Stasik

Hello, yes you will, provided the model is behind an API supported by the plugin.
I've released the extension to the marketplace, I am still waiting for JetBrains' approval.

FYI it's available on the marketplace :)


Doesn't convert to .egg

I have the latest version of Songe Converter yet my songs remain as .ogg files. Any help please? All other files get converted (.json to .dat).

Have you tried trying again? Does it always happen with the same songs?

Closing due to inactivity.


⚠ TechNews 科技新報 has degraded performance

In c6e0c44, TechNews 科技新報 (https://technews.tw/) experienced degraded performance:

HTTP code: 200
Response time: 1029 ms

Resolved: TechNews 科技新報 performance has improved in 293e413.


[UI] memory leak warning printed to the console

Problem
Right after starting the application:
Download the React DevTools for a better development experience: https://reactjs.org/link/react-devtools
main.cljs:34 Starting v1.0.3 ...
main.cljs:35 Config {:router {:routes [["/" :route/splash] ["/home" :route/home] ["/new-analysis" :route/new-analysis] ["/analysis-results" :route/analysis-results] ["/documentation" :route/documentation] ["/map" :route/map]], :default-route :route/splash, :scroll-top? true, :html5? true}, :analysis-viewer-url "http://localhost:8021", :logging {:level :debug, :console? true, :sentry false}, :graphql {:ws-url "ws://<IP_ADDRESS>:3001/ws", :url "http://<IP_ADDRESS>:3001/api"}, :environment "dev", :google {:client-id "806052757605-5sbubbk9ubj0tq95dp7b58v36tscqv1r.apps.googleusercontent.com", :redirect-uri "http://localhost:8020/?auth=google"}, :root-url "http://localhost:8020", :version "1.0.3", :public-key "-----BEGIN PUBLIC KEY-----\nMFwwDQYJKoZIhvcNAQEBBQADSwAwSAJBAJliLjOIAqGbnjGBM1RJml/l0MHayaRH\ncgEg00O9wBYvoNXrstFSzKTCKtG5MayUKgdG7C/98nu/TEzhvRFjINcCAwEAAQ==\n-----END PUBLIC KEY-----"}
logging.cljs:28 ui.events.general:9 Active page changed cljs.core/PersistentArrayMap [count: 3]
browser.cljs:20 shadow-cljs: #3 ready!
logging.cljs:28 ui.splash.page:58 component-did-mount null

react-dom.development.js:68 Warning: Can't perform a React state update on an unmounted component. This is a no-op, but it indicates a memory leak in your application. To fix, cancel all subscriptions and asynchronous tasks in a useEffect cleanup function.
    at f (http://localhost:8020/js/cljs-runtime/reagent.impl.component.js:756:31)

logging.cljs:28 ui.events.general:9 Active page changed cljs.core/PersistentArrayMap [count: 3]

Few problems spotted on the results-page:

Right hand side of menu is cut, including icons, search box and scroll. Add ellipsis to the text with the name of analysis and display a tooltip with the full name on-hover
Labels not centered
Not the same blueish color as in design

Problem with the "run new analysis" menu item:

This happens when both this and the next (Completed...) items are opened. There are always just 3 items there, no need for a scrollbar - just make it big enough so that they are always in view

Plenty of space on the right to display the full name, delete icon pretty big for that text size
No the same as in designs (and other inputs fields for that matter)
a little more vertical space here?


currentMediaItem is empty when a CastPlayer is being used

I'm using a MediaLibraryService that switches the player from ExoPlayer to CastPlayer when a CastSession is available. When the ExoPlayer is active, calling getCurrentMediaItem() from a MediaController returns the correct MediaItem, but if the same call is done when the CastPlayer is the active one, I receive an empty MediaItem.
The method getWindow(...) inside CastTimeline is the one creating and using an empty MediaItem instead of the expected currentMediaItem.

This is a known issue which becomes more sever with AndroidX Media3 MediaSession. Implementing the Player interface properly is more important when we aim to be able to build a MediaSession with a MediaSession.Builder.
For right now I'm afraid I can't really give a better advice than that you need to maintain your own list of MediaItems that represents the playlist and from where you can lookup the media item by index with the given window index.
Somewhat aside: I think an approach to have a MediaSession with a Player that can change (local vs. remote playback with Cast) would be to have a custom Player implementation that delegates to an actual Player implementation and that allows to swap the actual implementation. It may for instance have a setPlayer(Player) method for swapping the current player with another. This way you can build a MediaSession with a MediaSession.Builder and delegate commands sent by a MediaController to either of the implementations under the hood. If you do that, that custom Player interface would be the best place to maintain the list of MediaItems that is the playlist. It can also help you to work around  #26 that is related to the CastPlayer not properly implementing the Player interface in the current state.
--
Some comments for context when someone is going to fix this (I stick with the bug label because I agree this is now a bug in context of AndroidX Media3):
There is an unwritten contract that a Player should provide an equal MediaItem with Player.getMediaItemAt(int index) that have been passed to the  set/addMediaItem(MediaItem) methods. The same should be the case for Timeline.getWindow(...) as well.
CastPlayer currently fails to do that which we need to improve. At the very least CastPlayer should propagate the mediaId correctly which would be a first step although if we fix this that way we probably should comply with the whole MediaItem because the mediaId is not necessarily unique. There is a commit in the ExoPlayer repo for the dev2 branch which includes the mediaId when un/marshalling which is a first step towards this. CastTimeline is another step as mentioned above in the user's post.

Thank you very much @marcbaechinger for the idea of the custom Player. I was able to fix the empty MediaItem problem with a modified CastTimeline, CastTimelineTracker and a custom MediaItemConverter (like you already said, the default is a bit limited), but your idea is way better than calling MediaSession#setPlayer(...) each time the player changes. By the way, something like that inside the media3-cast would be awesome.
Thanks again!

@spun Thanks for the update. In the meantime i've decided to tackle the problem as well. I am currently working on my own implementation of a custom Player which indeed switches from local to cast internally. It works, but it is tailored to my use case. I don't have a list of items but only playback a single [MediaItem] at a time, so the solution is just to keep a reference to the current MediaItem and that's that. But I am sure i've missed some edge cases. I guess we have to wait.

Fixed by the commits above.

Nice work

Sorry to open this again, but during active play of CastPlayer getCurrentMediaItem() returns the correct MediaItem however when listening to onCastSessionUnavailable() callback and subsequently checking getCurrentMediaItem() is empty again. Is this intentional? Looking at the source it seems that the Timeline is cleared by the CastPlayer before the onCastSessionUnavailable() callback is invoked.
This means we still have to resort to locally caching the MediaItems, no? Or am I missing something?

When the remote media client is set to null the timeline is updated and set to an empty timeline.  I agree that's surprising and unfortunate, specifically that this happens before the callback is called.

we still have to resort to locally caching the MediaItem

I think this is a known behaviour though. CastPlayer is missing abilities to nicely connect/reconnect. To resume playback, the playlist media items need to be set again.  Apps probably mostly maintain the playlist, position, window index and the like to be able to either setup a local ExoPlayer or CastPlayer like PlayerManager in the demo app. UAMP uses a ForwardingPlayer.
However, now that we support the media items in the timeline with CastPlayer, we could and probably should do better. Re-opened to track decoupling cast session and CastPlayer state and keeping timeline, currentMediaIndex and currentPosition when the cast session ends  (probably similar to when CastPlayer.stop() is called). Then we probably also need to think about the behaviour when the session/remote media client returns again.
I don't know when we come around to look into this I'm afraid .

Thank you, seems logical. I was just verifying to be sure I didn't miss anything. I'll keep an eye on improvements in the future.

Would love an update on this issue, this is a major problem for us and definitely seems like it should be handled properly by CastPlayer.

Thank you for this comment @marcbaechinger. I stumbled upon it accidentally, I wish I could have read it before e.g. in README, it would have saved me a lot of time and debugging.

Hi @marcbaechinger
Just wondering if there were and updates on The empty media item issue?
It presents quite a significant problem, when resuming of a session is required. Caching a copy of the playlist, for the purpose of recovering from a Cast connection loss, if fairly ugly and highly prone to issues.
I am working around it by retaining a playlist of current items, which I reference rather than metadata on castplayer. I do this to avoid re-pushing a play queue if a resume is required. It's not ideal, due to potential to become out of sync, if queue items change for example. Have to be very careful to keep that list in sync on queue edits and so forth
I have noticed that the notification published by media 3 seems to have awareness of metadata post resume. Was wondering how notification obtaining this information?
Cheers

Hi @marcbaechinger
Was wondering if you had any updates or further advice on this issue, as yet?
Cheers


[Feature Request] a annotation tool for measuring 3D spheres

What feature or change would you like to see made?
I would like to have an annotation tool for measuring 3D spheres, with underlying metrics, like a combination of CircleROITool and SphereScissorsTool
Why should we prioritize this feature?
The current measurements are all based on a certain plane, and none of them are related to the volume in 3D

We have lablemap tools that can have statistics, does that help? https://www.cornerstonejs.org/live-examples/labelmapsegmentationtools  (the example does not show stats but you can get it)
Having said that it is not hard at all, you can implement it, take a look at more sophisticated tools like https://www.cornerstonejs.org/docs/examples


Why the select statement  "select count(1) from Order" take so long time?

OrientDB Version: Enterprise 2.2.26
Java Version: JDK1.8
OS: CentOS
Expected behavior
Can quick get the query result.
Actual behavior
The the select statement  "select count(1) from Order" take time about 50s.   The class Order has about 1200000 records.
Steps to reproduce

Hi @KishoDai
There is a hard-wired optimization for select count(*) from X, but it seems it doesn't apply when you do count(1).
My suggestion is to just use count(*) for now, hopefully we'll fix it in the near future.
Thanks
Luigi

@luigidellaquila
Got it. Thank you very much.


NDArray finished

Finished all functions in NDArray, with tests
Use additional ptr to handle java callback object in KVStore (tested)

This is awesome! Good work @javelinjs!  LGTM

LGTM too, @terrytangyuan , you can merge this~


Collect a FAQ and add to the proposal

Q: Are we stuck with CompletableFuture?
A: No, any API can be supported as a library
See #38

Q: What are the limitations?
A: We only have stack-less continuations

Closed. Obsolete.


hw2 instruction, Probable Typo?

Hi,
At the end of the  page three of the HW2 instruction, (Problem 1.a) Few equations sum over is. Shouldn't they be ts.

solved


[TIR] Add tir::builtin::assume

This PR introduces tir::builtin::assume, which is used to represent assumptions to be exposed to transformations, which would otherwise require out-of-scope information to prove.
This PR is part of the handling of padded buffer layout transformations (tracking issue, rfc).  Later PRs will introduce transformations that make use of the information provided by assumptions.

This is currently a draft PR, as the unit tests depend on functionality introduced in #12264.

#12264 has landed (Thank you @junrushao1994 for the quick review!), and this PR is ready for review.

CC @yzh119 who might be interested :-)


Upgraded a vulnerable package - json5

Issue reported by GitHub security and npm audit.
Fixes # (issue)
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation (including CHANGELOG updates)
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Don't think this is needed in package.json. Can't we merge:
https://github.com/trimble-oss/modus-react-bootstrap/pull/257 and
https://github.com/trimble-oss/modus-react-bootstrap/pull/258
instead?

update: I see it's in the resolutions section of the package.json so I guess it's ok. 👍


Add support for live update of free chargers in an area

Hi@all!
Would it be hard to add some kind of live update of free chargers in an area? I have it all the time. When I depart, I know where the nearest charger to my destination is. When I arrive, it is occupied. Stuttgart has it often enough... Now, would it be possible to pick one charger and have the chargers in vincinity be checked if they are free, too, so I can move fast to the next one? I would like to do the following steps:

turn on filter
Type my destination address, where I want to go to.
see the availability with little markers on the location markers like red, green and grey (not available). This would be fast and easy to see and adjust accordingly.
The app has to update this area in the background, obviously, so when I get it back from background when I think of it, I have good data. Perhaps a 5 min split would work, if the API can "swallow" it.
If the nearest charger would be occupied by now, it would show red, but the others around my destination would perhaps be free and I could simply use the pin and then navigation button to then take the next one easily, while driving or stopping at a red signal.

If I mark the nearest charger as temporary navigation "mark", it would be nice to see a message on the screen, if it is occupied or goes back to available, so I can adjust my route accordingly before passing by. This would be really helpful. :-)
Thank you for the great app! I like it very much for the clear UI and the good filters. :-) I just want it to be a little bit more helpful when really needed. :-)

Hi, thanks for the suggestion!
A combination of #166 and #42 would basically cover this use-case, right?

closing as it seems to be a duplicate and the original poster did not repond