Ensure accessible target size for commonly used controls

The WCAG 2.1 SC 2.5.5 Target Size requires targets for pointer input to be at least 44 by 44 in CSS pixels. Meeting this requirement has multiple benefits, it'd be great to pass this criteria for the commonly used controls:

General controls:
https://github.com/Leaflet/Leaflet/blob/0f904a515879fcd08f69b7f51799ee7f18f23fd8/dist/leaflet.css#L284-L286
Markers:
https://github.com/Leaflet/Leaflet/blob/bc918d4bdc2ba189807bc207c77080fb41ecc196/src/layer/marker/Icon.Default.js#L26
(Changing the marker's width and height to 44px would cause it to stretch. Either the image would need to be modified or padding (right and left) could be modified to achieve a 44 by 44 px target.)
Popup's Close button:
https://github.com/Leaflet/Leaflet/blob/0f904a515879fcd08f69b7f51799ee7f18f23fd8/dist/leaflet.css#L517-L518
(Here too padding can be utilized if keeping a visually small icon is desired.)

Increasing the default size of controls would also have the benefit of not needing extra CSS for conditional sizing (presumably, since the controls would be large enough by default). For example, the following lines could be removed:
https://github.com/Leaflet/Leaflet/blob/0f904a515879fcd08f69b7f51799ee7f18f23fd8/dist/leaflet.css#L316-L320
https://github.com/Leaflet/Leaflet/blob/0f904a515879fcd08f69b7f51799ee7f18f23fd8/dist/leaflet.css#L359-L362

In any case 44 by 44 px cannot be achieved for these targets (for example, for markers it can get tricky if increasing the size of the image itself is not desired, but setting a padding is also not desired as it may cause markers to be unexpectedly interacted with since they're larger than percieved) it should be added to as a known limitation in the accessibility statement.


pull request for e2e test at 01/13/2017 09:45:07

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


data drive may not automount

The command:
echo "/dev/xvdf /data ext4 defaults,nofail 0 2" >> /etc/fstab
fails with a permissions denied error with run from command line.

Second drive is no longer in use.


Compatibility with CUBA Platform 6.10 and above

Can you please make the component compatible with CUBA Platform 6.10 and above? Thx.

Hello,
we do not support the plugin any longer. Do you use it in your software?
Regards Matthias

Hello Matthias,
I have already updated the help-system appComponent to CUBA 6.10 and 7.0 (old screen structure).
Also fixed the multiple sidebars issue...
Do you willing to merge a PullRequest if I send it?

Or should I fork it and create a new appComponent in CUBA marketplace?
Thanks
Steven

Hi,
You can send a pull request. I will merge it. We will talk about the future
of the component in our team. I send you our decision.
Regards Matthias
Steven<EMAIL_ADDRESS>schrieb am Mo., 4. März 2019 10:59:

Hello Matthias,
I have already updated the help-system appComponent to CUBA 6.10 and 7.0
(old screen structure).
Also fixed the multiple sidebars issue...
Do you willing to merge a PullRequest if I send it?

Or should I fork it and create a new appComponent in CUBA marketplace?
Thanks
Steven
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/balvi/cuba-component-help-system/issues/8#issuecomment-469191067,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHsoOQuc3fQhvzPLFpPXiFuwr790IHHdks5vTO5qgaJpZM4a7jBj
.


Auto 1.6.0 issues

Hello! Thanks for maintaining latent couple!
I'm running the extension on auto 1.6.0 and its doesn't seem to work
The following error trace pops out and have confirmed the image splitting doesn't work, however rolling back to 2117fd9 works again

There are also these minor issues during startup

Thanks a lot!

@MaxLanc Thanks for the bug report! However, I found that it is somehow working as intended, but the UI has changed slightly:

You might have to push the button manually and select the type of separation - to prevent gradio 'state' affecting the actual process.
Here is the prompt I used:
masterpiece
AND BREAK 1girl, red hair, schoolgirl uniform
AND BREAK grass and ponds
AND BREAK 1girl, blue hair, police uniform

So I click "I've finished sketch" and then I can switch to rect mode? I'm using that mode due to simplicity sake

@MaxLanc
yes - its for just dummy image-
(

prompt
1girl, red hair, schoolgirl uniform
AND BREAK 1girl, blue hair, police uniform

Thanks a lot, tested and confirm it works
Painted whatever on the mask mode, clicked apply and then changed to rect mode, all other changes to rect mode work normal without any other extra clicks

Prompts:
lake background AND 2girls, blue hair AND 2girls, red hairr
lake background AND 2girls, blue hair AND 2girls, red hair AND large tower
Given this, I would suggest to add a small change to the repo's readme, adding this small workaround for people that get this bug

@MaxLanc Still it is bug that it requires blank image - I added some workaround in recent commit, can you check it?

@aria1th Nope, not fixed
Img using just latent couple:

Img using the workaround from before in the same build as pic above:

@MaxLanc I fixed it again, it should work now (at least tested once) even without apply button, can you check it?

@aria1th
It werks
Pic gen'd using rect and lora mask

Example without latent couple and only lora mask

And finally example with only latent couple

All where done same model, same 3 (one at zero weight) loras and prompt
Closing as completed, if I find something else will report back!
Thanks for the fix!


UP-4756: Ensure links have a valid href destination - rel-4-3-patches

Issue
https://issues.jasig.org/browse/UP-4756
Anchor element found with a valid href attribute, but no link content has been supplied.
 ...

Resolution
Use empty anchor instead.
 ...

Coverage increased (+0.002%) to 22.872% when pulling ce4a4a8e00d8f28f47e4f6946627a2a419f8d6c7 on ChristianMurphy:UP-4756-link-href-rel-4-3-patches into e6b4afdd506529d90275b783e859c6be70f204ab on Jasig:rel-4-3-patches.

the "javasript:void(0);" is a valid protocol

While the javascript: scheme is provisionally supported in the whatwg url standard.
Using the javascript: scheme violates the semantics of the  element.

If the a element has an href attribute, then it represents a hyperlink (a hypertext anchor) labeled by its contents.

Since scheme javascript: represents a command not a hyperlink.
The more semantically correct and more accessible way would be to use the <button> element with class="btn btn-link".
However using the <button> element has the potential to break skins/themes that style the  element rather than modifying the less @link-color and @link-hover-color variables.
For uPortal 5 switching javascript commands to the <button> element would be an option, but uPortal 4.3 should stick with the  element.

From the a11y POV, you've got downsides with href="#" : http://stackoverflow.com/a/1292546

Since the code is already calling preventDefault(), href="#" would have no scroll side effect.

as for me: +1 for href="javascript:void(0);"

Okay
Thoughts on moving uPortal 5 to using the <button> element?

Didn't notice that it calls on every  preventDefault(); , so that's ok.
That was my little fear.
+1 for your POV. 😀
@christianMurphy rocks! 👌

´´´ *  ´´´

links

Didn't notice that it calls on every preventDefault()

It is on most, I'll need to go though and confirm that it is called on every  element event listener.

@ChristianMurphy
Sorry for the late response :

Thoughts on moving uPortal 5 to using the <button> element?

You need to wonder two things (WCAG2 POV)
Does the hyperlink load a new page after clicking on it ? Or does all the content is still on the same page?
In the first case, links with href="#" is better, it is the primary function of link. It will be more robust, especially in case of JS loading delay. Furthermore it allows to navigate through the browser history, which could be very significant (thanks to Backspace).
In the second case , the buttons may agree, taking care to make them very explicit (with title or aria-label or aria-labelledby) because it's necessary to inform clearly the user about the action he will do. If there is in a long series of steps with buttons, it can quickly become difficult to know at which step the user stands.
You should know that links and button are different in a11y, because there are specific behaviors to 2 elements. For example: Space triggers a button but not a link. And if you touch / click on a button, keeping the pressure on it and then leave the touchable / clickable area, the button will not be triggered, while the link will be triggered (eg, on mobile browser, triggering the opening of a contextual menu).
hope this help. (If you think this can be usefull, do not hesitate to copy this into the jira for WCAG2 to keep track) 😸
@cousquer

Thanks much @cousquer !

After a conversation with @drewwills href="javascript:;" allows support for older IE and should not be removed in rel-4-3-patches this would still be an option from uPortal 5.


Load credentials from environment variables in python example

As a side note, please make sure the titles of the commit messages are <= 50 chars.
Cf. our guidelines on confluence & https://chris.beams.io/posts/git-commit/


feat(blog): add promise.all post

@tyndallm, would you please mind reading this and letting me know if this makes sense? 💟  Might try to wrap it all up with one final sentence, but this is the gist of the post.

Yo man, love it! Its a really great analogy. Forgetting your wallet is a good reject() example too. Now I need to find a Roy Rogers...

Haha for real. 🍗  Thanks for taking a look!


resolve failing CI

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #[issue number]
Description
Clear and concise code change description.
Alternative(s) considered
Have you considered any alternatives? And if so, why have you chosen the approach in this PR?
Type
Choose one: (Bug fix | Feature | Documentation | Testing | Code health | Builds | Releases | Other)
Screenshots (if applicable)
Checklist

[ ] I have read and acknowledged the Code of conduct
[ ] I have read the Contributing page
[ ] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA
[ ] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach
[ ] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project
[ ] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally
[ ] I have built and run the reference app(s) to verify my change fixes the issue and/or does not break the reference app(s)

@epicadk In my case, I got an OOM the last time I pushed a code. Works fine locally though

@epicadk In my case, I got an OOM the last time I pushed a code. Works fine locally though

Yeah what's weird is this time it's failing only on the coverage step (where I increased the ram).

@stevenckngaa not sure if you can shed a light on this.
What i've noticed locally is that if i run the instrumentation tests on my pixel6 everything passes, but if i run it on nokia 4.2 it fails.
so i wonder if this is related to any recent emulator update similar to what happened in https://github.com/ReactiveCircus/android-emulator-runner/issues/160
i am also trying to resolve this in https://github.com/google/android-fhir/pull/1118 but haven't gone very far

doesn't look like the change worked :(

not a ram issue maybe bumping the dependencies might work?

Amazing @epicadk

I think @stevenckngaa did something similar in #1125 😅

maybe it was a fluke when all the tests passed?


bug: bert_tokenize can not  find the longest token

Rollback to oringinal code to correct it.
test with "gpt", it should be splitted to ["gp", "t"].

I think the issue here is that "gpt" is not actually in the vocabulary. It works for other examples such as "hell"/"hello". I was having some issues with the original code going over n_max_tokens, so I ended up changing a couple of things.

Yes,the "hello" is right. But the "gpt" in HF's tokenizer is splited to ["gp", "t"]  instead of ["g", "pt"].
I think we should use HF's tokenizer as a benchmark for testing, right?

Ah, I see, thanks! Found the bug in the tokenizer, was double incrementing in one loop. Fixed in c7026df3abe1fb7cc8ecd1fab7d564a28d975d48. Should work now.


Fix CI

This repository's main branch currently does not pass all CI checks.
This makes detecting issues in PRs harder and might result in a broken track, and so is an important thing to fix.
The following CI workflows are currently failing:
https://github.com/exercism/nix/runs/3066699484
Tracking
https://github.com/exercism/v3-launch/issues/55

@ErikSchierboom
I think it's working now


Custom class for a DataTable row

Add ability to specify a custom class for a DataTable row.
Fixes https://github.com/salesforce/design-system-react/issues/2491

Are the tests not being run anymore?

@interactivellama Can you please review when you get a chance? Thanks!


Update default_data.json

Add mapping between zanata_id corin and launchpad_id corin-v

Thank you for contributing to openstack/stackalytics!
openstack/stackalytics uses Gerrit for code review.
If you have never contributed to OpenStack before make sure you have read the
getting started documentation:
http://docs.openstack.org/infra/manual/developers.html#getting-started
Otherwise please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


[Feature]: Shrink disk images on startup when preallocation is off

Is your proposal related to a problem?
Currently, when preallocation is off, disks will continue to grow as you add data but will not shrink after data is deleted.
Describe the solution you'd like.
Add an environmental variable that lets you shrink the disk image at startup.
Describe alternatives you've considered.
Buy larger disks and give Synology vDSM it's own reserved storage pool.
Additional context
No response

However, if you set DISK_FMT=qcow2 it should behave exactly like you want.

Yeah, I'm using qcow2 currently, but from what I read shrinking the qcow2 disk image isn't supported. Is this not correct?
I believe there is even a check that says that when you try and reduce the disk size via env variable.

Yes, I thought you ment shrinking the physical size of the image file (when its partially empty). That should already happen on qcow2, and may not always happen with the raw format (although it should work in theory).

Ahh, okay perfect. It seems to reclaim space automatically. I did not realize that it did this already. This is great, thank you! Seriously awesome stuff!


Support for Cryptographic Message Syntax (CMS) / RFC5652

Does the current library support CMS, aka RFC 5652, aka the latest update to PKCS#7?
If not, are there plans to support such a means to sign and verify a message with an x509 certificate?

@rmhrisk @microshine


Add proper error handling in SSA

Problem
In AcirVar we have a few expects which should be turned into Result<T, Error>
Happy Case
.
Alternatives Considered
No response
Additional Context
No response
Would you like to submit a PR for this Issue?
No
Support Needs
No response

This would essentially entail removing unecessary unwraps and expects, where we intended to return an error


Ignore default value for inputs if it's local file path

Resolve #49
When generating/updating config to valohai.yaml, we will ignore the value of inputs if it's pointing to a local file path

I'll rebase this PR once #123 is merged to master, it will fix the CI


No Video in Preview

I've been having this issue with my 2011 MacBook Pro. Maybe the screen saver isn't supported on the older hardware. But when I open System Preferences and go to Aerial, no preview that ever loads. I've uninstalled and reinstalled several times. When I open the the screen saver options I see the attached. This may be a duplicate issue and I do apologize if that is the case. Just curious if anyone else has seen this happen.

@cdwilkins1 Most likely if you are getting that symbol it means you are behind a firewall or corporate proxy that is blocking access to the videos. Are you able to connect your MacBook to another network and try caching the videos there?

I've tried multiple different networks. Current the firewall is off in System Preferences. I have a new 2016 MBP for work and it works perfectly there. Setup on the same network.

The firewall in System Preferences won't cause this problem. I meant a firewall from whoever is providing your internet. Or even a third-party firewall like Little Snitch perhaps, that has caused issues for people in the past.
Another way to test is to try these links in a web browser and see if either of them are blocked...
http://a1.phobos.apple.com/us/r1000/000/Features/atv/AutumnResources/videos/comp_GL_G004_C010_v03_6Mbps.mov
http://a1.v2.phobos.apple.com.edgesuite.net/us/r1000/000/Features/atv/AutumnResources/videos/comp_GL_G004_C010_v03_6Mbps.mov

After visiting the links you posted it is now working. Thanks for the assistance, I greatly appreciate it. I'm caching them now.

@cdwilkins1 Glad to hear that's working for you now. :)


The terminal process terminated with exit code: 127 after invalid command enter then exit in Git bash terminal

Steps to reproduce:

Update Git with latest build (I got v2.28.0.windows.1)
Update Visual Studio Code (I got v1.50.1)
Open Visual Studio code from any one directory as a workspace
Open terminal. In my case default terminal would be Git bash.exe
enter valid command like '$ ls -l', then type '$ exit' to close. here bash terminal close without any issue
Open terminal again (steps 4) then enter invalid command '$hellovsc' then type '$exit'. here I received 'The terminal process "C:\Program Files\Git\bin\bash.exe" terminated with exit code: 127.' error message.
I suspect invalid command response causing some issue with terminal exit status.
To confirm step 7 assumption opne terminal again (step 4) then enter a invalid command '$hellovsc' then type a valid command '$ls -l' then type '$exit' to close. This time terminal closed without any error.

Application info
VSC Version: 1.50.1 (user setup)
Commit: d2e414d9e4239a252d1ab117bd7067f125afd80a
Date: 2020-10-13T15:06:15.712Z
Electron: 9.2.1
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.18363
GIT: 2.28.0.windows.1
Systems info:

CPUs
Intel(R) Core(TM) i5-6200U CPU @ 2.30GHz (4 x 2400)

GPU Status
2d_canvas: enabled flash_3d: enabled flash_stage3d: enabled flash_stage3d_baseline: enabled gpu_compositing: enabled multiple_raster_threads: enabled_on oop_rasterization: disabled_off opengl: enabled_on protected_video_decode: unavailable_off rasterization: enabled skia_renderer: disabled_off_ok video_decode: enabled vulkan: disabled_off webgl: enabled webgl2: enabled

Load (avg)

Memory (System)
7.87GB (1.64GB free)

Process Argv
--crash-reporter-id 73688476-101c-4294-aa99-13c447d8f5ec

Screen Reader
no

VM
0%

Extension info:

Extension
Author (truncated)
Version

Bookmarks
ale
11.4.0

ng-template
Ang
0.1001.0

All-Autocomplete
Ati
0.0.23

gitlens
eam
10.2.2

go
gol
0.17.2

output-colorizer
IBM
0.1.2

vscode-dimmer-block
ima
2.2.0

vscode-essentials
jab
1.5.0

yuml
Jai
3.5.1

bash-ide-vscode
mad
1.11.0

git-graph
mhu
1.26.0

vscode-docker
ms-
1.6.0

vscode-devskim
MS-
0.4.193

vscode-kubernetes-tools
ms-
1.2.1

mssql
ms-
1.9.0

vscode-postgresql
ms-
0.3.0

python
ms-
2020.9.114305

remote-containers
ms-
0.145.0

remote-ssh
ms-
0.55.0

remote-ssh-edit
ms-
0.55.0

remote-wsl
ms-
0.50.1

vscode-sanddance
msr
3.1.0

subtle-brackets
raf
3.0.0

java
red
0.69.0

vscode-rsp-ui
red
0.23.5

vscode-server-connector
red
0.23.5

vscode-xml
red
0.13.0

vscode-yaml
red
0.11.1

sonarqube-inject
sil
2.1.0

sonarlint-vscode
Son
1.17.0

shellcheck
tim
0.12.1

vscodeintellicode
Vis
1.2.10

vscode-java-debug
vsc
0.29.0

vscode-java-dependency
vsc
0.13.0

vscode-java-pack
vsc
0.10.0

vscode-java-test
vsc
0.24.2

vscode-maven
vsc
0.24.2

I can't reproduce this, but if it's saying 127 after you run exit, that's just the code that git bash exited with. I know macOS also provides unexpected exit codes when running exit sometimes as well. You can workaround this by killing the terminal instead (the bin icon), or use this setting:
"terminal.integrated.showExitAlert": false

I also got the same error when running a program A.py on vscode remotely connecting to the server via SSH. But I can run other programs normally in this way. Even I copy the code of A.py to other programs and it works normally. I found that the only difference in A.py is that the name of its parent folder contains the symbol &. After I removed the & in the folder name, it returned to normal. Maybe this can help you.


Private Base URL should be configurable through environment variable

Currently there is no way to configure Private Base URL parameter through environment variable and this will come in handy for self-managed deployments automation. With current workflow you either have to hard-code this on integration setting through online dashboard or download the file and updated it with correct internal endpoint for your backend service and use THREESCALE_CONFIG_FILE environment variable to load the file instead of pulling it from the portal.
Version
dokcer-image: registry.access.redhat.com/3scale-amp21/apicast-gateway:1.4

Would also be nice to have for us.
We only have one service but have different private base URLs depending on the environment. Currently, we are taking the JSON file and manually replacing the URL in our staging APICast instance which is suboptimal.


Adding code for Kruskal Algorithm in C

I want to add code for Kruskal Algorithm in C

Sure, go ahead.


[RELEASE] 0.1.3-12 [CLOSES #136] Full SimpleNetwork Library

[CLOSES #136] Full HTTP methods coverage in SimpleNetwork library in @misk/core
Made generic handling function for methods that don't have a data parameter (DELETE, HEAD, GET)
Bump dependencies in tabs/palette to prepare for new SimpleNetwork demo component
Change dispatch function names to new idiom of {state domain}{action name} ie. dispatch.simpleNetworkGet

As a metapoint: in the future could you split out logical code changes from the various dependency bumps / release changes in distinct commits? Makes it a bit hard for me to distinguish what I should pay attention to.

As a metapoint: in the future could you split out logical code changes from the various dependency bumps / release changes in distinct commits? Makes it a bit hard for me to distinguish what I should pay attention to.

Good idea, I'll start to do that and will update the releasing doc accordingly

LG. Are there any unit tests we could add for this network library?

Yes I'm looking into a library that will let us mock out and unit test Redux flows like this. The actual network part probably doesn't need to be extensively tested because it's entirely provided by Axios.


Uses Passive Event Listeners to Improve Scrolling Performance

According to the Google devs, it is a good practice to flag every wheel, mousewheel, touchstart, and touchmove event listener with the passive attribute, see here.
There is a simple feature detection test to implement in order to see if the browser supports this flag here and it seems that the only place in Semantic UI where this flag needs to be passed is in one of the helper functions for managing events.
Is it possible to implement this?

I've been thinking about this previously. It's most likely contingent on jQuery supporting passive listeners https://github.com/jquery/jquery/issues/2871

Yes, sorry you are right. The helper I was mentioning is indeed part of Jquery.


use a transparent icon for linux desktop files

will be fixed once https://github.com/lockbook/aur-lockbook-desktop/commit/3458eb2eb4830aa2340bd5b631552eea6e6f3f7d is tested


Clearer k8s alert fields in slack

Currently, we output kubernetes-originating alerts to Slack with three
common fields: Product, Namespace and Pod.  However, the Pod field can
be quite confusing, because it's not at all clear whether the pod
label refers to:

a pod that is having a problem
a pod that is reporting a problem elsewhere

Therefore, I think we shouldn't output a Pod field for all alerts.
Instead, I thought we could:

add a Deployment field if there's a deployment label
add a Job field if there's a job_name label
add a Pod field only if we know it's an alert about a failing
pod (right now, this is done by matching the alertname against
KubePodCrashLooping)

Should we also add the pod field for this alert? https://github.com/helm/charts/blob/master/stable/prometheus-operator/templates/prometheus/rules/kubernetes-apps.yaml#L32

@samcrang good idea. Maybe we could match KubePod.*?

Sounds legit to me

@samcrang done


Refactor commands using spo.waitUntilFinished and spo.waitUntilCopyJobFinished

Closes #5079

Thanks for another awesome PR we will review it ASAP 👍

Put on draft until #5127 and #5125 has been merged.

@MathijsVerbeeck, both PRs have been merged, so we can resume our efforts on this.

@MathijsVerbeeck are you still working on this?


Add linting & formatting

In my opinion, this should be in charge of implementing #6 or #5 depending on which works best.
That is to say, there should ideally only be svelte-add/linting and not svelte-add/eslint nor svelte-add/prettier

I have started some work here: https://github.com/manuel3108/svelte-add-linting
I think it's pretty hard, because the svelte template and prettier are both opinionated.
That's why I choose to try to approach the svelte template as close as possible, by i.e. using tabs instead of spaces.
I'm quite happy with the results of prettier now, as it only modifies 4 now, instead of nearly every file when i was starting.
Eslint is currently giving me this error:
/mnt/files/dev/web/sveltekit/src/routes/index.svelte
  2:22  error  Unable to resolve path to module '$components/Counter.svelte'  import/no-unresolved

If you have an idea, please let me know.
Also I'm unsure for the scripts added. For the my past projects I added them like this:
		"lint": "eslint .",
		"format": "prettier --write .",
		"format-check": "prettier --check .",
		"check": "npm run lint && npm run format-check"

Any opinion on this?

I have started some work here: https://github.com/manuel3108/svelte-add-linting
I think it's pretty hard, because the svelte template and prettier are both opinionated.
That's why I choose to try to approach the svelte template as close as possible, by i.e. using tabs instead of spaces.
I'm quite happy with the results of prettier now, as it only modifies 4 now, instead of nearly every file when i was starting.
Eslint is currently giving me this error:
/mnt/files/dev/web/sveltekit/src/routes/index.svelte
  2:22  error  Unable to resolve path to module '$components/Counter.svelte'  import/no-unresolved

If you have an idea, please let me know.

I do not. Problems like this are why I was personally afraid of being responsible for the linting adder. Leaving it to solve later is fine by me.

Also I'm unsure for the scripts added. For the my past projects I added them like this:
		"lint": "eslint .",
		"format": "prettier --write .",
		"format-check": "prettier --check .",
		"check": "npm run lint && npm run format-check"

Any opinion on this?

I agree with combining linting and format checking into one script ("check") because they really are 2 just parts of the same step. I'd leave the name check open to type checking, though, so I'd personally rather have scripts like this:
"format": "prettier --write .",
"format:check": "prettier --check .",
"eslint": "eslint .",
"lint": "run-p format:check eslint",

and only tell people to use lint and format (the other two are just "implementation details").
What are your thoughts on that? Do you also know if it's always going to be the case that Prettier formats code in a way that ESLint accepts, or can people get caught on errors from that? I personally have only used ESLint's formatter to prevent such a thing from ever occurring.

I'm excited about this one. I played around with the no-unresolved issue and I don't see a way to get eslint or eslint-plugin-import to resolve those imports. It's expecting to use a resolver package, and the config is just a Vite alias.
https://github.com/benmosher/eslint-plugin-import/blob/master/README.md#resolvers
You can ignore those imports like this, in .eslintrc.json:
"rules": {
"import/no-unresolved": ["off", { "ignore": ["$\w/"] }]
}
I set it up that way instead of $components because SvelteKit changed the components path to lib now. It could be more specific if that feels off:
"import/no-unresolved": ["off", { "ignore": ["${lib|components}/"] }]

I played around with the no-unresolved issue and I don't see a way to get eslint or eslint-plugin-import to resolve those imports. It's expecting to use a resolver package, and the config is just a Vite alias.

Ahhh, you where there a bit too early :smile:
I found another way to remove this error: https://github.com/manuel3108/svelte-add-linting/commit/61e1a6ea200a54b65ff9e60f6e27e734d18f3941
Essentially I am using eslint-import-resolver-alias and adding a bit of config to the eslintrc.json.
The tricky thing is the $ in the $lib folders name. The package I am using, does not escape the string properly, that's why we need to escape it "map": [["\\$lib", "./src/lib"]].
Ideally we would create a PR for this upstream package, but the maintainer seems very inactive, so I'm not planning to do so.

I agree with combining linting and format checking into one script ("check") because they really are 2 just parts of the same step. I'd leave the name check open to type checking, though, so I'd personally rather have scripts like this:
"format": "prettier --write .",
"format:check": "prettier --check .",
"eslint": "eslint .",
"lint": "run-p format:check eslint",
and only tell people to use lint and format (the other two are just "implementation details").

I agree mainly, but I don't think we should use run-p here. It would add an additional dependency, and it might happen that the output of eslint and prettier is mixed together. I have changed the lint script to run them one after the other

and only tell people to use lint and format (the other two are just "implementation details").

Updated the readme

Do you also know if it's always going to be the case that Prettier formats code in a way that ESLint accepts, or can people get caught on errors from that?

That's why we are using eslint-config-prettier as this disables all eslint rules that would cause conflicts.

The default create-svelte process includes linting and formatting options now.


chore: use published version of fedimint-tonic-lnd

Needs a cargo update to get Cargo.lock in sync!

ping! would be nice to get this in


Cancel pending request

Describe the bug
When a request to check a password or email is sent - a loading icon is shown, if the back button is pressed we should cancel the request and hide the icon.
To Reproduce
Steps to reproduce the behavior:

Check password or email
When the loading icon is shown, click the back arrow or the hardware back button

Expected behavior
The request should be canceled and loading icon hidden
Screenshots or videos

Hey @michaeltintiuc, this looks like something I can help with? Any issue if I start taking a look?

@nosnibor89 not at all, please go for it!

Does this still need help?

Thanks for the help @machine-dev go for it!

I see that the back button is disabled while loading. Don't think if I will work on this or not. @michaeltintiuc


Bump libloading to 0.8

Main motivation is that in my projects, many dependencies use windows-sys to interface Windows API so and only ash keeps using winapi crate because of libloading. Libloading 0.8 switched to windows-sys so problem solved.

Note that you can also avoid this dependency by disabling the "loaded" feature, and using "linked" instead. This does require that the Vulkan SDK be installed in the build environment, though.

I recalled this being in the changelog already, and it turns out that's in #739 where we also bump libloading to 0.8 so I'll close this one to not have a dupe open.


CDPCP-732 - Added support for getting workload credentials for a user…

…… (#6271)
Squashed Commits:

CDPCP-732 - Added support for getting workload credentials for a user or machine user

CDPCP-732 - updated getActorWorkloadCredentials since passing CRN itself

CDPCP-732 - added support for internal actor since get workload credentials ums api requires internal crn

CDPCP-732 - Added support for getting hased credentials from UMS, Internal Actor Support, Updated FreeIPA Client and Config

CDPCP-732 - Added password sync to FreeIPA.
Added util class for Credentials and updated ASNUtil

CDPCP-732 - Added ANS encoded support and calling user-mod for password, reverted bk freeipaclient factory for hard coded test purpose password

https://github.com/hortonworks/cloudbreak/commit/888b2e2a8887def28a24e26efcd6d9ca5863733a

Closes CDPCP-732
@toddlipcon @PhilippeLan @handavid @keyki @lacikaaa @horadla23

just a couple of minor comments. The only serious issue I have with this is that the MockUserManagementService doesn't return a valid value. I think that this is needed to support local development (i.e., for folks running CBD)

Agreed and I have put back implementation.

It seems I am late, but I am missing more unit tests from this change (for new code in UserService and UmsUsersStateProvider for example)


chore: Update indirect dependencies (crypto, net and sys) to latest version

Update the following dependencies of the project to their latest version. To make sure we have the latest fixes/enhancements.
golang.org/x/crypto, Version: v0.22.0 -→ v0.24.0
golang.org/x/net, Version: v0.23.0 -→ v0.26.0
golang.org/x/sys, Version: v0.19.0 -→ v0.21.0
Definition of Ready

[ ] I am happy with the code
[ ] Short description of the feature/issue is added in the pr description
[ ] PR is linked to the corresponding user story
[ ] Acceptance criteria are met
[ ] All open todos and follow ups are defined in a new ticket and justified
[ ] Deviations from the acceptance criteria and design are agreed with the PO and documented.
[ ] No debug or dead code
[ ] My code has no repetitions
[ ] Critical parts are tested automatically
[ ] Where possible E2E tests are implemented
[ ] Documentation/examples are up-to-date
[ ] All non-functional requirements are met
[ ] Functionality of the acceptance criteria is checked manually on the dev system.

Thanks!


P10M: Aspirate function updates

overview
New aspiration function customization.
Things to post here:

Model number
Serial number format
New aspirate and dispense functions
Any other config elements for the new model that are different from previous models

Definition of done

All config changes and functions in the model config file
Entries for the serial number and model in write_pipette_memory

Config changes are all in #4721 , still need serial number changes

Serial number format: P10MV16\d+
Configs should be identical to 1.5 aside from the aspirate/dispense changes


ECWolf cfg file data path stuck on first game used.

Describe the bug
When playing more than one game with ECWolf, the first game launched will generate ecwolf.cfg with the path to its data directory. If you run another game from steam it will not change the path and will run with the first game's data files.
To Reproduce
Steps to reproduce the behaviour:

Launch a game (eg Wolf3d) with luxtorpeda-dev and exit.
Launch another ECWolf game (eg Spear of Destiny).
ECWolf will run Wolf3d.
Delete .config/ecwolf/ecwolf.cfg
Launch the second game again.
Second game will launch correctly.

Expected behaviour
ECWolf should run with data corresponding to the game launched.
Additional context
If you specify a config file with "--config wolf3d.cfg" for example, it will create that file and not make an ecwolf.cfg in the home config. That might be a good start.

Note: when checking this, check into bstone

https://github.com/luxtorpeda-dev/packages/commit/9e0d50c5b1860cbb0249359e5285cc6ddd929fc0 should fix this.
The next launch you should see this change.


Update doorbird to 1.7.0

Please update my adapter ioBroker.doorbird to version 1.7.0.
This pull request was created by https://www.iobroker.dev c0726ff.

RE-CHECK!


As init system on FreeBSD

FreeBSD does not have systemd, can tini be used as init system?

It probably would not be super valuable as an init system for a full
system. Tini is more useful for single process container in use cases.
On Wed, 4 Sep 2024 at 04:39, zoujiaqing @.***> wrote:

FreeBSD does not have systemd, can tini be used as init system?
—
Reply to this email directly, view it on GitHub
https://github.com/krallin/tini/issues/228, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANIHVTN2Q5PWNO6OSWFXFDZUZXELAVCNFSM6AAAAABNTKUVQGVHI2DSMVQWIX3LMV43ASLTON2WKOZSGUYDIMJXGAYDOMA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Need special handling for PRMS segid 3_1, 8_1,51_1, and 2124_1 in p2_FORESCE_LC_per_catchment_reclass_tot

Please see PR #123 draft for details on changes to main
No TOT Land Cover proportions calculated for 4 PRMS segid - 3_1, 8_1, 51_1, and 2124_1.
As such, target p2_FORESCE_LC_per_catchment_reclass_tot - which calculates all land cover proportions for the total upstream area for each PRMS_segid - has nrow = 445 (not the expected 459).
Case 1:
3_1, 8_1, 51_1 are not in the source dataset p1_prms_reach_attr that maps the upstream and downstream segments for every given segment, and allows us to recursively determine the all segments upstream of given segment. p1_prms_reach_attr just has 3_2, 8_2, and 51_2.
TOT prop values are therefore calculated for 3_2, 8_2, and 51_2. How should we assign proportions to 3_1, 8_1, and 51_1.

[x] Decide how to assign proportions to these 3 PRMS_segid

Case 2:
To produce the TOT FORESCE proportions target p2_FORESCE_LC_per_catchment_reclass_tot ,hru_segment values are used to merge  the “recursed” reach attributes dataset (p2_prms_attribute_df) with the foresce CAT LC proportions target, p2_FORESCE_LC_per_catchment_reclass_cat, which ultimately outputs p2_FORESCE_LC_per_catchment_reclass_tot.
Typically, the $hru_segment == $PRMS_segid without the suffix _*
However, the hru_segment for PRMS_segid:2124_1 is 2131 (not 2124 as expected).
> p2_FORESCE_LC_per_catchment_reclass_cat[[1]][p2_FORESCE_LC_per_catchment_reclass_cat[[1]]$PRMS_segid == '2124_1',]
# A tibble: 1 × 14
# Groups:   PRMS_segid [1]
  PRMS_segid hru_segment total_PRMS_area prop_lcClass_3 prop_lcClass_2 prop_lcClass_1 prop_lcClass_4 prop_lcClass_7 prop_lcClass_8
  <chr>      <chr>                 <dbl>          <dbl>          <dbl>          <dbl>          <dbl>          <dbl>          <dbl>
1 2124_1     2131              38784764.          0.405         0.0347         0.0293         0.0598          0.120        0.00207
# … with 5 more variables: prop_lcClass_9 <dbl>, prop_lcClass_10 <dbl>, prop_lcClass_6 <dbl>, prop_lcClass_5 <dbl>, Year <chr>

[ ] Determine why this is occurring to PRMS_segid:2124_1 and map out how to handle this case

For both cases:

[ ] Add special handling for both cases when creating the p2_FORESCE_LC_per_catchment_reclass_tot
[ ] Suggest producing a separate function for this process that will replace the lengthy code blocks that produce p2_prms_attribute_df and p2_FORESCE_LC_per_catchment_reclass_tot

I have an idea for handling Case 1, but it depends on the TOT area being correct for _1 and _2 segments (see #122).
TOT_1 = (TOT_2 * TOT_AREA_2 - CAT_2 * CAT_AREA_2)/(TOT_AREA_2 - CAT_AREA_2)
where TOT_AREA_2 - CAT_AREA_2 should equal TOT_AREA_1 + very small error

The equation I posted will not work as written for these segments because there are additional reaches upstream of _2 vs. _1 segments. Example for 3_1, 3_2, 8_1, and 8_2:

The equation I posted will not work as written for these segments because there are additional reaches upstream of _2 vs. _1 segments. Example for 3_1, 3_2, 8_1, and 8_2:

If _1 is always upstream of _2, then can we assume the TOT that is attributed to _2 includes _1  ?

Yes, _1 is always upstream of _2. The issue is that there are tributaries upstream of _2 that are not upstream of _1 (black segment in between the red and blue segments in the figure).

Thinking about case 1, I don't really have a clever solution here because of the reasons that Jared's already mentioned. I would recommend adding special handling within the p2_prms_attribute_df target as opposed to other downstream targets like the FORESCE ones. Two options stick out to me:

the first being to edit p2_prms_attribute_df to use the xwalk table and use NHD (example here).
brute force edit p1_prms_reach_attr to include the split segments with a "_2" suffix.

If you're hoping to edit what you already have using the recursive function that you and Jared came up with, I wonder if the code below to carry out the brute force method would work? I haven't spent much time with the target p2_prms_attribute_df so I'm not as familiar with the expected output and would appreciate checks by you or Jared.
munge_reach_attr_table <- function(attr_tbl){

  # Special handling to add split segments with _2 suffix to prms
  # reach attribute table; values for from_segs and to_seg were
  # derived based on visual inspection and manually entered
  attr_tbl_w_splitsegs <- attr_tbl %>%
    mutate(to_seg = as.character(to_seg)) %>%
    tibble::add_row(subseg_id = "3_1", from_segs = "1", subseg_seg = 3, to_seg = "3_2", seg_id_nat = 1437) %>%
    tibble::add_row(subseg_id = "8_1", from_segs = "", subseg_seg = 8, to_seg = "8_2", seg_id_nat = 1442) %>%
    tibble::add_row(subseg_id = "51_1", from_segs = "43", subseg_seg = 51, to_seg = "51_2", seg_id_nat = 1485) %>%
    # now adjust the split segments with _1 suffix
    mutate(
      from_segs = case_when(
        subseg_id == "3_2" ~ "3_1;8",
        subseg_id == "8_2" ~ "8_1;2",
        subseg_id == "51_2" ~ "51_1;50"),
      to_seg = case_when(
        subseg_id == "3_2" ~ "4",
        subseg_id == "8_2" ~ "3",
        subseg_id == "51_2" ~ "52")
      )

  # Find all segments upstream of a given PRMS segment
  attr_tbl_out <- attr_tbl_w_splitsegs %>%
    select(subseg_id,subseg_seg,from_segs,to_seg) %>%
    # Update `from_segs` col by splitting the individual segs in a list (can then loop through the list)
    mutate(from_segs = stringr::str_split(string = from_segs, pattern = ';', simplify = F)) %>%
    rowwise() %>%
    # Collect all upstream segs per individual seg_id using recursive_fun() (row wise application)
    mutate(all_from_segs = list(recursive_fun(x = subseg_seg,  df = ., col1 = 'subseg_seg', col2 = 'from_segs'))) %>%
    # unest to have new rows for each upstream catchment
    unnest(all_from_segs, keep_empty = TRUE) %>%
    # change col type to be able to compute
    dplyr::mutate(all_from_segs = as.integer(all_from_segs))

  return(attr_tbl_out)

}

Here is a plot for 2124_1. It seems correct that the hru_segment = 2131 for segment 2124_1 because the segment is entirely within this HRU. For now, I'll add manual handling for segment 2124_1 for this problem. @lekoenig: should the hru_segment for 2124_1 be edited in the GFv1_catchments_edited gpkg file?


docs: Add a design doc for the ocs to ocs StorageConsumer CR and its controller

Signed-off-by: kesavan<EMAIL_ADDRESS>

@kesavanvt is this PR still relevant?

Hi Malay
Kesaven has left the company a couple of months ago
Regards
-- Ohad
On Fri, Oct 14, 2022 at 10:33 AM Malay Kumar Parida <
@.***> wrote:

@kesavanvt https://github.com/kesavanvt is this PR still relevant?
—
Reply to this email directly, view it on GitHub
https://github.com/red-hat-storage/ocs-operator/pull/1422#issuecomment-1278597785,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADSMPT5OYNDSGKRY263UUGTWDEDKFANCNFSM5JJVHZOQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Adding new gRPC call to notify PDP about its readiness

we should also add a readiness check

I think it will be the next step. Even now we can perform an implicit healthcheck just trying to connect to tcp/5555


undefined: scope

The following program doesn't work:
package main
import (
	"os"
)
func main() {
	if o := os.Getenv("O"); o != "" {
		println(o)
	} else if s := os.Getenv("S"); s != "" {
		println(s)
	}
}

Thanks for the report! Fixing now.

Fixed in bc74b8dafd779a8ebdda5742aca1ff0cacd63d8f


MAINTENANCE: upper limits on dependencies

Pinging @SMoraisAnsys since he is the only one I know working in this package...
https://github.com/ansys/pyconceptev/blob/4b3c8c6386983c31a83888386f590f4741a0a21d/pyproject.toml#L35-L43
Is there really a need to put the cap on all these dependencies (I'm referring to the upper limits). Some libraries are using above https 0.27 and this package prevents them from using it. It is in general not a good practice to cap such standard dependencies... See other poetry-based projects:
https://github.com/ansys/pyacp/blob/90189482a8ad4cfc74057282bc84fb769716a4e6/pyproject.toml#L35-L46
Can we get this solved and get out a new patch release?

@RobPasMue This should be handled with #109.
Non main dependencies haven't been updated though, let us know if that's an issue

Nope, that's not a problem - as long as the main deps are not capped on the upper limit I'm happy  :)


feat: expand support for python versions to include path *version.py

Fixes https://github.com/googleapis/release-please/issues/1706 🦕

After offline discussion with @chingor13 , I'm going to explore using the extra-files option


Remove redundant initialization attributes of variables

Is your feature request related to a problem? Please describe.
no
Describe the solution you'd like

List services no need to initialize to null
Describe alternatives you've considered
Remove redundant initialization attributes of variables
Additional context
Add any other context or screenshots about the feature request here.

@ryanjbaxter  this also can be closed


403 when executing the nuget push of version 1.5.10

As per log
https://github.com/lolochristen/OpenLayers.Blazor/actions/runs/9174726234/job/25226186014
when the pipeline tried to push the new version it received a 403.

solved (key expired). 1.5.10 nuget published.


Update version to 1.0.70-6 on staging

Update version to 1.0.70-6

🚀 Deployed to staging in version: 1.0.70-6🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

I have read the CLA Document and I hereby sign the CLA

🚀 Deployed to production in version: 1.0.73-3🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Filter on name=gitea to get correct container id

name: Pull request
about: Installation of gitea for example
Changes proposed by this PR
Add --filter name=gitea to line that gets the container id for the gitea container.
Otherwise, if other containers are running besides those created by this script, it is possible the wrong container id will be retrieved.
PR Checklist
Note: Please do not remove items. Mark items as done [x] or use ~strikethrough~ if you believe they are not relevant

[-] Linked to a relevant issue. Eg: Fixes #123 or Updates #123
[-] Removed non-atomic or wip commits
[-] Filled in the Release Note section above
[-] Modified the docs to match changes

Thanks for submitting this!


Handle constructors in extraction semantic.

Fix #86
Previous PR fixed inly block-statement graph creation, but there was need to handle constructors in extraction semantic step.

I still have the same error:
Traceback (most recent call last):
  File "D:\git\veniq\veniq\ast_framework\block_statement_graph\_block_extractors.py", line 15, in extract_blocks_from_statement
    return _block_extractors[statement.node_type](statement)
KeyError: <ASTNodeType.CLASS_DECLARATION: 17>

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "D:\git\veniq\veniq\dataset_collection\validation.py", line 144, in validate_row
    opport = find_extraction_opportunities(ast_subtree)
  File "D:\git\veniq\veniq\dataset_collection\validation.py", line 28, in find_extraction_opportunities
    statements_semantic = extract_method_statements_semantic(method_ast)
  File "D:\git\veniq\veniq\baselines\semi\extract_semantic.py", line 12, in extract_method_statements_semantic
    block_statement_graph = build_block_statement_graph(method_ast)
  File "D:\git\veniq\veniq\ast_framework\block_statement_graph\builder.py", line 15, in build_block_statement_graph
    root_index = _build_graph_from_statement(method_ast.get_root(), graph)
  File "D:\git\veniq\veniq\ast_framework\block_statement_graph\builder.py", line 26, in _build_graph_from_statement
    new_block_index = _build_graph_from_block(block, graph)
  File "D:\git\veniq\veniq\ast_framework\block_statement_graph\builder.py", line 40, in _build_graph_from_block
    new_statement_index = _build_graph_from_statement(statement, graph)
  File "D:\git\veniq\veniq\ast_framework\block_statement_graph\builder.py", line 24, in _build_graph_from_statement
    blocks = extract_blocks_from_statement(statement)
  File "D:\git\veniq\veniq\ast_framework\block_statement_graph\_block_extractors.py", line 17, in extract_blocks_from_statement
    raise NotImplementedError(f"Node {statement.node_type} is not supported.")
NotImplementedError: Node Class declaration is not supported.
100%|██████████| 1/1 [00:00<00:00, 117.67it/s]
D:\git\veniq\venv\lib\site-packages\numpy\core\fromnumeric.py:3372: RuntimeWarning: Mean of empty slice.
  return _methods._mean(a, axis=axis, dtype=dtype,
D:\git\veniq\venv\lib\site-packages\numpy\core\_methods.py:170: RuntimeWarning: invalid value encountered in double_scalars
  ret = ret.dtype.type(ret / rcount)

KeymapPanel.zip

could you please update branch and then create a new issue?


Fix typos, add hyperlinks and add info on setting up the project in the "Contributor Guide"

Improved or typo fixed.
The "Contributor Guide" page has typos and missing hyperlinks.
Example:
Please ~fee~ free to report bugs
Please feel free to report bugs
It is also missing information about how to setup the project and test changes locally. This is also missing in the CONTRIBUTING.md file.
Solution
Fix the typos and add appropriate hyperlinks.
Structure the contributing docs and add details on how to setup the development environment and test changes locally for the project and the documentation website.

I can open a PR with the typo fix and we can maybe start a discussion on what should be added in the contributing docs if needed.

looks like have been fixed.


Different scaling behaviour in 1.14.0

Hello,
Thank you for this amazing package!
It seems that scaling behaviour has changed in https://github.com/ordo-one/package-benchmark/commit/0b15255d21fabe55bc67a34d167f1ed5e51f9af1.
New logic scales results if scale == false. This is different to before and seems semantically incorrect, but I see the change is applied in few places.
I am happy to open PR, but wanted to check if this is intentional first?

Hi, thanks - yes this is intentional - see release note here:
https://github.com/ordo-one/package-benchmark/releases/tag/1.14.0
Basically there was much confusion of not scaling to the inner loop count so we decided to change the default - you can still get the non-scaled result by specifying --scale (so, yes, we intentionally did flip the behavior, sorry!)

Gotcha, thank you!


Despite changing the language setting to Chinese in the settings

Issue Description
Despite changing the language setting to Chinese in the settings, the interface continues to display in English. I tried clearing the cache and restarting the app, but the issue persists.
Steps to Reproduce

Open the app and go to Settings.
Change the language setting to Chinese.
Save and exit the settings.
The interface language does not change to Chinese and continues to display in English.

Expected Behavior
After changing the language setting to Chinese, the app interface should display all text in Chinese.
Actual Behavior
After changing the language setting to Chinese, the app interface still displays in English.
Environment Information

Device: [Please fill in your device model, e.g., iPhone 12]
Operating System: [Please fill in your operating system version, e.g., iOS 15.4]
App Version: [If applicable, please provide the app version]

Additional Information
Please provide any additional information (such as screenshots) here.
I hope this can be resolved soon. Thank you!

I didn't install it using Docker; I installed it by running npm run build, and I can't switch languages.

I could not reproduce the issue in dev branch, @jxyk2007 Can you check if any errors are displayed in the browser's console?

The container running docker already supports Chinese, so there is no need to modify the language yourself, which is very good.

@jxyk2007 not sure if I understand your issue report entirely, Is the issue resolved then?


Gather list of default STAC catalogs

Hard code a list of STAC APIs right now, later on we'll add the feature to allow users to modify the list of APIs that the plugin pulls from.


Missing rMATS arguments

Description of the bug
Not a bug as such, but more of a missing argument which is critical for rMATS.
one of the key rMATS arguments is libType (https://github.com/Xinglab/rmats-turbo?tab=readme-ov-file#all-arguments):
  --libType {fr-unstranded,fr-firststrand,fr-secondstrand}
                        Library type. Use fr-firststrand or fr-secondstrand
                        for strand-specific data. Only relevant to the prep
                        step, not the post step. Default: fr-unstranded

however in the rnasplice docs I don't see it exposed in the pipeline. Is is possible to choose the library type and I just missed?
Command used and terminal output
No response
Relevant files
No response
System information
No response

Hello @adomingues
The library type parameter is inferred based on the value of the strandedness column in the input sample sheet file. See here.
If this solves your query, I will go-ahead and close the issue.

Thanks @jma1991, that answers my question perfectly.


Adds PR template to repo.

@dbernstein We need either an iCLA or a cCLA to cover any contribution to Islandora. We have a cCLA on file from LYRASIS - does that cover you at this point?
Our list: https://github.com/Islandora/islandora/wiki/Contributor-License-Agreements

Going on information that this work was done for Lyrasis and is as such covered under their cCLA.
See IRC 2019-06-24@16:20 UTC

Just realizing that I had a concern that you can't use the @Islandora-CLAW/committers tag in a different organization. If any Islandora 8 committers receive a notification from this, please respond otherwise we might need to use a team in this organization.

No responses, so what if I use @Islandora-Devops/committers to ping the crew?

I just got the notice.

Right ok, @dbernstein thanks for opening all these PRs in Islandora-Devops. But could I get you to replace
Tag (@ mention) interested parties or, if unsure, @Islandora-CLAW/committers

with
Tag (@ mention) interested parties or, if unsure, @Islandora-Devops/committers


New path to vimfiles on windows

Explain the problem here ...
VIM on Windows changed its default path of ~\vimfiles to ~\Vim\vimfiles
md ~\vimfiles\autoload
$uri = 'https://raw.githubusercontent.com/junegunn/vim-plug/master/plug.vim'
(New-Object Net.WebClient).DownloadFile($uri, $ExecutionContext.SessionState.Path.GetUnresolvedProviderPathFromPSPath("~\vimfiles\autoload\plug.vim"))

Type:

[ ] Bug
[x] Enhancement
[ ] Feature Request
[ ] Question

OS:

[ ] All/Other
[ ] Linux
[ ] OS X
[x] Windows

Vim:

[ ] Terminal Vim
[x] GVim
[ ] Neovim

Using gvim 8.0.586 I cannot confirm. ´~/vimfiles` does still work and I haven't seen any patches changing that path.

yep. And I noticed that VimPlug does install all its files in a hidden folder called .vim on User\
Although all Vim 8 files are installed,  by default, on User\Vim

I believe that's because you specifically told vim-plug to do so. i.e. call plug#begin('~/.vim/plugged'). You can change the path in the argument as you like.

haha...yep,  my bad


Problem Clone()

let attribute1 = TextAttributes().font(UIFont(name: "AvenirNext", size: 12))

let attribute2 = attribute1.clone()

attribute2.font  // => UIFont(name: "HelveticaNeue", size: 12) 😥
attribute2.dictionary[NSFontAttributeName] // => UIFont(name: "AvenirNext", size: 12)

I think that there is better to change StoredProperty to ComputedProperty.
How do you think?

Oh indeed 😱
I don't like the idea of changing the stored properties to computed ones though.
Indeed, we would have to cast the value every time we want to access a property like attrs.font.
I would prefer to do the ugly stuff in a specific place (like a function) that we will call only when cloning.

@delba
I understand your opinion.
So, Can you solve this problem in a different way?

I'll probably fix that during the weekend :smiley:

Thank you so much!!!

Hi @muukii,
It has been fixed in 0.2.4.
Thanks for the issue!

@delba thanks!


use github workflow matrix for  notebook tests

@ink-pad noticed this about GPU runners (from July): https://github.blog/changelog/2024-07-08-github-actions-gpu-hosted-runners-are-now-generally-available/

noticed this about GPU runners

Those are paid-only. So we cannot use them for no-cost.

See the following PRs for the implementation of this across the cookbook repos.
https://github.com/ibm-granite-community/granite-legal-cookbook/pull/22
https://github.com/ibm-granite-community/granite-snack-cookbook/pull/72
https://github.com/ibm-granite-community/granite-code-cookbook/pull/154
https://github.com/ibm-granite-community/granite-finance-cookbook/pull/8
https://github.com/ibm-granite-community/granite-timeseries-cookbook/pull/25


Add README file for the algorithms present in the DSA/JAVA/Algorithms folder

In the DSA/JAVA/Algorithms folder you need to add README file for Quick sort, Bubble sort, Selection sort  to describe its algorithm in details. Don't add source code because it is already there

Hello! Can you assign me this issue

Hello! I would like to work . Can you assign me this issue

okay do it except for Bubble Sort Algorithm because some is on that

Okay I will complete it as soon as possible.

Hi there, I'd like to contribute to this project. Let me know if you need extra help


Content Security Policy - Add Nonce

At the moment it is very hard to get this working on a implemented Content-Security-Policy.
It would be great if we could pass a nonce to the generated script tags in init().
There is already a fork by @Weisskpub containing this: https://github.com/Weisskpub/laravel-google-recaptcha-v3
I personally like this approach, but its your project and the decision how its implemented is yours.
Thanks for this great package!

@lxlang hey thanks! I will check @Weisskpub repo to see how it works. Thanks! I will try to update it within this week.

@lxlang hi, sorry for the delay. I got some urgent company tasks to do this weekend. I might not have time this weekend. just a quick update.

@RyanDaDeng Thanks alot :)


Deploy to Azure Static Webapp using Azure DevOps Service Repos and Pipeline

Can you please add some information about how to deploy to Azure Static Webapp using Azure DevOps Service Repos and Pipeline? E.g. Can I use Azure CLI and use Azure Repos URL with PAT? Because portal only forces to use GitHub but Azure CLI does provide an option to deploy from Git based repos.

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

ID: 51bfa022-2797-8b08-7e15-332229df9a92
Version Independent ID: a1645d12-f261-d303-0191-c8e0b668bcad
Content: Quotas in Azure Static Web Apps Preview
Content Source: articles/static-web-apps/quotas.md
Service: static-web-apps
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@Anand-Moghe
I have assigned the issue to the content author for further review.

Hello @Anand-Moghe ! Could you please help us by providing some further details?
In regards to deploying from Azure DevOps Repos, that functionality is currently there. When creating the static web app, ensure you select 'other' repository. We've published some documentation here: https://docs.microsoft.com/en-us/azure/static-web-apps/publish-devops
Would appreciate some feedback if you feel there's something beneficial that we can add to the article.
In regards to the question about the Azure CLI, are you looking to deploy a template or some other kind of mechanism?

Since it's been over two weeks since there has been any activity on this thread, I am going to ask to #please-close this issue.
@Anand-Moghe, if you have further questions, please feel free to open a new issue against the DevOps article.


How do I set error levels? Ie. Ignore INFO-level.

I was unable to find this in the docs. Thank you.

You have to set the error_types option of the client. It has the same functionality of the error_reporting PHP constants: http://php.net/manual/en/errorfunc.configuration.php#ini.error-reporting

@Jean85 Thank you for the reply. I am setting error levels, please see screenshot:

However I'm still getting errors submitted for INFO levels.

@otang on what PHP version are you and can you past the actuall error message you are seeing in Sentry. My assumption is that PHP 7+ might submit those info's as an Exception instead of Error (because some are now Catchable as Exceptions).

@stayallive We're using PHP 5.5.3. There is no error message, it's just that I'm unable to prevent sentry from sending log level INFO.

@stayallive We're using PHP 5.5.3

@otang Using this code I am not being able to reproduce the issue.
<?php
require_once __DIR__ . '/lib/Raven/Autoloader.php';
Raven_Autoloader::register();

$client = new<EMAIL_ADDRESS>array('environment' => 'test'));

$handler = new Raven_ErrorHandler($client);
$handler->registerExceptionHandler();
//$handler->registerErrorHandler(); // If this register is used the event id is filled
$handler->registerErrorHandler(true, (E_ERROR | E_PARSE)); // When this one is active no event is being submitted
$handler->registerShutdownFunction();

trigger_error('This is an info message!', E_USER_NOTICE); // Which translates to INFO

var_export($client->getLastEventID());

Can you show me the actual error that is being reported to Sentry?
Also if your are submitting an message or something yourself to Sentry (using the $client) the error types won't have effect. The error handler only deals with PHP generated errors / exceptions.

Closing this issue because it is quite old, if there is still a problem please open a new issue.


Write a C++ program to convert paise to rupees

Description
Write a C++ program to convert paise to rupees
Input  ( Ps ) : 100
Output ( Rs ) : 1

How to contribute
Save the solution in ConvertPaiseToRupees.cpp file
Add ConvertPaiseToRupees.cpp file in convert-paise-to-rupees folder

@Pal-Sandeep I will solve this issue.

Hey I can solve this issue. So if that's okay with you , you can assign this to me.

@AbhinavPrabhat01 Sure.
Make  sure to follow community guidelines and if you have any doubts you can ask here.


Build a CNN Model using Alexnet Architecture

Type
Advancement
Description
Create a new CNN Model using the Alexnet Architecture. Document the Code properly for better understanding.
Tools

Jupyter Notebook
PyTorch / Keras
Jupyter Markdown

Have you read the Contributing Guidelines on Pull Requests?
Yes

May I work on this please?

@HarshCasper Can I work on this issue using Keras.

I have assigned this to you @namanshah01
You can work on this using PyTorch @rjn01

Thank you. Is it fine if i take the same dataset form kaggle https://www.kaggle.com/iarunava/cell-images-for-detecting-malaria ? Or should i use the one in your repo

@HarshCasper I have not worked with PyTorch. I only know how to work using keras.


accept-options / connect options should be "sequence" to allows extenstions

In order to extend the accept-options and connect-options in the base XSD, the types should be "sequence."  This does enforce an order in the XSD, so to avoid unnecessary restrictions an XSLT will first be applied to the configuration file to reorder elements, and then XSD validation will occur.

See issue#70 for updates on the XSLT.

Fixed by pull request: https://github.com/kaazing/enterprise.gateway/pull/28 (note:  shaded JAR implementation used rather than sequence changes in the XSD to keep the flexible ordering of gateway-config.xml).


Sensorineural hearing loss

https://www.ebi.ac.uk/ols/ontologies/mondo/terms?iri=http%3A%2F%2Fpurl.obolibrary.org%2Fobo%2FMONDO_0005365
In the OLS link above, there is no entry for sensorineural hearing loss but its in the MONDO-Edit file
Also the term "deafness" is used interchangeably throughout many of the entries under "hearing loss" but deafness is only profound hearing loss and does not encompass milder forms of hearing loss. So ideally the term "deafness" should be replaced with hearing loss as many of the forms of syndromic and nonsyndromic hearing loss do not have deafness, but milder hearing loss. This fix is also needed in OMIM and I've discussed with Ada

We can do a mass replace, keeping deafness as synonym (narrow, not exact?). We did a similar thing with MR->ID

Chris,
I went through the ontology under hearing loss to decide if you should do a full search and replace for all uses. I think that’s probably not quite what we want because there are some syndromes where the formal name includes deafness and we don’t want to change that either because patients are in fact deaf in that particular syndrome and/or because the name is historical (e.g. KID syndrome which is keratitis-ichthyosis-deafness syndrome). I started going through all of the syndromes under “syndromic genetic deafness” and began listing ones you shouldn’t change but it’s a lot of work to copy and paste and my conclusion is that you should not change any of these syndrome names. So you’d change everything in the tree pictured below including all branches but not the one branch I highlighted in blue. For that you’d change “syndromic genetic deafness” to “syndrome genetic hearing loss” but then not touch the items under syndromic genetic deafness.
I did give some thought to the terminal branches under nonsyndromic hearing loss as these could be considered named diseases such as “autosomal dominant nonsyndromic deafness 1” in OMIM. However, these are not the names used by the genetic hearing loss community as can be seen on the website maintained by the hearing loss community here: https://hereditaryhearingloss.org/ so I think its fine to change these nonsyndromic names to be consistent with what the community actually uses despite the OMIM entries. The original issue probably arose because the loci were labeled “DFN” but the whole field now calls all of these loci “nonsyndromic hearing loss” loci despite the DFN locus symbols.
Let me know if this plan makes sense.
Thanks, Heidi
syndromic genetic deafness
Charcot-Marie-tooth disease-deafness-intellectual disability syndrome
Hirschsprung disease-deafness-polydactyly syndrome
autosomal dominant keratitis-ichthyosis-deafness syndrome
ichthyosiform erythroderma, corneal involvement, and deafness
ichthyosis, hystrix-like, with deafness
retinitis pigmentosa-deafness syndrome
ocular albinism with congenital sensorineural deafness
albinism-deafness syndrome
more but I got tired of cutting and pasting!!!!
From: Chris Mungall<EMAIL_ADDRESS>Sent: Thursday, November 29, 2018 7:08 PM
To: monarch-initiative/mondo<EMAIL_ADDRESS>Cc: Heidi Rehm<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [monarch-initiative/mondo] Sensorineural hearing loss (#551)
We can do a mass replace, keeping deafness as synonym (narrow, not exact?). We did a similar thing with MR->ID
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/monarch-initiative/mondo/issues/551#issuecomment-443041118 , or mute the thread https://github.com/notifications/unsubscribe-auth/AckkEBzDEU6ytkyzhnvwY-sKfxcDzu7oks5u0HbegaJpZM4Y4Xi1 .  https://github.com/notifications/beacon/AckkEIQSbW1Y0uoxp6F8lkQ9Spk44vvbks5u0HbegaJpZM4Y4Xi1.gif

to do:

[ ] change everything in the tree pictured above including all branches but not the one branch I highlighted in blue. For that you’d change “syndromic genetic deafness” to “syndrome genetic hearing loss” but then not touch the items under syndromic genetic deafness.
[ ] for terms under nonsyndromic hearing loss: review the names used by the genetic hearing loss community as can be seen on the website maintained by the hearing loss community here: https://hereditaryhearingloss.org/
[ ] change these nonsyndromic names to be consistent with what the community actually uses despite the OMIM entries.

partial list from above:

[ ] syndromic genetic deafness
[ ] Charcot-Marie-tooth disease-deafness-intellectual disability syndrome
[ ] Hirschsprung disease-deafness-polydactyly syndrome
[ ] autosomal dominant keratitis-ichthyosis-deafness syndrome
[ ] ichthyosiform erythroderma, corneal involvement, and deafness
[ ] ichthyosis, hystrix-like, with deafness
[ ] retinitis pigmentosa-deafness syndrome
[ ] ocular albinism with congenital sensorineural deafness

albinism-deafness syndrome

to do:
all the children of 'nonsyndromic genetic hearing loss' should be renamed to 'hearing loss' instead of deafness. I will ask Nico to help me with this.


What about Global Admin

I notice that the Global Admin role is not included in the table. I'm assuming that this role can get all of the data, but I'd like this confirmed. Please update the table as appropriate.

Readme has been updated.


Missing license

I'd like to port this software to a more modern deep learning framework, but the package does not have a license. Could you please add a license file for the project, for example MIT or Apache license?
Thanks,
Henrik Petander

I added the license.
Thank you for pointing out.


override the signature of plotfunc

This fixes the documented signature of the plotting functions. Not sure if using a decorator to override __wrapped__ is the best way to achieve this, though.

[x] Closes #4152
[x] Passes isort . && black . && mypy . && flake8
[x] User visible changes (including notable bug fixes) are documented in whats-new.rst

Could you add a comment why this is needed to the code?

should we merge this?

Seems  OK to me. We should do the same in dataset_plot.py too but that can happen later...


avoid crash when insert zero column data

What problem does this PR solve?
Avoid crash when insert zero column data.
Issue link:#687
Type of change

[x] Bug Fix (non-breaking change which fixes an issue)

Duplicated with #700


Three enhancements: @include, @charset, and css file extension

Minify version: 2.1.2
PHP version: 5.1.6

What steps will reproduce the problem?
1. Aggregate CSS-files that includes @charset and @include statements
2. Link the the result via: <meta http-equiv="Content-Style-Type"
content="text/css" /><link
href="/public_tools/min/g=myCSS" rel="stylesheet" type="text/css" />
3. Open the page with Safari 4 under OS X 10.5.6

Expected output: Nicely rendered web page.

Actual output:
 * Invalid CSS file since @charset is only allowed at the beginning
 * No CSS layout since Safari is confused how to handle "/min/g=myCSS"

Did any unit tests FAIL? (Please do not post the full list)
 * No tested

Please provide any additional information below. If this has to do with URI
rewriting, please
include your DOCUMENT_ROOT.

I propose the following changes:

1.) RewriteRule ^([a-z])_(.*)\.css$  index.php?$1=$2  [L,NE]
=> this will treat the script output a a regular CSS file (compression, content
type, ...) and Safari
is happy with it

2.) At the end of _process($css): $css = preg_replace('/@charset ".*";/', '',
$css);
=> this will remove the @charset statements (maybe you could add one at the
beginning of the
output based on the input encoding)

3.) At the end of _process($css): $css = preg_replace('/@import url\((.*)\);/',
'/*WARNING:
IMPORT of "\1!"*/', $css);
=> Just as a very simple default solution for this common problem

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 29 Apr 2009 at 2:45

1. Minify should be sending Content-Type: text/
css;charset=UTF-8. As long as this header is sent, the browser shouldn't care
about
the URL. On your page w/o CSS showing, what's the Content-Type sent? (You'll
need an
HTTP sniffer).

2. Re: @charset, I want to keep Minify simple: Use UTF-8 in your files. You can
change the charset sent by setting $min_serveOptions['contentTypeCharset'] in
config.php, but the minifiers can't handle encodings like UTF-16 anyway. I'm
willing
to discuss this more on the Google group.

3. Something like this already implemented in 2.1.2:
http://code.google.com/p/minify/source/browse/tags/release_2.1.2/min/config.php#
69
Minify should prepend a note to the CSS if you've ended up with an @import not
at
the top.

Original comment by<EMAIL_ADDRESS>on 29 Apr 2009 at 8:24

Changed title: Three enhancements: @include, @charset, and css file extension
Changed state: NeedInfo

@include essentially dup of Issue 50.

Original comment by<EMAIL_ADDRESS>on 1 Jul 2009 at 8:29

Changed state: Invalid


Fixed bug with BlackrockIO not loading because of spike times before/after t_start/t_stop

I fixed a bug (see #420) that caused BlackRockIO to fail loading certain files because of spikes otuside of the time range set by the nsX start and stop times. Now _segment_t_start in BlackrockRawIO returns the smaller of t_start (read from available nsX files) and the first spike timestamp instead of t_start. Also the general t_stop is now set to the higher of t_stop from nsX and the last spike timestamp, thus  _segment_t_stop now returns this.
Apart from that I changed the value of self._t_starts in BlackrockRawIO from 0 to the first spike timestamp if no nsX file is available.

Coverage decreased (-0.05%) to 48.983% when pulling ce50269972ef1623ff649f8608864bd8875566c9 on bjoern1001001:fix/spiketrain_times into d9950a9c03858f42df7f8ff64a9c46a55280b6d1 on NeuralEnsemble:master.

This looks very good. Thanks.
To avoid any confusion for the day we will be back on the code I would explicitly change:

self._t_starts to self._sigs_t_starts
self._t_stops to self._seg_t_stops
keep self._seg_t_starts

Otherwise, just curious : why to np.math.ceil() instead of int() in _get_timestamp_slice ?
Note that I am OK with that.

Thanks for your comment.
I think you're right here and I will change this (probably on Monday).
I needed to do this because if it rounds downwards then spikes are loaded from the lower value. Then the time of the first spike might be lower than t_start. Let's say there is a timestamp at 40 samples and t_start is set by the user to (40.5/30000)*pq.s then SpikeTrain will raise an error because that spike at 40 samples is earlier than t_start. I admit this is probably are very rare situation but I managed to create it and thus noticed and changed this.

Coverage decreased (-0.05%) to 48.983% when pulling 93cfce6f0e94bfba628eec07002b90e9b3bff19f on bjoern1001001:fix/spiketrain_times into d9950a9c03858f42df7f8ff64a9c46a55280b6d1 on NeuralEnsemble:master.

I guess this is ready to merge now.
Did you run noestests ?

Yes, I just ran them.
The result is the same as for the Unittests: Two tests fail when testing BlackRockIO because it expects Unit.name to be a regular string but it's a Unicode string now. The other tests pass.

I guess you are in python27. Have you tried on py3 ?
The problem come from basefromrawio.py I have forgotten

at line 145 and 151.
I will fiw it in a separated PR.


express get apis throwing 404

All the GET apis are returning 404 not found for all routes. My GET api code is
const router = new express.Router();
/* All the Orders Route */
router
.route('')
.get(readOrders)
Using node v 8.9.4
and express 4.16.3

At the very least can you provide complete code that we can run and then provide the exact call to make to that server that is getting a 404 you expect to get something different on?

I must concur with wesley, please try with the pattern of router.get(path, readOrders).
I can confirm that everything is working as expected.

ahh I see this is a sub route, You wanted a wildcard. '/*' in the router, '/api_name' in express.use
It leaves you with a responsibility to handle bad routes though.

You always have to handle "bad routes".  In the case above, all you really want do to do is call next without an error:
app.get('/:id (req, res, next) => {
  const resource = db.getResourceById(req.params.id)
  if (!resource) {
    return next()
  }

  // do thing with resource then return it
  res.json(resource)
})

This would result in the next set of middleware being called, and if none other matched you would want a default 404 handler at the end of your application middleware stack:
app.use((req, res) => {
  res.status(404).json({
    someBody: 'Route not found or missing resource.....'
  })
})

This is a very common express pattern, if you have any questions it might be a good idea to reference the docs on the website.  Good luck!
The last bit of code there should be in all of your express apps, because typically you don't want the default express behavior because you want to return a custom 404 response telling the consumer what happened and why it was a 404.

I'm going to close since there hasn't been any followup to our help.


torch2trt import error

TensorRT weight Generator will process the image with height 720 and width 1280
Traceback (most recent call last):
  File "/Project/FAST_Anime_VSR/main.py", line 107, in <module>
    main()
  File "/Project/FAST_Anime_VSR/main.py", line 97, in main
    parallel_process(input_path, output_path, parallel_num=configuration.process_num)
  File "/Project/FAST_Anime_VSR/process/single_video.py", line 212, in parallel_process
    model_full_name, model_partition_name = weight_justify(configuration, input_path)
  File "/Project/FAST_Anime_VSR/process/single_video.py", line 92, in weight_justify
    generate_weight(h, w)               # It will automatically read model name we need
  File "/Project/FAST_Anime_VSR/tensorrt_weight_generator/weight_generator.py", line 257, in generate_weight
    tensorrt_transform_execute(lr_h, lr_width)
  File "/Project/FAST_Anime_VSR/tensorrt_weight_generator/weight_generator.py", line 206, in tensorrt_transform_execute
    ins.run()
  File "/Project/FAST_Anime_VSR/tensorrt_weight_generator/weight_generator.py", line 160, in run
    self.weight_generate()
  File "/Project/FAST_Anime_VSR/tensorrt_weight_generator/weight_generator.py", line 145, in weight_generate
    self.model_weight_transform(self.sample_input)
  File "/Project/FAST_Anime_VSR/tensorrt_weight_generator/weight_generator.py", line 92, in model_weight_transform
    from Real_ESRGAN.rrdb import RRDBNet
  File "/Project/FAST_Anime_VSR/Real_ESRGAN/rrdb.py", line 16, in <module>
    import Real_ESRGAN.torch2trt_fix    # For the bug override, don't delete it
  File "/Project/FAST_Anime_VSR/Real_ESRGAN/torch2trt_fix.py", line 4, in <module>
    from torch2trt.module_test import add_module_test
ModuleNotFoundError: No module named 'torch2trt.module_test'

i got this error on latest version

To solve the problem with the ModuleNotFoundError: No module named 'torch2trt.module_test' error in the torch2trt project, you can try the following:
Check the installation. Make sure that the torch2trt module is installed correctly. To do this, run the pip list command and check whether torch2trt is listed among the installed packages. If not, reinstall it using the pip install torch2trt command.
Check the environment. Make sure that the code runs in the correct environment where torch2trt is installed.  Different environments, such as virtual or conda, can have separate installations.  You can check the list of installed packages in the environment by using the pip list or conda list commands.


dont mention upstream quarkus

just quarkus as no diff from this and rhbq when it comes to the starters.

/retest

More CI flakes 😢
/retest

To be honest, the test run on minikube already passed, and this change wouldn't have any effect on the tests, so I'm fine merging this either way.
Since OpenShift CI has been kinda flaky for the past few test runs for us, I've marked the check as optional for now


[#issue-313] cmdline lib function list and getinfo has no info return

Added information processing of function list and function getinfo
Close #issue-313
Signed-off-by<EMAIL_ADDRESS>

I modified the return data again。
Getinfo returns blockinfo in JSON format, and list only returns the list of added channels


add website collection list page

Pre-Flight checklist

[x] Screenshots and design review for any changes that affect layout or styling

[x] Desktop screenshots
[x] Mobile width screenshots

[x] Testing

[x] Code is tested
[x] Changes have been manually tested

What are the relevant tickets?
closes #356
What's this PR do?
This adds the first part of the website collection frontend, a list page which shows the collections that have been created (no UI for creating or editing yet).
How should this be manually tested?
Create some WebsiteCollections via the django admin. Then go to /collections, and you should see them appear in the list. If you make more than 10 you should be able to page through them with the standard pagination controls.
Screenshots (if appropriate)

Codecov Report

Merging #369 (6354408) into master (eace5af) will decrease coverage by 2.87%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #369      +/-   ##
==========================================
- Coverage   89.02%   86.14%   -2.88%
==========================================
  Files         123       63      -60
  Lines        3881     2505    -1376
  Branches      594      279     -315
==========================================
- Hits         3455     2158    -1297
+ Misses        384      306      -78
+ Partials       42       41       -1

Impacted Files
Coverage Δ

main/urls.py
75.00% <ø> (ø)

...atic/js/lib/ckeditor/plugins/MarkdownMediaEmbed.ts

static/js/components/Dialog.tsx

static/js/components/widgets/RelationField.tsx

static/js/selectors/websites.ts

static/js/lib/ckeditor/plugins/test_util.ts

static/js/lib/api/util.ts

static/js/util/incrementer.ts

static/js/lib/ckeditor/turndown.ts

...tatic/js/components/forms/SiteCollaboratorForm.tsx

... and 45 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 eace5af...6354408. Read the comment docs.


Download rate limit and broken links question

Idk why but after 3:00 AM (gmt -3) google drive broke some links and say "the download limit has excedeed" but i still can download the file only streaming links does not work. however after 6 AM the video works normally. the question is, does the script break the download limit? Someone having this problem?

(https://drive.google.com/file/d/1VM5dDSGy_7NyTtSVgCz5v2bSagqRjLQQRQ/view)
also, some old videos broken with no reason and not back to work :/

(https://drive.google.com/file/d/0B2r31YvbMD0rYWwyOUwwdGpka0U/view)

@Yokutto I'll be further investigating about this problem.

Totally agree, this script cache reset not working and limit still happen.

Google drive has two bandwidth limit. It is stream and download limit. So cache will be useless.

@aalfiann The cache system lowers the TTFB too much in my json script, so it's not useless. idk about streaming limit because I already had videos using 20GB per day and it was not limited, I think this is a bug in google drive side

You can find the solution here: https://github.com/ArdiArtani/Google-Drive-Player-Script - he bypass file limit.


0.8.0 dependency issue

Our python project to communicate with azure b2c broke today with the 0.8.0 release.  If we revert back to 0.5.1 it works agin:
packages/kiota_abstractions/request_information.py", line 8, in <module>
    from opentelemetry import trace
ModuleNotFoundError: No module named 'opentelemetry'

I'm having the same problem.


[1.10.2][RFTools 6.01] Storage Scanner causing massive lag even while not scanning

I'm not sure what caused it to start after being fine for a long time, but since today the storage scanner causes a massive amount of tick lag even though it's not scanning for inventories or anything. See:

[12:22:04] [EntityProfiler/INFO] [TickProfiler]:
TPS: 1.6333333
Single Entity | Time/Tick |      %
StorageScannerTileEntity {-54,52,251}:0 |   603.067 | 98.369
TileExternalStorage {-64,57,247}:0 |     0.914 |  0.149
tile.TileController {-54,52,249}:0 |     0.732 |  0.119
TileExternalStorage {-71,52,247}:0 |     0.480 |  0.078
tile.TileExporter {-45,52,242}:0 |     0.402 |  0.066
Chunk | Time/Tick |      %
0: -4, 15 |   605.881 | 98.828
0: -5, 15 |     1.166 |  0.190
0: -2, 14 |     0.932 |  0.152
0: -4, 14 |     0.818 |  0.133
0: -3, 15 |     0.688 |  0.112
All Entities of Type | Time/Tick |      %
StorageScannerTileEntity |   603.067 | 98.369
TileExternalStorage |     1.706 |  0.278
tile.TileController |     0.756 |  0.123
tile.TileExporter |     0.711 |  0.116
monster.EntitySkeleton |     0.429 |  0.070
Average Entity of Type | Time/tick | Calls
StorageScannerTileEntity |   603.067 |    49
tile.TileController |     0.252 |   147
TileExternalStorage |     0.213 |   392
tileentity.TileGrinder |     0.112 |    49
monster.EntityEnderman |     0.104 |   192

There is no log output while running. I'm happy to do more debugging for any info you need.

Was there an update of any kind? Is there an item in the storage scanner input slot that it is trying to insert somewhere? Is the storage full? Is there automation inserting/extracting out of this?
I need more info

Right, sorry. I always forget what kind of info you'd need to reproduce an issue. There was no update, but it was trying to insert an item into storage which was full. This shouldn't cause such an extreme amount of lag so I didn't think it was relevant. No automation was connected to it apart from a quick dump screen linked to the scanner. To be more specific about my setup, the range on the scanner was maxed out (20) and I used it to insert patterns into my Refined Storage autocrafting setup. The only inventories that were marked as routable were all those crafters (40-ish), with a ton of unrouted drawers and the likes.

It was probably relevant. If you have a really big storage system then this can be very slow as it will try to find room for that item constantly.

I may be able to solve this by putting a delay on this after it failed a few times

I was going to suggest that too. I like to put at least a second delay in between expensive checks like that. Anyways glad I could help with my report :)

Thank you for taking the time to create this issue, as we're no longer supporting versions below 1.12, I'm going to go ahead and close it but if this bug seems to continue over to newer versions, please do report it again so we can investigate the issue further.


Title on SVG download not centred

When you download the chart as an SVG, the title text isn’t vertically centred in the blue bar as it is on the website/PNG – it’s a little too high up. Please could we fix this?

Closing as discussed.

@gilesdring we've dug into this a bit more. Inkscape seems to struggle with 'dominant-baseline = "hanging"' and shifts everything up a bit. This applies to not just the title, but also the axis labels and the footer.
A fix that we've found is replacing all instances of 'dominant-baseline="hanging"' with 'dy="0.85em"' (see the attached file for Ben Bradshaw, with the unedited version for reference).
Do you think there would be any undesirable consequences from this? If not, could you edit the code for the title, axis labels and footer to replace the dominant-baseline with dy="0.85em"?

Have added as small fix - will include in next release

Great, thank you

Thanks for making this fix Giles - happy for it to be marked as closed.


Derive Snafu for TaggedBlobError.

This implements Display, which allows us to take TaggedBlob types on the command line using structopt.

@jbearer Can we just manually implement Display without depending on Snafu? That would be sufficient for the downsteam?


Remove or refactor fhir-replication-api

fhir-replication-api defines an API for "replication events" that can be used to broadcast create/update/delete interactions to a messaging/event engine (e.g. Kafka).
To assist with reliable delivery of messages, the server writes a fhir-replication record to the database within the managed transaction.
However, when there is nothing processing these records (i.e. no separate component deleting rows in this table like in a simple deploy), it will grow unbounded.
We should either remove this project or refactor it so that it can be disabled.
Additionally, some of the metadata fields on these tables are not appropriate for a general-purpose fhir server implementation.

PR https://github.com/IBM/FHIR/pull/301


ES6-ified the project

The quickstart is now fully in ES6 including; replaced the old anonymous function syntax with the new arrow function syntax. Segregated import directives. Made the code a little more pretty.

Thanks for looking into this @labsvisual! I'm going to close, however, because we want to keep this project simple and without prescribing much like the dependencies and syntax included here.


event.requestContext missing path

On a real API Gateway in AWS, the event.requestContext object has a path string.  Sam Local does not.
event.requestContext on AWS:
"requestContext": {
        "resourceId": "bmjxgk",
        "resourcePath": "/client_reports/{proxy+}",
        "httpMethod": "GET",
        "requestTime": "15/Feb/2018:19:27:20 +0000",
        "path": "/api//client_reports/search_activity/USA_142687",
        "accountId": "XXXXXXXXX",
        "protocol": "HTTP/1.1",
        "stage": "api",
        "requestTimeEpoch":<PHONE_NUMBER>794,
        "requestId": "3de2046b-1286-11e8-b38f-79c8acf67300",
        "identity": {
            "cognitoIdentityPoolId": null,
            "accountId": null,
            "cognitoIdentityId": null,
            "caller": null,
            "sourceIp": "<IP_ADDRESS>",
            "accessKey": null,
            "cognitoAuthenticationType": null,
            "cognitoAuthenticationProvider": null,
            "userArn": null,
            "userAgent": "PostmanRuntime/7.1.1",
            "user": null
        },
        "apiId": "sxfluki8re"
    },

On Sam Local:
{
   "resourcePath":"/client_reports/search_activity/USA_142687",
   "httpMethod":"GET",
   "stage":"prod",
   "identity":{
      "sourceIp":"<IP_ADDRESS>:57470"
   }
}

Note also that the format of resourcePath is inconsistent.

Both of these (resourcePath inconsistent and no path in the requestContext) were addressed and corrected in the 0.3.0 release.
We set the path on the RequestContext here
Closing as this was addressed.


feat: provide link to documentation for compile time error codes

Fixes #176
Adding an error-code.ts namespace where we will store all error code constants. Each error code has an ID, code, messageText and documentation in TypeScript code.
export namespace ErrorCodes {
  /**
   * The computations that [Amazon States Language](https://docs.aws.amazon.com/step-functions/latest/dg/concepts-amazon-states-language.html)
   * can do is restricted by JSON Path and the limited [Intrinsic Functions](https://docs.aws.amazon.com/step-functions/latest/dg/amazon-states-language-intrinsic-functions.html). Currently, arithmetic expressions are not supported.
   * ```ts
   * // ok
   * new StepFunction(scope, id, () => 1 + 2);
   *
   * // illegal!
   * new StepFunction(scope, id, (input: {num: number}) => input.number + 1);
   * ```
   */
  export const Cannot_perform_arithmetic_on_variables_in_Step_Function: ErrorCode =
    {
      code: 100,
      messageText: "Cannot perform arithmetic on variables in Step Function",
    };
}

TS doc generates a markdown file under website/docs/api/namespaces/ErrorCodes.md. This file is not easy to read and hidden away in the file tree, so a script is used to pre-process the file and create website/docs/error-codes.md at the top-level of the file tree and in a linkable and easy to read format.

All errors reported by the language service or CLI now provide a link directly to this documentation explaining the error.

[ ] Make the link clickable within VS code (current not sure if this is possible)

@thantos @brendo-m  after this change is complete, please adopt this standard and contribute to the error codes. We should begin sharing al error codes throughout both the TS code phase and CDK synth phase

Have an example of how to use this in the compile and synth code?

Here is an example within validate.ts
function validateStepFunctionNode(
  node: typescript.Node
): typescript.Diagnostic[] {
  if (
    (ts.isBinaryExpression(node) &&
      isArithmeticToken(node.operatorToken.kind) &&
      !checker.isConstant(node)) ||
    (ts.isPrefixUnaryExpression(node) && !checker.isConstant(node))
  ) {
    return [
      newError(
        node,
        ErrorCodes.Cannot_perform_arithmetic_on_variables_in_Step_Function
      ),
    ];
  }
  return [];
}

Have not thrown any of these errors during synth yet. Need to do that. See https://github.com/functionless/functionless/issues/183


Update to documentation to include (Import-PackageProvider DockerMsftProvider)

In documentation one step is missing for pulling images. Once you install package provider you need to import it which is missing.

Are you thinking about the docker run microsoft/sample-dotnet step? Docker will pull the images automatically as part of the run.

No. On this page
https://msdn.microsoft.com/en-us/virtualization/windowscontainers/quick_start/quick_start_windows_server

Friism is perfectly right, but I also ran into that misunderstanding like cheese, some weeks ago.
Voting for close

@enderb-ms can you take this?

Closing this issue as it has been resolved above.


createst: Create a default README with every test

Feature: #5210
Previous PR: #1010
Description:

Replace name and email autocompletion to manual completion.

Link to the redmine ticket: https://redmine.openinfosecfoundation.org/issues/5210

Thank you for your contribution, @AkakiAlice ! :)
Based on suggestions by Jason, I have made some updates and staged a new PR for merge/review. #1129

Your PR was merged as a part of https://github.com/OISF/suricata-verify/pull/1156. Thank you for your contribution! :)


SDK Asset Organization.

Moved a good amount of things to the SDK and organized cursor assets.
Fixed BaseFocusHandler to require focus by default.
Updated documentation.
Updated basic focus handling.
Rolled back Editor to 2018.1.8f1
Refactored usage of getting Input System references into base classes.

Okay Simon. I refactored a few more classes to inherit from a base with the input system in a single place.  Couldn't do it everywhere, but got the easiest ones first.

This is ready for another review

I want to merge the devices refactor before this

I want to merge the devices refactor before this

Sounds good.