Feature request: reporting error hooks

Hi all,
Recently was dealing with an internal error, and also noticed that sometimes I'd get 500 responses from the server (async). Was looking for hooks so I could track / monitor those situations, but couldn't find any. End up adding a code like this:
Rollbar::Notifier.prepend(Module.new do
   def handle_response(response)
      if response.code != '200'
        # tracking
      end
      super
    end

    def report_internal_error(exception)
      # tracking
      super
    end
end)

Would be really nice if there were hooks for async responses and internal errors when configuring Rollbar.

Thanks for the request!

Documentation on usage can now found here: https://docs.rollbar.com/v1.0.0/docs/gem-configuration-reference

@ArturMoczulski thanks for the change, appreciate it!


Postgresql without timezone : sails change it

Sails version: 1.1.0
Node version: 9.8.0
NPM version: 5.7.1
DB adapter name: sails-postgresql
DB adapter version: 1.0.2
Operating system:Windows

Hi,
I spent a lot of time to search any solution.
I think there is a "bug" or someting missing in configuration.
I store data in a timestamp without time zone, like this : 2019-03-15 00:00:00
Everything is okay in DB.
In sails model, the sendNativeQuery method returns 2019-03-14T23:00:00.000Z.
I tried to put "process.env.TZ = 'UTC';" in bootstrap.js file, it's not working. Same for timezone parameter in datastore.js with sails-postgresql adapter.
So, I think timezone is not implemented in sails, that's a problem.
Many thanks,

Hi @nlgi,
Maybe you could try setting a custom: timestamp to whatever attributes you want to change the formatting for.
https://sailsjs.com/documentation/concepts/models-and-orm/validations#?custom-validation-rules

Hi @raqem,
Many thanks for your answer.
I tried, but "custom" attribute is firing in update / insert command, not for select (in any case sendNativeQuery).
I found a workaround : "SELECT mydate::text FROM mytable". Sails doesn't parse the column as a date because it's now a string. And everything is ok, even for insert / update with ORM:
attributes: {         mydate: { type: 'ref', columnType: 'date'      },
It's working, but I think it's not a good practice, sails has to include timezone config.
Many thanks,


DPL-917 Submit plate of PBMC pools for cDNA prep

User story
As an SSR, I would like to be able to submit a plate of pooled PBMCs for cDNA prep, as part of the scRNA Core pipeline.
Who are the primary contacts for this story
Katy, Lesley, Abby
Who is the nominated tester for UAT
Lesley, Abby, SSR
Acceptance criteria
To be considered successful the solution must allow:

[x] PBMC pool plates imported from faculty (output of DPL-823) can be submitted to the cDNA prep part of the scRNA Core pipeline (request type limber_scrna_core_cDNA_prep)
[x] Update / create new Integration Suite test to include this submission.
[x] Update documentation, to say relevant submission templates, library types etc.

References
This story has a non-blocking relationship with:

https://github.com/sanger/sequencescape/issues/3857
https://github.com/sanger/sequencescape/issues/3900

Additional context
This is the second entrypoint into the pipeline - these plates are coming straight from faculty, rather than having being taken through the scRNA Core Cell Extraction pipeline in SeqOps.

Gitlab branch for 917 also made


masses, colors

remove "junk" entries from atomic_masses
replace AtomicMasses() with direct Dict definition
replace CPKColors() with direct Dict definition

All default atom properties are now loaded the same way (like covalent radii)

Codecov Report

Merging #76 (b4d4aa3) into master (849f6f3) will decrease coverage by 0.23%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #76      +/-   ##
==========================================
- Coverage   93.29%   93.05%   -0.24%
==========================================
  Files          10        8       -2
  Lines         924      893      -31
==========================================
- Hits          862      831      -31
  Misses         62       62

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 849f6f3...b4d4aa3. Read the comment docs.


Fixes #3682 - Use portAssignments for readiness checks

WARNING: this PR targets gk/add-port-assignments, DO NOT merge.

Nice job! Only nit-picks, would also merge as-is.

Superseded by #3712


Chat: clarify tooltip for remote search context

Small UI change.
Update the tooltip for file links included via remote search to explicitly mention "search (remote)" for unified context to improve clarity and make it easier to identify the search, as currently, we are showing it as via Search which overlaps with the search results from symf.

symf context
unified context

Test plan

Before

After

👍
(remote) seems redundant given the only enterprise search is remote search, but seems fine.

That's true! Maybe it'd be helpful in the future if we decide to support both 😆
For now I think this would still be helpful for supports and users to identity if the context sources are correct imo!


bear table should have a foreign key to refer the form

This is a following task of PR #71

Landed https://github.com/OH-SHOWN/ohshown-backend/pull/83 . Close issue.


String fortification warnings

Description
I got three string fortification warnings, hinting at incorrect string handling.
[195/337] /usr/lib/ccache/bin/x86_64-pc-linux-gnu-g++ -DRENDERDOC_EXPORTS -DRENDERDOC_LIB_SUFFIX=64 -DRENDERDOC_PLATFORM_LINUX -DRENDERDOC_SUPPORT_EGL -DRENDERDOC_SUPPORT_GL -DRENDERDOC_SUPPORT_GLES -DRENDERDOC_SUPPORT_VULKAN -DRENDERDOC_WINDOWING_XCB -DRENDERDOC_WINDOWING_XLIB -DVK_USE_PLATFORM_WIN32_KHR -DVK_USE_PLATFORM_XCB_KHR -DVK_USE_PLATFORM_XLIB_KHR -DVK_USE_PLATFORM_XLIB_XRANDR_EXT -D_RELEASE -I/tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc -I/tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/3rdparty  -DNDEBUG -O3 -pipe -march=native -std=c++11 -fno-strict-aliasing -fvisibility=hidden -fvisibility-inlines-hidden -Wall -Wextra -Wno-unused-variable -Wno-unused-parameter -Wno-unused-result -Wno-type-limits -Wno-missing-field-initializers -Wno-unknown-pragmas -Wno-reorder -Wno-unused-but-set-variable -Wno-maybe-uninitialized -Wno-class-memaccess -Wimplicit-fallthrough=2 -fPIC -MD -MT renderdoc/driver/vulkan/CMakeFiles/rdoc_vulkan.dir/wrappers/vk_device_funcs.cpp.o -MF renderdoc/driver/vulkan/CMakeFiles/rdoc_vulkan.dir/wrappers/vk_device_funcs.cpp.o.d -o renderdoc/driver/vulkan/CMakeFiles/rdoc_vulkan.dir/wrappers/vk_device_funcs.cpp.o -c /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/driver/vulkan/wrappers/vk_device_funcs.cpp
In file included from /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/os/os_specific.h:41,
                 from /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/common/timing.h:30,
                 from /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/driver/vulkan/wrappers/../vk_core.h:27,
                 from /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/driver/vulkan/wrappers/vk_device_funcs.cpp:26:
In member function ‘bool rdcstr::operator==(const char*) const’,
    inlined from ‘VkResult WrappedVulkan::vkCreateInstance(const VkInstanceCreateInfo*, const VkAllocationCallbacks*, VkInstance_T**)’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/driver/vulkan/wrappers/vk_device_funcs.cpp:766:62:
/tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/api/replay/rdcstr.h:868:19: warning: ‘int strcmp(const char*, const char*)’ of a string of length 25 and an array of size 23 evaluates to nonzero [-Wstring-compare]
  868 |     return !strcmp(o, c_str());
      |             ~~~~~~^~~~~~~~~~~~
/tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/driver/vulkan/wrappers/vk_device_funcs.cpp: In member function ‘VkResult WrappedVulkan::vkCreateInstance(const VkInstanceCreateInfo*, const VkAllocationCallbacks*, VkInstance_T**)’:
/tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/driver/vulkan/wrappers/vk_device_funcs.cpp:765:87: note: in this expression
  765 |   if(pCreateInfo->pApplicationInfo && pCreateInfo->pApplicationInfo->pApplicationName &&
      |      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~
  766 |      rdcstr(pCreateInfo->pApplicationInfo->pApplicationName) == "RenderDoc forced instance")
      |      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

In file included from /usr/include/string.h:519,
                 from /usr/include/python3.8/Python.h:30,
                 from renderdoc_python.cxx:178:
In function ‘void* memcpy(void*, const void*, size_t)’,
    inlined from ‘void rdcstr::insert(size_t, const char*, size_t)’ at ../../renderdoc-1.12/renderdoc/api/replay/rdcstr.h:379:11,
    inlined from ‘static swig_type_info* TypeConversion<T, isEnum>::GetTypeInfo() [with T = VKPipe::SpecializationConstant; bool isEnum = false]’ at ../../renderdoc-1.12/renderdoc/api/replay/rdcstr.h:325:61,
    inlined from ‘PyObject* array_selfconcat(arrayType*, PyObject*) [with arrayType = rdcarray<VKPipe::SpecializationConstant>]’ at ../../renderdoc-1.12/qrenderdoc/Code/pyrenderdoc/pyconversion.h:51:44:
/usr/include/bits/string_fortified.h:29:33: warning: ‘void* __builtin_memcpy(void*, const void*, long unsigned int)’ offset 30 is out of the bounds [0, 24] of object ‘baseTypeName’ with type ‘rdcstr’ [-Warray-bounds]
   29 |   return __builtin___memcpy_chk (__dest, __src, __len, __bos0 (__dest));
      |          ~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from renderdoc_python.cxx:4362:
../../renderdoc-1.12/qrenderdoc/Code/pyrenderdoc/pyconversion.h: In function ‘PyObject* array_selfconcat(arrayType*, PyObject*) [with arrayType = rdcarray<VKPipe::SpecializationConstant>]’:
../../renderdoc-1.12/qrenderdoc/Code/pyrenderdoc/pyconversion.h:42:12: note: ‘baseTypeName’ declared here
   42 |     rdcstr baseTypeName = TypeName<T>();
      |            ^~~~~~~~~~~~

[332/337] /usr/lib/ccache/bin/x86_64-pc-linux-gnu-g++ -DRENDERDOC_LIB_SUFFIX=64 -DRENDERDOC_PLATFORM_LINUX -DRENDERDOC_SUPPORT_EGL -DRENDERDOC_SUPPORT_GL -DRENDERDOC_SUPPORT_GLES -DRENDERDOC_SUPPORT_VULKAN -DRENDERDOC_WINDOWING_XCB -DRENDERDOC_WINDOWING_XLIB -D_RELEASE -D_qrenderdoc_EXPORTS -I/tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/qrenderdoc -I/tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/api/replay -I/usr/include/python3.8  -DNDEBUG -O3 -pipe -march=native -std=c++11 -fno-strict-aliasing -fvisibility=hidden -fvisibility-inlines-hidden -Wall -Wextra -Wno-unused-variable -Wno-unused-parameter -Wno-unused-result -Wno-type-limits -Wno-missing-field-initializers -Wno-unknown -pragmas -Wno-reorder -Wno-unused-but-set-variable -Wno-maybe-uninitialized -Wno-class-memaccess -Wimplicit-fallthrough=2 -Wno-cast-function-type -Wno-stringop-truncation -Wno-deprecated-declarations -fPIC -std=gnu++14 -MD -MT qrenderdoc/Code/pyrenderdoc/CMakeFiles/_qrenderdoc.dir/__/__/qrenderdoc_python.cxx.o -MF qrenderdoc/Code/pyrenderdoc/CMakeFiles/_qrenderdoc.dir/__/__/qrenderdoc_python.cxx.o.d -o qrenderdoc/Code/pyrenderdoc/CMakeFiles/_qrenderdoc.dir/__/__/qrenderdoc_python.cxx.o -c qrenderdoc/qrenderdoc_python.cxx
In file included from /usr/include/string.h:519,
                 from /usr/include/python3.8/Python.h:30,
                 from qrenderdoc/qrenderdoc_python.cxx:180:
In function ‘void* memcpy(void*, const void*, size_t)’,
    inlined from ‘void rdcstr::ensure_mutable(size_t)’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/api/replay/rdcstr.h:195:13,
    inlined from ‘char* rdcstr::data()’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/api/replay/rdcstr.h:164:8,
    inlined from ‘void rdcstr::insert(size_t, const char*, size_t)’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/api/replay/rdcstr.h:374:21,
    inlined from ‘void rdcstr::append(const char*, size_t)’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/api/replay/rdcstr.h:325:61,
    inlined from ‘static swig_type_info* TypeConversion<T, isEnum>::GetTypeInfo() [with T = ExtensionMetadata; bool isEnum = false]’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/api/replay/rdcstr.h:602:11,
    inlined from ‘static PyObject* TypeConversion<T, isEnum>::ConvertToPy(const T&) [with T = ExtensionMetadata; bool isEnum = false]’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/qrenderdoc/Code/pyrenderdoc/pyconversion.h:65:44,
    inlined from ‘PyObject* array_copy(arrayType*) [with arrayType = rdcarray<ExtensionMetadata>]’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/qrenderdoc/Code/pyrenderdoc/pyconversion.h:854:40,
    inlined from ‘PyObject* _wrap_rdcarray_of_ExtensionMetadata_copy(PyObject*, PyObject*)’ at qrenderdoc/qrenderdoc_python.cxx:4723:24:
/usr/include/bits/string_fortified.h:29:33: warning: ‘void* __builtin_memcpy(void*, const void*, long unsigned int)’ accessing<PHONE_NUMBER>867975535 bytes at offsets 0 and 0 overlaps<PHONE_NUMBER>881175263 bytes at offset<PHONE_NUMBER>86800272 [-Wrestrict]
   29 |   return __builtin___memcpy_chk (__dest, __src, __len, __bos0 (__dest));
      |          ~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In function ‘void* memcpy(void*, const void*, size_t)’,
    inlined from ‘void rdcstr::ensure_mutable(size_t)’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/api/replay/rdcstr.h:195:13,
    inlined from ‘char* rdcstr::data()’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/api/replay/rdcstr.h:164:8,
    inlined from ‘void rdcstr::insert(size_t, const char*, size_t)’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/api/replay/rdcstr.h:374:21,
    inlined from ‘void rdcstr::append(const char*, size_t)’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/api/replay/rdcstr.h:325:61,
    inlined from ‘static swig_type_info* TypeConversion<T, isEnum>::GetTypeInfo() [with T = ExtensionMetadata; bool isEnum = false]’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/api/replay/rdcstr.h:602:11,
    inlined from ‘static PyObject* TypeConversion<T, isEnum>::ConvertToPy(const T&) [with T = ExtensionMetadata; bool isEnum = false]’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/qrenderdoc/Code/pyrenderdoc/pyconversion.h:65:44,
    inlined from ‘static PyObject* TypeConversion<rdcarray<U>, false>::ConvertToPyInPlace(PyObject*, const rdcarray<U>&, int*) [with U = ExtensionMetadata]’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/qrenderdoc/Code/pyrenderdoc/pyconversion.h:596:54,
    inlined from ‘static PyObject* TypeConversion<rdcarray<U>, false>::ConvertToPy(const rdcarray<U>&, int*) [with U = ExtensionMetadata]’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/qrenderdoc/Code/pyrenderdoc/pyconversion.h:622:39,
    inlined from ‘PyObject* array_repr(arrayType*) [with arrayType = rdcarray<ExtensionMetadata>]’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/qrenderdoc/Code/pyrenderdoc/pyconversion.h:631:75:
/usr/include/bits/string_fortified.h:29:33: warning: ‘void* __builtin_memcpy(void*, const void*, long unsigned int)’ accessing<PHONE_NUMBER>867975535 bytes at offsets 0 and 0 overlaps<PHONE_NUMBER>881175263 bytes at offset<PHONE_NUMBER>86800272 [-Wrestrict]
   29 |   return __builtin___memcpy_chk (__dest, __src, __len, __bos0 (__dest));
      |          ~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In function ‘void* memcpy(void*, const void*, size_t)’,
    inlined from ‘void rdcstr::ensure_mutable(size_t)’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/api/replay/rdcstr.h:195:13,
    inlined from ‘char* rdcstr::data()’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/api/replay/rdcstr.h:164:8,
    inlined from ‘void rdcstr::insert(size_t, const char*, size_t)’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/api/replay/rdcstr.h:374:21,
    inlined from ‘void rdcstr::append(const char*, size_t)’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/api/replay/rdcstr.h:325:61,
    inlined from ‘static swig_type_info* TypeConversion<T, isEnum>::GetTypeInfo() [with T = ExtensionMetadata; bool isEnum = false]’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/renderdoc/api/replay/rdcstr.h:602:11,
    inlined from ‘static PyObject* TypeConversion<T, isEnum>::ConvertToPy(const T&) [with T = ExtensionMetadata; bool isEnum = false]’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/qrenderdoc/Code/pyrenderdoc/pyconversion.h:65:44,
    inlined from ‘PyObject* array_concat(arrayType*, PyObject*) [with arrayType = rdcarray<ExtensionMetadata>]’ at /tmp/portage/dev-util/renderdoc-1.12/work/renderdoc-1.12/qrenderdoc/Code/pyrenderdoc/pyconversion.h:854:40:
/usr/include/bits/string_fortified.h:29:33: warning: ‘void* __builtin_memcpy(void*, const void*, long unsigned int)’ accessing<PHONE_NUMBER>867975535 bytes at offsets 0 and 0 overlaps<PHONE_NUMBER>881175263 bytes at offset<PHONE_NUMBER>86800272 [-Wrestrict]
   29 |   return __builtin___memcpy_chk (__dest, __src, __len, __bos0 (__dest));
      |          ~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Steps to reproduce
Build it. g++10.2.0.
On development builds, this -Werrors.
Environment

RenderDoc version: 1.12
Operating System: Gentoo
Graphics API: doesn't matter.

I can't reproduce this. I don't have a Gentoo install to hand but on g++ 10.2.0 on ubuntu (g++-10 (Ubuntu 10.2.0-5ubuntu1~20.04) 10.2.0) the development build works with no warnings. If you have any extra CFLAGS or other extra compiler flags or customisations try disabling them in case they are causing the problem.
Looking at the warnings they appear to be nonsense, so this seems like a compiler bug to me - perhaps related to a particular flag if you are enabling any non-default flags.

I can't reproduce this. I don't have a Gentoo install to hand but on g++ 10.2.0 on ubuntu (g++-10 (Ubuntu 10.2.0-5ubuntu1~20.04) 10.2.0) the development build works with no warnings. If you have any extra CFLAGS or other extra compiler flags or customisations try disabling them in case they are causing the problem.
Looking at the warnings they appear to be nonsense, so this seems like a compiler bug to me - perhaps related to a particular flag if you are enabling any non-default flags.

@baldurk I was seeing this years ago, and I'm seeing this now again on a completely different system. Basically just configured the default build and tried to make it:
[nix-shell:/x/code/renderdoc/build]$ cmake ..
-- Calculating version
-- Building RenderDoc version 1.18
-- Building using Qt 5.15.3
-- Found BISON: /nix/store/6kc28ix1fng215s3dx88fi94nbjfp976-bison-3.8.2/bin/bison (found version "3.8.2")
-- PySide2 not found - Qt will not be accessible in python scripting. See https://github.com/baldurk/renderdoc/wiki/PySide2
-- Enabled APIs:
--   - OpenGL (with additional EGL support)
--   - OpenGL ES (EGL)
--   - Vulkan
-- Enabled Window System Support:
--   - XLib
--   - XCB
-- Configuring done
-- Generating done
-- Build files have been written to: /x/code/renderdoc/build

Error:
In file included from /x/code/renderdoc/renderdoc/os/os_specific.h:41,
                 from /x/code/renderdoc/renderdoc/common/timing.h:30,
                 from /x/code/renderdoc/renderdoc/driver/vulkan/wrappers/../vk_core.h:27,
                 from /x/code/renderdoc/renderdoc/driver/vulkan/wrappers/vk_device_funcs.cpp:26:
In member function ‘bool rdcstr::operator==(const char*) const’,
    inlined from ‘VkResult WrappedVulkan::vkCreateInstance(const VkInstanceCreateInfo*, const VkAllocationCallbacks*, VkInstance_T**)’ at /x/code/renderdoc/renderdoc/driver/vulkan/wrappers/vk_device_funcs.cpp:549:64:
/x/code/renderdoc/renderdoc/api/replay/rdcstr.h:868:19: error: ‘int strcmp(const char*, const char*)’ of a string of length 25 and an array of size 23 evaluates to nonzero [-Werror=string-compare]
  868 |     return !strcmp(o, c_str());
      |             ~~~~~~^~~~~~~~~~~~
/x/code/renderdoc/renderdoc/driver/vulkan/wrappers/vk_device_funcs.cpp: In member function ‘VkResult WrappedVulkan::vkCreateInstance(const VkInstanceCreateInfo*, const VkAllocationCallbacks*, VkInstance_T**)’:
/x/code/renderdoc/renderdoc/driver/vulkan/wrappers/vk_device_funcs.cpp:548:89: note: in this expression
  548 |       (pCreateInfo->pApplicationInfo && pCreateInfo->pApplicationInfo->pApplicationName &&
      |       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~
  549 |        rdcstr(pCreateInfo->pApplicationInfo->pApplicationName) == "RenderDoc forced instance");
      |        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1plus: all warnings being treated as errors
make[2]: *** [renderdoc/driver/vulkan/CMakeFiles/rdoc_vulkan.dir/build.make:524: renderdoc/driver/vulkan/CMakeFiles/rdoc_vulkan.dir/wrappers/vk_device_funcs.cpp.o] Error 1
make[1]: *** [CMakeFiles/Makefile2:437: renderdoc/driver/vulkan/CMakeFiles/rdoc_vulkan.dir/all] Error 2
make: *** [Makefile:136: all] Error 2

Compiler:
[nix-shell:/x/code/renderdoc/build]$ gcc --version
gcc (GCC) 10.3.0
Copyright (C) 2020 Free Software Foundation, Inc.

I don't know if it's a compiler bug or not, but it's very suspicious that the bug is still there. Something needs to be done about it:

either report upstream, or find an existing upstream bug
disable erroring on this particular warning in renderdoc's build
refactor the code to make gcc happy

With this in mind, could we re-open this?

Please do not comment on old issues that were closed long ago.


Hacker News Theme

I created a theme for Hacker News using Stylus!
Link for my repo here, and you can check the theme here

Hey, very cool! ✨
I just sent you an invitation to join the organization.
I also cloned your repository for Dracula's organization, so you can go ahead and delete or archive yours:

Your repository.
New repository.

Thanks for building this! 🎉

Awesome! Just opened a PR with the install instructions!

Thank you, the theme is already listed on the site! 🚀


[BUG]: Seek method in android , only integer seconds can make effect

Version
6.3.0
What platforms are you having the problem on?
Android
System Version
6.3.0
On what device are you experiencing the issue?
Real device, Simulator
Architecture
New architecture with interop layer
What happened?
Seek method in android , only support integer number
Reproduction
No response
Reproduction
1.load any local source
2. use the hook below ,try to keep seeking the video by 0.1 s
React.useEffect(() => {
    let index = 0;
    var ref = setInterval(() => {
          index = index +0.1
          console.log(index);
          videoRef?.current?.seek(index);

    }, 100);

    return () => {
      clearInterval(ref);
    };
  }, []);

3.then , when the second is 0.1 0.2.... nothing happened,if it come to 1.1  it will play one time . then keep stucked ,util it come to 2.0, 2.1

would this patch be enough:
diff --git a/android/src/main/java/com/brentvatne/react/VideoManagerModule.kt b/android/src/main/java/com/brentvatne/react/VideoManagerModule.kt
index 716f337d..b20497ba 100644
--- a/android/src/main/java/com/brentvatne/react/VideoManagerModule.kt
+++ b/android/src/main/java/com/brentvatne/react/VideoManagerModule.kt
@@ -49,7 +49,7 @@ class VideoManagerModule(reactContext: ReactApplicationContext?) : ReactContextB
             return
         }

-        val time = ReactBridgeUtils.safeGetInt(info, "time")
+        val time = ReactBridgeUtils.safeGetDouble(info, "time")
         performOnPlayerView(reactTag) {
             it?.seekTo((time * 1000f).roundToInt().toLong())
         }


Backmerge release 2.53.0

Release: https://github.com/safe-global/safe-config-service/releases/tag/v2.53.0

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.789%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
945

Relevant Lines:
947

💛  - Coveralls


[Feature] emphasis Added detailed iconStyle settings according to mouse motion.

What problem does this feature solve?
Is it possible to set the color of mouse hover and click separately in the following options?
If not, I thought
What does the proposed API look like?
 iconStyle: {                      emphasis: {                                 hover:{ borderColor: 'red'}                                 onClick:{borderColor: 'blue'}                             }                  },

yes, possible - hover=emphasis and click=select
Demo code

It does not go back to its original state when clicked again(default off).
At first it was off, but after clicking it changed to 'on' color.
I want to separate 'on&off' color and 'hover' color.

Thank you so much for your help. Solved.


I have added a file named experiments which I will use to play with V…

…Mware API.

You dont need to open a pull request for this.. Its not a sample.

Just FYI: if you ever do write a sample we stick to pep8 standards. [https://travis-ci.org/vmware/pyvmomi-community-samples/ Travis] watches things for us and you can see what he says about things. In this case, here's an example of what Travis does every time we see a pull request...
$ git clone --depth=50 git://github.com/vmware/pyvmomi-community-samples.git vmware/pyvmomi-community-samples
Cloning into 'vmware/pyvmomi-community-samples'...
remote: Counting objects: 544, done.
remote: Compressing objects: 100% (24/24), done.
remote: Total 544 (delta 12), reused 0 (delta 0), pack-reused 520
Receiving objects: 100% (544/544), 158.44 KiB | 0 bytes/s, done.
Resolving deltas: 100% (315/315), done.
Checking connectivity... done.
$ cd vmware/pyvmomi-community-samples
0.13s$ git fetch origin +refs/pull/183/merge:
remote: Counting objects: 5, done.
remote: Compressing objects: 100% (4/4), done.
remote: Total 5 (delta 3), reused 1 (delta 0), pack-reused 0
Unpacking objects: 100% (5/5), done.
From git://github.com/vmware/pyvmomi-community-samples
 * branch            refs/pull/183/merge -> FETCH_HEAD
$ git checkout -qf FETCH_HEAD
0.01s$ source ~/virtualenv/python2.6/bin/activate
$ python --version
Python 2.6.9
$ pip --version
pip 6.0.7 from /home/travis/virtualenv/python2.6.9/lib/python2.6/site-packages (python 2.6)
before_install.1
1.33s$ if [[ $TRAVIS_PYTHON_VERSION == '2.6' ]]; then pip install unittest2; fi
You are using pip version 6.0.7, however version 7.0.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
Collecting unittest2
  Downloading unittest2-1.0.1-py2.py3-none-any.whl (95kB)
    100% |################################| 98kB 2.4MB/s
Requirement already satisfied (use --upgrade to upgrade): argparse in /home/travis/virtualenv/python2.6.9/lib/python2.6/site-packages (from unittest2)
Collecting traceback2 (from unittest2)
  Downloading traceback2-1.4.0-py2.py3-none-any.whl
Collecting six>=1.4 (from unittest2)
  Downloading six-1.9.0-py2.py3-none-any.whl
Collecting linecache2 (from traceback2->unittest2)
  Downloading linecache2-1.0.0-py2.py3-none-any.whl
Installing collected packages: linecache2, six, traceback2, unittest2
  Compiling /tmp/pip-build-_hNpmo/linecache2/linecache2/tests/inspect_fodder2.py ...

Successfully installed linecache2-1.0.0 six-1.9.0 traceback2-1.4.0 unittest2-1.0.1
before_install.2
0.79s$ pip install pep8
You are using pip version 6.0.7, however version 7.0.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
Collecting pep8
  Downloading pep8-1.6.2-py2.py3-none-any.whl (40kB)
    100% |################################| 40kB 2.0MB/s
Installing collected packages: pep8
Successfully installed pep8-1.6.2
1.01s$ pep8 $TRAVIS_BUILD_DIR/samples/*.py
/home/travis/build/vmware/pyvmomi-community-samples/samples/add_disk_to_vm.py:117:80: E501 line too long (126 > 79 characters)
The command "pep8 $TRAVIS_BUILD_DIR/samples/*.py" failed and exited with 1 during .


As a user I should be able to disable the update notice

Is your feature request related to a problem? Please describe.
Semi-related. I thought of this request due to having to "pin" to not-latest-versions of portainer.
Describe the solution you'd like
I would like the "option" to disable the update notice in the bottom left of the portainer UI in the toolbox gutter.
Describe alternatives you've considered
None that I am aware of.
Additional context
This is the UI element I am referencing:

This feature request came to mind after having to hold back upgrades because of other (reported) portainer issues.

Issues like:

#3432
#3575

Sometimes means I can't upgrade to the latest right away. I totally understand that FOSS development is messy so sometimes one can't be on the latest-and-greatest. However, this notice is somewhat bothersome when I am on purpose a version or two behind.

As always portainer devs, your efforts and work help make this product and my life fantastic. Thank you for your time.

Thanks for the suggestion, much appreciated :)

Don't forget about the news banner that always shows up, after dismissing it a hundred times.
This even happens on a brand new install.

@sfkpmr IIRC Portainer keeps track of whether the MOTD banner has been disabled by the user in the browser cookies. So if you're constantly clearing cookies you're going to see that banner again and again. @xAt0mZ correct me if I am wrong on this
Also thanks for letting us know about the out of date MOTD, I will pass this on to the team to get this updated!

True, Portainer is keeping track of the UI State in the cookies (portainer.UI_STATE cookie to be exact) so yes, if you delete the cookies, state resets.

Is there a way to have portainer track this information instead of a cookie?

We can look to add an option in future versions.

any updates on this?

Any news? There's basically an update every week it seems so I have a permanent big orange update button. I understand, there's an update but Portainer is not critical to me at all so please don't put in my face.

I would still like this feature

Partial Fix: Use ublock origin 'element picker' feature on chrome to block the upgrade icon.
This is the most annoying thing. With everything nagging you nowadays, I was hoping to have my own local stuff not nag.

If your Portainer instance is running in container, just add host mapping for api.github.com:<IP_ADDRESS>

It looks like, given: https://github.com/portainer/portainer/blob/develop/api/http/handler/status/version.go#L62-L66

It should be possible to fetch a configuration value to just skip this check.

In 2.15, which should be released soon, we hope to add a 'Dismiss' option to the update banner.

Freaking awesome. Can't wait to try it out.


fix hardcoded --ram node type for cluster join

cc @michaelklishin @jjasghar
Tested and working:
[2015-12-16T01:26:23+00:00] INFO: Processing rabbitmq_cluster[[{"name":"rabbit@ec2-54-201-251-170.us-west-2.compute.amazonaws.com","type":"disc"}]] action join (gz-rabbitmq::join line 5)
[2015-12-16T01:26:23+00:00] INFO: [rabbitmq_cluster] Action join ...
[2015-12-16T01:26:24+00:00] INFO: DOUDOU =><EMAIL_ADDRESS>[2015-12-16T01:26:25+00:00] INFO: [rabbitmq_cluster] Clustering node<EMAIL_ADDRESS>with<EMAIL_ADDRESS>...

[2015-12-16T01:26:26+00:00] INFO: [rabbitmq_cluster] Node<EMAIL_ADDRESS>joined in<EMAIL_ADDRESS>with type disc
[2015-12-16T01:26:26+00:00] INFO: [{nodes,[{disc,['rabbit@ec2-54-200-235-67.us-west-2.compute.amazonaws.com','rabbit@ec2-54-201-251-170.us-west-2.compute.amazonaws.com']}]},{running_nodes,['rabbit@ec2-54-201-251-170.us-west-2.compute.amazonaws.com','rabbit@ec2-54-200-235-67.us-west-2.compute.amazonaws.com']},{cluster_name,<<"rabbit@ip-10-52-42-243.us-west-2.compute.internal">>},{partitions,[]}]

root@rabbitmq-i-fef72824 ~ # rabbitmqctl cluster_status
Cluster status of node<EMAIL_ADDRESS>...
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS> {partitions,[]}]

Default node type should definitely not be RAM. There is no reason to use RAM nodes for 95% of the workloads.

Any chance we can merge it @michaelklishin ? The failing check is related to the if condition related to the node type passed.

@scalp42 I havent had a chance to look into this yet. It's on my TODO and when i have some time i'll be visiting this.
Though i do trust @michaelklishin's opinion ;)

Can you add a skip for the travis failure too?

@jjasghar passing now.

@michaelklishin @jjasghar we should really merge this in, this is literally hardcoded right now.

Yep!
I just need to set up another release. It takes a bit of time so I have to schedule it.


refactor: Refactor and Scraper

Description
Improved project structure and added scraper functionality expanding upon Kingteena's code
Changes Made

refactor: Refactored and structured project to be more maintainable
feat: Added Test suite to test current classes and establish good testing habits to avoid breaking changes
feat: Tested example data (https://github.com/compsci-adl/courses-api/issues/2)
chore: Added .git ignore template
docs: Improved README
chore: Added makefile

Related Issue
Fixes #2
Additional Notes
Add any other notes or screenshots that might be helpful for the reviewers.
-tests passing with all example fields listed on issue #2 ticket.

Probably would need to wait until #4 and #5 are merged to do some more refactoring since those PR's would change multiple things.

Please edit your PR title to describe what you did in this PR. TY!

Please edit your PR title to describe what you did in this PR. TY!

Done: descriptions are in changes made section

It will be better if you follow https://github.com/compsci-adl/.github/blob/main/CONTRIBUTING.md#commit-messages to name your PR title.

@hughmancoder Can you update your branch with main?

@hughmancoder #4 and #5 have now been merged, so can you update your branch with main and make changes as needed?

Done. I've tested all the server routes and can confirm we get a response. Documentation updated too and unit tests passing

@hughmancoder We made some changes to the project structure already to get the scraper working and make the server work with the api, so if you can make a separate PR just for adding testing that would be great.


HA for Alert-manager

Hi for the sake of HA in alert-manager I have deployed two instances and those two were successfully configured in promethus now my point is how to enable the gossip between those two instances and i have see some warn logs here
level=info ts=2020-12-08T05:04:57.578Z caller=main.go:216 msg="Starting Alertmanager" version="(version=0.21.0, branch=HEAD, revision=4c6c03ebfe1009c546e4d1e9b92c371d67c021d)"
level=info ts=2020-12-08T05:04:57.578Z caller=main.go:217 build_context="(go=go1.14.4, user=root@dee3592357f, date=20200617-08:54:02)"
level=warn ts=2020-12-08T05:04:57.593Z caller=cluster.go:228 component=cluster msg="failed to join cluster" err="1 error occurred:\n\t* Failed to resolve alertmanager-test-0.alertmanager-operated.depl.svc:9094: lookup alertmanager-test-0.alertmanager-operated.depl.svc on x.x.x.x:53: no such host\n\n"
level=info ts=2020-12-08T05:04:57.593Z caller=cluster.go:230 component=cluster msg="will retry joining cluster every 10s"
level=warn ts=2020-12-08T05:04:57.593Z caller=main.go:307 msg="unable to join gossip mesh" err="1 error occurred:\n\t* Failed to resolve alertmanager-test-0.alertmanager-operated.depl.svc:9094: lookup alertmanager-test-0.alertmanager-operated.depl.svc on x.x.x.x:53: no such host\n\n"
level=info ts=2020-12-08T05:04:57.593Z caller=cluster.go:623 component=cluster msg="Waiting for gossip to settle..." interval=2s
level=info ts=2020-12-08T05:04:57.614Z caller=coordinator.go:119 component=configuration msg="Loading configuration file" file=/etc/alertmanager/config/alertmanager.yaml
level=info ts=2020-12-08T05:04:57.614Z caller=coordinator.go:131 component=configuration msg="Completed loading of configuration file" file=/etc/alertmanager/config/alertmanager.yaml
level=info ts=2020-12-08T05:04:57.617Z caller=main.go:485 msg=Listening address=:9093
level=info ts=2020-12-08T05:04:59.594Z caller=cluster.go:648 component=cluster msg="gossip not settled" polls=0 before=0 now=1 elapsed=2.000220306s
level=info ts=2020-12-08T05:05:07.594Z caller=cluster.go:640 component=cluster msg="gossip settled; proceeding" elapsed=10.00090527s
level=warn ts=2020-12-08T05:05:13.057Z caller=cluster.go:438 component=cluster msg=refresh result=failure addr=alertmanager-test-0.alertmanager-operated.depl.svc:9094 err="1 error occurred:\n\t* Failed to resolve alertmanager-test-0.alertmanager-operated.depl.svc:9094: lookup alertmanager-test-0.alertmanager-operated.depl.svc on x.x.x.x:53: no such host\n\n"

Thanks for your report. It looks as if this is actually a question about usage and not development.
It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


Write a function design

What is needed for the framework to really work
What are the standards to writing a function?

Kinda done on PSModule main page.


Adding GCP/Stackdriver support

I wonder if you would consider adding GCP/Stackdriver support to this repo.

I have written the provider for Auth0 and submitted a PR. We'll have to say if they'll accept it or put some work into it.

https---s3.us-west-2.amazonaws.com-temporary.notion-static.com-Export-77d0c2ae.md
@yinzara @haithem-souala @starsprung @starsprung @machuga @xer0x @haithem-souala @machuga


Toggle Task can not be used for uninstalled custom resources

What happened:
Tried to change a resource deployment to use a Toggle Task. The resource is a custom resource for a CRD that may not exist on the cluster. KUDO failed with:
06
          +        - message: 'A transient error when executing task deploy.nodes.pre-node.monitor-deploy.
16:17:06
          +            Will retry. failed to determine if object &{map[apiVersion:monitoring.coreos.com/v1
16:17:06
          +            kind:ServiceMonitor metadata:map[annotations:map[kudo.dev/last-plan-execution-uid:6521e1bf-e0c4-45ab-a295-356198b3557b
16:17:06
          +            kudo.dev/phase:nodes kudo.dev/plan:deploy kudo.dev/step:pre-node] labels:map[app:prometheus-operator
16:17:06
          +            heritage:kudo kudo.dev/instance:cassandra kudo.dev/operator:cassandra
16:17:06
          +            release:prometheus-kubeaddons] name:cassandra-monitor namespace:cassandra-install-test]
16:17:06
          +            spec:map[endpoints:[map[interval:30s port:prometheus-exporter-port]] namespaceSelector:map[matchNames:[cassandra-install-test]]
16:17:06
          +            selector:map[matchLabels:map[kudo.dev/instance:cassandra kudo.dev/servicemonitor:true]]]]}
16:17:06
          +            is namespaced: a resource with GVK monitoring.coreos.com/v1, Kind=ServiceMonitor
16:17:06
          +            seems to be missing in API resource list'

What you expected to happen:
The toggle task should be able to "delete" or not deploy a custom resource for which the CRD is not known to the cluster
How to reproduce it (as minimally and precisely as possible):
Use a toggle task with a custom resource which is not known to the cluster.
Anything else we need to know?:
The task_delete.go uses the enhancer which tries to determine if the resource to deploy is namespaced or not - which fails for an unknown custom resource.

The toggle task should be able to "delete" or not deploy a custom resource for which the CRD is not known to the cluster

that is a failed requirement and I think it should fail
if there is a CRD required for an operator installation that should be fixed independent of if its a Toggle task or not.

Well, if the parameter for the Toggle Task is "false", then the CRD is not required to install the operator, correct? And in this case the toggle tasks should simply do nothing and not fail the execution because it doesn't know about the CR that it wants to delete.

got it!
that would be a slightly different delete task. Would that be the same expectation from Delete task or just Toggle task?

Good question. My gut feeling says that it should only be like that for the Toggle task, as it depends on a parameter and is often used to enable/disable deployment of resources - I'm not fully sure what the DeleteTask would be used for, but I'd expect it to fail if one tried to delete an unknown custom resource.
But I wouldn't mind otherwise as well


Problem with the monotransquest-hter-en_cs model

Hi! I wanted to use the TransQuest/monotransquest-hter-en_cs-pharmaceutical model but it outputs the same constant for any input sentences. Other models work well so it seems that this model in particular is broken.
May I kindly ask you to upload fixed model? Thank you
For the bug replication check out this Colab: https://colab.research.google.com/drive/1ujKgV8aS7E9XjNz02pdh7vhYDpDAuH_m#scrollTo=npg2206l9Z7b

Hi!, was just wondering if you ever came up with a fix to this issue?

Hey, sadly not. If I remember correctly I used COMET reference free models instead. Hope that helps!

That does help, thank you!


Add modulemap for static compilation

When compiling statically, we have to link against DispatchStubs in addition to the other dependencies, so we are defining a separate modulemap for that.

@swift-ci test linux

@compnerd The only difference between the existing modulemap and the new one is
link "DispatchStubs"

otherwise they are exactly the same. So as to why it is required, I don't know.

Oh, I get it.  Its for the autolinking to its dependencies from the Swift module.  LGTM with grumbling.


ROX-24892: Fix accessibility issues in compliance clusters

Description
Problems according to axe DevTools

/main/compliance/clusters/id

Elements must meet minimum color contrast ratio thresholds

remote

/main/compliance/clusters/id/controls

Elements must meet minimum color contrast ratio thresholds

122 CONTROLS

Analysis

Classic text-primary-700 has insufficient 3.65 color contrast ratio.

Small, heavy, and dim: font-700 text-sm opacity-50
Was also unique combination of size, weight, and opacity.

Solutions

Replace text-primary-700 with pf-v5-u-link-color utility class.

Move and edit TableGroup.js file.

Render as ordinary text.
Because TableGroup is only for compliance standards, replace generic pluralize of UPPERCASE entity type with container-specific entityCountNounOrdinaryCase function.

Residue

Replace other occurrences of text-primary-700 class?
Investigate accessibility issue on many classic pages:

Scrollable region must have keyboard access

Checklist

[ ] Investigated and inspected CI test results
[x] ~Unit test and regression tests added~

Testing Performed

yarn lint in ui
yarn build in ui and then compute branch - master for the following

wc build/static/js/*.js
main.js 0 = 4636952 - 4636952
total -128 = 11719417 - 11719545
ls -al build/static/js/*.js | wc
files 0 = 175 - 175

yarn start in ui

Manual testing

Visit /main/compliance/clusters and click a row to display side panel.

Before change without sufficient color contrast ratio

After change with sufficient color contrast ratio

Click cluster link in side panel, and then click Controls tab in single page.

Before change without sufficient color contrast ratio

After change with sufficient color contrast ratio

/test gke-nongroovy-e2e-tests


[Form Recognizer ] Failed to deploy test resource in weekly pipeline

We are running live tests against other clouds like US Gov and Azure China Cloud. The goal is to check whether new azure sdk package work with other clouds or not.
Error Description:
When running the Form Recognizer  pipeline on Public and UsGov and Azure China cloud, it runs failed at section Deploy test resource and the error messages is shown as following, for more details please check here:

Expected Behavior:
The deploy test resource can pass in weekly pipeline.
@benbp, @joshfree, @mayurid, @meeraharidasa and @samvaity for notification.

@hongli750210 please update your testing branch. We added a new parameter to the scripts and updated the corresponding dynamic cloud configuration on the backend. The updated branch should have this line updated. Let me know if there is still an issue.

@benbp I run a pipeline test of Form Recognizer Service after updated testing branch. Although the test resources can be successfully delpoyed on Public cloud and entered the testing section, but still failed to deploy test resource on UsGov cloud and China cloud. The update of dynamic cloud configuration on backend on Public cloud has been not synchronized to UsGov Cloud and China Cloud I think. Please update the corresponding dynamic cloud configuration on the backend on UsGov Cloud and China Cloud.

@samvaity could you follow up with @hongli750210?

@hongli750210 I believe your branch still needs this change: https://github.com/Azure/azure-sdk-for-java/commit/fc918a0a5d6c94a270ea03813ab07c36b16d8945 so you need to git pull in the main branch.

@benbp After updating my testing branch again, the test resources can also be successfully delpoyed on UsGov and China cloud. Thank you for your reminder.


Update README.md

README.md reflects renaming of SWHttpTrafficRecorder.sharedRecorder() to SWHttpTrafficRecorder.shared().

@appleitung hmm, I am confused - SWHttpTrafficRecorder.sharedRecorder() is the right interface per how it is exposed in .h file?

@appleitung please confirm and close the pull request if agreed

I didn't actually check the interface, my PR was based on an error I encountered in Xcode directly:


Flomesh + Dapr blog duplicate

Please advise If any further action from my side is required to get it published to Dapr blog?

@naqvis the other maintainers are reviewing, but so far the feedback is confusion around the diagram showing Redis(flomesh_dapr_blog_demo_arch.png), specifically where Redis is deployed and how it relates to the cross cluster calls. Would you be able to update that to make it more clear?


chirp: 0.4.0-unstable-2024-05-10 -> 0.4.0-unstable-2024-07-05, python311Packages.suds: init at 1.1.2

Description of changes
There are three main things in this PR:

new mandatory chirp dependency added, python311Packages.suds: init at 1.1.2, as mentioned in #317593.
chirp package updated to 0.4.0-unstable-2024-07-05 and python 3.11 module issues addressed to allow it to build again.
added myself as a chirp maintainer if acceptable.

Result of nixpkgs-review run on x86_64-linux 1

4 packages built:

chirp
chirp.dist
python311Packages.suds
python311Packages.suds.dist

Request: This is my first time packaging for python with nixpkgs, please let me know if there are best practices or anything I missed on the new package.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[X] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Thank you for the detailed review @wegank.

Updated @natsukium, thanks as well for the review.

Result of nixpkgs-review pr 326171 at a445123f run on x86_64-linux 1

3 packages built successfully:

chirp
python311Packages.suds
python312Packages.suds


Feature request : remember the state of the workspace

VSCode Version: 1.4.0
OS Version: OS X 10.11

When i relaunch VS Code, my tabs are kept, it's what i want.
But the view "Output" is showed again while i decided to hide it before i quit.
Same thing for the breakline, it's systematically on when i launch VS Code.
It's only two examples but I would appreciate if my workspace is in the same state than when i quit previously.

@pom421 are you saying we keep restoring the output panel all the time after restart? can you show a screenshot?


Microsoft Forms Data is restored too

When an office 365 group is restored, any microsoft form and its data associated with the group is also restored. It is nice to add Forms to the list of content that will get restored

@officedocsbot assign @mypil

@sairao77 - May we know which Microsoft document link you used in reference to this issue?
Please share it with us so we can investigate further. Thank you.

@mypil - I didn't use a particular Microsoft document link, during a testing process to retrieve Forms data that is deleted as part of Office 365 Group, I found that Forms data can be retrieved when an Office 365 group is restored. So I would like to suggest this to be added to docs for anybody in the future to reference it.
Below link should help you reference why I would like to suggest this change
https://techcommunity.microsoft.com/t5/Microsoft-Forms/Form-dissapeared/m-p/475110#M3108

@sairao77 - Thank you for engaging with the community here for the docs.
I know the product teams are always looking for user feedback on how to make Office 365 better. They keep a close tab on user feedback and it would be awesome if you could add your voice in
User Voice https://office365.uservoice.com
You might find others have made a similar request that you can add to, or you can open a new one. Let us know here if you do so we can up-vote your request.
Thank you for your contribution to make the docs better! Much appreciated!

@mypil - but isn't this more of a documentation addition rather than a feature addition, for which we might have to use uservoice

@sairao77 - Based on the link you provided, the only MS article indicated is this link - https://docs.microsoft.com/en-us/office365/admin/create-groups/restore-deleted-group?view=o365-worldwide
Is this the article where you want the information to be added to? Kindly confirm. Thank you.

Yes @mypil , I would like the information to be added to this article :)

@sairao77 - a Pull Request (PR #454) has been created by @joinimran to incorporate your feedback in the article. We just have to wait until the doc owner merges the PR content for the changes to appear in the article. Thank you.

@sairao77 - From our understanding, the issue has been resolved but it may take a few days for the merged content to appear in the article. If you feel it hasn't been resolved, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!

@officedocsbot close


libxcrypt: fix on systems with 1GiB hugetables

patch is not load with fetchpatch since we don't have this during stdenv bootstrapping.
Description of changes
Still waiting to compile...

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

We asked upstream to provide a release, and since they've been pretty responsive we can wait a few days for their feedback.

There we go.

Please reference the release notes in the commit message as per CONTRIBUTING.md :wink:


change the favicon icon for a local tab such as "chrome://history/"

Hi @YousefMMS,
Unfortunately, it seems that the extension can't access to the chrome://* tabs.
Bests

Sorry to dredge up an old issue - is this the case for chrome-extension://  URLs as well? Trying to fix IETab using the same icon :stuck_out_tongue:

Yes @ZephireNZ, same behavior for other extensions.


Implement new difficulty cap setting when accumulating difficulty

@kelvinator07 the build by CircleCI is failing. If the reason why this can not be built is related to the need of using a specific branch from RSKJ, instead of master, please create a branch with the same name as this branch(implement-difficulty-cap), so it uses that rskj branch with your latest changes to compile.


Infinite Looping on Jovice

16/04/18 11:23:37.845|ME-D2-PLM|Exception while executing execute, retrying...
16/04/18 11:23:42.921|JOVICE|Database exception: The DELETE statement conflicted
with the REFERENCE constraint "FK_MC_MECustomer". The conflict occurred in data
base "jovice", table "dbo.MECircuit", column 'MC_MU'.
The statement has been terminated., SQL: delete from MECustomer where MU_ID = 'B
<GKmrDY4AUK?A8_UB0Z'
16/04/18 11:23:42.923|ME-D2-PLM|Exception while executing execute, retrying...
16/04/18 11:23:48.096|JOVICE|Database exception: The DELETE statement conflicted
with the REFERENCE constraint "FK_MC_MECustomer". The conflict occurred in data
base "jovice", table "dbo.MECircuit", column 'MC_MU'.
The statement has been terminated., SQL: delete from MECustomer where MU_ID = 'B
<GKmrDY4AUK?A8_UB0Z'
16/04/18 11:23:48.098|ME-D2-PLM|Exception while executing execute, retrying...
16/04/18 11:23:53.236|JOVICE|Database exception: The DELETE statement conflicted
with the REFERENCE constraint "FK_MC_MECustomer". The conflict occurred in data
base "jovice", table "dbo.MECircuit", column 'MC_MU'.
The statement has been terminated., SQL: delete from MECustomer where MU_ID = 'B
<GKmrDY4AUK?A8_UB0Z'

Apparently, it happened because necrow cannot delete MECustomer because it still bound to MECircuit that should not be exists in the database. I have gone through MEProbe.cs and found out that DELETE PEER and DELETE CIRCUIT wont be executed because if conditions was not correct. I will deploy the updated necrow as soon as possible.


xfce.tumbler: 4.16.0 -> 4.16.1

Description of changes
Update to version 4.16.1
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 186439 run on x86_64-linux 1

1 package built:

xfce.tumbler

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/1108


react-intl v2

Hi Guys,
I noticed you're using the defaultMessage attribute for FormattedHTMLMessage from react-intl. I tried using a different approach in a project I forked from yours, have a look at the linked commit and let me know if you dig it and I'll try to go all the way with it.
https://github.com/jdart/este/commit/f8d2e06d9771eceddb1974a78043c57002d04d8d
Edit:
Some details about the changes in v2: https://github.com/yahoo/react-intl/blob/master/UPGRADE.md

related to #739


Heapster API not returning correct number of values at higher resolution

Environment
I am running a 5 node(each m3.medium) k8s cluster on AWS with a 1 replica nginx server deployment as my test case.
My custom Heapster replication controller just has a 1 container pod replica with the image: kubernetes/heapster:canary and the resolution set to 10s via -metric_resolution=10s
I am querying the Heapster API from outside the cluster via the kubectl proxy for this experiment.
I have attached my custom-heapster's replication controller and service yaml files for reference.
heapster-controller.yaml.txt
heapster-service.yaml.txt
Also attached is my cluster.yaml file generated from setting up the cluster via kube-aws
AWS-cluster.yaml.txt
NAMESPACE       NAME                                                              READY     STATUS    RESTARTS   AGE       NODE
calico-system   calico-policy-agent-ip-10-0-0-50.us-west-1.compute.internal       2/2       Running   0          6d        ip-10-0-0-50.us-west-1.compute.internal
default         my-nginx-3800858182-cgh9i                                         1/1       Running   0          12h       ip-10-0-0-98.us-west-1.compute.internal
kube-system     heapster-custom-z4kdm                                             1/1       Running   0          3d        ip-10-0-0-98.us-west-1.compute.internal
kube-system     heapster-v1.0.2-743892168-57uf2                                   2/2       Running   0          6d        ip-10-0-0-96.us-west-1.compute.internal
kube-system     kube-apiserver-ip-10-0-0-50.us-west-1.compute.internal            1/1       Running   0          6d        ip-10-0-0-50.us-west-1.compute.internal
kube-system     kube-controller-manager-ip-10-0-0-50.us-west-1.compute.internal   1/1       Running   1          6d        ip-10-0-0-50.us-west-1.compute.internal
kube-system     kube-dns-v11-e82ff                                                4/4       Running   0          6d        ip-10-0-0-99.us-west-1.compute.internal
kube-system     kube-proxy-ip-10-0-0-50.us-west-1.compute.internal                1/1       Running   0          6d        ip-10-0-0-50.us-west-1.compute.internal
kube-system     kube-proxy-ip-10-0-0-96.us-west-1.compute.internal                1/1       Running   0          6d        ip-10-0-0-96.us-west-1.compute.internal
kube-system     kube-proxy-ip-10-0-0-97.us-west-1.compute.internal                1/1       Running   0          6d        ip-10-0-0-97.us-west-1.compute.internal
kube-system     kube-proxy-ip-10-0-0-98.us-west-1.compute.internal                1/1       Running   0          6d        ip-10-0-0-98.us-west-1.compute.internal
kube-system     kube-proxy-ip-10-0-0-99.us-west-1.compute.internal                1/1       Running   0          6d        ip-10-0-0-99.us-west-1.compute.internal
kube-system     kube-scheduler-ip-10-0-0-50.us-west-1.compute.internal            1/1       Running   0          6d        ip-10-0-0-50.us-west-1.compute.internal
kube-system     kubedash-3370066188-bowsi                                         2/2       Running   0          2d        ip-10-0-0-97.us-west-1.compute.internal

Problem
On querying the Heapster API I am not getting the correct number and value of data point (Timestamp, Value) pairs for some specific metrics.
Querying any metric for an interval of 5 minutes over a resolution of 10 seconds should return 29 or 30 data point pairs. For the cluster wide metrics such as cpu/usage_rate or memory/usage this is working fine. I have not posted the raw response to keep it brief but you can see that there are 29 values and I've verified that their timestamps are also fine according to the 10s resolution:
Request: http://localhost:8080/api/v1/proxy/namespaces/kube-system/services/heapster-custom/api/v1/model/metrics/cpu/usage_rate?start=2016-05-31T11:14:21-07:00&end=2016-05-31T11:19:21-07:00

Response:
cpu/usage_rate: [115 255 362 156 134 87 197 372 136 187 313 258 166 179 124 219 338 142 248 318 103 192 429 90 75 362 267 196 275]

Now doing the same for any node specific metric gives me a problem. So for cpu/node_utilization I get only 9-10 data points at max, even for an interval duration of 15 minutes. I’ve posted the raw and formatted response so you can see the timestamps as well
Request: http://localhost:8080/api/v1/proxy/namespaces/kube-system/services/heapster-custom/api/v1/model/nodes/ip-10-0-0-96.us-west-1.compute.internal/metrics/cpu/node_utilization?start=2016-05-31T11:14:21-07:00&end=2016-05-31T11:19:21-07:00

Response:
 {
  "metrics": [
   {
    "timestamp": "2016-05-31T18:17:00Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:17:10Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:17:20Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:17:40Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:18:00Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:18:10Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:18:30Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:18:40Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:18:50Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:19:10Z",
    "value": 0
   }
  ],
  "latestTimestamp": "2016-05-31T18:19:10Z"
 }

ip-10-0-0-96.us-west-1.compute.internal: [0 0 0 0 0 0 0 0 0 0]

As you can see there are only 9 data points returned. Moreover all of them are 0. This happens for all the nodes and also for memory/node_utilization. This cannot be the case since the cluster wide cpu and node utilization are clearly not zero.
I am also seeing some erratic behavior for the pod metrics as the values returned are right but they are fewer than the expected amount. So in this case for cpu/usage_rate I get 21 data points instead of 29. The values are correct since my nginx pod has no load on it currently but you can see from the timestamps that some values have been skipped such as the jump from 2016-05-31T18:15:20Z to 2016-05-31T18:15:50Z.
Request:
http://localhost:8080/api/v1/proxy/namespaces/kube-system/services/heapster-custom/api/v1/model/namespaces/default/pods/my-nginx-3800858182-cgh9i/metrics/cpu/usage_rate?start=2016-05-31T11:14:21-07:00&end=2016-05-31T11:19:21-07:00

Response:
 {
  "metrics": [
   {
    "timestamp": "2016-05-31T18:14:40Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:14:50Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:15:00Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:15:10Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:15:20Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:15:50Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:16:00Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:16:20Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:16:30Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:16:50Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:17:00Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:17:10Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:17:20Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:17:30Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:17:40Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:18:20Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:18:30Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:18:40Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:18:50Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:19:00Z",
    "value": 0
   },
   {
    "timestamp": "2016-05-31T18:19:10Z",
    "value": 0
   }
  ],
  "latestTimestamp": "2016-05-31T18:19:10Z"
 }

my-nginx-3800858182-cgh9i: [0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0]

I am not sure if the higher resolution is the problem since it seems to be working fine for cluster wide metrics. Any idea on how I might debug this?

By default cadvisor collect metrics every 10s so it's expected that metrics will be missed for pods when Heapster metrics_resolution is set to 10s. On the other hand aggregation across cluster requires at least one collected metric for some pod which is usually true.
In general if you are using default Kubelet setup you shouldn't decrease the the value of metrics_resolution. If you need to do it you should also change flag housekeeping_interval in Kubelet, but there is no guarantee it would work.


Unexpected char when inserting doc with Ü in uri

#1008 # Version of MarkLogic Java Client API
'com.marklogic', name: 'marklogic-client-api', version: '5.0.0'
Version of MarkLogic Server
10.0-2.1
Java version
java 12.0.1
OS and version
ProductName:	Mac OS X
ProductVersion:	10.14.6
BuildVersion:	18G1012
Input: Some code to illustrate the problem, preferably in a state that can be independently reproduced on our end
    GenericDocumentManager docMgr = DatabaseClientConfiguration.getInstance().getAdminClient().newDocumentManager();
    DocumentWriteSet writeSet = docMgr.newWriteSet();
    StringHandle handle = new StringHandle("{hello:\"test\"}");
    writeSet.add("/CXXXX_Ü_9999.json", handle);
    docMgr.write(writeSet);

Actual output: What did you observe? What errors did you see? Can you attach the logs? Exception in thread "main" java.lang.IllegalArgumentException: Unexpected char 0xdc at 42 in Content-Disposition value: attachment; filename="/CXXXX_Ü_9999.json"
Expected output: What specifically did you expect to happen?
document created in ML
Alternatives: What else have you tried, actual/expected?
Same issue with datamovementSDK writeBatcher
Works properly with:

TextDocumentManager
curl on /v1/documents
documentInsert()

We plan to fix this bug in the next version of the Java API, which should be released concurrent with MarkLogic 10.0-3 (currently scheduled for January).

Hi @jeremybrunetML . I was able to duplicate the error and it comes when we use TextDocumentManager as well.

Fixing this bug requires a change to the REST API (tracked with internal bug 53915) as well as the Java API.
On the Java API side, it will be necessary to check the server version to determine the syntax supported for the Content-Disposition header.

Internal bug 53915 is now committed on the trunk, 10, and 9 branches of the REST API.
IMPORTANT: Because the Java API fix depends on a REST API fix, the Java API can only support UTF-8 characters in document URIs for server releases starting with 9.0-12 and 10.0-3
When writing a batch, the Java API should check whether a document URI consists entirely of characters from the US-ASCII character set.
If so, the Java API should continue quote the document URI in the current approach for backward compatibility.
If not, the Java API should percent-encode the document URI according to this format:
filename*=UTF-8''THE_PERCENT_ENCODED_DOCUMENT_URI;

as in
filename*=UTF-8''%2FCXXXX_%C3%9C_9999.json;

One way to check the character set is to get an encoder for the US-ASCII character set (which cannot be shared across threads):
CharsetEncoder asciiEncoder = java.nio.charset.StandardCharsets.US_ASCII.newEncoder()
Then, call asciiEncoder.canEncode(THE_DOCUMENT_URI) to do the check.  Be sure to call asciiEncoder.reset() between each call to canEncode().
An alternative technique for checking would be to construct a java.util.regex.Pattern that matches against the POSIX ASCII character class from the start of the value to the end of the value.
The encoding can be implemented with URLEncoder.encode(THE_DOCUMENT_URI, "UTF-8")

Test added to DataMovement test StringQueryHostBatcherTest
 @Test
  public void testUTF8InUri() throws Exception
  {
        System.out.println("Running testUTF8InUri");
        DatabaseClient clientTmp = null;
        DataMovementManager dmManagerTmp = null;

        try {
          System.out.println("Running testUTF8InUri");

          String[] filenames = { "constraint1.xml", "constraint2.xml", "constraint3.xml", "constraint4.xml", "constraint5.xml" };
          String combinedQueryFileName = "combinedQueryOptionJSON.json";

          clientTmp = getDatabaseClient("eval-user", "x", getConnType());
          dmManagerTmp = clientTmp.newDataMovementManager();

          QueryManager queryMgr = clientTmp.newQueryManager();
          String dataFileDir = dataConfigDirPath + "/data/";
          String combQueryFileDir = dataConfigDirPath + "/combined/";

          // Use WriteBatcher to write the same files.
          WriteBatcher wbatcher = dmManagerTmp.newWriteBatcher();

          wbatcher.withBatchSize(2);
          InputStreamHandle contentHandle1 = new InputStreamHandle();
          contentHandle1.set(new FileInputStream(dataFileDir + filenames[0]));
          InputStreamHandle contentHandle2 = new InputStreamHandle();
          contentHandle2.set(new FileInputStream(dataFileDir + filenames[1]));
          InputStreamHandle contentHandle3 = new InputStreamHandle();
          contentHandle3.set(new FileInputStream(dataFileDir + filenames[2]));
          InputStreamHandle contentHandle4 = new InputStreamHandle();
          contentHandle4.set(new FileInputStream(dataFileDir + filenames[3]));
          InputStreamHandle contentHandle5 = new InputStreamHandle();
          contentHandle5.set(new FileInputStream(dataFileDir + filenames[4]));

          wbatcher.add(filenames[0], contentHandle1);
          wbatcher.add(filenames[1], contentHandle2);
          wbatcher.add(filenames[2], contentHandle3);
          wbatcher.add(filenames[3], contentHandle4);
          wbatcher.add("/CXXXX_Ü_testqa.xml", contentHandle5);

          wbatcher.flushAndWait();

          // get the combined query
          File file = new File(combQueryFileDir + combinedQueryFileName);

          // create a handle for the search criteria
          FileHandle rawHandle = (new FileHandle(file)).withFormat(Format.JSON);
          // create a search definition based on the handle
          RawCombinedQueryDefinition querydef = queryMgr.newRawCombinedQueryDefinition(rawHandle);

          StringBuilder batchResults = new StringBuilder();
          StringBuilder batchFailResults = new StringBuilder();

          // Run a QueryBatcher on the new URIs.
          QueryBatcher queryBatcher1 = dmManagerTmp.newQueryBatcher(querydef);

          queryBatcher1.onUrisReady(batch -> {
            for (String str : batch.getItems()) {
              batchResults.append(str)
                      .append('|');
            }
          });
          queryBatcher1.onQueryFailure(throwable -> {
            System.out.println("Exceptions thrown from callback onQueryFailure");
            throwable.printStackTrace();
            batchFailResults.append("Test has Exceptions");
          });

          dmManagerTmp.startJob(queryBatcher1);
          boolean bJobFinished = queryBatcher1.awaitCompletion(3, TimeUnit.MINUTES);

          if (bJobFinished) {

            if (!batchFailResults.toString().isEmpty() && batchFailResults.toString().contains("Exceptions")) {
              fail("Test failed due to exceptions");
            }

            // Verify the batch results now.
            String[] res = batchResults.toString().split("\\|");
            assertEquals("Number of reults returned is incorrect", 1, res.length);
            assertTrue("URI returned not correct", res[0].trim().contains("/CXXXX_Ü_testqa.xml"));

            // Read the document and assert on the value
            DOMHandle contentHandle = new DOMHandle();
            contentHandle = readDocumentUsingDOMHandle(clientTmp, "/CXXXX_Ü_testqa.xml", "XML");
            Document readDoc = contentHandle.get();
            System.out.println(convertXMLDocumentToString(readDoc));

            assertTrue("Document content returned not correct", readDoc.getElementsByTagName("id").item(0).getTextContent().contains("0026"));
            assertTrue("Document content returned not correct", readDoc.getElementsByTagName("title").item(0).getTextContent().contains("The memex"));
            assertTrue("Document content returned not correct", readDoc.getElementsByTagName("date").item(0).getTextContent().contains("2009-05-05"));
          }
        } catch (Exception e) {
          System.out.println("Exceptions thrown from testUTF8InUri");
          System.out.println(e.getMessage());
          fail("testUTF8InUri mathod failed");
        } finally {
          clientTmp.release();
          clearDB();
        }
      }

The regression isn't specific to this bug -- see issue 1242 cited above.

Issue is fixed.


Reproducing the mIoU of 81.4

Hi, would you please define the config for mIoU of 81.4?
What is the cv_split, inference mode, the scales and the pre-trained model?
Thanks

You mean the number in the paper table 3? It is cv split 0, sliding inference, single scale. That model is for ablation study in the paper, so we don't provide its weight here in this repo. If you want to reproduce the score, you can use cityscapes_cv0_wideresnet38_nosdcaug.pth  as an approximation. This model has a single scale mIoU of 81.5.

@bryanyzhu  @MahdiehP can we close this issue ?

I'm fine closing it.


[Event Hubs Client] Stabilize Live Tests

The tests adopted from the standalone repository, which were designated as Live are not fully self-contained nor resilient, which has resulted in some unpredictable failures when run as part of the nightly runs.
In order to ensure that the nightly run is reliable, the Event Hubs results should be investigated and tests that are not reliable should be analyzed.

If the test is not current and is not needed to verify functionality of the service, it should be deleted.

If the test can be improved in a reasonable amount of time and effort, it should be.

If the test is needed and requires more than a reasonable effort, it should be flagged to be skipped and a separate issue opened to cover the work required to fix the test.

For ServiceFabricProcessor, most of these were fixed by #7446. EventException test has a separate issue with the error count being off by one.


Find sum of n natural numbers using formula

Description
formula: n(n+1)/2
Input  : 5
Output : 15

Program added successfully :tada:
Thanks for your contribution :hugs:


User access structure

[ ] define needed user-roles and their access rights.
[ ] check if possible with django (admin).
[ ] implement / configure accordingly

Thoughts from #145:
A single invitation link for each city would be really nice! That way, we do not need to get active for each individual user. Could not find anything like that, so far. Looked at:

https://github.com/jazzband/django-invitations (Fairly active, but individual links sent to emails directly, only.)
https://github.com/pennersr/django-allauth (Big. Social. Works together with django-invitations.)
https://github.com/caffeinehit/django-inviter (Pretty dead)
https://github.com/mjschultz/django-inviter2 (Fork of the above, but not as dead. Dont think it does what we want.)

Alternative scheme:
Central org admin creates one or two accounts per city with the authorization to create more users - but just for the same city. This might be hard to achieve. https://github.com/bennylope/django-organizations might help, though.

@mdrie @cafischer  @kleins05  @fblampe @patrick-local-zero and others. Please comment/discuss:
LocalMonitoring security model
We use the Django authentication system based on users --> groups --> permissions.
Django support permissions per model as well as per object instance (using @user_passes_test or UserPassesTestMixin).
See also:

Article https://testdriven.io/blog/django-permissions/
Framework https://django-guardian.readthedocs.io
Framework https://pypi.org/project/rules/

For each city we dynamically create the privileges:
- cpmonitor.view_city_[city slug]	"Can read unpublished data for the city"
- cpmonitor.change_city_[city slug]  "Can edit and publish data for the city"
- cpmonitor.delete_city_[city slug]  "Can delete the city"
- cpmonitor.admin_city_[city slug]  "Can manage users for the city"

And groups
- cpmonitor.city_admins_[city slug] (has privileges cpmonitor.{admin,delete,change,view}_city_[city slug])
- cpmonitor.city_writers_[city slug] (has cpmonitor.{change,view}_city_[city slug])
- cpmonitor.city_readers_[city slug] (has cpmonitor.view_city_[city slug])

We also use global privileges
- cpmonitor.add_city
- cpmonitor.view_city
- cpmonitor.change_city
- cpmonitor.delete_city
- cpmonitor.admin_city

For users that can access all cities.
And groups
- cpmonitor.global_readers  "Can read all unpublished city data"
- cpmonitor.global_writers  "Can edit all citys"
- cpmonitor.global_admins   "Can manage users and permissions"

Django also uses superusers having all privileges.

Roadmap
TBD:

[ ]  Instead of city slug we might want to use [city ID] to avoid issues when renaming a city.

Step 1:

[ ] Create Privileges & Roles for existing cities
[ ] Automatically create new Privileges when new city is added.
[ ] Restrict access in adminpanel for city/task/etc to corresponding privileges.
[ ] Create useraccounts for existing users

In this phase new users/cities will be created by superusers/global_admins only.
Step 2:

Existing Users with privilege cpmonitor.admin_city_[city slug] can create new users for their city.
Create new users using invite links.

This is a good start. I think, we have at least one user story, that is not covered:

A city admin needs to be able to add users, but only give them permission to change the city(ies) for which (s)he has admin permissions.

I think, we need a list of such user stories in order to communicate / agree on a authorization model that suits our needs. They should not reference any existing django structure.
But... I still need to read through the django docs before a final verdict. ...thereby preparing myself for implementing it...

This is a good start. I think, we have at least one user story, that is not covered:

A city admin needs to be able to add users, but only give them permission to change the city(ies) for which (s)he has admin permissions.

@mdrie  This should covered by the privlege
- cpmonitor.admin_city_[city slug]  "Can manage users for the city"

You are right! I overlooked, that you added "admin" to the standard permission list.

Okay, after reading through the docs of django.contrib.auth, django-guardian, and django-rules I come to the following conclusions:

None my reading suggest to dynamically create Permission objects like "cpmonitor.view_city_[city slug]".
It seems, there are roughly two ways to achieve, what we need:

Either: Add user IDs to City object and use that dynamically to check permissions.
Or: Add individual per-object/row permissions and use user groups to get organized. (in the spirit of your description, but with differences. See below.)

I think we should agree on a few user stories, but not here on GitHub, though. I will post something in German on wechange.
It seems to be a good idea to use a django app providing object-level permissions. There are a few choices:

django-guardian

allows object-level permissions which can be set with e.g. assign_perm("change_profile", user, profile)
Seems suitable for the approach number 2 above.
Last commit at start of 2022. A lot of open issues and pull requests... Seems to be unmaintained.

django-rules

allows object-level permission by defining them as functions which can dynamically decide based on object properties.
Seems suitable for approach number 1 above.
Few commits, but into 2023. Seems to be actively maintained. Documentation quite decent.

A few other/older apps are out there, but even less maintained.

I prefer approach number 2 and django-rules. It allows an easy maintenance of access rights from within the City. City admin users do not need to see / work with the user list and can simply add users to their city.
Open points:

[ ] Registration of users (By invitation link per city would be really nice!)
[ ] Restricting user management in the admin e.g. for city admins to just manage users for their city.
[ ] Agree on user stories.

@Holger-GZ @mdrie thanks for all your work and thinking. I think you're on the right track so no need for me to make a lot of comments. As always I think we should start as simple as possible. Maybe it is not even necessary to distinguish between viewing and changing.
I also prefer approach 2 in @mdrie's last post. I think it is better to keep authentication data separate and not blend it into core business models such as City.

@kleins05 Sorry, i got something mixed up: I prefer approach 1 (corrected in my comment above). After playing around with django-rules a bit, it is really simple and natural. Approach 1 seems to need much more structure.

Open points:

[ ] Tests for invitation link handling and registration.
[ ] Fix failed CI run.
[ ] Beautify registration page (maybe later...)
[ ] Plan thorough review with a team member. Possibly synchronously.


If-Modified-Since, If-None-Match Conditional Headers Support

https://developer.mozilla.org/en-US/docs/Web/HTTP/Conditional_requests#cache_update

Isn't it supported implicitly?
https://docs.aws.amazon.com/ja_jp/AmazonCloudFront/latest/DeveloperGuide/RequestAndResponseBehaviorS3Origin.html

WriteGetObjectResponse API: https://docs.aws.amazon.com/AmazonS3/latest/API/API_WriteGetObjectResponse.html#API_WriteGetObjectResponse_RequestSyntax does not respond to ETag or LastModified headers by default. Therefore, the current S3 Object Lambda implementation requires generating a CloudFront cache every time it expires.
Additionally, if If-Modified-Since, If-None-Match Conditional Headers are present in the request, use HeadObject API: https://docs.aws.amazon.com/AmazonS3/latest/API/API_HeadObject.html#API_HeadObject_RequestSyntax You'll need to check the S3 Returns 304 Not Modified response code.


MathExpressions.max/min support Date

Why the new feature should be added
I want get the max value from tow datetime field. But  MathExpressions.max/min only support number field. I think date that is Comparable type. Should support it.
How the new feature should work
Code like this:
DateTimeExpression<Date> createTimeExpression = MathExpressions.max(
                QTableName.tableName.createtime1,
                QTableName.tableName.createtime2);

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Hi I am trying to help out on this issue and wonder what's the difference between max and greatest? What is the feature supposed to do?
Thank you!

Greatest returns the biggest value of two values (so it takes two expressions as arguments). Max is an aggregate function that returns the greatest value for a single expession.


Changes required in Contact Us page

Change theme according to the home page. Use bright colours and light background. Make changes in navbar and make it as it is in home page.

I would like to work on the issue

I would like to work on the issue

I would like to work on this issue too

I would like to work on this issue too


Create add user modal

Authors:
What does this PR contain?
How did you test these changes?
Attach images (if applicable)
Closes #37

need to fix:

page is not rendering the navbar anymore
dependencies from yarn are breaking; maybe related to dev branch


Postgres: inspect: for foreign key deferrable initially deferred

Hello, seeing that a diff is generating this (using postgres)
ALTER TABLE "public"."order_items" ADD
 CONSTRAINT "fk_order_items_order_id" FOREIGN KEY ("order_id") REFERENCES "public"."orders" ("id") ON UPDATE NO ACTION ON DELETE NO ACTION;

While I have set this in the schema .sql :
constraint fk_order_items_order_id foreign key (order_id) references orders (id) deferrable initially deferred

I would like to contribute opening a PR :
Seems like this codes need to be impacted :

https://github.com/ariga/atlas/blob/856a70c3c532b3c2111b803bc2be6594bed74228/sql/postgres/diff.go#L232
https://github.com/ariga/atlas/blob/856a70c3c532b3c2111b803bc2be6594bed74228/sql/schema/schema.go#L328
https://github.com/ariga/atlas/blob/856a70c3c532b3c2111b803bc2be6594bed74228/sql/internal/sqlx/diff.go#L502

Could you give me some hint so that I can open a pull request ?
Thanks you :)

This commit adds the inspect/diff pieces you need. I'm not familiar with the tooling functionality that atlas does since I use this package like Ent does. If support in the tooling is needed to merge this, someone else will have to take over.

The issue was resolved with latest Atlas build, and it will be officially released in next version. You can define the deferrable attribute on the following blocks: foreign_key, primary_key, unique, and exclude. For example:
table "users" {
  // ...
  foreign_key "constraint_name" {
    columns     = [...]
    ref_columns = [...]
    deferrable  = INITIALLY_IMMEDIATE // Or: INITIALLY_DEFERRED
  }
}

See the HCL docs for more info: https://atlasgo.io/hcl/postgres#table.foreign_key-attributes


MAINT: Installation using setup.cfg

Is there something I can read to learn why setup.cfg is preferred for specifying installation instructions over setup.py? Or is it a general principle to like configuration files over python/code files?

Mostly the latter. I don't know of anything already written, but to sketch out the reasons:

For the sake of not repeating yourself, it's nice to have stuff in a single place. This led to a lot of things like __about__.py and info.py and version.py, which would define these things inside the project, and then be evaluated in setup.py with something like eval(open('<project>/info.py', 'r').read()). In practice, though, you do need to repeat yourself for a few things, unless you want to exert a major effort to build helper scripts that correctly propagate these things (and documentation to onboard new maintainers...). Also, you don't need most of the stuff inside your project, making it all weird, hacky and probably unnecessary.

setup.py is ridiculously configurable, as it can run literally any Python code. This has resulted in some pretty unmaintainable installation scripts, so reducing the dynamic parts that you can't avoid evaluating to a minimum and putting the rest in a static file is appealing. That setuptools has settled on a preferred format gives high confidence that it will continue working for a while.

There's also a chicken/egg problem when dependencies need to be installed. Some of this is addressed in (1), by evaling a file that has no dependencies, but it goes a bit deeper in terms of "What if I want to use an alternative build system to setuptools and you don't have it installed?" See PEP 518 for more on this.

If we move to a common convention, such as that proposed in setuptools and PEP 517, this increases the legibility of our build system to new contributors or just us later on. Even if nobody else hops on this bandwagon, we have about 6-10 projects now that we maintain, and at least keeping them consistent will help reduce our cognitive load.

I have been thinking recently that it would be very nice to have a best practices guide for things like this, and especially for how and when to transition to new systems as the old ones fade away without causing disruption, similar to how the Python language itself has end-of-life schedules for each minor version.


Flag for dumping HTTPS traffic?

I find it pretty useful to be able to inspect the raw HTTP request/response being issued by a client library. This makes it easy to add integration tests to my own code, by copying the JSON responses from Github into my tests, then setting up a httptest.Server to replay those responses.
Here's how I implement this in my own API client - you can set DEBUG_HTTP_TRAFFIC=true in the environment to dump all requests/responses. https://github.com/kevinburke/rest/blob/master/client.go#L110-L134
And here's where I am using a response captured by that logic, in a test: https://github.com/saintpete/logrole/blob/master/server/messages_test.go#L88
Thoughts on adding something similar to this library?

Can this be done via a middleware http.RoundTripper implementation? If so, it can be orthogonal from this specific library.

We have exactly such a roundtripper internally at Google.

@willnorris bah haha, I just added https://github.com/kevinburke/rest/commit/da85e327c2c8c3668c19c41e17e6ea5b11e8f843 as a public implementation!

Is there anything left here, or can we close this?

nope you can close it


covid_hosp_facility data from Aug 7, 2023 is corrupted

This has broken acquisition since it appeared.  The data file (https://us-dhhs-aa.s3.us-east-2.amazonaws.com/anag-cw7u_2023-08-07T12-59-15.csv) seems to end prematurely and abruptly with some json-formated error text in the middle of a line.  There is no mention of this on the dataset page (https://healthdata.gov/Hospital/COVID-19-Reported-Patient-Impact-and-Hospital-Capa/anag-cw7u/about_data).  It is probably worth following up with the dataset owners about it.
We are going to attempt to skip the import of this file by inserting a dummy record into the covid_hosp_meta table, but that does leave a gap in the data.  The SQL is:
INSERT INTO covid_hosp_meta (dataset_name, publication_date, revision_timestamp, hhs_dataset_id, acquisition_datetime, metadata_json)
VALUES ('covid_hosp_facility', 20230807, 'https://us-dhhs-aa.s3.us-east-2.amazonaws.com/anag-cw7u_2023-08-07T12-59-15.csv', 'anag-cw7u', '2024-01-01 00:00:00', '{"error":"malformed data file"}');

We wrote to the healthdata.gov team and got a response:

We have looked into this malformed auto archive version and it appears the download to create the archive version was interrupted and uploaded only partially complete. It won’t be possible to recover this specific version of the archived asset but it is an overall unlikely occurrence.

So we tried our fix -- the suggested row was inserted into the covid_hosp_meta table and the covid_hosp_facility acquisition job successfully completed. It did take ~7h to run, as it had to catch up on the backlog.


improve tests

especially with the generation of dates for spanning multiple weeks

delete tests folder and start again?


Feature request: allow CORS on local dev server

OS?  Mac OSX (El Capitan?)

Versions.
angular-cli: 1.0.0-beta.1
node: 5.10.0
os: darwin x64

Repro steps. I'm running a Spring-Boot server as my RESTful API and to serve up the initial html page. The html page then gets the angular app from a CDN. The Angular cli server acts like the CDN while we are developing. Both Spring and Angular cli are running on my local machine (with different ports). When the call to System.import() is made, I get a CORS error from the browser.

The log given by the failure.
XMLHttpRequest cannot load http://localhost:4200/system-config.js. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'http://localhost:8080' is therefore not allowed access.
zone.js:323 Error: Error: XHR error loading http://localhost:4200/system-config.js(…)

Mention any other details that might be useful.
I'm sure there must be a way to do this, but I've tried everything I can think of.

+1 vote for enabling to set custom Access-Control-Allow-Origin parameter for ng serve

+1 vote for this. We had to rely on express server + https://github.com/nodejitsu/node-http-proxy to make cross-origin HTTP requests.

To be able to make CORS http request I had to use the "--proxy" i.e ng server --proxy http://localhost:8080.

I actually solved this issue by using the ember-cli-cors add on. All you have to do is install it using the ng command like so:
ng install ember-cli-cors

ng install ember-cli-cors
didnt help. What am I missing?

@wholladay I presume that you meant:
npm install ember-cli-cors
and not
ng install ember-cli-cors?

@brocco No, I first tried npm install ember-cli-cors and that did not work. When I ran ng install ember-cli-cors it did the trick for me.

+1
@wholladay did you configure anything after ng install ember-cli-cors ?
Or change the ng serve command at all?
In my scenario I am trying to port an Angular 1 app where in Dev we had to run "C:\..\chrome.exe" --disable-web-security to access a SharePoint oData source.  We also have to login to SharePoint first to get authenticated.  Eventually the app is deployed to SharePoint and all that is no longer an issue.
It would be great to have a way to ng serve it that doesn't modify the deployables in any way.

I tried ng install ember-cli-corsand nothing changed for me.
Is there an additional step to do to make it work?

@jmb-mage I did not configure anything differently that I can remember, not did I change the ng serve command. It just kind of... well, started working and I moved on to other issues.

@wholladay as a workaround I ran the npm http-server onto the 'public' or dist folder directly.
It has these attributes:
http-server -o --cors

Superseded by https://github.com/angular/angular-cli/issues/2872, as this was for a pre-webpack version of the CLI.


Update changelog and dependencies for version 2.6

Description

This PR adds some missing labels in draft_release_utils. It also filters all PR that have the automatedPR label so that these pull requests don't appear in the changelog. I have also added the new release dates in the script, because when I ran the script for the first time I was confused with the amount of PR that appeared because the dates were quite old. So maybe it will be easier for the next release manager to have these dates set by the previous release manager.
I also changed the environment dependencies to install ESMValCore v2.6.*

Closes #2670
Link to documentation:

Before you get started

[ ] ☝ Create an issue to discuss what you are going to do

Checklist
It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

[ ] 🛠 This pull request has a descriptive title
[ ] 🛠 Code is written according to the code quality guidelines
[ ] 🛠 Documentation is available
[ ] 🛠 Tests run successfully
[ ] 🛠 The list of authors is up to date
[ ] 🛠 Any changed dependencies have been added or removed correctly
[ ] 🛠 All checks below this pull request were successful

New or updated recipe/diagnostic

[ ] 🧪 Recipe runs successfully
[ ] 🧪 Recipe is well documented
[ ] 🧪 Figure(s) and data look as expected from literature
[ ] 🛠 Provenance information has been added

New or updated data reformatting script

[ ] 🛠 Documentation is available
[ ] 🛠 The dataset has been added to the CMOR check recipe
[ ] 🧪 Numbers and units of the data look physically meaningful

To help with the number of pull requests:

🙏 We kindly ask you to review two other open pull requests in this repository

It seems like matplotlib.org is down and the documentation cannot finish the build

It seems like matplotlib.org is down and the documentation cannot finish the build

Do you know if there is a way to restart the documentation build without using another commit? Matplotlib seems available again...

Do you know if there is a way to restart the documentation build without using another commit?

Could not find anything in readthedocs but a dot was missing so I can justify another commit

Ok that worked, but can I release ESMValTool as the final version now or do I have to do the whole release candidates thing again?

Ok that worked, but can I release ESMValTool as the final version now or do I have to do the whole release candidates thing again?

Release candidates were not used for the Tool in past releases. I think you could release the Tool v2.6 on Monday 🚀 Just don't forget to use the same date in #2738


Jar packaging type not supported by install-apps

I am trying to build and deploy a Spring Boot jar on my local machine. I do not see a jar deployed to my server apps directory. When running maven with -X option I see the following output during the install-apps goal:
[DEBUG] The project artifact cannot be installed to a Liberty server because jar is not a supported packaging type.

@paulmccrady When using the Liberty Maven Plugin to deploy a Spring Boot jar, there are two things you need to specify.

In the pom.xml, you need to specify <installAppPackages>spring-boot-project</installAppPackages> in the <configuration> of the liberty-maven-plugin.

You need to provide a server.xml that includes the correct Spring Boot feature for your application (either springBoot-1.5 or springBoot-2.0). This file is typically located under src/test/resources.

Let me know if adding these resolves your issue. You may also want to check out our new Boost plugin that is in alpha stage (0.1.2) that simplifies deploying Spring Boot applications with Liberty/Open Liberty here: https://github.com/OpenLiberty/boost/tree/master/boost-maven

@paulmccrady After reading my comments above, were you able to resolve your issue?

@cherylking apologies for not coming back. Yes I got this working. Was a combination of bad configuration in pom and server xml files. Thanks.


I wish /examples/guestbook-go// where is guide?

I wish /examples/guestbook-go// where is guide?

Fixed by adding new 404.md functionality that knows to forward users to examples/ on kubernetes/kubernetes.