undefined symbol: cudaEventDestroy

Hi,
I installed this package as a dependency for torch-dataset. After installing, I can't seem to load up the dataset module due to this problem in ipc:
th> require 'libipc'
/home/victor/torch/install/share/lua/5.1/trepl/init.lua:384: ...e/victor/torch/install/share/lua/5.1/luarocks/loader.lua:117: error loading module 'libipc' from file '/home/victor/torch/install/lib/lua/5.1/libipc.so':
        /home/victor/torch/install/lib/lua/5.1/libipc.so: undefined symbol: cudaEventDestroy
stack traceback:
        [C]: in function 'error'
        /home/victor/torch/install/share/lua/5.1/trepl/init.lua:384: in function 'require'
        [string "_RESULT={require 'libipc'}"]:1: in main chunk
        [C]: in function 'xpcall'
        /home/victor/torch/install/share/lua/5.1/trepl/init.lua:651: in function 'repl'
        ...ctor/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:199: in main chunk
        [C]: at 0x00406690

This is a fresh torch installation. I have cuda 7.0 and nvidia-smi shows the drivers:
 $ nvidia-smi                                                                                [13:48:01]
Tue Feb  9 13:48:05 2016
+------------------------------------------------------+
| NVIDIA-SMI 346.82     Driver Version: 346.82         |
|-------------------------------+----------------------+----------------------+

Any ideas what might be wrong? Am I missing a required library that defines cudaEventDestroy?

Hi,
Can you load cutorch? Just run th (the Torch repl) and type
require ‘cutorch’
cutorch.getDeviceCount()
Does that work?
On Tue, Feb 9, 2016 at 1:49 PM, Victor Zhong<EMAIL_ADDRESS>wrote:

Hi,
I installed this package as a dependency for torch-dataset. After
installing, I can't seem to load up the dataset module due to this
problem in ipc:
th> require 'libipc'
/home/victor/torch/install/share/lua/5.1/trepl/init.lua:384: ...e/victor/torch/install/share/lua/5.1/luarocks/loader.lua:117: error loading module 'libipc' from file '/home/victor/torch/install/lib/lua/5.1/libipc.so':
/home/victor/torch/install/lib/lua/5.1/libipc.so: undefined symbol: cudaEventDestroy
stack traceback:
[C]: in function 'error'
/home/victor/torch/install/share/lua/5.1/trepl/init.lua:384: in function 'require'
[string "_RESULT={require 'libipc'}"]:1: in main chunk
[C]: in function 'xpcall'
/home/victor/torch/install/share/lua/5.1/trepl/init.lua:651: in function 'repl'
...ctor/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:199: in main chunk
[C]: at 0x00406690
This is a fresh torch installation. I have cuda 7.0 and nvidia-smi shows
the drivers:
$ nvidia-smi                                                                                [13:48:01]
Tue Feb  9 13:48:05 2016
+------------------------------------------------------+
| NVIDIA-SMI 346.82     Driver Version: 346.82         |
|-------------------------------+----------------------+----------------------+
Any ideas what might be wrong? Am I missing a required library that
defines cudaEventDestroy?
—
Reply to this email directly or view it on GitHub
https://github.com/twitter/torch-ipc/issues/6.

Thanks for helping @zakattacktwitter :)
It does work:
th> require 'cutorch';
th> cutorch.getDeviceCount()
8

Did you end up figuring this out?

@zakattacktwitter No. In addition I reproduced this same problem on my home machine with CUDA as well.

Is nvcc in your path? What does which nvcc report?
On Thu, Feb 11, 2016 at 9:29 AM, Victor Zhong<EMAIL_ADDRESS>wrote:

@zakattacktwitter https://github.com/zakattacktwitter No. In addition I
reproduced this same problem on my home machine with CUDA as well.
—
Reply to this email directly or view it on GitHub
https://github.com/twitter/torch-ipc/issues/6#issuecomment-182967563.

yup
 $ nvcc --version                                                                                nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2015 NVIDIA Corporation
Built on Tue_Aug_11_14:27:32_CDT_2015
Cuda compilation tools, release 7.5, V7.5.17

what does which nvcc report?
On Thu, Feb 11, 2016 at 9:31 AM, Victor Zhong<EMAIL_ADDRESS>wrote:

yup
$ nvcc --version                                                                                nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2015 NVIDIA Corporation
Built on Tue_Aug_11_14:27:32_CDT_2015
Cuda compilation tools, release 7.5, V7.5.17
—
Reply to this email directly or view it on GitHub
https://github.com/twitter/torch-ipc/issues/6#issuecomment-182968145.

 $ which nvcc
/opt/cuda/bin/nvcc

Moreover I believe this cudaEventDestroy is defined in include/cuda_runtime_api.h. Could it be possible that this file isn't linked properly during the build process for ipc?
victor@archie: /opt/cuda
 $ grep -R cudaEventDestroy include                                                                                [1:36:15]
include/cuda_runtime_api.h: * with ::cudaEventDestroy will result in undefined behavior.
include/cuda_runtime_api.h: * ::cudaEventDestroy,
include/cuda_runtime_api.h: * This event must be freed with ::cudaEventDestroy.
include/cuda_runtime_api.h: * been freed with ::cudaEventDestroy will result in undefined behavior.
include/cuda_runtime_api.h: * ::cudaEventDestroy,
include/cuda_runtime_api.h: * any functions (including ::cudaEventRecord() and ::cudaEventDestroy()) may be
include/cuda_runtime_api.h: * ::cudaEventSynchronize, ::cudaEventDestroy, ::cudaEventElapsedTime,
include/cuda_runtime_api.h: * ::cudaEventSynchronize, ::cudaEventDestroy, ::cudaEventElapsedTime,
include/cuda_runtime_api.h: * ::cudaEventSynchronize, ::cudaEventDestroy, ::cudaEventElapsedTime,
include/cuda_runtime_api.h: * ::cudaEventSynchronize, ::cudaEventDestroy, ::cudaEventElapsedTime
include/cuda_runtime_api.h: * ::cudaEventQuery, ::cudaEventDestroy, ::cudaEventElapsedTime
include/cuda_runtime_api.h: * when ::cudaEventDestroy() is called, the function will return immediately and
include/cuda_runtime_api.h:extern __host__ __cudart_builtin__ cudaError_t CUDARTAPI cudaEventDestroy(cudaEvent_t event);
include/cuda_runtime_api.h: * ::cudaEventSynchronize, ::cudaEventDestroy, ::cudaEventRecord
include/thrust/system/cuda/detail/bulk/future.hpp:        cudaError_t e = cudaEventDestroy(m_event);
include/thrust/system/cuda/detail/bulk/future.hpp:          printf("CUDA error after cudaEventDestroy in future dtor: %s", cudaGetErrorString(e));
include/thrust/system/cuda/detail/cub/util_allocator.cuh:                    if (CubDebug(error = cudaEventDestroy(search_key.ready_event))) break;
include/thrust/system/cuda/detail/cub/util_allocator.cuh:            if (CubDebug(error = cudaEventDestroy(begin->ready_event))) break;
include/cuda_device_runtime_api.h:extern __device__ __cudart_builtin__ cudaError_t CUDARTAPI cudaEventDestroy(cudaEvent_t event);
include/cuda_runtime.h: * ::cudaEventSynchronize, ::cudaEventDestroy, ::cudaEventElapsedTime,

Following up on this, I've noticed similar issues with undefined symbols:
CUDA 7.0 (linux): torch/install/lib/lua/5.1/libipc.so: undefined symbol: cudaMemcpyAsync
CUDA 7.5 (osx):  dlopen(/Users/nico/torch/install/lib/lua/5.1/libipc.so, 6): Symbol not found: _cudaDeviceSynchronize
CUDA / cutorch otherwise appear healthy. I'm using the latest torch / nn / cunn. I haven't yet had time to dig any deeper...

Ok, I think the issue is the CMake version. The CMakeLists.txt relies on CUDA_SDK_ROOT_DIR, which is undefined in my version of CMake (3.4). I think it was deprecated in favor of:
CUDA_TOOLKIT_ROOT_DIR
See: https://cmake.org/cmake/help/v3.0/module/FindCUDA.html

This works for me:
INCLUDE_DIRECTORIES(${CMAKE_CURRENT_SOURCE_DIR})
INCLUDE_DIRECTORIES(${Torch_LUA_INCLUDE_DIR})
INCLUDE_DIRECTORIES(${CUDA_INCLUDE_DIRS})
CUDA_ADD_LIBRARY(ipc MODULE ${src})
TARGET_LINK_LIBRARIES(ipc luaT TH THC)

### Torch packages supposes libraries prefix is "lib"
SET_TARGET_PROPERTIES(ipc PROPERTIES
  PREFIX "lib"
  IMPORT_PREFIX "lib"
  INSTALL_NAME_DIR "@executable_path/${Torch_INSTALL_BIN2CPATH}")

IF(APPLE)
  SET_TARGET_PROPERTIES(ipc PROPERTIES
    LINK_FLAGS "-undefined dynamic_lookup")
ENDIF()

INSTALL(TARGETS ipc
  RUNTIME DESTINATION "${Torch_INSTALL_LUA_CPATH_SUBDIR}"
  LIBRARY DESTINATION "${Torch_INSTALL_LUA_CPATH_SUBDIR}")

INSTALL(FILES ${luasrc} DESTINATION "${Torch_INSTALL_LUA_PATH_SUBDIR}/ipc")

Does anyone solve the problem? I have the correct CUDA setup, but still have the same issue here...

I also got the "Symbol not found: _cudaDeviceSynchronize" error (OS X). Commenting the last if statement in CMakeLists.txt worked for me.
#IF (CUDA_FOUND AND NOT ("$ENV{CUDA}" STREQUAL "NO"))
#   INCLUDE_DIRECTORIES(${CUDA_INCLUDE_DIRS})
#   INCLUDE_DIRECTORIES("${CUDA_SDK_ROOT_DIR}/common/inc")
#   SET(CMAKE_C_FLAGS "-DUSE_CUDA ${CMAKE_C_FLAGS}")
#   IF (NOT "$ENV{TH_ONLY_STATIC}" STREQUAL "YES")
#      TARGET_LINK_LIBRARIES(ipc luaT TH THC)
#   ENDIF()
#ELSE()
   TARGET_LINK_LIBRARIES(ipc luaT TH)
#ENDIF()

Same here (osx). When I install with "CUDA=NO luarocks make ipc-scm-1.rockspec" it works but obviously it doesn't work with CUDA. When I install with "luarocks install ipc" I get "Symbol not found: _cudaDeviceSynchronize" error.

I had this problem too. I had to add set CUDA_TOOLKIT_ROOT_DIR.
Then changed TARGET_LINK_LIBRARIES(ipc luaT TH THC) to TARGET_LINK_LIBRARIES(ipc luaT TH THC ${CUDA_LIBRARIES})

FWIW I encountered the same problem (archlinux , /opt/cuda/bin/nvcc, CUDA V7.5.17, no problem to load cutorch via th).
If I explicitly link against CUDA libraries (see above) with TARGET_LINK_LIBRARIES(ipc luaT TH THC ${CUDA_LIBRARIES}) then /opt/cuda/lib64/libcudart_static.a is added at link stage:
[ 50%] Linking C shared module libipc.so
/usr/bin/cmake -E cmake_link_script CMakeFiles/ipc.dir/link.txt --verbose=1
/usr/bin/cc  -fPIC -DUSE_CUDA ... /opt/cuda/lib64/libcudart_static.a ...

So thanks to this static linking there is no more problem:
$ nm ~/torch/install/lib/lua/5.1/libipc.so | grep -i cudaEventDestroy
000000000005ac70 t cudaEventDestroy
0000000000069430 r _ZZ16cudaEventDestroyE12__FUNCTION__

And th -e "print(require('libipc'))" works like a charm.

Hi,
Glad everyone has found solutions. Can we post one as a pull request so
others can take advantage? I don't experience this problem so I can't post
a fix.
Thanks,
Zak
On Wednesday, April 6, 2016, Cédric Deltheil<EMAIL_ADDRESS>wrote:

FWIW I encountered the same problem (archlinux , /opt/cuda/bin/nvcc, CUDA
V7.5.17, no problem to load cutorch via th).
If I explicitly link against CUDA libraries (see above) with TARGET_LINK_LIBRARIES(ipc
luaT TH THC ${CUDA_LIBRARIES}) then /opt/cuda/lib64/libcudart_static.a is
added at link stage:
[ 50%] Linking C shared module libipc.so
/usr/bin/cmake -E cmake_link_script CMakeFiles/ipc.dir/link.txt --verbose=1
/usr/bin/cc  -fPIC -DUSE_CUDA ... /opt/cuda/lib64/libcudart_static.a ...
So thanks to this static linking there is no more problem:
$ nm ~/torch/install/lib/lua/5.1/libipc.so | grep -i cudaEventDestroy
000000000005ac70 t cudaEventDestroy
0000000000069430 r ZZ16cudaEventDestroyE12__FUNCTION_
And th -e "print(require('libipc'))" works like a charm.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/twitter/torch-ipc/issues/6#issuecomment-206401935

Done: see #21.

Pretty sure this is fixed now. Thanks!


fix: LSDV-3861: XSS in error messages

URLs are used directly in error messages about failed resource
loading. This allows to simply exploit XSS with short "urls" in data:
"data": {
  "url": "https://>"
}

Now all occurencies are escaped.
PR fulfills these requirements

[x] Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
[ ] Tests for the changes have been added/updated (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[x] Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
[x] Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)
(check all that apply)

[ ] Product design
[ ] Frontend

Describe the reason for change
All HTML files are secured before displaying in LSF, scripts and iframes are removed, handlers are removed as well. But error messages were not secured, allowing to exploit XSS there.
What does this fix?
Escapes all occurencies of URL from task data in error messages.
What is the new behavior?
Everything is escaped.
What is the current behavior?
Possible XSS.
What libraries were added/updated?
none
Does this change affect performance?
nope
Does this change affect security?
Improves it!
Does this PR introduce a breaking change?
(check only one)

[ ] Yes, and covered entirely by feature flag(s)
[ ] Yes, and covered partially by feature flag(s)
[x] No
[ ] Not sure (briefly explain the situation below)

What level of testing was included in the change?
(check all that apply)

[ ] e2e
[ ] integration
[ ] unit

Which logical domain(s) does this change affect?
Security

Codecov Report
Patch coverage has no change and project coverage change: +0.02 :tada:

Comparison is base (f01bd95) 65.37% compared to head (0bb7e73) 65.40%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1419      +/-   ##
==========================================
+ Coverage   65.37%   65.40%   +0.02%
==========================================
  Files         436      436
  Lines       27267    27267
  Branches     7222     7222
==========================================
+ Hits        17825    17833       +8
+ Misses       8107     8102       -5
+ Partials     1335     1332       -3

Impacted Files
Coverage Δ

src/utils/messages.js
30.76% <ø> (ø)

... and 2 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

/git merge master


Validate unique key values in list of dictionaries

I would like to be able to validate the uniqueness of key values in a list of dictionaries as well having normal validation for strings, floats etc. for these key values.
Example of .yml that should fail this kind of validation:
pets:
  - name: my_favourite_pet
    type: cat
  - name: my_favourite_pet
    type: dog

I'm able to make a custom validator to do this, but then I'm not able to apply other validations to 'name' and 'type' (like string validation for 'name' and enum for 'type'):
class Pets(Validator):
    """ Custom pets validator """
    tag = "pets"

    def _is_valid(self, value):
        pet_names = [pet['name'] for pet in value]
        if len(value) == len(set(pet_names)):
            return True
        return False

    def fail(self, value):
        return "Pet names are not unique."

Where the schema for this would be:
pets: pets()
Any tips on how to accomplish this, except wirting a validator that includes the default validators that I want to include?

You could override validators.List to add a new Constraint, something like UniqueItemsByProperty that checks each item on the list for a property, maybe like:
def _is_valid(self, value):
    if not value or not self.unique_property:
        return None
    errors = []
    seen = set()
    for item in value:
        property = getattr(item, self.unique_property, None) if item else None
        if property in seen:
            errors.append(self._fail(property, value))
        seen.add(property)
    return errors

Then list(include('Pet'), unique_property='name').
The same could be done with a kwarg in the List directly, if you prefer a smaller footprint.

Thank you for the suggestion @mechie, that worked like a charm :)
My implementation:
class UniqueItemsByProperty(Constraint):
    keywords = {'unique_property': str}

    def _is_valid(self, value):
        if not value or not self.unique_property:
            return True
        seen = set()
        for item in value:
            if self.unique_property in item.keys():
                property = item[self.unique_property]
            print(property)
            if property in seen:
                self.fail = f"Property '{self.unique_property}' is not unique. Duplicate value '{property}'"
                return False
            seen.add(property)
        return True

    def _fail(self, value):
        return self.fail

...

validators = DefaultValidators.copy()
validators[List.tag].constraints.append(UniqueItemsByProperty)

Thanks for your help @mechie !


Typo: linx -> linux

I think it would be great to move words "linux" and "windows" to separate class-constants, e.g. OS_TYPE_LINUX and OS_TYPE_WINDOWS to prevent such typos in future.

Thanks.

@sergiy-petrov-vakoms
sound good. :smile:


Update scalafmt-core to 2.5.3

Updates org.scalameta:scalafmt-core from 2.5.2 to 2.5.3.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

labels: library-update, semver-patch

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@bd007df). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #110   +/-   ##
=========================================
  Coverage          ?   37.80%
=========================================
  Files             ?        3
  Lines             ?       82
  Branches          ?        0
=========================================
  Hits              ?       31
  Misses            ?       51
  Partials          ?        0

Impacted Files
Coverage Δ

...ptics/src/main/scala/io/circe/optics/package.scala
100.00% <0.00%> (ø)

...tics/src/main/scala/io/circe/optics/JsonPath.scala
33.76% <0.00%> (ø)

.../main/scala/io/circe/optics/JsonNumberOptics.scala
100.00% <0.00%> (ø)

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 bd007df...f52beed. Read the comment docs.


Updated to inject storage into controlllers

Allows users to use the storage functionality from controllers

@asmith60 hey there, thanks for this pull request. I appreciate it. However instead of updating the initializer to inject into everything that might need the storage service I'm going to update the documentation to use Ember.inject.service('storage') as the recommended way.


Rethink / Document the tantivy index format.

@fulmicoton since there is a pretty big rethink open, have you looked at how Vespa does things? I've been reading a lot about that engine with all of the ES circus going on.

@drusellers Yes I did have a look at Vespa. Mostly for the way they distribute things, but not their index format.

https://www.youtube.com/watch?v=SzZ_A9G6PMY
Checkout 41:00 - where they talk about read/write workloads
59:00 - a discussion about in place updates

It is further development of BMW algorithm. I just leave it here, think there is no any other better place for it. https://doi.org/10.1145/3077136.3080780
I'm gonna dive into in a while. I had a look but although cannot say yet if it is applicable to Tantivy or is just another one pure academical research.

Assigning to @fmassot , but he will just lead the effort.  People who want to join the discussion is welcome.


limit search thread pool size

Don't take all threads for search, reserve one thread for incoming requests. Otherwise there may not be enough CPU left to answer health checks
This behavior can be overruled by manually setting the search thread pool size via QW_SEARCH_THREAD_POOL_SIZE.
improve logging:
log actual indices that are being queried on leaf nodes instead of the patterns from the user request
reduce env var spam
Related to https://github.com/quickwit-oss/quickwit/issues/5305

I suggest closing this PR as we are not reproducing the health checks issue with heavy CPU queries

Yes, I'll move the logging improvements to a new PR


How to add music from Tiktok to video when upload

Hi. This is a great tool to automatically upload videos to tiktok. Can you provide more functionality to add music on Tiktok to videos in future sessions?

Adding a "sound" to TikTok from their web portal is currently not available.
Unless this feature is added, I can not really upload the video.
Sorry about this.

Feature is added now on TikTok web portal
Please update


errors with keras image_dim_ordering()

When I try to run the second cell in predict.ipynb I get:
/media/nobu/Ubuntu Backup1/1DeepLearning/2pixelation/age-gender-estimation/wide_resnet.py in __init__(self, image_size, depth, k)
     24         self._weight_init = "he_normal"
     25
---> 26         if K.backend.image_data_format() == "th":
     27             logging.debug("image_dim_ordering = 'th'")
     28             self._channel_axis = 1

AttributeError: module 'keras.backend' has no attribute 'image_dim_ordering'

Tried changing to `K.backend.imaged_data_format()` but error stays the same.

What's the Keras and TensorFlow versions?

I'm having the same issue. I think it's because I'm using Keras v2, where the function has been renamed to 'image_data_format()'. Not sure if @nyck33 made a spelling error (you wrote imaged instead of image) when commenting here or if he actually mistyped it.

I've fixed it by switching to keras v2.2.4 instead of 2.2.5.

@AlecVandenbroeck I'll have to check that but thanks!


https://xtremestream.co/

Checklist

[ ] I'm reporting a new site support request
[ ] I've verified that I'm running youtube-dl version 2021.12.17
[ ] I've checked that all provided URLs are alive and playable in a browser
[ ] I've checked that none of provided URLs violate any copyrights
[ ] I've searched the bugtracker for similar site support requests including closed ones

Example URLs

Single video: https://xtremestream.co/...
Playlist: https://xtremestream.co/...

Description

Unable to download streams from sites using this player...

please provide example URL

As the site isn't supported now, I've converted this into a Site Support request. Please complete the template (use the Edit function of the ... menu at the top right of your post) including the full example URLs


Prevent sudo timeout

Issue:
Possibility for sudo timeout during mysqldump
Symptoms:
Could cause delays in unattended/partially attended installations

Sudo timeout is a security precaution


RealmCollectionType

/cc @segiddins

What about realm, `invalidated, and KVC?

realm is optional on List
invalidated can't happen to a Results
KVC is defined on NSObject

it should be fine to define it as an optional on the protocol but make it non-optional on Results
it actually can, if the realm is invalidated
true, but RealmCollectionType isn't limited to NSObject types, as far as I can see

it should be fine to define it as an optional on the protocol but make it non-optional on Results

That doesn't work.

it actually can, if the realm is invalidated

We don't expose that at the moment. Create a new issue if you think we should in both Objective-C and Swift results.

true, but RealmCollectionType isn't limited to NSObject types, as far as I can see

Ugh, I can't make RealmCollectionType conform to NSObject or @objc because then it couldn't also conform to CollectionType. I could make a KeyValueEncodable protocol I suppose...

What about realm?
realm is optional on List

We definitely need some way to access the realm from a RealmCollectionType. Otherwise you can't do any transactions on it without having to communicate the realm off-band.
Would it be enough to make the realm variable optional in Results as well?

Would it be enough to make the realm variable optional in Results as well?

We could, but that would slightly weaken the interface on Results. I suppose that's a worthwhile tradeoff, though. I'll do that.

Done.

@segiddins addressed your feedback. I think the only things left to do are to test aggregate operations on List. Marking WIP until that's done.

@jpsim anything we can do to help get this in for the next release?

@tgoyne @segiddins this should now be ready for review.

I think some of the List tests are now redundant?

I think some of the List tests are now redundant?

Good point, fixed in last commit.

:+1:

:+1:


Zip all playlist videos.

I have an idea to enhance the playlist which is of zipping all the playlist video into an one zip file, which will be easier for usage.
I think this maybe the alluring feature of pytube - YouTube Space.🌍
😇

hi @RamkumarMrj somebody in the past actually created a pull request for this, and I'm not sure if this is the best idea for pytube. As I mentioned in this comment, I believe a better course of action would be to allow the user to compress the files however they choose, because different users may want to use zip, gzip, bzip, tar, etc. My solution at the time was to return a list of filepaths where the videos were saved, but the playlist functionality has actually been changed significantly in this PR. Playlists no longer have a download_all() function, and instead users are expected to iterate over Playlist.videos to save them, so I don't think this feature is relevant as of version 10.0.0

dont zip youtube playlist videos.


Images

Currently the scale of the images varies.
This has to be corrected.

align the image of the tab view. The source files have different sizes -> the scale differently. However a single image spot should stay the same (this is almost perfect in the second tab view) -> so the change has to happen in the files: https://github.com/blogosfair/How-does-the-inductive-bias-influence-the-generalization-capability-of-neural-networks/blob/main/assets/img/2022-12-01-how-does-the-inductive-bias-influence-the-generalization-capability-of-neural-networks/Figure2.PNG and https://github.com/blogosfair/How-does-the-inductive-bias-influence-the-generalization-capability-of-neural-networks/blob/main/assets/img/2022-12-01-how-does-the-inductive-bias-influence-the-generalization-capability-of-neural-networks/Figure3.PNG

I changed the file (Figure3) to match the figure2 and pushed the changes to my fork of the repo. Unfortunately, I can't see if that worked. I tried to do a pull request to the original repo and it said it had worked but I can't find the changes here and in my fork it still says that I am one commit ahead.

I tried to improve that. It is definitely better but not perfect...


Add Kromatika Token

https://etherscan.io/token/0x3af33bef05c2dcb3c7288b77fe1c8d2aeba4d789
https://coinmarketcap.com/currencies/kromatika/
https://info.uniswap.org/#/tokens/0x3af33bef05c2dcb3c7288b77fe1c8d2aeba4d789

Hello David, XRN. Would you like to approve KROM ?

hi @sNnickK

Address has to be in EIP-55 format
SVG logo < 10kB size is a requirement, please replace PNG one

@xrn Everything fixed

Hi @sNnickK - thanks for the updates

Can you remove png file from PR?
Your image has > 200kB and poor quality - regular SVG logo should have < 10kB and be a vector file. Could you try to fix it?

Hi @sNnickK - thanks for the updates

Can you remove png file from PR?
Your image has > 200kB and poor quality - regular SVG logo should have < 10kB and be a vector file. Could you try to fix it?

Check it now

@sNnickK this is still PNG image encoded in base64 not vector one - any chance for such a beautiful logo to be provided in vector SVG format?

@sNnickK this is still PNG image encoded in base64 not vector one - any chance for such a beautiful logo to be provided in vector SVG format?

We dont have .svg format only PNG. :/

@xrn Wait for our new rebranded logo, ill send it to you in svg format.

Hi @sNnickK - your SVG file has still > 150kB which is much to big and I am not able to see this file, like it is broken.

Here it is uploading now.

@xrn Logo updated, you can marge.

Looks great - I love your new logo. Thanks for your contribution


Adding workflow for creating documentation issues

Signed-off-by: Vacha Shah<EMAIL_ADDRESS>Description
Adding workflow to automatically create issues in documentation-website repo when a needs-documentation label is added to a PR.
Issues Resolved
#425
CheckList:
[X] Commits are signed per the DCO using --signoff
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #458 (2809556) into main (95fa134) will increase coverage by 0.05%.
The diff coverage is n/a.

:exclamation: Current head 2809556 differs from pull request most recent head a68ee74. Consider uploading reports for the commit a68ee74 to get more accurate results

@@             Coverage Diff              @@
##               main     #458      +/-   ##
============================================
+ Coverage     76.74%   76.80%   +0.05%
  Complexity      176      176
============================================
  Files           166      166
  Lines          8369     8369
  Branches       1231     1231
============================================
+ Hits           6423     6428       +5
+ Misses         1342     1334       -8
- Partials        604      607       +3

Impacted Files
Coverage Δ

...destinationmigration/DestinationConversionUtils.kt
68.88% <0.00%> (-2.23%)
:arrow_down:

...nationmigration/DestinationMigrationCoordinator.kt
61.70% <0.00%> (-2.13%)
:arrow_down:

...rg/opensearch/alerting/BucketLevelMonitorRunner.kt
76.15% <0.00%> (-0.77%)
:arrow_down:

...ain/kotlin/org/opensearch/alerting/AlertService.kt
77.99% <0.00%> (-0.48%)
:arrow_down:

...ing/model/destination/DestinationContextFactory.kt
75.00% <0.00%> (ø)

.../kotlin/org/opensearch/alerting/util/IndexUtils.kt
72.91% <0.00%> (+2.08%)
:arrow_up:

.../alerting/transport/TransportIndexMonitorAction.kt
71.32% <0.00%> (+3.14%)
:arrow_up:

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 95fa134...a68ee74. Read the comment docs.


Improve File Import performance

I noticed when working with large files Advantage scope tends to take seemly exponential time to import log (WPILOG) files.
This pull request is a very rough/quick and dirty start/working draft to speeding up the ingest process for advantage scope. I replaced the implementation for the LogField time and value arrays with a Sorted Set from js-sdsl. This PR has gotten basic functionality working but still has some issues with data viewing due to possibly incorrect implementations of clearBeforeTime, getRange, toSerialized and fromSerialized.
With this change I was able to get an improvement of 6x for smaller wpilog files (7mb) and 2x improvement in larger ones (40mb)
I ran the importer though a CPU profiler and it identified a few hot methods taking a large amount of the commutation time, putdata, createBlankField, and parseURCLR2.
I tried to limit any external api changes if at all possible, but would love input on what changes you would be open to, along with any insight on where to look deeper, as It still seems to slow down quite a bit as the files get larger. Maybe writing a parser in C++ would be a approach to getting much better performance?

Updates. Moved away from Red black trees and ended up finding it was much faster to load in the data into memory and then sort and then process the data when sorted.
Discovered a few things, the optimization that removes duplicate repeating values had a issue where it assumed that records where coming in sequentially so if you got a record out of sink it would produce an incorrect array. It was also the main cause for the n^2 time complexity when importing log files. Doing all this processing at the end allows the code to make the assumption that all keys are ordered and speeds up the process a lot.
This current commit adds improves the log importing but now no longer correctly updates the log progress bar as that functionality is within the workers and not within the logField or log class.  I think now the log field only only has valid values after a call to getSerilizable is a bit of a hack and might warrant some refracting, but I wanted to limit impact on other parts of the program for this POC.
This improved file loading times on a 70mb file from 4m to ~5-10 seconds.

I got bored and started playing around with this fork, and I really love the improved speed. My only complaint is that when LogField first calls sortAndProcess(), it can take a bit of time (like 5-10 seconds) while the progress bar remains at 100%. What I think could be useful is having some sort of message or progress bar saying "Processing Data".
I haven't been able to find any bugs yet (although the logs I was using don't have any weird anomalies), but I think this could be a big improvement to advantagescope.

I got bored and started playing around with this fork, and I really love the improved speed. My only complaint is that when LogField first calls sortAndProcess(), it can take a bit of time (like 5-10 seconds) while the progress bar remains at 100%. What I think could be useful is having some sort of message or progress bar saying "Processing Data".
I haven't been able to find any bugs yet (although the logs I was using don't have any weird anomalies), but I think this could be a big improvement to advantagescope.

There is currently a few issues with this pr, it breaks streaming functionality im pretty sure as pointed out by jwbonner. It also uses 2x the memory as the old code and cases out of memory errors when processing data over 1gb ish. where as the prevous code would allow that to run even if it did take hours. The memory fix could just be working around it and allocating more memory that does work but is a bad solution.
A possible solution for the streaming issue is to split the log class into two one for streaming and one for file reading, and utilize different optimizations for both. Overall the architecture is quite challenging to deal with

I tried to limit any external api changes if at all possible, but would love input on what changes you would be open to, along with any insight on where to look deeper, as It still seems to slow down quite a bit as the files get larger. Maybe writing a parser in C++ would be a approach to getting much better performance?

Writing a parser in C++ seems like somewhat of a pain to integrate with an Electron app to me. I think if performance becomes enough of a bottleneck a switch to something like Tauri allowing you to offload intensive tasks to a native backend would be a good change while allowing less changes than writing a full native implementation.

@jwbonner I am running into a bit of difficulty with the structure of how the log class is set up. I attempted to split it up into two modes one where the log can take advantage of not being sorted at all times and sorted at the end and then a always sorted mode for when the logs are streaming.
This approach runs into problems with the custom Schemas for Photonvision and URCL. Some of these schemas attempt to read data from the log file and these calls can not be made on a unsorted log. (Least not efficiently). Currently the way that custom schemas are loaded is on a by record basis and then all the generated fields (there can be many) generated by a single wpilog record all have to be sorted. A more efficient approach would be to load in all the raw data (while retaining the type information) and then after processing and sorting parse the custom schemas. This way no sorting needs to be done on the resulting log fields as they would already be sorted.
I do have a working workaround to it that could be polished a bit (right now the only issue I see is the loading bar not correctly accounting for the parse time for custom schemas. ) I think it might be worth in the future reworking the structure of how log files are ingested and read and attempt to better decouple the file parsing and log handling code. Also likely split up the streaming and file loading logic. Maybe trying to utilize a common interface to a intermediary file format such as MCAP or a new revision of wpilog in the future.

Sorry that I haven't been keeping up with this very closely, but I really appreciate the work you've been doing.
I think the structure of separating live sorting from normal log loading makes sense. As log files get larger, I think it will ultimately be necessary to avoid storing the entire set of log data in memory. The issue is that it requires a log format that can be efficiently indexed, which would have to be an intermediate format as long as we're using the RIO (maintaining writing efficiency is of upmost importance). Electron also makes file IO a bit tricky, though I wonder if IndexedDB could be an option? Regardless I think that's a separate project from this PR.
In regard to the Photon/URCL schemas, maybe the parses could be turned into separate objects with persistent state (which would need to be serialized along with the log). They wouldn't need to be able to read back from the log, but could instead just store data as required and generate records as they go (with the assumption that records fed into them aren't necessarily in timestamp order, which I think is reasonable since they need to work with NT anyway). Would that allow you to do a single pass of reading records from the file while processing schemas, then a sort at the end?
I believe we talked about it before, but I think it's likely that I'll transition the rest of the codebase to deal with duplicate values in the log data. I've been working on a redesign for most of the tabs to add more sophisticated options, and being able to explicitly show individual values, process them for stats, etc. is likely to be helpful as long as the performance is reasonable. I'm not 100% sure that it will work yet, but if so that should provide a bit of a performance improvement for data ingestion (memory is potentially still a concern).

I'd like to see how we can move forward with these changes. However, I haven't been seeing a significant difference in performance from the previous version when running with 6328's AdvantageKit logs. Do you have an example of the log files you've been testing with and seeing a significant difference in performance?

Yeah sorry I have not worked on this in a while so right now I broke something when I fixed URCL. I need to look over it when I get a bit of time. Right now this pr is still very much a draft.

I just merged #192, which reworks how the decoding system works (see the description for details). The changes are essentially orthogonal to this PR, so I think the changes here should still be integrated at some point. The difference in decode performance is probably still significant even though it's deferred from the initial load.


[ARCTIC-1539][AMS] Make the updateConfigInternal method always execute

Why are the changes needed?
Fix #1539
Brief change log

Make the updateConfigInternal method always execute
Optimize updateConfigInternal logic

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before making a pull request

Documentation

Does this pull request introduces a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

LGTM


Waiting for first pageview

Hi there,
I have the module installed, but Im getting this on the dashboard,
Do I need to add the snippet as well? Or is it something that the module should handle?
<script defer data-domain="alvarosaburido.dev" src="https://plausible.io/js/script.js"></script>

Thanks

Technically, you don't have to add any configuration when adding the module to your nuxt.config.ts.
Have you customized the module's option? And if so, could you please provide its configuration? Does your deployed Nuxt application's domain match the one set up in Plausible?
Otherwise, Plausible may take a while to process the first request. Could you check if it is still missing?

Hey, thanks for the quick response, here is my config
modules: [
    'nuxt-plausible',
    ...
],
plausible: {
    domain: 'https://alvarosaburido.dev',
 },

Yes, the domain is the same on the dashboard. Still the same state as the screenshot

You don't need to configure the domain module option, it will default to the window.location.hostname.
Apart from that, your configuration is malformed. domain should be alvarosaburido.dev (without protocol).

I removed the options, waited a little bit and then started fetching data. Thanks for the support, I'm gonna close this


Visual indication if a filter has been applied to the transaction log

Description
If a transaction filter is applied to the transaction log, then some transactions may be hidden. It is not obvious, as a user, whether a filter has been applied or not. This can only be discovered by navigating to the Filter page to check the current settings. A visual indicator (like an icon) would be an improvement, so that it becomes clear that the full list of transactions is not currently shown.
Steps to Reproduce
Change view -> Filters -> Filter out some transaction types that you have in your transaction log.
Versions
1.2.0

@jens-concordium I can see that the UI was changed regarding "change view", but did you implement anything with regards to this issue?

I don't think we made any changes for this yet. I think we discussed it without finding a solution, didn't we @soerenbf?


Ask about video from winning pitch for announcment

Next Week, 11/20 Week - Upload video & Winning Pitch description on website and share with NC City Camp.  Brent/Leo/Natalie connected on video, description and timeline

Completed.


Can't modify my gnome

Hello. The changes you've made to your desktop are really good and I wanted to get this modifications to my gnome as well. I'm new to Linux and I'm having some difficulties in applying this changes. I did the git clone on your repo and I also entered the commands you've mentioned on my terminal. But some changes didn't work out as it should. I wanted to get the same "widgets" as those that are in your top screen, like temperature, the day and the song being played and I also wanted to get that bar notification on your top right corner.

apologize for that, i forgot mention list installed my gnome extension list
make sure you have been installed gnome shell integration, here for chrome: https://chrome.google.com/webstore/detail/gnome-shell-integration/gphhapmejobijbbhgpjhcjognlahblep
then open: https://extensions.gnome.org/
to check installed extension can check here: https://extensions.gnome.org/local/
search those listed extension
make sure those extension been installed:
++ mandatory to install

ArcMenu :: https://extensions.gnome.org/extension/3628/arcmenu/ (left corner menu mod)
Aylur's Widgets :: https://extensions.gnome.org/extension/5338/aylurs-widgets/ (add additional menu on top mod)
Burn my windows :: https://extensions.gnome.org/extension/4679/burn-my-windows/ (window animation mod)
Compiz alike magic lamp effect :: https://extensions.gnome.org/extension/3740/compiz-alike-magic-lamp-effect/ (fix glitch minimize gnome effect)
Dash to dock :: https://extensions.gnome.org/extension/307/dash-to-dock/ (add dock)
One window wonderland :: https://extensions.gnome.org/extension/5696/one-window-wonderland/ (window gaps when maximize)
OpenWeather :: https://extensions.gnome.org/extension/750/openweather/ (weather indicator)
Top bar organizer :: https://extensions.gnome.org/extension/4356/top-bar-organizer/ (organize top menu)
Useless gaps :: https://extensions.gnome.org/extension/4684/useless-gaps/ (window gaps too)

++ optional

Caffeine :: https://extensions.gnome.org/extension/517/caffeine/ (pause screen timeout)
AATWS - Advanced Alt-Tab Window Switcher :: https://extensions.gnome.org/extension/4412/advanced-alttab-window-switcher/ (alt tab mod)
Text scaler :: https://extensions.gnome.org/extension/1018/text-scaler/ (font scaling + listed to top menu)

after installed those extension, configure those extension too

here how to configure those extension: https://drive.google.com/file/d/1e1tV4NE63jmZjHC-FoiWNWrepUv6QKeQ/view?usp=sharing

Thanks a lot for your time and help.


Replacement Password field in IE7 breaking form submission

In the setPlaceholder function, when the replacement field is created for the password field, the name attribute is removed from the replacement field. In IE7 at least, any input in the form that follows the replacement field is not submitted by the browser to the server. This is in fact causing the password field to not be submitted to the server at all because this replacement field was added to the DOM before the real password field.
$replacement
	.removeAttr('name')
	// We could just use the `.data(obj)` syntax here, but that wouldn’t work in pre-1.4.3 jQueries
	.data('placeholder-password', true)
	.data('placeholder-id', id)
	.bind('focus.placeholder', clearPlaceholder);

I've fixed this issue for myself by just modifying the name attribute to be something we would never use in a form versus removing the name. For example, I just appended ".replacement" to the name and that solves the issue.

http://mathiasbynens.be/demo/placeholder?search=&name=&email=&url=&tel=&=e.g.+hunter2&password=lol&message= seems to work just fine.
I guess you’re saying the &=e.g.+hunter2 part messes things up? What kind of system are you on?

I had a similar problem. My password field was submitted by its ID. The replacement input had the id, but the real input field did not. I 'fixed' this by editing the lines below. On non HTML5 friendly browsers these inputs have two identical IDs, but at least the form works now.
$input = $input.removeAttr('id').hide().prev().attr('id', id).show();
to this:
$input = $input.hide().prev().attr('id', id).show();

@joshbt Can you provide a test case?

Closing due to lack of a reduced test case.


BootOrUpdateReceiver not being called on Wear OS

Seems like it's not being called when updating the app, it's unclear currently whether it's called on boot.

Confirmed BOOT_COMPLETED is received correctly, MY_PACKAGE_REPLACED doesn't seem to be fired at all, and PACKAGE_REPLACED is blocked. We'll need some alternative to keep capabilities updated.


change find_next

find with an iterator from should check the range [ from, end() ).
from == end() is valid and returns end()

This works correctly in fields_view

boost.url calls it find not find_next


Add colors to doc tree links

Codecov Report

Merging #87 (3ecce23) into main (b3f374f) will decrease coverage by 0.32%.
The diff coverage is 84.61%.

@@             Coverage Diff             @@
##              main      #87      +/-   ##
===========================================
- Coverage   100.00%   99.67%   -0.33%
===========================================
  Files            7        7
  Lines          598      608      +10
===========================================
+ Hits           598      606       +8
- Misses           0        2       +2

Impacted Files
Coverage Δ

meilisearch_cli/_docs.py
97.22% <84.61%> (-2.78%)
:arrow_down:

meilisearch_cli/_helpers.py
100.00% <0.00%> (ø)

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 b3f374f...3ecce23. Read the comment docs.


Manual backport #1247 to 2.x

Description
This PR is for manual backport #1247 to 2.x
Issues Resolved
[List any issues this PR will resolve]
Check List

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

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

@SuZhou-Joe @Hailong-am The automatic backport was failed, could you help me review it? Thank you.

for curios, what does this used for?

This site is for finding release tests videos in jenkins.


imgui[docking-experimental,glfw-binding,opengl3-glbinding-binding] version 1.81#1 build failure

Host Environment

OS: Windows
Compiler: cl.exe (Visual Studio 2019)

To Reproduce
Steps to reproduce the behavior:
./vcpkg install imgui[docking-experimental,glfw-binding,opengl3-glbinding-binding]
Failure logs
vcpkg install imgui[docking-experimental,glfw-binding,opengl3-glbinding-binding]
Computing installation plan...
The following packages will be built and installed:
imgui[core,docking-experimental,glfw-binding,opengl3-glbinding-binding]:x86-windows -> 1.81#1
Detecting compiler hash for triplet x86-windows...
Could not locate cached archive: C:\Users\Runik\AppData\Local\vcpkg\archives\9c\9c737409ab92c71ab19f461e32c97ac14bc96603.zip
Starting package 1/1: imgui:x86-windows
Building package imgui[core,docking-experimental,glfw-binding,opengl3-glbinding-binding]:x86-windows...
-- Note: imgui only supports static library linkage. Building static library.
-- Using cached D:/Dev/Libraries/vcpkg/downloads/ocornut-imgui-682249396f02b8c21e5ff333ab4a1969c89387ad.tar.gz
-- Cleaning sources at D:/Dev/Libraries/vcpkg/buildtrees/imgui/src/69c89387ad-b99c0b72cb.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source D:/Dev/Libraries/vcpkg/downloads/ocornut-imgui-682249396f02b8c21e5ff333ab4a1969c89387ad.tar.gz
-- Using source at D:/Dev/Libraries/vcpkg/buildtrees/imgui/src/69c89387ad-b99c0b72cb.clean
-- Configuring x86-windows
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:105 (message):
Command failed: ninja -v
Working Directory: D:/Dev/Libraries/vcpkg/buildtrees/imgui/x86-windows-rel/vcpkg-parallel-configure
Error code: 1
See logs for more information:
D:\Dev\Libraries\vcpkg\buildtrees\imgui\config-x86-windows-out.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_configure_cmake.cmake:319 (vcpkg_execute_required_process)
ports/imgui/portfile.cmake:64 (vcpkg_configure_cmake)
scripts/ports.cmake:131 (include)
Error: Building package imgui:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: imgui:x86-windows
Vcpkg version: 2021-01-13-768d8f95c9e752603d2c5901c7a7c7fbdb08af35
Additionally, attach any relevant sections from the log files above.
config-x86-windows-out.log
Additional context
Tried to copy the missing imgui_tables.cpp to the directory using --editable switch, it fixes this one error but throws a bunch more ... so I reverted back to version 1.79#4 for now.

The problem is that only the 'master' version is updated, not the one with the feature 'docking-experimental'
This is still on an old build (1.79) where the file imgui_tables.cpp does not exist yet.

The problem is that only the 'master' version is updated, not the one with the feature 'docking-experimental'
This is still on an old build (1.79) where the file imgui_tables.cpp does not exist yet.

Ah right.  Didn't think to update that as I thought it might be stuck to a particular check-in on purpose.  I can update to the latest commit in that branch which does include the imgui_tables.cpp file if that seems like an appropriate fix?

Looks like #16268 will fix this.  Sorry for the breakage!

Thanks a lot to everyone involved 👍

Is this issue regressed on v1.89 for vcpkg or there's versioning mismatch/conflict between vulkan-binding and docking-experimental, since can't build vulkan-binding if only docking-experimental included as well? Same error log as above. However, install-x64-windows-dbg-out.log gives this:
C:\dev\vcpkg\buildtrees\imgui\src\725fcbd0e3-ce3e4747fa.clean\backends\imgui_impl_vulkan.cpp(1684): error C2065: 'VkRenderingAttachmentInfo': undeclared identifier
C:\dev\vcpkg\buildtrees\imgui\src\725fcbd0e3-ce3e4747fa.clean\backends\imgui_impl_vulkan.cpp(1684): error C2146: syntax error: missing ';' before identifier 'attachmentInfo'
C:\dev\vcpkg\buildtrees\imgui\src\725fcbd0e3-ce3e4747fa.clean\backends\imgui_impl_vulkan.cpp(1684): error C2065: 'attachmentInfo': undeclared identifier
C:\dev\vcpkg\buildtrees\imgui\src\725fcbd0e3-ce3e4747fa.clean\backends\imgui_impl_vulkan.cpp(1684): error C3079: an initializer list cannot be used as the right operand of this assignment operator
C:\dev\vcpkg\buildtrees\imgui\src\725fcbd0e3-ce3e4747fa.clean\backends\imgui_impl_vulkan.cpp(1685): error C2065: 'attachmentInfo': undeclared identifier
C:\dev\vcpkg\buildtrees\imgui\src\725fcbd0e3-ce3e4747fa.clean\backends\imgui_impl_vulkan.cpp(1686): error C2065: 'attachmentInfo': undeclared identifier
C:\dev\vcpkg\buildtrees\imgui\src\725fcbd0e3-ce3e4747fa.clean\backends\imgui_impl_vulkan.cpp(1687): error C2065: 'attachmentInfo': undeclared identifier
C:\dev\vcpkg\buildtrees\imgui\src\725fcbd0e3-ce3e4747fa.clean\backends\imgui_impl_vulkan.cpp(1688): error C2065: 'attachmentInfo': undeclared identifier
C:\dev\vcpkg\buildtrees\imgui\src\725fcbd0e3-ce3e4747fa.clean\backends\imgui_impl_vulkan.cpp(1689): error C2065: 'attachmentInfo': undeclared identifier
C:\dev\vcpkg\buildtrees\imgui\src\725fcbd0e3-ce3e4747fa.clean\backends\imgui_impl_vulkan.cpp(1690): error C2065: 'attachmentInfo': undeclared identifier
C:\dev\vcpkg\buildtrees\imgui\src\725fcbd0e3-ce3e4747fa.clean\backends\imgui_impl_vulkan.cpp(1691): error C2065: 'attachmentInfo': undeclared identifier


StripPrefix conflict with server.contextPath

i added stripPrefix = false under zull configuration, like this
zuul:
  prefix: /api
  stripPrefix: false
  ignored-services: "*"
  routes:
    customer:
      path: /customer/**
      stripPrefix: false

my requirement is i want to black all downstream calls
e.g
1)http://localhost:9999/api/customer/orders      {need to allow all calls and validate request using zuulfilter}
2)http://localhost:9999/customer/orders   need to block direct access
even i tried to add server.contextpath =/api/customer after that zuul got disabled

Please format code according to https://help.github.com/articles/creating-and-highlighting-code-blocks/
What do you mean you need to block all downstream calls?  Can you describe what is happening with the configuration above and what you are trying to achieve?

Hi ryanjbaxter,
Thanks for quick replay,
Here thing is i want to block all downstream calls without spring security or any other interceptors
my controller looks like this
@RestController
@RequestMapping("/customer")
public class customer{
RequestMapping("/orders")
public Respnse getOrders(....){
return x;
}
}
and Configured zuul in yml like below
zuul:
prefix: /api
stripPrefix: false
ignored-services: "*"
routes:
customer:
path: /shop/**
stripPrefix: false
not added any server.contextPath
so URL looks like
DownStream Direct Access http://localhost:9999/customer/orders
Zuul routing URL   http://localhost:9999/api/shop/customer/orders
even throughi added strip prefix ,still i can access downstream API directly,
here thing is i want to validate a token using zuul proxy,without security downstream calls diverted

Again please read the github documenation on how to format code in your issues, see the link in my previous comment.
I don't see how setting stripPrefix to false would cause you not to be able to access the downstream API.  zuul.stripPrefix will make sure /api is added to the request to the downstream service and zuul.routes.customer.stripPrefix will make sure /customers is added to the request to the downstream service.
Now if you are trying to validate some kind of security token before the request is forwarded on you might be able to do that in a ZuulFilter but that did not seem to be what you are asking originally so I am still confused on what the issue is.

Hi @ryanjbaxter below config used
Sample Zuul Config used

zuul:
prefix: /api
stripPrefix: false
ignored-services: "*"
routes:
  customer:
      path: /shop/**
      stripPrefix: false

Sample ZuulFilter for validating token

  public class SecurityFilter extends ZuulFilter {
        @Override
	public boolean shouldFilter() {
		return true;
	}
        @Override
	public Object run() {
                RequestContext context = RequestContext.getCurrentContext();
		HttpServletRequest request = context.getRequest();
                String token = request.getHeader("token");
                //-----validation of token -----
        }
        @Override
	public String filterType() {
		return "pre";
	}
        @Override
	public int filterOrder() {
		return 1;
	}
  }

Sample Restful API /controller

   @RestController
   @RequestMapping("/customer")
   public class customer{
         RequestMapping("/orders")
         public Respnse getOrders(....){
              return x;
         }
    }

Case1:
If i use http://localhost:9999/api/shop/customer/orders
then call is landed to zuulfilter and i can check token
Case2:
if i use http://localhost:9999/customer/orders
then call landed to controller rather then zuulfilter,so any one can access APIs without token
can i block Case2 with stripPrefix  using zuulfilter?

/customer/orders shouldn't be mapped in zuul. Why is it? Can you show /routes?

Hi
All zuul configuration and APIs in the same app,
I configured only one route to access customer orders i.e.  /api/shop

zuul:
prefix: /api
stripPrefix: false
ignored-services: "*"
routes:
   gateway-app:
      path: /shop/**
      stripPrefix: false

I just modified to gateway-app, so final URL looks like below
case 1
http://localhost:9999/api/shop/customer/order
case 2
http://localhost:9999/customer/order
case 2 can't block  ,even through if i use context-path no use

See @spencergibb question above, can you show us what the /routes endpoint produces from Zuul?

There is no mapping like /routes in Controller

/routes is added automatically for when you use @EnableZuulProxy http://cloud.spring.io/spring-cloud-static/Dalston.RELEASE/#_the_routes_endpoint

And you have spring-boot-starter-actuator on the classpath.

Hi I added spring actuator to class path and tried, below is the response
**Response for /routes**
   /api/shop/**
**Response for /mappings**
   /api/shop/**
   /customer/orders

The behavior seems to make sense to me.  Since /api/shop/** is route created by Zuul than the Zuul filter will intercept the request.  Since /customer/orders is not it won't and there fore you wont be able to validate the token.

is it possible to block direct access to downstream API i.e. /customer/orders using stripPrefix ?,
Temporarily i am using plain old Filter and validating token, rather then overriding ZuulFilter

I don't understand what it means to "block direct access to downstream API".

i mean Case 2 as mentioned above

How do you envision Zuul blocking access to that API?

Used different filter and working,Thanks

Used different filter and working,Thanks


Remove ActiveSupport dependency?

Here are the ActiveSupport dependencies in FactoryGirl:
In lib/factory_girl/evaluator.rb
1:require 'active_support/core_ext/hash/except'
2:require 'active_support/core_ext/class/attribute'

In lib/factory_girl/factory.rb
1:require 'active_support/core_ext/hash/keys'
2:require 'active_support/inflector'

In lib/factory_girl.rb
2:require 'active_support/core_ext/module/delegation'
3:require 'active_support/deprecation'
4:require 'active_support/notifications'

In spec/support/macros/deprecation.rb
1:require "active_support"

For Rails apps, using ActiveSupport is a no-brainer since it's already required.
For other Ruby apps, ActiveSupport is a pretty big dependency. (Granted, only for testing, but it's still there)
I can work to remove the dependency on ActiveSupport. Is that a worthwhile effort? Are you interested in a PR that does that?
Things like ActiveSupport::Notifications can still be supported, by just checking if they're defined at the top of the file.
(The concrete use-case for this is several of us in the Hanami Gitter chat would like to use FactoryGirl but don't want to add ActiveSupport, since it monkey-patches)
Thanks!

@cllns thanks for opening this issue.
For a while, we'd avoided including AS because of the same reasons you'd mentioned. We've obviously  become more lax on this, and I think you bring up a good point about assessing whether it's a necessary dependency.
Looking through the source for some of those files, I don't think it makes sense right now to introduce similar behavior while avoiding AS - that's a significant amount of code that has the risk of being outdated, buggy, or otherwise needs maintenance over time. The overhead cost of doing that versus leveraging an existing gem is very high.
Were there a community-adopted alternative that didn't rely on reopening classes, it makes sense to consider introducing it given it was actively-maintained and widely-used, across all the versions of Ruby we support.

Thanks for the quick and thoughtful response @joshuaclayton.
Hanami::Utils offers some of the functionality that's needed (like inflectors), but not all of it. Also, it only supports MRI 2.2, MRI 2.3 and JRuby.

Bummer that some is missing - for the 5.0 release of FG/FGR, I'm planning on dropping support for MRI Ruby below the latest 2.2.x; however, I think we should continue to push towards RBX compatibility, at least for FG 5.0.
Again, thanks for putting forth the investigative work on this!

Now that I'm looking into it, I think all the ActiveSupport features (except pub/sub Notifications  /Instrumentation) can be supported with a combination of Hanami::Utils and pure-Ruby.
Note: the 'require' links to where lines where the methods are called.
in lib/factory_girl/evaluator.rb

require 'active_support/core_ext/hash/except' is not in Hanami::Utils, but can be a one-liner in pure-Ruby: overrides.tap { |h| h.delete(:strategy) }
require 'active_support/core_ext/class/attribute can be replaced by Hanami::Utils::ClassAttribute

In lib/factory_girl/factory.rb

require 'active_support/core_ext/hash/keys' can be replaced by Hanami::Utils::Hash#symbolize_keys!
require 'active_support/inflector' can be replaced by Hanami::Utils::String.classify and Hanmi::Utils::Class.load!

In lib/factory_girl.rb

require 'active_support/core_ext/module/delegation' can be replaced by pure-Ruby's def_delegators in Forwardable
require 'active_support/deprecation' can be replaced by Hanami::Utils::Deprecation
require 'active_support/notifications' There's no alternative in Hanami::Utils but, if I'm not mistaken, this can be supported if the user has ActiveSupport loaded. If they don't have it, then it just won't work. Not sure if this is OK?

Historically, Hanami::Utils supported Rubinius until recently, in order to focus on MRI and JRuby.
I can reach out to see if they'd want to support Rubinius again, if you're interested in converting ActiveSupport to Hanami::Utils.

Ping @joshuaclayton (figured you didn't see my response since this Issue is closed)

@cllns thanks again for the work here. I've thought about this and believe it makes sense to continue with ActiveSupport.
The largest is due to developer adoption and use of AS itself. For both developers who are interested in contributing to FG and for those who're interested in understanding what the library does, seeing a library that's not AS would likely be confusing and increase cognitive load ("What is Hanami?", "Why is AS avoided?", "Does Hanami (or another tool) do something different/better than AS that I should further source dive?").
Secondarily, as most developers using this are doing so within the context of a Rails application, introducing another dependency doesn't seem worth it. I understand the size aspects of AS, but it's also a safe, trusted, widely adopted piece of software that is working as expected.
Thanks again for bringing this up for consideration; I truly appreciate it. I'm going to leave the issue closed.

Thanks @joshuaclayton, I completely understand.
Personally, I think adding another dependency for vast majority of  users is the more compelling argument. Making people question ActiveSupport and teaching them about the dangers of reopening classes isn't a bad thing, in my opinion :)


Proposal: truncate files before uploading from liveview

In liveview, even when onchange is not set for an input type="file", every chosen file is serialised and sent as one big WS message. In case of choosing a large file, even accidentally, this leads to JS errors (for hundreds-of-megabytes-big files), WS disconnections (for tens of megabytes), or UI freezes (for couples of megabytes).
Eventually and ideally, file uploads would be streamed piece by piece. However, probably even in that case there should be a modicum of control over those uploads, e.g. a limitation on the size of the uploaded file, otherwise one can see a potential for DDoS.
This PR is very modest, but would go a long way toward helping me to overcome this limitation until a better solution is devised. I think the approach I take here would be quite useful even when the streaming uploads are done.
I'm really not sure about the exact API and naming, and whether it makes sense to limit this attribute to liveview-only.
The PR adds an attribute called _liveview_truncate_at, with a default of 1 mb. The files chosen in the file-input are truncated to this size prior to sending them along the WS. I'd also propose to send and expose the original file sizes (the client side of this is also implemented in the PR), to make sure that the truncated files aren't accidentally taken to be complete. However, I wanted to discuss this before I start reworking major APIs across the different engines.
(A simple way to expose the size, as commented out in the PR, is through exposing the sizes as a method. A type-safer method could perhaps return a file as an enum: Complete(Contents), Truncated(usize, Contents), Streamed(impl Stream, Contents)...)

DDoS needs to be handled on the server. You can't trust a client check to prevent uploading a large file. Adding support for streaming file uploads to dioxus-liveview would be great in the future, but I don't think exposing an attribute that limits the size of uploads is a good solution currently:

It makes it difficult to give feedback to the user if the upload is too big
It should be depreciated once streaming uploads are supported
Running into the default 1mb limit would be very confusing because there is no logging


Mapping of Collections without Default Constructor not working

Recently we have shifted from Automapper to Mapster and facing another issue which was working fine with Automapper.
We have several classes with the following structure:
public class ErpProductPropertyTranslationDictionary
	{
		public ErpProductPropertyTranslationDictionary()
		{
		}
		public System.Collections.Generic.ICollection<ErpProductPropertyTranslation> ProductPropertiesAsList	{ get; set; }//;
		public System.Collections.Generic.IEqualityComparer<string> Comparer	{ get; set; }//;
		public int Count	{ get; set; }//;
		public System.Collections.Generic.Dictionary<string, ErpProductPropertyDictionary>.KeyCollection Keys	{ get; set; }//;
		public System.Collections.Generic.Dictionary<string, ErpProductPropertyDictionary>.ValueCollection Values	{ get; set; }//;
		public ErpProductPropertyDictionary Item	{ get; set; }//;
	}

When mapping _mapster.Map<ErpProductPropertyTranslationDictionary,ErpProductPropertyTranslationDictionary>()
It gives exception for Comparer, Keys and Values properties as they dont have default constructors. This was working fine with Automapper. Is there any way we can make this work?
Your help is highly appreciated. Thank you

Can you show your ErpProductPropertyDictionary constructors?

public ErpProductPropertyDictionary()
		{
		}

Do you need deep copy? By default Mapster try to deep copy all types. Otherwise you can use ShallowCopyForSameType


Local variable with same name as global function

Elan allows this code, but run-time error says:
TypeError: f is not a function
nor is it fixed by writing:

The easy solution is to not allow local variables to shadow global functions or procedures. (This though is different from constants as local variables can shadow constants).
The harder solution (because it involves a big generated code change) is to move all functions and procedures into the 'global' namespace with constants. You could then use 'global' to deconflict.


kinit: Preauthentication failed while getting initial credentials

Hi,
after running the command, the register stop at renew script
+ [[ '' =~ ^[^#]*= ]]
+ read line
++ hostname
+ host=m1.freeipa.sxl.io
+ kinit -k -t /etc/lets-encrypt.keytab lets-encrypt/master.freeipa.example.com
kinit: Preauthentication failed while getting initial credentials

Any Help?

@antevens Hi, I left this issue behind for now. If I encounter it again, I will repoen the issue


Create new shell2http binaries to solve CVE-2016-2183

Current version of shell2http binaries are build with old go version which cause our security tooling to report CVE-2016-2183.
This seems to be solved with a new go version: link
Could you please create a new version of shell2http binaries by triggering github-action to publish the compiled code (like was done with last version link )
Thanks for your help

@jlvrhee i've created the new release - https://github.com/msoap/shell2http/releases/tag/v1.14.2

@msoap Thanks for the quick help. We will install the new version and test to see if the issue is resolved

@msoap Unfortunately The new release didn't solve our issue. After some investigation we found that improvents that have been done in the new go versions (crypto/tls package) were related to the client side but not to server side.
Therefor code changes are needed to really solve this issue. Could you please:

Disable TLS1.0 and TLS1.1 or atleast make this configurable, as these are seen as insecure.
Disable old ciphers that support 64 bit blocks. Best would be to only only modern block cipher with 128-bit blocks such as AES. Or atleast make this configurable. See for more info https://sweet32.info/


accuary on UCF101 or Kinetics

Hello! I use your code to train UCF101 dataset, but loss does not go down.Can you show me ｈ-yperparameters when you train the model?Does this code training model achieve the effect of the paper?

Hi @Kathrine94,
Sorry for the late response.
I successfully trained (or more precisely, finetuned) this network on other datasets  (not UCF101 though).
I used cross-entropy loss SGD  with:

lr 0.001
momentum 0.9
Hope this helps !
Best,

Yana


Can't execute curl-based to install turso

Downloading Linux_x86_64 ...
######################################################################## 100.0%
Installing to /home/.turso
gzip: stdin: unexpected end of file
tar: Child returned status 1
tar: Error is not recoverable: exiting now

I'm using ubuntu 20.04 please help

same here. I've tried the shell script, through pacman, and by downloading the tagged zip and manually installing - none work. error: 'turso-cli-0.87.4.zip': invalid or corrupted package

If this is still an issue please comment so I can re-open but I just tried on two other machines (one running Linux) and it seems to work fine.


Formating

Description
There was twice the page title in the page and the title of v3.4.1 was ## instead of ###.
Motivation and Context
Not applicable

Breaking Changes
No

How Has This Been Tested?
Not applicable

[ ] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[ ] I have tested and validated these changes using one or more of the provided examples/* projects

[ ] I have executed pre-commit run -a on my pull request

It is automatically generated, so there is no need to touch this file.


TensorRT  Support Deconvolution?

Dose TensorRT support  Deconvolution layer in Caffe?

Sorry, but this is not the right place to ask TensorRT questions.
Maybe @cgottbrath-nvidia can point you on the right direction.


Adding LDA and  QDA from sklearn + registry  update

see https://github.com/alan-turing-institute/MLJModels.jl/issues/57#issuecomment-548489874

minor fix needed to import Distributions instead of  using Distributions, will do this asap


cleanup & refactor

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 13 (0.0%)  changed or added relevant lines in 2 files are covered.
9 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.08%) to 26.675%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/cli/cc/iostream/iostream.go
0
6
0.0%

pkg/cli/cc/cc.go
0
7
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/cli/cc/iostream/iostream.go
3
0.0%

pkg/cli/cc/ports.go
6
76.27%

Totals

Change from base Build<PHONE_NUMBER>:
-0.08%

Covered Lines:
1210

Relevant Lines:
4536

💛  - Coveralls


SetBrightness Byte for revB

Hi,
Could you convert the input value of SetBrightness(byte) to int like the class of revA? Will help a lot. For revB we just need to convert the int 0-100 to 0-255 like this:
var val = ((float)intValue / 100) * 255;
Thanks!

Because RevB supports different brightness ranges for different versions, the values are directly specified without conversion.
I am also considering an abstraction that would treat RevA and RevB the same way, but at this stage, please keep the conversion on the caller side.

Yeah, i am using a dynamic object to store the rev classes inside my class helper as the methods are equals, the only difference is SetBrightness, but no problem for now!
private dynamic display;

        public DisplayTools(string rev, string port)
        {
            _rev = rev;
            if (rev == "a")
                display = new TuringSmartScreenRevisionA(port);
            else
                display = new TuringSmartScreenRevisionB(port);
        }

I added new abstraction API in v2.0.0, please try it.
The Brightness input for the new API is 0-100.
Please refer to README.md and Example project that have added for usage.


[Feature] Redesign the Product Card

Description
We already have a ModelViewer component having a 3D viewer and QR code ( for desktop ).
Just convert it to a well-designed card having both a 3D viewer and OR code combined.
It should be responsive for all viewports and the component should be reusable.
Screenshots
Just go through some similar e-commerce platforms to get an idea of how the UI should look.
Checklist

[X] I have checked the existing issues
[X] I have read the [Contributing Guidelines]
[ ] I am willing to work on this issue (optional)
[ ] I am a GSSoC'23 contributor

Greetings! I would like to work on this issue. Please assign it to me.

Of course, Go ahead! Apply your creativity to the next level.
Also, try to complete it within next 5 days. You're assigned.

Any updates or issues @tanisha2202 ? Please try to complete your changes by this weekend.
After that, it will be open to other contributors.

Any updates or issues @shraddha761 ?
Please try to complete the requested changes and update your PR within the next 3 days.
After that, This issue will be open to other interested contributors.

@ShwetKhatri2001 I'll text you at discord please check your dm

Let's give other contributors a chance to work on this issue. This issue is open to be assigned now.
I've updated the description to clear the objectives.

@ShwetKhatri2001 Please assign this to me under gssoc'23. I am ready to work on this issue in time

@akanksha984 You've shown interest to work on multiple issues, but I can only assign you 2 issues at a time.
There is one already assigned and one is left.
Let me know which issue you want to pick up next from the issues where you've commented.
This issue has been our high priority. So, It'd be good if you can pick this. But the final decision is yours.

Instead of the shoe image above we will have this whole 3D viewer ( nothing to be removed ),

You can add other things required for product details or actions. Apply your creativity freely, I've just provided an example.

Ok I will try designing it

Please check it @ShwetKhatri2001

The colors used here can be better.
Make the styles like this, ( I just changed some CSS in browser's developer-tools and got this )

Complete these and you will be done after that,

Make the card smaller ( around 350 in width ) since we will have multiple cards on this page later
Stick to 3-4 main colors. ( #eee, #ddd,rgba(235,88,51,0.75 )  and #aaa you may use )
Remove the URL as mentioned in this issue's description
Show those three details above the QR code row in this order - Product Name, Rating, and Price.

Same card color I created earlier so that it looks formal but then I changed it to bright colors.
Ok I will do the required changes and create the pr for it @ShwetKhatri2001

@ShwetKhatri2001 looks well??

please check this @ShwetKhatri2001


Fix bugs with spatial attack

LGTM once pep8 passes and small comments were addressed

This will probably need a rebase after the switch to 2 spaces.

Closing in favor of #623


moveit: 0.8.6-0 in 'jade/distribution.yaml' [bloom]

Increasing version of package(s) in repository moveit to 0.8.6-0:

upstream repository: https://github.com/ros-planning/moveit.git
release repository: https://github.com/ros-gbp/moveit-release.git
distro file: jade/distribution.yaml
bloom version: 0.5.24
previous version for package: 0.8.5-0

moveit
* [fix][moveit_ros_visualization] correct "simplify widget handling" #452 <https://github.com/ros-planning/moveit/pull/452>
* [enhancement][MoveGroup] Add getLinkNames function (#440 <https://github.com/ros-planning/moveit/issues/440>)
* [doc][moveit_commander] added description for set_start_state (#447 <https://github.com/ros-planning/moveit/issues/447>)
* Contributors: Yannick Jonetzko, henhenhen, Ravi Prakash Joshi

moveit_commander
* [doc][moveit_commander] added description for set_start_state (#447 <https://github.com/ros-planning/moveit/issues/447>)
* Contributors: Ravi Prakash Joshi

moveit_controller_manager_example

No changes

moveit_core

No changes

moveit_fake_controller_manager

No changes

moveit_kinematics

No changes

moveit_planners

No changes

moveit_planners_ompl

No changes

moveit_plugins

No changes

moveit_ros

No changes

moveit_ros_benchmarks

No changes

moveit_ros_benchmarks_gui

No changes

moveit_ros_control_interface

No changes

moveit_ros_manipulation

No changes

moveit_ros_move_group

No changes

moveit_ros_perception

No changes

moveit_ros_planning

No changes

moveit_ros_planning_interface
* [enhancement][MoveGroup] Add getLinkNames function (#440 <https://github.com/ros-planning/moveit/issues/440>)
* Contributors: henhenhen

moveit_ros_robot_interaction

No changes

moveit_ros_visualization
* [fix][moveit_ros_visualization] correct "simplify widget handling" #452 <https://github.com/ros-planning/moveit/pull/452>
* Contributors: Yannick Jonetzko

moveit_ros_warehouse

No changes

moveit_runtime

No changes

moveit_setup_assistant

No changes

moveit_simple_controller_manager

No changes

Waiting for another sync...


any timeline to CSS Selector ?

3q

Hello @zlzforever ,
We don't have any timeline for this feature.
We recommend you to try using some third party library for HAP that already supports it:

https://www.nuget.org/packages/HtmlAgilityPack.CssSelectors/
https://www.nuget.org/packages/HtmlAgilityPack.CssSelectors.NetCore/

Let me know if that answer correctly to your question.
Best Regards,
Jonathan

Thank you very much.


Configure CPU-allocation, the run.googleapis.com/cpu-throttling boolean in Cloud Run

Hi, is it possible to somehow configure the option that Cloud Run has for CPU allocation?
It's like this in the Run YAML:
apiVersion: serving.knative.dev/v1
kind: Service
metadata:
  name: SERVICE
spec:
  template:
    metadata:
      annotations:
        run.googleapis.com/cpu-throttling: 'BOOLEAN'
      name: REVISION

where the bool is:

BOOLEAN with true to set CPU allocation only during request processing, or false to set CPU to always allocated.

That's from https://cloud.google.com/run/docs/configuring/cpu-allocation#yaml
The Python lib here has many related options, such as cpu min-instances, and a way to set labels, but that annotations part is not mentioned in the docs, https://firebase.google.com/docs/reference/functions/2nd-gen/python/firebase_functions.options#firebase_functions.options.EventHandlerOptions

Oh I think it's actually possible to set preserve_external_changes to true and then edit the autodeployed YAML manually, we'll go with this now.

What sort of configuration options could be added here? Could you give me an example of a use case where you'd need to edit it?


Rename DetectChangePoint reference to DetectSpike

In the Spike Detection part
"Use the IidSpikeEstimator to train the model for spike detection. Add it to the DetectChangepoint() method with the following code:" --> DetectChangepoint() must be DetectSpike()
Fixes #13433

Thanks, @luisquintanilla!


fix for tour button + content

@agamaloni Please re-open as pull request to dev rather than master.


Crash when downloading a file using urllib.urlretrieve fails

This issue was originally created at: 2011-08-17 02:05:54.
This issue was reported by: gerard_.
gerard_ said at 2011-08-17 02:05:54

I noticed the monkey patch for this in my build scripts and thought you might
want a bug report for this. I've got the following code in my SConstruct:
def downloadOnlineHelpXML(target, source, env):
urllib.urlretrieve("http://.../onlinehelp.xml", str(target[0]))
env.Command(basedir+'onlinehelp.xml', '', downloadOnlineHelpXML)
When downloading this file fails a builderror is thrown that causes a crash when
the str function does not return a string (weird, right?). The following
monkey patch prevents this:
old_str = SCons.Errors.BuildError.str
def new_str(self):
return str(old_str(self))
SCons.Errors.BuildError.str = new_str
Is this even a bug in SCons? If you need a reproduction please let me know.

Fixed in (at least) 3.0.1 (probably earlier)


problem with context menus

I'm having trouble with blur in context menus:
https://imgur.com/TOLjBjm
how can I solve?

I can't test blur for context menus at the moment as BSPWM doesn't use context menus. It has something to do with your compositor for sure. What picom version/fork are you using?

I'm using this version:

Great! Glad you were able to solve it. Closing this issue now


backport: bitcoin#19522, #19809, #20993, #21075, #21126, #21138, #21221, #21354, #21542

Issue being fixed or feature implemented
Regular backports from bitcoin v22 and related fixes
What was done?
Follow-up fixes for #5055, bitcoin#21366 and bitcoin#19809
Backports:

bitcoin/bitcoin#21075
bitcoin/bitcoin#21126
bitcoin/bitcoin#21138
bitcoin/bitcoin#19522
bitcoin/bitcoin#21221
bitcoin/bitcoin#21354
bitcoin/bitcoin#21542
bitcoin/bitcoin#19809
bitcoin/bitcoin#20993

How Has This Been Tested?
Run unit/functional tests
Breaking Changes
N/A
Notice, that function name is included now by default to logs with -logfunctionnames and many logs have function name twice now such as:
 node0 2024-04-06T20:13:56.564123Z (mocktime: 2014-12-04T17:15:38Z) [httpworker.3] [masternode/sync.cpp:331] [NotifyHeaderTip] CMasternodeSync::NotifyHeaderTip -- pindexNew->nHeight: 5 fInitialDownload=0

For further development need to take it in account and do not use more direct calls __func__ from code as well as reduce usages in codebase.
Checklist:

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

@knst knst force-pushed the bp-v22-p5 branch from 32ec395 to d4dcdf0

rebased on the top of develop after merging master

LGTM but needs rebase to fix ff-only merge check


Add new testnet network id

Since testnet data is incompatible now becasue of the modification of native contract, we could create a new testnet N3T5. Wait for a better solution on native contract update.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 14.936%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
107

Relevant Lines:
694

💛  - Coveralls

Just curious, does the id has any special meaning? or its just randomly picked?

Just curious, does the id has any special meaning? or its just randomly picked?

Just curious, does the id has any special meaning? or its just randomly picked?

BigInteger -> Hex -> UTF8 string -> N3T5

BigInteger -> Hex -> UTF8 string -> N3T5
LOOOOOOL, Impressive~~~~~way more complex than i thought


Support boolean and number arrays

Background
Currently, minimist-options supports type: array. However, the values of the array are always parsed as 'string'. yargs/yargs-parser (which are minimist backwards compatible) have support for coercing values 'number' and 'boolean' as well (see opts.array under yargs-parser API).
Proposal
Therefore, I would like to see minimist-options supporting these types too.
Considerations
I have considered the following options for how that could look:

Add another property elementType: 'number'|'boolean' (potentially arrayElementType to tie it more to array). This would only be allowed when having type: array.
Extend type with support for ['number'] and ['boolean'].
Extend type with support for 'number-array' and 'boolean-array' (camelCasing possible if preferred).
Extend type with support for numbers and booleans. Note: plural.
Extend type with support for { type: 'array', elementType: 'number|boolean' }.

Note: Additionally, 'string' would of course be a valid element type.
#1 is explicit, but verbose, introduces an extra property which would only be valid under certain conditions (which is always a little "blah" and adds complexity).
#2 is a little magical, but I like how terse it is and find it quite expressive/intuitive when you look at it. ”Aha, it’s an array of (number|boolean)s”.
#3 would introduce magical (made-up) strings. Not very fond of.
#4 is probably the simplest of them all. My only concern is: is it distinct enough or too easy to mix up?
#5 Using an object is always neat, but having to specify type: 'array' again feels weird as it would be the only valid value, but leaving it out would be odd too.
I'm leaning towards #2 or #4.
Any other suggestions?
Backwards compatibility
To be backwards compatible and not break existing code bases, I suggest we still keep type: array and its behavior intact.
@vadimdemedes What say you?

@vadimdemedes Went ahead and implemented it. Hope you get around to have a look sometime soon. Would be much appreciated to have this merged. Thanks!


Finish Javadoc

Need to finish Javadoc before MapDB 1.0 is released.

Unsubscribe
Sent from my iPhone

On Jul 4, 2015, at 1:21 AM, Jan Kotek<EMAIL_ADDRESS>wrote:
Closed #204.
—
Reply to this email directly or view it on GitHub.


Update default workspace mapping for fairfax

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/bragi92/azure-cli-extensions.git@kaveesh/updateForFFSupport#subdirectory=src/$EXT&egg=$EXT"


Implement holding and carrying objects (not in inventory)

Intended for heavy objects, like big boxes, huge ammunition shells etc. Object displayed in front of the camera. While holding it, other interactions (inventory, context inventory) are not available.
Note: this will require basic physics (object can be dropped).
Alternative: very heavy object would take all inventory slots.

This is partially implemented already ("item holding") except the items can only be placed in certain areas. The feature might still be extended with some basic physics and the possibility to drop the item anywhere.


nnapi delegate and cpu output with my QAT int8 model got Inconsistent result

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

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):16.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):pip
TensorFlow version (use command below):2.3.0
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
QAT my keras model，got tflite model，when I test with CPU on my Android phone（R version），I got right result，after I set useNNapi（true），I got a wrong  result。
Describe the expected behavior
got the same result with CPU or nnapi
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.
nnapi log:
2020-12-04 12:09:34.687 30684-30714/org.tensorflow.lite.examples.detection D/score:: 0.011764707
2020-12-04 12:09:34.689 30684-30714/org.tensorflow.lite.examples.detection D/score:: 0.003921569
2020-12-04 12:09:34.690 30684-30714/org.tensorflow.lite.examples.detection D/score:: 0.007843138
2020-12-04 12:09:34.690 30684-30714/org.tensorflow.lite.examples.detection D/score:: 0.011764707
2020-12-04 12:09:34.692 30684-30714/org.tensorflow.lite.examples.detection D/score:: 0.007843138
2020-12-04 12:09:34.693 30684-30714/org.tensorflow.lite.examples.detection D/score:: 0.011764707
2020-12-04 12:09:34.693 30684-30714/org.tensorflow.lite.examples.detection D/score:: 0.003921569
2020-12-04 12:09:34.693 30684-30714/org.tensorflow.lite.examples.detection D/score:: 0.007843138
cpu log:
2020-12-04 12:19:04.204 31252-31925/org.tensorflow.lite.examples.detection D/score:: 0.78125
2020-12-04 12:19:04.204 31252-31925/org.tensorflow.lite.examples.detection D/score:: 0.78125
2020-12-04 12:19:04.391 31252-31925/org.tensorflow.lite.examples.detection D/score:: 0.7734375
2020-12-04 12:19:04.391 31252-31925/org.tensorflow.lite.examples.detection D/score:: 0.7734375
2020-12-04 12:19:04.586 31252-31925/org.tensorflow.lite.examples.detection D/score:: 0.80859375
2020-12-04 12:19:04.587 31252-31925/org.tensorflow.lite.examples.detection D/score:: 0.80859375
2020-12-04 12:19:04.749 31252-31925/org.tensorflow.lite.examples.detection D/score:: 0.81640625
2020-12-04 12:19:04.749 31252-31925/org.tensorflow.lite.examples.detection D/score:: 0.81640625
2020-12-04 12:19:04.895 31252-31925/org.tensorflow.lite.examples.detection D/score:: 0.765625
when I use nnapi the score is always low

@sunzhe09
Please share simple indented stand alone code for us to replicate, or a colab gist with the error reported.

@Saduf2019  is there any comanddline tools，I can test my model output，with nnapi and cpu ？if so I can post my test code for you

@Saduf2019 Can we loop some one in NNAPI team here?
We need to know what op and what input data cause the issue.

@thaink @Saduf2019 I have tested with the latested  tf-nightly aar the nnapi result seems to be the same,thanks

@sunzhe09 It looks like you are using an older Version of Tensorflow. Many bugs have been fixed in the latest version. Could you please execute your code using Latest  stable Version of TF 2.5 and let us know if the issue still persists? Thanks!