CodeBrowser focuses on menu bar after releasing forward/back hotkey (containing alt)

Describe the bug
CodeBrowser focuses on menu bar after releasing forward/back hotkey (containing alt)
To Reproduce
Navigate in the disasm view such that location history will be populated.
Use alt+ left arrow, right arrow to navigate back and forth through the history.
Note that sometimes, releasing the alt key results in focus being transferred to CodeBrowser's File...Edit...Analysis...etc menubar (the cursor in disasm view will be lighter color, and text in the menubar will have hotkey characters underlined).
-> need to hit esc or alt again to return focus to disasm view.
Expected behavior
Focus does not leave original window view.
Environment (please complete the following information):

OS: Microsoft Windows [Version 10.0.22631.3737]
Java Version: jdk-21.0.3+9
Ghidra Version: 3ec2dfb2011579933c43177e4d38d204ff248b7d
Ghidra Origin: locally built

Additional context
This is not a new problem (not related to recent focus changes).

I tried this on Linux, Mac and Windows.   Windows is the only OS I see the the problem on.  I'm investigating the issue more.

As a workaround, you can remap the keybindings to not use the Alt key.

We have no plans to make any changes to address this issue at this time.


Integrate listings CRUD methods with database

Create listings table within devdatabase locally
Create listing
Read listing
Update listing
Delete listing

Resolved with #7

adding users table to this issue since a listing requires an owner


[5.2] Prevent related models proliferation on ModelFactory

Allows to define closures within the model factory definition in order to avoid unused models to be written to the database when overriding the model attributes. (See #9245)
Here's a model definition example:
$factory->define(App\Post::class, function ($faker) {
    return [
        'title' => $faker->title,
        'content' => $faker->paragraph,
        'user_id' => factory(App\User::class)->create()->id
    ];
});

If you overwrite the user_id attribute when creating an instance of the model:
$post = factory(App\Post::class)->create([ 'user_id' => App\User::find(1)]);

You'll end up having written an unused App\User model to the database as the attribute merging happens after the evaluation of the definition.
The first fix has been provided by @devinfd with #9387 and looked something like this:
$factory->define(App\Post::class, function ($faker, $attributes) {
    $userId = isset($attributes['user_id']) ? $attributes['user_id']: factory(App\User::class)->create()->id;
    return [
        'title' => $faker->title,
        'content' => $faker->paragraph,
        'user_id' => $userId
    ];
});

This works but as you start defining more complex model relationship becomes ugly as hell.
My solution is to provide the ability to insert relationship model definition inside a closure and evaluate the closure only after the attributes merging.
So you will declare the definition as follow:
$factory->define(App\Post::class, function ($faker) {
    return [
        'title' => $faker->title,
        'content' => $faker->paragraph,
        'user_id' => function () {
            return factory(App\User::class)->create()->id;
        }
    ];
});

An array of evaluated definition is passed to the  closure in order to define complex relationships:
$factory->define(App\Post::class, function ($faker) {
    return [
        'title' => $faker->title,
        'content' => $faker->paragraph,
        'user_id' => function () {
            return factory(App\User::class)->create()->id;
        },
        'user_type' => function (array $evaluated) {
            return App\User::find($evaluated['user_id'])->type;
        }
    ];
});

This looks nice. It's true that my solution works but I agree that it can get muddy quickly with complicated relationships.

Devin Dombrowski

On Feb 10, 2016, at 1:45 AM, Alessandro Benoit<EMAIL_ADDRESS>wrote:
Allows to define closures within the model factory definition in order to avoid unused models to be written to the database when overriding the model attributes. (See #9245)
Here's a model definition example:
$factory->define(App\Post::class, function ($faker) {
return [
'title' => $faker->title,
'content' => $faker->paragraph,
'user_id' => factory(App\User::class)->create()->id
];
});
If you overwrite the user_id attribute when creating an instance of the model:
$post = factory(App\Post::class)->create([ 'user_id' => App\User::find(1)]);
You'll end up having written an unused App\User model to the database as the attribute merging happens after the evaluation of the definition.
The first fix has been provided by @devinfd with #9387 and looked something like this:
$factory->define(App\Post::class, function ($faker, $attributes) {
$userId = isset($attributes['user_id']) ? $attributes['user_id']: factory(App\User::class)->create()->id;
return [
'title' => $faker->title,
'content' => $faker->paragraph,
'user_id' => $userId
];
});
This works but as you start defining more complex model relationship becomes ugly as hell.
My solution is to provide the ability to insert relationship model definition inside a closure and evaluate the closure only after the attributes merging.
So you will declare the definition as follow:
$factory->define(App\Post::class, function ($faker) {
return [
'title' => $faker->title,
'content' => $faker->paragraph,
'user_id' => function () {
return factory(App\User::class)->create()->id;
}
];
});
An array of evaluated definition is passed to the closure in order to define complex relationships:
$factory->define(App\Post::class, function ($faker) {
return [
'title' => $faker->title,
'content' => $faker->paragraph,
'user_id' => function () {
return factory(App\User::class)->create()->id;
},
'user_type' => function (array $evaluated) {
return App\User::find($evaluated['user_id'])->type;
}
];
});
You can view, comment on, or merge this pull request online at:
https://github.com/laravel/framework/pull/12231
Commit Summary
Delay closures evaluation after merging attributes
File Changes
M src/Illuminate/Database/Eloquent/FactoryBuilder.php (22)
Patch Links:
https://github.com/laravel/framework/pull/12231.patch
https://github.com/laravel/framework/pull/12231.diff
—
Reply to this email directly or view it on GitHub.

Thank you @devinfd . Sorry for the "ugly as hell" part, your solution actually saved my sanity on my latest project and allowed me to get to this solution (waiting for someone else to propose a better one) :smile:

@GrahamCampbell I've replace the foreach loop with array_map but I'm not sure what you mean with "Please import this."

put use Closure at the top of the file instead of \Closure in evaluateClosures
Devin Dombrowski
<EMAIL_ADDRESS>
On Feb 10, 2016, at 7:15 AM, Alessandro Benoit<EMAIL_ADDRESS>wrote:
@GrahamCampbell https://github.com/GrahamCampbell I've replace the foreach loop with array_map but I'm not sure what you mean with "Please import this."
—
Reply to this email directly or view it on GitHub https://github.com/laravel/framework/pull/12231#issuecomment-182419045.

@GrahamCampbell @devinfd I've just realized that if we want to be able to pass the $attributes array to the closure we cannot even afford to use array_map as $attribute wouldn't be updated with the evaluated closures values.
So I can see only one solution here. To go back to the original deal, maybe using the ternary operator as follow:
protected function evaluateClosures(array $attributes)
{
    foreach ($attributes as &$attribute) {
        $attribute = $attribute instanceof Closure ? $attribute($attributes) : $attribute;
    }

    return $attributes;
}

Let me know what you think about it.

This definitely needs tests.

Can you send a PR to laravel/docs for this?

@taylorotwell Yep.

I am having trouble following the docs on closure attributes, did the docs get updated @micc83 ?
Thanks
The error I receive is:
Object of class Closure could not be converted to string
in helpers.php line 671
at HandleExceptions->handleError('4096', 'Object of class Closure could not be converted to string', '/home/vagrant/tm/cc/vendor/laravel/framework/src/Illuminate/Support/helpers.php', '671', array('search' => '\?', 'replace' => array(object(Closure), 'pass', 'http://idv.ivxs.uk/idv/check?forename=john&surname=smith', '2016-05-25 15:26:34', '2016-05-25 15:26:34'), 'subject' => 'insert into `idvchecks` (`idvofficer_id`, `result`, `internal_request`, `updated_at`, `created_at`) values (?, ?, ?, ?, ?)', 'value' => object(Closure)))
at preg_replace('/\?/', 'Object', 'insert into `idvchecks` (`idvofficer_id`, `result`, `internal_request`, `updated_at`, `created_at`) values (?, ?, ?, ?, ?)', '1') in helpers.php line 671````

For code in ModelFactory.php

$factory->define(\App\Idv\IdvCheck::class, function ($faker) {
$checkValues = ['pass', 'fail'];

return [
    'idvofficer_id'     => function() {
        return factory(App\Idv\IdvOfficer::class)->create()->id;
    },
    'result'            => 'pass',
    'internal_request'  => 'http://idv.ivxs.uk/idv/check?forename=john&surname=smith',

];

});
$factory->define(\App\Idv\IdvOfficer::class, function ($faker) {
return [
    'account_id'      => 1,
    'forename'          => $faker->firstName,
    'surname'           => $faker->lastName,
    'country_code'      => 'GB'
];

});

Invoked by:

$f = factory(App\Idv\IdvCheck::class)->create();

Oh, I've just found a solution here, specifying a closure as an attribute of the factory definition doesn't seem to work as advertised.
http://laravelfaq-ujesh.rhcloud.com/2016/02/29/laravel-model-relationships-with-closure-attributes-in-factory-definition

Hello @jonwhittlestone,
I've just tried and It's working fine. Which version (5.2.x) of Laravel are you using?


App Invite dialog: The operation couldn’t be completed

I sometimes get this error and app invite dialog is not opened. Has anyone has the same issue?
[AirFacebook][NATIVE] APPINVITE_ERROR error: Error Domain=com.facebook.sdk.core Code=9 "The operation couldn’t be completed. (com.facebook.sdk.core error 9.)"
I think it only happens when I don't have Facebook application installed but I am not sure.

I didn't think you could use the app invite feature unless you had the facebook app installed.  We added a call to one of our ANE's to check and see if it was installed.  Glad to see someone else is having the same problem, but I'd like to know if this is truly correct and not just our guess?


Enforce all constructor(options) function to be consistent for Puppet Implementations

https://github.com/Chatie/wechaty/blob/63713ba7e73d17e1a99b59cbd7e4f6feeb8e4747/src/wechaty.ts#L527
With the above code, we will meet the following error:
[ts] Cannot use 'new' with an expression whose type lacks a call or construct signature.
When we have different puppet with different constructor() args.
For example: PuppetA allow constructor() but PuppetB requires constructor(options)
SOLUTION: we enforce all the PuppetImplenmentation to have options default value.

Recap: all constructor of Puppet class should be allowed to omit the options param.
Which means it should support this declaration:
const puppet = new Puppet()


TimestampResponse.Time is not in UTC

Description
When generating a TimestampResponse, the time is returned as a Localized time instead of UTC.
Example: link
However, in RFC5280, the time field is supposed to ends with Z and be expressed in Greenwich Mean Time ( source )
The real culprit here is go itself.
Version : I'm on main

However, in RFC5280, the time field is supposed to ends with Z and be expressed in Greenwich Mean Time ( source )

On top of this: even outside of the RFC 5280 profile, DER requires that GeneralizedTime be encoded with Z, and not with any relative timezone offsets.
This restriction can be found in X.690 11.7.1, which is on page 19 of the spec: https://www.itu.int/ITU-T/studygroups/com17/languages/X.690-0207.pdf
So TL;DR: this needs to be a Z to be a valid encoding, and most conforming DER parsers will reject it if it isn't 🙂


Add sort priority to properties to change the order they show up in the inspector

[x] joeferner
[ ] mwizeman joeybrk372 jharwig sfeng88

Testing Instructions:

Add an entity with two different properties
Set the sort priority of the property with a sort order at the end to a number > 0. Verify the property now appears above the other property.
Set the sort priority of the property now at the bottom to a value larger than the previous step. Verify this property now shows up higher.
Finally set both properties back to 0 and verify the original sort order is correct

CHANGELOG
Added: sort priority to properties to change the order they show up in the inspector

@joeferner looks like there are some backend test failures

pushed a fix, should be working now


Fixed: set errored to false if warning no contains errors.

https://github.com/styled-components/stylelint-processor-styled-components/issues/22
BTW, i would recommend don't changed stylelintResult, just add node in README.md about disable no-empty-source and no-missing-end-of-source-newline, in next version we can change stylelintResult (example rename warnings to another name) and stylelint-processor-styled-components will cease to work as expected.

in next version we can change stylelintResult (example rename warnings to another name) and stylelint-processor-styled-components will cease to work as expected.

That would be a major version release anyway, in which case we'd have to update and release a major version too. Might as well make that explicit rather than only mentioning it in the docs and avoid a footgun for people?

Can you fix the linting errors?
   98:1  error  Trailing spaces not allowed                                     no-trailing-spaces
  100:7  error  Assignment to property of function parameter 'stylelintResult'  no-param-reassign

BTW, i would recommend don't changed stylelintResult

@evilebottnawi Next to this pr as a patch, I think it would be a good solution to define the rules that need to be disabled in a shareable config. Users can then extend that config and list it before all others so it overrides all other rules (https://stylelint.io/user-guide/configuration/#extends).
Say for example: stylelint-config-styled-components
Then we could leave stylelintResult as is. I think defining the shareable config as a peerDependency could be a good way to allow coupling a version of this lib to a version of the config. What do you think?
Also, would you mind fixing the linting errors? If you don't have time let me know. If you could then allow me access to the pr I'd be happy to help.

@ismay sorry, no time now for this, maybe late :disappointed:

sorry, no time now for this, maybe late

No worries, linting errors are fixed 👍

Coverage increased (+0.02%) to 99.225% when pulling 173ee908e2ca62f9df347dcf364be7c98f5abf7a on evilebottnawi:patch-1 into 9842e3edf4792d57581685db34870e2cf9862091 on styled-components:master.

I'll try and rebase it this weekend btw, so we can get this merged.

Coverage increased (+0.01%) to 99.225% when pulling 86fe76ac0f836d5266c043a2b5c4ed562a436db8 on evilebottnawi:patch-1 into 68b4c4fc53d35a52fa52a3b864f516bbd98a1b56 on styled-components:master.


Saving file after update_header

Hi,
I'm trying to write the file with the same name after I apply the update_header(), but I'm getting "Permission Denied". It happens even if I close the file.
I tried to check the documentation but I can't find a way to save the file after the update_header using the same name. Am I forced to write to another file name?
I hope you can help,
pm

Ok, I'm closing this; it is possible to write the file with the same name. I was just having problems not related with pylas.
pm

And update_header is automatically called when you write the file

Ah nice, thank you for the info!
pm


Backport 'disallow project_admin from adding subnets to admin networks'

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

ok to test

Verified / tested changes locally and :+1:

retest

This needs to be rebased on RELEASE-3.0.x

rebased


target_compile_options breaks other projects

In the cmake file the target options are made public.  This causes other "not so well behaved" code to break. It should be made PRIVATE.

Ok thanks I will fix.


Add scripts to load and parse google docs

These little scripts make exporting and parsing google docs easy. For example, to load, parse, and pretty print the latest version of the reference document do:
$ export DOC_ID=1E4lncZE2jDkbE34eDyYQmXKA9O26BHUiwguz4S9qyE8
$ ./bin/article_json_export_google_doc.rb $DOC_ID \
    | ./bin/article_json_parse_google_doc.rb \
    | jq .

Coverage remained the same at 100.0% when pulling d218f3abcd8a63067ef020e75508eb454b2821dd on dsager:scripts into 0f5789dbe26e45731a17662398ac1ab1d5f7ac05 on Devex:master.

+1

+1


Simulation simulator

I convert this to draft as an alternative to https://github.com/Cloud-Storage-Team/algnet/pull/20


build: add target to create jar

Currently artman invokes CodeGeneratorTool through gradle command.
A lot of time is wasted as Gradle repeatedly resolve the workspace.
The added command instead creates one JAR,
which can be repeatedly used.

Codecov Report

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

@@           Coverage Diff            @@
##             master   #1499   +/-   ##
========================================
  Coverage        87%     87%
  Complexity     4111    4111
========================================
  Files           405     405
  Lines         16492   16492
  Branches       1759    1759
========================================
  Hits          14349   14349
  Misses         1667    1667
  Partials        476     476

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 9241de5...35247ce. Read the comment docs.

@garrettjonesgoogle @ethanbao PTAL


Precision of metric returned to HPA from datadog

Output of the info page (if this is a bug)
(Paste the output of the info page here)

Describe what happened:
apiVersion: autoscaling/v2beta1
kind: HorizontalPodAutoscaler
metadata:
  name: assortment-event-listener
  namespace: xxxxxx
spec:
  minReplicas: 1
  maxReplicas: 8
  scaleTargetRef:
    apiVersion: apps/v1
    kind: Deployment
    name: assortment-event-listener
  metrics:
  - type: External
    external:
      metricName: rabbitmq.queue.messages
      metricSelector:
          matchLabels:
            rabbitmq_queue: "event_bus.case_management"
      targetValue: 50

 ➜  kubernetes git:(master) ✗ kubectl describe hpa
Name:                                        assortment-event-listener
Namespace:                                   xxxxxx
Labels:                                      <none>
Annotations:                                 kubectl.kubernetes.io/last-applied-configuration:
                                               {"apiVersion":"autoscaling/v2beta1","kind":"HorizontalPodAutoscaler","metadata":{"annotations":{},"name":"assortment-event-listener","name...
CreationTimestamp:                           Tue, 27 Aug 2019 19:14:51 -0400
Reference:                                   Deployment/assortment-event-listener
Metrics:                                     ( current / target )
  "rabbitmq.queue.messages" (target value):  329500m / 50
Min replicas:                                1
Max replicas:                                8
Deployment pods:                             8 current / 8 desired
Conditions:
  Type            Status  Reason               Message
  ----            ------  ------               -------
  AbleToScale     True    ScaleDownStabilized  recent recommendations were higher than current one, applying the highest recent recommendation
  ScalingActive   True    ValidMetricFound     the HPA was able to successfully calculate a replica count from external metric rabbitmq.queue.messages(&LabelSelector{MatchLabels:map[string]string{rabbitmq_queue: event_bus.case_management,},MatchExpressions:[],})
  ScalingLimited  True    TooManyReplicas      the desired replica count is more than the maximum replica count
Events:
  Type    Reason             Age                   From                       Message
  ----    ------             ----                  ----                       -------
  Normal  SuccessfulRescale  60m (x10 over 12h)    horizontal-pod-autoscaler  New size: 2; reason: external metric rabbitmq.queue.messages(&LabelSelector{MatchLabels:map[string]string{rabbitmq_queue: event_bus.case_management,},MatchExpressions:[],}) above target
  Normal  SuccessfulRescale  60m (x3 over 12h)     horizontal-pod-autoscaler  New size: 7; reason: external metric rabbitmq.queue.messages(&LabelSelector{MatchLabels:map[string]string{rabbitmq_queue: event_bus.case_management,},MatchExpressions:[],}) above target
  Normal  SuccessfulRescale  47m (x24 over 14h)    horizontal-pod-autoscaler  New size: 1; reason: All metrics below target
  Normal  SuccessfulRescale  4m3s (x17 over 14h)   horizontal-pod-autoscaler  New size: 4; reason: external metric rabbitmq.queue.messages(&LabelSelector{MatchLabels:map[string]string{rabbitmq_queue: event_bus.case_management,},MatchExpressions:[],}) above target
  Normal  SuccessfulRescale  3m48s (x24 over 14h)  horizontal-pod-autoscaler  New size: 8; reason: external metric rabbitmq.queue.messages(&LabelSelector{MatchLabels:map[string]string{rabbitmq_queue: event_bus.case_management,},MatchExpressions:[],}) above target

 ➜  kubernetes git:(master) ✗ kubectl get hpa
NAME                        REFERENCE                              TARGETS   MINPODS   MAXPODS   REPLICAS   AGE
assortment-event-listener   Deployment/assortment-event-listener   329500m/50      1         8         8          14h

root@datadog-cluster-agent-559df79864-9k4st:/# datadog-cluster-agent status
Getting the status from the agent.
==============================
Datadog Cluster Agent (v1.3.2)
==============================

  Status date: 2019-08-28 14:04:52.261167 UTC
  Agent start:
  Pid: 1
  Check Runners: 4
  Log Level: info

  Paths
  =====
    Config File: /etc/datadog-agent/datadog-cluster.yaml
    conf.d: /etc/datadog-agent/conf.d

  Clocks
  ======
    System UTC time: 2019-08-28 14:04:52.261167 UTC

  Hostnames
  =========
    ec2-hostname: ip-xxx-xx-xx-xx
    hostname: i-0b1360486e5c7ac93
    instance-id: i-0b1360486e5c7ac93
    socket-fqdn: datadog-cluster-agent-559df79864-9k4st
    socket-hostname: datadog-cluster-agent-559df79864-9k4st
    hostname provider: aws
    unused hostname providers:
      configuration/environment: hostname is empty
      gce: unable to retrieve hostname from GCE: status code 404 trying to GET http://<IP_ADDRESS>/computeMetadata/v1/instance/hostname

  Leader Election
  ===============
    Leader Election Status:  Running
    Leader Name is: datadog-cluster-agent-559df79864-9k4st
    Last Acquisition of the lease: Mon, 26 Aug 2019 19:07:40 UTC
    Renewed leadership: Wed, 28 Aug 2019 14:04:45 UTC
    Number of leader transitions: 15 transitions

  Custom Metrics Server
  =====================
    ConfigMap name: default/datadog-custom-metrics

    External Metrics
    ----------------
      Total: 1
      Valid: 1
      hpa:
      - name: assortment-event-listener
      - namespace: xxxxxx
      - uid: 78b82781-c920-11e9-ba51-0ec466f8e528
      labels:
      - rabbitmq_queue: event_bus.case_management
      metricName: rabbitmq.queue.messages
      ts: 1.56700101e+09
      valid: true
      value: 329.5

=========
Collector
=========

  Running Checks
  ==============

    kubernetes_apiserver
    --------------------
      Instance ID: kubernetes_apiserver [OK]
      Total Runs: 10,314
      Metric Samples: Last Run: 0, Total: 0
      Events: Last Run: 0, Total: 1,000
      Service Checks: Last Run: 3, Total: 30,927
      Average Execution Time : 170ms

=========
Forwarder
=========

  Transactions
  ============
    CheckRunsV1: 10,313
    Dropped: 0
    DroppedOnInput: 0
    Events: 0
    HostMetadata: 0
    IntakeV1: 241
    Metadata: 0
    Requeued: 0
    Retried: 0
    RetryQueueSize: 0
    Series: 0
    ServiceChecks: 0
    SketchSeries: 0
    Success: 20,867
    TimeseriesV1: 10,313

  API Keys status
  ===============
    API key ending with 3df00: API Key valid

==========
Endpoints
==========
  https://app.datadoghq.com - API Key ending with:
      - 3df00

Notice the precision of the metric in the result from datadog-cluster-agent status ran on the cluster-agent, and notice the precision after running kubectl get hpa and kubectl describe hpa.
The worse part is sometimes the precision is good, and we get the same values on both side (when it is an integer), so imagine the autoscaler seeing 329500m as some point and then 300 at another and scaling up and down.
Describe what you expected:
I expected 329.5.
Steps to reproduce the issue:
Additional environment details (Operating System, Cloud provider, etc):

Hey @oodiete, thanks for updating the issue with your information! The values reported by the autoscaler suffixed with m represents milli (1/1000). The cluster-agent was built to support this with https://github.com/DataDog/datadog-agent/pull/3090 (>= cluster-agent v1.3.0)and will carry the conversion over into the agent status as a float type when it's deemed necessary, so we should not expect any differences between the two values when scaling.

@DylanLovesCoffee thanks for the response although I don't think that solves my problem because the hpa does not understand the m and seems to see for example 329500m as a very very large value so the scaling behaviour is it scales really up when it get 329500m and down when it gets something like 329.500 even though they should be the same.
NAME                        REFERENCE                              TARGETS   MINPODS   MAXPODS   REPLICAS AGE
assortment-event-listener   Deployment/assortment-event-listener   329500m/50      1         8         8          14h

@oodiete, I facing the same issue, did you able to solve this?

Hi all,
Let me know if I am misunderstanding the issue, but I believe this is just a data representation quirck and is not impacting the actual lifecycle of this feature.
The m is one of the visual representations that is used by the Quantity type, this type is used throughout the codebase and if you want to implement the API interfaces.
For instance, the Cluster Agent, implementing the External Metrics API's interface and registered as the server here, returns the ExternalMetricValueList type, per the function GetExternalMetric.
As you can see in the Kubernetes library, the Value is a Quantity and used as such in the code of the Horizontal Pod Autoscaler Controller.
Now, when describing HPAs, as you can see here the status of HPAs gives the values with the Quantity type (aka, m) for the AverageValues (that is used for the External Metrics types) and an integer for AverageUtilization.
Per the doc on quantities:
// Before serializing, Quantity will be put in "canonical form".
// This means that Exponent/suffix will be adjusted up or down (with a
// corresponding increase or decrease in Mantissa) such that:
// - No precision is lost
// - No fractional digits will be emitted
// - The exponent (or suffix) is as large as possible.
//
// The sign will be omitted unless the number is negative.
// Examples:
// - 1.5 will be serialized as "1500m"
// - 1.5Gi will be serialized as "1536Mi"

The above is really geared towards addressing the following statement:

because the hpa does not understand the m and seems to see for example 329500m as a very very large value

For the HPA controller, 329500m=329.5. From the status, the reason there are scaling events is because the threshold is at 50 (though I might be missing context).
Lastly to address one other potential misunderstanding, I see that the value from the Cluster Agent status is used to compare the values representation. This is just how we chose to represent the value (as Dylan explained), per his template and the humanize function here


Houston-area toll road shields

Hot off the heels of #205, a map of Texas shields would be incomplete without the ones for city that started its toll road shield craze: Houston.
Dallas may have built the first few toll roads in Texas, and even gave the Dallas North Tollway a unique shield for a few decades (before changing their minds and adopting the state standard as they added more toll roads to their inventory), but the Houston area really took off with toll roads in 1983. The Harris County Toll Road Authority (network=US:TX:Harris:HCTRA) chose the standard county pentagon for its new roads, though they opted for purple instead of blue and encircled the Authority's logo with the name of the tollway. At the time, they had only two toll roads in the works, which don't intersect each other (Beltway 8's mainlanes are free at the point where it crosses the Hardy Toll Road, and thus it is not a part of the Sam Houston Tollway), so it wasn't a huge deal that the signs happen to be completely illegible at highway speeds, as long as the shape was recognizable.
Fast forward to today: HCTRA has now crossed the ring-like Sam Houston Tollway with 3 more radial tollways, and 3 suburban counties have formed their own toll road authorities. The newest one, the Brazoria County Toll Road Authority, was formed after the standard state toll road shield guidelines had taken ground elsewhere, so they made their own that conformed to it. But the other two, the Fort Bend County Toll Road Authority and the Montgomery County Toll Road Authority (network=US:TX:Fort_Bend:FBCTRA and network=US:TX:Montgomery:MCTRA, respectively), felt like one-upping HCTRA with shields even less legible.
HCTRA has 5 toll roads, and the only toll roads signed by FBCTRA are extensions of two of these into Fort Bend County. These counties spell out the full name of the toll road on their shields, though each relation is currently tagged with a ref value derived from an initialism of the roadway's name. This is a case similar to Kentucky Parkways, where the best solution would probably be to retag these relations without ref, and use hardcoded abbreviations for these routes as a special case.
MCTRA has only one toll road, the 249 Tollway. It may be illegible, but fortunately, no special casing is needed here for the shield text, since the ref value is present on the sign.
I hope that in the future, the Houston area toll road authorities can figure something out and redesign their shields to be more legible. But god forbid they ever replace them with the state standard, we'd be in deep SHT.

network=US:TX:Harris:HCTRA on top left, network=US:TX:Montgomery:MCTRA on top right, network=US:TX on bottom row:

network=US:TX:Fort_Bend:FBCTRA:

Montgomery County is vying with Ashland County, Ohio, for the most suboptimal county shape on a shield. 😄 Needless to say, all these shields will benefit from serious simplification. Are there any examples in online or print maps, perhaps locally, that could inform us as to the elements that need to be kept and emphasized?

Unfortunately, these shields have never made it to any print map that I'm aware of. Rand McNally prefers to spell out the names of the toll roads, and the only shields present are those of the free routes parallel to them (like Beltway 8 in the case of the Sam Houston Tollway). This has been a thorn in my side since I was a kid in the back of the car staring at the atlas on cross-country trips.
The shapes and colors of the shields should be retained, but the artwork and verbose text can go. Current initialisms tagged in ref values, for HCTRA and FBCTRA, are as follows:

Sam Houston Tollway → SHT
Hardy Toll Road → HTR
[Fort Bend] Westpark Tollway → WPT
Fort Bend [Parkway] Toll Road → FBP
Tomball Tollway → TBT

These mostly align with what the ref values would be, if they went by the state guidelines. The Westpark and Tomball Tollways have syllabic initialisms, which seems recognizable and fine by me. The only questionable one for me would be the Fort Bend Toll Road, which inexplicably has a 'P' in its ref value. I would recommend FBTR be displayed instead.

The aforementioned ref=* values have been retagged as short_name=*.


feat: builtin rfc{1123,3339} date format

Restore #10264
Resolve #10614
About ISO8601 and RFC3339 diffs, see https://ijmacd.github.io/rfc3339-iso8601/

@bep


Fix HMR for circular dependencies

Changes
Circular dependencies cause hmr to no longer refresh the page. This fixes that. I outlined the problem and my proposed solution in #3607.
I had a few questions on my code changes that I'll include here as well since they seem relevant:

Should reload always be triggered? Even if update has been triggered?
What does "bubble" mean in this context? To me it seems to mean traverse the dependents, but I normally see it used in reference to events. Maybe it means the file change event?
The visited.add('RELOAD_BROADCASTED'); seems a little hacky, we could add a parameter to the updateOrBubble function and change it to return a variable named isReloadBroadcasted or something. However, that seems to add a lot of complexity to the code that is already a little complex due to the recursion.

Testing
I didn't see any existing tests for hmr, and am not familiar enough with how hmr works to feel comfortable creating tests for that. I can create some if necessary.
yarn test:dev failed before and after my changes complaining about the number of assertions, but yarn test passed before and after.
Docs
bug fix only

Hi! This PR has been working great for me, but I had to tweak the code so reload isn't always triggered. I added visited.add('RELOAD_BROADCASTED'); after hmrEngine.broadcastMessage({ type: 'update', url, bubbled: isBubbled }); and now everything works perfectly. Without this line, the whole page reloads even when it shouldn't.


Luno Streaming API

Dear Xchange team,
Is there a plan to support Lino's Streaming API?
https://www.luno.com/en/developers/api#tag/Streaming-API-(beta)
It says beta, but it's working well.
Thanks!

If you want a new exchange the best route to go is to open a PR yourself or setup a bounty to have a developer tackle implementing the exchange.

maybe you could as Luno itself to implement it


[Hotfix] Add cross-spawn package

Signed-off-by: Vipul Gupta (@vipulgupta2048) <EMAIL_ADDRESS>Motivation/Description of the PR

Resolves: #2729

Thanks, I am not sure why this package was not included in current package.json...


How to get only the id of the relationships when request to /{id}/relationships/ ?

Hi,
My work is based on JsonApi spec. Currently, I am implementing the relationships paths of my server api, I am trying to get response format like : https://jsonapi.org/format/#fetching-relationships.
However I don't know how to remove the attributes part of the data field to get only the type and the id of each target items.
Could you help me?
Thank you

Well, if you are looking for a regex that gets the id form relationships urls, I think this works:
/\/([^\/]+)\/relationships\//

For example, in node:
> /\/([^\/]+)\/relationships\//.exec("/articles/1/relationships/comments")
[ '/1/relationships/',
  '1',
  index: 9,
  input: '/articles/1/relationships/comments',
  groups: undefined ]```

As you can see, the index 1 has the matched patter, `'1'`

Hi @chamini2
That's not what I was looking for, but thanks !
What I wanted is just to query the data with bookshelf and to retrieve only the 'id' of the queried data.  Then by parsing with jsonapi-mapper, to get data without attributes like:
  "data": [
    { "type": "tags", "id": "2" },
    { "type": "tags", "id": "3" }
  ]

Finally, I success to do this by using knex queryBuilder.select('id').innerJoin(...)  then jsonapi-mapper parsing with option attributes omit 'id'


Numba on the Apple M1

Starting this as a thread to track the support of this processor. It seems like most of the things work but I guess other people will come around and also look for "what's missing".
I got it to run using packages from conda-forge and LLVM 10. My personal code runs fine and most of the unit tests pass but not all.
This depends on the following PRs to be merged on some form:

[x] Support cross-compiling llvmlite: https://github.com/numba/llvmlite/pull/677
[x] Build LLVM 10 for osx-arm64 on conda-forge: https://github.com/conda-forge/llvmdev-feedstock/pull/111
[x] Build llvmlite for osx-arm64 on conda-forge: https://github.com/conda-forge/llvmlite-feedstock/pull/50
[x] Build numba for osx-arm64 on conda-forge: PR to be made once the above ones are merged.

I will post issues as I come along in separate comments below.

In general, the hashing algorithm of CPython leads to the UserWarning every time you use Numba:
/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/cpython/hashing.py:484: UserWarning: FNV hashing is not implemented in Numba. See PEP 456 https://www.python.org/dev/peps/pep-0456/ for rationale over not using FNV. Numba will continue to work, but hashes for built in types will be computed using siphash24. This will permit e.g. dictionaries to continue to behave as expected, however anything relying on the value of the hash opposed to hash as a derived property is likely to not work as expected.

We have one test that fails due to a different hash:
ERROR: test_basic_unicode (numba.tests.test_hashing.TestUnicodeHashing)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/test_hashing.py", line 80, in check_hash_values
    self.assertEqual(nb_hash, hash(val))
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/unittest/case.py", line 912, in assertEqual
    assertion_func(first, second, msg=msg)
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/unittest/case.py", line 905, in _baseAssertEqual
    raise self.failureException(msg)
AssertionError:<PHONE_NUMBER>409129890 !=<PHONE_NUMBER>822328427

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/test_hashing.py", line 267, in test_basic_unicode
    self.check_hash_values([kind1_string[:i]])
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/test_hashing.py", line 85, in check_hash_values
    print(abs(val), hashing._PyHASH_MODULUS - 1)
TypeError: bad operand type for abs(): 'str'

Stdout:
val, nb_hash, hash(val)
ab<PHONE_NUMBER>409129890<PHONE_NUMBER>822328427
abs(val), hashing._PyHASH_MODULUS - 1

All other 5 failures are due to wrongly compiled libraries. This is definitely something to look for in the setup.py and its dependencies as this was a cross-compiled build of numba:
======================================================================
ERROR: test_manage_memory (numba.tests.test_nrt.TestNrtExternalCFFI)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/test_nrt.py", line 609, in test_manage_memory
    ffi, mod = self.compile_cffi_module(name, source, cdef)
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/test_nrt.py", line 568, in compile_cffi_module
    mod = import_dynamic(name)
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/support.py", line 641, in import_dynamic
    __import__(modname)
ImportError: dlopen(/var/folders/km/wl0tx9197gz3__0_h7y9gfyr0000gn/T/numba-tests.501/cffi_test_TestNrtExternalCFFI_test_manage_memory-0jfvgy30/TestNrtExternalCFFI_test_manage_memory.cpython-38-darwin.so, 2): no suitable image found.  Did find:
	/var/folders/km/wl0tx9197gz3__0_h7y9gfyr0000gn/T/numba-tests.501/cffi_test_TestNrtExternalCFFI_test_manage_memory-0jfvgy30/TestNrtExternalCFFI_test_manage_memory.cpython-38-darwin.so: mach-o, but wrong architecture
	/private/var/folders/km/wl0tx9197gz3__0_h7y9gfyr0000gn/T/numba-tests.501/cffi_test_TestNrtExternalCFFI_test_manage_memory-0jfvgy30/TestNrtExternalCFFI_test_manage_memory.cpython-38-darwin.so: mach-o, but wrong architecture

Stderr:
/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/cffi/cparser.py:163: UserWarning: Global variable 'status' in cdef(): for consistency with C it should have a storage class specifier (usually 'extern')
  warnings.warn("Global variable '%s' in cdef(): for consistency "

======================================================================
ERROR: test_compile_nrt (numba.tests.test_pycc.TestCC)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/test_pycc.py", line 283, in test_compile_nrt
    with self.check_cc_compiled(self._test_module.cc_nrt) as lib:
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/contextlib.py", line 113, in __enter__
    return next(self.gen)
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/test_pycc.py", line 179, in check_cc_compiled
    with self.check_c_ext(self.tmpdir, cc.name) as lib:
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/contextlib.py", line 113, in __enter__
    return next(self.gen)
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/test_pycc.py", line 84, in check_c_ext
    lib = import_dynamic(name)
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/support.py", line 641, in import_dynamic
    __import__(modname)
ImportError: dlopen(/var/folders/km/wl0tx9197gz3__0_h7y9gfyr0000gn/T/numba-tests.501/test_pycc-nft583ic/pycc_test_nrt.cpython-38-darwin.so, 2): no suitable image found.  Did find:
	/var/folders/km/wl0tx9197gz3__0_h7y9gfyr0000gn/T/numba-tests.501/test_pycc-nft583ic/pycc_test_nrt.cpython-38-darwin.so: mach-o, but wrong architecture
	/private/var/folders/km/wl0tx9197gz3__0_h7y9gfyr0000gn/T/numba-tests.501/test_pycc-nft583ic/pycc_test_nrt.cpython-38-darwin.so: mach-o, but wrong architecture

======================================================================
ERROR: test_from_buffer_pyarray (numba.tests.test_cffi.TestCFFI)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/test_cffi.py", line 30, in setUp
"""
Hash implementations for Numba types
"""

import math
import numpy as np
import sys
import ctypes
import warnings
from collections import namedtuple

import llvmlite.binding as ll
import llvmlite.llvmpy.core as lc
from llvmlite import ir

from numba.core.extending import (
    overload, overload_method, intrinsic, register_jitable)
from numba.core import errors
from numba.core import types, utils
from numba.core.unsafe.bytes import grab_byte, grab_uint64_t

_py38_or_later = utils.PYVERSION >= (3, 8)

# This is Py_hash_t, which is a Py_ssize_t, which has sizeof(size_t):
# https://github.com/python/cpython/blob/d1dd6be613381b996b9071443ef081de8e5f3aff/Include/pyport.h#L91-L96    # noqa: E501
_hash_width = sys.hash_info.width
_Py_hash_t = getattr(types, 'int%s' % _hash_width)
_Py_uhash_t = getattr(types, 'uint%s' % _hash_width)

# Constants from CPython source, obtained by various means:
# https://github.com/python/cpython/blob/d1dd6be613381b996b9071443ef081de8e5f3aff/Include/pyhash.h    # noqa: E501
_PyHASH_INF = sys.hash_info.inf
_PyHASH_NAN = sys.hash_info.nan
_PyHASH_MODULUS = _Py_uhash_t(sys.hash_info.modulus)
_PyHASH_BITS = 31 if types.intp.bitwidth == 32 else 61  # mersenne primes
_PyHASH_MULTIPLIER = 0xf4243  # 1000003UL
_PyHASH_IMAG = _PyHASH_MULTIPLIER
_PyLong_SHIFT = sys.int_info.bits_per_digit
_Py_HASH_CUTOFF = sys.hash_info.cutoff
_Py_hashfunc_name = sys.hash_info.algorithm

# hash(obj) is implemented by calling obj.__hash__()

@overload(hash)
def hash_overload(obj):
    def impl(obj):
        return obj.__hash__()
"~/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/cpython/hashing.py" 719L, 26149C
    mod.init_ool()
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/cffi_usecases.py", line 142, in init_ool
    ffi_ool, mod = load_ool_module()
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/cffi_usecases.py", line 111, in load_ool_module
    mod = import_dynamic('cffi_usecases_ool')
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/support.py", line 641, in import_dynamic
    __import__(modname)
ImportError: dlopen(/var/folders/km/wl0tx9197gz3__0_h7y9gfyr0000gn/T/numba-tests.501/test_cffi-5bjdebon/cffi_usecases_ool.cpython-38-darwin.so, 2): no suitable image found.  Did find:
	/var/folders/km/wl0tx9197gz3__0_h7y9gfyr0000gn/T/numba-tests.501/test_cffi-5bjdebon/cffi_usecases_ool.cpython-38-darwin.so: mach-o, but wrong architecture
	/private/var/folders/km/wl0tx9197gz3__0_h7y9gfyr0000gn/T/numba-tests.501/test_cffi-5bjdebon/cffi_usecases_ool.cpython-38-darwin.so: mach-o, but wrong architecture

======================================================================
ERROR: test_two_funcs (numba.tests.test_cffi.TestCFFI)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/test_cffi.py", line 30, in setUp
    mod.init_ool()
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/cffi_usecases.py", line 142, in init_ool
    ffi_ool, mod = load_ool_module()
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/cffi_usecases.py", line 111, in load_ool_module
    mod = import_dynamic('cffi_usecases_ool')
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/support.py", line 641, in import_dynamic
    __import__(modname)
ImportError: dlopen(/var/folders/km/wl0tx9197gz3__0_h7y9gfyr0000gn/T/numba-tests.501/test_cffi-kfns3l73/cffi_usecases_ool.cpython-38-darwin.so, 2): no suitable image found.  Did find:
	/var/folders/km/wl0tx9197gz3__0_h7y9gfyr0000gn/T/numba-tests.501/test_cffi-kfns3l73/cffi_usecases_ool.cpython-38-darwin.so: mach-o, but wrong architecture
	/private/var/folders/km/wl0tx9197gz3__0_h7y9gfyr0000gn/T/numba-tests.501/test_cffi-kfns3l73/cffi_usecases_ool.cpython-38-darwin.so: mach-o, but wrong architecture

======================================================================
ERROR: test_user_defined_symbols (numba.tests.test_cffi.TestCFFI)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/test_cffi.py", line 30, in setUp
    mod.init_ool()
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/cffi_usecases.py", line 142, in init_ool
    ffi_ool, mod = load_ool_module()
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/cffi_usecases.py", line 111, in load_ool_module
    mod = import_dynamic('cffi_usecases_ool')
  File "/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/tests/support.py", line 641, in import_dynamic
    __import__(modname)
ImportError: dlopen(/var/folders/km/wl0tx9197gz3__0_h7y9gfyr0000gn/T/numba-tests.501/test_cffi-rwced46f/cffi_usecases_ool.cpython-38-darwin.so, 2): no suitable image found.  Did find:
	/var/folders/km/wl0tx9197gz3__0_h7y9gfyr0000gn/T/numba-tests.501/test_cffi-rwced46f/cffi_usecases_ool.cpython-38-darwin.so: mach-o, but wrong architecture
	/private/var/folders/km/wl0tx9197gz3__0_h7y9gfyr0000gn/T/numba-tests.501/test_cffi-rwced46f/cffi_usecases_ool.cpython-38-darwin.so: mach-o, but wrong architecture

I did run the tests using python -m numba.runtests -b -j --random='0.20' --exclude-tags='long_running' -m 1 -- numba.tests

@xhochy thanks for starting to look at this. The Numba build farm should get some suitable hardware soon and we'll be able to help!
In the above, the hash tests are likely failing because of this:

/Users/uwe/mambaforge/envs/numba-test/lib/python3.8/site-packages/numba/cpython/hashing.py:484: UserWarning: FNV hashing is not implemented in Numba. See PEP 456 https://www.python.org/dev/peps/pep-0456/ for rationale over not using FNV. Numba will continue to work, but hashes for built in types will be computed using siphash24. This will permit e.g. dictionaries to continue to behave as expected, however anything relying on the value of the hash opposed to hash as a derived property is likely to not work as expected.

Where did you get this Python from? It's been compiled without siphash24 which I think requires actually forcing that to happen: https://github.com/python/cpython/blob/master/configure.ac#L2953-L2972
The tests failing where a DSO fails to load looks like an incorrect toolchain/invalid tooling problem, I think these tests will be using ${CC} to compile a library and something is evidently wrong as it's likely targeting x86 (mach-o, but wrong architecture).

Where did you get this Python from? It's been compiled without siphash24 which I think requires actually forcing that to happen: https://github.com/python/cpython/blob/master/configure.ac#L2953-L2972

This is the Python build from conda-forge as defined in https://github.com/conda-forge/python-feedstock/blob/89902c593957440acefd3acd65dfb242bd2b8ace/recipe/build_base.sh. There seems to be some ignored issues when building in the log: https://github.com/conda-forge/python-feedstock/issues/430 , we're going to investigate.

With python    3.8.6  h12cc5a1_2_cpython the hashing issues are now resolved 👍

With python 3.8.6 h12cc5a1_2_cpython the hashing issues are now resolved +1

@xhochy great, thanks for confirming.

With https://github.com/conda-forge/python-feedstock/pull/434 and https://github.com/conda-forge/python-feedstock/pull/433 all the above issues are now fixed. The test suite passes smoothly. Closing as this is working as expected, conda-forge packages for the 0.52 release on osx-arm64 are also available now.

Nice!  Thanks for all the work on this.

Im still getting an error when using the Miniconda arm osx install. The same one as described here.. The link must be for an older version on the git. Can I update somehow or will I have to find another?

Im still getting an error when using the Miniconda arm osx install. The same one as described here.. The link must be for an older version on the git. Can I update somehow or will I have to find another?

Which error exactly of the many above?

All of them, I think the miniforge distro for Mac arm is using an older build. The fail seems to be a result of llvm. I'll post the traceback tomorrow if its useful

All of them, I think the miniforge distro for Mac arm is using an older build. The fail seems to be a result of llvm. I'll post the traceback tomorrow if its useful

It seems to be that Numba is incompatible with python 3.9

It seems to be that Numba is incompatible with python 3.9

It seems to be that Numba is incompatible with python 3.9

Yes but that is a different issue unrelated to the M1. It works with Python 3.8 there.

It seems to be that Numba is incompatible with python 3.9

Yes but that is a different issue unrelated to the M1. It works with Python 3.8 there.

In case anyone is curious, the issue tracking Python 3.9 support is #6345 and the in-progress PR adding 3.9 support is #6579.

In case anyone is curious, the issue tracking Python 3.9 support is #6345 and the in-progress PR adding 3.9 support is #6579.

Hello. Importing njit kills the Notebook Kernel.
Just installed miniforge along with Python=3.8.5on my shiny new Apple M1 MBP. Installed all the packages from the conda-forge channel:
NumPy 1.19.5
Numba 0.52
Notebook 6.2.0
When doing the following the Notebook Kernel dies:
from numba import njit

Hello. Importing njit kills the Notebook Kernel.
Just installed miniforge along with Python=3.8.5on my shiny new Apple M1 MBP. Installed all the packages from the conda-forge channel:
NumPy 1.19.5
Numba 0.52
Notebook 6.2.0
When doing the following the Notebook Kernel dies:
from numba import njit

@stnatter it might be worthwhile to open a new issue to track the error you encoutered.

@stnatter it might be worthwhile to open a new issue to track the error you encoutered.

done #6650

done #6650

@stnatter thanks!

@stnatter thanks!

Python 3.9.1 works natively on m1

On Android ARM64 using python3.9 on Termux
This error occurs when i import thermosteam
I had previously compiled git version of numba
/data/data/com.termux/files/usr/lib/python3.9/site-packages/numba-0.57.0.dev0+782.g04d4af3ce-py3.9-linux-aarch64.egg/numba/cpython/hashing.py:524: UserWarning: FNV hashing is not implemented in Numba. See PEP 456 https://www.python.org/dev/peps/pep-0456/ for rationale over not using FNV. Numba will continue to work, but hashes for built in types will be computed using siphash24. This will permit e.g. dictionaries to continue to behave as expected, however anything relying on the value of the hash opposed to hash as a derived property is likely to not work as expected.


(BUG) [RN] 2672 code withdraw failed Private key must be 32 bytes long

Description
This PR adds decoding of url for deeplinks for both web and android. It also fixes the regex used in all base64 conversions to remove all padding automatically added and prevent having links with '=' at the end
About #2672
How Has This Been Tested?
Generated various links by sending money between different accounts.

generate links by adding reason to the transaction
generate links by leaving empty the reason of the transaction

Checklist:

[x] PR title matches follow: (Feature|Bug|Chore) Task Name
[x] My code follows the style guidelines of this project
[x] I have followed all the instructions described in the initial task (check Definitions of Done)
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have added reference to a related issue in the repository
[ ] I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
[ ] I have added screenshots related to my pull request (for frontend tasks)
[ ] I have pasted a gif showing the feature.
[x] @mentions of the person or team responsible for reviewing proposed changes

the decodeURI is probably also required for deeplinking.native.js, on the incoming deep link
search code for other instances of /==$/ and replace

both requests done.
in regards to 1, without the decode it works fine too. I've tested with links sent by cli and it handled the params correctly. either way adding the decode is probably safer and better for consistency, plus we haven't tested native deeplinks in prod yet so without it it could fail in the future


Update setup.sh (Windows)

(10-31) creates a symbolic link between python3 and python for Windows Python users that don't have this set
(36-40) changed path to venv activate script for Windows

Awesome - thanks Ren!


added allow_replace_deleted in hnswlib.py

Description of changes
HNSWlib deletes nodes by marking them as deleted. These nodes will be excluded from the results, but they will still persist in the graph and in memory. Currently, there is no way to free the memory used by these deleted nodes. However, there is a workaround (which is also implemented by hnswlib itself) that involves replacing the deleted nodes with new nodes that are added to the graph.
https://github.com/chroma-core/chroma/blob/465495d16298c27c26d81043aa2716a26b221607/chromadb/db/index/hnswlib.py#L162
Currently, Chroma is using default paramerter for replace_deleted in add_items( ) i.e False.
Reference to HNSWLIB:
https://github.com/nmslib/hnswlib/blob/master/README.md#api-description

Improvements & Bug fixes

This PR  fixes  the above issue.

Test plan

Same as before

Documentation Changes

Nil

@Satyam-79 @HammadB will look into this next week

Closing this out, we can incorporate it into the new system in the future.


Add Texas Instruments DACx0501 driver

Adds a driver for Texas Instruments DACx0501 family of digital to analog converter devices

@martinjaeger Can you please review the DAC driver addition?
Thanks!

@martinjaeger Can you please review the DAC driver addition?
Thanks!

Sure, will try to have a look ASAP.
May I ask you already to fix the commit messages according to the links to documentation posted by the bot above. Merge commits are not allowed and have to be removed. Use git rebase instead of merge.


Push library to JCenter

The library releases only show up on Maven Central/Bintray and not JCenter search.
I know JCenter syncs from Maven Central, but do you have to do anything else to show up in their search? Android has switched to using JCenter rather than Maven Central, so showing up in their search will help with getting more devs using the library.

The library is published to JCenter first and then synced to Maven Central. Which search were you not able to find it in? From this page https://bintray.com/bintray/jcenter ?

Sorry about that, apparently their search is terrible and doesn't update quickly when you type or submit. I waited 10 more seconds and the result finally showed up.


Add ClusterIP to the services in the egctl test data

What type of PR is this? fix: fix egctl test data

What this PR does / why we need it:  xds endpoint configuration requires address under socketAddress field. We need to provide ClusterIP in the input data file.
Which issue(s) this PR fixes: #1742

Fixes #

thanks @Ronnie-personal ! does this allow you to add the ValidateAll() method back for Endpoints ?

thanks @Ronnie-personal ! does this allow you to add the ValidateAll() method back for Endpoints ?

Yes, we will be able to add  endpoints ValidateAll(), after this change.
I tested locally, egctl tests passed with ValidateAll() in place.

Can you add ValidateAll into this PR as well ?

Can you add ValidateAll into this PR as well ?

Sure, will do.


Evaluate and choose role/permission gem(s)

We currently plan to have two roles (Admin and User) and only one permission (hasMediaVaultAccess). However, our solution should assume neither of those are constant.

✅ Authentication: We're using Devise.
⏳ Authorization candidates:

CanCanCan
Pundit

⏳ Role definition/management candidates:

Rolify

I've dived through the documentation for the suggestions above, and done a little more surveying of options. There aren't a lot! I've also thought more about this and our actual current needs:

All our users have access to the FCI data
Some of those users may have access to Media Vault
All of our admins can do everything

Is there anything else even on the horizon? Perhaps… being able to upload your own things to Media Vault might be something we restrict to certain users? (I don't think we ever intend for people to be able to upload to the shared corpus, so this would just be the ability to have your own private archive.)
Given the simplicity of our current set of permissions, and that they are all application permissions (not resource permissions) I think we should strongly consider not adding the complexity of a gem. This would be much easier to add just as attributes on the User model.
Big caveat to all of this: We're not going to be using the "organization" concept that's hiding in the current codebase. If we were, we would definitely be considering a gem, since it would handle precisely this sort of scoped/relative authorizations.
@cguess @oneroyalace Would love have chat for a few minutes with y'all about this before moving forward!

Update: Okay, I actually dived a little more into the documentation for Rolify (specifically) and thought about it, and… I think Rolify on its own is a sensible middle ground that would minimize future disruption if we wanted to get more complex with our permission structure.
So I'd like to choose Rolify on its own without any authorization gem; in the future we can insert one as needed (I think).

Got sign-off from @cguess to start with Rolify sans CanCanCan.


bots: Replace some redundant logic with is_cross_realm_bot_email.

is_cross_realm_bot_email is just
email.lower() in settings.CROSS_REALM_BOT_EMAILS which is the same, aside of looking at .lower() - which is actually more correct.

Merged, thanks @mateuszmandera! Probably the .lower detail isn't important, since we should be comparing against the exact original value, but definitely more obviously correct this way :)


Elements following headers get header style in Mac Pages

Hi!
When I have a header (e.g. h1) with some following element(s) (e.g. p), the latter elements get interpreted as headers as well. For instance, if I pass the following html string:
<!DOCTYPE html>

   <meta charset='UTF-8'>
   <title>The useEffect hook (updated)</title>

Heading

This gets big and bold as well

the downloaded document looks like this:

The same goes for any kinds and amount of subsequent elements. If I would put the paragraph before the h1 in the example above, the paragraph would be rendered correctly as a p.
I tried to look around for any similar issue, but I couldn't find any. Can you tell if I'm doing something wrong, or is this a bug?

I personally don’t see this issue, but can say the best way to triage it is
to fix it by hand and do a diff of the OOXML tags to see how pages renders
it.
From there, you can make a PR with an option such as “forMacPages: true”
or something like that. Maybe there’s a way it can support both?
On Fri, Jan 13, 2023 at 3:53 PM Petter Tasola @.***>
wrote:

I've looked a bit more into this, and the problem might only be related to
Mac Pages. At least when I try to upload it to Google Documents, the
paragraphs are properly rendered.
When inspecting the paragraphs in Pages, it says they are headings (in the
case above, the paragraph would be interpreted as a Heading 1 in Pages).
I've tried to get all paragraphs (getElementsByTagName('p')) and applying
some style to them before sending the html to html-to-docx. I can decrease
the font-size by doing this, but the font-weight will still match the
preceding header's font-weight. I guess Pages overrides the font-weight
property to its default font-weight of the heading-element it interprets
the paragraph as.
I would still love to get some input on this issue. Is it a known bug?
—
Reply to this email directly, view it on GitHub
https://github.com/privateOmega/html-to-docx/issues/180#issuecomment-1381619352,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACHLNTSWRBWB6ULTKZ6LNADWSEUJDANCNFSM6AAAAAATYFFNX4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@nicolasiscoding Cool, thanks for the input. I'll look into it if I get some time.
Just to be clear - you're not experiencing the same thing if you run the function with the html in my initial post, with version 1.6.4 of html-to-docx and open it in Mac Pages...?

@tasola So I tested the corresponding output on multiple word processors, seems like the issue is on mac pages only. Attaching the screenshot for reference.
word

libre office

mac pages

@nicolasiscoding I think to debug this problem, its gonna be super hard considering each word processor might work differently based on order of tags etc. And I wouldnt suggest changes being proposed based on each word processor since that would complicate things further.
Closing this issue since issue isnt with html-to-docx, but with the viewer application. Please feel free to reopen it if you think I made a mistake.


Multilingual sentence translation

I have a sentence that has two languages or I may have a HTML page that is a mix of English and Chinese and I would like to know how does translation api handle such cases? Right now it just ignores the Chinese text. Please see screen shot below.

Thank you for your question.
Can you please share with us the URL of the document that you are having issues with?
Please note that we scope issues on this repo to feedback related to the docs. If you have a product question, please post it on Microsoft QnA or Stack Overflow. Thank you for your understanding.

Thank you for your question.
Can you please share with us the URL of the document that you are having issues with?
Please note that we scope issues on this repo to feedback related to the docs. If you have a product question, please post it on Microsoft QnA or Stack Overflow. Thank you for your understanding.

@neilmario70 - Since we have not heard back from you we will now proceed to close this thread.
However if you can share additional information regarding the documentation you are referring and circle back on this thread , we will gladly reopen the issue and continue the discussion. Thank you.


Don't decompress VEP FASTA

Decompressing takes forever and also uses a huge amount of disk space (the compressed file is just 1 GB, but it expands to tens of GBs).

Good question, but seems to have worked fine without :)


how to get csrf token

i can't find where to get the csrf token

The CSRF Token is included in Earnapp requests on the dashboard and is in the cookies storage, follow this method but with xsrf-token and not oauth-refresh-token.


The title of the safety points page is displayed incorrectly when backing up the mnemonic phrase

When did back up the mnemonic, the title of the safety points page is displayed as create a did

expected outcome:
display as Backup DID
version：Windows <IP_ADDRESS> macos <IP_ADDRESS>

win139&mac127 test passed.


Bloxstrap is unable to connect to roblox

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] My answer given in the checkbox above is a lie.

What problem did you encounter?

please help i've tried everything to fix, nothing works.


Remove vcredist workarounds for .NET Core 3

Following up from #501, we need to remove the vcredist workarounds added once the fixes are in the framework.

External: https://github.com/dotnet/wpf/issues/37

Done in Preview 6


Error searching for the file: Can't deserialize SearchResultsOrSearchResultsWithSharedLinks

Description of the Issue
I cant Search
Steps to Reproduce
`  let fileName='o2ma1000.ojn'
try {
const fileSearch = await client.search.getSearch({query:fileName});
} catch (error) {
console.error("Error searching for the file:", error);
return null;
}
`
Error searching for the file: Can't deserialize SearchResultsOrSearchResultsWithSharedLinks

It looks like problem on the SDK side. We'll look into it. I'll let you know of in case of any updates.

It's a spec problem. Will be solved by https://github.com/box/box-openapi/pull/357

Should work now 👍


Adds an extra Non-Existant Plex "Media Machine"

When going to Settings --> Plex and using the "Sign In/Add Servers" button, it adds 2 Plex servers. I only have one. The first one it adds is correct. It's a valid IP address, and the "Name" is my computer's name. The second one it adds has the Name "Media Machine," and the IP address does not exist on my network.
PS> I had to manually change the "HTTPS" to "HTTP" for the first server to connect. I did the same to the second to no avail.

 Just remove the second server.

Thank you!
 Just remove the second server.
—
Reply to this email directly, view it on GitHub
https://github.com/vexorian/dizquetv/issues/433#issuecomment-1401671265,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHXQ4QSGKRI4C6LIAZUQDNLWT6SZ5ANCNFSM6AAAAAASBQC4HY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Adds test about icmpv6 type and code detection

https://redmine.openinfosecfoundation.org/issues/3323
Base test case From #172
From the paper, the first failing test by Suricata is denial6-5
cf Table 4: IDSv6 Benchmark Results.
I think the authors expected a signature present by default to trigger (ie some app-layer event)
But we can easily create the signature to detect as is shown in this Suricata-Verify test.
Also note that 51 Authentication Header is a known extension header even if it is not supported by the target (ie 2001:db8:1::1)
and that attacker is in 2001:db8:1::2/128 rather than in 2001:db8:2::0/64

Added another commit in the same branch

Maybe for the latest smurf, we can reduce the pcaps from the paper : getting 591234 alerts on ipv6-rsmurf

Should we worry about this?
remote: warning: File tests/ipv6-evasion/ipv6-rsmurf/rsmurf6.pcap is 53.00 MB; this is larger than GitHub's recommended maximum file size of 50.00 MB

Should we worry about this?
remote: warning: File tests/ipv6-evasion/ipv6-rsmurf/rsmurf6.pcap is 53.00 MB; this is larger than GitHub's recommended maximum file size of 50.00 MB

Yes, we surely can take a smaller part of the pcap. Should I ?

Replaced by https://github.com/OISF/suricata-verify/pull/534


Sort Categories Alphabetically

Allow user to sort the categories alphabetically in EditCategories when clicking on the column header. For help, see how transactions are ordered by date in RawView.

When the user hits save on EditCategories Presenter, resort the categories in alphabetical order as well.


Intercept

Argument to call an interceptor function
intercept:filter:qualifiedMethod
example:
java -javaagent:/PathTo/javaAgent.jar=intercept:org.postgresql.jdbc.PgConnection#prepareStatement:com.test.Interceptor#staticFunction -jar yourapp.jar
Where
class com.test.Interceptor{
  public static void staticFunction(Object self, Object args[]){
     log(args);
  }
}

There is a problem. Intercepting at Functions and intercepting constructors are different things. Also are different intercepting at the function start and at the end.
More options will be needed. Maybe:

interceptCall
interceptReturn
interceptConstructor

interceptConstructor is done

Done


Fix typo in URL

There is a typo in the URL attribute in pom.xml that makes it so the README content isn't showing up on https://plugins.jenkins.io/cloudbees-enabler/ as expected.

needs releasing to fix the docs


Encode Global Settings File

This PR encrypts / decrypts strings in the global settings file.
fix #369

The values of the API keys in the config.json text file appear the same as on main for me 🤔
I even deleted the file and saved a fresh one through the app

Whoops! Was just logging the encoded values for testing, never actually saved them...

One small conflict now

Should be good to go!

@garrettmflynn The new config file contents look good, but I can't navigate away from the settings page in the app after resetting the values
The popup

blocks me even if I keep hitting 'save and continue'

Good catch. Fixed

OK looks good now 👍


Prototype/tasks v2 2 merge prune robot panel

This PR tracks a merge of various work into a single branch, and tracks the bugs that are still present that should be resolved before merging to prototype/tasks_v2-2. After that merge, the follow up target would be to merge into main.
Features of this branch to be merged into prototype/tasks_v2-2

Includes robot panel edits to remove redundant / unused components: https://github.com/open-rmf/rmf-web/pull/576
Includes event logs as described here: https://github.com/open-rmf/rmf-web/pull/573
Does not include bug fixes for multiple subscriptions as yet, which is present in main: https://github.com/open-rmf/rmf-web/pull/577 This has been reverted and includes the bugfix for multiple subscriptions

There are a few bugs/user "gotchas" as of now

Sending delivery requests from the create-task panel fails as we cannot specify a payload. This could be resolved with another field in the create-task panel

[full_control-12] [INFO] [1644943525.937784621] [tinyRobot_fleet_adapter]: [Bidder] Received Bidding notice for task_id [delivery.dispatch-0]
[full_control-12] [ERROR] [1644943525.938383205] [tinyRobot_fleet_adapter]: Received a request description for [delivery] with an invalid format. Error: At /dropoff/payload of "" - no subschema has succeeded, but one of them is required to validate
[full_control-12]
[full_control-12] Request:
[full_control-12] {
[full_control-12]   "dropoff": {
[full_control-12]     "handler": "coke_ingestor",
[full_control-12]     "payload": "",
[full_control-12]     "place": "hardware_2"
[full_control-12]   },
[full_control-12]   "pickup": {
[full_control-12]     "handler": "coke_dispenser",
[full_control-12]     "payload": "",
[full_control-12]     "place": "pantry"
[full_control-12]   }
[full_control-12] }
[rmf_task_dispatcher-10] [WARN] [1644943528.137506720] [rmf_dispatcher_node]: Dispatcher Bidding Result: task [delivery.dispatch-0] has no submissions during bidding. Dispatching failed, and the task will not be performed.
[rmf_task_dispatcher-10] [ERROR] [1644943528.137573326] [rmf_dispatcher_node]: No submission error[1]: {"category":"Invalid request format","code":5,"detail":"At /dropoff/payload of \"\" - no subschema has succeeded, but one of them is required to validate\n"}
[schedule_visualizer-5] [2022-02-16 00:45:28] [error] handle_read_frame error: websocketpp.transport:7 (End of File)
[schedule_visualizer-5] [INFO] [1644943528.470067827] [rmf_visualization_schedule_data_node]: [TrajectoryServer] Disconnected with a client
[schedule_visualizer-5] [INFO] [1644943530.202191853] [rmf_visualization_schedule_data_node]: [TrajectoryServer] Connected with a client
[full_control-12] [INFO] [1644943535.903235009] [tinyRobot_fleet_adapter]: Replanning for [tinyRobot/tinyRobot2] because of an interruption
[full_control-12] [INFO] [1644943535.904153263] [tinyRobot_fleet_adapter]: Replanning for [tinyRobot/tinyRobot1] because of an interruption
[gzserver-14] [INFO] [1644943535.909809612] [slotcar_tinyRobot2]: tinyRobot2 received a path request with 2 waypoints
[gzserver-14] [INFO] [1644943535.912172978] [slotcar_tinyRobot1]: tinyRobot1 received a path request with 2 waypoints

Unsuccessful ( unassigned ) tasks end up on the task panel anyway, with an "unknown" status, rather than failed:

Running under use_sim_time=true, we need to manually specify the start time of each task on the create-task panel as 0. Otherwise the panel submits the task start time as unix time, which causes task to be queued far in the future

Running under use_sim_time=true, the current approach to calculating progress is inaccurate

Task cancellation doesn't work, but clicking the button in the task panel reports success. There might be underlying issues as well in the RMF backend, issue here: https://github.com/open-rmf/rmf_ros2/issues/175

Tests are borked Tests are less borked by removing many of them, but need to fix them ASAP in a folllow up

Otherwise, submission of Loop Requests under use_sim_time:=false seems to be working well, seen in this video.
https://user-images.githubusercontent.com/12628669/154110886-747f68f0-d36d-4a8c-943f-aa3ac9fd23fc.mp4
Notably, these edits may not be ideal, but were necessary to handle some validation errors. Do advise if these are not the best solution:
https://github.com/open-rmf/rmf-web/pull/581/files#diff-f8cf7ad0d839656743c8f01b8df95728fd6921717927d8958b22ef11208d92c4L22-R23
For the following error:
line 127, in serialize_response
    raise ValidationError(errors, field.type_)
pydantic.error_wrappers.ValidationError: 2 validation errors for TaskDispatchResponseItem
response -> success
  field required (type=value_error.missing)
response -> state
  field required (type=value_error.missing)

Running under use_sim_time=true, we need to manually specify the start time of each task on the create-task panel as 0. Otherwise the panel submits the task start time as unix time, which causes task to be queued far in the future

I believe that is the correct behavior. The client always submits requests in unix time, and rmf always processes requests in unix time as well, this is same regardless if it is using sim time or not. Tasks are queued far in the future because rmf's time starts at 0.

Running under use_sim_time=true, the current approach to calculating progress is inaccurate

The calculation is borked because it uses the client's time, actually this is a problem even if sim time is not used because there is no guarantee that the client's and rmf's time is synced. It won't be accurate as long as the client is doing any calculation based on the time. We can have rmf provide the progress instead, which will always ensure a correct estimate, but it would only "tick" when a new task state comes in.

Summary of to-do after merging:

[ ] Modify create-task panel to add sku, quantity inputs for Delivery Requests
[ ] Unsuccessful dispatches should be displayed as "failed" on task-panel instead of "unknown"
[ ] In the longer term, figure out a way make task progress calculations independent of the client time
[ ] Make task cancellations work, after this possible bug is resolved: https://github.com/open-rmf/rmf_ros2/issues/175


DXCDT-405: Fix incorrect number rendered in header for auth0 apps list

🔧 Changes
When running auth0 apps list, the number of apps listed in the header is one higher than the actual number of apps rendered in the output. This is because the global client (mostly deprecated) is included in the header number but is filtered by the output. The fix here is to use the length of the filtered applications to determine the header number instead of the raw number of clients which likely include the deprecated global client.
🔬 Testing
Primarily manual but mostly covered by existing integration tests.
📝 Checklist

[ ] All new/changed/fixed functionality is covered by tests (or N/A)
[ ] I have added documentation for all new/changed functionality (or N/A)

Codecov Report
Patch coverage: 54.54% and no project coverage change.

Comparison is base (e10608c) 50.45% compared to head (e3b4f26) 50.45%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #674   +/-   ##
=======================================
  Coverage   50.45%   50.45%
=======================================
  Files          93       93
  Lines       11764    11764
=======================================
  Hits         5936     5936
  Misses       5400     5400
  Partials      428      428

Impacted Files
Coverage Δ

internal/display/apps.go
76.82% <54.54%> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Legendary Ship: Spacer Scavenger Blattodea

After beginning the quest 'Legendary Ship: Spacer Scavenger Blattodea', leaving and returning can cause all the ships to disappear from this area, making the quest incompletable.

Game Version
Full Release Version
New Game
No
Reproduction Steps
If you enter orbit of Volii Omega, you receive the quest Legendary Ship: Spacer Scavenger Blattodea.
After doing this and then leaving to different planets, locations, doing quests, sleeping days, returning, the ships can disappear.

It looks like the ships actually change systems if you level too much - check if the listed system / planet on the quest has changed.


Implement (De)Seralization for Pinned Smart Pointers

Implements SerializeAs and DeserializeAs for Pin<Box<T>>, Pin<Rc<T>>, and Pin<Arc<T>>

Seems the lints are bugged and are flagging the macros as unused when they are in fact used. I'll go ahead and add an #[allow(unused-macros)] for now.

Hi, thank you for submitting the PR. Everything looks good here. I extended the SerializeAs side with support for Pin<&'a T> and Pin<&'a mut T>.
I wonder if it is possible to support any Pin<T>, not just with specific pointer types. Pin::as_ref almost looks like it could work, but then is also return Deref::Target and I couldn't figure out how to make that work. On the deserialization side we need the well-known pointer types, since only they have safe conversions to Pin.
I leave this PR open for a day or two, in case you have an idea how to extend the serialization side, otherwise this looks good to merge.

Yea, honestly not sure about how to go about extending it to work using Deref stuff. Had a go at it myself and couldn't really get very far with it. Might come back to it later as it seems like an interesting idea, however I'm just going to leave it for now I think.


Fixes Array.get() implicit wrapping with negative index values

This is a response to #100.

Hi, as I wrote in #100 the issue was addressed to be fixed in the next major release.
I just pushed an update into the Swift-1.2 branch to do so.
Thanks anyway for your PR.


An unmigrated test is running on an NNBD bot

The test dart2js_extra/switch_test/00 expects a compile-time error because of a case block that doesn't end with one of the terminating statements, such as break.  Instead, the block contains a call to a function that never returns (it's return type is Never).  In NNBD it's allowed to end a case block with such an invocation, so the test should be migrated because it's running on a bot where NNBD is enabled: dart2js-weak-linux-x64-d8.

Ah, we should actually for now opt-out dart2js_extra and dart2js_native (we intended to run these in legacy mode for a while longer until we migrate them over)


Add additional_user_data and additional_users inputs, recreate ASG and EC2 on launch config updates

A new additional_users module input allows specifying additional SSH users to be created. The available per-user fields are login, gecos (full name), shell, supplemental groups, and SSH authorized_keys.
A new additional_user-data input allows specifying content to add toward the end of EC2 User Data. The additional User Data is executed before additional users are added.
The Auto Scaling Group (and its bastion EC2) will now be recreated when there is an update to the Launch Configuration. The new Auto Scaling Group will be created before the current one is deleted. Previously the EC2 would remain untouched, leaving its recycling to operator discretion.

 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.

@coreypobrien I was able to create 20 users, each with a single 4096-bit key now that the User Data is being compressed. Let me know if your use-case requires more and I'll work on Terraform pushing to SSH and User Data pulling from S3 to execute.

Awesome 👍


Anchor program mainnet: InvalidRentPayingAccount

I deployed a program that works fine on tests and Devnet, but on Solana Mainnet, I get an error which I haven't found info by Googling it:
Raw transaction 5SJvH8NPKLPumNWmbB6nscwdtr1LpGsF48kPGiBTy4vkFi4k78L6Q36gaCbqzCbvck4jRu42M64Y72a7DtwNNiKT failed ({"err":"InvalidRentPayingAccount"})

In the Mainnet explorer, the transaction is this one.
Interesting enough is that the same transaction for the same programId on Mainnet has 1 "System Program: Create Account" additional transaction that Devnet has not.
The place where I get the above error is in this logic:
const provider = getProvider();
      const { program, gameAccount, houseWallet, clientWallet } = getProgramByClient(provider, client, amount);

      solana.createPDA(solanaWallet.wallet.publicKey!, program.programId).then(async (playerAccount) => {
        try {
          const accounts = {
            game: gameAccount,
            player: solanaWallet.wallet.publicKey!,
            playerData: playerAccount,
            houseWallet,
            clientWallet,
          };

          const result = await program.methods.flip().accounts(accounts).rpc(); <== HERE

As I mentioned, this one won't fail on Devnet with a Phantom Wallet.
How can I solve the InvalidRentPayingAccount error on Mainnet?

@paul-schaaf you faced a similar error on this github issue: https://github.com/solana-labs/solana/issues/23670
I think my issue is related and can't find much info on the internet, do you have a clue?

Rent paying accounts are no longer allowed(https://github.com/solana-labs/solana/issues/24049):

The change adds a failure case for transactions: if the transaction leaves an account in a newly rent-paying state (ie. the account started as uninitialized or rent-exempt), the runtime now fails the transaction with TransactionError::InvalidRentPayingAccount.


Toggle connections via client api

Hello, I am using the client class to create an auth0 SPA.
I cannot find a way to enable/disable specific connections.
Is it currently possible?
If not it would be a nice feature to have.
Thanks.

I believe you can create connections and add/remove clients from the enabledClients list.


GH-1613: Fix Race in Producer Factory

Resolves
Do not clear the cache in destroy() - just drain the producer queues instead.
Fixes a race condition where a producer could be returned to the cache between draining
the queues and clearing the map.
Just leave the queue(s) in place to accept new returns.
cherry-pick to 2.5.x, 2.4.x, 2.3.x, 1.3.x

Cherry-picked to 2.5.x, 2.4.x & 2.3.x.
There is no cache.clear() in the 1.3.x branch. So, nothing to back-port.


Open new instance of Simulator to allow parallelizing UI tests

Adding the NSWorkspaceLaunchNewInstance launch option to the NSRunningApplication allows multiple simulators instances to be run concurrently on a single machine.

Hi Stephan,
Thanks for your PR.
Could you provide more details on how it will improve your workflow?
This change doesn't parallelize application (or yet unsupported UI) tests but I guess it makes it closer to that goal. I am just trying to understand why you need this and if it actually solves issues you are trying to solve.
Thanks!

You can create multiple simulators (with simctl), all of the same device type and runtime, but with unique UDIDs, and xcodebuild supports running them simultaneously. It doesn't work with XCTool because you basically need to launch the Simulator.app with the -n option, which as I understand it, is the NSWorkspaceLaunchNewInstance option in that NSWorkspace launchApplicationAtURL method.
With this change, you could parallelize your UI test by running something like this:
xctool -workspace "MyWorkspace.xcworkspace" -scheme "MyScheme" -sdk iphonesimulator -destination "id=F3939D21-EA01-4F6F-AF60-F944993926CA" run-tests -only MyUITestsTarget:Tests/test1,Tests/test2,Tests/test3,Tests/test4,Tests/test5
xctool -workspace "MyWorkspace.xcworkspace" -scheme "MyScheme" -sdk iphonesimulator -destination "id=4FCB211A-4533-4EF2-BAC4-BCB6239F6757" run-tests -only MyUITestsTarget:Tests/test6,Tests/test7,Tests/test8,Tests/test9,Tests/test10
xctool -workspace "MyWorkspace.xcworkspace" -scheme "MyScheme" -sdk iphonesimulator -destination "id=B9681E3D-E108-4E3D-AD94-6288583D5DE0" run-tests -only MyUITestsTarget:Tests/test11,Tests/test12,Tests/test13,Tests/test14,Tests/test15
xctool -workspace "MyWorkspace.xcworkspace" -scheme "MyScheme" -sdk iphonesimulator -destination "id=60BFC8F7-48CA-42E2-93D2-CB439C3266CD" run-tests -only MyUITestsTarget:Tests/test16,Tests/test17,Tests/test18,Tests/test19,Tests/test20

I should clarify what I mean by "UI" tests. This actually wouldn't be for the new Apple UI tests that were recently introduced with iOS 9, though it would probably work with that too. I'm using KIF for automated UI tests in my app and xctool to do that with CI.

I've got a working script run by a Jenkins job (that uses xctool) and have been using it extensively, but have up until now, have neede to run tests on multiple VMs (because of only being able to launch a single instance of the Simulator.app, per machine).
This change takes my existing scripts that use xctool, and allows me to launch multiple Simulator.app instances on the same machine/OS (without VMs) simultaneously.
My script basically lists all the tests with the -listTestsOnly option, then breaks that up into 20 chunks and spins off 20 parallel instances of the Simulator (in different VMs), and then runs them in parallel because UI tests are so slow. I'm able to take about 2 hours of UI tests and run them in 20 minutes. VMs are slow and add all kinds of flakiness to my tests, so being able to run my tests in parallel on a real machine, not a VM would be soooooo great!

Got it. Thanks for details of how you are going to rely on this feature. I have nothing against merging it today and now. The only thing you should keep in mind is that xctool might kill sometimes all simulators and other simulator related services (see https://github.com/facebook/xctool/blob/master/xctool/xctool/SimulatorWrapper/SimulatorUtils.m#L97). In that case xctool might kill simulators that used by another instances of xctool (just be aware that is might cause some flakiness of your tests).
Ideally we should add that feature to xctool itself. Xctool already could parallelize logic tests but can't application tests. With this small change we are one step closed to parallelize application tests as well and I think I should look into it.

What happens if you run xctool N times serially with the same arguments? Does it open N simulator apps or only 1?

100% agree. This just gets me what I need for now. I'd LOVE to see parallelizing app tests built into xctool.

I created a sample project and script so you can test this out. It just had 20 tests that take about 3 seconds each to run. This script will divide the tests up into 4 groups and run them in parallel on iPhone 6 Simulators.
https://www.dropbox.com/s/6ultzpjs3bnukh8/MyTestProject.zip?dl=0
python make-test-script.py > runTests.sh
Fetch this PR, compile it, replace the path to xctool in runTest.sh to the one you just built.
./runTests.sh

Thanks for a test project. I've checked it out - works great. So I have now only one concern. If you run xctool -project MyTestProject.xcodeproj/ -scheme MyTestProject -sdk iphonesimulator test -only MyTestProjectTests:MyTestProjectTests/test01 consequentially 4 times then there will be 4 simulator apps opened. We want to avoid that because locally I would expect people not to do what your script does.
I think for now we will need to pass extra option to run-tests step that will enable feature you are adding. Currently we have next options: -freshSimulator, -resetSimulator and -noResetSimulatorOnFailure. I would add something like -newSimulatorInstance or whatever you think best describes functionality you are adding.
Does it make sense to you?

Yes, I will add that. Thanks!

@ExtremeMan, is there anything else you need before you can merge this?

Since it is gated by new option I think we should merge! Thanks!


The MetaMask struggle continues

MetaMask has two presentations, one is an overlay that is always on top of the browser and the other is a popup which can disappear behind the browser if the user is not careful. The popup is forced open by eht_requestAccounts or wallet_requestPermissions. The former is a nightmare because calling it twice with the popup behind causes things to lock up. The user can only fix it by using the overlay. The later works better but still requires teh user to use the overlay for adjustments.

Mhh, what are you trying to achieve with MM?

@Siegrift : So this a component requested by Ashar and Ben. They are building out new guides for the docs and want an interface to MetaMask. It appears we may now have a handle on things and can do want Ashar wants to accomplish which is:

Deploy a contract to Goerli only.
Call [n] number of functions or get on a public variable.

See the (not finished component and it docs for use here)

I am unable to connect to MM. I don't see any connect button nor instructions how to connect. Am I missing something? (I like the idea of the widget though).

@Siegrift - There should be two widgets at the top of the page and one at the bottom. https://vitepress-docs.web.app/dev/using-metamask/
The top two are different  implementations. One (on left) uses the MetaMask overlay and the other (on right) MetaMask popup. I seem to remember you are on a Mac wit apple silicon, same as me, so they should be working. If not we are still working on it and I will keep you updated.

Just fyi:

And I am not able to connect to any of the widgets.

If not we are still working on it and I will keep you updated.

Yeah, I stumbled on this just by accident and was curious. Let me know if you need my help/testing.


Forms Builder

We are excited in introducing a powerful feature in spurtcms the concept of Forms Builder.

The Forms Builder in SpurtCMS will undoubtedly enhance user experience by allowing for easy and customizable form creation.It’s great to see such powerful tools.


Figure out bg color of none style somehow

The current method of forcing users of the none style to define their color with the solid color style is not friendly at all. It's actually worse now because if you don't read the readme, the plug will crash and you'll have no idea why.
We need to do gtk detection somehow. Whether we have to do that in the plug or in the socket, I don't presently know. If I can set the widget class to match xfce4-panel in the plug, then sweet, we can just do it in the plug. Otherwise, we'll have to have the socket do it and tell the plug somehow.
If we have to use the socket to do it, we could have the socket set the xfconf property, maybe.
One problem will be detecting GTK theme changes. There's probably not a way to work around this. There is a signal to let us do it or there isn't.

c52684835ca0e2cd3d5632afd8a81b8917eac070 should fix this at the basic level.


Limit permissions in server side

Hi guys.
Is there any possibility to limit a room stream to allow only audio streams and drop video streams(in server)?

please use discourse to ask questions to the community: http://discourse.lynckia.com/


Retrieving "Favorites" Pages

Hi, thanks for the great work with this library. Is there a way to get pages which a user has marked as "Favorite". These show up on the top of the navigation bar on the left of the Notion app.

That will be great to have this.

That will be great to have this.


[#6609] Add test

Codecov Report

Merging #7496 (5d7d2b0) into master (b24f9e7) will increase coverage by 0.21%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##           master    #7496      +/-   ##
==========================================
+ Coverage   38.49%   38.70%   +0.21%
==========================================
  Files        3465     3465
  Lines       99464    99471       +7
  Branches    12215    12215
==========================================
+ Hits        38285    38499     +214
+ Misses      58210    57991     -219
- Partials     2969     2981      +12

Impacted Files
Coverage Δ

...r/context/storage/AsyncQueueingUriStatStorage.java
73.52% <85.71%> (+73.52%)
:arrow_up:

...collector/handler/grpc/GrpcAgentStatHandlerV2.java
40.00% <100.00%> (+40.00%)
:arrow_up:

...corp/pinpoint/rpc/stream/StreamChannelManager.java
60.91% <0.00%> (-5.75%)
:arrow_down:

...m/navercorp/pinpoint/rpc/client/ConnectFuture.java
85.00% <0.00%> (-5.00%)
:arrow_down:

...rc/main/java/com/navercorp/pinpoint/test/Item.java
70.00% <0.00%> (-3.34%)
:arrow_down:

...tor/metric/datasource/DefaultDataSourceMetric.java
75.00% <0.00%> (-3.13%)
:arrow_down:

...vercorp/pinpoint/web/vo/ApplicationAgentsList.java
66.66% <0.00%> (-2.57%)
:arrow_down:

...java/com/navercorp/pinpoint/rpc/DefaultFuture.java
72.50% <0.00%> (-1.67%)
:arrow_down:

...pinpoint/profiler/monitor/DeadlockMonitorTask.java
19.76% <0.00%> (+1.16%)
:arrow_up:

...pinpoint/common/server/bo/stat/AgentUriStatBo.java
49.05% <0.00%> (+3.77%)
:arrow_up:

... and 14 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 b24f9e7...5d7d2b0. Read the comment docs.


Suggestion for a Meteor package

Recently Meteor community are trying to integrate React, React Native with Meteor.js.
I think Arch is a good pattern for frontend, with Meteor as backend it would be great to develop apps or websites and easily to deploy on any platform. If there is an Arch package, we will see Arch running on any devices very soon.
Best wishes for LiveScript community growing.
Thank you very much!
https://github.com/reactjs/react-meteor
https://github.com/grovelabs/meteor-react
https://github.com/hharnisc/react-native-meteor-websocket-polyfill
https://atmospherejs.com/i/publishing

Thanks for this idea! I unfortunately don't know much about Meteor - can you use it solely as a sort of a "push streaming API"? Integrating in that case should be fairly straightforward. Otherwise I'm not sure...
Arch is fairly heavily relying on having the server handler for isomorphic rendering and we didn't really do much work on cleanly separating that to enable a client-side use with something just serving the application js bundle (obviously losing the isomorphic aspect of Arch).
@asavin You looked at Meteor a bit closer, any comments/ideas?

@wuxianliang Hi and thanks for idea! As far as I know you can use npm packages in Meteor apps too? That might be a quick solution for now.
What you will loose with Meteor is isomorphism and pure server-side rendering - but other than that it sounds like a good pattern for an app with multiple clients, constant data sync. I can easily imagine Arch being responsible for UI and being fed with data by Meteor.
Haven't checked with Atmosphere for a while, but I can't imagine it's a huge effort to make a package.
Thanks for warm words about LiveScript!

Thank you very much. I will have a tray this weekend.


Revert "[NEW] Wear Module"

Reverts RocketChat/Rocket.Chat.Android#1280

@filipedelimabrito develop is now blocked for us to sent the 2.3.0, we'll need to wait before we ship this one.


Fix doc redirects

Adds a new set of redirects so links like "docs/foo" go to
"docs/vLatest/foo".
Fixes #1232
Signed-off-by: John Schnake<EMAIL_ADDRESS>

Seems to be working great, I don't see any conflicts here
LGTM!