CAS2-416: Allow Licence CA's to see prison dashboard

Context
CAS2 currently is setup for users with the POM
role to be able to create/read applications. We
have since found that the ROLE_POM is given out to
a broad range of users who may not be POMs - The
LICENCE_CA role is more in line with who our
users are, and is also used by HDC services to
perform similar actions that we require. We are
therefore going to add in the LICENCE_CA role and
remove the POM role permissions in stages.
This is the first stage - to allow them to view submitted applications for their prisons.
Changes in this PR
Having done the work on the API to allow read
access to LICENCE_CA this PR allows the users to see the prison dashboard tile.
A User with the LICENCE_CA role has been created for AP Tools locally, and I've setup one for the dev/test environment under our 1Password vault as 'DEV: CAS2 Licence CA user'
Screenshots of UI changes
Before
After
Release checklist
As part of our continuous deployment strategy we must ensure that this work is
ready to be released at any point. Before merging to main we must first
confirm:
Pre merge checklist

[ ] Are any changes required to the e2e tests?
[ ] If you've added a new route, have you added a new
auditEvent? (see server/routes/apply.ts for examples)
[ ] Are there environment variables or other infrastructure configuration which needs to be included in this release?
[ ] Are there any data migrations required. Automatic or manual?
[ ] Does this rely on changes being deployed to the CAS API?

Post merge checklist
Once we've merged it will be auto-deployed to the dev environment.
Find the build-and-deploy job in CircleCI.
Should a release fail at any step, you as the author should now lead the work to
fix it as soon as possible.

@patrickjfl I think the incremental approach gives us some value now - the LCA's can then see prison dashboard ASAP. It's going to take me at least a day or two to give them write access as well. It's also more work to also remove the POM role (and that could potentially be a big change so we might want to get a bit more data on our users and be prepared to roll back when we remove that).

@patrickjfl I think the incremental approach gives us some value now - the LCA's can then see prison dashboard ASAP. It's going to take me at least a day or two to give them write access as well (because it needs changes to API). It's also more work to also remove the POM role (and that could potentially be a big change so we might want to get a bit more data on our users and be prepared to roll back when we remove that).

Ah ok - I was under the impression that everyone with role LCA also has role POM, in which case they'd all be able to see the prison dashboard anyway, but is that not the case?

Ah ok - I was under the impression that everyone with role LCA also has role POM, in which case they'd all be able to see the prison dashboard anyway, but is that not the case?

I don't think there's any actual hard data for this! It's possible an LCA might not have role POM, according to an email and technically.
Having said that the API work has been a lot less than I expected to might just do all the UI in one go after all :)

Having said that the API work has been a lot less than I expected to might just do all the UI in one go after all :)
actually the API has gotta go through review etc. so might as well merge this as it's ready

Having said that the API work has been a lot less than I expected to might just do all the UI in one go after all :)

actually the API has gotta go through review etc. so might as well merge this as it's ready

👍


Ability to use attributes other than name for building the json

I'm developing a flask app with WTForms where the name attribute on fields is set by the python variable name used to store the field on creation of the form.  For example, the python code
field1 = DecimalFeild()
creates
<input name="field1" type="number">
This effectively makes the name attribute reserved but I can pass any other attribute like:
feild1 = DecimalFeild(render_kw={'json_name': 'object[fieild1]'})
which creates
<input name="field1" json_name="object[fieild1]" type="number">
I'd love to be able to use this code you've written but with custom attributes

The current code builds on top of jQuery serializeArray, which only works with name attributes, using the standard W3C rules for successful controls to determine which elements it should include.
If we wanted to change that, we would have to stop depending on serializeArray, and write our own code that is equivalent, and also allows for more customization. When the plugin was first written, I was opposed to doing that in order to keep the plugin simple and predictable, but giving its success, I would be open to such rewrite; thankfully the plugin has extensive tests to verify that it works as expected. But I don't think I will be able to embark in this task anytime soon.

Thanks for the reply.  It's not pretty but WTForms can be made to custom name attributes so that's what I have done and it is working with your code.  Thank you for writing this library.

This might be revisited with an option to supply attr to use instead of name in light of v3.2.0 taking over serializeArray from jQuery.


TLSv1.3: Use the new TLSv1.3 certificate_required alert where appropriate

TLSv1.3 introduces a new alert "certificate_required". We should use it.

Pushed. Thanks.


Response of keyvault set secret includes secret value by default

Describe the bug
The default behavior of az keyvault secret set will return the secret value in plain text. It is dangerous. In our scenario, we have a script to generate secret then save into kv. We think the secret only exists in memroy and kv, but actually is printed and logged.
Related command
az keyvault secret set --subscription xxxx --vault-name xxx--name test-secret --value '123'
Errors
{
"attributes": {
"created": "2023-09-15T18:41:41+00:00",
"enabled": true,
"expires": null,
"notBefore": null,
"recoveryLevel": "Recoverable+Purgeable",
"updated": "2023-09-15T18:41:41+00:00"
},
"contentType": null,
"id": "xxxxx",
"kid": null,
"managed": null,
"name": "test-secret",
"tags": {
"file-encoding": "utf-8"
},
"value": "123"
}
Issue script & Debug output
No error in script/cmd
Expected behavior
can we make --output none as default
Environment Summary
azure-cli: 2.41.0
Additional context
No response

Thank you for opening this issue, we will look into it.


Added Greek translation

See #50.

Thank you @imiordanov for the Greek translation :+1: Happy Hacktoberfest

Happy to help in any way I can! Happy Hacktoberfest! :)


Fix dependent packages loading when cross-compiling

When configuring xmake for android from Linux with a binary package (plat=linux), its dependencies will by fetched for android.`
add_requires("nzsl~host, {binary = true})
add_requires("nzsl")

(nzsl depends on nazarautils)
I noticed this because xmake wanted me to reinstall the same binary package over and over again because its dependency hash didn't match.
I added a print here:
-- load all required packages
function _load_packages(requires, opt)

    -- no requires?
    if not requires or #requires == 0 then
        return {}
    end

    -- load packages
    local packages = {}
    local packages_nodeps = {}
    for _, requireitem in ipairs(load_requires(requires, opt.requires_extra, opt)) do

        -- load package
        local requireinfo = requireitem.info
        local requirepath = opt.requirepath and (opt.requirepath .. "." .. requireitem.name) or requireitem.name
        local package     = _load_package(requireitem.name, requireinfo, table.join(opt, {requirepath = requirepath}))
        print(package:name(), package:plat())

prints
nzsl linux
nazarautils android

...

nzsl android
nazarautils android

this PR fixes the issue by setting the plat/arch values of requires_extra for dependencies, I'm not sure it's the best fix but it works:
nzsl linux
nazarautils linux

...

nzsl android
nazarautils android

no {binary = true}, it should be {host = true}
it should has warnings tips. like this
warning: add_requires("xxx") has unknown option: {binary=true}!

sorry I put the wrong code from my memory, here's the real code I used:
if is_plat("android", "iphoneos", "wasm") then
	-- compile shaders using host-binaries
	if has_config("compile_shaders") then
		add_requires("nzsl~host", { kind = "binary" })
	end
end

it's a binary package, which is the same as host = true if I understand correctly.
Plus you can see that package:plat() returns "linux" and it gets "android" dependencies

it's a binary package, which is the same as host = true if I understand correctly.

no, it's different, you should use {host = true}
add_requires("nzsl~host", { host = true })

kind = "binary" simply tells xmake to install the package as a binary, which is not a static or dynamic library, but it may also be a cross-compiled binary (even if perhaps it cannot be run on the host).
And host = true tells xmake to compile the package only as a host platform program, which may be a binary or a library, but it, and all its dependencies, will be forced to compile and run as a host program.
Translated with www.DeepL.com/Translator (free version)

I tested with add_requires("nzsl~host", { kind = "binary", host = true }) and it works as intended, however there still seem to be an issue as kind = "binary" made it so nzsl~host :plat() returns "linux" instead of "android", but this didn't affect dependent packages.
This is an inconstancy between the two behaviors.

It works for me.
add_requires("libpng", {system = false, kind = "binary", host = true})
target("testdep")
    set_kind("binary")
    add_files("src/*.cpp")
    add_packages("libpng")

ruki-2:testdep ruki$ xmake f -p android --ndk=~/files/android-ndk-r20b/ -c
checking for architecture ... armeabi-v7a
checking for Android SDK directory ... ~/Library/Android/sdk
checking for Build Tools Version of Android SDK ... 33.0.0
checking for NDK directory ... /Users/ruki/files/android-ndk-r20b
checking for SDK version of NDK ... 21
note: install or modify (m) these packages (pass -y to skip confirm)?
in xmake-repo:
  -> libpng v1.6.37 [macosx, x86_64, binary]. <---------------- macos, not android
please input: y (y/n/m)

It works for me with both host = true and binary = true.
But if I use binary=true only, I still get a Linux binary (and not Android binary), with :kind() being "linux", however its dependency are loaded as android.

It works for me with both host = true and binary = true.
But if I use binary=true only, I still get a Linux binary (and not Android binary), with :kind() being "linux", however its dependency are loaded as android.

Yes, I just made a bit of a mistake because the xmake-repo repository binary is mostly running on the host, so I forced it to gate as the host platform as well.
But setting kind = "binary" currently does not pass to its dependency packages. But host = true will make all dependencies set to the host platform as well.

Yes, this may be a bit hacky, and other types such as host binary may be extended later to improve it. But at the moment I haven't encountered a situation where I need to build a target host package, because usually it can't be run and used directly. So I didn't deal with it for now.


Wasm with Cairo 1

Wasm with Cairo 1
Closes #1827
Description
This PR adds the ability to run cairo 1 programs with wasm. The crate cairo1-run has been refactored in order for it to stop depending on the rust native standard library. There also has been added a new feature cli which allows cairo1-run to be used as a binary. This addition was made to avoid using the std library unless this specific case, where it needs to read from files and the std library must be used.
Checklist

[X] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[X] This change requires new documentation.

[X] Documentation has been added/updated.
[X] CHANGELOG has been updated.

Done!

Everything done


tabs bug

on pc,the tabs stay  below,however when browsed on android 4.4 chrome brower,it run on the top,why?i m very confused

Greetings @Jaware!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

This is part of our platform specific configurations. On iOS, tabs will appear at the bottom and on Android, tabs will appear below the nav-bar.
http://ionicframework.com/docs/nightly/api/provider/$ionicConfigProvider/
This can be configured though.
Added to the docs via https://github.com/driftyco/ionic/commit/a7b894447912c03274625942cab4c4bdf345a7e0

ok，but can the line behind the title be removed? the android chrome show the line(pc show too),but the android system browser not(this is what i want)


Duplicate of items in collection?

Seems we have duplicates in the collection?
https://beta.kodadot.xyz/rmrk/collection/C6E2CA836BD86CBD05-PLACE?page=1

👋

👋

Hey, there is already a PR for this issue  👀

👋

Hey, there is already a PR for this issue 👀

oh there is? should we close this one then?

lol yeah I seemed to have missed the message my bad

👋

Hey, there is already a PR for this issue 👀

oh there is? should we close this one then?

No, there's PR for this particular issue 😄 you're even assigned to review it ! 🚀

fix: Duplicate of items in collection #3483


server hangs when starting, showing connection refused errors

Describe the bug
The server hangs when starting, showing "connection refused" errors when attempting to connect to the local service on port 3986.
To Reproduce
Steps to reproduce the behavior:

Run the command daytona server start
Observe the terminal output

Expected behavior
The Daytona server should start without errors and be accessible on port 3986.
Screenshots

Desktop (please complete the following information):

OS: macOS
Daytona Version: 0.34.0

Looks like the same problem with 1071

@nkkko here is what i observed, since The waitForApiServerToStart()  checks every  30 seconds for the Api server to start. It might be that the API server takes longer to start, so i increased it to 60 seconds, it stopped displaying the connection refused error,  and the server started.
The code

@Tpuljak should decide on this. Btw. @Tpuljak is the solution a faster api server thou? :)

@Philip-21 the error does not manifest there. It was a problem with our reverse proxy.
@nkkko I think the issue might've been resolved with https://github.com/daytonaio/daytona/pull/1196. Does this error still happen on the latest version of Daytona?

Alright

This Issue has been automatically marked as "stale" because it has not had recent activity (for 14 days). It will be closed if no further activity occurs. Thanks for the feedback.

Due to the lack of activity in the last 7 days since it was marked as "stale", we are closing this Issue. Do not hesitate to reopen it later if necessary.


fix for #1405: ERROR:Invalid inlined_image_type in cached_result

I have signed CLA as a corporation. Company name - GDB Technologies Pvt. Ltd.

LGTM, thanks!
@jmarantz are you able to acknowledge we are ok CLA-wise?

Yes I think it's good.
On Mon, Apr 17, 2017 at 7:00 AM, Otto van der Schaaf <
<EMAIL_ADDRESS>wrote:

LGTM, thanks!
@jmarantz https://github.com/jmarantz are you able to acknowledge we
are ok CLA-wise?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pagespeed/mod_pagespeed/pull/1529#issuecomment-294452869,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB2kPYNcLawy5gQF0D_AydcHe5vD5J2Uks5rw0ZXgaJpZM4M8ytw
.


ipv6: bad workload entry name prevents VM auto registration

error   wle     auto-registration WorkloadEntry of vm-v1-84f64867f-9r9g7.echo failed: error creating WorkloadEntry: WorkloadEntry.networking.istio.io "vm-fd00:10:244::14" is invalid: metadata.name: Invalid value: "vm-fd00:10:244::14": a lowercase RFC 1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')

Hi @howardjohn, I re-assigned to you because I noticed that you have submitted a PR(https://github.com/istio/istio/pull/35847) for this. Sorry for the mis-take. ^_^


Capture Traces

As discussed in #959. Here is how it works (from the user perspective).
scala> import io.finch._, io.finch.syntax._
import io.finch._
import io.finch.syntax._

scala> val foo = get("foo" :: path[String]) { s: String => Ok(s) }
foo: io.finch.Endpoint[String] = GET /foo :: :string

scala> import com.twitter.finagle.http.Request
import com.twitter.finagle.http.Request

scala> val s = foo.toServiceAs[Text.Plain]
s: com.twitter.finagle.Service[com.twitter.finagle.http.Request,com.twitter.finagle.http.Response] = io.finch.ToService$$anon$4$$anon$2

scala> Trace.capture { s(Request("/foo/bar")).map(_ => Trace.captured) }
res0: com.twitter.util.Future[io.finch.Trace] = Promise@406161512(state=Done(Return(/foo/:string)))

Codecov Report

Merging #960 into master will decrease coverage by 0.44%.
The diff coverage is 44.44%.

@@            Coverage Diff            @@
##           master    #960      +/-   ##
=========================================
- Coverage   83.25%   82.8%   -0.45%
=========================================
  Files          50      50
  Lines         818     826       +8
  Branches       48      44       -4
=========================================
+ Hits          681     684       +3
- Misses        137     142       +5

Impacted Files
Coverage Δ

core/src/main/scala/io/finch/Trace.scala
70.58% <20%> (-21.08%)
:arrow_down:

core/src/main/scala/io/finch/ToService.scala
96.55% <75%> (-3.45%)
:arrow_down:

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 8d43990...b645508. Read the comment docs.

👍
Let's merge it and release. We have instant cases that will be solved using it

LGTM. I'm 👍 to getting this into the next release too.

@sergeykolbasov I'm debating myself on whether there is a need for a bootstrap option. Having both (bootstrap option and Trace.capture) seems to be redundant for me. I will run some benchmarks to see how expensive that local lookup (based on what we've seen in FInagle it could be quite expensive).

Nothing stands out of the benchmark runs (just a drop in the ocean). Also, it's pretty volatile so somehow the new code is faster than old. Again, I think it's just noise, nothing to be concerned about.
BEFORE:

[info] Benchmark                                                   Mode  Cnt      Score      Error   Units
[info] TextBootstrapBenchmark.foos                                thrpt   10  68824.182 ± 2454.004   ops/s
[info] TextBootstrapBenchmark.foos:·gc.alloc.rate.norm            thrpt   10  47888.021 ±    0.008    B/op

AFTER:

[info] Benchmark                                                   Mode  Cnt      Score      Error   Units
[info] TextBootstrapBenchmark.foos                                thrpt   10  70165.207 ± 2442.619   ops/s
[info] TextBootstrapBenchmark.foos:·gc.alloc.rate.norm            thrpt   10  47904.100 ±    0.308    B/op

@vkostyukov ok, that's nice. Have no questions


Make all "static" pages configurable in .env to show

The current coding coach agora setup of having about, 3π+, community pages should be configurable so that any page can be "disabled".  The product page is the biggest offender as some sites may not have a use for this.  It is also possible that they might not need an about page either.

Completed.


For step-based workflows in GUI use a st.empty()

The current implementation of steps is rather clumsy.
Essentially, where ever st.experimental_rerun() is called, that is, because we want to get rid of the previous step. But that way, we need to keep track of steps by using the steps in the state handler, and need to persist search results and stuff there too.
With a Streamlit's empty container, we could solve this in a much simpler way, as it allows to get rid of previous stuff by simply clearing the container. Then no re-run is required any more, thus no steps and intermediate data is needed either. So we should be able to get rid of this, if not the entire StateHandler as a whole.

This is proving more difficult to accomplish. Nested buttons, i.e. buttons that are nested in multiple instances of with st.something, will by default trigger a rerun. This cannot be prevented and is the very session states were implemented in the first place. Sort of forgot about that myself. So we will still need the state handler. However, we can greatly reduce complexity by using st.empty(). Not sure about getting rid of all of the st.experimental_rerun() instances BUT: What we can do is get rid of the meta functions, that select the search or result function and instead combine those in the search function, which will check, if there are results and then empty the container and fill it with the results container. Alternatively, the container could live at the level of the meta function and be filled with either content.

ah... how I hate this fiddly state stuff in streamlit! xD
but reducing the complexity of the search stuff will be an improvement in and by itself. Feel free to implement it the way that seems best to you. (Given you find time for it, that is, of course 😄 )


CES-11250 part II

added patches to fix an issue with nova scheduler preventing instances creation (bz 1846020)

Looks good.  Tested on Mobile Rack.


Improve error messages for dependency groups

Description
Given a paket.dependencies with a default Main group
And some other groups
When the user tries to update/remove a package ID not in the Main group
And he forgets to specify the group
Then the error message does only tell him that the package ID does not exist in group Main
Repro steps
Using a paket.dependencies with packages in default group and at least 1 dependency group named InternalTools with 1 package called A. The package A is only used in the dependency group, and not in default one

Call paket update nuget A

The error message only tells that package A does not exist in group Main

Expected behavior
PS C:\svn\ProjectSecret\trunk> ..paket\paket.exe update nuget A
Paket version <IP_ADDRESS>
Paket failed with:
      Package A was not found in paket.dependencies in group Main.
      However, Package A was found in groups:
        - InternalTools
      You can use the command "group" to specify the dependency group of your package
Actual behavior
PS C:\svn\ProjectSecret\trunk> ..paket\paket.exe update nuget A
Paket version <IP_ADDRESS>
Paket failed with:
      Package A was not found in paket.dependencies in group Main.
Known workarounds
Today, the developer needs to find the right group main himself.
He has to open the paket.dependencies file, search for his package ID and get the group name

:+1:


Add hash-join execution for left-outer-joins

Summary of the changes / Why this improves CrateDB
This implements hash-join execution for left-outer-joins according to https://github.com/crate/crate/issues/15147 by the following steps:

Extend HashInnerJoinBatchIterator to HashJoinBatchIterator which an additional option to emit non-matching join pairs with null values to support left-outer-join behaviour
Extend the planning pipeline to support Left Join-Type for Hash-Joins

Performance Improvements:
This is a comparison between the hash-join and nested-loop join execution on a table having 50 % matches and 50 % non-matches with different sizes running each time 100 iterations:
select count(*) from t1 left join t2 on t1.id = t2.id

Table sizes
Nestedloop (ms mean)
Hash-Join (ms mean)

10 / 10
2.045 ± 1.229
0.555 ± 0.110

100 / 100
4.455 ± 0.271
0.438 ± 0.028

1000 / 1000
169.494 ± 0.286
1.058 ± 0.018

10.000 / 10.000
21919.926 ± 653.076
7.955 ± 0.050

100.000 / 100.000
n/a
178.404 ± 4.188

@mfussenegger Could you give me some early feedback on this draft?

retest this please

New benchmarks are on its way.

There seems to be no overhead on the inner-hash-join according to the jmh bench:
this branch:
Result "io.crate.data.join.RowsBatchIteratorBenchmark.measureConsumeHashInnerJoin":
  2.646 ±(99.9%) 0.011 ms/op [Average]
  (min, avg, max) = (2.628, 2.646, 2.683), stdev = 0.014
  CI (99.9%): [2.636, 2.657] (assumes normal distribution)

Result "io.crate.data.join.RowsBatchIteratorBenchmark.measureConsumeHashInnerJoinWithHashCollisions":
  0.323 ±(99.9%) 0.003 ms/op [Average]
  (min, avg, max) = (0.318, 0.323, 0.331), stdev = 0.004
  CI (99.9%): [0.320, 0.326] (assumes normal distribution)

Benchmark                                                                 Mode  Cnt  Score   Error  Units
RowsBatchIteratorBenchmark.measureConsumeHashInnerJoin                    avgt   25  2.646 ± 0.011  ms/op
RowsBatchIteratorBenchmark.measureConsumeHashInnerJoinWithHashCollisions  avgt   25  0.323 ± 0.003  ms/op

master:
Result "io.crate.data.join.RowsBatchIteratorBenchmark.measureConsumeHashInnerJoin":
  2.679 ±(99.9%) 0.019 ms/op [Average]
  (min, avg, max) = (2.645, 2.679, 2.716), stdev = 0.025
  CI (99.9%): [2.660, 2.698] (assumes normal distribution)

Result "io.crate.data.join.RowsBatchIteratorBenchmark.measureConsumeHashInnerJoinWithHashCollisions":
  0.302 ±(99.9%) 0.002 ms/op [Average]
  (min, avg, max) = (0.297, 0.302, 0.311), stdev = 0.003
  CI (99.9%): [0.299, 0.304] (assumes normal distribution)

Benchmark                                                                 Mode  Cnt  Score   Error  Units
RowsBatchIteratorBenchmark.measureConsumeHashInnerJoin                    avgt   25  2.679 ± 0.019  ms/op
RowsBatchIteratorBenchmark.measureConsumeHashInnerJoinWithHashCollisions  avgt   25  0.302 ± 0.002  ms/op

@matriv Can you do another pass on this, please?

Please remove file .pom.xml.swp.

There seems to be no overhead on the inner-hash-join according to the jmh bench:
great, thank you!

I'd say to add a JMH benchmark as well.
Maybe you can do that already, but can also be done as a followup, as you like

I think it'll make sense to run end-to-end benchmarks for different sizes of tables to also see if for small sizes, NL could be faster than HashJoin.
Do you have any metrics for this? I think it would be nice to have an indication before merging the PR

Otherwise it look great, nice stuff!
Please give me just a bit more time, to take a closer look in the iterator (debug to understand it thoroughly), and I think we're good to go!

Do you have any metrics for this? I think it would be nice to have an indication before merging the PR

It's in the description of this pr.

(apologies, approved my accident)


Old Fuskator Test Link Fix

Would not work, even if human was looking at it.
This is probably why all travis.cl builds have not been compiling.
Updated URL to more popular album. Hopefully it will not get deleted in the future.
Category
This change is exactly one of the following (please change [ ] to [x]) to indicate which:

[X] a bug fix (Fix #362)
[ ] a new Ripper
[ ] a refactoring
[ ] a style change/fix

Description
New Fuskator link for test suite
Testing
Compile and ran on new test URL.
Nothing else should be affected.
Required verification:

[X] I've verified that there are no regressions in mvn test (there are no new failures or errors).
[X] I've verified that this change works as intended.

[X] Downloads all relevant content.
[ ] Downloads content from multiple pages (as necessary or appropriate).
[ ] Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.

[X] I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

[ ] I've added a unit test to cover my change.

Changes Unknown when pulling ed9b4c1ac351dc51ecaecd6cce5d67014cf97e6c on kevin51jiang:UpdatedFuskatorTestURL into ** on RipMeApp:master**.

LGTM


swag init could not find struct from imported package

Describe the bug
I'm trying to provide a struct as request body to @Param request body, but the issue is that the struct is imported from another package and not present in current project. So for instance:
import (
models "git.mySite.net/core/sample-models"
)

And in my function I use:
var profile models.UserProfile
Before my function I use swagger annotation:
// @Param request body models.UserProfile true "query params"
But it does not work. I have tried many different formats but with no luck.
To Reproduce
Steps to reproduce the behavior:

Define a Go package with some struct
Import your struct in your function and use it also as @Param
issue the command: swag init

You will get the below error:
2022/08/20 16:53:34 Generate swagger docs....
2022/08/20 16:53:34 Generate general API Info, search dir:./
2022/08/20 16:53:35 ParseComment error in file /Users/Username/GolandProjects/myProject/controller/sample_controller.go :cannot find type definition: models.UserProfile

Expected behavior
Successful document generation with no error.
Your swag version
1.8.4
Your go version
1.18.4
Desktop (please complete the following information):

OS: Mac OS X 12.1
Browser: Opera
Version: 89.0

Hello!
Try using the actual package name, not an alias:
// @Param request body sample-models.UserProfile true "query params"

I think, swag doesn't support aliases for packages.
Duplicate of https://github.com/swaggo/swag/issues/1291

if git.mySite.net/core/sample-models is not inside your project directory, you have to import it into your main package.
//main.go

import(
    _  "git.mySite.net/core/sample-models"
)

then
swag init  --parseDependency ……

 swag init  --pd
2022/09/16 18:13:56 Generate swagger docs....
2022/09/16 18:13:56 Generate general API Info, search dir:./
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x10 pc=0x141c928]

goroutine 1 [running]:
github.com/swaggo/swag.(*PackagesDefinitions).parseFunctionScopedTypesFromFile(0xc00000ea50, 0xc000bd2c80, {0xc0003b7c20, 0x18}, 0xc0023ab6e0)
        /Users/yangkun/dev/pkg/mod/github.com/swaggo/swag@v1.8.5/packages.go:168 +0xa8
github.com/swaggo/swag.(*PackagesDefinitions).ParseTypes(0xc00000ea50)
        /Users/yangkun/dev/pkg/mod/github.com/swaggo/swag@v1.8.5/packages.go:110 +0xc9
github.com/swaggo/swag.(*Parser).ParseAPIMultiSearchDir(0xc0001ce2a0, {0xc0001fa060, 0x1, 0x0}, {0x1512a84, 0x7}, 0x64)
        /Users/yangkun/dev/pkg/mod/github.com/swaggo/swag@v1.8.5/parser.go:362 +0x3da
github.com/swaggo/swag/gen.(*Gen).Build(0xc0001c7980, 0xc0001cd6c0)
        /Users/yangkun/dev/pkg/mod/github.com/swaggo/swag@v1.8.5/gen/gen.go:177 +0x5af
main.initAction(0xc00019b8c0)
        /Users/yangkun/dev/pkg/mod/github.com/swaggo/swag@v1.8.5/cmd/swag/main.go:151 +0x6f7
github.com/urfave/cli/v2.(*Command).Run(0xc0001997a0, 0xc0001dc400)
        /Users/yangkun/dev/pkg/mod/github.com/urfave/cli/v2@v2.3.0/command.go:163 +0x64a
github.com/urfave/cli/v2.(*App).RunContext(0xc0001d4340, {0x15be130, 0xc00002e090}, {0xc0000201b0, 0x3, 0x3})
        /Users/yangkun/dev/pkg/mod/github.com/urfave/cli/v2@v2.3.0/app.go:313 +0x81e
github.com/urfave/cli/v2.(*App).Run(...)
        /Users/yangkun/dev/pkg/mod/github.com/urfave/cli/v2@v2.3.0/app.go:224
main.main()
        /Users/yangkun/dev/pkg/mod/github.com/swaggo/swag@v1.8.5/cmd/swag/main.go:221 +0x55d

Did anyone find a solution to this?


Password required Validation error on user update

Hi,
I am using a standard meanjs base for my app, when I try to update an user, it gives me a save failed error :
Saving error is :
  {
    [ValidationError: Validation failed]
    message: 'Validation failed',
    name: 'ValidationError',
    errors:
     { password:
       { [ValidatorError: Path `password` is required.]
           message: 'Path `password` is required.',
           name: 'ValidatorError',
           path: 'password',
           type: 'required',
           value: undefined } } }

I have tried both, retrieving the password for that particular user and then trying to save -> In this case the password is rehashed, making the old password obsolete and making the user incapable of logging in again.
Second, deleting password field using :
delete user._doc.password;

before saving, in which case I get the same error. What could be the reason for this? I would really appreciate some help with this. The ONLY solution is for me to reload (window.location.reload) the edit profile page, after which everything seems to work just fine. But this is now a problem as I am trying to enable html5mode, which breaks on reload. More than a direct solution, I would love to know why this is happening? Why does a reload fix this? What is changing in the user object after a reload, only on this page, when I can access all the user fields on the display profile page already with ease, without reloading.

Hi @bluekrill,
The changed password issue got resolved on branch 0.4.0 which will be soon released.
If you need to continue supporting the current version of MEAN.JS which you used then you can easily backport the fix (it's very focused) to your version and apply it.

Thanks @lirantal, I would like to backport the fix to my version. Just to be sure, I would have to perform a git cherry-pick master-0.4.0 to get this done right? If not, could you please tell me the right way to go about it, would really appreciate it.

Or you could just take a look at the actual change in this PR https://github.com/meanjs/mean/pull/642 and copy&paste it to your code.

@lirantal Thanks, but I have already tried the isModified solution, in both cases -> without retrieving the password, in which case it still gives the Password validation error, and retrieving the password, and then trying to save, in which case it does not detect that the password is not modified and proceeds to rehash it.
I copied you code and tested it out again. No luck. Why would it not detect that the password is not modified?

I assume you are working on the latest 'master' code? if so can you give us step by step instruction how to reproduce the problem you are facing?

@lirantal Hey, I am not using the latest master code. Will update and get back to you with the result. Hopefully that will fix it :). Thanks for the help.

@lirantal I found the problem, apparently I had set the password field in my user model as required. This was causing any updates to check for the password and hence the error. Though, the error would disappear after a page reload was forced, so I never thought this could be the problem. It all works fine now, so I guess the issue can be closed. Still can't understand what the page reload did to cause the Password required error to disappear. Thanks for your help.

@lirantal I have now the updated master version, and now the earlier problem of a Password validation error no longer exists. The user profile is successfully updated, but unfortunately now it either deletes the password field if it does not exists, or rewrites it, if it does.
I am using the exact same server function to update as the master code, and retrieving the user from the exact same authentication service as the master code. No changes here except the additions to the User model to suit my app. Everything is updating correctly, unfortunately including the password which was removed by Passport at the time of retrieval. I still have the isModified function in place in my pre save method.
The password retrieval error was due to the fact that I had set my password field as required. Once I changed that, it is now overwriting my password, or deleting the field entirely, depending on how I send it across.

@bluekrill 2 things:

can you submit a PR for master branch which fixes this problem?
can I ask you to try out the 0.4.0 branch of meanjs where this issue has been resolved?

@lirantal Sorry for the late response. I can definitely do those things. Thanks.

Great, thanks.


jQuery/Ready event parameter injection issue

Note: for support questions, please use one of these channels: https://github.com/angular/angular.js/blob/master/CONTRIBUTING.md#question. This repository's issues are reserved for feature requests and bug reports.
Do you want to request a feature or report a bug?
BUG
What is the current behavior?
When jQuery is loaded before Angular then the ready event handler function can receive an unexpected parameter, which will be set to jQuery itself.
In the case below, MyMethod() supports an optional parameter, and the parameter being passed in was jQuery:
angular.element(document).ready(MyHelper.MyMethod);
In my own example, I accidentally booted angular and passed in jQuery as the app element, which resulted in multiple instances of jQuery being instantiated until memory was exhausted.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via https://plnkr.co or similar (template: http://plnkr.co/edit/tpl:yBpEi4).
http://plnkr.co/edit/BVxsUaluLp6A7C8YTBHS
What is the expected behavior?
I would have expected that no parameters are passed into MyMethod().
What is the motivation / use case for changing the behavior?
jQuery consistency. Consistency in general.
Which versions of AngularJS, and which browser / OS are affected by this issue? Did this work in previous versions of AngularJS? Please also test with the latest stable and snapshot (https://code.angularjs.org/snapshot/) versions.
Not browser related.
Other information (e.g. stacktraces, related issues, suggestions how to fix)

@mgol is this something you could take a look at?

Thanks for the report. However, current behavior is consistent with jQuery which does pass the jQuery instance into the invoked callback as the first parameter. In jQuery passing jQuery itself to itself is not supported and it does trigger an infinite loop so there's no difference here.
Where exactly do you see the inconsistency?

I'll agree on that. I think there was confusion on my part when using angular's ready() event handler without jquery loaded and no parameters are passed.


vue3-simple-icons components import seems to break HMR

It seem they break the Hot Module replacement feature.
I tried to register some globally, but happens also when using them individually.

I have not tried the hot module replacement and really don't understand it much
Is this the feature you are talking about
https://vitejs.dev/guide/api-hmr.html

Yup.
I don't know how it works either,
but it allows the dev server to reload components once they have been modified.
This feature seems broken when I use vue3-simple-icons in my project,
maybe someone will be able to tell us how to let it work.
For now I disabled icon imports when import.meta.env.DEV is true.
Here is my generic IconBrand component that loads vue3-simple-icons asynchronously.
When dev server is enabled, it loads a Quasar icon instead.
Usage: <IconBrand name="si-discord" />
<template>
  <Icon v-if="!devMode"/>
  <q-icon v-else name="mdi-cube"/>
</template>

Are you using Vite?

@wyatt-herkamp quasar uses webpack, not vite.

@dxlliv are you using quasar cli or the vue-cli plugin?

Using the default configuration with a fresh quasar project, yarn > webpack > vue 3.0.0. I am seeing hot module reload working with adding/swapping out icons and cannot reproduce the issue. My IndexPage.vue:
<template>
  <q-page class="flex flex-center">

      Hi my name is Gerald.
      <ApachemavenIcon style="fill: black" />
      <NpmIcon style="fill: purple" size="24px" />
      <ZigIcon style="fill: black" />

  </q-page>
</template>

I was unable to reproduce the issue on a new project too.
I'll try to investigate a little more, but the problem is apparently not related to vue3-simple-icons.
PS: I'm using Quasar with Vite 3.2.1 and Vue 3.2.41

Cool, it does not occur with Vite 4.0.3!
I'm going to close the issue.


Redesign of library to support class inheritance

name: Pull Request
about: Fill out this template before submitting a Pull Request.
title: First cut of library
labels: 'pull request'
assignees: ''

Description

First complete cut of the library that has been tested with Transformers.
Types of changes

Moved code into specialized files and removed some functions that didn't belong in a library (they were moved back to the Transformers that used them)
Checklist:

[x] I have read the documentation on CONTRIBUTING
[x] My updates require a change to the documentation
[x] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[x] All new and existing tests passed

Hints to reviewers:

There is minimal new code here, most of it has been moved from existing Transformers. The biggest changes are related to wrapping some code into classes and moving code out to other files (within the library).
An outstanding issue at this time is being able to publish the library automatically to PyPi when merging into the main branch

The library is now able to be automatically published to pypi upon merge to master

The library is now able to be automatically published to pypi...
It's building and pushing the library to PyPi on push to the main GitHub branch.


DSND-2508: Fix missing capital date bug

Describe the changes
This PR implements the fix for the capital date bug where the date field was not being added to the ObjectNode and therefore not sent in the PUT request body.
Related Jira tickets
DSND-2508
Developer check list
General

[ ] Is the PR as small as it can be?
[ ] Has the Jira ticket been updated with any relevant comments?
[ ] Is the README up to date?
[ ] Has the POM been updated for the latest versions of dependencies?
[ ] Has the code been double-checked against main?
[ ] Does the code adhere standards in this repository, including SonarQube checks?

Testing

[ ] Do the code changes have unit and integration tests with code coverage > 80%?
[ ] Has the code been tested locally and/or passed the API Karate tests on Tilt?
[ ] Have mandatory manual test cases been run?
[ ] Are extra Karate tests required?
[ ] Are all the test data resources up to date?

Release preparation
Where possible, add links to the respective Jira tickets when an item is checked

[ ] Have these changes been included in a release ticket?
[ ] Are changes required to the docker-chs-development deployment or test data?
[ ] Are any changes required to the environment added to chs-configs?

Notes to the tester
Add any testing hints or instructions here.

CI: No security warnings found

CI: No security warnings found


server/shadow: Fix black screen with win10 mstsc

Request full screen update on RDPGFX_CAPS_ADVERTISE_PDU. Win10 client seems to clean the screen after gfx channel opened. If there happens to be no screen update from server, we will get black screen in mstsc client.

@realjiangms Pull looks good so far, could you rebase it to current master?

Yes I'll do it later.

Can we get this one Merged?


malformed docx processing error: media file not in [Content_Types].xml

bad.docx
some malformed docx, for example, /word/media/* files not in [Content_Types].xml
python-docx raise a keyerror exception while open the file and refuse to proceed.

i've create a pull request:
https://github.com/python-openxml/python-docx/pull/1456


ui: allowed master node to be added to the cluster

So far we only allowed workers to be added to the cluster through the
unassigned nodes page.
With this patch, we are now allowing the users to select the roles of the
nodes they want to add to the cluster. Very similar to what is available
from the discovery bootstrap page.
This feature keep forcing the user to respect the contrainst of having an
odd number of master nodes so we have a stable topology.
feature#add_master_node
Signed-off-by: Vítor Avelino<EMAIL_ADDRESS>Screenshots

 Allow master nodes to be added through the UI


New Version: VMware.WorkstationPro version <IP_ADDRESS>00274

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

@stephengillie - here's another one...

/azp run

@wingetbot run

needs a manual merge just like previous versions


Can not get image data via rviz

Hi, I am using kachaka in ros2 humble environment, it seems that I can not get image data in rviz even if I subscribe to the /kachaka/front_camera/image_raw. I check the topic list and topic info, the image message was published and subscribed, but I just can not see it in rviz.

It looks like there is no data in /kachaka/front_camera/image_raw topic. Nothing shows when I run  ros2 topic echo /kachaka/front_camera/image_raw


Move Connect to Studio button into Actions tree

First very small step based on this feedback
Seems like a reasonable step as I try to work out what is next.
Demo
https://user-images.githubusercontent.com/37993418/222293911-5513b9af-17c3-4c8a-9559-a42c73071fa3.mov
Options I am considering for the 3rd button after Studio is connected:

Show settings
Share Experiment Live or checkbox or similar
Edit Studio Connection
No change

After sleeping on it this feels too artificial. There is not enough value available in connecting to Studio yet. Will revert back to running a separate tree for the time being.


Text field validation errors should use standard portal handling

Current errors are shown as red text below the text box.
We should be following the standard pattern used throughout the portal. The right-hand end of the text box should have a error indicator which I can hover over for details.
This will allow us to give longer, more actionable error messages.

I think that's probably a good idea for things that require async operations where we're explicitly provisioning something based on a user action.  But for general errors (like a null-ref), Ibiza doesn't have a way of showing these to the user, other than to log something in the debug console.  I think we could probably do better than that.

@Tuesdaysgreen - Are you talking about the regex validation stuff for input boxes? You seem to be talking about something different.

I'm talking about runtime issues.  Portal has a "notification" system which they use mainly for scenario's where you do an ajax call and want to log the status.  We don't normally use it in the case where you hit a null ref or something in code.  I guess we could go down that path, but we aren't doing that anywhere in the websites extension today.


fix: correctly retag node ca watcher to static during the build process

What type of PR is this?
Due to schema changes, node-ca-watcher image was no longer correctly retagged during build process.

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>3
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 71.271%

Totals

Change from base Build<PHONE_NUMBER>5:
0.0%

Covered Lines:
2590

Relevant Lines:
3634

💛  - Coveralls


How to upload PPT on S3 Without download into local

Hi @gitbrent
i want to upload ppt on s3. how can we achive it.
Thanks in advance.

Hi @AjayRawat232

Node app: use a stream and pass it amazon's API, or save the file locally and FTP/REST post, etc.
Client-side browser: There's no way to redirect the file.  Your browser downloads the presentation.

Hi,
I am unable to download the PPT in Firefox but the same PPT is downloading in Chrome.
I have checked Issue #81 which is not worked for me. Can you give me any other reliable solution for it.
Thanks in Advance

Follow-up:
As of version 2.5.0, client-browsers can save presentations as a blob, which could then be redirected to Amazon S3, OneDrive, DropBox, etc.
API Docs: Saving


There was an error when downloading SuiteCloud SDK dependency from https://system.netsuite.com/download/suitecloud-sdk/23.2/cli-2023.2.0.jar

Select the SuiteCloud tool where you encountered the issue
SuiteCloud CLI
Version of the SuiteCloud Tool
0.0.0
Operating system
macOS vertura 13.5.1
Node Version / VSCode Version
v21.1.0
JAVA Version
<IP_ADDRESS>
Bug description
No matter run command sudo npm install -g @oracle/suitecloud-clior  'sudo npm install -g --acceptSuiteCloudSDKLicense @oracle/suitecloud-cli' , always throws up error:
npm ERR! minTimeout is greater than maxTimeout

npm ERR!  There was an error when downloading SuiteCloud SDK dependency from https://system.netsuite.com/download/suitecloud-sdk/23.2/cli-2023.2.0.jar
npm ERR! Error: GET request timeout.

Steps To Reproduce

open terminal
run 'sudo npm install -g @oracle/suitecloud-cli'

Actual Output
npm ERR! minTimeout is greater than maxTimeout
npm ERR!  There was an error when downloading SuiteCloud SDK dependency from https://system.netsuite.com/download/suitecloud-sdk/23.2/cli-2023.2.0.jar
npm ERR! Error: GET request timeout.
Expected Output
successful result
Anything else?
no

Hi @ppk2017 ,
Is it possible that you are behind a proxy?
If so, you'll need to setup the SUITECLOUD_PROXY environment variable.
Extract from: https://docs.oracle.com/en/cloud/saas/netsuite/ns-online-help/section_155929845760.html#Related-Topics
 Note:

To configure a proxy server, create the SUITECLOUD_PROXY environment variable.
To be valid, the variable value must include an http or https protocol, a hostname, and a port number,
for example: http://proxyurl:proxyport. After creating the variable, restart the console.

I have also noticed that your node version is v21.1.0, and we recommend using 18.14.2 LTS.
Please don't use sudo to install the SuiteCloud CLI as it could lead to other future problems as described on this other issue https://github.com/oracle/netsuite-suitecloud-sdk/issues/662#issuecomment-1686329882

@ppk2017 were you able to try the things mentioned in previous comment?

Closed due to inactivity.
Please open the ticket again if you still experience the same issue


Correction: Possible Capitalization

URI of the page with the mistake
https://usaybia.net/person/1666
What should be corrected on the page?
bi-amr could be capitalized and the letter Hamza could be added. (since it is technically the beginning of a word, and the Hamza is generally left out at this point, it may be not included here as well)
How should it be corrected?
probably: bi-Amr, bi-ʾAmr
Citation
No response
Citation URL
No response
Public Acknowledgement
Yes
Comments
No response

https://usaybia.net/person/2347


feat: Entity::assert_all() (resolves #200)

This adds a function for asserting that all components of an Entity have a given attribute, panicking if any do not. Suggested by @ten3roberts in #200.

Note that I also ran rustfmt on entity.rs. I don't think that it'll be a problem, but it's worth mentioning since that's why git is saying that some of the imports changed.

Also, I just noticed that the doctest doesn't compile :woman_facepalming:.

Also, I just noticed that the doctest doesn't compile woman_facepalming.

Fixed, after probably too much effort...

Looks fantastic.
Yeah, not sure about the imports as things should already be formatted but some mysteries are best left alone.


Audio streaming error

How do I fix this?
 $ echo 'This sentence is spoken first. This sentence is synthesized while the first sentence is spoken.' |   ./piper --model en_US-libritts-high.onnx --output-raw |   aplay -r 22050 -f S16_LE -t raw -

aplay: main:834: audio open error: Operation not supported
[2024-02-03 13:27:34.711] [piper] [info] Loaded voice in 0.325418684 second(s)
[2024-02-03 13:27:34.713] [piper] [info] Initialized piper

One suggestion might be to change the reference to output raw to refer to an actual created file that can be examined independently in order to see if the piper or aplay part is having a problem.

Don't forget -c 1 on the aplay command for mono audio. You may also need to use -D <device> to choose a different audio device.

change the reference to output raw to refer to an actual created file

This works perfectly fine

Don't forget -c 1 on the aplay command for mono audio. You may also need to use -D  to choose a different audio device.

The device names look like this. Which part of the output do I specify as device name?
**** List of CAPTURE Hardware Devices ****
card 1: Generic [HD-Audio Generic], device 0: ALC1220 Analog [ALC1220 Analog]
  Subdevices: 1/1
  Subdevice #0: subdevice #0
card 1: Generic [HD-Audio Generic], device 2: ALC1220 Alt Analog [ALC1220 Alt Analog]
  Subdevices: 1/1
  Subdevice #0: subdevice #0

Use aplay -L (capital L) to get the device names. Look for ones that begin with plughw: since they will automatically convert the audio format for you.
For example, I have something like this:
plughw:CARD=PCH,DEV=0
    HDA Intel PCH, ALC285 Analog
    Hardware device with all software conversions

so I would need aplay -D "plughw:CARD=PCH,DEV=0" ...

It looks like you are running PipeWire. Have you tried the default one?
echo 'This sentence is spoken first. This sentence is synthesized while the first sentence is spoken.' |   ./piper --model en_US-libritts-high.onnx --output-raw |   aplay -D "default" -r 22050 -f S16_LE -t raw -

I'm noticing a similar issue in piper 1.2.0, but with an additional clue to add to the discussion. It seems that piping the output of piper to aplay will fail if there is more than one sentence to speak. It crashes with an illegal seek, presumably in regards to seeking in the output pipe.
echo "What was is the nature of this sentence? Is this sentence number two? Or is it number three?"  |  piper --model en_GB-aru-medium --speaker 4 --debug  |  aplay -r 22050 -f S16_LE -t raw -

Playing raw data 'stdin' : Signed 16 bit Little Endian, Rate 22050 Hz, Mono
DEBUG:__main__:Namespace(model='en_GB-aru-medium', config=None, output_file=None, output_dir=None, output_raw=False, speaker=4, length_scale=None, noise_scale=None, noise_w=None, cuda=False, sentence_silence=0.0, data_dir=['/home/user/AI/Models/piper-voices', '/home/user/AI/Models/piper-voices'], download_dir='/home/user/AI/Models/piper-voices', update_voices=False, debug=True)
DEBUG:piper.download:Loading /home/user/AI/Models/piper-voices/voices.json
DEBUG:piper.download:Checking /home/user/AI/Models/piper-voices/en_GB-aru-medium.onnx
DEBUG:piper.download:Checking /home/user/AI/Models/piper-voices/en_GB-aru-medium.onnx.json
DEBUG:piper.download:Checking /home/user/AI/Models/piper-voices/en_GB-aru-medium.onnx
DEBUG:piper.download:Checking /home/user/AI/Models/piper-voices/en_GB-aru-medium.onnx.json
Traceback (most recent call last):
  File "/home/user/.local/pipx/venvs/piper-tts/lib/python3.10/site-packages/piper/__main__.py", line 151, in main
    voice.synthesize(text, wav_file, **synthesize_args)
  File "/home/user/.local/pipx/venvs/piper-tts/lib/python3.10/site-packages/piper/voice.py", line 103, in synthesize
    wav_file.writeframes(audio_bytes)
  File "/usr/lib/python3.10/wave.py", line 439, in writeframes
    self._patchheader()
  File "/usr/lib/python3.10/wave.py", line 494, in _patchheader
    curpos = self._file.tell()
OSError: [Errno 29] Illegal seek

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/user/.local/bin/piper", line 8, in <module>
    sys.exit(main())
  File "/home/user/.local/pipx/venvs/piper-tts/lib/python3.10/site-packages/piper/__main__.py", line 150, in main
    with wave.open(sys.stdout.buffer, "wb") as wav_file:
  File "/usr/lib/python3.10/wave.py", line 332, in __exit__
    self.close()
  File "/usr/lib/python3.10/wave.py", line 446, in close
    self._patchheader()
  File "/usr/lib/python3.10/wave.py", line 494, in _patchheader
    curpos = self._file.tell()
OSError: [Errno 29] Illegal seek


connector/github: test LoginURL

Test case addressing the GitHub connector's LoginURL() function.
Relates to #975

Why is this test useful? It looks like it's just another way to express this logic.

Going to close this for now. We need to add more tests to verify functionality of the GitHub connector to make this useful.


Add Hugo Lin

Portfolio Author
@1chooo
Portfolio Link?
https://www.1chooo.com/
GitHub Link:
https://github.com/1chooo/1chooo.com
Tech Stack
React JS, Next.js, Nextra (V3), TypeScript

@israelmitolu Thank you!!!


chore: add test case for numeric generics failing in trait methods

Description
Problem*
Resolves
Summary*
It seems that #5156 has resulted in problems in unifying numeric generics in trait impl methods.
Additional Context
Documentation*
Check one:

[x] No documentation needed.
[ ] Documentation included in this PR.
[ ] [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt on default settings.

cc @michaeljklein


Add legacy thread dump parsing. Add thread pool filter.

Thread viewer now fully supports all Ignition thread dump formats. Added support for legacy (pre-8.1.10) thread dumps from both the scripting function and web page.
Also added thread filtering by "pool". Thread names are of the format pool-number and filtering by pool makes searching through threads of a specific type of subsystem quite easy.
Misc refactoring of Thread dump parsing as well as implemented tests for legacy threads.
Fixes #45

#YOLO


Move workflow#step into State#run

Moves state step logic from workflow into step

State#run! now ignores input
State#run! now returns the block status (0) instead of the output, next_state
State#start is no longer abstract. (sorry. I can get in another setup method if we want to keep this)

updates:

converted timeout syntax to your preferred wait until run style
successfully moved start/finish into state
moved more workflow state operations into state
state#start now has a super implementation (you may not like this)

update:

fixed hash rocket and guard clause cops

update:

rebased
changed to store strings


Add test data covering various options in the spec

Related to https://github.com/opengeospatial/geoparquet/issues/123
This is incomplete (more parameters should be added) and still draft (the script should be cleaned-up, ensure to add this to the CI to check the generated files, validate with json schema, etc). But wanted to open a PR already to see where and how we want to go with this.
This is a script that writes a bunch of .parquet files, and then also saves the metadata as a separate json file (extracted from the .parquet files using the existing script scripts/update_example_schemas.py).

@jorisvandenbossche - do you want to try to get this updated for the 1.1 release? It sounds like it's complementary to #134, right?

Yes, have to update it follow the same changes I made in https://github.com/opengeospatial/geoparquet/pull/204


ImportError: No module named models

Hi,
I've been using your code to do some analysis of game logs, and I've found it very useful.
I installed the module through pip, and was able to get it to work, but I had to comment out the following line in init.py:
from .models import Game
Best,
Jonathan

Sorry I must have lost track of my email notification. Are you using Python 3 or Python 2? This might be a compatibility problem with Py3's changes in relative imports.

I'm using Python 2.7. I don't think it's a problem with compatibility, but with what's included in the module installed by pip. The only files are api.py, utils.py, and init.py. The models/ folder is not in there.

I think those files are present on the develop branch. Can you see if you can checkout that and install as a develop version to see if this is fixed?
To install as a develop version, go to the root directory of the package and run python setup.py develop


support named instance

Currently there is no way to setup named instance for mssql linux. Often times customers would need to have multiple instances on the same machine. Thanks!

Why do you need support for multiple service instances in one docker container
Your OS can faciliate multiple docker instances thus different instances of your sql service


Issue running Invoke-Pronounsync.ps1

Everything has been set up successfully but still getting this error when it finds someone that needs to be updated. Saw that others were having similar issues but cannot seem to find a resolution.

Does this happen for every user that needs an update or for just a subset of the users?

Please see this issue. Are your users synced from on-prem?
https://github.com/merill/m365-gender-pronoun-kit/issues/6#issuecomment-1173610409


Theme is different from the official test server

Use nextcloud app richdocuments to integration with collabora, but the UI is different to  the collabora online development test server
This is my self host server.

This is the collabora online test server.

The version is completely the same.
In my config.json, user_interface.mode is default, user_interface.use_integration_theme is true.

I believe Collabora inserts their own CODE theme into it at build time, which is not open source. If you find the theme it could be easily inserted into the image at build time on your system by putting source in /build-assets/online/src/ .

coming back to this one - when using the notebookbar menus, some icons are missing so the 'File' menu is superimposed by the text local edit. Furthermore the sharing menu isn't available for nextcloud. Does anybody have a hint where to get the missing files and how to insert them? copying them from a running cool instance didn't help in my case.

https://forum.collaboraonline.com/t/collabora-online-weekly-meeting-162 pointed me to something similar regarding the overlay of text around the File menu..
occ config:app:set richdocuments open_local_editor --value='no' seems to solve it. I'll have to look into the sharing on a nextcloud instance. 28 has introduced significant regressions that have caused quite a problem in the organizations I am providing services for...

oh thanks for this one. I don't want them to edit locally anyway.

hi,

https://forum.collaboraonline.com/t/collabora-online-weekly-meeting-162 pointed me to something similar regarding the overlay of text around the File menu..
occ config:app:set richdocuments open_local_editor --value='no' seems to solve it. I'll have to look into the sharing on a nextcloud instance. 28 has introduced significant regressions that have caused quite a problem in the organizations I am providing services for...

i tried the occ (NC 27.1.11 & 24.04.5-1/latest) but it didnt change the overlapping with the menu:
firefox 128.0.3 linux:

chromium 127.0.6533.99 linux:

do you actually know which config value is talked about ?:
Julius pushed “Fix open locally with files lock and wopi allow list” , thanks! Furthermore we have added a config value for disabling but also address that
for /me, like joerg, removing edit locally is fine as the feature wouldnt work on most clients anyway ...
tia,tja...


Discord Invite In Readme Expired

Hello! Would love to join discord, invite in Readme is expired though :/

https://discord.gg/rM286W7J and will update! thank you


Drag support and basic go-to-point support for touch devices

Once solving on a touchscreen, you can lift your finger, resume dragging, cancel the line, and attempt a solve by moving to an end component.

This partly fixes #6


Window resized to fill 2 monitors can restore to be very large

Have two monitors with different scales. My setup: Left external monitor at 100%, right laptop monitor at 250%.
Open VS Code and snap it to the right (250%) monitor.
Grab the left edge of VS Code and drag it to fill the left monitor.
Grab the title bar of VS Code and shift it slightly to the left, so that it's partially off screen.
Close VS Code.
Open VS Code. The window will be extra wide and go far off screen to the right.

Found while verifying https://github.com/microsoft/vscode/issues/140305.

Screenshot after step 3:

Screenshot after step 4:

Screen shot of step 6 (you can see that the window close button is not visible on the screen):

I cannot reproduce, though I am on latest Windows 11, maybe something is fixed there?

I'm on Windows 11 Version 10.0.22000 Build 22000

No matter what I try, I see no issue, even though I can reproduce that in certain cases the buttons are off screen because I left them off screen. But that is expected, we try to restore the window as it was even if it was in a somewhat inconvenient location.
To clarify, did the location change or was it how you left it?

The width of the window, and therefore the location of the buttons, changed.

It happens 100% of the time for me.

Ok lets debug together when we are both in the office, I will be there on Monday next week.

This issue doesn't actually repro on my setup at the office. The major difference is that I have a have a high DPI external monitor at the office.

Ok thanks, that will probably have an impact as outlined by upstream issues in electron:

https://github.com/electron/electron/issues/10862
https://github.com/electron/electron/issues/29605

I think @gjsjohnmurray originally wanted to tackle these shortcomings in his PR https://github.com/microsoft/vscode/pull/155589 and maybe this one case is not covered.
Since this is depending on actual upstream fixes, let's close in favor of the upstream ones.


isContourConvex support

@Designify101 Get it. Will add the function when having some time. :)
Or do you have time to work on it? @abdelaziz-mahdy
References:

https://docs.opencv.org/4.x/d3/dc0/group__imgproc__shape.html#ga8abf8010377b58cbc16db6734d92941b
https://github.com/rainyl/dartcv/blob/1ba6e92ab11c563be9bee0607ceb182dcf18a6b6/dartcv/imgproc/imgproc.h#L807-L808

It's only a function call? I think I will have time next weekend if I start working on it will let you know

Yes，just add the C wrappers in https://github.com/rainyl/dartcv/blob/1ba6e92ab11c563be9bee0607ceb182dcf18a6b6/dartcv/imgproc/imgproc.h#L807-L808 and corresponding dart codes in https://github.com/rainyl/opencv_dart/tree/main/packages/dartcv.
I am kind of busy recently, thank you in advance 😄

hi i created this pr https://github.com/rainyl/dartcv/pull/12, but when running make for ffigen it generates empty files
when i looked at the configuration it tries to find the files in the src dir? which is empty since its in another repo, dont you think it should be in this repo to avoid this kind of problem?
i can make a pr for it if you dont mind, if there is a specific reason for it let me know
since there is always the workaround of cloning the repo in the make file before generating (i just dont like it)

Will see how can I make it work, I have zero experience with cocopods 😅 so I may make a script to pull the src files to use in make file and delete them after it finishes.
For now it's a problem for tomorrow. Good night

😄 Thanks, night~


Support pseudo-stokes parameter beam as done in Kohn et al.2018

Implemented Figure 1 (equation 8) of Kohn et al. 2018.

@Chuneeta I'd like to get this merged in, can you add some examples to the tutorial?


[Enhancement] Page header automatically truncating content?

We've got a need for a page header supporting large title lengths, these are templates where the full title is very necessary.
Seemingly, truncation within the page header is not possible to customise, and as a result our product (App Connect SaaS) have created a custom page header to support this use case.

It feels like this should be customisable for teams to determined when and how titles truncate.

@Laura-Marshall Just to follow up from what we discussed in Slack — Unsure at this moment if we need to add an entire prop just to opt out of truncation — in which case CSS override is the way to go. If we think that in most cases, we want to truncate, then our default should reflect that.
Your dev teams should be able to use a custom className to override the default behavior. But please let us know if you run into other issues when trying it this way!

We try to avoid CSS overrides if at all possible, because they can be fragile. For us, just truncating the text is not a usable option, at least not without some way for the user to see the rest of the text, such as via tooltip. I think it would be better if the title text just wrapped, the same as the subtitle text already does.

@asfordmatt Looking at the PageHeader guidance, it does look like we are missing the title tooltip in our current implementation. We can definitely add the browser tooltips.
However, because our guidance explicitly recommends a single line title, I don’t think we would pursue a new prop to enable this for numerous use cases unless we update the design guidance as well.
tl;dr CSS is the way unless we get design buy-in to update the existing behavior to support both (so really the decision to truncate would be left to product teams) or define when we break from the preferred behavior.


new(blog): 0.30.0 release

Signed-off-by: Frederico Araujo<EMAIL_ADDRESS>What type of PR is this?
/kind content
Any specific area of the project related to this PR?
/area blog
What this PR does / why we need it:
Blog post for Falco 0.30.0 release.
Which issue(s) this PR fixes:
NONE
Special notes for your reviewer:
/hold

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: araujof
To complete the pull request process, please assign mfdii after the PR has been reviewed.
You can assign the PR to them by writing /assign @mfdii in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Welcome @araujof! It looks like this is your first PR to falcosecurity/falco-website 🎉

@zuc: changing LGTM is restricted to collaborators

In response to this:

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/lgtm
/approve

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: araujof, danpopSD
To complete the pull request process, please assign mfdii after the PR has been reviewed.
You can assign the PR to them by writing /assign @mfdii in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 40e87c5a66472741b9ad8806f64e86c80a1b1487

@leogr Thanks for your comments on the blog!

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: araujof, danpopSD, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/unhold


[ Question ]

Hi,
I have a problem when I launch start.bat file.

This is a screenshot of cmd when I launch it, and it closes automatically instantly.
I had a problem during installation process which it could not locate package.json and some files in module folder (I don't remember them all), so I manually moved them to designated locations in windows/system32 folder.
The installation was done without any error after that, but I don't think my toolkit is working properly.
Could you help me with this?
Thank you very much!

There seems to be an issue with the authentication in the tool
To use the tool, for now, please deactivate it if you don't need it.
You can do that in module -> plugin-config -> config.json
There will be a patch in a few days to fix that. So you will be able to use auth again.

Oh and pls move the tool out of the System32 folder somewhere else
Since this is a security concern.

Thank you so much it works now!
Also, I extracted all files to C:\Program Files\Riot Community Volunteers\League Observer Tool.
I only moved a few necessary files mentioned during installation process (install.bat) to system 32 folder.
Do I still remove all the files I moved from system32?

This is what happens if I don't put folders or files to system32

You should create a folder only for the prod-tool somewhere on your hard drive and all files from this repository need to be in the same folder. There should not be any files in system 32

Just to reinforce; install.bat must not be moved from the extracted directory. If a Windows security message pops up, you can safely ignore this (or double check the contents of install.bat if you want to be sure).
The installer relies on the required files/folders to be present, this is why it failed when it was moved out of the toolkit folder.

Ok,
I have extracted all files to C:\league-prod-toolkit-1.1.0 and ran install.bat without adding any files to system32

and I get this message which is the first one I got yesterday.
So I put package.json to system32 and it asks me tells me it cannot locate dist and module folders, so I put them again and it works well.
If I am not supposed to add anything to system32 then  I think it is a installation bug if I have done everything correct.
I am also attaching the log files if it helps.
Thank you.
2022-07-20T10_27_27_142Z-debug.log
2022-07-20T10_49_39_090Z-debug.log
2022-07-21T08_02_59_541Z-debug.log

Are you running the installer as admin? This should not be necessary and can cause this problem. Please try running the installer in your normal user context.

Yes I have been running as admin. For some reason I thought it was necessary.

This is what I see when I run the installer in normal user context

Oh nevermind about the one above.
I just forgot to extract other folders.

I think it is all fixed now!
Thank you very much for your time and help guys!!

Hi,
I have another question about api keys.
I am getting error messages  when I access API keys in localhost:3003.

I have looked issues regarding API keys and please correct me if I am wrong, that API key is not RIOT API but related to authentication which has some issues right now.
Is the tool still can be used without the authentication?
Also, can I get an idea when the next patch will be?
Thank you very much

Oh yea that's related to the same issue with auth I mentioned before
This still needs fixing on our side.

If you update the database plugin this issue should also be gone now.

Thank you it works!

However I don't think my End of Game page is fetching correct font or default theme..

Thank you it works now!

However I don't think it is fetching the right font or default theme...can I get a help on this?
Thank you


Adding Support for Multiple Examples

It seems more and more frequent that we run into problems of not having a good way of sharing multiple sketches / examples for one video. Hopefully this will fix that.
The new feature allows for adding custom variations for each page. Which is done like so:
variations:
  - name: "Data Collection Sketch"
    folder: "1_data_collection"
    lang: "p5.js"
    live_example: true

This can be in addition to or instead of the normal P5, Processing, JavaScript, and Node folders.
I changed the format to allow for more unique names and added the lang tag. These two are combined in the button like p5.js name. I know lang isn't correct because processing and p5 are libraries but I couldn't think of a better term.
I also switched all pages using p5-multi to the new system and removed it from the possibilities.
Here's a screenshot of ml5 7.3 with filtering turned off because it checks against the repository. I've highlighted the ones that would be visible on the website.

This is great, love it! A couple small comments:

Let's make sure to run a WCAG accessibility test.
I think we can clean up / simplify the language. Maybe remove "sketch" and "program"? So it's just "p5.js", "Processing", "data collection", etc.
The one thing that is a little confusing is the use of "web editor". This is the same as the "p5.js" listing but in the web editor rather than github, correct? Maybe it should just always come first? Should we actually just include the web editor and not the github link? So "p5.js" goes to the web editor?

@shiffman @gruselhaus Sorry it took me so long to following up on this. I made some changes that hopefully fix all the previous problems with the drop downs.
I ended up switching them to badges (for lack of a better word) because the drop downs are difficult to make work with screen readers and even when they are accessible in my opinion they still are tiring to use with a screen reader.
I did a WCAG test and cleaned up naming as well as grouping the web editor with p5.js sketches.
If there's no objections then I'll merge it.
The new variation options look like:
variations:
  - name: "Data Collection"
    folder: "1_data_collection"
    lang: "p5.js"
    web_editor: Fe1ZNKw1Z

Here's a few screen shots:

Love this new version!!!!

This is fantastic! Let's merge and we can make any further from there! Thank you! 🚀


Canvas Support?

Is your feature request related to a problem? Please describe.
My database has a lot of canvases in there. Could it be possible to implement some kind of support for them?
Describe the solution you'd like
Opening a canvas goes to a page that renders the canvas, with the ability to scroll, zoom, and click links contained within.
Describe alternatives you've considered
Perhaps a rendered image may be simpler, at the cost of less functionality? Or at least providing a direct-download link to the canvas?
Additional context
Canvas is, I believe, an open source standard.

The easiest (and brute-force) way is to download your canvases as images and put them into separate pages.

find a interesting project xyflow,  and use obsidian's JSON canvas,  can be rendered in our website .  But the default layout isn't proper for full-screen canvas

closing in favour of https://github.com/jackyzha0/quartz/issues/927


[Bug]

Describe the bug
Prayer times with the Muslim World League parameters appear to be incorrect.
To Reproduce
Steps to reproduce the behavior:
Create a PrayerTimes object with coordinates (51.5074, 0.1278) [The coordinates of London] and DateComponents(2021, 5, 26), and the Muslim World League parameters, like so:
PrayerTimes(       Coordinates(51.5074, 0.1278),       DateComponents(2021, 5, 26),       CalculationMethod.muslim_world_league.getParameters(),  )
Then print out the DateTime objects for Fajr and Isha, and you'll end up with the following:
flutter: 2021-05-26 00:57:00.000 flutter: 2021-05-27 00:31:00.000
Expected behavior
These prayer times are incorrect, and according to IslamicFinder.org with the same coordinates and calculation method, the following should be output instead:

Environment(please complete the following information):

Dart Language Version: 2.12.0
Flutter Version: 2.0.1
Adhan-Dart Version: 1.1.15

Perhaps there is a way to alleviate this issue/a setting that I have not been using, in which case I apologise. Jazakallah for developing this package and may Allah reward you!

Apologies! This appears to be resolved by setting a HighLatitudeRule of twighlight_angle


Jfonrose/i67 writing results to cli

Description
Added a new utility to format the result of a check to print it nicely and organized to the terminal.

Finished in branch julio/i67-writing-results-to-cli.


How to assign Mesh operation ?

🔍 Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

🐞 Description of the bug
My mesh operation is created
mop = GrpcLengthMeshOperation(name=name,
net_layer_info=net_layer_op,
refine_inside=refine_inside,
mesh_region=mesh_region,
max_length=max_length,
restrict_max_length=restrict_length,
restrict_max_elements=restrict_elements,
max_elements=max_elements)

Mesh operation remains empty

also tried this

📝 Steps to reproduce
NA
💻 Which operating system are you using?
Windows
📀 Which ANSYS version are you using?
No response
🐍 Which Python version are you using?
3.10
📦 Installed packages
NA

@svandenb-dev You are correct that mesh ops should be set like this:
self.mesh_operations = [mop]

However, max_elements should be a str not an int. Changing it to an str will fix this issue.

@drewm102 I confirm it is working now


Let outlet only flow from higher to lower level

Since the outlet works by gravity, it should only produce flow if the level of its inneighbor is higher than the level of its outneighbor.

A minimum crest level seems like a good addition to the Outlet. The level then has to exceed this minimum crest level before checking the outneighbor level.


How does one uninstall a plugin?

What problem does this feature solve?
There does not appear to be any way to remove a plugin in vue ui. The list of plugins on the dashboard don't have a trash icon next to them, as the dependencies list do have. There's no information on whether it's even possible to remove a plugin.
Can't find anything in the existing issues, and this StackOverflow post doesn't have a solution beyond 'hack about and uninstall it manually'. (https://stackoverflow.com/questions/50155541/remove-plugins-with-vue-cli3)
If there is a way, it doesn't appear to be documented, if there isn't, there should be.
What does the proposed API look like?
A vue remove <plugin> command with a corresponding trash can icon in the plugins list.

Currently, vue remove X would do nothing else than npm uninstall X. There's no "cleanup" API so far.

Could this be documented, given that an explicit add command implies there's stuff going on that an npm uninstall X might not deal with?

Plugins can add and modify your project files. It's already quite difficult especially if you change the whole project structure. I don't think it's worth the trouble to have something similar that deletes and modifies files back to the previous state. There is 99% chances that the files would have been modified in the mean time. Also, you have git to check what the plugin changed and we recommend using it everywhere (including in the UI).

@Akryum If plugins are install-only, that doesn't seem a good idea long-term. One would only ever have more and more plugins and if one decided, for-instance, that a new kind of plugin handled a situation better than an existing one, or covered more cases, one would have two plugins competing with each other. Sounds like the project would easily become borked.
Aren't plugins meant to be self-contained and when adding and removing files, isn't that limited to files they know about?

They aren't install-only: you can uninstall them using yarn or npm and clean up the files they added/modified (for example with git). My point is there isn't an automatic way of doing this, we don't have an IA to do that (yet?).

Ok.

I think it would be nice to add vue remove x to give the plugin the ability to do some clean up. Even if the default is just running npm uninstall x. I think if you have an add, users expect the remove.
Also as an example I set up my project with mocha and want to move to jest, it would be nice if removing mocha would also remove it's injected task. Also I could even be asked if I want the created files to be removed. I can see what the plugin has done by reading https://github.com/vuejs/vue-cli/blob/dev/packages/%40vue/cli-plugin-unit-mocha/generator/index.js. Having old configs / files hanging around from removed plugins might lead to problems. At the very least you could warn the user (when running vue remove x) "this might not remove all configuration changes made by the plugin it is recommended to use git to roll back plugin changes"
The other thing could be a vue add --dry-run which outputs what the plugin is going to do, which would help in the removal / clean up.
P.S. @Akryum keep up the great work and nice talk at vuejs.london!

One thing I noticed is that webpack config changes made by plugins get left behind when the package is uninstalled, which means if the same configuration item is reused it may appear to not be working, as vue inspect will show that there are two versions of the same key- the one left behind by a plugin and a new one created, unbeknownst to a new configurator.

webpack config changes made by plugins get left behind when the package is uninstalled

I can't really imagine how that would happen, since the changes to the webpack config are made on the fly from the plugin 's index.js file - which would be gone once the plugin is uninstalled.

Ok, I'll have to investigate that further and see how it's happening.

Recently I did something dumb. I just added vuetify to my little personal project following the quick start directions. I probably should not have clicked something in the cli, because, Ughh it was a mess. Errors for days rewritten hello world views.  it would be nice to be able to pull the plug on plugins . Once i saw existing templates changed, and webpack  with no undo I realized the gravity of what i did.  I wouldnt think  a "plugin" would manipulate things.  That experience has rattled me.
this is what i saw:
`Successfully invoked generator for plugin: vue-cli-plugin-vuetify
The following files have been updated / added:
 .amplifyrc
 amplify/#current-cloud-backend/amplify-meta.json
 amplify/#current-cloud-backend/analytics/lambdatest/parameters.json
 amplify/#current-cloud-backend/analytics/lambdatest/pinpoint-cloudformation-template.json
 amplify/#current-cloud-backend/auth/cognito1/cognito95306251-cloudformation-template.yml
 amplify/#current-cloud-backend/auth/cognito1/parameters.json
 amplify/#current-cloud-backend/hosting/S3AndCloudFront/template.json
 amplify/.config/aws-info.json
 amplify/.config/project-config.json
 amplify/backend/amplify-meta.json
 amplify/backend/analytics/lambdatest/parameters.json
 amplify/backend/analytics/lambdatest/pinpoint-cloudformation-template.json
 amplify/backend/auth/cognito95306251/cognito95306251-cloudformation-template.yml
 amplify/backend/auth/cognito95306251/parameters.json
 amplify/backend/awscloudformation/nested-cloudformation-stack.yml
 amplify/backend/hosting/S3AndCloudFront/template.json
 src/aws-exports.js
 src/components/Login.vue
 src/components/Signup.vue
 src/plugins/vuetify.js
 src/routes/index.js
 babel.config.js
 package-lock.json
 package.json
 src/App.vue
 src/assets/logo.png
 src/components/HelloWorld.vue
 src/main.js`

Well there's a big warning in the docs, looking like this:

WARNING
It is recommended to commit your project's current state before running vue add, since the command will invoke the plugin's file generator and potentially make changes to your existing files.

Hmm yeah , this needs a big ol catch all disclaimer ... it does not sound safe.   I thought it would just expose some components/ new functionality but literally changed every file and even broke it .  To call that a plug-in imo is misleading.  I would expect “deep project manipulations” like this to be a more associated with cli.

I think this is like the migration in the database, you can roll back your operations. Similarly, a plugin should define an up operation and a down operation for the vue add and remove commands, respectively.

Yes in some cases plugins seem to be doing what should be called "Project Migration". To me that should not be lumped in with a plugin. thats a different tool  .  they should have all they need in their package and exposing some settings/or globals.

Why is this an issue when a framework like Ember has a simple ember uninstall add-on --save-dev and this works with no issues?

@Rup1

Why is this an issue when a framework like Ember has a simple ember uninstall add-on --save-dev and this works with no issues?

ember-cli has not a command named uninstall, it has a deprecated command named uninstall:npm.
And it's obviously that uninstall:npm just removes the npm package directly. Actually it would suggest you to call npm uninstall now. https://github.com/ember-cli/ember-cli/blob/master/lib/commands/uninstall-npm.js

@jkzing Apologies I stand corrected! Haven't played with ember in a while.
Nevertheless, the same logic that justifies having a vue add plugin command should equally justify having a vue remove plugin command IMHO. Why not just remove Vue add then?

@Rup1 On the contrary, from what ember-cli did (deprecating uninstall command), we can learn that it is not necessary to provide an uninstall command which just simply do npm uninstall.

Suggestion: Perhaps plugins can choose to provide an uninstall URL with help text on how to fully uninstall them. Then, running vue remove plugin could run npm uninstall plugin followed by a message to the effect of:

The node package has been removed. Please visit (URL here) for instructions on completing the uninstallation.

FWIW, the term plugin strongly suggests ease of adding (plugging) and removal (un-plugging). In fact, outside of JS frameworks, I can't think of any plugin architecture where that isn't the case.
Thus, I think it's a reasonable expectation that plugins be un-pluggable.

I agree with this. I certainly didn't expect that the plugin would update so many of my files on a small project. Around 30 at my last count. That feels pretty intrusive for a "plugin".

I found the solution:

run from terminal vue ui
go to Dependencies
Enjoy! https://share.getcloudapp.com/GGu0wPBN

I found the solution:

run from terminal vue ui
go to Dependencies
Enjoy! https://share.getcloudapp.com/GGu0wPBN

Alas, this does not let you remove a plugin, only a dependency.

I found the solution:

run from terminal vue ui
go to Dependencies
Enjoy! https://share.getcloudapp.com/GGu0wPBN

Alas, this does not let you remove a plugin, only a dependency.

Thumb up!


Remote MySQL Database Setup

Create a remote MySQL DB that all local (and live) servers can use.

Completed.


LUIS authoring - version exportMethod - format

The LUIS authoring code for exporting a version does not appear to support the 'format' request paramter, as documented here:
https://westeurope.dev.cognitive.microsoft.com/docs/services/5890b47c39e2bb17b84a55ff/operations/5890b47c39e2bb052c5b9c40
Could we add this property the 'exportMethod' method so that it gets included in the request body?

Thanks for reporting @jspruance
Can you share the name and the version of the luis authoring package you are using?
cc @kayousef

we're using 'azure-cognitiveservices-luis-authoring', v2.1.1
https://www.npmjs.com/package/azure-cognitiveservices-luis-authoring
thanks

Thanks @jspruance
That package is not in active development and has been replaced with @azure/cognitiveservices-luis-authoring. Please try out https://www.npmjs.com/package/@azure/cognitiveservices-luis-authoring.
If you want to target v2 of the luis authoring apis, then use version 3.0.0 of the above library
If you want to target the newer v3 of the luis authoring apis that is in preview, then use version 4.0.0-preview.1 of the above library.
@kayousef can answer any other questions you have

@ramya-rao-a @kayousef
Thanks for the suggestion. I tried out the '4.0.0-preview.1' since we're targeting LUIS API v3.0.
However, I don't see that the optional 'format' request parameter is supported in either the old (v2.1.1) or new (4.0.0-preview.1') for versions 'importMethod' or 'exportMethod.' Can you confirm if this is or is not supported? Thanks.
For reference - 'format' param (see 'Export format type'):
https://westeurope.dev.cognitive.microsoft.com/docs/services/luis-programmatic-apis-v3-0-preview/operations/5890b47c39e2bb052c5b9c40

@jspruance You are right, format is not yet supported in import/export scenarios. This will be available in the next release.

Hi @jspruance in the latest version of the SDK 4.0.0-preview.3, 2 new import APIs are exposed.
exportLuFormat is available for LU and exportMethod for JSON.
Please let us know if that addresses your problem


starter template plugin needed

Providing a "blank" starter template plugin for users to use would be a nice addition.

@ddesmond actually just made something for this. It is a make command. Just needs documentation.

That would be called then - Terra application plugin template.
Yes, i do: make app-demoapp, and basic plugin structure gets created so i can jump and work on it, instead copy pastin, renaming variables etc.. some manual work is required, some of those can be improved for sure.
Let me clean up and make docs, the branch is already live

Docs updated in separate branch.
We can officaly merge this.


Request: possible to publish sha256sum along with binaries?

Hello folks,
Thanks for creating and releasing both overmind and hivemind, these have become the first tool in my toolbelt for development! 😍
Background:
Until recently I've been using only with linux/amd64 binaries with Docker, but more and more, linux/arm64 (aarch64) is becoming popular.
In order to ensure integrity of overmind/hivemind installation, I've been using the following approach in my Dockerfiles:
ENV OVERMIND_VERSION=2.2.1 \
    OVERMIND_SHA256=4a425e8654219014bb7ea127eab6f8c6c2325418814a7c128a4355f4e06b4a79

RUN --mount=type=cache,target=/var/cache/apk \
    set -eux; \
    apk add \
        tmux \
    ; \
    { \
        curl --fail --progress-bar -Lo overmind.gz https://github.com/DarthSim/overmind/releases/download/v${OVERMIND_VERSION}/overmind-v${OVERMIND_VERSION}-linux-amd64.gz; \
        echo "${OVERMIND_SHA256} *overmind.gz" | sha256sum -c - >/dev/null 2>&1; \
        zcat overmind.gz > /usr/local/bin/overmind; \
        chmod +x /usr/local/bin/overmind; \
	    rm overmind.gz; \
    }; \
    # smoke test
    [ "$(command -v overmind)" = '/usr/local/bin/overmind' ]; \
    overmind --version | grep -q "${OVERMIND_VERSION}"

Which checks the downloaded package against the computed SHA256 of the file. While limited, it aims to provide some reproducible builds.
However, with the introduction of multiple platforms (and the detection, not showing in the snippet), will be better to download the SHA256 of each binary to validate its integrity.
Request:
Will be possible to publish a .sha256sum along each of the packages? (Eg. overmind-v2.2.1-linux-amd64.gz.sha256sum) Or just one for all files in the same release?
Thank you in advance and once again, thank you for creating and sharing Overmind and Hivemind with the world! ❤ ❤ ❤
Cheers.

Hey Luis!
Added sums to the latest release

Thank you @DarthSim!!!
Have a great week!
❤ ❤ ❤


bug fix in ListImage and gofmt

Bug(https://github.com/samalba/dockerclient/pull/130) fix and also gofmt
./dockerclient.go:527: undefined: argAll
./dockerclient.go:527: cannot assign to argAll
./dockerclient.go:529: undefined: argAll

Signed-off-by: Xian Chaobo<EMAIL_ADDRESS>

ping @vieux  @ehazlett  @donhcd

LGTM


Integrate with jupyterlab/extension-examples

Problem
Finding out what token to use is not be trivial. It could benefit newcomers to have examples close at hand when first visiting the playground. https://github.com/jupyterlab/extension-examples is a terrific resource which could be re-used here.
Proposed Solution
Either:

add jupyterlab-github extension to the Binder image pointing to https://github.com/jupyterlab/extension-examples, allowing users to explore the extensions easily

pros: easy to implement
cons: the extensions are in typescript so will not immediately work; will require keeping jupyterlab-github up-to-date

add a dedicated browser of extensions which will run typescript compilation (without asset minimization) prior to showing the examples

Additional context
The TypeScript playground includes a set of pre-defined examples: https://www.typescriptlang.org/play?q=415#example/tuples

With https://github.com/jupyterlab/jupyterlab-plugin-playground/pull/28 the extensions from https://github.com/jupyterlab/extension-examples now just "work". Regarding the delivery, jupyterlab-github is still looking good. An alternative would be to use git submodule which would freeze the examples (and after testing we can be confident that the examples will work with playground) - we only have a single version of @jupyterlab and lumino packages available at runtime anyways, so we don't need any dynamic capabilities of jupyterlab-github, and having a dedicated UI would be better UX-wise.

Very exciting, thanks a lot @krassowski for pushing on this!


chore: implement observer

issue: #177

Deprecated by this one: https://github.com/henomis/lingoose/pull/190


[Enum Request] table 5077 "Segment Line" - Evaluation option field to Enum Convert

Please convert option field to Enum

Thanks!
srinivas

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


master

Hi @expendable123!
Thanks for wanting to contribute to Bootstrap by sending this pull request!
Unfortunately, your pull request seems to have some problems:

The title of this pull request strongly suggests that you were using Bootstrap to test out Git or GitHub, rather than proposing a legitimate change. If that's accurate, please DON'T do that again! Instead, use your own personal repositories to experiment with how to use Git or GitHub. Using the repos of public projects (such as Bootstrap) for such experiments wastes the time of those projects' maintainers and is thus considered rude.

You'll need to fix these mistakes and revise your pull request before we can proceed further.
Once you've fixed these problems, you can either ask the maintainers to re-open this pull request, or you can create a new pull request.
Thanks!
(Please note that this is a fully automated comment.)

(For anyone following along, this is my alter ego testing https://github.com/twbs/rorschach/issues/32)