CDISC CORE Exits when encounterd non-ASCII Characters

I have several xpt datasets that use the non ASCII mu character to represent unit i.e. μm.
The controlled term for this unit is “um”.
If I run the datasets through CDISC Core 0.8.1 instead of reporting a non controlled term, CDISC CORE aborts early (it doesn’t check all remaining xpt files in the folder) with the following error.
File "pyreadstat\_readstat_parser.pyx", line 300, in pyreadstat._readstat_parser.convert_readstat_to_python_value
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xb5 in position 0: invalid start byte
[26824] Failed to execute script 'core' due to unhandled exception!
If I run the same xpt file with the μ removed, CDISC CORE proceeds and outputs as expected.
Expected result: CDISC CORE would detect the non controlled term and report a rule failure for the value. All remaining xpt files in the folder would also be processed. CDISC CORE would not abort early.
Was using standard – sending, version 3.1 but get the same result no matter what version used to check. Also xlsx/json output has no effect. Also with or without CT version has no effect

Can't attach the xpt file as its is not supported file type

As XPT does not support non-ASCII characters, it is impossible for the engine to know what the provided bytes represent if it is not an ASCII character. Even if it were, it cannot know what encoding is used for the non-ASCII character, it can just guess.
But I agree with you that the execution should not crash for that dataset in that case, I think that a simple "try-except" in the parser would solve this problem.


in controller_name_with_namespaces': undefined method camelize' for nil:NilClass

~/work/foobar ‹ruby-1.9.2›  ‹devel› $ rails_best_practices .
/Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/lib/rails_best_practices/core/routes.rb:27:in `controller_name_with_namespaces': undefined method `camelize' for nil:NilClass (NoMethodError)ooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooooo| ETA:  00:00:00
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/lib/rails_best_practices/reviews/remove_unused_methods_in_controllers_review.rb:85:in `block in after_review'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/lib/rails_best_practices/reviews/remove_unused_methods_in_controllers_review.rb:80:in `each'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/lib/rails_best_practices/reviews/remove_unused_methods_in_controllers_review.rb:80:in `after_review'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/lib/rails_best_practices/core/check.rb:224:in `block (2 levels) in included'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/lib/rails_best_practices/core/check.rb:66:in `instance_exec'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/lib/rails_best_practices/core/check.rb:66:in `block in node_end'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/lib/rails_best_practices/core/check.rb:65:in `each'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/lib/rails_best_practices/core/check.rb:65:in `node_end'
	from (eval):17:in `block in review'
	from (eval):15:in `each'
	from (eval):15:in `review'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/lib/rails_best_practices/core_ext/sexp.rb:16:in `review'
	from (eval):12:in `block in review'
	from (eval):10:in `each'
	from (eval):10:in `review'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/lib/rails_best_practices/core_ext/sexp.rb:16:in `review'
	from (eval):12:in `block in review'
	from (eval):10:in `each'
	from (eval):10:in `review'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/lib/rails_best_practices/core_ext/sexp.rb:16:in `review'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/lib/rails_best_practices/core/runner.rb:135:in `after_review'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/lib/rails_best_practices/analyzer.rb:90:in `process'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/lib/rails_best_practices/analyzer.rb:57:in `block in analyze'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/lib/rails_best_practices/analyzer.rb:57:in `each'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/lib/rails_best_practices/analyzer.rb:57:in `analyze'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/lib/rails_best_practices/command.rb:114:in `<top (required)>'
	from /Users/jason/.rvm/rubies/ruby-1.9.2-p290/lib/ruby/site_ruby/1.9.1/rubygems/custom_require.rb:39:in `require'
	from /Users/jason/.rvm/rubies/ruby-1.9.2-p290/lib/ruby/site_ruby/1.9.1/rubygems/custom_require.rb:39:in `require'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/gems/rails_best_practices-1.7.2/bin/rails_best_practices:6:in `<top (required)>'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/bin/rails_best_practices:19:in `load'
	from /Users/jason/.rvm/gems/ruby-1.9.2-p290/bin/rails_best_practices:19:in `<main>'

could you paste your route files to me?

devise_for :users, :controllers => { :registrations => "registrations" } do
  get "/register", :to => "registrations#new"
  get "/login", :to => "devise/sessions#new"
  get "/logout", :to => "devise/sessions#destroy"
  get "/signup_check", :to => "registrations#signup_check"
end

我查出来是这几行的问题

hmmm...I'm afraid it's not only the route file issue, does it an open sourced project? If so, just give me the project url.
If not, we need several steps to debug, first, please print @routes value in lib/rails_best_practices/reviews/remove_unused_methods_in_controllers_review.rb

I am running into the same problem and for me it seems like the atom.builder templates are causing a problem.

@mhenrixon, could you let me clone your repository? it's difficult for me to guess where is the issue

@flyerhzm wish I could. Any suggestions that might help me search for the root cause of the problem? It's custom actions and they exist in both routes, controllers and views.

please print @routes value in lib/rails_best_practices/reviews/remove_unused_methods_in_controllers_review.rb, thanks

I am sorry but I don't understand what you are after. I am trying to run remove_unused_methods_in_controllers and it crashes saying controller_name_with_namespaces: undefined method camelize for nil:NilClass (NoMethodError)
what do you mean with "print @routes value"?

I mean you can debug the rails_best_practices in your computer.
add the following codes on line 80 in lib/rails_best_practices/reviews/remove_unused_methods_in_controllers_review.rb
p @routes
p @controller_names

then run the rails_best_practices again

Project is on ruby 1.9.3. Would that matter? I can try both though I just
uninstalled 1.9.2 and rather not keep it around unless I have to :)

I get the same error when I run bin/rails_best_practices -d ../project nothing prints except for the usual stuff.

it should be fixed on master branch

No still have the same problem.
./app/views/users/wods.atom.builder
/Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/lib/rails_best_practices/core/routes.rb:27:in `controller_name_with_namespaces': undefined method `camelize' for nil:NilClass (NoMethodError)
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/lib/rails_best_practices/reviews/remove_unused_methods_in_controllers_review.rb:85:in `block in after_review'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/lib/rails_best_practices/reviews/remove_unused_methods_in_controllers_review.rb:80:in `each'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/lib/rails_best_practices/reviews/remove_unused_methods_in_controllers_review.rb:80:in `after_review'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/lib/rails_best_practices/core/check.rb:230:in `block (2 levels) in included'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/lib/rails_best_practices/core/check.rb:66:in `instance_exec'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/lib/rails_best_practices/core/check.rb:66:in `block in node_end'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/lib/rails_best_practices/core/check.rb:65:in `each'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/lib/rails_best_practices/core/check.rb:65:in `node_end'
	from (eval):17:in `block in review'
	from (eval):15:in `each'
	from (eval):15:in `review'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/lib/rails_best_practices/core_ext/sexp.rb:16:in `review'
	from (eval):12:in `block in review'
	from (eval):10:in `each'
	from (eval):10:in `review'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/lib/rails_best_practices/core_ext/sexp.rb:16:in `review'
	from (eval):12:in `block in review'
	from (eval):10:in `each'
	from (eval):10:in `review'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/lib/rails_best_practices/core_ext/sexp.rb:16:in `review'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/lib/rails_best_practices/core/runner.rb:136:in `after_review'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/lib/rails_best_practices/analyzer.rb:91:in `process'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/lib/rails_best_practices/analyzer.rb:58:in `block in analyze'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/lib/rails_best_practices/analyzer.rb:58:in `each'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/lib/rails_best_practices/analyzer.rb:58:in `analyze'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/lib/rails_best_practices/command.rb:124:in `<top (required)>'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/bin/rails_best_practices:6:in `require'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bundler/gems/rails_best_practices-3378118e35f6/bin/rails_best_practices:6:in `<top (required)>'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bin/rails_best_practices:19:in `load'
	from /Users/mikael/.rvm/gems/ruby-1.9.3-p125@wodstack/bin/rails_best_practices:19:in `<main>'

@mhenrixon, could you paste your routes to me?  It may solve your issue

tried and it crash on exactly the same spot unfortunately. Anything else I can provide?

could you remove all the routes and add one by one to detect which route cause the issue?

You were right, when I glanced over the routes again it was kind of easy to spot the problematic route. A left over technical debt.

could you tell me which route?

of course it's the only route left in my edited message above

I have updated codes that will not parse wrong route


Update en.json

Hello,
Typing error on munkireport - passprase instead of passphrase
It's seems that we have duplicate key ( "localonlymanifest on munkiinfo)

👍

Helle Arjen,
Could you verify these key :
"munkiinfo": {
"softwarerepourl": "Software Repo URL",
"localonlymanifest": "Run Type",
"duration": "Duration",
"applecatalog": "Apple Catalog URL",
"manifest": "Manifest",
"localonlymanifest": "Local Only Manifest",
"start": "Start"
},

Hmmm, that does not look right!

I pushed a fix, thanks for reporting.


Classes from packages PDFSharp and MigraDoc aren't imported to the project

Can't import the classes used in here, but these one I can.
Visual Studio 2017 simply doesn't seem to find them.
I'm using ASP.NET Core 2.1 on windows 10.
Maybe you can point which specefic packages from nugget should I install?

There are NuGet packages with just PDFsharp and there are NuGet packages with PDFsharp&MigraDoc. If you want to use MigraDoc classes, get a package that includes them.
https://www.nuget.org/profiles/PDFsharp-Team

I have installed this package, but still can't even import the Document class that says on the documentation.

Thats the installed package, and this is what VS2017 tells me to import:

Maybe a using is still missing.
Have you tried compiling the Hello World sample:
https://github.com/empira/MigraDoc-samples

Yes, I had to import it manually... Weird that VS2017 doesn't list it. Thanks for the help!


chore(docker): controller support multi-arch and refine Makefile

Description

Docker image of the controller support multi-arch
refine Makefile in docker dir, use the build-image which is based on buildx in all the release workflow

Modules

[ ] UI
[ ] Controller
[ ] Agent
[ ] Client
[ ] Python-SDK
[x] Others

Checklist

[ ] run code format and lint check
[ ] add unit test
[ ] add necessary doc

@jialeicui  https://github.com/star-whale/starwhale/actions/runs/4350126117/jobs/7600519020


[glib] Fix build failures

https://gitlab.gnome.org/GNOME/glib/-/merge_requests/3386
Some headers are not generated early enough in concurrent builds. The patch adds dependencies.

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

/azp run

Thanks for bringing us an upstream patch!

The feature has been tested successfully locally.


Create DashpageChart component to render a single chart in a dashpage

When showing a single metric, we make generous use of the screen real estate.
For dashpages, we'll want a more constrained view, as we want to show multiple metrics in the same area.
We need a component that displays:

A chart
A table
A means to toggle between chart/table
A means to show the filter

The toggle and filters should be accessed via a discrete icon in the corner. They must be accessible.

Single metric view

DashpageChart view (proposed)
Similar to Azure Data Explorer dashboard components:

We need to rebase main onto feature/dashpages in order to get some necessary components - specifically GridColumnManager. We ned it to allow us to show filters properly
(the value updates toggle has also not yet been added)


kotlinx immutable collections JSON adapters

Resolve issue https://github.com/ZacSweers/MoshiX/issues/460
Moshi JsonAdapters for kotlinx.collections.immutable collection types
Note the JsonAdapterFactory implementations in this PR are similar to the core Moshi versions of the adaptors (eg: MapJsonAdapter.kt).
Usage
Gradle dependency
dependencies {
  implementation("dev.zacsweers.moshix:moshi-adapters-kotlinx-immutable:<version>")
}

Immutable Collection Adapters
Use addKotlinXImmutableAdapters() to add PersistentListJsonAdapterFactory and PersistentMapJsonAdapterFactory type adapters. These will deserialize standard JSON list and dictionary types into kotlinx.collections.immutable PersistentList and PersistentMap types.
val moshi = Moshi.Builder()
  .addKotlinXImmutableAdapters()
  .build()

@JsonClass(generateAdapter = true)
data class MyImmutableModel(
  val stringList: PersistentList<String>, // Supports List of Strings
  val objList: PersistentList<SomeObject>, // Also supports List of Objects
  val stringMap: PersistentMap<String, String>, // Supports Map of <Strings, String>
  val objMap: PersistentMap<String, SomeObject>, // Also supports Map of <String, Object>
)

Thanks for starting this! There are some things I think we need to do along the way.

[ ] support Sets
[ ] support Collection
[ ] simplify the API to just ImmutableCollectionsJsonAdapter.Factory, rather than one for each.

@brentwatson are you planning to continue this?

@ZacSweers yes, I was hoping to get to it this week, but wasn't able to.
Next week I'm going to prep for DroidCon NY talk, and then AFK for a week
after that so it likely won't be until late Sept / early Oct. If you want
to close this you can and I can open a new PR when I have something ready.
Or we can merge this as a v1 and I'll do Set and Collection support as a
follow-up PR... up to you
On Sat, Sep 2, 2023, 8:36 PM Zac Sweers @.***> wrote:

@brentwatson https://github.com/brentwatson are you planning to
continue this?
—
Reply to this email directly, view it on GitHub
https://github.com/ZacSweers/MoshiX/pull/478#issuecomment-1703990998,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKE6OW2OX5XJGJK4J4HM23XYPUKHANCNFSM6AAAAAA3V64PIQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

No worries, just continue this PR when you're ready. Just asking as I'm about to cut a bugfix release

Implemented in #586


adding transform

for use via node which maybe expected to work by default
improving the great work done by @jonnyreeves
karma.config.js extract
var istanbulify = require('istanbulify');
module.exports = function(config) {
...
   browserify = {
        debug: true,
        transform: [ istanbulify({ exclude: ['**/vendor/**'] })]
   }
}

closing this as browserify-istanbul is recommended


Search button is a button

Description
Hi! This is my first PR in an open-source repo. I've never used Ghost or its themes before. However, I think I succeeded in making it scalable for color changes. Any and all corrections are highly appreciated.
Thanks!
Issue
Search "button" is a link #3
Screenshots
Desktop

Mobile

Great stuff, thanks a lot for the help! SEO is now 100 in lighthouse! 🥳


Updates bazel max supported version to 0.9.0

Fixes Issue: https://github.com/istio/istio/issues/2264
With this PR, istio builds using bazel 0.9.0:
$ make build
<SNIP>
bazel  build  //...
...........
INFO: Analysed 455 targets (748 packages loaded).
INFO: Found 455 targets...
INFO: Elapsed time: 406.458s, Critical Path: 53.54s
INFO: Build completed successfully, 1873 total actions

Hi @danehans. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

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. I understand the commands that are listed here.

/ok-to-test

@danehans: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-smoke.sh
7e583c7c2a9269c49c541e0d66dd133c256deeef
link
/test e2e-smoke

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. I understand the commands that are listed here.

/lgtm
/approve


Update mentions of 'ra_vfs' in architecture doc

Adjust the paragraph about ra_vfs to point to the vfs crate that is used now.

Note, the updated text is based on my limited knowledge of this project. I happy to rewrite it if there are things missing or wrong.

bors r+
Thanks!


Open and keeping multiple HTTP/TCP connections open

Hi @inaz2 ,
Thank you for developing proxy2.py, it has been very useful. I'm currently trying to modify your code to support multiple parallel HTTP/TCP connections. The goal is to intercept a HTTP GET request and turns it into multiple (N) GET requests to different parts of the file, then reassemble them before handing it back to the client. I really need to keep N connections open throughput the session to minimise overhead.
How do I achieve that? Would you mind pointing out where I should modify to open multiple sockets, keep them open, send N GET requests and reassemble them. I tried stepping through the code and modifying do_GET() to issue multiple GET requests, but without any success.
If you could assist me or provide me with some pointers, that would be much appreciated.
Thank you.
Regards,
Jonathan

How about implementing a custom request_handler which sends multiple GET requests, assembles their responses, and self.wfile.write(assembled_response) to the client?
If doing this, let the request_handler return None and remove self.send_error(403) in L132 not to send a unnecessary 403 response.

Thanks for your prompt reply @inaz2 . Do we need to explicitly open multiple sockets or does the rest of the code look after that after we implement a custom request_handler
I identify this line of code responsible for opening non SSL connection:
self.tls.conns[origin] = httplib.HTTPConnection(netloc, timeout=self.timeout)
Do we need a loop to create multiple httplib.HTTPConnection objects?
And how do we reassemble the responses if they do not return in order?
Thanks for your help.
Regards,
Jonathan

I suggest you to use Pool.map() in multiprocessing module 🤞

Thanks @inaz2 . I'll look into it.
Regards,
Jonathan


initial commit of the /variants/gene/search endpoint

Codecov Report

Merging #225 into develop will increase coverage by 0.17%.
The diff coverage is 87.67%.

@@             Coverage Diff             @@
##           develop     #225      +/-   ##
===========================================
+ Coverage    81.56%   81.74%   +0.17%
===========================================
  Files           38       38
  Lines        17473    17518      +45
  Branches      1755     1764       +9
===========================================
+ Hits         14252    14320      +68
+ Misses        2580     2562      -18
+ Partials       641      636       -5

Impacted Files
Coverage Δ

candig/server/exceptions.py
99.46% <100%> (ø)
:arrow_up:

candig/server/frontend.py
67.19% <50%> (-0.04%)
:arrow_down:

candig/server/backend.py
75.7% <87.69%> (+2.52%)
:arrow_up:

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 834536c...bf4e30c. Read the comment docs.


[Bug]: "Unexpected token 'with'" on Shopify App

Please confirm that you have:

[x] Searched existing issues to see if your issue is a duplicate. (If you’ve found a duplicate issue, feel free to add additional information in a comment on it.)
[X] Reproduced the issue in the latest CLI version.

In which of these areas are you experiencing a problem?
App
Expected behavior
I followed the tutorial on how to initialize a Shopify App using Shopify CLI v3.0 on Node. The app is supposed to start after installation and show the home screen from "app._index.ts".
Actual behavior
An error popped out showing "Unexpected token 'with'"
Verbose output
Prisma schema loaded from prisma\schema.prisma
11:46:05 │ remix    │
11:46:05 │ remix    │ ✔ Generated Prisma Client (v5.14.0) to .\node_modules\@prisma\client in 96ms
11:46:05 │ remix    │
11:46:05 │ remix    │ Start using Prisma Client in Node.js (See: https://pris.ly/d/client)
11:46:05 │ remix    │
11:46:05 │ remix    │ import { PrismaClient } from '@prisma/client'
11:46:05 │ remix    │ const prisma = new PrismaClient()
11:46:05 │ remix    │ ```
11:46:05 │ remix    │ or start using Prisma Client at the edge (See: https://pris.ly/d/accelerate)
11:46:05 │ remix    │ ```
11:46:05 │ remix    │ import { PrismaClient } from '@prisma/client/edge'
11:46:05 │ remix    │ const prisma = new PrismaClient()
11:46:05 │ remix    │ ```
11:46:05 │ remix    │
11:46:05 │ remix    │ See other ways of importing Prisma Client: http://pris.ly/d/importing-client
11:46:05 │ remix    │
11:46:05 │ remix    │ ┌─────────────────────────────────────────────────────────────┐
11:46:05 │ remix    │ │  Deploying your app to serverless or edge functions?        │
11:46:05 │ remix    │ │  Try Prisma Accelerate for connection pooling and caching.  │
11:46:05 │ remix    │ │  https://pris.ly/cli/--accelerate                           │
11:46:05 │ remix    │ └─────────────────────────────────────────────────────────────┘
11:46:05 │ remix    │
11:46:08 │ remix    │ Prisma schema loaded from prisma\schema.prisma
11:46:08 │ remix    │ Datasource "db": SQLite database "dev.sqlite" at "file:dev.sqlite"
11:46:09 │ remix    │
11:46:09 │ remix    │ SQLite database dev.sqlite created at file:dev.sqlite
11:46:09 │ remix    │
11:46:09 │ remix    │ 1 migration found in prisma/migrations
11:46:09 │ remix    │
11:46:09 │ remix    │ Applying migration `20240530213853_create_session_table`
11:46:09 │ remix    │
11:46:09 │ remix    │ The following migration(s) have been applied:
11:46:09 │ remix    │
11:46:09 │ remix    │ migrations/
11:46:09 │ remix    │   └─<PHONE_NUMBER>3853_create_session_table/
11:46:09 │ remix    │     └─ migration.sql
11:46:09 │ remix    │
11:46:09 │ remix    │ All migrations have been successfully applied.
11:46:09 │ webhooks │ App isn't responding yet, retrying in 5 seconds
11:46:11 │ remix    │ ✔ Console Ninja extension is connected to Remix, see https://tinyurl.com/2vt8jxzw
11:46:14 │ webhooks │ App isn't responding yet, retrying in 5 seconds
11:46:19 │ webhooks │ App hasn't started in time, giving up
11:46:19 │ webhooks │ APP_UNINSTALLED webhook delivery failed
Error forwarding web request: AggregateError
11:46:46 │ remix    │   ➜  Local:   http://localhost:50417/
11:46:46 │ remix    │   ➜  Network: use --host to expose
11:47:08 │ remix    │ [shopify-api/INFO] version 11.0.0, environment Remix
11:47:35 │ remix    │ 11:47:35 AM [vite] Error when evaluating SSR module /app/routes/app.tsx: failed to import "@shopify/shopify-app-remix/react"
11:47:35 │ remix    │ |- SyntaxError: Unexpected token 'with'
11:47:35 │ remix    │     at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:118:18)
11:47:35 │ remix    │     at callTranslator (node:internal/modules/esm/loader:273:14)
11:47:35 │ remix    │     at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:278:30)
11:47:35 │ remix    │
11:47:35 │ remix    │ 11:47:35 AM [vite] Error when evaluating SSR module virtual:remix/server-build: failed to import "/app/routes/app.tsx"
11:47:35 │ remix    │ |- SyntaxError: Unexpected token 'with'
11:47:35 │ remix    │     at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:118:18)
11:47:35 │ remix    │     at callTranslator (node:internal/modules/esm/loader:273:14)
11:47:35 │ remix    │     at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:278:30)
11:47:35 │ remix    │
11:47:35 │ remix    │ 11:47:35 AM [vite] Internal server error: Unexpected token 'with'
11:47:35 │ remix    │       at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:118:18)
11:47:35 │ remix    │       at callTranslator (node:internal/modules/esm/loader:273:14)
11:47:35 │ remix    │       at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:278:30)
11:55:37 │ remix    │ 11:55:37 AM [vite] Internal server error: Unexpected token 'with'
11:55:37 │ remix    │       at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:118:18)
11:55:37 │ remix    │       at callTranslator (node:internal/modules/esm/loader:273:14)
11:55:37 │ remix    │       at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:278:30)

Reproduction steps

Create a shopify app using shopify app init
Choose "Install with Remix" and "Typescript" for language.
Upon finish installation, start the app using shopify app dev
Preview the app by pressing "p".
Install the app on your partner store.
Error pop out.

Operating System
Windows 11
Shopify CLI version (check your project's package.json if you're not sure)
3.60.1
Shell
Powershell
Node version (run node -v if you're not sure)
v20.9.0
What language and version are you using in your application?
Typescript on Node v20.9.0

It was an issue server-side. It should work now no matter the version, sorry for the inconvenience! We'll try to ensure it doesn't happen again.

Hi @gonzaloriestra ! This issue is still happening to me, should it simply work again or do I need to do something in my App's code? I just generated a new app and am trying to run it

This is happening again with:
Ubuntu 24.04
Node 22.6.0
Shopify CLI 3.65.3
Same error message:
remix │ 6:38:29 PM [vite] Error when evaluating SSR module /app/routes/app.jsx: failed to import "@shopify/shopify-app-remix/react"
remix │ |- SyntaxError: Unexpected token 'with'
remix │     at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:152:18)
remix │     at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:298:14)
remix │
remix │ 6:38:29 PM [vite] Error when evaluating SSR module virtual:remix/server-build: failed to import "@shopify/shopify-app-remix/react"
remix │ |- SyntaxError: Unexpected token 'with'
remix │     at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:152:18)
remix │     at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:298:14)
remix │
remix │ 6:38:29 PM [vite] Internal server error: Unexpected token 'with'
remix │       at ModuleLoader.moduleStrategy (node:internal/modules/esm/translators:152:18)
remix │       at ModuleLoader.moduleProvider (node:internal/modules/esm/loader:298:14)

I tried it again this morning on my windows 10 machine, running node 22.3.0 and shopify CLI v3.65.3, and it worked fine.
The errors I had encountered before were running on an Ubuntu 24.04 VM on the remix template with default selections prompted by shopify app init.
I'm torn between a few projects at work right now, but I plan to investigate and submit a full ticket about the issue running on the VM when I have some time.
Thank you for your help, and for all the work you do to keep up this project!

I’m encountering the same "Unexpected token 'with'" error as mentioned in the discussion when trying to run my Shopify app. Here are the details of my setup:

Operating System: macOS 13.6.3
Shopify CLI Version: 3.66.1
Node.js Version: 22.7.0

Steps to Reproduce:

Created a new Shopify app using shopify app init.
Chose "Install with Remix" and "JavaScript" for the language.
After finishing the installation, started the app using shopify app dev.
Pressed "p" to preview the app and installed it on my development store.
Encountered the "Unexpected token 'with'" error in the console logs.

Error Details:
Here are some excerpts from my error logs:

[vite] Error when evaluating SSR module /app/routes/app.jsx: failed to import "@shopify/shopify-app-remix/react"
|- SyntaxError: Unexpected token 'with'
    at ESMLoader.moduleStrategy (node:internal/modules/esm/translators:117:18)
    at ESMLoader.moduleProvider (node:internal/modules/esm/loader:218:14)

I’ve tried updating my Node.js version and ensuring all dependencies are up to date, but the error persists.
It seems similar to the issues others have reported with Node versions 20.0.0 - 20.9.0, but I’m currently using 22.7.0.
Any insights or suggestions would be greatly appreciated!


JSON.parse can't parse special characters

JSON.parse errors when parsing strings with special chars, e.x. \t or \n.
https://play.crystal-lang.org/#/r/7qpu
require "json"

puts JSON.parse %({"body": "A bunch of data\tyo"})

Unhandled exception: Unexpected char '	' at 1:26 (JSON::ParseException)
  from /usr/lib/crystal/json/lexer.cr:371:7 in 'raise'
  from /usr/lib/crystal/json/lexer.cr:367:5 in 'unexpected_char'
  from /usr/lib/crystal/json/lexer.cr:366:11 in 'unexpected_char'
  from /usr/lib/crystal/json/lexer/string_based.cr:28:11 in 'consume_string'
  from /usr/lib/crystal/json/lexer.cr:61:37 in 'next_token'
  from /usr/lib/crystal/json/parser.cr:108:11 in 'next_token'
  from /usr/lib/crystal/json/parser.cr:83:11 in 'parse_object'
  from /usr/lib/crystal/json/parser.cr:35:7 in 'parse_value'
  from /usr/lib/crystal/json/parser.cr:13:12 in 'parse'
  from /usr/lib/crystal/json.cr:137:5 in 'parse'
  from /eval:3:6 in '__crystal_main'
  from /usr/lib/crystal/crystal/main.cr:97:5 in 'main_user_code'
  from /usr/lib/crystal/crystal/main.cr:86:7 in 'main'
  from /usr/lib/crystal/crystal/main.cr:106:3 in 'main'
  from __libc_start_main
  from _start
  from ???

Discovered while doing some manual testing of https://github.com/Blacksmoke16/oq/pull/37.

The exact same program fails in Ruby too: https://carc.in/#/r/7qrf
According to the JSON spec, a string is any codepoint except ", \ or control characters.


Introduce RuboCop to help clean up the code

Run rubocop -a and rubocop --auto-gen-config to clean up the code.
This is a cosmetic change only, so no CHANGELOG entry is recorded.

Coverage remained the same at 99.42% when pulling 28b7eee399cacdd3c6cebe1d0878fdc9811eaf05 on jlduran:rubocop-auto-correct into e7433c4d6c8088b8e20a1b1faa3b0bf6b5b6458c on kslazarev:master.

Coverage increased (+0.5%) to 99.894% when pulling 3e8df63d0fcfafa7cd248eb4afacd55c5e0dae3c on jlduran:rubocop-auto-correct into f351fe90bd66593fbde80d3c21a0824b7e6838be on kslazarev:master.

Coverage increased (+0.5%) to 99.894% when pulling 3e8df63d0fcfafa7cd248eb4afacd55c5e0dae3c on jlduran:rubocop-auto-correct into f351fe90bd66593fbde80d3c21a0824b7e6838be on kslazarev:master.

Coverage increased (+0.5%) to 99.894% when pulling 3e8df63d0fcfafa7cd248eb4afacd55c5e0dae3c on jlduran:rubocop-auto-correct into f351fe90bd66593fbde80d3c21a0824b7e6838be on kslazarev:master.


permission denied .kweb

when running docker compose on a freshly installed debian server (with actual docker version) web cannot startup
it always gets the error:
chown: /.kweb/locks: Permission denied
chown: /.kweb/acme: Permission denied
I created in the same folder of docker-compose a folder web and gave that one 777 permission - to try, without any success.
any ideas what i'm doing wrong?

Hi @f31n,
please provide the data from https://github.com/kopano-dev/kopano-docker#troubleshooting

I'm trying to run
https://github.com/Kopano-dev/kopano-docker/tree/master/examples/meet
Running Kopano Meet without Kopano (with only the LDAP backend), with the default setup.sh
after cloning the folder I ran setup.sh provinding a valid DNS name and added a valid email for letsencrypt leaving everything else at default.
after that I edited the .env and added my turn user and password.
and i didn't user "docker-compose up", but "docker-compose up -d"

Hi @f31n,
your last update only partly answers my question. The output of docker-compose config can be quite helpful to reproduce issues (I can then run issues that are configured in the exact same way your environment is). But knowing you are running the meet example also prived me with the required clue.
The problem should be resolved with the changes from https://github.com/zokradonh/kopano-docker/pull/375.
Please confirm if that is the case for you.

thank you very much, and sorry for the troubles. next time I'll definitely provide you with the docker-compose.yml // config!
yes, its working like a charm now!

No problem


fix(network): identify protocol: possible dead lock in identification

What this PR does / why we need it:
Status lock may still be hold when we wake up identification wait futures.
Which issue(s) this PR fixes:

Fixes #
Which docs this PR relation:
Ref #
Which toolchain this PR adaption:
No Breaking Change
Special notes for your reviewer:

/lgtm


[MM-34343] Migrate 'components/suggestion/search_channel_suggestion' and associated tests to TypeScript

Summary
This code migrate "components/suggestion/search_channel_suggestion" component to TypeScript.
Ticket Link
Jira card: https://mattermost.atlassian.net/browse/MM-34343
Github issue: https://github.com/mattermost/mattermost-server/issues/17280
Release Note
- Refactor component to typescript, no changes in UI

Hello @arvinDarmawan,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Hi  @devinbinnie ,  I already create the pull request for this issue  but I set it to "Draft" because I have some questions for this task.
I check the previous code and I see that the class "SearchChannelSuggestion" is a child class from parent class "Suggestion".
Currently I only translate "SearchChannelSuggestion" to typescript but for Suggestion still in javascript language.
Should I change Suggestion class to typescript too ?
Because if I change it to typescript, I think there are several classes (ChannelSuggestion, EmoticonSuggestion, SearchSuggestionList, etc) that need to be checked again since it's related with 'Suggestion'.
Or should I convert it from Inheritance to Composition style for class "SearchChannelSuggestion" ??

/update-branch


Don't send 'completed' event to tracker on client.seed

Follow-on to bittorrent-tracker/#185
Completed event now only emits when a previously started torrent download completes

@DiegoRBaquero From a glance it looks like _checkDone is eventually called via _onTorrentId, but this is from client.seed. So we could remove _checkDone from torrent.load, but now torrent.js's proper operation is dependent on properly using it through client.seed
I'm not very close to this code, so let me know if I'm missing something obvious!

@DiegoRBaquero _checkDone needs to get called or else file.on('done') and torrent.on('done') won't fire on torrents that are added when they are already complete on disk.
I think this PR is correct to just change the message that is sent to the tracker server.
However, I have style comments. See inline.

0.98.2


return early from transaction submission if reverted or canceled

In case of Revert or Cancel error type, we definitely know what happened -> transaction is mined. There is no need to wait for other futures.
This prevents racing when, for example, we have two submissions running at the same time, first one finishes with Revert error while other detects change in nonce, but does not detect mined transaction because it has a different set of submitted transaction hashes, therefore it returns SimulationRevert. In this case, we want Revert as a final error.

Codecov Report

Merging #1601 (a348a2e) into main (9bba97a) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #1601      +/-   ##
==========================================
- Coverage   66.63%   66.60%   -0.03%
==========================================
  Files         171      171
  Lines       33639    33647       +8
==========================================
- Hits        22415    22412       -3
- Misses      11224    11235      +11


Migrate core + shuttle infrastructure away from Equinix

### Tasks
- [ ] https://github.com/application-research/estuary-hosted-infrastructure/issues/30
- [ ] https://github.com/application-research/estuary-hosted-infrastructure/issues/21
- [ ] https://github.com/application-research/estuary-hosted-infrastructure/issues/32
- [ ] https://github.com/application-research/estuary-hosted-infrastructure/issues/31
- [ ] https://github.com/application-research/estuary-hosted-infrastructure/issues/33
- [ ] https://github.com/application-research/estuary-hosted-infrastructure/issues/29
- [ ] https://github.com/application-research/estuary-hosted-infrastructure/issues/28
- [ ] https://github.com/application-research/estuary-hosted-infrastructure/issues/27
- [ ] https://github.com/application-research/estuary-hosted-infrastructure/issues/26

Done


WIP: Additional Gutenberg Support

Description of the Change
This proposes additional tools to make writing tests for blocks in the Gutenberg editor faster and easier, as well as a new set of standard tests that can be used for core blocks. This is a work in progress and there is much more that can be added, but I would like to get feedback on the approach and see if you are interested in this feature.

Alternate Designs

Benefits

It will make it easier and faster to write block tests for Gutenberg
Can provide easy to follow examples for testing blocks
Helps to demonstrate the usefulness of wpacceptance to look for block regressions.
This pairs very nicely with wpsnapshots in that you can make sure that block changes will not create validation errors on existing blocks.

Possible Drawbacks
Gutenberg is still very much in flux, so any functions using core Gutenberg classes for click events could change in future releases of core.

Verification Process
I have tested the new functionality locally and the tests pass. This should not introduce any regressions since it does not make any changes to existing functionality.

Checklist:

[X] I have read the CONTRIBUTING document.
[X] My code follows the code style of this project.
[X] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my change.
[ ] All new and existing tests passed.

Applicable Issues

Changelog Entry

@PypWalters this would be pretty fantastic to get added in, what else do you feel is needed before this can get moved out of WIP?

@jeffpaul This would be different for different versions of Gutenberg because they have changed a lot of classes over time. So there is a lot of testing, and there are conditionals that would need to be put in place depending on WP version.
I am also interested in feedback regarding whether this is a good approach or whether there is anything else I should be considering.

@PypWalters I have been wanting to explore a different idea for testing Gutenberg blocks, but my idea would be to test them in isolation, outside of WordPress by rendering the block and the block editor separately and running the assertions against them.
This would work well for blocks that aren't dependent on WP stuff,

@nicholasio that sounds a little bit like what @fabiankaegy has worked on extending BlockBook with his preliminary BlockGuide plugin that displays individual blocks outside the editor.  Maybe some of what he's built there could be leveraged here?

@nicholasio that sounds a little bit like what @fabiankaegy has worked on extending BlockBook with his preliminary BlockGuide plugin that displays individual blocks outside the editor. Maybe some of what he's built there could be leveraged here?

Yeah but what he has done would still require WordPress in the mix, I was thinking of a testing strategy without WordPress.

@nicholasio With WPA you are already spinning up an environment to run tests on, if we tested blocks outside of WP would we need to spin up something else entirely?

@nicholasio With WPA you are already spinning up an environment to run tests on, if we tested blocks outside of WP would we need to spin up something else entirely?

Yes, one approach I'm thinking of is to just render Gutenberg in jsdom and run assertions with jest, on this approach we wouldn't need to spin up any environment making it much easier to run tests on CI (and much faster).
Another approach is to spin up a react application and run end-to-end tests with cypress or puppeteer.
I'm not sure how well those two approaches would work and also not sure about the limitations yet. I think truly e2e tests should still be considered though


[Feature request] Add better support for Brazilian Portuguese

feature
I did a test to OCR scanned documents in Brazilian Portuguese, and I saw that LLaVA makes a lot of mistakes on scanned documents in Portuguese
result from https://huggingface.co/spaces/badayvedat/LLaVA
Prompt: transcribe image

The image features a white piece of paper with a stamp on it. The stamp is placed in the upper left corner of the paper, and it appears to be a government document. The paper is also marked with a signature, which is located in the middle of the document. The overall appearance of the paper suggests that it is an official document, possibly related to a government agency or a legal matter.

the correct thing would be
Here is the transcription of the text from the image:

ESTADO DO RIO DE JANEIRO
Prefeitura Municipal de Rio das Ostras
PROTOCOLO GERAL

Processo: 18457 / 2003
Data: 03/09/2003
Hora: 10:53:56
Requerente: COSCARELLI E CIA LTDA ME
Sec. Destino: Secretaria Municipal de Fazenda
Dept. Destino: Depto. de Tributos e Fiscalização
Assunto: ALVARÁ

result

The image features a white piece of paper with a sticker on it. The sticker has a barcode and a label, which reads "Estado do Rio de Janeiro." The paper appears to be a certificate or a document related to the city of Rio de Janeiro. The paper is placed on a table, and the sticker is prominently displayed on the surface.

the correct thing would be
Here is the transcription of the text from the image:

ESTADO DO RIO DE JANEIRO
Prefeitura Municipal de Rio das Ostras
PROTOCOLO GERAL

Processo: 15314 / 2003
Data: 25/07/2003
Hora: 16:18:28

Requerente: COLÔNIA DE PESCADORES Z-22
Sec. Destino: Sec. Mun. Urbanismo Obras e S. Pub.
Dept. Destino: 0
Assunto: AGRADECIMENTO / FAZ

perhaps training on this dataset could help improve accuracy
https://zenodo.org/records/7872951


Wordpress 6.5.3 upgrade

Just upgraded the Wordpress version to 6.5.3
Overview
Upgraded Wordpress version to 6.5.3 as a response to the issue
Rationale
Never and shinier Wordpress.
Juju Events Changes

Module Changes

Library Changes

Checklist

[x] The charm style guide was applied
[x] The contributing guide was applied
[x] The changes are compliant with ISD054 - Managing Charm Complexity
[x] The documentation is generated using src-docs
[x] The documentation for charmhub is updated.
[x] The PR is tagged with appropriate label (urgent, trivial, complex)

I ran the test locally and it seems like there's a problem with the launchpad wordpress plugin. That plugin hasn't been upgraded for a while; I think we might need to put this on hold.
2024-08-01T05:18:13.763Z [wordpress] [Thu Aug 01 05:18:13.763761 2024] [php7:error] [pid 181] [client <IP_ADDRESS>:58743] PHP Fatal error:  Uncaught Error: Call to a member function query() on null in /var/www/html/wp-includes/widgets.php:2090\nStack trace:\n#0 /var/www/html/wp-includes/class-wp-hook.php(324): wp_check_widget_editor_deps()\n#1 /var/www/html/wp-includes/class-wp-hook.php(348): WP_Hook->apply_filters()\n#2 /var/www/html/wp-includes/plugin.php(517): WP_Hook->do_action()\n#3 /var/www/html/wp-content/plugins/openid/common.php(764): do_action()\n#4 /var/www/html/wp-content/plugins/openid/common.php(746): openid_page()\n#5 /var/www/html/wp-content/plugins/openid/consumer.php(59): openid_repost()\n#6 /var/www/html/wp-content/plugins/openid/consumer.php(177): openid_redirect()\n#7 /var/www/html/wp-content/plugins/wordpress-launchpad-integration/openid-launchpad.php(173): openid_start_login()\n#8 /var/www/html/wp-includes/class-wp-hook.php(326): openid_launchpad_wp_authenticate()\n#9 /var/www/html/wp-includes/class-wp-hook.php(348): WP_Hook->apply_filters()\n#10 /var/www/html/wp-includes/plugin.php(565): WP_Hook->do_action()\n#11 /var/www/html/wp-inclu in /var/www/html/wp-includes/widgets.php on line 2090, referer: http://<IP_ADDRESS>/wp-login.php?redirect_to=http%3A%2F%2F<IP_ADDRESS>%2Fwp-admin%2F&reauth=1


[PDI-15018] DI Server security not behaving properly.

Since we have a separate "Schedule content" permission in permission list, we separately check permissions for this operation.
Checkstyle applied
Test written

More details:
AbsSecurityProvider

Added check for scheduling action
Refactored code a little

IAbsSecurityProvider

Added constant for scheduling action
Removed unnecessary public static final on fields, as this is done by default since this are variables of an interface.

RepositoryOperation

Added enums for job and trans scheduling

@akhayrutdinov, could you please review it?

@pamval, @pedrofvteixeira, run WM for this PR please

wingman

Build Commands:
Build:

rm -r ~/.kettle; rm -r ~/.pentaho; echo removed hidden folders
ant -Dpentaho.coding.standards.version=1.0.4 -Djunit.jvmargs=-XX:PermSize=256m -f engine/build.xml clean-all resolve jacoco jacoco-integration checkstyle publish-local
ant -Dpentaho.coding.standards.version=1.0.4 -Djunit.jvmargs=-XX:PermSize=256m -f plugins/pdi-pur-plugin/build.xml clean-all resolve jacoco jacoco-integration checkstyle publish-local

Cleanup:
rm -r ~/.ivy2/local || echo no publish local to remove
Changed files:
engine/src/org/pentaho/di/repository/RepositoryOperation.java
plugins/pdi-pur-plugin/src/org/pentaho/di/repository/pur/AbsSecurityProvider.java
plugins/pdi-pur-plugin/src/org/pentaho/di/ui/repository/pur/services/IAbsSecurityProvider.java
plugins/pdi-pur-plugin/test/org/pentaho/di/repository/pur/AbsSecurityProviderTest.java
Unit test coverage change:
org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta:

branchPercentCoverageChange : +0.53%

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta:

instructionPercentCoverageChange : +0.28%
branchPercentCoverageChange : +3.33%
complexityPercentCoverageChange : +1.85%

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta:

instructionPercentCoverageChange : +0.18%

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta:

instructionPercentCoverageChange : +0.18%

org.pentaho.di.trans.step.StepMeta:

instructionPercentCoverageChange : +0.19%
linePercentCoverageChange : +0.27%

org.pentaho.di.repository.pur.AbsSecurityProvider.new Object() {...}:

linePercentCoverage :100.00%
methodPercentCoverage :100.00%
complexityPercentCoverage :100.00%
instructionPercentCoverage :87.23%

org.pentaho.di.repository.pur.PurRepositoryMeta.new RepositoryCapabilities() {...}:

linePercentCoverage :10.00%
methodPercentCoverage :10.00%
complexityPercentCoverage :10.00%
instructionPercentCoverage :25.00%

org.pentaho.di.repository.pur.UserRoleListDelegate:

instructionPercentCoverageChange : +17.54%
linePercentCoverageChange : +22.73%
complexityPercentCoverageChange : +11.11%
methodPercentCoverageChange : +11.11%

org.pentaho.di.repository.pur.PurRepositorySecurityProvider:

linePercentCoverage :18.00%
methodPercentCoverage :22.22%
complexityPercentCoverage :18.18%
instructionPercentCoverage :19.25%

org.pentaho.di.repository.pur.AbsSecurityProvider:

branchPercentCoverage :80.00%
linePercentCoverage :70.97%
methodPercentCoverage :60.00%
complexityPercentCoverage :63.64%
instructionPercentCoverage :69.66%

org.pentaho.di.repository.pur.AbsSecurityProvider.new ActiveCacheLoader() {...}:

linePercentCoverage :50.00%
methodPercentCoverage :50.00%
complexityPercentCoverage :50.00%
instructionPercentCoverage :46.15%

org.pentaho.di.repository.pur.PurRepositoryMeta:

instructionPercentCoverageChange : +3.55%
linePercentCoverageChange : +3.03%
complexityPercentCoverageChange : +8.33%
methodPercentCoverageChange : +9.09%


[BUG] realtime update not working in jupyter book

Expected Behavior
update chart in realtime
Current Behaviour
not update in time
Reproducible Example
import pandas as pd
from time import sleep
from lightweight_charts import JupyterChart

if __name__ == '__main__':

    chart = JupyterChart()

    df1 = pd.read_csv('ohlcv.csv')
    df2 = pd.read_csv('next_ohlcv.csv')

    chart.set(df1)

    chart.load()

    last_close = df1.iloc[-1]['close']

    for i, series in df2.iterrows():
        chart.update(series)

        if series['close'] > 20 and last_close < 20:
            chart.marker(text='The price crossed $20!')

        last_close = series['close']
        sleep(0.1)

Environment
- OS:Mac OS 15.1
- Library:v2.1

detail:

PyCharm 2024.1.7 (Professional Edition)
Build #PY-241.19416.19, built on October 17, 2024
Licensed to XXX
You have a perpetual fallback license for this version.
Subscription is active until September 5, 2025.
Runtime version: 17.0.12+1-b1207.37 aarch64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
macOS 15.1
GC: G1 Young Generation, G1 Old Generation
Memory: 2048M
Cores: 10
Metal Rendering is ON
Registry:
  debugger.new.tool.window.layout=true
  ide.experimental.ui=true
  ide.images.show.chessboard=true
Non-Bundled Plugins:
  com.intellij.ideolog (<IP_ADDRESS>)
  String Manipulation (9.14.1)
  PlantUML integration (7.11.2-IJ2023.2)
  mobi.hsz.idea.gitignore (4.5.4)
  com.github.copilot (<IP_ADDRESS>13)

https://lightweight-charts-python.readthedocs.io/en/latest/reference/charts.html#JupyterChart
class JupyterChart
The JupyterChart object allows the use of charts within a notebook, and has similar functionality to the Chart object for manipulating data, configuring and styling.
This object only supports the displaying of static data, and should not be used with the update_from_tick or update methods. Every call to the chart object must occur before calling load.


re-invest profits

What are you addressing in this PR

[X] Functionality
[ ] Ease of use
[ ] Bug fix

How have you tested this PR
tested both in test mode and in live mode, however, any mode would be enough here as the change is not directly linked to any particular running mode.
In your own words, please share how this makes the codebase better.
well, it doesn't really. The codebase is quite a mess due to the BVT original repo not having any tests. What this does is change the original behaviour so that the bot re-invests all the profits (and losses) throughout its current session.
so that the bot on a 1% daily return, would return something along the lines of:
https://www.thecalculatorsite.com/dailycompound?a=100&p=1&pp=daily&y=1&m=0&d=0&dr=100&iw=y&dw=y&md=0&rp=monthly&sd=2021-11-29&c=1

Thanks for PR Azul. I'm currently testing it just to make sure it works as expected.
Will let you know how I go!!

I've got a small change to make it work for TRADE_SLOTS > 0 but I can't figure out how to do it in here.
So I'll just merge then update my source and recommit.


create stream: Fix stream email not rendering on stream creation.

Fixes #8817
Testing Plan:
Manual testing
GIFs or Screenshots:

One things is that, this email address is still present in html code(hidden) even after user unsubscribed from stream. This can be security bug in case if stream is private. Do let me know if we should fix this, and remove email address from widget on unsubscription from private streams.

We should remove the widget when a user unsubscribes, yeah.  Even if there's no really an exploit here, we should aim for precisely implementing our security model, and that includes taking away access from a user who is unsubscribed immediately.

(This PR otherwise lgtm; I think when you go about fixing that, but please also test the cases around what happens for an admin who is not subscribed to a private stream).

Thanks for reviewing! It's updated. Now, when user unsubscribe from stream email address gets cleared, from both html widget and sub object in frontend.
About the admin part, I think we don't want to display email address if user is not subscribed, so here admin can be treated as simple user. If admin is subscribed to stream, email address will be visible. If not it will not. In case of unsubscribed private stream listed in admin subscriptions, as admin is not subscribed email address should be visible or hidden in html widgets.

Great, merged, thanks @YJDave!  I opened an issue for another little real-time sync bug that I found while playing with this.


Make UseStatusCodePagesWithReExecute clear Endpoints

#13715 There was a regression when switching to Endpoint based routing. When the status code component tries to re-execute the pipeline there is some state that needs to be cleared first such as the endpoint, otherwise the request will be routed to the old endpoint again.
Normally a middleware would do it's own cleanup (e.g. routing), but that significantly impacted the perf in the case of routing.
In 3.0 @NTaylorMullen Fixed this in UseExceptionHandlerWithReExecute but we missed UseStatusCodePagesWithReExecute.
https://github.com/aspnet/AspNetCore/commit/1f0641f5c0c8552f9605ff782ba1a27c06f4cedf#diff-abb2b51ad2d2d988debc78909b2674d9
FYI: I split the test files for exception handler and status code handler.

Will need @Pilchie approval to take a bug fix in 3.1

@Pilchie @anurse this is ready to merge.

Seems good to me given that we did it for UseExceptionHandlerWithReExecute. I endorse this for 3.1

Agreed.  Approved for 3.1


Electron update to latest stable release

Electron updated to latest stable release.
( the auth-macaroon PR is inside this one, I believe it's not really a problem because it will be merged eventually, let me know if you still want me to separate things)

@joaodealmeida I recommend splitting them up - that way it will be easier to e.g. revert either independently if needed, and will generally keep the history clean and comprehensible.

Yeah I'm going to close this. I'll submit a new PR with the updated electron


add NaN check to temperature and humidity sensors

Proposed Changes

Add a NaN check to the temperature and humidity sensors because they report NaN% or NaN C when unavailable

Related Issues

Related issues or pull requests

whoops forgot about conventional commits


Error with google drive

Hello, I m trying to download a file from google drive but i have an issue. Here is the code
var path = require('path');
var fs = require('fs');
var ytdl = require('..');
var video = ytdl("https://drive.google.com/uc?export=download&id=0B5hBTUDdATCCNmtqM3ItcVAyWGM", ['--output=test.output'], {
cwd: __dirname
});
var size = 0;
video.on('info', function(info) {
'use strict';
size = info.size;
console.log('Got video info');
var file = path.join(__dirname, info._filename);
video.pipe(fs.createWriteStream(file));
});
var pos = 0;
video.on('data', function data(chunk) {
'use strict';
pos += chunk.length;
if (size) {
    var percent = (pos / size * 100).toFixed(2);
    process.stdout.cursorTo(0);
    process.stdout.clearLine(1);
    process.stdout.write(percent + '%');
}

});
video.on('end', function end() {
'use strict';
console.log('\nDone');
});
It appears output options is not sent to youtube-dl exe. Any help is appreciated. Regards

I think this is a youtube-dl thing more than this specific Node.js connector.
Can you try again with the latest connector/youtube-dl version?


Can't retrieve messages, always returns 0

Hello
No matter what I try , I always get a zero message count.
I've tried the standard sample. I've tried $folder->query()->all()->count() , I've tried $folder->query()->all()->get()->first() I've tried with getSubject() in foreach .... nothing seems to work.
When I foreach trough folders and echo $folder->name , I get "INBOX" as result
ANy starter advise to get this going ? Much appreciated !!

Same problem here.
When I'm just doing a simple:
        $client->checkConnection();
        $folder = $client->getFolderByPath($path);
        $messages = $folder->messages()->all()->get();
        return var_dump($messages);

The result is the following:
object(Webklex\PHPIMAP\Support\MessageCollection)#314 (3) {
  ["items":protected]=>
  array(0) {
  }
  ["escapeWhenCastingToString":protected]=>
  bool(false)
  ["total":protected]=>
  int(5)
}

The "total" amount is correct, I have 5 mails in this folder. Although I don't get any info regarding the actual messages.
I'm on Laravel 9 with the latest laravel-imap from composer, IMAP server is plain 'ol Dovecot.

I seem to have the same issue created over at the php-imap repository: https://github.com/Webklex/php-imap/issues/341
Just leaving this here as reference if it gets resolved at one place or the other.

I am reposting my solution from the php-imap repository here as well, maybe it helps:
After diving into the libraries code myself I figured out the issue. In my configuration file, for some reason the sequence-option was set to \Webklex\PHPIMAP\IMAP::ST_MSGN instead of \Webklex\PHPIMAP\IMAP::ST_UID. After changing it, the MessageCollection contained the items as expected.

Thanks for figuring it out, @manraf !
I'm not sure though if I will continue using this library. As you can see for yourself, the support is underwhelming - for two months we haven't got a single response in this issue from a maintainer and there are only like 10 open issues.

I am reposting my solution from the php-imap repository here as well, maybe it helps:
After diving into the libraries code myself I figured out the issue. In my configuration file, for some reason the sequence-option was set to \Webklex\PHPIMAP\IMAP::ST_MSGN instead of \Webklex\PHPIMAP\IMAP::ST_UID. After changing it, the MessageCollection contained the items as expected.

Really big thanks! I had the same issue and it helped!

Big thanks @manraf. You really saved my day!


round function's behavior?

The round function implemented in the major RDBMS seems to do rounding, is rounding ideal for the behavior of the PRQL's round function?
In other words, I am wondering if the behavior of the round function should be changed to bankers' rounding.
In general, rounding is very easy to implement on the user side compared to bankers' rounding, since all we have to do is add 0.5 and trunc that.
Related duckdb's issue: duckdb/duckdb#2348

(no strong view on this end. Unless anyone has a strong view, I would tend to delegate to the DB's implementation until we have one...)

From that DuckDB thread:

Banker's rounding is more numerically stable (https://stackoverflow.com/questions/45223778/is-bankers-rounding-really-more-numerically-stable)

In particular from that SO thread, looks like Banker's rounding has no "bias" (in the statistical sense) but slightly larger variance (no idea why that might be).

Sounds like a good idea to default to Banker's rounding then. There should probably be an option/different function to default to the RDBMs target default though for consistency with existing SQL code.

There should probably be an option/different function to default to the RDBMs target default though for consistency with existing SQL code.

Perhaps simply using an S-string s"round()" would suffice. (If not, we may have to provide two versions of every function.)

As discussed on dev call, the tentative proposal is:

Implement round as "bankers rounding"
Possibly implement round_common too
Don't add any of the others

@max-sixty I will like to take this up if someone else is not already working on it?

That sounds great @varunu28
We'd want to write functions such as https://github.com/PRQL/prql/blob/7c510444ce8fdbe4a53b6102095276f9ecfb3c1d/prqlc/prqlc/src/sql/std.sql.prql#L330-L335 for dialects that don't do bankers' rounding by default, and confirm tests such as https://github.com/PRQL/prql/blob/3dcace736ca74d4e273b65026434411bb22b2cc1/prqlc/prqlc/tests/integration/queries/arithmetic.prql#L28 work with this.
Always feel free to ask any questions / push partially complete code...


add lnc attached wallet

This

adds a litd container to sndev for testing
implements lnc as an attached wallet

This works but still has a few issues:

wasm loading is wonky ... in strictMode, lnc.connect() will be called twice which causes an infinite loop in wasm seemingly

I need to investigate how to better handle this and only conditionally load the wasm

I'm not yet sure how to make sure permissions are limited (so that we can only pay invoices) on the LNC connection
lnc encrypts the keys with a stacker provided password before storing in the browser

alby hardcodes a password to deal with this but we probably give stackers the option to encrypt the storage
so for stackers that want encryption, I'll need a UI for prompting for the password to spend or to display the pairing phrase in a form
we have enough webln providers and shared code that it's time for a refactor/improvement to the abstraction .. I'll take stab at it possibly

Regarding passwords:
I envisioned that we stay close to bitcoin wallet UI/UX so we wouldn't prompt for a password but simply generate a BIP39 mnemonic for them that they should backup. I would prefer to avoid user-generated passwords. I see them as security holes since humans are bad at being random. Since it should be a familiar experience for most stackers, I think it would also meet expectations and thus be good UX. That's also not only what bitcoin wallets do, that's also what other websites with E2EE do (e.g. Matrix, RocketChat).
We could let stackers generate the optional BIP39 passphrase.
We could also use this passphrase for everything, including E2EE stuff (#56, #519) in the future.

so for stackers that want encryption, I'll need a UI for prompting for the password to spend or to display the pairing phrase in a form

Is there no such concept as unlocking a wallet? This sounds like we need to prompt for the password for every payment. Is that true?
I envision unlocking in a browser context to work like loading decrypted content from localStorage into memory. There are security implications to consider: global variable vs only available inside a closure vs web worker etc.

This sounds like we need to prompt for the password for every payment. Is that true?

No, just once for each session.


Initialize test github actions

Describe the task
depends on: #112, #73, #72

jest
e2e
backend

And configure the GH repo to have these as status checks on thedevelop branch
Acceptance Criteria

[ ] first
[ ] second
[ ] third

Additional context

Add any other context about the task here.
Or here

Closing, this is not applicable with the monorepo


updated PTR task on  new version of PTR library

Release Logs are available here : https://dev.azure.com/mseng/AzureDevOps/_releaseProgress?releaseId=13035328&environmentId=148848611&_a=release-environment-logs

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

Please check in the changes in the task.loc.json as well.

Let's change the task version to 2.160.2 otherwise it will be hard to track in telemetry.


update readme.md to include installation in beginning

I really thought that there was no installation procedure, since its placed way down. I hope this is a logical change.

thanks!


Compound objects and item page refresh

Adds compound objects options for "compound_object" and "multiple" display template, as well as "panorama" option.

I would like to swap out object examples with public domain items

@owikle you want to sign off your approval? then I'll merge the thing!

@owikle did you want to add your updated compound metadata?

I am merging this so we can finally start working on other stuff!!

@evanwill can't believe you merged this. I had dibs. See https://github.com/CollectionBuilder/collectionbuilder-csv/pull/36#issuecomment-1540909552
I HAD DIBS!!!

eh. I had work to do on it, sick of waiting!

p.s. @dcnb you could always go finish up the cb-docs documentation and merge that if you need that excitement!


WIP | user auth

Fix #196

To simplify the user experience, we're going to support service account authentication only. Service accounts are the official recommended way of testing in CI. There isn't a strong value proposition for adding an additional auth mechanism that doesn't work in CI.


[20.03] shotcut: 19.12.31 -> 20.09.13, mlt: 6.18.0 -> 6.22.1

Motivation for this change
fixes #99851
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

See #100753 for a minimal invasive solution.


Rebuild artifacts with 3 years of deadline

Windows and macOS might not work, but that's the best we can do


Added #fetchMoreResults analytics event

Needed for KIT-170.

Codecov Report

Merging #225 (5b161f8) into master (762ebef) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #225      +/-   ##
==========================================
+ Coverage   89.69%   89.71%   +0.02%
==========================================
  Files          27       27
  Lines         883      885       +2
  Branches      117      117
==========================================
+ Hits          792      794       +2
  Misses         59       59
  Partials       32       32

Impacted Files
Coverage Δ

src/searchPage/searchPageClient.ts
97.36% <100.00%> (+0.04%)
:arrow_up:

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 762ebef...5b161f8. Read the comment docs.


Symbol lookup for plugin fails on iOS with static linkage

In order to use a Google MLKit plugin in my Flutter App on iOS, I had to set :linkage => :static in my Podfile. After doing so, the webcrypto plugin (which worked fine before) fails to lookup native boringssl symbols at runtime (it uses DynamicLibrary.executable() here).
Stack trace (in case it helps):
flutter: Unsupported operation: package:webcrypto cannot be used from scripts or `flutter test` unless `flutter pub run webcrypto:setup` has been run for the current root project.
flutter: #0      lookup.<anonymous closure> (package:webcrypto/src/boringssl/lookup/lookup_symbol_flutter.dart:47:5)
flutter: #1      lookup (package:webcrypto/src/boringssl/lookup/lookup_symbol_flutter.dart:53:2)
flutter: #2      lookup (package:webcrypto/src/boringssl/lookup/lookup_symbol_flutter.dart)
flutter: #3      _cachedLookup (package:webcrypto/src/boringssl/lookup/lookup.dart:26:21)
flutter: #4      _cachedLookup (package:webcrypto/src/boringssl/lookup/lookup.dart)
flutter: #5      ssl (package:webcrypto/src/boringssl/lookup/lookup.dart:29:44)
flutter: #6      ssl (package:webcrypto/src/boringssl/lookup/lookup.dart)
flutter: #7      _aesCbcEncryptOrDecrypt (package:webcrypto/src/impl_ffi/impl_ffi.aescbc.dart:41:52)

So the original exception seems to appear here.
I already tried setting the Strip-Style to "Non-Global Symbols" in Xcode, as suggested in the docs, but it doesn't make a difference.
Note: I also created an issue in the webcrypto plugin itself, but I'm not sure where it rather belongs and it currently blocks the release of our application, which is why I need to find a way to solve it fast. Hopefully here it's visible to more experts on this (I'm a noob with anything related to ffi on ios).
I'd really appreciate any help on this, as I'm completely stuck here. Please let me know if I need to provide more information.

I can reproduce your issue by changing flutter_app/ios/Podfile:
target 'Runner' do
  use_frameworks! :linkage => :static
  use_modular_headers!

  flutter_install_all_ios_pods File.dirname(File.realpath(__FILE__))
end

For a FFI plugin to support static linking, I need to meddle with linker flags to avoid stripping symbols. For example in mylib_staticlib/ios/mylib_staticlib.podspec:
  s.pod_target_xcconfig = {
    'DEFINES_MODULE' => 'YES',
    'EXCLUDED_ARCHS[sdk=iphonesimulator*]' => 'i386',
    "OTHER_LDFLAGS[sdk=iphoneos*]" => "-force_load $(PODS_TARGET_SRCROOT)/Frameworks/mylib_staticlib.xcframework/ios-arm64_armv7/libmylib_staticlib.a",
    "OTHER_LDFLAGS[sdk=iphonesimulator*]" => "-force_load $(PODS_TARGET_SRCROOT)/Frameworks/mylib_staticlib.xcframework/ios-arm64_x86_64-simulator/libmylib_staticlib.a",
  }
  s.vendored_frameworks = 'Frameworks/mylib_staticlib.xcframework'

However, when building from source, we just depend on s.source_files in mylib_source/ios/mylib_source.podspec:
  s.source_files = 'Classes/**/*'
  s.dependency 'Flutter'
  s.platform = :ios, '10.0'

  s.pod_target_xcconfig = {
    'DEFINES_MODULE' => 'YES',
    'EXCLUDED_ARCHS[sdk=iphonesimulator*]' => 'i386'
  }

And the lookup can be done in DynamicLibrary.open('$_libName.framework/$_libName') rather than DynamicLibrary.executable() which can fail when there's duplicate symbols in different frameworks.
This is the setup used in package:webcrypto but it still uses DynamicLibrary.executable(). If it would have been using DynamicLibrary.open('$_libName.framework/$_libName') that step would fail instead of trying to lookup the symbol.
If we were to add those linker flags, the linking would fail without :linkage => :static. So, I'm not sure which route to go here. We would need to investigate how to make cocoapods packages that work both with static and dynamic linking and when used with static linking we want to ensure the symbols are not stripped out.
(As a side note, we would in Dart code also need to fallback to DynamicLibrary.executable().)


Update adopters list

Q
A

Bug fix?
no

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets
n/a

License
Apache 2.0

What's in this PR?
Adds ViaBill to the list of adopters. I've also allowed myself to alphabetize the list, as suggested in the first line ;-)
Why?
We're using bank-vaults every day and we love it. Thanks for your effort!

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


ARROW-4112: [Packaging] Add support for Gandiva .deb

https://github.com/kou/crossbow/branches/all?utf8=✓&query=build-58

+1
I'll merget his.


cargo-nextest: 0.9.37 -> 0.9.38

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/cargo-nextest/versions.
meta.description for cargo-nextest is: "Next-generation test runner for Rust projects"
meta.homepage for cargo-nextest is: "https://github.com/nextest-rs/nextest"
meta.changelog for cargo-nextest is: ""
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.9.38 with grep in /nix/store/ki8p6isg5nmmshvkcs3y6afihq4rxshx-cargo-nextest-0.9.38
found 0.9.38 in filename of file in /nix/store/ki8p6isg5nmmshvkcs3y6afihq4rxshx-cargo-nextest-0.9.38

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cargo-nextest

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ki8p6isg5nmmshvkcs3y6afihq4rxshx-cargo-nextest-0.9.38 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cargo-nextest https://github.com/r-ryantm/nixpkgs/archive/f7a27972b51361231fea2aa1d2efd7dd78cbcd2b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ki8p6isg5nmmshvkcs3y6afihq4rxshx-cargo-nextest-0.9.38
ls -la /nix/store/ki8p6isg5nmmshvkcs3y6afihq4rxshx-cargo-nextest-0.9.38/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cargo-nextest

Maintainer pings
cc @Ekleog for testing.

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

1 package built:

cargo-nextest


How can I allow mic/camera and block notifications permissions using EdgeOptions? Selenium 1.141

No more reproducible

Edge WebDriver and ChromeDriver both support the standard W3C WebDriver Set Permission command. Please check the docs for the version of Selenium you are using to see how to call this command.


[Event Request] codeunit 816 "Purch. Post Invoice" - RunOnBeforePrepareDeferralLine

Describe the request
Hello, can you please add a new Event RunOnBeforePrepareDeferralLine in codeunit 816 "Purch. Post Invoice"?
    local procedure PrepareDeferralLine(PurchHeader: Record "Purchase Header"; PurchLine: Record "Purchase Line"; AmountLCY: Decimal; AmountACY: Decimal; RemainAmtToDefer: Decimal; RemainAmtToDeferACY: Decimal; DeferralAccount: Code[20]; PurchAccount: Code[20])
    var
        DeferralTemplate: Record "Deferral Template";
        DeferralPostingBuffer: Record "Deferral Posting Buffer";
        IsHandled: Boolean;
    begin
        IsHandled := false;
        PurchPostInvoiceEvents.RunOnBeforePrepareDeferralLine(TempDeferralHeader, TempDeferralLine, PurchHeader, PurchLine, AmountLCY, AmountACY, RemainAmtToDefer, RemainAmtToDeferACY, DeferralAccount, PurchAccount, InvoicePostingParameters."Document No.", InvDefLineNo, IsHandled);
        if IsHandled then
            exit;
        DeferralTemplate.Get(PurchLine."Deferral Code");

        if TempDeferralHeader.Get(
            "Deferral Document Type"::Purchase, '', '', PurchLine."Document Type", PurchLine."Document No.", PurchLine."Line No.")
        then begin
            if TempDeferralHeader."Amount to Defer" <> 0 then begin
                DeferralUtilities.FilterDeferralLines(
                  TempDeferralLine, "Deferral Document Type"::Purchase.AsInteger(), '', '',
                  PurchLine."Document Type".AsInteger(), PurchLine."Document No.", PurchLine."Line No.");

codeunit 826 "Purch. Post Invoice Events"
.
.
.
    procedure RunOnCalculateVATAmountsAfterVatCalculationType(var TempInvoicePostingBuffer: Record "Invoice Posting Buffer" temporary; var PurchHeader: Record "Purchase Header"; VatPostingSetup: Record "VAT Posting Setup")
    begin
        OnCalculateVATAmountsAfterVatCalculationType(TempInvoicePostingBuffer, PurchHeader, VatPostingSetup);
    end;

    [IntegrationEvent(false, false)]
    local procedure OnCalculateVATAmountsAfterVatCalculationType(var TempInvoicePostingBuffer: Record "Invoice Posting Buffer" temporary; var PurchHeader: Record "Purchase Header"; VatPostingSetup: Record "VAT Posting Setup")
    begin
    end;

Additional context
We want to use our own logic for the PrepareDeferralLine function.
Internal work item: AB#550247

Availability update: We will publish a fix for this issue in the next update for release 25.
Build ID to track: 26491.


SAA-727 - Handle alerts updated event

Tested via localstack

A couple of things:

Can you add integration test coverage in the InboundEventsIntegrationTest please.
Is there a corresponding cloud platform PR to subscribe to the new inbound domain event?  Without this it won't work as we won't actually be listening for it.

Codecov Report
Patch coverage: 88.88% and no project coverage change.

Comparison is base (c776f51) 85.20% compared to head (a9133d8) 85.21%.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #506   +/-   ##
=========================================
  Coverage     85.20%   85.21%
- Complexity     2347     2348    +1
=========================================
  Files           294      294
  Lines          8592     8601    +9
  Branches        545      546    +1
=========================================
+ Hits           7321     7329    +8
- Misses          994      995    +1
  Partials        277      277

Files Changed
Coverage Δ

...itiesmanagementapi/service/events/InboundEvents.kt
68.75% <87.50%> (+2.08%)
:arrow_up:

...service/events/handlers/InterestingEventHandler.kt
80.95% <100.00%> (+0.46%)
:arrow_up:

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


Make RecipeBook accept "search" command

Resolved in #78

And implemented the whole command while I was at it


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_4c8_vht5bf/clone/recipe/meta.yaml:18: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_4c8_vht5bf/clone/recipe/meta.yaml:23: missing_wheel: For pypi packages, wheel should be present in the host section
WARNING: /tmp/abs_4c8_vht5bf/clone/recipe/meta.yaml:28: python_build_tool_in_run: The python build tool setuptools is in run depends
WARNING: /tmp/abs_4c8_vht5bf/clone/recipe/meta.yaml:46: missing_description: The recipe is missing a description
ERROR: /tmp/abs_4c8_vht5bf/clone/recipe/meta.yaml:46: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_4c8_vht5bf/clone/recipe/meta.yaml:46: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_4c8_vht5bf/clone/recipe/meta.yaml:46: missing_license_family: The recipe is missing the about/license_family key.
Errors were found


BadgeNotification not working

Describe the bug
Please see attached boiler plate repo: clicking the button should show a badge on the taskbar icon ... but doesn't.

Windows 11 with latest updates
Settings -> System -> Personalization -> Taskbar -> Taskbar behaviors -> Show badges (...) is checked
WindowsAsppSdk 1.1.0

Badge.zip
Steps to reproduce the bug
Please see above
Expected behavior
Badges should work
Screenshots
No response
NuGet package version
No response
Packaging type
Packaged (MSIX)
Windows version
Windows 11 version 21H2 (22000)
IDE
Visual Studio 2022
Additional context
No response

We appreciate the bug report! Since this is a Windows platform bug and not a Windows App SDK bug, I've filed a bug with the platform team (internal bug #39959734) and asked them to follow up here with any updates. Closing this bug since this isn't an active issue with Windows App SDK itself.


Refactor Allocator

[x] ArrayList
[ ] Deque

Coverage increased (+0.2%) to 71.993% when pulling 3c128c52c8c7aa765d54dc6e7aa61827a23dd037 on Allocator into bcfe1479d8ab96b80cd079a8264833375ed28843 on dev.


New command: pa app permission ensure

Usage
m365 pa app permission ensure [options]
Description
Assigns/updates permissions to a Power Apps app
Options

Option
Description

-e, --environmentName [environmentName]
The name of the environment. Specify only a value when running the command as admin.

--appName <appName>
The name (GUID) of the Microsoft Power App.

--roleName <roleName>
Permission level given to the app. Valid values: CanEdit, CanView. Sharing with the entire tenant is only supported with CanView role.

--userId [userId]
The ID of the user. Specify either userId, userName, groupId, groupName or tenant.

--userName [userName]
User principal name of the user. Specify either userId, userName, groupId, groupName or tenant.

--groupId [groupId]
The ID of the group. Specify either userId, userName, groupId, groupName or tenant.

--groupName [groupName]
The name of the group. Specify either userId, userName, groupId, groupName or tenant.

--tenant
Share the app with the entire tenant. When specifying this, only CanView role is supported.

--asAdmin
Run the command as admin for apps you don't own.

--sendInvitationMail
When set, an invitation mail will be sent.

Examples
Share a Power App app with another user with run only permissions
m365 pa app permission ensure --appName bc9f0a7e-53df-46af-b669-5888bb2f63d0 --userName<EMAIL_ADDRESS>--roleName CanView --sendInvitationMail

Add co-owner permissions for a security group to a Power Apps app
m365 pa app permission ensure --appName bc9f0a7e-53df-46af-b669-5888bb2f63d0 --groupName Developers --roleName CanEdit

Add permissions as admin to a Power App app that you do not own
m365 pa app permission ensure --environment Default-e535e809-c6a1-47b5-8951-d0898a64bf8a --appName bc9f0a7e-53df-46af-b669-5888bb2f63d0 --userId 42a28ec6-972f-4fdd-b483-50ee4b237f91 --roleName CanEdit --asAdmin

Share a Power Apps app with everyone in the tenant
m365 pa app permission ensure --appName bc9f0a7e-53df-46af-b669-5888bb2f63d0 --roleName CanView --tenant

Default properties
No response
Additional Info
This is  a suggestion from our Discord community server.
Docs
Remarks:

Only security groups can be used to share apps with

API request
When app is owned by the user
POST https://api.powerapps.com/providers/Microsoft.PowerApps/apps/<APP ID>/modifyPermissions?api-version=2022-11-01
{
    "put": [
        {
            "properties": {
                "principal": {
                    "id": "42a28ec6-972f-4fdd-b483-50ee4b237f91", // ID of user, group or tenant
                    "type": "User" // Supported values: User, Group, Tenant
                },
                "NotifyShareTargetOption": "Notify", // Supported values: DoNotNotify, Notify
                "roleName": "CanView" // Supported values: CanView, CanEdit
            }
        }
    ]
}

When running as admin
POST https://api.powerapps.com/providers/Microsoft.PowerApps/scopes/admin/environments/<Environment ID>/apps/<App ID>/modifyPermissions?api-version=2022-11-01
{
    "put": [
        {
            "properties": {
                "principal": {
                    "id": "42a28ec6-972f-4fdd-b483-50ee4b237f91", // ID of user, group or tenant
                    "type": "User" // Supported values: User, Group, Tenant
                },
                "NotifyShareTargetOption": "Notify", // Supported values: DoNotNotify, Notify
                "roleName": "CanView" // Supported values: CanView, CanEdit
            }
        }
    ]
}

looks solid 👍. Let's wait for the Discord votes outcome 👍

Since this issue originates from a Discord thread, I'm labeling it again as Feature request

Looks great. I'd say: move the environmentName option down below asAdmin and add the 'Specify either...' message for tenant as well.

Thx for the review @martinlingstuyl!

Can I work on this?

Definitely!


Publish asset: btc/usdt on Ocean CEXA, into Polygon

[x] Publish asset: btc/usdt on Ocean CEXA, into Polygon. Example: eth/usdt on Ocean CEXA.
[ ] Update eth-predict2.md to also load it

Parent epic: https://github.com/oceanprotocol/predict-eth/issues/3

The CEXA asset is published here: https://market.oceanprotocol.com/asset/did:op:b4584760f5133b27d91c337b9c10b56448b84a1bae39b8c1037d0de33023b4dc
The data retrieved by the CEXA is currently not what we need, the PR to fix this is available here: https://github.com/oceanprotocol/cex_aggregator/pull/2

@juanpablovs We fixed the cexa aggregator and now the asset: https://market.oceanprotocol.com/asset/did:op:b4584760f5133b27d91c337b9c10b56448b84a1bae39b8c1037d0de33023b4dc returns the latest 1000 values/hour.
Please take this issue over and update updated the eth-predict2 with this new asset


Not working on web

when I press the button of scan document . it came this error . I am using mac book pro intel chip. and run flutter web on both safari and chrome, same error occurred .
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36

Currently, this supports only Android and ios.


Merge master to live

:warning: Validation status: warnings

File
Status
Preview URL
Details

:warning:Warning

Details

docs/docfx.json
:white_check_mark:Succeeded

docs/how-to-add-a-comment-to-a-slide-in-a-presentation.md
:white_check_mark:Succeeded

docs/how-to-apply-a-theme-to-a-presentation.md
:white_check_mark:Succeeded

docs/how-to-change-the-fill-color-of-a-shape-in-a-presentation.md
:white_check_mark:Succeeded

docs/how-to-delete-a-slide-from-a-presentation.md
:white_check_mark:Succeeded

docs/how-to-delete-all-the-comments-by-an-author-from-all-the-slides-in-a-presentatio.md
:white_check_mark:Succeeded

docs/how-to-get-all-the-external-hyperlinks-in-a-presentation.md
:white_check_mark:Succeeded

docs/how-to-get-all-the-text-in-a-slide-in-a-presentation.md
:white_check_mark:Succeeded

docs/how-to-get-the-titles-of-all-the-slides-in-a-presentation.md
:white_check_mark:Succeeded

docs/how-to-get-worksheet-information-from-a-package.md
:white_check_mark:Succeeded

docs/how-to-insert-a-chart-into-a-spreadsheet.md
:white_check_mark:Succeeded

docs/how-to-insert-a-new-slide-into-a-presentation.md
:white_check_mark:Succeeded

docs/how-to-insert-a-picture-into-a-word-processing-document.md
:white_check_mark:Succeeded

docs/how-to-move-a-paragraph-from-one-presentation-to-another.md
:white_check_mark:Succeeded

docs/how-to-move-a-slide-to-a-new-position-in-a-presentation.md
:white_check_mark:Succeeded

docs/how-to-open-a-presentation-document-for-read-only-access.md
:white_check_mark:Succeeded

docs/how-to-remove-a-document-part-from-a-package.md
:white_check_mark:Succeeded

docs/how-to-remove-hidden-text-from-a-word-processing-document.md
:white_check_mark:Succeeded

docs/how-to-replace-the-header-in-a-word-processing-document.md
:white_check_mark:Succeeded

docs/how-to-retrieve-a-list-of-the-hidden-worksheets-in-a-spreadsheet.md
:white_check_mark:Succeeded

docs/how-to-retrieve-comments-from-a-word-processing-document.md
:white_check_mark:Succeeded

docs/open-xml-docs/manifest.json
:white_check_mark:Succeeded

docs/open-xml-docs/media/Octocat.png
:white_check_mark:Succeeded

docs/open-xml-docs/media/OpenXMLCon_HowToConvertDOCMtoDOCX_Fig1.gif
:white_check_mark:Succeeded

docs/open-xml-docs/media/OpenXmlCon_CreateCharacterStyle_Fig1.gif
:white_check_mark:Succeeded

docs/open-xml-docs/media/OpenXmlCon_HowToReplaceStyles_Fig1.gif
:white_check_mark:Succeeded

docs/open-xml-docs/media/OpenXmlCon_HowToSetCustomPropertyFig2.gif
:white_check_mark:Succeeded

docs/open-xml-docs/media/OpenXmlCon_HowToSetCustomProperty_Fig1.gif
:white_check_mark:Succeeded

docs/open-xml-docs/media/a-theme01.gif
:white_check_mark:Succeeded

docs/open-xml-docs/media/odc_oxml_ppt_documentstructure_fig01.jpg
:white_check_mark:Succeeded

docs/open-xml-docs/media/odc_oxml_ppt_documentstructure_fig02.jpg
:white_check_mark:Succeeded

docs/open-xml-docs/media/odc_oxml_wd_documentstructure_fig01.jpg
:white_check_mark:Succeeded

docs/open-xml-docs/media/odc_oxml_xl_documentstructure_fig01.gif
:white_check_mark:Succeeded

docs/open-xml-docs/media/odc_oxml_xl_documentstructure_fig02.gif
:white_check_mark:Succeeded

docs/open-xml-docs/media/w-comment01.gif
:white_check_mark:Succeeded

docs/open-xml-docs/media/w-delparagraphs01.gif
:white_check_mark:Succeeded

docs/open-xml-docs/media/w-delparagraphs02.gif
:white_check_mark:Succeeded

docs/open-xml-docs/media/w-headerref01.gif
:white_check_mark:Succeeded

docs/open-xml-docs/xrefmap.yml
:white_check_mark:Succeeded

docs/open-xml-sdk.md
:white_check_mark:Succeeded

docs/structure-of-a-presentationml-document.md
:white_check_mark:Succeeded

docs/structure-of-a-spreadsheetml-document.md
:white_check_mark:Succeeded

docs/what-s-new-in-the-open-xml-sdk.md
:white_check_mark:Succeeded

docs/working-with-animation.md
:white_check_mark:Succeeded

docs/working-with-comments.md
:white_check_mark:Succeeded

docs/working-with-conditional-formatting.md
:white_check_mark:Succeeded

docs/working-with-handout-master-slides.md
:white_check_mark:Succeeded

docs/working-with-notes-slides.md
:white_check_mark:Succeeded

docs/working-with-paragraphs.md
:white_check_mark:Succeeded

This comment lists only the first 50 files in the pull request.

[Warning] Unable to handle following files: open-xml-docs/xrefmap.yml. Do they miss YamlMime as the first line of file, e.g.: ### YamlMime:ManagedReference?

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


The ASSERT condition was violated

When I run the report It crashes and shows the following error in row 25418

The ASSERT condition was violated

I am getting the same exact error. This happens at line 25418 because table ZABAPGIT is empty.

thanks, I'll have a look, will most likely fix it within an hour or two

@a13marquez @nununo can you try again, make sure to get the latest version from http://larshp.github.io/abapGit/build/zabapgit.txt
thanks for the bug report :+1:

Hi Lars,
Thanks for you quick action.
Unfortunately the problem remains in v1.16.2...
At line 25418:
IF mv_show IS INITIAL.
READ TABLE lt_repos INTO lo_repo INDEX 1.

 ASSERT sy-subrc = 0.

On Wed, Jul 20, 2016 at 4:48 PM, Lars Hvam<EMAIL_ADDRESS>wrote:

@a13marquez https://github.com/a13marquez @nununo
https://github.com/nununo can you try again, make sure to get the
latest version from http://larshp.github.io/abapGit/build/zabapgit.txt
thanks for the bug report 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/larshp/abapGit/issues/292#issuecomment-233991959, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAgeUTJdC9y0bqyJjAgFcU443PT4C35Nks5qXkNggaJpZM4JQ0_v
.

yeah, will try again :) my user has the mv_show set, so I fixed the wrong case(but also a possible dump)

Hi @larshp an thanks for your help but the same that happens to  @nununo happens to me

try again, new Version released :smile:

It works for me @larshp. Thanks a lot

cool, feel free to open a new issue if you find more bugs, or have any questions/comments/suggestions
closing issue

Perfect. It works now for me also. Thanks @larshp !


FLEX-1803_Rename_Dto_Classes_Using_Dto_Postfix

 You should sign our Contributor License Agreement in order to get your pull request merged.

I tested this version of the code using my dev-environment.
Everything looks good.

SonarQube analysis reported 59 issues

 53 major
 2 minor
 4 info

Watch the comments in this conversation to review them.
Top 10 extra issues
Note: The following issues were found on lines that were not modified in the pull request. Because these issues can't be reported as line comments, they are summarized here:

 DeviceManagementService.java#L83: Method has 8 parameters, which is greater than 7 authorized.
 DeviceManagementService.java#L147: Method has 9 parameters, which is greater than 7 authorized.
 DeviceRegistrationService.java#L94: The Cyclomatic Complexity of this method "checkDeviceRandomAndPlatformRandom" is 11 which is greater than 10 authorized.
 OslpChannelHandlerServer.java#L126: Remove those useless parentheses.
 OslpDeviceService.java#L77: The Cyclomatic Complexity of this class is 226 which is greater than 200 authorized.
 OslpDeviceService.java#L539: Method has 10 parameters, which is greater than 7 authorized.
 OslpDeviceService.java#L566: Method has 10 parameters, which is greater than 7 authorized.
 OslpDeviceService.java#L567: Remove this unused method parameter "ipAddress".
 OslpDeviceService.java#L631: Add a nested comment explaining why this method is empty, throw an UnsupportedOperationException or complete the implementation.
 OslpDeviceService.java#L1466: The Cyclomatic Complexity of this method "handleOslpResponseGetStatus" is 26 which is greater than 10 authorized.

All is well.


fix: Corrected NGINX forward rules for both rules engines

PR Checklist
Please check if your PR fulfills the following requirements:

[X] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[X] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?)
[X] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions

https://localhost:8443/app-rules-engine/api/v2/ping should get the normal EdgeX ping response
https://localhost:8443/rules-engine/rules should get a empty Javascript array

Previously, both of these urls returned 404's
New Dependency Instructions (If applicable)

Codecov Report

Merging #4420 (9da3e52) into main (58b8fd4) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main    #4420   +/-   ##
=======================================
  Coverage   41.76%   41.76%
=======================================
  Files         106      106
  Lines        9740     9740
=======================================
  Hits         4068     4068
  Misses       5325     5325
  Partials      347      347

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