BGR2RGB in examples

Hi, it seems like in the "Test on random set of 20 images" section of your README you did not change the colors order from BGR to RGB (that is basically why the images are blue). Please correct it :D

face_recognition library in python uses BGR format. Training data consists of BGR formatted 224x224 images of faces. So, in examples also BGR format was used. But in  the demo app, images converted to RGB for better visual image. Try to use demo app in huggingface 👍
Thank you for the comment!


Improvements to file selection

Is your feature request related to a problem? Please describe.
In file explorer you are currently able to select a specific amount of files in a custom order but not in Files UWP.
Describe the solution you'd like
When I have marked a few files and press after the last one ctrl+shift, I then be able to select a few files in a row additionally. Currently in Files UWP it is deselecting the former selected files when doing that, so I cannot choose more files in a row.
When selecting a few files it looks like this: (the last one selected is the lonely selected pdf file)

When I know press ctrl+shift and click on the certificat file, this happens: (it resets the current selection and restarts the selection from the pdf file)

But it should look like this afterwards:

Thank you for opening this issue, unfortunately, this is out of control and it's a behavior of the Windows Community Toolkit DateGrid control. The good news is WinUI is working on a brand new DataGrid control that will improve the behavior in many areas. I am linking to the issue here so you can add your feedback about what you would like to see in this new control.

@yaichenbaum would you suggest to like the comment that you have linked? So is the comment already covering this issue here?
Or should I add a comment there?

@szaimen You should probably add a comment there.


fix(ci): Remove duplicate task not used anymore

What does this PR do?
Remove the old lint-go task that was moved to linter.go namespace

Motivation
We kept the task to make a non breaking change in macos-lint.
Now it's changed on macos side we can deprecate this old task

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Remove duplication 1691

Removed duplication from TestDelete.java, TestAppend.java, TestBatch.java, TestTimestamp.java

Added copyrights to all abstract methods.
On Thu, Mar 15, 2018 at 9:24 AM, Solomon Duskis<EMAIL_ADDRESS>wrote:

@sduskis requested changes on this pull request.
We need the header included for legal reasons. Other than that I'm
assuming you basically cut-and-pasted the Abstract classes and ran the four
sets of tests (HBase vs Bigtable, v1 vs. v2).
In bigtable-hbase-parent/bigtable-hbase-integration-
tests-common/src/main/java/com/google/cloud/bigtable/
hbase/AbstractTestAppend.java
https://github.com/GoogleCloudPlatform/cloud-bigtable-client/pull/1733#discussion_r174670819
:

@@ -0,0 +1,203 @@
+package com.google.cloud.bigtable.hbase;

Please add the standard header:
/*

Copyright 2018 Google Inc. All Rights Reserved.

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
implied.
See the License for the specific language governing permissions and
limitations under the License.
*/

In bigtable-hbase-parent/bigtable-hbase-integration-
tests-common/src/main/java/com/google/cloud/bigtable/
hbase/AbstractTestTimestamp.java
https://github.com/GoogleCloudPlatform/cloud-bigtable-client/pull/1733#discussion_r174670970
:

@@ -0,0 +1,91 @@
+package com.google.cloud.bigtable.hbase;

Please add the standard header.
In bigtable-hbase-parent/bigtable-hbase-integration-
tests-common/src/main/java/com/google/cloud/bigtable/
hbase/AbstractTestDelete.java
https://github.com/GoogleCloudPlatform/cloud-bigtable-client/pull/1733#discussion_r174671011
:

@@ -0,0 +1,330 @@
+package com.google.cloud.bigtable.hbase;

Please add the standard header.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/cloud-bigtable-client/pull/1733#pullrequestreview-104068628,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AiYn90tcXUHugwfYh-0O-Hi7RXCdD8LZks5teeYBgaJpZM4Srft6
.


Refactor: AVS

Right now, there's a lot of intertwined code regarding AVS communication (requests, response handling), triggering, recording, playback.
Splitting AVS into a separate module is especially important since we could then potentially support other services if available (Google Assistant for example), but what's more important, we could then start the work on new AVS API without much problems.

Note: Have a look at @ewenchou's https://github.com/ewenchou/alexa-client if it makes any sense to use it.


How to get additional details of the fund?

Thanks for providing API for Indian MF.
At the moment, API is providing important stats like AUM, NAV & expense ratio. Is there a possibility to get other details like standard deviation, r-squared, sharpe ratio, portfolio turn over etc?

Currently no plans for any derived data, as I don't have expertise in that and just dealing with primary data is work enough - I'd rather keep it at low scope and low chance of errors.
While I don't want to work on portfolio turnover specifically, I do want to track Portfolio composition over time. SEBI mandates fortnightly disclosures, which are in a prescribed format. But no guarantees of if/when i get around to it. (Help is welcome!)


Editor window - selected task not shown beside project name

💻 Environment
Platform: macOS
OS Version: macOS 10.14
Toggl Version: 7.4.1023
🐞 Actual behavior
Edit an time-entry in the editor window, click on the "Select project"-field and select a task below a project, the task-name is not shown after after the selection.
💯 Expected behavior
The selected project- and task-name will be shown in the editor window.
🔨 Steps to reproduce

Create a new task for a project
Edit an time-entry in the editor-window and select a this task
Only the project-name will be displayed but not the task-name.

📦 Additional info

Thank for the report. I'm able to reproduce it too. I'm going to put this bug in next milestone 👍


When I know two countdowns on the same page countdown the two have the same value error

When I know two countdowns on the same page and useRef defines a different countdown the two have the same value
code
const countdownDareRef = useRef();
const countdownRef = useRef(null);
code 1
<Countdown
ref={countdownRef}
style={styles.timer}
textStyle={{ fontSize: 20, color: '#fff', textAlign: 'center' }}
initialSeconds={onEnd ? 1 : duration / 1000}
formatTime="hh:mm:ss"
autoStart={true}
onEnd={() => {
if (duration / 1000 < 0) {
console.log('End', duration)
                            }
                        }}

                    />

code 2
<Countdown
ref={countdownDareRef}
textStyle={{ fontSize: 20, color: '#fff', textAlign: 'center' }}
initialSeconds={300}
formatTime="hh:mm:ss"
autoStart={false}
onEnd={() => {  }}
                    />

To Code Teak same initialSeconds  but  I need different  initialSeconds

hi @AhmadAl-Ghalban ,
This issue is resolved. Please install later version.

hi @AhmadAl-Ghalban , This issue is resolved. Please install later version.

I tried it and it hasn't been resolved yet and there is a problem


xfail Hive text tests failing on CDH

#7423 indicates that Hive text reads fail on Cloudera's Spark distribution, for some Date, Timestamp, and Decimal input.
This commit adds xfail to those tests until the root cause is determined.
Signed-off-by: MithunR<EMAIL_ADDRESS>

build


Remove DatabaseCleaner

It's not needed in Rails 5.1: https://github.com/thoughtbot/suspenders/pull/864

Hmmm looking at the spec failure, maybe we need to set something so tests get wrapped in transactions?

Yeah, everything is failing 😞
Even with config.use_transactional_fixtures set to true, everything is failing.


Failing task due to incomplete data

Sometimes Anomaly detection job fails or the RCA job fails due to fewer data in a few buckets in that case we don't even show the results for the other buckets.
    if not allTasksSucceeded:
        runStatusObj.status = ANOMALY_DETECTION_ERROR

We have added such checks to the code. But I think this is not a good thing to do as the user is not aware of the reason for the failure. Better is to skip this bucket and show the result of the rest of the buckets.

@vincue Let me know if you agree with my point, I will send a PR to fix the above issue.

Correct me if I'm wrong @AakarSharmaHME , by bucket you mean Anomaly Definitions? The keyword allTasksSucceeded here means all tasks within an anomaly definition, so that wouldn't show error for other Anomaly Definitions if that's what you meant.

By bucket, I mean the values of dimension under consideration for an Anomaly Definition. This is a snapshot of code that I took from anomalyDetectionTasks.py. Here allTasksSucceeded is defined as
allTasksSucceeded = all([anomalyTask["success"] for anomalyTask in result]).
So for example, if a user has selected the Top 10 Dimension value and the anomaly detection task is successful only for 8 of them then we should show anomaly detection of the 8 instead of marking the status as ERROR.


Error in Cadabra

Hi
I realigned my reads using abra and wants to do somatic calling using Cadabra but keep having a problem and I do not understand why.
Here is my command:
java -Xmx24G -cp /home/gwenneg/bin/abra-0.95-SNAPSHOT-jar-with-dependencies.jar abra.cadabra.Cadabra $REF $INNORM $INTUM > $OUTPUT
And my error:
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 2
at abra.cadabra.Cadabra.main(Cadabra.java:508)
Command exited with non-zero status 1
I used the bam files output from abra and the same hg19.fa file that was used with abra. I also tried with bam and sorted bam but with the same result.
Thanks for helping
Regards
Gwenneg Kerdivel

Sorted abra bams are the required input.
Please confirm that all of your script variables have values:
$REF $INNORM $INTUM  $OUTPUT

Yes all the variables have values

So, the error you posted can only occur if an input parameter is missing.  I'd suggest printing out your variables in your script prior to the call.  I'll add better input checking in the next release.

Thank you for you answer. I 'll see what I can do.


Add support for icm42686 and tune for NXP VMURT1170

Support both icm42688 and icm42686 through the same driver
Add support for setting Anti Alias Filter and some additional features for Accelerometer and Gyroscope
Tune icm4268x sensors for VMURT1170 for NXP's Mobile Robotics products

This PR has same commits as - PR #75187
PR #75187 got closed since nxp-zephyr repo had to be deleted.
Requesting the  stakeholders to kindly continue your review here

Hi @teburd @decsny .. Kindly continue the review of  #75187 here

Hi @teburd @decsny .. Kindly continue the review of  #75187 here

@teburd @decsny
Is there any change that you need to be done ?

@MaureenHelm can you help review this PR ?


Environment visual enhencement not compatible with 1.2.2 in CKAN

Environment visual enhencement is not compatible with 1.2.2 in CKAN, can you add it in compatibility list?
It cause problems with some depedencies

It doesn't work due to the ksp_version_max restriction requested by the author so I guess we'll ping @WazWaz for confirmation on how to resolve the issue. Is a new version of EVE on its way or should we bump it up another notch?

As see in the forum the actual version works : http://forum.kerbalspaceprogram.com/index.php?/topic/149733-121-environmentalvisualenhancements-12-2/&page=8

Perfect, that's all I need. See the above PR


make some Java examples ready to be opened, runnable in IDE

I have read the CLA Document and I hereby sign the CLA


Dropwizard Testing with environment variables

Hi everybody,
I would like to use environment variables when testing my Dropwizard implementation. Right now I use for testing the DropwizardClientRule class.
@ClassRule
public static final DropwizardClientRule dropwizard = new DropwizardClientRule(new PingResource());

Is there any way on how to set environment variables within the particular resource (i.e. PingResource) when performing testing?
Moreover I would like to use EnvironmentVariableSubstitutor but that requires me to trigger somehow my initialize method. Does anyone of you know if this is possible with the the DropwizardClientRule class.
Best regards
Andy

Is there any way on how to set environment variables within the particular resource (i.e. PingResource) when performing testing?

You could use System Rules for that: https://stefanbirkner.github.io/system-rules/#EnvironmentVariables

Thank you for your suggestion. Is there a way to use EnvironmentVariableSubstitutor together with System Rules?

@aschilling What do you think EnvironmentVariableSubstitutor is doing? What do you want to achieve?

As described in https://www.dropwizard.io/0.8.0/docs/manual/core.html#environment-variables I would like to have in my code environment variables declared like:

defaultSetting: ${DW_DEFAULT_SETTING:-default value}

However, to do so I need to modify the initialize method to add the SubstitutingSourceProvider, which is located in the Application class which in turn is not used if I use DropwizardClientRule to test my code

I would like to have in my code environment variables declared like:

Could you provide specific examples?


Proposed fix for the behaviour of STP

I have 2 setups, one using a real WDC65C02 connected to a Teensy 4.1 and another using vrEmu6502 on a Pico.
The behaviour of STP in the emu did not correspond to reality, so here is a proposed fix that seems to work well for me.
I am not confident in the new code of the STP-function, it might need some more code, but the patched behaviour of STP reflects what I observe on the WDC65C02 version.

Thanks for the bug report and proposed fix. Appreciated!
I think your proposed fix is good, but (for minor performance gains where branch prediction isn't available) it might be a reasonable idea to combine jam and stp struct members into a single member. That avoids the second if() (branch) in vrEmu6502Tick. Perhaps we replace jam with stp? I'd also be open to combining the two if() statements into a single one if (vr6502->jam || vr6502->stp)

I agree, combining the 2 if would be faster and also a bit clearer. I like the single stp version best, jam can use that and still be easy to understand. I have pushed a new version in my fork reflecting this, hopefully it is what you envisioned.

Looks good, Adam. Merged :)


feat(email_sender): add max size to the attachments

Feature/Implement Maximum Attachment Size for Email Messages
Description
This pull request introduces a new feature that allows setting a maximum size for attachments in EmailMessage instances within the gomail package. The feature includes:

A default maximum attachment size of 25 MB.
A method to set a custom maximum attachment size.
Logic to exclude attachments that exceed the specified maximum size during the email sending process.
Comprehensive updates to the documentation to describe the new feature and provide instructions on its usage.

This enhancement improves the usability and reliability of the gomail package by preventing large attachments from causing email delivery failures. It aligns with best practices for email transmission and assists users in avoiding common issues.

Related Issue: Closes #11
Type of Change:

New feature (non-breaking change which adds functionality)

Checklist
Please ensure the following guidelines are met:

[x] The code follows the style guidelines of this project.
[x] A self-review has been performed on the code.
[x] The code is well-documented, and comments have been added where necessary.
[x] Tests have been added to prove that the fix is effective or that the feature works. All existing tests pass.
[x] Commit messages follow the convention type(scope): description.
[x] The pull request has no conflicts with the base branch.
[x] Any dependent changes have been merged and published in downstream modules.

Additional Information
This feature will enhance the usability and reliability of the gomail package by preventing large attachments from causing email delivery failures. The implementation includes comprehensive tests to ensure the feature works as intended and does not introduce regressions. The documentation has been updated to reflect these changes and provide guidance on using the new functionality.

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
:information_source: You can also turn on project coverage checks and project coverage reporting on Pull Request comment
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Build error with python 3.7

FreeBSD 12.1 amd64.
With DOCS option OFF build fine with python 3.7 and python 2.7.
But with DOCS option ON build fine with python 2.7 only. With 3.7 I got error and search didn't help.
Build log:
-- The C compiler identification is Clang 8.0.1
-- The CXX compiler identification is Clang 8.0.1
-- 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
-- Detecting C compile features
-- Detecting C compile features - 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
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Setting Build Type to: Release
-- Setting Namespace to: OpenColorIO
-- Exec prefix not specified, defaulting to /usr/local
-- Use Boost Ptr: OFF
-- Setting python bin to: /usr/local/bin/python3.7
Python library: PYTHON_LIBRARY-NOTFOUND
-- Setting EXTDIST_BINPATH: /tmp/work/usr/ports/graphics/opencolorio/work/.build/ext/dist/bin
-- Setting EXTDIST_PYTHONPATH: /tmp/work/usr/ports/graphics/opencolorio/work/.build/ext/dist/lib/python3.7/site-packages
-- Found TinyXML: /usr/local/lib/libtinyxml.so
-- TinyXML version: 2.6.2
-- External TinyXML will be used.
-- Found PkgConfig: pkgconf (found version "1.6.3")
-- Found yaml-cpp: /usr/local/lib/libyaml-cpp03.so
-- Generate Documentation: true
-- Create sphinx conf.py from conf.py.in
-- Copying doc to staging area
-- Copy extra doc files to staging area
-- Extracting .rst files from C++ headers
-- SSE Optimizations: ON
-- Could NOT find Truelight (missing: Truelight_INCLUDE_DIR Truelight_LIBRARIES Truelight_LIBRARY_DIR)
-- Not building truelight transform support. Add the flag -D TRUELIGHT_INSTALL_PATH=... or set the TRUELIGHT_ROOT environment variable
-- Create OpenColorABI.h from OpenColorABI.h.in
-- Setting OCIO SOVERSION to: 1
-- Create OpenColorIO.pc from OpenColorIO.pc.in
-- Build Unit Tests: ON
-- Create ocio_core_tests.sh.in from ocio_core_tests.sh
-- Disable build of apps. See cmake options : OCIO_BUILD_APPS and OCIO_BUILD_SHARED/OCIO_BUILD_STATIC (requiered)
-- Python library to include 'lib' prefix: OFF
-- Python 3.7 okay (UCS: ucs4), will build the Python bindings against /usr/local/include/python3.7m
-- Python variant path is lib/python3.7/site-packages
-- Found PythonLibs: /usr/local/lib/libpython3.7m.so (found version "3.7.7")
PYTHON_VARIANT_PATH: lib/python3.7/site-packages
-- Configuring done
-- Generating done
CMake Warning:
  Manually-specified variables were not used by the project:

    BOOST_PYTHON_SUFFIX
    CMAKE_CXX_FLAGS_DEBUG
    CMAKE_C_FLAGS_DEBUG
    CMAKE_C_FLAGS_RELEASE
    CMAKE_VERBOSE_MAKEFILE
    THREADS_HAVE_PTHREAD_ARG

-- Build files have been written to: /tmp/work/usr/ports/graphics/opencolorio/work/.build
===>  Building for opencolorio-1.1.1_1
[1/200] cd /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs && /usr/local/bin/cmake -E make_directory /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools && /usr/local/bin/cmake -E make_directory /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools && /usr/local/bin/cmake -E make_directory /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix && /usr/local/bin/cmake -E make_directory /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/tmp && /usr/local/bin/cmake -E make_directory /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools-stamp && /usr/local/bin/cmake -E make_directory /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src && /usr/local/bin/cmake -E make_directory /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools-stamp && /usr/local/bin/cmake -E touch /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools-stamp/setuptools-mkdir
[2/200] cd /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src && /usr/local/bin/cmake -P /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools-stamp/verify-setuptools.cmake && /usr/local/bin/cmake -P /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools-stamp/extract-setuptools.cmake && /usr/local/bin/cmake -E touch /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools-stamp/setuptools-download
CMake Warning at setuptools-stamp/verify-setuptools.cmake:15 (message):
  File will not be verified since no URL_HASH specified

-- extracting...
     src='/tmp/work/usr/ports/graphics/opencolorio/work/OpenColorIO-1.1.1/ext/setuptools-1.1.6.tar.gz'
     dst='/tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools'
-- extracting... [tar xfz]
-- extracting... [analysis]
-- extracting... [rename]
-- extracting... [clean up]
-- extracting... done
[3/200] cd /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs && /usr/local/bin/cmake -E echo_append && /usr/local/bin/cmake -E touch /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools-stamp/setuptools-update
[4/200] cd /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs && /usr/local/bin/cmake -E echo_append && /usr/local/bin/cmake -E touch /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools-stamp/setuptools-patch
[5/200] cd /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools && /usr/local/bin/cmake -E make_directory /tmp/work/usr/ports/graphics/opencolorio/work/.build/ext/dist/lib/python3.7/site-packages && /usr/local/bin/cmake -E touch /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools-stamp/setuptools-configure
[6/200] cd /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools && PYTHONPATH=/tmp/work/usr/ports/graphics/opencolorio/work/.build/ext/dist/lib/python3.7/site-packages: /usr/local/bin/python3.7 setup.py build && /usr/local/bin/cmake -E touch /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools-stamp/setuptools-build
FAILED: docs/setuptools-prefix/src/setuptools-stamp/setuptools-build
cd /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools && PYTHONPATH=/tmp/work/usr/ports/graphics/opencolorio/work/.build/ext/dist/lib/python3.7/site-packages: /usr/local/bin/python3.7 setup.py build && /usr/local/bin/cmake -E touch /tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools-stamp/setuptools-build
Traceback (most recent call last):
  File "setup.py", line 17, in <module>
    exec(init_file.read(), command_ns)
  File "<string>", line 8, in <module>
  File "/tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools/setuptools/__init__.py", line 11, in <module>
    from setuptools.extension import Extension
  File "/tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools/setuptools/extension.py", line 5, in <module>
    from setuptools.dist import _get_unpatched
  File "/tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools/setuptools/dist.py", line 16, in <module>
    import pkg_resources
  File "/tmp/work/usr/ports/graphics/opencolorio/work/.build/docs/setuptools-prefix/src/setuptools/pkg_resources.py", line 1435, in <module>
    register_loader_type(importlib_bootstrap.SourceFileLoader, DefaultProvider)
AttributeError: module 'importlib._bootstrap' has no attribute 'SourceFileLoader'
[7/200] /usr/bin/c++  -DOpenColorIO_EXPORTS -DUSE_SSE -I/tmp/work/usr/ports/graphics/opencolorio/work/OpenColorIO-1.1.1/export -Iexport -I/tmp/work/usr/ports/graphics/opencolorio/work/OpenColorIO-1.1.1/ext/oiio/src/include -Iext/dist/include -O2 -pipe -march=core2 -fstack-protector-strong -isystem /usr/local/include -fno-strict-aliasing  -Wno-deprecated-register -isystem /usr/local/include -msse2 -O2 -pipe -march=core2 -fstack-protector-strong -isystem /usr/local/include -fno-strict-aliasing  -Wno-deprecated-register -isystem /usr/local/include -fPIC   -DOLDYAML  -fPIC -fvisibility=hidden -Werror -std=c++11 -MD -MT src/core/CMakeFiles/OpenColorIO.dir/AllocationOp.cpp.o -MF src/core/CMakeFiles/OpenColorIO.dir/AllocationOp.cpp.o.d -o src/core/CMakeFiles/OpenColorIO.dir/AllocationOp.cpp.o -c /tmp/work/usr/ports/graphics/opencolorio/work/OpenColorIO-1.1.1/src/core/AllocationOp.cpp
ninja: build stopped: subcommand failed.

Hi @VVD
OCIO 1.x does not support Python 3. But the coming OCIO v2 supports both Python versions.

That's a known limitation of OCIO v1 so I close the defect.
Please feel free to reopen it if needed.

But it build and work with python 3.7 without DOCS…
When v2 will be released?

OCIOv2 feature complete is planned for this summer.
Note: As any open source project we are open to contributions from the community. So, you can definitively submit a pull request to fix the Python 3 support in OCIOv1 if that's a blocker for you. Refer to CONTRIBUTING.md for details.

Thanks. I'm not python developer and I'll wait v2.

Is master support python 3 already?

All FreeBSD ports with mandatory dependency from python 2 will be removed on 2020-12-31 if not fixed:
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=249695

There are issue already: https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=249534


Feature testing update

I found a handful of items that seem like are either malfunctioning, or are features that could be cool for additions.
Races:
DAE effects from races/racial features don't carry through
Damage resistance/vulnerabilities/immunities options would be cool
Additional movements options would be helpful
Armor and Weapon proficiencies are each currently titled "tool proficiencies" (I think this was already identified in another issue)

Classes:
Classes don't seem to drag on any level 1 features, nor are there options for proficiencies, saving throws or otherwise

Thanks,

First topic is already identified and corrected, it was just a matter of the translations. About the second topic, I would need to investigate this a little further. Can I contact you through discord one of these days to see it together ?

First topic is already identified and corrected, it was just a matter of the translations. About the second topic, I would need to investigate this a little further. Can I contact you through discord one of these days to see it together ?

Absolutely. It's should be the same username in Discord.

Breaking this into multiple issues to keep track of them independently.

Armor/Weapon proficiencies already taken care on https://github.com/HeroCreationLab/hero-creation-tool/issues/26

There seemed to be a misalignment of compendia sources - sorted with TyphosTheD, closing this!


add two papers

Thanks for your commits! The paper "The case for 4-bit precision: k-bit inference scaling laws" is not a survey paper and the paper "Towards Efficient Post-training Quantization of Pre-trained Language Models" is not related to LLM (experiments only conducted on small scale models such as BERT.)


x/vgo: package fails to resolve from within external test package

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go version go1.10.3 linux/amd64
vgo commit 8a71b2bd9c54c265c37322f928113c849582486b

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/myitcv/.cache/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/tmp/tmp.mLuCNaVLVO"
GORACE=""
GOROOT="/home/myitcv/gos"
GOTMPDIR=""
GOTOOLDIR="/home/myitcv/gos/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build619975550=/tmp/go-build -gno-record-gcc-switches"

What did you do?
cd `mktemp -d`
export GOPATH=$PWD
mkdir hello
cd hello
vgo mod -init -module example.com/hello
mkdir blah
cd blah
cat <<EOD > blah_test.go
package blah_test

import (
        _ "example.com/hello/blah/other"
)
EOD
mkdir other
cd other
cat <<EOD > other.go
package other
EOD
cd ..
vgo list -test -json .

What did you expect to see?
JSON output with a zero exit code.
What did you see instead?
A non-zero exit code with the error message:
can't load test package: blah_test.go:4:9: unknown import path "example.com/hello/blah/other": no such package in module

This package does exist however:
$ vgo list example.com/hello/blah/other
example.com/hello/blah/other

/cc @rsc @bcmills

Well that's wrong. Reproduced.


Update QueryBuilder to use higher level arguments

e.g.
cache.query(q => q.relatedRecord(type, id, relationship));

should be changed to
cache.query(q => q.relatedRecord(record, relationship));

Agreed :+1:
We should also support record(identity) where identity is {type, id}


Support specifying both header and path filter for stealing HTTP

IMO we should probably just allow for any set/combination of filters.
Remember to verify config after fetching mirrord-protocol version used by agent/operator

I agree technically, but from UX side we wouldn't want too complex configuration

Currently the config looks like:
{
    "http_filter": {
        "header_filter": "user: me"
    }
}

or
{
    "http_filter": {
          "path_filter": "/api/v1/my-route.*"
    }
}

I'd allow for third variant like this:
{
    "http_filter": [
        {"header": "user: me"},
        {"path": "/api/v1/my-route.*"},
    ]
}

How about allowing both to be set and take it as an AND?
{
    "http_filter": {
          "path_filter": "/api/v1/my-route.*",
		          "header_filter": "user: me"
    }
}

Sounds ok, but it would be the same amount of work on our end with less flexibility :c

Another, more extensible option:
{
    "http_filter": {
        "all": [
            {"header": "user: me"},
            {"path": "/api/v1/my-route.*"}
        ]
    }
}

(allow for "all" | "any")

I like how explicit this is tbh

I'm okay with it, as long as the basic config options remain (so users can just specify under http_filter one condition..)


Allow text in error message popups to be copied or at least selected

The possibility to copy the message in an error popup (the one with a red X in a circle. This would be nice for:

searching error messages
getting key info to resolve the error oneself

Below is my specific usecase:

The server I work on does not have ftp access, so I need to download that file on my workstation and scp it onto the server.
As it is I cannot select that text to copy the url, so I'm left with typing a 100 character ftp url by hand. Which is painful enough that I'm writing this issue.

This makes sense 👍


Enable easier debugging for python

This issue has been moved from a ticket on Developer Community.

Add debugging rewind feature to python code stepping.  When stepping through code, enable option to step backwards...

Original Comments
Feedback Bot on 5/24/2021, 02:10 AM:
Thank you for taking the time to provide your suggestion.  We will do some preliminary checks to make sure we can proceed further.  We'll provide an update once the issue has been triaged by the product team.

Thanks for the suggestion.


After outputting content from the owlshell terminal, clear the command line window. After adding a space to the cd, it will jump to the bottom

Windows Terminal version
No response
Windows build number
No response
Other Software
No response
Steps to reproduce
Create a new Powershell terminal, enter the pip list, output information, and clear the terminal
Enter cd with a space and enter, the cd will be displayed at the bottom

Expected Behavior
Display CD with space in its original position
Actual Behavior
After adding spaces to cd, it will jump to the bottom

Two things:

This looks just like #2432, which I believe was fixed with an updated version of PsReadline.
However, it looks like you're using the VsCode terminal, not the Windows Terminal. This is the repo for the Windows Terminal, not the vscode one.

If updating PsReadline doesn't help, then I'd file an issue over at https://github.com/microsoft/vscode


Add IncludeRequiredDataConnectors,OmitDataTypeExtension - requiredDataConnectors to ARM template

Adds requiredDataConnectors back to the ARM template - I use it to validate the workspace prior to deployment whether it contains required tables/columns.
IncludeRequiredDataConnectors
Switch to include requiredDataConnectors to the ARM template when converting from yaml
OmitDataTypeExtension
Removes the stuff inside of () and the brackets themselves - as this is not a table name - useless for me, opt in, not default behaviour.
Exec w/ -OmitDataTypeExtension and -IncludeRequiredDataConnectors

Exec w/ -IncludeRequiredDataConnectors

Also @f-bader where the Pester tests at 😄 ?

Nice additional. I will review the PR over the next week but looks really cool

Hi @Kaloszer thank you very much for the additional functionality. I checked the offical ARM template description and it seems that RequiredDataConnectors is not a valid field.
Have you successfully tested the deployment of an ARM template created this way?

@f-bader It's not, that's why it's an optional field here. I filter it out in the CD workflow just before it gets sent as an ARM template.


[BUG] Cython 0.29.30 failing build with pymssql

Describe the bug
I am trying to build a alpine python 3.7 container with pymssql==2.1.5 and Cython==0.29.30. I have already build this image many times with success, but today (18 july 2023) it start to failed in many machines. I saw equal error in Cython==3.0.0 and even updating cython version the error persist. Someone knows why this older version crash? Have someone passed through this and knows how to solve this problem?
Collecting pymssql
Downloading pymssql-2.2.7.tar.gz (170 kB)
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 171.0/171.0 KB 7.0 MB/s eta 0:00:00
Installing build dependencies: started
Installing build dependencies: finished with status 'done'
Getting requirements to build wheel: started
Getting requirements to build wheel: finished with status 'done'
Preparing metadata (pyproject.toml): started
Preparing metadata (pyproject.toml): finished with status 'done'
Building wheels for collected packages: pymssql
Building wheel for pymssql (pyproject.toml): started
Building wheel for pymssql (pyproject.toml): finished with status 'error'
error: subprocess-exited-with-error
× Building wheel for pymssql (pyproject.toml) did not run successfully.
│ exit code: 1
╰─> [119 lines of output]
setup.py: platform.system() => Linux
setup.py: platform.architecture() => ('64bit', '')
setup.py: platform.libc_ver() => ('', '')
setup.py: include_dirs => []
setup.py: library_dirs => []
running bdist_wheel
running build
running build_py
creating build
creating build/lib.linux-x86_64-cpython-37
creating build/lib.linux-x86_64-cpython-37/pymssql
copying src/pymssql/init.py -> build/lib.linux-x86_64-cpython-37/pymssql
running build_ext
warning: src/pymssql/_mssql.pyx:27:0: The 'DEF' statement is deprecated and will be removed in a future Cython version. Consider using global variables, constants, and in-place literals instead. See https://github.com/cython/cython/issues/4310
warning: src/pymssql/_mssql.pyx:28:0: The 'DEF' statement is deprecated and will be removed in a future Cython version. Consider using global variables, constants, and in-place literals instead. See https://github.com/cython/cython/issues/4310
warning: src/pymssql/_mssql.pyx:29:0: The 'DEF' statement is deprecated and will be removed in a future Cython version. Consider using global variables, constants, and in-place literals instead. See https://github.com/cython/cython/issues/4310
warning: src/pymssql/_mssql.pyx:30:0: The 'DEF' statement is deprecated and will be removed in a future Cython version. Consider using global variables, constants, and in-place literals instead. See https://github.com/cython/cython/issues/4310
warning: src/pymssql/_mssql.pyx:31:0: The 'DEF' statement is deprecated and will be removed in a future Cython version. Consider using global variables, constants, and in-place literals instead. See https://github.com/cython/cython/issues/4310
warning: src/pymssql/_mssql.pyx:32:0: The 'DEF' statement is deprecated and will be removed in a future Cython version. Consider using global variables, constants, and in-place literals instead. See https://github.com/cython/cython/issues/4310
warning: src/pymssql/_mssql.pyx:35:0: The 'DEF' statement is deprecated and will be removed in a future Cython version. Consider using global variables, constants, and in-place literals instead. See https://github.com/cython/cython/issues/4310
warning: src/pymssql/_mssql.pyx:36:0: The 'DEF' statement is deprecated and will be removed in a future Cython version. Consider using global variables, constants, and in-place literals instead. See https://github.com/cython/cython/issues/4310
warning: src/pymssql/_mssql.pyx:38:0: The 'DEF' statement is deprecated and will be removed in a future Cython version. Consider using global variables, constants, and in-place literals instead. See https://github.com/cython/cython/issues/4310
warning: src/pymssql/_mssql.pyx:82:0: The 'IF' statement is deprecated and will be removed in a future Cython version. Consider using runtime conditions or C macros instead. See https://github.com/cython/cython/issues/4310
warning: src/pymssql/_mssql.pyx:230:0: The 'DEF' statement is deprecated and will be removed in a future Cython version. Consider using global variables, constants, and in-place literals instead. See https://github.com/cython/cython/issues/4310
warning: src/pymssql/_mssql.pyx:264:4: The 'IF' statement is deprecated and will be removed in a future Cython version. Consider using runtime conditions or C macros instead. See https://github.com/cython/cython/issues/4310
warning: src/pymssql/_mssql.pyx:333:4: The 'IF' statement is deprecated and will be removed in a future Cython version. Consider using runtime conditions or C macros instead. See https://github.com/cython/cython/issues/4310
warning: src/pymssql/_mssql.pyx:831:8: The 'IF' statement is deprecated and will be removed in a future Cython version. Consider using runtime conditions or C macros instead. See https://github.com/cython/cython/issues/4310
warning: src/pymssql/_mssql.pyx:919:8: The 'IF' statement is deprecated and will be removed in a future Cython version. Consider using runtime conditions or C macros instead. See https://github.com/cython/cython/issues/4310
warning: src/pymssql/_mssql.pyx:1005:12: The 'IF' statement is deprecated and will be removed in a future Cython version. Consider using runtime conditions or C macros instead. See https://github.com/cython/cython/issues/4310
warning: src/pymssql/_mssql.pyx:1379:16: The 'IF' statement is deprecated and will be removed in a future Cython version. Consider using runtime conditions or C macros instead. See https://github.com/cython/cython/issues/4310
warning: src/pymssql/_mssql.pyx:1713:8: The 'IF' statement is deprecated and will be removed in a future Cython version. Consider using runtime conditions or C macros instead. See https://github.com/cython/cython/issues/4310
  Error compiling Cython file:
  ------------------------------------------------------------
  ...

  cdef void init_mssql():
      if dbinit() == FAIL:
          raise MSSQLDriverException("dbinit() failed")

      dberrhandle(err_handler)
                  ^
  ------------------------------------------------------------

  src/pymssql/_mssql.pyx:2173:16: Cannot assign type 'int (DBPROCESS *, int, int, int, char *, char *) except? -1 nogil' to 'EHANDLEFUNC'

  Error compiling Cython file:
  ------------------------------------------------------------
  ...
  cdef void init_mssql():
      if dbinit() == FAIL:
          raise MSSQLDriverException("dbinit() failed")

      dberrhandle(err_handler)
      dbmsghandle(msg_handler)
                  ^
  ------------------------------------------------------------

  src/pymssql/_mssql.pyx:2174:16: Cannot assign type 'int (DBPROCESS *, DBINT, int, int, char *, char *, char *, LINE_T) except? -1 nogil' to 'MHANDLEFUNC'
  Compiling src/pymssql/_mssql.pyx because it changed.
  [1/1] Cythonizing src/pymssql/_mssql.pyx
  Traceback (most recent call last):
    File "/usr/local/lib/python3.7/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 363, in <module>
      main()
    File "/usr/local/lib/python3.7/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 345, in main
      json_out['return_val'] = hook(**hook_input['kwargs'])
    File "/usr/local/lib/python3.7/site-packages/pip/_vendor/pep517/in_process/_in_process.py", line 262, in build_wheel
      metadata_directory)
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/build_meta.py", line 417, in build_wheel
      wheel_directory, config_settings)
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/build_meta.py", line 401, in _build_with_temp_dir
      self.run_setup()
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/build_meta.py", line 488, in run_setup
      self).run_setup(setup_script=setup_script)
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/build_meta.py", line 338, in run_setup
      exec(code, locals())
    File "<string>", line 340, in <module>
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/__init__.py", line 107, in setup
      return distutils.core.setup(**attrs)
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/_distutils/core.py", line 185, in setup
      return run_commands(dist)
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/_distutils/core.py", line 201, in run_commands
      dist.run_commands()
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/_distutils/dist.py", line 969, in run_commands
      self.run_command(cmd)
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/dist.py", line 1234, in run_command
      super().run_command(command)
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/_distutils/dist.py", line 988, in run_command
      cmd_obj.run()
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/wheel/bdist_wheel.py", line 343, in run
      self.run_command("build")
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/_distutils/cmd.py", line 318, in run_command
      self.distribution.run_command(command)
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/dist.py", line 1234, in run_command
      super().run_command(command)
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/_distutils/dist.py", line 988, in run_command
      cmd_obj.run()
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/_distutils/command/build.py", line 131, in run
      self.run_command(cmd_name)
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/_distutils/cmd.py", line 318, in run_command
      self.distribution.run_command(command)
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/dist.py", line 1234, in run_command
      super().run_command(command)
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/_distutils/dist.py", line 988, in run_command
      cmd_obj.run()
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/_distutils/command/build_ext.py", line 345, in run
      self.build_extensions()
    File "<string>", line 181, in build_extensions
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/_distutils/command/build_ext.py", line 467, in build_extensions
      self._build_extensions_serial()
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/setuptools/_distutils/command/build_ext.py", line 493, in _build_extensions_serial
      self.build_extension(ext)
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/Cython/Distutils/build_ext.py", line 123, in build_extension
      ext,force=self.force, quiet=self.verbose == 0, **options
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/Cython/Build/Dependencies.py", line 1134, in cythonize
      cythonize_one(*args)
    File "/tmp/pip-build-env-1e60ghn3/overlay/lib/python3.7/site-packages/Cython/Build/Dependencies.py", line 1301, in cythonize_one
      raise CompileError(None, pyx_file)
  Cython.Compiler.Errors.CompileError: src/pymssql/_mssql.pyx
  [end of output]

note: This error originates from a subprocess, and is likely not a problem with pip.
ERROR: Failed building wheel for pymssql
Failed to build pymssql
ERROR: Could not build wheels for pymssql, which is required to install pyproject.toml-based projects
WARNING: You are using pip version 22.0.4; however, version 23.2 is available.
You should consider upgrading via the '/usr/local/bin/python -m pip install --upgrade pip' command.
failed to solve: process "/bin/sh -c pip install pymssql" did not complete successfully: exit code: 1
Code to reproduce the behaviour:
No response
Expected behaviour
No response
OS
Linux 18.04; Docker alpine3.16
Python version
3.7.15
Cython version
0.29.30
Additional context
No response

I created this issue because I have only seen the problem to Cython==3.0.0, but resemble it happens on older versions too

This definitely looks like you're actually using Cython 3.0 even if you think you're using 0.29.x. Would it be possible to add a print for the version in setup.py to check?

Do you say it is Cython 3 because something in the log, or only because the error??

Because of the error.

Relevant post in discourse: https://discuss.python.org/t/no-way-to-pin-build-dependencies/29833/14

Wanted to mention, that this happens inside docker of my M1 machine also.
Not on docker containers on intel machines as it seems.

perhaps the with gil / no gil is the problem here - i try a bit

Just wanted to bump this by saying that I am facing the same issue currently, it is happening to me on Azure Pipelines as well as locally.

This also affects intel machines;
What i saw right now:

pip freeze shows i am using cython 0.29.35

when i install pymssql i just saw that it is downloading the latest cython 3.0.0 version when pip installing the pymssql.
Seems if i can avoid downloading the cython 3.0.0 at install it will work again:

ok by this it works again:

Setting pyproject.toml and forcing not to use latest Cython by setting it to the latest best 0.29.35 cython version.
I make a pull request and a branch.
Installing could then work with
pip3 install git+https://github.com/....@branch
When compiling the libs must be present:

I will check out installing them on ubuntu based image and see if it also works.

In order to compile fully those libs are required on ubuntu:
apt-get install libcurl4-openssl-dev zlib1g-dev libkrb5-dev libssl-dev

I will now make a pull request to restrict the cython to 0.29.35. I hope the maintainer is merging that very soon, that this is available on pypi.

i branched away from 2.2.7 version - now doing pull request. please help me reach the maintainers.

I really need this pull request to go through as I depend on the package a lot currently, hopefully it won't take long

Thanks a lot @marcwimmer, unfortunately on my way of working I can not install packages like that for security reasons, but I will be on lookout for the pull request to go through!

https://github.com/pymssql/pymssql/pull/827
Pull request done - lets call the maintainers somehow :) which timezone are they?

A temporary solution until @marcwimmer's solution goes live:
echo "Cython==0.29.35" >> constraints.txt
export PIP_CONSTRAINT="$(pwd)/constraints.txt"

Thanks @blink1073 for providing that link!

The conclusion for futher readers,
This is right, the problem is that pymssql take another cython to compile and that version is simply bigger than 0.29.x, so it takes the new release of cython 3.0.0 (@da-woods you were right) and breaks on build. The solution is take the pymssql repo and change the build configuration fixing cython.
I need to do this on an older version of pymssql, so I did a fork and I created a new branch from the tag. But the next version of pymssql would probably be fixed with the @marcwimmer's PR or some other.
Thanks everyone that helped.

A temporary solution until @marcwimmer's solution goes live:
echo "Cython==0.29.35" >> constraints.txt
export PIP_CONSTRAINT="$(pwd)/constraints.txt"

Thanks @blink1073 for providing that link!

You can also add a link to constraints.txt to your requirements.txt:
--constraint constraints.txt
anyio==3.6.1
...

Good thing is that the constraint is applied for dependent packages of packages used in requirements.txt


Mention of docs-strategy repository probably only for staff?

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/contributing/style-guide-and-content-model/style-guide#buttons
What part(s) of the article would you like to see updated?

"If a button encourages people to navigate away from the GitHub Docs site, follow the call to action (CTA) button guidelines. If you want to include another type of button on a landing page or article, open an issue in the docs-strategy repository to share your use case for approval by the Content Strategy team."

This is either outdated, or unavailable to outside collaborators. There is no public @github/docs-strategy repository to file an issue with. If this is only for Hubbers there might be a note it's not available publicly — with instructions for outside collaborators how to proceed(?); or if outdated/superseded new instructins might be needed.
Additional information
No response

@janbrasna Thank you for opening this issue! I'll get this triaged for review ✨

Thanks for highlighting this, @janbrasna. This is unavailable to outside collaborators, so I've opened an internal issue to update this section of the style guide and I'm going to close this one out. Thank you for helping make the GitHub Docs better!


fix Mapsui.Native.nuspec

by re-adding Mapsui.Rendering.Xaml.dll
(which was accidentally removed in dadd2989f69ce081c9fb22d469ccc4242446081c)

@janusw I have split up this file, so I think this PR has become irrelevant and could be close. Do you agree?

Sure.


feat: enabled invite_accepter

Allow for use of this module within AWS Organizations that already have guard duty enabled and are under management from a Security Hub admin account.
Types of changes
What types of changes does your code introduce?

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation Update (if none of the other choices apply)

Checklist

[x] I have read the CONTRIBUTING.md doc.
[x] I have added necessary documentation (if appropriate).
[x] Any dependent changes have been merged and published in downstream modules.

Further comments
Without the ability to disable the invite acceptor or the initial enablement of the detector itself then this module could never be used within an AWS setup that already has the Guard Duty enabled at the Organization level.
The variables I introduced are none-breaking, in the sense that anyone using the module will not have to update definitions as the default is the same as the previous module version.
All the changes I have tested in my own AWS env which contains the situation I was solving for.

@soareseds @arthurbdiniz @adenot @caiovfernandes This PR has sat for a month and a half. Mind taking a look and merging?

@soareseds @arthurbdiniz @adenot @caiovfernandes poke.

@caiovfernandes I did you one better - I added types to all the variables (they were not defined sans one, which is why I left them off originally).


Decoupled Master Themes

Removed application specific properties from the master theme definitions
Added build source boiler plate.
Added the ability to create application theme templates from master themes.


Unexpected newline when parsing JSON

I'm trying to parse a json file and then parse it into a string.  It seems that when I'm parsing in the way that I'm used to, an additional newline is being put into my data.  In the examples below, my ConvertFrom-Json call does not include any empty members, but it appears as soon as I start processing the object it produces.
Steps to reproduce
lints.json:
{
    "lints": [
        "needless_return",
        "module-inception",
        "many_single_char_names",
        "implicit-hasher"
    ]
}

Get-Content ./lints.json | ConvertFrom-Json | % lints

Expected behavior
needless_return
module-inception
many_single_char_names
implicit-hasher

Actual behavior

needless_return
module-inception
many_single_char_names
implicit-hasher

Environment data

Name                           Value
----                           -----
PSVersion                      7.0.0
PSEdition                      Core
GitCommitId                    7.0.0
OS                             Darwin 19.4.0 Darwin Kernel Version 19.4.0: Wed Mar  4 22:28:40 PST 2020; root:xnu-6153.101.6~15/RELEASE_X86_64
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

I cannot reproduce this behavior with the sample input string - are you sure it is representative?
Does the following also reproduce your symptom?
@'
{
  "lints": [
    "needless_return",
    "module-inception",
    "many_single_char_names",
    "implicit-hasher"
    ]
 }
'@ | ConvertFrom-Json | % lints

@mklement0 Do you try on MacOS too?

The sample code appears to work just fine on MacOS (Catalina) for me. 😕

Thanks for confirming, @vexx32. @iSazonov, yes, I tried on macOS too (it's usually where I try first).

I guess the file contains broken newlnes.

More information might come from:
Format-Hex -Path ./lints.json


[WIP] Optimize trivial gather offsets

Update gather/scatter offset generation to handle trivial contiguous offsets (i.e. [0,1,2,3]) and generate contiguous load/store instead.

@harsh-nod here is the patch to get rid of trivial gathers

@harsh-nod here is the patch to get rid of trivial gathers

Thanks

Do we not have tests to showcase this?

Yeah, it's kind of hard to construct artificial example for it.


Nothing being passed in Payload

For some reason, the payload that comes from Slack is empty, is that an issue with the repo, or slack?
I followed your directions, setup a route that responds with the URL verification, that works just fine, but when I send a message to the bot, Slack sends a message with the $request variable being completely empty. Am I missing something here?

Could you try and log the Slack payload?
// Inside your controller method
\Log::info('Request content: ', $request->getContent());

When doing $request->getContent(), I get this:
[2016-11-14 23:30:38] production.INFO: Request content: {"token":"***","team_id":"***","api_app_id":"***","event":{"type":"message","user":"U043B5E5L","text":"Hello World","ts":"1479166237.000120","channel":"***","event_ts":"1479166237.000120"},"type":"event_callback","authed_users":["***"]}

When I try following your guide, and doing $request->getJson(), assigning it to $payload, I get this:
[2016-11-14 23:30:38] production.INFO: Request Payload: {}

I don't quite get where you check for the $payload variable and what exactly your error is.
It looks like Slack is sending you the correct data, so this should work:
\SlackBot::hears('hello world', function($bot) {
    $bot->respond('Hello!');
});

I was following the guide you have:
    $payload = $request->json();

    if ($payload->get('type') === 'url_verification') {
        return $payload->get('challenge');
    }

    $slackBot = app(SlackBot::class);
    $slackBot->initialize(env('SLACK_TOKEN'));

    $slackBot->hears('hello', function (SlackBot $bot) {
        $bot->respond('hi');
    });

Okay - now I got you :)
Could you try it with this code instead?
The API of the bot changed quite a bit and there's no need to reinitialize the bot.
Just add this to your config/services.php:

    'slack' => [
        'bot_token' => env('SLACK_TOKEN')
    ]

And the implementation would be:
use SlackBot;
use Mpociot\SlackBot\SlackBot as Bot;

$payload = $request->json();

if ($payload->get('type') === 'url_verification') {
    return $payload->get('challenge');
}

\SlackBot::hears('keyword', function (Bot $bot) {
    $bot->respond('lets begin');
});

My problem is that this bot will be used by multiple companies, doesn't that require me to change the token between different companies/installations on slack? If so, how would I go about that? Re-initialize the bot with the token corresponding to the company?

The other issue is that hears isn't static, so it has to be called doing $slackbot->hears unless you are doing some sort of witchcraft. 😃

If you're using Laravel, make sure you use the app()->make(classs) to use the IOC container instead of simply instantiating     $slackBot = new SlackBot(); That'll return a new instance of the SlackBot to use. 😃

As stated in a previous example of mine:
    $team = Company::find(1);
/** @var \Mpociot\SlackBot\SlackBot $slackBot */
        $slackBot = app('slackbot');

 $slackBot->initialize($team->slack_info->bot->bot_access_token);
      $slackBot->hears('ping', function (SlackBot $bot) {
            $bot->reply('PONG');
        });
        $slackBot->listen();

I have this now:
/**
 * @param Request $request
 */
public function handleBotRequest(Request $request)
{
    $payload = $request->json();
    if ($payload->get('type') === 'url_verification') {
        return $payload->get('challenge');
    }

    $team = Company::find(1);

    $slackBot = app('slackbot');

    $slackBot->initialize($team->slack_info->bot->bot_access_token);
    $slackBot->hears('ping', function (SlackBot $bot) {
        $bot->reply('PONG');
    });
    $slackBot->listen();
}

and nothing, I type ping in chat, and nothing returns. Do I have to do @bot ping, or in the dm chat just ping?

My full file is:
use App\Models\Company;
use Illuminate\Http\Request;
use Mpociot\SlackBot\SlackBot;

/**
 * Class SearchController
 * @package App\Http\Controllers
 */
class SlackBotController extends Controller
{
    /**
     * @param Request $request
     */
    public function handleBotRequest(Request $request)
    {
        $payload = $request->json();
        if ($payload->get('type') === 'url_verification') {
            return $payload->get('challenge');
        }

        $team = Company::where('slack_info', 'LIKE', '%"team_id":"' . $request->input('team_id') . '"%')->first();
        $team = Company::find(1);

        $slackBot = app('slackbot');

        $slackBot->initialize($team->slack_info->bot->bot_access_token);
        $slackBot->hears('ping', function (SlackBot $bot) {
            $bot->reply('PONG');
        });
        $slackBot->listen();
    }
}

GOT IT!

Nice! :D So the above is example is working correctly?

Is there a reason that it is sending double responses?

I've experienced that when I had config/services.php token was set as well as setting $slackBot->initialize() while listening for a response

I feel like the ServiceProvider shouldn't try to initialize the bot if the config isn't configured. @mpociot thoughts?

@darkgoldblade01  to clear things up: You can use SlackBot as if the methods were static in Laravel, when you have registered the SlackBot alias. This uses the Laravel facades feature and requires you to use the \SlackBot class.
@jerairrest I modified this behavior.

Wonderful! Thank you sir!


Smite Support

Hello, can u add functionality for smite, i see lol and dota 2 but not smite which is the 3rd of the big 3 mobas. thank you

I think they can't since that game uses easy anticheat, and this aurora software uses some hacks to read the memory of the game to get the game stats, and with smite that's imposible since that anti-cheat engine is so intrusive it will ban people even just for having some unsigned drivers.

@Dagda52 The only game support that does memory reading is Rocket League (which is why it has to be updated all the time) and Borderlands 2. Most games either use GSI or a wrapper for Logitech/Razer/LightFX so the native lighting is sent to Aurora instead of the normal software. If there is anything like GSI or modding for Smite then I can probably support it, otherwise, I can't do much.

Smite is supported by LGS. (Logitech wrapper should work?)


Applied the correct validation to iso and pxe dialogs when selected i…

…n the provisioning process
https://bugzilla.redhat.com/show_bug.cgi?id=1255190
Refactored how the update_field_required method validates if supports_pxe?
or supports_iso? are required fields.
We now simply apply the :edit, or :hide flags to the MiqProvisionVirtDialog class
which implicitly does not validate on hidden fields

@gmcculloug - Please review

@syncrou In the future (or if you end up needed to amend this PR anyway), please keep commit message subject under 80 chars (preferably 72).  This is because both GitHub titles as well as tools like git log rely on that width (see how your PR title is truncated).  See https://github.com/ManageIQ/guides/blob/master/coding_style_and_standards.md#commits

@gmcculloug - Requested changes were made.  Please review.


Manual approval required for workflow run<PHONE_NUMBER>

Workflow is pending manual review.
URL: https://github.com/ashishjullia/terraform-w-rollback/actions/runs/3475317219
Required approvers: [ashishjullia]
Respond "approved", "approve", "lgtm", "yes" to continue workflow or "denied", "deny", "no" to cancel.

approve

All approvers have approved, continuing workflow and closing this issue.


support for ranges

Does this tox plugin supports ranges for pip version or only pinned ones?
I often need to update pip to a minimal version.

@ssbarnea It does not support ranges currently. Should be an easy thing to add.


Wrong return type in PHPDoc

Laravel Version: master
PHP Version: -
Database Driver & Version: -

Description:
The return type of \Illuminate\Database\Eloquent\Model#resolveChildRouteBindingQuery should be from type \Illuminate\Database\Eloquent\Builder instead of \Illuminate\Database\Eloquent\Model|null
 /**
     * Retrieve the child model query for a bound value.
     *
     * @param  string  $childType
     * @param  mixed  $value
     * @param  string|null  $field
     * @return \Illuminate\Database\Eloquent\Model|null
     */
    protected function resolveChildRouteBindingQuery($childType, $value, $field)
    {
        $relationship = $this->{Str::plural(Str::camel($childType))}();

        $field = $field ?: $relationship->getRelated()->getRouteKeyName();

        if ($relationship instanceof HasManyThrough ||
            $relationship instanceof BelongsToMany) {
            return $relationship->where($relationship->getRelated()->getTable().'.'.$field, $value);
        } else {
            return $relationship->where($field, $value);
        }
    }

Steps To Reproduce:
N/A

Just send in a pr. Please don't open issues for this.


Getting authentication error message from server response JSON

(maybe) related - https://github.com/SpoonX/aurelia-api/issues/134
Hi!
So my server returns a JSON response message to authentication requests so I can identify what error specifically happened and display that in my view (user doesn't exist, password doesn't match, field empty, etc).
{
	"success": false,
	"message": "User doesn't exist."
}

However, I can't access these response messages from my VM.
return this.authService.login(this.email, this.password)
	.catch(error => {
		console.log(error);
	});

All I get is a generic "Token not found in response" error. Is there any official way of getting this response or at least any workaround? Thanks!

@jemhuntr Hey, thanks for the question.
It's not related, no, but I've posted an answer on that issue again. :)
The "token not found" error you mention gets thrown when it, well, can't find the token on the response. (see here and here).
It tries to set the response object because your server is returning a valid response code, and authentication tries to access your token (here).
All you have to do to fix this, is fix your server to return the correct status code (401) so that the authentication library can throw an error and you can catch it as expected. Then it won't be a Token not found in response error.
Does this answer your question? :)

@RWOverdijk Thanks much! I got my setup working by fixing the server, although a console warning keeps popping up.
From what I read, I would have to reconfigure the fetch-client which is being used by aurelia-api, which is in turn being used by aurelia-authentication. I tried directly modifying the config for my authentication endpoint on my main.js, but it seems it's still rejecting error responses.

@jemhuntr I'm not sure I follow, what do you mean? I use this all the time and never get warnings.

I'm guessing that the rejectErrorResponses() method on the (HttpClientConfiguration)[http://aurelia.io/hub.html#/doc/api/aurelia/fetch-client/1.0.1/class/HttpClientConfiguration] is causing this issue.
Since my server is now sending a 401 (instead of a 2xx response) on failed logins, aurelia-fetch-client is rejecting the response messages on these calls.
I can't seem to figure out how to configure aurelia-authentication or aurelia-api or aurelia-fetch-client to prevent it from rejecting error responses.

I'll try rebuilding my server from scratch.

I am having a similar problem.
After calling authservice.login('invalid-credentials')
I get an error response:
body:ReadableStream
bodyUsed:false
headers:Headers
ok:false
status:401
statusText:""
type:"cors"
url:"https://api.test.com/api/v2/user/session"

It is my understanding that an error is thrown because the session token is missing which is correct, but I can't figure out how to access the actual json response object returned from my server.
Below is what is returned from url:"https://api.test.com/api/v2/user/session" that I need to get access to.
{"error":{"context":null,"message":"Invalid credentials supplied.","code":401}}
Thanks

I got, just needed to call .json on the err object.  Thanks

Awesome! Glad you got it.

@RWOverdijk Hey, I get a console error if I return a 401 Unauthorized, (everything is working tho). Anyway to suppress this error?

http://stackoverflow.com/questions/42532109/aurelia-auth-custom-response-message-console-error

@reft The console error is for the developer to know why. What are you looking for?

@RWOverdijk  If I return a json result with a message and an empty token I get:
login failure: Error: Token not found in response
As I understand it I get this because I have not returned an invalid response message.. BUT.. Can I read my json message from the server here? I dont want to return a response message because that generates a console error.

Currently I have to put a generic message here:
    return this.authService.login(this.email, this.password)
        .then(response => {
            console.log("success logged " + response);
        })
        .catch(err => {
            ALERT('GENERIC MESSAGE');

            //err returns: "Token not found in response"
        });

"Token not found in response" message is found in file: authentication.js. There really should be support for returning a server message instead :(

@reft you need to call .json() on the err object you're getting from .catch(). Try this:

return this.authService.login(this.email, this.password)
	.then(response => {
		// do stuff
	})
	.catch(response => {
		if (response instanceof Error) {
			// server returned no response
				this.errorMessage = "Unable to contact authentication server.";
		} else {
			// server handled the response gracefully,
			// but since it's an error code we got, we're expecting
			// an error message from the server along with the error code
			response.json().then(error => {
				// `error` in this scope would be the JSON
				// that your server has sent back from your request
				console.log(error);
				this.errorMessage = error.message;
			});
		}
	});

Take note that for this to work, your authentication server has to respond with a non-success error code (401, for example).
Until now, there still is an ominous console warning I'm getting even if everything is working as expected in this setup. Tell me if you get that warning too: Warning: a promise was rejected with a non-error: [object Response]

I also had a problem with .json as I was unfamiliar with the fetch api.  The .json is part of the new fetch api   specifically the body mixin.  Also,  Warning: a promise was rejected with a non-error: [object Response]  this is a bluebird warning.   I also get this error message when my server sends a 401 response.  You can turn all warnings off but I am also interested in finding the correct solution besides disabling warnings as per below
Promise.config(Object {
warnings: boolean=false,
longStackTraces: boolean=false,
cancellation: boolean=false,
monitoring: boolean=false
}
Oh, just ran across this regarding the aurelia-fetch-client.  Note: the user Bergi says it can be ignored, he seems to know what he is talking about if you look at his profile.

@jemhuntr
As you mentioned the Warning: a promise was rejected with a non-error: [object Response] seems to be a result of config.rejectErrorResponses() in the  aurelia-fetch-client under the Helpers section.  I can see how to disable it in aurelia-fetch-client but not in the spoonx aurelia-api.  Did you figure this out?

I think I see the problem. The reason it throws that error is because it can't find the token on the response. What we can do, perhaps, is add the server response to the error. Would that solve your issues?
If there's a token, it won't complain. But if the status code is a valid one, it will look for the token.

I can live with simply ignoring the warning, assuming that won't show up in production.
Although being able to access the server response from the error object would be a great addition.

@jemhuntr Awesome middle ground I'd say.

I'm a little busy with schoolwork, but I'll try to look at it over the weekend.

For anyone still having this issue, pull from the latest release and make sure your server returns a response code 200 along with the errors it generated.

return this.authService.login(this.email, this.password)
	.then(response => {
		// do something
	})
	.catch(error => {
		// The entire server response object is now directly accessible via
		// the error object's `responseObject` key, given that the server
		// responded with a 200 status code and aurelia-authentication
		// didn't find a token from the server response.
		console.log(error.responseObject);

		// If authService returned a response object from the server,
		// use that for outputting error message on the UI.
		// Otherwise, use the error as is (access error.message).
		this.error = error.responseObject || error;
	});


expose 1194/tcp aswell

The tcp should be exposed as well, since non docker runtime(e. g. rkt) can't expose ports on the fly

These can be specified during docker run.


Creation of HTTP Virtual Directory fails if default website is missing

This one took me quite a while to diagnose.  The deployment framework piece that installs virtual directories will fail if there is not a #1 (i.e. default) website in the IIS Metabase.  I peeked into the code of DeploymentFramework.BuildTasks.dll and found reference to where you are trying to do the following:
new DirectoryEntry("IIS://localhost/W3SVC/1/Root").  This is where it will fail if "/1/Root" doesn't exist.

There is a workaround to recreate the default website manually, so that the script will work, but I would think that the deployment framework should let you define the target website - especially if you do not want it to go into the default website.
This work item was migrated from CodePlex
CodePlex work item ID: '4015'
Assigned to: 'tfabraham'
Vote count: '1'

[Rokhead@3/11/2010]
Just a note to let you know that this issue still exists on IIS6.  It is a known security practice to remove the default website:
http://www.windowsitpro.com/article/security/removing-the-default-web-site.aspx
Recreating the default website via the IIS UI will not work because it is not identified internally as #1.  It is possible to do a switcheroo and move it to the #1 spot via script, but the real fix would be to resolve the bad assumption in the framework task as outlined above.

[UnknownUser@6/28/2010]

[UnknownUser@6/28/2010]
Added new property IISMetabasePath and modified all IIS configuration steps to use it.  Default value is IIS://localhost/w3svc/1/Root, which is the Default Web Site.

[UnknownUser@6/28/2010]
Resolved with changeset 48059.

[UnknownUser@2/21/2013]

[UnknownUser@5/16/2013]


Added support for pycharm's type checking with dataframe

Made DataFrame a regular pandas dataframe when in IDE and use the pandera DataFrame on runtime.

@cosmicBboy I think a great idea is to add the "hacktoberfest2020" label to the project! Might get a bit of attraction and will help me plant a tree (as I'm focused to help to improve this library since we are starting to use it in our company)

thanks for the PR @amitripshtos! and also thanks @jeffzi for the simple solution ⭐

I think a great idea is to add the "hacktoberfest2020" label to the project

excellent idea! lemme add that (also so you can get your PR in 🙂)

hey @amitripshtos I just made a small change to this PR to prevent the pylint error: https://travis-ci.org/github/pandera-dev/pandera/jobs/733198258
please confirm that the change doesn't make pycharm complain again (it didn't on my machine).

Codecov Report

Merging #283 into class-api will decrease coverage by 0.09%.
The diff coverage is 50.00%.

@@              Coverage Diff              @@
##           class-api     #283      +/-   ##
=============================================
- Coverage      97.90%   97.80%   -0.10%
=============================================
  Files             18       18
  Lines           1670     1733      +63
=============================================
+ Hits            1635     1695      +60
- Misses            35       38       +3

Impacted Files
Coverage Δ

pandera/typing.py
95.16% <50.00%> (-4.84%)
:arrow_down:

pandera/model.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

I can confirm that the change you made works well in pycharm.
Do you understand the codecov issue? I suspect it's because the condition but I'm not sure how to test it our since in runtime TYPE_CHECKING is false.

Codecov Report

:exclamation: No coverage uploaded for pull request base (class-api@48059ab). Click here to learn what that means.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             class-api     #283   +/-   ##
============================================
  Coverage             ?   97.90%
============================================
  Files                ?       18
  Lines                ?     1673
  Branches             ?        0
============================================
  Hits                 ?     1638
  Misses               ?       35
  Partials             ?        0

Impacted Files
Coverage Δ

pandera/typing.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


determine if redirector needs to be AWS-region aware

We have two scenarios still on the table for handing off traffic to AWS for AWS clients:

We route to cloudfront + s3 or ecr or otherwise some other endpoint that is global and handles serving regional copies internally.

We ourselves have regional data storage (e.g. an s3 bucket per continent, region ...) and we need to perform region mapping ourselves before redirecting.

Both of these should be doable in the redirect service, it's mostly a matter of what makes the most sense on the AWS architecture side from a cost perspective. Both have been discussed but we haven't settled on which we'll provide.
Once we know this, we know what the redirection logic needs to look like in the redirector.
Parsing the AWS data is very easy, but the implementation and optimization will look different depending on if we need to detect regions or not.
/assign @jaypipes
I have a branch locally with benchmarks for 1) that I can easily clean up and turn into 1) or 2), but they look fairly different and I'd like to wait until we know this first.

cc @jaypipes @dims

Adding: k8s.gcr.io currently handles mapping to {us,eu,asia}.gcr.io/$actual-registry*, so the redirector uses approach 1) today. I think approach 2) looks quite doable for us though (we will need to push to each and then match region IPs https://docs.aws.amazon.com/general/latest/gr/aws-ip-ranges.html in the redirector), I'm not sure if this is better cost wise than e.g. cloudfront + s3.

The GCP version of this we have today is a multi-regional GCS/GCR in us,eu,asia, with a frontend routing to the appropriate mutli-regional copy, but this is handled transparently for the project specially, our tools are relatively unaware of this. This is just because that is what the google teams setup, I am clarifying this just for context.

Hopefully one of the AWS experts can recommend which is best for cost, we should have a workload dominated by downloads.

ref: https://github.com/kubernetes-sigs/oci-proxy/issues/20#issuecomment-1077629290

The k8s.gcr.io bucket is public read, should be able to list and collect reported sizes. I think the right call to gsutil ls would even work + summing the results. @BenTheElder
From https://kubernetes.slack.com/archives/CCK68P2Q2/p1648159238576869?thread_ts=1648085463.736569&cid=CCK68P2Q2

I'll take on generating this via gustil or go. Do we have a list of buckets used by k8s.gcr.io?

The GCP version of this we have today is a multi-regional GCS/GCR in us,eu,asia

To be more specific, k8s.gcr.io do the mapping to 3 mutli-regions buckets and the image layers are replicated between them :

Bucket name
Total Size
Object  total count*

gs://asia.artifacts.k8s-artifacts-prod.appspot.com
1.07 TB
93875

gs://eu.artifacts.k8s-artifacts-prod.appspot.com
1.07 TB
93875

gs://us.artifacts.k8s-artifacts-prod.appspot.com
1.07 TB
93875

(found the list in https://github.com/kubernetes/k8s.io/blob/main/images/public-log-asn-matcher/buckets.txt)
*Object total count: From https://kubernetes.slack.com/archives/CCK68P2Q2/p1648172955716699?thread_ts=1648085463.736569&cid=CCK68P2Q2

It's also probably worth noting that the bulk of those images are likely not often requested, since we continue to host e.g. the very oldest Kubernetes releases that hopefully few are using today :-)

@jaypipes @hh is this info enough for you?

I believe so. @jaypipes ?

@jaypipes @hh is this info enough for you?

yup.

per the call we are doing 2) with a mapping from regions to the nearest bucket, we'll have a bucket in the top N regions.
I've taken the action item to update the redirector to handle this, we also have related items to have image promotion support pushing to these buckets and to figure out what regions to use.
we should move region selection to a new issue. s3 bucket pushing also will have an issue in the image promoter repo.


Fix output of hyperparameter search notebook

Hi :)
While going through the examples, I have noticed that one of the cells of the text-classification_hyperparameter-search.ipynb notebook raised an Exception.
I just rerun that notebook, and now it looks fine.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

So this is only reran, no other changes? The diff, and even ReviewNB, is a bit of a mess as you can imagine, haha.
It looks good at a glance.

Yes, just rerun it in Colab no changes to the code/markdown!
Oh yeah, the diff in ReviewNB looks messy too. Mmm… The diff on VSCode looked good. 😕 I actually made sure to only commit the outputs to keep the diff small (because Colab changed the metadata of each cell a bit too).
Are there any tricks to make ReviewNB show a more helpful diff?

I'm guessing it's not possible. It's a bit surprising, really, because it should just be able to recognize which cells correspond between main and this PR. After all, the number of cells didn't change, and only the outputs differ...
That said, this is great! Thank you for taking the time to fix this for us! I have no issues merging this.


KeyError: 'message'

Worked like charm for many months. Then recently the new authorisation process came. I updated pyporscheconnectapi. Worked again for 3 days and then ....
Since a few days, I am getting errors while running my script with pyporscheconnectapi even in the latest version (0.1.3).
Now as a reference, using both of your example scripts in the 'Library usage' section, I am getting the following error:
Traceback (most recent call last):
File "C:\Users\plane\AppData\Roaming\JetBrains\PyCharmCE2022.3\scratches\scratch_3.py", line 23, in
loop.run_until_complete(vehicles())
File "C:\Program Files\Python\Python_311\Lib\asyncio\base_events.py", line 653, in run_until_complete
return future.result()
^^^^^^^^^^^^^^^
File "C:\Users\plane\AppData\Roaming\JetBrains\PyCharmCE2022.3\scratches\scratch_3.py", line 13, in vehicles
vehicles = await client.getVehicles()
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\plane\PycharmProjects\PyPorsche\venv\Lib\site-packages\pyporscheconnectapi\client.py", line 448, in getVehicles
vehicles = await self._connection.get(
^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\plane\PycharmProjects\PyPorsche\venv\Lib\site-packages\pyporscheconnectapi\connection.py", line 233, in get
headers = await self._createhead(application)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\plane\PycharmProjects\PyPorsche\venv\Lib\site-packages\pyporscheconnectapi\connection.py", line 287, in _createhead
token = await self._requestToken(
^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\plane\PycharmProjects\PyPorsche\venv\Lib\site-packages\pyporscheconnectapi\connection.py", line 202, in _requestToken
await self._login()
File "C:\Users\plane\PycharmProjects\PyPorsche\venv\Lib\site-packages\pyporscheconnectapi\connection.py", line 149, in _login
raise WrongCredentials(message['message'])
~~~~~~~^^^^^^^^^^^
KeyError: 'message'
Unclosed client session
client_session: <aiohttp.client.ClientSession object at 0x000001AA684A0C50>
Unclosed connector
connections: ['[(<aiohttp.client_proto.ResponseHandler object at 0x000001AA6845B850>, 185288.109)]']
connector: <aiohttp.connector.TCPConnector object at 0x000001AA684A0CD0>
Am I the only one?
Any idea what I need to change?

I have the same problem, it comes from, that the Porsche API sometimes now wants a cpatcha to get resolved for first login :-( it depends on the IP Adress it looks like they now have a black list,...
After some time not trying to login it works again, but than (after relogin) maybe due to missing token refresh I'll be banned again,..
Any updates on token refresh?
Greetings

I got it back working! The fix is:
If you get the Mail from Porsche<EMAIL_ADDRESS>You must only click: reactivate account here!
Do not reset you Password on the upcoming homepage, because if you do your IP is again deleted from the whitelist and it is not working. After clicking here, I got it back working.
Kind regards

I am closing this issue as the APIs we relied on have been retired. C.f #46

I am closing this issue as the APIs we relied on have been retired. C.f #46


How about explaining the subscription header requirement???

I get the following message:
Trace is not available for requests with no subscription key specified.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0a3dd020-373d-9807-fcfd-30fedf005040
Version Independent ID: 34fe30d0-f225-ff8d-1f6b-2494f7aad42d
Content: Debug your APIs using request tracing in Azure API Management
Content Source: articles/api-management/api-management-howto-api-inspector.md
Service: api-management
GitHub Login: @vladvino
Microsoft Alias: apimpm

@eschneider999 Thank you for the feedback!  We are investigating this and will get back to you shortly.

@eschneider999 I went through the tutorial again myself and can confirm that I was able to get a successful trace.

Did you include Ocp-Apim-Subscription-Key?

That's the point, what is the subscription key? where do I get it? How do I set one up?

From: "kobulloc-MSFT"<EMAIL_ADDRESS>Sent: Saturday, July 7, 2018 2:01 PM
To: "MicrosoftDocs/azure-docs"<EMAIL_ADDRESS>Cc: "eschneider999"<EMAIL_ADDRESS>"Mention"<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] How about explaining the subscription header requirement??? (#11098)
@eschneider999 I went through the tutorial again myself and can confirm that I was able to get a successful trace.
Did you include Ocp-Apim-Subscription-Key?

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@eschneider999 That should be automatically generated and filled in by default (or at least it was for me).  When you went through the documentation tutorials, did you do the Quickstart > Create an instance - Portal and Tutorials > 1 - Import your first API before starting on the Tutorial > 6 - Debug your APIs?

no, was trying to do a trace on an existing site/api and it was not filled in and had no idea what to do.

From: "kobulloc-MSFT"<EMAIL_ADDRESS>Sent: Monday, July 9, 2018 12:47 PM
To: "MicrosoftDocs/azure-docs"<EMAIL_ADDRESS>Cc: "eschneider999"<EMAIL_ADDRESS>"Mention"<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] How about explaining the subscription header requirement??? (#11098)
@eschneider999 That should be automatically generated and filled in by default (or at least it was for me). When you went through the documentation tutorials, did you do the Quickstart > Create an instance - Portal and Tutorials > 1 - Import your first API before starting on the Tutorial > 6 - Debug your APIs?

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@eschneider999

Go to the Developer Portal
In the upper right, click on the drop down menu and select Profile
In your profile, you will be able to view/regenerate/copy your keys
Click on Show next to your key and copy it
Repeat the steps to test your API
Use the copied key as the OCP-Apim-Subscription-Key.

@vladvino We may want to add a line in the documentation about how to get the Ocp-Apim-Subscription-Key value as this question has appeared a couple times in the forums.  Something like:
"You can get the Ocp-Apim-Subscription-Key by going to the Developer Portal, looking at your Profile, and showing the keys under Your subscriptions."

@eschneider999 We are going to close this thread as resolved but if there are any further questions regarding this matter, please tag me in your reply and we will be happy to continue the conversation.


Missing arguments in documentation

I noticed there sometimes are arguments missing in the documentation, as for BootstrapFewShotWithRandomSearch
https://dspy-docs.vercel.app/api/optimizers/BootstrapFewShotWithRandomSearch
In this case it's:
        max_errors=10,
        stop_at_score=None,
        metric_threshold=None,

Is this on purpose or is it okay for me to open a PR?

Just updated! Thanks for raising.


fix typo resulting in autolog feature not working

Coverage remained the same at 37.871% when pulling 1ebcf8a960b06c9061be6d3e9c40832d152c9a9a on u9520107:rcint-4293 into 4b05c2fda9e90e92d33f42cd94974bb4ae10c7f1 on ringcentral:master.


Clippy complains about use of 0 as *const _

When I use lazy_static, for every static ref I create I get a clippy warning:
warning: `0 as *const _` detected. Consider using `ptr::null()`
  --> src/a.rs:b:9
   |
b |         lazy_static! {
   |         ^
   |
   = note: #[warn(zero_ptr)] on by default
   = help: for further information visit https://github.com/Manishearth/rust-clippy/wiki#zero_ptr
   = note: this error originates in a macro outside of the current crate

Not sure how to best address this, since the warning is about a line of code that runs as a constexpr.

ah... so you can't use ptr::null() because it's not const? 😞

The problem is that calling a const is a unstable feature.

Oh, right. Well, then this issue is blocked until calling const functions is stabilised.


Never send Content-Length for 204

With flask-restful, it's easy to make a malformed response by doing return '', 204 or return None, 204. It might be idiomatic, but it would result in a 204 response with a non-empty body (containing a JSON payload such as '' or null).
We used to reset the Content-Length to 0 for status code 204, but since 1c70c3e1855d32a285843c0851f52e20160f6656 we keep Content-Length: N while the body is eliminated (see BaseResponse.get_app_iter). This broke existing flask-restful applications:
https://github.com/flask-restful/flask-restful/issues/736
Per section 3.3.2 of RFC 7230,  "a server MUST NOT send a Content-Length header field in any response with a status code of 1xx (Informational) or 204 (No Content)". This change enforces it in get_wsgi_headers.

LGTM

What version did this fix get released to?

@randyesq still waiting for this to be released. The relevant issue has been locked by the project maintainer:
https://github.com/pallets/werkzeug/issues/1296


[404 page] Malay version

For Malay version, let's proceed with using google translation to translate it to Malay from English.
We will get this validated from client during UAT.

Updated the translations. Please check and close this card.
https://main--fonterra-fernleaf--hlxsites.hlx.live/ms/dssd

@mkbansal1 The alignment of text and layout as a whole is different, please consider aligning/styling it as per the original URL for the content part

@asthabh23 Live site use 404 page of Fonterra, so Fonterra design is rendered.
As we have created 404 page for Fernleaf, it has Fernleaf theme.

ok, I will ask the customer to validate this as part of UAT. for now I added the relevant label

This has been shared with customer


Fix formatting conditionals with comments

Description
Closes https://github.com/FuelLabs/sway/issues/4935

In order to write comments for the else block when it only contains a comment, I had to add the span to CodeBlockContents

Checklist

[x] I have linked to any relevant issues.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have updated the documentation where relevant (API docs, the reference, and the Sway book).
[x] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
[ ] I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
[ ] I have requested a review from the relevant team or maintainers.

I'm not sure why the markdown lint is failing...


Boolean metadata properties are being ignored when converting to descriptor

Overview
If I have a custom subclass of Metadata with the following metadata profile:
    metadata_profile = {
        "type": "object",
        "properties": {
            "name": {"type": "string", "pattern": fl.settings.NAME_PATTERN},
            "type": {"type": "string"},
            "path": {"type": ["string", "null"]},
            "filepaths": {"type": "array"},
            "files": {"type": "object"},
            "errors": {"type": "array"},
            "valid": {"type": "boolean"}
        }
    }

And I perform the to_descriptor() method the boolean "valid" will  be filtered out by:
https://github.com/frictionlessdata/framework/blob/5e092abae1167c6d8e452cb934b9f202ecf28801/frictionless/metadata.py#L409
I think one need a more robust parser given the fact that the type information is also there. If I write:
self.metadata_profile.get("properties", {})["valid"]

I get:
{'type': 'boolean'}

I think the implementation should be easy.

Nvm It was a typo from my side


Multi-user Capability

The current IVCTool does not support multiple user. This effects the GUI/UI, the Command Messages on the JMS bus, the test case engine, and the log-file generation.

Priority of this feature has been discussed and will be scheduled as future enhancements


[feat] Allow any test attribute to be logged

EDIT: @vkarak
This PR adds the ability to log any test attribute. If the attribute cannot be found, <undefined> will be logged instead. If an attribute is deferrable it will be evaluated and if this is not possible <error> will be logged.

Codecov Report

Merging #1600 (33df163) into master (9406445) will decrease coverage by 0.62%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master    #1600      +/-   ##
==========================================
- Coverage   87.80%   87.18%   -0.63%
==========================================
  Files          45       45
  Lines        7112     6858     -254
==========================================
- Hits         6245     5979     -266
- Misses        867      879      +12

Impacted Files
Coverage Δ

reframe/core/logging.py
82.24% <66.66%> (-1.14%)
:arrow_down:

reframe/core/runtime.py
89.47% <0.00%> (-5.04%)
:arrow_down:

reframe/core/modules.py
57.48% <0.00%> (-2.11%)
:arrow_down:

reframe/core/systems.py
86.56% <0.00%> (-1.97%)
:arrow_down:

reframe/core/schedulers/slurm.py
52.69% <0.00%> (-0.92%)
:arrow_down:

reframe/core/deferrable.py
96.64% <0.00%> (-0.87%)
:arrow_down:

reframe/core/schedulers/pbs.py
64.00% <0.00%> (-0.71%)
:arrow_down:

reframe/core/pipeline.py
91.59% <0.00%> (-0.48%)
:arrow_down:

reframe/core/environments.py
96.62% <0.00%> (-0.43%)
:arrow_down:

reframe/core/schedulers/__init__.py
97.35% <0.00%> (-0.42%)
:arrow_down:

... and 21 more

Continue to review full report at Codecov.

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

I have finalized this PR. @rsarm @ekouts Have a look in case something is missing.

lgtm


Activations REST - course code missing from JSON

The activations endpoint /activation/item/{uuid}/{version} has a reference to the course contained in the JSON which used to have the course code included and some external scripts were relying on that so we need to put it back.

fixed


Offset in the links of the website Private Stake section

⚠ There is an offset in the links of the website Private Stake section ⚠
💠Delegate explorer button link ▶  https://docs.xcash.foundation/dpops/vote-and-staking ▶  http://delegates.xcash.foundation/delegates 😉
💠ROI Calculator button link ▶ https://delegates.xcash.foundation/  ▶ https://calculator.xcash.foundation/ 😉
💠Docs button link ▶ https://calculator.xcash.foundation/ ▶ https://docs.xcash.foundation/dpops/vote-and-staking 😉

fix is on the way!
Thank you!


Null Reference Error occurs sometime when retrieving data from elastic search when maximum RAM was utilized in server

NEST/Elasticsearch.Net version:
Elasticsearch version:
Description of the problem including expected versus actual behavior:
Steps to reproduce:
1.
2.
3.
Provide ConnectionSettings (if relevant):
Provide DebugInformation (if relevant):

Describe the feature:

This sounds unfortunate but without a stacktrace or anything else we have far too little information to look into this. Closing this but feel free to update the ticket with more infomation and we'll reopen this.