GH-5024 Fix 'nuclear' delete bug in RDF4JTemplate.delete(IRI, List)

GitHub issue resolved: #5024
Briefly describe the changes proposed in this PR:
RDF4JTemplate.delete(IRI, List<PropertyPath>) is meant to

delete the 'star' around IRI
find all nodes from IRI by the specified property paths and delete the 'star' around them.

The problem was that the  source - propertypath - target pattern was optional, and so the star pattern around the target could match all triples. Now that pattern is not optional, which limits deletes to any matches of that pattern.

PR Author Checklist (see the contributor guidelines for more details):

[x] my pull request is self-contained
[x] I've added tests for the changes I made
[x] I've applied code formatting (you can use mvn process-resources to format from the command line)
[x] I've squashed my commits where necessary
[x] every commit message starts with the issue number (GH-xxxx) followed by a meaningful description of the change

@hmottestad no good idea on how I can help make the slow-tests pass... ideas?

I restarted it. I believe it's unrelated to your PR.

I fixed the small formatting issues here so that this can be included in 5.0.1.


k-exercises not checked in

mvn verify fails because k-exercises is missing. It seems like it is not checked into runtimeverification (https://github.com/runtimeverification).
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-antrun-plugin:1.7:run (exec-ktest) on project k-distribution: An Ant BuildException has occured: The directory /Users/danielsf/runtimeverification/k-exercises does not exist
[ERROR] around Ant part ...<exec dir="/Users/danielsf/runtimeverification/k/k-distribution/../../k-exercises" executable="make">... @ 4:104 in /Users/danielsf/runtimeverification/k/k-distribution/target/antrun/build-main.xml

This should be fixed.


Removing assumption that python is available when running --install

Moved installing the Example plugin till after the environment is created since install_plugins.py needs the python command to exist.
Added logic to the conda install path to find a python command that is in the PATH, by checking PYTHON_COMMAND, python and python3, and using the first that is available.

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
Closes #2033
What are the significant changes in functionality due to this change request?
Now can be installed if only python3 is available in the path by modifying establish_conda.sh

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[x] 1. Review all computer code.
[x] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[x] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[x] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[x] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[x] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[x] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[x] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[x] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

Checklist is good, and PR can be merged.


[Enhancement] Link players to their Discord accounts

Checks

[X] I confirm that I have searched for existing issues before reporting to avoid duplicate reporting.

Description
Example usage: In-game chat message can also display user role colour.

This feature is cumbersome to implement and has little practical use. Therefore, I will not work on this.

One use for this that I have seen many servers use is for sync'ing the users discord roles so that they can assign different permissions in-game based on discord roles.

One use for this that I have seen many servers use is for sync'ing the users discord roles so that they can assign different permissions in-game based on discord roles.

This should be done by other mods that are only for that purpose, and should not be implemented in MCDC.

This feature is cumbersome to implement and has little practical use. Therefore, I will not work on this.

On the contrary, the feature would be very useful, I'm in favour of this kind of functionality!

@Kisuny
What other useful features can you think of?

@Kisuny What other useful features can you think of?

Are we talking about being able to sync the discord account with mine server chat?

@Kisuny What other useful features can you think of?

Are we talking about being able to sync the discord account with mine server chat?

Yes


Source: megalobiz.com

Song - https://en.wikipedia.org/wiki/Satisfied_(Hamilton_song)
https://www.megalobiz.com/lrc/maker/hamilton+-satisfied.54542334
https://www.google.com/search?q=hamilton+satisfied+site%3Amegalobiz.com

Song - https://en.wikipedia.org/wiki/Believe_(Cher_song)
https://www.megalobiz.com/lrc/maker/Cher+-+Believe+[Official+Music+Video].54552148

https://www.megalobiz.com/lrc/maker/Video+games.54516021

Song - https://en.wikipedia.org/wiki/Runaway_(Bon_Jovi_song)
https://www.megalobiz.com/lrc/maker/Bon+Jovi+-+Runaway.54805058

// File: src_sources.cpp

// Include the necessary headers
#include "stdafx.h"
#include "openlyrics_source.h"

// Function to fetch lyrics from www.megalobiz.com
std::string GetLyricsFromMegalobiz(const std::string& url)
{
    // Implement the logic to fetch the lyrics from megalobiz.com
    // You can choose your preferred web scraping or HTTP library for this task

    // Here's an example using libcurl:
    CURL* curl = curl_easy_init();
    std::string lyrics;

    if (curl)
    {
        // Set the necessary curl options
        curl_easy_setopt(curl, CURLOPT_URL, url.c_str());
        curl_easy_setopt(curl, CURLOPT_FOLLOWLOCATION, 1L);
        curl_easy_setopt(curl, CURLOPT_WRITEFUNCTION, WriteCallback);
        curl_easy_setopt(curl, CURLOPT_WRITEDATA, &lyrics);

        // Perform the curl request
        CURLcode res = curl_easy_perform(curl);

        if (res != CURLE_OK)
        {
            // Handle the error gracefully
            lyrics = "Apologies! Failed to retrieve lyrics from megalobiz.com. Please try again later.";
        }

        // Clean up the curl instance
        curl_easy_cleanup(curl);
    }
    else
    {
        // Handle the curl initialization error
        lyrics = "Apologies! Failed to initialize curl for fetching lyrics. Please check your network connection.";
    }

    return lyrics;
}

// Modify the openlyrics_source.cpp file to incorporate the new source
void OpenLyricsSource::GetLyrics(const std::string& url)
{
    if (url.find("megalobiz.com") != std::string::npos)
    {
        // Fetch the lyrics from megalobiz.com
        std::string lyrics = GetLyricsFromMegalobiz(url);
        if (!lyrics.empty())
        {
            // Display or process the fetched lyrics
            // Perform necessary operations based on the retrieved lyrics
        }
        else
        {
            // Handle the case when no lyrics are found
            lyrics = "No lyrics found on megalobiz.com. Perhaps the song is instrumental?";
        }
    }
    else
    {
        // Handle other sources gracefully
        // Implement appropriate behavior for sources other than megalobiz.com
    }
}

@jacquesh

New example - https://www.megalobiz.com/lrc/maker/Selena+Gomez+-+Hands+To+Myself.54465621


C#

Best documentation to start with c#

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

ID: 8053bb96-cc40-8369-c07f-f9fd14060c1e
Version Independent ID: 293260e2-f444-537b-0ba7-14d1fb6ebcf2
Content: Hello World - Introduction to C# interactive C# tutorial
Content Source: docs/csharp/tour-of-csharp/tutorials/hello-world.yml
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thank you @Shivam7283
Closing as kudos


Testing - this should fail

Looks like CheckResults still skipped even though it wasn't supposed to.


mathjax does not get installed properly

From<EMAIL_ADDRESS>on 2012-04-16T01:21:12Z
What steps will reproduce the problem?

after installation using 'python setup.py install'
rich text mode does not work correcly for object inspector

The reason is that mathjax and the templates don't get installed. The fix is (I guess) the following:
diff -r 53f93819ad49 setup.py
--- a/setup.py	Mon Apr 02 14:17:20 2012 +0200
+++ b/setup.py	Mon Apr 16 08:17:48 2012 +0200
@@ -85,11 +85,11 @@
platforms=['any'],
packages=get_subpackages(LIBNAME)+get_subpackages('spyderplugins'),
package_data={LIBNAME:

               get_package_data(LIBNAME, ('.mo', '.svg', '.png', '.css')),

               get_package_data(LIBNAME, ('.mo', '.svg', '.png', '.css','.html','.js')),
               'spyderplugins':
               get_package_data('spyderplugins', ('.mo', '.svg', '.png'))},

Original issue: http://code.google.com/p/spyderlib/issues/detail?id=994

From pierre.raybaut on 2012-04-16T05:30:30Z
Carlos, can you check that everything will indeed be bundled by adding those file extensions?
Owner: ccordoba12
Cc<EMAIL_ADDRESS>Labels: MS-v2.2 Cat-ObjectInspector

From ccordoba12 on 2012-04-16T05:45:23Z
Ops, I didn't try to run setup.py install! I guess that's the problem with always using
$ python bootstrap.py
to run spyder :)
At first sight it seems Christoph patch is enough to solve the issue. I'll try it ASAP on a virtualenv.
Thanks for pointing it out and for the fix

From ccordoba12 on 2012-04-28T07:50:14Z
issue #1007 has been merged into this issue.
Cc: pierre.raybaut

From ccordoba12 on 2012-04-28T08:04:15Z
This issue was closed by revision 679233a3d9a3 .
Status: Fixed


enable downloading of MERS biological assemblies

Description
Had a shower thought that it hadn't occurred to me to check whether we could download the biological assembly in CIF format and whether that would enable OpenEye to read the two "states" as two different chains. And it does!
Todos
Notable points that this PR has either accomplished or will accomplish.

[x] added tests for pdb download
[x] added minor code changes that downloads the CIF biological assembly
[x] probably should change 'download_and_align_pdbs.py' to just 'download_pdbs.py'
the next step would be to add a protein prep script that can prepare these downloaded models! but that should be in a separate PR

Questions
Status

[x] Ready to go

Codecov Report

Merging #135 (f4da465) into main (cd82a6a) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

:exclamation: Current head f4da465 differs from pull request most recent head 3817ff9. Consider uploading reports for the commit 3817ff9 to get more accurate results

I was thinking about the whole "download PDBs" functionality, and I think we might benefit from a general refactor of it (not necessarily a big effort). In that sense, maybe some of the proposed changes in my review (especially the optional ones) can be in a separate issue/PR.

I changed the behavior and signature of the download_pdb_structure as per my comments in the previous review. It runs fine with the code in test-pdb-download.py but I bet it can still create issues deeper in a pipeline. @kaminow @apayne97 can you please review this and make sure it won't create deeper issues with your workflows? Thanks!

@kaminow  @apayne97 By the way, I'm more than happy to make the changes if you don't want to spend time on these, since they are basically minor changes.

@kaminow @apayne97 By the way, I'm more than happy to make the changes if you don't want to spend time on these, since they are basically minor changes.

That would be great!

@kaminow Can you take a look into this one and review after the namespace and file layout changes from merging main? Just to have a second pair of eyes to double check things look as expected. Thanks!

@ijpulidos not a result of your changes but this test now fails due to module not found, what are we supposed to use instead? #143
test-pdb-download.py:
ModuleNotFoundError: No module named 'asapdiscovery.data.pdb'

it might be an issue with your path? with the new refactor I think you need to make sure each of the individual directories asapdiscovery-* are in your path

Hmm so if I leave the data/__init__.py uncommented, I get this error:
Traceback (most recent call last):
  File "/Users/alexpayne/Scientific_Projects/covid-moonshot-ml/asapdiscovery-data/asapdiscovery/data/tests/test-pdb-download.py", line 6, in <module>
    from asapdiscovery.data.pdb import download_pdb_structure, load_pdbs_from_yaml
  File "/Users/alexpayne/Scientific_Projects/covid-moonshot-ml/asapdiscovery-data/asapdiscovery/data/__init__.py", line 5, in <module>
    __version__ = version("asapdiscovery-data")
  File "/Users/alexpayne/opt/anaconda3/envs/mers-docking2/lib/python3.8/importlib/metadata.py", line 530, in version
    return distribution(distribution_name).version
  File "/Users/alexpayne/opt/anaconda3/envs/mers-docking2/lib/python3.8/importlib/metadata.py", line 503, in distribution
    return Distribution.from_name(distribution_name)
  File "/Users/alexpayne/opt/anaconda3/envs/mers-docking2/lib/python3.8/importlib/metadata.py", line 177, in from_name
    raise PackageNotFoundError(name)
importlib.metadata.PackageNotFoundError: asapdiscovery-data

If I comment out this file as in https://github.com/choderalab/asapdiscovery/pull/135/commits/dad9bca5ed0fd39b2f4b13f97b9dcd9b8c725c42, then it works just fine. This feels like a question for @dotsdl

@ijpulidos not a result of your changes but this test now fails due to module not found, what are we supposed to use instead? #143 test-pdb-download.py:
ModuleNotFoundError: No module named 'asapdiscovery.data.pdb'

I think the approach here is that we should do pip install -e . from the asapdiscovery-data subdir, that should work.

I think the approach here is that we should do pip install -e . from the asapdiscovery-data subdir, that should work.

This worked!

@kaminow Can you please give this one more review iteration and check if it seems good to be merged? Thanks!


Some typing issues

When trying to compile the equivalent of cad.js in typescript I get this error:
error TS2339: Property 'shell' does not exist on type 'Shell | Solid | CompSolid | Compound | Vertex | Edge | Wire | Face'.
  Property 'shell' does not exist on type 'Vertex'.

Similarly, when trying to compile
  const OC = await opencascade({
    locateFile: () => opencascadeWasm,
  });

there's an error
error TS2554: Expected 0 arguments, but got 1.

(and indeed the .d.ts shows declare function init(): Promise<OpenCascadeInstance>;)

What is the context of the first one?
For the second one I looked into it at some point - if I remember well, it comes from the upstream project

I just tried to compile this in Typescript:
https://github.com/sgenoud/replicad/blob/main/packages/replicad-app-example/src/cad.js#L4
and that was the error shown


My webcam doesn't turn off when I switch scenes

I am using one of your examples (SelfieBarracuda) and my webcam doesn't turn off when I switch scenes in game. There is have a class called Compositor.cs and in this class there is have OnDestroy method you are destroying or disposing the webcam in that method but its not working I guess because webcam doesn't turn off. I hope I was more descriptive.

Which platform are you using? Mac, Windows or Linux?
What device are you using? A built-in webcam or an external one?

Windows 10 and logitech brio 4k

any help?

I'll look into it when I have time. Please don't expect it happens in one or two weeks.


feat: OPTIC-1242: Ask new OSS installs how they found about us

PR fulfills these requirements

[x] Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
[ ] Tests for the changes have been added/updated (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[ ] Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
[x] Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)
(check all that apply)

[ ] Product design
[ ] Backend (Database)
[x] Backend (API)
[x] Frontend

Describe the reason for change
adding an optional field on signup to ask how the new user heard about Label Studio

/git merge develop


Upgrade dependencies

Most of the dependencies are quite outdated. After the upgrade the tests didn't work anymore, especially the on create callback of PouchDB does seem to be gone. I've also removed the async function and replaced it with promises.
The reason for the upgrade was a conflict:
warning Pattern<EMAIL_ADDRESS>is trying to unpack in the same destination "/home/node/.cache/yarn/v6/npm-esprima-fb-3001.1.0-dev-harmony-fb-b77d37abcd38ea0b77426bb8bc2922ce6b426411-integrity/node_modules/esprima-fb" as pattern<EMAIL_ADDRESS>This could result in non-deterministic behavior, skipping.

@nolanlawson @marten-de-vries It would be great if you can look into this within a week. Sorry for the messy commits with the yarn lock file.

@openscript Looks good to me. In the interest of also being able to merge a similar effort 5 years from now, I thought I'd increase the bus factor a bit by adding you as a collaborator on Github and NPM. I did the first, but it turns out I don't have access to the latter myself. Anyway, feel free to merge this into the main branch/release (when possible). :+1:
@nolanlawson Could you add me (https://www.npmjs.com/~commandoline) and @openscript (https://www.npmjs.com/~openscript, I presume) as NPM collaborators for pouchdb-all-dbs? Thanks!

I've just sent Nolan an email. He's moved on to other things, so it's not surprising he didn't see the pings so far. He did a very good job giving access to other contributors to other PouchDB libraries, but he authored so many it's not surprising that one was overlooked. Especially as pouchdb-all-dbs is as close to 'done' as its gets for a node project...

I also totally understand him.. sometimes just other things become more interesting. Let's hope that he sees the email.  😊
It would be great if Github and NPM had a feature for such situations. Like handing out write access when someone request it for a package or repo which was not maintained for a while. Or even privileges sharing between Github and NPM ..

@openscript We're in luck, Nolan responded already:
marten@procyon:~/git/pouchdb-all-dbs$ npm access ls-collaborators
{
  "nolanlawson": "read-write",
  "commandoline": "read-write",
  "openscript": "read-write"
}

Will you do the honours? (i.e. npm publish)

Awesome! :) Thank you @marten-de-vries for company! Just published 1.1.0. Hopefully I did everything right.. I didn't do this too often.


[Feature][UI Next] Adjust the projects directory

#7928

Codecov Report

Merging #8128 (b70a26e) into dev (98ae331) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                dev    #8128   +/-   ##
=========================================
  Coverage     41.41%   41.41%
  Complexity     3732     3732
=========================================
  Files           637      637
  Lines         27141    27141
  Branches       3089     3089
=========================================
  Hits          11240    11240
  Misses        14814    14814
  Partials       1087     1087

Impacted Files
Coverage Δ

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

...er/master/dispatch/host/assign/RandomSelector.java
83.33% <0.00%> (+5.55%)
:arrow_up:

Continue to review full report at Codecov.

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


Would love more examples

I've created a relatively robust chalice app, and am literally just getting into my very first foray of unit testing (having never done it in python before).
I've created the same confttest and test_app file you created, but honestly have no idea where to go from there.

Thanks for giving me a feedback, I'm gonna add some more examples and enhance the document ASAP


pipenv lock -r unnecessarily produces dependencies included in install_requires which results in a conflict

Issue description
Creating a requirements.txt file with pipenv lock -r > requirements.txt produces dependencies from imported packages with install_requires
Expected result
requirements.txt does not contain dependencies from shared-a  setup.py install-requires
-i https://pypi.org/simple
-e ssh://git@github.com/company/shared_a.git@babfb191416822bc8673bb89d83b720a09a267a9#egg=shared-a
python_full_version >= '3.6.1'

Actual result
requirements.txt contains a dependency from shared-a  setup.py install-requires
-i https://pypi.org/simple
-e ssh://git@github.com/company/shared_a.git@babfb191416822bc8673bb89d83b720a09a267a9#egg=shared-a
ssh://git@github.com/company/shared_b.git@a1ffa042215360f4a16ea7e38dba93aa656f040f8#egg=shared-b
python_full_version >= '3.6.1'

pip install requirements.txt output:
ERROR: Cannot install -r reqs.text (line 3) and shared-b 1.0 (from ssh://git@github.com/company/shared_b.git@a1ffa042215360f4a16ea7e38dba93aa656f040f8#egg=shared-b) because these package versions have conflicting dependencies.

The conflict is caused by:
    The user requested shared-b 1.0 (from ssh://git@github.com/company/shared_b.git@a1ffa042215360f4a16ea7e38dba93aa656f040f8#egg=shared-b)
    shared-a 1.0 depends on shared-b 1.0 (from<EMAIL_ADDRESS>
Steps to replicate
project-a - Contains a Pipfile
shared-a - Contains setup.py file
shared-b - a simple python package
Pipfile in project_a packages section contains the following entry:
shared_a = {editable="true", git =<EMAIL_ADDRESS>setup.py in shared_a contains the following:
setup(name='shared_a',
      version='1.0',
      description='Test',
      author='Me',
<EMAIL_ADDRESS>      license='MIT',
      packages=find_packages(),
      install_requires=[
            'shared_b<EMAIL_ADDRESS>      ])

the WHY
pip is already take care of importing dependencies listed under install_requires

$ pipenv --support
Pipenv version: '2021.5.29'
Pipenv location: '/usr/local/Cellar/pipenv/2021.5.29/libexec/lib/python3.9/site-packages/pipenv'
Python location: '/usr/local/Cellar/pipenv/2021.5.29/libexec/bin/python3.9'
Python installations found:

3.9.6: /Users/ewok/.local/share/virtualenvs/kalamate-shared-model-hW_i7_dZ/bin/python3
3.9.6: /Users/ewok/.local/share/virtualenvs/kalamate-shared-model-hW_i7_dZ/bin/python3.9
3.9.6: /Users/ewok/.local/share/virtualenvs/kalamate-shared-model-hW_i7_dZ/bin/python3
3.9.6: /Users/ewok/.local/share/virtualenvs/kalamate-shared-model-hW_i7_dZ/bin/python3.9
3.9.6: /usr/local/bin/python3
3.9.6: /usr/local/bin/python3.9
3.7.3: /Users/ewok/.pyenv/versions/3.7.3/bin/python3

PEP 508 Information:
{'implementation_name': 'cpython',
 'implementation_version': '3.9.6',
 'os_name': 'posix',
 'platform_machine': 'x86_64',
 'platform_python_implementation': 'CPython',
 'platform_release': '20.6.0',
 'platform_system': 'Darwin',
 'platform_version': 'Darwin Kernel Version 20.6.0: Tue Oct 12 18:33:42 PDT '
                     '2021; root:xnu-7195.141.8~1/RELEASE_X86_64',
 'python_full_version': '3.9.6',
 'python_version': '3.9',
 'sys_platform': 'darwin'}

System environment variables:

PATH
LDFLAGS
TERM
SHELL
CPPFLAGS
TMPDIR
TERM_SESSION_ID
ZSH
__INTELLIJ_COMMAND_HISTFILE__
USER
COMMAND_MODE
SSH_AUTH_SOCK
__CF_USER_TEXT_ENCODING
LOGIN_SHELL
VIRTUAL_ENV
PAGER
LSCOLORS
TERMINAL_EMULATOR
_
LaunchInstanceID
__CFBundleIdentifier
PWD
XPC_FLAGS
XPC_SERVICE_NAME
SHLVL
HOME
LESS
LOGNAME
LC_CTYPE
SECURITYSESSIONID
PIP_DISABLE_PIP_VERSION_CHECK
PYTHONDONTWRITEBYTECODE
PIP_SHIMS_BASE_MODULE
PIP_PYTHON_PATH
PYTHONFINDER_IGNORE_UNSUPPORTED

Pipenv‚Äìspecific environment variables:
Debug‚Äìspecific environment variables:

PATH: /usr/local/Cellar/pipenv/2021.5.29/libexec/tools:/Users/ewok/.local/share/virtualenvs/kalamate-shared-model-hW_i7_dZ/bin:/Users/ewok/.pyenv/shims:/usr/local/bin:/Users/ewok/.pyenv/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/Applications/VMware Fusion.app/Contents/Public:/usr/local/go/bin:/usr/local/share/dotnet:~/.dotnet/tools:/Library/Frameworks/Mono.framework/Versions/Current/Commands
SHELL: /bin/zsh
PWD: /Users/ewok/Desktop/sources/kalamate-shared-model
VIRTUAL_ENV: /Users/ewok/.local/share/virtualenvs/kalamate-shared-model-hW_i7_dZ

</details>

I think the whole point of the lock file, as I understand it, is to lock ALL packages that the project depends on, and snapshot at that point in time their package hashes.  It would be similar to if you took your minimal requirements.txt file that also installed dependent packages through install_requires and then ran pip freeze > requirements.txt because at that point you are documenting what all requirements are to rebuild that virtual environment, even the ones that came from install_requires.
I suspect that Pipenv has had this behavior for a long time, as it is the intended behavior.   Another analogy I could make is comparing it to the npm lock file of a node project.  I haven't found anything in the documentation that would suggest this isn't intended behavior, but more importantly how would you guarantee the package hashes of packages your virtualenv depends on, but aren't in the lock file?


[Need to discuss] has_space_for_loot() changes

5 is not always correct number for gained items from fort.
I have example of gaining 8 items from fort. Maybe this number can be greater?

[00:15:20] Now at Pokestop: Brass Dancers
[00:15:20] Spinning ...
[00:15:22] Loot:
[00:15:22] 100 xp
[00:15:22] - 3x Pokeball (Total: 33)
[00:15:22] - 2x Greatball (Total: 72)
[00:15:22] - 2x Potion (Total: 2)
[00:15:22] - 1x Revive (Total: 31)

9 items poketstop

[13:05:45] Now at Pokestop: Saint Luke's Lutheran Church
[13:05:45] Spinning ...
[13:05:46] Loot:
[13:05:46] 100 xp
[13:05:47] - 4x Pokeball (Total: 51)
[13:05:47] - 2x Greatball (Total: 103)
[13:05:47] - 1x Hyper Potion (Total: 31)
[13:05:47] - 1x Potion (Total: 11)
[13:05:47] - 1x Revive (Total: 32)

Happy to change this value if necessary. It would be good to know what happens when you go over the max number. It's possible that the pokestop won't award that many if you don't have available bag space. It is also possible that you go over your max.

@TheSavior if your inventory is: 349/350, the pokestop will give you all the items and your inventory could be: 353/350, but you can't get more items, so maybe validation is not required.

@robertpro If that is the case, then this should be fine, correct? We could even lower it to 1 theoretically.

@TheSavior yes, I think so we can lower it to 1, 1 unless someone say no.

@Nihisil @douglascamata What are your thoughts? I'm thinking that changing this value to 1 would be fine. It should probably have a comment in the code explaining why 1 is fine: Pokestops will let you go over the max.

Surprisingly, this one line change has merge conflicts. I'd still be happy accepting a change to this value to make it 1.

Closing this PR due to inactivity. If you want to reopen it and fix the merge conflicts, we'd be happy to accept this change


Updated PublisherID

Required items, please complete
Change(s):

Updated PublisherID
Re-generated Solution Package

Reason for Change(s):

Corrected PublisherID for adding Solution in to Azure Marketplace and Content Hub

Version Updated:

No

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Deployed Maintemplate looking good

Arm-ttk also running fine

All are looking good


running scipy.interpolate.tests.test_fitpack::test_bisplev_integer_overflow on 32 bit with debug python segfaults

This is a bit obscure, but it (amongst other things) is currently blocking scipy 1.1 from being part of the next release of Ubuntu.
Reproducing code example:
On Ubuntu Cosmic, at least:
python3-dbg runtests.py -v -t scipy.interpolate.tests.test_fitpack::test_bisplev_integer_overflow

Error message:
scipy/interpolate/tests/test_fitpack.py::test_bisplev_integer_overflow Segmentation fault (core dumped)

Scipy/Numpy/Python version information:
1.2.0.dev0+641bc92 1.14.5 sys.version_info(major=3, minor=6, micro=6, releaselevel='final', serial=0)

As all the other problems I'm having are on architectures other than amd64/x86_64, I feel I should ask: what is the status of scipy support on other architectures? If I file bugs is there anyone who can/will help me with debugging them?

As all the other problems I'm having are on architectures other than amd64/x86_64, I feel I should ask: what is the status of scipy support on other architectures? If I file bugs is there anyone who can/will help me with debugging them?

We aim to support all of them, however it's less "official". Meaning that we don't have CI for anything but 32/64-bit (i686/amd64) on Windows/Linux/macOS, and therefore support is on a best-effort basis as issues are reported. Due to the lack of CI, regressions and build issues tend to show up at the time when Ubuntu or another distro wants to incorporate a new SciPy version.
We'll do our best to help debug. We don't have access to hardware though, so usually it's a lot easier for a Ubuntu/Debian dev to send patches for us to review/incorporate.

We don't have a python-dbg TravisCI build matrix entry, looks like we should add one. NumPy does have one that works, we can steal that.

We aim to support all of them, however it's less "official". Meaning that we don't have CI for anything but 32/64-bit (i686/amd64) on Windows/Linux/macOS, and therefore support is on a best-effort basis as issues are reported. Due to the lack of CI, regressions and build issues tend to show up at the time when Ubuntu or another distro wants to incorporate a new SciPy version.

That all makes sense.
That said I've been seeing failures on 32-bit intel too... the full log is here<EMAIL_ADDRESS>but it's fairly incomprehensible, I should try to reproduce the failures individually with current git and report a bug for each one I guess.
Do you actually run 32 bit Python in CI anywhere? I had a poke but couldn't find a run. (I also think given scipy's scope, dropping 32 bit support would be totally justifiable).

We'll do our best to help debug. We don't have access to hardware though, so usually it's a lot easier for a Ubuntu/Debian dev to send patches for us to review/incorporate.

Well it might have to be a bit more iterative than that, i.e. I paste a failure, someone tells me where to poke and then I have a look and come back with a patch? :)

I should try to reproduce the failures individually with current git and report a bug for each one I guess.

yes, thanks for the ones you already sent

Do you actually run 32 bit Python in CI anywhere? I had a poke but couldn't find a run. (I also think given scipy's scope, dropping 32 bit support would be totally justifiable).

No we don't, forgot about that - we really should. Will open a new issue for that. Numpy has a 32-bit TravisCI build config that we can steal.

I've reactivated build-time testing on Debian builds.  It's not CI testing, but binary rebuilds do happen from time to time. It catches the bisplev segfaults on all 32 bit arches (test failures at build time are currently ignored so the builds continue to completion, which is probably a build configuration bug but convenient for this scipy issue).  https://buildd.debian.org/status/package.php?p=python-scipy
The problem will also show up in Debian build reproducibility tests once 1.2.2-4 starts building on the 32 bit arches there. So this can be considered as CI.
https://tests.reproducible-builds.org/debian/rb-pkg/unstable/i386/python-scipy.html
https://tests.reproducible-builds.org/debian/rb-pkg/unstable/armhf/python-scipy.html


openboardformat.org - Web site is down for maintenance?

Hi,
I've been trying to access https://www.openboardformat.org/ since yesterday, is it down on purpose?

Works now


How to represent rounds and channels in ISS data

For using ISS data one needs to know the "round" and "channel" for each image.
Is it within scope to represent this metadata in SpatialData?
I could not see anything about this here: https://github.com/scverse/spatialdata-io/blob/main/src/spatialdata_io/readers/iss.py

In our case, we assume our 7-round, 5-channel-per-round image is one single multichannel stack, and they are pre-aligned. So, it's in the end one stack with 35 channels. I didn't know about @melonora mentioned. It seems to be a neat solution for now.

Note that it only works if affine transformation is enough. Bilinear transforms are not supported yet but it is in the planning.

you can pass c_coords right now

Would/could that also be serialised to disk?

Yes, it is serialized on disk in the OMERO metadata (handled by ome-zarr-py), you can find the relevant code here in spatialdata and here in ome-zarr-py.

Thanks @LucaMarconato, however I could not find anything specific to store round and channel indices.
Summarising the previous conversation, my feeling is that one can simply define arbitrary channel names or is there anything else that I am missing?

Yes you can indeed just create ordinary channel names and store the information like that. Personally I would give the images the name of 'roundn_suffix_of_choice'  and then store the channel names using 'c_coords'.
As stated before we are going to more properly support this kind of metadata.

Hi all, again, were there any new developments regarding "officially" supporting the notion of imaging rounds and channels in SpatialData.
To be honest, personally, I am not even sure this is a good idea, image data is so diverse that one cannot have a spec for everything :-)
One the other hand, both protein and gene based spatial-omics assays seem to have this data structure...
I think I'd be happy with having one multi-scale image for each round, containing all the channels, and indicate the round in the image name. That should be also easy to generate from microscopy input data, because, I think, typically it would already have this structure.

not directly in SpatialData. I would say this is more some work with the current OME-NGFF challenge on metadata. The structure you state here is also what I use myself and then with c_coords that I mentioned before you can give the names to each individual channel.
I do have some work on metadata and how to support particular specifications, but exactly what the specification looks like is not something SpatialData specific.

I think I'd be happy with having one multi-scale image for each round, containing all the channels, and indicate the round in the image name. That should be also easy to generate from microscopy input data, because, I think, typically it would already have this structure.

This is what currently is implemented 😊


Internal Server Error popup

Hi Guys,
I recently upgraded to OpenSlides Version: 4.0.1-main-20230309-497e22e
When trying to save some settings, e.g. under "Settings" on the left menu, I get a popup at the footer saying:
  <title>Internal Server Error</title>

Internal Server Error

Settings are not saved.
What can I do?
Thanks!

I found this on the logs:
backendAction_1     | [2023-03-09 16:25:51 +0000] [10] [ERROR] Error handling request /system/action/handle_request
backendAction_1     | Traceback (most recent call last):
backendAction_1     |   File "/usr/local/lib/python3.10/site-packages/gunicorn/workers/gthread.py", line 271, in handle
backendAction_1     |     keepalive = self.handle_request(req, conn)
backendAction_1     |   File "/usr/local/lib/python3.10/site-packages/gunicorn/workers/gthread.py", line 323, in handle_request
backendAction_1     |     respiter = self.wsgi(environ, resp.start_response)
backendAction_1     |   File "/app/openslides_backend/http/application.py", line 103, in __call__
backendAction_1     |     return self.wsgi_application(environ, start_response)
backendAction_1     |   File "/app/openslides_backend/http/application.py", line 93, in wsgi_application
backendAction_1     |     response = self.dispatch_request(request)
backendAction_1     |   File "/app/openslides_backend/http/application.py", line 44, in dispatch_request
backendAction_1     |     response_body, access_token = view_instance.dispatch(request)
backendAction_1     |   File "/app/openslides_backend/http/views/base_view.py", line 117, in dispatch
backendAction_1     |     return func(request)
backendAction_1     |   File "/app/openslides_backend/http/views/action_view.py", line 30, in action_route
backendAction_1     |     assert_migration_index()
backendAction_1     |   File "/app/openslides_backend/migrations/__init__.py", line 33, in assert_migration_index
backendAction_1     |     datastore_migration_index = get_datastore_migration_index()
backendAction_1     |   File "/app/openslides_backend/migrations/__init__.py", line 23, in get_datastore_migration_index
backendAction_1     |     datastore_migration_index = read_db.get_current_migration_index()
backendAction_1     |   File "/usr/local/lib/python3.10/site-packages/datastore/shared/postgresql_backend/sql_read_database_backend_service.py", line 347, in get_current_migration_index
backendAction_1     |     raise InvalidDatastoreState(
backendAction_1     | datastore.shared.util.exceptions.InvalidDatastoreState: The datastore has inconsistent migration indices: Minimum is 27, maximum is 39.

However, I did the upgrade and it returned no errors to me:
(...)
Position 1142 from MI 27 to MI 39 ...
Position 1143 from MI 27 to MI 39 ...
Position 1144 from MI 27 to MI 39 ...
Position 1145 from MI 27 to MI 39 ...
Position 1146 from MI 27 to MI 39 ...
Position 1147 from MI 27 to MI 39 ...
Done. Finalizing is still needed.
root@ga:~/openslides# ./openslides migrations finalize -a MYIPADDRESS:443
Progress:
Finalize migrations.
No migrations to apply, but finalizing is still needed. Current migration index: 39
Cleaning collectionfield helper tables...
Calculate helper tables...
Deleting all migration keyframes...
Swap events and migration_events tables...
Set the new migration index to 39...
Clean up migration data...
root@ga:~/openslides#

I suspect, according to the error on the log, there's an issue with the upgrade.
What can I do?
Thanks!

Hi, sorry for the late answer. If it's still relevant and for future reference: I'm unsure of the origin of this error, but assuming the migrations where indeed successfull, you can manually set the migration index to your current version by executing UPDATE positions SET migration_index=39. This may however have unusual errors as a consequence if the migrations were for some reason not successful.

I do have the same problem when following the install instructions of openslides 4 when entering ./openslides initial-data
./openslides check-server gives a Server is ready.
where to enter UPDATE positions SET migration_index=39 ?

This is not a general solution, but rather a last resort. Try other things like restarting the whole stack first (there currently is a known caching problem with the migration index, so this might already help) or running `./openslides migrations finalize".
What exactly is your error when executing ./openslides initial-data? It sounds like another problem.
If you however want to try it anyway (be warned that this may make your database unusable!), you can run the following command to forcefully update the migration index:
docker-compose exec postgres psql openslides -U openslides -c 'UPDATE positions SET migration_index=39'

(You must be in the same folder as the openslides executable and the generated docker compose file for this to work. Also, you must use the maximum migration index available as given by the ./openslides migrations finalize command.)


Cluster TLS certs/keys rotation

@redbaron @c-knowles @danielfm I'm still scratching my head on how to achieve this safely and automatically.
For now, I'm recreating clusters until the certs expire but that's painful!
In general, we'd want to replace all the certs/keys targeted for rotation across all the nodes very fast, without replacing the node one by one just for cert/key update, right?
// Please see https://github.com/kubernetes/kubernetes/issues/25379#issuecomment-281533623 for more context.
Otherwise, delay in update process could result in downtime. For example, when apiservers w/ old key/cert on their nodes got restarted due to some reasons after etcd-server key/cert rotation on etcd nodes but before etcd-client key rotation on controller nodes, each restarted apiserver will fail connecting etcd with old key.
Related topics:

For kubelets we can rely on TLS bootstraping. See #922.
For etcd-ca, #907

I'm not really wanting to introduce an another moving-part to kube-aws but would it be feasible to utilize aws-ssm-agent or somethig for fast key/cert rotation?

I think for rotation the minimum requirement is for the target system to support two public keys as noted here. We could use that facility and introduce a new kube-aws command or deal with it automatically in each update call?

I found myself needing to rotate credentials today which is really painful.
The --service-account-key-file option accepts public and private key files, perhaps we can keep previously generated apiserver cert files and use them for this option? Currently this option is set to apiserver-key.pem which is recreated each time kube-aws render credentials.
I think we can do this as part of kube-aws render credentials, but I think it is also okay to do this in a new command.

@cheungpat Thanks a lot for reminding that flag! I have been forgotten about that since https://github.com/kubernetes-incubator/kube-aws/issues/907#issuecomment-345665123
Yes, it would work.
Regarding the command, something like kube-aws rotate service-account-key would be cool but i'm unsure whats a realistic option from implementors perspective, as you need multiple steps to complete a rotation, even with the flag.

Add a new private and public key. The public key is appended to the pem file passed to service-account-key-file.
Ensure that old service account tokens are deleted and regenerated(any way to automate this process?
Remove the old pub key from the pem file.

For example, how render credentials work on the 3rd point?

Also see #1146 #1147

Rotating of service account keys comes with risks - I wouldn't do it until there is a consensus within k8s as to how it should be done.  The issue is that you can't rotate out old service keys without regenerating any service accounts created by them, and restarting any pods that use those service accounts (which is disruptive).
I've done live certificate rotation before on k8s but the implementation was vault specific.  I don't personally believe that the cluster deployment tool, i.e. kube-aws, should be run in order to rotate certs- I do think, however, that there should be an automated facility utilising an online pki/api such as Vault.


how to use cfltk with scons

I don't much like cmake since it never seems to work right for me.
However scons works great.
I'd like to try cfltk.
I have done a git clone of cfltk in $HOME/opt/cfltk and I have my project (a copy of the first example) in $HOME/app/c/hellofltk.
My SConstruct file is:
AddOption('--dbg', dest='dbg', action='store_true', default=False,
          help='Debug build')
env = Environment()
if GetOption('dbg'):
    env.Append(CCFLAGS='-Wall -Wextra -pedantic -g')
else:
    env.Append(CCFLAGS='-Wall -s -O3 -DNDEBUG')
FLTK = '/home/mark/opt/cfltk'
env.Append(CPPPATH=['.',  f'{FLTK}/include'])
env.ParseConfig([f'{FLTK}/build/fltk/fltk-config', '--cflags',
                 '--ldstaticflags'])

app = env.Program('hellofltk', ['hellofltk.c'])

Default(app)
appname = str(app[0])
runner = Command(target=f'run_{appname}', source=app,
                 action=f'-./{appname}')
alias = Alias('run', runner)

When I try building I get:
$ scons
scons: Reading SConscript files ...
scons: done reading SConscript files.
scons: Building targets ...
gcc -o hellofltk.o -c -Wall -s -O3 -DNDEBUG -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -D_THREAD_SAFE -D_REENTRANT -I. -I/home/mark/opt/cfltk/include -I/home/mark/opt/cfltk/build/fltk -I/home/mark/opt/cfltk/fltk -I/home/mark/opt/cfltk/fltk/jpeg hellofltk.c
gcc -o hellofltk hellofltk.o /home/mark/opt/cfltk/build/fltk/lib/libfltk.a -lm -lX11 -lXext -lpthread -lXinerama -lXfixes -lXcursor -lXft -lXrender -lfontconfig -ldl
/bin/ld: hellofltk.o: in function `cb':
hellofltk.c:(.text+0x8): undefined reference to `Fl_Widget_set_label'
/bin/ld: hellofltk.o: in function `main':
hellofltk.c:(.text.startup+0x7): undefined reference to `Fl_init_all'
/bin/ld: hellofltk.c:(.text.startup+0xc): undefined reference to `Fl_register_images'
/bin/ld: hellofltk.c:(.text.startup+0x11): undefined reference to `Fl_lock'
/bin/ld: hellofltk.c:(.text.startup+0x2d): undefined reference to `Fl_Window_new'
/bin/ld: hellofltk.c:(.text.startup+0x50): undefined reference to `Fl_Button_new'
/bin/ld: hellofltk.c:(.text.startup+0x5b): undefined reference to `Fl_Window_end'
/bin/ld: hellofltk.c:(.text.startup+0x63): undefined reference to `Fl_Window_show'
/bin/ld: hellofltk.c:(.text.startup+0x74): undefined reference to `Fl_Button_set_callback'
/bin/ld: hellofltk.c:(.text.startup+0x7f): undefined reference to `Fl_run'
collect2: error: ld returned 1 exit status
scons: *** [hellofltk] Error 1
scons: building terminated because of errors.

For cfltk itself all I've done so far is:
cd ~/opt
git clone https://github.com/moalyousef/cfltk
cd cfltk
git submodule update --init --recursive
mkdir build
cd build
cmake ..
make

Hi Mark
It’s my first time looking at a SConstruct file. It seems to me you’re linking fltk libs by oarsing the output of fltk-config ldstaticflags. This however misses the libcfltk.a library that’s generated by your CMake build.
I guess it would be something like:
app = env.Program('hellofltk', ['hellofltk.c'], LIBS = ['cfltk'], LIBPATH = [f'{FLTK}/build/lib'])

Hi Mo,
I added libcfltk.a but that led to even more errors (although different). Here's the start:
scons: Reading SConscript files ...
scons: done reading SConscript files.
scons: Building targets ...
gcc -o hellofltk hellofltk.o -L. -L/home/mark/opt/cfltk/build /home/mark/opt/cfltk/build/libcfltk.a /home/mark/opt/cfltk/build/fltk/lib/libfltk.a -lm -lX11 -lXext -lpthread -lXinerama -lXfixes -lXcursor -lXft -lXrender -lfontconfig -ldl
/bin/ld: /home/mark/opt/cfltk/build/libcfltk.a(cfl_image.cpp.o): warning: relocation against `_ZTI12Fl_SVG_Image' in read-only section `.text'
/bin/ld: /home/mark/opt/cfltk/build/libcfltk.a(cfl.cpp.o): in function `Fl_Widget_Tracker_delete':
cfl.cpp:(.text+0x906): undefined reference to `operator delete(void*, unsigned long)'
/bin/ld: /home/mark/opt/cfltk/build/libcfltk.a(cfl.cpp.o): in function `Fl_load_system_icons':
cfl.cpp:(.text+0x135e): undefined reference to `Fl_File_Icon::load_system_icons()'

This was only out of curiosity since I've been relearning C recently. I'll give xcb a go instead.
Thanks!

I found xcb far too low level and also tried exforms2 which wasn't satisfying. So it would be nice to be able to try cfltk, but I really dislike cmake so want to use scons.

This seems to link to the fltk bundled libz. Try linking to libfltk_z.a

That library doesn't exist in cfltk.
However it does exist in go-fltk so I copied it and tried using it to no avail:
$ scons
scons: Reading SConscript files ...
scons: done reading SConscript files.
scons: Building targets ...
gcc -o hellofltk.o -c -Wall -s -O3 -DNDEBUG -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -D_THREAD_SAFE -D_REENTRANT -I. -I/home/mark/opt/cfltk-lib/include/cfltk -I/home/mark/opt/cfltk-lib/include/FL -I/home/mark/opt/cfltk-lib/include -I/home/mark/opt/cfltk-lib/include/FL/images -I/home/mark/opt/cfltk-lib/include/zlib -I/home/mark/opt/cfltk-lib/include/jpeg hellofltk.c
gcc -o hellofltk hellofltk.o -L. /home/mark/opt/cfltk-lib/lib/libfltk_z.a /home/mark/opt/cfltk-lib/lib/libcfltk.a /home/mark/opt/cfltk-lib/lib/libfltk_forms.a /home/mark/opt/cfltk-lib/lib/libfltk_images.a /home/mark/opt/cfltk-lib/lib/libfltk_jpeg.a /home/mark/opt/cfltk-lib/lib/libfltk_gl.a /home/mark/opt/cfltk-lib/lib/libfltk.a -lm -lX11 -lXext -lpthread -lXinerama -lXfixes -lXcursor -lXft -lXrender -lfontconfig -ldl
/bin/ld: /home/mark/opt/cfltk-lib/lib/libfltk_images.a(Fl_SVG_Image.cxx.o): undefined reference to symbol 'inflateEnd'
/bin/ld: /lib/x86_64-linux-gnu/libz.so.1: error adding symbols: DSO missing from command line
collect2: error: ld returned 1 exit status
scons: *** [hellofltk] Error 1
scons: building terminated because of errors.

My guess is the link order matters. I think CMake resolves link order, not sure about scons.
https://stackoverflow.com/questions/45135/why-does-the-order-in-which-libraries-are-linked-sometimes-cause-errors-in-gcc
That means /home/mark/opt/cfltk-lib/lib/libfltk_z.a has to come after /home/mark/opt/cfltk-lib/lib/libfltk_images.a and /home/mark/opt/cfltk-lib/lib/libfltk_png.a both of which depend on libz. Normally if you didn't find a libfltk_z in the cfltk build directory, it means that it's using your installed libz. in that case -lz has to come after libfltk_images and libfltk_png.

@

I did try reordering but to no avail. So I'm giving up.


obstacles in a negative area are likely not drawn correctly.

    "obstacles": [
        {
            "name": "obst_top",
            "vertices": [
                [0, 5],
                [200, 5],
                [200, 10],
                [0, 10]
            ]
        },
        {
            "name": "obst_bottom",
            "vertices": [
                [0, -5],
                [200, -5],
                [200, -10],
                [0, -10]
            ]}
    ]


Failed to load NIF library: geef_nif.so: undefined symbol:

Trying to compile gitgud will lead to the following error:
Debian (Docker (elixir-1.9.1 image) and Jessie (no Docker)):
23:28:53.168 [error] Process #PID<0.6967.0> raised an exception
** (RuntimeError) got 'Failed to load NIF library: \'/gitgud/_build/dev/lib/gitrekt/priv/geef_nif.so:                                undefined symbol: git_diff_to_buf\'' while retrieving Exception.message/1 for %RuntimeError{mes                               sage: 'Failed to load NIF library: \'/gitgud/_build/dev/lib/gitrekt/priv/geef_nif.so: undefined symbo                               l: git_diff_to_buf\''} (expected a string)
    lib/gitrekt/git.ex:120: GitRekt.Git.load_nif/0
    (kernel) code_server.erl:1340: anonymous fn/1 in :code_server.handle_on_load/5

Ubuntu (Docker (built from 18.04 image)):
23:28:53.168 [error] Process #PID<0.6967.0> raised an exception
** (RuntimeError) got 'Failed to load NIF library: \'/gitgud/_build/dev/lib/gitrekt/priv/geef_nif.so:                                undefined symbol: git_commit_committer\'' while retrieving Exception.message/1 for %RuntimeError{mes                               sage: 'Failed to load NIF library: \'/gitgud/_build/dev/lib/gitrekt/priv/geef_nif.so: undefined symbo                               l: git_commit_committer\''} (expected a string)
    lib/gitrekt/git.ex:120: GitRekt.Git.load_nif/0
    (kernel) code_server.erl:1340: anonymous fn/1 in :code_server.handle_on_load/5

Both use the same erlang + elixir version:
Erlang/OTP 22 [erts-10.5.1] [source] [64-bit] [smp:4:4] [ds:4:4:10] [async-threads:1] [hipe]

Elixir 1.9.1 (compiled with Erlang/OTP 20)

On Ubuntu the following libgit2-dev version is installed: v0.26.0
On Debian the following libgit2-dev version is installed: v0.25.1
This error seems to be (loosely) related to #61. However adding -Wl,--no-as-needed to gitrekt makefile does not resolve the issue for me.
I am opening this issue ticket here, because you maintainers know certainly more about geef than me. If this is a bug in geef, please open an issue on the related issue tracker.

Here's the complete log of the debian docker container.
buildlog.txt

This error seems to be (loosely) related to #61. However adding -Wl,--no-as-needed to gitrekt makefile does not resolve the issue for me.

From your output logs, this seems very related to #61. Can you runldd on geef_nif.so (see comment) and paste the results here?
Did you modify the LD_FLAGS variable in Makefile as indicated here? Could you run make clean && make in apps/gitrekt and past the output here?

@redrabbit

From your output logs, this seems very related to #61. Can you runldd on geef_nif.so (see comment) and paste the results here?

ldd.txt

Did you modify the LD_FLAGS variable in Makefile as indicated here? Could you run make clean && make in apps/gitrekt and past the output here?

make.txt
After make, recompiling using mix compile --force does not change the error or output.

I've just tried with a fresh Ubuntu 19.04 and did not get any trouble compiling and running the server. I'll check with 18.04 asap.
Could you maybe try with Ubuntu 19.04. I don't really understand how we can get different results using Docker containers.
How did you install Elixir and Erlang?  With the standard Ubuntu package repositories, I've got version 1.7.4 installed.

You are right, on 19.04 it works - but only with the changes to gitrekt's Makefile. libgit2-dev is 0.27.7 on 19.04 - I'll try to see if I can manually compile a newer version of libgit2 on 18.04 and test if it's related to the version.

Update: I installed in the debian container libgit2-dev 0.27.4 from stretch-backports - with the backport I can successfully compile the application.

@CharlotteDunois did you have to update the Makefile as well with Debian or did it work out of the box?


steampipePackages.steampipe-plugins-azure: init at 1.0.0

Description

https://github.com/turbot/steampipe-plugin-azure

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[X] sandbox = true

[ ] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 351808

x86_64-linux

:white_check_mark: 1 package built:

steampipePackages.steampipe-plugin-azure

aarch64-darwin

:white_check_mark: 1 package built:

steampipePackages.steampipe-plugin-azure

Thank you, and welcome to the NixOS community!
Your PR looks good to me! :+1:
However, there is just one error in the commit messages.
The PR should contain 2 separate commits for the package and the maintainer list addition:

steampipePackages.steampipe-plugin-azure: init at 1.0.0
maintainers: add petee

Thank you, and welcome to the NixOS community! Your PR looks good to me! 👍

Thank you for the warm welcome!  I'm excited to be here.

However, there is just one error in the commit messages. The PR should contain 2 separate commits for the package and the maintainer list addition:
* maintainers: add petee

I believe I've done this correctly (after many tries).  I'm glad to learn some new git tricks today!
I will probably have a few more PRs coming for some of these steampipe plugins.  :)


migrate test for read-only Java editor mode from Selenium

Reimplementation of https://github.com/droolsjbpm/kie-wb-common/pull/364

jenkins retest this

@wmedvede Could you please review this for @barboras7 thanks.

@wmedvede yes, I forgot to remove that comment, it is fixed now

jenkins retest this

+1, thanks


Update couchbase-server-community to 4.5.0

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask's name and version.

Thank you for the contribution. It was merged directly as e368328d7bd8dceab212408a7c2f56baa76c584a to keep commit history cleaner. Your contribution is still credited to you.


Fix context menu overflow & review tag margin

Resolves #9161

Also applies a small CSS fix for Review tags.
Before

After

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@a00d720). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #9231   +/-   ##
=========================================
  Coverage          ?   41.09%
=========================================
  Files             ?      555
  Lines             ?    72393
  Branches          ?        0
=========================================
  Hits              ?    29748
  Misses            ?    38900
  Partials          ?     3745

Continue to review full report at Codecov.

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


Compile Issue

Hello There,
I am interested in your project, and I was wondering if you could point me in the right direction for compiling your engine?
I have all of the dependencies required, but I am attempting to compile it on MacOS. I would like to contribute to this project in some way, and I will continue to search for how to compile it with MacOS.
The error I am getting is the following:
CMake Error at cmake/FindGWEN.cmake:6 (add_subdirectory):
The source directory
 ~/RadixEngine/RadixEngine/external/gwen

does not contain a CMakeLists.txt file.

Call Stack (most recent call first):
CMakeLists.txt:85 (find_package)
CMake Error at cmake/FindVHACD.cmake:9 (add_subdirectory):
The source directory
  ~/RadixEngine/RadixEngine/external/vhacd-lib

does not contain a CMakeLists.txt file.

Call Stack (most recent call first):
CMakeLists.txt:99 (find_package)
 Found FreeImage: headers at /usr/local/include, libraries at  :: /usr/local/lib/libfreeimage.dylib

CMake Error at cmake/FindRadixEntity.cmake:7 (add_subdirectory):
The source directory
  ~/RadixEngine/RadixEngine/external/RadixEntity

does not contain a CMakeLists.txt file.

Call Stack (most recent call first):
CMakeLists.txt:117 (find_package)
--
To me it looks like a missing file CMakeLists might be necessary, when I check the directory, I found that there was nothing at all in the directory.
Thanks

@amwoods3 Check the command here: https://github.com/GlPortal/glPortal/blob/master/COMPILE.md#downloading-source-code
git clone https://github.com/GlPortal/glPortal.git
cd glPortal
git submodule update --init --recursive

@amwoods3 You might still get some errors in a logger class. However you should be able to comment out the offending lines.

Should the build directory be put directly into the RadixEngine directory? If I try to build in glPortal, it tells me RadixEngine is missing. If I try to build in RadixEngine, it gives me the same error as previously.

@amwoods3 Yes. Neither glportal nor radix have their submodules initialized. The git submodule command fixes that.

@amwoods3 This will initialize and update all the submodules for glportal and radix in one command. Result: Said directories will no longer be empty.

Got it, I did the submodule for the RadixEngine, and it works now. I found the logger error that you were talking about, which was the only error. I started the make again, and it compiled successfully.
Thanks.

@amwoods3 Sweet. If you run glportal you can have a look at the engine in action. You can chat with me on irc and gitter. #glportal on freenode and https://gitter.im/GlPortal/glPortal

@amwoods3 I am also interested in fixes to these logger problems if you like to cook up a patch.

Thanks. I actually only built the RadixEngine. I am kind of new with CMake as I have never used it before, so I don't know what the glPortal is trying to do in order to find RadixEngine (it says it cannot find it).

@amwoods3 The following commands should be enough to get glportal started for you.
git clone https://github.com/GlPortal/glPortal.git
cd glPortal
git submodule update --init --recursive
cmake ./
make run


docs: add note about test mode for deployment commands

Stack:

#38 ⬅
#37
#36
#35

⚠ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.

alpha

#35

#36

#37

#38 :point_left:

#39

Closing pull request: commit has gone away


Turn off raw when drawing

This enables raw mode only when we're taking in input from the user, otherwise it turns it off so drawing doesn't have to worry about it

seems to straighten out my multi-line history.

but wow, on the last line of the terminal, fuggedabowdit.

and until you hit enter


Find-PSResource -Name * does not return anything on 1.0.5+

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.

Steps to reproduce
Find-PSResource -Name * does not return any results on 1.0.5+.
Expected behavior
All resources in a repository are returned

Actual behavior
No resources in a repository are returned

Error details
N/A

Environment data
1.0.5

Visuals
No response

Seems it errors because $filter gets added twice.
PS > Find-PSResource -Name '*' -Repository 'PSGallery' -Debug

DEBUG: In FindPSResource::ProcessResourceNameParameterSet()
DEBUG: Filtering package name(s) on wildcards
DEBUG: In FindHelper::FindByResourceName()
DEBUG: Parameters passed in >>> Name: '*'; ResourceType: 'None'; VersionRange: ''; NuGetVersion: ''; VersionType: 'NoVersion'; Version: ''; Prerelease: 'False'; Tag: ''; Repository: 'PSGallery'; IncludeDependencies 'False'
DEBUG: Searching through repository 'PSGallery'
DEBUG: In FindHelper::SearchByNames()
DEBUG: No version specified, package name is '*'
DEBUG: In V2ServerAPICalls::FindAll()
DEBUG: In V2ServerAPICalls::FindAllFromTypeEndPoint()
DEBUG: In V2ServerAPICalls::HttpRequestCall()
DEBUG: Request url is 'https://www.powershellgallery.com/api/v2/items/psscript/Search()?$filter=$filter=IsLatestVersion&$inlinecount=allpages&$skip=0&$top=6000&$orderby=Id+desc'
DEBUG: Response is empty
Find-PSResource: 'Response status code does not indicate success: 400 (Bad Request).' Request sent: 'https://www.powershellgallery.com/api/v2/items/psscript/Search()?$filter=$filter=IsLatestVersion&$inlinecount=allpages&$skip=0&$top=6000&$orderby=Id+desc'

PS >

If going to that URL in the browser:
<m:error xmlns:m="http://schemas.microsoft.com/ado/2007/08/dataservices/metadata">
<m:code/>
<m:message xml:lang="en-US">URI or query string invalid. Syntax error at position 8 in '$filter=IsLatestVersion'.</m:message>
</m:error>


Options should be placed over/outside containing DIV

I'm running into an issue when a MultiSelect is too near the bottom of a div, that the options are 'cut off'.
In the image below the 'darker grey' is the containing div, the next couple of options below "Avestan" are not visible due to not enough height:

In my opinion the options should be popping out of the select box, thus be overlaying everything else. And therefore also be visible outside a container div.

Did you set overflow: hidden on the parent div? To my knowledge, there are only two ways around this, neither are CSS-only solutions.

MultiSelect violates component encapsulation patterns and changes its parent element style to overflow: visible. Probably a bad idea since this can have other unintended side-effects.
Use position: fixed instead of the current position: absolute on the ul.options since position: fixed is unaffected by parent container overflow. But that would require JS to get the current component location on the page and its width to position and size the dropdown correctly. Might look awkward during scrolling, have not tried.

So the simplest fix would be for you to not put overflow: hidden on the parent div... if you did and removing it is an option for you. If not, could you share more details about your use case?

Another option would be to display the dropdown above the input if insufficient space below is detected. Might also look a bit weird though.

I'll close this as user responsibility for now. Meaning devs need to ensure the options dropdown has sufficient vertical space not to be obscured. As outlined above, I don't think there's a good one-size-fits-all solution from within svelte-multiselect. Happy to reopen this issue if someone wants to suggest a solution.


Test and add AWS ECS to the list of supported registries

According to this blog post, AWS ECR should have support for storing OCI artifacts, meaning it should accept Wasm modules as well.
We should test that this is the case, then add it to the list.
If anyone with an AWS account wants to test it, that would be great!

I can test this

I came here to post an issue to track a potential PR and I found this one. Cool.
I have tested it a few minutes ago and it (seems) to work just fine:
$ aws ecr get-login-password --region us-west-2 | docker login --username AWS --password-stdin<PHONE_NUMBER>39.dkr.ecr.us-west-2.amazonaws.com
Login Succeeded

$ wasm-to-oci push python3.wasm<PHONE_NUMBER>39.dkr.ecr.us-west-2.amazonaws.com/python3.wasm:latest
INFO[0009] Pushed:<PHONE_NUMBER>39.dkr.ecr.us-west-2.amazonaws.com/python3.wasm:latest
INFO[0009] Size: 26193349
INFO[0009] Digest: sha256:6e723ad57cadd700e0fed37aa62b0dbe973af7597efaabac4265ad53ff90901e

$ rm python3.wasm

$ wasm-to-oci pull<PHONE_NUMBER>39.dkr.ecr.us-west-2.amazonaws.com/python3.wasm:latest --out python3.wasm
INFO[0010] Pulled:<PHONE_NUMBER>39.dkr.ecr.us-west-2.amazonaws.com/python3.wasm:latest
INFO[0010] Size: 26193349
INFO[0010] Digest: sha256:a29013c02f0b76edbf2d7f57ca2f60c71cf33925119ebd3f7fb8c1d8c313386d

$

PS the subject of this issue says ECS but it should really say ECR


Can't seem to limit to just the first color option in template

I'm trying to return the first color for a field's selection in the template instead of looping through them:
	{% set style = block.styling.theme.color.one() %}
	{{ style .color }}

And I get this error:
Impossible to invoke a method ("one") on an array.

Shouldn't one() work in this case?
My color swatch field is inside a Matrix field and then inside a Supertable field.

Hi @mediagirl,
We don't use the .one() notation in our plugin. You can update your piece of code like this in order to work:
{% set style = block.styling.theme.color[0] %}
{{ style .color }}

I hope this helped out your issue?

To clarify @mediagirl:
.one() notation can only be used on an Element Query, since our colour swatches are simply arrays you can't use .one() .all() .count() or any of those. You'd simply loop over the colours or you pick a specific one with the bracket notation as @cookie10codes mentioned here above.

I was able to get to the block.styling.theme.color[0] solution last night after Googling for a bit.
I didn't realize one() was only on on an Element Query. Thank you for that tidbit!


Update to 1.2.0 and make noarch

Fixes #13
Fixes #10

Checklist

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

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge/happybase This is ready for review/merge.


[folding] finish language-aware folding

Finish the work on the HTML and JSON folding providers:

handle cancellation
limit the number of ranges transmitted
support multi-line comments
stable ranges while editing

Investigate:

custom folding decorations (...) (for #31966)

@aeschli I noticed this is in the release notes for March but currently only a proposed API - is this planned to ship as a stable API in this release? (If so, will it be moved to vscode.d.ts in advance? dev is simpler when things aren't proposed 😁)

Yes, this is planned to go to vscode.d.ts once we have enough confidence and reviews.

@aeschli I'm slightly confused - the top of the release notes say there's language-aware folding, but further down it says the API is proposed. It's unclear whether I can actually implement and ship this yet?

These are the built-in extensions from VSCode

Ah, didn't realise they could use them, all makes sense now!
I was trying to figure out if I could put them in my release towards the end of the month but I guess it'll need to wait. If I get chance after completing the stuff for this release I'll wire this up on a branch and ensure it all works for us. Thanks!


Issues with comparisons

Cuid2 appears to be a good format and library overall, but the readme has some issues around its comparisons with other formats.
The comparisons to other id formats seem to conflate some things when making comparisons:

The spec and output format
The implementation and PRNG used

Taking UUIDv4 as an example - going by the spec, UUIDv4 should roughly match in every aspect other than "Not URL or name friendly".
If we break it down:

Cuid2 uses: Cuid2 CSPRNG -> ~124 random bits -> Cuid2 text
UUIDv4 with a CSPRNG uses: CSPRNG -> 122 random bits -> UUID

You could use the Cuid2 CSPRNG in an UUIDv4 implementation, and get exactly the same security (ignoring the 124 vs 122 bits). So what you're really comparing is the security of the CSPRNG, rather than the format itself.
There is something to be said for specifying the CSPRNG used in the format, versus UUIDv4 that leaves it up to the implementation. JWTs had similar issues where the spec wasn't specific enough about some details, leading to major security issues in many implementations.
When comparing the PRNGs
As long as the PRNG satisfies the required properties for a CSPRNG, there is little meaningful difference between different implementations.

Id generation should be fast enough that humans won't notice a delay, but too slow to feasibly brute force (even in parallel)

The faster you can generate ids, the faster you can run collision attacks

If you start at the output of the PRNG, you can generate IDs equally fast - the speed of the PRNG doesn't matter. You have 122+ bits of entropy either way, and there's no way you're going to find a collision.
Using the PRNG itself to generate collisions only makes sense if the PRNG isn't secure to begin with.
A slow PRNG isn't a requirement for security.

Cuid2 uses multiple, independent entropy sources and hashes them with a security-audited, NIST-standard cryptographically secure hashing algorithm (Sha3).

That the hashing algorithm is NIST-standard doesn't prove on its own that the Cuid2 PRNG is secure - it's just one component of the PRNG. NIST does actually go further and provides recommendations for creating a CSPRNG using hash functions or other crypto primitives - why are one of those recommendations not used?
Using a completely custom PRNG here instead of using a known secure has the possibility of introducing new security issues.
For example, the Wikipedia page on CSPRNGs mentions this requirement:

Every CSPRNG should withstand "state compromise extension attacks". In the event that part or all of its state has been revealed (or guessed correctly), it should be impossible to reconstruct the stream of random numbers prior to the revelation.

Does the Cuid2 implementation meet that requirement? Has that been audited?

there may be bugs in browser CSPRNGs

The linked Chromium issue was from 2015/2016. Are there any currently known issues / weaknesses? Otherwise there is no more reason to believe there is a bug in the browser CSPRNGs than there being a bug in Cuid2.

Thanks for bringing up these questions!
Where's the maintainer? This issue is over a year old and without a response. That doesn't make it easy for me to trust this library, unfortunately.


Sleuthkit from GIFT is missing libewf support

Must rebuild into the SIFT repository with a high version number.

This should be resolved now.


What's the meaning of fhd,lite and onestage in config files

That means there is no downsample layer in RPN, RPN consist of just some simple convolutional layers.

So lite means no layer in RPN, and onestage means RPN consist of just some simple convolutional layers?


Translate the index page to Japanese

Translated the index page.
@JohnTitor please review it!

@phil-opp  translation and review is complete. Please merge this.

@phil-opp  translation and review is complete. Please merge this.

Thanks!

Thanks!


Provide Angular-based UI

Leaving this PR open even if, finally, I decided to use plain Golang templating to serve HTML files inside the addon.
I never got the Angular UI to work properly inside the HA addon, cause I should be able to customize all URLs to start with a particular prefix (%%ingress_entry%%).


Check augmented wheel icons when CRUD operations are performed on the table

Right now, the augmented wheels are ignored when a table is renamed or deleted.
The Studio should know about its own file modification and take are of the during these CRUD operations.

Fixed with 2.11.2


Field description

Useful for documenting the schema of an index and further display. Seen in BigQuery ("description"), Hive ("comment"), ...

I strongly agree.

Closed via #1563.


Attaching last targeted is failing

https://ptb.discord.com/channels/915186263609454632/1167221795103985764/1256848798857170965
Foundry: v12.328
D&D5e: v3.2.1
Walled Template: v0.8.0
libWrapper: v<IP_ADDRESS>

Closing as not replicable.


new external script to adds quotes to comma-separated lists

Looks great, two comments:

Empty input should produce empty output
name should be capitalized with Chicago Manual of Style

Fixed it


z-base 32: potential bug: decoding inaccuracy

I'm not sure if I expect the library to do something that the algorithm doesn't support, but Wiry.Base32.Base32Encoding.ZBase32.ToBytes, given different values, produces exact same set of bytes. For example, this particular string, and any other that is comprised by the same and another character appended, produces the same set of bytes below.
base:
"gr3doqbw8radnqb3go"
other that produce same binary:
"gr3doqbw8radnqb3goa"
"gr3doqbw8radnqb3gob"
"gr3doqbw8radnqb3goc"
"gr3doqbw8radnqb3god"
{byte[11]}
[0]: 49
[1]: 50
[2]: 56
[3]: 56
[4]: 52
[5]: 57
[6]: 48
[7]: 49
[8]: 56
[9]: 57
[10]: 52
I guess, if the length of the encoded argument doesn't match the algorithm, I would expect an error, rather than trailing characters being ignored. This is very important to me, as I'm using these values as data identifiers, and while "gr3doqbw8radnqb3go" points to existing data record, I don't want to make it appear that ""gr3doqbw8radnqb3goa" is an alias to it. Please provide some guidance. If there is any work-around (e.g. my validating the length of the string using some formula, prior to feeding it to the 'ToBytes' method), please tell me.
Thanks!

For the time being, I'm decoding the entrusted zbase32 input, then encoding the decoding-result back to zbase32, then comparing the encoding-result with the untrusted input. For example:
string untrusted-zbase32:
"gr3doqbw8radnqb3goa"
Base32Encoding.ZBase32.ToBytes + Encoding.UTF8.GetString  produce decoding-result:
"12884901894" (which is same the result when "gr3doqbw8radnqb3go" + any-[valid]-char is decoded)
Then, to validate, I zbase32 encode the "decoding-result":
Base32Encoding.ZBase32.GetString("12884901894"), which gives me this:
"gr3doqbw8radnqb3go"
Since "gr3doqbw8radnqb3go" != "gr3doqbw8radnqb3goa" (trailing "a" char), I conclude that the input is bad.
Any other more performant suggestion would be appreciated.
Thanks.

zbase32 encoding have some unobvious "alias" combinations. For example "r3oo" and "r3oi" decoded as "&a" string. If we convert "&a" string to zbase32 we get "r3oo" string, so "r3oo" is a primary value rather than "r3oi" (an alias).
Best workaround is: generate an identifiers as a bytes and then convert it to zbase32 by Base32Encoding.ZBase32.GetString method.
But I agree, that current input validation is poor and this is cause a bug you have found.

I plan to add a method to verify input zbase32 string. So ToBytes method will just work with "primary" and with "alias" values, but users can be able to check their data if they need.


Adds migrations version

Adds initial migrations

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

In the file services/server/migrations/versions/001_initial_migrations.py:

Line 4:9: W291 trailing whitespace


Add alternatives key for listing other possible language matches

Contains files that are classified using the fallback language and lists what languages it could be.
Lists files and a list of matches under files.alternatives

Resolves #15

Should alternatives include the result?

"files": {
  "/x.pluginspec": "Ruby",
},
"alternatives": {
  ".pluginspec": ["Ruby", "XML"],
},

or

"files": {
  "/x.pluginspec": "Ruby",
},
"alternatives": {
  ".pluginspec": ["XML"],
},

?

Should alternatives include the result?

I'll go with yes

Should alternatives include the result?

I'll go with no so that if (alternatives) {...} may be used


Self Signup Page

Create a page where a user can signup for an account.
This should collect:

Email
Name
Password
Accept T+Cs and privacy policy
Organisation Name

Should send to the mutation created in https://github.com/impactasaurus/server/issues/169.
If 200, send to login page.

 Self signup

Decided to let first login through without email validation.


truncated menu on right-clicked menu

in right-click menu, if we click on higher item position, the menu will show like this:

I hope we can fix this, so the menu will never truncated by screen anymore

Please adhere to the issue template.


WIP: Initial implementation of daily backups to cloud storage.

Also, I think it's totally fine, and maybe even preferable, if we write all of the backup logic in python, and throw it in as a docker service.
If we're concerned about reliability, we can include a heartbeat discord ping that keeps us in the loop that our backups are firing every day or every week or however often we want a heartbeat discord ping.


Update Common Voice recipes

@lucadellalib

@poonehmousavi

You should run the experiment starting from the latest dev branch. You can
start the experiments for all the languages except French. Salima should
update the French preprocessing soon.
On Thu, Jun 22, 2023 at 11:31 AM Pooneh Mousavi @.***>
wrote:

@mravanelli https://github.com/mravanelli Should I use the main branch
for running the experiments? Or there are some updates in other pull
requests that need to be tested for common-voice recipes like Salima's
update to french preprocessing?
—
Reply to this email directly, view it on GitHub
https://github.com/speechbrain/speechbrain/issues/2039#issuecomment-1602849068,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEA2ZVQG66OG4Z45SM7BDEDXMRQN3ANCNFSM6AAAAAAZNLP2KY
.
You are receiving this because you were mentioned.Message ID:
@.***>

You should run the experiment starting from the latest dev branch.

Hey @mravanelli, what do you think about training smaller Whisper models for each languages? On the hub we have several Whisper large-v2 models, but they consumes a lot of vram and they are slow at inference time. I propose to train (and release on the HF hub) Whisper models such as the base (74M) or the small (244M) instead of the large-v2 (1550M). Furthermore, if we have time, it would be great to replicate a little bit what @bofenghuang did in his blog post (see: https://medium.com/@bofenghuang7/what-i-learned-from-whisper-fine-tuning-event-2a68dab1862) in terms of hyper parameters (I believe we should be able to improve the WER with this setting).

Yes,
we can release a fine-tuned Whisper for all the languages currently in
CommonVoice. Pooneh has already experience with that.
On Thu, Jun 22, 2023 at 3:37 PM Adel Moumen @.***>
wrote:

You should run the experiment starting from the latest dev branch.
Hey @mravanelli https://github.com/mravanelli, what do you think about
training smaller Whisper models for each languages? On the hub we have
several Whisper large-v2 models, but they consumes a lot of vram and they
are slow at inference time. I propose to train (and release on the HF hub)
Whisper models such as the base (74M) or the small (244M) instead of the
large-v2 (1550M). Furthermore, if we have time, it would be great to
replicate a little bit what @bofenghuang https://github.com/bofenghuang
did in his blog post (see:
@.***/what-i-learned-from-whisper-fine-tuning-event-2a68dab1862)
in terms of hyper parameters (I believe we should be able to improve the
WER with this setting).
—
Reply to this email directly, view it on GitHub
https://github.com/speechbrain/speechbrain/issues/2039#issuecomment-1603211853,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEA2ZVUGPUDGO4CLY7ZEQ7TXMSNGXANCNFSM6AAAAAAZNLP2KY
.
You are receiving this because you were mentioned.Message ID:
@.***>

I will do that but currently, as @Adel-Moumen we have several Whisper large-v2 models, but they consume a lot of vram and they are slow at inference time. Should we train medium or base  Whisper models instead of the large one so it could be more applicable for the release on the HF hub?

Yes, small models are fine.
On Thu, Jun 22, 2023 at 4:03 PM Pooneh Mousavi @.***>
wrote:

I will do that but currently, as @Adel-Moumen
https://github.com/Adel-Moumen we have several Whisper large-v2 models,
but they consume a lot of vram and they are slow at inference time. Should
we train medium or base Whisper models instead of the large one so it could
be more applicable for the release on the HF hub?
—
Reply to this email directly, view it on GitHub
https://github.com/speechbrain/speechbrain/issues/2039#issuecomment-1603244541,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEA2ZVT5G66IRD5PPJU7ML3XMSQKLANCNFSM6AAAAAAZNLP2KY
.
You are receiving this because you were mentioned.Message ID:
@.***>


Event metadata can get out-of-sync when using this sample code for event hubs trigger

In this repository there are some samples on how to fetch metadata from messages: https://github.com/Azure/azure-functions-dotnet-worker/blob/main/samples/Extensions/EventHubs/EventHubsTriggerMetadata.cs.
The code referring too is this:
 [Function("EventHubsTriggerMetadata-Parameters")]
        public static void UsingParameters([EventHubTrigger("src-parameters", Connection = "EventHubConnectionAppSetting")] string[] messages,
            DateTime[] enqueuedTimeUtcArray,
            long[] sequenceNumberArray,
            string[] offsetArray,
            Dictionary<string, JsonElement>[] propertiesArray,
            Dictionary<string, JsonElement>[] systemPropertiesArray)
        {
            // You can directly access binding data via parameters.
            for (int i = 0; i < messages.Length; i++)
            {
                string message = messages[i];
                DateTime enqueuedTimeUtc = enqueuedTimeUtcArray[i];
                long sequenceNumber = sequenceNumberArray[i];
                string offset = offsetArray[i];

                // Note: The values in these dictionaries are sent to the worker as JSON. By default, System.Text.Json will not automatically infer primitive values
                //       if you attempt to deserialize to 'object'. See https://docs.microsoft.com/en-us/dotnet/standard/serialization/system-text-json-migrate-from-newtonsoft-how-to?pivots=dotnet-5-0#deserialization-of-object-properties
                //
                //       If you want to use Dictionary<string, object> and have the values automatically inferred, you can use the sample JsonConverter specified
                //       here: https://docs.microsoft.com/en-us/dotnet/standard/serialization/system-text-json-converters-how-to?pivots=dotnet-5-0#deserialize-inferred-types-to-object-properties
                //
                //       See the Configuration sample in this repo for details on how to add this custom converter to the JsonSerializerOptions, or for
                //       details on how to use Newtonsoft.Json, which does automatically infer primitive values.
                Dictionary<string, JsonElement> properties = propertiesArray[i];
                Dictionary<string, JsonElement> systemProperties = systemPropertiesArray[i];
            }
        }

This code can however create bugs in software. If you have a message without a body there is no corresponding item in the string[] messages array but there are properties in the Dictionary<string, JsonElement>[] propertiesArray. If you parse these in batch the indexes between the different arrays are not equals which leads to wrong properties being used with wrong messages.
Let's take this example:
We have two messages coming in with the same batch. One (1) with a property but without body (yes, we have those) and another (2) where the body is filled. These are coming in, in the following way:
messages = int[1]{ messageBodyFrom2 }
propertiesArray = int [2] { propertiesFrom1, propertiesFrom2 }
How can we correlate the correct properties to the messagebody?
Is it not better that all arrays are always having the equal size? Something like this:
messages = int[2]{ nullValueAsMessageBodyFrom1, messageBodyFrom2 }
propertiesArray = int [2] { propertiesFrom1, propertiesFrom2 }

Assigning this to 123 to update the sample while we investigate the issue with the trigger

Hi @fabiocav I'm a colleague of @wesselkranenborg is and know the issue at hand. We are using the following packages for our eventhub triggers:
    <PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.EventHubs" Version="5.0.0-beta.6" />     <PackageReference Include="Microsoft.Azure.Functions.Worker.Sdk" Version="1.3.0" OutputItemType="Analyzer" />     <PackageReference Include="Microsoft.Azure.Functions.Worker" Version="1.6.0" />

Hi @fabiocav I'm a colleague of @wesselkranenborg. Know the issue at hand pretty well,  I can also answer the question about the used packages and their version:
    <PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.EventHubs" Version="5.0.0-beta.6" />     <PackageReference Include="Microsoft.Azure.Functions.Worker.Sdk" Version="1.3.0" OutputItemType="Analyzer" />     <PackageReference Include="Microsoft.Azure.Functions.Worker" Version="1.6.0" />

@fabiocav I was a few days on leave but my colleague already responded. Am curious about the issue with the trigger. Do you have an issue Id from that or is that tracked with this issue?
Thanks @Privarozenbeek.

Thank you @wesselkranenborg  @Privarozenbeek
Update from the investigation so far:

For out-of-process functions, The GRPC payload received in the worker from the host is missing the message entry which has null/empty string as value.  The meta properties for the missing entry is still present in the TriggerMetaData property of the InvocationRequest instance received.
We get the correct data in the host from the event hub listener/processor module and in the host code, when we convert the event data object to RPC invocation request instance, we are skipping the empty entry in the ToRpcStringArray method.

The messages are passed in the InputData property of the InvocationRequest instance while the properties are passed inside TriggerMetadata property.

This issue does not happen in the in-proc function app where an array of Azure.Messaging.EventHubs.EventData type is used as function method parameter.

This might be an issue in all out-of-process workers.

I will follow up with the team to discuss the right fix & impact of it and share an update as we make progress.

Initial investigation is done and root cause identified. The fix requires a change in the host. The below issue will track the progress of that.
Introduce new app setting to control skipping null entries from event payload when sending to OOP layer

The host level fix is on v4.12.2. Waiting for that release via core tools.


Error: Could not get code signature for running application (Catalina)

I just updated macOS to Catalina (10.15.4) and SoundCleod 1.4.0 would freeze when I launched it, so I tried SoundCleod 1.4.1-pre.0 before filing a bug report.
Now SoundCleod (1.4.1-pre.0) works, but throws the following error every time I launch it:
A JavaScript error occurred in the main process

Uncaught Exception:
Error: Could not get code signature for running application
    at startAutoUpdater (/Applications/SoundCleod.app/Contents/Resources/app.asar/auto-updater.js:26:15)
    at checkAutoUpdater (/Applications/SoundCleod.app/Contents/Resources/app.asar/auto-updater.js:8:26)
    at fs.access (/Applications/SoundCleod.app/Contents/Resources/app.asar/auto-updater.js:18:7)
    at FSReqWrap.oncomplete (fs.js:169:20)

Hey, thanks for writing in! I am working on some updates including fixing SoundCleod on Catalina. Expect a new version in the next couple of weeks!

New version is out, should fix all Catalina issues.


Build failure

Bug Report
Madara version:
62609f6808c882f8fe2301af2f4a14e56e06ebfb
Current behavior:
Build (from scratch fails)
Steps to reproduce:
clone
cargo run --release -- --dev
Other information:
The error is:
❯ cargo run --release -- --dev
    Updating git repository `https://github.com/xJonathanLEI/starknet-rs`
error: failed to get `starknet-crypto` as a dependency of package `mp-starknet v0.1.0-alpha (/home/ittay/workspace/madara/crates/primitives/starknet)`

Caused by:
  failed to load source for dependency `starknet-crypto`

Caused by:
  Unable to update https://github.com/xJonathanLEI/starknet-rs?branch=dev/jsonrpc_0_3_0#403f02e8

Caused by:
  object not found - no match for id (403f02e83fb3fd9f87654aae1f34c94ad480e9ef); class=Odb (9); code=NotFound (-3)

hey @ittayd thanks for submitting. Don't know if it can help but can you please share information about your machine (OS, etc)

can I try to fix this one @abdelhamidbakhta ? In any case it's mandatory for my other issue #427

can I try to fix this one @abdelhamidbakhta ? In any case it's mandatory for my other issue #427

yes please


signature verification failed

Bug: signature verification failed
Error Code: 110775B5D101FB36BC6C911BEB774491D9FF06E2
OS: Kali Linux WSL 2 / Windows 2004
Already tried to refresh keys etc.

I've got exactly the same error, but on Arch Linux

Same on Fedora

Related to #481

Same on Ubuntu 18.04

I get the same error code on Ubuntu 20.04

Same error happening on OpenSUSE Tumbleweed.

On Debian 10 buster the same problem

Same issue on Manjaro.

SIGNATURE VARIFICATION FAILED
Error Code: 110775B5D101FB36BC6C911BEB774491D9FF06E2 : Key expired
You might be under attack, there might be a network problem, or you may be missing a recently added Tor Browser verification key.
A copy of the Tor Browser files you downloaded have been saved here:
[ path where the file is saved tar.xz.verification_failed and tar.xz.asc.verification_failed]
Click START to refresh the keyring and try again
But even after trying again the same

Can we please stop bumping this? Yes. We are all getting the same issue. Use the 👍 reaction.

If you're unwilling to wait you can update tor-browser-developers.asc manually.

bump

This fixed the issue for me https://github.com/micahflee/torbrowser-launcher/issues/481#issuecomment-681585974

SIGNATURE VARIFICATION FAILED
Error Code: 110775B5D101FB36BC6C911BEB774491D9FF06E2 : Key expired
You might be under attack, there might be a network problem, or you may be missing a recently added Tor Browser verification key.
A copy of the Tor Browser files you downloaded have been saved here:
/home/user/.var/app/com.github.micahflee.torbrowser-launcher/cache/torbrowser/download/tor-browser-linux64-9.5.4_ru.tar.xz failed
/home/user/.var/app/com.github.micahflee.torbrowser-launcher/cache/torbrowser/download/tor-browser-linux64-9.5.4_ru.tar.xz.verification_failed failed
Click START to refresh the keyring and try again
without any success  after restarting

On Debian bullseye the same problem

same issue on debian 10 buster and pop os 20.04

Grab Key. Put it into Tor-Browser Launcher's folder.
wget -v 'https://github.com/sysrqb/torbrowser-launcher/raw/develop/share/torbrowser-launcher/tor-browser-developers.asc' ; sudo cp -v ~/tor-browser-developers.asc /usr/share/torbrowser-launcher/ ; torbrowser-launcher ; echo "DONE"
This will replace the Key(old/expired) of the same name in there. Everything works now. Don't forget to thumbs up. thx

Tanks, it's working

@micahflee Would it be possible to release a new torbrowser-launcher version with this issue fixed?

Grab Key. Put it into Tor-Browser Launcher's folder.
wget -v 'https://github.com/sysrqb/torbrowser-launcher/raw/develop/share/torbrowser-launcher/tor-browser-developers.asc' ; sudo cp -v ~/tor-browser-developers.asc /usr/share/torbrowser-launcher/ ; torbrowser-launcher ; echo "DONE"
This will replace the Key(old/expired) of the same name in there. Everything works now. Don't forget to thumbs up. thx

Tanks, it's working

worked fine for mint 19.3

Sorry, it isn't working on Ubuntu 20.04
Error message during installation:
"The version of Tor Browser you have installed is earlier than it should be, which could be a sign of an attack!"
Installation on hold

@anmarko That is a different issue, see #498.

For Ubuntu 18.04, the command
"~/.local/share/torbrowser/tbb/x86_64/tor-browser_en-US/Browser/start-tor-browser --verbose" execute/run the browser fine but I am trying to put it in the start-tor-browser.desktop file
sudo nano /home/raj/.local/share/torbrowser/tbb/x86_64/tor-browser_en-US/start-tor-browser.desktop
replacing "--detach" with "--verbose"
and then trying to start from the app icon but doesn't work :( .
It starts the tor launcher and try to update again though the version is 10.0
and also replace the "--verbose" with "--detach" by itself
and then shows
"The version of Tor Browser you have installed is earlier than it should be, which could be a sign of an attack!"
again.
:(

@paulrajib Works fine for me
This is my command
sh -c '"/home/nathanfranke/.local/share/torbrowser/tbb/x86_64/tor-browser_en-US/Browser/start-tor-browser" --verbose || ([ !  -x "/home/nathanfranke/.local/share/torbrowser/tbb/x86_64/tor-browser_en-US/Browser/start-tor-browser" ] && "$(dirname "$*")"/Browser/start-tor-browser --verbose)' dummy %k

(Copied from Exec in the desktop file)

Thanks  AsciiWolf !
I've done a new installation, it's really easy:
https://linuxconfig.org/how-to-install-tor-browser-in-ubuntu-18-04-bionic-beaver-linux
It works also for UBUNTU 20.04
Before deleting of the complete old TOR-directory I've only saved the bookmarks from the "bookmarks backup" folder

error code:110775B5D101FB36BC6C911BEB774491D9FF06E2: key expired

Помогите!error code:110775B5D101FB36BC6C911BEB774491D9FF06E2: key expired. Англиского не знаю,что делать?

Перезагружал, всё равно  error code:110775B5D101FB36BC6C911BEB774491D9FF06E2: key expired      o.s.linux64

Ответьте, пожалуйста,что мне надо сделать.

Been trying to install tor for about an hour now. Have tried all the public key refreshing suggestions and whatnot. Still getting the Key Expired failure. can some kind soul please help me.

└─$ ./start-tor-browser.desktop --register-app
zsh: segmentation fault  ./start-tor-browser.desktop --register-app


[Storage] Update Storage ARM template for built-in Files RBAC role

Same change as https://github.com/Azure/azure-sdk-for-net/pull/39018 but in Python SDK 😄

/azp run python - storage - tests


Code rearranged, curly braces added, typos fixed.

Just some code beautification, rearrangement of code, curly braces are placed at all appropriate locations and typos fixed.

@gautamabhishek46 This PR looks good to me.


Getting error on Linux Ubuntu

[ERROR] HTTPError 503: Service Temporarily Unavailable
[ERROR] Just a moment...
[ERROR] Body:
Full error:- https://bin.theindex.moe/?f228b7d8baccf8e4#Eu83onTahabRwiFby6C7YjJpXgbetXCKXNN3tTsyD4sA

@izu-co can you help sir?

It seems like the login page is being blocked by CloudFront. This will require more work, and I am currently pretty tied up.

Sir is there any quick fix @izu-co

None I can think of

How much it will take sir for the fix? @izu-co

In terms of time its hard to say without investigation. Since I have no time atm, maybe 5 weeks?

Can one of you (@thehenil or @KuroProjects) confirm whether this issue is Linux specific or if it also happens on Windows?

Sir, I tried on windows server it's giving same error @izu-co

Thank you

I'm going to close this issue since it doesn't seem to be a problem anymore. If it is however, feel free to re-open