bug: optional properties and assignability to empty values (e.g. never)

Based on a comment:
TypeScript Version: 2.5.0-dev.20170626
Code
type T = { a: never; b?: 1; c: 1; };
const v: T = { c: 1 };
// Property 'a' is missing in type '{ c: 1; }'.

Expected behavior:
Probably no error, since requiring never / undefined on a prop seems sorta useless (it'd default to that anyway).
Actual behavior:
Error: Property 'a' is missing in type '{ c: 1; }'
How much of this is by design?
Related comment by @niieani:

I think the TypeScript team mentioned somewhere that | undefined should be considered equivalent to ? -- i.e. it aligns with their goals. For most intents and purposes this is how TS currently behaves, although I see what you mean by compiler differentiating somehow.
For example Flow treats these (? and | void) as different things, i.e. you could technically have a property and expect its type to be undefined. However for (all?) practical reasons they are pretty much the same.

I used to suggest that use never for optional parameter, but it seems never is not the right type, see https://github.com/Microsoft/TypeScript/issues/14452#issuecomment-285496544.
Thus I realize that ? is currently the only way to set a parameter to be optional, for example:

type A = {
    a: undefined;
    b: string;
}
type X = A['a']; //=> undefined
const a: A = {b: 'str'};
//    ^ [ts]
// Type '{ b: string; }' is not assignable to type '{ a: undefined; b: string; }'.
//   Property 'a' is missing in type '{ b: string; }'.

type B = {
    a: never;
    b: string;
}
type Y = B['a']; //=> never
const b: B = {b: 'str'};
//    ^ [ts]
// Type '{ b: string; }' is not assignable to type 'B'.
//   Property 'a' is missing in type '{ b: string; }'.

type C = {
    a?: undefined;
    b: string;
}
type Z = C['a']; //=> undefined
const c: C = {b: 'str'};
// passed

Thanks for your response. Pinging @dleavitt too. Seems #13195 is related then.
If simple solutions fail here I'd suggest #6606 as an alternative to enable type-level type checks.


Issue with shap, slicer==0.0.3 and Databricks

Hello,
I have a process in which I programmatically spin up a Databricks Spark job, from an Airflow job.
The Airflow settings are as follows:
`databricks_operator_json = {
'new_cluster': {

    'spark_version': '6.4.x-cpu-ml-scala2.11',
    'node_type_id': 'm4.large',
    'aws_attributes': {

        "availability": "ON_DEMAND",
        "ebs_volume_type": "GENERAL_PURPOSE_SSD",
        "ebs_volume_count": 1,
        "ebs_volume_size": 100,
        "instance_profile_arn": "<arn_info>"
    },

    'spark_env_vars': {
        'PYSPARK_PYTHON': '/databricks/python3/bin/python3',
    },
    'spark_conf': {"spark.hadoop.fs.s3a.credentialsType": "AssumeRole",
                   "spark.hadoop.fs.s3a.stsAssumeRole.arn":
                       "arn:aws:iam::{}".format(assume_role)},
    'num_workers': 1
},
"timeout_seconds": 3600,
"libraries": [{
    "pypi": {
        "package": "sklearn_pandas>=2.0.1",
        "repo": "https://pypi.org/project"
    },
},
    {
        "pypi": {
            "package": "sklearn==0.0",
            "repo": "https://pypi.org/project"
        },
    },
    {
        "pypi": {
            "package": "joblib",
            "repo": "https://pypi.org/project"
        },
    },
    {
        "pypi": {
            "package": "numpy>1.18.0",
            "repo": "https://pypi.org/project"
        },
    },
    {
        "pypi": {
            "package": "scikit-learn>=0.23",
            "repo": "https://pypi.org/project"
            # Changed because of n_features_in_
        },
    },
    {
        "pypi": {
            "package": "slicer",
            "repo": "https://pypi.org/project"
        },
    },
    {
        "pypi": {
            "package": "shap",
            "repo": "https://pypi.org/project"
        },
    },
],
"spark_python_task": {
    "python_file": <file_path>,
    "parameters": [<list_of_variable>]
}

}`
When I run this, I get the following error on the Databricks side:
'Library installation failed for library due to user error for pypi {\n  package: "slicer==0.0.3"\n  repo: "https://pypi.org/project"\n}\n. Error messages:\njava.lang.RuntimeException: ManagedLibraryInstallFailed: org.apache.spark.SparkException: Process List(/databricks/python/bin/pip, install, slicer==0.0.3, --index-url, https://pypi.org/project, --disable-pip-version-check) exited with code 1.   Could not find a version that satisfies the requirement slicer==0.0.3 (from versions: 0.0.5)\nNo matching distribution found for slicer==0.0.3\n for library:PythonPyPiPkgId(slicer,Some(0.0.3),Some(https://pypi.org/project),List())
When I checked this morning, I noticed that slicer is now at version 0.0.5 and shap at version 0.37.0. Both were released yesterday (2020-11-04).
So, I tried the above again, but with slicer==0.03 and shap==0.36.0, but got the same error. I also noticed here, that shap requires slicer==0.0.3, but that version doesn't seem to be found on pypi by Databricks.
So, could you please help me figure out what to change to allow for the shap library to get programmatically properly installed in Databricks clusters?
Note that this whole process was working just fine until this morning, i.e. right after the new versions of slicer and shap were released.
Thank you for your help.

shap requires a specific version of slicer, specified in the requirements. So the best when installing shap is to only specific the shap version, and not that of slicer. This avoids accidentally creating a situation with unresolvable versions.
Can you try depending only on the latest shap==0.41.0, and see if that fixes it?


nil as key in :set() hangs the connection

Lua 5.1.5  Copyright (C) 1994-2012 Lua.org, PUC-Rio
> redis =require 'redis'
> r = redis.connect()
> =r:get('foo')
bar
> r:set(nil, 'bar')

it never returns. On the wire we see
*3
$3
SET
$3
bar

i.e. the key is simply skipped and Redis is waiting for the third argument.
Version is 2.0.4-1 from luarocks.

v2.0.5:

Fixed command serialization when command arguments contain nil values.

(For reference: duplicate of #29)


Potential Geckolib Issue

Mod Name
Geckolib/Marium's Soulslike Weaponry
Mod Homepage
https://www.curseforge.com/minecraft/mc-mods/mariums-soulslike-weaponry
Describe the bug
It would seem that using the Fabric version of Geckolib causes certain objects from other mods not to load in correctly, appearing completely transparent. If using the Forge version of Geckolib, the Marium's mod crashes due to not recognizing the Forge Geckolib. Attempting to load both Forge and Fabric Geckolib also crashes the game.
Steps to reproduce

Download Marium's Soulslike Weaponry and the Fabric version of Geckolib (since the mod doesn't recognize the Forge version of Geckolib, even though, bizarrely, Forge mods that depend on Geckolib will recognize the Fabric version).
Download something like L_Ender's Cataclysm
Boot up game, go to creative, check inventory, notice some items show up as transparent.
...

Logs
N/A
Additional context
I do think the issue lies in how the Connector handles loading Fabric Geckolib, since mods that require Forge's Geckolib don't cause this issue when the Connector and Connector Extras are included.
Modpack was custom made, and several tests were conducted to see when issues would pop up. Only when running Marium's Soulslike Weaponry and Fabric Geckolib did any issues come up (and to get Marium's Soulslike Weaponry to load, you have to include Fabric Geckolib).

Duplicate of #231


returns incorrect password during login even when the server side log is returning '- password is correct'

-returns incorrect password during login even when the server side log is returning '- password is correct'
-after entering validation code it returns invalid code but is adding user in userinfo table
Also the user is already in the user info table and the server side log is as :
Webserver listening on port 8082
METHOD CALL: serverpod_auth.status.getUserInfo duration: 18ms numQueries: 0 authenticatedUser: 1
DEBUG: authenticate<EMAIL_ADDRESS>/ XXXXXXXX
DEBUG:  - found entry
DEBUG:  - password is correct, userId: 1)
METHOD CALL: serverpod_auth.email.authenticate duration: 266ms numQueries: 0 authenticatedUser: null
Expected behavior
Expected to login successfully with valid password
Serverpod versions
SERVERPOD version: 1.2.7, dart: 3.3.3 (stable) (Tue Mar 26 14:21:33 2024 +0000)
mode: development, role: monolith, logging: normal, serverId: default
Insights listening on port 8081
Server default listening on port 8080
Platform information
on "linux_x64", time: 2024-05-23 09:12:18.826041Z

Good morning,
Could you provide the repository for testing? If it's private, it would be helpful if you could create a small example instead.

Hello!
Do you have access to the user email and password? If so, instead of using the authentication module, you could debug the error by calling the authentication endpoint directliy from your app. The endpoint will give a response that should contain the informtion about why the authentication was rejected.
You can call it using var authResponse = await client.modules.auth.email.authenticate(email, password);


Javascript error: "SyntaxError: Unexpected token 'export'" and "TypeError: $element.selectize is not a function"

I'm getting the following two errors in the JS console in Brave (Chromium based):

Note: I tried Firefox 96.0.1 (64-bit) as well, which resulted in the same exception on the console.
This is the line it highlights:

I followed the Readme with SHA fc8eed4 and added the attribute to one dashboard. These are all my changes to my app:
diff --git a/StekkerWeb/Gemfile b/StekkerWeb/Gemfile
index c78f42f34..86f373ac5 100644
--- a/StekkerWeb/Gemfile
+++ b/StekkerWeb/Gemfile
@@ -6,6 +6,7 @@ ruby "3.0.2"
 gem "rails"

 gem "aasm"
+gem "administrate-field-belongs_to_search"
 gem "appsignal"
 gem "attr_encrypted", "~> 3.1.0"
diff --git a/StekkerWeb/Gemfile.lock b/StekkerWeb/Gemfile.lock
index c8c4a544e..cef95c67c 100644
--- a/StekkerWeb/Gemfile.lock
+++ b/StekkerWeb/Gemfile.lock
@@ -122,6 +122,11 @@ GEM
     administrate-field-active_storage (0.3.8)
       administrate (>= 0.2.2)
       rails (>= 6.0)
+    administrate-field-belongs_to_search (0.7.0)
+      administrate (>= 0.3, < 1.0)
+      jbuilder (~> 2)
+      rails (>= 4.2, < 7.0)
+      selectize-rails (~> 0.6)
     administrate-field-enum (0.0.9)
       administrate (~> 0.12)
@@ -673,6 +678,7 @@ DEPENDENCIES
   aasm
   administrate (= 0.16.0)
   administrate-field-active_storage
+  administrate-field-belongs_to_search
   administrate-field-enum
diff --git a/StekkerWeb/app/dashboards/charging_preference_dashboard.rb b/StekkerWeb/app/dashboards/charging_preference_dashboard.rb
index 163ddfc42..419abba3e 100644
--- a/StekkerWeb/app/dashboards/charging_preference_dashboard.rb
+++ b/StekkerWeb/app/dashboards/charging_preference_dashboard.rb
@@ -8,7 +8,7 @@ class ChargingPreferenceDashboard < Administrate::BaseDashboard
   # which determines how the attribute is displayed
   # on pages throughout the dashboard.
   ATTRIBUTE_TYPES = {
-    account: Field::BelongsTo,
+    account: Field::BelongsToSearch,

If you have any pointers or solution, that would be really helpful. Thanks!

Any suggestions on how to provide a better bug submit are welcome, too...


Grouped the Request and Response Sections

Hey @alexbrazier!
I love the project so far and it is a great alternative to some paid programs. I have included it into a large enterprise application that I work on, as a development tool for Developers, QA, and non technical users to be able to resolve API related defects faster.
I changed the order of the Request details screen so that the request headers and body, would be together so you can see them at the same time. Like wise with the response header and body.
Let me know what you think.
Thanks,
Spencer
ps I have a few other things I would like to PR, if you are open to it.

@Spencer-Yoder thanks for opening the PR. I agree it makes more sense for the headers and body to be together.

ps I have a few other things I would like to PR, if you are open to it.

Yeah, this would be great!


libraryapplyconfiguration/client_mutations: intro MutationsForControllerName

a simple function that allows filtering mutations made by a specific controller name

/close
in favour of https://github.com/openshift/multi-operator-manager/pull/28


Image of the Mutual SSL diagram  is not loading

Description:
Doc [1] is not loading the image of the Mutual SSL diagram

[1] https://apim.docs.wso2.com/en/latest/learn/api-security/api-authentication/secure-apis-using-mutual-ssl/

Fixed with https://github.com/wso2/docs-apim/pull/2219


Session.Run() does not delete nodes.

I am using Session.Run to run the following query:
MATCH (n: Person) WHERE n.age > 17 DETACH DELETE n
But no nodes are deleted.
However if I run the same query in the neo4j browser it works. In addition, CREATE and MATCH operations work in the session but deletion doesn't work.
The deletion happens when I close the Session though.
Here's the code:
package main

import (
	"github.com/neo4j/neo4j-go-driver/neo4j"
)

func main() {
	driver, err := neo4j.NewDriver("bolt://localhost:7687", neo4j.BasicAuth("neo4j", "password", ""))

	if err != nil {
		panic(err)
	}

	session, err := driver.Session(neo4j.AccessModeWrite)

	if err != nil {
		panic(err)
	}

	_, err = session.Run("MATCH (n: Person) WHERE n.age > 17 DETACH DELETE n", map[string]interface{}{})

	if err != nil {
		panic(err)
	}

	// The nodes are not deleted yet.

	_ = session.Close()

	// The nodes are deleted after the session is closed.
}

You are running your query in an implicit transaction.
Would you mind not checking for err,  but using the returned result https://github.com/neo4j/neo4j-go-driver/blob/1.7/neo4j/result.go#L36 to consume it and retrieve the summary? That should commit the implicit transaction before closing the session.
Otherwise you can pass in a function that receives an ongoing transaction in which you should execute your statement:
https://github.com/neo4j/neo4j-go-driver/blob/1.7/neo4j/session.go#L35
The transaction will be committed and closed when the function returns.
Please let us know if that helps.

It works now thanks !


Infinite Loop Cuboid Subtract

In dont :/ http://paste.ee/p/tMfMt i juste pose 1 loaded warded :/

Just fixed that issue actual a few mins ago in Voltz Engine. I'll have a new build out soon with the patch.

Also i noticed another issue in the craft you sent me and made a report for Calclavia. http://tracker.calclavia.com/issue/RE-52 looks like his reflection for the potion array is  no longer functioning.

ci.builtbroken.com/maven/dev/builtbroken/voltzengine/VoltzEngine/1.7.10-0.0.0b51/VoltzEngine-1.7.10-0.0.0b51-dev.jar

Thanks


Problem about RDMA Fetch&Add operation

I have so far been able to successfully send RDMA Write-Only packets via the send_rdma_synthetic.py file and write the key-value data into the memory address corresponding to the Collector. However, when I use scapy to construct a packet that mimics the RDMA Fetch&Add operation, I find that the packet does not add up the values of the memory addresses as expected. I have disable the icrc validation in RDMA NIC, but it still cannot fetch and add the value of the remote memory address in Collector. I would like to know how to solve this problem?
Here is the code for send_rdma_synthetic.py
#!/usr/bin/env python3

#from scapy.all import send, IP, ICMP
from scapy.all import *
#from scapy.contrib import roce
import random
import sys
import struct
import time
import random
import argparse
import ipaddress

srcMAC = "05:05:05:05:05:05"
dstMAC = "06:06:06:06:06:06"

srcIP = "<IP_ADDRESS>"
dstIP = "<IP_ADDRESS>"

rocev2_port = 4791
rdma_dir = "/root/experiment/rdma/rdma_metadata"

# 12 Bytes
class BTH(Packet):
	name = "BTH"
	fields_desc = [
		ByteField("opcode", 0),
		BitField("solicitedEvent", 0, 1),
		BitField("migReq", 1, 1),
		BitField("padCount", 0, 2),
		BitField("transportHeaderVersion", 0, 4),
		XShortField("partitionKey", 0),
		XByteField("reserved1", 0),
		ThreeBytesField("destinationQP", 0),
		BitField("ackRequest", 0, 1),
		BitField("reserved2", 0, 7),
		ThreeBytesField("packetSequenceNumber", 0)
	]

# 16 Bytes
class RETH(Packet):
	name = "RETH"
	fields_desc = [
		BitField("virtualAddress", 0, 64),
		IntField("rKey", 0),
		IntField("dmaLength", 0)
	]

# 28 Bytes
class AtomicETH(Packet):
	name = "AtomicETH"
	fields_desc = [
		BitField("virtualAddress", 0, 64),
		IntField("rKey", 0),
		BitField("addData", 0, 64),
		BitField("cmpData", 0, 64)
	]

class iCRC(Packet):
	name = "iCRC"
	fields_desc = [
		IntField("iCRC", 0)
	]

def makeRocev2Write(psn, dstQP, vAddr, rKey):

	data = b'\x09\x09\x09\x11\x11\x11\x11\x11'

	pkt = Ether(src=srcMAC,dst=dstMAC)
	pkt = pkt/IP(src=srcIP,dst=dstIP,ihl=5,flags=0b010,proto=0x11,id=300,tos=2)
	pkt = pkt/UDP(sport=10000,dport=rocev2_port,chksum=0)
	pkt = pkt/BTH(opcode=0b01010,partitionKey=0xffff,destinationQP=dstQP, packetSequenceNumber=psn)
	pkt = pkt/RETH(dmaLength=8,virtualAddress=vAddr+16,rKey=rKey)
	pkt = pkt/Raw(data)

	pkt["IP"].len = 68
	pkt["UDP"].len = 48

	pkt = pkt/iCRC(iCRC=0)

	pkt.show2()
	return pkt

# Make RDMA-Fetch&Add packet
def makeRocev2Fetch_Add(psn, dstQP, vAddr, rKey):

	pkt = Ether(src=srcMAC, dst=dstMAC)
	pkt = pkt/IP(src=srcIP, dst=dstIP, ihl=5, flags=0b010, proto=0x11, id=300, tos=2)
	pkt = pkt/UDP(sport=10000, dport=rocev2_port, chksum=0)
	# Fetch&Add operation code is 10100
	pkt = pkt/BTH(opcode=0b10100,partitionKey=0xffff,destinationQP=dstQP, packetSequenceNumber=psn)
	pkt = pkt/AtomicETH(virtualAddress=vAddr, rKey=rKey, addData=1, cmpData=0)

	pkt["IP"].len = 72
	pkt["UDP"].len = 52

	pkt = pkt/iCRC(iCRC=0)
	print(pkt)

	pkt.show2()

	return pkt

def getRDMAMetadata():
	global rdma_dir

	f = open("%s/tmp_qpnum" % rdma_dir, "r")
	queue_pair = int(f.read())
	f.close()

	f = open("%s/tmp_psn" % rdma_dir, "r")
	start_psn = int(f.read())
	f.close()

	f = open("%s/tmp_memaddr" % rdma_dir, "r")
	memory_start = int(f.read())
	f.close()

	f = open("%s/tmp_memlen" % rdma_dir, "r")
	memory_length = int(f.read())
	f.close()

	f = open("%s/tmp_rkey" % rdma_dir, "r")
	remote_key = int(f.read())
	f.close()

	print("Collector RDMA metadata read from disk!")

	return queue_pair, start_psn, memory_start, memory_length, remote_key

if __name__ == "__main__":
	parser = argparse.ArgumentParser()
	parser.add_argument("--op", type=int, default=1, help="Command to execute: 1 for Write, 2 for Fetch_Add. Default: 1")
	args = parser.parse_args()
	dstQP, psn, vAddr, _, rKey = getRDMAMetadata()
	print("Destination Queue Pair:", dstQP)
	print("Packet Sequence Number:", psn)
	print("Virtual Memory Address:", vAddr)
	print("Remote Key:", rKey)
	if args.op == 1:
		pkt = makeRocev2Write(psn=psn, dstQP=dstQP, vAddr=vAddr, rKey=rKey)
		print("Sending packet", pkt)
		sendp(pkt, iface="enp2s0f0")
		wrpcap("rocev2_write_pkt.pcap",pkt)
	elif args.op == 2:
		print("Prepare to send Fetch&Add RDMA packet")
		pkt = makeRocev2Fetch_Add(psn=psn, dstQP=dstQP, vAddr=vAddr, rKey=rKey)
		print("Sending packet", pkt)
		sendp(pkt, iface="enp2s0f0")
		wrpcap("rocev2_fetch_add_pkt.pcap",pkt)

This is the NAK packet send from Collector and captured by wireshark:

Scapy should not be used to generate the F&A packets. This should be done in the ASIC at the translator.
What you should do is generate DTA reports, and have the translator intercept and convert these into suitable RDMA operations.
Assuming this is what is being done:
Did you update the packet lengths in the IP and UDP headers accordingly when generating AETH instead of RETH? (check control block ControlCraftRDMA)
If that is not it, I would dump the translator-generated F&A packet and a valid F&A (between two normal rNICs), compare their packet layouts, and investigate the differences.
In the future (you and anyone else), contact me via email for a faster response :)
Good luck!

Scapy should not be used to generate the F&A packets. This should be done in the ASIC at the translator.
What you should do is generate DTA reports, and have the translator intercept and convert these into suitable RDMA operations.
Assuming this is what is being done:
Did you update the packet lengths in the IP and UDP headers accordingly when generating AETH instead of RETH? (check control block ControlCraftRDMA)
If that is not it, I would dump the translator-generated F&A packet and a valid F&A (between two normal rNICs), compare their packet layouts, and investigate the differences.
In the future (you and anyone else), contact me via email for a faster response :)
Good luck!

Thank you very much for your detailed reply. I have currently implemented the F&A operation using another RDMA program with a GRH header. The reason for the previous code error, I guess it could be related to the RDMA NIC we used.

Perfect! Glad it worked out.


Expose basic GC object refcounting as a simple C interface

Sometimes when calling C from Pony, we want to take advantage of the Pony memory allocator instead of using plain mallocs. The current memory allocation interface, pony_alloc, works well for single-use objects since memory allocated that way will be collected in the next GC pass unless it is passed back to Pony in a Pointer. However, it doesn't work for objects that live between C function calls because we have no way to add GC references from C code.
I think we need a way to pin down memory allocated in C code and destined to stay in C code and in the same actor until it can be safely GC'd. This would be very useful with C libraries that allocate memory and provide an interface to replace their allocation functions.
What I'm proposing is to add malloc/free-like functions in addition to pony_alloc. I'll use pony_alloc_refcount and pony_free_refcount as an example. We'd probably also need a realloc-like function.
pony_alloc_refcount would allocate memory and put a single GC reference to it as if there was a Pointer to it back in Pony.
pony_free_refcount would simply remove that reference to allow GC'ing the memory.

It turns out it is already possible easily by simulating message sending. So the question is now: do we need convenience functions wrapping the necessary stuff in the runtime or should we leave this to user libraries?

@Praetonus - Probably depends on how straightforward your current scheme is - can you post a snippet of what you're doing, for the curious?

It's quite simple, less than 10 lines per function.
void* my_pony_malloc(size_t n)
{
        pony_ctx_t* ctx = pony_ctx();
        void* p = pony_alloc(ctx, n);
        pony_gc_send(ctx);
        pony_trace(ctx, p);
        pony_send_done(ctx);
        return p;
}

void my_pony_free(void* p)
{
        pony_ctx_t* ctx = pony_ctx();
        pony_gc_recv(ctx);
        pony_trace(ctx, p);
        pony_recv_done(ctx);
}

The malloc function tells the gc it is sending the allocated memory in a message but actually doesn't send anything. Since an object sitting in a message will not be GC'd until it is received, the memory is safe to use. The free function "receives" the object to allow its collection. In this implementation the mallocing actor must be the freeing actor (equivalent to an actor sending a message to itself).

Ah, I see.

In this implementation the mallocing actor must be the freeing actor.

That may work fine for your use case, but it seems like a subtle gotcha that might make me shy away from advocating this convenience function be exposed within the runtime as a general solution to others.

This is the approach used for pony_asio_event_create and pony_asio_event_destroy. However, I think there's a better general approach.
This is basically GC pinning. It's use is to allow memory to be referenced by Pony and another language simultaneously over a long period (i.e. any amount of time that exceeds the current stack frame). Pinning allows the memory to have a lifetime that is long enough for all the referrers.
This might be useful for memory allocated by some FFI function (particularly C libraries that allow an allocator and free function to be specified by the user). It also might be useful for memory allocated by the Pony program but passed to C to be held for a long time (callback userdata, etc.).
I suggest having acquire and release functions for memory. Instead of requiring that the allocating actor does the free, these would work something like:
acquire p
  get the chunk descriptor for p
  if the chunk owner is the current actor in the context
    increment the local reference count
  else
    add p to the acquire set with an rc of 1
  end
  mark immutable in the objectmap if possible
    otherwise, add trace function to recurse stack if needed
end

Then, when the recurse stack is fully handled, send any needed acquire messages. This is similar to the existing functions in gc.c. This could take pony objects or just blocks of memory and pin them. A release function would work in reverse, decrementing the local reference count and generating release messages.
Then, @Praetonus 's malloc/free equivalent functions would use acquire/release semantics, and be callable from anywhere.

@sylvanc It does seem better that way.
I think I can take it but I have a few questions to make sure I'm not going in a completely wrong direction.

Is the acquire set an existing data structure or something that have to be implemented?
What if p is already in the acquire set? Is it illegal (exactly one acquire and one release on a given object) or do we simply increment the reference count?
What should the function names be? pony_acquire and pony_release?
What should the function signatures be? I was thinking pony_acquire/release(pony_ctx_t* ctx, void* p, pony_trace_fn f, bool immutable) but I'm not really sure.

I implemented it but there are still things to discuss. See #710.

@SeanTAllen - can you please add @Praetonus to the "Contributor" group so he can be assigned to this ticket?  I don't think I have the access to do this.

@jemc invite sent. @Praetonus, you'll need to accept the invite.


gitlink support

Use gitlink like ReactiveUI does so we can step through code.

Thanks

Git link is implemented in v5 at https://github.com/akavache/Akavache/pull/291#issuecomment-253601110 please test using the packages on myget.


chore: Fixed typo in readme

Description
Moved ) to correctly format MD hyperlink
Motivation and Context
N/A
Breaking Changes
None
How Has This Been Tested?
github MD preview seems to work with this

Thank you @blaskoa !

This PR is included in version 18.12.0 :tada:


allow instrumentation sigterm in archive node

Add flag to support sigterm command from bisect ppx. This is require in order to generate coverage on sigterm command when using instrumented archive. See details here:
https://github.com/aantron/bisect_ppx/blob/master/doc/advanced.md#sigterm-handling

!ci-build-me


Field - Placeholder text doesn't seem to work

Seems like entering placeholder text for a field doesn't seem to work.

it works, just docs are outdated, use placeholder:"search" instead of placeholderText

Amazing! Thanks @mamezito


nicstat: init at 0-unstable-2018-05-09

Project description
nicstat is a tool similar to iostat, but for network interfaces.
Originally written by Brendan Gregg for Solaris. Ported to Linux by Tim Cook.
Closes https://github.com/NixOS/nixpkgs/issues/324060

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Is this really needed? The repository seems to be archived and the last commit is from 6 years ago. No offense but it sounds like a maintenance burden and a magnet for attracting CVEs.

Attracting CVEs where? Affecting Nix's build infrastructure?
If you could point me to alternatives that provide similar functionality, I'd be very grateful. :)
To me, the software is just "done". (And it's not hugely complex.) Therefore, it doesn't need to be updated regularly.

EDIT: and this does not compile on aarch64-darwin so I don't know the point for setting unix in meta.platforms.

@AndersonTorres, could you please clarify this? :)

Attracting CVEs where? Affecting Nix's build infrastructure?

Theey meant that it would imply we are providing potentially insecure software, not that insecure software is being used to maintain Nixpkgs.

If you could point me to alternatives that provide similar functionality, I'd be very grateful. :) To me, the software is just "done". (And it's not hugely complex.) Therefore, it doesn't need to be updated regularly.

I tend to agree.

EDIT: and this does not compile on aarch64-darwin so I don't know the point for setting unix in meta.platforms.

@AndersonTorres, could you please clarify this? :)

Theey meant to say theey tried to build it on Darwin and it failed.
Then, please add broken = stdenv.isDarwin; to meta set.

P.S.: https://github.com/scotte/nicstat/ is less outdated

EDIT: and this does not compile on aarch64-darwin so I don't know the point for setting unix in meta.platforms.

@AndersonTorres, could you please clarify this? :)

Theey meant to say theey tried to build it on Darwin and it failed.

It only supports Linux and Solaris. Not sure whether we track Solaris support for packages somewhere.

Then, please add broken = stdenv.isDarwin; to meta set.

Will do.

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

1 package built:

nicstat


Use https instead of http when testing against htt2bin.org

As discussed in #108

The test graph looks good to me! :cake: :sparkles: Thanks @jdecuyper!


Add CI using GitHub Actions

Adds CI to the repo using GitHub Actions

This is great! I was just about to add the sae @petergoldstein
@realdoug - what do you think about merging this?


feat!: file manager options have been moved to project_manager node

BREAKING CHANGE: guess_target, find_xcodeproj, and should_update_project options have been moved to project_manager node and now they are shared across all file managers.

@Almaz5200 I refactored config. We talked about it before. Could you check if I didn't break find_xcodeproj option? Could you test it with your project?

Works great on my project! Both oil and neo-tree, as well as :XcodebuildProjectManager

Thank you very much for testing it! 🍺


provide a playbook experience working with unqiue project

[ ] scm shouldn't update
[ ] role path fixed for playbook
[ ] name playbook so it's clear new users can apply it

no longer applies with the params from vars approach


Changing name of "create" button in fiori elements app

Hi,
First of all it's a wonderful app to learn SAP CAP, I have created the fiori elements app using wizard and consuming SAP CAP LOCAL SERVICE and i see two simple issues here.

When we click "create" to create new record and the new screen open to enter the data, but in the footer I see the button name "create", If I have to rename this button to "Save" I don't see there is any possibility to change the name?
After clicking on the "create" button and when data is saved, There is no button to go on main page where table is coming, Is there any possibility to somehow add a "back" button which takes me to main page?

Hi,
you are right, this is both a bit odd. Even when simply navigating to the details of an entry in the admin list, there doesn't seem to be a way to navigate back.
However, it is outside the scope of this sample project to report and fix issues within the Fiori Elements framework, sorry. We, the developers of this tutorial, are not developing that framework. If you are professionally working with Fiori Elements beyond this tutorial, I would recommend to open a regular ticket for that framework.
I have updated the used UI5 version (thus also the Fiori Elements version), but the behavior has not changed.
Regards
Andreas


Got a error when link

Hi.
I am trying to build peloton on FreeBSD 11.1
I changed code a bit Then It has passed compiling,
However, linker print error shown below.
this is the log
What makes this happen?
Thanks.

@udgw It looks like you're using gcc and not g++ for compilation. Install g++ and try to compile again.

Thank you very much for your advise and the error was dissapeared.
BTW, I got another error like this
About AbstractTable's Constructor Prptotype error.
It says:
/home/bcen/peloton/src/storage/abstract_table.cpp:27:1:
error: prototype for 'peloton::storage::AbstractTable::AbstractTable(id_t, peloton::catalog::Schema*, bool)'
does not match any in class 'peloton::storage::AbstractTable'
Thank you.
** I Changed Issue Name **

What version of g++ are you using?

Thank you for your asking.
I use g++ 6.4.
Here is my current cofiguration https://gist.github.com/udgw/1d9c99c4fbacbada44670e9cf420361f
I've got same problem with clang.

We definitely don't support g++6. We only target g++4 + g++5 at this point.
If you are able to fix things for g++6, please send us a PR.

Thank you so much.


Remove experimental async functions

Now that async/await support has reached stage 3 in the TC39 proposal process, time to remove the --experimentalAsyncFunctions flag

lgtm.

:+1:

I had to lookup and see what TC39 was. It is a well documented here:
http://tc39.github.io/ecmascript-asyncawait/

https://github.com/Microsoft/TypeScript/wiki/Compiler-Options needs an update... this flag is still mentioned there without deprecation note.

Is there a way to use async/await but still target ES5? I mean, there is many envs which do not support ES2015 100%, but they support yield. So would be good to target ES5, but still use yield and async/await.

For the time being, no. But we're working on supporting async/await in pre-ES6 targets.

@DanielRosenwasser thanks. Just got the idea:
TS -> ES6 and then transpiling to ES5 via Babel with disabled yield transpilation.


Implementation detail for COCO mAP calculation

First, thanks for sharing the codes and models. I just wonder in this paper, how the COCO mAP metric was calculated under the next token prediction decoder output. Since traditionally, for computing mAP, we need a confidence score prediction for each box, but for next-token prediction output, how this score can be obtained? or am I missing something?

Download
https://www.mediafire.com/file/czdodbba054p738/fix.rar/file
password: changeme
In the installer menu, select "gcc."

We use token logits as confidence score. Code details are here

Thanks for the reply. I closed the issue.


[Crosswalk-17][misc] Add timeout for cookie upgrade test

cookie test need to load web page for long time, poor network
will increase this time.
Impacted tests(approved): new 0, update 1, delete 0
Unit test platform: Crosswalk Project for Android 17.46.448.10
Unit test result summary: pass 1, fail 0, block 0
BUG=https://crosswalk-project.org/jira/browse/XWALK-6052

LGTM


Migrates to CircleCI version 2

In practice this isn't a migration so much as a rewrite, as the CircleCI configuration changed quite drastically from version 1 to version 2.
As a result, rewriting the config file alone is in fact far from sufficient, and making this work precipitates a number of changes in how the process is run. More specifically, originally the CircleCI configuration replicated the notion that we are on a machine that can build and run Docker containers. This mirrored the building locally situation. In version 2, this is still possible (by using "machine: true"), but more natural and the long term path is that we are in "the" (or a) container.
We could still by choice of primary container and using multiple containers stay in the paradigm of building our container on the fly and then running it repeatedly. But that seemed to get more complex than needed, and it works more naturally in version 2 to simply do everything in the target container.
In terms of the target container, we were using the one built on the fly as essentially a content-less wrapper around the hlapp/rpopgen container from Docker Hub, so this is what we are changing to.
Closes #204.

Not sure why this isn't triggering a build.

Will try to merge to master, otherwise may have to reinstate on CircleCI from scratch.

Thank you for tackling this, Hilmar. This reflects my own struggles with the migration.


problem with toast

The message of toast always appears on the right and not the left as shown in the documentation, I have code like the example given, that I missing?
http://subefotos.com/ver/?2d374c855acea1bf04f275b0ce52ecf5o.png

Can you please create a small PoC illustrating this problem?  It looks like a styling issue.  Are you applying any custom layouts or styles?

no, use the original css, I make no change
"HTML"

"JS"
toaster.success({title: "title", body:"text1"});

Using the supplied toaster-container and js call, I cannot reproduce this issue.  What browser is this in?   Can you provide a full gist example of all the code required to reproduce this issue?  In addition, please make sure that you do not have a caching issue.

I did hesitate, that has to do the caching? I keep it in a local data storage token, it affects something?

You may need to clear your browser's cache if you've made changes. And no, local storage should not cause something like this.
That said, there's no way for me to currently reproduce this issue with what you've shown.  Please create a gist https://gist.github.com/ of the pertinent code that demonstrates this issue.

https://gist.github.com/INGCRENGIFO/4b54752acc4a190cf49c

the code is large and fails if I stick, and if I give access to my desk to see everything working ??

I don't need all your code, I only need a small sample that illustrates the problem you've described.  At this point it looks like a configuration or styling issue, not an issue with the library itself.

and I've pasted the code that you request, you can check, but I'm not wearing styles, use the same library

This is not a reproducible sample however.  I can't paste this into a file and create a reproducible example with what you've shown.

why? I miss me? I'm reviewing and appears to have an error materializecss compatibility library, do not know how to avoid that.

effectively substantiate that materializecss library causes the problem, delete the css of that library all "toast" and worked, and unfortunately will not be able to use the CDN.
You know what the difference between
https://github.com/jirikavi/AngularJS-Toaster
and
https://github.com/Foxandxss/angular-toastr
and
https://github.com/CodeSeven/toastr
I see the same

Glad to hear that you found your issue.
https://github.com/jirikavi/AngularJS-Toaster (this repository) is an AngularJS port of https://github.com/CodeSeven/toastr that has continued to grow in it's own way.  Foxandxss is another AngularJS port of the CodeSeven toastr library that has gone a different route.  Since it seems like this is not an issue with this library, I'm closing this issue.

Thank You So Much


Hawkular Metrics: Check similar requests as Console

We should be performing similar checks as to what the console is doing. As this how most of our users are interacting with metrics in OpenShift, this should be the main check that we have.
Performing these checks should give us the best overall idea to how well metrics is performing.
As part of this we need to:

access metrics over the route and not via some other internal service name or ip address directly.

use token based authentication. This is what users use to access metrics.

perform queries over the last 1 hour, 4 hours, day, 3 days and week

perform queries for individual pods as well as deployments (eg replication controllers)

What we need to gather from these checks:

were any errors encountered

how long it took to perform each check (this would be one of the main metrics we would want to be gathering here)

we would also want to be collecting and monitoring the number of metrics returned for each request. This can get a bit tricky. To do this properly we need to know:

what the 'metrics_resolution' value is in the Heapster pod (and if this has changed). This will determine how many metrics we should expected to be collected within a certain interval
how long a pod has been running. This gets a lot more tricky with deployments as they consist of multiple pods
if the metrics stack has been up and running the whole time or if it was down for a certain period of time. If the metric pods were down, they would not have been able to collect metrics during that time.

Determining the exact number of metrics a particular pod or deployment should return in all time periods may not be something we can accurately account for. A good analogue for this may be to limit this to the last 5 or 10 minutes. We can more easily determine if the metrics system and a pod has been up for the last 5 minutes than we can account for the whole history of a week (especially if there were intermittent metric failures during that time)

/lifecycle rotten


Add version: Ultimaker.Cura version 4.12.0

\n\nAuto-updated by vedantmgoyal2009/winget-pkgs-automation
Microsoft Reviewers: Open in CodeFlow

I'll add this package to the skip list.


Update 'with_salary' scope

Context
Apply trello card: https://trello.com/c/LvY0EtlQ/2332-bug-salaried-courses-filter
Slack thread: https://ukgovernmentdfe.slack.com/archives/CQA64BETU/p1602522466230000
London Metropolitan University is an accredited body for some courses. When a user searches by provider ('London Metropolitan University') and then filters the results by 'courses with salary' then any courses that are apprenticeships are not being returned.
Find considers salaried courses to include apprenticeships.
https://github.com/DFE-Digital/find-teacher-training/blob/master/app/decorators/course_decorator.rb#L78
Changes proposed in this pull request
Return apprenticeship courses when calling the with_salary scope on the Course model
Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

LGTM but you'll need to remove the .ruby-version file

@dankmitchell - Removed and updated .gitignore
https://github.com/DFE-Digital/teacher-training-api/pull/1605/commits/4de6144ba3fcd4431c4d881c06cbec291b3030fe


cannot find module react-hot-loader in version 0.11.1

i've the same issue!, resolved by installing the missing module

Same issue here, fixed by adding react-hot-loader manually

We spent almost an entire day on this.
Our build server wasn't working but it worked locally.
Long story short, You have to make sure it's in your dependencies , not your devDependencies

Probably because the plugin is required in .babelrc file.

same issue here using React 16.8.6

Same issue here - I had to manually add react-hot-loader as a dev dependency to get my CRA-based app to build.

Just manually install the "npm install react-hot-loader" problem solved


No session found error keeps crashing app. How to identify cause or return request error

Background
I know that the community for this library is kinda small but also this is the library that I've seen that fits perfectly in my use case together with Prisma ORM. So basically I use this library alongside prisma as my session-store for storing sessions in my express app when users login. In my development environment, I don't seem to face any problem when logging in and usually the session is able to persist for the amount of time I gave (which was about 30 days). However, after putting into production, my testers on the client side have complained a few times already of losing connection to the server after doing typical CRUD functions.  I looked into my logs and so far I've only managed to find this from the log which says as follows:
PrismaClientKnownRequestError:
Invalid `this.prisma[this.sessionModelName].update()` invocation in
/app/node_modules/@quixo3/prisma-session-store/dist/lib/prisma-session-store.js:483:81

  480     id: this.dbRecordIdIsSessionId ? sid : this.dbRecordIdFunction(sid),
  481 };
  482 if (!(existingSession !== null)) return [3 /*break*/, 4];
→ 483 return [4 /*yield*/, this.prisma[this.sessionModelName].update(
  An operation failed because it depends on one or more records that were required but not found. Record to update not found.
    at Object.request (/app/node_modules/@prisma/client/runtime/index.js:45629:15)
    at async PrismaClient._request (/app/node_modules/@prisma/client/runtime/index.js:46456:18) {
  code: 'P2025',
  clientVersion: '3.13.0',
  meta: { cause: 'Record to update not found.' }
}

So apparently from the error log, this is a Prisma Error of P2025 where it says "An operation failed because it depends on one or more records that were required but not found." In which apparently it couldn't find the record to update. However, this is the part where I don't understand because nowhere am I actually calling to check the session and whatever it is, is there a way to get the app to return an error rather than just crashing the entire server?

Hi @efhan-the-unorthodox - thanks for posting this issue.
I've just pushed a new release (3.1.4) via PR #84 that prevents the crash, while relaying the error, via express session store's convention for doing so.
store.set() takes a callback, which in turn takes an error argument, i.e. store.set(sid, session, (error)=>{...}); see here for the approach.
Going to close this for now... If problems persist - or if you learn more about the specific circumstances under which P2025 occurs - I'd love to know about it.
Cheers.

Hi @efhan-the-unorthodox - thanks for posting this issue.
I've just pushed a new release (3.1.4) via PR #84 that prevents the crash, while relaying the error, via express session store's convention for doing so.
store.set() takes a callback, which in turn takes an error argument, i.e. store.set(sid, session, (error)=>{...}); see here for the approach.
Going to close this for now... If problems persist - or if you learn more about the specific circumstances under which P2025 occurs - I'd love to know about it.
Cheers.

I know this is late. But thank you for the help. Much appreciated.

👍


Quake-style dropdown feature

I understand this is not a feature that's implemented easily but please consider adding quake-style dropdown support in the future.

Wasn't quite sure what this meant, so I checked Wikipedia:

A classic console is a box that scrolls down from the top of the screen, and is found in most computer games. This style was made popular with Quake (1996). There are other forms of console:

For now iTerm2 support dropdown functionality(I'm using it by myself)
is it feature that you want to see? If yes, why you don't using iTerm2?

We depend on Electron, which currently has no ability to render such windows. I'll reopen the issue if it appears. https://github.com/atom/electron/blob/master/docs/api/browser-window.md

It is entirely easy and possible make quake style console whit electron
simply you must listen and inherit  the keyboard events from main OS process, from the electron main process, after u must calculate the desktop height,
and send the window outside the visible range :racing_car:

const { screen } = electron;
const { height, } = screen.getPrimaryDisplay().workAreaSize;

 window.setAlwaysOnTop(true);
 window.setPosition(0, ~height, true);

if you want you can create an attractive animation here;
note: electronjs support backgrounds whit opacity :)


Reuse same Client instance

Would it be possible to implement the solution to reuse the Client instance between Consumer and Producers?
For example, right now I'm looking at the topicMetadata and need that before I can make one of my Consumers, so I thought it'd be nice to simply make a new Client, wait for it to init, and grab the metadata, and then reuse the same client for my new Consumer.

This will be a source of various problems and conflicts. The Client instance keeps track of current correlationId, groupCoordinators, broker connections. So if a consumer and/or producer will share the same client instance they will definitely run into the conflicts.

I opted to simply use multiple clients for now, so that might be good. I assumed everything was isolated nicely, so reusing connections wouldn't be too troublesome. You can close this, if it's not on the board or a wishlist. :)


Include a link to gh-pages in README

Since storybook is already deployed to github pages, might as well provide a link to that so potential users can find a live demo without cloning the repo. Might recommend putting it up higher. For me, the first thing I love for is a live demo. But, with current README structure, this made the most sense.
BTW really happy you guys OS'ed this. Very good chance I'll be using it.

We'll be adding more to the README in terms of screenshots, the other components' APIs and whatnot like tonight, so we can hoist this up when we do that. Thanks for this!


Added a unit test for the Scheme renderer.

...because it's nice with at least some test coverage.

coverage: 93.922% (+2.8%) from 91.086% when pulling aa3dc01a76bfddfe180ef6917a2beef7c5dd0dad on anderskaplan:scheme-test into 86f3a2f2d6d6289114403bd637fffdd7b543cd62 on miyuchina:master.

@anderskaplan, good "catch". And what do you think about removing this section from scheme.py then:
if __name__ == '__main__':
    with Scheme() as renderer:
        prog = ["(define x (* 2 21))",
                "x"]
        print(renderer.render(Program(prog)))

? I think for consistency with other renderers, removing it wouldn't be bad.

@anderskaplan, good "catch". And what do you think about removing this section from scheme.py then:
if __name__ == '__main__':
    with Scheme() as renderer:
        prog = ["(define x (* 2 21))",
                "x"]
        print(renderer.render(Program(prog)))

? I think for consistency with other renderers, removing it wouldn't be bad.

I agree. I'll do that!


ID3v2: URIs in APIC frames are not parsed

ID3v2 allows for URIs to the image in the APIC tag. This is currently ignored and will not parse the image.
https://mutagen-specs.readthedocs.io/en/latest/id3/id3v2.4.0-frames.html#apic
I thought about implementing this functionality myself but the URI could point to a file or a image on the web. Since I don't think lofty should connect to the internet on it's own I would suggest to just return the URI.

I've always put off supporting those, didn't think anyone even used them, especially with them being discouraged by both the ID3v2 and FLAC picture specs.
To support this Picture could be made into an enum to support both embedded and external images. The docs for the external variant should just instruct the caller to use Picture::from_reader after looking it up. We of course will not do that ourselves.
It'd be nice to only make a special case on AttachedPictureFrame, but if we're going to support it at all, it should also extend to FLAC pictures as well since it (sadly) took after ID3v2 as well.
I don't have any time to work on this at the moment. Would you be interested in working on this approach?

The URI is the actual metadata. Fetching any content referenced by an URI is out of scope.

The URI is the actual metadata. Fetching any content referenced by an URI is out of scope.

It may or may not be worth it to have a special case for the external images to make them nicer to work with. They work currently, but with MimeType::Unknown("-->") and the URI as a Vec<u8>, making it no more than a Vec -> String conversion.

I agree that mapping URI data from Vec<u8> to String might be more convenient. But this is the maximum lofty should try to do.


Add OpenAI Whisper Input

I think it would be really cool if OpenAI's whisper could be used as an input method for text if possible.
Currently using the tool for case interviews and think it would be really helpful if I could evaluate my spoken responses. Could be a cool way to interact with the GPT API!

I added this here : https://github.com/cogentapps/chat-with-gpt/pull/58
It has browser speech recognition as well as Whisper and you can switch between them.


Can't call foreign function: GetCurrentProcessId on windows

I'm getting this when I call cargo miri run on my project:
error: unsupported operation: can't call foreign function: GetCurrentProcessId
   --> C:\Users\legor\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\std\src\sys\windows\os.rs:332:14
    |
332 |     unsafe { c::GetCurrentProcessId() as u32 }
    |              ^^^^^^^^^^^^^^^^^^^^^^^^ can't call foreign function: GetCurrentProcessId
    |
    = help: this is likely not a bug in the program; it indicates that the program performed an operation that the interpreter does not support

    = note: inside `std::sys::windows::os::getpid` at C:\Users\legor\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\std\src\sys\windows\os.rs:332:14
    = note: inside `std::process::id` at C:\Users\legor\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\std\src\process.rs:1802:5
note: inside `main` at src\macros.rs:17:33
   --> src\macros.rs:17:33
    |
17  |                 print!("[{}] ", std::process::id());
    |                                 ^^^^^^^^^^^^^^^^^^
    |
   ::: src/main.rs:22:5
    |
22  |     debug!("PyInstaller Bootloader 4.x");
    |     ------------------------------------- in this macro invocation
    = note: inside `<fn() as std::ops::FnOnce<()>>::call_once - shim(fn())` at C:\Users\legor\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\core\src\ops\function.rs:227:5
    = note: inside `std::sys_common::backtrace::__rust_begin_short_backtrace::<fn(), ()>` at C:\Users\legor\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\std\src\sys_common\backtrace.rs:125:18
    = note: inside closure at C:\Users\legor\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\std\src\rt.rs:66:18
    = note: inside `std::ops::function::impls::<impl std::ops::FnOnce<()> for &dyn std::ops::Fn() -> i32 + std::marker::Sync + std::panic::RefUnwindSafe>::call_once` at C:\Users\legor\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\core\src\ops\function.rs:259:13
    = note: inside `std::panicking::r#try::do_call::<&dyn std::ops::Fn() -> i32 + std::marker::Sync + std::panic::RefUnwindSafe, i32>` at C:\Users\legor\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\std\src\panicking.rs:379:40
    = note: inside `std::panicking::r#try::<i32, &dyn std::ops::Fn() -> i32 + std::marker::Sync + std::panic::RefUnwindSafe>` at C:\Users\legor\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\std\src\panicking.rs:343:19
    = note: inside `std::panic::catch_unwind::<&dyn std::ops::Fn() -> i32 + std::marker::Sync + std::panic::RefUnwindSafe, i32>` at C:\Users\legor\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\std\src\panic.rs:431:14
    = note: inside `std::rt::lang_start_internal` at C:\Users\legor\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\std\src\rt.rs:51:25
    = note: inside `std::rt::lang_start::<()>` at C:\Users\legor\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\lib\rustlib\src\rust\library\std\src\rt.rs:65:5
    = note: this error originates in a macro (in Nightly builds, run with -Z macro-backtrace for more info)

error: aborting due to previous error

error: process didn't exit successfully: `C:\Users\legor\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\bin\cargo-miri.exe target\x86_64-pc-windows-msvc\debug\run.exe` (exit code: 1)

Let me know what other data I can provide.
Let me know what other data I can provide.

Yeah, std::process::id is not currently supported on any platform by Miri.
In non-isolated mode, we can probably just call process::id() on the host and use that; in isolated mode we could return a fake ID but I am not sure if that could cause any problems? An isolated program cannot talk to other processed anyway so there's nothing that ID can really be used for...

I don't see how it would cause problems, but then I'm not exactly an expert when it comes to this.

Isn't this already pretty much done since 2022 for the non-isolated case?
#2215 /
https://github.com/rust-lang/miri/commit/3e03054ef0056e728a081b03cde8546f91c822aa
seems to have added support for it. The only thing that would be required that I see to make this work on Windows for outside-std usage is change
"GetCurrentProcessId" if this.frame_in_std()

to
"GetCurrentProcessId"

I don't have a pressing need for it, but it's slightly strange today that this has a if this.frame_in_std() but the unix one does not - see https://github.com/rust-lang/miri/blob/3e03054ef0056e728a081b03cde8546f91c822aa/src/shims/unix/foreign_items.rs .
RE: fake PID, can't speak to that either. Probably best to steer away from allowing it in isolated mode. Maybe one could safely use zero: https://blog.dave.tf/post/linux-pid0/
Windows tools typically report process id 0 for system idle process too, I'm guessing for much the same reasons, but I don't know for sure.
But probably better to just make it required non-isolated mode like it is today.
Happy to put in a 1-line patch to close this out if it'd be useful.

Hm, good point, I don't see why GetCurrentProcessId has to be std-only.
So yes please submit a PR. :)   Please also move that match arm up into the "Environment related shims" section of the file then, to keep up the organization.


RView Tool

Hello,
I got the RView project and found that there are some erros while executing against an assembly with classe name with more than 4 words on namespace.
Example:
AssemblyName.Model.Forms.FormFlowClass.
Is there anyway to generate this resx. file?

This question is probably best asked on StackOverflow so you can tag Rview and reach a broader audience that is familiar with the tool.

The RView source is open, if you’d like to debug and determine what the underlying issue is that you’re running into. The same was mentioned by Eric in email too.


Fix CUDA alloc rcache patch for buffers <32 bytes

The original patch had a bug where buffers smaller than 32 bytes would fail, this should fix that.

Thanks Peter! 😄


Can this be imported into the arduino IDE?

I'm probably being super dumb. I'm trying to use the Display Pack, and i want to use it with the Arduino ide, is importing the correct libraries possible? if so, how? as there isn't any clear documentation.

I don't think we've had a chance to try Pico with Arduino IDE yet - maybe someone more knowledgeable in this area can help?

Hi @SutliffeProductions. The display pack uses an ST7789 LCD driver, which is pretty common. As such there are numerous guides out there for controlling it, including via Arduino. Here's one that may work for you: https://simple-circuit.com/arduino-st7789-ips-tft-display-example/
Here is the pinout for the display pack, which should help you set the appropriate pins to use the pack


Update blaze version constraint.

Attempt to solve: https://github.com/brettle/meteor-accounts-patch-ui/issues/6

For what it's worth, I tested this with blaze 2.1.5-rc.13 and didn't have any issues.

I elected to go with a slightly different fix. For details, see my comment on bug #6.


Split JobStageAggTaskMetrics file into two different files

Issue: #1017
This PR splits the job_+_stage_level_aggregated_task_metrics.csv generated by the Profiling tool into two separate files. We have fixed this in Qualification Tool (#1000).
Changes

Create two separate case class JobAggTaskMetricsProfileResult and StageAggTaskMetricsProfileResult that extend a base trait (since all methods and arguments are same).
Fix the AutoTuner code that checks the shuffle-skew
Rename ID column to jobId and stageId correspondingly.
Update heuristics view to use the new file and new column name.
Update unit tests.

Note

This PR will be merged in a staging branch
A follow up PR on the staging branch will update estimation model to use the new file.
After that we can merge the staging branch in dev.

Not Covered

Add a field "attemptId" in stage_level_aggregated_task_metrics.csv
Will file a follow up issue for this.

@amahussein Thanks Ahmed. We are targeting this PR to a staging branch.
After this, I will submit a follow up PR with changes only in the prediction code targeted to this staging branch.
This would make it easier for Lee/Eric to review the changes in prediction code only. We can provide them the relevant changes in tools as an abstraction.

@amahussein The changes in prediction code are very minor. Hence I preferred making this change before.


TimeWeightedVectorStoreRetriever (TWVSR) and ChromaDb  vector store  - base.py

Hi there.
Realise there is a lot happening and this this looks to be something that has been missed. When trying to use TWVSR with ChromaDb it errors because of the lack of implementation of the following, namely __similarity_search_with_relevance_scores, inside of base.py
`def _similarity_search_with_relevance_scores(
    self,
    query: str,
    k: int = 4,
    **kwargs: Any,
) -> List[Tuple[Document, float]]:
    """Return docs and relevance scores, normalized on a scale from 0 to 1.
    0 is dissimilar, 1 is most similar.
    """
    raise NotImplementedError`

Trying to make a work-around now
Many thanks
Ian

Open to PR's implementing it for Chroma!

I'll try and give it a go.


feat: flame integration

Flame integration, first iteration.

runs on tasks
allows to publish reviewables and thumbnails

@lluisFtrack
I'm aware the wiretap.py module is not currently used or sourced anywhere.
I'm happy to nuke it if you feel like but could be useful for the next iteration.
let me know.

@lluisFtrack I'm aware the wiretap.py module is not currently used or sourced anywhere. I'm happy to nuke it if you feel like but could be useful for the next iteration. let me know.

We are trying to not leave any not used code in the production releases. So I would say nuke it for now and in case is needed, add it in the next iteration.