Service Worker cache offline is it possible with Grav ? and how ?

@rhukster
Sorry, but I wasn't clear with my first question about PWA , I would like to use a Service Worker that is a require to be PWA, by this way I could access to my Grav Website completely offline, and I don't know how to do that exactly because Grav it's dynamic without any static file, what will be the best tools for that, there are many tools like :
https://github.com/NekR/offline-plugin
https://github.com/GoogleChrome/workbox
https://github.com/GoogleChromeLabs/sw-precache
Thanks for your help !

Grav is written in PHP, which is stateless and runs under a webserver.  You would need to create your service worker in another language that can then run as a service.

I know this issue is old and closed but there is very little information out there on making grav into a PWA.

you CAN use a service worker to precache anything with static links, like /user/themes/mytheme/fonts/myfont.woff2 .
Since the service worker usually installs after the first page load that only really helps speed up other pages that use other fonts, other js scripts etc
you can NOT PREcache dynamic pages but you CAN cache them into the applications runtime cache as they are loaded first time.
make sure your server does NOT cache the service worker, setting a no-cache header or you will not be able to update it, something you need to do to cache bust when you update your content
This one https://googlechrome.github.io/samples/service-worker/basic/ does precache and runtime and works offline once the pages are cached. Some of the other service workers I found seemed to be out of date or just plain broken.
use the lighthouse audit in chrome dev tools to test it is working.
you need a site manifest to get some of the PWA stuff to work like "add to home screen"
the service worker goes into the site root, getting to to register at the bottom of the body of your base or homepage template seems to work for me. It should only update the service worker if it has changed (assuming you have done 3) and the browser cache doesn't interfere).

Hope that helps someone trying to add PWA features to a grav site in future.


[ConstraintSystem] Fix a bogus missing function call error message.

If there is no function expression written in the source code, telling the user to insert an explicit function call is really misleading. This can happen when attempting to match the type of a property with attached property wrapper against the wrappedValue type:
@propertyWrapper
struct StringWrappedValue {
  var wrappedValue: String
}

struct SR_14730 {
  @StringWrappedValue var value: () -> String
}

Previously, the constraint system attempted to diagnose a missing function call because the result of the function type () -> String matches the wrapped-value type String, but this isn't the right fix because there's no expression written in the code to actually call (not to mention the diagnostics code not knowing how to handle this case and crashing!).
Resolves: rdar://79051798 / SR-14730

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please test Windows platform

@swift-ci please test Windows platform


Added pyepsg as it is an optional dependency of cartopy needed by geo…

…array.

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender


refactor(recommend): refactor code, add pubsub test

AMLNG-831
Description
Increase Unit Test Coverage
Acceptance Criteria
Unit Test Coverage has been increase
How to test code
make ptest
Please Review
@aml-development/ozp-backend-team
Checklist

[x] Read CONTRIBUTING.md
[x] Write code to complete task
[x] Python Code is PEP8 Compliant
[x] Add unit tests for new code
[ ] All tests must pass before merging the pull request
[ ] At least 2 people reviewed code

:+1:
All tests pass

:+1:


Add simple traceback to VmException

Depends on https://github.com/lambdaclass/cairo-rs/pull/657

[ ] Update cairo-rs version

Codecov Report

Merging #188 (3b1cb67) into main (8f080da) will decrease coverage by 0.20%.
The diff coverage is 91.66%.

@@            Coverage Diff             @@
##             main     #188      +/-   ##
==========================================
- Coverage   82.57%   82.36%   -0.21%
==========================================
  Files          14       14
  Lines         746      709      -37
==========================================
- Hits          616      584      -32
+ Misses        130      125       -5

Impacted Files
Coverage Δ

src/instruction_location.rs
97.61% <ø> (ø)

src/lib.rs
0.00% <0.00%> (ø)

src/ids.rs
91.00% <90.00%> (+1.86%)
:arrow_up:

src/cairo_runner.rs
71.17% <100.00%> (ø)

src/relocatable.rs
100.00% <100.00%> (ø)

src/run_context.rs
100.00% <100.00%> (ø)

src/vm_core.rs
92.04% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


WeightedPolyline Opacity Changes

I am using a weighted polyline for a project and have them inside of a layer group that is turned on and off by the user. I have found that when removing and then adding the layer back to the map, the opacity of the weighted polyline increases each time the layer is added back. Is this a known issue?

This sounds like a new issue.  I'll look into it and hopefully have a fix soon.  Is this the 0.7.3 version (master branch) or the 1.0dev branch or both?

The master branch, specifically the 0.2.4 release. I haven't tried it with 1.0dev.
Here is a fiddle I made to show it: https://jsfiddle.net/wSytj/92/
click the map to remove and then add

Just pushed a fix.  Let me know if this works for you.

Works great!


overwriting the default entries in server.env generated by server create.

liberty:create generates the keystore password by default in target server.env file due to a change in Liberty. https://github.com/OpenLiberty/open-liberty/issues/1627.
When user has liberty.property.env.* in pom.xml, the target server.env will be overwritten and the generated keystore password is lost.
Similarly, there is also a generated server.env entry, WLP_SKIP_MAXPERMSIZE=true to avoid a warning msg in server log. https://github.com/cloudfoundry/ibm-websphere-liberty-buildpack/pull/245
We might need to consider to preserve these two environment variables if they are not set in the liberty.env.* properities.

So to be clear, this is not a new problem. We previously allowed the specification of a server.env file either through the default configDirectory or the serverEnvFile parameter. In both of those cases, any server.env file produced when the server was created would get overridden. Also, it is not liberty:create per se that generates the keystore password. It is the server create command provided by Liberty, which gets invoked by liberty:create.
It would be difficult for our plugin to determine what should and should not be kept in the server.env file located in the server. Especially when liberty:create is called multiple times to update the server configuration.

It is not a new problem.

This has been reported in ci.gradle issue https://github.com/OpenLiberty/ci.gradle/issues/459 and I have proposed adding an appendServerEnv parameter that defaults to false. Issue #832 will be used to implement the fix for ci.maven.

I think we can close this since there are fixes in Maven and Gradle already.


things to add/fix

To fix:

word detail view text wrapping leaves artifacts and doesn't  work correctly. (took lazy approach ;-) )
full help view dose not render.
I confused paginator with an sppiner  so whole paginator implementation needs to be removed.
some functions need to be cleaned up/ relocated.

to add:

make searching for word an gorutine and  display spinner while searching.
kanji searching dose not work (not supported by library )
jisho api dose not sometimes that word is English so it searches as Japanese. (ex. china -> by the way).

**for any more addition please make new issue **

Text wrapping/rendering
found out that  charmbraclet has glamour package witch solves all problems with wrapping and color rendering.


modImplementation for addons is no longer functional

This effects both Origins mod and FallFlyingLib.
Bintray shut down download link functionality recently and because of this mod developers are unable to download these two mod sources to their workspace.
See for FallFlyingLib fix:
#286

Currently, I'd recommend switching to Jitpack for now... We're looking into doing our own Maven but it's not our main priority, and Jitpack works fine.

Fixed in 0.7.2. :)

Just a small update in our part: the Cafeteria Development Team finally managed to set up our own Maven Repository, so expect FallFlyingLib to be released from now on.


Corrupting item caused build to be unable to be opened

Added "1 extra curse" corruption to a carnage heart on a slayer build and I'm now getting the following error any time I attempt to open the build.
Error:
In 'OnFrame': Modules/Calcs.lua:256: bad arguement #1 to 'ipairs' (table expected, got nil)
stack traceback:
[C]: in fuction 'ipairs'
Modules/Calcs.lua:256: in fuction 'buildOutput'
Classes/CalcsTab.lua:378: in fuction <Classes/CalcsTab.lua:363>
Modules/Build.lua:484: in function 'CallMode'
Modules/Main.lua:248: in function <Modules/Main.lua:239>
[C]: in function 'Pcall'
C:\ProgramData\Path of Building\Launch.lua:98: in function <C:\ProgramData\Path of Building\Launch.lua:95>
Press enter/Escape to Dismiss, or F5 to restart Application.
Restarting or completely closing out the program and reopening does not chance this.

Sorry about that; it seems I forgot to test Punishment against the modifications to the buff code that were in 1.4.31. That curse causes all kinds of problems, given its debuff/buff duality. Anyway, it will be fixed in the next update; I'll try and get that out soon.


[Keyboard Manager]: CCA fails for keyboard focus on combo boxes.

Microsoft PowerToys version
0.45.0
Running as admin

[ ] Yes

Area(s) with issue?
Keyboard Manager
Steps to reproduce
Steps to reproduce
Test Environment:
OS: Windows 11 Version Dev
App version: 0.45.0
App name: PowerToys
Screen Reader: Narrator
Tool: Colour Contrast Analyzer
Repro Steps:

Launch 'PowerToys' application.
In General tab, navigate to 'Keyboard manager’ button and activate.
Navigate to ‘Remap a key’ and activate it using enter/Space key.
Now, Remap a key dialog box will appear.
Turn on Colour Contrast Analyzer.
Navigate 'Key1' combo box.
Observe the issue.

Note:
Same issue repro with Other Comb box.
User Impact: -> Priority 2
Visual user is impacted as the Color convey the information.
Guidelines Reference:-
https://www.w3.org/TR/WCAG21/#non-text-contrast
Attachments:

✔ Expected Behavior
The Minimum Contrast ratio should be at least 3:1.
❌ Actual Behavior
The Current Contrast ratio is 2.8:1 > 3:1
Other Software
No response

Closing as fixed on v 0.76.2:


Simplify the lifetimes around engines.

Description
Checklist

[ ] I have linked to any relevant issues.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have updated the documentation where relevant (API docs, the reference, and the Sway book).
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
[ ] I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
[ ] I have requested a review from the relevant team or maintainers.

This is probably more maintainable but is it worth the extra cloning?


docs(openssl): openssl_pkcs7_encrypt accepts also an array of certifi…

…cates

@kesselb Please run php-cs-fixer to fix code style problems

@isfedorov done


chore: add name-squatting policy

Generally I am positive on adding an item to the FAQ that covers this.


FTXUI fails to compile

Describe the bug
The third_party dependency FTXUI fails to compile.
To Reproduce
Steps to reproduce the behavior:

git clone https://github.com/eclipse-ecal/ecal
git submodule init && git submodule update
cd ecal && mkdir _build && cd _build
cmake .. -DCMAKE_BUILD_TYPE=Release -DECAL_THIRDPARTY_BUILD_PROTOBUF=OFF -DECAL_THIRDPARTY_BUILD_CURL=OFF -DECAL_THIRDPARTY_BUILD_HDF5=OFF -DECAL_THIRDPARTY_BUILD_QWT=OFF
make VERBOSE=1
Build fails with:

[  1%] Building CXX object thirdparty/ftxui/CMakeFiles/dom.dir/src/ftxui/dom/automerge.cpp.o
cd /tmp/ecal/_build/thirdparty/ftxui && /usr/bin/c++  -I/tmp/ecal/thirdparty/ftxui/include -I/tmp/ecal/thirdparty/ftxui/src -O3 -DNDEBUG   -Wall -Wextra -std=c++14 -Wall -Wextra -pedantic -Werror -Wmissing-declarations -Wdeprecated -Wshadow -MD -MT thirdparty/ftxui/CMakeFiles/dom.dir/src/ftxui/dom/automerge.cpp.o -MF CMakeFiles/dom.dir/src/ftxui/dom/automerge.cpp.o.d -o CMakeFiles/dom.dir/src/ftxui/dom/automerge.cpp.o -c /tmp/ecal/thirdparty/ftxui/src/ftxui/dom/automerge.cpp
In file included from /tmp/ecal/thirdparty/ftxui/include/ftxui/dom/elements.hpp:162,
                 from /tmp/ecal/thirdparty/ftxui/src/ftxui/dom/automerge.cpp:4:
/tmp/ecal/thirdparty/ftxui/include/ftxui/dom/take_any_args.hpp: In function ‘ftxui::Elements ftxui::unpack(Args ...)’:
/tmp/ecal/thirdparty/ftxui/include/ftxui/dom/take_any_args.hpp:27:33: error: fold-expressions only available with ‘-std=c++17’ or ‘-std=gnu++17’ [-Werror]
   27 |   (Merge(vec, std::move(args)), ...);
      |                                 ^~~

Expected behavior
Should build.
Desktop (please complete the following information):

OS: Ubuntu
Version 22.04
gcc 11.2.0

Additional context
It seems that the "-std=c++14" compiler flag is populated all the way through to ftxui from the ecal main CMakeLists.txt.
Removing the two lines make it compile successfully.
  set(CMAKE_CXX_STANDARD 14)
  add_definitions(-Wall -Wextra -std=c++14)

I am wondering why it doesn't fail in the github pipelines.

I tried it on a fresh Ubuntu VM and encountered the same issue. Thanks for reporting 👍

I am wondering why it doesn't fail in the github pipelines.

I am puzzled, too. It neither shows that issue on GH Actions nor on my main development Ubuntu 22.04.

I recommend that we remove / modify
https://github.com/eclipse-ecal/ecal/blob/f87f8242db312d62743c0b406cc90b856a7ad912/CMakeLists.txt#L59-L60
Also -Wall,  -Wextra should not be forced on users.
Instead I propose to add to mon_cli
target_compile_features(foxglove_websocket PRIVATE cxx_std_17)

and to the other ones whatever they need (e.g C++14) (PUBLIC / PRIVATE depending on if they are a lib / executable, ...)
target_compile_features(foxglove_websocket PUBLIC cxx_std_14)

I removed the hardcoded C++14 settings in  PR #832. I didn't set the target compile features, as there is a pending Refactoring PR (#829) that we should merge first.


Moving vertexai (2), compute(1), dataproc(2) resources to v1beta1 version

Moving vertexai 2 resources to v1beta1 version

[ ] google_vertex_ai_endpoint
[ ] google_vertex_ai_featurestore_entitytype_feature

Moving compute 1 resources to v1beta1 version

[x] google_compute_router_peer

Moving dataproc 2 resources to v1beta1 version:

[ ] google_dataproc_cluster_iam_member
[ ] google_dataproc_job_iam_member

All these resources fails with an error:
Cannot determine region: set in this resource, or set provider-level 'region' or 'zone'.:

We need to open an issue to discuss supporting the region and zone fields in providerconfig.
@sergenyalcin

Blocked by: https://github.com/upbound/provider-gcp/issues/212

The google_compute_router_peer resource is configured in this PR: https://github.com/upbound/provider-gcp/pull/341


getkap.co has no gifs using kap created with kap

would love to see a gif of how to use Kap made with Kap on the Kap homepage :)

We'll update the homepage (with gifs) along with the v2 release 🙌
Would you mind closing this issue and re-opening it on the kap-website repo? https://github.com/wulkano/kap-website ❤


Jenv use breaks PATH

I'm attaching the output of a couple of commands.
Let me know if you need more info.
C:\Users\hconstantin>java -version
java version "1.8.0_301"
Java(TM) SE Runtime Environment (build 1.8.0_301-b09)
Java HotSpot(TM) 64-Bit Server VM (build 25.301-b09, mixed mode)

C:\Users\hconstantin>cd dev

C:\Users\hconstantin\dev>jenv.bat use jdk11
Changed to jdk11

C:\Users\hconstantin\dev>java -version
'java' is not recognized as an internal or external command,
operable program or batch file.

Closed because of:
https://github.com/FelixSelter/JEnv-for-Windows/issues/2

Closed because of:
https://github.com/FelixSelter/JEnv-for-Windows/issues/2


[HOLD for payment 2023-12-07] [HOLD for payment 2023-12-06] Wallet - On first try, SSN is verified after second attempt to submit the form

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: unable to check production
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Issue found when executing PR https://github.com/Expensify/App/pull/31401
Action Performed:

Go to staging.new.expensify.com
Login with an expensifail account.
Go to Settings > Wallet > Enable Wallet.
Follow the flow until your reach the "Additional details" page.
Fill the form with following data: First name: Alberta, Last name: Bobbeth, SSN: 3333, Any other information for the rest of the fields.
Tap on "Save & continue".

Expected Result:
The user should be taken to the nest step without any issues.
Actual Result:
An error message appears the first time the user hits the "Save & continue" button. If the user taps on it again without any change on the fields, then the SSN is verified.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/dbd0c478-d1ce-4e92-8804-ae064ee9b3be
View all open jobs on GitHub

We CPed a revert. Closing.

This does not seem to be a regression from https://github.com/Expensify/App/pull/31401. It just happen that it was found during its QA

@luacmartins Apparently this failed on the latest staging https://github.com/Expensify/App/issues/32266#issuecomment-1834427850 maybe there is some misunderstanding with expected resutlts, but I am reopning just so we can check if this is fine.
Feel free to close again if all if expected

This is expected. Closing!


The cursor disappears in the preview panel when resizing yazi.

What system are you running Yazi on?
Linux Wayland
What terminal are you running Yazi in?
kitty 0.35.2
yazi --debug output
Yazi
    Version: 0.3.0 (4150f401 2024-08-09)
    Debug  : false
    OS     : linux-x86_64 (unix)

Ya
    Version: 0.3.0

Emulator
    Emulator.via_env: ("xterm-kitty", "")
    Emulator.via_csi: Ok(Kitty)
    Emulator.detect : Kitty

Adapter
    Adapter.matches: Kitty

Desktop
    XDG_SESSION_TYPE: Some("wayland")
    WAYLAND_DISPLAY : Some("wayland-0")
    DISPLAY         : Some(":0")

SSH
    shared.in_ssh_connection: false

WSL
    /proc/sys/fs/binfmt_misc/WSLInterop: false

Variables
    SHELL              : Some("/usr/bin/fish")
    EDITOR             : Some("nvim")
    YAZI_FILE_ONE      : None
    YAZI_CONFIG_HOME   : None
    ZELLIJ_SESSION_NAME: None

Text Opener
    default: Some(Opener { run: "${EDITOR:-vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })
    block  : Some(Opener { run: "${EDITOR:-vi} \"$@\"", block: true, orphan: false, desc: "$EDITOR", for_: None, spread: true })

tmux
    TMUX   : false
    Version: 3.4

Dependencies
    file             : 5.45
    ueberzugpp       : 2.9.6
    ffmpegthumbnailer: 2.2.2
    magick           : 7.1.1-36
    fzf              : 0.54.3
    fd               : 10.1.0
    rg               : 14.1.0
    chafa            : 1.14.1
    zoxide           : 0.9.4
    7z               : 17.05
    7zz              : No such file or directory (os error 2)
    jq               : 1.7.1

--------------------------------------------------
When reporting a bug, please also upload the `yazi.log` log file - only upload the most recent content by time.
You can find it in the "/home/tlss/.local/state/yazi" directory.

Did you try the latest nightly build to see if the problem got fixed?
Yes, and I updated the debug information above (yazi --debug) to the nightly that I tried
Describe the bug
When the window size is reduced, the out-of-bounds cursor cannot be displayed in the preview pane.
Preview panel cursor shows in fullscreen.

Preview panel cursor disappears after resizing the window.

Minimal reproducer

Open yazi in fullscreen.

Go to the bottom of this directory.
Resize the yazi window.

Anything else?
No response

Anyone wants to give it a go?
When the terminal size changes, the resize command is triggered:
https://github.com/sxyazi/yazi/blob/dac72eb39a846d15db7f1a7c54e9261675a90e53/yazi-fm/src/app/commands/resize.rs#L16
and this method calls self.current_or_mut(opt.tab).repos:
https://github.com/sxyazi/yazi/blob/dac72eb39a846d15db7f1a7c54e9261675a90e53/yazi-fm/src/app/commands/resize.rs#L21
https://github.com/sxyazi/yazi/blob/dac72eb39a846d15db7f1a7c54e9261675a90e53/yazi-core/src/manager/commands/hover.rs#L30
I think we can handle it there.

I noticed that the cursor disappears in both the preview and current panels when resizing the window.
Thanks for the hint. I checked the repos code, which should reposition the cursor, but it doesn't seem to fix the issue.
From what I understand, the cursor's position is correct after resizing, but the displayed content isn't. I noticed that the offset variable represents the starting position of the first file shown in the current panel within the entire file list. I managed to fix the cursor disappearing in the current panel by adjusting the offset value, but the issue persists in the preview panel.

We can start by only implementing the current panels.


Update Seldon Community Operator to 1.2.2

Signed-off-by: Clive Cox<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.

Flat operator directory structure is obsolete from 23-rd of October 2019, only nested directory structure will be accepted.

/test CVP


feat: gasbot disable chain switching

Description
Update @gasbot/widget to version 0.3.19. Add avoidChainSwitching prop to ensure that users cannot switch chains away from Mode. This allows users to sign permit messages on any stablecoins held on other Gasbot-supported chains to receive Mode ETH.
Type of change

[ ] Docs change / dependency upgrade
[ ] Configuration / tooling changes
[ ] Refactoring
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Requires changes in customer code

High-level change(s) description - from the user's perspective
The user will be able to click the "Get Gas" button in the navbar which opens the Gasbot modal. The modal scans the connected wallet for stablecoins across various networks that can be used to obtain Mode ETH.
Related Issue(s)
None
Related pull request(s)
https://github.com/ionicprotocol/monorepo/pull/212

Need to close to re-assess tokens affected by this prop.


Seems dapr cli has not deregister consul after dapr cli was closed?

Note: If you have a general support question and are looking for a quicker response, please checkout our discord channel for answers from the community:
https://aka.ms/dapr-discord
In what area(s)?

CLI version: 1.9.1
Runtime version: 1.9.0

Ask your question here
when i use dapr cli slim mode to started a server, i actually this server had register to consul successfully, but when i close dapr cli use ctrl+c, i still saw the server i just close has still in consul's serverlist, is somthing i forgot?

Can you double-check that the daprd process was killed? I’ve seen situations when running with the Dapr CLI (if the runtime crashes) that daprd is still around: sudo ps ax | grep daprd
Just double-checking if the issue is a lingering daprd process, or an issue with the component

Closing this as a duplicate of https://github.com/dapr/components-contrib/issues/2489 Which is newer, but has more details


Integrated terminal always eats up tab key

#8222

It is not possible to tab out of the terminal which can make it hard for keyboard accessibilty
This also makes the actions in the terminal header not accessible by keyboard

The editor has a command to use tab key for moving focus, maybe something similar is needed for the terminal.

Duplicate of https://github.com/Microsoft/vscode/issues/6731


Save and Sync button - HTTP ERROR 500

Hi there,
We did install this plugin. We are able to connect and create the fields mapping but when we click on Save and Sync button on an order we are getting the following error:
_This page isn’t working
magasales.com is currently unable to handle this request.
HTTP ERROR 500_

Having the same issue here - did you get anywhere?

having the same issue, the error printed to the logs is the following:
PHP message: PHP Fatal error:  Uncaught Error: Call to a member function read() on null in /var/www/test-ecommerce/wp-content/plugins/woocommerce/includes/legacy/abstract-wc-legacy-order.php:628
Stack trace:
#0 /var/www/test-ecommerce/wp-content/plugins/WooCommerce-Salesforce-integration-master/includes/models/class-nwsi-order-model.php(23): WC_Abstract_Legacy_Order->init(1639)
#1 /var/www/test-ecommerce/wp-content/plugins/WooCommerce-Salesforce-integration-master/includes/controllers/core/class-nwsi-salesforce-worker.php(80): NWSI_Order_Model->__construct(1639)
#2 /var/www/test-ecommerce/wp-content/plugins/WooCommerce-Salesforce-integration-master/includes/views/class-nwsi-orders-view.php(45): NWSI_Salesforce_Worker->handle_order(1639)
#3 /var/www/test-ecommerce/wp-includes/class-wp-hook.php(300): NWSI_Orders_View->process_order_sync_request(1639)
#4 /var/www/test-ecommerce/wp-includes/class-wp-hook.php(323): WP_Hook->apply_filters(NULL, Array)
#5 /var/www/test-ecommerce/wp-includes/plugin.php(453): WP_H" while reading response header from upstream

I am encountering this issue as well, HTTP Error 500.

Hello,
It still does not work. Is there any planned fix in the nearest feature?

Same as issue #12.
Should be fixed with https://github.com/Neuralab/WooCommerce-Salesforce-integration/commit/6d68c166ca073711da0627e19bc8dadad4226b38 commit.


2.5 upstream merge

What this PR does / why we need it:
Merge upstream commits on release-2.5 branch.
Release note:

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 5 committers have signed the CLA.:white_check_mark: briantopping:x: chethanv28:x: sipriyaa:x: adikul30:x: divyenpatelYou have signed the CLA already but the status is still pending? Let us recheck it.


Allow blank username

Right now metabase will not allow adding a connection with no username but this is a valid config.

Hi @eni9889 , what does your server's configuration XML look like? Does it work if you enter "default" into the Metabase database connection setup screen?


is autoPlay option available ? moving items by its own ?

It is not available yet. It can be used with when scroll controller is added to the widget.


The profile page should make it more obvious that they can edit it

Is your feature request related to a problem? Please describe.
Yes https://github.com/freeCodeCamp/freeCodeCamp/issues/49132, it's easy to add broken results for beginners.
Describe the solution you'd like
Add button that navigate to the portfolio section in the setting page, this will be QoL improvement, if a URL becomes broken or image becomes broken, and camper can move fast to that section of the page
Describe alternatives you've considered
The ability to edit it in the portfolio page itself?
Additional context
No response

i would like to work on this issue. Please do assign if no one else is working on it right now.

Only issues with the https://github.com/freeCodeCamp/freeCodeCamp/labels/help wanted or https://github.com/freeCodeCamp/freeCodeCamp/labels/first timers only label is open for contribution. The first comprehensive PR created will be reviewed and merged. We typically do not assign issues to anyone other than long-time contributors.
If you would like to contribute, and have not read the contributors docs, please do so here: https://contribute.freecodecamp.org/#/
If you have any issues with contributing, be sure to join us on the contributors channel, or on the contributors sub-forum

Based on my understanding, we are moving the portfolio setting's to the portfolio itself.

That what Sem suggested, if you are okay with it, I can open this for contribution or start working on it.
I really don't mind if it's button that send to setting page, or profile page has an edit button for this, just want a better experience for the users.

Yes, we can open an umbrella issue with some user stories from our discussions. Once that is done, I could go ahead with a mock/wireframe and the feature prs could follow up.

Point mentioned by Tom in the dev meeting, the setting page isn't well organized, it would be better to move the setting related to profile page, and let campers edit it in the profile section.

Hey @ahmaxed,
What do you think about adding a button to the profile page that takes you to the settings page to edit your profile?
Or does it not fit the upcoming design?

The profile settings should be relocated to the profile page.
Here is the mock:
https://www.figma.com/design/4M7uEe9kH8hgZdB07lBvj9/Untitled-(Copy)?node-id=0-1

Alright, thanks Ahmad. Looks like the feature is already in the works, so we can close this.


Sorting numbers or currency only sorts the first digit

If i sort my Column it only sorts from the first digits, but not the whole number.
How can I fix this? it doesn't matter if I use the currency sort function or numbers.
The numbers are fetched from objects, do I need to convert those to strings maybe?
Currency:
88
860
810000
756
74500
5240
472200
4651
297100
241700
23390
1...
etc...
sortable={{
  column: 'Currency', Currency: function (a, b) {
    return a > b ? 1 : -1;
   }
}}

+1


ln not recognized as log

Error: Error sending data to Herbie server at http://<IP_ADDRESS>:8000/api/sample:
Herbie server: Invalid program
web::38: Unknown operator ln
See https://herbie.uwplse.org/doc/2.0/faq.html#invalid-syntax for more.
at c (https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/misback/.vscode/extensions/herbie-fp.odyssey-fp-1.0.4/dist/webview.bundle.js:2:1256127)
at async Object.getSample (https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/misback/.vscode/extensions/herbie-fp.odyssey-fp-1.0.4/dist/webview.bundle.js:2:1231181)
at async https://file+.vscode-resource.vscode-cdn.net/c%3A/Users/misback/.vscode/extensions/herbie-fp.odyssey-fp-1.0.4/dist/webview.bundle.js:2:1233871

This issue is more serious since it occurs in our asinh demo and there is no warning about ln in the spec entry!

I've hacked this so that ln just looks like log to Herbie rather than crashing. This is not technically correct and we should warn the user in this case!

Nvm log is natural log for Herbie and we now explain this on the landing page, closing.


Followup #283 - RTEMS - select() fixes

As suggested by @Acuadros95 I simplified the code copying the set to a local variable. Also fixed the discovery.

Fixing DCO after uncrustify test

Build status:

Linux
Windows

LGTM!

Ahh sorry messed up. Will fix

Build status:

Linux
Windows

@Acuadros95 now clean and signed. Sorry for the confusion.

Build status:

Linux
Windows

@Acuadros95 remember to move the ros2 and foxy branches to the last commit in develop


Update Site settings “info”

Automatically generated by Netlify CMS

👷 Deploy Preview for uvz processing.

Name
Link

🔨 Latest commit
f93579618483f2755af37dec9112b2411575767c

🔍 Latest deploy log
https://app.netlify.com/sites/uvz/deploys/6242dbbe2ccf060009dd5a94


implemented a PutItem file based queue with exponential backoff

hey there, I've implemented a file based queue with exponential back off here: https://github.com/sansmischevia/node-dynamodb as we have a use case for writes that must exceed, that also happen in bursts.
Check out the repo, and let me know if this is something you're interested in pushing into this core project.
I've included mocha tests for fileQueue, and wrote some simple ones for the other operations. The tests can be run with a 'make test'

Hi! I really like the idea since we're having the same issues at teleportd. We had to implement a queue to minimize the impact of Throughput Exceeded exceptions.
Nevertheless, I think it should be a wrapper to ddb.js rather than a modification to ddb.js.
What do you think?

Could you explain what the benefits are to using a wrapper? currently it's just another parameter you'd pass into the constructor, but I suppose I can easily make a wrapper

the main benefit I see is 'total' independency between the two: no need to hook into the execute workflow, which results in increased simplicity and modularity.

fixed by #26


Enforce MPI progress during event_wait

Avg response time
coverage on master

Summary of changes
Summarize what you changed
Rationale for changes
Why did you make these changes?
Additional info and certifications
This pull request (PR) is a:

[x] Bug fix
[ ] Feature addition
[ ] Other, Please describe:

I certify that

[x] I have reviewed the contributing guidelines and followed the
policies on:
- Pull request (PR) naming to indicate work in progress (WIP),
and to attach the PR to the appropriate bug report, or feature
request issue
- White space (no trailing white space or white space errors may
be introduced)
- Commenting code where it is non-obvious and non-trivial
- Running tests locally, to ensure all of them pass
- Maintaining or increasing test coverage
- Logically atomic, self consistent and coherent commits
- Commit message content
- Waiting 24 hours before self-approving the pull request to
give another OpenCoarrays developer a chance to review my
proposed code

Fixes #411 . Let's discuss this before merging.

Fixes #411 . Let's discuss this before merging.

OK, I'll hold off on approval until we discuss...

@vehre The MPI_Win_flush polls the MPI engine and ensures progress (here I assume that CAF_Win_unlock points to flush). MPI_Win_sync sometimes (like in this case I guess) is implemented as a no-op operation and it does not ensure progress.
In a situation where the MPI implementation takes advantage of the interconnect capabilities, the CAF_Win_unlock is not needed. That's why I was curious about the behavior with MVAPICH.
Polling the MPI engine with CAF_Win_unlock requires time and because we don't know a priori if we really need the CAF_Win_unlock (because we don't know which MPI implementation and interconnect the user is using) I increased the timer to avoid polling the MPI engine too ofter (which may take more than the timer increase). I hope this clarifies.

(because we don't know which MPI implementation and interconnect the user is using)

We might be able to  setup some system introspection to determine this, no? Might it be worthwhile?

@zbeekman sure, it would be helpful but not exhaustive. We need to know the MPI implementation AND the network interconnect.

@afanfa hmmm I see... the network interconnect may be difficult, because if you compile on a login/head node, you may not have access to that information... so I guess the best we can do might be check which MPI lib we're using?

@zbeekman , yes, I think we can speculate. For example, if we found MVAPICH installed we can assume that an Infiniband interconnect will be used and it will probably behave as we expect (hopefully).

@afanfa what is the status of this? What needs to happen to get this fix into master?

@zbeekman , please go ahead and merge this little change. We'll ask Ambra to rerun her tests with the trunk version.
Thanks.

LGTM

Sorry, but this does not 'look good to me'. I am sorry, that I was in the impression that when I ask a rhetorical question, it is clear that I am pointing out a mistake. Obviously it is not understood that I can provide the required MPI knowledge.
So @afanfa: Answering a MPI_Win_sync() with a MPI_Win_unlock() will neither lead to progress nor to a program running on all systems. You are right that usually CAF_Win_unlock() is expanded to MPI_Win_flush(), but not always. Therefore when one wants a flush, one writes a flush.
I apologize, that did not make myself more clear. It will not happen again, even when I then seem to be impolite. I do have very limited time, so voicing anything directly now will be my policy.

Andre,
We can easily revert this merge.
I apologize if I did not understand your comment completely. We are
definitely short on man power and expertise (because our experts like you
and Alessandro have many obligations) so I do the best job I can to keep
progress moving forwards. I don't believe this code has made it into a
release yet, either.
On Sat, Aug 5, 2017 at 7:36 AM Dr. Andre Vehreschild <
<EMAIL_ADDRESS>wrote:

Sorry, but this does not 'look good to me'. I am sorry, that I was in the
impression that when I ask a rhetorical question, it is clear that I am
pointing out a mistake. Obviously it is not understood that I can provide
the required MPI knowledge.
So @afanfa https://github.com/afanfa: Answering a MPI_Win_sync() with a
MPI_Win_unlock() will neither lead to progress nor to a program running on
all systems. You are right that usually CAF_Win_unlock() is expanded to
MPI_Win_flush(), but not always. Therefore when one wants a flush, one
writes a flush.
I apologize, that did not make myself more clear. It will not happen
again, even when I then seem to be impolite. I do have very limited time,
so voicing anything directly now will be my policy.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/sourceryinstitute/OpenCoarrays/pull/412#issuecomment-320438820,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAREPKjrgoByIr_EkBo5vEhfG3zrwahrks5sVFPXgaJpZM4OWDsV
.

No issue Zaak. My comment was more intended to Alessandro. He has the knowledge and was expected to adapt the code. Your work on progress of opencoarrays is very much appreciated.

No need to revert. The pull request #426 is on top of Alessandro's.


Fix design changes for mobile login page

closes #449LiveURL: https://smartfaq-449-fix-design-changes-for-mobile-login-page.surge.sh

I think this desings should also include iPhoneX


Support environment variable passing

Do you know if this has been implemented? I need this tool to accept access_token every time the test is fired.


Added basic support for Lightware SF/LW-20 I2C connected LIDAR Modules

Added Configuration for the Lightware SF20 / LW20 Type LIDAR Modules (Range and Update Rate) as well as Parameter Config Option.
This refers to #7899

Unintentional sensor_params.c changes?

I have seen but did not know how it happened nor how to revert it. The Line for SF1XX with Value 5 is relevant. Can you discard the rest?

I ve seen some how this PR is messed up. I'll check this again an issue a new one.


Key Vault data plane SDK

Key Vault Data plane SDK and a couple of sample applications

Can one of the admins verify this patch?

Hi @divyakgupta, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (divyag). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

@azuresdkci add to whitelist


[CLI] Add version to help screen, move down the docs links

Before

After

@nezaj @dwwoelfel

Help for individual command:

It will expand the options there : )
Going to go through each one as I push changes, and see if we can make them more informative / nicer to look at


Script creates CSV file of relevant door movements

Note to self: The data specifies whether the status of the door changed from event to event, so I can use that information to filter through events instead of my weir syntax. I should also edit lines 92-94.

I would definitely make the change to use the stateChange field as you already commented about before we merge this.


[VS Code] No usable version of the libssl was found

I am running Fedora Workstation 28 with latest VSCode installed from the official page. The VS Live Share extension does not work since there seems to be missing a libssl dependency. Letting VSCode auto install the required dependencies does not fix this problem.
Verbose logs
Product and Version: VSCode
OS Version: Fedora Workstation 28
Live Share Extension Version: 0.3.1011
Target Platform or Language: N/A
Steps to Reproduce / Scenario:

Download and install Visual Studio Code from the official page.
Open VSCode and install the "VS Live Share" extension
Restart the editor and allow VSCode to automatically install the required dependencies.
Restart the editor again after the dependency installation succeeds
Each time the editor is opened the following message appears: "VS Live Share activation failed. You may be missing key Linux dependencies. Install them now?
Export the VS Live Share logs where you can find the message No usable version of the libssl was found

Screenshots

Well... It turns out that I am running Fedora 29. I don't know why I thought it was 28 :man_facepalming:  I don't know if this changes anything, I am updating the main post to reflect this, sorry.
In any case. This is a fresh installation from yesterday with nothing special in it. I ran the Anaconda installer and then installed some of my preferred apps, including VSCode.
Running dnf list installed | grep openssl provides the following output:
apr-util-openssl.x86_64         1.6.1-8.fc29           @koji-override-0
openssl.x86_64                  1:1.1.1-3.fc29         @anaconda
openssl-devel.x86_64            1:1.1.1-3.fc29         @fedora
openssl-libs.x86_64             1:1.1.1-3.fc29         @anaconda
openssl-pkcs11.x86_64           0.4.8-2.fc29           @koji-override-0
xmlsec1-openssl.x86_64          1.2.25-5.fc29          @anaconda

It seems that Fedora 29 comes with a newer OpenSSL version that can not be used by VS Live Share. I've just tried to install the package compat-openssl10 and everything is working properly now.
Running dnf list installed | grep openssl provides the following input:
apr-util-openssl.x86_64         1.6.1-8.fc29           @koji-override-0
compat-openssl10.x86_64         1:1.0.2o-3.fc29        @fedora
openssl.x86_64                  1:1.1.1-3.fc29         @anaconda
openssl-devel.x86_64            1:1.1.1-3.fc29         @fedora
openssl-libs.x86_64             1:1.1.1-3.fc29         @anaconda
openssl-pkcs11.x86_64           0.4.8-2.fc29           @koji-override-0
xmlsec1-openssl.x86_64          1.2.25-5.fc29          @anaconda

@crbelaus Ah yes that could explain it! Historically the compat-openssl10 came along with the openssl-libs package but that must have changed in 29. I agree adding it to the dependency script is a good idea. If you'd like to submit a PR as you mentioned above, I'd be happy to review it or we can move it forward.

Oops!  Looks like my PR merge auto-closed this issue.  It will go with the next release.  Reopening until then.


Replace internal:split and internal:replace usages with stringutils module

Purpose
This PR replaces the internal:split and internal:replace usages with stringutils:split and stringutils:replace respectively.
Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #18657 into master will decrease coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #18657      +/-   ##
==========================================
- Coverage    15.1%   15.04%   -0.06%
==========================================
  Files          48       48
  Lines        1258     1263       +5
  Branches      198      199       +1
==========================================
  Hits          190      190
- Misses       1054     1059       +5
  Partials       14       14

Impacted Files
Coverage Δ

...ool-plugins/vscode/src/debugger/config-provider.ts
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7eaf30b...d8a6970. Read the comment docs.


Export mamba models

Can you add a small example on how to export mamba models?
See https://github.com/pytorch/pytorch/issues/130150
https://pytorch.org/tutorials/intermediate/torch_export_tutorial.html#custom-ops
Thanks

Iirc, I think the team at meta showed it in a pretty nice way over here: https://github.com/facebookresearch/lingua/tree/main/apps/mamba/component

I am asking this cause https://github.com/huggingface/transformers/issues/33839.
But yes after exporting the model it could be also compiled so a complete example of export+compile it will be very helpful.


Create 12

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

1


Feature/display user-set time units

Resolves:

Feature - implement set time unit in simularium-website
Bug - Some local uploads break the playback slider

Summary
simularium-viewer v2.7.0 (installed this new version in this PR) converts all data to the latest simularium file version upon receipt. This PR makes simularium-website work with the latest v2 data format.
Results
I tested with v1 and v2 local uploads. All are as expected:

v2 files - Times and scale bars display correctly and properly. Trajectories with both non-second and second time units play properly.

v1 files - Times and scale bars display correctly but are "ugly" for some trajectories (because everything displays in seconds and meters). Functionality is same as before - trajectories with very small time values break the slider.

I also tested with the 2 streamed trajectories. They are broken, but I believe it's unrelated to this PR (localhost:8080/public/ shows the same bug on simularium-viewer test site when rolled back to the commit before the version update changes were merged). I let @EricJIsaac know in case it's related to array buffer changes.

Remaining related work

Determine scale bar label in simularium-viewer

Pull request recommendations:

[x] Name your pull request your-development-type/short-description. Ex: feature/read-tiff-files
[x] Link to any relevant issue in the PR description. Ex: Resolves [gh-##], adds tiff file format support
[x] Provide description and context of changes, including any helpful screenshots.
[ ] Provide relevant tests for your feature or bug fix.
[ ] Provide or update documentation for any feature added by your pull request.

Thanks for contributing!

As a note - the yellow bug pre-dates both this change and the array-buffer changes:
https://allencellscience.slack.com/archives/CMQFRJPPF/p1608252745398400?thread_ts=1608248656.391700&cid=CMQFRJPPF

As a note - the yellow bug pre-dates both this change and the array-buffer changes:
https://allencellscience.slack.com/archives/CMQFRJPPF/p1608252745398400?thread_ts=1608248656.391700&cid=CMQFRJPPF

Oh that's right, it did happen before. The only thing is that the all-yellow bug only happened sometimes previously, but now the it happens every time reliably for both of the trajectory files. 🤔

did you also check this with ?trajUrl=https://aics-agentviz-data.s3.us-east-2.amazonaws.com/trajectory/endocytosis.simularium? Should be the same as loading it locally, just want to be certain

did you also check this with ?trajUrl=https://aics-agentviz-data.s3.us-east-2.amazonaws.com/trajectory/endocytosis.simularium? Should be the same as loading it locally, just want to be certain

I totally did, and it worked great!


Support more platforms via LIEF

Is your feature request related to a problem? Please describe.
binsider should support more binary formats.
Describe the solution you'd like
Abstract away the parsing details via LIEF format.
See: https://github.com/lief-project/LIEF
Describe alternatives you've considered
None.
Additional context
Ideas welcome!

@HadrienG2 raised some good questions in https://github.com/orhun/binsider/issues/67#issuecomment-2396104783


directory  is not empty and cannot be removed in method keybase.1.createPersonalRepo

All keybase git operations fail with some variant of above.
Also included: Error 218.

#26158


Implement custom alignment to remove screen size knowledge when offsetting element in MotionController

For offsetting UI elements correctly in SpotlightSlider and similar motion controllers we use the container's width value multiplied by position in the list. We could remove knowledge about this from MotionControllers and offset elements with Alignment as it contains all the information about the parent's size and its own size

Can be closed? #564


Bump to go 1.18

Description
Merge requirements
Please take into account the following non-code changes that you may need to make with your PR:

[ ] CHANGELOG entry for user-facing changes
[ ] Updated the relevant documentation

How do we decide when to upgrade to golang version?
BTW 1.19 is already out we got the following PR

https://github.com/timescale/promscale/pull/1502/files
https://github.com/timescale/promscale/pull/1539

How do we decide when to upgrade to golang version?

I don't know. I don't think that we have any process. I opened this PR because I wanted to see the result of the benchmark compared to another PR which I have open which would also bump the golang version (https://github.com/timescale/promscale/pull/1501).
I would suggest that once those other PRs are merged, we bump to 1.19, but I like being on the bleeding edge 😂

But this is just the minimum Go version supported, right? Because all the build references I found use go 1.18

https://github.com/timescale/promscale/blob/8ccdc0f9992cfe8210134027d7be905354fc4488/.github/workflows/go-releaser.yml#L15
https://github.com/timescale/promscale/blob/8ccdc0f9992cfe8210134027d7be905354fc4488/build/Dockerfile#L2 ->
https://github.com/timescale/promscale/blob/8ccdc0f9992cfe8210134027d7be905354fc4488/.goreleaser.yml#L129

But this is just the minimum Go version supported, right? Because all the build references I found use go 1.18

Yes.


feat(client): ts-migrate multiple files

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Related to #42256, #43778

Migrated Files

/client/src/templates/Introduction/Intro.js
/client/src/templates/Introduction/SuperBlockIntro.js

@vishwasnavadak, It the has been an issue with the rebase that is why it has picked up some translation files among other commits.
I suggest force pushing your changes after a clean rebase, and let us know if you are still experiencing issues.

@vishwasnavadak, It the has been an issue with the rebase that is why it has picked up some translation files among other commits.
I suggest force pushing your changes after a clean rebase, and let us know if you are still experiencing issues.

Thanks for this suggestion. I did that, Now the history seems cleaner without the translation changes. Let me know if there's anything else to in this PR.

Thanks for the change suggestions. I'll try to make the fixes as per your suggestion. I had added some ts-ignore comments since I wasn't sure what is the right type to be applied. This helps.
I'll update the PR with these changes

@ojeytonwilliams thanks for the help. I have done some of those changes in local, hadn't got time to run the tests and update the PR. Will take your suggestions and update this in couple of days.

Quick Update: I was able to update the types to correct types. There's one last thing I'm trying to fix on the conflicted file. Will update that soon.


Create Justfile

Here are a few commands I think are common enough to warrant wanting to avoid typing them in all the time.
I am using Age (https://github.com/FiloSottile/age) to encrypt the mnemonic, having this in the program would be much more secure but I figured I would add it for the time being (since its better than nothing - I hope). The idea is to open the mnemonic when in use and close when not in use.
I also added a command that takes a file path from the .env folder, encrypts the database and writes to the folder location. This folder would ideally be somewhere durable.
I had some commands to make it easy to test gun by yourself, so I added them.
Just (https://github.com/casey/just) is like Make but simpler and it can be installed using cargo. I like using it with my bitcoin cli apps, I use it as a way to take notes of which shell commands work. You can change wallets on here by updating your .env file, makes it easy to test stuff or manage multiple wallets.
Along with this file comes a .env file, mine looks like this.
gundir = $HOME/.gun
backup = $HOME/backup

If this is out of scope for the project then no problem, no need to merge.
Figured I would share anyways.

@keblek First I've heard of just. Looks really handy. I think it would be a maintenance/documentation burden to keep this kind of thing in the repo though. The most useful commands you've made are things that really should be in gun itself e.g. backup/restore and password encryption.
Might be worth linking to your Justfile as a gist in the documentation as a hint on how to simplify thing if you're an advanced user though.


Check if we still need to use paket with the .NET Core setup

It might be that paket has become obsolete, as our build script is now fairly straightforward.

This issue is on-hold pending the work in #214.


Inference with finetuned checkpoints

Could you provide documentation on running inference with a finetuned Cogvideox-fun model? My finetune has output into something other than diffusers, and it seems like inference only supports the original diffusers format.

Could you list your finetuned model dir?

Could you list your finetuned model dir?

If you use predict_t2v.py, you can modify the transformer_path


Image Ref missing title tag in rendered HTML

If a URL is referenced from a non-ref link then the rendered image link will be missing a title
![alt text](/images/icons/up_16.gif)

![alt text][id]

[id]: /images/icons/up_16.gif "Title"

Actual:

Expected:

Also affect link references with title definition.

Fixed in 0.19.6, repo updated. Maven released, may take time for central to update.


[Robots.txt] Close groups of rules as defined in RFC 9309

This PR addresses:

Clarify behavior when to close blocks of multiple user-agents (resolves #390)
Handle robots.txt with missing sections (and implicit master rules (resolves #114)

Changes:

rule groups are closed as specified in RFC 9309: on a user-agent line, if at least one allow/disallow line was read before
that is, Sitemap, Crawl-delay and other directives not covered by RFC 9309 do not finish a rule group
the Crawl-delay is set independently from grouping:

it's set for a given agent or the wildcard agent
but a value defined for a specific agent is never set using a value defined for the wildcard agent

unit tests are updated or added accordingly

for changed tests it's verified, that the RFC reference parser (google/robotstxt) behaves the same

This PR should unblock #245 (all unit tests added there should now pass).

Thanks for the review, @jnioche!


[proposal] [tool] flutter run is too chatty

Steps to Reproduce

Run flutter run --debug myapp.
Interact with the app in any way (scrolling, touching, etc.)

Expected results:
Only relevant information like state events or print statements, etc in the debug console
Actual results:

Logs
D/ViewRootImpl( 3768): ViewPostImeInputStage ACTION_DOWN
D/ViewRootImpl( 3768): ViewPostImeInputStage ACTION_DOWN
D/ViewRootImpl( 3768): ViewPostImeInputStage ACTION_DOWN
D/ViewRootImpl( 3768): ViewPostImeInputStage ACTION_DOWN
D/ViewRootImpl( 3768): ViewPostImeInputStage ACTION_DOWN
D/ViewRootImpl( 3768): ViewPostImeInputStage ACTION_DOWN
D/ViewRootImpl( 3768): ViewPostImeInputStage ACTION_DOWN`

`flutter doctor -v`
[✓] Flutter (Channel stable, v1.17.0, on Mac OS X 10.13.6 17G66, locale en-IN)
    • Flutter version 1.17.0 at /Users/sanskarjaiswal/flutter
    • Framework revision e6b34c2b5c (6 days ago), 2020-05-02 11:39:18 -0700
    • Engine revision 540786dd51
    • Dart version 2.8.1

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/sanskarjaiswal/Library/Android/sdk
    • Platform android-29, build-tools 29.0.3
    • Java binary at: /Users/sanskarjaiswal/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/192.6392135/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✗] Xcode - develop for iOS and macOS
    ✗ Xcode installation is incomplete; a full installation is necessary for iOS development.
      Download at: https://developer.apple.com/xcode/download/
      Or install Xcode via the App Store.
      Once installed, run:
        sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer
        sudo xcodebuild -runFirstLaunch
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To install:
        sudo gem install cocoapods

[!] Android Studio (version 3.6)
    • Android Studio at /Users/sanskarjaiswal/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/192.6392135/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[✓] VS Code (version 1.44.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.10.1

[✓] Connected device (1 available)
    • SM E700H • <IP_ADDRESS>:5555 • android-arm • Android 5.1.1 (API 22)

! Doctor found issues in 2 categories.

Even packing the same messages into one message with a counter would be great.

The tool makes a best effort attempt to filter out logs, unfortunately android itself is chatty.


BuildRequest not using resource parameter

I'm just getting started with websocket programming and thought that the Beast library would be a good place to started. I've been trying to put together a simple app that communicates with a websockets server and I noticed that the handshake is always issued as GET / rather than, for example, GET /myserver/serverpoint.
I notice that although you are expected to provide a resource during the handshake call this parameter, which is passed into the build_request function, is ignored and the url is explicitly set to "/". This looks like an oversight - shouldn't the resource be used in the construction of the req.url?
PortableNuke.

@PortableNuke You are correct, and this issue has been fixed in 1.0.0-b11 as of August 26th:
https://github.com/vinniefalco/Beast/commit/878c0f2a19887114ca30c2ae4d298b8008985d5b
Current version is 1.0.0-b13 you should upgrade :-)
Thanks for the heads up!

Okay, that's good to know. I'll make sure to upgrade to the latest version.
Thanks,
PortableNuke


Sample vs Readme examples

Hi Dusan,
your sample contains the following example:
[HttpGet("")]
public IActionResult Index(ODataOptions options)
{
return View(new ODataSqlQuery(
"SELECT FROM Entities WHERE OwnerId = @p0",
10456, options
));
}
but the readme contains this which I cannot figure out how to make it work:
[HttpGet("api/entities")]
public Task<IOData> GetEntities(ODataOptions options)
{
// Do whatever you want with OData query options,
// build custom Micro ORM integration, etc.
var query = new ODataSqlQuery(
    "SELECT FROM Entities WHERE OwnerId = @p0",
    10456, options
);

}
How should I convert the query into the required Task result?
Thanks!
Rok

I don't understand your problem. The ODataSqlQuery is here to generate SQL command form OData options. The Task result is not required at all.

Ok, but the
public Task> GetEntities(ODataOptions options)
has to return something, currently it only defines a local query variable, doesn't it?

Of course. You need to use the query somehow (retrieve data from database, etc.). I've created a gist to show you how to integrate OData SQL with MicroLite ORM, but you can use any other library (ADO.NET, Dapper, etc.).
(The code is probably not working, but you should get the idea.)


newa user config file

Users should be able to define all important variables (like ET URL, Jira token,...) in a config file, for example ~/.newa/config
At the same time, each such config option should be also configurable through environment variable having NEWA_ prefix to simplify integration into pipelines.

I tried to approach this in https://github.com/RedHatQE/newa/pull/30

@kkaarreell I think we might want to reconsider having jira_project in the newa config. The thing is that project is a kind of component-specific thing. Different teams will want to use their own projects for their components. Hence I believe this particular config fits better in to component configuration (as implemented in #20 ).

i think that it might be handy to be able to specify it at multiple levels (envvar, newa config, jira subcommand option, issue config) with defined (hardcoded) priorities. Also, partially related problem is splitting issue config into multiple files.

https://github.com/RedHatQE/newa/pull/30 got merged.


The container property is not supported

https://developer.mozilla.org/en-US/docs/Web/CSS/container

As mentioned in the top level vscode issue https://github.com/microsoft/vscode/issues/170589
The features needed for container queries are:
properties

container shorthand

container-name
container-type

at-rule

@container

length units

cqw
cqh
cqi
cqb
cqmin
cqmax

The container property seems to be working now, so I think this can be closed.
Supports appears to have been added in 1ded897e653d5129bdc4c372917d6580d7ae22c3 (in webCustomData.ts), released in 6.2.7.


Create zh-HANT.json

https://github.com/switchbrew/libnx/blob/d91a2e22e7942a7e0e0b840c55f13675bc4066ac/nx/include/switch/services/pl.h#L18-L24
typedef enum {
    PlSharedFontType_Standard             = 0,  ///< Japan, US and Europe
    PlSharedFontType_ChineseSimplified    = 1,  ///< Chinese Simplified
    PlSharedFontType_ExtChineseSimplified = 2,  ///< Extended Chinese Simplified
    PlSharedFontType_ChineseTraditional   = 3,  ///< Chinese Traditional
    PlSharedFontType_KO                   = 4,  ///< Korean (Hangul)
    PlSharedFontType_NintendoExt          = 5,  ///< Nintendo Extended. This font only has the special Nintendo-specific characters, which aren't available with the other fonts.
    PlSharedFontType_Total,                     ///< Total fonts supported by this enum.
} PlSharedFontType;

I'm not sure if  you have add the ChineseTraditional font in your Aether lib.Just a reminder : )

Thanks! And yes, Aether is all set for the other fonts :)


Is there a way to query parsec-service from parsec-tool regarding the current parameters in the config.toml

Depending on the platform, if we want to cross-verify the current configuration of the parsec-service, Is there a way to have a command option to read out the current config parameters set for the parsec-service?
If there are any passwords/pins in the config.toml, probably those can be encrypted as done by /dev/random?

Neither the parsec-tool nor any parsec client should be able to query the configuration of the parsec-service. For this reason, we are closing this ticket.
For debugging purposes on the parsec-service side, you can directly check the written config.toml.


Fix StartGameSerializer 527 not reading unsigned var int for default player permission

This is an Issue on Hive end so im closing this.


Greenkeeper/@types/node 10.0.9

Coverage remained the same at 94.595% when pulling 0c5a6af5d8ba24d2ecdfd4fa09eb96df11247891 on<EMAIL_ADDRESS>into a8c2e2b6741f58376c865fb4e9c49f1d1b040819 on master.


readme on npmjs outdated

on npmjs section
#difference-from-client-side-fetch different from
HEAD
version.
old version point to https://github.com/node-fetch/node-fetch/blob/HEAD/LIMITS.md (Page not found)

You are probably looking at the README for the v2 version.

i see from npm https://www.npmjs.com/package/node-fetch/v/3.0.0-beta.9
that v2 is latest stable , and v3 is beta


Avoiding require cycles

Given you have to import screens in the navio file, as well as export the navio instance for use in said screens, is there a trick to avoiding require cycles?
My cycle looks like this:
src/navio.tsx -> src/screens/chat.tsx -> src/services/index.tsx -> src/navio.tsx

I've found some documentation for a metro config line that does fix it.
https://facebook.github.io/metro/docs/configuration/#requirecycleignorepatterns
I set mine to:
  /(^|\/|\\)node_modules($|\/|\\)/,
  /^src\/navio.tsx$/,
];```

Unless there's a clean solution, it seems this is the best we've got.

Hi @nick-potts! I've been always ignoring that issue as it didn't bring any problems, but it had to be fixed! Thank you, that works! I'll add that to docs.

No worries. Doesn't seem like the worst fix given the require cycle is bourne by the nav library importing the screens that instantiate the require cycle.


Allow loading images from MEDIA_URL

Coverage remained the same at 89.037% when pulling afbdde75c0c4da4d34a6ca2976ae556b65ded726 on flamingspaz:csp into 3a3d39906f4653cdd6ce5561a920ed634c23ccd9 on mozilla:master.

Coverage increased (+0.003%) to 89.041% when pulling afbdde75c0c4da4d34a6ca2976ae556b65ded726 on flamingspaz:csp into 3a3d39906f4653cdd6ce5561a920ed634c23ccd9 on mozilla:master.


fix(package): relax node version range (engines)

[x] bugfix
[ ] new feature
[ ] code refactor
[ ] test update
[ ] typo fix
[ ] metadata update

Motivation / Use-Case
Motivation described in #154

Thank you @evilebottnawi !

Released in v1.1.2 🎉


proxy_protocol_filter: Add configuration to match only specific proxy protocol versions, new stats

Commit Message:
proxy_protocol_filter: Configuration to match only specific proxy protocol versions, new stats
Additional Description:
Currently the Proxy Protocol Listener filter will try to match incoming connections against both proxy protocol v1 and v2 signatures. While this is convenient, it:

Increases the attack surface of the filter.
(when allow_requests_without_proxy_protocol is enabled) Increases the chance of signature conflicts between proxy protocol v1 requests and non-proxy protocol requests.

This change adds a new config option disallowed_versions that scopes down the set of proxy protocol versions that the filter matches. The configuration is optional and defaults to current behavior when not specified.
This change also adds new statistics per matched proxy protocol version. See doc update for details.
downstream_cx_proxy_proto.not_found_disallowed
downstream_cx_proxy_proto.not_found_allowed
downstream_cx_proxy_proto.v1.found
downstream_cx_proxy_proto.v1.disallowed
downstream_cx_proxy_proto.v1.error
downstream_cx_proxy_proto.v2.found
downstream_cx_proxy_proto.v2.disallowed
downstream_cx_proxy_proto.v2.error

Pre-existing stat downstream_cx_proxy_proto_error is kept at it's own scope for backwards-compatibility.
Risk Level: Low

All client-facing behavior changes are guarded by new filter config field.
Only default behavior change is incrementing new counters.

Testing:

Extensive unit tests
Some integration tests

Docs Changes:
Updated proto and filter docs
Release Notes:
Updated
Platform Specific Features:
N/A
Fixes https://github.com/envoyproxy/envoy/issues/32425

/review @fzhong-connect

cc @fzhong-connect @jaysonjdmello

@markdroth or @envoyproxy/api-shepherds PTAL, we have a question for you in the comments.

PTAL

Please add 1 integration test case to validate disallowed versions; probably the case of disallowed v1, send a request with v1, verify connection gets closed.
/wait

Thanks, added. I also found the stats were not rooted under listener stats via the integration test, so fixed that as well (only for the new stats).

/lgtm api

Please add tag-extraction rules for the new stats to extract the version, so for example in prometheus stats they would have a label for proxy protocol version. Add them here, and see other examples in this file for how to do it: https://github.com/envoyproxy/envoy/blob/main/source/common/config/well_known_names.cc#L212
/wait

Done, thanks for the pointers. Didn't know this existed.

/retest

PTAL

PTAL @ggreenway

Ready for review @ggreenway

/retest

/retest

/retest

Ready for review @ggreenway . Not sure how to re-run the flaky CI failure

This mostly LGTM, aside from these small details.
I'm going to be out for awhile starting tomorrow, so one of the other maintainers will need to finish review of this and get it merged.

Thanks @ggreenway . If you have time, feel free to review today. Slightly worried Github may not let other reviewers merge as the status checks say:

1 review requesting changes by reviewers with write access. Merging can be performed automatically once the requested changes are addressed.

But no worries if you have taken off already, I can ping other reviewers tomorrow. Thanks for the thorough feedback!

/retest


Font

please help me with this.
i don't know how to create fonts in json

Not an issue with bitmap-manupulation, please use google.


Implement a mechanism for accepting old check-IDs

Based on Lib/fontbakery/legacy_checkids.py

The mechanism will be temporary, displaying a message to users:
DEPRECATION WARNING: By late-December 2024, FontBakery version 0.13.0 introduced a new naming scheme for the check-IDs. Fontbakery detected usage of old IDs and performed an automatic backwards-compatibility translation for you. This automatic translation will be deprecated in the next major release. Please start using the new check-IDs as documented at https://github.com/fonttools/fontbakery/blob/83db7cc2a6ad58585ddec9397306e0420843edb1/Lib/fontbakery/legacy_checkids.py


"Error maximum recursion depth exceeded"

At the end of the backTrack function, I try to call checkNext but I get an error message saying Error maximum recursion depth exceeded
I don't know how to work around it ? I need to call the function
def backTrack(square, row, column):
    previous=emptyCells[currentCell-2]
    s=previous[0]
    r=previous[1]
    c=previous[2]
    square=int(s)
    row=int(r)
    column=int(c)
    num=sudoku[square][row][column]
    checkNext(square, row, column, num)

def checkNext(square, row, column, num):
    if num == 9:
        backTrack(square, row, column)
    else:
        num=num+1
        squareClash = inSquare(square, num)
        if squareClash != 'Y':
            rowClash = inRow(square, row, num)
            if rowClash != 'Y':
                columnClash = inColumn(square, column, num)
                if columnClash != 'Y':
                    sudoku[square][row][column]=num
                else:
                    checkNext(square, row, column, num)
            else:
                checkNext(square, row, column, num)
        else:
            checkNext(square, row, column, num)

Am I right in thinking this has been made obsolete using the new implementation discussed this afternoon?

I'm not sure yet, I'll try it with the new code and let you know how I get on


Feature request: Add support for 2FA or hardware key based authentication

Hello! I think that it would be great if we could have the option to use 2FA authentication in pocketbase, or even hardware based authentication. I think this is a necessity to have at least for the admin authentication. Are you planning to implement this feature? If not, how would I go about implementing something like this myself?

You can check https://github.com/pocketbase/pocketbase/discussions/1553.


Make DefaultReflectionService.IsStringEnum virtual

Hi Rico,
could you please make DefaultReflectionService.IsStringEnum also virtual? I add a custom StringEnumConverter to JsonSerializationSettings.Converters that excludes certain enum types so they (the excluded enum types) have to be handled as numeric enums by NJsonSchema/NSwag.
I hope there are now objections to this suggestion.
Thank you,
Daniel

https://github.com/RicoSuter/NJsonSchema/commit/1e343ee5c063ae059527bef95397dc6ce74aa16c


RangeError: wrong data size

Hello! Great library by the way.
I am getting this error:
RangeError: wrong data size. Found 196608, expected 65536
Context code:
const res = await fetch(icon_url);
const uint = new Uint8Array(await res.arrayBuffer())

const resizeIcon = await resize(uint, { width: 256, height: 256 });

icon_url is an url pointing to PNG, JPG and JPEG file of different sizes

Can you give me an image that doesn't work?


Update journeys-edits.md info on Journey Settings

Proposed changes
No mention of editing limitations.
Merge timing

ASAP once approved

Related issues (optional)
Slack thread

Hey team just wanted to check in on this docs update. Could someone check what's blocking this merge?

@lizkane222 Merged and live!


fix(package): update react to version 16.4.2

Coverage remained the same at 98.986% when pulling 7aab8bc9469fde3badc60e92edacf8eb08b9025c on tuchfarber/upgrade_react_16_4_2 into 559ba85eac53b1dff8797946ca25dfe307830235 on master.

@tuchfarber I think it's a great idea to upgrade React! Out of curiosity, is there a particular reason for the upgrade? Also, did you happen to go through and test that this wouldn't be a breaking change for any of our existing components?

@adamstankiewicz 16.3 is when createRef was introduced and I need that for #322. I updated and flipped through the storybook and everything seemed to work. Tests all pass as well.


Fails to load planar DDS format textures

In DirectX 12, each plane of a planar format is it's own subsurface--in DX 11 all the planes were in the same subsurface. This means the current implementation of DDSTexutreLoader fails for:

DXGI_FORMAT_NV12 4:2:0 8-bit
DXGI_FORMAT_P010 4:2:0 10-bit
DXGI_FORMAT_P016 4:2:0 16-bit
DXGI_FORMAT_420_OPAQUE 4:2:0 8-bit
DXGI_FORMAT_NV11 4:1:1 8-bit

It would also fail for the JPEG Hardware decode formats added in DXGI 1.4

DXGI_FORMAT_P208 4:2:2 8-bit
DXGI_FORMAT_V208 4:4:0 8-bit
*` DXGI_FORMAT_V408 4:4:4 8-bit

Subresource indexing changed for planar types:
inline UINT D3D11CalcSubresource( UINT MipSlice, UINT ArraySlice, UINT MipLevels )
{
    return MipSlice + (ArraySlice * MipLevels);
}

is now:
inline UINT D3D12CalcSubresource( UINT MipSlice, UINT ArraySlice, UINT PlaneSlice, UINT MipLevels, UINT ArraySize )
{
    return MipSlice + (ArraySlice * MipLevels) + (PlaneSlice * MipLevels * ArraySize);
}

Updated to support load of planar formats


Is there a "isRotating" option?

Hey, is there an option to retrieve information about the wheel's rotating status?
It would be really helpful!
Edit: Just saw your "//TODO onWheelItemSettled? "

See my pull request.


Clear up a few issues regarding elementaryOS

It would be best to clear the air between Sodalite and elementaryOS, to make sure there's no treading on toes and no confusion of users. Although this project is currently quite small it would be a good idea to sort these issues out from the get-go. Things we need to discuss:

[ ] Are we allowed to pack the AppCenter Flatpak repository by default into the OS?

As we are using GNOME Software currently (boo!), the pay-what-you-want system is being circumvented and developers are possibly losing out on potential sales. In the future I would like AppCenter to be working on Sodalite along with the payment system.
If we're not allowed, could we do this as an optional thing during on-boarding since curated apps are an important part of elementary, but still letting the user know about the pay-what-you-want system somehow?

[ ] What's the best way to refer to the DE?

In documentation it would be nice to always mention the DE is "Pantheon" and not "elementary", but there doesn't seem to be a canonical link to Pantheon on elementary's website to explain the DE to users: it's all just elementary(OS).
We could make our own explanation of Pantheon?

[ ] Making sure to send beginner users to elementaryOS instead, as Sodalite is far from finished and targets more novice users.
[ ] Potentially out-of-scope, it would be nice to make sure Sodalite-specific issues are not posted in elementary's subreddit or other discussion areas.
[ ] Are we breaking any other licensing issues with elementary?
[ ] Any other potential issues?

Would be nice to get appropriate people from elementary on this (@cassidyjames? @danrabbit?).

Sure you can pre-install the AppCenter remote that's not a problem. We're working on a cross-platform payment solution.
Yes the desktop environment is called Pantheon. The only real reference we have of it on our website is on our open source page: https://elementary.io/open-source
If you're concerned about infringing on trademark/licensing you'll need to consult a lawyer. We're not able to answer legal questions or give legal advice. What I can tell you is that Pantheon is available under the terms of various GPL licenses as described in their respective source code repositories, but our trademarks used in commerce (the elementary "e", the elementary workmark, etc) are not openly licensed.

Sure you can pre-install the AppCenter remote that's not a problem. We're working on a cross-platform payment solution.

That's great to hear!

Yes the desktop environment is called Pantheon. The only real reference we have of it on our website is on our open source page: https://elementary.io/open-source

Hm, alright. Suppose I could do a little bit of explianing in the docs of what Pantheon is and how it differs from elementaryOS. Although, tbf, the users using this should already be aware of the differences :thinking:.
Thanks for the input :smile:.

Also be sure to read through the brand page on the elementary website if you haven't for clarity in usage of the elementary names and marks:
https://elementary.io/brand


Double-clicking a word selects the word plus the leading or trailing space

In the InlineStyleTextArea, when I double-click on a word, it selects the word and its leading or trailing space. I'd like for the double-click to only select the word itself, without the leading or trailing space (similar to the JavaFX text area). Is this possible, and if so, how should I do so?

Here's a quick workaround:
area.addEventHandler(MouseEvent.MOUSE_CLICKED, new EventHandler<MouseEvent>() {
    @Override
    public void handle(MouseEvent event) {
        if (event.getClickCount() == 2) {
            IndexRange range = area.getSelection();
            String selectedText = area.getSelectedText();
            if (selectedText.startsWith(" ")) {
                area.selectRange(range.getStart() + 1, range.getEnd());
            } else if (selectedText.endsWith(" ")) {
                area.selectRange(range.getStart(), range.getEnd() - 1);
            }
        }
    }
});

Thanks for reporting and thanks for the work-around. We should fix this at the core, though.

Although I don't fully understand BreakIterator, I found a way to make it work.

Remove NavigationActions.previousWord()'s line: "wordBreakIterator.previous()"
Remove NavigationActions.nextWord()'s line: "wordBreakIterator.next()"

That will fix this issue, but previousWord and nextWord are used elsewhere, so it will likely break other usages.

So should my workaround be placed after StyledTextAreaBehavior.selectWord() does its normal code? Or should something else be done?

My above question was probably dumb... Oh well.
If a new method was created that does previous/nextWord's method but without that line, would that have any adverse effects?

Something similar to this issue is when the user presses the combination SHIFT+Shortcut+LEFT/RIGHT. The selection will also include an extra space.
For example, let's say "|" represents the caret position in the following sentence
"This sim|ple sentence gives an illustration,"
Pressing the above combination to the right will select "ple " instead of just "ple" (shown by angle brackets)
"This sim<ple >sentence gives an illustration"
Every additional selection, if doing the same key press, will still include the extra space char.
"This sim<ple sentence >gives an illustration"
"This sim<ple sentence gives >an illustration"

If going to the left, the selection will be " sim" instead of "sim"
"This< sim>ple sentence gives an illustration."

How about changing previousWord() and nextWord() to the following (not sure how the javadoc would need to be updated):
default void previousWord(SelectionPolicy selectionPolicy, boolean applyPrevious) {
    if(getLength() == 0) {
        return;
    }

    BreakIterator wordBreakIterator = BreakIterator.getWordInstance();
    wordBreakIterator.setText(getText());
    wordBreakIterator.preceding(getCaretPosition());
    if (applyPrevious) { wordBreakIterator.previous(); }

    moveTo(wordBreakIterator.current(), selectionPolicy);
}

I was actually thinking of something similar, but instead of a boolean, use an int n and then call wordBreakIterator.previous() n-1 times. A behavior-preserving refactoring would be to change all previousWord() calls to previousWord(2). Then to address this issue would be to change it to previousWord(1) somewhere. Maybe changing the method name would also make sense.

That makes sense, so something like this:
/**
* Skips n number of word boundaries backwards.
* Based on the given selection policy, anchor either moves with
* the caret, stays put, or moves to the former caret position.
*/
default void skipToPreviousWordBoundary(int n, SelectionPolicy selectionPolicy) {
    if(getLength() == 0) {
        return;
    }

    BreakIterator wordBreakIterator = BreakIterator.getWordInstance();
    wordBreakIterator.setText(getText());
    wordBreakIterator.preceding(getCaretPosition());
    if (n < 1) n = 1;
    // Do I need a maximum limit based on getLength() here ?
    for (int i = 0; i <= n - 1; i++) {
        wordBreakIterator.previous();
    }

    moveTo(wordBreakIterator.current(), selectionPolicy);
}

Renaming it would make sense, but I'm not sure if my above suggestion is the best naming scheme.

If there's no need for a maximum limit, I guess I better way to write that for loop code:
if (n >= 2) {
    for (int i = 0; i < n-1; i++) {
        wordBreakIterator.previous();
    }
}

I don't really see a difference. What about just
    wordBreakIterator.preceding(getCaretPosition());
    for (int i = 1; i < n; i++) {
        wordBreakIterator.previous();
    }

Yeah, that's simpler.
What about the method's name?

What about wordBreaksBackwards(int n), wordBreaksForwards(int n)?

I think that's good.

OK, if you want to go ahead and submit a PR, go ahead ;)

FYI: Just stumbled across this problem too and looking forward for a fix.

This was fixed in fca5d95273334feb166bbb961167363ee77e5ae2. Tomas just hasn't made a new release with the bug fix included yet.

:+1:
Would be great if this could be released.
Will this also fix the fact double clicking the first word of a new line also selects the previous line separator ?
Thanks

@Crystark (I know it's not exactly ideal but...) you could always use the snapshot version. Also, does that issue arise in the snapshot version? Or only in the 0.6.10 release?

@JordanMartinez The thing is i would have to build the snapshot version myself and make sure it's in my local m2 repository each time i want to build my project. i'm not working on only one computer and i don't see a way to automate that through a maven dependency. Any ETA on a new version (even a beta) being pushed to maven central ?

Forget what i said. looks like the snapshots are being deployed. i'll test that

@JordanMartinez I can confirm my problem with selecting the previous line separator is also fixed in the latest snapshot. Although i lost the cursor turning to text

@JordanMartinez I can confirm everything works fine in the snapshot version. Thanks.
It would be nice if you could release some milestones so that one doesn't have to depend on a moving version.

@Crystark Good to know! Thanks for checking. AFAIK, Tomas hasn't made a new release because when I implemented #152, it broke the undo handling by making it more complex. So, once issue #233 is fixed (you can follow the progress via TomasMikula/UndoFX#8), he should make a new stable release. As for an ETA, I'm not sure. I'll be working on the issue during this week, but it is a complicated issue.
Also, the current snapshot release should be reliable because there's no other work planned besides the undo issue (and fixing any bugs that may be discovered until then). For reference, you could look at #259.

Fixed in 0.7-M1.


Rename Spring Profile dev to debug

Checklist

[x] Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
[x] Make sure mvn install runs for the whole project and, if you touched any code in the respective service, submission and distribution service can be run with spring-boot:run

Description

Rename spring-profile 'dev' to 'debug'
Adjusted documentation accordingly

As the documentation PR were merged, these PR can be updated


Weird "unexpected fault address" error

Hey, I'm not sure if this is at all my fault, but I got this stack trace today. I can't figure out what happened wrong and was wondering if you had seen anything like it before or knew something that I didn't. Thanks!
fatal error: fault
[signal SIGSEGV: segmentation violation code=0x80 addr=0x0 pc=0x4a82e8]

goroutine 212813943 [running]:
runtime.throw(0xb9e8b4, 0x5)
	/usr/local/go/src/runtime/panic.go:605 +0x95 fp=0xc422f0e970 sp=0xc422f0e950 pc=0x42bc15
runtime.sigpanic()
	/usr/local/go/src/runtime/signal_unix.go:374 +0x227 fp=0xc422f0e9c0 sp=0xc422f0e970 pc=0x4424c7
reflect.StructTag.Lookup(0x646465722e777777, 0x6d6f632e7469, 0xba451f, 0xc, 0xc422f0ee90, 0xfc0e50, 0xc4201b5030)
	/usr/local/go/src/reflect/type.go:1165 +0x38 fp=0xc422f0ea20 sp=0xc422f0e9c0 pc=0x4a82e8
reflect.StructTag.Get(0x646465722e777777, 0x6d6f632e7469, 0xba451f, 0xc, 0x0, 0xa6b9c0)
	/usr/local/go/src/reflect/type.go:1148 +0x49 fp=0xc422f0ea68 sp=0xc422f0ea20 pc=0x4a8289
vendor/github.com/mitchellh/mapstructure.(*Decoder).decodeStruct(0xc421a01370, 0x0, 0x0, 0xaac580, 0xc425d207b0, 0xb92040, 0xc4236cec00, 0x199, 0xc422f0f4e8, 0x4109d2)
	/go/src/vendor/github.com/mitchellh/mapstructure/mapstructure.go:706 +0xe26 fp=0xc422f0f400 sp=0xc422f0ea68 pc=0x6ec176
vendor/github.com/mitchellh/mapstructure.(*Decoder).decode(0xc421a01370, 0x0, 0x0, 0xaac580, 0xc425d207b0, 0xb92040, 0xc4236cec00, 0x199, 0x1, 0xc42956a380)
	/go/src/vendor/github.com/mitchellh/mapstructure/mapstructure.go:225 +0x6c3 fp=0xc422f0f518 sp=0xc422f0f400 pc=0x6e64d3
vendor/github.com/mitchellh/mapstructure.(*Decoder).Decode(0xc421a01370, 0xaac580, 0xc425d207b0, 0x0, 0x1)
	/go/src/vendor/github.com/mitchellh/mapstructure/mapstructure.go:180 +0xb3 fp=0xc422f0f578 sp=0xc422f0f518 pc=0x6e5de3
vendor/github.com/mitchellh/mapstructure.Decode(0xaac580, 0xc425d207b0, 0xa51900, 0xc4236cec00, 0x0, 0x0)
	/go/src/vendor/github.com/mitchellh/mapstructure/mapstructure.go:122 +0xa4 fp=0xc422f0f5b8 sp=0xc422f0f578 pc=0x6e59b4
vendor/github.com/turnage/graw/reddit.parsePost(0xc422f0f728, 0xc425d207b0, 0xba3d68, 0xb)
	/go/src/vendor/github.com/turnage/graw/reddit/parse.go:183 +0x6a fp=0xc422f0f600 sp=0xc422f0f5b8 pc=0x6f125a
vendor/github.com/turnage/graw/reddit.parseListing(0xc422a6eea0, 0x1a1e2, 0x1fe00, 0xa51b00, 0xc422a6eea0, 0x0, 0x0, 0x0, 0x0, 0xc423979810, ...)
	/go/src/vendor/github.com/turnage/graw/reddit/parse.go:147 +0x559 fp=0xc422f0f750 sp=0xc422f0f600 pc=0x6f0b39
vendor/github.com/turnage/graw/reddit.parseRawListing(0xc4230c8000, 0x1a1e2, 0x1fe00, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
	/go/src/vendor/github.com/turnage/graw/reddit/parse.go:79 +0x11b fp=0xc422f0f7c8 sp=0xc422f0f750 pc=0x6f027b
vendor/github.com/turnage/graw/reddit.(*parserImpl).parse(0xfea8d8, 0xc4230c8000, 0x1a1e2, 0x1fe00, 0x1fe00, 0x0, 0x0, 0x411200, 0x120, 0xb1fda0, ...)
	/go/src/vendor/github.com/turnage/graw/reddit/parse.go:54 +0x5a fp=0xc422f0f888 sp=0xc422f0f7c8 pc=0x6efe1a
vendor/github.com/turnage/graw/reddit.(*reaperImpl).reap(0xc420b32780, 0xc4227f4b00, 0xa, 0xc4272b6a20, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
	/go/src/vendor/github.com/turnage/graw/reddit/reaper.go:76 +0x239 fp=0xc422f0f978 sp=0xc422f0f888 pc=0x6f16e9
vendor/github.com/turnage/graw/reddit.(*scanner).Listing(0xc4203a0400, 0xc4227f4b00, 0xa, 0xc423962de0, 0x9, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
	/go/src/vendor/github.com/turnage/graw/reddit/scanner.go:42 +0x1f1 fp=0xc422f0fa98 sp=0xc422f0f978 pc=0x6f21b1
vendor/github.com/turnage/graw/reddit.(*bot).Listing(0xc420b35140, 0xc4227f4b00, 0xa, 0xc423962de0, 0x9, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
	<autogenerated>:1 +0xde fp=0xc422f0fbb8 sp=0xc422f0fa98 pc=0x6f312e
vendor/github.com/turnage/graw/streams/internal/monitor.(*monitor).harvest(0xc429fe9c80, 0xc423962de0, 0x9, 0x14, 0xc424068000, 0x14, 0x0, 0x0, 0x0, 0x0, ...)
	/go/src/vendor/github.com/turnage/graw/streams/internal/monitor/monitor.go:96 +0xc9 fp=0xc422f0fce8 sp=0xc422f0fbb8 pc=0x9c9bc9
vendor/github.com/turnage/graw/streams/internal/monitor.(*monitor).Update(0xc429fe9c80, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, ...)
	/go/src/vendor/github.com/turnage/graw/streams/internal/monitor/monitor.go:87 +0xd2 fp=0xc422f0fe58 sp=0xc422f0fce8 pc=0x9c97c2
vendor/github.com/turnage/graw/streams.flow(0xf6ea20, 0xc429fe9c80, 0xc422708cc0, 0xc422708d20, 0xc4236d57a0, 0xc4236d5800, 0xc4236d5860)
	/go/src/vendor/github.com/turnage/graw/streams/streams.go:253 +0x1c7 fp=0xc422f0ffa8 sp=0xc422f0fe58 pc=0x9cb2b7
runtime.goexit()
	/usr/local/go/src/runtime/asm_amd64.s:2337 +0x1 fp=0xc422f0ffb0 sp=0xc422f0ffa8 pc=0x45c131
created by vendor/github.com/turnage/graw/streams.stream
	/go/src/vendor/github.com/turnage/graw/streams/streams.go:230 +0xe9

I won't have time to look into this in detail soon, but it appears that Reddit's response doesn't match the expectations I set up for mapstructure.
That happens here. If I were to debug this myself I'd log the responses there and inspect one that causes the failure. I had some trouble catching various oddities in the Reddit responses, such as fields being boolean sometimes and integers other times. It's likely I missed one of those things.

Should probably be closed since no further information was given and it's been 2 years 😄


Feature Request: auto_arima as a sklearn.model_selection method

First off, thank you for this package, it's great.
It appears you made pmdarima estimator classes to be compatible with sklearn, including Pipeline objects. It would be nice to have auto_arima as a sklearn.model_selection-type optimizer so it could be used in conjunction with a pipeline, unless there's something I missed in the docs. Not sure about the feasibility of this without a major refactor, but everything else seems to be in place for it.

I've thought about this a lot and I think it would be a cool and useful feature. It's a bit tricky, however...
First off, for typical scikit-learn transformers, the fit method's signature is:
def fit(X, y=None):
    ...

where X is a 2d array and y is a an optional 1d array. As you know, for pmdarima, the inverse is true:
def fit(y, exogenous=None):
    ...

which could be re-written:
def fit(y, X=None):
    ...

Obviously, we could make it work by just swapping the X and the y, but that is a bit of a backwards-compatibility-breaking change just to fit it into the Pipeline. It also means that our current practice of making y required and X optional wouldn't work. For instance:
def fit(X=None, y):  # not valid
    ...

Of course there are ways around it like making y default to None and failing out if y is not provided, but that's unintuitive if you're not reading the source code: "why would you make it optional if it's required?"
The scikit code:
Here is where the transformer fitting happens in the pipeline itself (and the estimator portion is pretty similar):
def _fit_one_transformer(transformer, X, y, weight=None, **fit_params):
    return transformer.fit(X, y)

So you'd see our signature would have to change. Otherwise, we'd have to force the fit signature to accept **kwargs only, and that seems like a really kludgy solution.
Short answer
All of this is to say, I'd really love to make this happen but I still haven't thought of an elegant way to handle it without breaking backwards compatibility. So I wouldn't call it a "wontfix" issue but more of a "leave-it-on-the-back-burner-to-see-if-it-gains-any-steam" issue. Fair?

Point taken; thank you for the detailed explanation.

The pipeline was added in #121 and will be released in v1.2.0

Thanks; I'll check it out 👍


v0.6.1 release

Move SDKs into aperture repo (#751)
Scheduler: Load Multiplier (#783)
Add tutorial for signal processing (#784)
Run jb install also in generate-doc-assets hook (#785)
rollupprocessor: Apply per-batch attribute cardinality limit (#782)
Add jobs for building aperture-java (#787)
Update protoc-gen-go-deepcopy and copy go stubs to sdks/aperture-go (#789)
Add send_batch_max_size option (#792)

Description of change
Checklist

[ ] Tested in playground or other setup
[ ] Documentation is changed or added
[ ] Tests and/or benchmarks are included
[ ] Breaking changes

This change is

In theory it shouldn't be a patch release – https://github.com/fluxninja/aperture/pull/783 is a breaking change in policies.

@krdln Sure, what changes should go into 0.6.1 release?

@kklimonda-fn https://github.com/fluxninja/aperture/pull/782 and #792


Event documents transformer

Issue

added contributors to the eventDoc query: that makes its type the same as series on articles
the above could actually be removed, since I ended up not including contributors in display or query. Reason is, it doesn't seem important enough to display on the result list, and I don't see people querying for specific hosts or facilitator. Happy to discuss 💬
Any idea what we could use as "default event format"? it's sometimes undefined
not sure about the times in the display

the above could actually be removed, since I ended up not including contributors in display or query. Reason is, it doesn't seem important enough to display on the result list, and I don't see people querying for specific hosts or facilitator. Happy to discuss 💬

I thiiiink this was because of curators or specific artists, but tbh I'm not sure! Happy to leave out for now and add later if we want it back?