Please consider fixing issues detected by repository checker

Notification from ioBroker Check and Service Bot
Dear adapter developer,
I'm the ioBroker Check and Service Bot. I'm an automated tool processing routine tasks for the ioBroker infrastructure. I have recently checked the repository for your adapter snips for common errors and appropiate suggestions to keep this adapter up to date.
This check is based the current head revisions (master / main  branch) of the adapter repository
Please see the result of the check below.
ioBroker.snips
   -
ERRORS:

[ ] :heavy_exclamation_mark: [E026] "{'engines': {'node'>='16'}}" is required at package.json, "{'engines':{'node'>='18'}}" is recommened
[ ] :heavy_exclamation_mark: [E033] @iobroker/adapter-core 2.4.0 specified. 3.1.4 is required as minimum, 3.1.6 is recommended. Please update dependencies at package.json
[ ] :heavy_exclamation_mark: [E035] No devDependency declared for @iobroker/testing. Please add "@iobroker/testing":"4.1.3" to devDependencies at package.json
[ ] :heavy_exclamation_mark: [E150] No "common.connectionType" found in io-package.json
[ ] :heavy_exclamation_mark: [E152] No "common.dataSource" found in io-package.json
[ ] :heavy_exclamation_mark: [E204] Versions "1.3.1, 1.2.1, 1.2.0" listed at common.news at io-package.json do not exist at NPM. Please remove from news section.

WARNINGS:

[ ] :eyes: [W105] Missing suggested translation into ru,pt,nl,fr,it,es,pl,uk,zh-cn of "common.titleLang" in io-package.json.
[ ] :eyes: [W109] Missing suggested translation into pt,nl,fr,it,es,pl,uk,zh-cn of "common.desc" in io-package.json.
[ ] :eyes: [W115] "common.tier" is required in io-package.json. Please check https://github.com/ioBroker/ioBroker.docs/blob/master/docs/en/dev/objectsschema.md#adapter.
[ ] :eyes: [W130] Many "common.news" found in io-package.json. Repositorybuilder will truncate at 7 news. Please remove old news.
[ ] :eyes: [W145] Missing suggested translation into ru,pt,nl,fr,it,es,pl,uk,zh-cn of some "common.news" in io-package.json.
[ ] :eyes: [W181] "common.license" in io-package.json is deprecated. Please define object "common.licenseInformation"
[ ] :eyes: [W184] "common.materialize" is deprecated for admin >= 5 at io-package.json. Please use property "adminUI".
[ ] :eyes: [W184] "common.title" is deprecated and replaced by "common.titleLang". Please remove from io-package.json.
[ ] :eyes: [W513] "gulpfile.js" found in repo! Think about migrating to @iobroker/adapter-dev package
[ ] :eyes: [W853] .npmignore found - consider using package.json object "files" instead.

SUGGESTIONS:

[ ] :pushpin: [S522] Please consider migrating to admin 5 UI (jsonConfig).

Please review issues reported and consider fixing them as soon as appropiate.
Errors reported by repository checker should be fixed as soon as possible. Some of them require a new release to be considered as fixed. Please note that errors reported by checker might be considered as blocking point for future updates at stable repository.
Warnings reported by repository checker should be reviewed. While some warnings can be ignored due to good reasons or a dedicated decision of the developer, most warnings should be fixed as soon as appropiate.
Suggestions reported by repository checker should be reviewed. Suggestions can be ignored due to a decision of the developer but they are reported as a hint to use a configuration which might get required in future or at least is used be most adapters. Suggestions are always optional to follow.
You may start a new check at any time by adding the following comment to this issue:
@iobroker-bot recheck
Please note that I (and the server at GitHub) have always plenty of work to do. So it may last up to 30 minutes until you see a reaction. I will drop a comment here as soon as I start processing.
Feel free to contact me (@iobroker-bot) if you have any questions or feel that an issue is incorrectly flagged.
And THANKS A LOT for maintaining this adapter from me and all users.
Let's work together for the best user experience.
your
ioBroker Check and Service Bot
@mcm1957 for evidence
Last update at Fri, 06 Sep 2024 10:29:14 GMT

Issue outdated due to RECREATE request. Follow up issue #60 has been created.
This issue will be closed.
your
ioBroker Check and Service Bot


Fork you!

FOAAS needed a wordpress plugin. Now it has one: https://github.com/mmatthews1981/wp-foaas

It's official: https://wordpress.org/plugins/wp-foaas/

Sweet


Inconsistent application of metatags in HTML, RTD generation; cache-control

(Reproduced both on a local rtd build as well as on RTD server, see http://pact-language.readthedocs.io/en/latest, github conf.py etc is at https://github.com/kadena-io/pact/tree/master/docs .)
I'm trying to use conf.py to set cache-control tags as follows:
html_context = {
    'metatags' : '<meta http-equiv="Cache-Control" content="no-cache, no-store, must-revalidate" /><meta http-equiv="Pragma" content="no-cache" /><meta http-equiv="Expires" content="0" />'
}

Looking at layout.html, we see {{metatags}} here: https://github.com/rtfd/sphinx_rtd_theme/blob/d24f7ca6b57052eaf0843d56da46e1bfc71aa7f5/sphinx_rtd_theme/layout.html#L16 . So you would think this would show up on all generated pages.
Instead, my metatags only show up in genindex.html and search.html. index.html and my doc htmls (pact-reference.html) have nothing there.
I also tried setting metatags in the doc, as follows:
.. meta::
   :metatags: STUFF

This only gets inserted into my main doc file (pact-reference.html) but is missing everywhere else.
Is metatags being overwritten somewhere? I can't figure out why it shows up only on some pages and not others.
ALTERNATELY: is there any objection to simply adding the cache-control tags directly to the templates? It's really not ideal that RTD docs don't have them, and it means users end up inadvertently seeing old versions of the docs if they don't force-refresh their browser. If there is appetite for this I'm happy to provide a PR.

https://github.com/rtfd/sphinx_rtd_theme/pull/467 might fix this.

Hi, the metatags are injected by sphinx and expose it as metatags for templates https://www.sphinx-doc.org/en/master/templating.html#metatags
As the docs mention, to add custom metatags you need to use the meta directive
.. meta::
   :foo: bar

If you want to add some metatags to all pages, you can use https://www.sphinx-doc.org/en/master/usage/configuration.html#confval-rst_prolog.
Why it works with pages like search and genindex? Not sure, and looks like using rst_prolog doesn't work for the search page as well. But that's a sphinx problem, please report that problem upstream.


Provide libxsmmf library accompanying the MODule file

Providing a libxsmmf.[a|so|dll] library which is accompanying the MODule file (already generated) allows for using LIBXSMM without including the header file and the related implications. Including libxsmm.f and linking against the regular libxsmm.[a|so|dll] is just an additional option for users who prefer working with a compiler-dependent module file.

As of commit 5e8e152, the issue is resolved and tested along with an updated documentation.


no nice output

example data
{'description': u'',
 'emails'<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS> 'firstname': u'adnan',
 'git_aydo': u'',
 'git_github': u'',
 'groups': [u'sales', u'dubai', u'mgmt'],
 'id': u'fatayera',
 'lastname': u'fatayerji',
 'mobiles': [u'971507192009'],
 'skype': u'',
 'telegram': u'971507192009'}

print contoml.dumps(data)
telegram = "971507192009"
description = ""
firstname = "adnan"
lastname = "fatayerji"
git_aydo = ""
emails =<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>git_github = ""
groups = ["sales", "dubai", "mgmt"]
skype = ""
id = "fatayera"
mobiles = ["971507192009"]

is not the expected behaviour

Auto-formatting is actually not turned on by default. I added an optional boolean parameter to contoml.dumps() that auto-formats before it writes out.
You can now do:
>>> import contoml

>>> d = {'description': u'',
 'emails'<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS> 'firstname': u'adnan',
 'git_aydo': u'',
 'git_github': u'',
 'groups': [u'sales', u'dubai', u'mgmt'],
 'id': u'fatayera',
 'lastname': u'fatayerji',
 'mobiles': [u'971507192009'],
 'skype': u'',
 'telegram': u'971507192009'}

>>> print(contoml.dumps(d, prettify=True))
description = ""
firstname = "adnan"
lastname = "fatayerji"
git_aydo = ""
groups = ["sales", "dubai", "mgmt"]
skype = ""
emails =<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>id = "fatayera"
git_github = ""
telegram = "971507192009"
mobiles = ["971507192009

the arguments should be sorted
e.g. emails comes before skype
it should be prettified standard !!!
why would anyone want to turn this off
its the goal of this library

The original goal of the module is to read/update TOML files while doing as little as possible to the original structure of the text. Auto-formatting is a requirement that I was made aware of only very recently, so having the module automatically re-format the user's TOML data without their explicit command goes against the original selling point of the module.
I'll add the lexicographic key sorting to the implemented prettifiers.
Are you sure you want the module to prettify by default during calls to dumps()? I'm quite sure this would confuse everyone else who could use this module.

I am going to close this issue for now. Please re-open if you have further comments.


[VS Code] Could not connect to the server. Live Share requires openssl 1.0 for authentication, please verify you have the dependency installed.

Error:
Could not connect to the server. Live Share requires openssl 1.0 for authentication, please verify you have the dependency installed.
Steps to Reproduce:

Version Data

extensionName
VSLS

extensionVersion
1.0.67

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.33.1

platformName
Linux

platformVersion
4.19.0-4-amd64

@sirgloveface Thanks for reporting the issue. What flavor of Linux do you have?
You may find Linux installation details on https://docs.microsoft.com/en-us/visualstudio/liveshare/reference/linux. Please let us know if they're not sufficient.
@Chuxel Is this error expected?

Hello, my distro is Debian!  Version buster 9.0.
I follow the Linux instalación but it does work
Tony García

On Apr 25, 2019, at 5:46 PM, Ilya Biryukov<EMAIL_ADDRESS>wrote:
@sirgloveface Thanks for reporting the issue. What flavor of Linux do you have?
You may find Linux installation details on https://docs.microsoft.com/en-us/visualstudio/liveshare/reference/linux. Please let us know if they're not sufficient.
@Chuxel Is this error expected?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

is anybody here?

@sirgloveface Did you try installing libssl1.0.0 ?

I have the same issue on Debian buster(10, not 9.0), i've installed libssl1.0.0 from the debian jessie repo, but it still doesn't work.

dupe #2030

LiveShare version 1.0.181 was just released to the marketplace and contains a fix for this issue.

Thank's a lot for all guys....
On Fri, May 10, 2019 at 12:47 PM Daniel Lebu<EMAIL_ADDRESS>wrote:

LiveShare version 1.0.181 was just released to the marketplace and
contains a fix for this issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/live-share/issues/2019#issuecomment-491355269,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF2DS4N7W5UL6VLDBZVLXATPUWRI3ANCNFSM4HIHKVQA
.


Strength meter for password sign up

System information
the operating system used, including its version, e.g. Windows 10, Ubuntu 18.04
What browser you are using e.g. Chrome, Firefox, Edge etc.
Windows 10 LTS
6-10-22
Describe the feature
A clear and concise description of what the feature is.
Read read me.md
Additional context
Add any other context or screenshots about the feature request here.
https://www.solodev.com/blog/web-design/creating-a-password-strength-indicator.stml#
Read after school.

nah not doing this, way too complicated lol...........


how to cmake xgboost dll with debug info

i want to build xgboost dll with debug info in order to debug xgboost in python
i can only find release version in xgboost doc
how can i use cmake to build xgboost dll with debug info?

Please visit CMake document: https://cmake.org/cmake/help/latest/variable/CMAKE_BUILD_TYPE.html .


feature request: add option for multiple data directories

currently (as much as I understand) you can configure only one directory that is tracked in dvc
it would be useful to be able to specify more then one directory (for repositories that contain more than one project)

@ophiry I thought about that but don't have clear requirements so far. Could you please give more details?
A few possible solutions:

Why are two directories not enough? For example: data/raw and data/db_dump?
Would you like to use these two directories as completely separate projects\repositories?
Let say I work on data/db_dump and you share data/db_dump with me and you have an additional repo data/raw.

What is your scenario?

The scenario is adding the data files to a repository with multiple projects
proj1
proj2
...
I prefer to have the data files of each project as a sub folder of the project folder (which is the current standard for all other project files, like configurations, small data files, etc.)
proj1/data
proj2/data
...


BUG: FIT export fails when metadata contain non latin-1 ASCII character (like micro)

In the metadata of gaiaedr3 some wavelenght (like pseudo color) have a unit in "per micro-meter" introducing the non latin-1 ASCII character micro:

Dec
Hex
Char
Octal
Rawencoding
UTF8encoding
HTMLentity
Description

181
00b5
μ
265
0xB5
0xC2,0xB5
µ
Micro sign

A simple solution could be to modify the metadata.
A better to add a latin-1 rule in the FITs exporter (made by hand)
A final solution would be to use a library to export to FITs files.

in order to be compatible with other library (like astropy) we could rename the per micrometer unit into:
um-1 see: https://docs.astropy.org/en/stable/units/#module-astropy.units

In order to add a FITs compatible unit we could use the FIT format from astropy: https://docs.astropy.org/en/stable/api/astropy.units.format.Fits.html#astropy.units.format.Fits this guarranty compatibility.

A better way to go would be to export in a similar way than the following:

from astropy.table import Table
import astropy.units as u

a = [1., 2., 3.] * u.m / u.s # the colum a with its units
b = [4., 5., 6.] / u.micron # the column b with its units

table = Table([a, b], names=('a', 'b'), meta={'name': 'dummy'}) # the table to export with its table metadata

# exports to FITs format
table.write('test.fits', format='fits')

Note: It also may be worth to write a tutorial for our user.


Ghostscript version from homebrew is too new

Close

Please close this issue, I have made something wrong

Hi @bbqaaq,
If you install ghostscript through homebrew the entire solution is in this following folder:
/usr/local/Cellar/ghostscript/{VERSION}/ the most simple thing to do is to create a symbolic link in /usr/local/lib that point to  /usr/local/Cellar/ghostscript/{VERSION}/lib/libgs.dylib.
Hope that this will help you.

Ok I provide to close feel free to open new issue if you need other help.

@NickNaso Sorry that I have made sth wrong.
Turn out the problem can be solved by reinstalling the package.
But I found that another issue #47 happens on my side too, as Im using 9.27 also.
See if that issue is related to gs 9.27 with ghostscript4js.
Thanks!


chore: find kor exceptions script

What this PR does / why we need it?
This PR add a hack folder into kor, to include future useful scripts for contributors.
It also includes the initial script find_exceptions.sh to discover false-positive default resources, the output of this script could be merged to pkg/kor/exceptions to be ignored by kor.
This script should be used in issues like #236, #240 for intsance.
Output Example
$ ./find_exceptions.sh
Processing completed for ConfigMaps, output saved to exceptions/configmaps.json
Processing completed for Secrets, output saved to exceptions/secrets.json
Processing completed for ServiceAccounts, output saved to exceptions/serviceaccounts.json

$ cat exceptions/configmaps.json
{
  "exceptionConfigMaps": [
    {
      "Namespace": "kube-system",
      "ResourceName": "bootstrap"
    },
    {
      "Namespace": "kube-system",
      "ResourceName": "cluster-config-v1"
    },
    {
      "Namespace": "kube-system",
      "ResourceName": "openshift-service-ca.crt"
    },
    {
      "Namespace": "kube-system",
      "ResourceName": "root-ca"
    }
  ]
}

PR Checklist

[ ] This PR adds K8s exceptions (false positives)
[x] This PR adds new code
[ ] This PR includes tests for new/existing code
[x] This PR adds docs

GitHub Issue
[XX-XX]

Notes for your reviewers
The find_exceptions.sh does not squash resources that appear in multiple namespaces, for example, if a ConfigMap is created in every namespace, the output will not display as so:
    {
      "Namespace": "*",
      "ResourceName": "kube-root-ca.crt"
    }

The script eases the process to map the default resources, but requires manual intervention (regex, merging, etc).

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

Project coverage is 40.96%. Comparing base (852a6f9) to head (f31cc96).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #288   +/-   ##
=======================================
  Coverage   40.96%   40.96%
=======================================
  Files          58       58
  Lines        2910     2910
=======================================
  Hits         1192     1192
  Misses       1530     1530
  Partials      188      188

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

Rebasing issues, I’ll re-submit it.

We should add a way to sort the JSON (alphabetize it) as well.


serialize.py: np.float128 and np.complex256 are not implemented on some platforms

np.float128 and np.complex256 do not exist for numpy on some platforms (including Windows, ARM-based MacOS, which breaks scripts when trying to import them.
Deleting np.float128 and np.complex256 on lines 191, 192 in serialize.py fixed the issue for me.
https://github.com/vivarium-collective/vivarium-core/blob/aae63ff9dc3d26a2fea8fb8d74ccf58bce24fe67/vivarium/core/serialize.py#L191
A couple of other examples:
https://github.com/mpi4jax/mpi4jax/issues/112
https://github.com/winpython/winpython/issues/613

It looks like float64 is the default for numpy, so we'd only get np.float128 if someone specified that explicitly, which seems unlikely. We could remove these and note in the docs that we don't support those data types?

who put those in and why? Can you check the git blame @cyrus-bio?
Maybe we can check if the type exists, and only use it if it does?

From a cursory search it seems that most people just use np.float64 at most, or do something along the lines of:
if hasattr(np, 'float128'):     do A else:     do B

I think I put those in. They just tell Vivarium that the numpy serializer should be used for objects of type np.float128. If someone does provide a np.float128 object after we remove the np.float128 reference from this file, nothing will break. Vivarium will just take slightly longer to find the right serializer (it will loop through the serializers instead of looking the right serializer up in a dictionary)

Sounds like we can safely remove it then. @cyrus-bio -- want to open a pull request in vivarium-core?


How to download the project RazorPagesMovie60?

"Tutorial: Get started with Razor Pages in ASP.NET Core" from the following url:
https://docs.microsoft.com/en-us/aspnet/core/tutorials/razor-pages/razor-pages-start?view=aspnetcore-6.0&tabs=visual-studio
has a section "Troubleshooting with the completed sample" and that states that we can "download completed project" from GitHub.
But I do not see a project under that name, it is actually a folder -  does that mean that I have to download the whole "dotnet/AspNetCore.Docs" in order to download that?

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

ID: a25bf940-8977-fec4-461b-542a9f550fc8
Version Independent ID: 5492c641-af15-64b8-beaa-ddf511be73e6
Content: Tutorial: Get started with Razor Pages in ASP.NET Core
Content Source: aspnetcore/tutorials/razor-pages/razor-pages-start.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

Hello @hsnprojects ... That's correct. Take the ZIP of the repo, decompress, and navigate to the sample per the How to download a sample section at ...
https://docs.microsoft.com/aspnet/core/introduction-to-aspnet-core#how-to-download-a-sample


Simplify asserts introduced in #512.

Note that the second one was incorrect.

On second thoughts I think they are equivalent, but simpler is probably better.

Could simplify to a single static assert with ==, but the current gives more descriptive error messages

Yes, originally we just had the equality but needed the more detailed error message for debugging!


Processes for climatology use cases

Processes for climatology use cases as defined with @jwagemann this week.
Reviews are appreciated.
cc: @claxn

This PR is now open for final review. @jwagemann


Rootless mariadb with persistent storage via bindmount: permissioned denied

Is this a BUG REPORT or FEATURE REQUEST? (leave only one on its own line)
/kind bug
Description
I'm having trouble getting a mariadb instance running with rootless podman.
When I run this, I get the following issue:
chown: cannot read directory '/var/lib/mysql/data': Permission denied

Steps to reproduce the issue:

Make a directory to hold the database

mkdir $HOME/mysql-data
sudo chown -R mysql:mysql mysql-data

Start the database

podman run --security-opt label=disable --rm  -v $HOME/mysql-data:/var/lib/mysql/data:z -e MYSQL_USER=user -e MYSQL_PASSWORD=pass -e MYSQL_DATABASE=db -p 3306:3306 mariadb/server

Describe the results you received:
chown: cannot read directory '/var/lib/mysql/data': Permission denied

Describe the results you expected:
Mariadb container starts with correct permissions. Note
sudo chown 100998:100997 mysql-data

Fixes the issue
Additional information you deem important (e.g. issue happens only occasionally):
Output of podman version:
podman version 1.9.2

Output of podman info --debug:
❯ podman info --debug
debug:
  compiler: gc
  gitCommit: ""
  goVersion: go1.14.2
  podmanVersion: 1.9.2
host:
  arch: amd64
  buildahVersion: 1.14.8
  cgroupVersion: v1
  conmon:
    package: conmon-2.0.16-2.fc32.x86_64
    path: /usr/bin/conmon
    version: 'conmon version 2.0.16, commit: 1044176f7dd177c100779d1c63931d6022e419bd'
  cpus: 8
  distribution:
    distribution: fedora
    version: "32"
  eventLogger: file
  hostname: desktop-fedora
  idMappings:
    gidmap:
    - container_id: 0
      host_id: 1000
      size: 1
    - container_id: 1
      host_id: 100000
      size: 65536
    uidmap:
    - container_id: 0
      host_id: 1000
      size: 1
    - container_id: 1
      host_id: 100000
      size: 65536
  kernel: 5.6.13-300.fc32.x86_64
  memFree: 864075776
  memTotal:<PHONE_NUMBER>0
  ociRuntime:
    name: runc
    package: containerd.io-1.2.13-3.1.fc31.x86_64
    path: /usr/bin/runc
    version: |-
      runc version 1.0.0-rc10
      commit: dc9208a3303feef5b3839f4323d9beb36df0a9dd
      spec: 1.0.1-dev
  os: linux
  rootless: true
  slirp4netns:
    executable: /usr/bin/slirp4netns
    package: slirp4netns-1.0.0-1.fc32.x86_64
    version: |-
      slirp4netns version 1.0.0
      commit: a3be729152a33e692cd28b52f664defbf2e7810a
      libslirp: 4.2.0
  swapFree:<PHONE_NUMBER>
  swapTotal:<PHONE_NUMBER>
  uptime: 140h 13m 49.62s (Approximately 5.83 days)
registries:
  search:
  - registry.fedoraproject.org
  - registry.access.redhat.com
  - registry.centos.org
  - docker.io
store:
  configFile: /home/michael/.config/containers/storage.conf
  containerStore:
    number: 1
    paused: 0
    running: 1
    stopped: 0
  graphDriverName: overlay
  graphOptions:
    overlay.mount_program:
      Executable: /usr/bin/fuse-overlayfs
      Package: fuse-overlayfs-1.0.0-1.fc32.x86_64
      Version: |-
        fusermount3 version: 3.9.1
        fuse-overlayfs: version 1.0.0
        FUSE library version 3.9.1
        using FUSE kernel interface version 7.31
  graphRoot: /home/michael/.local/share/containers/storage
  graphStatus:
    Backing Filesystem: extfs
    Native Overlay Diff: "false"
    Supports d_type: "true"
    Using metacopy: "false"
  imageStore:
    number: 2
  runRoot: /tmp/1000
  volumePath: /home/michael/.local/share/containers/storage/volumes

Package info (e.g. output of rpm -q podman or apt list podman):
podman-1.9.2-1.fc32.x86_64

Additional environment details (AWS, VirtualBox, physical, etc.):
physical

podman run --security-opt label=disable --rm  -v $HOME/mysql-data:/var/lib/mysql/data:z -e MYSQL_USER=user -e MYSQL_PASSWORD=pass -e MYSQL_DATABASE=db -p 3306:3306 mariadb/server
No reason to disable SELinux here. And you figured out the solution.  Anything in your homedir is owned by "root" in your container.  So if you want the database to be owned by the mariadb user, then you need to chown it to match the correct user.

Ah, I think I was just confused about the mapping of the uid of the "root" container user to host, I expected it to match the mysql id outside of the container but I'm realizing that was a flawed assumption. I can close

You inspired me to write my third blog today.  If you want to preview it, please send email to<EMAIL_ADDRESS>I think I cover what you did to figure this out.  BTW Good detective work by you, other then disabling SELinux. :^)


Unable to view Excel worksheets. "General exception while processing"

Describe the bug
I'm trying to list worksheets in a workbook in Microsoft Graph Explorer
To Reproduce

Go https://developer.microsoft.com/en-us/graph/graph-explorer
Sign in
Grant yourself Files.ReadWrite permissions
Make a GET request to: https://graph.microsoft.com/v1.0/me/drive/items/<replace_me>/workbook/worksheets
(update <replace_me> with an excel file you have in your one drive)
Get the following response: Internal Server Error - 500 - 275 ms

{
    "error": {
        "code": "generalException",
        "message": "General exception while processing",
        "innerError": {
            "date": "2024-07-15T14:48:29",
            "request-id": "3b2f22df-944f-48bb-bc64-bb44336bc465",
            "client-request-id": "537a1bbf-21a8-0c28-e514-c618a0b1a509"
        }
    }
}

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Additional context
I tried to do this via Node and wasn't successful so I looked to get some resolution via the explorer. Received the same response:
async function getWorksheets(accessToken: string, userId: string, fileId: string): Promise<any> {
  const client = await getAuthenticatedClient(accessToken);
  const worksheets = await client.api(`/users/${userId}/drive/items/${fileId}/workbook/worksheets`).get();
  return worksheets;
}

The file id i was using: 6BB86F66F0707C42!122

Hi @eugene-kim , could you please confirm if you are still experiencing this issue?

@ElinorW yes. Are you asking because there was a fix in place?

@eugene-kim no, but I believe it's an issue with the Excel API's.
Please post this on the Microsoft Graph Q&A forum where the appropriate team will be able to help

@ElinorW I'm trying to post in the forum but it says I'm violating the Code of Conduct.
All I did was post the text of the original question and referred back to this issue.
Are you able to assist?
Thank you

@eugene-kim,
that's strange. I've tried to post it as well but I got the same error. 😞
Let me try and find an alternative

@eugene-kim if you have a paid azure subscription or an m365 tenant, you can create a support ticket instead
https://developer.microsoft.com/en-us/graph/support

I'm trying to create a support request but I'm prevented from doing so. I've put in my details and I only see self help solutions. Do you have any suggestions on how I can proceed?
I have the standard support plan

Hey @eugene-kim
click on the 'Return to support request" and then click Next

Thank you! I was able to create a support ticket

Closing this for now. If you encounter any issues on Graph Explorer, please feel free to create an issue


Create Getters on service's maps

I saw in quite a lot of files this boring piece of code to access value of maps and return an error if not found:
https://github.com/mesg-foundation/core/blob/bd84ad1d54eadfb8aa2bd79ac0c7c68f775f9c03/execution/complete.go#L13-L19
I suggest to create getters that will return the expected data and the error if not found.
We should have getters for:

[ ] service's event. func (service *Service) GetEvent(eventKey string) (*Event, err)
[ ] service's task. func (service *Service) GetTask(taskKey string) (*Task, err)
[ ] output of service's task. func (service *Service) GetTaskOutput(taskKey string, outputKey string) (*Output, err)
[ ] maybe on all map of Service?

@mesg-foundation/core what do you think?

This validations also shouldn't be made in Complete(). We should first check data for execution and only then call Complete() with valid data in the api handlers. Passing an invalid data to Complete() doesn't make much sense.
event and execution packages needs to be more agnostic and only do their own jobs.


Correction for raw affiliation Food and Healthy Living Group,  Aeres University of Applied Sciences, The Netherlands

Correction needed for raw affiliation Food and Healthy Living Group,  Aeres University of Applied Sciences, The Netherlands
raw_affiliation_name: Food and Healthy Living Group,  Aeres University of Applied Sciences, The Netherlands
new_rors: 03jnx2c74
previous_rors: 015d5s513
works_examples: W2905682528
contact: 3d31f6be0bc36fc0bf1716fc3ef72dbf:019aa20aab2b4d20 @ aeres.nl

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


I want to help

Hi... I'm Peter
I enjoy to use your software... thanks for all your effort.
I really would like to contribute with some minor improvements. I'm not a good programmer but I want to help.
If you agree, could you give me some basic orientation/guides?
Compiling, software to use, tips, etc.
Best regards.

Sure, if you hop on the discord server (https://discord.gg/Bya5HaA) I'll be able to fill you in on anything you want information on. College classes are draining a lot of my time as of late, but I'll be online later in the day almost daily.

I see that you're in. Closing.

Thanks


Hook for writing final substituted config to disk

When using a SubstitutingSourceProvider or other ConfigurationSourceProvider that modifies the byte stream of the config file read from disk, it would be helpful to have a hook to write the final modified byte stream to disk (presumably somewhere in tmpDir).
I think the "final" byte stream is generated in ConfigurationFactory.build(). I don't have a specific design in mind for such a hook, but what I'm looking for is a mechanism to provide a file path where the ConfigurationFactory will attempt to write the stream once it gets created.
This would be useful particularly in the case where the service fails to start up due to some syntax problem with the configuration file. If the stream has been modified, it can be difficult to tell from the source config what's wrong.
I'd be willing to do development work if somebody with more complete knowledge of dropwizard's config machinery can suggest a particular design that would be generically useful.

My preference would be to just get this via a ConfigTask and/or a new config command. We have both of these in our implementation at Cvent and it's worked quite well so far.

@brentryan - Can you go into a little more detail about that? I'm not familiar with ConfigTasks, and it looks like this might be an interface that no longer exists? Is this the same thing as a Task accessible through the admin http port?

@jklukas I think @brentryan is referring to (admin) tasks: http://www.dropwizard.io/0.9.2/docs/manual/core.html#tasks
Generally :+1: for this feature (either as a task or on startup).

a bit similiar issue: #1156

While I too think this is a good idea, and very useful for debugging, be
careful about how you expose credentials. Depending on how and where this
information is exposed, you may end up revealing sensitive information that
was segregated for a reason. Just my 2 cents.
Glenn
On Fri, Apr 8, 2016 at 9:29 AM Patryk Najda<EMAIL_ADDRESS>wrote:

a bit similiar issue: #1156
https://github.com/dropwizard/dropwizard/issues/1156
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/dropwizard/dropwizard/issues/1514#issuecomment-207432445

@glennmcallister Very good argument, thanks for weighing in! Making this feature a task which is not available by default might be the best solution then. So the developers can decide themselves if they want to expose that functionality.

An admin task that needs to be configured sounds like a reasonable approach.

+1


Update this test with the correct dag.Engine() newly generated type

@marcosnils can you clarify which type needs updating? I can't seem to find any test that needs an updated test.
In this TestEngineError, the values are discarded, so we don't need any conversion, but maybe there's another usage?
https://github.com/dagger/dagger/blob/64165b972e6caf7a20c56380ea4515d770ee1dea/core/integration/module_test.go#L2447-L2450

This has been addressed already as a v0.14 follow-up https://github.com/dagger/dagger/pull/8901 :pray: . I'm closing this one


[SPARK-18471][CORE] New treeAggregate overload for big large aggregators

What changes were proposed in this pull request?
The zero for the aggregation used to be shipped into a closure which is
higly problematic when this zero is big (100s of MB is typical for ML).
This change introduces a new overload of treeAggregate which only ships a
function able to generate this zero.
How was this patch tested?
Test suite for the core and mllib modules launched locally.

@srowen here is the companion PR to #16037.

Can one of the admins verify this patch?

This is not really related to the other change or the JIRA though. It also doesn't seem to address the problem as you describe it elsewhere. It's not that the zero value is big (a sparse 0 vector serializes into something tiny) but that some other representation gets into the closure?

It is related as if this PR get accepted, then the fix for PR to #16037 becomes trivial: replace treeAggregate with a call to treeAggregateWithZeroGenerator( () => (Vectors.zeros(n), 0.0)), in which case only the n gets caught in the closure if I'm not mistaken. This fix then gets much easier to port to other similat use cases in MLlib.

If you're right here, then https://github.com/apache/spark/pull/16037 does nothing, right?
In which case my understanding at https://github.com/apache/spark/pull/15905#pullrequestreview-8844854 is wrong.
But if your other change does work, and that would be explained by my understanding, then this doesn't help.
At https://issues.apache.org/jira/browse/SPARK-18471?focusedCommentId=15670795&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-15670795 you describe aggregating a dense vector, but this is different from having a dense vector zero value.

Maybe we could have a live talk/chat this evening Paris time / Morning West Coast time?
If I'm not mistaken both  #16037 and this change fix the issue at two different levels.

#16037 consider the the is specific to ML and that we can & should work around it in MLlib
This change consider the root cause is an inefficiency of treeAggregate in core whose fix would enable a better fix in MLlib. I updated this PR to reflect this.

I might have misunderstood your comment
https://github.com/apache/spark/pull/15905#pullrequestreview-8844854

(I am in the UK, but let's keep the discussion online here)
OK, I had assumed that Vectors.zeros makes a sparse vector. It does not; it makes a dense vector. If it were sparse, then it would be tiny to serialize. Hence I didn't understand this to be the issue, and though you were saying that another copy of a dense representation was accidentally in the closure.
However, this means there's a more direct solution without changing or adding to the public API: pass a sparse vector (of zeroes) as the zero value. It looks like this should work, even be a little faster on the first pass when the gradient is 0. If for some reason something doesn't like the sparse vector, it's easy enough to call .toDense such that it's only expanded once and not on the driver.
In that case, I would not make the change in this PR, and change https://github.com/apache/spark/pull/16037 to make the initial vector sparse only.

No this does not do the trick as the result of the aggregation IS dense. And the zero in (tree)aggregate has the same type as the result. Said otherwise, in L-BFGS, we do aggregate vectors that are each pretty sparse but whose aggregation is dense as they have different support. So taking a dense vector as the zero of the aggregator make perfect sense, adding a sparse contribution to a dense aggregator yield a dense aggregator and this is what is desired. We just need not to waiste bandwith by sending this huge zero to executors.
If you do not want to change or add a public API in core, we might contribute a treeAggregateWithZeroGenerator to MLlib using a third way to solve the issue: use an Option[DenseVector] as the aggregate type and unwrap it as a dense zero bby wrapping the seqOp and comboOp, we have a draft of that locally...

Yes, but the result is inherently dense -- not going to be many zeroes in the gradient if any. There's no way around that. The discussion has been about the initial value, the zero vector, right? That you agree is sparse. I don't see what the problem is then. A huge sparse 0 vector is not huge when serialized.

Yes sure the zero Vector is very sparse. Bu ti do not get your suggestion ? I see no way to pass a Sparse Vector as zero and get the type of vector to change underway to Dense Vector with only operations on Sparse vectors.
What do you propose ? We do want to accumulate the Sparse Vector built from each datapoint into a Dense Vector. And because the type of the zero is the type of the return then we need to use a Dense vector as the zero, doesn't it ? You might have seen a way out of this I haven't, have you ?

It might be as simple as writing Vectors.sparse(n, Seq()) as the zero value. Everything else appears to operate on a Vector that's either sparse or dense then. (Of course, the first call to axpy should produce a dense vector, but, that's already on the executor.)
The method does ultimately want to return a dense vector, and does insert a cast that assumes it's already a dense vector. If that's a problem, it's easy to insert a call to .toDense on the Vector at the end, which is a no-op if it's already dense, which definitely produces a dense vector.

THe big doubt I have is on Of course, the first call to axpy should produce a dense vector, but, that's already on the executor: the other operand is Sparse and has to be sparse, and this axpy call between to sparse vectors has to return a sparse vector. And we do not want to cast at the end because the performance penalties of using sparse instead of dense will already have been paid.
Fully exploring the trade-off between sparse and  dense aggregation os ot-of-scope and likely very application specific.  I'll have a new look at the problem trying to manage a work around along the line of Sparse vectors, but this will not solve the typing issue as far as i can see for now.

Right now, everything is dense here, right? That's the worst case. Your goal is to avoid serializing a dense zero vector and I say it can just be sparse, which solves the immediate problem. From there, some operations may or may not result in sparse vectors, but that's not relevant -- it can only be better than being all dense, which is the current case. It matters at the end because the return type is dense, but, making it dense is easy. Paying the cost of copying to a dense representation is the only downside, but that's small compared to the saving in serialization (I presume).
I don't understand the problem you're suggesting? are you saying that you don't want any operations to be on sparse vectors? I'd leave that choice to the implementations, but, if you're worried about it, simply force the argument of the seqOp to become dense. Then, the only change is the smaller serialization and everything should be identical afterwards.

Actually aggregating (and thus sending on the network) on quite dense SparseVectors with 10s of million elements is not to taken lightly. This would required serious benchmarking. What I tell is that we can not do that lightly because we know that aggregating  acc += partial  when partial is sparse and acc dense is efficient locally (i.e. within a partition) and that after a partition has been aggregated the vector is already quite dense (enough so that the dense and sparse representation have similar size).
Anyway if you consider this is not worth an new API and an optimization in core, the discusion should probably go on on the MLlib only PR #16037 for which we already have a specific work around.
I just naively assumed  that aggregating over big accumulator was typical enough to get some support from core, if this is not the case, I'll fall back on a MLlib only workaround.
Feel free to close this PR is you deem it (ir)relevant.

I see, you are saying that encoding an actually-dense vector as sparse is somewhat less efficient, and if the implementations happened to make that choice given sparse input, could be bad. In that case, just short-cut the problem and use .toDense to force a dense representation immediately.

Without understanding the specifics of the ML part here - wont the actual impact of a large dense vector on Task 'bytes' be minimal at best ?
We do compress the task binary; and 1B zero's should have fairly low compressed footprint, no ?

@mridulm we don't know how to monitor the size of the serialize task. Sure it would not 10MB due to all those zeros. But we nonetheless measure a significant increase in performance and (more importantly) in stability when using the workaround and our custom TreeAggregatorWithZeroGenerator
@srowen when the density of a Sparse Vector increases it became very inefficient, not only because it is using almost twice the size of memory, but mainly because you fragment memory access on overload the GC :(
See #16078 for a generic wrapper around treeAggregate with uses Option[U] do denote the zero of U by None and generate a full representation by need when calling seqOp or comboOp.

As in https://github.com/apache/spark/pull/16078 I do not think we should merge a change like this. Let's fix the problem directly in https://github.com/apache/spark/pull/16037

Agreed this is too ML specific to be deserve a fix in core.


wip: Sync upstream only network unavailable commits

Thanks for sending a pull request! A few things before we get started:

If this is your first time, please read the contributing guide
For non-trivial pull requests, please file an
issue first, and get
agreement that the change is a good idea, and a general guideline
for how it should be implemented, before sending code. Large PRs
that weren't first discussed and agreed upon in an issue won't be
accepted.
If the PR fixes a particular bug, please include the words "Fixed
#" in the PR text, so that the bug auto-closes when
the PR is merged.

/test metallb-e2e-metal-ipi

Waiting for the latest commits to merge U/S and will rerun the full sync PR, changing the override frr image to


Global namespace conflict between analytics.min.js and sjcl.min.js

sjcl.min.js defines a globally namespaced function t
analytics.min.js defines a globally namespaced variable t (when this file fails to be loaded the conflict does not occur)
Because analytics.min.js is lazily loaded (and loaded last) it overwrites the function t
aws cognito identity sdk depends on sjcl.
Using the segment snippet stops a user from being able to authenticate with aws cognito user pool, using the js sdk.
See related issues
https://github.com/aws/amazon-cognito-identity-js/issues/197
https://github.com/bitwiseshiftleft/sjcl/issues/196

Looking at my personal analytics.min.js, it passes in window.define as t in the IIFE. I'm assuming this is to make the library AMD compliant.
Lines 1,2 and 10814 from my personal analytics.min.js:formatted in Chrome DevTools:
!function(t) {"function" == typeof t && t.amd && (t = void 0);
  // analytics.js code
}(window.define);

Because t is scoped to the function, it should not conflict with the global namespace.
Try this example in the console:
var t = 'global t';
!function (t) {
    console.log(t) // 'scoped t'
}('scoped t');
console.log(t); // 'global t'

Please provide a link that demonstrates the collision behavior, and I will investigate further.


Error 2 see post it

After the first database search the program automatically runs the TCGA Expression analysis and redirects the user to the TCGA Expression tab. See post it for exact options to reach the bug.

Done.


external-dns-private should depend on prometheus-operator-crd app due to ServiceMonitors

What this PR does / why we need it
Towards https://github.com/giantswarm/roadmap/issues/3720
Checklist

[X] Updated CHANGELOG.md.

Trigger E2E tests

/run cluster-test-suites

/run cluster-test-suites TARGET_SUITES=./providers/capz/private

I've managed to confirm this specific change works manually. There seems to be an unrelated issue causing the CAPZ private tests to timeout during creation. I'm going to add the skip label so we can get this merged in and released.


Always return nil when named column isn't found.

This improves named column support so that it acts predictably (returns a nil value for all rows) if the named column isn't found.

Thanks a tonne for these patches.

Gem has been updated :smile:


Investigate Storybook 8 changes

Looking into Storybook 8 changes against the some of the Storybook simplification.

Lee and Matt to follow up after Matt’s return from vacation.

Made a little progress here, but it was hard going.
It refused to process the MDX file used for getting started reporting Invariant failed: No matching indexer found for this despite a clean create react app project with storybook 8 being perfectly happy to process it.
Next up was complaints about the way some of our stories are written. The best course of action may be to cut down to a single file to find a pattern that can be applied elsewhere.
https://github.com/lee-chase/ibm-products/tree/storybook8

Cutting it down just to APIKeyModal we get this error, however the same seem to be possible in the a clean project.

This works in test react app project

We'll be moved over to Storybook 8 with #5019, moving to Vite from Webpack resolved a lot of issues that I think both @lee-chase and myself encountered when initially looking at this.


Been experiencing crashes in Windows Explorer since installing this

I'll be back with more details once I get the time to investigate it a little.

Can't figure it out, might not be FileMeta after all...


Compatibility with React.js 15.2.1

Hi,
I was wondering if the module is compatible with the latest React 15.2.1.

i see no breaking change here, (no reactDOM)

updated the build for 13 < react < 16


Script cleanup

Make the node scripts more modular.
Make the example more self-contained.
Add public path handling to the tests.

👍


Remove URL parameters from FileName

Signed-off-by: Maxime Gervais<EMAIL_ADDRESS>

I think I prefer to use Reader_libcurl_FileNameWithoutPassword() everywhere (also in the error logs), possible to use it with the additional filter on "?"?

I think I prefer to use Reader_libcurl_FileNameWithoutPassword() everywhere (also in the error logs), possible to use it with the additional filter on "?"?

@JeromeMartinez I can chain the two functions call, or add a parameter to Reader_libcurl_FileNameWithoutPassword()

@JeromeMartinez I can chain the two functions call, or add a parameter to Reader_libcurl_FileNameWithoutPassword()

add a parameter to Reader_libcurl_FileNameWithoutPassword(), I consider the part after ? as like a password (it is actually the case for AWS presigned URLs!).

add a parameter to Reader_libcurl_FileNameWithoutPassword(), I consider the part after ? as like a password (it is actually the case for AWS presigned URLs!).

So I delete the parameters also from the logs?

So I delete the parameters also from the logs?

Yes

Yes


Filepond UI error message doesn't go away when files that exceed the limit are removed

Describe the bug:
I have been working on filepond for a while now and I came across this issue recently.
Right now I have a limit of 10MB for total files size and a 1MB limit for each file.
I get the error message saying total files size must not exceed 10mb. But when I remove the file with bigger size, other files that display the error message are not getting updated.
How to reproduce

Have a total file size limit equal to 10MB : maxTotalFileSize="10MB"
First, attach a file with the size around 5MB
Add multiple files with smaller size less than 5MB parallelly until most of the files display the error message "Total must not exceed 10MB"
Remove the file with the bigger size i.e., file with size 5MB

Current Behavior

when I remove the file with the size 6.7MB, other files are actually within the 10MB limit, but it still says total must not exceed 10MB for those files  as shown below
Expected Behavior
When a file is removed, filepond has to re calculate the size of the existing files in the drop box and remove the errors that are not relevant.

Additional Information

Reactjs
"filepond" : 4.19.2
"react-filepond" : 5.0.0
"react": ^16.13.1

Is there anyway I can update the files and error messages on removing a file?
I'd appreciate any help here.
Thank you.

We require the issue template to be filled out on all new issues. The issue template helps us collect all the information we need to address your submission efficiently.
The more information we have, the faster the issue can be resolved.
We will be able to more closely look at your issue once the issue template has been filled out. The issue will be closed until the above is addressed.
Please leave a comment after you've updated the issue.

Hello @rikschennink ,
I've updated the issue template. Could you please take a look and let me know if you require any more information?
Thanks,
Kruthika M

Thanks, will take a look on Monday.

Thanks for reporting, this indeed looks like a bug.

Hello @rikschennink,
Since this issue is a bug, could you please re-open it?
I'm really looking forward to a solution from your end for this issue!

Ah yes, forgot to reopen after the update.

Hello @rikschennink,
Hope you're doing fine.
Just checking if you found any solution for the above bug. Please let me know.
Thanks!

Hi @rikschennink
Any update on this?
Thanks!

No sorry, I'm working on FilePond v5 which addresses issues like these.

do we have any updates? faced with the same error

I am facing the same issue. I am currently trying to make something like: when other files get deleted, retry to load this file to remove the error.
  pond.on('removefile', (removedFile) => {
    pond.getFiles().forEach((file) => {
      if (file.status === 8) {
        file.retryLoad();
      }
    });
  });

I saw retryLoad was one of the available functions, but it doesn't help. Maybe there's another function I can call to make the error reevaluate?

Honestly the error state per file is a bit weird, it's not the file itself that is at fault, it's the collection as a whole. The warning should be shown at the a component level error/warning, not at the file level. But unfortunately that's not what past-rik decided to go with.
For now I think it's best to not use the maxTotalFileSize prop and instead use a basic function like this instead:
const pond = FilePond.create(inputElement, {
    allowMultiple: true,
});

pond.onupdatefiles = files => {
    const totalFileSize = files.reduce((total, file) => {
        return total + file.fileSize;
    }, 0);

    // check if exceeds maximum file size
    const showErrorState = totalFileSize > 1000000;

    // show an error
    console.log({ totalFileSize, showErrorState });
};

If showErrorState is true you then display an error message in an element outside of FilePond.


[Docs] Update README for install notes

Changes the suggested shell when running the install script to $SHELL
which will return the default shell.
Adds a note for how to fix the issue when the install script is not
updating the correct profile file.
Reorders the list of profile files to match the order of files the install script looks for.

NOTES ON CHANGES:
This documentation change is to help users who are finding that the install script is not updating the correct shell profile file. $SHELL will return the path to the default shell used, which will set the environment variables used by the install script to decide which profile file to update. There are reports of users not sure why their preferred profile file is not being updated here https://github.com/nvm-sh/nvm/issues/2196.
I suspect that many of these are from people running the given curl command, which currently pipes the install script into bash explicitly, even if the user has zsh or another shell as their default. This will especially bite MacOS Catalina users who now have zsh as their default shell.
There is also a way for users to explicitly set which profile file to change by the install script, which some users have asked for not knowing it already exists. Here https://github.com/nvm-sh/nvm/issues/2188, https://github.com/nvm-sh/nvm/issues/869       . This lets them know how to do that.
CAVEATS:
I would prefer to use an variable like $0 instead of $SHELL as $SHELL does not return the currently used shell, but the default shell. The reason why $0 can't be used is that it returns the current shell with a - in front of it. Once you switch to another shell and then back, the - is gone, but its there for all uses before switching to another shell and back. $SHELL however is better than the current example which states bash explicitly.
Also, it would be better for the install script to be a smarter about which profile file to change. Right now it is using the existence of $BASH_VERSION, .bashrc, .bash_profile, $ZSH_VERSION, .zshrc, and then .profile, in that order, to decide. It could instead look at which shell is being used and then set the corresponding profile file, through something like $SHELL or $0. I'm not versed in shell script so will leave those changes to someone with more experience.

@ljharb Would we want to include this, since we determined that piping through other shells might break backward compatibility.


test-branch/execute-job/wrong-commit-sha-on-pr/83906773-83fd-4921-94c6-43ebc21b7789

PR for testing 6d92283850230c6e146f8cd5b756341f55cae8dd

/canonical/self-hosted-runners/run-workflows 931ad21dd9ace68aaa0a7d341e94de96ad93fa6b


Add prop type utility for vue

This probably needs a documentation since usage and utility isn't really obvious :)

I've added a documentation entry under documentation/vue.md inside the generated code.


Error on dynamic requiring img src

I use the webpack plugin with the default settings.

So basically, requiring the variable throws an error, but requiring the output of the variable works perfectly. Am I missing something here or is this somehow a bug?

You can't make webpack require a resource that is only an expression (which is only evaluated at runtime) - which makes sense, because the require could end up being anything, includeing a node_modules package, and webpack shouldn't pre-load all of those possibly hundreds of megabytes of files, right?
You can however use a partly dynmaic require string:
https://webpack.js.org/guides/dependency-management/#require-with-expression

You can try something like this
image() {     return require("@/assets/images/" + this.image}

You can try concatenating the string in Computed. Something like this
	computed: { 		image() { 			return require("@/assets/images/modals/aiesec/" + this.ima); 		} 	}


[kie-issues#1446] BPMN Editor - Decision name UI field removal

Closes https://github.com/apache/incubator-kie-issues/issues/1446
The "Decision Name" field in the UI does not do anything. I've gone through the backend, we don't even haven anyway of referencing a "decision name." Remove this field as it is unnecessary.
Thanks to @jomarko for a starting point.

Fixes #2542

Fixes https://github.com/apache/incubator-kie-issues/issues/1446

Is there anything to be done for the Code scanning job? Looks like we're missing something for that job to complete.

I'm not sure @LightGuard. I remember seeing it a while ago but thought GitHub was unstable. Feel free to look into it if you got time...

Thank you @jomarko. Can you please file an issue on kie-issues for us to track those outdated places? Thanks!

@tiagobento , sure, ticket created https://github.com/apache/incubator-kie-issues/issues/1449


after refreshing the page the initialvlues are gone

I guess this is a bug.
I am using "redux-form": "^8.1.0", and noted that if initialValues are flexible (for instance in my case I get title from my redux state ) after refreshing my page the initial values are not displayed in the field unless I put  this.props.initialize(this.props.initialValues); inside my render
const mapStateToProps = ({ streams }, props) => {
  const stream = streams[props.streamId];
  return {
    initialValues: {
      title: stream ? stream.title : null
    }
  };
};
export default connect(
  mapStateToProps
)(blabla);

Please reproduce your issue with a sandbox, then post the link here. Thanks in advance

Closing due to inactivity.


Define vocabulary

Please define the/your definition of job, stage and step as early as possible.

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

ID: 95227592-b2a9-62be-59d1-fe8c7f4eae8e
Version Independent ID: abf0c659-ce8c-b187-2784-22ddc061d69f
Content: Jobs in Azure Pipelines and TFS - Azure Pipelines
Content Source: docs/pipelines/process/phases.md
Product: devops
Technology: devops-cicd-process
GitHub Login: @juliakm
Microsoft Alias: jukullam

Thanks for the feedback. I'll add a reference to our key concepts article.


Support the msvc compiler.

On Windows, using -cc=vcc fails because GNU-style arguments are being passed to cl.exe. This can be resolved by using -cc=clang, but this causes a compilation error in GLFW because types like WINBOOL aren't defined, leading to a misleading syntax error:
naylib-5.0.2-9979f2e337750d00c4e9feeb34641600f83dd86b\raylib\src\external\glfw\deps\mingw\xinput.h:227:26: error:
      a parameter list without types is only allowed in a function definition
  227 | void WINAPI XInputEnable(WINBOOL);
      |                          ^
1 error generated.

If the compiler can be determined, the best solution would be to just pull in the windows headers on a windows target and assume that the user is building with MSVC (most users will probably go this route, MinGW/MSys2 doesn't exactly work well on Windows these days particularly with security mitigations enabled).

No, the official way to install Nim is to use MinGW. Try winlibs.com Unfortunately I cannot help you fix this, as I don't own a windows machine.

 Since there's no response to call for help closing.


feat: change default script and predicate program generated at init

Description
Changes default predicate to:
predicate;

fn main() -> bool {
    true
}

and default script to:
script;

use std::logging::log;

configurable {
    SECRET_NUMBER: u64 = 0
}

fn main() -> u64 {
    log(SECRET_NUMBER);
    return SECRET_NUMBER;
}

Thank you @kayagokalp 🙏!
Before the defaults was predicates returning false and script was simply empty which means you can't make any meaningful example templated assertions to them. These new examples are for the new templates for scripts and predicates
https://github.com/FuelLabs/sway/pull/5238


Remove cygwin and openssh builds?

With winrm, we don't need these builds any more, and they just overly complexify things. Also, if a user needs them, we can modify bin/tweak-json.py to regenerate them.
Therefore, I volunteer to remove them. @annawake?

I was unable to make vmware//win7x86-enterprise today.  I kept getting an error when the VMware tools were suppose to upload, right after WinRM successfully connected.  I tried make clean before trying again and it failed the same way.  I then tried make vmware//win7x86-enterprise-ssh and it worked.  So removing ssh and cygwin would be removing a quick work around when WinRM is obviously not working right.

I fixed that issue locally, using
winrm set winrm/config/winrs @{MaxMemoryPerShellMB="2048"}

rasa, I'm new to using packer/boxcutter/vagrant/chocolately.  I'm currently on a Windows 10 PC using VMware Workstation and I'm not aware of where I would use that winrm set command?  Is it something I configure for my Windows environment or for VMware somehow?  Does it go into a config file?  Thanks.

rasa, never mind, I figured it out.

openssh/cygwin builds are pretty useful, for example I use packer/vagrant for running windows builds on remote bare-metal boxes which only have remote access over ssh.
Being able to do ssh into one of those remote vbox builds is priceless.
(ssh -X/vnc/rdp is not an option)

This would make the boxes unusable on linux and OSX as these platforms don't have winrm client support in vagrant.

Linux and OSX had the vagrant-winrm plugin available and I believe in more recent versions the functionality has been rolled into Vagrant proper, so it shouldn't even require a plugin anymore.

I'm all for this removal as well.

It appears that the vote is to not remove these templates.
How about consolidating them? There aren't too many differences between the templates, it might be possible to use something like jq to merge the ssh or cygwin parts into the main template?
If there's no interest in either of the aforementioned solutions, I'll likely close this in a month or so as "wontfix" if nobody has any objections. Let me know.

Now that Packer supports writing templates in HCL I wonder if there is an option to write them in HCL and do an import or merge on top of a base with the SSH and cygwin functionality and export to JSON. I'd definitely hope to avoid going too crazy with having code that generates templates just to avoid a little bit of repeated code.

Yeah, agreed on avoiding going crazy with generating templates for everything.
With regards to jq or whatever other mechanism, I was thinking something along the lines of the following...like you can do relatively quick merges like the following to add/replace some fields to an individual builder.
jq '.builders[] |= . + {"communicator":"ssh","ssh_username":"blah","ssh_password":"blah"}' eval-win7x86-enterprise.json

This logic simply adds the communicator, ssh_username, and ssh_password from specified dict into each of the builder elements for a template.
But with regards to integrating this into boxcutter/windows, it'd probably look something more like the following if we've isolated the ssh-specific/cygwin-specific components into their own json. I've been doing this sort of stuff in my own fork of boxcutter/windows to de-vagrantify all of the templates, and to change the default usernames/passwords.
jq --slurpfile builders /path/to/ssh-builders.json --slurpfile provisioners /path/to/ssh-provisioners.json '(.provisioners[].scripts += $provisioners) | (.builders[] |= . + $ssh)' eval-win7x86-enterprise.json

But, to avoid forcing people to have to build these templates themselves (which'd require having jq), we could split up the logic for ssh-ifying (or cygwin-ifying) a template into the makefile, and then have a maintainer just ensure that its part of a pre-commit hook or something similar so that from the users' perspective, nothing appears to have changed.
Re-writing all of the json templates in HCL kind of feels out of scope for this project, especially since its kind-of in "maintenance" mode. It might be better to consider a whole rewrite for non-json templates in some newer project like github.com/boxcutter/windows-ps or something similar

This issue has had no discussion or input for over 6 months. I've tagged it as stale and as a result this issue will be closed in 1 month.
If you disagree and feel this decision is a mistake, please let me know and I'll remove the label.

Closing this issue as it's been stale for over a month.


Swift 3.0 branch SR1951 Fix 2

Explanation: IRGen failed to update one of its data structures which allows to lookup arche type access types. This can lead to crashes when the data structure is consulted.
Scope: This occured on a github project. It can occur with generic types with same type constraints.
Origination: I think this was introduced within the swift 3 time frame.
Reviewed by: John McCall, Slava Pestov
Testing: The project that crashes, compiler unit tests
Directions for QE: This bug is tested as part of the compiler unit tests.

Resolves one bug of SR-1951.

@swift-ci Please smoke test OS X Platform

@swift-ci Please test


Fixes #5: Build not requiring any Conjur connection are failing

Desired Outcome
This PR fixes issue #5

Issue #5 was addressed by PR #9 so I believe this PR can be closed.


Pylance creates dozens of errors with Huggingface Transformers

I'm using VScode's Jupyter notebook feature, and although my code runs fine, there are dozens of Pylance errors.  I'm going over the Huggingface Transformers Course on their website.  For example from transformers import AutoModelForSequenceClassification generates an error to import from transformers.models.auto.tokenization_auto instead, and this fails to run.

can you link example

This is the course.  If you paste these text fragments into a Jupyter notebook in VSCode, you'll get about 34 errors, but the notebook will run fine.  [https://huggingface.co/course/chapter3/2?fw=pt]

@MikeyBeez, with "python.analysis.typeCheckingMode" set to "basic" I'm not seeing any errors with the code sample from the "Processing the data" section on the page you linked to above.
Can you please provide more details? Is this the code that you were referring to? Did you pip install both torch and transformers[sentencepiece]? What errors are you seeing?
import torch
from transformers import AdamW, AutoTokenizer, AutoModelForSequenceClassification

# Same as before
checkpoint = "bert-base-uncased"
tokenizer = AutoTokenizer.from_pretrained(checkpoint)
model = AutoModelForSequenceClassification.from_pretrained(checkpoint)
sequences = [
    "I've been waiting for a HuggingFace course my whole life.",
    "This course is amazing!",
]
batch = tokenizer(sequences, padding=True, truncation=True, return_tensors="pt")

# This is new
batch["labels"] = torch.tensor([1, 1])

optimizer = AdamW(model.parameters())
loss = model(**batch).loss
loss.backward()
optimizer.step()

I’m afraid I’ve moved on.  I’m not working on this any more.

On Oct 6, 2022, at 8:56 PM, Erik De Bonte @.***> wrote:
@MikeyBeez https://github.com/MikeyBeez, with "python.analysis.typeCheckingMode" set to "basic" I'm not seeing any errors with the code sample from the "Processing the data" section https://huggingface.co/course/chapter3/2?fw=pt#processing-the-data on the page you linked to above.
Can you please provide more details? Is this the code that you were referring to? Did you pip install both torch and transformers[sentencepiece]? What errors are you seeing?
import torch
from transformers import AdamW, AutoTokenizer, AutoModelForSequenceClassification
Same as before
checkpoint = "bert-base-uncased"
tokenizer = AutoTokenizer.from_pretrained(checkpoint)
model = AutoModelForSequenceClassification.from_pretrained(checkpoint)
sequences = [
"I've been waiting for a HuggingFace course my whole life.",
"This course is amazing!",
]
batch = tokenizer(sequences, padding=True, truncation=True, return_tensors="pt")
This is new
batch["labels"] = torch.tensor([1, 1])
optimizer = AdamW(model.parameters())
loss = model(**batch).loss
loss.backward()
optimizer.step()
https://user-images.githubusercontent.com/12820898/194449602-9cf15ad5-c190-41bd-9b64-67e83e9f9de3.png
—
Reply to this email directly, view it on GitHub https://github.com/microsoft/pylance-release/issues/3306#issuecomment-1271007498, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADM2NQCMRIJI5OWBRA2OYBDWB57OLANCNFSM6AAAAAAQHZFYDU.
You are receiving this because you were mentioned.


Left-align figure captions

My workaround is to redefine it in yaml (with the addition of a space around a figure):
format:
  preprint-typst:
    include-in-header:
      text: |
        #show figure: set block(inset: (top: 1em, bottom: 1em))
        #show figure: it => {
          align(center, it.body)
          align(left, it.caption)
        }

That's great, thank you! I'll add that snippet in the preprint template file so it's automatically set for all captions. Optimally the alignment would be set upstream from Quarto/knitr options...
tbd in the near future.


Remove clarice and bnotions

edX doesn't do anything with them anymore

@nedbat and @sarina: @aleffert says that edX doesn't do anything with Clarice or bNotions anymore. Does this change look good?

:+1:

They are also in the data repo in orgs.yaml


add sample data eli5, msmarco, asqa

close #108

Plus, edit dataset as the real structure. This data must be the perfect example for evaluation dataset. It must be followed our rules for dataset rules.


Add night mode

Hi, I'm new to iOS dev. No way to add nightmode to WebView. You have any good idea?

try to solve  https://github.com/amitburst/HackerNews/issues/15

Nice, thanks! I will take a look at this over the weekend.

Ended up scraping this feature to keep this as a small example app. Thanks though!


New field border is not visible in dark mode

The new field border is not visible in dark mode.

This is fixed in 1.1.0-alpha.4 on the update-help branch and should be released in the official 1.1.0 release.

fixed in 1.1.0.


Signup homepage

Once you sign up for your account you are logged out when you click to return to home page.

Found the bug on line 104 of controllers/api/user-routes


[5.2] Fix Eloquent Builder accessing $orders property

$orders and $unionOrders are properties of Illuminate\Database\Query\Builder and not defined in Illuminate\Database\Eloquent\Builder. Doing this solve my problem. Is there any problem doing this?

Ping @taylorotwell as you wrote this line in https://github.com/laravel/framework/commit/0b4835e86e6a9c2a1c082f8a6a85aaff525e5271.


Theme picker

Support multi theme (colors).
In web interface put theme picker.

:+1:

:+1:

@tvn87 or disable this in .vimrc.local

@avelino may we can use some things like this:
http://cocopon.me/app/vim-color-gallery/
And a colorscheme editor like vivify: http://bytefluent.com/vivify/

I have been unable to find a way disabling the colorscheme in .vimrc.local. Since it is defined in .vimrc and there seems to be no colorscheme disable command.

The suggestion is a plugin to choose themes installed?

A group of option in the web interface for staff select the theme (vim color) that you want.

@tvn87 may :syntax off?

@lerrua I am not talking about the syntax highlight. A theme overrides the
colorscheme I've set in the terminal. So I have :syntax on and no theme but
still color ;-). The only way to override a theme in vim is to load a new
theme, but that is something I don't want.

Or this colorscheme editor: https://terminal.sexy/

I prefer a light color theme. At least a setting for dark or light would be nice.

I've use vim bootstrap. But when I download a theme and paste into 'init.vim', the theme that i download before wasn't work perfectly. Font color change, but not the background. Any solution?

solved with 6d9850259db00337c26469897bf7993e9ecb89e9

We can improve our frontend asap.


Touch gesture support

Great work! A pan-gesture support for touchscreens would it make even more awesome ;)

Hm, could look at it tomorrow @Blankarsch. If touch-action property will do nicely I can add that. If not - I don't want to make the code any heavier by adding JS.

Something like this? Touch Action

@Blankarsch @MaximilianSchieder I just checked and it doesn't seem it can be done easily with touch-action property since we are using checkboxes here (tried on a fieldset and form and didn't work).
I will drop this idea for now then, as I don't want to add unnecessary scripts here.


Is there any feature to download/list the following details.

How to get license details in tabular form??
or Any template example which shows the following items in list:
Package Name
Package Version
Package type
Copyright Detail
License Detail

Hi @parvjain639, this should be possible using a custom template: https://github.com/anchore/syft#using-templates. There is a new blog post about templates (in Grype, but the concepts are similar) if you need a bit more info: https://anchore.com/blog/customizing-grype-vulnerability-reports-with-templates/ -- and please let us know if you need any help.

Thank you so much we have just solved it by using a Template...
Please clear our one more Query:

Does Syft and Grype have open source license files keyword scanners to facilitate obligations compliance.  Having a tool to scan OSS components license files for keywords such as IP: patents, royalties, legal, ECC: export, cryptography, AI, newtech, GDRP: privacy, regulations, chatgpt, OSS: attribution, contribution, distribution streamlined obligations compliance.

At the moment, no, the tools don't have keyword scanners like you suggest. It might be difficult to automate these kinds of things, but we would be happy to consider contributions if you implemented a new scanner.

Thank you so much for your response.
We will see what we can contribute!!
I am having one more doubt!
How to integrate SYFT and GRYPE with GITLAB 16??

 For Gitlab integration, I would start here and read through the Gitlab docs: https://docs.gitlab.com/ee/development/integrations/secure.html
Good luck!

When it comes to a keyword search within license text that is really outside of the intended use case for syft (which is to create SBOMs). However, we don't support raising up the raw license text in the SBOM, which would at least enable downstream tooling to do this (and is supported in at least SPDX).

If we supported such a feature (getting full license text in the SBOM) would that be useful to you?

Yes, this will be very helpful to us. And What about Dependencies and Depth (Level) of Dependencies??
If you can add this feature, this will be also very helpful for us and other users too.

indeed! that work is being tracked under https://github.com/anchore/syft/issues/572

We will close this issue but please let us know if you need any more help. Thanks!


Lack of developer documentation

See https://googlecloudplatform.github.io/gcloud-node/#/docs/v0.13.1/datastore
It mentions "some convenience methods"
Convenience based on What?
Do I need to master the low level Datastore APIs? Imo, the gcloud layer should be self sufficient and complete for documentation, exposing all the Datastore concepts assuming the user knows nothing, and without the need for the user to read other docs....
The doc issue seems generic for all gcloud APIs for all the runtimes (same for Java, I did not look at ruby)

Hi @ludoch thanks for the feedback. We pride ourselves on our documentation so it's sad to hear you're not satisfied.
Datastore is a rather tricky API to make an idiomatic client for while also giving users like you the ability to do everything you need to do. This is because the Datastore concepts are strongly linked to the API itself, e.g. what a key is, what an entity is, how these play in when you're just trying to save your data to the datastore. We hope this library tries to soften the edges by abstracting away the details that don't matter and in some cases making assumptions on what you want to do given the specific context.
In any case, we believe that as a developer you should ramp yourself up on the various high level concepts of each API from https://cloud.google.com/docs/. We should also likely make this clear in our docs.

"some convenience methods"

This is just talking about the datastore#double and datastore#int methods available on that object for specifying the type of number you'd like to save to the datastore because JavaScript only has a single number type.
We'd suggest you start looking at the dataset documentation because that gives you the power to make calls with the API. We should also make this more clear in our documentation.

The doc issue seems generic for all gcloud APIs for all the runtimes

If you believe that other languages documentation is also bad, I'd suggest you open separate issues on their libraries for your concerns as different people work on different libraries.

It would be nice to add in a quick link in each module's "overview" section to the official docs.

Sent in #507 that links to Google developer docs for each service. This will hopefully give newcomers an easy path to all the information they will need.


eosio-cpp abi generation "type is invalid", eosiocpp correct. Ignoring 'key_names' and 'key_types'

If we will try to generate abi for that contract with eosio-cpp :
#include <eosiolib/eosio.hpp>
using namespace eosio;

//@abi table tests i64
struct[[eosio::table]] tst
{
	tst() {};

	uint64_t id;
	vector<uint64_t> vec;

	uint64_t mysize() const {return vec.size();}
	EOSLIB_SERIALIZE(tst, (id)(vec))
};

typedef eosio::multi_index
<N(tests), tst,
	indexed_by<N(tstssize), eosio::const_mem_fun<tst, uint64_t, &tst::mysize>>
> tstsT;

class hello : public eosio::contract {
  public:
      hello(account_name self) : eosio::contract(self)
		, tests(self, self)
		{}
      tstsT tests;
      //abi action
      [[eosio::action]]
      void hi( tst val ) {
         print("size: ", val.vec.size());
      }
};

EOSIO_ABI( hello, (hi) )

with eosio-cpp ./hello.cpp -o ./hello.wasm --abigen
It's ABI will contains "type": "vector<uint64_t>" , which will cause error Error 3015004: The type defined in the ABI is invalid when calling cleos push action contractname hi '[[1,[2,3]]]' -p contractname.
The old way to compile eosiocpp -g ./hello.abi ./hello.cpp will create type "uint64[]"
Moreover, if we will look at section tables, eosiocpp will generate
"tables": [{
      "name": "tests",
      "index_type": "i64",
      "key_names": [
        "id"
      ],
      "key_types": [
        "uint64"
      ],
      "type": "tst"
    }
  ]

and eosio-cpp will ignore key_names and key_types and generate that:
"tables": [
        {
            "name": "tests",
            "type": "tst",
            "index_type": "i64",
            "key_names": [],
            "key_types": []
        }
    ],

Duplicate of https://github.com/EOSIO/eosio.cdt/issues/109
bug with vectors abi gen

What did u mean? U meant that #109 is duplicate of #106? If I'm not wrong, 109 is greater than 106.

I also encountered this situation, how to resolve it? help me!!!

I also encountered this situation, how to resolve it? help me!!!

The easiest way is to not use eosio-cpp, just compile with old eosiocpp .
More complex way is to to fix your abi manualy: replace vectors with arrays and add key_names and key_types.

I can't describe the mood at the moment in words, I hope I can give priority to this problem.

@larryk85

The type is invalid issue with vectors has been resolved in v1.3.0.  As for key_names and key_types, they are ignored, as they are a deprecated part of the abi, these fields are a hold over from a different system for secondary tables and no longer make any sense.


Can we drop the telemetry for VT sequences?

Description of the new feature/enhancement
We've now reached the point that if we add another VT sequence to the telemetry tracker, the TraceLoggingWriteActivity call will fail to compile. I'm not exactly sure what the problem is, but the initial warning/error is: pasting '_tlg_FOR_imp' and '(' does not result in a valid preprocessing token, after which there are another 20 or so lines of errors.
It's possible there is a way to work around that, but if we don't really need this telemetry, it would be nicer if we could just drop it altogether. And maybe that will save a bit of parsing overhead in the process.
Proposed technical implementation details (optional)
Drop the TermTelemetry class, and then remove all code that was making calls to it.

I'm all for it. This was more valuable when we like, hardly implemented any sequences. I don't think this is even hooked up to anything at this point.

YES. PLEASE. Thank you!

The way we implemented this telemetry is wrong, anyway - TraceLogging is supposed to be zero-cost if it is not enabled, so every time we put the log behind a function call we're wasting a branch and likely cache fault to determine we aren't supposed to be doing anything.

I am very new to programming. I am confused how can I contribute to open source. I am proficient in html, css, javascript and learning react now.  Can I contribute somehow?


cherry-pick remove some CHECKs in Tablet::revise_tablet…

Proposed changes
Issue Number: close #xxx
cherry-pick #31268
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...
Proposed changes
Issue Number: close #xxx

run buildall


Control Toastposition

Feature request
I'd like to have a way to control Toast position on the screen.
I have already implemented the necessary changes locally and would like to make a pull request.
Please give me permissions to push a new branch to create PR.

Sounds the same as https://github.com/segmentio/evergreen/pull/625
I think there were some limitations with that approach. PRs welcome! You'll have to create a fork to open a pull request. :)

sorry for the double post. created PR.


Return value of JouwWeb\SendCloud\Model\Parcel::JouwWeb\SendCloud\Model{closure}() must be of the type string, null returned

When you create a parcel and call the toArray() function on the resulting parcel it fails with the error:
Return value of JouwWeb\SendCloud\Model\Parcel::JouwWeb\SendCloud\Model{closure}() must be of the type string, null returned

Problem is that wrapper expects an label created to convert an parcel into an array. Sometimes you wanna split these two processes and this won't work with the current version.

Changing
jouwweb/sendcloud/src/Model/Parcel.php:299
to
'labels' => array_map(function (int $format): ?string {
solves the issue

Thanks for the report and a way of fixing the issue. Sorry for not getting back to you sooner. I didn't have this repository watched yet. I'll fix it and follow up with a minor version bump immediately after that.


Error when pulling changes

I receive an error when pulling objects from abapGitServer.

When I click on the error message, it just show the message again. So, no details available.

closing, duplicate of #2235


ci: render quarto directly

Depends on #15.
Avoid the manually generated intermediary jupyter notebook file(s).
https://github.com/mcanouil/awesome-quarto

Closing this because I'm using markdown as the intermediate format now, and it will only be stored in the gh-pages branch.


🍒 Cherry pick PR #8892 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/App/pull/8892 to staging 🍒

🚀 Cherry-picked to staging by @OSBotify in version: 1.1.57-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by @chiragsalian in version: 1.1.57-17 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


⚠ Bofrost.at has degraded performance

In 0b64cd3, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1006 ms

Resolved: Bofrost.at performance has improved in b7f7eb1.


Simplifying the documentation

Fixes https://github.com/Azure/Azure-Orbital-Analytics-Samples/issues/83
Following message will be removed after we merge https://github.com/Azure/Azure-Orbital-Analytics-Samples/pull/86
Please review only the last commit titled "Updated documentation", the other two commits are already covered in https://github.com/Azure/Azure-Orbital-Analytics-Samples/pull/86

Please review only the last commit titled "Updated documentation", the other two commits are already covered in #86

@sushilkm, As we merged #86 we can remove this line I think.


Remove unnecessary eslint ignore

Stumbled into this one — it was causing a failure on main for me. Haven't investigated what caused it yet.
Merge checklist

[ ] Added/updated tests
[ ] Added/updated documentation
[ ] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

Whoops, maybe I have an environment issue. Looks like it wasn't a problem in CI.


Storage account keys renew takes 'account-name' instead of 'name'

$ az storage account keys renew -h

Command
    az storage account keys renew: Regenerates the access keys for the specified storage account.

Arguments
    --account-name -n   [Required]: The storage account name.
    --key               [Required]: The key to regenerate.  Allowed values: primary, secondary.
    --resource-group -g [Required]: Name of resource group.

Lokos like this should be --name here.

Based on our conventions, if anything, it would be the key that should correspond to name, not the account.

Understand.
Then what about this?
$ az storage account keys list -h

Command
    az storage account keys list: Lists the access keys for the specified storage account.

Arguments

Resource Id Arguments
    --ids              : One or more resource IDs. If provided, no other 'Resource Id' arguments
                         should be specified.
    --name -n          : The storage account name.
    --resource-group -g: Name of resource group.

Then this should be --account-name?

Yes, it should.

Fixed in https://github.com/Azure/azure-cli/issues/1796

How to check whether storage account keys are regenerated using azure policy using json script ?


Wishes

Hi.
I want to see the output of the generated dictionary to the stdout console. As well as setting the minimum word length. Thanks!

I can do that. 👍

These features are in the latest release.


I couldn't use with the new yara 3.5

+ export CGO_CFLAGS=-I/usr/local/includeBuilding info Go binary...

+ export CGO_LDFLAGS=-L/usr/local/lib
+ go version
go version go1.6.3 linux/amd64
+ go get
# github.com/hillu/go-yara
../../hillu/go-yara/rules-callback.c: In function 'rules_callback':
../../hillu/go-yara/rules-callback.c:40:76: error: 'YR_MATCH {aka struct _YR_MATCH}' has no member named 'length'
         addString(user_data, string->identifier, m->offset, m->data, (int)m->length);

Ah well, the API changed in 89a305f91de07ebfbeaf56a228140e10f27a8623, just a few weeks ago. Thanks for spotting this.

@blacktop could you please verify that my commit actually fixes your problem? Thanks.

it worked perfectly! thank you!! 👍


Mozilla Addons reviewer: "Please add a privacy policy"

Just to keep you in the loop to help you understand why Firefox is stuck on version 19.12.9:

More information requested by Cse a day ago
Please add a privacy policy to this add-on that details which user data is being sent and to what services. The critical things to describe in the policy are how your extension collects, uses, stores, and shares or discloses information about people.If your add-on makes it apparent to websites that it is installed, this must also be mentioned. The policy should be about the extension only, not a copy of the website's privacy policy. It should also be the actual text, as opposed to a link to a privacy policy on a website. The privacy policy can be added in the add-on settings under “Manage Authors & License” on AMO.

My response:

Refined GitHub does not collect, use, store, nor share any data; we don't have servers. The extension only operates locally and on GitHub itself.

Help wanted to add a privacypolicy file in case they make this a hard requirement.

Latest:

There should be a privacy policy that states what kind of data is sent to github servers

Help.

I was just about to open an issue about the missing Firefox version.

Some info I found on the internet about privacy policies for extensions:

Documentation: https://extensionworkshop.com/documentation/publish/add-on-policies/
Examples of privacy policies:

https://addons.mozilla.org/en-US/firefox/addon/weather-extension/privacy/
https://addons.mozilla.org/en-US/firefox/addon/lastpass-password-manager/privacy/
https://addons.mozilla.org/en-US/firefox/addon/decentraleyes/privacy/

English is not my native language, so I'm not the best person to write a policy.

Perhaps:

This extension communicates with GitHub to display additional data related to files, repositories and users. GitHub will receive the same unmodified cookies from GitHub.com or your Personal Access Token, if you provide one. Some actions like "Revert Changes to File" will use the GitHub API to create a commit only when initiated by the user. The authentication is done directly between the extension and GitHub, no third party server is involved. There's no advertising and no analytics. All features that cause additional HTTP request can be disabled.

@sindresorhus can you add it here? It's read-only for me https://addons.mozilla.org/en-US/developers/addon/refined-github-/ownership

@sindresorhus also Chrome is stuck on 4 versions ago and the 20.1.6 publish command returned "OK". Does the Chrome Store say anything about it?

can you add it here? It's read-only for me addons.mozilla.org/en-US/developers/addon/refined-github-/ownership

✅ Very well written.

also Chrome is stuck on 4 versions ago and the 20.1.6 publish command returned "OK". Does the Chrome Store say anything about it?

Wow... worse waiting times than AMO.
Let's see if the policy is enough. Could still be rejected for being too vague

It looks like the Firefox version was accepted; I got an update to 20.1.6 today. 🐰

Hallelujah!

@sindresorhus also Chrome:

            "message": "Publish condition not met: To publish your item, you must provide mandatory privacy information. This can be done in the new Developer Dashboard."

Why do we need the activeTab permission (specifically)?

To enable GH on Enterprise https://github.com/fregante/webext-domain-permission-toggle

Ok, submitted. They warned that it could take up to several weeks to review...

While we wait for Google, maybe there's something we can improve with the privacy form I filled out?

Single Purpose Description
The extension adds power-user features to GitHub.com.

activeTab
We need this to be able to enable the extension on GitHub Enterprise. activeTab lets us read the URL of the tab where the user clicks "enable". https://github.com/fregante/webext-domain-permission-toggle

Host Permission
We need access to https://github.com/* to add the features and access to https://api.github.com/* to fetch required data for some of the features we add.


animated-border button

A button with animated colour gradient borders like this:
https://user-images.githubusercontent.com/55590289/136796072-872416e7-4689-47ff-a4d2-29e74071b2df.mp4

I'm working on this, please assign me on this