weather-mv bq raster issue while reading ecmwf grib file

weather-mv is unable to read ecmwf grib files with the following set of Raster errors being thrown out. Here is the command trace:
`weather-tools) jupyter@gi-asset:~/weather-tools$ python weather_mv/weather-mv bigquery --uris "gs://gi_asset-ecmwf-ensemble-data/hres-sample/single-levels_hres_*.gb" --output_table "$PROJECT.ecmwf.hres-sample-check" --temp_location "gs://gi_asset-ecmwf-ensemble-data/tmp/" --runner DataflowRunner --num_workers 2 --project $PROJECT --region  us-central1 --job_name hrse-sample-mv-003 --disk_size_gb 200
no previously-included directories found matching 'test_data'
INFO:loader_pipeline.bq:Validating regions for data migration. This might take a few seconds...
INFO:loader_pipeline.bq:Region validation completed successfully.
INFO:apache_beam.internal.gcp.auth:Setting socket default timeout to 60 seconds.
INFO:apache_beam.internal.gcp.auth:socket default timeout is 60.0 seconds.
INFO:apache_beam.io.gcp.gcsio:Starting the file information of the input
INFO:apache_beam.io.gcp.gcsio:Finished listing 1 files in 0.05787062644958496 seconds.
WARNING:loader_pipeline.sinks:Assuming grib.
INFO:loader_pipeline.sinks:Normalizing the grib schema, name of the data variables will look like '<attrs['GRIB_stepType']>_'.
ERROR:loader_pipeline.sinks:Unable to open file 'gs://gi_asset-ecmwf-ensemble-data/hres-sample/single-levels_hres_2020-01-01T00_00_00z-u100-v100-u10-v10-u200-v200-2t-2d-ssr-str-sp-msl-tprate-ptype-blh-sr-tp.gb': only size-1 arrays can be converted to Python scalars
Traceback (most recent call last):
File "weather_mv/weather-mv", line 74, in
cli(['--extra_package', pkg_archive])
File "/home/jupyter/weather-tools/weather_mv/loader_pipeline/init.py", line 23, in cli
pipeline(*run(sys.argv + extra))
File "/home/jupyter/weather-tools/weather_mv/loader_pipeline/pipeline.py", line 71, in pipeline
paths | "MoveToBigQuery" >> ToBigQuery.from_kwargs(vars(known_args))
File "/home/jupyter/weather-tools/weather_mv/loader_pipeline/sinks.py", line 55, in from_kwargs
return cls({k: v for k, v, in kwargs.items() if k in fields})
File "", line 17, in init
File "/home/jupyter/weather-tools/weather_mv/loader_pipeline/bq.py", line 154, in post_init
with open_dataset(self.first_uri, self.xarray_open_dataset_kwargs,
File "/opt/conda/envs/weather-tools/lib/python3.8/contextlib.py", line 113, in enter
return next(self.gen)
File "/home/jupyter/weather-tools/weather_mv/loader_pipeline/sinks.py", line 385, in open_dataset
xr_dataset: xr.Dataset = __open_dataset_file(local_path,
File "/home/jupyter/weather-tools/weather_mv/loader_pipeline/sinks.py", line 311, in __open_dataset_file
return _add_is_normalized_attr(__normalize_grib_dataset(filename), True)
File "/home/jupyter/weather-tools/weather_mv/loader_pipeline/sinks.py", line 229, in __normalize_grib_dataset
forecast_hour = int(da.step.values / np.timedelta64(1, 'h'))
TypeError: only size-1 arrays can be converted to Python scalars
(weather-tools) jupyter@gi-python:~/weather-tools$ python weather_mv/weather-mv bigquery --uris "gs://gi_asset-ecmwf-ensemble-data/hres-sample/2020-01-01T00:00:00z-u100-v100-u10-v10-u200-v200-2t-2d-ssr-str-sp-msl-tprate-ptype-blh-sr-tp.gb" --output_table "megatron-389205.ecmwf.hres-sample-check" --temp_location "gs://gi_asset-ecmwf-ensemble-data/tmp/loadprocess/" --runner DataflowRunner --num_workers 2 --project megatron-389205 --region  us-central1 --job_name hrse-sample-mv-003 --disk_size_gb 200 --disable_grib_schema_normalization
no previously-included directories found matching 'test_data'
INFO:loader_pipeline.bq:Validating regions for data migration. This might take a few seconds...
INFO:loader_pipeline.bq:Region validation completed successfully.
INFO:apache_beam.internal.gcp.auth:Setting socket default timeout to 60 seconds.
INFO:apache_beam.internal.gcp.auth:socket default timeout is 60.0 seconds.
WARNING:loader_pipeline.sinks:Assuming grib edition 1.
INFO:rasterio._env:GDAL signalled an error: err_no=4, msg='/var/tmp/tmphj2fbdcj is a grib file, but no raster dataset was successfully identified.'
ERROR:loader_pipeline.sinks:Unable to open file 'gs://gi_asset-ecmwf-ensemble-data/hres-sample/2020-01-01T00:00:00z-u100-v100-u10-v10-u200-v200-2t-2d-ssr-str-sp-msl-tprate-ptype-blh-sr-tp.gb': /var/tmp/tmphj2fbdcj is a grib file, but no raster dataset was successfully identified.
Traceback (most recent call last):
File "rasterio/_base.pyx", line 302, in rasterio._base.DatasetBase.init
File "rasterio/_base.pyx", line 213, in rasterio._base.open_dataset
File "rasterio/_err.pyx", line 217, in rasterio._err.exc_wrap_pointer
rasterio._err.CPLE_OpenFailedError: /var/tmp/tmphj2fbdcj is a grib file, but no raster dataset was successfully identified.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "weather_mv/weather-mv", line 74, in
cli(['--extra_package', pkg_archive])
File "/home/jupyter/weather-tools/weather_mv/loader_pipeline/init.py", line 23, in cli
pipeline(*run(sys.argv + extra))
File "/home/jupyter/weather-tools/weather_mv/loader_pipeline/pipeline.py", line 71, in pipeline
paths | "MoveToBigQuery" >> ToBigQuery.from_kwargs(vars(known_args))
File "/home/jupyter/weather-tools/weather_mv/loader_pipeline/sinks.py", line 55, in from_kwargs
return cls({k: v for k, v, in kwargs.items() if k in fields})
File "", line 17, in init
File "/home/jupyter/weather-tools/weather_mv/loader_pipeline/bq.py", line 154, in post_init
with open_dataset(self.first_uri, self.xarray_open_dataset_kwargs,
File "/opt/conda/envs/weather-tools/lib/python3.8/contextlib.py", line 113, in enter
return next(self.gen)
File "/home/jupyter/weather-tools/weather_mv/loader_pipeline/sinks.py", line 399, in open_dataset
with rasterio.open(local_path, 'r') as f:
File "/opt/conda/envs/weather-tools/lib/python3.8/site-packages/rasterio/env.py", line 442, in wrapper
return f(*args, **kwds)
File "/opt/conda/envs/weather-tools/lib/python3.8/site-packages/rasterio/init.py", line 277, in open
dataset = DatasetReader(path, driver=driver, sharing=sharing, **kwargs)
File "rasterio/_base.pyx", line 304, in rasterio._base.DatasetBase.init
rasterio.errors.RasterioIOError: /var/tmp/tmphj2fbdcj is a grib file, but no raster dataset was successfully identified.
`
The sample file mentioned can be read without any issues locally using cfgrib open datasets. Please help in troubleshooting/fixing the above issue with weather-mv

Hey Hemanand! I think what's going on here is that we're trying to user rasterio (and thus GDAL) to parse the CRS information for the data. Searching around, it seems like GDAL may not support the type of grid that these files are using.
Can you run grib_ls and paste it here?
I think the best path forward to fix this issue for weather-mv bq would be to catch the rasterio error and set default projection information and then return the dataset. Projection info is not relevant for this data sink anyway.

Hey Alex,
Attached the file here.
grib_ls.pdf


Primary key can not int64?

Arithmetic overflow error when converting expression to data type int

Users table

public Int64Field UserId;


Flags parsing error

I am trying to do test run for just installed Alphafold.  However, stuck with the below error message ?

I strongly recommend running AlphaFold using the way described in the readme, i.e. using docker/run_docker.py which will set a lot of things up for you, including many of the flags you are missing.
As for your problem, AlphaFold is complaining that some of the flags that must be set are not being set, you must run the run_alphafold script with all of the 5 flags set.


overriding highlights

With context indent highlighted by treesitter
how can i override this highlights by treesitter

The highlight group for this is IndentBlanklineContextChar

ok thanks. will try it after my tuition


[CONTROLLER/RECORDER] publishes to tagrecorder when cleaning expired …

This PR is for:

Server

publishes to tagrecorder when cleaning expired resource data
Checklist

[ ] Added unit test.

Backport to branches

main

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.zhengya 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.


Add option to disable crafting recipe

This would be really good to add, especially if you can disable the recipe on certain worlds.


Fix time range selector timezones

This PR removes all uses of date-fns-tz (except from test code). In my opinion, date-fns-tz actually complicates things significantly. From a time range point of view, there are no timezones, there's a start and end timestamp. So, the timezone only becomes relevant for display.
Intl.DateTimeFormat [1] handles this well. In my opinion, date-fns-tz can be useful to handle string to timestamp if a timezone is involved, i.e., figuring out the timestamp of "17 Mar 11:05".
If I understood things right, there's actually three timezones involved, local time, utc, and zoned time. Having the browser (or the tests) run in Europe/Zurich looks correct, using GMT though shows the discrepancy.
[1] https://caniuse.com/?search=DateTimeFormat

Before, correct when browser is in CET (start of next interval is end of previous):

Before, incorrect when browser time is in GMT (one hour missing in DST crossing):

After, issue is fixed when browser time is in GMT (start of next interval is end of previous):

For completeness sake, correct behaviour also when browser is in CET:

Updated node from 12 to 14. v12 did not yet contain full ICU support by default, see:
https://github.com/nodejs/node/issues/19214

Thank you Mezzo!

Thanks for the review, @nicholas-os . @justmejulian , do you also wanna have a quick look?

Discussed with @sosostris , @justmejulian & Thomas, merging now.

@neonnoon I merged all the changes to local repo and somehow the tests were failing due to formatting (the values are all correct), like this:
Expected: "01.11.2020, 01:59 - 02.11.2020, 00:59"
Received: "11/01/2020, 01:59 AM - 11/02/2020, 12:59 AM"

But the tests are all passing here, so it's super weird. Checked a bit together with @Sakai-san and we found that the implementation of the method Intl.DateTimeFormat seems to vary with machine-os-language-set-up, e.g. there are similar issues reported here https://stackoverflow.com/questions/52797070/intl-datetimeformat-with-format-options-doesnt-work-for-it-ch
I will create a ticket to follow up with this. It would be nice to unify all the display to be one format :)

Ah.. I had the same issue.
See build result: See https://github.com/open-ch/pyrene/runs/2129628724
Fixed for Github like so: fa7937484d2861206f0b8b9cc77a1c8aa48f300f
Also mentioned further up in this thread: https://github.com/open-ch/pyrene/pull/6#issuecomment-800977148
You might have to check the bamboo build agents.


Postgres hasTable does not exist on withSchema

Environment
Knex version: 2.4
Database + version: Postgres (localhost) v14
OS: MacOs
Scenario: Migration
Bug

Explain what kind of behaviour you are getting and how you think it should do

I have various tables on different schemas within a single database.  I was using the knex.schema.createTableIfNotExists() method but saw a log warning that I should use hasTable and then createTable.
So I switched my code to knex.schema.hasTable('sys.sometable') and I noticed it was not matching the table correctly because it was checking INFORMATION_SCHEMA.tables and using current_schema.
So then I tried knex.withSchema('sys').hasTable('sometable') but its throwing an error.
knex.withSchema(...).hasTable is not a function
TypeError: knex.withSchema(...).hasTable is not a function

IMO all of the migration functions available on knex.schema should be available on knex.withSchema() and if schema is not pertinent to the function it can just ignore it.
OR hasTable should split the table name if a schema is provided to use in its query.

I thing you have to do:
knex.schema.withSchema('sys').hasTable('sometable')


Pull out SQL rendering into dialect-specific modules

SQL rendering will be dialect-specific. We could have generic rendering but that should be only for debugging / diagnostics.
Now that we have separate modules for Postgres, SQL Server, Oracle, and MySQL, we should create traits for each of these dialects, and implement SQL rendering functionality there.

This has been addressed by @robmwalsh in https://github.com/zio/zio-sql/pull/100


5.9: [InstructionDeleter] Don't delete-as dead instructions which produce owned move-only values.

Description: Many passes use the InstructionDeleter, requesting that it try to delete an instruction if its dead.  The deleter has a number of checks to determine what counts as "dead".
Among those checks is an early exit, affirming that an instruction which satisfies a few other conditions is indeed dead if it satisfies getSingleValueCopyOrCast.  At least one instruction (move_value) satisfying that predicate may produce an owned move-only value.  Such an instruction must not be deleted as dead.
Here, an additional condition is added prior to considering the getSingleValueCopyOrCast predicate; namely, whether the instruction produces an owned move-only value.  If it does, the deleter determines that the instruction is not dead and does not delete it.
Risk: Low.
Scope: Narrow.  The change only affects move-only values.
Original PR: https://github.com/apple/swift/pull/68108
Reviewed By: Andrew Trick ( @atrick )
Testing: Added a test.
Resolves: rdar://114351349

@swift-ci please test

Build failures are caused by lacking the overhaul to the SIL testing infrastructure on the release/5.9.  Will raise a further patch to adapt.

@swift-ci please test


nrf5_drv_uart build failed

info: ci/examples/ble_peripheral/ble_app_blinky, SDK 16.0.0
command:

cmake . -D NRF5_SDK_PATH=/tmp/nRF5_SDK -D NRF5_TARGET=nrf52832 -D NRF5_SOFTDEVICE_VARIANT=s132 && make

[ 66%] Building C object CMakeFiles/nrf5_drv_uart.dir/tmp/nRF5_SDK/integration/nrfx/legacy/nrf_drv_uart.c.o
In file included from /tmp/nRF5_SDK/integration/nrfx/legacy/nrf_drv_uart.c:41:
/tmp/nRF5_SDK/integration/nrfx/legacy/nrf_drv_uart.h:122:13: error: unknown type name 'nrf_uarte_baudrate_t'
     typedef nrf_uarte_baudrate_t        nrf_uart_baudrate_t;
             ^~~~~~~~~~~~~~~~~~~~
/tmp/nRF5_SDK/integration/nrfx/legacy/nrf_drv_uart.h:127:13: error: unknown type name 'nrf_uarte_error_mask_t'
     typedef nrf_uarte_error_mask_t      nrf_uart_error_mask_t;
             ^~~~~~~~~~~~~~~~~~~~~~
/tmp/nRF5_SDK/integration/nrfx/legacy/nrf_drv_uart.h:130:13: error: unknown type name 'nrf_uarte_hwfc_t'
     typedef nrf_uarte_hwfc_t            nrf_uart_hwfc_t;
             ^~~~~~~~~~~~~~~~
/tmp/nRF5_SDK/integration/nrfx/legacy/nrf_drv_uart.h:133:13: error: unknown type name 'nrf_uarte_parity_t'
     typedef nrf_uarte_parity_t          nrf_uart_parity_t;
             ^~~~~~~~~~~~~~~~~~
/tmp/nRF5_SDK/integration/nrfx/legacy/nrf_drv_uart.h:134:13: error: unknown type name 'nrf_uarte_task_t'
     typedef nrf_uarte_task_t            nrf_uart_task_t;
             ^~~~~~~~~~~~~~~~
/tmp/nRF5_SDK/integration/nrfx/legacy/nrf_drv_uart.h:135:13: error: unknown type name 'nrf_uarte_event_t'
     typedef nrf_uarte_event_t           nrf_uart_event_t;
             ^~~~~~~~~~~~~~~~~
/tmp/nRF5_SDK/integration/nrfx/legacy/nrf_drv_uart.c: In function 'nrf_drv_uart_init':
/tmp/nRF5_SDK/integration/nrfx/legacy/nrf_drv_uart.c:124:27: warning: variable 'config' set but not used [-Wunused-but-set-variable]
     nrf_drv_uart_config_t config = *p_config;
                           ^~~~~~
make[2]: *** [CMakeFiles/nrf5_drv_uart.dir/build.make:82: CMakeFiles/nrf5_drv_uart.dir/tmp/nRF5_SDK/integration/nrfx/legacy/nrf_drv_uart.c.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:1454: CMakeFiles/nrf5_drv_uart.dir/all] Error 2
make: *** [Makefile:103: all] Error 2

Hi! I'm afraid that calling CMake with just
cmake . -D NRF5_SDK_PATH=/tmp/nRF5_SDK -D NRF5_TARGET=nrf52832 -D NRF5_SOFTDEVICE_VARIANT=s132 && make

won't work. There are few problems with that:

Consider creating a CMake binary directory separate from the source directory.
You need to set the CMAKE_TOOLCHAIN_FILE and TOOLCHAIN_PREFIX variables in order to use a proper toolchain.
The official nRF5 examples are meant for boards, not targets per se. It would be better to pass NRF5_BOARD=pca10040 instead of NRF5_TARGET=nrf52832, the target will be deduced from the board. Thing is, nRF5 examples use the  Board Support Package library (nrf5_bsp) which relies on compile definitions related to the board.
The compile errors you got are most likely due to incorrect entries in the sdk_config.h. Since you did not specify the NRF5_SDKCONFIG_PATH, the generic SDK configuration file was used which won't work in this case. You need to use the sdk_config.h file meant for the ble_peripheral/ble_app_blinky example for the pca10040 board.
Finally, you need to use the right linker script for the example and the board, otherwise it won't work after being loaded into the board. You can point to a linker script by setting the NRF5_LINKER_SCRIPT variable.

Assuming you're running CMake from the ci/examples/ble_peripheral/ble_app_blinky directory, the command line would look more or less like this (you need to adjust the paths for your system, of course):
cmake -S . -B build -DCMAKE_TOOLCHAIN_FILE=../../../../cmake/arm-none-eabi.cmake -D TOOLCHAIN_PREFIX=D:/tools/gcc-arm-none-eabi-9-2019-q4-major-win32 -DNRF5_SDK_PATH=D:/tools/nRF5SDK160098a08e2 -DNRF5_BOARD=pca10040 -DNRF5_SOFTDEVICE_VARIANT=s132 -DNRF5_SDKCONFIG_PATH=D:/tools/nRF5SDK160098a08e2/examples/ble_peripheral/ble_app_blinky/pca10040/s132/config -DNRF5_LINKER_SCRIPT=D:/tools/nRF5SDK160098a08e2/examples/ble_peripheral/ble_app_blinky/pca10040/s132/armgcc/ble_app_blinky_gcc_nrf52.ld -G "Unix Makefiles"

Thank you.
Specifying NRF5_SDK_PATH did the trick and using NRF5_BOARD=pca10040
allowed to build till the end.
One thing I couldn't overcome: set-ing NRF5_SDK_PATH in CMakeLists.txt
doesn't work.

Glad to help.
Actually you can specify NRF5_SDK_PATH (and some other variables) in CMakeLists.txt. The important thing is that it must occur before including the nrf5 module, for example:
# ...

set(NRF5_SDK_PATH "D:/tools/nRF5SDK160098a08e2" CACHE PATH "" FORCE)

list(APPEND CMAKE_MODULE_PATH "../cmake")
include("nrf5")

# ...

If it works for you - that's OK but I wouldn't recommend doing that in production. By setting configuration variables in the command line your CMakeLists.txt can be more universal so you can keep it unchanged for building for multiple targets, SDK versions etc.
BTW. You can build examples more easily if you setup the CI locally and run some scripts. Go the ci/scripts directory and run prepare.sh. It's going to download and install some dependencies, check it out and make sure it's OK for you. After that, you can just type:
./build_example.sh --example=ble_peripheral/ble_app_blinky --sdk_version=16.0.0 --board=pca10040 --sd_variant=s132

That will configure and build the example.


[hw,csr_utils,dv] Wait for register field to be free before write access

Description
In the current implementation of csr_wr, there is no option to wait for register/field to be free before calling write method
Update csr_wr with an option to explicitly wait for register to be free.
https://github.com/lowRISC/opentitan/blob/c3d3e284a6346c373c55da82120586eb401d9060/hw/dv/sv/csr_utils/csr_utils_pkg.sv#L145-L181

For reference, here is an example of how this can be implemented with code before the call to csr_wr:
https://github.com/lowRISC/opentitan/blob/a32c4052d455841c530d66cc091c5fac844ee821/hw/ip/spi_device/dv/env/seq_lib/spi_device_pass_base_vseq.sv?plain=1#L633-L637
and I agree it would be nice if csr_wr had an argument to enable this waiting directly inside the function

CC @matutem @rswarbrick @jdonjdon

This would be nice to have but it's rather a cleanup thing. What the issue asks for is already possible today by writing some code around the csr_wr function. It's not realistic that we will have time for this. -> Backlog / FutureReleases.


Something went wrong! V55

I'm using version 55 and this error occurs crashing the plugin. Any solution?
Minified React error #31; visit https://reactjs.org/docs/error-decoder.html?invariant=31&args[]=object with keys {value} for the full message or use the non-minified dev environment for full errors and additional helpful warnings.
Restart the plugin and try again.

Could you provide me with a recording of when that error occurs? Is it after a specific action?
Also, if you can, a link to the Figma file could help greatly, you can send the invite to<EMAIL_ADDRESS>

I just open the extension in the file and scroll the plugin.
Unfortunately I can't share the file.

That is odd, I'm getting the feeling this might have to do with tooltips. Do any of the tokens give a tooltip for you when hovering over them? Or does it crash as soon as you move your mouse over them?

I just scroll, nothing more than that, I just need to go from the beginning to the end of the height of the container.


Weird clipping issue

Depending on which window I'm focusing on, this clipping happens. I see it more often while focused and scrolling through my terminal.
https://user-images.githubusercontent.com/842850/143053053-316b16d2-ac81-4b90-a167-98582a0586fa.mov

What WM are you using?

i3wm 4.20.1
Kernel 5.14.18-1-MANJARO
Wired 0.9.4-1

I can't seem to reproduce this on i3wm 4.20.1 or in bspwm:
https://user-images.githubusercontent.com/6947819/143327003-c6baa9d9-986f-4bb2-b68e-657be84ff62f.mov
I'm using basically stock i3wm and picom with the following (maybe) relevant settings:
mark-wmwin-focused = true;
mark-ovredir-focused = false;
detect-rounded-corners = false;
unredir-if-possible = true;
refresh-rate = 0;
vsync = false;

Perhaps send over your wired config and I'll try that directly?  Also I see you have gaps -- are you using i3-gaps or is there a way to do gaps with vanilla i3?

Here are the packages that might be useful to reproducing it:
i3-gaps | 4.20.1-1 | config
Wired | 0.9.4-1 | config
Picom | 8.2-1 | config
Kitty | 0.23.1-1 | config
Polybar | 3.5.5 | config
Kernel | Kernel 5.14.18-1-MANJARO
I've dug a tiny bit more and found out that I can reproduce the clipping by trigerring the notification from polybar's volume module but when running the script mannually it works flawlessly. So maybe there's some weird behavior with how Polybar & Wired a playing together.
https://user-images.githubusercontent.com/842850/143440731-3d904f1b-9759-4c05-8339-63256bf127c2.mov

[ ]

Thanks for all the info!  So weird!
Will keep looking into this soon.

Ah man.  I've basically duplicated your setup and I still cannot reproduce this issue.
Same Wired/Picom/Polybar config, using Kitty, Kernel 5.15.4-arch1-1:
https://user-images.githubusercontent.com/6947819/143726593-199a5305-55e8-4344-8838-e1d92e9d9c3e.mp4
For reference, I'm using nvidia 495.44-12.
My best guess at this point is that there's a weird graphics interaction or something with X11 is misbehaving.  I don't think this is an issue in Wired, but it could be in the windowing library we're using.  I'm still interested in trying to solve it, but it's difficult to move forward.
The works-when-script-run-separately-but-not-with-polybar especially makes me think something super weird is going on.  I had a brief look into polybar's script run stuff but couldn't see anything that would make this happen.
Also, consider the replacing_resets_timeout option which might make those volume controls a little more pleasant :).

Given I can't provide a reproducible setup, lets close this one.


Add basic vssapi support

Not all of the VSS api is finished, but at its current state, it is usable.
This lets you link statically against the microsoft vssapi.lib import library.  But this library is a bit different.  All symbols in the import library are c++ mangled names and need to be specified with the link_name attribute.  On x86_64, I would always hit rust issue 23216 and it would crash.  On x86, I had to discover the trick of starting the link_name attribute with \x01, or else rust would further mangle the specified link_name attribute and it would fail to link.
One struct used a field named "type" that I had to rename "type_".
I hit one c union that will probably need to be cleaned up later.
I ran into a lot of c++ enum types.  Initially I modeled them with rust enums, but now I don't think rust enums are a good fit.  They're more cumbersome to use because they have their own namespace, and they don't let you apply the | operator to them, and about half of them are flags that need the | operator.  C++ doesn't guarantee a size of a normal enum, but from what I can tell, they're always i32 in Microsoft's ABI, right?  I settled to using INT or DWORD depending on if they were numbers or flags.  This needs more thought.  Maybe they should all be the same type, but sometimes they're negative and sometimes they're >= 2^31.

For flags using DWORD is fine, but for enums that aren't flags, please use rust enums. You can do pub use SOME_ENUM::*; to replicate C enum contents not being namespaced. Enums should use #[repr(i32)]. Values being outside of the valid range of a type can be handled with a simple cast Blah: DWORD = -1i32 as u32;.
Also, please try to match the existing style by not vertically aligning things.

Thanks for reviewing it.  I fixed everything.
The Travis CI build failure isn't my fault.  It's a missing close parenthesis on lib/secur32-sys/src/lib.rs:114

Ha, whoops, that build failure was my fault. :stuck_out_tongue:


Uploading large files (>10M) through nginx proxy?

Error description:
Error

When uploading a file I get the above message
Steps to reproduce
Upload a >2M file to a site behind it
Expected result
An OK
Error result
Failed upload
Comments
Hi Evert
I'm trying to get it to allow uploading some really large files through this.
I have a website in DMZ that is forwarded through an nginx behind your setup.
So your nginx forwards to my nginx that forwards to an inside site.
The docker-compose file I've made for this:
# web forward for site
  site-url:
    image: nginx
    container_name: site-url
    restart: always
    volumes:
      - "/docker/site:/etc/nginx/conf.d"
#    depends_on:
#      - name_of_all_your_other_services
    environment:
      VIRTUAL_HOST: siteurl
      LETSENCRYPT_HOST: siteurl
      LETSENCRYPT_EMAIL: email

In 'my own' nginx conf.d I've given the line:
client_max_body_size 100M;
To solve this, but how do I get this added to the hosts that is automatically generated by the environment settings?

Doh, got it working, never mind :-D

@fribse What did you do to get it working?

In the new version, I don't find inside the .env files the variable:
#USE_NGINX_CONF_FILES=true
So how do I increment the "client_max_body_size"?


publish ignores ssh key; wants to use password

I'm trying to publish a subproject as a library to my own repository using an SSH key.  SBT asks for a password (which I don't have) instead of using the key.
I'm using SBT 1.9.0.  Also tested wth 1.6.2 and get the same results.  This is on a Mac (M1 processor) running Ventura 13.4.  SBT was installed using Homebrew.
steps
The build for the subproject:
lazy val solver = (project in file("_solver"))
  .settings(commonSettings)
  .settings(
    name := "solver",
    libraryDependencies ++= Seq(
      "ca.uwaterloo" %% "pprint"    % "0.1.5-SNAPSHOT",
      "com.lihaoyi"  %% "fastparse" % "3.0.1",
      "com.lihaoyi"  %% "fansi"     % "0.4.0",
      "com.lihaoyi"  %% "scalatags" % "0.12.0"
    ),
    publishTo := Some(
      Resolver
        .ssh("Degree Audit", "ubuntu2204-102.cs.uwaterloo.ca", "/u1/cs-oat/public_html/maven")
        .as("cs-oat", file(sys.env("HOME") + "/.ssh/oat_rsa"))
        .withPublishPermissions("0644")
    )
  )

commonSettings has name, version, etc.  Nothing related to publishing.
The results of attempting to publish (I manually "pretty printed" the results of show publishTo to make reading easier):
bwbecker@beta-2 oat-degreeAudit3 % sbt
[info] welcome to sbt 1.9.0 (Homebrew Java 11.0.19)
...
sbt:cli> project solver
[info] set current project to solver (in build file:/Users/bwbecker/oat/src/oat-degreeAudit3/)
sbt:solver> show publishTo
[info] Some(SshRepository(
    Degree Audit,
    Patterns(ivyPatterns=Vector(),
                   artifactPatterns=Vector(/u1/cs-oat/public_html/maven/[organisation]/[module](_[scalaVersion])(_[sbtVersion])/[revision]/[artifact]-[revision](-[classifier]).[ext]),
    isMavenCompatible=true,
    descriptorOptional=false,
    skipConsistencyCheck=false),
    SshConnection(Some(KeyFileAuthentication(cs-oat, /Users/bwbecker/.ssh/oat_rsa, None)),
                               Some(ubuntu2204-102.cs.uwaterloo.ca),
                               None),
    Some(0644)))
[success] Total time: 0 s, completed Jun. 10, 2023, 8:29:55 a.m.
sbt:solver> publish
[info] Wrote /Users/bwbecker/oat/src/oat-degreeAudit3/_solver/target/scala-2.13/solver_2.13-0.3.0-SNAPSHOT.pom
[error] stack trace is suppressed; run last publish for the full output
[error] (publish) java.io.IOException: SSH_MSG_DISCONNECT: 2 Too many authentication failures
[error] Total time: 28 s, completed Jun. 10, 2023, 8:30:29 a.m.
sbt:solver>

The above showed this dialog box:

Clicking "OK" with a blank password resulted in the java.io.IOException.  The first bit of last publish produces:
sbt:solver> last publish
[debug] :: SSH :: connecting to ubuntu2204-102.cs.uwaterloo.ca...
[error] java.io.IOException: SSH_MSG_DISCONNECT: 2 Too many authentication failures
[error] 	at org.apache.ivy.plugins.repository.ssh.SshCache.getSession(SshCache.java:338)
[error] 	at org.apache.ivy.plugins.repository.ssh.AbstractSshBasedRepository.getSession(AbstractSshBasedRepository.java:108)
[error] 	at org.apache.ivy.plugins.repository.ssh.SshRepository.put(SshRepository.java:238)
[error] 	at org.apache.ivy.plugins.repository.AbstractRepository.put(AbstractRepository.java:130)
[error] 	at org.apache.ivy.plugins.resolver.RepositoryResolver.put(RepositoryResolver.java:234)
[error] 	at org.apache.ivy.plugins.resolver.RepositoryResolver.publish(RepositoryResolver.java:216)
[error] 	at sbt.internal.librarymanagement.IvyActions$.$anonfun$publish$5(IvyActions.scala:504)
[error] 	at scala.runtime.java8.JFunction0$mcV$sp.apply(JFunction0$mcV$sp.java:23)
[error] 	at scala.util.Try$.apply(Try.scala:213)
[error] 	at sbt.internal.librarymanagement.IvyUtil$.retryWithBackoff(IvyUtil.scala:22)
[error] 	at sbt.internal.librarymanagement.IvyActions$.$anonfun$publish$4(IvyActions.scala:503)
[error] 	at sbt.internal.librarymanagement.IvyActions$.$anonfun$publish$4$adapted(IvyActions.scala:501)
[error] 	at scala.collection.Iterator.foreach(Iterator.scala:943)

expectation
I expect SBT to publish the artifact using my SSH key rather than asking for a password.
notes

SSHing manually works:

bwbecker@beta-2 oat-degreeAudit3 % ssh<EMAIL_ADDRESS>-i ~/.ssh/oat_rsa
Last login: Sat Jun 10 07:25:59 2023 from <IP_ADDRESS>
cs-oat@ubuntu2204-102:~$

Another library that is set up similarly was successfully published on May 2, 2022.  It now shows the same issue.  I don't know what changed.
If I remove the keyfile from the .ssh directory, I get the same results.  So SBT must not even be trying the SSH key since it surely doesn't know where I put it.
I have multiple SSH keys in ~/.ssh.  But I don't think that's a factor because if I rename the .ssh directory, SBT's behaviour is exactly the same.
The ssh key does not use a passphrase.

Bug is probably upstream around there
https://github.com/sbt/librarymanagement/blob/55893d357bcc5a0cdb617ba1cf24491023839cf6/ivy/src/main/scala/sbt/internal/librarymanagement/ConvertResolver.scala#L356-L373

Cannot reproduce it (Windows 10, sbt 1.10.4, Java 21)
sbt:sbt-7293> publish
[info] Wrote D:\Repos\sbt-7293\target\scala-3.3.4\sbt-7293_3-0.1.0-SNAPSHOT.pom
[warn] versionScheme setting is empty; set `ThisBuild / versionScheme := Some("early-semver")`, `Some("semver-spec")` or `Some("pvp")`
[warn] so tooling can use it for eviction errors etc - https://www.scala-sbt.org/1.x/docs/Publishing.html
[info]  published sbt-7293_3 to maven/sbt-7293/sbt-7293_3/0.1.0-SNAPSHOT/sbt-7293_3-0.1.0-SNAPSHOT.pom
[info]  published sbt-7293_3 to maven/sbt-7293/sbt-7293_3/0.1.0-SNAPSHOT/sbt-7293_3-0.1.0-SNAPSHOT.jar
[info]  published sbt-7293_3 to maven/sbt-7293/sbt-7293_3/0.1.0-SNAPSHOT/sbt-7293_3-0.1.0-SNAPSHOT-sources.jar
[info]  published sbt-7293_3 to maven/sbt-7293/sbt-7293_3/0.1.0-SNAPSHOT/sbt-7293_3-0.1.0-SNAPSHOT-javadoc.jar
[success] Total time: 49 s, completed Nov 1, 2024, 2:20:33ΓÇ»p.m.

build.sbt
ThisBuild / version := "0.1.0-SNAPSHOT"

ThisBuild / scalaVersion := "3.3.4"

lazy val root = (project in file("."))
  .settings(
    name := "sbt-7293",
    publishTo := Some(
      Resolver
        .ssh("Degree Audit", "localhost", "maven")
        .as("Jerry Tan", file(sys.env("HOME") + "/.ssh/id_rsa"))
        .withPublishPermissions("0644")
    )
  )

Shall try later with a Mac.


Add auto-generated scenes by Unity play mode tests

Reasons for making this change:

Unity Test Framework is official testing framework for Unity.
Unity Test Framework has two testing mode, Edit mode and Play mode.
Play mode tests is run in the Scene.
Unity Test Framework are auto-generate one scene file when run play mode tests.
This scene file is delete when all tests finished, normally.
But, scene file may not be deleted when test is abnormally terminated.
e.g.
-rw-r--r--    1 ko2hase  staff  14582  2  5 23:37 InitTestScene638112370725555120.unity
-rw-r--r--    1 ko2hase  staff    155  2  5 23:37 InitTestScene638112370725555120.unity.meta

Links to documentation supporting these rule changes:
No documents for auto-generated scene file.

See this issue comment. (still not fixed)
https://issuetracker.unity3d.com/issues/testrunner-playmode-test-leaves-behind-inittestscene-dot-dot-dot-scene-assets-after-test-runs


Missing LICENSE

There is no license for this project. The default is copyright.
You should license this project under the AGPL3 or GPL3 license. If you would like me to submit a PR to make this easier, I would be happy to do that.

Thanks for reminding, I added an MIT License.


ngAfterContentInit in ListView template component

Hi!
I found a strange behavior with ListView I can't understand it...
If I have created a very basic list:
  template: `
    <ListView [items]="rows" (itemTap)="onItemTap($event)">
      <ng-template let-item="item" let-i="index">
        <list-with-picture-template-cmp [document]="item"></list-with-picture-template-cmp>
      </ng-template>
    </ListView>
  `,

With a simple component as template:
import { Component, Input, AfterContentInit } from '@angular/core';

@Component({
  selector: "list-with-picture-template-cmp",

  template: `
  <GridLayout style="background-color: white" columns="auto, *">
      <Label text="{{ document.designation }}"></Label>
  </GridLayout>
  `
})
export class ListWithPictureTemplateComponent implements AfterContentInit {

  @Input() document: any;

  ngAfterContentInit() {
    console.log('ListWithPictureTemplateComponent - ngAfterContentInit');
  }
}

When I log ngAfterContentInit, I see that the function is called only for the initialy displayed elements. When I move the mouse to reach other elements, the ngAfterContentInit is called on more time for the first next element but that’s it, no more call no matter how long is the list. It’s a problem because I need to execute some code for each element in the list.
I think it's bug or I don't understand this behavior...

This is due to the view recycling done by the native views. You can use ngOnChanges callback if you want to get notified about views data binding has been changed. (Change detection is triggered by the list view component on the views created from the template(s)).

Thnak you very much, works for me !


Investigate VictoriaMetrics exporter

VictoriaMetrics is designed as stand-in for Prometheus. I already use it to scrape Harvest 2.0 targets. I found no downside so far.

Thanks for the feedback @mamoep


Notification after mfa

Send Notification if WDE receives the service updates with timeout that is less than 5 minutes.
Send notifications after mfa re-auth

i'm not sure this is really needed, the service imo probably does not need to track this sort of state in the long run. for now it's probably fine

I thought it is better for WDE to know when do the services become unavailable. That would help in troubleshooting issues. Jeremy, Could you please check and let us know if you can handle the notifications from UI completely when you have some time. For now, we will go ahead and merge this logic.


Insert into button button doesn't work with Videos

Bug Description
When 'Insert into page' is clicked, it doesn't inserts the video into the page. See Video.
I get the following console error.
getResourceFromMediaPicker.js:52 Uncaught TypeError: Cannot convert undefined or null to object
    at Function.entries (<anonymous>)
    at getResourceFromMediaPicker (getResourceFromMediaPicker.js:52)
    at onSelect (mediaPane.js:128)
    at h.Collection.<anonymous> (useMediaPicker.js:101)
    at p (backbone.min.js?ver=1.4.0:2)
    at f (backbone.min.js?ver=1.4.0:2)
    at l (backbone.min.js?ver=1.4.0:2)
    at h.Collection.n.trigger (backbone.min.js?ver=1.4.0:2)
    at h.Collection._onModelEvent (backbone.min.js?ver=1.4.0:2)
    at p (backbone.min.js?ver=1.4.0:2)

Expected Behaviour
When 'Insert into page' is clicked, it inserts the asset into the page.
Steps to Reproduce

Open media upload dialog.
Upload  video file ( mp4 file).
Select video.
Click insert into page.

Screenshots
https://youtu.be/F0IUuYIFwlA
Additional Context

Plugin Version: 1.0.0-beta2
Operating System: Mac OS
Browser: Chrome 85

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

QA'ed and works as expected now.


[release-1.4] fix(virtctl): Ensure the log verbosity flag is supported

What this PR does
Manual backport of https://github.com/kubevirt/kubevirt/pull/13555
Before this PR:
virtctl does not support the -v / --v log verbosity flag.
After this PR:
virtctl supports setting the log verbosity with the -v / --v flag.
Fixes https://github.com/kubevirt/kubevirt/issues/12779

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

None

/cherry-pick release-1.3

/hold Wait for https://github.com/kubevirt/kubevirt/pull/13555 to be merged.

/approve

/lgtm

/hold cancel
/retest-required

/test pull-kubevirt-fuzz-1.4


⚠ Bofrost.nl has degraded performance

In f15e8f8, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1232 ms

Resolved: Bofrost.nl performance has improved in 8b472b0.


Use scoped styles for .notification

When building a Vue CLI 3 app for production, it seems that the .notification style in the Notifications component takes priority, thus overriding the styles (specifically the default background colour) of Bulma notifications. This does not happen when serving a development server, only in production, and naturally, the styles are bundled with the production build when another class name is used.
The only thing I can suggest is making use of scoped styles here, or perhaps a more unique name is in order.
Thanks for this package - really works well 👍

Please can this be looked into?

I would really love to see that change too! @mikerockett have you tried adding scoped here https://github.com/euvl/vue-notification/blob/master/src/Notifications.vue#L323 directly in your node_modules folder followed by a npm i && npm build (inside the packcage dir, you could run tests too) to rebuild the package before building your app again? A pull request for this change would be trivial then.

I've since forked this package and made significant changes to it, including moving the stylesheet out into its own SASS file with default variables.


Edge case for index >10

While discussing deserialization from JSON (see https://github.com/FasterXML/jackson-databind/issues/2020#issuecomment-387257273)
Decade is the longest tuple, and it has no add() method.
I think it maybe better to have some way to produce Collection<Object> (unlimited tuple), than throw Exception when trying to deserialize JSON array with 10+ elements.
ref https://github.com/FasterXML/jackson-future-ideas/issues/25

or make Tuple non abstract class that would hold tuple/array of any size


Link to brackets if pod is full

I think it would help if the modal when you try to join a full pod (https://github.com/Senryoku/MTGADraft/blob/master/server.js#L820) also had a link to the read-only bracket for that pod. People don't link to read-only brackets that often, but usually the original link to a pod remains available (on discord, twitch chat, etc.) to people beyond the first 8 players. If we kicked them over to a spectator-type experience, or just gave them an option for that before starting a new pod, I think it would help for people to follow along with a pod that they're not playing in. This would go especially well with #198 , but even just being able to see decks people build for KHM right now, and being able to see live match results in the team draft showdown this week (if they bother to put them in the bracket this time, grrr) would be really valuable!

Yep, this will be important to remember if we do add a spectator mode.
I've added a link to bracket with a10fd5ade69eefb65290c89a6fdc2604566dd312 so I'm closing this issue and will add a note in the spectator mode discussion.

Yep, this will be important to remember if we do add a spectator mode.
I've added a link to bracket with a10fd5ade69eefb65290c89a6fdc2604566dd312 so I'm closing this issue and will add a note in the spectator mode discussion.


Proposal (my thoughts): Externalize tree shaking from the core to separate package/lib/tool

What about tree-shaker or tree-shaking? Why not frog - free, short, animal, cool?

Hi there. Just playing around more deeply last few days with Rollup and Webpack.
Rollup is extremely focused, small, fast and awesome. And his idea for tree shaking is unique, moved tooling years forward, saves million bytes. That's great, but it seems that it has much problems with the tree shaking, just for few hours found couple of cases that seems easy.
So I ended up with thought that Webpack has good support for tree shaking, but is awful when it comes to "getting started", "up and running", "setup/config". Okey, it easy to "get started", but these configs reminds me to Grunt times or something worst (i'm step into the community around when gulp pops up, or a bit later after it). Never touched gruntfile and never worked with grunt, and even can't imagine to work with - it is total mess even in to moment you just look its config. Nevermind. That's not some hate, rant or etc. That's my 2c about "configuration style".

In short: We need just tree-shaking package, which deals with only es6 modules. Just gets entry then returns a bundle. Period. Nothing more, nothing less. Actually babylon is great, so I think it should be used in that tree-shaking package as a base. So babel-based? This allow more and more advanced things to be built on top of it?
In short 2: If we have such package, both rollup and webpack would be more stable, built with one core with joined forces (+ babel ecosystem and its users). Rollup will still be the same thing, Webpack too. If someone just don't want to use webpack (in some cases and small projects it not worth), or rollup (because some reason) - actually it has 180 issues currently, or it just want to built another bundle tool/service, will just use that package.

Another thing is that both Rollup and Webpack can't load config from package.json? Or I'm missing something? hm

Webpack "activates" its tree shaking only when "optimize minimize" thing is enabled, meaning you can't get tree shaked bundle without it to be minified?

Webpack must use babel-loader to get tree-shaked bundle (because the minify-ing stuff) because uglifyjs. (yea we have some harmony branch out there but..) And babili is too young, but i believe we should use it as base.

In short 3: We just don't need another features. We want stable tree shaking and optimized bundles, no painful config and boilerplate. We don't want more and more config files into repo root. I, specifically, hate to mix tools inside another tools (i mean with "plugins" or no matter how you'll call it) - most of them are totally focused and does its job great. We as developers should think in this way.
If we build just a tool that deals with that job, then we can just pass that bundle to babel. One scenario can be: you are user, you write packages that are mostly ES5 but you use es6 modules because the minify-ing and tree shaking stuff. You pass your code to that tool, gets final cool es5 bundle without import/exports. Second scenario: you are writing ES6 with more features than just es6modules, pass your code to such tool that does not deal with another ES6 features but only with the ES6 modules, gets a bundle, pass that bundle to babel and you get final es5 code again. Minifying is totally another story and we should not deal with it in these tools - we can use babili/babel ecosystem, i believe it will explode in some point in future.

I'm playing with babylon to get import and exports, and thinking for object structure that will contain the things.

Another thing is that both Rollup and Webpack can't load config from package.json? Or I'm missing something? hm

It's not feasible to build an automatic mapping between JSON and webpack's configuration format as far as I'm aware of. You'll run into issues with plugins in particular.
I don't know how feasible it would be to extract the tree shaking related functionality to package of its own. It's quite possible it's somewhat tied to webpack's internal implementation making it difficult.
That said, I agree pushing features to packages is a good way to go as that enables collaboration across projects, but it can also limit (stable API needed, more communication required). Sometimes those limitations can be good, though, and foster quality.

:+1:

It's not feasible to build an automatic mapping between JSON and webpack's configuration format as far as I'm aware of. You'll run into issues with plugins in particular.

I don't get what problems you'll have when you just want to use babel and some preset. For example such things are totally JSON valid
module.exports = {
  entry: './index.js',
  output: {
    path: __dirname,
    filename: 'bundle.min.js'
  },
  module: {
    loaders: [{
      test: /\.js$/,
      exclude: /(node_modules|bower_components)/,
      loader: 'babel-loader'
    }]
  }
}

of course, regex/patterns can be nanomatch or micromatch.
AVA, ESLint, Standard, Verb, PostCSS are good examples too. They can accept pretty complex config through package.json. And really, worth nothing, just some cool config loader package (i'm not talking for webpack package), like my idea for load-cfg (tunnckoCore/ideas)

@tunnckoCore A plugin definition could look like this:
plugins: [
  new webpack.HotModuleReplacementPlugin({
    multiPass: true
  })
]

I've tried the JSON approach myself and I got it working, but it got very frustrating with plugins for this reason. Especially third party ones are painful.
That said, if you know how to solve it, please show me how to do it.

@bebraw @tunnckoCore
package.json
"webpack": {
  "plugins": {
     "html-webpack-plugin":  {..options}
  }
}

plugins.js
const load = (Plugin, options) => {
  if (Object.keys(options).length === 0) {
    try {
      Plugin = require(Plugin)

      return new Plugin()
    } catch (err) {
      console.log(err)
    }
  }
  try {
    Plugin = require(plugin)

    return new Plugin(options)
   } catch (err) {
     console.log(err)
   }
 }
}

if (typeof webpack.plugins === 'object') {
    Object.keys(webpack.plugins).forEach((Plugin) => {
        Plugin = load(Plugin, wepack.plugins[Plugin])

        config.plugins.push(Plugin)
    })
}

But please don't 😛

It's absolutely understandable that one JSON config won't support fully what user can do through js file config, but it not mean that the tool should not support JSON config and that it should not try to load config from couple of places and finally merge them in one.
edit: I'm not talking for plugins especially. It's totally normal to not be able to accomplish what we can with js file. I'm talking for most of other things.

@tunnckoCore Plugins are the bread and butter of webpack, even internally most of the logic is implemented as a plugin (tapable instance), being unable to use plugins would not just limiting 'advanced' usecases to some point, you would lose the ability to configure and use Core Plugins (UglifyJS, CommonChunks etc..) aswell.

@michael-ciniawsky omg :laughing: core will still work with plugins, config has nothing to mess with that. Worth nothing to support package.json as config for simple things that in some cases are totally enough and the user just don't adds dozens of config and dot files.

@tunnckoCore This issue is a good conversation, but I do not feel that it is an issue. Maybe a feature request is a good label for this thread.

Valid.


Add flat style?

Not sure why but the typescript badge looks different from the other badges I usually add.
I guess it could be that it is not "flat" style, perhaps that could be added?
See example below:

The image on the readme in this repo seems to be flat but then it is bigger than the others. See example below:


Revert "sgurary: added labs/sawtooth-secure-transaction-families.md for pull …"

Reverts hyperledger-labs/hyperledger-labs.github.io#49

See #51


Missing description of partnerOrganization

There are no annotation properties for OP nfdicore:partnerOrganization.
In nfdico it used to be: The property is a partner relationship between organizations. The property is symmetric.

Thanks for the suggestion!


Update Analyzer.ruleset.json

Summary
This does nothing
Work Item(s)
closees #7

Could not find linked issues in the pull request description. Please make sure the pull request description contains a line that contains 'Fixes #' followed by the issue number being fixed. Use that pattern for every issue you want to link.

Could not find linked issues in the pull request description. Please make sure the pull request description contains a line that contains 'Fixes #' followed by the issue number being fixed. Use that pattern for every issue you want to link.

Could not find linked issues in the pull request description. Please make sure the pull request description contains a line that contains 'Fixes #' followed by the issue number being fixed. Use that pattern for every issue you want to link.

Issue #7 is not valid. Please make sure you link an issue that exists, is open and is approved.


fixed some translations error for zh docs #31888

This was a Bug Report. The problem was:

content/zh/docs/concepts/cluster-administration/system-metrics.md
line 131 , '启用' should be '弃用'

content/zh/docs/reference/access-authn-authz/_index.md
Title '访问 API' should be ‘API 访问控制'
I have corrected the documentation error. Please review

/hold
This PR is dup with #31889 .

Closing as a dup. Thanks for the contribution anyway!
/close


Remove Followers | Unfollow Users Who doesn’t have a profile picture

Form

[x] I've searched this similar issue
[x] I've read the basic concepts
[x] I'm using the latest version

Question
Is there a way to remove followers of unfollow users who doesn’t have a profile picture?

possible.
Check user image, if they have default picture or not.
Check my app built with api
https://github.com/kingbotss/igportable


even after traing with large data.model is prdeicting the unintended part

I ran the code as it is : It generates me the plotted output : but The thing is The output comes as an empty one . on all 3 instances of Test Image .
I have attached my screenshot of the result.png file . I haven't made even a single modification in the code . What might be wrong ?
result

me to have the same issue, i have trained the model with  200 images .still it is predicting the unintended object in the image .Any clue to sort out this issue is highly appreciable


How to integrate guide

After integrating with Cirrus CI Agent it worth to document a guide on how to integrate the terminal within your system.

Perhaps we should research if it's possible to create a JS module that ships the minimalistic web UI, otherwise it only seems to be useful in the Cirrus Cloud. One potential option to achieve this is to create an Xterm.js addon.


Clear loadend event after it has been handeled

Task & test plan (description of the problem)

The change looks very specific and from the discussion I believe this is the right fix. But I couldn't test it because of the errors before all tests. Maybe @venetak can test tomorrow and verify it. Probably 150ms / 100% should be enough (see clumsy screenshot) + remember to press the Start button.

The errors are because either the URL search params are not bundled into the form control that now Veni is fixing, or the form control is not linked correctly but I assume it is the first problem and hope the fix from Veni will fix it.


Feature/fix

Codecov Report

Merging #120 into develop will decrease coverage by 0.84%.
The diff coverage is 40%.

@@             Coverage Diff             @@
##           develop     #120      +/-   ##
===========================================
- Coverage    62.52%   61.68%   -0.85%
===========================================
  Files          204      204
  Lines        10845    10849       +4
===========================================
- Hits          6781     6692      -89
- Misses        4064     4157      +93

Impacted Files
Coverage Δ

mediaPlayer/SuperMediaPlayer.cpp
67.7% <0%> (+0.72%)
:arrow_up:

framework/demuxer/play_list/HlsParser.cpp
77.85% <80%> (ø)
:arrow_up:

framework/tests/dataSource/ipList.h
0% <0%> (-100%)
:arrow_down:

framework/tests/decoder/decoderUnitTest.cpp
0% <0%> (-93.48%)
:arrow_down:

framework/tests/dataSource/dataSourceUnitTest.cpp
0% <0%> (-93.34%)
:arrow_down:

framework/data_source/ffmpeg_data_source.cpp
9.37% <0%> (-65.63%)
:arrow_down:

framework/data_source/curl/CURLShareInstance.cpp
69.44% <0%> (-27.78%)
:arrow_down:

framework/data_source/ffmpeg_data_source.h
80% <0%> (-10%)
:arrow_down:

framework/data_source/curl/CURLConnection.cpp
56.84% <0%> (-8.64%)
:arrow_down:

mediaPlayer/media_packet_queue.cpp
50.4% <0%> (-4%)
:arrow_down:

... and 16 more

Continue to review full report at Codecov.

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


Invalid memory address

After running serve2d for weeks, it crashed :) I don't know, maybe something in my setup broke it or it just has a bug :beetle:. When this crash occured, I was restarting my webserver very often (serve2d is used as a proxy in front of nginx).
Panic:
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xb code=0x1 addr=0x0 pc=0x514c78]
goroutine 30051 [running]:
github.com/joushou/serve2.(*Server).handle(0xc208056140, 0x7fe1ca1b3f58, 0xc20800ca80, 0x7fe1ca1b3f88, 0xc20802c060, 0x83e568, 0x0, 0x0, 0xc20800a600, 0x20, ...)
/home/janne/servepack/src/root/src/github.com/joushou/serve2/serve.go:116 +0x418
github.com/joushou/serve2.(*Server).HandleConn(0xc208056140, 0x7fe1ca1b3f88, 0xc20802c060, 0x83e568, 0x0, 0x0, 0x0, 0x0)
/home/janne/servepack/src/root/src/github.com/joushou/serve2/serve.go:213 +0x685
github.com/joushou/serve2.func·001()
/home/janne/servepack/src/root/src/github.com/joushou/serve2/serve.go:236 +0x53
created by github.com/joushou/serve2.(*Server).Serve
/home/janne/servepack/src/root/src/github.com/joushou/serve2/serve.go:237 +0x162
goroutine 1 [IO wait]:
net.(*pollDesc).Wait(0xc208010290, 0x72, 0x0, 0x0)
/usr/lib/go/src/net/fd_poll_runtime.go:84 +0x47
net.(*pollDesc).WaitRead(0xc208010290, 0x0, 0x0)
/usr/lib/go/src/net/fd_poll_runtime.go:89 +0x43
net.(*netFD).accept(0xc208010230, 0x0, 0x7fe1ca1b2b70, 0xc20802ab90)
/usr/lib/go/src/net/fd_unix.go:419 +0x40b
net.(*TCPListener).AcceptTCP(0xc20802c020, 0x7fe1ca1a1000, 0x0, 0x0)
/usr/lib/go/src/net/tcpsock_posix.go:234 +0x4e
net.(*TCPListener).Accept(0xc20802c020, 0x0, 0x0, 0x0, 0x0)
/usr/lib/go/src/net/tcpsock_posix.go:244 +0x4c
github.com/joushou/serve2.(*Server).Serve(0xc208056140, 0x7fe1ca1b2ea8, 0xc20802c020)
/home/janne/servepack/src/root/src/github.com/joushou/serve2/serve.go:230 +0x76
main.main()
/home/janne/servepack/src/root/src/github.com/joushou/serve2d/serve2d.go:324 +0xa1e
goroutine 17 [syscall, 9932 minutes, locked to thread]:
runtime.goexit()
/usr/lib/go/src/runtime/asm_amd64.s:2232 +0x1

Ah! Definitely a bug in serve2. The lib (serve2) can call a logging callback if one is interested in such information. Unfortunately, I had missed a check for whether a logger was defined in a single spot, which was where it would print if a handler failed to handle the connection. That's what you hit, probably due to the constant server restart.
I have fixed it in the library, and will make a new release of serve2d soon.

Fixed in release v0.3.1


From<failure::Context<&str>> is not implemented for std::io::Error

Hello!
This is issue is likely due to my noobness, but that's the error I get:
let mut file = File::open(path).context("Missing Cargo.toml")?;
    |          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ the trait `std::convert::From<failure::Context<&str>>` is not implemented for `std::io::Error`

The line above in shamelessly copied from the documentation. What am I doing wrong (or what am I missing to do)?

The function you are doing this in likely returns std::io::Result<...> or Result<..., std::io::Error>. To use failure, you need to have it return Result<..., failure::Error> instead.

You certainly have skills. Actually, I'm returning std::io::Result<...> :smiley:

@livingsilver94 It's compatible one way but not the other!
You can convert an std::io::Error into a failure::Error, but not the other way around.
This is because failure::Error can store extra information like context, backtraces and any other information different kinds of errors might store. There just isn't a place for that information to be stored in an std::io::Error. It's as much compatibility as you can expect between a type built specifically for IO errors and one which is generalized for any kind of error handling.
I'd recommend you have anything which can fail with extra information either return a custom error type deriving Fail with the extra information built into the type's variants, or failure::Error.

I don't  know what to say more. You explained everything already, thank you for your help :smile: It's probably the best idea to implement the Fail trait in my project.


feat: Add sql.js fallback for sqlite in wasm

This PR adds support for another SQLite driver sql.js is a browser orientated driver which compiles SQLite to wasm.
Enabling the possibility to run cds with @cap-js/sqlite in more environments.
For example better-sqlite3 cannot be loaded into the default npm promoted online environment runkit:
https://runkit.com/bobdenos/cds-test
Error: Could not locate the bindings file. Tried:
 →<EMAIL_ADDRESS> →<EMAIL_ADDRESS> →<EMAIL_ADDRESS> →<EMAIL_ADDRESS> →<EMAIL_ADDRESS> →<EMAIL_ADDRESS> →<EMAIL_ADDRESS> →<EMAIL_ADDRESS> →<EMAIL_ADDRESS> →<EMAIL_ADDRESS> →<EMAIL_ADDRESS> →<EMAIL_ADDRESS> →<EMAIL_ADDRESS>
There are some limitations with sql.js when comparing to better-sqlite3. Which is mostly that it uses more memory and is around 50% slower for running the @cap-js/sqlite tests. Where better-sqlite3 takes ~10 seconds to run all tests. It takes sql.js ~14 seconds, but all current tests are green. Which should be good enough for experimentation scenarios.

Verification that the Test WASM job actually uses the fallback driver: log


Dutch aside text

Updated the aside text for note, caution and danger.

Description

Translated the aside text into Dutch.

This PR still needs a changeset, which I'll write in the coming hours. Busy now releasing Astro 4.0, but after that I should have time for that!


Removed unsupported clouds from Atlas Mongodb

Description
We had included GCP and azure in the registry values for mongodb-atlas; but in reality they are not supported in the code.
Safety checklist

[  X] This change is backwards compatible and safe to apply by existing users
[ X] This change will NOT lead to data loss
[ X] This change will NOT lead to downtime who already has an env/service setup

How has this change been tested, beside unit tests?
Mnaually tested AWS + Mongo

Confirmed working in apply for mongodb-atlas provider.


Trivy

@parkand1 Any updates on this?

@parkand1 Are you still working on this? Could you provide an ETA to complete this?


vpc-sample: sample is unused

The vpc-sample was created but the companion tutorial never published.  Should we delete?

We should connect with iX if there is the ability to quickly finish the tutorial or repurpose this sample.

Merged in CL to incorporate vpc-sample. Closing.


Circular marker support

Proposal
Create a VMapCircle or VMapCircleMarker component with a position as LatLng and a radius in meters (number)
References

Circle
CircleOptions

Hi, thanks for the suggestion. The library is under development, I plan to cover all the functionality of the leaflet and there will also be additional interesting components.


Revert "neovim: 0.9.5 -> 0.10.0"

Please refer to https://github.com/NixOS/nixpkgs/issues/303285.
Reverts NixOS/nixpkgs#311047 as it was not supposed to happen for this release cycle.
A full message is provided in the commit message.

I guess we need to revert these related PRs.
https://github.com/NixOS/nixpkgs/pull/312312 @bew
https://github.com/NixOS/nixpkgs/pull/312630 @PriceHiller

I guess we need to revert these related PRs. #312312 @bew #312630 @PriceHiller

Can you push the revert commits on this PR additionally and provide a reason for their reverts as well?

I guess we need to revert these related PRs. #312312 @bew #312630 @PriceHiller

Can you push the revert commits on this PR additionally and provide a reason for their reverts as well?

Can do in about an hour when I'm at a computer.

@RaitoBezarius
Not sure that I have perms to push onto your PR -- feel free to incorporate my revert commit seperate from me or point me to what I need to do to get those perms.
Thanks 🙂.
I'll likewise track for the next release cycle to update Neovide.

I have reverted those two commits.
@PriceHiller
You need to become a committer to push your commits to PRs by others.

From what I can see, the release branches and channels are expected to be created today, so this can be re-reverted then, right? What a bummer this didn't make into the release, but thank Nix for flake locks.

Yes, this PR can be reverted on master after branch-off.

Thanks @RaitoBezarius for reverting. While I'd love 0.10 in 24.05, the upgrade definitely broke things.

No problem, you're welcome! Thanks for everyone's understanding on this matter and swift resolution, you all are amazing :)


unable to connect to X server: No such file or directory

With the latest chrome OS update in the stable channel my crouton stopped working. So I deleted the existing chroot and started installing it fresh with the latest crouton version. Installation succedes. But ends up with the following error message when starting xfce.
chronos@localhost / $ sudo startxfce4
Entering /mnt/stateful_partition/crouton/chroots/precise...
Unknown username "dnsmasq" in message bus configuration file
/usr/bin/startxfce4: Starting X server

(EE)
Fatal server error:
(EE) Cannot move old log file "/tmp/Xorg.crouton.1.log" to "/tmp/Xorg.crouton.1.log.old"
(EE)
(EE)
Please consult the The X.Org Foundation support
         at http://wiki.x.org
 for help.
(EE)
/usr/bin/xinit: giving up
/usr/bin/xinit: unable to connect to X server: No such file or directory
/usr/bin/xinit: server error
Unmounting /mnt/stateful_partition/crouton/chroots/precise...

My chrome OS version is as follows:
Version 43.0.2357.81 (64-bit)
Platform 6946.55.0 (Official Build) stable-channel swanky
Firmware Google_Swanky.5216.238.5

How can I fix this?

output of croutonversion
(precise)keheliya@localhost:~$ croutonversion
crouton: version 1-20150519153807~master:30cf02c9
release: precise
architecture: amd64
xmethod: xorg
targets: xfce,keyboard,extension
host: version 6946.55.0 (Official Build) stable-channel swanky
kernel: Linux localhost 3.10.18 #1 SMP Tue May 26 18:03:35 PDT 2015 x86_64 x86_64 x86_64 GNU/Linux
freon: yes

@keheliya I think you did try to start it as root once from inside the chroot according to Cannot move old log file "/tmp/Xorg.crouton.1.log" to "/tmp/Xorg.crouton.1.log.old" just reboot to clear /tmp or use sudo rm  /tmp/Xorg.crouton.1.log from a crosh shell. Then try again starting the chroot.

Got it fixed. Thanks for the help.


Click on any move in Current Game box copy evaluation with opposite sign to the clicked move when Continuous Evaluation is on

In the current box the current move is 31.b4 and is evaluated as -7.93:

If I now click on any move in the current box, the evaluation of -7.93 is copied to that move:

example of clicking on 1.e4

example of clicking on 21.Rc4

Fixed


Project Broken Due To Docker / Docker Compose - Unsupported Compose File Version - Root Additional Property Name Is Not Allowed

This project does not work anymore.
With a clean cookiecutter, if you follow the directions in dockerswarm.rocks you get the error: "Unsupported compose file version 1.0"
This happens when you get to docker stack in /scripts/deploy.sh
I think it's because docker stack and docker compose are not compatible.
If you try to add a version to the top of the file after the compose config, i.e.  ...
(echo -e "version: '3.8'\n"; DOMAIN=${DOMAIN?Variable not set} \
TRAEFIK_TAG=${TRAEFIK_TAG?Variable not set} \
STACK_NAME=${STACK_NAME?Variable not set} \
TAG=${TAG?Variable not set} \
docker-compose \
-f docker-compose.yml \
config)

... then you still get this error "(root) additional property **name** is not allowed"

This is because tiangolo updated docker-with-compose image.
update .gitlab-ci.yml will fix....
tiangolo/docker-with-compose:2021-09-18

Yep, there's a lot of things I have to update here, starting with migrating to SQLModel. But first I have to finish dinner things in SQLModel. 🤓

For quick solution update your .gitlab-ci.yml file with following line:
image: tiangolo/docker-with-compose:2021-09-18

I do not like breaking changes.

Yep, there's a lot of things I have to update here, starting with migrating to SQLModel. But first I have to finish dinner things in SQLModel. 🤓

Glad to hear there are plans on the board to update this, I appreciate the project - thank you. I've been working on a project structure loosely based on your project for the last month or so. Do you have any major adjustments for the front end in mind at this stage, or is that loosely going to stay intact?


Add tools for Verilog

Okay now I feel a bit silly for adding a few Verilog tools myself before checking the open PRs. 😆
I'll merge your code in from CLI. Thanks a lot for your support @h313! ❤


Fix 33136 move period inside quotation

Why:
Move period inside quotation in "upstream branch" term to align with style guide
Closes: #33136
What's being changed (if available, include any code snippets, screenshots, or gifs):

Move the period to the inside of the ending quotation

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).
[x] For content changes, I have completed the self-review checklist.

@SeanAverS Thanks so much for opening a PR with this fix! I'll update the branch and get this merged once tests are passing 🍏


go<EMAIL_ADDRESS>unknown revision 9f4d16fe7c2d

hello, guys!
i am a new learner for go-micro.when i fllow this guid https://micro.mu/docs/install-guide.html doing this command 'go get github.com/micro/examples/greeter/srv && srv
' , i get this error:
go: finding github.com/micro/examples/greeter/srv latest
go: finding github.com/micro/examples/greeter latest
go: finding github.com/nats-io/nats.go v1.8.2-0.20190607221125-9f4d16fe7c2d
go<EMAIL_ADDRESS>unknown revision 9f4d16fe7c2d
go: error loading module requirements
may be the version error?

I believe this has been in master, see also: https://github.com/nats-io/nats.go/issues/493

I believe this has been fixed in https://github.com/micro/examples/pull/75
Feel free to re-open the issue if you still have problems getting the dependency.


Use pro-rata output in constructing SwapClaimProof

Fixes #1497

@zbuc should we also have a guard on tx submission that prevents swaps from being submitted when there's no liquidity to begin with? you could fail the transaction entirely, in check_tx_stateful, rather than turning it into a swap NFT that needs to be redeemed.

@zbuc should we also have a guard on tx submission that prevents swaps from being submitted when there's no liquidity to begin with? you could fail the transaction entirely, in check_tx_stateful, rather than turning it into a swap NFT that needs to be redeemed.

@plaidfinch I considered that but decided against it because the available liquidity could change during the block the swap is submitted, i.e. there's no way to tell with certainty whether a swap will succeed or fail when you submit it. But maybe there could be a prompt instead? "There doesn't seem to be liquidity for this trading pair and the swap may fail as a result, are you sure you wish to submit?"

Oh, that's a good point. I think that we should not do interactivity here, per previous conversations with @hdevalence where we decided that interactivity in pcli is probably not a good thing. Let's just have the swap generate a failed NFT that can be immediately claimed back, like you already did :)


webm not working on Safari Browser

I have React Application .webm videos will not play on Safari Browser. Other formats work just not .webm
All it show is a play button with line in the middle.

Same with HLS urls
(pure  tag working well)

I believe it's safari itself and not React player as I have been battling with this issue the entire week.
I'm not an expert but my finding so far is that:
Safari won't play WebM (VP9 codec) but it will play .mov (HEVC h265 codec) as type QuickTime.
Safari might play mp4 files but depends on codec as well and not necessarily with audio.
If you need transparency, i.e., alpha channel videos, it has to be freshly exported as the previously mentioned HEVC h265 codec .mov file. If you compress/convert it there's a good chance it won't have alpha channel enabled afterward.

Facing same issue on Safari 16

indeed this is the reason the url accepts an array. so you can define different formats to support all browsers.
Safari doesn't support webm, try mov or mp4.
or move to video streaming like HLS or DASH which offers more benefits.
see https://howvideo.works


676 release ois 2.3

Closes #676
Release OIS v2.3 without a compatible Airnode version. Not that on the versions page the chart shows the Airnode version as "to be released". Maybe there is a better phrase?

What about "Pending" ?

I did start with pending and changed it. Was not a good choice on my part so Pending it is.

think there needs to be changes throughout the Airnode docs that point to OIS.

Makes sense.


Add FAQ

Pull Request
Description
I just talked to Bitmarck about a validation problem and this is the outcome.
Closes

Thanks for looking this up! That is reassuring, I was afraid they had changed something recently. How did you contact bitmarck?

Via mail and phone:
<EMAIL_ADDRESS>0800 24862725


Can we have an option to reverse the buttons order?

In Sweetalert, the buttons are left to right, while here, in Sweetalert2, the buttons are right to left (check 'Are you sure' example)
Can we have an option to reverse the order of the buttons, for them to be ltr again?

there's reverseButtons parameter

Thanks, didnt see that one


Be able to add multiple compositeKeyboardVector steps

Right now if you add two compositeKeyboardVector steps to a Control then only the last one takes effect. The use case for having more than two would for example have wasd and arrow keys control player movement.

That's a very good point - thanks for submitting this issue!


Quests aren't working

In your issue, please include:

Spigot and Quests versions (do not say "latest")
git-Spigot-7228328-14b597f (MC 1.12) (Implementing API version 1.12-R0.1-SNAPSHOT)
Quests 2.9.5
Description of problem & console errors
Quest editor just isn't processing any messages in chat since new update, quests are duplicating on reload and oak logs aren't being counted.
Pastebin an example quest from quests.yml for testing, if applicable
https://pastebin.com/kbHgTzhn

Duplicate of #73


[Task]: Implement a test suite for snowflake connector

Connector Name
module/snowflake (SnowFlake)
Task Description
Need do do the $subject as its required for all the connector modules to have 80+% test coverage and, this is also required to verify the GraalVM compatibility of the connector runtime.

Hi @NipunaRanasinghe, I would like to work on this issue if available.

Thanks for reaching out @Harshal0902.
Yes, you can allocate this task and start working on it.
Feel free to ask questions, propose ideas, or report issues. Together, we can make Ballerina even better!
Happy coding! 🎉


Context type inference on createRouter is not working anymore

Describe the bug
When creating a new router with createRouter and a context value, the returned router doesn't contain valid typing on the options.context properties. This was working in version 1.34.5 but doesn't work anymore in version 1.44.2 (I did not test versions in between).
Your Example Website or App
https://stackblitz.com/edit/github-dysjpq?file=src%2Fmain.tsx
Steps to Reproduce the Bug or Issue
import ReactDOM from 'react-dom/client';
import { RouterProvider, createRouter } from '@tanstack/react-router';
import { routeTree } from './routeTree.gen';

// Set up a Router instance
const router = createRouter({
  routeTree,
  defaultPreload: 'intent',
  context: {
    value: 'test',
  },
});

// Type inference is not working there (Property 'value' does not exist on type 'AnyContext'.)
const value = router.options.context.value;

Expected behavior
Type inference on router.options.context.value should be inferred correctly.
Screenshots or Videos
No response
Platform

OS: Windows
Browser: Chrome

Additional context
No response

It is failing again on ^1.49.2.
Using the same stackblitz from OP.
When loading main.tsx for the first time inference seems to work but if you delete and retype the value inference fails.


Requesting support for discovery/v1 CRDs

Support dies for discovery/v1 CRDs. e.g., there's no die for EndpointSlice

Thanks!


ShimmerEffect alters widget dimensions

Animation widget has its dimensions 1 px at the bottom and 1 px on the right side
In this snipet container will be overflowed at the bottom by 1 px
Container(
  height: 86,
  color: Colors.red,
  child: Animate(
    effects: [
      ShimmerEffect(
      duration: 1000.ms,
      color: AppColors.shimmerAnimationCollor
      )
    ],
    controller: _animationController,
    onPlay: (controller) => controller.repeat(),
    child: Container(
      height: 86,
    )
 )
)

This is because ShaderMask doesn't do a particularly good job with edges, so by default ShimmerEffect adds 0.5px of padding around the widget to avoid visual artifacts.
You can disable this by setting padding: EdgeOffsets.zero.
I think I can work around this by creating a PaddedShaderMask, having it apply the padding, but return the original dimensions for layout. I'll put it on my list, but feel free to look into it in the interim.


Crashing on MESSAGE: EVENT_STA_DISCONNECTED

Prerequisites

[x] I have written a descriptive issue title
[x] I have verified that I am using the latest version of esp32-wifi-manager
[x] I have searched open and closed issues to ensure it has not already been reported

Description
Using esp32-wifi-manager, it works flawlessly until the ESP32 cannot connect to the wireless router. As soon as the wifi connection is not seen the ESP crashes. It seems to be some form of heap corruption and occurs when the disconnect message is added to the queue. I have tried on multiple devices and multiple applications, including stripped down application and even the examples app provided. All have the same behavior.
Is this reproducible or am I doing something stupid?
Below are logs from running the "default_demo" example
Rebooting...
ets Jun  8 2016 00:22:57

rst:0xc (SW_CPU_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:2
load:0x3fff0030,len:4
load:0x3fff0034,len:7008
ho 0 tail 12 room 4
load:0x40078000,len:13860
load:0x40080400,len:3952
0x40080400: _init at ??:?

entry 0x40080678
I (31) boot: ESP-IDF v4.3-dev-907-g6c17e3a64-dirty 2nd stage bootloader
I (31) boot: compile time 20:49:49
I (32) boot: chip revision: 1
I (36) boot_comm: chip revision: 1, min. bootloader chip revision: 0
I (43) boot.esp32: SPI Speed      : 40MHz
I (47) boot.esp32: SPI Mode       : DIO
I (52) boot.esp32: SPI Flash Size : 4MB
I (56) boot: Enabling RNG early entropy source...
I (65) boot: ## Label            Usage          Type ST Offset   Length
I (73) boot:  0 nvs              WiFi data        01 02 00009000 00006000
I (80) boot:  1 phy_init         RF data          01 01 0000f000 00001000
I (88) boot:  2 factory          factory app      00 00 00010000 00100000
I (95) boot: End of partition table
I (99) boot_comm: chip revision: 1, min. application chip revision: 0
I (106) esp_image: segment 0: paddr=0x00010020 vaddr=0x3f400020 size=0x1ecd0 (126160) map
I (164) esp_image: segment 1: paddr=0x0002ecf8 vaddr=0x3ffb0000 size=0x01320 (  4896) load
I (166) esp_image: segment 2: paddr=0x00030020 vaddr=0x400d0020 size=0x7b970 (506224) map
0x400d0020: _stext at ??:?

I (363) esp_image: segment 3: paddr=0x000ab998 vaddr=0x3ffb1320 size=0x027d8 ( 10200) load
I (367) esp_image: segment 4: paddr=0x000ae178 vaddr=0x40080000 size=0x00404 (  1028) load
0x40080000: _WindowOverflow4 at X:/esp-idf/components/freertos/xtensa/xtensa_vectors.S:1730

I (370) esp_image: segment 5: paddr=0x000ae584 vaddr=0x40080404 size=0x15828 ( 88104) load
I (428) boot: Loaded app from partition at offset 0x10000
I (429) boot: Disabling RNG early entropy source...
I (440) cpu_start: cpu freq: 160
I (440) cpu_start: Pro cpu up.
I (440) cpu_start: Starting app cpu, entry point is 0x400813d4
0x400813d4: call_start_cpu1 at X:/esp-idf/components/esp_system/port/cpu_start.c:112

I (428) cpu_start: App cpu up.
I (458) cpu_start: Pro cpu start user code
I (459) cpu_start: Application information:
I (459) cpu_start: Project name:     wifi_manager
I (464) cpu_start: App version:      5e8b4c6-dirty
I (469) cpu_start: Compile time:     Sep 16 2020 20:47:24
I (476) cpu_start: ELF file SHA256:  2fa8800e99479cd5...
I (482) cpu_start: ESP-IDF:          v4.3-dev-907-g6c17e3a64-dirty
I (489) heap_init: Initializing. RAM available for dynamic allocation:
I (496) heap_init: At 3FFAE6E0 len 00001920 (6 KiB): DRAM
I (502) heap_init: At 3FFB80C0 len 00027F40 (159 KiB): DRAM
I (508) heap_init: At 3FFE0440 len 00003AE0 (14 KiB): D/IRAM
I (514) heap_init: At 3FFE4350 len 0001BCB0 (111 KiB): D/IRAM
I (521) heap_init: At 40095C2C len 0000A3D4 (40 KiB): IRAM
I (528) spi_flash: detected chip: generic
I (532) spi_flash: flash io: dio
I (537) cpu_start: Starting scheduler on PRO CPU.
I (0) cpu_start: Starting scheduler on APP CPU.
I (629) wifi_manager: Set STA IP String to: <IP_ADDRESS>
I (639) main: free heap: 251892
I (649) system_api: Base MAC address is not set
I (649) system_api: read default base MAC address from EFUSE
I (669) wifi_init: rx ba win: 6
I (669) wifi_init: tcpip mbox: 32
I (669) wifi_init: udp mbox: 6
I (669) wifi_init: tcp mbox: 6
I (669) wifi_init: tcp tx win: 5744
I (679) wifi_init: tcp rx win: 5744
I (679) wifi_init: tcp mss: 1440
I (689) wifi_init: WiFi IRAM OP enabled
I (689) wifi_init: WiFi RX IRAM OP enabled
I (1559) phy: phy_version: 4370, 4e803b3, Aug 11 2020, 14:18:07, 0, 0
I (1569) wifi_manager:
I (1569) http_server: Registering URI handlers
I (1579) wifi_manager: wifi_manager_fetch_wifi_sta_config: ssid:AzCOM_MICRO password:xxxxxxxx
I (1579) wifi_manager: Saved wifi found on startup. Will attempt to connect.
I (1589) wifi_manager: MESSAGE: ORDER_CONNECT_STA
I (3649) wifi_manager: WIFI_EVENT_STA_DISCONNECTED
Guru Meditation Error: Core  0 panic'ed (LoadProhibited). Exception was unhandled.

Core  0 register dump:
PC      : 0x4008aecb  PS      : 0x00060233  A0      : 0x8008b42c  A1      : 0x3ffbf420
0x4008aecb: is_free at X:/esp-idf/components/heap/multi_heap.c:138
 (inlined by) get_prev_free_block at X:/esp-idf/components/heap/multi_heap.c:194

A2      : 0x004f5243  A3      : 0x3ffc926c  A4      : 0x3ffbc178  A5      : 0x00000000
A6      : 0x00000000  A7      : 0x00000001  A8      : 0x00000001  A9      : 0x00000001
A10     : 0xfffffffc  A11     : 0x00000000  A12     : 0x3ffc9238  A13     : 0x00060223
A14     : 0x00000001  A15     : 0xfffffffc  SAR     : 0x00000004  EXCCAUSE: 0x0000001c
EXCVADDR: 0x004f5243  LBEG    : 0x4000c2e0  LEND    : 0x4000c2f6  LCOUNT  : 0xffffffff

Backtrace:0x4008aec8:0x3ffbf420 0x4008b429:0x3ffbf440 0x40081986:0x3ffbf460 0x4008e8a5:0x3ffbf480 0x4014ad5e:0x3ffbf4a0 0x4014add5:0x3ffbf4e0 0x40087a51:0x3ffbf500
0x4008aec8: get_prev_free_block at X:/esp-idf/components/heap/multi_heap.c:191 (discriminator 1)

0x4008b429: multi_heap_free_impl at X:/esp-idf/components/heap/multi_heap.c:547

0x40081986: heap_caps_free at X:/esp-idf/components/heap/heap_caps.c:305

0x4008e8a5: free at X:/esp-idf/components/newlib/heap.c:47

0x4014ad5e: post_instance_delete at X:/esp-idf/components/esp_event/esp_event.c:436
 (inlined by) esp_event_loop_run at X:/esp-idf/components/esp_event/esp_event.c:604

0x4014add5: esp_event_loop_run_task at X:/esp-idf/components/esp_event/esp_event.c:115

0x40087a51: vPortTaskWrapper at X:/esp-idf/components/freertos/xtensa/port.c:169

Steps to Reproduce

Load default_demo
Connect to AP using and configure local wifi
Remove power from Wifi router to simulate a  WIFI_EVENT_STA_DISCONNECTED
ESP crashes with "Guru Meditation Error: Core  0 panic'ed (LoadProhibited). Exception was unhandled."

System Configuration
Building on Windows 10
Latest esp-idf (ESP-IDF v4.3-dev-907)
Lastest esp32-wifi-manager

As a further note. I tried reverting back to ESP-IDF 4.2 with the same result

Thank you for the detailed report. Hopefully it should be an easy one to reproduce on my end. I’ll check this weekend.

Hi thank you for the quick response (and sharing this really cool app). I think that I have found the issue.
In the wifi_manager_event_handler function at the WIFI_EVENT_STA_DISCONNECTED case, the code is allocating memory for the disconnected param but it is using the variable rather than type to determine size.
wifi_event_sta_disconnected_t* wifi_event_sta_disconnected = (wifi_event_sta_disconnected_t*)malloc(sizeof(wifi_event_sta_disconnected));`
I believe rather it should be
wifi_event_sta_disconnected_t* wifi_event_sta_disconnected = (wifi_event_sta_disconnected_t*)malloc(sizeof(wifi_event_sta_disconnected**_t**));`
I have added this change locally and it seems to now be behaving correctly with no heap corruption.
Please let me know your thoughts

Yeah that's a big oops. I have checked the code and the rest of the params are ok.
eg
wifi_event_sta_scan_done_t* event_sta_scan_done = (wifi_event_sta_scan_done_t*)malloc(sizeof(wifi_event_sta_scan_done_t));

and
ip_event_got_ip_t* ip_event_got_ip = (ip_event_got_ip_t*)malloc(sizeof(ip_event_got_ip_t));


chore(firestore-bigquery-export): updated CHANGELOG and version for 0.1.15

Codecov Report

Merging #683 (27c2140) into next (5fc7ef6) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 27c2140 differs from pull request most recent head 2dfdbb0. Consider uploading reports for the commit 2dfdbb0 to get more accurate results

@@           Coverage Diff           @@
##             next     #683   +/-   ##
=======================================
  Coverage   50.65%   50.65%
=======================================
  Files          30       30
  Lines        1313     1313
  Branches      193      193
=======================================
  Hits          665      665
  Misses        647      647
  Partials        1        1


v11 LevelDB databases missing in zip

Since the package is minimum compatibility v11, could I please request that v11 LevelDB databases be included in upcoming releases?
This would allow users on the Forge to use the Bazaar version of the v11 system.
Currently, they need to install the package as a custom package so that Foundry can have write access in the directory (which exists in the Forge Bazaar) to migrate the databases from .db files to LevelDB databases.

There may be a valid reason for keeping the v10 databases, though. If is the case and this module doesn't aim to include the v11 databases in the zip, instead relying on Foundry's v10 -> v11 migration to LevelDB when the system is first accessed after an update, please let me know! 🙏

I thought that letting Foundry run the upgrade against the v10 databases seemed to work well for my Windows Electron setup as well as my self-hosted Ubuntu server setup. But I don't use the Forge so that wasn't a scenario I tested. I did have one other issue from a Forge user, but they had a workaround for it.
The only real reason I kept the v10 databases was that the v11 database files are constantly changing, which was a bit annoying from a GIT perspective. I can look into the right way to include the v11 files instead, since I was not shooting for backwards compatibility with v10 anyways.

@rikmarais I'm not that familiar with the Forge, could you try the new version that just released (v1.2.6) and see if it resolves your issues? @jalensailin  has changed our packaging process so that hopefully now we will just have the v11 databases for compendia automatically deployed.

This fix is brilliant 🙏 Thank you and all contributors for your efforts. Testing is all green on our side.


Support sending transactions from other signers

Currently we send all transactions using the default signer account (index zero).
We should allow specifying a different signer to send from, per transaction:
proposed API:
const sample = m.contract('Sample', {
  signer: ethers.Signer;
})

m.call(sample, 'foo', {
  signer: ethers.Signer;
})

Part of #205

This is in reference to the old api.


Special-case FileHandle(forWritingAtPath: "CONOUT$") on Windows.

This PR special-cases calls to FileHandle.init(forWritingAtPath:) when passed "CONOUT$" (the reserved "console output" file) on Windows such that it does not open a new file but just returns a copy of .stdout instead.
This change is necessary because, on Windows, opening and locking "CONOUT$" does not lock stdout (they're considered entirely different files) and output written to "CONOUT$" is wrapped at the console's column limit even though it's being written as binary data. The VSCode Swift plugin authors would like to be able to specify writing the JSON event stream to "CONOUT$", but the observed behaviour is stopping them from doing so.
Checklist:

[ ] Code and documentation should follow the style of the Style Guide.
[ ] If public symbols are renamed or modified, DocC references should be updated.

@swift-ci test

LOL, got to love Windows. Nice catch!

@swift-ci test


Wrong migration when foreign key property belong to base class

I have following classes (important notice Parent inherits ParentBase)
public class ParentBase
{
    public Guid ChildId { get; private set; }

    public ParentBase(Guid childId)
    {
        ChildId = childId;
    }
}

public class Parent : ParentBase
{
    public Guid Id { get; private set; }

    public string Code { get; private set; }

    public Parent(Guid id, string code, Guid childId)
		: base(childId)
    {
        Id = id;
        Code = code;
    }
}

public class Child
{
    public Guid Id { get; private set; }

    public Child(Guid id)
    {
        Id = id;
    }
}

and following mapping configuration
modelBuilder.Entity<Parent>(b =>
{
    b.HasKey(e => e.Id);
    b.HasOne<Child>().WithMany().HasForeignKey(x => x.ChildId).IsRequired();
});

When I add migration then it generates:
Actual result
migrationBuilder.CreateTable(
    name: "Parent",
    schema: "Test",
    columns: table => new
    {
        Id = table.Column<Guid>(type: "uuid", nullable: false),
        ChildId = table.Column<Guid>(type: "uuid", nullable: false),
        ChildId1 = table.Column<Guid>(type: "uuid", nullable: true),
        Code = table.Column<string>(type: "text", nullable: true)
    },
    constraints: table =>
    {
        table.PrimaryKey("PK_Parent", x => x.Id);
        table.ForeignKey(
            name: "FK_Parent_Child_ChildId",
            column: x => x.ChildId,
            principalSchema: "Test",
            principalTable: "Child",
            principalColumn: "Id",
            onDelete: ReferentialAction.Cascade);
        table.ForeignKey(
            name: "FK_Parent_Child_ChildId1",
            column: x => x.ChildId1,
            principalSchema: "Test",
            principalTable: "Child",
            principalColumn: "Id",
            onDelete: ReferentialAction.Restrict);
    });

As you see, there is additional nullable column  ChildId1 which is wrong. When I remove ParentBase and move properties ChildId to Parent class then it generates correct migration as excepted result.
Expected result
migrationBuilder.CreateTable(
    name: "Parent",
    schema: "Test",
    columns: table => new
    {
        Id = table.Column<Guid>(type: "uuid", nullable: false),
        ChildId = table.Column<Guid>(type: "uuid", nullable: false),
        Code = table.Column<string>(type: "text", nullable: true)
    },
    constraints: table =>
    {
        table.PrimaryKey("PK_Parent", x => x.Id);
        table.ForeignKey(
            name: "FK_Parent_Child_ChildId",
            column: x => x.ChildId,
            principalSchema: "Test",
            principalTable: "Child",
            principalColumn: "Id",
            onDelete: ReferentialAction.Cascade);
    });

Further technical details
EF Core version: 2.0.0
Database Provider: Npgsql.EntityFrameworkCore.PostgresSQL 2.0.0
Operating system: Windows 10
IDE: Visual Studio 2017

@myshon I was able to reproduce this and it looks like it's a side-effect of #9705. Workarounds are to either give ChildId a public setter or explicitly map it before configuring the relationship:
modelBuilder.Entity<Parent>(b =>
{
    b.HasKey(e => e.Id);
    b.Property(e => e.ChildId);
    b.HasOne<Child>().WithMany().HasForeignKey(x => x.ChildId).IsRequired();
});

@ajcvickers I would like keep DDD approach so public setter doesn't sound good, but explicit configuration is fine. Thanks!

Same root cause as #9705, and fixed by the fix for that issue which already went into preview2. Closing as duplicate.


Cannot set property id of # which has only a getter

Hi
I have started getting the following error when changing layouts dynamically since updating to 0.3.26
Do you have any ideas ? If not, I'll try and replicate the issue in https://codesandbox.io/ so you can have a look, hopefully I will be able to replicate the issue in a simple case.
I get the error when changing from one set of layouts to a completly separate set of layouts at the same breakpoint (screen size). The id's are different so its not a duplicate id issue. The crash caues the debug view not to update, but the error still occurs with debug turned off.
TypeError: Cannot set property id of #<DashItem> which has only a getter     at VueComponent.handler (vue-responsive-dash.common.js?e6d2:10032)     at Watcher.run (vue.runtime.esm.js?2b0e:4568)     at flushSchedulerQueue (vue.runtime.esm.js?2b0e:4310)     at Array.eval (vue.runtime.esm.js?2b0e:1980)     at flushCallbacks (vue.runtime.esm.js?2b0e:1906)

If you can setup a repo that always makes it easier for me to see whats going on.

I used another test we did in the past but could not replicate this issue: [https://codesandbox.io/s/vue-responsive-dash-3zorp?file=/src/App.vue]9https://codesandbox.io/s/vue-responsive-dash-3zorp?file=/src/App.vue)

Hi
I was able to replicate it, I found the cause was using the index as the key
it works with
<dash-item v-for="item in layout.items" v-bind.sync="item" :key="item.id">
but not with
<dash-item v-for="(item, index) in layout.items" v-bind.sync="item" :key="index">
Replicated here:
https://codesandbox.io/s/vue-responsive-dash-qd5d3?file=/src/App.vue
Does the key have to be the item.id ?
I think I changed it to index when i was trying to get the drag drop to work.
Thanks
Peter

Hi,
I believe it does need to be item.id otherwise the system may not sync back to the array of items in the correct order or change the wrong item. This is why the error occurs as its actually changing the dash items internally.

ok thanks, i will change it back to id.


[aws provider] moto library not available!

following the AWS set at http://cloudbridge.readthedocs.io/en/latest/getting_started.html
with
image_id = 'ami-2d39803a'  # Ubuntu 14.04 (HVM)
are there other setup steps required?

Thanks for reporting this. It's caused by a print statement when it should have been a log statement, and has since been fixed: https://github.com/gvlproject/cloudbridge/blob/master/cloudbridge/cloud/providers/aws/provider.py#L12
You can safely ignore this - it only comes up when you have a non dev-mode installation of cloudbridge.

Thanks -- I was impatient -- it takes a while for the demonstration commands
print(provider.compute.images.list())
print(provider.security.security_groups.list())
print(provider.block_store.snapshots.list())
print(provider.object_store.list())
to return... I took the message to indicate an error state and killed the
script.
On Wed, Sep 27, 2017 at 10:41 PM, Nuwan Goonasekera <
<EMAIL_ADDRESS>wrote:

Thanks for reporting this. It's caused by a print statement when it should
have been a log statement, and has since been fixed:
https://github.com/gvlproject/cloudbridge/blob/master/
cloudbridge/cloud/providers/aws/provider.py#L12
You can safely ignore this - it only comes up when you have a non dev-mode
installation of cloudbridge.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gvlproject/cloudbridge/issues/87#issuecomment-332711306,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEaOwvbezKcyimuuc1BZCnIipC7aY5BLks5smwdcgaJpZM4Pmli9
.

compute.images.list() is particularly slow, because it doesn't filter images by default. We are thinking of optimising that to only list images owned by the current user by default. It should probably not be the first command to try out in the getting started guide so we'll remove it. The other commands should finish reasonably quickly though. Thanks for the useful feedback.

Documentation fixed in commit: https://github.com/gvlproject/cloudbridge/commit/940fc7504d4bee75180816e70f6f67d7f608402c#diff-9bc8bf6e8d9db020cefce1fbc0426795R72


V3 API Mediation Layer related cleanup

Describe your pull request here:
I went through the API ML related doc for V3 and cleansed pieces that aren't relevant for V3 or claim information such as supported from specific Zowe version.
List the file(s) included in this PR:
After creating the PR, follow the instructions in the comments.

@anaxceron Resolved all the comments.


feat: add intrinsic attributes from HTML components

support intrinsic attributes from button, a (in Link), input, label, paragraph, span and headings
refactor types
cleanup

:tada: This PR is included in version 4.7.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Merge v2 of overseas passport

I would copy over a full description of changes from the V2 PR to the second commit message, so that a dev looking at git history could read why those changes where made to the V1 files.


Remove register link

Signed-off-by: Julien Pivotto<EMAIL_ADDRESS>

LGTM.
Sent by mobile; please excuse my brevity.
On Tue, May 4, 2021, 20:18 Julien Pivotto @.***> wrote:

@roidelapluie https://github.com/roidelapluie requested your review on:
#190 https://github.com/promcon/website/pull/190 Remove register link.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/promcon/website/pull/190#event-4684949119, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFYII4P7UTV7BILDCZZVH3TMA26ZANCNFSM44DGQQIQ
.