invalid mutual definition

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Checked that your issue isn't already filed.
Reduced the issue to a self-contained, reproducible test case.

Description
mutually recursive definitions produce some unclear error message
Steps to Reproduce
universe u

inductive Foo : Type u → Type u → Type (u+1)
| foo {as bs cs: Type u}: Foo as bs → Foo bs cs → Foo as cs

namespace Foo
  mutual
    def bar    : {as bs: Type u} → Foo.{u} as bs → bs := bar_aux
    def bar_aux: {as bs: Type u} → Foo.{u} as bs → bs
    | as, bs, foo _ x => x.bar
  end
  termination_by
    bar     => 0
    bar_aux => 0
  decreasing_by sorry
end Foo

Expected behavior: this example should be accepted or produce a clearer message
Actual behavior: got error
invalid mutual definition, result types must be in the same universe level

Reproduces how often: 100%
Versions
nightly-2022-03-17
Additional Information
corresponding lean3 code doesn't work either
universe u

inductive Foo : Type u → Type u → Type (u+1)
| foo {as bs cs: Type u}: Foo as bs → Foo bs cs → Foo as cs

namespace Foo
  mutual def bar, bar_aux
  with bar     : ∀{as bs: Type u}, Foo as bs → bs | as bs           := bar_aux
  with bar_aux : ∀{as bs: Type u}, Foo as bs → bs | as bs (foo _ x) := x.bar
end Foo

The error message has been improved. It is now
 error: invalid mutual definition, result types must be in the same universe level, resulting type for `Foo.bar` is
  Foo as bs → bs : Type (u + 1)
and for `Foo.bar_aux` is
  bs : Type u

It makes it clear that Foo.bar is being interpreted by Lean as a function that returns  Foo as bs → bs : Type (u + 1).
You can write the function as
namespace Foo
  mutual
    def bar2    : Foo as bs → bs
      | x => bar_aux2 x

    def bar_aux2 : Foo as bs → bs
      | foo _ x => x.bar2

  end
  termination_by
    bar2     x => (sizeOf x, 1)
    bar_aux2 x => (sizeOf x, 0)
end Foo


[Feature] IBrowser only implements IAsyncDisposable and not IDisposable

Seeing that Playwright is a Microsoft product I thought it would be right if it would follow the recommendations from the Microsoft Doc.

It is typical when implementing the IAsyncDisposable interface that classes will also implement the IDisposable interface. A good implementation pattern of the IAsyncDisposable interface is to be prepared for either synchronous or asynchronous dispose.

https://docs.microsoft.com/en-us/dotnet/standard/garbage-collection/implementing-disposeasync

Hi, is this still planned for 1.26 release? I just bumped into this issue in my code too.

Disposing APIRequestContext, APIResponse, Browser, BrowserContext, and JSHandle are asyncronous, since they either interact with the browser itself to shutdown them gracefully or interact also with the file-system / Playwright driver to shut them down, cleanup files etc. It's discouraged to either ignore the exception so it runs in the background or force it to be sync by using GetAwaiter().GetResult() since it might be ending up being blocking.
Closing by that as a won't fix.

Adding more context to the discussion, it isn't a case of being a "best practice", it can literally cause leaks in users' applications:

Caution
If you implement the IAsyncDisposable interface but not the IDisposable interface, your app can potentially leak resources. If a class implements IAsyncDisposable, but not IDisposable, and a consumer only calls Dispose, your implementation would never call DisposeAsync. This would result in a resource leak.

As stated here: https://docs.microsoft.com/en-us/dotnet/standard/garbage-collection/implementing-disposeasync
From my understanding, anyone using an old hosting model or any disposing mechanism that wasn't designed when IAsyncDisposable was available will have leaks unless the user explicitly call this async disposable with sync-over-async.
So, as much as I understand that all those APIs are async, In my opinion this should be handled on Playwright's side, or at the very least be explicitly documented as a warning. It seems to me that simply expecting the user to know about this and do a workaround is a recipe for leaks happening.
What do you think? Is there something else that can be done on Playwright's side to avoid pitfalls?

IMHO. Playwright is a new, end-to-end testing library, async by nature. I don't think we need investments in old hosting models.

IMHO. Playwright is a new, end-to-end testing library, async by nature. I don't think we need investments in old hosting models.

That's a fair point, but is there something that can be done documentation-wise?

We updated the Microsoft docs to reflect the change best practices - should be live soon!: https://github.com/dotnet/docs/pull/30852
cc @akamud @kblok


 或者有什么办法修复吗

Yes, one area for improvement in a v2 API is within the offset commit flows. Currently the (manual) 'Commit' call is just a fire and forget func and any actual errors from the commit itself are flowed back onto the Consumer's errors channel (assuming Consumer.Return.Errors config setting is true), rather than being returned from the Commit call itself — so it is more like a "flush"
https://github.com/IBM/sarama/blob/a4eafb450c57b9a0243dc51f4cb9116f1f1bedbb/offset_manager.go#L251-L277
So you can check for any errors in your range loop on the consumer Errors channel and attempt to log or handle offset commit ones there


Feature: Default setting after PresenceLight is switched off

A color, a dimming and an on/off switch.
Currently, If I stop PL, the hue band switches to bright white
Dimming would be good as the light could be little less bright while PL is on.
Maybe I've overseen a setting?

I reset the light to white by design. When you say dim, are you referring to changing the brightness for the light always??

Yep, when I stop presencelight on my 2m Hue band, all of a sudden the room in front of my home office looks like a runway at the airport.
a fixed dimming would be good

Latest nightly has brightness added

Awesome!


[Mutator] DSP_Mutator

The author of this mutator (sadly no longer with us and much missed as a prolific and humourous contributor on UT99.org) is  MrLoathsome
Source : the mutator Readme

URL: https://unrealarchive.org/mutators/unreal-tournament/D/dsp_mutator_b2a16881.html
Hash: b2a16881aa3fb0e89ad28a6089b8507f59048700
Current name: DSP_Mutator

:cry:
Fixed


How to open new vim session

Hi,
I just tried out this plugin and it opens the file in an existing vim session. How do I configure to always open in a new vim session instead?

There's no such option at the moment.

Thank, I appreciate the quick answer.


3228: Make dashboard report use AJAX exclusively.

This change was done to improve load time on login and to show
the dashboard more quickly.

Good start. See my comments above. Thanks.


Dockstore fails to parse WDL if parameter_meta values aren't strings

Describe the bug
A WDL with a parameter_meta section will fail to update on Dockstore if any of the values are not of a string type.
See here for an example; all the values are booleans.
To Reproduce

Have a WDL workflow with a .dockstore.yml in GitHub
Add or update a parameter_metasection with at least one non-string value (see example in description)
Commit and push
Dockstore will not get updated, GitHub app logs will show an error.

Expected behavior
It should parse correctly, and the version on Dockstore should get updated/created.
Additional context
Webservice - 1.11.5
UI - 2.8
Compose Setup - 1.11
Deploy - 1.11.0
┆Issue is synchronized with this Jira Story
┆Issue Number: DOCK-1895
┆Sprint: Sprint 70- Whale shark
┆Issue Type: Story

Informally verified my own ticket on staging.

Forked to coverbeck/gatk
Pushed kc_test_dockstore
Double-checked the WDL has boolean meta parameters
I see a Success message in GitHub App Logs
The Info tab looks correct.


Allowed stdlib

Built on top of rspec/rspec-support#179.

This one is also ready for a review, @JonRowe.

:+1:


Resource management should be done by the framework

as of version 4.x , resource management is handled by the user:
Connection connection = .. ; // get a JDBC connection

Job job = aNewJob()
		.reader(new JdbcRecordReader(connection, "select * from tweet"))
		.jobListener(new JdbcConnectionListener(connection))
		.build();

In this example, the user has to:

open a connection to the database and pass it to the JdbcRecordReader
close the connection using a JdbcConnectionListener or keep it open using the keepAlive parameter

This is clearly error prone and should be handled by the framework:
DataSource dataSource = .. ; // get a JDBC data source

Job job = aNewJob()
		.reader(new JdbcRecordReader(dataSource , "select * from tweet"))
		.build();

The JdbcRecordReader should be created with a data source from which it will open/close a connection.
This is the same for:

JpaRecordReader: it should be created with EntityManagerFactory and not EntityManager
HibernateRecordReader: it should be created with SessionFactory and not Session

JdbcConnectionListener, JpaEntityManagerListener and HibernateSessionListener should be removed

implemented in 33bd7eabdf7bdc66ee00ea7dcbefcc793f1f9cbb and 009769d0ffe3c9aa23a9ede1067559e79b9ec22e


array arguments are interpreted as float

'use strict';
var yargs = require('yargs');

var argv = yargs
    .option('v', {
      type: 'array'
    })
    .argv;

console.log(argv.v);

Given the simple script I get the following output;
$ node test.js -v 0.10 -v 0.12
[ 0.1, 0.12 ]

$ node test.js -v "0.10" -v "0.12"
[ 0.1, 0.12 ]

This is reproducible with v3.9.1 & v3.10.0.
Question: Would it be possible to configure a type for the array arguments?

perhaps something like this?
var argv = require('yargs')
  .array('v', {type: 'string'}

The caveat being you'd have to use the array method rather than the option shorthand.

I like that.

@skenqbx There hasn't been any movement on this lately, but if you're ok with passing multiple flags (i.e. -v 0.10 -v 0.12) you can avoid the float coercion using type: 'string' for your option instead of type: 'array':
var argv = require('yargs')
  .option('v', {
    type: 'string'
  })
  .argv
console.log(argv.v)

$ node test.js -v 0.10 -v 0.12
[ '0.10', '0.12' ]

The only real benefit of using type: 'array' is to support a variable number of arg values for an option without requiring duplicate flags, like:
$ node test.js -v 0.10 0.12
[ 0.1, 0.12 ]

As you've found, this is where float coercion is a problem.

@nexdrew I can perfectly work with your solution, thank you!


TRG 7.02 License and copyright header

Some of the files in the repository don't include the copyright "Copyright (c) 202x,20yy Contributors to the Eclipse Foundation" and therefore needed to be added in the header.
https://eclipse-tractusx.github.io/docs/release/trg-7/trg-7-02#copyright-header

MARKDOWN/HTML:
<!--
  Catena-X - Digital Product Passport Application

  Copyright (c) 2022, 2024 BASF SE, BMW AG, Henkel AG & Co. KGaA
  Copyright (c) 2022, 2024 Contributors to the Eclipse Foundation

  See the NOTICE file(s) distributed with this work for additional
  information regarding copyright ownership.

  This program and the accompanying materials are made available under the
  terms of the Apache License, Version 2.0 which is available at
  https://www.apache.org/licenses/LICENSE-2.0.

  Unless required by applicable law or agreed to in writing, software
  distributed under the License is distributed on an "AS IS" BASIS
  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND,
  either express or implied. See the
  License for the specific language govern in permissions and limitations
  under the License.

  SPDX-License-Identifier: Apache-2.0
-->

JS:
/**
 * Catena-X -  Digital Product Passport Application
 *
 * Copyright (c) 2022, 2024 BASF SE, BMW AG, Henkel AG & Co. KGaA
 * Copyright (c) 2022, 2024 Contributors to the Eclipse Foundation
 *
 * See the NOTICE file(s) distributed with this work for additional
 * information regarding copyright ownership.
 *
 * This program and the accompanying materials are made available under the
 * terms of the Apache License, Version 2.0 which is available at
 * https://www.apache.org/licenses/LICENSE-2.0.
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND,
 * either express or implied. See the
 * License for the specific language govern in permissions and limitations
 * under the License.
 *
 * SPDX-License-Identifier: Apache-2.0
 */

YAML/SH:
#################################################################################
# Catena-X -  Digital Product Passport Application
#
# Copyright (c) 2022, 2024 BASF SE, BMW AG, Henkel AG & Co. KGaA
# Copyright (c) 2022, 2024 Contributors to the Eclipse Foundation
#
# See the NOTICE file(s) distributed with this work for additional
# information regarding copyright ownership.
#
# This program and the accompanying materials are made available under the
# terms of the Apache License, Version 2.0 which is available at
# https://www.apache.org/licenses/LICENSE-2.0.
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND,
# either express or implied. See the
# License for the specific language govern in permissions and limitations
# under the License.
#
# SPDX-License-Identifier: Apache-2.0
#################################################################################

JAVA:
/*********************************************************************************
 *
 * Catena-X - Digital Product Pass Application
 *
 * Copyright (c) 2022, 2024 BASF SE, BMW AG, Henkel AG & Co. KGaA
 * Copyright (c) 2022, 2024 Contributors to the Eclipse Foundation
 *
 *
 * See the NOTICE file(s) distributed with this work for additional
 * information regarding copyright ownership.
 *
 * This program and the accompanying materials are made available under the
 * terms of the Apache License, Version 2.0 which is available at
 * https://www.apache.org/licenses/LICENSE-2.0.
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND,
 * either express or implied. See the
 * License for the specific language govern in permissions and limitations
 * under the License.
 *
 * SPDX-License-Identifier: Apache-2.0
 ********************************************************************************/


RealDebrid api change? links are stuck on loading.

Idk if RD changed their api or something but on stremio when i click on any RD torrentio link they dont start and are stuck on loading. This is weird cuz it was working flawlessly before today.
I also noticed that torrents were not being fetched on my account which they normally would when it was working that why im guessing RD changed their api?
If you could look into that'd be great! Besides that, torrentio rocks! Appreciate what you've made.

nvm still happening


Adding code of conduct

Issue #2
I have added code of conduct file to this repository.

@ChameeraD


Fix publishing

This PR fixes the publishing of JsDocs.
update-gh-pages exposes a list of generatedDocs via ext, which other tasks can write to, if they wish to expand the list of docs.
At some point during migration to Kotlin, we have substituted a call of ext.generatedDocs += newDocs to ext["generatedDocs"] += newList(predefinedDocs).add(newDocs). add returns true, and thus generatedDocs became a boolean.
This PR also advances the versions.

@armiol, PTAL.


Ci-Chore: run nightly --tests-- on push/merge to main only

Summary
To cut down on PR queues, let's run nightly tests (everything else stays the same) only on merges/pushes to the main branch.

Closing, as this is less necessary.


Wrote fixDateTime() method

Wrote fixDateTime() method: attempts to fix invalid DateTime object by decrementing invalid time/date components and incrementing next components to compensate. E.g. If DateTime object represents the 32 July 2019 then running fixDateTime() will change it to 1 August 2019.
Also wrote getDaysInMonth() and isLeapYear() functions. These are used by fixDateTime() hence the inclusion.

I did a small test that shows the effect of the mm field overflowing. fixDateTime() performs the following conversions:
2000-01-01T00:255:59 (INVALID) -> 2000-01-01T04:15:59 (valid)
2000-01-01T00:255:60 (INVALID) -> 2000-01-01T00:00:00 (valid)

The first one is the correct, expected behavior. The second one is incorrect: it should return 2000-01-01T04:16:00.


Impossible to launch the software

Whan i double click to open the soft, it lauches 2 sec in the task manager and it closes, i tried all versions but no one works

The any of the solutions in the following issue help?
https://github.com/ptasev/Ego-Engine-Modding/issues/27


Fedora 30 Problems with permissions when using ly

when using startx /path/to/windowmanager or using other display managers (so far sddm and lightdm) applications that need permissions to do certain actions like file managers mounting drives works as expected. I have tried this with  pcmanfm and nemo so far
when I use ly as the display manager both pcmanfm and nemo give the message
"not authorized to perform action"
when I try to mount an external device

This should be fixed in 4c17b2f, forgot to close (again)


Ensure ShibbolethRestController only redirects to trusted URLs after authentication

This bug was discovered/reported by LGTM in the ShibbolethRestController
Description
The ShibbolethRestController will allow redirection to arbitrary URLs after successful authentication.  This is a security issue if individuals are fooled into clicking the below link while logged in.
This behavior can even be reproduced on the demo REST API by doing the following:

Login the the REST demo site: https://dspace7.4science.cloud/server/login.html
Then, visit this link:  https://dspace7.4science.cloud/server/api/authn/shibboleth?redirectUrl=http://google.com
You will be redirected to the Google homepage

In this PR, I've modified this Controller to ONLY allow redirects to URLs that have the same hostname as either dspace.server.url or dspace.ui.url configurations.  This should allow Shibboleth to work for any REST API or UI path, but not allow redirects to arbitrary locations.
Instructions for Reviewers

Review code & verify the new ITs which prove the functionality.
Optionally, retest above example on a local REST API, and verify that passing "http://google.com" now returns a 400 Bad Request response.
Ideally, someone with Shibboleth can test this new setup to verify that it works as expected with Shibboleth enabled for DSpace.

I've just tested and it works as expected on both sides (HAL browser and Angular).

Merging with one approval (NOTE: I added a final commit here just to add an @SuppressWarnings annotation to let LGTM know we fixed this bug.  It seems like LGTM didn't automatically detect the fix in this case)


Please tag 2.8.1

2.8.1 is missing a tag, could you push it please?

cc @randx

Sure I will

Done - https://github.com/gitlabhq/grit/releases/tag/v2.8.1


Conclusion

This is awesome!
Great idea, and nice execution!
I felt like the conclusion was a little anti-climactic (unless I missed something). After figuring out the murderer's name it was sort of... "well, am i done now?"
Would have enjoyed more of a catch at the end of the chase.
But most importantly I learned some new commands and flags, so job well done :grinning:
Are you planning on continuing this project?

This is great! @veltman, The idea is really enjoyable! but needs a longer story, keep the idea alive! ;)


Proofing reading, grammar & syntax corrections made to all docs

Your work is very much appreciated and that's why I put in the time to make the documents as professional  sounding as possible. I have plans to convert all of the docs - once you approve them - into mp3 files so I listen to them while driving etc. to absorb the info even more. Thanks again and please contact me if you need to add more docs etc. so I can proof them. It's been a pleasure!
Takis

The changes look great to me! Incredible work @eliotis

The changes look great to me! Incredible work @eliotis

Thanks @itstimetogoleft !

Made requested changes/corrections, please let me know if there is anything else.

@eliotis
Thank you once again for the amazing work you've done ❤


Doctests should be removed in favor of unit tests and documentation.

Doctests are being removed in favor of unit tests in this branch: https://github.com/cheyneh/permpy/tree/unit-tests


Server certs need to be readable by user and group

The server certificate is used by httpd (running as root) and MiqRemoteConsoleWorker (running as manageiq).
https://github.com/ManageIQ/manageiq/issues/22065

I didn't realize that the remote console worker needed the certificates, I thought that httpd took care of all of the TLS termination here

Yeah it loads them here https://github.com/ManageIQ/manageiq/blob/master/lib/remote_console/client_adapter/ssl_socket.rb#L23-L24 I'm guessing remote consoles aren't proxied by apache?
Here was the original discussion https://github.com/ManageIQ/manageiq/discussions/22060

Backported to oparin in commit 6b8bdf351eb6dbbf0644cc9fba3d71ac79f1ddd1.
commit 6b8bdf351eb6dbbf0644cc9fba3d71ac79f1ddd1
Author: Brandon Dunne<EMAIL_ADDRESS>Date:   Mon Aug 29 15:33:21 2022 -0400

    Merge pull request #365 from agrare/server_certs_need_rw_group

    Server certs need to be readable by user and group

    (cherry picked from commit 7348101fd3528b7aa95cdaeceeace8d7933f9819)

Backported to najdorf in commit 22fc36b8e731b8effef862b8d7e6bfa08d486c35.
commit 22fc36b8e731b8effef862b8d7e6bfa08d486c35
Author: Brandon Dunne<EMAIL_ADDRESS>Date:   Mon Aug 29 15:33:21 2022 -0400

    Merge pull request #365 from agrare/server_certs_need_rw_group

    Server certs need to be readable by user and group

    (cherry picked from commit 7348101fd3528b7aa95cdaeceeace8d7933f9819)


wg-performance Status Update 2022-03-28

cc @ampproject/wg-performance
What was accomplished
Lessons Learned
What's Next

What was accomplished
Vacation! 🌴

What was accomplished

Team planning
Brotli Prepend prototype nearing completion

Lessons Learned

Brotli is still very complicated, even with an already implemented decompressor

What's Next

TC39
Brotli Prepend

What was accomplished

updated the amp story translations transformer with new caching mechanism
disabled size adjust experiment temporarily

Lessons Learned
What's Next

amp e2e use mjs files
amp-story translations request transformer


x/crypto/ssh: login with incorrect credentials

func LinkSSH(host, user, pass string) (info string) {
config := &ssh.ClientConfig{
	User: user,
	Auth: []ssh.AuthMethod{
		ssh.Password(pass),
	},
}
client, err := ssh.Dial("tcp", host, config)
if err != nil {
            info = "connect error"
	return info
} else {
	session, err := client.NewSession()
	defer session.Close()

	if err != nil {
                    info = "login error"
		return info
	} else {
		info = "login ok"
		client.Close()
		return info

	}
}

}
if ssh-server is Dropbear user and pass error return login ok

Are you saying that against a dropbear server, providing incorrect
credentials allows you to login to the server anyway?
On Sun, 11 Oct 2015 20:59 tobor<EMAIL_ADDRESS>wrote:

func LinkSSH(host, user, pass string) (info string) {
config := &ssh.ClientConfig{
User: user,
Auth: []ssh.AuthMethod{
ssh.Password(pass),
},
}
client, err := ssh.Dial("tcp", host, config)
if err != nil {
info = "connect error"
return info
} else {
session, err := client.NewSession()
defer session.Close()
if err != nil {
                info = "login error"
    return info
} else {
    info = "login ok"
    client.Close()
    return info

}

}
}
if ssh-server is Dropbear user and pass error return login ok
—
Reply to this email directly or view it on GitHub
https://github.com/golang/go/issues/12901.

can't reproduce. I've tested SSH-2.0-dropbear_2015.67 in OpenWRT.
Note that if the ssh client is unable to authenticate, the Dial call will
fail.
Please provide more information on how to reproduce.

Not able to reproduce either, please open with more details if it is still the case.

Are you saying that against a dropbear server, providing incorrect
credentials allows you to login to the server anyway?
Provided the wrong username and password procedure returns successfully, it should return failure

Provided the wrong username and password procedure returns successfully, it should return failure

Can you replicate this behaviour with another ssh client (you may have to mess with the order and types of authentication attempted).
In this case the ssh package is acting as a client to the dropbear server, so
a. this isn't the fault of the client
b. you may have discovered a lucrative bug in dropbear.

thank you all，I tested xshell and putty return success, but will be asked to re-enter the password，this isn't the fault of the client

Are you saying that against a dropbear server, providing incorrect
credentials allows you to login to the server anyway?

Sorry for not providing little context in my previous comment. It is not reproducible against other servers. I wanted to say the issue is probably dropbear, not the ssh client.

Thank you for confirming, I agree with your assessment.
On Sat, 17 Oct 2015, 08:03 rakyll<EMAIL_ADDRESS>wrote:

Are you saying that against a dropbear server, providing incorrect
credentials allows you to login to the server anyway?
Sorry for not providing little context in my previous comment. It is not
reproducible against other servers. I wanted to say the issue is probably
dropbear, not the ssh client.
—
Reply to this email directly or view it on GitHub
https://github.com/golang/go/issues/12901#issuecomment-148836438.


Update Goreleaser

Ref. https://github.com/goreleaser/goreleaser/issues/2196#issuecomment-846316315

Fixed with https://github.com/gohugoio/hugo/pull/8620


Update Android LTS NDK to 27

Tool name
Android NDK
Tool license
Custom
Add or update?

[ ] Add
[X] Update

Desired version
27
Approximate size
No response
Brief description of tool
No response
URL for tool's homepage
No response
Provide a basic test case to validate the tool's functionality.
No response
Platforms where you need the tool

[ ] Azure DevOps
[X] GitHub Actions

Runner images where you need the tool

[ ] Ubuntu 20.04
[X] Ubuntu 22.04
[ ] Ubuntu 24.04
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[ ] Windows Server 2019
[ ] Windows Server 2022

Can this tool be installed during the build?
No response
Tool installation time in runtime
No response
Are you willing to submit a PR?
No response

Hi @finagolfin--Thank you for bringing this issue to us. We are looking into this issue and will update you shortly on this issue after investigating.

Hi @finagolfin ,
We added new NDK Version , kindly try with it latest version . Thank you !

I'm now using this, works well. 😃

Thanks for the confirmation @finagolfin

macos is missing


Update README.md

Added badges.
Closes #2
Closes #3

@AleksandarDev please, update the CHANGELOG and bump the version as well.

@radovix Updated changelog and bumped version


Milestone 4 - App improvements

At least five improvements:

Styling:

Recolour plot headers for readability,
Make plot sizes same for ratings and Netflix movies over time
Reduce size of table and increase size for world map since world map looks shrunken;
Add a year column to the table, then sort by descending order. (probably a note of the order)

Comments from the peer review: documentation

Rename world map title with short description
Add more explanation for movie rating plot in description README;
Styling for table of examples for readability;

TA feedback (do we have any?)

You should paste the link to the commits in the submission with a brief description of the change (similar to a meaningful commit message).

App improvement

Blockquotes are the comments from the peer review
Link to peer review is here

Improvements based on the peer review:

Recolour plot headers for readability,

The header color for “World Map” appears to be a lighter shade than the other headers and is harder to read with the yellow font. A darker shade might help increase contrast and readability.

The color schemes of header cards have been changed to the Netflix color schme with black dark fill-in color and red font color
[TODO] add the commit link

Make plot sizes same for ratings and Netflix movies over time

The plot of Ratings and Netflix Movies over Time are of different plot sizes which keeps changing based on genre selected. It would be great if you could make both the plots sizes the same.

Plots now have the fixed scales, but since the scales are fixed, now there will be values going over the fixed scale since the number of movies per each genre vary a lot
[TODO] add the commit link

Reduce size of table and increase size for world map since world map looks shrunken;

The world map plot is an interesting plot . It would be great if you could provide interactivity to it . Also the plot looks shrunken , the table plot can be reduced in size to increase the visual representation of the world map .

Not sure what proper interactivity can be added, but the arrangement of the plots are now changed, such that the world map plot would be larger

The suggested sliding bar for year would not be appropriate here, since the data would be grouped for counting, year data would not be easily available (even though it is doable, it requires way too much time given the current deadline); plus the time plot is given for some ideas of the global changes over time

A title also has been added to improve communication of what this world map is showing, regarding to this comment:

For example, in the plot named “World map”, I found it hard to understand what the app was showing. It could mean how many users in different countries were using Netflix as well without providing sufficient context information.

[TODO] add the commit link

Add a year column to the table, then sort by descending order. (probably a note of the order)

Table of Examples can incorporate other information like film’s year so that stakeholders can get more insight on these films and perhaps be ordered in descending order of year.

The year of release and the duration of the movie are now added in the table, and the results are ordered in descending oredr of the release year
[TODO] add the commit link

Rename world map title with short description

Change the Card label to a short description instead of "World Map"
This has been handled by the previous commit

Add more explanation for movie rating plot in description README;

However, it would be better if more explanation could be provided about the movie rating. This part is not that intuitive.

Extra explanation and clarafication added for the movie rating plot
The name of the rating plot has also been changed to avoid confusion with the actual rating of the movie (here the data we have is the maturity rating)

This was added in a separate commit: [TODO]

[TODO] add the commit link

Styling for table of examples for readability;

Styling could be added to the Table of Examples to help increase readability, such as a bolded header, striped rows, and left alignment for text columns.

The improved styles have been aded for the table (now changed name to Table of Netflix Movies for more meaningful title)
Also added sorting option for the columns
[TODO] add the commit link

TA feedback (No feedbacks from TA yet, and we should already have plentiful of improvements above)


Application do_render method

Just noticed that the method takes three arguments, but

it doesn't really need the first two, as they are instance variables
it doesn't use the third parameter

Again, this is simple enough, i could do it if you want

That will be great! Thanks a lot!


inconsistent tensor size

Hi, do you know the following errors. thanks
/root/torch/install/bin/luajit: /root/torch/install/share/lua/5.1/threads/threads.lua:183: [thread 1 callback] .../Photo-Realistic-Super-Resoluton-master/data/dataset.lua:339: inconsistent tensor size at /root/torch/pkg/torch/lib/TH/generic/THTensorCopy.c:7
stack traceback:
[C]: in function 'copy'
.../Photo-Realistic-Super-Resoluton-master/data/dataset.lua:339: in function 'tableToOutput'
.../Photo-Realistic-Super-Resoluton-master/data/dataset.lua:359: in function <.../Photo-Realistic-Super-Resoluton-master/data/dataset.lua:347>
[C]: in function 'xpcall'
/root/torch/install/share/lua/5.1/threads/threads.lua:234: in function 'callback'
/root/torch/install/share/lua/5.1/threads/queue.lua:65: in function </root/torch/install/share/lua/5.1/threads/queue.lua:41>
[C]: in function 'pcall'
/root/torch/install/share/lua/5.1/threads/queue.lua:40: in function 'dojob'
[string "  local Queue = require 'threads.queue'..."]:15: in main chunk

Hi, I also have the same problem. In /data/dataset.lua line 339, I checked the size of data[i] and dataTable[i] and found that they really had different sizes. The size of data[i] is 1x96x96, while dataTable[i] has four different sizes (e.g., 1x128x96, 1x96x128, 1x143x96, 1x96x144) corresponding to 'nThreads = 4'. As a result, the error 'inconsistent tensor size' occurred.
The error information are as follows:
lunge@lunge:/media/lunge/Users/projects/SRGAN/Photo-Realistic-Super-Resoluton$ th run_sr.lua
{
ntrain : inf
beta1 : 0.9
name : "super_resolution"
dataset : "folder"
niter : 250
lr : 0.001
model_folder : "model"
gpu : 1
nThreads : 1
t_folder : "/media/lunge/Data/IMAGENET2015/CLS_LOC_dataset/train"
batchSize : 32
loadSize : 96
}
Starting donkey with id: 1 seed: 1
table: 0x41e4bfe0
Loading train metadata from cache
Dataset: folder	 Size: 	1281167
cunn used
/home/lunge/torch/install/bin/luajit: /home/lunge/torch/install/share/lua/5.1/threads/threads.lua:183: [thread 1 callback] ...s/SRGAN/Photo-Realistic-Super-Resoluton/data/dataset.lua:339: inconsistent tensor size at /home/lunge/torch/pkg/torch/lib/TH/generic/THTensorCopy.c:7
stack traceback:
[C]: in function 'copy'
...s/SRGAN/Photo-Realistic-Super-Resoluton/data/dataset.lua:339: in function 'tableToOutput'
...s/SRGAN/Photo-Realistic-Super-Resoluton/data/dataset.lua:359: in function <...s/SRGAN/Photo-Realistic-Super-Resoluton/data/dataset.lua:347>
[C]: in function 'xpcall'
/home/lunge/torch/install/share/lua/5.1/threads/threads.lua:234: in function 'callback'
/home/lunge/torch/install/share/lua/5.1/threads/queue.lua:65: in function </home/lunge/torch/install/share/lua/5.1/threads/queue.lua:41>
[C]: in function 'pcall'
/home/lunge/torch/install/share/lua/5.1/threads/queue.lua:40: in function 'dojob'
[string "  local Queue = require 'threads.queue'..."]:15: in main chunk
stack traceback:
[C]: in function 'error'
/home/lunge/torch/install/share/lua/5.1/threads/threads.lua:183: in function 'dojob'
...ects/SRGAN/Photo-Realistic-Super-Resoluton/data/data.lua:76: in function 'getBatch'
run_sr.lua:85: in function 'opfunc'
/home/lunge/torch/install/share/lua/5.1/optim/adam.lua:37: in function 'adam'
run_sr.lua:133: in main chunk
[C]: in function 'dofile'
...unge/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:145: in main chunk
[C]: at 0x00405d50
lunge@lunge:/media/lunge/Users/projects/SRGAN/Photo-Realistic-Super-Resoluton$
The size of dataTable[i] should be 1x96x96, I think. Would you please help us solve this problem? Thank you! @leehomyc

Hi I don't have this problem while training.
Yes data[i] has size 96x96. However dataTable[i] uses sampleHookTrain which relates to the function local trainHook=function(self,path) in donkey_folder_supres.lua. The function returns two outputs (input_y, and input_y2) where the first one is of size 96x96 and the second one has size 24x24. I do not know where other sizes come from.
If you know what caused the problem, please let me know and I can update the repository. Thanks!

@leehomyc
I have also got this problem.

Can you tell us your torch and other libs version?

Hello!
Sorry I fixed the issue which is caused by image size. When the input image's height and width are different there will be such error.

@leehomyc  thanks.


"heroku local" does not print full output in Go (golang) web apps

When using heroku local to run Go (golang) web apps, the full stack trace of panics is not output.
For example, here is the full output of an error in my app that causes a panic. As you can see, only the first line of the output is displayed:
$ heroku local web
[00] forego | starting web.1 on port 5000
[00] web.1  | 2015/12/31 18:49:53 http: panic serving [::1]:63130: runtime error: invalid memory address or nil pointer dereference
[00] web.1  | t/http.HandlerFunc.ServeHTTP(0xc20800b220, 0x913cb0, 0xc208094960, 0xc208030ea0)

it's a known issue in an upstream dependency: https://github.com/ddollar/forego/issues/67

Until this is fixed, running this (instead of "heroku local web") works for me:
$ forego start


Autoform 6 autosave problem

Hi
When using autosave for Autoform 6 it is really slow with page load times of 15-20 seconds.
Without autosave everything runs quick, like in a few seconds, as the forms have around 50-80 fields.
What I've noticed is that with Autosave my helpers (3 of them) are called multiple times (about 30-50) before on render even happens.
Please can you investigate the issue.
Thanks

@peardigital Please create a repo with an app that minimally reproduces this issue, and post a link to it.

Here is the slow copy with autosave turned on. https://bitbucket.org/libertyapps/node/commits/6c143526fc826d271712cade9750d8601d4b6500
The working copy without Autosave:
https://bitbucket.org/libertyapps/node/commits/03452f9aef5dd021dffa38fc02ca621e81e3bd08
This is the route / page that is problematic
FlowRouter.route('/user/view/:id', {
name: 'user-view',
action() {
BlazeLayout.render('MainLayout', {main: 'ViewUser'});
}
});


Multiple alert rules of groups in one file

The prometheus version i use is   2.0.0-beta.2, and i want to make some alert rules in one file, like this:
first.rules:
 groups:
  - name: test
      rules:
      - alert: test
      expr: go_memstats_heap_released_bytes<1

 groups:
 - name: test1
   rules:
   - alert: test1
   expr: go_memstats_heap_released_bytes<1

There are many groups in first.rules.
But only the last alert rule of groups takes effect when i lanch prometheus，it is alert:test1 will take effect.
Is there a problem with my operation？
Does this release support this？
Thanks

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


Dropdown 'openOnSpace' triggered through TextArea

Hey guys,
I've multiple Dropdown components at one page, and also other form components like TextArea included. When I use the dropdown components everything works fine, but when I then use my TextArea and press space inside, all my previous selected dropdowns are opening.
Looks like it has something to do with the search-attribute for the dropdown, when I remove this, it works.
Steps

Select item in dropdown
select textarea
start typing
press space

Expected Result
The listener for space-key should be removed for dropdowns if losing focus.
Actual Result
Dropdown/s is/are opened.
Version
0.56.0
Testcase
http://codepen.io/treckerflx/pen/ALPdPg?editors=0010

Thank you for the superb report.  I've labeled this issue for fixing.

I've got a fix in progress, PR coming soon.

Released in<EMAIL_ADDRESS>

Thank you guys, incredible fast fix. :) 👍

No problem!


merge package template from arlac77/npm-package-template

package.json

chore(devDependencies): update<EMAIL_ADDRESS>from template

Coverage remained the same at 97.143% when pulling b0bb68b14f243f65d375492d4a97b57fc84cc774 on template-sync-1 into 235bebc85bf416cc1a8f5e4b7dcd0ab5a7fa5aa2 on master.

Codecov Report

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

@@          Coverage Diff          @@
##           master    #31   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      2
  Lines          29     29
=====================================
  Hits           29     29

Continue to review full report at Codecov.

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


CB-SDK: PeerConnection: IceConnectionState change to RTCIceConnectionState.RTCIceConnectionState Failed for opponent in

What is the reason behind this.. This is happens when user open app through notification and attend call and after some seconds i get above mention error or message and call ended for caller....

How to implement push notifications for video call?

@ShyamKrishna122 i am sending notification(call) to the receiver person using fcm and receiver get notification to attend the call.
i know this is not best approach but i have achieved this.

here short guide, how to implement background call in Flutter using our SDK

I want a sample code for implementing push notification for video call

@TatankaConCube CB-SDK: PeerConnection: IceConnectionState change to RTCIceConnectionState.RTCIceConnectionState Failed for opponent...what is the reason behind this error. is it because of poor internet?

it means you have some problems with establishing a connection between peers, it can happen when peers didn't exchange with icecandidates, or something happen with internet connection after/during an exchange with icecandodates

On which step do you get this error?

@TatankaConCube after attending the call opposition's screen(camera view) becomes white screen for sometime and i get this error and call cuts automatically(For caller). This usually happens when attending the call first time in application then work as expected

Should be fixed in the latest (1.1.1) release. Closing. If the issue still reproduces, please create a new ticket with details.

Should be fixed in the latest (1.1.1) release. Closing. If the issue still reproduces, please create a new ticket with details.


added endTask function; added D-score calculation and display;

Issues

Functions are not firing or printing to the console

@yago1994 the D score is being shown and added the feedback right below it for extra context.
Question: is the D score supposed to be calculated in a different way from the d score provided by the iat? Because I am currently using the d score provided from the iat extension and not calculating it with your functions.

Let’s use the D-score provided by the IAT because that’s what gets used for the test, so that’s alright.
However let’s remove the feedback so as not to bias participants to perform differently in the next round of the test. The page should just display the score
Sent from Outlook for iOShttps://aka.ms/o0ukef

From: Wiza Munthali @.>
Sent: Wednesday, July 17, 2024 3:32:27 AM
To: AppHatchery/Gender-career-IAT @.>
Cc: SantiagoA @.>; Mention @.>
Subject: Re: [AppHatchery/Gender-career-IAT] added endTask function; added D-score calculation and display; (PR #2)
@yago1994https://github.com/yago1994 the D score is being shown and added the feedback right below it for extra context.
Question: is the D score supposed to be calculated in a different way from the d score provided by the iat? Because I am currently using the d score provided from the iat extension and not calculating it with your functions.
—
Reply to this email directly, view it on GitHubhttps://github.com/AppHatchery/Gender-career-IAT/pull/2#issuecomment-2232626106, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AD5EOKDGCP4RM7BSLYFQ5OLZMYMYXAVCNFSM6AAAAABK4SCFSOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZSGYZDMMJQGY.
You are receiving this because you were mentioned.Message ID: @.***>


Update usage of api

Update usage of apis

Can go.mod be updated with the version of crd-generator with istio types as part of the PR

Good idea - that will make sure we really got rid of everything.

Can go.mod be updated with the version of crd-generator with istio types as part of the PR

Yes, I will do that and also for any other repo that is using crd-generator

Can go.mod be updated with the version of crd-generator with istio types as part of the PR

go.mod is updated latest  new verrazzano-crd-generator changes.  I also updated the commit hash to other verrazzano components that were out of date.
Will rerun an acceptance test suite with the new operator image.


Default export from next-routes

Hi !
I'm trying to get next-routes working with TypeScript.
I have this code right now :
import Routes from 'next-routes';

const routes = new Routes();

routes
  .add('about')
  .add('post', '/p/:id')

export default routes;

But when I'm trying to compile I get this error :

const routes = new Routes();
^
TypeError: next_routes_1.default is not a constructor

I followed example here
Could you please help me ?

Same here. Solved it with:
import * as createRoutes from "next-routes"
import Routes from "next-routes"

// @ts-ignore Types are broken
const routes: Routes = createRoutes()

routes
  .add("about")

export default routes
export const Link = routes.Link
export const Router = routes.Router

It works but requires a @ts-ignore and has pretty a bad semantic. I would like to see this fixed too :) @fridays do you accept PR on this issue? Is there something we do not know?

@prevostc I agree, that's not good and should be fixed but.. It works atleast :D

@fridays I believe there's a pending PR to fix this. Can we get it merged please?

Anything happening with this?


Last updated blank for City of Surrey

Switched to CKAN's revision_timestamp instead... need to chat with Devin on this.

Looks like the tag we want is "metadata_modified".  Changed to this instead of "revision_timestamp"


docs: Update

Description
Checklist
Please confirm that this pull request has done the following:

[ ] Tests added
[x] Documentation added (where applicable)
[x] Changelog item added to changelog/

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

Project coverage is 91.96%. Comparing base (cf58495) to head (59ccd5c).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #114   +/-   ##
=======================================
  Coverage   91.96%   91.96%
=======================================
  Files          15       15
  Lines         672      672
  Branches       72       72
=======================================
  Hits          618      618
  Misses         31       31
  Partials       23       23

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


feat(sdk): SDK changes

SDK Changes
Description
Makes some changes to sdk:

Derives batcher_url from Network paramater.
Adds --private_key flag to DepositToBatcher
Adds msg to batcher errors telling a user there funds have not been spent.

Type of change
Please delete options that are not relevant.

[x] New feature
[ ] Bug fix
[ ] Optimization
[ ] Refactor

Checklist

[ ] “Hotfix” to testnet, everything else to staging
[ ] Linked to Github Issue
[ ] This change depends on code or research by an external entity

[ ] Acknowledgements were updated to give credit

[ ] Unit tests added
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

[ ] This change is an Optimization

[ ] Benchmarks added/run

[ ] Has a known issue

Link to the open issue addressing it

[ ] If your PR changes the Operator compatibility (Ex: Upgrade prover versions)

[ ] This PR adds compatibility for operator for both versions and do not change batcher/docs/examples
[ ] This PR updates batcher and docs/examples to the newer version. This requires the operator are already updated to be compatible

@uri-99, your comments have been addressed at: 2e8465c.


[FEATURE] New Release (v0.62.0?)

Is there an existing issue for this?

[X] I have searched the existing issues

Feature description
Would it be possible for a new release to be published? There's been quite a few new features, bug fixes, and updates since the last release in June.
What might the configuration look like?
n/a
Anything else?
https://github.com/nerdswords/yet-another-cloudwatch-exporter/compare/v0.61.2...master

Thanks for the patience @patrickdash, we've been busy with the move of the project to prometheus-community, will cut a release as soon as things are settled.


Error on My Goals

ErrorException:
ob_end_flush(): Failed to send buffer of zlib output compression (0)
http://performance-332842-dev.apps.silver.devops.gov.bc.ca/goal/current

Closed: wrong repo.


I want to develop single-module wrappers by default, and not be constrained/guided by "Query" or "Mutation" notions

Description
Previously #835
Today, users are restricted to programming their wrapper's logic into 2 different Wasm modules: query.wasm and mutation.wasm This is because in the schema interface layer, we are abiding by standard GraphQL best practices, and exposing all executable methods via the Query and Mutation types.
Based on prior considerations, we are moving farther away from GraphQL (more information here), so it only makes sense to also allow developers to name their own custom modules.
And not only that, because of this GraphQL notion, we've been assuming wrappers to contain multiple modules by default; when this is not the regular case with regular libraries and SDKs. Working with an arbitrary number of named modules inside of a wrapper, comes with a non-trivial increase in complexity and capability, just like working with a monorepo.
Therefore, we should assume single-module wrappers as the default case. We would develop additional, multiple module specific, infrastructure in the future
Planning
After a conversation with @dOrgJelli and later with @cbrzn, we agreed on removing the concept of modules in the current implementation altogether; and in the future, if needed, re-add it with a different implementation and mindset completely.
This would not only simplify the complexity around the core parts of the toolchain, but also simplify the development, usage and invocation of wrappers in general.
A summary of the conversation can be found here
Implementation Plan
A safe way of implementing the necessary changes in parallel without much conflict would be separating the changes related to the schema pipeline, from the rest of the core infrastructure's changes. Hence, the necessary subtasks are:

[ ] #887
[ ] #888
[ ] #889
[ ] #890
[ ] Update docs

Note that templates packages needs to be updated too - Is it worth to create a subtask for this?


Please include Async documentation on docs.rs

There is no mention of AsyncLineEventHandle anywhere on docs.rs. The only way to see this documentation is to build it locally with cargo doc, which is a bit of a pain, especially since many of the people using gpio-cdev are working on a raspberry pi, and if they are running the lite version of Raspbian, without a GUI, as I am, there is no browser, so cargo doc --open won't work. We need to manually go in and copy the docs directory to a different computer to read it.
When I did cargo doc on my project, it generated 16k files. Using scp to copy them over to my other computer was going to take like half an hour. I had to interrupt the scp copy operation, tar the 16k of doc html files, transfer it as one file, and unarchive it before opening it. The whole process took like 20 minutes. It would have been nice if I could have just found the documentation on docs.rs in seconds, as I do with most other documentation.

Someone asked about this on SO:
https://stackoverflow.com/questions/66519446/is-it-possible-to-see-feature-documentation-on-docs-rs/66519709#66519709

@tal-zvon Not great, but you might be able to use something like python3 -m http.server to just serve the doc html directory and browse the files from your workstation. Should be faster than SCP/tar-ball/etc.

Yup - that's one way to go. Would be nice if we didn't need to jump through hoops to see that section of documentation though.

Hey there, thanks for opening the issue. It's possible to use docsrs attributes (#[cfg_attr(docsrs, doc(cfg(feature = "async-tokio")))]) to show these features on docs.rs, which were recently added here, (hopefully) just need to publish a release.

Awesome! I'd close the issue, but it seems that it's set to auto-close when 0.4.1 is released. I guess I'll keep it open for people who run into the same issue, so they can find the answer easier, until the 0.4.1 is released, which will both solve, and auto-close this.
Thanks


Added performance test

This package will create some tests that generate data, in particular: system cpu load and system memory usage.
To test this program compile this package, you should remove COLCON_IGNORE and then run:
reset && colcon test --merge-install --event-handlers console_direct+ --packages-select vehicle_gateway_sim_performance

If you want to run a particular case:
reset && colcon test --merge-install --event-handlers console_direct+ --packages-select vehicle_gateway_sim_performance --ctest-args -R test_performance_position_control_vtol_9

When all tests are done, in another terminal execute:
gnuplot

plot './build/vehicle_gateway_sim_performance/test/system_collector_1.csv' using 1:2 with lines title 'CPU 1', \
     './build/vehicle_gateway_sim_performance/test/system_collector_3.csv' using 1:2 with lines title 'CPU 3', \
     './build/vehicle_gateway_sim_performance/test/system_collector_5.csv' using 1:2 with lines title 'CPU 5', \
     './build/vehicle_gateway_sim_performance/test/system_collector_7.csv' using 1:2 with lines title 'CPU 7', \
     './build/vehicle_gateway_sim_performance/test/system_collector_9.csv' using 1:2 with lines title 'CPU 9'
set term qt 2 size 500,300

plot './build/vehicle_gateway_sim_performance/test/system_collector_1.csv' using 1:3 with lines title 'Memory 1', \
     './build/vehicle_gateway_sim_performance/test/system_collector_3.csv' using 1:3 with lines title 'Memory 3', \
     './build/vehicle_gateway_sim_performance/test/system_collector_5.csv' using 1:3 with lines title 'Memory 5', \
     './build/vehicle_gateway_sim_performance/test/system_collector_7.csv' using 1:3 with lines title 'Memory 7', \
     './build/vehicle_gateway_sim_performance/test/system_collector_9.csv' using 1:3 with lines title 'Memory 9'

You will see some graphs like this:

I included the Gazebo RTF:

With more vehicles we need more time for the experiment because the RTF is lower

I added some improvements.  In this link you can find the data of the experiments:
system_collector_data.zip


Bar chart does not emit cross-filter if dimension is not set

A clear and concise description of what the bug is.
How to reproduce the bug
on superset 3.0.0
dashboard below before using cross filters

dashboard below after using cross filters on blue bar chart

same dashboard but using cross filters on the pie chart

same dashboard but using dimensions on bar chart

Expected results
cross filters work with bar chart without the need of dimensions
Actual results
cross filters on bar chart only work when using dimensions
Screenshots
If applicable, add screenshots to help explain your problem.
Environment
(please complete the following information):

browser type and version:
superset version: superset version
python version: python --version
node.js version: node -v
any feature flags active:

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[ ] I have reproduced the issue with at least the latest released version of superset.
[ ] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

Interesting, people have reported cross filters not working for bar chart but I hadn't seen anyone point out it was dimension-specific.  Thanks for the bug report.

@sfirke yes, because you can't reuse same column for metrics and dimensions on bar chart  you need to add another column to be used in dimensions

Running into the same problem (3.0) and also noticed it's dimension specific.
It seems that it tries to filter by the metric of the bar chart. When I click in one of the bars and hover over the filter applied in other charts it shows like this:

In contrast, doing it instead with a pie chart it shows on hover that is being filter by the correct dimension (name of salesperson in this case)

Someone gave a good description of the bug in Slack, with videos.  https://apache-superset.slack.com/archives/C0170U650CQ/p1698853673073119?thread_ts=1684878085.826729&cid=C0170U650CQ

I have also seen this issue.Is there any update?Or any solution?

It seems to be related to DIMENSIONS.If category axis is not a date,you can set like this.It works for me.

I think the ideal fix would be: bar charts without dimension emit cross-filters as if the series was a dimension.  That seems to be the behavior people expect.  I have no idea how hard that is to implement.

I've noticed this problem with 4.0 as well. Has anything changed recently? or a possible fix?
I want only one color, however if I use dimensions, the bar color will show every bar as every color.

@visharavana this issue persists through 4.0.0. I do not know of a fix or workaround at this time.

Hi..
This issue persits on 4.0.1.

Correct, it persists on the latest version 4.0.2 as well. I expect that if/when this is addressed, the pull request will link back here and close the issue.

Hi!
I didn't understand the X-Axis in this graph, this field is mandatory and at the same time it has no Cross-Filter action...
The funniest thing is that the legacy chart does not have this X-Axis and only the Dimensions field (and it is not mandatory).
Even because the X-Axis can become Y-Axis just by changing the format of the graph... so I didn't quite understand the meaning of this field. Dimensions field has everything we need.
I believe that making Dimensions mandatory instead of X-Axis will solve everything!
Thank you in advance for your efforts in resolving the issue. I'm just getting to know the tool now and it's very promising!

If category axis is not a date,you can set like this.It works for me.
(https://github.com/apache/superset/assets/49085215/7f1cd5c8-2267-45fe-92f3-927ac9898486) (https://github.com/apache/superset/assets/49085215/b404037f-c14f-46e4-a2e5-83343be0c4b5)

Thanks, but when doing this, char bars get too narrow

If category axis is not a date,you can set like this.It works for me.
(https://github.com/apache/superset/assets/49085215/7f1cd5c8-2267-45fe-92f3-927ac9898486) (https://github.com/apache/superset/assets/49085215/b404037f-c14f-46e4-a2e5-83343be0c4b5)

Thanks, but when doing this, char bars get too narrow

Oh.. I forget...
In CUSTOMIZE, you need choose "STACK" option...

If category axis is not a date,you can set like this.It works for me.
(https://github.com/apache/superset/assets/49085215/7f1cd5c8-2267-45fe-92f3-927ac9898486) (https://github.com/apache/superset/assets/49085215/b404037f-c14f-46e4-a2e5-83343be0c4b5)

Thanks, but when doing this, char bars get too narrow

Oh.. I forget...
In CUSTOMIZE, you need choose "STACK" option...

It works like a champ, thanks for your support

I would like this to work so that the cross filter field is the category on x axis not the value (i have categrized data by warehouse zone stock and when i click the zone i expect to filter the neighbour table to be crossfiltered only to the item stored in that zone... curently the cross filter sends the number of pallets in the zone not the zone category... I think this may be common problem with the one reported


[Backport to main] add more parameters for text embedding model (#640)

Backport PR #640 to main branch.
Description
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

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

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

Codecov Report

Merging #756 (bef97c8) into main (a08e482) will increase coverage by 0.33%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main     #756      +/-   ##
============================================
+ Coverage     84.55%   84.89%   +0.33%
- Complexity      986      990       +4
============================================
  Files            92       92
  Lines          3542     3542
  Branches        326      326
============================================
+ Hits           2995     3007      +12
+ Misses          410      399      -11
+ Partials        137      136       -1

Flag
Coverage Δ

ml-commons
84.89% <ø> (+0.33%)
:arrow_up:

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

Impacted Files
Coverage Δ

.../opensearch/ml/action/handler/MLSearchHandler.java
86.66% <ø> (ø)

...earch/ml/action/load/TransportLoadModelAction.java
89.32% <ø> (ø)

...ml/action/load/TransportLoadModelOnNodeAction.java
94.44% <ø> (ø)

...arch/ml/action/models/GetModelTransportAction.java
78.78% <ø> (ø)

...earch/ml/action/profile/MLProfileNodeResponse.java
89.18% <ø> (ø)

...pensearch/ml/action/profile/MLProfileResponse.java
94.73% <ø> (ø)

...pensearch/ml/action/stats/MLStatsNodeResponse.java
95.65% <ø> (ø)

...ensearch/ml/action/stats/MLStatsNodesResponse.java
94.73% <ø> (ø)

.../ml/action/syncup/TransportSyncUpOnNodeAction.java
93.93% <ø> (ø)

...search/ml/action/tasks/GetTaskTransportAction.java
78.12% <ø> (ø)

... and 27 more

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


conversations.kick always returns not_in_channel error

Description
I'm writing a Slack app with Bolt that provides a custom onboarding experience for new users in my Slack organization. One of the things this app does is kick users from any channel they try to join if they have not completed the onboarding process. Here's what my code roughly looks like:
app.event('member_joined_channel', async body => {
  // if user has not completed the onboarding process {
    app.client.conversations.kick({
      token: process.env.SLACK_BOT_TOKEN,
      channel: body.event.channel,
      user: body.event.user
    })
  }
});

But even though I am definitely in the channel, I keep getting a not_in_channel error when the app tries to kick me.
I even wrote a bit more code to make sure my app was aware I'm in a channel before kicking me:
app.event('member_joined_channel', async body => {
  // if user has not completed the onboarding process {
    const people = await app.client.conversations.members({
      token: process.env.SLACK_BOT_TOKEN,
      channel: body.event.channel
    })
    if (people.members.includes(body.event.user)) {
      app.client.conversations.kick({
        token: process.env.SLACK_BOT_TOKEN,
        channel: body.event.channel,
        user: body.event.user
      })
    }
  }
});

but I'm still getting the not_in_channel error when the app tries to kick me.
This leads me to believe there is a problem with Bolt that causes conversations.kick to always return not_in_channel, even when the user being kicked is in the channel.
What type of issue is this? (place an x in one of the [ ])

[x] bug
[ ] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
package version: 2.0.1
node version: 11.14.0
OS version(s): Ubuntu 16.04.6 LTS (project is hosted on glitch.com)
Steps to reproduce:

Create a Slack app & add it to your organization
Set up events & subscribe to member_joined_channel
Give your app the necessary scopes for conversations.kick
Create a Bolt app & add the above code to the app
Join a channel

Expected result:
Users are kicked from any channel they try to join if they have not completed the onboarding process.
Actual result:
Users are not kicked and the API returns a not_in_channel error, even if the user is in the channel.

The bot wasn't in the channel. Stupid mistake on my part.


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Latest Chrome update 62.0.3202.94 causing freeze if  is present

After latest Chrome update 62.0.3202.94 our clients (various platforms) reports that our app is unusable. I discovered really weird behaviour - when <form> with inputs is present on web page with <BootstrapTable>, first time clicking (focusing) the (first) input causes browser to lag/freeze. It depends on the count of rows/cells in bootstrap table. With complex table and slow computers the app is totally frozen.
I made JSFiddle to reproduce this - https://jsfiddle.net/5asms7j7/116/ . I tried to upgrade libs, but it seems that all react/react-bootstrap-table versions are affected. Only latest Chrome v62 is affected. Firefox/Safari works fine.
I couldn't find what's causing this behaviour, but in our case I've found really ugly temporary workaround. Seems like only first <input> is affected so if I put another small - not meant to be used - input before inputs.. following inputs works ok.

Oh I thought, I was the only one experiencing this problem. I have a table (with roughly 300 rows) and a modal windows which shows a form. Whenever I click on the first input available to type in, the page freeze. I will try your workaround, does it work if I put an hidden input?

Same here.
Using the developer tools performance tab, I noticed that the mouse down event lasts over 10 seconds.

@AllenFang do you have any clue why this is happening?

I tried to put a extra input in the beginning and then hide it using css. It works until I hide it.

I'll spend some time to check, however, like I mentioned on README, react-bootstrap-table already stop development and start forward to react-bootstrap-table2 and performance issue will be fixed on react-bootstrap-table2 only.

Thank you for looking into it Allen. I think this isn't common performance issue, the lag is larger than just slow table rerendering and makes forms ununsable. Maybe you could have an idea if BootstrapTable is doing something with focus event handling of (unrelated) form inputs on page .

Could not find a npm package with the exact name react-bootstrap-table2. Only found this. Is this the right one?

@MichalRemis ok, I'll look into on it
@mustafamamun react-bootstrap-table2 not released yet, repo: https://github.com/react-bootstrap-table/react-bootstrap-table2/tree/develop  it's still in progress.

hi all,  please add withoutTabIndex prop on BootstrapTable. This prop default is false and it will add tabIndex on td element but I dont know the root cause, still investigating,  thanks

not sure is about https://bugs.chromium.org/p/chromium/issues/detail?id=788933

thank you for solution @AllenFang , I confirm with withoutTabIndex freezing doesnt occur

👍

With the latest Chrome update v63 this got fixed and it is not a problem anymore. @AllenFang I think this issue may be closed.

thanks @MichalRemis, that's a good news, thanks your contribution :)


Type error because memberContext ist undefined

No matter what I do, I get the error message:

I got this same error, and found it was because I was putting the wrong Netflix password for the account at the prompt.

Precicely what @argoolsbee said. I can reproduce this error by entering incorrect login information but with the correct information, everything works just fine. If you'd like, you can open an issue in LBBO/node-netflix2 so that wrong login data throws its own error.
Please check your login data and close this issue if that fixed it.

Well my Login data was definitely correct, because I tried it multiple times either by handtyping or copy pasting the password.
Maybe the issue is that the profile I tried to access has has space characters?

I can't find it now, but I remember seeing the same error could occur if the netflix subscription was inactive.

@argoolsbee you are probably referring to #24 and yes, an inactive account would probably cause this exact or a very similar error message. As I don't have an inactive account, I can't check that.
@ogierm If your profile name includes a space, you can just surround it with quote marks like so:
netflix-migrate --email<EMAIL_ADDRESS>--profile "Your profile name" --export netflixData.json

However, the problem is your actual login (with email and password). If the login had completed, the script would have logged Switching to profile Thorsten. Since it didn't the crash happened during login. Does your password contain any characters that might confuse the power shell? I'm especially thinking of quotation marks or backslashes. If it does, could you try escaping them?
Also, could you try logging in to your account with your browser using the exact same data? If that doesn't work, netflix-migrate won't be able to login either.

I can login with the data it is definitely correct (I have to login per hand each time I open Netflix on my TV, because it constantly "forgets" the session)
Since I already deleted the account data I wanted to have I haven't touched it after writing the issue.
I guess I'll expand a bit on the situation I had. Basically my profile was the first profile on the account. We wanted to use Netflix on the TV, but the Netflix App on the TV doesn't support profiled (the TV is from 2012) so I had to basically give up my profile and start a new one. I wanted to use the tool to move over some data to my new profile.
I renamed my old profile to "Thorsten der TV" and added a new profile which had the name of my old profile. Besides the spaces the names didn't use any crazy characters.
I assumed the spaces were a problem so I tried wrapping it with quotation marks, only using the first word before the space etc. Basically everything you do when troubleshooting.
To be honest I don't really have a use case for this anymore so yeah. If you want me to test something specific I can do that, but beyond that I don't need it to work anymore.

I get this error as well. I'm sure credentials are correct. My account is deactivated though (doesn't happen for an activated account)

Has the issue been fixed? Let me know if I can help.

@KevinAlexandro I don't think so, as I haven't been able to reproduce the issue. Is there anything special about your account (such as unusual characters or the account being deactivated) and have you double-checked for correct spelling? I guess, any additional information that could help me reproduce this issue would help!

I am trying to migrate between an account that will be inactive on Monday and another one which is active. Also, I tried to migrate the other way around but I got the same error when the login takes place. I get the following error:

I thought that the issue was related with the netflix2 version so I tried updating with the latest available version I found, but the issue persisted. Then I tried with another OS (Ubuntu 20LTS and Windows 10), but the issue persisted.
Is there any chance I need an old version of netflix2, the one used when migrate Netflix was released?

I am trying to migrate between an account that will be inactive on Monday and another one which is active. Also, I tried to migrate the other way around but I got the same error when the login takes place. I get the following error:

I thought that the issue was related with the netflix2 version so I tried updating with the latest available version I found, but the issue persisted. Then I tried with another OS (Ubuntu 20LTS and Windows 10), but the issue persisted.
Is there any chance I need an old version of netflix2, the one used when migrate Netflix was released?

I am trying to migrate between an account that will be inactive on Monday and another one which is active. Also, I tried to migrate the other way around but I got the same error when the login takes place. I get the following error:

I thought that the issue was related with the netflix2 version so I tried updating with the latest available version I found, but the issue persisted. Then I tried with another OS (Ubuntu 20LTS and Windows 10), but the issue persisted.
Is there any chance I need an old version of netflix2, the one used when migrate Netflix was released?

First of all, sorry for taking so long to get back to you. I really hope you didn't loose your account's data because of this!
After doing some research, I was able to consistently reproduce the following behavior:

Use netflix-migrate in some way with correct login data. This command will succeed
Execute any other netflix-migrate command (even the exact same command as in 1) and it will fail.

The problem might be that Netflix doesn't allow two sessions from the same network. If I wait a few minutes after the successful command, I can execute another command successfully and the command after that will fail again (if I don't wait long enough). This might be fixable by logging off after every command, but I haven't gotten to that yet.
@KevinAlexandro and @jn-harvey (from #46): did your errors occur after running netflix-migrate once successfully? Perhaps the export worked, but then the import didn't work. And if so, could you try the import again and see if it works?

It's also worth noting that you can't have an open tab with Netflix while using netflix-migrate. That's definitely something I will add to the README

I could log in to the Netflix account successfully and had everything closed and executed the command but I have the same error message.

Ok, after doing some more experimentation, my current guess is that Google reCAPTCHA is stopping most (but not all) login attempts. Here's why I think that:

I just cannot for the life of me reproduce this issue consistently. Without changing any code at all (or anything else, for that matter - I just leave my computer for a few minutes), some requests go through and some fail.
After checking archive.org, it looks like Netflix added Google reCAPTCHA around Feb 27 2020, which roughly coincides with this issue gaining traction. It was opened in 2019, but it seems to have gotten way more common now and I also wasn't able to reproduce it back then.

I'm not really sure how to confirm this theory (please feel free to share any ideas you might have on that!). And even if I was able to confirm it, I have no idea how to get around Google's reCAPTCHA. One option might be to have the user enter their authUrl from an active browser session, but that would be a pretty terrible UX.
For now, all I can suggest is trying it again. Perhaps wait a few minutes in between requests, maybe even a day, I just don't know.  Please let me know how things go, though!

I have the same issue running Ubuntu in WSL2.
ill try to check after some time, maybe in an hour or in a day.

I was able to export the data fine but I get the error when I import.
I made sure that the password is correct.
May be I will wait for a day or two and try since I just created my netflix account

Got the very same issue. Export works fine, but I get the very same error every time I import. So far I tried:

waiting a day
using both the existing profile name and a nonexisting profile name as an import target

@LBBO
I really care about solving this issue, if you suggest some ways of debugging the problem (like getting more descriptive error) I could try to do some investigation myself. I'd really appreciate that :)

Hey @dstronczak!
Sorry for not getting back to you for a while. To be honest, I haven't looked into this issue for a while, so I might be forgetting details or my information might be outdated.
One thing you could do is to compare the Netflix response to responses from other sites that use reCAPTCHA. This way, we might be able to confirm that reCAPTCHA is actually the source of this issue.
As for other ideas: it might be possible to create a browser plugin (or even a bookmarklet?) that either brings this plugin's functionality to the browser or transmits any necessary data to the command line tool. Turning this project into a plugin actually makes a lot of sense, especially in terms of the UI. The user could then just log in on their own and the plugin would then execute pretty much the same scripts, I think. There might be issues with limitations for plugin's network traffic, though.
There are more important things for me to take care of in the next weeks and months, so it might take a while before I get around to looking into this change, though. I wouldn't expect it to happen before March.

Is it possible to access the API by other means and give the correct reCAPTCHA?

same problem here, I was able to export but when I tried to import i got undefined :( going to wait a few hours to try again

same problem here, I was able to export but when I tried to import i got undefined :( going to wait a few hours to try again

I´m facing the same issue.
Some how I was able to export, however after many tries, but import seams to be impossible.
Is there any way to edit netflix2.js to manually add a valid token after the browser login?

I´m facing the same issue.
Some how I was able to export, however after many tries, but import seams to be impossible.
Is there any way to edit netflix2.js to manually add a valid token after the browser login?

Hey everyone!
I really like the idea of manually logging in and then just adding some token as a parameter. I will look into it these next days and will get back to you once I have an update.

Hey everyone!
I really like the idea of manually logging in and then just adding some token as a parameter. I will look into it these next days and will get back to you once I have an update.

Alright, I have a working prototype!
Netflix-migrate will no longer email address and username. Instead, users must manually log in to Netflix, open the console, and copy the cookie by running document.cookie. That value will then have to be provided to netflix-migrate and everything should work (at least, it does for me).
Let me turn this into something actually usable (instead of hard-coding the cookie 😅 etc.) and then a beta version will be on its way.

Alright, I have a working prototype!
Netflix-migrate will no longer email address and username. Instead, users must manually log in to Netflix, open the console, and copy the cookie by running document.cookie. That value will then have to be provided to netflix-migrate and everything should work (at least, it does for me).
Let me turn this into something actually usable (instead of hard-coding the cookie 😅 etc.) and then a beta version will be on its way.

Hey everyone,
I just published a beta version that should fix this issue! You can install it with npm i netflix-migrate@beta or run it directly with npx netflix-migrate@beta .... However, please be aware that the rest of the commands have changed. For more information on that, please refer to the README I linked below.
Sadly, I was wrong about how the cookie could be extracted. Apparently, a part of the cookie is needed that cannot be extracted via JS, which I accidentally missed last evening. The cookie must, instead, be extracted from the network tab, which is a bit more complicated. I wrote some instructions on how to do this, though. You can find them in the README of the branch I'm currently working on.
If you'd like to, please feel free to give this a try and leave some feedback in #54 if you run into any trouble or if you find the instructions to be unclear in any way.

Hey everyone,
I just published a beta version that should fix this issue! You can install it with npm i netflix-migrate@beta or run it directly with npx netflix-migrate@beta .... However, please be aware that the rest of the commands have changed. For more information on that, please refer to the README I linked below.
Sadly, I was wrong about how the cookie could be extracted. Apparently, a part of the cookie is needed that cannot be extracted via JS, which I accidentally missed last evening. The cookie must, instead, be extracted from the network tab, which is a bit more complicated. I wrote some instructions on how to do this, though. You can find them in the README of the branch I'm currently working on.
If you'd like to, please feel free to give this a try and leave some feedback in #54 if you run into any trouble or if you find the instructions to be unclear in any way.

Hi, I tried out the beta and it is bugging over import due to several reasons it seems. At first it bugs out because of the version number in the JSON. Tried to edit that to 0.3.6 and it moves a bit forward but it halts here:
npx: installed 146 in 8.822s
Using new Netflix2 class!
Logging into Netflix using Cookies
Login successful!
Switching to profile Hilde
Successfully switched profile!
Reading data from hilde3_netflix.json
Found rating history and viewing history
Setting rating for The White Tiger: thumbs up
TypeError: Request with GET/HEAD method cannot have body
    at new Request (C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\node_modules\node-fetch\lib\index.js:1199:10)
    at C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\node_modules\node-fetch\lib\index.js:1409:19
    at new Promise (<anonymous>)
    at fetch (C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\node_modules\node-fetch\lib\index.js:1407:9)
    at Netflix.__request (C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\node_modules\netflix2\lib\netflix2.js:58:18)
    at Netflix.__apiRequest (C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\node_modules\netflix2\lib\netflix2.js:501:35)
    at Netflix.__setRating (C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\node_modules\netflix2\lib\netflix2.js:355:35)
    at Netflix.setThumbRating (C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\node_modules\netflix2\lib\netflix2.js:381:16)
    at C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\main.js:255:19
    at processTicksAndRejections (node:internal/process/task_queues:94:5)
Error: There was something wrong with your request. For more information, see previous log statements.
    at Netflix.__apiRequest (C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\node_modules\netflix2\lib\netflix2.js:509:13)
    at processTicksAndRejections (node:internal/process/task_queues:94:5)
    at async Netflix.__setRating (C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\node_modules\netflix2\lib\netflix2.js:355:24)
    at async Netflix.setThumbRating (C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\node_modules\netflix2\lib\netflix2.js:381:5)
    at async C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\main.js:255:5
    at async main (C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\main.js:51:4)
Error: Couldn't set thumb rating. For more information, see previous log statements.
    at Netflix.__setRating (C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\node_modules\netflix2\lib\netflix2.js:362:13)
    at processTicksAndRejections (node:internal/process/task_queues:94:5)
    at async Netflix.setThumbRating (C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\node_modules\netflix2\lib\netflix2.js:381:5)
    at async C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\main.js:255:5
    at async main (C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\main.js:51:4)
Error: Could not set thumb rating for The White Tiger. For more information, please see previous log statements.
    at C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\main.js:263:10
    at processTicksAndRejections (node:internal/process/task_queues:94:5)
    at async main (C:\Users\micro\AppData\Roaming\npm-cache\_npx\21008\node_modules\netflix-migrate\main.js:51:4)

memberContext seems to be renamed to loginContext. I can make the login working by changing memberContent to loginContext but then i have an issue about profile guid

I'm trying to use the cookie from my account on netflix2 and email and password  on netflix-migrate and  I'm receiving the same issue  Cannot read property 'data' of undefined

memberContext seems to be renamed to loginContext. I can make the login working by changing memberContent to loginContext but then i have an issue about profile guid

If this is the issue then the netflix2 part is the issue

Getting same error with master branch. Did lots of reading, about to download and try https://github.com/LBBO/netflix-migrate/tree/cookie-login.

OK, so is the solution for the import: use another computer? Don't have a 4K account? Have a simple profile name?
Or is it just open and not solved yet?
I would love to transfer my old profile to my new account

OK, so is the solution for the import: use another computer? Don't have a 4K account? Have a simple profile name?
Or is it just open and not solved yet?
I would love to transfer my old profile to my new account

OK, so I tried again, but this time a clean install from my debian server. Same result. So I shortened the command and simplified my password. To my surprise: import worked.
Unfortunately as indicated in the read me: only ratings were imported (as far as I can tell). Viewed or lists remain pending.
Hoping there is a update for this in the near future.
For.information: profile name was the same and new profile is a 4K account.

Since Netflix added native support for account transfer (including viewing history and My List 🎉), I'm archiving this project (see #73). Please refer to their official documentation if you still need to transfer you account data.


Remove frame.append since it is being deprecated.

evr_parser is using df.append to append rows. Append to list and concat instead.

@valentina-s : has this been fixed?

I forgot to say during our last meeting, but frame.append is what is causing a lot of the errors mentioned in #79, making this issue redundant and of smaller scope than #79. I will make a PR to #79 soon and close this issue once that PR has been reviewed and merged.

Resolved with #79


Fixes CB-11074: WKWebView configuration is not considered

I updated the configuration before the WKWebView initialisation in order to consider it.
It implies that the updateWithInfo method doesn't update it anymore. I'm not sure it is really a problem but I'd prefer @shazron to confirm.
Thanks for the reviews !

Any feedback ?

It still needs to be able to update it dynamically. I would refactor it to not remove the existing code, but create a new - (WKWebViewConfiguration*) createConfigurationFromSettings:(NSDictionary*)settings;
That way it can be used in both updateSettings and initWithFrame

As I commented in the base repo, this PR doesn't do what it says it does yet.
@shazron is it normal to change the settings after the WebView has already been inited? When? It seems like this feature (of changing settings after init) is creating unnecessary work for us.
My preferred approach would be to init the CDVPlugin/WebViewEngine instance, but not the WKWebView immediately, because for that to work, we need the commandDelegate to be set. The problem with that approach is that this check in CDVViewController.m will fail, causing self.webViewEngine to fallback to UIWebView:
if (!self.webViewEngine || ![self.webViewEngine conformsToProtocol:@protocol(CDVWebViewEngineProtocol)] || ![self.webViewEngine canLoadRequest:[NSURLRequest requestWithURL:self.appUrl]]) {
    self.webViewEngine = [[NSClassFromString(defaultWebViewEngineClass) alloc] initWithFrame:bounds];
}

The only thing I can image that would work in the current CDVViewController.m structure is that we init a WKWebView with default options on init, then re-init the WKWebView in pluginInitialize with the user's actual settings, because it's only then that commandDelegate.settings is finally available.

@ephemer not just settings (which as you said will probably not a frequent scenario), but setting the other delegates. updateWithInfo will be used for certain apps that need to set their own delegates on the webViewEngines -- a minor case, but I encountered this just last week.
I don't think self.webViewEngine will fail in the case of your 'preferred approach' you mentioned since that is just the CDVPlugin object wrapper that will always be instantiated unless WKWebView is not available: https://github.com/apache/cordova-plugin-wkwebview-engine/blob/33a75172a1450e8922788cd23382cc6ec33845c7/src/ios/CDVWKWebViewEngine.m#L47-L49
I'll comment on your #8 pull request.

A drive-by comment:
In our app, we ended up subclassing CDVWKWebViewEngine and creating another instance of WKWebView in -pluginInizialize so that we can use configuration values that we want. I'd suggest factoring out the code to create WKWebViewConfiguration to another method so that Cordova users can customize the logic by code rather than static configurations in a file. This means you can't create a web view instance in the initializer any more. This is OK because CDVWKWebViewEngine doesn't have to conform to the usual plugin semantics and you can add a factory method like -webView.

It seems that this bug prevents autoplay in   and  elements on IOS, because the config "MediaPlaybackRequiresUserAction" is never passed to the WkWebview instance and therefore always true (default in the code).
I needed to switch back to UiWebview in my meteor application because of this.
See https://github.com/meteor/meteor/issues/7173

I'm creating failing unit tests for this first, I think we have passed the point where we can argue about the implementation, but we need objective verification: https://issues.apache.org/jira/browse/CB-11496

Any update on this? In my opinion, this is a very severe issue with WKWebview.

I have failing unit tests that do objectively verify that as others have shown, yes, we can't update the configuration dynamically (I've tried all sorts of clever runtime tricks, to no avail).
The WKWebViewConfiguration.preferences (WKPreferences) can however be updated dynamically. I will update https://issues.apache.org/jira/browse/CB-11496 with the unit tests to reflect this, as well as document this. After doing so, I will re-review this pull request with that in mind.

@shazron : cool, thanks. Is there any short-time workaround known?

Please see #13, and let's continue discussion there.

Please provide feedback on #13, I will be pulling that in soon.


[Telegram->XMPP] Chokes with >2 forwarded messages at a time

Describe the bug
Only some of the simultaneously forwarded messages in Telegram are passed to XMPP.
Voice messages don't seem to be the issue, I tried with text messages too.

Expected behavior
Messages are not lost.
Screenshots/debug logs

Environment (please complete the following information):

OS: debian
Matterbridge version: latest

Could you run with -debug ?
And put the log in pastebin for example?

@XRevan86 I told you should report this!

Here's something from XMPP that never reached Telegram:
time="2020-05-31T19:58:46+03:00" level=debug msg=PING func=func1 file="bridge/xmpp/xmpp.go:210" prefix=xmpp
time="2020-05-31T19:58:56+03:00" level=debug msg="== Receiving<EMAIL_ADDRESS>Type:"groupchat", Text:"Some message", Subject:"", Thread:"", Ooburl:"", Oobdesc:"", ID:"aacda", ReplaceID:"", Roster:xmpp.Roster(nil), Other:[]string{"", ""}, OtherElem:[]xmpp.XMLElement{xmpp.XMLElement{XMLName:xml.Name{Space:"urn:xmpp:mam:tmp", Local:"archived"}, InnerXML:""}, xmpp.XMLElement{XMLName:xml.Name{Space:"urn:xmpp:sid:0", Local:"stanza-id"}, InnerXML:""}}, Stamp:time.Time{wall:0x0, ext:0, loc:(*time.Location)(nil)}}" func=handleXMPP file="bridge/xmpp/xmpp.go:237" prefix=xmpp
time="2020-05-31T19:58:56+03:00" level=debug msg="<= Sending message from person on xmpp.room to gateway" func=handleXMPP file="bridge/xmpp/xmpp.go:281" prefix=xmpp
time="2020-05-31T19:58:56+03:00" level=debug msg="<= Message is config.Message{Text:"Some message", Channel:"room", Username:"person"<EMAIL_ADDRESS>Avatar:"", Account:"xmpp.room", Event:"", Protocol:"", Gateway:"", ParentID:"", Timestamp:time.Time{wall:0x0, ext:0, loc:(*time.Location)(nil)}, ID:"aacda", Extra:map[string][]interface {}(nil)}" func=handleXMPP file="bridge/xmpp/xmpp.go:282" prefix=xmpp
time="2020-05-31T19:58:56+03:00" level=debug msg="=> Sending config.Message{Text:"Some message", Channel:"room", Username:"person"<EMAIL_ADDRESS>Avatar:"", Account:"xmpp.room", Event:"", Protocol:"xmpp", Gateway:"room", ParentID:"", Timestamp:time.Time{wall:0xbfad16743b6ab20c, ext:252551040982136, loc:(*time.Location)(0x311d640)}, ID:"aacda", Extra:map[string][]interface {}(nil)} from xmpp.room (room) to telegram.room (-1001220027368)" func=SendMessage file="gateway/gateway.go:421" prefix=gateway
time="2020-05-31T19:58:56+03:00" level=debug msg="=> Receiving config.Message{Text:"Some message", Channel:"-1001220027368", Username:"<person> "<EMAIL_ADDRESS>Avatar:"", Account:"xmpp.room", Event:"", Protocol:"xmpp", Gateway:"room", ParentID:"", Timestamp:time.Time{wall:0xbfad16743b6ab20c, ext:252551040982136, loc:(*time.Location)(0x311d640)}, ID:"71036", Extra:map[string][]interface {}(nil)}" func=Send file="bridge/telegram/telegram.go:59" prefix=telegram

I'd also need the next lines after the last message.

closed because of inactivity


docs: improve and organize openai docs

Related Issues/PRs

#xxx
What changes are proposed in this pull request?
Briefly describe the changes included in this Pull Request.
How is this patch tested?

[ ] I have written tests (not required for typo or doc fix) and confirmed the proposed feature/bug-fix/change works.

Does this PR change any dependencies?

[ ] No. You can skip this section.
[ ] Yes. Make sure the dependencies are resolved correctly, and list changes here.

Does this PR add a new feature? If so, have you added samples on website?

[ ] No. You can skip this section.
[ ] Yes. Make sure you have added samples following below steps.

Find the corresponding markdown file for your new feature in website/docs/documentation folder.
Make sure you choose the correct class estimators/transformers and namespace.
Follow the pattern in markdown file and add another section for your new API, including pyspark, scala (and .NET potentially) samples.
Make sure the DocTable points to correct API link.
Navigate to website folder, and run yarn run start to make sure the website renders correctly.
Don't forget to add <!--pytest-codeblocks:cont--> before each python code blocks to enable auto-tests for python samples.
Make sure the WebsiteSamplesTests job pass in the pipeline.

/azp run

Codecov Report

Merging #1937 (e2c0782) into master (f1b7b6a) will decrease coverage by 79.67%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #1937       +/-   ##
==========================================
- Coverage   86.87%   7.21%   -79.67%
==========================================
  Files         301     301
  Lines       15783   15783
  Branches      848     848
==========================================
- Hits        13712    1139    -12573
- Misses       2071   14644    +12573

see 288 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


error:0909006C:PEM routines:get_name:no start line

I'm using azure-functions-auth module with 1.4.2 latest version.
This is my code:

const validateJwt = require('azure-functions-auth')({
  clientId: '<myClientId>',
  clientSecret: '<myClientSecret>',
  domain: 'https://MyDomain.auth0.com',
  algorithms: ['RS256'],
});

const main = (context, req) => {
  context.log('token is valid. (you shouldn\'t log like that in production code)')

  return new Promise(resolve => {
    resolve('the function will return this exact string as body with a status code of 200')
  }).then(asyncResult =>{
    return asyncResult
  })
}
module.exports = validateJwt(main, true)

I'm using a machine to machine auth0 communication, when I call my azure function with a valid JWT TOKEN (validated with jwt.io) I'm always getting this error:
{
  "message": "error:0909006C:PEM routines:get_name:no start line"
}

I'm not sure but I think it's related to the encryption part for the RSA algorithme.
Thanks for your feedback and ideas to how handle this error.

Did you solve this issue?how?

I used an other solution instead of azure-functions-auth package

did you used different package?

I used the native packages for node and auth0 and set up the configuration with my azure function


MAINT aspect ratio of ASCII animation cannot be changed...

This is undesirable for several reasons.

It may look good on one computer, but looks awkward on another.
It's currently portable (txts > jpgs), but not a very good solution if we want to run on multiple computers/platforms.

I think this can be fixed by simply deleting the last 16 characters from every line, from every anim file. This has been done in 075860f0cfb9158a1642a140b426c2e0161f1cfd.