MediaOutcome.error of Response code: Forbidden

I tried to consume media like this:
        Aws::KinesisVideoMedia::KinesisVideoMediaClient mediaClient(Aws::Auth::AWSCredentials(redacted,redacted),config);

        Aws::KinesisVideoMedia::Model::StartSelector start;
        start.SetStartSelectorType(Aws::KinesisVideoMedia::Model::StartSelectorType::NOW);

        Aws::KinesisVideoMedia::Model::GetMediaRequest mediaReq;
        mediaReq.SetStreamName("my-stream");
        mediaReq.SetStartSelector(start);

        Aws::KinesisVideoMedia::Model::GetMediaOutcome mediaOutcome;
        mediaOutcome = mediaClient.GetMedia(mediaReq);

But that resulted in a MediaOutcome.error of Response code: Forbidden with inner error of Failed to parse error payload. I reread the c++ docs and I don't see anything obvious I am missing. A known good example would be very helpful.

MediaOutcome.error
```
Sucess= 1Printing description of mediaOutcome.error:
(Aws::Client::AWSError) error = {
  m_errorType = UNKNOWN
  m_exceptionName = {
    __r_ = {
      std::__1::__compressed_pair_elem, Aws::Allocator >::__rep, 0, false> = {
        __value_ = {
           = {
            __l = (__cap_ = 0, __size_ = 0, __data_ = 0x0000000000000000)
            __s = {
               = (__size_ = '\0', __lx = '\0')
              __data_ = {
                [0] = '\0'
                [1] = '\0'
                [2] = '\0'
                [3] = '\0'
                [4] = '\0'
                [5] = '\0'
                [6] = '\0'
                [7] = '\0'
                [8] = '\0'
                [9] = '\0'
                [10] = '\0'
                [11] = '\0'
                [12] = '\0'
                [13] = '\0'
                [14] = '\0'
                [15] = '\0'
                [16] = '\0'
                [17] = '\0'
                [18] = '\0'
                [19] = '\0'
                [20] = '\0'
                [21] = '\0'
                [22] = '\0'
              }
            }
            __r = {
              __words = ([0] = 0, [1] = 0, [2] = 0)
            }
          }
        }
      }
    }
  }
  m_message = {
    __r_ = {
      std::__1::__compressed_pair_elem, Aws::Allocator >::__rep, 0, false> = {
        __value_ = {
           = {
            __l = (__cap_ = 97, __size_ = 89, __data_ = "Failed to parse error payload with address : <IP_ADDRESS> with address : <IP_ADDRESS>")
            __s = {
               = (__size_ = 'a', __lx = 'a')
              __data_ = {
                [0] = '\0'
                [1] = '\0'
                [2] = '\0'
                [3] = '\0'
                [4] = '\0'
                [5] = '\0'
                [6] = '\0'
                [7] = 'Y'
                [8] = '\0'
                [9] = '\0'
                [10] = '\0'
                [11] = '\0'
                [12] = '\0'
                [13] = '\0'
                [14] = '\0'
                [15] = '\0'
                [16] = '?'
                [17] = 'p'
                [18] = '\x03'
                [19] = '\x01'
                [20] = '\0'
                [21] = '\0'
                [22] = '\0'
              }
            }
            __r = {
              __words = ([0] = 97, [1] = 89, [2] =<PHONE_NUMBER>)
            }
          }
        }
      }
    }
  }
  m_responseHeaders = size=4 {
    __tree_ = {
      __begin_node_ = 0x00000001037037e0
      __pair1_ = {
        std::__1::__compressed_pair_elem *>, 0, false> = {
          __value_ = {
            __left_ = 0x0000000103703630
          }
        }
      }
      __pair3_ = {
        std::__1::__compressed_pair_elem = (__value_ = 4)
      }
    }
  }
  m_responseCode = FORBIDDEN
  m_isRetryable = false
}
```

m_responseHeaders
Printing description of mediaOutcome.error.m_responseHeaders:
(Aws::Http::HeaderValueCollection) m_responseHeaders = size=4 {
  [0] = {
    first = {
      __r_ = {
        std::__1::__compressed_pair_elem<std::__1::basic_string<char, std::__1::char_traits<char>, Aws::Allocator<char> >::__rep, 0, false> = {
          __value_ = {
             = {
              __l = (__cap_ =<PHONE_NUMBER>067597076, __size_ = 7237481, __data_ = <no value available>)
              __s = {
                 = (__size_ = '\x14', __lx = '\x14')
                __data_ = {
                  [0] = 'c'
                  [1] = 'o'
                  [2] = 'n'
                  [3] = 'n'
                  [4] = 'e'
                  [5] = 'c'
                  [6] = 't'
                  [7] = 'i'
                  [8] = 'o'
                  [9] = 'n'
                  [10] = '\0'
                  [11] = '\0'
                  [12] = '\0'
                  [13] = '\0'
                  [14] = '\0'
                  [15] = '\0'
                  [16] = '\0'
                  [17] = '\0'
                  [18] = '\0'
                  [19] = '\0'
                  [20] = '\0'
                  [21] = '\0'
                  [22] = '\0'
                }
              }
              __r = {
                __words = ([0] =<PHONE_NUMBER>067597076, [1] = 7237481, [2] = 0)
              }
            }
          }
        }
      }
    }
    second = {
      __r_ = {
        std::__1::__compressed_pair_elem<std::__1::basic_string<char, std::__1::char_traits<char>, Aws::Allocator<char> >::__rep, 0, false> = {
          __value_ = {
             = {
              __l = (__cap_ =<PHONE_NUMBER>18634, __size_ = 0, __data_ = <no value available>)
              __s = {
                 = (__size_ = '\n', __lx = '\n')
                __data_ = {
                  [0] = 'c'
                  [1] = 'l'
                  [2] = 'o'
                  [3] = 's'
                  [4] = 'e'
                  [5] = '\0'
                  [6] = '\0'
                  [7] = '\0'
                  [8] = '\0'
                  [9] = '\0'
                  [10] = '\0'
                  [11] = '\0'
                  [12] = '\0'
                  [13] = '\0'
                  [14] = '\0'
                  [15] = '\0'
                  [16] = '\0'
                  [17] = '\0'
                  [18] = '\0'
                  [19] = '\0'
                  [20] = '\0'
                  [21] = '\0'
                  [22] = '\0'
                }
              }
              __r = {
                __words = ([0] =<PHONE_NUMBER>18634, [1] = 0, [2] = 0)
              }
            }
          }
        }
      }
    }
  }
  [1] = {
    first = {
      __r_ = {
        std::__1::__compressed_pair_elem<std::__1::basic_string<char, std::__1::char_traits<char>, Aws::Allocator<char> >::__rep, 0, false> = {
          __value_ = {
             = {
              __l = (__cap_ =<PHONE_NUMBER>581218076, __size_ =<PHONE_NUMBER>0502317, __data_ = <no value available>)
              __s = {
                 = (__size_ = '\x1c', __lx = '\x1c')
                __data_ = {
                  [0] = 'c'
                  [1] = 'o'
                  [2] = 'n'
                  [3] = 't'
                  [4] = 'e'
                  [5] = 'n'
                  [6] = 't'
                  [7] = '-'
                  [8] = 'l'
                  [9] = 'e'
                  [10] = 'n'
                  [11] = 'g'
                  [12] = 't'
                  [13] = 'h'
                  [14] = '\0'
                  [15] = '\0'
                  [16] = '\0'
                  [17] = '\0'
                  [18] = '\0'
                  [19] = '\0'
                  [20] = '\0'
                  [21] = '\0'
                  [22] = '\0'
                }
              }
              __r = {
                __words = ([0] =<PHONE_NUMBER>581218076, [1] =<PHONE_NUMBER>0502317, [2] = 0)
              }
            }
          }
        }
      }
    }
    second = {
      __r_ = {
        std::__1::__compressed_pair_elem<std::__1::basic_string<char, std::__1::char_traits<char>, Aws::Allocator<char> >::__rep, 0, false> = {
          __value_ = {
             = {
              __l = (__cap_ = 808661254, __size_ = 0, __data_ = <no value available>)
              __s = {
                 = (__size_ = '\x06', __lx = '\x06')
                __data_ = {
                  [0] = '1'
                  [1] = '3'
                  [2] = '0'
                  [3] = '\0'
                  [4] = '\0'
                  [5] = '\0'
                  [6] = '\0'
                  [7] = '\0'
                  [8] = '\0'
                  [9] = '\0'
                  [10] = '\0'
                  [11] = '\0'
                  [12] = '\0'
                  [13] = '\0'
                  [14] = '\0'
                  [15] = '\0'
                  [16] = '\0'
                  [17] = '\0'
                  [18] = '\0'
                  [19] = '\0'
                  [20] = '\0'
                  [21] = '\0'
                  [22] = '\0'
                }
              }
              __r = {
                __words = ([0] = 808661254, [1] = 0, [2] = 0)
              }
            }
          }
        }
      }
    }
  }
  [2] = {
    first = {
      __r_ = {
        std::__1::__compressed_pair_elem<std::__1::basic_string<char, std::__1::char_traits<char>, Aws::Allocator<char> >::__rep, 0, false> = {
          __value_ = {
             = {
              __l = (__cap_ =<PHONE_NUMBER>52, __size_ = 0, __data_ = <no value available>)
              __s = {
                 = (__size_ = '\b', __lx = '\b')
                __data_ = {
                  [0] = 'd'
                  [1] = 'a'
                  [2] = 't'
                  [3] = 'e'
                  [4] = '\0'
                  [5] = '\0'
                  [6] = '\0'
                  [7] = '\0'
                  [8] = '\0'
                  [9] = '\0'
                  [10] = '\0'
                  [11] = '\0'
                  [12] = '\0'
                  [13] = '\0'
                  [14] = '\0'
                  [15] = '\0'
                  [16] = '\0'
                  [17] = '\0'
                  [18] = '\0'
                  [19] = '\0'
                  [20] = '\0'
                  [21] = '\0'
                  [22] = '\0'
                }
              }
              __r = {
                __words = ([0] =<PHONE_NUMBER>52, [1] = 0, [2] = 0)
              }
            }
          }
        }
      }
    }
    second = {
      __r_ = {
        std::__1::__compressed_pair_elem<std::__1::basic_string<char, std::__1::char_traits<char>, Aws::Allocator<char> >::__rep, 0, false> = {
          __value_ = {
             = {
              __l = (__cap_ = 33, __size_ = 29, __data_ = "Thu, 25 Jun 2020 13:56:26 GMT")
              __s = {
                 = (__size_ = '!', __lx = '!')
                __data_ = {
                  [0] = '\0'
                  [1] = '\0'
                  [2] = '\0'
                  [3] = '\0'
                  [4] = '\0'
                  [5] = '\0'
                  [6] = '\0'
                  [7] = '\x1d'
                  [8] = '\0'
                  [9] = '\0'
                  [10] = '\0'
                  [11] = '\0'
                  [12] = '\0'
                  [13] = '\0'
                  [14] = '\0'
                  [15] = '`'
                  [16] = '2'
                  [17] = 'p'
                  [18] = '\x03'
                  [19] = '\x01'
                  [20] = '\0'
                  [21] = '\0'
                  [22] = '\0'
                }
              }
              __r = {
                __words = ([0] = 33, [1] = 29, [2] =<PHONE_NUMBER>)
              }
            }
          }
        }
      }
    }
  }
  [3] = {
    first = {
      __r_ = {
        std::__1::__compressed_pair_elem<std::__1::basic_string<char, std::__1::char_traits<char>, Aws::Allocator<char> >::__rep, 0, false> = {
          __value_ = {
             = {
              __l = (__cap_ =<PHONE_NUMBER>345323040, __size_ =<PHONE_NUMBER>880126322, __data_ = <no value available>)
              __s = {
                 = (__size_ = ' ', __lx = ' ')
                __data_ = {
                  [0] = 'x'
                  [1] = '-'
                  [2] = 'a'
                  [3] = 'm'
                  [4] = 'z'
                  [5] = 'n'
                  [6] = '-'
                  [7] = 'r'
                  [8] = 'e'
                  [9] = 'q'
                  [10] = 'u'
                  [11] = 'e'
                  [12] = 's'
                  [13] = 't'
                  [14] = 'i'
                  [15] = 'd'
                  [16] = '\0'
                  [17] = '\0'
                  [18] = '\0'
                  [19] = '\0'
                  [20] = '\0'
                  [21] = '\0'
                  [22] = '\0'
                }
              }
              __r = {
                __words = ([0] =<PHONE_NUMBER>345323040, [1] =<PHONE_NUMBER>880126322, [2] = 100)
              }
            }
          }
        }
      }
    }
    second = {
      __r_ = {
        std::__1::__compressed_pair_elem<std::__1::basic_string<char, std::__1::char_traits<char>, Aws::Allocator<char> >::__rep, 0, false> = {
          __value_ = {
             = {
              __l = (__cap_ = 49, __size_ = 36, __data_ = "894c0d82-c602-41f3-a3dd-9faa5e6a9f8c")
              __s = {
                 = (__size_ = '1', __lx = '1')
                __data_ = {
                  [0] = '\0'
                  [1] = '\0'
                  [2] = '\0'
                  [3] = '\0'
                  [4] = '\0'
                  [5] = '\0'
                  [6] = '\0'
                  [7] = '$'
                  [8] = '\0'
                  [9] = '\0'
                  [10] = '\0'
                  [11] = '\0'
                  [12] = '\0'
                  [13] = '\0'
                  [14] = '\0'
                  [15] = 'p'
                  [16] = '5'
                  [17] = 'p'
                  [18] = '\x03'
                  [19] = '\x01'
                  [20] = '\0'
                  [21] = '\0'
                  [22] = '\0'
                }
              }
              __r = {
                __words = ([0] = 49, [1] = 36, [2] =<PHONE_NUMBER>)
              }
            }
          }
        }
      }
    }
  }
}
(lldb)

Originally posted by @jelling in https://github.com/aws/aws-sdk-cpp/issues/1399#issuecomment-649565932

@KaibaLopez here's the new issue and here is the debug log:

debug log

[INFO] 2020-06-25 21:44:46.065 Aws_Init_Cleanup [0x100118dc0] Initiate AWS SDK for C++ with Version:1.7.357
[TRACE] 2020-06-25 21:44:46.065 FileSystemUtils [0x100118dc0] Checking HOME for the home directory.
[DEBUG] 2020-06-25 21:44:46.065 FileSystemUtils [0x100118dc0] Environment value for variable HOME is /Users/jelling
[DEBUG] 2020-06-25 21:44:46.065 FileSystemUtils [0x100118dc0] Home directory is missing the final / appending one to normalize
[DEBUG] 2020-06-25 21:44:46.065 FileSystemUtils [0x100118dc0] Final Home Directory is /Users/jelling/
[INFO] 2020-06-25 21:44:46.065 Aws::Config::AWSConfigFileProfileConfigLoader [0x100118dc0] Initializing config loader against fileName /Users/jelling/.aws/credentials and using profilePrefix = 0
[TRACE] 2020-06-25 21:44:46.065 FileSystemUtils [0x100118dc0] Checking HOME for the home directory.
[DEBUG] 2020-06-25 21:44:46.065 FileSystemUtils [0x100118dc0] Environment value for variable HOME is /Users/jelling
[DEBUG] 2020-06-25 21:44:46.065 FileSystemUtils [0x100118dc0] Home directory is missing the final / appending one to normalize
[DEBUG] 2020-06-25 21:44:46.065 FileSystemUtils [0x100118dc0] Final Home Directory is /Users/jelling/
[INFO] 2020-06-25 21:44:46.065 Aws::Config::AWSConfigFileProfileConfigLoader [0x100118dc0] Initializing config loader against fileName /Users/jelling/.aws/config and using profilePrefix = 1
[TRACE] 2020-06-25 21:44:46.065 FileSystemUtils [0x100118dc0] Checking HOME for the home directory.
[DEBUG] 2020-06-25 21:44:46.065 FileSystemUtils [0x100118dc0] Environment value for variable HOME is /Users/jelling
[DEBUG] 2020-06-25 21:44:46.065 FileSystemUtils [0x100118dc0] Home directory is missing the final / appending one to normalize
[DEBUG] 2020-06-25 21:44:46.065 FileSystemUtils [0x100118dc0] Final Home Directory is /Users/jelling/
[DEBUG] 2020-06-25 21:44:46.066 Aws::Config::ConfigFileProfileFSM [0x100118dc0] found profile pw
[DEBUG] 2020-06-25 21:44:46.066 Aws::Config::ConfigFileProfileFSM [0x100118dc0] found region us-east-1
[DEBUG] 2020-06-25 21:44:46.066 Aws::Config::ConfigFileProfileFSM [0x100118dc0] found access key AKIA5FQKCPUZ75PN72EW
[DEBUG] 2020-06-25 21:44:46.066 Aws::Config::ConfigFileProfileFSM [0x100118dc0] found profile default
[DEBUG] 2020-06-25 21:44:46.066 Aws::Config::ConfigFileProfileFSM [0x100118dc0] found region us-west-2ear
[DEBUG] 2020-06-25 21:44:46.066 Aws::Config::ConfigFileProfileFSM [0x100118dc0] found access key AKIA5FQKCPUZ75PN72EW
[INFO] 2020-06-25 21:44:46.066 Aws::Config::AWSProfileConfigLoader [0x100118dc0] Successfully reloaded configuration.
[TRACE] 2020-06-25 21:44:46.066 Aws::Config::AWSProfileConfigLoader [0x100118dc0] reloaded config at 2020-06-25T21:44:46Z
[TRACE] 2020-06-25 21:44:46.066 FileSystemUtils [0x100118dc0] Checking HOME for the home directory.
[DEBUG] 2020-06-25 21:44:46.066 FileSystemUtils [0x100118dc0] Environment value for variable HOME is /Users/jelling
[DEBUG] 2020-06-25 21:44:46.066 FileSystemUtils [0x100118dc0] Home directory is missing the final / appending one to normalize
[DEBUG] 2020-06-25 21:44:46.066 FileSystemUtils [0x100118dc0] Final Home Directory is /Users/jelling/
[DEBUG] 2020-06-25 21:44:46.067 Aws::Config::ConfigFileProfileFSM [0x100118dc0] found profile default
[DEBUG] 2020-06-25 21:44:46.067 Aws::Config::ConfigFileProfileFSM [0x100118dc0] found region us-east-1
[INFO] 2020-06-25 21:44:46.067 Aws::Config::AWSProfileConfigLoader [0x100118dc0] Successfully reloaded configuration.
[TRACE] 2020-06-25 21:44:46.067 Aws::Config::AWSProfileConfigLoader [0x100118dc0] reloaded config at 2020-06-25T21:44:46Z
[INFO] 2020-06-25 21:44:46.067 CurlHttpClient [0x100118dc0] Initializing Curl library with version: 7.64.1, ssl version: SecureTransport (LibreSSL/2.8.3)
[DEBUG] 2020-06-25 21:44:46.068 ClientConfiguration [0x100118dc0] ClientConfiguration will use SDK Auto Resolved profile: [default] if not specified by users.
[WARN] 2020-06-25 21:44:46.068 ClientConfiguration [0x100118dc0] Retry Strategy will use the default max attempts.
[INFO] 2020-06-25 21:44:46.068 CurlHandleContainer [0x100118dc0] Initializing CurlHandleContainer with size 25
[TRACE] 2020-06-25 21:44:46.068 AWSClient [0x100118dc0] Found body, but content-length has not been set, attempting to compute content-length
[DEBUG] 2020-06-25 21:44:46.068 AWSAuthV4Signer [0x100118dc0] Calculated sha256 b92ece021048ba8ac0873acd369022212246b79ca7120e7b7b395a6266022876 for payload.

[DEBUG] 2020-06-25 21:44:46.068 CurlHandleContainer [0x100118dc0] No current connections available in pool. Attempting to create new connections.
[DEBUG] 2020-06-25 21:44:46.068 CurlHandleContainer [0x100118dc0] attempting to grow pool size by 2
[INFO] 2020-06-25 21:44:46.068 CurlHandleContainer [0x100118dc0] Pool grown by 2
[INFO] 2020-06-25 21:44:46.068 CurlHandleContainer [0x100118dc0] Connection has been released. Continuing.
[DEBUG] 2020-06-25 21:44:46.068 CurlHandleContainer [0x100118dc0] Returning connection handle 0x10480aa00
[DEBUG] 2020-06-25 21:44:46.068 CurlHttpClient [0x100118dc0] Obtained connection handle 0x10480aa00
[TRACE] 2020-06-25 21:44:46.155 CurlHttpClient [0x100118dc0] HTTP/1.1 100 Continue

[TRACE] 2020-06-25 21:44:46.155 CurlHttpClient [0x100118dc0]

[TRACE] 2020-06-25 21:44:46.200 CurlHttpClient [0x100118dc0] HTTP/1.1 200 OK

[TRACE] 2020-06-25 21:44:46.200 CurlHttpClient [0x100118dc0] x-amzn-RequestId: 5fac6d44-a3e1-45ef-a77d-c57066497fdd

[TRACE] 2020-06-25 21:44:46.200 CurlHttpClient [0x100118dc0] Content-Type: application/json

[TRACE] 2020-06-25 21:44:46.200 CurlHttpClient [0x100118dc0] Content-Length: 74

[TRACE] 2020-06-25 21:44:46.200 CurlHttpClient [0x100118dc0] Date: Thu, 25 Jun 2020 21:44:45 GMT

[TRACE] 2020-06-25 21:44:46.200 CurlHttpClient [0x100118dc0]

[TRACE] 2020-06-25 21:44:46.200 CurlHttpClient [0x100118dc0] 74 bytes written to response.
[DEBUG] 2020-06-25 21:44:46.200 CurlHttpClient [0x100118dc0] Returned http response code 200
[DEBUG] 2020-06-25 21:44:46.200 CurlHttpClient [0x100118dc0] Returned content type application/json
[TRACE] 2020-06-25 21:44:46.200 CurlHttpClient [0x100118dc0] Response content-length header: 74
[TRACE] 2020-06-25 21:44:46.200 CurlHttpClient [0x100118dc0] Response body length: 74
[DEBUG] 2020-06-25 21:44:46.200 CurlHttpClient [0x100118dc0] Releasing curl handle 0x10480aa00
[DEBUG] 2020-06-25 21:44:46.200 CurlHandleContainer [0x100118dc0] Releasing curl handle 0x10480aa00
[DEBUG] 2020-06-25 21:44:46.200 CurlHandleContainer [0x100118dc0] Notified waiting threads.
[DEBUG] 2020-06-25 21:44:46.200 AWSClient [0x100118dc0] Request returned successful response.
[TRACE] 2020-06-25 21:44:46.200 AWSClient [0x100118dc0] Request successful returning.
[INFO] 2020-06-25 21:44:46.200 CurlHandleContainer [0x100118dc0] Initializing CurlHandleContainer with size 25
[TRACE] 2020-06-25 21:44:46.200 AWSClient [0x100118dc0] Found body, but content-length has not been set, attempting to compute content-length
[DEBUG] 2020-06-25 21:44:46.200 AWSAuthV4Signer [0x100118dc0] Calculated sha256 701876e5fcbb604dbbfe5c8e07d01e701b6dd35e22d14c52a67dff9bd7693d61 for payload.

[DEBUG] 2020-06-25 21:44:46.200 CurlHandleContainer [0x100118dc0] No current connections available in pool. Attempting to create new connections.
[DEBUG] 2020-06-25 21:44:46.200 CurlHandleContainer [0x100118dc0] attempting to grow pool size by 2
[INFO] 2020-06-25 21:44:46.200 CurlHandleContainer [0x100118dc0] Pool grown by 2
[INFO] 2020-06-25 21:44:46.200 CurlHandleContainer [0x100118dc0] Connection has been released. Continuing.
[DEBUG] 2020-06-25 21:44:46.200 CurlHandleContainer [0x100118dc0] Returning connection handle 0x104820c00
[DEBUG] 2020-06-25 21:44:46.200 CurlHttpClient [0x100118dc0] Obtained connection handle 0x104820c00
[TRACE] 2020-06-25 21:44:46.253 CurlHttpClient [0x100118dc0] HTTP/1.1 100 Continue

[TRACE] 2020-06-25 21:44:46.253 CurlHttpClient [0x100118dc0]

[TRACE] 2020-06-25 21:44:46.276 CurlHttpClient [0x100118dc0] HTTP/1.1 403 Forbidden

[TRACE] 2020-06-25 21:44:46.276 CurlHttpClient [0x100118dc0] x-amzn-RequestId: f514138c-85e5-4b2f-b8a3-f26a9b80e79e

[TRACE] 2020-06-25 21:44:46.276 CurlHttpClient [0x100118dc0] Content-Length: 130

[TRACE] 2020-06-25 21:44:46.276 CurlHttpClient [0x100118dc0] Date: Thu, 25 Jun 2020 21:44:46 GMT

[TRACE] 2020-06-25 21:44:46.276 CurlHttpClient [0x100118dc0] Connection: close

[TRACE] 2020-06-25 21:44:46.276 CurlHttpClient [0x100118dc0]

[TRACE] 2020-06-25 21:44:46.276 CurlHttpClient [0x100118dc0] 130 bytes written to response.
[DEBUG] 2020-06-25 21:44:46.278 CurlHttpClient [0x100118dc0] Returned http response code 403
[TRACE] 2020-06-25 21:44:46.278 CurlHttpClient [0x100118dc0] Response content-length header: 130
[TRACE] 2020-06-25 21:44:46.278 CurlHttpClient [0x100118dc0] Response body length: 130
[DEBUG] 2020-06-25 21:44:46.278 CurlHttpClient [0x100118dc0] Releasing curl handle 0x104820c00
[DEBUG] 2020-06-25 21:44:46.278 CurlHandleContainer [0x100118dc0] Releasing curl handle 0x104820c00
[DEBUG] 2020-06-25 21:44:46.278 CurlHandleContainer [0x100118dc0] Notified waiting threads.
[DEBUG] 2020-06-25 21:44:46.278 AWSClient [0x100118dc0] Request returned error. Attempting to generate appropriate error codes from response
[ERROR] 2020-06-25 21:44:46.278 AWSClient [0x100118dc0] HTTP response code: 403
Exception name:
Error message: Failed to parse error payload with address : <IP_ADDRESS>
4 response headers:
connection : close
content-length : 130
date : Thu, 25 Jun 2020 21:44:46 GMT
x-amzn-requestid : f514138c-85e5-4b2f-b8a3-f26a9b80e79e
[WARN] 2020-06-25 21:44:46.278 AWSClient [0x100118dc0] If the signature check failed. This could be because of a time skew. Attempting to adjust the signer.
[DEBUG] 2020-06-25 21:44:46.278 AWSClient [0x100118dc0] Server time is Thu, 25 Jun 2020 21:44:46 GMT, while client time is Thu, 25 Jun 2020 21:44:46 GMT
[INFO] 2020-06-25 21:44:46.278 CurlHandleContainer [0x100118dc0] Cleaning up CurlHandleContainer.

[DEBUG] 2020-06-25 21:44:46.278 CurlHandleContainer [0x100118dc0] Cleaning up 0x104809200
[DEBUG] 2020-06-25 21:44:46.278 CurlHandleContainer [0x100118dc0] Cleaning up 0x10480aa00

If you are building SDK from source, could you log the actual response before this line:
https://github.com/aws/aws-sdk-cpp/blob/569c8895e841d23bea9d63f2e24ac465d8084163/aws-cpp-sdk-core/source/client/AWSErrorMarshaller.cpp#L39
AWS_LOGSTREAM_TRACE(AWS_ERROR_MARSHALLER_LOG_TAG, "Response:  " << httpResponse.GetResponseBody().redbuf());

Or a breakpoint.
Just curious what's the incomplete payload.
A guess is that Kinesis failed to return all the responses and then timeout.
So could you also try to increase the timeout to see if it helps?


How to introduce custom models and even customizing Team model?

Title explains..
Custom models with some relationships ...
How can we handle those stuff? 🤔

Figured out just adding rails models. It's not yet clear why the models directory is not working just like actions or events or commands.


Random behaviour of Depends functionality.

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
'''Main Websocket Route'''

@app.websocket("/{obj_id}")
async def websocket_endpoint(websocket: WebSocket, obj_id: str, active_clients = Depends(get_active_clients)):
    await websocket.accept()
    try:
        await ws_manager.connect(websocket, obj_id, active_clients)
    except WebSocketDisconnect:
        await websocket.close()

'''Connection Manager'''
class ConnectionManager:

    async def connect(self, websocket: WebSocket, obj_id: str, active_connections):
        client = ServerPoint(obj_id, websocket)
        try:
            active_connections[cp_id] = client
            await client.start()

        except WebSocketDisconnect:
            active_connections.pop(cp_id)

ws_manager = ConnectionManager()
active_clients: Dict[str, ServerPoint] = {}

'''Dependency function'''
async def get_active_clients() -> Dict[str, ServerPoint]:
    return active_clients

'''Sample Test'''
@app.get("/test-active)
def test_dict_keys(active_clients: Dict[str, ServerPoint] = Depends(get_active_clients)):
    clients = list(active_clients.keys())

    return clients

Description
Hi,
I have been working with FastAPI for managing websocket connections, and I am saving the websocket connections in a dictionary. However, whenever I try to access this active connection array, there are seemingly random instances wherein this array returns empty while at other times, it outputs the correct connection instances.
My workflow is derived from the websocket tutorial on the FastAPI website. I am using an external library that manages this websocket connection to perform various functionalities (defined as ServerPoint in the ConnectionManager class).
The major issue arises wherein I have defined an additional HTTP route which looks to perform certain operations defined in the ServerPoint class on an individual connection stored in the dictionary, but the behaviour of the Depends(get_active_clients) is pretty erratic with it returning an empty dictionary on some occasions and returning full instances on another.
I created a minimal testing route to check the contents of the dictionary but somehow, it gives mixed results at each call. It is difficult to reproduce the error as I am not able to understand any real pattern to the behaviour.
It would be greatly helpful if someone could guide me to solve this issue and be able to generate a robust workflow.
Thanks
Operating System
Linux
Operating System Details
No response
FastAPI Version
0.85.0
Python Version
3.9.13
Additional Context
No response

Can you provide an example of the issue that you're facing?

Are you running with multiple workers? - just trying to figure out.


release 0.4.2 on conda/pypi

This is the follow-on to #29. Since no new features were added and the public API didn't change, I think this should be 0.4.2, and not 0.5.0. Could be argued otherwise!

This is now shipped.
pip install nbpresent
# or
conda install -c nbcio nbpresent

Do we want to wait a tick before pushing this to the anaconda-cloud channel?


theme-product-form doesn't work with some third party apps

In the theme-product-form _onSubmit function, there is a call to the _setIdInputValue function:
this._setIdInputValue(event.dataset.variant.id);

This finds the input with the name="id" and changes the value to the selected variant ID. For some third parties, eg. Recharge, this is incompatible with how they work (in the case of Recharge, the default input is replaced with an alternative to add a different product to cart). This means the replacement input is incorrectly assigned a value which doesn't exist, resulting in an add to cart error.
It seems to me that re-setting the value of this input here is quite opinionated and doesn't allow for some adjustments to behaviour that might be quite legitimate.

@t-kelly @statementtom
ReCharge add custom select and remove name="id" from original select to own one. When _setIdInputValue try to set variant value to new element variant value does not exist in ReCharge select, so value sets to "". Possible solution to allow users to use custom selectors:
https://github.com/savchukoleksii/theme-scripts/blob/theme-product-form/selectors-option/packages/theme-product-form/theme-product-form.js#L52.
On the other side, many apps depends on id input and add custom event listeners to id select. ReCharge is one of them.
You can explore more about this problem here: https://github.com/Shopify/theme-scripts/issues/146#issuecomment-767773548

@t-kelly @statementtom
ReCharge add custom select and remove name="id" from original select to own one. When _setIdInputValue try to set variant value to new element variant value does not exist in ReCharge select, so value sets to "". Possible solution to allow users to use custom selectors:
https://github.com/savchukoleksii/theme-scripts/blob/theme-product-form/selectors-option/packages/theme-product-form/theme-product-form.js#L52.
On the other side, many apps depends on id input and add custom event listeners to id select. ReCharge is one of them.
You can explore more about this problem here: https://github.com/Shopify/theme-scripts/issues/146#issuecomment-767773548


fn matcher_from_integration_json in mockserver/bodies.rs doesn't support all MatchingRules

fn matcher_from_integration_json in mockserver/bodies.rs doesn't support all MatchingRule types.
v4 Matching rules like server, boolean, statusCode` etc are not supported using the format described in https://github.com/pact-foundation/pact-reference/blob/master/rust/pact_ffi/IntegrationJson.md.
Also, the documentation says pact:matcher:type for date/time values should be datetime but the code uses timestamp.

Just a note that matcher_from_integration_json is marked as deprecated and should be replaced with MatchingRule::create.

matcher_from_integration_json has now been refactored and supports all matching rules


Get messages from comments

Hi. In #13 you instructed how to get retrieve additional data and I 'm trying here:
My goal is to get all the comments from a given post in the moderate view.
I 'm tryng inside "from_raw" method :
messages = []

        if 'comments' in raw:
            for x in raw['comments']['data']:
                messages.append(x['message'])
        mensagens = {el for el in messages}
        print (mensagens)

and add to the cls return:
return cls(
comments=mensagens, )

I succesful print a dict with all the comments from the posts retrieved. I 'm having troubles to view those in the moderate page. I 'm tryng right under https://github.com/LUKKIEN/wagtailsocialfeed/blob/master/wagtailsocialfeed/templates/wagtailsocialfeed/admin/moderate.html#L51 :+1:
                {% for item in post.comments %}

{{ item }}

                {% endfor %}

nothing happens.
could you help me to achieve this ?
cheers

If you return a FeedItem like you demonstrated with your code, the FeedItem will be missing all of the other required information.
The current attributes that are being passed on to the FeedItem are;
return cls(
            id=raw['id'],
            type='facebook',
            text=item_type.get_text_from(raw),
            image_dict=image,
            posted=cls.get_post_date(raw),
            original_data=raw,
        )

Before you return the new FeedItem, you could set extra attributes and then return it.
Or, as all the original information is still available via FeedItem.original_data, in your template code you could do something like:
{% for comment_data in post.original_data.comments.data %}

{{ comment_data.message }}

{% endfor %}


device went offline and never recovered

i have been using an older version of this integration for months (since march-ish) and its worked flawlessly with my lazyspa milan. honestly this integration was the single best thing in the world!
Bestway device
lazy spa milan
Describe the bug
a few days ago i noticed that my automation that sets the spa temp and turns it on stopped working. Upon investigation i noticed that all the entities were offline. I checked for updates to the integration this morning and installed the latest version (1.2.0) this duplicated all my entities but they still show as offline.
i went ahead and removed all the devices within the integration and confirmed that my username and password were correct in the bestway mobile app.
i then added a new device entered my details and now have 1 device and 11 entities.  (screenshot 1)
if i look at the entities themselves i can see they all have a warning status except for spa connected.  (screenshot 2)
Logs
looking under logs and searching for bestway im not seeing any logging at all. i have enabled debug logging.
as i said before this was working perfectly for months. Im guessing something has changed with the spa or the api you call in bestway. happy to get some packet sniffing done or anything else to try and get to the bottom of it. but i would need some direction on how to do it.
thanks
James

Morning guys.
so this morning I got up to find my automation was working and the tub was showing as online in HA.
Very strange behaviour.

Unfortunately the HTTP API used by the integration seems a bit flaky at times. I think the app uses a websocket connection which explains why the app can seem fine when the integration does not. Anyway, glad to see it working again.


Improvement: Add an image to README.md

People would like to see how it looks before clone it. I'll try to solve this later.

There's a link in the readme to a fully working demo, I would think that would be enough to see how it looks before cloning. Are people wanting more than that?

Yes please! :-)
The problem with the biogears demo is that, at least as far as we can tell, that page has been heavily customized beyond the a default doxygen build. It would be really help to generate a doxygen build without doxygen-bootstrapped and one with it, but with no additional modifications to see how they differ.
If additional modifications are required to get some different behavior, then an example or two with images would be very helpful as well.

Sorry, I just noticed the example site, so maybe additional examples are not necessary. My bad!


Dino model

Finalized DINO model architecture split into three main files:

dino.py containing the DINONet and Spec classes
layers.py containing the inner layers that the DINI model builds on top: DINO Back Bone ( vision Transfomer) and DINO head (projection head)
utils.py containing utility functions that the different DINO modules use

and a testing script under test_dinonet.py, which displays the DINO model in action against image data.

@OmarJabri7 That's a great work you did, can you message me when this is merged so that we can have a blog post draft about it?


Avoid logging Data and Args when dequeuing

https://bugzilla.redhat.com/show_bug.cgi?id=1297576
Informational messages were being logged when a msg was dequeued from MiqQueue.
MiqPassword.sanitize_string was failing to clean LDAP credentials from the msg.args
Initially I had set out to solve this by improving MiqPassword.sanitize_string to better
clean LDAP credentials from the msg.args and realized it might not always be possible
given the unpredictable nature of what could be stored in msg.args in the future.
@kbrock suggest simply never logging msg.args or msg.data, avoiding the possibility of
every accidentally logging credentials and improve processing in this section of the code.
This is what this PR proposes.
@Fryguy @chessbyte Please review/comment

I'm not really sure on this approach.  I get the security things with args, but the downside is we can't see what the message being acted upon is given.  What really should happen is that unencrypted things should not be put on the queue.  In many other places, we've found those locations and made it enrypt onto the queue, and decrypt off of the queue.  @chessbyte Thoughts?
I don't understand the data removal here at all.  The old code didn't reveal any data...only the size.

@Fryguy Thank you for the comments.
I only removed the data too as a way to reduce time/size logging

@Fryguy @chessbyte;
After talking with @Fryguy I have decided to encrypt the creds when putting them on the queue and decrypt when removing them.
Stay tuned for that update.

@jvlcek we were not logging the data, just the size of the data.  Not sure why you removed that.  Args are important to continue logging, but without credentials.

@Fryguy I agree it would be best to store credentials in a table/security vault. And the next best thing is to encrypt them when sending over the wire. The logger knows to hide these.
It does seem easy to forget to encrypt entries that are passed into the queue. And we won't be able to detect these (especially when in an array)
Also, this is our background processor / message bus / rpc mechanism. Do we really need to log every message that goes over the wire including all the parameters?
I bet we would be able to notice/measure a performance boost If we remove args and data from logging.

@kbrock Thank you for the comments.
@chessbyte Yes I talked with @Fryguy and I will encrypt and then mask the credentials and log the Args.
Closing this PR and will open a new one with the alternate solution


feat(lint/noRestrictedImports): add rule

Adds no-restricted-imports rule to Biome.

Btw, we’ve had some discussion in the past about where we want to go with the useImportRestrictions rule, which can still be found over here: https://github.com/rome/tools/issues/4678
Unfortunately most of the ideas got blocked on needing project-level analysis (on the roadmap for 2024 now), so only the package private logic ended up being implemented. And there’s no config yet, since we only have one option implemented :)
So if you’re okay with adding bannedModules to the configuration for useImportRestrictions, then we can put the existing logic behind the packagePrivate option we discussed there. It’s fine to make ”none” the default, and setting it to ”all” would enable the existing logic in that rule.

This rule, without options, does nothing. This means that the options are mandatory for this rule, and it is in contrast with all the rules we have.
Which means that if I enabled this rule like this:
{
	"noRestrictedImports": "warn"
}

Biome is happy, but the rule is never triggered. I have mixed feelings about creating a rule that only works if options are passed. And our infrastructure/deserialization wasn't designed for this case.
I like @arendjr suggestion. We could merge the options/logic of this rule to noImportRestrictions.

@ematipico Sounds like a plan! :+1:
We may just need to bikeshed a bit about the naming for the allowPrivateImports option :) I do think it should add least remain an enum, so we can add a third option to support package private annotations when we have the infra to analyze annotations from imports.

Always open to bikeshed names! I'm bad at that

I personally think we should have 1-to-1 correlation when we are migrating existing eslint rules but I'm ok with merging it with another rule as well.

This rule, without options, does nothing. This means that the options are mandatory for this rule, and it is in contrast with all the rules we have.
Which means that if I enabled this rule like this:
{
	"noRestrictedImports": "warn"
}

Biome is happy, but the rule is never triggered. I have mixed feelings about creating a rule that only works if options are passed. And our infrastructure/deserialization wasn't designed for this case.
I like @arendjr suggestion. We could merge the options/logic of this rule to useImportRestrictions, although the options of this rules have a different objective compared to useImportRestrictions. The latter prevents the use of sub-paths.
So here my proposal:
1. Merge this rule

2. Update `useImportRestrictions` to accept two options:

   * `bannedModules`: the options of **this** rule
   * `allowPrivateImports`: allows to turn **off** the main logic of `useImportRestrictions`

3. Add the logic of **this** rule to `useImportRestrictions`

4. Delete `noRestrictedImports`

@anonrig @arendjr @Conaclos, how does that sound?
@anonrig if you like the idea, would like to take charge of the proposal?

I have mixed feeling about merging the two rules together. Maybe the name useImportRetrictions is not enough informative? I don't remember the rationals behind useImportRetrictions. I need more time to think about that. In the meantime we could merge this PR. The rule is a nursery rule, we could still change it / merge it with useImportRetrictions before the 1.6 release or in a future release.

I think this PR is ready to merge. Regarding merging with another rule, we can make that decision later since this is a Nursery linting rule

Nice, any plan to also support the patterns option from https://eslint.org/docs/latest/rules/no-restricted-imports?

@yann-combarnous

Nice, any plan to also support the patterns option from https://eslint.org/docs/latest/rules/no-restricted-imports?

I plan to add this in a future version.


warn: False ignored by ansible

Describe the bug
Hello,
It seems that ansible does not understand syntax of warn command:
statically imported: ansible-zabbix-agent/tasks/RedHat.yml
[WARNING]: Ignoring invalid attribute: warn
Installation method/version

Github using latest/master

Ansible Version
ansible 2.6.1
Targetted hosts
Concerns the following OS(es):

Ubuntu

Expected behavior
Ansible should be able to parse role without warninigs.
Additional context

I merged a PR that fixes this warning. Thank you!


Need a tag for 0.0.12-pre1(for Bower)

We're being hit by the bug fixed here:
https://github.com/christopherthielen/ui-router-extras/commit/5aba1345152f589d6cf913d26b5b00c6ee0f1884
It looks like Bower won't resolve the 0.0.12-pre1 version without a tag.
Thanks for the all the work on this.  It saved us a significant amount of time.

+1

I forked it and ran the build.
From my bower file:
"ui-router-extras": "https://github.com/impatient/ui-router-extras.git#7ca4f9f283fab8d097aef5f1e02318e7dfe21efa"
I would obviously prefer to use the main repo.  I'm assuming this ticket will get bumped when a new release is made.

Thanks!


Add Photoshop Beta support section to docs

Photoshop has added native support for OCIO. Currently in beta

@carolalynn @doug-walker Can someone push the change please? It'll be good to have the web page up to date with what's out there.


test: reduce duration of E2E tests

Describe what should be investigated or refactored
Our E2E tests currently take about 20 minutes to run. We should aim to speed up this feedback loop by removing low-value tests and parallelizing where possible.

[ ] Remove TestMismatchedArchitectures

TestValidatePackageArchitecture unit test covers the same scenarios and more


Create Android app from server-side Blazor app

I already have my Blazor server-side app created with Electron.Net. It's working good as a Windows application. Now, I would like to package it as an Android app. Is it possible with BlazorMobile and how can I achieve it with my existing app?

The default BlazorMobile template should be already for packaging both.
Actually the main difference with a regular Electron.NET with Blazor and the one is BlazorMobile is that the BlazorMobile implementation don't reference Electron.NET API directly from the Blazor project, instead it use a dependency injection service for interoping with your specific API, in order to keep your Blazor project not linked with any platform specific library.
See Communication between Blazor and Native section, and try to update your code to follow this pattern on your Blazor Electron.NET Project.
In my opinion just start from the BlazorMobile template and follow the README.
If you have any platform specific call to do on Android (like for storage, SQLite or else), you will have to implement them on your projects with the Xamarin Dependency service, in order to call them from the class libraru Xamarin project.
Actually it would be too long for me to explain everything here. Read the manual first, to see how it works, and then i suggest you to create a BlazorMobile solution project, and then use a diff tool, like WinMerge, or anything else, or inspecting yourself manually how Startup.cs, and the additionnal services, are registered, named etc.
Actually BlazorMobile support ElectronNET too so maybe the it will be easier to you to inspect the ElectronNET template for BlazorMobile.

Alright, thank you!

Don't hesitate to ask me detailled questions here if you have an issue to make this working.


Pull folders only

I have a Google Drive with a huge, deeply-nested file structure.  It is not feasible to store the contents of the Drive locally, and the folder structure is too complicated to keep in my head.
I would like to use drive pull to recreate the entire folder structure locally; then I could navigate the structure in the terminal and pull down individual folders when I need them.  Is this possible with the current version of drive?

Roger that. We could have a flag to just pull directories kind of the way you can specify only files with find . -type f or only directories with find . -type d. This is a useful feature. In fact I think we have it enabled for drive list --files to only list files.

Hello there @brandoncurtis, thank you for this request.
Please get the latest code and then you'll be able to do
$ drive push --directories a1/m1
$ drive pull --files tf1/tf2

Please see PR https://github.com/odeke-em/drive/pull/673.
If there are any issues, feel free to reopen this issue.

Brilliant.  Works great!  Thanks @odeke-em!

Thanks for checking it out @brandoncurtis and for the kind words :). Good morning!


Demos are broken

Hey @Grsmto,
it seems that the demo page is broken.
Both js and css sources of simplebar cannot be found (see:. http://grsmto.github.io/simplebar/demos/).

Hope this helps.
Kind Regards
Gregor

I can confirm this.


Suggestions for README.md updates

These are the things I noticed right now.
About removing some abbreviation like "they're", it because, when in positive form, it is usually considered better to avoid contraction of the words when writing a text.
In addition, I was wondering if the package also supports JS projects out of the box. This maybe worth a mention.

I have read about the contraction part.
I understand your point  and since Vue docs uses them, let's stick to it.
But if I remember correctly, the abbreviations are more used while speaking, while the written world is different. For example, even on our blog we do not use positive contractions, if I remember correctly.

the written world is different
More the formal world, and you know how I dislike these little formal details 😁


[Windows] Remove Go 1.18.x

Description
Per software support policy, remove Go 1.18.x from Windows images
Related issue: https://github.com/actions/runner-images/issues/8237
Check list

[ ] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run windows2019,windows2022


reStructuredText

Adds support for exporting datasets and databooks as reStructuredText.
This is useful for using data from a range of sources in documentation.
>>> data = Dataset().load(open('data.csv').read())
>>> data.export(rst)
=====  =====  =====
  A      B    A and
                B
=====  =====  =====
False  False  False
True   False  False
False  True   False
True   True   True
=====  =====  =====

(Travis failure seems unrelated. Build attempts to install numpy 1.13.1 (not a requirement of this PR) on Python 3.3, but it requires Python >= 3.4.)


Any reason why receive_batch_multipart requires 'static for body?

While updating the Rocket integration for 0.5.0-rc.1, I found that it is not possible to use async_graphql::http::receive_batch_body since it requires 'static lifetime for the data parameter and the recent Rocket version changed to require the lifetime of the body data to be the same with the lifetime of the request.
However, removing 'static requirement from both receive_batch_body and receive_batch_multipart simply fixed all the issues without any errors. Is there any specific reason that data parameter of receive_batch_multipart should be marked as 'static? Otherwise, I'll just make a pull request with the 'static requirement removed.

It seems that this 'static is unnecessary. 😁

I tried to remove the 'static constraint on the body, and no problem was found.

Great, just created the pull request as mentioned above(#545)


No obfuscate

You can optionally leave the obfuscation and minification. It's only for GitHub pages /docs build.
yarn build:dev will build pretty code to /public, for local development.

Awesome thanks very much, sorry about merging to your master, meant to keep it in my fork


brew zsh autocompletion broken

Operating system
MacOS 12.6
Helmfile Version
0.147.0
Helm Version
3.10.0
Bug description
After the latest call of brew update ; brew upgrade the helmfile command fails for autocompletion. brew uninstall and brew install of helmfile did not help.
Example helmfile.yaml

Error message you've seen (if any)
helmfile unknown flag: --generate-bash-completion
                 helmfile --version

Steps to reproduce

Working Helmfile Version

Relevant discussion
No response

try:
helmfile completion --help

Hi @yxxhero
I have found that help page. But beforehand the autocomplete install with brew worked 100% fine, why is it broken now?

@aeimer maybe since helmfile changed to use cobra as the CLI library. I will post an issue for the brew to resolve this.

@aeimer please show your helmfile version.

@aeimer BTW. brew is not maintained by us.

@aeimer if you have other questions. Please post a new issue. thanks very much.

@aeimer please show your helmfile version.

Well I did: 0.147.0
@yxxhero have you opened a ticket? Would you mind refrencing it? Thanks for the help

@aeimer I haven't done it yet. If you have time, you can propose an issue. If need my help. Please tell me.

Here we go: https://github.com/Homebrew/homebrew-core/issues/114158


N23: Add Medium.com Rust GameDev Article

_Part of #677 _
Adds an entry for my medium.com article, "Game Development with Rust"

Thanks for the PR! Good luck with the book's release! :)


[Epic] How the PAF progress looks in Hypha activity feed view

Additional context
This is a sub-issue of #2911
Description
This issue is to show how the PAF progress looks in the activity feed.

[x] PM submits completed PAF for approval from CA
[x] CA requests more information
[x] CA submits PAF to CEO, Finance, Compliance for approval
[x] CEO, Finance, Compliance request change
[x] CEO, Finance, Compliance for approval
[ ] Contract approved/signed/mutually executed

-- both CEO and contractor signed contract (I think we need to show this, but not sure right now. Unsure if we need to display contract has been signed. We need to find out if contract needs to be displayed).
Priority

High (keeping you from completing day-to-day tasks)

Ideal deadline
October 2022

Record: It is already done by an unknown PR. As it is tested and approved for live means it is already on test, so closing this issue.


istioctl analyzer warn on namespace not injected for istio injection

Bug description
I ran the cmd for my entire cluster and it is generating too many warnings for me.  I would much prefer the The namespace is not enabled for Istio injection an info vs a warning because this is most likely by user's choice, similarly as Info [IST0118] (Service cert-manager.cert-manager) Port name  (port: 9402, targetPort: 9402) doesn't follow the naming convention of Istio port.
Also, why do we give Error at the end after a bunch of Warn and Info?
$ istioctl analyze --all-namespaces
Warn [IST0102] (Namespace bookinfo) The namespace is not enabled for Istio injection. Run 'kubectl label namespace bookinfo istio-injection=enabled' to enable it, or 'kubectl label namespace bookinfo istio-injection=disabled' to explicitly mark it as not needing injection
Warn [IST0102] (Namespace cert-manager) The namespace is not enabled for Istio injection. Run 'kubectl label namespace cert-manager istio-injection=enabled' to enable it, or 'kubectl label namespace cert-manager istio-injection=disabled' to explicitly mark it as not needing injection
Warn [IST0102] (Namespace ibm-cert-store) The namespace is not enabled for Istio injection. Run 'kubectl label namespace ibm-cert-store istio-injection=enabled' to enable it, or 'kubectl label namespace ibm-cert-store istio-injection=disabled' to explicitly mark it as not needing injection
Warn [IST0102] (Namespace ibm-operators) The namespace is not enabled for Istio injection. Run 'kubectl label namespace ibm-operators istio-injection=enabled' to enable it, or 'kubectl label namespace ibm-operators istio-injection=disabled' to explicitly mark it as not needing injection
Warn [IST0102] (Namespace ibm-system) The namespace is not enabled for Istio injection. Run 'kubectl label namespace ibm-system istio-injection=enabled' to enable it, or 'kubectl label namespace ibm-system istio-injection=disabled' to explicitly mark it as not needing injection
Warn [IST0102] (Namespace istio2) The namespace is not enabled for Istio injection. Run 'kubectl label namespace istio2 istio-injection=enabled' to enable it, or 'kubectl label namespace istio2 istio-injection=disabled' to explicitly mark it as not needing injection
Warn [IST0102] (Namespace istio3) The namespace is not enabled for Istio injection. Run 'kubectl label namespace istio3 istio-injection=enabled' to enable it, or 'kubectl label namespace istio3 istio-injection=disabled' to explicitly mark it as not needing injection
Warn [IST0102] (Namespace kube-node-lease) The namespace is not enabled for Istio injection. Run 'kubectl label namespace kube-node-lease istio-injection=enabled' to enable it, or 'kubectl label namespace kube-node-lease istio-injection=disabled' to explicitly mark it as not needing injection
Warn [IST0102] (Namespace test) The namespace is not enabled for Istio injection. Run 'kubectl label namespace test istio-injection=enabled' to enable it, or 'kubectl label namespace test istio-injection=disabled' to explicitly mark it as not needing injection
Info [IST0118] (Service cert-manager.cert-manager) Port name  (port: 9402, targetPort: 9402) doesn't follow the naming convention of Istio port.
Error: Analyzers found issues when analyzing all namespaces.
See https://istio.io/docs/reference/config/analysis for more information about causes and resolutions.

[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ x] User Experience
[ ] Developer Infrastructure
Expected behavior
Steps to reproduce the bug
Version (include the output of istioctl version --remote and kubectl version --short and helm version if you used Helm)
How was Istio installed?
Environment where bug was observed (cloud vendor, OS, etc)
1.7 on IKS

@esnible @therealmitchconnors thoughts on this?

Howard and Lin suggest lowering level to INFO
We should not say ERROR if the highest level is WARNING


Update Issue Template: Bug report

Noting some issues I found in the issue template "Bug report":

[ ] Code of Conduct could include a link to the file

Have you read CoronaTracker's Code of Conduct? By filing an Issue, you are expected to comply with it, including treating everyone with respect

[ ] Add a troubleshooting guide or remove TODO line

Followed all applicable steps in the TROUBLESHOOTING guide: TODO

[ ] Add info under Versions sub-heading or remove TODO line

Versions
TODO

Closing in favor of .github repo #3


WIP: Gotty

Codecov Report

Merging #36 (7fb5e45) into main (c90bffe) will decrease coverage by 3.24%.
The diff coverage is 7.46%.

@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
- Coverage   34.59%   31.35%   -3.25%
==========================================
  Files          11       12       +1
  Lines        1474     1665     +191
==========================================
+ Hits          510      522      +12
- Misses        889     1060     +171
- Partials       75       83       +8

Impacted Files
Coverage Δ

util/tmux.go
0.00% <0.00%> (ø)

bot/bot.go
32.98% <8.21%> (-6.64%)
:arrow_down:

bot/session.go
38.80% <8.82%> (-5.56%)
:arrow_down:

util/shellescape.go
11.11% <11.11%> (ø)

config/config.go
92.30% <100.00%> (+0.20%)
:arrow_up:

Continue to review full report at Codecov.

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


xService Resource Missing Localization Files

Description of issue
xService Resource with latest xPSDesiredStateConfiguration (3.11.0) seems couldn't run on non-English OS.
Reason
This is because un-comment out Import-LocalizedData  LocalizedData -filename MSFT_xServiceResource.strings.psd1 with this commit.
https://github.com/PowerShell/xPSDesiredStateConfiguration/commit/463376df047e0f46c28e3b9f3e69fd9db5a9fd6a#diff-4b82dc2dcab9c16ff7ca0e0913b1e441R45
Previously this section was comment-out.
https://github.com/PowerShell/xPSDesiredStateConfiguration/blob/12ccf4555dfc358174386f40812678a981654094/DSCResources/MSFT_xServiceResource/MSFT_xServiceResource.psm1#L36
Because xService resource missing localization file, you can see only en-US is included.
https://github.com/PowerShell/xPSDesiredStateConfiguration/tree/12ccf4555dfc358174386f40812678a981654094/DSCResources/MSFT_xServiceResource

only en-US is there.

BTW In-Box PSDesiredStateConfiguration Service Resource never happen this issue as it contains localized file in C:\Windows\System32\WindowsPowerShell\v1.0\Modules\PSDesiredStateConfiguration

How to resolve
Add Localization directory for your language, or comment-out while covering languages.
However add language by user-self means you need to do another work after Install-Module xPSDesiredStateConfiguration, this is too sad. Maybe copy and paste same content of en-US and create all languages folder is much easier :(

I will re-comment the localization for now and re-release.

Thank you for prompt action.


Manual deployment mode

Automatic deployment can be a great feature but in some cases I'd like to just disable it so that images are pulled and services are deployed only when I want them to be, for example using the command "kontena stack deploy". Can you provide an option to disable the automatic features on the master at one or all of the levels: master, grid, stack, service? E.g.:
$ kontena grid update --auto-deploy 0 mygrid
$ kontena service update --auto-deploy 0 myservice

Would these "flags" disable all automatic deployments including reaction to missing instances (due to nodes gone awol)?

I would think reaction to failures should continue to operate as normal but I suppose there could be different levels like "off", "failures-only" and "on" instead of just 0/1.


Enable uwsgi cache

We cache home, blog and mediacenter. Cache is flushed every 5min.
Gain is about 8x faster on the KoomBook.
If the user is logged in as administrator the cache is disable.

I'm not a expert in uwsgi but :
When is the cache cleared ? Only every 5 minutes ?
If an admin changes the displayed cards, does a user have to wait 5 minutes before seeing the change ?
expires=60. Is it 60 seconds ? Where are you specifying the 5 minutes (300 seconds) ?
Why there are some route = ... and some route-uri = ... ? They seem to do the same things.

We decided to clear the cache every 5min (the description was not reflecting the code). It means of course that the user might have to wait few minutes before seeing the updated version. We think though, that this frequency is good enough as regular use of our device does not implies a lot of upload/modification/delete of the media-center content. This will be anyway checked on the field in order to have feedbacks.
route and route-uri does not behave exactly the same way. route-uri match exactly the given regex whereas route does not take care of the regex and cache the whole URL. For instance, I would like to avoid caching the tags, type and so on. Main pages are enough for now.

Well, the route option uses regex and the regexes of route-uri(as route) explicitly use a ^ for start of line and $ for end of line.
I doubt (in the way you use them) they really behave differently.
(Yet again, I may have wrong, but I will be happy to learn)


Bot Framework Emulator & Suggested Action

Based on your doc (below) I implemented a suggested actions.
In the Bot Emulator Framework the text is coming but not the action buttons.
https://docs.microsoft.com/en-us/azure/bot-service/rest-api/bot-framework-rest-connector-add-suggested-actions?view=azure-bot-service-3.0

Hello @thibaultcrouzet - I have just tested the activity in that document, replacing the from, conversation, recipient, and replyToId fields with applicable values, and also using the correct service URL (which should be an ngrok URL in the case of Emulator) and my own bearer token, and it works just fine. The suggested actions appear as expected. I would like to help, but I need some more information from you:

Please provide the HTTP request you are sending (though obscure the bearer token)
Please provide screenshots of what you're seeing in Emulator
Please tell me what version of Emulator you're using
Please let me know what other requests you're sending

Right now, my best guess is that you're sending an activity to the bot after the activity with the suggested actions, which would make the suggested actions disappear.

Hi @thibaultcrouzet - Are you still working on this?

@v-kydela yes

I use ngrok

Version 4.5.2 (<IP_ADDRESS>281) Mac
I used the example that you provided in JSON here : https://docs.microsoft.com/en-us/azure/bot-service/rest-api/bot-framework-rest-connector-add-rich-cards?view=azure-bot-service-3.0#adaptive-card

you can see that the buttons are missing in the emulator

@thibaultcrouzet - From the screenshot, it looks like the user has sent one message to the bot and then the bot sent a typing activity and four message activities. Are your suggested actions in the message that says "hello User" or one of the Adaptive Card messages?

@v-kydela like I explained in the beginning in the Bot Emulator Framework the text is coming but not the action buttons in the Adaptive Card.
I don't see any of the action buttons in the Framework Emulator from the JSON below :
            "actions": [
                {
                    "type": "Action.Http",
                    "method": "POST",
                    "url": "http://foo.com",
                    "title": "Snooze"
                },
                {
                    "type": "Action.Http",
                    "method": "POST",
                    "url": "http://foo.com",
                    "title": "I'll be late"
                },
                {
                    "type": "Action.Http",
                    "method": "POST",
                    "url": "http://foo.com",
                    "title": "Dismiss"
                }
            ]

This below should be the result based on your documentation :

But this is what I get :

@thibaultcrouzet - Thank you for the clarification. I understand the problem now. Allow me to explain.
Suggested actions are a term used in the Bot Framework to refer to some kind of temporary button that disappears as soon as a message is sent. They're usually used to automatically send a text message without the user having to type it. Because the Bot Framework connects to many different channels with their own messaging platforms and possibly their own suggested action implementations, the Bot Framework tries to unify these potentially different channel-specific features under the umbrella of suggested actions. For example, in Facebook Messenger the feature is called quick replies and they might look like this:

As you can see, suggested actions exist outside of cards. What you want to do is add actions to an Adaptive Card, but these Adaptive Card actions are different from suggested actions. You have the right idea, but unfortunately there is a problem with the Adaptive Cards document you linked. I have logged that as an issue here: https://github.com/MicrosoftDocs/bot-docs/issues/1346
The problem is that Adaptive Cards don't support Action.Http. There's something called Outlook Actionable Messages that's based on Adaptive Cards, and that's where Action.Http comes from. You can see the list of supported actions in the Adaptive Cards schema explorer.
If you really want the client application to trigger an HTTP request then you could try Action.OpenUrl and link to some page that makes the request. But what I normally recommend is Action.Submit which sends a message to your bot, because your bot can respond to the message by making whatever HTTP requests you want. For more information, see my latest blog post.
Does this resolve your issue?

@v-kydela yes it does. thanks!

@v-kydela yes it does. thanks!


Add FSDP support to Qwen2-72B model

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.

🔖 Feature description
FSDP can efficiently shard large models into multiple GPUs. This was very successful with Llama-3-70B. However, Qwen2 doesn't have support for FSDP and even QLoRA fails to fit the model in 4 A100 with 80G each.
I have failed to fine-tune Qwen2-72B with QLoRA+DPO on 4 A100 devices.
✔ Solution
Adding FSDP support to Qwen2 should help
❓ Alternatives
I have tried DeepSpeed all zero3 strategies (offload params, all, etc.), none of them actually worked in terms of allowing a simple QLoRA+DPO on 4 A100. In theory, each model should be around take around 36G-40G in 4bit. But it gets OOM. Might be failing to load the model correctly in bf16 and leaves some on 32bit.
📝 Additional Context
No response
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this feature has not been requested yet.
[X] I have provided enough information for the maintainers to understand and evaluate this request.

@maziyarpanahi what yaml are you using?

@winglian I use this one, but changing it to DPO (very simple Intel Orca): https://github.com/OpenAccess-AI-Collective/axolotl/blob/main/examples/qwen/qlora.yml
bf16: auto
fp16: false
tf32: false

I tried to add DeepSpeed (all zero3 variations), but they all are failing with OOM. I see these right when the training starts, so I was wondering if there are some stuff left on float32 and couldn't cast back.
[2024-06-07 12:43:28,508] [INFO] [axolotl.callbacks.on_train_begin:771] [PID:5276] [RANK:0] The Axolotl config has been saved to the WandB run under files.
  0%|
                                                                         | 0/6430 [00:00<?, ?it/s]The input hidden states seems to be silently casted in float32, this might be related to the fac
t you have upcasted embedding or layer norm layers in float32. We will cast back the input in torch.bfloat16.
The input hidden states seems to be silently casted in float32, this might be related to the fact you have upcasted embedding or layer norm layers in float32. We will cast back the input in torc
h.bfloat16.
The input hidden states seems to be silently casted in float32, this might be related to the fact you have upcasted embedding or layer norm layers in float32. We will cast back the input in torc
h.bfloat16.
The input hidden states seems to be silently casted in float32, this might be related to the fact you have upcasted embedding or layer norm layers in float32. We will cast back the input in torc
h.bfloat16.

for Qwen/Qwen2-72B , the qwen2-7b fsdp example is not working , out of memory all the time ,even with 7*80GB A100 . for Qwen/Qwen2-7B , the example is working .

@deter3 I am using the exact same FSDP on 4 A100 GPUs with QLoRA, and it fits. (batch size is 1 though)
I cannot fit FFT of 7B in the same GPUs via FSDP! Are you using QLoRA or FFT?

@deter3 I am using the exact same FSDP on 4 A100 GPUs with QLoRA, and it fits. (batch size is 1 though)
I cannot fit FFT of 7B in the same GPUs via FSDP! Are you using QLoRA or FFT?

I am using 4 A100 GPUs with QLoRA & FSDP ,or 7 A100 GPUs with QLoRA & FSDP for 72b model , none of them fits . my max length is 9100 . I used my own data (English + Chinese mixed data )  for fine tune , I will double check other dataset , if it is okay or not .

i figured out the problem : sequence_len . if sequence_len is 2100 , example qwen2-72b qlora fsdp example is working , out of memory when sequence_len changed to 9100 . do you have any good suggestion , since even i am using 7*a100 80gb is still out of memory .


python312Packages.requests-pkcs12: 1.24 -> 1.25

Diff: https://github.com/m-click/requests_pkcs12/compare/1.24...1.25
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

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

12 packages built:

audiness
audiness.dist
faraday-agent-dispatcher
faraday-agent-dispatcher.dist
python311Packages.pytenable
python311Packages.pytenable.dist
python311Packages.requests-pkcs12
python311Packages.requests-pkcs12.dist
python312Packages.pytenable
python312Packages.pytenable.dist
python312Packages.requests-pkcs12
python312Packages.requests-pkcs12.dist


prevent flushing twice

If flush_mode = interval and timekey = 5m, this would be flushed twice.
Though it is configuration mistake, I think it's also unexpected thing.
Any suggestions welcome and let me know what I do not know:)

Did you test this patch?
I think your code has syntax error.

BTW, could you paste your reproducible configuration here?

@repeatedly
I corrected the wrong syntax. Thanks :)
As I mentioned, my configure file was like below.
<buffer tag, time>
		@type               file
		flush_mode          interval
		timekey             5m
</buffer>

The default value of the flush_interval is 60 seconds. and there's no setting flush_interval outside the buffer tag.
Then I got the result applied both configure, timekey and flush_interval, means flush_mode=lazy and interval.
Every one minutes later, it's flushed and also timekey works.
The full configuration is like this.
<match sometag>
	@type						webhdfs
	namenode					hdfs_url
	append						no
	path						/some_path/%Y%m%d/%H/${hostname}.${chunk_id}.log
	<buffer tag, time>
		@type               file
		flush_mode          interval
		timekey             5m
		path                /var/log/td-agent/buffer
		chunk_limit_size    128mb
		flush_at_shutdown   true
		retry_forever       true
	</buffer>
</match>

@repeatedly
Hi, repeatedly
I think branching by flush_mode is more clear and it can prevent issue I mentioned.
If I misunderstand about flush_mode and chunk_time_key, please let me know.


Consider using rattler-build as our packaging system

I had this idea while writing #2
At some point in the near future, we should consider "eating our own dog food" and use rattler-build to package this project.
For a period of time, we might want to consider having two build-recipe workflows on PRs/merges; 1 for conda-build and one for rattler-build

As of d3b046f09553b8f098496f0f185d35ed8af9b01d there is now a build-recipe-conda and a build-recipe-rattler action. The latter takes the recipe that works with conda-build, converts it, and passes that output to rattler-build


Fix AppNotification activation when no foreground handlers defined

There was a bug with using REG_SINGLEUSE on app launch for AppNotification activation. The issue was that the com object was not activated the first time and only on the second activation would another process be created.
Now if the app has no foreground handlers, was launched by any ExtendedActivationKind that is not AppNotification, and receives a AppNotification activation, we CoCreateInstance the app and propagate the arguments through Activate.

Can we add a prompt in the demo app to let the user choose to register or not foreground handlers?

/azp run

/azp run

Can we add a prompt in the demo app to let the user choose to register or not foreground handlers?

I think this is a good idea. Let's have a PR that contains demo app improvements


An error occurred in the main loop: 'balanceKeys'

[2024-08-12 03:02:10] An error occurred in the main loop: 'balanceKeys'

How to fix?

game  loop  ,  i not get game code

Use the last version!


Dear All

Hi,
How to give horizontal bar for wider dropdown option,
tried overflow-x:auto, but it doesn't works.

You do realize what is the difference between issue tracker and forum, right :)?

Hi, your issue reads like a support question. This github repo is only for bug reports and feature requests. Please try one of the listed Support Channels.

I have tried with blur admin, but it didn't work, i understand the support issue vs Portal(application) bugs. here i raised this point  because i tried same for blur admin.


Rebase over main failing due to OOM persist dir missing

After rebasing on today's main, there is a compilation error because a new method, OOMFilePath was added to the runtime interface.
libpod/oci_shimv2_common.go:147:9: cannot use runtime (variable of type *ShimV2OCIRuntime) as OCIRuntime value in return statement: *ShimV2OCIRuntime does not implement OCIRuntime (missing method OOMFilePath)

This was added as part of commit 667311c7d509794146114e05e7699bf3109bcee3, which is 11 months old but was apparently only merged recently

The calling code does not seem too ready to handle an error silently. However, it seems to be happy with a file that does not exist, so that's probably the safe thing to return.
func (c *Container) handleExitFile(exitFile string, fi os.FileInfo) error {
    ...
	oomFilePath, err := c.oomFilePath()
	if err != nil {
		return err
	}
	if _, err = os.Stat(oomFilePath); err == nil {
		c.state.OOMKilled = true
	}

	c.state.Exited = true
    ...
}


Add charge mode for cardputer

Add something that turns off the screen so the cardputer can charge the second battery. It is to prevent burn in

I'll make a charging mode, reducing brightness and lowering CPU frequency..


added NNX vs JAX transformations guide

added NNX vs JAX transformations guide

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 61.19%.  Consider uploading reports for the commit 751215a to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3819   +/-   ##
=======================================
  Coverage   61.19%   61.19%
=======================================
  Files         105      105
  Lines       13148    13148
=======================================
  Hits         8046     8046
  Misses       5102     5102

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Auto FOV selection

Depending on the current resolution, FOV can be calculated.
The task is not trivial, for three OS at the same time. But it will simplify patching for people who do not have to understand, and who have monitors other than 16: 9

Hi! I love all the tweaks - thank you.
But, I prefer the default FOV in the vanilla client on my 16:9 monitor. How do I disable the FOV patch, but retain all the other tweaks?
Thanks in advance!

@sambonz
vanilla-tweaks.exe --no-fov WoW.exe
see readme and --help command

SuperWoW (SuperAPI) solves this problem, almost


Filtered replication

Hi,
Does it work with server-side filtering using a design doc filter? I have tried implementing it but with no success.

Hi,
it should work, tested it actually one. Can you provide me an Sample, so that I can check?

The problem I'm facing is actually a couchdb bug: https://github.com/pouchdb/pouchdb/issues/1641
If you still have the design document you used when you tested it can you post it here?
Thanks!

Never mind, this issue appear only to happen on couchdb on my pc, its working using a cloud service.

currently i just test against express-pouchdb in my node service, but there the Filter looks like
const userFilter = {
  _id: '_design/userEventFilter',
  filters: {
    myfilter: function (doc, req) {
      return doc.userId && doc.userId === req.query.userId
    }.toString()
  }
}

but automatic testing is still an issue...
https://github.com/stockulus/pouchdb-react-native/issues/13


terms of service link in erc-20 staking modal routing to dashboard/terms of service stays in url

Overview
clicking on the terms of service in ERC-20 staking modal opens up a new window that goes to the dashboard instead of a terms of service page. also after you click the terms of service link the legal/terms-of-service stays in your url. you have to copy past the url back in to get rid of it.
References and additional details

go to https://3202a9af.web-29e.pages.dev/ or app.shapeshift.com and connect with any wallet
click on the defi tab
click one of your ERC-20 vaults
click on the deposit tab on the top of the widget
go the the bottom of the widget and click on the terms of service link.
notice that it opens a new tab and directs you back to the dashboard.
also notice that /legal/terms-of-service# stays in the url as you natigate the app

happening on all browsers

Acceptance Criteria
the terms of service link should take you to a new tab with the terms of service. also the /legal/terms-of-service# should not stay in your url when you navigate off of the page.
Need By Date
No response
Screenshots/Mockups

Estimated effort
No response

Looking at it right now!


Fix for pointer events pen input in Edge/IE11

Remapping of "pen" to "touch" now also implemented (fairly naively, admittedly) for Edge/IE11 (where event.pointerType is NOT a number, so previously wasn't being remapped at all and resulted in a value of "pen" for data-what-input). Additionally, this also now registers pointerenter, as otherwise pen interactions keep flipping back to being recognised as mouse when the pen is moving over the page (due to the compatibility mouse events being fired for pointerenter)

made a quick video showing the problem, and the effect after applying this patch https://www.youtube.com/watch?v=5gGtE-pSvCY

Thanks! I'm on vacation over the long weekend but am looking forward to checking this out next week!

force-pushed a tiny amendment to the first commit, which removes the need for the touch handler to be registered if pointer events handlers were already registered

Thanks @patrickhlauke, this is a great enhancement!

My pleasure :+1:


Where are xref namespace from?

The xref use CURIEs in namespaces.
I think these are identifiers.org namespaces as explained in external-references but we need to update xref to use hyperlinks to that.
(also then the so example should be a valid http://identifiers.org/so/ identifier starting with SO: )

@stain ,
I am not sure I understand exactly what you are saying here. Do you mean that
            {
                "namespace": "so",
                "name": "Sequence Ontology",
                "ids": ["0000048"],
                "access_time": "2018-13-02T10:15-05:00"
            },

should become something like
            {
                "namespace": "so",
                "name": "Sequence Ontology",
                "ids": ["http://identifiers.org/so/0000048"],
                "access_time": "2018-13-02T10:15-05:00"
            },

Hadley, that still isn't the right, the namespace is so and the identifier is SO:0000048
http://identifiers.org/so/0000048 → '0000048' does not match the regular expression '^SO:\d{7}$' for so.
http://identifiers.org/so/SO:0000048 correctly pulls up information at https://www.ebi.ac.uk/ols/ontologies/so/terms?obo_id=SO:0000048 where we learn the term is obsolete, so perhaps a better example should be chosen?

substitute SO:1000002 for SO:0000048

[ ] user_guide.md:487:
[ ] user_guide.md:488:
[ ] user_guide.md:551:
[ ] description-domain.md:81:
[ ] HCV1a.json:59:
[ ] HCV1a.json:60:
[ ] HCV1a.json:120:
[ ] usability-domain.md:14:
[ ] usability-domain.md:16:


More fine grained commands

At the moment, when calling "nodeshift" a rather large promise chain is executed.  I would like to have a more fine grained approach.
Off the top of my head, i think there could be:

deploy - A command that does everything(combo of both deploy and build)
build - buildconfig/imagestream/actual build stuff
applyEntities/applyResouces? - the thing that does those service/routes/deploymentconfig stuff
archive - just does the archiving stuff
Delete - this removes everything, or one thing

Thinking about this again since i've been doing #80 . I think we should start off with 3 or 4 "goals"
Build - does just the build steps, which include tar'ing the source code, creating the buildConfig and pushing to the ImageStream
resources - creates the application resources(these are the service/deployment config, routes etc..), but doesn't push up to the cluster
resource-apply(maybe) - does what resource does, but also pushes to the cluster
deploy - does the whole thing(build + resource-apply)

#111 is laying the groundwork for this

the default command(deploy), resource and apply-resource are in this commit e0fce605858d901dcb4bcc7c7d0a2c22af96dc1e

undeploy here 5f04b36

All the commands from this comment: https://github.com/bucharest-gold/nodeshift/issues/4#issuecomment-339799785 are now implemented.
closing


PSSA doesn't respect the config I give it.

This is how I call PSSA :
$scriptAnalyzerParams = @{
    Path = $ModulePath
    Severity = @('Error', 'Warning')
    ExcludeRule=@('PSUseBOMForUnicodeEncodedFile','TypeNotFound')
    Recurse = $true
    Verbose = $false
}

$saResults = Invoke-ScriptAnalyzer @scriptAnalyzerParams

This is the file failing.
https://gist.github.com/webtroter/d44a5725849ad4cd30bbadd1efb69bd2

This is a duplicate of #1041 because TypeNotFound is not a rule per se but emitted to still warn the user that the AST could only be partially analysed. I discussed at the time being if it might be confusing to return TypeNotFound as a DiagnosticRecord, which might be confusing but it was decided to rather return an object than emit a warning (which cannot be seen in editors such as VSCode. cc @JamesWTruher
@tylerl0706  @rjmholt  Would it be possible for PSES/VSCode to display warnings from PSSA as well so that we can fix those issues by returning a PowerShell warning instead of a DiagnosticRecord object in PSSA?

Thank you @bergmeister.
How would your proceed to ignore that error? It's blocking my build process.
Thank you.

@bergmeister I'm not sure I follow you there on the PowerShell warning side of things.
There are probably a few points to address here:

TypeNotFound is a legitmate parse error - PowerShell itself will not run a script that results in the emission of this error. See https://github.com/PowerShell/PowerShell/issues/6722, https://github.com/PowerShell/PowerShell/issues/2074 and https://github.com/PowerShell/PowerShell/issues/3641. Converting the parse error to an information record may be ergonomic for users, but it masks a real issue in the script -- that the required types need to be imported ahead of time before that script/module is parsed. (This is a PowerShell issue I've been working to rectify in my spare time)
PSSA is a linter/diagnostic tool, and I think it should behave the way the parser does with respect to errors -- by making a clear distinction between diagnosing an error in the script its analysing and showing an actual error in itself. Writing to the error stream conventionally means there is a problem in execution, but PSSA currently writes errors diagnosed in the script under analysis as errors, rather than diagnostics. My opinion is that PSSA should write to the error stream only if there has been an error in PSSA. PSSA probably needs a new diagnostic category for parse errors and the ability to suppress any given parse error by FullyQualifiedErrorId. Writing diagnosed errors to the error stream makes it harder for PSES and other PSSA-embedding tools to handle PSSA's output, especially if some parse errors are treated differently.
The other problem with the way PSSA currently treats parse errors is that it strips out important information, such as where the error occurs (see https://github.com/PowerShell/PSScriptAnalyzer/issues/264#issuecomment-414093378). So in fact we have to get our parse error diagnostics in PSES elsewhere (by parsing the script ourselves). If a TypeNotFound error were emitted as a warning like this and information were similarly stripped out, we would not be able to display it.

@webtroter I think this whole discussion might be tangential to what you're trying to achieve. You should be able to filter your Invoke-ScriptAnalyzer to weed out TypeNotFound diagnostics. Something like this:
$saResults = Invoke-ScriptAnalyzer @scriptAnalysisParams | Where-Object { $_.RuleName -ne "TypeNotFound" }

Anyway, I understand that all the things I'm suggesting are breaking changes. But they are ones that I think will improve the ease of use and API consistency for PSSA for everyone, especially anyone trying to embed PSSA in a hosted scenario. I was trying to write a PSSA GitHub bot the other week and the inability to extract an extent from the parse errors it emits was frustrating to say the least.

Thank you @rjmholt .
I went ahead and changed it to ignore my specific Type, instead of the whole "TypeNotFound" rule.
I'm using
$saResults = Invoke-ScriptAnalyzer @scriptAnalyzerParams | Where-Object { $_.Extent.Text -ne "Microsoft.Online.Administration.User" }

I'm glad you could sort it out @webtroter
P.S.: a slightly better way would be to filter like this:
Invoke-ScriptAnalyzer @scriptAnalyzerParams | Where-Object { $_.RuleName -ne 'TypeNotFound' }


not working with https

https://github.com/dvcama/LodLive/blob/e90f5887cdec2bf347597c33653504cdf4e4e01f/js/lodlive.core.js#L134
Lodlive not work with https url, should be:
var base = uri.replace(/(^http[s]{0,1}:\/\/[^\/]+\/).+/, "$1");

done
https://github.com/dvcama/LodLive/blob/master/js/lodlive.core.js#L134
thank you for pointing it out


Post build index creation

It may be desirable for some index types to be build post segment file has been created. This PR allows for that.

Codecov Report

Merging #11711 (f2a3c6a) into master (54a1ca0) will decrease coverage by 48.62%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             master   #11711       +/-   ##
=============================================
- Coverage     63.05%   14.44%   -48.62%
+ Complexity     1118      201      -917
=============================================
  Files          2342     2342
  Lines        125787   125815       +28
  Branches      19335    19337        +2
=============================================
- Hits          79314    18171    -61143
- Misses        40813   106104    +65291
+ Partials       5660     1540     -4120

Flag
Coverage Δ

integration
<0.01% <0.00%> (-0.01%)
:arrow_down:

integration1
<0.01% <0.00%> (-0.01%)
:arrow_down:

integration2
0.00% <0.00%> (ø)

java-11
14.43% <0.00%> (-48.58%)
:arrow_down:

java-17
14.42% <0.00%> (-48.49%)
:arrow_down:

java-20
0.00% <0.00%> (-62.93%)
:arrow_down:

temurin
14.44% <0.00%> (-48.62%)
:arrow_down:

unittests
14.43% <0.00%> (-48.62%)
:arrow_down:

unittests1
?

unittests2
14.43% <0.00%> (+<0.01%)
:arrow_up:

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

Files
Coverage Δ

...ment/creator/impl/SegmentColumnarIndexCreator.java
0.00% <0.00%> (-86.27%)
:arrow_down:

.../segment/index/dictionary/DictionaryIndexType.java
0.00% <0.00%> (-83.84%)
:arrow_down:

...al/segment/index/nullvalue/NullValueIndexType.java
0.00% <0.00%> (-84.00%)
:arrow_down:

.../org/apache/pinot/segment/spi/index/IndexType.java
0.00% <0.00%> (-66.67%)
:arrow_down:

...t/creator/impl/SegmentIndexCreationDriverImpl.java
0.00% <0.00%> (-80.12%)
:arrow_down:

... and 1503 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I approved the PR by accident 😛

cc @gortiz


NoNothingToNothing

See https://stil4m.github.io/elm-analyse/#/messages/MapNothingToNothing
What the rule should do:
Forbid case expressions where Nothing is matched and then Nothing is returned.
What problems does it solve:
It makes the code more complicated than necessary. Use May.andThen or Maybe.map instead.
Example of things the rule would report:
module Greet exposing (greet)

greet : Maybe String -> Maybe String
greet maybeName =
    case maybeName of
        Nothing ->
            Nothing

        Just name ->
            Just ("Hello " ++ name)

Example of things the rule would not report:
module Greet exposing (greet)

greet : Maybe String -> Maybe String
greet maybeName =
    case maybeName of
        Nothing ->
           Just "Hello"

        Just name ->
            Just ("Hello " ++ name)

When (not) to enable this rule:
You don't need this rule. If you're writing code like this then there may be some wider code restructures to consider.
I am looking for:

Reasons you would / would not want to use this rule.

Hi! I don't have concrete code examples to contribute right now, but we sometimes opt out of the Maybe.map/andThen constructs because it seems to obscure the intent in some instances, especially when there are several nested Maybes or longer chains.
Maybe (ah) we're wrong about it, though!

Hi! I don't have concrete code examples to contribute right now, but we sometimes opt out of the Maybe.map/andThen constructs because it seems to obscure the intent in some instances, especially when there are several nested Maybes or longer chains.
Maybe (ah) we're wrong about it, though!

I don't have concrete code examples to contribute right now.

That's good! Anything that brings doubt on to whether a rule should always be applied is what I'm looking for. An elm-review rule cannot be selectively applied.

I don't have concrete code examples to contribute right now.

That's good! Anything that brings doubt on to whether a rule should always be applied is what I'm looking for. An elm-review rule cannot be selectively applied.

Reopening this so people can contribute reasons for/against this rule.

Reopening this so people can contribute reasons for/against this rule.