sqlmesh prompt command failing

I discovered the prompt command today. I tried to use but it failed. Help command wasn't sufficient. Anything else I need to do?
❯ sqlmesh prompt "get me top n  employees"
Traceback (most recent call last):
  File "/Users/sumanth/Library/Python/3.10/bin//sqlmesh", line 8, in <module>
    sys.exit(cli())
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/click/core.py", line 1659, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/click/decorators.py", line 26, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/Users/sumanth/Library/Python/3.10/lib/python/site-packages/sqlmesh/cli/__init__.py", line 20, in wrapper
    return func(*args, **kwargs)
  File "/Users/sumanth/Library/Python/3.10/lib/python/site-packages/sqlmesh/cli/main.py", line 417, in prompt
    from sqlmesh.integrations.llm import LLMIntegration
  File "/Users/sumanth/Library/Python/3.10/lib/python/site-packages/sqlmesh/integrations/llm.py", line 5, in <module>
    from langchain import LLMChain, PromptTemplate
ModuleNotFoundError: No module named 'langchain'

There are some dependencies missing I guess, can you try running the following?
pip install -e ".[llm]"

It worked. Thanks!
Let me know if should close this right away or you want to add some dependencies and close this.

Cool!
I think we want to keep this as optional instead of installing the dependencies by default, since it's an experimental feature and not really needed for the core product.
I'm gonna go ahead and close the issue, cc @izeigerman


Add unsupported style and parameter attributes to the export

Not all IMSC/TTML style and parameter attributes are supported yet. If an input file contains unsupported attributes, they should be at least be exported with an equivalent value compared to original file.

following attributes added to the export:

overflow
padding
position
ruby
rubyAlign
rubyPosition
rubyReserve
textCombine
textDecoration
shear
linePadding
disparity (still not supported at imscJS)


Algorithm: Inner product

Thanks for your contribution but it looks like this method has already been contributed before.


[BUG] Llamalend squeeze bot: I can't see my bot in mybots api

just tested the Llamalend squeeze bot and the bot was created successfully. But I can't see my bot in mybots api result.
https://etherscan.io/tx/0x471ae8ed1c6e50db5e68e8e2ac6c210e52659c229d4b6e81885e0e922509a394

https://service.palomabot.ai/robots shows it active


traceur.get alternatives

This function has been critical to allow creating custom transformers for use with Traceur given the current API.
For an example see how we compile CommonJS modules in SystemJS builder at https://github.com/systemjs/builder/blob/master/compilers/cjs.js.
The current workaround involves replacing this use of traceur.get with eg:
$traceurRuntime.ModuleStore.get('traceur@0.0.95/src/codegeneration/ParseTreeTransformer.js');
Where $traceurRuntime is the environment global.
I tried adding everything needed to compiler-imports.js but it quickly becomes obvious that this isn't an approach that will scale, although can provide the PR to cover my own use cases if that is preferred over the traceur.get API?
Otherwise perhaps a comprehensive export * mechanism is needed here to provide all the namespaced APIs for transformer creation?

Far and away the best solution to accessing eg ParseTreeTransformer is
import {ParseTreeTransformer} from './ParseTreeTransformer.js';

and transcoding especially in the cjs example.  Maybe we can do something to make it easier to take this path?
Next best would be expanding the API via more export calls.  But as you note our API is (deliberately) incomplete. Perhaps we should reduce our use of the API now that we have es6 based testing, to further encourage transpiling modules over es5 API calls? That said, I'm not opposed to expansion if the alternative is very painful.
I wouldn't even put $traceurRuntime.ModuleStore.get on the list, I thought it was internal API only global because we have to do that for bootstrap.

Far and away the best solution to accessing eg ParseTreeTransformer is
import {ParseTreeTransformer} from './ParseTreeTransformer.js';

and transcoding especially in the cjs example.  Maybe we can do something to make it easier to take this path?
Next best would be expanding the API via more export calls.  But as you note our API is (deliberately) incomplete. Perhaps we should reduce our use of the API now that we have es6 based testing, to further encourage transpiling modules over es5 API calls? That said, I'm not opposed to expansion if the alternative is very painful.
I wouldn't even put $traceurRuntime.ModuleStore.get on the list, I thought it was internal API only global because we have to do that for bootstrap.

It's really about if Traceur wants to encourage users to create their own transformers with it. I've already built my transpilers with Traceur so that's the reason for me, but I also feel that Traceur is potentially a faster compiler than Babel for these cases (although haven't done the deeper performance analysis).
I do think extending the API sounds like the best way here. Let me look again at doing a PR for my needs. In terms of scaling the approach, I think just being able to structure it into subfiles to allow export * would help manage this file better. I can possibly attempt something along these lines as well.

So you don't think users will use es6 to create transformers?
Subfiles that matched the namespace hierarchy, eg codegeneration for traceur.codegeneration would make sense to me.

We used to map everything into es5 style namespace objects. We went away from that because es modules provided a cleaner abstraction.
What we should do is allow import {ParseTreeTransformer} from 'traceur/src/codegeneration/ParseTreeTransformer.js' in an npm environment.

If allowing the above, then perhaps it would make sense to do a separate file dist build of Traceur as well?
Along the lines of -
traceur --dir src dist --modules=commonjs

The issue otherwise is the graph duplication of having to load effectively two versions of Traceur - one for the transformer and one for the main global itself.

Yes! We should do that!
I would probably compile it to dist/commonjs and also make sure we include src/ in the npm output.

Snags. We need to add a way to not include all files in a directory since we have those *.js-template.js files.

With those 2 changes and ./traceur --dir src/ dist/ --modules=commonjs the following works as expected:
'use strict';

require('./bin/traceur-runtime.js');

const ParseTreeTransformer = require('./dist/codegeneration/ParseTreeTransformer.js').ParseTreeTransformer;
const Script = require('./dist/syntax/trees/ParseTrees.js').Script;
const parseStatements = require('./dist/codegeneration/PlaceholderParser.js').parseStatements;
const parseExpression = require('./dist/codegeneration/PlaceholderParser.js').parseExpression;
const STRING = require('./dist/syntax/TokenType.js').STRING;
const LiteralExpression = require('./dist/syntax/trees/ParseTrees.js').LiteralExpression;
const LiteralToken = require('./dist/syntax/LiteralToken.js').LiteralToken;
const IdentifierExpression = require('./dist/syntax/trees/ParseTrees.js').IdentifierExpression;
const IdentifierToken = require('./dist/syntax/IdentifierToken.js').IdentifierToken;
const BindingIdentifier = require('./dist/syntax/trees/ParseTrees.js').BindingIdentifier;
const createUseStrictDirective = require('./dist/codegeneration/ParseTreeFactory.js').createUseStrictDirective;

console.log(
    ParseTreeTransformer,
    Script,
    parseStatements,
    parseExpression,
    STRING,
    LiteralExpression,
    LiteralToken,
    IdentifierExpression,
    IdentifierToken,
    BindingIdentifier,
    createUseStrictDirective);

@arv how does the performance compare for startup time?

@guybedford. Good  question. I haven't checked. I'll run some tests when I get the time.

@guybedford Using the code snippet in the comment above. test.js is using dist and and test2.js is using traceurGet from https://github.com/systemjs/builder/blob/master/compilers/cjs.js:
$ time node test.js
120ms

real	0m0.175s
user	0m0.153s
sys	0m0.027s

$ time node test2.js
256ms

real	0m0.312s
user	0m0.278s
sys	0m0.042s

In other words, performance is better in your case.
Changing this to import src/traceur.js which pulls in almost everything I get:
$ time node test.js
237ms

real	0m0.295s
user	0m0.271s
sys	0m0.036s

$ time node test2.js
249ms

real	0m0.311s
user	0m0.278s
sys	0m0.042s

So the performance is still in favor for the dist version.


Added Center property to PixelLine.cs

Fixes #4318
Hi! Just added a center property based on the issue above.
If the center property was just for the Pixel 1, please let me know and I can fix it.

This looks great @brokoli777, thanks! 🚀


Multiple API permission selection during role creation under Applciation->Roles tab is not clear

Describe the issue:

Create an application
Authorize to few APIs by going to the application's "Authorize API" tab
Go to the Roles tab of the application. select the application audience
Try to create a role by selecting permissions from multiple APIs.
Once you assign permission from one API, and try to select another API the first one will disappear. Feels like only one API's permissions can be assigned. But when you check on the created role, the selected permissions of multiple APIs were added to the role.

https://github.com/wso2/product-is/assets/25483865/97cb88de-f958-4243-82f8-9e65542c520c
If you check the permission assignment of role creation under User Management -> Roles, it clearly shows the selected APIs and permissions
https://github.com/wso2/product-is/assets/25483865/bf632bcf-9d6c-4b3c-b890-1dd381f83898

Fix: https://github.com/wso2/identity-apps/pull/4820

Verified in latest wso2is-7.0.0-rc1-SNAPSHOT
https://github.com/wso2/product-is/assets/37938529/1487bacf-8871-4987-b9c9-2e3eafe8aaef


fix for request.log exceptions

When a user is not passing a log BaseRequestHandler we are using Dummy object by default but later on in _resolve_call_info we are using bind function from log to bind some values to log
If user did not provide a log then it would be dummy object bind function which will return Nonetype
This Nonetype object is stored in request.log which is being used in handle request to log something
as request.log is None it is throwing following errors

NoneType object has no attribute 'exception'
NoneType object has no attribute 'info'

@jaswanth098 here is the behavior I see
(venv) root@prashanth-dev:~# python
Python 3.6.7 (default, Oct 22 2018, 11:32:17)
[GCC 8.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import deeputil
>>> log = deeputil.Dummy()
>>> log.bind()
<deeputil.misc.Dummy object at 0x7fd3c7357908>
>>> log = log.bind()
>>> log
<deeputil.misc.Dummy object at 0x7fd3c7357c18>
>>> log.info('sdsdsd', a=10)
<deeputil.misc.Dummy object at 0x7fd3c7357898>

Based on this, I don't understand why there should be a problem. What am I missing?

@prashanthellina Got it we were using the older version of deeputil where it was returning None when we use log.bind()
(dev) root@dev ~/Goutham-new/kwikui_new2/kwikui/test # python
Python 3.5.2 (default, Nov 12 2018, 13:43:14)
[GCC 5.4.0 20160609] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from deeputil import Dummy
>>> log = Dummy()
>>> log = log.bind()
>>> log.info("testing")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: 'NoneType' object has no attribute 'info'

We will update the version of deeputil
Thanks for the information


[BUG] limit can not push down to tidb when cast expression appears

Describe the bug

When the expression of org.apache.spark.sql.catalyst.expressions.Cast appears in parsing SQL, limit cannot be pushed down.
What did you do

first , in tidb create a table :
CREATE TABLE `test2` (
  `id` int(10) NOT NULL,
  `name` varchar(255) NOT NULL,
  `age` decimal(20,0) DEFAULT NULL,
  PRIMARY KEY (`id`) /*T![clustered_index] CLUSTERED */
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_bin

than ,use tispark run sql :
select * from test.test2 where age > 0.0 limit 1;

We can see that the limit can not push down to tidb.
What do you expect

Limit can push down to tidb when cast expression appears.
What happens instead

Spark and TiSpark version info

tispark : 2.4.3
spark : 2.4.0

The problem is here. When converting GreaterThan to TiExpression during spark parsing, match fails because org.apache.spark.sql.catalyst.expressions.Cast cannot be converted.
https://github.com/pingcap/tispark/blob/2f5badb5bd0d90cd5a4d5f9bc5facc69d096e6c1/core/src/main/scala/org/apache/spark/sql/catalyst/expressions/BasicExpression.scala#L114
My question is whether it is necessary to add a tiexpression's Cast to tidb to adapt to this situation.
@shiyuhang0 @xuanyu66  cc

Supplementary explain information

I will check it today


Support for Eufy T1016 Lumos 2.0 Tunable Bulb

NOTE: For a feature to be created it has to be doable in the Amazon app because we are basically replicating the app for HA. If you can't find it in the app, we probably can't create it here.
Is your feature request related to a problem? Please describe.
Support for Eufy T1016 Lumos 2.0 Tunable Bulb, as this bulb can be controlled fine via the Alexa app.
Describe the solution you'd like
It appears that this device should be able to be controlled via this addon, though it shows as unsupported. It does NOT have the capability of Alexa.ColorController, though it does have a capability of Alexa.BrightnessController and Alexa.ColorTemperatureController, which it appears alexa_entity.py is looking for. This particular lightbulb is a tunable white bulb and not a full color bulb, hence why Alexa.ColorController isn't part of its support.
Describe alternatives you've considered
n/a
Additional context
Output from the add-on in debug mode:
2023-06-22 09:04:09.678 DEBUG (MainThread) [custom_components.alexa_media.alexa_entity] Found unsupported device
{
  "applianceId": "SKILL_eyJza2lsbElkIjoiYW16bjEuYXNrLnNraWxsLjBkNDJmYzEyLTI3ZDAtNGIzNy1iMzYyLTg4ODExNzI1MDxxxxxxjoibGl2ZSJ9_51xxxxbfa7",
  "endpointTypeId": "",
  "driverIdentity": {
    "namespace": "SKILL",
    "identifier": "eyJza2lsbElkIjoiYW16bjEuYXNrLnNraWxsLjBkNDJmYzEyLTI3ZDAtNGIzNy1iMzYyLTgxxxxxMyIsInN0YWdlIjoibGl2ZSJ9"
  },
  "manufacturerName": "EufyHome",
  "friendlyDescription": "T1016 Lumos 2.0 Tunable Bulb, connected via EufyHome",
  "modelName": "",
  "deviceType": "CLOUD_DISCOVERED_DEVICE",
  "version": "0",
  "friendlyName": "Office Lamp",
  "friendlyNameModifiedAt":<PHONE_NUMBER>429,
  "capabilitiesModifiedAt":<PHONE_NUMBER>612,
  "ipAddress": "",
  "port": "",
  "entityId": "43ee1006-c2da-464b-a872-fa3xxxxxc",
  "applianceNetworkState": {
    "reachability": "REACHABLE",
    "lastSeenAt":<PHONE_NUMBER>858,
    "createdAt":<PHONE_NUMBER>612,
    "lastSeenDiscoverySessionId": {
      "value": "20452be7-98cd-4803-8b60-0cxxxxx1"
    }
  },
  "tags": {
    "tagNameToValueSetMap": {
      "groupIdentity": [
        "amzn1.HomeAutomation.ApplianceGroup.AY817KFE6MFW1.6da58906-99d0-4642-866c-8xxxxxx5a"
      ]
    }
  },
  "additionalApplianceDetails": {
    "additionalApplianceDetails": {
      "eufy_product_code": "T1016"
    }
  },
  "firmwareVersion": "0",
  "actions": [],
  "capabilities": [
    {
      "capabilityType": "AVSInterfaceCapability",
      "type": "AlexaInterface",
      "version": "3",
      "interfaceName": "Alexa"
    },
    {
      "capabilityType": "AVSInterfaceCapability",
      "type": "AlexaInterface",
      "version": "3",
      "properties": {
        "supported": [
          {
            "name": "powerState"
          }
        ],
        "proactivelyReported": true,
        "retrievable": true,
        "readOnly": false
      },
      "interfaceName": "Alexa.PowerController"
    },
    {
      "capabilityType": "AVSInterfaceCapability",
      "type": "AlexaInterface",
      "version": "3",
      "properties": {
        "supported": [
          {
            "name": "colorTemperatureInKelvin"
          }
        ],
        "proactivelyReported": true,
        "retrievable": true,
        "readOnly": false
      },
      "interfaceName": "Alexa.ColorTemperatureController"
    },
    {
      "capabilityType": "AVSInterfaceCapability",
      "type": "AlexaInterface",
      "version": "3",
      "properties": {
        "supported": [
          {
            "name": "brightness"
          }
        ],
        "proactivelyReported": true,
        "retrievable": true,
        "readOnly": false
      },
      "interfaceName": "Alexa.BrightnessController"
    },
    {
      "capabilityType": "AVSInterfaceCapability",
      "type": "AlexaInterface",
      "version": "3",
      "properties": {
        "supported": [
          {
            "name": "connectivity"
          }
        ],
        "proactivelyReported": true,
        "retrievable": true,
        "readOnly": false
      },
      "interfaceName": "Alexa.EndpointHealth"
    }
  ],
  "applianceTypes": [
    "LIGHT"
  ],
  "isEnabled": true,
  "aliases": [],
  "connectedVia": "",
  "alexaDeviceIdentifierList": [],
  "applianceKey": "43ee1006-c2da-464b-a872-faxxxxxxxc",
  "identifiers": {
    "networkInterfaceIdentifiers": []
  },
  "applianceDriverIdentity": {
    "namespace": "SKILL",
    "identifier": "eyJza2lsbElkIjoiYW16bjEuYXNrLnNraWxsLjBkNDJmYzEyLTI3ZDAtNGIzNy1iMzYyLTg4ODExNzI1MDxxxxxxxx2ZSJ9"
  },
  "ipaddress": "",
  "applianceLambdaControlled": true,
  "mergedApplianceIds": [
    "SKILL_eyJza2lsbElkIjoiYW16bjEuYXNrLnNraWxsLjBkNDJmYzEyLTI3ZDAtNGIzNy1iMzYyLTg4ODExNzI1MxxxxxxxoibGl2ZSJ9_514xxxxxxebfa7"
  ]
}

Thanks for the good report along with the json capabilities. Hopefully someone can easily put in a PR to add it.


sudo: cyber_recorder: command not found (in Apollo6.0)

System information

Linux Ubuntu 18.04):
**https://gitee.com/ApolloAuto/apollo**:
Apollo version (6.0):
Output of apollo.sh config if on master branch:

Steps to reproduce the issue:

Please use bullet points and include as much details as possible:
failed to run demo
1、i have tried this command "source scripts/apollo_base.sh" ,  but it don't work.

Supporting materials (screenshots, command lines, code/script snippets):

find something helpful

but still not work

sloved
no sudo ,it can run
cyber_recorder play -f docs/demo_guide/demo_3.5.record --loop


Most ADVANCED actions do not make sense for the controller

I've highlighted them red:

The firmware upgrade may become useful in the future but it will work differently than the OTA upgrade for nodes.

Them are just a way to make some operations faster, after you select one of them you will be asked to insert the nodeId.
Instead you should firstly find the node in the table, expand the row, click on advanced button

Uh this is on the advanced actions button under the controller node. Not the one in the top corner.

Shit. 🤣

Fix is coming


Metal Gear Solid

Cutscenes play very slow (1~2 FPS), the game plays normally.
Happens with all Wine/Proton versions tested.

I think the solution is to add DLL override WINEDLLOVERRIDES=ddraw=n

Fixed in cc0a27a21100310bc96b4d50f25949655853c815
I haven't tested if dinput that ships in latest version breaks anything, feel free to add necessary changes. If dinput is broken I'd suggest to add it as dinput=n,b for compatibility with old builds that don't ship it

Is keeping compatibility with older builds kinda pointless when we're shipping Protonfixes with Proton itself? Usage of Protonfixes requires staying on the very lastest version at any time.

I meant the game build. When you compare GOG builds https://www.gogdb.org/product/1771973390#builds
the latest one introduces dinput.dll.
I think I'll add dinput override for now, we'll see if any issues appear


A customizable skin

This is an enhancement or feature.

Summary
This PR implements a feature that users can make their skins.
I have moved all color-related properties into a single file _sass/so-simple/_skin.scss, and made a default skin file assets/css/skins/default.scss:
---
---

@charset "utf-8";

@import "so-simple/functions";
@import "so-simple/variables";
@import "so-simple/notices";

/* Colors */
$base-color: #000 !default;
$text-color: #000 !default;
$accent-color: #34374C !default;
$nav-color: #fff !default;
$background-color: #fff !default;
$nav-background-color: #000 !default;
$code-background-color: tint($base-color, 95%) !default;
$border-color: tint($base-color, 80%) !default;

$primary-color: #000 !default;
$success-color: #62c462 !default;
$warning-color: #f89406 !default;
$danger-color: #ee5f5b !default;
$info-color: #52adc8 !default;

/* Syntax highlighting (base16) colors */
$base00: #fafafa !default;
$base01: #073642 !default;
$base02: #586e75 !default;
$base03: #657b83 !default;
$base04: #839496 !default;
$base05: #586e75 !default;
$base06: #eee8d5 !default;
$base07: #fdf6e3 !default;
$base08: #dc322f !default;
$base09: #cb4b16 !default;
$base0a: #b58900 !default;
$base0b: #859900 !default;
$base0c: #2aa198 !default;
$base0d: #268bd2 !default;
$base0e: #6c71c4 !default;
$base0f: #d33682 !default;

@import "so-simple/skin";

A skin file would be referenced in the template head.html according to the user's configuration:
<link rel="stylesheet" href="{{ '/assets/css/main.css' | relative_url }}">
<link rel="stylesheet" href="{{ site.skin | default: '/assets/css/skins/default.css' | relative_url }}">

If the user wants to use a custom skin, he or she just needs to copy default.css, and rename it, and override these color variables.
I have also made two additional skins: light.scss and dark.scss.
I think this implementation is very flexible. Users can even select different skins according to different hours in a day, for example, they can write below scripts in their head-custom.html:
<script>
    var hours = (new Date()).getHours();
    var node = document.createElement('link');
    node.rel = 'stylesheet';
    if (hours >= 6 && hours < 18) {
      node.href = "{{ '/assets/css/skins/light.css' | relative_url }}";
    } else {
      node.href = "{{ '/assets/css/skins/dark.css' | relative_url }}";
    }
    document.getElementsByTagName('head')[0].appendChild(node);
</script>

Context

#344

Great work on this pull request @alxddh!
I'd like some time to give it a better review but everything is looking good from my point of view. Might want to adjust some of the color variables for the light and dark skins, but overall I think it's solid.

Great work on this pull request @alxddh!
I'd like some time to give it a better review but everything is looking good from my point of view. Might want to adjust some of the color variables for the light and dark skins, but overall I think it's solid.

Thanks. Looking forward to your follow-up improvement.


MLIBZ-1476 Check that storage adapter is supported correctly

Description
This PR checks if a storage adapter is supported asynchronously.
Changes

Update isSupported for storage adapters.
Add CookieStorage storage adapter.

Current coverage is 20.49% (diff: 11.29%)

Merging #5 into master will decrease coverage by 2.36%

@@             master         #5   diff @@
==========================================
  Files            12         12
  Lines           936       1093   +157
  Methods         129        158    +29
  Messages          0          0
  Branches        134        148    +14
==========================================
+ Hits            214        224    +10
- Misses          722        869   +147
  Partials          0          0

Powered by Codecov. Last update d220ee3...0d2f754


Fixes #328

The build is failing because tslint failed on commentFormatRuleTests.ts. Please run ensure the project builds (simply via grunt) on your local machine.

Fixed.

Please also update the readme to include this new option.

No problems


documentation build

I guess in the near future we want to provide a generated online documentation, preferable automatically (e.g. travis build uploaded to s3).
Maybe Adam's new documentation generator is worth looking at.

Maybe Adam's new documentation generator is worth looking at.

Have you build something with Adam's docs?

Nope, was just an idea. However I have done this Travis to s3 many times, so if you have a favorite doc generator, I would be happy to hack something together next week.
On March 31, 2016 5:44:25 PM GMT+03:00, Ilya Yaroshenko<EMAIL_ADDRESS>wrote:

Maybe Adam's new documentation generator is worth looking at.

Have you build something with Adam's docs?

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/DlangScience/mir/issues/32#issuecomment-203970511

Nope, was just an idea. However I have done this Travis to s3 many times, so if you have a favorite doc generator, I would be happy to hack something together next week.

I like https://github.com/kiith-sa/harbored-mod and Adam's doc engine looks good too. Mir requires all DDOC macros from Phobos. This will add some problems.

Mir requires all DDOC macros from Phobos. This will add some problems.

Hmm yeah this is going to be tricky, but maybe taking the macro definition from the dlang.org repo is already enough.
I am more worried about references to Phobos modules, so when using REF the link won't resolve, but perhaps as a workaround a simple regex could find these occurrences
An alternative would be to use the setup from dlang.org, but modified to only build Mir.
The advantages here is that we avoid ui bugs due to different documentation engines if a module will be merged to Phobos and their css design is pretty sweet.
On March 31, 2016 5:58:11 PM GMT+03:00, Ilya Yaroshenko<EMAIL_ADDRESS>wrote:

Nope, was just an idea. However I have done this Travis to s3 many
times, so if you have a favorite doc generator, I would be happy to
hack something together next week.

I like https://github.com/kiith-sa/harbored-mod and Adam's doc engine
looks good too. Mir requires all DDOC macros from Phobos. This will add
some problems.

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/DlangScience/mir/issues/32#issuecomment-203975728

as bootDoc is working again - do you want to add that back to the Readme?
Btw should we archive every documentation version? We could just execute the s3 copy command to the git describe --abbrev=0 --tags (latest tag) too, but then we should probably provide a nice way for the user to switch between documentations and at the moment we didn't introduce any breaking changes  - so just an idea once the first official version is released.

as bootDoc is working again - do you want to add that back to the Readme?

We still need to add macros like LREF and others.

Btw should we archive every documentation version? We could just execute the s3 copy command to the git describe --abbrev=0 --tags (latest tag) too, but then we should probably provide a nice way for the user to switch between documentations and at the moment we didn't introduce any breaking changes - so just an idea once the first official version is released.

This would be helpful

After testing ddox, bootDoc, harbored-mod I gave@adamdruppe's new adrdox a try:
http://docs.mir.dlang.io/adrdox/mir.combinatorics.html
It's still in an experimental stage, but I really like the fact how easy it is to give it a custom skeleton :)

Looks good

I like https://github.com/kiith-sa/harbored-mod and Adam's doc engine looks good too

Seems like harbored is working again.
I gave it a try too: http://docs.mir.dlang.io/harbored/

Fixed with #203


updates missing

i tried getting updates for little big planet 3 and everything was in order but the 1.13 update was missing
i changed the download location but that just made more updates dissapear
help please

I'm gonna need you to post a .log file after downloading an update that disappears

For reference, that'd be a text file named rusty-psn_ and the date and hour when you launched the app, located alongside the .exe file.

Run rusty-psn, reproduce your issue, then close it and upload that file here.

the file is too big
how do i send it?

If compressing the log file doesn't help, then upload it somewhere else (like Google drive) and post a link to it

Is this still an issue? It's been a while


Ensure that all grpc errors include the message if available

In some cases, only the error code is being forwarded to the caller. This makes debugging and identifying problems difficult. Potential sources for this omission include the unary calls in threads-client and threads-network-client, as well as any streaming calls in those two clients.

This has now been done here and in https://github.com/textileio/js-hub


[Question] Integration with Azure DevOps / Pipeline

Hello,
I'm in the middle of selecting test framework for canvas app.
Looks like this project covers all my needs - easy to use, tests from studio are portable, simple configuration.
My next step is to integrate it with CI in Azure DevOps.
I couldn't find any instruction how to do this correctly.
Have you tired this so you can provide some simple guid?

Hi @arexms, we don't currently have any guides for incorporating the current PowerAppsTestEngine into CI\CD, but this is definitely something we're considering for the future. I'd suggest waiting for incorporation into a more fully supported solution like the PAC CLI or a Github Action, but if you want to take a look at what this might look like today then I'd suggest looking at how we're testing this component in the build action for this repository which runs Test Engine against some example canvas apps.


SD: Formatting issue with 1-line mode, SPI works fine

I noticed the following: When i try to save some dump (generated by network traffic / promiscuous mode) in 1-line mode to my sd card, the format is broken. Looks like this:
000068 02 0C 00 0B 05 02 00 0C
000070 00 00 46 05 32 08 01 00
00007878 00 2D 1A BC 09 1B FF FF

0000D8 D4 6C 38 00 ED FF000000
80 00 00 00F F FF FF FF
000008 FF FF 8C 34 FD 31 08 9F

Just as an example. Some pakets are 'normal' and some others looks like this. Maybe a paragraph is missing, maybe a space, maybe there are 3 chars printed, etc.
So i tried the sd card in SPI. And here is everythings fine. My code is here.
Basically i put the buffer from the promiscuous callback into a ringbuffer and fomat the buffer in a nice hexdump with the following code
for (int i = 0; i < packet->rx_ctrl.sig_len; i++) {
    if (i % 8 == 0) {
        //Offset hinzufügen
        snprintf(hexbuf, sizeof(hexbuf), "%06X ", offset);
        strcat(hexbuf_dump, hexbuf);
        offset += 8;
    }
    //Hex anfügen
    snprintf(hexbuf, sizeof(hexbuf), "%02X ", packet->payload[i]);
    strcat(hexbuf_dump, hexbuf);
    //Absatz, wenn: 8x Hex oder Frame zuende
    if ((i % 8 == 7) || ((i + 1) == packet->rx_ctrl.sig_len)) {
        snprintf(hexbuf, sizeof(hexbuf), "\n");
        strcat(hexbuf_dump, hexbuf);
    }
}
sprintf(filenamebuf, "/sdcard/%s.pcap", filename);
printf("%s", hexbuf_dump);
savestuff(hexbuf_dump);
memset(&hexbuf_dump[0], 0, sizeof(hexbuf_dump));
offset = 0;

saving function is here
void savestuff(const char* data)
{
    //fprintf("%c", data);
    sprintf(filenamebuf, "/sdcard/%s.pcap", filename);
    ESP_LOGI(TAG, "Opening file");
    FILE* f = fopen(filenamebuf, "a");

    if (f == NULL) {
        ESP_LOGE(TAG, "Fehler! Datei konnte nicht geoeffnet werden!");
        digitalWrite(led1, HIGH);
        vTaskDelay(500 / portTICK_PERIOD_MS);
        digitalWrite(led1, LOW);
        ESP.restart();
    }
    ESP_LOGI(TAG, "File open");
    //fprintf(f, "%s", data);
    fwrite(data, sizeof(char), strlen(data), f);
    fclose(f);
    ESP_LOGI(TAG, "File written");

}

One of the issues you may be seeing, where some characters appear doubled, has been recently fixed in b260f6cb25bf7ab9be907fe4fdebaac7aacb8900. Please try updating IDF and check if it works any better.

I updated the IDF (https://github.com/espressif/esp-idf/releases -> Obtaining v2.1 -> using git), but in the new folder 'esp-idf-v2.1' are still the old files (i have compared the content of those files with your updated versions). Also in the .zip file are only the old files.

Well, thanks. That works perfectly.


Improved exception handling

In order to run against large quantities of content, we need better exception handling, with custom exceptions for the errors that seem likely to occur (e.g. local file not found, etag mismatch, communication error).

@jwestgard First chunk of additional exceptions and exception handling. I'm sure more cases will come up under further testing, but this is most of the major ones (local files not found and profile or S3 errors).


Examine plugin positive overflow

Describe the bug
Examine plugin is overflowing when the total price is over max integer value.
To Reproduce
Steps to reproduce the behavior:

Enable the examine plugin
Examine some item stack valued at over max integer value

Expected behavior
Should be displaying the real value instead of a negative value (overflowing)
Screenshots
https://github.com/runelite/runelite/blob/f8b801f6cdad414a66349b808f45f7eda78dd8b0/runelite-client/src/main/java/net/runelite/client/plugins/examine/ExaminePlugin.java#L359
https://github.com/runelite/runelite/blob/f8b801f6cdad414a66349b808f45f7eda78dd8b0/runelite-client/src/main/java/net/runelite/client/plugins/examine/ExaminePlugin.java#L379
Environment (please complete the following information):

OS: Ubuntu 18.04.3 LTS
RuneLite version: <IP_ADDRESS>
Launcher version: 1.6.2

I fixed this and put in a pull request. The same overflow occurs with the mouse-over price text as well


Microsoft.Cpp.Default.props was not found

🚨 The issue tracker is not for questions 🚨
I am not able execute a .cpp in vs code
using MSBUILD properties in taks.json
when i tries to build the application i am getting an error
Microsoft.Cpp.Default.props was not found
"D:\Coding\Data Structures\Learn\Learn\Learn.vcxproj" (build target) (1) ->
  D:\Coding\Data Structures\Learn\Learn\Learn.vcxproj(28,3): error MSB4019: The
 imported project "D:\Microsoft.Cpp.Default.props" was not found. Confirm that
the path in the <Import> declaration is correct, and that the file exists on di
sk.

0 Warning(s)
1 Error(s)

Where tasks.json
{
    // See https://go.microsoft.com/fwlink/?LinkId=733558
    // for the documentation about the tasks.json format
    "version": "2.0.0",
    "tasks": [
        {
            "label": "build",
            "type": "shell",
            "command": "msbuild",
            "args": [
                // Ask msbuild to generate full paths for file names.
                "/property:GenerateFullPaths=true",
                "/t:build"
            ],
            "group": "build",
            "presentation": {
                // Reveal the output only if unrecognized errors occur.
                "reveal": "silent"
            },
            // Use the standard MS compiler pattern to detect errors, warnings and infos
            "problemMatcher": "$msCompile"
        }
    ]
}

"D:\Coding\Data Structures\Learn\Learn\Learn.vcxproj" (build target) (1) ->
D:\Coding\Data Structures\Learn\Learn\Learn.vcxproj(28,3): error MSB4019: The
imported project "D:\Microsoft.Cpp.Default.props" was not found. Confirm that
the path in the  declaration is correct, and that the file exists on di
sk.
0 Warning(s)
1 Error(s)

This means that the build task is running correctly and it is in fact finding a build error. For general help and "how to" questions checkout Stack Overflow using the tag vscode.


Issue #122 - Added sh.mv with tests and documentation

New request -_- Sorry about the last one.

This is a pretty good patch!
@modeswitch Is this approach to mv what we want? I'm fairly sure fs.rename can do the majority of the work for us though I'm still investigating. I need this for #273 , so if rename isn't enough I'll probably update this patch instead of the one I'm developing.

I had a quick look at it seems reasonable. The code probably needs to be updated to bring it in line with recent refactoring.
Can you give it a review (adding comments to the patch) and we can discuss any changes you feel are required. If most of the work is done already, let's use this.

@modeswitch The mv utility can accept multiple source files to be moved to a directory. Do we want to support this?

The new version of this PR is #346

The POSIX standard says that the mv utility relies on rename(), except for cross-filesystem moves. At the moment, this patch is doing the rename() work manually, but it's working. Should I switch it to use fs.rename()?

This can't be automerged. Can you investigate?

@modeswitch Here's the updated PR: https://github.com/filerjs/filer/pull/346
It's missing a test case, which shouldn't be too difficult to add. Other than that I believe it's ready for review.


Updates for v3.12.2

Description

Related issues/PRs

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

Deploy preview for calico ready!
Built with commit 27590fa81199cc45879c44ff48d5da222ed50d28
https://deploy-preview-3405--calico.netlify.com


Design Doc for PaddlePaddle TrainingJob k8s CRD

hi, @typhoonzero @Yancey1989 ,  I'd like you to review this issue. In this issue. This is the design doc for paddlepaddle cluster job.

too many commits for one pr, you'd better rebase and merge the commits.

@typhoonzero , I fix the comments  you mentioned above. How do you think about this commit?


Instaling MRTK after StreamVR Plugin causes a compile error.

Describe the bug
When instaling MRTK after SteamVR Plugin, existing openvr_api will be overridden and causes a compile error.
To reproduce

Install SteamVR Plugin
Install MRTK Foundation.
Many compile errors.

Expected behavior
Screenshots

Your setup (please complete the following information)

Unity 2019.1.14f1
MRTK v2.3
StreamVR Plugin v2.5.0

Target platform (please complete the following information)

HoloLens 2
OpenVR

Additional context

Assets/SteamVR/Plugins/openvr_api.cs
MixedRealityToolkit.Providers/OpenVR/Headers/openvr_api.cs

Their paths are different, but they have the same GUID.
UnityPackage has a tendency to override the same GUID regardless of the path.
So,I think the GUID of openvr_api.cs included in MRTK needs to be changed.
regards.

but they have the same GUID.

Ah shoot, I thought I had changed this. I'll update

Though, what do the compile errors look like? I wouldn't expect that out of a GUID clash.
Is it actually because two of the same script exist?

Still happens for SteamVR 1.2.3. The errors now are:
Assets\MixedRealityToolkit.Providers\OpenVR\OpenVRRenderModel.cs(10,45): error CS0234: The type or namespace name 'Headers' does not exist in the namespace 'Microsoft.MixedReality.Toolkit.OpenVR' (are you missing an assembly reference?)
Assets\MixedRealityToolkit.Providers\OpenVR\OpenVRRenderModel.cs(235,45): error CS0246: The type or namespace name 'CVRRenderModels' could not be found (are you missing a using directive or an assembly reference?)
...

@eusebiu Which version of MRTK were you using? This should be resolved in our upcoming 2.4 release.

@eusebiu Which version of MRTK were you using? This should be resolved in our upcoming 2.4 release.

2.3.0


The hydra module in Nixpkgs doesn't honor the "port" option

# nixos-option services.hydra.port
Value:
8080

# systemctl status hydra-server.service
● hydra-server.service
   Loaded: loaded (/nix/store/vca1ggrsj7krp4jzcqxb8s39y2bjya5k-unit-hydra-server.service/hydra-server.service; bad; vendor preset: enabled)
   Active: active (running) since Wed 2016-06-08 21:24:23 CEST; 30min ago
 Main PID: 1392 (starman master )
   CGroup: /system.slice/hydra-server.service
           ├─1392 starman master
           ├─1406 starman worker
           ├─1407 starman worker
           ├─1408 starman worker
           ├─1409 starman worker
           └─1410 starman worker

Jun 08 21:24:25 hydra hydra-server[1392]: Unknown option: f
Jun 08 21:24:28 hydra hydra-server[1392]: [warn] Unicode::Encoding plugin is auto-applied, please remove this from your appclass and make sure to define "encoding" config
Jun 08 21:24:39 hydra hydra-server[1392]: DEPRECATION WARNING: The Regex dispatch type is deprecated.
Jun 08 21:24:39 hydra hydra-server[1392]:   It is recommended that you convert Regex and LocalRegex
Jun 08 21:24:39 hydra hydra-server[1392]:   methods to Chained methods. at /nix/store/ykp208nmg541lrsqbi621ba62nlm109f-hydra-perl-deps/lib/perl5/site_perl/5.22.2/Catalyst/DispatchType/Regex.pm line 210.
Jun 08 21:24:47 hydra hydra-server[1392]: 2016/06/08-21:24:47 Starman::Server (type Net::Server::PreFork) starting! pid(1392)
Jun 08 21:24:47 hydra hydra-server[1392]: Resolved [*]:3000 to [<IP_ADDRESS>]:3000, IPv4
Jun 08 21:24:47 hydra hydra-server[1392]: Binding to TCP port 3000 on host <IP_ADDRESS> with IPv4
Jun 08 21:24:47 hydra hydra-server[1392]: Setting gid to "122 122 122"
Jun 08 21:24:47 hydra hydra-server[1392]: Starman: Accepting connections at http://*:3000/

The hydra-server listens on port 3000 regardless of what I configure.
Ping @domenkozar.

Also, note the "Unknown option: f". That doesn't seem right.

@peti could you also cat /nix/store/vca1ggrsj7krp4jzcqxb8s39y2bjya5k-unit-hydra-server.service/hydra-server.service?

[Unit]
After=hydra-init.service
Requires=hydra-init.service

[Service]
Environment="COLUMNS=80"
Environment="HYDRA_CONFIG=/var/lib/hydra/hydra.conf"
Environment="HYDRA_DATA=/var/lib/hydra"
Environment="HYDRA_DBI=dbi:Pg:dbname=hydra;user=hydra;"
Environment="HYDRA_TRACKER="
Environment="LOCALE_ARCHIVE=/nix/store/kn6aljmxnifxa6q1arhbpdy7dw19pghw-glibc-locales-2.23/lib/locale/locale-archive"
Environment="NIX_REMOTE=daemon"
Environment="NIX_REMOTE_SYSTEMS=/etc/nix/machines"
Environment="PATH=/nix/store/iawkwcizv0b10q41k3f0pr1ggdx7v8rg-coreutils-8.25/bin:/nix/store/kjfz70ng7lsln6a0czccmdhm0s88fsxw-findutils-4.6.0/bin:/nix/store/phlj8c4bl0bgijm287lbv7sizb4z0dfk-gnugrep-2.25/bin:/nix/store/9xq9cj2dcvdmmm6nxi0xyz3v8v0x1l1v-gnused-4.2.2/bin:/nix/store/3k7lx2fx36jfpwwih0i3pdsmsqnbr9cg-systemd-230/bin:/nix/store/iawkwcizv0b10q41k3f0pr1ggdx7v8rg-coreutils-8.25/sbin:/nix/store/kjfz70ng7lsln6a0czccmdhm0s88fsxw-findutils-4.6.0/sbin:/nix/store/phlj8c4bl0bgijm287lbv7sizb4z0dfk-gnugrep-2.25/sbin:/nix/store/9xq9cj2dcvdmmm6nxi0xyz3v8v0x1l1v-gnused-4.2.2/sbin:/nix/store/3k7lx2fx36jfpwwih0i3pdsmsqnbr9cg-systemd-230/sbin"
Environment="PGPASSFILE=/var/lib/hydra/pgpass-www"
Environment="SSL_CERT_FILE=/etc/ssl/certs/ca-certificates.crt"
Environment="TZDIR=/nix/store/c924jsm0xyir2d9szpdnxl3cn71vbhyl-tzdata-2016d/share/zoneinfo"

ExecStart=@/nix/store/bm0fqbnjh3i2i71z017qlhc1akpzixz5-hydra-2016-04-15/bin/hydra-server hydra-server -f -h '*' -p 8080 --max_spare_servers 5 --max_servers 25 --max_requests 100
PermissionsStartOnly=true
Restart=always
User=hydra-www

It seems none of the arguments passed are taken in account:
$ /nix/store/al5xxcdp7750k01wb6zddvckdgc20vjw-hydra-2016-04-15/bin/hydra-server -f -p 8080 --max_spare_servers 5 --max_servers 25 --max_requests 100 --background
Unknown option: f
[warn] Unicode::Encoding plugin is auto-applied, please remove this from your appclass and make sure to define "encoding" config
DEPRECATION WARNING: The Regex dispatch type is deprecated.
  It is recommended that you convert Regex and LocalRegex
  methods to Chained methods. at /nix/store/naxjf7f9370hajz4z0la62fkmrkkrg1i-hydra-perl-deps/lib/perl5/site_perl/5.22.2/Catalyst/DispatchType/Regex.pm line 210.
2016/06/08-22:23:20 Starman::Server (type Net::Server::PreFork) starting! pid(14866)
Resolved [*]:3000 to [<IP_ADDRESS>]:3000, IPv4
Binding to TCP port 3000 on host <IP_ADDRESS> with IPv4
Setting gid to "498 498 1 57 67 72 498"
Starman: Accepting connections at http://*:3000/

@peti removing -f helps, not sure why that's not recognized anymore.

Unknown option should result in the system exiting with exit status 1, IMHO.

I wish I knew how to operate with Catalyst. Not sure how those options are parsed even.

Aha, Catalyst upgrade broke this in master.

@domenkozar according to our conversation on list. What do you think is good strategy here? You will be porting back those changes to hydra repo or somehow pinning catalyst version for hydra or something else?

@spinus we can't backport this to hydra repo as it's not using nixpkgs master. How this will be fixed in master, I don't know yet, I'll look into it tomorrow.


Enhancement for QBO diags

When preparing for QBO diags for a WaterCycle run, I noticed that:

Year range being evaluated need to be printed out on plots. Current view shown as below:

With 34 years of low res model data and high res ERA data, it took quite long time to finish the run. May need to optimize for performance. Results from timing the run:
real	48m5.190s
user	6m3.675s
sys	8m35.370s

@forsyth2 Thanks for the figure using ERA-interim data.

The results seems very close to use ERA5 data. I think we can default to use ERA5 Interim in this particular case.

Just adding one plot that shows the comparison in QBO index between ERA5 and ERA Interim. In the figure below, test is ERA-interim, reference is ERA5.

Reopening this issue because even with a smaller datasets (ERA-Interim), i get inconsistent timing with same configuration. Possibly a memory leak somewhere.


Rename AppSignals configs with backward compatibility

Description of changes:
SMP is just a component of Application Signals, so it makes sense that we appropriately name the config that controls the Application Signals.
Also, http://cloudwatch-agent.amazon-cloudwatch:4317 as default exporter endpoint doesn't really work since it points to CW pod in EKS environment. Plus 4317 is also wrong as we have switched to 4315 for AppSignals.
Testing
Tested without and with updating the environment variables in contract tests to ensure the change is backward compatible.

Test run with SMP config: https://github.com/srprash/aws-otel-java-instrumentation/actions/runs/7836933969/job/21386408375?pr=1
Test run with AppSignal config: https://github.com/srprash/aws-otel-java-instrumentation/actions/runs/7837485201/job/21387233082?pr=1

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report
Attention: 103 lines in your changes are missing coverage. Please review.

Comparison is base (09e6487) 85.71% compared to head (dc8ac3e) 50.72%.
Report is 234 commits behind head on main.

Files
Patch %
Lines

...ent/providers/AwsAppSignalsCustomizerProvider.java
25.00%
36 Missing and 3 partials :warning:

...gent/providers/AwsSpanMetricsProcessorBuilder.java
0.00%
20 Missing :warning:

...ders/AttributePropagatingSpanProcessorBuilder.java
0.00%
16 Missing :warning:

...viders/AwsMetricAttributesSpanExporterBuilder.java
0.00%
11 Missing :warning:

...try/javaagent/providers/AwsSpanProcessingUtil.java
90.16%
1 Missing and 5 partials :warning:

...vaagent/providers/AwsMetricAttributeGenerator.java
96.89%
2 Missing and 3 partials :warning:

...y/javaagent/providers/AwsSpanMetricsProcessor.java
91.48%
0 Missing and 4 partials :warning:

...t/providers/AttributePropagatingSpanProcessor.java
94.59%
2 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@              Coverage Diff              @@
##               main     #744       +/-   ##
=============================================
- Coverage     85.71%   50.72%   -34.99%
- Complexity       19      264      +245
=============================================
  Files             3       39       +36
  Lines            49     1303     +1254
  Branches          5      141      +136
=============================================
+ Hits             42      661      +619
- Misses            3      610      +607
- Partials          4       32       +28

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

Contract test failed due to a timeout. https://github.com/aws-observability/aws-otel-java-instrumentation/actions/runs/7848701629/job/21420410781?pr=744
@aws-observability/adot-sdk-maintainers  Can you help re-run the workflow and also review the PR? TY!


Update the link to demo in README

Codecov Report

Merging #2177 (9c570e6) into master (fede38a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2177   +/-   ##
=======================================
  Coverage   80.02%   80.02%
=======================================
  Files          57       57
  Lines        2303     2303
  Branches      636      636
=======================================
  Hits         1843     1843
  Misses        271      271
  Partials      189      189

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 fede38a...9c570e6. Read the comment docs.


Delete even more code and trim down our default message

This officially removes support for "download", "init", and "upgrade".
This also adjusts the default message so that it fits comfortably in 80 columns:
$ ./boot2docker download

  WARNING: The 'boot2docker' command line interface is officially deprecated.

  Please switch to Docker Machine (https://docs.docker.com/machine/) ASAP.

  Docker Toolbox (https://docker.com/toolbox) is the recommended install method.

error in run: Sorry, command "download" is no longer supported.

cc @nathanleclaire

I'm thinking that taking out init might be a bit too far, but I don't see a clean way to do so without totally removing the download code completely, which makes it kind of gimped. :confused:

Argh.  Maybe we should just remove upgrade then?  download will still download a new ISO but if we're moving to a new repository that shouldn't be an issue, right?

Yeah, download will still be reasonably safe.

Maybe just the warning is actually enough?

Yeah, I liked the idea of disabling upgrade completely, but I had forgotten how tightly coupled everything is to cmdDownload.

Ok, we tried. :+1:


Enable bdf writing for BladeStiffenedShellConstitutive

Opening as a draft to get your opinion on how to implement this properly.
There is no direct equivalent of the Blade-stiffened shell constitutive model in NASTRAN, based on the approach taken in the previous version of TACS I am using the PSHELL propery card along with 4 MAT2 cards that directly describe the A, B, D, and As matrices of the stiffened shell.
The main issue is that the current BDF writer in pytacs relies on the material card being written by the TACSMaterialProperties object stored in each constitutive object. This is because the generateBDFCard method takes no inputs, so the only way a constitutive object knows which material card IDs to use in its property card is to get them from its TACSMaterialProperties objects.
To get around this I did 2 things:

Created a new cython-only MAT2MaterialProperties class. This class is not designed to be used at all by TACS, only to write out MAT2 cards. 4 instances of this class are returned by the getMaterialProperties method of BladeStiffenedShellConstitutive
Altered the writeBDF method in pyTACS so that it keeps track of which material cards are associated with each constitutive object and then passes and passes their ids to generateBDFCard. Currently this only works with BladeStiffenedShellConstitutive, we would need to modify the generateBDFCard methods of the other constitutive models if we want to stick with this approach.

It looks like these changes are causing the write_bdf tests to hang. Any idea what's going on here?

@timryanb are you happy with the basic idea behind the current implementation? If so I can go ahead and alter the other constitutive class generateBDFCard methods to accept the mat_ids input.


Vertical Grid

I was wondering if there is a way to show the grid vertically? With the fields as rows instead of columns. Easier than using the Row Rendered option I mean.

I would go for rowRenderer for this case, but apparently redefining display: block for <td> also seem to work (http://stackoverflow.com/questions/360649/vertically-stack-table-td-elements-using-css)


Defining Standards

The standards listed in the readme are really old. It would be nice to redefine them and break them down into a checklist so we can tell at a glance what needs to be done for each species.
Things that might be on this list include:

Assigned its own classifier (since most of these currently reuse C3 classifiers and we certainly don't want our new scripts messing up docked ships)
Conforms to Creatures Development Standards (this might be broken up into multiple checklist items, I dunno)
Does not require CAs
Does not depend on another specific species classifier to survive (IE bugs should eat plants/leaves in general, not one specific plant/leaf agent)
Has population control limits (should we define these more strictly?)
Has extinction control limits (these aren't supposed to be foolproof, but should be pretty tough)
Is just a little bit more special in some way (??)
Has been bugtested (We should probably set up a standard testing system for this too)

I also don't want to imply that things can't be created outside of these standards (and of course anyone can fork this repo and do whatever they want with it), but for the purposes of making the species as global and useful to everybody as possible, this feels like a good starting place.
Would really appreciate any thoughts/feedback/discussion on this!

See also https://github.com/Creatures-Developer-Network/C2toDS-Refresh/issues/41 which may be useful in the Standards portion of this issue.


Apt package google-chrome-stable=41.0.2272.101-1 randomly fails to install

There is an issue with google-chrome-stable=41.0.2272.101-1. Installing Chrome on Ubuntu randomly fails with the following error:
WARNING: The following packages cannot be authenticated!
     google-chrome-stable
STDERR: E: There are problems and -y was used without --force-yes
---- End output of apt-get -q -y install google-chrome-stable=41.0.2272.101-1
---- Ran apt-get -q -y install google-chrome-stable=41.0.2272.101-1 returned 100

This issue has been reported to Google.
FYI, Running the following command does NOT help:
wget -q -O - https://dl-ssl.google.com/linux/linux_signing_key.pub | sudo apt-key add -

the underlying problem is that the apt repo is not correctly signed
i think we can bypass this in the mean while by adding --force-yes option
package "google-chrome-#{node['chrome']['track']}" do
action :nothing
options "--force-yes"
end.run_action(:install)

I don't think that is the case either, because the key outputted to the screen was correct.  I think Google has a corrupt package out on one of their CDN servers.


Search and replace should focus in search not replace

Issue Type: Feature Request
Clicking ctrl+h brings up the search and replace popup however focus is in the replace text box, not the search.  I always end up shift tabbing to get back to the search textbox first.
Not a big deal, just an annoyance.
VS Code version: Code 1.26.1 (493869ee8e8a846b0855873886fc79d480d342de, 2018-08-16T18:38:57.434Z)
OS version: Windows_NT x64 6.1.7601

You should see it focus the search box if there is no text persisted in there. If there's text there it should focus the replace box. If that's what you see, this is by design.
You can tab back or ctrl+f to focus the search box.

I guess it is just an odd workflow to me because I have used visual studio for years and that is not how visual studio handles the same scenario.
Once there is text in the search and replace boxes there is never not text in those boxes unless you manually remove it.
I would bet if you did a usability study you would find that when a user presses ctrl+h they want to change the text in the search box first, tab, then replace the text in the replace box 99% of the time.
Out of curiosity, after you have completed a search and replace why would you want to slightly change your replace text to search and replace the same search string?  It doesn't make any sense to me because the search string changed after you did the last search and replace so it won't find any results... either way you have to tab back to the search text box before you can continue.
Like I said, it isn't a big deal, this is a throw away issue, however some would say that usability can make or break an application.  Just trying to help.
If it is by design then I shall close it.

I think it's optimized for the case where the user selects some text and presses ctrl+h, the selected text goes to the search input, and the cursor goes to replace. I think it's worth considering not moving the focus in this case where the user presses ctrl+h and there is some random old text in the search input.
Find widget/search should match, thoughts @rebornix ?

We have two similar issues talking about this case https://github.com/Microsoft/vscode/issues/32617 and https://github.com/Microsoft/vscode/issues/37102 . IMO the request makes sense, especially when you have Find opened once, and then

Put the cursor on a word, do not select
Run Ctrl-H to open Replace dialog
The previous query is populated into the search box, and we move the focus into the replace box.

@roblourens I agree with you that we can at least optimize for this case.


DEX-1283 RTL Support

What this PR does
The PR addresses some fixes applied to the universal checkout and overall UI.
It is now simplified and supports natively right-to-left locales.
Notable decisions & other stuff
Notion link
Instructions on how to test this
Let other reviewers know how they can test this.
Before merging
QA

[x] I manually tested this with a demo application (simulator)
[ ] I manually tested this with a demo application (real device)
[ ] I wrote unit tests for each new util-like function
[ ] I wrote at least basic e2e tests for the new functionalities or new paths
[ ] If this PR introduces UI changes, I manually tested this PR on iPhone 8 (iOS 10, 12, 14), iPhone 12, and iPhone 12 Max
[ ] If this PR modifies the behavior of an API, I did my best to make my new API backward compatible
[ ] If this PR modifies the API, I attempted to implement the feature with an example

Documentation

[ ] If this PR modifies the API, I updated the API Reference.
[ ] If this PR adds new options to the API, I wrote some documentation or guide in the online documentation
[ ] If the PR modifies the API, I wrote some documentation to deprecate it

After merging

Make sure a new release has been pushed to Cocoapods trunk

@EvansPie
Also providing more context based on the JIRA message:

I’ve gone through a check and it looks like the “increased padding” in the RTL case is due to the nature of the iOS Toolbar insets. As you can notice, for RTL we have the “mirror effect” ad the back button :arrow_backward: is reversed.


Detected a mismatch for the compiler version between your conan profile   settings and CMake:

I have compiled arm64-v8a sucessfully, but when I make the project, there is an error:
C/C++ debug|arm64-v8a : CMake Error at /home/ming/AndroidStudioProjects/pcl-for-android/example-app/app/.cxx/cmake/debug/arm64-v8a/conanbuildinfo.cmake:1065 (message):
Detected a mismatch for the compiler version between your conan profile
settings and CMake:
Compiler version specified in your conan profile: 8
Compiler version detected in CMake: 9.0
Please check your conan profile settings (conan profile show
[default|your_profile_name])
How should I do? Thanks.

I modified the NDK to android-ndk-r19c, the problem has been solved, but another error exists:
C/C++ debug|arm64-v8a : CMake Error at /home/ming/Android/Sdk/cmake/<IP_ADDRESS>88404/share/cmake-3.10/Modules/FindPackageHandleStandardArgs.cmake:137 (message):
Could NOT find Eigen3 (missing: EIGEN3_INCLUDE_DIR EIGEN3_VERSION_OK)
(Required is at least version "2.91.0")

The error
Could NOT find Eigen3 (missing: EIGEN3_INCLUDE_DIR EIGEN3_VERSION_OK)
(Required is at least version "2.91.0")
has been addressed by delete the file "FindEigen3.cmake" in :
~/.conan/data/eigen/3.3.7/conan/stable/package/5ab84d6acfe1f23c4fae0ab88f26e3a396351ac9/
And, the NDK version of r20 also works, I did not read the readme clearly before.
Finally, if git clone errors occurs in your Cross-compilation, modify all url from"https" to "git" in conanfile.py in ~/AndroidStudioProjects/pcl-for-android-master/conanfiles/flann,
~/AndroidStudioProjects/pcl-for-android-master/conanfiles/lz4,
and
~/AndroidStudioProjects/pcl-for-android-master/conanfiles/pcl.
This may help someone else.

The solutions really help me! Thanks a lot!


Fixed reference for ECS as it was written as EC2

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks for catching this, I will accept your pull request.


feature gate serde usage

This PR feature gates serde functionality, fixing clippy and build errors.
While linting the project I ran into these errors reported by clippy --no-default-features.
Interestingly, the errors are not reported if clippy is run with the --workspace option, which is what is used in GH Actions Lints workflow.
I have confirmed the errors are correct trying to build rgb-lib depending on rgb-runtime with the serde feature disabled. This way I get the exact same errors clippy reports without --workspace. I have also checked that it compiles and runs all tests fine when including this PR.
I think the issue with specifying --workspace is that dependency resolution is different. Specifically, rgb-wallet's dependency on rgb-runtime with all enabled features, including serde, might be masking the errors by inadvertently enabling the serde feature on rgb-runtime even if --no-default-features is used.
I am quite confident this might be the case as clippy correctly reports the errors if either cli is removed from the workspace members or the serde feature is removed from rgb-wallet's dependency on rgb-runtime.
I guess the usage of the --workspace option might also be the reason why some GH Actions jobs, like Build > no-default, pass instead of failing (cargo check --no-default-features fails without --workspace on master and passes with this PR).
I think cargo commands need to be run without --workspace, possibly separately for each package, in order to avoid hiding errors.

cfg_eval is a PITA: I tried to use it here before - and it was a mess everywhere, so I got rid of it. I will check whether there any other way of fixing the lint and the way dependencies are computed...

Maybe I am missing something, but I was able to fix the issue with no cfg_eval, which was unexpected for me. Seems everything works: https://github.com/RGB-WG/rgb/commit/f9813b780bf0b0295b13df23d5017565840d3994#diff-f643199b78cd917deee88c751028d85bc710961b6f7b874f3c99cf6935fd68b0L109-L118

Closing this in favor of fix in https://github.com/RGB-WG/rgb/pull/175/commits/ac43eb2709fe06cb102b7083ea6205634573f2df


Get raw 1.8.0/2.8.0 values as seen on electric meter

I have an optical sensor connected to the energy manager which reads out the optical signals of my electric meter.
Is it possible to get the raw 1.8.0 (consumed from grid) and 2.8.0 (sent to grid) values as they can be seen on the meter?
I noticed that there’s a huge discrepancy by now between what I get from this integration as energymanager_workin and energymanager_workout and what I can see on the electric meter as 1.8.0 and 2.8.0, e.g. ~28 MWh for 1.8.0 vs ~22MWh for energymanager_workin.
The optical sensor was plugged out for a bit and I then noticed that it didn’t jump straight to a way higher value after plugging it back in. Instead, it just continued with what it could read from that point on—which apparently wasn’t the electric meter work sum but instead only the current power which the energy manager seems to add up itself??
Can you shed some light on what’s happening here and which data can be read from the energy manager and how to interpret it? 😊
I also checked other values than the two mentions. None of them match what I see on the electric meter.

The integration just reads the values from the energy manager and makes it available to home assistant.
I have no idea how it handles other devices connected to it. Maybe you can login to the energy manager and try to configure it?

I wouldn’t know where to look. The interface is very barebones with very few options (if any). 😕

@Mas2112 Ah, found the data! I checked $localIP/rest/kiwigrid/wizard/devices and found the correct values in result->items->8->tagValues->WorkIn.
The ones I had used (wrongly) so far are in result->items->1->tagValues->WorkIn and result->items->1->tagValues->WorkOut
Any idea how I could get the integration to get me that value? Would the integration need to be changed for that or the underlying Python library that connects with the energymanager?

I think I know what the problem is. The integration assumes there are only
one set of work values and takes the first one. Can you post your Json
data? (delete personal data from the Json before posting)
On Tue, Aug 13, 2024, 21:29 Patrick Geselbracht @.***>
wrote:

@Mas2112 https://github.com/Mas2112 Ah, found the data! I checked
$localIP/rest/kiwigrid/wizard/devices and found the correct values in
result->items->8->tagValues->WorkIn.
The ones I had used (wrongly) so far are in
result->items->1->tagValues->WorkIn and
result->items->1->tagValues->WorkOut
Any idea how I could get the integration to get me that value? Would the
integration need to be changed for that or the underlying Python library
that connects with the energymanager?
—
Reply to this email directly, view it on GitHub
https://github.com/Mas2112/solarwatt-energymanager-homeassistant/issues/21#issuecomment-2286980990,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABJLA4S6UZ73F6RIS5HKU3ZRJNCVAVCNFSM6AAAAABMNXAZ36VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOBWHE4DAOJZGA
.
You are receiving this because you were mentioned.Message ID:
@.***
.com>

energymanager.json
The WorkIn/WorkOut values of the device with the SMLDriverService ("guid": "urn:sml:09-RE-DA-CT-ED-RE-DA-CT-ED-bf",) are the ones from the electric meter which match what it shows

You added an optical sensor to the energy manager?    I only have read from the possibility, to connect an S0 energy meter to the S0 interface (wire connection)
My values also differ   1.8.0 / 2.8.0: 14,1MWh / 21.4MWh   Energymanager:  WorkIN/Out  10,2MWh / 17,3MWh  (Both started at the same day 4 years ago at 0)
BTW:  Before using this integration, I just called the RestAPI with the HA restful integration.  Here is some code for this:
rest_solarwatt_energymanager.yaml.zip

You added an optical sensor to the energy manager?
Yes, I have an optical sensor attached to the energy manager via USB that is then attached to the optical interface of the energy meter. It had been installed like that initially and we didn’t change it ourselves.

BTW: Before using this integration, I just called the RestAPI with the HA restful integration.

Yeah, after I saw the JSON response for the call, I was thinking of just getting that. But I’d prefer an integration that is also used/tested and updated by others. 😊
It’s just a pity that I went with the wrong data for more than two years and all the historic data in the energy dashboard is wrong and I should start anew with the correct values once available.

Oh I see the problem. My integration doesn't support the com.kiwigrid.devices.powermeter.PowerMeter device.
By default, I pull the values from the com.kiwigrid.devices.location.Location device.
I need to add support for the PowerMeter device, and then it should appear as a new entity in Home Assistant.
Please be patient, it might take a while.

Please be patient, it might take a while.

@Mas2112 Absolutely, thank you kindly!
If there’s anything I could assist you with, just tell me. Unfortunately, my Python expertise is basically non-existent, otherwise I would create a PR for the base library myself.

By default, I pull the values from the com.kiwigrid.devices.location.Location device.

What is that though? There seems to be substantial difference in values for both @m67hoff and I so I wonder what that device is supposed to represent? 🤔

Since its terrible weather here, I decided to use the time to add support for the Power Meter. I have created pre-release 0.10.0 with this support. Can you test it, see if it is working?

Oh, nice, thank you! @Mas2112
I’ll test that today or tomorrow and will get back to you.

@Mas2112 It’s working! 🥳 Thank you kindly!
I got (at least?) two new sensors:

energymanager_edl_zaehler_09_lo_ng_hex_st_ri_ng_workin
energymanager_edl_zaehler_09_lo_ng_hex_st_ri_ng_workout
which represent 1.8.0 and 2.8.0


[P1] Added BLIP-ITM model definitions

Description
Current BLIP implementation by @aryamanarora is for VQA, but our upcoming experiments with ColorSwap will require BLIP-ITM. I've added new model definition files specifically for the BLIP-ITM setup, as it's fairly different in architecture (vis enc -> text enc -> ITM head) from BLIP-VQA (vis enc -> text enc -> text dec). It's probably possible to capture both of them with a single general definition, but I wanted to take the lowest risk path for my first PR.
Testing Done
Haven't added any tests or done any testing yet. Just wanted to open a PR with the [mostly complete] work I've done already as it's Sunday night and I'm not sure when I'll be able to get back around to this next.
Checklist:

[X] My PR title strictly follows the format: [Your Priority] Your Title
[ ] I have attached the testing log above
[X] I provide enough comments to my code
[ ] I have changed documentations
[ ] I have added tests for my changes

@frankaging Did you get rid of the topological order enforcement for the intervention list? Does that mean we can specify arbitrary names for keys in the type_to_module_mapping? That would mean we can add the text decoder + image encoders as well for VLMs.

@frankaging Did you get rid of the topological order enforcement for the intervention list? Does that mean we can specify arbitrary names for keys in the type_to_module_mapping? That would mean we can add the text decoder + image encoders as well for VLMs.

yes! that is removed. you can add arbitrary names. and you can also intervene without specifying the mapping now if it is a standard intervention point, e.g., no QKV split, no head split as in transformer.

I am merging this PR as it seems like it will not break anything, and adds additional model supports. Future bug PRs can be opened separately if there is any issue.


Select with many entries not scrollable in Visual Builder

Problem Description
When there are many entries in a select field, the list overflows the screen and is not scrollable. This is an issue on small screens and depending on the list size also on large screens. The solution would be to remove the max-height property so the list becomes scrollable.

QA Checklist
Reporter

[ ] Helpful title used

Helpful title format: Scope(s) :: One Sentence To Describe The Issue
Example: VB :: Floating Toolbar :: Jumps To Top Of Page

[ ] Detailed description

Description should usually be 150+ characters.

[ ] Steps to reproduce included
[ ] Support Ticket URL included
[ ] Image and/or gif

Developer

[ ] Steps to reproduce proven solved
[ ] Test included and coverage acceptable and all Tests passing?
[ ] PR description has layman's description of problem and solution and "Areas Affected" listed/explained?
[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Estimated points on issue
[ ] Proper code documentation and conventions?
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Peer Review

[ ] PR branch includes latest commits from master branch (at the time of checking)?
[ ] Steps to reproduce proven solved
[ ] Affected areas are tested and ok
[ ] Proper code documentation and conventions
[ ] Proper RTL appearance
[ ] i18n used if applicable
[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?

Quality Assurance

[ ] All tests passed on CircleCI.
[ ] Steps to reproduce passed, the fix works.
[ ] No errors in console or output
[ ] Affected areas are tested and ok
[ ] The fix does not change a customer's website in any surprising or unintended way

[ ] If Divi's design is adjusted, settings are migrated gracefully and invisibly

Design Review

[ ] UI is to spec
[ ] UX is to spec
[ ] Design is to spec
[ ] No surprising visual changes to a customer's website will be experienced after updating

[ ] Any/all visual changes are migrated gracefully

Dev Review

[ ] Test(s) included and coverage acceptable
[ ] Ajax/Form used? Nonce used? Perms checked? All user input sanitized and validated? Intentional unsanitized/unescaped functions used?
[ ] Database saved/updated? Data sanitized and validated?
[ ] Variable output properly escaped and validated?
[ ] i18n used if applicable
[ ] Implementation acceptable, DRY, Scaleable, future safe
[ ] Best practices used, cleverness and grokage, ES6 use, etc.

In the latest version it seems to work as expected. Not sure what you did but it worked 👍


Content Issue In Image

There is a content spelling issue in the image: Test Script run by suitable "ranguage" runtime.

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

ID: a8db1c71-2ade-27cb-9e60-61d6f1c86ac2
Version Independent ID: e151c18c-b900-0676-a6e2-b06ae9b7a301
Content: UI Testing for Apache Cordova Apps with Appium - Cordova
Content Source: cordovadocs/2017/debug-test/ui-testing-with-appium.md
Product: visual-studio-dev15
GitHub Login: @kraigb
Microsoft Alias: kraigb

Thanks for reporting this! For some reason I'd fixed it in the source image but never got the change propagated to the live docs.

#conceptdev Please merge the PR and close the issue at your convenience.

Updated image merged - thanks :)


Update calico_node url reference in document

calico/node moved from projectcalico/calicoctl to
projectcalico/calico since v2.4.0[1]. Updating doc reference
accordingly.
[1] commit id: b4e4c1aa12b41f44b84d455ecb486e4e54ee31d3
Description

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

None required

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

This is a small doc fix as I was crawling through Calico document and code base. Let me know if somewhere else needs to be touched as well. Thanks!

@caseydavenport


Create actable plus related acts_as product

Not as much an issue as a question i have for my needs.
I get it when I create a Pen, it will automatically also create a Product (examples from the docs).
Which is all fine. But in this case I have to create Pen... which is a bit counter intuitive to me.
In reality i will always create new Product, which happens to be of Pen type.
So what I would basically try to ask is whether it is possible to have something like this in a view:

new product
select that it is a Pen
open additional Pen related attributes
create Product with Pen

As far as i am testing it, there is actually no way to create Product and related Pen records.
Only vice-versa.
Wouldn't be easier to have one (although more complex) form, than a separate forms for each of the types? I could be shooting in totally different direction and MTI is not even good solution to do it...in that case I would really appreciate any tips or tricks :)

+1

That highly depends on your app, but nothing prevents that.


Support mixing compression modes in TopicTree

This is needed for per-tenant TopicTrees to work as it disconnects the TopicTree from a specific WebSocketContext. It is also needed to properly support all settings in x-webkit-deflate.

 We also don't care about what some non-standard extension might do. Closing!


Add rand.range command

Seems like this would be useful:
rand.range [start, ]stop[, step]

I.e., a random number 0-9 would be rand.range 10, etc

done https://github.com/voidqk/sink/commit/fbfee33f7dc6ebcf6d2f9a33c2610df88c010380


add getChannelTimezone to redux

Summary
Add the getChannelTimezone to redux
Ticket Link
N/a
Other Prs:
webapp: https://github.com/mattermost/mattermost-webapp/pull/1607
server: https://github.com/mattermost/mattermost-server/pull/9286
api reference: https://github.com/mattermost/mattermost-api-reference/pull/387
Checklist

[x] Ran make check-style to check for style errors (required for all pull requests)
[x] Ran make test to ensure unit tests passed
[x] Ran make flow to ensure type checking passed
[x] Added or updated unit tests (required for all new features)
[x] All new/modified APIs include changes to the drivers

Test Information
This PR was tested on: mac ox 10.13.6

@lieut-data @jwilander Here is the associated Redux PR for https://github.com/mattermost/mattermost-server/pull/9286

@hmhealey PTAL


Missing plugin socket

I'm not sure if this is the right place, But recently after updating to kong 3.2.2, I started seeing ciritical errors at start up
2023/05/18 15:33:19 [notice] 1206#0: *148 [kong] process.lua:252 Starting js, context: ngx.timer
2023/05/18 15:33:19 [crit] 1206#0: *149 connect() to unix:/usr/local/kong/js_pluginserver.sock failed (2: No such file or directory), context: ngx.timer
2023/05/18 15:33:19 [crit] 1206#0: *149 connect() to unix:/usr/local/kong/js_pluginserver.sock failed (2: No such file or directory), context: ngx.timer
2023/05/18 15:33:19 [crit] 1206#0: *149 connect() to unix:/usr/local/kong/js_pluginserver.sock failed (2: No such file or directory), context: ngx.timer
2023/05/18 15:33:19 [crit] 1206#0: *149 connect() to unix:/usr/local/kong/js_pluginserver.sock failed (2: No such file or directory), context: ngx.timer
2023/05/18 15:33:19 [crit] 1206#0: *149 connect() to unix:/usr/local/kong/js_pluginserver.sock failed (2: No such file or directory), context: ngx.timer
2023/05/18 15:33:19 [crit] 1206#0: *149 connect() to unix:/usr/local/kong/js_pluginserver.sock failed (2: No such file or directory), context: ngx.timer
2023/05/18 15:33:19 [crit] 1206#0: *149 connect() to unix:/usr/local/kong/js_pluginserver.sock failed (2: No such file or directory), context: ngx.timer

2023/05/18 15:33:28 [warn] 1206#0: *160 [kong] plugins.lua:125 /plugins/schema/:name endpoint is deprecated, please use /schemas/plugins/:name instead (deprecated after 1.2.0, scheduled for removal in 3.0.0), client: <IP_ADDRESS>, server: kong_admin, request: "GET /plugins/schema/prometheus HTTP/1.1", host: "<IP_ADDRESS>:8444"

Things seem to have a real hard time connecting to kong when this happens.
Here are the env vars used to set this up
KONG_PLUGINSERVER_JS_QUERY_CMD=node /usr/local/bin/kong-js-pluginserver --plugins-directory /opt/plugins/js --dump-all-plugins
KONG_PLUGINSERVER_JS_SOCKET=/usr/local/kong/js_pluginserver.sock
KONG_PLUGINSERVER_JS_START_CMD=node /usr/local/bin/kong-js-pluginserver --plugins-directory /opt/plugins/js
KONG_PLUGINSERVER_NAMES=js

I in fact had to patch the deployment in production to disable this plugin server because it looked like we were loosing connections

After I patched it and the error stop things normalized

Hi. Have you checked if the plugin functions normally? I believe this is an expected behavior.
We introduced a check to prevent requests before the plugin server is ready. Due to the implementation limit, we have to check by trying to connect to the socket, which generates error messages and we cannot mute it.
We're considering removing this check logic as it does not really help when a plugin server is slow.
Reference:
https://github.com/Kong/kong/pull/9507
https://github.com/openresty/lua-nginx-module/issues/2138


Create a new directory to hold the new plugin version when recompiling

I tend to get one compiler error that it will then fix, but it used to break when the directory already existed, so we move it beforehand.

Hey @Zinnusl 👋
Thanks for the feedback! I hope you don't mind but I went with a slightly different approach, but I appreciate you taking the time to open a PR anyway 🙏
The change I pushed allows it to overwrite the existing source code to avoid ending up with potentially large numbers of directories containing invalid source 😅
It does mean we lose the history of GPT's failed attempts, but I'm going to add a conversation logger to write the full conversation to disk.


MaterialDialogCompat item/button listeners

OnClickListener passed to setSingleChoiceItems is called after setPositiveButton/setNeutralButton/setNegativeButton listeners.

This is not specific to MaterialDialogCompat. That class just wraps around the non-deprecated MaterialDialog.Builder API.
This is also intended behavior. The button is pressed, and then the item callback is sent.


Import Error

I keep getting this error:
ImportError: attempted relative import with no known parent package
How do I fix this?

Hi,
Can you give me more details? Are you trying to run example.py or a script you wrote using the library?


add config example

Description
fixes https://github.com/gruntwork-io/cloud-nuke/issues/786.

TODOs
Read the Gruntwork contribution guidelines.

[ ] Update the docs.
[ ] Run the relevant tests successfully, including pre-commit checks.
[ ] Ensure any 3rd party code adheres with our license policy or delete this line if its not applicable.
[ ] Include release notes. If this PR is backward incompatible, include a migration guide.
[ ] Attention Grunts - if this PR adds support for a new resource, ensure the nuke_sandbox and nuke_phxdevops jobs in .circleci/config.yml have been updated with appropriate exclusions (either directly in the job or via the .circleci/nuke_config.yml file) to prevent nuking IAM roles, groups, resources, etc that are important for the test accounts.

Release Notes (draft)

Added / Removed / Updated [X].
Migration Guide

Hey @denis256, can you please approve the PR once again?


slnStartupProject clobbers Project Dependencies in MSVC 2008

So, just running it on a 2008 sln just causes all checkboxes to be unchecked seen under the startup project's dependencies (and probably all other projects')
Need more info?

Hi,
Yes. Thanks for the feedback. Could you send me your sln file and I'll be
happy to fix the issue.
M
Thanks for the feedback. Could you
On Wednesday, May 20, 2015, ddurham2<EMAIL_ADDRESS>wrote:

So, just running it on a 2008 sln just causes all checkboxes to be
unchecked when going under the startup project's dependencies (and probably
all other projects')
Need more info?
—
Reply to this email directly or view it on GitHub
https://github.com/michaKFromParis/slnStartupProject/issues/3.

--
Michel Courtine | CEO | iVoltage
+33 6 189 354 89 |<EMAIL_ADDRESS>http://ivoltage.me

Hopefully the foo.zip attachment will make it through email... Let me
know if not.
If you extract the zip, you'll find foo\foo-app.sln which references 3
projects: foo-app, baz-app and bar-lib  where foo-app and baz-app both
depend on the bar-lib
foo-app is already the startup project.
If you run "slnStartupProject foo-app.sln foo-app", then open the sln
back in MSVC 2008, you'll find that it clobbers the dependency of
foo-app on bar-lib.   I diffed the sln file from before and after the
run, and it looks like it's messing up one of the the XML tag names a
bit.  So I wouldn't think the fix should be too hard.
Thanks!
On 05/20/2015 12:00 AM, Michel Courtine wrote:

Hi,
Yes. Thanks for the feedback. Could you send me your sln file and I'll be
happy to fix the issue.
M
Thanks for the feedback. Could you
On Wednesday, May 20, 2015, ddurham2<EMAIL_ADDRESS>wrote:

So, just running it on a 2008 sln just causes all checkboxes to be
unchecked when going under the startup project's dependencies (and
probably
all other projects')
Need more info?
—
Reply to this email directly or view it on GitHub
https://github.com/michaKFromParis/slnStartupProject/issues/3.

--
Michel Courtine | CEO | iVoltage
+33 6 189 354 89 |<EMAIL_ADDRESS>http://ivoltage.me
—
Reply to this email directly or view it on GitHub
https://github.com/michaKFromParis/slnStartupProject/issues/3#issuecomment-103756932.

Unfortunately, the attachment didn't make it through. Could you please send it to github ---at--- michak.net ?


Make NuclideBases not global

The directory in nuclideBases is global, which leads to considerable pain:

Import order is very sensitive, and lots of things have to happen at import time, and cant happen again or else 💥.
The actual contents change when a burn chain is imposed, which is really case dependent and can only happen once, or else 💥
Some code changes the LFP properties, and these changes probably don't survive round-tripping through the database. Not exactly related to global state, but part of the reason we haven't been thinking of it. See collapseFissionProducts and updateYieldVector methods within the fissionProductModel.

So, we should be thinking of another place for this to live, and spin up an object containing the valid nuclides for that scope. Reactor? Case? Somewhere else? Even a global database that is actually an object instead of a magical singleton-esque horror would be better than it is now. Also, if it is mutable beyond the initial setup with a burn chain, then we need to think about how to recover it's state from the database.

We made a step in a direction where global nuclide bases are less bad than they used to be by adding more nuclides in #998. Unfortunately, the nuclide data is still global and the burn chain modifications still may occur when it's being imposed. I specifically added logging statements if a user overwrites any default parameter from the burn-chain.yaml or custom file, but the transmutation and decay data that is a subset of the nuclides still gets modified based on the user-defined burn chain data.
We can probably take this further still.

@opotowsky
This has reared it's head again, and will have to be put on the calendar for ARMI 0.4.0.  It turns out ARMI enforces everyone to use RIPL nuclear data, but we have some users that use ENDF in other parts of their modeling chain.  So I would LOVE to give people a setting to choose which nuclear data they want.  But we can't do that (easily) if the nuclear data lives in the global scope at import time.
So, finally, I think we have enough interest to make this ticket happen.

I notice that it is not just nuclideBases that is global, but also elements and thermalScattering. We even have an unrelated ticket open to rip thermalScattering out of the global scope.
So, let's try to itemize the nuclide information that is currently in the global scope, and where (outside the nucDirectory area) it is used:
nuclideBases

Name
Usage

byName
* very popular in downstream projects* various materials* isotopicOptions* component* composite* densityTools

byDBName
* one downstream plotting function

byLabel
* two downstream projects* Uranium.setDefaultMassFracs* xsNuclides.updateBaseNuclides

byMcc2Id
* unused

byMcc3Id
* downstream dif3d and depletion codes* dlayxs* isotopicOptions.getAllNuclideBasesByLibrary

byMcnpId
* downstream burnx and mcnp code

byAAAZZZSId
* single isotxsSample file in nucleardata-uq

elements

Name
Usage

byZ
* blocks::getPuMoles* isotopicOptions::autoUpdateNuclideFlags()* Composite.constituentReport()

byName
* isotopicOptions.eleExpandInfoBasedOnCodeENDF(cs)

bySymbol
* many places downstream* isotopicOptions::ALLOWED_KEYS and .expandMassFracs* Composite::getNuclidesFromSpecifier and .constituentReport()* Water.setDefaultMassFracs

thermalScattering

Name
Usage

byNbAndCompound
* materials like BE9 and Graphite * OpenMC plugin

Materials
I am a touch worried about the current state of ARMI materials because of this "global nuclides" situation. Currently, if you update the burn chain, all the global nuclides can be updated. Which means that the mass fractions of materials can be updated... during a run?
How how does that work with parallel unit tests? Our materials are changing at random points during unit tests because one test or another happens to touch the burn chain?  Ewww

@aaronjamesreynolds @opotowsky And anyone else who is interested... I have a branch out now to start this this ticket: obliterate_global_nuclides
All these global will be moved into this new class:
https://github.com/terrapower/armi/blob/fe4be83e444165db8697f7c5bbf872eb8afc8e34/armi/nucDirectory/nuclideBases.py#L1467-L1485
Note the big table above. This is only the start. But merge freeze or not, I can start thinking about all this.

This looks good @john-science. Are you thinking that this class would be instantiated on an application basis and then once the application is configured we can call through app.nuclides or did you have something else in mind for how it should be connected and configurable?

Are you thinking that this class would be instantiated on an application basis ...

Well, at the moment, I am guessing the nuclides bases have to live on the Reactor, so r.nuclideBases. But that's just a guess. The truth is the solution will have to derive from the table above.
The table above shows where the nuclide bases are used. So, the usage of x.nuclideBases will determine where they have to live.  Like... where can we put this new object that causes the fewest changes in ARMI and downstream.
It is possible (probable?) that no matter where I put the new object, there will have to be a some breaking changes to the process (my guess is this will happen in the materials). BUT choosing a good place to put this new object will cut the work needed for this change down by 2x or even 10x.

Two cents: R.nuclides over App.nuclides
Rationale: interfaces always have access to the reactor, not necessarily the application. And the app is already a global thing, so putting the nuclides there isn't really getting rid of global nuclides. It's just moving them.


mozc not activated

Happening on: t440s, kudu1
Not happening on: serval1

On serval,

writing in ~/.emacs directly the following worked.

(load "~/link/github_repos/130s/compenv_ubuntu/config/emacs/emacs.el")
(load "~/link/github_repos/130s/compenv_ubuntu/config/emacs/emacs_130s-serval.el")

sym linking ~/.emacs  to emacs_130s-serval.el causes the same error in *Warnings* buffer:

Warning (initialization): An error occurred while loading `/home/n130s/.emacs':

File error: Cannot open load file, no such file or directory, ibus

To ensure normal operation, you should investigate and remove the
cause of the error in your initialization file.  Start Emacs with
the `--debug-init' option to view a complete error backtrace.

On serval, the same issues is observed with the tweak below. This confirms that the issue only occurs on < Ubuntu 15.04.

writing in ~/.emacs directly the following worked.

(load "~/link/github_repos/130s/compenv_ubuntu/config/emacs/emacs.el")
(load "~/link/github_repos/130s/compenv_ubuntu/config/emacs/emacs_130s-serval.el")

sym linking ~/.emacs  to emacs_130s-serval.el causes the same error in *Warnings* buffer:

Warning (initialization): An error occurred while loading `/home/n130s/.emacs':

File error: Cannot open load file, no such file or directory, ibus

To ensure normal operation, you should investigate and remove the
cause of the error in your initialization file.  Start Emacs with
the `--debug-init' option to view a complete error backtrace.

On serval, the same issues is observed with the tweak below. This confirms that the issue does NOT only occur on < Ubuntu 15.04.

writing in ~/.emacs directly the following worked.

(load "~/link/github_repos/130s/compenv_ubuntu/config/emacs/emacs.el")
(load "~/link/github_repos/130s/compenv_ubuntu/config/emacs/emacs_130s-serval.el")

sym linking ~/.emacs  to emacs_130s-serval.el causes the same error in *Warnings* buffer:

Warning (initialization): An error occurred while loading `/home/n130s/.emacs':

File error: Cannot open load file, no such file or directory, ibus

To ensure normal operation, you should investigate and remove the
cause of the error in your initialization file.  Start Emacs with
the `--debug-init' option to view a complete error backtrace.


Provide more informative setup instructions

As I commented at https://github.com/Ikuyadeu/vscode-R/issues/240#issuecomment-590014087:

We may need a major improvement of README to make clear the setup of multiple parts including vscode-R, vscode-r-lsp, languageserver, and radian on Windows, Linux/macOS so that it is easier for user to have the full experience these parts combined has to offer.

I suggest we write more detailed setup instructions in README or in wikis.

I agree. A couple of thoughts:

I think we should suggest that users install radian as part of setup, i.e., have radian be vscode-r’s ‘recommended console’
On vscode-R activation, we could check if vscode-r-lsp is installed, and prompt for installation if it isn’t
In vscode-r-lsp, we could check if languageserver is installed (I think this is possible), and prompt for installation if it isn’t

I'm also wondering if we have a way to find R binary automatically on Windows? In my knowledge, most beginners don't know where the R binary is located or can be confused by x86, x64 stuff.

I suggest we create a wiki to present more documentation:

Getting started
Installation

Windows
Linux/macOS

R Terminal
Keyboard shortcuts
R Session watcher
Recommended extensions and configuration
(such as C/C++ to write Rcpp, Path Autocomplete, TODO highlight, etc. mainly for RStudio user to find similar editor features in VSCode)

I created several wiki pages at https://github.com/Ikuyadeu/vscode-R/wiki. I'll spend some time to finish these pages later.

I've finished writing some introduction pages and installation guides. Will elaborate on other pages.

Great works!
Also, I put the contribution guide from CONTRIBUTING.md. It can be updated by just putting the wiki link too.

I'm pretty new to vscode and am looking to migrate from RStudio. Therefore, some of my comments might be a bit too basic and self-evident, so I apologise for that.
I think there are still some aspects of the setup missing, and maybe they're more to do with understanding vscode, than specifically vscode-R.
I'm missing some simple steps to follow regarding setting up a workspace environment to launch just the way I like it. I have a setup/layout I like to use, but I seem to somehow default back to just one large pane view whenever I open a new folder, which I then need to split into the multi-pane view I prefer. Also, first thing I do when I start vscode now is hit F1 and select Attach Active Terminal. There must be something I've missed in the setup instructions for this to happen automatically on startup.
Or something like the equivalent of RStudio's Project scaffolding to create a default Workspace view/layout.
Basically I'm looking for the most friction free way for a user to migrate to vscode from RStudio. It could be a comparison table perhaps of common options from RStudio, and how to do the equivalent in vscode. e.g. from RStudio:

New Project (Workspace) with default layout
New R Package
New R markdown document
etc

@phillc73 Thanks for your kind suggestions!
I was just thinking about a feature table of RStudio and VSCode R language support. I'll write something like this when I have time.

Some more documentation on features like session watcher and terminals would be helpful as well. As it is written now it sounds like just by adding ```source(file.path(Sys.getenv(if (.Platform$OS.type == "windows") "USERPROFILE" else "HOME"), ".vscode-R", "init.R"))

I think for a lot of RStudio converts like me one of the major drives behind sticking with it so long is the way it "GUIifies" R for us with the packages and environment panes. Especially when it comes to things like stepping through a function and seeing every object update live.

Closing as resolved.


[Bug] container for service "songs" is unhealthy

Search before asking

[X] I had searched in the issues and found no similar issues.

Apache SkyWalking Component
OAP server (apache/skywalking)
What happened
apple macbook pro m1, something went wrong with docker-compose

git clone https://github.com/apache/skywalking-showcase.git
cd skywalking-showcase
make redeploy.docker
container for service "songs" is unhealthy
make[1]: *** [deploy] Error 1
make: *** [deploy.docker] Error 2
What you expected to happen
the SkyWalking can not run
How to reproduce
Just do with the document
https://skywalking.apache.org/docs/skywalking-showcase/next/readme/
Anything else
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I am not sure whether we have m1 support for the showcase. @Yemsheng At least, you should provide information rather than two error messages.

Just do with the document
https://skywalking.apache.org/docs/skywalking-showcase/next/readme/

This doesn't make sense. You got the m1 to run locally, not us.

We don't even build Docker images for arm architecture, let alone m1 support.

OK, so, we don't support m1 and arm env in the showcase for now.
@Yemsheng If you want to, we are welcome to accept contributions.