Add new instance id

/run recipe-test recipe_run_id=63

@cisaacstern can we naively rerun 94? I set the subset value to a quite high number, and just want to check if that is indeed the issue.

Sure thing. As a side note, I'm confused as to how we jumped from recipe run #s in the low sixties to the low nineties over the weekend, when I don't really see where those additional increments are coming from. But that's another issue, running 94 now!

Another killed worker. @jbusecke have you tried running this is in a local notebook? That's probably the best way tweak the subset_inputs kwarg at this point.

/run recipe-test recipe_run_id=98

/run recipe-test recipe_run_id=104

/run recipe-test recipe_run_id=108

/run recipe-test recipe_run_id=109

/run recipe-test recipe_run_id=109

@rabernat, Julius and I are seeing the following error when running a test of recipe_4 from this PR on Pangeo Forge Cloud:
AttributeError: 'NoneType' object has no attribute 'lock_release'

2022-04-29T19:56:16.625736338Z stderr F distributed.nanny - INFO -         Start Nanny at: 'tcp://<IP_ADDRESS>:40227'
--
  |   | 2022-04-29T19:56:20.833508624Z stderr F distributed.worker - INFO -       Start worker at:      tcp://<IP_ADDRESS>:32987
  |   | 2022-04-29T19:56:20.833573941Z stderr F distributed.worker - INFO -          Listening to:      tcp://<IP_ADDRESS>:32987
  |   | 2022-04-29T19:56:20.833825584Z stderr F distributed.worker - INFO -          dashboard at:            <IP_ADDRESS>:44343
  |   | 2022-04-29T19:56:20.834045602Z stderr F distributed.worker - INFO - Waiting to connect to: tcp://dask-jovyan-d39353ef-8.pangeo-forge-columbia-staging-bakery:8786
  |   | 2022-04-29T19:56:20.834167535Z stderr F distributed.worker - INFO - -------------------------------------------------
  |   | 2022-04-29T19:56:20.885289315Z stderr F distributed.worker - INFO -               Threads:                          1
  |   | 2022-04-29T19:56:20.885424473Z stderr F distributed.worker - INFO -                Memory:                   3.90 GiB
  |   | 2022-04-29T19:56:20.885434672Z stderr F distributed.worker - INFO -       Local Directory: /home/jovyan/dask-worker-space/worker-i_k7r44c
  |   | 2022-04-29T19:56:20.885554599Z stderr F distributed.worker - INFO - -------------------------------------------------
  |   | 2022-04-29T19:56:20.904091335Z stderr F distributed.worker - INFO -         Registered to: tcp://dask-jovyan-d39353ef-8.pangeo-forge-columbia-staging-bakery:8786
  |   | 2022-04-29T19:56:20.904171627Z stderr F distributed.worker - INFO - -------------------------------------------------
  |   | 2022-04-29T19:56:20.905137091Z stderr F distributed.core - INFO - Starting established connection
  |   | 2022-04-29T19:56:36.085992583Z stderr F distributed.core - INFO - Event loop was unresponsive in Worker for 15.04s.  This is often caused by long-running GIL-holding functions or moving large chunks of data. This can cause timeouts and instability.
  |   | 2022-04-29T19:56:36.341685239Z stdout F [2022-04-29 19:56:36+0000] INFO - prefect.CloudTaskRunner \| Task 'store_chunk[5]': Starting task run...
  |   | 2022-04-29T19:56:36.460479844Z stderr F INFO:pangeo_forge_recipes.recipes.xarray_zarr:Opening inputs for chunk Index({DimIndex(name='time', index=5, sequence_len=15, operation=<CombineOp.SUBSET: 3>), DimIndex(name='time', index=0, sequence_len=2, operation=<CombineOp.CONCAT: 2>)})
  |   | 2022-04-29T19:56:36.460575417Z stderr F INFO:pangeo_forge_recipes.recipes.xarray_zarr:Opening input with Xarray Index({DimIndex(name='time', index=0, sequence_len=2, operation=<CombineOp.CONCAT: 2>)}): 'http://aims3.llnl.gov/thredds/fileServer/css03_data/CMIP6/CMIP/NOAA-GFDL/GFDL-CM4/historical/r1i1p1f1/SImon/sithick/gn/v20180701/sithick_SImon_GFDL-CM4_historical_r1i1p1f1_gn_185001-194912.nc'
  |   | 2022-04-29T19:56:36.460611843Z stderr F INFO:pangeo_forge_recipes.storage:Opening 'http://aims3.llnl.gov/thredds/fileServer/css03_data/CMIP6/CMIP/NOAA-GFDL/GFDL-CM4/historical/r1i1p1f1/SImon/sithick/gn/v20180701/sithick_SImon_GFDL-CM4_historical_r1i1p1f1_gn_185001-194912.nc' from cache
  |   | 2022-04-29T19:56:36.460719823Z stderr F DEBUG:pangeo_forge_recipes.storage:file_opener entering first context for <contextlib._GeneratorContextManager object at 0x7ff83a255ac0>
  |   | 2022-04-29T19:56:36.461193991Z stderr F DEBUG:pangeo_forge_recipes.storage:entering fs.open context manager for pfcsb-bucket/cache/d3dad268f59c0449814856790ecb884d-http_aims3.llnl.gov_thredds_fileserver_css03_data_cmip6_cmip_noaa-gfdl_gfdl-cm4_historical_r1i1p1f1_simon_sithick_gn_v20180701_sithick_simon_gfdl-cm4_historical_r1i1p1f1_gn_185001-194912.nc
  |   | 2022-04-29T19:56:36.543465161Z stderr F DEBUG:pangeo_forge_recipes.storage:FSSpecTarget.open yielding <File-like object GCSFileSystem, pfcsb-bucket/cache/d3dad268f59c0449814856790ecb884d-http_aims3.llnl.gov_thredds_fileserver_css03_data_cmip6_cmip_noaa-gfdl_gfdl-cm4_historical_r1i1p1f1_simon_sithick_gn_v20180701_sithick_simon_gfdl-cm4_historical_r1i1p1f1_gn_185001-194912.nc>
  |   | 2022-04-29T19:56:36.543535256Z stderr F DEBUG:pangeo_forge_recipes.storage:file_opener entering second context for <File-like object GCSFileSystem, pfcsb-bucket/cache/d3dad268f59c0449814856790ecb884d-http_aims3.llnl.gov_thredds_fileserver_css03_data_cmip6_cmip_noaa-gfdl_gfdl-cm4_historical_r1i1p1f1_simon_sithick_gn_v20180701_sithick_simon_gfdl-cm4_historical_r1i1p1f1_gn_185001-194912.nc>
  |   | 2022-04-29T19:56:36.543614333Z stderr F DEBUG:pangeo_forge_recipes.recipes.xarray_zarr:about to enter xr.open_dataset context on <File-like object GCSFileSystem, pfcsb-bucket/cache/d3dad268f59c0449814856790ecb884d-http_aims3.llnl.gov_thredds_fileserver_css03_data_cmip6_cmip_noaa-gfdl_gfdl-cm4_historical_r1i1p1f1_simon_sithick_gn_v20180701_sithick_simon_gfdl-cm4_historical_r1i1p1f1_gn_185001-194912.nc>
  |   | 2022-04-29T19:56:42.227516552Z stderr F DEBUG:pangeo_forge_recipes.recipes.xarray_zarr:successfully opened dataset
  |   | 2022-04-29T19:56:42.240394762Z stderr F DEBUG:pangeo_forge_recipes.recipes.xarray_zarr:<xarray.Dataset>
  |   | 2022-04-29T19:56:42.240458564Z stderr F Dimensions:    (bnds: 2, time: 1200, y: 1080, x: 1440, xTe: 1441, yTe: 1081, vertex: 4)
  |   | 2022-04-29T19:56:42.240473445Z stderr F Coordinates:
  |   | 2022-04-29T19:56:42.240497895Z stderr F   * bnds       (bnds) float64 1.0 2.0
  |   | 2022-04-29T19:56:42.240508252Z stderr F   * time       (time) object 1850-01-16 12:00:00 ... 1949-12-16 12:00:00
  |   | 2022-04-29T19:56:42.24051703Z stderr F   * x          (x) float64 -299.7 -299.5 -299.2 -299.0 ... 59.53 59.78 60.03
  |   | 2022-04-29T19:56:42.240540578Z stderr F   * xTe        (xTe) float64 -299.8 -299.6 -299.3 -299.1 ... 59.66 59.91 60.16
  |   | 2022-04-29T19:56:42.24055001Z stderr F   * y          (y) float64 -80.39 -80.31 -80.23 -80.15 ... 89.73 89.84 89.95
  |   | 2022-04-29T19:56:42.24055787Z stderr F   * yTe        (yTe) float64 -80.43 -80.35 -80.27 -80.19 ... 89.78 89.89 90.0
  |   | 2022-04-29T19:56:42.240565975Z stderr F     lon        (y, x) float32 ...
  |   | 2022-04-29T19:56:42.24057309Z stderr F     lat        (y, x) float32 ...
  |   | 2022-04-29T19:56:42.240580755Z stderr F Dimensions without coordinates: vertex
  |   | 2022-04-29T19:56:42.240587148Z stderr F Data variables:
  |   | 2022-04-29T19:56:42.240593795Z stderr F     sithick    (time, y, x) float32 ...
  |   | 2022-04-29T19:56:42.240600735Z stderr F     time_bnds  (time, bnds) object ...
  |   | 2022-04-29T19:56:42.240606969Z stderr F     lat_bnds   (y, x, vertex) float32 ...
  |   | 2022-04-29T19:56:42.240614762Z stderr F     lon_bnds   (y, x, vertex) float32 ...
  |   | 2022-04-29T19:56:42.240621578Z stderr F Attributes: (12/46)
  |   | 2022-04-29T19:56:42.240628795Z stderr F     history:                File was processed by fremetar (GFDL analog of CM...
  |   | 2022-04-29T19:56:42.240635632Z stderr F     table_id:               SImon
  |   | 2022-04-29T19:56:42.24064225Z stderr F     activity_id:            CMIP
  |   | 2022-04-29T19:56:42.240648831Z stderr F     branch_method:          standard
  |   | 2022-04-29T19:56:42.240655586Z stderr F     branch_time_in_child:   0.0
  |   | 2022-04-29T19:56:42.240662247Z stderr F     comment:                <null ref>
  |   | 2022-04-29T19:56:42.240669499Z stderr F     ...                     ...
  |   | 2022-04-29T19:56:42.240676278Z stderr F     variable_id:            sithick
  |   | 2022-04-29T19:56:42.240683107Z stderr F     variant_info:           N/A
  |   | 2022-04-29T19:56:42.240691358Z stderr F     references:             see further_info_url attribute
  |   | 2022-04-29T19:56:42.240698917Z stderr F     variant_label:          r1i1p1f1
  |   | 2022-04-29T19:56:42.240705843Z stderr F     branch_time_in_parent:  36500.0
  |   | 2022-04-29T19:56:42.240713216Z stderr F     parent_time_units:      days since 0001-1-1
  |   | 2022-04-29T19:56:42.24103179Z stderr F INFO:pangeo_forge_recipes.recipes.xarray_zarr:Subsetting input according to time-5
  |   | 2022-04-29T19:56:42.241106002Z stderr F DEBUG:pangeo_forge_recipes.recipes.xarray_zarr:Subsetting dataset with indexer {'time': slice(400, 480, None)}
  |   | 2022-04-29T19:56:42.247336395Z stderr F INFO:pangeo_forge_recipes.recipes.xarray_zarr:Combining inputs for chunk 'Index({DimIndex(name='time', index=5, sequence_len=15, operation=<CombineOp.SUBSET: 3>), DimIndex(name='time', index=0, sequence_len=2, operation=<CombineOp.CONCAT: 2>)})'
  |   | 2022-04-29T19:56:42.287602094Z stderr F DEBUG:pangeo_forge_recipes.recipes.xarray_zarr:<xarray.Dataset>
  |   | 2022-04-29T19:56:42.287656856Z stderr F Dimensions:    (bnds: 2, time: 80, y: 1080, x: 1440, xTe: 1441, yTe: 1081, vertex: 4)
  |   | 2022-04-29T19:56:42.287686494Z stderr F Coordinates:
  |   | 2022-04-29T19:56:42.287695499Z stderr F   * bnds       (bnds) float64 1.0 2.0
  |   | 2022-04-29T19:56:42.287704274Z stderr F   * time       (time) object 1883-05-16 12:00:00 ... 1889-12-16 12:00:00
  |   | 2022-04-29T19:56:42.287712385Z stderr F   * x          (x) float64 -299.7 -299.5 -299.2 -299.0 ... 59.53 59.78 60.03
  |   | 2022-04-29T19:56:42.287720188Z stderr F   * xTe        (xTe) float64 -299.8 -299.6 -299.3 -299.1 ... 59.66 59.91 60.16
  |   | 2022-04-29T19:56:42.287728Z stderr F   * y          (y) float64 -80.39 -80.31 -80.23 -80.15 ... 89.73 89.84 89.95
  |   | 2022-04-29T19:56:42.287734388Z stderr F   * yTe        (yTe) float64 -80.43 -80.35 -80.27 -80.19 ... 89.78 89.89 90.0
  |   | 2022-04-29T19:56:42.287740812Z stderr F     lon        (y, x) float32 dask.array<chunksize=(1080, 1440), meta=np.ndarray>
  |   | 2022-04-29T19:56:42.287747939Z stderr F     lat        (y, x) float32 dask.array<chunksize=(1080, 1440), meta=np.ndarray>
  |   | 2022-04-29T19:56:42.287755222Z stderr F Dimensions without coordinates: vertex
  |   | 2022-04-29T19:56:42.287761484Z stderr F Data variables:
  |   | 2022-04-29T19:56:42.287769416Z stderr F     sithick    (time, y, x) float32 dask.array<chunksize=(80, 1080, 1440), meta=np.ndarray>
  |   | 2022-04-29T19:56:42.287786406Z stderr F     time_bnds  (time, bnds) object dask.array<chunksize=(80, 2), meta=np.ndarray>
  |   | 2022-04-29T19:56:42.287794244Z stderr F     lat_bnds   (y, x, vertex) float32 dask.array<chunksize=(1080, 1440, 4), meta=np.ndarray>
  |   | 2022-04-29T19:56:42.287800505Z stderr F     lon_bnds   (y, x, vertex) float32 dask.array<chunksize=(1080, 1440, 4), meta=np.ndarray>
  |   | 2022-04-29T19:56:42.287807212Z stderr F Attributes: (12/46)
  |   | 2022-04-29T19:56:42.287814043Z stderr F     history:                File was processed by fremetar (GFDL analog of CM...
  |   | 2022-04-29T19:56:42.287820676Z stderr F     table_id:               SImon
  |   | 2022-04-29T19:56:42.287827816Z stderr F     activity_id:            CMIP
  |   | 2022-04-29T19:56:42.28783502Z stderr F     branch_method:          standard
  |   | 2022-04-29T19:56:42.287842076Z stderr F     branch_time_in_child:   0.0
  |   | 2022-04-29T19:56:42.287849585Z stderr F     comment:                <null ref>
  |   | 2022-04-29T19:56:42.28785678Z stderr F     ...                     ...
  |   | 2022-04-29T19:56:42.287863685Z stderr F     variable_id:            sithick
  |   | 2022-04-29T19:56:42.287870506Z stderr F     variant_info:           N/A
  |   | 2022-04-29T19:56:42.287878045Z stderr F     references:             see further_info_url attribute
  |   | 2022-04-29T19:56:42.287884675Z stderr F     variant_label:          r1i1p1f1
  |   | 2022-04-29T19:56:42.287891102Z stderr F     branch_time_in_parent:  36500.0
  |   | 2022-04-29T19:56:42.287897495Z stderr F     parent_time_units:      days since 0001-1-1
  |   | 2022-04-29T19:56:42.64369656Z stderr F DEBUG:pangeo_forge_recipes.recipes.xarray_zarr:Converting variable sithick of 497664000 bytes to `numpy.ndarray`
  |   | 2022-04-29T19:56:51.028790066Z stderr F DEBUG:pangeo_forge_recipes.recipes.xarray_zarr:Acquiring locks ['sithick-time-33']
  |   | 2022-04-29T19:56:51.092226537Z stderr F DEBUG:pangeo_forge_recipes.utils:Acquiring lock pangeo-forge-sithick-time-33...
  |   | 2022-04-29T19:56:51.093594791Z stderr F DEBUG:pangeo_forge_recipes.utils:Acquired lock pangeo-forge-sithick-time-33
  |   | 2022-04-29T19:56:51.093881024Z stderr F INFO:pangeo_forge_recipes.recipes.xarray_zarr:Storing variable sithick chunk Index({DimIndex(name='time', index=5, sequence_len=15, operation=<CombineOp.SUBSET: 3>), DimIndex(name='time', index=0, sequence_len=2, operation=<CombineOp.CONCAT: 2>)}) to Zarr region (slice(400, 480, None), slice(None, None, None), slice(None, None, None))
  |   | 2022-04-29T19:56:57.438654101Z stderr F DEBUG:pangeo_forge_recipes.utils:Released lock pangeo-forge-sithick-time-33
  |   | 2022-04-29T19:56:57.520012731Z stderr F DEBUG:pangeo_forge_recipes.recipes.xarray_zarr:Converting variable time of 640 bytes to `numpy.ndarray`
  |   | 2022-04-29T19:56:57.522017763Z stderr F DEBUG:pangeo_forge_recipes.recipes.xarray_zarr:Acquiring locks ['time-time-33']
  |   | 2022-04-29T19:56:57.522829331Z stderr F DEBUG:pangeo_forge_recipes.utils:Acquiring lock pangeo-forge-time-time-33...
  |   | 2022-04-29T19:56:57.524663995Z stderr F DEBUG:pangeo_forge_recipes.utils:Acquired lock pangeo-forge-time-time-33
  |   | 2022-04-29T19:56:57.524769957Z stderr F INFO:pangeo_forge_recipes.recipes.xarray_zarr:Storing variable time chunk Index({DimIndex(name='time', index=5, sequence_len=15, operation=<CombineOp.SUBSET: 3>), DimIndex(name='time', index=0, sequence_len=2, operation=<CombineOp.CONCAT: 2>)}) to Zarr region (slice(400, 480, None),)
  |   | 2022-04-29T19:56:57.749296599Z stderr F distributed.worker - INFO - Stopping worker at tcp://<IP_ADDRESS>:32987
  |   | 2022-04-29T19:56:57.77712317Z stdout F [2022-04-29 19:56:57+0000] ERROR - prefect.CloudTaskRunner \| Task 'store_chunk[5]': Exception encountered during task execution!
  |   | 2022-04-29T19:56:57.777637929Z stdout F Traceback (most recent call last):
  |   | 2022-04-29T19:56:57.777662797Z stdout F   File "/srv/conda/envs/notebook/lib/python3.9/site-packages/prefect/engine/task_runner.py", line 861, in get_task_run_state
  |   | 2022-04-29T19:56:57.777672349Z stdout F     value = prefect.utilities.executors.run_task_with_timeout(
  |   | 2022-04-29T19:56:57.777584345Z stderr F ERROR:prefect.CloudTaskRunner:Task 'store_chunk[5]': Exception encountered during task execution!
  |   | 2022-04-29T19:56:57.778682249Z stdout F   File "/srv/conda/envs/notebook/lib/python3.9/site-packages/prefect/utilities/executors.py", line 323, in run_task_with_timeout
  |   | 2022-04-29T19:56:57.778715778Z stdout F     return task.run(*args, **kwargs)  # type: ignore
  |   | 2022-04-29T19:56:57.77874032Z stdout F   File "/usr/local/lib/python3.9/site-packages/registrar/flow.py", line 113, in wrapper
  |   | 2022-04-29T19:56:57.778749981Z stdout F   File "/srv/conda/envs/notebook/lib/python3.9/site-packages/pangeo_forge_recipes/recipes/xarray_zarr.py", line 573, in store_chunk
  |   | 2022-04-29T19:56:57.778756568Z stdout F     zarr_array[zarr_region] = data
  |   | 2022-04-29T19:56:57.778763299Z stdout F   File "/srv/conda/envs/notebook/lib/python3.9/contextlib.py", line 126, in __exit__
  |   | 2022-04-29T19:56:57.778769801Z stdout F     next(self.gen)
  |   | 2022-04-29T19:56:57.778776439Z stdout F   File "/srv/conda/envs/notebook/lib/python3.9/site-packages/pangeo_forge_recipes/utils.py", line 118, in lock_for_conflicts
  |   | 2022-04-29T19:56:57.778783258Z stdout F     lock.release()
  |   | 2022-04-29T19:56:57.778803439Z stdout F   File "/srv/conda/envs/notebook/lib/python3.9/site-packages/distributed/lock.py", line 151, in release
  |   | 2022-04-29T19:56:57.778810718Z stdout F     self.client.scheduler.lock_release, name=self.name, id=self.id
  |   | 2022-04-29T19:56:57.778817494Z stdout F AttributeError: 'NoneType' object has no attribute 'lock_release'
  |   | 2022-04-29T19:56:57.778705956Z stderr F Traceback (most recent call last):
  |   | 2022-04-29T19:56:57.778831704Z stderr F   File "/srv/conda/envs/notebook/lib/python3.9/site-packages/prefect/engine/task_runner.py", line 861, in get_task_run_state
  |   | 2022-04-29T19:56:57.778838359Z stderr F     value = prefect.utilities.executors.run_task_with_timeout(
  |   | 2022-04-29T19:56:57.778845231Z stderr F   File "/srv/conda/envs/notebook/lib/python3.9/site-packages/prefect/utilities/executors.py", line 323, in run_task_with_timeout
  |   | 2022-04-29T19:56:57.778852071Z stderr F     return task.run(*args, **kwargs)  # type: ignore
  |   | 2022-04-29T19:56:57.778858386Z stderr F   File "/usr/local/lib/python3.9/site-packages/registrar/flow.py", line 113, in wrapper
  |   | 2022-04-29T19:56:57.778865144Z stderr F   File "/srv/conda/envs/notebook/lib/python3.9/site-packages/pangeo_forge_recipes/recipes/xarray_zarr.py", line 573, in store_chunk
  |   | 2022-04-29T19:56:57.778871675Z stderr F     zarr_array[zarr_region] = data
  |   | 2022-04-29T19:56:57.778878222Z stderr F   File "/srv/conda/envs/notebook/lib/python3.9/contextlib.py", line 126, in __exit__
  |   | 2022-04-29T19:56:57.778884679Z stderr F     next(self.gen)
  |   | 2022-04-29T19:56:57.778893125Z stderr F   File "/srv/conda/envs/notebook/lib/python3.9/site-packages/pangeo_forge_recipes/utils.py", line 118, in lock_for_conflicts
  |   | 2022-04-29T19:56:57.778900143Z stderr F     lock.release()
  |   | 2022-04-29T19:56:57.778906574Z stderr F   File "/srv/conda/envs/notebook/lib/python3.9/site-packages/distributed/lock.py", line 151, in release
  |   | 2022-04-29T19:56:57.778935059Z stderr F     self.client.scheduler.lock_release, name=self.name, id=self.id
  |   | 2022-04-29T19:56:57.778942054Z stderr F AttributeError: 'NoneType' object has no attribute 'lock_release'
  |   | 2022-04-29T19:56:57.780001303Z stderr F distributed.dask_worker - INFO - Exiting on signal 15
  |   | 2022-04-29T19:56:57.780026469Z stderr F distributed.nanny - INFO - Closing Nanny at 'tcp://<IP_ADDRESS>:40227'
  |   | 2022-04-29T19:56:57.948725733Z stdout F [2022-04-29 19:56:57+0000] INFO - prefect.CloudTaskRunner \| Task 'store_chunk[5]': Finished task run for task with final state: 'Failed'
  |   | 2022-04-29T19:56:57.948866006Z stderr F INFO:prefect.CloudTaskRunner:Task 'store_chunk[5]': Finished task run for task with final state: 'Failed'
  |   | 2022-04-29T19:56:57.950702711Z stderr F distributed.nanny - INFO - Worker closed
  |   | 2022-04-29T19:56:59.385116591Z stderr F distributed.nanny - WARNING - Worker process still alive after 1 seconds, killing
  |   | 2022-04-29T19:56:59.386855884Z stderr F distributed.dask_worker - INFO - End worker
  |   | 2022-04-29T19:56:59.387340386Z stderr F distributed.process - INFO - reaping stray process <SpawnProcess name='Dask Worker process (from Nanny)' pid=315 parent=1 started daemon>
  |   | 2022-04-29T19:56:59.405694188Z stderr F Exception in thread AsyncProcess Dask Worker process (from Nanny) watch process join:
  |   | 2022-04-29T19:56:59.40574961Z stderr F Traceback (most recent call last):
  |   | 2022-04-29T19:56:59.405761984Z stderr F   File "/srv/conda/envs/notebook/lib/python3.9/threading.py", line 973, in _bootstrap_inner

Could this be related to https://github.com/pangeo-forge/noaa-coastwatch-geopolar-sst-feedstock/issues/2#issuecomment-1104167875 ?
We've just tried to reproduce in a notebook with
import prefect

flow = recipe_4.copy_pruned().to_prefect()
flow.executor = prefect.executors.DaskExecutor()

but were unsuccessful, perhaps because there were not enough workers in the LocalCluster to induce the locking issue?

I've opened https://github.com/pangeo-forge/cmip6-feedstock/pull/6 as a replacement for this PR, so going to close this now.


Getting started with automated testing

setting up initial (unit) tests for easy implementation of future features

@rishabhrkaushik are you interested in working on some test classes for test automatization?
You already started with run_test.py - so this would be a great way to continue the work!

@2Max4 I am working on GUI currently. I'll let you know if I'd be able to take this task up after completion of first drat of GUI.
Also, can I have you mail id. I need to discuss some logic with you. Ping me on my email id. Available on my github profile.

Hey @rishabhrkaushik - you're right GUI has prio!
Don't you think it would be a good Idea to do those discussions in the  Issue #25 - so everyone who is interested always knows why we made which decission....


sokol_fetch.h doesn't work for compressed responses

Aaaaargh. When the web server sends a compressed response back, the Content-Length field contains the compressed size, not the uncompressed size.
And worse, it's forbidden to override the Accept-Encoding request header in XMLHttpRequest, so I can't even "request" the uncompressed file size with a HEAD request with Accept-Encoding NONE.

Fixed in 74a19f9ede1939d0d29bb3f6e82ddcdd64425a83


Importing public files with /* @vite-ignore */ dynamic imports throws an error

Description
My project generates JavaScript templates for non-technical users.  Before Vite, I published a zip file that looked like this:

index.html
config.js
bundles/

mount.js

I'm trying to port this project to Vite.
To prevent Vite from minifying the source and mangling the name, I've tried putting config.js in public.  However, when I run vite serve, I'm met with this error message:
Failed to load url /config.js (resolved id: /config.js). This file is in /public and will be copied
as-is during build without going through the plugin transforms, and therefore should not be
imported from source code. It can only be referenced via HTML tags.

Reproduction
While I respect the effort to prevent people from making mistakes, there ought to be the ability to say "no really, this is on purpose."
In short, I want to be able to tell Vite "Treat this JS file like any other asset.  Don't change its name or its contents, and allow it to be imported from the devserver."
Suggested solution
The easiest solution from both an implementation and a user's POV would be a config flag, like server.allowDangerousPublicDirImports.  If it's set, Vite skips throwing the error and serves the requested file as if it's any other asset.
This gives the user the final decision about what is and is not permissible in their architecture, without resorting to crazy backdoors like writing a plugin to serve JS as an assetInclude.
The term Dangerous in the flag name warns people to not set it without consideration.  Using a flag gives it an obvious place to live in the documentation to make it easier for people to find.  (A nod could also be included in ERR_LOAD_PUBLIC_URL.) I'm not convinced that the risk of serving files from publicDir is so high that they need to individually be whitelisted.
Alternative
Whitelisting, either by:

an inline comment, like import(/* vite-treat-as-asset */ 'config.js), or
a config field, like jsModulesInPublicDir: ['**/*config.js'].

The comment is less discoverable than a config field, but easier to use.  You explicitly grant permission at the call site, so there is nothing to maintain.  (Imagine if the file name changes.)
A config field is DRYer (for instance, if many packages in a monorepo share a vite.config, one setting repairs the whole project).
Additional context
config.js is effectively a JSON with generous commenting and some dependency injection to provide functionality like shuffling and centering.
It's designed to be hand-edited by people who aren't technical enough to create or maintain a web app, but are capable of editing some strings.  All the complexity is in mount.js, which is just a blob to them.  They can edit the config file, drag it to a file server, and have an app that meets their needs without touching the command line.  Moreover, config can be generated by a web app after mount has already been minted.
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

I think this is a bug.
I guess it'll work if && !hasViteIgnore is added here. (hasViteIgnore would need to be moved outside the if block together)
https://github.com/vitejs/vite/blob/66caef369dbe80bb3fbaabdae5ffb05c42124f32/packages/vite/src/node/plugins/importAnalysis.ts#L654-L657

Thanks for the speedy reply!
I believe the problem is that the devserver is getting a request for a JS file it doesn't recognize.
https://github.com/vitejs/vite/blob/66caef369dbe80bb3fbaabdae5ffb05c42124f32/packages/vite/src/node/server/transformRequest.ts#L255-L259
I'm new to Vite so I may be mistaken, but it looks like importAnalysis.ts is part of the build infrastructure.
There may also be a problem with importing (leading to hacks like const importWIthoutVite = (p) => import(p)), but this FR is to allow .js requests to pass when the devserver sees they're in publicDir.

I tested out my change and it worked for your reproduction. So I created the PR: #14851

Thanks again.  You're definitely the expert here.
However, I tried using your PR:
├─ vite@portal:../vite/packages/vite::locator=root-workspace-0b6124%40workspace%3A.
…
* 42fee9ee (HEAD -> main) fix: don't add `?import` for `@vite-ignore`ed import (#14851)

(I ran pnpm install and pnpm build.)
and I'm still seeing
Failed to load url /experiment-config.js (resolved id: /experiment-config.js). This file is in ../passthrough and will be copied as-is during build without going through the plugin transforms, and therefore should not be imported from source code. It can only be referenced via HTML tags.
    at loadAndTransform (file://~/Projects/vite/packages/vite/dist/node/chunks/dep-qg0kuhpq.js:47748:21)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at viteTransformMiddleware (file://~/Projects/vite/packages/vite/dist/node/chunks/dep-qg0kuhpq.js:57367:32

Are you sure this isn't something that needs to be solved in the server (e.g. `loadAndTransform)?

Thanks for the knowledge and quick fix!
Should I need to do this?
const importWithoutVite = (path: string) => import(/* @vite-ignore */path);

importWithoutVite('./config.js').then(

If I try to
import(/* @vite-ignore */'./config.js)

directly, I get:
Internal server error: Failed to resolve import "./experiment-config.js" from "src/mount.tsx". Does the file exist?

Thanks for the knowledge and quick fix!
Should I need to do this?
const importWithoutVite = (path: string) => import(/* @vite-ignore */path);

importWithoutVite('./config.js').then(

If I try to
import(/* @vite-ignore */'./config.js')

directly, I get:
Internal server error: Failed to resolve import "./config.js" from "src/mount.tsx". Does the file exist?

After further investigation, since build puts the bundle in assets, but serve serves it from /, I end up with a path conflict.  Is there a better strategy than handling the manually?
const importWithoutVite = (path: string) => import(/* @vite-ignore */path);

importWithoutVite(
  // handle JS going into `assets` when bundled
  import.meta.env.PROD
    ? '../config.js'
    : './config.js'
).then(

Since you're opting-out of Vite handling normalizing the import paths, you would have to do that manually.

Reopening as the PR was reverted #14866 due to breakage: https://github.com/vitejs/vite/pull/14851#issuecomment-1791941779

Turns out you can skip the path switching for the devserver, because the devserver mounts at the root, and /../ is the same as /.
So import('../my-asset.js') will work whether you're in /mount.jsx or /assets/mount.js.

Here's a prototype that seems to be working:
// Copyright 2023 Google LLC.
// SPDX-License-Identifier: Apache-2.0

const literalDynamicImports = (literals: Array<string>): PluginOption => {
  let inDevServer = false;

  return {
    name: 'vite-plugin-literal-dynamic-imports',

    configureServer () {
      inDevServer = true;
    },

    resolveDynamicImport (specifier, importer, options) {
      if (literals.includes(specifier as string)) {
        return false;
      }
    },

    transform (code, id, options){
      // Vite only intermediates when the dev server is on, so we only have to
      // work around it when it's on.
      if (inDevServer) {
        // `MagicString` tracks string modifications to generate a matching source
        // map.
        const magicCode = new MagicString(code);

        for (let result of code.matchAll(/import\(([^\)]*?('([^']+?)'|"([^"]+?)")[^\)]*?)\)/g)) {
          const specifier = result[3] ?? result[4];
          const startIndex = result.index!;
          const endIndex = startIndex + result[0].length;

          magicCode.update(
            startIndex,
            endIndex,
            // `.split('').join('')` is a no-op, but it prevents
            // `importAnalysis` from analyzing (and flagging) the specifier.
            //
            // `@vite-ignore` disables the warning that dynamically generated
            // strings won't be analyzed.
            `import(/* @vite-ignore */ "${ specifier }".split('').join(''))`,
          );
        }

        return {
          code: magicCode.toString(),
          map: magicCode.generateMap({}),
        };
      }
    },
  };
}

RegExes suck - I don't promise it's perfect, but it's working for me.
Does this look good?  If so, I can publish it.

Dammit - just tried upgrading to<EMAIL_ADDRESS>and this error is back:
Failed to load url /experiment-config.js (resolved id: /experiment-config.js). This file is in
../passthrough and will be copied as-is during build without going through the plugin transforms,
and therefore should not be imported from source code. It can only be referenced via HTML tags.

I feel like I keep going down this route, I'm gonna spend more time that I can afford reverse engineering Vite's dev server to figure out precisely what I need to do in configureServer to make this work.  @bluwy, do you have tips for me?
It seems that Vite has no problem serving the file (so I don't need to call sirv), but that the caller is the one throwing the error.  That part is confusing to me.  I don't know why I can load the imported JS file directly without causing the error, but opening the HTML+JS bundle that imports it does throw the error.
transformMiddleware appears to be what's calling the erroring loadAndTransform, but it's doing it with a path to the imported file.  If that was the source of the error, I'd expect requesting the imported file directly in the browser would cause the error.  The error only occurs when imported from HTML+JS, but I don't know how/why.

Here's how I solved it for my little use case in a vite react app...
I added 2 files to a subfolder in public.

public/app/

config.js
importer.js

// config.js
export const myPlugin = {
  id: "yolo",
  main(id) {
    console.log("Yolo!", id);
  },
};

// importer.js
function importNative(path) {
  return import(path);
}

Then in my index.html file I added <script src="/app/importer.js"></script> to the <head> area.
To test it in my src/main.tsx file I did:
const importNative = window["importNative"] as (path: string) => any;

importNative("/app/config.js").then(config => {
  const myPlugin = config?.myPlugin;
  if (!myPlugin) { console.log("Config?", config); return; }
  myPlugin.main(myPlugin.id);
});

Any progress with that? After updating to Vite 5 dynamic import throws an error and @vite-ignore is not working properly.

I was able to work around this issue by creating a custom import function for when I want to import a js file from the public folder.  Works for me with vite: ^5.0.8.
/**
 * This is a patched version of the native `import` method that works with vite bundling.
 * Allows you to import a javascript file from the public directory.
 */
function importStatic(modulePath) {
    if (import.meta.env.DEV) {
        return import(/* @vite-ignore */ `${modulePath}?url`);
    } else {
        return import(/* @vite-ignore */ modulePath);
    }
}

const path = './content/foo.js`; //  -> `/public/content/foo.js`
const module = await importStatic(path);

@waynebloss - clever!  You could inline importer into the HTML to skip a download.
@connorgmeehan Did you try that in both vite serve and vite build?  I feel like that's where I started, but IIRC, I was seeing error overlays in vite serve.
Does the ?url get Vite to skip the path validation?  You might be better with Date.now, so you're always getting a fresh copy of modulePath

I just tested @connorgmeehan's workaround in the reproduction, and adding a trailing ? does seem to work around this.

Yep it worked for me for both serve as well as build/preview (again, only tested with ^5.0.8)  Is the the Date.now required to avoid caching of the module (I'm guessing in instances where you're hot-reloading)?
import(/* @vite-ignore */ `${modulePath}?{Date.now()}`)

Yeah. I wouldn't say it's "required," but it can be nice to ensure you're getting unique, uncached URLs on each load.
I haven't played with it in this context to see how well it plays with HMR. It's just an old habit to avoid caching.

The workaround seems really fragile.  I tried to implement it in the literalDynamicImports function above and was getting errors.  I tried other techniques to make it dynamic like .split('').join('') and got errors.
We seem to be lucky that this one sneaks by Vite now, but I don't trust that to stay in the future.  Regardless, I'm glad to be on a stable version now (rather than monkeypatching my local module with sapphi-red's PR).
Thanks again Connor!

One of the errors was Unknown variable dynamic import.  I forgot to write the other one down.

Not sure if this is exactly the same problem as above but in case its useful here's the workaround I'm using, in a web component that allows users to specify wasm files:
    const base = this.src.startsWith('http') ? undefined : window.location.href
    const src = new URL(this.src, base)
    const module = await import(/* @vite-ignore */src.href)

Issue still exists, I unable to enforce optional dynamic import.

For me, await import(/* @vite-ignore */`${location.origin}/xxx.js`) is work

await import(/* @vite-ignore */ 'zip:xxx.zip#index.js') still can not work.

This issue still exists.


Adding test for vs code cli command

Re-created branch and PR, because there was still an issue with the previous one even if I created it only yesterday

Is this ready for review?


feat: add exists, isFile, and isDir cheatcodes

Motivation
Resolve #5637 and provide a way to verify if a path is valid, points to a file or a directory.

Solution
Implement three new cheatcodes - exists, isFile and isDir
    // Returns true if the given path points to an existing entity, else returns false
    function exists(string calldata path) external returns (bool);

    // Returns true if the path exists on disk and is pointing at a regular file, else returns false
    function isFile(string calldata path) external returns (bool);

    // Returns true if the path exists on disk and is pointing at a directory, else returns false
    function isDir(string calldata path) external returns (bool);

@Evalir updated forge-std Vm.sol with the interface: foundry-rs/forge-std#441
Will update the book in a bit too


refine faq for resource allocation

add default configuration

Coverage increased (+8.2%) to 84.274% when pulling 48903558fc1964ae989ea81b15682eabb0db1a90 on yanjga/faq_detail into 6d09e02be0311be8e77a45e30fcdb987a630ea83 on master.

Coverage increased (+8.2%) to 84.274% when pulling 48903558fc1964ae989ea81b15682eabb0db1a90 on yanjga/faq_detail into 6d09e02be0311be8e77a45e30fcdb987a630ea83 on master.

Coverage increased (+8.2%) to 84.274% when pulling 48903558fc1964ae989ea81b15682eabb0db1a90 on yanjga/faq_detail into 6d09e02be0311be8e77a45e30fcdb987a630ea83 on master.

The item could added.
For use test case:
I guess :

not use many user to test and make the percent to touch the config percent ? (see official intro )

The offical intro:
mapred.capacity-scheduler.queue..minimum-user-limit-percent |
Each queue enforces a limit on the percentage of resources allocated to a user at any given time, if there is competition for them. This user limit can vary between a minimum and maximum value. The former depends on the number of users who have submitted jobs, and the latter is set to this property value. For example, suppose the value of this property is 25. If two users have submitted jobs to a queue, no single user can use more than 50% of the queue resources. If a third user submits a job, no single user can use more than 33% of the queue resources. With 4 or more users, no user can use more than 25% of the queue's resources. A value of 100 implies no user limits are imposed.
-- | --


Fix typescript types

Fix of the issue mentioned in #68 - the main Typescript API is broken, stating description instead of descriptions.

Please, create a tag for this fix.


Add App to Orchestrator and allow Name to be read from it

This PR needs to be merged and a new service-extension library release be created.
After that the new lib should be referenced in the corresponding orchestrator PR:


[Feature request] Treating run duration as a summary metric

I'd like to be able to treat run duration like any other final metric, produces by the run (e.g. like a loss that I record)
Use-case example:
I'm currently running a sweep (on a single machine) over my hyperparameters. Looking at the parameter importance table w.r.t. losses makes me feel that some parameters that influence the runtime significantly are not actually of much importance to loss values. If true, this might mean that I can make the model more efficient without losing much of accuracy. In this case, I'd need to see an importance plot for the run duration to quickly evaluate this suspicion (and hopefully save running time) =)
P.S. I understand that I can probably make a workaround by recording duration by myself from the training script, but why would I want to do that, if the info is already attached to the run?

Hi Maria, thanks for sharing this feature request. To clarify, the specific feature you'd like is to be able to select the run duration as a target metric in the parameter importance panel. Is that right?

Hi @cvphelps!
No, what I wanted is the other way around:

Select duration as a target metric
See how my hyperparameters have an effect on this metric (e.g. larger number of Conv layers -> longer training?)

Something like this. But your understanding also sounds interesting, actually 🤔

In the past year we've majorly reworked the CLI and UI for Weights & Biases. We're closing issues older than 6 months. Please comment to reopen.

I would also be interested in something like this as @maria-korosteleva mentioned;

Select duration as a target metric
See how my hyperparameters have an effect on this metric (e.g. larger number of Conv layers -> longer training?)

I would also find it useful to be able to query the run duration from the api

Is this planned? It would be very usefule, plotting eval metrics over duration for examples can give valuable insights when comparing multiple runs or models against each other. It's a bit frustrating that this is currently not possible, despite the value being logged with the run (but not under summary).


Merge two accounts using IdentityServer4

We built OAuth identity server using .net core identity server 4 third party library. The identity server support both direct login and facebook login. We are now thinking to merge direct login user and facebook user if both have same email ID. So can you please suggest how to integrate this feature in our identity server. Is there any built in support or best practice to implement this?
Any suggestion or ideas greatly appreciated.
Thanks!

This seems to be a general question about IdentityServer - not a bug report or an issue.
Please use one of the our free or commercial support options
See here for more details.
Thanks!


Task00 Леонид Тернопол SPbU

Локальный вывод

$ ./enumDevices
Number of OpenCL platforms: 2
Platform #1/2
    Platform name: AMD Accelerated Parallel Processing
    Platform vendor: Advanced Micro Devices, Inc.
    Number of devices: 1
    Device #1/1
        Device name: gfx1103
        Device type: GPU
        Device memory: 14537.5MB
        Is device little endian: true
        Max supported OpenCL C version: OpenCL C 2.0
        Max work group size: 256
Platform #2/2
    Platform name: Intel(R) OpenCL
    Platform vendor: Intel(R) Corporation
    Number of devices: 1
    Device #1/1
        Device name: AMD Ryzen 7 7840HS with Radeon 780M Graphics
        Device type: CPU
        Device memory: 28356.9MB
        Is device little endian: true
        Max supported OpenCL C version: OpenCL C 3.0
        Max work group size: 8192

Process finished with exit code 0

Вывод Github CI

Run ./build/enumDevices
Number of OpenCL platforms: 1
Platform #1/1
    Platform name: Intel(R) CPU Runtime for OpenCL(TM) Applications
    Platform vendor: Intel(R) Corporation
    Number of devices: 1
    Device #1/1
        Device name: AMD EPYC 7763 64-Core Processor
        Device type: CPU
        Device memory: 15991.5MB
        Is device little endian: true
        Max supported OpenCL C version: OpenCL C 2.0
        Max work group size: 8192

Все хорошо, задача зачтена, 5/5 баллов :+1:


Fix undefined label in the InputLookupQA component

Fixes #660
Adds some new spoofed resource templates for testing this fix.
If the defaultLiteral value is undefined, use the defaultURI instead.

LGTM; I don't see anything at this point that would prevent future work.


Fix log message for not found WIT

A work item type (WIT) is not identified by it's ID but its name, hence
the log message needed to be changed.

Current coverage is 49.61% (diff: 0.00%)

Merging #415 into master will not change coverage

@@             master       #415   diff @@
==========================================
  Files            39         39
  Lines          1685       1685
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            836        836
  Misses          786        786
  Partials         63         63

Powered by Codecov. Last update cb52b20...dc10149


Rename repo.

As we will eventually have support for cosmwasm contracts written in go (not just rust), the name of this repo has already led to some confusion.
I propose to rename it sooner rather than later. This repo is just go-rust FFI glue between cosmwasm (rust) and wasmd (cosmos sdk). I propose the name cosmwasm-ffi, but I am sure there are better ideas. Maybe @webmaster128 or @alpe can think of a better one.

It is not only the glue code. It also bundles the entire CosmWasm VM and makes it available in Go applications, such as x/wasm.
I propose CosmWasm/go-vm.

After long back and forth with @alpe, we concluded that wasmvm is a good compromise, respecting

Length
Differentiation to contract development in Go
The VM's view
The x/wasm's view


Update http-parser

Brings the http parser up to the tip of the upstream master branch

Current coverage is 67.20%

Merging #1588 into develop will increase coverage by +0.03% as of c03481a

@@            develop   #1588   diff @@
=======================================
  Files           893     892     -1
  Stmts         64299   64309    +10
  Branches          0       0
  Methods           0       0
=======================================
+ Hit           43193   43219    +26
  Partial           0       0
+ Missed        21106   21090    -16

Review entire Coverage Diff as of c03481a

Powered by Codecov. Updated on successful CI builds.

Both gcc and clang warn about unused enum in a switch statement:
src/beast/beast/http/impl/joyent_parser.cpp:64:12: warning: enumeration value ‘HTTP_BIND’ not handled in switch [-Wswitch]
switch (m)
^
src/beast/beast/http/impl/joyent_parser.cpp:64:12: warning: enumeration value ‘HTTP_REBIND’ not handled in switch [-Wswitch]
src/beast/beast/http/impl/joyent_parser.cpp:64:12: warning: enumeration value ‘HTTP_UNBIND’ not handled in switch [-Wswitch]
src/beast/beast/http/impl/joyent_parser.cpp:64:12: warning: enumeration value ‘HTTP_ACL’ not handled in switch [-Wswitch]
src/beast/beast/http/impl/joyent_parser.cpp:64:12: warning: enumeration value ‘HTTP_MKCALENDAR’ not handled in switch [-Wswitch]
src/beast/beast/http/impl/joyent_parser.cpp:64:12: warning: enumeration value ‘HTTP_LINK’ not handled in switch [-Wswitch]
src/beast/beast/http/impl/joyent_parser.cpp:64:12: warning: enumeration value ‘HTTP_UNLINK’ not handled in switch [-Wswitch]

If you're OK with modifying that file,this commit resolves the warnings: https://github.com/seelabs/rippled/commit/6b2e3a4f9938ef02b4412537d86e3531e1f8222e

Added the missing case labels

:+1:

:+1:


Add page_manager patch

Closes #51
Ref https://www.drupal.org/project/page_manager/issues/3398407

Using only last commit from patch as it only makes sense


changed Debug to Release

This now accurately describes how to get the correct build output

I'd rather just fix the .csproj.


locked / unlock loop

Bug description
WebSSH 25.8 on iPadOS 17.3.1 is stuck in an unlock loop. A small dialog appears including "Locked" and an Unlock button. Clicking the button makes the dialog go away for a second then it comes back.
Screenshots

Steps to reproduce

Start app
Locked dialog appears, click Unlock
Dialog goes away for a second. Go to step 2.

Tried but didn't help:

killing the app (swipe up from bottom for app cards, swipe WebSSH card up to end process)
changing lock settings under Settings/WebSSH
disabling FaceID under Settings/WebSSH
restarting iPad

Hello @stuntsman 👋
Does Face ID is enrolled on the device?

Thanks for the tip, I've found that FaceID was disabled:

a few days earlier I had to disable Passcode on the device temporarily, but then forgot about that
however disabling Passcode also automatically disabled FaceID (I didn't notice that until now)
since then WebSSH stuck in an unlock loop
re-enabling FaceID systemwide solved the issue (iPadOS required setting a passcode too)

Suggestion: the WebSSH unlock dialog could show a message if FaceID and/or passcode is disabled systemwide, as it seems to make unlocking impossible.

Thanks for your strong feedback about your resolution 🙏
I will check if I can retrieve Face ID (full) enrollment from the Apple SDK and so improve WebSSH as well in order to avoid this check loop

Merging this issue with this old one : https://github.com/isontheline/pro.webssh.net/issues/1101#issuecomment-2061731451

Fixed inside the latest beta release :
https://testflight.apple.com/join/QSrBK59z


SegmentRangeReader: new format for reading multiple segments

only supports the segment indices for now (doesn't support time based)

similar syntax to numpy for segment ranges
344c5c15b34f2d8a/2024-01-03--09-37-12 -> all segments
344c5c15b34f2d8a/2024-01-03--09-37-12--0 -> segment 0
344c5c15b34f2d8a/2024-01-03--09-37-12/0 -> segment 0
344c5c15b34f2d8a/2024-01-03--09-37-12/5 -> segment 5
344c5c15b34f2d8a/2024-01-03--09-37-12/0/2 -> segment 0 and 1
344c5c15b34f2d8a/2024-01-03--09-37-12/-1 -> last segment
344c5c15b34f2d8a/2024-01-03--09-37-12/0/-1 -> all except last segment

@adeebshihadeh I think this initial version is ready for review, I can switch over more tools once this is merged

tests for matching numpy syntax
tests for parsing qlogs and rlogs
defaults to an "automatic source" which tries internal and openpilotci before falling back to the comma api

The slash separator for a range seems confusing to me (slash normally infers heirarchy), I think something like python array slicing (colon separator) would be a lot more intuitive.


[Bug]: Terraform import of snowflake_tag_masking_policy_association is not working correctly

Terraform CLI Version
1.9.8
Terraform Provider Version
0.97.0
Terraform Configuration
While trying to import an existing terraform resource of snowflake_tag_masking_policy_association
using the below statement -
terraform import snowflake_tag_masking_policy_association.example 'ANALYTICS_PRODUCTION_DB.SNAPSHOTS.ON_CHURN_ARCHIVE'

as per the documentation.
https://registry.terraform.io/providers/Snowflake-Labs/snowflake/latest/docs/resources/tag_masking_policy_association
# format is tag database name | tag schema name | tag name | masking policy database | masking policy schema | masking policy name
terraform import module.tag.snowflake_tag_masking_policy_association.KIPI_ANALYSIS_DATA_MASKING_privacy_category_custom_PHI_REDACTED_MASK "KIPI_ANALYSIS|DATA_MASKING|PRIVACY_CATEGORY_CUSTOM|RAW_PRODUCTION_DB|PG_LOG_PUBLIC|PHI_REDACTED_MASK"

But while looking at the terraform state file I am not able to understand why everything is blank.
So this import statement is not able to import the resource correctly.
Please help me on this.

Category
category:resource
Object type(s)
No response
Expected Behavior
tag_id            = snowflake_tag.this.fully_qualified_name
masking_policy_id = snowflake_masking_policy.example_masking_policy.fully_qualified_name
Actual Behavior
{
"address":"module.tag.snowflake_tag_masking_policy_association.KIPI_ANALYSIS_DATA_MASKING_privacy_category_custom_PHI_REDACTED_MASK",
"mode":"managed",
"type":"snowflake_tag_masking_policy_association",
"name":"KIPI_ANALYSIS_DATA_MASKING_privacy_category_custom_PHI_REDACTED_MASK",
"provider_name":"registry.terraform.io/snowflake-labs/snowflake",
"schema_version":0,
"values":{
"id":"KIPI_ANALYSIS|DATA_MASKING|PRIVACY_CATEGORY_CUSTOM|RAW_PRODUCTION_DB|PG_LOG_PUBLIC|PHI_REDACTED_MASK",
"masking_policy_id":null,
"tag_id":null
},
"sensitive_values":{
                 }
              }

Steps to Reproduce
Import any snowflake_tag_masking_policy_association association in terraform state.
How much impact is this issue causing?
High
Logs
No response
Additional Information

Would you like to implement a fix?

[ ] Yeah, I'll take it 😎

Hi @saugandhik-kipi 👋
This resource will be deprecated in one of the releases - please use snowflake_tag_association like this:
resource "snowflake_tag_association" "example" {
  object_identifier {
    name     = snowflake_masking_policy.example_masking_policy.name
    database = snowflake_masking_policy.example_masking_policy.database
    schema   = snowflake_masking_policy.example_masking_policy.schema
  }
  object_type = "MASKING POLICY"
  tag_id      = snowflake_tag.this.fully_qualified_name
  tag_value   = "<tag_value>"
}

you can import it with
terraform import snowflake_tag_association.example 'KIPI_ANALYSIS.DATA_MASKING.PRIVACY_CATEGORY_CUSTOM'
If this still doesn't work, please provide logs with TF_LOG=DEBUG.
Note that snowflake_tag_association will also be reworked soon.

Hi @sfc-gh-jmichalak ,
Having similar issue with snowflake_tag_association.
Created a bug https://github.com/Snowflake-Labs/terraform-provider-snowflake/issues/3145

Sorry, I didn't connect these two. As it was mentioned in the comment there, I'll handle this during tag rework, but still, snowflake_tag_masking_policy_association will probably be deprecated.

Hi @saugandhik-kipi   👋
We've just released v0.100.0 (release, migration guide) in which we concluded rework of tags and tag associations. Please use masking_policies in the tag resource instead. For managing tags on objects, use the tag_association resource. Please upgrade to this version and let us know if you have any issues.


Update PTQ example to fix new compile_spec requirements

Signed-off-by: Nick Comly<EMAIL_ADDRESS>Description
Fixed compile(model, compile_spec) to use kwargs instead. See 3557747.
Another option is to just use **compile_spec but we have been have been pushing kwargs instead
Fixes # (issue)
Type of change
Please delete options that are not relevant and/or add your own.

Bug fix (non-breaking change which fixes an issue)

Checklist:

[ ] My code follows the style guidelines of this project (You can use the linters)
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas and hacks
[ ] I have made corresponding changes to the documentation
[ ] I have added tests to verify my fix or my feature
[ ] New and existing unit tests pass locally with my changes
[ ] I have added the relevant labels to my PR in so that relevant reviewers are notified

Note: I have not run this code (don't have a system). Would be good to run it once to ensure everything is correct.


Servercrash because of an error in SQL syntax

Why the server is crashing when I make this wrong query:

SELECT UNIX_TIMESTAMP(time) AS time FROM table_1 WHERE userID = '1' ORDER BY timeTaken LIM

Yes I know there is IMIT missing after LIM, but the server should not crash because of this little string-size-misstake.
Crashdetect-Log:

[debug] Server crashed while executing
[debug] AMX backtrace:
[debug] #0 native sql_num_rows () from sql.so
[debug] #1 0000b304 in ?? (0) from
[debug] #2 0000d1fc in public OnDialogResponse (0, 11000, 1, 0, 38220) from
[debug] Native backtrace:
[debug] #0 e632ce8b in _ZN10StackTraceC1EPv () from plugins/crashdetect.so
[debug] #1 e6325bcf in _ZN11CrashDetect20PrintNativeBacktraceERSoPv () from plugins/crashdetect.so
[debug] #2 e6326dbc in _ZN11CrashDetect20PrintNativeBacktraceEPv () from plugins/crashdetect.so
[debug] #3 e6327226 in _ZN11CrashDetect11OnExceptionEPv () from plugins/crashdetect.so
[debug] #4 e632cadc in ?? () from plugins/crashdetect.so
[debug] #5 e667e410 in __kernel_rt_sigreturn () from linux-gate.so.1
[debug] #6 e5f8db98 in _ZN7Natives12sql_num_rowsEP6tagAMXPi () from plugins/sql.so
[debug] #7 080950e4 in ?? () from sampServer
[debug] #8 e632894b in ZN11CrashDetect13DoAmxCallbackEiPiS0 () from plugins/crashdetect.so
[debug] #9 e632b8f8 in ?? () from plugins/crashdetect.so
[debug] #10 e632f916 in amx_Exec () from plugins/crashdetect.so
[debug] #11 e6327be6 in _ZN11CrashDetect9DoAmxExecEPii () from plugins/crashdetect.so
[debug] #12 e632b659 in ?? () from plugins/crashdetect.so
[debug] #13 e55882ea in ?? () from plugins/streamer.so
[debug] #14 080a31d4 in ?? () from sampServer
[debug] #15 080b2c05 in ?? () from sampServer
[debug] #16 08071d38 in ?? () from sampServer
[debug] #17 08071e32 in ?? () from sampServer
[debug] #18 0807bc50 in ?? () from sampServer
[debug] #19 080aed3d in ?? () from sampServer
[debug] #20 080aef02 in ?? () from sampServer
[debug] #21 080aa13a in ?? () from sampServer
[debug] #22 e6369a63 in __libc_start_main () from /lib32/libc.so.6

My fault, there is a problem with the FCNPC-Plugin.


Set up 404-page links and trigger

During the audit, we discover the links on the 404-page is not set up with the correct labels and links (see below).

The label is updated but for some reason, the links are not working and need dev assistance.

Help - https://ibm-dotcom-library.mybluemix.net/help
Slack - https://cognitive-app.slack.com/archives/C2PLX8GQ6
Github repo - https://github.com/carbon-design-system/ibm-dotcom-library/issues/new/choose

Task

[ ] Link the Help, Slack, and Github repo to its proper pages.
[ ] Figure out how to trigger the 404 on mybluemix.net, so all errors can be linked to the 404-page.

@jeffchew Let me know which dev will work on the task, please. Thank you.

@jeffchew Any update on this task?

Followed up with Jeff via Slack on 06/25.
Per Jeff:
I think it might involve webmaster team as they handle the overall 404 behavior on ibm.com. How important is it for us to have our own custom 404?
From Jenny:
The 404-error page is on our website but not for general IBM.com. It's in the Gatsby Theme. I updated the link label but the link (URL) is not working/connecting, so the task issue is created for the dev assistant around that - FYI.
The 404-error page is important  because as we have more traffic on the IBM.com Libray website and was presented with the 404-error page, they won't be stocked on that page.
also, tina williams owns the main 404 page

@jeffchew Please let me know if my comment above makes sense.

Reached out to webmaster team, haven't received a definitive answer on this yet but they pretty much own how redirects work, especially for the standard 404 behavior and we would need to find the appropriate way of bypassing it on the EI server. Even then, we might need approval from them for this change.

@jeffchew Thank you for the info. I'd like to keep this task open so we can keep an eye on it and continually record the progress. Please feel free to reach out if you have any further info from the webmaster team.

@RobertaJHahn Can we update the milestone to Sprint 20-14? Thank you.

@JennySanchez Done

@jeffchew Any updates from the Webmaster Team?

7/23/2020:

The 404 page is a standard Gatsby page.  Jenny is asking to have several links added to the page.  (Help, Slack, Github Repo)

Link to the DDS website 404
Link to the Carbon 404 page

The Carbon page works because they are hosted outside of IBM.  DDS website is hosted on EI.
We must work with the webmaster team to get Akami configured

Can we run a link checker on the website to identify the broken links on the site to be fixed?

More discussion is needed to define the next steps.

7/23/2020 Slack message from Jeff:

Task PR can be viewed here.


[refactor] mapstructure

As github.com/mitchellh/mapstructure was archived  on Jul 22, 2024 and miss a lot o features, don't handle well time.Time
I suggest to move to  github.com/go-viper/mapstructure/v2

@TuSKan Awesome work 💪


Initial Viewer CLI

To test:

From a folder outside of your repo, execute "node path/to/viewer/packages/modules/create-itwin-viewer/bin
Follow the prompts

When you run the application, you will receive a runtime error about a dompurify import that should be resolved in this PR
Still need to implement the uiConfig, the desktop template, and some other items. Requesting comments in the meantime

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.4 out of 5 committers have signed the CLA.:white_check_mark: aruniverse:white_check_mark: kckst8:white_check_mark: johnnyd710:white_check_mark: samejima-san:x: iTwinViewerWorkflowiTwinViewerWorkflow seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Update shrooms meta & inscriptions :)

attention: when use inscriptions as icon, the correct code is "inscription_icon", but not "icon". I helped you modify it. Please update your local meta.json file. @Bloudraad


fix(orchestrator): Fix a problem in the scheduler algorithm

When checking whether a worker can be executed in the current phase of an ORT run, all its transitive dependencies must be taken into account. Otherwise, it can happen that it is started too early if its direct dependencies are configured to be skipped.
Fixes #236.

Could we have this merged soon, as it's blocking some of the UI work?


Yobipy tests (and BuildSys)

This is ready for merge to master - I like to keep things in sync so branches don't drift too far apart.
In this PR:

"BuildSys" - a set of bash scripts that installs all required dependencies, compiles everything and runs tests, should work on any debian-based distro, we can extend it later for others and for OS X and such
Code coverage output (for yobipy, yobipyll and libyobicash test suites - ask me if you want to see the HTML output hosted on my server)
pytest fully setup - I can now start porting the unit tests from libyobicash into yobipy, then following that using a TDD approach, adding the implementations in

Tomorrow i'm going to take a break from the tests to implement the skeleton wallet app, that'll help guide what parts of libyobicash have priority when planning further development out.

Coverage remained the same at 87.386% when pulling 74cd6087823e6a138f5ad92089b3ea692dc54b42 on yobipy-tests into 7bbb855daab85d1519b2334f1f34413fd9484da1 on master.


Hotkey to hide/show arrows in case if you don't need them

As an "emergency" mode or simply because some don't need them arrows all the time.

Will implement when I figure out an elegant way to create 'hotkey' config elements.

Implemented in https://github.com/0mlml/chesshook/commit/ffa911e9b91dd8bf707ebdd207dbcc986afc7229 as "Clear Arrows Hotkey".


[HW]

Name: AYESHA HANNURE
Email<EMAIL_ADDRESS>Linkedin Profile: https://www.linkedin.com/in/ayesha-hannure-7b3205201/
Attach the homework step I screenshots below:
<

Attach the homework step II screenshots below:
< I am start working with the Ania Video , I will send Screen shot of that soon thank you for the Video it was very helpful  >

I am start working with the Ania Video , I will send Screen shot of that soon thank you for the Video it was very helpful
That's great, would love to see it

Hi Ayesha, looks like you have the local environment working just fine. Nice! When I checked your deployed site at https://tender-darwin-796422.netlify.app I just get the blank TikTok page which makes me wonder if your env vars were not injected into netlify. I suggest you repeat the steps from step 9 https://github.com/datastaxdevs/appdev-week2-tiktok#9-connect-netlify-to-your-site and see if you can get that working. Please provide a screen or the link to your deployed site with the videos loaded.

Nice job Ayesha, thanks for following up.

Hey AYESHA HANNURE, great job! Congrats! Here is your badge! https://api.badgr.io/public/assertions/KGc40QpvRjaAx2XmbpjxIQ


Unwrap, Masswrap & Massunwrap

Which problem is this PR solving?
Addresses unwrap, masswrap and massunwrap functionality as well as moving some things around to make things more Node friendly.
Short description of the changes

Added unwrap functionality + test
Added massWrap functionality + test
Added massUnwrap functionality + test
Moved the ESM specific wrap function logic to the node specific instrumentation.ts file so that we aren't doing Node specific things in methods that might be used in the browser.

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] Existing tests pass
[x] Added new tests for new functionality
[x] Tested with the ESM example app with http autoinstrumentation

Codecov Report

:exclamation: No coverage uploaded for pull request base (esm-support@b722486). Click here to learn what that means.
 Consider uploading reports for the commit 4401576 to get more accurate results

@@              Coverage Diff               @@
##             esm-support       #3   +/-   ##
==============================================
  Coverage               ?   92.90%
==============================================
  Files                  ?      143
  Lines                  ?     3409
  Branches               ?      701
==============================================
  Hits                   ?     3167
  Misses                 ?      242
  Partials               ?        0

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Update DataFrame.merge docstring

Partially resolves #8769

The failing windows-py3.8 test is a known flaky test, unrelated to this PR. :)

@bryanwweber This is a quick docs update, would you mind taking a look?


.removeProperty that is an embeddedList still returns array with null value in select.

OrientDB Version: 3.0
Java Version: 1.8
OS: Centos
Hi,
In Hooks, I do vertex.removeProperty("embedded_list_property") but when I select it still comes back.  Not sure if the problem occurs outside of Hooks as well.
Thanks,
Safwan

Hi @luigidellaquila
I believe, this is related to not calling save() in Hooks.  if I don't call save(), the item is actually removed from the list and is reflected on the database with a [null] but the removeProperty will not work unless save() is called.  I have a similar issue where changes to vertex are taken without a call to save(), they should not unless a specific save() is called, I think that would be best.
Thanks,
Safwan


Rotation matrix output columns aligned.

Hello,
I love this library.  I changed the format of the output rotation matrix so that the columns align.  To do this I added a function to convert to fixed width strings of each matrix element.
Mark Brudnak

I made all requested changes.

Thanks again for your patch, merged it.


Add 'webapp' short name to Razor Pages web application template

So that folks can do dotnet new webapp

This might not be possible with the current features of dotnet new 😢

@mlorbetske / @seancpeters - would you guys need to build a feature in order to make this possible? Does it boil down to adding some sort of "alias" feature or allowing multiple short names for templates?

I actually recently made a PR to allow templates to have multiple short names, here:
https://github.com/dotnet/templating/pull/1431
As noted in that PR, with this change:
Short name can be either a string or an array of strings. When in array context, the first entry in the array will be the “preferred” short name for the template.
Displaying a list of templates using dotnet new: when there are multiple templates installed with the same group identity, those templates are coalesced into one entry in the list, using info from the template with the highest precedence when there is a need to choose among the templates (this is existing behavior). The displayed short name will be preferred short name from the highest precedence template in the group.
Attempting to invoke a template using dotnet new: Based on the user inputs, a search is performed on the installed templates to determine the best matched template. The first argument on the command line represents the template “name”, which is used for substring matching on the template name and short name. When there are multiple templates installed with the same group identity, all the short names for all the templates in the group will be considered as short names for every template in the group. This search behavior will be applicable when a template gets invoked because it’s the single unambiguous match. It’s also used when the match is ambiguous, causing a template list or help on a template group to be displayed; and also when the –list or –help flags are explicitly provided by the user.

@seancpeters sounds cool! We'll look at using that feature once it's available in builds.

@kichalla, can you please look into this? Thanks!

Moving to RC1.

It looks like this is present in CLI version 2.1.300-preview2-008530 (dotnet/templating commit hash 9331d3937ca61f1183ffab80ecba6bd6ce9ba3d9)

I tried https://github.com/dotnet/cli 2.1.300-preview3-008443 which did not work

I can try the preview2 version you suggested. Thanks

Is this a prodcon build? If not, then we'll need to manually insert update bits for the change to be there.

See #477


📎 Create a new git hooks recipe

Description
Essentially, on this page we want to add some recipes for how to add Biome to commit hooks:

lint-staged
lefthook
etc.

@nhedger:  I know you set up a commit hook in biome-vscode, this knowledge could live in that page
@Conaclos : I know you created a repository around this, but honestly, I don't know what that is about. We can have some knowledge in that page
The name of the page isn't set on stone, feel free to come up with a better name

Hey! I'm using a simplified pre-commit hook only for formatting. Putting here in case it's useful for this task:
#!/usr/bin/env sh

EXTENSION="js|jsx|ts|tsx|json|mjs|cjs"
STAGED_FILES=$(git diff --cached --name-only --diff-filter=ACM | grep -E "(${EXTENSION})$" | tr '\n' ' ')

if [ -z "$STAGED_FILES" ]; then
  exit 0
fi

yarn biome format --write --colors=off $STAGED_FILES
git add $STAGED_FILES

if git diff --cached --quiet; then
  echo "No changes left to commit. Aborting."
  exit 1
fi

Hey! I'm using a simplified pre-commit hook only for formatting.

Any downsides to running check instead of format have you been bit by something?

Hey! I'm using a simplified pre-commit hook only for formatting.

Any downsides to running check instead of format have you been bit by something?

check will also run the linter, so I guess it depends on what you're trying to achieve.


Deprecate classes Variable, Variables, Datasets

Since the classes Datasets, Variable and Variables are not maintained anymore, they should be deprecated. See discussion here: https://github.com/ESMValGroup/ESMValTool/issues/1925#issuecomment-739987634
@bouweandela would it make sense to add a test that fails in version 2.3 to remind us to remove these functions?

Yes, that might be nice if it's not too much work.
I realized I made a mistake with the version numbers in my earlier comment though, I think it would be best to keep deprecated features at least two minor releases, so this will then be deprecated (i.e. scheduled for removal) in v2.2 and actually removed in v2.4. That will probably be next autumn, that should give people enough time to adjust their code.

@jvegasbsc @sloosvel I hope it's not too big a problem for you if we remove this feature, since you're using it in the so far unmerged PRIMAVERA diagnostics. It would be really nice if we could remove this, because it will leave us with a single way of doing things and that would smoothen moving parts of this code to ESMValCore (see #1873).

Not a problem at all, it will be easier to maintain in the future


[FEATURE] Don't display passive effects

Is your feature request related to a problem? Please describe.
Right now, this great module displays the passive effects.
The user can right-click and delete a passive effect.
Describe the solution you'd like
An option to not display passive effects (list can be quite long)
Describe alternatives you've considered
At least, do not disable/delete passive effect on right-click.

Just saw that the following are also displayed and shouldn't:

Disabled effects
Suspended effects

Good catches, thanks! I'll remove displaying suspended/disabled, and add a setting for showing passive or not.

Fixed in v1.1.0


Bug: Motion Sensor on C200 not detected or unavailable

Describe the bug
C200 no longer gets motion detection (entity) detected thus cannot be used.
Works fine on APP.
To Reproduce
Steps to reproduce the behavior:

Upgrade to 1.1.15 Build 211111 Rel. 34670n(4555)
Add to Home Assistant
Check Entities Detected.
See error

Expected behavior
Motion Detector would be available.

ONVIF

Log
If applicable, add error logs.
Camera (please complete the following information):

Device Model: C200
FW: 1.1.15 Build 211111 Rel.34670n(4555)

Using stream component
yes
Camera works through onvif component OR different cameras work in general
yes
Camera has all attributes filled out properly in developer tools
Yes
model_name: C200
brand: TP-Link
motion_detection: 'on'
device_type: SMART.IPCAMERA
device_model: C200
device_name: C200 1.0
device_info: C200 1.0 IPC
hw_version: '1.0'
sw_version: 1.1.15 Build 211111 Rel.34670n(4555)
device_alias: Tapo_Camera_E999
features: '3'
barcode: ''
motion_detection_sensitivity: normal
privacy_mode: 'off'
alarm: 'off'
alarm_mode:
  - sound
led: 'on'
day_night_mode: auto
auto_track: 'off'
presets: {}

HASS Environment
Docker on Unraid (Home-Assistant-Core) with HACS added.

Read readme troubleshooting section.
Also, if motion sensor is not available also inside onvif, it is not an issue of this integration.

Read readme troubleshooting section.
Also, if motion sensor is not available also inside onvif, it is not an issue of this integration.

It was available before the update as far as I could see it, then it went to Unavailable.
It was available for sometime then gone. Not sure why.
Do you have a C200 with motion sensor working with HA?

What is written in troubleshooting section in readme?
All that you are asking is written in readme.

What is written in troubleshooting section in readme?
All that you are asking is written in readme.

Not sure why it went AWOL but after triggering a new detection it does show up again, on the new firmware.

If you read the readme like I told you 3 times now you what know 🤷

If you read the readme like I told you 3 times now you would know shrug
Sure. Thanks


Add support for Nodejs16.x

cfn-lint version: (cfn-lint --version)
Description of issue.
AWS has added support for nodejs16.x for Lambda runtime. However this will generate an error using cfn-lint as it doesnt support this yet
E3030 You must specify a valid value for Runtime (nodejs16.x). Valid values are ["dotnetcore1.0", "dotnetcore2.0", "dotnetcore2.1", "dotnetcore3.1", "go1.x", "java11", "java8", "java8.al2", "nodejs", "nodejs10.x", "nodejs12.x", "nodejs14.x", "nodejs4.3", "nodejs4.3-edge", "nodejs6.10", "nodejs8.10", "provided", "provided.al2", "python2.7", "python3.6", "python3.7", "python3.8", "python3.9", "ruby2.5", "ruby2.7"]

Included in #2261
#2263
Should be included in any release since v0.60.0

Is support for Nodejs16.x added in previous releases? I cannot see it listed in any recent releases https://github.com/aws-cloudformation/cfn-lint/releases?page=1

Its included the automated maintenance scripts so it always doesn't get automatically called out.
From the current version of the spec
"LambdaRuntime": {
      "AllowedValues": [
        "dotnet6",
        "dotnetcore1.0",
        "dotnetcore2.0",
        "dotnetcore2.1",
        "dotnetcore3.1",
        "go1.x",
        "java11",
        "java8",
        "java8.al2",
        "nodejs",
        "nodejs10.x",
        "nodejs12.x",
        "nodejs14.x",
        "nodejs16.x",
        "nodejs4.3",
        "nodejs4.3-edge",
        "nodejs6.10",
        "nodejs8.10",
        "provided",
        "provided.al2",
        "python2.7",
        "python3.6",
        "python3.7",
        "python3.8",
        "python3.9",
        "ruby2.5",
        "ruby2.7"
      ]
    },

whats your cfn-lint --version output?


SLIVR Volume bad image on Windows

Open multi_rect3d.siloAdd volume plot of 'd'.Turn off lighting.Draw.Image should have green circles on center faces, fading through yellow to orange at the edges of the faces.Change rendering method to SLIVR. You will see a lot of blue, and no longer a square, but stepped face.I've attached two images, 1 rendered with 3d Texture, 1 with SLIVR.
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 1633
Status: Rejected
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: SLIVR Volume bad image on Windows
Assigned to: -
Category: -
Target version: -
Author: Kathleen Biagas
Start: 10/29/2013
Due date:
% Done: 0%
Estimated time:
Created: 10/29/2013 01:01 pm
Updated: 11/19/2013 02:02 pm
Likelihood: 3 - Occasional
Severity: 4 - Crash / Wrong Results
Found in version: trunk
Impact:
Expected Use:
OS: Windows
Support Group: Any
Description:
Open multi_rect3d.siloAdd volume plot of 'd'.Turn off lighting.Draw.Image should have green circles on center faces, fading through yellow to orange at the edges of the faces.Change rendering method to SLIVR. You will see a lot of blue, and no longer a square, but stepped face.I've attached two images, 1 rendered with 3d Texture, 1 with SLIVR.
Comments:
Apparently I had a bad silo file, as I can no longer reproduce.


SD PromptSaver node adds an extra 12seconds to first run

Description
I'm seeing a 12 second run time that isn't there for other save nodes, and that is really hurting me when using comfy with an API. It seems like the SD PromptSaver node might actually be flushing the flux model and reloading it into memory; I'm guessing that if not using the SD ParameterGenerator, this is what you're doing to get model name, which seems really inneficient. if model name could just be a string that would be ideal, or really anything that wouldn't trigger a reload
Reproduction steps
run any basic workflow using the SD PromptSaver, take a look at the inference time reported for that node on the first run from a cold boot
Image file
first run:

second run:

first run with a dummy empty model nullModel.ckpt:

interestingly enough, if i convert model_name to an input and don't connect anything, this isn't a problem


Validating arrays

Hi, I dont think this handles array validation in 5.1. Which can be seen being used here:
https://ericlbarnes.com/2015/04/04/laravel-array-validation/
Are you aware of a way I can overcome this or could this be an idea for a feature?

Are you ok with me pulling this and pushing a fix for this?

I have opened a pull request for this:
https://github.com/JesseLeite/Laravel-Bootstrap-Modal-Form/pull/6


Fix + appearing on ConfigurableSources in Browse > Extensions

"+" needs to be added only if the following conditions are met:

Extension is Installed and is a ConfigurableSource (regardless of it
having an Update or not)
Extension is in InstallStep.Idle state

Also tried to align the text (Settings/Settings+ etc) to the right end rather
than letting it be centred. Can remove if not required.
Images

Before
Now
Now (non-Idle state)

I would remove the text alignment for now, I dont feel there is that good of a difference. Also put the first equality check on the next line

Also put the first equality check on the next line

nice catch, thanks! If I understand correctly, installStep.idle is true more often than ext.installed on average and so, having the installed check first can help skip if block faster (think its called shorting the circuit)


Suggested feature: Add ability to inspect NUTS running diagnostics

Allow user to look into tree size and step size in each NUTS iteration
Consider sampler is a nut sampler, the user can plot
fig, ax= plt.subplots(1,2, figsize=(8,3))
plt.suptitle('N='+ str(N) +', Nb='+ str(Nb) + ', max_depth='+ str(sampler.max_depth))
plt.sca(ax[0])
plt.plot(sampler.iteration_list, sampler.num_tree_node_list, label='num_tree_node')
plt.legend()
plt.sca(ax[1])
plt.plot(sampler.iteration_list, sampler.epsilon_list, label='epsilon')
plt.plot(sampler.iteration_list, sampler.epsilon_bar_list, label='epsilon_bar')
plt.legend()

@jakobsj @nabriis
The PR associated with this issue adds ability to store epsilon, epsilon_bar (both related to the nuts step size and affect the tree size -- heuristically speaking), number of nodes per iteration inside the NUTS sampler, after sampling. I am adding you both as reviewers to the PR. I appreciate it if you can take a look and let me know what you think.
The second plot is the one I showed in the meeting but in semiology scale here. These two plots are for the Poisson paper II example.

For the sake of some completion here, I show below the trace and ci plot for the two cases above (Nb=20 and Nb =30), Ns=10 for both cases.
Nb=20:

Nb = 30


Fix parse functions that weren't properly updated

This fixes issues with #89. I didn't properly update the ParseOpenFlagArgument and ParseExecFlag helpers to remove the fact that they're methods.
Signed-off-by: grantseltzer<EMAIL_ADDRESS>

LGTM. I'm assuming you will now change all the calls to helpers.Parse*** in (t *Tracee) parseArgs(...) function. Right ?

Yes, I will update the libbpfgo dependency in tracee and fix the breaking changes as soon as I cut the libbpfgo release today.


KOGITO-3036: Remove KafkaClient from Kogito Runtimes

JIRA: https://issues.redhat.com/browse/KOGITO-3036
Description: Remove KafkaClient utility as it's causing issues running the pipelines and does not bring up any particular functionality.
Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[x] You have read the contributors guide
[x] Pull Request title is properly formatted: KOGITO-XYZ Subject
[x] Pull Request title contains the target branch if not targeting master: [0.9.x] KOGITO-XYZ Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains link to any dependent or related Pull Request
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket

It relates to https://github.com/kiegroup/kogito-examples/pull/329 (but not blocking)


Support #[ignore] attribute in defmt_test

This PR adds support for the #[ignore] attribute in defmt_test to solve #390. Similar to the ignore attribute, tests can be annotated with the attribute to ignore particular tests.

use trybuild crate to check multiple conditions with the proc macro

Note the message is different from the suggestion in #390 and should be revised accordingly if needed

bors r+


CI:  run tests via grunt instead of npm?

"If you are always going to need some node modules like grunt I would suggest making your own image instead of using our node base"
https://discuss.circleci.com/t/global-npm-grunt-etc-with-workflows-and-jobs/14226/2

Per Jack and Drew, it will be better to use the node scripts (in package.json) approach.


Don't use SSLv3 method

The new version of openssl in Debian has been compiled without ssl3 support. This means that the SSLv3_method function no longer is available. For this reason the compilation of canl-c fails:
https://buildd.debian.org/status/package.php?p=voms
error: 'SSLv3_method' was not declared in this scope
I don't know what is the best way to fix this. One way is to use SSLv23_method instead, as in this pull request.
But you might have other ideas.

Thanks Mattias,

I don't know what is the best way to fix this. One way is to use SSLv23_method instead, as in this pull
request. But you might have other ideas.

Probably it's time to move  to TLS_method or something  along those lines, but it will have to be tested..
Thanks,
Andrea

SSLv23_method means "Negotiate highest available SSL/TLS version".
All the others requests one specific version. I think "highest available" makes sense - but the lowest versions available should be disallowed using SSL_OP_NO_SSLv2 and really also SSL_OP_NO_SSLv3.
in Client.cpp it currently says SSL_OP_NO_TLSv1 | SSL_OP_NO_SSLv2, which is a bit weird, This allows SSLv3, TLSv1.1 and TLSv1.2, but it disallows TLSv1 which is between SSLv3 and TLSv1.1. But maybe there is a reason for this discontinuous setting.

According to this:
https://www.openssl.org/docs/manmaster/ssl/SSL_CTX_new.html
SSLv23_method is deprecated.

SSLv23_method(void), SSLv23_server_method(void), SSLv23_client_method(void)
Use of these functions is deprecated. They have been replaced with TLS_Method(),
TLS_server_method() and TLS_client_method() respectively.
New code should use those functions instead.

It also says "TLS_method, TLS_server_method and TLS_client_method were introduced in OpenSSL 1.1.0." Which is why I hadn't heard of them. Even Fedora rawhide and Debian testing are still using OpenSSL 1.0.2.

Yes, you're right. I guess it will be SSLv23 for the next 5 or 6 years then :)
Will merge this, but on the develop branch.
Thanks.


Use system-ui font by default

Before

After

Info
system-ui is the font that your browser uses to render the 'chrome' UI: toolbars and menus. It is the most 'neutral' font for UI. inherit would use the default font by the display system, which is likely a font designed for paragraphs of text, not UI.
https://drafts.csswg.org/css-fonts-4/#system-ui-def

bump

Sorry for the very late reply.
I think this PR is reasonable. However, I don't think "inherit" is a bad choice.
Setting "system-ui" disables font specification on the container side.


add associations

In this project, I worked on
- New Classroom, Rental, and Book classes
- Has-many/belongs-to relationship between Classroom and Student
- Many-to-many relationship between Person and Book using the intermediate class Rental

Hi @osbentech,
While you made a great effort in this project, I cannot proceed to review your code.
Invalid Code Review Request
Kindly ensure you push your changes before requesting for review. Your MIT link in the readme is still broken as pointed out earlier.

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Native go fuzzing

Please provide a description of this PR:

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@howardjohn: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
110644e3ff4878da1d18c8c616f4d921cabb81a7
link
true
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
110644e3ff4878da1d18c8c616f4d921cabb81a7
link
true
/test release-notes_istio

lint_istio
110644e3ff4878da1d18c8c616f4d921cabb81a7
link
true
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
802900d15da17388e0448f292c12eaff628c59f4
link
true
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
802900d15da17388e0448f292c12eaff628c59f4
link
true
/test lint_istio

unit-tests_istio
802900d15da17388e0448f292c12eaff628c59f4
link
true
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


coq: lablgtk is only needed for coqide

Motivation for this change
Being able to build Coq without building lablgtk.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg eval
(sorry for the noise, master was broken by a merge)


Provide Angular's LOCALE_ID

This pull request requires a change to @skyux/i18n.
See: https://github.com/blackbaud/skyux-i18n/pull/22
Purpose: This change will automatically set Angular's LOCALE_ID based on what's provided by SkyAppHostLocaleProvider. The side effect is that Angular's DatePipe (and any other localized utilities) will be automatically provided with the same locale string.

Codecov Report

Merging #515 into master will decrease coverage by 0.05%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master     #515      +/-   ##
==========================================
- Coverage     100%   99.94%   -0.06%
==========================================
  Files          54       54
  Lines        1724     1729       +5
  Branches      257      258       +1
==========================================
+ Hits         1724     1728       +4
- Misses          0        1       +1

Flag
Coverage Δ

#builder
100% <ø> (ø)
:arrow_up:

#runtime
96.87% <88.88%> (-3.13%)
:arrow_down:

#srcapp
100% <ø> (ø)
:arrow_up:

Impacted Files
Coverage Δ

lib/sky-pages-module-generator.js
100% <ø> (ø)
:arrow_up:

runtime/i18n/host-locale-provider.ts
93.75% <88.88%> (-6.25%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 36a0bdc...c127073. Read the comment docs.

Closing in favor of SDK Builder.


[Feature]: Add hide_name configuration variable to fan card

Is your feature request related to a problem?
Sort of. The only way to hide the name is to set it to a space (' '), but that appears to take up space resulting in alignment issues.
Describe the solution you'd like
Add a configuration variable to fan card (and maybe the other cards that don't have it) that would provide a way to completely remove the name field.
Describe alternatives you've considered
Set name to a space (' '), but that appears to take up space resulting in alignment issues.
Additional context
No response

Hello 🙂
It will be possible with https://github.com/piitaya/lovelace-mushroom/pull/528


Don't free instanced scenes when recreating tiles

Fixes #58034
Fixes #68347
Scenes are no longer re-instantiated when quadrant is recreated, including when setting cells and changing layer properties (modulate etc). Only modifying TileSet will recreate the scenes (although there's high chance they'll just get unlinked at that point...).
To me it's rather weird that the scenes are re-instanced at all. The purpose of scene collections is to put objects on your level. At no point you want them freed and recreated by the TileMap.
That said, not sure if my solution is good. Probably not 🙃

Hmm, it does not look right to me. How do you know whether or not the packed scene was modified or not ?
If I am not mistaking, this function is also called when the TileSet is modified, which might cause cached scenes to be wrong.

If I am not mistaking, this function is also called when the TileSet is modified, which might cause cached scenes to be wrong.

When exactly this can happen at runtime?

When exactly this can happen at runtime?

I guess procedural modifications of the TileSet, which is possible right now.
I guess that the "runtime" check also means that in the editor, the issue of scene "resetting" will still be there, though most of the time they have no tool scripts so that's fine.
To be honest, I am not perfectly happy with that implementation. I think we should make sure somehow that the underlying PackedScene wasn't modified. This, I suppose, should be maybe done by passing an argument to the update functions ?
(This seems the simplest solution IMO)

I guess procedural modifications of the TileSet, which is possible right now.

I reset the instanced scenes on tileset changed, so this is handled.

To be honest, I am not perfectly happy with that implementation. I think we should make sure somehow that the underlying PackedScene wasn't modified. This, I suppose, should be maybe done by passing an argument to the update functions ?

The way I implemented it, only tiles that weren't instantiated previously will instantiate. Not sure what argument should be passed to preserve this behavior. You need to store the used tiles somewhere.
Also not sure what you mean by modified scene. Scenes don't get modified at runtime, unless you use live edit.
I'm also not 100% happy with this implementation (as I noted in the OP), but I don't have much better idea.

I reset the instanced scenes on tileset changed, so this is handled.

Ah that's true, my bad.

The way I implemented it, only tiles that weren't instantiated previously will instantiate. Not sure what argument should be passed to preserve this behavior. You need to store the used tiles somewhere.

The idea would be to have a p_tile_set_modified argument in _scenes_update_dirty_quadrants that will bypass the cache if the TileSet resource was updated.
I think ultimately it would allow optimizing quadrants even more, to only update parts of the quadrant that need an update. But well that might be for later.

Also not sure what you mean by modified scene. Scenes don't get modified at runtime, unless you use live edit.

Not the scene itself, but you can replace a scene in the TileSet resource at runtime. For example using tileset.get_source(0).set_scene_tile_scene(id, new_packed_scene). Which is a valid use case IMO (though not the most efficient for sure).
Anyway, I didn't see the cache was cleared when the TileSet was modified, so I think this implementation should be fine in the end. I feel an implementation with the added argument as I suggested could maybe be cleaner, but I am not sure about how much rewrite that would need. So well, as you prefer. :)

The idea would be to have a p_tile_set_modified argument in _scenes_update_dirty_quadrants that will bypass the cache if the TileSet resource was updated.

What does that change exactly? Clearing cache is the same as bypassing it. You need to remove the old node and cache the new one. (although it might change something if the used cells have changed)
Also what about _scenes_cleanup_quadrant()?
tbh I'm not sure how scene tiles are supposed to work. I'd just make them instance and forget, no clearing.

What does that change exactly? Clearing cache is the same as bypassing it. You need to remove the old node and cache the new one. (although it might change something if the used cells have changed)

I think it's more about the way the code is organized right now, where a method is dedicated to each type of sources, and I avoided spreading the responsibilities over. So like, I'd rather have the scene cache cleaning the method dedicated to scene tiles rather than having it in the generic _tileset_changed callback. But I don't mind much to be honest, it is more of a cosmetic change.
Even if I didn't have the time to implement it, and to find interesting use cases, I kind of wanted to allow users to implement their own TileSetSource if needed. So I kept things modular just in case. This special handling in the  _tileset_changed function kind of goes against this modularity, but since it's not fully implemented anyway, it's not a big deal.

Also what about _scenes_cleanup_quadrant()?

I guess I would have added the same argument.

Thanks!

I think this PR made it so that it's not possible to use set_cell with a scene tile if that tile is not empty (already had a value wether it was a regular tile or a scene tile) because all tiles are added to the instantiated_scenes set, not only the scene tiles, and they are also never removed for that set (the set is only emptied when the tileset changes). See the attached MRE, left click doesn't place new tiles on the bottoom layer which already has tiles set while right click works on an empty layer above.
TestPR.zip


[BUG] Cannot import parts

Hi,
I am unable to import Parts from file using procedure from the manual.
Steps to Reproduce
Steps to reproduce the behavior:

Go to InvenTree Admin \Parts \ Import
Select Excel file (previously prepared by Export option)
Press import

Cannot call get_descendants on unsaved Part instances
1, None, None, None, None, None, None, None, None, None, None, None, Test, None, Description, None, None, None, None, None, None, None, None, None, None, None, None, None, None, None, None, None, None, None, None, None
Traceback (most recent call last):
File "/home/inventree/.local/lib/python3.9/site-packages/import_export/resources.py", line 644, in import_row
diff = self.get_diff_class()(self, original, new)
File "/home/inventree/.local/lib/python3.9/site-packages/import_export/resources.py", line 204, in init
self.left = self._export_resource_fields(resource, instance)
File "/home/inventree/.local/lib/python3.9/site-packages/import_export/resources.py", line 225, in _export_resource_fields
return [resource.export_field(f, instance) if instance else "" for f in resource.get_user_visible_fields()]
File "/home/inventree/.local/lib/python3.9/site-packages/import_export/resources.py", line 225, in
return [resource.export_field(f, instance) if instance else "" for f in resource.get_user_visible_fields()]
File "/home/inventree/.local/lib/python3.9/site-packages/import_export/resources.py", line 845, in export_field
return field.export(obj)
File "/home/inventree/.local/lib/python3.9/site-packages/import_export/fields.py", line 122, in export
value = self.get_value(obj)
File "/home/inventree/.local/lib/python3.9/site-packages/import_export/fields.py", line 98, in get_value
value = value()
File "/home/inventree/InvenTree/part/models.py", line 1403, in allocation_count
self.build_order_allocation_count(**kwargs),
File "/home/inventree/InvenTree/part/models.py", line 1327, in build_order_allocation_count
query = self.build_order_allocations(**kwargs).aggregate(
File "/home/inventree/InvenTree/part/models.py", line 1313, in build_order_allocations
variants = self.get_descendants(include_self=True)
File "/home/inventree/.local/lib/python3.9/site-packages/mptt/models.py", line 391, in _fn
raise ValueError(
ValueError: Cannot call get_descendants on unsaved Part instances
Deployment Method
Docker
Version Information
0.7.0

Hey @jkanaka thanks for reporting. This will be fixed in the upcoming 0.8.0 release - see https://github.com/inventree/InvenTree/pull/3235
Please note that the whole import process will be overhauled in the near future - https://github.com/inventree/InvenTree/issues/3101