Added AR module

New Docs for AR module. To Do:

Release notes
KB?
Docs landing page img: Light and dark
Get Started guide (once done add links to landing page and home page)
Content check to make sure open source version of docs line up with enterprise version
LandingPage.tsx and featurelist (What does this do?)
Future work: Roadmap.

Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66fb140210ed252a65c5d693--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6720022b73cab6dfe5bd709c--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6721c4253b8ac40d3f7afafb--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6732ae1f4d4c3b0434443320--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6735168ba685c258326b59ee--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://673cf7c0dc638167533c3a8a--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://673d05d41d9cb162a2c9fa9a--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://673d102b4ea95b9cf3bc1c49--harness-developer.netlify.app

And @dewan-ahmed for the open source docs change.


Observability: Update OpenTelemetry doc

Pull Request Template
What are you trying to address
This PR aims to make the OpenTelemetry doc language agnostic and more consistent with OpenTelemetry documentation for the naming and concept definitions. It also adds more links to the original OpenTelemetry documentation pages to make sure the most up to date information are referenced in the playbook.
Closes #886
Description of new changes

Update the OpenTelemetry doc to be more generic and language agnostic.
Provide links to the OpenTelemetry documentation for more information/details on the OpenTelemetry concepts.
Make the usage of OpenTelemetry Agent term consistent with the OpenTelemetry documentation

For all pull requests

[x] Link to the issue you are solving (so it gets closed)
[x] Label the pull request with the appropriate area(s)
[x] Assign potential reviewers (you may also want to contact them on Teams to ensure timely reviews)
[x] Assign the project - Engineering Playbook Backlog
[x] Assign the pull request to yourself

Thanks @Izzzu for adding this. I have some minor suggestions

Thanks! I applied your suggestions

@Izzzu can I merge this?

Thank you for contributing @Izzzu, can I merge this?

Yes, please :)


No quantiles test in the unit tests for exp_mod_normal_rng

Summary:
There aren't any tests that validate exp_mod_normal_rng via assert_matches_quantiles. These tests are there for all the other distributions.
Description
Current code: https://github.com/stan-dev/math/blob/develop/test/unit/math/prim/scal/prob/exp_mod_normal_test.cpp
Example _rng with test: https://github.com/stan-dev/math/blob/develop/test/unit/math/prim/scal/prob/normal_test.cpp
The issue with this is it's not easy to test that exp_mod_normal_rng is working correctly. I found this out after I introduced a bug into it in my local working branch (trying to make it a little faster -- these changes didn't get committed) and the unit tests passed.
Current Version:
v2.17.0

Thanks for filing the issue.  The CDFs are implemented, so it shouldn't be hard to add them.

P.S.  I assigned to you (@bbbales2), but if you don't want to deal with it, please unassign yourself.

I believe this was fixed in #833.


feat(component): add SfHighlight

Related issue

Scope of work

a part of search result issues
<SfHighlight
  :tag="tag"
  :content="content"
  :highlighted="highlighted"
/>

Screenshots of visual changes
storybook:

figma

Checklist

[x] No commented blocks of code left
[x] Run tests and docs
[x] Self code-reviewed

If applicable:

[x] I followed composition rules for my component
[x] I tested the component in most common device sizes (can be tested in Storybook from viewport addon in top menu)
[x] I accept Individual Contributor License Agreement

remove v-html
use simplest API use only queries property
use  instead


Using parentData() instead of currentData to get state

Is this creating any problem?
At the moment we have some places where currentData is used while in some other there's parentData: see here.
I think we should make this uniform...
If you look at the documentation for parentData it says that currentData is equivalent to parentData(0).
IMHO currentData is more clear in that it clearly refers to the current data context for the template while parentData makes me think about the data context of the parent template...
What's your view on this?

I admit using parentData as the name for the local variable is a bit misleading...
Perhaps we should change all local variables to currentData and use Template.currentData everywhere?


Major API troubleshooter update v1.6.1

Issue #, if available:
Description of changes:
Major API troubleshooter update v1.5.7
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Looks good, I tried it in my environment.
Can we keep the filename same API-Server.json, so that it reflects the component name.


Support for TextEncoder

Hello again, is TextEncoder support in the roadmap? I don't see it there.
Regards.

It is! In development: https://developer.microsoft.com/en-us/microsoft-edge/platform/status/encodingstandard/?q=Encoding


Use a faster hash for the state diff map

The default hasher is pretty slow for HashMaps. We should consider using something like https://docs.rs/seahash/latest/seahash/

Found this:
https://gist.github.com/daboross/976978d8200caf86e02acb6805961195?permalink_comment_id=4205660#gistcomment-4205660

@rakita yep, it seems faster than seahash according to that benchmark which also has several even better candidates https://github.com/rurban/smhasher
both are non-cryptographic and should be ok for some of the revm usecases

@rakita yep, it seems faster than seahash according to that benchmark which also has several even better candidates https://github.com/rurban/smhasher
both are non-cryptographic and should be ok for some of the revm usecases

Should be okaу for all revm usecases, it is just i didnt spend time changing it 😅
Good link, thank you for sharing

TIL that hashbrown already uses ahash by default, and revm uses hashbrown for all performance-critical things 🙂
/// Default hasher for `HashMap`.
#[cfg(feature = "ahash")]
pub type DefaultHashBuilder = ahash::RandomState;

source: https://github.com/rust-lang/hashbrown/blob/1d2c1a81d1b53285decbd64410a21a90112613d7/src/map.rs#L11-L13

hashbrown it is :)


Refactor authentication into separate object

The way the authentication logic was build into the other modules made unit tests of the authentication itself and of service calls executed with a specific authentication type very hard.
Moving the authentication into a authenticator object not only better separates the concerns of the library, but also enables simple unit tests and mocking of the authentication.

Replaced arrow functions as in #21.
Tests running successfully down to node v0.10.x now.


Drop down selection does not work in React apps

Repo with example to reproduce: https://github.com/riccardoNovaglia/taikoselect
Expected behavior
Selecting from a dropdown causes the value to change
Actual behavior
The value looks to be selected, but even tho the value in the dropdown looks selected, the app does not respond
Steps to reproduce

checkout repo above, npm install top-level and my-app
start app (cd my-app, npm run start)
manually select from dropdown at the bottom of the page (below "Learn React"), see result update just below
run taiko reproduceIssue.js --observe and see result not updating

Versions
➜  taikoselect git:(master) ✗ node --version
v12.12.0
➜  taikoselect git:(master) ✗ npm --version
6.12.0
➜  taikoselect git:(master) ✗ taiko --version
Version: 1.0.2 (Chromium: 76.0.3803.0) RELEASE

@riccardoNovaglia This issue has been fixed and available in master but not yet released.
For now you can change the package.json file in taikoselect folder to the below code which fetched taiko from master and things should work.
{
  "name": "taikoselect",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "author": "",
  "license": "ISC",
  "dependencies": {
    "taiko": "https://github.com/getgauge/taiko"
  }
}

Amazing, thanks!

Any thoughts as to when the next version with the fix might be released?


add a github component

I was reading mkdoc based documetnation and I saw this little component in their headerbar. Would you like to add it to the theme ?
It would display the link to the repository, the version tag, the number of stars and the number of forks.

IMO the navbar-icon-links support for GitHub is enough. The great thing about sphinx themes is that there are many to choose from, so we don't need to make our theme do everything out of the box.

closing as we already have enough on our plates


Remove mk-initrd.sh script

As pointed out by @SvenDowideit, this script is no longer used anywhere.

LGTM


NIP-05 verification

Add a simple URLSession request that validates NIP-05 identifiers.
https://github.com/nostr-protocol/nips/blob/master/05.md

Working on this...


feature: Support more arch for docker image: casdoor-all-in-one

Hey, casdoor community.
Currently, the casdoor-all-in-one docker image only provide linux/amd64 arch. Should we add more arch support? For example, linux/amd64?
Thanks, and Best Regards.

Duplicated, see: https://github.com/casdoor/casdoor/issues/2416#issuecomment-1766442931


th175: skipgame and instant_ending

Implement skipgame and instant_ending for the full version of th175

instant_ending done in thpatch/thcrap-tsa@d1f06e43627d821ab6bcb6cc9f36ed5a9c207611, moving skipgame into a new issue


Broken/invalid URL

This URL is invalid/broken:
http://riotgear.js.org/components/sidemenu/
It's the "Sidemenu" link under Components here:
http://riotgear.js.org/

v3.2.0 release of the site removed this link just now ;-)
Ps. and added Charts


Frameworks

Is it possible to integrate with Rails?

Unfortunately I have never worked with Rails. You may have more luck if you post your question directly in the single-spa repo here: https://github.com/CanopyTax/single-spa

Discussion has been moved to https://github.com/CanopyTax/single-spa/issues/219. Closing this issue here.


A few fixes in the EVP_KDF-KB and EVP_KDF-KRB5KDF manpages

The context and label is info and salt respectively - fix
the example in KB manpage for that.
There are some typos and bug in EVP_KDF_derive call in the
KRB5KDF example.
Checklist

[x] documentation is added or updated

Appveyor failure is unrelated.

Merged to master as a90311fead


polyostotic fibrous dysplasia unsatsifiable

Note the conflict in ORDO, a genetic disease under FDoB which the text says is acquired


I can't set the root_dir in spectral lsp

Note: If this issue is not exactly related to the installer please let me know, Any guidance is much appreciated, I've very new to all of this btw 😅.
What I've tried
I've created this lua file for my custom server
local servers = require "nvim-lsp-installer.servers"
local server = require "nvim-lsp-installer.server"
local npm = require "nvim-lsp-installer.installers.npm"

local installer = npm.packages { "spectral-language-server" }

local server_name = "spectral"

local get_root_dir = function (sn)
  return server.get_server_root_path(sn)
end

local spectral_server = server.Server:new {
  name = server_name,
  root_dir = get_root_dir(server_name),
  installer = installer,
-- default options copied from the server config in lsp config
  default_options = {
        cmd = { "spectral-language-server", "--stdio" },
    filetypes = { "yaml", "json", "yml" },
    root_dir = function() -- [1] I don't know exactly that I should write here
        return get_root_dir(server_name)
      end,
    settings = {
      enable = true,
      run = "onType",
      validateLanguages = { "yaml", "json", "yml" }
    },
    single_file_support = true
  }
}
servers.register(spectral_server)

part of my issue is I can't write a suitable root_dir function [1], in the config guide https://github.com/neovim/nvim-lspconfig/blob/master/doc/server_configurations.md#spectral There is M I don't know what it does refer to in this context.
The log from lsp-install seems normal
[DEBUG Mon 29 Nov 2021 04:02:47 AM EET] ...art/nvim-lsp-installer/lua/nvim-lsp-installer/server.lua:75: Attaching server to buffers spectral
[DEBUG Mon 29 Nov 2021 04:02:47 AM EET] ...art/nvim-lsp-installer/lua/nvim-lsp-installer/server.lua:86: Successfully attached server to buffers spectral

The log from lsp is
[START][2021-11-29 04:02:47] LSP logging initiated
[ERROR][2021-11-29 04:02:51] .../vim/lsp/rpc.lua:417	"rpc"	"spectral-language-server"	"stderr"	"node:internal/modules/cjs/loader:936\n  throw err;\n  ^\n\nError: Cannot find module 'vscode-languageserver/lib/utils/is'\nRequire stack:\n- /home/a/.npm/lib/node_modules/spectral-language-server/dist/spectral-language-server/server.js\n- /home/a/.npm/lib/node_modules/spectral-language-server/bin/spectral-language-server\n    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:933:15)\n    at Function.Module._load (node:internal/modules/cjs/loader:778:27)\n    at Module.require (node:internal/modules/cjs/loader:999:19)\n    at require (node:internal/modules/cjs/helpers:102:18)\n    at Object.<anonymous> (/home/a/.npm/lib/node_modules/spectral-language-server/dist/spectral-language-server/server.js:7:14)\n    at Module._compile (node:internal/modules/cjs/loader:1097:14)\n    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1149:10)\n    at Module.load (node:internal/modules/cjs/loader:975:32)\n    at Function.Module._load (node:internal/modules/cjs/loader:822:12)\n    at Module.require (node:internal/modules/cjs/loader:999:19) {\n  code: 'MODULE_NOT_FOUND',\n  requireStack: [\n    '/home/a/.npm/lib/node_modules/spectral-language-server/dist/spectral-language-server/server.js',\n    '/home/a/.npm/lib/node_modules/spectral-language-server/bin/spectral-language-server'\n  ]\n}\n\nNode.js v17.1.0\n"

Hello! Ah yes it seems like you're on the right track. The idea of the root_dir is different between lspconfig and nvim-lsp-installer. It's a bit unfortunate I ended up using the same name for a different concept. The root_dir  is responsible for the following ,respectively:
lspconfig
root_dir is a function that receives a filename and a bufnr as arguments. It needs to return the absolute path to a working directory in which lspconfig should start the server (aka root_dir, aka workspace folder). This is generally done by scanning the file system for the existence of certain files and/or directories. Usually one would start this scan from the directory of the provided filename argument, and continue upwards until the search pattern is found or until there are no more parent directories to traverse. A very common search pattern is the existence of a .git/ directory, but some LSPs may also search for a specific file (for example GraphQL looks for a .graphqlrc* file).
nvim-lsp-installer
The root_dir property of a server instance controls where the server should be installed. This is only used when:

installing a server
uninstalling a server
telling lspconfig where to find the cmd to the server (this is done when calling server:setup())

I think we should add spectral to nvim-lsp-installer, it seems like a really straight forward server to install as it's just an npm package.

Since lspconfig already provide a default config for spectral, all you really need to do is override the cmd, like so:
local root_dir = get_root_dir(server_name)

local spectral_server = server.Server:new {
  name = server_name,
  root_dir = root_dir,
  installer = installer,
-- default options copied from the server config in lsp config
  default_options = {
        cmd = { npm.executable(root_dir, "spectral-language-server"), "--stdio" },
  }
}

See #295 for adding this to nvim-lsp-installer. However there seem to be some potential issues with the npm package, see https://github.com/luizcorreia/spectral-language-server/issues/1

Thanks for your work I will check https://github.com/luizcorreia/spectral-language-server/issues/1. I think this issue should be closed now


Microsoft.TestPlatform only discovers .NET Core unit tests

Description
This follows on from #808.
I can successfully discover and run tests in the following configurations:

NUnit 3 (NUnit3TestAdapter <IP_ADDRESS>-alpha1) targeting netcoreapp1.1
XUnit (default .NET Core test project) targeting netcoreapp1.1
MSTest (default .NET Core test project) targeting netcoreapp1.1

In the following configurations, VSTestConsoleWrapper does not discover any tests:

NUnit 3 (NUnit3TestAdapter 3.8.0-alpha1) targeting net45
NUnit 3 (NUnit3TestAdapter 3.8.0-alpha1) targeting net46
Universal Windows Unit Testing Application using default MSTest adapter

The target framework this appears to be baked in as .NET Core 5.0

MSTest (default .NET Core test project) targeting net46

Steps to reproduce
I've uploaded a copy of my test client code to https://gist.github.com/yaakov-h/19f4cc856d4b83e606c19005ba36b961
Using:

Microsoft.TestPlatform (from MyGet)
Microsoft.TestPlatform.TranslationLayer (from NuGet)

You can create a new Unit Test Project from the template in VS, then modify the csproj to have <TargetFrameworks>netcoreapp1.1;net46</TargetFrameworks> to use as a test for the two MSTest configurations listed above.
Then, run the sample client, giving it the path to vstest.console.exe and the path to the compiled unit-test assembly.
Expected behavior
C:\temp\VSTest>"D:\Team Data\Spike\VSTest\VSTest\bin\Debug\VSTest.exe" "C:\Temp\VSTest\packages\Microsoft.TestPlatform.15.0.0\tools\net46\vstest.console.exe" C:\Temp\UnitTestRepro\UnitTestProject1\bin\Debug\net46\UnitTestProject1.dll
Target Framework Version: .NETFramework,Version=v4.6
Log message: [Informational] Logging TestHost Diagnostics in file: C:\temp\VSTest\VSTestLog_20170622-150806.host.17-06-22_15-08-06_74174_1.log
Discovered 1 tests.
- TestMethod1
Log message: [Informational] Logging TestHost Diagnostics in file: C:\temp\VSTest\VSTestLog_20170622-150815.host.17-06-22_15-08-17_01148_4.log
-  Passed

C:\temp\VSTest>"D:\Team Data\Spike\VSTest\VSTest\bin\Debug\VSTest.exe" "C:\Temp\VSTest\packages\Microsoft.TestPlatform.15.0.0\tools\net46\vstest.console.exe" C:\Temp\UnitTestRepro\UnitTestProject1\bin\Debug\netcoreapp1.1\UnitTestProject1.dll
Target Framework Version: .NETCoreApp,Version=v1.1
Log message: [Informational] Logging TestHost Diagnostics in file: C:\temp\VSTest\VSTestLog_20170622-150815.host.17-06-22_15-08-16_23948_4.log
Discovered 1 tests.
- TestMethod1
Log message: [Informational] Logging TestHost Diagnostics in file: C:\temp\VSTest\VSTestLog_20170622-150815.host.17-06-22_15-08-17_01148_4.log
-  Passed

C:\temp\VSTest>

Actual behavior
C:\temp\VSTest>"D:\Team Data\Spike\VSTest\VSTest\bin\Debug\VSTest.exe" "C:\Temp\VSTest\packages\Microsoft.TestPlatform.15.0.0\tools\net46\vstest.console.exe" C:\Temp\UnitTestRepro\UnitTestProject1\bin\Debug\net46\UnitTestProject1.dll
Target Framework Version: .NETFramework,Version=v4.6
Log message: [Informational] Logging TestHost Diagnostics in file: C:\temp\VSTest\VSTestLog_20170622-150806.host.17-06-22_15-08-06_74174_1.log
Discovered 0 tests.

C:\temp\VSTest>"D:\Team Data\Spike\VSTest\VSTest\bin\Debug\VSTest.exe" "C:\Temp\VSTest\packages\Microsoft.TestPlatform.15.0.0\tools\net46\vstest.console.exe" C:\Temp\UnitTestRepro\UnitTestProject1\bin\Debug\netcoreapp1.1\UnitTestProject1.dll
Target Framework Version: .NETCoreApp,Version=v1.1
Log message: [Informational] Logging TestHost Diagnostics in file: C:\temp\VSTest\VSTestLog_20170622-150815.host.17-06-22_15-08-16_23948_4.log
Discovered 1 tests.
- TestMethod1
Log message: [Informational] Logging TestHost Diagnostics in file: C:\temp\VSTest\VSTestLog_20170622-150815.host.17-06-22_15-08-17_01148_4.log
-  Passed

C:\temp\VSTest>

Environment
Windows 10.0.15063.332 (v1703 "Creator's Update") (i.e. w/ .NET Framework 4.7)
Microsoft Visual Studio Enterprise 2017 15.2.26430.13

I might be experiencing the same thing. This is what I'm doing.

Clone vstest repo and build
On the samples project I set Microsoft.TestPlatform.Protocol as my startup project
Set the testAssembly, testAdapterPath to the corresponding dlls on samples/UnitTestProject/bin/Debug/.netcoreapp1.0
Run the program and all test cases are found
If I switch to samples/UnitTestProject/bin/Debug/net46, and set the framework to .NETFramework,Version=v4.6, no tests are discovered.

Any idea why is this happening? Same settings as above.
Luis

@yaakov-h I think you need to update the InitializeExtensions call (here) to include the test adapter path. It can be the output directory of test project, send a list of *.testadapter.dll files. This is required only for net46, for netcore projects adapters are picked up automatically by the test platform.
@luis-cortina for the net46 scenario, are you also setting the testadapter path? It is possible to generate diagnostic logs by modifying this line, add --diag log.txt to the process args. At end of run, it will generate 2 files log.txt and log.host*.txt, these will have more information on internals of vstest.

I was using the wrong adapter. It works ok now.
Btw the --diag log.txt was hanging the app, I switch to /diag:log.txt  and worked. All good on my side.

Thanks @codito, that's nearly got me all the way.
Interestingly, in my test cases:

NUnit 3 adapter for .NET Core calls HandleDiscoveredTests with tests, then HandleDiscoveryComplete with lastChunk of null.
NUnit 3 adapter for .NET Framework calls HandleDiscoveredTests with tests, then HandleDiscoveryComplete with lastChunk being the same tests already discovered.

I assume that's an adapter issue to take up with the NUnit team?
Also, the MSTest adapter has an odd error for UWP, then returns 0 tests:

[Warning] File does not exist: C:\Workspace\DevTools\bin\NetCoreSampleTests\UWP\UniversalWindowsMSTestTests.exe

The file does exist. Is this one a bug for the MSTest team?

NUnit 3 adapter for .NET Framework calls HandleDiscoveredTests with tests, then HandleDiscoveryComplete with lastChunk being the same tests already discovered.

Looks like a bug in vstest. Can you please create an issue to track this?

Also, the MSTest adapter has an odd error for UWP, then returns 0 tests:

It shouldn't be working :) UWP support is not available yet in test platform. We're working to add it in near future.

@codito ah, dangit. Any ETA on UWP?

We are tracking UWP work here


Pseudogenome file

Description of feature
Good day,
Apologies if I'm at the wrong place here...
I tried joining the Slack channel but can't seem to find it.
Am I correct in assuming that the pseudogenome alignment can be used directly as an input for iqtree for example to generate a phylogenetic tree? Or does it need further processing/filtering?
Thank you!
Really enjoying this pipeline!

Or rather the snp-sites extracted from the pseudo-alignment, sorry.

Hi,
Thanks for getting in touch. Yes, the two files in the snpsites directory can be fed directly to iqtree. This is my usual go to command (you may need to adjust memory/threads for your system):
iqtree -fconst $(cat constant_sites.txt) -s filtered_alignment.fas -nt AUTO -ntmax 8 -mem 8G -bb 1000 -m MFP
Kind regards,
Andries


Footer

Develop footer based on design elements

[x] Design (Mockups Assigned)
[ ]  Stakeholder Signoff
[ ] Tech Feasible Sign Off
[ ] Final Sign Off

Mockups: https://zpl.io/JEzg6X9 (That is documentation, the rest of the mockups have all been updated to have footers.)


readshapefile fails on certain filenames

Hi,
(Tested with basemap 1.0.6)
This works:
m.readshapefile('foo.shp', 'my_shape')
while this fails:
m.readshapefile('bar_0.1.shp', 'my_shape')
raise IOError('error reading shapefile %s.shp' % shapefile)
  fields = shf.fields
  coords = []; attributes = []

IOError: error reading shapefile bar_0.1.shp

Note that foo.shp and bar_0.1.shp are exactly the same file except for the filename. So it seems that the additional . causes the problem. Minimal example to reproduce behavior:
Rename  huralll020.{shp,dfb,shx} to huralll020_0.1.{shp,dbf,shx} and change filename in examples/hurrtracks.py accordingly. Run hurrtracks.py.
Of course the obvious workaround is not to use . in filenames. But this is somewhat impracticable as I have a large number or files and would like keeping the filenames consistent accross filetypes (netCDF, shapfiles, pngs, etc).
Is there an easy fix?
Thanks very much,
Andy

This issue can be closed here, if the issue is still there, then it should be reported to the pyshp project (I am guessing here: https://github.com/GeospatialPython/pyshp).


Mb 5518 static analysis g306

Description
Adds annotation to justify usage of permission level 644 for the writefile (the linter expects 600 or lower).  There is only 1 occurrence of G306.
Reviewer Notes
Permissions of 600 mean that the owner has full read and write access to the file, while no other user can access the file. Permissions of 644 mean that the owner of the file has read and write access, while the group members and other users on the system only have read access.In this situation, we want to protect from any changes to migration files once they have been deployed, and changes to the db should happen only in a new migration file.
Setup
pre-commit run -v --all-files golangci-lint | grep G306
Code Review Verification Steps

[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change
Article about permission levels.

Thanks for including the file permissions article! This one is a new concept for me and it's great to have the ref handy. Will review when I'm done reading, though without additional context the annotation seems good.


New version: ProfileLikelihood v0.3.5

Registering package: ProfileLikelihood
Repository: https://github.com/DanielVandH/ProfileLikelihood.jl
Created by: @DanielVandH
Version: v0.3.5
Commit: 1c8b17127b520c4bc25646e6827dd776af25b95c
Reviewed by: @DanielVandH
Reference: https://github.com/DanielVandH/ProfileLikelihood.jl/commit/1c8b17127b520c4bc25646e6827dd776af25b95c#commitcomment-137613281
Description: Methods for profile likelihood analysis.

[noblock]
Looks like there is a merge conflict.
@DanielVandH Can you re-trigger Registrator? This will fix the merge conflict.

Oh, sorry @DilumAluthge I should've commented that I replaced this by https://github.com/JuliaRegistries/General/pull/99356. I thought I had already registered 0.3.4 but I hadn't. This can be closed.


universal cable crash

Issue Description: using universal cable, connecting to Laser Relay, causes server to crash, any time this chunk loads the server crashes.
What happens: Ticking block entity
What you expected to happen: provide power.
Steps to reproduce:
1.connect universal power cable to laser relay.
Version of All the Mods:13c
Added mods (Optifine, Fastcraft, etc.): none
crash report.
http://pastebin.com/m2tH74iU

Fixed in dev, thanks!


Force Diagram issues

https://github.com/compas-dev/compas_ags/blob/f50259bc35ec76946867b595f8c80cdce0213ab6/src/compas_ags/diagrams/forcediagram.py#L49

Change:
network_find_faces(formdiagram, formdiagram.leaves())
For:
network_find_faces(formdiagram, formdiagram.breakpoints())

As in compas_tna to generate the dual? Otherwise when generating the faces with formdiagram.leaves() delete the exterior face using face_network.delete_face(0)

Did you and Tom resolve this? Can the issue be closed?

many things have been updated since then. please check if the issue still exists...

Yes it is solved but probably not implemented. I would have to check.
Ricardo Maia Avelino
On Jul 20, 2018, at 10:58, Andrew Liew <notifications<EMAIL_ADDRESS>wrote:
Did you and Tom resolve this? Can the issue be closed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcompas-dev%2Fcompas_ags%2Fissues%2F6%23issuecomment-406626798&data=02|01||b7a29177b2cf4f1964f608d5ee514622|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636676955184314872&sdata=uJgPfkOovwt4I%2B3GXHL700HLQQOwHnuoUg%2F3VpZafhE%3D&reserved=0, or mute the threadhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAido5PQlWtBytW06GxWJRR5ThhgLgRrCks5uIfAdgaJpZM4T4R3S&data=02|01||b7a29177b2cf4f1964f608d5ee514622|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636676955184314872&sdata=mzmo5qe%2BH9YOsV4pPklw9JTmScUxFlE6VyGe4%2BWDmSA%3D&reserved=0.


#689: Added url and removed try-catch-block

Fixes: #689
Added url to ide-projects/build and removed try-catch-block

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.02%) to 66.764%

Files with Coverage Reduction
New Missed Lines
%

com/devonfw/tools/ide/version/VersionIdentifier.java
1
86.0%

com/devonfw/tools/ide/url/model/UrlMetadata.java
1
83.87%

com/devonfw/tools/ide/log/IdeLogger.java
2
63.04%

Totals

Change from base Build<PHONE_NUMBER>4:
-0.02%

Covered Lines:
6255

Relevant Lines:
9027

💛  - Coveralls

Before we merge, I would like to discuss how we handle such test urls files...

I have created issue #706 for the idea of serving the software as compressed archive via wiremock.
As a preparation the suggested format for the urls files is as written in that issue:
${testbaseurl}/download/«tool»/«edition»/«version»/«tool»-«version».tgz

The values for «tool», «edition» and «version» need to be filled in while ${testbaseurl} remains as variable in the files and will later be resolved dynamically by implementation of #706.


document how to switch to different language of mobile app

https://github.com/MerginMaps/mobile/issues/743

https://support.google.com/android/answer/12395118?hl=en (Android 14+)
https://apple.stackexchange.com/questions/231000/ios-how-to-set-language-of-app-independent-of-iphone-ipad-language (iOS 13 +)

Currently not supported by our app on Android, only on iOS.


Olga/conditional render

added codntional rendering with gifs and fix for content to persist after extension is closed

redoing


sliders horizontal instead of vertical

Should try putting conditional sliders inline horizontally rather than vertically.. will simplify layout and potential be cleaner

i think mostly address with w/ referenced commit, will leave in for a couple days to see/report any draw backs.

closing as no issues w/ horizontal sliders :-)


LL-7265 - Make empty account tokens' names uppercase

Type
UI Polish
Context
LL-7265
Parts of the app affected / Test plan
Add an empty account for a SubAccount Compatible Chain -> check the text

Apparently this is a forgotten PR from @lambertkevin from when you switched to your personal account. Either close it, or follow the review (and update it) so we can get it merged 🙏🏼


Some minor cleanups on Unix

Build winapi crates on Windows only
Fix an unused import warning

Merged, thanks for the patch.
Maybe we could avoid a spurious warning on 1.15.x with a #![cfg(...)] guard? I'm not sure if that's standard practice in Rust, but it's a small price to pay for a warningless compile.


Update cocoon on devicelab agents

Manually update the remaining agents that aren't on LUCI to pick up https://github.com/flutter/cocoon/pull/1059 to help track down flaking flutter config call https://github.com/flutter/flutter/issues/65413.
https://github.com/flutter/flutter/issues/65413#issuecomment-783618007

Whenever changing cocoon agent, we need to manually update the machines. Feel free to file an infra ticket to get the machines updated with the change.

mac-{1,2,3,4,5} updated
mac-12 was updated but continues to be in a problematic state (error 65).
windows-{1,2} updated


Should BLEProcess and BLEApp run dispatch_forever()?

I'd prefer if an EventQueue I created was only truly executed in the code I control.
BLEProcess has signfiicantly simplified some of the code I previously had, but I had to comment out the call to dispatch_forever() in start() in my local copy so I could execute it when I needed it.

Yes, this is something that I would've done differently and will do differently if I get to write the production version of BLEApp. These utils here are not production code - it's just code do simplify the mbed ble examples, so they can be simpler to concentrate on showing only the features that are the focus of each example.


Use drupol/launcher

Hi,
I just created a new PHP package: drupol/launcher.
Basically this is a very basic helper that will open any resource that are passed as parameter with the default application of your operating system.
Using this package will remove a bunch of code from GraphViz.php.

Hi @drupol, thanks for bringing this to our attention!
I've identified ~10 lines of code that deal with process execution and could potentially be replaced by an external implementation. The amount of code appears reasonable to me and the benefit of using an external dependency is unclear to me at the moment, so in this case I would rather avoid adding an external dependency with its dependency tree.
I believe this has been answered, so I'm closing this for now. Please come back with more details if this persists and we can reopen this :+1:

Sure no problem :-) I was just proposing :)


Incomplete documentation around .NET 8 SDK with Visual Studio for Mac

Type of issue
Missing information
Description
From the current documentation, we could suppose everything would work with .NET 8 after we followed steps mentioned below:

Download the Visual Studio 2022 For Mac
...
Install .NET 8 through the standalone installer.
After .NET 8 has finished installing, run dotnet workload install maui in a terminal.

But, according to this ticket, there is one missing step which is to activate .NET 8 SDK under "Preferences/Preview Features":

Hi, in order to use the .NET 8 SDK with Visual Studio for Mac you have to enable this feature in Preferences - Other - Preview Features - Use the .NET 8 SDK if installed.
Also note that the .NET 8 support in Visual Studio for Mac is still in preview and some things may not work or be fully implemented.

Page URL
https://learn.microsoft.com/en-us/dotnet/maui/get-started/installation?view=net-maui-8.0&tabs=vsmac#installation-2
Content source URL
https://github.com/dotnet/docs-maui/blob/main/docs/get-started/installation.md
Document Version Independent Id
9a62d421-d009-30b3-843e-a92bfc21c061
Article author
@davidbritch
Metadata

ID: 7607155a-9a56-d752-c5ac-c3aeb6b128bc
Product: dotnet-mobile
Technology: dotnet-maui

Hi @Kapusch
This is explained when you build your first app, at https://learn.microsoft.com/en-us/dotnet/maui/get-started/first-app?pivots=devices-ios&view=net-maui-8.0&tabs=vsmac, rather than in the installation doc.

Thank you @davidbritch , I understand this is not a miss then.
But still, this section is confusing for scenario where I am redirecting my readers to the installation steps before they can launch my open source project.
Shouldn't we "duplicate" the same information there as well?

@Kapusch Sure, why not. One for tomorrow.


tcmalloc cache too much memory

Refer to: https://gist.github.com/chzhuo/b52fb6c898d55e7bf96149e18892ede1
There is three files:
mem.log:  every line means the size that to malloc
mem.c: malloc all blocks, then free all
After free done, will see about 1GB RES when using tcmalloc.
My testing environment：
using docker image: ubuntu:18.04
then install package: apt install libgoogle-perftools-dev gcc

You can explicitly request TCMalloc to release memory to the system using tcmalloc::MallocExtension::ReleaseMemoryToSystem or increase the memory release rate https://github.com/gperftools/gperftools/blob/fe85bbdf4cb891a67a8e2109c1c22a33aa958c7e/src/gperftools/malloc_extension.h#L294
More on this topic
https://google.github.io/tcmalloc/tuning.html#memory-releasing
But generally my opinion is this behaviour is by design (I realise I cannot speak for the maintainers of TCMalloc)

@lennoxho ， Seems there's no function called ReleaseMemoryToSystem  for tcmalloc@gperftools, but exist in https://github.com/google/tcmalloc.git


Bump cucumberJS version: 7.0.0

The latest version of cucumberJS is available and offer new Gherkin enhancement.
https://github.com/cucumber/cucumber-js/releases/tag/v7.0.0

So far the CucumberJs API remain the same the main breaking change is on the format.
Then the impact is on the project https://github.com/restqa/cucumber-export


Custom assertions

I'm not married to this design at all--it might be easier for the user and to integrate state into the it block to keep track of per-assertion pass/fail status.

Yeah, it's difficult because of the original design.
If we can start over, we can simplify all this to:

<PASSED::> unless something throws
catch AssertionError for <FAILED::>
otherwise <ERROR::>

(or just use established test frameworks!)

Yeah, it's not hard to do this with a bit of redesign and keeping state differently. Just give a holler if you want me to work on it. I'm sure there's a good reason for it, but I guess unittest has to be adapted to work with the runner so that the proper stdout can be printed?

I'm sure there's a good reason for it

Do you mean the reason behind the original design? I don't think there was any "design" and that's the problem. If I remember correctly, Codewars test framework was created for JavaScript in the early days and it was decided that other languages should also have the similar API so custom frameworks for Ruby and Python were also written in the same way.
The only reason we're keeping it is backwards compatibility.

I guess unittest has to be adapted to work with the runner so that the proper stdout can be printed?

I'm not sure how it can be adapted, but I'm interested if you have some idea in mind.
I was thinking maybe we can change how the assertions work only when it's used inside it. That should allow us to switch to the simplified flow I wrote above without affecting the tests written without describe/it, unless I'm missing something.

I don't have an idea for making unittest work with the necessary CW stdout elements but I figured there was some trick that Q uses for this and could be adapted for CW as well.
Yeah, the simplified flow is promising but I think it'd take a bit of a redesign that felt invasive to propose. The current approach prints PASSED tags explicitly for each assertion rather than a single PASSED for the entire block when no exception is raised, so is this behavior acceptable? A lot of CW users have one it block (especially for random tests) and then run hundreds of assertions in it that each get a pass/no pass mark. Assuming I'm understanding correctly!

I figured there was some trick that Q uses for this and could be adapted for CW as well.

No, Q just uses unittest with custom runner/reporter. Qualified Test Framework == Codewars Test Framework and we've been discouraging our customers from using it.

The current approach prints PASSED tags explicitly for each assertion rather than a single PASSED for the entire block when no exception is raised, so is this behavior acceptable? A lot of CW users have one it block (especially for random tests) and then run hundreds of assertions in it that each get a pass/no pass mark.

I think changing to a single <PASSED::> per test case is acceptable. Most of the test frameworks work that way. If the authors wants to provide more information, they can still output some custom messages in between assertions. Or maybe we can make it configurable somehow.

Let's discuss and do small experiments before actually working on it.
Also, we should have some tests set up. Something like:
.
├── codewars_test
│   ├── __init__.py
│   └── test_framework.py
├── tests
│   ├── fixtures
│   │   ├── inside_it.expected.txt
│   │   ├── inside_it.py
│   │   ├── toplevel_assertions.expected.txt
│   │   └── toplevel_assertions.py
│   └── test_output.py
└── setup.py

And compare the actual output against the expected after removing things that's expected to be different like durations.

I like the single PASSED per test, and in fact this might discourage CW users from putting all of their assertions into one it. It does change functionality though, but hopefully in a non-harmful way.
On tests, yes--I was thinking of this when working on the repo.
Right, that makes sense for unittest, but I'm still not quite understanding why the same strategy isn't available for CW--backwards compatibility locks in the custom library, though.

Can you elaborate on the same strategy? I don't think I'm following. Qualified have the same problem.

Isn't it possible to use both unittest with a customer reporter alongside CW/Q unit tests? I know there is a dropdown option, but I'm assuming ultimately they have to emit the same stdout tags that the runner needs to display the results, so I assume that it's possible to plug n play different test libraries with the runner, as long as they ultimately generate the right standard output.

That should be possible. Just need to also include the code to set up the custom reporter and to run the tests.
You actually don't even need to use any code from this repo or unittest and just produce your own outputs.
But these are not officially supported or recommended because it makes it even more difficult to maintain. We have fields like testFramework in the payload because we have different ways of handling code base on the information provided. We already have some hacks in place for backwards compatibility for CW/Q test framework and I'm not going to officially support mixing things.
In the future, Codewars will allow authors to choose the test framework to use just like Qualified.


[HUDI-3192] Spark metastore schema evolution broken

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

This PR is to solve this issue

@xushiyan @nsivabalan Hello,can you please take a review?

@xiarixiaoyao : Can you please review this patch. thanks.

@dongkelun : can you please check if HUDI-3192 and https://issues.apache.org/jira/browse/HUDI-2682 are duplicates. if yes, please mark one of them as duplicate and close it.

@dongkelun : can you please check if HUDI-3192 and https://issues.apache.org/jira/browse/HUDI-2682 are duplicates. if yes, please mark one of them as duplicate and close it.

I think it's a duplicate.HUDI-3192  has been turned off

hi @xiarixiaoyao . thx for looking at this. not sure we can solve this from hudi. the problem happens on spark vanilla to. see my explainations here https://lists.apache.org/thread/9mmrnc5o7w42z723s2yqgcrdpwwtts3x

hi @xiarixiaoyao . thx for looking at this. not sure we can solve this from hudi. the problem happens on spark vanilla to. see my explainations here https://lists.apache.org/thread/9mmrnc5o7w42z723s2yqgcrdpwwtts3x

Hello, I think this PR can explain why it is necessary

hi @xiarixiaoyao . thx for looking at this. not sure we can solve this from hudi. the problem happens on spark vanilla to. see my explainations here https://lists.apache.org/thread/9mmrnc5o7w42z723s2yqgcrdpwwtts3x

I packed and verified it today. It should solve this problem However, adding columns with Hive SQL is not supported

@parisni    we want sparksql tread hudi as DataSource table to have a better performace.
when spark read dataSource table, spark will restore table metadata from table properties(include table schema )
you can see the original code in spark HiveExternalCatalog.restoreTableMetadata

It reads table schema, provider, partition column names and bucket specification from table
properties, and filter out these special entries from table properties.

@dongkelun  we have no way to control the behavie of hive, so i think this pr is ok.  thanks for your contribution.
LGTM， wait for CI  pass

adding columns with Hive SQL is not supported

we have no way to control the behavie of hive

does this mean the hive_sync shall be equal to jdbc/hms and distinct from hiveql when syncing the metastore ?


Can somebody help me with the question that about  connect by node.js module

Can somebody help me with following issue:
When i use the module, i start my db server with server.disable-authentication = false, and i write the code:
var db = new Database('http://<IP_ADDRESS>:8000');
db.createDatabase('mydb', function(err, newdb) {
  if (err) {
    console.log('Failed to create database: %j',
      err);
  } else {
    console.log('Database created: %j', newdb.name);
  }
});

and
var db = new Database('http://username:password@<IP_ADDRESS>:8000');
db.createDatabase('mydb', function(err, newdb) {
  if (err) {
    console.log('Failed to create database: %j',
      err);
  } else {
    console.log('Database created: %j', newdb.name);
  }
});

Then i found they all  print
Database created: "mydb"

why?

I can't replicate your problem. Are you sure authentication is correctly enabled?
Your code fails with UnauthorizedError: Unauthorized for me in the first example when running arangod with --server.disable-authentication false.


Manually select Window title bar/borders/background colors

Description of the new feature / enhancement
Add the ability to manually (or randomly) assign different colors to the title bar, borders, or background of application windows to help differentiate between files.
Scenario when this would be used?
When working on multiple files that appear similar, such as a current year and a previous year's report, being able to pick the color of the title bar or borders would help distinguish which file is which. File names are often long, and between split screens and all the quick menu icons, the full file name isn't always visible at the top. Same goes for Windows Explorer windows - when working on a project there may be files needed in multiple network locations, and having them color coded would make them easier to navigate.
Supporting information
No response

This is neccesary and should be a default Windows Setting. I dont understand why Microsoft dominates the Industry and sleeps on features like this for decades!!!
Anyway some additions:
File Explorer AND General Program Instance/ Window Border Coloring Shemes
Usecase: If the User has opened many Instances/ Windows of the same Program and their placement is on the same position not differenciate between them can avoid copy/ paste/ relocate/ deletion errors

Instances:: Each Instance of a Window should automatically have a different Color Sheme when started/ opened to differenciate between them easily.

The Settings should allow for:

Specific Order of Color: RGB, HEX or HSL for minimally 6 Levels or up to 12 before the color repeats itself.
(Context: Obsidian uses 6 Levels and i want them to look the same so i know without thinking which Instance Number this is)
Random Order of Color

Easy implementation would allow for just a different shade of one color.
Detailed implementation would allow to set the color of the Border independently from the Background or Text color.
If an Instance is closed while multiple others are opened and the closed Instance wasnt the last opened one:

Keep track of the new Order of Instances and change colors accordingly so that all color levels in between are used.
example:
Level 1 Red
Level 2 Orange
Level 3 Yellow
Level 4 Green
Level 5 Blue
Level 6 Purple
If i open 4 Instances and close the 2 the color of Instance 3 should change from Yellow to Orange and the color of Instance 4 should change from Green to Yellow

Items:: Each Item should be colored automatically according to:

Index Number

Or any Column Sorting already available: Name, Type, Date, Created, Last Modified, Status, Size,

Specific Order of Color:

Random Order of Color


Update acyclic to 0.3.7

About this PR
📦 Updates com.lihaoyi:acyclic from 0.3.6 to 0.3.7
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

🔍 Files still referring to the old version number
The following files still refer to the old version number (0.3.6).
You might want to review and update them manually.
upickleReadme/Readme.scalatex

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.lihaoyi", artifactId = "acyclic" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.lihaoyi", artifactId = "acyclic" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, old-version-remains, commit-count:1

Superseded by #494.


Resolve controlKeyTimer name collision

I introduced this bug in c0e915b when I renamed control_key_timer to
controlKeyTimer. It turns out microphone.lua also has a function of
the same name.
To address this, I saw two potential solutions. First, renaming the
function to alleviate the conflict altogether. And second, declaring
functions as local. I was tempted to implement the latter, and
successfully did so locally, but decided against submitting that work as
I'm not fully confident in my understanding of lexical scoping in Lua.
This closes #50

@landongrindheim Thanks for the super-fast fix! :bow:


Add usage collection for queries

This changelist adds usage collection for queries. The collection is done
at the Cursor level. The hope is that collecting at this level, there will
be less code changes, and more uniformed code changes.
A UsageCollector is added to each Query type to store necessary information
to collect usage. This UsageCollector is passed to Cursor, so that the
collection can be done here.

I am running through all tests, and will fix the above check failure.


Add run in componentDidUpdate

Found in next-ga it was only running in the first page render and not on page changes.

Thanks! Sorry for the delay, for some reason I didn't receive the notification about this PR. 🍻


QR-Code Generator

Motivation
To avoid entering the code for adding additional disciplines, it should be possible to scan a QR Code in the app. For content creators, e.g. teachers, it should be possible to create these QR-Codes and share them with their students
Proposed Solution
Add an service to the cms, so when creating the api-key, also a qr code is generated and displayed to the teacher.
Alternatives
If the proposed solution is too much work, we could at least add a link to a online service that creates qr codes, this for example takes the text which should be converted to a QR-Code in the query:
e.g. https://api.qrserver.com/v1/create-qr-code/?size=150x150&data=our-api-key
Additional Context
none

Generally a great a idea, which we should implement in the long run. Currently, creating api keys should only be done by the lunes staff (and is therefore restricted for teachers- this can be defined in the settings). We need to make same (time consuming) adaptations until it can be published to "normal" users as well

Not sure what you mean by publish to "normal" users?
I thought the workflow would be:

Teacher creates api key
Teacher creates qr-code
Teacher shares qr code with their students by printing it, using a beamer or sending it to them per Mail / WhatsApp

At least realizing the alternative solution i proposed would be really helpful to avoid the typewriting of the long api-keys for all students.

The workflow is as follows:

Lunes staff creates API Key for an organization
Lunes staff eventually creates QR code
Lunes staff sends API Key & QR code to organization. They can share that information with their students

PR #279 already implements a shorter API Key.


loss.backward() hangs after burn update 0.12 -> 0.13

Describe the bug
loss.backward() started to hang in my tests for both ndarray and wgpu.
To Reproduce

git clone https://github.com/pointsgame/oppai-rs.git -b burn && cd oppai-rs/zero-burn
cargo test --release -- train_ndarray
It hangs in this line.

Desktop:

OS: linux

Additional context
I haven't tried to extract it to a minimal example, but the test is pretty straightforward. Say if you need it.

@kurnevsky Found the problem, working on it, thanks !

@kurnevsky I fixed my recursive code in #1710 , ran the test in your project and it's fast again. I guess we'll have to release a new patch soon 😅

Thanks! Checked the MR - it's possible to use recursive approach as well, you just need to pass visited nodes as an additional argument. But I have no idea what's going on there, so up to you how it should be :)


Shortlist test

changed H1_std smiles col to base_rdkit_smiles and updated relevant json's so this file works with shortlists
added shortlist_test
it takes ~15 minutes IF the calculated descriptor files are saved
takes ~20 minutes if they should be deleted and recreated during the test
please check the relative / absolute path logic. In clean() I used ../ style paths

At some point we need to think of a way for AMPL to automatically set the bank. Otherwise this test passed for me.


Migrate dev off of vagrant-lxc

Our standard dev environment is a VM configured through Vagrant, and on Linux we use LXC to get "VMs" (containers) that are much more efficient and pleasant than the VMs we fall back to on other platforms. To configure those LXC containers through Vagrant, we use "vagrant-lxc".
Unfortunately vagrant-lxc is basically past EOL. In the medium term we'll want to move off of it.
Some notes on the vagrant-lxc situation:

The "box" we use is this: https://app.vagrantup.com/fgrehm/boxes/trusty64-lxc
It hasn't had a release since 2014. Nor has any other box from this user: https://app.vagrantup.com/fgrehm who also seems to be the author / ex-maintainer of vagrant-lxc itself.
The docs link to https://github.com/fgrehm/vagrant-lxc-base-boxes as a repo for the code to make these "boxes". There's a handy notice there about fgrehm stepping down from maintaining vagrant-lxc, and a link to a fork someone attempted as a successor, https://github.com/obnoxxx/vagrant-lxc-base-boxes ; that fork lasted a little less than a year past the announcement, to 2016.
On the web, I'm not finding much evidence of people using Vagrant with LXC after 2014. There's the Drupal dev VM, optionally; it links to a Xenial box, which is good news, but that user in turn has not updated that or released any other boxes since 2016.

Options for moving off:

LXC is not fundamentally that complicated. I made a perfectly serviceable prod/install test environment in our tools/test-install/, with commit bf5f1b5f2 and thereafter. I'm kind of tempted to see if with a focused day or two I could hook up Vagrant to LXC, i.e. make a replacement for vagrant-lxc. (With more bells and whistles than my test-install setup, but fewer than vagrant-lxc has, e.g. no NFS.)
On the Arch wiki, I learn about vagrant-libvirt. That is promising -- particularly as libvirt supports LXC as a backend, so it might even be possible to use this "vagrant-libvirt" with LXC containers and continue to skip the performance impact of full VMs. Also promising: vagrant-libvirt is actively maintained (compare e.g. closed PRs and open ones), and there are multiple posts from people in the last year describing using vagrant-libvirt.
Sadly, it looks like vagrant-libvirt doesn't currently support LXC; the maintainers are open to a PR for it, but think it would be significant work and don't plan to build it themselves. Still, a maintained thing using full VMs (with KVM) would be a lot better than something increasingly unmaintained.
There's always VirtualBox, like people use on macOS and Windows.
Potentially some non-Vagrant solution, but I don't know offhand of an alternative I expect to be an improvement. (The one almost-alternative to Vagrant that I do know of that might be an improvement is Docker -- but I think that will be helpful only if we've ported Zulip itself to run in the Docker deployment model, which is different from the traditional-Unix-server deployment model it currently runs in. That will probably happen but not in the near term, probably not the medium term.)

My rough plan is to try that first idea of hooking up Vagrant to LXC myself; then if that turns out to be too hard or messy, try vagrant-libvirt with KVM. This isn't urgent, so no plans to do this real soon.
(See #8047 for what caused me to look into this recently. Most of the above is copied from a comment I made there.)

Related:

#7861 dev provision: Add docker compose integration
#7901 Vagrantfile: Bump Ubuntu version to Xenial
#9597 Update vagrant guest images to Bionic
#12357 Vagrantfile: Support Vagrant Docker provider

Fixed by #12357; we’re using the Vagrant Docker provider now.


Nested classes and functions are not checked

If I check a file containing
class A:

    """Test documentation."""

    class B:
        pass

I notice that it will not complain that B does not contain any PyDoc. I guess this is incorrect behaviour?

My two cents on this issue:
Nested classes are public. They are public because names defined inside a class definitions are accessible from outside of the class, such as methods, class methods, class members, etc. For all of these, we consider them public unless they are prepended with an underscore.
Nested function are private. They are private because names inside a function definitions are not accessible from the outside, namely local variables.

Just to make it clear - in "nested class" I meant a class inside a class, not a class inside a function.


Service Worker immediately becomes redundant on refresh

Library Affected:
*workbox-sw
Browser & Platform:
*Chrome 71 for mac os x.
Issue or Feature Request Description:
My service worker works as expected while there are two tabs open, but when I have only one tab open a refresh of that tab doesn't serve anything from the service worker, the previous service worker becomes redundant immediately on refresh and a new one is registered. The result is that in a real world use case I cannot "browse offline". I am using precacheAndRoute as well as some runtimeCaching that all works with two tabs open.
I am including my service worker here for assistance:
self.__precacheManifest = [
  {...} // trimmed purposefully, i can paste these if they are needed.
];

importScripts("https://storage.googleapis.com/workbox-cdn/releases/3.6.3/workbox-sw.js");

workbox.skipWaiting();
workbox.clientsClaim();
self.__precacheManifest = [].concat(self.__precacheManifest || []);
workbox.precaching.suppressWarnings();
workbox.precaching.precacheAndRoute(self.__precacheManifest, {});

workbox.routing.registerRoute(
  /.*\.(?:png|jpg|webp|webm|jpeg|svg|gif)/,
  workbox.strategies.cacheFirst({
    cacheName: 'media',
    plugins: [new workbox.cacheableResponse.Plugin({ statuses: [0, 200] })],
  }),
  'GET',
);
workbox.routing.registerRoute(
  /.*/,
  workbox.strategies.staleWhileRevalidate(),
  'GET',
);

Also, if this is still an issue, could you consider upgrading to the latest v4 release candidate? If you're using, e.g., the webpack plugin, you can do so via npm install --save-dev workbox-webpack-plugin@beta, or just manually swap in the updated CDN URL for the workbox-sw loader library: https://storage.googleapis.com/workbox-cdn/releases/4.0.0-rc.3/workbox-sw.js

@jeffposnick sorry for the delay/lack of response here from our side. I will give the release candidate a whirl in the next few days and report back. Also just want to praise you for how helpful you've been for something that is so very clearly an implementation issue on our end. Really appreciate the extra mile you've gone.

I am facing the same issue. Have you come across any solution?

@thebedroomprogrammer I have discovered an issue in my bundle related to how I was manually splitting out some dependencies using a regex. I am still diving into this because we still haven’t enabled our worker but have at least found that there is a silent error or two that is impacting our users without the worker. I want to fix these up before I re-enable worker because my theory is the silent error is preventing the worker from persisting

Ok I came across this issue when we migrated our project from https://games.gamezop.com to https://www.gamezop.com we did not do any change in the script and started getting very less service worker activation. Thanks for them help though. I will update on this thread if I get any concrete solution.

@jeffposnick sorry for the long delay, finally got around to testing with version 4. I will say the version 4 experience is much better but I'm having the same result in my next.js app. I will provide the details I have gathered here to see if it sparks anything.
Notes

[x] webpack set to maximum pre-gzip chunk size of < 2 mb
[x] upgraded to latest workbox

This is my service worker file:
self.__precacheManifest = [
  {
    "url": "/_next/static/O_OckWoVTaY0N5wsWRLFw/pages/contact.js"
  },
];

importScripts("https://storage.googleapis.com/workbox-cdn/releases/4.1.1/workbox-sw.js");

workbox.precaching.precacheAndRoute(self.__precacheManifest, {});

I've actually trimmed this way down just to see if i could get it working with as minimal as required setup. This is the file that registers the worker in the client:
export const registerWorker = async () => {
  if ('serviceWorker' in navigator) {
    const { Workbox } = await import('workbox-window');

    const wb = new Workbox('/service-worker.js');
    wb.addEventListener('activated', event => {
      // `event.isUpdate` will be true if another version of the service
      // worker was controlling the page when this version was registered.
      if (!event.isUpdate) {
        console.log('Service worker activated for the first time!');

        // If your service worker is configured to precache assets, those
        // assets should all be available now.
      }
    });

    wb.addEventListener('waiting', () => {
      console.log(
        `A new service worker has installed, but it can't activate` +
          `until all tabs running the current version have fully unloaded.`,
      );
    });

    wb.addEventListener('message', event => {
      if (event.data.type === 'CACHE_UPDATE') {
        const { updatedURL } = event.data.payload;

        console.log(`A newer version of ${updatedURL} is available!`);
      }
    });

    wb.addEventListener('installed', event => {
      if (!event.isUpdate) {
        console.log('First install');
      } else {
        console.log('updated install');
      }
    });

    wb.addEventListener('controlling', event => {
      if (!event.isUpdate) {
        console.log('First control');
      } else {
        console.log('updated control');
      }
    });

    wb.addEventListener('externalinstalled', event => {
      if (!event.isUpdate) {
        console.log('external first install');
      } else {
        console.log('external update install');
      }
    });

    wb.addEventListener('externalwaiting', event => {
      if (!event.isUpdate) {
        console.log('external first waiting');
      } else {
        console.log('external update waiting');
      }
    });

    wb.addEventListener('externalactivated', event => {
      if (!event.isUpdate) {
        console.log('external first acvtive');
      } else {
        console.log('external update acvtive');
      }
    });

    wb.addEventListener('redundant', event => {
      // `event.isUpdate` will be true if another version of the service
      // worker was controlling the page when this version was registered.
      if (!event.isUpdate) {
        console.log('Service worker redundant for the first time!');

        // If your service worker is configured to precache assets, those
        // assets should all be available now.
      }
      console.log(event);
    });

    wb.register();
  }
};

I've just tried to log everything out that I could to see what happens.
results:

Side note, if I use skipWaiting/clientsClaim, no redundant message ever occurs:

but that - deleted flag is there and on the next refresh i'll get the exact same console messages.

The problem that I faced was with the precache manifest. So basically the error occurred and service worker became redundant because the links that I had in my precache manifest file was posing cors error when fetched via service workers. I resolved the CORS issue and I was able to get my service worker up and running.

How did you diagnose the issue and confirm the CORS issue? @thebedroomprogrammer thanks for sharing!

@thebedroomprogrammer I will work on confirming that I have actually fixed the issue but your feedback made me look at my script/link tags more closely and I discovered that some of the resources we are attempting to cache had an improper 'crossOrigin' attribute. It seems to be working but I'm going to put it through its paces and report back once I am confident it's not like the other flukes I've had :D.

@thebedroomprogrammer @jeffposnick i have confirmed that the issue is one of our third party scripts we are loading (?) When i fixed our head script tags up I added proper crossOrigin attributes in a few places they were missing and one of those scripts started not responding to our request (throwing a real error in the console about cross origin). So essentially that script was never loaded, but the service worker persisted and worked as expected. Once reverting that change and allowing the script to load the old behavior returns of the service worker immediately marking itself as deleted.
VERY OCCASIONALLY i get a Uncaught DOMException in promise with no stack trace error and always after refresh its gone. Maybe related? The script in question is under 200kb and there is no message about quota exceeded.

Also we do not have a caching strategy for that third party script/route so the service worker shouldn't be requesting it, right?

Sorry for replying so late. I resolved my issue by mainly trial and error. On precaching files I was getting an error in my service worker like this Uncaught (in promise) TypeError: Failed to fetch . On examining the issue I came to know that there were some files in my precache that I was getting from AWS bucket. As both my sites and bucket URLs were different I was getting the CORS error hence the fetch request was failing. I allowed the CORS on my bucket and the error was gone. My service worker was registering and running from then on without any issue.

@brad-decker if there's an error while attempting to precaching files, Workbox should be logging that to the console. https://github.com/GoogleChrome/workbox/blob/v4.1.1/packages/workbox-precaching/PrecacheController.mjs#L191-L198
So if you're not seeing any error logged, then most likely precaching is not the issue. Do you have a demo of this hosted publicly somewhere?

@philipwalton @thebedroomprogrammer @jeffposnick - Back from paternity leave sorry for resurrecting this again. So the third party we use has enabled cors options for this script that is killing the sw. Still the same problem. The thing is we aren't even attempting to cache it or respond to it with our service worker. Removing the script makes the service worker work as expected. I don't have a public demo that I can share at this point. We are pre-launch still.

@jeffposnick @philipwalton @thebedroomprogrammer -- in this case it happens to be that the third party script was manually unregistering service workers as a workaround for a problem they were having with their own implementation. We've asked that they remove it. Thanks for all the support during this crucible of mine!

Glad you got it figured out!

I'm facing the same issue. It only happens on Google Chrome. Already unregistered all service workers but somehow I can't get it to work in Google Chrome. In firefox the service worker works as it should.
What can I do to fix this in Google Chrome?
I'm running on MacOS big Sur and Google Chrome 88.0.4324.96.
It seems not to be tight to a single subdomain, following urls are the same deployment.

https://nextjs-blog-git-pwa.marcofranssen.vercel.app/
https://nextjs-blog-f902dghv2.vercel.app/

Following info I retrieved from chrome://serviceworker-internals/?devtools.
Error: {"columnNumber":-1,"lineNumber":-1,"message":"Failed to register a ServiceWorker for scope ('https://nextjs-blog-f902dghv2.vercel.app/') with script ('https://nextjs-blog-f902dghv2.vercel.app/sw.js'): ServiceWorker failed to install: ServiceWorker failed to handle event (event.waitUntil Promise rejected)","sourceURL":""}
Console: {"lineNumber":0,"message":"Uncaught (in promise) TypeError: Failed to fetch","message_level":3,"sourceIdentifier":1,"sourceURL":"https://nextjs-blog-f902dghv2.vercel.app/sw.js"}

Running the same code on localhost works properly with having a service worker being registered.

I'm facing the same issue. It only happens on Google Chrome. Already unregistered all service workers but somehow I can't get it to work in Google Chrome. In firefox the service worker works as it should.
What can I do to fix this in Google Chrome?
I'm running on MacOS big Sur and Google Chrome 88.0.4324.96.
It seems not to be tight to a single subdomain, following urls are the same deployment.

https://nextjs-blog-git-pwa.marcofranssen.vercel.app/
https://nextjs-blog-f902dghv2.vercel.app/

Following info I retrieved from chrome://serviceworker-internals/?devtools.
Error: {"columnNumber":-1,"lineNumber":-1,"message":"Failed to register a ServiceWorker for scope ('https://nextjs-blog-f902dghv2.vercel.app/') with script ('https://nextjs-blog-f902dghv2.vercel.app/sw.js'): ServiceWorker failed to install: ServiceWorker failed to handle event (event.waitUntil Promise rejected)","sourceURL":""}
Console: {"lineNumber":0,"message":"Uncaught (in promise) TypeError: Failed to fetch","message_level":3,"sourceIdentifier":1,"sourceURL":"https://nextjs-blog-f902dghv2.vercel.app/sw.js"}

Running the same code on localhost works properly with having a service worker being registered.

On my side I have also discovered that adblock seems to block service worke registration as well. So, please keep that in mind.
Disabling it fixed the issue.

I have same issue with service worker when using workbox. Does anyone have the solution for this?

Have the same issue as @huykon. Bad precaching response.
Anyone know how to solve it please ?
Thanks

Hello!
I am still having this issue on next js 14 and using next-with-workbox plugin...
My service worker is registered but I get this error :

An error is thrown because this HTTP request failed :

I don't understand why this request is having /public appended.
We don't need to add "public" to the url to fetch the resource at build time.
I can access http://localhost:3000/sw.js by going on the URL but of course not http://localhost:3000/public/sw.js
I am not sure if it's a CORS issue, I don't think so since I am basically not fetching anything yet from my SW...


Support for mutable arrays

Currently, when attempting
let mut my_array: [u64; 3] = [1, 2, 3];
a[1] = 42;

The compiler errors with This expression is not valid on the left hand side of a reassignment
It would be good to support mutable arrays. This should also be possible in storage, so that:
storage {
    my_array: [u64; 3] = [1, 2, 3],
}

//...
storage.my_array[1] = 42

also works. (Note: storage arrays would be mutable by definition)

Yeah mutable arrays on the stack are not supported yet: https://github.com/FuelLabs/sway/issues/428. We have not prioritized that work yet because Vec is now available.
Arrays in storage are also not supported yet: https://github.com/FuelLabs/sway/issues/1182.
@simonr0204 are you blocked by this? We are aware of these deficiencies, we just have not prioritized the work yet.

Not really blocked, just seen occasions where a statically sized array is just the right way to model something.
@sezna said

"Would you mind filing an issue if there isn’t already one? It should not be too hard. Please label it as critical as we should really be able to do this"

So... I did 🙂

Makes sense! Thanks for filing. I can organize the issues soon :)


Support ENUM type in parquet reader

This PR adds ENUM type support in parquet and fixes logging

will wait for #8000 to merge and then rebase

@yingsu00 @mbasmanova all tests pass. please take a look. thanks!

@qqibrow Thanks. Would you confirm that the new test fails without the code changes?

yes.

@qqibrow Thank you for the confirmation.


Async and Paralell Drawing

If we enable Paralell or async drawing will buff performance when many values and series

Instead lets ignore data located "close"


WYSIWYG fields within an ACF repeater field

Good Day,
Quick question, I have a WYSIWYG field inside a repeater. The docs on Timber mention that in order for the content of a WYSIWYG field to be formatted (tags stripped etc) properly you must call it like:
{{post.get_field('wysiwyg_field')}}

However, later in the docs it mentions that subfields of ACF repeater fields should never be called with the .get_field('some_field') syntax and instead they should be called like repeater.wysiwyg_field
While the repeater.some_field works for all other fields I have tried, the proper formatting for the wysiwyg field is not being outputted.
Anyone run into this issue before and have a solution?
Thanks!

Have you looked at using the wpautop filter?
https://github.com/jarednova/timber/wiki/Text-Cookbook
Alternatively you could add a filter to Twig that pushes text through WordPress' the_content filter.
https://github.com/jarednova/timber/wiki/Extending-Timber

I think you can also use:
{{ repeater.wysiwyg_field |apply_filters('the_content') }}

Thanks, appreciate the help! Looks like this solved the issue.

@vincentGustav - While I am a huge fan of ACF, it's also a good idea to not use ACF functions so that you're project isn't so tightly coupled to that plugin - http://www.billerickson.net/advanced-custom-fields-frontend-dependency/

@corradomatt really interesting read and also a good point. Definitely going to do this on the next project.    Thanks for passing along the link!


Format tokens.

Format tokens locally to resolve issue w/ /list failing after new/unconfirmed token is created.

Pull Request Test Coverage Report for Build 1428

0 of 0   changed or added relevant lines in 0 files are covered.
104 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.5%) to 75.102%

Files with Coverage Reduction
New Missed Lines
%

dist/routes/v2/slp.js
104
45.71%

Totals

Change from base Build 1418:
-0.5%

Covered Lines:
2928

Relevant Lines:
3701

💛  - Coveralls

:tada: This PR is included in version 1.14.5 :tada:
Your semantic-release bot :package::rocket:


cilium-agent connects to local storage with --consul flag provided

If the cilium-agent is called with --consul "<IP_ADDRESS>:8500", the agent connects to the local storage.
It would be helpful to provide a mechanism that allow us to use the --consul and --etcd flags without writing --kvstore <kvstore> otherwise the upgrade from 0.8.x won't be smooth.

I think we should error out if --consul is provided and --kvstore is not equal to  "consul".

I agree with @tgraf.

On top of that, could we error out if a kvstore flag isn't provided? It would ensure that the user explicitly has to specify it.


Fix the Readme so that it has valuable information.

I want to see a better title, description, and maybe a roadmap.

Working on this now.

Included in pre-release v0.0.1-alpha


Innocent and Criminal record are switched: https://www.cleanslatedc.com/#/eligibility/50

https://www.cleanslatedc.com/#/eligibility/50

Looks like a quick fix in the json file. I'll swap it and submit a PR.

Awesome thanks
On Sep 10, 2015 4:58 PM, "David Ginzberg"<EMAIL_ADDRESS>wrote:

Looks like a quick fix in the json file. I'll swap it and submit a PR.
—
Reply to this email directly or view it on GitHub
https://github.com/codefordc/clean-slate/issues/121#issuecomment-139376921
.


Adding MySQL connector and prepare functionality for marketplace

Adding support for MySQL connectors and its prepare query execution for datanimbus marketplace nodes.

Renamed the branch and raising a new PR. Closing this one.


Gists "double load" on refresh

Go to https://ethereum.github.io/browser-solidity/#version=soljson-v0.4.10+commit.f0d539ae.js&gist=ea236029aacdd5a6b443d6a82905f2c8
Refresh the page

All the contracts will have been loaded twice (there will be a "Thank.sol" and a "Thanks.sol1")

that story should fix this:
https://github.com/ethereum/browser-solidity/issues/487


call methylation_segmentation fault

Hi,i have some problem when i run nanopolish call-mythlation for GM12878.
The indexing is normal, but in call-mythlation i met this issue:
nanopolish-0.14.0/nanopolish call-methylation -b GM12878/chr1/GM12878.sort.bam -g human_ref/GRCh38.p13.genome.fa -r GM12878/fast5/chr1_basecall/basecall.fastq -q dam -t 100 > GM12878/chr1/GM12878.methdam_2.tsv

Segmentation fault
I used the version both v0.13.2 and v0.14.0. the two version all have this issue above.
I followed the suggestion you provided：
nanopolish fast5-check -r reads.fastq
and got this :
The readdb file contains 143291 fast5 files
[fast5] ERROR: failed to open
[fast5] OK: opened /GM12878/fast5/chr1_basecall/workspace/Bham/FAB39043-3709921973/LomanLabz_PC_20160923_FNFAB39043_MN17250_mux_scan_Human_1D_ligation_R9_4_23348_ch101_read18_strand.fast5
        [read] OK: found 134405 raw samples for ses
terminate called after throwing an instance of 'std::out_of_range'
  what():  basic_string::substr: __pos (which is 5) > this->size() (which is 3)
Aborted

but i cannot locate which fast5 file leads to the error
I find difference between GM12878 fast5 file and K562 fast5 file, which could be processed by nanopolish without error
K562:
/                        Group
/Analyses                Group
/Analyses/Basecall_1D_000 Group
/Analyses/Basecall_1D_000/BaseCalled_template Group
/Analyses/Basecall_1D_000/BaseCalled_template/Fastq Dataset {SCALAR}
/Analyses/Basecall_1D_000/Summary Group
/Analyses/Basecall_1D_000/Summary/basecall_1d_template Group
/Analyses/Segmentation_000 Group
/Analyses/Segmentation_000/Summary Group
/Analyses/Segmentation_000/Summary/segmentation Group
/Raw                     Group
/Raw/Reads               Group
/Raw/Reads/Read_27792    Group
/Raw/Reads/Read_27792/Signal Dataset {59935/Inf}
/UniqueGlobalKey         Group
/UniqueGlobalKey/channel_id Group
/UniqueGlobalKey/context_tags Group
/UniqueGlobalKey/tracking_id Group

GM12878:
/                        Group
/Raw                     Group
/Raw/Reads               Group
/Raw/Reads/Read_146      Group
/Raw/Reads/Read_146/Signal Dataset {6969/Inf}
/UniqueGlobalKey         Group
/UniqueGlobalKey/channel_id Group
/UniqueGlobalKey/context_tags Group
/UniqueGlobalKey/tracking_id Group

any useful suggestion would be appreciate.

It appears the GM12878 files are one-read-per-fast5 (the old format), rather than multi-fast5. As a test could you try to re-pack the fast5s using single_to_multi to see if it resolves the problem?

Or given Nanopolish 0.14.0 supports slow5, those fast5 can be attempted to be converted to slow5 first. slow5 f2s <single_fast5_dir> -d blow5_dir -a -p <num_processes>. slow5tools f2s may be able to catch the problematic fast5 file perhaps.

Hi, sorry for the late reply, I have tried the method [single_to_multi], and used the new multi_fast5 files doing basecalling, mapping. Then I got this problem again when doing nanopolish.
  #000: H5A.c line 642 in H5Aread(): unable to read attribute
    major: Attribute
    minor: Read failed
  #001: H5Aint.c line 661 in H5A_read(): datatype conversion failed
    major: Attribute
    minor: Unable to encode value
  #002: H5T.c line 4816 in H5T_convert(): data type conversion failed
    major: Attribute
    minor: Unable to encode value
  #003: H5Tconv.c line 3274 in H5T__conv_vlen(): can't read VL data
    major: Datatype
    minor: Read failed
  #004: H5Tvlen.c line 891 in H5T_vlen_disk_read(): Unable to read VL information
    major: Datatype
    minor: Read failed
  #005: H5HG.c line 622 in H5HG_read(): unable to protect global heap
    major: Heap
    minor: Unable to protect metadata
  #006: H5HG.c line 262 in H5HG_protect(): unable to protect global heap
    major: Heap
    minor: Unable to protect metadata
  #007: H5AC.c line 1320 in H5AC_protect(): H5C_protect() failed.
    major: Object cache
    minor: Unable to protect metadata
  #008: H5C.c line 3574 in H5C_protect(): can't load entry
    major: Object cache
    minor: Unable to load metadata into cache
  #009: H5C.c line 7954 in H5C_load_entry(): unable to load entry
    major: Object cache
    minor: Unable to load metadata into cache
  #010: H5HGcache.c line 141 in H5HG_load(): bad global heap collection signature
    major: Heap
    minor: Unable to load metadata into cache
error reading attribute channel_number
nanopolish: src/nanopolish_squiggle_read.cpp:305: void SquiggleRead::load_from_raw(const Fast5Data&, uint32_t): Assertion `this->base_model[strand_idx] != NULL' failed.

I will try the "slow5" method soon to see if it resolves the problem.

Hi, sorry for the late reply, I have tried the method [single_to_multi], and used the new multi_fast5 files doing basecalling, mapping. Then I got this problem again when doing nanopolish.
HDF5-DIAG: Error detected in HDF5 (1.8.14) thread<PHONE_NUMBER>73344:
  #000: H5A.c line 642 in H5Aread(): unable to read attribute
    major: Attribute
    minor: Read failed
  #001: H5Aint.c line 661 in H5A_read(): datatype conversion failed
    major: Attribute
    minor: Unable to encode value
  #002: H5T.c line 4816 in H5T_convert(): data type conversion failed
    major: Attribute
    minor: Unable to encode value
  #003: H5Tconv.c line 3274 in H5T__conv_vlen(): can't read VL data
    major: Datatype
    minor: Read failed
  #004: H5Tvlen.c line 891 in H5T_vlen_disk_read(): Unable to read VL information
    major: Datatype
    minor: Read failed
  #005: H5HG.c line 622 in H5HG_read(): unable to protect global heap
    major: Heap
    minor: Unable to protect metadata
  #006: H5HG.c line 262 in H5HG_protect(): unable to protect global heap
    major: Heap
    minor: Unable to protect metadata
  #007: H5AC.c line 1320 in H5AC_protect(): H5C_protect() failed.
    major: Object cache
    minor: Unable to protect metadata
  #008: H5C.c line 3574 in H5C_protect(): can't load entry
    major: Object cache
    minor: Unable to load metadata into cache
  #009: H5C.c line 7954 in H5C_load_entry(): unable to load entry
    major: Object cache
    minor: Unable to load metadata into cache
  #010: H5HGcache.c line 141 in H5HG_load(): bad global heap collection signature
    major: Heap
    minor: Unable to load metadata into cache
error reading attribute channel_number
nanopolish: src/nanopolish_squiggle_read.cpp:305: void SquiggleRead::load_from_raw(const Fast5Data&, uint32_t): Assertion `this->base_model[strand_idx] != NULL' failed.

I will try the "slow5 method soon to see if it resolves the problem."


Equates not propagating to Decomple View at 9.2.1 and later

Describe the bug
Equate does not work correctly under certain conditions.
An issue that was fixed in 9.2 reappears in 9.2.1 and later.
https://github.com/NationalSecurityAgency/ghidra/issues/2386
To Reproduce
Steps to reproduce the behavior:

Check your Decompile View for a statment like if (x == 1)
Go to the Listing View and update the constant "1" to any appropriate EQUATE constant
Look back at the Decompile View
See error

Expected behavior
Scalar will be replaced by Equate.
Screenshots
Bug reproduction
9.2

9.2.1 and later

Other
9.2.1 and later

Environment (please complete the following information):

OS: Windows10
Java Version: <IP_ADDRESS>
Ghidra Version:  9.2.1 and 9.2.2

I've not been able to reproduce this.  If you can provide a "Debug Function Decompilation" dump (from the menu in the upper right corner of the Decompiler Window), it might help to diagnose.

Attach the "Debug Function Decompilation" dump and the gzf of the exe that caused the problem.
This is a program for studying reverse engineering.
bug.zip

Thanks for the sample. This was patched in Ghidra 10.0, specifically the commit 1743a70ede3f96e627442b845d108b729d5d6974


Allow setting the paint used on icons

For example, if you wanted to draw a stroked icon rather than filled, it is not possible without custom code today. We should allow specifying a foreground paint on icons, which would allow for specifying shaders or stroke/fill styles.

Adding the foreground property to the Icon class should be very straightforward as it's just a wrapper for RichText. This would also give the ability to create shadows on icons. While this is being done I recommend making a small modification to the build method. Currently it looks like this:
return Semantics(
      label: semanticLabel,
      child: ExcludeSemantics(
        child: SizedBox(
          width: iconSize,
          height: iconSize,
          child: Center(
            child: iconWidget,
          ),
        ),
      ),
    );

This would result in some changes to how icons are rendered as a whole but changing it to something like the below code will fix some of the problems with alignment on certain icon packages. For instance, FontAwesomeIcons doesn't center very well as the icons go outside of the bounding box.
return Semantics(
      label: semanticLabel,
      child: ExcludeSemantics(
        child: SizedBox(
            width: iconSize,
            height: iconSize,
            child: FittedBox(
              alignment: Alignment.center,
              child: iconWidget,
            )
        ),
      ),
    );

I also think that a more simplified version of the Icon widget would be to create its own TextPainter to sidestep the creation of an entire RichText component all to render a single character in it's given TextSpan. Just my two cents.

Yeah, I were asked to make outline icon or at least shadow, and I just said - I can't
Whatever I tried, it won't work


Error on TypeScript Watch

node_modules/react-native-linear-gradient/index.d.ts:6:15 - error TS2430: Interface 'LinearGradientProps' incorrectly extends interface 'ViewProps'.
  Types of property 'end' are incompatible.
    Type '{ x: number; y: number; }' is not assignable to type 'ReactText'.
      Type '{ x: number; y: number; }' is not assignable to type 'number'.

6     interface LinearGradientProps extends ReactNative.ViewProperties {

After install and use the lib all works fine but this message appear on my terminal
best regards
Victor C Tavernari

Same here

same here

i think this is fixed


Added Portuguese Brazilian (pt_BR) translation

Hi, please add in your repository.
I added PT_BR (Brazilian Portuguese) translation.
Thx!

Thanks for your contribution. You forgot to update lib/src/supported_locales.dart but I'll do that for you.


Code cleanup

Propagate @deprecated annotation to implementations
Remove redundant "static final" definition from interface
Use Collections.emptyMap instead of Collections.EMPTY_MAP to preserve type

@pavolloffay - Re-raised code cleanup PR here...

Retargeted to v0.31.0 branch: https://github.com/opentracing/opentracing-java/pull/196