undefined method `job_status_circle_class' for an instance of

I decided to try out the gem but when i try to enter the dashboard i get this error:
undefined method `job_status_circle_class' for an instance of #Class:0x00007f770a223e80
10:47:58 web.1  |
10:47:58 web.1  |
10:47:58 web.1  |
10:47:58 web.1  | ActionView::Template::Error (undefined method `job_status_circle_class' for an instance of #<Class:0x00007f770a223e80>):
10:47:58 web.1  |
10:47:58 web.1  | Causes:
10:47:58 web.1  | NoMethodError (undefined method `job_status_circle_class' for an instance of #<Class:0x00007f770a223e80>)
10:47:58 web.1  |     18:

10:47:58 web.1  |     19:

10:47:58 web.1  |     20:           <%= status.to_s.titleize %>
10:47:58 web.1  |     21:           <span class="ml-0.5 -translate-y-px circle <%= job_status_circle_class(status) %>">
10:47:58 web.1  |     22:

10:47:58 web.1  |     23:

10:47:58 web.1  |     24:           <%= number_with_delimiter(@jobs.with_status(status).count) %>
10:47:58 web.1  |
10:47:58 web.1  | solid_queue_dashboard (0.2.0) app/views/solid_queue_dashboard/dashboard/index.html.erb:21
10:47:58 web.1  | solid_queue_dashboard (0.2.0) app/views/solid_queue_dashboard/dashboard/index.html.erb:16:in `each'
10:47:58 web.1  | solid_queue_dashboard (0.2.0) app/views/solid_queue_dashboard/dashboard/index.html.erb:16
10:47:58 web.1  | actionview (7.2.1) lib/action_view/base.rb:281:in `public_send'
10:47:58 web.1  | actionview (7.2.1) lib/action_view/base.rb:281:in `_run'
10:47:58 web.1  | actionview (7.2.1) lib/action_view/template.rb:275:in `block in render'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/notifications.rb:212:in `instrument'
10:47:58 web.1  | actionview (7.2.1) lib/action_view/template.rb:574:in `instrument_render_template'
10:47:58 web.1  | actionview (7.2.1) lib/action_view/template.rb:263:in `render'
10:47:58 web.1  | actionview (7.2.1) lib/action_view/renderer/template_renderer.rb:66:in `block (2 levels) in render_template'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/notifications.rb:210:in `block in instrument'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/notifications/instrumenter.rb:58:in `instrument'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/notifications.rb:210:in `instrument'
10:47:58 web.1  | actionview (7.2.1) lib/action_view/renderer/template_renderer.rb:60:in `block in render_template'
10:47:58 web.1  | actionview (7.2.1) lib/action_view/renderer/template_renderer.rb:76:in `block in render_with_layout'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/notifications.rb:210:in `block in instrument'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/notifications/instrumenter.rb:58:in `instrument'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/notifications.rb:210:in `instrument'
10:47:58 web.1  | actionview (7.2.1) lib/action_view/renderer/template_renderer.rb:75:in `render_with_layout'
10:47:58 web.1  | actionview (7.2.1) lib/action_view/renderer/template_renderer.rb:59:in `render_template'
10:47:58 web.1  | actionview (7.2.1) lib/action_view/renderer/template_renderer.rb:11:in `render'
10:47:58 web.1  | actionview (7.2.1) lib/action_view/renderer/renderer.rb:58:in `render_template_to_object'
10:47:58 web.1  | actionview (7.2.1) lib/action_view/renderer/renderer.rb:31:in `render_to_object'
10:47:58 web.1  | actionview (7.2.1) lib/action_view/rendering.rb:135:in `block in _render_template'
10:47:58 web.1  | actionview (7.2.1) lib/action_view/base.rb:308:in `in_rendering_context'
10:47:58 web.1  | actionview (7.2.1) lib/action_view/rendering.rb:134:in `_render_template'
10:47:58 web.1  | actionpack (7.2.1) lib/action_controller/metal/streaming.rb:258:in `_render_template'
10:47:58 web.1  | actionview (7.2.1) lib/action_view/rendering.rb:121:in `render_to_body'
10:47:58 web.1  | actionpack (7.2.1) lib/action_controller/metal/rendering.rb:186:in `render_to_body'
10:47:58 web.1  | actionpack (7.2.1) lib/action_controller/metal/renderers.rb:142:in `render_to_body'
10:47:58 web.1  | actionpack (7.2.1) lib/abstract_controller/rendering.rb:29:in `render'
10:47:58 web.1  | actionpack (7.2.1) lib/action_controller/metal/rendering.rb:167:in `render'
10:47:58 web.1  | actionpack (7.2.1) lib/action_controller/metal/instrumentation.rb:32:in `block (2 levels) in render'
10:47:58 web.1  | /home/blago/.rvm/rubies/ruby-3.3.5/lib/ruby/3.3.0/benchmark.rb:313:in `realtime'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/core_ext/benchmark.rb:14:in `ms'
10:47:58 web.1  | actionpack (7.2.1) lib/action_controller/metal/instrumentation.rb:32:in `block in render'
10:47:58 web.1  | actionpack (7.2.1) lib/action_controller/metal/instrumentation.rb:101:in `cleanup_view_runtime'
10:47:58 web.1  | activerecord (7.2.1) lib/active_record/railties/controller_runtime.rb:46:in `cleanup_view_runtime'
10:47:58 web.1  | actionpack (7.2.1) lib/action_controller/metal/instrumentation.rb:31:in `render'
10:47:58 web.1  | actionpack (7.2.1) lib/action_controller/metal/implicit_render.rb:39:in `default_render'
10:47:58 web.1  | actionpack (7.2.1) lib/action_controller/metal/basic_implicit_render.rb:9:in `send_action'
10:47:58 web.1  | actionpack (7.2.1) lib/abstract_controller/base.rb:226:in `process_action'
10:47:58 web.1  | actionpack (7.2.1) lib/action_controller/metal/rendering.rb:193:in `process_action'
10:47:58 web.1  | actionpack (7.2.1) lib/abstract_controller/callbacks.rb:261:in `block in process_action'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/callbacks.rb:121:in `block in run_callbacks'
10:47:58 web.1  | turbo-rails (2.0.10) lib/turbo-rails.rb:24:in `with_request_id'
10:47:58 web.1  | turbo-rails (2.0.10) app/controllers/concerns/turbo/request_id_tracking.rb:10:in `turbo_tracking_request_id'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/callbacks.rb:130:in `block in run_callbacks'
10:47:58 web.1  | audited (5.7.0) lib/audited/sweeper.rb:16:in `around'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/callbacks.rb:130:in `block in run_callbacks'
10:47:58 web.1  | audited (5.7.0) lib/audited/sweeper.rb:16:in `around'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/callbacks.rb:130:in `block in run_callbacks'
10:47:58 web.1  | actiontext (7.2.1) lib/action_text/rendering.rb:25:in `with_renderer'
10:47:58 web.1  | actiontext (7.2.1) lib/action_text/engine.rb:71:in `block (4 levels) in <class:Engine>'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/callbacks.rb:130:in `instance_exec'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/callbacks.rb:130:in `block in run_callbacks'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/callbacks.rb:141:in `run_callbacks'
10:47:58 web.1  | actionpack (7.2.1) lib/abstract_controller/callbacks.rb:260:in `process_action'
10:47:58 web.1  | actionpack (7.2.1) lib/action_controller/metal/rescue.rb:27:in `process_action'
10:47:58 web.1  | actionpack (7.2.1) lib/action_controller/metal/instrumentation.rb:77:in `block in process_action'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/notifications.rb:210:in `block in instrument'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/notifications/instrumenter.rb:58:in `instrument'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/notifications.rb:210:in `instrument'
10:47:58 web.1  | actionpack (7.2.1) lib/action_controller/metal/instrumentation.rb:76:in `process_action'
10:47:58 web.1  | actionpack (7.2.1) lib/action_controller/metal/params_wrapper.rb:259:in `process_action'
10:47:58 web.1  | activerecord (7.2.1) lib/active_record/railties/controller_runtime.rb:39:in `process_action'
10:47:58 web.1  | actionpack (7.2.1) lib/abstract_controller/base.rb:163:in `process'
10:47:58 web.1  | actionview (7.2.1) lib/action_view/rendering.rb:40:in `process'
10:47:58 web.1  | actionpack (7.2.1) lib/action_controller/metal.rb:252:in `dispatch'
10:47:58 web.1  | actionpack (7.2.1) lib/action_controller/metal.rb:335:in `dispatch'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/routing/route_set.rb:67:in `dispatch'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/routing/route_set.rb:50:in `serve'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/journey/router.rb:53:in `block in serve'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/journey/router.rb:133:in `block in find_routes'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/journey/router.rb:126:in `each'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/journey/router.rb:126:in `find_routes'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/journey/router.rb:34:in `serve'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/routing/route_set.rb:896:in `call'
10:47:58 web.1  | railties (7.2.1) lib/rails/engine.rb:535:in `call'
10:47:58 web.1  | railties (7.2.1) lib/rails/railtie.rb:226:in `public_send'
10:47:58 web.1  | railties (7.2.1) lib/rails/railtie.rb:226:in `method_missing'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/routing/mapper.rb:33:in `block in <class:Constraints>'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/routing/mapper.rb:62:in `serve'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/journey/router.rb:53:in `block in serve'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/journey/router.rb:133:in `block in find_routes'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/journey/router.rb:126:in `each'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/journey/router.rb:126:in `find_routes'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/journey/router.rb:34:in `serve'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/routing/route_set.rb:896:in `call'
10:47:58 web.1  | rack-attack (6.7.0) lib/rack/attack.rb:103:in `call'
10:47:58 web.1  | rack-attack (6.7.0) lib/rack/attack.rb:127:in `call'
10:47:58 web.1  | warden (1.2.9) lib/warden/manager.rb:36:in `block in call'
10:47:58 web.1  | warden (1.2.9) lib/warden/manager.rb:34:in `catch'
10:47:58 web.1  | warden (1.2.9) lib/warden/manager.rb:34:in `call'
10:47:58 web.1  | rack (2.2.9) lib/rack/tempfile_reaper.rb:15:in `call'
10:47:58 web.1  | rack (2.2.9) lib/rack/etag.rb:27:in `call'
10:47:58 web.1  | rack (2.2.9) lib/rack/conditional_get.rb:27:in `call'
10:47:58 web.1  | rack (2.2.9) lib/rack/head.rb:12:in `call'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/http/permissions_policy.rb:38:in `call'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/http/content_security_policy.rb:36:in `call'
10:47:58 web.1  | rack (2.2.9) lib/rack/session/abstract/id.rb:266:in `context'
10:47:58 web.1  | rack (2.2.9) lib/rack/session/abstract/id.rb:260:in `call'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/middleware/cookies.rb:704:in `call'
10:47:58 web.1  | activerecord (7.2.1) lib/active_record/migration.rb:674:in `call'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/middleware/callbacks.rb:31:in `block in call'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/callbacks.rb:101:in `run_callbacks'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/middleware/callbacks.rb:30:in `call'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/middleware/executor.rb:16:in `call'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/middleware/actionable_exceptions.rb:18:in `call'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/middleware/debug_exceptions.rb:31:in `call'
10:47:58 web.1  | web-console (4.2.1) lib/web_console/middleware.rb:132:in `call_app'
10:47:58 web.1  | web-console (4.2.1) lib/web_console/middleware.rb:28:in `block in call'
10:47:58 web.1  | web-console (4.2.1) lib/web_console/middleware.rb:17:in `catch'
10:47:58 web.1  | web-console (4.2.1) lib/web_console/middleware.rb:17:in `call'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/middleware/show_exceptions.rb:32:in `call'
10:47:58 web.1  | railties (7.2.1) lib/rails/rack/logger.rb:41:in `call_app'
10:47:58 web.1  | railties (7.2.1) lib/rails/rack/logger.rb:29:in `call'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/middleware/remote_ip.rb:96:in `call'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/middleware/request_id.rb:33:in `call'
10:47:58 web.1  | rack (2.2.9) lib/rack/method_override.rb:24:in `call'
10:47:58 web.1  | rack (2.2.9) lib/rack/runtime.rb:22:in `call'
10:47:58 web.1  | activesupport (7.2.1) lib/active_support/cache/strategy/local_cache_middleware.rb:29:in `call'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/middleware/server_timing.rb:61:in `block in call'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/middleware/server_timing.rb:26:in `collect_events'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/middleware/server_timing.rb:60:in `call'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/middleware/executor.rb:16:in `call'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/middleware/static.rb:27:in `call'
10:47:58 web.1  | rack (2.2.9) lib/rack/sendfile.rb:110:in `call'
10:47:58 web.1  | actionpack (7.2.1) lib/action_dispatch/middleware/host_authorization.rb:143:in `call'
10:47:58 web.1  | rack-mini-profiler (3.3.1) lib/mini_profiler.rb:334:in `call'
10:47:58 web.1  | railties (7.2.1) lib/rails/engine.rb:535:in `call'
10:47:58 web.1  | puma (6.4.3) lib/puma/configuration.rb:272:in `call'
10:47:58 web.1  | puma (6.4.3) lib/puma/request.rb:100:in `block in handle_request'
10:47:58 web.1  | puma (6.4.3) lib/puma/thread_pool.rb:378:in `with_force_shutdown'
10:47:58 web.1  | puma (6.4.3) lib/puma/request.rb:99:in `handle_request'
10:47:58 web.1  | puma (6.4.3) lib/puma/server.rb:464:in `process_client'
10:47:58 web.1  | puma (6.4.3) lib/puma/server.rb:245:in `block in run'
10:47:58 web.1  | puma (6.4.3) lib/puma/thread_pool.rb:155:in `block in spawn_thread

@tripplesteel, thank you for your feedback. It looks like the helpers aren’t loading correctly. Could you provide a bit more information about your app? Is it API-only? Also, could you share your Gemfile with me?

@akodkod Yes of cource. Its not API only. The app is new, running ruby 3.3.5 and rails 7.2.1. It's still in development phase, because of migration from old rails 4 app. Here you are the Gemfile
source 'https://rubygems.org'

ruby '3.3.5'

# Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
gem 'rails', '= 7.2.1'
gem "pg"
# Use Puma as the app server
gem 'puma'
gem 'kamal'

# The original asset pipeline for Rails [https://github.com/rails/sprockets-rails]
gem "sprockets-rails"
gem 'sassc-rails'

# Bundle and transpile JavaScript [https://github.com/rails/jsbundling-rails]
gem "jsbundling-rails"

# Hotwire's SPA-like page accelerator [https://turbo.hotwired.dev]
gem "turbo-rails"

# Hotwire's modest JavaScript framework [https://stimulus.hotwired.dev]
gem "stimulus-rails"

# Bundle and process CSS [https://github.com/rails/cssbundling-rails]
gem "cssbundling-rails"

gem 'haml'
gem 'haml-rails'
gem 'htmlentities'
gem 'nokogiri'

gem "bootsnap", require: false

gem 'devise'
gem 'httpclient'
gem 'rails-i18n'
gem 'multi_json'
gem 'lru_redux'
gem 'rack-attack'
gem 'net-sftp'
gem 'http-cookie'
gem 'sitemap_generator'
gem 'audited'
gem 'scenic'
gem 'dotenv'
gem "image_processing"
gem "solid_queue"
gem "solid_queue_dashboard"

group :development do
  # Access an interactive console on exception pages or by calling 'console' anywhere in the code.
  gem 'web-console'
  # Display performance information such as SQL time and flame graphs for each request in your browser.
  # Can be configured to work on production as well see: https://github.com/MiniProfiler/rack-mini-profiler/blob/master/README.md
  gem 'rack-mini-profiler'
  gem 'listen'
  gem 'rainbow'
  gem 'terminal-table'
  gem 'brakeman'
  gem 'foreman'
end

getting the same error now.
it was working earlier tho but i updated my rails app, maybe solid_queue did an update and that method is gone now


Moved from Google Chrome to electron-prebuilt for libffmpegsumo.so

As discussed in #100, Google Chrome is no longer providing a libffmpegsumo.so. Since we currently haven't invested in distributing .deb/.rpm (or any archive), then we cannot provide a prebuilt libffmpegsumo.so file. As a result, we are introducing a 100MB hack for a 6MB file (hopefully less bandwidth with gzipping) which downloads electron-prebuilt and copies its libffmpegsumo.so file.
In this PR:

Replaced copying from Google Chrome with copying from electron-prebuilt
Updated documentation

/cc @wlaurance

Hmm...
I'm looking into why the merge caused a failure @twolfson

Seems to run properly locally.
https://travis-ci.org/plaidchat/plaidchat/jobs/77733665 is the failing job (only on node v0.11)

I am on vacation for 2 weeks starting last Friday. I won't have internet after today. Please try to resolve this without me =/


Ghost Cloud Storage

When using a storage adapter in Ghost, the purge script reports :
Completed but nothing to do here...
Congrats, all images are being used 👍

Regards.

Hi @rchampagneca,
Please could you describe which Ghost version & storage adapter are you using?
Thank you

Ghost-CLI version: 1.16.3
Ghost version: 4.1.2 (at /var/www/rchampagneca)

I'm using this version of a gCloud Storage Adapter : https://github.com/thombuchi/ghost-google-cloud-storage


Use alternate SSH client

Is it possible to use another SSH client or wrapper other than the native SSH client? I found a script that allows me to carry all of my local aliases to whatever hosts I shell into, I would like to use this along with i2cssh but don't see a way other than symlinking /usr/local/bin/ssh to the script, which I can't do since this is a shared workstation.

I figured out how to hack the source to use sshrc, just had to mod lib/i2cssh.rb


Update license

Purpose

Updates the licenses files

Codecov Report

Merging #20482 into release-stage will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           release-stage   #20482   +/-   ##
==============================================
  Coverage          15.21%   15.21%
==============================================
  Files                 50       50
  Lines               1341     1341
  Branches             208      208
==============================================
  Hits                 204      204
  Misses              1121     1121
  Partials              16       16

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 8e96b20...508af0e. Read the comment docs.


Back-end collapse API - Convert generic JSON objects to Java POJOs for strong typing

[x] joeferner
[ ] sfeng88
[ ] mwizeman joeybrk372 jharwig

This commit gives API users a type safe interface to collapse entities.
CHANGELOG
Changed: Back-end collapse API - Convert generic JSON objects to Java POJOs for strong typing

On my project I am generating workspaces and I need a way to collapse vertices using the new API. Typing in weakly types JSON seemed very fragile.

@joeferner would this prevent plugins from adding to extendedData?

@jharwig It works with the graph styling plugin (I assume it should work with others as well). Vertices, edges, and compound nodes are a little different in the extended data structure in that they aren't stored anywhere but are computed when asked for so this change doesn't effect the other parts of the extended data because of this.
Existing plugin extended data is retrieved in the getProductExtendedDataJson method https://github.com/visallo/visallo/pull/1309/files#diff-a0f439d5c2ad6a598890917b6f2915daR893 which still loads the property as JSON and appends to the JSON.
The saving is handled here https://github.com/visallo/visallo/blob/bd264f6b90f702c78da05ffc3046233af65962f3/core/plugins/model-vertexium/src/main/java/org/visallo/vertexium/model/workspace/VertexiumWorkspaceRepository.java#L1073 which isn't effected by these changes.

Go ahead and merge after travis succeeds


Revert "Add CPU Time since last sample data on JFR events (#16)"

Need to revert #16 as it leads to deadlocks in the signal handler

LGTM


Proposal to Relicense Theia Blueprint to MIT: Seeking Approvals

👋 Hey all,
In our last Theia Dev call, we reached a consensus on considering a change of license for the Theia Blueprint project. We're thinking of moving from EPL 2.0 (or GNU General Public License with classpath exception) to MIT. As Theia Blueprint is meant to be an example as well as a template for adopters, the aim is to clear up any gray areas around the reuse and modification of the Blueprint code in proprietary projects.
This should make it easier for everyone to adopt and modify Blueprint without worrying over the current licensing complexities. Note that this is only about the code contained in this repository (Theia Blueprint), not Theia in general.
What We Need
To officially make this change, we need a thumbs-up ("+1") from a representative of every copyright holder who has contributed to Theia Blueprint.
Please reply to this issue with a +1 if you're on board with this license change.
If you have any concerns or questions, feel free to comment on this issue or reach out to us directly.
Thanks for your cooperation! 🙏
More information

Help desk ticket at the Eclipse Foundation
Approval from the Eclipse Foundation to use MIT for Non-Code, Example, and Other Content

List of contributors

[x] @thegecko
[x] EclipseSource (@jfaltermeier, @lucas-koehler, @AlexandraBuzila, @JonasHelming, @tortmayr)
[ ] Ericsson (@marcdumais-work, @vince-fugnitto, @paul-marechal)
[ ] @danepowell
[ ] STMicroelectronics (@Niklas-Dahlquist, @ErikMallbergSTM)
[ ] @Atharva321
[x] @duncdrum
[x] TypeFox (@msujew)
[ ] @DucNgn

+1

+1

+1

+1 from STMicroelectronics (@ErikMallbergSTM and @Niklas-Dahlquist)

+1

I have checked with Ericsson, we all approve the re-licensing.  (Marc D, Paul M, Duc and Vince F) Please confirm Duc did not contribute after 2021.

@Atharva321 Please let me know what you think about our plans for re-licensing Theia Blueprint (see above). Thank you!

+1
On Tue, 31 Oct 2023 at 20:35, Philip Langer @.***>
wrote:

@Atharva321 https://github.com/Atharva321 Please let me know what you
think about our plans for re-licensing Theia Blueprint (see above). Thank
you!
—
Reply to this email directly, view it on GitHub
https://github.com/eclipse-theia/theia-blueprint/issues/310#issuecomment-1787404073,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKR742HPUJALPQW3F63YLMDYCEHTZAVCNFSM6AAAAAA6PTNU5WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBXGQYDIMBXGM
.
You are receiving this because you were mentioned.Message ID:
@.***>

the README still says EPL/GPL

Thanks for reporting, see: https://github.com/eclipse-theia/theia-ide/pull/415


Godot4.4dev7: Using @warning_ignore annotation at function signature level is broken

Tested versions

Reproducible in v4.4.dev7.mono.official [46c8f8c5c]
Works on v4.4.dev6.mono.official [1f47e4c4e]

System information
Windows 11
Issue description
I updated my CI-PR workflow to Godot4.4dev7 and it fails now
It reports now the warning and is ignoring the @warning_ignore("unsafe_method_access")
[Ignore]Line 13 (UNSAFE_METHOD_ACCESS):The method "line_number()" is not present on the inferred type "Node" (but may be present on a subtype).
The annotation was working before on function signature level but is now ignored, this is a regression
Steps to reproduce
extends Node

@warning_ignore("unsafe_method_access")
func check(test_case :Node) -> void:
	#@warning_ignore("unsafe_method_access")
	test_case.line_number()

Minimal reproduction project (MRP)
n/a

Looks like @warning_ignore is broken general.
With introducing the https://github.com/godotengine/godot/pull/76020 the annotation is broken,

This functionality was:

Undocumented
Not really intended

See:

https://github.com/godotengine/godot/pull/83037#discussion_r1477302018

This change was intentional

There has been a correction https://github.com/godotengine/godot/pull/75620 do not sound as originally intended.
This change is a step backwards and only makes it more cumbersome. It had worked fine for many versions and feels broken.
It is definitely a regression whether intentional or not!

#75620 is for the specific case, the method declaration, it's not for the whole function, at least the issue it solved, can you point to where in that PR that part of this was touched on?
The PR is about the scope of shadowing not the scope of the warning,

#75620 is for the specific case, the method declaration, it's not for the whole function, at least the issue it solved, can you point to where in that PR that part of this was touched on?
The PR is about the scope of shadowing not the scope of the warning,

From the issue description, https://github.com/godotengine/godot/issues/75208 is related to the warning, I would say.
Still, I don't see why this should be a problem at the class or method level.
Sure, it may hide future problems, but that may well be intentional, we're talking about warnings here.
If you don't want that, you can define the blocks with the new annotation.
I think that forcing developers to switch to the new annotations after the fact is the wrong way to go.
This only creates frustration, may not be a problem for smaller projects, but for larger projects it only creates unnecessary effort, especially as it used to work well.
Translated with DeepL.com (free version)

It working this way was never documented, and was not by design given the discussions, but regardless the removal of it was entirely intentional

It working this way was never documented, and was not by design given the discussions, but regardless the removal of it was entirely intentional

Yes, I understood that, but who says it was bad?
Please understand that it's a real problem when you maintain a plugin that has to be backwards compatible.
I can't change everything to the new annotations at once because they are not available in older versions and would cause compiler errors.
To handle this so strictly afterwards is simply not right and also unnecessary.
Both functionalities can coexist and would not break anything.

See the discussion I linked for the background, this was undocumented behaviour that was never officially supported, so therefore there's no breakage

I can't change everything to the new annotations at once because they are not available in older versions and would cause compiler errors.

Use local ignore annotations I'd say, that was the supported usage

It working this way was never documented, and was not by design given the discussions, but regardless the removal of it was entirely intentional

Yes, I understood that, but who says it was bad?

This is bad because the user can accidentally ignore existing and future warnings in the function body without actually intending to do so, see the video in the comment.
The problem with @warning_ignore on functions is that the scope of the annotation is unclear. Some might expect it to apply only to the next line, some might expect it to apply only to the function header, and some might expect it to apply to the entire function, including its body.
If we were committed to never breaking compatibility even for undocumented and unintended behavior, we could solve this by introducing a new annotation like @warning_ignore_once and deprecating @warning_ignore. This would be quite sad, given that @warning_ignore works well in most cases and we essentially have to rename it because of an unfortunate mistake in the past.
We can't guarantee absolute compatibility, because otherwise we wouldn't be able to fix any bugs. General rule: don't rely on bad behavior, fixing a bug is not considered breaking compatibility. The problem is that some behaviors are not obviously bad. They can be controversial: convenient in some cases and confusing in others.
In this particular situation, we decided to remove the behavior because it is:

Undocumented.
Unintentional.
Can be confusing for users and lead to unintentional ignoring of warnings.
For new versions of the engine, there are @warning_ignore_start and @warning_ignore_restore annotations. If you want to maintain compatibility with older engine versions, you can use single @warning_ignores.
We expect that this change should not significantly affect most projects. Only individual projects that relied on this behavior and don't want to use the new annotations to maintain compatibility with older engine versions within a single project version.

I'm sorry to hear that this has caused inconvenience to your plugin, but I don't see enough justification to revert this change. In my opinion, the only reason we should reconsider this is if the assumption in p.5 turns out to be wrong. If many users report serious problems, even though they didn't rely on this behavior and are not constrained by the need to maintain compatibility with older engine versions.
As for addons, I think the problem may be that you want to have one version of the addon compatible with many engine versions. You could adapt the engine workflow: have a master/main branch of the plugin and several maintenance branches for older engine branches (4.3, 4.2, etc.). Compatible changes you could cherry-pick into maintenance branches.
Note that using any new engine feature makes the addon incompatible with older versions of the engine. For example, 4.4 introduces typed dictionaries. If you use them, this will make your addon incompatible with 4.3, just like using @warning_ignore_start and @warning_ignore_restore.
Also note that this change has a positive consequence. Users who unintentionally ignored warnings for an entire function will now see them. From this perspective, the change was a bugfix, not a regression.

@dalexeev thanks for the very detailed explanation.

Also note that this change has a positive consequence. Users who unintentionally ignored warnings for an entire function will now see them. From this perspective, the change was a bugfix, not a regression.

I am with you on this reasoning, as much as it hurts. I found the option to disable the warnings of the entire function or class quite practical, but yes, it is stricter this way, and you are now “forced” to take a closer look at the problem areas.
Nevertheless, I see a bug here, as the editor console output differs from the system console and is even logged as an error.
I have set the level to warning and the editor console shows it as warnings

But running at system console, it is reported as errors.

@dalexeev I fixed the warnings with success, but I run now into a compile error on v4.2.2.stable.official [15073afe3]

Just for info, looks like I need a fix customized for Godot2.2.x, all higher versions has no issues.

@MikeSchulze @warning_ignore support for match branches was added in #83037 (i.e. 4.3).


Add additional types for translators and other TS files; fix building…

… before testing

Pull Request Test Coverage Report for Build 54

0 of 0   changed or added relevant lines in 0 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.2%) to 87.112%

Files with Coverage Reduction
New Missed Lines
%

lib/translators/InfoTranslator.js
2
79.66%

lib/translators/object/ObjectsTranslator.js
4
93.33%

Totals

Change from base Build 53:
0.2%

Covered Lines:
752

Relevant Lines:
823

💛  - Coveralls


Question for the wanted accessory name.

if the accessory name has a space in it, do I include the space?

thanks


FFM-9030 Update JS SDK docs for polling, and correct install script

What
To Support FFM-9030 this adds polling documentation for the JavaScript SDK
Also minor correction for install step to reference the correct distribution for non es module browsers

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

@erdirowlands this has the "Do not merge" label. Is it still a work in progress?

@erdirowlands this has the "Do not merge" label. Is it still a work in progress?

I forgot to mention the reason for using "do not mege". This is still a work in progress - we aim to release a new SDK version early next week, where I'll remove the label and ask for merge.

Hi @KimberlyFields this can now be merged - thanks.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


README.md: fixed link for akka.io

Hi! Just looking at this library, I noticed a broken link in README.md so here's the fix :)

Thanks a lot, i didn't realize it and it was kinda bad to have a bad readme :)


[SPARK-20923] turn tracking of TaskMetrics._updatedBlockStatuses off

What changes were proposed in this pull request?
Turn tracking of TaskMetrics._updatedBlockStatuses off by default. As far as I can see its not used by anything and it uses a lot of memory when caching and processing a lot of blocks.  In my case it was taking 5GB of a 10GB heap and I even went up to 50GB heap and the job still ran out of memory.  With this change in place the same job easily runs in less then 10GB of heap.
We leave the api there as well as a config to turn it back on just in case anyone is using it.
How was this patch tested?
Ran unit tests that were modified and manually tested on a couple of jobs (with and without caching).  Clicked through the UI and didn't see anything missing.
Ran my very large hive query job with 200,000 small tasks, 1000 executors, cached 6+TB of data this runs fine now whereas without this change it would go into full gcs and eventually die.
Author: Thomas Graves<EMAIL_ADDRESS>Author: Tom Graves<EMAIL_ADDRESS>Closes #18162 from tgravescs/SPARK-20923.
(cherry picked from commit 5b5a69b)

tests ok

LGTM, please backport it for spark 2.1 also as it si still the main released used on the cluster


Alamofire 4 beta / Swift 3 - how to get suggestedFilename from response?

Hi,
I'm in the process of migrating my projects to XCode 8 / Swift 3.
Currently most stuff works, however I can't find how to get the suggestedFilename from the response?
self.downloadManager.request(requestString, method:.get, parameters: nil, encoding:URLEncoding.default).response { response in
                 if let filename = response.suggestedFilename {
         }
    }

This gives me the error: Value of type 'DefaultDataResponse' has no member 'suggestedFilename'.
How should I access the suggested filename in Alamofire 4?
Thanks a lot!

First of all, suggestedFilename is a member of NSURLResponse, so you first need to access the DefaultDataReponse's response member :
self.downloadManager.request(requestString, method:.get, parameters: nil, encoding:URLEncoding.default).response { response in
        if let filename = response.response?.suggestedFilename {
    }
}

Moreover, what filename are you expecting to get using request(_:method:parameters:encoding)? Shouldn't you be using download(_:method:parameters:encoding)to actually write something to disk?

Thanks @thebluepotato. 🍻
In the future, I'd recommend opening a question on Stack Overflow for questions like this. We use our GitHub project for bug reports and feature requests.

From our Contribution Guidelines
Asking Questions
We don't use GitHub as a support forum. For any usage questions that are not specific to the project itself, please ask on Stack Overflow instead. By doing so, you'll be more likely to quickly solve your problem, and you'll allow anyone else with the same question to find the answer. This also allows maintainers to focus on improving the project for others.


Preflight Checklist

[X] I agree to follow the Code of Conduct that this project adheres to.
[X] I have searched the issue tracker for an issue that matches the one I want to file, without success.
[X] I am not looking for support or already pursued the available support channels without success.

Version
1.11.0
Installation Type
Official SaaS Service
Service Name
DongTai-agent-java
Describe the details of the bug and the steps to reproduce it

我查看项目漏洞里面的组件漏洞时报错，经排查在pod里面进行curl这个接口发现7分多钟之后才会返回正常的数据，但是这对于web界面来说已经远远大于超时时间了，请问大佬该怎么优化。

看代码查数据库表，发现应该不是数据库的问题，

Additional Information
No response
Logs
No response

The problem here is known, in the upcoming 1.12.0 version, the component vulnerability part will be reworked

@Bidaya0  收到 感谢

close, since 1.12.0 has been released


[Failed pipeline] Network - ApplicationGateways

failed run: https://github.com/Azure/ResourceModules/actions/runs/4168125391

successful run: https://github.com/Azure/ResourceModules/actions/runs/4176965104


Demo, XYZ

All good

Another test

Another test

Hopefully final test

So so close

Another test

Another one


file loss when editing a row.

Tables that contain file fields lose their already loaded files when updating any row. because in the edit form, the file field is always empty. and when the form is sent to the server it deletes the existing one.
It mainly occurs when you want to make a file type field for an avatar in the user table

I want to second this issue. If an edit view has a field component referencing a file, even if the field view is set to a read only (not editable) type such as show image, when the edit view is saved, the file field is overwritten as empty. The file fields that are not within the view are not overwritten.
For anyone else needing a quick solution to this and only needs a view only reference to the field, you can create a separate view with the "Show" pattern or similar that only contains the field component referencing the file and then add a view component referencing this new view into the edit view to display the image. When the edit view is saved, the file field is not overwritten as it is in a show view.


Make ElfParser::from_mmap() infallible

The ElfParser::from_mmap() constructor can't error out. Remove the theoretical error return path by dropping the Result from the return type.

Trivial refactoring; merging.


Can't release the cli module

Ever since I added the cli module to the project, I can't release due to:
Error: [ERROR] Rule failure while trying to close staging repository with ID "ioquarkiverse-1781".
Error: [ERROR]
Error: [ERROR] Nexus Staging Rules Failure Report
Error: [ERROR] ==================================
Error: [ERROR]
Error: [ERROR] Repository "ioquarkiverse-1781" failures
Error: [ERROR]   Rule "sources-staging" failures
Error: [ERROR]     * Missing: no main jar artifact found in folder '/io/quarkiverse/authzed/quarkus-authzed-cli/0.2.1'
Error: [ERROR]   Rule "javadoc-staging" failures
Error: [ERROR]     * Missing: no main jar artifact found in folder '/io/quarkiverse/authzed/quarkus-authzed-cli/0.2.1'
Error: [ERROR]
Error: [ERROR]
Error: [ERROR] Cleaning up local stage directory after a Rule failure during close of staging repositories: [ioquarkiverse-1781]
Error: [ERROR]  * Deleting context 166ff053e5f61f.properties
Error: [ERROR] Cleaning up remote stage repositories after a Rule failure during close of staging repositories: [ioquarkiverse-1781]
Error: [ERROR]  * Dropping failed staging repository with ID "ioquarkiverse-1781" (Rule failure during close of staging repositories: [ioquarkiverse-1781]).

Details: https://github.com/quarkiverse/quarkus-authzed-client/actions/runs/4364294724/jobs/7631435499
It seems to me that cli is producing an artifact with the classifier runner and there is no unclassified artfiact. Nexus doesn't seem to like it.

@gastaldi, @gsmet, @geoand: Any ideas here? How do we deal with this issue for the Quarkus CLI itself?

No ideas here unfortunately

If that doesn't work, you might need to generate your CLI as an uber-jar:
        <quarkus.package.type>uber-jar</quarkus.package.type>


Create new MET TC app

Replace italics below with details for this issue.
Describe the New Feature
The next new METexpress app will be MET TC.
Acceptance Testing
App must pass automated tests upon completion.
Time Estimate
1-2 weeks.
Sub-Issues
Consider breaking the new feature down into sub-issues.

[ ] Write metadata script for TC app.
[ ] Write new app code.
[ ] Create automated tests for this app.

Relevant Deadlines
List relevant project deadlines here or state NONE.
Funding Source
Define the source of funding and account keys here or state NONE.
Define the Metadata
Assignee

[ ] Select engineer(s) or no engineer required
[ ] Select scientist(s) or no scientist required

Labels

[ ] Select component(s)
[ ] Select priority
[ ] Select requestor(s)

Projects and Milestone

[ ] Review projects and select relevant Repository and Organization ones or add "alert:NEED PROJECT ASSIGNMENT" label
[ ] Select milestone to next major version milestone or "Future Versions"

Define Related Issue(s)
Consider the impact to the other METplus components.

[ ] METplus, MET, METdatadb, METviewer, METexpress, METcalcpy, METplotpy

New Feature Checklist
See the METplus Workflow for details.

[ ] Complete the issue definition above, including the Time Estimate and Funding source.
[ ] Fork this repository or create a branch of development.
Branch name: feature_<Issue Number>_<Description>
[ ] Complete the development and test your changes.
[ ] Add/update log messages for easier debugging.
[ ] Add/update unit tests.
[ ] Add/update documentation.
[ ] Push local changes to GitHub.
[ ] Submit a pull request to merge into development.
Pull request: feature <Issue Number> <Description>
[ ] Define the pull request metadata, as permissions allow.
Select: Reviewer(s), Project(s), Milestone, and Linked issues
[ ] Iterate until the reviewer(s) accept and merge your changes.
[ ] Delete your fork or branch.
[ ] Close this issue.

The initial app version has been completed and all automated tests pass.


[BUG] DocumentSplitter‘s maxOverlapSize parameter sometimes fails

DocumentSplitter‘s maxOverlapSize parameter sometimes fails .
Look at this method: dev.langchain4j.data.document.splitter.HierarchicalDocumentSplitter#overlapFrom
private String overlapFrom(String segmentText) {
        if (maxOverlapSize == 0) {
            return "";
        }

        SegmentBuilder overlapBuilder = new SegmentBuilder(maxOverlapSize, this::sizeOf, joinDelimiter());

        String[] sentences = new DocumentBySentenceSplitter(1, 0, null, null).split(segmentText);
        for (int i = sentences.length - 1; i >= 0; i--) {
            String part = sentences[i];
            if (overlapBuilder.hasSpaceFor(part)) {
                overlapBuilder.prepend(part);
            } else {
                return overlapBuilder.build();
            }
        }

        return "";
    }```

Should it be returned like this?

```private String overlapFrom(String segmentText) {
        if (maxOverlapSize == 0) {
            return "";
        }

        SegmentBuilder overlapBuilder = new SegmentBuilder(maxOverlapSize, this::sizeOf, joinDelimiter());

        String[] sentences = new DocumentBySentenceSplitter(1, 0, null, null).split(segmentText);
        for (int i = sentences.length - 1; i >= 0; i--) {
            String part = sentences[i];
            if (overlapBuilder.hasSpaceFor(part)) {
                overlapBuilder.prepend(part);
            } else {
                return overlapBuilder.build();
            }
        }

        **return return overlapBuilder.build();**
    }```

- LangChain4j version: e.g. 0.24.0

I'm not sure if this is correct, but it can satisfy my understanding of the maxOverlapSize parameter.

@18856317221 thank you for reporting! It is indeed a bug, here is a fix: https://github.com/langchain4j/langchain4j/pull/464


Accounting: Listpayments fee_msat missing

Background
We show msat for values sent but do not for fees paid.
Your environment

version of lnd
which operating system (uname -a on *Nix)
version of btcd, bitcoind, or other backend
any other relevant environment details

Steps to reproduce
Tell us how to reproduce this issue. Please provide stacktraces and links to code in question.
Expected behaviour
Tell us what should happen
Actual behaviour
Tell us what happens instead

Closing as duplicate of #1359.


Fix Device Shadow demo build on Cypress64 IDE

Fix linker issue undefined reference to RunDeviceShadowDemo linker error in Cypress64 MTB build from FreeRTOS console download due to missing demo and library files in the project makefile.
Also, update the aws_tests Makefile with the library and test files of Device Shadow

Thanks @mingyue86010 !

/bot run checks


re-install of the bee-clef debian package fails

Install bee-clef. Uninstall it. Then try to install it,
$ dpkg -i bee-clef_0.4.7_amd64.deb
Selecting previously unselected package bee-clef.
(Reading database ... 140013 files and directories currently installed.)
Preparing to unpack bee-clef_0.4.7_amd64.deb ...
useradd: group bee-clef exists - if you want to add this user to that group, use -g.
dpkg: error processing archive bee-clef_0.4.7_amd64.deb (--install):
 new bee-clef package pre-installation script subprocess returned error exit status 9
Errors were encountered while processing:
 bee-clef_0.4.7_amd64.deb

sudo groupdel bee-clef solves your problem

Is there something we can do to prevent this from happening?

Perhaps the uninstall script should remove the group.


AL-Pipeline: Table "XXXX" for the extension object is not found, despite symbols being installed correctly

Describe the issue
Pipeline is set up in DevOps with 3rd Party Apps and Dependencies located in InstallApps Folder.
The sync and publish step of the 3rd parties works without any errors.
The problem is later on when the app folders are gettings compiled with external dependencies.
Some Apps do compile correctly with packages but one of the big ones throws the error:
The target Table 'XXXX' for the extension object is not found, despite the apps definitely being already installed.
The symbols used are either directly provided by the partners or downloaded from the production env. via "download symbols".
If compiled via VS Code, it works.
Dont mind the override to input the DLLs.
If you need additional Info, feel free to ask.
Scripts used to create container and cause the issue
Run-AlPipeline @params `
    -pipelinename $pipelineName `
    -containerName $containerName `
    -imageName $imageName `
    -bcAuthContext $authContext `
    -environment $environmentName `
    -artifact $artifact.replace('{INSIDERSASTOKEN}',$insiderSasToken) `
    -memoryLimit $memoryLimit `
    -baseFolder $baseFolder `
    -licenseFile $LicenseFile `
    -installApps $installApps `
    -installTestApps $installTestApps `
    -previousApps $previousApps `
    -appFolders $appFolders `
    -testFolders $testFolders `
    -doNotRunTests:$doNotRunTests `
    -testResultsFile $testResultsFile `
    -testResultsFormat 'JUnit' `
    -installTestRunner:$installTestRunner `
    -installTestFramework:$installTestFramework `
    -installTestLibraries:$installTestLibraries `
    -installPerformanceToolkit:$installPerformanceToolkit `
    -enableCodeCop:$enableCodeCop `
    -enableAppSourceCop:$enableAppSourceCop `
    -enablePerTenantExtensionCop:$enablePerTenantExtensionCop `
    -enableUICop:$enableUICop `
    -azureDevOps:($environment -eq 'AzureDevOps') `
    -gitLab:($environment -eq 'GitLab') `
    -gitHubActions:($environment -eq 'GitHubActions') `
    -failOn 'error' `
    -AppSourceCopMandatoryAffixes $appSourceCopMandatoryAffixes `
    -AppSourceCopSupportedCountries $appSourceCopSupportedCountries `
    -additionalCountries $additionalCountries `
    -buildArtifactFolder $buildArtifactFolder `
    -CreateRuntimePackages:$CreateRuntimePackages `
    -appBuild $appBuild -appRevision $appRevision `
    -credential $credential `
    -NewBcContainer {
        Param([Hashtable]$parameters)
        New-BcContainer @parameters

        $dllFiles = Get-ChildItem -Path '.\.netpackages\' -Recurse -Filter '*.dll' -File -Force -Verbose
        foreach ($dll in $dllFiles) {
            Copy-FileToBcContainer -containerName $containerName -localPath $dll.FullName -containerPath "C:\Program Files\Microsoft Dynamics NAV\230\Service\Add-Ins\$($dll.Name)"
            # Copy-FileToBcContainer -containerName $containerName -localPath $dll.FullName -containerPath "C:\Program Files\dotnet\shared\$($dll.Name)"
        }

        $serverdllFiles = Get-ChildItem -Path '.\.resources\ServerAddIns\' -Recurse -Filter '*.dll' -File -Force -Verbose
        foreach ($serverdll in $serverdllFiles) {
            Copy-FileToBcContainer -containerName $containerName -localPath $serverdll.FullName -containerPath "C:\Program Files\Microsoft Dynamics NAV\230\Service\Add-Ins\$($serverdll.Name)"
            # Copy-FileToBcContainer -containerName $containerName -localPath $serverdll.FullName -containerPath "C:\Program Files\dotnet\shared\$($serverdll.Name)"
        }

        Invoke-ScriptInBcContainer $parameters.ContainerName -scriptblock {
            $progressPreference = 'SilentlyContinue'
            Set-NavServerInstance -ServerInstance 'BC' -Restart
        }
    } `
    -PublishBcContainerApp {
        Param([HashTable] $parameters)
        $parameters.sync = $true
        $parameters.syncMode = 'ForceSync'
        Publish-BcContainerApp @parameters
    }

Full output of scripts
Processing dependency CKL Software GmbH_Kostenrechnung 365 / Cost Accounting 365_<IP_ADDRESS> (32a8e514-216f-487a-82ea-23709b370af9)
Downloading symbols: CKL Software GmbH_Kostenrechnung 365  Cost Accounting 365_<IP_ADDRESS>.app
Url : .../packages?appId=32a8e514-216f-487a-82ea-23709b370af9&versionText=<IP_ADDRESS>&tenant=default

Compiling...
.\alc.exe /project:"c:\sources\XXXX" /packagecachepath:"c:\sources\.packages" /out:"c:\sources\.output\XXXXX.Base_1.0.11639.0.app" /BuildBy:"BcContainerHelper,6.0.26-preview1247" /assemblyprobingpaths:"C:\Program Files\dotnet\shared","C:\ProgramData\BcContainerHelper\Extensions\XXXXX\.netPackages\Service"
Microsoft (R) AL Compiler version <IP_ADDRESS>432
Copyright (C) Microsoft Corporation. All rights reserved

Compilation started for project 'XXX.Base' containing '1604' files at '12:37:42.793'.

##[error]./c:/sources/XXXX/src/tableextension/BaseAllocationAssignmentCOA.TableExt.al(1,63): error AL0247: The target Table 'Allocation Assignment COA' for the extension object is not found
##[error]./c:/sources/XXXX/src/tableextension/BaseCostAccJnlLine.TableExt.al(1,57): error AL0247: The target Table 'Cost Acc. Journal Line COA' for the extension object is not found
##[error]./c:/sources/XXXX/src/tableextension/BaseCostAccLedgEntry.TableExt.al(1,59): error AL0247: The target Table 'Cost Acc. Ledger Entry COA' for the extension object is not found

You would need to include the full script.

Ist mostly based on the old ms repo for devops.
Param(
    [Parameter(Mandatory=$false)]
    [ValidateSet('AzureDevOps','GithubActions','GitLab')]
    [string] $environment = 'AzureDevOps',
    [Parameter(Mandatory=$true)]
    [string] $version,
    [Parameter(Mandatory=$false)]
    [int] $appBuild = 0,
    [Parameter(Mandatory=$false)]
    [int] $appRevision = 0
)

if ($environment -eq "AzureDevOps") {
    $buildArtifactFolder = $ENV:BUILD_ARTIFACTSTAGINGDIRECTORY
}
elseif ($environment -eq "GitHubActions") {
    $buildArtifactFolder = Join-Path $ENV:GITHUB_WORKSPACE "output"
    New-Item $buildArtifactFolder -ItemType Directory | Out-Null
}

# XXX-DOC: Determine base Folder from PSScriptRoot
$scriptFullPath = (Get-Item (Join-Path $PSScriptRoot "...")).FullName
$baseFolder = (Get-Item (Join-Path $scriptFullPath "..\..")).FullName
. (Join-Path $PSScriptRoot "Read-Settings.ps1") -environment $environment -version $version
. (Join-Path $PSScriptRoot "Install-BcContainerHelper.ps1") -bcContainerHelperVersion $bcContainerHelperVersion -genericImageName $genericImageName

Write-Host "Using BaseFolder: $baseFolder"

$authContext = $null
$refreshToken = "$($ENV:BcSaasRefreshToken)"
$environmentName = "$($ENV:EnvironmentName)"
if ($refreshToken -and $environmentName) {
    $authContext = New-BcAuthContext -refreshToken $refreshToken
    if (Get-BcEnvironments -bcAuthContext $authContext | Where-Object { $_.Name -eq $environmentName -and  $_.type -eq "Sandbox" }) {
        Remove-BcEnvironment -bcAuthContext $authContext -environment $environmentName
    }
    $countryCode = $artifact.Split('/')[3]
    New-BcEnvironment -bcAuthContext $authContext -environment $environmentName -countryCode $countrycode -environmentType "Sandbox" | Out-Null
    do {
        Start-Sleep -Seconds 10
        $baseApp = Get-BcPublishedApps -bcAuthContext $authContext -environment $environmentName | Where-Object { $_.Name -eq "Base Application" }
    } while (!($baseApp))
    $baseapp | Out-Host

    $artifact = Get-BCArtifactUrl `
        -country $countryCode `
        -version $baseApp.Version `
        -select Closest

    if ($artifact) {
        Write-Host "Using Artifacts: $artifact"
    }
    else {
        throw "No artifacts available"
    }
}

$params = @{}
$insiderSasToken = "$ENV:insiderSasToken"
$licenseFile = "$ENV:licenseFile"
$codeSigncertPfxFile = "$ENV:CodeSignCertPfxFile"
if (!$doNotSignApps -and $codeSigncertPfxFile) {
    if ("$ENV:CodeSignCertPfxPassword" -ne "") {
        $codeSignCertPfxPassword = try { "$ENV:CodeSignCertPfxPassword" | ConvertTo-SecureString } catch { ConvertTo-SecureString -String "$ENV:CodeSignCertPfxPassword" -AsPlainText -Force }
        $params = @{
            "codeSignCertPfxFile" = $codeSignCertPfxFile
            "codeSignCertPfxPassword" = $codeSignCertPfxPassword
        }
    }
    else {
        $codeSignCertPfxPassword = $null
    }
}

$allTestResults = "testresults*.xml"
$testResultsFile = Join-Path $baseFolder "TestResults.xml"
$testResultsFiles = Join-Path $baseFolder $allTestResults
if (Test-Path $testResultsFiles) {
    Remove-Item $testResultsFiles -Force
}

# XXX-DOC: Set credentials for persistent use, adviced to be used with -keepContainers and reUseContainer
$credential = New-Object pscredential 'admin', (ConvertTo-SecureString -String 'admin' -AsPlainText -Force)

# XXX-DOC: Sets the AppFolders to compile
$appFolders = @(Get-ChildItem -Path ".\XXX*" -Directory)

# XXX-DOC: AL Language to use, gets loaded from Artifact
$vsixFile = Get-AlLanguageExtensionFromArtifacts -artifactUrl (Get-BCArtifactUrl -type OnPrem -country "de" -version "23.5." -select Latest) -Verbose
Write-Host "Using vsix-File located in $vsixFile"

Run-AlPipeline @params `
    -pipelinename $pipelineName `
    -containerName $containerName `
    -imageName $imageName `
    -bcAuthContext $authContext `
    -environment $environmentName `
    -artifact $artifact.replace('{INSIDERSASTOKEN}',$insiderSasToken) `
    -memoryLimit $memoryLimit `
    -baseFolder $baseFolder `
    -licenseFile $LicenseFile `
    -installApps $installApps `
    -installTestApps $installTestApps `
    -previousApps $previousApps `
    -appFolders $appFolders `
    -testFolders $testFolders `
    -doNotRunTests:$doNotRunTests `
    -testResultsFile $testResultsFile `
    -testResultsFormat 'JUnit' `
    -installTestRunner:$installTestRunner `
    -installTestFramework:$installTestFramework `
    -installTestLibraries:$installTestLibraries `
    -installPerformanceToolkit:$installPerformanceToolkit `
    -enableCodeCop:$enableCodeCop `
    -enableAppSourceCop:$enableAppSourceCop `
    -enablePerTenantExtensionCop:$enablePerTenantExtensionCop `
    -enableUICop:$enableUICop `
    -azureDevOps:($environment -eq 'AzureDevOps') `
    -gitLab:($environment -eq 'GitLab') `
    -gitHubActions:($environment -eq 'GitHubActions') `
    -failOn 'error' `
    -AppSourceCopMandatoryAffixes $appSourceCopMandatoryAffixes `
    -AppSourceCopSupportedCountries $appSourceCopSupportedCountries `
    -additionalCountries $additionalCountries `
    -buildArtifactFolder $buildArtifactFolder `
    -CreateRuntimePackages:$CreateRuntimePackages `
    -appBuild $appBuild -appRevision $appRevision `
    -credential $credential `
    -NewBcContainer {
        Param([Hashtable]$parameters)
        New-BcContainer @parameters

        $dllFiles = Get-ChildItem -Path '.\.netpackages\' -Recurse -Filter '*.dll' -File -Force -Verbose
        foreach ($dll in $dllFiles) {
            Copy-FileToBcContainer -containerName $containerName -localPath $dll.FullName -containerPath "C:\Program Files\Microsoft Dynamics NAV\230\Service\Add-Ins\$($dll.Name)"
            # Copy-FileToBcContainer -containerName $containerName -localPath $dll.FullName -containerPath "C:\Program Files\dotnet\shared\$($dll.Name)"
        }

        $serverdllFiles = Get-ChildItem -Path '.\.resources\ServerAddIns\' -Recurse -Filter '*.dll' -File -Force -Verbose
        foreach ($serverdll in $serverdllFiles) {
            Copy-FileToBcContainer -containerName $containerName -localPath $serverdll.FullName -containerPath "C:\Program Files\Microsoft Dynamics NAV\230\Service\Add-Ins\$($serverdll.Name)"
            # Copy-FileToBcContainer -containerName $containerName -localPath $serverdll.FullName -containerPath "C:\Program Files\dotnet\shared\$($serverdll.Name)"
        }

        Invoke-ScriptInBcContainer $parameters.ContainerName -scriptblock {
            $progressPreference = 'SilentlyContinue'
            Set-NavServerInstance -ServerInstance 'BC' -Restart
        }
    } `
    -PublishBcContainerApp {
        Param([HashTable] $parameters)
        $parameters.sync = $true
        $parameters.syncMode = 'ForceSync'
        Publish-BcContainerApp @parameters
    }

if ($environment -eq 'AzureDevOps') {
    Write-Host "##vso[task.setvariable variable=TestResults]$allTestResults"
}

@freddydk inserted full script

@freddydk quick additional info from not directly related issue #3604
To confirm that the used package is a valid one I ran Extract-AppFileToFolder with it, to check its integrity.
The extraction worked and in the extracted SymbolReference.json there are definitely the hints to the missing table "Allocation Assignment COA". Hope this helps.

If possible I used the runtime versions. The first error throwing app "CKL Kostenrechnung 365 / Cost Accounting 365" is an symbol app now after the full app did not work. The error further down with "ECM" in it is from EASY SOFTWARE AG and as runtime version.

The way it works (by default) is to publish all apps from installedApps to the container and then when building apps - downloading the symbols from the container for the compiler.
This process doesn't work if you use symbol apps.
If you add -doNotPublishApps to your Run-AlPipeline, then it might build - but you won't be able to run tests (but you cannot do that anyway if some of your apps are symbol apps)

Alright, as far as I can see the customer does not use tests. I will the try the -doNotPublishApps switch. Is their a way to check if an app is a symbol, full or runtime app?

After looking into https://github.com/microsoft/navcontainerhelper/blob/main/AppHandling/Run-AlPipeline.ps1 for the -doNotPublishApps, does this mean I would need to use $useCompilerFolder? And do I need a database for the pipeline to compile against, or does that work with just the compiler?

You can use compilerfolder if you want - else it will use a filesonly container.

I think I have to use it, else the .NET-DLLs are not there for compilation? Or do I have a thinking error rn?

@freddydk quick update, thanks to your help it worked with a filesonly container. But somehow the script does not get to seem the $buildArtifactFolder = $ENV:BUILD_ARTIFACTSTAGINGDIRECTORY right.
I get the following output after all extensions get compiled:
Compilation ended at '13:05:36.328'.

In your log, the buildartifactsFolder seems to be the same as the basefolder:
2024-10-10T10:52:09.8288772Z BuildArtifactFolder             C:\azure_pipeline\bcnas01_agent1_work\1\a
(or maybe it is empty - don't know)
In AL-Go I use this parameter to specify where I want the artifacts.

I think I have to use it, else the .NET-DLLs are not there for compilation? Or do I have a thinking error rn?

A Files-Only container will have the DLLs as well

Looks like the build now succeeds, will close this issue.


Add useful Rewrite Policies to resolve or parametrize URLs automatica…

…lly. Implements #45 #46 #47
These policies are implementes as decorators which are meant to be used with a trivial "PassThroughPolicy" which doesn't do any rewriting.

Note, I've implemented these 3 stories in a single PR because they are kind of related.

A RewritePolicy is not a URI decorator. A URI decorator modifies the represented URI, but the RewritePolicy decides if an how the URI needs modification. This can be as trivial as resolving it against another URI or always adding parameters, but it also can involve conditional operations like matching with regular expressions and acting on the result of these. I'm even considering to add HttpRequest to the parameters of rewritten so the result can also depend on the request headers and method.
A RewritePolicy is more like a decorator factory that adds decorators based on certain conditions.

Ok, it makes sense with these examples you mentioned that RewritePolicy can have its place, especially with the option to use headers as well.
Finished with the review, no more comment from me.

Added HttpRequest to policy method and fixed unit tests (removed verify calls).
@lemonboston please have another look before I squash and merge.

The changes look good to me.


possible bug in is_valid_false_atom

The is_valid_false_atom assigns the result of the check to a 32 bit value and cheks if it's 0;
false has 5 characters so the check only covers if it's a valid fals atom, for example falsy would also pass the check in question.

Thanks. I will work on this.

FWIW: I found that using a 64 bit error value instead of a 32 bit one solves the issue. but I'm not sure about the negative impact on the performance of that.

I'm not sure about the negative impact on the performance of that

I have not measured it, because I expect that there won't be any impact. It is going to be the same number of instructions (I expect).
Anyhow, if you have a document where parsing "false" atoms is the bottleneck... it is going to be a fun document.

Let us close this.

The new code might actually be faster...
https://godbolt.org/z/ORcNo5
I'll do a performance comparison later but I am confident that there won't be a regression and there might be a small gain. In any case, it is bug fix so it is not like we can have a choice. We can't leave the old code.


[PR] add Android support to location_background package

I would really appreciate if you guys can put location_background Android support on the roadmap. TBH - it seems a bit funny lack of Android version in Flutter.
Any thoughts on this one would be greatly appreciated...

As far as I know this package is supposed to provide similar functionality as https://github.com/flutter/plugins/tree/master/packages/android_alarm_manager which obviously is for Android only. The underlying platforms provide very different ways for background execution and therefore the different approaches.
You might want to follow #24278

Thanks, @zoechi for pointing out #24278, it seems however, android alarm manger falls into category 2  (periodically updates), while location background is into category 3 (platform driven events), which seems to me as entirely different use case. Anyway - looking at the above mentioned by you issue, seems to be the place for general inquiry re dart / flutter background execution.

Closing since this package doesn't exist anymore in plugins repo
Thank you


More space for pool name in edit pool

Create the component,
Add it to the index file,
Use the component in the edit pool page

I checked it, does not show 3 rows, only 2 rows!


Cannot use import statement outside a module (in monorepo)

What version of Remix are you using?
latest
Steps to Reproduce
https://github.com/ahoyahoy/remix-super-error
run in root
npm i
npm run dev

open http://localhost:3000 and http://localhost:3008
Expected Behavior
Next.js (port 3000) app works correctly and import Button from UI package.
I want same result in Remix app (without build UI package).
Also I want to use same approach with shared tsconfig.
Actual Behavior
Remix app return error SyntaxError: Cannot use import statement outside a module .
Also remix always rewrite my tsconfig...  which is pretty annoying (https://github.com/remix-run/remix/discussions/3180)

I have confirmed this is still an issue on 1.7.5
remix has major issues resolving local dependencies in mono repos, i'm not sure if its something to do with the symlinks - but i've been pulling my hair out all day trying to import a TS local library into my Remix app.
Here is another reproduction of this issue
https://github.com/dan-cooke/remix-esbuild-resolve-bug

Run yarn
yarn build --filter remix-example
Inspect the build output

You will see that the ui has not been bundled
var import_react2 = require("@remix-run/react"), import_ui = require("ui"), import_jsx_dev_runtime = require("react/jsx-dev-runtime"), meta = () => ({
  charset: "utf-8",
  title: "New Remix App",
  viewport: "width=device-width,initial-scale=1"
});

And of course if you try to execute this you will get the OP error "Cannot use ipmort statement outside a module"
Because node require will try to require the TS version of the UI module.
This issue makes developing Remix in a monorepo incredibly annoying as you will have to create buildable versions of all your TS libraries and pre-build them before your remix app.

Okay I've figured it out.
After digging into the compiler code and finding what I assumed to be a buggy plugin serverBareModulesPlugin - you see it assumes that any module that does not start with './ or / to be external on the server build.
https://github.com/remix-run/remix/blob/main/packages/remix-dev/compiler/plugins/serverBareModulesPlugin.ts#L123
I discovered a hidden option in the remix config
serverDependenciesToBundle
You can specify your local modules in here and remix will include them in the bundle.
Hope this helps someone

@ahoyahoy Can you give @dan-cooke suggestion a try and see if it fixes your issue? 🙏🏼


Remove tests for allennlp and chainer

Motivation

Development has ceased for AllenNLP and Chainer. In this PR, we will gradually cease the use of these functions by eliminating the tests of the integration functions that have been discontinued.
Description of the changes

Remove tests for allennlp and chainer

@gen740 Could you review this PR?

There are some conflicts; please merge the main branch.


Bad Request Error

Log says:
1> Refreshing account Coinbase Pro.
1> Web Banking Engine: Using user-supplied extension GDAX.lua version 1.00.
1> Protocol version: Web Scraping
1> Server address: https://api.gdax.com
1> Getting portfolio for Coinbase Pro...
...
1> Sending: GET https://api.coinbase.com/v2/exchange-rates?currency=ETC
22:28:05  1> HTTPS response: Bad Request
1> The server responded with the error message: Bad Request. Retrying...
1> Sending: GET https://api.coinbase.com/v2/exchange-rates?currency=ETC
22:28:06  1> HTTPS response: Bad Request
1> The API server of your bank responded with the error message: »Bad Request«  Please try again later or contact the support of your bank with this error message.

See https://github.com/leoschweizer/moneymoney-gdax/issues/3#issuecomment-411324815

I think this may need to be revised since they started trading but it's still thrown Bad Request. My strong suspicion is that they are not going to add this new currency to the deprecated API and an update may be necessary to their new API endpoint to make this work.
 1> Sending: GET https://api.coinbase.com/v2/exchange-rates?currency=ETC
          1> HTTPS response: Bad Request
          1> The server responded with the error message: Bad Request. Retrying...
          1> Sending: GET https://api.coinbase.com/v2/exchange-rates?currency=ETC
11:39:24  1> HTTPS response: Bad Request

I have also added ETC to my account so that shouldn't be an issue.
Maybe it's time to move to their new API endpoint 😄

@TheSerapher the exchange rates API is not a GDAX API but a Coinbase API and thus not subject to the deprecation at all!

My bad! But isn't that the API endpoint for Coinbase and not Coinbase Pro (which is replacing GDAX)? Since Coinbase itself didn't add the new currency it can't get the new data. Hence it's causing a Bad Request error.
Maybe it would make sense to add a new plugin that supports Coinbase Pro and is named as such to avoid any confusion with other plugins?
I think calling https://api.gdax.com/accounts and then later using the Coinbase (non-PRO) API is mixing two different API endpoints for different purposes. There is a Coinbase Plugin for MoneyMoney which would be used to call that endpoint (I would assume, didn't check the plugin code).
What do you think?

It is the Coinbase API, the Coinbase PRO API (which really IS the GDAX API with a new address) unfortunately does not have an exchange rates endpoint: https://docs.pro.coinbase.com (neither did the GDAX API).
So the problem won't really go away when switching the GDAX endpoints to the Coinbase Pro endpoints...

So we'd need a new plugin specifically for the Coinbase Pro API if I am not mistaken. Maybe this endpoint can be used to catch a most recent trade value and use it for the Last quote in MoneyMoney:
https://api.pro.coinbase.com/products/BTC-EUR/ticker
Comparing the ticker price with the price displayed on the WebUI, they seem to match and could be used?
I wish I could give it a go but using the App Store version of MoneyMoney I am not able to disable signature checks (unless you know a way or how to create a signature?).

The ticker API could certainly be used for the time being but I wouldn't be too happy with it. Semantically, it does not express what I wan't to see in MoneyMoney - the actual worth of my cryptocurrencies in EUR. The ticker API just gives you the current bid and ask prices and sizes, which doesn't mean that you could convert your cryto assets to EUR with the current bid price (which is only the case when the current bid size is greater than your portfolio). But the Coinbase Exchange rate does mean exactly that, at least that was my understanding.
Regarding signatures: Only the author of MoneyMoney can sign extensions, but as long as you can verify that you bought the app they will happily send you a license for the non-appstore version.

What if we used the ticker to fetch the current price and get the actual value based on your account portfolio. We have the amount of each coin and use the ticket to calculate the value to display in MoneMoney.
Sent them a mail and waiting for a response, may take a while the automated reply said :-)

Went ahead and checked again and it seems you are already doing it in a two-step process. You fetch all accounts (CryptoCurrencies and real ones). If the currency you find is the native one in MoneyMoney (e.g. EUR), you handle it different since there is no exchange rate. For all others you fetch the current trade value from the exchange API:
https://github.com/leoschweizer/moneymoney-gdax/blob/master/GDAX.lua#L76
Instead of taking this as a fixed value, we can just use the quantity available in our Account listing:
https://github.com/leoschweizer/moneymoney-gdax/blob/master/GDAX.lua#L78
and call the ticker API to fetch the current price, then multiply by quantity to get the same result.
I'd think making this a new moneymoney-coinbasepro extension would make sense since it's not related to GDAX anymore with all changes done.
If/once I get my license I can aid in testing if you need a volunteer.

I think you really overestimate the difference between GDAX and Coinbase Pro :wink: This was just a rebranding. The API is exactly the same. All that needs to be changed in that regard is the API base url.
That alone would however still cause the current exchange rate problem, since this calculation always did use the Coinbase API which currently causes this problem but is itself unrelated to the GDAX <-> Coinbase Pro rebranding. We could as well use a completely different source for exchange rates since this never was based on any Coinbase account specific information. It was merely used to multiply your GDAX account balances with an exchange rate to get a fiat currency.
So all that needs to be done is 1) change the API endpoints and 2) integrate another exchange rate source (e.g. the Coinbase Pro ticker API but it could be any exchange rate API) and maybe 3) rename the repository / extension :wink:

I guess you are right, overthinking it it seems  😉. I just thought if we brand it a Coinbase Pro extension it would make sense to also ask the Coinbase Pro ticker even though it may not really be required to show a proper value for the coins a user has. So in short:

Yes
Coinbase Pro Ticker preferred to stay in the same "realm"
Yes 😄

Thanks for your feedback! I may look into it but it would be mostly a copy of the existing GDAX extension so maybe it would be best if you took it under your repository. Don't wanna steal any work you have done under a repo in my name - while keeping most of the code the same. But that'll be your call.

@TheSerapher if you want to make the adjustments, that's what pull requests are for 😉

You'll create a new repository or just rename this one?
I'll have to wait for a license but I can give it a go then.

I would rename this repository once the changes are made, because it doesn't make sense to keep it separate (GDAX no longer exists under this name).

True. I'll see what I can do!

I've started a branch with the necessary adjustments.
https://github.com/leoschweizer/moneymoney-gdax/tree/feature/coinbase-pro-support
Only the exchange rate API fix is missing.

Awesome! Once ready I will install it and give it a go. Thanks for your work on this, makes my life a lot easier 😃

The extension for Coinbase Pro is now available at https://moneymoney-app.com/extensions/
You will have to remove your GDAX account from MoneyMoney and add a new account for Coinbase Pro.

Got me the license but installed the Signature version anyway. Works! Thanks a lot :-)


2200301 - Tributary to Hammett Creek

[x] road permit holder is Canfor and ff-id = R01860
[x] special characters in the comments. spent hours troubleshooting problems that arise in report tables before due to this. Nightmare. important to remove
[x] loaded  gps points and tracks to the QGIS project to review location to see if it is correct (its not - see waypoint). Please adjust and repopulate spreadsheet coordinates

Closed via 24045e5b016c1585d480241f26b116805f0ab08b


[labs/observers] Consider adding polyfill for ResizeObserver

Should this be an RFC?

[X] This is not a substantial change

Which package is this a feature request for?
Observers (@lit-labs/observers)
Description
Currently, the controller for the ResizeObserver simply logs a console warning when a browser appears not to support the API, and there's no way to alter this behavior.  This means that, in order to load a polyfill dynamically, you need to hook into connectedCallback anyway, which defeats a chunk of the convenience the observers package provides.
It would be nice if the package either:

Builds in polyfill support (similar to the virtualizer package), or
Provides an option to pass a function or other means to dynamically import a user-defined polyfill.

Alternatives and Workarounds
It's possible to either:

Load a polyfill in connectedCallback, which has the disadvantages described in the previous section, or
Load a polyfill globally, which may override the polyfill inside other packages (e.g. virtualizer)

I thought we were going to remove the ResizeObserver from virtualizer. @usergenic @graynorton ?
Including polyfills in libraries is usually a bad idea. It ends up impacting apps that support only browsers that support the feature, and it bakes in a specific polyfill when an app may want to choose a different one, or other libraries may bundle one.
Loading the polyfill globally is generally the right approach. Only the app knows the browser environments it's deployging to.

The code in the virtualizer comments that the polyfill was "lightly modified", so perhaps it was only included because the released version didn't work for some reason?  The only one I see commonly used is https://github.com/que-etc/resize-observer-polyfill.
The global approach is fine I think.  Perhaps the only reason we do it dynamically is not to collide with virtualizer (cc @bramkragten).

For the record, yes we do plan on removing the ResizeObserver polyfill from Virtualizer, and I think we'll try to get that done relatively soon.  The one we provide is currently a tweaked version that I believe enabled more reliable observation through layers of Shadow DOM etc.  I don't know if it is vended elsewhere in a ready-to-go form yet, but we'll ensure that there's an option before we remove it.

Yes, the history here is that neither of the two most widely used polyfills properly supported detection within shadow roots, and the one that performed better in our tests and was most straightforward to patch was essentially unmaintained.
This is why we ended up forking the polyfill and shipping it with the library; we weren't interested in vending and supporting a general-purpose ResizeObserver polyfill, but we needed it for IE support. We came up with a mechanism for auto-loading it that added minimal weight to the library itself and didn't affect anything outside of Virtualizer, but with native support now at close to 100% due to the declining usage of IE11 and legacy Edge, we've decided to remove the auto-loading functionality and instead direct developers who need to support these older browsers to conditionally load it themselves.
@usergenic @justinfagnani I actually don't see much upside in removing the polyfill from the Virtualizer package altogether as long as IE11 and old Edge are still in Lit's official support matrix. A logical time to remove would be when the support matrix changes, presumably with a Lit 3.0.

Thanks for the info @graynorton !
The more things that pop up that are constrained by IE support (WeakRef use in lit-labs/context for example), the more I want to just remove IE support altogether sooner rather than later.


@MaxWofford's clubs for the week

See #244 for full details.
Check off each club once the requirements in #244 are met. These are
@zachlatta's clubs:

[ ] San Leandro High School
[ ] Armijo High School
[ ] South Brunswick High School

@zachlatta

These are @zachlatta's clubs:

Whoops, these are actually your clubs. My issue is the one that's incorrect.

I take that back. They were both correct.

I'm gonna steal Glenaeon Rudolf Steiner School back, since I've already been talking with him about this.

As discussed with @jonleung, I'll be contacting @gemmabusoni about submitting her case study

Closing this because the week ended.


Bytecode to boogie tests

This fixes a few bugs and adds unit tests to the bytecode_to_boogie translator.

@bors-libra r+

:pushpin: Commit ef46bf5 has been approved by sblackshear

:hourglass: Testing commit ef46bf5160ccfe444f6cb8f3ff67d7479e0fd6d1 with merge 189ca919b9a8f0cf65b0a6b70c9e73c90b81307e...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: sblackshear
Pushing 189ca919b9a8f0cf65b0a6b70c9e73c90b81307e to master...


add mcs example

Signed-off-by: lmxia<EMAIL_ADDRESS>
What type of PR is this?
Example add.
What this PR does / why we need it:
Add multi cluster service related examples.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
May add doc description about this example, notice multi-cluster-service has prerequiement for network condition.
First apply workload to child cluster.
kubectl clusternet applay -f examples/scheduling-with-mcs-api/scheduling
Then apply service import object in parent cluster.
kubectl create -f examples/scheduling-with-mcs-api/service-import.yaml

Codecov Report

Merging #455 (0dd87b5) into main (3185d87) will decrease coverage by 0.04%.
The diff coverage is n/a.

:exclamation: Current head 0dd87b5 differs from pull request most recent head e0cda31. Consider uploading reports for the commit e0cda31 to get more accurate results

@@            Coverage Diff             @@
##             main     #455      +/-   ##
==========================================
- Coverage   13.59%   13.55%   -0.05%
==========================================
  Files          62       62
  Lines        6678     6678
==========================================
- Hits          908      905       -3
- Misses       5700     5702       +2
- Partials       70       71       +1

Impacted Files
Coverage Δ

...ernet/clusternet/pkg/hub/apiserver/shadow/utils.go
89.28% <0.00%> (-10.72%)
:arrow_down:

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

The doc will be updated in clusternet/website#36.


Ensure delay timer is initialised

Work around an issue in 2020.1 where constructor functions aren't being called.
Fixes: Xilinx/PYNQ-Dev#281

Codecov Report

Merging #1142 into image_v2.6.0 will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           image_v2.6.0    #1142   +/-   ##
=============================================
  Coverage         47.82%   47.82%
=============================================
  Files                25       25
  Lines              4604     4604
  Branches            762      762
=============================================
  Hits               2202     2202
  Misses             2346     2346
  Partials             56       56

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 af369bb...8f7bd9f. Read the comment docs.


Handle references with missing type

Description
This change fixes an issue with the SQL data provider where we would null-ref when handling a request like
POST https://localhost:44348/Patient
Content-Type: application/json

{
    "resourceType": "Patient",
    "managingOrganization": {
        "reference": "abc"
    }
}

where the type of the managingOrganization reference is missing.
This requires a change to the schema because a table column needs to be made nullable.
In addition, in cases where the a reference type can only be a single type - like in the above example, which can only be of type Organization - we automatically add the type to the search parameter index in the DB.
Finally, for the SQL provider, for searches like Patient?organization=abc, turn this into Patient?organization=Organization/abc.
Related issues
Addresses #1458

            allowedReferenceResourceTypes.Any())

This can be changed to allowedReferenceResourceTypes?.Count >0 I would assume

Refers to: src/Microsoft.Health.Fhir.Core/Features/Search/TypedElementSearchIndexer.cs:203 in 4640916. [](commit_id = 464091685ef57cc91d8e167b164e6b4297cc64f3, deletion_comment = False)

        _searchParameterDefinitionManager = searchParameterDefinitionManagerResolver();

nit: looks like this can be removed as well.

Refers to: src/Microsoft.Health.Fhir.Core/Features/Search/Expressions/Parsers/SearchParameterExpressionParser.cs:41 in a8d18a5. [](commit_id = a8d18a5ac250807fb5c1fb2b9bd6570f136584da, deletion_comment = False)

Thank you for the review and feedback , @Ivanidzo4ka!


478387-FRDRSG1543

Before opening:

Search for duplicate or closed issues
Validate and lint any HTML to avoid common problems
Read the contributing guidelines

Bug reports must include:

Operating system and version (Windows, macOS, Android, iOS, Win10 Mobile)
Browser and version (Chrome, Firefox, Safari, IE, MS Edge, Opera 15+, Android Browser)
Reduced test case and suggested fix using CodePen or JS Bin

Please don't test on public repos!  Not helpful to anyone 😟


[Bug] Doesn't compile with iOS 13

Description
I'm using flutter_video_compress: 0.3.7+8 to deploy to an iPhone XS Max on iOS 13. It works in the Android simulator, but I get the following compilation errors in Xcode 11:
Xcode's output:
↳
    leveldb-library-fugunjpkrwxlsrdkdyyylozzzgaq
    /Users/karsten/workspaces/me/services/company-c/apps/ca_test_flutter/ios/Pods/Regift/Regift/AVAssetImageGeneratorTimePoints.swift:12:5: warning: 'public' modifier is redundant for instance method declared in a public extension
        public func generateCGImagesAsynchronouslyForTimePoints(timePoints: [TimePoint], completionHandler: @escaping AVAssetImageGeneratorCompletionHandler) {
        ^~~~~~~

    leveldb-library-fugunjpkrwxlsrdkdyyylozzzgaq
    /Users/karsten/workspaces/me/services/company-c/apps/ca_test_flutter/ios/Pods/Regift/Regift/Regift.swift:251:45: error: missing argument label 'preferredTimescale:' in call
                let time = CMTimeMakeWithSeconds(seconds, Constants.TimeInterval)
                                                ^
                                                          preferredTimescale:
    /Users/karsten/workspaces/me/services/company-c/apps/ca_test_flutter/ios/Pods/Regift/Regift/Regift.swift:303:46: error: missing argument label 'preferredTimescale:' in call
            let tolerance = CMTimeMakeWithSeconds(Constants.Tolerance, Constants.TimeInterval)
                                                 ^
                                                                       preferredTimescale:
    /Users/karsten/workspaces/me/services/company-c/apps/ca_test_flutter/ios/Pods/Regift/Regift/Regift.swift:251:45: error: missing argument label 'preferredTimescale:' in call
                let time = CMTimeMakeWithSeconds(seconds, Constants.TimeInterval)
                                                ^
                                                          preferredTimescale:
    /Users/karsten/workspaces/me/services/company-c/apps/ca_test_flutter/ios/Pods/Regift/Regift/Regift.swift:303:46: error: missing argument label 'preferredTimescale:' in call
            let tolerance = CMTimeMakeWithSeconds(Constants.Tolerance, Constants.TimeInterval)
                                                 ^
                                                                       preferredTimescale:
    /Users/karsten/workspaces/me/services/company-c/apps/ca_test_flutter/ios/Pods/Regift/Regift/AVAssetImageGeneratorTimePoints.swift:12:5: warning: 'public' modifier is redundant for instance method declared in a public extension
        public func generateCGImagesAsynchronouslyForTimePoints(timePoints: [TimePoint], completionHandler: @escaping AVAssetImageGeneratorCompletionHandler) {
        ^~~~~~~

    leveldb-library-fugunjpkrwxlsrdkdyyylozzzgaq
    note: Using new build systemnote: Planning buildnote: Constructing build description

Could not build the precompiled application for the device.

Error launching application on o8z.

Platform
iOS
Backtracking step (if has)

Include flutter_video_compress: 0.3.7+8 in pubspec.yaml
Build with Flutter 1.9 & Xcode 11
Get Compilation error

Expected solution
This project uses the Regift iOS Cocoa Pod, I think in version 1.5. Now in version 1.6, these problems are solved. So I added this line to my ios/Podfile:
pod 'Regift', '1.6'

But Regift 1.6 has iOS 11.1 as the minimum requirement. This clashes with the rest of Flutter  which has iOS 8.0 as the minimum compilation target. I'm unable to change that to iOS 11.1. I can change the deployment target to 11.1, but that hasn't solved the error.

Is there any news about this?? I have the same issue and the solution of fixing it with xcode every time pubspec runs is not good enough for me

Is there any news about this?? I have the same issue and the solution of fixing it with xcode every time pubspec runs is not good enough for me

I had to migrate to this package: https://pub.dev/packages/video_compress
It's still new and does the same thing, but no problem so far.

Is there any news about this?? I have the same issue and the solution of fixing it with xcode every time pubspec runs is not good enough for me

I had to migrate to this package: https://pub.dev/packages/video_compress
It's still new and does the same thing, but no problem so far.

Testing it!

Is there any news about this?? I have the same issue and the solution of fixing it with xcode every time pubspec runs is not good enough for me

I had to migrate to this package: https://pub.dev/packages/video_compress
It's still new and does the same thing, but no problem so far.

Testing it!

Works for me! Thanks @nthungdev !!!!

Is there any news about this?? I have the same issue and the solution of fixing it with xcode every time pubspec runs is not good enough for me

I had to migrate to this package: https://pub.dev/packages/video_compress
It's still new and does the same thing, but no problem so far.

I'm getting an error, if I preview the video before compressing it I get "PlatformException(error, java.io.IOException: Failed to instantiate extractor., null)", could this be because of the preview theart that doesn't allow me to open the video again?

I'm getting an error, if I preview the video before compressing it I get "PlatformException(error, java.io.IOException: Failed to instantiate extractor., null)", could this be because of the preview theart that doesn't allow me to open the video again?

Possibly. Are you able to open the video after the compression?

Once I preview the video with native camera preview I’m unable to compress,
I get this error: PlatformException(error, java.io.IOException: Failed to
instantiate extractor., null). I tried to make a copy of the video, but
still fails
El El sáb, 11 abr. 2020 a la(s) 14:36, Hung Nguyen<EMAIL_ADDRESS>escribió:

I'm getting an error, if I preview the video before compressing it I get
"PlatformException(error, java.io.IOException: Failed to instantiate
extractor., null)", could this be because of the preview theart that
doesn't allow me to open the video again?
Possibly. Are you able to open the video after the compression?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rurico/flutter_video_compress/issues/59#issuecomment-612473253,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADK7AEAA635M3SCGTZJM4A3RMCTANANCNFSM4IZKYFZA
.

Same issue here, and seems duplicate with #56?

The problem still persists. Running the project on Xcode and fixing relevant errors worked, but sometimes I needed to repeat the process. Will there be any update to fix this?

Also after upgrading to flutter 1.22 and iOS 14, this version is not working anymore (flutter build ios throws a lot of errors. Android worked)

This error still exists and I can't build on IOS now

This error still exists and I can't build on IOS now


Remove duplicate library include

Remove duplicate

:+1: merged it in. Thx
Did it manually, to accommodate your commit message to our contribution guidelines.


Filter Query on a ManagedSet

I want to filter a "city" ManagedObject by it's "starts" ManagedSet.
var citiesQuery = new Query<City>()
      ..matchOn.starts.includeInResultSet = true
      ..matchOn.starts.isNotEmpty;
return new Response.ok(await citiesQuery.fetch());

But this code return all City records from database. Output is this.
[
  {
    "id": 1,
    "name": "București",
    "coordinates": "44.4377401,25.9545541",
    "country": {
      "id": 1
    },
    "starts": [
      {
        "id": 1,
        "start": {
          "id": 1
        },
        "end": {
          "id": 2
        }
      }
    ]
  },
  {
    "id": 2,
    "name": "Londra",
    "coordinates": "51.528308,-0.3817765",
    "country": {
      "id": 2
    },
    "starts": []
  },
  {
    "id": 3,
    "name": "Berlin",
    "coordinates": "52.5335457,13.5169484",
    "country": {
      "id": 3
    },
    "starts": []
  }
]

Please help me with this situation.

That's a good request. It's not currently possible, but it should be with (what I think) is a minor change and a slightly different syntax.
Currently, the query builder works by assigning properties in matchOn to 'matchers' (values/methods that start with the word where). So, adding isEmpty to the matcher wouldn't alter the query, but there should be a way to filter the result set in this way. I'll investigate a level of effort for making this work, as it should be a relatively common use case. The syntax will likely be something like this:
var citiesQuery = new Query<City>()
      ..matchOn.starts.includeInResultSet = true
      ..matchOn.starts = whereNotNull;

Ok. I wait a solution for that, beacause a have a project who needs this filter. Thanks.

I try to find a alternative solution like:
class City extends ManagedObject<_City> implements _City {
  int get startsnumber => startsnumber;
  void set startsnumber(un) {
    starts.length = un;
  }
}

class _City {
  @managedPrimaryKey
  int id;

  @ManagedColumnAttributes(indexed: true)
  String name;

  @ManagedColumnAttributes(unique: true)
  String coordinates;

  @ManagedRelationship(#cities)
  Country country;

  ManagedSet<Route> starts;
  ManagedSet<Route> ends;
}

And I get  NoSuchMethodError: Class 'ComparisonMatcherExpression' has no instance method '>'.
The code for controller is:
@httpGet
  getStartCities() async {
    var citiesQuery = new Query<City>()
      ..matchOn.starts.includeInResultSet = true
      ..matchOn.startsnumber = whereGreaterThan(0);

    return new Response.ok(await citiesQuery.fetch());
  }

The properties declared on City aren't database columns, so you won't be able to include those in a query. A workaround right now would either be fetching all of the cities and then filtering them in the application:
var q = new Query<City>()
  ..matchOn.starts.includeInResultSet = true;

var cities = (await q.fetch())
  .where((city) => city.starts != null)
  .toList();

Or it would be to do two queries of questionable performance:
var q = new Query<Route>()
  ..matchOn.cityStart = whereNotNull;
var allRoutes = await q.fetch();

var cityQuery = new Query<City>()
  ..matchOn.id = whereIn(allRoutes.map((r) => r.cityStart.id));
var cities = await cityQuery.fetch();

// pair together cities with their routes.

When you send me this message, I was trying first option :)). And it works.
I write more code, so I am able to do something wrong :)).
So please put in todo list :

filter empty for 'has one' and 'has many' relations
get in transident managed object attribute the count of rows of some ManagedSet from persistent type

P.S. I like Aqueduct and I want to use it in production. I am a novice in Dart and your project, but I was tried to look in source and in this moment is too much for me. I will try to contribuite in the future.

This is now resolved for 2.0