Release

:book::rocket: RELEASE NOTES Preview

:rocket: Release
2021-11-04

:sparkles: Ignore tags
2021-11-04
Changes

📖 Add ignoreTag attribute.
Now you can ignore Pull Request text surrounded with ignoreTag.
Default ignore tag is `

Generated with RNG :robot:


[microTVM] Project API infrastructure

This PR implements the Project API RFC, a plugin-like infrastructure that allows TVM to interact with build tools for third-party platforms. By implementing the Project API interface for a platform, developers can run core TVM workflows such as host-driven inference and AutoTVM without needing to build the full TVM library on the platform. This is initially targeted at microTVM use cases, where platforms don't have the OS facilities nor the flash space to run a full TVM C++ runtime nor a Python interpreter.
More discussion is available in the RFC. Implementations are provided here for Zephyr and for the POSIX subprocess test platform.
NOTE: This PR is based on top of #8072 and will merge following that one.
@mehrdadh @guberti @tqchen @jroesch @tkonolige @csullivan @leandron @u99127 @Mousius @giuseros @gromero @stoa @hogepodge

@mehrdadh @gromero please take another look!

@areusch Fixes in https://github.com/apache/tvm/pull/8380/commits/0d0c504a158ac9e03c301b88a5ca1127c57bd06e are good. I think the only remaining issue is that prj.conf generation mechanism currently doesn't account for boards that don't have a FPU, so CONFIG_FPU=y (like the mps2_an521 board). So I think we need to fix it because it's a regression of https://github.com/apache/tvm/pull/8055. Otherwise I think it's pretty much ready to land once CI is happy too :)

tested offline against nrf5340 and stm32f746xx_disco; waiting for CI to be fixed to retrigger. @gromero @mehrdadh can you take a look when you have a minute?

@areusch Thanks a lot for addressing all the issues and concerns. LGTM!

I'm happy for anyone to merge this once the conflicting files are solved and CI is green again. cc @tmoreau89

Thank you @areusch , @leandron, @gromero, @guberti, @mehrdadh  the PR was merged!

@gromero right now we haven't broadly addressed data dependencies in the wheel. i think we need to at some point build a central e.g. tvm.utils.get_packaged_resource("...") function to mimic tvm.micro.get_standalone_crt_dir (and use the former internally in the latter). this will allow us to add arbitrary data dependencies to the wheel and refer to their paths later on.


1.10 Completed Form Retention

1.10 Form submissions currently do not remove from the DGJ platform after completion. This ticket can specify and implements this retention period.
AC:
Find out the retention period.
Add the retention to the 1.10 complete form submissions to be removed from DGJ.
@MeghanStothers  @morezooomm
Could you please let me know how many days 1.10 submissions should be kept in DGJ before getting removed? The telework form has 60 days.

@iman-jamali-fw 60 days is the maximum unless there are business reasons why this period should be extended.

I am good with 60 days.
Don Smallwood, Director of Investigations & Intake
Employee Relations Division | BC Public Service Agency
167 Lorne Street | Kamloops BC | V2C 1V9
Office:<PHONE_NUMBER> | Mobile:<PHONE_NUMBER>
Living with gratitude and respect on the unceded territory of the Splatsin
Government confidentiality and privilege requirements apply to this message and any attachments.  If you are not the intended recipient, you are hereby notified that any review, retransmission, conversion to hard copy, copying, circulation or other use is strictly prohibited. If you are not the intended recipient, please notify the sender immediately, and delete this message and any attachments from both your inbox and deleted items folder.
@.***
From: Meghan Stothers @.>
Sent: Monday, March 6, 2023 12:52 PM
To: bcgov/digital-journeys @.>
Cc: Smallwood, Don PSA:EX @.>; Mention @.>
Subject: Re: [bcgov/digital-journeys] 1.10 Completed Form Retention (Issue #958)
[EXTERNAL] This email came from an external source. Only open attachments or links that you are expecting from a known sender.
@iman-jamali-fwhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fiman-jamali-fw&data=05|01|Don.Smallwood%40gov.bc.ca|c99f197e95a944e3eb7208db1e849215|6fdb52003d0d4a8ab036d3685e359adc|0|0|638137326953117602|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=qmtiD5RCMnhdp3iweEiUjPsfKUExcHyOUl7gFpmeJLs%3D&reserved=0 60 days is the maximum unless there are business reasons why this period should be extended.
—
Reply to this email directly, view it on GitHubhttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fbcgov%2Fdigital-journeys%2Fissues%2F958%23issuecomment-1456971131&data=05|01|Don.Smallwood%40gov.bc.ca|c99f197e95a944e3eb7208db1e849215|6fdb52003d0d4a8ab036d3685e359adc|0|0|638137326953117602|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=2XamqC3ORZ0aUaivXZYz0Ww%2FEQiLSzqAYUttSYHNI2g%3D&reserved=0, or unsubscribehttps://can01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FA37NTNNIS6RYFREAVWRLQLTW2ZE5LANCNFSM6AAAAAAVRPTYYA&data=05|01|Don.Smallwood%40gov.bc.ca|c99f197e95a944e3eb7208db1e849215|6fdb52003d0d4a8ab036d3685e359adc|0|0|638137326953117602|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=y4WChzH0qdQfEw8wZs25GR3NCgU2UPrEYVL%2F7TQ3keQ%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

Excellent - thank you.

@Andrew-Vargas-bcgov @Stella-Archer
This is ready for testing on TEST along with changes to the workflow that Andrew sent me.
For easier testing, retention time is set to 50 seconds (instead of 60 days). The submitted form under the Submitted Forms tab, should be removed 50 seconds after emails are sent.

Tested and working as intended.


Ignoring visible gpu device (device: 0, name: GeForce GTX 780M compute capability: 3.0) with Cuda compute capability 3.0. The minimum required Cuda capability is 3.5.

Please make sure that this is a build/installation issue. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:build_template
System information

OS Platform and Distribution: Linux Ubuntu 18.04
TensorFlow installed from (source or binary): From source
TensorFlow version: 2.4.1
Python version: 3.8.5
Installed using virtualenv? pip? conda?: conda
Bazel version (if compiling from source): bazel 3.1.0
GCC/Compiler version (if compiling from source): gcc (Ubuntu 7.5.0-3ubuntu1~18.04) 7.5.0
CUDA/cuDNN version:  10.1/8
GPU model and memory: dual GeForce GTX 780M with 2 GB each

Describe the problem
I built tensorflow (version 2.4.1) from source becuase my old gpu compute capability is 3.0.
I followed the instructions from https://www.tensorflow.org/install/source
and from<EMAIL_ADDRESS>I did the build process 5 times, every time I change some parameters in ./configure
I also manualy edited .tf_configure.bazelrc (suggested parameters from https://github.com/tensorflow/tensorflow/issues/24126#issuecomment-443847200) to turn XLA off by
removing --config=XLA
and
adding the line build --define with_xla_support=false
and every time I run python to check if tensorflow is using my gpu, I got False as shown below.
>>> tf.test.is_gpu_available(True,3.0) 2021-01-25 09:21:44.146701: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:941] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero 2021-01-25 09:21:44.148484: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1720] Found device 0 with properties:  pciBusID: 0000:01:00.0 name: GeForce GTX 780M computeCapability: 3.0 coreClock: 0.797GHz coreCount: 8 deviceMemorySize: 3.94GiB deviceMemoryBandwidth: 149.01GiB/s 2021-01-25 09:21:44.148617: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:941] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero 2021-01-25 09:21:44.150416: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1720] Found device 1 with properties:  pciBusID: 0000:07:00.0 name: GeForce GTX 780M computeCapability: 3.0 coreClock: 0.797GHz coreCount: 8 deviceMemorySize: 3.94GiB deviceMemoryBandwidth: 149.01GiB/s 2021-01-25 09:21:44.150466: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcudart.so.10.1 2021-01-25 09:21:44.150508: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcublas.so.10 2021-01-25 09:21:44.150535: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcublasLt.so.10 2021-01-25 09:21:44.150559: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcufft.so.10 2021-01-25 09:21:44.150585: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcurand.so.10 2021-01-25 09:21:44.150611: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcusolver.so.10 2021-01-25 09:21:44.150638: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcusparse.so.10 2021-01-25 09:21:44.150664: I tensorflow/stream_executor/platform/default/dso_loader.cc:49] Successfully opened dynamic library libcudnn.so.8 2021-01-25 09:21:44.150773: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:941] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero 2021-01-25 09:21:44.152515: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:941] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero 2021-01-25 09:21:44.154623: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:941] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero 2021-01-25 09:21:44.156318: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1816] Ignoring visible gpu device (device: 0, name: GeForce GTX 780M, pci bus id: 0000:01:00.0, compute capability: 3.0) with Cuda compute capability 3.0. The minimum required Cuda capability is 3.5. 2021-01-25 09:21:44.156445: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:941] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero 2021-01-25 09:21:44.158101: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1816] Ignoring visible gpu device (device: 1, name: GeForce GTX 780M, pci bus id: 0000:07:00.0, compute capability: 3.0) with Cuda compute capability 3.0. The minimum required Cuda capability is 3.5. 2021-01-25 09:21:44.158151: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1261] Device interconnect StreamExecutor with strength 1 edge matrix: 2021-01-25 09:21:44.158161: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1267]      0 1  2021-01-25 09:21:44.158167: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1280] 0:   N Y  2021-01-25 09:21:44.158173: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1280] 1:   Y N  False
So I did not know what do now.

@eagle-hub
Please share simple indented stand alone code to replicate the issue or if possible share a colab git with the error reported.

@eagle-hub
Please share simple indented stand alone code to replicate the issue or if possible share a colab git with the error reported.

`
import tensorflow as tf
tf.test.is_gpu_available()
tf.config.list_physical_devices('GPU')
tf.test.is_gpu_available(True,3.0)
tf.test.is_gpu_available()
`
I used jupyter to run code and also from terminal. I will attach 2 files for both running.
Please note that I'm running the code on my machine as I mentioned in the first comment.

`
import tensorflow as tf
tf.test.is_gpu_available()
tf.config.list_physical_devices('GPU')
tf.test.is_gpu_available(True,3.0)
tf.test.is_gpu_available()
`
I used jupyter to run code and also from terminal. I will attach 2 files for both running.
Please note that I'm running the code on my machine as I mentioned in the first comment.

@tensorflowbutler @Saduf2019
Any advice on how to build from source and support GeForce GTX 780M compute capability: 3.0

@tensorflowbutler @Saduf2019
Any advice on how to build from source and support GeForce GTX 780M compute capability: 3.0

Unfortunately we don't have an exhaustive guide for TF build with cuda compute 3.0
I think reaching out to community can be your best bet here.
I found one article that may help in your case.

I have the same problem but I tried with version 2.3.2 since it supports cuda 10.1 that works with compute capability 3.0.
In my case I followed [https://github.com/tensorflow/tensorflow/issues/27840] to turn off XLA and edited .tf_configure.bazelrc, after running ./configure script, and appended  --define with_xla_support=false to build:xla line.
After a successful compilation I run the example:
python3 -c "import tensorflow as tf; print(\"Num GPUs Available: \", len(tf.config.list_physical_devices('GPU')))"

and the output:
...
... I tensorflow/core/common_runtime/gpu/gpu_device.cc:1812] Ignoring visible gpu device (device: 0, name: Quadro K600, pci bus id: 0
000:04:00.0, compute capability: 3.0) with Cuda compute capability 3.0. The minimum required Cuda capability is 3.5.
Num GPUs Available:  0

I gave it a second try but this time I removed build:xla line and added
build --define with_xla_support=false
build --action_env=TF_ENABLE_XLA=0

unfortunately with the same result.
Is there a proper way to disable XLA in v2.x?
is build --define=with_xla_support=false the same as build --define with_xla_support=false?
Also notice that ./configure script explicitly says that compute capability 3.0 is not supported
Please note that each additional compute capability significantly increases your build time and binary size,
and that TensorFlow only supports compute capabilities >= 3.5 [Default is: 3.0]: 3.0

WARNING: XLA does not support CUDA compute capabilities lower than 3.5. Disable XLA when running on older GPUs.

does this mean that v2.x can never run on a gpu with compute capability 3.0?
@ymodak answer does not help because the article is for v1.x and v1.x is known to work with cc 3.0

I solved the problem in my case using tensorflow 2.1.3 + cuda 10.1 + cudnn 7.6.5 + bazel 0.27.2:
...
2021-04-26 19:28:16.651806: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:981] successful NUMA node read from SysFS had negative value (-1), but the
re must be at least one NUMA node, so returning NUMA node zero
2021-04-26 19:28:16.652966: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1697] Adding visible gpu devices: 0
Num GPUs Available:  1

maybe versions > 2.1 work with bazel 0.27 or it is related to the additional option of v2.1 in ./configure script
Do you wish to build TensorFlow with XLA JIT support? [Y/n]: n
No XLA JIT support will be enabled for TensorFlow.

that is not in v2.3

This is to confirm that the following recipe solved this problem in my case with tensorflow 2.3.2 + cuda 10.1 + cudnn 7.6.5 + bazel 3.10:

disable XLA (not supported with compute capability 3.0) by removing the line build:xla in .tf_configure.bazelrc and adding:

build --define=with_xla_support=false
build --action_env TF_ENABLE_XLA=0

for tensorflow > 2.2 the code changed and it is necessary to define TF_EXTRA_CUDA_CAPABILITIES in bazel command line:

--copt=-DTF_EXTRA_CUDA_CAPABILITIES=3.0

Which version of gcc are you using? Another question: is it bazel 3.1.0? I can't find bazel 3.10

This is to confirm that the following recipe solved this problem in my case with tensorflow 2.3.2 + cuda 10.1 + cudnn 7.6.5 + bazel 3.10:

disable XLA (not supported with compute capability 3.0) by removing the line build:xla in .tf_configure.bazelrc and adding:

build --define=with_xla_support=false
build --action_env TF_ENABLE_XLA=0

for tensorflow > 2.2 the code changed and it is necessary to define TF_EXTRA_CUDA_CAPABILITIES in bazel command line:

--copt=-DTF_EXTRA_CUDA_CAPABILITIES=3.0

Which version of gcc are you using? Another question: is it bazel 3.1.0? I can't find bazel 3.10

gcc (Debian 8.3.0-6) 8.3.0
yes it is a typo, I used bazel 3.1.0

Thanks for the prompt reply!

Which version of gcc are you using? Another question: is it bazel 3.1.0? I can't find bazel 3.10

gcc (Debian 8.3.0-6) 8.3.0
yes it is a typo, I used bazel 3.1.0


Maps using URLs instead of local storage

The maps should be able to load over HTTP instead of storing on the file system.
IE...
ad_agency_at {   type = "content";   filter = "oneline";   regexp = true;   one_shot = true;   map = "https://raw.githubusercontent.com/martinschaible/rspamd-rules/main/maps.d/_ad_agency_at.map";   score = 20.0;   symbol = "AD_AGENCY_AT";   description = "Agencies sending Advertising, Austria"; }
Then Rspamd can automatically update the maps once the etag changes.

Very good idea! I will do this soon. Thanks for the advice.


Adding Spanish descriptions for CT

I also increased the character limit for Spanish descriptions in the schema test from 160 to 200. On Slack, we had discussed increasing to 180 based on the idea that on average Spanish is 20% longer than English. However, even 200 still doesn't cover a few of the longer descriptions. I left those out for now but they should be added back when we have a better way to display longer descriptions.
Here's the spreadsheet with the descriptions for reference.

Chatted with Monique offline, and we decided that it's worse for there to be no Spanish description for a few of these than for a few of them to be a bit longer, so I increased the limit and added them back. Will bring it up in the standup later to see what action we want to take about it.


What is AI content added

Great Work ✅
If you have great ideas you are welcome to open up more issues and work on them.
Have a great day :)


old release processes still present after rollback

Here's the scenario and how I got here:
><> deis releases
=== rb Releases
v2      2016-03-18T16:07:12UTC  bacongobbler deployed e159088
v1      2016-03-18T16:04:44UTC  bacongobbler created initial release
><> deis rollback v1
Rolling back to v1... Error:
400 Bad Request
detail: No build associated with this release

><> deis releases
=== rb Releases
v3      2016-03-18T16:12:14UTC  bacongobbler rolled back to v1
v2      2016-03-18T16:07:12UTC  bacongobbler deployed e159088
v1      2016-03-18T16:04:44UTC  bacongobbler created initial release
><> deis rollback v2
Rolling back to v2... done, v4
><> deis releases
=== rb Releases
v4      2016-03-18T16:13:08UTC  bacongobbler rolled back to v2
v3      2016-03-18T16:12:14UTC  bacongobbler rolled back to v1
v2      2016-03-18T16:07:12UTC  bacongobbler deployed e159088
v1      2016-03-18T16:04:44UTC  bacongobbler created initial release
><> deis ps
=== rb Processes
><> deis scale web=2
Scaling processes... but first, coffee!
done in 9s
=== rb Processes
--- web:
rb-v4-web-7m9yf up (v4)
rb-v4-web-fo8g5 up (v4)
><> kubectl --namespace=rb get po
NAME              READY     STATUS    RESTARTS   AGE
rb-v2-web-85cnj   1/1       Running   0          7m
rb-v4-web-7m9yf   1/1       Running   0          1m
rb-v4-web-fo8g5   1/1       Running   0          1m

This would be fine except that the v2 app is still in the router mesh, which is something we fixed in v1 with deis/publisher only publishing the latest release deployed to the cluster:
><> curl rb.<IP_ADDRESS>.xip.io
Powered by Deis
Running on container ID rb-v2-web-85cnj
><> curl rb.<IP_ADDRESS>.xip.io
Powered by Deis
Running on container ID rb-v4-web-fo8g5
><> curl rb.<IP_ADDRESS>.xip.io
Powered by Deis
Running on container ID rb-v4-web-7m9yf

Also note that the structure object is correct:
><> curl -s deis.<IP_ADDRESS>.xip.io/v2/apps/rb -H 'Authorization: token c3250f7181a4eb6dbac18022c17881ddae0659e0' | jpretty
{
    "created": "2016-03-18T16:04:44UTC",
    "id": "rb",
    "owner": "bacongobbler",
    "structure": {
        "web": 2
    },
    "updated": "2016-03-18T16:24:23UTC",
    "uuid": "7298ef12-125d-4287-a42d-bf55926b957b"
}

promoting to beta showstopper after conversing with @mboersma about this. Normally we could punt this, but because there's no way to reasonably recover from this unless you have kubernetes administration access, this is a showstopper.

For future reference, here's some logs with a new deployment to reproduce this:
><> deis rollback v1
Rolling back to v1... Error:
400 Bad Request
detail: No build associated with this release

><> deis rollback v2
Rolling back to v2... done, v4
><> deis scale web=1
Scaling processes... but first, coffee!
done in 0s
=== rc Processes
><> deis scale web=2
Scaling processes... but first, coffee!
done in 7s
=== rc Processes
--- web:
rc-v4-web-4jt0m up (v4)
rc-v4-web-uh0hr up (v4)

Notice that deis scale web=1 worked, which seems to indicate the scheduler code recognized that the v2 pod was recognized. Controller logs are as follows:
INFO rc: release rc-v3 created
INFO [rc]: bacongobbler rolled back to v1
INFO rc: No build associated with this release to publish
INFO [rc]: No build associated with this release to publish
DEBUG create rc
<IP_ADDRESS> "POST /v2/apps/rc/releases/rollback/ HTTP/1.1" 400 50 "Deis Client v2.0.0-dev"
INFO rc: release rc-v4 created
INFO [rc]: bacongobbler rolled back to v2
<IP_ADDRESS> "POST /v2/apps/rc/releases/rollback/ HTTP/1.1" 201 13 "Deis Client v2.0.0-dev"
<IP_ADDRESS> "GET /healthz HTTP/1.1" 200 2 "Go 1.1 package http"
<IP_ADDRESS> "GET /healthz HTTP/1.1" 200 2 "Go 1.1 package http"
DEBUG create rc
<IP_ADDRESS> "POST /v2/apps/rc/scale/ HTTP/1.1" 204 - "Deis Client v2.0.0-dev"
<IP_ADDRESS> "GET /v2/apps/rc/pods/?page_size=100 HTTP/1.1" 200 24 "Deis Client v2.0.0-dev"
DEBUG create rc
DEBUG create rc
DEBUG scale rc-v4-web, img http://<IP_ADDRESS>:9000/git/home/rc:git-e1590887/push/slug.tgz, params {'cpu': {}, 'tags': {}, 'envs': {}, 'app_type': 'web', 'build_type': 'buildpack', 'healthcheck': {}, 'num': 2, 'version': 'v4', 'memory': {}}, cmd "start web"
DEBUG scaling RC rc-v4-web in Namespace rc from 0 to 2 replicas
DEBUG waiting for RC rc-v4-web to get a newer resource version than 1004 (30s timeout)
DEBUG RC rc-v4-web has a new resource version 1005
DEBUG waiting for 2 pods in rc namespace to be in services (120s timeout)
<IP_ADDRESS> "GET /healthz HTTP/1.1" 200 2 "Go 1.1 package http"
<IP_ADDRESS> "GET /healthz HTTP/1.1" 200 2 "Go 1.1 package http"
DEBUG 2 out of 2 pods in namespace rc are in service
INFO rc: bacongobbler scaled pods web=2
INFO [rc]: bacongobbler scaled pods web=2
<IP_ADDRESS> "POST /v2/apps/rc/scale/ HTTP/1.1" 204 - "Deis Client v2.0.0-dev"
<IP_ADDRESS> "GET /v2/apps/rc/pods/?page_size=100 HTTP/1.1" 200 225 "Deis Client v2.0.0-dev"

One thing that I started wondering was if the v4 rollback was in fact copying from v3 instead of v2.
@bacongobbler noted that only create is called and not deploy. The code around deploy is generally guarded with release.build is not None so it would somewhat checkout that we are dealing with a dud release.

Any idea how to reproduce this now?
$ deis releases -a indoor-offshoot
=== indoor-offshoot Releases
v2	2016-04-12T01:40:30UTC	helgi deployed deis/example-go
v1	2016-04-12T01:40:20UTC	helgi created initial release

$ deis rollback v1 -a indoor-offshoot
Rolling back to v1... Error:
400 Bad Request
detail: No build associated with this release

$ deis releases -a indoor-offshoot
=== indoor-offshoot Releases
v2	2016-04-12T01:40:30UTC	helgi deployed deis/example-go
v1	2016-04-12T01:40:20UTC	helgi created initial release

A different fix may have made it harder to get to this point

ah hang on a minute, the RC and release is no longer created but it is shows up in the logs that the steps started and now I can't run a config:set on the app... sounds like a diff problem than described here but problem none the less


Add GET /docmap_for/doi

WIP.
Description
Adds the method GET /docmap_for/doi. Because it is a GET, this should be cacheable, and at the moment, this is reliant on there existing such a docmap already.
Related Issues
Contributes to
#24
#13
Checklist

[x] I have tested these changes locally and they work as expected.
[x] I have added or updated tests to cover any new functionality or bug fixes.
[ ] I have updated the documentation to reflect any changes or additions to the project.
[x] I have followed the project's code of conduct and conventions for commit messages.

Additional Information
Open questions:

[ ] What error codes are best? How do we want to distinguish between "nothing found" and "bad data found"?
[ ] what is the result when there are multiple docmaps found that match?

okay, as of EOD today the only outstanding issue is handling cases of multiple matches in the database. For now will go with implementing a most-recent strategy that pulls all triples that apply to any matches into a single transactional sub-graph, then we do operations in-memory with N3 to get "smart" results.


TIMsort.py

https://github.com/swenson/sort/commit/63ac6d4e7a7f6b0019add8c9ef823fe8162e6bc2

time sort

Hi. Thanks for your contribution, but I don't think this is the right place for it. This is a C sorting library.
Also, I think this is more of a merge sort than a tim sort, since it doesn't take advantage of natural runs or unbalanced merges.


Package acceptance validation error: Apps converted with the Desktop Bridge and that require the .NET Native framework must be pre-compiled by the .NET Native tool chain.

Trying to follow guide here (Migrate step) and using this sample, I'm getting following error when uploading to the Store

Package acceptance validation error: Apps converted with the Desktop Bridge and that require the .NET Native framework must be pre-compiled by the .NET Native tool chain.

Is there a way to upload to Store?

Yes, be sure to follow the steps precisely for creating an acceptable appxupload file manually. Then it will be accepted by the Store.
“To create the new appxupload file, we will create a new zip archive to include the generated appxsym and appxbundle from the _Test folder. Create a new zip file that contains the appxsym and appxbundle files, and then rename the extension to appxupload”
Thanks,
Stefan Wick
From: Sergey Galich<EMAIL_ADDRESS>Sent: Thursday, February 23, 2017 8:28 PM
To: Microsoft/DesktopBridgeToUWP-Samples<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [Microsoft/DesktopBridgeToUWP-Samples] Package acceptance validation error: Apps converted with the Desktop Bridge and that require the .NET Native framework must be pre-compiled by the .NET Native tool chain. (#27)
Trying to follow guide here (Migrate step)https://docs.microsoft.com/en-us/windows/uwp/porting/desktop-to-uwp-packaging-dot-net and using this samplehttps://github.com/Microsoft/DesktopBridgeToUWP-Samples/tree/master/Samples/AppServiceBridgeSample, I'm getting following error when uploading to the Store
Package acceptance validation error: Apps converted with the Desktop Bridge and that require the .NET Native framework must be pre-compiled by the .NET Native tool chain.
Is there a way to upload to Store?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/DesktopBridgeToUWP-Samples/issues/27, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ASRavKqSH1NIyyyc8GYc15Qfkd-MoKMPks5rflxcgaJpZM4MK0My.

@wickste aw, i totally missed that bit, let me give it a try, thank you

confirmed, manual repackaging helps to get sample uploaded to the Store, thank you Stefan!
now it's time to try with the real app...

perfect, while I hope tooling will get better to avoid all these manual steps, it works and 1Password lives 👍

Glad to hear it worked – congratulations!
Tooling improvements are high on our list for a future update.
Sent from my Windows 10 phone
From: Sergey<EMAIL_ADDRESS>Sent: Thursday, February 23, 2017 8:53 PM
To<EMAIL_ADDRESS>Cc: Stefan<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [Microsoft/DesktopBridgeToUWP-Samples] Package acceptance validation error: Apps converted with the Desktop Bridge and that require the .NET Native framework must be pre-compiled by the .NET Native tool chain. (#27)
perfect, while I hope tooling will get better to avoid all these manual steps, it works and 1Password lives 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/DesktopBridgeToUWP-Samples/issues/27#issuecomment-282203962, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ASRavEpMTljJvpyXQrAKVjkMVsGUdYSYks5rfmI6gaJpZM4MK0My.


does it supoort http proxy?

if client side network can only access network by a http proxy, how to use this tool? is it supported?

This tool use tcp protocol which more basic than http, thus a http proxy won't help. If you must use a proxy you may setup a tcp proxy in your env.


Remove Internal Types from Exports

MeasurementMetadata and Identifier are exported by the mlte.measurement module. These are now an implementation detail that should be hidden.

Note that there is still a use case when they are used, though I think we should try to remove that case as well. In the demo/docs, there is an example of when an Image is created manually to record some evidence, which uses both classes. If we want to handle this as an ExternalMeasurement as well, even if it is just creating an image, then removing those exports would be fine, while updating these examples creating the image to use ExternalMeasurement instead of manually creating the result class.

Ok, so as discussed today, we'll try to add a method to ExternalMeasurements, something like .collect(), to be used with this type of external evidence data. This would allow us to continue removing the export for those internal items.


extend subdomain delete capability to domain admins

Description

Contribution Checklist:

[x] The pull request does not introduce any breaking changes
[x] I have read the contribution guidelines.
[x] Create an issue and link to the pull request.

Attach Screenshots (Optional)

addresses issue #2373


multiple redirects.. I think I know why.. but confirming

so, over the weekend, i was able to get this (sorta) working with Okta.. I'm now getting the callback from okta..but it doesn't look like the JWT is set properly on the domain, so it goes into a redirect loop (with okta).  I think it's because I don't have an actual live domain for the lasso service so it doesn't set the JWT correctly (but I'm not sure).  Here is the output from lasso:
ERRO[0015] no jwt found in request
INFO[0015] | 401 |  120.372µs /validate      avgLatency=66.802768ms host="<IP_ADDRESS>:9090" ipPort="<IP_ADDRESS>:49242" latency=" 120.372µs" method=GET path=/validate referer= request=7 statusCode=401
INFO[0015] | 302 |  242.791µs /login         avgLatency=58.482771ms host=ec2-18-234-210-151.compute-1.amazonaws.com ipPort="<IP_ADDRESS>:49244" latency=" 242.791µs" method=GET path=/login referer= request=8 statusCode=302
INFO[0015] OpenID userinfo body:  {"sub":"00u9cpj0gaOBYVZ4v0h7","email":"chris.mollis@gmail.com","email_verified":true}
INFO[0015] | 302 | 208.432955ms /auth        avgLatency=75.143902ms host=ec2-18-234-210-151.compute-1.amazonaws.com ipPort="<IP_ADDRESS>:49246" latency=208.432955ms method=GET path=/auth referer= request=9 statusCode=302
ERRO[0015] no jwt found in request
INFO[0015] | 401 |   71.296µs /validate      avgLatency=67.636642ms host="<IP_ADDRESS>:9090" ipPort="<IP_ADDRESS>:49248" latency="  71.296µs" method=GET path=/validate referer= request=10 statusCode=401
INFO[0015] | 302 |  256.479µs /login         avgLatency=61.511173ms host=ec2-18-234-210-151.compute-1.amazonaws.com ipPort="<IP_ADDRESS>:49250" latency=" 256.479µs" method=GET path=/login referer= request=11 statusCode=302
INFO[0016] OpenID userinfo body:  {"sub":"00u9cpj0gaOBYVZ4v0h7","email":"chris.mollis@gmail.com","email_verified":true}
Initially nginx redirects to /auth (lasso), which redirects to Okta.  I log into okta, and then it redirects back to /auth (lasso), which appears to extract my user info (I had thought from the JWT), but errors with no JWT found, which redirects to /validate back to /login back to Okta, which redirects back to /auth.. yada yada.
If you remember, I'm somewhat faking the 'lasso.mydomain.com' with a proxy set up on a different port, but the same domain.  here:
server {
listen 81 ;
server_name ec2-18-234-210-151.compute-1.amazonaws.com;
#location / {
root   html;
#        index  index.html index.htm;
 #   }

#ssl_certificate /etc/letsencrypt/live/login.avocado.lol/fullchain.pem;
#ssl_certificate_key /etc/letsencrypt/live/login.avocado.lol/privkey.pem;
Proxy to your Lasso instance
location / {
proxy_set_header  Host  ec2-18-234-210-151.compute-1.amazonaws.com;
proxy_set_header  X-Forwarded-Proto http;
proxy_pass        http://<IP_ADDRESS>:9090;
}
}
instead of:
server {
listen 80 default_server;
server_name lasso.yourdomain.com;
location / {
proxy_set_header Host lasso.yourdomain.com;
proxy_pass http://<IP_ADDRESS>:9090;
}
}
there's a note about setting the setting the proxy header or the jwt isn't set correctly.. (this JWT is probably managed by lasso itself).

I'm going to close this and move us back to #36


Remove obsolete ObfuscatorTask

It has been obsolete for quit a while. Replace with YGuardTask. See https://github.com/yWorks/yguard/blob/0dad808605b695e696c747d92ff1324370a7ae58/src/main/java/com/yworks/yguard/ObfuscatorTask.java#L109.
Go to 2.10 release (because this is a breaking change for some users) afterwards.

After talking to @thomasbehr we agreed that this should be postponed / handled differently.


Do not merge: June23 demo differences from dynamics

With dynamics merged into develop, this is now redundant.


Add Reset method to SKPaint and add constructor to SKMatrix

It's quite a bit faster to reuse SKPaint objects (due to the all pinvoke, extra bookkeeping for the references, etc.). But without having an reset method (https://skia.org/user/api/SkPaint_Reference#SkPaint_reset) it is hard to actually reuse objects, as it will be quite slow to reset properties one by one.
Native changes in: https://github.com/mono/skia/pull/59
Additionally I added a constructor to SKMatrix as it leads to better codegen, especially when inlining (like Avalonia conversion methods).
Benchmark code: https://gist.github.com/MarchingCube/3bcfbc8894a93b92cb2997b650a05508
Benchmark results for SKPaint reuse:

BenchmarkDotNet=v0.11.4, OS=Windows 10.0.17763.316 (1809/October2018Update/Redstone5)
AMD Ryzen 7 2700X, 1 CPU, 16 logical and 8 physical cores
.NET Core SDK=3.0.100-preview-009812
  [Host]     : .NET Core 2.0.9 (CoreCLR 4.6.26614.01, CoreFX 4.6.26614.01), 64bit RyuJIT
  DefaultJob : .NET Core 2.0.9 (CoreCLR 4.6.26614.01, CoreFX 4.6.26614.01), 64bit RyuJIT

Method
Mean
Error
StdDev
Ratio

AllocatePaint
697.9 us
15.9502 us
13.3191 us
1.00

ResetPaint
211.7 us
0.5672 us
0.5028 us
0.30

Results for SKMatrix change:

Method
Mean
Error
StdDev
Ratio

MatrixConstructor
7.926 ns
0.0267 ns
0.0250 ns
0.79

MatrixPropertyInitialization
10.050 ns
0.1238 ns
0.1158 ns
1.00

OPS Build status updates of commit 5b9e08d:
:clock10: Preparing: average preparing time is 1 min(s) 58 sec(s)

 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.:x: MarchingCube sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

OPS Build status updates of commit 5b9e08d:
:clock10: Incremental building: average incremental building time is 19 sec(s)

OPS Build status updates of commit 5b9e08d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.gitmodules
:white_check_mark:Succeeded

binding/Binding/SKMatrix.cs
:white_check_mark:Succeeded

binding/Binding/SKPaint.cs
:white_check_mark:Succeeded

binding/Binding/SkiaApi.cs
:white_check_mark:Succeeded

externals/skia
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

I merged the native PR, so you should be able to switch the submodule back to the mono org.

Thanks for this PR, and the benchmarks. It is a major saving - especially if there are several properties used.

OPS Build status updates of commit b51ac90:
:clock10: Preparing: average preparing time is 1 min(s) 58 sec(s)

OPS Build status updates of commit b51ac90:
:clock10: Incremental building: average incremental building time is 19 sec(s)

OPS Build status updates of commit b51ac90:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

binding/Binding/SKMatrix.cs
:white_check_mark:Succeeded

binding/Binding/SKPaint.cs
:white_check_mark:Succeeded

binding/Binding/SkiaApi.cs
:white_check_mark:Succeeded

externals/skia
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Bot joining problem

Please tick all applicable boxes.

[x] I am using Python 3.5 or higher (run python --version on the command line)
[X] I have followed the official guides to install the bot for my system
[X] I have updated my dependencies to the latest version using the appropriate update script

Which version are you using?

[X] The latest master version (1.9.7)
[ ] The latest review version

What type of issue are you creating?

[X] Bug
[ ] Feature request
[ ] Question

Description of issue
I'm using CentOS 7.4 and Python 3.5.4
Bot appears in voice server but then that error count starts. I have tried changing server region, doesn't work.
http://prntscr.com/jww7yy
Also when I directly download that same MusicBot folder to my pc and then run "run.py" it works.
Steps to reproduce

Log file
Please attach your MusicBot log file (located at logs/musicbot.log) to this issue. You can do so by dragging and dropping the file here. If you do not include your log file, you WILL be asked to provide one.
musicbot.log

Still same problem

You said you downloaded it and running it works, that indicates to me that the issue has nothing to do with the bot, but is an issue with your hosts connection to discord's voice servers.

The problem was with Hetzner hosting. They are still blocking Hetzner ips, but only in voice servers, text servers work just fine. Also there is no workaround.


Tickets System 2

Description
I've added tickets to the main bot, this is okay for now if you want I'll figure out how to get a transcript of chat in a txt file.
I swear this is the last PR.
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Other (Please specify):

Has this been tested:

[x] Yes
[ ] No
[ ] Not needed

Checklist:
If any are false, you may still submit, but may be asked to make changes.

[x] My code follows the style guidelines of this project

[] I have changed/added the help doc at the beginning of any commands changed/added.

[x] I have performed a self-review of my own code

[x] I have commented my code, particularly in hard-to-understand areas

[x] My changed files do not include any files regularly changed in saves (something where my bot would interfere with this).

[x] My changes generate no new warnings

[x] I have tested and concluded that my changes are effective or that my feature works - If you have not tested, dont check this so I know to review it more. You may still make a PR for untested code.

[x] My code does not cause issues with existing code

[] Any dependent changes have been merged and published in downstream modules

[ ] Some of these options did not apply to my change (Please specify which):

So @BobDotCom are you going to take the pull request or will this just remain as a folder on my PC :(
Please either close it or take it, please get back ASAP. I'm ready to add transcripts of chat, or any other bug fixes / features.
Also I haven't coded dpy in like 2 weeks so I might be rusty.

This does show some potential so I will merge it into the tickets branch, but it lacks some of the features that we currently have on the tickets bot, such as ticket logging and such. I can work on this when I have time, or @NightZan999 can if you want (of course anyone else would be welcome to contribute as well).

I have ticket logging, but not the content (see code), but of course the content of the ticket for higher mods to review their moderator's support team for staff checks and promotes / demotes is understandable.
Would you prefer a txt file the same way that Tickets does it?


[BUG] Bug in json_to_roidb.py and fixed

Describe the bug
A clear and concise description of what the bug is.
code in line 12-13
 parser.parse_args()
 return parser.json

are supossed to be change into
arg = parser.parse_args()
return arg.json

or you wil get

AttributeError: 'ArgumentParser' object has no attribute 'json'

Thanks for bringing this up. Would you mind make a PR?

Thanks for bringing this up. Would you mind make a PR?

Ok, I will do it later..


Documentation for hub configurator not working to change docker image

I was following these docs:
https://github.com/CryoInTheCloud/hub-image#updating-the-cryocloud-jupyterhub-to-use-a-new-image
But no matter what the configurator is set to (I tried the latest http://quay.io/cryointhecloud/cryo-hub-image:b25e78527b0c) hub will pull the image set here:
https://github.com/2i2c-org/infrastructure/blob/0f265f493b8280f6432cc275a431229774c848e6/config/clusters/nasa-cryo/common.values.yaml#L140
possibly related to using the profile list for different node types (https://github.com/2i2c-org/infrastructure/issues/3135#issuecomment-1764328026)

@scottyhq yes, unfortunately when the image is overriden per-profile, it doesn't pull from the configurator anymore. https://github.com/yuvipanda/z2jh-configurator is the eventual answer to this, but it's not been deployed anywhere right now.

We need to update the readme (https://github.com/CryoInTheCloud/hub-image?tab=readme-ov-file#updating-the-cryocloud-jupyterhub-to-use-a-new-image) to use the Other option to test the new image.

This has been added. Let me know if you have any further issues or updates.


custom webpack config doesn't work with arrays, config factory functions, or promises

Expected Behaviour
This library now supports Webpack config files that export plain config objects. But Webpack itself supports other configuration types, some of which are necessary for some basic functionality.

To build config based on runtime conditions, the Webpack config file should be able to export a factory function which receives an env object and returns a plain config object.
To build config that requires some data from an async operation, the Webpack config file should be able to export a Promise which resolves to a plain config object.
It should also be possible to export an async factory function that receives an env object and returns a Promise for a plain config object.
In order to compile multiple bundles in parallel, you should be able to export an array of any of the above types.

Actual Behaviour
Only the plain object type of config is supported in webpack-config.js.
Reproduce Scenario (including but not limited to)
Steps to Reproduce

Put the following in webpack-config.js:module.exports = { mode: "development" };

Run aio app:deploy -v.
Observe that the app is built in dev mode.
Change webpack-config.js to the following:module.exports = async () => ({ mode: "development" });

Run aio app:deploy -v again.
Observe that the app was not built in dev mode. This legal Webpack config type was not honored.

Platform and Version
{
  '@adobe/aio-cli': '5.0.1',
  npm: '7.5.2',
  node: '12.19.1',
  '@adobe/aio-lib-runtime': '1.1.0'
}

Hi @zetlen and thanks for opening this issue!
Let me loop-in @purplecabbage and @Himavanth.
Quick question: is this a webpack 5 feature or was already available in webpack 4?
We do not support webpack 5 yet. In that case, the current approach for now would be to write your custom build script, as documented at https://github.com/AdobeDocs/project-firefly/blob/master/guides/app-hooks.md.

Thanks for the confirmation @zetlen !
@purplecabbage @Himavanth what would be the possible impacts (if any) if we'd support these additional ways of providing a webpack config?


Added http_socket param to client config #776

Pull Request Checklist
Added parameter http_socket to client config.
Description
Added parameter to main class, and then reference to it in sunse::client is passwd to the sensu_client_config type used the new json write approach.
Related Issue

Fixes #776
Motivation and Context

How Has This Been Tested?
Tested os samples vms and spec.
General

[x] Update README.md with any necessary configuration snippets

[x] New parameters are documented

[x] New parameters have tests

[x] Tests pass - bundle exec rake validate lint spec

Thanks! Released in v2.34.0


ion-content: tabbar height miscalculation when using multiple ion-tabs

Ionic version:  (check one with "x")
[ ] 1.x
[x] 2.x
I'm submitting a ...  (check one with "x")
[x] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:
When using 2 ion-tabs on a page (with them having differing heights), the margin calculation for ion-content is incorrect.
Expected behavior:
The ion-content should have the correct margin set, based on the height of the closest ion-tabs.
Steps to reproduce:

Create a starter tabs app
Add more tabs with tabsPlacement="top"
Make the bottom tabs larger than the top tabs
The ion-content margins are incorrect for the top tabs

Related code:
See repo: https://github.com/metz0r/ionic-issue-myApp-1
PR on the way
Other information:
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Cordova CLI: 6.5.0
Ionic Framework Version: 2.3.0
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.1.4
ios-deploy version: 1.9.0
ios-sim version: 5.0.13
OS: macOS Sierra
Node Version: v6.9.2
Xcode version: Xcode 8.3 Build version 8E162

Here is a workaround:
home { // your selector name
  ion-tabs .tabbar {
    max-height: 30px; //set pixels here
  }
}

tab-bar really a problem in ionic


DS-42: typography setup

I think it's worth adding the font loading to the app as well

I'm not sure why frontend/apps/core-app/src/react-app-env.d.ts and the tsconfig.json changes has been made. They seem to have been caused by react-scripts, but the app doesn't use react-scripts. Can you revert the changes and try again?

I'm not sure why frontend/apps/core-app/src/react-app-env.d.ts and the tsconfig.json changes has been made. They seem to have been caused by react-scripts, but the app doesn't use react-scripts. Can you revert the changes and try again?

tried again after reverting these changes, seems to have worked, no idea what I did there

/lgtm

/lgtm


reduce logging and update hard coded seed nodes

Issue being fixed or feature implemented

What was done?

How Has This Been Tested?

Breaking Changes

Checklist:

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation

For repository code-owners and collaborators only

[ ] I have assigned this pull request to a milestone

Codecov Report

Merging #173 (7e48d19) into master (7a9471c) will decrease coverage by 0.04%.
The diff coverage is 25.00%.

@@             Coverage Diff              @@
##             master     #173      +/-   ##
============================================
- Coverage     45.22%   45.18%   -0.05%
  Complexity     5329     5329
============================================
  Files           379      379
  Lines         41410    41421      +11
  Branches       5666     5671       +5
============================================
- Hits          18729    18715      -14
- Misses        20733    20756      +23
- Partials       1948     1950       +2

Impacted Files
Coverage Δ

...c/main/java/org/bitcoinj/params/MainNetParams.java
80.83% <ø> (ø)

...ava/org/bitcoinj/quorums/SimplifiedQuorumList.java
51.40% <0.00%> (-1.83%)
:arrow_down:

...va/org/bitcoinj/evolution/QuorumRotationState.java
61.37% <19.04%> (-2.23%)
:arrow_down:

core/src/main/java/org/bitcoinj/core/Context.java
54.54% <50.00%> (-0.09%)
:arrow_down:

...main/java/org/bitcoinj/core/NetworkParameters.java
70.52% <100.00%> (ø)

...e/src/main/java/org/bitcoinj/core/BloomFilter.java
74.65% <0.00%> (-2.06%)
:arrow_down:

...inj/evolution/SimplifiedMasternodeListManager.java
38.49% <0.00%> (-1.51%)
:arrow_down:

.../java/org/bitcoinj/quorums/QuorumRotationInfo.java
61.48% <0.00%> (-0.75%)
:arrow_down:

... and 3 more

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


feat(helper/html): Support Promise<string | HtmlEscapedString> in html tagged template literals

This PR allows Promise<string | HtmlEscapedString> to be passed to the html tagFunction variables as well.
Fixes: #1812
Author should do the followings, if applicable

[x] Add tests
[x] Run tests
[x] yarn denoify to generate files for Deno

@yusukebe @Code-Hex
Would you please review?

Great! LGTM!


did_disconnect

I have another issue working the basic example:
I can see the message "Connected to:  some numbers"
But when I disconnect the device for example turn it off de device or stop my belututhfrom my computer, then I except to see the message ""Disconnected from: ", some numbers" , but it still scanning.
Could you please check it and let me know if I need to change some definitions?

Hi, does this still occur in the latest version? If so, please provide information on what system you are using and what version of AidlabSDK?


Incorrect diff with rust analyzer

Original code:
pub fn foo() -> i32 {
    let x = 1;
    x + x
}

Plugin diff:
 pub fn foo() -> i32 {
-    let x = 1;
-    x + x
+    1
+ + x1
 }

Actual result:
pub fn foo() -> i32 {
    1 + 1
}

Plugin diff:
 pub fn foo() -> i32 {
-    let x = 1;
-    x + x
+    consX:i32  x = 1;
+    X + X
 }

Actual result:
pub fn foo() -> i32 {
    const X: i32 = 1;
    X + X
}

actions-preview.nvim works as expected.

Hello,
Which LSP do you use?

Which LSP do you use?

In the title, rust analyzer.

Maybe did you configure your LSP ? I have the default config, and it did not shows the same code actions as yours:

I'm not sure, but the problem also exists in your second screenshot.

It should be good now. Can you test it please ?

There is an extra new line in the diff.

It should now be ok (I hope) as of commit https://github.com/rachartier/tiny-code-action.nvim/commit/8fb55da53c5ed3f5a25ad07916d40f50b0b59172:

There is an extra new line in the diff.

This one is still the same. I'm not sure why you are seeing a different list of available code actions. I'm using rust-analyzer 1 (4afe0d5393 2024-07-21) and cargo 1.80.0 (376290515 2024-07-16).

I see that this plugin uses custom logic to parse the language server outputs. Can we use vim.lsp.util.apply_text_edits instead? See https://github.com/aznhe21/actions-preview.nvim/blob/9f52a01c374318e91337697ebed51c6fae57f8a4/lua/actions-preview/action.lua#L59-L66

apply_text_edits

Unfortunately, LSPs send back textChanges for some actions, which is not supported (I think, I haven't found anything about it) with native LSP in Neovim, so that's why I used custom logic
I really can't have the same actions as yours. I do not understand why, I did install rust-analyzer (2024-07-22) (as 2024-07-21 was not in mason) and cargo 1.80.0 (376290515 2024-07-16). I've tried to get code actions for every characters in the demo file, and I can't have the same as yours... Do you have some specifics configurations? It will made my life easier if I can debug a real example.

Do you have some specifics configurations?

I don't think so. I just use the default configuration from nvim-lspconfig.

Okay I did reproduce the issue.
You probably have rustacean installed. I've modified the code to merge lines if needed, so you shouldn't have the issue anymore.


Unexpected Cancel?

Files are not uploading.
{
"message": ""UnexpectedCancel""
}
Very unhelpful error message

I get the same error

@kkennett @raviabhiram
A couple of starter questions:

Storage Explorer version?
Operating system?
What are you transferring to and from?

Finally, after you see this error message, can you check your azcopy logs directory (/.azcopy) for a log file which has a date created at about the same time you started the transfer? If so can you please share that log file here.

Closing due to lack of response. Please open a new issue if you are still having problems.


Example chapter: Improve status code display and table entry

Small improvement to example chapter by displaying the status name and code instead of only asserting the status code. Displaying the solver status more explicitly gives the reader a better idea of what this is before seeing the table of status codes right after.
Also, a small fix to last row in the table.

Good idea, accepted!


feat: arn validation

The idea is to have validation of AWS ARNs per services.

@deathbywedgie You can checkout the PR if you have time.


Error while downloading data using azcopy

I am using azcopy to download the simulation data used for css training keeps interrupting, ERROR LOG:.
2024-02-22 14:21:28,912 [INFO] [azure_storage] Downloading to temp dir first: /tmp/tmp9mivr0cw
INFO: scanning...
INFO: will not process any empty folders, as the source and/or target does not support full folders
Task 4d0542a2-a3a6-e844-6c3e-d8367fb18524 started
Log file located at /root/.azcopy/4d0542a2-a3a6-e844-6c3e-d8367fb18524.log
19.6 %, 1963 Done, 0 Failed, 8037 Pending, 0 Skipped, 10000 Total (scanning...) , 2-sec Throughput (Mb/s): 81.9549
Failed to execute replication command due to error: unable to start job due to error: unable to list file due to reason Read tcp x.x.x.x:x->X.X.X.X:443: Read: Connection was reset by peer.

Thanks for your interest in our challenge.
If you share the script you ran we'll try to reproduce the issue.
In the meantime, please try calling azcopy command directly:
azcopy copy https://notsofarsa.blob.core.windows.net/css-datasets/v1.5/200hrs/train <destination> --recursive
Note that the 200 hour training set requires about 1TB. Make sure you have enough disk space and a stable connection.

azcopy copy https://notsofarsa.blob.core.windows.net/css-datasets/v1.5/200hrs/train  --recursive

According to your command, I still received the same error.
erro log:
INFO: Scanning...
INFO: Any empty folders will not be processed, because source and/or destination doesn't have full folder support
Job 3ca4fc71-013b-5149-7f2d-6cf0398cb350 has started
Log file is located at: /home/.azcopy/3ca4fc71-013b-5149-7f2d-6cf0398cb350.log
22.6 %, 2296 Done, 0 Failed, 7704 Pending, 0 Skipped, 10000 Total (scanning...), 2-sec Throughput (Mb/s): 79.8252
failed to perform copy command due to error: cannot start job due to error: cannot list files due to reason read tcp x.x.x.x:x-><IP_ADDRESS>:443: read: connection reset by peer.

Thanks for trying again. I wasn't able to reproduce the issue.
Could you share your azcopy version, os version, and the log file at /home/.azcopy/3ca4fc71-013b-5149-7f2d-6cf0398cb350.log?

Thanks for trying again. I wasn't able to reproduce the issue. Could you share your azcopy version, os version, and the log file at /home/.azcopy/3ca4fc71-013b-5149-7f2d-6cf0398cb350.log?

Uploading 3ca4fc71-013b-5149-7f2d-6cf0398cb350.log…

Thanks for trying again. I wasn't able to reproduce the issue. Could you share your azcopy version, os version, and the log file at /home/.azcopy/3ca4fc71-013b-5149-7f2d-6cf0398cb350.log?

3ca4fc71-013b-5149-7f2d-6cf0398cb350.log

Closing since this specific issue seems to be resolved with downloading through Azure Storage Explorer:

Open storage explorer, click "Open connect dialog", then "Blob container or directory".
Select "anonymously (my blob container allows public access)", paste the url and click "connect". e.g. url=https://notsofarsa.blob.core.windows.net/css-datasets/v1.5/200hrs/train.
You can then download the different folders or individual files.


Publish artifact to S3

Motivation
See https://github.com/falcosecurity/falco/blob/master/proposals/20221129-artifacts-distribution.md#http-distribution
Required to use rules package in the Falco cmake.
Feature

Alternatives

Additional context

We need to delete the git tag and release it again to make this work after the fix introduced by #16

will do.

done! https://download.falco.org/?prefix=rules/


Update helm, opencontainers, containerd.

What this PR does / why we need it:
These dependencies include a minor update to the OCI spec to address https://github.com/opencontainers/distribution-spec/security/advisories/GHSA-mc8v-mgrf-8f4m and https://github.com/opencontainers/image-spec/security/advisories/GHSA-77vh-xpmg-72qh
Release note:
NONE

/kind cleanup

/lgtm


Can original manim be supported?

The current extension works for manim CE(community version).
Is it possible to tailor it so that it also works for original manim ver.(gl ver. by Grant)?
I would really appreciate it if the author can do it. XD

Hey IcyChlorine,
I have not previously considered this but if there is such a need.
As I haven't really used Grant's version before, I'll have a look into it and keep the updates posted
in this issue.
Thank you for the suggestion!

Hey IcyChlorine,
Grant's version of manim is a personal project, and he quickly iterates on it.
The community version is a more stable version, and therefore a gallery like this would need to be regularly updated to his latest version. Therefore, I think it's not worth the effort to create such a gallery for his version.


Should be updated to reflect new licensing

Mention of the transition period should be removed
Threasholds should be checked and eventually updated to reflect the new API calls limits described here

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0a7b1f84-92bd-7188-a7d0-359a930525dd
Version Independent ID: dd7d2d01-bdee-b656-ca63-e19305963d28
Content: Limits and configuration - Power Automate
Content Source: articles/limits-and-config.md
Service: power-automate
Sub-service: cloud-flow
GitHub Login: @MSFTMan
Microsoft Alias: deonhe

@Nicokirr Thanks for using Power Automate. There's no longer any mention of the transition period in this document since the most recent updates.


Screenshot is broken when ReShade's screenshot feature used.

When you take a screenshot with the function of FF14, the result will be as you intended, but when you take a screenshot with the function of ReShade, the colors to be superimposed will be reversed. Probably the same thing will happen in other games.

Some games have alpha channel. So, ReShade has clear alpha channel when screenshot taken. No need to fix.


Add typing for bind() which is also needed for demo

Sorry I missed this one in the previous push. We were not working off the newest complete example app.

@basarat again, sorry to bother you

@alexeagle :+1:


Terrible!

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: f5c733e4-5a8c-56a3-ff18-b5e589ac6d91
Version Independent ID: aba35600-3d59-5751-2072-33cc24ec5601
Content: Power Apps portals Studio - Power Apps
Content Source: powerapps-docs/maker/portals/portal-designer-anatomy.md
Service: powerapps
Sub-service: portals
GitHub Login: @neerajnandwana-msft
Microsoft Alias: nenandw

Thanks for your comment, @standenman. This issue has been assigned to the doc author for review.

Thank you for you feedback.
Please provide some more details as to what information you feel is missing or what you don't find helpful?


feat(pageserver): add fast path for sparse keyspace read

Problem
In https://github.com/neondatabase/neon/issues/9441, the tenant has a lot of aux keys spread in multiple aux files. The perf tool shows that a significant amount of time is spent on remove_overlapping_keys. For sparse keyspaces, we don't need to report missing key errors anyways, and it's very likely that we will need to read all layers intersecting with the key range. Therefore, this patch adds a new fast path for sparse keyspace reads that we do not track unmapped_keyspace in a fine-grained way. We only modify it when we find an image layer.
In debug mode, it was ~5min to read the aux files for a dump of the tenant, and now it's only 8s, that's a 60x speedup. Not benched in release mode because I'd like to discuss the next step of this pull request before refining and doing further benchmarks -- are we okay with a separate code path, or we should integrate it with the current one?
Summary of changes

A new code path for sparse keyspace read. It does not consume done_keys for updating unmapped keyspaces, and only update unmapped keyspaces when image layers are found. This update only involves a single key range, which is significantly faster.
Allow ValueReconstructSituation::Complete to be updated again in ValuesReconstructState::update_key.
Aux file scan now uses the new sparse keyspace fastpath.

Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

would like to know if @VladLazar has any opinion about, thanks :)

Given the performance boost, this might indicate further optimization opportunities in remove_overlapping_with() in the non-sparse case. Did you explore any other optimizations?

The issue here is that the aux keyspace is huge. On non-aux queries, the keyspaces in question are much smaller (max 16 keys per vectored read), so I doubt too much time is being spent in remove_overlapping_with() there.

I'll also update the RFC to include this change


fix import snippet

number is required to load choice menu correctly

thanks !

Great catch! And thanks for submitting the fix. I'll merge and push an update today


More win32/unix I/O woes: libtiff crashing in poppler's pdftocairo on Windows

As reported in https://github.com/conda-forge/poppler-feedstock/issues/73 , it seems that there are TIFF-related crashes in the poppler program pdftocairo on Windows. It can be recreated thus:

My test file is: https://arxiv.org/pdf/2005.01489
Run: pdftocairo -tiff 2005.01489.pdf
The program silently exits with an error code.
Resulting file 2005.01489-1.tif is empty.

The exit code is reported as -1073740791 = 0xC0000409 = STATUS_STACK_BUFFER_OVERRUN, which means that a stack buffer overflow is occuring.
Running the program in a Windows debugger, I get this stack trace:

Resolved by https://github.com/conda-forge/poppler-feedstock/pull/103

Yeah, I haven't heard about any issues on this front for a while, so I'm going to go ahead and close this.


Improve test coverage

At the moment, as evidenced by #1, ovos_utils is 8% covered. That ain't great.

Consider at least stubbing out test classes for all remaining modules. Moving forward, added methods should always add something to tests, even if not complete at least basic tests and #TODOs will help keep things working

closing this one for now https://app.codecov.io/github/OpenVoiceOS/ovos-utils
coverage is almost at 50%, a far cry from the original 8% when this issue was opened


Breakpoint/stopped symbols

Hey there. Thanks for taking this on.
Is/will there be a way to assign a custom breakpoint/stopped symbol?
I tried this in my init:
vim.fn.sign_define('DapBreakpoint', {text = ' ', texthl = '', linehl = '', numhl = ''})
vim.fn.sign_define('DapStopped', {text = ' ', texthl = '', linehl = '', numhl = ''})

But it seems that go.nvim overrides this when started.

Hi 👋🏿 ,
I just found this issue and thought it might be resolved but looking through the code dap is required, and then this plugin replaces daps markers which I have configured to my taste and don't want changed. As far as I can tell unless my config for dap is read after this plugin explicitly my signs won't to precendence

@akinsho
in go setup
you can define your icons:
  icons = { breakpoint = "🧘", currentpos = "🏃" },

Is what you are looking for ?

Hi @ray-x
Not really I saw that option but basically I already configured dap with my preferred icons using sign define I basically don't want this plugin to change it or to have to redefine it for this plugin

Well, I can add an option
icons = false

Thanks @ray-x that's exactly what I was thinking 👍🏿


The issue to run PyGCGOpt on Colab

Dear support team,
I hope you are doing well. I am trying to solve the scheduling problems by PyGCGopt on Colab in order to compare the solving time of different formulations between GCG and SCIP. Some days ago, I could install Conda on the Colab with !pip install conda, but it does not work for now. I try to import and run that in a different way that works for PySCIPopt, but it caused a major error on GCG. Please, see this MRE.

I was wondering if, how can I fix that?
Best regards
Abbas

Dear support team,
Is it possible to check the above issue and say how can I fix this?
Regards
Abbas

Sorry. I will take a look at it in the next days.

I run the notebook on Colab. It crashes but it gives me the following output
presolving:
(round 1, fast)       55 del vars, 10 del conss, 0 add conss, 15 chg bounds, 0 chg sides, 0 chg coeffs, 0 upgd conss, 0 impls, 10 clqs
(round 2, fast)       55 del vars, 10 del conss, 0 add conss, 16 chg bounds, 0 chg sides, 0 chg coeffs, 0 upgd conss, 0 impls, 10 clqs
(round 3, exhaustive) 55 del vars, 10 del conss, 0 add conss, 16 chg bounds, 0 chg sides, 0 chg coeffs, 10 upgd conss, 0 impls, 10 clqs
   (0.0s) probing: 51/85 (60.0%) - 0 fixings, 0 aggregations, 0 implications, 0 bound changes
   (0.0s) probing aborted: 50/50 successive totally useless probings
presolving (4 rounds: 4 fast, 2 medium, 2 exhaustive):
 55 deleted vars, 10 deleted constraints, 0 added constraints, 16 tightened bounds, 0 added holes, 0 changed sides, 0 changed coefficients
 0 implications, 10 cliques
presolved problem has 96 variables (85 bin, 0 int, 0 impl, 11 cont) and 384 constraints
     10 constraints of type <setppc>
    374 constraints of type <linear>
Presolving Time: 0.01
 Consclassifier "nonzeros" yields a classification with 3  different constraint classes
 Consclassifier "constypes" yields a classification with 2 different constraint classes
 Consclassifier "constypes according to miplib" yields a classification with 3 different constraint classes
 Conspartition "constypes according to miplib" is not considered since it offers the same structure as "nonzeros" conspartition
 Consclassifier "gamsdomain" yields a classification with 1  different constraint classes
 Consclassifier "gamssymbols" yields a classification with 1  different constraint classes
 Conspartition "gamssymbols" is not considered since it offers the same structure as "gamsdomain" conspartition
 Varclassifier "gamsdomain" yields a classification with 1  different variable classes
 Varclassifier "gamssymbols" yields a classification with 1  different variable classes
 Varpartition "gamssymbols" is not considered since it offers the same structure as "gamsdomain"
 Varclassifier "vartypes" yields a classification with 2 different variable classes
 Varclassifier "varobjvals" yields a classification with 2 different variable classes
 Varclassifier "varobjvalsigns" yields a classification with 2 different variable classes
 Varpartition "varobjvalsigns" is not considered since it offers the same structure as "varobjvals"
 in dec_consclass: there are 3 different constraint classes
  the current constraint classifier "nonzeros" consists of 3 different classes
  the current constraint classifier "constypes" consists of 2 different classes
  the current constraint classifier "gamsdomain" consists of 1 different classes
 dec_consclass found 11 new partialdecs
dec_densemasterconss found 1 new partialdec
dec_neighborhoodmaster found 1 new partialdec
POSTPROCESSING of decompositions. Added 1 new decomps.
Found 15 finished decompositions.

Chosen structure has 11 blocks, 85 linking variables, and 0 linking constraints.
This decomposition has a maxwhite score of 0.002984.
Warning: Discretization with continuous variables is only an experimental feature.

  time | node  | left  |SLP iter|MLP iter|LP it/n| mem |mdpt |ovars|mvars|ocons|mcons|mcuts|  dualbound   | primalbound  |  deg   |  gap
   0.1s|     1 |     0 |      0 |      0 |     - |6120k|   0 |  96 |   0 | 384 |   0 |   0 | 8.200000e+01 |      --      |   --   |    Inf
r  0.1s|     1 |     0 |      0 |      0 |     - |6125k|   0 |  96 |   0 | 384 |   0 |   0 | 8.200000e+01 | 1.030000e+02 |   --   |  25.61%
   0.1s|     1 |     0 |      0 |      0 |     - |6437k|   0 |  96 |  51 | 385 |  92 |   0 | 8.200000e+01 | 1.030000e+02 |   0.00%|  25.61%   0.1s|     1 |     0 |      0 |      0 |     - |6437k|   0 |  96 |  51 | 385 |  92 |   0 | 8.200000e+01 | 1.030000e+02 |   0.00%|  25.61%   0.5s|     1 |     0 |    938 |    128 |     - |6905k|   0 |  96 | 136 | 385 |  92 |   0 | 8.200000e+01 | 1.030000e+02 |  84.74%|  25.61%   0.5s|     1 |     0 |    984 |    174 |     - |7257k|   0 |  96 | 136 | 385 |  92 |   0 | 8.200000e+01 | 1.030000e+02 |  84.74%|  25.61%
   0.5s|     1 |     0 |    984 |    174 |     - |7257k|   0 |  96 | 136 | 385 |  92 |   0 | 8.200000e+01 | 1.030000e+02 |  84.74%|  25.61%
   0.5s|     1 |     0 |    984 |    174 |     - |7292k|   0 |  96 | 147 | 385 |  92 |   0 | 8.200000e+01 | 1.030000e+02 |  84.74%|  25.61%
r  0.5s|     1 |     0 |    984 |    174 |     - |7327k|   0 |  96 | 158 | 385 |  92 |   0 | 8.200000e+01 | 8.900000e+01 |  84.74%|   8.54%
   0.5s|     1 |     0 |    984 |    174 |     - |7327k|   0 |  96 | 158 | 385 |  92 |   0 | 8.200000e+01 | 8.900000e+01 |  84.74%|   8.54%
   0.5s|     1 |     0 |    984 |    174 |     - |7329k|   0 |  96 | 158 | 385 |  92 |   0 | 8.200000e+01 | 8.900000e+01 |  84.74%|   8.54%
   0.5s|     1 |     0 |    984 |    174 |     - |7329k|   0 |  96 | 158 | 385 |  92 |   0 | 8.200000e+01 | 8.900000e+01 |  84.74%|   8.54%
   0.5s|     1 |     0 |    984 |    174 |     - |7328k|   0 |  96 | 158 | 374 |  92 |   0 | 8.200000e+01 | 8.900000e+01 |  84.74%|   8.54%
o  0.5s|     1 |     0 |    984 |    174 |     - |7383k|   0 |  96 | 169 | 374 |  92 |   0 | 8.200000e+01 | 8.300000e+01 |  84.74%|   1.22%
L  0.5s|     1 |     0 |    984 |    174 |     - |7413k|   0 |  96 | 180 | 374 |  92 |   0 | 8.200000e+01 | 8.200000e+01 |  84.74%|   0.00%
  time | node  | left  |SLP iter|MLP iter|LP it/n| mem |mdpt |ovars|mvars|ocons|mcons|mcuts|  dualbound   | primalbound  |  deg   |  gap
   0.5s|     1 |     0 |    984 |    174 |     - |7413k|   0 |  96 | 180 | 374 |  92 |   0 | 8.200000e+01 | 8.200000e+01 |  84.74%|   0.00%

SCIP Status        : problem is solved [optimal solution found]
Solving Time (sec) : 0.55
Solving Nodes      : 1
Primal Bound       : +8.20000000000000e+01 (5 solutions)
Dual Bound         : +8.20000000000000e+01
Gap                : 0.00 %

start task  1  on machine 3 = 61.0
finish task  1  on machine 3 = 65.0
-----------------------------------
start task  2  on machine 3 = 59.0
finish task  2  on machine 3 = 68.0
-----------------------------------
start task  3  on machine 3 = 1.0
finish task  3  on machine 3 = 8.0
-----------------------------------
start task  4  on machine 3 = 65.0
finish task  4  on machine 3 = 70.0
-----------------------------------
start task  5  on machine 3 = 8.0
finish task  5  on machine 3 = 13.0
-----------------------------------
start task  6  on machine 3 = 13.0
finish task  6  on machine 3 = 17.0
-----------------------------------
start task  7  on machine 3 = 67.99999999999996
finish task  7  on machine 3 = 72.99999999999996
-----------------------------------
start task  8  on machine 3 = 70.0
finish task  8  on machine 3 = 79.0
-----------------------------------
start task  9  on machine 3 = 79.0
finish task  9  on machine 3 = 82.0
-----------------------------------
start task  10  on machine 3 = 75.0
finish task  10  on machine 3 = 82.0
-----------------------------------
```.

@jurgen-lentz,
Thanks a lot for your attention. I should note, in many cases, it crashes and in a few one, it can solve the models. Actually. I cannot take a solution from the MRE. I think you are so lucky. :)
I am looking forward to hearing from you.
Regards
Abbas

Dear support team,
Would you say please, do you have any update on pygcgopt running on the colab to resolve this issue?
Regards
Abbas

Did the answer of Matthias not work?
Hi Abbas,
"!pip install conda" is not going to work. You should use the following to get conda in Colab:
!pip install -q condacolab
import condacolab
condacolab.install()
I hope that helps!
Cheers
Matthias

Dear @jurgen-lentz,
Thanks for your attention. Actually I have used condalab instead of conda because recently colab does not support conda at all.
I have checked pygcgopt on the different problems and almost in all of the cases the issues is remaining. Even though I try to install pyscipopt first and then pygcgopt, but the solver behavior is the same as the before.
I was wondering if, you can check that and see this strange behavior.
All the best
Abbas

Dear @abb-omidi,
yes, I also have the same behavior on Colaboratory.
But running your code as a python script with conda locally, did work perfectly.
Can you look if it crashes also locally?
Best wishes
Jurgen

Dear @jurgen-lentz,
Thank you so much for checking and verifying that. Actually, I used pygcgopt locally with colab and it works well. At the moment I am willing to work with that as a part of DDOM on colab and it would be great if its developer team could fix this issue.
Best regards
Abbas


Union of generic indexed types works incorrectly

I have two generic indexed types
type ExtendedService<T> = {
    [K in keyof T]: T[K] & {
        __$daemonMode?: string;
        __$action?: string;
    };
};

type Service<T> = {
    [K in keyof T]: T[K] & {id?: string};
};

And use them like this
export const createService = <T>(
    ServiceCtr: ExtendedService<T> & Service<T>
) => {
    ...
    Object.keys(ServiceCtr).forEach(key => {
        const method = (ServiceCtr)[key as keyof T];
        const {__$daemonMode, __$action, id} = method;
        ...
    }
}

VS Code output for method type is
const method: ({ [K in keyof T]: T[K] & {
    __$daemonMode?: string;
    __$action?: string;
}; } & Service<T>)[keyof T]

Expected behavior:
properties __$daemonMode, __$action, id exists on method variable;
const {__$daemonMode, __$action, id} = method;  has no errors;
Actual behavior:
I got an error
[ts] Type '(ExtendedService<T> & Service<T>)[keyof T]' has no property '__$daemonMode' and no string index signature.

__$action and id produces the same error;
P.S.
Expression
const x: (ExtendedService<{z: {}}> & Service<{z: {}}>)['z'];
const {__$daemonMode, __$action, id} = x;

works as expected

Fixed by #26281


Can't install nodegit

System information

node version: v17.3.0
npm or yarn version: 1.22.17
OS/version/architecture: Linux tiago-A-F-K20CE 5.11.0-49-generic #55-Ubuntu SMP Wed Jan 12 17:36:34 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Applicable nodegit version: latest

v17.3.0
8.3.0
linux
5.11.0-49-generic
x64

I can't install node git, it throws this error when building it:
In file included from /home/tiago/.node-gyp/17.3.0/include/node/openssl/././archs/linux-x86_64/asm/include/openssl/ssl.h:21,
                 from /home/tiago/.node-gyp/17.3.0/include/node/openssl/./ssl_asm.h:11,
                 from /home/tiago/.node-gyp/17.3.0/include/node/openssl/ssl.h:4,
                 from ../vendor/libgit2/src/netops.h:17,
                 from ../vendor/libgit2/src/fetch.h:14,
                 from ../vendor/libgit2/src/fetch.c:8:
/home/tiago/.node-gyp/17.3.0/include/node/openssl/macros.h:155:4: error: #error "OPENSSL_API_COMPAT expresses an impossible API compatibility level"
  155 | #  error "OPENSSL_API_COMPAT expresses an impossible API compatibility level"
      |    ^~~~~

I was using an unsupperted node version, i downgraded to 14.10.0 with asdf and it worked


Reorganize navigation, add testing basics page

Add README FAQ instructions for how to run the docs locally
Hide blank debugging pages from nav
Reorganize navigation so the "Getting Started" section is not bloated
Add a page for testing basics

LGTM!
I would wait for @digorithm to give it one final look before merging.


Set image stream imports to upstream

All images come from quay.io upstream versions

@pleimer Are all of these changes handed by PR #33?

This is intermediary to #33


Ability to verify the EAD handler type

See discussion here.

This is partially solved by #170 and #174. What is missing is some code to actually test the EAD type and act accordingly.


moment(undefined) returns the current date (i.e. equivalent to moment())

Is this really expected behavior? This makes it difficult to determine that an undefined variable is the source of an invalid date rather than an incorrect signal coming from an API on the server for example.

I suppose we could explicitly check arguments.length for whether an undefined argument was passed or no argument was passed. This represents a major behavioral change for the library though. I'm honestly not sure if that's something we would be able to do in a minor release.

I'm not aware of the internals of moment.js. It seems like a fairly straightforward fix from an outside perspective. Just check arguments.length and if the length is greater than 0, check each argument against typeof === "undefined"

Yes, it is a straightforward change. However, there is no way to know how many people are relying on the library working the way that it does - thus we can't just make that kind of change without analysis.

Strong 👎 . See also #2729. I just don't think that checking arguments.length at the top of every Javascript function is the way JS is supposed to work. There's a reason that null and undefined are different things; perhaps that distinction was unwise on the part of JS's designers, but it's how it works, and why it's so troublesome to try to "fix".
Consider, for example, the caller who wants to choose which arguments to pass moment():
moment(needsAnArg() ? something : undefined);

I'd certainly expect that in the case where needsAnArg() returns false, the code ought to be equivalent to just moment(). That's what undefined is sort of for: "It's not that I have any empty value; it's that I don't have one at all."

Issac, I'm sorry, but I don't agree with your perspective. The successful design of a library is strongly coupled with the APIs it exposes. From an outside perspective, moment(undefined) should at the very lease return an invalid date. It's, in my opinion, nonsensical that undefined somehow means the current date.
With that said, I understand your point in the difficulty that this requirement would impose on the existing design of the library. Perhaps there is an alternative implementation other than checking arguments.length at the beginning of every function. I'm sure some refactoring (not taken lightly in JS) would solve this problem.
Thirdly, I would argue that it makes much more sense to call moment(needsAnArg() ? something : new Date()) if that really was my intention.
Finally, the discovery of this was not due to lazy programming. It was due to an idiosyncrasy with using Typescript. I build object definitions to match the object model coming in from an API on the Server. I unfortunately added a typo and the variable was always undefined. In trying to find this bug, I first thought it was a bug with Typescript. It turns out, it was the result of a type-o. I spent several hours on trying to find the source of this bug. It would have saved me a lot of grief if moment reported an invalid date or threw an exception.

It's unfortunate that typos in JS can cause vars to be undefined, and perhaps it would be best if JS just exploded when you try to use an undefined var.
But your case that moment(undefined) should behave in a special way is a complaint about JS, not about Moment. It really is the case that JS is supposed to work by making foo() and foo(undefined) equivalent. That's why foo's first parameter gets bound to undefined when you don't provide it. If we start doing this thing where we say "JS works this way but it should really work this other way so we're coding around it" we're swimming upstream.
Do you think it is the case that most JS libraries with optional arguments check the arguments count and throw as your suggesting? I don't believe it is. Here's a built-into-your-browser one I just happened to be playing with recently:
new Intl.NumberFormat().format(1) //=> "1"
new Intl.NumberFormat(undefined).format(1); //=> "1", just like above
new Intl.NumberFormat("ar").format(1) //=> "١", how this arg is meant to be used
new Intl.NumberFormat(null).format(1); //error

So it's doing what we do. (There are some interesting places where built-in JS APIs do treat undefined differently and they always trip me up.)

I would argue that it makes much more sense to call moment(needsAnArg() ? something : new Date()) if that really was my intention.

That may be, but the question is what the user's expectations are when doing what I wrote.

I'm going to agree with @icambron on this one. Trying to work around the language doesn't seem right. Checking arguments.length at the start of every function is the definition of a code smell.


Make Rectangle Resizable

The rectangle should be resizable instead of manually choose the rows and columns from the property menu.
Are you working on this? (Yes/No)
NO

@mahmodHammad  The size of rectangle  can be increased by arrow keys  since if user don't want to put values in properties

@ryuk156
I didn't figure out that.
Most of users will not too.
So it will be better if we enable resizing by dragging, users are most. Likely on hand users (mouse )

Okay👍

Its kinda hard to implement stuff like this

Hello @mahmodHammad @satu0king
I am planning to work on this issue, and I hope it's still available.
Thank you!


Windows Stand Alone Player returns null reference when you instantiate in Scene 0

Version Number and Operating System(s):
03/10/2018-19.53.38,04 Windows, Unity Editor 2019.2.21f1
Expected behavior:
The Editor and Windows Stand Alone version behave the same, regardless of what i am doing, and it doesn't return null when making a network object
Actual behavior:
The Windows player behaves differently than the editor, when i instantiate the chat object it returns null but the editor works
Steps to reproduce:
(not tried these actual steps but have reproduced similar) Add the multiplayer scene into a build, turn off automatically change scenes, make it spawn the chat window/network object on a key press. Build for Windows (debug), log in as host and press that key to spawn the chat object all in one scene, note you get a null reference as the whole GameObject[] is null which should hold your Chat prefab - i believe the whole thing is null, not just it has no entries but could not attach the debugger to my windows build to confirm. Also note, if you do this in the editor in Unity, it is not null.
If you make a scene 0 that doesn't really do anything but go to scene 1 (where you have the multiplayer scene we used before and the Chat on Keypress), then make a windows build the same thing will actually work. I am not sure if you need the forge manager script in scene 0 and have it not destroy on scene change and this is doing something, or just the fact this is all done (the instantiating) in a scene with index != 0 will work regardless. I do know you don't need to become host in scene 0, before spawning chat in scene 1, you can do both these things in scene 1 and it will still work.
[Optional] Discord Username:
Gwom

I'm not sure I follow what you are describing here. Would you be able to provide a minimal project where I can see what you are trying to describe above.
Also proven reproduce steps would be much appreciated.

@phalasz unfortunately if make a project using basic steps it seems to not be reproducible so it must be some kind of conflict with my setup and slicing it out of the project would take quite a while. I quickly tried the following and the issue does not happen on Windows Player:

Make a project with the multiplayer scene
Turn off Automatic Scene on the NetworkManager prefab
Turn on DontChangeSceneOnConnect on the MutiplayerManager component
Add to the MultiplayerManager.cs at ln 168 else if (Input.GetKeyDown(KeyCode.Q)) NetworkManager.Instance.InstantiateChatManager();
Make a Windows build, run, connect as client and press Q

It is very difficult to see what I am doing that is different to this, and due to another issue I cannot attach my debugger to the windows player. As such, it is difficult to know why I am getting a null reference for the Instantiate but only on the Windows version, not on the Editor. If i can get more details or a more concrete example that would be good but for now, I am unsure how to provide more info (sorry). If you can think of anything that may affect Windows only, to do with scenes that would be good to know.

I personally haven't see an issue like this and without concrete steps that always reproduce the problem I will not be able to help.


Add maven to the project

Issue #5
I have added maven with maven-compiler-plugin and maven-jar-plugin. The Java version is 15 as specified in the repository's README.
I have seen that there is already another PR to add Gradle, so feel free to reject this if you don't need it finally ;).

Update the README.md with the new way of running the project with Maven.

Update the README.md with the new way of running the project with Maven.

Done! I have also added the plugin maven-exec-plugin, so you can run the game directly using a maven goal.

Hi! Thanks for your contribution!
^^


Role issuance order when multiple roles are assigned to the same principal for a given app

Certain SAML service providers can only evaluate the first value from the Role claim, if it contains multiple values.
There are scenarios where the issuance order for the role values needs to be controlled, in case a user has multiple roles assigned to them.
Currently, neither assignment order nor the order in which the roles are defined in the app registration manifest do not influence the order or set a precedence to on how the roles should be issued.
This CAN be achieved by controlling the GUID value from the role.
Practical example:
generating a list of GUIDs and incrementing the value one by one, can control the order in which the roles get placed in the Role claim inside a SAML token, should multiple roles be assigned to the same user:
00000000-b5c0-4e2f-bd28-4c8e565e291f
00000001-b5c0-4e2f-bd28-4c8e565e291f
00000002-b5c0-4e2f-bd28-4c8e565e291f
00000003-b5c0-4e2f-bd28-4c8e565e291f
This behavior should be documented if it is consistent since it could address the above mentioned scenarios.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0184d69e-7219-f4f0-6583-32f79f5a76e1
Version Independent ID: 49e5b0f0-69cb-4ab1-c8d9-62eb07778a88
Content: Application roles - Azure Architecture Center
Content Source: docs/multitenant-identity/app-roles.md
Service: architecture-center
Sub-service: reference-architecture
GitHub Login: @adamboeglin
Microsoft Alias: pnp

Thanks for reporting - this issue is under review.  This is a Microsoft Internal DevOps Bug ID AB#249461

Hey there @boboldi, that's unfortunate that the SAML consumer was programmed only to look at the first item in the appRoles  (roles in SAML) array.  Out of curiosity, what consumer of this token was written to only look at the first item in this array?  Any chance a bug (read: feature) could be opened up upstream to address that?
I don't see any documentation or internal implementation guarantees that implies that appRoles (roles) has a stable sort order; matter of fact we have at least one documentation example showing a non-alphabetical sort on id.  Taking a dependency on the order in which they return would be a high risk activity, that we'd advise against to ensure no hard-to-track down issues arise . For example, depending on how it's used, you could end up having all admins locked out of the system because of the order in which they return changed.
While I see where you're coming from, I hesitate to recommend a practice that there no evidence or documented guarantee that it won't change in the future. I might have missed the documentation on where we make this promise, so if you have evidence of this, other than "it works that way now" I'd love to dig in.  I just can't find that evidence.  If you have it, I'll follow through then with the Microsoft Identity Platform team to validate it will be stable.  Until then, it's best to assume it's an unstable ordering.
Maybe even better, you could alternatively ask this question right to the Microsoft Identify Platform team on this page and they can give you an authoritative answer (which I cannot).  If you learn from them that it's, by contract, consistent, then we can include guidance to the same here, as you suggested.

I'm going to go ahead and close this issue based on my recommendation to avoid taking a dependency on the order.  If however you do find evidence that order is stable and will remain stable, we can continue the conversation (you can reopen this or start a new issue) and consider how we should incorporate that guidance.  Thanks.


[Vote] Donate DevLake to Apache Software Foundation

Background
DevLake is going to be donated to Apache Software Foundation. Initial committers are required before transferring the project. Considering the past contribution and long-term willingness to engage in the project, we nominated several initial committers:

@klesh
@e2corporation
@mindlesscloud
@warren830
@joncodo
@hezyin
@MaximDub

We nominate the initial committers mainly according to their code contributions measured by this paper. Below are the original top five code contributors in the past six months.

We added Hezheng to the list because he is responsible for everything technical, and Maxim because he is responsible for user interviews and has made a lot of non-technical contributions.
Voters can either respond "agree, yes, +1", "disagree, no, -1", or "abstain".

+1

I agree! 👍

+1 yay~

+1

+1

Agree

+1

+1

+1

already finished


Using WASM modules in Comlink-ed Webworker

For my project I have a WASM (Rust based) module imported as a package using wasm-pack.
In the regular (non-Web Worker) workflow the package works just fine, but when I put everything under comlink-loader things start to break and I get
_wasm_bg_wasm__WEBPACK_IMPORTED_MODULE_0__.add_two_ints is not a function

initially coming from
callback
src/comlink.ts:312
  309 |   break;
  310 | case MessageType.APPLY:
  311 |   {
> 312 |     returnValue = rawValue.apply(parent, argumentList);
      | ^  313 |   }
  314 |   break;
  315 | case MessageType.CONSTRUCT:

is there a good way to keep comlink-loader and fix that?

I'm running into the same problem, @sklinov did you find any workaround for this?

@outkine Sorry to write that here, but I've migrated from comlink-loader to another tool.
You can check out my boilerplate example here https://github.com/sklinov/react-wasm


Spacing of negative inside parens

The spacing between the negative sign and the number is incorrect when the negative is preceeded by a parenthesis.  For example in (-3,6), the negative is spaced like subtraction, like in 5-3, rather than the tighter spacing in -3
From a quick glance, it looks like the TeX renderer uses − rather than - for the negative sign; I'm presuming this is what's causing the different spacing.
We might want to see if we can find a way to distinguish in the AM parser between subtraction and a negative symbol so we can mimic the spacing of the TeX renderer.

Here's what's happening:  currently, (-3,6) produces
<math xmlns="http://www.w3.org/1998/Math/MathML">
  <mstyle displaystyle="true">
    <mrow>
      <mo>(</mo>
      <mo>-</mo>
      <mn>3</mn>
      <mo>,</mo>
      <mn>6</mn>
      <mo>)</mo>
    </mrow>
  </mstyle>
</math>

This means that the minus sign is an infix operator, so gets the space of a binary minus sign, not a unary negation.  The recommended way to code -3 is <mrow><mo>-</mo><mn>3</mn></mrow>.  The extra <mrow> will cause the minus to be a prefix operator, and get the appropriate spacing.
An alternative would be to use <mo form="prefix">-</mo><mn>3</mn>, but the <mrow> approach is what the spec recommends.
Another reason that the TeX output differs from the AsciiMath is that AsciiMath uses MathML spacing, but TeX uses TeX spacing (the HTML-CSS and SVG output renders use TeX spacing rather than MathML spacing unless told otherwise, but we turned on MathML spacing for AsciiMath, since it assumed those rules in some instances, namely things like {x|x>1}).

I noticed that the TeX processor produces for $(-3,6)$
<math xmlns="http://www.w3.org/1998/Math/MathML">
  <mo stretchy="false">(</mo>
  <mo>−<!-- − --></mo>
  <mn>3</mn>
  <mo>,</mo>
  <mn>6</mn>
  <mo stretchy="false">)</mo>
</math>

I'm guessing the use of ࢤ is also contributing to the layout difference?  I was figuring that I was going to need to adjust the AM processor to return that symbol for negation - is that correct?

No, the HTML-CSS and SVG output converts the dash to U+2212 automatically.  For native MathML, it will depend on the browser's implementation.  Firefox does seem to convert it as well.  I doubt WebKit does.

Ahh, ok.  So the correct fix is to wrap the negation and what it's applying to in a separate .  I'll see if I can figure a way to get that to work.

So digging into this again, it appears the only way to address this issue would be to separate the CONST token type into two subtokens: ones that "separate" expressions (like , ; : = < -> and a bunch others) and ones that don't (like numbers, variables, basic operators).
A negative following a separating CONST or a LEFTBRACKET would be treated like a negation, while a negative following a standard CONST would act as a minus operator.
Do you see any pitfalls of this approach?

Found an issue already. The negative in  4,-2/3 would display differently than in -2/3,4 which isn't sensible.

Could you resolve this problem by looking for separating CONST, LEFTBRACKET, or no previous token?
I think there are other situations you might need to worry about, like summations and integrals.  I would hope that int -2x dx would use negation, though I suppose you could do int {:-2x:} dx if you needed to.

Could you resolve this problem by looking for separating CONST, LEFTBRACKET, or no previous token?
I think there are other situations you might need to worry about, like summations and integrals. I would hope that int -2x dx would use negation, though I suppose you could do int {:-2x:} dx if you needed to.

The solution is very much appreciated.


Annotations have stopped working

I can see there's code for the iNaturalist and AnnoSys annotations, but it looks to have stopped working.

it was disabled on purpose if I remember correctly. It i has since moved into a generic feedback form for all sorts of feedback