Storybook broken out of the box in latest version

When trying to set up a SvelteKit project with the storybook option enabled, the project is broken out of the box.
npx sv --version
(node:303808) ExperimentalWarning: CommonJS module /usr/lib/node_modules/npm/node_modules/debug/src/node.js is loading ES Module /usr/lib/node_modules/npm/node_modules/supports-color/index.js using require().
Support for loading ES Module in require() is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
0.6.1

npx sv create test-app
(node:304086) ExperimentalWarning: CommonJS module /usr/lib/node_modules/npm/node_modules/debug/src/node.js is loading ES Module /usr/lib/node_modules/npm/node_modules/supports-color/index.js using require().
Support for loading ES Module in require() is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
┌  Welcome to the Svelte CLI! (v0.6.1)
│
◇  Which template would you like?
│  SvelteKit minimal
│
◇  Add type checking with Typescript?
│  Yes, using Typescript syntax
│
◆  Project created
│
◇  What would you like to add to your project? (use arrow keys / space bar)
│  storybook
│
◇  Which package manager do you want to install dependencies with?
│  npm
│
◇  Running external command (npx storybook@latest init --skip-install --no-dev)
(node:304169) ExperimentalWarning: CommonJS module /usr/lib/node_modules/npm/node_modules/debug/src/node.js is loading ES Module /usr/lib/node_modules/npm/node_modules/supports-color/index.js using require().
Support for loading ES Module in require() is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
(node:304214) ExperimentalWarning: CommonJS module /usr/lib/node_modules/npm/node_modules/debug/src/node.js is loading ES Module /usr/lib/node_modules/npm/node_modules/supports-color/index.js using require().
Support for loading ES Module in require() is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
╭──────────────────────────────────────────────────────╮
│                                                      │
│   Adding Storybook version 8.4.2 to your project..   │
│                                                      │
╰──────────────────────────────────────────────────────╯
 • Detecting project type. ✓
 • Adding Storybook support to your "SvelteKit" appWARN An issue occurred while trying to find dependencies metadata using npm.

  ✔ Getting the correct version of 9 packages
  ✔ Installing Storybook dependencies
WARN An issue occurred while trying to find dependencies metadata using npm.
. ✓

attention => Storybook now collects completely anonymous telemetry regarding usage.
This information is used to shape Storybook's roadmap and prioritize features.
You can learn more, including how to opt-out if you'd not like to participate in this anonymous program, by visiting the following URL:
https://storybook.js.org/telemetry

╭──────────────────────────────────────────────────────────────────────────────╮
│                                                                              │
│   Storybook was successfully installed in your project! 🎉                   │
│   To run Storybook manually, run npm run storybook. CTRL+C to stop.          │
│                                                                              │
│   Wanna know more about Storybook? Check out https://storybook.js.org/       │
│   Having trouble or want to chat? Join us at https://discord.gg/storybook/   │
│                                                                              │
╰──────────────────────────────────────────────────────────────────────────────╯
│
◆  Successfully setup add-ons
│
◇  Successfully installed dependencies
│
◇  Project next steps ─────────────────────────────────────────────────────╮
│                                                                          │
│  1: cd test-app                                                          │
│  2: git init && git add -A && git commit -m "Initial commit" (optional)  │
│  3: npm run dev -- --open                                                │
│                                                                          │
│  To close the dev server, hit Ctrl-C                                     │
│                                                                          │
│  Stuck? Visit us at https://svelte.dev/chat                              │
│                                                                          │
├──────────────────────────────────────────────────────────────────────────╯
│
└  You're all set!

cd test-app/

ls
node_modules  package.json  package-lock.json  README.md  src  static  svelte.config.js  tsconfig.json  vite.config.ts
npm run storybook

><EMAIL_ADDRESS>storybook
> storybook dev -p 6006

@storybook/core v8.4.2

SB_CORE-SERVER_0002 (CriticalPresetLoadError): Storybook failed to load the following preset: ./.storybook/main.js.

Please check whether your setup is correct, the Storybook dependencies (and their peer dependencies) are installed correctly and there are no package version clashes.

If you believe this is a bug, please open an issue on Github.

SB_CORE-SERVER_0002 (CriticalPresetLoadError): Storybook failed to load the following preset<EMAIL_ADDRESS>
Please check whether your setup is correct, the Storybook dependencies (and their peer dependencies) are installed correctly and there are no package version clashes.

If you believe this is a bug, please open an issue on Github.

Error [ERR_UNSUPPORTED_DIR_IMPORT]: Directory import './node_modules/@storybook/addon-svelte-csf/dist/compiler/post-transform/story' is not supported resolving ES modules imported from<EMAIL_ADDRESS>    at finalizeResolution (node:internal/modules/esm/resolve:263:11)
    at moduleResolve (node:internal/modules/esm/resolve:932:10)
    at defaultResolve (node:internal/modules/esm/resolve:1056:11)
    at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:650:12)
    at #cachedDefaultResolve (node:internal/modules/esm/loader:599:25)
    at ModuleLoader.getModuleJobForRequire (node:internal/modules/esm/loader:349:53)
    at new ModuleJobSync (node:internal/modules/esm/module_job:338:34)
    at ModuleLoader.getModuleJobForRequire (node:internal/modules/esm/loader:409:11)
    at new ModuleJobSync (node:internal/modules/esm/module_job:338:34)
    at ModuleLoader.getModuleJobForRequire (node:internal/modules/esm/loader:409:11)

More info:

    at loadPreset (./node_modules/@storybook/core/dist/common/index.cjs:16477:13)

More info:

    at loadPreset (./node_modules/@storybook/core/dist/common/index.cjs:16477:13)
    at async Promise.all (index 1)
    at async loadPresets (./node_modules/@storybook/core/dist/common/index.cjs:16487:55)
    at async getPresets (./node_modules/@storybook/core/dist/common/index.cjs:16520:11)
    at async buildDevStandalone (./node_modules/@storybook/core/dist/core-server/index.cjs:37145:11)
    at async withTelemetry (./node_modules/@storybook/core/dist/core-server/index.cjs:35757:12)
    at async dev (./node_modules/@storybook/core/dist/cli/bin/index.cjs:2591:3)
    at async s.<anonymous> (./node_modules/@storybook/core/dist/cli/bin/index.cjs:2643:74)

WARN Broken build, fix the error above.
WARN You may need to refresh the browser.

WARN   Failed to load preset<EMAIL_ADDRESS>on level 1
Error [ERR_UNSUPPORTED_DIR_IMPORT]: Directory import './node_modules/@storybook/addon-svelte-csf/dist/compiler/post-transform/story' is not supported resolving ES modules imported from<EMAIL_ADDRESS>    at finalizeResolution (node:internal/modules/esm/resolve:263:11)
    at moduleResolve (node:internal/modules/esm/resolve:932:10)
    at defaultResolve (node:internal/modules/esm/resolve:1056:11)
    at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:650:12)
    at #cachedDefaultResolve (node:internal/modules/esm/loader:599:25)
    at ModuleLoader.getModuleJobForRequire (node:internal/modules/esm/loader:349:53)
    at new ModuleJobSync (node:internal/modules/esm/module_job:338:34)
    at ModuleLoader.getModuleJobForRequire (node:internal/modules/esm/loader:409:11)
    at new ModuleJobSync (node:internal/modules/esm/module_job:338:34)
    at ModuleLoader.getModuleJobForRequire (node:internal/modules/esm/loader:409:11)
(node:305056) ExperimentalWarning: CommonJS module<EMAIL_ADDRESS>is loading ES Module<EMAIL_ADDRESS>using require().
Support for loading ES Module in require() is an experimental feature and might change at any time
(Use `node --trace-warnings ...` to show where the warning was created)
✔ Would you like to help improve Storybook by sending anonymous crash reports? … yes

Yeah, sadly the 8.4.x storybook versions are currently broken, i think we already pinned this in the repo, but did not release up until now.
https://github.com/sveltejs/cli/blob/0d3ecb7b25a085c342fafff2ccabb8ae69378123/packages/adders/storybook/index.ts#L12
We just released 0.6.2, which pins storybook to 8.3.6 which is the latest version that seems to be working.

Interestingly creating a new project with the latest 0.6.2 version does not seem to work still. If it works to you, then something about my env might be messed up, but I am unsure what that could be.


Combinator or_else and conditionals

In the process of trying to understand https://github.com/xd009642/tarpaulin/issues/1066, I believe combinators (i.e.: or_else) are not parsed as a conditional (i.e. ExprIf or similar).
Can somebody please confirm, how are they treated?

As far as I can tell, this is working as intended. or_else method calls are definitely not an ExprIf. That refers to use of the if keyword.

Hi @dtolnay,
Yes, it makes sense. Into which type are or_else lexed?


Unable to evaluate tracers while using a subdomain.

Hi,
I am currently attempting to use the latest version (18/12/2020) of TRACMASS with ORCA025 fields with dimensions (75 x 1021 x 1442) and a subdomain in the North Atlantic. Unfortunately, each time I enable tracers to be evaluated along trajectories I experience a Fortran runtime error in read_field.F90 - I have copied the output below for reference. I'm not sure whether this can be fixed directly in read_field.F90 (my previous attempts to modify line 179 directly crashed mod_loop.F90) and I would greatly appreciate some further advice on this issue.
Thanks in advance,
Ollie

Model information:
Project code  : ORCA025_Sim01
Case name     : ORCA025_Sim01
Namelist file : projects/ORCA025_Sim01/namelist_ORCA025_Sim01.in

Configuration options:

Forward in time (nff = 1)
A subdomain has been choosen
i index:   844 - 1122
j index:   673 - 856
Three-dimensional trajectories (vertical fluxes computed)
Evaluate the value of these tracers along the trajectories:
1- To (degC)
2- S (PSU)
3- sigma0 (kg/m3)

At line 179 of file projects/ORCA025_Sim01/read_field.F90
Fortran runtime error: Array bound mismatch for dimension 1 of array 'tracers' (1442/279)

Following my question above:
Having explored this issue further, I believe the problem may arise as the tracer arrays are defined in init_tracer before the subdomain is defined in init_subdomain. The mismatch would then occur since imt and jmt are redefined once the tracers are already allocated to the global grid size (imt = 1442) rather than the subdomain (imt = 279).

Thank you for raising the issue,
As you pointed out the tracers were initialised before the subdomain was defined. The order has been reversed in the latest commit (23/02/2021). Moreover, we have found more potential issues in the stream function calculation and in the initialisation of the trajectories if TRACMASS is initialised with a subdomain.
Do not hesitate to contact us if you find more problems.

Thanks for making these changes so quickly!
Much appreciated,
Ollie


Eliminate a superfluous List.reverse call in Random.list

Replaces https://github.com/elm-lang/core/pull/522, which became collateral damage in the branch killing spree.
Why remove that List.reverse call? Without it, generation is more efficient. And its presence or absence doesn't carry meaning for the semantics of randomness.
An argument has been made in the other discussion that with that call one has some sort of stability. But I have pointed out that it's a very fragile stability, and might even be harmful in terms of raising expectations, and in any case isn't compositional.
In this:
import Random
import Html

seed =
  Random.initialSeed 314159

triple n =
  let list = Random.list n (Random.int 0 100)
      gen = Random.map3 (,,) list Random.bool list
  in Random.step gen seed |> fst

main =
  Html.text <| toString (triple 2) ++ " " ++ toString (triple 5)

the outputs are ([2,22],True,[50,42]) vs. ([2,22,89,50,42],False,[47,79,82,79,6]). So only an appearance of stability.

Added to #724


Refactor Calculator.js to function component

Refactor to function component

Use the "useState" hook to add state to the component

Thankl you sir


Dockerhub images

This will push a built Ubuntu Bionic docker image with the K package installed on it to DockerHub under runtimeverificationinc/kframework-k:ubuntu-focal-COMMIT_ID.
The naming scheme is runtimeverificationinc/REPO_IDENTIFIER:DISTRO_IDENTIFIER-COMMIT_ID, where REPO_IDENTIFIER is the full GitHub path to the repo with / replaced by -.
This will make it so that repos which depend on a K submodule can just grab the corresponding Docker image by inspecting the submodule's K commit ID. That way, we get to:

Still test downstream repos (eg KEVM) with custom versions of K to be sure our changes to K will fix our KEVM problems.
Use a pre-built docker image with K installed on CI jobs to shave off 8-12 minutes every semantics job we have (the make deps steps, usually).

I have the changes to make it so that we also do Debian Buster and Arch docker images, but I don't think it's necessary. The primary target here is to make CI faster so adding different distros as examples seems lower priority.

Here is an example of it working: https://github.com/kframework/wasm-semantics/pull/325

One thing I'm not sure about is the tagging scheme. I like having the commit-id in the tag name so that we have the ability to peg to specific versions of K, and there is no limit on the number of tags in a given public Docker Hub repo.
But it may also be nice to provide a master tag for ease of use.

It may be easier to review this one with git diff origin/master origin/dockerhub-images -b

Having both specific and rolling tags is fairly normal for docker, so I +1 that idea.

@dwightguth this should also update the ubuntu-bionic-master tag each time master branch changes too.
I also publish the docker image before the github release, so that the release notes will point to a valid Docker Hub image.

It worked, ready for re-review @dwightguth https://office.runtimeverification.com/jenkins/blue/organizations/jenkins/k/detail/PR-1228/36/pipeline/

Evidence that it works: https://office.runtimeverification.com/jenkins/blue/organizations/jenkins/k/detail/PR-1228/49/pipeline/152
So now I am just running the docker commands directly on the LXC container, which at the very least is much simpler than all other solutions and means that the main CI docker container doesn't need docker or sudo privileges.
@dwightguth ready for re-review.


[gha] use latest version of actions

Motivation
Use the latest version of diem/actions, with new port/protocol for hyperjump
Have you read the [Contributing Guidelines on pull requests]
Yes
Test Plan
CI
Related PRs
None
If targeting a release branch, please fill the below out as well

Justification and breaking nature (who does it affect? validators, full nodes, tooling, operators, AOS, etc.)
Comprehensive test results that demonstrate the fix working and not breaking existing workflows.
Why we must have it for V1 launch.
What workarounds and alternative we have if we do not push the PR.

Looks good:  https://github.com/diem/diem/pull/8120/checks?check_run_id=2251088459

/land


Fixes minor_version for CE 3.20-2

Product Version(s):

Issue:

Link to docs preview:

SME review:

[ ] An SME has approved this change.

DOCS review:

[ ] A member of the docs team has approved this change.

Additional information:

Merge checklist:

[ ] Deploy preview inspected wherever changes were made
[ ] Build completed successfully
[ ] Test have passed

CC @rene-dekker


#2 Merging this if statement with the enclosing one

This will decrease the nesting of the code and improve its readability.
Thanks for your contribution.
PLEASE REMOVE
To support us in providing a nice (and fast) open-source experience:

Verify that the tests are passing
Check that the code is properly formatted (using AndroidStudio's autoformatter)
Provide write access to the branch
If the PR is not ready for review, please submit it as a draft

PLEASE REMOVE
Describe the pull request
A clear and concise description of what the pull request changes/adds.
Link to the the issue
(If available): The link to the issue that this pull request solves.
License agreement
By opening this pull request, you are providing your contribution under the Apache License 2.0 (see LICENSE.md).
Note: new dependencies/libraries
Please refrain from introducing new libraries without consulting the team.

Merging this if statement with the enclosing one - Changes Done

PR Merged for issue: https://github.com/OmniaAlam/OpenTracks-Winter-SOEN-6431_2024/issues/6#issue-2129210066


Add User Authentication to Your Application (SAP HANA Cloud)

Tutorials: https://developers.sap.com/tutorials/hana-cloud-cap-add-authentication.html
Hi,
when I executed the folling command to create the XSUAA services instance with the xs-security.json configuration, the following error is thrown:
Incorrect Usage: Invalid configuration provided for -c flag. Please provide a valid JSON object or path to a file containing a valid JSON object.
My xs-security.json has the following content:
{
"xsappname": "myhanaapp",
"tenant-mode": "dedicated",
"scopes": [
{
"name": "uaa-user",
"description": "UAA"
}
],
"attributes": [],
"role-templates": [
{
"name": "Token_Exchange",
"description": "UAA",
"scope-references": [
"uaa_user"
]
}
]
}
Thanks

First that isn't the correct content for this tutorial.  You are missing the oauth2-configuration section you were supposed to add in  task 3 of step 1.  You also have an empty attributes section which is likely the root of the error message you are receiving.   The complete content of the xs-security.json should be this:
{
  "xsappname": "myhanaapp",
  "tenant-mode": "dedicated",
  "description": "Security profile of called application",
  "scopes": [
    {
      "name": "uaa.user",
      "description": "UAA"
    }
  ],
  "role-templates": [
    {
      "name": "Token_Exchange",
      "description": "UAA",
      "scope-references": [
        "uaa.user"
      ]
    }
  ],
  "oauth2-configuration": {
    "redirect-uris": [
        "https://*.applicationstudio.cloud.sap/**"
    ]
}
}


[PE-D][Tester B] Phone number has too many digits

It seems like there is no restriction about phone number digits. In reality, there should be a restriction about this, such as the Singapore phone number is eight digits.

Labels: type.FunctionalityBug severity.Medium
original: QQH0828/ped#10

Is this really necessary?

Will Implement feature constraint such that phone number can only have up to 15 digits, as that is the longest possible mobile phone number.

Is this really necessary?

I think we could make the argument that since it is a phone number attribute and the maximum phone number lenght is 15 digits world-wide, it makes sense for an extremely long phone number to be a functionality bug. Will check if tutor/ prof Damith on forum

After reviewing the forum I think it is safe to say that this is a feature flaw, rather than a bug.
Suggested actions
Future features
Add the following to future features:

Will Implement feature constraint such that phone number can only have up to 15 digits, as that is the longest possible mobile phone number.

UG for add/edit command

Mention that phone numbers can be be infinitely long, which will be fixed in the future, and current users can use it as you see fit.

Possible Bug fix

[ ] Additionally, since phone number is stored as an Integer throw exception if number given is greater than max int


Replace Alert with Modal on FASET Form Submit

I really want a simple and elegant animation and then the form should auto-clear
Something like this:

I want the modal to open, show the message briefly with copy along the lines of "You're all set!" and then auto-close and clear the form for the next person.


Empty completionEntryDetails for auto import entry with misspelling

From: https://github.com/Microsoft/vscode/issues/46457
TypeScript Version: 2.9.0-dev-20180402
Search Terms:

completionEntryDetails
auto imports

Code
For two files in a typescript project
a.ts:
export const abc = 123;

b.ts:

In b.ts, type acb. This trigger suggestions. Go to the entry for abc
Bug:
Due to how vscode's suggestion fuzzy filtering works, we show a suggestion for abc  However the completionEntryDetails request in this case returns an empty response:
[Trace  - 13:39:48] Sending request: completionEntryDetails (236). Response expected: yes. Current queue length: 0
Arguments: {
    "file": "/Users/matb/projects/san/b.ts",
    "line": 1,
    "offset": 2,
    "entryNames": [
        {
            "name": "Abc",
            "source": "/Users/matb/projects/san/b"
        }
    ]
}
[Trace  - 13:39:48] Response received: completionEntryDetails (236). Request took 6 ms. Success: true
Result: []

Playground Link:
Related Issues:

This is because of the if (stringContainsCharactersInOrder... check in completions.ts -- we don't actually add the completion at acb, you must be using a cached version. We could remove that check, but then we'd be returning all completions from all external modules unconditionally and that might be slow. @mhegazy thoughts?

@mjbvz
so there is nothing you can do with that?
I just don't understand, why do we need an auto-import "feature" that does NOT insert auto-import paths?
Or to be more accurate, does it always from SECOND try. Sp, you type name of a module or directive.... find the auto-import snippet, choose it - it didn't work, delete 1 char retype - choose snippet again and it works.
For my opinion, this ridiculous crippled "feature" should be put to rights, or removed at all.

@andy-ms   @mjbvz
not sure, but seems like the same case:


syncRouteParams: false results in an error in plugin.routing when using switchLocalePath

Version
v6.7.0
Reproduction link
https://not-needed.com
Steps to reproduce
Add the following config to vuex:
        vuex: {
          syncRouteParams: false,
        },

Reload the server / page
What is expected ?
App should load and work
What is actually happening?
App fails to load with an error coming from plugin.routing.js - store.getters[${vuex.moduleName}/localeRouteParams](locale) is not a function.
Additional comments?
If syncRouteParams option is false, then the localeRouteParams getter within plugin.main.js does not get registered. However, switchLocalePath function within plugin.routing.js depends on localeRouteParams getter being present (if vuex and store are present, which both holds true with the configuration listed above)

This bug report is available on Nuxt community (#c429)

Thanks. Fixing in #645.


ci: use original release please action

Checklist

[ ] make test passes
[ ] tests and/or benchmarks are included
[ ] documentation is changed or added

Affected core subsystem(s)

Description of change

Just a question, is that normal that we want to use codecov v4 in beta ?

I don't understand . This PR doesn't change codecov


Passing a Function into a Java Class

When you pass a Function into a normal Java Class you don't get the V8ValueFunction you get the Function Entity but that can't be turned into a V8ValueFunction.
This is quite an Annoying Issue because I have to turn all my Java Classes into V8ValueObjects with import statements and make work-arounds for the V8ValueFunction.

That was a decision I made when designing proxy based injection. I thought proxy based injection should only be applied to regular Java objects that don't know Javet at all. In your case, if your Java objects know Javet, please use the standard way to avoid double conversions.

That was a decision I made when designing proxy based injection. I thought proxy based injection should only be applied to regular Java objects that don't know Javet at all. In your case, if your Java objects know Javet, please use the standard way to avoid double conversions.

how about add like a options system into the proxy so we can choose for our application

how about add like a options system into the proxy so we can choose for our application

I'll find a simple solution.

any solution for this?

any solution for this?

It is possible, but requires considerable coding effort and careful design. I'll spend sometime on it later.

This is enabled by this commit. You are welcome taking a try.


fix: Controller buttons use material renderQueue instead of shader renderQueue

With URP, since the usingMaterial is Universal Render Pipelin/Unlit, the shader renderQueue will have a wrong rendering order. Therefore, the material renderQueue is used.
after the fix

before the fix

merged with #889

merged with #889


How to specify different output names for audio and video files (without including format/codec)?

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
For simplicity's sake, let's say I'm using -f ba,bv to get separate audio and video files. I don't care what the format/codec of either actually is, but I want the audio file to have a .a extension and the video file to have .v
What should my -o parameter look like? I've read the section about replacement values.
I've tried:
-o '%(id)s.%(hasaud&a|v)s' -> This has both files ending in .v
-o '%(id)s.%(hasvid&v|a)s' -> This has both files ending in .a
-o '%(id)s.%(vocodec&v|a)s' -> This has both files ending in .v (Why? The audio file vcodec should have "empty" value per the docs so the second replacement should be applied)
-o '%(id)s.%(vocodec=none&a|v)s' -> This has both files ending in .NA (This absolutely defeats me, because I can't make any sense whatsoever of how this could be the output. "a" and "v" should be the only possibilities.)
-o '%(id)s.%(vcodec=NA&a|v)s' -> Same as above.
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

--parse-metadata 'video:%(acodec)s:^(?P<has_audio>(?!none).+)$'
--parse-metadata 'video:%(vcodec)s:^(?P<has_video>(?!none).+)$'
-o '%(id)s.%(has_video&v|)s%(has_audio&a|)s'

Okay, thanks. But I'd really like to understand what I'm doing, and especially what I'm doing wrong, so why doesn't -o '%(id)s.%(vcodec=none&a|v)s' work? And especially, how does the syntax allow NA to be the output?

vcodec=none is not valid in an output template. The = operator is not supported at all, and it is breaking the parsing.
--parse-metadata 'video:acodec:^(?P<has_audio>(?!none).+)$'
--parse-metadata 'video:vcodec:^(?P<has_video>(?!none).+)$'

Here you are parsing values for custom fields has_audio and has_video by using regex to match anything other than 'none'. An audio-only format will have a vcodec value of 'none' (string) and a video-only format will have an acodec value of 'none' (string)
-o '%(id)s.%(has_video&v|)s%(has_audio&a|)s'

Then you're putting the new custom fields to use in the output template... if has_video then 'v' else '' etc

I see now. The tripmine for me was that I assumed the first part of a replacement can be an expression, but it only takes a plain field name.
I was able to figure out the --parse-metadata after some headscratching (although the ?P<> syntax isn't really specified anywhere, it's only given in examples.)
Thank you for the help.


fix(mongoose): Fix issue when a mongoose model self reference on his …

…props
Information

Type
Breaking change

Feature/Fix/Doc/Chore
Yes/No

Description
A few sentences describing the overall goals of the pull request's commits.
Usage example
Example to use your feature and to improve the documentation after merging your PR:
import {} from "@tsed/common";

Todos

[ ] Tests
[ ] Coverage
[ ] Example
[ ] Documentation

:tada: This PR is included in version 6.26.3 :tada:
The release is available on:

v6.26.3
GitHub release

Your semantic-release bot :package::rocket:


eval() function is not compatible with hermes in iOS

Description
I have hermes enabled in my iOS project and trying to execute following code snippet,
 let testCall = () => this.testCall();
 let funct = '{() => testCall()}';   // function in string format
 let finalFunc = eval(funct);

Now, If I execute the finalFunc() then am getting following error,

But if I  I execute the finalFunc() when hermes is disabled then testCall() function is called successfully.
Note: In Android there is no issue no matter hermes is enabled or disabled.
Version
0.67
Output of npx react-native info
System:
     OS: macOS 11.6
     CPU: (8) x64 Apple M1
     Memory: 40.87 MB / 8.00 GB
     Shell: 5.8 - /bin/zsh
Binaries:
    Node: 12.22.1 - /usr/local/bin/node
    Yarn: Not Found
    npm: 6.14.12 - /usr/local/bin/npm
   Watchman: Not Found
Managers:
       CocoaPods: 1.11.2 - /opt/homebrew/bin/pod
 SDKs:
      iOS SDK:
      Platforms: DriverKit 21.2, iOS 15.2, macOS 12.1, tvOS 15.2, watchOS 8.3
      Android SDK: Not Found
IDEs:
      Android Studio: 4.2 AI-202.76<IP_ADDRESS>51085
      Xcode: 13.2.1/13C100 - /usr/bin/xcodebuild
Languages:
      Java: 11.0.10 - /usr/bin/javac
npmPackages:
      @react-native-community/cli: Not Found
      react: 17.0.2 => 17.0.2
      react-native: 0.67.0 => 0.67.0
      react-native-macos: Not Found
      npmGlobalPackages:
      *react-native*: Not Found

Steps to reproduce

Kindly clone the reproducible demo

2.Once the project is ran you will be able to see a text called 'Button'
3.Just tap on the button then you will face the error which is in above screen shot
Snack, code example, screenshot, or link to a repository
Please find the reproducible demo here - -https://github.com/Yandamuri/Test-Hermes-In-IOS

As per this comment - https://github.com/facebook/react-native/issues/27116#issuecomment-555933140, eval() should be avoided if hermes is enabled.
Can some one justify this?


Recommended Gdstime fan is not really better

Comparing the specs between default heatsink fan and Gdstime, it's evident that the default one is better performer and quieter. So I don't really get why the other one i recommended.
Ruiapple RA-DC3010 24M
Current : 0.08A
Speed: 10000RPM
AirPressure: 4.2 CFM
Noise: 32dB
Gdstime GDA3010-5F 24V
Current : 0.06A
Speed: 12000RPM
AirFlow: 4.1 CFM
Noise: 35dB

Myself and @stew675 checked the model numbers of our stock fans which were RA-DC3010 M, which is only rated for 3.8CFM. I could be Qidi started shipping the more powerful version RA-DC3010 H which is on par with the GSDtime 3010. If you have a different model of fan stock on your printer, let us know.

https://www.fanscooling.com/dc-axial-fans/ra-dc3010.html

Granted, but I would always look at the official vendor specifications when comparing products:
https://www.ruiapple.cn/Products/DC-Axial-Fans/450.html
https://www.ruiapple.cn/uploads/soft/210810/3010.pdf
Yes, it's still the same model, but the specs on their page are a little different.

I don't have an accurate way to measure fan CFM, but the GSDtime fans push a lot more air than the stock fan even though the CFM ratings are close on paper. You can tell just by putting your hand on the other side of the hotend when the GSDtime fan is installed. It wouldn't be the first time specs on components are 'inflated'.
Either way, it seems the stock fan can keep up with cooling the cold side of the hotend with the newer ceramic heatbreak. So changing out the fan isn't as critical as it was for us on the initial units.
GSDtime isn't an A-tier supplier of fans, Sunon or Delta are better, but don't make 24V fans in 3010 size. So the GSDtime recommendation will stick around.


Exception while bouncing banned IP (not output captcha/ban page)

Hello all! :wave:
First of all, I would like to say that Crowdsec is a powerful thing, and this WordPress plugin is a total bomb! :zap:
But, I recently noticed a possible problem with this plugin, in a test, and then in a production environment with plugin version - 1.11.0. The captcha/ban page is not displayed, for example, if add your IP address to Crowdsec decisions manually or use a banned IP address by inserting it into the "Forced test IP" field in "Advanced Settings". It simply passes the banned IP further to the site, instead of displaying the captcha/ban page.
In the logs (prod.log) there is an exception message (captcha):
2023-02-06T16:17:18.152008+00:00|300|{"type":"NON_AUTHORIZED_X_FORWARDED_FOR_USAGE","original_ip":"XXX.XXX.XXX.XXX","x_forwarded_for_ip":"type not as expected"}
2023-02-06T16:17:18.167364+00:00|300|{"type":"BAD_VALUE","value":" XXX.XXX.XXX.XXX","scope":"Ip","remediation":"ban","cache":"miss"}
2023-02-06T16:17:18.167583+00:00|200|{"type":"FINAL_REMEDIATION","ip":" XXX.XXX.XXX.XXX","remediation":"ban"}
2023-02-06T16:17:18.169490+00:00|400|{"type":"WP_EXCEPTION_WHILE_BOUNCING","message":"An exception has been thrown during the compilation of a template (\"include(/var/www/<site_dir>/wp-content/plugins/crowdsec/vendor/composer/../twig/twig/src/TwigTest.php): failed to open stream: No such file or directory (Error level: 2)\") in \"captcha.html.twig\".","code":0,"file":"/var/www/<site_dir>/wp-content/plugins/crowdsec/vendor/twig/twig/src/Environment.php","line":521}

And the same for the page with a ban (here briefly, only an exception, everything repeats):
2023-02-06T16:27:41.991953+00:00|400|{"type":"WP_EXCEPTION_WHILE_BOUNCING","message":"An exception has been thrown during the compilation of a template (\"include(/var/www/<site_dir>/wp-content/plugins/crowdsec/vendor/composer/../twig/twig/src/TwigTest.php): failed to open stream: No such file or directory (Error level: 2)\") in \"ban.html.twig\".","code":0,"file":"/var/www/<site_dir>/wp-content/plugins/crowdsec/vendor/twig/twig/src/Environment.php","line":521}

debug.log:
2023-02-08T16:45:29.912247+00:00|100|{"type":"REST_CLIENT_INIT","base_uri":"http://<IP_ADDRESS>:8080","timeout":120,"user_agent":"WordPress CrowdSec Bouncer/v1.11.0"}
2023-02-08T16:45:29.912563+00:00|100|{"type":"API_CLIENT_INIT","user_agent":"WordPress CrowdSec Bouncer/v1.11.0","rest_client":"CrowdSecBouncer\\RestClient\\Curl"}
2023-02-08T16:45:29.913331+00:00|100|{"type":"API_CACHE_INIT","adapter":"Symfony\\Component\\Cache\\Adapter\\TagAwareAdapter","mode":"live","fallback_remediation":"captcha","exp_clean_ips":5,"exp_bad_ips":20,"exp_captcha_flow":86400,"exp_geolocation_result":86400,"warmed_up":"false","geolocation":{"enabled":false,"type":"maxmind","save_result":false,"maxmind":{"database_type":"country","database_path":"/var/www/<site_dir>/wp-content/plugins/crowdsec/inc/../geolocation/"}}}
2023-02-08T16:45:29.913362+00:00|100|{"type":"BOUNCER_INIT","logger":"Monolog\\Logger","max_remediation_level":1,"configs":{"api_key":"***","auth_type":"api_key","tls_cert_path":"/var/www/<site_dir>/wp-content/plugins/crowdsec/inc/../tls/","tls_key_path":"/var/www/<site_dir>/wp-content/plugins/crowdsec/inc/../tls/","tls_verify_peer":false,"tls_ca_cert_path":"/var/www/<site_dir>/wp-content/plugins/crowdsec/inc/../tls/","api_url":"http://<IP_ADDRESS>:8080","use_curl":true,"api_user_agent":"WordPress CrowdSec Bouncer/v1.11.0","api_timeout":120,"debug_mode":true,"disable_prod_log":false,"log_directory_path":"/var/www/<site_dir>/wp-content/plugins/crowdsec/inc/../logs/","forced_test_ip":"XXX.XXX.XXX.XXX","forced_test_forwarded_ip":"","display_errors":false,"bouncing_level":"flex_bouncing","trust_ip_forward_array":[],"fallback_remediation":"captcha","stream_mode":false,"cache_system":"phpfs","fs_cache_path":"/var/www/<site_dir>/wp-content/plugins/crowdsec/inc/../.cache","redis_dsn":"","memcached_dsn":"","clean_ip_cache_duration":5,"bad_ip_cache_duration":20,"captcha_cache_duration":86400,"geolocation_cache_duration":86400,"geolocation":{"enabled":false,"type":"maxmind","save_result":false,"maxmind":{"database_type":"country","database_path":"/var/www/<site_dir>/wp-content/plugins/crowdsec/inc/../geolocation/"}},"max_remediation_level":"captcha","excluded_uris":[]}}
2023-02-08T16:45:29.913409+00:00|300|{"type":"NON_AUTHORIZED_X_FORWARDED_FOR_USAGE","original_ip":"XXX.XXX.XXX.XXX","x_forwarded_for_ip":"type not as expected"}
2023-02-08T16:45:29.913651+00:00|100|{"type":"START_IP_CHECK","ip":"XXX.XXX.XXX.XXX"}
2023-02-08T16:45:29.913799+00:00|300|{"type":"BAD_VALUE","value":"XXX.XXX.XXX.XXX","scope":"Ip","remediation":"ban","cache":"hit"}
2023-02-08T16:45:29.913907+00:00|200|{"type":"FINAL_REMEDIATION","ip":"XXX.XXX.XXX.XXX","remediation":"ban"}
2023-02-08T16:45:29.914806+00:00|400|{"type":"WP_EXCEPTION_WHILE_BOUNCING","message":"An exception has been thrown during the compilation of a template (\"include(/var/www/<site_dir>/wp-content/plugins/crowdsec/vendor/composer/../twig/twig/src/TwigTest.php): failed to open stream: No such file or directory (Error level: 2)\") in \"captcha.html.twig\".","code":0,"file":"/var/www/<site_dir>/wp-content/plugins/crowdsec/vendor/twig/twig/src/Environment.php","line":521}

If enable WP_DEBUG and "Enable errors display":
Fatal error: Uncaught CrowdSecBouncer\BouncerException: include(/var/www/<site_dir>/wp-content/plugins/crowdsec/vendor/composer/../twig/twig/src/TwigTest.php): failed to open stream: No such file or directory (Error level: 2) in /var/www/<site_dir>/wp-content/plugins/crowdsec/inc/Bounce.php:319
Stack trace:
#0 /var/www/<site_dir>/wp-content/plugins/all-in-one-seo-pack/vendor/composer/ClassLoader.php(571): Bounce->{closure}()
#1 /var/www/<site_dir>/wp-content/plugins/all-in-one-seo-pack/vendor/composer/ClassLoader.php(571): include()
#2 /var/www/<site_dir>/wp-content/plugins/all-in-one-seo-pack/vendor/composer/ClassLoader.php(428): Composer\Autoload\includeFile()
#3 [internal function]: Composer\Autoload\ClassLoader->loadClass()
#4 /var/www/<site_dir>/wp-content/plugins/crowdsec/vendor/twig/twig/src/Extension/CoreExtension.php(243): spl_autoload_call()
#5 /var/www/<site_dir>/wp-content/plugins/crowdsec/vendor/twig/twig/src/ExtensionSet.php(431): Twig\Extension\CoreExtension->getTests()
#6 /var/www/<site_dir>/wp-con
in /var/www/<site_dir>/wp-content/plugins/crowdsec/vendor/twig/twig/src/Environment.php on line 521

This exception appears in versions 1.10.0 and 1.11.0. In the test environment, deactivation of all other plugins was tried, and for version 1.11.0, a test was made in a clean installation of WordPress. The result is the same.
This TwigTest.php file is missing from the plugin directory.
Version 1.9.0 displays pages with captcha/ban when banned or manually "Forced test IP" without any problems.
Also, I have already tried plugin version 2.0.0 on a clean installation of WordPress. The problem still remains. But, in the logs there seems to be no information about the error (debug.log):
2023-02-09T15:59:08.230416+00:00|100|Instantiate client|{"type":"CLIENT_INIT","configs":{"api_key":"***","auth_type":"api_key","tls_cert_path":"/var/www/<site_dir>/wp-content/plugins/crowdsec/inc/../tls/","tls_key_path":"/var/www/<site_dir>/wp-content/plugins/crowdsec/inc/../tls/","tls_verify_peer":false,"tls_ca_cert_path":"/var/www/<site_dir>/wp-content/plugins/crowdsec/inc/../tls/","api_url":"http://<IP_ADDRESS>:8080","api_timeout":120,"user_agent_version":"v2.0.0","user_agent_suffix":"WordPress"}}
2023-02-09T15:59:08.231299+00:00|100|Instantiate cache|{"type":"CACHE_INIT","configs":{"fs_cache_path":"/var/www/<site_dir>/wp-content/plugins/crowdsec/inc/../.cache"},"adapter":"Symfony\\Component\\Cache\\Adapter\\TagAwareAdapter"}
2023-02-09T15:59:08.231581+00:00|100|Instantiate remediation engine|{"type":"REM_INIT","configs":{"fallback_remediation":"captcha","stream_mode":false,"clean_ip_cache_duration":60,"bad_ip_cache_duration":120,"geolocation":{"enabled":false,"type":"maxmind","cache_duration":86400,"maxmind":{"database_type":"country","database_path":"/var/www/<site_dir>/wp-content/plugins/crowdsec/inc/../geolocation/"}},"ordered_remediations":["ban","captcha","bypass"]},"cache":"CrowdSec\\RemediationEngine\\CacheStorage\\PhpFiles"}
2023-02-09T15:59:08.231894+00:00|100|Instantiate bouncer|{"type":"BOUNCER_INIT","logger":"CrowdSec\\Common\\Logger\\FileLog","remediation":"CrowdSec\\RemediationEngine\\LapiRemediation","configs":{"use_curl":true,"debug_mode":true,"disable_prod_log":false,"log_directory_path":"/var/www/<site_dir>/wp-content/plugins/crowdsec/inc/../logs/","forced_test_ip":"XXX.XXX.XXX.XXX","forced_test_forwarded_ip":"","display_errors":true,"bouncing_level":"normal_bouncing","trust_ip_forward_array":[],"cache_system":"phpfs","captcha_cache_duration":86400,"hide_mentions":false,"custom_css":"","excluded_uris":[]}}
2023-02-09T15:59:08.232324+00:00|100|Cache result|{"type":"LAPI_REM_CACHED_DECISIONS","ip":"XXX.XXX.XXX.XXX","result":"miss"}
2023-02-09T15:59:08.232351+00:00|100|Now processing a bouncer request|{"type":"BOUNCER_CLIENT_REQUEST","method":"GET","endpoint":"/v1/decisions","parameters":{"ip":"XXX.XXX.XXX.XXX"}}
2023-02-09T15:59:08.239015+00:00|100|Decisions have been sorted by priority|{"type":"REM_SORTED_DECISIONS","decisions":[{"0":"ban","1":1675958468,"2":"CAPI-ban-ip-XXX.XXX.XXX.XXX","priority":0}]}

If enabled, "Enable errors display" (without WP_DEBUG), then almost immediately a letter arrives with a plugin exception from WordPress:
An error with type E_ERROR occurred on line 35 of the file /var/www/<site_dir>/wp-content/plugins/crowdsec/inc/bounce-current-ip.php.
Error message: Uncaught Error: Class 'Twig\TwigTest' not found in /var/www/<site_dir>/wp-content/plugins/crowdsec/vendor/twig/twig/src/Extension/CoreExtension.php:245
Stack trace:
#0 /var/www/<site_dir>/wp-content/plugins/crowdsec/vendor/twig/twig/src/ExtensionSet.php(448): Twig\Extension\CoreExtension->getTests()
#1 /var/www/<site_dir>/wp-content/plugins/crowdsec/vendor/twig/twig/src/ExtensionSet.php(428): Twig\ExtensionSet->initExtension()
#2 /var/www/<site_dir>/wp-content/plugins/crowdsec/vendor/twig/twig/src/ExtensionSet.php(399): Twig\ExtensionSet->initExtensions()
#3 /var/www/<site_dir>/wp-content/plugins/crowdsec/vendor/twig/twig/src/Environment.php(817): Twig\ExtensionSet->getUnaryOperators()
#4 /var/www/<site_dir>/wp-content/plugins/crowdsec/vendor/twig/twig/src/Lexer.php(457): Twig\Environment->getUnaryOperators()
#5 /var/www/<site_dir>/wp-content/plugins/crowdsec/vendor/twig/twig/src/Lexer.php(108): Twig\Lexer->getOperatorRegex()
#6 /var/www/<site_dir>/wp-content/plugins/crowdsec/vendor/twig/twig/src/Envi

Additionally, I provide information about the production environment:
wordpress-prod.txt
Server OS:
Distributor ID: Debian
Description:    Debian GNU/Linux 11 (bullseye)
Release:        11
Codename:       bullseye

Debian 11.6

Crowdsec:
2023/02/10 00:50:32 version: v1.4.5-debian-pragmatic-a9a2186a76af63551352aa3bc296bdbe80ca4893
2023/02/10 00:50:32 Codename: alphaga
2023/02/10 00:50:32 BuildDate: 2023-01-19_15:06:57
2023/02/10 00:50:32 GoVersion: 1.19.2
2023/02/10 00:50:32 Platform: linux
2023/02/10 00:50:32 Constraint_parser: >= 1.0, <= 2.0
2023/02/10 00:50:32 Constraint_scenario: >= 1.0, < 3.0
2023/02/10 00:50:32 Constraint_api: v1
2023/02/10 00:50:32 Constraint_acquis: >= 1.0, < 2.0

Plugin settings that are changed from default in production environment:

Use cURL to call Local API - Yes
Local API request timeout - 10 sec
Bouncing level - Flex bouncing
Public website only - No
Theme customization - Messages and custom CSS code
Captcha flow cache lifetime - 604800 sec
Geolocation cache lifetime - 604800 sec

I'm sorry to face this problem, but I'm ready to help :slightly_smiling_face:
I hope that the problem will be confirmed by you.
If you need any additional information, tests on this issue, I am always ready to provide and help with this.
Thank you in advance!
Best regards,
Maksim Strekalovskikh

Hi @MaxSH97, thanks for your message.
Indeed, it seems that the TwigTest.php file is missing in the zip file.
I will try to debug this asap.

Hi @MaxSH97, I just published a 2.0.1 release with a fix.
Could you test it to see if it works as expected now ?
For information, the TwigTest.php files was present on the source of this repo (https://github.com/crowdsecurity/cs-wordpress-bouncer/blob/v2.0.0/vendor/twig/twig/src/TwigTest.php) but the release process is using a .distignore file to create the zip and it was removing all files ending by Test.php (https://github.com/crowdsecurity/cs-wordpress-bouncer/blob/v2.0.0/.distignore#L34).
I will add a test GitHub action to test the final release zip (and not only the sources, as it is currently the case). Thus, it should help to avoid such mishaps.
Thanks again for reporting this issue.

Hello, @julienloizelet!
Thank you very much for checking and making a fix very quickly!
During this day I will test this fix on a test environment, and I will return with the results of the test.

@julienloizelet I'm back with test results on a test environment.
2.0.1 now works as expected. Ban and captcha walls are displayed without problems.
I have no questions or doubts.
I'll update plugin in production environment right now.
Thank you very much again for the response and the quick fix!
I'm closing this issue.
Have a nice day! :wink:


Bump dependencies

Bumped pkg/xattr.
Please release version 2.1 (miss -dry-run badly) :)

Merged, thanks!

v2.1.0 released now


update cdn - all to jsdelivr.net

some countries in SEA having issue connecting maxcdn.bootstrapcdn.com for past few months, contacted provider - no luck.
(optional) Issue number:
Summary of the change:

Thanks @NavanithanS !


How to get SS_cons when building my own cm file?

I am trying to analyze tRNAs of non-model organisms (parasites) because I found out that Rfam doesn't include the tRNA sequences of my interest. I aligned all sequences of my interest and then tried to convert it to Stockholm format having WUSS notations. However, I can't find how to convert WUSS notations and then add SS_cons.
This case has been quite challenging for me over the past week. Could you help me out with this?

Hi Wonwill,
I understand that you have an alignment and a separate secondary structure (SS) in WUSS format, and you want to combine them.
Here are some options depending on your specific goal:

Adding tRNA Rfam family SS_cons to your alignment:

We recommend using Infernal https://github.com/brendanf/inferrnal for this task.
You'll likely need a FASTA file containing your sequences and the Rfam family CM file.

Extracting SS_cons from an existing alignment:

Use R-scape https://github.com/EddyRivasLab/R-scape to retrieve both the alignment and the more covering corresponding SS_cons.
If either of these scenarios describes your situation, please let me know.
For further assistance:
Clarify your goal: Briefly describe your SS source.
Provide an example: A snippet of your alignment and the SS_cons, and CM file (if applicable) would be very helpful.

Thanks for your kind and quick reply.

a separate secondary structure (SS) in WUSS format

Actually, not.
I have only an alignment file (ClustalW). I don't know how to get individual or consensus SS  in WUSS format.
I am trying to analyze the tRNA of Echinostoma sp. (Parasitic fluke) as the attached example file.
E_trnaA_v2.aln.txt

My first goal is to build my own Echinostoma tRNA CM file.
The second goal is to add my own CM file to a typical RFam CM file.
I would like to compare two results because it is currently difficult to annotate my mitogenome.

It seems like you're aiming to predict secondary structures (SS_cons) for your specific alignment, rather than simply annotating the existing Rfam families SS_cons.
Here are two potential approaches:
R-scape: This tool is excellent for predicting secondary structures from alignments. You can input your alignment and R-scape will generate the corresponding SS_cons.
RNAalifold: https://www.tbi.univie.ac.at/RNA/RNAalifold.1.html This tool from the ViennaRNA package can also be used for secondary structure prediction, especially for individual sequences or smaller alignments.
Given that tRNA secondary structures are highly conserved, I'd recommend relying on the Rfam SS_cons for accurate predictions. You can use Infernal to create a Stockholm file containing both the alignment and the Rfam SS_cons.
Please let me know if you have any other questions or if you'd like to discuss this further.

Hi,
Regarding

R-scape output.

R-scape tends to predict a large number of covarying base pairs, including non-canonical pairs and coaxial interactions. This can sometimes exceed the level of complexity found in Rfam families. To align with Rfam's criteria, you might use the --Rfam option in R-scape.
I've tested R-scape on your 4-sequence Hydra vulgaris alignment, using the Rfam SEED and the --Rfam option. Please find the results attached.
# STOCKHOLM 1.0
# Rfam RF04010 SEED

URS000075BBC7_6087/2-77 AGCTATTTATATCATTACCAAATTGCACACTTAACATAACTGTTAAATGTGCAAACTGGTAATGAAATAAATACTG
URS0000D558DF_6087/1-76 TGCTATTTATATCATTACCAGTGTGCACATTTAACATTATTGTTAAATGTGCGAACTGGTAATGAAATAAATACTG
#=GC SS_cons            :::<<<<<<<-<<<<<<<<<<<<<<<<<<<<<<<<______>>>>>>>>>>>>>>>>>>>>>>>>->>>>>>>:::
#=GC RF                 uGCuauuuauAuCauuaCCaauuuGCaCauuuaaCaUuAuuGuuaaauGuGCaAACuGGuaauGaAauaaauaCUG
//

# STOCKHOLM 1.0
# Hydra vulgaris

#=GS NW_004171009.1/56862-56787 DE Hydra vulgaris strain 105 unplaced genomic scaffold, Hydra_RP_1.0 HYDRAscaffold_35197, whole genome shotgun sequence
#=GS NW_004170022.1/71698-71773 DE Hydra vulgaris strain 105 unplaced genomic scaffold, Hydra_RP_1.0 HYDRAscaffold_36186, whole genome shotgun sequence
#=GS NW_004170022.1/71773-71698 DE Hydra vulgaris strain 105 unplaced genomic scaffold, Hydra_RP_1.0 HYDRAscaffold_36186, whole genome shotgun sequence
#=GS NW_004171009.1/56787-56862 DE Hydra vulgaris strain 105 unplaced genomic scaffold, Hydra_RP_1.0 HYDRAscaffold_35197, whole genome shotgun sequence

NW_004171009.1/56862-56787 AGCUAUUUAUAUCAUUACCAAAUUGCACACUUAACAUAACUGUUAAAUGUGCAAACUGGUAAUGAAAUAAAUACUG
NW_004170022.1/71698-71773 UGCUAUUUAUAUCAUUACCAGUGUGCACAUUUAACAUUAUUGUUAAAUGUGCCAACUGGUAAUGAAAUAAAUACUG
NW_004170022.1/71773-71698 CAGUAUUUAUUUCAUUACCAGUUGGCACAUUUAACAAUAAUGUUAAAUGUGCACACUGGUAAUGAUAUAAAUAGCA
NW_004171009.1/56787-56862 CAGUAUUUAUUUCAUUACCAGUUUGCACAUUUAACAGUUAUGUUAAGUGUGCAAUUUGGUAAUGAUAUAAAUAGCU
#=GC SS_cons               :::<<<<<<<-<<<<<<<<<<<<<<<<<<<<<<<<______>>>>>>>>>>>>>>>>>>>>>>>>->>>>>>>:::
//

RF04010_test_Hydra_vulgaris.tar.gz

@cykim1029 : infernal's cmbuild program does not require the SS_cons annotation be in WUSS format. The dot parentheses  notation will work just fine.
To build a CM starting with your clustal alignment, do the following steps:

install infernal 1.1.5 (http://eddylab.org/infernal/)

convert your clustal alignment to 'pfam' format (a special kind of stockholm format in which each sequence is on a single line - this will make it easier to add your SS_cons notation in step 3), by running the following command:
infernal-1.1.5/easel/miniapps/esl-reformat pfam in.clustal > out.pfam

Manually modify the out.pfam file in a text editor by adding your dot parantheses structure prediction from RNAalifold after the final sequence line (but before the // line) with the prefix #=GC SS_cons, like this:
#=GC SS_cons ...((((...))))...
And then save the file.
You could also add the R-scape predicted structure here. It is not a big problem that this is different from the Rfam structure in the RF04010 test you performed. As @nancyontiveros wrote above, R-Scape may predict additional or different basepairs from what the Rfam structure has, this is because predicting the correct structure is a difficult problem and different approaches will yield slightly different results. Rfam uses structures from different sources and from a variety of prediction methods.

Build a CM with the command:
infernal-1.1.5/src/cmbuild -F my.cm out.pfam
The output should indicate that there are N>0 basepairs in your model in the bps field.


Update PYPI package to the latest

Seems like the latest version at PYPI is outdated.

This was resolved with https://github.com/Shizmob/pydle/commit/3c1a6d11e949f0f06f4f3e059f859f34e149b4a2.


Fix confusing code around plot marker size vs stroke width

What

Part of https://github.com/rerun-io/rerun/issues/6540

The existing code was using the same field to mean different things at different times. I have no idea if the original code was buggy, since it didn't document expected behavior.
Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[x] I've included a screenshot or gif (if applicable)
[ ] I have tested the web demo (if applicable):

Using examples from latest main build: rerun.io/viewer
Using full set of examples from nightly build: rerun.io/viewer

[x] The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG
[x] If applicable, add a new check to the release checklist!

PR Build Summary
Recent benchmark results
Wasm size tracking

To run all checks from main, comment on the PR with @rerun-bot full-check.

Assuming there is a change. The previous MarkerSize did not document if the size referred to the radius, diameter, or something else.


clarify the task section of the tutorial documentation

Changes

As a first time tekton user, when I was following the tutorial in the creating and running a task section it was not immediately obvious to me that I had to create and apply both the task and taskrun yamls to kubernetes. I applied only the taskrun and got an error saying the task didn't exist. It was pretty clear to me what to do at that point (apply the task) but if the guide had made it a little more clear to apply both files it would have saved me the step of figuring out what went wrong.
In this PR, I am proposing a super simple tutorial documentation change to just clarify that step a little bit, and hopefully aid future first time users.
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[x] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
~If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.~ N/A
Release Notes
clarify the difference between task/taskrun

I went ahead and completed the CLA 👌

Sorry for the force-push spam. I wanted to make sure my commit message adhered to the guidelines and that my committer info was properly attached to my github account. I had forgotten to set the local git config. 😅

/test pull-tekton-pipeline-build-tests

/kind documentation
/ok-to-test

/test pull-tekton-pipeline-build-tests


Brazilian Portuguese Translation Final Tasks!

Congratulations!
You all are almost done translating the React docs into Portuguese! There are couple more errands to do before we can mark the project as "completed"

[x] Replace "reactjs.org" to "pt-br.reactjs.org" in the following places:

[x] package.json
[x] site-constants.js
[x] gatsby-config.js
[x] MarkdownPage (the link href in "Edit this page")

[x] Change html.js to have the attribute lang="pt-BR"
[ ] Go over each section in the site to:

[ ] Make sure there are no formatting errors (dangling markdown/html tags) or untranslated English
[ ] Make sure the sections and style are consistent.

[ ] Change the DocSearch API key. Signed up. Waiting for DocSearch API Key One of the maintainers needs to sign up for DocSearch

We're almost there everyone!

@tesseralis
Hey Nat! Will this have any immediate effect on something?
I mean, should we wait to have at least all the core pages done to make this changes or could we do it now just to make sure it will be ready?
Thanks!

Thank you for pointing our last steps @tesseralis! We're almost there indeed :tada:
I've made some of the changes on #124. I've also signed up for the DocSearch key, will comment here when we get their response.

@tesseralis awesome. We have to say thank you for all the brazilian community help. It was insane. But we will see about the topics that you listed it and then finish this issue. :)

@halian-vilela hmm. I think I'd wait until a certain section (like "main concepts") is completed, and then go over all of it to see if the whole thing feels consistent.

These items listed by @tesseralis, can I help with any of these or is there someone already doing that?

That's awesome, sorry to be away guys, but I had some personal issues that took a lot of my free time

@WendellAdriel you can take it this one:
* Change the [DocSearch API key](https://github.com/reactjs/es.reactjs.org/blob/ed07fa53dc9ab7950e630900f7da3a7c821d63a6/src/components/LayoutHeader/DocSearch.js#L25). **Signed up. Waiting for DocSearch API Key** One of the maintainers needs to [sign up for DocSearch](https://community.algolia.com/docsearch/#join-docsearch-program)

Great @glaucia86, I'll do that then!!!
Has any of the other maintainers signed up already for DocSearch?

Hi everyone. I've already signed for DocSeach. I'm awaiting for their response and will comment here. We will probably hear from them this week since I applied on the weekend. Cheers!

@tibuurcio awesome! I just got the email from the Spanish repo after around a week. Shouldn't take too long.

I have a little hack i used to make suggestions in PR's, and this can be useful to spot untranslated/misspelled words:
document.designMode === 'on'
// focus on some text or scroll the and you will see autocorrect suggestions
document.designMode === 'off'

Got the email from DocSearch yesterday already! I'll update our code as soon as I can :)

@tibuurcio I'm sorting out issues with docsearch for spanish here: https://github.com/reactjs/es.reactjs.org/pull/184#issuecomment-467345047
it may be useful to you

[ ] Go over each section in the site to:

[ ] Make sure there are no formatting errors (dangling markdown/html tags) or untranslated English
[ ] Make sure the sections and style are consistent.

I'm pointing some items out on https://github.com/reactjs/pt-BR.reactjs.org/issues/134
I'll need some help on that as well :)

@eduardomoroni about the #134 I already did a code review about the Nav bar situation and I asked to the contributor to make some changes on it! So... he did it and I already approve the contribution #131. Try to see right now @eduardomoroni if everything it's correct right now! ;)

It seems everything all right. Should mark this as done?

It seems everything all right. Should wemark this as done?

Agree w/ u @eduardomoroni. Let's finish this translation! :)


WinAppDriver 1.0 How to get BackgroundColor/ForegroundColor of element in Native Wins App

I tried to use GetAttribute of element to get text patterns like background color, foreground color... but unlucky. Please can help!

Hi @hainguyenh,
You can use GetAttribute to get all element attributes shown on inspect.exe such as Name, ControlType, IsEnabled, etc. Some examples can be found on our sample test below:
https://github.com/Microsoft/WinAppDriver/blob/v1.0-RC/Tests/WebDriverAPI/ElementAttribute.cs#L38
Unfortunately, other attributes like background color and foreground color are not exposed through the accessibility layer which made it not accessible in Windows Application Driver.

Hi @timotiusmargo,
Thanks a lot for your confirmation.

What accessibility layer are you referring to?  Is this in Appium or the Windows control?


About Documentation

Please can you provide documentation on how to implement or please can you guide me how to implement it.

Hey! you can look at the Readme.md file, which has steps to run the backend

Add a .env file with
GOOGLE_CLIENT_ID=<CLIENT ID FROM GOOGLE CREDENTIALS>
GOOGLE_CLIENT_SECRET=<CLIENT ISECRET FROM GOOGLE CREDENTIALS>
JWT_SECRET=someSecret"
REDIRECT_URL=<redirect base url for google auth>
REDIRECT_FRONTEND=<redirect base url to frontend>
EMAIL_REDIRECT=<redirect base url within email>
AWS_ACCESS_KEY_ID=<AWS IAM access key id>
AWS_SECRET_ACCESS_KEY=<AWS IAM secret key>
AWS_DEFAULT_REGION=<AWS IAM region>
SES_EMAIL=<email to be used for SES>
RECAPTCHA_SECRET=<Google Recaptcha Secret>
ADMIN_PASSWORD =<admin portal email>
ADMIN_EMAIL=<Admin portal password
AWS_S3_BUCKET=<AWS S3 Bucket name> ```

i have uploaded your backend in my server but when i start i can't see admin panel. https://codeapi.sabrangjklu.tech this is the website

This repo has the apis for the admin portal, you can find admin portal over here https://github.com/CodeChefVIT/DEVSOC21-Admin-Portal

Sir then how to create database. I mean Mongo URL and configure it.

You can use the free tier of https://www.mongodb.com/atlas/database , you will get connection url of mongodb database

Default password for admin panel please.


Adding an install method for no qwerty

I have no need for the qwerty layer, and the provided xkbmap didn't work without some tweaking as a result. This removes that layer and makes halmak easy to install.
Also I didn't include this, but I highly recommend, much like colemak and other keyboards, to use capslock -> backspace .
key <CAPS> {[BackSpace]};

Hey friend, thanks for the PR! Do you mind adding some more wording into the readme please? Just to avoid confusion between the no querty and the querty controls variations. Thanks!

Done, let me know how this looks.

amazing, thanks!


[Bug]: Application is throwing "Resource not found" randomly

Is there an existing issue for this?

[X] I have searched the existing issues

Description
This issue is observed in CI , where on navigating between api's, application is throwing "Resource not found" error randomly

Steps To Reproduce

Login to Appsmith
Create an app
Create 2 or more api's
Navigate between api's

Public Sample App
No response
Environment
Release
Issue video log
https://github.com/appsmithorg/appsmith/assets/119562824/9bdab387-6db5-41aa-bd82-65dc4151f0ec
Version
Self hosted

Hi, my initial analysis here is that the UI is trying to access a resource while it it getting updated i.e. we can see the saving icon on the top left still under progress while the app is trying to access something (the 2nd API maybe) ? and it is possible that the 2nd API is still not persisted in the API server DB in the desired way. Overall, at the moment seems like a performance issue.


Fix server-side keepalive shutdown process

#9468 added client-side keepalive timers, #9902 extended them to servers. But these timers were not canceled explicitly on servers when channels were closed by close_transport_locked. They were canceled in the cancellation path of the bandwidth estimator.
This PR cancels keepalive timers explicitly on servers, so that the cancellation of keepalive timers won't rely on the cancellation of the bandwidth estimator.

No significant performance differences

Added more context in the description. It should be canceled on both the client and server. #9902 forgot to cancel it on server, but luckily the cancellation path of bandwidth estimator helped to cancel it, so it did not cause any trouble (also not caught by our tests :dizzy_face: ) .

Thanks Yuchen. LGTM

Performance differences noted:
Benchmark                                                                         cpu_time    real_time
------------------------------------------------------------------------------  ----------  -----------
BM_ErrorCreateAndSetStatus                                                          +75.50       +75.50
BM_ErrorStringOnNewError<ErrorWithGrpcStatus>                                      +261.00      +261.00
BM_PollEmptyPollset_SpeedOfLight/1/0                                                 +7.50        +7.50
BM_PollEmptyPollset_SpeedOfLight/1/100                                               +7.50        +7.50
BM_PollEmptyPollset_SpeedOfLight/1/1000                                              +8.50        +8.50
BM_PollEmptyPollset_SpeedOfLight/1/9.76562k                                          +7.50        +7.50
BM_PollEmptyPollset_SpeedOfLight/1/97.6562k                                          +8.00        +8.00
BM_PollEmptyPollset_SpeedOfLight/10/1                                                +9.50        +9.50
BM_PollEmptyPollset_SpeedOfLight/1000/1                                              +7.50        +7.50
BM_StreamCreateSendInitialMetadataDestroy<RepresentativeClientInitialMetadata>     +342.50      +342.50
BM_TransportEmptyOp                                                                 +11.50       +11.50
BM_TransportStreamRecv/128M                                                     +297601.00   +297600.50
BM_TransportStreamRecv/16M                                                       +35076.00    +35075.50
BM_TransportStreamRecv/256k                                                        +583.00      +583.00
BM_TransportStreamRecv/2M                                                         +4288.00     +4288.50
BM_TransportStreamRecv/32k                                                         +165.00      +165.00
BM_TransportStreamRecv/4k                                                           +91.50       +91.50
BM_TransportStreamRecv/64                                                           +89.00       +89.50
BM_TransportStreamSend/0                                                            +47.50       +47.50
BM_TransportStreamSend/1                                                            +45.00       +45.00
BM_TransportStreamSend/128M                                                      +98201.00    +98203.00
BM_TransportStreamSend/16M                                                       +12727.50    +12727.50
BM_TransportStreamSend/256k                                                        +240.50      +240.50
BM_TransportStreamSend/2M                                                         +1440.00     +1439.00
BM_TransportStreamSend/32k                                                          +76.00       +76.00
BM_TransportStreamSend/4k                                                           +52.50       +52.50
BM_TransportStreamSend/512                                                          +46.00       +46.00
BM_TransportStreamSend/64                                                           +61.00       +61.00
BM_TransportStreamSend/8                                                            +59.00       +59.00

No significant performance differences

Known issues:
Basic Tests Linux: #9147


[BUG]: Aggegrate error using soundcloud as source

NodeLink version
GitHub commit: e871afa
Description
This is the error I get upon attempting to play soundcloud track
12|npm              | [http1makeRequest]: Failed sending HTTP request to https://a-v2.sndcdn.com/assets/0-b17d3b6c.js: AggregateError
12|npm              | [soundcloud]: Failed to fetch clientId.
12|npm              | file:///root/root/nodelink/src/sources/soundcloud.js:27
12|npm              |   const { body: data } = await http1makeRequest(assetId, {
12|npm              |                 ^
12|npm              | TypeError: Cannot destructure property 'body' of '(intermediate value)' as it is undefined.
12|npm              |     at Object.init (file:///root/root/nodelink/src/sources/soundcloud.js:27:17)
12|npm              |     at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
12|npm              |     at async file:///root/root/nodelink/src/connection/handler.js:817:7
12|npm              | Node.js v21.1.0

Client
FastLink (1b19ba6)
Steps to reproduce
Connect to NodeLink
Search/play a track using scsearch
The issue should appear in console
Example code
No response
Confirmations

[X] My environment meets the minimum requirements.
[X] I have verified that this is not a duplicate issue.

Code of Conduct

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

Supposedly created by a network issue, nothing much to be done about the fail itself, however I'll implement better error handling there to avoid crash and using the next release system to disable sources on runtime, to simply disable SoundCloud instead of taking the entire node down.

https://github.com/PerformanC/NodeLink/commit/e998f7d1d265fe13b910dcc89f20f2395a1d51c5 should better handle those.


Linkify the related objects

I have uploaded a few documents and when viewing them from the plugin side, I would like to directly click on the related objects to open those pages up
Here, I would prefer if I could click on the Site DUBLIN

Maybe the filename should also be made clickable to view it?

I'll release v0.3.1 likely tomorrow with these changes. Nice suggestion, thanks.


Why when shutting services down you can still see them and get their url's

I am using spring boot 2.1.0
I started two instances of microservice X and i can see them on eureka ui.
When i shut down one of them, i can see it for a long time and the load balancer gives the url of the turned off service 50% of the times..
Can i configure eureka to remove the service after couple of seconds?

Eureka and ribbon both have built-in caches at the server and client. You can certainly change those values, but eureka is built as an eventually consistent store. If you need immediate consistency, another discovery service (such as consul) might be a better choice.

Can i configure eureka to remove the service after couple of seconds?

Hi @VitalyKheifets
So far there is not configure you can set that.For eureka server has a background task which is delete expired instance(default value is 60s).
As far as i know you can reduce this delay through configure, but it won't be done immediately(As mentioned by spencergibb).
Turn off self-protection mechanism
Turn off the eureka server cache mechanism
Thanks

@VitalyKheifets if you want to properly shutdown service while using Eureka you should firstly remove it from Eureka, wait around 60 seconds until it is removed from cache after that you can safely shutdown your service.

By removing from Eureka I mean putting it manually into DOWN or OUT_OF_SERVICE status via REST API.

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


try loading as TF SavedModel instead of HDF5

In trying to solve #87, I have converted the models and changed the loader by trying to omit the .h5 suffix.
Unfortunately, I now get a new error:
00:49:28.543 INFO eynollah - INPUT FILE PHYS_0005 (1/19)
00:49:28.824 INFO eynollah - Resizing and enhancing image...
00:49:28.825 INFO eynollah - Detected 300 DPI
Traceback (most recent call last):
  File "/local/ocr-d/ocrd_all/venv/bin/ocrd-eynollah-segment", line 8, in <module>
    sys.exit(main())
  File "/local/ocr-d/ocrd_all/venv/lib/python3.8/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/local/ocr-d/ocrd_all/venv/lib/python3.8/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/local/ocr-d/ocrd_all/venv/lib/python3.8/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/local/ocr-d/ocrd_all/venv/lib/python3.8/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/local/ocr-d/ocrd_all/venv/lib/python3.8/site-packages/qurator/eynollah/ocrd_cli.py", line 8, in main
    return ocrd_cli_wrap_processor(EynollahProcessor, *args, **kwargs)
  File "/local/ocr-d/ocrd_all/venv/lib/python3.8/site-packages/ocrd/decorators/__init__.py", line 117, in ocrd_cli_wrap_processor
    run_processor(processorClass, ocrd_tool, mets, workspace=workspace, **kwargs)
  File "/local/ocr-d/ocrd_all/venv/lib/python3.8/site-packages/ocrd/processor/helpers.py", line 107, in run_processor
    processor.process()
  File "/local/ocr-d/ocrd_all/venv/lib/python3.8/site-packages/qurator/eynollah/processor.py", line 58, in process
    Eynollah(**eynollah_kwargs).run()
  File "/local/ocr-d/ocrd_all/venv/lib/python3.8/site-packages/qurator/eynollah/eynollah.py", line 2310, in run
    img_res, is_image_enhanced, num_col_classifier, num_column_is_classified = self.run_enhancement()
  File "/local/ocr-d/ocrd_all/venv/lib/python3.8/site-packages/qurator/eynollah/eynollah.py", line 1993, in run_enhancement
    is_image_enhanced, img_org, img_res, num_col_classifier, num_column_is_classified, img_bin = self.resize_and_enhance_image_with_column_classifier()
  File "/local/ocr-d/ocrd_all/venv/lib/python3.8/site-packages/qurator/eynollah/eynollah.py", line 408, in resize_and_enhance_image_with_column_classifier
    _, page_coord = self.early_page_for_num_of_column_classification(img_bin)
  File "/local/ocr-d/ocrd_all/venv/lib/python3.8/site-packages/qurator/eynollah/eynollah.py", line 654, in early_page_for_num_of_column_classification
    img_page_prediction = self.do_prediction(False, img, model_page)
  File "/local/ocr-d/ocrd_all/venv/lib/python3.8/site-packages/qurator/eynollah/eynollah.py", line 528, in do_prediction
    img_height_model = model.layers[len(model.layers) - 1].output_shape[1]
  File "/local/ocr-d/ocrd_all/venv/lib/python3.8/site-packages/keras/engine/base_layer.py", line 2132, in output_shape
    raise AttributeError(
AttributeError: The layer "activation_56" has never been called and thus has no defined output shape.

Given that this seems to be the output layer, I am at a loss here. Creating as draft only.
Ideas anyone?

@vahidrezanezhad could it be that the way the layers get defined in sbb_pixelwise_segmentation should use  input_shape as kwarg instead of data_format?

Ah, here's the reason:
WARNING:tensorflow:SavedModel saved prior to TF 2.5 detected when loading Keras model. Please ensure that you are saving the model with model.save() or tf.keras.models.save_model(), *NOT* tf.saved_model.save(). To confirm, there should be a file named "keras_metadata.pb" in the SavedModel directory.

So I must do the model conversion on Python <= 3.7 but TF >= 2.5 ...

Ok, it does work.
I can also update ocrd-tool.json here to include the changed resource URL. But first I need some place to upload it permanently. How about qurator-data.de?
(@kba knows where to find the new archive...)

@bertsky Thank you so much. If you can upload the saved model somewhere shareable, we can publish it to qurator-data.
btw sorry for being unresponsive here, we are currently fighting the ICDAR2023 submission deadline for a paper on...Eynollah ;)

BTW, CI failure is because there is no TF2 for Py 3.6 anymore. Config in GH Actions should be changed (back) to 3.7 or 3.8.

BTW, CI failure is because there is no TF2 for Py 3.6 anymore. Config in GH Actions should be changed (back) to 3.7 or 3.8.

This is what I tried in https://github.com/qurator-spk/eynollah/pull/90.

@kba asked me to upload this, but 1. I'd prefer @vahidrezanezhad to do it 2. that he produces the updated models himself. Is this just about - skims the issue comments - being able to load the files with a newer Python/TF version? Then I could maybe be persuaded to do it.
Not trying to be a pedant, it's just professional diligence before I upload some variant of some model I don't fully understand myself.

(Alternatively, I offered @kba access to the relevant resources (our internal git annex + qurator-data.de), then the diligence is up to him.)

Skimming the PR, it's not only about loading the model, but some other stuff as well? (Cleaning up and a change of a larger code part)

(And yes, I did test, of course.)

;-) As I will only be uploading the files, that's not my concern. (I would try to test myself if it was - had the tiniest change break something unexpected too often.)

Regarding model upload: if you could do that first, then I could update this PR accordingly – adding the new URL to the local ocrd-tool.json, so it would immediately become available as the current model location (independent of old URLs registered in core).
(I really just loaded as HDF5 and stored as SavedFormat, nothing fancy.)

Yeah, I think @kba will give me the files and I'll upload them (or I give him access.)

Sure – I would also recommend to first allow @vahidrezanezhad to take a look, even if the changes are more or less cosmetic.

He's on vacation btw.

@cneud wants to discuss this with @kba (and me, i think) at our regular meeting, which is understandable.

It would be great if we can put the TF saved model versions of the models on qurator-data already (and possibly HF later as well), for testing. About the other changes and the PR, I would like to have some discussion to see if we can review/merge while Vahid is away.
So

a. I would upload the data if necessary,

Yes please.

b. reviewing, merging this PR must be done by someone else, there is too much going on

To be discussed.

a. I would upload the data if necessary,
Yes please.

Still waiting for the data :) @kba, @bertsky?

b. reviewing, merging this PR must be done by someone else, there is too much going on
To be discussed.

Yes, that's 100% "in meinem Sinne"/my opinion. I'd rather be not involved, as I am currently not familiar with the code base.

Still waiting for the data :) @kba, @bertsky?

https://ocr-d.kba.cloud/2021-04-25.SavedModel.tar.gz

Fixes #92
Fixes #87

https://qurator-data.de/eynollah/2021-04-25/SavedModel.tar.gz
I've created a short README, to document how this file was created, see https://qurator-data.de/eynollah/2021-04-25/. @bertsky Could you post a short code snippet here, how you converted this? Could be useful in the future.
@joergleh Relevant to your efforts, probably.

Many thanks @mikegerber!
@bertsky This should be the location for the model(s), rather than https://ocr-d.kba.cloud/2021-04-25.SavedModel.tar.gz, which was just for temporary file transfer, so https://github.com/qurator-spk/eynollah/pull/91/commits/00f431a5d6152aba96675b4f26d89f44bfdef211 would need to be adapted again.

@mikegerber it boils down to:
import os
from tensorflow.keras.models import load_model
for path in os.listdir('default'):
    if not path.endswith('.h5'):
        continue
    model = load_model(path, compile=False)
    model.save(path[:-3], save_format="tf")

@cneud done!

Wait! I just found out my "cosmetic" changes are not so cosmetic after all … investigating.

@bertsky Thanks!
Where does the default directory name come from? I was wondering the same while examining @kba's tar. Is it OCR-D convention?

@bertsky Sorry for the laughing emoji :) Couldn't resist!

@mikegerber

Where does the default directory name come from? I was wondering the same while examining @kba's tar. Is it OCR-D convention?

That's just the model name (i.e. last directory inside the archive = directory to unpack). Put your absolute or relative path there.

Sorry for the laughing emoji :) Couldn't resist!

No worries. I shouldn't have been so bold to begin with!

I just realized that the conversion .h5 -> TF SavedModel was done using the models from https://qurator-data.de/eynollah/2021-04-25/ where rather the models from https://qurator-data.de/eynollah/2022-04-05/ should have been used.
Other than having more comprehensible model names, the 2022-04-05 dir also contains an additional model eynollah-main-regions_20220314.h5 which is required for full functionality.

@cneud,

I just realized that the conversion .h5 -> TF SavedModel was done using the models from https://qurator-data.de/eynollah/2021-04-25/ where rather the models from https://qurator-data.de/eynollah/2022-04-05/ should have been used.
Other than having more comprehensible model names, the 2022-04-05 dir also contains an additional model eynollah-main-regions_20220314.h5 which is required for full functionality.

you should know better than me, but as of Vahid's earlier response I think this only belongs to the (now public) eynollah_light branch.
I can convert these models as well, if you like, but this must be done similarly against that branch.

I see (the mess...).
But since that eynollah_light branch is "approved" despite merge conflicts, that is sth we likely have to keep waiting until Vahid is back. :confused:

Wait! I just found out my "cosmetic" changes are not so cosmetic after all … investigating.

I removed these parts, rebased and force-pushed. (Sorry for the noise!)

But since that eynollah_light branch is "approved" despite merge conflicts, that is sth we likely have to keep waiting until Vahid is back.

I have converted these models as well and placed them in the same location for @kba to upload. I suggest merging master into eynollah_light after this has been merged into master (as the easiest path AFAICS).

But since that eynollah_light branch is "approved" despite merge conflicts, that is sth we likely have to keep waiting until Vahid is back.

I have converted these models as well and placed them in the same location for @kba to upload. I suggest merging master into eynollah_light after this has been merged into master (as the easiest path AFAICS).

Available from https://ocr-d.kba.cloud/2022-04-05.SavedModel.tar.gz

I refuse to upload 2022-04-05.SavedModel.tar.gz as the original is not fully in our internal data repository, sorry. This must be corrected by Vahid first.
Here, I am pedantic because it is not the first time this kind of situation comes up and while I am happy to support, I don't want to clean up all the time.

Take your time! This PR can wait, and even more so any further action on eynollah_light.

No worries. I shouldn't have been so bold to begin with!

I appreciate the boldness, I just think you should have been bold in another PR ;) I know it's more work but for me, while reviewing a PR like this (even if it's just to understand the upload situation), it's more work if the PR contains more than just the changes necessary.
However, this is not my project, so it's not up to me, actually.

Side note: This PR seems to fix the CircleCI tests, it that correct? Great stuff if this is true!

Side note: This PR seems to fix the CircleCI tests, it that correct? Great stuff if this is true!

No. This PR only fixes Python 3.8 onwards. The CI check comes with @cneuds config changes (throwing out broken 3.6, activating 3.7 onwards).

Note, since we have moved towards Python 3.8 as base version, this is becoming rather urgent. See ocrd_all – where I am now considering switching to this PR version.


Close button on View Snippet is not clickable

🔍 Is there already an issue for your problem?

[X] I have checked older issues, open and closed

ℹ Environment / Computer Info
- Spotify version: <IP_ADDRESS>4.g39a25e2c
- Spicetify version: v2.36.14

📝 Description
Click the close button does not do anything.
📸 Screenshots

Works just fine.


added data-test atttribute for checkboxes

Added the ability to have a data-test attribute on a checkbox input

Noice. Will publish with the next release.


In case someone looking for fish init scripts

In case of someone looking for fish init script.
here you go https://gist.github.com/alialrahahleh/0738969d28f4e340dc0d6ab6216bcc4c

I just updated the script https://gist.github.com/alialrahahleh/229f21327a0228ee4b3d770e074f245e, instruction:
Place the files in specified directory. ( I replaced '/' with '_') because gist doesn't allow me to use the real path.
Please note this doesn't expose sdk to fish but it only expose the candidates to fish. sdk is written as bash function and it will be time consuming to rewrite that code in fish shell.

You may be interested in reitzig/sdkman-for-fish.


Sort crates by downloads

This has been discussed previously in https://github.com/rust-gamedev/arewegameyet/issues/157. The main issues with sorting by downloads are:

We'd only be able to do it for items that are hosted on Crates.io.
It disproportionately favours older and low-level crates. For example, Piston is the most downloaded game engine, because it was one of the first released, but it's not the most widely used today.


Fix tests in some environments

My environment doesn't have securerandom loaded by default, and needs an explicit require 'securerandom' for the tests to pass. I was getting this:
$ rake
/home/eric/.rbenv/versions/2.2.0/bin/ruby -I/home/eric/.rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/rspec-support-3.1.2/lib:/home/eric/.rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/rspec-core-3.1.7/lib /home/eric/.rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/rspec-core-3.1.7/exe/rspec --pattern spec/\*\*\{,/\*/\*\*\}/\*_spec.rb
................F....F.....

Failures:

  1) U2F#authentication_requests returns an array of requests
     Failure/Error: let(:requests) { u2f.authentication_requests(key_handle) }
     NameError:
       uninitialized constant SecureRandom
     # ./lib/u2f/u2f.rb:75:in `challenge'
     # ./lib/u2f/u2f.rb:24:in `block in authentication_requests'
     # ./lib/u2f/u2f.rb:23:in `map'
     # ./lib/u2f/u2f.rb:23:in `authentication_requests'
     # ./spec/lib/u2f_spec.rb:36:in `block (3 levels) in <top (required)>'
     # ./spec/lib/u2f_spec.rb:38:in `block (3 levels) in <top (required)>'

  2) U2F#registration_requests returns an array of requests
     Failure/Error: let(:requests) { u2f.registration_requests }
     NameError:
       uninitialized constant SecureRandom
     # ./lib/u2f/u2f.rb:75:in `challenge'
     # ./lib/u2f/u2f.rb:86:in `registration_requests'
     # ./spec/lib/u2f_spec.rb:77:in `block (3 levels) in <top (required)>'
     # ./spec/lib/u2f_spec.rb:79:in `block (3 levels) in <top (required)>'

Finished in 0.01373 seconds (files took 0.346 seconds to load)
27 examples, 2 failures

Failed examples:

rspec ./spec/lib/u2f_spec.rb:37 # U2F#authentication_requests returns an array of requests
rspec ./spec/lib/u2f_spec.rb:78 # U2F#registration_requests returns an array of requests
Coverage report generated for RSpec to /home/eric/konklone/ruby-u2f/coverage. 192 / 192 LOC (100.0%) covered.
[Coveralls] Outside the Travis environment, not sending data.
/home/eric/.rbenv/versions/2.2.0/bin/ruby -I/home/eric/.rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/rspec-support-3.1.2/lib:/home/eric/.rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/rspec-core-3.1.7/lib /home/eric/.rbenv/versions/2.2.0/lib/ruby/gems/2.2.0/gems/rspec-core-3.1.7/exe/rspec --pattern spec/\*\*\{,/\*/\*\*\}/\*_spec.rb failed

This test explicitly requires the Ruby stdlib's securerandom library, which fixes the tests for me. It's not clear to me why Travis doesn't need this -- maybe because it uses SecureRandom itself.

(I should add that though the above was run with Ruby 2.2.0, I got an identical error with Ruby 2.1.5.)

Coverage remained the same when pulling a1f7588e979960738c181cbc951c950bf5130294 on konklone:errors into 9c2c6d7ade11bb62129b4325b084cb5859541587 on userbin:master.


fix bug. use f-string format

mistake occur in f-string in reading widerface.
mmdet/datasets/wider_face.py
see in detail.

@bailiqun
Thanks for the fix!

Hi @bailiqun ！First of all, we want to express our gratitude for your significant PR in the mmdetection project. Your contribution is highly appreciated, and we are grateful for your efforts in helping improve this open-source project during your personal time. We believe that many developers will benefit from your PR.
We would also like to invite you to join our Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us ：https://discord.gg/raweFPmdzG
If you are Chinese or have WeChat，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
Thank you again for your contribution❤


fix(number-input): add aria attributes for a11y

Closes #1743
Related: https://github.com/IBM/carbon-components-react/issues/1658
This PR adds aria-live and aria-atomic attributes to improve a11y

cc @dakahn to confirm 👍

@emyarod Could you please solve the merge conflict...? Thanks!

should be resolved now @asudoh

Oh @emyarod could you please also solve Handlebars errors seen in the Netlify build...? Thanks!

yes taking a look right now

a bad rebase broke the netlify build, it should be fixed now although I'm not entirely sure if I dropped any changes

Thanks @emyarod! The latest code looks good to me, let's see what @dakahn thinks.

👋 Hi! The error state should have a role="alert" with aria-atomic="true" so screen readers announce the value is invalid when an error is made. Other than that it looks good to me!