I get failed to reload

It starts tryng to load the lib even when still compiling packages (for example if I do cargo build --release) it start downloading things, then building and I get the error while it is still building.
Windows 10 here.

What errors are you seeing? Could you please include the errors generated here.

Well, I dont see errors (as a backtrace)
PS C:\Users\tyoc\Documents\GitHub\dynamic_reload> cargo run --example example  --release
   Compiling dynamic_reload v0.2.0 (file:///C:/Users/tyoc/Documents/GitHub/dynamic_reload)
    Finished release [optimized] target(s) in 2.16 secs
     Running `target\release\examples\example.exe`
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Value 4
Failed to reload

If you know how can I debug I would give a try on win...

And as I have said, it try to reload almost at the same time I hit cargo build --release or cargo build so it doesnt give time for the process to finish.

When I test now I actually get the same issue. I will try to look at this.

Maybe the "timestamp" of files is now changed before building? or something like that? if prior testing did not result on this "misses".

Might be. I have some code in there to handle if I can't load the file. Also I always copy the file to a separate directory and load it from there (you can't write an open dll on windows) I will try to investigate it but right now a bit busy but I will try to do it next week unless I get some free time this one.

@emoon if oyu provide a little direction I could try :)

Oh crap. I totally forgot about this. Let me get back to you.

Hi again,
So the way this library works:

Before loading a DLL (or sharedlib) it will make a copy of it
The copy of the dll will be loaded and the original file will be watched for changes.
If a change is detected in the original file the copy will be unloaded and a new copy of the original file will be done and reloaded.

The reason for this code is that Windows doesn't allow for modifying already loaded dlls which this works on macOS and Linux for example.
I have a bunch of code in there that should ensure that reload/copy of the file should work but there might be some cases that doesn't work. Like sometimes when you get a notification that a file has changed it's actually not possible to copy it yet. I have some code that tries to deal with this by checking we can read from it. If we can't we sleep for a while and try again with fixed number. If this still fails I just bail and say fail reload.
Hopefully this gives you an overview of how it works. If you have any more specific question I will love to help you out and I really appreciate that you want to take a look at this :)

@tyoc213 You could also try adding a delay to the UpdateState::Before handler to give the build system time to finish the build.

@emoon hmm.  It might be a safer to watch the .cargo-lock file.  When .cargo-lock is opened, only mark libraries for reload.  Then .cargo-lock is closed, reload marked libraries.

That won't work if you produce dynamic libs from somewhere else than cargo tho.

It could be added as an optional feature.  Also I think we could look for .cargo-lock in the same folder as the library.  When any .cargo-lock file is opened, delay reloading.

Yeah that might work but I would rather have the general case be reliable that this wouldn't be needed.

I think the general case would need to wait X seconds after the last detected change, before reloading.  While testing I have seen multiple events trigger a reload, even when using the Debounce watcher.  It doesn't seem to be easy to be 100% sure that the compiler has finished writing the library.

Yeah I guess so :/ It's annoying that there seems to be no reliable way to do this. Maybe wait, hash file, wait rehash to see if file has still changed may be a way forward?

watching .cargo-lock with the Debounce watcher doesn't work, since it is only opened/closed.  Would have to switch back to the raw watcher and do our own debouncing.

yes.  Hashing would be good.  Could do the hash check after shadow copying the lib.

Shadow copy.
hash copy.
wait 100ms.
hash original and compare.
Goto 1 if hash doesn't match?

Something like that yeah

Someone told me to look at https://github.com/fungos/cr for reference as they say it works there. Might be interesting to see if they have solved these problems in a good way there.

cr just checks timestamps to detect library changes.
One important feature from cr is that they version the library file each time they reload.  I was just adding timestamps to the library filename to fix some crashes I have been having on Linux.
Calling dlopen() with the same filename, doesn't seem to be reliable.  Using a unique name for each reload and not closing the old copy seems to fix the issue with TLS usage in the library.

I see

aah.  They rollback to the previous library version if the reload fails.  Also they validate the library file sections and reload data in the sections.
Seems it would be easier to just wrap cr.h in a crate and use that instead of libloading.

Yeah that might be a good idea

I might look into wrapping cr.h in a crate later.
For now, versioning the library copy on each reload, hash checks, and leaking/forgetting the previous instance would help a lot.

Indeed. Sounds good 👍

Any update on this? :)
Isn't it the case that this crate only gets notified by the OS when the dll file was closed after writing to it? In that case, why would debounce be necessary?
Or is it the case that this crate can only get notified every time data is written to the dll file, but not when it gets closed after being opened for writing?
Btw, this recent article goes into detail about the issue that the lib is not reloading due to the TLS destructor:
https://fasterthanli.me/articles/so-you-want-to-live-reload-rust

@Boscop I ran into those same TLS destructor issues over a year ago https://github.com/fungos/cr/issues/35 while making a Rust binding for cr.h: A Simple C Hot Reload Header-only Library.  See my comments on that issue.  If I remember correctly, the simplest solution is to alway use a different filename (rename/copy the SO file and add a version number before loading).  That way the if the old SO version is not released/unloaded because of TLS destructors, the new version can still be loaded.

I've made new tool that fixes thread-locals issue and more: https://github.com/xxshady/relib


Make "force90degLinear" handle any gradient type

Also, adjust parameter name from "string" to "color"

Thanks missed that one, feature live in 2.0.14


EKS Audit Log Collected by Security Lake Not Loaded

Is EKS audit log collected by Security Lake supported atm?
https://aws.amazon.com/about-aws/whats-new/2024/02/amazon-security-lake-audit-logs-eks/
I am trying to load EKS audit log from Security Lake S3 bucket to OpenSearch.
es-loader gets invoked. However, the log is not loaded to OpenSearch with the following messages.

/var/task/aws_lambda_powertools/metrics/provider/base.py:208: UserWarning: No application metrics to publish. The cold-start metric may be published if enabled. If application metrics should never be empty, consider using 'raise_on_empty_metrics'
self.flush_metrics(raise_on_empty_metrics=raise_on_empty_metrics)

Does anyone know how to load EKS audit logs to OpenSearch?
Can we use aes-es-loader Lambda function?


Stop running broken end-to-end mobile test

Description
As far as I can tell, this test is failing every time. Remove it so we can (sometimes) get green builds.
Tested
Not tested.

This is still broken, can we remove?


ValueError: Model is not loaded for type: run setup() before using predict()

Can you provide more details?


Fix set, but never read variable within MVKDevice::getFirstViewIndexInMetalPass.

A super basic cleanup, caught by enabling -Wunused-but-set-variable in clang.

Good catch! Thanks for submitting this!


Add extra attributes to inserted DOM element on Twitch

I'd like to be able to change the styling of the badge added in on Twitch but there currently isn't any selector that match it specifically.
What do you think to adding a data attribute like data-provider="pronoundb"? I can submit a PR adding it if you think it's a good idea.

I'm not opposed to adding ways to select PronounDB generated elements, I think adding a class name would be a bit better than a data attribute, easier to query and I don't think it'd cause any issues. Feel free to PR!

Cool I'll open a PR with a class!

Version 0.7.0 (with class names on elements (Twitch only)) is now available in AMO and will go live on other extension stores in the following days. 🎉

Thank you @cyyynthia !


Sample for azure-ai-documentintelligence fails with "NameError: name 'AnalyzeDocumentRequest' is not defined"

azure-ai-documentintelligence:
1.0.0b4:
Windows Server 2025 Build 26304.5000:
3.13.0:

Describe the bug
The python sample in https://learn.microsoft.com/en-us/training/modules/plan-form-recognizer-solution/3-resources?pivots=python fails with
"NameError: name 'AnalyzeDocumentRequest' is not defined"
To Reproduce
Steps to reproduce the behavior:

Install vanila Windows Server 2025
Python Releases for Windows | Python.org --> download and install python 3.13.0
curl https://bootstrap.pypa.io/get-pip.py -o get-pip.py
Add path to user variable PATH
pip install azure-ai-documentintelligence
Python IDLE shell, open a document and paste the python code

from azure.core.credentials import AzureKeyCredential
from azure.ai.documentintelligence import DocumentIntelligenceClient
from azure.ai.documentintelligence.models import AnalyzeResult

endpoint = "xxxxx"
key = "xxxxx"

docUrl = "https://www.ensi.ch/en/wp-content/uploads/sites/5/2024/09/Organigramm_E_01.10.2024.pdf"

document_analysis_client = DocumentIntelligenceClient(endpoint=endpoint,
    credential=AzureKeyCredential(key))

poller = document_analysis_client.begin_analyze_document("prebuilt-layout", AnalyzeDocumentRequest(url_source=docUrl))
result: AnalyzeResult = poller.result()

Accordingly to https://github.com/Azure/azure-sdk-for-python/blob/azure-ai-documentintelligence_1.0.0b4/sdk/documentintelligence/azure-ai-documentintelligence/MIGRATION_GUIDE.md, the function has been migrated.
Expected behavior
Sample runs smoothly.
Screenshots

Hi @dcasota , thanks for your contact!
You didn't import AnalyzeDocumentRequest, please add from azure.ai.documentintelligence.models import AnalyzeDocumentRequest and try running again.

@YalinLi0312 Thank you. Yes, that function import was missing. I didn't start thinking and simply processed the docs' content instead of browsing the class. Shame on me.
from azure.core.credentials import AzureKeyCredential
from azure.ai.documentintelligence import DocumentIntelligenceClient
from azure.ai.documentintelligence.models import AnalyzeResult
from azure.ai.documentintelligence.models import AnalyzeDocumentRequest

endpoint = "<your endpoint>"
key = "<your key>"

docUrl = "<your url>"

document_analysis_client = DocumentIntelligenceClient(endpoint=endpoint,
    credential=AzureKeyCredential(key))

poller = document_analysis_client.begin_analyze_document("prebuilt-layout", AnalyzeDocumentRequest(url_source=docUrl))
result: AnalyzeResult = poller.result()


Update trace datastore test to use datastore module

The google-cloud module is not recommended for interacting with Google
APIs and the version we were using in our tests required a version of
gRPC that could not be compiled with Node 8 on windows.

Done.


loadCurrentSession when returning to an installed app

Issue summary
Shopify.Auth.loadCurrentSession() fails to load a session when opening an already installed app.
Expected behavior
An offline session should be retrieved.
Actual behavior
No session is found.
Steps to reproduce the problem

Install the app using the steps described in the documentation for this library while using CustomSessionStorage (beginAuth, validateAuthCallback, save offline session to DB, etc.). All this happens successfully, I have a record in the DB for offline_mystore.myshopify.com with accessToken, etc.
On the shop, go back to the list of Apps and open the recently installed app again.
The loadCurrentSession function on my app entry route fails to find an active session, so instead it overwrites the session entry in my DB with a temporary one (without the access token).

The query parameters for the app entry route when opening the app for the first time (to install) are hmac, shop and timestamp. When opening the app the second time (after it has been installed), the parameters are hmac, host, locale, session, shop and timestamp. And the call to load the session is const session = await Shopify.Utils.loadCurrentSession(req, res, false), i.e. I want to find an offline session.
I could, of course, try to query my DB for 'offline_' + req.query.shop, but this feels very hacky...
Perhaps I'm just misunderstanding how sessions need to be looked up?

@artooras where do you get shop from tho?
Do you always pass it in the URL and do ctx.query.shop ? 🤔


Implement the closed kinematic simulator block for handling any number of closed chain kinematics

In the commit d27c4a077706011f7b4523581b76a70bfbf62630, I implemented the simulator block that handle the closed chain dynamics having two chains. In this issue, we want to generalize it for the dynamics with any number of the closed-chains.

It is fixed in #58


Example Metadata Entry Tool entries to showcase FAIR & CARE

There are already lots of scored items in that demo cat but you try to aim for ranges of scores, you will put the tool through pretty good paces, and have specific examples to show it.
A range of metadata examples
a) we need to update our demo catalogue, now that we have scores calculable via the Metadata Entry Tool and our Python Scores Calculator tool too
b) we need a range of high, medium & low FAIR & CARE scoring resources
c) make up clearly dummy researchers/examples to achieve this

13 May - just waiting on Jamie to update FAIR and CARE scores this week & for catalogue
22 Apr - Create Metadata Entry Tool examples using items in our demo catalogue at https://data.idnau.org/c/catalogs/pd:democat/resources and try and create a High, Medium & Low FAIR & CARE resource.

Currently waiting on the completion of the CARE and FAIR scoring

Need to discuss with Nick

Improved (?) existing metadata record:
,,,
@prefix dcat: http://www.w3.org/ns/dcat#.
@prefix dcterms: http://purl.org/dc/terms/.
@prefix geo: http://www.opengis.net/ont/geosparql#.
@prefix prov: http://www.w3.org/ns/prov#.
@prefix rdf: http://www.w3.org/1999/02/22-rdf-syntax-ns#.
@prefix rdfs: http://www.w3.org/2000/01/rdf-schema#.
@prefix role: https://w3id.org/idn/def/idn-role-codes/.
@prefix sdo: https://schema.org/.
@prefix xsd: http://www.w3.org/2001/XMLSchema#.
https://data.idnau.org/pid/AUSLANG a dcat:Resource;
dcterms:title "AustLang Database";
dcterms:type https://data.idnau.org/pid/vocab/indigeneity/about-indigenous-people, https://data.idnau.org/pid/vocab/indigeneity/about-indigenous-things;
dcterms:description "AustLang provides a controlled vocabulary of persistent identifiers, a thesaurus of languages and peoples and information about Aboriginal and Torres Strait Islander languages which has been assembled from referenced sources.\nThe alpha-numeric codes function as persistent identifiers, followed by a 'string of changeable text'. This allows changes to the name or spelling of a language variety, according to community preference. In cases where there is more than one preference, two or three versions of the name are included, e.g. E6: Dhanggati / Dunghutti^. \nThis vocabulary of persistent identifiers supports archives, libraries, galleries and other agencies to identify materials or projects in or about Indigenous Australian languages and peoples, without the confusion of a multitude of language names and spellings. The codes maintain an identity if a change is made to the spelling or the name.\nAustLang can be searched with language names (including a range of spellings); the codes, for example E6; placenames and via the map. AustLang has links to MURA the AIATSIS catalogue and other online resources.";
prov:qualifiedAttribution _:b7;
dcat:distribution _:b11;
dcterms:spatial _:b9;
dcterms:temporal _:b12;
dcterms:license http://purl.org/NET/rdflicense/cc-by4.0;
dcterms:rights "© The Australian Institute of Aboriginal and Torres Strait Islander Studies";
dcterms:accessRights https://linked.data.gov.au/def/data-access-rights/conditional;
dcat:theme https://vocabularyserver.com/apais/xml.php?skosTema=8, https://vocabularyserver.com/apais/xml.php?skosTema=15;
dcterms:created "1990-01-01"^^xsd:date;
dcterms:modified "2019-01-01"^^xsd:date;
dcterms:issued "2008-01-01"^^xsd:date.
_:b7 prov:agent https://data.idnau.org/pid/org/aiatsis;
dcat:hadRole https://linked.data.gov.au/def/data-roles/author, https://linked.data.gov.au/def/data-roles/custodian, https://linked.data.gov.au/def/data-roles/pointOfContact, https://linked.data.gov.au/def/data-roles/publisher.
_:b9 a geo:Geometry;
geo:asWKT "POLYGON ((96.0 -45.0, 168.0 -45.0, 168.0 -9.0, 96.0 -9.0, 96.0 -45.0))"^^geo:wktLiteral.
_:b11 dcat:accessURL "https://data.gov.au/data/dataset/70132e6f-259c-4e0f-9f95-4aed1101c053/resource/e9a9ea06-d821-4b53-a05f-877409a1a19c/download/aiatsis_austlang_endpoint_001.csv"^^xsd:anyURI.
_:b12 prov:startedAtTime "2008-01-01"^^xsd:date;
prov:endedAtTime "2024-12-31"^^xsd:date.
,,,


Terraform script to add provisioning on GCP, DO and AWS

As part of #2647, we need a way to provision servers on all cloud providers, e.g.:

Google Cloud Platform
Digital Ocean
Amazon Web Services

Terraform is a declarative way to do so and can therefore be convenient here.

Closing, as change is part of: weaveworks/build-tools/pull/53.


[WebGL] - GL_INVALID_OPERATION: Insufficient buffer size.

Godot version
4.0.stable
System information
WebGL, Windows 11
Issue description
Some .glb (.fbx and .dae too) mesh don't show after a certain distance
Scene on web - WebGL:

Scene running on Desktop - OpenGL

Error:

Steps to reproduce

Create project
Add .glb to Scene
Add Camera3D
Play on Web

Minimal reproduction project
RenderExample.zip

As a workaround, select the glTF scene in the FileSystem dock and disable Generate Lods in the Import dock.

This works, thanks

I think there needs to be a global disable for lod and be set on by default for the web.


Invalid QR code

Looks like he might have entered a title with invalid characters or something.

Actually it looks like the amount is displaying incorrectly. 0.01630000000000000000002 btc

Oh that's my end calculating the amount incorrectly. Doh


Usage Documentation

Any documentation on how to use this repo would be great.
I am trying to do a simple clone/redeploy from my existing Synapse Workspace to a new Synapse Workspace, this repo seems to fit but there is minimal documentation.

Hello
We have a github action available on the marketplace. You can directly use that instead of using this repository.
https://github.com/marketplace/actions/synapse-workspace-deployment
Please let me know if any additional help is required.


bug in v0.3.0 hauler version doesn't correctly specify version of hauler

Environmental Info:
linux/amd64
Hauler Version:
0.3.0
System CPU architecture, OS, and Version:
OS = Centos 7
uname -a
Linux vm1 3.10.0-1160.59.1.el7.x86_64 #1 SMP Wed Feb 23 16:47:03
UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Describe the bug:
hauler version doesn't correctly specify the version of hauler
Steps To Reproduce:
On a x86_x64 based linux machine copy paste the following 4 commands:
curl -L https://github.com/rancherfederal/hauler/releases/download/v0.3.0/hauler_0.3.0_linux_amd64.tar.gz > hauler.tar.gz
tar -xzf hauler.tar.gz
sudo mv hauler /usr/bin/hauler && rm README.md
hauler version

Expected behavior:
expect to see version 0.3.0 specified in the output
Actual behavior:
hauler version
GitVersion:  devel
GitCommit:   unknown
BuildDate:   unknown
GoVersion:   go1.17.6
Compiler:    gc
Platform:    linux/amd64

Bumping this issue. hauler version still does not show all information and does not show the Hauler Version.
zackbradys@Zacks-MacBook-Pro bin % hauler version
GitVersion:  devel
GitCommit:   unknown
BuildDate:   unknown
GoVersion:   go1.20.5
Compiler:    gc
Platform:    darwin/arm64

fixed in PR #136!


[Modal] Request for passive modal

I would like to use a passive Modal component without an ok or cancel button.

:tada: This issue has been resolved in version 1.6.5 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


TFlite on xtensa lx7

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
tf 2.16
Custom code
Yes
OS platform and distribution
Linux, ubuntu 22.04
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
Hi,
I need to compile tensorflow lite on xtensa lx7 platform. Currently, I used cross compilation information from tflite documentation but build errors (file not found errors, undefined references, syntax errors) are generated.
Please provide instruction on the build steps.
Standalone code to reproduce the issue
cross compilation :
cmake   -DCMAKE_C_COMPILER=${XTENSA_PREFIX}/clang   -DCMAKE_CXX_COMPILER=${XTENSA_PREFIX}/clang++   -DCMAKE_C_FLAGS="${XTENSA_FLAGS}"   -DCMAKE_CXX_FLAGS="${XTENSA_FLAGS}"   -DCMAKE_VERBOSE_MAKEFILE:BOOL=ON   -DCMAKE_SYSTEM_NAME=Linux   -DCMAKE_SYSTEM_PROCESSOR=xtensa   -DCMAKE_THREAD_LIBS_INIT="-lpthread"   -DCMAKE_HAVE_THREADS_LIBRARY=1   -DCMAKE_USE_PTHREADS_INIT=1   -DTHREADS_PREFER_PTHREAD_FLAG=ON
../tensorflow_src/tensorflow/lite

build:
cmake --build . -j

Relevant log output
No response

Hi @devapriyas2001 ,
The error you are facing is due to compatibility issue between EDK version of your device and TFLite. Could you please make sure the EDK version is compatible with the TFLite version you're targeting and the build includes the necessary header files in the path from Xtensa libraries and TFLite. For faster resolution could you please post this issue on tflite/micro.
Thank You!

Hi @Venkat6871 ,
How to find the compatible EDK version for the device and TFlite?
I posted the query on https://github.com/tensorflow/tflite-micro but didn't get the desired response.
Thanks!

Hi @devapriyas2001, do you have a link to your query & response so that we may gain better context on what the issue may be?


fix(amplify-util-mock): add defaults for streamViewType

Description of changes

Use sane defaults for StreamSpecification in CFN parser if there exist any custom DDB tables
Close the simulators etc, in case of any error in starting up the API mock process and exit.

Issue #, if available

https://github.com/aws-amplify/amplify-category-api/issues/624
Description of how you validated changes
unit test added
Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #10726 (22cb312) into dev (a189648) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##              dev   #10726      +/-   ##
==========================================
- Coverage   47.36%   47.35%   -0.01%
==========================================
  Files         669      669
  Lines       33071    33075       +4
  Branches     6674     6676       +2
==========================================
  Hits        15663    15663
- Misses      15727    15731       +4
  Partials     1681     1681

Impacted Files
Coverage Δ

...-mock/src/CFNParser/resource-processors/appsync.ts
0.00% <0.00%> (ø)

packages/amplify-util-mock/src/api/api.ts
0.00% <0.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Web support

is there another package for web support?

This package targets Windows only.
Please check this package. It probably meets your application and is of higher quality. (The referenced project appears to be developed as part of a commercial use.
https://pub.dev/packages/desktop_drop


Added aliases for most commands

Added shorthard keywords for all commands except help. Exit has shorthand "x", while all other commands have aliases corresponding to the first letter of the command

Changes Unknown when pulling 66866e50d5e8e6f0933f9422d776ce7d2d4df9d6 on zacharytang:add_aliases into ** on CS2103T-W09B1:master**.


"OnSuccess" callback is still invoked when there is not camera

When failed to request device we should inform the right status to content,
since there might have some JavaScript listening to the result,  neither success or error.
BUG=https://crosswalk-project.org/jira/browse/XWALK-3133

@pozdnyakov @shaochangbin @rakuco @kenchris
Please spare some time to have a look.

lgtm

Please remember to backport to crosswalk-11.


feat: allow passing custom descriptions to queries and mutations

Close #137
Contributor checklist

[ ] Bug fixes include regression tests
[x] Features include unit/acceptance tests

🚀 Thank you for your contribution! 🚀


fix(release): add packaging dep

Codecov Report

Merging #105 (8e74a64) into main (b35fb96) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #105   +/-   ##
=======================================
  Coverage   88.43%   88.43%
=======================================
  Files          14       14
  Lines         536      536
=======================================
  Hits          474      474
  Misses         62       62

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 b35fb96...8e74a64. Read the comment docs.


QrInputTooLongException

I need to add a longer string, but I get this: [QR] ERROR - QrInputTooLongException: Input too long. 1116 > 640.
Is it a way to fix this?

Hi @silviucs,
This input length cap is determined by which version QR code you are using. You can read more about QR versions here: https://www.qrcode.com/en/about/version.html
Try increasing your QR version by manually specifying like this:
QrImage(version=25, data: _dataString)
The link above also explains how to find the best version# based on the length of your data.

Thanks @rrisque!

Thank you sir, it's working perfectly ;)


dot.c early bail fix

The dot kernel should return 0 for N=0. An existing early bail checks for N<0 and returns 0 in this case. Adjust this to N<1 to simplify maintenance: with this change we don't have to reason through subsequent code to confirm it handles that corner case correctly, and future platform ports have a slightly more pleasant baseline to begin from.

ah, sorry, that would be against the main develop branch (not sure if/how I could redirect the merge there ?)

Gotcha, I'll close this and make a new one. One sec


mmj2: init at 2.5.2-unstable-2023-06-27

Description of changes

Add mmj2 package. This can be run with ./result/bin/mmj2 -d set.mm, where set.mm is the file https://github.com/metamath/set.mm/blob/develop/set.mm.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] sandbox = true

[x] 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/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Looks like you didn't test your changes :p by changing the version you broke it when you are copying the jar file.
Other stuff:

No runHook in installPhase
You didn't set meta.maintainers, please add yourself as a maintainer and then set that field
Did you really test this package on all platforms ? Otherwise just specify the platform on which you tested in meta.platforms

Here's a patch to make it work and with runHooks:
diff --git a/pkgs/by-name/mm/mmj2/package.nix b/pkgs/by-name/mm/mmj2/package.nix
index f851baa33d94..65f60af716fd 100644
--- a/pkgs/by-name/mm/mmj2/package.nix
+++ b/pkgs/by-name/mm/mmj2/package.nix
@@ -1,11 +1,13 @@
 { lib, maven, fetchFromGitHub, jre, makeWrapper }:

-maven.buildMavenPackage {
+let
+  lastVersion = "2.5.2";
+in maven.buildMavenPackage {
   pname = "mmj2";

   # The latest stable version is from 2017 and doesn't include the mmj2jar/mmj2
   # wrapper script, so use the unstable one for now
-  version = "2.5.2-unstable-2023-06-27";
+  version = "${lastVersion}-unstable-2023-06-27";

   src = fetchFromGitHub {
     owner = "digama0";
@@ -20,13 +22,17 @@ maven.buildMavenPackage {
   nativeBuildInputs = [ makeWrapper ];

   installPhase = ''
+    runHook preInstall
+
     local jar=$out/share/$pname/$pname.jar
     local bin=$out/bin/$pname
-    install -Dm644 target/$pname-$version-SNAPSHOT-jar-with-dependencies.jar $jar
+    install -Dm644 target/$pname-${lastVersion}-SNAPSHOT-jar-with-dependencies.jar $jar
     install -Dm744 mmj2jar/$pname $bin
     wrapProgram $bin \
       --set MMJ2_JAR $jar \
       --set JAVA ${jre}/bin/java
+
+    runHook postInstall
   '';

   meta = with lib; {

Thanks @TheBrainScrambler, I fixed all the problems.

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

1 package built:

mmj2

@ofborg build mmj2


okey new mess...

on every android build!

okey this is not yours mess it guy's mess

ok it not yours mistake it guy's mistake


Nick: Watch atlmoneytransfer.com

Nick requests the watch of the watch_keyword atlmoneytransfer\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

atlmoneytransfer\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Ryan M in Charcoal HQ


Adding subscript and superscripts in text for writing formulas

Adding subscripts and superscripts is essential for writing mathematical formulas, chemical equations, and scientific notations.

Can you please give some clear information about your proposed idea?

sir could you please assign this to me so that i can contribute in adding features like subscripts and superscripts for writing formulas

The similar issue is already assigned.
Refer issue #22
You may create another issue

I would like to solve this issue

@vishanurag I wanted to add subscript and superscript like for chemical formal H2O (2 is subscript) and 2^2 that square power is superscript

@aditiverma-21  Can you please! assign me to work on this issue.


Cannot reuse a MaterialPageRoute after disposing it.

Steps to Reproduce

Video: https://i.imgur.com/A85IETe.mp4

Install Flutter Gallery on iOS simulator
Open and close any sample using the swipe mechanism
After a few successful swipes pick up the pace of opening and closing the sample
Crashes

Logs

flutter: ══╡ EXCEPTION CAUGHT BY ANIMATION LIBRARY ╞═════════════════════════════════════════════════════════
flutter: The following assertion was thrown while notifying status listeners for AnimationController:
flutter: Cannot reuse a MaterialPageRoute<dynamic> after disposing it.
flutter: 'package:flutter/src/widgets/routes.dart': Failed assertion: line 218 pos 12:
flutter: '!_transitionCompleter.isCompleted'
flutter:
flutter: Either the assertion indicates an error in the framework itself, or we should provide substantially
flutter: more information in this error message to help you determine and fix the underlying cause.
flutter: In either case, please report this assertion by filing a bug on GitHub:
flutter:   https://github.com/flutter/flutter/issues/new?template=BUG.md
flutter:
flutter: When the exception was thrown, this was the stack:
flutter: #2      TransitionRoute.didPopNext (package:flutter/src/widgets/routes.dart:218:12)
flutter: #3      NavigatorState.pop (package:flutter/src/widgets/navigator.dart:1944:23)
flutter: #4      _CupertinoBackGestureController._handleStatusChanged (package:flutter/src/cupertino/route.dart:651:17)
flutter: #5      _AnimationController&Animation&AnimationEagerListenerMixin&AnimationLocalListenersMixin&AnimationLocalStatusListenersMixin.notifyStatusListeners (package:flutter/src/animation/listener_helpers.dart:192:19)
flutter: #6      AnimationController._checkStatusChanged (package:flutter/src/animation/animation_controller.dart:736:7)
flutter: #7      AnimationController._tick (package:flutter/src/animation/animation_controller.dart:752:5)
flutter: #8      Ticker._tick (package:flutter/src/scheduler/ticker.dart:228:5)
flutter: #9      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:990:15)
flutter: #10     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding.handleBeginFrame.<anonymous closure> (package:flutter/src/scheduler/binding.dart:906:11)
flutter: #11     __InternalLinkedHashMap&_HashVMBase&MapMixin&_LinkedHashMapMixin.forEach (dart:collection/runtime/libcompact_hash.dart:367:8)
flutter: #12     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding.handleBeginFrame (package:flutter/src/scheduler/binding.dart:904:17)
flutter: #13     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding._handleBeginFrame (package:flutter/src/scheduler/binding.dart:834:5)
flutter: #17     _invoke1 (dart:ui/hooks.dart:203:10)
flutter: #18     _beginFrame (dart:ui/hooks.dart:156:3)
flutter: (elided 5 frames from class _AssertionError and package dart:async)
flutter:
flutter: The AnimationController notifying status listeners was:
flutter:   AnimationController#f0974(⏮ 0.000; paused; DISPOSED; for
flutter:   MaterialPageRoute<dynamic>(/cupertino/alert))
flutter: ════════════════════════════════════════════════════════════════════════════════════════════════════
flutter: Another exception was thrown: 'package:flutter/src/widgets/navigator.dart': Failed assertion: line 1507 pos 12: '!_debugLocked': is not true.
flutter: Another exception was thrown: 'package:flutter/src/widgets/navigator.dart': Failed assertion: line 1507 pos 12: '!_debugLocked': is not true.
flutter: Another exception was thrown: 'package:flutter/src/widgets/navigator.dart': Failed assertion: line 1507 pos 12: '!_debugLocked': is not true.
flutter: Another exception was thrown: 'package:flutter/src/widgets/navigator.dart': Failed assertion: line 1507 pos 12: '!_debugLocked': is not true.
flutter: Another exception was thrown: 'package:flutter/src/widgets/navigator.dart': Failed assertion: line 1507 pos 12: '!_debugLocked': is not true.
flutter: Another exception was thrown: 'package:flutter/src/widgets/navigator.dart': Failed assertion: line 1507 pos 12: '!_debugLocked': is not true.
flutter: Another exception was thrown: 'package:flutter/src/widgets/navigator.dart': Failed assertion: line 1507 pos 12: '!_debugLocked': is not true.
flutter: Another exception was thrown: 'package:flutter/src/widgets/navigator.dart': Failed assertion: line 1507 pos 12: '!_debugLocked': is not true.
flutter: Another exception was thrown: 'package:flutter/src/widgets/navigator.dart': Failed assertion: line 1507 pos 12: '!_debugLocked': is not true.

Analyzing flutter_gallery...
No issues found! (ran in 5.6s)

[✓] Flutter (Channel master, v1.2.1-pre.50, on Mac OS X 10.14.2, locale en-US)
    • Flutter version 1.2.1-pre.50 at /Users/droidery/flutter
    • Framework revision dbd1b317ff (76 minutes ago), 2019-01-30 21:31:44 -0500
    • Engine revision 15f2b92cce
    • Dart version 2.1.1 (build 2.1.1-dev.3.2 71bee8f05e)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/droidery/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
    • All Android licenses accepted.

[!] iOS toolchain - develop for iOS devices (Xcode 10.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.1, Build version 10B61
    ✗ Verify that all connected devices have been paired with this computer in Xcode.
      If all devices have been paired, libimobiledevice and ideviceinstaller may require updating.
      To update with Brew, run:
        brew update
        brew uninstall --ignore-dependencies libimobiledevice
        brew uninstall --ignore-dependencies usbmuxd
        brew install --HEAD usbmuxd
        brew unlink usbmuxd
        brew link usbmuxd
        brew install --HEAD libimobiledevice
        brew install ideviceinstaller
    • ios-deploy 1.9.4
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 31.3.3
    • Dart plugin version 182.5124
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[✓] IntelliJ IDEA Community Edition (version 2018.3.4)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 32.0.3
    • Dart plugin version 183.5429.25

[✓] VS Code (version 1.30.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 2.22.2

[✓] Connected device (1 available)
    • iPhone SE • 58F34A3E-1576-453E-B1B4-3DCB22E1FEDD • ios • iOS 12.1 (simulator)

Might be related to https://github.com/flutter/flutter/issues/27333 but error message and reproduction steps are difference.

I am also getting this. I navigate user from Login to Profile if the user have been already logged in by     Navigator.pushReplacementNamed(context, "/profile");. It correctly redirects to login, but when I Navigate back with Navigator.pushReplacementNamed(context, "/login"); and then try to navigate back to profile after logging in, I get this error
[VERBOSE-2:shell.cc(186)] Dart Error: Unhandled exception:
'package:flutter/src/widgets/routes.dart': Failed assertion: line 226 pos 12: '!_transitionCompleter.isCompleted': Cannot reuse a MaterialPageRoute<dynamic> after disposing it.
#0      _AssertionError._doThrowNew (dart:core/runtime/liberrors_patch.dart:40:39)
#1      _AssertionError._throwNew (dart:core/runtime/liberrors_patch.dart:36:5)
#2      TransitionRoute.didChangeNext (package:flutter/src/widgets/routes.dart:226:12)
#3      NavigatorState.pushReplacement (package:flutter/src/widgets/navigator.dart:1631:27)
#4      NavigatorState.pushReplacementNamed (package:flutter/src/widgets/navigator.dart:1494:12)
#5      Navigator.pushReplacementNamed (package:flutter/src/widgets/navigator.dart:788:34)
#6      __LoginScreenState&State&BaseState.redirectToProfile (package:mobile/main.dart:59:15)
#7      _LoginScreenState.doLogin.<anonymous closure> (package:mobile/Screens/login.dart:69:9)
#8      Api.login (package:mobile/Api.dart:34:11)
<asynchron<…>
flutter: ══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
flutter: The following assertion was thrown building Navigator-[GlobalObjectKey<NavigatorState>
flutter: _WidgetsAppState#3e2b3](dirty, state: NavigatorState#0518a(tickers: tracking 2 tickers)):
flutter: 'package:flutter/src/widgets/navigator.dart': Failed assertion: line 1995 pos 12: '!_debugLocked':
flutter: is not true.
flutter:
flutter: Either the assertion indicates an error in the framework itself, or we should provide substantially
flutter: more information in this error message to help you determine and fix the underlying cause.
flutter: In either case, please report this assertion by filing a bug on GitHub:
flutter:   https://github.com/flutter/flutter/issues/new?template=BUG.md
flutter:
flutter: When the exception was thrown, this was the stack:
flutter: #2      NavigatorState.build (package:flutter/src/widgets/navigator.dart:1995:12)
flutter: #3      StatefulElement.build (package:flutter/src/widgets/framework.dart:3809:27)
flutter: #4      ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3721:15)
flutter: #5      Element.rebuild (package:flutter/src/widgets/framework.dart:3547:5)
flutter: #6      StatefulElement.update (package:flutter/src/widgets/framework.dart:3878:5)
flutter: #7      Element.updateChild (package:flutter/src/widgets/framework.dart:2742:15)
flutter: #8      ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3732:16)
flutter: #9      Element.rebuild (package:flutter/src/widgets/framework.dart:3547:5)
flutter: #10     ProxyElement.update (package:flutter/src/widgets/framework.dart:3990:5)
flutter: #11     Element.updateChild (package:flutter/src/widgets/framework.dart:2742:15)
flutter: #12     SingleChildRenderObjectElement.update (package:flutter/src/widgets/framework.dart:4867:14)
flutter: #13     Element.updateChild (package:flutter/src/widgets/framework.dart:2742:15)
flutter: #14     ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3732:16)
flutter: #15     Element.rebuild (package:flutter/src/widgets/framework.dart:3547:5)
flutter: #16     StatelessElement.update (package:flutter/src/widgets/framework.dart:3781:5)
flutter: #17     Element.updateChild (package:flutter/src/widgets/framework.dart:2742:15)
flutter: #18     ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3732:16)
flutter: #19     Element.rebuild (package:flutter/src/widgets/framework.dart:3547:5)
flutter: #20     StatelessElement.update (package:flutter/src/widgets/framework.dart:3781:5)
flutter: #21     Element.updateChild (package:flutter/src/widgets/framework.dart:2742:15)
flutter: #22     ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3732:16)
flutter: #23     Element.rebuild (package:flutter/src/widgets/framework.dart:3547:5)
flutter: #24     StatelessElement.update (package:flutter/src/widgets/framework.dart:3781:5)
flutter: #25     Element.updateChild (package:flutter/src/widgets/framework.dart:2742:15)
flutter: #26     ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3732:16)
flutter: #27     Element.rebuild (package:flutter/src/widgets/framework.dart:3547:5)
flutter: #28     ProxyElement.update (package:flutter/src/widgets/framework.dart:3990:5)
flutter: #29     Element.updateChild (package:flutter/src/widgets/framework.dart:2742:15)
flutter: #30     ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3732:16)
flutter: #31     Element.rebuild (package:flutter/src/widgets/framework.dart:3547:5)
flutter: #32     ProxyElement.update (package:flutter/src/widgets/framework.dart:3990:5)
flutter: #33     Element.updateChild (package:flutter/src/widgets/framework.dart:2742:15)
flutter: #34     SingleChildRenderObjectElement.update (package:flutter/src/widgets/framework.dart:4867:14)
flutter: #35     Element.updateChild (package:flutter/src/widgets/framework.dart:2742:15)
flutter: #36     ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3732:16)
flutter: #37     Element.rebuild (package:flutter/src/widgets/framework.dart:3547:5)
flutter: #38     StatefulElement.update (package:flutter/src/widgets/framework.dart:3878:5)
flutter: #39     Element.updateChild (package:flutter/src/widgets/framework.dart:2742:15)
flutter: #40     ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3732:16)
flutter: #41     Element.rebuild (package:flutter/src/widgets/framework.dart:3547:5)
flutter: #42     ProxyElement.update (package:flutter/src/widgets/framework.dart:3990:5)
flutter: #43     Element.updateChild (package:flutter/src/widgets/framework.dart:2742:15)
flutter: #44     ComponentElement.performRebuild (package:flutter/src/widgets/framework.dart:3732:16)
flutter: #45     Element.rebuild (package:flutter/src/widgets/framework.dart:3547:5)
flutter: #46     BuildOwner.buildScope (package:flutter/src/widgets/framework.dart:2286:33)
flutter: #47     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding&PaintingBinding&SemanticsBinding&RendererBinding&WidgetsBinding.drawFrame (package:flutter/src/widgets/binding.dart:685:20)
flutter: #48     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding&PaintingBinding&SemanticsBinding&RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:219:5)
flutter: #49     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:990:15)
flutter: #50     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:930:9)
flutter: #51     _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding._handleDrawFrame (package:flutter/src/scheduler/binding.dart:842:5)
flutter: #52     _invoke (dart:ui/hooks.dart:159:13)
flutter: #53     _drawFrame (dart:ui/hooks.dart:148:3)
flutter: (elided 2 frames from class _AssertionError)
flutter: ════════════════════════════════════════════════════════════════════════════════════════════════════
flutter: Another exception was thrown: setState() called after dispose(): OverlayState#b7857(lifecycle state: defunct, not mounted, tickers: tracking 6 tickers, entries: [OverlayEntry#7a6bc(opaque: true; maintainState: false), OverlayEntry#9dc6f(opaque: false; maintainState: true), OverlayEntry#fba79(opaque: true; maintainState: false), OverlayEntry#34189(opaque: false; maintainState: true)])
flutter: getProfile ok
flutter: Martin
flutter:
[VERBOSE-2:shell.cc(186)] Dart Error: Unhandled exception:
setState() called after dispose(): ProfileScreenState#ac0c4(lifecycle state: defunct, not mounted)
This error happens if you call setState() on a State object for a widget that no longer appears in the widget tree (e.g., whose parent widget no longer includes the widget in its build). This error can occur when code calls setState() from a timer or an animation callback. The preferred solution is to cancel the timer or stop listening to the animation in the dispose() callback. Another solution is to check the "mounted" property of this object before calling setState() to ensure the object is still in the tree.
This error might indicate a memory leak if setState() is being called because another object is retaining a reference to this State object after it has been removed from the tree. To avoid memory leaks, consider breaking the reference to this object during dispose().
#0      State.setState.<anonymous closure> (package:flutter/src/widgets<…>

And just fixed.
In my logout routine from profile, I have had called Navigator.popAndPushNamed(context, "/login"); instead of Navigator.pushReplacementNamed(context, "/login");.
When I changed it to the second one, it started to working. Could somebody elaborate why is it the case and what are the differences between these two? @zoechi @Hixie ?

To investigate we would need a minimal runnable reproduction
as a single file so that we can just copy your code into lib/main.dart of a new project and run to reproduce.

Reminder for future: Matt hypothesizes that the swipe dismiss animation may still be ongoing after a next route has been pushed by tapping the next demo.

I am also getting this. I'm using CupertinoPageRoute. If I swipe the previous page, and click quickly to open another page, the error occurs.
Every time, I should spend 300 or 400ms to wait for the previous page animation finish. As @xster said.
But how to solve this?

I finally get a working code:
// this is a global variable, used to wait before page route animation finish.
Completer page_transition_promise;

Future openPage(context, Widget builder) async {
  if (page_transition_promise != null && !page_transition_promise.isCompleted) {
    await page_transition_promise.future;
  }
  page_transition_promise = Completer();

  final result = await Navigator.of(context).push(
    CupertinoPageRoute(builder: (ctx) => builder),
  );
  page_transition_promise.complete(result);
  return result;
}

The sliding is not finished, but you can click back or other at the same time.

@TomLiu-GitHub I filed a separate bug for it. #30416

This bug not been noticed. I think it's a very serious bug.

I use Future.dealyed solved this problem
Future.delayed(Duration(seconds: 0), () {
    ...push....
  });


[7.x] Make parsedown swappable

This PR introduces the "illuminate/markdown" component. It should "just work". No configuration needed. As long as some markdown package is installed, it should find it, and use it.
This design is super easy to configure now. Previosuly it was impossible to configure the parsedown renderer, or use a different markdown implementation. Now it is super easy to do. Simply bind to the Markdown interface, and you're done!
By default, we support league/commonmark, erusev/parsedown, michelf/php-markdown, out of the box, and we search for them in that order.

I think there are two sane choices for the default parser. Either we:

stay with parsedown (https://github.com/laravel/laravel/pull/5192),
or we switch to commonmark (https://github.com/laravel/laravel/pull/5193).

We do this simply by means of what we include by default in new apps - the framework itself has no preference, other than the order it looks for parsers out of the box.

So, we're now super flexible, and everything still works without needing to install extra dependencies on apps based from laravel/laravel due to us including a parser in "require".

What was some of the reasoning for this? Is Parsedown not a very good default?

Isn't also the point of a specification like Markdown that you don't need to support multiple drivers because they all do the same basically?

Isn't also the point of a specification like Markdown that you don't need to support multiple drivers because they all do the same basically?

Unfortunately, markdown is not a specification. It's the name given to a family of languages. For example, GitHub flavoured markdown was historically popular, until the CommonMark specification was developed. Only the league CommonMark library is actually following this specification. Parsedown has no specification, and seems to make breaking changes to parsing in minor releases.

Is Parsedown not a very good default?

So, yeh, as I said above, Parsedown doesn't actually have a spec, and is not stable between versions.

What was some of the reasoning for this?

The main reasoning for this is that it's totally impossible (without replacing the framework's view files) to customize the markdown parser. After this PR, it's now super easy to do this, since we resolve the markdown contract from the ioc container.
I think the league CommonMark package is the best default, and it's actually more used that Parsedown. This is because most of Parsedown's downloads are coming from laravel/framework or illuminate/mail downloads. There were 2156952 downloads of laravel/framework or illuminate/mail 5.4, 5.5, 5.6, 5.7, 5.8 and 6.x in the last 30 days, and 2165578 parsedown downloads. This means only 8626 people intentionally downloaded Parsedown in the last 30 days, compared to league's commonmark, which had 96691 downloads in the last 30 days.
Note also that my graham-campbell/markdown package had 39048 downloads in the last 30 days, and it brings commonmark into laravel, and makes it easy to customize, so that shows that there is demand for it. 11x as many people downloaded commonmark for laravel (using my package alone) than those downloading parsedown to use outside of laravel!

Finer details for calculating the laravel downloads over the last 30 days:
Laravel 5.4: 98410  + 3813
Laravel 5.5: 221912 + 7482
Laravel 5.6: 143645 + 5988
Laravel 5.7: 237423 + 9959
Laravel 5.8: 553445 + 14515
Laravel 6.x: 851005 + 9355

Customising markdown parsers is totally a common thing to do, I should add. One could view parsedown extra as doing that. The CommonMark package has an extension system for doing this, and there are many extentions. For example, if people wanted to have twitter handles parsed and links, they'd need to customize the parser to do that. Something they can't do easily without this PR.

I'm wondering if I should remove the locator class, and just move the content of create within the closure in the service provider?

Can you given an example how I would use a custom markdown implementation, I mean specifically where exactly would I specify it to replace the existing one?

Yeh. If you want a totally custom implementation, just bind the "Markdown" contract in your app service provider. Laravel will then just use it.
If you are not overriding that, and want to customise the commonmark instance used, just rebind the commonmark interface, then Laravel will get that from the container when it looks to see if it can use commonmark, rather than creating the default commonmark setup.

Got it!
So there seems to reason to have the locator for further customization and I think your suggestion for the closure makes sense 👍

So there seems to reason to have the locator for further customization and I think your suggestion for the closure makes sense

I'm going to delete the locator class, since I actually didn't intend for anyone to ever call it.
Here are my two examples more clarified:
Example 1
Suppose you want to totally replace the markdown renderer the framework will use. Simply bind the thing you wanna use in your app service provider:
$this->app->singleton(\Illuminate\Contracts\Markdown\Markdown::class, function ($app) {
    return new ...
});

Example 2
Suppose you have installed commonmark in your app's composer.json instead of parsedown, and you wanna install a commonmark extension. No need to rebind the markdown contract. Just bind the commonmark environment in your app service provider:
$this->app->singleton(\League\CommonMark\EnvironmentInterface::class, function ($app) {
        $environment = \League\CommonMark\Environment::createCommonMarkEnvironment();

        $environment->addExtension(...);

        return $environment;
});

Added a test to demonstrate using a custom commonmark environment:
    public function testResolvesCustomCommonMarkEnv()
    {
        $app = new Application();
        $app->register(new MarkdownServiceProvider($app));

        $app->singleton(EnvironmentInterface::class, function () {
            $environment = Environment::createCommonMarkEnvironment();

            $environment->addExtension(new class implements ExtensionInterface {
                public function register(ConfigurableEnvironmentInterface $environment)
                {
                    throw new Exception('Ext was resolved!');
                }
            });

            return $environment;
        });

        $markdown = $app->make(Markdown::class);

        $this->expectException(Exception::class);
        $this->expectExceptionMessage('Ext was resolved!');

        $markdown->render('foo');
    }

It shows that laravel automatically used the environment I had bound, since when I asked it to render, it involved the extension I had added.

Can we instead of creating a new component move the Markdown stuff into the Support or View component? I'm not sure is a new component is needed for this.

I'm not really sure where it fits. Laravel does already have quite small components for some things, like cookie and hash. There would certainly be an argument for moving hash into support or cookie into http, if those components were being created today.

Same with the pipeline component, which could have easily been put in support.

I don't think we should make this swappable. I think we should use one good default. If a better default is now CommonMark I am fine switching to that. I think original I used Parsedown for the table support possibly. But, I see there is an official table extension for CommonMark. I am fine with switching to that if we want to do a smaller PR for that.


Simple naming convention for Memberships & MCRs

ACs

PB or Flow using a naming convention that can be cloned and updated a la PMM
Could be changed later to something more robust according to issue #15

is this really good first issue? As the Membership Name is (system) mandatory the user still have to fill it in and it will be later on changed by flow, so while easy to solve it is not a great user experience


Nightly builds do not have the correct download links

e.g. aarch64 openj9 build on https://adoptopenjdk.net/nightly.html?variant=openjdk11&jvmVariant=openj9 seems to be serving up the debugimage build, not the JDK

EDIT: Just occurred to me - "jdk" is no longer first alphabetically - are we just serving up the first matching binary for the platform in each release?

Was an API issue, now resolved.


Skip google-signin-happy-path.spec.ts

Description
Skip google sign in test until RHIDP-2265 is fixed
Which issue(s) does this PR fix

Fixes #?

PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

/test all

/lgtm


Activate suspicious request blocking tests

Description
Activate suspicious request blocking tests.
Motivation
Not much point having deactivated tests.

APPSEC_BLOCKING

yes, forgot these tests where part of a scenario, should be good now.


[SHIRO-776] refactor: JUnit5 Best Practices

Following this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GitHub issue filed
for the change (usually before you start working on it).  Trivial changes like typos do not
require a GitHub issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] Format the pull request title like [#XXX] - Fixes bug in SessionManager,
where you replace #XXX with the appropriate GitHub issue. Best practice
is to use the GitHub issue title in the pull request title and in the first line of the commit message.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] add fixes #XXX if merging the PR should close a related issue.
[x] Run mvn verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[x] If you have a group of commits related to the same change, please squash your commits into one and force push your branch using git rebase -i.
[x] Committers: Make sure a milestone is set on the PR

Trivial changes like typos do not require a GitHub issue (javadoc, comments...).
In this case, just format the pull request title like [DOC] - Add javadoc in SessionManager.
If this is your first contribution, you have to read the Contribution Guidelines
If your pull request is about ~20 lines of code you don't need to sign an Individual Contributor License Agreement
if you are unsure please ask on the developers list.
To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

[x] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004
[ ] In any other case, please file an Apache Individual Contributor License Agreement.

Hi! Noticed that the JUnit 5 migration had missed a few small steps to optimize the methods called, and sometimes swap the argument order. The compiler won't complain about such issues, but it's best to conform to JUnit 5's best practices. Hope this is helpful! It's a mostly automated change, prompted by the summons here: https://twitter.com/lobaorn/status/1763676683012317411
Tagging @bmarwell for review, since he opened

https://github.com/apache/shiro/pull/237

Use this link to re-run the recipe: https://app.moderne.io/recipes/builder/hBA8IygoJ?
organizationId=YzQ4NzM2ZmItNmRjOS00ZmU2LWJhOTUtNDQ4NTViYTRhNWFk

Thank you for your contribution!

Thank you for your contribution!

 There's potentially some more automated changes we can do here, but don't want to flood you with PRs.  Will work through them

Thanks @timtebeek, this is awesome!
Do you have a public Moderne dashboard you can point us to?

Thanks @timtebeek, this is awesome!
Do you have a public Moderne dashboard you can point us to?

Glad to hear! Yes you can run any of our recipes through https://app.moderne.io/marketplace, which is free for OSS.
You'll likely want to create a user organization for Shiro via de selector in the top left corner, and then select Shiro

Or you can select the existing Open Source organization we already have for Apache, Maven, Gradle and others.
From there you can go to our marketplace to select and run recipes; I like looking over our best practice recipes. Here's the typical list of recipes to run when first starting out. All of those should complete in about a minute for Shiro, and from there you can then create PRs as I did here.
Some recipes also come with dedicated visualizations, which we're wrapping up into a dashboard of insights and fixes to announce coming week. I'll share a link when the blog is out! :)

As an example of what you could do next, I see you're right now using assertj for one single test:
https://github.com/search?q=repo%3Aapache%2Fshiro assertj&type=code
If you were to want to switch over to AssertJ we have a recipe that should do most of the work for you:
https://app.moderne.io/recipes/org.openrewrite.java.testing.assertj.Assertj
If not, then it's maybe best to phase out that single use to prevent a long period of dual use or slow adoption.

Ha, yes. We wanted to switch to assertj at some point, and that test is so far the only one which was added.

Thanks @timtebeek, this is awesome!
Do you have a public Moderne dashboard you can point us to?

Yes finally I can announce our DevCenter, as posted on our blog:
https://www.moderne.io/blog/moderne-devcenter-dashboard-used-to-migrate-secure-visualize-large-codebases
Here's a quick preview of what's explained on the above link

We have this available for all of Apache, Gradle, Maven and a few others via https://app.moderne.io/devcenter
You'll want to select the corresponding organization in the selector in the top left corner, to then see the following for Apache Maven.

Needless to say some work to do with regards to the recent Java 17 migration effort for Apache Maven, but exited to help out! :)

Looks great @timtebeek !  Thanks for the link!

Ha, yes. We wanted to switch to assertj at some point, and that test is so far the only one which was added.

@bdemers / @bmarwell based on this comment I've gone ahead and started a branch with a mostly automated migration to AssertJ. Feel free to turn this into a pull request! :)
https://github.com/apache/shiro/compare/main...timtebeek:shiro:refactor/assert-j-best-practices


AuthHttpClient.java:45-46: Start decorating our internal...

The puzzle 99-3f73ab5f from #99 has to be resolved:
https://github.com/amihaiemil/docker-java-api/blob/ab0d78ec783de53306d63a66dd459598d58dc8c4/src/main/java/com/amihaiemil/docker/AuthHttpClient.java#L45-L46
The puzzle was created by rultor on 23-May-18.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@amihaiemil please pay attention to this new issue.

@amihaiemil/z please, pay attention to this issue

Job #172 is now in scope, role is DEV

The job #172 assigned to @bkuzmic/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this; there will be no monetary reward for this job

@0crat wait for clearing the requirement

@bkuzmic Can you open a new ticket for clarification? I would like to assign it to @llorllale -- he implemented all this auth stuff, maybe he knows it better than me :)

@amihaiemil Sure, no problem.

@0crat wait for clearing the requirement (here)

@bkuzmic The impediment for #172 was registered successfully by @bkuzmic/z

@bkuzmic I think you already spent some time on it so make a PR and change the puzzle (change the ticket origin and reference your comment here, for the next assignee.

@rultor/z the issue is closed not by @0pdd/z (its creator); I won't close the order; please, re-open it and ask @0pdd/z to close it

@amihaiemil/z please, pay attention to this issue

@amihaiemil I think we can close this one, right? I've also noticed that for building images and using X-Registry-Config header, there can be multiple key-value pairs for registries, like this:
{
  "docker.example.com": {
    "username": "janedoe",
    "password": "hunter2"
  },
  "https://index.docker.io/v1/": {
    "username": "mobydock",
    "password": "conta1n3rize14"
  }
}

I will have to create another puzzle to extend recently added RegistyConfigAuth and support multiple registries.

@bkuzmic Yes, we can close it.

I will have to create another puzzle to extend recently added RegistyConfigAuth and support multiple registries.

Just open a ticket directly :)

There is an unrecoverable failure on my side. Please, submit it here:
PID: 4@eb89e8d3-0095-4fbe-a3fc-12ba67735974, thread: AsyncProc-3
com.jcabi.xml.StrictXML[124] java.lang.IllegalArgumentException: 1 error(s) in XML document: -1:-1: Duplicate unique value [gh:amihaiemil/docker-java-api#172] declared for identity constraint of element "verbosity".

0.39.7: CID: 483bfdca-879b-49b4-9aa1-0125c15dd5c4, Type: "Close job"

The job #172 is now out of scope

Order was finished: +30 point(s) just awarded to @bkuzmic/z

Job gh:amihaiemil/docker-java-api#172 doesn't exist in WBS, can't create order


Add hacker news plugin

@Joehoel  Sorry for the late approval. I tested it and it works well.

I hope there's a simple icon. as you know, Currently, the default icon that appears when no icon is present.

Depending on the setting, it would be good to see a specific page right away. I only look at the front page, but I think it would be good to have a setting that shows the front page right away when you enter hn

Thank you for registering the plug-in.

Thanks! No worries. I will add the icon and work on your suggestions!


Deployment fails with an error - Failed to list classic administrators of subscription

Was following steps shown in this video article here https://docs.microsoft.com/en-us/azure/active-directory-b2c/azure-monitor and provided correct inputs to the template, however the deployment fails with an error message below
{   "code": "DeploymentFailed",   "message": "At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.",   "details": [     {       "code": "ClassicAdministratorListFailed",       "message": "**Failed to list classic administrators of subscription** '<subscription-id>': 'The subscription ID was not found.'."     }   ] }
Can you help?
Is there any pre-requisite to deploy this template with Owner permissions on Azure Subscription? Currently facing it while the deploying account has Contributor permissions on the Azure subscription.

Update - This indeed seems to be an issue with steps mentioned in the video because the documentation mentions you would require Owner permissions on the subscription to complete the deployment and indeed it fails if you do not have these permissions.


Jl/mmp 1962/transaction multichain destroy

Explanation

add #stopAllTracking() that stops all trackers, incl global
add addIncomingTransactionHelperListeners()
add removeIncomingTransactionHelperListeners()
add destroy()

use destroy in specs

use global clock variable and afterEach to ensure timers are properly restored between tests even if a test fails

References

Fixes https://github.com/MetaMask/MetaMask-planning/issues/1962

Changelog

@metamask/package-a

: Your change here
: Your change here

@metamask/package-b

: Your change here
: Your change here

Checklist

[ ] I've updated the test suite for new or updated code as appropriate
[ ] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
[ ] I've highlighted breaking changes using the "BREAKING" category above as appropriate

Looks good to me! Were the package updates added for any particular reason?

unsure why this in there


New version: CIAOAlgorithms v0.1.1

Registering package: CIAOAlgorithms
Repository: https://github.com/kul-optec/CIAOAlgorithms.jl
Created by: @pylat
Version: v0.1.1
Commit: 8d0376e8a636279568212f06187d172423732614
Reviewed by: @pylat
Reference: https://github.com/kul-optec/CIAOAlgorithms.jl/commit/8d0376e8a636279568212f06187d172423732614#commitcomment-52518393

@pylat please see the message above.  Once you fix the automerge issues you can register again the new revision without changing the version number and this pull request will be automatically updated.
[noblock]


scriptHandler removes bin/*.phar files from different packages

When I have a dependency which installs a .phar file into bin directory, then during ScriptHandler::installPharTools the phar gets removed.
Steps to reproduce:

Create composer.json:

{
	"name": "test/test",
	"require": {},
	"require-dev": {
		"phpstan/phpstan-shim": "^0.9.2",
		"tm/tooly-composer-script": "^1.2.5"
	},
	"scripts": {
		"post-install-cmd": [
			"Tooly\\ScriptHandler::installPharTools"
		],
		"post-update-cmd": [
			"Tooly\\ScriptHandler::installPharTools"
		]
	}
}

run composer install
vendor/bin now contains only phpstan file but phpstan/phpstan.shim package has two files specified in bin section of composer.json - https://github.com/phpstan/phpstan-shim/blob/master/composer.json#L11-L14

Expected is that after composer install two files (phpstan and phpstan.phar) files are in vendor/bin directory.

Hi @voda,
thanks for your feedback and the detailed issue report! As I developed tooly I was assuming that all PHAR files would be managed by tooly. So that a cleanup simply remove all PHAR files in the binary directories.
But anyway, can you please test this again with this tooly version?
"require-dev": {
    "tm/tooly-composer-script": "dev-master#bd936ef64693354e5700df8e5f8336333febfbdd"
  }

If this also works for you and I get the broken TravisCI stuff fixed, I'll release a new bugfix version.
Thanks,
Tommy

Hi @tommy-muehle, tested the new version and it works as expected. Thanks


README: osbs build requires -b param

+1
I've also experienced that $ osbs build isn't very user friendly.


Documentation  to explain what is happening when we run the integration test suite

As a new developer
I want to unerstand how to run the app
So I know what software works together and how, so that I can develop such software
Definiton of done

[x] video explaining what happens when you run the different ./app.sh commands in the integration

done
https://www.loom.com/share/f44be75ce80044a08a73c53ea64a3afd


fix: OSError when retrieving source code for lambda functions

While training a big network I got: OSError: could not get source code from the _save_model() method of the Trainer class, in particular when we do str_expr = inspect.getsource(key["transform"]). The error indicates that the inspect module is unable to retrieve the source code for the lambda function stored in key["transform"].

I fixed it by using dill to serialize and deserialize the functions, which appears to be more reliable.
I changed the scipy.signal.bspline import to scipy.interpolate.BSpline to be up to date with a more recent version of scipy (minimum 1.13.1) as indicated in one of their latest releases (1.13.0). I did this since we prefer to not fix the versions, but just to indicate the minimum, and the previous requirement on scipy made the tests break because of this expired deprecation.

coverage: 83.933% (-0.1%) from 84.071%
when pulling 69776c5032f6a9a098d85b908cd6bc098387423c on fix_inspect_function_trainer_gcroci2
into e422fb387e2b65d910a3488fda193c5112d5223c on main.

I fixed it by using dill to serialize and deserialize the functions, which appears to be more reliable.

It's not completely clear to me what this does, but if it works now, I'm happy to accept the change.

inspect.getsource (old way) retrieves the source code of the function or lambda expression passed to it.
It looks up the function definition in the current environment. Lambda functions are often anonymous and defined inline, making it challenging for inspect to locate their source code reliably, especially if they're not defined in a module or a directly accessible script. When I run the code on Snellius (I get the error only in that case), it's very likely that the exact location or the context where the lambda was defined might not be accessible or might not be present in the expected form.
With the new way, the serialization (dill.dumps) serializes the lambda function into a byte stream. The deserialization (dill.loads) converts the byte stream back into the original function object. This means that even if the function's source code is not directly accessible, dill (that captures the complete context of the function) can reconstruct it from the serialized data. The deserialized function retains the context and source information (captured in the serialized data) that inspect can then access.

I hope it's clearer but in case we can have a chat about this :)

Given that we have much less time for this project now, maybe we can discuss fixing versions of many packages to avoid similar problems in the future? Also given that development of this package will slow down, at least for the moment, and current package versions work well with what we are currently doing, it may be less important to stay up to date with newest package versions. This would be a separate PR though.

Very good point, I opened issue #607 :)


Enabling diagnostics PowerShell example script should have Set-AzStorageServiceLoggingProperty

$storageAccount = Get-AzStorageAccount -ResourceGroupName <resourceGroup> -Name <storageAccountName>
Set-AzStorageServiceLoggingProperty -Context $storageAccount.Context -ServiceType Blob -LoggingOperations read,write,delete -RetentionDays <days>.

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

ID: 46d15a48-4780-edbb-4485-121f901cbac9
Version Independent ID: fc65764c-2d2a-4901-5c31-82b4566bf3bc
Content: Known issues with Azure Data Lake Storage Gen2
Content Source: articles/storage/blobs/data-lake-storage-known-issues.md
Service: storage
Sub-service: data-lake-storage-gen2
GitHub Login: @normesta
Microsoft Alias: normesta

@manishgonsalves Thanks for your question. We are checking on this and will respond to you soon.

Thank you @manishgonsalves for reporting the issue. You are correct. I've changed the doc and that update should appear live shortly. Thanks again.

#please-close


feat(pixels): add my new pixel(25, 0)

Checklist

[x] I ran npm test locally and it passed without errors.

[x] I only edited the _data/pixels.json file.

[x] I entered the username in the pixels.json that I'm also using to create this pull request.
[x] I acknowledge that all my contributions will be made under the project's license.

resubmitting soon


Algolia Bug Fixes

This PR contains a few bug fixes around the recent contacts API and algolia search.
Upon first twitter login (first signin managed via local state), the user's DID is automatically published to the Contacts API. Users can delete or update this Did in Settings > Identity
If you are connected to twitter, the "This is your avatar" section in settings is replaced with your twitter profile to reinforce that you are signed in
Algolia dropdown items show a "more prominent" download button to import a contact. I tried to set the button to only appear if the contact wasn't already in your contacts but for some reason the algolia search bar wasn't honoring that logic. Super weird. Should revisit.
Clicking on links to a user's twitter profile opens in a new tab instead of redirecting them.
Fixed a small issue in the listDids graphql call that was allowing users to publish duplicate dids to the contacts api.

Looks good to me


how to close a mqtt that it's no longer needed?

in a webapp with auth user input, i will need to create a mqtt after a user login and destroy it when user logout.
any way to disconnect (apart reloading the page)?

Currently this isn't implemented, but that's a not very far fetched usecase, so I'll have a look at that…

1.7.1 now offers you to subscribe on a few events.
Feel free to open another issue, if you've got any issues or questions.


License missing

It's not 100% necessary but currently, the CLAs do not have a license. For anyone who'd want to reuse them, it'd be great if one could be added.

The CNCF CLA is mostly copied from Apache and they added this to their FAQ recently: https://www.apache.org/foundation/license-faq.html#CLA-Usage
Might be worth doing the same here.

Sorry I missed the LICENSE in the repo, that's an oversight: https://github.com/cncf/cla/commit/e399091da4c29a7f14867b678617798aa4e9b64c
The CNCF IP Policy is specified in the charter: https://github.com/cncf/foundation/blob/master/charter.md#11-ip-policy


Pronunciation assessment - word level result

I am facing issue with word level result, as per the docs in error type we should get the parameters as inserted, ommited, missspelled. but we are only getting missspelled if the word is missspelled.

@anushaa01 inserted, ommit can only show up when miscue is enabled, did you enable miscue?
https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/how-to-pronunciation-assessment?pivots=programming-language-python#pronunciation-assessment-configuration-parameters

Hi Yunhew,
Let me check and get back to you.
Thanks,
Anusha Agarwal
On Wed, Jul 7, 2021 at 8:24 AM yinhew @.***> wrote:

@anushaa01 https://github.com/anushaa01 inserted, ommit can only show
up when miscue is enabled, did you enable miscue?
https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/how-to-pronunciation-assessment?pivots=programming-language-python#pronunciation-assessment-configuration-parameters
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/1157#issuecomment-875230900,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATZI6LCCWKXOBZOZUYSYIX3TWO6WNANCNFSM47JIQLCA
.

Hi @anushaa01 ,
Were you able to try what @yinhew suggested?

Closed since no further comments from the customer in 27 days. Please open a new issue if more support is needed.


Questions about the framework...

Hi,
I'm evaluating the framework for my company before buying a license.
I have few questions if I may...

I some cases, you want to control the form HTML.
Can I override the form and write it in HTML / typescript. But still, benefit from the framework form fields. So let's say just as an example I want a different layout of the form. 10 fields in parallel...
I assume that eventually some piece of code in the framework is looping the form definition and display it.
So I want to break that apart and write the HTML myself and maybe reuse the form fields.
Is it possible?

Row Level security, Looking at the ORM I assume that it is not supported out of the box. Correct?
Did someone manage to implement it in the framework?

Can I replace the ORM to something with Linq support such as NHibernate? if so, what are the pros and cons? and how to do that?

Thanks

I know you are pushing me to buy a license (-: which is ok.
But I want to run some tests, Licenses for my developers is after I have managed to accomplish that.
Can you point me to some resources to start and create it?
Thanks

First of all, he is not pushing you as he is just a community member, not a company representative.
It is possible to use custom HTML and create Serenity widgets inside that. There are some samples in demo. Also StartSharp contains React integration for Serenity widgets.
Multi tenancy is a sample for row level security, i recommend you should add the guide to your evaluation.
You may use any ORM you like, but Serenity won't be using your ORM, so you won't be getting any advantages of Serenity.


Cannot Set Value for display props

I am following the documentation to display partial rating, but display props throw errors when i pass the rating state into it.
The error message is as follows:
Json.stringify cannot serialize cyclic structure.
Any solution to solve this issue?

Will take a look this weekend - thanks for bringing this up!

Will take a look this weekend - thanks for bringing this up!

Thank you for writing this library. After i do some inspection in my code. I found out there is no problem when i implement the star rating using local image, but there is issue when i try to use icon component from Ionicon. And one more question, is there anyway to standardise the display of star rating in partial format? Because the display of 0.1 and 0.9 rating are too similar with 0 and 1 rating.

the JSON.stringify issue should be solved as of the most recent release - so I'll be closing this issue.
On a separate note: I've noticed this issue as well - I will add a note in the Roadmap in the README about adding a partial display amounts interpolation feature which would give a more natural look and feel to marginal partial display amounts like .1 and .9


Dist package ?

It would be really helpful to have a dist folder for people who want to import the JS files directly into their project.

Although it's okay for quick prototypes I don't believe this is good for real projects and I want to discourage this. Why someone would like to have a browser version? What's the advantage here?

It's for a Scala.js project. I'm using SBT to build the project and I'm not using webpack or browserify hence the reason for the dist package request :)
I usually see a dist folder in all JS projects so I thought I should ask. But if it's not a good practice then I will try compile it locally with webpack.

I'm not sure I'm following. There is a dist/index.js but it doesn't work in the browser out of the box. Would this be enough for you or you want to make it UMD-compatible?

A committed dist/* would be very helpful for people using bower with a non node backend (and not using webpack or browserify).
I checkout resources during deploy (instead of them being checked into the repo) the advantage is that no one gets the bright idea to change the vendor scripts, the downfall tends to be the attitude that you have

Although it's okay for quick prototypes I don't believe this is good for real projects

and because I can't just go into the react-dropzone package folder and run webpack ( well I could but it would only run locally as it is never committed)
The idea behind this is a 3rd party library is not the primary purpose of a project it is a component, or utility and as such there is no reason that a 3rd parties code should be checked into the project repo.  Its a similar philosophy to Ruby and bundled gems or PHP and composer.

There is a dist in this package. You should use npm to install it. See README.md.


timber.scss.liquid 500 error

Not sure what happend but I just reuploaded timber.scss.liquid and started getting a 500 error.
When I accessed timber.scss.liquid directly, the contents of the file was
"\xE2" from ASCII-8BIT to UTF-8 scss
Not sure if relevant but I tried compiling the scss file locally and I kept on getting indentation errors
$ sass timber.scss.liquid
Syntax error: Inconsistent indentation: 2 spaces were used for indentation, but the rest of the document was indented using 4 spaces.
        on line 18 of timber.scss.liquid
  Use --trace for backtrace.

I fixed this by adding @charset "UTF-8"; to timber.scss.liquid.

This is still happening. Using 1.1.0.
http://cdn.shopify.com/s/files/1/0751/9403/t/2/assets/timber.scss.css?114

This is still happening. Using 2.0.1.
https://cdn.shopify.com/s/files/1/0751/9403/t/2/assets/timber.scss.css?453

No luck with that link @tyteen4a03. Any other info you can provide?

Just ran bootstrap from the theme gem and got this error again.

The theme gem's bootstrap is currently bringing in an old version. We're looking at a fix for that. In the meantime, grab the latest from master here, or https://github.com/Shopify/Timber/releases/tag/v2.0.2


Fix EIP-2124 Fork identifier for chain compatibility checks.

PR description
Before this pull request Besu was using the latest known fork id to create status message for Ethereum P2P protocol handshake.
This latest known fork id was created based on a list of forks retrieved from the Genesis.
For private networks it is possible that all fork blocks number are set to 0.
The algorithm to compute the valid fork hashes excludes 0 values.
As a result, the list was empty and the getLatestForkId was returning null. This is an issue when you support capabilities >= to Eth/64 sub protocol because other peers expect the fork id value in the RLP encoded message.
Moreover, the algorithm to compute the fork id should be aware of the chain head number and update CRC value only for fork blocks below the current head.
This pull request fixes this issue by fetching the chain head number and update accordingly the CRC value.
Unit tests have been extended to cover an exhaustive list of possible combinations on named networks (goerli, rinkeby, ropsten and mainnet).
Fixed Issue(s)
fixes #1044
Testing
Automated
Added unit tests.
Test Results - EIP2124Test.pdf
Manual

Network
Status

mainnet
OK

ropsten
OK

goerli
OK

mordor
OK

classic
NOT STARTED

private
OK

Signed-off-by: Abdelhamid Bakhta<EMAIL_ADDRESS>

Mainly just read the code. Didn't run the tests locally as that would be enforced.


NextJS with Tailwind CSS not showing the photo

Describe the bug
When using Tailwind CSS on NextJS project the React Image Mapper not working. The image is in the source of the code, but not showing.
To Reproduce
Steps to reproduce the behavior:

npx create-next-app -e with-tailwindcss my-project (or create nextjs project and then add the tailwindcss, its the same)
npm install react-img-mapper --force (cause its not working without force, cause react versions)
add the code from Usage section on this repo
See there is no image show in the project (but when inspect you can see the code is there and in network the image is downloading)
go to styles/global.css and remove first line - @tailwind base;
observe everything (except tailwind stuff) is working

Error Stack
No error shown in console.
Expected behavior
The image and mappers should work with tailwindcss.
Screenshots
https://prnt.sc/Vg-DRyCv4qrY
Desktop (please complete the following information):

OS: Windows 11
Browser: Chrome
Version: 103

Hello @ventsislavnikolov, for now, Next.js is not fully supported due to some issues.

Hello @ventsislavnikolov, for now, Next.js is not fully supported due to some issues.

There are any workaround?

Hello @ventsislavnikolov, for now, Next.js is not fully supported due to some issues.

There are any workaround?

Adding :
#img-mapper {
height: 100% !important;
width: 100% !important;
}
to the CSS works for now.

Hello @ventsislavnikolov @flosrn @jhowards Resolved in V1.5.0
Please check and free feel to reopen if you found any bugs related to this issue.


prompt_status:15: command not found: ✘

I'm getting this error on C-c, I'm guessing it's some kind of character compatibility but I have the patched fonts and everything else is fine.

Which OS and Terminal app are you using?

Sorry for late reply!
Mac OSX - El Capitan w/ iTerm2

Could you git pull and retest?

@conrman Could you re-test?


left click checkboxes in w2ui 1.5.x

Hy All,
I notify that in the last version 1.5.X, in grid, if there are checkboxes, clicking on the record with left click of the mouse it's ok and the checkbox will be checked but clicking on the checkbox(so not the record) with left click the checkbox can't be selected,it's like there is a preventDefault, is it right in this way?
Again my compliments for this usefull library.Good WE

I do not understand what you mean. Can you please rephrase?

Hi mrg2001,
In the grid with checkboxes enabled, if i click with the mouse on the rectangle of the checkboxes, for sure click with the left button of the mouse(the usual click for the checkbox), the checkbox will not selected instead if i click on the record the checkbox will be selected.
I don't use JS fiddle and i don't know how to put in JSFiddle a PHP call so i uploaded an example on my site where i use w2ui 1.5.x nightly...You can see that if you click on the checkbox with the mouse (left click)the checkbox will not selected....I notified that if i put multiSelect true after it works with the left click of the mouse, maybe something is going to conflict.
But if you click with the right button of the mouse over the checkbox it will work, but usually the selection is done with the left click.
Here's the example
Regards
http://fractalcosmo.com/MvcWebLog/paginaDefault.php

Yes, there was a bug. Will recompile JS a bit later tonight, then you can download


Add a unique ID to widget props

I'm trying to use a stateless widget like so:
'ui:widget': (props) ->
  <input type="checkbox" id="ID"/>I agree

The issue is generating the ID. I could obviously use a real component and do it inside the component. I could hope the component only loaded once (ouch). I could also probably hack the onChange into the onClick of the label. But all of those options aside, would it make sense for the package to generate a unique ID for each field and pass it in? _.uniqueId() maybe?

OK, I see the error of my ways here. The widget doesn't render the , it only renders the input. The label is rendered as well as whatever I supply for ui:widget. I guess the id question remains, even though my example no longer makes sense. My example does raise a different question, how I could replace a whole component, including the label, but that's for a separate issue...

I'm surprised your code snippet works; JSX requires to have a single root element, so you should use something like this:
'ui:widget': (props) ->

<input type="checkbox" id="ID"/>I agree

For unique id generation, the issue is probably no persist them along the entire lifecycle of the component, so that means persisting this in comp state, or keeping a whole tree of nested ids in root component state.
To be noted, maybe #45 would help achieving what you're after here.
Also, we don't have identified use cases for this in our current projects using this lib; so would you be willing working on implementing this?

I'm trying to use a stateless widget (...) I could hope the component only loaded once (ouch)

Stateless components are indeed fully rendered every time they receive props. If you want to "initialize" a component, eg with something generated once attached to it, you need state, hence you can't use stateless components.

We now have generated unique ids for forms (docs).

@n1k0 Yet more awesomeness from y'all. Thanks.


AsciiDoc include macro isn't working

AsciiDoc has a macro that lets you pull in adoc content from other files. The include:: macro isn't working on our Franklin pages. See the official documentation for include:: here.
This page shows the issue (scroll to the bottom of the page):
Link: Onboard Accounts for Agentless Scanning
Relevant underlying doc source:
include::frag_start-agentless-scan.adoc[leveloffset=1]

Screenshot:

Re-opening this issue. I've found other instances where the include:: macro still isn't being processed properly.
See the following links. This isn't a comprehensive list, it's just what I spot checked:

agentless-scanning/onboard-accounts/onboard-azure - See agentless-scanning/onboard-accounts/onboard-azure at the bottom of the page.
agentless-scanning/agentless-scanning-results - See onboard-accounts/frag_start-agentless-scan.adoc at the bottom of the page
agentless-scanning/onboard-accounts/onboard-gcp - See frag_start-agentless-scan.adoc at the bottom of the page
install/install-openshift-4 - Search for fragments/install_defender_twistcli_export_oc_4.adoc
waas/deploy-waas/deployment-hosts - See fragments/create-waas-rule.adoc and fragments/add-app-policy.adoc

since the HTML format changed on the worker to make the include fragments, those docs were broken until republish
just republished all, seems to be working now

@maxakuru This still isn't completely working. This page still has the issue:

waas/deploy-waas/deployment-hosts

@maxakuru Also, quick question: What's the best way to force a republish on all content?

@iansk I republished all, seems to be working now.. could you take another look?
by design we don't have a convenient button to republish all docs, since we want to be selective for performance... however there is a script in the prisma-cloud-docs repo that can be used for batch publishing:
# publish one
npm run publish:batch docs/en/compute/pcee/admin-guide/waas/deploy-waas/deployment-hosts

# publish all in specific folder
npm run publish:batch docs/en/compute/pcee/admin-guide/waas/deploy-waas/fragments/**/*

# publish all (takes ~30 mins)
npm run publish:batch docs/**/*

@maxakuru Awesome, that's good stuff. Thanks for providing the script

@maxakuru I'm still seeing issues with this. If I grep through the doc content for include::, and check each page, I'm finding issues with virtually every page.
As an example, these pages still show the issue. I'm not sure if was previously fixed, and now showing up again. But currently, the latest published content shows the issue:

agentless-scanning/onboard-accounts/onboard-azure - See agentless-scanning/onboard-accounts/onboard-azure at the bottom of the page.
agentless-scanning/agentless-scanning-results - See onboard-accounts/frag_start-agentless-scan.adoc at the bottom of the page
agentless-scanning/onboard-accounts/onboard-gcp - See frag_start-agentless-scan.adoc at the bottom of the page
install/install-openshift-4 - Search for fragments/install_defender_twistcli_export_oc_4.adoc

@iansk these are because we changed the rules for document naming to normalize paths, but didn't change the content of the documents to correctly reference those new names.

for the first 3 issues: frag-start-agentless-scan is being included as frag_start-agentless-scan
for the last: install-defender-twistcli-export-oc-4 is being included as install_defender_twistcli_export_oc_4

for each of these we just need to replace the _ with -.. does that help?

@maxakuru Yes, that helps, thanks. I updated the underscore to a dash, but it's still not working completely working for me. Also, I now see some HTML code being rendered on the page.
Franklin preview:
https://main--prisma-cloud-docs-website--hlxsites.hlx.page/prisma/prisma-cloud/en/compute/pcee/admin-guide/agentless-scanning/onboard-accounts/onboard-azure?branch=ian/fix-include
GitHub pull request:
https://github.com/hlxsites/prisma-cloud-docs/pull/69
BTW, our embedded Gdocs stopped working since you merged some recent fixes. I see the same weird HTML code on the page, so it may be related to this issue.

https://redesign--prisma-cloud-docs-website--hlxsites.hlx.page/prisma/prisma-cloud/en/compute/pcee/admin-guide/welcome/launch?branch=ian-complex-gdoc
https://redesign--prisma-cloud-docs-website--hlxsites.hlx.page/prisma/prisma-cloud/en/compute/30-xx/admin-guide/welcome/field-guide?branch=ian-complex-gdoc2

@iansk yep my bad, I forgot to merge a fix for a typo yesterday, fix was merged in https://github.com/hlxsites/prisma-cloud-docs/pull/70

@maxakuru Thanks, the gdocs are loading again now. But the include:: macro still isn't working:
https://main--prisma-cloud-docs-website--hlxsites.hlx.page/prisma/prisma-cloud/en/compute/pcee/admin-guide/agentless-scanning/onboard-accounts/onboard-azure?branch=ian/fix-include
I tried republishing the page, but nothing changed.
When I inspect the element on the page, I see it's rendered (incorrectly) as a link:

frag-start-agentless-scan.adoc

looks like this was an error during fragment loading that only occurred when the branch query param was set
should be fixed in #142, could you try again @iansk

@maxakuru I'm still not seeing include:: content being loaded.
Example pages:

https://main--prisma-cloud-docs-website--hlxsites.hlx.page/en/compute-edition/30-xx/admin-guide/waas/deploy-waas/deployment-containers?branch=ian-update-compute-ag-30
https://main--prisma-cloud-docs-website--hlxsites.hlx.page/en/compute-edition/30-xx/admin-guide/agentless-scanning/onboard-accounts/onboard-aws?branch=ian-update-compute-ag-30
https://main--prisma-cloud-docs-website--hlxsites.hlx.page/en/compute-edition/30-xx/admin-guide/alerts/jira?branch=ian-update-compute-ag-30

@iansk looking at logs and I see [worker] book is undefined, path resolution may fail for these
also on main I don't see a 30-xx folder: https://github.com/hlxsites/prisma-cloud-docs/tree/main/docs/en/compute-edition
remember that the book.yml needs to exist on main before the pages will work