40 add monte carlo pricer lecture 3

@KYLChiu, please send help 🆘
As you can see from CI, there are a few tests that a failing at the moment. Most crucially, the test L4, VanillaCallPricer in here is failing.
In line 324 of lecture4_test.cpp , I can see that the seed of the random number generator is not incrementing...

@KYLChiu, please send help 🆘
As you can see from CI, there are a few tests that are failing at the moment. Most crucially, the test L4, VanillaCallPricer in here is failing.
In line 324 of lecture4_test.cpp , I can see that the seed of the random number generator is not incrementing...
I have a separate test (line 12 of lecture4_test.cpp) that checks that the RNG on its own is working ok, i.e. able to increment the seed and throw out different random number each time the method Gauss() is being called.

all fixed! (ish...commented out some failing tests lol)


fix: process parallel events and potential deadlock in on_record

Motivation
#94 - Mutli-threaded tracing drops most of the events
Solution
I have reorganized the code so that the lock on extensions is held, but not while record is called which should solve the deadlock originally solved in #59
Benchmark
I've used the benchmark from #93, please ignore the "filtered" vs "non_filtered" distinction, for the purpose of this PR they should be identical cases.
See results in later comment as the code got changed.

I've dug into it a bit more and found out #59 fixed the deadlock only in on_event and it was actually present in on_record the whole time. I've added a test for this but since the fix for both overlaps a lot, I've squashed it into a single commit where a different structure from SpanBuilder is used in the visitor and the changes recorded there are then merged into the builder itself.
Benchmark against current HEAD
The results here seem hopeful that this is actually even improvement. But again, first time really using criterion so if you double-checked it, it would be great.
     Running benches/trace.rs (target/release/deps/trace-811e6591eded5464)
Gnuplot not found, using plotters backend
otel_many_children/spec_baseline
                        time:   [22.606 µs 22.610 µs 22.614 µs]
                        change: [-0.8576% -0.8330% -0.8084%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 2 outliers among 100 measurements (2.00%)
  2 (2.00%) high severe
otel_many_children/no_data_baseline
                        time:   [50.365 µs 50.379 µs 50.394 µs]
                        change: [+1.1564% +1.2176% +1.2804%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 7 outliers among 100 measurements (7.00%)
  7 (7.00%) high mild
otel_many_children/data_only_baseline
                        time:   [58.989 µs 59.005 µs 59.021 µs]
                        change: [-3.9068% -3.8545% -3.8043%] (p = 0.00 < 0.05)
                        Performance has improved.
otel_many_children/full time:   [114.33 µs 114.34 µs 114.35 µs]
                        change: [-1.3542% -1.3192% -1.2730%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 9 outliers among 100 measurements (9.00%)
  3 (3.00%) low mild
  4 (4.00%) high mild
  2 (2.00%) high severe

otel_many_events/spec_baseline
                        time:   [42.680 µs 42.713 µs 42.749 µs]
                        change: [+0.2011% +0.3659% +0.5146%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 3 outliers among 100 measurements (3.00%)
  3 (3.00%) high mild
otel_many_events/no_data_baseline
                        time:   [104.22 µs 104.25 µs 104.27 µs]
                        change: [+1.4316% +1.4605% +1.4944%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 6 outliers among 100 measurements (6.00%)
  1 (1.00%) low severe
  3 (3.00%) high mild
  2 (2.00%) high severe
otel_many_events/data_only_baseline
                        time:   [145.88 µs 145.92 µs 145.96 µs]
                        change: [-1.4142% -1.3765% -1.3420%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 4 outliers among 100 measurements (4.00%)
  4 (4.00%) high mild
otel_many_events/full_filtered
                        time:   [457.29 µs 457.37 µs 457.46 µs]
                        change: [-15.471% -15.440% -15.409%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 6 outliers among 100 measurements (6.00%)
  5 (5.00%) high mild
  1 (1.00%) high severe
otel_many_events/full_not_filtered
                        time:   [465.70 µs 465.76 µs 465.82 µs]
                        change: [-16.094% -16.059% -16.028%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 4 outliers among 100 measurements (4.00%)
  1 (1.00%) high mild
  3 (3.00%) high severe

Sorry for the late change to this.

@jtescher do you think you'll have some time to take a look at this? I'd like to try and tackle the span limits next but I think it should be built upon this so I'd like to have even a soft go-ahead on this first.


Unable to link Entra Tenant with MSA account after enrolling for Edge Developers Program

Unable to link Entra Tenant with MSA account after enrolling for Edge Extensions Developers Program and getting below error while linking:
An error happened while processing your request. Please try again.
Something went wrong. Please try again, if this problem persists please contact Microsoft support.
Request ID: 71804c2b-0f4b-3c97-eee9-96cdcb32534a
Correlation ID: 9017955d-f9a7-4c50-948d-e92b9fb18562
After this error Tenant shows under Linked Tenants however on the Linked Tenant we are not able to manage Edge extensions. Unable to assign user's to it. Accounts we are using are already Global Admins. We also added MSA account as Global admin on the Entra Tenant still same. We need urgent assistance for this.

Hi @ParanvirSingh1, could you please help us with the screenshot of the error along with the seller ID so that we can assist you further.

Hello @ManikanthMSFT  could you please share an update on this. We are stuck with the deployment.

Hi @ParanvirSingh1, we are looking into this and will let you know as soon as we have an update.

@ManikanthMSFT
Seller ID: 89401550
For now, we are trying to provide Edge Developer's program Access to our Developers and other team members so do not have edge extensions details.

Hey @ParanvirSingh1, could you please try the following steps to resolve the problem:

Clear your browser cache.
Retry linking- Unlink and relink the tenant in the Partner Center.

If the issue persists after trying these steps, please let us know, and we will be happy to assist you further.


gh-2560 GetTraitsHandler is initialised with store traits

Related Issue

Resolve #2560

Codecov Report

Merging #2577 (227c44d) into develop (4ccb03f) will increase coverage by 6.76%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #2577      +/-   ##
=============================================
+ Coverage      66.69%   73.46%   +6.76%
+ Complexity      2522      228    -2294
=============================================
  Files            995       19     -976
  Lines          32457      731   -31726
  Branches        3936       57    -3879
=============================================
- Hits           21647      537   -21110
+ Misses          9114      145    -8969
+ Partials        1696       49    -1647

Impacted Files
Coverage Δ

.../src/main/java/uk/gov/gchq/gaffer/store/Store.java

...ffer/store/operation/handler/GetTraitsHandler.java

.../gov/gchq/gaffer/store/schema/SchemaOptimiser.java

...n/java/uk/gov/gchq/gaffer/rest/SystemProperty.java

...k/gov/gchq/gaffer/accumulostore/AccumuloStore.java

...gov/gchq/gaffer/federatedstore/FederatedStore.java

...java/uk/gov/gchq/gaffer/hbasestore/HBaseStore.java

...ain/java/uk/gov/gchq/gaffer/mapstore/MapStore.java

.../uk/gov/gchq/gaffer/parquetstore/ParquetStore.java

...java/uk/gov/gchq/gaffer/proxystore/ProxyStore.java

... and 966 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 f54e069...227c44d. Read the comment docs.

Closed as this needs to be merged into https://github.com/gchq/Gaffer/pull/2564


Internal error involving quotation of struct tuple

The following quotation (involving struct tuple)
let actual = struct (0,0)
let code =
    <@ match actual with
       | struct (0,0) -> true
       | _ -> false @>

Produces the following error

Error FS0073 internal error: Unexpected expression shape

Meanwhile, the following quotation (involving struct DU)
let actual: Result<string, string> = Ok "foo"
printfn "%A"
    <@ match actual with
       | Ok _ -> true
       | Error _ -> false @>

Produces the following output

IfThenElse (UnionCaseTest (AddressOf (ValueWithName (Ok "foo", actual)), Error),
Value (false), Value (true))

So it compiles without error, but I am unsure if AddressOf makes sense here.
Visual F# 4.1

WIP see https://github.com/Microsoft/visualfsharp/pull/3125


#[model] macro does not work in RustRover

Anything defined in the #[model] macro does not resolve and prevents code completion from working on basically everything in iroha_data_model. Commenting out the #[model] macro call makes everything defined there immediately available.
It would be nice to debug why this fails and make sure it either gets fixed in the intellij rust extension or worked around in our macro (for example by detecting the ij-rust invocation, see https://github.com/intellij-rust/intellij-rust/pull/9711, and emitting simpler code)
When trying to manually expand the #[model] macro I get the following error message: "Failed to expand macro: cannot find ModData for mod containing the macro call (internal error)". I am also getting a lot of errors stating that the "Module model is private" (not sure why it thinks that it's private)

While this macro does seem to yield some trouble with my IDE, the effects seems to be transient =(
It disappeared after yet another cache invalidation and I can't seem to reproduce it anymore


how to install graphviz on Laravel Sail

i want to use your pacakge in a Laravel Sail environment.
How do i install graphviz on Laravel Sail?
thanks,
Noud

sail composer require beyondcode/laravel-er-diagram-generator --dev
sail root-shell
apt-get update
apt-get install graphviz
php artisan generate:erd

This worked for me.

On Fri, 2024-05-03 at 07:37 -0700, Romek wrote:

sail composer require beyondcode/laravel-er-diagram-generator --dev

i use Laravel 11,
could it be that it is a too new Laravel version 11
to work with beyondcode/laravel-er-diagram-generator?
if i do, i get:
./vendor/bin/sail composer require beyondcode/laravel-er-diagram-
generator --dev
./composer.json has been updated
Running composer update beyondcode/laravel-er-diagram-generator
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of
packages.
Problem 1

doctrine/dbal[v2.5.0, ..., v2.5.2] require doctrine/common

=2.4,<2.6-dev -> satisfiable by doctrine/common[v2.4.0, ..., v2.5.3].

doctrine/dbal[v2.5.3, ..., v2.5.5] require doctrine/common

=2.4,<2.7-dev -> satisfiable by doctrine/common[v2.4.0, ..., v2.6.2].

doctrine/dbal[v2.5.6, ..., v2.5.13] require doctrine/common

=2.4,<2.8-dev -> satisfiable by doctrine/common[v2.4.0, ..., v2.7.3].

doctrine/dbal[v2.4.0, ..., v2.4.5] require doctrine/common ~2.4 ->
satisfiable by doctrine/common[v2.4.0, ..., 2.13.3].
laravel/framework v11.3.1 requires nesbot/carbon ^2.72.2|^3.0 ->
satisfiable by nesbot/carbon[3.2.4].
laravel/framework v11.3.1 requires symfony/http-foundation ^7.0 ->
satisfiable by symfony/http-foundation[v7.0.6].
beyondcode/laravel-er-diagram-generator[0.1.0, ..., 0.2.3, 1.0.0,
..., 1.5.0] require php ^7.1 -> your php version (8.3.4) does not
satisfy that requirement.
doctrine/common[v2.9.0, ..., 2.13.0] require php ^7.1 -> your php
version (8.3.4) does not satisfy that requirement.
doctrine/dbal[v2.6.0, ..., v2.9.3] require php ^7.1 -> your php
version (8.3.4) does not satisfy that requirement.
beyondcode/laravel-er-diagram-generator 2.0.0 requires nikic/php-
parser ^2.0|^3.0|^4.0 -> found nikic/php-parser[v2.0.0, v2.0.1, v2.1.0,
v2.1.1, v3.0.0, ..., v3.1.5, v4.0.0, ..., v4.19.1] but the package is
fixed to v5.0.2 (lock file version) by a partial update and that
version does not match. Make sure you list it as an argument for the
update command.
doctrine/common[v2.4.0, ..., v2.5.3] require doctrine/inflector 1.* -

found doctrine/inflector[v1.0, ..., 1.4.4] but the package is fixed
to 2.0.10 (lock file version) by a partial update and that version does
not match. Make sure you list it as an argument for the update command.

doctrine/common[v2.6.0, ..., v2.6.2] require php ~5.5|~7.0 -> your
php version (8.3.4) does not satisfy that requirement.
doctrine/common[v2.7.0, ..., v2.7.3] require php ~5.6|~7.0 -> your
php version (8.3.4) does not satisfy that requirement.
doctrine/common[v2.8.0, ..., v2.8.1] require php ~7.1 -> your php
version (8.3.4) does not satisfy that requirement.
doctrine/common[2.13.1, ..., 2.13.3] require doctrine/inflector ^1.0
-> found doctrine/inflector[v1.0, ..., 1.4.4] but the package is fixed
to 2.0.10 (lock file version) by a partial update and that version does
not match. Make sure you list it as an argument for the update command.
doctrine/dbal[v2.10.0, ..., 2.10.4] require php ^7.2 -> your php
version (8.3.4) does not satisfy that requirement.
doctrine/dbal[2.11.0, ..., 2.11.3] require php ^7.3 -> your php
version (8.3.4) does not satisfy that requirement.
laravel/framework is locked to version v11.3.1 and an update of this
package was not requested.
carbonphp/carbon-doctrine-types 3.2.0 conflicts with doctrine/dbal
3.8.4.
beyondcode/laravel-er-diagram-generator 3.0.0 requires doctrine/dbal
~2.3|^3.3 -> satisfiable by doctrine/dbal[2.3.0, ..., 2.13.9, 3.3.0,
..., 3.8.4].
nesbot/carbon 3.2.4 requires carbonphp/carbon-doctrine-types * ->
satisfiable by carbonphp/carbon-doctrine-types[3.2.0].
symfony/http-foundation v7.0.6 conflicts with doctrine/dbal 3.5.4.
symfony/http-foundation v7.0.6 conflicts with doctrine/dbal 3.4.6.
symfony/http-foundation v7.0.6 conflicts with doctrine/dbal 3.3.8.
symfony/http-foundation v7.0.6 conflicts with doctrine/dbal 3.3.5.
symfony/http-foundation v7.0.6 conflicts with doctrine/dbal 2.13.9.
symfony/http-foundation v7.0.6 conflicts with doctrine/dbal 2.13.8.
symfony/http-foundation v7.0.6 conflicts with doctrine/dbal 2.13.1.
symfony/http-foundation v7.0.6 conflicts with doctrine/dbal 2.12.1.
symfony/http-foundation v7.0.6 conflicts with doctrine/dbal 2.3.5.
symfony/http-foundation v7.0.6 conflicts with doctrine/dbal 2.3.2.
Root composer.json requires beyondcode/laravel-er-diagram-generator *
-> satisfiable by beyondcode/laravel-er-diagram-generator[0.1.0, ...,
0.2.3, 1.0.0, ..., 1.5.0, 2.0.0, 3.0.0].

Use the option --with-all-dependencies (-W) to allow upgrades,
downgrades and removals for packages currently locked to specific
versions.
You can also try re-running composer require with an explicit version
constraint, e.g. "composer require beyondcode/laravel-er-diagram-
generator:*" to figure out if any version is installable, or "composer
require beyondcode/laravel-er-diagram-generator:^2.1" if you know which
you need.
Installation failed, reverting ./composer.json and ./composer.lock to
their original content.
$

@noud It seems you're having the same issue as people in this issue: #112

Romek,
yes, same issue, Laravel is too new.
Can you make a new distribution with these patches to composer.json
-        "php": "^7.1|^8.0",
+        "php": "^7.1|^8.0|^8.1|^8.2",

-        "doctrine/dbal": "~2.3|^3.3",
+        "doctrine/dbal": "~2.3|^3.3|^3.8|^4.0",

-        "phpunit/phpunit": "^7.0| ^8.0|^9.5.10",
+        "phpunit/phpunit": "^7.0| ^8.0|^9.5.10|^10.5.20|^11.1.3",

and make a new distribution?
thanks,
Noud


old Climb model

@mayork, this is an old climb model by @pgkirsch. I'm hoping with this PR we can get the best of it and your climb model. Note this is from back when we had setup() methods so it needs some cleanup before merging.
A long term goal I'd like to move towards is shared models for some key aircraft things like MIssionSegment, Climb, Cruise, BreguetRange, etc. E.g. we currently have three climb models:

this one
your D8 one
@mjburton11's gas male one

It would be ideal if they were all imported from the same place.

taking a look right now


Fix(addLayers): unspiderfy before adding new Marker(s)

Hi again,
Fix https://github.com/Leaflet/Leaflet.markercluster/issues/879
Add the call to _unsiperfy within the process subtask of addLayers, so that it works even if for some reason a cluster has been spiderfied while addLayers is in the middle of processing.
As previously, I will push this PR in 2 steps (test, then fix).

Thanks again!


Add InstanceStatus.Phase.

Also add phases for user and restore in phase.go

Thanks for the PR, Yufeng! :-)
Can you run make -C oracle check and make -C oracle unit-test and include any new changes in the commit? See https://github.com/GoogleCloudPlatform/elcarro-oracle-operator/blob/main/docs/contributing.md#testing.

LGTM. Thanks Yufeng!

/retest


Fix physical function name in documentation and tests

cc @egernst

@rkamudhan Can you take a look at this?

@amshinde Can we have internal sync up before creating any pull request ?


Promote env var usage instead of JSON file for auth

Currently, in various places of the gcloud cloud storage documentation, following code is recommended to initialise the storage:
//Create a client with explicit credentials

const storage = new Storage({
  projectId: 'your-project-id',
  keyFilename: '/path/to/keyfile.json'
});

See: https://googleapis.dev/nodejs/storage/latest/Storage.html#Storage-examples
Making code rely on credentials stored in files is a bad practice from a security point of view, requires extra steps in kubernetes like mounting drives in runtime, also if builders/transpilers are used, the file needs to be copied around for the path to match.
From looking at the code, I got a solution working by providing the service account credentials directly, which I would kindly ask to promote in the docs instead:
Example:
 storage = new Storage({
    projectId: process.env.GCLOUD_PROJECT_ID,
    credentials: {
      client_email: process.env.GCLOUD_STORAGE_SERVICE_ACCOUNT_CLIENT_EMAIL,
      private_key: process.env.GCLOUD_STORAGE_SERVICE_ACCOUNT_PRIVATE_KEY
    }
  });

Environment variables are easier and safer to handle and exist in nearly every project, so it's trivial to put the service account details there simply, instead of handling that additional JSON file that could potentially leak.
Cheers

I agree that using a file could be suboptimal, but if I understand correctly we do not recommend the use of env variables for this purpose either. We instead suggest the use of a secure keystore such as Cloud Key Management. I think that might be too complex to integrate with our samples, so we should just aim for consistency with other Cloud samples; for now I'll assign to a language owner for a second opinion.

Hi @nikita-fuchs ,
Our samples aim to show the simplest way to perform an operation. In order to simplify the authentication, piece we show JSON file example. Different authentication strategies are appropriate in different scenarios. We have full documentation on authenticating with Cloud Storage here.

I 100% agree with @nikita-fuchs and think the use of a JSON file is something that will inevitably lead to security breaches because it's an anti-pattern for 99% of secrets in Node.js and most projects are not configured to ignore json files.
I also disagree that using a json file is "the simplest way to perform an operation"; I'm here because I thought it was a mistake that the README doesn't mention how to auth with environment variables...


Add Bosnia and Herzegovina holidays.

Resolves #683

Coverage increased (+0.003%) to 99.605% when pulling 0446d93754f52145c79ec3d3e21264ae0e8ae6e9 on kasya:bos_and_herz into be4cd58c46ed19263e985b0bf473488d2ec65b19 on dr-prodigy:beta.

Hi @kasya + @arkid15r thank you very much, great contribution.. Merged in beta.
Cheers


Sensitive info when failure on attaching a deleted/recovered volume

VMware, master
WHen a volume is deleted, then recovered - there is an issue attaching a volume (@weizhouapache didn't you work on a PR for allowing delete/expunge vs. just deletting a volume? perhaps you can advise here?) - sensitive info is sent by the backend and displayed in UI:

The simple fix might be to remove the vSphere volume location from the message - just keep the first sentence of the error response, but ensure that the full info is logged in debug/elsewhere in the mgmt log.

cc @DaanHoogland - FYI

cc @Spaceman1984

@andrijapanicsb why is the path sensitive? to my knowledge authorisation of the user had already been done when this error occurs? is the user not allowed to know where their disks reside?

@DaanHoogland I'm sure you are not saying that an average joe ("user" role) is allowed to learn your internal IPs (and internal VM name?)

ah, the ip i missed, sure . just don't see harm in the internal name


HTML code removed when scrolling down

Hi, I'm in trouble with Nanoscroller.
I've installed it on my project and it seems to work normally but, when I scroll down, the whole html code inside the .nano class is litterally removed!
Is that a bug, or did I miss something?

Well hard to say without seeing your code. Have you followed the instructions in README and had a look at how the demo for nanoScroller is set up?

Closing as inactive, let's reopen if something new comes up.


Dockerfile.centos8.dapper: use AlmaLinux8 instead of CentOS8

As there is no updated CentOS8 image that is built on CentOS Stream and the repositories for non-stream have been archived, it would need workarounds to get CentOS8 to run properly.
See https://github.com/rancher/rancher-selinux/pull/10/commits/fb35171078397b7dc5e67cf02a0ee4a99d51c31e and https://github.com/rancher/rancher-selinux/pull/10/commits/8fe92aa1f0c856fda78451260ac8ba1c35daaef2 for example.
Rather than hacking workarounds, use the AlmaLinux8 image which promises to be bit-compatible with RHEL8.

Superseded by https://github.com/rancher/rancher-selinux/pull/19.


XXXXXX.worker.js 404

I've an app mounted in Vue + Cordova. Vue pdf works in browser, but doesnt't work when i run my app on real device. I've an error 404 on XXXXXX.worker.js.
Any idea?

path not found

How to fix it？


[BUG] removal of old-style buffer special methods breaks pyarrow on py313

Describe the bug
When trying to cythonize pyarrow with cython and cpython default branches it fails with:

      Error compiling Cython file:
      ------------------------------------------------------------
      ...
              buffer.obj = self
              buffer.shape = self.shape
              buffer.strides = self.strides
              buffer.suboffsets = NULL

          def __getsegcount__(self, Py_ssize_t *len_out):
                                    ^
      ------------------------------------------------------------

      pyarrow/io.pxi:1449:30: Cannot convert Python object argument to type 'Py_ssize_t *'

      Error compiling Cython file:
      ------------------------------------------------------------
      ...
          def __getsegcount__(self, Py_ssize_t *len_out):
              if len_out != NULL:
                  len_out[0] = <Py_ssize_t>self.size
              return 1

          def __getreadbuffer__(self, Py_ssize_t idx, void **p):
                                                      ^
      ------------------------------------------------------------

      pyarrow/io.pxi:1454:48: Cannot convert Python object argument to type 'void **'

      Error compiling Cython file:
      ------------------------------------------------------------
      ...
                  raise SystemError("accessing nonexistent buffer segment")
              if p != NULL:
                  p[0] = <void*> self.buffer.get().data()
              return self.size

          def __getwritebuffer__(self, Py_ssize_t idx, void **p):
                                                       ^
      ------------------------------------------------------------

      pyarrow/io.pxi:1461:49: Cannot convert Python object argument to type 'void **'

I was able to bisect it to 9a22b49cb8573c5085994d8c9c5da2de6b2b577d being the first bad commit which looks like it removes these methods from  self.PyBufferProcs.
Oddly, it does not fail with py312 and I do not have a good theory of why.
Code to reproduce the behaviour:
https://github.com/apache/arrow/blob/22f88fa4a8f5ac7250f1845aace5a78d20006ef2/python/pyarrow/io.pxi#L1449-L1468
# example code
    def __getsegcount__(self, Py_ssize_t *len_out):
        if len_out != NULL:
            len_out[0] = <Py_ssize_t>self.size
        return 1

    def __getreadbuffer__(self, Py_ssize_t idx, void **p):
        if idx != 0:
            raise SystemError("accessing nonexistent buffer segment")
        if p != NULL:
            p[0] = <void*> self.buffer.get().data()
        return self.size

    def __getwritebuffer__(self, Py_ssize_t idx, void **p):
        if not self.buffer.get().is_mutable():
            raise SystemError("trying to write an immutable buffer")
        if idx != 0:
            raise SystemError("accessing nonexistent buffer segment")
        if p != NULL:
            p[0] = <void*> self.buffer.get().data()
        return self.size

Expected behaviour
Compilation should not fail.
If these are going to be removed, then I think it needs to be better advertised and removed from https://cython.readthedocs.io/en/latest/src/userguide/special_methods.html#buffer-interface-legacy-no-python-equivalents-see-note-1
OS
linux
Python version
main branch
Cython version
development branches
Additional context
From searching code I have checked out on my machine I think this may also hit mpi4py.

I can definitely update the docs not to mention them.
I can try to make it a warning so that code doesn't immediately break - I doubt that's too hard.
They are useless though unless you're supporting Python 2 (which I don't think pyarrow is?). So whatever Cython does, it might be a good opportunity to remove them.


Formal and informal

I would like to choose when to use formal or informal tone in those languages supporting those tones. Now it mixes those styles in the translation.
I know this is a feautre on DeepL Pro, but I would like to know if it can be done with DeepLX somehow.
Thank you.

Unfortunately, it seems that this is not possible. This is because the project uses the same API as the DeepL web side and the DeepL iOS/Android client. As far as I know, this is not possible on the client side or the web side. @adocampo

Ok, good to know.
Thank you anyway.


Tribe nodes not getting pass values in elasticsearch.yml config file

This is because Elastic is restricting what is being passed through.  ES v2.2.0
https://github.com/elastic/elasticsearch/pull/16893
From debugging, I see that nothing from "searchguard.*" properties is making it through when setting up tribes.   This is needed of course as the clusters the tribe is talking to is using transport over SSL.
Ugh.  My patching didn't help.   Need to somehow get the properties from elasticsearch config file through tribe blocks.
For now, my workaround is to have a static init that give all the defaults I needed from the config.  Of course that meant re-compiling new jar each time it changes :-(

see https://github.com/floragunncom/search-guard-ssl/blob/216bb6190bcd714adc2caaf06c0fe7438826f63e/src/main/java/com/floragunn/searchguard/ssl/SearchGuardSSLModule.java and https://github.com/floragunncom/search-guard-ssl/blob/master/src/main/java/com/floragunn/searchguard/ssl/SearchGuardSSLSettingsFilter.java
All SG properties will be filtered for security reasons

The filter does not seem to work.   I compiled from HEAD last night and in debug mode, I can see that nothing for searchguard.* gets through when running in tribe mode.
Probably a bug on ES filtering side.

When nothing comes through the filter DID work

I guess I am confused.   None of the searchguard.* properties get passed through to sg plugin, and that is working as designed?
How so?

sorry got you wrong - i mean when you call /_nodes/settings via http/rest settings are filtered out. Installing SG on a tribe node is currently untested, so would be great to figure out whats wrong here.

I think it's a bug on ES side.   In tribe mode, they are not passing through most properties in ES config file.   Unfortunately, it's buggy at moment.   They are passing through things that should not be, and blocking others that should be.
The long term fix for SG is to add to the init function a check in config dir for SG's own config file.  E.g. check in $CONFIG_DIR for search-guard-ssl.yml, and if it is there read it (values there override everything else).   Users still have option of putting SG properties in ES config.

will investigate

Figured it out!
If I copied searchguard.* properties and put the same under each tribe downstream cluster, then they get passed through.... to tribe (wtf!).   Anyway, doing that allow SG to work.
E.g.
tribe.downstream1.searchguard.ssl.etc
Will pass through searchguard.ssl.etc

Closing as it is not an SG bug, but ES one.   I'll document how to setup ES tribe with SG on my blog.

thx, can you pls post the link here so we can refer to it (and include them in the docs)?

OK.   I've posted blog on how to get SG-SSL working with ES 2.x and with security policy manager turn on.
http://blog.tinle.org/?p=505
I'll post how to use tribe with SG-SSL as a followup.   I am waiting for time to test ES 2.2.1 as that supposedly fixed the tribe node bug in ES.   If that is the case, it will change my how-to post, so I want to test first.

thats great, thx a lot


[Nominations Open] Top CDF Ambassador Award 2022 🏆

This issue is to receive nominations for the Top CDF Ambassador Award 2022.
This award recognizes a champion for the continuous delivery space. This individual helps spread awareness of the CDF and its projects. The CDF Ambassador leverages multiple platforms, both online as well as speaking engagements, driving interest and excitement around the ecosystem.
To nominate someone, reply to this issue with the following:

Full name of the person you’re nominating (they have to be a CDF Ambassador, view lists of current and previous Ambassadors.
Short description of their contributions to the Continuous Delivery Foundation and why they should win.

Nomination Deadline: Tuesday, March 29, 2022 [EXTENDED to April 11, 2022]
More details are available here: https://cd.foundation/cdf-community-awards/

Garima Bajpai has served the CDF as an Ambassador since the program was started. She has been an eager and willing participant in the program. It was no surprise to me when she was elected to take over as the Ambassador Chair. This was a hard job to take on. With over 200 Ambassadors, some of who had never participated, she knew she had to revamp the program. A brave undertaking. She now has around 40 Ambassadors that participate in 3 different working groups. From this organization, Garima can begin building an Ambassador program that makes an impact. Garima, above all others, deserves this recognition.

Tracy Miranda has been involved in CDF since the early days and even served as CDF GM before moving to a new role: https://tracymiranda.com/2022/01/19/im-joining-chainguard/
She continues to be an advocate for CDF and improving security in the software supply chain.

Fatih Degirmenci has done a great job in helping with the https://github.com/cdfoundation/sig-interoperability bringing together different people and resulted in the CDEvents project https://github.com/cdfoundation/sig-interoperability/tree/master/workstreams/archived/events_in_cicd

cc @tracymiranda + @fdegir :)

Nomination: @salaboy Mauricio Salatino
He has been an ambassador and wonderful advocate since the early days. He's even writing a book about Continuous Delivery + Kubernetes!

🚫 Nominations are now closed!
Voting will start April 13, link will be posted here.


Add CCO2 holdings (NOT RETIREMENTS) to polygon-digital-carbon

Still need to add retirement handling logic for CCO2 but this will at least unblock

closing


test(extension): smoke lw-2440 quickfix

Checklist

[ ] JIRA - <link>
[ ] Proper tests implemented
[ ] Screenshots added.

Proposed solution
Explain how does this PR solves the problem stated in JIRA ticket.
You can also enumerate different alternatives considered while approaching this task.
Testing
Describe here, how the new implementation can be tested.
Provide link or briefly describe User Acceptance Criteria/Tests that need to be met
Screenshots
Attach screenshots here if implementation involves some UI changes

Changes unknown
when pulling b6f8c22c62a20b0111be35cfcc1a667c1964b921 on e2e-smoke-quickfix
into ** on main**.


[QUESTION] Player.Task.RunTo(somePosition);

Is there anyway to make the player run to the way-point on the map. So far out of everything I've tried, I'm starting to think this is just not possible to do.

It only works if the player is within a certain range of the destination. So is there a way I can load paths that lead to the way-point??

I'm not sure about purely using the API here, but if you're willing to use function hash code,
TASK_FOLLOW_NAV_MESH_TO_COORD is available.
It'd be EG Function.Call(Hash.TASK_FOLLOW_NAV_MESH_TO_COORD, PED.Handle, X, Y, Z, SPEED, TIMEOUT, STOPRANGE, PERSIST, 0);

@mcmonkey4eva Thanks I'll give it a try. But I don't think this will work because I'm trying to get the player to go to a position, and not a ped.

@mcmonkey4eva Still no success :(
If there was anyway to load a navmesh that would help me so much. (There is a native for it but I have to pass in the coordinates of a square and I'm not sure how I would do that).

Closing for now, since this is no direct bug report or enhancement idea.


Update aws-sources.md

There is also an issue on lines 80 to 82, it is not formatting properly on the page itself.  I'm not sure why it is behaving this way.  It should have the green window background and "INFO" but it does not render that way on the help page itself. It should look like how the text on lines 99 - 101 renders. On the help page itself it displays as plain text, ":::important Sumo Logic highly recommends using an SNS Subscription Endpoint for its ability to maintain low-latency collection. This is essential to support up-to-date Alerts. :::"
Purpose of this pull request
This pull request (PR) ...
Issue number:

Select the type of change:

[x] Minor Changes - Typos, formatting, slight revisions
[ ] Update Content - Revisions and updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

The article references a Step "B" but as far as I can tell there is no Step B. I think it's referencing Step 1.

This is an image of the unformatted plain text on the doc I mentioned above. Not sure it's supposed to display that way.

This is an image of the unformatted plain text on the doc I mentioned above. Not sure it's supposed to display that way.

Yes you're correct in that it's broken. I assigned this ticket to content owner @JainM6 Monday and will follow up with her again today.

This is an image of the unformatted plain text on the doc I mentioned above. Not sure it's supposed to display that way.

Yes you're correct in that it's broken. I assigned this ticket to content owner @JainM6 Monday and will follow up with her again today.
@kimsauce I think we need a correction here. I am not the owner of this doc, this was done before I joined the organization.

This is an image of the unformatted plain text on the doc I mentioned above. Not sure it's supposed to display that way.

Yes you're correct in that it's broken. I assigned this ticket to content owner @JainM6 Monday and will follow up with her again today.
@kimsauce I think we need a correction here. I am not the owner of this doc, this was done before I joined the organization.

This is an image of the unformatted plain text on the doc I mentioned above. Not sure it's supposed to display that way.

Yes you're correct in that it's broken. I assigned this ticket to content owner @JainM6 Monday and will follow up with her again today.
@kimsauce I think we need a correction here. I am not the owner of this doc, this was done before I joined the organization.

@JainM6 , to clarify, each docs writer is responsible for different topics. I'm responsible for observability/traces/OTEL, etc., Marie is CSE/metrics, Nishant does Apps, and /send-data docs (in this PR) falls under your purview. The edits I make are almost all from before I joined the org :) cc @stacykor

This is an image of the unformatted plain text on the doc I mentioned above. Not sure it's supposed to display that way.

Yes you're correct in that it's broken. I assigned this ticket to content owner @JainM6 Monday and will follow up with her again today.
@kimsauce I think we need a correction here. I am not the owner of this doc, this was done before I joined the organization.

@JainM6 , to clarify, each docs writer is responsible for different topics. I'm responsible for observability/traces/OTEL, etc., Marie is CSE/metrics, Nishant does Apps, and /send-data docs (in this PR) falls under your purview. The edits I make are almost all from before I joined the org :) cc @stacykor

@kimsauce I think you have completely misunderstood what I said. You mentioned "I am the content owner of this doc", that's not correct, I did not create this doc. If I were in your place, I'd have simply forwarded this request and informed the requester about the new "Area owner."  That's just my approach. Thanks

I think this is still broken. The info window on the page is displaying in code block format. The same for the text below, "To set up the subscription you need to get an endpoint URL from Sumo to provide to AWS. This process will save your Source and begin scanning your S3 bucket when the endpoint URL is generated. Click on Create URL and use the provided endpoint URL when creating your subscription in step C."

I think this is still broken. The info window on the page is displaying in code block format. The same for the text below, "To set up the subscription you need to get an endpoint URL from Sumo to provide to AWS. This process will save your Source and begin scanning your S3 bucket when the endpoint URL is generated. Click on Create URL and use the provided endpoint URL when creating your subscription in step C."

@JainM6 ☝

@kimsauce @bnartiff  Thank you for pointing this out. I have created a new PR #1433


Split Generators tests from Sequence tests

Summary
We can now test the Sequence API on Python3. This PR aims at enabling those tests on Travis.
Related Issues
PR Overview

[ ] This PR requires new unit tests [y/n] (make sure tests are included)
[ ] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[x] This PR is backwards compatible [y/n]
[ ] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Thanks for the pull request!


Changing codec on existing partitions appears to have no effect

CrateDB version
4.8.0
CrateDB setup information
No response
Steps to Reproduce

Create a partitioned table:CREATE TABLE IF NOT EXISTS "tmp"."conditions" (
      "time" TIMESTAMP WITH TIME ZONE NOT NULL,
      "device_id" TEXT,
      "temperature" DOUBLE PRECISION,
      "humidity" DOUBLE PRECISION,
      "partition_key" INTEGER
 )
 PARTITIONED BY ("partition_key");

Insert some data to create a partition (should be enough data to notice the effect of compression later on)
Change the partitions' codec:ALTER TABLE tmp.conditions PARTITION (partition_key = 1) CLOSE;
ALTER TABLE tmp.conditions PARTITION (partition_key = 1) SET (codec = 'best_compression');
ALTER TABLE tmp.conditions PARTITION (partition_key = 1) OPEN;

Verify the codec was changed:SELECT settings['codec']
FROM information_schema.table_partitions
WHERE table_schema = 'tmp'
  AND table_name = 'conditions';

Returns: best_compression
To be on the safe side (not sure if needed), also run OPTIMIZE TABLE tmp.conditions

Observation
After changing codec, the table size doesn't decrease. Creating a copy of the same data with codec = 'best_compression' set during CREATE TABLE shows that the partition size is noticeably smaller if compression is applied successfully. Therefore, it seems as if compression with the workflow above isn't actually changed.
Also see the related community discussion.
Expected Result
Not sure if changing the codec on an existing partition is supposed to work, but I would expect one of the following outcomes:

The partition size does decrease
There is an error message stating that changing codec for existing partitions isn't supported

Actual Result
Compression doesn't change, even after waiting for a few hours.

@hammerhead A changed codec of a table/partition will only take effect after it's segments have been merged. This can be forced by using the OPTIMIZE statement. Have you tried that?

@hammerhead

Changing codec on non-partitioned tables seems not possible, is this correct?

No this looks like a bug, there isn't any reason to forbid that for whole tables. Also as you mentioned, the behaviour is contradicting.


Upgrade react-scripts

for Node v18 compatibility (https://stackoverflow.com/questions/69692842/error-message-error0308010cdigital-envelope-routinesunsupported/71334532#71334532)

Alternatively,:

consider shifting out of CRA

e.g. migrating to vitejs
CRA is not as actively maintained anymore

This would also allow us to remove the following hack:
https://github.com/source-academy/frontend/blob/46ade71e85c5dfcf478f906ed62201667c97aced/package.json#L133-L136
See #2214 for context.

Unfortunately, we cannot upgrade react-scripts (CRA) to v5.0x because another dependency that we use, Create React App Configuration Override (CRACO) does not have a stable version that supports CRA v5.x yet

It seems that this is no longer true with the release of CRACO v7, so we might be able to fully resolve the issue in #2214 as well as update Node.js once we upgrade react-scripts.


Fix TranslogSequencerTests

Description
"test sequencer out of order" Test turned flaky after recent change, this change fixes the test.
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

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

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

runTest introduced in kotlin 1.6, backport accordingly


Generic requirements

Thanks for sharing!
Some questions:

[ ] What is required to use this from scratch?
[ ] Relative and OS agnostic paths?
[ ] Remove unused code?
[ ] Include requirements.txt?
[ ] Example?

I will get all of these added!! Thanks for your interest! I'll try to do that later tonight.


update: add a application.yml file for launchpad deployments

connects to #21

Coverage remained the same at 86.207% when pulling 15298f3e90eed9d8bb90d47070f0ca7d92dc7c14 on lholmquist:21-application-yaml into 073839cc00f50caba5f5d568324b5dd2fb1a186d on bucharest-gold:master.


Unable to start because of Multiple primary key defined.

Unable to start Casdoor.
[xorm] [warn]  2023/06/19 01:11:00.191853 Table organization has column phone_prefix but struct has not related field
[xorm] [warn]  2023/06/19 01:11:00.191885 Table organization has column password_complex_options but struct has not related field
[xorm] [warn]  2023/06/19 01:11:00.194243 Table user has column country_codes but struct has not related field
[xorm] [warn]  2023/06/19 01:11:00.195444 Table group has column id but struct has not related field
[xorm] [warn]  2023/06/19 01:11:00.195451 Table group has column parent_group_id but struct has not related field
panic: Error 1068: Multiple primary key defined

goroutine 1 [running]:
github.com/casdoor/casdoor/object.(*Adapter).createTable(0xc00083d500)
        /srv/casdoor/casdoor/object/adapter.go:150 +0xa85
github.com/casdoor/casdoor/object.CreateTables(0x25?)
        /srv/casdoor/casdoor/object/adapter.go:64 +0x36
main.main()
        /srv/casdoor/casdoor/main.go:43 +0x3e

Go: 1.20.5
Casdoor: v1.342.1

@mzch please delete the DB table: user_group_relation (backup the data if still needed) and restart Casdoor again.

Oh, fixed! Thanks @hsluoyz ! :)


expo is not working in version "@livekit/react-native": "^1.1.2" but works in ^1.0.0

Describe the bug
This library does not work with Expo if the version is ^1.1.2.

I investigated the reason and found it works in following repository and uses version ^1.0.0

https://github.com/qweasd1/livekit-webrtc-expo-sample/tree/main

To Reproduce
Steps to reproduce the behavior:

follow these instruction  https://github.com/livekit/client-sdk-react-native/wiki/Expo-Development-Build-Instructions
run eas build --profile development --local (iOS target)
See error

Expected behavior
@livekit/react-native is available in expo in latest version
Screenshots

Device Info:

Device: iPhone Simulator 14
OS: iOS17 (beta)

Dependencies Info (please reference your package-lock.json or yarn.lock file):

@livekit/react-native: ^1.1.2
react-native-webrtc: ^111.0.1

And this library looks like currently not work with @livekit/react-native-webrtc (expo environment)
got following error
[INSTALL_PODS] [!] The 'Pods-test' target has frameworks with conflicting names: webrtc.xcframework.

@yareyaredesuyo
please add @livekit/react-native-webrtc in your package.json.
you need to 'ios' dir to run pod install, or if you  mac is M1 , the pod install maybe fail, you can download webrtc lib, and move it on pod cache

I'm working on Expo 49 (custom dev client) (iOS + Android) having no issue with it

[INSTALL_PODS] [!] The 'Pods-test' target has frameworks with conflicting names: webrtc.xcframework.
Having the same issue with Expo 48.
Is upgrading to Expo 49 a requirement?

[INSTALL_PODS] [!] The 'Pods-test' target has frameworks with conflicting names: webrtc.xcframework.

Had the same problem and removing react-native-webrtc from my package.json fixed it.

Closing as obsolete. Works on most recent Expo versions now.


reorganised imports for FOV module

Removed default imports for the FOV module in order to enable intellisense on exported classes.
Example valid code after the change:
import { PreciseShadowcasting } from "rot-js/lib/fov/precise-shadowcasting";
const fov: PreciseShadowcasting = new PreciseShadowcasting(lightPassesFunc);

Interesting! I have two comments:

can you explain why and how does this change fix the issue you were encountering? Completely eradicating export default from the whole codebase is not something I welcome with joy. When a source file provides exactly one class, a default export seems pretty straightforward to me.

is there a particular reason for removing file extensions from import statements? This change breaks the native module implementation in browsers that requires file extensions to be present (the result has to resolve to a valid URI, after all).

1. can you explain why and how does this change fix the issue you were encountering? Completely eradicating `export default` from the whole codebase is not something I welcome with joy. When a source file provides exactly one class, a default export seems pretty straightforward to me.

Ah, I missed your post in https://github.com/ondras/rot.js/issues/148#issuecomment-448410672. So this is already explained.

As explained in the issue you linked, eliminating default exports and replacing them with named ones allows the IDE to provide intellisense (code completion) and automatic imports. More on why default exports are bad: https://basarat.gitbooks.io/typescript/docs/tips/defaultIsBad.html and https://blog.neufund.org/why-we-have-banned-default-exports-and-you-should-do-the-same-d51fdc2cf2ad

In Node.js, modules don't care about the file extension. Truth be told, this is the first time I've ever seen them used at all. My IDE automatically adds imports without extensions too. I wasn't aware browsers even implemented ES6 modules, much less that they complained about missing extensions. I will add them back.

BTW, if you decide to phase away from default exports, I can also extend the PR to go through the entire codebase. I only included the FOV module because this is the only module I am using.

BTW, if you decide to phase away from default exports, I can also extend the PR to go through the entire codebase. I only included the FOV module because this is the only module I am using.

Yeah, I will have to think this over. (Also educate myself by reading those articles you linked.)


Fix issue 78 - Bug with backup timestamp for dismounted DB's or stopped IS

This fix for issue #78 resolves:

a bug with the comparison of "hours since last backup" to see if it's greater than 24 (buggy code was comparing a string to an int)
condition where dismounted databases were reported as "Never" for last backup. Now they report as dismounted.
condition where a stopped information store service caused all databases on that server to be reported as "Never" for last backup. Now they report as "could not be reached".

Closes #78


[Bug] one of the created Ray actors die during docid transform

Search before asking

[X] I searched the issues and found no similar issues.

Component
Transforms/universal/doc_id
What happened + What you expected to happen
Could be related #719
full error
(orchestrate pid=5751) created [Actor(RayTransformFileProcessor, 37cbb100b21c9012ce68444101000000), Actor(RayTransformFileProcessor, e063fda8d2ea6f56efc152e201000000)], alive [ActorState(actor_id='37cbb100b21c9012ce68444101000000', class_name='RayTransformFileProcessor', state='ALIVE', job_id='01000000', name='', node_id='08deddebac466e0100f6c1015c266849f52c8ff173595c8170cdee12', pid=5915, ray_namespace='aa965802-2b25-4b0b-83c8-3e939d83ab34', serialized_runtime_env=None, required_resources=None, death_cause=None, is_detached=None, placement_group_id=None, repr_name=None)]
(orchestrate pid=5751) Traceback (most recent call last):
(orchestrate pid=5751)   File "/usr/local/lib/python3.10/dist-packages/data_processing_ray/runtime/ray/transform_orchestrator.py", line 89, in orchestrate
(orchestrate pid=5751)     processors = RayUtils.create_actors(
(orchestrate pid=5751)   File "/usr/local/lib/python3.10/dist-packages/data_processing_ray/runtime/ray/ray_utils.py", line 121, in create_actors
(orchestrate pid=5751)     raise UnrecoverableException(f"out of {len(actors)} created actors only {len(alive)} alive")
(orchestrate pid=5751) data_processing.utils.unrecoverable.UnrecoverableException: out of 2 created actors only 1 alive
(orchestrate pid=5751) 05:26:08 ERROR - Exception during execution out of 2 created actors only 1 alive: None
(orchestrate pid=5751) Traceback (most recent call last):
(orchestrate pid=5751)   File "/usr/local/lib/python3.10/dist-packages/data_processing_ray/runtime/ray/transform_orchestrator.py", line 138, in orchestrate
(orchestrate pid=5751)     stats["processing_time"] = round(stats["processing_time"], 3)
(orchestrate pid=5751) KeyError: 'processing_time'
(orchestrate pid=5751) 05:26:08 ERROR - Exception during execution 'processing_time': None
05:26:18 INFO - Completed execution in 2.309 min, execution result 1
INFO:data_processing_ray.runtime.ray.transform_launcher:Completed execution in 2.309 min, execution result 1

Reproduction script
code
colab link
error happens on step 5.2
Works with release 0.2.1
pip install  --default-timeout=100  \
        data-prep-toolkit-transforms==0.2.1 \
        data-prep-toolkit-transforms-ray==0.2.1 \
        deepsearch-toolkit

Fails with release 0.2.2.dev1
pip install  --default-timeout=100  \
        data-prep-toolkit[ray]==0.2.2.dev1 \
        data-prep-toolkit-transforms[ray,all]==0.2.2.dev1 \
        deepsearch-toolkit

Anything else
No response
OS
Other
Python
Other
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Based on the logged exception this seems to be a duplicate of #719 and is fixed in PR #721 that has been merged.

@sujee can you verify the fix?

code : https://colab.research.google.com/github/sujee/data-prep-kit-examples/blob/main/dpk-intro/process_pdfs_1_ray.ipynb
tried with 0.2.2
Error at step 5.2 (docid ray transform)
@shahrokhDaijavad can you verify please?  thx

@sujee I just tested this again and it fails with the default ray settings on Google Colab at step 5.2 (as you reported above). Either we have to play with the ray settings to see if we can get it working (considering the limitations of Colab with the number of CPUs, etc), or now that we have a python version of fdedup in 0.2.3 release, change the Colab example to use python and not ray.


Better highlight which tab is active

Fixes #1623

Wasn't really able to test the ASP.NET change, but it's pretty straight forward to fix the expressions for the active tab. The CSS I tested in F12 tools in Edge.
@praveenkuttappan could you update the internal wiki (or somewhere) which dev test settings?

I have queued a job to deploy this to staging instance. We can get this merged after it is deployed to staging. We use staging instance for testing for the time being. But I can create a new dev test resources and update wiki with those details.

I have queued a job to deploy this to staging instance. We can get this merged after it is deployed to staging. We use staging instance for testing for the time being. But I can create a new dev test resources and update wiki with those details.

+1 for updating the wiki with the new resource information. I noticed that localHost testing no longer works with the local settings I was using.

Same here, @JoshLove-msft . @praveenkuttappan is going to look into that. I get an error when GitHub is calling back after authentication.

Same here, @JoshLove-msft . @praveenkuttappan is going to look into that. I get an error when GitHub is calling back after authentication.

Yep same error


Custom set peer names are not saved

More of a feature request maybe, but it would be great if peer names would be saved, so on next startup you can still recognize your neighbors.

Available in the latest release.


Resolved, On hold, Stale, Closed, othe states of the issues

Comparing to bitbucket, we have lost the resolve button for issues (not only this one, also other states are missing). Everything is in close, but it doesn't make sense.

@matthew-morris-cessda

This can be acheived by using labels alongside the GitHub issue states (closed as completed; closed as not done). I've added labels for "Resolved" and "On hold". Please let us know if there are any more states that you would like.

Ok. Thanks for the explanation

S pozdravom
Martin Šeleng
Institute of Informatics
Slovak Academy of Sciences
Dubravska cesta 9
845 07 Bratislava
Slovak Republic
Tel: +421 (0)2 5941-1256
Fax: +421 (0)2 5477-1004
URL: ikt.ui.sav.sk
E-mail: @.*** @.***>

On Mon, Mar 27, 2023 at 10:22 AM Matthew Morris @.***>
wrote:

You have to use labels for this purpose. GitHub only understands 3 issue
states: open, closed as completed and closed as not done. See
https://docs.github.com/en/issues/tracking-your-work-with-issues/closing-an-issue
for GitHub's documentation.
—
Reply to this email directly, view it on GitHub
https://github.com/cessda/cessda.cvs.two/issues/591#issuecomment-1484712626,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB23PTUABT3QAU4FZRXPM5LW6FE4ZANCNFSM6AAAAAAWIY6INA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


histogram with log scale has display issues for small values

It seems matplotlib can not properly render histograms, when log scale is active for the y-axis and the absolute frequency is smaller than ~2.7, ie. 1 and 2.
The Python code:
import pandas as pd
import matplotlib.pyplot as plt
from matplotlib2tikz import save as tikz_save

plot_values = pd.DataFrame(
    {
        'test0': pd.Series([0, 0, 0, .1, .1, .2]),
        'test1': pd.Series([0, .1]),
    }
)
plot_values.plot.hist(stacked=False)  # Should not use stacked for log plot
plt.gcf().set_size_inches(4, 3)
plt.yscale('log')
plt.ylim(ymin=.1)

plt.savefig('/tmp/test_a.png')
tikz_save('/tmp/test_a.tikz')

The png looks fine:

However, from the pdf it seems histogram bars are plotted with their lower bound starting at ~2.7 (instead of -inf). This will especially cause issues when the frequency is 1 or 2.

Can you cook up an example code without pandas? It's such a heavy dependency...

I have a similar issue with my histogram while using a log scale on the y axis.
The matplotlib version looks fine.
The .tex version has a similar problem as MarcoFalke described.
Matplotlib PNG Version:

.tex Version after compiling in latex:

I have the same issue as @derbach. I solved it by manually replacing values in the tikz output file. The lower value for plotting a bar was set to 0, and I replaced it by 0.01. A bit dirty but it was enough in my case.
e.g., \draw[draw=white,fill=color0,line width=0.32pt] (axis cs:1,0.01) rectangle (axis cs:2,13357);


Add SOCKS4 support for outbound proxy

Please see this original message:
https://t.me/adguard_en/138147
It appears that in Iran AdGuard's outbound proxy functionality is often used for accessing proxies and there're users from there that badly need SOCKS4 and not just SOCKS5. In theory, it shouldn't be a problem to add it so why not?

    OPROXY_MODE_SOCKS4,

🤔

Please Add Socks5 To local proxy Mode
and Import .TXT Proxies to Network Setting Option


[AMQ-6586] Compilation of activemq-unit-test fails

Should be merged only in activemq-5.14.x branch


[FluentUI Vnext] Integrating Core functionality, Avatar and AvatarGroup.

Platforms Impacted

[x] iOS
[ ] macOS

Description of changes
This is the first integration bringing the vnext-prototype contents to main.
The components being integrated to main are:

Core functionality of the Design Token System.
Avatar (Vnext version in SwiftUI), which replaces the current AvatarView.
AvatarGroup (Vnext version in SwiftUI), which replaces the current AvatarGroupView.

Verification

Local build and run of the Demo app.
-- Exercising controls' functionality through their demo controllers.

Devmain integration local build to ensure public API changes will be accomodated.

Pull request checklist
This PR has considered:

[ ] Light and Dark appearances
[ ] VoiceOver and Keyboard Accessibility
[ ] Internationalization and Right to Left layouts
[ ] Different resolutions (1x, 2x, 3x)
[ ] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)

Microsoft Reviewers: Open in CodeFlow

Noticed a weird animation on Avatar when adding and removing the status badge. Regression from legacy?
https://user-images.githubusercontent.com/4934719/127694702-a22dd45b-967a-4169-8308-4521eb90c8e8.mov

Noticed a weird animation on Avatar when adding and removing the status badge. Regression from legacy?
RPReplay_Final1627667057.mov

Noticed a weird animation on Avatar when adding and removing the status badge. Regression from legacy?
RPReplay_Final1627667057.mov

As we spoke before, this is an issue caused by the change of the outer frame of the avatar which applies for any transition that causes a change on that frame. I'll issue a separate PR to fix this after the merge is done.


Update virtual-machines-windows-sql-server-agent-extension.md

Added
-Location "East US 2"
to
Set-AzureRmVMSqlServerExtension -ResourceGroupName "resourcegroupname" -VMName "vmname" -Name "SQLIaasExtension" -Version "1.2" -Location "East US 2"
Location parameter seems to be required when running this command per latest version of Azure Powershell.

Hi @oedenfield, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Orrin Edenfield). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@oedenfield : Thanks for your contribution to the Azure documentation! The author, @rothja, has been notified to review your proposed change.

@oedenfield Thanks for the catch.

#sign-off


Import Error from click.termui

Hi, Honestly, I have no idea where this is coming from, I used the package a couple of days ago to update some dependencies and it worked fine. I'm currently using the latest 0.6.0 version, I tried to downgrade to 0.5.4 and 0.5.3 and same issue.  I think it happened during a poetrup upgrade, but I don't know when.
Traceback (most recent call last):
  File "/Users/tobi/Library/Caches/pypoetry/virtualenvs/fwellbeing-hsd0M_FB-py3.10/bin/poetryup", line 5, in <module>
    from poetryup.main import app
  File "/Users/tobi/Library/Caches/pypoetry/virtualenvs/fwellbeing-hsd0M_FB-py3.10/lib/python3.10/site-packages/poetryup/main.py", line 7, in <module>
    import typer
  File "/Users/tobi/Library/Caches/pypoetry/virtualenvs/fwellbeing-hsd0M_FB-py3.10/lib/python3.10/site-packages/typer/__init__.py", line 12, in <module>
    from click.termui import get_terminal_size as get_terminal_size
ImportError: cannot import name 'get_terminal_size' from 'click.termui' (/Users/tobi/Library/Caches/pypoetry/virtualenvs/fwellbeing-hsd0M_FB-py3.10/lib/python3.10/site-packages/click/termui.py)

Hi @Tobi-De, thanks for reporting an issue.
poetryup depends on Typer which in turn depends on click, and click has removed get_terminal_size in version 8.1.0. An issue has been reported in Typer: https://github.com/tiangolo/typer/issues/377 and a fix seems to be around the corner. A temporary workaround is to lock click to 8.0.4.

Hi @Tobi-De, thanks for reporting an issue.
poetryup depends on Typer which in turn depends on click, and click has removed get_terminal_size in version 8.1.0. An issue has been reported in Typer: tiangolo/typer#377 and a fix seems to be around the corner. A temporary workaround is to lock click to 8.0.4.

Ok got it, thanks

I just released Typer 0.4.1 that should handle this. :rocket: :nerd_face:


mm-grid-item click event

@dlasky @jcmoore @shuwen The stopgap solution to mm-grid-item (and inner items) click handling.

LGTM pending shadow testing on those event targets (i suspect its not normalized) but the approach is fine.


fix memcpy

As we are not capable of garante that a type won't change during the compilation passes, we cannot use copymemory as types might have changed between the time we create the node and when we want to produce the equivalent spirv.
Thus always rely on load/store to perform the copy.
fix #1109

I need to add some more comments in the code, and I forgot to add a test as well. I'll do that tomorrow


Implement process for ion-js cli

Description:
As part of ion-test-driver, this commit is for ion-js-cli process. It supports:

--output(-o) : rewrite given input files to specified output format
--output-fomrat(-f): specify ion output format
--error-report(-e): generate error report to specified location

Completeted Tasks:

Implement processFiles function for binary/text/pretty formats to rewrite given input files.
Implement processEvents for generating event stream out of ion stream.
Supports help menu with all options and usage
Supports multiple input files rewritten into same output file

TO DO:

[X] Support symbol[READ | WRITE | STATE] for error-report.
[X] Support multiple input files for error-report.
[X] Support None output format
[ ] Implement other options --catalog, --imports, --filter

Test:
tested using multiple input files for all output formats and events.
e.g. ts-node process example.ion --output-format text --output output.ion --error-report error_report

Commit: e200c9c

Support compare command

compare  method for basic comparison
compareEquivs for equivs, non-equips and equiv-timeline comparison.

Commit : fde92fc

fixed suggested changes
solved merge conflicts

Commit: 441a1c8

write to file only once. (remove duplicate writes)

TO DO:

Investigate on the CI build fail.

Commit: 9c1d0d2

IonEventStream

changed the way embedded streams were handled.
added doctype comment for equivs and compareEquivs.

Process

changed the way embedded streams were handled to make it work properly for List. (writeToEventStream)

Previously suggested changes


Is this project no longer maintained?

There is only one commit between the latest release and now.
Have you guys stopped maintaining the project?
There's still a lot of work to be done in terms of documentation and performance benchmarks.

The project is maintained, but it is not actively developed at the moment as we don't currently have need for new features in it and it works well as-is in our use cases.  Documentation could certainly be improved, but I don't think I'll do anything about it until I need more changes to the project.
I'm happy to review and help with documentation and other changes if someone else creates a pull request.

Thanks for the heads up.


Get path for first page of paginated pages

When a page is paginated, the edit link includes the pagination part of the URL. This grabs the path from the first page of the pagination collection.
I would appreciate some help with the tests as I'm not familiar enough with Ruby.
Closes #213

Okay. The existing tests pass.  Therefore, no regressions.
Now you need to add the test that proves this change works (with your use-case) as claimed.

Hmm, not what I was expecting. Although I note that the tests are running against Jekyll 4.2.1 and I was using 3.9.0 to match the version in github-pages gem.

Seems like a useful bug fix.


Suggestion: decide on an official stackoverflow tag, maybe "wincomposition"

The title says it all: it'd probably be better if people would use SO to ask random questions than github.

I agree with wincomposition, that's was my first search term I entered on SO when I looked for win composition questions.

But there's also the old Composition. :P How about winrt-composition?

We've been considering  wincomposition, we're less worried about the old composition confusion.

Let's use win-composition then!!

My vote is for "wincomposition" :+1:


Use regular date when no internal date found

app crashes when trying to fetch "Sent" email folder because the sent
emails don't have an INTERNALDATE field in their summary. In this case
use the envdate field. Fix corresponding parsing function.
Signed-off-by: Aurelien Aptel<EMAIL_ADDRESS>

The date in the envfield look like standard email dates (eg Mon, 11 Apr 2016 14:43:35 +0200)

Merged.  Thanks!!  -rt


Fixes #19

Hi @weierophinney ,
It may be good to merge this first and tag a version.
Then merge the PR https://github.com/phly/http/pull/18 .
Thank you.

Any chance of a unit test to make sure it stays fixed? :-)

aah is it breaking ?

aah ok. I will add one :) .

Fixed :) .

On a side note, I did some experiments over https://github.com/harikt/ConduitExperiments . It took only a few hours to make some thing which works.
If you are willing, do you like to have a router middleware here ? Or do you have any other plans ?

Thanks!

@weierophinney did you forget to merge or is looking for something ?

I've been away from my computer and not had a chance to merge yet. (Been following the pr from my tablet.)

oh you don't use github merge :) .

Nope. On this repo, in particular, I tend to update the changelog on each merge, and, in this case, will be doing an immediate patch release after the merge.


Cesium doesn't understand the Alpha-F5-8 'Execute Program' feature

When in a program, using the built-in edit menu to run the program simply quits to the Cesium launcher.

Soooo, what's your point. It's known to do that.

That menu should run the program directly.

Can't fix because TI weirdness


Add cocos-creator and modDB.

Hey @HaitaoBio there are some conflicts with the current file, can you fix them first, please?

sorry,now have done.


feat: check INSTALL.md (TRG 1.02)

Implemented new method IsOptional in interface QualityGuideline to cover optional checks (aka INSTALL.md or AUTHOR.md(→ TRG 2.03)).
Added new feature to test if optional file 'INSTALL.md' exists. If the file does not exist, this test fails (for now) but not the test run itself (does not exit 1):
$ go run main.go checkLocal
Running local checks of eclipse-tractusx release guidelines
Testing Quality Guideline: TRG 1.01 - README.md
Testing Quality Guideline: TRG 1.02 - INSTALL.md
Failed! Test failed, but marked as optional. More infos: Did not find optional INSTALL.md file in current directory.
https://eclipse-tractusx.github.io/docs/release/trg-1/trg-1-2
Testing Quality Guideline: TRG 1.03 - CHANGELOG.md

Just a general sidenote: if we have more checks, should we try to prefix them or organize them more clearly?

IMHO, I would rephrase the Message since "Failed! Test failed, but marked as optional." sounds quite harsh.
Maybe something like "Warning! Optional Test failed, but is considered best practice..."

IMHO, I would rephrase the Message since "Failed! Test failed, but marked as optional." sounds quite harsh. Maybe something like "Warning! Optional Test failed, but is considered best practice..."

Technically the test fails, therefore wording is correct. But I get your point. Currently there is nothing between Passed and Failed implemented.
Would that be a better output:
$ go run main.go checkLocal
Running local checks of eclipse-tractusx release guidelines
Testing Quality Guideline: TRG 1.01 - README.md
Testing Quality Guideline: TRG 1.02 - INSTALL.md
Warning! Test failed, but test is marked as optional.
More infos:
        Optional INSTALL.md file not found in current directory.
        https://eclipse-tractusx.github.io/docs/release/trg-1/trg-1-2
Testing Quality Guideline: TRG 1.03 - CHANGELOG.md

Maybe aligning it a bit more with the other messages:
Warning! Testing (Optional) Quality Guideline: TRG 1.02 - INSTALL.md failed, more info: ...

Maybe aligning it a bit more with the other messages: Warning! Testing (Optional) Quality Guideline: TRG 1.02 - INSTALL.md failed, more info: ...

I'd leave it for not this way and redesign message handling with another PR.

New check already applied to this PR: https://github.com/eclipse-tractusx/tractusx-quality-checks/actions/runs/4628495387/jobs/8187637882

@carslen & @SebastianBezold can we make it more github action native? Like showing it in the checks instead of indirectly in the github action?
Also potentially you can indent it to make it easy seeable that the warning is part of TRG 1.02

@carslen & @SebastianBezold can we make it more github action native? Like showing it in the checks instead of indirectly in the github action?
Also potentially you can indent it to make it easy seeable that the warning is part of TRG 1.02

In general, yes we can do that. In this case I would suggest to create a separate command, that "knows" or "assumes" it is running on PR basis and therefore has the GitHub features available.
I would create it as separate commands, so we do not need to add too many conditions, checking in which context the checks are run.
The checks themselves run under the assumptions, that the files to check are available locally, relative to where the command is run from. So if we build something that matches this assumption and makes the results accessible via GitHub check API, then this can definitely be done


DNS Resolution from within a Chroot?

Unknown if Question or Bug, and sorry if this is not the proper location for documenting this behavior...
Please fill out the below information:

Your Windows build number:  (Type ver at a Windows Command Prompt)

Microsoft Windows [Version 10.0.17763.529]

What you're doing and what's happening:  (Copy&paste the full set of specific command-line steps necessary to reproduce the behavior, and their output. Include screen shots if that helps demonstrate the problem.)

Special Note, this is a Lenovo Yoga C630 device with a 64-bit Qualcomm Snapdragon Arm processor, not an x86 machine.......However, inside of WSL Ubuntu 18.04, DNS resolution works properly and an "apt-get update" will resolve the Ubuntu repo's normally.  However, I need to do some work on a 32-bit Arm filesystem that will eventually live on a Raspberry Pi, so, I have installed Qemu and can Chroot into the 32-bit Arm rootfs.  Once inside of that chroot, name resolution does not work, and "apt-get update" results in a failure to resolve ports.ubuntu.com to read the package lists.
The resolv.conf file within the 32-bit filesystem just uses the <IP_ADDRESS> and <IP_ADDRESS> nameservers, so there should not be an issue, but something else must be impacting the system.
Here is the complete log, noting the first set of commands are in WSL Ubuntu 18.04, then the chroot occurs, switching the filesystem:
david@DESKTOP-B007DR3:~/python/rootfs$ cat /etc/resolv.conf
This file was automatically generated by WSL. To stop automatic generation of this file, remove this line.
nameserver <IP_ADDRESS>
david@DESKTOP-B007DR3:~/python/rootfs$ sudo apt-get update
[sudo] password for david:
Hit:1 https://download.docker.com/linux/ubuntu bionic InRelease
Hit:2 http://ppa.launchpad.net/projectatomic/ppa/ubuntu bionic InRelease
Hit:3 http://ports.ubuntu.com/ubuntu-ports bionic InRelease
Get:4 http://ports.ubuntu.com/ubuntu-ports bionic-updates InRelease [88.7 kB]
Hit:5 http://ports.ubuntu.com/ubuntu-ports bionic-backports InRelease
Hit:6 http://ports.ubuntu.com/ubuntu-ports bionic-security InRelease
Get:7 http://ports.ubuntu.com/ubuntu-ports bionic-updates/main arm64 Packages [509 kB]
Get:8 http://ports.ubuntu.com/ubuntu-ports bionic-updates/universe arm64 Packages [850 kB]
Fetched 1448 kB in 5s (276 kB/s)
Reading package lists...   They have been ignored, or old ones used instead.
root@DESKTOP-B007DR3:/#

What's wrong / what should be happening instead:
DNS should resolve.

Strace of the failing command, if applicable:  (If some_command is failing, then run strace -o some_command.strace -f some_command some_args, and link the contents of some_command.strace in a gist here).

I am having the same issue, Windows 10 ARM64 build 20215.. I setup a qemu chroot into x86_64 from Ubuntu 18.04 but when I try to apt update and ping anything I get the same error resolving address..
Tried the same edits to 'resolv.conf' but no joy..


Cant't install

This is SBCL 2.1.3, an implementation of ANSI Common Lisp.
More information about SBCL is available at <http://www.sbcl.org/>.

SBCL is free software, provided as is, with absolutely no warranty.
It is mostly in the public domain; some portions are provided under
BSD-style licenses.  See the CREDITS and COPYING files in the
distribution for more information.
* (ql:quickload :lisp-stat)
To load "lisp-stat":
  Load 1 ASDF system:
    lisp-stat
; Loading "lisp-stat"
..
debugger invoked on a SB-INT:SIMPLE-PROGRAM-ERROR in thread
#<THREAD "main thread" RUNNING {70088A3863}>:
  SELECT:HEAD already names an ordinary function or a macro.

Type HELP for debugger help, or (SB-EXT:EXIT) to exit from SBCL.

restarts (invokable by number or by possibly-abbreviated name):
  0: [CONTINUE                     ] Replace the function binding
  1: [TRY-RECOMPILING              ] Recompile pprint and try loading it again
  2: [RETRY                        ] Retry
                                     loading FASL for #<CL-SOURCE-FILE "data-frame" "pprint">.
  3: [ACCEPT                       ] Continue, treating
                                     loading FASL for #<CL-SOURCE-FILE "data-frame" "pprint">
                                     as having been successful.
  4:                                 Retry ASDF operation.
  5: [CLEAR-CONFIGURATION-AND-RETRY] Retry ASDF operation after resetting the
                                     configuration.
  6:                                 Retry ASDF operation.
  7:                                 Retry ASDF operation after resetting the
                                     configuration.
  8: [ABORT                        ] Give up on "lisp-stat"
  9: [REGISTER-LOCAL-PROJECTS      ] Register local projects and try again.
 10:                                 Exit debugger, returning to top level.

(ENSURE-GENERIC-FUNCTION HEAD)

This is sbcl on m1 Mac, same issue with ccl on Intel Mac.

Lisp-Stat isn't yet in Quicklisp. I submitted an issue for it's inclusion (quicklisp/quicklisp-projects#2016). Normally quicklisp projects are updated once a month, so I expect it to be there at the end of April. The alternative is to do a manual install, described in the installation section of the docs.

That's what I've done.
I'm at ”Try it out“, which is where the above occurs.

When did you do the checkout? I removed head and tail about a week ago after encountering this error. It could be that you pulled down the repositories before that change. Might want to pull from the select repository again.
Also, I pushed changes to data-frame yesterday and fixed a bug in dfio this morning that I'll push out now.

I used the method described in the manual, have just run git pull everywhere, noticed some changes, but the issue stil stands.

What OS are you running? I'll try to reproduce with a fresh checkout in the morning.
In the meantime, let's delete all your fasl files and start with a clean rebuild.

What I tried myself:
I cloned lisp-stats' repository in QL's local-projects
(asdf:clear-source-registry)

quickload "lisp-stat" ->

System "data-frame" not found
[Condition of type QUICKLISP-CLIENT:SYSTEM-NOT-FOUND]

The main repository doesn't say to clone all other projects :] (https://github.com/Lisp-Stat/lisp-stat) I supposed it brings the dependencies itself, but maybe not yet.
(ql:dist-version "quicklisp")
"2020-12-20"

Ah, I think I see the problem. Thank you @vindarel for providing the clue.
The manual installation instructions left out the select repository, and that dependency was being pulled in from Quicklisp. Check out the Lisp-Stat version of select with:
git clone https://github.com/Lisp-Stat/select.git

and see if it builds properly.

lisp-stat is now in the latest Quicklisp: http://blog.quicklisp.org/2021/04/april-2021-quicklisp-dist-update-now.html
to upgrade: (ql:update-dist "quicklisp")

@rolfmblindgren, are you still having problems installing?

not as such, but still
? (in-package :ls-user)
#<Package "LS-USER">
? (load #P"LS:DATASETS;CAR-PRICES")
#P"/Users/roffe/quicklisp/dists/quicklisp/software/lisp-stat-20210411-git/datasets/car-prices.lisp"
? (mean 'car-prices)

Error: There is no applicable method for the generic function:
          #<STANDARD-GENERIC-FUNCTION CENTRAL-SAMPLE-MOMENTS #x302000E3B62F>
        when called with arguments:
          (CAR-PRICES :DEGREE 1 :WEIGHTS NIL)
While executing: #<CCL::STANDARD-KERNEL-METHOD NO-APPLICABLE-METHOD (T)>, in process listener(1).
Type :GO to continue, :POP to abort, :R for a list of available restarts.
If continued: Try calling it again
Type :? for other options.

--
Rolf Lindgren
+47 455 19 940
http://www.grendel.no/
On 21 Apr 2021, 04:43 +0200, Steve @.***>, wrote:

@rolfmblindgren, are you still having problems installing?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

OK, it works in GNU Emacs’ sly-mode.  But not when the same lisp executable is called from the shell.
--
Rolf Lindgren
+47 455 19 940
http://www.grendel.no/
On 21 Apr 2021, 08:22 +0200, Rolf Marvin Bøe Lindgren @.***>, wrote:

not as such, but still
? (in-package :ls-user)
#<Package "LS-USER">
? (load #P"LS:DATASETS;CAR-PRICES")
#P"/Users/roffe/quicklisp/dists/quicklisp/software/lisp-stat-20210411-git/datasets/car-prices.lisp"
? (mean 'car-prices)

Error: There is no applicable method for the generic function:
          #<STANDARD-GENERIC-FUNCTION CENTRAL-SAMPLE-MOMENTS #x302000E3B62F>
        when called with arguments:
          (CAR-PRICES :DEGREE 1 :WEIGHTS NIL)
While executing: #<CCL::STANDARD-KERNEL-METHOD NO-APPLICABLE-METHOD (T)>, in process listener(1).
Type :GO to continue, :POP to abort, :R for a list of available restarts.
If continued: Try calling it again
Type :? for other options.

--
Rolf Lindgren
+47 455 19 940
http://www.grendel.no/
On 21 Apr 2021, 04:43 +0200, Steve @.***>, wrote:

@rolfmblindgren, are you still having problems installing?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

OK, ignore that … pass the variable unsuited, and this happens
? (mean car-prices)
2.810199998617172D0
? (median car-prices)

Error: Undefined function :DOCUMENTATION called with arguments ("Returns median of SAMPLE.  SAMPLE must be a sequence of real numbers.") .
While executing: #<STANDARD-METHOD MEDIAN (SEQUENCE)>, in process listener(1).
Type :GO to continue, :POP to abort, :R for a list of available restarts.
If continued: Retry applying :DOCUMENTATION to ("Returns median of SAMPLE.  SAMPLE must be a sequence of real numbers.").
Type :? for other options.
1 >

--
Rolf Lindgren
+47 455 19 940
http://www.grendel.no/
On 21 Apr 2021, 08:25 +0200, Rolf Marvin Bøe Lindgren @.***>, wrote:

OK, it works in GNU Emacs’ sly-mode.  But not when the same lisp executable is called from the shell.
--
Rolf Lindgren
+47 455 19 940
http://www.grendel.no/
On 21 Apr 2021, 08:22 +0200, Rolf Marvin Bøe Lindgren @.***>, wrote:

not as such, but still
? (in-package :ls-user)
#<Package "LS-USER">
? (load #P"LS:DATASETS;CAR-PRICES")
#P"/Users/roffe/quicklisp/dists/quicklisp/software/lisp-stat-20210411-git/datasets/car-prices.lisp"
? (mean 'car-prices)

Error: There is no applicable method for the generic function:
          #<STANDARD-GENERIC-FUNCTION CENTRAL-SAMPLE-MOMENTS #x302000E3B62F>
        when called with arguments:
          (CAR-PRICES :DEGREE 1 :WEIGHTS NIL)
While executing: #<CCL::STANDARD-KERNEL-METHOD NO-APPLICABLE-METHOD (T)>, in process listener(1).
Type :GO to continue, :POP to abort, :R for a list of available restarts.
If continued: Try calling it again
Type :? for other options.

--
Rolf Lindgren
+47 455 19 940
http://www.grendel.no/
On 21 Apr 2021, 04:43 +0200, Steve @.***>, wrote:

@rolfmblindgren, are you still having problems installing?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Fixed. You'll need to pull the latest change from numerical-utilities.

Still an issue?

The issue seems to have been resolved. Thanks! :)
--
Rolf Lindgren
+47 455 19 940
http://www.grendel.no/
On 26 Apr 2021, 02:53 +0200, Steve @.***>, wrote:

Still an issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Field and Column filters on API Endpoints

I wanted to double check and make sure I'm not missing something in your API documentation, but I have yet to see how to include the fields and columns query string parameters that are allowed for within the CW API for GET requests.
I use these pretty extensively to limit the amount of data returned to a client on a mobile device/hotspot as well as within Azure to limit the overall amount of bandwidth use needed (and charged for!) to iterate through a large amount of records.  For instance, I have an Azure function that will go grab all of our active customers (2500 out of 13000 total) and bringing back the full company record for 2500 customers uses a lot of bits and bytes.
Is this possible, or would this be an enhancement request?

It is possible now, however the typings and documentation could definitely be improved.
You can filter the response by using  extraQueryParams?: any like { fields: 'id, name' }  so a request would look something like
'use strict';

import { Connectwise as CW } from 'connectwise-rest-api';

export const ConnectWise = new CW('url', 'company', 'publicKey', 'privateKey');

ConnectWise.CompaniesApi.companyCompaniesGet( { conditions: '(status/id=1 or status/id=20)', pageSize: 1000 }, { fields: 'id,name,status' }).then( response => {
	console.log(response);
});

that worked. thanks for the help


Patches

Various updates, mostly to do with FC level handling

Codecov Report
Attention: 21 lines in your changes are missing coverage. Please review.

Comparison is base (99915ed) 59.89% compared to head (7117057) 59.85%.

Files
Patch %
Lines

mth5/tables/channel_table.py
50.00%
15 Missing :warning:

mth5/groups/fc_dataset.py
14.28%
6 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #190      +/-   ##
==========================================
- Coverage   59.89%   59.85%   -0.05%
==========================================
  Files         139      139
  Lines       15403    15425      +22
==========================================
+ Hits         9226     9233       +7
- Misses       6177     6192      +15

Flag
Coverage Δ

tests
59.85% <60.37%> (-0.05%)
:arrow_down:

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

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


Remote Execution of Powershell Command with PowerShell-Core Version

I am trying to execute Powershell command remotely, my problem is that the Powershell is used is version 4.0, and I want to execute my command remotely using Powershell Core 6.0.4.
I tried to use this command on the remote host
Set-PSSessionConfiguration -name microsoft.powershell -psversion 6.0 -FORCE
And getting this error:
Set-PSSessionConfiguration : Cannot bind parameter 'PSVersion' to the target. Exception setting "PSVersion": "The
value 6.0 is not valid for the PSVersion parameter. The available values are 2.0,3.0 and 4.0."

I got version 6.0.4 installed on my remote machine.
I know it uses version 4 to execute my remote command, because
Invoke-Command -Session $session -ScriptBlock {$PSVersionTable.PSVersion};
returns:
Major  Minor  Build  Revision PSComputerName
-----  -----  -----  -------- --------------
4      0      -1     -1       IIS-DEV2

Any ideas how to force it to use version 6?

Don't think you can do this directly, because the cmdlet originates from version 4.
It might be as simple as just calling pwsh to open PS core directly before you do anything else in the remote session

Why do you care about 4.0 6.0?
You only need to know the difference between the specific commands, you need to write your own script to test this difference.


sensors only publish airspeed if finite

This was causing one of the intermittent CI failures.
Switching the VTOL land detector to indicated airspeed while I'm at it.

@dagar Removing the hrt_elapsed_time calls from the land detector is something we can also do in future.


Crash when provided train data and test data do not have same number of columns

flight_delays dataset as per https://www.kaggle.com/c/flight-delays-spring-2018
On tangram 0.7:
 $ tangram train --file-train flight_delays_train.csv --file-test flight_delays_test.csv --target dep_delayed_15min
✅ Inferring train table columns. 0s 43ms
✅ Loading train table. 0s 40ms
✅ Loading test table. 0s 45ms
✅ Shuffling. 0s 9ms
✅ Computing train stats. 0s 121ms
✅ Computing test stats. 0s 112ms
✅ Finalizing stats. 0s 1ms
error: panicked at 'removal index (is 8) should be < len (is 8)', library/alloc/src/vec/mod.rs:1385:13
   0: backtrace::capture::Backtrace::new
   1: tangram::train::train::{{closure}}
   2: std::panicking::rust_panic_with_hook
   3: std::panicking::begin_panic_handler::{{closure}}
   4: std::sys_common::backtrace::__rust_end_short_backtrace
   5: _rust_begin_unwind
   6: core::panicking::panic_fmt
   7: alloc::vec::Vec<T,A>::remove::assert_failed
   8: tangram_core::train::Trainer::prepare
   9: tangram::main
  10: std::sys_common::backtrace::__rust_begin_short_backtrace
  11: _main

Noticed slightly different stacktrace with tangram latest (main branch):
 $ tangram-tip train --file-train flight_delays_train.csv --file-test flight_delays_test.csv --target dep_delayed_15min
✅ Inferring train table columns. 0s 48ms
✅ Loading train table. 0s 41ms
✅ Loading test table. 0s 44ms
✅ Shuffling. 0s 9ms
✅ Computing train stats. 0s 115ms
✅ Computing test stats. 0s 119ms
✅ Finalizing stats. 0s 1ms
error: panicked at 'removal index (is 8) should be < len (is 8)', crates/core/train.rs:176:58
   0: backtrace::capture::Backtrace::new
   1: tangram::train::train::{{closure}}
   2: std::panicking::rust_panic_with_hook
   3: std::panicking::begin_panic_handler::{{closure}}
   4: std::sys_common::backtrace::__rust_end_short_backtrace
   5: _rust_begin_unwind
   6: core::panicking::panic_fmt
   7: alloc::vec::Vec<T,A>::remove::assert_failed
   8: tangram_core::train::Trainer::prepare
   9: tangram::main
  10: std::sys_common::backtrace::__rust_begin_short_backtrace
  11: _main

@comunidadio thanks for opening this issue. We should show a good error message in this case. We will have a fix pushed shortly.

Good catch @comunidadio - this commit will now quit gracefully in this case.


Specify username for executing a plugin

To change username, sudo must be used as work-around like:
command="sudo -u xxxx ..."

Specifying username is as follows.
[plugin.metrics.vmstat]
command = "ruby /etc/sensu/plugins/system/vmstat-metrics.rb"
username = nobody

resolved at #250