Intellisense not working with vue files

I made a new vite3/vue3/vuetify3/typescript project. Installed VS code and volar extension. All latest version. But when I edit the vue files, I don't get any intellisense on the typescript. How can I resolve it?

This also the same for me

Could you provide repro case?

I just ran npm init vue@latest and choosing typescript 'Yes', everything else 'No'.

<script setup lang="ts">
import { ref } from "vue";
import { group } from "src/...";

As I'm typing the file path I was expecting to see directory/file suggestions. Also no suggestions for relative paths.

@robmolloyntt please add "src/*": ["./src/*"], to compilerOptions.paths in tsconfig, or use relative path instead of.

I've just seen that I'm getting this eslint error displayed in vscode

apologies @johnsoncodehk  I should have said that I have added compilerOptions.paths as suggested but it hasn't worked

I get this
I get this

node:internal/bootstrap/switches/does_own_process_state:112
rawMethods.chdir(directory);
^
Error: ENOENT: no such file or directory, chdir 'c:\Users\user\Documents\GitHub\CompanyDocumentationViewer' -> 'c:\Users\user\source\repos\GlobalReports'
at process.wrappedChdir [as chdir] (node:internal/bootstrap/switches/does_own_process_state:112:14)
at Object.apply (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1292:20408)
at c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29857
at Generator.next ()
at c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29461
at new Promise ()
at Xa (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29258)
at owt (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29755)
at Object.add (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:34421)
at c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1292:17750 {
errno: -4058,
code: 'ENOENT',
syscall: 'chdir',
path: 'c:\Users\user\Documents\GitHub\CompanyDocumentationViewer',
dest: 'c:\Users\user\source\repos\GlobalReports'
}
[Info  - 3:26:55 PM] Connection to server got closed. Server will restart.
node:internal/bootstrap/switches/does_own_process_state:112
rawMethods.chdir(directory);
^
Error: ENOENT: no such file or directory, chdir 'c:\Users\user\Documents\GitHub\CompanyDocumentationViewer' -> 'c:\Users\user\source\repos\GlobalReports'
at process.wrappedChdir [as chdir] (node:internal/bootstrap/switches/does_own_process_state:112:14)
at Object.apply (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1292:20408)
at c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29857
at Generator.next ()
at c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29461
at new Promise ()
at Xa (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29258)
at owt (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29755)
at Object.add (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:34421)
at c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1292:17750 {
errno: -4058,
code: 'ENOENT',
syscall: 'chdir',
path: 'c:\Users\user\Documents\GitHub\CompanyDocumentationViewer',
dest: 'c:\Users\user\source\repos\GlobalReports'
}
[Info  - 3:27:11 PM] Connection to server got closed. Server will restart.
node:internal/bootstrap/switches/does_own_process_state:112
rawMethods.chdir(directory);
^
Error: ENOENT: no such file or directory, chdir 'c:\Users\user\Documents\GitHub\CompanyDocumentationViewer' -> 'c:\Users\user\source\repos\GlobalReports'
at process.wrappedChdir [as chdir] (node:internal/bootstrap/switches/does_own_process_state:112:14)
at Object.apply (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1292:20408)
at c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29857
at Generator.next ()
at c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29461
at new Promise ()
at Xa (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29258)
at owt (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29755)
at Object.add (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:34421)
at c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1292:17750 {
errno: -4058,
code: 'ENOENT',
syscall: 'chdir',
path: 'c:\Users\user\Documents\GitHub\CompanyDocumentationViewer',
dest: 'c:\Users\user\source\repos\GlobalReports'
}
[Info  - 3:27:24 PM] Connection to server got closed. Server will restart.
node:internal/bootstrap/switches/does_own_process_state:112
rawMethods.chdir(directory);
^
Error: ENOENT: no such file or directory, chdir 'c:\Users\user\Documents\GitHub\CompanyDocumentationViewer' -> 'c:\Users\user\source\repos\GlobalReports'
at process.wrappedChdir [as chdir] (node:internal/bootstrap/switches/does_own_process_state:112:14)
at Object.apply (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1292:20408)
at c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29857
at Generator.next ()
at c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29461
at new Promise ()
at Xa (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29258)
at owt (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29755)
at Object.add (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:34421)
at c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1292:17750 {
errno: -4058,
code: 'ENOENT',
syscall: 'chdir',
path: 'c:\Users\user\Documents\GitHub\CompanyDocumentationViewer',
dest: 'c:\Users\user\source\repos\GlobalReports'
}
[Info  - 3:27:36 PM] Connection to server got closed. Server will restart.
node:internal/bootstrap/switches/does_own_process_state:112
rawMethods.chdir(directory);
^
Error: ENOENT: no such file or directory, chdir 'c:\Users\user\Documents\GitHub\CompanyDocumentationViewer' -> 'c:\Users\user\source\repos\GlobalReports'
at process.wrappedChdir [as chdir] (node:internal/bootstrap/switches/does_own_process_state:112:14)
at Object.apply (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1292:20408)
at c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29857
at Generator.next ()
at c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29461
at new Promise ()
at Xa (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29258)
at owt (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:29755)
at Object.add (c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1291:34421)
at c:\Users\user.vscode\extensions\vue.volar-0.40.13-win32-x64\dist\node\server.js:1292:17750 {
errno: -4058,
code: 'ENOENT',
syscall: 'chdir',
path: 'c:\Users\user\Documents\GitHub\CompanyDocumentationViewer',
dest: 'c:\Users\user\source\repos\GlobalReports'
}
[Error - 3:27:44 PM] The Volar - Language Features Server server crashed 5 times in the last 3 minutes. The server will not be restarted. See the output for more information.

@TheInvoker please try v1.0.x, if still not working please provide minimal reproduction.
@Readpato you can try check tsconfig includes file use this plugin: https://marketplace.visualstudio.com/items?itemName=johnsoncodehk.vscode-tsconfig-helper

@Readpato you can try check tsconfig includes file use this plugin: https://marketplace.visualstudio.com/items?itemName=johnsoncodehk.vscode-tsconfig-helper

Thank you very much it solved my problem <3

@TheInvoker please update the project vue-tsc version, and pin @types/node version to 18.8.0.


add domain replication logic

pending unit test.
solve #569
close in favor of #586

Coverage decreased (-0.5%) to 66.965% when pulling 09255e5632d1c8ca03d052c79c7165ae5afbdf76 on r-domain into 2740e6b28a91b734e1c8bfd28426654eae8f3df9 on master.

close in favor of #586


How to apply pan/zoom to rotated X Axis labels?

I am an absolute newbie to D3
I am able to follow and use the recipe of Rotated axis labels in v4 ( https://bl.ocks.org/d3noob/3c040800ff6457717cca586ae9547dbf )
However; if I apply pan/zoom to such plot, all labels vanish and even ticks do not move with the zoom or pan.
However; if the axis labels are kept without any special formatting of text-anchor and rotation etc, the zoom behavior for the axis works just fine.
Please let me know ( or point me to any example about ) how to use pan / zoom with rotated or formatted x axis labels.
(I have created same issue in d3/zoom (https://github.com/d3/d3-zoom/issues/155))
Thanks in advance
-Mahesh

Please use Stack Overflow tag d3.js to ask for help. Stack Overflow provides a better collaborative forum: thousands of D3-related questions have been asked there, and some answers may be relevant to you.
When asking for help, please include a link to demonstrate the issue, preferably as an Observable notebook. It is often impossible to debug from code snippets alone. Isolate the issue and reduce your code as much as possible before asking for help. The less code you post, the easier it is for someone to debug, and the more likely you are to get a helpful response.
If you have a question about D3’s behavior and want to discuss it with other users, also consider the d3-js Google Group or joining the d3-js Slack.
Thank you! 🤗


Remove os.Std* from roxctl/central

Description
We have misconfigured linter so it shipped too much not only central but also roxctl/central.
This PR fixes some issues and enable linting on roxctl/central.
Testing Performed
CI

/retest


Outputs are missing @description

In the json files, all outputs had the metadata to include a description. Although bicep supports that with the @description tag, the decompiler ignored that. We will need to add them again plus update the readme accordingly. This can probably done in a scripted way

Note: After updating templates also corresponding readme files should be regenerated to include the description

Regex

^(?!@).*
output


Idea

Option to set how many tabs are open at once.
I don't know about you guys, but my tabs very quickly get filled, and are thus rendered useless.
I think it would be great to have an option to set only have two tabs open or 3, so you can switch back and forth quickly to those tabs without having to think about which tab you were last on and where it is.
90% of the time, I just want to use the last tab I had open, and I really cannot use the tabs at all, because there's so many, and I can't tell the names on them. If it automatically closed the 6th to the last tab you had open or so, you could only have the last 5 tabs you had open.

There is a package called Zen Tabs that seems like it can give you the feature you want. Have you tried that package?

There are many packages that handle this.  See
https://atom.io/search?utf8=✓&q=tabs
On Thu, Mar 5, 2015 at 7:37 PM, Tyler Mumford<EMAIL_ADDRESS>wrote:

There is a package called Zen Tabs https://atom.io/packages/zentabs
that seems like it can give you the feature you want. Have you tried that
package?
—
Reply to this email directly or view it on GitHub
https://github.com/atom/atom/issues/5879#issuecomment-77501434.

Fantastic. I should have known someone would have thought of this. Thanks


No Permission error while modifying quicklook.exe.config file

I tried editing the file but i couldn't save it. tried different ways such as editing it through vsc or even through terminal with administrative perms but no luck.
Anyone knows a fix?

Are you using the Microsoft Store version of QuickLook?

yes Indeed

Close as duplicate (#3).


python311Packages.es-client: 8.11.0 -> 8.12.3

Diff: https://github.com/untergeek/es_client/compare/refs/tags/v8.11.0...v8.12.3
Changelog: https://github.com/untergeek/es_client/releases/tag/v8.12.3
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 285620 run on x86_64-linux 1

4 packages built:

elasticsearch-curator
elasticsearch-curator.dist
python311Packages.es-client
python311Packages.es-client.dist


Can't successfully run seldon deployment with tensorflow image (wrong port)

Creating seldon deployment that is run using tensorflow/serving image.
Here is the yaml file of the job - https://pastebin.ubuntu.com/p/KpNyz8qDf9/
Essentially the job is instructing the manager to probe model container on port 9000 (which is not defined anywhere), while the container is listening on the port 8501 (API).
Consequently the manager never claims the model to be ready, though model container starts successfully.
Log output from the model container: https://pastebin.ubuntu.com/p/8PqmgQC3mV/
Pod description: https://pastebin.ubuntu.com/p/Qnmfj26Jmw/

@grebennikov: Looks like you'll need to specify the port that the tf serving container is listening to, in the SeldonDeployment yaml:
ports:
  - name: http
    containerPort: 8501

Seldon will then appropriately set up the engine to talk to that port, and it otherwise defaults to using port 9000 if no port in the container was specified.


cant import project

I add root build.gradle  maven { url 'https://jitpack.io' } but getting error:  "failed to resolve : compile 'com.github.jiang111:CProgressButton:1.2.7' "
how can i import my project?

com.github.jiang111:CProgressButton:v1.2.7     (version: v1.2.7)

Thanks.


Feature Parity with OpenLane 1

[x] Macro Flows
[x] LVS/Spice
[x] LEC
[x] Antenna Checker
[x] KLayout Streamout
[x] KLayout DRC
[ ] Padframe Gen
[ ] run_designs alternative
[ ] GF180 Support
[x] Docker support

New PRs

[ ] https://github.com/The-OpenROAD-Project/OpenLane/pull/1687
[ ] https://github.com/The-OpenROAD-Project/OpenLane/pull/1675
[x] https://github.com/The-OpenROAD-Project/OpenLane/pull/1715

Updated main comment with
Additional checks in QUIT_ON_SYNTH_CHECK￼
Hierarchical sta

GF180 support blocked on The-OpenROAD-Project/OpenROAD#3155


tfs content links here but repo is missing docs/server folder

If you create an issue from https://docs.microsoft.com/en-us/tfs/server/install/sql-server/use-always-on-groups?view=vsts it point to https://github.com/MicrosoftDocs/vsts-docs/blob/live/docs/server/install/sql-server/use-always-on-groups.md which doesn't exist here.
Is it just on the private repo for now, or a comment system bug?

@nschonni Let me look into this and see what is happening - some of the TFS docs are in a different place now (note the /tfs/ part of the URL that replaces the /vsts/ part that most of the other docs have (except release notes).


Bump racer to 2.0.9

Hi Joe,
First of all, thanks a lot for this project, it really helps me a lot in my daily development in Rust!
You know, the guys from racer have added some cool features to the recent versions, and one of them which I especially missed a lot is a try! macro support.
So I've bumped the racer version (which led to some other packages to be bumped as well) and it seems working to me quite well.
Could you please consider to merge the request?
Thanks!

hi @mexus racer has already bumped to 2.0.10. How about bumping the bump ;)

I've somehow totally missed this PR O_o I think it's not relevant anymore.


Create docker-prod.yml

Takes the Dockerfile and produces image

sorry folks added the wrong workflow, we need to push the image to the cloud :)


[FEATURE REQUEST] CORS as an option

Is your feature request related to a problem? Please describe.
CORS is hardcoded currently. It ill be good to have an option for it.
Describe the solution you'd like and provide pseudocode examples if you can
Edit *.json configuration file. Update Go's structs, add some conditions checks.
Describe alternatives you've considered and provide pseudocode examples if you can
nope
Additional context
nope

cloed by #10


Fixed missing file bug in a test case.

This issue was introduced in PR#102.

Execution is verified
09/14/2018 09:23:34 : [INFO ] ~~~~~~~~~~~~~~~TEST END : VERIFY-IFUP-IFDOW-ETH0-RMMOD-MODPROBE-HV_NETVSC-Synthetic~~~~~~~~~~
09/14/2018 09:23:34 : [INFO ] CURRENT - PASS    - 1
09/14/2018 09:23:34 : [INFO ] CURRENT - FAIL    - 0
09/14/2018 09:23:34 : [INFO ] CURRENT - ABORTED - 0
09/14/2018 09:23:34 : [INFO ] Checking background cleanup jobs..
09/14/2018 09:23:34 : [INFO ] Removed 0 context files.
09/14/2018 09:23:34 : [INFO ] Cycle Finished.. TC-92570
09/14/2018 09:23:34 : [INFO ]
Test Results Summary
ICA test run on 09/14/2018 09:17:43
ARM Image under test Canonical : UbuntuServer : 16.04-LTS : latest
Total Executed TestCases        1 (       1 Pass,        0 Fail,        0 Abort)
Total Execution Time(dd:hh:mm) 0:0:5
XML file: TestConfiguration

VERIFY-IFUP-IFDOW-ETH0-RMMOD-MODPROBE-HV_NETVSC-Synthetic Execution Time: 5.8 minutes
	VERIFY-IFUP-IFDOW-ETH0-RMMOD-MODPROBE-HV_NETVSC-Synthetic : PASS

Logs can be found at .\TestResults\2018-14-09-09-17-37-2992

Verified on -

Ubuntu 1604
RHEL 7.4

Unsupported -

Ubuntu 1804
Error: STDERR: verify-ifdown-ifup-eth0-rmmod-modprobe-hv-netvsc.sh: line 33: ifdown: command not found
SLES 15
Error: STDERR: rmmod: ERROR: Module hv_netvsc is builtin.


Helm Beta

Added the changes to deploy Network Service Mesh

Hi @Harshini-M, will you sign-off these commits by executing these steps? - https://github.com/layer5io/meshery-nsm/pull/26/checks?check_run_id=261948888
You can configure automatic sign-off on each of your commits using the instructions in the main meshery repo’s contributing.md.

@Harshini-M here's a link to sign off on this PR's commits - https://github.com/layer5io/meshery-nsm/pull/26/checks?check_run_id=261948888


Add missing locale strings to y18n lookup

:heavy_plus_sign: Further testing revealed the following strings used in index.js were missing from y18n lookup:
"Path to JSON config file": "Path to JSON config file",
"Show help": "Show help",
"Show version number": "Show version number"

This PR fixes that.
Note that the first commit takes the naive approach, which results in using the en version for 'Show help' in this scenario:
var argv = require('yargs').help('h').locale('pirate').argv

The second commit addresses this by deferring the y18n lookup until the whole usage content is generated, instead of doing it when the option is defined.
Also note that this PR does not attempt to address the problem of updating locale strings before the desired locale is set (if it even is a problem):
var argv = require('yargs')
  .help('h')
  .updateStrings({
    'Show help': 'Use this here help desc instead' // applies to default locale en
  })
  .locale('pirate') // changes locale and thus discards previously updated strings
  .argv

See related discussion in PR #220.

@bcoe Just so I'm clear, :shipit: = ok to merge?
Do you want me to attempt a release once the es.json and fr.json PRs are merged, or is that something I should leave to you? I assume the general steps are something like:

update CHANGELOG.md in master
npm version
git push && git push --tags
npm publish --tag next
test yargs@next
npm dist-tag add<EMAIL_ADDRESS>latest

:shipit: = ship it.
You've got the steps correct for the release, but let's wait for @kat to update the Spanish translation before we release. I talked to her today she's on it :)
You can push the next release once we get things merged :+1: -- I think I'll have time for some yargs hacking this weekend, perhaps we can close a few more issues.


rbenv shims path not set on Ubuntu 18.04

hi, few minutes ago i install rebenv with git and run rbenv doctor, all test passed except the shims path
I have follow instruction to run  echo 'eval "$(rbenv init -)"' >> ~/.bashrc  and restart the terminal but isn't fix the error
p.s: also installing goenv, phpenv, pyenv, and have no problem

Sorry you're having trouble! So far you're doing everything right; I'm not sure why the new line in your ~/.bashrc isn't sourced. You haven't given us any information on what part of rbenv isn't working for you.
Open your ~/.bashrc and check for the following lines:
export PATH="$HOME/.rbenv/bin:$PATH"
eval "$(rbenv init -)"

Good luck! I'm gonna close this in favor of the related discussion in https://github.com/rbenv/rbenv/issues/1130#issuecomment-446906104.


Possible improvements to extension docs

I tried making the extension docs a bit clearer.

Thanks for the interest, but I don't feel that this changes the clarity of those sections.


[BUG] Type for CellHyperlinkValue is wrong, sometimes has nested CellRichTextValue

🐛 Bug Report
Lib version: 4.2.0
Steps To Reproduce

Parse a xlsx file with lots of colours and hyperlinks.

The expected behaviour:
Cells including an hyperlink should follow the CellHyperlinkValue interface:
export interface CellHyperlinkValue {
	text: string;
	hyperlink: string;
}

The current behaviour:
Not all cells will match the given interface. We get cells nesting CellRichTextValue objects inside a linked cell:
{
  "text": {
    "richText": [
      {
        "font": {
          "underline": true,
          "size": 12,
          "color": {
            "indexed": 17
          },
          "name": "Calibri"
        },
        "text"<EMAIL_ADDRESS>      }
    ]
  },
  "hyperlink"<EMAIL_ADDRESS>}

Possible solution:
It might be enough to change the value for text on the CellHyperlinkValue interface. But I am not familiar enough with the innerworkings of ExcelJS to say wether this is enough:
export interface CellHyperlinkValue {
	text: string | CellRichTextValue;
	hyperlink: string;
}

It might need an even wider type for text? Maybe straight up CellValue if anything can be nested?

By coincidence, our project ran into the same issue just now, when we noticed .text doesn't return the textual representation of a link in cases where the cell included rich text formatting. Instead, the above occurs. Would be grateful if there were a dependable way to access the textual content of the cell that we wouldn't need to double check.

By coincidence, our project ran into the same issue just now, when we noticed .text doesn't return the textual representation of a link in cases where the cell included rich text formatting. Instead, the above occurs. Would be grateful if there were a dependable way to access the textual content of the cell that we wouldn't need to double check.

@ajuvonen Are there any test files and test code to reproduce this error?

The issue may be related to users who are saving .xlsx files with Apple Numbers, but I haven't been able to verify this.

Same issue here, I'm having to use cell.text.richText[0].text to get my value, and TypeScript isn't happy with it

@Wissam-Yelhow If I understand exceljs rich text correctly, the content of the cell may be split in multiple rich text formats, so until a fix exists, you should pull the values from all possible array items and join them, not just look at the text property in the first item. Not sure if there's any shortcut for this.

Thanks @ajuvonen , we're currently using:
case Exceljs.ValueType.RichText:     case Exceljs.ValueType.Hyperlink:       return (cellValue as Exceljs.CellRichTextValue).richText       .map(richText => richText.text)       .join(' ');

@skypesky:

Are there any test files and test code to reproduce this error?

I will try to create an XLSX file that I am able to share publicly that shows this problem. Probably later tonight (CET). I haven’t found a clearcut way of creating these types of cells myself, but I should be able to copy one from an existing workbook and anonymise it.
@Wissam-Yelhow:

I'm having to use cell.text.richText[0].text to get my value, and TypeScript isn't happy with it.

we're currently using:
return (cellValue as Exceljs.CellRichTextValue).richText.map(richText => richText.text).join(' ');

Be careful with your code there unless you are certain of the origin of your workbook! The same Excel file can contain hyperlink cells with and without nested richtext. If you cast all cell values as you are doing there you may end up casting actual string values to CellRichTextValue which will also break.
I like to take the defensive approach. I created a type alias with a fixed text and then another where I switch the old hyperlink type for the new one. This way TypeScript can continue to help me in writing code and making sure I cover all bases:
import { CellHyperlinkValue, CellRichTextValue, CellValue } from "exceljs";
// Dynamically modify the CellHyperlinkValue type to allow CellRichTextValue on `text`.
type DefensiveCellHyperlinkValue = {
    [key in keyof CellHyperlinkValue]: key extends "text" ? CellRichTextValue | string : CellHyperlinkValue[key];
};
// Swap the CellHyperlinkValue out for DefensiveCellHyperlinkValue.
type DefensiveCellValue = Exclude<CellValue, CellHyperlinkValue> | DefensiveCellHyperlinkValue;
// Use a function to recast into our defensive type.
const makeDefensivelyTyped = (cellValue: CellValue): DefensiveCellValue => cellValue;

With these I can cast a cell’s value and work on them knowing the hyperlink values are complete:
const cellValue = makeDefensivelyTyped(workbook.getWorksheet(1).getCell(1,1).value);
if (typeof cellValue === "object" && "hyperlink" in cellValue) {
    const text = cellValue.text; // Type here is now string | CellRichTextValue
    if (typeof text === "string") {
        return text;
    } else {
        return text.richText.map(({ text }) => text).join("");
    }
} else {
    // If the cell is not a hyperlink …
}

Not only can I hover the variable in VSCode to check the type every step of the way, in the else block after the typeof type narrowing it will also help me write the code as if the typings were always correct:

I’ve now published a full reproduction repo that shows this problem. It also includes an XLXS file that shows this behaviour, having both a field where text is a string and one where text is a CellRichTextValue.
Hope that makes it super clear :D

@Zegnat
I looked at your demo,This is indeed a problem.
As you think, I think the definition of cellhyperlinkvalue should be as follows:
export interface CellHyperlinkValue {
	text: string | CellRichTextValue;
	hyperlink: string;
}

Can you submit a PR for this?


Add Promise support

This patch introduces Promise support. Once merged, when a callback argument is not provided, a Promise will be returned.
This is a breaking change.
Closes #5.

Any updates here?

Hey fellas, it's super discouraging to have PRs sit untouched for multiple weeks. I understand that you're probably busy, but I think an update saying "I'm busy" is justified.

Apologies @stephenmathieson. Thanks for the PR!
I've made you an author of this package on npm and given you commit bit here on the repo. Have at it :dancer:

Awesome, thanks dude!

@jonschlinkert is there anything blocking this now? You've requested changes, but I believe all necessary updates have been made.

I'm rebasing the project for a migration and then I'll cherry-pick this into a 2.0

This has been published as 2.0.0 and node <4 support has been dropped unless Promise is polyfilled.  Thanks @stephenmathieson for all the work!


Button Assignment not working

Some controllers do permanently sent a button pressed command (e.g. Fanatec MCLaren GT3 Wheel). Crew Chief immediately recognizes this button and therefore no other button could be assigned e.g. for talk to crew chief.

This is not an issue with Crew Chief, you need to change configuration of your rim, see here for instruuctions;
https://forum.sector3studios.com/index.php?threads/mclaren-650s-gt3-steering-wheel.8455/page-4#post-149633


refactor: check gofmt error.

Codecov Report

Merging #54 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #54   +/-   ##
=======================================
  Coverage   76.63%   76.63%
=======================================
  Files           1        1
  Lines         184      184
=======================================
  Hits          141      141
  Misses         42       42
  Partials        1        1

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 a3ac7c5...70df3be. Read the comment docs.


Placeholder for refactoring proposal on reducing scope of Payment ExitGame

This doc was originally in confluence: here. Put an issue for it for public availability and easier reference with PR and issues.
Background
We start the audit of Payment ExitGame with the ability to support DEX feature in the future without the need to re-write the main code. As the time of this doc is written, the latest commit of plasma-contract is 493ddeadc821fae7293cfc3088939eddf89cb559. This is still with the ability to seamlessly support DEX feature extension. Here is a demo test of extension to DEX with existing Payment ExitGame implementation: https://github.com/omisego/plasma-contracts/blob/493ddeadc821fae7293cfc3088939eddf89cb559/plasma_framework/test/endToEndTests/Extendiblility.e2e.test.js#L35
During the start of the project, DEX was the next highest priority in the schedule. As a result, it makes sense to take DEX into consideration at that time. However, as priority switches as things changes. Also, the spec of the DEX is changed from Restricted Custody to Velocity which the detail of it is not fine tuned yet.
Current implementation takes quite a lot of complexity into consideration. One is that sadly the tx type dependency is not trivial to solve if we want the code base to be a more generalized solution which is ignorant of tx types and the protocol (MVP, MoreVP) that they are using. Also, some abstraction we introduced have apis or arguments only for MVP protocol, which is unclear when we would ever start to use that.
As auditors have pointed out, current code base have too many extra abstraction/complexity that does not seem straight forward to understand as there is no concrete example of using them as the first version of our launching scope would be without DEX feature. This is some extra unnecessary risks to bear for our first launch. Also, those feature would not be well tested as other feature would be now.
Following are some issue related:
https://github.com/omisego/plasma-contracts/issues/456
https://github.com/omisego/plasma-contracts/issues/457
https://github.com/omisego/plasma-contracts/issues/458
https://github.com/omisego/plasma-contracts/issues/471
https://github.com/omisego/plasma-contracts/issues/472
As a result, to be able to run security on the launch, and to not be burned by those not fully tested feature, we are considering to reduce the scope of Payment ExitGame V1 contract by removing the ability of seamlessly supporting DEX.
Some analysis (personal thought) among the issues
Issue 457: Recommendation: Remove optional arguments and clean unused code
https://github.com/omisego/plasma-contracts/issues/457
This one is probably less controversial. This recommends to remove those unused optional args from our exit game apis + remove the abstraction on outputGuard.
To remove the unused optional args, we can first hardcode those args with empty value to the interface we are relying.
We are only using outputGuard to hold the address of owner directly for our initial launch. However, we abstracted things to outputGuardHandler to be able to support multiple output types which have different outputGuard logics.
Currently the only real useful function of outputGuardHandler is getExitTarget(). The isValid function is really…doing nothing and the getConfirmSigAddress is obviously only from MVP protocol. We can remove the outputGuardRegistry and outputGaurdHandler and just get the owner out from the outputGuard field directly.
issue 456: Recommendation: Remove WireTransaction and PaymentOutputModel. Fold functionality into an extended PaymentTransactionModel
https://github.com/omisego/plasma-contracts/issues/456
So the concept of WireTransaction is not really well documented during the start of audit. And indeed it is a controversial abstraction. The idea is that we have assumption on what the data structure of transaction would be looking like, and we rely on that abstraction to get the field or data we are looking for when needed.
Although DEX is de-prioritized on the timeline, one of our next priority - Fee, would need this feature of easily accept input transaction with different tx types. We are planning to spend a fee transaction to a Payment transaction, and then we rely on the exit game of Payment transaction to exit the fee.
My personal proposal is to re-structure our code a bit. Instead of defining the struct in PaymentTransactionModel and PaymentOutputModel, define it in WireTransactionModel. PaymentTransactionModel would just reuse the struct defined in WireTransactionModel. However, PaymentTransactionModel would have the custom verification logic during decode (Potentially we should rename the contract to sth like PaymentTransactionDecoder ?)
So ExitGame code would be using struct from WireTransactionModel only. This is more explicit stating our assumption of different tx types would have same data structure.
issue 458: Recommendation: Remove multiple decoding steps and redundant tx model
https://github.com/omisego/plasma-contracts/issues/458
Part of the recommendation of removing WireTransaction would be handled using above proposal.
However, it does mentioned that we are duplicating our decoding even within same contract. Probably we should cache it on first decoding if it is not too hard to make the change. Cross contract (eg. spending condition uses bytes instead of struct) I would prefer to be more conservative and not make that change now.
issue 471: Recommendation: Remove TxFinalizationModel and TxFinalizationVerifier. Implement stronger checks in Merkle
https://github.com/omisego/plasma-contracts/issues/471
Again, the concept of tx finalization is not well documented during the start of audit. Also, it is something grown during the implementation phase. This is to abstract the complexity and have a more common phrase for both MVP and MoreVP protocol.
I would like to keep using the new phrases of protocol finalized and standard finalized. So we get used to the new definition of our more generalized protocol of M(ore)VP which can be used for both MVP and MoreVP.
However, to reduce the scope and simplify stuff, we can remove the TxFinalizationModel and ITxFinalizationVerifier. We give up on not re-write when introducing MVP tx type dependency. So we have only one TxFinalizationVerifier implementation and it would only take care of MoreVP protocol without the useless args for MVP protocol.
issue 472: Recommendation: Remove PaymentOutputToPaymentTxCondition and SpendingConditionRegistry
https://github.com/omisego/plasma-contracts/issues/472
I am strongly against making the change proposed here, unless we have better abstraction that can dispatch spending condition with types instead of branching inside the ExitGame controller directly.
Also, take into the fact that we already have 2 spending conditions: PaymentV1 → PaymentV1 and PaymentV1 → Payment V2. Also, the next which is coming soon is Fee → Payment V2. I see strong benefit of keeping this unless better abstraction idea is proposed.
However, it does let me realize some bug of current implementation. see: https://github.com/omisego/plasma-contracts/issues/473
Impact of the change
Need to re-write the Payment Exit Game contract when we want to integrate with DEX. However, we would have better understanding on the requirement of Velocity at that time. We can have the feature set just enough to support Velocity and Payment potentially.
Timeline wise, it would probably add some extra 2 weeks to refactor those.

closing this as the proposal has been implemented


Bug when using np.average without validation data

List of changes:

fixed a bug that occurred when not using TrainSplit(0), which was introduced in a recent PR
add test to catch this kind of error
add line breaks in base.by

Accidentally branched from wrong branch


custom config file removed after first use

Hi,
#Description of issue
I wanted to implement collectd input plugin for telegraf so I wrote custom file.
I took me a while to realize, that with custom_conf enabled I need full telegraf.conf file, with all options, but it's fine, perhaps worth to mention in docs.
Unfortunately found issue: after first use seems the file is moved from /share/telegraf.conf to /etc/telegraf/telegraf.conf. In result after hassio, plugin or docker restart it fails to start and complains for missing file, as below.
Logs
[21:37:07] INFO: Using custom conf file
mv: cannot stat '/share/telegraf.conf': No such file or directory

this relates to issue

0.5.4 should fix this :)


Menu unreadable when using a dark theme

I use the CDL feature of firefox to integrate with the gnome native look and feel with adwaita dark theme. Opening the sync-tab-groups menu display black text over dark background. Here is a sceenshot.

So this extension seems interesting as a tab-groups replacement now that it's gonna with firefox 57, but it's not usable as is for me.

Ok, I added a few lines to my ~/.mozilla/firefox/xxxxxxxx.default/chrome/userContent.css
/*
sources used to fill this file:
 https://userstyles.org/styles/90565/firefox-adjust-white-flash-when-opening-new-tab
 https://userstyles.org/styles/142191/remove-new-tab-flash
*/
@-moz-document url-prefix(about:preferences), url-prefix(about:blank), url-prefix(about:newtab) {
  html, body, #newtab-customize-overlay {
    background: #303030 !important;
    color: #b2b2b2 !important;
  }
}

@-moz-document url(chrome://browser/content/browser.xul)
{

  browser[type="content-primary"]
  {
    background: #303030 !important;
    color: #b2b2b2 !important;
  }
}

@-moz-document url(chrome://browser/content/browser.xul)
{

  browser[type="content-primary"], tabbrowser tabpanels, #appcontent > #content
  {
    background: #1B1B1B url("chrome://global/skin/media/imagedoc-darknoise.png") repeat fixed !important;
    color: #b2b2b2 !important;
  }
}

@-moz-document url(about:blank),
               url(about:preferences),
               url(about:config),
               url(about:newtab)
{
  #newtab-window,
  html,
  body,
  #newtab-customize-overlay
  {
    background: #1B1B1B url("chrome://global/skin/media/imagedoc-darknoise.png") repeat fixed;
    color: #b2b2b2 !important;
  }
}

html>body {
  background: #303030 !important;
  color: #b2b2b2 !important;
}

And tada:

Which is far better than what is rendered inside the Multi-account-container extension menu with the same configuration:

I hope this feedback will help you to improve the default rendering you provide.
Thank you @Morikko for your quick reaction in triage.

The application you are showing is not Sync Tab Groups. So the lines you added are for the other extension isn't it ?

Do you know other extensions that switch between dark and light theme ?
I am looking for examples.
Is it done automatically or do you have to set something is the preferences ?

I played a little with this behaviour. The Gtk theme appears to be applied automatically to all menus (from Firefox and Sync Tab Groups), regardless of the theme set in Firefox.
Here are three screenshots showing the effect with the light and dark versions of the Adwaita Gtk theme:

Adwaita:light Gtk theme, “Default” Firefox theme

Adwaita:dark Gtk theme, “Default” Firefox theme

Adwaita:dark Gtk theme, “Light” Firefox theme

It would be awesome to have Sync Tab Groups adapt to the Gtk theme, that is for sure. But maybe the first step could be to display the same menu colours regardless of the theme. This way the extension would be usable with any theme.
And by the way, thanks @Morikko for the good work! 🎉

But maybe the first step could be to display the same menu colors regardless of the theme. This way the extension would be usable with any theme.

This is the easiest and first step.

The second would be to detect the theme (dark or light) and render the extension in dark or light mode.

Later...

Thank you everybody for taking time and paying attention to this issue.


Destroy layer does not clear tile queue

Platform: Widows 8.0 IE10 Openlayers 2.13.1, using WMS layers
Steps to reproduce:
Have tiles in a tile queue
Call the destroy method on the layer
The tile queue will now have tiles in it with many properties being null
When one of these tiles are processed an exception occurs
I believe this is due to the order in which a layer is destroyed, the first step is to destroy tiles, which first nullifies the properties on the tiles, this also has the effect of nullifying the properties on the tiles in the tile queue.  The prelayerdestory event which is fired after the tile have been destroyed, triggers an event to try and clear the tile queue, but as the tiles now have their layer set to null, it is unable to
As a work around after destroying a layers I manually clear the queue:
var tileCount = this.map.tileManager.tileQueue[this.map.id].length - 1;
            for (var j = tileCount; j >= 0; j--) {
                var l = this.map.tileManager.tileQueue[this.map.id][j];
                if (!l.layer) {
                    this.map.tileManager.tileQueue[this.map.id].splice(j, 1);
                }
            }

I'm seeing this as well switching layouts via Fullscreen API. Monkey patching this with the follwing seems to work but is less than ideal.
drawTilesFromQueue: function(map) {
    var tileQueue = this.tileQueue[map.id];
    var limit = this.tilesPerFrame;
    var animating = map.zoomTween && map.zoomTween.playing;
    var tile;
    while (!animating && tileQueue.length && limit) {
        tile = tileQueue.shift();
        if (tile.layer) tile.draw(true);
        --limit;
    }
}


Alternate package repos

At this point I fear this PR is rather large, so quick pretext that I am totally fine if you don't want all (or any) of this to be merged.
Resolves #17
Some notable things this PR does:

Creates a config.yaml in the base .webman dir, which is used to control things like refresh interval and package repositories

By default the refresh interval is set to 6h (same as current), and a single package repo is written (webman-pkgs)
Recipes are moved one layer deeper and are identified by their name in the config.
e.g. the default lives in ~/.webman/recipes/webman now
When any commands are processed, they now look through package repos to find a match, meaning the first to match wins.

I ran gofumpt, which is why some of the formatting is changed. gofumpt is just a stricter gofmt, so they are compatible.
I removed the refresh files and am instead simply checking the directory modtime. Since we remove the entire dir, the modtime is effectively the same as the last time we "refreshed".

For me, personally, the reason for wanting this is because I have some homegrown projects that I would love to benefit from the UX simplicity of webman, but I'm not sure make sense to add to the central webman-pkgs repo.
As well, this allows me to potentially use this in a work setting where, again, the tools used may not make sense to add to the central location.

Currently there aren't any dedicated tests, so I will report back after using this for a while and let you know if I run into any hiccups. (And also when I find time I will try to write some tests)

I'm gonna take some time to review this later today. Exciting work!

Just for future reference, separating out a formatting-type change that touches lots of files into its own PR makes it a lot easier to review. Less noise :)

Yeah, that's my bad. I'm so used to using gofumpt that it's muscle memory, and I included it in the commit.
Will try to keep it in mind next time. 👍


Build for Android fails, conflicting assets messages

My Unity project build for Android target fails, running with 2019.4.22f1.
Any idea how I can start troubleshooting?
Thanks,
Vitaly
The only suspicious messages I see in the log are:
2022-02-23T10:13:32.2786155Z GUID [d742f2caefd9f934d9f19dad07a08e6f] for asset<EMAIL_ADDRESS>conflicts with: 2022-02-23T10:13:32.2787248Z   'Packages/com.unity.test-framework/UnityEditor.TestRunner/Api/ICallbacksHolder.cs.meta' 2022-02-23T10:13:32.2787797Z Assigning a new guid.
and later in the log
2022-02-23T10:19:33.5621563Z Assembly with name 'Unity.TextMeshPro.Editor.Tests' already exists<EMAIL_ADDRESS> 2022-02-23T10:19:33.5622137Z (Filename:  Line: 0)
and in the end
2022-02-23T10:19:34.5950490Z Scripts have compiler errors. 2022-02-23T10:19:34.5950831Z  #0 GetStacktrace(int) 2022-02-23T10:19:34.5951688Z  #1 DebugStringToFile(DebugStringToFileData const&) 2022-02-23T10:19:34.5952161Z  #2 CheckBatchModeErrorString(core::basic_string<char, core::StringStorageDefault<char> > const&) 2022-02-23T10:19:34.5952572Z  #3 Application::ParseARGVCommands() 2022-02-23T10:19:34.5953109Z  #4 Application::FinishLoadingProject() 2022-02-23T10:19:34.5953424Z  #5 InitializeUnity(void*) 2022-02-23T10:19:34.5953675Z  #6 g_source_get_time 2022-02-23T10:19:34.5953921Z  #7 g_main_context_dispatch 2022-02-23T10:19:34.5954174Z  #8 g_main_context_dispatch 2022-02-23T10:19:34.5954406Z  #9 g_main_loop_run 2022-02-23T10:19:34.5954625Z  #10 gtk_main 2022-02-23T10:19:34.5954833Z  #11 main 2022-02-23T10:19:34.5955066Z  #12 __libc_start_main 2022-02-23T10:19:34.5955286Z  #13 _start 2022-02-23T10:19:34.5955490Z   2022-02-23T10:19:34.5955749Z (Filename: ./Runtime/Utilities/Argv.cpp Line: 376)

Looks to me like https://github.com/game-ci/unity-builder/pull/308 is intended to fix this issue.
@keRin7 Is that right? Can you explain if there is a workaround?

https://github.com/game-ci/unity-builder/pull/348 (which superseded #308) is now merged.
@vitalyk-multinarity, could you try your workflow using unity-builder@main to see if that fixes your problem?

@webbertakken - many thanks!
Unfortunately, my build using unity-builder@main ('game-ci/unity-builder@main' (SHA:eb42a8c9b870c7449f64fe91e316eebc74ca570c)) failed as well.
I'll be glad to share our full log via PM or email.

Great, thanks for testing that out.
I'll close this issue, but always feel free to create another one if you're still having problems getting it to work.


Separate start and end depots

This has come up before (in #538, and originally in #305). Essentially, I propose we allow for the following:
m = Model()
veh_type = m.add_vehicle_type(
    ...,
    start_depot=idx1,
    end_depot=idx2,
)

This is a breaking change but it should make re-optimization a lot easier. Only question is how to deal with unused vehicles travelling between depots empty - do we count that towards the objective?

Only question is how to deal with unused vehicles travelling between depots empty - do we count that towards the objective?

TODO: how do other solvers do this? I am leaning no because such unused vehicles are not part of a solution.


Sir,I would like to know the meaning of each item in the CSV file, is there any relevant introduction?

See: https://github.com/StevenMHernandez/ESP32-CSI-Tool/blob/master/python_utils/parse_csi.py

What I mean by this is the meaning of these contents。I have found.

The link is https://github.com/espressif/esp-idf/blob/9d0ca60398481a44861542638cfdc1949bb6f312/components/esp_wifi/include/esp_wifi_types.h#L314

Thank you! SIr

Oh, I understand now. Typically you can ignore these items, they are for advanced use only. Unfortunately I do not have any additional documentation beyond the code that you found.
However, I believe if you use the information in the code-comments, you can learn much more on other websites. This code is from the espressif, so I did not create the attributes, but if there is specific ones that are confusing, I can try to help explain some.


Format: {{ CoinName }} | {{ Symbol }}
Nano | NANO |  *Coin symbol changed
MediShares | MDS
Time New Bank  | TNB
Theta Token | THETA
SuperNET | UNITY
BLOCKv | VEE
Dynamic Trading Rights | DTR
Asch | XAS
Telcoin | TEL
IoT Chain | ITC
Bluzelle | BLZ
DEW | DEW
SaluS | SLS
Ink | INK
SONM | SNM
QUANTUM RESISTANT LEDGER | QRL
CRYPTO20 | C20
DATA | DTA
BIBOX TOKEN | BIX
WINGS | WINGS
NULS | NULS
EIDOO | EDO
JIBREL NETWORK | JNT
AGORAS TOKENS | AGRS
SPANKCHAIN | SPANK
ION | ION
WAGERR | WGR
WEPOWER | WPR
COBINHOOD | COB
XTRABYTES | XBY
HTMLCOIN | HTML
CYBERMILES | CMT
BREAD | BRD
UTRUST | UTK

If I can also make a suggestion - since you're signing up for a nearly endless amount of work with shitcoins coming out daily and dropping in price wildly, keeping up with the cmc 1 - 200 could be a full-time job.... What if you made a fallback icon, so devs could use a generic image if a particular shitcoin doesn't yet have a logo. It would make replacing the icons a lot less urgent if a placeholder was available.
Anyways, thanks for all your hard work, this repo is amazing!

Just updated the xrb > nano issue. Will need to set aside time to conquer all these.
The idea of a default coin is a great idea indeed. 👍

Took me a while, but with the exception of C20 and the placeholder, they're done. https://github.com/cjdowner/cryptocurrency-icons/commit/2c5fe1715311c6306e5985025fae78c674abfde7

Thank you for your support. If you’d like to tip me for my time in maintaining and updating this project, it would be very much appreciated.


Remove staging directory before populate it again

without this, the directory keeps accumulating the old files when user never call 'clean'

preClean configuration is added with default is off

fixed at https://github.com/mojohaus/appassembler/commit/d4cf5f278e95af4f46fa83fcbd3136757935d0ce


Use less bits for ABA protection in CAS loops

The ABA bits are now the 5 (POOL_MIN_BITS) lowest bits of the pointers. I used a fixed number to avoid runtime computations, even though more bits are available in pool_push and pool_pull as the pool index goes up. I think 5 bits should be sufficient in every case.
I benchmarked this change on x86_64, the benchmark program can be found here (it uses the benchmark lib). The old version takes ~50ns per iteration while the new one takes ~30ns per iteration. I don't know how it would perform on ARM or other platforms but I expect similar ratios.

The AppVeyor failure seems unrelated.

@Praetonus i restarted the Appveyor "just in case"


audit: Fix python bindings

Motivation for this change
Fixes the python bindings for libaudit, which currently don't build to a missing dependency on swig, and a hardcoded path in upstream source to /usr/include/linux/audit.h.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 123342 at ce801ab7 run on aarch64-linux 1

891 packages marked as broken and skipped:

OVMF-CSM
adoptopenjdk-hotspot-bin-13
adoptopenjdk-hotspot-bin-14
adoptopenjdk-jre-hotspot-bin-13
adoptopenjdk-jre-hotspot-bin-14
agdaPackages.iowa-stdlib
aldor
ams-lv2
amuleDaemon
aqemu
...

15725 packages skipped due to time constraints:

AusweisApp2
CHOWTapeModel
DisnixWebService
EBTKS
EmptyEpsilon
Fabric (python38Packages.Fabric)
MIDIVisualizer
MMA
OVMF
OVMF-secureBoot
...

7 packages built successfully:

audit
cryptsetup
libfido2
libusb1
lvm2
openssh
systemdMinimal

2 suggestions:

warning: unclear-gpl
gpl2 is a deprecated license, please check if project uses gpl2Plus or gpl2Only and change meta.license accordingly.
Near pkgs/os-specific/linux/audit/default.nix:74:5:
   |
74 |     license = lib.licenses.gpl2;
   |     ^

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/os-specific/linux/audit/default.nix:76:5:
   |
76 |     maintainers = with lib.maintainers; [ ];
   |     ^

/rebase staging

rebased (sorry for all the noise, should have changed the base branch before pushing the rebase ^^')

The diff looks pretty good, if you want to fix the conflicts, I can get this through

I rebased, and it builds locally. Not sure what's up with ofborg's failure?

python2 was deprecated, I pushed a fix

Building git is failing on staging.

@GrahamcOfBorg eval

This PR broke build of pkgsStatic.audit with
configure: error: no suitable Python interpreter found

and consequently other static packages like nixStatic.

python3 should probably be in nativeBuildInputs?

For reference, mitigated in PR #164622.
(I'm not sure why GitHub doesn't show a back-reference here.)


Implementation Plan: Rekognition Data Evaluation

For #393 we need a quick implementation plan which outlines how we will determine to use the Rekognition data. There are three components to this project:

Determine which tags to reject
Determine which accuracy threshold to use
List and evaluate all tags generated by Rekognition

Closing this as it's been superseded by #431


twitter video

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2019.03.01. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x ] I've verified and I assure that I'm running youtube-dl 2019.03.01

Before submitting an issue make sure you have:

[x ] At least skimmed through the README, most notably the FAQ and BUGS sections
[ x] Searched the bugtracker for similar issues including closed ones
[ x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
PS C:\Users\xxx\Downloads\yt> .\youtube-dl.exe --verbose "https://twitter.com/realDonaldTrump/status/1102020668135272454"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://twitter.com/realDonaldTrump/status/1102020668135272454']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2019.03.01
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.17763
[debug] exe versions: ffmpeg 4.1, ffprobe 4.0.2, phantomjs 2.1.1
[debug] Proxy map: {}
[twitter]<PHONE_NUMBER>135272454: Downloading webpage
[twitter:card]<PHONE_NUMBER>135272454: Downloading webpage
[twitter:card]<PHONE_NUMBER>135272454: Downloading guest token
[twitter:card]<PHONE_NUMBER>135272454: Downloading JSON metadata
ERROR: Unable to download JSON metadata: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmplyslo9xs\build\youtube_dl\extractor\common.py", line 605, in _request_webpage
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmplyslo9xs\build\youtube_dl\YoutubeDL.py", line 2225, in urlopen
  File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
  File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
  File "C:\Python\Python34\lib\urllib\request.py", line 508, in error
  File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
  File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_default
...
<end of log>

I received a request to delete this issue at the request of the author, so I'll proceed. It may be related to  issue 17820.


Groups version bump

Bump the backup version and force a full backup if
there's a backup for teams/groups that has base(s)
from an older version of corso
This will avoid propagating older details formats
forward. Those formats don't have all the data
newer formats do
This is mostly a stop-gap, a more robust solution
can be added later
Manually tested that it forces a full backup

Does this PR need a docs update or release note?

[ ] :white_check_mark: Yes, it's included
[ ] :clock1: Yes, but in a later PR
[x] :no_entry: No

Type of change

[ ] :sunflower: Feature
[x] :bug: Bugfix
[ ] :world_map: Documentation
[ ] :robot: Supportability/Tests
[ ] :computer: CI/Deployment
[ ] :broom: Tech Debt/Cleanup

Test Plan

[x] :muscle: Manual
[ ] :zap: Unit test
[ ] :green_heart: E2E

ok, thought up one more case for this: we need to add a check on the version of the assist bases and drop those separately since it's possible to have assist bases without a merge base. The other half of the reason for this check is that assist bases also source backup details and we don't want to pull forward older backup details versions for teams


Run 0.9 Version

I've tried to run UI mode with command ./map-matching.sh action=start-server , but I got error :
using java
2017-11-15 17:46:54,345 [main] INFO  com.graphhopper.http.CmdArgsModule - jsonp disabled
Exception in thread "main" java.lang.IllegalStateException: Cannot load properties to fetch EncodingManager configuration at: ./graph-cache/
	at com.graphhopper.routing.util.EncodingManager.create(EncodingManager.java:155)
	at com.graphhopper.GraphHopper.load(GraphHopper.java:725)
	at com.graphhopper.GraphHopper.importOrLoad(GraphHopper.java:617)
	at com.graphhopper.http.GraphHopperModule$3.start(GraphHopperModule.java:182)
	at com.graphhopper.http.GHServer.start(GHServer.java:118)
	at com.graphhopper.http.GHServer.start(GHServer.java:62)
	at com.graphhopper.matching.http.MatchServer.main(MatchServer.java:36)

I used version 0.9, maven version 3.3.9 and java openjdk version "1.8.0_151". Did I miss something during instalation ?

Please follow the instructions like that an import is necessary before and questions should go first to our forum, then if this turns out to be a bug they'll be reported as issue


ARROW-15297: [C++] The write node options shouldn't require a schema [WIP]

The Execution node for write operation doesn't require the schema, the schema can be obtained from the
input node. The required modification is done.
TODO

[ ] Verify the output readability

[ ]


@LordRalex ??

Please use the community forums for issues, questions, or help with installation or configuration of PufferPanel.
This issue tracker is only for confirmed bug reports. Please consider opening a thread on the community forums before opening an issue here.

@LordRalex Okey, i'll use it


Add separate package with debug symbols.

Checklist

[x ] Used a personal fork of the feedstock to propose changes
[x ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x ] Ensured the license file is being packaged.

These changes add a separate "dbg" package with debug symbols.

@conda-forge-admin, please rerender


Support Elliptic curve signatures in JWT module

Currently
https://github.com/GoogleCloudPlatform/google-auth-library-python/blob/master/google/auth/crypt.py
Only supports RSA
It would be great to add support for ES* such as ES256 signing algorithms as well

@ptone Can you point to some docs for Google Auth services that accept elliptic curve signed JWTs?

@dhermes confirmed internally that a new (unannounced) service will support these. This isn't critical priority, but we should do this.

Cloud IAP provides JWT signed with ES256 only.


python controller ble scan returns NoneType

chip-device-ctrl > ble-scan -t 120 CHIP-3840
2021-02-25 17:51:01,595 ChipBLEMgr   INFO     use default adapter
2021-02-25 17:51:01,840 ChipBLEMgr   INFO     scanning started
2021-02-25 17:51:02,591 ChipBLEMgr   INFO     Name            = Family Room TV
2021-02-25 17:51:02,591 ChipBLEMgr   INFO     ID              = ad13e92b-b0fa-39ca-b3aa-64c01ccb39cc
2021-02-25 17:51:02,593 ChipBLEMgr   INFO     RSSI            = -88
2021-02-25 17:51:02,594 ChipBLEMgr   INFO     Address         = 5F:33:85:F6:6C:DC
2021-02-25 17:51:02,599 ChipBLEMgr   INFO     Adv UUID        = 0000fea0-0000-1000-8000-00805f9b34fb
2021-02-25 17:51:02,601 ChipBLEMgr   INFO     Adv Data        = 03fa8fca59300e6d2020203fff
2021-02-25 17:51:02,601 ChipBLEMgr   INFO
Traceback (most recent call last):
File "/home/ubuntu/.local/lib/python3.8/site-packages/chip/ChipBluezMgr.py", line 896, in running_thread
target(**kwargs)
File "/home/ubuntu/.local/lib/python3.8/site-packages/chip/ChipBluezMgr.py", line 957, in scan_bg_implementation
scanned_peripheral_list = self.adapter.find_devices(
File "/home/ubuntu/.local/lib/python3.8/site-packages/chip/ChipBluezMgr.py", line 280, in find_devices
for i in device.uuids:
TypeError: 'NoneType' object is not iterable

Is this resolved? Please reopen if not.


Refresh DoH stamps and minisign key

Quad9 had updated the TLS certificate which is now signed by a new intermediary. Since the fingerprint for the certificate has changed, we need to update the stamps to reflect the change.
While at it, we also rotated the minisign key and signed all the files using the new key.

Thank you!

TLS hashes don't seem to be correct.
I think it should be b01920744bbb76c9ab053e01e07b7e050e473d20f79f7bea435fafe43c9d242f, not 2a15f5d6acb6e7c0901ade4ebbc743b2ccd489032b46e1642f0693683001258a:
[2024-08-07 12:59:39] [NOTICE] Advertised cert: [CN=dns.quad9.net,O=Quad9,L=Zurich,ST=Zurich,C=CH] [e3e13daef1fd3012db80b3b002b5d2a7f24a7c8bb82b318694bdcaf061d1ba02]
[2024-08-07 12:59:39] [NOTICE] Advertised cert: [CN=DigiCert Global G3 TLS ECC SHA384 2020 CA1,O=DigiCert Inc,C=US] [b01920744bbb76c9ab053e01e07b7e050e473d20f79f7bea435fafe43c9d242f]
[2024-08-07 12:59:39] [CRITICAL] [quad9-doh-ip4-port443-filter-pri] Certificate hash [2a15f5d6acb6e7c0901ade4ebbc743b2ccd489032b46e1642f0693683001258a] not found

Ah, actually the issue seems to be that quad9-resolvers-doh.md was updated, but not quad9-resolvers.md that encompasses both DNSCrypt and DoH.

So DOH dont work ? ive got the same error every time.
[CRITICAL] [quad9-doh-ip6-port5053-filter-alt2] Certificate hash [2a15f5d6acb6e7c0901ade4ebbc743b2ccd489032b46e1642f0693683001258a] not found

This commit should have addressed the issue: https://github.com/Quad9DNS/dnscrypt-settings/commit/5da49928f4a734e9d3a259b904e36b5260840c38

Yep, looks good!
Thank you!


Adding more events

It might be helpful to add some event emits to the plugin around the beforeSave and beforeMarshal so that userland devs can hook the data being passed around the plugin.
Such a use-case might be wanting to maintain the users uploaded file between form submissions if the form fails submission the user should not have to resubmit their file.
Need to consider the impact of allowing this level of control, as it could create more bugs with people changing data in unintended ways.

A use-case for a new event would be renaming the file before it's written to the filesystem


Module without exports and with "@module" not included

Search terms
@module exports side effects
Expected Behavior
If a module has a doc string at the top with @module it should be included even if it doesn't have any exports. This allows to document modules that are only imported for their side effects, like polyfills.
Actual Behavior
The module is omitted in the docs.
Steps to reproduce the bug
/**
 * This module has side-effects only.
 *
 * @module
 */

(<any>globalThis).sideEffect = 42;

Environment

Typedoc version: 0.20.33
TypeScript version: 4.2.3

There's a couple things here.

A request to revert #1126, which with 0.20's behavior, I'm happy to do.
A bug, where TypeDoc doesn't pick up module comments from global files


Update utils

small pipeline for development uploads to render:

fix file permissions (posix only)
fix image/mask Urls (in robocopy destination)
upload to render

@jaybo I recall getting render to work on windows, we had to set validate=False somewhere. Let's try that again today and track down where that needs to go.
@RussTorres this is mainly for development. But, will start to give you an idea of the files we're generating that I'd like to move into the workflow. Also, comments welcome.

Hello @djkapner! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file on_scope/utils/set_permissions.py:

Line 36:33: W605 invalid escape sequence '('
Line 40:24: W605 invalid escape sequence '*'
Line 41:13: W605 invalid escape sequence ')'

In the file on_scope/utils/set_update_upload.py:

Line 10:80: E501 line too long (102 > 79 characters)
Line 11:80: E501 line too long (117 > 79 characters)
Line 22:80: E501 line too long (120 > 79 characters)

In the file on_scope/utils/update_urls.py:

Line 12:80: E501 line too long (119 > 79 characters)

In the file on_scope/utils/upload_to_render.py:

Line 13:80: E501 line too long (120 > 79 characters)
Line 17:80: E501 line too long (104 > 79 characters)
Line 18:80: E501 line too long (118 > 79 characters)

Codecov Report

Merging #28 into develop will increase coverage by 4.16%.
The diff coverage is 16.77%.

@@             Coverage Diff             @@
##           develop      #28      +/-   ##
===========================================
+ Coverage    76.31%   80.47%   +4.16%
===========================================
  Files           14       14
  Lines         1068     1045      -23
===========================================
+ Hits           815      841      +26
+ Misses         253      204      -49

Impacted Files
Coverage Δ

on_scope/utils/set_update_upload.py
0% <0%> (ø)

on_scope/utils/upload_to_render.py
0% <0%> (ø)

on_scope/utils/set_permissions.py
0% <0%> (ø)

on_scope/utils/update_urls.py
0% <0%> (ø)

on_scope/utils/schemas.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Testing Results are very low

Hi, I manage to train PTv3 on a custom dataset, and got pretty good results for validation, over 80% iou for most classes. However, when I test using the test script, the least sampled class drops iou to 9%, while the accuracy is over 80% I do want to mention that all other classes achieve decent, normal results. with all metrics around 80%. Is it because of the interpolation or the chosen grid_size is too big ? I decided to go with this grid_size because I have a very high dense point cloud, about 37 points / m^2 and each pc has around 90k points.
I am thinking that the model predicts to many false positives for that specific class, and when it gets to interpolation, those values are mutlplying themselves. Any other ideas , why this might happen ? Any help is highly appreciated.
This is my config for the test set.
        transform=[
            dict(type='CenterShift', apply_z=True),
            dict(type='Copy', keys_dict=dict(segment='origin_segment')),
            dict(
                type='GridSample',
                grid_size=0.25,
                hash_type='fnv',
                mode='train',
                keys=('coord', 'strength', 'segment'),
                return_inverse=True),
            dict(type='CenterShift', apply_z=False),
        ],
        test_mode=True,
        test_cfg=dict(
            voxelize=dict(
                type='GridSample',
                grid_size=0.25,
                hash_type='fnv',
                mode='test',
                return_grid_coord=True,
                keys=('coord', 'strength')),
            crop=None,
            post_transform=[
                dict(type='ToTensor'),
                dict(
                    type='Collect',
                    keys=('coord', 'grid_coord', 'index'),
                    feat_keys=('coord', 'strength'))
            ],
            aug_transform=[[{
                'type': 'RandomRotateTargetAngle',
                'angle': [0],
                'axis': 'z',
                'center': [0, 0, 0],
                'p': 1
            }]]),

From past issue like this #164 , I do understand that validation results are not that accuracte. However I dont think it justifies this sudden drop.
Update: I found issue https://github.com/Pointcept/PointTransformerV3/issues/9 . I will check now the inconsistencies in the trasnformation pipeline between training vlaidation and testing.

Update: I solved the issue. Got a 85% mIoU, and all classes have normal-looking metrics I removed the GridSample from transform in the test dict and made sure that test pipeline matches with train pipeline. This is my final config.
transform=[
            dict(type='CenterShift', apply_z=True)
        ],
        test_mode=True,
        test_cfg=dict(
            voxelize=dict(
                type='GridSample',
                grid_size=0.25,
                hash_type='fnv',
                mode='test',
                return_grid_coord=True,
                keys=('coord', 'strength')),
            crop=None,
            post_transform=[
                dict(type='ToTensor'),
                dict(
                    type='Collect',
                    keys=('coord', 'grid_coord', 'index'),
                    feat_keys=('coord', 'strength'))
            ],
            aug_transform=[[{
                'type': 'RandomRotateTargetAngle',
                'angle': [0],
                'axis': 'z',
                'center': [0, 0, 0],
                'p': 1
            }]]),

It's good to see the issue is already solved. Sorry for the late response.


Theme Support: Enfold

I'm attempting to use this plugin with the Enfold theme. Unfortunately they use the wp_get_attachment_image_src method along with the get_posts method to get the image attachment posts for that post/page and then further get the image src with an associated ID, which, if the ID doesn't exist (or is not the correct ID because it's associating locally) on the sub-site it can't get the correct post-type for the image associated with the native post/page.
I've tried to hack the theme by determining if this plugin is active, and if so switch the context to the site which has the global media library. That works, but then images that are uploaded to the non-global media library site end up not working after. There's no way to determine if an attachment image is local or global.

Any news on this? Did you find a workaround?


Build Cache docs recommend that periodic cleanup configuration be done globally in init script

The docs currently point out that the clean-up affects all projects. The docs will now recommend to configure how often the clean-up runs at the global level. Screenshot of how it looks like now below:

@bot-gradle test and merge

OK, I've already triggered a build for you.


base path missing in v2/api-docs

springfox-boot-starter 3.0.0
@EnableSwagger2
application.properties: server.servlet.context-path= /value
base path is missing in v2/api-docs :

when server.servlet.context-path not appear base path appear with "/"

Hi, we migrated from springfox-swagger2 and springfox-swagger-ui to springfox-boot-starter:3.0.0, but I noticed the basePath which was shown in  springfox-swagger2, is not present in springfox-boot-starter:3.0.0 api definitions (v2 definitions).
Is there any update on this?


Declare identifiers in linkml model

We should make sure that identifers like mapping_set_id or mapping_registry_id are declared as LinkML identifiers.
Originally posted by @matentzn in https://github.com/mapping-commons/sssom/pull/159#discussion_r852281360

but only in the owning class of course, not when they are foreign keys


Adding synchronous methods to create a store

Fixes #188

Seb this introduces a breaking changes, so can bump the version?

I know, but no ;)

As you wish ;)


README.md Wrong

In the README.md to download I used:
defp deps do
  [ { :geolix, "~> 0.10" } ]
end

And followed the documentation in the README I could not start the Database.Loader correctly. After exploring my deps and the current source code it seems that the README should work for the 0.11.0 code, whereas it does not work for ~> 0.10. Maybe updating the version should be the same version in the mix.exs file?

You are probably looking for the README of the tag you are using (GitHub and HexDocs).
The dependency statement is valid for real releases and not the 0.11.0-dev only available via GitHub. Rest assured you are not the only one tripping over that :D
Could you think of some statement I could include to mention that difference? Might help to prevent the confusion...

Yeah I was able to find the correct docs in the README found from the hex package under my deps directory, and from there everything worked great, so thanks for building a solid working package!
As far as including a statement, I am really bad at writing these types of things but maybe in include something along the lines of:
Documentation below is for the 0.11.0-dev release. Please see <insert some link or instructions here> for documentation for ~> 0.10 releases.

By the way, I am very excited about 0.11 after exploring the source code and docs for both versions!  Cannot wait for the release, and thank you for working on this package!

I tried to word it in a way that someone reading it on Hex is not confused and I don't have to modify it for upcoming versions. Should be good enough for now. And as a new release is around the corner the current problem will vanish for some time anyway :D


Uncaught SyntaxError: Use of const in strict mode since v3.8.0

Version
3.8.0
Environment info
  System:
    OS: macOS Sierra 10.12.6
    CPU: (4) x64 Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz
  Binaries:
    Node: 11.10.1 - /usr/local/bin/node
    Yarn: 1.9.4 - /usr/local/bin/yarn
    npm: 6.9.0 - /usr/local/bin/npm
  Browsers:
    Chrome: 74.0.3729.169
    Firefox: Not Found
    Safari: 12.1.1
  npmPackages:
    @vue/babel-helper-vue-jsx-merge-props:  1.0.0
    @vue/babel-plugin-transform-vue-jsx:  1.0.0
    @vue/babel-preset-app:  3.8.0
    @vue/babel-preset-jsx:  1.0.0
    @vue/babel-sugar-functional-vue:  1.0.0
    @vue/babel-sugar-inject-h:  1.0.0
    @vue/babel-sugar-v-model:  1.0.0
    @vue/babel-sugar-v-on:  1.0.0
    @vue/cli-overlay:  3.8.0
    @vue/cli-plugin-babel: ^3.0.3 => 3.8.0
    @vue/cli-plugin-eslint: ^3.0.3 => 3.8.0
    @vue/cli-service: 3.8.0 => 3.8.0
    @vue/cli-shared-utils:  3.8.0
    @vue/component-compiler-utils:  2.6.0
    @vue/preload-webpack-plugin:  1.1.0
    @vue/web-component-wrapper:  1.2.0
    eslint-plugin-vue:  4.7.1
    vue: ^2.5.17 => 2.6.10
    vue-cli-plugin-cordova: ^2.0.3 => 2.1.3
    vue-eslint-parser:  2.0.3
    vue-hot-reload-api:  2.3.3
    vue-lazyload:  1.2.3
    vue-loader:  15.7.0
    vue-router: ^3.0.1 => 3.0.6
    vue-style-loader:  4.1.2
    vue-template-compiler: ^2.6.7 => 2.6.10
    vue-template-es2015-compiler:  1.9.1
  npmGlobalPackages:
    @vue/cli: 3.8.2

Steps to reproduce
with the plugin "vue-cli-plugin-cordova" installed/set up, run "npm run cordova-serve-android"
What is expected?
N/A
What is actually happening?
N/A

When running "npm run serve" for mobile/cordova, it throws the exception "Uncaught SyntaxError: Use of const in strict mode" causing the app to hang. I understand this is a babel-related issue but since vue-cli is handling babel, I thought I should report this here. The issue on started from v3.8.0. Prior versions since v3.0.3 all work and so I'm using v3.7.0 for now. I must also mention that the issue does not apply when building, only for when serving/dev

Hi
Thanks for filing this issue! Could you please provide a small reproduction on jsfiddle.net so we can debug this without trying to replicate this ourselves?`
That would be very helpful, thanks!
Also: As this only happens with the cordova setup, did you open an issue there as well?

We receive this error a lot:
https://sentry.io/share/issue/1cbe4aae4a3d40d28be7038b78f8f181/

We receive this error a lot:
https://sentry.io/share/issue/1cbe4aae4a3d40d28be7038b78f8f181/


add development api bypass

Pull Request Considerations
Although some of these things can only be done after merged, please ensure all items have been considered when doing pull requests.

Updated README.md(s) for new functionality or changes.
Did you update .env / create a new environment variable?

Did you update/create an explanation in the appropriate README.md file for it?
Did you make/update an Openshift configmap/secret template for it?

Have you changed any Openshift configurations/templates?

Did you execute the appropriate oc process command on the template to put the configmap/secret in openshift?

Did changes trigger Openshift image rebuilds?

Check Openshift instance of application to ensure your changes are working.

Did you update the changelog CHANGELOG.md.

used to keep track of development changes including (env changes, docker changes, functionality changes)
help developers keep track of new/updating changes in code.

Code looks good. What about modifying the "userCan" function? Is that not required anymore?

Code looks good. What about modifying the "userCan" function? Is that not required anymore?

Sorry! I see in the ticket that the function shouldn't be committed. All good

Code looks good. What about modifying the "userCan" function? Is that not required anymore?

Sorry! I see in the ticket that the function shouldn't be committed. All good

No worries, and thanks!


fix gopkg path parsing

dep does not correctly understand the valid gopkg.in path "gopkg.in/juju/charm.v6-unstable".
It prints:
	% dep init
sm.DeduceProjectRoot: could not parse "6-unstable" as a gopkg.in major version

Ahh, yeah. This is entirely a gps thing - see referenced issue.

This is now (finally) fixed 😄


Alert close button align issues

Expected Behavior

To be aligned as expected
Current Behavior

Align issues
Steps to Reproduce

Access https://designrevision.com/docs/shards-vue/components/alert
At the dismissables
Check the close button

Context (Environment)

Possible Solution

// fix
.alert-dismissible .close {
    top: 0;
    right: 0;
    padding: .75rem 1.25rem 1.25rem;
    transition: all .25s cubic-bezier(.27,.01,.38,1.06);
    line-height: 1.2rem;
}

// Previous
.alert-dismissible .close {
    top: 0;
    right: 0;
    padding: .75rem 1.25rem;
    transition: all .25s cubic-bezier(.27,.01,.38,1.06);
}

@rodrigorhas Thanks for taking the time to submit this Rodrigo! 😊


Add support for managing workspace access

new resource resource_workspace_access to manage workspace access to apps, security groups, and users.

@alex-davies, please have a look at this PR as GitHub action tests were not initiated due to branch to branch PR.

Thanks again for implementing this
I have pulled your changes into a separate branch ( https://github.com/codecutout/terraform-provider-powerbi/pull/17) so I could perform the rebase and do some minor tweaks to get the tests running on the build server
Unfortunately github doesn't seem to associate your commits to your github username so my PR doesn't give you credit for your contributions. If you want to recognition for your changes i'm happy for you to reraise my PR in your name

Thats great if you can do so, or let me know if I have to raise PR again to master, as you have already merged branch client-credential to master.

Yes i've merged the client-credential change and i've rebased your workspace-access changes on the new master and done a few minor changes to update docs and get the automated build running. I have it on a branch on a feature/workspace-access branch in this repo.
If you create the PR from that branch to master (https://github.com/codecutout/terraform-provider-powerbi/compare/master...feature/workspace-access) I will merge and it should mean you get your github name on the squash merge commit.
If you not concerned about having your name on the commit let me know and I can go ahead and raise it myself. But you do deserve credit for the commit

Yes i've merged the client-credential change and i've rebased your workspace-access changes on the new master and done a few minor changes to update docs and get the automated build running. I have it on a branch on a feature/workspace-access branch in this repo.
If you create the PR from that branch to master (https://github.com/codecutout/terraform-provider-powerbi/compare/master...feature/workspace-access) I will merge and it should mean you get your github name on the squash merge commit.
If you not concerned about having your name on the commit let me know and I can go ahead and raise it myself. But you do deserve credit for the commit

Yes i've merged the client-credential change and i've rebased your workspace-access changes on the new master and done a few minor changes to update docs and get the automated build running. I have it on a branch on a feature/workspace-access branch in this repo.
If you create the PR from that branch to master (master...feature/workspace-access) I will merge and it should mean you get your github name on the squash merge commit.
If you not concerned about having your name on the commit let me know and I can go ahead and raise it myself. But you do deserve credit for the commit

Thanks, I've raised new PR #18 from this branch

Yes i've merged the client-credential change and i've rebased your workspace-access changes on the new master and done a few minor changes to update docs and get the automated build running. I have it on a branch on a feature/workspace-access branch in this repo.
If you create the PR from that branch to master (master...feature/workspace-access) I will merge and it should mean you get your github name on the squash merge commit.
If you not concerned about having your name on the commit let me know and I can go ahead and raise it myself. But you do deserve credit for the commit

Thanks, I've raised new PR #18 from this branch


Append Elastic/synthetics to UA string

Fixes https://github.com/elastic/synthetics/issues/232
Appends Elastic/Synthetics to UA string.
This PR uses a pretty ugly way to determin current user agent string and append the desired string into it.
Not sure if there is a way better way to do this. Struggling with playwright docs on how to get the current user agent string without opening a browser context.

:broken_heart: Tests Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2021-09-20T12:05:52.521+0000

Duration: 7 min 29 sec

Commit: f30f62205c30ccda4b038e40fd023f32fc4ac16a

Test stats :test_tube:

Test
Results

Failed
2

Passed
127

Skipped
0

Total
129

Trends :test_tube:

Test errors
Expand to view the tests failures
Test / CLI screenshots with device emulation – CLI screenshots with device emulation

no error details
Expand to view the stacktrace
 Error: Failed: "Error SyntaxError: Unexpected end of JSON input could not parse data '{\"type\":\"screenshot/block\",\"_id\":\"71bb77ed1acf85f13f1e8a2ab3cf3401850f7b9d\",\"@timestamp\":1632139976353934.8,\"root_fields\":{\"os\":{\"platform\":\"linux\"},\"package\":{\"name\":\"@elastic/synthetics\",\"version\":\"1.0.0-beta.14\"}},\"blob\":\"/9j/2wBDAAYEBQYFBAYGBQYHBwYIChAKCgkJChQODwwQFxQYGBcUFhYaHSUfGhsjHBYWICwgIyYnKSopGR8tMC0oMCUoKSj/2wBDAQcHBwoIChMKChMoGhYaKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCj/wAARCAFyALQDASIAAhEBAxEB/8QAFQABAQAAAAAAAAAAAAAAAAAAAAj/xAAUEAEAAAAAAAAAAAAAAAAAAAAA/8QAFAEBAAAAAAAAAAAAAAAAAAAAAP/EABQRAQAAAAAAAAAAAAAAAAAAAAD/2gAMAwEAAhEDEQA/AKpAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA'"
    at Env.it (/var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-380/src/github.com/elastic/synthetics/node_modules/jest-jasmine2/build/jasmineAsyncInstall.js:95:24)
    at it (/var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-380/src/github.com/elastic/synthetics/node_modules/jest-jasmine2/build/jasmine/jasmineLight.js:100:21)
    at Suite.<anonymous> (/var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-380/src/github.com/elastic/synthetics/__tests__/cli.test.ts:163:3)
    at addSpecsToSuite (/var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-380/src/github.com/elastic/synthetics/node_modules/jest-jasmine2/build/jasmine/Env.js:444:51)
    at Env.describe (/var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-380/src/github.com/elastic/synthetics/node_modules/jest-jasmine2/build/jasmine/Env.js:414:11)
    at describe (/var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-380/src/github.com/elastic/synthetics/node_modules/jest-jasmine2/build/jasmine/jasmineLight.js:88:18)
    at Object.<anonymous> (/var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-380/src/github.com/elastic/synthetics/__tests__/cli.test.ts:30:1)
    at Runtime._execModule (/var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-380/src/github.com/elastic/synthetics/node_modules/jest-runtime/build/index.js:1299:24)
    at Runtime._loadModule (/var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-380/src/github.com/elastic/synthetics/node_modules/jest-runtime/build/index.js:898:12)
    at Runtime.requireModule (/var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-380/src/github.com/elastic/synthetics/node_modules/jest-runtime/build/index.js:746:10)

Test / Gatherer append Elastic/Synthetics as part of userAgent – Gatherer append Elastic/Synthetics as part of userAgent

no error details
Expand to view the stacktrace
 Error: expect(received).toBe(expected) // Object.is equality

Expected: "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) HeadlessChrome/94.0.4595.0 Safari/537.36 Elastic/Synthetics"
Received: "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) HeadlessChrome/94.0.4595.0 Safari/537.36 Elastic/Synthetics"
    at Object.<anonymous> (/var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-380/src/github.com/elastic/synthetics/__tests__/core/gatherer.test.ts:61:67)

Steps errors
Expand to view the steps failures

Runs the tests

Took 1 min 57 sec . View more details here
Description: ./node_modules/.bin/jest --ci --reporters=default --reporters=jest-junit

Log output
Expand to view the last 100 lines of log output

[2021-09-20T12:12:59.244Z] PASS __tests__/index.test.ts
[2021-09-20T12:12:59.508Z] FAIL __tests__/core/gatherer.test.ts
[2021-09-20T12:12:59.508Z]   ● Gatherer › append Elastic/Synthetics as part of userAgent
[2021-09-20T12:12:59.508Z]
[2021-09-20T12:12:59.508Z]     expect(received).toBe(expected) // Object.is equality
[2021-09-20T12:12:59.508Z]
[2021-09-20T12:12:59.509Z]     Expected: "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) HeadlessChrome/94.0.4595.0 Safari/537.36 Elastic/Synthetics"
[2021-09-20T12:12:59.509Z]     Received: "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) HeadlessChrome/94.0.4595.0 Safari/537.36 Elastic/Synthetics"
[2021-09-20T12:12:59.509Z]
[2021-09-20T12:12:59.509Z]       59 |     const driver = await Gatherer.setupDriver({ wsEndpoint });
[2021-09-20T12:12:59.509Z]       60 |
[2021-09-20T12:12:59.509Z]     > 61 |     expect(await driver.page.evaluate(() => navigator.userAgent)).toBe(
[2021-09-20T12:12:59.509Z]          |                                                                   ^
[2021-09-20T12:12:59.509Z]       62 |       'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) HeadlessChrome/94.0.4595.0 Safari/537.36 Elastic/Synthetics'
[2021-09-20T12:12:59.509Z]       63 |     );
[2021-09-20T12:12:59.509Z]       64 |
[2021-09-20T12:12:59.509Z]
[2021-09-20T12:12:59.509Z]       at Object.<anonymous> (__tests__/core/gatherer.test.ts:61:67)
[2021-09-20T12:12:59.509Z]
[2021-09-20T12:13:00.459Z] PASS __tests__/plugins/plugin-manager.test.ts
[2021-09-20T12:13:01.855Z] PASS __tests__/plugins/performance.test.ts
[2021-09-20T12:13:01.855Z] PASS __tests__/config.test.ts
[2021-09-20T12:13:03.254Z] PASS __tests__/sdk/trace-processor.test.ts
[2021-09-20T12:13:04.642Z] PASS __tests__/plugins/tracing.test.ts
[2021-09-20T12:13:05.584Z] PASS __tests__/core/browser-service.test.ts
[2021-09-20T12:13:05.584Z] PASS __tests__/dsl/journey.test.ts
[2021-09-20T12:13:06.156Z] PASS __tests__/core/logger.test.ts
[2021-09-20T12:13:18.407Z] FAIL __tests__/cli.test.ts (51.036 s)
[2021-09-20T12:13:18.407Z]   ● CLI › screenshots with device emulation
[2021-09-20T12:13:18.407Z]
[2021-09-20T12:13:18.407Z]     Failed: "Error SyntaxError: Unexpected end of JSON input could not parse data '{\"type\":\"screenshot/block\",\"_id\":\"71bb77ed1acf85f13f1e8a2ab3cf3401850f7b9d\",\"@timestamp\":1632139976353934.8,\"root_fields\":{\"os\":{\"platform\":\"linux\"},\"package\":{\"name\":\"@elastic/synthetics\",\"version\":\"1.0.0-beta.14\"}},\"blob\":\"/9j/2wBDAAYEBQYFBAYGBQYHBwYIChAKCgkJChQODwwQFxQYGBcUFhYaHSUfGhsjHBYWICwgIyYnKSopGR8tMC0oMCUoKSj/2wBDAQcHBwoIChMKChMoGhYaKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCj/wAARCAFyALQDASIAAhEBAxEB/8QAFQABAQAAAAAAAAAAAAAAAAAAAAj/xAAUEAEAAAAAAAAAAAAAAAAAAAAA/8QAFAEBAAAAAAAAAAAAAAAAAAAAAP/EABQRAQAAAAAAAAAAAAAAAAAAAAD/2gAMAwEAAhEDEQA/AKpAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA'"
[2021-09-20T12:13:18.408Z]
[2021-09-20T12:13:18.408Z]       161 |   });
[2021-09-20T12:13:18.408Z]       162 |
[2021-09-20T12:13:18.408Z]     > 163 |   it('screenshots with device emulation', async () => {
[2021-09-20T12:13:18.408Z]           |   ^
[2021-09-20T12:13:18.408Z]       164 |     const cli = new CLIMock()
[2021-09-20T12:13:18.408Z]       165 |       .args([
[2021-09-20T12:13:18.408Z]       166 |         join(FIXTURES_DIR, 'fake.journey.ts'),
[2021-09-20T12:13:18.408Z]
[2021-09-20T12:13:18.408Z]       at Env.it (node_modules/jest-jasmine2/build/jasmineAsyncInstall.js:95:24)
[2021-09-20T12:13:18.408Z]       at Suite.<anonymous> (__tests__/cli.test.ts:163:3)
[2021-09-20T12:13:18.408Z]       at Object.<anonymous> (__tests__/cli.test.ts:30:1)
[2021-09-20T12:13:18.408Z]
[2021-09-20T12:13:18.408Z] A worker process has failed to exit gracefully and has been force exited. This is likely caused by tests leaking due to improper teardown. Try running with --detectOpenHandles to find leaks.
[2021-09-20T12:13:18.408Z] Summary of all failing tests
[2021-09-20T12:13:18.408Z] FAIL __tests__/core/gatherer.test.ts
[2021-09-20T12:13:18.408Z]   ● Gatherer › append Elastic/Synthetics as part of userAgent
[2021-09-20T12:13:18.408Z]
[2021-09-20T12:13:18.408Z]     expect(received).toBe(expected) // Object.is equality
[2021-09-20T12:13:18.408Z]
[2021-09-20T12:13:18.408Z]     Expected: "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) HeadlessChrome/94.0.4595.0 Safari/537.36 Elastic/Synthetics"
[2021-09-20T12:13:18.408Z]     Received: "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) HeadlessChrome/94.0.4595.0 Safari/537.36 Elastic/Synthetics"
[2021-09-20T12:13:18.408Z]
[2021-09-20T12:13:18.408Z]       59 |     const driver = await Gatherer.setupDriver({ wsEndpoint });
[2021-09-20T12:13:18.408Z]       60 |
[2021-09-20T12:13:18.408Z]     > 61 |     expect(await driver.page.evaluate(() => navigator.userAgent)).toBe(
[2021-09-20T12:13:18.408Z]          |                                                                   ^
[2021-09-20T12:13:18.408Z]       62 |       'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) HeadlessChrome/94.0.4595.0 Safari/537.36 Elastic/Synthetics'
[2021-09-20T12:13:18.408Z]       63 |     );
[2021-09-20T12:13:18.408Z]       64 |
[2021-09-20T12:13:18.408Z]
[2021-09-20T12:13:18.408Z]       at Object.<anonymous> (__tests__/core/gatherer.test.ts:61:67)
[2021-09-20T12:13:18.408Z]
[2021-09-20T12:13:18.408Z] FAIL __tests__/cli.test.ts (51.036 s)
[2021-09-20T12:13:18.408Z]   ● CLI › screenshots with device emulation
[2021-09-20T12:13:18.408Z]
[2021-09-20T12:13:18.408Z]     Failed: "Error SyntaxError: Unexpected end of JSON input could not parse data '{\"type\":\"screenshot/block\",\"_id\":\"71bb77ed1acf85f13f1e8a2ab3cf3401850f7b9d\",\"@timestamp\":1632139976353934.8,\"root_fields\":{\"os\":{\"platform\":\"linux\"},\"package\":{\"name\":\"@elastic/synthetics\",\"version\":\"1.0.0-beta.14\"}},\"blob\":\"/9j/2wBDAAYEBQYFBAYGBQYHBwYIChAKCgkJChQODwwQFxQYGBcUFhYaHSUfGhsjHBYWICwgIyYnKSopGR8tMC0oMCUoKSj/2wBDAQcHBwoIChMKChMoGhYaKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCgoKCj/wAARCAFyALQDASIAAhEBAxEB/8QAFQABAQAAAAAAAAAAAAAAAAAAAAj/xAAUEAEAAAAAAAAAAAAAAAAAAAAA/8QAFAEBAAAAAAAAAAAAAAAAAAAAAP/EABQRAQAAAAAAAAAAAAAAAAAAAAD/2gAMAwEAAhEDEQA/AKpAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA'"
[2021-09-20T12:13:18.408Z]
[2021-09-20T12:13:18.408Z]       161 |   });
[2021-09-20T12:13:18.408Z]       162 |
[2021-09-20T12:13:18.408Z]     > 163 |   it('screenshots with device emulation', async () => {
[2021-09-20T12:13:18.408Z]           |   ^
[2021-09-20T12:13:18.408Z]       164 |     const cli = new CLIMock()
[2021-09-20T12:13:18.408Z]       165 |       .args([
[2021-09-20T12:13:18.408Z]       166 |         join(FIXTURES_DIR, 'fake.journey.ts'),
[2021-09-20T12:13:18.408Z]
[2021-09-20T12:13:18.408Z]       at Env.it (node_modules/jest-jasmine2/build/jasmineAsyncInstall.js:95:24)
[2021-09-20T12:13:18.408Z]       at Suite.<anonymous> (__tests__/cli.test.ts:163:3)
[2021-09-20T12:13:18.408Z]       at Object.<anonymous> (__tests__/cli.test.ts:30:1)
[2021-09-20T12:13:18.408Z]
[2021-09-20T12:13:18.408Z]
[2021-09-20T12:13:18.408Z] Test Suites: 2 failed, 19 passed, 21 total
[2021-09-20T12:13:18.408Z] Tests:       2 failed, 113 passed, 115 total
[2021-09-20T12:13:18.408Z] Snapshots:   13 passed, 13 total
[2021-09-20T12:13:18.408Z] Time:        52.021 s
[2021-09-20T12:13:18.408Z] Ran all test suites.
[2021-09-20T12:13:19.637Z] Post stage
[2021-09-20T12:13:19.666Z] Recording test results
[2021-09-20T12:13:19.761Z] [Checks API] No suitable checks publisher found.
[2021-09-20T12:13:19.901Z] Stage "E2e Test" skipped due to earlier failure(s)
[2021-09-20T12:13:19.984Z] Stage "Release" skipped due to earlier failure(s)
[2021-09-20T12:13:20.025Z] Stage "Release" skipped due to earlier failure(s)
[2021-09-20T12:13:20.112Z] Stage "Release" skipped due to earlier failure(s)
[2021-09-20T12:13:20.627Z] Running on Jenkins in /var/lib/jenkins/workspace/nt-rum_elastic-synthetics_PR-380
[2021-09-20T12:13:20.718Z] [INFO] getVaultSecret: Getting secrets
[2021-09-20T12:13:20.771Z] Masking supported pattern matches of $VAULT_ADDR or $VAULT_ROLE_ID or $VAULT_SECRET_ID
[2021-09-20T12:13:21.788Z] + chmod 755 generate-build-data.sh
[2021-09-20T12:13:21.788Z] + ./generate-build-data.sh https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-rum/elastic-synthetics/PR-380/ https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-rum/elastic-synthetics/PR-380/runs/1 FAILURE 448988
[2021-09-20T12:13:21.788Z] INFO: curl https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-rum/elastic-synthetics/PR-380/runs/1/steps/?limit=10000 -o steps-info.json

We discussed offline and decided to go with using browser based session approach to get the user agent.
Benchmarks if anyone is curious
Session: 3.505ms
Context: 245.072ms

i tested manualy, it seems to work with iframe and new window as well.
will try to write some tests for those use cases.

Added tests for iframe and popup


Tag Questions

List of questions for the larger group about tags:

Should all tags be proper cased? (ie, Association of Governments, Shelved, Bioscience, etc) (code is currently written to enforce this)
Should shelved data be tagged with SGID (will also be tagged with Shelved, Utah, and AGRC)?

(More can be added later...)

i prefer lowercase but i doubt my opinion will be popular.

+1 for lowercase/kebab case for tags
sgid, cadaster, two-words

I'm going to stick with proper case. The data team expressed a preference for proper case, and I feel like this is appropriate because the tags are primarily targeted to human beings, not computers. If these were purely for processing/automation, I'd agree with kebab case.