Template binary sensor attributes

Description:
Added option to add custom attributes to the template binary sensor via a new attribute_templates config option. This is an updated version of PR #20410 concentrating on just the binary_sensor.
Related issue (if applicable): fixes #
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#9106
Example entry for configuration.yaml (if applicable):
binary_sensor:
  - platform: template
    sensors:
      test_sensor:
        value_template: >-
          {{ states.device_tracker.my_phone.state == "home" }}
        attribute_templates:
          latitude: >-
            {{ states.device_tracker.my_phone.attributes['latitude'] }}
          longitude: >-
            {{ states.device_tracker.my_phone.attributes['longitude'] }}

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

@rohankapoorcom Here is the new PR, concentrating on the binary_sensor.

I'll review this tonight.

Attributes are for the frontend. I.e. the GPS is on device tracker/person Entity defined. This feature ends up in some hacks between different layers. Otherside, some component implemented also a dynamic attribute handling.

There are other components which already allow attributes to be created, such as the RESTful sensor. Would a different example be better, one which doesn't just directly map attributes from other entities? Here is an actual example that I have used (with this code running as a custom component) which detects if my keys are lost (location more than 100 metres way) and has an attribute for the distance they are away in metres:
lost_keys:
  friendly_name: Keys Lost
  value_template: >-
    {{ (distance(states.device_tracker.keys, states.device_tracker.phone) * 1000) > 100 }}
  attribute_templates:
    distance: >-
      {{ distance(states.device_tracker.keys, states.device_tracker.phone) * 1000 }}
  entity_id:
    - sensor.keys
    - device_tracker.phone

This is good IMHO. I was planning something similar, but putting it in a base class for all the template and rest components so they all work the same. I have got that work ready to go, it's just waiting on getting some other PRs merged though, so don't roll this out to all the others yet.

Linking to PRs waiting to merge: #23590, #23293, Issue: #22587

Arch issue: home-assistant/architecture#220

What is the status and plan for this PR?

I like this approach.

Nice. Been wanting something like this.

Awesome  !  🐬

@balloob I also have a branch for attributes on the template sensor (I originally had both in the same branch but split them apart following earlier feedback).
I can submit a PR for this once I have checked that this includes all the changes from your feedback on this one.

@gadgetchnnel Hi, do you plan to create similar PRs for switch templates and light templates?  If not, I might do that myself.
Thanks in advance!


Bug: on new specify views command

When using the new specify views command I  ran the following:
php artisan infyom:api_scaffold Product --fieldsFile='resources/model_schemas/Product.json' --views='index,create,edit' --skip='migration'
It worked great but when I  went to view the outcome there was an error. it did generate the table.blade.php not the fields.blade.php file.

@phillipmadsen fixed it.


CD-18 Expose aws box url

@vasili-zolotov @d3ming
This creates a link to the aws box that's created during ci. Right now it uses the users github icon as the button, but this can be changed later with a real icon. (I didn't want to take a stab at art)

Super old.


Improve standardize_dataframe documentation

The docstring could be improved by giving some indication of how the col_mapper object should be constructed

@andim happy to update the docs! But was unsure what exactly you had in mind- the docs already say that it should be a mapping object like a dictionary?

I just chatted to Sankalan about this issue and he said that maybe it would be good to add examples here. I will make time to do that.


Example in README uses 'paths', not 'path'

https://github.com/Madrapps/jacoco-report/blob/7a334255fbce42f385d7567c25d986a9c62e2971/README.md?plain=1#L51-L55
This produces an error
Warning: Unexpected input(s) 'paths', valid inputs are ['path', 'token', 'min-coverage-overall', 'min-coverage-changed-files']

Which version of the action are you using? path was replaced with paths in v1.2. Could you try using the action with v1.2 ?

I was using v1 (no minor version) so this probably used v1.1 so the problem is probably fixed now :)
      - name: Add coverage to PR
        uses: madrapps/jacoco-report@v1
        if: ${{ always() }}
        continue-on-error: true
        with:
          path: ${{ github.workspace }}/target/site/jacoco/jacoco.xml
          token: ${{ secrets.GITHUB_TOKEN }}


feat: make text support italic and underline

Summary
Type

Feature

Summarise concisely:
What is expected?
Make text support italic and underline
The following changes where made:

add optional italic and underline props to text

Why PhoneInput visual has been updated ?

I'm trying to understand as well, I'm on it


Fix wrong parameter was passed on debug_backtrace()

https://www.php.net/manual/en/function.debug-backtrace.php
This 0 is undefined.
define('DEBUG_BACKTRACE_PROVIDE_OBJECT', 1);
define('DEBUG_BACKTRACE_IGNORE_ARGS', 2);

Do you mind explaining what are you trying to achieve with this change? Passing 0 should disable DEBUG_BACKTRACE_PROVIDE_OBJECT, which is the default value for the parameter

The corresponding value of this constant is 1 , so I think I should change it to DEBUG_BACKTRACE_PROVIDE_OBJECT ?

As explained above, we're passing 0 to avoid PHP to fill the object in the backtrace. What use-case do you have for needing such piece of info?

As explained above, we're passing 0 to avoid PHP to fill the object in the backtrace. What use-case do you have for needing such piece of info?

I looked at the definition of PHP built-in constant, and this DEBUG_BACKTRACE_PROVIDE_OBJECT actually has a value of 1 .
define('DEBUG_BACKTRACE_PROVIDE_OBJECT', 1);

The $options parameter is a bitmask, so you can combine the various options which are DEBUG_BACKTRACE_PROVIDE_OBJECT and DEBUG_BACKTRACE_IGNORE_ARGS. If you don't want to use any of them, you have to pass 0 which effectively disables all those options

As explained above, we're passing 0 to avoid PHP to fill the object in the backtrace. What use-case do you have for needing such piece of info?

https://github.com/php/php-src/blob/b698108133f5403bb440bc94dbebe92ff487e667/Zend/zend.h#L412

I'm sorry, I understand your intention.


Taking Raw Data

Hi i'm taking queries using java driver and need take the raw statementResult, how i can get that? Thanks you. I will put a piece code like example.
try (Connection con = DriverManager.getConnection(urlOfServer, user, password)) {
  try (PreparedStatement stmt = con.prepareStatement("MATCH (n) RETURN n")) {
     try (ResultSet rs = stmt.executeQuery()) {
        //Take here raw data
        return rs;
     }
  }
}

I expect something like this:
[
  {
    "keys": [
      "n"
    ],
    "length": 1,
    "_fields": [
      {
        "identity": {
          "low": 23,
          "high": 0
        },
        "labels": [
          "Node"
        ],
        "properties": {
          "name": "EG - DIRECCION DE CENTRAL COMPRAS BAZAR",
          "identificator": "MODU426"
        }
      }
    ],
    "_fieldLookup": {
      "n": 0
    }
  },
  {
    "keys": [
      "n"
    ],
    "length": 1,
    "_fields": [
      {
        "identity": {
          "low": 24,
          "high": 0
        },
        "labels": [
          "Node"
        ],
        "properties": {
          "name": "CIRCUITOS RCM",
          "identificator": "APLI238"
        }
      }
    ],
    "_fieldLookup": {
      "n": 0
    }
  }
]

Why?
And why don't you the just use the java driver
Please note that there is no json in the java driver at most you get a list of maps

Because i need neo4j format for paint a Graph using something like this framework [https://github.com/eisman/neo4jd3]https://github.com/eisman/neo4jd3), and i can't attack database from client, first need pass by business layer.

Well, this driver is just a pure implementation of the JDBC interface. It doesn’t provide any additional conversion.
Maybe you could rely on some APOC and call them via this driver, or implement you own one, if you can do in altre middle tier

I'm closing this since this is not a real issue.


chore: bdd-nh-import-1587850315 to 0.0.1

chore: Promote bdd-nh-import-1587850315 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Add Creating Docker Images guide

This guide has no associated code -- it uses the Quickstart example, so I've put the adoc file directly in  src/main/docs/guides/

I wonder if using jib would create smaller images 🤔


Reset ast builder state (dotnet)

I found some time to do the changes today :-)
I added the necessary constructors and methods to Gherkin/Parser.Extensions.cs. Is that the right place? If they should go into the generated code (Parser<T>): how should I do that?

@aslakhellesoy @brasmusson I think I addressed all your concerns. Could you have another look please?

Great work @dirkrombauts !

:-)


[Trade] Trading View refresh on each subpage change

On the trade page, when we switch to a new subpage (form or explorer), the trading view iframe refresh.
https://github.com/user-attachments/assets/573f9c88-ff42-49ab-bba2-6d77f0e61a8c

doesnt happen as often as the design has changed.
still happens when you change strategy type
https://github.com/user-attachments/assets/3dbe9464-2cff-4c97-826c-6c81874df380


Install pytest-asyncio

(message created by the CI based on PR content)

Since we're touching the file, I pushed a commit to also remove pytest from azure-identity's dev requirements.

Core async tests are executed now:
examples/test_example_async.py::test_example_trio PASSED                 [  0%]
examples/test_example_async.py::test_example_asyncio PASSED              [  1%]
examples/test_example_async.py::test_example_aiohttp PASSED              [  1%]


Whisper message notification sent to other members when a new member joins

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

Version Number: 1.4.74-4
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @quinthar
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1715964357862879
Action Performed:
Prerequisite: Set room push notifications to immediate

User A creates a public room in a workspace chat
User B (iOS) joins the room
User C joins the room

Expected Result:
No whisper notification for user A and B
Actual Result:
Push notification received for user A and B
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[X] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01b0be0badd692dcec
Upwork Job ID:<PHONE_NUMBER>894093312
Last Price Increase: 2024-05-20

I have some changes prepped but they're held on a refactor for https://github.com/Expensify/Expensify/issues/381189 and also the merge freeze

I think the merge freeze is over right, @arosiclair ?

Doesn't look like it to me 😕

You're right, sorry that was wrong from me! @arosiclair

Hey there! I am going OOO until the 9th of June. Please can you handle this and I will take it back once I return? Thanks!

Web-E PR is off hold and out for review. Unassigning the C+ since none is needed for this.

This hit prod so we're all done here


Please invite me to the Recode-Hive GitHub Community Organization

Name
Anamitra Joshi
Discord Username (if applicable)
anamitrajoshi
Additional Context
I would like to contribute to this project through the GSSOC'24.

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/Recode-Hive/people


chore(deps): latest

Supersedes https://github.com/hi-ogawa/vite-plugins/pull/177

It looks like Vite Runtime stacktrace changed from 5.1.0 to 5.1.6
https://github.com/hi-ogawa/vite-plugins/actions/runs/8306381095/job/22734378807?pr=196#step:8:50
In this PR, we pin 5.1.0 and investigate the change separately.


"Can't find bash-language-server on your PATH...." on Windows

I've installed the package as advised but the extension fails with this error:

PS C:\Users\fakui> npm i -g bash-language-server
C:\Users\fakui\AppData\Roaming\npm\bash-language-server -> C:\Users\fakui\AppData\Roaming\npm\node_modules\bash-language-server\bin\main.js
+<EMAIL_ADDRESS>updated 1 package in 1.253s
PS C:\Users\fakui> node -v
v10.0.0
PS C:\Users\fakui> (Get-ChildItem Env:Path).Value
C:\Python27\;C:\Python27\Scripts;C:\Program Files (x86)\Common Files\Oracle\Java\javapath;C:\Program Files (x86)\NVIDIA Corporation\PhysX\Common;C:\ProgramData\Oracle\Java\javapath;C:\Program Files\Python36\Scripts\;C:\Program Files\Python36\;C:\WINDOWS\system32;C:\WINDOWS;C:\WINDOWS\System32\Wbem;C:\WINDOWS\System32\WindowsPowerShell\v1.0\;C:\Program Files\Microsoft SQL Server\120\Tools\Binn\;C:\Program Files\Common Files\Autodesk Shared\;D:\Program_Files\Strawberry\c\bin;D:\Program_Files\Strawberry\perl\site\bin;D:\Program_Files\Strawberry\perl\bin;C:\ProgramData\chocolatey\bin;C:\Program Files\Git\cmd;C:\Program Files\dotnet\;C:\Program Files\nodejs\;C:\Users\fakui\AppData\Local\Microsoft\WindowsApps;;C:\Program Files (x86)\Microsoft VS Code\bin;C:\Users\fakui\AppData\Roaming\npm

Thanks for reporting this. It might be related to https://github.com/mads-hartmann/bash-language-server/pull/40

@fakuivan Thanks for reporting this. I've released version 1.3.1 which includes #40. Please upgrade the bash language server (npm i -g bash-language-server) and re-open if this is still an issue :)

Nope, still having the same issue.
PS C:\Windows\system32> npm i -g bash-language-server
C:\Users\fakui\AppData\Roaming\npm\bash-language-server -> C:\Users\fakui\AppData\Roaming\npm\node_modules\bash-language-server\bin\main.js
+<EMAIL_ADDRESS>updated 1 package in 5.686s

@fakuivan Damn, alright, as I don't have access to windows machines at the moment I'd love your help debugging this issue. Head over to #5 to see what other users have tried.

@mads-hartmann As far as I understand it, finding the path to the executable is not the problem:
PS C:\Windows\system32> &"$(Join-Path "$(npm.cmd bin -g)" "bash-language-server.cmd")" -v
Version is 1.3.1

(if that's what vscode-client/src/util.ts does)

I have the same issue:

Linux Mint 18.3 (Ubuntu 16.04)
NodeJS 10.1.0
Bash Language Server 1.3.1

Running it from shell I get this error:

$ bash-language-server
internal/modules/cjs/loader.js:573
throw err;
^
Error: Cannot find module './build/Debug/tree_sitter_runtime_binding'
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:571:15)
at Function.Module._load (internal/modules/cjs/loader.js:497:25)
at Module.require (internal/modules/cjs/loader.js:626:17)
at require (internal/modules/cjs/helpers.js:20:18)
at Object. (/usr/local/lib/node_modules/bash-language-server/node_modules/tree-sitter/index.js:5:13)
at Module._compile (internal/modules/cjs/loader.js:678:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:689:10)
at Module.load (internal/modules/cjs/loader.js:589:32)
at tryModuleLoad (internal/modules/cjs/loader.js:528:12)
at Function.Module._load (internal/modules/cjs/loader.js:520:3)

Just to clarify, this issue is supposed to be open.

I think this is failing to require the Debug build because of this error (or similar) trying to get the Release build:
> require('./build/Release/tree_sitter_runtime_binding')
Error: The module '/usr/lib/node_modules/bash-language-server/node_modules/tree-sitter/build/Release/tree_sitter_runtime_binding.node'
was compiled against a different Node.js version using
NODE_MODULE_VERSION 59. This version of Node.js requires
NODE_MODULE_VERSION 64. Please try re-compiling or re-installing
the module (for instance, using `npm rebuild` or `npm install`).

this is my setup with npm:6.1.0 node:v10.2.1 on ubuntu

I think it's happening here in main.js. Maybe "spawn" isn't smart enough to find the npm package in Windows
    const childProcess = ChildProcess.spawn(command, args, {
      cwd: projectPath
    });

A couple of things to look into:

https://stackoverflow.com/questions/27688804/how-do-i-debug-error-spawn-enoent-on-node-js
https://stackoverflow.com/questions/37288044/atom-electron-cant-find-path-of-child-process

I am running window 10 home. I am having the same issues.
I tried installing and re-installing
VS code reports

I also tried

Possible alternatives to Node spawn on windows:

https://github.com/MarcDiethelm/superspawn
https://www.npmjs.com/package/cross-spawn

Nothing seems to work still getting same error
PS C:\WINDOWS\system32> bash-language-server -v
Version is 1.3.1
Any Fix for this?

Having the same issue on mac :/

@pryorda I think that might be an unrelated issue... But how did you install the server? Yarn or npm? And which Shell are you using?

zsh, npm i -g bash-language-server
CO007-mac ➜  /tmp npm i -g bash-language-server
npm WARN checkPermissions Missing write access to /usr/local/lib/node_modules/bash-language-server
npm ERR! path /usr/local/lib/node_modules/bash-language-server
npm ERR! code EACCES
npm ERR! errno -13
npm ERR! syscall access
npm ERR! Error: EACCES: permission denied, access '/usr/local/lib/node_modules/bash-language-server'
npm ERR!  { [Error: EACCES: permission denied, access '/usr/local/lib/node_modules/bash-language-server']
npm ERR!   stack:
npm ERR!    'Error: EACCES: permission denied, access \'/usr/local/lib/node_modules/bash-language-server\'',
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'access',
npm ERR!   path: '/usr/local/lib/node_modules/bash-language-server' }
npm ERR!
npm ERR! The operation was rejected by your operating system.
npm ERR! It is likely you do not have the permissions to access this file as the current user
npm ERR!
npm ERR! If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm ERR! the command again as root/Administrator (though this is not recommended).

npm ERR!      /Users/dpryor/.npm/_logs/2018-06-30T23_28_24_260Z-debug.log
CO007-mac ➜  /tmp sudo npm i -g bash-language-server
Password:
/usr/local/bin/bash-language-server -> /usr/local/lib/node_modules/bash-language-server/bin/main.js
+<EMAIL_ADDRESS>updated 1 package in 0.752s
CO007-mac ➜  /tmp bash-language-server
internal/modules/cjs/loader.js:596
    throw err;
    ^

Error: Cannot find module './build/Debug/tree_sitter_runtime_binding'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:594:15)
    at Function.Module._load (internal/modules/cjs/loader.js:520:25)
    at Module.require (internal/modules/cjs/loader.js:650:17)
    at require (internal/modules/cjs/helpers.js:20:18)
    at Object.<anonymous> (/usr/local/lib/node_modules/bash-language-server/node_modules/tree-sitter/index.js:5:13)
    at Module._compile (internal/modules/cjs/loader.js:702:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:713:10)
    at Module.load (internal/modules/cjs/loader.js:612:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:551:12)
    at Function.Module._load (internal/modules/cjs/loader.js:543:3)
CO007-mac ➜  /tmp

Additional Output.
521 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/node-gyp-bin:/usr/local/lib/node_modules/bash-language-server/node_modules/tree-sitter/node_modules/.bin:/usr/local/lib/node_modules/bash-language-server/node_modules/.bin:/usr/local/lib/node_modules/.bin:/opt/chefdk/bin:/Users/dpryor/.chefdk/gem/ruby/2.4.0/bin:/opt/chefdk/embedded/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/Users/dpryor/go/bin:/usr/lib/go-1.9/bin:/sbin/bin:/snap/bin:/Users/dpryor/.local/bin:/usr/local/opt/go@1.9/libexec/bin:/opt/local/bin:/opt/local/sbin:/usr/local/sbin:/usr/local/bin:/opt/chefdk/gitbin
522 verbose lifecycle<EMAIL_ADDRESS>CWD: /usr/local/lib/node_modules/bash-language-server/node_modules/tree-sitter
523 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'node-gyp rebuild' ]
524 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
525 info lifecycle<EMAIL_ADDRESS>Failed to exec install script
526 timing action:install Completed in 469ms
527 verbose unlock done using /Users/dpryor/.npm/_locks/staging-3a08f0df5026584d.lock for /usr/local/lib/node_modules/.staging
528 timing stage:rollbackFailedOptional Completed in 73ms
529 timing stage:runTopLevelLifecycles Completed in 4730ms
530 verbose stack Error<EMAIL_ADDRESS>install: `node-gyp rebuild`
530 verbose stack Exit status 1
530 verbose stack     at EventEmitter.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/index.js:304:16)
530 verbose stack     at EventEmitter.emit (events.js:182:13)
530 verbose stack     at ChildProcess.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/lib/spawn.js:55:14)
530 verbose stack     at ChildProcess.emit (events.js:182:13)
530 verbose stack     at maybeClose (internal/child_process.js:961:16)
530 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:248:5)
531 verbose pkgid<EMAIL_ADDRESS>532 verbose cwd /usr/local/lib/node_modules
533 verbose Darwin 17.6.0
534 verbose argv "/usr/local/Cellar/node/10.5.0_1/bin/node" "/usr/local/bin/npm" "i" "-g" "bash-language-server"
535 verbose node v10.5.0
536 verbose npm  v6.1.0
537 error code ELIFECYCLE
538 error errno 1
539 error<EMAIL_ADDRESS>install: `node-gyp rebuild`
539 error Exit status 1
540 error Failed at the<EMAIL_ADDRESS>install script.
540 error This is probably not a problem with npm. There is likely additional logging output above.
541 verbose exit [ 1, true ]

Looks like I had to install Xcode, accept the license, blah blah.
Thanks

I've just released version 1.3.1 of the VSCode extension which adds a new configuration option bashIde.path.  Please see if you can use that to run it on Windows. As for Atom I think it's a good idea to use cross-spawn.

I'm on Gentoo, it can't seem to find the server, because it's dumping the following:
module.js:549
    throw err;
    ^

Error: Cannot find module './build/Debug/tree_sitter_runtime_binding'
    at Function.Module._resolveFilename (module.js:547:15)
    at Function.Module._load (module.js:474:25)
    at Module.require (module.js:596:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (/usr/lib64/node_modules/bash-language-server/node_modules/tree-sitter/index.js:5:13)
    at Module._compile (module.js:652:30)
    at Object.Module._extensions..js (module.js:663:10)
    at Module.load (module.js:565:32)
    at tryModuleLoad (module.js:505:12)
    at Function.Module._load (module.js:497:3)

Why is it wanting a debug module?

@Lucretia Are you setting the path explicitly using the new config options?
I'm not sure about the missing debug module. @maxbrunsfeld any idea?

@mads-hartmann I've had the option not set and had it set, same issue. Running the bash-language-server directly produces the above error.

@Lucretia How did you install bls?

Why is it wanting a debug module?

I think that on an older version of Tree-sitter, that error was thrown even if the Release module failed to load because it would try to 'fall back' to the Debug module. That's been fixed for more recent Tree-sitter versions.

I'm not sure about the missing debug module. maxbrunsfeld any idea?

I'm guessing that people are installing your module with npm manually, so the native modules are not getting compiled against the correct version of node.js?

@pryorda npm i -g bash-language-serveras root (on gentoo).

Have you tried to build from source to see what happens?

Version 1.3.1 on Ubuntu 18.04 works for me! :+1:

@mads-hartmann Once [this PR] lands, we could do something similar for tree-sitter-bash, and then users would be able to install the package without having a C compiler toolchain.

@maxbrunsfeld I think you forgot the link? 😊

🤦♂ thanks. I added it above.

@maxbrunsfeld Just had a look - that would be amazing - the current install process for this project is way more complicated then I would've liked

If you change the prefix before installing, it works.  You can specify your global lib folder.  For example, I did npm config set prefix="/usr/local/" before installing and that fixed the problem

@fakuivan @thehandcoder please try to install the latest bash-language-server by running npm install -g bash-language-server. We hope to have fixed the issue in version 1.5.1 by the help from @wingrunr21.

I was having this problem as well, and oddly, by running the following command it fixed it for me on Ubuntu 18.04, VS Code 1.27.2:
sudo npm i -g bash-language-server tree-sitter tree-sitter-bash --unsafe-perm

I ran the install command for each of the modules separately, and that didn't work... it was only when I did it all at once did it resolve things.

@alexford-bg Thank you so much, work like a charm on MacOSX, too.

# npm install -g bash-language-server
npm ERR! code EAI_AGAIN
npm ERR! errno EAI_AGAIN
npm ERR! request to https://registry.npmjs.org/whatwg-mimetype/-/whatwg-mimetype-2.2.0.tgz failed, reason: getaddrinfo EAI_AGAIN registry.npmjs.org:443

npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2018-10-02T14_02_39_858Z-debug.log

The tail of that log is:
885 silly pacote prebuild-install@https://registry.npmjs.org/prebuild-install/-/prebuild-install-5.2.0.tgz extracted in 6514ms
886 verbose unlock done using /root/.npm/_locks/staging-5416abac55ed2dd0.lock for /usr/lib64/node_modules/.staging
887 verbose type system
888 verbose stack FetchError: request to https://registry.npmjs.org/whatwg-mimetype/-/whatwg-mimetype-2.2.0.tgz failed, reason: getaddrinfo EAI_AGAIN registry.npmjs.org:443
888 verbose stack     at ClientRequest.req.on.err (/usr/lib64/node_modules/npm/node_modules/pacote/node_modules/make-fetch-happen/node_modules/node-fetch-npm/src/index.js:68:14)
888 verbose stack     at ClientRequest.emit (events.js:180:13)
888 verbose stack     at TLSSocket.socketErrorListener (_http_client.js:395:9)
888 verbose stack     at TLSSocket.emit (events.js:180:13)
888 verbose stack     at emitErrorNT (internal/streams/destroy.js:64:8)
888 verbose stack     at process._tickCallback (internal/process/next_tick.js:178:19)
889 verbose cwd /root
890 verbose Linux 4.19.0-rc4
891 verbose argv "/usr/bin/node" "/usr/bin/npm" "install" "-g" "bash-language-server"
892 verbose node v9.11.2
893 verbose npm  v5.6.0
894 error code EAI_AGAIN
895 error errno EAI_AGAIN
896 error request to https://registry.npmjs.org/whatwg-mimetype/-/whatwg-mimetype-2.2.0.tgz failed, reason: getaddrinfo EAI_AGAIN registry.npmjs.org:443
897 verbose exit [ 1, true ]

I was having this problem as well, and oddly, by running the following command it fixed it for me on Ubuntu 18.04, VS Code 1.27.2:
sudo npm i -g bash-language-server tree-sitter tree-sitter-bash --unsafe-perm

I ran the install command for each of the modules separately, and that didn't work... it was only when I did it all at once did it resolve things.

Thank you! That is THE only thing I could find that worked.

Setting an absolute path for Bash Ide: Path worked for me.

@nottoseethesun just to be clear, where did you set this? It seems bash-language server and other npm scripts are not on your path....

@skovhus I set it in the VSCode Prefs. I did put the server in my path, and I did also do the "hack" of installing the dep's globally as well.  I am using nvm, but I used the correct path to the binary.   For me anyway, using the shell path didn't work.

I have the same problem. Although I had installed bash-language-server

incurred this problem on mac Catalina, have Xcode 10.2.1 installed with accepted license.

Still not working same Brian's report.

With the new version (1.4.0) of the VS Code extension, we got rid of the manual npm install step of the bash-language-server. Hopefully, this will resolve npm installation issues. :)
The latest version of bash-language-server (the npm package) should also be easier to install.
Let me know if this solves the issues.


fix: fix thermostat device types

me === 🤡
another kick in the butt to use public types

The version previously approved didn't expose min/max setpoints


fix: block and arguments is not equal

clickhouse: when table field is Array(string)

Hi @knkp1209
Can you write a test case for the clickhouse driver?

Closing this PR for now, please open a PR with test cases for clickhouse to reproduce the issue.
Thank you.


docs(sumologicextension): add information about running the collector as systemd service

Resolves #683

One more change to consider, update the systemd section in Installation.md and have the user confirm that the opentelemetry user has a home directory defined and if not explicitly set a directory for storing the creds in the otel config.
If for some reason the user's system already has an opentelemetry user defined without a home directory the user creation instructions won't generate a home directory.


Verify Airnode v0.11 uses node v18

@aTeoke This is to look into your request.

Yes: https://github.com/api3dao/airnode/pull/1647

Or wait- do you mean environment in processing specifications of config.json?

@dcroote @aTeoke - So Airnode uses node 18, but I thought that the environment field  for pre/post processing was user defined. So why can't it be whatever node version they want?

environment is restricted to two values here: https://github.com/api3dao/ois/blob/59b3dc3f53e45a8e700b340d3af0c0db64a9fade/src/ois.ts#L273

Closing after our discussion related to https://github.com/api3dao/api3-docs/pull/1207


⚠ Bofrost.be has degraded performance

In 0a1f684, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1412 ms

Resolved: Bofrost.be performance has improved in 351e5a3 after 25 minutes.


Rolling back columns and indices added in separate migrations clashes with index recreation

When a column with an index is being added in a separate migration, this will, when being rolled back, cause an error when the column is being dropped, as the fix from #2040 will try to re-add the index for which the referenced column(s) doesn't exist anymore.
I mistakenly assumed that reversed plans would drop indices before removing columns, however while the inverted commands do indeed generate index drops for the to be removed columns, these index drops are being filtered out by the plans conflict resolution:
https://github.com/cakephp/phinx/blob/3cfd71a5caad784711c6525f5fcc246ff35a1aca/src/Phinx/Db/Plan/Plan.php#L193-L204

The error can be reproduced using these two migrations, where table creation comes first, and then table alteration. Rolling back the alteration migration will trigger the error.

==<PHONE_NUMBER>0113 RollbackFailureAlter: reverting
In PdoAdapter.php line 192:
SQLSTATE[HY000]: General error: 1 no such column: foo_id

<?php
declare(strict_types=1);

use Phinx\Migration\AbstractMigration;

final class RollbackFailureCreate extends AbstractMigration
{
    public function change()
    {
        $this
            ->table('test')
            ->create();
    }
}

<?php
declare(strict_types=1);

use Phinx\Migration\AbstractMigration;

final class RollbackFailureAlter extends AbstractMigration
{
    public function change()
    {
        $this
            ->table('test')
            ->addColumn('foo_id', 'integer')
            ->addIndex(['foo_id'], ['name' => 'FOO_ID_IDX'])
            ->update();
    }
}

Ugh, yeah, now I more greatly remember why I wrote https://github.com/cakephp/phinx/pull/2040#issuecomment-986092852 in the first place and marked the PR as a draft 🤦
I think a similar bug may occur then on renaming a column that has an index on it?

Well after a quick test it seems that this is only true for certain indices depending on the DBMS, there are indices for which the column references will be renamed accordingly too when renaming a column.
So I guess renaming columns would be affected too 😢

Thinking about it, it is known what column are being renamed/removed, so shouldn't it be possible to parse the index creation SQL and compare it against the change, and then either exclude the index in case one or more of its columns are being dropped, or rename its columns accordingly?

Yes, and it's probable that can do this via a simple regex as I'd expect the SQL we pull out of schema_master to be somewhat well-formed, but would have to validate that.

Given that they can be created manually, and support expressions, there can probably be quite a variety in syntax. But I guess it would be reasonable to expect that people who are using fancy custom indices, to then handle updating them themselves when columns are being renamed/dropped, ie Phinx would only need to support the kind of indices syntax that can be created through its own API?

 For example:
CREate INdex test_idx On test (foo asc , aBs(bar)   DEsc )

is stored as-is, except for CREate INdex, meaning sqlite_master will contain/return:
CREATE INDEX test_idx On test (foo asc , aBs(bar)   DEsc )

I also looked into the behavior of other DBMS for renaming/dropping columns with indices, and it varies. MySQL will error out if any column of the index is a function expression, Postgres drops indices and renames columns even when function expressions are involved (not sure if there are edge cases), and in SqlServer it seems that using expressions isn't possible, dropping columns seems to always trigger an error, and renaming columns seems to give some kind of notice (which I assume needs to be obtained manually, I'd need to check on that again).
Long story short, there is no consistent behavior that the SQLite adapter would necessarily have to consider.
Unless you're already working on something @MasterOdin, I can give it a shot.

Checking SqlServer again, it actually has a similar problem as Sqlite here, on rollback it will trigger an error because that's what SqlServer seems to do when dropping columns that have dependent indices.
@MasterOdin Do you remember whether filtering out the index drop actions from the plan was actually required to solve a specific problem? If it was just an optimization, then maybe it can be removed.

It is almost certainly just an optimization, and there's probably no harm in doing a drop index before dropping a column during a migration, other than a few more queries run. However, I don't think that people really demand their migration library runs the absolute minimal set of queries, and are probably fine with a bit of non-optimization for correctness.
If we really wanted to keep it, we could just do an adapter type check that wraps around that block that checks if this optimization is supported or not.

I'll create a new issue for the renaming problems, and probably also one for explicit column dropping, as the implications of the implicit dropping as part of a rollback, are a bit different. For explicit drops the question is whether/when to automatically drop indices for SQLite.

For explicit drops the question is whether/when to automatically drop indices for SQLite.

I would see if we can just remove that optimization for now and if there's nothing breaking about it, I'd go with that for now.

I would classify this as two different problems, removing the optimization will not affect a removeColumn() that one would explicitly invoke in a migration, and since it is kind of "affected" by the index restoration change, I figured that it might still be worth looking into it in time.


Spotify Search Functionality

This functionality utilizes the Spotify Web API (user must be authorized) to be able to search for Tracks, Artists, Albums, Podcasts, and Shows on Spotify. Once the user has entered more than 2 characters the current query string will be searched and the search suggestions will be updated in the suggestion model as they are fetched. The search suggestion screen isn't very flexible with how it displays results so when the user clicks OK on the input state they will be taken to BrowseView that shows the search results with cover art and formatted similarly to how the Spotify app displays results.
Demo of Car UI:
https://user-images.githubusercontent.com/11298573/106551348-a6dbc180-64da-11eb-8774-99d7d958307b.mp4
I've also added a "Search" tab to the phone UI that allows for simple searching of a query.
Demo of PhoneUI:
https://user-images.githubusercontent.com/11298573/106551340-a0e5e080-64da-11eb-9483-cade5b0c922c.mp4

Can you also change the MusicPermissionsFragment layout to show a different string like Ability to search Spotify to emphasize the user benefit of clicking the button?

The SpotifyAppController.Connector has logic to trigger MusicAppDiscovery again after logging in with the control API to add the Browseable flag. Can you add similar logic to add the Searchable flag after the Web API is authenticated?

Can you also change the MusicPermissionsFragment layout to show a different string like Ability to search Spotify to emphasize the user benefit of clicking the button?

Updated the MusicPermissionsFragment string for the Spotify Web API authorization to be the suggested "Ability to search Spotify". It now fits in so much better with regards to the other permission text!

The SpotifyAppController.Connector has logic to trigger MusicAppDiscovery again after logging in with the control API to add the Browseable flag. Can you add similar logic to add the Searchable flag after the Web API is authenticated?

Yeah definitely, I updated the searchable flag after the SpotifyWebApi is authenticated along with the fixes to the MusicAppDiscovery class

Looks great, thank you so much!


no Understand-determine-basal.md in gitbooks

Seems that Understand-determine-basal.md is not deployed correctly in gitbooks. See below:

We are trying to deprecate gitbooks.  Where do you still see links to it?  We'd rather delete them all so people stop trying to use it.

Hmm, actually I don't remember. I think that I googled "openaps documentation" or similar.

The gitbook has now been removed.  Going through all the docs and removing references to it.  If you come across any, please PR to fix.


rbd: failed to modprobe rbd error:executable file not found in $PATH

Hi:
rancher server 1.6.2
k8s 1.5.4
CentOS 7.3
I got a problem here! I try to create a kubernetes statefull nginx service with ceph rbd as the backend dynamically provision persistent volume on rancher, but failed, it shows the error "rbd: failed to modprobe rbd error:executable file not found in $PATH ". I checked the  image rancher/k8s:v1.5.4-rancher1-4 and  the container kubernetes-controller-manager-1, they both have the rbd command!
I try to follow the example web.yaml on https://kubernetes.io/docs/tutorials/stateful-application/basic-stateful-set/

web-0  logs:
1h		1m		60	{kubelet ceph-osd219}			Warning	  FailedMount	MountVolume.SetUp failed for volume "kubernetes.io/rbd/fc8b7dab-6ec2-11e7-8665-02672b04ad55-pvc-e6a430b4-6eb3-11e7-aa48-02672b869d7f" (spec.Name: "pvc-e6a430b4-6eb3-11e7-aa48-02672b869d7f") pod "fc8b7dab-6ec2-11e7-8665-02672b04ad55" (UID: "fc8b7dab-6ec2-11e7-8665-02672b04ad55") with: rbd: failed to modprobe rbd error:executable file not found in $PATH
1h		51s		48	{kubelet ceph-osd219}			Warning	  FailedMount	Unable to mount volumes for pod "web-0_default(fc8b7dab-6ec2-11e7-8665-02672b04ad55)": timeout expired waiting for volumes to attach/mount for pod "default"/"web-0". list of unattached/unmounted volumes=[www]
1h		51s		48	{kubelet ceph-osd219}			Warning	  FailedSync	Error syncing pod, skipping: timeout expired waiting for volumes to attach/mount for pod "default"/"web-0". list of unattached/unmounted volumes=[www]
kubectl get pvc|grep web
www-web-0   Bound     pvc-e6a430b4-6eb3-11e7-aa48-02672b869d7f   1Gi        RWO           3h
www-web-1   Bound     pvc-e6ab7435-6eb3-11e7-aa48-02672b869d7f   1Gi        RWO           3h
==================================================
also, there's another problem with ceph rbd
ceph-secret.yaml
apiVersion: v1
kind: Secret
metadata:
name: ceph-secret
namespace: default
type: "kubernetes.io/rbd"
data:
key: "QVFDM0htNVphbjlMRGhBQVhaSENkQUYzOWJYY0V3ZHBWNnkvY0E9PQo="
ceph-rbd-storageclass.yaml
apiVersion: storage.k8s.io/v1beta1
kind: StorageClass
metadata:
name: fast
provisioner: kubernetes.io/rbd
parameters:
monitors: <IP_ADDRESS>:6789,<IP_ADDRESS>:6789,<IP_ADDRESS>:6789
adminId: k8s
adminSecretName: ceph-secret
adminSecretNamespace: default
pool: k8s
userId: k8s
userSecretName: ceph-secret
kubernetes on rancher 1.6.2 only works when the ceph secret's namespace is default, if change the namespace to kube-system.The PVC can not be bound!

Looks like a k8s specific bug: https://github.com/kubernetes/kubernetes/issues/38923

Please let us know in a comment if you can still reproduce the issue, and we'll reopen it


Enable async downloading of individually specified articles

The docstring of newspaper.mthreading.NewsPool.__init__ appears to indicate that one can pass it a list of source or article objects. The class implementation, however, cannot handle article objects; I modified newspaper.mthreading.NewsPool.set to remedy this.

Hey @lebedov, this is an awesome fix! Will merge now 👍 💯


[SPARK-10738][ML] Refactoring Instance out from LOR and LIR, and also cleaning up some code

Refactoring Instance case class out from LOR and LIR, and also cleaning up some code.

Merged build triggered.

Merged build started.

Test build #42781 has started for   PR 8853 at commit 9410c24.

Test build #42781 has finished for   PR 8853 at commit 9410c24.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42781/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #42785 has started for   PR 8853 at commit 1d13918.

Test build #42785 has finished for   PR 8853 at commit 1d13918.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42785/
Test PASSed.

@dbtsai The main question would be where we should put Instance. I'm not sure ml.feature is a good place for it, though I don't have good alternatives. Another place is ml.util, not ideal either.

I know ml.feature is mostly for feature engineering, but I don't think ml.util is a good place. How about make it as first citizen in ml? I will implement different type of Instance for unweighted version.

It will be quite hard, e.g., in survival model, we have a column called censor. In the future, we might want to add offset to instances used in linear models.

Okay. It makes sense. But it's still private, and cleans up some code, do u think it worths to merge it first and we can think about where is the best place to have it?

Merged build triggered.

Merged build started.

Test build #43126 has started for   PR 8853 at commit 2ac1d03.

Test build #43126 has finished for   PR 8853 at commit 2ac1d03.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/43126/
Test PASSed.

Merged build finished. Test PASSed.

LGTM. Merged into master. Sorry for the delay!


Please add @todo jsDoc to checkTypes

warning: jsDoc unavailable tag todo
See usejsdoc: @todo.

Thanks for the report @piotr-cz!
Can you please post the exact code which is creating the error and your configuration?
In my quick tests everything worked and according to https://github.com/jscs-dev/jscs-jsdoc/blob/master/lib/tags/jsdoc3.json#L70 the @todo token should exist.

Sorry for false alarm.
I'm using the google preset and looks like this is the desired behavior for checkannotations: closurecomiler option.

I think this is a problem that should be solved. I thought about using multiple tag sets.
@markelog What you think about it? Should we add jsdoc3 tag set to google preset?
Ref: jscs-dev/jscs-jsdoc#69

I'd love such solution - but I'm not sure if this is not overcomplicating things.


Dugup - Fan to Earn (KPOP Real Use Case)

Track
Track 2
team participants
Giovanni
https://github.com/aeither
discord: aithez
Sujin
https://github.com/sujinkim89
discord: stay_keen

Dugup
Fan to Earn (KPOP Real Use Case)
Demo
https://github.com/aeither/ICP_HUBS_Hacker_House/assets/36173828/97fff9a1-549f-4acb-9835-19e099555b12

Slides:
https://docs.google.com/presentation/d/1IWWew-CG7Tyd11E1ODv4_PuhhmtanVdX/edit?usp=sharing&ouid=103129906474846662491&rtpof=true&sd=true
Description
DUGUP is not a Fan to Earn but a Fan to Grow Artist social-fi app. The KPOP community has shown the power to elevate BTS to the top of the Billboard charts. This growth is driven not by agency plans but by fan engagement, creating artists together. Dugup is designed so that fans' engagement contributes to the growth of artists.
How to Collect DUG COIN:
Scan the Offline Merchandise: Use your device to scan the QR code or barcode on the merchandise associated with DUG COIN promotions.
Insert the Code: After scanning, a unique code will appear. Enter this code into the designated area on the DUG COIN platform or app.
Add to My Moments: Share this event by posting it to your 'Moments', a feature on the platform that allows you to share updates or achievements.
Badge (POAP) Distribution: Receive a Proof of Attendance Protocol (POAP) badge as acknowledgment of your participation and proof of ownership.
My Page: View and manage your collected DUG COINS and badges on your personal page within the platform.
How to Spend DUG COIN:
Community Voting: Use DUG COINS to vote on community decisions. Each DUG COIN might represent one vote in community polls or decisions.
Exclusive Access: Spend DUG COINS to gain access to special events, content, or merchandise only available to coin holders.
Marketplace Purchases: Use DUG COINS to buy, sell, or trade goods and services on the platform’s marketplace.
Staking or Investment: Allocate DUG COINS towards staking or investment opportunities within the platform to potentially earn more.
Charity or Crowdfunding: Contribute DUG COINS towards charity drives or crowdfunding campaigns hosted on the platform.
Track

Fully on-chain dapps

Track
Team participants
Giovanni.

http://github.com/aeither
Discord: aithez

Sujin.

https://github.com/sujinkim89
Discord: stay_keen

Canister ID(s) of mainnet deployment
abtwt-7yaaa-aaaap-qcgca-cai

https://a4gq6-oaaaa-aaaab-qaa4q-cai.raw.icp0.io/?id=abtwt-7yaaa-aaaap-qcgca-cai

Feedback
What have you learned

ICP glossary: Canisters, Cycles, Candid, Internet idendity, ledger cannister
Different languages to build program. azle for typescript, bitfinity evm for solidity...
How to build with azle
dfx CLI

What were the challenges you ran into

the image size is huge
deployment issues
ai chatbot and documentation not straightforward to follow in order to make a deployment
cors

What are you proud of

fullstack app. frontend + backend
deployment on mainnet

Screenshots


Image creation reponse changed with new url field

Hello,
I was working with this package for a study project and noticed that the image generation API always returns null but prints out the response to the console.
It seems that the image generation API output format has changed. The docs still state that a field b64_json is present, but when running the examples here: together AI image generation, the output has changed and apparently the field is now url containing a URL to the image on the together AI server.
I changed that in the model definition and wrote a test for the image generation. Also, I added a method for generating a b64 image data URL out of the URL from the API response.

Hello friend bern, thank you so much for making this request.
Please give me few days to check and approve :)
Thank you!

Hello, I just wanted to ask if you could check and approve the changes :)

Hey @bernhard759 !
I apologize for the long delay in addressing this PR. I'll be reviewing and closing this shortly - thank you for your patience!


Remove unused wordCount utility

issue TryGhost/Ghost#9507

Removed unused wordCount utility

Coverage increased (+0.2%) to 71.945% when pulling 304e4c957dc48097f2bde1b3d3ab5cc50f184e4b on rkalis:removed-unused-wordcount into fddf679a27d8898554f9a43ae7789532a555c8c8 on TryGhost:master.


Language Variant File Names should be aligned with ISO 3166

This ISO standard defines country codes in capitals:
http://en.wikipedia.org/wiki/ISO_3166-1_alpha-2
Also this standard defines language tags in the form de-AT or en-GB etc.
http://en.wikipedia.org/wiki/IETF_language_tag
Same point I made here for momen.js:
https://github.com/timrwood/moment/issues/869
Currently, the lang files mix upper and lower case for the variant suffix which is not good I think.

I come from the future: Is this handled? I want to use navigator.language but I want unhandled cases (say es-CL, es-AR, etc.) to fallback to es.

Duplicate of #55


Ready to release v1.5.2

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

Comparison is base (42b0594) 100.00% compared to head (483db5a) 100.00%.
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #14   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           11        11
  Lines          277       274    -3
  Branches        13        13
=========================================
- Hits           277       274    -3

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


Broken links in log messages

Launching pcm-core.exe from CI artifacts on Win 10 gives me this error:
ERROR: winpmem_64.sys not found in current directory. Download it from https://volatility.googlecode.com/svn-history/r2813/branches/scudette/tools/windows/winpmem/binaries/winpmem_64.sys .

The link doesn't exist anymore. Related piece of code: https://github.com/opcm/pcm/blob/456ff2fd8734fc1b51114d9e5bf92b6b42fb588e/client_bw.cpp#L52

you can download the file from https://github.com/google/rekall/tree/master/tools/pmem/resources/winpmem
Any chance you can create a pull request with a fix?

Thanks for your reply. Sure, I'll do it.

thanks!!


PREPROC_BAD_INDENT is not triggered when tabs are used for indent

Describe the bug
When you insert tabs inside preproc indent it doesn't trigger PREPROC_BAD_INDENT norm error.
I think it should be considered as an error.
Erroneous code
# define TEST_H

# define HELLO_WORLD "Hello World"
# 		define FIRST					1
# 		 define SECOND			2
# 		  define THIRD	3

# include <stdio.h>
# 						include <stdlib.h>

#endif

Additional infos

OS: Ubuntu 20.04.3 (WSL)
python --version: 3.8.10
norminette -v: 3.3.40

Additional context

.

Hello,
This looks fixed as of my latest build.
Thank you for the issue.


Mirror on F.Cu and B.Cu?

I think I must be being a bit thick but I am trying to get the attached SVG to footprint where the pattern is on both the front and back copper layers. When I run the command
 ~/bin/svg2mod/svg2mod.py -i cap.svg --format=pretty
I get a kicad_mod file but the copper pattern is only ever on F.Cu. And I'm sorry but I don't really understand the bit in your README.md:

Drawing elements will be mapped to front layers by default. Mirrored images of these elements can be automatically generated and mapped to back layers in a separate module (see --front-only option).

Is there a way I can get this pattern on both sides of the footprint?
TIA
cap.svg.zip

I am not sure but I think you might have to split this into two SVGs or amend the script.

Just FYI - I simply copied all the data from the mod file which was on F.Cu and did a search and replace to replace F.Cu with B.Cu, then pasted this back into the mod file. Seems to work correctly.

Ah, that's smarter, yes. Still we should probably amend the script to specify exact layers.


Move validator to manager. Add validator

Coverage decreased (-0.4%) to 52.334% when pulling 117971628e61d3d6b143dc369162d3a8f702a4ed on oleksiyVeretiuk:master into 0e5facd6b406c0eee768a228f6ddab7de12209a3 on openprocurement:master.


P44 missing $applyAsync there

When you describe all the places where we should "move on" when we get exceptions, you forgot to name $applyAsync there.

Mentioned now, thanks


UHH-9755: Small fix for tpr unit form default configuration

UHF-9755

What was done

Change show_www to be visible on the tpr_unit form, it used to be both visible and hidden according to the configuration

How to install

Make sure your instance is up and running on latest dev branch.

git pull origin dev
make fresh

Update the Helfi Platform config

composer require drupal/helfi_platform_config:dev-UHH-9755_remove_show_www_from_hidden

Run make drush-updb drush-cr

How to test

[ ] This fixes the show_www being in two places at the same time in the configuration and this change will fix the configurations next time automatic update is run. The site works correctly on both occasions but the configuration shouldn't be the way it is to avoid confusion.
[ ] Check that code follows our standards

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

Project coverage is 14.53%.  Consider uploading reports for the commit 6c41162 to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #716   +/-   ##
=========================================
  Coverage     14.53%   14.53%
  Complexity      256      256
=========================================
  Files            30       30
  Lines           977      977
=========================================
  Hits            142      142
  Misses          835      835

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


Is Skype supposed to works?

I haven't tried previous versions, so I don't know if this had been introduced in the last build fo Skype.
but the container don't start, content of logs/skype-startup.log:
Failed to move to new namespace: PID namespaces supported, Network namespace supported, but failed: errno = Operation not permitted

it seem that --privileged is required

Checkout #350

Same problem here. User namespaces are enabled, so #350 doesn't help.
Running:
github.com/jessfraz/dockerfiles/skype$ docker build -t skype .
github.com/mviereck/x11docker$ ./x11docker --home --newprivileges skype
$ cat ~/.local/share/x11docker/skype/.config/skypeforlinux/logs/skype-startup.log
Failed to move to new namespace: PID namespaces supported, Network namespace supported, but failed: errno = Operation not permitted
$ cat /proc/sys/kernel/unprivileged_userns_clone
1
$ uname -a
Linux machine 5.9.0-1-amd64 #1 SMP Debian 5.9.1-1 (2020-10-17) x86_64 GNU/Linux
$ cat /etc/debian_version
bullseye/sid
$ docker --version
Docker version 19.03.13, build 4484c46

Running ./x11docker --home --newprivileges skype -- --cap-add=SYS_ADMIN --security-opt seccomp=unconfined doesn't help either, same error.


Let bazel run unit tests with the golang race detector enabled

What this PR does / why we need it:
We never enabled the golang race detection feature when switching from native go-executed unit-tests to bazel executed unit-tests.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:
NONE

/cc @jean-edouard

/close
superseeded by #4251


Python 3.7 compatibility

test_simpleion fails on Python 3.7, as does vectors test data.
See https://travis-ci.org/jayvdb/ion-python/jobs/495377901

I will look into this.


Crash during MessageQueue::flush

Godot version
v4.0.2.stable.custom_build
System information
Windows 10, Forward+
Issue description
Seeing hundreds of reports of crash on exit which causes some of my own cleanup code to not be run (saving settings, etc.)
Example stack trace: https://kar-games.sentry.io/share/issue/ec148845fd2841feab9cd51472c540b7/
Steps to reproduce
Unknown
Minimal reproduction project
Unknown

The provided stack trace seems kinda messed up and contains a bunch of frames that are not actually part of the call stack (probably artifacts of stack walking without proper symbols)
The frames I have numbered here give a coherent stack trace, while the remaining ones just don't fit into it at all:
1  Object::get_instance_id (object.h:750)
2  CallableCustomMethodPointer<T>::get_object (callable_method_pointer.h:97)
3  Callable::get_object (callable.cpp:132)
   _Mtx_unlock (mutex.cpp:170)
   TextServerExtension::cleanup (text_server_extension.cpp:1406)
4  MessageQueue::flush (message_queue.cpp:266)
   TextServerAdvanced::_cleanup (text_server_adv.cpp:6608)
5  Main::cleanup (main.cpp:3300)
6  widechar_main (godot_windows.cpp:181)
   RDPipelineColorBlendStateAttachment::get_write_g (rendering_device_binds.h:647)
   IUnknown_Set
   StrStrW
   SHGetInputLanguageTag
   SHGetInputLanguageTag
7  _main (godot_windows.cpp:203)
8  main (godot_windows.cpp:222)
9  invoke_main (exe_common.inl:78)
10 __scrt_common_main_seh (exe_common.inl:288)
11 BaseThreadInitThunk
12 RtlUserThreadStart

Thus neither TextServerExtension or TextServerAdvanved are likely to have anything to do with this crash.
Instead when enabling verbose function names on sentry the callable in question: CallableCustomMethodPointer<NavigationServer2D,RID>::get_object() const turns out to be a function on NavigationServer2D that takes a single RID argument and returns void, which could be _emit_map_changed, map_force_update, or free. None of which are called deferred by the engine tho, and I don't have any more information or code to continue looking into.
As a quick workaround I would remove all the #ifdef / #endif from this section and always enable the debug code that prevents issues like this: https://github.com/godotengine/godot/blob/7a0977ce2c558fe6219f0a14f8bd4d05aea8f019/core/object/callable_method_pointer.h#L81-L116

@RedworkDE Well that's even stranger since I don't use NavigationServer2D. I hadn't realized it was stack walking as I uploaded symbols for the binary.
I'll reach out to Sentry to determine what is going on and circle back with some better data.
Thanks for looking into it!

I figured out whats going on with the CallableCustomMethodPointer<NavigationServer2D,RID>::get_object() const stack frame, and what the NavigationServer has to do with this crash. And the answer is nothing. Nothing at all.
Instead the compiler has identified that CallableCustomMethodPointer<>::get_object doesn't depend on the template arguments at all an uses the same code for all instantiations, making this bit of information completely useless as well.
(Ie it is just random that sentry has identified the function as CallableCustomMethodPointer<NavigationServer2D,RID>::get_object; in the build I use for testing that function is instead identified as CallableCustomMethodPointer<GraphEdit,float,Vector2,Ref<InputEvent>>::get_object)

I can confirm this has nothing to do with TextServerExtension now - here is a proper stack trace, now based on 4.0.3.
https://kar-games.sentry.io/share/issue/0c3d18711c9e4249ad8cd7097a913f4a/
But now I can't really say where the problem is.

You can try to create build with ICF disabled (https://github.com/godotengine/godot/pull/78120) to get useful type information for the CallableCustomMethodPointer<T>::get_object frame to hopefully narrow things down a bit.


Don't fail if /sys/class/hidraw is missing

On a Linux system, if /sys/class/hidraw is missing, the keymaster client fails. One has to use the -noU2F flag to make it work. This is frustrating.

New PR added to try to fix this: https://github.com/Symantec/keymaster/pull/201

Fixed with merge of: https://github.com/Symantec/keymaster/pull/201


Fixed Issue when initialized Client threw error because cache_retain_time and  use_cache had no default values in nepse/core

Sorry I had already noticed and fixed this issue before noticing your pull request. It would be better if you had opened a issue before submitting a pull request :)
This issue has already been addressed and fixed in the latest release. v1.0.
Thank you for your contribution.


[Bug] [Python Task] Execute python task failed with java error

Search before asking

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

What happened
just create a workflow with only one python task node, which simply execute command print("hello m") and when execute this workflow an error happened.
execute log below
1717146925671.log
env:
os: centos7
database: postgresql-15
java: jdk1.8
python: python 3.6
zookeeper: zookeeper 3.7.x
dolphinscheduler: 3.2.1
deploy way: cluster deploy
I use new user dolphinscheduler to deploy and set dolphinschedueler's tenant is dolphinscheduler too.
What you expected to happen
execute python node successfully
How to reproduce
create a simple workflow with python node, execute command print("hello m") and execute the workflow.
Anything else
No response
Version
3.2.x
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

I didn't meet this problem, you can have a try to use JDK 11 instead

solved it by change enviroment PYTHON_LAUNCHER and PYTHON_HOME to actual python path in ~/.bashrc.
this error info occured again but can scheduler python node now.


[11.x] Add mariadb to databases config

This adds proper support for MariaDB, as the default/recommended collation for mysql has changed.
See https://github.com/laravel/laravel/pull/6239
See https://github.com/laravel/laravel/pull/6240
See https://github.com/laravel/laravel/pull/6241

Any ideas why the MariaDB 10 tests fail? I can't find a location were I missed adding mariadb as an available connection.

This should be expected since Testbench holds it's own configuration files.

@Tristan-MyAnaPro this seems like a good idea.
Seems like there are already plans to make uca1400_ai_ci the default. See https://jira.mariadb.org/browse/MDEV-30164

@Jubeki are you still working on this one?

Note -  MDEV-32336 deb default config - use collation-server = utf8mb4_uca1400_ai_ci https://github.com/MariaDB/server/pull/2775  for 11.3
That without an explicit collation utf8mb4 uses the collation uca1400_ai_ci. Applying this to Debian/Ubuntu will flow through to the mariadb:11.3 container on the next release.
If you want to ensure that the next version of MariaDB (server and container) to be released is going to be Laravel compatible (and give you a chance to tell MariaDB to fix things before release :-) ) you can use the following container images:

quay.io/mariadb-foundation/mariadb-devel:very-latest (beta/rc)
quay.io/mariadb-foundation/mariadb-devel:latest (stable)
quay.io/mariadb-foundation/mariadb-devel:latest-lts (stable and long term maintained)
quay.io/mariadb-foundation/mariadb-devel:ealiest-lts (old and still maintained)

These are the latest version of completed code merged into the main branches - https://quay.io/repository/mariadb-foundation/mariadb-devel?tab=tags . The code there has passed review and is expected to work so this isn't a beta tree, just an unreleased version.
information:

https://mariadb.org/new-service-quay-io-mariadb-foundation-mariadb-devel/ (service)
https://mariadb.org/mariadb-server-docker-official-images-healthcheck-without-mysqladmin/ (healthcheck)

Or just ask me. Happy to help.

@Jubeki we might want to resend in this PR as Taylor probably force-pushed his commits.

@grooverdan Thanks for the information about the changes in MariaDB. I applied the default collation in the config accordingly. I don't think the newer container image is needed here, because the tested mariadb version already supports the new collation, but of course the Laravel Team can decide to test against the new image. (FYI Laravel 11 will probably be released in February 2024 as far as I know).
@driesvints I rebased against slim-skeleton-11.x. Should I mark this PR as Ready for Review?


fs: Higher units for ETA - Issue #3221

What is the purpose of this change?
The ETA can now display higher units: days, weeks, and years, in addition to the previous units
Was the change discussed in an issue or in the forum before?
https://github.com/ncw/rclone/issues/3221
Checklist

[x] I have read the contribution guidelines.
[x] I have added tests for all changes in this PR if appropriate.
[x] I have added documentation for the changes if appropriate.
Note: I have looked for other details regarding the ETA String representation, and found nothing specific.

[x] I'm done, this Pull Request is ready for review :-)

That looks really good now - thank you.  Will merge.
(Apologies for the delay - I'm a bit behind).


Rename FetchClients.fetchClients to ClientRepo.all, and refactor

This is probably as good as its gonna get for now, so I'm merging.


Library that requires another, both in requirements.txt for Python Function App

Using the requirements.txt file to install the dependencies for a Python Function App can be limiting.
Say that one library (hawkeslib) requires another (Cython) to be installed first. If both are in the same requirements file, the installation of hawkeslib fails with ImportError: No module named 'Cython'.
The alternative is to use custom image to create Function Apps, but there has to be a middle ground in terms of flexibility, say allowing for additional prior commands to be executed on the remote environment, without having to resort to Docker.

@ankitkumarr - can you add documentation before closing this based on investigation?

This is an unusual scenario and the reason why a package requires Cython because the package you want to use is no longer supported in the current Python version and needs to be recompiled.
Currently, there are two blockers for performing multistage dependencies resolution in Python function app.

The dependency resolution service is provided by Microsoft/Oryx a limitation on our side which only supports a single requirements.txt file.
The Oryx Build does not take in installed Python Libraries for build process. Installing Cython on our build server doesn't mean you can use it right away.

Instead, one thing you can try out is by setting PRE_BUILD_COMMAND app settings in your function app to perform arbitrary build steps. Docs Here. You can try uploading your build_script.sh and execute it in the pre build command.


Error with load_credentials

hello
after installing the packages I created a YAML file as shown here
search_tweets_premium:
  account_type: premium
  endpoint: https://api.twitter.com/1.1/tweets/search/fullarchive/dev.json
  # if you have a bearer token, you can use it below. otherwise, swap the comment marks and use
  # your app's consumer key/secret - the library will generate and use a bearer token for you.
#bearer_token: <A_VERY_LONG_MAGIC_STRING>
  consumer_key: <Sznpkb******************************>
  consumer_secret: <lP8Xr*****************************************************>

but when I run the code as shown here
premium_search_args = load_credentials("C:/Users/DRC/Desktop/.twitter_keys.yaml",
                                       yaml_key="search_tweets_premium",
                                       env_overwrite=False)

I got this error
cannot read file C:/Users/DRC/Desktop/.twitter_keys.yaml
Error parsing YAML file; searching for valid environment variables
Account type is not specified and cannot be inferred.
        Please check your credential file, arguments, or environment variables
        for issues. The account type must be 'premium' or 'enterprise'.

---------------------------------------------------------------------------
KeyError                                  Traceback (most recent call last)
<ipython-input-55-953b455c2646> in <module>
      1 premium_search_args = load_credentials("C:/Users/DRC/Desktop/.twitter_keys.yaml",
      2                                        yaml_key="search_tweets_premium",
----> 3                                        env_overwrite=False)

~\Anaconda3\lib\site-packages\searchtweets\credentials.py in load_credentials(filename, account_type, yaml_key, env_overwrite)
    187                    if env_overwrite
    188                    else merge_dicts(env_vars, yaml_vars))
--> 189     parsed_vars = _parse_credentials(merged_vars, account_type=account_type)
    190     return parsed_vars
    191

~\Anaconda3\lib\site-packages\searchtweets\credentials.py in _parse_credentials(search_creds, account_type)
     80         """
     81         logger.error(msg)
---> 82         raise KeyError
     83
     84     try:

KeyError:

is that because of bearer token?
or there is any mistake in my YAML file?

Hi @fatimamb, it looks like your error message is truncated in the snippet you posted. Can you update it to show the entire thing?

@jrmontag thank you for replying, but there is not truncated that is all the error message

Hi--in the YAML you pasted, the commented-out bearer token line isn't indented. I think that indentation matters in YAML files, if you delete that line and make sure that all of the keys underneath the main key are indented the same amount does it work?
Also yeah, seems like in the first example your code couldn't find the file at all, but it looks like it found it when you removed the "."

Hi @fatimamb, thanks for adding more context. The first argument to load_credentials() should be the path to the yaml file and have the appropriate name - if your yaml starts with a . character, then make sure that is reflected in the load_credentials() argument.
The exception is suggesting that the account_type isn't being read correctly, but in the snippet that you posted, I see you have set account_type: premium so that looks correct. Your snippet does include the < and > characters around your masked credentials. Those are intended to denote the example <VERY_LONG_STRING> but should not be included around your consumer key and secret. Once you verify that those characters are not present in the yaml, report back with the current exception you're seeing.

thank you @fionapigott and @jrmontag,
I did what you wrote and I think it worked as shown here
  search_tweets_fullarchive_dev:
  account_type: premium
  endpoint: https://api.twitter.com/1.1/tweets/search/fullarchive/dev.json
  consumer_key: Szn************************************
  consumer_secret: lP8X************************************************

but I got another error
`Error parsing YAML file; searching for valid environment variables

TypeError                                 Traceback (most recent call last)
 in
1 premium_search_args = load_credentials("C:/Users/DRC/Desktop/twitter_keys.yaml",
2                                        yaml_key="search_tweets_fullarchive_dev",
----> 3                                        env_overwrite=False)
~\Anaconda3\lib\site-packages\searchtweets\credentials.py in load_credentials(filename, account_type, yaml_key, env_overwrite)
186     merged_vars = (merge_dicts(yaml_vars, env_vars)
187                    if env_overwrite
--> 188                    else merge_dicts(env_vars, yaml_vars))
189     parsed_vars = _parse_credentials(merged_vars, account_type=account_type)
190     return parsed_vars
~\Anaconda3\lib\site-packages\searchtweets\utils.py in merge_dicts(*dicts)
82         return merged
83
---> 84     return reduce(_merge_dicts, dicts)
85
86
~\Anaconda3\lib\site-packages\searchtweets\utils.py in _merge_dicts(dict1, dict2)
79     def _merge_dicts(dict1, dict2):
80         merged = dict1.copy()
---> 81         merged.update(dict2)
82         return merged
83
TypeError: 'NoneType' object is not iterable`
are this is environment variables?
export SEARCHTWEETS_ENDPOINT=
export SEARCHTWEETS_USERNAME=
export SEARCHTWEETS_PASSWORD=
export SEARCHTWEETS_BEARER_TOKEN=
export SEARCHTWEETS_ACCOUNT_TYPE=
export SEARCHTWEETS_CONSUMER_KEY=
export SEARCHTWEETS_CONSUMER_SECRET=

but where should I write them in YAML file or python file?
also, what are password and username? is it for my twitter account?

@fatimamb
  search_tweets_fullarchive_dev:
  account_type: premium
  endpoint: https://api.twitter.com/1.1/tweets/search/fullarchive/dev.json
  consumer_key: Szn************************************
  consumer_secret: lP8X************************************************

It looks like there's 2 spaces before the YAML key search_tweets_fullarchive_dev: - it's still valid YAML format, but it's not what the code expects.
it should be:
search_tweets_fullarchive_dev:
  account_type: premium
  endpoint: https://api.twitter.com/1.1/tweets/search/fullarchive/dev.json
  consumer_key: Szn************************************
  consumer_secret: lP8X************************************************

so that the key search_tweets_fullarchive_dev matches what the code is looking for yaml_key="search_tweets_fullarchive_dev"

@igorbrigadir thank you for replying
I did as you said and I got this message, is that means it works?
Grabbing bearer token from OAUTH

I saw my dashboard and I think it works, thank you for helping me all

Thanks for chiming in, @igorbrigadir. And I'm glad you got it working, @fatimamb.


No file paths or sourcemaps on failing tests?

I first submitted this to https://github.com/bySabi/karma-tap-pretty-reporter/issues/15#issuecomment-257114305
But it seems it's not their fault.
So, my failing tests come up with: "name of failure at undefined". I assume undefined should be a file path. But I can't get it to work.
Outputting result parameter in this.specFailure shows up as:
{ description: 'whaaat',
  success: false,
  log: [ '{\n  "operator": "fail"\n}' ],
  suite: [ '# timing test\n' ],
  time: 9 }

The guys from karma-tap-pretty-reporter pointed me to:
https://github.com/bySabi/karma-tap/blob/master/src/adapter.js#L32
However I am not into karma plugins so I'm just looking for a solution. Is there a config I'm missing?

@antitoxic it seems a source map problem. Could you isolate a failing example test with your settings.
Karma-tap is a parser on top of tap-parser, it only know about TAP Protocol.
This is probably not karma-tap issue but a  karma-webpack, tape and source maps
Anyway is you share a repo I can help you to found the problem/solution

I can't share a public repo. I will try to isolate a small demo and share the code.

Closed due inactivity. Feel free to open again is needed.


Add the ability to disable analytics with config option

This way it's easily disabled with an environment variable.

LGTM, thank you Grant 👍


Allow blacklisting srpms from source container images

OSBS-8749

Signed-off-by: Robert Cerven<EMAIL_ADDRESS>Maintainers will complete the following section:

[x] Commit messages are descriptive enough
[x] "Signed-off-by:" line is present in each commit
[x] Code coverage from testing does not decrease and new code is covered
[x] JSON/YAML configuration changes are updated in the relevant schema
[x] Changes to metadata also update the documentation for the metadata
[ ] Pull request includes link to an osbs-docs PR for user documentation updates
[x] New feature can be disabled from a configuration file

The CI failure is unrelated. See https://github.com/containerbuildsystem/osbs-client/pull/957

rn updated: Allow blacklisting srpms from source container images


./crawler: line 16: ./bin/swoole-cli: cannot execute binary file

When attempting a crawl on the siteone-crawler-v1.0.6-linux-x64 version, I get this error:
./crawler: line 16: ./bin/swoole-cli: cannot execute binary file

Hi @justinholtweb,
Thank you for issue report.
Could you please provide output of these commands executed from crawler root directory?

stat ./bin/swoole-cli
lsb_release -a
uname -a ?

stat ./bin/swoole-cli --> 16777234 975860688 -rwxr----x 1 jholt staff 0 68954472 "Dec 11 11:52:37 2023" "Apr 26 07:16:16 2023" "Dec 11 11:52:37 2023" "Apr 26 07:16:16 2023" 4096 134680 0 ./bin/swoole-cl
lsb_release -a --> zsh: command not found: lsb_release
uname -a --> Darwin MacBook-Pro.local 21.6.0 Darwin Kernel Version 21.6.0: Wed Oct  4 23:56:02 PDT 2023; root:xnu-8<IP_ADDRESS>4.15~1/RELEASE_ARM64_T6000 arm64

Aaaa, Thank you, that explains it.
You have to use build for macOS and also for arm64 architecture, not x64.
This is build for you: https://github.com/janreges/siteone-crawler/releases/download/v1.0.6/siteone-crawler-v1.0.6-macos-arm64.tar.gz
Btw, you can try this command-line version, of course, but I recommend you also desktop application for macOS, see https://crawler.siteone.io/getting-started/quick-start-guide/
In the following hours, I will also ensure that the application is signed with a Apple Developer certificate - until then, it is necessary to run the sudo xattr -cr /Applications/SiteOne\ Crawler.app command once between installation and the first start, as it is also written in the documentation.
Please let me know if it works for you.

Oh gosh sorry thought that was the one I was installing. In the meantime, I did get the desktop app working and running. Thank you.


translation with indentation

Hi,
I want to try translation between java and python. As current datasets from CodeXGLUE were representing functions in a single line, it is easy to finetune and test.
But what if I want to do this for python where indentation is very important? i.e, how will the tokenization take care of it?
I saw an example in TransCoder that they used below format for indentation using INDENT, DEDENT and NEWLINE -
def rm_file ( path ) : NEWLINE try : NEWLINE INDENT os . remove (path) NEWLINE print ( " Deleted " )
DEDENT except : NEWLINE INDENT print ( " Error _ while _ deleting _ file " , path ) DEDENT

Can you suggest how to proceed further with indentation using PLBART?

Figured out that above is possible by using code_tokenizer.py

Yes, we follow TransCoder, so during preprocessing, we take care of it.


Consistent ruff checks with imap_processing

Change Summary
Overview

This PR adds in the D and S ruff checks in attempt to be more consistent with the checks that happen over in the imap_processing repo. I also made the corresponding fixes to get the checks passing (mostly adding/modifying docstrings).
New Dependencies

Removed the black checks and dependencies, like we did in imap_processing

Updated Files

A whole bunch! The vast majority of the updated files just have edits to their docstring formatting (and not their content). I also added a bunch of new docstrings for the modules/classes/functions that didn't have them.

pre-commit.ci autofix


Top to bottom icon hiding the footer content

Pull Request Details
Description
Top to bottom icon hiding the footer content
Fixes
Fixes #950
Type of PR

[x] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Refactoring
[ ] Other (specify): _______________

Summary
Top to bottom icon hiding the footer content that i fixed and make rsponsive for all type device
Screenshots (if applicable)
[If your changes include UI updates, provide screenshots to illustrate the changes.]
Additional Notes
[Include any additional information or context that might be helpful for reviewers.]
Checklist

[x] I have read and followed the Pull Requests and Issues guidelines.
[ ] The code has been properly linted and formatted using npm run lint:fix and npm run format:fix.
[x] I have tested the changes thoroughly before submitting this pull request.
[ ] I have provided relevant issue numbers, snapshots, and videos after making the changes.
[ ] I have not borrowed code without disclosing it, if applicable.
[ ] This pull request is not a Work In Progress (WIP), and only completed and tested changes are included.
[ ] I have tested these changes locally.
[x] My code follows the project's style guidelines.
[ ] I have updated the documentation accordingly.
[ ] This PR has a corresponding issue in the issue tracker.

Summary by CodeRabbit

Style

Adjusted styling of the ScrollToTop component for improved height, width, text size, and margin.

Already done that changes in pr #956


Using Nvidia docker instead of reinventing it

I have the issue that the custom docker provided by the project is unable to fetch the device list for my GTX 1060 6G, and I check the the other docker for OpenCL, then I found this.
Nvidia provides the docker for OpenCL, we can just use that for supporting newer devices (GPUs).

Hi @seadog007, thanks for the PR! Do you mind updating this to nvidia/opencl:runtime-ubuntu18.04?

Codecov Report

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

@@            Coverage Diff             @@
##           master      #53      +/-   ##
==========================================
+ Coverage   22.86%   22.87%   +0.01%
==========================================
  Files          68       68
  Lines        3604     3602       -2
==========================================
  Hits          824      824
+ Misses       2712     2710       -2
  Partials       68       68

Impacted Files
Coverage Δ

worker/engines/hashcat/helpers.go
0% <0%> (ø)
:arrow_up:

server/web/server.go
20.72% <0%> (+0.18%)
:arrow_up:

Continue to review full report at Codecov.

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

OK, changed

Thanks @seadog007!


Typing experience is really bad when markdown preview is opened and word wrap is off

VSCode Version: Code 1.15.0 (8b95971d8cccd3afd86b35d4a0e098c189294ff2, 2017-08-09T20:09:15.006Z)
OS Version: Linux x64 4.10.0-26-generic
Extensions:

Extension
Author (truncated)
Version

EditorConfig
Edi
0.9.4

ActiveFileInStatusBar
Ros
1.0.3

lorem-ipsum
Tyr
1.0.0

sort-lines
Tyr
1.3.0

vscode-terminal-here
Tyr
0.0.5

vscode-svgviewer
css
1.4.2

vscode-eslint
dba
1.2.11

git-project-manager
fel
1.3.2

annotator
ryu
0.10.1

vscode-header-source
ryz
1.1.0

gitblame
wad
2.1.0

(2 theme extensions excluded)

Not sure if this is a regression or not but it happens in 1.15.0
Steps to Reproduce:

Open a markdown file
Open markdown preview to side
Make sure word wrap is off (I think this is the default)
Type on a long line

Reproduces without extensions: Yes

I'm not a fan of how it shifts the editor even when word wrap is on as well, makes me stop typing to find the cursor.

I'm unmarking this as a candidate fix for July recovery:

I don't think it is a regression.
To hit this, you have to explicitly turn off word wrap in markdown (disabling it normally keeps it enabled in markdown files).

@Tyriar I'm having trouble reproducing this. Can you test the latest insiders build and also share any other markdown settings you may have in your workspace

Still happens on latest insiders, on macOS too. My settings:
{
  "editor.autoClosingBrackets": false,
  "editor.codeLens": false,
  "editor.folding": false,
  "editor.fontFamily": "Hack",
  "editor.formatOnPaste": false,
  "editor.minimap.enabled": true,
  "editor.minimap.renderCharacters": false,
  "editor.renderControlCharacters": true,
  "editor.renderIndentGuides": true,
  "editor.renderWhitespace": "boundary",
  "editor.roundedSelection": false,
  "editor.rulers": [ 80, 100, 120 ],
  "files.useExperimentalFileWatcher": true,

  "explorer.autoReveal": false,
  "explorer.openEditors.visible": 0,

  "extensions.autoUpdate": true,
  "extensions.ignoreRecommendations": true,

  "files.defaultLanguage": "markdown",
  "files.hotExit": "onExitAndWindowClose",

  "terminal.integrated.cursorBlinking": true,
  "terminal.integrated.scrollback": 5000,
  "terminal.integrated.shell.windows": "C:\\Program Files\\Git\\bin\\bash.exe",

  "window.reopenFolders": "all",
  "window.title": "${rootPath}${separator}${activeEditorName}${separator}${appName}",

  "workbench.editor.enablePreviewFromQuickOpen": false,
  "workbench.welcome.enabled": false,
  "workbench.iconTheme": "seti",

  // Languages
  "typescript.check.tscVersion": false,
  "[markdown]": {
    "editor.quickSuggestions": false
  },

  // Extensions
  "gitProjectManager.baseProjectsFolders": [
    "$HOME/dev",
    "%USERPROFILE%\\Documents\\dev"
  ],
  "gitProjectManager.maxDepthRecursion": 2,
  "window.zoomLevel": 0,
  "workbench.colorTheme": "Sapphire"
}

Thanks @Tyriar. I can repo it now, but only reliably in the express readme https://raw.githubusercontent.com/expressjs/express/master/Readme.md
The bug seems to be an extra call to revealLine after we render the document. This event is trigged by scrolling in the markdown webview but we should disabled these events when we programmatically scroll the document or when we first load the document (which also produces a scroll event). It looks like it is not disabled in this case

Should be fixed by new markdown editor -> preview scroll sync added in 1.21

Still an issue:

VS Code version: Code - Insiders 1.21.0-insider (9a04587cc6b42c332609a08bfd7e2dbeac592c21, 2018-03-01T06:39:57.509Z)
OS version: Darwin x64 16.7.0

Should be fixed in the current insiders (tested with the long longs on the express release notes)


Load testing verifications

Review the endpoints exposed by VC-AuthN so that it is possible to load test the system by generating verification events without having to go through the UI/QR Code.

Let’s do this in stages, starting from easiest first. Here’s what I propose.  We'll should do the first one now driven by this issue, and based on the results, we’ll look at doing the other steps (2, 3 below) as new issues:

Copy the existing “Verify” script (locustMediatorPresentProof.py) to a new script (e.g., locustMediatorVerifier.py) and adjust it by moving the “get credential” step out of the loop and into the setup, so that we are only doing verifies in the loop. We’ll run that with Traction Dev and see how we do in the loop step.  The credential can be trivial for this — just the default one with the single attribute.  The goal is to see how many presentations the Traction Verifier can process per minute.

Notes:

Use the existing logic, but please look at the code and let us know if the process does a connection with the verifier, or if a connection-less presentation is done. Connection-less would be prefered, but for now, stick with the existing functionality.
We’ll run this with and without revocation in the source credential, to see if the revocation addition has an impact. AFAIK, there should be no difference in the script to add/remove verification. I did that in the issuer testing we did earlier without a change to the script.

The rest of this will be in different tickets if we decide we need to do them. Included here just to outline the direction.

Make a new “IAS” version of the script, where we replace the “generic issuer” get credential in the start up with the IAS “get person credential” processing.  We still would just do the get credential once, and loop on the verifications, but we would be using a much bigger credential.

We add a way to use in an instance of VC-Authn as the verifier. I think it can be done be deploying the version of VC-Authn app with the “deep link” on it, and in the loop “curl”-ing that URL, extracting the invitation from the deep-link URL (we don’t actually call the deep-link — we just extract the base64-encoded invitation and process it).  I think that should result in the verification being done.

For item 1 above:
Used the Sandbox env for these tests and created 2 tenants
Issuer
75c346aa-d0bc-492e-9664-f5e230fa8dc9
b1c14dd4-3b0e-4092-8a07-d06df2b26146
Copied in the schema used previously (LjFT93n4TcoUZZ8b5CcwMp:2:load_testing_prefs:1.0) and created a credential and a revocable credential from it.
Verifier
a14cd46e-0118-4b22-82ce-c1419c70a5ab
d674295f-b298-4c1f-9ccd-d7ddab6eeac3
Connection with the verifier?
Yes existing locustMediatorPresentProof scripts use a connection between the local agent and the Traction verifier and then that connection ID is used in the call to request verification.
The step that does the verification is invoking the Traction /present-proof/send-request endpoint with the connection ID
Akrida changes
Based locustMediatorVerifier.py off of locustMediatorPresentProof.py
but the Akrida code seems to have a bunch of copy/paste errors (I think?) where the issuer agent and config is being used for verifier stuff. So I've adjusted that locally and will make a PR.
In the script start up 1-time step does

Create invitation through issuer Traction agent
Accept that with local agent
Issue cred through Traction agent and receive it
Create invitation through verifier Traction agent
Accept that with local agent

Then the repeating presentation_exchange step invokes the verifier Tratction agent /present-proof/send-request
Sample run
Ran this through Sandbox, just did an arbitrary 1 minute manual timed start/stop (some of that time is taken up with startup connection/cred steps)
Did this twice, once for the non-revocable cred, once with the revocable.

So through Tenant UI can see (look at feb 27 around 3:30 and after) Issuer has 20 connections created and 10 Cred issued for LCpcJGQ3n3HKoYBbEHh6JL:3:CL:249336:default and 10 Cred issued for LCpcJGQ3n3HKoYBbEHh6JL:3:CL:249336:default_rev

LCpcJGQ3n3HKoYBbEHh6JL:3:CL:249336:default

LCpcJGQ3n3HKoYBbEHh6JL:3:CL:249336:default_rev  (also happened to land on 88 requests but that's mostly luck depending on when I hit stop)

For item 2 (use person credential):
As discussed just copied the IDIM person credential to one of my own and issue that from Sandbox issuer.
Make the same schema as the IDIM person cred, but on BCOVRIN-TEST
LCpcJGQ3n3HKoYBbEHh6JL:2:Person:1.0
Make 2 creds on Sandbox Issuer tenant
CRED_DEF=LCpcJGQ3n3HKoYBbEHh6JL:3:CL:405243:person
CRED_DEF=LCpcJGQ3n3HKoYBbEHh6JL:3:CL:405243:person_rev
Set CRED_ATTR to the values as shown:
[ { "name": "street_address", "value": "123 FAKE STREET" }, { "name": "country", "value": "CA" }, { "name": "locality", "value": "CITYTOWN" }, { "name": "birthdate_dateint", "value": "20011222" }, { "name": "given_names", "value": "TEST" }, { "name": "region", "value": "BC" }, { "name": "postal_code", "value": "V0H 1V0" }, { "name": "family_name", "value": "USER" }, { "name": "picture", "value": "data:image/jpeg;base64,BASE64GOESHERE...}, { "name": "expiry_date_dateint", "value": "" } ]
Used this image (in place of BASE64GOESHERE above):

Same pattern as above (start, time one minute, stop), do non-revocable, then revocable.
CRED_DEF=LCpcJGQ3n3HKoYBbEHh6JL:3:CL:405243:person

CRED_DEF=LCpcJGQ3n3HKoYBbEHh6JL:3:CL:405243:person_rev

A little slower on the average, but not orders of magnitude or anything, possibly just environmental. Can refine with less abitrary test runs later.

Trying to set up on Test. I’m using “stock” Akrida with the .env updated (I think) appropriately, and the new “Verifier” script added. None of the “load-agent” changes (locustAgent, agent.ts) from the IAS testing remain — is that right?
When I run it, the "on start” task goes fine (60 with 0 errors), all agents start successfully, but all of the verifications fail.  Really fast fails (so there is that), but they all fail (384 of them)
In the Locust UI I get: # lookup in project for bcovern CRED_ATTR='[{"mime-type": "text/plain","name": "score","value": "test"}]' SCHEMA="LjFT93n4TcoUZZ8b5CcwMp:2:load_testing_prefs:1.0" # anchor schema VERIFIED_TIMEOUT_SECONDS=60 # seconds for verified: true
In the docker compose log, I get a bunch of these — although not 384 of them:
load-agent-1  | (node:16) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 ProofStateChanged listeners added to [EventEmitter]. Use emitter.setMaxListeners() to increase limit
load-agent-1  | (Use `node --trace-warnings ...` to show where the warning was created)
load-agent-1  | DEBUG: Verifier Connection: {'_tags': {'state': 'initial', 'role': 'receiver', 'recipientKeyFingerprints': ['z6Mkfqh7zQKy6YuNDvbu5HEeNuaBpncATVknhdZhGqEcm7nS'], 'threadId': 'ef576547-03ad-408c-af21-dcccc8286ae8', 'invitationId': 'ef576547-03ad-408c-af21-dcccc8286ae8'}, 'metadata': {'_internal/legacyInvitation': {'legacyInvitationType': 'connections/1.x'}}, 'id': 'da37ff6b-dc88-467c-9d56-51faac3ab44d', 'createdAt': '2024-03-01T19:38:35.017Z', 'outOfBandInvitation': {'@type': 'https://didcomm.org/out-of-band/1.1/invitation', '@id': 'ef576547-03ad-408c-af21-dcccc8286ae8', 'label': 'Test', 'accept': ['didcomm/aip1', 'didcomm/aip2;env=rfc19'], 'handshake_protocols': ['https://didcomm.org/connections/1.0'], 'services': [{'id': '#inline', 'serviceEndpoint': 'https://traction-acapy-test.apps.silver.devops.gov.bc.ca', 'type': 'did-communication', 'recipientKeys': ['did:key:z6Mkfqh7zQKy6YuNDvbu5HEeNuaBpncATVknhdZhGqEcm7nS']}]}, 'role': 'receiver', 'state': 'prepare-response', 'autoAcceptConnection': True, 'reusable': False, 'updatedAt': '2024-03-01T19:38:35.022Z'}

Perhaps we can do a debug session later?

Issue above was resolved. Another test run with some questions (see #9 comments).
Not sure we want to bother, since the processing would be about the same, but for future consideration if we decide to do more:
I would suggest that we drop the verifier connection establishment in the "on_start", and instead use do a connection-less presentation request in the loop. That would be more like what would happen in, for example, a VC-Authn example.


feat: Add option to put the cursor at end of line

Fixes #15
Adds a simple option to switch the cursor position after the buffer-edit.
Feedback (if any) would be highly appreciated :)

Thank you Moritz! This is asuper helpful addition.