search_up_on_func potential bug but nor urgent

there is a potential bug in search_up_on_func  - needs some rework. if the parent of theorem has no childs and no space or newline before thereom the programm will throw an Error

probably fixed :D


fix unsaved_files_array TypeError in reparse method

In reparse method, the way unsaved_files_array is filled is not consistent with from_source and codeComplete methods and raises TypeError when passed a python3 unicode str.

Sorry but we don't use github, please report your patch on https://reviews.llvm.org/


Add GIF to README

Hey @psy2k, your lib is really cool, but your README can have some improvements. Since you have an UI Library, it would be good to add a gif to show to devs how it looks like. You can get some inspiration from this lib

@lfarah already had a .gif. It was on my dropbox public folder and it just stopped working with the recent dropbox changes. Now it is fixed https://github.com/psy2k/NMPopUpView/commit/c636bc9a5ae13a1f289ec7cddda414555e5305cf


cmod_hybrid needs to propagate errors from lower level cmods

Errors thrown in lower level cmods are ignored, and the error shown to the user is the next arbitrary error from cmod_hybrid.
Check the error status from each cmod called in cmod_hybrid and propagate notices, warnings, and errors to the user.

See ssc_module_exec_with_error in https://github.com/NREL/ssc/pull/1091/commits/a68f4138fac5e26dd008eac1235958d7cd178af8#diff-0bb15765ddde12ea78e017fc6256d566c64c615d9b31a135366ff72277e0c80d


ExasolConnectionManager could be created several times

We have several createRelation methods in DefaultSource and every variant of those methods create ExasolConnectionManager instance, which supposed to be a singleton (caching jdbc connections). But we call those methods from each other (example: https://github.com/exasol/spark-connector/blob/main/exasol-jdbc/src/main/scala/com/exasol/spark/DefaultSource.scala#L138), so, it is possible to get several ConnectionManager instances.

After closer look in the code logic, it turned out not a problem, as ConnectionManager just creates new connections without pooling them.


deliver lane screenshot upload failed

Fastlane version: 2.153.0
deliver lane screenshot upload fails with following error.
Error log:
DEBUG [2020-07-27 04:25:02.53]: Version '3.10.1' is the first version on App Store Connect
WARN [2020-07-27 04:25:02.53]: [33mWill begin uploading metadata for '3.10.1' on App Store Connect[0m
ERROR [2020-07-27 04:25:02.53]: [31mSkipping 'release_notes'... this is the first version of the app[0m
INFO [2020-07-27 04:25:02.53]: Uploading metadata to App Store Connect for version
INFO [2020-07-27 04:25:03.08]: Uploading metadata to App Store Connect for localized version 'en-US'
INFO [2020-07-27 04:25:03.60]: Uploading metadata to App Store Connect for localized info 'en-US'
INFO [2020-07-27 04:25:09.11]: Uploading app review information to App Store Connect
ERROR [2020-07-27 04:25:09.56]: [31mError fetching app store review detail - The specified resource does not exist - There is no resource of type 'appStoreReviewDetails' with id 'null'[0m
call_with_default_configuration Exception is... The provided entity includes an unknown attribute - 'demo_account_name' is not an attribute on the resource 'appStoreReviewDetails'
The provided entity includes an unknown attribute - 'contact_phone' is not an attribute on the resource 'appStoreReviewDetails'
The provided entity includes an unknown attribute - 'demo_account_password' is not an attribute on the resource 'appStoreReviewDetails'
The provided entity includes an unknown attribute - 'contact_last_name' is not an attribute on the resource 'appStoreReviewDetails'
The provided entity includes an unknown attribute - 'demo_account_required' is not an attribute on the resource 'appStoreReviewDetails'
The provided entity includes an unknown attribute - 'contact_first_name' is not an attribute on the resource 'appStoreReviewDetails'
The provided entity includes an unknown attribute - 'contact_email' is not an attribute on the resource 'appStoreReviewDetails'
Logger: Upload screenshots lane completed 2020-07-27 04:25:10.036
@joshdholtz

Will take a look at this morning!

@mganapathy Oh, can you update to 2.153.1? 😇

@joshdholtz - Thanks for your quick update! we will update to 2.153.1 and will let you know the status.

wanted to confirm that I had 2.153.0 and had this exact issue with Error fetching app store review detail - The specified resource does not exist - There is no resource of type 'appStoreReviewDetails' with id 'null'.
i upgraded to latest fastlane 2.153.1 and reran the deliver and the issue was resolved.

Closing this! ☝

@joshdholtz - after upgrading 2.153.1 version deliver lane screenshot issue resolved now. Thanks!


Missing named objects do not error out at runtime

Problem
Invalid or missing named objects / resources can lead to invalid configuration at runtime leading to issues like #208 where endpoints can be left in an insecure state.
Example
kind: IPPolicy
apiVersion: ingress.k8s.ngrok.com/v1alpha1
metadata:
  name: test-policy
spec:
  description: "Deny all policy"
  rules:
  - action: "deny"
    cidr: <IP_ADDRESS>/0
    description: "Deny all"
---
kind: NgrokModuleSet
apiVersion: ingress.k8s.ngrok.com/v1alpha1
metadata:
  name: ip-restrictions
modules:
  ipRestriction:
    policies:
    - "test-policy-typo"

Expected behavior

Check existence of all named resources / objects that are being used
Error when named resources / objects do not exist

To clarify, we're not simply erroring, but fatal erroring when the objects do not exist, is that correct? So a tunnel won't start for an unnamed resource/object.

To clarify, we're not simply erroring, but fatal erroring when the objects do not exist, is that correct?

Correct, as per this line from Alex:

On the second ing2 object though, it will error finding the module ip-restrictions-typo and discard that route.

Which then results in:

The problem here lies in that it first creates/updates the routes, and then it tries to lookup and add the ip policy information which errors.

Reading all of this, it looks like we actually aren't erroring when we encounter an unknown module set. We are logging it as an error, but we don't do anything with it as such.
Am I correctly understanding that we want the behavior to actually return the error here? Since the outer function calculateHTTPSEdges doesn't return an error, this should cause the parent Sync call to panic (aka unresolvable error).


Remove-AzADSpCredential   doesn't remove credentials

I'm using  Remove-AzADSpCredential  to remove password credentials but it doesn't do anything.
I don't receive any error  but when check credentials they are still there.
In debug mode I see it's calling correct API and the body doesn't contain the deleted one
HTTP Method:
PATCH
https://graph.windows.net/xxxxx/servicePrincipals/xxxxx/keyCredentials?api-version=1.6
Body:
{
  "value": [
{
      "startDate": "2019-04-12T00:00:00Z",
      "endDate": "2020-04-12T00:00:00Z",
      "keyId": "550fd04",
      "usage": "Sign",
      "type": "AsymmetricX509Cert",
      "customKeyIdentifier": "F77E9B42",
      "value": null
    }
]
}

Status Code:
NoContent

PsVersion
Name                           Value
----                           -----
PSVersion                      5.1.19041.1682
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.19041.1682
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Azure version   6.0.0

It seems it can be the API error.

@rezarms Please log this issue in the azure-powershell source repo since it appears to be a product issue. This repo is for only for documentation.


Generic Dynamo Based Hash Table Implementation

Clusternator now has a dynamo db backed generic key/value hash table system.
The code is well tested wrt unit tests.  Manual REPL tests have been successful but there is a problem doing a write/createTable at the same time.  We should probably just deprecate this feature since we'll be checking table existence on server init anyway

:+1: Paired on this, looked good.

This will hopefully serve as a template for the rest of the AWS refactorings.  I'm going to do some research into the function boxing approach.  I'd like to make sure it will scale properly to the rest of the AWS wrappers.
I do really like the parameterized approach, I found it easier to test than the massive closure approach

I concur.


Allow For memory chunks bigger than 4GB

As discussed this seems to be a "technical unnecessary restriction."
From my experiments it seems possible to fix it by tweaking some of the types
Discussed in https://github.com/eclipse-iceoryx/iceoryx/discussions/2149

Originally posted by kozakusek January  3, 2024
When trying to start RouDi with a 4GB segment.mempool I encountered an unpleasant error:
 Condition: (t >= Minimum) && (t <= Maximum) in iox::range<T, Minimum, Maximum>::range(T) [with T = unsigned int; T Minimum = 1; T Maximum =<PHONE_NUMBER>] is violated. (/workdir/iceoryx/iceoryx_hoofs/primitives/include/iox/algorithm.hpp:208)

Then after reading into the source code I noticed that in many cases (e.g. iox_pub_loan_chunk) the variable that refers to size is of type uint32_t.
Is there a reason for this limitation? Are you planning to add support for bigger memory chunks by adding option to hold size in 64bit integers?
Thanks

@kozakusek looking forward to your PR :)

I've created a draft PR https://github.com/eclipse-iceoryx/iceoryx/pull/2158.
I just need to delete the temporary modifications made in iceperf example and I think it will be ready for review.
In the meantime if you have a spare second I would apprieciate if you could check if I didn't messed up anything with the issue/commit/pr format :smiley:


sasjs run target parameter should be optional

The -t on sasjs run should be optional - it would be nice if it would just pick the first target

already resolved


Dynamic tables with buttons + dynamic HTML from C# not working?

The table is the same HTML like in the template, the only difference is that the HTML is being created in the C# and inserted via PlaceHolder control. It is in a update panel.. Before I update the panel, I call the script where the datatable buttons should create and initialize, but they don't appear. :(
Any ideas?

Any screen shots?

I did it. I had to include again the scripts and call again the functions. Now it is working.


Incompatibility with Aframe 1.3.0 (most recent release)

Do you want to request a feature or report a bug?
Bug.

What is the current behavior?
When importing AR.js in combination with the latest version of Aframe - 1.3.0 (released a few days ago), the module throws two errors in the console and fails to load any items onto the AR scene.
The errors are:
Uncaught TypeError: arContext.addEventListener is not a function
    at new ARjs.Session (aframe-ar.js:4041:12)
    at HTMLElement.<anonymous> (aframe-ar.js:7365:48)
    at HTMLElement.<anonymous> (aframe.min.js:45177:2943)
    at HTMLElement.<anonymous> (aframe.min.js:45187:12532)
    at HTMLElement.<anonymous> (aframe.min.js:45177:2943)
    at aframe.min.js:45177:1550

and
Uncaught TypeError: _this.dispatchEvent is not a function
    at done (aframe-ar.js:2527:15)
    at ARCameraParam.onload (aframe-ar.js:2679:9)
    at aframe-ar.js:1377:10
    at writeCallback (aframe-ar.js:1576:18)
    at writeByteArrayToFS (aframe-ar.js:1602:3)
    at XMLHttpRequest.oReq.onload (aframe-ar.js:1619:4)

If the current behavior is a bug, please provide the steps to reproduce.
I've made a codepen that shows the issue: https://codepen.io/vasturiano/pen/NWwdwxG
Essentially it's just importing<EMAIL_ADDRESS>AR.js and adding an a-scene tag to the body.
<head>
  <script src="https://aframe.io/releases/1.3.0/aframe.min.js"></script>
  <script src="https://raw.githack.com/AR-js-org/AR.js/3.3.3/aframe/build/aframe-ar.js"></script>
</head>

<body>
  <a-scene></a-scene>
</body>

Please mention other relevant information such as the browser version, Operating System and Device Name
What is the expected behavior?
Is there something that can be done externally to have AR.js work with Aframe 1.3.0?
If this is a feature request, what is motivation or use case for changing the behavior?

Hi @vasturiano have you tested the master or dev branch?

@kalwalt the example is importing v3.3.3, the latest version of AR.js. I believe that's the same as master.

@vasturiano can you try with the dev branch? I applied some changes related to new Three.js that should probably apply to aframe.

@kalwalt I've just tested with the dev branch and it works just fine, so that's great!
Is there plans sometime soon to merge it to master and release a new version with these fixes?
Also just a side question, any specific reason why the recent versions haven't been published to the npm registry? Looks like the latest published version there was 2.2.2, about 2y ago.
https://www.npmjs.com/package/ar.js

@vasturiano that is not our official npm registry, you should go instead to<EMAIL_ADDRESS>Consider that is an experimental package until will be merged in the master branch, in fact at the moment the aframe version has some issues.

This issue was solved with release https://github.com/AR-js-org/AR.js/releases/tag/3.4.5 see aframe version https://github.com/AR-js-org/AR.js/blob/35ea5555cd8d5b6c6aca0180cd28728e98a17203/package.json#L63


Cannot run through example dataset in 3_Pixie_Cluster_Cells.ipynb

Describe the bug
When running 3_Pixie_Cluster_Cells.ipynb directly (WITHOUT running 2_Pixie_Cluster_Pixels.ipynb) before it, I get this error:

I don't think the example dataset in Hugging Face was updated after some code changes.
Expected behavior
Run through notebook 3 completely using example dataset with no errors.
To Reproduce
Delete any existing example_dataset folders and run notebook 3 only (don't run notebook 2).

Ah sorry my bad, it's because I had some old versions of the example dataset in the cache, so it was using the cached version instead of the new version.


New IIO C-API to read configuration attributes

Task Description
The existing IIO C API does not allow the users to read out the existing configuration. The API only support reading out the sensor raw data or processed data.
Need new API be introduced to allow users to read and modify the attributes of IIO devices.

May I know when is the target date for v2?

no dates set in stone, but we're looking into 2-3 months maximum.

@laykuanloon @fulong82  could you share more information about which configuration items are needed for the reading/modifying?

Hi @yongli3,
Sorry for the late response. Here are the list that we would like to have a read API that can be used in sensor application:

sampling frequency
hysteresis
scale
offset

@laykuanloon do you have any interfaces want to add on?
Thanks & Regards,
Wilson

@fulong82 which sensor has the "hysteresis" ?

Hi @yongli3,
All of the IIO-HID sensors would have "hysteresis" attributes:
http://lxr.free-electrons.com/source/drivers/iio/common/hid-sensors/hid-sensor-attributes.c#L224
For example:

HID accelerometer: http://lxr.free-electrons.com/source/drivers/iio/accel/hid-sensor-accel-3d.c#L150
HID gyro: http://lxr.free-electrons.com/source/drivers/iio/gyro/hid-sensor-gyro-3d.c#L150
HID magneto: http://lxr.free-electrons.com/source/drivers/iio/magnetometer/hid-sensor-magn-3d.c#L197
HID ambient light: http://lxr.free-electrons.com/source/drivers/iio/light/hid-sensor-als.c#L127
HID pressure: http://lxr.free-electrons.com/source/drivers/iio/pressure/hid-sensor-press.c#L124
HID inclinometer: http://lxr.free-electrons.com/source/drivers/iio/orientation/hid-sensor-incl-3d.c#L150

Regards,
Wilson

Just like the sol_iio_mount_calibration API, we can add a new one as below:
Int sol_iio_read_device_config(struct sol_iio_device *device, struct iio_device_config *config);
The API will query the device configuration, and save them into config
struct iio_device_config {
double hysteresis;
double offset;
double sampling_frequency;
double scale;
}
@fulong82 @laykuanloon @edersondisouza @ceolin  any comment/suggestion ?

I suppose we should have 2 different APIs to get parameter:
eg:
In fbp level:  The principle is that we should let user get config parameter easily and clearly. So write like this to get config:   tick OUT -> OFFSET gyro CONFIG-> IN _(console:prefix="offset: ")
In nodes.c level:  we should check two key parameter from fbp include what config parameter(offset, scale, ...) and what kind of sensor type (different type has different config data type)
In sol-iio API level:

int sol_iio_read_device_config(struct sol_iio_device *device,
const char * config_name,
double *value);
int sol_iio_read_device_config(struct sol_iio_device *device,
const char * config_name,
sol_direction_vector *value);

For considering the return of error code , the improvement for API design is :

int sol_iio_read_device__double_config(struct sol_iio_device *device, const char * config_name, double *value);
int sol_iio_read_device_vector_config(struct sol_iio_device *device, const char * config_name, sol_direction_vector *value);
@fulong82 @laykuanloon @edersondisouza @ceolin @yongli3 any more comment/suggestion ?

@guchaojie
Do you consider to add API to read a list of supported value?
For e.g:
sampling_frequency_available
0.781000 1.563000 3.125000 6.250000 12.500000 25 50 100 200<PHONE_NUMBER>
in_accel_scale_available
0.009582 0.019163 0.038326

For get sensor config available config , I suggest two APIs:
To get the count of available value for user to malloc double array to store

int sol_iio_read_device_available_config_count(struct sol_iio_device *device, const char * config_name, uint32_t *count)
To get the available config value into user malloc memory
int sol_iio_read_device_available_config(struct sol_iio_device *device, const char * config_name, uint32_t *count, double *values)

@fulong82 @laykuanloon @edersondisouza @ceolin @yongli3 any comment/suggestion ?

@guchaojie
Can just combine into 1 API ? malloc in this function also
2. int sol_iio_read_device_available_config(struct sol_iio_device *device, const char * config_name, uint32_t *count, double *values)

@laykuanloon   If do so,  when to release the memory in API internal function?
2 APIs can let user  malloc and free the memory

@guchaojie
I think you can use pointer to pointer if want to combine the API. I am fine with what your propose.

I think memory allocation/free is not the key issue, though it's better to be released by assigner itself.
Actually the problem is, usually user does not know how may values can be supported for that configuration, so it's difficult to predict the size of the list. A query-response mechanism could be helpful to get the actual size here.

For Summary,  we finally define 4 APIs for two categories:

To read specific current configuration attribute:

int sol_iio_read_device__double_config(struct sol_iio_device *device, const char * config_name, double *value);
int sol_iio_read_device_vector_config(struct sol_iio_device *device, const char * config_name, sol_direction_vector *value);

To read specific configuration available attributes

int sol_iio_read_device_available_config_count(struct sol_iio_device *device, const char * config_name, uint32_t *count)
To get the available config value into user malloc memory
int sol_iio_read_device_available_config(struct sol_iio_device *device, const char * config_name, uint32_t *count, double *values)

@fulong82 @laykuanloon @edersondisouza @ceolin @yongli3 @peijiajames
If no more comments, we  are going on new C-API work.

@guchaojie
I am ok. Why sol_iio_read_device__double_config have 2 underscore?

Hi everyone,
I'd say that int sol_iio_read_device_double_config(struct sol_iio_device *device, const char * config_name, double *value); and int sol_iio_read_device_vector_config(struct sol_iio_device *device, const char * config_name, sol_direction_vector *value); are fine. They blend well with Soletta current APIs.
However, I'm a bit concerned about the one to get a list of available configurations. Soletta has sol_vector - couldn't it come to help?
What about something like struct sol_vector *sol_iio_list_available_config(struct sol_iio_device *device, const char *config_name)?
It would return NULL on error. It would be caller responsibility to clear the sol_vector, and sol_vector use is quite common throughout Soletta.

Hi everyone,
According to the deep implementation,  Some changes have to be modified,  Let me classify two category to make statement clear.

For reading current configuration attributes,  we found two issues:

parameter "config_name" generally would be prefix name such as "in_anglvel_x" or "in_anglvel_y"...
So for this condition, we input only one config_name which cannnot get sol_direction_vector data   type. It would be better to let user format double data from C-API into sol_direction_vector for output if necessary such as in fbp

In existing Soletta sol-iio design, we found configuration has been classified to two structure:  sol_iio_config and sol_iio_channel_config.
we can design this sytle API:
int sol_iio_read_device_iio_config(struct sol_iio_device *device, const char * config_name, sol_iio_config *value);
int sol_iio_read_device_iio_channel_config(struct sol_iio_device *device, const char * config_name, sol_iio_channel_config *value);
However I think this sytle is not good because two reasons:
1st is that it will make user confused and get all the config attribute for one time which let user has no choice.
2nd is that if these two structure happen changed, this style will impact exist implementation by using old C-API

So  I think new C-API should be like this, 1st version would be include 3 APIs , and would be easy to add more configuration reading API in future.
int sol_iio_device_get_scale(struct sol_iio_device *device, const char * config_name, double *scale);
int sol_iio_device_get_offset(struct sol_iio_device *device, const char * config_name, double *offset);
int sol_iio_device_get_sampling_frequency(struct sol_iio_device *device, const char * config_name, double *sampling_frequency);

For reading a list of available configuration attributes,
@edersondisouza  your proposal is good idea,  more details would be better to discuss in another new issue thread, this implementation would not be include 1st version of PR which is totally different from getting current configuration attribute in fact.

@fulong82 @laykuanloon @edersondisouza @ceolin @yongli3 @peijiajames
Sorry for updating the C-API design because of actual issue found in implementation, Thanks!

It was merged. Closing it.


Add VersionScheme constants?

I would like to have :
object VersionScheme {
   val Semver-Spec = "semver-spec"
...
}

in sbt library.

It was there:)


[GOBBLIN-1463] Remove pentaho dependency

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-1463

Description

[x] Here are some details about my PR, including screenshots (if applicable):

Pentaho's library for aggdesigner is used in tests. https://mvnrepository.com/artifact/org.pentaho/pentaho-aggdesigner-algorithm/5.1.5-jhyde However, the nexus repository frequently goes down and is unreliable.
Proposal to move to a newer version of aggdesigner hosted on maven central, similar to calcite https://issues.apache.org/jira/browse/CALCITE-1474.
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #3303 (e2c8627) into master (bf37c76) will decrease coverage by 3.50%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3303      +/-   ##
============================================
- Coverage     46.50%   42.99%   -3.51%
+ Complexity    10041     9313     -728
============================================
  Files          2041     2041
  Lines         79349    79349
  Branches       8845     8845
============================================
- Hits          36898    34117    -2781
- Misses        39020    41995    +2975
+ Partials       3431     3237     -194

Impacted Files
Coverage Δ

.../org/apache/gobblin/util/filters/HiddenFilter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...g/apache/gobblin/cluster/HelixMessageSubTypes.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...gobblin/runtime/mapreduce/GobblinOutputFormat.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...obblin/compaction/source/CompactionFailedTask.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/cluster/event/ClusterManagerShutdownRequest.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...in/compaction/action/CompactionCompleteAction.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/compaction/mapreduce/orc/OrcKeyDedupReducer.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...action/audit/KafkaAuditCountHttpClientFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...askStateCollectorServiceHiveRegHandlerFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ion/mapreduce/orc/OrcKeyCompactorOutputFormat.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 142 more

Continue to review full report at Codecov.

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

@aplex Ahh unfortunately it looks like hive-exec version also has a dependency on pentaho...need to think of how to prevent that as well.


-[__NSCFString getBytes:length:]: unrecognized selector sent to instance 0x16eed1b0

got 2 crash:
-[__NSCFString getBytes:length:]: unrecognized selector sent to instance 0x16eed1b0
-[UINibStorage getBytes:length:]: unrecognized selector sent to instance 0x17df5170
why an NSString/UINibStorage is being passed instead of NSData?
I'm sorry I don't know how to reproduce.

I've noticed a substantial increase in unrecognized selector exceptions with an update to the latest sdk. It may be that this project has relied on apis that are changing underneath it.

@huangguiyang are you still having these crashes

closing this. please reopen if you have troubles again

I got similar crash with v 3.7.2.
NSInvalidArgumentException in NSData(ImageContentType)
-[__NSCFString getBytes:length:]: unrecognized selector sent to instance 0x156c9df0
Stacktrace
Pods/SDWebImage/SDWebImage/NSData+ImageContentType.m:13 - +[NSData(ImageContentType) sd_contentTypeForImageData:]
Pods/SDWebImage/SDWebImage/UIImage+MultiFormat.m:22 - +[UIImage(MultiFormat) sd_imageWithData:]
Pods/SDWebImage/SDWebImage/SDImageCache.m:270 - -[SDImageCache diskImageForKey:]
Pods/SDWebImage/SDWebImage/SDImageCache.m:310 - _42-[SDImageCache queryDiskCacheForKey:done:]_block_invoke


remove cleanup login when cluster is deleted

run as non root
fix unit test
add service type when calculate hash
remove cleanup login when cluster is deleted

update the PR comments.


DNM: incomplete v13 with 46

Just in case we need it
...but looks like we won't!

I'm going to close this because I think that we can genuinely work with 47 for the next release


extend OAuth to other providers

Grant proxies 200+ Oauth providers. I should not require lots of efforts to plug it into Hasura-auth.

shipped in 0.16


Support mac Touch bar

OS:
macOS
Issue Descriptions:
2016 MBPs's touch bar has a limit of only 4 buttons that can be shown by default
It would be great if we could show media related buttons on the touchbar when the app is in focus
Electron docs on touch bar:
https://github.com/electron/electron/blob/master/docs/api/touch-bar.md

I'd just like to take a moment to point out that @MarshallOfSound, the creator of this app, also helped write the touch bar integration with electron, including the documentation. Also a duplicate issue #2411 and even further back #1965, though I don't see resolution in the original issue.

ah, apologies, I did a search for "touch bar" but should have also searched for "touchbar"
closing as duplicate


Swift 3.0.2 Ubuntu 14.04

Hi Norio,
Thank you for your help so far with https://github.com/jpsim/SourceKitten/issues/367
Again, sorry for all the questions, I really don't know what I'm doing here :/
I forked the repo and made the following changes based on your suggestion:
https://github.com/jeffreyhanna/docker-sourcekit-builder/commit/2f1cc3dd8f5c971b436ee322fd5fb9c7d29664d1
I tagged that commit "302_14_04" and then ran the following command
docker build -t sourcekit-builder:302_14_04 https://github.com/jeffreyhanna/docker-sourcekit-builder.git

That seems to have gone well. The next command in your instructions is:
docker run --rm sourcekit-builder:302 context | docker build -t norionomura/sourcekit:302 -

I understand that the first half of this switches to the newly generated context, so I need to replace that with:
docker run --rm sourcekit-builder:302_14_04 context | docker build -t norionomura/sourcekit:302 -

But I don't understand what the second command does, do I need to change it to 302_14_04?
Here's the output:
Sending build context to Docker daemon    370MB
Step 1/4 : FROM ubuntu:16.04
 ---> f7b3f317ec73
Step 2/4 : MAINTAINER Norio Nomura<EMAIL_ADDRESS> ---> Using cache
 ---> 203bd6f240f2
Step 3/4 : RUN apt-get update &&     apt-get install -y       clang       curl       git       libblocksruntime0       libcurl4-openssl-dev       libedit2       libicu55       libkqueue0       libpython2.7-dev       libxml2       python2.7       uuid-dev       &&     apt-get clean &&     rm -rf /var/lib/apt/lists/* /tmp/* /var/tmp/*
 ---> Using cache
 ---> 3b8a0e71080c
Step 4/4 : ADD swift-3.1.1-RELEASE-3f82132-with-sourcekit.tar.gz /
 ---> Using cache
 ---> bbffe83f6a70
Successfully built bbffe83f6a70
Successfully tagged norionomura/sourcekit:302_14_04

It seems that this command is still using 16.04 and swift 3.1.1. Could you let me know what I need to do for 14.04 and 3.0.2? Any help is appreciated!

Ok, I see https://hub.docker.com/r/norionomura/sourcekit/ and there is a tag for 302, I'm assuming I should use that. When I try, it still says FROM ubuntu:16.04 and ADD swift-3.1.1 though. Any idea?

docker build -t sourcekit-builder:302_14_04 https://github.com/jeffreyhanna/docker-sourcekit-builder.git

This builds image from master branch of your repository.
You need to specify your 302_14_04 branch as documented in docker build - Docker Documentation instead of master.
docker build -t sourcekit-builder:302_14_04 https://github.com/jeffreyhanna/docker-sourcekit-builder.git#302_14_04
Other steps looks good to me.


Make necessary changes to add support to AMD's MIGraphX backend

Make necessary changes to add support to AMD's MIGraphX backend. The structures of the files are very similar to those of IPU's and STC's. There is a readme file filled with specs of the hardware, models enabled and the directions on how run the benchmarks.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Zhou seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add ggandor/flit.nvim and ggandor/leap-spooky.nvim

Repo URL:
Hey this PR adds two plugins cause they're pretty similar (both plugins/enhancements for leap). Hope that's ok.

https://github.com/ggandor/flit.nvim
https://github.com/ggandor/leap-spooky.nvim

@ggandor
Checklist:

[x] The plugin is specifically built for Neovim, or if it's a colorscheme, it supports treesitter syntax.
[x] The lines end with a .. This is to conform to awesome-list linting and requirements.
[x] The title of the pull request is Add/Update/Remove `username/repo`  (notice the backticks around `username/repo`) when adding a new plugin.
[x] The description doesn't mention that it's a Neovim plugin, it's obvious from the rest of the document. No mentions of the word plugin unless it's related to something else.
[x] The description doesn't contain emojis.
[x] Neovim is spelled as Neovim (not nvim, NeoVim or neovim), Vim is spelled as Vim (capitalized), Lua is spelled as Lua (capitalized), Tree-sitter is spelled as Tree-sitter.
[x] Acronyms should be fully capitalized, for example LSP, TS, YAML, etc.

Thanks for the PR!


add: ContentPanel

new Component for display Information.
Change  main Layout  for new Component.

Welcome Alex! 👌🥳

Very nice proceedings 👍

I love the design!
Added i18n and did a little cleanup...
One test in Header.test.js fails when running in single test mode. I will fix that...

Found the reason why a test in Header.test.js failed when running in single test mode:
The instrumentation of a spy on the window object does not take effect, because the component uses the window object of the iframe in which the current tests is run. The spy however works on the parent window.
Therefore we had to mock the window object for all responsive layout tests...(see last commits)

Sorry, didn't recognized it earlier, but the name of the module must be menu (menue is a typo)'.
I've already renamed all occurrences  within the module.
Would you just rename the packages in src and test , so that we have

src/modules/menu
test/modules/menu

?


Make istioctl x post-install webhook commands work with istiod

istioctl x post-install webhook assumes the webhooks use key/certs from a k8s secret. Istiod generates its owns certs. istioctl needs to be updated to handle this case.

@howardjohn @ostromart @sdake Are these commands still needed?
If so, as part of this work we should add examples to istioctl x post-install webhook.
Note that it is very unclear when and how to use this command:
$ istioctl x post-install webhook status
...
Error: error when displaying validation webhook configuration: error getting ValidatingWebhookConfiguration istio-galley: validatingwebhookconfigurations.admissionregistration.k8s.io "istio-galley" not found
$ istioctl x post-install webhook disable
Are you sure to delete webhook configuration(s)?
y
Error: error when disabling validation webhook configuration: validatingwebhookconfigurations.admissionregistration.k8s.io "istio-galley" not found
$ istioctl x post-install webhook enable
Error: --namespace <namespace-of-webhook-secret> is required
$ istioctl x post-install webhook enable --namespace istio-system
Error: --validation-path <yaml-file> is required for the validation webhook configuration

@esnible this was added in commit: 04e758e76bbf4cb00614709672aa5f51614ded4a
Looking at the commit, the PR was https://github.com/istio/istio/pull/17288 and the issue is: https://github.com/istio/istio/issues/17061.
@lei-tang @incfly is this functionality still necessary after the transition to istiod?
Cheers,
-steve

@lei-tang @incfly last call before I make the suggestion for removal of these operations.
I have never seen them used not documented, and if we can reduce the footprint of the CLI, we should do so.
Cheers,
-steve

The documentation for this command and the motivation behind this work can be found at https://istio.io/v1.4/docs/tasks/security/webhook/ and https://istio.io/latest/blog/2019/webhook/.
In short, the motivation behind this work is to enhance Istio security by letting the operator manage the webhooks, which still applies to istiod. However, the plan for updating this work to stay current with the latest istiod is TBD. Any helps from the community are welcome and appreciated.

@lei-tang @sdake istioctl x post-install webhook is deprecated in 1.8 and will be removed in 1.9.  Please don't work on it.  webhook was the only command in post-install and it is likely that istioctl x post-install will be removed at the same time.


Add Charterhouse School

@WilliamBurns Pull request merged. Thank you!

@philipto I know this is very old, but thank YOU very much!


NTV-306 : Migrate to Kotlin Extension Discovery Utils classes

📲 What
Migrate to Kotlin Extension Discovery Utils classes
🛠 How
Migrate from Java to Kotlin extension the next Utils classes
DiscoveryUtils.java to DiscoveryUtils.Kt
DiscoveryParamsUtils.java And DiscoveryDrawerUtils.java toDiscoveryParamsExt.kt
Story 📖
https://kickstarter.atlassian.net/browse/NTV-306

two tests failing, once they are fixed, I'll review the PR


Do not configure ports for host-networked pods in secondary networks

The previous condition was incorrect and was ignoring host-networked pods only when addPort was false, additionally it didn't ignore cluster-networked pods that do not want a port configured(addPort==false).
This means that the controller tried to configure the logical port for new host-networked pods and on updates for cluster-networked pods.
Fix the condition by exiting when the pod is host networked or addPort is false.
This is likely a mistake from reversing the default network condition:
https://github.com/ovn-kubernetes/ovn-kubernetes/blob/d20b22110a912a244f4cd42f0969df60eaf434fc/go-controller/pkg/ovn/ovn.go#L166
The issue can be reproduced by creating a host networked pod in a UDN namespace:
E1125 13:40:08.269965   17921 obj_retry.go:671] Failed to update *v1.Pod, old=nad-l3/host-network-pod, new=nad-l3/host-network-pod, error: failed to add logical port of Pod nad-l3/host-network-pod for NAD nad-l3/l3-network: failed to update pod nad-l3/host-network-pod: admission webhook "ovn-kubernetes-admission-webhook-pod.k8s.io" denied the request: user: "system:ovn-node:ovn-worker" is not allowed to set k8s.ovn.org/pod-networks on pod "host-network-pod": the annotation is not allowed on host networked pods

@cathy-zhou ptal

lgtm


What About Foundation-Rails for Version 6 ?

Just wanna know how early we can expect Foundation 6 for Ruby on Rails ?

+1 :smile:

We're working on it this week and next week!

Update: the gem is almost ready, just have a few kinks to work out.

We shipped the gem last week :) foundation-rails
Foundation 6.1.1 is the first version to work with it.


ui: remove outdated members from about section

Result

Thank you @karolina-siemieniuk-morawska could you please also add the reference of the issue?


old dependency remove

Remove

#945 - This will remove an obsolete dependency on iobio.js which has been replaced with an npm module


New updates

The Pinball Arcade just announced that they're adding VR support BUT in a standalone spinoff of the original game called "The Stern Pinball Arcade" only offering a small subset of a subset of the games available in the original version. This version will also be released cross platform.
I really do not understand their decision to implement VR this way, but I think this revives the need for Pinball VRcade. I want to play the full version of TPA in VR.

I'm not a developer, so I can't contribute much to reviving this, but any ideas on getting someone who can? This has potential to be an amazing VR mod.


String is van empty or null

Hi Kaushik,
I like your example of the form validation, thanks for that!
Is there a reason why you use the
com.google.common.base.Strings.isNullOrEmpty instead of the TextUtils.isEmpty() in there?

laziness and legacy code :)
thanks for pointing it out. i should totally get rid of it. that would additionally help me get rid of the guava dependency.


KhrSurface API not finding any native functions

Summary
I'm following the tutorial over at vulkan-tutorial.com, but in c# with Silk.Net.Vulkan and Silk.Net.GLFW. Most of the tutorial translated well, except for GetPhysicalDeviceSurfaceSupport which throws Silk.NET.Core.Loader.SymbolLoadingException: Native symbol not found (Symbol: vkGetPhysicalDeviceSurfaceSupportKHR).
Steps to reproduce

Platform: Desktop (PopOS/Linux)
Framework Version: .NET 6
API: Vulkan
API Version: Vulkan 1.2

Initialize vulkan instance with GLFW extensions
Enumerate over GetPhysicalDeviceQueueFamilyProperties
Check if a family index returns true when using GetPhysicalDeviceSurfaceSupport
Crash with the exception above

Comments
I am new to vulkan, but decently know c#. I have tested the tutorial's c++ example, it compiled an ran, so the function does exist on my computer.

How did you call the function? Did you specify the extension when creating the device?

Thanks for getting back to me. I first call  var glfwExtensions = _glfw.GetRequiredInstanceExtensions(out var glfwExtensionCount); to get the extensions glfw needs, and pass that to my InstanceCreateInfo:
 var createInfo = new InstanceCreateInfo()             {                 SType = StructureType.InstanceCreateInfo,                 PApplicationInfo = &appInfo,                 EnabledExtensionCount = extensionCount,                 PpEnabledExtensionNames = extensions,                 EnabledLayerCount = 0             };
(glfwExtensions/glfwExtensionCount and extensions/extensionCount are the same)

Are you using the TryGetExtension method to get your KhrSurface?

I call if (!vk.TryGetInstanceExtension(_instance, out _surfaceApi)) throw new Exception("Failed to get surface api, which is required for managing vulkan windows.");

Everything seems to be in order then, I’m not sure why this is happening. Other than manually validating that GLFW is giving you the correct extensions (using SilkMarshal.PtrToStringArray and then writing that to Console or Debug) I’m not sure what to suggest

I did that, and it returned VK_KHR_surface and VK_KHR_xcb_surface

Can you run the SDK included vkcube? If not it's probably not our issue but a problem with your computer.
If yes, please send us a full repo (csproj + cs files) to run locally.

I can run VKCube locally.
I don't want to share the whole project, but I will share the stuff with Vulkan in it (Also pls ignore the stupid name). The error is in VulkanDeviceManager.cs, line 127. Link: https://drive.google.com/file/d/1WsWFzBzquacGMImiwlyZkDXLDI-d-MYV/view?usp=sharing

Going to migrate this to a discussion until we know for sure that this is a silk bug.


Fallback for no-js or feature detection / activation class for Sal.js

When testing different scenarios on my website I noticed if JavaScript is disabled or sal.js couldn't be loaded for some reason, the animated elements are staying invisible.
On my sites I’m usually detecting disabled js with removing a „no-js“ class from the html tag and loading a different css-file. But this doesn’t solve the problem if js is enabled but Sal.js couldn’t be loaded, so I think it would be a simple and effective but really useful feature for sal.js to have a „native“ way for creating a fallback.
I’m not sure what the best practice would be for sal of the following two simple ways:

having a specific class on the html element which is removed if sal is loaded. So there only a class must be added in manually/not by sal.js, which has the advantage that the according css is smaller as the additional class is only needed for the fallback-part and no additional class has to be included when creating additional/custom animations.

adding a sal.js specific (Like „sal-enabled“) class to the html or body element, like most js librarys do. The advantage would be that no “manual“/separate js work has to be done, as sal handles this by its own and there is done nothing if it couldn’t be loaded. Disadvantage is that the sal-activated class has to be included in all css selectors which makes the the css file a bit larger (not much larger than version 1 when using gzip) and as written above the class has to be included in all custom animations.

I personally prefer the second way, which I’m currently integrating in my project, as my compiled css is already has already around 15k lines of code and I’m not done yet, so i don’t mind the bit additional code I don’t have to care about adding the class on the html element.

This one is a bit complicated, because adding such class will cause flash of content when sal is initialised. Imagine, that at the beginning we show all animated elements. Then when sal is launched, we hide them. So user with working JS for split of a second will see animated content. I'm not sure if I'd like to introduce such change to library, that will cause in most of the cases flash of animated content. I think it should be handled by developers themselves.

Maybe we can use the  Tag and then apply inline styles which makes the elements visible?

Maybe we can use the <noscript> Tag and then apply inline styles which makes the elements visible?

That's a great idea! Honestly, it can't be a feature of this library, but I can make it as a recipe and put it into docs, so anyone can use it later. I think that something like this should work:
<noscript>
  <style type="text/css">
    [data-sal|='fade'] {
      opacity: 1;
    }

    [data-sal|='slide'],
    [data-sal|='zoom'] {
      opacity: 1;
      transform: none;
    }

    [data-sal|='flip'] {
      transform: none;
    }
  </style>
</noscript>

That works well! Thanks!

Closing with https://github.com/mciastek/sal/commit/212402e95982020310fbe66776b744bb83ebf64c


[MENFORCER-314] - Warn if EnforcerRuleException has no message

Following this checklist to help us incorporate your
contribution quickly and easily:

[x] Make sure there is a JIRA issue filed
for the change (usually before you start working on it).  Trivial changes like typos do not
require a JIRA issue.  Your pull request should address just this issue, without
pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Format the pull request title like [MENFORCER-XXX] - Fixes bug in ApproximateQuantiles,
where you replace MENFORCER-XXX with the appropriate JIRA issue. Best practice
is to use the JIRA issue title in the pull request title and in the first line of the
commit message.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Run mvn clean verify to make sure basic checks pass. A more thorough check will
be performed on your pull request automatically.

If your pull request is about ~20 lines of code you don't need to sign an
Individual Contributor License Agreement if you are unsure
please ask on the developers list.
To make clear that you license your contribution under
the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

[x] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004

[ ] In any other case, please file an Apache Individual Contributor License Agreement.

I pushed an alternative approach because I couldn't really decide which one is better: #44
Please merge whichever PR you like best.

I jumped ahead and extended TestEnforceMojo. IMHO this should be sufficient for such a small change.
Please note that I won't be able to put much more effort into this entire issue as I am busy elsewhere.
PS: Yes, I did see that there is a custom MockEnforcerRule but I saw no real sense in extending this with more logic as mockito is just way more flexible at such tasks (and you need to write less code).

@famod great work.
Will commit soon

waiting for CI
https://builds.apache.org/job/maven-box/job/maven-enforcer/job/MENFORCER-314/

You are welcome and thank you, too!


Fixed ClusterStateVerifier to consider timeout

ClusterStateVerifier now pays attention to the timeout setting that the caller may provide.
This was needed while testing upgrade to helix-0.8.0

Codecov Report

Merging #2497 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2497      +/-   ##
==========================================
+ Coverage   69.42%   69.44%   +0.01%
==========================================
  Files         848      848
  Lines       39530    39530
  Branches     5327     5327
==========================================
+ Hits        27444    27450       +6
+ Misses      10297    10295       -2
+ Partials     1789     1785       -4

Impacted Files
Coverage Δ

...a/manager/realtime/RealtimeSegmentDataManager.java
75% <0%> (-25%)
:arrow_down:

...e/impl/dictionary/LongOnHeapMutableDictionary.java
82.22% <0%> (-13.34%)
:arrow_down:

.../impl/dictionary/FloatOnHeapMutableDictionary.java
86.66% <0%> (-6.67%)
:arrow_down:

.../impl/dictionary/LongOffHeapMutableDictionary.java
87.27% <0%> (-5.46%)
:arrow_down:

...impl/dictionary/FloatOffHeapMutableDictionary.java
87.27% <0%> (-5.46%)
:arrow_down:

...not/core/query/pruner/DataSchemaSegmentPruner.java
53.65% <0%> (-4.88%)
:arrow_down:

...linkedin/pinot/common/metrics/AbstractMetrics.java
75.39% <0%> (-1.59%)
:arrow_down:

.../broker/routing/HelixExternalViewBasedRouting.java
90.45% <0%> (-0.71%)
:arrow_down:

...pby/NoDictionarySingleColumnGroupKeyGenerator.java
92.92% <0%> (+1.01%)
:arrow_up:

...lix/core/realtime/PinotRealtimeSegmentManager.java
73.44% <0%> (+1.12%)
:arrow_up:

... and 9 more

Continue to review full report at Codecov.

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

I guess it makes sense to provide some context. In this case, the tool was used to run a 24-hour test with new helix package, taking turns to kill a random server/broker/controller and wait for tables to get stable. Realtime tables were also adding segments in the meantime on a couple of tables.


fix: properly handle deserialization from Reader

The pattern "let s: &str = Deserialize::deserialize(deser)" used in multiple places is not ideal, as it generates errors when the deserializer uses an IO Reader and not a string. To fix this, implementing a visitor is preferred, as it gives the deserializer the choice of allocating or not a string.
To ensure the issue is fixed, the tests now deserialize both from a string and from a reader.
I haven't used the same fix for the integer/string deserialization for numbers. Using a Visitor is much more cumbersome as it would depend on each numeric type, and using a Cow type is enough to fix the issue.
Closes #55

Thanks, I have signed the CLA

Thank you :smile:


Added Gujarati Translation

Thanks for taking the time to contribute this localization pull request. However, we are not able to accept contributions to this historical reference repository at this time.


Upgrade to react-admin v5

Marked as a WIP because I can't test locally, and installing npm from my github is not working.
Fixes #60

Thanks for initializing this PR!
I'll try to take over to see if I can finish the migration.
I'll also need to do some more checks to ensure ra-data-postgrest is fully compatible with RA v5 (in which case I'll open a PR to fix the peer dependencies declaration).

Great. Looking forward to hearing from you.

PR to update ra-data-postgrest's peerDeps: https://github.com/raphiniert-com/ra-data-postgrest/pull/142

PR is all green! :white_check_mark:
Just waiting to see with the team if this needs to be merged on the main branch or the next branch. Stay tuned.

We can merge on master. However the 3.0.0 release will have to wait for either https://github.com/raphiniert-com/ra-data-postgrest/pull/142  to get merged, or for us to fork and release ra-data-postgrest.


add training example

for example learning a few non linear layers on top of clip embeddings

https://huggingface.co/datasets/imagenet-1k use this to build a linear probe layer on top of clip embeddings for imagenet


Rebrand Debts into Credits

Credits have a more positive feeling vs debts, so we're rebranding! Please reflect this in the UI.

The raw vectors are available on this figma - check page 2
https://www.figma.com/file/t7mstvFjXU057Gr5nEx8eQ/Ubiquity-Token-Icons?node-id=124%3A30

hey btw we’re in the process of rebranding all the debts into Ubiquity Credit (uCR) and Ubiquity Credit NFT (uCR-NFT) so this should be reflected in the UI!

Chris will analyze and compile the questions.

uAR -> Ubiquity Credit (uCR)
uDEBT -> Ubiquity Credit NFT (uCR-NFT)

https://github.com/ubiquity/ubiquity-dollar-development/pull/149 Use this branch

Waiting for the confirmation from @Zequez before closing this issue.

Are we using the same icon for the fungible and non-fungible versions?

Alex - we will use the same icon for now


right click "format disk..." stopped working in Gnome Files

Describe the bug
I can not format disk in Gnome anymore since updating to 20.03
To Reproduce
Steps to reproduce the behavior:

Insert a disk
Right click "format" in Gnome Files
Nothing happens

Expected behavior
The format disk dialog pops up
Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Gnome files logs the following to journalctl -xef
Apr 29 13:41:57 t490s org.gnome.Nautilus[1875]: Cannot parse integer value “1e00007” for --xid

Ubuntu also seems affected: https://bugs.launchpad.net/ubuntu/+source/nautilus/+bug/1853784
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

Just found out that this issue is also being tracked upstream: https://gitlab.gnome.org/GNOME/nautilus/issues/1265

Just found out that this issue is also being tracked upstream: https://gitlab.gnome.org/GNOME/nautilus/issues/1265

Looks like it got fixed upstream. Closing.


Edge case with RbxUtils Signal cleanup

The cleanup function for tables first checks if the table includes the .destroy key, and then it checks if there is a .Destroy key. This causes the Signal module by sleitnick to throw an error since the strict metatable wrapping Signal errors when indexing a key that doesn’t exist.
The solution would be to swap the order in which the cleanup function checks for these keys.

If it would be best to submit the issue to sleitnick, we can close this issue.


Fix broken links

The installation guides are referring to a no-longer valid section name in the developer guide.

@jodh-intel : So IIUC,  CentOS, Fedora, Red Hat, Ubuntu installation guides are not pointing to the right Kubernetes link when deciding which container manager to use.  The correct link being: https://github.com/kata-containers/documentation/blob/master/Developer-Guide.md#install-kubernetes
I can fix this, if this is what is to be done.

Hi @nitkon - thanks for offering but I'd raised this for @scooley on #183.

I wish we could find a way to allow both users to self-assign issues and allow us to assign to new contributors :)

@jodh-intel : Ahh okay, no problem.

It's rather annoying actually because those links changed but our URL checker didn't notice:
https://github.com/kata-containers/tests/blob/master/.ci/static-checks.sh#L363
I'm really not sure what we can do about that aside from downloading the document and grep-ing for the anchor name HTML.

@jodh-intel - as for enabling external contributors, sadly it looks like you should not hold your breath.... https://github.com/isaacs/github/issues/100 :-(

Wow - 5 years....


Generate PyPI encrypted password with com option

Follow up for https://github.com/graphql-python/graphql-core-next/pull/44
Travis deployment fails with a 403 from PyPI during the deployment step.
Steps taken:
brew install travis
cd ~/graphql-core-next
travis encrypt <password> --add deploy.password --com

Unfortunately, this didn't work, either.


Use an ID Attribute to style an Element

CatPhotoApp
Click here for cat photos.

Things cats love:

cat nip
laser pointers
lasagna

Top 3 things cats hate:

flea treatment
thunder
other cats

 Indoor
 Outdoor
 Loving
 Lazy
 Energetic

Submit

Your form element should have the background-color of green. Haven't I?

Nope. Read the directions carefully: your 'form element' should have the background-color of green. You have your "background-color:green" in the wrong place(div id).

Please use the Help Chat for assistance with challenges and only create Github issues once others have confirmed the problem is with the site and not your code.
Thanks and happy coding!

<EMAIL_ADDRESS>On Jan 5, 2016 3:32 AM, "nandinimenon77"<EMAIL_ADDRESS>wrote:

#cat-photo-element{ background-color: green; } red-text { color: red; }
h2 { font-family: Lobster, Monospace; } p { font-size: 16px; font-family:
Monospace; } thick-green-border { border-color: green; border-width: 10px;
border-style: solid; border-radius: 50%; } smaller-image { width: 100px; }
gray-background { background-color: gray; }
CatPhotoApp
Click here for cat photos
Things cats love:
cat nip
laser pointers
lasagna
Top 3 things cats hate:
flea treatment
thunder
other cats
Indoor Outdoor Loving Lazy Energetic Submit
Your form element should have the background-color of green Haven't I?
—
Reply to this email directly or view it on GitHub.


WIP add windows support

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@basnijholt that rerender removed all the windows stuff!

It claims the windows build succeeded, but if you look at the Appveyor logs you'll see that it didn't actually do anything!

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Selectors are suggested to take a <two spaces>#<one space>[<expression>] form. See lines [29]

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Ok it builds but there are 3 tests failing. The logs are all mangled (looks like appveyor might be running several things in parallel and interleaving the output) so it's difficult to tell what's going on, but it seems that what is failing is:

test_rcond_from_lu: there is a comment above the failing test that seems to imply that the test should not be too strict. I do not know enough about this specific issue to tell whether we should just disable the test
test_symm_algorithm_equivalence: Seems our check triggers some internal assertions withing numpy? A serious WTF going on in here.
test_finalization: the check that all the sites in the system are also in the builder fails. WTF

To add insult to injury, appveyor for some reason decides that the build succeeds despite the failing tests: https://ci.appveyor.com/project/conda-forge/kwant-feedstock/build/1.0.29

Action plan
test_rcond_from_lu: we just have to decide whether this test is important. My vote would be just to disable it on windows
test_symm_algorithm_equivalence: the apparent exchange of the degrees of freedom could be due to assumptions we make about the way eigenvalues are ordered, maybe in some very degenerate case? We would need to look at the relevant code in Kwant to see where the problem could be, and investigate in isolation on our own machines (because modifying Kwant and getting conda forge to rebuild is far too slow).
test_finalization: could be due to the fact that we are comparing tinyarrays with tuples. On linux assert tinyarray.array([1, 2]) in  {(1, 2)} works, but maybe (for some crazy reason) this is not the case on windows

@basnijholt you have a windows machine: we should block a few hours to debug this.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

When defining a source/url please add a sha256, sha1 or md5 checksum (sha256 preferably).

For recipe:

Using toolchain directly in this manner is deprecated.  Consider using the compilers outlined here.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@basnijholt merge time :tada: :tada:


[Backport 2.x] [Segment Replication] Trigger a round of replication for replica shards during peer recovery when segment replication is enabled

Signed-off-by: Rishikesh1159<EMAIL_ADDRESS>
Description
This PR Backports #5332
Issues Resolved
Resolves #5313
Check List

[x] New functionality includes testing.

[x] All tests pass

[x] New functionality has been documented.

[x] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff
[x] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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

Codecov Report

Merging #5533 (d09e300) into 2.x (69e9701) will decrease coverage by 0.10%.
The diff coverage is 23.07%.

@@             Coverage Diff              @@
##                2.x    #5533      +/-   ##
============================================
- Coverage     70.75%   70.64%   -0.11%
+ Complexity    58373    58337      -36
============================================
  Files          4701     4701
  Lines        279161   279198      +37
  Branches      40765    40772       +7
============================================
- Hits         197526   197249     -277
- Misses        65145    65508     +363
+ Partials      16490    16441      -49

Impacted Files
Coverage Δ

...ch/indices/cluster/IndicesClusterStateService.java
62.72% <23.07%> (-6.31%)
:arrow_down:

...pensearch/client/cluster/RemoteConnectionInfo.java
0.00% <0.00%> (-75.00%)
:arrow_down:

...n/indices/forcemerge/ForceMergeRequestBuilder.java
0.00% <0.00%> (-75.00%)
:arrow_down:

.../indices/forcemerge/TransportForceMergeAction.java
25.00% <0.00%> (-75.00%)
:arrow_down:

...a/org/opensearch/client/cluster/ProxyModeInfo.java
0.00% <0.00%> (-63.16%)
:arrow_down:

...a/org/opensearch/client/cluster/SniffModeInfo.java
0.00% <0.00%> (-56.25%)
:arrow_down:

...ava/org/opensearch/action/NoSuchNodeException.java
0.00% <0.00%> (-50.00%)
:arrow_down:

...adcast/BroadcastShardOperationFailedException.java
55.55% <0.00%> (-44.45%)
:arrow_down:

...earch/index/fielddata/SortingNumericDocValues.java
57.14% <0.00%> (-42.86%)
:arrow_down:

.../opensearch/client/cluster/RemoteInfoResponse.java
61.53% <0.00%> (-38.47%)
:arrow_down:

... and 448 more

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


Merging to dev

Highilights:

DBUtils now reads db info and credentials from db.properties file in resources folders.
Fixed potential sql-injection using prepared statements.
Fixed code smells by updating variable names to follow proper conventions.
Improved coverage through use of EqualsVerifier.

Highilights:

DBUtils now reads db info and credentials from db.properties file in resources folders.
Fixed potential sql-injection using prepared statements.
Fixed code smells by updating variable names to follow proper conventions.
Improved coverage through use of EqualsVerifier.


Request to add check for CSCvw33061

CSCvw33061 - Prevent "internal" Encap Blocks (AVE) from leaf programming to avoid overlapping VLAN VNID behavior
https://bst.cloudapps.cisco.com/bugsearch/bug/CSCvw33061

Pulled from #39
Validation Type
[ ] - Fault
[X] - Config
[ ] - Bug
[ ] - Other
What needs to be validated
for fvnsEncapBlk.role=internal, since ACI 4.2(6m) or 5.2(1g) , the corresponding stpAllocEncapBlkDef is deleted post upgrade, as a result, those vlans are deleted from the related switch. If they were used for any user tenant before, outage can be expected.
Why it needs to be validated
Outage can be triggered post upgrade if customer is not aware of this behavior change, introduced by [CSCvw33061]
Additional context
This behavior change is publicly release noted, documented however we see customer's still facing this trouble.

`______________________________________________________________________________________________________________________________________________________________ test_pos_internal_vlanpool_check _______________________________________________________________________________________________________________________________________________________________
-------------------------------------------------------------------------------------------------------------------------------------------------------------------- Captured stdout call ---------------------------------------------------------------------------------------------------------------------------------------------------------------------
Starting Positive internal_vlanpool_check ...
[Check  1/6] Internal VLAN Pool Check...                                                                          FAIL - OUTAGE WARNING!!
Target Version  VLAN Pool        Warning

5.2(2a)         jr-dynamic-pool  VLAN Pool contains an encap block with role "internal" and is associated to a non-AVE domain
Recommended Action: Ensure any VLAN Encap blocks explicitly used for leaf front panel VLAN programming are set to "external (on the wire)"
Reference Document: https://bst.cloudapps.cisco.com/bugsearch/bug/CSCvw33061
[Check  2/6] Internal VLAN Pool Check...                                                                                PASS
[Check  3/6] Internal VLAN Pool Check...                                                                          FAIL - OUTAGE WARNING!!
Target Version  VLAN Pool        Warning

5.2(6a)         jr-dynamic-pool  VLAN Pool contains an encap block with role "internal" and is associated to a non-AVE domain
Recommended Action: Ensure any VLAN Encap blocks explicitly used for leaf front panel VLAN programming are set to "external (on the wire)"
Reference Document: https://bst.cloudapps.cisco.com/bugsearch/bug/CSCvw33061
[Check  4/6] Internal VLAN Pool Check...                                                                          FAIL - OUTAGE WARNING!!
Target Version  VLAN Pool        Warning

4.2(7d)         jr-dynamic-pool  VLAN Pool contains an encap block with role "internal" and is associated to a non-AVE domain
Recommended Action: Ensure any VLAN Encap blocks explicitly used for leaf front panel VLAN programming are set to "external (on the wire)"
Reference Document: https://bst.cloudapps.cisco.com/bugsearch/bug/CSCvw33061
[Check  5/6] Internal VLAN Pool Check...                                                                                PASS
______________________________________________________________________________________________________________________________________________________________ test_neg_internal_vlanpool_check _______________________________________________________________________________________________________________________________________________________________
-------------------------------------------------------------------------------------------------------------------------------------------------------------------- Captured stdout call ---------------------------------------------------------------------------------------------------------------------------------------------------------------------
Starting Negative internal_vlanpool_check ...
[Check  1/6] Internal VLAN Pool Check...                                                                                PASS
[Check  2/6] Internal VLAN Pool Check...                                                                                PASS
[Check  3/6] Internal VLAN Pool Check...                                                                                PASS
[Check  4/6] Internal VLAN Pool Check...                                                                                PASS
[Check  5/6] Internal VLAN Pool Check...                                                                                PASS
================================================================================================================================================================== 11 passed in 0.09 seconds ==================================================================================================================================================================
`

Check working on live script..Committing changes.
admin@fab3-apic1:techsupport> python aci-preupgrade-validation-script.py
==== 2022-11-21T12-52-28-0500 ====
Enter username for APIC login          : admin
Enter password for corresponding User  :
Checking current APIC version (switch nodes are assumed to be on the same version)...5.2(7f)
Gathering APIC Versions from Firmware Repository...
What is the Target Version?     : 1
You have chosen version "aci-apic-dk<IP_ADDRESS>f.bin"
...
[Check 40/40] Internal VLAN Pool Check...                                                                          FAIL - OUTAGE WARNING!!
Target Version           VLAN Pool        Warning

aci-apic-dk<IP_ADDRESS>f.bin  jr-dynamic-pool  VLAN Pool contains an encap block with role "internal" and is associated to a non-AVE domain
Recommended Action: Ensure any VLAN Encap blocks explicitly used for leaf front panel VLAN programming are set to "external (on the wire)"
Reference Document: https://bst.cloudapps.cisco.com/bugsearch/bug/CSCvw33061
=== Summary Result ===
PASS                     : 32
FAIL - OUTAGE WARNING!!  :  3
FAIL - UPGRADE FAILURE!! :  2
MANUAL CHECK REQUIRED    :  1
N/A                      :  2
ERROR !!                 :  0
TOTAL                    : 40
=== Summary Result ===
PASS                     : 32
FAIL - OUTAGE WARNING!!  :  3
FAIL - UPGRADE FAILURE!! :  2
MANUAL CHECK REQUIRED    :  1
N/A                      :  2
ERROR !!                 :  0
TOTAL                    : 40


Revert "fix graphs interface for weighted graphs"

Reverts Circuitscape/SpatialGraphs.jl#6. That was a unneeded workaround. I just needed to tighten the version requirements for SimpleWeightedGraphs

Codecov Report

Merging #7 (5f69234) into main (a4fd38a) will increase coverage by 4.79%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main       #7      +/-   ##
==========================================
+ Coverage   92.89%   97.68%   +4.79%
==========================================
  Files           5        5
  Lines         211      173      -38
==========================================
- Hits          196      169      -27
+ Misses         15        4      -11

Impacted Files
Coverage Δ

src/graph_interface.jl
95.83% <100.00%> (+15.18%)
: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 a4fd38a...5f69234. Read the comment docs.


Propagate 'response' event through file.createReadStream()

Background:
We run a proxy service that serves a large number of files of varying size to downstream clients. In cases where we encounter an error serving the requested file (such as 404s) we intercept the error to the downstream client and pass through a custom error and template. See #522 for an oversimplified version.
Request:
The previous request to remove the required call to file.getMetadata() before invoking file.createReadStream() has been a big help in reducing mean latency since it removed a roundtrip for each download request. Our next challenge has been intercepting errors (namely those 404s) in a way that continues to let us simply pipe the result to the downstream client (and not buffer the download file until completion), and send the appropriate status code + headers.
Currently, in the case of a 404, the stream returned from file.createReadStream() will first emit a data event (an XML response containing an error) and then emits an error event (the formatted error object). Ideally, we could inspect the response headers before deciding whether or not to stream the result from GCS down to the client, and currently we have to buffer some number of data event payloads until we're sure we haven't encountered a GCS error.
Details:
The Node http.ClientRequest class specifies a response event that fires once and exposes (amongst other things) the HTTP response headers. If we could subscribe to that event, it would allow us to inspect the response from GCS, send the appropriate status code and headers to our downstream client, prior to piping any data.
diff --git a/lib/storage/file.js b/lib/storage/file.js
index d64aa09..137baba 100644
--- a/lib/storage/file.js
+++ b/lib/storage/file.js
@@ -464,6 +464,8 @@ File.prototype.createReadStream = function(options) {
         request(authorizedReqOpts)
           .on('error', done)

+          .on('response', throughStream.emit.bind(throughStream, 'response'))
+
           .on('data', function(chunk) {
             if (crc32c) {
               localCrcHash = crc.calculate(chunk, localCrcHash);

Don't hesitate to let me know if you have any thoughts or questions, and thanks for all of your hard work on this library!

Interesting use case. I don't see why we couldn't support this. I'm all for exposing this functionality if it helps our users :) @stephenplusplus what do you say?

@robertdimarco I'm going to make an executive decision here and say "why not!". Could you create a PR for this change with a unit test or two and assign me to review when you're ready? Thanks!

@ryanseys Sure thing, I'll follow-up with that shortly.

:+1:

Thanks, @robertdimarco!


ServiceManager v2-v3 compatibility

This builds on @ezimuel's #64 adding v2 factories, tests and fixes, plus incorporating some of the comments made on that PR.

@marc-mabe @weierophinney Think this one is ready if you can check.
The fail happened touching the filesystem - don't think that's me:
https://travis-ci.org/zendframework/zend-cache/jobs/107458284#L544
Man I love seeing those PHP7 execution times :)

Merged with #69 (which was rebased).


error in installing bosh_cli

can anyone provide me clear steps in installing ruby(1.9.3)version and bosh_cli ?

As far as I understand this issue is duplicate of this one https://github.com/cloudfoundry/bosh/issues/1024 and can be removed


High-Scale IPcache: Chapter 2

This picks back up closed PR #25436, which needed some changes to get it past the finish line.
Differences from that PR:

The manager now uses the Resource paradigm
The BPF map is now exported as a Cell in the Hive

This is in the context of the high-scale ipcache feature described at https://github.com/cilium/design-cfps/pull/7.
This pull request simply adds a new CRD, used to describe IP addresses of entities outside the Cilium high-scale ipcache domain, typically outside the clustermesh.
Updates: https://github.com/cilium/cilium/issues/25243.
New CiliumWorldCIDRSet CRD to define IP addresses outside the Cilium domain in the context of high-scale ipcache.

/test

@tommyp1ckles @christarazi updated based on feedback, thanks!

/test


Roon Bridge on Coreelec device - Mecool K3 Pro

Hi,
I have Synology NAS, running Roon server. And it's working great. I can access it from all my devices, no issues.
I bought a new DAC for living room and before I spend money for music streamer device or raspberry pie, I decided to try installing "Roon bridge" in Mecool KIII Pro box, running CoreElec OS. I installed Docker, then Portainer, Watchover and finally Roon Bridge.
So, Roon Bridge container is up and running, but It's not visible to Roon server for some reason.
This is the command I used deploying it:
docker run --name RoonBridge --net=host -e TZ=Europe/Dublin -d -v /storage/.config/docker/roon:/var/roon dubodubonduponey/roon-bridge:latest
Log:
00:00:00.015 Warn: get lock file path: /tmp/.rnbgem0-
00:00:00.369 Trace: [childprocess] using unix child process
Initializing
00:00:00.558 Info: Starting /boot/bin/RoonBridge/Bridge/RoonBridgeHelper
00:00:00.588 Info: ConnectOrStartAndWaitForExit RAATServer, path: /boot/bin/RoonBridge/Bridge/RAATServer
Not Running (.o)
00:00:00.034 Warn: get lock file path: /tmp/.rnbhgem0-
Running
Thank you for any help with this

Hi @Martynet

docker run --name RoonBridge --net=host -e TZ=Europe/Dublin -d -v /storage/.config/docker/roon:/var/roon dubodubonduponey/roon-bridge:latest

This ^ is not the command I provided in the README, right?
Is there a reason why you are using this instead and removed the other options?

So, unless you had good reasons to use your own command, you should really start with using the right command instead.
If that still does not work, you very likely have a networking problem:

what EXACT command did you use to start roon server?
is your Roon server on the same vlan as your Roon bridge?
do you believe mDNS / bonjour / zeroconf is working fine on your lan, between your server and bridge?
does CoreElec have any sort of firewalling / fancy networking that would prevent multicast from working?

Hi, thank you for quick reply. Sorry, I didn't look in readme previously. So now I did and found this:
docker run -d
--net host
--name bridge
--read-only
--cap-drop ALL
--group-add audio
--device /dev/snd
--rm
ghcr.io/dubo-dubon-duponey/roon-bridge
But if I try that, I just get an error:
Unable to find image 'ghcr.io/dubo-dubon-duponey/roon-bridge:latest' locally

docker: Error response from daemon: Get https://ghcr.io/v2/dubo-dubon-duponey/roon-bridge/manifests/latest: unauthorized.
See 'docker run --help'.

docker run --name RoonBridge --net=lsio -e TZ=Europe/Dublin -d -v /storage/.config/docker/roon:/var/roon dubodubonduponey/roon-bridge:latest

Yes, all my devices are on the same network and can see each other. CoreElec is basically light Linux with KODI layer on top. You install Docker as an add-on and then you can install images from KODI environment through "CoreElec - linuxserver.io" repository. There's things like Plex server, TVHeadednd, NetData etc... I have few of these installed and all working fine - Portainer and Netdata.

Communication should be fine. I'm running LOTS of images in Docker in my Synology server, where Roon server is running. There's for example Navidrome music server which I can connect to from CereElec perfectly. Everything just works between all my servers and clients.

No.

One of my friends, who is familiar with Docker much more than me, think that the problem is permissions of some sort...
Thank you for all the help.

You can just use "dubodubonduponey/roon-bridge" instead of ghcr.io/dubo-dubon-duponey/roon-bridge if you can't m/won't authenticate with ghcr.

The way you start roon server is unlikely to work. Roon need host or macvlan networking for its discovery mechanism to work

Roon relies on multicast for discovery. This is very different from a networking perspective. This is not the same thing as being able to access an http server on another machine. So, are you positive mDNS reolution works between these two machines?

As stated above, the fact that other containers "work" is no guarantee - the issue is networking here

Summarizing:

start the bridge with the provided command from the readme (feel free to use the hub version instead of the ghcr version if you prefer)
make sure you start the server in host or macvlan networking (no other networking mode will work)

If you do these two and it still does not work, we can try to debug this further.
Also, it's unlikely that your problem is a permission problem.
Hope that helps.

Thank you, will try now

Re reading your answer to question 1: this is the bridge command line, not the server one.

I'm not 100% sure about that, because I don't know how to test it. But, there's Airplay active on CoreElec device and Roonserver can see it and it works. Does that help answer your question?

I am trying to install Bridge, not the server

so the command line for Bridge is right. Bridge Server is already running and working on Synology NAS and I can access it from my Mac with Roon app and iOS devices.

You still need RoonServer, and you still need it to be started with the right networking mode, otherwise you will not see your bridge - hence my question.

You mean another RoonServer? Not just the one on Synology?

I mean the one server you already have.
Can you just copy the exact command you used to start it?

Trying now

Docker stopped working altogether... Even Portainer and Netdata don't work anymore. When typing command Docker ps, it's empty... I think I installed and removed that RoonBridge so many times, that it just got tired of me...  I don't have the strength to reinstall the CoreElec so I'm giving up on this and I ordered 2012 Mac mini from eBay to run the RoonBridge.

Docker stopped working altogether...
Even Portainer and Netdata don't work anymore. When typing command Docker ps, it's empty... I think I installed and removed that RoonBridge so many times, that it just got tired of me... I don't have the strength to reinstall the CoreElec so I'm giving up on this and I ordered 2012 Mac mini from eBay to run the RoonBridge.

Ok. Wishing you better luck with that one then.
I'm going to close this ticket then.


Fixing misspelled word: "Consolidating"

Tiny change, but it caught my eye.

Hi @thr3-arendt, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;


API design of stencil.Delete vs stencil.RemoveAll

Just some feedback:
stencil.Delete is a bit awkward to use, since it's stateful - you have to run stencil.Create and then stencil.Delete.
It's also analogous to stencil.RemoveAll, but RemoveAll takes an argument and Delete does not.
My request would be to allow Delete to take an argument (perhaps an optional one if you can't break compatibility).

Do you mean file.Delete? The idea behind that was to keep everything centric to the current "file" which is what file is scoped to. stencil functions are meant to be not scoped to a single file, and thus RemoveAll, and Exists are set there.
That being said, I can't really think of something better than the current file.Delete paradigm, or just using stencil.RemoveAll instead.

Whoops, yes I meant file.Delete and didn't even think of just using stencil.RemoveAll directly...I was thinking it was only for directories.  That basically does what I want.   Thanks Jared.


Handle housekeeping with shared clusters

What would you like to be added:
We recently introduced cluster mode based on the shard key. This shard key is used to assign APIs requested by clients to servers in the cluster based on project or document.
Housekeeping operations should also apply this sharded cluster structure. We can think of two options:

Only the master server executes housekeeping in the cluster
Split housekeeping tasks per project and execute them on servers mapped by project

Why is this needed:

Bugs may occur in housekeeping of sharded clusters.

The options that you've suggested look good to me.
I have listed some of the considerations for these options below.

Only the master server executes housekeeping in the cluster.

Implementation: Using etcd with the raft leader election algorithm will ease this implementation. We can use K8s's etcd for this operation: https://github.com/travisjeffery/kube-leaderelection.
Pros: It looks easy to implement.
Cons: As workload for housekeeping increases, housekeeping overhead on the master server will increase, which may lead to server panic or failure.

Split housekeeping tasks per project and execute them on servers mapped by project.

Implementation: We need to map the server with the project range to housekeeping in etcd and pull this information before housekeeping. Maybe we can reference Yorkie's etcd implementation in the sync package for this task: https://github.com/yorkie-team/yorkie/blob/fdd62a95e802073584e861e1e305f93b24a53d8b/design/cluster-mode.md
Pros: We can distribute housekeeping workload just like document workload sharding, which will ease the overhead of servers to handle housekeeping.
Cons: Implementation seems to be more difficult than the first option.

These options seem like opposite options to me, but I think they both can be implemented by using etcd.
I'll keep searching for more information on these options.

I have implemented leader election with K8s lease object, but this will make yorkie k8s dependent.
Therefore, I will change this implementation with MongoDB, where yorkie already have dependency on.
To be specific about the implementation, I will use the way of nodes try to update on the same document(same _id) by checking lease_expire_at field.
Also, to ensure that only one node becomes the leader when a lease has expired, I will use TTL indexes to remove document on expiry, preventing nodes to update the same document simultaneously.

No, it's still on hold. During housekeeping, it seems that Client Deactivation competing across nodes doesn't cause significant issues, so we haven't proceeded with this.


Visual Studio Code finds TypeScript errors in the TypeScript source itself

git clone https://github.com/Microsoft/TypeScript.git
Code TypeScript &
Code reports 41 errors.

Example code it doesn't like is the "string" in the below.
Perhaps it is a TypeScript version issue?
How can I update the TypeScript version?
    /* @internal */
    export interface CommandLineOptionBase {
        name: string;
        type: "string" | "number" | "boolean" | Map<number>;    // a value of a primitive type, or an object literal mapping named values to actual values
        isFilePath?: boolean;                                   // True if option value is a path or fileName
        shortName?: string;                                     // A short mnemonic for convenience - for instance, 'h' can be used in place of 'help'
        description?: DiagnosticMessage;                        // The message describing what the command line switch does
        paramType?: DiagnosticMessage;                          // The name to be used for a non-boolean option's parameter
        experimental?: boolean;
    }

Code version:
Version 0.10.6
Commit a80232bbcfe8a5cdad1ebc98638673d9dcb02458
Date 2015-12-19T11:26:42.359Z
Shell 0.34.5
Renderer 45.0.2454.85
Node 4.1.1

The problem here is that VSCode 0.10.6 ships with typescript 1.7.5 but master of the repository already uses features coming in 1.8.0. You can control the tsc version to us in VSCode using the typescript.tsdk setting (see https://code.visualstudio.com/updates/vNovember)

Thanks!
In case anyone else finds this issue later, I'll note you also need to run
jake generate-diagnostics

to generate the file needed to resolve the Diagnostics symbol.
With that in place, I only see warnings in the Jakefile.js that are likely out of scope for this bug.


Terminal Issue in VSCode WSL fish shell

I installed Ubuntu WSL. and Installed fish shell in it. But I am getting this in VS code terminal .

I typed hello but it shows ello and also it shows a h infront of coder@LAKSHIT (completely unrelated to command hello) . It just not showing things correctly in vscode terminal. In powershell, or in cmd

Does this issue occur when all extensions are disabled?: No

VS Code Version: 1.62.0
OS Version: Win 10

Steps to Reproduce:

Just install wsl ubuntu. open vscode terminal install fish. And see yourself.

can't reproduce looks similar to #122676


[url] Add API version into URL request

Older versions of vspd have the API version into the request, for example: /api/v1/getpurchaseinfo, but right now this new version does not have it, example: /api/vspinfo.
We should call it /api/v3/vspinfo to keep consistency.

Agree that vspd API routes should include a version number, however I don't think we should start at v3. vspd is a new piece of software and its API should start at v1. This is not version 3 of the dcrstakepool API, its version 1 of the vspd API.
This highlights that we need to consider how we distinguish dcrstakepool and vspd in dcrwebapi -
https://api.decred.org/?c=gsd
However that is a seperate problem

related: decred/dcrwebapi#98


[move-vm-types]: remove dependency on libra-types

Motivation
Remove dependency on libra-types from move-vm-types.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
No test needed.
Related PRs
None.

const MAX_TRANSACTION_SIZE_IN_BYTES is moved from libra-types to nearby of calculate_intrinsic_gas in gas_schedule.
The const is used to check txn's size.
I'm not sure whether we should use max_transaction_size_in_bytes of GasConstants arguments and remove the const once and for all.

:exclamation: before this PR can be merged please make sure that you enable "Allow edits from maintainers".
This is needed for tooling to be able to update this PR in-place so that Github can properly recognize and mark it as merged once its merged into the upstream branch

:exclamation: failed to update PR in-place; halting merge.
Make sure that that "Allow edits from maintainers" is enabled before attempting to reland this PR.

@tzakian  thanks.  The bot failed to land, it asked my to allow edit from maintainers, which I think is a missing feature for forks from organizations. See https://github.com/isaacs/github/issues/1681 and https://github.community/t/how-can-we-enable-allow-edits-from-maintainers-by-default/2847/8. Didn't know this.
So I'm going to fire another PR from my own fork, and close this.

New PR is here: https://github.com/libra/libra/pull/6277


fix(iodredis): change incompatible syntax with nodejs 16

:tada: This PR is included in version 7.7.1 :tada:
The release is available on:

v7.7.1
GitHub release

Your semantic-release bot :package::rocket:


The next VIDEO in the playlist does not load.

I don't know what is happening but: when i'm in a playlist even if the video finish to play or i press next button or if I select another video from the playlist the video doesn't load, The page is loading, it says that it is the next video the title is the comments are even in the playlist says that it is the next one but when I press play is the same old one even the progress bar remains at the same point, i need to refresh every time the page, otherwise nothing happens do you know what causes this ???

Just a hunch, but try disabling SPF in YTC settings.

Thanks man it works :) finally


installing pyq on redhat 2.7 with anaconda 5.0

I tried to install pyq with python 3.6 under anaconda 5.0.  I got the following error:
Processing /apps/download/pyq-master
    Complete output from command python setup.py egg_info:
    running egg_info
    creating pip-egg-info/pyq.egg-info
    writing pip-egg-info/pyq.egg-info/PKG-INFO
    writing dependency_links to pip-egg-info/pyq.egg-info/dependency_links.txt
    writing requirements to pip-egg-info/pyq.egg-info/requires.txt
    writing top-level names to pip-egg-info/pyq.egg-info/top_level.txt
    writing manifest file 'pip-egg-info/pyq.egg-info/SOURCES.txt'
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-29sp_qa8-build/setup.py", line 658, in <module>
        run_setup(METADATA)
      File "/tmp/pip-29sp_qa8-build/setup.py", line 654, in run_setup
        setup(**keywords)
      File "/apps/anaconda3/envs/env_py3/lib/python3.6/distutils/core.py", line 148, in setup
        dist.run_commands()
      File "/apps/anaconda3/envs/env_py3/lib/python3.6/distutils/dist.py", line 955, in run_commands
        self.run_command(cmd)
      File "/apps/anaconda3/envs/env_py3/lib/python3.6/distutils/dist.py", line 974, in run_command
        cmd_obj.run()
      File "/apps/anaconda3/envs/env_py3/lib/python3.6/site-packages/setuptools/command/egg_info.py", line 280, in run
        self.find_sources()
      File "/apps/anaconda3/envs/env_py3/lib/python3.6/site-packages/setuptools/command/egg_info.py", line 295, in find_sources
        mm.run()
      File "/apps/anaconda3/envs/env_py3/lib/python3.6/site-packages/setuptools/command/egg_info.py", line 526, in run
        self.add_defaults()
      File "/apps/anaconda3/envs/env_py3/lib/python3.6/site-packages/setuptools/command/egg_info.py", line 562, in add_defaults
        sdist.add_defaults(self)
      File "/apps/anaconda3/envs/env_py3/lib/python3.6/site-packages/setuptools/command/py36compat.py", line 36, in add_defaults
        self._add_defaults_ext()
      File "/apps/anaconda3/envs/env_py3/lib/python3.6/site-packages/setuptools/command/py36compat.py", line 119, in _add_defaults_ext
        build_ext = self.get_finalized_command('build_ext')
      File "/apps/anaconda3/envs/env_py3/lib/python3.6/distutils/cmd.py", line 299, in get_finalized_command
        cmd_obj.ensure_finalized()
      File "/apps/anaconda3/envs/env_py3/lib/python3.6/distutils/cmd.py", line 107, in ensure_finalized
        self.finalize_options()
      File "/tmp/pip-29sp_qa8-build/setup.py", line 449, in finalize_options
        ('define', 'define'))
      File "/apps/anaconda3/envs/env_py3/lib/python3.6/distutils/cmd.py", line 287, in set_undefined_options
        src_cmd_obj.ensure_finalized()
      File "/apps/anaconda3/envs/env_py3/lib/python3.6/distutils/cmd.py", line 107, in ensure_finalized
        self.finalize_options()
      File "/tmp/pip-29sp_qa8-build/setup.py", line 384, in finalize_options
        ('q_version', 'q_version'))
      File "/apps/anaconda3/envs/env_py3/lib/python3.6/distutils/cmd.py", line 287, in set_undefined_options
        src_cmd_obj.ensure_finalized()
      File "/apps/anaconda3/envs/env_py3/lib/python3.6/distutils/cmd.py", line 107, in ensure_finalized
        self.finalize_options()
      File "/tmp/pip-29sp_qa8-build/setup.py", line 284, in finalize_options
        self.python_dll = get_python_dll(sys.executable)
      File "/tmp/pip-29sp_qa8-build/setup.py", line 242, in get_python_dll
        raise RuntimeError('no python dll')
    RuntimeError: no python dll

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-29sp_qa8-build/

I took a look at setup.py, looks like pyq installer is looking for python shared library libpython3.6m.so by using ldd  But in this case, the share lib has been statically compiled into python3.6 executable. How do I get around this issue?
/apps/anaconda3/lib$ ldd /apps/anaconda3/bin/python
	linux-vdso.so.1 =>  (0x00007ffd2fd37000)
	libpthread.so.0 => /lib64/libpthread.so.0 (0x00007fb7daf6d000)
	libc.so.6 => /lib64/libc.so.6 (0x00007fb7dabab000)
	libdl.so.2 => /lib64/libdl.so.2 (0x00007fb7da9a7000)
	libutil.so.1 => /lib64/libutil.so.1 (0x00007fb7da7a4000)
	librt.so.1 => /lib64/librt.so.1 (0x00007fb7da59b000)
	libm.so.6 => /lib64/libm.so.6 (0x00007fb7da299000)
	/lib64/ld-linux-x86-64.so.2 (0x00007fb7db6ea000)

You might want to try code changes from #29.

We are working on improving the situation for conda users.  Meanwhile, the workaround is to add a setup.cfg file to the root of the source tree with the following contents:
[config]
python_dll=libpython3.6m.so

See Installing from source code for details on how to obtain PyQ source code.

I tried to install via source by having this setup.cfg. But I got some other error message
[root@MiWiFi-R2D-srv ~]# python
Python 3.6.3 |Anaconda, Inc.| (default, Oct 13 2017, 14:21:34)
[GCC 7.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import pyq
Traceback (most recent call last):
  File "/root/miniconda3/lib/python3.6/site-packages/pyq/__init__.py", line 22, in <module>
    from ._k import K as _K, error as kerr, Q_VERSION, Q_DATE, Q_OS
ImportError: /root/miniconda3/lib/python3.6/site-packages/pyq/_k.cpython-36m-i386-linux-gnu.so: undefined symbol: dot

I'm using Miniconda3 (x86) on CentOS 6.9 (x64). kdb+ binary is 32 bit.

This is odd.  What version of kdb+ do you use?  What do you see when you enter the following command?
nm $QHOME/l32/q | grep dot

Please note that we recently started using Miniconda in the Travis CI builds.  You can follow the steps from the log or from .travis.yml.

@abalkin I understand the whole error message now.
ImportError: Importing pyq from stock python is not supported. Use pyq executable.

So if I run pyq, then it works.

Starting with release, 4.1.4 use
conda install -c kx -c enlnt pyq

to install both kdb+ and PyQ.