fix broken link to extensions page

a trailing slash is required, it seems

Cleaning up old PRs :grinning: :+1:


Idea: Add a lint that looks for un-remembered instantiation of types or functions whose names end in "State"

This would be similar to the new UnrememberedStateDetector in androidx, but be a warning lint to catch likely cases. @StateFactoryMarker works for functions, but it doesn't help with cases where people instantiate state classes directly.
Thoughts @chrisbanes @mrmans0n?

Sounds good to me! I thought we had something like that already? Or maybe it was an idea we had but never got around to implementing.

We have the view model factory lint but not sure there's anything else

Note for the future - could just be a modified version of the first-party lint https://cs.android.com/androidx/platform/frameworks/support/+/androidx-main:compose/runtime/runtime-lint/src/main/java/androidx/compose/runtime/lint/UnrememberedStateDetector.kt


[Feature]: manage access iplist to support client-side IP validity check

Is your feature request related to a problem? Please describe.
In some scenarios, it is necessary to restrict client requests to enhance system reliability and force clients to either voluntarily exit or fail to mount,manage a list of abnormal IPs on the server side to prohibit their access to the system.

commit：b525c104a5c1e99fde1a10250722de55f05d5c32


Git clone on the repo fails on windows because filenames contain ':'

I've been using the repo by adding dependency_overrides in my pubspec.yaml as suggested in this Issue https://github.com/juancastillo0/leto/issues/3.
My primary OS is Windows 10.
Because they contain a colon, the "Validation: ..." json files located in https://github.com/juancastillo0/leto/tree/main/leto_schema/test/validation/scenarios cause git clone to fail.
This is because a colon is an invalid character for a Windows filename.

Hi, thanks for the issue!
I think the last couple of commits should fix this, I already tested it in a Windows machine.
Please let me know if it isn't working for you or if there is more I can help you with.

Hi, thanks for the quick response.
The updates to those files has now fixed the issue.


YAMLFormatter: change standard quote to single

What the title says ;)

This doesn't change the Yaml at all, but it looks like a nice cleanup still, but you might have to find a bug here.


Support PyTorch set_per_process_memory_fraction

Summary
PyTorch allows a limit for GPU memory. This is useful, for example, when a GPU resource is shared.
set_per_process_memory_fraction(fraction, device=None): Set memory fraction for a process. The fraction is used to limit an caching allocator to allocated memory on a CUDA device. The allowed value equals the total visible memory multiplied fraction. If trying to allocate more than the allowed value in a process, will raise an out of memory error in allocator.

Proposal
This setting takes a percentage [0-1] and a device (optional). Use an environment variable alongside ENABLE_CUDA of the format CUDA_MEMORY_FRACTION where the value is 0.0-1.0 and passed to fraction. Additionally, if set, check and prefer CUDA_MEMORY_FRACTION_... variable(s), where the value is the same format, and the ... is passed to device for each variable found.
Questions

[ ] Is there a better name than CUDA_MEMORY_FRACTION/CUDA_MEMORY_FRACTION_...?
[ ] Do we need multiple device support initially? We don't seem to currently support device selection.

One use case is for AWS vGPU support so that multiple consumers of the vGPU device(s) don't assume they have exclusive rights to the full resource usage.


Adding provisionAfterExtensions property for extension sequencing

This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[x] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[x] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[x] My spec meets the review criteria:

[x] The spec conforms to the Swagger 2.0 specification.
[x] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

Automation for azure-sdk-for-js

Automation for azure-sdk-for-python

Can one of the admins verify this patch?

Automation for azure-sdk-for-node

Automation for azure-sdk-for-go

Hi @sirajudeens
Looks good to me, but could you give me more context about this change? Was it forgotten and always here from the beginning? Is this something you added recently to the two latest API version? etc. Because I'm not able to find it in real payload.
Thanks!
FYI @hyonholee

Discussed by Skype with @sirajudeens and in my understanding the node was already there, but not declared in both 2018-06-01 and 2018-10-01


unable to edit wiki

I'm unable to update this wiki page
https://github.com/google/closure-compiler/wiki/Debugging-Uncompiled-Source-Code
to reflect the new procedures discussed in unable to generate deps.js
Can you give me permission to update that page?

I was exploring if it was possible to open a PR for the github wikis, but it looks like the software that github wikis are built upon don't support this workflow: https://github.com/gollum/gollum/issues/265

I was exploring if it was possible to open a PR for the github wikis, but it looks like the software that github wikis are built upon don't support this workflow: https://github.com/gollum/gollum/issues/265

I made a few adjustments and posted it: https://github.com/google/closure-compiler/wiki/Debugging-Uncompiled-Source-Code/_compare/c09396780561d6cec64e0db5af392fa04599edbf...4b4fffc1a4814b39a6c57080bd06364d6865245d
I'll close this bug now, thanks for bringing the issue up!


If there is only one child, don't return an array

Rendering children as an array breaks some components, such as TouchableWithoutFeedback.
Fixes https://github.com/jrans/react-native-smart-scroll-view/issues/31.

Would be good to get this merged in, at the moment just stealing this for our fork thanks @RobinClowers !


Feature Request: add support for websockets in broker

Hello!
One more feature request: can you, please, add a support for websockets (standard and secure) in broker, so we can implement "MQTT over Websockets"-thing natively with SurgeMQ? Combined with Paho's JS library we can build very modern web applications with the benefits of MQTT.
Thanks!

Yeah, this would be nice.
Maybe this would be a good project to work off of for the websocket-specific stuff?

Ack.
Thanks for the requests, and apologies for the late ack. This is definitely on my radar to add.

+1

Pull request added: https://github.com/surgemq/surgemq/pull/19

@jcw @RangerMauve @bakirov Is there anyway you guys can take a look at @edwardchoh's PR and see if that works? Really appreciate all your help!

See @edwardchoh 's PR https://github.com/surgemq/surgemq/pull/19


Missing Function "detectFace"

Am I not sure, but this code uses function detectFace, i cant find the same func in functions.py
`#using DEEPFACE WRAPPER
from deepface.commons import functions
backends = ['opencv', 'ssd', 'dlib', 'mtcnn', 'retinaface']
img1 = functions.detectFace("img1.jpg", target_size=input_shape, detector_backend = backends[3])
img2 = functions.detectFace("img2.jpg", target_size=input_shape, detector_backend = backends[3])`
As per the file functions.py in line 96:
`def detect_face(img, detector_backend = 'opencv', grayscale = False, enforce_detection = True, align = True):
img_region = [0, 0, img.shape[0], img.shape[1]]

#----------------------------------------------
#people would like to skip detection and alignment if they already have pre-processed images
if detector_backend == 'skip':
	return img, img_region

#----------------------------------------------

#detector stored in a global variable in FaceDetector object.
#this call should be completed very fast because it will return found in memory
#it will not build face detector model in each call (consider for loops)
face_detector = FaceDetector.build_model(detector_backend)

try:
	detected_face, img_region = FaceDetector.detect_face(face_detector, detector_backend, img, align)
except: #if detected face shape is (0, 0) and alignment cannot be performed, this block will be run
	detected_face = None

if (isinstance(detected_face, np.ndarray)):
	return detected_face, img_region
else:
	if detected_face == None:
		if enforce_detection != True:
			return img, img_region
		else:
			raise ValueError("Face could not be detected. Please confirm that the picture is a face photo or consider to set enforce_detection param to False.")`

it is replaced with preprocess_face function


Filling local service cluster with target port of service

Please describe the Improvement and/or Feature Request

In cds/cluster.go#getLocalServiceCluster, filling endpoints of load assignment with <IP_ADDRESS> and service's target port instead of endpoint.
Scope (please mark with X where applicable)

New Functionality          [ ]
Install                    [ ]
SMI Traffic Access Policy  [ ]
SMI Traffic Specs Policy   [ ]
SMI Traffic Split Policy   [ ]
Permissive Traffic Policy  [ ]
Ingress                    [X]
Egress                     [ ]
Envoy Control Plane        [ ]
CLI Tool                   [ ]
Metrics                    [ ]
Certificate Management     [ ]
Sidecar Injection          [ ]
Logging                    [ ]
Debugging                  [ ]
Tests                      [ ]
CI System                  [ ]
Project Release            [ ]

Possible use cases

When mesh service started, the envoy will generate RDS_Inbound cluster with service's endpoints (port of endpoint is used to fill local cluster).
But without healthcheck success, there is no endpoint for service. This is antinomy because RDS_Inbound requiring endpoint to handle inbound http health check request, but without inbound handling healthcheck always failed.
This is for http probe only, it's ok for exec command probe.

@shashankram Is there any solution for this? With the workaround solution is to replace http probe to exec probe, which execute 'curl -sIk [http.schema]://localhost:[http.port]/[http.path]'. This should be done in injection phase and required curl command existing in docker image.
Another solution is fill the port of local cluster with the port in service, instead of endpoint.


DS-1965: Adds date fields to the form used to edit policies (JSPUI).

Transferring my comment from https://github.com/DSpace/DSpace/pull/915
Tested locally, +1.
I would like to see some more error handling (eg putting "201" into the date field currently leads to the date being set to 0201-01-01), but XMLUI doesn't do error handling either, so that is a separate issue.


Implement prototype attribute in body of classes

What I'd like to see implemented is a shortcut for declaring/defining something for a class prototype through its body. This existed on ECMAScript 4, with a prototype attribute before a declaration:
class URLRequest {

    prototype var contentType: string =
        'application/x-www-form-urlencoded';
}

It's not standard in today ECMAScript, but helpful, it helps code readability. I don't see troubles on repeating the existence of the prototype attribute.

Available today under alternate syntax
class URLRequest {
    contentType: string;
}
URLRequest.prototype.contentType = 'application/x-www-form-urlencoded';

@RyanCavanaugh When I set more prototype properties in this way I optionally shorthand the prototype getter (store URLRequest.prototype elsewhere), that's what made me think it'd be a little helpful feature.


Design discussion: Enabling maximal customizability and adaptability of composite strategies

This issue is for discussion of design choices for composite strategies (which can be potentially nested having more than 1 level of "leafs"). Getting the configuration is less a challenge than getting the strategies fully compatible with Hypothesis adaption capabilities. As example for the discussion we use the std_msgs/Point strategy (docs.ros.org - geometry_msgs/Point).

@Zac-HD No worries. I'll continue somewhen the next evenings. Thx a lot.

OK, I'm just going to do a stream-of-consciousness dump as I read builtin_msg_field_types.py:

Constant literals always make me nervous.  I'd define INT64_MIN_VALUE as  −1 * 2^63, rather than -9223372036854775808 - better to be obviously than non-obviously correct, and save the time readers will use to check for typos.
You don't need to define FLOAT64_MIN_VALUE and FLOAT64_MAX_VALUE; Hypothesis only generates 64-bit floats in the first place.
For strategies such as int8(), add a check that user-supplied bounds are in range.  This can be as simple as assert INT8_MIN_VALUE <= min_value <= max_value <= INT8_MAX_VALUE!  Never trusting user input can make life a lot easier later, and sprinkling this kind of assertion through internal code too is a habit which makes PBT more powerful.
Docstrings: strategies don't shrink values, the underlying engine produces smaller examples from the strategy.
If you're generating HTML or other docs from the docstrings, I'd link to the ROS docs on the corresponding field from each strategy.  If they also use Sphinx you can use the intersphinx extension to get a nice markup for this.

Overall: a solid start!  I don't know how different ROS1 and ROS2 are, but it might be worth factoring out fundamentals like this into a shared section.

notebooks/point.ipynb:  I think you're over-complicating this.  Instead of passing kwargs with which to construct a strategy, in Hypothesis we simply make you pass a strategy directly.  For example:
@composite
def point(draw, x=float64(), y=float64(), y=float64()):
    x_val, y_val, z_val = draw(x), draw(y), draw(z)
    # check that each value is valid for the respective field, e.g.
    assert isinstance(x_val, float), f'drew invalid x={x_val} from {x} for float64 field'
    # Obviously the check is more precise for sized integers!
    return _Point(x, y, z)

Customising one leaf of a deeply nested tree is kinda painful, but that's always going to be the case.  In Hypothesis, we just do our best to infer missing strategies, and trust that the user can assign parts to variables etc. if they really need to.  To some extent though, this kind of fiddly customisation really does indicate that PBT might be the wrong way to write the test in question.

I think this is off to a solid start!  The next things I'd do are:

Write strategies for more detailed message types.  These are the basic primitives that everything else is built on.
Write a function that can take a message type, and infer the corresponding strategy.  This is the key piece that will save your users a lot of time.  It also encourages high-quality code, by increasing the friction of functions that only accept some of the values representable by a message type.
Get users and listen to their feedback!  Probably a good idea to apply it to your own projects first, then ideally try it out at e.g. a makerspace where you can get live feedback, but at some point you've just got to release it into the wild and see what happens :smile:

Feel free to ask about any of this, or just ping me again whenever you want another round of comments!

Thanks a lot for the review!

Constant literals always make me nervous. I'd define INT64_MIN_VALUE as −1 * 2^63, rather than -9223372036854775808 - better to be obviously than non-obviously correct, and save the time readers will use to check for typos.
You don't need to define FLOAT64_MIN_VALUE and FLOAT64_MAX_VALUE; Hypothesis only generates 64-bit floats in the first place.

That's right.

For strategies such as int8(), add a check that user-supplied bounds are in range. This can be as simple as assert INT8_MIN_VALUE <= min_value <= max_value <= INT8_MAX_VALUE! Never trusting user input can make life a lot easier later, and sprinkling this kind of assertion through internal code too is a habit which makes PBT more powerful.

That's on my TODO list already. Sticking to an implementation like in the Hypothesis NumPy extension check_argument should do the job.

Docstrings: strategies don't shrink values, the underlying engine produces smaller examples from the strategy.

Added to the TODO list.

If you're generating HTML or other docs from the docstrings, I'd link to the ROS docs on the corresponding field from each strategy. If they also use Sphinx you can use the intersphinx extension to get a nice markup for this.

Hyperlinking the ROS and ROS2 types is on my TODO list already as well. (ROS and ROS2 do not use Sphinx and using intersphinx is not possible.)

Overall: a solid start! I don't know how different ROS1 and ROS2 are, but it might be worth factoring out fundamentals like this into a shared section.

You are totally right. I wanted to get the things done from a technical/implementation point of view first. As soon as I am sure which is shared, ROS1 only or ROS2 only this kind of refactoring is usually done in minutes. (But I am not sure right now that I did not miss something :wink: .)

notebooks/point.ipynb: I think you're over-complicating this. Instead of passing kwargs with which to construct a strategy, in Hypothesis we simply make you pass a strategy directly. For example:

@composite
def point(draw, x=float64(), y=float64(), y=float64()):
    x_val, y_val, z_val = draw(x), draw(y), draw(z)
    # check that each value is valid for the respective field, e.g.
    assert isinstance(x_val, float), f'drew invalid x={x_val} from {x} for float64 field'
    # Obviously the check is more precise for sized integers!
    return _Point(x, y, z)

Ah, right :smiley: . Customizing is way easier and more convenient with your suggested design. As long as one modifies the check assert isinstance(x_val, float), f'drew invalid x={x_val} from {x} for float64 field' to also accept strategies in general it should even be possible to use adapted strategies for x, y and z like e.g. x=float64().map(...), right?
I thought about to accept optional, callable keyword arguments like x_map=map(...) which would have been added to x=float64().map(...) to allow adaption of "leaf" strategies. Equivalent for filter() and flatmap()... But your design is way cleaner.

Customising one leaf of a deeply nested tree is kinda painful, but that's always going to be the case. In Hypothesis, we just do our best to infer missing strategies, and trust that the user can assign parts to variables etc. if they really need to. To some extent though, this kind of fiddly customisation really does indicate that PBT might be the wrong way to write the test in question.

I think if (detailed) adaption of "leaf" strategies is required or not depends much on the property a user wants to check. E.g. in case of ROS there should not be any need to adapt strategies to check the property "ROS node still alive". However there could be observers for ROS node output data in the future which could assert in-range data. In this case a user would want to define input ranges by using adapting strategies. Therefore I would like to provide as much as possible adaptability and let the user decide how to use it.

I think this is off to a solid start! The next things I'd do are:

Write strategies for more detailed message types. These are the basic primitives that everything else is built on.
Write a function that can take a message type, and infer the corresponding strategy. This is the key piece that will save your users a lot of time. It also encourages high-quality code, by increasing the friction of functions that only accept some of the values representable by a message type.
Get users and listen to their feedback! Probably a good idea to apply it to your own projects first, then ideally try it out at e.g. a makerspace where you can get live feedback, but at some point you've just got to release it into the wild and see what happens smile

I'll do so... :wink:

Feel free to ask about any of this, or just ping me again whenever you want another round of comments!

Thx a lot for your help! Would be great if you could help to solve the "adaptability" design issue (above). Then the most things should be clarified for now...

There are no docs specific to single types. ROS and ROS2 do not use Sphinx and using intersphinx is not possible.

Ouch - a wiki is better than nothing, but I wouldn't call it documentation :disappointed: (not versioned, untestable, etc).  A few "custom directives" for sphinx would be trivial to write (just a mapping of stubs to URL patterns) and might make linking easier?  Not urgent but good to know if you revisit later.

Customizing is way easier and more convenient with your suggested design [all args are strategies]. As long as one modifies the check assert isinstance(x_val, float) to also accept strategies in general it should even be possible to use adapted strategies for x, y and z like e.g. x=float64().map(...), right?

Yep, adapted strategies are just strategies to Hypothesis and will work as expected.  Note that the assertion is about the value drawn from the strategy, not the strategy itself - e.g. to ensure that you always get an in-range integer for a uint8 field - so this should not accept a strategy for x_val.  Of course x must be a strategy or draw(x) will fail loudly...

I would like to provide as much as possible adaptability and let the user decide how to use it.

Agreed!  We have a couple of design tactics for this:

Provide a solid layer of basics.  Users with really weird edge cases can build their strategy up from primitives like int8() all the way to complex messages.
Provide some easy ways to compose them, which only need things spelled out for edge cases.  For example, builds() will use type hints to infer a strategy for missing required arguments - by analogy, users of rospbt would need to provide the chain of branches down to the leaf, but not the whole tree!  IMO you can't really do better than this without compromising usability for the majority of non-pathological cases.
Provide a magic top layer from_message(msg, **kwargs) that infers everything except strategies given in kwargs.  This is also the implementation for (2) if it goes well!  For example:  given a "move" message of a pair of points (start, end) which must have z>=0... from_message(move, start=from_message(point, z=float64(min_value=0)), ...).  Note that in actual code I'd pull out a positive_altitude_point variable for easier reference!

So: adaptability first by composition and second by customisation, but never at the expense of the default case because that's where everyone will start, and where most time will be spent.

A few "custom directives" for sphinx would be trivial to write (just a mapping of stubs to URL patterns) and might make linking easier? Not urgent but good to know if you revisit later.

I will use intersphinx to hyper link to Hypothesis docs (strategies, etc.). Would you guys consider to deploy versioned hypothesis rtd docs for the major versions? According to the Hypothesis Changelog linking to minor and patch versions would imply massive maintenance overhead :smile: .

Note that the assertion is about the value drawn from the strategy, not the strategy itself - e.g. to ensure that you always get an in-range integer for a uint8 field - so this should not accept a strategy for x_val. Of course x must be a strategy or draw(x) will fail loudly...

Right. The assert is just fine.

Agreed! We have a couple of design tactics for this:

Provide a solid layer of basics. Users with really weird edge cases can build their strategy up from primitives like int8() all the way to complex messages.
Provide some easy ways to compose them, which only need things spelled out for edge cases. For example, builds() will use type hints to infer a strategy for missing required arguments - by analogy, users of rospbt would need to provide the chain of branches down to the leaf, but not the whole tree! IMO you can't really do better than this without compromising usability for the majority of non-pathological cases.
Provide a magic top layer from_message(msg, **kwargs) that infers everything except strategies given in kwargs. This is also the implementation for (2) if it goes well! For example: given a "move" message of a pair of points (start, end) which must have z>=0... from_message(move, start=from_message(point, z=float64(min_value=0)), ...). Note that in actual code I'd pull out a positive_altitude_point variable for easier reference!

So: adaptability first by composition and second by customisation, but never at the expense of the default case because that's where everyone will start, and where most time will be spent.

I totally agree. The first use case for rospbt will be to assert property "node alive" anyway which will not require customization at all. I will implement the composite strategies with 1-level depth leaf strategies according to your proposed design.
After checking out the builds() docs and source I (hope that I) understand how builds() helps and why one cannot do better :wink: . And how I actually "Provide some easy ways to compose them" is done with 3. ...
Something similar to hypothesis/extra/numpy.ps from_dtype() I guess. Ok, I understand how 3. helps in 2. now.

Thx for your time... again :+1:

I actually removed the versioned docs recently, because there were so many links that it broke the rtd styling (oops!). We won't put docs online for 1.x or 2.0, (libraries.io suggests that almost everyone is using Hypothesis 3) but once 4.x is out we'll leave the last 3.x docs up for a good length of time to enable an orderly transition.  I'd recommend just linking to the latest version though!
All the rest sounds great and I'm glad I could help :smile:

I actually removed the versioned docs recently, because there were so many links that it broke the rtd styling (oops!).

I can imagine :wink:

I'd recommend just linking to the latest version though!

:thumbsup:

All the rest sounds great and I'm glad I could help 😄

Thx a lot Zac.


undefined symbol: _Z8bind_guiv

I use python_examples/compile_dlib_python_module.bat build dlib.so,and copy dlib.so to
/usr/lib/python2.7/site-packages/
Logs:
[mersdk@SailfishSDK python_examples]$ python
Python 2.7.6 (default, May 09 2013, 12:34:56)
[GCC 4.8.3 20140106 (Mer 4.8.3-1)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import dlib
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ImportError: ./dlib.so: undefined symbol: _Z8bind_guiv

The same issue like http://sourceforge.net/p/dclib/discussion/442518/thread/c05888de/

Try using the 'pip install dlib' command.  Does that work?

@davisking
I had installed python2.7-devel,python3,boost
[root@Jolla nemo]# python3 -m pip install dlib
You are using pip version 6.0.8, however version 7.1.2 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
Collecting dlib
  Downloading dlib-18.17.100.zip (4.4MB)
    100% |################################| 4.4MB 18kB/s
    package init file 'dist/dlib/__init__.py' not found (or not a regular file)
Installing collected packages: dlib
  Running setup.py install for dlib
    Detected Python architecture: 32bit
    Configuring cmake ...
    -- The C compiler identification is GNU 4.8.3
    -- The CXX compiler identification is GNU 4.8.3
    -- Check for working C compiler: /usr/bin/cc
    -- Check for working C compiler: /usr/bin/cc -- works
    -- Detecting C compiler ABI info
    -- Detecting C compiler ABI info - done
    -- Check for working CXX compiler: /usr/bin/c++
    -- Check for working CXX compiler: /usr/bin/c++ -- works
    -- Detecting CXX compiler ABI info
    -- Detecting CXX compiler ABI info - done
    -- Could NOT find Boost
    -- Boost version: 1.51.0
    -- Found the following Boost libraries:
    --   python
    CMake Error at /usr/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:108 (message):
      Could NOT find PythonLibs (missing: PYTHON_LIBRARIES PYTHON_INCLUDE_DIRS)
      (Required is at least version "3.4")
    Call Stack (most recent call first):
      /usr/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:315 (_FPHSA_FAILURE_MESSAGE)
      /usr/share/cmake/Modules/FindPythonLibs.cmake:186 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)
      /tmp/pip-build-nnavp_5m/dlib/dlib/add_python_module:64 (FIND_PACKAGE)
      CMakeLists.txt:5 (include)
    -- Configuring incomplete, errors occurred!
    error: cmake configuration failed!
    Complete output from command /usr/bin/python3 -c "import setuptools, tokenize;__file__='/tmp/pip-build-nnavp_5m/dlib/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /tmp/pip-czyu8uuf-record/install-record.txt --single-version-externally-managed --compile:
    running install

    running build

    Detected Python architecture: 32bit

    Configuring cmake ...

    -- The C compiler identification is GNU 4.8.3

    -- The CXX compiler identification is GNU 4.8.3

    -- Check for working C compiler: /usr/bin/cc

    -- Check for working C compiler: /usr/bin/cc -- works

    -- Detecting C compiler ABI info

    -- Detecting C compiler ABI info - done

    -- Check for working CXX compiler: /usr/bin/c++

    -- Check for working CXX compiler: /usr/bin/c++ -- works

    -- Detecting CXX compiler ABI info

    -- Detecting CXX compiler ABI info - done

    -- Could NOT find Boost

    -- Boost version: 1.51.0

    -- Found the following Boost libraries:

    --   python

    CMake Error at /usr/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:108 (message):

      Could NOT find PythonLibs (missing: PYTHON_LIBRARIES PYTHON_INCLUDE_DIRS)

      (Required is at least version "3.4")

    Call Stack (most recent call first):

      /usr/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:315 (_FPHSA_FAILURE_MESSAGE)

      /usr/share/cmake/Modules/FindPythonLibs.cmake:186 (FIND_PACKAGE_HANDLE_STANDARD_ARGS)

      /tmp/pip-build-nnavp_5m/dlib/dlib/add_python_module:64 (FIND_PACKAGE)

      CMakeLists.txt:5 (include)

    -- Configuring incomplete, errors occurred!

    error: cmake configuration failed!

    ----------------------------------------
    Command "/usr/bin/python3 -c "import setuptools, tokenize;__file__='/tmp/pip-build-nnavp_5m/dlib/setup.py';exec(compile(getattr(tokenize, 'open', open)(__file__).read().replace('\r\n', '\n'), __file__, 'exec'))" install --record /tmp/pip-czyu8uuf-record/install-record.txt --single-version-externally-managed --compile" failed with error code 1 in /tmp/pip-build-nnavp_5m/dlib

You seem to be missing the python development files.  You need to install
those too to be able to compile python modules.

@davisking I had installed python3-dev,python2.7-dev and some development packages.
I can build out dlib.so .

Use dlib-0.18.17.tar.gz and disable SSE4 fixed this issue on sailfish os.
Though has other issue,i think it's time to close this.

@0312birdzhang Were you ever able to get this completely working. I am seeing the same issue at the moment with 18.17.100.

Just in case someone runs across this in the future. Diving a bit deeper it appeared that the bind_gui symbol from tools/python/src/dlib.cpp was getting included in the generated dlib.so even when I compiled with the DLIB_NO_GUI_SUPPORT flag. Since I had this flag disabled anyways I ended up commenting out the bind_gui definition and execution in tools/python/src/dlib.cpp and then recompiling.
I verified that the symbol was no longer present in my dlib.so by doing: nm dlib.so | grep bind_gui
At this point everything started to work like normal and I was able to import dlib in my python project.


2.0 - Multi-Image - placing images in same height and width rows

I have a collection of images that are different sizes, different aspect ratios, etc.  I am trying to display my images in multiple rows that are the same width and the same height with equal distance between each image.  I'm shooting for a "block" of images with the same border around each image - and not having much luck.
The number of images to be displayed will vary per call.  The last row of images wouldn't need to be aligned on the left and right at this time, meaning, if there is one image on the last row, it's not expected to be stretched the entire width of the other rows.
I'm using viewer.addTiledImage to add images to my display.  This method does not allow setting both the width and the height of each image.  For my first pass, I'm setting the height to 1 which gives me equal height rows and that's doing a great job of calculating the associated width. I can set the images N pixels apart so I get the equidistant margin that I desire.   I just have a jagged right side on the display.  I can't seem to coax OpenSeaDragon into making the width work out so the rows are equal width and the images align on both the left and right.
Here's what I'm trying and my approach:
Displaying multiple images that have different sizes (width/height):
Image1: 952/641
Image2: 10079/7559
Image3: 600/287
Image4: 1280/720
Image5: 620/388
Image6: 615/394
Using the aspect ratio for each image, total all aspect ratios - in essence, a total width.
Calculate how many rows I want to display the images in - in this case 3.
Average Row width = total aspect ratios/rows.
Assign images per row - as long as the total aspect ratio per row < Average Row width, add another image to the row.  This results in a jagged right side to the display.
So - using the above images, row 1 has 2 images, row 2 has 3 images, and row 3 has 1 image.
Attempt 1 was to go back through and adjust the width of each image so they could be equidistant apart (a margin between images in a row, and between rows).   Since Height is already set on the image, setting the width is not allowed.
Attempt 2 was to adjust the actual image width that is passed in the addTiledImage call, but that results in a different aspect ratio for the image, which causes OpenSeaDragon to expect different tiled images to be present in the DZI which not only causes errors when it attempts to access those tiles, but also changes the look of the image.
At this point, I'm thinking I have 2 options, either live with the jagged right edge, or change the spacing between images on a row.
Is there a way that I can tell OpenSeaDragon to display an image at X,Y and display it in a rectangle that is Width/Height and then zoom it from there without mangling the image's aspect ratio?
Thanks much!

I am not sure I understand how you are trying to lay out the images.
Is it different from what we have in the last 2 examples in this page:
http://openseadragon.github.io/examples/tilesource-collection/

It's similar to that, except I think what I want to do is control the viewport size for each image, not use a single viewport size for all images.
When I set the height to 1 for the addTiledImage call, OpenSeaDragon calculates the width for the viewport, which gives me a view that looks like:

|           |  |                         |  |         |

|                 |  |                         |  |                 |

What I would like to do is keep those rows all with the same height, like they are now, but "stretch" the images in the first row so they align on the left and right with the images on the second row - and still maintain the equal margin between images.
I believe in the example that you mentioned, all of my images would be presented in the same sized viewport, but wouldn't necessarily fill the viewport completely?   I think that since the image is centered within that viewport, some images would have margins within the tile and others would fill it due to their aspect ratio.  I would like the image to completely fill the viewport regardless of the aspect ratio of the image.
Does that make it clearer? If not - I can try again!

Well that didn't go well!   I'll use the preview button next time!

Here's an example of what I am trying to do.

Ha thanks, much clear with an example :+1:
You are definitely on good tracks by setting a fixed height of 1.
For the width, I suppose that in the provided example, the images are reorganized in a magic order letting them fit approximately (compensating by varying the margins a little). Finding the best order is definitely not an easy algorithm but sounds feasible as it looks similar to the knapsack problem https://en.wikipedia.org/wiki/Knapsack_problem
The other option is to distort the images which would probably looks ugly and is currently not supported by OpenSeadragon.

I created a plugin for that:
https://github.com/Pin0/openseadragon-justified-collection


Getting Lua Error

I keep getting this error in WOW using your addon.

Auctionator Installed and opened and option on main buy screen
I have ELVUI installed and your addon installed and when I have ELVUI Setting open the data texts under DataTexts: Panels: Datatext Panel (Left): Select Spirit in any of the DataTexts
then I get error:
I can send you more info if needed just let me know and how to get info and I will send you.
Lua Error
Message: ...ace\AddOns\D4KiR MoveAndImprove\elements/minimap.lua:249: Attempt to access forbidden object from code tainted by an AddOn
Time: Fri Dec 24 11:23:29 2021
Count: 1
Stack: ...ace\AddOns\D4KiR MoveAndImprove\elements/minimap.lua:249: Attempt to access forbidden object from code tainted by an AddOn
[string "=[C]"]: ?
[string "=[C]"]: in function GetParent' [string "@Interface\AddOns\D4KiR MoveAndImprove\elements/minimap.lua"]:249: in function MAIIsMMBTNSVisible'
[string "@Interface\AddOns\D4KiR MoveAndImprove\elements/minimap.lua"]:272: in function <...ace\AddOns\D4KiR MoveAndImprove\elements/minimap.lua:258>
Locals:

I can’t replicate this error.  I have no idea what D4KiRMoveAndImprove is but the only similarity I can see is you also have GSE installed? The information supplied says that if you have x mods installed and you do something in them you get an error and GSE is the cause?

BTW. Ace3 is a common library used by a number of addons.  The first instance loaded is the one used.  These errors are being thrown and the references to Ace3 are coming from GSE but if you disable GSE you will get these errors thrown by any other instance of Ace3 installed.  You need to take this up with Elv, Ace3 or D4KiR


It is not possible to open a link in dropdown placed in top bar when using Tor browser

The title describes it pretty well. Tor browser ignores click event and does not follow the link. Would it be possible to fix the JavaScript to make it compatible with Tor?

I noticed that the "Reveal Modal" also doesn't work in the Tor browser. Would be great to get this fixed.

Tor isn't officially supported by Foundation, we'd be happy to accept a PR if people can fix these issues though.


fix: follow-us section in Student's main page

📋 Description

Fixed the bug of "follow us" section not being displayed properly in Student's main page
🔨 Changes Made

Fixed the width of footer column in "main_page_style.css"

✅ Checklist
Before submitting the PR, please make sure you have completed the following:

[x] I have followed all the guidelines mentioned in CONTRIBUTING.md.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation (if applicable).
[x] I have tested it locally and it works fine.
[x] Any dependent changes have been merged and published in downstream modules.

🏷 Types of Changes
What type of changes does your code introduce? (Check all that apply)

[x] Bug fix (non-breaking change which fixes an issue) 🐛

[ ] New feature (non-breaking change which adds functionality) ✨

[x] UI enhancement (non-breaking change which enhances UI) 🎨

[ ] Documentation update 📚

Fixes #241

📸 Screenshots (if applicable)

Desktop/laptop

Tab

Phone

Thank you for contributing to Research Nexus! We look forward to reviewing your PR and getting it merged!

@Harshdev098 I have made the required changes. Please review it and add the 'hacktoberfest', 'hacktoberfest-accepted', 'gssoc-ext' and 'level3' tags.


YouTube app not responding for a few seconds when it comes to the foreground

when I have this module enabled, the YouTube app gets stuck for literally a few seconds whenever I switch to it. the video is stuck and the UI does not respond at all, for example scrolling through the comments is impossible. the audio does play though, even while the rest of the app is stuck.

the same thing happens while opening youtube videos from other apps

sorry for triple posting. I can't edit posts with the mobile client. what I I mean in the last post is that when I click on a youtube link inside a browser, or feedly, or any other app and that app opens YouTube using an intent, the YouTube app is also stuck for a few seconds in that scenario.
this is on a note 3 n9005 with stock 4.3
tell me if you need any more info like logcat output for example.

Having the same problem. Videos often freeze with unresponsive UI. Running stock Verizon LG G3.

No audio for me though. Video doesn't load at all. Had to uninstall.

Same thing happening here on my Moto X 2013 running 4.4.4. I found it very annoying so I also had to uninstall.

Same thing happening here on my Moto X 2013 running 4.4.4. I found it very annoying so I also had to uninstall.

Do you guys maybe use Flat Style Colored Bars module?
I had some lags in yt app recently neverthless if the YouTubeBackgroundPlayback was enabled or disabled.
I was investigating the problem and the probable cause was FSCB module which causes disturbing yt lags after some recent updates. The problem is discussed in the module thread on xda. Anyway, after disabling it I experience no lags or freezees.

No, it's definitely this YouTube module in my case.


feat(APIM-450): check file exists and does not exceed max size

Introduction
As part of APIM-138, we need to check if a file exists in DTFS Azure File storage and check that it does not exceed a maximum allowed size. This PR adds this functionality in the form of a temporary endpoint (the endpoint will ultimately be removed but the associated services, methods etc. will remain). Note that this functionality currently uses shared key auth with the Azure storage account; I'm still investigating whether the either of the other options, Azure AD or SAS auth, would be better options, hence this PR is a draft at present.
,
Resolution
I've added a dtfs module which handles sending/receiving requests to/from DTFS file storage using the Azure SDK, which also handles auth. I've also added a temporary file check module containing a file service which calls through to a service in the dtfs module and contains the logic about checking the file isn't too large and handling if it does exist.

Closed as draft PR, changes merged in under different branch/PR


Added backwards compatible way to do sanity checking on permissions and raise exceptions when you make a typo.

@bubenkoff fixed :)


Reorg the helper functions

Issue
There are some state transition helper functions are described in the spec especially. Reorg them together into helpers.py can help to improve readability.
Proposed implementation

Basically, follow the spec:

get_shuffling: has been implemented,
get_attesters_and_proposer: implemented in #16
get_crosslink_shards: #10
get_crosslink_notaries: #10

Add tests.

Closing due to updated spec


Exception Handling

Add the ExceptionNotification gem to capture application errors:
https://github.com/smartinez87/exception_notification

Targeting Slack as the potential notification channel.

@little9 @ewlarson should I go ahead and pull the exception notifications out and create a new issue to add Signal or App Signal?

Yes! Let's remove this gem in favor of the third party tools.


Get debug/test mode value correctly

Adds new helper to get test mode status, using new functions from ckantools for retrieving settings and debug state.
Fixes #87

coverage: 68.884% (+0.3%) from 68.615%
when pulling a758ceca1fc8d18fd5e2c8edb377d9e9cf383076 on ginger/debug
into 0506762e9d835de816d9b54ced7784c97af66318 on dev.


Add Chaper 0, 1, and 2 from foss-primer as requirements to litreview2

Where are these chapters from?

:grapes:


Fix concurrent push

The pushing step would fail if there are 2 or more repos do push at the same time to another repo, in that case we need to rebase the commit before pushing again.

@dmnemec please help me review this PR

Just saw the notification. I'll check this out this weekend when I get some time. Pretty sure I'll need to write a new test for this case.

Okay, checks are failing. This PR (and all others) should now run checks when you commit changes. Failure looks like it might be related to the branch names for the receiving repo, I'll have to dig into it. There's only a main and dev branch on that repo right now.


[BUG] failed create-parallel-world in databricks build

Describe the bug
seems related to https://github.com/NVIDIA/spark-rapids/pull/3976, failed both 301db and 311db when doing mvn package
rapids_databricks_nightly-dev-github build ID:203

[2021-11-02T01:19:00.709Z] [ERROR] Failed to execute goal org.apache.maven.plugins:maven-antrun-plugin:3.0.0:run (create-parallel-world) on project rapids-4-spark_2.12: An Ant BuildException has occured: The following error occurred while executing this line:

[2021-11-02T01:19:00.709Z] [ERROR] /home/ubuntu/spark-rapids/dist/maven-antrun/build-parallel-worlds.xml:57: The following error occurred while executing this line:

[2021-11-02T01:19:00.709Z] [ERROR] /home/ubuntu/spark-rapids/dist/maven-antrun/build-parallel-worlds.xml:67: exec returned: 1

[2021-11-02T01:19:00.709Z] [ERROR] around Ant part ...<ant antfile="/home/ubuntu/spark-rapids/dist/maven-antrun/build-parallel-worlds.xml" target="build-parallel-worlds" />... @ 9:123 in /home/ubuntu/spark-rapids/dist/target/antrun/build-main.xml

likely a dupe/ fixed by #3987


autoPlaySpeed Being Ignored?

#I am setting autoPlaySpeed to 5000 and it appears to be ignored since the slides are transitioning much faster. I've tried setting it it 15000 and it still has no effect. The other options I set are being honoured.
Here is a link to an S3 bucket link illustrating the problem: https://s3.amazonaws.com/aws-website-asgctdatavisualizationdevelopment-p93i0/speed-test.html
Here are the settings I am currently using:
$("#slide-container").slick({
          dots: false,
          infinite: true,
         slidesToShow: 1,
        slidesToScroll: 1,
       speed: 1000,
      autoplay: true,
     autoPlaySpeed: 10000
 });

Actually nevermind. There is clearly a plugin bug since changing autoPlaySpeed in the slick.js file fixes the problem.

the setting is autoplaySpeed not autoPlaySpeed


Slack Invite link is no longer active

What would you like to share?
Trying to join the Score slack. However the join link found both on the score website and the README of this repo just resolve to a page informing you that the invite link is no longer active. Would like to join the score slack!
Additional information
No

Hi @danijam, thanks for pointing this out! Here's a new invite link that should work for you: https://join.slack.com/t/scorecommunity/shared_invite/zt-2a0x563j7-i1vZOK2Yg2o4TwCM1irIuA. I'll look into updating the outdated link everywhere.


Create App Skeleton

Clone the repo then create maven project with 3 submodules as seen in the root of the Nexmo Spring Boot Starter repo. Only that is needed for this issue is the initial file structure and the pom files.
To make things simpler, you can copy and paste the pom files from the nexmo starter app into the corresponding pom file for this repo then make the necessary changes.
After making the changes run the mvn clean build command to make sure there are no build errors.

Please assign.

@yallen011 Looks like this isn't being worked on? Can I pick it up?

sure, you definitely can.


Create corresponding keys while importing xPub or creating Hot Wallet

When a wallet is created, it needs to have its corresponding key. Currently when you import xPub or create a Hot Wallet there are no corresponding keys created.

[x] For Hot Wallet - add the corresponding Mobile Key if not already added and subtitle for the key will be the wallet name

[x] Side point - Unknown Signer or Key should be called Unidentified Signer or Key

Single sig wallets takes care of these

Not relevant .


#8682: Add binary mod support for WH_B0

Issue #8682 [ #8680 ]
Binary MOD support in WH_B0
Details :

Test File : tests/tt_eager/python_api_testing/sweep_tests/pytests/tt_dnn/test_remainder.py

Comparison : comp_pcc

Working and Tested Range (for both inputs) : [ -100, 100 ]

Implementation details : Used Typecast to avoid precision issues in bfloat16 datatype

CI :

All post-commit tests - PASSED
[post-commit] Fast Dispatch unit tests - PASSED
[post-commit] Slow Dispatch unit tests - PASSED

All Post Commi Test - https://github.com/tenstorrent/tt-metal/actions/runs/9597499069 [PASSED]


Oklahoma

This looks to be a total re-write. The historical script was just getting data from ARCGIS for Tulsa county. But it looks like there is data for all counties now
https://looker-dashboards.oklahoma.gov/embed/dashboards/63

Thanks @zweissman!
This one looks like it will be a great second PR for someone. They offer downloadable files and it will hopefully be pretty easy to get it integrated once the PR author is familiar with the our scraper system


login is not checking the password

... you can use anything!

I remember not checking it on the token check/filter, but in order to get a token then the user & password needs to be checked. So the place to look into the code (assuming I dont grab this story) is the AuthenticationRestController.class /login method on the pathfinder-server & the authentication manager implementation

actually.... it's the CustomAuthenticationProvider.authenticate method.. That's the implementation but for some reason the password check isn't causing a failure. perhaps returning null on that method is not the correct approach...

Now shows an error message if you put the wrong password in.


Update Get-PendingReboot to latest version

Need to update Get-PendingReboot to the latest on https://gallery.technet.microsoft.com/scriptcenter/Get-PendingReboot-Query-bdb79542

PR #140 was merged.


Add Component System

Implement a component system that can take props, and each has their own scope, etc.

Added in v0.5.0


CFBundleIdentifier", Does Not Exist

this is my init new project  but it has error ...

** BUILD FAILED **

The following build commands failed:
	PhaseScriptExecution Install\ Third\ Party /Users/genilex3/Desktop/reactNative/reactthree/ios/build/Build/Intermediates/React.build/Debug-iphonesimulator/double-conversion.build/Script-190EE32F1E6A43DE00A8543A.sh
(1 failure)

Installing build/Build/Products/Debug-iphonesimulator/reactthree.app
An error was encountered processing the command (domain=NSPOSIXErrorDomain, code=2):
Failed to install the requested application
An application bundle was not found at the provided path.
Provide a valid path to the desired application bundle.
Print: Entry, ":CFBundleIdentifier", Does Not Exist

Command failed: /usr/libexec/PlistBuddy -c Print:CFBundleIdentifier build/Build/Products/Debug-iphonesimulator/reactthree.app/Info.plist
Print: Entry, ":CFBundleIdentifier", Does Not Exist

I bet you are using react-native 0.46 on mac...

#14604 this is it

You should try to remove react-native and reinstall the 0.44 version, it worked for me.

Hey Baptiste, how to remove the react-native and install the 0.44 version? please leme know.

everybody has this issue, and so far it's being ignored.

I've done exactly what @yozef said and it somehow worked.
@gopaldevra You might have some issues if you have already an app and try to downgrade it. If it is the case, they are mostly related to the folder structure.
Let me know if it works

https://github.com/facebook/react-native/issues?utf8=✓&q=is%3Aissue is%3Aopen CFBundleIdentifier

@Baptiste-Garcin     ,  no no no ,  I tried many version because of this error, a few days ago my project version is 45  , i do nothing in project  and when i  using run command it's wrong. then i tred version of 0.44.0 ,0.45.1, 0.45.0 , 0.46.1  but it also was wrong.

@fanyuexiang  啥意思 , 你说的那个已经关闭了啊, 也没有成功的解决方案啊 ? 我猜你看得懂中文

@wei63w 把"package.json"里面"react"换成这个版本, "16.0.0-alpha.6","react-native": "0.44.3",然后删除node_modules文件夹,重新npm install, 再react-native run-ios,如果还报错,你打开ios项目里面,target-buildphases-bundle react native code and images, 里面有个路径是写错了的,好像是script,正确的是"packager",你按照那个路径打开,把正确的路径写上去再运行试试

Thanks, @yozef, @Baptiste-Garcin, Now I am running the app through the Xcode and it's working.
So I am not using the command to run on the iOS.

@fanyuexiang    thanks for you , 额 谢谢啦, 跟你说的一样 那个shell会报错 写错东西了 script需要改成packager ,

Duplicate of #14423

I had the same problem, and I solved it with the react-native upgrade command. It asks you to reset several settings of the project like the Launch screen and etc.
I think this issue is caused by changing the name of the app name from within XCode instead from react-native project. Or at least that was what caused the issue in my case.

The solution by @vish-udp fixed the issue for me. I always set my build location relative to my current project but react-native doesn't seem to like that.

The solution for me was in the Xcode Build Location, under Xcode -> Preferences -> Locations, I changed the "Command Line Tools" from an old version to the latest version of Xcode 9.4 and it worked.

This happens for me whenever the React Native cache gets corrupted. Clearing cache and retrying works for me:
$ rm -rf ~/.rncache
$ rm -rf node_modules
$ npm install
$ react-native run-ios

I downgraded react-native to 0.52.0 and build work fine

I tried all the other suggestions multiple times infact, the only thing that worked for me was the downgrade to version 0.52.0!
My Xcode version is 8.3
yarn add<EMAIL_ADDRESS>or
npm i<EMAIL_ADDRESS>

Try to run your project by building it from XCode. It worked for me


Use crates for targets from git

This has a few advantages:
a) It enables continous fuzzing
b) It means you don't need to update them on every release
c) It lets you easily validate fixes as upstream commits them

Makes sense to me!
bors r+


Allow disable of auto image download while keeping attachements.

Hello!  I would love if there was an option that would allow you to disable the automatic image downloading feature while leaving the attachment downloads enabled.  The reason for this is some artists only put a preview image, or one of a set up, and then zip the rest.

Hi there,
thank you for your request, a function like this makes good sense in the scenario you described.
However, the development of the addon has been stalled in favor of other projects, and I'm unsure if and when I'll get back to it. Your request will remain open as a reminder of desired features to be implemented. In the meantime, the repository is of course open to pull requests of all sorts.
Cheers!


Typo in Book 1, Section 20.1.1

In section 20.1.1 (page 653), "we can use the represenation" should be "representation".

In section 20.1.1 (page 653); Representation is misspelled, "we can use the represenation" should be "representation".

fixed, thanks


[Question] Inquiry about the Status and Release Date of 'fcmV1Credential' Resource

API Spec link
https://github.com/Azure/azure-rest-api-specs/tree/main/specification/notificationhubs/resource-manager
API Spec version
2023-10-01-preview
Question/Query
Hello,
I'm seeking an update regarding the status of the 'fcmV1Credential' resource in the 'Microsoft.NotificationHubs namespaces/notificationHubs 2023-10-01-preview' pre-release version. With Google's impending discontinuation of GCM support on June 20, 2024, it's crucial to ensure compatibility with FCM as soon as possible.
Could you kindly provide information on when we can expect the stable endpoint for 'fcmV1Credential' to be released?
(ive opened this issue also somewhere, and have been redericted to this github project : https://github.com/MicrosoftDocs/azure-docs/issues/120662 )
Kind regards,
Sylber
Environment
No response

As the end of GCM support on June 20, 2024, is rapidly approaching, could you please provide an update on the status of the fcmV1Credential resource that will be released in the stable versio?


python37Packages.ftfy: 5.8 -> 5.9

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python37Packages.ftfy is: "Given Unicode text, make its representation consistent and possibly less broken"
meta.homepage for python37Packages.ftfy is: "https://github.com/LuminosoInsight/python-ftfy"
meta.changelog for python37Packages.ftfy is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 5.9 with grep in /nix/store/01gayb28za0fsb34xah982n751gdx6r1-python3.7-ftfy-5.9

directory tree listing: https://gist.github.com/373e023d6f3df00f63031de4980181eb

du listing: https://gist.github.com/e2d04ecce71a82d1b5da4e703af06740

Rebuild report (if merged into master) (click to expand)

24 total rebuild path(s)

6 package rebuild(s)

6 x86_64-linux rebuild(s)
6 i686-linux rebuild(s)
6 x86_64-darwin rebuild(s)
6 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python37Packages.ftfy
python37Packages.wordfreq
python38Packages.ftfy
python38Packages.wordfreq
python39Packages.ftfy
python39Packages.wordfreq

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/01gayb28za0fsb34xah982n751gdx6r1-python3.7-ftfy-5.9 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python37Packages.ftfy https://github.com/r-ryantm/nixpkgs/archive/7273800f1e796e98a34a417d621f7ecae09c218a.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/01gayb28za0fsb34xah982n751gdx6r1-python3.7-ftfy-5.9
ls -la /nix/store/01gayb28za0fsb34xah982n751gdx6r1-python3.7-ftfy-5.9/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package failed to build:

python39Packages.wordfreq

5 packages built:

python37Packages.ftfy
python37Packages.wordfreq
python38Packages.ftfy
python38Packages.wordfreq
python39Packages.ftfy

Maintainer pings
cc @sdll @aborsu for testing.

Result of nixpkgs-review pr 112993 at 7273800f run on aarch64-linux 1

1 package failed to build:

python39Packages.wordfreq

5 packages built:

python37Packages.ftfy
python37Packages.wordfreq
python38Packages.ftfy
python38Packages.wordfreq
python39Packages.ftfy

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

4 packages built:

python38Packages.ftfy
python38Packages.wordfreq
python39Packages.ftfy
python39Packages.wordfreq


Two-factor authentication

I use two-factor authentication. It should be a pretty easy add, and I'm working on a way to add it. If I get it working, is it okay if make a PR?

Of course it's ok


Fix Bson.opt not working with Bson[]

Pushed a modified fix which keeps calling opt on const(Bson) working and adds a unit test.


Merge feature/metrics branch to 2.4

Description
Cherry picked commits from feature/metrics to 2.4
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

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

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

Codecov Report

Merging #1234 (9751001) into 2.4 (b3a541a) will decrease coverage by 18.61%.
The diff coverage is 34.69%.

@@              Coverage Diff              @@
##                2.4    #1234       +/-   ##
=============================================
- Coverage     71.87%   53.26%   -18.62%
- Complexity      291      292        +1
=============================================
  Files            42      282      +240
  Lines          2311     9690     +7379
  Branches        240     2274     +2034
=============================================
+ Hits           1661     5161     +3500
- Misses          509     4358     +3849
- Partials        141      171       +30

Flag
Coverage Δ

dashboards-observability
47.65% <42.18%> (?)

opensearch-observability
70.35% <28.91%> (-1.52%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...ability/public/components/common/search/search.tsx
54.54% <ø> (ø)

...ces/saved_objects/event_analytics/saved_objects.ts
5.40% <0.00%> (ø)

...ensearch/observability/model/SavedVisualization.kt
60.73% <28.91%> (-33.17%)
:arrow_down:

.../public/components/custom_panels/helpers/utils.tsx
73.38% <31.81%> (ø)

...event_analytics/explorer/save_panel/save_panel.tsx
47.61% <45.45%> (ø)

...isualization_container/visualization_container.tsx
69.56% <66.66%> (ø)

...nents/event_analytics/home/saved_objects_table.tsx
75.60% <85.71%> (ø)

...hboards-observability/common/constants/explorer.ts
100.00% <100.00%> (ø)

...c/components/common/search/queries/data_queries.ts
100.00% <0.00%> (ø)

...mponents/visualizations/assets/chart_datatable.tsx
33.33% <0.00%> (ø)

... and 238 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Awesome!! 😎


Remove version: Jackett.Jackett version 0.21.97

404 - release deleted
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/Jackett/Jackett/releases/download/v0.21.97/Jackett.Installer.Windows.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 918)


Update turnstile FAQ and other pages to clarify how to use turnstile on sites that are proxied by Cloudflare

Which Cloudflare product does this pertain to?
Turnstile
Existing documentation URL(s)

https://developers.cloudflare.com/turnstile/migration/
https://developers.cloudflare.com/turnstile/frequently-asked-questions/

What changes are you suggesting?
I am using Cloudflare DNS/nameservers to proxy a few domains for DDoS protection. I recently set up an email server (listmonk.app) on a subdomain and enabled the cloudflare proxy to protect my IP from DDoS attacks.
My concern is that when people click the "unsubscribe" links they will be presented with a captcha puzzle and will close the tab and click "spam" instead. You can lower your security level to “Essentially off”, but that's not a good fix because it does it for the entire domain instead of just the subdomain that listmonk is installed on. I know that Cloudflare uses hcaptcha by default. I learned about turnstile and it seems to be the solution for this. But there is no simple checkbox in https://dash.cloudflare.com/ to switch a website or subdomain to turnstile instead of hcaptcha.
So I landed on the "migration from hcaptcha" page https://developers.cloudflare.com/turnstile/migration/migrating-from-hcaptcha/ since that seems to be the only option. Besides "inserting the Turnstile script snippet in your HTML’s <head> element", I have no clue what the rest of the instructions are telling me to do. Note that I have done nothing to set up hcaptcha. As far as I know, hcaptcha is built in and automatically enabled when you proxy a domain/subdomain with cloudflare.
After clicking on the recaptcha tab https://developers.cloudflare.com/turnstile/migration/migrating-from-recaptcha/ my understanding of these "migration" directions changed, and it now seems that they do not apply to my situation at all, and are rather for people who have explicitly set up these captchas on their website?
Can you please clarify the docs for people in my situation? One specific request would be to mention whether or not the default hcaptcha gets automatically disabled if someone in my case enables turnstile on their website. Or would that just run turnstile on top of hcaptcha?
There really should be an option in the https://dash.cloudflare.com/ to "use turnstile instead of hcaptcha", but I don't see any place to send feedback for that.
Additional information
No response

Howdy @MaximilianKohler, thanks for raising this issue. TL;DR, by using Cloudflare, you're getting Turnstile behavior on proxied hostnames by default.
I'll raise this to the team to help make that clearer in the documentation.
More details
The blog you're referring to is a bit dated (2020). We've since switched the default Cloudflare behavior to be Managed challenges (blog | docs).
All proxied hostnames use these Managed Challenges instead of CAPTCHAs. Managed challenges are designed to reduce the friction associated with hcaptcha or recaptcha and primarily use non-interactive challenges.
Turnstile also uses Managed Challenges. One of the main points of Turnstile as a product is to open this behavior up to folks who aren't using the CF proxy.
All that said, if you were worried about challenges triggering more often on a certain subdomain, you could:

Monitor your security events to see whether those challenges were issued.
Use a Configuration Rule to adjust your Security settings specifically for your email subdomain.

Let me know if that solved your question or issue. If I don't hear anything in a few days, I'll close this out.

Yes, thank you very much! That answered my questions and concerns very well. I would suggest adding that information to these pages:

https://www.cloudflare.com/products/turnstile/
https://dash.cloudflare.com/sign-up?to=/:account/turnstile
and various pages in the docs https://developers.cloudflare.com/turnstile/

This issue should only be closed after that, no?

As a team / repo, we'll consider this closed when it's updated in various places throughout our docs.
Can file internal tickets on the other ones.... but those are outside the scope of our team's work, so would just consider this issue "addressed" once we link a closing PR here (apologies earlier, got a bit click happy :) ).
cc: @patriciasantaana for the docs updates

@kodster28 @patriciasantaana Do you see any opportunities for additional docs per the above info? If not, I think this is good to be closed.

All of the information here has already been addressed either in Blog posts, or WAF docs.

This is not sufficient. We shouldn't be expected to spend hours reading through blog posts and other docs for something as simple as being informed that Cloudflare is currently using turnstile so if we have our domain/subdomain proxied there's nothing we need to do. The searches I did brought me to the blog post saying that Cloudflare switched from recaptcha to hcaptcha.
I don't see any problem with adding some text (to the locations I listed) saying "Domains that are proxied by Cloudflare already use turnstile by default so no additional configuration is needed".

however, further, Cloudflare runs all Challenges using Turnstile now days, so maybe that's where the confusion is.

Yes, that's it. I came across no such information until I opened this issue.


Add link to resource provider details page

The type column should have a link to something that helps customers construct the "type" property. I think the link I used is good, but something like https://docs.microsoft.com/en-us/azure/templates/ could work too.

@camillemarie : Thanks for your contribution! The author, @tfitzmac, has been notified to review your proposed change.

@camillemarie - thanks for this addition.
#sign-off


did_set_breakat() should be in optionstr.c

Problem:  did_set_breakat() should be in optionstr.c as 'breakat' is a
string option.
Solution: Move did_set_breakat() to optionstr.c.

thanks!


Add basic documentation functionality

Add a doc module and process the AST to gather and associate documentation comments with SV entities.

Basic documentation capability is ready now. I would suggest that we postpone any styling of the HTML and proper templating to separate issues and PRs.

This is incredibly nice! I agree on following up with separate MRs regarding stylistic improvements. Merging!


Hit Box is incredibly frustrating

How can we improve the collision detection of ships and land?

Fixed in d946a01


Binascii

After finishing this I found about PyByteInner.hex, PyByteInner.fromhex methods, I guess we should merge these impls? I'm not too fluent with Rust to be able to tell which of them is more efficient, though...

You can't name the test containing import binascii as binascii.py, due to the way Python import works. Consider renaming to stdlib_binascii.py as done for other similar cases.

You can't name the test containing import binascii as binascii.py, due to the way Python import works. Consider renaming to stdlib_binascii.py as done for other similar cases.

Actually it does work (As it also does in https://github.com/RustPython/RustPython/pull/948), I guess because the imports for builtin modules are attempted first?
Anyway I agree it's a bad practice and if there's a convention to use stdlib_*.py, so I'll use.

Codecov Report

Merging #957 into master will increase coverage by 0.03%.
The diff coverage is 77.77%.

@@            Coverage Diff             @@
##           master     #957      +/-   ##
==========================================
+ Coverage   64.78%   64.81%   +0.03%
==========================================
  Files          89       90       +1
  Lines       15994    16039      +45
  Branches     3623     3634      +11
==========================================
+ Hits        10361    10396      +35
- Misses       3239     3241       +2
- Partials     2394     2402       +8

Impacted Files
Coverage Δ

vm/src/stdlib/mod.rs
100% <100%> (ø)
:arrow_up:

vm/src/stdlib/binascii.rs
77.27% <77.27%> (ø)

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 0a683f2...322aff7. Read the comment docs.


Possible not to errorOnFail when watching?

It's really cool to break the processs when there's an error in order not to push it in production but when I'm doing a "watch", I don't want a fail on error. Is it possible?

The error is thrown only when a eslinted file is passed through the errorOnFail stream/step. You might either avoid using the stream when watching (preferable) or downgrading errors to warnings with a custom formatter (semi-hacky). How simple this is to do depends largely on how your tasks are arranged.

Sounds good to me, thanks. ;)


mention

in the following section of the kubeadm docs:
https://kubernetes.io/docs/setup/production-environment/tools/kubeadm/create-cluster-kubeadm/#control-plane-node-isolation
we should mention that users should optionally remove the known label:
exclude-from-external-load-balancers
https://kubernetes.io/docs/reference/labels-annotations-taints/#node-kubernetes-io-exclude-from-external-load-balancers
in cases of single CP node clusters.
similar was done for kind recently:
https://github.com/kubernetes-sigs/kind/pull/3507
/kind feature
/sig cluster-lifecycle

/retitle mention node.kubernetes.io/exclude-from-external-load-balancers on the kubeadm create cluster page

I will try to help with this.
/assign

I have raised PR #45057 to resolve this.


Library not work

when i run the project, the messages is show like that

Lookup the value of PI to six decimals (I think, maybe more) and replace that PI constant with your own PI variable.

@RaulMarquezInclan that's what the pi constant in the standard library is for lol
@akbarraihan this was written some time ago, and the dart language has changed a bit. The PI constant was redefined to just "pi". See here -- dart:math
This is a great opportunity to make a pull request! Please do not do what Raul suggested.

Is this issue going to be solved? I am in the need of using this library and would be happy if it was solved.

@RCHowell
Hi!
I prepared pull request with a solution of this issue
https://github.com/RCHowell/flutter_plot/pull/4

Latest revision has been uploaded
https://pub.dev/packages/flutter_plot/versions/0.0.3


Tools page sort dropdown does nothing

The sort dropdown gives alphabetical by name no matter which of the dropdowns I choose.
http://long-read-tools.org/tools.html?sort=techs

This problem seems to exist on the categories and technologies tabs too.

Fixed in the new facelifted database

Fixed in the new facelifted database


Documentation is always 0.0.1 version

We're missing autoversioning in our documentation

Here it says 'latest' https://robocop.readthedocs.io/en/latest/index.html but:

If you go deeper it will be 0.0.1
You can't go back to previous version in docs

@bhirsz I believe You need to tag it using GIT

@bhirsz I believe You need to tag it using GIT

Youre right, but I also forgot to remove hardcoded '0.0.1' in conf.py for my docs generator ;) If I will do any of two (either remove hardcoded or switch to git tags) the issue will be fixed


Anitube changes URL

Source information
Anitube 14.13
Source language
Português Brasil
Source new URL
Anitube.ninja
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated all installed extensions.
[X] I have opened WebView and checked that the source URL is not updated yet.
[X] I will fill out all of the requested information in this form.

AniTube continues to be anitube.vip, the site anitube.ninja is another site


Add width to the options and fix bug in the size setting

fixes #22
and adds a setting to set the width of the video tag

Thanks!


Fix around getting transaction id when sending RB from API -> Rogue

What's this PR do?

Changes the way we are grabbing the transaction id (X-Request-Id) from the header.
Changes a check to isset() to get rid of log messages.

How should this be reviewed?
Same as testing from this PR.
Any background context you want to provide?

When testing on Thor, it thought that it had a transaction id so skipped lines 365-370 and went straight to 373 to save to Phoenix.
I saw this error in the logs and realized that it wasn't grabbing the header correctly:

Checklist

[ ] Documentation added for new features/changed endpoints.
[ ] Tested on staging.
[ ] Pinged a PM if this is a larger PR that would benefit from some additional testing love
[ ] Post a message in #phoenix if this includes something that causes a rebuild!

👍


SpaceAroundOperators raised when using safe navigation with an array

SpaceAroundOperators is raised when safe-navigating a square brackets method (foo&[:bar]).

Expected behavior
I would expect SpaceAroundOperators to ignore the safe navigation operator.
Actual behavior
SpaceAroundOperators is raised and auto-correct produces:
foo & [:bar]

Steps to reproduce the problem
Using the Rails config. The following code produces the problem:
foo&[:bar]

RuboCop version
$ rubocop -V
0.54.0 (using Parser <IP_ADDRESS>, running on ruby 2.5.1 x86_64-darwin17)

Well. You aren't safe navigating a bracket method. 🙂
foo&[:bar]

Is this:
foo & [:bar]

What you meant to do was probably:
foo&.[](:bar)

Uhh, momentary insanity. My bad. :p

For what it is worth (and I get this in an old issue), I recently stumbled to this myself. I think foo&.[](:bar) is not good syntax to use, as it has much visual overhead. I stumbled to this because SafeNavigation cop wants to guard against foo["bar"], where IMHO an if is better here. But there is no way to disable the cop only for [] case as far as I see it. At least documentation makes no mention of it.


Managed Cloud SDK installation requires python

Trying to install the Managed Cloud SDK on a fresh Windows box fails as the installation requires Python.  Here's what we see on our console:
Installing/upgrading the Cloud SDK... (may take several minutes)
[Installing Google Cloud SDK]
Welcome to the Google Cloud SDK!

To use the Google Cloud SDK, you must have Python installed and on your PATH.
As an alternative, you may also set the CLOUDSDK_PYTHON environment variable
to the location of your Python executable.
Google Cloud SDK installer will now exit.
Press any key to continue . . .

and the process then hangs.

On the console, I see:
Feb 27, 2018 12:14:23 PM com.google.cloud.tools.managedcloudsdk.install.Downloader download
INFO: Downloading https://dl.google.com/dl/cloudsdk/channels/rapid/google-cloud-sdk.tar.gz to C:\Users\IEUser\AppData\Local\google-cloud-tools-java\managed-cloud-sdk\downloads\google-cloud-sdk.tar.gz

and nothing else.  Would be useful to have a little bit more logging info.

There is a windows specific archive with bundled python that I'll change this to.... on the pausing, I thought we ran everything with --quiet, but I'll look again

I guess all the test machines have python on the path.

I just noticed that on Windows, the following prompt appears even on successful installation and SdkInstaller.install() won't return.
Google Cloud SDK installer will now exit.
Press any key to continue . . .

I mentioned that to @loosebazooka yesterday on hangouts:

I think you need to run install.sh and install.bar with —disable-prompts: https://cloud.google.com/sdk/downloads#non-interactive_silent_deployment

--quiet should disable prompts from the script we're using, I'm going to have to ask the cloud sdk team about that too.

--disable-prompts is for the installer shell script on Linux: install_google_cloud_sdk.bash. Moreover, we don't even download or use that install script. We just call install.sh included in the downloaded archive.

I think we can use call install.bat.... to prevent windows from injecting this PAUSE ?

One workaround for the inject PAUSE is to make echo do the keystroke:
    script.add("cmd.exe");
    script.add("/c");
    script.add("@echo");
    script.add("|");
    script.add("install.bat");

This works in my testing. Not sure if it should be @echo | call install.bat" or if cmd /c is unnecessary.

The docs on —disable-prompts say it should work for the Windows install.bat too:

The install.sh/install.bat are simply wrappers around invoking Python on google-cloud-sdk/bin/bootstrapping.install.py.  Oddly, running install.sh —help doesn't list —disable-prompts, but does include —quiet, and install.bat —disable-prompts doesn't seem to do the right thing on Windows:
C:\Users\IEUser\AppData\Local\Google\Cloud SDK\google-cloud-sdk>install.bat --disable-prompts
Welcome to the Google Cloud SDK!
usage: install.py [-h] [--usage-reporting USAGE_REPORTING] [--rc-path RC_PATH]
                  [--command-completion COMMAND_COMPLETION]
                  [--path-update PATH_UPDATE] [--disable-installation-options]
                  [--override-components [OVERRIDE_COMPONENTS [OVERRIDE_COMPONENTS ...]]]
                  [--additional-components ADDITIONAL_COMPONENTS [ADDITIONAL_COMPONENTS ...]]
                  [--quiet]
install.py: error: unrecognized arguments: —disable-prompts

But looking at the install.bat, it does invoke PAUSE but does check for some variables:
echo %CmdCmdLine% | find /i "%~0" >nul
SET INTERACTIVE=%ERRORLEVEL%
[…]
IF _%INTERACTIVE%_==_0_ (
  IF _%CLOUDSDK_CORE_DISABLE_PROMPTS%_==__ (
    echo Google Cloud SDK installer will now exit.
    PAUSE
  )
)

It looks like setting CLOUDSDK_CORE_DISABLE_PROMPTS=1 before running install.bat should do the trick.

Another possibility would be to invoke bin/bootstrapping/install.py directly.


Create Linux installation script

do something like this curl -fsSL https://tailscale.com/install.sh | sh  & src

https://github.com/progrium/httpfs#install


facing issues while running tfrecord code:

facing issues while running tfrecord code:
!python ~/models/research/object_detection/dataset_tools/create_pet_tf_record.py --label_map_path=label_map.pbtxt --data_dir=. --output_dir=. --num_shards=1
ERROR
/root/datalab
Traceback (most recent call last):
File "/root/models/research/object_detection/dataset_tools/create_pet_tf_record.py", line 318, in
tf.app.run()
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py", line 125, in run
_sys.exit(main(argv))
File "/root/models/research/object_detection/dataset_tools/create_pet_tf_record.py", line 271, in main
label_map_dict = label_map_util.get_label_map_dict(FLAGS.label_map_path)
File "/usr/local/lib/python2.7/dist-packages/object_detection-0.1-py2.7.egg/object_detection/utils/label_map_util.py", line 164, in get_label_map_dict
label_map = load_labelmap(label_map_path)
File "/usr/local/lib/python2.7/dist-packages/object_detection-0.1-py2.7.egg/object_detection/utils/label_map_util.py", line 138, in load_labelmap
label_map.ParseFromString(label_map_string)
google.protobuf.message.DecodeError: Error parsing message
@RomRoc any help will be great .
Originally posted by @sam-ai in https://github.com/RomRoc/objdet_train_tensorflow_colab/issues/3#issuecomment-449877789

Check this it's working but Done all in a hurry 💯 : https://colab.research.google.com/drive/1U6naXzr5F9HyNkFZBZlUHOsqk4uYxzJP

Change this -> !echo "item {\n id: 1\n name: 'dog'\n}" -> label_map.pbtxt to !echo "item { id: 1 name: 'dog'}" > label_map.pbtxt, tfrecord will work


add docstrings

Codecov Report

Merging #12 into master will decrease coverage by 0.59%.
The diff coverage is 88.88%.

@@            Coverage Diff            @@
##           master      #12     +/-   ##
=========================================
- Coverage   90.86%   90.26%   -0.6%
=========================================
  Files          11       11
  Lines         372      380      +8
  Branches       51       51
=========================================
+ Hits          338      343      +5
- Misses         20       22      +2
- Partials       14       15      +1

Impacted Files
Coverage Δ

aiomisc/utils.py
92% <88.88%> (-0.4%)
:arrow_down:

aiomisc/periodic.py
75.67% <0%> (-5.41%)
:arrow_down:

Continue to review full report at Codecov.

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

Pull Request Test Coverage Report for Build 117

8 of 9 (88.89%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.5%) to 92.434%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

aiomisc/utils.py
8
9
88.89%

Files with Coverage Reduction
New Missed Lines
%

aiomisc/periodic.py
1
80.85%

aiomisc/utils.py
1
94.03%

Totals

Change from base Build 110:
-0.5%

Covered Lines:
358

Relevant Lines:
380

💛  - Coveralls


Customising overflow position for Text Widget

Currently, Text widget supports only limited options for painting overflowed text.
Can we customize the overflow options to support custom truncation positions eg. before @ for emails? For reference, iOS SDK also supports three different types of modes.
Other open issues related to this -

https://github.com/flutter/flutter/issues/45336
https://github.com/flutter/flutter/issues/26748

Internal bug - b/180975303

Hi @Anushi1998
The iOS types are covered in https://github.com/flutter/flutter/issues/45336.

Can we customize the overflow options to support custom truncation positions eg. before @ for emails?

Can you please elaborate how this will work for text with more than one @? assuming that the text is not a Email.
Thank you

With our current requirements of feature, it is certain that there is only one @. Also, I would assume flutter to delegate choosing ellipsis place to the invoker, so cases of multiple @ can be handled gracefully.

Duplicate of https://github.com/flutter/flutter/issues/45336

I think this is distinct from #45336.
That issue asks for a standard set of three options: ellipsis at start, middle, or end. This one asks to go beyond that to more arbitrary flexibility:

custom truncation positions eg. before @ for emails

Not that I personally have an idea for a good API for providing that feature, though. Given the level of activity on this issue, it might just hang out at P2 or P3 indefinitely, and that's fine.


Alphabet dividers

Hello everyone,
want an extra PR for Hacktoberfest?  You have come to the right place!
We need dividers for each letter of the alphabet.  Here are the requirements:

[X] Need a divider for every letter of the alphabet
[X] 12 of that letter per divider
[X] Dividers need to be numbered along with existing ones

Important: make sure to open a pull request on the alphabet_divs branch, not master!
Thanks and happy Hacktoberfest!

I can work on this. Would you like to have capitalized and lowercased letters?

WIP: Fixed by #39


feat: update to Angular 18

Upgrade the packages to the latest version of Angular.
ran

ng update @angular/cli
✔ for migrating to new build system

manual update version

Version has been published to npm, thank you for your merge request.


chore: remove published charts

This PR

⚠ Before merging this change the Github Pages branch setting to gh-pages here
Removes published charts (to be served from the gh-pages branch).
Once this is merged a PR will be created to gh-pages merging main and re-adding the removed charts.
Related Issues

#219
Notes

Follow-up Tasks

How to test

As long we are 100% sure the gh-pages branch has been deployed with the right packages!

We'll change the pages setting to point to gh-pages and verify things work before merging this.


ping6 fails with warning and error

Ping6 command fails to work properly. See below for error details:
root@hostname:~# ping  google.com
PING google.com (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from ord36s01-in-f14.1e100.net (<IP_ADDRESS>): icmp_seq=1 ttl=55 time=17.0 ms
64 bytes from ord36s01-in-f14.1e100.net (<IP_ADDRESS>): icmp_seq=2 ttl=55 time=17.6 ms
^C
--- google.com ping statistics ---
2 packets transmitted, 2 received, 0% packet loss, time 1000ms
rtt min/avg/max/mdev = 17.036/17.338/17.640/0.302 ms
root@hostname:~# ping6 google.com
WARNING: your kernel is veeery old. No problems.
setsockopt(RAW_CHECKSUM) failed - try to continue.setsockopt(ICMP6_FILTER): Invalid argument

@SecurityWaffle - Thanks for trying out WSL and the feedback. Yes, we are aware of this issue and are tracking it internally.

I'm having a bit different issue, still about the ping and ping6 commands:
gregor@HOMEPC: ~
$ ping www.google.com
ping: icmp open socket: Permission denied
gregor@HOMEPC: ~
$ ping6 www.google.com
ping: icmp open socket: Permission denied

@lordgreg ping requires running bash.exe as admin, ping6 is on the backlog.

ping6 works in 14926, which is now publicly available for Insiders. And also starting build 14926, thanks to the Windows networking team, admin is no longer a requirement for ping.


'convert' returns dynamic Ruby Interface instead of String when converting to String

The following example converts an AsciiDoc source to a String, but has 2 problems:
  var options = Options.builder()
      .catalogAssets(true)
      .safe(SafeMode.UNSAFE)
      .build();
  var doc = asciidoctor.convert(Files.readString(sourceDocument.toPath()), options, Document.class);

The returned internal type is a dynamic coerced interface (in the example org.jruby.gen.InterfaceImpl682315553)

The returned object wraps in fact an String but we asked for Document.class.

Both happen because we don't validate the types, but just delegate in jRuby https://github.com/asciidoctor/asciidoctorj/blob/0240f02d54268bc484d6da210d0803c0156578e0/asciidoctorj-core/src/main/java/org/asciidoctor/jruby/internal/JRubyAsciidoctor.java#L314.
I think we should:

Validate that the expected type and the returned one are compatible. Not sure if we should return null or return exception :thinking: What do ppl think? Or should we just cover known cases and leave the current behavior as lat resort?
Never return a dynamic JRuby interface.

It was not my idea to close this yet. The latest fixes prevent returning invalid types but I still think it's worth thinking deeply about how we should expose the convert/load methods in a more Java-like way.


generating client libs creates multiple versions of the same object when its used in multiple endpoints

http://stackoverflow.com/questions/28527698/how-do-i-share-entity-objects-between-multiple-endpoints-in-google-app-engine-us

Yeah this is a consequence of the generation service.  It actually generates and archive for each endpoint.  The gradle plugin merges those in to one archive but preserves all the package paths.  I think a way around this is to manage your API as one endpoint and as multiple classes.  https://cloud.google.com/appengine/docs/java/endpoints/multiclass

That sounds like a good solution. Would you care to answer the same question on stack overflow?

done.


Integration test framework does not support multicluster

Describe the feature request
The current integration framework does not support multicluster. You can default to the current k8s context or you can specify a kubeconfig directory for the tests. The e2e framework does allow specifying a remote cluster kubeconfig.
Describe alternatives you've considered
Slacking with @ozevren on this, it was discussed that we can preserve the current command line arg for the simple case, but support an array of kubeconfigs.  This would lead to an array of accessors, with most tests using the 0th. The index of the cluster can be a configuration to each component. Or, we can create a "sub" ResourceContext with only the particular cluster usable. One issue may be that the base environment does not have a notion of a cluster, which might be needed.
I previously coded up a change similar to the e2e test with the additional parameters and was able to get two clusters configured with Istio, but ran into some underlying issues trying to handle the second cluster in some of the components. The above discussion would allow n clusters which is an improvement.
Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ x ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Additional context

This is a prerequisite for #15706

First PR: #16060


Forego CI builds for documentation only changes

To test this I think you need to mark ready for review then edit md.

To test this I think you need to mark ready for review then edit md.

@pmconne Hmm, looks like you might be right.
Also, not sure the best way to test this without making a ton of small commits (which is fine...) and/or notifying multiple people with each of these commits. I tried running the pipelines directly on azure, but that didn't seem to be picked up as a CI- or PR- triggered build to be able to test these changes.

without making a ton of small commits (which is fine...) and/or notifying multiple people with each of these commits.

Meh, that's what the unsubscribe button is for.


Automatic error report

File "/home/john/anaconda3/lib/python3.6/site-packages/spyder/plugins/configdialog.py", line 214, in accept
configpage.apply_changes()
File "/home/john/anaconda3/lib/python3.6/site-packages/spyder/plugins/configdialog.py", line 104, in apply_changes
self.apply_callback()
File "/home/john/anaconda3/lib/python3.6/site-packages/spyder/plugins/configdialog.py", line 269, in
self.apply_settings(self.changed_options))
File "/home/john/anaconda3/lib/python3.6/site-packages/spyder/plugins/configdialog.py", line 1187, in apply_settings
self.main.help.apply_plugin_settings(['color_scheme_name'])
File "/home/john/anaconda3/lib/python3.6/site-packages/spyder/plugins/help.py", line 552, in apply_plugin_settings
self.editor.apply_plugin_settings(options=[connect_n])
AttributeError: 'NoneType' object has no attribute 'apply_plugin_settings'

This is fixed in our latest version (3.2.8). Please update by opening a terminal (xterm, konsole, etc) and running there
conda update spyder

Thanks for reporting. This error is a duplicate of #6519 , which is fixed with Spyder 3.2.8.


Attempting to add useful info, mostly specific to Monero.

Havent tested the changes on a live version of Moneropedia.

Deleted easiest_buy (Circle no longer sells Bitcoin) and wrote a new guide for buying Monero via Shapeshift and securely storing it on a paper wallet.  Pics to come later.
Reorganized the index and added the guide -^

Please fix conflicts and squash commits:)

Thank you!  I've never used github before. :)

No problem @apezio , but now you have to squash all those commits in one, or at least fewer. Did you use only github or you have a local repository managed with git? because if you are using just github i have no idea if it's possible to squash the commits. If you are using git, this link helped me (i started with git and github just few weeks ago). Hope this will help


Testing "legacy-os" Buildkite Pipeline Option

Change Description
With release 1.7.1 we have announced the deprecation of support for the following operating systems:

Mint 18
Fedora 27
Amazon Linux 1
macOS High Sierra

Since develop is working towards 1.8, I have removed these operating systems from the Buildkite Pipelines.
To build and test against these operating systems, add a file named ".pipelinebranch" to the root of eosio on your branch containing only the text "legacy-os":
$ echo 'legacy-os' > .pipelinebranch
This branch is being used to test this feature in Buildkite and will not be merged.
Consensus Changes

[ ] Consensus Changes
None.

API Changes

[ ] API Changes
None.

Documentation Additions

[ ] Documentation Additions
None.

Done with my testing :)


clean up HRG

Because we are actively using it, the applications folder is especially a bit of a mess...
Also the HRG lacks documentation. It doesn't need much, but we should say a little.

i took care of it


Disabling Keep-Alive support

I see no documentation for disabling Keep-Alive. How would this be done?

@connected-mgosbee I recently implemented a change that allows Keep-Alive to be disabled (#1148).
You can now pass the keepAlive: .disabled parameter to Kitura.addHTTPServer().


Adding <> around bare URLs

Adding <> around bare URLs as they can cause layout issue for loc without MD syntax.

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

Hi @JasonCard - we probably have many more cases of links like this. If we need to change the guidance to wrap with <>, we should probably look at getting others involved, update the contributor guide link guidance, and do a complete sweep of this and other repos. Also, it would be best if this PR were done in the private repo so we can get a full build validation and preview.

Hi @JasonCard - As @bryanla notes, the PR should be opened in the private repo instead of in the public repo. The contributor guide article Public vs. private repo guidance indicates that "Content developers and core product team PMs should work in the private repositories so they can address quality feedback from the quality automation present in the private repos."
To edit an article in the private repo from within the documentation, after you select Edit on an article page,  add “-pr” to the repo name in the URL.

We'll close this PR. Thanks!

Related links in the internal Microsoft Docs contributor guide:
Public vs. private repos | Move a commit to a different repo (public -> private) | GitHub for content reviews
Quick reference for contributing to the docs | Make changes to a Docs article | PR review quality criteria


Avoid clipping responsive social SVG icons

After #677 , social icons slightly clips, probably due to issues similar to https://gsap.com/community/forums/topic/16844-svg-overflowing-its-bounding-box-while-scaling/
For example, see the right edge of the GitHub icon

The CSS property added in this PR fixes it.

Thank you @caizixian
P.S. @caizixian Do you also encounter a flaky situation where sometimes the social-icons do not render at all in spite of being configured properly in the config file and include_file?
@jekyllbot: merge +fix

Thank you @caizixian
P.S. @caizixian Do you also encounter a flaky situation where sometimes the social-icons do not render at all in spite of being configured properly in the config file and include_file?
@jekyllbot: merge +fix

Sorry that I somehow missed this. I haven't encountered this issue.