Can't tokenize alias as key without seperator

js-yaml: @3.7.0
Node v7.2.1
Windows 10 Pro
disclaimer: I'm not an expert on the YAML spec
the bug
":" is a valid character when defining anchors and using aliases.
It is valid to use aliases as keys.
":" is also used to denote a node.
This seems to cause a bug where you are unable to use aliases as keys unless you separate the alias from the ":" with whitespace.
Trailing ":"'s look like they're being included in the alias token by the while loop in loader.readAlias().
I don't know the code very well, but a fix might need some extra logic in loader.composeNode().
some tests
const tests = [
  // working cases  (space before `:`)
  `
    adjective: &adj  stealthy!
    *adj :           box :)
  `,
  `
    adjective: &adj: stealthy!
    *adj: :          box :)
  `,
  `
    adjective: &:    stealthy!
    *: :             box :)
  `,
  // broken cases  (no space)
  `
    adjective: &adj  stealthy!
    *adj:            box :)
  `,
  `
    adjective: &adj: stealthy!
    *adj::           box :)
  `,
  `
    adjective: &:    stealthy!
    *::              box :)
  `,
]
for ( const content of tests )
  try{ console.log(require('js-yaml').load(content)) }
  catch(e) { console.log(e) }

output
{ adjective: 'stealthy!', 'stealthy!': 'box :)' }
{ adjective: 'stealthy!', 'stealthy!': 'box :)' }
{ adjective: 'stealthy!', 'stealthy!': 'box :)' }
{ Error
    at generateError (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:162:10)
    at throwError (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:168:9)
    at readAlias (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1244:5)
    at composeNode (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1336:20)
    at readBlockMapping (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1004:16)
    at composeNode (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1327:12)
    at readDocument (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1489:3)
    at loadDocuments (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1545:5)
    at Object.load (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1562:19)
    at Object.<anonymous> (C:\Users\leigh\repos\puppet\test-site\data\yaml.js:31:25)
  name: 'YAMLException',
  reason: 'unidentified alias "adj:"',
  mark:
   Mark {
     name: null,
     buffer: '\n    adjective: &adj  stealthy!\n    *adj:            box :)\n  \n\u0000',
     position: 41,
     line: 2,
     column: 9 },
  message: 'unidentified alias "adj:" at line 3, column 10:\n        *adj:            box :)\n             ^' }
{ Error
    at generateError (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:162:10)
    at throwError (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:168:9)
    at readAlias (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1244:5)
    at composeNode (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1336:20)
    at readBlockMapping (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1004:16)
    at composeNode (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1327:12)
    at readDocument (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1489:3)
    at loadDocuments (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1545:5)
    at Object.load (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1562:19)
    at Object.<anonymous> (C:\Users\leigh\repos\puppet\test-site\data\yaml.js:31:25)
  name: 'YAMLException',
  reason: 'unidentified alias "adj::"',
  mark:
   Mark {
     name: null,
     buffer: '\n    adjective: &adj: stealthy!\n    *adj::           box :)\n  \n\u0000',
     position: 42,
     line: 2,
     column: 10 },
  message: 'unidentified alias "adj::" at line 3, column 11:\n        *adj::           box :)\n              ^' }
{ Error
    at generateError (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:162:10)
    at throwError (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:168:9)
    at readAlias (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1244:5)
    at composeNode (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1336:20)
    at readBlockMapping (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1004:16)
    at composeNode (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1327:12)
    at readDocument (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1489:3)
    at loadDocuments (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1545:5)
    at Object.load (C:\Users\leigh\repos\puppet\test-site\data\node_modules\js-yaml\lib\js-yaml\loader.js:1562:19)
    at Object.<anonymous> (C:\Users\leigh\repos\puppet\test-site\data\yaml.js:31:25)
  name: 'YAMLException',
  reason: 'unidentified alias "::"',
  mark:
   Mark {
     name: null,
     buffer: '\n    adjective: &:    stealthy!\n    *::              box :)\n  \n\u0000',
     position: 39,
     line: 2,
     column: 7 },
  message: 'unidentified alias "::" at line 3, column 8:\n        *::              box :)\n           ^' }

This is an edge case. The spec is not very clear on that.
@flyx mentioned that he had been asking about that on the mailing list two years ago: https://sourceforge.net/p/yaml/mailman/message/34909032/
I would generally add a space after an alias key to make sure it's portable. It should also be noted that libyaml and pyyaml do not support : anywhere in aliases/anchors.
When playing with that, it seems I found a bug in libyaml, pyyaml and snakeyaml:
foo: &a:b bar is parsed the same as foo: &a :b bar
So I would avoid : in aliases completely.

@perlpunk
In personal experience, I have not seen other usage in which people add spaces after their aliases to mitigate this problem, but it's good advice for compat /w this version of js-yaml.
I agree about disallowing : in aliases in the tokenizer.
Since the spec is so lax on this point, it's probably a source of implementation specific divergences like you discovered.
Neat find!


nightshade upgrade

5.6.0 Nightshade upgrade.
Also adds some molecule tests. The verify step of the webclient test currently hangs on Travis but passes locally, so is disabled. Previously there were no tests at all for these playbooks so this is still an improvement.

Last two commits add OMERO_TMPDIR and webtagging. If everyone's happy with these settings downtime can be scheduled.

The last two commits aren't deployed as it requires a restart and therefore downtime. I'll send out an announcement for Monday morning.

Last few commits deployed this morning


Setup initial Electron shell wrapper - [3]

Includes basics for build system, root js, and renderer. Verify build and runtimes.

Hi James, I will be in there hangout now. Meet me there when you get a chance. Thanks
Slim Geransar
(646)706-3382
slim<EMAIL_ADDRESS>[cid:CE92FAD3-CF75-4CDD-A3D3-9B390309CBAB]
On Jul 27, 2015, at 1:14 PM, James T <notifications<EMAIL_ADDRESS>wrote:
Includes basics for build system, root js, and renderer. Verify build and runtimes. [3]
—
Reply to this email directly or view it on GitHubhttps://github.com/PancakeBot/PancakeCreator/issues/2.

Root is in, basic build is working on Windows & Mac with size constraints. Will complete tomorrow morning.


Change default server URL

here https://github.com/lutraconsulting/mergin-db-sync/blob/master/dbsync.py#L42
to https://app.merginmaps.com/

https://github.com/MerginMaps/mergin-db-sync/pull/64


TypeScript Errors

This project looks really interesting. However, I am getting several TypeScript compile errors:
node_modules/dynamodb-onetable/dist/cjs/Model.d.ts:44:11 - error TS2304: Cannot find name 'OneType'.

44     type: OneType,
             ~~~~~~~

node_modules/dynamodb-onetable/dist/cjs/Model.d.ts:188:2 - error TS1036: Statements are not allowed in ambient contexts.

188 };
     ~

node_modules/dynamodb-onetable/dist/cjs/Table.d.ts:21:25 - error TS2314: Generic type 'Model<T>' requires 1 type argument(s).

21     intercept?: (model: Model, op: string, rec: {}, params: OneParams, raw?: {}) => void,
                           ~~~~~

node_modules/dynamodb-onetable/dist/cjs/Table.d.ts:57:2 - error TS1036: Statements are not allowed in ambient contexts.

57 };
    ~

Found 4 errors.

Any ideas? Thanks

Can you rename OneTypes to OneType in that file and see if that works?

changed it. cleaned up a few files. one last error:
node_modules/dynamodb-onetable/dist/cjs/Table.d.ts:21:25 - error TS2314: Generic type 'Model<T>' requires 1 type argument(s).

21     intercept?: (model: Model, op: string, rec: {}, params: OneParams, raw?: {}) => void,
                           ~~~~~

it looks like the type definitions need to be cleaned up

changed to intercept?: (model: Model<any>, op: string, rec: {}, params: OneParams, raw?: {}) => void, and it compiled

Cool. Probably should be AnyModel.

yep. AnyModel worked.  FYI I also had to remove the trailing ; in the following files Model.d.ts, Table.d.ts

Great. I don't understand removing the trailing semicolon?  Can you explain a bit more? Thanks.

It seems that type declarations can use either semicolon or comma as a line terminator. However, the class declarations (Model) require a semicolon.
We have some types line OneTableSchema that use commas:
type OneIndexSchema = {
    hash: string,
    sort?: string,
    description?: string,
    follow?: boolean,
}

and others that use semicolo:
export type Entity<T extends OneTypedModel> = {
    [P in keyof T]?: EntityField<T[P]>;
};

I'd like to be consistent. What are the best practices / rules. Comma, semicolon or nothing?

Background:
https://stackoverflow.com/questions/27994253/why-is-the-separator-in-a-typescript-typememberlist-semicolon-as-opposed-to-comm/27994757#27994757

these are the errors:

node_modules/dynamodb-onetable/dist/cjs/Model.d.ts:188:2 - error TS1036: Statements are not allowed in ambient contexts.

188 };
     ~
node_modules/dynamodb-onetable/dist/cjs/Table.d.ts:57:2 - error TS1036: Statements are not allowed in ambient contexts.

57 };

have no idea why. if i remove the semicolon at the end of the files, it works.

What version of TSC are you using?
I don't see that with 4.0.3

And that is complaining only about the very last semicolon in the file but not the ones before that?

I am at Version 4.1.5. yeah, it doesn't complain about the ones before. just the last one.

This is our tsconfig we use when testing:
{
    "compilerOptions": {
        "allowJs": true,
        "allowSyntheticDefaultImports": true,
        "baseUrl": "src",
        "declaration": true,
        "esModuleInterop": true,
        "inlineSourceMap": false,
        "lib": ["esnext"],
        "listEmittedFiles": false,
        "listFiles": false,
        "moduleResolution": "node",
        "noFallthroughCasesInSwitch": true,
        "pretty": true,
        "resolveJsonModule": true,
        "rootDir": "src",
        "skipLibCheck": true,
        "strict": true,
        "traceResolution": false
    },
    "compileOnSave": false,
    "exclude": ["node_modules", "dist"],
    "include": ["src"]
}

Can you please share yours and we'll test.
I'm happy with the fix, but I'd like to understand why.

{
  "compilerOptions": {
    "resolveJsonModule": true,
    "esModuleInterop": true,
    "lib": ["es2020", "dom"],
    "target": "es2020",
    "removeComments": true,
    "moduleResolution": "node",
    "noUnusedLocals": true,
    "noUnusedParameters": true,
    "inlineSourceMap": true,
    "inlineSources": true,
    "module": "commonjs",
    "isolatedModules": true,
    "outDir": "./dist",
    "rootDir": "./src",
  },
  "include": ["./**/*.ts"],
  "exclude": ["node_modules/**/*", ".serverless/**/*", ".webpack/**/*", "_warmup/**/*", ".vscode/**/*"]
}

skipLibCheck in our tsconfig was the culprit.
Sorry, we'll apply the fix.  Good to understand why we were not seeing this.

nice! thanks for validating!

That has uncovered a few other issues ... :-(

Thank you for assisting. Appreciated.


Default Timezone

System Information:

Version: 2.4
Language: en-US
Timezone Offset: -04:00
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/45.0.2454.85 Safari/537.36

Problem/Needed Feature: Please allow a way to change timezone on this extension. I travel frequently and I rely on my timezone staying consistent. Thanks!

+1


Why you using website of weheartit for requests?

Why you using website of weheartit for requests? when offical mobile app of weheartit have an api for requests -> api.weheartit.com

The app's API requires account authentication to request and receive weheartit objects, as far as I know. This API client is provided for people who do not wanna authenticate with an account before proceeding to receive things from weheartit. (Though I understand there's some limitations on the number of objects you can receive, not a bad trade off).
I would be interested in making a wrapper out of their app's API but I'm not sure how the authentication works entirely. if you could find out about the authentication I'd be more than happy to do something with it, would be a huge improvement for the project.

The app's API requires account authentication to request and receive weheartit objects, as far as I know. This API client is provided for people who do not wanna authenticate with an account before proceeding to receive things from weheartit. (Though I understand there's some limitations on the number of objects you can receive, not a bad trade off).
I would be interested in making a wrapper out of their app's API but I'm not sure how the authentication works entirely. if you could find out about the authentication I'd be more than happy to do something with it, would be a huge improvement for the project.

Here i made a simple login function that uses the api of application
i captured the request that app makes for login with tool
class WeHeartIt:
    def __init__(self):
        self.api = "https://api.weheartit.com"
        self.headers = {
            "user-agent": "okhttp/3.14.9",
            "x-weheartit-client": "os: 'Android', sdkVersion: '25', device: 'ASUS_Z01QD', appVersion: '9.0.1.RC-GP-Free(21892) (21892)'"
        }

    def login(self, username: str, password: str):
        data = {
            "username": username,
            "password": password,
            "signature": "af3fc56e9223a0f327cac372cd56225b",
            "grant_type": "password"
        }
        response = requests.post(
            f"{self.api}/oauth/token",
            data=data,
            headers=self.headers).json()
        if "access_token" in response:
            self.access_token = response["access_token"]
            self.headers["authorization"] = f"Bearer {self.access_token}"
        return response

you can use it like this
weheartit = WeHeartIt()
weheartit.login(username="", password="")

That seems really interesting. I will definitely add that and work on bringing in new features and improving the codebase, hopefully moving away from the website and using BeautifulSoup itself.
Would you like to fork the repo and open a pull request with the changes? I think it'll give you the credits you deserve because this method looks very promising and I didnt even think of it before.

That seems really interesting. I will definitely add that and work on bringing in new features and improving the codebase, hopefully moving away from the website and stop using BeautifulSoup. Would you like to fork the repo and open a pull request with the changes? I think it'll give you the credits you deserve because this method looks very promising and I didnt even think of it before.

I've made a branch where I'll be implementing this for now. I'll be populating it with new code every now and then. What tool did you use to capture the app requests? I'm looking to use it for capturing other requests for the API

That seems really interesting. I will definitely add that and work on bringing in new features and improving the codebase, hopefully moving away from the website and stop using BeautifulSoup. Would you like to fork the repo and open a pull request with the changes? I think it'll give you the credits you deserve because this method looks very promising and I didnt even think of it before.

I've made a branch where I'll be implementing this for now. I'll be populating it with new code every now and then. What tool did you use to capture the app requests? I'm looking to use it for capturing other requests for the API

can you send me your telegram or discord
i will write

That seems really interesting. I will definitely add that and work on bringing in new features and improving the codebase, hopefully moving away from the website and stop using BeautifulSoup. Would you like to fork the repo and open a pull request with the changes? I think it'll give you the credits you deserve because this method looks very promising and I didnt even think of it before.

I've made a branch where I'll be implementing this for now. I'll be populating it with new code every now and then. What tool did you use to capture the app requests? I'm looking to use it for capturing other requests for the API

can you send me your telegram or discord i will write

Sure, here's my discord ID, goldy.#5911

I've made a github repo where I've listed all the API endpoints I enumerated from the latest version of the app. Hoping to test most of them with the right data soon.

I've made a github repo where I've listed all the API endpoints I enumerated from the latest version of the app. Hoping to test most of them with the right data soon.

i sent friend request sazerac#1516

Update: due to the complications of replicating oauth access this improvement might take some time, but it's definitely something weheartpy would like to achieve. Closing the issue for now, a new branch will be created when its time to implement these features.


python-wrapper Add integration test for all branch commands

          Can you open an issue for this? :)

Originally posted by @N-o-Z in https://github.com/treeverse/lakeFS/pull/7019#discussion_r1398474649

added in #7019


Fix(eos_cli_config_gen): Router bgp vlan config order

Issue Summary
Router bgp vlan config order.
Which component(s) of AVD impacted
eos_cli_config_gen
How do you run AVD ?
Ansible CLI (with virtual-env or native python)
Steps to reproduce
router_bgp:
    as: 65000
    vlans:
        66:
            rd: <IP_ADDRESS>:66
            redistribute_routes:
            - learned
            route_targets:
                both:
                - 11100:66
        67:
            rd: <IP_ADDRESS>:67
            redistribute_routes:
            - learned
            route_targets:
                both:
                - 11100:67
        400:
            rd: <IP_ADDRESS>:400
            redistribute_routes:
            - learned
            route_targets:
                both:
                - 11100:400
        500:
            rd: <IP_ADDRESS>:500
            redistribute_routes:
            - learned
            route_targets:
                both:
                - 11100:500
        600:
            rd: <IP_ADDRESS>:600
            redistribute_routes:
            - learned
            route_targets:
                both:
                - 10101:600
        666:
            rd: <IP_ADDRESS>:666
            redistribute_routes:
            - learned
            route_targets:
                both:
                - 11100:666
        667:
            rd: <IP_ADDRESS>:667
            redistribute_routes:
            - learned
            route_targets:
                both:
                - 11100:667
        700:
            rd: <IP_ADDRESS>:700
            redistribute_routes:
            - learned
            route_targets:
                both:
                - 10101:700

Expected Config:
router bgp 65000
   !
   vlan 400
      rd <IP_ADDRESS>:400
      route-target both 11100:400
      redistribute learned
   !
   vlan 500
      rd <IP_ADDRESS>:500
      route-target both 11100:500
      redistribute learned
   !
   vlan 600
      rd <IP_ADDRESS>:600
      route-target both 10101:600
      redistribute learned
   !
   vlan 66
      rd <IP_ADDRESS>:66
      route-target both 11100:66
      redistribute learned
   !
   vlan 666
      rd <IP_ADDRESS>:666
      route-target both 11100:666
      redistribute learned
   !
   vlan 667
      rd <IP_ADDRESS>:667
      route-target both 11100:667
      redistribute learned
   !
   vlan 67
      rd <IP_ADDRESS>:67
      route-target both 11100:67
      redistribute learned
   !
   vlan 700
      rd <IP_ADDRESS>:700
      route-target both 10101:700
      redistribute learned

Actual Config:
router bgp 65000
   !
   vlan 66
      rd <IP_ADDRESS>:66
      route-target both 11100:66
      redistribute learned
   !
   vlan 67
      rd <IP_ADDRESS>:67
      route-target both 11100:67
      redistribute learned
   !
   vlan 400
      rd <IP_ADDRESS>:400
      route-target both 11100:400
      redistribute learned
   !
   vlan 500
      rd <IP_ADDRESS>:500
      route-target both 11100:500
      redistribute learned
   !
   vlan 600
      rd <IP_ADDRESS>:600
      route-target both 10101:600
      redistribute learned
   !
   vlan 666
      rd <IP_ADDRESS>:666
      route-target both 11100:666
      redistribute learned
   !
   vlan 667
      rd <IP_ADDRESS>:667
      route-target both 11100:667
      redistribute learned
   !
   vlan 700
      rd <IP_ADDRESS>:700
      route-target both 10101:700
      redistribute learned

Relevant log output
No response
Contributing Guide

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

It seems EOS is sorting vlans under router bgp as strings ie vlan 66 comes before vlan 666, BUT after vlan 600.


Add weather to internal datasets

🚀 Feature Request
We want to add weather dataset with 10 min frequency. It contains 21 series for 2020.
Proposal

Add weather to datasets_dict with 10 min freq:

datasets_dict = {
    "weather_10T":  {
        "get_dataset_function": get_weather_dataset,
        "freq": "10T",
        "parts": ("train", "test", "full")
    },
}

Add get_weather_dataset function for loading weather dataset. We have to add dataset description and reference link to docstring:

def get_weather_dataset(dataset_path: Path) -> None:

Dataset has no missing values.
All series have the same length.
We define test part as last 24 hour period (144 points).

Test cases

Check weather datasets statistics (number of segments, min/max dates).

Additional context
See https://github.com/etna-team/etna/issues/84 as an example.

Closed by https://github.com/etna-team/etna/pull/125.


Httpd error:httpd.conf: Invalid LoadModule path D:/\xc8\xed

CurrentBehavior
After installing wampserver, the chart will continue to display red after startup and cannot be started. Then, follow the method on Baidu to open a command prompt line in the corresponding folder, and run the httpd command with an error

ExpectedBehavior
Wampserver can be used normally

To the maintainers of this repo, this account is a bot, I have reported them to github, but it's very strange.
It seems to copy people's issues from other places and post them.

:disappointed: thanks for the heads up :heart:


[BUG] Crash due to duplicate __PYX_XCLEAR_MEMVIEW

Describe the bug
When using a property setter that accepts a memory view, Python crashes with the following error:
Fatal Python error: __pyx_fatalerror: Acquisition count is -1 (line 16959)
Python runtime state: initialized

Current thread 0x00000001e83a1e00 (most recent call first):
  File "/private/tmp/cdtmp.l8Y/test.py", line 7 in <module>

Extension modules: cython.cimports.libc.math, foo (total: 2)

Code to reproduce the behaviour:
# foo.pyx
cdef class Foo:
    @property
    def bar(self):
        return b'bar'

    @bar.setter
    def bar(self, const unsigned char[::1] bar):
        pass

# test.py
#!/usr/bin/env python3

from pyximport import install; install()
from foo import Foo

c = Foo()
c.bar = b'some string'  # <-- this crashes

In the generated foo.c, I can see the following code where __PYX_XCLEAR_MEMVIEW is called twice on the same variable which is likely the cause of the bug:
/* Python wrapper */
static int __pyx_pw_3foo_3Foo_3bar_3__set__(PyObject *__pyx_v_self, PyObject *__pyx_arg_bar); /*proto*/
static int __pyx_pw_3foo_3Foo_3bar_3__set__(PyObject *__pyx_v_self, PyObject *__pyx_arg_bar) {
  /* ... */

  /* function exit code */
  __PYX_XCLEAR_MEMVIEW(&__pyx_v_bar, 1);
  __PYX_XCLEAR_MEMVIEW(&__pyx_v_bar, 1);
  __Pyx_RefNannyFinishContext();
  return __pyx_r;
}

Expected behaviour
The code is executed without crashing the Python interpreter.
OS
macOS
Python version
Python 3.11.4
Cython version
Cython version 3.0.0
Additional context
I can also reproduce the error on Debian 11 with Cython 3.0.0 and Python 3.9.2

So the double __PYX_XCLEAR_MEMVIEW isn't right, but it should be safe. The crash is actually on the first one

Thank you for debugging the code and creating a patch.


Geras don't have wanted icon

Description

Wanted icon from the Criminal Records Computer
Reproduction

set a slime to wanted
turn into a geras and have no icon

Geras 1984'd
Fixed by #29731


Fix health discrepancies

Health isn't updating properly for players. Need to fix this

Just ran a few ore test games.  The local player should be the only thing trying to updates it's own health.

Change the Bullet.hit method to only test against the local player

Bullets now not colliding with players at all. Will look into this later on

console is logging hits,  but I think the destroy method is broken

Should have been fixed by the merge. Closing it manually.


Remove double forward slash "/" from function call

Remove double forward slash "/" from function call. Noticed today that after upgrading to the latest 2.0.4 @nhost/react package that function calls made with the Nhost client were being requested using double // forward slashes in the url.
http://localhost:1337/v1/functions//path/subpath
this happens regardless of which way I call it
nhost.functions.call("/path/subpath",.....
or
nhost.functions.call("path/subpath",.....
Small change but this should fix the issue.

Nice catch!
Closed in favor of https://github.com/nhost/nhost/pull/1670


Display simple pdf on android like iOS when there is only one screeen

Thanks for the awesome component. Is there possible to remove the extra file option on android like remove plus and minus buttons and up, down arrow buttons on bottom and page counter on top.
iOS seems okay. It is happening on android because of allowFileAccess webView props. but is there any workaround to remove the extra content.
Thanks

No this is the reason why we have this lib...


Fixes crash when loading previously saved file with connections

This fixes #158

Great, thanks a lot!
I see now what happened. I renamed the internal class member Connection::_id to Connection::_uid by blindly replacing all the entries in the source file and broke this function.
The original name _id was conflicting with the moc-generated code.
Dmitry


.Net 2.0 Version : ExcelBinaryReader is not adding first row to dataset table even if _isFirstRowAsColumnNames==false

Hello,
just a small bug.
In the .NET 2.0 version even if _isFirstRowAsColumnNames==false in the method readWholeWorkSheetNoIndex the first row is not added to the table of the dataset.
The line
if (!justAddedColumns && m_depth > 0 && !(_isFirstRowAsColumnNames && m_maxRow == 1))
should become
if (!(justAddedColumns && _isFirstRowAsColumnNames) && m_depth > 0 && !(_isFirstRowAsColumnNames && m_maxRow == 1))

I think the issue is that initializeSheetRead reads to the first row, and then moveToNextRecordNoIndex starts by reading until the next row skipping the first one.

Closing because inactivity, and there's been so many changes to the code in question, chances are this was fixed.


New feature "Identify" is great, but doesn't clear lights when complete

Clicking the 'Identify' button flashes the landing lights on the PZ55. However they don't clear or reset after a short time. Nor does the 'identify' act as a toggle.
If it's going to become a toggle, then fine, when it's toggled off the lights need to go back out (or to their previous state if already coloured)
If it's not going to be a toggle and the flashing lasts for a set period of time, then they need to go back out (or to their previous state if already coloured)

I do so that all lights go out after the process if finished.


Adjust firewall script for El Capitan

Drops anchor in pf
Resolves issue #507

:+1:

Woo!

:+1:

:+1: Please merge if this resolve #507.

This fixed things for me too :+1:

Does this work on Mavericks / Yosemite as well, or is the anchor required there?

This PR is obsolete.  This change was only required for El Cap Public Beta 1.


Add support for using -target in terraformOptions

I have 2 modules in my terraform file and would like to invoke only one module from the terraform file via my Terratest suite. Unfortunately, there is no support in Terratest for this.
Ideally in my test suite, I would want to call the first module via -target and then call the second module via -target, thereby I can test upgrading to v1.4.6 does not cause any issues.
Terraform does not support using 'count' on modules yet and hence this feature to use -target in terraformOptions of Terratest will be of great help.
Terraform Module
provider "aws" {
region = "${var.region}"
}
module "aws_s3_previous" {
#Testing source from this module
//count = "${var.current == false ? 1: 0}"
source   = "git::ssh://xxxx-xxxxxx-xxxxxxxx-xxxxxx-bucket.git?ref=v1.4.5"
region   = "${var.region}"
app_name = "${var.app_name}"
owner    = "${var.owner}"
allow_force_destroy = "${var.allow_force_destroy}"
}
module "aws_s3_current" {
#Testing source from this module
//count = "${var.current == true ? 1: 0}"
source   = "git::ssh://xxxx-xxxxxx-xxxxxxxx-xxxxxx-bucket.git?ref=v1.4.6"
region   = "${var.region}"
app_name = "${var.app_name}"
owner    = "${var.owner}"
allow_force_destroy = "${var.allow_force_destroy}"
}
terraform {
backend "local" {
path = "../stateFiles/terraform.tfstate"
}
}
Use case
Ability to do "terraform apply -target=${module.aws_s3_previous}" in Terratest.

Sure, a PR for that would be welcome. You could add it to terraform.Options and if set, add the appropriate CLI args.

I want to contribute on this one, my question is, how do you do variable interpolation? Are these variables available at all? Or do I need to read the plan and find that

This must be closed


Unit Testing the IRespository

Hello again I have another question
I was wondering how I could go about Unit Testing the Reposiotories that I make since they depend on IRespository and to add that I would be forced to add all the other dependencies of IRespository
1)Dapper
2)Dapper Fast Crud
Thanks again

Hi
I have lots of unit tests on the project site under the "tests" folder in the code. There is even some more difficult mocking with it.
Yes, repository project does depend on dapper and FastCRUD. If you do not want to use them, then you can fork the code and pull the interfaces out into there own project. It was just a decision I had to make for the package.
There is a separate Uow without Repository project precisely to keep it clean. But as I said, I just had to make a compromise for the repo project. Normally when I develop I separate classes and interfaces. I could do this here, but then the package is not 1 dll. Compromises are difficult...
I can rip into 3 projects (uow, repo.interface and repo.core) but honestly, is it an issue to have dapper and FastCRUD as nuget in the test project?
Let me know what you think :-)
Also, what IOC/Di are you using?

Path to repo tests https://github.com/generik0/Smooth.IoC.Dapper.Repository.UnitOfWork/tree/master/src/Smooth.IoC.Dapper.Repository.UnitOfWork.Tests/ExampleTests/Repository

I am using Autofac

Yeah I understand the difficulties of the development :-)

Ok. I am using autofac for a aspnetcore project right now also. I used to be a castle guy. But autofac is good too.
I have to update the examples to use a autofac module. I believe it is best practice. Simple enough. The Load just has to be overritten.
Br Rik


include jade file gulp-pug 2.0

I use now version 2.0 and I did not have this issue with version 1.3.0 but  now include jade file creates text instead of code block of pug
index.pug
doctype html
html
  head
    title  Yusuf Özlü (ozluy)
    meta(name='viewport' content ='width=device-width, initial-scale=1, user-scalable=no')
    meta(charset='utf-8')
    meta(http-equiv='X-UA-Compatible', content ='IE=edge')
    link(href='styles/bundle.css', rel='stylesheet', type='text/css')
    link(rel='shortcut-icon', href='favicon.ico')
  body
    include partials/_footer.jade
    script(src='scripts/bundle.js' )

footer.jade
footer
  p hello I am footer

html

<!DOCTYPE html>
<html>
  <head>
    <title> Yusuf Özlü (ozluy)</title>
    <meta name="viewport" content="width=device-width, initial-scale=1, user-scalable=no">
    <meta charset="utf-8">
    <meta http-equiv="X-UA-Compatible" content="IE=edge">
    <link href="styles/bundle.css" rel="stylesheet" type="text/css">
    <link rel="shortcut-icon" href="favicon.ico">
  </head>
  <body>  footer
    p hello I am footer

    <script src="scripts/bundle.js"></script>
  </body>
</html>

Hello, thanks for reporting.  Me and the owner of gulp-jade actually recently came to a compromise where the old gulp-jade package is going to be renamed and repurposed for pug.  That should make support from gulp-jade equal to the new coming gulp-pug.

I changed your package(gulp-pug) with gulp-jade, and problem solved
cheers

@ozluy: gulp-pug@3 has been release.  Closing now.


A few questions

Hello
I have read the Readme and seen a video or two on youtube about Digger.
It looks great!
From my understanding it is a continueous delivery system for Android and Mobile apps.

Does digger have a custom s2i image builder?
Or is this not required because building is done in jenkins - and because the final artifact is not a docker image, but a mobile application?
Where is the binary stored once it is built?
Are there any plans for it to push all the way into the app stores/google play?

Thanks

Hello @magick93, sorry for the delayed response:

Correct, we don't use a custom s2i image builder because the end result is the actual mobile app binary (the build itself runs inside a openshift pod)
The app binary is stored in jenkins as a job artifact.
Our worker android container  contains a pre-installed version of fastlane, so you could use fastlane to publish your app binary in google play/app store, fastlane samples can be found here.

There is also this fastlane docs on how to publish your app: https://docs.fastlane.tools/actions/supply/#uploading-an-apk
You can also find us (the aerogear community) in the following communication channels:
Mailing lists: aerogear-users and aerogear-dev
IRC: #aerogear channel in the freenode network.

thanks @odra !
One more question. What do you use nagios  for?

And one more question: as you are not using s2i, how do I get started?
I have a git repository with mobile app source. How / where in OpenShift do I add this git url?

Nagios is used to monitor certain components such as jenkins slave availability, android sdk volume file permissions, etc.
You just need to create a new job in jenkins and trigger a build there but you can also use our nodejs cli tool: https://github.com/aerogear/aerogear-digger-node-client


[@azure/identity] Uncaught Error: EnvironmentCredential is not supported in the browser.

Package/Version<EMAIL_ADDRESS>Is your feature request related to a problem? Please describe.
When I attempt to use @azure/identity in the browser I get this error:
Uncaught Error: EnvironmentCredential is not supported in the browser.
Describe the solution you'd like
I would like this use case to actually work. We inject env vars into the build using webpack. Its a convenient way to inject various bits in test scenarios. In our case we make a "fake" Azure Portal wrapper around our project that injects these variables into the page. Webpack will handle the injection for us, but it won't work when this case is explicitly rejected by the identity SDK.

cc @jeremymeng, @chradek, @bterlson, @xirzec, @maorleger for any additional thoughts before closing

I think I'm fine with manually plumbing environment variables. It might be convenient for tests to have this auto-picked up but it feels like such a risky thing to enable/have built in support for. You really should think hard using env vars in the browser.

@ramya-rao-a @southpolesteve I don’t think this makes a lot of sense, since the only credential we really support in the browsers is the InteractiveBrowserCredential. EnvironmentCredential uses client secret or client certificate authentication, that can be made to work by disabling browser security in the browsers. Browsers will block these requests because they imply that CORS has been enabled for these services, which if possible, would require further configuration on the outgoing requests. At the moment, InteractiveBrowserCredential is the only credential that we can advertise that works for browsers.

@southpolesteve For the specific scenario you mention, would you be willing to consider writing a custom TokenCredential that does what you want?


How to create .ogg video file?

This works:
sd run -it -rm -v $(pwd):/v jrottenberg/ffmpeg -i /v/video.mp4 -movflags faststart /v/video.webm

This does not work:
sd run -it -rm -v $(pwd):/v jrottenberg/ffmpeg -i /v/video.mp4 -movflags faststart /v/video.ogg

Error:
Encoder (codec theora) not found for output stream #0:0

How do I encode to OGG Theora?

You are absolutely right, it's a bug, testing a fix, but there should be a new build by end of day.
Thanks a lot !

Built on the hub and Tested :
docker run -it --rm -v $(pwd):/tmp/workdir jrottenberg/ffmpeg  -i video.mp4 -movflags faststart   video.ogg
[...]
Stream mapping:
  Stream #0:0 -> #0:0 (h264 (native) -> theora (libtheora))
  Stream #0:1 -> #0:1 (ac3 (native) -> vorbis (libvorbis))
Press [q] to stop, [?] for help
[...]

docker pull jrottenberg/ffmpeg
(updated 2.8 tag as well)

Thanks, perfect.


not working

I'm using SQL Express 2019 and SSMS 18.4 and trying to delete sql express instances but find that when following this, there is no installation pop up for me to carry out. Instead it unintalls straight leaving the sql instances behind

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

ID: 5559d56d-e787-2c02-970d-a2825911d81e
Version Independent ID: 99f7b922-9fca-dc06-f934-d4a87c098764
Content: Uninstall existing instance - SQL Server
Content Source: docs/sql-server/install/uninstall-an-existing-instance-of-sql-server-setup.md
Product: sql
Technology: install
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

Hi @tjcheah , thanks for contacting Microsoft! Unfortunately, it sounds like you're running into a support issue, and this forum is intended for issues with the documentation.
Although in this case I can offer some preliminary assistance, you may still need to reach out to the community for assistance, or possibly open a support ticket with Microsoft.
Would you mind taking a screenshot of your Programs and Features, and highlighting which program you're choosing to uninstall when attempting to uninstall SQL Server Express?
Thanks!
Masha from the SQL Docs team

Hi,
thanks for coming back to me. I have attached the screenshot with all of my
SQL programs and features listed. The highlighted line was what i was
uninstalling as per the instructions found on the forum. However when I did
it, the uninstall pop up never came up so hence I could not choose parts to
uninstall and was only given the option to uninstall everything.
[image: Mailtrack]
https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality5&
Sender
notified by
Mailtrack
https://mailtrack.io?utm_source=gmail&utm_medium=signature&utm_campaign=signaturevirality5&
16/04/20,
09:13:42
On Wed, Apr 15, 2020 at 12:15 AM Masha at MSFT<EMAIL_ADDRESS>wrote:

Hi @tjcheah https://github.com/tjcheah , thanks for contacting
Microsoft! Unfortunately, it sounds like you're running into a support
issue, and this forum is intended for issues with the documentation.
Although in this case I can offer some preliminary assistance, you may
still need to reach out to the community for assistance, or possibly open a
support ticket with Microsoft.
Would you mind taking a screenshot of your Programs and Features, and
highlighting which program you're choosing to uninstall when attempting to
uninstall SQL Server Express?
Thanks!
Masha from the SQL Docs team
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/sql-docs/issues/4545#issuecomment-613537137,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOGKZR2Q36XTQQD5EGRIIRTRMSDYRANCNFSM4MHOBOPA
.

Hi @tjcheah , you'll need to paste the pictures into the comment on GitHub :( pictures don't come through over email.
Thanks!
Masha from the SQL Docs team

Hi @tjcheah , I tested it out, and I was able to uninstall SQL Server 2019 Express from the Add or remove programs settings page. When I chose to uninstall Microsoft SQL Server 2019 (64-bit), my installation window came up, and I was able to select the features that I wanted to remove.

Thanks!
Masha from the SQL Docs team

Hi @tjcheah, it's been some time since we we've heard from you, so we're going to assume that your issue is resolved and close out this Git Issue.
However, please feel free to comment should you need anything further!
Thanks again for contacting Microsoft and I hope you have a great rest of your day!
Masha from the SQL Docs team


⬆ Update dependencies

Nevermind, doesn't work with the newer versions of Node.
I'm doing a hard-fork that will resolve these issues.


mime type application/json not supported

I have some cells generating as the first output json data (mime type: application/json). I get the following warning:
WARNING: skipping unknown output mime type: application/json [mystnb.unknown_mime_type]

which comes from https://github.com/executablebooks/MyST-NB/blob/master/myst_nb/core/render.py#L746
First it would be great if the same kind of nicely displayed json as in jupyter would be displayed in the documentation.
Second, I'd like to be able to silent this warning, which is not interesting for us (and we try to keep the documentation build with zero warning).

I am also very much interested in this! And it is quite disappointing that this seemingly standard IPython output type which works in Jupyter goes away when using myst-nb/nbsphinx.
Further investigation shows that it is not necessarily Myst-NB's fault:

We could implement a rendering plugin for myst-nb. But it is actually not a custom type, rather a standard type from IPython.
The IPython JSON display object only stores the data, but does not provide mime type representations for it. However for other display types like Audio, it implements _repr_html_.
The representation in JupyterLab comes from a JupyterLab "json-extension".
Apparently, the execution output is stored in MIME types "application/json" and "text/plain", but there is no "text/html" (which is otherwise stored, e.g. for Pandas dataframes).

So although it seems to be a standard output type, it is left to HTML renderers to implement the representation. JupyterLab is able to handle the missing "text/html" and just generates it from the JSON.
I think we should look whether we can patch whatever executes the notebooks to store "text/html" (not necessarily dynamic with javascript). Does this make sense? And I'm not sure in which project this is, IPython or jupterlab?

I created this plugin: https://pypi.org/project/myst-nb-json/
Just add myst-nb-json to your dependencies and the built HTML will show JSON similar to Jupyter.


Fix spacing issues in example files

         paragraph temporary lorem test tag

I had to use a workaround involving spaces in italic inner html
       i.innerHTML = ' ' + element.innerHTML + ' ';

You can close this, i tested it on my app end. it is working well!
Thanks!


Add support for local weather station data (same as on iOS)

Let's add this feature on Android because our target is to offer similar features on the both applications. Users also need a way to test the application without purchasing physical Ruuvi sensors first and it's possible with these location based "virtual sensors".
Let's try to simplify the feature by adding no map but a search bar only. If it seems that a map is not mandatory, then we can remove the map also from the iOS app.
We also may would like to rename this feature from virtual sensors to something else. So, please use unified translation strings with the iOS.

Our current plan is not to add this but instead we'll implement demo Ruuvi Cloud sensors. Closing this one.


Creating the Cold Pool Index Mask

The original mask is located in GFDL and require mask export from GFDL folks. Currently, we decided to use the cold pool index mask generated solely from the GLORYS bottom temperature (1993-2022) using the same criteria from the code in CEFI-regional_MOM6. Since the mask can be switch from one to another. Current approach is to make a mask from only GLORYS and replace the mask file when it become available. The module should be designed to handle the mask file changes.

[x] create mask from GLORYS data
[x] the module only need to read in the mask data for CPI calculation
[ ] the mask replace by the GFDL mask

#34 will be the cold pool index implementation. Mask from the GFDL will be replace when it becomes available.


Added com.amazon.aws.cloudfront/wd_access_log/jsonschema/1-0-4 (close…

…s #199)

Hey @fblundun ! This looks good - but I think that if we are adding maxLength constraints on an existing field like scStatus, we have to backport ("patch") the prior schemas to have the same constraints, or else we don't strictly have an additive change... Do you agree?

Good point! Should I add the maxLengths to the previous versions, or just make this version 2-0-0?

Me on HipChat in case anybody is interested:
I think a good test of these sorts of questions is to imagine we were starting from scratch today
so imagine we are sitting down to draft 1-0-0
knowing what we know about schema-guru
we would add those constraints from day 1
therefore, I think we should patch the existing schemas

Thanks Fred, looks good


[Selfhost] Instance keeps redirecting to http

Currently, there is no way of setting https only with docker.
I needed to open port 80 and redirect it manually to https using reverse proxy.
I have tried setting APP_DOC_URL APP_HOME_URL GRIST_ROUTER_URL , etc.. but none works.

Not sure I entirely understand the question. The Grist docker image doesn't deal with https, that is true. There is a complete example of hosting Grist, with certificates using Let's Encrypt, at https://community.getgrist.com/t/a-template-for-self-hosting-grist-with-traefik-and-docker-compose/856

What I need is for grist to also specify the protocol, instead of only domain.

Do you mean you'd like the Grist docker container to have something within it that:

terminates SSL connections (perhaps using certificates that you provide?)
handles redirection of http requests to https
so that you don't need to handle that yourself using a reverse proxy? If so, I can understand wanting that, and I'll mark this as a feature request. Just want to be sure you're not having some more fundamental problem with getting Grist working.

Yes, it's sort of a feature request. Although I think it would be best to let reverse proxy handles ssl certs. I just need to be able to specify http scheme using variable.

I just need to be able to specify http scheme using variable.

It really sounds like you are seeing a problem of some kind, but I can't guess what it is. Maybe #149 is relevant?

Nope, I was able to get it working. But it would be cleaner if there is a way to specify protocol without having to open port 80 and redirect it to 443. The method you're suggesting using traefik will create more frictions as I have my own reverse proxy in place.

I don't understand what port 80 or 443 would have to do with the Grist container. If a user attempts to visit your site via http on port 80, it would be the job of the reverse proxy to redirect them, right? Grist should only see the traffic after ssl termination, passed along on port 8484 or whatever you choose?

My ideal setup is to only open port 443 and map port 8484:8484 to grist container. Then the reverse proxy 443=> localhost:8484=>grist container. This setup is not possible, because grist container defaults the address to http. I needed to do reverse proxy 80=>redirect to 443=> localhost:8484 => grist container.

Can you tell me more about what you mean by This setup is not possible, because grist container defaults the address to http. What address?

Grist container keeps redirecting me to http://gristdomain/o/docs instead of https://gristdomain/o/docs

For your setup, I think the main thing you need is to set APP_HOME_URL to https://gristdomain (i.e. include the protocol and make it https -- it should be the URL for browsers to use). This is what's done in the link that @paulfitz shared earlier (https://community.getgrist.com/t/a-template-for-self-hosting-grist-with-traefik-and-docker-compose/856). That setup is similar to yours and is known to work.
BTW, I don't think either APP_DOC_URL or GRIST_ROUTER_URL is needed, but it's possible that setting GRIST_SINGLE_ORG=grist is important.

Like I said many times, it doesn't address the issue that grist can't handle https. It creates more frictions by adding another layer of reverse proxy on top of my reverse proxy.

Hello @finzzz i think you install docker without GRIST_SINGLE_ORG env.
This is a known issue that is currently being resolved, while it is not possible to install without
GRIST_SINGLE_ORG=myorganization
GRIST_ORG_IN_PATH=true
https://community.getgrist.com/t/grist-core-multi-user-docker-setup/666/8

I tried everything after searching similar issues in the forum and github. None works.

Just to confirm, this is definitely a bug, and can be easily replicated. If you visit https://gristdomain, it will redirect to http://gristdomain/o/docs; it should instead redirect to https://gristdomains/o/docs. Installations that redirect from http to https hide the problem. This is without GRIST_SINGLE_ORG set. You can see the code building a redirect url incorrectly here: https://github.com/gristlabs/grist-core/blob/05c89fb65a3e377449e31ec1e3662c7254686b70/app/server/lib/FlexServer.ts#L1521
This doesn't work because of the reverse proxy, the request protocol will be http but the user should be directed to https.

Just to clarify, now we can remove env:
GRIST_SINGLE_ORG=myorganization
And use GRIST_ORG_IN_PATH=true for multiple org for selfhosted install?

@Vladimir-Va if you wish to, yes - that is a separate issue though. The status of multi-org support hasn't changed recently (it works, but creating new orgs/teams is a manual process).


[PROPOSAL] Vary build numbers for every build

Is your feature request related to a problem? Please describe
Currently there's no way to distinguish between two builds of OpenSearch or OpenSearch Dashboards during development by looking at their version number, nor is there a way to introduce a new build when the build changes, but the semantics don't (e.g. as a result of https://github.com/opensearch-project/opensearch-build/pull/1130). This makes it difficult to release a new version where binaries are identical but the way we package them together, or the container itself, have changed.
Describe the solution you'd like
Introduce a build number. This number can be, for example, date/time-based (e.g.<PHONE_NUMBER>18), a random serial number, or a sequential number from the build system. The complete version number would be 1.2.4+202201131018 or <IP_ADDRESS>201131018 if we wanted to replace the last digit with this number. Every time anything is rebuilt, it gets a new build number, without affecting the major/minor/patch parts of it.

Policy / procedure question related to this: is it okay for the current build number to be different for various distribution artifacts at any point in time? Or should there be an expectation that they all match. Now that I type it out, I think that as an administrator I would prefer the former: I don't want to be prompted to update my docker image because an update to the windows installer was released. But if the build numbers vary, will users be concerned that anything they're running with a less-than-highest build number isn't 'current'?

Policy / procedure question related to this: is it okay for the current build number to be different for various distribution artifacts at any point in time? Or should there be an expectation that they all match. Now that I type it out, I think that as an administrator I would prefer the former: I don't want to be prompted to update my docker image because an update to the windows installer was released. But if the build numbers vary, will users be concerned that anything they're running with a less-than-highest build number isn't 'current'?

Haven't we already done this once with 1.0.1 that didn't release a new OpenSearch-min (remained at 1.0.0)? In this issue we're just generalizing this slightly, but whichever way we cut it, if we released distribution 1.2.3.A, then distribution 1.2.3.B is effectively a new release.

Two things:

SemVer states that build metadata can only be separated by +, so <IP_ADDRESS>201131018 is not possible under the SemVer Spec.
In the case of sequential or date-based identifiers, we should be careful not to treat this as a version precedence factor SemVer spec item 10  states that two version with different build metadata have the same version precedence. We shouldn't say 1.2.4+000002 is newer release than 1.2.4+000001.

@stockholmux ,
Regarding the +: Docker tags do not support +. So if we choose to create per-build tags in Docker, we'd need to use - or .. e.g. docker pull 1.2.4-202201131018.
Also, SemVer is focused on solving the issue of dependency resolution. When it comes to precedence, the OpenSearch maintainers will perform this resolution. It happens via both:

Updating the artifacts URLs to download the latest build (e.g. https://artifacts.opensearch.org/releases/bundle/opensearch/1.2.4/opensearch-1.2.4-linux-x64.tar.gz downloads build 1.2.4+000002)
The Docker tag will be moved to new images and we produce new Docker builds. (e.g. 1.2.4 is tagged on the 1.2.4-000002 image)

By the way, I created a different proposal which may have some impact on this particular discussion in #1621.

@zelinh @peterzhuamazon @prudhvigodithi  Can you please update this issue with the current status?

We have build number after docker tags right now.
We also have build number separate each tar build.
This can be closed for now.


Adjust TS2691 message for .ts import sources

Description
The advice given by error TS2691 is confusing, and following its instructions produces output which throws ERR_MODULE_NOT_FOUND at runtime when outputting an ES module, i.e. when scriptTarget >= ScriptTarget.ES2015 and moduleKind >= ModuleKind.ES2015.
An import for a module located at ./test.ts must be specified as import ... from './test.js' in order to get a valid ES module out, but users are directed to rename it to ./test. This error message should be updated so users are given the correct advice based on their use case, and are not confused why their output throws despite the source being perfectly valid TS. (The ./test format can be used to generate valid CommonJS modules, just not ES modules.)
Closes #42151.
Example
Current behavior
import { test } from './test.ts'

Currently throws TS2691 which says:
An import path cannot end with a '.ts' extension. Consider importing './test' instead.ts(2691)

This adjustment does silence the compiler, but the compiled JS looks like:
import { test } from './test';
console.log(test);

Which is not a valid ES module, as the import source is located at ./test.js. Executing the output throws ERR_MODULE_NOT_FOUND.
Suggested changes
This PR adjusts checker.ts to detect if the output is an ES module, and if so advises the user to replace ./import.ts with ./import.js instead of ./import.
After these changes, trying to import ./test.js while outputting an ES module throws:
An import path cannot end with a '.ts' extension. Consider importing './test.js' instead.ts(2691)

Which will compile to a valid ES module that behaves as expected.

Sure @andrewbranch - if you see this in time, is there a way to run just a single test in this workspace?

Sure @andrewbranch - if you see this in time, is there a way to run just a single test in this workspace?

npx gulp runtests --tests substringOfTestFileName

npx gulp runtests --tests substringOfTestFileName

@andrewbranch, so sorry to bug you again for such a low-priority PR, but I can't figure out how to do the equivalent of snapshot updating for these tests in /tests/baselnes/reference? I split the moduleResolutionNoTs.ts test into moduleResolutionNoTsCJS.ts and moduleResolutionNoTsESM.ts, but can't find out how to create corresponding files like /tests/baseline/reference/moduleResolutionNoTsCJS.errors.txt, .symbols.txt, etc.
Thank you so much for your help, super sorry for pinging you, did my best to search for context before asking.

@andrewbranch, so sorry to bug you again for such a low-priority PR, but I can't figure out how to do the equivalent of snapshot updating for these tests in /tests/baselnes/reference? I split the moduleResolutionNoTs.ts test into moduleResolutionNoTsCJS.ts and moduleResolutionNoTsESM.ts, but can't find out how to create corresponding files like /tests/baseline/reference/moduleResolutionNoTsCJS.errors.txt, .symbols.txt, etc.
Thank you so much for your help, super sorry for pinging you, did my best to search for context before asking.

super sorry for pinging you

Don’t be! Contributions are always appreciated and we expect first-time contributors to have some questions 👍

super sorry for pinging you

Don’t be! Contributions are always appreciated and we expect first-time contributors to have some questions 👍


iOS - Attachments - Not possible to attach photo from the Gallery which was taken with Live mode off

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
Preconditions: Settings --> Camera --> Formats --> should be set to "High Efficiency"

Open Native iOS Camera App
Turn off the Live Photo mode and take the photo
Open the NewDot App and try to upload previous photo from the Gallery

Expected Result:
The user user should be able to upload photos from Gallery without any issues
Actual Result:
Photo preview is available, but at the last strep format error appears.
Workaround:
Unknown
Platform:

Where is this issue occurring?

iOS

Version Number: <IP_ADDRESS>
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/93399543/196812712-44faba5b-b346-4b5e-9179-7cf183d30a3e.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Interesting! We have a related issue about attachment types that I will look for, but this seems weird b/c the image filename ends with .jpg which is definitely allowed 🤔  I thinkkkkk this needs to be internal

Trying to get someone to take over for me, got too many other issues on my plate atm

Pinged in #bug-zero for a CME to take over since I don't own a physical iPhone

@MonilBhavsar  volunteered to take over 🙏

This needs a BZ member, so adding myself.

Issue not reproducible during KI retests. (First week)

Ok, I just re-tested and we're talking about this same issue in multiple places. This is a dupe of https://github.com/Expensify/App/issues/12459. I'm closing this issue.


Added mercurial support

Now this theme also works with mercurial! :)
Only thing is:
I had to implement a very poor dirty check. Unfortunately I didn't find a better one that works with mercurial and git.. Now I just check, whether the main VCS_INFO-prompt contains a '●' or '✚'..

Thanks for the pull-request! Support for other VCSes would be awesome.
As you said, I'm not a huge fan of checking status by the existence of that character. I'm going to dig around and see if I can sort out another way to do this. If not, I'll merge this changeset.

Okay, I took a shot at doing this differently, and in the process changed the git printout somewhat. Have a look at my mercurial_support branch (bhilburn/powerlevel9k@3d4a1eba7da) and give it a shot. I don't really use Hg much, so I'm honestly not sure it's behaving correctly.
Feel free to send me feedback or pull-requests on that branch. If it looks good, I'll merge into master.

What bothers me most in this approach is that we build a functionality for an information that the vcs_info-subsystem has already gathered. For me that was the point to use the existance of the staged/unstaged character, to transport the information from the vcs_info to our needs.
Secondly we have to implement that functionality for every vcs we want to support. So there is a lot of clutter in the template.
Vcs_info supports staged/unstaged only for git, mercurial and bazaar (as the zshcontrib manpage says).
I'll search for another solution at the weekend.

Ah, I see what you are saying. You had changed the prompt_vcs function to simply detect staged / unstaged, and create the prompt status from that, rather than detecting a dirty local in that function and creating the prompt based on that. Yes, I agree that the method I implemented is not ideal, either. The other problem, as you mentioned, is that this is only supported for git, Hg, and bzr. What do we do about SVN or CVS?
That said, the more I think about it, the more I like your original solution to this more than the one I provided. If we can't come up with something else, I'll probably just merge this original pull request.

Okay. I found a slightly better solution. Basically it is the same idea, but it uses a vcs_info-hook to check if one of ${hook_com[staged]} or ${hook_com[unstaged]} is empty. If not, the variable VCS_WORKDIR_DIRTY is set to true..

And regarding the CVS/SVN (or: all which does not support staged/unstaged): I think it is possible to implement such a feature for them, but it might not be a good idea as we need to make an HTTP-call to query the server...
But maybe it is worth the thing (at least for SVN, CVS is IMHO pretty dead). A quick search found this (well in japanese, but google-translate for the rescue): http://www.techscore.com/blog/2013/08/02/zshのvcs_infoのhookを利用してsvn-stを表示する/
For now, at least the theme won't be broken. The just see always a green background..

Awesome! I like this method much better. Merged into master =)
Thanks for your awesome improvements, @dritter!

You are welcome :)
I just love this theme. Thanks for creating it!


ZD-4632051: Add West Berkshire custom branding

See https://govuk.zendesk.com/agent/tickets/4632051
Question for reviewers - do I add the versioning commit to this PR, or raise a fresh on once it's merged?

Thanks @sfount - I've added a commit with the version bump.


[Question]: ReAct agent, not passing action input correctly to tool, but hallucinating at first step

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I created 2 different query engines, DB query engine and generic LLM query engine
# Define DB query engine

db_query_engine = SQLTableRetrieverQueryEngine(
    sql_database,
    obj_index.as_retriever(similarity_top_k=3),
    service_context=service_context,
    text_to_sql_prompt=text_to_sql_prompt,
)

# Define LLM query engine

qa_prompt = PromptTemplate(
    "Context information is below.\n"
    "---------------------\n"
    "{context_str}\n"
    "---------------------\n"
    "Given the context information and not prior knowledge, "
    "answer the query.\n"
    "Query: {query_str}\n"
    "Answer: "
)

class LLMStringQueryEngine(CustomQueryEngine):
    llm: OpenAILike
    qa_prompt: PromptTemplate

    def custom_query(self, query_str: str):
        response = self.llm.complete(query_str)

        return str(response)

llm_query_engine = LLMStringQueryEngine(
    llm=llm,
    qa_prompt=qa_prompt,
)

I created tools based on these query engines:
query_engine_tools = [
    QueryEngineTool(
        query_engine=db_query_engine,
        metadata=ToolMetadata(
            name="db_query",
            description=(
                "Provides information about entries from database"
                "Use a detailed plain text question as input to the tool."
            ),
        ),
    ),
    QueryEngineTool(
        query_engine=llm_query_engine,
        metadata=ToolMetadata(
            name="llm_query",
            description=(
                "Provides general information based on model knowledge"
                "Use a detailed plain text question as input to the tool."
            ),
        ),
    ),
]

I created agent based on these tools:
agent = ReActAgent.from_tools(query_engine_tools, llm=llm, verbose=True)
When I am querying each query engine directly I am getting correct response,
But when I am asking agent to provide answer it looks like agent is hallucinating on agent decision,  instead of just passing the initial question to agent it is trying to construct SQL query as action input and is inventing table and column names
Strangely it was working for brief time, but then started this hallucinations.
I tried using agent.reset() as I thought that agent history can be issue, as sometime it started answering without going to database.
I am using Mixtral 7x8B as LLM accessed throught API (OpenAILike LLM definition). Could it be the case that it is Mixtral keeping it somehow in the history, is there any context reset that could be sent to LLM?
here is some generic question
response = agent.chat('use db_query to show me entry with latest ACTIVE status, display name and status date')
print(response.response)

and verbose log:
Thought: I need to use the db_query tool to retrieve the information about the entries  with the latest ACTIVE status.
Action: db_query
Action Input: {'input': "SELECT entry_name, entry_date FROM Entries WHERE entry_status = 'ACTIVE' ORDER BY entry_date DESC LIMIT 1;"}

Trace: chat
|_CBEventType.AGENT_STEP ->  59.037927 seconds
|_CBEventType.LLM ->  2.264976 seconds
|_CBEventType.FUNCTION_CALL ->  56.771783 seconds
|_CBEventType.RETRIEVE ->  0.098935 seconds
|_CBEventType.EMBEDDING ->  0.098451 seconds
|_CBEventType.TEMPLATING ->  8e-06 seconds
|_CBEventType.LLM ->  1.885245 seconds
|_CBEventType.SYNTHESIZE ->  54.518435 seconds
|_CBEventType.TEMPLATING ->  2.6e-05 seconds
|_CBEventType.LLM ->  2.995604 seconds
|_CBEventType.TEMPLATING ->  2.4e-05 seconds
|_CBEventType.LLM ->  1.347868 seconds
|_CBEventType.TEMPLATING ->  3.2e-05 seconds
|_CBEventType.LLM ->  1.226248 seconds
|_CBEventType.TEMPLATING ->  1.9e-05 seconds
|_CBEventType.LLM ->  2.022342 seconds
|_CBEventType.TEMPLATING ->  2.7e-05 seconds
|_CBEventType.LLM ->  1.039552 seconds
|_CBEventType.TEMPLATING ->  1.9e-05 seconds
|_CBEventType.LLM ->  0.996389 seconds
...
|_CBEventType.LLM ->  3.137375 seconds
|_CBEventType.TEMPLATING ->  2.3e-05 seconds
|_CBEventType.LLM ->  0.0 seconds

I have discovered that it looks like issue can be agent definition, and initial query parsing from agent side.
when requesting from agent to process complex query like:
response = agent.chat('list 10 persons from personstatus table with most recent ACTIVE status and MASTER rank, display person name, status date,  company, birthdate and full address')
agent converts it to following:
Thought: To answer this question, I need to use the 'db_tool' tool to retrieve the required information from the 'personstatus' table.
Action: db_tool
Action Input: {'title': 'Query', 'description': 'The query to execute on the personstatus table', 'type': 'string'}

this means that whole query is lost and not transferred to tool, can you somehow specify in agent definition that for this specific tool original query could be passed?
How do you display / modify agent prompt?

I think you are slightly misunderstanding how agents work
They get a user message, look at a list of tools/descriptions, and decide whether to call a tool or respond directly. As part of this process, the LLM rewrites the input to the tool, following the schema of that tool.
If you want it to pass in the original user message, you kind of have to tell it to do that, either in a system prompt or tool description.
Also, Open-Source LLMs are (in my experience) not great at being agents right now (hence the hallucinating you described originally)

hi @logan-markewich,
as I am handing data stored in DB and some of it is considered sensitive privately hosted open source model is the only viable solution. I am using Mixtral 7x8b and I am pleasantly surprised how usable it is.
I managed to find the solution,
I am actually wrapping SQLTableRetrieverQueryEngine in CustomQueryEngine and creating DBStringQueryEngine(CustomQueryEngine). Then I am found out that when defining the tool you can instruct it I"nput is human language" and also tweeking is needed in text_to_sql_prompt for SQLTableRetrieverQueryEngine
query_engine_tools = [
QueryEngineTool(
query_engine=db_query_engine,
metadata=ToolMetadata(
name="fromRDBMS",
description=(
"Provides information about people from database. Input is human language"
"Always ignore previous conversation history and fetch data from database unless told differently."
"Use a detailed plain text question as input to the tool."
),
),
),
QueryEngineTool(
query_engine=llm_query_engine,
metadata=ToolMetadata(
name="fromModel",
description=(
"Provides general information based on model knowledge"
"Use a detailed plain text question as input to the tool."
),
),
),
]
This way I managed to get input pretty much only as instruction text, despite the fact that sometimes it still suggest something that looks very similar to SQL query.
One thing that I found out is that agent keeps conversation and activity history. Despite this being usefull, sometimes this creates disastrous results, If there is for example specific person mentioned previously and any data relevant for him was already fetched, with next question system can hallucinate and not fetch data from database but present this false date straight up.
I know that I can reset get and agent.memory, but is there any other option to control QueryEngineTool itself.
I can see that question is routed to correct QueryEngineTool, but this QueryEngineTool then  despite being defined as SQLTableRetrieverQueryEngine does not fetch from DB but from some memory.
How can I purge this?

Hi @dinonovak ,
Mine also hallucinates quite a bit, but I was able to alleviate the problem by overriding the system prompt.
    # Override the default system prompt for ReAct chats.
    with open("system_prompt.md") as f:
        MY_SYSTEM_PROMPT = f.read()

    class MyReActChatFormatter(ReActChatFormatter):
        system_header = MY_SYSTEM_PROMPT

    chat_formatter = MyReActChatFormatter()
    return ReActAgent.from_tools(
        tools=all_tools,
        llm=local_llm,
        verbose=True,
        react_chat_formatter=chat_formatter,
    )

where system_prompt.md is my spin on the default prompt, with emphasis on tool usage.
Try if this helps. Good luck!


UIViewController+Rx.swift:50:57: error: type 'UIViewController' has no member 'viewIsAppearing'

After update start catch subj error while build time.
Maybe we something missing?
p.s. ios14+

Hi @bimawa Thanks for the feedback. Unfortunately I cannot reproduce this error on my side. Do you have more details? Are you using viewIsAppearing in your code?

No, we don't use this method. I just resolved SPM deps. And can't compile with this version anymore.
Currently change on exact: 1.7.1 - this compiled perfect.
I can try to reproduce it later, currently busy.

Please also let me know if there are any updates on this, since I am the one who is works on the PR of viewIsAppearing.

I am also stuck in this issue please any one can help, I am blocked due to this in ios 14, previously I have solved by installing carthage but again don't know it's showing error

previously I have solved by installing carthage

Hey can you provide a sample app with that issue? For Tonny. I'm still busy ...

yes, I am working on one project in xcode 14.3, but I can also provide one demo app I have also downloaded, in that app also showing error, https://github.com/pixeldock/RxAppState?tab=readme-o

v-file, in that same app showing this error

I think this method not available in older sdk.
Maxim Bunkov - Chat @ Spike	[2ixb1q]
On March 26, 2024 at 6:36 GMT, piyushdscg @.***> wrote:
yes, I am working on one project in xcode 14.3, but I can also provide one demo app I have also downloaded, in that app also showing error, https://github.com/pixeldock/RxAppState?tab=readme-o
Uploading RxappstateIssue.png…
v-file, in that same app showing this error
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@bimawa @piyushdscg Does this error still occur in RxAppState 1.8.0?

@bimawa @piyushdscg @tommyming
Hi guys, I think I found the reason for this issue. Apparently you need Xcode 15 to use viewIsAppearing. The reason for this is that while viewIsAppearing was already added in iOS 13 it was only made public in iOS 17. (Checkout this article for a more detailed explanation)
The easiest solution for this issue would be to use Xcode 15. If for some reason you cannot do that, I added the fix suggested in the article to a separate branch fix/xcode14-viewIsAppearing. I could not test this fix, because I cannot run Xcode 14 on my machine.
Could you please test if this fixes the issue for you?

@pixeldock Thanks for checking out this issue.
I agree with your investigation that the method is only public since iOS 17, which is available starting from iOS 15.
I haven't tested on it below Xcode 15 when I create the PR, since it is not available on my Mac anymore.

I think we may close this issue. I think no reason to fix it. All my users will move on Xcode 15 very soon too.
Maxim Bunkov - Chat @ Spike	[2iz1ud]
On March 27, 2024 at 0:49 GMT, Tommy Han @.***> wrote:
@.***(https://github.com/pixeldock) Thanks for checking out this issue.
I agree with your investigation that the method is only public since iOS 17, which is available starting from iOS 15.
I haven't tested on it below Xcode 15 when I create the PR, since it is not available on my Mac anymore.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Thanks for support, you can close this, I am compelled due to some reason, So can not move to xcode 15, last time I fixed this issue in xcode 14.3 by installing some libraries like carthage etc., but don't know in just 3-4 days this issue again triggered without doing any thing.
Thanks again for the support and for quick response.

I found the problem again.
It was built in version 15.2 of XCode
'viewIsAppearing' is only available in iOS 13.0 or newer
The error occurs in line 47 of the UIView Controller+Rx.Swift.
The error I think is in Package.swift
platforms: [
.iOS(.v10)
],
I said that, so I need to modify it to iOS 13
In the code above, I think I need to make an exception that I have to do it on iOS13 or later.

Hi @greatsk55 thanks for your comment! Have you found ".iOS(.v10)" in the 1.8.0 release of RxAppState? Or are you talking about an older version?

I found it in the master branch.
Probably 1.8.0.

Thanks @greatsk55 for your fix! I'll release a 1.8.1 patch release.


Add Laravel 5.7 support

for when could the compatibility of shinobi with laravel 5.7 be available? Thanks!!

I created PR #83 yesterday to add 5.7+ support, so we just have to wait until it gets accepted and merged in!


[google_maps_flutter]Marker drag event

Description
Flutter bridge to OnMarkerDragListener
Related Issues
https://github.com/flutter/flutter/issues/28929

signed it
On Tue, Jun 4, 2019 at 6:18 PM googlebot<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project (if not, look below for help).
Before we can look at your pull request, you'll need to sign a Contributor
License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I
signed it!) and we'll verify it.
What to do if you already signed the CLA Individual signers

It's possible we don't have your GitHub username or you're using a
different email address on your commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.

Corporate signers

Your company has a Point of Contact who decides which employees are
authorized to participate. Ask your POC to be added to the group of
authorized contributors. If you don't know who your Point of Contact is,
direct the Google project maintainer to go/cla#troubleshoot (Public
version https://opensource.google.com/docs/cla/#troubleshoot).
The email used to register you as an authorized contributor must be
the email used for the Git commit. Check your existing CLA data
https://cla.developers.google.com/clas and verify that your email is
set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
The email used to register you as an authorized contributor must
also be attached to your GitHub account
https://github.com/settings/emails.

ℹ Googlers: Go here
https://goto.google.com/prinfo/https%3A%2F%2Fgithub.com%2Fflutter%2Fplugins%2Fpull%2F1702
for more info.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter/plugins/pull/1702?email_source=notifications&email_token=AALJYT2NODAJ6UTFS2QUGBLPY2BTNA5CNFSM4HS5NZP2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODW45B7Q#issuecomment-498716926,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALJYT7TDP3QZY2MBLD6XGDPY2BTNANCNFSM4HS5NZPQ
.

IPA build check is seemingly broken. All recent PR's fail to pass it.

are there any news?

It looks like all the requests are done but its still showing up as one missing. @duzenko Any idea on how to fix that?

It looks like all the requests are done but its still showing up as one missing. @duzenko Any idea on how to fix that?

@iskakaushik ?

@iskakaushik this should be good to go, right?

Yup. This looks good to me. @duzenko can you please update the pubspec and changelog?

Done

sorry for the trouble, but could you please fix the conflicts, i will keep and eye out and make sure i merge this PR in first :-)

sorry for the trouble, but could you please fix the conflicts, i will keep and eye out and make sure i merge this PR in first :-)

@iskakaushik
No problem
But I'm traveling this and the next weeks and AFK most of the time. I'll try to find some time for this.

Fixed

The callback receives the new position after drag event in the marker object?

The callback receives the new position after drag event in the marker object?

That was the idea


Extend @webstone/cli to wrap svelte-add

The svelte-add project (repo link) is a convenient way to configure various things in SvelteKit apps, e.g. Tailwind CSS, mdsvex, etc.
The Webstone CLI should provide a command which acts as a thin wrapper around svelte-add. For example, webstone web configure tailwind is a wrapper for npx svelte-add@latest tailwindcss (docs).
Update: Since this is the first CLI command, let's also set us up for success as we add more commands. See best practices docs.

The Webstone CLI wrapper command is webstone web svelte-add <integration>. This makes it clear we delegate to svelte-add and gives us an opportunity to use webstone web configure for other, Webstone-specific configuration tasks.


Vera state tracking broken since ~0.85

Home Assistant release with the issue:

Last working Home Assistant release (if known):
0.84
Operating environment (Hass.io/Docker/Windows/etc.):
Hass.io
Component/platform:
Vera
https://www.home-assistant.io/components/vera/
Description of problem:
my Hass.io had been working great as the front end to my Vera Plus (managing Zwave devices) but from about v0.85 hassio no longer updates device states from my Vera Plus.
I can trigger things on but then the slider switch goes right back to the off position and never updates - which also means I cant turn them off. If something is turned on physically it also never updates in hassio (everything is working fine in Vera).
I’m running 100% lovelace and have been since around v0.74 without issues till 2019.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
vera:
  vera_controller_url: http://<Vera_IP>:3480/
  lights: [83, 56, 117, 154]

Traceback (if applicable):

Additional information:
From the logs:
- dictionary {'name': 'Lounge Lamp', 'altid': '2', 'id': 117, 'category': 3, 'subcategory': 1, 'room': 3, 'parent': 1, 'configured': '1', 'commFailure': '0', 'kwh': '0.3130', 'watts': '0.000', 'light': '0.000', 'categoryName': 'On/Off Switch'}
10:11 AM components/vera/light.py (ERROR)
Could not set Status for Lounge Lamp (key does not exist).
10:11 AM components/vera/light.py (ERROR)

I am running the latest version and it works for a day or so and stops updating.

I'm also seeing the same issues.

Hopefully it can be fixed relatively soon. I use the Vera for a few things, one of which is a scene switch that doesn't appear to be supported by OpenZwave, and therefore home assistant.

I think there is better tracking here:
https://github.com/home-assistant/home-assistant/issues/24987

I've given up and ditched HA altogether - luckily the latest version of Vera is stable now and I can live without all the neat stuff in HA now that the core Vera platform is stable.

While Vera may be somewhat stable, it's a decade behind so many other
platforms. Without PLEG (or possibly reactor, I've never used it), the Vera
platform would be nearly useless. I have automations that would be
impossible to do with Vera in one step. Going back would not be possible
for me. Many use the Vera for the hardware interface and HA for the rest.
Probably the best use of Vera....
On Mon, Jul 29, 2019, 5:52 PM dJOS1475<EMAIL_ADDRESS>wrote:

I've given up and ditched HA altogether - luckily the latest version of
Vera is stable now and I can live without all the neat stuff in HA now that
the core Vera platform is stable.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/23415?email_source=notifications&email_token=ABGBF5XLFDWKBHIQ3HULF33QB5RBTA5CNFSM4HIRPA72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3CDUCY#issuecomment-516176395,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABGBF5SV2REBQB5P7ZSLRYDQB5RBTANCNFSM4HIRPA7Q
.

I believe this issue has been resolved, but I'm not totally certain. I have almost completely moved away from Vera being a part of my HA setup...


Add adminstration UI

Over time, we will have multiple tools for administrators or moderators to change content on the site. We prooobably want a dedicated area that contains all those tools. Quick idea would be to add a link in the current main menu (if the user has the corresponding rights) called "Administration" and linking to... /admin maybe. There, it lists links to all tools we have, like "Change configuration" (global config), "Server status" and "Change realm structure".
But yeah, we need to think about this.

This was started in #242
It will surely still evolve in the future, but I don't think keeping this issue does not serve any purpose for now.


Add planning and tender document counts to release_summary

The release_summary table includes total_award_documents and total_contract_documents aggregates, but is missing similar aggregates for planning, tender and implementation documents.

 kingfisher-views: Add new aggregates/fields

I see total_contract_implementation_documents so I assume this just leaves planning and tender.

Here's a diff from a branch that was not made into a PR and which now conflicts. Putting @pindec's commit here so it can be taken forward from master.
diff --git a/sql/008-release.sql b/sql/008-release.sql
index 4a32e7a..1e4737c 100644
--- a/sql/008-release.sql
+++ b/sql/008-release.sql
@@ -29,6 +29,49 @@ create unique index tmp_release_party_aggregates_id on tmp_release_party_aggrega

 ----

+--- Adding planning document counts by document_type
+
+drop table if exists tmp_planning_documents_aggregates;
+create table tmp_planning_documents_aggregates
+AS
+select
+    id,
+    jsonb_object_agg(coalesce(documentType, ''), documentType_count) planning_documentType_counts
+from
+    (select
+        id, documentType, count(*) documentType_count
+    from
+        planning_documents_summary
+    group by
+        id, documentType
+    ) AS d
+group by id;
+
+create unique index tmp_planning_documents_aggregates_id on tmp_planning_documents_aggregates(id);
+-- end of adding planning document counts
+
+--- Adding tender document counts by document_type
+
+drop table if exists tmp_tender_documents_aggregates;
+create table tmp_tender_documents_aggregates
+AS
+select
+    id,
+    jsonb_object_agg(coalesce(documentType, ''), documentType_count) tender_documentType_counts
+from
+    (select
+        id, documentType, count(*) documentType_count
+    from
+        tender_documents_summary
+    group by
+        id, documentType
+    ) AS d
+group by id;
+
+create unique index tmp_tender_documents_aggregates_id on tmp_tender_documents_aggregates(id);
+-- end of adding tender document counts
+
+
 drop table if exists tmp_release_awards_aggregates;

 create table tmp_release_awards_aggregates
@@ -261,6 +302,18 @@ from
 left join
     tmp_release_party_aggregates
 using(id)
+left join
+    (select id, documents_count AS total_planning_documents from planning_summary) AS planning_summary
+using(id)
+left join
+    tmp_planning_documents_aggregates
+using(id)
+left join
+    (select id, documents_count AS total_tender_documents from tender_summary) AS tender_summary
+using(id)
+left join
+    tmp_tender_documents_aggregates
+using(id)
 left join
     tmp_release_awards_aggregates
 using(id)
@@ -298,6 +351,8 @@ drop table if exists tmp_release_party_aggregates;
 drop table if exists tmp_release_awards_aggregates;
 drop table if exists tmp_release_award_suppliers_aggregates;
 drop table if exists tmp_award_documents_aggregates;
+drop table if exists tmp_tender_documents_aggregates;
+drop table if exists tmp_planning_documents_aggregates;
 drop table if exists tmp_release_contracts_aggregates;
 drop table if exists tmp_contract_documents_aggregates;
 drop table if exists tmp_contract_implementation_documents_aggregates;

 Finish up and merge work on Views that's been started and responds to analyst requests


[1.5.2] Bow & Fishing Rod model issue

Bow:
Arrow missing despite texture being correct
Pulling stages aren't shown correctly.
This is stage 2 (final stage)

Vanilla for comparison

Item sprite on bar doesnt update as it should

Vanilla for comparison

Fishing Rod:
When cast it still uses the standard model

Fairly confident my models are correct :)
Attached zip with model json and textures
files.zip

nvm sorted. missed client renderer..


'Something went wrong' message: custom title for FirstPageErrorIndicator and NewPageErrorIndicator

Would be great to pass title for FirstPageErrorIndicator and NewPageErrorIndicator

The API's provide firstPageErrorIndicatorBuilder on which you can display a widgets of your choice.


Get rid of eaa-kbc & verdictd CI test

Oh, it has been done in https://github.com/confidential-containers/enclave-cc/pull/165/files . Let me close this

My preference is we drop KBC=eaa_kbc completely since we are not validating it and I was thinking this issue covers all the work. I'm OK to submit a new issue about it too

Sounds reasonable to me. Let me reopen this issue and assign this to me.


allow stub templates

I have seen in other migration cli's the use of stubs which are really nice.
I am thinking a command like
npx kysely migrate:stub {stub_name} {migration_name}
Would need to add a stubsFolder option under migrations option in the config file
Then I could have a folder called whatever I want with file names like: my_stub_file.stub.  This file would be just like your template file in your src->templates folder.
When the migration is created it will use the stub file as the template, name the file as it does now and use the correct extension.

Hey 👋
Thank you for raising this! 🙏
Looking at https://github.com/knex/knex/pull/3316, knex's stub feature works a bit differently.
A --stub flag on migrate:make and seed:make that can accept a filename (looks for it in the migrations/seeds folder) or a path relative to the config file.
This is a welcome enhancement, and gets us closer to 100% knex CLI coverage.


Split change bundles with excessive asset counts

Issue Number
ADP-727
Overview
This PR has two main areas:
Refactoring

[x] Relocates all existing type-specific equipartition functions into the modules that define their types. We now have:

Module
Function

Numeric.Util
equipartitionNatural

Coin
equipartition

TokenQuantity
equipartition

TokenMap
equipartitionQuantities

TokenMap
equipartitionQuantitiesWithUpperBound

TokenBundle
equipartitionQuantitiesWithUpperBound

Justification:

This PR will add more functions for splitting up bundles and maps, and the existing naming scheme was becoming too cumbersome to extend.
Equipartitioning a value of type T is something that relates to the definition of type T, so the natural home for such a function is within module T.

[x] Adjusts makeChange to accept a record type, and unifies that type with MakeChangeData from the test suite.
Justification:

Adding further arguments to this function is now less cumbersome, since the compiler will tell us that a field is missing in all the required call sites (both in implementation code and in test code), rather than producing a series of long type errors.
We cut down on some boilerplate from the test suite.

New Functionality

[x] Adds the following functions for equipartitioning the asset sets of maps and bundles:

Module
Function

TokenMap
equipartitionAssets

TokenBundle
equipartitionAssets

[x] Adds an assessBundleSize parameter to performSelection and makeChange:
assessBundleSize :: TokenBundle -> TokenBundleSizeAssessment

data TokenBundleSizeAssessment
    = TokenBundleSizeWithinLimit
    -- ^ Indicates that the size of a token bundle does not exceed the maximum
    -- size that can be included in a transaction output.
    | TokenBundleSizeExceedsLimit
    -- ^ Indicates that the size of a token bundle exceeds the maximum size
    -- that can be included in a transaction output.

[x] Adds function splitBundlesWithExcessiveAssetCounts to split change bundles with excessive asset counts, and calls it from makeChange.

[x] Provide a definition of assessBundleSize based on functionality exported by Cardano.Api.

Testing
This PR adds:

[x] Property tests for equipartitionAssets functions (in their relevant modules).
[x] Boundary unit tests for performSelection to show that change bundles are correctly split when they exceed the maximum size.

bors try

bors cancel

bors try

bors cancel

bors try

bors cancel

bors try

bors cancel

bors try

bors try


updating master

Codecov Report

Merging #172 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #172   +/-   ##
=======================================
  Coverage   99.37%   99.37%
=======================================
  Files          43       43
  Lines        1275     1275
  Branches      142      142
=======================================
  Hits         1267     1267
  Misses          8        8

Impacted Files
Coverage Δ

src/proptypes.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Fix CA2000: Dispose objects before losing scope

Enable CA2000 as a suggestion and fix a few rule violations:

src/System.Management.Automation/engine/SessionStateVariableAPIs.cs
src/System.Management.Automation/engine/remoting/common/RunspaceConnectionInfo.cs
src/System.Management.Automation/help/UpdatableHelpSystem.cs

Looking for feedback on whether any further changes like this would be worthwhile.

Tracking PR

What are we tracking?

Tracking PR

What are we tracking?

It was going to track progress on CA2000, but there are a huge number of issues, so I've decided to seek review before making any more changes.

@PowerShell/powershell-maintainers agree that this change is worthwhile and should be taken early in the release cycle to allow time to stabilize any side effects due to resource clean up.


readme: fix typo, availabe -> available

(deleted template as listed in it, since this is not a package submission)
Fixes a typo in README.md I ran across while looking up on how to install local packages.
It might also be useful to add running typos (can also be used as an installed binary without GH actions) to the submission guidelines since there are quite a few typos in the packages, too, on the other hand I'm not sure if you're fine with that, so I left it out.

We can add it to the guidelines

Yeah, thanks!


Still can't get captive portals working :(

Hi, I'm back on this after a long hiatus - Sorry for the delay.
I can't seem to be able to re-open this previous issue so I have created a new one:
https://github.com/belyalov/tinyweb/issues/26
I just can't get tinydns to work - or I just don't understand what I am doing (most likely).
I'm an asyncio noob as well, so that doesn't help at all.
I have all of the captive routes setup, I still have my own tinyweb hack always falls back to '/' if it can't find a route, and no matter what I do, I can't get it to serve a page once I connect to my AP.
The exact same code using MicroDNSSrv works as expected.
I'd gladly keep using MicroDNSSrv, but I still have this outstanding issue:
#25
I've tried setting the DNS to <IP_ADDRESS>:53 and to the IP of the AP, but same issue. I'm thinking maybe the DNS is just not running as I can't see how to make it "run forever" like I can with tinyweb.
I've followed your implementation examples from your int-platform example, but I just can't get it to work :(
So I really don't know where to go from here. I'd love to only be using tinyweb & tinydns for my MP web projects.

I'm moving this to tinydns, as it's clearly not tinyweb related ;)

Hey,
As I remember there was some issue with binding to <IP_ADDRESS> - maybe related WiFi interface readiness (e.g. you need to wait until DHCP done / re-bind once interface configured).
I can try to get repro, within next 2 weeks, but if it is too late - you may want to play around network configuration..
Thanks for feedback! :)


Added MIT License

Added MIT License.

Hello @elishaking, thanks for your contribution

Hello @elishaking, thanks for your contribution

Pleasure 💯


Enable to configure global remote option and it to affects sortable helper

Hello.
I happy to use SmartListing on my company projects.
So I want to configure global remote option and it to affect sortable helper.

Merged. Thanks @kitabatake


Running CSD using METRA codebase

Hi Seohong,
As you mentioned in comment of CSD repo, METRA repo contains CSD implementation. Could you please indicate how can we run CSD using the METRA codebase?
Thanks for your help and time!
Best,
Zhao

Hi Zhao, you can use --dual_dist s2_from_s --dual_lam 3000 --dual_slack 1e-06 to run CSD.


New Site Errors - bad fields [Description,IconLink] in [ChildPage List, Text Box]

addonInstallClass.UpgradeAppFromLocalCollection_InstallAddonNode_Phase1, cause=[n/a], ex=[System.Exception: bad field found [Description], in addon node [ChildPage List], of collection [Child Page List]]
addonInstallClass.UpgradeAppFromLocalCollection_InstallAddonNode_Phase1, cause=[n/a], ex=[System.Exception: bad field found [IconLink], in addon node [Child Page List], of collection [Child Page List]]
addonInstallClass.UpgradeAppFromLocalCollection_InstallAddonNode_Phase1, cause=[n/a], ex=[System.Exception: bad field found [Description], in addon node [Text Box], of collection [Text Box]]
addonInstallClass.UpgradeAppFromLocalCollection_InstallAddonNode_Phase1, cause=[n/a], ex=[System.Exception: bad field found [IconLink], in addon node [Text Box], of collection [Text Box]]
addonInstallClass.UpgradeAppFromLocalCollection_InstallAddonNode_Phase1, cause=[n/a], ex=[System.Exception: bad field found [Description], in addon node [Content Box], of collection [Content Box]]
addonInstallClass.UpgradeAppFromLocalCollection_InstallAddonNode_Phase1, cause=[n/a], ex=[System.Exception: bad field found [IconLink], in addon node [Content Box], of collection [Content Box]]

fixed in the addons


Add option to toggle file extension HTTP Post

Add the option to remove the file extension(png) from the url copied to the clipboard.

Thanks!

Feel free to reword anything you need :P