feat: Add document saver class and support save/load/delete user journeys.

Add document saver class and support save/load/delete user journeys.
In detail:

Load documents via default table
Load documents via custom table/metadata
Load documents via custom page content columns
Load documents via custom metadata columns
Initialize MySQLDocumentSaver with existing table
Initialize custom MySQLDocumentSaver table & add documents with metadata
Delete documents

Added do not merge label for now since this PR is quite large. Let's wait for the MySQLChatMessageHistory PR to get merged into main first before merging this into our staging branch.

@jackwotherspoon @totoleon Added unit tests and confirmed text spilter is no-op. Code is ready for review.

Removed do not merge label and switched base to main since this is a large enough PR to go right to main


Fix edit link field cursor behaviour

Fixes https://github.com/subconsciousnetwork/subconscious/issues/796
Port the RenameSearchView input field to ValidatedFormField. Reworked how caption works on ValidatedTextField in the process.

caption is now of type String.LocalizationValue?, forcing us to do the right thing at the callsite.


How to setup the thumb ratings plugin with the admin pannel ?

I was reading the docs of this plugin and i didn't find any informations about how to setup this plugin with the pannel ... Can you explain ? Thank's !

Please, what is the "pannel"? Did you mean the Admin plugin?

Admin panel, Admin board. Yeah, the Admin plugin panel/board.

Yes

The string {{ thumbs(page.slug) }} must be inserted in a twig file.
If you use the Quark theme you can insert it after this line https://github.com/getgrav/grav-theme-quark/blob/develop/templates/partials/blog-list-item.html.twig#L7, that is between the line 7 and 8.
This is the screen of what you see after installing the plugin: these are default values that can be modified as desired.


📝[README.md] fix syntactic highlighting on crates.io

GitHub seems to support  ```rs for syntactic highlighting of rust files. But it looks like crates.io needs  ```rust

Thank you, I had forgotten about that.


New version of sceneform does not support previous converted SFB

Hi! We are using sceneform for our mobile app. Currently we have 500+ 3D assets in GLTF format both animated and non animated. Since GLTH animation is not currently supported we are converting these assets to SFB to work on Android.
We faced a major issue that after each update of Sceneform all previous SFB models are not supported. Here is an error description:
019-11-11 11:48:17.834 8406-8406/com.snaappy E/Filament: Panic     in filament::Material *filament::Material::Builder::build(filament::Engine &):86     reason: Material version mismatch. Expected 3 but received 1. 2019-11-11 11:48:17.836 8406-8406/com.snaappy A/libc: Fatal signal 6 (SIGABRT), code -6 (SI_TKILL) in tid 8406 (com.snaappy), pid 8406 (com.snaappy)
Is there any compatibility of SFB files with newer version of the engine? How should we address this issue - there is no chance we can handle regular conversion of several hundreds 3D assets.

Did you find any solution to this problem ? Seems that I have got a similar issue. Thanks.

The only solution is to reconvert all the models on the newer version on sceneform. Which takes too much time to be honest


documentation which employs acronyms/jargon must define terms

We arrived at this page from an Office 365 notification with the following title:
"Secure by Default - Honoring EOP/ATP detonation verdicts"
Nowhere in that article was EOP defined.
Nowhere in that article was ATP defined.
After two layers of linking, we arrived at this page where EOP is defined, and ATP can only be inferred.
Please teach new documentation writers to define acronyms and jargon terms without assuming the audience knows their meaning. A footnote is a minimally intrusive solution.

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

ID: 472b3a0c-7ad8-8245-9b57-29431490370b
Version Independent ID: 8671f04a-b7b3-402c-1e65-59a52c1cdaf5
Content: Secure by default in Office 365 - Office 365
Content Source: microsoft-365/security/office-365-security/secure-by-default.md
Service: o365-seccomp
GitHub Login: @Dansimp
Microsoft Alias: dansimp

@trcoley Thank you for your feedback. Sorry about the confusion. Microsoft Defender for Office 365 is the new name for Office 365 Advanced Threat Protection. We have now added a note on the document to explicitly mention this change.
Thanks
Sri

@trcoley, I totally agree. Currently, I can find no references to advanced threat protection or ATP within the topic. I made sure the one reference to Microsoft Defender for Office 365 was linked. I also made sure that a link exists for Exchange Online Protection (EOP).
Thanks for the feedback.
Clsoing


Ensure properties is only invoked once

The static get properties was invoked twice because of an if-guard. Instead, invoke it once and reassign to a variable and use that in the if-guard instead.
Reference Issue
Fixes #5252

Test failure is in dom-repeat which seems unrelated. Not sure why this is suddenly breaking. Works just fine on master.

This PR is ready for review again


Add Icons To Footer

Description
Hello Project Admin
I would like to add icons to Footer to make website look attractive
Can you assign me this issue under GSSOC'24 @pooranjoyb
Screenshots

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)

Congratulations, @sailaja-adapa! 🎉 Thank you for creating your issue. Your contribution is greatly appreciated and we look forward to working with you to resolve the issue. Keep up the great work!We will promptly review your changes and offer feedback. Keep up the excellent work! Kindly remember to check our contributing guidelines

Yes do this in one PR only.

You mean you want me to add icons to Navbar and footer in one PR @pooranjoyb

You mean you want me to add icons to Navbar and footer in one PR @pooranjoyb

Yes do it in #315 itself :)

Yeah okk @pooranjoyb

@pooranjoyb can you please clarify me why u r Mentioning #315??

Opps sorry, my bad :/

You want me all the icons and Rate us option in One PR right??

I will do that no problem @pooranjoyb :)


Add pulse animation for the thumbnail

It is a tried to figure it out how to inject CSS ans animation to an element.
If you think it's a good idea I can further

It's excellent. I had the idea to make something like, like injecting CSS to the current tab. I did not know exactly how to do it and did not have time to investigate.
I don't see bad aspect with this. The only thing is to make the pulse only once and take car that if you change your thumbnail providing a new one, you must remove the last style and put it again to play the animation again once. You see what I mean?


Should Sinatra::Reloader reload i18n files (.yml)?

The extension Sinatra::Reloader should reload other files such as yml with translations used by the gem i18n? Here is the setup:
# Gemfile

gem 'i18n'
gem 'sinatra'
gem 'sinatra-contrib'

# my_app.rb

require "sinatra/base"
require "sinatra/reloader"

class MyApp < Sinatra::Base
  configure :development do
    register Sinatra::Reloader
  end

  configure do
    I18n::Backend::Simple.send(:include, I18n::Backend::Fallbacks)
    I18n.load_path = Dir[File.join(settings.root, 'config/locales', '*.yml')]
    I18n.backend.load_translations
  end

  ...
end

# config/locales/pt.yml

pt:
  hello: "Olá Sinatra"

After changing the pt.yml file, the changes would be visible only after restarting the server.

Can't you use also_reload to include the .yml files, see http://www.sinatrarb.com/contrib/reloader.html#label-Changing+the+Reloading+Policy ?

Hi @canton7, thank you for the suggestion.
Before opening this issue I tried to use also_reload, however I ended up with an error while trying to require an .yml file, and then I realised that perhaps only .rb would be reloaded, but I am still not sure.

I'm having the same experience, I tried using also_reload and it is fine starting up, but after changing the content in my yml file I get an error saying it can't load the file and the error message says it is looking in my home directory even though I have specified the path the same way as other also_reload files that are .rb.

I think because '.yml' files aren't technically required but read from source?
It looks like, from here, that the reloaded essentially just calls require on the file after removing the element from Sinatra and removing the file from $LOADED_FEATURES.

We could add a method after_reload that would accept a block allowing the user to call I18n to update yml files or run anything he needs after the reload. What do you think @zzak ?

@stjhimy Sounds good to me, will you work up a patch for after_reload feature as well as documentation regarding this behavior?

@zzak yes, I'm on it.


App is out of date?

Tried the app yesterday on MacOS:
Android Studio Koala | 2024.1.1
Build #AI-241.15989.150.2411.11948838, built on June 10, 2024
Runtime version: 17.0.10+0-17.0.10b1087.21-11609105 aarch64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
macOS 14.5
Kotlin Plugin: K2 Kotlin Mode (Alpha)
GC: G1 Young Generation, G1 Old Generation
Memory: 8192M
Cores: 16
Metal Rendering is ON
Registry:
ide.instant.shutdown=false
debugger.new.tool.window.layout=true
ide.experimental.ui=true
On emulator:
Pixel_3a_API_34_extension_level_7_arm64-v8a
The App compiles and run which is great in itself, but seem there are maybe a few problems? For instance the Sign-in button doesn't do anything for me. There's nothing in logcat coming out.
Just putting this here in case there is interest in trying to debug.
Best,
Pierre

Hi Pierre,

If your emulator does not have Play Store / Google Chrome installed, that would explain the problem, since the code sample uses a Chrome Custom Tab.
Also ensure that you configure a PIN on the emulator before clicking Sign In, since the app is meant to do nothing if Sign In is clicked when a PIN is not configured.

I ran the app OK this morning on may macOS 14.5 system. I run an x64 architecture, though that shouldn't change anything.

If you need to debug, this code is hit when you click Sign In, so you could output statements to logcat or set breakpoints there. Hope this gets you up and running. Always interested in ideas for improvements to code and docs ...
Regards,
Gary


Chart not rendering inside Side Drawer

Describe the bug
I was trying to implement line chart on side drawer. But not able to load chart
Expected behavior
Chart should be loaded on side drawer.
Screenshots

If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

FYI, I think putting things like charts and tables in the side drawer goes against design guidelines.
However, it does seem like a legitimate bug that the chart doesn't render inside a container initially set to display: none.

This was actually a CSS positioning issue with the Side Drawer component in shidoka-applications. It is resolved in v1.6.2.

Hi @brian-patrick-3 , Still this is not working with sample code i have attach. Can you have a look please

Can you create an example in CodeSandbox that reproduces it? I am unable to reproduce in<EMAIL_ADDRESS>

@brian-patrick-3 PFB link for codeSandbox
https://codesandbox.io/p/devbox/gifted-ellis-jj8dq2?file=%2Fsrc%2Fapp%2Fapp.component.ts%3A11%2C25&workspaceId=a63da9bb-5496-4111-be41-7fd9d81ab734

The issue is in how you've written your code:

Remove *ngIf="loadChart" from each <kd-chart>. This is the main problem.
Remove (click)="drawerOpen()" from anchor span. The button to open the drawer is built-in so this is not needed.
Remove extraneous wrapper divs: <div style="height: 500px;">. Unnecessary.
Remove any inline arrays/objects you are passing to <kd-chart> props (labels, options, datasets), and use variables instead. Inline array/object type props can cause unnecessary re-renders.
On <kyn-side-drawer>, [hideFooter]="true" can just be hideFooter since it will never change.

Forked working example

Hi @brian-patrick-3 , Still its not working after doing all the changes suggested by you. Can you please give me access to fork working example. Its a private folder. Need to move out to public. Thanks
https://codesandbox.io/p/devbox/sidedrawerwithchart-jj8dq2?file=%2Fsrc%2Fapp%2Fapp.component.html%3A52%2C21

It should be public now I think

@brian-patrick-3 Not working on your codesandbox URL too. First chart work for firefox but not for chrome. And 2nd chart is not working in both browser

Thanks for the update. I have one more theory to test tomorrow, it may be related to the chartjs deferred plugin.

Removing chartjs-plugin-deferred from being included by default seems to have resolved this in 1.1.7. It's not ideal, but it can still be added manually if anyone needs it.


Update exposing-stream-as-managed-api-in-service-catalog.md

Fixing minor typos in this page.

Addresses https://github.com/wso2/docs-apim/issues/4529


SMNumberWheel

Please add SMNumberWheel under UI/PickerView

I have created a CocoaPod for the custom made UI (SMNumberWheel) and would like to have it in your list.
Project URL
https://github.com/SinaMoetakef/SMNumberWheel respecting your contribution guidelines.
Description
I added the link and description under UI/PickerView.
Why it should be included to awesome-ios (optional)
The UI component is written is swift 3.0 and supports iOS 9.0 and later. It also supports sounds and the new haptic feedback feature (iOS 10 for iPhone 7 and 7+). It is very smooth, highly customizable, renders in InterfaceBuilder, very accurate and very fast.
The sample project demonstrating the wheel is very well done.
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 or later
[x] Supports Swift 3
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Error

:no_entry_sign:
Found 2 link issues

Link issues by awesome_bot

Line
Status
Link

757
301
https://github.com/ideaismobile/IDMPhotoBrowser redirects tohttps://github.com/thiagoperes/IDMPhotoBrowser

1586
301
https://github.com/k9101/FCAlertView redirects tohttps://github.com/krispenney/FCAlertView

  Generated by :no_entry_sign: danger

I think it's better under /Switch.
@dkhamsing what do you think?

How about Stepper

Stepper it is. @SinaMoetakef please change to the /Stepper section 👍

@lfarah
Okay, I'm going to create another pull request under stepper.
Thanks 👍👍


Add Hour_Of_Day Function As An Alias Of Hour

Description
Adds support for the hour_of_day function as an alias for the hour function which currently exists in opensearch
Issues Resolved
#722
Check List

[X] New functionality includes testing.

[X] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[X] New functionality has user manual doc added

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1226 (f931b9a) into main (1b58f7d) will decrease coverage by 2.48%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main    #1226      +/-   ##
============================================
- Coverage     98.32%   95.84%   -2.49%
- Complexity     3550     3551       +1
============================================
  Files           344      354      +10
  Lines          8762     9425     +663
  Branches        554      673     +119
============================================
+ Hits           8615     9033     +418
- Misses          142      334     +192
- Partials          5       58      +53

Flag
Coverage Δ

query-workbench
62.76% <ø> (?)

sql-engine
98.32% <100.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

...c/main/java/org/opensearch/sql/expression/DSL.java
100.00% <100.00%> (ø)

...arch/sql/expression/datetime/DateTimeFunction.java
100.00% <100.00%> (ø)

...h/sql/expression/function/BuiltinFunctionName.java
100.00% <100.00%> (ø)

workbench/public/components/Header/Header.tsx
100.00% <0.00%> (ø)

workbench/public/utils/PanelWrapper.tsx
100.00% <0.00%> (ø)

workbench/public/components/SQLPage/SQLPage.tsx
100.00% <0.00%> (ø)

workbench/public/components/PPLPage/PPLPage.tsx
56.52% <0.00%> (ø)

workbench/public/application.tsx
0.00% <0.00%> (ø)

workbench/public/components/app.tsx
0.00% <0.00%> (ø)

...ublic/components/QueryResults/QueryResultsBody.tsx
68.32% <0.00%> (ø)

... and 3 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

BWC Test failed, let's ignore it for now.


UI is small

The UI is pretty small. The fonts being thin are not visible properly on a large window.
Either changing the fonts to a bolder variant, or increasing the size of the app based on editor font size could be two solutions.
The other actionable buttons are fine but the input font is illegible.
Here is a screenshot for reference:

Made font size relative to editor size with latest release (1.3.9)


feat(api): Add api endpoints on lab repo

Adds three endpoints to fetch icons from lab repo:

/api/icon-nodes
/api/icon-details
/api/icon-metadata

This will be deployed on Vercel: lab.lucide.dev

Idea
What about loading this data from https://unpkg.com/<EMAIL_ADDRESS>instead, maybe that's simpler?

Yeah, we had this before using lucide-static, not a bad idea. but I tried to quickly setup with the current setup we also use for the site.
I think not a bad idea to eventually create scripts package that generates all these files for you.
But to continue and make lab available on the site, I first continue with this approach and refactor it later


Update STYLE.md with instructions that actually work

Signed-off-by: Raven Black<EMAIL_ADDRESS>
Commit Message: Update STYLE.md with instructions that actually work
Additional Description: While trying to send a pull request for an extension I was baffled multiple times by poor error messages and surprising behaviors. The alterations here each derive from a confusing thing that mostly also didn't get any clear "someone understood it better" signs from the dev slack channel.
First surprise, if you use option (udpa.annotations.file_status).work_in_progress = true; like the doc suggested, proto_format.sh would remove it. Suggestion from the slack channel was to use xds.annotations.v3.file_status instead, which this updates STYLE.md to reflect. (related slack thread)
Added a link to security posture and status documentation, which was otherwise hard to make sense of without foreknowledge.
Adjusted the section that suggests using work_in_progress=true or package_version_status=ACTIVE - any combination of these (with either xds or udpa version of work_in_progress) that doesn't involve package_version_status=ACTIVE causes proto_format.sh to delete the file entirely. So the only viable options appear to be just package_version_status=ACTIVE, or that and xds work_in_progress=true which is already covered in an earlier bulletpoint.
Clarified that without ACTIVE, proto_format.sh will delete the file (which hopefully will make it more discoverable for developers experiencing that symptom).
Added a mention of api/BUILD's v3_protos section, without which proto_format.sh will automatically remove any imports of your new proto from other protos.
Added mention of [#extension:] tag, which is also necessary, and extra detail about where [#extension-category:] has to go (it doesn't appear to work if present at the top-level). (related slack thread)
Risk Level: None, documentation only.
Testing: Verified that the old documentation doesn't work for at least one developer, and the new documentation is better. :)
Docs Changes: That's all this is.
Release Notes: n/a
Platform Specific Features: n/a

Sorry about force-push - it was required because apparently clicking to apply a suggestion commit makes an unsigned commit which upsets DCO...

/retest

This is great @phlax. Is there anyone in @envoyproxy/first-pass-reviewers who hasn't already written an extension with proto and would have some cycles to run through these instructions? I feel this is the ultimate test :)

just to clarify @htuch - @ravenblackx is the author - my only contrib is a minimal review - but agree this is a great improvement
as a way forward im wondering if its best to land and then post to the slack/dev channel to encourage some acceptance testing

+1 to that - given this has been stalled with no volunteers for a few

Still found some confusion following the updated doc, it turns out - adjusted it to be more helpful in those regards.
(I had the #extension annotation above the third message in the proto file, which is the actual config message - doing that causes the annotation to not be usefully detected, and you don't get a helpful error like "this annotation should be at the top", it just acts the same as if the annotation was absent from the file entirely.)


Possible to use without xcode?

It would be great to use this without xcode, or to be more exact to run it in docker, like the official swagger codegen? Having xcode as a dependency really limits options for using this where the team maintaining the swagger definition is not iOS devs (no mac or no xcode). Not sure how it fits but swift can run server-side via clang on linux, does that help? 🤷♂ 🤞

Hi @nhoughto, once https://github.com/yonaskolb/SwagGen/pull/157 is merged I'll build a docker image with SwagGen and make it publicly available. Stay tuned!

Sounds great @Kastet!

@Kastet So... #157 has been merged. Were you able to build a docker image?


Shared libraries not loading correctly on Linux

Trying to use swiftenv with Vapor, I ran into an issue where trying a command like vapor build would return error while loading shared libraries: libswiftCore.so: cannot open shared object file: No such file or directory. I found in Perfect-Ubuntu/install.sh a fix for this (run as sudo):
touch /etc/ld.so.conf.d/swift.conf
echo "/usr/lib/swift/linux" >> /etc/ld.so.conf.d/swift.conf
ldconfig

So I used /home/kyle/.swiftenv/versions/3.1.1/usr/lib/swift/linux (3.1.1 because that's the current global version I'm using) in place of /usr/lib/swift/linux and all now works well. Wasn't sure if you wanted to incorporate this fix into your install scripts.
Unrelated P.S. - Nice name/initials.

@kylefrost Are you able to produce the same problem with swift build?
I'm trying to understand if this is a problem specifically with something that Vapor is doing, or if there is a problem in swiftenv with the swift compiler not finding the dynamic libraries relative to itself.


Open EQUELLA Documentation isn´t available

Open EQUELLA Documentation isn´t available in https://www.apereo.org/projects/open-equella
Steps to reproduce the behavior:

Go to 'Open EQUELLA Documentation'
Click on 'Documentation for Getting Started', for example
See error '404: Sorry, we can’t seem to find this page’s pixylls.'

Documentation links have been updated


Add helper function ElemwiseAttrHelper

Sometimes, we need to call ElemwiseAttr without NodeAttrs for example when calling from legacy op infer type in MXNet. Adding a helper function for the same.
Suggestions for better name / other suggestions welcome!

Closing, didnt realize these functions are also in MXNet.


Something is odd in digits-classification plot

There seems to be an error in the last plot (line 32) of 05.02-Introducing-Scikit-Learn.ipynb (digits classification example). The green labels are not matching those in the figure in line 23.

Good eye – thanks for pointing that out! The plot should show the test points rather than the training points. I'll try to push a fix tonight.

Fixed in e912ac11209db796f3c0d4b3a91d6c622c2756b8. Thanks!


[FEAT] Enable moving in Light Microscope from Minimap

We should be able to click on a region in the overview image and move to the same position in the light microscope
in practice i think this is just applying the offset?

Feature flagged, and ready for testing


MacBook10,1 is not compatible with macOS Sonoma

The model MacBook10,1 is listed in the supported Macs for macOS Sonoma which is wrong. The latest compatible macOS version according to Mactracker is macOS Ventura. Unfortunately Apple does not list the latest compatible macOS version in this article: https://support.apple.com/en-ca/103257

Thanks for recording the issue, how would you recommend we resolve that?

(Whoops, totally overlooked that's our data 😅)


[Bug] Default Github Pages workflow gets triggered alongside merging pull request

It appears that somehow the default Github Pages workflow gets triggered: https://github.com/alloy-rs/book/actions/runs/8970561854

For future reference: to work around this make sure to always use PRs and not push directly to main


New services menu for nav

Codecov Report

Merging #5 (d8516f0) into develop (8d8757c) will decrease coverage by 0.28%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop       #5      +/-   ##
===========================================
- Coverage   100.00%   99.71%   -0.29%
===========================================
  Files           32       34       +2
  Lines          329      349      +20
  Branches        34       37       +3
===========================================
+ Hits           329      348      +19
- Misses           0        1       +1

Impacted Files
Coverage Δ

src/components/NavBar.jsx
100.00% <ø> (ø)

src/components/ServicesLink.jsx
100.00% <100.00%> (ø)

src/utils/canViewPage.js
75.00% <0.00%> (ø)

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 8d8757c...d8516f0. Read the comment docs.


Marker behind line limit

Hi Daniel,
I'm working on loading multiple charts with different datasets. Most of the charts have a line limit and marker called on each data point. For some reason my marker is being drawn behind the line. Any thoughts?

I checked the code, drawLimitLine is called before drawMarkers, so it should not happen...
I tried with ChartsDemo's line chart, negative as well.
Could you debug on your side?

Ok I looked into it more and I think the issue is on our custom BarChart class. We have implemented the limit line on drawRect
// after render chart, render limit line again
override func drawRect(rect: CGRect) {
super.drawRect(rect)
    // we want to render the limit line label (7k) to the left of the data. The default impl clips the drawing context
    // we need to redraw it here w/o clipping the context
    if let context = UIGraphicsGetCurrentContext() {
        leftYAxisRenderer.renderLimitLines(context: context)
    }
}

If I remove this the marker appears on top of the line but the left axis value doesn't show up (7,000)

in your custom bar chart class, usually you don't have to change drawRect, just override the drawing functions.
If you don't want the limit line to be clipped beyond contentRect, then you have to change your rendering order.
I think this can be closed since you find the root cause.


Define API contract between API Gateway and ORM

Fixed here https://github.com/airavata-courses/dexlab/commit/d167d6fb1b1a51feb940cf642b20917a7a6122f5


ISA: return proper type for edge/qubit dead identifier

The dead identifier of an qubit/edge is not a Pythonic True, rahter string 'true':
>>> for qubit in device.isa.qubits:
...     print(qubit)
...
Qubit(id=0, type='Xhalves', dead=False)
Qubit(id=1, type='Xhalves', dead=False)
Qubit(id=2, type='Xhalves', dead=False)
Qubit(id=3, type='Xhalves', dead='true')
Qubit(id=4, type='Xhalves', dead=False)
...

This PR fixes it to be actually set to True:
>>> for qubit in device.isa.qubits:
...     print(qubit)
...
Qubit(id=0, type='Xhalves', dead=False)
Qubit(id=1, type='Xhalves', dead=False)
Qubit(id=2, type='Xhalves', dead=False)
Qubit(id=3, type='Xhalves', dead=True)
Qubit(id=4, type='Xhalves', dead=False)
...

Thanks for pointing this out, it'd be more ideal to fix the cloud to return actual Boolean values instead of strings /cc @pkaralekas @ecp-rigetti

Thanks for reporting this @tbabej. I looked and this is indeed an issue with the form of the data stored in the cloud. We'll have this fixed soon.

I just checked: the compiler guts expect a JSON boolean in this field. An empty field defaults to true; "true" acts as if it were true; true acts as if it were true; false acts as if it were false, and "false" acts as if it were true. We should be able just to change this at exactly the point @cbosborn suggests.

@tbabej This should now be resolved on the cloud end. Please check the device info for acorn and see that the attribute is now True. Closing this PR, if anything is wrong feel free to reopen and comment. Thanks again.


Activity bar disappears when clicking on the active tab when in "top" or "bottom" positions

In vanilla VSCode, clicking the active tab icon only causes the activity bar tab to collapse, but the bar with the icons stay visible. So when it's integrated into the tab, the whole tab disappears.
I guess a solution would be to block the collapse action entirely when in "top" or "bottom" positions.

Is this supposed to be fixed in APC v0.1.1? Unfortunately I can still reproduce the issue (using latest VSCode v1.79.2)

working
macos vscode 1.80.0-insider
macos vscode 1.79.2
macos vscodium 1.79.2
I need more details
os, settings.json etc

Windows 11 running a totally fresh VSCode:
Version: 1.79.2
Commit: 695af097c7bd098fbf017ce3ac85e09bbc5dda06
Date: 2023-06-14T08:57:04.379Z
Electron: 22.5.7
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621

settings.json:
{
    "apc.activityBar": {
        "position": "top",
        "size": 35
    }
}

And just for the good measure:

this is expected behavior,
science activity bar integrated in the sidebar, when you close sidebar and activity bar is closed
when you open sidebar through the menu or ctrl+b/cmd+b activity bar opens too

if you want to have an activitybar and a closed sidebar,
remove setting for activity bar
the current layout in vscode does allows integration into tabs

Ok, my issue is that I'm comparing behaviour to iocave's Customize UI, in which clicking on the current active "icon" doesn't close the sidebar.
In the example below I click multiple times on the 'explorer' and 'search' icons but the sidebar doesn't close (I would consider this expected behaviour), while in APC it closes the sidebar.

I'm assuming this will be marked as a non-issue then.

solution for you will be
"workbench.activityBar.iconClickBehavior": "focus"
since the default behavior is that clicking on the active icon closes sidebar
but I won't block the closing sidebar
I use it all the time ctrl+b

solution for you will be
"workbench.activityBar.iconClickBehavior": "focus"

Ah thank you, that is an adequate fix for me. 👍

solution for you will be
"workbench.activityBar.iconClickBehavior": "focus"

Ah thank you, that is an adequate fix for me. 👍

note
this has nothing to do with this extension
it is vscode default behavior


Update AppVeyor CI matrix.

Remove duplicate CI run for PowerShell Core Edition in WMF 5 OS image.

Hi @bmanikm, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;


Add Orange Unified School District

Add Orange Unified School District to the existing approved list.
Unfortunately, there's no way to pinpoint which email belongs to which schools or students or teachers, as they all share @orangeusd.org domain.

@huy-kunaico First stage of verification passed, official domain is orangeusd.org as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@huy-kunaico First stage of verification passed, official domain is orangeusd.org as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@philipto Appreciate it! Looking forward to the good news!

@huy-kunaico Thank you for your patience while waiting for a review and processing your request. Pull request merged.


nixos/pam: pam_mkhomedir umask to 0077

Motivation for this change
pam_mkhomedir should create homedirs with the same umask as the rest of the system. Currently it creates homedirs with go+rx which makes it readable for other non-privileged users.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

The change LGTM, and I agree it's a good idea to use 0077 by default.
Or maybe even make an option out of it...
The commit message should be split into title+body, though.

The commit message should be split into title+body, though.

Sorry, first MR here.
Looking at the CONTRIBUTING.md I think you mean split by an extra new line. If so, that is now done.

@oxij I splitted the title+body

@ofborg eval

Maybe the urgency is not clear; this is a security issue:
pam_mkhomedir creates home directories which are world readable (755). This is unexpected because normal (adduser/useradd) behavior is to create home directories with 700.
The umask set here does not relate to the umask of the newly created user when logged in.
Please tell me what to do to get this merged.

I wonder if this qualifies for a release notes entry.

@mweinelt yes, release notes would be good here.
@nberlee are you able to add a quick entry to the release notes mentioning this change? I promise we'll merge ASAP!

@aanderse i added release notes

Thanks everyone!


GTDB API Error

The GTDB API is currently down. BGCFlow should check if the API is down or not, and give choice to the use to continue the run or not.
import requests

url = "https://gtdb-api.ecogenomic.org/"

try:
    response = requests.get(url)
    if response.status_code == 200:
        print("API is up and running")
    else:
        print(f"API is down or experiencing issues. Status code: {response.status_code}")
except requests.RequestException as e:
    print(f"An error occurred: {e}")

Solved in https://github.com/NBChub/bgcflow/pull/295


RegionProps constructor returns error for hemisphere'ed region(s)

version/hash: 6831d2df53fe5715738ccb752604fcdfabc32340
to reproduce:
import siibra as sb
atlas=sb.atlases.MULTILEVEL_HUMAN_ATLAS
atlas.select_region('4p left')
regionprop=sb.region.RegionProps(
    atlas.selected_region,
    sb.spaces.MNI152_2009C_NONL_ASYM
)

expected result: returns regionprop
actual result: AttributeError:
[siibra:WARNING]  No mask could be computed for Area 4p (PreCG) - left hemisphere
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/xiao/dev/projects/siibra-api/venv/lib/python3.6/site-packages/siibra/region.py", line 488, in __init__
    M = np.asanyarray(mask.dataobj)
AttributeError: 'NoneType' object has no attribute 'dataobj'

some diagnosis seems to suggest that ParcellationMap is indexed by non hemisphered region (ie, parent node of 4p left)

Confirmed. This had been fixed in an earlier version already, and included an update to the confiugration files which had been messed up when merging branches of the configuration repository. I will fix it on develop.

This error should now be fixed on develop. @xgui3783 could you check this again?


Rename providors validator pkg to validation

Rename the provider specific validator package, which is a noun, to validation to align with coding guidelines.

/cc @gwesterfieldjr @jacobweinstock @pokearu @panktishah26 @ptrivedi @abhinavmpandey08

Closing as we might merge with the existing validations package at the top level.


[Feat]: Any plan to refactor in C++?

Is your inference  system running on python?

Our inference demo is based on the TensorFlow Serving framework, the official serving framework for TensorFlow based on C++. The python code is simply a demo for building the grpc request. You can use any language you like to build the grpc request.


command 'lsp-start' already defined

I'm attempting to follow the installation instructions.
My (reduced) kakrc file is
 Enable the plugin manager
source "%val{config}/plugins/plug.kak/rc/plug.kak"

# Configure the lsp server
plug "ul/kak-lsp" do %{
    cargo build --release --locked
    cargo install --force
}
eval %sh{kak-lsp --kakoune -s $kak_session}
lsp-enable

On starup I get the following errors in the *debug* buffer:

Closing as it's #180 duplicate.


Doesn't work with Bootstrap modal

Hi,
I just tried to implement this with one of our internal tools and noticed that typeahead.js will not work correctly in a modal dialog.
I already use the special bootstrap css.
JSFiddle:
http://jsfiddle.net/xcfbs/1/
Screenshot of the issue:

Sadly just fiddling with the z-index doesn't help the issue and I fear that it is a layout issue.

@KMontag42 did you ever find a solution for BS 3?

@nickmerwin this was a long time ago, I believe I did, but I do not recall the solution. Sadly I do not have access to the code anymore due to a change in job.
I can take a look at this over the weekend and see if I recall my solution.

Thanks @KMontag42, I was able to solve it by specifying a data-container=... value of an element ID within the modal.

I have tried the z-index and overflow fixes without any luck, @nickmerwin could you explain more in detail how you solved it with a data-container=... ?

.typeahead.dropdown-menu {
	z-index: 1051;
	position:relative;
	top:0!important;
	left:0!important;

}

Not a perfect solution but this did the trick for me. Using Bootstrap 3

You need jquery-2.2.4 and bootstrap 3.3.7 to make typeahead works on
Bootstrap 3
I think that was a problem, right?
On 13 February 2017 at 10:35, Sudeep Rane<EMAIL_ADDRESS>wrote:

.typeahead.dropdown-menu {
z-index: 1051;
position:relative;
top:0!important;
left:0!important;
}
Not a perfect solution but this did the trick for me. Using Bootstrap 3
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/twitter/typeahead.js/issues/147#issuecomment-279336238,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AK99tubTaeUR8FdFy0Gike9izKfk8GJdks5rcCPygaJpZM4AiHu9
.

--
Sve najbolje,
Velimir Majstorov
White Hat Security Serbia
Www.Majstorov.Info


Report without devDependencies

For my understanding the Node.js Analyzer in conjunction with the Node Audit Analyzer inspects the package-lock.json. And the package-lock.json always includes the devDependencies.
The dependency-check-report.html reflects this and is including all dependencies.

On a clean environment (no npm command before, no package-lock.json available) I did an npm install --production
As expected ;-) only the production dependencies are installed, but
The package-lock.json was created with all dependencies (prod and dev)
I run an OWASP/DC scan
The reports shows also all dependencies (prod and dev)

Now my question: Is there a optional parameter/possibility to avoid the inclusion of the devDependencies in the report ? The devDependencies in the package-lock.json shows "dev": true,, the production dependencies have no "dev" entry.

At the moment no. Everything in the package-lock.json is analyzed.


Update tests to refect we don't have the consumer mode, since ages

Signed-off-by: Matthias Wessendorf<EMAIL_ADDRESS>Fixes #

Proposed Changes

clean uo

Release Note

Docs

/assign @pierDipi


Coaches: redirect to 404 when edition is invalid

fixes #423

Instead of doing this manually for every request I think it's better to use an interceptor. I've had this assigned to myself for a long time but never really got around to it, so I'll look into getting it done tonight!


Rule svelte/mustache-spacing doesn't accept values "never" and "always"

Before You File a Bug Report Please Confirm You Have Done The Following...

[X] I have tried restarting my IDE and the issue persists.
[X] I have updated to the latest version of the packages.

What version of ESLint are you using?
8.34.0
What version of eslint-plugin-svelte are you using?
2.19.0
What did you do?
You can just try it here in the playground of the docs: https://ota-meshi.github.io/eslint-plugin-svelte/rules/mustache-spacing/
Instead of /* eslint svelte/mustache-spacing: "error" */ on line 2, write /* eslint svelte/mustache-spacing: ["error", "always"] */, and you'll get the error message
Configuration for rule "svelte/mustache-spacing" is invalid:
	Value "always" should be object.

What did you expect to happen?
I expect the plugin to lint and fix according to the provided config.
What actually happened?
The plugin throws an error stating that the provided config is invalid.
Link to GitHub Repo with Minimal Reproducible Example
n/a
Additional comments
No response

Please check the documentation.  The rule does not accept that option.
https://ota-meshi.github.io/eslint-plugin-svelte/rules/mustache-spacing/

Oh, then I would suggest switching up the wording. The way it reads now tells me that you can either use the two strings "always" or "never", or supply an object. It's also confusing that both, "always" or "never", somehow are default values.

Could you please send us a PR to improve the documentation?


core/iwasm: fix nuttx build on arm64 host (macOS)

Add aarch64 files on nuttx platform
Signed-off-by: Peter Bee<EMAIL_ADDRESS>Change-Id: Idf94cf6aba01f1f9191412a71ef31c1a3b944c78

@no1wudi Could you help review the PR? Thanks.


invalid literal for int() with base 10: 'b'  when volume up work 5% then crash

Version 0.0.2 volume- works OK, but when volume up it's maximize 5% then crash with error invalid literal for int() with base 10: 'b'
Old your version with “Volumouse_pactl_1_right.py” work OK, but volume+ without max_volume limit.
Now I delete line 661:                               if int(str(subprocess.check_output("pactl get-sink-volume "+Top_right_card+" | grep % | cut -d '/' -f2 | cut -d'%' -f1", shell=True).rstrip()).replace("b' ","").replace("'","")) > max_volume_top_right :
And it's work without crash, but volume up at one moment 150%. Volume down minimize OK(5%)
My OS - Rosa Fresh KDE (kde 5.27.10) In other OS on my PC it works without problems.
Is there any thoughts about it?
In any case - Thank you for your volumouse!

You can comment or delete lines 660, 661, 662, 663 and 680, 681, 682, 683 and 698, 699, 700, 701 and 716, 717, 718, 719.
To comment a line just add # at he beginning of the line :
#                          if str(Top_right_card) != 'no_soundcard':
#                              if int(str(subprocess.check_output("pactl get-sink-volume "+Top_right_card+" | grep % | cut -d '/' -f2 | cut -d'%' -f1", shell=True).rstrip()).replace("b' ","").replace("'","")) > max_volume_top_right :
#                                print("Volume exceeded the maximum value : "+str(max_volume_top_right)+"% !")
#                                subprocess.run("pactl set-sink-volume "+Top_right_card+" "+str(max_volume_top_right)+"% & pid=$!",shell=True)

In this case the volume will exceed max_volume (150%)
Can you first try to modify the line 661 to :
                              if int(str(subprocess.check_output("pactl get-sink-volume "+Top_right_card+" | grep % | cut -d '/' -f2 | cut -d'%' -f1", shell=True).rstrip()).replace("b' ","").replace("'","").replace("b","")) > max_volume_top_right :
and tell me if it's ok ?
Thanks

Thanks for your answer.
Replacing line 661 did not help. Error - invalid literal for int() with base 10: ''
Comments lines it's like your first version - works step +&- 5%, but without limit max level 150%.
I think its problem my RosaOS. Linux Mint and Alt OS(also kde) works with your app very well.
So its the better then without one. My notebook isn't so much loudness.

Thank you for your answer.
Could you modify line 661 to :
if int(subprocess.check_output(f"pactl get-sink-volume {Top_right_card} | grep % | cut -d '/' -f2 | cut -d'%' -f1", shell=True, text=True).strip().replace('"','')) > max_volume_top_right:
And tell me if it's ok ?
If not, it could be a problem with python or your system has not been updated for a long time

Again  crash - invalid literal for int() with base 10: ''
Well, thats all right without line 661. Its problem RosaOS. Don't worry.
Thanks for your attention and efforts to resolve it.
Anyway, thanks for your app! Very usefull for me.

Ok.
I will give you a last try for modifying line 661 :
if int(str(subprocess.check_output(f"pactl get-sink-volume {Top_right_card} | grep % | cut -d '/' -f2 | cut -d'%' -f1", shell=True, text=True).strip().replace("'","").replace('"',''))) > max_volume_top_right: