⚠ Consejo de Estado has degraded performance

In 5ec16a9, Consejo de Estado (https://consejodeestado.gov.co/) experienced degraded performance:

HTTP code: 200
Response time: 7414 ms

Resolved: Consejo de Estado performance has improved in ced702c.


Define scanCompleteCB with using std::function

I want to use bind for scanCompleteCB like this.
class Controller {
  void startScan() {
    scanning = true;
    auto pScan = NimBLEDevice::getScan();
    pScan->setAdvertisedDeviceCallbacks(advDeviceCBs);
    pScan->setInterval(45);
    pScan->setWindow(15);
    Serial.println("Start scan");
    // pScan->start(scanTime, scanEndedCB);
    pScan->start(scanTime,
                 std::bind(&Core::scanEndedCB, this, std::placeholders::_1)); // <<<<<<<<<< ======== Here
  }

  void scanEndedCB(NimBLEScanResults results) {
    Serial.println("Scan Ended");
    scanning = false;
  }
};

Related issue: https://github.com/h2zero/NimBLE-Arduino/issues/342

Sorry for the delay.
This is a good idea but we have a problem.
This will break the API for users that do not provide the is_continue parameter when they provide a callback that does not use  std::bind (such as the examples). In this scenario the callback argument is converted to a bool(true) by the compiler and results in calling the wrong start() method, and the callback is not called.
In order to support this correctly another solution is needed. If we are going to break the API, I think the best way would be to have a setScanEndCallback method or a callback class added to NimBLEScan and remove the function parameter from the start calls. This would also require renaming the start() method to support the blocking and non-blocking calls with different naming.
I'm open to other suggestions, this is just my first thoughts.
Note: merging this will need to wait until the other breaking changes I have in mind are implemented with the upcoming 2.0 releases.

Thank you for the response.
I understand that this change breaks current code.

have a setScanEndCallback method or a callback class added to NimBLEScan

I agree passing a class instance for a function because we can support additional callback easily.

I have in mind are implemented with the upcoming 2.0 releases.

I see, thanks.

@asukiaaa Please take a look at #389, it is part of a few breaking changes I have planned for a version 2 release and should suit your needs.

Thank you for updating but it seems renaming m_pAdvertisedDeviceCallbacks to m_pScanCallbacks and reuse it as m_scanCompleteCB.
Is it possible to distinguish the timing of finding some device and finishing scanning?

Sorry, I need to update an examples to make it clearer. The advertised device callback class has been replaced with the scan callback class which contains the virtual function onResult and onScanEnd that you can define for your needs.

Thank you for the explanation.
I could use the updated NimBLE without using global functions and variables.
https://github.com/asukiaaa/arduino-XboxSeriesXControllerESP32/commit/57e50c181ff621cfd5c6521d0ea36bd6ab32c68d

You got it 😄

This request is satisfied by onScanEnd callback on https://github.com/h2zero/NimBLE-Arduino/pull/389.
Thank you.


Add arrow navigation for command history

This mimics the behavior of a terminal: You can use arrow keys ⬆ and ⬇ to navigate to past commands. The user can use this feature when the command box is focused (the behavior is handled in CommandBox.java only).

I will add more test later

I guess we can skip the codecov check for now. It does not affect the overall coverage (-0.65%)

Can you add missing testcases for the logic manager?

I also think it is difficult to test UI.

Can you add missing testcases for the logic manager?

It is 1 line missing 🤔. I only add a getter for history. So no need?

There are no test cases for other getters in LogicManager too

LGTM.


populate bf:source/Source when a heading is added

Attempts to pre-supply thesaurus URIs and labels for genre/form headings in the BF editor have not been successful. Currently, the cataloger must select the heading, and then select the applicable thesaurus.

It's two lookups -- the first one searches https://id.loc.gov/authorities/genreForms and the second one searches https://id.loc.gov/vocabulary/genreFormSchemes
Would there be a way to assign the thesaurus URI/label based on the lookup?

Source of the heading can also be inferred from id.loc.gov URI, and some template changes have lessened the need for this.


Wrong title for donate page

Actual Behaviour
The donate page current title is download | p5.js
Expected Behaviour
It should be donate | p5.js
Screenshots of the issue

Would you like to work on the issue?
Yes, I would love to fix this.

You can go ahead with a fix. Thanks.

Hey I was changing the 'title' in pages/download/support.hsb. But after running it throws an error: Warning: Cannot read property 'Skip-To-Content' of undefined Use --force to continue. What should be the reason?

Can you share a more complete terminal output? Just several lines before it quits may be enough.

That is kinda odd. Do you have the changes pushed to your fork? I'll have a look if so.

Yes, just pushed it in my fork. with commit "update donate title"

hii, i have tried to do the change in the title ,but it throws an error plz.. anyone help me out .

The problem comes from the way that i18n for the web pages look up translated strings in the data structure we use for translations, ie. it expects the title to be a certain string as a result. It is not easy to disentangle the structure here as there are possible overlaps with other strings in the download folder. I personally wouldn't really recommend working this out at the moment.

Yup, figured out the same while going through codebase. I guess we can close this issue for now


AVT camera use different property names

Some cameras use ExposureTime, others ExposureTimeAbs.
Some cameras have StreamBytesPerSecond, others do not.
More properties like these?

Optional: GevTimestampTickFrequency
Different: AcquisitionFrameRate and AcquisitionFrameRateAbs


Not correct map path

Describe the bug
As you can see in following immage, the robot path is strange and confusing, not always allineated to the map and it's note the same of the Xiaomi Home software, and looks like that everytime it update the path it change position and draw on the map
To Reproduce
Just run a clean, it's happen every time
Expected behavior
Just clean 2 rooms everyday
Screenshots
If applicable, add screenshots to help explain your problem.

Additional Information (please complete the following information)

Dreame W10 Pro Model Name [e.g. dreame.vacuum.p2028]:
Firmware Version 4.3.3_1127
Home Assistant Version: 2022.10.0
Configuration Type [With or without map support]: With map support
Errors or warnings shown in the HA logs (if applicable): # grep -r dream home-assistant.log | grep -i dream
2022-11-03 01:05:13.328 WARNING (SyncWorker_0) [homeassistant.loader] We found a custom integration dreame_vacuum which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2022-11-03 01:06:32.368 WARNING (MainThread) [homeassistant.bootstrap] Waiting on integrations to complete setup: dreame_vacuum
2022-11-03 03:27:48.917 WARNING (Thread-1186) [custom_components.dreame_vacuum.dreame.device] Update failed, retrying 1: Unable to discover the device <IP_ADDRESS>
2022-11-03 10:39:28.346 WARNING (Thread-4773) [custom_components.dreame_vacuum.dreame.device] Update failed, retrying 1: Unable to discover the device <IP_ADDRESS>
2022-11-03 10:39:38.350 WARNING (Thread-4774) [custom_components.dreame_vacuum.dreame.device] Update failed, retrying 2: Unable to discover the device <IP_ADDRESS>
2022-11-03 10:39:47.438 WARNING (Thread-4776) [custom_components.dreame_vacuum.dreame.device] Update failed, retrying 3: No response from the device
2022-11-03 12:07:43.491 WARNING (SyncWorker_6) [homeassistant.loader] We found a custom integration dreame_vacuum which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant

Regular connected to the WiFi

How can I do?

3 mails sent!

perfect! Thank you!

Great!

After lastes update it changed a bit but still not ok

ops sorry, just I received a notice on HACS and upgrade that mod :P


Fix potential unused parameter warnings

Some unused parameters in the new BLAS-less implementations of BLAS routines, as well as one in the existing cvodes_anyode headers.

Thanks!


Update Appveyor config to use new codecov uploader

On June 9, codecov released a new version of their coverage uploader program to replace the old bash uploader, which will stop working on February 1 and is currently experiencing scheduled brownouts; see this blog post for more information.  This repository's Appveyor config installs the codecov uploader provided by Chocolatey, which has not yet updated to the newer version.  If the Chocolatey package is not updated in time, the Appveyor config must be updated to install the new codecov uploader directly; see the linked blog post for instructions.

Fixed by PR #35


[BUG] Cluster restoration from k8s v1.25 to v1.24 fails

Rancher Server Setup

Rancher version: v2.7.5-rc5
Installation option (Docker install/Helm Chart): Docker
Proxy/Cert Details: Self-signed

Information about the Cluster

Kubernetes version: v1.24
Cluster Type (Local/Downstream): Downstream

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider): Node Driver AWS

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom) Admin

Describe the bug
Cluster restoration from k8s v1.25 to v1.24 fails due to failure in installation of following charts.

helm-install-rke2-snapshot-controller
helm-install-rke2-snapshot-validation-webhook

Pod log shows following error.
Error: INSTALLATION FAILED: chart requires kubeVersion: >= v1.25.9 which is incompatible with Kubernetes v1.24.13+rke2r1
To Reproduce

Create a rancher server v2.7.5-rc5
Provision a RKE1 k8s v1.24 cluster.
Create a snapshot once the cluster is up and active.
Upgrade the k8s cluster to v1.25.
Once cluster is active, try to restore the cluster to snapshot created in step 3.

Result
Cluster version restores to the k8s v1.24 successfully, however, the above mentioned pods fails to install successfully.
Expected Result
Charts should install successfully along with the cluster restoration.

You could try to remove the kubeVersion Check from the corresponding Helm Charts, i posted a solution for this here https://github.com/rancher/rancher/issues/43092#issuecomment-2423135070


Provide helper method to get windows version info

Add new Windows::VersionHelper module.
These helpers rely on kernel.cs_info ohai instead of Win32 api calls.
Using this helper instead of WindowsVersion object, allows you to rspec test your recipes on linux, or just easily mock the version you want to test.
cc: @aboten

lgtm

:+1:  More reviews @chef-cookbooks/client-windows  @chef-cookbooks/windows-cookbook-collaborators ?

@Annih I was half way into adding this functionality myself. Thanks for putting it all together.

You're welcome @tas50
I just rebase on master branch. Let's review it and merge it ^^


ANTs not finding VTK

Hello, am posting this here on the advice of Brian after sharing a few messages on the ANTs discussion site.
Am trying to build ANTs with VTK so that I can use antssurf. Ave been trying for over a week first on a CENTOS VM and then on Neurodebian. The basic gist of the error message is that ants make can't find VTK whether I add the VTK directory to my path or explicitly define the VTK path in make.
Any suggestions on best practice for installing and building VTK/ANTS+VTK would be much appreciated.
Thank you very much and aw the best, David.

There was a problem with when ITK was built with ITKVTKGlue where including ITK after VTK would clobber the VTK library definitions.  You may want to try find_package(ITK) first followed by find_package(VTK)
find_package(VTK REQUIRED NO_MODULE )
include(${VTK_USE_FILE})
include_directories(${VTK_INCLUDE_DIRS})
set(INIT_VTK_LIBRARIES ${VTK_LIBRARIES})
ITK must be found before VTK
ITK calls find_package(VTK with limited components)
and that creates a limited ${VTK_LIBRARIES} call
find_package(ITK REQUIRED NO_MODULE )
include(${ITK_USE_FILE})
THERE IS A BUG IN ITK that clobbers the VTK_LIBRARIES
variable when ITK was built with itkVTKGlue
set(VTK_LIBRARIES ${INIT_VTK_LIBRARIES})
unset(INIT_VTK_LIBRARIES)
This is just a guess.
Hans
From:  weedavy<EMAIL_ADDRESS>Reply-To:  stnava/ANTs<EMAIL_ADDRESS>Date:  Saturday, April 23, 2016 at 9:39 AM
To:  stnava/ANTs<EMAIL_ADDRESS>Subject:  [stnava/ANTs] ANTs not finding VTK (#310)
Hello, am posting this here on the advice of Brian after sharing a few messages on the ANTs discussion site.
Am trying to build ANTs with VTK so that I can use antssurf. Ave been trying for over a week first on a CENTOS VM and then on Neurodebian. The basic gist of the error message is that ants make can't find VTK whether I add the VTK directory to my path or explicitly define the VTK path in make.
Any suggestions on best practice for installing and building VTK/ANTS+VTK would be much appreciated.
Thank you very much and aw the best, David.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

Thank you very much Hans, I'll give that a go.
Aw the best, David.

Hi,
I am getting the same error: VTK_DIR-NOTFOUND
Although I have installed VTK.
When I set: `VTK_DIR: \usr\local\lib\cmake\vtk-7.1'
which is the same path which ITK is using, when I compile it with Module_ItkVtkglue ON
I am getting follwing error
CMake Error at SuperBuild/External_VTK.cmake: 201 (find_package): Could not find a configuration file for package "VTK" that is compatible with requested version "6.2"
The following configuration files were considered but not accepted:
/usr/local/lib/cmake/vtk-7.1/VTKConfig.cmake, version: 7.1.0
/usr/local/lib/cmake/vtk-7.1/VTKConfig.cmake, version: 7.1.0
/usr/lib/cmake/vtk-6.0/VTKConfig.cmake, version: 6.0.0
ANTs configuration:

ANTs Error Msg:

Can you tell me how to solve this issue?

i dont think this has been resolved.  it may be specific to linux as i
havent seen the same thing on osx.
brian
On Sat, May 14, 2016 at 3:54 AM, Raghav Mehta<EMAIL_ADDRESS>wrote:

Hi,
I am getting the same error: VTK_DIR-NOTFOUND
Although I have installed VTK.
When I set: `VTK_DIR: \usr\local\lib\cmake\vtk-7.1'
which is the same path which ITK is using, when I compile it with Module_ItkVtkglue
ON
I am getting follwing error
CMake Error at SuperBuild/External_VTK.cmake: 201 (find_package): Could
not find a configuration file for package "VTK" that is compatible with
requested version "6.2"
The following configuration files were considered but not accepted:
/usr/local/lib/cmake/vtk-7.1/VTKConfig.cmake, version: 7.1.0
/usr/local/lib/cmake/vtk-7.1/VTKConfig.cmake, version: 7.1.0
/usr/lib/cmake/vtk-6.0/VTKConfig.cmake, version: 6.0.0
ANTs configuration:
[image: ants_cmake_config]
https://cloud.githubusercontent.com/assets/13105636/15267145/f1a5f108-19d6-11e6-8dd1-82fde22b6f9f.png
ANTs Error Msg:
[image: ants_error]
https://cloud.githubusercontent.com/assets/13105636/15267144/f1a59190-19d6-11e6-8431-020ac4be1cb4.png
Can you tell me how to solve this issue?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/stnava/ANTs/issues/310#issuecomment-219206903

I have a little bit of info for this, in the case of the Superbuild, turning on VTK has some issues:

VTK is configured to build after ITK, so ITK build breaks looking for it, I worked around this was a make -j5 build, which starts the VTK build in parallel.
ITK can't find VTK until I add the VTK-install path to CMAKE_PREFIX_PATH (after successfully getting VTK built via the trick in 1

There was a bug in earlier versions of ITK that caused problems when both ITK and VTK were used.
It was addressed on April 5th in b64b0e7c49ebd14f98b47c5d67558adb26a74d24
You may want to verify that ITK is new enough to include that fix.
Hans
From: "Gabriel A. Devenyi"<EMAIL_ADDRESS>Reply-To: stnava/ANTs<EMAIL_ADDRESS>Date: Monday, May 16, 2016 at 5:21 PM
To: stnava/ANTs<EMAIL_ADDRESS>Cc: Hans Johnson<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [stnava/ANTs] ANTs not finding VTK (#310)
I have a little bit of info for this, in the case of the Superbuild, turning on VTK has some issues:

VTK is configured to build after ITK, so ITK build breaks looking for it, I worked around this was a make -j5 build, which starts the VTK build in parallel.
ITK can't find VTK until I add the VTK-install path to CMAKE_PREFIX_PATH (after successfully getting VTK built via the trick in 1

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub

To be clear, I'm saying it works, with my workarounds.
I'm just using the Superbuild. If it's not up-to-date, then it's missing the "fixes"

I think I may have something useful to add here. The ANTs superbuild tries to get VTK4.0 from git but it appears that git no longer houses VTK4.0 (they are at version 7 now apparently) and so the build crashes here. But when you try to use a system VTK with a version other than 4.0 it won't even start because the superbuild looks for v4.0. It might then be useful to update/set the superbuild to accept newer releases of VTK, i.e., v7.
Peace,
David

From: Gabriel A. Devenyi<EMAIL_ADDRESS>Sent: 17 May 2016 15:17:53
To: stnava/ANTs
Cc: DICKIE David; Author
Subject: Re: [stnava/ANTs] ANTs not finding VTK (#310)
To be clear, I'm saying it works, with my workarounds.
I'm just using the Superbuild. If it's not up-to-date, then it's missing the "fixes"

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHubhttps://github.com/stnava/ANTs/issues/310#issuecomment-219731665
The University of Edinburgh is a charitable body, registered in
Scotland, with registration number SC005336.

I resolved this in PR #355


feat: add Webiew::navigate_to_url, closes #776

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[x] No

Checklist

[x] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[x] I have added a convincing reason for adding this feature, if necessary

Other information

Some quick comments, because this looks like it would be important for the usefulness of the library.
First, this feature is not yet in the main branch of the library and I would encourage you to push it as soon as possible. What I am doing is using an IPC_Channel (through Servo, not between Javascript code) to run this library in conjunction with Rust/Iced.
Here's is what I'm working on -
https://github.com/patientplatypus6/ipciceexample
Specifically, I'd like to be able to programmatically push new URLs based on state changes that occur at top level - see line marked below. I can save local state in an enum, but I can't push a new URL.
https://github.com/patientplatypus6/ipciceexample/blob/main/webview/src/main.rs#L84
Thanks for all your hard work on this library.

@patientplatypus6 this has been released https://docs.rs/wry/latest/wry/webview/struct.WebView.html#method.load_url

@amrbashir Oh...ok. Couple of things. First, it should be noted for anyone for anyone that looks for this that the syntax name has changed from "navigate_to_url" to "load_url". The other thing that I don't understand is that this is in a Webview as opposed to a Webviewbuilder and according to the docs on that page --
WebViewBuilder / WebView are the basic building blocks to construct WebView contents and scripts for those who prefer to control fine grained window creation and event handling. WebView presents the actual WebView window and let you still able to perform actions during event handling to it. WebView also contains the associate Window with it.
From https://docs.rs/wry/latest/src/wry/webview/mod.rs.html#771-773 the function is this -
  pub fn load_url(&self, url: &str) {
    self.webview.load_url(url)
  }

So maybe I'm not understanding how to use this with when I load the site with Webviewbuilder. The example on the main page doesn't use Webview. The docs tell me not to use Webview. But there are functions that are only available to use with Webview, including the ability to load urls? I don't get it.
Can you tell me what I'm not understanding?


Error detected while processing "nvim/runtime/autoload/provider/python.vim"

Everytime I try to launch neovim with python support I receive this error:
Error detected while processing /usr/local/Cellar/neovim/HEAD/share/nvim/runtime/autoload/provider/python.vim:
line   14:
E117: Unknown function: remote#host#RegisterClone
line   15:
E117: Unknown function: remote#host#RegisterPlugin

I read this post about how it was having issues with virtualenv.
post >>   https://github.com/neovim/python-client/issues/16
So, I uninstalled virtualenv. But I still have same issue.
Vim does launch; only without python support.
nvim --version
NVIM 0.0.0-alpha+201412041032 (compiled Dec  4 2014 21:18:39)
Commit: 45ae6eadfd3184b860ac19bd2661e1338c4efc89
Build type: RelWithDebInfo
Compilation: /usr/local/Library/ENV/4.3/clang -Wconversion -O2 -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -DINCLUDE_GENERATED_DECLARATIONS -DHAVE_CONFIG_H -I/tmp/neovim-JJDRaW/build/config -I/tmp/neovim-JJDRaW/src -I/tmp/neovim-JJDRaW/.deps/usr/include -I/tmp/neovim-JJDRaW/.deps/usr/include -I/tmp/neovim-JJDRaW/.deps/usr/include/luajit-2.0 -I/usr/local/opt/gettext/include -I/usr/include -I/tmp/neovim-JJDRaW/build/src/nvim/auto -I/tmp/neovim-JJDRaW/build/include
Compiled by<EMAIL_ADDRESS>Features included (+) or not (-):
+acl             +farsi           -mouse_gpm       -tag_any_white
+arabic          +file_in_path    +mouse_netterm   +terminfo
+autocmd         +find_in_path    +mouse_sgr       +termresponse
-browse          +float           -mouse_sysmouse  +textobjects
+byte_offset     +folding         +mouse_urxvt     +title
+cindent         -footer          +mouse_xterm     -toolbar
-clipboard       +fork()          +multi_byte      +user_commands
+cmdline_compl   +gettext         +multi_lang      +vertsplit
+cmdline_hist    +iconv           +path_extra      +virtualedit
+cmdline_info    +insert_expand   +persistent_undo +visual
+comments        +jumplist        +postscript      +visualextra
+conceal         +keymap          +printer         +viminfo
+cscope          +langmap         +profile         +vreplace
+cursorbind      +libcall         +quickfix        +wildignore
+cursorshape     +linebreak       +reltime         +wildmenu
+dialog_con      +lispindent      +rightleft       +windows
+diff            +listcmds        +scrollbind      +writebackup
+digraphs        +localmap        +signs           -X11
-dnd             +menu            +smartindent     -xfontset
-ebcdic          +mksession       +startuptime     -xsmp
-emacs_tags      +modify_fname    +statusline      -xterm_clipboard
+eval            +mouse           +syntax
+ex_extra        -mouseshape      +tag_binary
+extra_search    +mouse_dec       +tag_old_static
   system vimrc file: "$VIM/nvimrc"
     user vimrc file: "$HOME/.nvimrc"
 2nd user vimrc file: "~/.nvim/nvimrc"
      user exrc file: "$HOME/.exrc"
  fall-back for $VIM: "/usr/local/Cellar/neovim/HEAD/share/nvim"

This is happening to me still on OS X, with nvim installed via brew cask install neovim/neovim/neovim:
→  nvim --version
NVIM 0.1.5
Build type: RelWithDebInfo

→ pip show neovim
---
Metadata-Version: 1.1
Name: neovim
Version: 0.1.9

Even a blank init.vim gets me this error when doing :python print "hello".


SMTP email doesn't work in Docker container - 535-5.7.8 Username and Password not accepted

I'm running the docker image huginn/huginn on Synology diskstation. I've tried all combinaties I could find on the internet but, I always get the error
"535-5.7.8 Username and Password not accepted. Learn more at"
I have no idea how to debug this further. Anybody have any ideas?

How did you configure the docker containers and what is your email provider?

Hi,
thanks for the reply.
I pulled in the image huginn/huginn.
I've tried all the stuff I could find here (different ports, authentication plain vs login, etc)
These are the current env variables.

Did you enable access for less secure apps for your GMail account? https://www.google.com/settings/security/lesssecureapps

Did you enable access for less secure apps for your GMail account? https://www.google.com/settings/security/lesssecureapps

No. I use two factor auth, so I use an app password. I verified the settings via an independent email test.

Did you try the setting? To my understanding an app password effectively bypasses the 2FA which should count as a "less secure app".

I tried the setting, but it's not available for 2-factor

Does setting SMTP_DOMAIN=gmail.com help?

I deleted the entire container and started from scratch. it seems that changed a few stuff and my env variables are "read". Now I got some new error when I tries to send an email
Exception during receive. end of file reached: /usr/lib/ruby/2.5.0/net/protocol.rb:189:in rbuf_fill' /usr/lib/ruby/2.5.0/net/protocol.rb:157:in readuntil' /usr/lib/ruby/2.5.0/net/protocol.rb:167:...

https://github.com/huginn/huginn/issues/2592 https://github.com/huginn/huginn/issues/1894 https://github.com/huginn/huginn/issues/1604
These issues look similar, they seem to have had success with using SMTP_PORT=587

after spending hours of debugging, checking time, ssl, veryfing I could make an smtp connection inside the container and so forth..., I finally got around to printing environment variables inside the docker container and noticed that I misspelled "SMTP_USER_NAME" as "SMPT_USER_NAME"
god damn it

Oh no, typos are always the most annoying errors 😿 Do the Agents now work as expected?

Yeah! I couldn't get the gmail to work with the app passwords, but I have an email account from my isp which I don't use and does work.
I'm really enjoying it, I love the simple UI and the power of the liquid syntax, it's clutterfree.

That is great! Feel free to reopen the issue if you have additional questions.

@Tieno
I am having similar issues with Synolgoy/docker/huginn. Can you advise on a few things. Huginn is running well, agents work but i cannot manage to get emails to send with any provider. I have tried sendgrid, gmail and a couple of others.
Have you made any specific settings to your docker install?
Is your Synology connected with a domain name?
Could you provide a full example (redacted when needed) of your environment settings for Huginn?
I would really love to get this working but i am at my wits end.
Best regards

Hi,
I exported the docker settings for the container and redacted it, it should give you an idea.
huginn-huginn1_exported_redacted.zip

@Tieno
Wow, thank you so much, i really appreciate the help! Ill get back to you in the progress.
Thank you again!

Hello,
Have a related question here... I'm running Huginn on Synology, and trying to get email to work but am a little bit confused about .env file and its whereabouts. Should it be called .env to begin with? And where should I locate it? (A bit of a noob here)
Will appreciate any help! Thanks.

So found out the .env file is respected over environment variables. I had my docker container up before i decided to set up SMTP settings, had endless issues using the environment variables on the container. Getting this error and not understanding what was going on. So I got a shell and tried editing the .env with sed. After a restart i got an end of file error when sending, tried changing the EMAIL_FROM_ADDRESS and restarted one more time and email works. So seems after the .env file is generated changing the environment variables does nothing, guessing the file is generated on container creation from the comments above.

This seems like a thing that should be added to the wiki.  Would you mind?

For those whose email spontaneously stopped working, for me it was becauseGoogle disable "less secure" access on May 30 2022
The solution is to (1) use an app specific password and then (2) reset the container because of what @Logiar noted


Problems on color:white

Hi everyone
when compiling a file with this style inside
.ciao
  color:white

the compiler doesn't output anything and it doesn't give me errors

is this a bug?
massimiliano

Thanks for the report @Thorgrim901. When running this code with Ruby Sass I get the following warning and no output.
WARNING on line 2 of test.sass:
This selector doesn't have any properties and will not be rendered.

Node Sass must compile the same as Ruby Sass, so this is not a bug.

@Thorgrim901 You were missing a space between : and w
This is what it should be
.ciao
  color: white


No PWM on a raspberry pi 5

using the following code i am unable to produce a pwm signal on a raspberry pi 5. See attached scope probe image
Its possible i'm just doing something wrong but this code compiles and runs just fine, and is basically pulled straight from the examples

const PERIOD_MS: u64 = 10;
const PULSE_MIN_US: u64 = 500;
const PULSE_NEUTRAL_US: u64 = 1250;
const PULSE_MAX_US: u64 = 2500;

 tokio::spawn(async move {
        let mut pwm = Pwm::with_period(
            Channel::Pwm0,
            Duration::from_millis(PERIOD_MS),
            Duration::from_micros(PULSE_MAX_US),
            Polarity::Normal,
            true,
        )?;
        loop {
            for pulse in (PULSE_MIN_US..=PULSE_NEUTRAL_US).step_by(10) {
                let pwm_res = pwm.set_pulse_width(Duration::from_micros(pulse));
                println!("Res: {:?}", pwm_res);
                sleep(Duration::from_millis(20)).await;
            }
        }
        Ok::<(), rppal::pwm::Error>(())
    });
    ```

Thanks!
Keith Sachs

Hey @whattheflukz. Unfortunately I don't have a Pi 5 available for testing, so I can't run your code myself. I do have a few suggestions you could try to pinpoint the cause of the issue though.
Besides running a minimal synchronous version of the code and seeing if that works, and making sure PWM is enabled in /boot/config.txt, googling for PWM channels on the Pi 5 seems to suggest the RP1 PWM controller uses pwmchip2 on the Pi 5, while hardware PWM on previous Pi models was accessible through pwmchip0. You could verify this with ls /sys/class/pwm and check which devices are available. RPPAL is currently hardcoded to always use pwmchip0, so it's possible that code needs to be tweaked for the Pi 5.
If you have some time to check, grab a copy of the RPPAL repo, and in the pwm::sysfs module, try changing any pwmchip0 paths to pwmchip2 and see if that helps.

Hello @golemparts !
ls /sys/class/pwm reports pwmchip0 pwmchip2 pwmchip6' I tried cloning the repo and changing all instances of pwmchip0to pwmchip2inside of thepwm::sysfsmodule  I also tried running the example code found here https://github.com/golemparts/rppal/blob/master/examples/pwm_servo.rs verbatim with bothpwmchip2as well aspwmchip0set inpwm::sysfs`.
 It's worth checking pwmchip6 as well since, as far as I'm aware, it just uses the next available device number. However, if that also doesn't fix the issue, I'm not sure what else it could be. Let me know whether using pwmchip6 resolves the issues. If not, I'll have to mark this as Help Wanted and get a hold of a Pi 5 myself so I can figure things out.

Hi again @golemparts
I just tried pwmchip6 , also I did make sure to enable pwm in /boot/firmware/config.txt (as I'm using raspberry pi OS) I tried both pwm as well as pwm-2chan unfortunately no luck with any of this.

I also just tried doing the exact same setup on a raspberry pi 4, running the example code verbatim as well, and that works as expected so it's definitely a problem specific to the pi 5. I imagine because of the hardware differences with the new RP1 chip.

Thanks for the additional checks. Yeah, it definitely sounds like an RP1 specific issue. I'll have a Pi 5 coming in soon so I can figure out what's going on.

Hello Im experiencing the same issue, anyone figured out any workaround to get pwm for rpi5 to work?

I finally have a Pi 5 available for testing, so I should be able to dive into this as soon as I get a chance.
RPPAL also has a software PWM available if you're going with @whattheflukz 's alternative suggestion. Worth a shot until this is resolved.

I did some testing today. GPIO18 on the Pi 5 is tied to pwmchip2/pwm2, while RPPAL is hardcoded to use pwmchip0 with pwm0/pwm1 on all Pi models. I assume this change is related to the RP1 now handling the hardware PWM channels. I'll add an exception for Pi models that have an RP1 (and also check the other hardware PWM channels), after which everything should be working as expected.

A fix has been implemented on the master branch. I've run some tests on both Pwm0 and Pwm1 on a Pi 4 and a Pi 5, and everything seems to be working. I'll schedule a 0.18.0 release for tonight or tomorrow, which will include the fix.

A fix has been implemented on the master branch. I've run some tests on both Pwm0 and Pwm1 on a Pi 4 and a Pi 5, and everything seems to be working. I'll schedule a 0.18.0 release for tonight or tomorrow, which will include the fix.

Thanks so much for your effort!

Fixed with release 0.18.0


🐛 BUG: Vue template doesn't work out of the box

What version of astro are you using?
0.22.18
What package manager are you using?
npm
What operating system are you using?
Mac
Describe the Bug
The Vue starter has all kinds of issues out of the box:

There's no Vue shim, so TS can't deal with Vue SFC files
The tsconfig is empty (except for moduleresolution), so TS won't pick up Vue files properly anyway
The Vue component throws error because of 1 and 2
The index.astro file contains a <style global> tag that throws a JS type error: Type '{ global: true; }' is not assignable to type 'ElementAttrs<StyleHTMLAttributes>'. Property 'global' does not exist on type 'ElementAttrs<StyleHTMLAttributes>'.js(2322)

I can't seem to fix 4 at all, so I don't know how to proceed, but these all seem like very important missing things?
Note: Stackblitz won't show you the errors, you can literally add anything to the astro template and it'll be fine with it (I've added a bogus and invalid meta element as an illustration), this needs to be tested locally in a real IDE like VSCode.
Link to Minimal Reproducible Example
https://stackblitz.com/edit/github-uhx2lz-uxjhxt?file=src%2Fpages%2Findex.astro&on=stackblitz

Transfered to language-tools since these all relate to TS issues in VSCode when using Vue. @Princesseuh has done recent work on this with React.

I believe all the issues here have been fixed :smiley:
Some additional settings inside tsconfig.json or a jsconfig.json might be needed for ex: full Volar support depending on if you want to use TS or JS etc but there shouldn't be any errors at least!
Don't hesitate to open another issue if there's anything!

I believe all the issues here have been fixed 😃
Some additional settings inside tsconfig.json or a jsconfig.json might be needed for ex: full Volar support depending on if you want to use TS or JS etc but there shouldn't be any errors on our side at least!
Don't hesitate to open another issue if there's anything!

I download the Astro Framework Vue Example, but volar does not work


operator recipient must improve

currently, operator calls ring two numbers until one answers. We need to be able to add more numbers, ideally stop ringing before voicemail picks up. Probably the best way is with a call queue.

here's one strategy
https://wiki.asterisk.org/wiki/display/AST/Call+Pickup
operator call rings operator extension, which may or may not be connected to a phone
one extension connects user to operator extension
have a public incoming line with a passcode, or admin option on existing line, connect here
have internal extension that connects directly
when operator extension is ringing, notification occurs - text messages, direct calls
text message tells user to call incoming line or internal extension for operator call
direct call calls user, asks user to enter 1 for operator call, connects user to operator extension

Would the operator extension be a callgroup/pickup group? According to this article http://www.voip-info.org/wiki/view/Asterisk+callgroups+and+pickupgroups these are not intended to do multiple outbound calls.
If the operator extension is just going to be a AGI script or something similar that sends notification I don't think we will need to use the Pickup application for this.

no, operator extension is just an extension, which is not even attached to a phone. Or maybe it is, whatever.
This is all based on my reading of those docs, I have not tried any of this, but:
calling user selects operator
dialplan rings operator extension, which probably is not picked up
operator users are notified (however this happens - phone, sms, chatroom, bot)
operator user selects diaplan context that calls CallPickup on ringing operator extension (howerver this happens - operator selects admin menu entry, or confbridge thing, or just picks up call)
call to operator extension is picked up, operator user and calling user are connected


NHS for triangles or edges

After a discussion in slack with @efaulhaber, this now lives in TrixiParticles.jl again.


Add default for NumFollowing field (fixes #2261)

We set the default value for the non-NULL field NumFollowing of the User model to 0, which stops an error when the ORM tries to sync.

Thanks!


Topic/chronos

On my way to try this out...

In Progress...

Using Spark's default log4j profile: org/apache/spark/log4j-defaults.properties
16/03/09 12:58:06 INFO RestSubmissionClient: Submitting a request to launch an application in mesos://spark-dispatcher.service.consul:31266.
16/03/09 12:58:13 INFO RestSubmissionClient: Submission successfully created as driver-20160309125811-0001. Polling submission state...
Dispacher works but not clear how to use chronos UI (it works too)
16/03/09 12:58:13 INFO RestSubmissionClient: Submitting a request for the status of submission driver-
20160309125811-0001 in mesos://spark-dispatcher.service.consul:31266.
16/03/09 12:58:13 INFO RestSubmissionClient: State of driver driver-20160309125811-0001 is now QUEUED.
16/03/09 12:58:13 INFO RestSubmissionClient: Server responded with CreateSubmissionResponse:
{
  "action" : "CreateSubmissionResponse",
  "serverSparkVersion" : "1.5.2",
  "submissionId" : "driver-20160309125811-0001",
  "success" : true
}


JSON-LD schema markup

I think we should consider moving to JSON-LD for the offer (and other) schema markup. It's more easy to manage updates as the standards evolve. How it is set up right now it is too easy to break the structure, and updating is overly complex.
See https://developers.google.com/search/docs/data-types/products
Thoughts and feedback very welcome 🤓

Agreed 👍
nice tidy snippet.

See #78


[Question]: Does it use the linux kernel of the host OS?

Is your question not already answered in the FAQ?

[X] I made sure the question is not listed in the FAQ.

Is this a general question and not a technical issue?

[X] I am sure my question is not about a technical issue.

Question
Does the Windows OS in the container make use of the linux kernel service of the host OS?
It may sound vague. Let me explain a bit. As we know if it is a linux container running, the container will share the linux kernel of the host OS and the container is just a user space application from the host OS's perspective. I'd like to know the situation when the Windows OS runs in the container and the host OS is linux.
I ask because I have a user-space SCTP stack implemented using raw IP. It cannot co-exists with LKSCTP(linux kernel SCTP stack), but we cannot guarantee the LKSCCTP is not installed in the kernel.
Thanks a lot!

Thank you for the detailed explanation.


"Pătroaia-Vale", "Potlogeni-Vale" are listed in the drop-down menu

Description:
When user writes ”Pătroaia-Vale” and "Potlogeni-Vale" the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Pătroaia-Vale"

Expected results
The location appears in the drop-down menu as "Sat Pătroaia-Vale, DÂMBOVIȚA (Crângurile)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Potlogeni-Vale"

Expected results
The location appears in the drop-down menu as "Sat Potlogeni-Vale, DÂMBOVIȚA (Crângurile)".


Possible to make navbar dropdown label take HTML instead of text only?

Is it possible to have navbar dropdown label allow HTML instead of plain text? I need to add an icon to the left of the caret, sort of like user icon in the image below.

Try passing in new HtmlString("<foo>...</foo>") where you would normally pass in the content. Does that help? If not, care to post an example so I can take a closer look?

Html.Bootstrap().DropDown() takes string, so passing an HtmlString does not work. Here's the code I'm using.
@using(var dropdown = Html.Bootstrap().DropDown("Text Label").Begin())
{
    ...
}

@daveaglick Did you get a chance to look at this?

Haven't quite gotten to it yet. I was on the verge of shipping a new version of another OSS project so that took all my free cycles. I'll try to take a look in the next few days.

No rush, I think I can work something out for now. Thanks for this awesome library!

I finally had a chance to take a close look at this. Unfortunately, I don't think what you want can be achieved right now.
Most components are really flexible and you can use methods like .Content() or .AddChild() to insert HTML and other elements into them. With navbar dropdowns that also works, but it always inserts stuff inside the dropdown. That's because all the stuff that sets up the dropdown inside the navbar is generated before it starts processing children.
The text in the constructor gets inserted via a TextContent property that all components have. The navbar checks to see if this is set and then adds it inside all the stuff at the top of the dropdown. Right now, this TextContent property is just a string and it automatically gets escaped. To make something like what you have in mind work, we would probably to somehow indicate whether this property should be escaped when used (maybe by changing to an object and then allowing it to take IHtmlString or something). That would be a pretty major change since this property is used in lots of components.
Which brings us to: I just don't see having the time to tackle it at the moment. While I still fix bugs and answer questions about this project, it's currently in "maintenance mode" while I shift attention to other projects. I plan to revisit it at some point, but not right now. Sorry!

I'll leave this issue open as a reminder to revisit later.

@daveaglick Thanks for a detailed response. Not an issue, I understand you're quite busy. I'll play around with it in a fork. If I ever get it to work, I'll send a PR your way for you to review.
Once again, thanks for the awesomeness!


tile.wrap(true)

Related to http://leafletjs.com/reference.html#tilelayer-nowrap
For Mercator tiles it would be nice to enable wrapping.

It looks like @jasondavies attempted this in his example Wrapped d3.geo.tile. The tile provider there is broken, so I made a fork that uses OSM. It doesn't work properly, but maybe there are some ideas there we can draw from.

So I tried implementing this in d3-tile, but it turns out to be a bit tricky. This is because the tile image transform needs to depend on the original x coordinate, but the tile URL x needs to be wrapped.
Some ideas:

Since wrapping only concerns the URL, it could be implemented inside the URL generator.
We could add a special property, say x, to each tile address array. This could be used for generating the URL, whereas the original d[0] x tile coordinate can still be used to drive the transform.

Maybe the structure returned for each tile could be modified completely, to look something like this:
{
  x: // Tile X for URL (with wrapping applied)
  y: // Tile Y for URL
  z: // Tile Z for URL
  transform: // transform to apply to image (without wrapping applied)
}

Having transforms returned per tile like this would obviate the need for tiles.transform. It would also make it easier to deal with tiles at different zoom levels at the same time (e.g. for showing a blurry tile in the background until all of its quadtree children load).

Maybe the structure returned for each tile could be modified completely, to look something like this:
{
  x: // Tile X for URL (with wrapping applied)
  y: // Tile Y for URL
  z: // Tile Z for URL
  transform: // transform to apply to image (without wrapping applied)
}

Having transforms returned per tile like this would obviate the need for tiles.transform. It would also make it easier to deal with tiles at different zoom levels at the same time (e.g. for showing a blurry tile in the background until all of its quadtree children load).

I think I nailed the wrapping logic in a second attempt:

If we unpack the existing logic for the rows and columns ranges, it looks like this:
colMin = Math.max(0, Math.floor((x0 - x) / k)),
colMax = Math.max(0, Math.ceil((x1 - x) / k)),
rowMin = Math.max(0, Math.floor((y0 - y) / k)),
rowMax = Math.max(0, Math.ceil((y1 - y) / k)),
cols = d3Array.range(colMin, colMax),
rows = d3Array.range(rowMin, rowMax);

Using Math.max(0... on colMin is what was preventing Westward wrapping. That was changed to this:
colMin = Math.floor((x0 - x) / k),

Using Math.max(0... on colMax was confusing to me, and I'm not sure what case the Math.max is handling here, so I changed it to:
colMax = Math.ceil((x1 - x) / k),

Using Math.max(0... on rowMin has the perfect behavior of excluding tiles beyond the top of the map.
Using Math.max(0... on rowMax was als confusing to me, and I'm not sure what case the Math.max is handling here. The value of rowMax going too high was causing the invalid image tiles to be requested, going too far South on the map. Since the maximum row value is part of a quadtree address, the maximum valid value depends on the zoom level, namely 2^z. To account for this, I changed the logic for rowMax to this:
rowMax = Math.min(Math.ceil((y1 - y) / k), 1 << z0),

Testing this out by zooming in and out at the southern end of the map seems to indicate it is working correctly.

Here's an example that shows wrapping being turned on and off.
With the addition of a wrap boolean, the row and column logic looks like this:
colMin = Math.max(wrap ? -Infinity : 0, Math.floor((x0 - x) / k)),
colMax = Math.min(Math.ceil((x1 - x) / k), wrap ? Infinity : 1 << z0),
rowMin = Math.max(0, Math.floor((y0 - y) / k)),
rowMax = Math.min(Math.ceil((y1 - y) / k), 1 << z0),

cols = d3Array.range(colMin, colMax),
rows = d3Array.range(rowMin, rowMax);

wrap(true)

wrap(false)

Updating images to be centered:

Love love love this feature. Any current plan for an upcoming release?


Modify Chiasm constructor to take a selection string

Currently, the Chiasm constructor expects a DOM element. This means that writing code to use Chiasm involves something like this:
var containerDiv = document.getElementById("chiasm-container");
var chiasm = Chiasm(containerDiv);

It would be nicer to be able to simply pass a selector string into the constructor, like this:
var chiasm = Chiasm("#chiasm-container");

Both of the above snippets should work for instantiating Chiasm.

Decided against this strategy, and instead solved the issue by making containerSelector a public property in chiasm-layout.


yarn "dev" doesn't compile *.hbs files

After running "yarn dev", changing the *.hbs files doesn't update the HTML outputs. I have to run "yarn build" and then "yarn dev" again to see the changes in HTMLs.

#dfecf10

dfecf10078cdd17983e31a3c552e2cb4cbf55da4


Remove long obsolete DB-to-Endpoint mapping page

The ability to restrict an endpoint to certain databases is last mentioned in 3.0. Was probably a 2.x feature.
https://github.com/arangodb/arangodb/blob/3.0/Documentation/Books/Manual/Administration/Configuration/Endpoint.md

@jsteemann WDYT about my suggestion?
Database Organization
A single ArangoDB instance can handle multiple databases in parallel. By default,
there will be at least one database which is named _system.
Data is physically stored in .sst files in a sub-directory engine-rocksdb
that resides in the instance's data directory. A single file can contain
documents of various collections and databases.
ArangoSearch stores data in database-specific directories underneath the
databases folder.
Foxx applications are also organized in database-specific directories but inside
the application path. The filesystem layout could look like this: ...


Travis: harden compilation the way Fedora and Ubuntu do

Fedora and Ubuntu use these options to turn some bugs into compilation
failures and runtime crashes:

https://fedoraproject.org/wiki/Changes/HardeningFlags28
https://fedoraproject.org/wiki/Changes/HardenedCompiler
https://wiki.ubuntu.com/ToolChain/CompilerFlags

I might add "-Wformat -Wformat-security" to the Makefile, because
they're only affecting the compilation. The rest are runtime checks, and
I'm on the fence about enabling them for all downstream users. Let's
keep deferring the decision to distributions for now.
This commit is inspired by #542.

Coverage increased (+0.02%) to 40.901% when pulling efda9d415359cb67e86f31049fb450d45ee60ec3 on feature/fortified-ci into 3320e011ae9004519af8b7e6b6b90bf45565a9e1 on master.


Add Jamf Protect to the Content Hub

Adding a new workbook to the Content Hub that provides visual insight into events captured by Jamf Protect.
Required items, please complete
Change(s):

New workbook for Jamf Protect into the Sentinel Content Hub.

Reason for Change(s):

Adding new workbook into the Content Hub

Version Updated:

N/A

Testing Completed:

YES

Checked that the validations are passing and have addressed any issues that are present:

YES

Please fix the validation errors, hope below information will help in fixing them:

Logo file added is not valid. If you are adding GUID make sure it is unique.
In workbook fallbackResourceIds field is having Subscription details make sure to remove hardcoded values and follow the workbook template for fixing any other validation issues.
Please share the sample data to test the workbook.
Let us know if you need any help over here.
Thanks.

@v-mchatla Thank you for the support on getting this into the Content Hub.

Any guidance on creating a proper SVG?
Could you guide me on how to export sample data and have it transferred to you?

@v-spadarthi @v-mchatla I have updated the logo, the workbook and provide sample data in the related folders. Please let me know if anything needs to be done from my side still!

Hi @txhaflaire,
Thanks for the update. I will verify the changes and let you know if anything required from your end.
Thanks

Hi @txhaflaire
Changes looks good to me. Just need small confirmation - We have PR #6372, seems both the Pr's having same workbook. Have you created 2PR's for the same workbook changes. If yes, please close this PR as it is part of individual folder, and 6372 Pr is already having package for the same.
Thanks

I think this one can be closed; my main goal was to start publishing in the Content Hub, which seems it needs to be in the Solutions folder and creating a Solution package. So if having the Workbook in this location is not required let me know so we can close the PR.
Just for my understanding, what could be a reason to publish a Workbook in this Workbook folder?

Hi @txhaflaire
Thanks for the confirmation. You can close this PR, we will review and merge the PR #6372.
To give some context around the old way of maintaining content in individual folders has some drawbacks for example each folder will gets deployed individually, user may see stale data till all the content gets deployed to avoid this we are maintaining all the content of a solution in single folder, so all the content gets deployed in one go.
Hope this answers your question.
Thanks


Update Sentry env to align with UI

Ike Plugins (test-keeper)
Thank you @aslakknutsen for this contribution!

It appears that no tests have been added or updated in this PR.
Automated tests give us confidence in shipping reliable software. Please add some as part of this change.
If you are an admin and you are sure that no test is needed then you can use the command /ok-without-tests as a comment to make the status green.
For more information please head over to official documentation. You can find there how to configure the plugin - for example exclude certain file types so if PR contains only them it won't be checked.

/ok-without-tests

@aslakknutsen snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1969-1

Codecov Report

Merging #1969 into master will not change coverage.
The diff coverage is 0%.

@@          Coverage Diff           @@
##           master   #1969   +/-   ##
======================================
  Coverage    64.2%   64.2%
======================================
  Files         147     147
  Lines       13718   13718
======================================
  Hits         8808    8808
  Misses       4073    4073
  Partials      837     837

Impacted Files
Coverage Δ

configuration/configuration.go
67.18% <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 f194fd3...f937bb9. Read the comment docs.

@aslakknutsen snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1969-2

/ok-without-tests


GH-133 && GH-347 - latedef now plays nice with undef

Fixes GH-133
Fixes GH-347
I added all the test cases that were present in the comments as well as a few others.

Closing this pull request because it includes commits from another pull request.


[Composer] Design view is not getting loaded when trying out with "try catch block"

Description:
Design view is not getting showed when we try out with try catch block
Steps to reproduce:
Open the blob type .bal file
Add a try catch block in the design view.
Then design is disappeared and below error showed in the UI.
(attached a screenshot)
Affected Versions:
ballerina-tools-0.970.0-beta15-SNAPSHOT.
OS, DB, other environment details and versions:
macOS High Sierra
ballerina-tools-0.970.0-beta15-SNAPSHOT
JDK 1.8.0_152
Suggested Labels (optional):
Type/Bug
Severity/Blocker
Priority/High
Affected/ballerina-tools-0.970.0-beta15-SNAPSHOT

Duplicate of https://github.com/ballerina-platform/ballerina-lang/issues/8058


Gps level

Why does this PR have the commit from the previous PR? They should be seperate. Please get rid of it ( the commit with gains import and export) otherwise it'll cause a merge conflict. Make sure you switch to master before creating a new branch so that this won't happen.


[WIP] Namespace the mappable object types, add Amazon VM and Image types.

This adds Amazon images and vm's as a mappable type to the ContainerLabelTagMapping model. Furthermore, I've scoped the names by provider type.
This serves two purposes. First, it modifies the drop down menu, and lets the user know exactly which type of resource that they're mapping:

Second, rather than use the hard coded AUTOTAG_PREFIX constant, the prefix will be parsed from the name. This will require a minor modification on the UI side. The net result will be that you will see entries in this form inside the tags table:
/managed/kubernetes:container_node:container_stuff
/managed/amazon:vm:test_label
/managed/kubernetes::label_for_all
/managed/amazon:image:aws_image_label

The UI modification is here: https://github.com/ManageIQ/manageiq-ui-classic/pull/666

@blomquisg Please review.

See comment I just left on #14236.
I wonder if we really need combinatorial explosion of providers * entities, especially considering that current UI won't let you map 2 entity types to same category.
I'm NOT certain anybody actually needs this on containers side, we implemented it because "the backend already allowed it and it wasn't too hard in UI" ;-)  So in case you're only extending it because we already had it, let's talk :-)
And if we do need it, I wonder what resolution of "provider" we need.  E.g. "kubernetes" here actually stands for both Kubernetes & Openshift — i.e. any containers provider.  And while currently Amazon will be the only other option, will one want separate mappings for Amazon Vms vs Azure Vms vs oVirt Vms?

@cben, the goal is to automatically map custom attributes from AWS vm's and images with any mappings the user has created. This would then be applied during a refresh. @blomquisg please correct me if I'm wrong.

Can user have both vms and images with custom attribute
"department=propulsion"?  Would they want those mapped to one "Departments"
category or independent categories for Vms and Images?
I suppose user could even have both amazon vm images and container images
labeled with "project=warp-drive".  Would they want those mapped to one
"Projects" category or independent categories for amazon and containers?

@cben I will defer to @blomquisg and @bascar for the answer to your questions.

@cben vm and images would use the same tagging, not independent. I think we would need containers and amazon separate since in many cases it is two independent groups who run those environments and they may not want to cooperate on the tagging.

So reading through this, I guess we need:
Amazon/All
Amazon/Vm
Amazon/Image
Containers/All
Containers/Project
Containers/...

Instead of one encompassing <ALL> group.
@bascar, @cben, @djberg96 sound about right?
@djberg96 is that signing up for a whole mess of work? Or, is that easy to finesse here?

@blomquisg Unfortunately that's not so easy because the current code is assuming nil is ALL.

I tested against SCVMM and SCVMM2016 and both seem to be working fine.

Unfortunately that's not so easy because the current code is assuming nil is ALL.

@bascar given the current timeframe for the Fine release, I'm thinking the first phase should be to assume ALL means literally anything that can be mapped from any provider enabled for Tag Mapping.
Currently ALL means anything that can be mapped from any containers provider.
We want to have as little change to the mapping logic as possible to avoid the risk of impacting the Container-specific mappings.
@djberg96 continue under this assumption for now.  We can figure out a course correction later if @bascar comments otherwise.

I am good with the minimized impact of the "All" mapping

The net result will be that you will see entries in this form inside the tags table:
/managed/kubernetes:container_node:container_stuff
/managed/amazon:vm:test_label
/managed/kubernetes::label_for_all
/managed/amazon:image:aws_image_label

The 3rd one is subtle, it's an "All" mapping, with kubernetes: prefix just for compatibility.  There will be no amazon::foo categories for now.  I'm OK with that, though at some point kubernetes:: would change meaning from really All to Containers/All (if you do a migration, perhaps we could avoid it).

Need to change map_labels() calls in kubernetes/container_manager/refresh_parser to use entity_type when with Kubernetes:: prefix.

And support no prefix for compat?

I'd prefer to see a migration adding prefix to all existing mappings, rather than compatibility code in several places...

@cben I agree that we'll need to do a migration that adds a provider column eventually. But, at the moment, there are internal methods depending on that nil to check for ALL.
That will be the next refactoring, but for now I'm keeping it as simple as possible.

This is great @djberg96 !

I meant a migration changing Foo -> Kubernetes::Foo within a single string column, so you don't need compatibility checks to support both Foo and Kubernetes::Foo.
But if you prefer to have checks first, your call.
:+1:, a provider column would be even better.

I actually had written a message here and forgot to click "Comment", then I saw @cben's comment saying the same thing.
I think there needs to be a data migration to move the existing labeled_resource_type mappings in Container_Label_Tag_Mapping from things like ContainerNode to Kubenetes::ContainerNode.

@blomquisg At the moment that value isn't touched and just stores "ContainerNode" or "Vm". I'm afraid of what changing that will do. Plus, once we add a provider column to the table, it would become redundant.
What I could do is just add a provider column now and populate it, but not do anything with it just yet.

Ignore those last rubocop warnings, they were for a commit that I yanked.

@miq-bot remove_label wip

@blomquisg Ok, ready to go.

So, the only thing we're looking for as a follow on PR is a provider column in the container_label_tag_mapping.  For now, it's fine because it's just two different provider types with mutually exclusive labeled_resource_types.

Sorry, had a frantic week.
Wait, Kubernetes::Foo mappings created after this will not be understood by kubernetes refresh.
OK, let me make a fix PR real quick.
I'm going to lift some of your UI compatibility logic into the mapping model, you'll then be able to call it from UI.

@cben I don't see how. If you select "Kubernetes::ContainerNode", then the tag it generates is "/managed/kubernetes:container_node:foo".

See map_labels( calls in container refresh_parser.


Fix outdated apt cache issues

Hey,
I had failures due to removing apt update commands and relying on the postgres role. However the very first apt install commands are run without update_cache and failed. This should solve this kind of problems for all distros using apt.

Hi,
Ok. I'll update the version number as you wish.
I can not accept your last commit.
It is not idempotent.
I got your point. I'll solve it.
Give me 1-2 days.
Regards,
Balázs
On 2023. január 17. 14:07:02 janArb @.***> wrote:

PS: Could you create a new tag 2.3.1 after applying the fix, please?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


SwiftGd is not doing anything on mac

I have the image uploaded to disk
            let url = URL(fileURLWithPath: path)
            let image = Image(url: url)
            if let im = image {
                if let mg = im.resizedTo(width: 250, height: 250){
                    mg.write(to: url)
                }
            }

I tried to resize the image and save it with no hope?

My bad
I have to add a new path


Link creating challenges tutorial

Link the github wiki pages which seem to have some 'creating challenges' tutorial info

Thanks!


Update json4s-native to 4.0.7

About this PR
📦 Updates org.json4s:json4s-native from 4.0.5 to 4.0.7
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.json4s", artifactId = "json4s-native" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.json4s", artifactId = "json4s-native" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (862ac3a) 60.51% compared to head (60410ed) 60.60%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #409      +/-   ##
==========================================
+ Coverage   60.51%   60.60%   +0.09%
==========================================
  Files         152      152
  Lines        3277     3277
  Branches      101      105       +4
==========================================
+ Hits         1983     1986       +3
+ Misses       1294     1291       -3

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


Replace UUID based IDs with a atomic-counted ones

Objective

alternative to #2895
as mentioned in #2535 the uuid based ids in the render module should be replaced with atomic-counted ones

Solution

instead of generating a random UUID for each render resource, this implementation increases an atomic counter

this might be replaced by the ids of wgpu if they expose them directly in the future

I have not benchmarked this solution yet, but this should be slightly faster in theory.

Bevymark does not seem to be affected much by this change, which is to be expected.

Changelog
Changed

Nothing of our API has changed, other than that the IDs have lost their IMO rather insignificant documentation.
Maybe the documentation could be added back into the macro, but this would complicate the code.

If nothing has publicly changed, please remove the Changelog section in the PR description. That part gets scraped to produce the public facing changelog.


Added Russian - Armenian University (RAU)

Added Russian - Armenian University (RAU)

Also here is the home page of Institute of Mathematics and Informatics in RAU, but it is in Russian only.

@vanushwashere I am sorry, you requested a change in a file KotlinJavaRuntime.xml, and I was not able to remove the change. Please submit a new request again, which includes only one file, lib/domains/am/rau/student.txt
Thank you!


Be more permissive of duplicate fields with different selections

GraphQL allows you to do this:
query {
  myField {
    # these two get merged:
    subField { subSubField1 subSubField2 }
    subField { subSubField3 subSubField4 }
  }
}

and even this:
query {
  myField {
    subField { subSubField1 subSubField2 }
    ... on OnePossibleConcreteType {
      subField { subSubField3 subSubField4 }  # merged in if the concrete type matches
    }
  }
}

(Well, it sometimes lets you do that.  The rules are very arcane.))
For now, to simplify things, I've simply disallowed any duplicate fields of composite type in genqlient.  There's no structural limitation preventing us from adding support, if we do find it useful, we'd just need to do more complicated merging in convertSelectionSet.  (there would be some subtleties to handle around type naming.)  But my guess is this is quite rare (and easy enough to work around).
Note that we do already allow duplicated leaf fields, because they're much easier to deduplicate.  We also allow duplicated fields if at least one is in a name fragment, because in that case we needn't merge the fields at all.

👍


Problem with Talktorials 008-010

Hello,
I want to notice that the support of mmtf.rcsb.org is no longer available, so the function mmtf.fetch() in opencadd.Structure doesn't work anymore. I ask you, please, change this code so it can work. If you know the avaliable url for mmtf library, I could change it in utils/constants, so I can use these talktorials.
It returns me Errno 11001: gaierror and URLError
Terminal: pip install mmtf-python
Console:
import mmtf
mmtf.fetch('4TRO')
4TRO is for example

Hello,
Many thanks for reporting this to us. Apparently the MMTF format is no longer supported by the RCSB and replaced by the BCIF format.
https://www.rcsb.org/news/feature/65a1af31c76ca3abcc925d0c
Hence, also the mmtf package, which is a Python implementation of the MMTF API, is not in use any more.
We will have a look how to use the BCIF API instead.
As the corresponding function in opencadd‘s Structure object mimics mdanalysis‘ Universe object, I think we also have to check in there.
https://github.com/MDAnalysis/mdanalysis/blob/develop/package/MDAnalysis/coordinates/MMTF.py#L56
@AndreaVolkamer is there anyone in your lab who can take a look at this or even has already dealt with this?
Many thanks!

I also noticed that I couldn't install package teachopencadd where was function seed_everything () because of the tensorflow-base.2.18.0 (something like this). Conda told me that I had corrupted file, pip told me that there was parse-function problem(if I remember correctly). I only want to say that I could resolve this problem with the help of pip install pytorch-lightning=1.7.7, where pytorch-lightning utilities.seed had the same function.

I am sorry for interrupting, I also have a question, when can the problem with mmtf package be resolved (changing the library, I mean)?
Anyway, thank you very much for noticing my message_s!

I also noticed that I couldn't install package teachopencadd where was function seed_everything () because of the tensorflow-base.2.18.0 (something like this). Conda told me that I had corrupted file, pip told me that there was parse-function problem(if I remember correctly). I only want to say that I could resolve this problem with the help of pip install pytorch-lightning=1.7.7, where pytorch-lightning utilities.seed had the same function.

Hi @Q84-wq, I moved this issue to a stand-alone issue:
https://github.com/volkamerlab/teachopencadd/issues/421

I am sorry for interrupting, I also have a question, when can the problem with mmtf package be resolved (changing the library, I mean)? Anyway, thank you very much for noticing my message_s!

Hi @Q84-wq,
I cannot give a good timeline as it is holiday season for most people in the lab atm and I am on mat leave. I'd think a couple of weeks at least to get all the updates in. I am very sorry for the inconvenience!

Progress on solving this issue on opencadd's end here: https://github.com/volkamerlab/opencadd/issues/159

T008: HTTP error to retrieve ligand information
1.3.5. Get metadata of ligands from top structures
While trying to fetch the PDB information about the top top_num ligands using get_ligands,
command 49 and 50 showing HTTP error and unable to retrieve the ligand information.
def _fetch_ligand_expo_info(ligand_expo_id):
"""
Fetch ligand data from ligand-expo.rcsb.org.
"""
r = requests.get(f"http://ligand-expo.rcsb.org/reports/{ligand_expo_id[0]}/{ligand_expo_id}/")
The error message:
HTTPError                                 Traceback (most recent call last)
Cell In[50], line 14
12 rows = []
13 for pdb_id in selected_pdb_ids:
---> 14     ligands = get_ligands(pdb_id)
15     # If several ligands contained, take largest (first in results)
16     ligand_id, properties = max(ligands.items(), key=lambda kv: kv[1]["Molecular weight"])
Cell In[49], line 15, in get_ligands(pdb_id)
13 ligands = {}
14 for ligand_expo_id in ligand_expo_ids:
---> 15     ligand_expo_info = _fetch_ligand_expo_info(ligand_expo_id)
16     ligands[ligand_expo_id] = ligand_expo_info
18 return ligands
Cell In[49], line 53, in _fetch_ligand_expo_info(ligand_expo_id)
49 """
50 Fetch ligand data from ligand-expo.rcsb.org.
51 """
52 r = requests.get(f"http://ligand-expo.rcsb.org/reports/{ligand_expo_id[0]}/{ligand_expo_id}/")
---> 53 r.raise_for_status()
54 html = BeautifulSoup(r.text)
55 info = {}
File ~.conda\envs\teachopencadd\lib\site-packages\requests\models.py:1024, in Response.raise_for_status(self)
1019     http_error_msg = (
1020         f"{self.status_code} Server Error: {reason} for url: {self.url}"
1021     )
1023 if http_error_msg:
-> 1024     raise HTTPError(http_error_msg, response=self)
HTTPError: 403 Client Error: Forbidden for url: http://ligand-expo.rcsb.org/reports/8/8AM/
########
Kindly help me to solve this issue.

Sorry to bother you:
How to deal with Structure.from _ atomgroup ( complex _.select _ atoms ( f 'resname { ligand } ' ) )

ValueError                                Traceback (most recent call last)
Cell In[58], line 1
----> 1 ligands = [
2     Structure.from_atomgroup(complex_.select_atoms(f"resname {ligand}"))
3     for complex_, ligand in zip(complexes, pairs.values())
4 ]
5 ligands
Cell In[58], line 2, in (.0)
1 ligands = [
----> 2     Structure.from_atomgroup(complex_.select_atoms(f"resname {ligand}"))
3     for complex_, ligand in zip(complexes, pairs.values())
4 ]
5 ligands
Cell In[41], line 73, in Structure.from_atomgroup(cls, *args)
71 for a in args:
72     if len(a) == 0:
---> 73         raise ValueError("cannot merge empty AtomGroup")
75 # Create a new topology using the intersection of topology attributes
76 blank_topology_attrs = set(dir(Topology(attrs=[])))
ValueError: cannot merge empty AtomGroup


Add OpenSSL-related file extensions

There are a number of OpenSSL-related file extensions (e.g., .pem, .crt,
etc..) that contain data that are generally best not committed to
repositories.  This file contains several common file extensions that
often correlate to these types of files.
Reasons for making this change:
Often times keys, certificates, etc., when they're committed, it's on accident.  This file excludes several common extensions used by OpenSSL, a popular tool for working with SSL / TLS certificates and such.  As I couldn't find resources for excluding these files, I'm submitting a new .gitignore file.
Links to documentation supporting these rule changes:
https://datatypes.net/openssl-file-types

@wesley-dean-flexion thanks!


widget unmounted error with multiple swipers in listView

observed with card_swiper v1.0.4
running into an issue where I have a listView of swipers, they are appropriately unmounted as they go out of scrollview, however if I reverse scroll back (with a previous swiper coming back into view), I occasionally get the following error:
"This widget has been unmounted, so the State no longer has a context (and should be considered defunct)."
it looks like it is triggered when the TransformerPageView tries to get the renderObject from the context (below), but not sure why it's being called after it's unmounted? possibly a timing issue? open to any suggestions.  -- thanks!
void _onGetSize(_) {
    Size? size;

    // if (context == null) {
    //   onGetSize(size);
    //   return;
    // }
    final renderObject = context.findRenderObject();
    if (renderObject != null) {
      final bounds = renderObject.paintBounds;
      size = bounds.size;
    }
    _calcCurrentPixels();
    onGetSize(size);
 }

update: I notice that there is no "if (mounted)" check before the context.findRenderObject() is called, and also am curious why the context == null check has been commented out?

yup, i am facing same issue right now

checked on latest v2.0.1, still an issue.  I'm ok with simply commenting back in the null check for the context, but I'd be a little concerned since it was purposely commented out as some point, so not sure of the repercussions?

I am facing same issue too at 2.0.1

I was using a CustomScrollView so instead of uding SliverToBoxAdapter i shifted to SliverPersistentHeader and then issue was gone.
I hope it helps others as well


Switch from Travis CI to AppVeyor BYOC

Motivation:
AppVeyor recently launched BYOC self-hosted jobs, which allows us to
utilize our own machines for building projects. It is highly
cost-effective because we can overcome the concurrency/performance limit
enforced by AppVeyor or Travis CI.
Modifications:

Switch from Travis CI to AppVeyor BYOC for building on Linux.
Ignore ArmeriaGrpcServerInteropTest.deadlineExceeded() temporarily
on Windows.

Result:

Faster build

Successfully set up 5 dockerized build agents with 8 vCPUs, 16 GB RAM and 100GB SDD. Currently configured to run up to 20 concurrent jobs, i.e. 20 x 5 = 100 concurrent jobs.

Seems to work fine now, except that the cache is missing. I could not set it up because somehow it keeps saying there's no such access key ID. If we fail to set up the cache, maybe we should look for something else.

BYOC with transparent caching seems hard as a concept since cross-cloud transfers would be slow anyways.
Can we manually add steps to upload/download to verda object store?

For reference, maybe something like this
https://github.com/curioswitch/curiostack/blob/master/cloudbuild.yaml#L37

Can we manually add steps to upload/download to verda object store?

Verda provides S3-compatible object storage but it just doesn't work for some reason. Let me contact the Verda admins early next week just in case I misconfigured anything. If it doesn't work out well, we could run an S3-compatible server like https://github.com/minio/minio

Cache is now working. I just used a local volume and rsync for speed and simplicity. Let me know what you think.

This is now ready for reviews.

Thanks for reviewing!


Supporting dashboards links and search

first, thank you for the great panel. We have been searching for a replacement for the regular built-in table panel that can show multiple metrics within the same panel.
There is 2 things that holding us from using this panel. So we were wondering to suggest those as enhancements be added at some point.

Having the ability to link to different dashboards

the ability to search inside the panel content with just like the screenshot below (attached screenshot is from the datatable  panel)

Appreciate the help.

Great suggestion for the links. Having a cell clickable for drill-down would make this plugin even more flexible. If links are implemented, please aim to be compatible with dashboard variables.

Hi @JosephFY Thanks for the suggestion.
Suggestion 1 : It is pending from my side for the long time. I will consider this as my next priority. It is one of the big feature in my list ("Clickable cells"). But expect only by mid of October as i will on vacation for next few weeks.
Suggestion 2 : We don't require the search feature right now, as this is only applicable if pagination is supported. I will consider this option if pagination is implemented in future.
Feel free to send pull requests, if possible.

@JosephFY @xkilian - Clickable links are available in master now
https://github.com/yesoreyeram/yesoreyeram-boomtable-panel#table-cell-links

Thank you very much. This is awesome


Anima Stone not doubling golem buff

I added an Anima Stone to my tree. I currently have 1 stone golem and wanted the Anima stone for a second golem to get the extra life regen on my blood magic character. I put the Anima stone in my passive tree and the Life Regen stat didn't change.
Here's a pastebin of the build: https://pastebin.com/5zLwysJT

Apparently stone golem regen doesn't stack.


improve CONTRIBUTING.md

Instead of just adding make the changes, you can add few points on how to do it exactly like how to make changes, make a PR in a detailed way and add it to README.md in the last where you wrote contribute here

Thank you for opening this issue @starlightknown, I'll edit that part.


Blur Icons

Description

Steps To Reproduce
None
Requirements
No blur
Files Version
<IP_ADDRESS>
Windows Version
10.0.22621.2215
Log File
debug.log

Thank you for your feedback, we're tracking a similar issue in #10067. You can click the subscribe button in that issue to be notified when the issue is resolved.


Fix langage error

Fix error for non-russian/english player.
Let en as the default langage and don't try to load their langage

Closed due to better fix

Your "better fix" doesn't work ... Just use mine it's 2 lines ...


Fix bug where unmark, mark doesnt save attributes

There is a bug where when mark and unmark is called, the newly task does not store the custom attributes in the previous tasks

Codecov Report

Merging #147 (11e6d48) into master (97cd714) will decrease coverage by 0.06%.
The diff coverage is 23.07%.

@@             Coverage Diff              @@
##             master     #147      +/-   ##
============================================
- Coverage     31.40%   31.34%   -0.07%
  Complexity      445      445
============================================
  Files           171      171
  Lines          3206     3213       +7
  Branches        375      377       +2
============================================
  Hits           1007     1007
- Misses         2100     2107       +7
  Partials         99       99

Impacted Files
Coverage Δ

.../address/logic/commands/tasks/MarkTaskCommand.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/model/task/Task.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
56.34% <50.00%> (ø)

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


Update Cache-Status RFC URL

Just that.

Thank you @szepeviktor for your contribution! 🚀

You're welcome.


Nested And / Or Statements Fail

Describe the bug
When creating nested AND / OR statements for WHERE, the search fails.
Example
Works, but does not match my desired query:
[
    ['account_id', '=', this.account.id],
    'AND',
    ['name', 'LIKE', `%${this.searchQuery}%`]
]

Does not work:
[
    ['account_id', '=', this.account.id],
    'AND',
    [
        ['name', 'LIKE', `%${this.searchQuery}%`]
    ]
]

Does not work, but matches the final query desired:
[
    ['account_id', '=', this.account.id],
    'AND',
    [
  	['name', 'LIKE', `%${this.searchQuery}%`],
  	'OR',
  	['vin', 'LIKE', `%${this.searchQuery}%`]
    ]
]

Expected behavior
I would expect nested operators to correctly resolve themselves. For example, I would expect the final code example above to resolve to a similar SQL statement as:
SELECT * FROM Vehicles WHERE account_id = 1 AND (name LIKE "%chevy%" OR vin LIKE "%chevy%")

Could you provide your data models so I can test this?

Thanks for helping debug this!
Here's a simplified version of the data model, but it includes the relevant fields:
nSQL('Vehicles')
.model([
    { key: 'id',              type: 'int',     props: ['pk']   },
    { key: 'account_id',      type: 'int',     props: ['idx']  },
    { key: 'name',            type: 'string' 		       },
    { key: 'license_plate',   type: 'string'                   },
    { key: 'vin',             type: 'string'                   }
]);

I don't assume there to be anything out of the ordinary with this model, and please let me know if I can provide any further details.

@ClickSimply Any updates on this? This is a blocker currently and I'm facing removing the library and starting over with another framework.

I've narrowed the problem down to an issue with the secondary indexes, should be fixed in the next day or so.

This should be resolved as of v1.7.7, just pushed it to NPM

Thanks! I can confirm this fixes the issue.

Actually - When not using a nested where like: ['account_id', '=', 18], I'm now seeing duplicate results returned. I've verified that duplicates do not exist in the database.
Duplicate results are however fixed when making the filter a nested array
[
   ['account_id', '=', 18]
]

I'm having trouble recreating the new bug as described, here's a codepen. https://codepen.io/clicksimply/pen/xyrQOK

Hey Matt, just curious if this is still an issue or if it can be closed.

I'll have to double check my local code, then, but it does indeed appear to be working for you. Happy to close and review my local codebase again.

@ClickSimply I've been able to identify this as an issue specifically with the cordova-plugin-nano-sqlite. It works as expected in the browser (as demonstrated by your codepen example above), however when running on an actual device or iOS simulator the issue can be reproduced. Yesterday, for example, I had several unexpected issues on devices that were all solved by wrapping the where() with extra [].
Does not work:
[
  ['inspection_form_id', '=', form.id],
  'AND',
  ['vehicle_id', '=', form.schedule.vehicleId]
]

Does work:
[
  [
    ['inspection_form_id', '=', form.id],
    'AND',
    ['vehicle_id', '=', form.schedule.vehicleId]
  ]
]

This should be resolved as of NanoSQL v1.8.0 and Cordova/SQLite3 adapters v1.6.0.  You may need to export your databases and reimport them, the issue was due to a type conflict with the primary key of the SQLite database.

I'm using the 2.1.9 version and something very similar is happening
So this query returns nothing:
[[['userId', '=', 'user-10'], 'AND', ['cardId', '=', '1']], 'AND', ['status', '=', 'promised']]
but this one returns expected rows
[['userId', '=', 'user-10'], 'AND', ['cardId', '=', '1'], 'AND', ['status', '=', 'promised']]

2.2.4 was just released, should be good to go now!


Bugfix 3.5: Calculate Write Amplification / Flag to disable StatisticsWorker

This PR adds two independent changes, and a related Agency change:

rocksdb Write Amplification is now calculated and reported with existing rocksdb statistics. This is essential for judging impact of compactions (and changes to compaction options).

StatisticsWorker has demonstrated a tendency to cause increased compactions which heavily impact throughput. The new --server.statistics-history option allows just the StatisticsWorker portion of statistics to be disabled.

statistics are now active for Agency. Previously AgencyFeature.cpp completely disabled statistics for an agent daemon. The disabling code is removed. StatisticsFeature.cpp now notices if the daemon is for an agent and automatically disables statistics-history to avoid current performance issues.

Jenkins running:  http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/6123/

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/6276/

http://jenkins01.arangodb.biz:8080/job/arangodb-matrix-pr-linux/7959/EDITION=community,STORAGE_ENGINE=mmfiles,TEST_SUITE=cluster,limit=linux&&test/artifact/testfailures.txt - seems to be sporadic failure. Not reproduced locally.


Customize material for change default texture (flashes white blicking)

Hey, everybody. I'm creating an Entity Plane and I'm adding
ImageMaterialProperty. When I update the position, the white background flashes. How can I access the original texture and change it?
cesium.viewer.entities.add({   name: "Red plane with black outline",   position: new Cesium.Cartesian3.fromDegrees(-107.0, 40.0, 300000.0),   plane: {     plane: new Cesium.Plane(Cesium.Cartesian3.UNIT_Z, 0.0),     material: new Cesium.ImageMaterialProperty({       image: './logo512.png',       transparent: true,     }),     dimensions: new Cesium.Cartesian2(60000.0, 61000.0)   } });

Hi @Meybiz , this is a great question for the forum.
https://community.cesium.com/c/cesium-for-unreal/10
Can you ask your question there? (if you search, it may have already been answered)
Thanks!

@Meybiz that looks like a CesiumJS question, so please post in the CesiumJS category instead. Thank you!


chore/person lists

Remove references to user-proxy Person
Remove Person user-proxy model
Migration for new non-proxy Person model
Add data migration to create new non-proxy Person records
Migrations to change Membership User to Person
Register Person as snippet and make searchable
Update exodus script to populate person images
Preliminary wagtail admin for new Person model
Update membership migrations to re-set Person to non-nullable field
Create User->Person migrations for Positions
Warn and skip missing images in exodus script
Move wagtail admin out of admin.py
Configure wagtail admin for People and Titles
Remove unused import
Support inline editing of titles on people
Convert UserResource to PersonResource
Fix ImportError in pages models
Rework initial pages migration
Create special People LandingPage subtype
Create ProfilePage type and move fields from Profile
Configure ProfilePage in wagtail admin
Exodize profiles to ProfilePages
Update and test init_person_from_ldap
Make exodus script work on fixtures with no homepage
Add global wagtail admin css hook
Add ProfilePage template
Enable ProfilePage titles and exodize them
Set allowed tags on ProfilePage education section
Define default ordering for People
Update admin for People and ProfilePages
Move people model tests to new file; remove fixtures
Move PeopleLandingPage to People app
Rename wagtail page tests to test_pages.py
Add wagtail page tests for People app
Skip failing tests until apps are exodized
Fix ProfilePage Person delete handler
Fix failing test for Person.current_title
Update People page tests
Use 'load static' to suppress django warning
Add and test recent blog post display on ProfilePages
Add template tests for ProfilePage
Remove profile view from urlconf
Convert ProfileQuerySet to PersonQuerySet
Add and test PersonListPage
Update and test default PersonListPage
Separate out PersonListPage methods more
Update archive nav on PersonListPages
Add all PersonListPage subtypes
Update exodus script to create new PersonListPages
Skip failing pages tests with todo notes
Remove unused people admin
Add convenience functions for person profile url and card image
Remove unused people urls
Remove unused people views
Update generic person card
Stub templates for each PersonListPage subtype
Delete old Profile template
Port old person list view tests to template tests

Codecov Report

Merging #269 (e847574) into develop (8d7c85b) will decrease coverage by 14.47%.
The diff coverage is 78.32%.

@@             Coverage Diff              @@
##           develop     #269       +/-   ##
============================================
- Coverage    97.68%   83.20%   -14.48%
============================================
  Files           99      126       +27
  Lines         3150     3644      +494
============================================
- Hits          3077     3032       -45
- Misses          73      612      +539

Codecov Report

Merging #269 (e847574) into develop (8d7c85b) will decrease coverage by 14.47%.
The diff coverage is 78.32%.

@@             Coverage Diff              @@
##           develop     #269       +/-   ##
============================================
- Coverage    97.68%   83.20%   -14.48%
============================================
  Files           99      126       +27
  Lines         3150     3644      +494
============================================
- Hits          3077     3032       -45
- Misses          73      612      +539


Cannot remove alias

In Windows PowerShell, the correct way to remove an alias is
> rm alias:ls

In Pash it doesn't work:
PASH /home/fornever> rm alias:ls
The requested feature is not implemented.
  +CategoryInfo: NotSpecified, Reason: NotImplementedException
  +FullyQualifiedErrorId: NotImplementedException,System.Management.Automation

This is because some functionality is not yet implemented in the SessionStateProvider. So this basically affects all SessionState related itemes as also variables.


Unskip .NET tests for OperationName and W3C

Description
v2.42.0 is released, but NuGet has been degraded and the package we use for the parametric tests hasn't been listed yet on NuGet.
Unskips the OpenTelemetry and W3C parametric tests for .NET, specifically to support the new operation name mapping and W3C propagation introduced in v2.42.0 of the .NET Tracer.
Motivation
The tests were skipped before due to yet being supported by the .NET Tracer.
Workflow

⚠⚠ Create your PR as draft
Follow the style guidelines of this project (See how to easily lint the code)
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Once your PR is reviewed, you can merge it! :heart:
Reviewer checklist

[ ] Check what scenarios are modified. If needed, add the relevant label (run-parametric-scenario, run-profiling-scenario...). If this PR modifies any system-tests internal, then add the run-all-scenarios label (more info).
[ ] CI is green

[ ] If not, failing jobs are not related to this change (and you are 100% sure about this statement)

[ ] if any of build-some-image label is present

is the image labl have been updated ?
just before merging, locally build and push the image to hub.docker.com

[ ] if a scenario is added (or removed), add (or remove) it in system-test-dasboard nightly

Hi @bouwkast , do you still plan to work on this ?

Hi @bouwkast , do you still plan to work on this ?

Looks like it has been done in a different PR, closing.


Pundit::AuthorizationNotPerformedError on ActiveAdmin

So, here is my scenario:
I've a ActiveAdmin class, any change[1] I do on this file and try reload the page I receive this error and need reset my app:
Pundit::AuthorizationNotPerformedError
What's going on here? Should I do something to avoid this error or it's the regular behavior?
[1] Just add a block like this:
  action_item only: :index do
    link_to 'Example', admin_example_path
  end

Pundit::AuthorizationNotPerformedError means that you do not have a policy being called and your rails app is configured to ensure one is being called for each controller action.
Here is the doc section for an "ordinary rails" app that would apply: https://github.com/elabs/pundit#ensuring-policies-and-scopes-are-used
Also, here's the suggested ApplicationPolicy from ActiveAdmin for use with Pundit:
https://github.com/activeadmin/activeadmin/blob/master/spec/support/templates/policies/application_policy.rb
That's the most likely resolution to your issue...

I already have this application policy in my project.
The strange part of this Pundit::AuthorizationNotPerformedError error is:

start rails app, access a page(eg: /admin/users), authorization works fine;
change the controller, eg: create a new action; add a policy for this new action; access an old action((eg: /admin/users); app raises: Pundit::AuthorizationNotPerformedError

There is nothing in Pundit that would do this. ActiveAdmin is crazy magic, I can't imagine the multitude of ways it could break things.


Test Issue

Just a test Github Issue to check if all integrations are working 🕵♂
import this

Looks like it goes!

Done 💚


Upload targeting policy

@microsoft-github-policy-service agree

@yifeike  - Please upload to the correct file path: teams/teams-ps/teams


labelle: 1.1.0 --> 1.2.0

Description of changes
Bump version from 1.1.0 to 1.2.0.
Changelog: https://github.com/labelle-org/labelle/releases/tag/v1.2.0
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

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

2 packages built:

labelle
labelle.dist


spot-perpetual arbitrage strategy: It does not check that the order size exceeds the minimum tradable in each leg before executing any of them.

Feature Suggestion
// A succinct description of how you want to achieve the desired behavior.
// If you have looked at the code and know exactly what code changes are needed then please consider submitting a pull request instead.
Impact
// A succinct description of why you want the desired behavior specified above.
The desired behavior will allow me to ...
Additional context
Add any other context or screenshots about the feature request here.

In the spot-perp arbitage strategy there is no prior check when sending the orders as to whether they will be able to execute both.
My experience was that the bot executed the future and then sent an error message that it could not execute the spot because the order_amount was less than the minimum allowed to trade by the exchange.
Then, I was left with the future traded and the spot not.


Change codeowner for index.json

If this PR is for a new extension or change to an existing extension, use the following to try out the changes in this PR:
docker run -it microsoft/azure-cli:latest
export EXT=<NAME>
pip install --upgrade --target ~/.azure/cliextensions/$EXT "git+https://github.com/derekbekoe/azure-cli-extensions.git@code-owner#subdirectory=src/$EXT&egg=$EXT"

@williexu @tjprescott This is so you get notifications when someone changes the file


JEID: Machine Preview is all *ite forms of stone

Compact Machines version: compactmachines3-1.12.2-3.0.18-b278
Forge version: forge-<IP_ADDRESS>38
I am not using Optifine
https://media.discordapp.net/attachments/344165687603757056/612728266502963270/unknown.png?width=1052&height=702
https://media.discordapp.net/attachments/344165687603757056/612730027498405901/unknown.png
I expect to see the inside of the machine, instead i see the *ite forms of stone
Steps to reproduce:
Install JEID&Compact machines
Install other mods
Place modded blocks in compact machine
View compact machine via GUI

Duplicate of #357