Changes in order to deploy on OpenShift

Here are my changes to the reference application in order to deploy the java backend and angular front-end on the Red Hat OpenShift Container Platform.

Hi Michael,
I've seen that the OpenShift build uses an environment that needs to have http://localhost:8081 URL setup. Instead of changing the environment.prod.ts file, I've changed the assemble file of s2i-angular so that the used environment is the dev one, because it will always point to this local URL:
https://github.com/cbeldacap/s2i/blob/master/s2i/angular/bin/assemble#L23 (yet to be PR-ed)
I think it maybe fits better with everyone's needs. Does it make sense?

Hi Cristobal,
I might miss something but I think it will break the setup. There are three artefacts that depend on each other:
(1) environments.prod.ts or environment.ts
in https://github.com/oasp/my-thai-star/tree/develop/angular/src/environments
(2) https://github.com/oasp/s2i/blob/master/s2i/angular/bin/assemble
(3) https://github.com/oasp/s2i/blob/master/templates/mythaistar/oasp-mythaistar-angular-template.json
(3) is the template that defines the build and deployment of the angular parts of the app, i.e. this is where the chain of dependencies starts. There is a env variable in the template $BUILD_TARGET (ca. line 121) that defaults to 'prod' but could be used to change to build for a different environment.
$BUILD_TARGET is used in (2) to tell the assemble script which environment config file to select and modify, i.e. looking for $REST_ENDPOINT_PATTERN, replacing it with $REST_ENDPOINT_URL. Both are also defined in (3).
TARGET_ENV_FILE="src/environments/environment.$BUILD_TARGET.ts"
sed -i "s|$REST_ENDPOINT_PATTERN|$REST_ENDPOINT_URL|" $TARGET_ENV_FILE

With this said, I do not know how the behaviour of yarn changes with the change. The reason I'm raising this is that e.g. in RAILS, there is different behaviour when you start the app with an environment variable RAILS_ENV='DEV' vs. 'PROD'. Example: in DEV, the runtime does code-reloading and does not optimise assets e.g. does not compress js/css files whereas in PROD it is the opposite. No idea if similar stuff happens in yarn, depending of the environment.
Nevertheless, for your change to work, you have change line
TARGET_ENV_FILE=

also, otherwise the script patches an environment file that yarn ignores ...

Hi @mickuehl ,
I've started fixing some "collateral" Jenkinsfiles to merge your PR in the application.

@mickuehl
I'm adding this PR's changes plus some others related with DevOps (deployment, usage in CICD environments, etc.) in #53 .
I'll proceed to close this when after the testing period.

As the devonfw-angular base image for OpenShift lets the user specify the string that is going to be substituted, is this PR still needed?  @mickuehl

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Coding profiles and link to the mobile number

Can you please tell me how can I add coding profiles into the header section or create a new section? Also, how can I make a mobile number as a link?

You can make phone number a link by changing xxx-xxx-xxxx to \href{tel:xxx-xxx-xxxx}{xxx-xxx-xxxx}

You can make a new section as follows:
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
%     CODING PROFILES
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
\section{Coding Profiles}
\hrulefill

\pt Profile 1
\sectionsep

First of all, thank you. Also, i have more two questions,

can i add coding profiles in the header section itself? Will it be bad?
There are links in the resume, but it does not have an underline or anything. So the recruiter will be able to identify that there is a link? or i should add an underline too?

You can add underline as well if you like it. No harm in that.
To add coding profile on the header, you would have to remove something else to make space. You can check for available fonts at http://mirrors.ibiblio.org/CTAN/fonts/fontawesome5/doc/fontawesome5.pdf and see if any is relevant for it and update https://github.com/kevalmorabia97/Latex-Resume-Keval-Morabia/blob/master/Resume.tex#L22 line with it

Ok, I'll try it out.
Unfortunately, i have to use some other template:( because I'm a 2023 pass out and i don't have many details to show. I added my all details even my 10th, and 12th-grade details but still resume has almost 2 section space left at the bottom, which is bad I guess.
Let me know if you have any idea that I can use the same template though. I really really loved this template.
Here I'm attaching an image of how much space is still i need to fill, which i don't know what should i add to fill...

A very easy thing to do would be to increase heating font size and space between lines in Resume.cls. That can very easily fill most of the page and still make it look good

Still remains spaces.
One more doubt, if we make changes to the resume template or if there are any unsolved errors while using LaTeX on overleaf, does it make the resume unstructured and hence won't get shortlisted?

As long as it looks good and there aren't any big errors, you should be fine

Okay. Thank you.


cyan attempts to compile lua files as if they were teal files.

See repro: cyan-lua-compilation-repro.zip
When I have a following file structure:

And tlconfg.lua set to:
return {
    build_dir = "build",
    include_dir = {"src"},
    source_dir = "src"
}

Then cyan build fails:
$ cyan build
     Error 2 type errors in src/test.lua
       ... src/test.lua 3:7
       ...    3 | local a
       ...      |       ^
       ...      | variable 'a' has no type or initial value
       ...
       ... src/test.lua 4:7
       ...    4 | local b
       ...      |       ^
       ...      | variable 'b' has no type or initial value
      Info Type checked src/main.tl

It looks like cyan tries to typecheck a lua file as if it was a teal file (even though there is a corresponding .d.tl file!).

Cyan doesn't currently handle this use case too well. An awkward solution would be to move the lua file into the build directory and use the dont_prune config.
return {
   build_dir = "build",
   include_dir = {"src"},
   source_dir = "src",
   dont_prune = { "build/test.lua" },
}

Or keep your lua sources in a different directory and use a build script to copy them over.
But this use case seems reasonable enough to be supported by default.

@euclidianAce I think I'm hitting the same issue when trying to use Cyan with the teal branch of LuaRocks.
Right now I have two kinds of Lua files under src, and I want to ignore both:

generated .lua files from the .tl files (I'm still keeping them under src/ to keep the build scripts compatible). I managed to get Cyan to ignore this by adding exclude = { "**/*.lua" } in my tlconfig.lua file
vendored Lua modules such as src/luarocks/vendor/argparse.lua, for which I have a src/luarocks/vendor/argparse.d.tl file. Looks like Cyan is still checking the .lua file (and thus outputting lots of errors) because these are referenced by other files as dependencies.


CTO HaloITSM 1.0.0 beta

This plugin supports creating tickets for Netskope alerts on the HaloITSM Platform. Only supports ticket creation and ticket status syncing. The HaloITSM plugin does not support updating the tickets (incidents).


Legacy InjectionParameterValue

Description
InjectionParameterValue do not exist anymore. I have some legacy code that uses this class in order to allow resolving contructor arguments generically by order based on the following article:
https://mikaelkoskinen.net/post/unity-passing-constructor-parameters-to-resolve
I would like an alternative way of doing that on current version (5.x).
To Reproduce
Try using InjectionParameterValue.

It was renamed to ParameterValue

@ENikS Thank you for the fast reply. I will take a look and try to adapt the code using this new classes.

@ENikS I upgraded my code with your help and it worked like a charm. Thank you!

@xcap2000 @ENikS do you guys have a solution on how you modified orderred parameters override?

@ershad65 take a look at the following gist and tell me whether it helped you:
https://gist.github.com/xcap2000/2d264960cae55cf131edaa907a1a3eae

Thanks @xcap2000. That helps

@ershad65 you are welcome!


CAUSEWAY-3522: Entities referenced from within Viewmodels may stay hollow during followup AJAX request

[ ] verify that we can accurately track the load/detach lifecycle of viewmodels, that is for both regular and AJAX requests
[ ] if so, we should be able to remove the wicket viewmodel refresh hackery

abandoned: instead we fixed this by caching titles within EntityIconAndTitlePanel


Remove nbformat: 3 | 4 | 5  type in types.ts

Remove  nbformat: 3 | 4 | 5 because they are number typed rather than a type for ipynb.

To be honest, it is no problem to delete it as it is not currently used internally either.
However, I would like a more detailed explanation. How are you having trouble?

I can't load the notebook because of it. It looks like you are using nbformat: 3 | 4 | 5  as a type for loading the notebook, but there is no such type of that.

So there are files without nbformat?
OK. Merging.


Project Monitoring and Alert Manager Configs

Summary
Fixes #5541
Fixes #2612

Occurred changes and/or fixed issues
Implements UI changes for Project Monitoring (as projectHelmCharts) and AlertmanagerConfigs
Specific UI changes to be defined in this mockup: https://xd.adobe.com/view/3fd60696-ee50-4f66-b516-93eb6841bb4b-6a85/ (note: chart installation handled by charts to be implemented by @aiyengar2)

Technical notes summary

Areas or cases that should be tested

Areas which could experience regressions

Screenshot/Video

Just merged in the AlertManagerConfig stuff and runs/builds/tests locally for me so I think we're good(ish) for now.


Vr Buttons error with not setup in udon

Pretty much any vr button from this list https://docs.google.com/spreadsheets/d/1_iF0NjJniTnQn-knCjb5nLh6rlLfW_QKM19wtSW_S9w/edit
Exemple from my own code of error that halts udon
Input Button Oculus_CrossPlatform_Button4 is not setup.
Did look into code but not sure how input entirely works in unity

All the "Oculus_CrossPlatform" axes have been implemented with 4f8cd5c2968f75b207e1fa1e043d7e5b57f71ffa
Do you believe the other axes should also be added?

Not sure since i got for the crossplatform ones personaly but maybe since more control?

Based on discussions in prefabs and U# servers - most people seem to use Oculus_CrossPlatform + horizontal/vertical stuff

While not all inputs have been added, I'm closing this as it covers the majority of cases. We can open a new issue if people really want the other input axes.


Can´t return to my mobile app after login

Hello everyoe.
I finally can sing up and login with gmail,twiiter,facebook on my website app (https://rails-tutorial-denialtorres.c9.io) with your module, I'm using ionic framework with cordova on the client side (android) and ruby on rails for the server side.
the problem is that when i´m login with my social credentials a little pop-up window (inappbrowser plugin) appears but never closes, so i´m navigating my website app on the pop-up window instead of the mobile app,
and the mobile app does nothing, it still remains in the main window where you are prompted for credentials
any suggestian with that ??
regards

Check to make sure your redirectUri is configured properly.

Hi @denialtorres,
Let us know if @sahat's suggestion works. Feel free to re-open this issue if you think there might still be an issue in Satellizer.
Thanks!


style(js,json,less): use 'prettier' to handle code style

Rather than throwing an error during precommit hook.
prettier will reformat all the code to meet the styleguide standards automatically.
Check out this talk for more information on the 'why' to use prettier.

part 1 of #535
part 2 is in https://github.com/uPortal-Project/uportal-app-framework/pull/581

Contributor License Agreement adherence:

[x] This Contribution is under the terms of Individual Contributor License Agreements (and also Corporate Contributor License Agreements to the extent applicable) appearing in the Apereo CLA roster.

@davidmsibley Currently I have prettier setup to use all of its defaults.
Those defaults are very similar, but subtly different than the Google Code style, we can adjust the settings to use single quotes.

@davidmsibley updated to use single quotes

I'm confused about a couple things in this. I gotta read it more, but I was referring to the single to double quote replacement that was happening in the js, not the less. Now the less is switched from double to single quotes.
Unrelated to whether or not we use prettier, I'm WAY more opposed to code being changed on commit than stopping you from committing. That sounds like many, many unintended problems waiting to happen.

:laughing: The codeclimate failure is occurring because the normalized code style allows codeclimate to detect nine places with code similarity that it wasn't able to before.

Does this automatically re-format to use Google's code style or does this use a separate style that we would have to match to Google's?

subtly different than the Google Code style

Okay, I hadn't understood that nuance. -1.  Code is either in Google Style or it's not. Automatically different, even subtly, is not Google Style. I'd only be open to automatic code styling if it's automatically styling to the adopted code style. :)
This also may be a tool that it'd be best to adopt on a less-core module or three before adopting it here in uportal-application-framework. I'm not sure I share @davidmsibley 's prediction of many unintended problems, but I certainly see how a reasonable person would fear that. I think we can be way more open to experimental friction and risk in modules that we're less likely to need to release frequently for adoption by other teams, which is what this module is.

Code is either in Google Style or it's not. Automatically different, even subtly, is not Google Style.

Does this automatically re-format to use Google's code style or does this use a separate style that we would have to match to Google's?

The subtle differences are trailing commas and quotes.
Both can be configured to match Google Code Style. :white_check_mark:

I'd only be open to automatic code styling if it's automatically styling to the adopted code style.

Or we could make the automated style the adopted style. :wink:

That sounds like many, many unintended problems waiting to happen.

I understand the concern, in the past formatters could be flaky. :fearful:
Prettier and ESLint are based of ESTree an AST standard developed by Mozilla for accurately representing and safely manipulating code structure. :+1:
Using AST for code manipulation resolves the invalid parsing and serialization errors common in regular expression based parsers (which did have major issues with producing invalid code). :white_check_mark:
Furthermore, Prettier is in production use today on some major projects: React, Jest, Yarn, and others.
The software is pretty well battle tested. :crossed_swords:

This also may be a tool that it'd be best to adopt on a less-core module or three before adopting it here in uportal-application-framework

:man_shrugging: With the clean breakup of modules, it is arguably questionable if there is a "core" and what isn't core. Until recently I thought uPortal-home was "core".

I think we can be way more open to experimental friction and risk in modules that we're less likely to need to release frequently for adoption by other teams, which is what this module is.

Since all the modules are closely interrelated, what is module is sufficiently "low risk" in your mind?

uPortal-home is lower risk than uPortal-app-framework, in that only the latter has downstream projects depending upon it.
widget-creator is much lower risk, in that its audience is developers.

Closing this, til decision is made over in https://github.com/UW-Madison-DoIT/widget-creator/pull/25


Adding Node not possible

Hi,
I have problems with adding nodes to my shipyard master (both on Ubuntu 16.04, Docker 1.9.1; with no TLS for shipyard).
On my master node (e.g. <IP_ADDRESS>) everything works fine (I can see alls containers, can deploy containers, etc.) - from my second node I can ping the shipyard master as well as curl all shipyard ports (in case of etcd port 4001: curl http://shipyard-ip:port = 404 not found). I've also checked the iptables logs - I noticed no blocked connection.
Unfortunately, it is not possible to add the second node to the master - I tried it with automatic delopyment:
curl -sSL https://shipyard-project.com/deploy | ACTION=node DISCOVERY=etcd://<IP_ADDRESS>:4001 bash -s
as well as with manually deployment:
docker run
-ti
-d
--restart=always
--name shipyard-swarm-agent
swarm:latest
join --addr <IP_ADDRESS>:2375 etcd://<IP_ADDRESS>:4001
The shipyard-agent container only logs:
INFO[0000] Initializing discovery without TLS
INFO[0000] Registering on the discovery service every 1m0s...  addr=<IP_ADDRESS>:2375 discovery=etcd://<IP_ADDRESS>:4001
ERRO[0003] client: etcd cluster is unavailable or misconfigured
On the master I have checked all shipyard container logs - but found only this in the logs of "shipyard-swarm-manager":
2016-09-19T22:01:55.602791282Z [31mERRO[0m[0002] Unable to update state of container "7c5d6961588f6164ba5977332c726e7d30f8fccd38b8b3310ab1456a01a07165": Error response from daemon: Unknown device 7c5d6961588f6164ba5977332c726e7d30f8fccd38b8b3310ab1456a01a07165  [31mid[0m=Z3RM:FEM6:RE45:ZV3L:YMNZ:CEQ5:BRIP:D7UA:VN4O...
Thanks and best regards,
Patrick

first you should start firewalld service and then add the source by
firewall-cmd --zone=public --add-source=<IP_ADDRESS>/24,
--addr should be append with agent's address,


Any plans for Material3 components?

It looks like the library only has Material 2 components. Any plans for Material 3?

Okay. Let me see if I can do the changes and make a PR.

That's nice, glad to see that.


how to disable authentication

Hi,
I want to integrate dashboard to another application. But i didn't find a way to disable caravel authentication. I tried to uncomment AUTH_ROLE_PUBLIC in config.py, but it still asks for login. Is there anyway to do this?
# Uncomment to setup Public role name, no authentication needed
AUTH_ROLE_PUBLIC = 'Public'

Thanks

I haven't tested public access much, but you can try opening the Public role from the Security menu in the app and adding all/some permissions to it. I just did a bit of testing and it seems to work. Beware that anyone on your network will then have those rights.

After copied all admin's roles to Pubilc, I can see http://<IP_ADDRESS>:8088/caravel/welcome, but no dashboard or slice displayed.
It throws exception after anonymous access dashboard directly, here is the screenshot,

Looks like this is fixed in master. I'll push a release today or tomorrow.

Shoule be fixed, closing.

In my config.py I've set AUTH_ROLE_PUBLIC to "Public"  and give all the necessary grants to this Public role but the application still asking for login and password
I've superset 0.28.1 version.
Thanks.

Same here, same issue

Please provide a fix, it's not fixed.

I can verify that this functionality works properly in 0.37 - please share additional information on your issue, such as your current Superset version and configuration, if this is not the case for you.

Can you please share documentation on how to disable default authentication. I tried to set AUTH_ROLE_PUBLIC to public in config.py file. But it still asks for login.

facing the same issue i create my own image
### Uncomment to setup Public role name, no authentication needed
AUTH_ROLE_PUBLIC = 'Public'
### AUTH_USER_REGISTRATION_ROLE = "Public"
## Grant public role the same set of permissions as for a selected builtin role.
PUBLIC_ROLE_LIKE: Optional[str] = None
class CeleryConfig:  # pylint: disable=too-few-public-methods
ROBOT_PERMISSION_ROLES = ["Public", "Gamma", "Alpha", "Admin", "sql_lab"]

Facing the same issue here:
-> uncommented "AUTH_ROLE_PUBLIC = 'Public'" on config.py, no login is necessary when accessing Superset from link, but it is still requesting authentication when adding dashboard link to iframe content on html.

still unfixed ,i have got

set superset_config.py
# Uncomment to setup Public role name, no authentication needed
AUTH_ROLE_PUBLIC = 'Public'
PUBLIC_ROLE_LIKE: Optional[str] = "Gamma"

We are facing same issue, these are the steps that we followed

Clone Superset's GitHub repository git clone https://github.com/apache/superset.git
Updated config.py
# Uncomment to setup Public role name, no authentication needed
AUTH_ROLE_PUBLIC = 'Public'
Docker compose (tried it with both dev and non-dev)
NONDEV
docker-compose -f docker-compose-non-dev.yml pull
docker-compose -f docker-compose-non-dev.yml up
DEV
docker compose build
docker-compose up -d
On UI still redirecting to login page

The issue with public access is faced by me too. I have enabled the configurations needed and created a role to support public access. But the public dashboard access is still prompting for a password.
Uncomment to setup Public role name, no authentication needed
AUTH_ROLE_PUBLIC = "Public"
PUBLIC_ROLE_LIKE = "Restricted Gamma No Charts No CSV"

Is there any update or fix for this issue?

Facing the same issue. Any updates?

Is it possible to disable auth at all?

It still appears broken.  I've tried just about every combination of config options and I'm always redirected to /login

maybe this will help
my superset_config.py file:
`SECRET_KEY="*******************************************"
TALISMAN_ENABLED = False
ENABLE_PROXY_FIX = True
HTTP_HEADERS = {'X-Frame-Options': 'ALLOWALL'}
AUTH_ROLE_PUBLIC = 'Gamma'
FEATURE_FLAGS = {
"DASHBOARD_RBAC": True,
"EMBEDDED_SUPERSET": True,
"EMBEDDABLE_CHARTS": True,
"ENABLE_TEMPLATE_PROCESSING": True,
"DASHBOARD_NATIVE_FILTERS": True,
"DASHBOARD_CROSS_FILTERS": True,
}
CONTENT_SECURITY_POLICY = {
'default-src': ['self', 'http://...', 'http://...:8088'],
}
ENABLE_JAVASCRIPT_CONTROLS = True`


Unit of Work

We should cover Unit of Work.

I can take this one as it'll line up with my NServiceBus article on implementing UoW and Outbox.


Swift 5.9 / Xcode 15 Warning: forming 'UnsafeMutableRawPointer' to a variable of type '[Type]'

I'm working on getting ready for Xcode 15 and Swift 5.9 and seeing some warnings coming from IndexStore.swift.
Looks like these were added with this swift evolution proposal, in which they provide some #workarounds-for-common-cases
Here's the build warnings we're seeing:
INFO: From Compiling Swift module @com_github_lyft_swift_index_store//:IndexStore:
external/com_github_lyft_swift_index_store/Sources/IndexStore/IndexStore.swift:49:67: warning: forming 'UnsafeMutableRawPointer' to a variable of type '(String) -> ()'; this is likely incorrect because '(String) -> ()' may contain an object reference.
        indexstore_store_units_apply_f(self.store, /*unsorted*/0, &context) { context, unitName in
                                                                  ^
external/com_github_lyft_swift_index_store/Sources/IndexStore/IndexStore.swift:173:66: warning: forming 'UnsafeMutableRawPointer' to a variable of type '(UnitReader, (UnitDependency) -> ())'; this is likely incorrect because '(UnitReader, (UnitDependency) -> ())' may contain an object reference.
        indexstore_unit_reader_dependencies_apply_f(self.reader, &context) { context, unitDependency in
                                                                 ^
external/com_github_lyft_swift_index_store/Sources/IndexStore/IndexStore.swift:229:80: warning: forming 'UnsafeMutableRawPointer' to a variable of type '(RecordReader, (Symbol) -> ())'; this is likely incorrect because '(RecordReader, (Symbol) -> ())' may contain an object reference.
        indexstore_record_reader_symbols_apply_f(self.reader, /*nocache*/true, &context) { context, symbol in
                                                                               ^
external/com_github_lyft_swift_index_store/Sources/IndexStore/IndexStore.swift:242:67: warning: forming 'UnsafeMutableRawPointer' to a variable of type '(RecordReader, (SymbolOccurrence) -> ())'; this is likely incorrect because '(RecordReader, (SymbolOccurrence) -> ())' may contain an object reference.
        indexstore_record_reader_occurrences_apply_f(self.reader, &context) { context, occurrence in
                                                                  ^
external/com_github_lyft_swift_index_store/Sources/IndexStore/IndexStore.swift:278:66: warning: forming 'UnsafeMutableRawPointer' to a variable of type '(SymbolOccurrence, (Symbol, SymbolRoles) -> ())'; this is likely incorrect because '(SymbolOccurrence, (Symbol, SymbolRoles) -> ())' may contain an object reference.
        indexstore_occurrence_relations_apply_f(self.occurrence, &context) { context, relation in
                                                                 ^

can you test with this branch https://github.com/lyft/swift-index-store/pull/45

but yea i hadn't seen that proposal until after I did that so maybe we can simplify by using withUnsafePointer

Here's another take doing that https://github.com/lyft/swift-index-store/pull/48

Sorry, haven't looked at this in a long time since the issue seemed to go away a while ago. Since #48 has been merged though I'm guessing we can close this out? I'll go ahead and do so in a few days (or next time I remember to look at it) if no one objects. Or feel free to close it out for me if you believe it is done!

ah yes thanks! lmk if you hit any issues!


fix(BasePointerRenderer): handle destroyed origin transform follow

Reset pointer origin transform follow on ResetPointerObjects.
Add null check in event of a destroyed pointer origin.

merged 3.3.0


Dev-master (fa0da57) returns error on all requests

Dev-master (fa0da57) returns the following error on all requests.
Fatal error: Declaration of Prettus\\Repository\\Eloquent\\BaseRepository::with() must be compatible with that of Prettus\\Repository\\Contracts\\RepositoryInterface::with()

My guess is it has something to do with merge 40452e4

@hiddeco Fixed ! Thank you for reporting this error! ;)

@andersao 👍


Optionally turning on customized file manager

This addresses #185
Implementation detail:
Adding an additional parameter is a little clunky, rather, this PR combines existing param classFileManager and the new boolean useCustomizedFileManager into IncToolOptions because both are for inc compile used for java tools.
As a by product, by using Maybe<ClassFileManager> allows to kill NoopClassFileManager.
If this looks ok I will make the downstream change in https://github.com/sbt/sbt, probably replace https://github.com/sbt/sbt/pull/2791

@eed3si9n can you review this? mostly done with testing this change (along with the earlier #181 ), ready to ask for a new release if this PR goes in

So the title of this PR should be flipped to optionally turning "on" if the default is off?

yea, :) fixed

Merged! Thanks for the contribution :)

cool thanks!
I will update https://github.com/sbt/sbt/pull/2791


Docker image fails

I tried to build the dockerfile provided, but it failed at the first apt update.
This is is the output
$ sudo docker build -t hero .
 Use with caution in scripts.

Get:1 http://archive.ubuntu.com/ubuntu bionic InRelease [242 kB]
Get:2 http://security.ubuntu.com/ubuntu bionic-security InRelease [88.7 kB]
Get:3 https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64  InRelease [1581 B]
Err:3 https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64  InRelease
  The following signatures couldn't be verified because the public key is not available: NO_PUBKEY A4B469963BF863CC
Get:4 http://archive.ubuntu.com/ubuntu bionic-updates InRelease [88.7 kB]
Get:5 http://security.ubuntu.com/ubuntu bionic-security/universe amd64 Packages [1567 kB]
Get:6 http://archive.ubuntu.com/ubuntu bionic-backports InRelease [83.3 kB]
Get:7 http://archive.ubuntu.com/ubuntu bionic/multiverse amd64 Packages [186 kB]
Get:8 http://archive.ubuntu.com/ubuntu bionic/main amd64 Packages [1344 kB]
Get:9 http://archive.ubuntu.com/ubuntu bionic/restricted amd64 Packages [13.5 kB]
Get:10 http://archive.ubuntu.com/ubuntu bionic/universe amd64 Packages [11.3 MB]
Get:11 http://security.ubuntu.com/ubuntu bionic-security/multiverse amd64 Packages [22.9 kB]
Get:12 http://security.ubuntu.com/ubuntu bionic-security/restricted amd64 Packages [1307 kB]
Get:13 http://security.ubuntu.com/ubuntu bionic-security/main amd64 Packages [3094 kB]
Get:14 http://archive.ubuntu.com/ubuntu bionic-updates/multiverse amd64 Packages [30.0 kB]
Get:15 http://archive.ubuntu.com/ubuntu bionic-updates/universe amd64 Packages [2342 kB]
Get:16 http://archive.ubuntu.com/ubuntu bionic-updates/restricted amd64 Packages [1347 kB]
Get:17 http://archive.ubuntu.com/ubuntu bionic-updates/main amd64 Packages [3519 kB]
Get:18 http://archive.ubuntu.com/ubuntu bionic-backports/universe amd64 Packages [20.5 kB]
Get:19 http://archive.ubuntu.com/ubuntu bionic-backports/main amd64 Packages [64.0 kB]
Ign:20 https://developer.download.nvidia.com/compute/machine-learning/repos/ubuntu1804/x86_64  InRelease
Get:21 https://developer.download.nvidia.com/compute/machine-learning/repos/ubuntu1804/x86_64  Release [564 B]
Get:22 https://developer.download.nvidia.com/compute/machine-learning/repos/ubuntu1804/x86_64  Release.gpg [833 B]
Get:23 https://developer.download.nvidia.com/compute/machine-learning/repos/ubuntu1804/x86_64  Packages [73.8 kB]
Reading package lists...
W: GPG error: https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64  InRelease: The following signatures couldn't be verified because the public key is not available: NO_PUBKEY A4B469963BF863CC
E: The repository 'https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64  InRelease' is not signed.
The command '/bin/sh -c apt update' returned a non-zero code: 100

I tried to run this on ubuntu server 22.4.

Maybe you can try to join this command.
RUN rm /etc/apt/sources.list.d/cuda.list

Still no sucsess with that command but I got it running by changing base image and a fix from nvidias forum:
#FROM nvcr.io/nvidia/cuda:10.1-cudnn7-runtime
FROM nvidia/cuda:10.0-base-ubuntu18.04

RUN apt-key del 7fa2af80
RUN apt-key adv --fetch-keys https://developer.download.nvidia.com/compute/cuda/repos/ubuntu1804/x86_64/3bf863cc.pub


Vitest

Another FYI @aryaemami59, I'm moving my repos gradually over from jest to vitest. Figured this would be a good one to do since vitest uses expect-type!
Adding the vitest dependency enables removing jest, @types/jest, ts-jest, jest.config.js and the globals_jest eslint config.

@mmkal That's awesome news, Vitest is the way to go.


Rework the NavigationHeader

Add ability to specify custom left, right components, and title component. Style the NavigationBar according to the Platform.
Refer https://github.com/ericvicenti/navigation-rfc/pull/21
cc @ericvicenti

@ericvicenti Updated!

Can we move NavigationExampleBackButton into CustomComponents instead of having it in the example? I think we should have a built-in back button as a default prop to renderLeftComponent so each app will not need to re-implement the back button if they don't want to

@ericvicenti Ok. Will do. What should I name the component? NaviagtionBackButton?

It is part of the header so we should probably call it NavigationHeaderBackButton

@ericvicenti Updated. Ideally we'll want to back button asset to be of @1.5x, @2x etc, but that's for another PR.

I would rather not change renderOverlay right now. I recently realized we have a bug in the header because the header needs access to this.state.scenes to display the stale/popped scene header as it gets animated away.
So we should really pass this.state.scenes into renderOverlay instead of this.props.navigationState. Would you mind doing me a favor, switching that, and fixing the bug? Or in a followup PR if you prefer?

I recently realized we have a bug in the header because the header needs access to this.state.scenes to display the stale/popped scene header as it gets animated away.

Switched to this.state.scenes. But what's the bug?

Ping @ericvicenti

Can you rebase and make sure flow still passes?

@ericvicenti Done.

@facebook-github-bot shipit

Damn. Conflicts again :(

@facebook-github-bot shipit

bot stuck? cc @bestander

@facebook-github-bot shipit

@facebook-github-bot-0 shipit

@facebook-github-bot shipit

@facebook-github-bot import

Yeah, looks like this PR is stuck.
A guess - because it contains changes to binaries and our import had troubles with images.
@mkonicek can you have a look if there is an exception for some reason?

ping @mkonicek
Let's merge this before there are conflicts again 😁

We currently have no way at all to import PRs that touch binary files.
Screenshot from an internal fb group:

Would it make sense to send the PR without images for now so I can merge it easier?

@mkonicek I can do that. But I still need to move and rename an image (hope it works).
We need the images though. So we need to add them anyways.

Even moving and renaming binary files won't work I'm afraid :(
Should find time to try to merge this manually. Please hold on and hope the bug will get fixed.

@mkonicek Whoa! Hope it gets fixed soon.

@satya164 I've merged this internally but it broke some tests so we had to revert unfortunately. Here is the test run: https://travis-ci.org/facebook/react-native/builds/113225174

@mkonicek No probs. I'm working on fixing the Flow errors right now. I'll uodate the PR in some time.

@bestander @mkonicek Fixed the Flow errors. Also rebased against master.

UIExplorer runs fine too.

Hi @satya164 :
Your commits looks good, thank you for doing this!.
However, we're working on APIs changes internally and merge your commits will immediately break a lot our our internal apps.
Please do allow us spending couple more days to unify the renderer APIs and we'll try to merge your commits.
In case you're wondering what the APIs will be look like, bellow is the snippet of the type definitions that we plan to apply soon.
export type NavigationState = {
  key: string,
};

export type NavigationParentState = {
  index: number,
  key: string,
  children: Array<NavigationState>,
};

export type NavigationAction = {
  type: string,
};

export type NavigationLayout = {
  height: NavigationAnimatedValue,
  initHeight: number,
  initWidth: number,
  width: NavigationAnimatedValue,
};

export type NavigationScene = {
  index: number,
  isStale: boolean,
  state: NavigationState,
};

export type NavigationSceneRendererProps = {
  layout: NavigationLayout,
  state: NavigationState,
  onNavigate: (action: NavigationAction) => void,
  position: NavigationAnimatedValue,
  scenes: Array<NavigationScene>,
};

// Object Instances

export type NavigationAnimatedValue = {
  __getValue: () => number,
};

// Functions.

export type NavigationReducer = (
  state: ?NavigationState,
  action: ?NavigationAction,
) => NavigationState;

export type NavigationSceneRenderer = (
  props: NavigationSceneRendererProps,
) => ?ReactElement;

export type NavigationAnimationSetter = (
  position: NavigationAnimatedValue,
  newState: NavigationParentState,
  lastState: NavigationParentState,
) => void;

@hedgerwang Okies. No problem. I hope it gets merged before next RC though :)

nudge nudge, still red

@bestander Damn. New errors after fixing the example. Anyways, fixed them. But I think we need to hold of before merging this until the NavigationExperimental API changes are done, as @hedgerwang said.

Yeah, let's do it after 0.22 branch cut which is on Tuesday.

@bestander Awesome!

BTW what happened to the bot which was commenting on Flow errors earlier? @bestander

It is done by this weird string https://github.com/facebook/react-native/blob/master/.travis.yml#L32.
I would not be surprised if it broke down, needs to be tested.
On the other hand I remember there was a limitation that it does not follow updates properly for ESLing, i.e. it was only reporting for the first commit and then ignoring when you fixed it.

@bestander Yeah, know that bug, where sometimes the line number on which it reported the errors stay unchanged, and hence GH doesn't remove the comments from that line. Still very helpful though.

Could you please create an example PR with Flow errors that should be
reported by the bot?
I'll investigate.
On Fri, Mar 4, 2016 at 8:18 PM, Satyajit Sahoo<EMAIL_ADDRESS>wrote:

@bestander https://github.com/bestander Yeah, know that bug, where
sometimes the line number on which it reported the errors stay unchanged,
and hence GH doesn't remove the comments from that line. Still very helpful
though.
—
Reply to this email directly or view it on GitHub
https://github.com/facebook/react-native/pull/5971#issuecomment-192447843
.

@bestander Sure. Will do in a minute.

ping @hedgerwang @ericvicenti

@hedgerwang @ericvicenti Rebased against master. There is one error with Flow, so it's not in a  mergeable state right now.

@ericvicenti @hedgerwang Updated

Ok, this looks good!

@facebook-github-bot shipit


New provider: Timeweb

Hi! I created new provider Timeweb, they API https://timeweb.cloud/api-docs#tag/Domeny/operation/getDomainDNSRecords

Awesome, thank you! I will create a repo and give you push access to it. I'll also add a repo for the caddy module in case you're interested in that.


Update documentation file.

Previous documentation had outdated screenshots and instructions

Task: Replace Documentation File

Merging by jonatha'ns request


Private Link GA or Public Preview

So here is a link that proves that Private Link is GA for AI + Machine Learning:
https://docs.microsoft.com/en-us/azure/machine-learning/how-to-manage-workspace?tabs=azure-portal#networking
what should we believe?

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

ID: 2caff87f-9f38-988a-12f3-fc96798e3c99
Version Independent ID: 80ab1cdf-282d-29d6-c146-68d1ce8d00b8
Content: Manage workspaces in portal or Python SDK - Azure Machine Learning
Content Source: articles/machine-learning/how-to-manage-workspace.md
Service: machine-learning
Sub-service: core
GitHub Login: @sdgilley
Microsoft Alias: sgilley

Hello @thebeautiful , it says "the private endpoint is on public preview". Where you saw the "GA"? I want to make the document consistent, please let me know. Thanks.

We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.

Hi YutongTie,
I think I wasnt clear, sorry for that. I might not remember what I saw last week, but at least we have the following:
In the blue box here (scroll up a few lines) you see that "private endpoint with Azure Machine Learning workspace" is in public preview
In this link you see that multiple private endpoint is in preview, but in the same article it does not mention that single endpoint is in preview (which might make us believe it's generally available).


chore: minor bug fixes

This PR resolves []
Description
This PR makes the following changes:

Update dimensions order to match my c
Refresh my collection after submission
Add spacing after provenance
Fix screen cropping after scrolling

PR Checklist

[ ] I have tested my changes on iOS and Android.
[ ] I hid my changes behind a feature flag, or they don't need one.
[ ] I have included screenshots or videos, or I have not changed the UI.
[ ] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

Warnings

:warning:

❌ An error occurred while validating your changelog, please make sure you provided a valid changelog.

  Generated by :no_entry_sign: dangerJS against 2e51fe93cac256924649efbe522107b216189728

Makes sense - I pushed this after we had the call with Barney over zoom and didn't separate it into different PRs. I updated the title of the PR though and hope the commits helped reviewing it


tracer won't start on mac os

I'm trying to install traccar server on mac OS. i have installed JDK and tried running the commands mentioned on the site.
the first one runs successfully, but the second one gives a prompt that java is not installed.
Abids-MacBook-Air:traccar-macosx-64-3.5 Satguru$ sudo ./traccar.run
Verifying archive integrity... All good.
Uncompressing traccar  100%
./setup.sh: line 7: [: -lt: unary operator expected
Unable to find any JVMs matching version "(null)".
No Java runtime present, try --request to install.
Detected Mac OSX:
The traccar daemon is already installed.
Abids-MacBook-Air:traccar-macosx-64-3.5 Satguru$

Looks like you don't have Java installed. What's the output of the following command:
java -version

Thanks. I had the wrong version of java installed. It works now. Thank you for your help.


I am so confused by everything

What do all of these keys mean?

only few of them are actually specified, and the documentation doesn't do anything to help.

i simply printed the responces(in this case variable "history") before writing other part about iterating through history

I will update the documentation, you can output function responses, so you can understand what is needed for what

I've managed to sort it out in the end. :)


Home Screen

ios small devices
Why there is movement it should behave like a screen
https://user-images.githubusercontent.com/107177634/222045813-3f0bb3cf-7067-4218-b4ab-cf5f79dc281f.MP4

Completed


ARROW-2432: [Python] Fix Pandas decimal type conversion with None values

This fixes conversion of Pandas decimal types to Arrow with None values. Previously, if the type was specified, an error would occur when checking if the object was Decimal. If the type was not specified, a segmentation fault would occur when attempting to find the max precision and scale.
Added new tests which include None values for both the above cases.

Ping @cpcloud and @pitrou, please take a look when you can, thanks!

Thanks @pitrou!


Webcomics extension doesn't work

Source information
Webcomics 1.4.3
Source language
English
Steps to reproduce
Search does display results (without covers), but not always correct results. Search sometimes only comes out with this same manga title, no matter what you were searching for. Also, when you go into one of the search results, they're blank.
Expected behavior
Should display cover in search, and the page of the comic should have a description and the chapters displayed.
Actual behavior
No cover, no description, no chapters, and the error message "No chapters found" displays for every comic.
Mihon/Tachiyomi version
Tachiyomi 0.15.3
Android version
Android 12
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.15.3.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

There's "No chapters found" error


Reconnect WiFi client

Hi,
in the Unifi App and Webinterface you have the option to reconnect a WiFi client initiated by the AP. This is useful if you have IoT devices that sometime hangup in the WiFi connection. (can be recognized if the "experience" value of the client goes down)
If you press "reconnect" the AP drops the connection of the client and the client reconnects and everything is fine. I would like to automated this "reconnect", but the adapter is missing this functionality.

Would be a perfect and very useful feature

I think it would be a very useful feature for many people. I hope it will be available soon.


Improve incremental builds by 710% by dropping webpack

I spent this morning messing with drawing a voting screen, which for someone who isn't good at geometry involves a lot of trial and error, and the incremental build perf started to really get to me.
So, currently we're using webpack to bundle our JS for production, and for a devserver. That came in from the wasm-pack starter kit so I figured it was a reasonable enough choice. However, we've got just a couple KB of JS, basically the entire project is written in Rust, and it seemed like webpack was taking a decent amount of time to build, so I wanted to see if we could do better.
I started out doing a few timing runs of incremental builds and wasm-pack -> webpack commands were together taking ~12s. The first thing I tried was to upgrade to webpack 5, which is supposed to have better caching and maybe be faster. Looks like it makes our build about 1s slower, so that's no help.
Looking at the output that wasm-pack was producing, it included a synchronous import of the generated .wasm file which isn't yet standard (though import assertions are probably going to unblock making that work, the syntax is likely to be import game from "./client.wasm" assert { type: "wasm" };).
It turns out that wasm-pack by default runs with --target bundler which emits code designed for webpack to consume. Kinda gross to emit pseudostandard output by default, but it means that your imports can all by synchronous which is kinda nice. But there's a --target web version, what's that do?
Well, while the import syntax is worked out, there are some interim methods for loading and instantiating a WebAssembly module. Because they involve loading data from the network they're asynchronous, but that's no big deal, our initialization function is already async, so we can just add an async initWasm() no problem. With that, we can drop webpack entirely, bringing us down from 12s to 4s.
3x faster!? Our 90% Rust project that has one JS file and zero JS dependencies was spending 3s in webpack for every second in the Rust compiler? The infamously slow Rust compiler that everyone warns you about when you say you're gonna try out this new Rust thing?
But then, poking around a bit, it turns out that's just the start! Because I switched us run wasm-pack with the --release flag even during development because it was net faster, as the larger unoptimized wasm file made webpack take tens of seconds to process. With webpack out of the picture we can afford to run in dev mode, which gets the build time down to 1.69s! That's 7x faster!
The difference is night and day, finally I can quickly iterate again!
Now potentially we'd still want to do some bundling, maybe if we add more JS to our repo. Seems like esbuild is a good option there if we go that route. For a devserver I'm using @web/dev-server just for livereload, but it can also downlevel if we end up needing that.

Congrats on the perf! Coming from your tweet, just wanted to say that this project looks really fun. Let me know if you're looking for any beta testers :)

It's a fun project, and some neat things are working (shared code between server and client! TLS in rust! type safe network protocol!), but the game is definitely not at a level I think anyone would call "fun" yet

Was it only really using webpack before to account for commonjs imports? So by the sounds of it because you don't have any, you can just output es modules as-is?
Nice find for sure. Interesting that we often assume we need a bundler when actually we might not (no cjs imports).

Was it only really using webpack before to account for commonjs imports?

It was using it to hide the asynchrony of initializing the wasm module. People want to write import {game} from './game.wasm'; but that's not yet supported.


Minor enhancements

Add CircleCI config
Add eslint and related stuff to devDependencies
Run linting on CI
Move screenshot out of repo
Remove extra activate and deactivate methods
Disable lintOnFly because the linter is executed on a file path
General simplification in error mapping

Instructions for reviewer

Goto circleci.com
Enable CI on this project

Neat, thanks again @steelbrain! The Circle CI error above is caused by https://github.com/eslint/eslint/issues/5476 and is fixed by using eslint: "2.2.x" in the package.json (for now). I’m going ahead and merging this PR—I really appreciate the cleanup and review!

CI can be fixed by https://github.com/steelbrain/linter-proselint/commit/c1214e7311a90b95ebe849f9fa524836adfaabf1


Work around #1769 by doing the DPI/height calculation ourselves.

This change is

:shipit:

:shipit:

Could we get this in today and spin off a full build 👯♂


Duplicate actions in notification controller on Android 13+

Describe the Bug
I've got reports from my users who uses ColorOS 13+(which is the default system on some OnePlus and Oppo devices), they found  duplicate rewind and ff actions in notification controller. After digging for a while, I found KotlinAudio will add custom actions for rewind, ff and stop actions on Android 13+ aligning with the official behavior change, it still attempt to add these actions to ExoPlayer, which may result in duplicate actions on some custom Android system.
I have already create a PR to solve this issue, I'm posting this bug report here to see if there are any scenerios I might have missed.

Steps To Reproduce
Add Capability.JumpForward, Capability.JumpBackward or Capability.Stop to capabilities when invoking updateOptions
Code To Reproduce
Playback.updateOptions({
  forwardJumpInterval: 15,
  capabilities: [
    Capability.Play,
    Capability.JumpForward,
    Capability.JumpBackward
  ]
})

Replicable on Example App?
Yes
Environment Info:
Paste the results of npx react-native info
0.73.8
Paste the exact react-native-track-player version you are using
4.1.1
Real device? Or simulator?
OnePlus Ace3
What OS are you running?
ColorOS 14(Based on Android 14)
How I can Help
See PR

thanks for the PR!
do u know if these specific OS providers map bluetooth events to forward/rewind as well when forward/rewind is enabled? like KeyEvent.KEYCODE_MEDIA_FAST_FORWARD (https://github.com/lovegaoshi/KotlinAudio/commit/a8990b57ae6cff0d85f9139d20d725ffbe224ed8). I wonder if for these OSes its better to use the native forward/rewind mapping instead of via custom actions.
unrelated, the whole customaction system needs to be overhauled at some point.


chore(CI): address uvicorn dropping py36 support

(Works around https://github.com/encode/uvicorn/issues/1327)


label collision in activation summary

there are only three activations in this case, 1 for each of the actions

here's one of the divs

after adding another activation

this view no longer exists

this view no longer exists


Create protocol for manual lesion segmentation

Some relevant discussions:

https://github.com/ivadomed/model_seg_ms_mp2rage/issues/56
https://github.com/ivadomed/model_seg_ms_mp2rage/issues/13
https://github.com/ivadomed/canproco/issues/20

We should use https://github.com/spinalcordtoolbox/manual-correction

Catherine Spino and I are working on manual correction of MS lesion in the spinal cord predicted by the model (refer to ACTRIMS abstract). So far, the proposed method is the following :

Download SCT
Clone manual correction repository
Download segmentation predictions : from `duke/projects/canproco/2023_11_08_lesion_seg_models_and_results/M12_inference/inference_results_Dataset555_nnUNet222_fold2_2d_BIDS``
Download the config file (looks like the following):

FILES_LESION:

sub-cal056_ses-M12_STIR.nii.gz
sub-cal072_ses-M12_STIR.nii.gz
sub-edm011_ses-M12_PSIR.nii.gz
sub-edm013_ses-M12_PSIR.nii.gz
sub-mon001_ses-M12_PSIR.nii.gz
sub-mon004_ses-M12_PSIR.nii.gz
sub-tor006_ses-M12_PSIR.nii.gz
sub-tor007_ses-M12_PSIR.nii.gz
sub-van005_ses-M12_PSIR.nii.gz
sub-van010_ses-M12_PSIR.nii.gz

Create a virtual environment  :

python3 -m venv venv
source venv/bin/activate
pip install -r requirements.txt

Run the following command for manual correction :

python manual_correction.py -path-img <INPUT_PATH> -config <CONFIG_FILE>  -path-label <LABEL_PATH> -suffix-files-lesion _lesion-manual

However, we identified certains things to do beforehand:

[ ] Copy predictions to CanProCo branch
[ ] Create a json file for each file with generated by "Generated by 2D nnUNet model available at duke/projects/canproco/2023_11_08_lesion_seg_models_and_results/nnunet_region-based_(SC+lesion)_training_on_STIR_and_inverted_PSIR_data/nnUNet_results/Dataset222_RegionBasedLesionSeg/nnUNetTrainer__nnUNetPlans__2d model used in ACTRIMS 2024 abstract"
[ ] Modify manual_correction script to change Fsleyes parameters : such as the intensity parameters depending on the contrast
[ ] Manually correct images

@jcohenadad Could you give me feedback on the json file data for the automatically segmented lesions

Great summary! 👍🏻
For SCI derivatives (segmented using the SCISeg model and then manually corrected), I use the following jSON sidecar:
{
    "GeneratedBy": [
        {
            "Name": "SCISeg nnUNet3D",
            "Version": "Dataset183_tSCI3SitesALPhase2Seed710",
            "Date": "2024-01-16"
        },
        {
            "Author": "Jan Valosek",
            "Note": "Manually corrected",
            "Date": "2024-01-18 10:42:24"
        }
    ]
}

Or, if a mask is created completely manually, I use:
{
    "GeneratedBy": [
        {
            "Author": "Jan Valosek",
            "Date": "2024-01-18 10:42:24"
        }
    ]
}

Unfortunately, manual-correction.py supports only the second JSON option. I usually write a simple loop to add info about a model to the JSON sidecar.
Also see our intranet convention here.

Modify manual_correction script to change Fsleyes parameters : such as the intensity parameters depending on the contrast

This is already supported; see the '-flies-dr' flag here.

In our case, what do you think of ?
{
    "GeneratedBy": [
        {
            "Name": "nnunet_region-based_(SC+lesion)_training_on_STIR_and_inverted_PSIR_data nnUNet2D",
            "Version": "Dataset222_RegionBasedLesionSeg",
            "Date": "2024-01-16"
        },
        {
            "Author": "Pierre-Louis Benveniste",
            "Note": "Manually corrected",
            "Date": "2024-01-18 10:42:24"
        }
    ]
}

"Name": "nnunet_region-based_(SC+lesion)_training_on_STIR_and_inverted_PSIR_data nnUNet2D" --> ideally it should point to the actual model (eg: URL to the model hosted on the assets of a published release on github)
"Note": "Manually corrected", --> i don't think this field is necessary

Release done !  Format of json files will be :
{
    "GeneratedBy": [
        {
            "Name": "2D nnUNet model stored at https://github.com/ivadomed/canproco/releases/tag/r20240125",
            "Version": "2D region-based nnUNet fold 2",
            "Date": "2024-01-16"
        },
        {
            "Author": "Pierre-Louis Benveniste",
            "Date": "2024-01-18 10:42:24"
        }
    ]
}

Great! I have a comment about naming the model https://github.com/ivadomed/canproco/issues/74
Also, putting the version URL under "Name" defeats the purpose of the field "Version".
How about instead putting the name of the model (ie: the packaged ZIP file, like model_ms_seg_sc-lesion_regionBased.zip) under "Name", and the URL to the release under "Version"?

"Note": "Manually corrected", --> i don't think this field is necessary

I used this because, in some cases, the automatic seg was okay and needed no corrections. To distinguish this, I used:
    "Note": "Visually verified",

or, if manual correction was required:
    "Note": "Manually corrected",

"Note": "Visually verified",

I agree, this is a good idea. We should probably add this information in our intranet so as to standardize our terminology @NathanMolinier
BTW: Does your manual_correction script automatically identifies if there was a modification in the segmentation, and then applies either "Visually verified" or "Manually corrected"?

BTW: Does your manual_correction script automatically identifies if there was a modification in the segmentation, and then applies either "Visually verified" or "Manually corrected"?

Yes! The script tracks the modification times of a file to distinguish if the file was just opened or actually modified.

The model name was updated in the release.
Segmentations were added to branch plb/m12_lesion_seg and pushed to git-annex
Json files were created for each predicted segmentation

The json files have the following format :
{
        "GeneratedBy": [
            {
                "Name": "2D nnUNet model model_ms_seg_sc-lesion_regionBased.zip",
                "Version": "https://github.com/ivadomed/canproco/releases/tag/r20240125",
                "Date": "2024-01-26"
            }
        ]
    }

"Note": "Visually verified",

I agree, this is a good idea. We should probably add this information in our intranet so as to standardize our terminology @NathanMolinier
BTW: Does your manual_correction script automatically identifies if there was a modification in the segmentation, and then applies either "Visually verified" or "Manually corrected"?

Regarding this point, I discussed with @valosekj and I think that we should stick to our standards and use the field "Name" instead of "Note". For more information about this please check this comment in manual correction.
Manual correction script needs to be updated for sure regarding json sidecars.

The field "SpatialReference": "orig" should also be added

Great idea @NathanMolinier ! However, because of consistency, I suggest adding this field to the entire dataset and not just to these specific segmentations. An issue was created #75

Yes indeed ! All the json sidecars should have this "SpatialReference" 👍

Regarding the "SpatialReference", you could test manual-correction PR #76, which now adds "SpatialReference" automatically.


Update javadoc workflow

There was a javadoc plugin behavior change from 3.8 to 3.10 (there doesn't seem to be a 3.9 release).
The diff shows that they changed site to reports:
https://github.com/apache/maven-javadoc-plugin/compare/maven-javadoc-plugin-3.8.0...maven-javadoc-plugin-3.10.0


"type 'String' is not a subtype of type 'ResponseBody?' in type cast" when using dio.download.

New Issue Checklist

[x] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
e.g. flutter / ios / android
flutter

Platform Version
e.g.  1.5.0 / 12.0 / 9.0
3.3.1

Dio Version
e.g.  2.1.0 / 1.0.17
4.0.6

Android Studio / Xcode Version
e.g. Android Studio 3.3.2 / Xcode 10.2.1

Repro rate
e.g. all the time (100%) / sometimes x% / only once
100%

Repro with our demo prj
e.g. does it happen with our demo project?

Demo project link
e.g. link to a demo project that highlights the issue

Issue Description and Steps
We are using the following code to download a file:
dio.download(
        'https://file-examples.com/storage/feb2e515cc6339d7ba1ffcd/2017/10/file-sample_150kB.pdf',
        './savePath/test.pdf',
      );

The problem we are having is that we get an error:
DioError [DioErrorType.other]: type 'String' is not a subtype of type 'ResponseBody?' in type cast

When using an interceptor to log the response, we see that the body is the content of the file in plain text. Are we using download wrong, or why do we get this error?

i am facing same issue?? do you know ho to solve??

i am facing same issue?? do you know how to solve??
if i click again then it download but for first time it throws an error
flutter: DioError [DioErrorType.other]: type 'String' is not a subtype of type 'ResponseBody?' in type cast
#0 DioMixin.assureResponse
package:dio/src/dio_mixin.dart:818
https://github.com/cfug/dio/issues/1 DioMixin.fetch.
package:dio/src/dio_mixin.dart:630
https://github.com/cfug/dio/issues/2 DioForNative.download
package:dio/…/entry/dio_for_native.dart:84

@AlexV525  do you know how to solve it ???

I got same problem and solved it.
The cause of the error was that option.responseType was not inherited when retrying with Interceptor, and I found that I could avoid the error by explicitly specifying option.responseType.
The code to avoid the error is as follows.
  Future<Response<dynamic>> _retry(RequestOptions prevOptions) async {
    final options = Options(
        method: prevOptions.method,
        headers: prevOptions.headers,
        receiveDataWhenStatusError: prevOptions.receiveDataWhenStatusError,
        responseType: prevOptions.responseType);
    return _dio.request(prevOptions.path,
        data: prevOptions.data, queryParameters: prevOptions.queryParameters, options: options);
  }

Thanks @tijins it worked

I got same problem and solved it.
The cause of the error was that option.responseType was not inherited when retrying with Interceptor, and I found that I could avoid the error by explicitly specifying option.responseType.
The code to avoid the error is as follows.
  Future<Response<dynamic>> _retry(RequestOptions prevOptions) async {
    final options = Options(
        method: prevOptions.method,
        headers: prevOptions.headers,
        receiveDataWhenStatusError: prevOptions.receiveDataWhenStatusError,
        responseType: prevOptions.responseType);
    return _dio.request(prevOptions.path,
        data: prevOptions.data, queryParameters: prevOptions.queryParameters, options: options);
  }

I have the same problem, but only on Windows desktop. MacOS and Android/iOS seem to work fine. Where in my code should I apply this code block?


Documentation still refers to removed spring-vault-dependencies artifact

I cannot find org.springframework.vault:spring-vault-dependencies for the latest version, but it is mentioned in the documentation (section 7.1, example 1).

This is an oversight in the docs, we removed spring-vault-dependencies with 3.0 and failed to update the documentation.


The documentation regarding Azure PowerShell installation is outdated

Prerequisites

[X] Existing Issue: Search the existing issues for this repository. If there is an issue that fits your needs do not file a new one. Subscribe, react, or comment on that issue instead.
[X] Descriptive Title: Write the title for this issue as a short synopsis. If possible, provide context. For example, "Typo in Get-Foo cmdlet" instead of "Typo."
[X] Verify Version: If there is a mismatch between documentation and the behavior on your system, ensure that the version you are using is the same as the documentation. Check this box if they match or the issue you are reporting is not version specific.

Links
https://learn.microsoft.com/en-us/powershell/azure/servicemanagement/install-azure-ps?view=azuresmps-4.0.0
Summary
The steps required to install and login to Azure using the Azure Powershell module are outdated, and do not take into account dependencies and the current need to enable TLS 1.2.
Details
TLS 1.2 must be enabled for the commands to interact correcty with the repositories and with the Azure infrastructure.
Repository NuGet must be installed on PowerShell 5.1 for the Azure module to be found and installed.
Suggested Fix
Correct steps, as validated during a case resolution:
Set-ExecutionPolicy -ExecutionPolicy RemoteSigned -Scope CurrentUser
[Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12
Install-PackageProvider -Name NuGet
Install-Module -Name Azure -AllowClobber
Add-AzureAccount

@OrlandoAMunoz
Thanks for your feedback! We will investigate and update as appropriate.
I've delegated this to @mikefrobbins, a content author, to review and share their valuable insights.

@OrlandoAMunoz Closing due to this being a duplicate issue. See #2593 for tracking of this problem.

Hi Mike,
The #2593 was posted by mistake with my personal account.
Please consider this one #2593 posted with my work @.***) account.
Otherwise it will not count for my performance review.
Kind regards,
Orlando Alvarez Muñoz
Support Engineer
Microsoft Azure Technical Support
Customer Service and Support
Office: +351 (21) 0491844
<EMAIL_ADDRESS>Working hours: 8:30-17:30 UTC+1
If you need to work with another Support Engineer outside of my working hours, please send email to<EMAIL_ADDRESS>with your case number, and availability.
We are always interested to hear your feedback. Please feel free to reach my manager regarding the level of service you have received, at<EMAIL_ADDRESS>.
[Microsoft Azure logo horizontal]
From: Mike F. Robbins @.>
Sent: Tuesday, June 6, 2023 20:25
To: MicrosoftDocs/azure-docs-powershell @.>
Cc: Orlando Muñoz @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs-powershell] The documentation regarding Azure PowerShell installation is outdated (Issue #2594)
@OrlandoAMunozhttps://github.com/OrlandoAMunoz Closing due to this being a duplicate issue. See #2593https://github.com/MicrosoftDocs/azure-docs-powershell/issues/2593 for tracking of this problem.

Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs-powershell/issues/2594#issuecomment-1579326415, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXZ7ICZBFJLJ46MAKA6OL5DXJ57Y7ANCNFSM6AAAAAAY4KQSAM.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@OrlandoAMunoz, The issue with enabling TLS 1.2 and installing NuGet on Windows PowerShell 5.1 is due to the PowerShellGet module requiring an update. I've added a link to Update PowerShellGet for Windows PowerShell 5.1 in the prerequisites section to resolve this issue (see PR #2593).

Set-ExecutionPolicy -ExecutionPolicy RemoteSigned -Scope CurrentUser <<- Already part of the prerequisites section
[Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12  <<- Covered by updating PowerShellGet
Install-PackageProvider -Name NuGet <<- Covered by updating PowerShellGet
Install-Module -Name Azure -AllowClobber <<- Already part of the installation section
Add-AzureAccount <<- Already part of the sign in section

Hi Mike,
Thank you for the feedback 😊
Kind regards,
Orlando Alvarez Muñoz
Support Engineer
Microsoft Azure Technical Support
Customer Service and Support
Office: +351 (21) 0491844
<EMAIL_ADDRESS>Working hours: 8:30-17:30 UTC+1
If you need to work with another Support Engineer outside of my working hours, please send email to<EMAIL_ADDRESS>with your case number, and availability.
We are always interested to hear your feedback. Please feel free to reach my manager regarding the level of service you have received, at<EMAIL_ADDRESS>.
[Microsoft Azure logo horizontal]
From: Mike F. Robbins @.>
Sent: Monday, June 12, 2023 18:22
To: MicrosoftDocs/azure-docs-powershell @.>
Cc: Orlando Muñoz @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs-powershell] The documentation regarding Azure PowerShell installation is outdated (Issue #2594)
@OrlandoAMunozhttps://github.com/OrlandoAMunoz, The issue with enabling TLS 1.2 and installing NuGet on Windows PowerShell 5.1 is due to the PowerShellGet module requiring an update. I've added a link to Update PowerShellGet for Windows PowerShell 5.1https://learn.microsoft.com/powershell/gallery/powershellget/update-powershell-51 in the prerequisites section to resolve this issue (see PR #2593https://github.com/MicrosoftDocs/azure-docs-powershell/issues/2593).
Set-ExecutionPolicy -ExecutionPolicy RemoteSigned -Scope CurrentUser <<- Already part of the prerequisites section
[Net.ServicePointManager]::SecurityProtocol = [Net.SecurityProtocolType]::Tls12 <<- Covered by updating PowerShellGet
Install-PackageProvider -Name NuGet <<- Covered by updating PowerShellGet
Install-Module -Name Azure -AllowClobber <<- Already part of the installation section
Add-AzureAccount <<- Already part of the sign in section
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs-powershell/issues/2594#issuecomment-1587749831, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AXZ7ICZPQUKRHKTWVUHYURDXK5F2RANCNFSM6AAAAAAY4KQSAM.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


fix issue #2890

Codecov Report
Attention: 19 lines in your changes are missing coverage. Please review.

Comparison is base (00e83fe) 77.31% compared to head (e982920) 77.27%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3002      +/-   ##
==========================================
- Coverage   77.31%   77.27%   -0.05%
==========================================
  Files         645      638       -7
  Lines       54347    54149     -198
==========================================
- Hits        42019    41842     -177
+ Misses      10217    10206      -11
+ Partials     2111     2101      -10

Flag
Coverage Δ

go-1.18-386
77.26% <69.84%> (-0.14%)
:arrow_down:

go-1.18-amd64
?

go-1.19-amd64
?

go-1.20-386
?

go-1.20-amd64
77.23% <69.84%> (-0.14%)
:arrow_down:

go-1.21-386
?

go-1.21-amd64
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

util/gvalid/gvalid.go
81.25% <ø> (+2.67%)
:arrow_up:

util/gvalid/gvalid_register.go
23.68% <ø> (-10.94%)
:arrow_down:

util/gvalid/gvalid_validator.go
78.83% <100.00%> (-5.60%)
:arrow_down:

util/gvalid/gvalid_validator_check_map.go
86.71% <100.00%> (-2.00%)
:arrow_down:

util/gvalid/gvalid_validator_check_struct.go
81.31% <100.00%> (-4.24%)
:arrow_down:

util/gvalid/gvalid_validator_check_value.go
93.02% <100.00%> (-5.64%)
:arrow_down:

util/gvalid/internal/builtin/builtin.go
100.00% <ø> (ø)

util/gvalid/internal/builtin/builtin_enums.go
50.98% <42.42%> (-10.56%)
:arrow_down:

... and 12 files with indirect coverage changes

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


Add links to manuals

Overview
TL;DR
We are open sourcing the training manuals. This PR adds links to those manuals, so that the training-kit landing page is a resource for finding other learning content in addition to cheat sheets.
Questions
<If relevant, write a list of questions that you would like to discuss related to the changes that you have made.>
N/A
Next Steps
<If incomplete, create a task list of items that are still being worked on within the Pull Request.>
As we move the training manual for developers into the GitHub org, we will need to update the link here.
Review
<If complete, or ready for :eyes:, use @mentions for quick questions, specific feedback, and progress updates.>
cc @parkerbxyz @github/ps-programs for awareness and review 👀

🎉 🎉 🎉

@github/ps-programs @parkerbxyz Would anyone have a few minutes to review this? Once this is merged, I think we can build on it with #765, and also configure a redirect from training.github.com to the landing page for training kit.

click click - double feature click - wrong, false


Write usage documentation

Need to write usage documentation in the README. It should include:

any differences from the API of Express (notably, mounting sub-routers)
examples of middleware, request handlers, error handlers
hello world

Explanation of the event paths you need, e.g.:
functions:
   api:
      name: 'your-fn-name'
      handler: src/YourAPI.handler
      memorySize: 512
      timeout: 2
      events:
         - http: 'ANY /'
         - http: 'ANY /{proxy+}'


Uncontrolled memory consumption

Bug report
Host type : VM
OS : Ubuntu 16.04
4 vCPU
25 GB of memory
Influxdb data disk : 500GB of SSD
Influxdb version : 1.0.0-beta3
Steps to reproduce:
We have multiple source of data :

telegraf (50GB of data)
prometheus (99 GB of data)
heapster (< 3GB of data)

Expected behavior:
We expect influxdb not to consume more memory than available in the system.
Actual behavior:
Very few minutes after startup, influxdb is eating more memory than the 25GB of RAM and get OOM killed.
This system has been working in production with great stability for a long time.
We do not use yet CQ so we are basically appending data.
We tried to purge some measurements but Influxdb got killed before dropping data.
Gists:
block : https://gist.github.com/rvrignaud/40ec0a3e2a36356e85a108d52d797d2b
goroutines : https://gist.github.com/rvrignaud/c8a96202628f0ec39518246d43e99961
iostat : https://gist.github.com/rvrignaud/920bc3d158d46b23d05d68ddf8bffdbe
vars : https://gist.github.com/rvrignaud/25c51d774f817c493ce7e6f9c2753e14

Can you also collect the following as well as attach the log with the full trace when you got the OOM?
curl -o heap.txt "http://localhost:8086/debug/pprof/heap?debug=1"
influx -execute "show shards" > shards.txt
influx -execute "show stats" > stats.txt
influx -execute "show diagnostics" > diagnostics.txt

Hi @jwilder, thanks for your answer! We upgraded the node from 26Gb of RAM to 52Gb  (so we don't hit again the issue for the moment :innocent: ) but the logs logs are here.

https://gist.github.com/sbadia/0cceb4dfd94029a3b6dd6e8dfa5d6fab#file-heap-txt
https://gist.github.com/sbadia/0cceb4dfd94029a3b6dd6e8dfa5d6fab#file-shards-txt
https://gist.github.com/sbadia/0cceb4dfd94029a3b6dd6e8dfa5d6fab#file-stats-txt
https://gist.github.com/sbadia/0cceb4dfd94029a3b6dd6e8dfa5d6fab#file-diagnostics-txt

I've tried influxdb from 0.13 up to 1.0.nightly on 36GB RAM Linux VM. Once it consumed 10mln rec. stocks log it OOM killed and now it crashes persistently. Upon start the process consumes all RAM available until next OOM.
Dump is here https://gist.github.com/eodin/52d0af58cd29f35dc10b3f80a0e5d10f
It looks promising but unstable :-(

@eodin I'd recommend using the 1.0rc1 build and not 0.13 going forward.

Are you writing historical data?
Are you overwriting points w/ the same timestamp?
Can you describe your writes, perhaps provide some sample writes in line protocol format?

I've reinstalled clean copy but it come to OOM again in couple of minutes. So I've shutdown grafana dashboard and now I ingested already 20mln of 100mln of historical records w/o any issue. Mem consumption is about 700Mb.
Seems some grafanas query kill it to dead. I'll continue the debug once dataset will be fulfilled

@eodin In 0.13, querying across a lot of shards can OOM the process.  1.0.0rc1 has some fixes for that, but it's still possible if you have sparse data and shard durations are not configured correctly.  If you could provide some details on your writes, that would help determine whether these is an  issue with your schema.
If queries are the problem, there are a number of limits you can enable to identify and kill the problem ones.
[coordinator]
  max-concurrent-queries = 0 # limits the number of concurrent queries, 0 is disables
  query-timeout = "0" # kills queries after a certain duration, "0" disables
  log-queries-after = "0"# logs slow queries after a certain duration, "0" disables
  max-select-point = 0 # kills queries that return more than a certain number of points, 0 disables
  max-select-series = 0 # kill queries that query more than a certain number of series, 0 disables
  max-select-buckets = 0 # kill queries that return more than a certain number of time buckets, 0 disables

Is this still occurring w/ 1.2.2?

Same issue with 1.6 and prometheus integration via remote write, 32GB is reached in a matter of minutes

@jwilder
I am running the influxdb in the docker container and when I check the process usage, I could see the influxd process taking high memory consumption even if it is not used. When I use the influxDB for inserting the data, the memory usage goes very high and not getting released even after the process is completed.
My config:
8vCPU,30G Ram
InfluxDB Version=1.6.3
Please suggest me the solution for this issue


fix(cos): add file upload for deploy

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


RET-2151: ET1 Vetting - Display saved data in ECM tab

https://tools.hmcts.net/jira/browse/RET-2151
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x ] No

Vetting tab showed before vetting event happened, restored branch to fix this now.


Something like v-bind, but also for other directives.

What problem does this feature solve?
Cleaner template code, especially with libs like Vuetify, heavily depending on bunch of props / directives.
What does the proposed API look like?
<template>
  <my-comp expand="attrs.myComp">
    <nested-comp expand="attrs.nested" />
  </my-comp>
</template>

<script>
export default {
  computed: {
    attrs () {
      return {
        myComp: {
          'v-on:click': this.myCallback
          class: 'some css classes'
          // bunch of other props / directives
        },
        nested: {
          // same here
        }
      }
    }
  }
}
</script>

you are probably looking for v-bind and v-on with no arguments: v-bind="{ value: myValue, disabled: isDisabled }"

Unfortunately, I'm not. As I said, I'm looking something like v-bind, not for v-bind. Because in v-bind you can use props only, not the directives too.

If you need that much dynamicity, it's probably time to use render functions with js 🙂


Seeking advice on optimizing user authorization with RBAC using Casdoor Go SDK

Hello,
I'm currently exploring the best practices for implementing user authorization in a project utilizing the Casdoor Go SDK, particularly within the context of the Role-Based Access Control (RBAC) model. As I am relatively new to this area, I am keen to learn from your expertise to ensure I'm adopting the most efficient and secure approach.
My current setup involves the default model configuration as follows:
[request_definition]
r = sub, obj, act

[policy_definition]
p = sub, obj, act

[role_definition]
g = _, _

[policy_effect]
e = some(where (p.eft == allow))

[matchers]
m = g(r.sub, p.sub) && r.obj == p.obj && r.act == p.act

For path access control, I specify permissions in the router like this:
http.HandleFunc("/private", tokenAuthMiddleware(privateHandler, "service1.read"))

And I perform the permission check as shown below:
func tokenAuthMiddleware(next http.HandlerFunc, requiredPermission string) http.HandlerFunc {
	return func(w http.ResponseWriter, r *http.Request) {
		authHeader := r.Header.Get("Authorization")
		if authHeader == "" || !strings.HasPrefix(authHeader, "Bearer ") {
			http.Error(w, "Unauthorized: No token provided", http.StatusUnauthorized)
			return
		}

		token := strings.TrimPrefix(authHeader, "Bearer ")
		if !validateTokenWithCasdoor(token, requiredPermission) {
			http.Error(w, "Unauthorized: Invalid or expired token", http.StatusUnauthorized)
			return
		}

		next(w, r)
	}
}

func validateTokenWithCasdoor(token string, requiredPermission string) bool {
	claims, err := casdoorsdk.ParseJwtToken(token)
	if err != nil {
		fmt.Println("Error validating JWT token:", err)
		return false
	}

	user, _ := casdoorsdk.GetUser(claims.User.Name)
	for _, perm := range user.Permissions {
		if perm.Name == requiredPermission && perm.IsEnabled {
			fmt.Println("Permission verified for user with claims:", claims)
			return true
		}
	}

	fmt.Println("Permission not found or not enabled for user:", claims.Name)
	return false
}

I am confident that there might be a more optimized or standardized method to handle this process, especially in verifying roles/permissions effectively. Could you provide any recommendations or best practices on this matter? Additionally, if there are any improvements or alternative approaches to the current method I'm using, I'd be greatly appreciative of your guidance.
Thank you for your time and support.

@nemirlev why don't you just use the permissions property inside the JWT? I think the JWT already contains permissions by default? See:
https://github.com/casdoor/casdoor/blob/57431a59ad67d5c992d505f261d1bd14470e9b03/object/token_jwt.go#L363-L387
If you are using "JWT" format option, you already have:
https://github.com/casdoor/casdoor/blob/57431a59ad67d5c992d505f261d1bd14470e9b03/object/token_jwt.go#L242-L244

@hsluoyz Hello.
Because the token is transferred from the client to the server.  It is stored in cookies, and this is a problem. There is a size limit. We can use roles instead, and we will solve this problem, but then we will not have atomicity.

You should store it in local storage then

@hsluoyz It's not safe.
https://auth0.com/docs/secure/security-guidance/data-security/token-storage#don-t-store-tokens-in-local-storage


Update sbt-updates to 0.6.2

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

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.timushev.sbt", artifactId = "sbt-updates" } ]

labels: sbt-plugin-update, early-semver-minor, semver-spec-patch, commit-count:1

Codecov Report

Merging #223 (4a97a20) into main (5b87f68) will increase coverage by 0.80%.
The diff coverage is 23.07%.

@@            Coverage Diff             @@
##             main     #223      +/-   ##
==========================================
+ Coverage   59.31%   60.11%   +0.80%
==========================================
  Files         153      153
  Lines        3200     3184      -16
  Branches       96       93       -3
==========================================
+ Hits         1898     1914      +16
+ Misses       1302     1270      -32

Impacted Files
Coverage Δ

...c/main/scala/surge/core/SurgePartitionRouter.scala
100.00% <ø> (ø)

...surge/internal/core/SurgePartitionRouterImpl.scala
22.22% <ø> (-1.22%)
:arrow_down:

...ain/scala/surge/javadsl/command/AggregateRef.scala
0.00% <ø> (ø)

...in/scala/surge/javadsl/command/CommandModels.scala
15.38% <ø> (-1.29%)
:arrow_down:

...ain/scala/surge/javadsl/command/SurgeCommand.scala
0.00% <0.00%> (ø)

...la/surge/javadsl/command/SurgeCommandBuilder.scala
0.00% <0.00%> (ø)

...a/surge/javadsl/common/AggregateRefBaseTrait.scala
0.00% <ø> (ø)

.../main/scala/surge/javadsl/event/AggregateRef.scala
0.00% <ø> (ø)

...rc/main/scala/surge/javadsl/event/SurgeEvent.scala
0.00% <0.00%> (ø)

...in/scala/surge/scaladsl/command/AggregateRef.scala
0.00% <ø> (ø)

... and 24 more

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 f4e2bfe...4a97a20. Read the comment docs.


Xamarin.Forms- API Coverage for iOS and Android

The following properties need to be exposed from the native animation views through to Xamarin Forms AnimationView:

Autoplay
Loop
Duration
IsPlaying

Fixed in [PR #13 ] (https://github.com/martijn00/LottieXamarin/pull/13)


Add #odd and #even to all methods lists

These are new in jQuery 3.5

How do we want to handle this? Mark them as "deprecated" as "This function did not exist until jQuery 3.5"?

Handle what? What's deprecated?

Using .odd() on jquery 3.1 will not work; shouldn't we warn about this kind of breakage?

Well that wouldn't be part of our deprecations, that would be a much bigger project to track when every feature was introduced. We'd have to know that $.find was introduced in 0.2 or whatever. I'm not sure it would be worth it.


Refine grammar, clarity, and precision in documentation

Fixed:
Replaced to with from for accuracy.
Simplified integer result of the signed integer division to signed integer division result.
Updated If the denominator is 0, the result will be 0. to If the denominator equals 0, the result is 0. for improved clarity.
Clarified All values are treated as two’s complement signed to All values are treated as two's complement 256-bit signed for precision.
Changed to sign extend to to be sign-extended for grammatical correctness.
Why it's useful:
These changes enhance precision, grammatical accuracy, and readability, making the text clearer and more professional.

I'm sorry, I fixed it


More documentation touchups

Fixes KOL-2499
Fixes KOL-2511
Fixes KOL-2521
Fixes KOL-2524
Fixes KOL-2525
Fixes KOL-2526
Fixes KOL-2513
Fixes KOL-2523

Feels bad to say, but it now feels like the FAQs are a bit too drab 😅

Unrelated to this PR specifically, but the help icon makes me feel like it's the unrecognized unicode character � (especially when repeated the number of times it is on the FAQ page)

Related to your comments on FAQs @munkyshi — I've added a custom FAQ admonition type with styles matching table styles. Also repurposed the flag icon for FAQs, and changed the "core concepts" icon to something thematically similar to the flag.
Let me know what you think.


Persistent TOC icon not visible in chunked output

The persistent TOC icon is not visible in the chunked output. The reason is this commit: d26e26b32d2061f53c8669f4ebd580b023d5fdbd, which bumped the z-index of nav.top to 100. Unfortunately, the styling of the nav.tocopen is done by persistent-toc.js, which sets the z-index to 3, which is lower than 100, so the icon isn't displayed.
This can be observed, for example on https://xsltng.docbook.org/guide/2.2.0/parti, the persistent toc icon is missing from the top-right corner, while on https://xsltng.docbook.org/guide/2.1.9/parti it's present.

It looks like the problem is fixed by changing persistent-toc.js from
  tocOpen.style.display = "inline";
  tocOpen.style.zIndex = 3;

  toc.style.zIndex = 4;

to
  tocOpen.style.display = "inline";
  tocOpen.style.zIndex = 101;

  toc.style.zIndex = 102;

The change to tocOpen.style ensures the icon is shown, the change to toc.style is necessary because otherwise the persistent TOC can not be closed as the "Table of Contents" title and the close button are under the nav.

I think z-index 100 might have been overkill. On the one hand, I'm tempted to adjust both values down, but...on the other hand, I could just accept the patch :-)


Fix alignment issues

Docs Build status updates of commit 3673868:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-RuningBuildFailed] Some unexpected errors happened when running build, please open a ticket in https://SiteHelp and include the error report for our team to troubleshoot

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 94335f0:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch surbhigupta12-patch-6 has been deleted or has been force pushed

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Use cloudsdk appcfg

Fixes #30

@meltsufin @patflynn

The test needs to be fixed to compile.

weird that mine local test didn't catch this :\
On Wed, May 4, 2016 at 1:56 PM, meltsufin<EMAIL_ADDRESS>wrote:

The test needs to be fixed to compile.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/app-tools-lib-for-java/pull/34#issuecomment-216948229

:ship: it


When copying to OneNote, the spaces used for indentation disappear

VSCode Version: Code 1.17.0 (be377c0faf7574a59f84940f593a6849f12e4de7, 2017-10-04T23:45:46.804Z)
OS Version: Windows_NT x64 10.0.15063
Extensions:

Extension
Author (truncated)
Version

vscode-eclipse-keybindings
alp
0.9.6

I submitted an issue #35288  before. It's been closed because it's written in Chinese, so I re-submit in English. My English is not good, so I use screenshots more.
Indent using spaces

When copy to OneNote(desktop version) ,The spaces are gone

If "Indent using tabs",  there is no problem.

Reproduces without extensions: Yes/No

The text is stored properly in the clipboard and I cannot reproduce this in Word (see below). So this looks like a OneNote specific behaviour. Nothing we can addresson the side of VS Code.

I guess the content is HTML format when copy to clipboard from VS Code. When copying content to OneNote, the performance is usually normal. Sometimes it is  not normal if the content has special formats. Whether can check the code display HTML.
When I search online, I found that when copying code with highlighting from IDE to Onenote, the disappearance of spaces for indentation is frequent.
This is not urgent. Now I can first convert indentation to Tabs for copying to OneNote, then undo.

Any movement on this? Pasting into Evernote strips all the spacing :/

Looks like pasting into Evernote is working now. Spaces are retained. @BrambleXu what version of EN are you using? I'm using the Mac client and it says version 7.8. Also, I'm using VS Code version 1.30.2 for Mac as well. Perhaps updating both to the latest version will solve this for you.

@BrambleXu Not sure if its relevant, but I use a monospace font in EN for everything. Just figured I'd mention it.

@qodesmith I am using latest Evernote and VS code just like you. I also tried to use the monospace font in EN. But the problem still exists.

It works for users who uses Tab as indentation, not Spaces
I believe it's caused by how these programs handle spaces in HTML, actually if you copy the following line and paste it in OneNote, the leading spaces will also be trimmed.

    copy me!

This is not the VSCode issue. This is OneNote issue. It has very poor attention at OneNote Suggestions place. https://onenote.uservoice.com/forums/327186-onenote-for-windows/suggestions/33254596-rich-text-paste And sadly to say it tends to be never fixed. (Probably there are other similar issues there because I see many duplicates for the missed line spacing option).

As for Evernote, this is the VS code issue. If set indention with space in VS code, and paste to Evernote, it would lose the indention. Other editors have no such problem.

Here is the link

Interestingly, if the paste function is done with the "Keep Text only" option the formatting obviously disappears but the indentation remains.
PS: The problem also occurs in the new OneNote UWP app for Windows 10.

This is not the VSCode issue. This is OneNote issue. It has very poor attention at OneNote Suggestions place. https://onenote.uservoice.com/forums/327186-onenote-for-windows/suggestions/33254596-rich-text-paste And sadly to say it tends to be never fixed. (Probably there are other similar issues there - I see many duplicates for the missed line spacing option).

Man, this is really a VSCode issue!
When Editor:Detect indentation is Off, then you type codes in vsc editor, then copy the codes, then paste in many other applications such as Word/Evernote, the indentations are lost!
But if Editor:Detect indentation is On, then do the same things, the indentations are there, not lost!
So, This is an issue of vscode, not of Evernote.
Developers of vscode, fix this issue asap please... Thanks a lot!!!

I find another thing:
When paste into Plain text editors, indents will not disappear.
When paste into Rich text editors, indents disappear!
So, I think it shouldn't be an issue of those other rich text editors. It is of VSCode.

Yes, when you use Space indentation in your code, the idents will lost in OneNote, but use Tab indentation is normal.
So you'd better convert indents to Tabs in VSCode before copy：

Click ctrl + shift + p
Find command Convert Indentation to Tabs
Execute it and then copy your code to OneNote


Add new test case to 'bank-account' exercise

Fixes #1520

I had to do a force push due to a typo in the commit message.
Let me know if I should create another PR.

I will close and re-open this to try to fix the hanging CI job.


[BUG] Remove "?" Badge On Events Page

Description
There's a badge / chip (or whatever you called it) that contain "?" as the value, i don't know what's that for. maybe it means to show the total events that available on the website(?).
i'm suggesting to remove it, because if it is intended to show the total number of available events, that information has been shown in the "show all" section. hence it's not necessary to have a badge that contains the total number of available events anymore
Screenshots

Do you want to work on this issue?
Yes
Additional information
No response

Thank you for raising. If you hover the ? you will see that this refers to help - and takes you to the docs for events.

However as we are now using forms mainly this can be removed.
I have assigned this to you.

I would like to work on this issue as the assignee has not made a pull request yet

Done! sorry for taking a long time to open a PR, had a long day at work, but i submitted my PR

I would like to work on this issue as the assignee has not made a pull request yet

@viditagrawal56 this an unreasonable request - when you commented the assignee had only been given this issue an hour before.  I would suggest you read the Contributing Guide for this project - we give assignees two weeks to create a PR. We would be grateful if you could please avoid creating extra notifications for maintainers and noise.

if you found any error in Assignees then you have a choice of another pull request. thank you
sorry for void the "code of conduct" but this pull request is only done for practice purposes.

@Sumit21kumar that you for your enthusiasm but this creates more work, noise and confusion to everyone. It would be better for the project and your GitHub profile to provide help and support the the person it is assigned to. Please think collaboration not competition. I will be closing your pull request.


Cannot run

Hi, I am trying to run this application and does not start at all.

Hi, I have the database in the root directory. And still having the same
problem.
Vijay.
On Tue, Jul 23, 2024, 8:02 PM Mohamed El Sayed @.***>
wrote:

it's require sqllite database file to start it
—
Reply to this email directly, view it on GitHub
https://github.com/mohamedelareeg/WPF-POS/issues/1#issuecomment-2245047589,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD3U4NZJAVLWFCY7RF3RK53ZNZA4HAVCNFSM6AAAAABLKHZ7E6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBVGA2DONJYHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

你好，我的数据库位于根目录中。并且仍然有相同的 问题。 维杰.
…
2024 年 7 月 23 日星期二晚上 8:02 Mohamed El Sayed @.> 写道： 它需要 sqllite 数据库文件来启动它 — 直接回复此邮件，在GitHub上查看 < #1 (comment) >, 或取消订阅 < https://github.com/notifications/unsubscribe-auth/AD3U4NZJAVLWFCY7RF3RK53ZNZA4HAVCNFSM6AAAAABLKHZ7E6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBVGA2DONJYHE > . 您收到此消息是因为您创作了该线程。消息 ID： @.>

Copy the rovaShop.db database file in the project root directory to RovaPOS\bin\Debug\