Weaver does not think this PDF has fields, but it does!

Revocation of Power of Attorney(20231023_152110).pdf

The problem is with has_fields; it only looks at the Annots, and assumes that each has a Type and Subtype. None of the fields in that PDF have a Type, just the Subtype.
I'm less familiar with the Annotation side of PDFs, but in FormFyxer when we get the actual fields, we look at the AcroForm instead. I haven't had problems with that before; I think that's the best solution.
@nonprofittechy, do you remember where you found the current code? (just so we know why it was implemented like that) https://github.com/SuffolkLITLab/docassemble-ALWeaver/commit/c1e171c1b8fdb28c024b61f100aec19ebf979152

I think we didn't have the code to check for fields initially, and this seemed like a good way to check that was lighter weight than just getting all of the fields and counting to see if there were more than 0.
Maybe we can combine the fast and slow methods, so it succeeds quickly but fails a little more slowly. Or fix the logic in this function--whatever seems safest.


[BUG] Task success on evaluator.py

Describe the bug
At file convlab2/policy/evaluate.py, on line 224 we need the code to be task_success[key] = [task_succ], right?
The way it is now, we do not count the task_succ when keyappears the first time.

@truthless11 @liangrz15 please check this

Done.


Is this still up to date?

Dear,
i tried out this software, i'm not a programmer but trader. I got some issues at multiple occasions trying to run this software. I'm wondering if this is still up to date or doesn't work anymore? It would have been a great tool otherwise...
FetchError: invalid json response body at https://manifold.markets/api/v0/markets?limit=1000&before=KxesWYLx2QAKmkQhQh1b reason: Unexpected token 'u', "upstream c"... is not valid JSON
at C:\Users\niel\market-maker\node_modules\node-fetch\lib\index.js:272:32
at processTicksAndRejections (node:internal/process/task_queues:95:5)
at async getMarkets (C:\Users\niel\market-maker\src\api.ts:17:33)
at async getAllMarkets (C:\Users\niel\market-maker\src\api.ts:31:35)
at async betOnTopMarkets (C:\Users\niel\market-maker\src\market-maker.ts:43:19)
at async main (C:\Users\niel\market-maker\src\market-maker.ts:38:5) {
type: 'invalid-json'
}
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
TypeError: Cannot read properties of undefined (reading 'filter')
at C:\Users\niel\market-maker\src\market-maker.ts:59:42
at processTicksAndRejections (node:internal/process/task_queues:95:5)
at async Promise.all (index 3)
at async batchedWaitAll (C:\Users\niel\market-maker\src\api.ts:118:19)
at async betOnTopMarkets (C:\Users\niel\market-maker\src\market-maker.ts:56:3)
at async main (C:\Users\niel\market-maker\src\market-maker.ts:38:5)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Ok seems like the api is not up to date anymore, hope it works now.

Oh sorry, yes I haven't looked at this code in a long time

No problem haha. I just made some changes that i found in the pull requests. Let's see if it works now.
I have not much programming experience, first time doing this but it works with the help of chatgpt. lol

I seem to still get an error:
error Command failed with exit code 1. at https://api.manifold.markets/v0/market/I9MKafdaZoi5mSMsCDmO reason: Unexpected token '<', "<!doctype "... is not valid JSON
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
at processTicksAndRejections (node:internal/process/task_queues:95:5)
C:\Users\niel\market-maker>(C:\Users\niel\market-maker\src\api.ts:10:30)
at async C:\Users\niel\market-maker\src\market-maker.ts:59:26
at async Promise.all (index 7)
at async batchedWaitAll (C:\Users\niel\market-maker\src\api.ts:125:19)
at async betOnTopMarkets (C:\Users\niel\market-maker\src\market-maker.ts:57:3)
at async main (C:\Users\niel\market-maker\src\market-maker.ts:39:5) {
type: 'invalid-json'
}
So seems there are now these markets that have the type: "doc". It's like just a question you can answer on. I will have to make the script skip these.
Hope this works:

ok i give up, tried lot of things.


Shell prompt does not appear

Shell prompt does not appear when docker command is invoked from PowerShell. It works from CMD shell.

I don't think, this is by design and is true for all. I usually call all these commands from powershell

Please may you confirm it works for you on Windows Server 2016 with the
latest module "DockerMsftProvider" and the latest package "docker" ?
I have used a fresh new AWS EC2 machine, launched from AMI
"Windows_Server-2016-English-Full-Containers-2016.10.18 (ami-d08edfc7)",
first following this guide:
https://msdn.microsoft.com/en-us/virtualization/windowscontainers/quick_start/quick_start_windows_server
and then this guide:
https://msdn.microsoft.com/en-us/virtualization/windowscontainers/quick_start/quick_start_images
Mirek Svoboda | +1<PHONE_NUMBER>  | Skype: xsvobo10
On 29 October 2016 at 20:39, rs38<EMAIL_ADDRESS>wrote:

I don't think, this is by design and is true for all. I usually call all
these commands from powershell
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/Virtualization-Documentation/pull/452#issuecomment-257108330,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHcWNAk-Vd1QODou5Zw6866lpnUYGakgks5q45LZgaJpZM4KkFNN
.

I can confirm that at least a EC2 nano Host behaved very strange during my own tests when powershell remoted. It experienced sustained cpu 100% after that docker run commands with nano docker image.

Thanks. Seems dust needs to settle a bit.
Mirek Svoboda | +1<PHONE_NUMBER>  | Skype: xsvobo10
On 29 October 2016 at 22:17, rs38<EMAIL_ADDRESS>wrote:

I can confirm that at least a EC2 nano Host behaved very strange during my
own tests when powershell remoted. It experienced sustained cpu 100% after
that docker run commands with nano docker image.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/Virtualization-Documentation/pull/452#issuecomment-257113570,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHcWNK2YcBuDpKuWUP9M_BOLgfDyW7WQks5q46nwgaJpZM4KkFNN
.

Can you share an example of what you see when the prompt is missing? I'm getting prompts after running docker run -it microsoft/nanoserver from PowerShell for example.

I see nothing at the moment when I run docker run -it microsoft/nanoserver from PowerShell. When I stop the container then command prompt returns. In W10 on my workstation it works fine, but it is HyperV isolation, not Windows Containers isolation. I tend to believe this is AWS AMI specific issue. I did not install WS2016 elsewhere than on AWS yet, so cannot confirm it.
I will close this issue now and would reopen if able to reproduce elsewhere.


tech(GRA-726): refactor join network functions

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


metadata_startup_script

TL;DR
Commands in metadata_startup_script are not properly executed once the compute instance is launched.
Expected behavior
Flask and the dependencies should be installed.
Observed behavior

terraform apply
go to VM instances and log in with SSH in the browser.

xyz@flask-vm:~$ pip list
Package             Version
------------------- -------
dbus-python         1.2.16
distro-info         1.0
pip                 20.3.4
python-apt          2.2.1
setuptools          52.0.0
unattended-upgrades 0.1
wheel               0.34.2

no flask installed.
4. If I installed it manually, create the app, and launch it, it works as expected.
Terraform Configuration
provider "google" {
  credentials = file("credentials.json")
  project = "NAME_OF_PROJECT"
  region  = "us-east1"
  zone    = "us-east1-b"
}

# [START compute_flask_quickstart_vpc]
resource "google_compute_network" "vpc_network" {
  name                    = "my-custom-mode-network"
  auto_create_subnetworks = false
  mtu                     = 1460
}

resource "google_compute_subnetwork" "default" {
  name          = "my-custom-subnet"
  ip_cidr_range = "<IP_ADDRESS>/24"
  region        = "us-east1"
  network       = google_compute_network.vpc_network.id
}
# [END compute_flask_quickstart_vpc]

# [START compute_flask_quickstart_vm]
# Create a single Compute Engine instance
resource "google_compute_instance" "default" {
  name         = "flask-vm"
  machine_type = "f1-micro"
  zone         = "us-east1-b"
  tags         = ["ssh"]

  boot_disk {
    initialize_params {
      image = "debian-cloud/debian-11"
    }
  }

  # Install Flask
  metadata_startup_script = "sudo apt-get update; sudo apt-get install -yq build-essential python3-pip rsync; pip install flask"

  network_interface {
    subnetwork = google_compute_subnetwork.default.id

    access_config {
      # Include this section to give the VM an external IP address
    }
  }
}
# [END compute_flask_quickstart_vm]

# [START vpc_flask_quickstart_ssh_fw]
resource "google_compute_firewall" "ssh" {
  name = "allow-ssh"
  allow {
    ports    = ["22"]
    protocol = "tcp"
  }
  direction     = "INGRESS"
  network       = google_compute_network.vpc_network.id
  priority      = 1000
  source_ranges = ["<IP_ADDRESS>/0"]
  target_tags   = ["ssh"]
}
# [END vpc_flask_quickstart_ssh_fw]

# [START vpc_flask_quickstart_5000_fw]
resource "google_compute_firewall" "flask" {
  name    = "flask-app-firewall"
  network = google_compute_network.vpc_network.id

  allow {
    protocol = "tcp"
    ports    = ["5000"]
  }
  source_ranges = ["<IP_ADDRESS>/0"]
}
# [END vpc_flask_quickstart_5000_fw]

# Create new multi-region storage bucket in the US
# with versioning enabled

# [START storage_bucket_tf_with_versioning]
resource "random_id" "bucket_prefix" {
  byte_length = 8
}

resource "google_storage_bucket" "default" {
  name          = "${random_id.bucket_prefix.hex}-bucket-tfstate"
  force_destroy = false
  location      = "US"
  storage_class = "STANDARD"
  versioning {
    enabled = true
  }
}
# [END storage_bucket_tf_with_versioning]

Terraform Version
Terraform v1.3.7

Additional information
No response

Hi @philippemiron, sorry for delay. I have tested this issues multiple times.
It's only my speculation but I believe the background process that is just slow to realise that flask is installed. In future, please use sudo journalctl -u google-startup-scripts.service for checking logs of startup script. Hope this help.


Split list and clear commands to Student and Tutor List

Things done in this PR:
Split clear commands into student/tutor so that we do not delete them together when we wish to clear only one list. This gives users greater flexibility in controlling their data stored.
Split Clear Command into Student and Tutor List

[x]  Updated ClearCommand
[x]  Added ClearCommandParser
[x]  Updated AddressBookParser to properly take in ClearCommand
[x]  Updated AddressBookParserTest
[x]  Updated AddressBookTest
[x]  Updated AddCommandTest
[x]  Updated ClearCommandTest
[x]  Added ClearCommandParserTest
[x]  Change the Addressbook clearing system to clear tutor and student list separately

[x]  Added setTutorData and setStudentData methods in Model.java
[x]  Added resetTutorData and resetStudentData

Split list commands into student/tutor so that the application does not override one of the student/tutor lists after a find command when the user did not want/expect to.
Split List Command into Student and Tutor List

[x]  Updated ListCommand
[x]  Added ListCommandParser
[x]  Updated AddressBookParser to properly take in ListCommand
[x]  Updated LogicManagerTest
[x]  Updated ListCommandTest
[x]  Updated AddressBookParserTest
[x]  Added ListCommandParserTest

Additional Updates

[x]  Update user guide with new command descriptions

Closes #95

Codecov Report

Merging #100 (44d4216) into master (7834385) will increase coverage by 0.20%.
The diff coverage is 87.71%.

@@             Coverage Diff              @@
##             master     #100      +/-   ##
============================================
+ Coverage     76.41%   76.61%   +0.20%
- Complexity      570      582      +12
============================================
  Files            85       87       +2
  Lines          1755     1800      +45
  Branches        201      204       +3
============================================
+ Hits           1341     1379      +38
- Misses          364      369       +5
- Partials         50       52       +2

Impacted Files
Coverage Δ

.../java/seedu/address/logic/commands/AddCommand.java
86.95% <0.00%> (-0.55%)
:arrow_down:

...eedu/address/logic/parser/DeleteCommandParser.java
100.00% <ø> (ø)

src/main/java/seedu/address/model/Model.java
100.00% <ø> (ø)

...ava/seedu/address/logic/commands/ClearCommand.java
78.57% <76.92%> (-21.43%)
:arrow_down:

...java/seedu/address/logic/commands/ListCommand.java
81.81% <80.00%> (-18.19%)
:arrow_down:

.../seedu/address/logic/parser/AddressBookParser.java
100.00% <100.00%> (ø)

...seedu/address/logic/parser/ClearCommandParser.java
100.00% <100.00%> (ø)

.../seedu/address/logic/parser/ListCommandParser.java
100.00% <100.00%> (ø)

src/main/java/seedu/address/model/AddressBook.java
97.82% <100.00%> (+0.32%)
:arrow_up:

...rc/main/java/seedu/address/model/ModelManager.java
95.94% <100.00%> (-2.63%)
:arrow_down:

... and 3 more

Continue to review full report at Codecov.

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


Add license file

Related to issue ( https://github.com/dask/dask-jobqueue/issues/1 ), item 2.
Includes the Dask license file (BSD 3-Clause) and makes sure it will be packaged.
Sets the date to begin in 2018 as that seems to be the beginnings of this codebase even on the Pangeo side.

If we are all good with this, could someone please merge?

Done.  @jakirkham do you not have permissions to merge here?

Thanks. I do. However I didn't feel comfortable self-merging this. Hope that makes sense.


HTTPUpgradeError

Hi @daltoniam!
Found a lot of Starscream.HTTPUpgradeError at Crashlytics console:
NSLocalizedDescription: The operation couldn’t be completed. (Starscream.HTTPUpgradeError error 0.)
Error is not permanent, sometimes all works fine.
I'm wondering, is it smth inside the SDK or troubles at the BE side?

Figured out the issue. It was caused by expired access token


Authentication does not work with 2-Step verification

Moved from GitbookIO/gitbook#1564

Closed in favor of https://github.com/GitbookIO/feedback/issues/250


Error executing MDX: org.olap4j.mdx.WithMemberNode cannot be cast to org.olap4j.mdx.WithSetNode

First of all - great stuff! Cool to see more open source OLAP and XML/A
I just deployed olaper.war v 0.0.2 to my apache tomcat (8.0.29).
I run this on windows and I could succesfully configure olaper by doing this:

copy cubes.json and tables.json from https://github.com/Wondersoft/olaper/tree/master/example, and store them in webapps/olaper

edit olaper/WEB-INF/web.xml and configure the OlapDriverConnectionString param of Olap4jXmlaServlet so it reads:
jdbc:olapserver:C:\apache-tomcat-8.0.29-windows-x64\webapps\olaper\cubes.json

edit olaper/cubes.json and set catalogs.physical_schema to
"C:\\\\apache-tomcat-8.0.29-windows-x64\\\\webapps\\\\olaper\\\\tables.json"

copy Mysql JDBC driver to tomcat's lib directory.

I tested with xavier XML/A client (https://github.com/rpbouman/xavier) and this can discover the schema, the cube, the measures, the dimensions, and the hierarchies just fine. But then it executes this:

<SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/" SOAP-ENV:encodingStyle="http://schemas.xmlsoap.org/soap/encoding/">
SOAP-ENV:Body

WITH MEMBER [Measures].[One] AS 1
SELECT {[Measures].[One]} ON COLUMNS
FROM [Store sales]
WHERE ([Customer].[Member card].[All],[Customer].[Name].[All],[Date].[Date].[All],[Date].[Year by quarters].[All],[Product].[Products by brands].[All],[Promotion].[Name].[All],[Store].[Stores by types].[All])

Provider=Mondrian;DataSource=Foodmart;
Foodmart
Multidimensional
TupleFormat
SchemaData

</SOAP-ENV:Body>
</SOAP-ENV:Envelope>
and the response is

<SOAP-ENV:Envelope xmlns:SOAP-ENV="http://schemas.xmlsoap.org/soap/envelope/" SOAP-ENV:encodingStyle="http://schemas.xmlsoap.org/soap/encoding/" >
SOAP-ENV:Header
</SOAP-ENV:Header>
SOAP-ENV:Body
SOAP-ENV:Fault
SOAP-ENV:Server.00HSBD02
XMLA MDX execute failed
Mondrian

<XA:error xmlns:XA="http://mondrian.sourceforge.net">
00HSBD02
The Mondrian XML: org.olap4j.mdx.WithMemberNode cannot be cast to org.olap4j.mdx.WithSetNode
</XA:error>

</SOAP-ENV:Fault>
</SOAP-ENV:Body>
</SOAP-ENV:Envelope>

Roland, thank you for posting this bug. Apparently there is a bug in MDX parser, will fix it.
Olaper was tested with Saiku and manual-written MDX only, i will try it with Xavier

Hi Aleksey, thanks for picking this up! Much appreciated :)
Best way to get set up is xavier probably to download the zip (
https://github.com/rpbouman/xavier/raw/master/dist/xavier.zip).
Extract it, and then put the extracted xavier directory in your webapps
dir.
Then, you still need to tell xavier where to find the xmla service.
I will probably change the xavier discovery code so it can find olaper by
itself in a short while.
But for now you can edit xavier/resources/html/index.html, and set the
xmlaUrl property of the xavierOptions object.
Best regards,
Roland
On Wed, Mar 23, 2016 at 7:24 AM, Aleksey<EMAIL_ADDRESS>wrote:

Roland, thank you for posting this bug. Apparently there is a bug in MDX
parser, will fix it.
Olaper was tested with Saiku and manual-written MDX only, i will try it
with Xavier
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/Wondersoft/olaper/issues/1#issuecomment-200204067

--
Roland Bouman
blog: http://rpbouman.blogspot.com/
twitter: @rolandbouman
linkedin: http://www.linkedin.com/profile/view?id=5142800&trk=tab_pro
Author of "Pentaho Solutions" (Wiley, ISBN: 978-0-470-48432-6
http://tinyurl.com/lvxa88) and "Pentaho Kettle Solutions" (Wiley, ISBN:
978-0-470-63517-9 http://tinyurl.com/33r7a8m)


How to provide both arc cost and node cost constraints in capacitated vehicle routing problem (CVRP)?

How to minimize the cost and path in routing problem when we have variable costs for each capacity at the location(node)?
Cost for delivery at each location is different and is not directly proportional to the distance. Objective is to optimize the route and number of vehicles while considering the cost at the nodes.
Example:
Let's say I have 5 locations with depot A.(A,B,C,D,E)
num_vehicles = 5
vehicle_capacities = {20,20,20,20,20}
demand_at_location = {10, 20, 5, 15, 10}
demand_cost_at_location = {50, 100, 200, 100, 200}
distance_matrix = {
{0, 20, 40, 30, 10},
{20, 0, 60, 50, 30},
{40, 60, 0, 70, 50},
{30, 50, 70, 0, 40},
{10, 30, 50, 40, 0}
}
Based on the cvrp ortools example, it optimizes the route and capacities and assigns route to the vehicles. In the example vehicle only considers arc cost (which is distance) to the nodes and minimizes for the route. However, there is no way we can give cost constraints to the nodes. How do we provide node cost for the location so that route is optimized both for cost and the distance along with vehicle capacities.
Please help in providing cost constraint to the nodes.
Thanks,
Hanu

You can duplicate each node into an in_node and an out_node.
With all incoming arcs going to the in_node, and all outgoing coming from the out_node, and only one arc from in_node to out_node.
Then you add the node cost on this arc.

Thanks Laurent for the quick response.
Is there any example that we can refer to for this kind of implementation?
Please let me know.
Thanks,
Hanu
On Sun 8 Sep, 2019, 12:49 PM Laurent Perron<EMAIL_ADDRESS>wrote:

You can duplicate each node into an in_node and an out_node.
With all incoming arcs going to the in_node, and all outgoing coming from
the out_node, and only one arc from in_node to out_node.
Then you add the node cost on this arc.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1560?email_source=notifications&email_token=AIYUTUZ2LVXD35OWSHSY3GLQISRQFA5CNFSM4IUSJWGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6FJU3I#issuecomment-529177197,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIYUTU7S5H3O3NZ4OVOSKH3QISRQFANCNFSM4IUSJWGA
.

Not this directly, but duplicating node has been explained a lot in the
mailing list.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le dim. 8 sept. 2019 à 19:06, hanuhyd<EMAIL_ADDRESS>a écrit :

Thanks Laurent for the quick response.
Is there any example that we can refer to for this kind of implementation?
Please let me know.
Thanks,
Hanu
On Sun 8 Sep, 2019, 12:49 PM Laurent Perron<EMAIL_ADDRESS>wrote:

You can duplicate each node into an in_node and an out_node.
With all incoming arcs going to the in_node, and all outgoing coming from
the out_node, and only one arc from in_node to out_node.
Then you add the node cost on this arc.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<
https://github.com/google/or-tools/issues/1560?email_source=notifications&email_token=AIYUTUZ2LVXD35OWSHSY3GLQISRQFA5CNFSM4IUSJWGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6FJU3I#issuecomment-529177197
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AIYUTU7S5H3O3NZ4OVOSKH3QISRQFANCNFSM4IUSJWGA
.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1560?email_source=notifications&email_token=ACUPL3KBP5RHWOMHEQVAVZDQIUWIFA5CNFSM4IUSJWGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6FUU6Y#issuecomment-529222267,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACUPL3PBYG6ANVKUFJVQ2DTQIUWIFANCNFSM4IUSJWGA
.


Loose Dep's

Anybody know if there's anything depending on chai or deep-equal?
I've had both removed from my own build for a while... No issues, and only seeing one instance of either across the entire repo.
Guillotine?
Other potentials: immutable, invariant, kerberos, node-libs-browser
Note: duplicate dev-dependency redux-mock-store in dependencies.
Sorry if any of these are obvious necessities, my build is pretty light.

Kill it!!
On Mon, 12 Sep 2016 02:28 John Rodler<EMAIL_ADDRESS>wrote:

Anybody know what's depending on chai?
I've had it removed from my own build for a long time... Never had any
issues, and I only see one instance of "chai" across the entire repo.
Guillotine?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/reactGo/reactGo/issues/504, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADnMbQ_5ZVYcDlYQ9Gk8Qg-oMLa-WYotks5qpCwTgaJpZM4J6CyL
.

Hehe nice work
On Mon, 12 Sep 2016 06:49 choon ken ding<EMAIL_ADDRESS>wrote:

Kill it!!
On Mon, 12 Sep 2016 02:28 John Rodler<EMAIL_ADDRESS>wrote:

Anybody know what's depending on chai?
I've had it removed from my own build for a long time... Never had any
issues, and I only see one instance of "chai" across the entire repo.
Guillotine?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/reactGo/reactGo/issues/504, or mute the thread
https://github.com/notifications/unsubscribe-auth/ADnMbQ_5ZVYcDlYQ9Gk8Qg-oMLa-WYotks5qpCwTgaJpZM4J6CyL
.

anything to reduce the number of greenkeeper PRs

Yes. Agreed. Kill them with 🔥
If you get to test them and have no issues, just merge them!

@caranicas greenkeeper seems like overkill, it clogs-up the commit log

I agree with @jrodl3r npm update -S is enough


[MM-40582] Use user_updated event's user to state, to prevent extra GetMe() ping

Summary

With user_update event now sending unsanitized user to source, ping to server to get user is redundant.
Ticket Link

https://github.com/mattermost/mattermost-server/issues/19361
Related Pull Requests

Has server changes (https://github.com/mattermost/mattermost-server/pull/19444)
Has mobile changes (please link here)

Release Note

Refactored `user-update` websocket event handler to prevent extra get request to server for unsanitized user.

Hello @plant99,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Adding myself to this PR since I have a question I wanted to ask about this implementation. I'll post it on the server PR since it's more to do with the serverside logic though.

/update-branch

/e2e-test

Triggering e2e testing with options:
<nil>

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/147887

E2E report looks good, no PR related failures.


Stormblessed: Watch skyalerts.io

Stormblessed requests the watch of the watch_keyword skyalerts\.io. See the MS search here and the Stack Exchange search here.

skyalerts\.io has been seen in 1 true positive, 0 false positives, and 0 NAAs.

!!/approve

Approved by double-beep in Charcoal HQ


Enable int64 tests for torchlib | test(torchlib)

Enable int64 tests and marked all failures as xfail with information to follow up.

Fix annotation for div and logsumexp
Fix bernoulli for int64 inputs
Use the ONNX native Bernoulli to implement bernoulli

cc @wschin

There are testing errors about min/minimum in WIN.

Those are known and will be addressed by #761

Merging now. Future comments will be addressed in a follow PR. Thanks!


Point-cloud-based methods in Open3d-ML

Create a new subclass of SensoryAgent that returns a Sample subclass representing the objects and volume bounding boxes.
Steps:

[ ] 1. Experiment with pre-trained models in Open3D-ML
[ ] 2. Define appropriate Sample subclass that contains 3d volume bounding box information.
[ ] 3. Load  weights from select object-detection and semantic-segmentation models from Open3D-ML and upload to HuggingFace
[ ] 4. Implement SensoryAgent interface.
[ ] 5. Add unit tests and create Gradio App with examples for qualitative inspection.

I'll take this
@sebbyjp Just to be clear, the SensoryAgent subclass implements the sense() function which takes the PointCloudSample type as an input parameter, uses an object detection model from open3d-ml to get the bounding box, and returns it as a SensorReadingSample type?

Yes, sir! sense() is an alias for act() in this case.


Document pointer cast semantics

#13 documents which casts are possible, as well as the semantics of the casts that don't involve pointers. For pointers it's not clear if a complete set of semantics can be given until the unsafe guidelines/memory model are determined, but maybe some statements can be made.

Including *const T as *const u8 casts where T: ?Sized.


[image_picker] Implement methods for picking both images and videos simultaneously.

This PR implements both pickImageOrVideo and pickMultiImageAndVideo on both Android and iOS.
This PR is still in draft mode due to the lack of unit tests, as well as the dependency on the changes to the platform interface: #4675. I will be adding tests later.
Relevant issue:

flutter/flutter#89159

Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [relevant style guides] and ran [the auto-formatter]. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the [CLA].
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the [pub versioning philosophy], or this PR is [exempt from version changes].
[x] I updated CHANGELOG.md to add a description of the change, [following repository CHANGELOG style].
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is [test-exempt].
[x] All existing and new tests are passing.

It sounds like this is obsolete with https://github.com/flutter/plugins/pull/4675 now, so I'm going to close this PR. Please don't hesitate to submit it again if I'm wrong about this.

@BeMacized Is this PR read for review?

No, it's not (also because the changes to the platform interface have not been merged yet).
It will be superceded by two new PRs that I'll submit in a moment. Closing.


feat: send chain exchange requests in parallel

Summary of changes
Changes introduced in this pull request:

send chain exchange requests in parallel (maximum N=2 at the same time)
Changing SHUFFLE_PEERS_PREFIX from 16 to 100
[x] manual testing on calibneet
[x] manul testing on mainnet

Hopefully this will mitigate or maybe even fix the below error.
Error = TipsetRangeSyncer(NetworkMessageQueryFailed("ChainExchange request failed for all top peers"))

Corresponding lotus code comment
// Try the request for each peer in the list,
// return on the first successful response.
// FIXME: Doing this serially isn't great, but fetching in parallel
//  may not be a good idea either. Think about this more.

Reference issue to close (if applicable)

Closes Related to https://github.com/ChainSafe/forest/issues/2244
Other information and links

I do wonder if this will double the network activity. Time will tell.

It should double the chain exchange requests


chore: ensure hover styles get applied

Current dependencies on/for this PR:

next

PR #235

PR #236

PR #237

PR #238

PR #239

PR #240

PR #241

PR #242


Bug with Prompts.time

Hello,
In french, with Webchat, Prompts.time doesn't work with Aujourd’hui but works well with Aujourd'hui.
(the difference is in the characters ’ ' )
Unfortunately, the first version (Aujourd’hui) is suggested by default on ios 11.
Could you fix this bug please ?
Thanks

That sounds like an SDK problem to me. Please report this bug to https://github.com/microsoft/botbuilder. I'm closing this issue, but let me know if I have this wrong.


AOC (up to 2022) Merge

Merged because I don't intend to come back to these problems for a while.

Rejected because this didn't capture the solutions.


Update background image for board to work on mobile displays

Consider using a flat color or a cartoonish image so the page isn't so busy.


add greek sample

Regex chars:
a-zA-ZÀ-ÖØ-öø-ȳͰ-Ͽἀ-ῼ

Sample story: https://www.greek-language.gr/certification/dbs/teachers/show.html?id=5
Figure out the dictionary

from Cynthios in slack:
Modern Greek
https://en.wiktionary.org/wiki/###
https://www.wordreference.com/gren/###
*https://www.deepl.com/translator#el/en/###

Done in the dev branch, looks good to me. :-)


Added V syntax support

V is a new general purpose language, inspired mainly by Go, Rust, Pascal and C.
Main site: https://vlang.io/
Github: https://github.com/vlang/v

🤣

With the default micro color theme, it looks like this:

@theAkito thanks for the feedback I guess ...

@theAkito thanks for the feedback I guess ...

@spytheman It's not about your syntax highlighting. It's about this scam of a language.

Why, what's wrong with V?

The V language is not a scam. I write programs in it every day and like it very much (mostly CLI tools and small games). As any young language it has weaknesses, but it progresses very fast too.

Why, what's wrong with V?

I follow V since pretty much its inception. Basically, to summarize everything I researched about it in a very short manner:
V's author has a big mouth, makes huge claims, but proves almost nothing.

I write programs in it every day and like it very much (mostly CLI tools and small games).

"Scam" does not mean it does not work at all. You can buy a scammy USB stick and if it does not give the 2TB, but only 2GB space, then it WORKS, but it is still a scam.

As any young language it has weaknesses, but it progresses very fast too.

There are tons of other young languages that this applies to. Difference is, they don't lie about their capabilities.

give us some evidence

You can visit the v repo and judge by the commits and the working examples, that proves almost nothing is not true.
https://github.com/vlang/v/tree/master/examples
Anyway. I like V, I like micro, I like emacs too and many others. I want to use them together (micro is a very VIM replacement for when I edit stuff on remote servers, due to its small size and very easy distribution). I would like to use micro to edit .v files => here is a syntax highilighting file for that.

give us some evidence

No problem, read the locked Github Issues on the V repository. There are plenty of examples showing how people just ASK how something is supposed to work and he shuts down the issue because he does not want (i.e. cannot) answer these questions, because his claims are too huge.

You can visit the v repo and judge by the commits and the working examples, that proves almost nothing is not true.
https://github.com/vlang/v/tree/master/examples
Anyway. I like V, I like micro, I like emacs too and many others. I want to use them together (micro is a very VIM replacement for when I edit stuff on remote servers, due to its small size and very easy distribution). I would like to use micro to edit .v files => here is a syntax highilighting file for that.

As I said, I follow V since pretty much ever. I know that stuff and read that stuff.

Why do you laugh then?
Do you find it funny that people want to use it with micro?

Why do you laugh then?
Do you find it funny that people want to use it with micro?

Absolutely not. I just find it funny that so many people use V without knowing the stuff that happens behind the scenes. When I first saw V, I thought too "amazing" but then I started researching, reading up on how it is supposed to work and the scam became more evident the more I read about it.

Just to be clear:
This commit should be merged. Nothing against using V in general. Everyone can decide for themselves, it's your choices.
My point is: if you want a great language, don't use V. It has nothing to do with Micro.

This is getting very off-topic, I'd recommend dropping it to avoid getting the issue locked.

This is getting very off-topic, I'd recommend dropping it to avoid getting the issue locked.

I agree.
Again: just criticising the language, not you guys!

And what happens behind the scene? As one of the contributors to v, I've read the entire code base, and know its limitations, strong features and weaknesses - the language works for me and I like it, just as I like micro. I do like to use them together, and I thought that others may like that too => this PR.

@Calinou you are probably right. I think that everything has its place under the sun, and did not expect that my PR will be welcomed in such a way...
Have a nice day.

No problem, read the locked Github Issues on the V repository. There are plenty of examples showing how people just ASK how something is supposed to work and he shuts down the issue because he does not want (i.e. cannot) answer these questions, because his claims are too huge.

Hasn't happened once.
Please go ahead and post links to such issues.
Also since it's a scam, it would be easy to have a list of why it's a scam:

Otherwise it's just empty words.

@medvednikov Stop spamming this issue. You know exactly how much you closed and how many people you muted on Github, by closing their threads when you could not answer how your language is supposed to work out your empty claims.
If you make these claims, go give PROOF.

Stop spamming this issue

I've literally posted once.
The one spamming is you.

You know exactly how much you closed and how many people you muted on Github, by closing their threads when you could not answer how your language is supposed to work out your empty claims.

Again, this hasn't happened once. You didn't provide any proof.

@medvednikov
You are an amazing con-artist. This is my honest compliment to you. But that is it.
Exhibit 1:
https://github.com/vlang/v/issues/35

I've literally posted once.

The one spamming is you.

If telling people the truth until you are called as the V-Police is "spam" then sure, I am the biggest spammer on earth.

Let's just wait until V becomes more developed and see how good it becomes

Exhibit 1:
vlang/v#35

Yes, an issue saying the language is 90% not what it's advertised, and the reasoning is...
you can't do JSON encoding without an AST (you can), and no way Vid and V are written in V (they are), playground crashing. Ok.
Do you have any actual evidence of V being a scam?
Again, just a small list is enough:
1
2
3

It's too early to decide if it's a scam

@dullbananas agreed. Things can change by time and I hope it will.

@theAkito there are a lot of things that were going on behind the scenes with attackers / trolls / infini-nude-Thanoses (don't ask). It has been quite a ride and frankly @medvednikov is doing a great job, not only in getting people psyched / excited, but also in delivering his promises. Perhaps the only thing I can see so far that has really proven a bit much is due to such a big (good and ill) reception the contributions and general interaction have seemed to be a bit taxing and have seemingly caused some delays / slippage in schedule of deliverable features... but they are coming and he's working tirelessly on it.
Let us refrain from making ad hominem attacks - especially when someone really is working so hard to keep his ALL of his promises.

@ylluminate
What you said is partially true.
Except I made no ad hominem attacks. I was the addressee of one, as you can see above.
That said, I find it dishonest to stigmatise valid criticism as "trolling".

I said there was trolling going on - I did not specifically call out any particular person for doing so. I was saying these terms to give context of the environment that could have lead to perhaps over-or-under-reacting to certain posts, tickets/issues, etc. that might have caused you a point of pain.
I personally believe it is always important to try to put yourself in someone else's shoes before hammering on them in such ways, especially in a more personal fashion.
In a world focused on tearing others down and offering criticism (even so called constructive criticism which deserves an entire treatise in and of itself), I think we should try to be more patient and measured in our wrath or ridicule.
/end_offtopic_remarks

I personally believe it is always important to try to put yourself in someone else's shoes before hammering on them in such ways, especially in a more personal fashion.

That is what I do, which is evident in the way, that I express myself verbally the first time after watching V-lang for so much time. Before today, I did not express anything on any V-lang related discussion platform, because I was watching and reading.

I think we should try to be more patient and measured in our wrath or ridicule.

If the language's author said something among the lines of "maybe at some point in the future, we will have feature X and feature Y" there would be no discussion. The point is, that the author was showing it off like the language already has these capabilities and not only that, even does it better than that. At the same time he said he had this game, this program, etc. even though none of it was released at the time. These were empty repositories. So anyone who just read the descriptions with the links provided, thought that it is already released and working as described. But the truth was, that you had to follow the links to see the truth. Nothing was out at the time and nothing done or working even close to how it was advertised. Which is dishonest and false advertising. Such things, like this example, made the project smell fishier and fishier. It is not an obvious rip-off like some Asian wife delivery website, but it consists of many many many small things that just aren't right.
All I (and others) want is honesty. If he would describe the features of his language honestly and truthfully, everyone would be alright with it and no one would complain.
If you pay for a car with 6 wheels and after your payment you see it only has 4, then you have the right to be upset and at least ask the seller about that issue.

we should make a code of conduct

At the same time he said he had this game, this program, etc. even though none of it was released at the time. These were empty repositories.

What game, what program?

If he would describe the features of his language honestly and truthfully, everyone would be alright with it and no one would complain.

What features were not described truthfully? Still waiting for the list.

we should make a code of conduct

Yeah, contributors laughing at others's contributions, insulting, calling people scammers and con artists without any evidence, goes against the spirit of open source.

This isn't the right place to argue about V. The syntax file looks good so I merged it, though the .v extension does conflict with Verilog (for now .v will detect as V but it may be best to change this in the future since I think Verilog is more widely used, and I use Verilog frequently). Users can manually give one precedence over the other though so it's not a huge deal.

What game, what program?

What features were not described truthfully? Still waiting for the list.

Now you are just plain trolling.

Yeah, contributors laughing at others's contributions, insulting, calling people scammers and con artists without any evidence, goes against the spirit of open source.

I rest my case. This proves my point.
This and most other comments show, that I am being personally attacked by a lot of fanboys of a wannabe language, just for criticising their favourite language. As usual, their author tries to make the critic look bad, etc. and pretends to not know what he is talking about, even though the author met plenty of people criticising him about the exact same issues.
Due to all these random ad hominem attacks against me and purposeful bad mouthing of my bad person, without any roots in reality, I see no point in joining a war, I never wanted to join in the first place. Therefore I will ignore this discussion from now on. Additionally, I fully agree with micro's author, that this is not the right place to discuss this.
Just let me say one last thing, and that will be it:
With your malicious attitude, your language will never thrive. No (especially open source) project can be successful in a good way, if their leader is a huge asshole, that thinks he is God and no one else should criticise what he is doing. You should finally accept that there are people on earth that by accident happen to not have the same opinion(s), as you do. Get over that, learn from it and your project will be successful. As long as you keep being demeaning and just keep attacking anyone who you remotely could possibly identify as a critic, your language will be dead in less than a couple of years.

He is actually being sincere and wanting input. I'm not sure where you're coming from or how you're coming up with this crap, but he's definitely not being malicious. I think you're having cultural and personality conflict issues here as far as I can tell here and thus probably why you're making these strange assumptions and not getting to the root of the issue / trouble you're having with him.

Let's stop talking about this. I will follow anyone who doesn't comment starting now


Create Stratified Sampling

Hi,
I am wondering if there exist any SQL examples to generate stratified samples.
Currently, METHOD could only be (uniform | hash)
The paper claims the stratified samples are generated in a tricky way using two-pass approach, but could you please provide more details about how to generate the stratified sample?
Thanks,

I don't think the stratified samples are in the open-source version. But there's a plan on adding it to the open source version too, hopefully in a week or two.

Thanks for the info.


Correct datetime types on PostgreSQL

Hi,
we're using requery for a project that supports both SQLite and PostgreSQL. We have a few columns that we defined as Date on Java level which works perfectly with SQLite. However on PostgreSQL, it creates a column of type date and therefore silently throws all time information away! What is the recommended way to store a datetime with requery on PostgreSQL?
Best
Raphael

You should use java.sql.Timestamp which has both a date and timestamp in Postgres


Created CODE_OF_CONDUCT.md

I have created the code of conduct.
fixes. #19

Hi @suryanshsk ,
I request you to please add the gssoc-extd, hacktoberfest-accepted and level 1 label to this PR. So, that there will be no conflict in my gssoc-extd in future.
Thank you for your time.


Improve error messages for failed plugin loads

This improves the error messages you get when elvish fails to load a go module.
Currently elvish almost always just fails with a NoSuchModule error, even when the error was actually a library mismatch or a missing Ns symbol in the .so file.

Forgot to run the linters/tests. Updated it now.

Thanks, there are some style issues that I'll fix up after merging.


Can't use copy & paste when launch vscode from tmux session

VSCode Version: 1.17.2
OS Version: Osx 10.12.6

Steps to Reproduce:

open tmux session
open vscode from tmux session code
Can't use cmd+c and cmd+v in vscode

Reproduces without extensions: Yes

I have a similar issue ,
it also does not load the correct font for some reason , I am using "Fira Code" , it just load a system font .
I am also using vscodevim and it has missed up some of the key binding .

@a-eid sounds like a different problem, we can can discuss in a separate issue if you want.

@Tyriar I have the same issue .
However I am not sure if the font issue related to it or not.
@lifez please confirm if your custom fonts are loading correctly  . Thabks

When you're in tmux with mouse mode on a regular mouse event will be sent to the terminal program to be handled (ie. tmux). There is an override to force "regular selection" by holding alt, this should show a selection color the same as when you are outside of tmux. This normally works fine and allow the use of cmd+c/v even in mouse mode but it broke recently due to https://github.com/sourcelair/xterm.js/issues/1090

I think @lifez and @a-eid are describing a different issue than what you fixed. I have the copy/paste problems in the editor after launching vscode from a tmux session, and the font is missing.

Yeah I linked to the wrong issue, removing from October.

weirdly enough after applying my linux machine tmux config , it worked fine (font + copy pasting) ,
I will make a gist of the config and share it in a few min

This will solve it: https://medium.com/@joaomoreno/persistent-terminal-sessions-in-vs-code-8fc469ed6b41

@bpasero I'm not so sure about that? This issue is about launching VS Code from tmux which then breaks copy and paste.

@Tyriar I just did that from my tmux setup and copy/paste works.

Here is my tmux.conf:
set -g mouse on
set -g status off

set-option -g pane-border-fg colour235 #base02
set-option -g pane-active-border-fg colour240 #base01

set -g history-limit 10000

set-option -g default-command "reattach-to-user-namespace -l bash"


All preliminary contracts written and tested

This looks great. Let's go through this when you are in the office.
I'd like some clarity on the setup for the migrations, deploying this to a network and running the tests. Then let's update the readme and then we can merge this.


Support sending secure files with empty message

Allow sending a secure file with an empty message from both the welcome screen and the chat transcript screen. Also added a check for file upload limit, just as the secure conversations chat transcript is doing.
MOB-1904

!merge


Optional parameter to force column Data Type

To the extension class "ExcelDataReaderExtensions" was added a new optional parameter. This new parameter allow specify the value type associated to a column.
The structure of this parameter is a list of ExcelDataType.
The definition of an ExcelDataType is:
public class ExcelDataType
{
public string ColumnName { get; set; }
public Type ColumnType { get; set; }
}

Hi and thank you! Unfortunately I believe this is a problematic change. In particular because of Mono, which apparently has slight differences in its DataSet implementation from the "official" ones. See #409 / #73 / #30.
Currently by default ExcelDataReader tries to autodetect the type for DataColumn.DataType and the linked issues are reporting problems when there are nulls in the data. I think we should fix the autodetector so that it always works (i.e: assign nullable types in case of nulls, also configure pr sheet), rather than providing a type override mechanism which can easily break (on mono)
If you can tell a bit more about your scenario, maybe we can come up with a better solution?

Hi, this is my scenary.
When I read and Excel document that contains an value of type Decimal, it always is save in the DataTable returned by the extensor method as a Double. But in this case I want it as a Decimal cause I'll save the DataTable into the database as Decimal.
This is just a specific scenario. But a very common situation is that someone want to read the value from the Excel with an specific data type. Otherwise he/she must retrieve the information from the original DataTable, perform a casting or type conversion and then create a new DataTable to be stored into the Database. This is memory consuming solution obviously.

Some kind of new ExcelDataTableConfiguration.TransformValue(reader, columnIndex) callback could be a flexible way to handle this and other scenarios. F.ex I had a use case where I wanted to apply formatting on every value in the DataSet, which is similar to yours, but converting to DB types instead. Happy to accept a PR for this.
Alternatively, if you need a quick resolve you might find it easier to build the dataset manually yourself, f.ex by copying the AsDataSet method into your project and adapt+simplify as necessary. AsDataSet() is theoretically quite simple, but comes with lots of bells and whistles.

Was this ever introduced to the library?
I have some issues with excel files that are produced by machine, but later modified by users. So for instance, some fields are numbers written as text, so I read them as text, but when users have been in and modified these files, they are read as Double.
I'm using "UseColumnDataType=true", but these columnts with varying types are then returned as object, instead of what I expected.
I think an TransformValue method would help here, but it looks like it doesn't exist?  (columnIndex wouldn't cut it though, if a map to columnName doesn't exists though)


Revise dates in the 2017 cores (and other cores) to be a bit more aligned, using lots more subjective decisions. Record these decisions, and record the information in new LUT files.

current status is that I've copied the depth-age model files and added the suffix mod.csv to each. I have identified a few events in the 2017-2019 cores that need refining. For example: in 2017, I will choose winter (20170201) to be the first moment that there is a depleted value in d18O.


What is the recommended version of nodejs?

I've been using node 11 in my development. Since it's not a LTS, I decide to downgrade to node 10. But the app no longer works.  The following code provides a minimal reproduction of the problem:
'use strict';
require('@babel/register');
require('babel-polyfill');

const XVIZWriter = require('@xviz/builder').XVIZWriter;
const XVIZMetadataBuilder = require('@xviz/builder').XVIZMetadataBuilder;

const writer = new XVIZWriter();
const builder = new XVIZMetadataBuilder();
builder
  .stream("/vehicle_pose")
  .category('pose')

const metadata = builder.getMetadata();
writer.writeMetadata("/tmp", metadata);

With node 11 it works fine and generates /tmp/1-frame.glb successfully. But with node 10 it results in the following error:
/home/caros/Developer/test/node_modules/@loaders.gl/gltf/dist/es5/glb/glb-builder.js:204
    var textEncoder = new _core.TextEncoder('utf8');
                        ^

TypeError: _core.TextEncoder is not a constructor
    at GLTFBuilder._convertObjectToJsonChunk (/home/caros/Developer/test/node_modules/@loaders.gl/gltf/dist/es5/glb/glb-builder.js:204:25)
    at GLTFBuilder._createGLBBuffer (/home/caros/Developer/test/node_modules/@loaders.gl/gltf/dist/es5/glb/glb-builder.js:161:28)
    at GLTFBuilder.encodeAsGLB (/home/caros/Developer/test/node_modules/@loaders.gl/gltf/dist/es5/glb/glb-builder.js:67:19)
    at encodeBinaryXVIZ (/home/caros/Developer/test/node_modules/@xviz/builder/dist/es5/writers/xviz-writer/xviz-binary-writer.js:34:22)
    at writeBinaryXVIZtoFile (/home/caros/Developer/test/node_modules/@xviz/builder/dist/es5/writers/xviz-writer/xviz-binary-writer.js:38:23)
    at XVIZWriter.writeMetadata (/home/caros/Developer/test/node_modules/@xviz/builder/dist/es5/writers/xviz-writer/xviz-writer.js:104:53)
    at Object.<anonymous> (/home/caros/Developer/test/main.js:15:8)
    at Module._compile (internal/modules/cjs/loader.js:776:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:787:10)
    at Module.load (internal/modules/cjs/loader.js:653:32)

The project is attached.
test.tar.gz
However, the kitti example still works fine. Maybe I've done something wrong in my project, but I   really fail to see it. Could you please provide some help?

I will try your example. I would expect that our modules include the appropriate polyfill where necessary, although you may be able to work around it by including it in your project for the time being.

I would expect that our modules include the appropriate polyfill where necessary,

Summary: Yes, going forward, my recommendation is that XVIZ modules should import @loaders.gl/polyfills directly for any modules that are mainly intended to run under node.
Long story:
The initial version of 2loaders.gl/core automatically installed the TextEncoder polyfills under Node.js, however the polyfills we use are quite big and there were subtle bundler-dependent issues that sometimes caused unnecessary polyfills to be bundled in the browser which caused hard to solve issues, so we moved them to a separate polyfills module so that the user is in control.
There is information on this in the v1.1. notes in https://loaders.gl/docs/upgrade-guide, however it is belated since these changes happened as hot-fixes in the v1.0.x release series.
In retrospect we should probably have done this fix as a major version bump rather than a patch.

I will try your example. I would expect that our modules include the appropriate polyfill where necessary, although you may be able to work around it by including it in your project for the time being.

Thanks for the help. I required @loaders.gl/polyfills in my project and now it works.

I was able to reproduce. I'm waiting on some dependent changes and should have a fix in for our next beta release. in a day or so

@pkulijing I tested with the latest with node v10 and I didn't run into the problem.  We updated a dependency that I believe may have been causing an issue with some globals.


[New chain]: Moonbeam 1.4.1

Summary

https://github.com/safe-global/safe-deployments/pull/563
https://github.com/ethereum-lists/chains/blob/master/_data/chains/eip155-1284.json

Chain ID
1284
Chain detail URL
https://chainlist.org/chain/1284
RPC URL
https://rpc.api.moonbeam.network
Blockscout Client URL
No response
Etherscan Client URL
https://moonbeam.moonscan.io
Etherscan Client API URL
https://api-moonbeam.moonscan.io
Version
1.4.1
Address (Master copy)
0x41675C099F32341bf84BFc5382aF534df5C7461a
Deployment Tx hash (Master copy)
0xa3fd87e4a25f7c7f0f115f8a7cd1bd9790f1b78c4afaac3f552303af6b3f9621
Block explorer URL (Master copy)
No response
Address (Master copy L2)
0x29fcB43b46531BcA003ddC8FCB67FFE91900C762
Deployment Tx hash (Master copy L2)
0xd396e840189b2500d5c5efdc0fdb9ff8b88729cd476525d7e8fb259984b589bf
Block explorer URL (Master copy L2)
No response
Address (Proxy factory)
0x4e1DCf7AD4e460CfD30791CCC4F9c8a4f820ec67
Deployment Tx hash (Proxy factory)
0x65afce9aca6c892e17d897675e5ed72f5815613687c4576caab4d879efd1d917
Block explorer URL (Proxy factory)
No response

/execute


Scoped hooks aren't working in demo app

It appears scoped hooks aren't working in the demo app. When you run all tests, the before hook that states it only runs during the Google tests is run during all tests.
Google test suite

Facebook test suite

repro steps: clone the demo app, run cypress, and run all tests.
I was having issues with scoped step definitions in a project and I wanted to make sure it wasn't a weird configuration on my end. I'm wondering if this is a bug with scoped step definitions and if I should file an issue against the main repo.
Thanks

Hello,
this is the problem described here:
https://github.com/TheBrainFamily/cypress-cucumber-preprocessor/issues/139
which is an acknowledged bug in cypress itself:
https://github.com/cypress-io/cypress/issues/3323
I will add an info here to NOT use "run all" in the GUI - I don't think this is something worth doing anyway - once you have a realistic number of features, you wouldn't be running them all through GUI anyway (and it works properly when you do cypress run)

Thanks so much for the quick reply @lgandecki. I'm glad to hear it was just a problem with how I was running tests. I'll go ahead and close the issue.


Rewrite Design Values Wiki

The wiki page is framed as "Bocoup Design Values" and I think it might make sense to frame this in a way similar to how we framed the design process.

I cleaned up the language here. The wiki page has now been changed to "Open Design Values."
I suggest  that we keep this, close the issue for now and continue to iterate on the wiki over the next year. 👍 or 👎


Typo in React Docs

Authentication Hooks
Email and Password Sign-Un

Thank you @quarkstars ! We will soon migrate our documentation page where this issue has already been fixed.
I am linking the PR to this issue so it will be closed once we have the new site up and running.


@JBean relationships

@JBean
public class AbstractTypeA {
    protected boolean hasSomething;
}
@JBean
public class AbstractTypeB {
    protected TypeA type;
}
The above example does not compile, it errors out because the TypeA class is not generated.

@DHuckaby Thank you for issue. I hope it will be fixed in the next release.

Fixed.


fix RHP animation on Safari

Details

hotfix for the bug with RHP screens
Stop using interpolator from react-navigation for RHP because it seems to be the source of the bug on Safari
instead use an interpolator that was already written and used in another place

Fixed Issues

$ https://github.com/Expensify/App/issues/44765
PROPOSAL:
Tests

verify opening RHP and navigating within it
flow: FAB -> Assign Task -> Next
flow: Report -> ReportDetails -> ShareScreen
flow: Search -> click on any row -> ReportDetails (click on header inside RHP)

Offline tests

QA Steps

verify opening RHP and navigating within it
flow: FAB -> Assign Task -> Next
flow: Report -> ReportDetails -> ShareScreen
flow: Search -> click on any row -> ReportDetails (click on header inside RHP)

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari
Chrome
https://github.com/Expensify/App/assets/3929868/4695b37f-010b-40ff-91f1-532759f3c1d7
Safari
https://github.com/Expensify/App/assets/3929868/da379567-0cba-4db5-8e22-c9b90b787f7a

MacOS: Desktop

@mountiny this is the fix, please take a look - perhaps we could run a build here and test it?
I have tested locally on Safari and did not get the bug
CC @ShridharGoel @s77rt

cc @ishpaul777 (https://github.com/Expensify/App/issues/44765#issuecomment-2209428056)

I see, it's a shame that this doesn't solve the bug.
However the funny thing is that this solves it for me 100%. I literally cannot trigger the bug anymore which means I can't try to fix it anymore 😓
Check my video. Interestingly if you specifically look at the period around 0:25 - 0:40 you can see the very similar faulty animation - but for me it always comes back to correct layout. However indeed there is something there still problematic. But I still think its these interpolators that break it.
https://github.com/Expensify/App/assets/3929868/a68fd607-2b2f-453d-8244-8c8898efba34
Not sure what to do next, since I cannot reproduce it anymore - either via clicking or via keyboard and I tried multiple types of RHP modals.
One more thing to try: There is a few more places where we use CardStyleInterpolators.forHorizontalIOS - people that still see the bug could try to remove it everywhere and see if this helps?
Btw I'm on mac M3 Pro, Sonoma v14.4.1, Safari v17.14.1

I can still reliably repro on this branch as well when going from the task title to confirmation page
https://github.com/Expensify/App/assets/36083550/95bdff9d-5c11-49bb-b712-fa2cde2e31de
@Kicu nobody in SWM can reproduce this?

This PR is now using the custom interpolators only for Safari as a temporary solution, until we can fix all the animations correctly.

@ishpaul777 PR updated, pushed a with commit with usememo

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/mountiny in version: 9.0.5-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/mountiny in version: 9.0.5-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
failure ❌

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/Julesssss in version: 9.0.5-13 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/thienlnam in version: 9.0.6-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


APP-2781: DatafeedV2 Implementation

I would recommend to put a description body in the git commit logs so we get a bit more context about the change, why it has been done and the technical choices made
And also to reference the corresponding ticket (it seems we do it even if the repo is open, the tracker is internal) (usually in the commit title SDK-XXX: blahblah) and in the PR branch name

Were you able to test it against a bot generated by the bdk?

Read datafeed method should be handled in a new thread instead running on main thread which will block other waiting tasks forever.

@symphony-youri I really do not know how to write an unittest for a infinite loop which presents in DatafeedEventsService class. Do you any suggestions?

As we have a way to stop it, we could have a test that runs the service, mock the client and then call the stopDatafeedService method to stop the thread

@symphony-youri I just updated my PR. Can you review this PR please?

does it work in case of failure? because to me an error won't fail the test
why do we need this to start in a thread? can't we just create the service calls some methods and then the service at the end of the test?

yes tried it locally if you change the asserts the tests still pass

Here is the kind of test I would write:
@Test
    public void readDatafeedTest() {
        DatafeedV2 id = new DatafeedV2();
        id.setId("123");
        when(datafeedClient.listDatafeedId()).thenReturn(Collections.singletonList(id));
        when(datafeedClient.getAckId()).thenReturn("ack_id_string");

        DatafeedEventsServiceV2 service = new DatafeedEventsServiceV2(datafeedClient);
        service.stopDatafeedService();

        verify(datafeedClient, times(1)).listDatafeedId();
// this is in the loop so it will happen multiple times before the thread is stopped
        verify(datafeedClient, atLeastOnce()).readDatafeed("123", "ack_id_string");
    }

without inhering the bot test
no need to json files and mock webservices since we can mock the datafeedclient
DatafeedEventsServiceV2 can be changed to avoid the dependency on Symbotclient and only pass the required info (the datafeed client, bot user id, errors timeout)

Thanks for the rebase, regarding the commit's description we are more interested in the why rather than the what, for instance we will see with the diffs that unit test were added or that DatafeedClient is now final but we are more interested why we did that (for instance to prevent users of the API to subclass)
This blog post is interesting to read https://chris.beams.io/posts/git-commit/


Ecovac broken in 0.96.0

Home Assistant release with the issue:

0.96.0
Last working Home Assistant release (if known):
0.95
Operating environment (Hass.io/Docker/Windows/etc.):

Hass.io
Component/platform:

Ecovacs
Description of problem:
Ecovac component fails to load after update
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
ecovacs:
  username: !secret ecovacuser
  password: !secret ecovacpass
  country: us
  continent: na

Traceback (if applicable):
Thu Jul 18 2019 09:21:12 GMT-0600 (Mountain Daylight Time)
Component refresh requests failed to reach VacBot. Will try again later.

<iq type="error"<EMAIL_ADDRESS><EMAIL_ADDRESS>id="ce98a50f-e88b-428b-be5f-67f4b53db874-16C"><error type="wait" code="404"><recipient-unavailable xmlns="urn:ietf:params:xml:ns:xmpp-stanzas" /></error></iq>
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/websocket_api/commands.py", line 121, in handle_call_service
    connection.context(msg))
  File "/usr/src/homeassistant/homeassistant/core.py", line 1150, in async_call
    self._execute_service(handler, service_call))
  File "/usr/src/homeassistant/homeassistant/core.py", line 1172, in _execute_service
    await handler.func(service_call)
  File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 194, in handle_service
    required_features
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 316, in entity_service_call
    future.result()  # pop exception if have
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 337, in _handle_service_platform_call
    await getattr(entity, func)(**data)
  File "/usr/src/homeassistant/homeassistant/components/vacuum/__init__.py", line 301, in async_turn_on
    partial(self.turn_on, **kwargs))
  File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/src/homeassistant/homeassistant/components/ecovacs/vacuum.py", line 144, in turn_on
    self.device.run(Clean())
  File "/usr/local/lib/python3.7/site-packages/sucks/__init__.py", line 416, in run
    self.send_command(action.to_xml())
  File "/usr/local/lib/python3.7/site-packages/sucks/__init__.py", line 413, in send_command
    self.xmpp.send_command(xml, self._vacuum_address())
  File "/usr/local/lib/python3.7/site-packages/sucks/__init__.py", line 482, in send_command
    c.send()
  File "/usr/local/lib/python3.7/site-packages/sleekxmppfs/stanza/iq.py", line 235, in send
    raise IqError(result)
sleekxmppfs.exceptions.IqError: <iq type="error"<EMAIL_ADDRESS><EMAIL_ADDRESS>id="ce98a50f-e88b-428b-be5f-67f4b53db874-16C"><error type="wait" code="404"><recipient-unavailable xmlns="urn:ietf:params:xml:ns:xmpp-stanzas" /></error></iq>
Failed to call service vacuum/turn_on. <iq type="error"<EMAIL_ADDRESS><EMAIL_ADDRESS>id="ce98a50f-e88b-428b-be5f-67f4b53db874-16C"><error type="wait" code="404"><recipient-unavailable xmlns="urn:ietf:params:xml:ns:xmpp-stanzas" /></error></iq>

Additional information:

I apologize, this error was not due to 0.96.0, but was my own mistake.
I'm now on 0.96.1, runs fine, there is no issue...


[Vue warn]: Failed to resolve component: strike

Vue version
3.3.4
Link to minimal reproduction
https://play.vuejs.org/#eNqNUUtPGzEQ/iuuL2kldq2KnqIQ0VZILQdAgMTFl7A7SQzrh8azIdJq/ztjL4QlPMTNM994vsd08ncI5aYFOZWzWKEJJCJQG+baGRs8kugEwlL0YoneigmPTrTTrvIukrBxJY4S/n3yD5rGixuPTf1t8kO7mRrW8SIuCGxoFgRcCTFb/5x3Xf7c9zPFVe4aF1oSm8L6GpojLRnXcoAiobmH+Tpx8OKhShy7vfJADnoLuwjlXfSOHXXps34CopZTkTupxz5SreWaKMSpUlXt+Bszmw2WDki5YNUxjylsHRkLRe3t8WF5WP5StYk0bpcQbXGL/iEC8hItD0Y0ipsbwALB1YCAX6Xd+zam3oPe0Cf2XrueQ6HIl1qa1V4klbfBNIDngQxf8lU0Cw754TT3CFvYeanWUN2/07+L28HTBUJWNvJPC1wBDfDJ1Rls+b0D+dBt83SGD8BLiL5pk8Zh7E/rapY9mstq/+cLG7e6jidbAhefTSWhOY08n+/x9xPrL3I57V2K/SN7DA/N
Steps to reproduce
Open the playground link and see the warning.
What is expected?
No warning using . Vue should recognize it as a native tag.
What is actually happening?
Warning using  as Vue does not recognize it as a native tag.
System Info
SFC Playground

Any additional comments?

Markdown use ~~strike~~/strike officially and that translated to
Previously reported #6841.

 is not supported in HTML5. Similar tags for HTML5 would be either <s> or .  Either one works for the purpose of strikiethrough, but <s> is commonly used to show text that is no longer accurate and <d> is used to show text that has ben deleted. An alternative would by setting the CSS text-decoration propery to line-through for a text element.
Examples:
The <s> tag:
<s>This text is no longe accurate</s>
The <de> tag:
This section has been deleted
Using the  text-decoration property:

Why does this have a line through it?

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/strike

This element is deprecated in HTML 4 and XHTML 1, and obsoleted in the HTML Living Standard. If semantically appropriate, i.e., if it represents deleted content, use  instead. In all other cases use .


How to Config & Send Message to Telegram from SharpTelegram

Hi dear friend
How do I configure and send a message by sharptelegram library
Please refer to the help or documentation
Thanks to your human brother Mehdi
email<EMAIL_ADDRESS>

hi
please send example send TelegramClient Example for me
my email<EMAIL_ADDRESS>

Hi dear friend
How do I configure and send a message by sharptelegram library
Please refer to the help or documentation
tankyou
email<EMAIL_ADDRESS>

hi
please send example send TelegramClient Example for me
my email<EMAIL_ADDRESS>

this is a fake library , haven't any document or help , its not work

HAVE YOU READ THE WARNING ON THE MAIN PAGE?
"WARNING: the library is not implemented yet (consider donation to speed up development process)."

How much money you want to complete the project?

1500$ good?

1000 $ only for send message , do not complete implement
ok?

Do you know that exist fully functional library for sending Telegram messages from C#? Try Notificatio. Http://landing.notificatio.me

i work on it
On Tue, Sep 8, 2015 at 8:40 AM, Ilya Pirozhenko<EMAIL_ADDRESS>wrote:

Do you know that exist fully functional library for sending Telegram
messages from C#? Try Notificatio. Http://landing.notificatio.me
—
Reply to this email directly or view it on GitHub
https://github.com/Everbytes/SharpTelegram/issues/2#issuecomment-138425381
.

So I'm ready to quickly implement base stable functionality of the Telegram .net client lib if you all raise funds up to $3000. I think in 1-2 month you will get a stable lib.

hi guys
you can use [tg] (https://github.com/vysheng/tg) with [php-client] (https://github.com/zyberspace/php-telegram-cli-client) , that's works together perfectly...


Built-ins ModelPlugin needs obligatory comments documentation to be functional.

The parser from parsing.py is looking for documentation of models (under the class name and routine method) obligatory or it will generate an error.

 Models implemented a la demos don't have this issue.

@rdevon OK

Right. Is it an issue then?

Resolved in PR #159.


Add support for non-analytic return for DefaultQubit.probability

Context:
Currently, when qml.probs() is returned from a QNode, the QubitDevice.probability() abstract method is returned directly to the user.
For hardware-like devices that don't support analytic mode, this should behave as expected; the probability will be computed on the device by binning the generated computational basis samples.
However, for analytic devices, such as default.qubit, care must be taken here. Currently, there is a bug in DefaultQubit.probability(), in that the analytic probability is always returned, regardless what the value of device.analytic is.
Description of the Change:
DefaultQubit.probability() is modified to return the non-analytic probability when called with device.analytic = False.
Furthermore, the analytic calculations of DefaultQubit.probability() is moved to a separate function called DefaultQubit._analytic_probability() which is called by DefaultQubit.probability() when device.analytic = False and from QubitDevice.generate_samples(), if available. If a QubitDevice has no _analytic_probability() function, then QubitDevice.generate_samples() will simply call the probability() function of the device directly.
Benefits:
DefaultQubit.probability() returns the correct probability when called with device.analytic = False or device.analytic = True.
Possible Drawbacks:
There is now an extra function called _analytic_probabilities(), that will only be called by QubitDevice.generate_samples() if that device has implemented it. Otherwise it will, as before, simply call probabilities() directly. This could cause some confusion (especially with plugins) whether to have this function or not.
Related GitHub Issues:
closes #531

@thisac, what is the status of this PR, is it ready for another round of review?

@josh146, I was planning to finish this later today. Sorry for the delay. The hackaton (and other things) came in the way last week.

@antalszava, @thisac: just moving the conversation top level, since GitHub has a habit of sometimes making the line comments hard to find.
There are potentially two logic flows here, which is why this is a bit confusing:

The user requests an expectation value or variance of an observable.

analytic=True: QubitDevice calls the devices probability method.

Only supported on statevector devices.
This should return the analytic probability, and the expectation value is computed via lambda @ probs, where lambda is the observable eigenvalues.

analytic=False: QubitDevice calls generate_samples to extract finite number of samples from the device. The mean and the variance of the samples is then calculated.

Supported on both statevector and hardware-like devices.
For hardware-like device, samples returned directly. No probability method is called.
If a statevector device, QubitDevice generates samples via the analytic probability.

The user requests the probability on particular wires be returned.

analytic=True: return the analytic probability (only supported on statevector devices)

analytic=False: return an estimate of the probability, either calculated from samples, or returned from the device itself.

So looking over this, perhaps the solution is:

Analytic devices must provide an analytic_probability() method. QubitDevice.generate_samples() calls analytic_probability() internally, as in this PR.

Hardware devices must overwrite generate_samples()

QubitDevice._estimate_probability(wires=None) is a private method containing general logic (i.e., the logic from this PR) for estimating probabilities from self._samples. This allows it to be easily applied to other plugins, such as the forest.wavefunction device.

QubitDevice has a new built in probability method:
def probability(wires=None):
    if self.analytic:
        return self.analytic_probability(wires=wires)

    return self._estimate_probability(wires=wires)

Hardware devices may optionally overwrite probability() if they provide a more-efficient built-in method (e.g., qiskit devices return the probability without needing first generate samples).

Summary
With these changes, from the perspective of plugin developers:

Statevector devices must provide analytic_probability()

Hardware-like devices must provide generate_samples() and may optionally provide probability().

I think this is relatively clear for plugin developers, while also minimizing breaking changes (as only statevector devices will need to be updated), and moving common code into QubitDevice.

I've added an _estimate_probability method to DefaultQubit which, together with the analytic_probability and probability methods, can easily be transferred to QubitDevice in a future PR together with the changes suggested by @josh146 above.
I also added some NOTEs mentioning what should be changed/removed when implementing the above suggestions (e.g. the try-except statement in QubitDevice.generate_samples).


OCM-5017: Autoscaler resource - Add validation for not negative values

The following attribiues are included here:

log_verbosity
core min
memory min
gpu min

/lgtm
/approve

/retest


HHH-13607 : Exception thrown while flushing uninitialized enhanced proxy with immutable natural ID

https://hibernate.atlassian.net/browse/HHH-13607

Superseded by https://github.com/hibernate/hibernate-orm/pull/3024.


Syntax for passing generic into function type annotation

I have a function type StatelessFunctionComponent (aka SFC) that I use as an annotation like so:
import * as React from 'react';
import { SFC } from 'react';

// Imported types look like:
// import { ReactNode, ReactElement } from 'react';
// type SFC
 = (props: P & { children?: ReactNode }, context?: any) => ReactElement<any> | null

type Props = {};
const MyComponent: SFC<Props> = props =>
Hello, World!

I would like to pass a generic into this function type via the annotation. However, that does not appear to be possible.
Pseudo-code:
import * as React from 'react';

type Props<T> = { t: T };
// How can we make this function receive a generic that is passed into
// the function type annotation?
const MyComponent: <T> SFC<Props<T>> = props =>

Hello, World!

Is there any existing or suggested syntax that could support this?
One workaround is to avoid the type annotation altogether, and instead just write the function signature out manually:
import * as React from 'react';
import { ReactElement } from 'react';

type Props<T> = { t: T };
// Since we have no way of passing the generic into the function type annotation,
// we have to write out the full function type
const MyComponent = <T>(props: Props<T>): ReactElement<any> | null =>

Hello, World!

Another workaround is to use a createSubType helper:
import * as React from 'react';
import { SFC } from 'react';

const createSubType = <T extends any>() => <SubType extends T>(
    subType: SubType,
) => subType;

const createGenericSFC = createSubType<SFC<any>>();

type Props<T> = { t: T };
const MyComponent = createGenericSFC(<T extends any>(props: Props<T>) => (

Hello, World!

));

I've found a better workaround that allows you to re-use the function type.
Example:
const createSubType = <T>() => <SubType extends T>(subType: SubType) => subType;

const createGenericSFC = createSubType<SFC<any>>();

// No error
const fn1 = createGenericSFC(
(props: { p: P }) => null);
// Error, string is not an allowed return type according to `SFC` type
const fn2 = createGenericSFC(
(props: { p: P }) => 'null');

It's not exactly clear what you're trying to do here - can you make these examples self-contained and provide more context on expected behavior?

@RyanCavanaugh I've tried to improve the original description. They are self-contained examples but they do rely on the React typings. However, I have provided an inline definition of the SFC type, which should make them self-contained.)

As per https://github.com/microsoft/TypeScript/issues/27124#issuecomment-456980444, can we re-open this and mark it as a suggestion? /cc @RyanCavanaugh @weswigham

Another example: we have a HOC function type which we want to use as an annotation on a const. However we can't because the HOC is generic:
type HOC
 = (
    ComposedComponent: ComponentType
,
) => ComponentType<P2>;

// Ideally we could somehow re-use the `HOC` function type:
// declare const myHoc:
 HOC

// But we can't, so we have to write out the whole of the `HOC` type, but inline in the function:
declare const myHoc:
(
    ComposedComponent: ComponentType
,
) => ComponentType<P2>;

@OliverJAsh if you really think the feature would be useful, you should probably open an issue with some kind of basic proposal for what new syntax/behavior we'd need to support.

@weswigham https://github.com/microsoft/TypeScript/issues/32973


QUESTION: Use this lib in docker

Hello,
is possible use this library in docker container please?
Because in my docker container not working.
Thanks

Ok I found where the problem is...
in my container is variable
 process.env.USER = undefined
then package wants bad dbus address
/tmp/omxplayerdbus.undefined
instead of
/tmp/omxplayerdbus.root


Require using AMD broken in 10.0.0

Loading via SystemJS in AMD used to work with v9.18.1. Broken in v10:
system.js:4 Uncaught (in promise) ReferenceError: hljs is not defined
    at eval (highlight.js:1566)
    at eval (highlight.js:6322)
    at eval (<anonymous>)
    at Qe (system.js:4)
    at system.js:4
    at async Promise.all (:63342/Kidoju.Widgets/src/js/widgets/index 10)

hljs is undefined when calling  L1566
hljs.registerLanguage('apache', function () {
   ...
}());

In System.config.js, I have:
packages: {
    'src/js': {
        defaultExtension: 'js',
        meta: {
           '*.js': {
                babelOptions: {
                    es2015: false,
                    stage2: false,
                    stage3: false,
                },
                format: 'amd',
            },
            '*.es6': {
                format: 'esm',
            },
            '*.jsx': {
                format: 'esm',
            },
            '*.mjs': {
                format: 'esm',
            },
         },
      },
 },

A workaround for now is to add an exception in meta:
 'vendor/highlight/highlight.js': {
      // Note: AMD broken as of v10
      format: 'global',
},

I'm not sure we ever were truly AMD compatible, the old init just did this:
  globalObject.hljs = factory({});

    // Finally register the global hljs with AMD.
    if(typeof define === 'function' && define.amd) {
      define([], function() {
        return globalObject.hljs;
      });
    }

IE, define a global, and then tell AMD to use that global... the new build system actually is actually far more proper UMD but the problem is now when we're trying to register the language modules there is no top-level hljs yet likely because I would assume AMD isn't even going to make one for us?
So the behavior is different, but I'm not sure that's actually a problem - since it was "all globals all along" so far as I can tell.
I'm not sure what the right answer is here because I don't think what we're doing before is actually correct... so perhaps someone with more knowledge can weigh in here.

Perhaps we should just remove our AMD support?  Perhaps someone super familiar with AMD could explain what we're have to do with the language modules to make them truly AMD compatible... just going back to how we had it before seems quite broken to me.

I think the problem is in the structure of the build output file which is:
(function (global, factory) {
  typeof exports === 'object' && typeof module !== 'undefined' ? module.exports = factory() :
  typeof define === 'function' && define.amd ? define(factory) :
  (global = global || self, global.hljs = factory());
}(this, (function () { 'use strict';
  const HLJS = function(hljs) {
    ....
  }
  var highlight = HLJS({});
  return highlight;
})));

hljs.registerLanguage('apache', function () { // <-- hljs is undefined when using AMD
  ...
}());

Yes, I understand exactly why it is failing.  The new AMD code fails to setup a global object (as it did before), so this now fails.
I'm less sure of the path forward.
This is how the build process works (and always has).  The languages are "modules" that are added AFTER the core library is setup by calling registerLanguage.  This is also  currently a requirement of the website builder/download tool too... the languages are just appeneded to the end of the core library.

After removing all hljs.registerLanguage, AMD loading works fine and returns the hljs object to the loader.

And now you have no languages, so the highlighter won't work. :)

From a different angle: What benefits do you gain by using AMD here vs having to declare it "global"?

I know but the problem is not with your UMD wrapper which is fine.

I know but the problem is not with your UMD wrapper which is fine.

Well it depends how you define the problem. :-)  You might say the wrapper is EXACTLY the problem.  The old wrapper setup a global, the new wrapper does not - and as a result it fails.
@egor-rogov Any thoughts here?

From my end, I can certainly use a workaround either with SystemJS (development) or Webpack (production). I'll always find a solution. I just thought it was nice to report an issue.

I just thought it was nice to report an issue.

It certainly was. :-)  And thanks.  I'm just wondering the value of saying we support AMD when truly we don't and we register a global anyways...

The issue is you are mixing modules and code outside of modules in the same file. I have also tried loading via CommonJS: same error:
Uncaught (in promise) ReferenceError: hljs is not defined
    at Object.eval (highlight.js:1566)
    at eval (highlight.js:6322)
    at eval (highlight.js:6323)
    at eval (<anonymous>)
    at Qe (system.js:4)
    at system.js:4
    at T (system.js:4)
    at U (system.js:4)
    at system.js:4
    at system.js:4

For this, simply configure SystemJS with:
'vendor/highlight/highlight.js': {
    format: 'cjs',
},

Your npm module works because it does not use the same code as the build output, highlight.pack.js. The npm module actually loads the module before loading languages:
var hljs = require('./core');
hljs.registerLanguage('apache', require('./languages/apache'));

In highlight.pack.js, the module id defined but not loaded before loading language packs, unless it has been assigned to global.
I

The cleaner option which I have just tested and which works is to change the structure as follows:
(function (global, factory) {
  typeof exports === 'object' && typeof module !== 'undefined' ? module.exports = factory() :
  typeof define === 'function' && define.amd ? define(factory) :
  (global = global || self, global.hljs = factory());
}(this, (function () { 'use strict';
  const HLJS = function(hljs) {
    ....
  }
  var highlight = HLJS({});

  highlight.registerLanguage('apache', function () {
      ...
  }());

  return highlight;
})));

This contactenates the file a bit differently but the result works both with AMD and CJS in addition to global = true UMD!

But that’s currently not possible for the website builder and would also break 3rd party languages that expect a global. So if we are going to solve this we can’t just solve it half way.

Do you know what the proper AMD way would be to call the registration at a later point?

So if we do say:
  typeof define === 'function' && define.amd ? define('highlight.js', factory) :

Then I think later you could do:
let = register(mod)
if (typeof define === 'function' && define.amd) {
  define(null, ["highlight.js"], (hljs) => {
   hljs.registerLanguage('apache',...
});
} else {
   hljs.registerLanguage('apache'...
}

But I'm wondering what we gain by jumping thru all those hoops since no one has ever seemed to mind a global in the past?

@yyyc514,
IMHO:

UMD loading in highlight.pack.js v10 is broken both for AMD and CommonJS,
highlight is a nice piece of code and you cannot have a broken feature: fix it or remove it,
your suggestion here above, is not the proper way to fix it for AMD and it does not fix it for CommonJS (I could explain but I am not sure it is worth the trouble).
In the long term, modules are now part of ECMAScript and are the way forward,
in the short term, most loaders including webpack can accommodate globals,
so in the end this is a question of product roadmap driven by user requirements.

I would recommend:

Until you have a proper fix, stick to global only to avoid misleading your users;
Give me a few days to look at the code for building highlight.pack.js and see if I can come up with an easy suggestion to fix this. If I can, I'll make a PR.

Just a suggestion, but why reinvent the wheel? i.e. why not use webpack to build your library? cf. https://webpack.js.org/guides/author-libraries/#expose-the-library

@egor-rogov Any thoughts here?

No idea. Until today I was pretty sure that AMD is for Advanced Micro Devices (:

Youur suggestion here above, is not the proper way to fix it for AMD

I was talking solely about how to do it "best" for AMD above... is that not how you would do it? Register the library with an ID and then make that id a dependency of the languages (which would also load via AMD)?  Remember we have to support languages loaded at any time, not just those bundled with the JS file.

highlight.pack.js

Do you mean .min.js? There is no pack file anymore with version 10.

In the long term, modules are now part of ECMAScript and are the way forward,

Agree, and when Node.js natively supports them broadly then I think we'd look at publishing an NPM package that was ES6 modules vs require... until then anyone is free to use the raw source which is ahead of the curve (since v10).

I just realized there is a lot of custom code to build highligh.js but why reinvent the wheel? i.e. why not use webpack to build your library? cf. https://webpack.js.org/guides/author-libraries/#expose-the-library. Any specific requirements? Just asking as you might consider delaying UMD as part of your product roadmap.

We have very specific requirements and the JS build code is all quite clear and easy to follow IMHO.  You only need to know Javascript (and a bit of rollup).  If we simplify it further in the future it'll likely be on top of rollup, not webpack. I've never cared for webpack; never seemed simple or intuitive to me at all.
The goal was never to ship a UMD build, just rather to "preserve" the existing AMD functionality from before... sadly I didn't realize how "hacked" it was, so we failed to do that.

If you want to explain how you feel the "proper" way to do AMD in this situation is, I will consider it but at this point I think the best way forward may be to just drop the previous hacky AMD support and not pretend to do AMD at all - and for the browser just ship a global version - which is really what we'd done all along anyways.

So if we just fallback to IIFE for the browser build we'll have:
/*
  Highlight.js 10.0.0 (3875088a)
  License: BSD-3-Clause
  Copyright (c) 2006-2020, Ivan Sagalaev
*/
var hljs = (function () {
  'use strict';
  ...

Just a top level variable...

@jlchereau Out of curiosity how were you using HLJS before?  Just the default build with "common" languages built-in via AMD?

@jlchereau Can you take a quick look at the PR.  It falls back to an IIFE build but also hooks up the variable to module.exports if it's around.  I think this:

Works in the browser (IIFE, global var hljs)
Works for CommonJS (my Node.js test file can requite it and use it just fine)
Drops support for AMD which we never truly supported properly anyways

@jlchereau Any further thoughts?  If not I'm going to move forward with just supporting CommonJS and IIFE/global and drop support for AMD.


fix: discord invite is invalid

Discord invite link shown at the readme is invalid,
can't find the link anywhere so can't fix and make a pull request.

Not this again...
Here's a temporary invite (30d) https://discord.gg/HDVhWrZ3v

Yeah, not sure how we can get a persistent link. For now I updated temp link again today.

Here's a new permanent link: https://discord.gg/bvrS72gaKS


Check the correct error variable for sandbox creation

In newSandbox(), the deferred func checks err variable which is different from the one declared within the useOldStore(ctx) case.
This PR makes the deferred func check the correct error variable.

@tedyu, there are errors in the travis CI because of:

You're missing a commit message (just the Signed-off-by is not allowed);
You must file an issue and have the Fixed: #nnn as part of the commit message;

You can check the errors here: https://travis-ci.org/github/kata-containers/runtime/builds/696648757

@fidencio
I have modified the commit message.

For 2.0-dev branch, can I use the same issue #2759 in the commit message ?

For 2.0-dev branch, can I use the same issue #2759 in the commit message ?

Yes, you can use the very same one. Thanks!

/test-ubuntu