Internal: Failed to enqueue async memset operation: HIP_ERROR_InvalidValue

Hi! I cannot run a minimal example
It is because the my GPU is not supported?
System information

OS: Archlinux
TensorFlow i tried installing from source and  with the one from pip (tensorflow-rocm)
TensorFlow version: merge-200420-402-gb9588d1af6 2.1.0
Python version: Python 3.7.7
Bazel version (if compiling from source): bazel-3.0.0
GCC/Compiler version (if compiling from source): gcc 9.3.0
CUDA/cuDNN version: 10.1
GPU model and memory: Radeon RX 470

-MIOpen: miopen-2.3.0
-HIP: The version tagged with ROCm 3.3.0
Describe the current behavior
Every time I call the fit method it finishes with the following error:
2020-04-29 01:07:16.372850: E tensorflow/stream_executor/stream.cc:5929] Internal: Failed to enqueue async memset operation: HIP_ERROR_InvalidValue
2020-04-29 01:07:16.372894: I tensorflow/stream_executor/stream.cc:317] did not allocate timer: 0x7f5fbfffdf20
2020-04-29 01:07:16.372902: I tensorflow/stream_executor/stream.cc:2186] [stream=0x55613341ada0,impl=0x55613341b2c0] did not enqueue 'start timer': 0x7f5fbfffdf20
2020-04-29 01:07:16.372946: I tensorflow/stream_executor/stream.cc:2198] [stream=0x55613341ada0,impl=0x55613341b2c0] did not enqueue 'stop timer': 0x7f5fbfffdf20
2020-04-29 01:07:16.372967: F tensorflow/stream_executor/gpu/gpu_timer.cc:65] Check failed: start_event_ != nullptr && stop_event_ != nullptr

Describe the expected behavior
Run without crashing
Standalone code to reproduce the issue
https://www.tensorflow.org/tutorials/quickstart/beginner?hl=en

Can you post the complete output?

After creating the model:
020-04-29 20:42:13.635622: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libhip_hcc.so
2020-04-29 20:42:13.736471: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1561] Found device 0 with properties:
pciBusID: 0000:01:00.0 name: Ellesmere [Radeon RX 470/480/570/570X/580/580X/590]     ROCm AMD GPU ISA: gfx803
coreClock: 1.23GHz coreCount: 32 deviceMemorySize: 4.00GiB deviceMemoryBandwidth: 98.35GiB/s

2020-04-29 20:42:13.796267: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library librocblas.so
2020-04-29 20:42:13.849208: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libMIOpen.so
2020-04-29 20:42:13.893434: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library librocfft.so
2020-04-29 20:42:13.910734: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library librocrand.so
2020-04-29 20:42:13.910845: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1685] Adding visible gpu devices: 0
2020-04-29 20:42:13.943421: I tensorflow/core/platform/profile_utils/cpu_utils.cc:102] CPU Frequency:<PHONE_NUMBER> Hz
2020-04-29 20:42:13.943707: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x56326ec9c890 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2020-04-29 20:42:13.943725: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
2020-04-29 20:42:13.943869: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1561] Found device 0 with properties:
pciBusID: 0000:01:00.0 name: Ellesmere [Radeon RX 470/480/570/570X/580/580X/590]     ROCm AMD GPU ISA: gfx803
coreClock: 1.23GHz coreCount: 32 deviceMemorySize: 4.00GiB deviceMemoryBandwidth: 98.35GiB/s
2020-04-29 20:42:13.943916: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library librocblas.so
2020-04-29 20:42:13.943934: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libMIOpen.so
2020-04-29 20:42:13.943949: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library librocfft.so
2020-04-29 20:42:13.943963: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library librocrand.so
2020-04-29 20:42:13.944013: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1685] Adding visible gpu devices: 0
2020-04-29 20:42:13.944066: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1084] Device interconnect StreamExecutor with strength 1 edge matrix:
2020-04-29 20:42:13.944077: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1090]      0
2020-04-29 20:42:13.944084: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1103] 0:   N
2020-04-29 20:42:13.944181: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1229] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 3540 MB memory) -> physical GPU (device: 0, name: Ellesmere [Radeon RX 470/480/570/570X/580/580X/590], pci bus id: 0000:01:00.0)
2020-04-29 20:42:14.176568: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x56326fd7c4c0 initialized for platform ROCM (this does not guarantee that XLA will be used). Devices:
2020-04-29 20:42:14.176601: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Ellesmere [Radeon RX 470/480/570/570X/580/580X/590], AMDGPU ISA version: gfx803

After calling fit:

Epoch 1/5
2020-04-29 20:42:49.779373: I tensorflow/core/graph/gpu_fusion_pass.cc:505] ROCm Fusion is enabled.
2020-04-29 20:42:49.794449: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library librocblas.so
2020-04-29 20:42:52.636045: E tensorflow/stream_executor/stream.cc:5929] Internal: Failed to enqueue async memset operation: HIP_ERROR_InvalidValue
2020-04-29 20:42:52.636263: I tensorflow/stream_executor/stream.cc:317] did not allocate timer: 0x7f98ca7faf20
2020-04-29 20:42:52.636348: I tensorflow/stream_executor/stream.cc:2186] [stream=0x56326fcc8000,impl=0x56326fcc8520] did not enqueue 'start timer': 0x7f98ca7faf20
2020-04-29 20:42:52.636479: I tensorflow/stream_executor/stream.cc:2198] [stream=0x56326fcc8000,impl=0x56326fcc8520] did not enqueue 'stop timer': 0x7f98ca7faf20
2020-04-29 20:42:52.636578: F tensorflow/stream_executor/gpu/gpu_timer.cc:65] Check failed: start_event_ != nullptr && stop_event_ != nullptr

Try to run /opt/rocm/bin/rocm-smi and /opt/rocm/bin/rocminfo and post their outputs.

rocm_info.txt
rocm_smi.txt

The setup looks in order. I suspect that the GPU is the problem - it is theoretically supported, but, in practice, newer releases are not tested on gfx803 GPUs and therefore YMMV.
Let's try two more things:

Rename the file /opt/rocm/miopen/share/miopen/db/gfx803_36.HIP.fdb.txt into /opt/rocm/miopen/share/miopen/db/gfx803_32.HIP.fdb.txt and retry.
Run the test after setting the environment variable HIP_TRACE_API=14 and post the output.

:(
HIP_TRACE_API_14.txt
I know that around July of last year I could make tensorflow works with this GPU in my PC. Perhaps some change broke it for me.

@ekuznetsov139 Thanks for taking the first look at the issue, I'm assigning the issue to you now. In case you need the machine to reproduce, we can sync offline or talk with @ekuznetsov139

I have been able to run the tutorial script without errors on a Radeon RX 480 with ROCm 3.3. It's the same chip except with 36 CUs instead of 32.
You could try to load the docker image  rocm/tensorflow:rocm3.3-tf2.1-dev and see if it works there.
Also, what's the exact commit hash of the version of miopen that you compiled?

Thank you very much for your help.
In my pc originally I was using the following commit for MIOpen: 869a484773f59a79eb53eac9fa9233e52d80b3c3
now I tried with 4c1b0ca4987eb5931d694b1be73137267861c369 the last one so far, but I got the same error about HIP_ERROR_InvalidValue.

These are my devices in the host
crw-rw----+ 1 root video 238, 0 may 16 13:29 /dev/kfd

total 0
drwxr-xr-x  2 root root         80 may 16 13:29 by-path
crw-rw----+ 1 root video  226,   0 may 16 13:29 card0
crw-rw-rw-  1 root render 226, 128 may 16 13:29 renderD128

Docker image
In order to be able to run rocminfo with the least amount of errors I  installed kmod and I added the render group, and I changed the group ownership for the devices. I attach the rocminfo output
rocminfo_docker.txt
rocm-smi
GPU  Temp   AvgPwr  SCLK    MCLK    Fan     Perf  PwrCap  VRAM%  GPU%
0    47.0c  9.047W  466Mhz  300Mhz  31.76%  auto  105.0W    5%   0%

tensorflow in python
when I ran the test code, the GPU is not being recognized and it fallbacks  to the CPU
2020-05-16 11:45:37.232898: I tensorflow/stream_executor/platform/default/dso_loader.cc:44] Successfully opened dynamic library libhip_hcc.so
2020-05-16 11:45:37.249434: E tensorflow/stream_executor/rocm/rocm_driver.cc:975] could not retrieve ROCM device count: HIP_ERROR_NoDevice
2020-05-16 11:45:37.253050: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
2020-05-16 11:45:37.253289: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x3fe5170 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2020-05-16 11:45:37.253317: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
2020-05-16 11:45:37.254101: E tensorflow/stream_executor/rocm/rocm_driver.cc:975] could not retrieve ROCM device count: HIP_ERROR_NoDevice

With another user different from root I got the following error:
ROCk module is loaded
Failed to get user name to check for video group membership
hsa api call failure at: /data/jenkins-workspace/compute-rocm-rel-3.3/rocminfo/rocminfo.cc:1102
Call returned HSA_STATUS_ERROR_OUT_OF_RESOURCES: The runtime failed to allocate the necessary resources. This error may also occur when the core runtime library needs to spawn threads or create internal OS-specific events.

Jajaj I suppose my stack installation is more broken than what I initially thought

Over here, /dev/kfd is
crw-rw-rw- 1 root root 241, 0 May 15 17:08 /dev/kfd
According to your rocminfo_docker.txt, rocminfo does not see GPU.
The typical command line to launch docker is
sudo docker run -it --network=host --device=/dev/kfd --device=/dev/dri --ipc=host --shm-size 16G --group-add video --cap-add=SYS_PTRACE --security-opt seccomp=unconfined
The HSA_STATUS_ERROR_OUT_OF_RESOURCES error often means that the user does not have the right access rights to /dev/kfd.

Hi!
Apparently I had a problem with the kernel module, I recompile it and now tensorflow is working in the  docker image rocm/tensorflow:rocm3.3-tf2.1-dev .
In the host the problem still persists.


Added support for geometry shaders

I went ahead and created this branch/pull request as an alternative/enhancement to #432 and fixes issue #428.
Here are the differences/additions:

This patch doesn't remove the member function group loadFrom...(resource, type).
I've added a new example to the Shader example utilizing a a geometry shader to draw billboards from single points (great for particle systems; this is rendering 10.000 moving/resizing/rotating sprites).

The code is once again fully backwards compatible with existing code.

Superseded by #886.


clean up unused DB queries for repositories

Summary
Provide a brief overview of the changes and the issue being addressed.
Explain the rationale and any background necessary for understanding the changes.
List dependencies required by this change, if any.
Fixes #(related issue)
Change Type
Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[x] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[ ] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 39.513% (-0.006%) from 39.519%
when pulling 1fb84281bc12389d27a3494d31fa701bc3c4393c on remove-unused-db-queries
into 45750b4e9fb2de33365758366e06c19e999bd2eb on main.


Multiprocess leaflet tile generation

Hi,
I've applied parallelise base tile generation patch to the
gdal2tiles.py , please view patch details:
http://trac.osgeo.org/gdal/ticket/4379
then ported commenthol's leaflet tile support scripts to patched gdal2tiles.py
so that it takes advantage of multi core processing and builds tiles faster.
I didn't modify the original gdal2tiles.py in the commenthol's repository however created a new one for testing/evaluating the multicore version.
please try following script for tile generation:

python gdal2tiles-multiprocess.py -l -p raster -z 0-5 -w none  karta.jpg tilesc

Bests,
Guvener

Hi @guvener,
That patch on gdal2tiles is really impressive with regards to speed.
Splitting up in two files (the normal one and the multiprocess) is a good idea, because of the multiprocess status in the Gdal project.
Thanks for contributing.

Hi @commenthol,
Thank you for merging multiprocess script in the project, always happy to contribute.
Bests,
Guvener


[BXMSPROD-1930] Changing build-chain version

Thank you for submitting this pull request
JIRA: https://issues.redhat.com/browse/BXMSPROD-1930
Referenced Pull Requests:

https://github.com/kiegroup/droolsjbpm-build-bootstrap/pull/2263
https://github.com/kiegroup/kogito-pipelines/pull/935

Notes:

The former folders/files are not going to be removed in this PR.

You can check Kiegroup organization repositories CI status from Chain Status webpage

How to replicate CI configuration locally?

Build Chain tool does "simple" maven build(s), the builds are just Maven commands, but because the repositories relates and depends on each other and any change in API or class method could affect several of those repositories there is a need to use build-chain tool to handle cross repository builds and be sure that we always use latest version of the code for each repository.
build-chain tool is a build tool which can be used locally on command line or in Github Actions workflow(s), in case you need to change multiple repositories and send multiple dependent pull requests related with a change you can easily reproduce the same build by executing it on Github hosted environment or locally in your development environment. See local execution details to get more information about it.
A general local execution could be the following one, where the tool clones all dependent projects starting from the -sp one and it locally applies the pull request (if it exists) in order to reproduce a complete build scenario for the provided Pull Request.

Note: the tool considers multiple Pull Requests related to each other if their branches (generally in the forked repositories) have the same name.

$ build-chain-action -df 'https://raw.githubusercontent.com/${GROUP:kiegroup}/droolsjbpm-build-bootstrap/${BRANCH:main}/.ci/pull-request-config.yaml' build pr -url <pull-request-url> -sp kiegroup/kie-wb-distributions [--skipExecution]

Consider changing kiegroup/kie-wb-distributions with the correct starting project.

How to retest this PR or trigger a specific build:

a pull request please add comment: Jenkins retest (using this e.g. Jenkins retest this optional but no longer required)

for a full downstream build

for jenkins job: please add comment: Jenkins run fdb
for github actions job: add the label run_fdb

a compile downstream build please  add comment: Jenkins run cdb

a full production downstream build please add comment: Jenkins execute product fdb

an upstream build please add comment: Jenkins run upstream

for windows-specific os job add the label windows_check

How to backport a pull request to a different branch?

In order to automatically create a backporting pull request please add one or more labels having the following format backport-<branch-name>, where <branch-name> is the name of the branch where the pull request must be backported to (e.g., backport-7.67.x to backport the original PR to the 7.67.x branch).

NOTE: backporting is an action aiming to move a change (usually a commit) from a branch (usually the main one) to another one, which is generally referring to a still maintained release branch. Keeping it simple: it is about to move a specific change or a set of them from one branch to another.

Once the original pull request is successfully merged, the automated action will create one backporting pull request per each label (with the previous format) that has been added.
If something goes wrong, the author will be notified and at this point a manual backporting is needed.

NOTE: this automated backporting is triggered whenever a pull request on main branch is labeled or closed, but both conditions must be satisfied to get the new PR created.

@bacciotti do you mind to rebase the branch? I see some old commits

@lampajr Done 😇


Changes to settings do not apply to certain users

When using the launcher, some users are unable to have any settings changes take effect. The same users are able to change the settings in the vanilla game, however.
This is possibly unrelated to #4, as some users have reported having no non-Latin characters in their Windows username.

Fixed in 3979885.


Refactoring to adhere to Credo

Solving these issues:

Pipe chain should start with a raw value (x2)
Cond statements should contain at least two conditions besides true
Space missing after comma (Alias declaration)

Changes Unknown when pulling 2fc948ba690d4ec0bb45fd4724a045b41516e9f6 on credo-refactor into ** on master**.


There should be what to do after saving the trained Scikit-Learn model (e.g., using joblib)

The code show how to save but not say how to use, access it later! So it not meaningful yet.

Oh I see! It's just identical to my_model. So you can make predictions with it, just like you would with my_model:
my_model_loaded.predict(X_new)

Makes sense?


Alternative yaml config convention

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/3149
Originally created by @kgpayne on 2022-01-10 17:32:04

As the Meltano feature-set and average project size grows, managing configuration as either a single large yaml document or with explicit include_paths imports may impose an upper-limit on readability/maintainability. This might be overcome by taking a similar approach to Helm or dbt, whereby all files of a certain type below a specified directory are assumed to contain Meltano config objects and are parsed (and therefore referencable). Following the Helm/Kubernetes resource-centric model (and resurrecting and repurposing the .m5o file type?), a Loader Plugin might look like:
# loaders/athena.m5o
---
version: 1
kind: Loader
metadata:
  name: target-athena
  namespace: target_athena
spec:
  pip_url: git+https://github.com/pnadolny13/target-athena.git@bump_sdk_version
  settings:
  - name: aws_access_key_id
    kind: password
  - name: aws_secret_access_key
    kind: password
  - name: s3_bucket
    kind: string
  - name: athena_database
    kind: string
  - name: s3_staging_dir
    kind: string
  - name: work_group
    kind: string
  - name: add_record_metadata
    kind: boolean
  - name: flatten_records
    kind: boolean
  - name: object_format
    kind: string
  config:
    aws_region: us-east-2
    aws_access_key_id:
      secretKeyRef:
        name: athena-secrets
        key: AWS_ACCESS_KEY_ID
    aws_secret_access_key:
      secretKeyRef:
        name: athena-secrets
        key: AWS_SECRET_ACCESS_KEY
    object_format: jsonl
    temp_dir: .output
    add_record_metadata: true
    flatten_records: true

Adding new plugins off the same kind might result in either a new file (sharing the name of the plugin) or a new document in an existing loaders.m5o file (using the --- yaml document syntax). Each resource type would have its own schema definition and be validatable. Plugins registering new resource types will be expected to supply this schema.
This approach lends itself well to Inventory too:
# inventory/snowflake.m5o
---
version: 1
kind: Warehouse
metadata:
  name: snowflake
  label: snowflake-prod
spec:
  account: "<snowflake account name>"
  default_database: "<snowflake database name>"
  region: "<warehouse region>"
  warehouse: "<snowflake warehouse name>"

---
version: 1
kind: WarehouseConnection
metadata:
  name: snowflake-connection-dbt
  label: snowflake-prod
spec:
  warehouse: snowflake # references the warehouse above
  database: "analytics"
  username: "dbt"
  password:
    secretKeyRef:
      name: snowflake-secrets
      key: password
  role: "dbt"
  session_parameters: "<session level parameters>"

View 8 previous comments from the original issue on GitLab

Closing. This was logged, I think, before we had support for include_paths.


EDDI not starting/not connecting to Companion App

Hi,
I was using EDDI perfectly fine last night, no problems at all.  Today when I tried to start EDDI, it tries to load, but does not appear.  After a bit of trouble shooting (deleting credentials.json) it seems there is a problem with connecting to the Companion App.
After deleting the credentials, EDDI loads up.  When I try to reconnect to the Companion App, it throws the following error:
System.ArgumentException: An item with the same key has already been added.
   at System.ThrowHelper.ThrowArgumentException(ExceptionResource resource)
   at System.Collections.Generic.Dictionary`2.Insert(TKey key, TValue value, Boolean add)
   at System.Linq.Enumerable.ToDictionary[TSource,TKey,TElement](IEnumerable`1 source, Func`2 keySelector, Func`2 elementSelector, IEqualityComparer`1 comparer)
   at System.Linq.Enumerable.ToDictionary[TSource,TKey](IEnumerable`1 source, Func`2 keySelector)
   at EddiCompanionAppService.CompanionAppService.AugmentShipInfo(Ship ship, List`1 storedShips)
   at EddiCompanionAppService.CompanionAppService.ProfileFromJson(JObject json)
   at EddiCompanionAppService.CompanionAppService.ProfileFromJson(String data)
   at EddiCompanionAppService.CompanionAppService.Profile(Boolean forceRefresh)
   at Eddi.MainWindow.companionAppNextClicked(Object sender, RoutedEventArgs e)

If I close EDDI, it then will not start again until I've deleted the credentials.json.  This is on beta 3, I also tried going back to beta 2, but it's the same.  Would I be right in thinking this is a problem with the Companion App, and as such nothing you can do?

This is an error in -b3 that caused NULL entries to be written to the ships.json file.  I've released -b4 that has a fix for this, but you should remove the NULL entry from the ships.json after you upgrade (and before as well, if EDDI isn't starting).

Thanks for your reply.  I've upgraded to beta 4, but it's still happening.  I've deleted the ships.json and EDDI starts fine.  I've been able to log back into the Companion App and it repopulates my ship yard.  All works fine until I quit EDDI, it then doesn't start up again.  I looked at the new ships.json and the null entry is back at the end of the file.
This is a fresh copy of my ships.json after deleting it and reloading EDDI:
{
  "ships": [
    {
      "LocalId": 11,
      "name": null,
      "phoneticname": null,
      "role": "Multi-purpose"
    },
    {
      "LocalId": 2,
      "name": null,
      "phoneticname": null,
      "role": "Multi-purpose"
    },
    {
      "LocalId": 0,
      "name": null,
      "phoneticname": null,
      "role": "Multi-purpose"
    },
    {
      "LocalId": 1,
      "name": null,
      "phoneticname": null,
      "role": "Multi-purpose"
    },
    {
      "LocalId": 3,
      "name": null,
      "phoneticname": null,
      "role": "Multi-purpose"
    },
    {
      "LocalId": 4,
      "name": null,
      "phoneticname": null,
      "role": "Multi-purpose"
    },
    {
      "LocalId": 5,
      "name": null,
      "phoneticname": null,
      "role": "Multi-purpose"
    },
    {
      "LocalId": 6,
      "name": null,
      "phoneticname": null,
      "role": "Multi-purpose"
    },
    {
      "LocalId": 7,
      "name": null,
      "phoneticname": null,
      "role": "Multi-purpose"
    },
    {
      "LocalId": 8,
      "name": null,
      "phoneticname": null,
      "role": "Multi-purpose"
    },
    {
      "LocalId": 9,
      "name": null,
      "phoneticname": null,
      "role": "Multi-purpose"
    },
    {
      "LocalId": 10,
      "name": null,
      "phoneticname": null,
      "role": "Multi-purpose"
    },
    {
      "LocalId": 0,
      "name": null,
      "phoneticname": null,
      "role": null
    }
  ]
}

When I start EDDI after deleting the ships.json, and go look at my ship yard, there is a blank entry at the bottom.  I'm guessing this is the culprit creating a null entry in the ships.json file.

I'm getting the same error in my log as the OP and can confirm that the null entry is still being written to ships.json in beta -b4. I've tried both removing the .json file and editing the .json file, but the null entry keeps getting re-written.
{
"ships": [
{
"LocalId": 6,
"name": "the Windblown Leaf",
"phoneticname": null,
"role": "Exploration"
},
{
"LocalId": 0,
"name": "Ragnarök",
"phoneticname": null,
"role": "Multi-purpose"
},
{
"LocalId": 11,
"name": "Atremis",
"phoneticname": null,
"role": "Bounty hunting"
},
{
"LocalId": 12,
"name": "Prospector",
"phoneticname": "prəsˈpɛktə",
"role": "Mining"
},
{
"LocalId": 13,
"name": "Serenity",
"phoneticname": null,
"role": "Mining"
},
{
"LocalId": 14,
"name": "Sovingaard",
"phoneticname": null,
"role": "Multi-purpose"
},
{
"LocalId": 16,
"name": "the Bird of Prey",
"phoneticname": "ðəbɜːdɒvpreɪ",
"role": "Combat"
},
{
"LocalId": 19,
"name": "Mischief",
"phoneticname": null,
"role": "Multi-purpose"
},
{
"LocalId": 0,
"name": null,
"phoneticname": null,
"role": null
}
]
}
EDDI does not start unless I first delete the entry:
{
"LocalId": 0,
"name": null,
"phoneticname": null,
"role": null
}
and even after I delete the entry EDDI does not say "hello" or "goodbye" when entering or exiting EDDI... Ships.json appears to be getting updated to include the null entry every time that the VA plugin or EDDI.exe is started.

I also noticed that EDDI doesn't say 'Goodbye' anymore.  I just thought Cmdr. McDonald had decided to remove it.

I've just pushed 2.1.1-b5 which should clean up the ships.json automatically and allow it to operate.
And yeah, I removed the 'goodbye' bit because it didn't do a lot other than delay the shutdown of EDDI and it caused confusion when EDDI auto-restarted after an upgrade.

Yes, this fix works perfectly. Thank you!

This is fixed in release 2.2.0.


⚠ Samsung has degraded performance

In 62f1513, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 1704 ms

Resolved: Samsung performance has improved in bfbbb02.


White screen crash on iOS following default example

Version
6.4.3
What platforms are you having the problem on?
iOS
System Version
17.5.1
On what device are you experiencing the issue?
Real device
Architecture
Old architecture
What happened?
Following the provided documentation example and playing an empty, 10s blank .mp4 file I experience a white screen / fatal app crash.
I get a warning message:
MSVEntitlementUtilities: - Process app_name PID[1234] - Group: (nil) - Entitlement: com.apple.mediaremote.external-artwork-validation - Entitled: NO - Error: (nil)

So, I need a capability to use this? What's missing for out of the box support on iOS? I do not yet have an Apple Developer program membership.
Reproduction
n/a
Reproduction

Install with NPM & pod install
Put the default Video, VideoRef example into a new .jsx file
Include it from the app root
Lauch the app
Crash, whitescreen, no video is displayed.

Where I'm not using any React hooks (Class components only) and no TypeScript, I had to change
useRef<VideoRef>(null)
to
useRef(VideoRef).


Feature Request: VSCode Extension "Rename Symbol" rename upstream module parameters

Is your feature request related to a problem? Please describe.
When using rename symbol on a module, the bicep file where that module is referenced does not get its parameter updated and breaks the config and must be found and renamed manually. This can get complicated quickly in larger, more modularized deployments.
Describe the solution you'd like
If a module is referenced by another module in the same project, and a parameter is renamed, that parameter rename should also carry through to the parameter definition in the upstream module. Implementing this relationship should probably also work for "find all references" action to find where the parameter is referenced in upstream bicep files.

Yes, definitely.
This is also related to #658.


Tests sometimes hang

What happened?
Sometimes the tests hang. Here is a log of the problem: https://gist.github.com/Marenz/1ace8c7c0ccf01db70ceee8f767bb6f9#file-different-eventloop-py-L188
Notes:

Thread is always the same when the problem appears
Possibly related to channels / Timer?
Works when using the old event_loop replacement method

What did you expect instead?
.
Affected version(s)
No response
Affected part(s)
Build script, CI, dependencies, etc. (part:tooling)
Extra information
.

This started when merging pull request https://github.com/frequenz-floss/frequenz-dispatch-python/pull/54 introduced a timing issue with tests, making them flaky in amd64 but probably consistently failing in arm64 because the CI runs on qemu, which is extremely sloooooowwww.

Possibly related to channels / Timer?
The channels dependency was bumped in the mentioned PR via the bump of client-dispatch.

The issue seems to be that some condition variable is run in a different loop than the one it was created:
    | RuntimeError: <asyncio.locks.Condition object at 0x7f3d2bac0e50 [unlocked]> is bound to a different event loop

The error seems to always happen (at least the error about using the wrong loop) inside the clean-up code from select(), it might help adding some logging there, like printing a stack trace when the select() was created and when it is being cleaned-up to see if both actions are done in different tests (and different loops).

There seems to be a --setup-show flag that might help checking if we don't somehow have multiple loops overlapping.
Reading a bit more about pytest-asyncio, it seems there have been a few big changes in how this library is supposed to work between 0.21, 0.23 and 0.24. Probably all our code was written using the API 0.21 or older, so maybe issues might be connected to the upgrade to 0.24, although that was done one month ago so I'm not sure it's that likely. But maybe this issue is just surfacing some misconfiguration or misused of 0.24.
Here are official migration guides, it might be worth checking them and making sure we are using pytest-asyncio properly:

https://pytest-asyncio.readthedocs.io/en/v0.24.0/how-to-guides/migrate_from_0_21.html
https://pytest-asyncio.readthedocs.io/en/v0.24.0/how-to-guides/migrate_from_0_23.html

duplicate of #61


Add common release-drafter configuration

Introduce release-drafter across the
organization repositories. This change
should enable use to produce release notes.

No fun on this mypy failure.  I tried repping the issue locally and cannot.  The versions of mypy and mypy-extensions match what the runner uses.
python -m pip freeze | grep mypy
mypy==0.930
mypy-extensions==0.4.3

mypy transformers4rec --install-types --non-interactive --no-strict-optional --ignore-missing-imports
Success: no issues found in 78 source files


New version: LibreWolf.LibreWolf version 125.0.2-1

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Hi @mariobrostech,

This PR's version number 125.0.2-1 has 3 parameters (sets of numbers between dots - major, minor, etc), which is greater than the current manifest's version 123.0-1, which has 2 parameters.

Does this PR's version number exactly match the version reported in the Apps & features Settings page? (Feel free to attach a screenshot.)
(Automated response - build 877)

Yes, the stated version number is correct. Here is a screenshot from Apps & Features on my PC.
As a side-note, can this check please be exempted for this package? LibreWolf uses this versioning scheme, so every release will have this problem until someone reports that the stated version number matches.

Hi @stephengillie,
Thanks for publishing the update! However, you never replied to my request for an exemption of version number checks for this package. Is this something that you'd still be willing to consider? Please see my previous mention of this issue below:

As a side-note, can this check please be exempted for this package? LibreWolf uses this versioning scheme, so every release will have this problem until someone reports that the stated version number matches.

Oh, sorry.

No worries! Is this something you can help with directly, or is there someone else that I should tag who would be able to comment further?

Yes - currently I'm implementing the override list manually.

Great, thanks for your help! Hopefully, this process will be smoother next time then 🙂


[PT Run] Set SelectedIndex to 0 after delayed execution of plugins

Summary of the Pull Request
What is this about:
Note: Not sure what was the initial idea by introducing this conditional setting of SelectedIndex to 0 only if it's not delayed execution plugin.
SelectedItem -> Item from the list which will receive the Action (Enter)
SelectedIndex -> Hovered item from the list
Anyway, for both "normal" plugins and "delayed execution" after getting the results, SelectedItem is set to FirstOrDefault() result from sorted list, but SelectedIndex is not updated after delayed executed plugins finished querying
This only happens with Indexer plugin as it's the only "delayed execution" plugin
What is include in the PR:
How does someone test / validate:
https://github.com/microsoft/PowerToys/issues/12283#issuecomment-877766337
Quality Checklist

[X] Linked issue: #12283
[ ] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[ ] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

@jaimecbernardo So, after discussion, go with as it is now? No logic changes, only align hovered item and item selected in the background...

I think the actual fix should be that it should actually open the option that is selected. Not change the selection, right?


[bazel,otp,manuf] Support immutable ROM_EXT enabling during personalization

This PR has two commits that:

Update ROM_EXT immutable section OTP gen tooling

Consider _rom_ext_start_address when determining the manifest start address to support the common rom_ext binary
Updates the OTP  JSON file with data from an immutable ROM_EXT section only if the immutable ROM_EXT is enabled.

Delay IMMUTABLE_ROM_EXT_EN provisioning to end of personalization

Separate the provisioning of the IMMUTABLE_ROM_EXT_EN OTP field from the rest of the CREATOR_SW_CFG partition
Use the ROM_EXT immutable section OTP gen tool by default when building personalization binary

This PR addresses #24610 partially.
Note: this depends on #24783 and #24789. Only review the last 2 commits.

The CI pipeline was failing (task:CI (Checkout repository) ) for this PR.


Flatten the curve!

flattened
linted
added prevention of premature switch

This change is

Looks cool.. ill test out the docker hub build (branches are built too)

Will test this out soon as I had to apply some commits that was left out
https://hub.docker.com/layers/lncm/neutrino-switcher/branch-leftist-flatland/images/sha256-95378b8f438d921ac50d4b1a613644f47c5505c26d21528584cfd240de8fffdf?context=repo

$ docker logs neutrino-switcher -f
Checking if synced...
If set to neutrino then lets check bitcoind
jq: error: syntax error, unexpected INVALID_CHARACTER, expecting $end (Unix shell quoting issues?) at <top-level>, line 1:
{"result":null,"error":{"code":-28,"message":"Loading block index..."},"id":"switchme"}} | .error
jq: 1 compile error
false
Not pruned
jq: error (at <unknown>): null (null) and null (null) cannot be subtracted
Sync seems complete!  Will switch on next check.

using lncm/neutrino-switcher:branch-leftist-flatland

though it does seem like its working as usual.. If someone could test this out on a full node it would be great

Cool will give this another go


ENYO-5920: Fix Pickers to not error when min > max

Checklist

[X] I have read and understand the contribution guide

[X] A CHANGELOG entry is included

[ ] At least one test case is included for this feature or bug fix

[X] Documentation was added or is not needed

[ ] This is an API breaking change

Issue Resolved / Feature Added
When min exceeded max, EditableIntegerPicker, Picker, and RangePicker would throw a runtime error from ramda's clamp method.
Resolution

Add a clamp method to core/util that safely clamps values
Use core/util.clamp in the Pickers and Slider (which doesn't currently error but should use the same logic)

Additional Considerations
The logic used in clamp mirrors the logic used by <input type="range"> in that it uses min as the value when min exceeds max.

As much as is possible, I think it's best to warn but to fail gracefully. This impl follows how HTML behaves and, as such, seems appropriate. Happy to discuss more!

@teckliew thanks for the note! validateRange warns for that scenario and is already in use by the various pickers so I think we're okay. Thoughts?


Trigger Autocomplete

How can i prefill the autocomplete result in the input box. I am having issues while achieving  this as when i set the model value to text address, the validator is not marking it as valid. How can i resolve it?

Any Help would be appreciated....


AttributeError in MistralClient when using ChatMessage with model_dump()

Environment:
Python version: 3.9
MistralAI package version: 0.0.8

Description:
Encountered an AttributeError when attempting to use the MistralClient.chat() method with ChatMessage objects. The error suggests that ChatMessage objects do not have a model_dump() method, which is expected by the _make_chat_request method in client_base.py.
Code to Reproduce:
import os
from mistralai.client import MistralClient
from mistralai.models.chat_completion import ChatMessage

api_key = 'your-api-key'
model = "mistral-tiny"

client = MistralClient(api_key=api_key)

chat_response = client.chat(
    model=model,
    messages=[ChatMessage(role="user", content="What is the best French cheese?")],
)
print(chat_response.choices[0].message.content)

Expected Behavior:
The chat() method should correctly process the ChatMessage object and return a chat completion response.
Actual Behavior:
An AttributeError is raised:
AttributeError: 'ChatMessage' object has no attribute 'model_dump'

This occurs at the line in client_base.py where it tries to call model_dump() on each ChatMessage object in the messages list.
Steps to Reproduce:
Initialize a MistralClient with a valid API key.
Attempt to send a chat request using the chat() method with a ChatMessage object.
Observe the AttributeError.

I cannot reproduce this.
For me, the chat message is correctly parsed.
Can you check your version of pydantic? It should be: pydantic = "^2.5.2"

Current installed version is pydantic==2.5.2

I get the same error.
Running on google colab,
pip install mistralai~=0.0.8
from mistralai.client import MistralClient
from mistralai.models.chat_completion import ChatMessage

model = "mistral-tiny"

client = MistralClient(api_key=userdata.get('MISTRAL_KEY'))

messages = [
    ChatMessage(role="user", content="What is the best French cheese?")
]

# No streaming
chat_response = client.chat(
    model=model,
    messages=messages,
)

Also tried with the langchain integration and got the same error
pip install langchain~=0.1.0
pip install langchain_mistralai
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
[<ipython-input-22-656503716d17>](https://localhost:8080/#) in <cell line: 14>()
     12
     13 # No streaming
---> 14 chat_response = client.chat(
     15     model=model,
     16     messages=messages,

2 frames
[/usr/local/lib/python3.10/dist-packages/mistralai/client.py](https://localhost:8080/#) in chat(self, model, messages, temperature, max_tokens, top_p, random_seed, safe_mode)
    125     ) -> ChatCompletionResponse:
    126         """A chat endpoint that returns a single response.
--> 127
    128         Args:
    129             model (str): model the name of the model to chat with, e.g. mistral-tiny

[/usr/local/lib/python3.10/dist-packages/mistralai/client_base.py](https://localhost:8080/#) in _make_chat_request(model, messages, temperature, max_tokens, top_p, random_seed, stream, safe_mode)
     35         self._api_key = api_key
     36         self._logger = logging.getLogger(__name__)
---> 37
     38     def _make_chat_request(
     39         self,

[/usr/local/lib/python3.10/dist-packages/mistralai/client_base.py](https://localhost:8080/#) in <listcomp>(.0)
     35         self._api_key = api_key
     36         self._logger = logging.getLogger(__name__)
---> 37
     38     def _make_chat_request(
     39         self,

AttributeError: 'ChatMessage' object has no attribute 'model_dump'

Ah I see that this was fixed on main, don't get the error anymore with
pip install git+https://github.com/mistralai/client-python


this.c.compile is not a function

Hello, I am receiving this error when using mustache-loader.
this.c.compile is not a function
I believe problem is coming with #9 when including only the hogan.js/lib/template file.
Later on, line https://github.com/deepsweet/mustache-loader/blob/master/index.js#L61 you are passing the wrong compiler to Hogan.js, since the current H doesn't have any compiler in it (only exposed in the main file).
function (Hogan) {
  Hogan.Template = function (codeObj, text, compiler, options) {

var Hogan = require('./compiler');
Hogan.Template = require('./template').Template;
Hogan.template = Hogan.Template;
module.exports = Hogan;

return 'var H = require("hogan.js/lib/template");\n' +
           'module.exports = function() { ' +
           'var T = new H.Template(' +
           Hogan.compile(source, { asString: true }) +
           ', ' +
           JSON.stringify(source) +
           ', H);' + suffix;

sorry, reverted, should be ok now.


[PEP] Fix using loadStyle from loadCSS

Fix a fn naming issue caused by code conflicts
Resolves: [Desktop] RTP CCX0137- Direct Path to Product
Test URLs:

Before: https://stage--express--adobecom.hlx.page/express/create/advertisement/cyber-monday
After: https://pep-css--express--adobecom.hlx.page/express/create/advertisement/cyber-monday?martech=off

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (f91d253) 31.95% compared to head (37b69a5) 31.86%.

Additional details and impacted files
@@            Coverage Diff             @@
##            stage     #528      +/-   ##
==========================================
- Coverage   31.95%   31.86%   -0.10%
==========================================
  Files         126      126
  Lines       24107    24107
==========================================
- Hits         7704     7682      -22
- Misses      16403    16425      +22

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


Leverage LazyGhostTrait when possible

This PR builds on https://github.com/doctrine/orm/pull/10045 and on https://github.com/doctrine/orm/pull/6719 (see there for motivations)
It's an experiment to replace Doctrine's proxies by VarExporter's LazyGhostTrait.
It needs https://github.com/nicolas-grekas/symfony/pull/42 to be applied on VarExporter.

As hinted in the description of the PR, Doctrine relies on the fact that setting a property would initialize a proxy. This forces a round-trip to the DB to fetch data that could be updated directly instead. It be great to figure out if this requirement really makes sense, and ditch it of not.

This requirement is a must-have. Without it, change tracking would be broken as Doctrine would not have the old value initialized to compare it on flush.

or at least, it is a must-have with the current implementation of the UnitOfWork.

share your opinion on the PR

The changes in the docs directory are the ones I can understand, and it appears you are lifting a good number of restrictions. Besides, I'm really looking forward to getting rid of doctrine/common, so big 👍 from me :slightly_smiling_face:

Only one failing test remains: ENABLE_SECOND_LEVEL_CACHE=1 ./vendor/bin/phpunit --filter testCloneProxy
The reason of the failure is that when the cloned proxy is initialized, it registers itself in the manager. The previous proxies were initialized specially when cloning. I'm not sure how to adjust the code for the new proxies.


Missing Semicolon after static const generates error

Hi,
in the following code:
public static const RENDER_NORMAL_HEIGHT:int = 74

[Bindable("dataChange")]
public function get product():Product
{
        return data as Product;
}

If I don't add the semicolon after "74" I get the following error:
"Call to a possibly undefined method Bindable."

I just checked how classic Flex SDK compiler behaves, and it produces the exact same error. With that in mind, I believe that the Royale compiler is behaving correctly here.
The opening character [ for metadata probably does not result in automatic semicolon insertion because it might also be intended as array access (even if it's not on the same line).

Hi Josh,
maybe the solution is not about a semicolon (I think that's ok too), but the error seems to me a bug (even if flex worked the same, doesn't seem to me ok). I mean, it's normal to write sentences without semicolons, but the start of a metadata that should be detected and the error ("Call to a possibly undefined method Bindable."), seems to me inapropiate.
do you think this should remain in the current shape?

I think that both compilers are behaving correctly, and we should not change this.
Consider the following example, which I think helps to demonstrate why making any changes here would be problematic.
public static function NotMetadata(value:String):int {
	return 0;
}

public static const MY_ARRAY:Array = [1, 2, 3];

public static const RESULT:int = MY_ARRAY[NotMetadata("foobar")]

This code unambigiously uses array access and calls a static function, named NotMetadata.
However, ActionScript allows us to add some whitespace between MY_ARRAY and the [ character, like in the next example below:
public static const RESULT:int = MY_ARRAY

[NotMetadata("foobar")]

Regardless, we shouldn't treat certain metadata (like Bindable) as a special case because ActionScript allows developers to invent arbitrary new metadata with any name that they want. They'll expect their custom metadata to behave the same, but we cannot guarantee that without breaking valid code, like what I included in my previous example.


Dialogs are truncated and not scrollable when overflowing the browser viewport

When I have a small browser window and open a long more menu, it is truncated and I cannot access the menu points at the bottom.
To Reproduce

Decrease the size of the browser window
Click on a more menu (e.g., of a numerical column)
Open another submenu

Expected behavior
The menu should overflow and show a scrollbar to access the last menu items that are now outside the viewport. Also the submenu should be fully accessible and/or scrollable.
Screenshots

lineup-menu-outside-viewport.webm
Context

Version: v4.6.2
Browser: Chrome 125

Additional context

Done.


Trigger Counts and Single Stat

Describe the bug
The Single Stat plugin doesn't appear to render trigger counts correctly after upgrading to Zabbix to 4.0.x and Grafana and this Zabbix plugin to the latest available. I either see a blank panel or 'undefined' depending on the browser.
Other Single Stat panels backed on to Zabbix (not showing counts, but specific metrics) don't appear to be affected and are working as before the upgrades.
Expected behavior
Previously the Single Stat plugin would show the count of the number of triggers returned by the query.
Screenshots

Network data
This is the return data of one of two affected queries.
{
    "jsonrpc": "2.0",
    "result": "3",
    "id": 1
}

Other Notes
I've tried deleting the panel and adding it back in again but see the same results.

I'm having the same issue after upgrading Zabbix from 4.2.0 to 4.2.1.
The default Zabbix System Status dashboard is no longer displaying the number in the blocks.

Same Issue for upgrading from 4.0.4 to 4.0.7
The API Query returns values, but they are not showed in the Single Stat Trigger counter
Seems like Zabbix changed the API Count Output Datafield in the last Update.
"Fixed api returns "countoutput", "select*": "count" results and "suppressed" property as integer"
https://support.zabbix.com/browse/ZBX-15762
Object
xhrStatus:"complete"
request:Object
method:"POST"
url:"api/datasources/proxy/1"
data:Object
jsonrpc:"2.0"
method:"trigger.get"
params:Object
output:"extend"
hostids:Array[50]
min_severity:4
filter:Object
value:1
expandDescription:true
expandData:true
expandComment:true
monitored:true
skipDependent:true
selectLastEvent:"extend"
selectGroups:"extend"
selectHosts:Array[2]
0:"host"
1:"name"
countOutput:true
applicationids:Array[50]
lastChangeSince:1546297200
lastChangeTill:1556083617
id:1
auth:"7e67168c67c0fd46dbce38e19783e4aa"
response:Object
jsonrpc:"2.0"
result:"1"
id:1

Yes, now Zabbix returns result field as a string (was an integer number previously).

Can the commit be released in a 3.10.3 update? Seems like it just missed out on 3.10.2.

Hello Guys
I'm having the same problem but I don't understand how to fixed, can anyone explain in more details the solution, by the way I'm a System Admin :)
Software Version
Zabbix Grafana Zabbix-Grafana-Plugin
4.0.7       6.1.6    3.10.2

Well, but its explained here for Sys Admins -> you cant fix it until the next update is released, because its a bug in the current version, in cause of the change in the latest Zabbix Version you running.

@alexanderzobnin 2 months have passed since the patch was committed to master. Could a new release be pushed out so that this plugin will work with current versions of Zabbix?

I am tested new pre release  verison zabbix 4.2.2rc1, still has problem


Doesn't support the latest version of transformers. IndexError: tuple index out of range

Hi! I tried to install the program on Mac M2  and found the error while using fresh python3.11 and transformers 4.34.0 as well as tokenizers 0.14.1. When I ran pipeline.py I got the next error, I also printed outputs variable and its type from model.py file:
python pipeline.py --model models --input tests/data/raw.txt --output out.json
Loading product extractor from models/prod...Some weights of the model checkpoint at models/prod were not used when initializing BertCRFForTagging: ['bert.pooler.dense.weight', 'bert.pooler.dense.bias']

This IS expected if you are initializing BertCRFForTagging from the checkpoint of a model trained on another task or with another architecture (e.g. initializing a BertForSequenceClassification model from a BertForPreTraining model).

This IS NOT expected if you are initializing BertCRFForTagging from the checkpoint of a model that you expect to be exactly identical (initializing a BertForSequenceClassification model from a BertForSequenceClassification model).
done
Loading role extractor from models/role...Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
Some weights of the model checkpoint at models/role were not used when initializing BertCRFForRoleLabeling: ['bert.pooler.dense.weight', 'bert.pooler.dense.bias']

This IS expected if you are initializing BertCRFForRoleLabeling from the checkpoint of a model trained on another task or with another architecture (e.g. initializing a BertForSequenceClassification model from a BertForPreTraining model).

This IS NOT expected if you are initializing BertCRFForRoleLabeling from the checkpoint of a model that you expect to be exactly identical (initializing a BertForSequenceClassification model from a BertForSequenceClassification model).
done
BaseModelOutputWithPoolingAndCrossAttentions(last_hidden_state=tensor([[[-7.2273e-01, -1.3122e-01, -1.1214e+00,  ...,  1.3058e+00,
-7.9485e-01,  1.7867e+00],
[-7.0862e-01,  8.6485e-01, -8.8587e-01,  ..., -9.6626e-01,
1.1189e+00,  8.2820e-01],
[ 4.5367e-01, -1.0321e+00, -4.2794e-02,  ..., -1.1382e-01,
-6.6842e-01, -2.5176e-01],
...,
[ 4.6953e-01, -4.4274e-01, -2.4798e-01,  ..., -7.2705e-02,
1.0744e-01, -6.4336e-01],
[ 2.9490e-01, -6.9888e-01,  4.3615e-01,  ...,  2.6774e-01,
-2.0582e-01, -5.2261e-01],
[ 3.9034e-01, -4.5327e-01,  7.8649e-02,  ...,  9.6617e-02,
1.2183e-01, -6.4031e-01]],
  [[-7.0996e-01, -1.4535e-01, -1.1391e+00,  ...,  1.2885e+00,
    -7.9856e-01,  1.7955e+00],
   [-6.9886e-01,  8.5938e-01, -8.7897e-01,  ..., -9.6362e-01,
     1.1165e+00,  8.3504e-01],
   [ 4.8238e-01, -1.0400e+00, -3.7558e-02,  ..., -1.3158e-01,
    -6.7545e-01, -2.3461e-01],
   ...,
   [ 3.2453e-01, -4.9036e-01, -6.7992e-02,  ...,  2.5228e-01,
    -3.6437e-01, -5.2642e-01],
   [-1.5090e-01, -5.1699e-01,  8.8815e-01,  ..., -6.2315e-02,
    -5.7753e-02,  3.7765e-01],
   [ 2.5326e-01, -7.0302e-01,  4.8222e-01,  ...,  3.4928e-01,
    -3.8039e-01, -3.5345e-01]],

  [[-7.1101e-01, -1.8783e-01, -1.1474e+00,  ...,  1.3191e+00,
    -8.0083e-01,  1.8080e+00],
   [-6.7759e-01,  8.8157e-01, -8.7389e-01,  ..., -9.1561e-01,
     1.0664e+00,  8.2624e-01],
   [ 2.9864e-01, -1.1082e+00, -6.4317e-02,  ..., -4.3113e-01,
    -6.0993e-01, -3.3053e-04],
   ...,
   [ 6.3764e-01, -6.6906e-01,  6.6547e-02,  ...,  3.8974e-01,
    -1.4845e-01, -3.4640e-02],
   [ 6.1456e-01, -7.3927e-01,  2.4568e-02,  ...,  5.0147e-01,
    -1.6768e-01, -5.0775e-02],
   [ 8.0470e-01, -8.2970e-01,  3.1760e-01,  ...,  2.6904e-01,
    -2.7527e-01,  7.3508e-02]],

  ...,

  [[-6.5839e-01, -1.3844e-01, -1.1229e+00,  ...,  1.3440e+00,
    -8.8502e-01,  1.8236e+00],
   [-6.7542e-01,  8.9506e-01, -8.3126e-01,  ..., -9.7451e-01,
     1.0457e+00,  8.9183e-01],
   [ 5.9978e-01, -8.9523e-01, -4.6267e-01,  ..., -6.4620e-01,
    -1.1831e+00, -4.5417e-02],
   ...,
   [ 5.6698e-01, -8.3520e-01, -5.7488e-02,  ...,  9.3281e-02,
    -4.7615e-01, -6.3753e-01],
   [ 5.2334e-01, -8.4141e-01, -5.8037e-02,  ...,  2.9975e-01,
    -4.9307e-01, -5.2515e-01],
   [ 5.3504e-01, -8.5081e-01, -2.9094e-02,  ...,  2.7717e-01,
    -5.0541e-01, -5.6720e-01]],

  [[-6.4393e-01, -1.1897e-01, -1.1036e+00,  ...,  1.3327e+00,
    -8.7286e-01,  1.8048e+00],
   [-6.6012e-01,  9.5893e-01, -8.2026e-01,  ..., -9.3506e-01,
     1.0918e+00,  8.8813e-01],
   [ 2.1059e-01, -6.8708e-01, -1.2977e-01,  ..., -9.9983e-01,
    -5.8769e-01,  1.2027e-01],
   ...,
   [ 9.4669e-01, -4.1373e-01, -1.5878e-01,  ..., -5.9621e-02,
    -3.9900e-01, -2.0323e-01],
   [ 5.5884e-01, -6.9609e-01, -1.5338e-01,  ...,  4.2257e-01,
    -4.0991e-01, -2.6131e-01],
   [ 6.7532e-01, -6.0004e-01, -1.8916e-01,  ...,  2.6614e-01,
    -3.2382e-01, -1.0976e-01]],

  [[-6.4499e-01, -1.1385e-01, -1.1036e+00,  ...,  1.3126e+00,
    -8.9420e-01,  1.7905e+00],
   [-6.6014e-01,  9.2368e-01, -8.3718e-01,  ..., -9.3139e-01,
     1.0758e+00,  8.7325e-01],
   [ 6.0877e-01, -9.7736e-01, -2.6773e-01,  ..., -6.1211e-01,
    -8.7993e-01, -1.7766e-01],
   ...,
   [ 1.1020e+00, -2.3823e-01, -1.2828e-01,  ..., -1.8427e-02,
    -2.1446e-01,  6.2169e-02],
   [ 1.1225e+00, -1.1497e-01, -3.5708e-01,  ..., -1.2119e-01,
    -2.2195e-02,  3.4493e-01],
   [-1.9072e-01, -5.9194e-01,  9.7846e-01,  ...,  7.0911e-02,
    -2.7881e-01,  5.8151e-01]]]), pooler_output=None, hidden_states=None, past_key_values=None, attentions=None, cross_attentions=None)

<class 'transformers.modeling_outputs.BaseModelOutputWithPoolingAndCrossAttentions'>
Traceback (most recent call last):
File "/Users/alekseikrasov/Desktop/OntoChem/work/ChemRxnExtractor/ChemRxnExtractor/pipeline.py", line 22, in
rxns = rxn_extractor.get_reactions(sents[:10])
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/alekseikrasov/Desktop/OntoChem/work/ChemRxnExtractor/ChemRxnExtractor/chemrxnextractor/cre.py", line 224, in get_reactions
outputs = self.role_extractor(
^^^^^^^^^^^^^^^^^^^^
File "/Users/alekseikrasov/miniforge3/envs/ontochem/lib/python3.11/site-packages/torch/nn/modules/module.py", line 1518, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/alekseikrasov/miniforge3/envs/ontochem/lib/python3.11/site-packages/torch/nn/modules/module.py", line 1527, in _call_impl
return forward_call(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/Users/alekseikrasov/Desktop/OntoChem/work/ChemRxnExtractor/ChemRxnExtractor/chemrxnextractor/models/model.py", line 321, in forward
extended_cls_h = outputs[1].unsqueeze(1).expand(batch_size, seq_length, hidden_dim) # FIXME: here is the errer causing IndexError: tuple index out of range in
~~~~~~~^^^
File "/Users/alekseikrasov/miniforge3/envs/ontochem/lib/python3.11/site-packages/transformers/utils/generic.py", line 405, in getitem
return self.to_tuple()[k]
~~~~~~~~~~~~~~~^^^
IndexError: tuple index out of range
I tried to use the old version of transformers v.3.0.2, however, I cannot compile tokenizers==0.8.1.rc1 (from transformers==3.0.2) with existing Rust compiler. The same problem was on Linux Opensuse machine.
Could you please help to solve this problem and maybe update the code and requirements so that one could use the latest version of Python, transformers, and tokenizers?

I have met the same problems. Could there is a solution?

Unfortunately, the installation on Mac with ARM architecture failed, however, I managed to install and run it on a Linux machine with the required dependencies. The only thing I could recommend is using Python<3.9 on Linux with the rest of the installation instructions.

Hi @LingjieBao1998, I have found the solution for Mac users with Apple silicon. You can follow next instruction to install ChemRxnExtractor:
1. Install compatible conda with either Miniforge or Anaconda(Miniconda). We recommend using Miniforge.
wget https://github.com/conda-forge/miniforge/releases/latest/download/Miniforge3-MacOSX-arm64.sh
bash Miniforge3-MacOSX-arm64
and follow instructions.
if you have installed Anaconda(Miniconda) update it to the latest version:
update the conda package manager to the latest version
conda update conda
use conda to update Anaconda to the latest version
conda update anaconda
check the version of Anaconda. The 2022.05 release of Anaconda Distribution features native compiling for Apple M1’s ARM64 architecture.
Set conda-forge as prior channel:
conda config --add channels conda-forge
conda config --set channel_priority strict
2. Create environment and install essential libraries. Python lower than  3.10 works fine.
conda create --name ENV_NAME "python<3.10"
conda activate ENV_NAME
pip install install pyproject-toml torch tqdm numpy seqeval -U
It’s important to use channel conda-forge for installation next version of tokenizers and transformed:
conda install -c conda-forge tokenizers==0.10.1
conda install -c conda-forge transformers=3.0.2
git clone https://github.com/jiangfeng1124/ChemRxnExtractor
cd ChemRxnExtractor
3. In file requirement.txt comment installation of

torch
transformers.

pip install -r requirements.txt
pip install -e .


Fix Host typo

@osvyrydenko : Thanks for your contribution! The author(s) have been notified to review your proposed change.


peerUrlTls removed from etcd CR when shoot hibernated

How to categorize this issue?

/area control-plane high-availability
/kind bug
What happened:
We noticed an issue that etcd.Spec.Etcd.PeerUrlTls was removed from the ETCD CR when shoot is hibernated and is re added when the shoot is then woken up.
This causes an issue that when a shoot is marked for deletion while hibernated, etcd is scaled up without adding etcd.Spec.Etcd.PeerUrlTls and etcd members throw up the following error
2023-02-15 14:20:31.492960 W | rafthttp: health check for peer dda1e1b5de5194a3 could not connect: tls: first record does not look like a TLS handshake

What you expected to happen:
etcd.Spec.Etcd.PeerUrlTls to always be present in the ETCD CR
How to reproduce it (as minimally and precisely as possible):

Create an HA shoot with failure tolerance zone
Hibernate the shoot
Mark the shoot for deletion while the shoot is hibernated

Anything else we need to know?:
Environment:

Gardener version: v1.64.0
Kubernetes version (use kubectl version):
Cloud provider or hardware configuration:
Others:

Related code: https://github.com/gardener/gardener/blob/e74cc7cc70de5a78bc2f0268a8178c283f8f69f8/pkg/operation/botanist/component/etcd/etcd.go#L463-L478

Also noticed that the peer secrets for etcd are being deleted when the shoot is hibernated. This is not the case for client secrets, which remain when a shoot is hibernated

Also noticed that the peer secrets for etcd are being deleted when the shoot is hibernated. This is not the case for client secrets, which remain when a shoot is hibernated

Related code: https://github.com/gardener/gardener/blob/e74cc7cc70de5a78bc2f0268a8178c283f8f69f8/pkg/operation/botanist/component/etcd/etcd.go#L924-L928 and ff.

Thanks @rfranzke and @aaronfern 🙂
The code misses to check if the peerTLS is already set and should not unset it if that's the case. Do you want to work on a fix @aaronfern?

Yes, I'd like to work on a fix for this 🙂

/assign @aaronfern

Until we have this fix: Is there a workaround to resolve this situation for an existing cluster?

@aaronfern And would it be fine to als create a hotfix once we have the fix? Thanks!


Update dxvk.conf

Feel free to use these values as local defaults for all applications or only specific applications, as described here. Your PR attempts to change global defaults however, which isn't a good idea.

Besides that, when you're making a PR to an open source repo, you need to actually describe what your changes do and why. Same goes for the commit message.


Final pass over tick events for hour of code, educator, and game jam pages

new tick events across the three pages:
Hour of Code

hourofcode2021.video.open
hourofcode2021.video.close

Educator Page

hourofcode2021.educator.learnmore.link
hourofcode2021.educator.learnmore.button
hourofcode2021.educator.forum

Game Jam

hourofcode2021.gamejam.submit
hourofcode2021.gamejam.skillmap
hourofcode2021.gamejam.hoc
hourofcode2021.gamejam.training
hourofcode2021.gamejam.box

@soniakandah @ElizabethRiffle please let me know if you add any more links or buttons (anything the user can click), otherwise i think we're set on telemetry!
for https://github.com/microsoft/pxt-arcade/issues/4179, will close the issue after this change goes in and i classify the new tick events

Looks good to me!


Potions are converted incorrectly from 1.8 to 1.9

What version of ViaVersion are you using? Type /ver viaversion
0.8.4
What version of Spigot are you using? Type /ver
git-Spigot-5f38d38-12698ea (MC: 1.8.8) (Implementing API version 1.8.8-R0.1-SNAPSHOT)
How does this error happen?
1.8 uses a byte flag based system to manage potion effects, it's not based on a fixed key or id, so the same potion effect can be obtained from many different data values.
For example you can get the same fire resistance potion on 1.8 with:
/give @p 1 3
/give @p 1 8227
/give @p 1 12323
/give @p 1 10275
/give @p 1 11299

And there are many more possibilities to get the exact same effect with different data values as long we don't change some specifics bits.
That's why it's incorrect to assume that only 8227 must be converted to fire_resistance.
This incorrect conversion affected my server because I use the Bukkit's potion API to create potions and it does not generate the same potion ids as the creative tabs.
This is an example of valid item stack on 1.8 that is converted incorrectly to 1.9, it's shown as a normal water bottle on 1.9.
import org.bukkit.potion.*;
ItemStack brokenOn1_9 = new Potion(PotionType.SPEED).toItemStack(1);

I've already fixed it in a branch and I'm going to send a pull request soon :wink:

We only port certain ids as they are the vanilla way of obtaining them. I mean if you have a better way to convert them that isn't ugly and gives same results, that's also cool. :)


chore: constants extraction and logic change

With this PR I aim to:

Speed it up a bit by replacing the endsWith and startsWith with faster check by index
Remove useless async on functions that doesn't use await

Good catch with spotting those redundant async's!


Fix custom token adding

Custom token addition is broken in 4.11.7 release

@patitonar it looks like a blinking test. It works fine locally. I have no clue for now why it doesn't work in CI.

298 passing (8m)
14 pending

https://github.com/poanetwork/nifty-wallet/issues/296


Batch Job/ Merge Question

Hey @babreu-ncsa,
I was experimenting with the batch job workflow for the smokeppe data set and Jupyter gave me an error indicating that it could not find the necessary function/types for this (ex. smokePPEGaussianMLTrainingJobOptions ). Based on the error, it gave me I tried to merge the file folder for these functions (smokePPEGaussianML) into dev/smokeApp/rc1 so jupyter could see it but I ran into trouble doing this as I am still learning version control. Could you help me with this merge or let me know if this is not necessary?
This is the error I was getting in the notebooks: Type 'smokePPEGaussianMLTrainingJobOptions' not found for tenant 'smokeApp' tag 'devsmokeApprc1'
This is the note book I was using: https://devsmokeapprc1-smokeapp.devrc01.c3aids.cloud/jupyter/notebooks/sandbox/Batch Trials/Batch Trial 1.ipynb
Thank you in advance,
Victor

Hi @vasanchez16, thanks for reporting it! I found a couple of things that need to be adjusted before this works. Do you have time for a zoom call today or tomorrow?

Hey @babreu-ncsa,
I have all my classes today but I can meet anytime tomorrow before 2PM EST or after 4PM EST. Let me know what works for you.

Hi @vasanchez16, sorry for the delay in getting back to you about the meeting. There were two small issues that I fixed; it should be working now:

Types need to start with a capital letter. I renamed all files and types.
The code was never provisioned to the tag, so the c3 server would not know about these new types. Provisioning sends the code to the remote c3 server and makes it render the new version, effectively implementing your new definitions. To do that, you must have a pull request open from the development branch (in this case, smoke-dev) into the target tag branch (in this case, dev/smokeApp/rc1). PR #29 is one example of that. Having the PR open will trigger the provisioning workflow when you push commits from your local machine into the remote (GitHub-based) smoke-dev using git push.

Let me know if you have any questions.

Hi @babreu-ncsa,
Great, thanks so much! I'll get back to you if I have any questions.


[IOTDB-842] Better Export/Import-CSV Tool

Description
Update the Export/Import-CSV tool.
Import common-csv
Using common-csv to do CSV file reading and writing.
Using Java stream
Using Java stream to simplify code.
The New Import Tool
The new import tool support 4 data formats. Including the data aligned by time or device, with or without data type of headers. Besides, you can specify a directory to save file which save failed lines.
The New Export Tool
The new export tool can print data type of timeseries in the head line of CSV file. Besides, you can directly specify a query comman that you want to execute by setting parameter.

Coverage decreased (-0.04%) to 67.245% when pulling 02729fbc4adf2d7fd1383935d4ed903b5289e067 on xuanronaldo:xuanronaldo into 8ec3e12865699064958e36a37a50083d85e11f9c on apache:master.

Ok, I got it.
发自我的iPhone
------------------ Original ------------------
From: Xiangdong Huang @.>
Date: Wed,Aug 25,2021 11:16 PM
To: apache/iotdb @.>
Cc: Xuan Ronaldo @.>, Author @.>
Subject: Re: [apache/iotdb] [IOTDB-842] Better Export/Import-CSV Tool (#3820)
@jixuan1989 commented on this pull request.
In cli/src/test/java/org/apache/iotdb/tool/integration/ImportCsvTestIT.java:
> @@ -16,9 +16,10 @@   * specific language governing permissions and limitations   * under the License.   */ -package org.apache.iotdb.tool; +package org.apache.iotdb.tool.integration;
this IT should be moved to cross-test module.
It is because cli module has no dependency of iotdb-server (in my memory).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.


IIOP subsystem graceful shutdown + library update

Linux Build 80 is now running using a merge of 170d3b0c17f92606b25c36761bcd0f0b865b6a20

Linux with security manager Build 76 is now running using a merge of 170d3b0c17f92606b25c36761bcd0f0b865b6a20

Windows Build 83 is now running using a merge of 170d3b0c17f92606b25c36761bcd0f0b865b6a20

Linux with security manager Build 76 outcome was SUCCESS using a merge of 170d3b0c17f92606b25c36761bcd0f0b865b6a20
Summary: Tests passed: 827, ignored: 317 Build time: 0:05:46

Windows Build 83 outcome was SUCCESS using a merge of 170d3b0c17f92606b25c36761bcd0f0b865b6a20
Summary: Tests passed: 2865, ignored: 372, muted: 4 Build time: 0:46:24

Linux Build 80 outcome was SUCCESS using a merge of 170d3b0c17f92606b25c36761bcd0f0b865b6a20
Summary: Tests passed: 2869, ignored: 372 Build time: 0:48:07


Pachyderm REPL

Capturing an offline discussion. We'd like to make exploratory data analysis smoother in Pachyderm. Even with Update Pipeline and jobs, fast iteration is a pretty kludgy process -- write some code, build a container, and update the JSON pipeline spec.
One idea for making this better is something like Pachctl Shell, pachctl sh. Basically, a REPL to interactively  explore data in repo(s) and then an smooth process for capturing that and "productionizing" it into a pipeline. Tools like iPython or SQL notebooks might be a good interface to support/mimic.

@JoeyZwicker this is done now, no?

No, this is not done. I renamed this issue because the Pachctl Shell (which does something pretty different) is feature we just released and changing to REPL is an attempt to differentiate. #3709 has additional information of the use case for this.


[Enhancement]: Create a Staging branch and add version tags

Contact Details
<EMAIL_ADDRESS>What this feature/enhancement tries to solve?
This issue intends to solve the problem of the website review before pushing to prod. As of the moment, the branch main serves as the staging, although it is in constant change, making the review process a little hectic.
Also, it would make the process much cleaner by tagging the main branch so we can version the website accordingly.
If want to provide us a more details about how to implement.

Add a new staging branch.
Add tags for the website releases.
Add this branch to the CodePipeline that publishes the site, with its own URL.

Relevant documentation
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

We've been discussing getting a better flow for the site reviews, and we discouraged the addition of a new branch for this purpose. Another potential solution would be tagging the main branch.


Visual Studio: Definitions don't update in hover panels when the defintions file is updated.

Describe the bug
In most IDEs when editing the defintions file, the updated definitions appear in the hover panel immediately.  In Visual Studio, they don't.  This is because Visual Studio supports didChangeWatchedFile notifications, but it doesn't support dynamic registration, which is an approach where the language server can nominate which files to watch.  See https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#didChangeWatchedFilesRegistrationOptions for details.
To Reproduce
Open a folder with a .contextive/definitions.yml file, in Visual Studio with Contextive installed.  See the hover panel text.  Modify a term definition. Hover to see the hover panel, and see the old definition.
Expected behavior
The hover panel should show the new definition.
Additional context
According to the initialize logs, the language client advertises support for didChangeWatchedFiles with an empty object, which means it doesn't offer dynamic registration.
{
   ...
   "didChangeWatchedFiles": {}
   ...
}

Tracked in Visual Studio Developer Community here


add table main menu command #3778. Reset all

kick jenkins

does not reset sort column

sort reset was fixed. also sorting info is saved to state now

still not working for me.
on the r-examples i click a column to sort, then reset has no visible effect and no error.

:+1:

kick jenkins

looks like you merged when i had a bad commit in master...
please merge again.  sorry my fault.

bring up to date please

done


fix race condition w/ start-time, use correct event for kernel idle, and...

... fix css loading

execution end time was reliant on a non-existent kernel event, probably from a previous version
CSS file now loaded from a more dynamic path
Execution start time now getting recorded after the cell execution rather than before it (not a huge issue since execution is delegated to kernel, but I believe recording from before makes more sense)

Thanks for the PR. Please use the 3.x branch to make your PR, as the master branch is not kept up-to-date, at least as long the big split of IPython/Jupyter is ongoing.
Some of your changes should already be in the 3.x branch.

Ah!  I see, I had mistakingly thought that master followed 3.x, I didn't see those changes.  Looks like my changes are roughly the same, I'll close this PR.
Thanks!


Support external diff tools in place of the CPython difflib approach

The Python diff approach will be slow for large files.  It would be helpful to be able to use diff -u and similar executables for the diff execution.

#14


ci: macOS job - use Xcode 15

@canepat 👌 implemented here: https://github.com/erigontech/silkworm/pull/1928

It's OK for me to bump the Xcode version in CI job, but I would pair this change with a proper fix for the build regression introduced by #1926 surrounding add_link_options with a check on compiler version:
  if(CMAKE_CXX_COMPILER_VERSION VERSION_GREATER_EQUAL 15)
    # https://stackoverflow.com/questions/77164140/
    add_link_options(-Wl,-no_warn_duplicate_libraries)
  endif()


XAPTC-62  DELETE a Test Case

-added unit tests

All changes implemented

Closing for Camelia's work


Create an Ingress controller for Tempus on k8s

Child of #361
We need an ingress controller to expose NiFi and Tempus on one load balancer

Completed.


Add detail page for presos

Fixes #51.
Solution
This PR adds a details page for presos.
Testing
⚠ If you don't have any presos created, you'll want to create at least 1 first. ⚠

Start ngosi, yarn start
Navigate to the Presos page, http://localhost:3000/presos
Click on one of the presentations
Observe that you're taken to a page with a URL similar to http://localhost:3000/preso/kb-lCfk

Resources
Preso detail page

@Yohanan-Negash

The only issue I am facing right now is adding/creating presos. When I click on the Generate QR Code button it seems like there is no event happening, so I can't navigate to the preso details page.

I was able to reproduce this. Perhaps we can pair and you walk me through it?


A new prop for searchText?

I need a way to query options based on what searchText currently is. As the user is typing.
Can we use the DOM input element value?
Do we need a new prop to bind?
What are your thoughts on this?

And since we're talking about it, a debounce option for changing this text would be great.
Something like:
import { debounce } from "lodash-es";

const newSearchText = debounce((val) => (searchText = val), 1000);

I know I can do it, but having a prop like:
debounce={1000}

is unvaluable.

I agree it makes sense to expose the user-entered search text. And the open/close state of the options dropdown as well, while we're at it:
+ export let searchText = ``
+ export let showOptions = false

Just to be clear re debouncing, you want to prevent the user from updating the value several times a second while he's typing?

you want to prevent the user from updating the value several times a second while he's typing?

Yep. Just so as not to clog the server with requests.

I think it that case it makes more sense to debounce your requests rather than the prop, no? Would that a problem for some reason?
I'm a bit worried about UX suffering from it being less snappy for devs not doing server requests with it (which I assume would be the majority).

Yeah. You're right. Not something to debounce in SMS.


Use current account as funding method by default when creating a new account & re-use recovery key

Original issue: https://github.com/near/near-wallet/issues/2284
We implemented phase one of the implicit account onboarding in https://github.com/near/near-wallet/pull/2313. Phase two includes:

Using current account as funding method by default when creating a new account (while logged in)
Enabling users to re-use the same seed phrase / key when creating a new account, essentially making it possible for users to add the same key to multiple accounts.

Figma design: https://www.figma.com/file/yR4qz7oAjAiVAJ0hJSnQQj/Onboarding-Exploration?node-id=1%3A606

@MaximusHaximus and @corwinharrell - the UI for this feature is done, but I know @MaximusHaximus has some concerns with shipping this 'shared seed phrases' feature, at least without any ability for users to manage multiple accounts with a single phrase outside of the initial setup flow. @corwinharrell any thoughts on this?

I think it's a valid approach to hold off on the shared phrase stuff until we can also design and plan for better key management, so that a user has the following options:

See which accounts are secured by which keys
See which accounts share keys
Easily rotate keys, change keys, etc...
Have messaging around actions that ensure the user understands the implications of changes to keys


I want to update README.md file

Key points that will enhance the README file include installation instructions, a usage guide, visual representations, API documentation, and a section on future features and improvements.

What changes are you planning to do?

I want to include
project purpose and vision
Preview Link
Future Improvements
Changelog- track progress and version changes


Support request timeout

Hi!
It would be cool if we could support HTTP request timeouts like so:
https://github.com/hyperium/hyper/blob/ce3dbd123d9cd10f4f1e09392ba6f3e4ac949d98/tests/client.rs#L252
It's a minor thing but might be worth adding.
Thanks for your library!

That sounds useful! I can take a look at it in a week or so as I'm away from my development machine right now. Or you could send a pull request and I'll take a look right away.

I've pushed a new commit to add this feature, and updated the documentation. Feel free to reopen if your issue is not solved!

Sweet. Thanks!


Perfdash label refresh

This change will prevent from changing selected label values due to data refresh.
Labels will be reset only if needed.

/assign shyamjvs

/lgtm
Thanks for this much-needed fix!


Drag the scroller out of browser window

Hello, Malihu,
The problem happens when I drag the scroller and move the mouse out of the browser window and release it. When I move the mouse back to the browser, the scroller still moves according to the mouse movement until I click the mouse again.
Is there a way to prevent the scroll event from propagating, when I move the mouse back to the browser?
I've tried this, but didn't work.
$(scroll).mouseenter(function (event) {
   $buttonsDiv.mCustomScrollbar("stop");

}).mouseleave(function () {
    $buttonsDiv.mCustomScrollbar("update");

});
Thank you.

Hi,
I can't seem to reproduce this. Can you provide more info (browser, OS, option parameters used etc.) or a test page/code?

It happens on IE (I tried on IE 11). On Chrome works fine.
I used one of your horizontal scrollbar examples.
Thank you.

Doesn't seem to happen when I test the main demo on IE11 (in Windows 7).

I tested with Windows 8... it only happens once in a while.
Thanks anyway :)

I still can't reproduce this issue (tested it on latest IE/Win 8.1). Maybe some application in your windows  installation gained focus the moment you release the dragger(?)
In any case, this seems like an issue out of the plugin scope or something that can't be solved within the script.


Enable Schema Evolution

Is there a way to enable schema evolution? I noticed that when adding additional columns which are not attribute columns those columns are not added to the schema after the type_2_scd_upsert.

Thanks for opening this issue.  I'm actually not sure what type of schema evolution you'd like to enable here.  Can you please send me your code snippet, so I can reproduce on my end and understand better?

I hope this clarifies it:
import mack
from pyspark.sql.types import *
import datetime

# suppose you have some initial data with the following schema
initial_schema = StructType([
    StructField("pKey", StringType(), False),
    StructField("attr1", StringType(), True),
    StructField("attr2", StringType(), True),
    StructField("is_current", BooleanType(), False),
    StructField("effective_time", TimestampType(), False),
    StructField("end_time", TimestampType(), True),
])

initial_df = spark.createDataFrame([
    ("1", "A", "A", True, datetime.datetime.fromisoformat("2019-01-01"), None),
    ("2", "B", "B", True, datetime.datetime.fromisoformat("2019-01-01"), None),
    ("4", "D", "D", True, datetime.datetime.fromisoformat("2019-01-01"), None),
], initial_schema)

initial_df.write.mode("overwrite").option("overwriteSchema", True).save("dbfs:/FileStore/ch/mack/testtable/")

# Now new data arrives with a new column added. After the upsert i would like to see this column also added to the schema of the table (even if its not added to attr_col_names)
additional_column_schema = StructType([
    StructField("pKey", StringType(), False),
    StructField("attr1", StringType(), True),
    StructField("attr2", StringType(), True),
    StructField("foo_column", StringType(), True), # new column was added
    StructField("effective_time", TimestampType(), False),
])

additional_column_df = spark.createDataFrame([
    ("2", "Z", None, "Foo", datetime.datetime.fromisoformat("2020-01-01")), # upsert data including new column
    ("3", "C", "C", "Foo", datetime.datetime.fromisoformat("2020-09-15")), # new pkey
], additional_column_schema)

mack.type_2_scd_upsert(test_path, additional_column_df, "pkey", ["attr1", "attr2"])

# However the new column is not added to the schema
spark.read.load(test_path).show()

@MrPowers I agree that enabling schema evolution would bring value.
I see three options that we have to solve this.

AutoMerge schema by default
Add a flag to method whether to merge the schema or not
Provide a new method -> something like scd2_with_schema_evolution

WDYT?

So it sounds like if spark.conf.set("spark.databricks.delta.schema.autoMerge.enabled", "true") is set and the user runs mack.type_2_scd_upsert with additional_column_df, then the existing implementation will just ignore the additional columns of data.  I think the first step is to update the existing implementation to error out with a descriptive message, something like: "The type_2_scd_upsert can only be run with DataFrames that contain col_a, col_b, and col_c.  You supplied a DataFrame with col_a, col_b, col_c, and col_d."
Let's get this check in the code first and then brainstorm next steps.  Perhaps ALTER TABLE ADD COLUMN could work if that's supported by OSS Delta Lake.

@MrPowers ah, okay I though that it is about us setting spark.conf.set("spark.databricks.delta.schema.autoMerge.enabled", "true").
Why does Schema Evolution not work when setting this configuration though?

@robertkossendey - we should start by setting spark.conf.set("spark.databricks.delta.schema.autoMerge.enabled", "true"), running the user example, and see what happens 🤓
Then we should see if we can run ALTER TABLE ADD COLUMN, run the user example again, and then see what happens.
I wonder if there are any other work-arounds that we should consider to get this working...

@MrPowers 👍🏼, you are right, seems to be unstable

@MrPowers I think we currently don't even allow for new columns to be added because of this line

@robertkossendey I noticed in the documentation that "Schema evolution occurs only when there is either an updateAll (UPDATE SET *) or an insertAll (INSERT *) action, or both." whenNotMatchedInsert which is used  at the moment does not support schema evolution.


terra-form-select 'search' variant does not focus when there is a large number of options

Bug Report
Description

When the terra-form-select 'search' variant has a large number of options (400+), the menu is not focused correctly which results in the dropdown remaining open when clicking on other fields.
Steps to Reproduce

Create a search select field with 400+ options
Click the dropdown arrow to open the dropdown
Click elsewhere on the screen - the dropdown does not close because it was not focused
*Happens most frequently on the first opening of the dropdown

Additional Context / Screenshots

Video:
https://user-images.githubusercontent.com/85512188/121061339-bb361080-c791-11eb-9124-dd06a25dbb71.mov
Expected Behavior

The dropdown should still be focused even if there is a large number of options.
Possible Solution

The menu is not yet painted to the DOM when trying to focus here https://github.com/cerner/terra-core/blob/main/packages/terra-form-select/src/shared/_FrameUtil.js#L150. In other words, that 100 ms timeout is not long enough. One solution I have noticed to fix the issue is adding a requestAnimationFrame inside the timeout, around the frame.selectMenu.focus();.
Environment

Component Name and Version: terra-form-select 6.29.0
Browser Name and Version: Chrome 91.0.4472.77
Node/npm Version: [e.g. Node 8/npm 5] Node 10/npm 6
Webpack Version: Webpack v4
Operating System and version (desktop or mobile): Mac, issue only occurs on mobile

@ Mentions

@supreethmr

From some offline discussion, even though performance can be expected to degrade with an increasingly sized list of options, we shouldn't be left in a "broken" state.
As mentioned above the initial focus is part of the issue, but we should also make sure this cleanup method properly closes the menu in either case:
https://github.com/cerner/terra-core/blob/main/packages/terra-form-select/src/search/Frame.jsx#L217
The useLayoutEffect could be used to great effect to solve this as well. The two locations I seeing being possible are switching the Menu component to hooks, or making a sub-component written in Hooks for the UL, and provide a callback trigger within the useLayoutEffect hook, ensuring the DOM has fulling laid out before we set focus:
https://github.com/cerner/terra-core/blob/main/packages/terra-form-select/src/search/Menu.jsx#L423


Switch e2e tests to a binary

Summary
Currently the knuu e2e tests are part of the go test suite. The problem is that these e2e tests are permissioned. You need to have the kube config setup to run them. We also want them separated from our unit tests and integration tests as they are part of a separate CI workflow. This problem becomes more apparent when we use knuu for performance based tests that we don't want continually running. Since environment variables are set for the lifetime of the terminal, I frequently accidentally run e2e tests when I run make test-short on my branch.
Proposal
I would like to propose that we put all our e2e tests in their own binary. To run them, one would use go run tests/e2e. We would not need to set environment variables
For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

Works for me. Can we also modify the existing make test-e2e command to run that separate binary?

As additional context. I would want the correctness tests run separately from the performance/experimental tests. These should be split into two packages like
go run ./test/e2e // for our correctness based tests i.e. compatibility and upgrades
go run ./test/benchmark // for our standardised benchmarks
go run ./test/experimental/two_node_test // for experimental work
(maybe the last two should be the same for now)

todo: also disable throughput tests and move them to benchmark or experimental


Update Expo domain to expo.dev

Summary
Some time ago, we moved the Expo domain over to expo.dev (see this PR for example). This does the same thing here :)
Changelog

Internal Changed - Update Expo domain to expo.dev
Test Plan

Just a minor change, unrelated to React Native code.

@byCedric please fix the changelog format. Square brackets are required there.


Bottom Navigation and overlay

Hi there is a way to make overlay over the BottomNavigationBar?

Hi, still not yet but we will update soon.


String template literals as object keys don't get validated correctly

🔎 Search Terms
String template literal object keys return type function
🕗 Version & Regression Information

This is the behavior in every version I tried, and there is nothing in the FAQ about it

⏯ Playground Link
https://www.typescriptlang.org/play/?ts=5.4.5#code/MYewdgzgLgBBCGBbADgGwKYEYYF4YAoAPALjAFdEAjdAJwEpiYAldUGgEwB4ADBFDAPoASAN7kqtAL7cANNBoBLMAHMAfLnX4RAKBh6YAbV5I06YSMLSAuowBEACXSpUIWzO2S627aEiw+pgBMuAQk4tT0jCxsXMb8ZqLhUrLySmoaBDr6MAD0OTBKbDSsUKgAnjAAbvCoCuwwANboZbr6RgBC8PW1ULQ1MKKW3DYwtpRdMOPsbh5ePuDQcCYYAMwhRKQUEQzMrCAcPAGCiVvJclCKKuo4mln6eTBFJeUF7OhgUAoAZgro9VONZqtPS2ZTwGjjZToWx2YoxWyzIA
💻 Code
const sample1 = (x:number): Record<`sample_${number}`,string> => ({
    [`sample_${x}`]: "Hello",
})

const sample2 = (x:number): Record<`sample_${number}`,string> => ({
    // incorrectly valid key
    [`Bad literal ${x}`]: "bad bad",
})

const sample3 = (x:number): Record<`sample_${number}`,string> => ({
    // correctly identified bad key
    "garbage": "record"
})

🙁 Actual behavior
Typescript does not error on incorrect string template literals
🙂 Expected behavior
Typescript should show sample2 as having a bad object key like it does in sample3
Additional information about the issue
Bonus feature request/question: Is this the only way to get this sort of return type from a function. Putting "as const" in the key or after the object doesn't seem to correctly infer the type. It stays wide {[x:string]: string}

Duplicate of #13948.

Note that Record<`sample_${number}`, string>` doesn’t restrict the type to only those keys, it just guarantees that any keys matching the pattern are typed as string.

I mean I guess strictly from the perspective of something receiving the type but TS still checks that you're putting in the correct things when you're initially creating the object
const aa: Record<number, boolean> = {
	[2]: true,
	salami: "sandwich",
};

https://www.typescriptlang.org/play/?#code/MYewdgzgLgBAhnAXDASgU1AJwCYB4wCuAtgEZqYA0MJIIANmnGAHwwC8MA3gFACQA2gCYAusiiYCaCnwhw6cIgEtkAIllhsAd0XAAFiukBfANzcgA

Only when assigning the object literal to the type directly:
const bb = {
	[2]: true,
	salami: "sandwich",
};
const aa: Record<number, boolean> = bb;  // ok, no error

The excess property check isn't comprehensive, you shouldn't rely on it to constrain the allowed values of a type.  That's why issue #12936 exists.


question/discussion: option for blank rows in csv

Not sure if this is already possible and if not if it makes sense to implement. What if users could optionally check a box to indicate that their .csv file will contain blank rows. If that box is checked, blank rows are skipped. Maybe the first row would always need to have headers.
In a demo today, one of my users suggested that it would be useful for her workflow if she could visually separate groups of data using blank lines.
Apologies if this is already possible.

You could do this by modifying the CSV parser. Just don't yield the line
when it is blank.
On Feb 23, 2017 4:19 PM, "Jacob Stoebel"<EMAIL_ADDRESS>wrote:

Not sure if this is already possible and if not if it makes sense to
implement. What if users could optionally check a box to indicate that
their .csv file will contain blank rows. If that box is checked, blank rows
are skipped. Maybe the first row would always need to have headers.
In a demo today, one of my users suggested that it would be useful for her
workflow if she could visually separate groups of data using blank lines.
Apologies if this is already possible.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stephskardal/rails_admin_import/issues/86, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AB1KwadfZzgwvqWjRsfCcF2LKSds13Q5ks5rfff9gaJpZM4MKg0-
.

Blank lines will now be skipped


404 Not Found: @grial/cli@latest

Hi, i'm very interested in your project, but unfortunately can't try it out due to the error i'm getting from npm install (other packages are installed fine on the same machine).
Any idea how to fix the problem?

Try just installing npm i @grial/cli instead of npm i @grial/cli@latest.


Give LETSENCRYPT_HOST a default value

You should make LETSENCRYPT_HOST host have a default value og e.g. VIRTUAL_HOST

I prefer that the LETSENCRYPT_HOST is distinct from VIRTUAL_HOST


Split out template engine into “x-template.js”.

Conceptually “XElement” and “XTemplate” do very different things. The former is all about defining element interfaces and the latter is all about turning interpolated markup into dom.
Over time, even our PR diffs have highlited this separation of concerns. Often a PR either changes XElement or XTemplate — rarely both.

FYI @klebba — Just working through some of the decisions we’ve been chatting about. In this case, we agreed that splitting this into two files is probably the right move. I went ahead and bumped some dev dependencies as well.


girder DB item refresh sometimes messes up UpSet

sometimes after a project has been closed and re-opened, Upset remembers the defined sets, but the values are all blank.  In this case there is only one resulting row in UpSet with all variables = 0.    Even if the mapping is changed, UpSet doesn't correctly reinitialize.  It seems to often happen when the Girder DB item takes a while to refresh.  This is with the version from 5/30.  Should I create a new instance with master of candela to see if the problem is resolved in origin/master?

You could try that but I'm not sure if it will help. More likely we will have to debug to see what's going on. Let me know what you'd like to do.

Where in ResLab code is the repo of the project?  I am wondering about adding a delay before restoring the visualization to give the girder_db_item more time to initialize?
I will package up the VM as is and forward a copy to NCI for experimentation since it works most of the time.  This VM could be available to Kitware as well, so we can see if we can replicate the issue. We may be seeing this because I’m pushing the biggest data through the pipeline currently and we’ve induced a timing race conditions.
I’ll let you know when I have it ready.  In the meantime, it has been exciting to see a flurry of action around the extensions of Res Lab capabilities!

On Jun 8, 2016, at 7:33 AM, Roni Choudhury<EMAIL_ADDRESS>wrote:
You could try that but I'm not sure if it will help. More likely we will have to debug to see what's going on. Let me know what you'd like to do.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/ronichoudhury/nci-nanoparticles-vm/issues/8#issuecomment-224563151, or mute the thread https://github.com/notifications/unsubscribe/ACDZ9qmlUrjqNOsAHWTjp4NTxNIj_CYPks5qJqiOgaJpZM4IwUVn.

@alex-r-bigelow is this bug related to the one you were/are working on today?

This sounds similar to the issue that Curt mentioned that I couldn't seem to reproduce—not sure if it's exactly the same, but there's a chance the latest fix has dealt with it.
@curtislisle, about how big was the file that was causing problems (just to help me reproduce the issue)?

Yes, this email thread is one and the same issue that Alex may have already fixed.  The source file was 3200 rows by about 65 columns.  The source CSV file used to load into mongo is here:
https://github.com/ronichoudhury/nci-nanoparticles-vm/blob/master/files/nano-0516-filled.csv https://github.com/ronichoudhury/nci-nanoparticles-vm/blob/master/files/nano-0516-filled.csv
I am thinking the majority of any delay is coming from pulling these records through from mongo as a girder db item.  I was thinking that I would upload this file twice to mongo, then I’d have plenty or records :-)
I could see us wanting to run pretty long databases through this girder DB item path eventually.  It is a very powerful concept!

On Jun 14, 2016, at 3:59 PM, Alex Bigelow<EMAIL_ADDRESS>wrote:
This sounds similar to the issue that Curt mentioned that I couldn't seem to reproduce—not sure if it's exactly the same, but there's a chance the latest fix https://github.com/Kitware/candela/commit/48358b7992d194b446a835f7f877671e5eb00f36 has dealt with it.
@curtislisle https://github.com/curtislisle, about how big was the file that was causing problems (just to help me reproduce the issue)?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/ronichoudhury/nci-nanoparticles-vm/issues/8#issuecomment-225998254, or mute the thread https://github.com/notifications/unsubscribe/ACDZ9i23ybKMThocZ_O7jEWm9aw089ZMks5qLwgygaJpZM4IwUVn.