Updated ML.NET Guide topic for SEO Report Items

Updated ML.NET Guide topic for SEO Report Items during hackadoc

Thanks, @rpetrusha!


[err-msg] Expected state running but got [error | stopped | something-else ] ...

Rancher versions:
1.6.5+
Symptom:
This error can occur whenever a container fails to start. It can be due to a failed docker pull, but also when a container exits immediately either due to runtime failure or improper container image.
For e.g. Expected state running but got error:
Error response from daemon: error unmarshalling content: unexpected end of JSON input
This error is of little value and only creates confusion.
Expected behavior:
The message should indicate exactly what went wrong. E.g. "container failed to start because the image could not be pulled due to incorrect credentials" or "The container exited immediately upon start. Check the logs for more information."

https://github.com/rancher/cattle/blob/v0.183.45/code/iaas/service-discovery/server/src/main/java/io/cattle/platform/servicediscovery/deployment/impl/unit/DefaultDeploymentUnitInstance.java#L201
Need to check if its possible to retrieve any failure message of the instances from the db and propagate in the message.

Tested with latest build for 1.6 from master:
When trying to deploy a service with invalid image name , user is presented with the following message:
Activating (Container should have been running but is in error state. Check logs for more information.: Error response from daemon: pull access denied for <image-name>, repository does not exist or may require 'docker login')

When trying to deploy a service with image name that requires credentials (and we do not have them configured ) , user is presented with the following message:
 Activating (Container should have been running but is in error state. Check logs for more information.: Error response from daemon: unauthorized: access to the requested resource is not authorized)


Cause Exception is Swallowed when ChannelOperationsHandler#channelRead throws

I had the case that my application throwed an IllegalStateException when ChannelOperationsHandler#channelRead was invoked. The Exception is caught by
catch (Throwable err) {
    Exceptions.throwIfFatal(err);
    exceptionCaught(ctx, err);
    ReferenceCountUtil.safeRelease(msg);
}

(ChannelOperationsHandler:163)
But the Throwable is never printed since ReferenceCountUtil#safeRelease is invoked without passing the original error.
This behavior is unexpected because the Exception that gets thrown in the end does not reflect the original cause but instead an arbitrary Netty error.
I use Netty-common version 4.1.31.Final

@mid-ger Netty does not contain any class Called ChannelOperationsHandler.

I think the issue refers to reactor-netty project. @mld-ger right?

You're absolutely right @kachayev. I got confused by the stacktrace. I'll open an issue there.


FR: migrate to curried // Array

@raveclassic not sure about fold and slice though, I feel like they shouldn't be curried

@gcanti Why not?

@raveclassic because of their typical usage
/** Break an array into its first element and remaining elements */
export const fold = <A, B>(nil: Lazy<B>, cons: (head: A, tail: Array<A>) => B, as: Array<A>): B =>
  as.length === 0 ? nil() : cons(as[0], as.slice(1))

/** Extract a subarray by a start and end index */
export const slice = (start: number, end: number) => <A>(as: Array<A>): Array<A> => as.slice(start, end)

what's the point of currying fold (nil, cons arguments) or slice (start, end arguments)? I mean, I never had a use case for partial application here
The pairs (nil, cons) and (start, end) feel like tangled arguments

It's ok then, I don't disagree


try parse JSON and catch error

if JSON.parse fails it should return {}
maybe you can do someting like
    try {
      return JSON.parse(data);
    }
    catch (error) {
      return {};
    }

in my case the server returns not json if the server is overloaded

I'm not so sure that {} is really reasonable. Maybe null would be more apt, but then still I think that most users would want to know if the parsing failed with a proper error rather than just by receiving null.


version: support post versions

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?
[x] Have you successfully run brew man locally and committed any changes?

Add support for PEP 440 post versions. Relates to https://github.com/Homebrew/brew/pull/8832. In support of https://github.com/Homebrew/homebrew-core/pull/61861 so the version number does not need to be manually specified.

Not sure what to do here. This PR makes the explicit version line in legit.rb (https://github.com/Homebrew/homebrew-core/blob/092220c889944cd078f6a7416f9580b19e34e699/Formula/legit.rb#L7) redundant and causes testing on this change to fail, but changing that line in legit before another tagged brew version probably wouldn't work either.

Not sure what to do here. This PR makes the explicit version line in legit.rb (https://github.com/Homebrew/homebrew-core/blob/092220c889944cd078f6a7416f9580b19e34e699/Formula/legit.rb#L7) redundant and causes testing on this change to fail, but changing that line in legit before another tagged brew version probably wouldn't work either.

I'm not sure what best to do, either. Could just override the required step here and merge the two PRs in quick succession before a tag?

Not sure what to do here.

Add an exception for the formula to the cop and remove it once it is fixed?

I submitted a PR for legit (https://github.com/Homebrew/homebrew-core/pull/62121). Merging both quickly and then tagging should work, but is a little risky. A temporary allowlist for redundant versions until the next tag should work, but the challenge is trying to balance doing it "the right way" without requiring too much throwaway work.

Why is a new tag necessary? homebrew-core CI runs on master anyways, right?

Why is a new tag necessary? homebrew-core CI runs on master anyways, right?

@reitermarkus It'll break installation for anyone installing not yet on master.

Thanks @dtrodrigues!


Failed to establish a new connection: [Errno 110] Connection timed out

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

From 'https://github.com/Azure/azure-cli/issues/9515', I got detailed information and repost it with the appropriate template.
It is regularly executing below Azure CLI commands from Azure Virtual Machine. This command was executed from March 2019, and the error occurred about once or twice a month before 5/19, but the error frequency has recently increased rapidly.
Describe the bug
The following az command fail frequently.
az monitor metrics list --resource /subscriptions/xxxxxxxxxx/resourceGroups/xxxxxxxxxx/providers/Microsoft.Cache/Redis/xxxxxxxxxx --start-time 2019-06-12T07:05:53Z --end-time 2019-06-12T07:06:53Z --metrics cachehits cachemisses cacheRead cacheWrite connectedclients percentProcessorTime evictedkeys expiredkeys getcommands operationsPerSecond serverLoad setcommands totalkeys totalcommandsprocessed usedmemory usedmemoryRss -o table --debug

Error message:
Failed to establish a new connection: [Errno 110] Connection timed out',))

When it specify the debug option and get the details of the error, it looks like an error has occurred in authentication. I would like to improve the occurrence of the error. How can I improve it?
To Reproduce
na
Expected behavior
The az command is executed without error.
Environment summary
Install Method (e.g. pip, interactive script, apt-get, Docker, MSI, edge build)
azure-cli-2.0.31-1.el7.x86_64.rpm
CLI version (az --version)
# az --version
azure-cli (2.0.31)
acr (2.0.23)
acs (2.0.31)
advisor (0.5.1)
appservice (0.1.31)
backup (1.1.1)
batch (3.2.0)
batchai (0.2.0)
billing (0.1.8)
cdn (0.0.14)
cloud (2.0.13)
cognitiveservices (0.1.12)
command-modules-nspkg (2.0.1)
configure (2.0.15)
consumption (0.3.0)
container (0.1.22)
core (2.0.31)
cosmosdb (0.1.20)
dla (0.0.19)
dls (0.0.21)
eventgrid (0.1.12)
eventhubs (0.1.2)
extension (0.0.12)
feedback (2.1.1)
find (0.2.9)
interactive (0.3.19)
iot (0.1.19)
keyvault (2.0.21)
lab (0.0.21)
monitor (0.1.5)
network (2.0.28)
nspkg (3.0.2)
profile (2.0.22)
rdbms (0.2.1)
redis (0.2.12)
reservations (0.1.2)
resource (2.0.27)
role (2.0.22)
servicebus (0.1.2)
servicefabric (0.0.12)
sql (2.0.25)
storage (2.0.31)
vm (2.0.30)

Python location '/usr/lib64/az/bin/python'
Extensions directory '/root/.azure/cliextensions'

Python (Linux) 2.7.5 (default, May  3 2017, 07:55:04)
[GCC 4.8.5 20150623 (Red Hat 4.8.5-14)]

Legal docs and information: aka.ms/AzureCliLegal

OS version
/etc/redhat-release
Red Hat Enterprise Linux Server release 7.4 (Maipo)
Shell Type (e.g. bash, cmd.exe, Bash on Windows)
bash
Additional context
# cat az.sh
date
/bin/az monitor metrics list --resource /subscriptions/xxxxxxxxxx/resourceGroups/xxxxxxxxxx/providers/Microsoft.Cache/Redis/xxxxxxxxxx --start-time 2019-06-12T07:05:53Z --end-time 2019-06-12T07:06:53Z --metrics cachehits cachemisses cacheRead cacheWrite connectedclients percentProcessorTime evictedkeys expiredkeys getcommands operationsPerSecond serverLoad setcommands totalkeys totalcommandsprocessed usedmemory usedmemoryRss -o table --debug
date

# sh -x az.sh
+ date
Wed Jun 12 15:55:54 JST 2019
+ /bin/az monitor metrics list --resource /subscriptions/xxxxxxxxxx/resourceGroups/xxxxxxxxxx/providers/Microsoft.Cache/Redis/xxxxxxxxxx --start-time 2019-06-12T07:05:53Z --end-time 2019-06-12T07:06:53Z --metrics cachehits cachemisses cacheRead cacheWrite connectedclients percentProcessorTime evictedkeys expiredkeys getcommands operationsPerSecond serverLoad setcommands totalkeys totalcommandsprocessed usedmemory usedmemoryRss -o table --debug
Command arguments: ['monitor', 'metrics', 'list', '--resource', '/subscriptions/xxxxxxxxxx/resourceGroups/xxxxxxxxxx/providers/Microsoft.Cache/Redis/xxxxxxxxxx', '--start-time', '2019-06-12T07:05:53Z', '--end-time', '2019-06-12T07:06:53Z', '--metrics', 'cachehits', 'cachemisses', 'cacheRead', 'cacheWrite', 'connectedclients', 'percentProcessorTime', 'evictedkeys', 'expiredkeys', 'getcommands', 'operationsPerSecond', 'serverLoad', 'setcommands', 'totalkeys', 'totalcommandsprocessed', 'usedmemory', 'usedmemoryRss', '-o', 'table', '--debug']
Event: Cli.PreExecute []
Event: CommandParser.OnGlobalArgumentsCreate [<function on_global_arguments at 0x172df50>, <function on_global_arguments at 0x1824aa0>, <function on_global_arguments at 0x18361b8>]
Event: CommandInvoker.OnPreCommandTableCreate [<function remove_logger_flags at 0x172e050>]
Installed command modules ['acr', 'acs', 'advisor', 'appservice', 'backup', 'batch', 'batchai', 'billing', 'cdn', 'cloud', 'cognitiveservices', 'configure', 'consumption', 'container', 'cosmosdb', 'dla', 'dls', 'eventgrid', 'eventhubs', 'extension', 'feedback', 'find', 'interactive', 'iot', 'keyvault', 'lab', 'monitor', 'network', 'profile', 'rdbms', 'redis', 'reservations', 'resource', 'role', 'servicebus', 'servicefabric', 'sql', 'storage', 'vm']
Loaded module 'acr' in 0.002 seconds.
Loaded module 'acs' in 0.002 seconds.
Loaded module 'advisor' in 0.001 seconds.
Loaded module 'appservice' in 0.004 seconds.
Loaded module 'backup' in 0.003 seconds.
Event: CommandLoader.OnLoadCommandTable []
Loaded module 'batch' in 0.007 seconds.
Loaded module 'batchai' in 0.001 seconds.
Loaded module 'billing' in 0.001 seconds.
Event: CommandLoader.OnLoadCommandTable []
Loaded module 'cdn' in 0.002 seconds.
Loaded module 'cloud' in 0.001 seconds.
Loaded module 'cognitiveservices' in 0.001 seconds.
Loaded module 'configure' in 0.001 seconds.
Loaded module 'consumption' in 0.001 seconds.
Loaded module 'container' in 0.001 seconds.
Loaded module 'cosmosdb' in 0.002 seconds.
Loaded module 'dla' in 0.003 seconds.
Loaded module 'dls' in 0.002 seconds.
Loaded module 'eventgrid' in 0.001 seconds.
Loaded module 'eventhubs' in 0.003 seconds.
Loaded module 'extension' in 0.001 seconds.
Loaded module 'feedback' in 0.000 seconds.
Loaded module 'find' in 0.001 seconds.
Loaded module 'interactive' in 0.000 seconds.
Loaded module 'iot' in 0.003 seconds.
Loaded module 'keyvault' in 0.003 seconds.
Loaded module 'lab' in 0.002 seconds.
Loaded module 'monitor' in 0.003 seconds.
Loaded module 'network' in 0.018 seconds.
Loaded module 'profile' in 0.002 seconds.
Loaded module 'rdbms' in 0.002 seconds.
Loaded module 'redis' in 0.001 seconds.
Loaded module 'reservations' in 0.001 seconds.
Loaded module 'resource' in 0.005 seconds.
Loaded module 'role' in 0.002 seconds.
Loaded module 'servicebus' in 0.004 seconds.
Loaded module 'servicefabric' in 0.001 seconds.
Loaded module 'sql' in 0.003 seconds.
Event: CommandLoader.OnLoadCommandTable []
Loaded module 'storage' in 0.020 seconds.
Loaded module 'vm' in 0.010 seconds.
Loaded all modules in 0.124 seconds. (note: there's always an overhead with the first module loaded)
Extensions directory: '/root/.azure/cliextensions'
Event: CommandInvoker.OnPreCommandTableTruncate []
Event: CommandInvoker.OnPostCommandTableCreate [<function add_id_parameters at 0x1eb7cf8>]
Event: CommandInvoker.OnCommandTableLoaded []
Event: CommandInvoker.OnPreParseArgs [<function _documentdb_deprecate at 0x213f668>]
Event: CommandInvoker.OnPostParseArgs [<function handle_output_argument at 0x1824b18>, <function handle_query_parameter at 0x1836230>, <function handler at 0x228ca28>]
Getting management service client client_type=MonitorManagementClient
attempting to read file /root/.azure/accessTokens.json as utf-8-sig
adal-python : 5f877713-0009-4ae3-a506-e811d330095a - Authority:Performing instance discovery: ...
adal-python : 5f877713-0009-4ae3-a506-e811d330095a - Authority:Performing static instance discovery
adal-python : 5f877713-0009-4ae3-a506-e811d330095a - Authority:Authority validated via static instance discovery
adal-python : 5f877713-0009-4ae3-a506-e811d330095a - TokenRequest:Getting token with client credentials.
adal-python : 5f877713-0009-4ae3-a506-e811d330095a - TokenRequest:No user_id passed for cache query
adal-python : 5f877713-0009-4ae3-a506-e811d330095a - CacheDriver:finding with query keys: {'_clientId': '...'}
adal-python : 5f877713-0009-4ae3-a506-e811d330095a - CacheDriver:Looking for potential cache entries: {'_clientId': '...'}
adal-python : 5f877713-0009-4ae3-a506-e811d330095a - CacheDriver:Found 0 potential entries.
urllib3.connectionpool : Starting new HTTPS connection (1): login.microsoftonline.com
adal-python : 5f877713-0009-4ae3-a506-e811d330095a - OAuth2Client:Get Token request failed
Traceback (most recent call last):
  File "/usr/lib64/az/lib/python2.7/site-packages/adal/oauth2_client.py", line 267, in get_token
    timeout=self._call_context.get('timeout', None))
  File "/usr/lib64/az/lib/python2.7/site-packages/requests/api.py", line 112, in post
    return request('post', url, data=data, json=json, **kwargs)
  File "/usr/lib64/az/lib/python2.7/site-packages/requests/api.py", line 58, in request
    return session.request(method=method, url=url, **kwargs)
  File "/usr/lib64/az/lib/python2.7/site-packages/requests/sessions.py", line 508, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/lib64/az/lib/python2.7/site-packages/requests/sessions.py", line 618, in send
    r = adapter.send(request, **kwargs)
  File "/usr/lib64/az/lib/python2.7/site-packages/requests/adapters.py", line 508, in send
    raise ConnectionError(e, request=request)
ConnectionError: HTTPSConnectionPool(host='login.microsoftonline.com', port=443): Max retries exceeded with url: /e546ca04-20ab-465c-9378-592ab55daafd/oauth2/token (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x39383d0>: Failed to establish a new connection: [Errno 110] Connection timed out',))
Please ensure you have network connection. Error detail: HTTPSConnectionPool(host='login.microsoftonline.com', port=443): Max retries exceeded with url: /e546ca04-20ab-465c-9378-592ab55daafd/oauth2/token (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x39383d0>: Failed to establish a new connection: [Errno 110] Connection timed out',))
+ date
Wed Jun 12 16:12:52 JST 2019

You are running an extremely out-of-date version of the CLI. You have 2.0.31 and the current release is 2.0.66. Please update and reopen if this is still a problem.


Allow internal LB and choice for egress

Fixes #57
Fixes #82
NOTE: This release forces creation of new template and thus Vault nodes will need to be replaced.

Add new variable for choosing if you want external LB created or not. Default to external to ease migration burden
Add ability for users to specify an http_proxy such as Squid and choose not to create a NAT.

@sethvargo is there anything else that needs to be done before we merge and release this?

Could you split this into two PRs, one for the internal/external load balancer option, and one for the http proxy/nat option? I think it will make reviewing a bit easier.

@lbordowitz Thanks for the comments. These seem like reasonable additions to the PR so I added one more commit. However since this PR has already been reviewed and approved in its current state, it doesn't make much sense to break it into multiple PRs at this point.
@sethvargo would you mind reviewing that latest commit and merging when you have a moment?

Thanks for pointing that out. I had upgraded the provider locally but didn't put it in the versions file

Good catch. I was looking at the wrong green check in the console when I tested.


contentLoaded fire before content fill

done fire after always

a new callback onContentReady is introducted in v3. it is called after the content is rendered in DOM


[RFC] Vudu 2.0, smaller pkg size and simpler api

January 20, 2019
Summary
In this RFC, I’m proposing a modified approach to writing styles in JS with Vudu.
See an associated PR 👉🏼 https://github.com/dhunninghake/vudu/pull/23
Vudu 2.0...

Is a smaller package size — 7.2kb vs the current 18.6kb.
Takes a less opinionated approach to writing and composing JS objects.
Exposes a simpler API for creating and interacting with styles.

Basic example
The main difference is that objects passed to vudu do not need to be named. Like the previous version, it will generate a random class name and return it after appending the styles:
import vudu from 'vudu';

export const Example = () =>
;

const className = vudu({
  color: 'red',
  ':hover': {
    color: 'green',
  },
});

It is also possible to achieve a deterministic class name by passing a string as the first argument and currying your style object:
import vudu from 'vudu';

export const Example = () =>
;

const className = vudu('.hi')({
  color: 'red',
  ':hover': {
    color: 'green',
  },
});

You can pass multiple objects and get an array of classes back:
import vudu from 'vudu';

export const Example = () => (
  <>

  </>
);

const [c1, c2] = vudu({ color: 'red' }, { color: 'blue' });

Motivation
A few observations about the package after using it for some time:

Naming each style object is cumbersome - previously it was required that any object passed to vudu needed a key. Over time this became a nuisance, especially when I just wanted styles generated without having to name them.

const className = vudu({
  container: {               // <---- this
    color: 'red',
  },
});

Composes keyword is unnecessary with es6 - one of the key concepts with vudu 1.0 was @composes. This was an array of pre-generated objects that could be imported and used in components. Instead of parsing this array, I've been using the spread operator to compose smaller atomic style objects.
The package src is hard to follow - I was the one who wrote the initial version of vudu, and yet every time I come back to make updates or optimizations, I think "how the f does this work". So I started a rewrite one day and found that I could get nearly the same functionality with way less code. Now, nearly all logic takes place in a single recursive function.

Detailed design
Breaking changes
The following features have been removed in Vudu 2.0, and the reasons why:

v.composes - Support for this monolithic object of atomic styles is being removed to keep the package lean. This functionality can easily be achieved by writing a thin wrapper around vudu.
@composes - As mentioned before, this feature can be achieved with simply spreading cached atomic style objects into an object passed to vudu.
@font-face - Setting up typefaces is mostly a static operation. Do it once at the top of the app and then leave it alone. Because of this, a static stylesheet for fonts seems like a better option.
@keyframes - Generating keyframes with JS can be very dynamic and powerful, but I rarely ever used this feature. I might add support for this feature back in, but still on the fence.
Custom style sheet - Only included this feature for testing purposes. Vudu 2.0 takes a different approach to testing.

Stayed the same

Random class names - Vudu continues to generate and return a random class names for unnamed style objects passed to it.
Vendor prefixing - The inline-style-prefixer package accounts for 5.8kb of the 7.2kb vudu is now becoming. It’s a lot, but I consider it a worthy cost in order to build robust components which stand up to varying browser css feature support.

New additions

Deterministic class names - This means it’s possible to assign a class name to style objects. Additionally, this opens the door to setting css for global elements.
New methods - .reset() and .css() are now available to empty the current stylesheet or read from it.

Migration Path
A look at how to move forward regarding the breaking changes above:
v.composes
Currently the only way to get this monolithic object would be to write a wrapper around Vudu 2.0. It would be pretty simple. However, although this functionality is being removed from the base package, we could consider a secondary package in the repo that offers support for this object. That way the base stays lean while not removing this altogether. It might look something like:
import composes from 'vudu/composes';

@composes
This one is easy, it can be achieved with es6.
Before
const className = v({
  container: {
    @composes: [red, pt2]
  }
});

After
const className = vudu({
  ...red,
  ...pt2,
});

@font-face
Create a static stylesheet with font declarations.
@keyframes
Similar to font face, these can be added to a static sheet. It’s possible support for this will be added back in.
Custom style sheet
Put all non-vudu styles in their own stylesheet. This feature was only meant for testing purposes.
Looking ahead
One goal I want to keep in mind with vudu is that it continues to offer an option for inline styles with JS outside the context of a modern framework. Whether your build is in React, Vue, Ember, the next hot JS framework, or plain old JS, I want it to be flexible enough to adapt to your use case. Any refinements to the inner workings or the API in the future will be in the service of making it faster and more accessible to more developers.
<3

@hhff @sepowitz ready for another 👀 whenever you two find some time! I'm curious about a couple things from your perspective:

How difficult would it be to upgrade the apps you've got using vudu 1.x to 2.0? Is there anything that can be changed/added to make that easier?
Are there use cases for vudu that it doesn't yet support? Seb mentioned a couple in the rewrite PR, but I'd love to know more and come up with a plan to support them.

Hi big D! We had a bunch of sites go down due to a CMS provider deprecating, so I've been timepoor lately.
Will review ASAP!

@hhff @sepowitz Question for you guys: do you use snapshot tests with your components? I personally don't love snaps for many reasons, but in a case like upgrading apps from vudu v1 to v2 you could throw some snapshot tests in there and make sure nothing looks too crazy before merging the upgrade. Would love to know if there's anything blocking on your side (code concerns or otherwise) for me merging this new version!


Add an option to disable the device screen

Hy, thanks for your great work.
Is it possible to add a command line argument, that disable the device screen (or set it to the lowest brightness)? It is possible with tasker, but would be nice if your app can do it automatically.
I usee your app to control my device with my computer and so I don't need the device screen, and my OLED Screen can burn in, if it shows the same screen for hours.

Hi,
I implemented it few days ago on dev branch, see #175.
It will be available in the next release.


Can I go over directory instead of playbook?

Usage calls for a specific playbook. How can I go over all playbooks in dir instead?

It will now go over a role directory. The problem with a playbook directory is that not everything in it might be a playbook (e.g. we tend to store rolesfiles in the same directory).
It might well be easier to use find and xargs, but implementations that include --exclude filters will be considered (probably best to suggest what an implementation might look like before implementing it though)

If run from bash, doing something like ansible-linter *.yml should work, right?

@ssbarnea apart from rolesfile.yml, sure.

I don't like that the fact that doing this generates lots of new lines which are not desired. It seems that ansible-lint adds an extra empty line to the output.

That sounds like a separate bug.

find . -type f -name "*.yml" -exec /usr/local/bin/ansible-lint -p -q {}  Works well enough usually - not sure this actually needs fixing.

Obsolete, the linter can already auto-detect repository layout or scan folders. It is no longer needed to use find.


conversionType: "object" option does not work anymore in new library version (2.0.0)

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[X] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior
In my projects I am importing the images as an object with the following config as it is in this article and in the previous docs:
"svg-to-ts": {
    "conversionType": "object",
    "srcFiles": [
      "libs/shared/icons-lib/src/lib/icons/*.svg"
    ],
    "outputDirectory": "libs/shared/icons-lib/src/lib",
    "fileName": "svg-icons",
    "svgoConfig": {
      "plugins": [
        {
          "removeDimensions": true,
          "cleanupAttrs": true
        }
      ]
    }
  }

This generates the following svg-icons.ts file:
/* 🤖 this file was generated by svg-to-ts*/
export default {
  checkmark:
    '<svg fill="currentColor" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 48 48"><path fill-rule="evenodd" clip-rule="evenodd" d="M24 44c11.046 0 20-8.954 20-20S35.046 4 24 4 4 12.954 4 24s8.954 20 20 20zm-1.895-11.667c-.505 0-.992-.234-1.31-.631l-4.788-6.025c-.56-.703-.411-1.695.318-2.236a1.688 1.688 0 012.32.306l3.347 4.24c2.133-3.14 6.453-9.272 9.202-11.87a1.715 1.715 0 012.338.019 1.57 1.57 0 01-.018 2.254c-3.442 3.247-9.95 13.114-10.025 13.204-.28.433-.786.721-1.328.74h-.056z"/></svg>',
  done:
    '<svg xmlns="http://www.w3.org/2000/svg" fill="currentColor" viewBox="0 0 24 24"><path d="M0 0h24v24H0z" fill="none"/><path d="M9 16.2L4.8 12l-1.4 1.4L9 19 21 7l-1.4-1.4L9 16.2z"/></svg>'

However I then have issues while registering the SVGIconsModule, since the expected icons type is SvgIconType | SvgIconType[] and this brings a breaking change in my current projects:
import { SvgIconsModule } from '@ngneat/svg-icon';
import * as icons from './svg-icons';

@NgModule({
  imports: [
    CommonModule,
    TranslateModule,
    SvgIconsModule.forRoot({
        icons
    })]
})

As a different type is expected:
Type 'typeof import("/Users/.../libs/shared/icons-lib/src/lib/svg-icons")' is not assignable to type 'SvgIconType | SvgIconType[] | undefined'.

Expected behavior
I would like to be able to keep using     "conversionType": "object", as until version 1.0.3
Minimal reproduction of the problem with instructions
Create a brand new Angular project with at least one svg icon and set the config as described above, using     "conversionType": "object"
What is the motivation / use case for changing the behavior?
It seems to be a breaking change compared to the previous version.
Environment

Angular version: 11.0.0

Browser:
- [ ] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [X] Chrome (iOS) version 86.0.4240.183
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:
- Node version: 12.18.2
- Platform:  Mac

We released version 2, and we focused more on lazy load icons in this version. You can continue to use v1 without any worries.

If you want to use version 2, you can change the conversionType to constants or files

What would be the best approach in case of a NX worspace with multiple apps that share on average 20-30 icons, plus some further one specific to each project.
In order to use the lazy load feature, we could import all common icons in a shared module and then the specific ones individually. Would this be the right approach in order to repeat importing the same common icons in every app or would you suggest a better way?

Sounds good


Option to not run build runner after generating JSON to Dart

This extension is awesome and thanks for developing this. I don't like to run build runner instantly for a single time. I prefer to run build runner with watch mode which I am manually doing while working with freezed. So it will be great to have a option to disable build runner after generating JSON to Dart.

@2shrestha22 Hi 👋
Thank you for your suggestion. It is easy to fix. What do you think maybe it's better allow choose run or watch builder in the settings instead to disable it. Or disabling it is the best option. I do not work with Freezed it therefore I ask you what do you think about it.

@iamarnas Currently when new freezed data class is created builder runner, opens a new terminal and builds. So I think completely disabling build runner will be a good option because everyone using freezed must have known build runner too. 😀

@iamarnas Currently when new freezed data class is created builder runner, opens a new terminal and builds. So I think completely disabling build runner will be a good option because everyone using freezed must have known build runner too. 😀

@2shrestha22 OK. Right now I'm on vacation. I fix it when I find free time.

@iamarnas ok thank you

@2shrestha22 Hi again!
I added option to disable build runner in the Settings/Extensions/JSON To Dart Model. Let me know if it works as you expected. Thank you.

Which option should we use to disable build runner? I have 4 but it is still running build runner in terminal.

@2shrestha22 Hi 👋

Which option should we use to disable build runner? I have 4 but it is still running build runner in terminal.

Have you latest version 3.2.5?

Yes I have 3.2.5

@2shrestha22 Then you would see the new option Run Builder.

@iamarnas Let me try again. And I will inform you.

Its working now, found new setting as you described. I will suggest a new feature. What about option in Explorer to create model class there?

@2shrestha22 Thanks for suggestion.

I will suggest a new feature. What about option in Explorer to create model class there?

This feature should work same as from the clipboard or from the file models just would pick faster target directory. If you setup JSON Dart To Model configuration in the settings and run fast mode. You can generate JSON string in few seconds by using hot keys. Tips: Try hot keys :)


Is it possible to apply conditional formatting for columns greater than or equal to another columns value?

I have an excel with a "Used" column and a "Total" column. Ideally, I would like to apply conditional formatting (Turning the row red) whenever the value in the "Used" column reaches 90% of the value of the "Total" column. For now, I would be happy with having the row turn red when the value in the "Used" column is greater than or equal to the value in the "Total" column. Is this possible?

@aleoyakas Do you know if this can be done in Excel itself with conditional formatting?

@dfinke It is possible. By applying a formula, for example: $A1:$A1000 > ($B1:$B1000) * 0.9

Add-ConditionalFormatting -WorkSheet $Excel.Workbook.Worksheets[1] -Range "B2:B500" -BackgroundColor "Red" -RuleType GreaterThan -ConditionValue "A2"

@Poolshark314 this will add conditional formatting to the B column, and it will only highlight cells in column B if the condition is met.
Also, if any of column B are greater than the A2 cell, then the condition will be met - if B56 is less than A56, but greater than A2, the cell will still be highlighted red.

Sorry didn't see you wanted the full row highlighted.  How about:
Add-ConditionalFormatting -WorkSheet $excel.Workbook.worksheets[1] -Range "$A2:$C16" -BackgroundColor "Red" -RuleType Expression -ConditionValue "=$C2>$B2"
My sheet then looks like this:

and the rule:

Also for your 90% condition:
Add-ConditionalFormatting -WorkSheet $excel.Workbook.worksheets[1] -Range "$A2:$C16" -BackgroundColor "Yellow" -RuleType Expression -ConditionValue "=$C2>($B2*.9)"


update IMAS compat

untested, but this came up as I was updating the compats for other packages

Doing this in https://github.com/ProjectTorreyPines/FusionSyntheticDiagnostics.jl/pull/44
I had removed IMAS from deps, so it needed to come back in.


Enable IMDSv2 query for getAvailabilityZoneImpl

IMDSv2 uses session-oriented requests to mitigate several types of vulnerabilities that could be used to attempt to access the IMDS, protecting against malicious activities such as SSRF attacks. The change does not impact existing usage of "getAvailabilityZoneImpl" call.
Before:
➜  ~ curl --silent --max-time 10 --connect-timeout 5 http://<IP_ADDRESS>/latest/meta-data/placement/availability-zone
us-east-1a%

After:
➜  ~ TOKEN=$(curl -X PUT -H "X-aws-ec2-metadata-token-ttl-seconds: 300" http://<IP_ADDRESS>/latest/api/token 2> /dev/null) && curl --silent --max-time 10 --connect-timeout 5 -H "X-aws-ec2-metadata-token: $TOKEN" http://<IP_ADDRESS>/latest/meta-data/placement/availability-zone
us-east-1a%

No description provided.

hi @xia0pin9 can you add description on why you are making this change, and what you have done to test/verify it

No description provided.

hi @xia0pin9 can you add description on why you are making this change, and what you have done to test/verify it

updated

The change does not impact existing usage of "getAvailabilityZoneImpl" call.

Just to confirm: what happens if the curl command to fetch token fails? will that cause the entire command to fail and hence failing to retrieve AZ?
Additionally, I would like to see test plan for this change. E.g. the command output for the new command you listed in the description. And ideally also a private build: you can follow the instruction in https://w.pinadmin.com/display/KVSYS/Rocksplicator+Release+Guide#RocksplicatorReleaseGuide-DevTesting(PrivateBuild) to produce a private build, and ask in #kv-systems team to help with a deployment.

The change does not impact existing usage of "getAvailabilityZoneImpl" call.

Just to confirm: what happens if the curl command to fetch token fails? will that cause the entire command to fail and hence failing to retrieve AZ?
Additionally, I would like to see test plan for this change. E.g. the command output for the new command you listed in the description. And ideally also a private build: you can follow the instruction in https://w.pinadmin.com/display/KVSYS/Rocksplicator+Release+Guide#RocksplicatorReleaseGuide-DevTesting(PrivateBuild) to produce a private build, and ask in #kv-systems team to help with a deployment.

IMDS token endpoint has 1000 TPS (Transaction per Second) per instance, so the token fetch failure should be very rare, it will also break lots of existing services in that case. And if it indeed fail, it will cause the entire command to fail and failing to retrieve AZ info.
The output for both before/after commands are already listed in the description, they are the same (us-east-1a)

The change does not impact existing usage of "getAvailabilityZoneImpl" call.

Just to confirm: what happens if the curl command to fetch token fails? will that cause the entire command to fail and hence failing to retrieve AZ?
Additionally, I would like to see test plan for this change. E.g. the command output for the new command you listed in the description. And ideally also a private build: you can follow the instruction in https://w.pinadmin.com/display/KVSYS/Rocksplicator+Release+Guide#RocksplicatorReleaseGuide-DevTesting(PrivateBuild) to produce a private build, and ask in #kv-systems team to help with a deployment.

IMDS token endpoint has 1000 TPS (Transaction per Second) per instance, so the token fetch failure should be very rare, it will also break lots of existing services in that case. And if it indeed fail, it will cause the entire command to fail and failing to retrieve AZ info.

Ideally we'd make it more resilient by breaking up the commands: fetch the token first, and if that fails, fall back to IMDSV1.
But if we are confident about the reliability of the IMDSV2 calls, I think this simple change is okay.  Note that most of our services uses this call upon start-up, and if the command fails, the process will crash immediately. We'll probably need some retry mechanism as a follow up to make it more resilent -- if we ever observed such failures.

The output for both before/after commands are already listed in the description, they are the same (us-east-1a)
got it. Sorry, missed that earlier.

Some background on this change: https://docs.aws.amazon.com/AWSEC2/latest/UserGuide/configuring-instance-metadata-service.html


Test b00002.vtc fails with Varnish 6.0.4

Trying to build libvmod-geoip2 with Varnish 6.0.4. But test b00002.vtc fails.
Relevant output from test-suite.log:
***  v1    1.1 debug|Debug: Version: varnish-6.0.4 revision 14d4eee6f0afc3020a044341235f54f3bd1449f1
***  v1    1.1 debug|Debug: Platform: Linux,3.10.0-957.27.2.el7.x86_64,x86_64,-junix,-sdefault,-sdefault,-hcritbit
***  v1    1.1 debug|200 327
***  v1    1.1 debug|-----------------------------
***  v1    1.1 debug|Varnish Cache CLI 1.0
***  v1    1.1 debug|-----------------------------
***  v1    1.1 debug|Linux,3.10.0-957.27.2.el7.x86_64,x86_64,-junix,-sdefault,-sdefault,-hcritbit
***  v1    1.1 debug|varnish-6.0.4 revision 14d4eee6f0afc3020a044341235f54f3bd1449f1
***  v1    1.1 debug|
***  v1    1.1 debug|Type 'help' for command list.
***  v1    1.1 debug|Type 'quit' to close CLI session.
***  v1    1.1 debug|Type 'start' to launch worker process.
***  v1    1.1 debug|

...

***  v1    2.9 debug|VCL "vcl1" Failed initialization
***  v1    2.9 debug|Message:
***  v1    2.9 debug|\tObject vo_db2 not initialized
***  v1    2.9 debug|
***  v1    2.9 debug|Debug: Stopping Child
**** v1    3.0 vsl|          0 CLI             - Rd vcl.load "vcl1" vcl_vcl1.1569759457.139654/vgc.so 1auto
**** v1    3.0 vsl|          0 Debug           - geoip2.geoip2: Using maxminddb 1.2.0
**** v1    3.0 vsl|          0 Debug           - geoip2.geoip2: Using maxminddb 1.2.0
**** v1    3.0 vsl|          0 Error           - geoip2.geoip2: Error opening the specified MaxMind DB file
**** v1    3.0 vsl|          0 CLI             - Wr 300 73 VCL "vcl1" Failed initialization
Message:
        Object vo_db2 not initialized

**** v1    3.0 vsl|          0 CLI             - EOF on CLI connection, worker stops

Build with Varnish 6.0.3 works fine, test b00002.vtc is passed.

Adding full test-suite.log

Thanks for the report. Which branch are you using?

I am using branch oldstable.

Thanks. I will take a look and update this ticket.

Can you try using devel?
Now that they have more or less converged I will unify the development in a single branch (probably master) soon.

Using branch devel the build process passes all tests.

Thanks for confirming it.
I've move the support for 6.0, 6.2 and 6.3 to master, and tagged v1.2.1.
Please use that going forward.


repo-header-info shows info for ci actions

Description
Repo-header-info show the stars and lock on to many elements. On the CI actions popover I hav enoticed this.

How to replicate the issue + URL

Goto https://github.com/refined-github/refined-github
Click on the CI checkmark

Extension version
23.9.17
Browser(s) used
Chrome

🥲🥲🥲

I love it 😂


Add all auto tests back with new updates

Description
add all automation tests back and update (homepage, profile manager, twin, transfer, bridge) pages.
Related Issue
https://github.com/threefoldtech/tfgrid-sdk-ts/issues/1285#issuecomment-2054004864
https://github.com/threefoldtech/tfgrid-sdk-ts/pull/1667
Checklist

[ ] Tests included
[ ] Build pass
[ ] Documentation
[ ] Code format and docstrings
[ ] Screenshots/Video attached (needed for UI changes)

@khaledyoussef24 Can you please give it a look?

test_dedicate.py kept failing

Screencast from 04-15-2024 01:08:14 PM.webm

Can we add a section on how to run these tests in the docs?

I updated the docs 893cf3a

test_dedicate.py kept failing

I know I only updated the (home page, profile manager, twin, transfer, bridge) test pages because the PR is already big enough. and I'll create more PRs to update other pages and add new tests.

@A-Harby
What is the status here? if the work still in progress please daft if :)

@A-Harby What is the status here? if the work still in progress please daft if :)

No, the PR is done and waiting for review.

I have updated the four pages xpaths with all the possible xpaths with improved selectors using button names, input labels, or unique classes if found.
All the tests are passing except for test_edit_twin_email which is failing and has been reported at https://github.com/threefoldtech/tfgrid-sdk-ts/issues/2555.
Don't forget to run the latest code locally, and you can use my address for testnet stellar if you want GAK2AN6ZC4REV2GXZPTMJG2JKLRJQX746JNG7ACKNC4RSJE7ETAZSE7D.
I will create another PR for the other pages: farm, node, and dedicate.

@A-Harby What are the ready tests? Also, can you remove any incomplete tests from this PR and change the PR name and description accordingly?

Since there were all the automated tests and its workflow, so I had to add them again, and also updated these tests to run on the new UI.
I also removed the dedicated page and updated docs to have all the needed information.
When you run the tests, you will find some failing tests:

All the farm tests will fail until these issues are fixed: https://github.com/threefoldtech/tfgrid-sdk-ts/issues/2578, https://github.com/threefoldtech/tfgrid-sdk-ts/issues/2610.
The node public config tests will fail until this issue is fixed: https://github.com/threefoldtech/tfgrid-sdk-ts/issues/2637.
Some tests will fail because of this issue: https://github.com/threefoldtech/tfgrid-sdk-ts/issues/2703.

Please fix the actions, if you still working on it lets draft it


Support for beginning of time --since in event

Fixes #11555. This is part of the docker
second birthday courtesy of Seattle.
Signed-off-by: Aaron Torres<EMAIL_ADDRESS>

LGTM

Dup of https://github.com/docker/docker/issues/11736  ?

Dup of https://github.com/docker/docker/pull/11571 ?

It looks like it is a dupe unfortunately.

Sorry for this @agtorre :-( We're gonna go with #11571 for the extra catch and test implementation.


Client Allocation Request for: LeoCheung

Client Allocation Questions
Core Information

Name: LeoCheung
Website / Social Media: https://github.com/LeoZCH
Region: Greater China
DataCap Requested: 10TiB
Addresses to be Notarized: f3v7x4a2aapgx6o2r477tenoin3u5oadaeqyd7kjdsitykvf4ok7vq2utcyh34lmu5u7oybs25ff6s4dbudpma
Notary Requested: IreneYoung

@LeoZCH Please subscribe to notifications for this Issue to be aware of updates. Notaries may request additional information on the Issue.

we have another communication: https://github.com/filecoin-project/filecoin-plus-client-onboarding/issues/1240

It will be appreciated if you can give me 50T DataCap. I really need it for the slingshot repair project. Otherwise, few providers is willing to join it.

@IreneYoung please review it again, I need your help

@LeoZCH Hello, I noticed that you already successfully applied for a large dataset (https://github.com/filecoin-project/filecoin-plus-large-datasets/issues/151) and I also signed it for you. So do you still wanna request this 10TiB DataCap?


⚠ Bofrost.at has degraded performance

In d99246e, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1773 ms

Resolved: Bofrost.at performance has improved in ef48dd8 after 21 minutes.


No data since October 19th at 3PM BST

Describe the bug
My sensors all worked fine as configured for a period of 5 days, but stopped working on October 19th at 3PM. No electricity or gas usage has been reported since.

Reproduction steps
Set up presumably as I have my sensors - however I note others experienced disruption with 13.0.3, my issues started before and in fact updating made no change to me. I also can't comment on the BST others have seen, as my integration broke before this point
Expected behaviour
To see electricity and gas usage tracked as per the previous 5 days, shown below. I'll show the timespan to show things worked on each day:

Tariff Code
1012773505176
Integration Version
13.0.2 and 13.0.3
Home Assistant Version
Core 2024.10.2 with frontend 20241002.3
Fresh Install?
Fresh install
Home Assistant Logs
Logs lost upon update to 13.0.3 - I can get some on request.
Confirmation

[X] I confirm that I cannot find my solution within the documentation
[X] I confirm that I cannot find my solution within the FAQ

Wasn't sure on tariff code if I put the right thing so please let me know if I need to update that.

Hello and sorry that you're having issues. It looks like you're using the home mini to provide data. Is this connected to the meter and internet? Can you see data for today in the app?

Also do you have any errors or warnings in your logs? Can you also download the supply the diagnostics as provided by following these instructions -https://bottlecapdave.github.io/HomeAssistant-OctopusEnergy/faq/#ive-been-asked-for-my-meter-information-in-a-bug-request-how-do-i-obtain-this

Hi - yes I can see the data in the app for every day up until now, so I'm pretty confident the Mini is working.

Downloading diagnostics - here's electric:
octopus_energy-01JBCE112X0FDSB77QPAT8GQXK-Electricity Meter-ed9ebc2508a6a9ef07109bd16969c4a9.json
And here's gas:
octopus_energy-01JBCE112X0FDSB77QPAT8GQXK-Gas Meter-100496d0e8ffe0bbeecc5713cdb568b2.json

For logs:

I see rate update events. I actually can't find anything about the electric or gas usage.

Thanks for those. Nothing is jumping out in the diagnostics other than the consumption sensors not updating as much as they should be. There are diagnostic entities which state when the underlying data was last updated which could shed some light on the situation (they should all be updating within the hour). I also need about an hours worth of debug logs to see if there are any underlying errors being raised.

If you're wishing to supply the requested data @quietbiker, then this might help solve both issues

I get this too, but just with Gas. Have added/removed the addon many times and re-generated API key.
Electric is fine
octopus_energy-01JBWADZBR4A7PD6ZM3AWY9GYQ-Gas Meter-72bd1f34665572ba103bd7e9c988d0cd.json

Sorry for the late response @PedroC1999. According to your data it can't get the gas information from the API. This however might have been a bug in the diagnostics. If you could update to v13.1.2 and try again, this should confirm what data is available.

octopus_energy-01JBWADZBR4A7PD6ZM3AWY9GYQ-Gas Meter-72bd1f34665572ba103bd7e9c988d0cd.json
octopus_energy-01JBWADZBR4A7PD6ZM3AWY9GYQ-Electricity Meter-484e6f70fbcf851b668767ee89751c88.json
Here you go, still the same situation.

@BottlecapDave
Hi, just to say the last couple updates that seemed relevant have also not changed the situation and it is still happening as before. Let me know if there's anything I can do.
Are there any "wgets" or API commands I can try and manually hit to check?
Thanks

The issue is the API, I get zero data back for Gas, but Electric is fully populated.

Hi
I have the same issue with gas, data stops on 7th October. All working fine until that point. I'm mostly just following instructions so if you need data then please point me clearly to what you'd like!

Hi, I contacted OE today and they have sent me a list of instructions. Following on from my message above, the app and IHD also has no data for Gas, and polling the API manually with a wget reveals no data at all for gas. On the phone, they were able to confirm this.
The steps below should pair the Gas smart meter with the HAN, which should restore data flow to Octopus.

Not yet confirmed as I am not home, will edit this post with any result.

I can give it a go. But I'm pretty sure my data is reaching Octopus as they seem to be billing ok.

@PedroC1999 Sorry for the lack of response. The diagnostic data you supplied suggested that there was no data available, but I wanted to confirm this wasn't a bug in the diagnostic data. Your manual proding confirmed the diagnostic data was indeed correct. But as you have confirmed, as the data isn't available via the API, then this was the cause of the issue. I'm hoping the instructions from OE solve your issue.
@curiousiguana Please confirm if you see data past the 7th October on the website. The app data will be skewed by a Home Mini if you have one, so isn't a great source. If the website doesn't have data either then you'll need to contact OE as it's something wrong on their end.
If the website does have data, then please upgrade to the latest release and follow the instructions to get your diagnostic data and post the results here. It might also be worth enabling the diagnostic entities as well to make sure HA is attempting to retrieve the data correctly.

Hi Dave, I've finally found time.  I did call Octopus in case they could spot something, but the agent has no details on their API and didn't appear to know anything about it.

If you're wishing to supply the requested data @quietbiker, then this might help solve both issues

Core: 2024.11.3
Supervisor: 2024.11.4
Operating System: 13.2
Frontend: 20241106.2
Octopus Energy 13.1.3
Error seen (HA Logs) as

Errors in request (https://api.octopus.energy/v1/graphql/): [{'message': 'Unauthorized.', 'locations': [{'line': 2, 'column': 2}], 'path': ['smartMeterTelemetry'], 'extensions': {'errorType': 'AUTHORIZATION', 'errorCode': 'KT-CT-1111', 'errorDescription': 'The viewer is not authorized to execute the query/mutation. Check authentication or roles/permissions.'}}]

CURL from command line does appear to work fine on GAS and Electric.
I'm running in a Proxmox VM - I don't think this makes any difference (standard networking and CURL example from Octopus does not show errors)
HA Log attached with DEBUG enabled.
home-assistant_2024-11-26T17-56-35.690Z.log

@quietbiker, it looks like for some reason the Home Mini device that is being reported in your account by the OE API is not accessible via the Home Mini API.
If you're comfortable running GraphQL, if you could run the following query to make sure there's not some clashes somewhere
query {
  account(accountNumber: "{account_id}") {
    electricityAgreements(active: true) {
			meterPoint {
				mpan
				meters(includeInactive: false) {
					serialNumber
          smartDevices {
            id
            deviceId
            type
            status
            manufacturer
            model
            firmwareVersion
          }
				}
			}
    }
    gasAgreements(active: true) {
			meterPoint {
				mprn
				meters(includeInactive: false) {
					serialNumber
          smartDevices {
            id
            deviceId
            type
            status
            manufacturer
            model
            firmwareVersion
          }
				}
			}
    }
  }
}

This will confirm if you need to contact OE based on the error. I'm guessing your home mini data is accessible via the app?

@PedroC1999 Sorry for the lack of response. The diagnostic data you supplied suggested that there was no data available, but I wanted to confirm this wasn't a bug in the diagnostic data. Your manual proding confirmed the diagnostic data was indeed correct. But as you have confirmed, as the data isn't available via the API, then this was the cause of the issue. I'm hoping the instructions from OE solve your issue.
@curiousiguana Please confirm if you see data past the 7th October on the website. The app data will be skewed by a Home Mini if you have one, so isn't a great source. If the website doesn't have data either then you'll need to contact OE as it's something wrong on their end.
If the website does have data, then please upgrade to the latest release and follow the instructions to get your diagnostic data and post the results here. It might also be worth enabling the diagnostic entities as well to make sure HA is attempting to retrieve the data correctly.

Octopus do have consumption for me.
I have the diagnostic data, how do I post without any account information? Is there anything within the file that I need to remove?

@curiousiguana  Sorry for the late response. The diagnostic data should have identifiable information removed, but there may be additional information that HA adds that you might want to remove.


feat: Temporarily disable, because I am poor.

Thanks for all your hard work you put into it! You are awesome!


change default namespace substitution for yasgui

Our data contains a lot of Classes and properties from https://schema.org.
When we edit the default query with 'schema:' then yasqe adds PREFIX schema: <http://schema.org/>. I want this behaviour to change, so that PREFIX schema: <http**s**://schema.org/> is inserted.
How can I set this prefix when I use the standard code for creating yasgui?
    const yasgui = new Yasgui(document.getElementById("yasgui"), {
  requestConfig: { endpoint: "my-custom-endpoint" },
  copyEndpointOnNewTab: true,
});

Note: the autocompletion of classes and properties is a nice feature and we don't want to ruin that.

Note: I tried this, but that doesn't achieve what I need, it seems.
    const yasgui = new Yasgui(document.getElementById("yasgui"), {
  requestConfig: { endpoint: "http://my_endpoint/sparql",
   },
   yasqe: { defaults: `"PREFIX schema: <https://schema.org/>
PREFIX rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#>
PREFIX rdfs: <http://www.w3.org/2000/01/rdf-schema#>
SELECT * WHERE {
  ?sub ?pred ?obj .

} LIMIT 10"`},
  copyEndpointOnNewTab: true,
});```

In regards to picking a prefix, we currently use prefix.cc to get autocompletion suggestions for prefixes,
Currently the http version of schema has the highest amount of votes (https://prefix.cc/schema)
As a workaround you can use the sdo prefix (https://prefix.cc/sdo) which does refer to the https version of schema
As for your changes, your config seems to miss a "layer". With this snippet, each new query should open with the desired query
const yasgui = new Yasgui(document.getElementById("yasgui"), {
  requestConfig: { endpoint: "http://my_endpoint/sparql",
   },
   yasqe: { defaults: { value: `"PREFIX schema: <https://schema.org/>
PREFIX rdf: <http://www.w3.org/1999/02/22-rdf-syntax-ns#>
PREFIX rdfs: <http://www.w3.org/2000/01/rdf-schema#>
SELECT * WHERE {
  ?sub ?pred ?obj .

} LIMIT 10"`}},
  copyEndpointOnNewTab: true,
});```

Let me know if this has resolved your issues

Ah thanks! Great to know it uses prefix.cc. I changed the prefix on the server and now using sdo: in yasgui triggers indeed the https version. Only autocompletion doesn't seem to be available for sdo: ?

Hmm, I unfortunately need to disappoint you here, for the autocompletion we use the LOV API, and it seems they only have the http version of schema in there

Thanks. I guess we need to do without completion then. Thanks for explaining a bit about yasgui.
By the way, the example snippet for the defaults configuration didn't result in another default for new tabs. I might have to get back to that later. For now it's OK.


Can't get service to work (on El Capitan 10.11.3)

I can't compare to any previous versions of OS X, but I've tried everything I can think of and I can't seem to get BTHSControl to respond to bluetooth evens from my headphones (e.g. Play/Pause still starts iTunes). I've also tried rebooting between service installations to no avail.
Here's a sample of what my syslog output looks like:
30/01/16 19:49:56,000 kernel[0]: IOBluetoothDevice::setProperties() -- calling mHCIController->CallCreateDeviceReporter() -- this = 0x3c00
30/01/16 19:49:56,000 kernel[0]: IOBluetoothHostController::CallCreateDeviceReporter -- calling enqueueAction() -- device = 0x3c00
30/01/16 19:49:56,000 kernel[0]: IOBluetoothHostController::CreateDeviceReporterAction -- device = = 0x3c00
30/01/16 19:49:56,000 kernel[0]: IOBluetoothHostController::CreateDeviceReporterAction -- calling CreateDeviceReporter() device = = 0x3c00
30/01/16 19:49:56,000 kernel[0]: IOBluetoothHostController::CreateDeviceReporter -- device = = 0x3c00
30/01/16 19:49:56,099 blued[87]: Device Connected with address: XX-XX-XX-XX-XX-XX and classOfDevice 0x240404 and connectionHandle 0x000c
30/01/16 19:49:56,000 kernel[0]: BT: returning false because owner <ptr> does not match new owner <ptr>
30/01/16 19:49:56,384 AVRCPAgent[2279]: Setting the delegate returned 0
30/01/16 19:49:56,388 blued[87]: Trigger core audio to connect to bluetoothaudiod
30/01/16 19:49:56,000 kernel[0]: Setting BTCoex Config: enable_2G:1, profile_2g:0, enable_5G:1, profile_5G:0

Trying to run AVRCPAgent manually in /Library/Services/AVRCPAgent (no other output):
./AVRCPAgent
Failed to initialize sandbox
2016-01-30 19:50:24.097 AVRCPAgent[2305:142740] BTHSControlPlugin installed
2016-01-30 19:50:24.098 AVRCPAgent[2305:142740] Setting Delegate: <BluetoothAVRCPAgent: 0x7fc982408c90>
2016-01-30 19:50:24.102 AVRCPAgent[2305:142740] Setting the delegate returned 0

Hmm... Did you run install.sh with sudo? The script shouldn't be run with sudo, so if it was, you should do a "sudo install.sh uninstall", then do "install.sh install".
I'm the most experienced with creating installation scripts, so I know it could probably be constructed better and I'll probably get to that eventually.
I'm not assuming you ran it with sudo, I just know that's a reason why it wouldn't work. I just did a fresh uninstall/install on 10.11.4 and everything is working smooth.
If you've gotten to this point and it's still not working, I would suggest:
1.) Turn your BT headset off
2.) Run "./install.sh uninstall"
3.) Reboot your computer (to possibly clear any rouge services or something)
4.) Perform a fresh install: "./install.sh install"
5.) Turn on your BT headset
6.) Test it with something like VLC or your desired music player open
And if it still doesn't work, send me the output of the "./install.sh install" and we'll see if that helps. Let me know how it goes.

I haven't run the script with sudo, but I've also tried to both run the launch agent as my regular user and sudo. I've gone through the script as well and verified the steps to see what it's doing, nothing seems to be missing.
I've not performed the steps exactly as you laid them out, but essentially I've done what you have indicated in your instructions. The only difference is that I've not done a complete reboot where the launch agent wasn't already installed and the headphones had never been connected to the computer before the launch agent got installed, to be honest if that's a problem then it would probably be a problem any time I reboot my computer or the like.
The output from the installation script is probably as expected:
❯ ./install.sh install
Installing
Building plug-in: BTHSControl
=== BUILD TARGET BTHSControl OF PROJECT BTHSControl WITH THE DEFAULT CONFIGURATION (Release) ===

Check dependencies

** BUILD SUCCEEDED **

Setting up Service: AVRCPAgent
Setting up LaunchAgent: com.bths.AVRCPAgent
Activating Service: AVRCPAgent
Installation complete!

Got it. Yeah, no, the reboot isn't required, it's just a step I like to include to possibly be thorough. You definitely don't need to unpair the device then repair.
Have you tried turning the headset off after installation, then back on again? If the agent is started while the headset is connected, it wont work.
Also, if I remember correctly, the agent should only be launched as the user, not with sudo.
I've seen a case where the play/pause doesn't work, but the forward/back does, so let me know if the forward/back is doing anything for you.
If it still doesn't work, let me know what kind of headphones you have and I'll see if I am not accounting for different types of headphones.

So far I've tested with my Sennheiser Momentum Wireless headphones and a TDK TREK Max bluetooth speaker, neither works with BTHSControl. To me it looks like AVRCPAgent is not intercepting any of the bluetooth commands since it's essentially not displaying any new output after a device has been connected. I've tried to be as thorough with the logs and steps here as possible:
No bluetooth device connected (and make sure we have a fresh AVRCPAgent process):
❯ ps aux|grep "[A]VRCPAgent"
mafredri         4591   0.0  0.0  2497056   7004   ??  S    10:05PM   0:00.03 ./AVRCPAgent
❯ launchctl unload /Library/LaunchAgents/com.bths.AVRCPAgent.plist
❯ launchctl load /Library/LaunchAgents/com.bths.AVRCPAgent.plist
❯ ps aux|grep "[A]VRCPAgent"
mafredri         5226   0.0  0.0  2498628   7000   ??  S    10:35PM   0:00.02 ./AVRCPAgent

Connect headphones:
30/01/16 22:35:37,468 Gear Player[3415]: No write key
30/01/16 22:35:42,250 blued[87]: Device Connected with address: XX-XX-XX-XX-XX-XX and classOfDevice 0x240404 and connectionHandle 0x000b
30/01/16 22:35:42,457 blued[87]: Trigger core audio to connect to bluetoothaudiod
30/01/16 22:35:43,873 AVRCPAgent[5226]: Setting the delegate returned 0
30/01/16 22:35:43,000 kernel[0]: BT: returning false because owner <ptr> does not match new owner <ptr>

Start playing music from Gear Player (manually):
30/01/16 22:35:46,475 AVRCPAgent[5226]: New AVRCP data: <10110e03 48000019 58310000 05010000 0000>
30/01/16 22:35:46,475 AVRCPAgent[5226]: [parseNOTIFY] registering for notification: 'Playback Status Changed'
30/01/16 22:36:01,000 kernel[0]: Setting BTCoex Config: enable_2G:1, profile_2g:0, enable_5G:1, profile_5G:0
30/01/16 22:36:10,937 Console[5300]: Failed to connect (_consoleX) outlet from (NSApplication) to (ConsoleX): missing setter or instance variable
30/01/16 22:36:22,469 Gear Player[3415]: No write key
30/01/16 22:37:07,471 Gear Player[3415]: No write key
30/01/16 22:37:15,000 kernel[0]: Setting BTCoex Config: enable_2G:1, profile_2g:0, enable_5G:1, profile_5G:0

No buttons worked, pause / play, forward track, backward track, rewind, forward. Disconnect headphones:
30/01/16 22:37:26,636 AVRCPAgent[5302]: BTHSControlPlugin installed
30/01/16 22:37:26,637 AVRCPAgent[5302]: Setting Delegate: <BluetoothAVRCPAgent: 0x7fb059511070>

❯ ps aux|grep "[A]VRCPAgent"
mafredri         5302   0.0  0.0  2490436   7124   ??  U    10:37PM   0:00.02 ./AVRCPAgent

Disconnecting also caused the AVRPCAgent to get killed and start with a new PID.
I'd be happy to help debug this issue if you've got any ideas where to start, but I'm quite sure I'm doing nothing wrong with the installation and timing part. I've also tried to build BTHSControl with -configuration Debug but that didn't give me any new output at all :).


Carbon 'Debug' Flag

Near the end of Coeus, I started to implement a DEBUG macro for making strongly-typed variants of methods without hampering the performance of release/production builds.
With Carbide (or even without), having a Carbon.Debug flag would be extremely useful for the framework and for user code, as could some other debug flags for things like verbose logs.

I've introduced mild use of the Debug flag so far with Carbide, but I haven't integrated anything completely yet.
In addition to having a Debug flag, it might also be nice to be able to disable/enable features one-by-one in addition to whatever the setting is. Setting debug mode might require a function call, but I'd prefer that debug checks and setting lookups don't require function calls.

This is going in as the Debug module in 1.0.1

Actually, this is just going to be a Feature for 1.0.1!


WIP: Add initial test base for mysensors

Description:

Mock mysensors library, gateway, node, child and message.
Fix validation return value.
Fix inconsistent setup_mysensors_platform return.
Calculate coverage for mysensors component modules.

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] Tests have been added to verify that the new code works.

I'll close this until I've figured out best approaches.


exception caught when get_proxy tries to write to proxy file

Hi @PalNilsson
I noticed
2022-08-24 16:18:58,609 | INFO     | pilot.util.https                 | get_curl_command          | request: curl -sS --compressed --connect-timeout 100 --max-time 120 --capath /cvmfs/atlas.cern.ch/repo/ATLASLocalRootBase/etc/grid-security-emi/certificates --cert /proxy/x509up_u25606_prod --cacert /proxy/x509up_u25606_prod --key /proxy/x509up_u25606_prod -H 'User-Agent: pilot/3.3.4 (69) (Python 3.8.13; Linux x86_64)' -H 'Accept: application/json' --config /pilotdir/PanDA_Pilot3_4235_1661356884/curl_getProxy.config https://aipanda057.cern.ch:25443/server/panda/getProxy
2022-08-24 16:18:58,609 | INFO     | pilot.util.container             | print_executable          | executing command: curl -sS --compressed --connect-timeout 100 --max-time 120 --capath /cvmfs/atlas.cern.ch/repo/ATLASLocalRootBase/etc/grid-security-emi/certificates --cert /proxy/x509up_u25606_prod --cacert /proxy/x509up_u25606_prod --key /proxy/x509up_u25606_prod -H 'User-Agent: pilot/3.3.4 (69) (Python 3.8.13; Linux x86_64)' -H 'Accept: application/json' --config /pilotdir/PanDA_Pilot3_4235_1661356884/curl_getProxy.config https://aipanda057.cern.ch:25443/server/panda/getProxy
2022-08-24 16:19:00,645 | ERROR    | pilot.util.proxy                 | get_proxy                 | exception caught:
[Errno 30] Read-only file system: '/proxy/x509up_u25606_prod',
traceback: Traceback (most recent call last):
  File "/pilotdir/atlas_FVvipxe6/pilot3/pilot/util/proxy.py", line 87, in get_proxy
    _file = os.open(proxy_outfile_name, os.O_WRONLY | os.O_CREAT | os.O_TRUNC, 0o600)
OSError: [Errno 30] Read-only file system: '/proxy/x509up_u25606_prod'

On k8s systems the $X509_USER_PROXY file is a mounted secret volume that (like all k8s mounted secrets) is read-only in the pod. It is regularly refreshed by Harvester so jobs always see a recent proxy.
It looks like the get_proxy function tries to open the proxy for writing:
https://github.com/PanDAWMS/pilot3/blob/master/pilot/util/proxy.py#L85
Incidentally the ownership and permission of a mounted secret in k8s have to be like this:
-rw-r--r--. 1 root localuser 11958 Aug 24 20:43 x509up_u25606_pilot
-rw-r--r--. 1 root localuser 12596 Aug 24 20:43 x509up_u25606_prod

which is fine since there are no other users in the pod, but can cause compatibility issues if applications assume/expect a traditional multi-user environment.
Is the exception an issue, or just cosmetic?  Even if the latter it would be nice to avoid it always happening. Perhaps the pilot could only try to open the proxy file for writing if it does not exist?
Thanks!

Hi Ryan,
Can you please forward the Panda ID for that? I'd like to see exactly what happened (and which pilot version you are using). In the case the proxy has expired, the pilot will download a new proxy so it will certainly need to open the file in write mode. The other case is for payload proxies in user jobs, which are also downloaded by the pilot and also needs to be written. Check if the proxy actually had expired. If these mechanisms are not desired on kubernetes, we need some logic to turn them off.
Cheers, Paul

Hi Paul,
It seemed to be a dev pilot running on our prod cluster, does that indicate it is a new feature that will arrive in the prod pilot soon?
pilotUrlOpt=--piloturl http://cern.ch/atlas-panda-pilot/pilot3-dev.tar.gz

2022-08-25 18:00:20,689 [wrapper] pandaids:<PHONE_NUMBER>

I can send the whole log file.
This job executed only 40 minutes before I checked the proxy, it had > 95 hours remaining. Harvester refreshes it regularly and the same proxy is used by all jobs on the cluster, so if there were a problem with the proxy there would have been widespread failures. However, I checked every job on the cluster and only 5 jobs had this message.  Actually all of them were dev pilots according to pilotUrlOpt=--piloturl http://cern.ch/atlas-panda-pilot/pilot3-dev.tar.gz
The pilot will not be able to update or write a proxy file, unless it places it in a different location from the provided $X509_USER_PROXY.
Thanks!

Hi Ryan, sorry about the delay. Can you please try this version instead?
http://cern.ch/atlas-panda-pilot/pilot3-<IP_ADDRESS>.tar.gz
It should have a fix for read-only file systems.
Cheers, Paul

Hi Paul,
Thanks, I am not sure how though. Is there a CRIC parameter to use a different pilot URL?
https://atlas-cric.cern.ch/atlas/pandaqueue/detail/CA-VICTORIA-K8S-TEST-T2/
-rt
Ryan Taylor
Research Computing Specialist
Research Computing Services, University Systems
University of Victoria

From: Paul Nilsson @.***>
Sent: August 31, 2022 7:35:42 AM
To: PanDAWMS/pilot3
Cc: Ryan Taylor; Author
Subject: Re: [PanDAWMS/pilot3] exception caught when get_proxy tries to write to proxy file (Issue #52)
Notice: This message was sent from outside the University of Victoria email system. Please be cautious with links and sensitive information.
Hi Ryan, sorry about the delay. Can you please try this version instead?
http://cern.ch/atlas-panda-pilot/pilot3-<IP_ADDRESS>.tar.gz
It should have a fix for read-only file systems.
Cheers, Paul
—
Reply to this email directly, view it on GitHubhttps://github.com/PanDAWMS/pilot3/issues/52#issuecomment-1233023141, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAM3YY5TXYLWSKQRIJY3KQDV35UT5ANCNFSM57QZXYXQ.
You are receiving this because you authored the thread.Message ID: @.***>

Hi,
I've updated it for you. In the "Associated Params" section you just click "Add Param" and then add "pilot_url" + a value.
Cheers, Paul

Just a note: it's better that we use fixed pilot versions, like <IP_ADDRESS>, than the generic pilot3-dev version since that might not be stable (as you actually saw). pilot3-dev will still be used for the automatic RC test jobs, but if you send a normal test job to the queue, it should use the <IP_ADDRESS> version. RC jobs arrive very slowly and I don't see any such jobs since yesterday.

Hi @rptaylor I have updated the pilot version in CRIC (close to final version), but I don't see any jobs running
http://apfmon.lancs.ac.uk/q/CA-VICTORIA-K8S-TEST-T2
Can you please check?
Cheers, Paul

The dev cluster was in scheduled downtime because I was rebuilding it.  Should be running again now.

it's better that we use fixed pilot versions
Do you mean temporarily for cases like this to confirm a specific issue is fixed, or in general? If the latter surely we will forget about the version in CRIC and it will get out of date.

If you use the default dev pilot, it means that the wrapper is downloading the pilot3-dev.tar.gz tarball. This one is inherently unstable, ie risky to use for anything but pure tests. A fixed version will have the version number in the file name, like pilot3-<IP_ADDRESS>.tar.gz. I will only create those files for release or to test specific things, like on your queue. In fact, we're just now in the process of moving to use fixed pilot versions also for production since we're using the pilot in three different experiments, and they will in general not care about changes only relevant for the others.
I updated the version on your queue yesterday to the latest stable version, <IP_ADDRESS>, looks good as far as I can see. If there's a need to download a proxy, it will do so to the work directory which will be writable..

Okay sounds good, as long as I don't have to do or remember anything to make sure the right pilot version is used. I guess I will just leave the pilot_url option alone and you can change it as needed.
Thanks for the fix! I suppose this can be closed then.


Event forwarding through Shadow DOM polyfill does not unwrap event.target

If you attach this event handler in a browser where Shadow DOM is polyfilled:
window.addEventListener('resize', function(evt) {
  console.log(evt.target === window);
});

you will discover that the output is false.  The reason is that evt.target is actually Object { __impl4cf1e782hg__: window } or similar.
This seems like a very specific problem, except that this code is almost precisely what exists in emscripten's library_html5.js, and is called when attaching resize events to the window from within C++ code.  This issue therefore renders Polymer inherently incompatible with the event handling code in emscripten.
This could be fixed by unwrapping the event target before forwarding the event, though that could conceivably break existing code.

Closing this issue due to age and the release of version 1 of Polymer - please feel free to re-open if this is incorrect.


feat(streamingVolumeLoader): added IMAGE_VOLUME_LOADING_COMPLETED event

Added IMAGE_VOLUME_LOADING_COMPLETED to make it possible to know when a volume is done loading

can you run yarn run build:update-api? and include them? https://www.cornerstonejs.org/docs/contribute/update-api

can you run yarn run build:update-api? and include them? https://www.cornerstonejs.org/docs/contribute/update-api

Done


IPFS as file system

Just curious how NodeOS can win from using IPFS as a default tool?

I know of IPFS a long time ago, but didn't know there got so far! :-D
Ok, NodeOS itself would not get too much benefict... but on the other hand, IPFS would be an interesting filesystem that users would mount their home dir on top of, and that would be something really cool :-D I was thinking to mount a WebDAV-based filesystem so the user files could be on the cloud and local hard disk just works as a cache, but using IPFS would give a lot more oportunities :-) Are you volunteer to work on that? :-P

Sooo NodeOS with the Plan9 philosophy? Distributed microkernel? That's sounds pretty cool actually.

Not sure about the distributed microkernel, but definitely, some NodeOS concepts are heavily influenced by Plan9 since I studied it for my operating system lessons :-)

The Dat Project would be another good choice!


Location doesn't work on Android

Location doesn't work on Android. Always opens the first page. Locator props works fine.
<ReadiumView
file={{url: filePath}}
settings={settings}
location={{
href: '/EPUB/text/ch002.xhtml',
locations: {
position: 5,
progression: 0,
totalProgression: 0.024390243902439025,
},
type: 'application/xhtml+xml',
}}
onLocationChange={locator => locationChanged(locator)}
/>
Error: lateinit property navigator has not been initialized

@M-HARIS-97 interesting, this was working before. I'll try to take a look at it today. A full stacktrace from the android side would be helpful.

@M-HARIS-97 , I just tested this in the example app and I'm unable to reproduce it.
Can you please pull the latest version of the project and see if you can reproduce it in the example app? I've added instructions on how to run the example app.
You can test it by selecting one of the chapters from the table of contents. You can see that the navigator correctly navigates to the chapter. I'm going to close this until you figure out a repro in the example.

@M-HARIS-97 you need to describe to me how to reproduce the error in the example app. Have you tried that?

@jspizziri My application has 2 screens. First is an epub file list screen and second is a readium reading screen. I select an epub file, flick through some pages and store the last page locator value in my local db (sqlite), then return to the listing page. I select the same book again, that time I retrieve the locator value from the local DB and pass the value to the location property. That time I got an error message which I shared earlier.

@jspizziri I was able to reproduce the problem using the example app. I made some changes to the example app. Please refer to the following to reproduce the issue;
1 - Install sqlite db
2 - Store the locator value in DB
3 - Close the app
4 - Then, open the app again
5 - Retrieve the locator value from DB
6 - Pass that value to the ReadiumView location parameter

@jspizziri Please go through the following steps;
1 - Open the book. Flick some pages and console the locator value.

2 -  Copy that specific locator value from console.
3 - Then apply that locator value directly to the location props

4 - Close the app
5 - Reopen the app
6 - Expected
6.1 - Opens the book and navigates to the specified location
But What I'm Seeing
6.2 - The book opens and shows its first page
After updating to the latest version 1.0.4 its not working on iOS too.

@jspizziri I'm still facing this problem on iOS and android. Could you pleas verify this?

@M-HARIS-97 upon opening the App you should be using initialLocation which is part of the File object. The location attribute should really only be used during navigation once the reader view is booted. The reason for the duplicate property is to avoid a race condition. In order to properly build the view at the correct initial location the location needs to be guaranteed to be present at the time the view is built. Because of the asynchronous nature of the react-native bridge layer we can't guarantee that the location property will arrive at the native layer before or at the same time that the file property does. Therefore they need to be passed as the same object.
In short, location was never intended to be used for boot position, and it never should've been working on iOS or Android unless it was by pure luck that when you happened to run it, the async calls randomly happened to resolve in the correct order to make it work.
With all that said, I could see a potential need for a feature that reports the state of the views initialization process. Something like the following to help determine if you're allowed to start passing locations, etc:
onStateChange={(state: 'loading' | 'initialized' | 'errored' ) => { ... }}

@jspizziri Thanks for your detailed explanation. Currently it works well on iOS and android. Previously I was not using the initialLocation property. I think that's why I faced problems in my project. Once again Thanks @jspizziri for your great help.

@M-HARIS-97 thanks for the bug reports! I should probably add something to the README.md to make it more clear to folks what that property is for.

I've updated the readme for better clarity here https://github.com/5-stones/react-native-readium/commit/dc92e1e38c4c9f37aa8e7e078efba8501e2bdc91

@jspizziri Ok Noted


Update BaseSyntaxTest to use Eq[A] instead of java .equals

The current code is inconsistent: the Field syntax test uses Eq[A], while the other tests use cooperative equality.

Regrouped.


Support the ability to generate templates on tests based on a mocked Kubernetes cluster

As a followup to https://github.com/aiyengar2/hull/issues/2, while the solution requested in the previous issue allows users to be able to mimic a helm install instead of a helm template / helm install --dry-run, it would not allow for environment-agnostic tests since the underlying Kubernetes cluster used may be different for different users, which can produce different results in tests.
Instead of assuming that a consistent environment is used in all test runs, Hull should offer the ability to start a HTTP server on a port that mocks one or more "read-only" API servers that are preconfigured to contain certain manifests of resources.
Ideally, from a user standpoint, a user would simply need to provide a manifest of resources (either a manifest string that can be go embedded from a file or a []runtime.Object) and a mock API server should be generated with a rest.Config that points to it that can be used for the e.RenderWithClient call; as a result, this mocked API server will be used as the basis for calling Helm which can result in tests that are fully environment-agnostic as they don't need a live Kubernetes cluster to be pre-setup to run tests with specific expectations.

This can be done simply by integrating with envtest, which handles setting up a mocked API Server and etcd only (no kubelet, so no actual running processes).


[Bug]: Module - Card View - name

Prerequisites

[X] I have searched the backlog for duplicate or closed feature requests

Your test device
iPhone 10
OS Version
iOS 15.3.1
App version
0.2.0
Describe the issue
Name is incorrect should be 'Card View'

Expected Behavior
update name to 'Card View'

This change was performed on the card in the Module page but not in the page itself (still named ListCard). The name of the card and the name of the page should be the same.

based on comment on the 21st of June the name was update to  "Card collections".  In the version I test build<PHONE_NUMBER> the name was updated to "Card View".
A new card was updated to correct this at #219


Do not write schema to schema_check table

Background
#2603 implementation
Description
Do not write schema/composite schema/ supergraph schema to schema_check db table if there were no changes
Checklist

[x] Testing

What is missing here is the business logic to instead load the schema from the linked schema_version reocrd when serving it via the API.

Hi @n1ru4l  can u pls look at this version?

@andriihrachov I added two comments.


Video Player

_ No description provided. _

@SunnyCoder0013 Please add problem statement and approach in the code file.


carbonlink: make it easilly importable, delay initialization

Also make readers.py pull less dependency and add
graphite.singleton to make it easier to have singleton.
This could also be used for some of our other singletons
to allow easier lazy-initialization.
See #1014 , #1886

Also see https://github.com/criteo/biggraphite/pull/259

Codecov Report

Merging #1891 into master will decrease coverage by 0.77%.
The diff coverage is 52.32%.

@@            Coverage Diff             @@
##           master    #1891      +/-   ##
==========================================
- Coverage    66.3%   65.53%   -0.78%
==========================================
  Files          56       57       +1
  Lines        6603     6644      +41
  Branches     1318     1321       +3
==========================================
- Hits         4378     4354      -24
- Misses       2002     2073      +71
+ Partials      223      217       -6

Impacted Files
Coverage Δ

webapp/graphite/singleton.py
48.88% <48.88%> (ø)

webapp/graphite/carbonlink.py
24.24% <52.77%> (-27.22%)
:arrow_down:

webapp/graphite/readers.py
41.46% <80%> (+0.47%)
: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 bbfab10...46690f1. Read the comment docs.

(wait before reviews, some things apparently don't work yet)

ok good now :)

@deniszh : I'll probably use the same singleton.py in carbon as there are a few singleton that could benefit from it

:+1:

I'll probably use the same singleton.py in carbon too as there are a few singleton that could benefit from it

Yep, that's a good idea ! 👍
Thanks!


refactor(api): dex prices integration

What kind of PR is this?:

/kind refactor
What this PR does / why we need it:
Which issue(s) does this PR fixes?:

Fixes #2318
Additional comments?:
Developer Checklist:

[ ] Read your code changes at least once
[ ] Tested on iOS/Android device (e.g, No crashes, library supported etc.)
[ ] No console errors on web
[ ] Tested on Light mode and Dark mode*
[ ] Your UI implementation visually matched the rendered design*
[ ] Unit tests*
[ ] Added e2e tests*
[ ] Added translations*

Hey just some things to check while you're still working on it
1. Should we display the 0 USD value?
This branch:

Main branch:

2. Composite Swap not working
Steps:

Choose from: dBTC to to: dETH
Click MAX
No errors but Continue button is still disabled


Carthage update hangs at download

Platform:

[X]  iOS
[ ]  tvOS
[ ]  MacOS
[ ]  watchOS

Swift:

[X]  Yes -> 3.1
[ ]  No

sentry-swift installed with:

[ ] Cocoapods
[X] Carthage
[ ] Manually

Version of sentry-swift:
2.1.5

I have following issue:
Cartage update starts downloading the framework and gets stuck.
Steps to reproduce:

carthage update --platform iOS --verbose

Actual result:
➜  krosskast-IOS git:(master) ✗ carthage update --platform iOS --verbose
*** Fetching sentry-swift
*** Fetching OAuth2
*** Fetching SwiftyBeaver
*** Fetching SDWebImage
*** Checking out AudioPlayer at "c88b6f0a4ed8f43680609f3ef8df806334671056"
*** Checking out KSCrash at "1.15.3"
*** Checking out OAuth2 at "3.0.3"
*** Checking out SDWebImage at "4.0.0"
*** Downloading sentry-swift.framework binary at "2.1.5"
Expected result:

Expect sentry to be checked out

Hey, please try again .... something went wrong when building sentry ... we will try to improve our build process.


word-wrap: break-word /* comment please? */

Silly me, I sometimes send perfectly good stuff through the W3C CSS validator as CSS3 just to see what happens.  For instance, this snippet recently caught my interest:
/*
 * Improves readability of pre-formatted text in all browsers
 */

pre {
    white-space: pre;
    white-space: pre-wrap;
    word-wrap: break-word;
}

(Uh-huh.  But please don't roll your eyes just yet...)
As I'm sure you already know, the validator complained:
6 	pre 	Property word-wrap doesn't exist : break-word

Wha...? Says I, and so I go looking.  (Time passes...)
AFAIK, word-wrap is not in the W3C Spec, although I did find: overflow-wrap along with this interesting quote:

For legacy reasons, UAs may also accept ‘word-wrap’ as an alternate name
for the ‘overflow-wrap’ property. However this syntax non-conforming
in author style sheets.

"Legacy? I don't want no stinkin' 'Legacy,'" I huff. And so, I scamper back to the validator to try it out the "Modern" way:
/*
 * Improves readability of pre-formatted text in all browsers
 */

pre {
    white-space: pre;
    white-space: pre-wrap;
    overflow-wrap: break-word;
}

which promptly results in
8 	pre 	Property overflow-wrap doesn't exist : break-word

Picture me having one of those hilariously-funny-if-it's-not-you "What the...?" Yosemite Sam kind of moments:  "Not in the...  Why you...
Since then, found a little more discussion both hereabouts, and elsewhere.
After which point, to quote the inimitable Jar-Jar Binks, "My give up! My give up!"
So what's the deal here?  Well, my point is not to say that I know what ought to go there in the code.  Rather, it's to raise the possibility of inserting a handy-dandy comment--perhaps something along the lines of, "Danger, Pard'--Tar Pit Ahead!"  It's too late for me.  My bones are already bleaching in the sun.  But, it would doubtless be much appreciated by some other cowpokes coming along later.  After all, there are so many truly great and helpful other comments in there.  Why not provide a little heads-up about this?  ;-)  (P.S. Hope you got a laugh.)

There are a handful of things in normalize that the validator is going to complain about, but I don't want to start labelling lines of code with validator results.


"Failed to mount the InstallESD.dmg" Error

Hello!
I'm running a fresh copy of Sierra and a fresh download of the Sierra installer (10.12.3) in my Applications directory. When I run this script, I get the following error:
Mount the installer image
-----------------------------------------------------------
$ hdiutil attach /Applications/Install macOS Sierra.app/Contents/SharedSupport/InstallESD.dmg -noverify -nobrowse -mountpoint /Volumes/install_app
hdiutil: attach failed - Resource busy
Failed to mount the InstallESD.dmg from the instaler at Install macOS Sierra.app.  Exiting. (1)

Any thoughts?

Oh.
I went into Disk Utility and unmounted the image, and the script worked great. 🤦♂

@xd1936 - No problem, I'm happy you posted the issue and solution, at least—future Google searches will thank you :)

True that :)
Thanks!

It worked.
Thanks!


Update ts0601_dimmer.py

Signature added to dimmer config and confirmed working.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.319%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5440

Relevant Lines:
6773

💛  - Coveralls

Adjusting PR, please disregard.


Ignore build_tools for the language bar

I'm sorry, this PR is really nitpicking.
Currently, the language bar shows that skopt has 15.8% Mako and 2% Shell, although these are only for the build tools.
Wouldn't it be better to exclude them and indicate that skopt is 100% Python?
We could also include the Jupyter notebooks in the examples folder.

We could also include the Jupyter notebooks in the examples folder.

Agreed! Could you also add that?

Sure!
...
Oh. Looks not so good: https://github.com/mp4096/scikit-optimize

Oh wow! This is a whole new level of github-fu!

Hmm, why do we still see 93% for jupyter?

Current coverage is 84.76% (diff: 100%)

Merging #251 into master will not change coverage

@@             master       #251   diff @@
==========================================
  Files            25         25
  Lines          1313       1313
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1113       1113
  Misses          200        200
  Partials          0          0

Powered by Codecov. Last update 332f761...339e7aa

@glouppe Sorry, I think I got you wrong. The original PR excluded build_utils and the language bar was 100% Python.
Then I thought maybe it's nice to force include notebooks to the language bar, but they're obviously much larger than the Python sources (probably due to images).
Ok, I'll now update the PR to show Python only.

So where to see this language bar? I can't seem to find it anywhere?

Current:

Proposed:

Whoa! I never noticed that bar till now.

The only PR where less 🌈 is better! Thanks for this.


Add Windows Mixed reality Support

This could potentially be a huge step in the difference between ALVR and the possibly upcoming wireless oculus link. This would offer a bigger variety of games to be played like for example, Minecraft bedrock edition, Halo Recruit, etc. that can be accessed in the windows mixed reality portal.

just wait, soon there will probably be a steamvr to wmr portal translator and it will probably work better than whatever we implement

Yeah a WMR-SteamVR bridge would probably be better than adding WMR to ALRV directly

The Windows Mixed Reality runtime does not offer an API to build device drivers, so support on ALVR is just not possible. We need a more advanced third party tool for the SteamVR-WMR bridge, but since the WMR ecosystem is much smaller than SteamVR, this may never happen.

https://docs.microsoft.com/en-us/windows/mixed-reality/develop/platform-capabilities-and-apis/device-portal-api-reference

As far as I can tell this just allows you to control a WMR-enabled device from a computer, kind of like their version of ADB.


Adds git pull commands

Coverage remained the same at 100.0% when pulling 8e64f5dc441f07eb821280d9ad31ded2cb0d69d3 on command/git-pull into 39846e1a628c39e0e687307cda8d35d2fd36cacd on master.


Add my contribution to challenge 102

Summary
2D water ripples implemented in C++ using SDL library for graphics and OpenMP to multithread the algorithm.
I don't have a web version of this due to the language and tools used however I uploaded a gif of the result while running the program.
Changes

Edited the file _CodingChallenges/102-2d-water-ripple.md to add my contribution

Tests

Clone https://github.com/1475963/be-water-my-friend and follow README's content

Looks great!


UMD support added

Implemented following the UMD recommendations for jQuery plugins.
Moreover I've added the "resources" folder to have a common root folder for all the resources.
Thanks! The library really rocks!  :+1:

Thanks a lot . Really useful enhancement. :+1:

added you as a contributor :smile:

Thanks, I'll playing these next days with it at work, so you'll know more about me. :)

Would love to know more about you :)


Basics: update version number

The current main branch on Swift indicates 5.11, follow the same version.

@swift-ci please smoke test

Seems to be a duplicate of https://github.com/apple/swift-package-manager/pull/6995

Ah, nice; we should figure this out.  It makes it very confusing to debug SPM issues.


Split __aenter__ and __aexit__ logic into functions

Move __aenter__ and __aexit__ logic into separate functions, to benefits from the SLResult<()> return annotation, which in turns enables us to handle channel and redis operations with the ? operator.

Codecov Report
Base: 98.62% // Head: 98.55% // Decreases project coverage by -0.06% :warning:

Coverage data is based on head (9d33747) compared to base (3b4c979).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #15      +/-   ##
==========================================
- Coverage   98.62%   98.55%   -0.07%
==========================================
  Files           9        9
  Lines         435      416      -19
==========================================
- Hits          429      410      -19
  Misses          6        6

Impacted Files
Coverage Δ

src/_utils.rs
82.75% <ø> (-2.96%)
:arrow_down:

src/_tests.rs
98.76% <100.00%> (-0.13%)
:arrow_down:

src/semaphore.rs
100.00% <100.00%> (ø)

src/token_bucket.rs
100.00% <100.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[SPARK-45954][SQL] Avoid generating redundant ShuffleExchangeExec node

What changes were proposed in this pull request?
This PR enhances EnsureRequirements to avoid generating redundant ShuffleExchangeExec node from the Spark plan.
Why are the changes needed?
Reduce shuffle to improve query performance. The use case is user add a repartition to increase parallelism for broadcast join, but this is useless for shuffle join.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Unit test.
Was this patch authored or co-authored using generative AI tooling?
No.

It should only take effect in the following cases, usually the user wants to increase the parallelism for BroadcastHashJoin/BroadcastNestedLoopJoin:
df1.repartition(num).join(df2, )
df1.repartition(num).select(xxx).join(df2, "id")

df1.repartition is a user-specified shuffle, we should better respect it, or at least we should add a new config. e.g., it's likely people want to add a shuffle to solve skewed data before aggregation or something else.

This change does not affect adding repartition before aggregation:
spark.table("t1").repartition(10).groupBy("a").count().explain(true)

== Physical Plan ==
AdaptiveSparkPlan isFinalPlan=false
+- HashAggregate(keys=[a#16L], functions=[count(1)], output=[a#16L, count#23L])
   +- Exchange hashpartitioning(a#16L, 5), ENSURE_REQUIREMENTS, [plan_id=62]
      +- HashAggregate(keys=[a#16L], functions=[partial_count(1)], output=[a#16L, count#28L])
         +- Exchange RoundRobinPartitioning(10), REPARTITION_BY_NUM, [plan_id=58]
            +- FileScan parquet spark_catalog.default.t1[a#16L]


K8s native clean up part 1

I've started to homogenize fabric8 and discovery client implementations slowly... More PRs like this coming your way :)

@ryanjbaxter forgot to tag you, sorry


changing names to experiment_settings.py following design doc. move s…

…et_values and initailizeCombo in presenter
Short description of the changes:

Long description of the changes:

Check list for the pull request

[ ] I have read the [CONTRIBUTING]
[ ] I have read the [CODE_OF_CONDUCT]
[ ] I have added tests for my changes
[ ] I have updated the documentation accordingly

Check list for the reviewer

[ ] I have read the [CONTRIBUTING]
[ ] I have verified the proposed changes
[ ] best software practices

[ ] all internal functions have an underbar, as is python standard
[ ] clearly named variables (better to be verbose in variable names)
[ ] code comments explaining the intent of code blocks

[ ] All the tests are passing
[ ] The documentation is up to date
[ ] code comments added when explaining intent

Manual test for the reviewer

References

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

Project coverage is 95.39%. Comparing base (38ece48) to head (d62d1ec).

Additional details and impacted files
@@            Coverage Diff             @@
##             next      #14      +/-   ##
==========================================
+ Coverage   95.07%   95.39%   +0.32%
==========================================
  Files           9        9
  Lines         325      326       +1
==========================================
+ Hits          309      311       +2
+ Misses         16       15       -1

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

@AndreiSavici I saw in Shiver there is no tests for presenter. Only three folders Data, View, Model.


Would you recommend this over symfony/session?

Hi Odan, Thank you for this package, I'm looking into your slim-4-skeleton and see that the symfony session is used there, which one would you recommend? Thank you for your work in the slim community.

Hi @notflip This session library was created with PSR-7 and PSR-15 (Middleware) in mind. The focus is on sessions only, while the symfony/http-foundation component is not PSR-7 and PSR-15 based and also brings a lot of additional classes on board but also more session storage options, a stable API and very good support from the community. If you want to use a more widely used component and more storage adapters, like the PdoSessionHandler or the RedisSessionHandler then better use the Symfony session component.