Add a greedy tree search option

Policy learning with exact search is only feasible on "medium" sized data. For users who want to use this package on datasets of arbitrary size we should offer a greedy option.
We should also consider automatically dispatching to this greedy solver when |X| passes some threshold, such that first-time users don't unintentionally hit an estimated runtime of 100 hrs+.

What do you @kanodiaayush and @swager think of the following API  tree_search(X, Y, solver = c("auto", "exact", "greedy") where auto means we dispatch to greedy search when the input dimensions would lead to a problem that takes days to compute (can determine exact threshold based some experiments)
With some flexibility I guess we can consider this a "backwards" compatible release (i.e v2.0 not necessary) as we do not expect "many" users to have run policy_tree with default options on data that takes days to compute...

I think that we'd like to give the user some more control, but help guide them through how to use it with some further documentation and specific examples.  A fully greedy algorithm would be nice because in order to apply, say, rpart to policy learning, there are several steps and people find it hard to code/easy to make a mistake.  However, there would be even more value added to have something between fully optimal and fully greedy, e.g. something that looks one step down the tree (and thus considers interactions among variables) but still runs in reasonable time.  In terms of coding, it would be some work for the user to implement a hybrid algorithm like this, so there would be value from using it.  I am worried that the "auto" doesn't really solve the user's problem, it may be too black box.  An alternative to "auto" would be a helper function that returns some info to the user, combined with documentation that shows examples of what to do in different scenarios.

Ideally a user should  be ready for most scenarios by reading the through a succinct @example docstring (some improvements in #95). For whichever solver alternative everyone decides on, fully greedy or hybrid, the intention was that policy_tree would elicit a useful warning when "auto" switched from the default ("exact") to the alternative (sorry, this wasn't clear above). The @example docstring would have explained both of these to the diligent user.
Regarding the hybrid solver, I think it would be beneficial to know more about which (|n|,p,k) configurations you/users that contacted you, expect for this to be beneficial/feasible? If someone expects this to allow them to throw in say 500 covariates with depth 6: am not sure how that would play out...

I think the docstring and helper functions support the same goal but not sure which is better aesthetically.
WRT what is a good intermediate algorithm. Hmmmmm..... I am not sure I can think up one immediately.
I have ideas but this might need a literature review. @erikcs do you have any intermediate solutions in mind? Also what specific algorithm do you mean by greedy?

we already have a 'look every so often' option - in my understanding scaling that is going to be the best as it scales smoothly and feels like it will have smooth performance degradation.
Also to Erik's point, would be very useful to understand the specific situation that we are trying to fix.

Basic problem trying to fix--users give up because they cannot compute a depth 3 or 4 tree.  Use cases encountered in the lab include tens of thousands to millions of observations, but weak treatment effect heterogeneity, with dozens of covariates.  We might coach people to use fewer covariates, but reducing the sample size is not a great idea with weak signal. They may need depth of 3-5 in order to allow interactions among covariates.  There should be no problem doing trees of depth up to 6 with a hybrid model.  The algorithm I suggested is that you build a depth-2 policy tree.  Then, prune the level 2 splits, and build two more depth-2 policy trees off of the first split.  Keep going.    You can try another version where you substitute 3 for 2.

Perfect. I independently came up with the exact same algorithm in my head too! this also does allow for smooth scaling exactly as in by subbing 3 for 2.
Thanks!
@erikcs lets talk about how to do this

@susanathey have you seen this idea in some paper which has theoretical performance analysis of this vis a vis the full algorithm since you described it? Perhaps an equivalent for Classification or Regression Trees?

CART is fully greedy (doesn't look ahead to future splits) so the hybrid approach is a direct improvement over it

Sounds good! Just to set expectations on runtime here, say we reduce the "deep" L depth tree to several globally optimal k-depth trees (k < L): k=2 will already hit a long runtime way before a milllion observations, we have a short point of reference table here:
https://github.com/grf-labs/policytree/issues/46#issuecomment-686658293
but, combined with careful split.step use and rounding/relabeling it can maybe scale, that's an empirical question!

got it @susanathey just to be clear, in extension to depth 3, the method i have in mind is you always prune except at the root split, but the tree depth you go to to create the root split is variable. another method might be to prune the lower most level only.

I would say a routine should "globally" search to level k and "prune" to level k'<k, these can each be user-specified parameters.

With appropriate split.step or rounding, there is no reason that having millions of observations needs to be problem.  If # observations was a problem even after the other optimizations, we could additionally sample.  For example couldn't we downsample but then draw in more observations if considered leaves don't have enough data.  There are simpler options too; we could use a smaller dataset to narrow down to a leading set of trees or to optimize with a more coarse grid, and then use more data to fine-tune things.

noted; how should we aim to do this? it feels like this is sufficient to think of in terms of an improvement paper? Or is this just code optimization?
Regards,

I think this can be code optimization for now--I think Erik should be able to implement it pretty quickly

Ayush I can come over to your place one day and we can have a go at it.

Yes that sounds good!

An R implementation of hybrid_policy_tree is here: https://github.com/grf-labs/policytree/pull/118
@kanodiaayush also started a C++ version in https://github.com/grf-labs/policytree/tree/hybrid_search, I am fine with the R version for now as it makes zero changes to the existing package.

The approach in #118 is simple to define and implement, and it does not involve pruning. Compared with the two other ways of building a depth 3 tree using depth 2 exact search, which both involves "pruning", it does better:
Alternative 1: on avg. #118 gives ~ 30% higher reward
Alternative 2: on avg. #118 gives ~ 70% higher reward


error with cmake (using eigen from gtsam)

Hi, I run into error when doing cmake in pcl with eigen from gtsam(Ubuntu 14.04 & Indigo):
CMake Error at cmake/Modules/FindEigen.cmake:29 (file):
file Internal CMake error when trying to open file:
/home/bona/software/gtsam-3.1.0/gtsam/3rdparty/Eigen/Eigen/src/Core/util/Macros.h
and "cmake -DEIGEN_INCLUDE_DIR="~/software/gtsam-3.1.0/gtsam/3rdparty/Eigen" .." failed.
Any suggestion?
Thanks.

What stage of the installation process as described by the wiki page was this?
https://github.com/CognitiveRobotics/omnimapper/wiki/Installation-&-Dependencies
Are you specifying omni mapper to build using the same 3rdparty/Eigen folder gtsam ships with?

It was "Build OmniMapper Base", however same error also when compiling pcl. I did pcl with PPA though.

I remember gtsam and package installed pcl having issues with each other, as gtsam ships with a patched version of Eigen that has not been ported upstream in Eigen (last I checked, but that was a year ago). Take a look at the last revision of the Dockerfiles (the COPY lines), and see how I patched the original headers in PCL to avoid compiling it from scratch:
https://github.com/CognitiveRobotics/omnimapper/blob/6d9b6d7404affc3ac07ac4281a3957085305fdc9/docker/demo/Dockerfile
I'm not sure if this still works, but I plan on revisiting the Dockerfiles in a week or so to update them to use the official ros images. Let me know how it goes.

COPY HouseholderQR.h /usr/include/eigen3/Eigen/src/QR/HouseholderQR.h
COPY CommaInitializer.h /usr/include/eigen3/Eigen/src/CommaInitializer.h
Did you mean these lines? Where do I get the packed headers from?
2016-04-29 13:11 GMT-04:00 Ruffin<EMAIL_ADDRESS>
I remember gtsam and package installed pcl having issues with each other,
as gtsam ships with a patched version of Eigen that has not been ported
upstream in Eigen (last I checked, but that was a year ago). Take a look at
the last revision of the Dockerfiles (the COPY lines), and see how I
patched the original headers in PCL to avoid compiling it from scratch:
https://github.com/CognitiveRobotics/omnimapper/blob/6d9b6d7404affc3ac07ac4281a3957085305fdc9/docker/demo/Dockerfile
I'm not sure if this still works, but I plan on revisiting the Dockerfiles
in a week or so to update them to use the official ros images. Let me know
how it goes.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/CognitiveRobotics/omnimapper/issues/17#issuecomment-215817412

Actually, I think the error might be these lines in FindEigen.cmake:
if(EIGEN_INCLUDE_DIR)
file(READ "${EIGEN_INCLUDE_DIR}/Eigen/src/Core/util/Macros.h"
_eigen_version_header)
string(REGEX MATCH "define[ \t]+EIGEN_WORLD_VERSION[ \t]+([0-9]+)"
_eigen_world_version_match "${_eigen_version_header}")
set(EIGEN_WORLD_VERSION "${CMAKE_MATCH_1}")
string(REGEX MATCH "define[ \t]+EIGEN_MAJOR_VERSION[ \t]+([0-9]+)"
_eigen_major_version_match "${_eigen_version_header}")
set(EIGEN_MAJOR_VERSION "${CMAKE_MATCH_1}")
string(REGEX MATCH "define[ \t]+EIGEN_MINOR_VERSION[ \t]+([0-9]+)"
_eigen_minor_version_match "${_eigen_version_header}")
set(EIGEN_MINOR_VERSION "${CMAKE_MATCH_1}")
set(EIGEN_VERSION
${EIGEN_WORLD_VERSION}.${EIGEN_MAJOR_VERSION}.${EIGEN_MINOR_VERSION})
endif(EIGEN_INCLUDE_DIR)
If I simply comment these lines, cmake .. will work, however compile failed
as missing eigen dependency.
Could you confirm that the lines above is correct while configuring make
files?
Thanks!
2016-04-29 13:28 GMT-04:00 shi bai<EMAIL_ADDRESS>
COPY HouseholderQR.h /usr/include/eigen3/Eigen/src/QR/HouseholderQR.h
COPY CommaInitializer.h /usr/include/eigen3/Eigen/src/CommaInitializer.h
Did you mean these lines? Where do I get the packed headers from?
2016-04-29 13:11 GMT-04:00 Ruffin<EMAIL_ADDRESS>
I remember gtsam and package installed pcl having issues with each other,
as gtsam ships with a patched version of Eigen that has not been ported
upstream in Eigen (last I checked, but that was a year ago). Take a look at
the last revision of the Dockerfiles (the COPY lines), and see how I
patched the original headers in PCL to avoid compiling it from scratch:
https://github.com/CognitiveRobotics/omnimapper/blob/6d9b6d7404affc3ac07ac4281a3957085305fdc9/docker/demo/Dockerfile
I'm not sure if this still works, but I plan on revisiting the
Dockerfiles in a week or so to update them to use the official ros images.
Let me know how it goes.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/CognitiveRobotics/omnimapper/issues/17#issuecomment-215817412

The patched headers are in the same folder for the docker build context:
https://github.com/CognitiveRobotics/omnimapper/tree/6d9b6d7404affc3ac07ac4281a3957085305fdc9/docker/demo
If you use say cmake-qt-gui or cmake-gui to configure cmake, do you see the eigen path properly pointed to that of gtsam's in it 3rdparty folder?

yes, I think I have the correct path for eigen, please find the details in
attachment:

As I just skipped pcl and jumped to omnimapper, it can pass cmake ..(with
the 3rd party eigen), however failed when make.
Error was :
In file included from /usr/include/pcl-1.7/pcl/common/common.h:41:0,
from
/home/bona/software/omnimapper/base/src/transform_tools.cpp:2:
/usr/include/pcl-1.7/pcl/pcl_base.h:49:27: fatal error: Eigen/StdVector: No
such file or directory
#include <Eigen/StdVector>
Not sure whether it is something wrong with Eigen path.
2016-04-29 16:25 GMT-04:00 Ruffin<EMAIL_ADDRESS>
The patched headers are in the same folder for the docker build context:
https://github.com/CognitiveRobotics/omnimapper/tree/6d9b6d7404affc3ac07ac4281a3957085305fdc9/docker/demo
If you use say cmake-qt-gui or cmake-gui to configure cmake, do you see
the eigen path properly pointed to that of gtsam's in it 3rdparty folder?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/CognitiveRobotics/omnimapper/issues/17#issuecomment-215871491

I think you're using email reply to this issue, although github issue threads do use attachment, perhaps use something like pastebin?
I'll have to check if i can reproduce this with the current debian packages for trusty when I refactor the dockerfiles.

Oops, sorry for missing the attachment, yes I did reply to email. Please find it here, I hope this time it will work. Sorry I am not familiar with pastebin, so I put it onto google drive.
Thanks!

Any workaround to this issue? I can't run any PCL code due to this error


blocklauncher still crashing - doesn't work

Accer iconia a1-840FHD crashing
Acer liquid works
Both use Intel so repair it.

And they're running the same Minecraft PE APK and BlockLauncher version?

I think no but i try it. And in version 0.10.5 you said "blocklauncher doesn't support Intel" but on acer liquid bl worked perfect! And on my tab a1-840FHD mods didn't work after firmware upgrade (added only split screen)

I don't have an Intel device myself, so I really don't know what's going on here. I know that there are two versions of MCPE that will run on an Intel device: the ARM version (for regular phones), and a special version of MCPE optimized for Intel Atom devices. If you downloaded MCPE from Google Play on an Intel device, you're getting the Intel optimized version; otherwise, you're probably getting the ARM version.

THX a lot i have arm version :D so lets go to buy MCPE! and when i'll have Intel version of MCPE so bl won't work?

No, neither works properly right now - I've been mostly looking into compatibility with the Intel version though. Which version is installed on the Liquid?

ARM


Add cosign

Tool name
cosign
Tool license
Apache License 2.0
Add or update?

[X] Add
[ ] Update

Desired version
1.4.1
Approximate size
80MB
Brief description of tool
Container Signing, Verification and Storage in an OCI registry.
It is promoted in a GitHub blog post^1 and in starter workflows^2
URL for tool's homepage
https://github.com/sigstore/cosign
Provide a basic test case to validate the tool's functionality.
No response
Virtual environments affected

[X] Ubuntu 18.04
[X] Ubuntu 20.04
[X] macOS 10.15
[X] macOS 11
[X] Windows Server 2016
[X] Windows Server 2019
[X] Windows Server 2022

Can this tool be installed during the build?
See https://github.com/sigstore/cosign-installer

Tool installation time in runtime
a couple of seconds
Are you willing to submit a PR?
yes

Hi @JamieMagee!
The started workflow example executes install action and this is a recommended way to use consign as you can set up any version you like or stay on the latest. The installation takes not more than 2-3 seconds:
jobs:
  build:
    runs-on: ubuntu-latest

    name: Install Cosign and test presence in path
    steps:
      - name: Install Cosign
        uses: sigstore/cosign-installer@main
        with:
          cosign-release: 'v1.4.1'
      - name: Check install!
        run: cosign version

However, the action doesn't support other OSs at the moment and we recommend creating an issue in the https://github.com/sigstore/cosign-installer repository to add the support.
Until that you can use the following snippets to install consign in runtime, it won't take more than 5 seconds for macOS and 20 for windows:
macOS
csDownloadUrl=$(curl -s https://api.github.com/repos/sigstore/cosign/releases/latest | jq -r '.assets[].browser_download_url | select(contains("darwin-amd64"))' | head -n 1)
curl -L -o /tmp/cosign "$csDownloadUrl"
sudo xattr -rd com.apple.quarantine /tmp/cosign
mv /tmp/cosign /usr/local/bin/cosign
sudo chmod +x /usr/local/bin/cosign
cosign version

windows
$csBinary = Get-GitHubPackageDownloadUrl -RepoOwner "sigstore" -RepoName "cosign" -BinaryName "cosign-windows-amd64.exe" -Version "latest" -UrlFilter "*{Version}/{BinaryName}"
Start-DownloadWithRetry -Url $csBinary -DownloadPath "." -Name "cosign.exe"
./cosign.exe version

Considering fast installation time in runtime and maintenance concerns we would not like to add the tool to the image. Please use the action\snippets provided.
I'm going to close the issue, feel free to contact us if you have any concerns.
Thank you!

@miketimofeev Thanks for the quick response and detailed feedback.
My aim with adding cosign to the default image was to simplify the whole process for users to start using it. The starter action is simple, but could be simpler 😄 I totally understand the maintenance aspect, and the need to keep the VM images small as well.
Thanks again!


Implement api-key's apis for web client

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #1145
Does this PR introduce a user-facing change?:

NONE

Looks good
/approve


Add nvmrc to the project and updated developing docs

Addresses #2648
Replaces PR for #2730

Merged in 493a30b


VS Code fails to launch no macOS Big Sur 11.3.1

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:

1.57.0
b4c1bd0a9b03c749ea011b06c6d2676c8091a70c
x64

OS Version: macOS Big Sur 11.3.1

Steps to Reproduce:

work with VS Code
try launching it the next day
observe VS Code failing to launch

Diagnostics:
$ code --verbose
[main 2021-06-15T11:01:47.907Z] Error: connect EACCES /Users/michael/Library/Application Support/Code/1.57.0-main.sock
    at PipeConnectWrap.afterConnect [as oncomplete] (net.js:1146:16)
[main 2021-06-15T11:01:47.909Z] Lifecycle#kill()

$ code --verbose --disable-extensions
Warning: 'disabled-extensions' is not in the list of known options, but still passed to Electron/Chromium.
[main 2021-06-15T10:55:19.365Z] Error: connect EACCES /Users/michael/Library/Application Support/Code/1.57.0-main.sock
    at PipeConnectWrap.afterConnect [as oncomplete] (net.js:1146:16)
[main 2021-06-15T10:55:19.366Z] Lifecycle#kill()

$ ls -l "/Users/michael/Library/Application Support/Code/1.57.0-main.sock"
srwxr-xr-x  1 root  staff  0 12 Jun 21:25 /Users/michael/Library/Application Support/Code/1.57.0-main.sock

$ cat "/Users/michael/Library/Application Support/Code/1.57.0-main.sock"
cat: /Users/michael/Library/Application Support/Code/1.57.0-main.sock: Permission denied

$ sudo cat "/Users/michael/Library/Application Support/Code/1.57.0-main.sock"
Password:
cat: /Users/michael/Library/Application Support/Code/1.57.0-main.sock: Connection refused

$ ls "/Users/michael/Library/Application Support/Code"
1.56.2-main.sock           Network Persistent State
1.57.0-main.sock           Preferences
Backups                    QuotaManager
Cache                      QuotaManager-journal
CachedConfigurations       Service Worker
CachedData                 Session Storage
CachedExtensionVSIXs       TransportSecurity
CachedExtensions           User
Code Cache                 Workspaces
Cookies                    blob_storage
Cookies-journal            databases
Crashpad                   languagepacks.json
CrashpadMetrics-active.pma logs
CrashpadMetrics.pma        machineid
Dictionaries               rapid_render.json
GPUCache                   storage.json
Local Storage              webrtc_event_logs

If 1.57.0-main.sock is owned by root it means you possibly started VSCode once via sudo?


Document how to add custom protocol implementations

The CLI has --http{1,2,3}-impl options that allow to choose between protocol implementations, but in the default setup only the default implementations are available.
Document how to register custom implementation. Maybe publish httptools  or nghttp2 adapters as examples.

It is a question how to test alternative implementations. hface test suite checks all combinations of know protocols. When working on library with some alternative protocol, I should be able to run hface tests to check compatibility.


Add some docs on how-to get the VPN working

Nice addition, thanks!
r+

Stupid question, what in olympia requires access to the VPN?

Signing, for example, if you use the test signing servers.

Oh interesting, hmm.
On Tue, Sep 22, 2015 at 6:54 AM, Mathieu Agopian<EMAIL_ADDRESS>wrote:

Signing, for example, if you use the test signing servers.
—
Reply to this email directly or view it on GitHub
https://github.com/mozilla/olympia/pull/772#issuecomment-142296083.


Removing preview alert

Description
Azure Functions extension for Dapr is now GA. Removing preview label from docs
Announcement

@holopin-bot @cecilphillip Enjoy the Dapr docs badge!


Add support for supplying a path to DefaultRoute

It would be nice, as mentioned in #883, to be able to give a default route a path so that, for example, when someone hits a top level path /settings/ they would be redirected to /settings/general/ or similar. Just a piece of the code I'm working on for an example:
<Route name="settings" handler={SettingsBase}>
  <DefaultRoute name="settings_general" path="general" handler={SettingsGeneral}/>
  <Route name="settings_team" path="team" handler={SettingsTeam}/>
</Route>

Default routes match when their parent matches, they don't have a path. Maybe you're after a defaultRedirect? in which case you redirect in the default route's transitionTo.

hm, I'm not sure what you mean by the default route's transitionTo. I just want /settings/ to redirect to /settings/general/ but don't see an easy way to make this happen. I tried the above first and next tried:
<Route name="settings" handler={SettingsBase}>
  <Redirect from="/" to="general"/>
  <Route name="settings_general" path="general" handler={SettingsGeneral}/>
  <Route name="settings_team" path="team" handler={SettingsTeam}/>
</Route>

but that also does not work.

that being said, the following does work, but I'd rather the above.
<Redirect from="settings" to="settings_general"/>
<Route name="settings" handler={SettingsBase}>
  <Route name="settings_general" path="general" handler={SettingsGeneral}/>
  <Route name="settings_team" path="team" handler={SettingsTeam}/>
</Route>

+1 — would much prefer the former, as it keeps all related items encapsulated in the parent <Route />.


Use  and  for docs

Part of #15948.
Testing Plan:
GIFs or Screenshots:

I merged everything but the semantic HTML tags and the warning.  I think without the sidebar sliders, I'm not sure the two h1 elements in the /help/ left sidebar are actually helpful?  So potentially the solution is to just remove those h1s?
Beyond that, one thing that still feels problematic about the accessibility of this setup is that it's basically impossible to move focus from the left sidebar with its ~100 items on it to the right side of the page.

Are we sure that having ~100 items on the left sidebar was a good idea? It makes it hard to even see which top-level topics exist.

No, not 100% sure.  I found the slidey thing really annoying when trying to find an article on something specific that might be in a different top-level topic (and the organization wasn't perfect), but I suppose I'm not a typical user.
I wonder if there's a better solution to both of these problems (well, we could also add search).

Want to post in #general for broader feedback on that question?

We’ve always had a non-sliding list of all articles on the /help/ home page.

Because the source repository for this pull request was deleted, this pull request has been automatically closed by GitHub as an unintended side effect of renaming our default Git branch. If this change is still desired, please open a new pull request, referencing this one in the description.


No matching version found for<EMAIL_ADDRESS>

Bug description
I am trying to install prisma migrate (npm i @prisma/migrate) since last night with no success. I keep getting the below messages:
npm ERR! code ETARGET
npm ERR! notarget No matching version found for<EMAIL_ADDRESS>npm ERR! notarget In most cases you or one of your dependencies are requesting
npm ERR! notarget a package version that doesn't exist.
npm ERR! notarget
npm ERR! notarget It was specified as a dependency of '@prisma/migrate'
npm ERR! notarget

Prisma 2.20.1
Windows 10
I think this might be related to https://github.com/prisma/prisma/issues/6422#issue-850779191

Thanks for reporting @martineboh ! We just released a<EMAIL_ADDRESS>with the correct dependency versions.

Oh! Thanks


Scale the Geometer grid properly

I send the scale to the grid but don't scale the graphics.

Closed by 96de0ac


Fix VPC lookup in AWS dualstack examples

What this PR does / why we need it:
Currently, the defaulting for vpc_id in the terraform example for AWS with dual stack networking is not working. Since we are passing on default as an input for the VPC ID.
Which issue(s) this PR fixes:

Fixes #
What type of PR is this?

/kind bug
Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:

Fix the defaulting for `vpc_id` in the terraform example for AWS with dual stack networking

Documentation:

NONE

/test pull-kubeone-e2e-aws-default-install-containerd-v1.25.11


Issue pairing with Brava TV (XBR850C & XBR830C)

Greetings,
I'm not sure if this is the correct venue for this, but I'm having issues pairing my TVs with the braviarc HomeAssistant implementation.
I tried using the HA configurator as directed, but after I enter 0000, the TV flashes a code momentarily and then says the pair was cancelled. HA immediately adds the TV without ever prompting for the code and then the TV shows up as a media player card, however it doesn't do anything or update status (obviously the pair didn't complete successfully.) I've deleted the bravia.conf in the hass root directory and tried to repair multiple times with the same result.
Also, every time I reboot HA, the TVs display the pairing request dialog from HA, but HA never gives me the option to enter a code.
I have two Bravia XBRs, 850C & 830C, same result with both.
Any advice?

Let me figure out what's wrong...

I've hit the same issue and found that if I wrote the PIN into bravia.conf manually and restarted HA, it worked. I then found out the power commands don't seem to work on the 850C (only Android-based one I have). But that's something for another issue, which I logged here: https://github.com/home-assistant/home-assistant/issues/2599

I thinks this issue is fixed by #4


adding UI for field files / links in edit form

Fixes #1244 .

Changes proposed in this PR:

validation 'sample links' in edit form

placeholder 'file url' during edit

WIP:

[ ] on some assets, it doesn't get picked up

found one issue, if the asset you want to edit already has sample files, the submit button is always disabled even if you update the sample field or any of them, on the other side the submit button is always enabled on assets without sample files set, even if you have even touched or updated any form input fields.
https://user-images.githubusercontent.com/13447142/195508812-78725c38-68b8-4476-8ffd-dccd22db4ecb.mov

found one issue, if the asset you want to edit already has sample files, the submit button is always disabled even if you update the sample field or any of them, on the other side the submit button is always enabled on assets without sample files set, even if you have even touched or updated any form input fields.
Screen.Recording.2022-10-13.at.08.13.21.mov

oh boy 😅 getting that validation to work was challenging 😂 please let me know if it needs more changes 👍🏽

i updated this https://market-git-feature-issue-1244-edit-ui-oceanprotocol.vercel.app/asset/did:op:50c60ce0810bf8f8b1c6ca421e5231b106cf4de784945037498c84147e11dbc5 and when i download i still get the old file

Updating just the sample file didn't work

think we can merge this, just double checked latest changes and all seems to be in place


Added support for testbench and script generation

Adds settings to JSON file
Adds completely manual testbench and scripts for reference
Adds testbench unittest
Preparing to remove baseaddroffset as concept

Resolves #24

First unit tests #31 - more to come


[Feature]: Change the email changing method for iOS

As a
User I want to have opportunity to change Email before activation
I want to
Change email right after OTP code check
So that
We should use another method for it
Preconditions
No response
Flow
No response
Postconditions
No response


IDE: Report warnings if @JvmBlockingBridge is applied to a function with @JvmSynthetic

As annotations are copied, generated blocking bridges are invisible from Java.

REDUNDANT_JVM_BLOCKING_BRIDGE_WITH_JVM_SYNTHETIC

REDUNDANT_JVM_BLOCKING_BRIDGE_WITH_JVM_SYNTHETIC


[FLINK-27746] Flink kubernetes operator docker image could not build with source release

Flink kubernetes operator docker image could not build with source release at present. The .git directory should be copied when the directory exists.

@wangyang0918 I'm thinking of COPY . . with a .dockerignore file. But yes, we can investigate better way after release 1.0.0.

I have created a new ticket FLINK-27759 to follow the potential improvement.


Add support for all fix commands to pr action

This way all fix:* commands in the package.json are recognized by the PR action, so we can also run /fix: dictname or /fix: markdown ...

/fix:all

/fix:submodules

ah... that doesn't work because the repo is checked out at a later stage


Widget Wishlist

Wishlist of widget from geoinfo,

Widget
Application
Ranking

Button
Buttons allow users to take actions, and make choices, with a single tap.
Needed

Select
Select components are used for collecting user-provided information from a list of options.
Needed

Slider
Sliders allow users to make selections from a range of values.
Needed

Table
Tables display sets of data.
Needed

Tooltip
Tooltips display informative text when users hover over, focus on, or tap an element.
Needed

Typography
Use typography to present your design and content as clearly and efficiently as possible.
Needed

Progress
Progress indicators (spinners) express an unspecified wait time or display the length of a process.
Needed

Tabs
Tabs make it easy to explore and switch between different views.
Needed

Container
The container centers your content horizontally. It's the most basic layout element.
Needed

Button Group
Group related buttons.
Nice to have

Switch
Switches toggle the state of a single setting on or off.
Nice to have

Divider
Provides a thin, unobtrusive line for grouping elements to reinforce visual hierarchy.
Nice to have

Icons
Guidance and suggestions for using icons with Material UI.
Nice to have

Alert
Alerts display brief messages for the user without interrupting their use of the app.
Nice to have

Dialog
Dialogs inform users about a task and can contain critical information or require decisions.
Nice to have

Skeleton
Displays a placeholder preview of your content before the data gets loaded.
Nice to have (lower prio)

Accordion
Lets users show and hide sections of related content on a page.
Nice to have (lower prio)

App Bar
Displays information and actions relating to the current screen.
Nice to have (lower prio)

Drawer
Navigation drawers provide ergonomic access to destinations or app functionality.
Nice to have (lower prio)

Checkbox
Allows the user to select one or more items from a set.
Nice to have (might be covered by Button)

We already have Button, Select, Typography, Checkbox, and Box.
And of course Plot, which holds a Vega-Lite Chart.
For ease of use I suggest we make Icon and Tooltip properties of other components, e.g., btn = Button(icon="update", tooltip="Update the chart").


add reload when comming back from session detail

Issue

#232

Overview (Required)

Search result scree  can  reload when coming back from session detail.

Links

https://github.com/DroidKaigi/conference-app-2017/pull/264

Screenshot

Before
After

I notice session title is overlapping with speaker image.

:eyes:

LGTM!! 💯

Thanks for contribution!!

Thank you for reviewing!


Expose ERROR log level to custom libraries

Here's the supported log levels in the logging api for robot: TRACE, DEBUG, INFO and WARN.
While there is no rule, it is a common convention to distinguish between a event that indicates something unexpected happened that could indicate a problem (WARNING) versus an event that is a serious problem that resulted in the failure of an operation (ERROR). For example, the official python documentation about the logging module and log4j make this distinction.
I believe that most engineers share this understanding and would treat events logged at ERROR differently then at warning. Consequently, I think it would be a good feature to support the ability to log at the ERROR log level in both test cases and custom libraries. The purpose would be to indicate that a serious problem has occurred that resulted in a failure.

The main reason ERROR level isn't officially exposed to custom keywords is that it could be confusing that a keyword that causes and error nevertheless passes. Do you think that is a problem? Or should an error automatically also fail a keyword?

IMHO, whether a custom keyword passes or not should be determined by the implementation and context. My view is that ERROR indicates something has failed within the function, whether that means the keyword should fail or not would be context dependent.

I agree that error log messages should not automatically cause a keyword to fail. It's easy to raise an exception if/when failure appropriate.
I guess it's fine to officially expose the error level to custom libraries. I can see some valid use cases for it, and don't think the possibility that libraries can log too many errors is a good reason to prevent them altogether. I thus tentatively assigned this to RF 2.9 scope, but actually getting it in requires a bit changes than the current PR contains:

If a keyword logs something with the error level, the message on test execution errors in log file should automatically be linked to that keyword. We already have this functionality in place for warnings, and it ought to be easy to use it also with errors.

Should map messages originating from Python's logging module with level logging.ERROR or higher to the new error level. Nowadays anything above logging.WARNING is considered a warning.

Several acceptance tests are needed (some may also require more changes to code):

Libraries using robot.api.logger.error().
Libraries using logging.error() or above.
Libraries using print '*ERROR* Message.
Using ERROR level with Log keyword.
Linking errors in execution errors to keywords causing them.

Error level needs to be documented in the User Guide.

Are you @jaloren interested to take care of the above? You should be able to get started by investigating how WARN level is handled/tested/documented currently, but I can also answer questions here or we can have a chat on #robotframework IRC channel.

@pekkaklarck sorry for the delayed response. Yes, i would be interested in taking a swing at that. My next PR will attempt to incorporate all your feedback.

@jaloren great! No worries, there's still plenty of time to get this into RF 2.9. My plan is to get RF 2.9 alpha 1 out next week so that's already pretty close, though.

How is the work with this going @jaloren ?

@stasiek slowly unfortunately. My day job has been intruding and preventing me from doing much else. I'll try to dig into this in the next week.

@pekkaklarck  so i have started digging into this. Can you point me to the modules that implement functionality you described in 1 and 2? I want to make sure I am looking at the right files.

Sorry for a bit late reply due to business trip.
Creating links from the execution errors section to actual keywords is in jsmodelbuilders.py and jsbuildingcontext.py. See especially MessageBuilder.build in the former. There may be other places that need to be modified, but this is a good place to start.
Mapping messages logged using logging module is in pyloggingconf.py. Probably don't need to edit other parts than RobotHandler._get_logger_method.

@pekkaklarck thanks for the pointers. I'll take a look. Btw, i noticed that these two modules seem to be identical though presumably they are there for different purposes.
src/robot/output/librarylogger.py
src/robot/api/logger.py
Since they both seem to be doing the same thing (write at a certain log level), I added a function to handle the error leg to both. Is that okay or should i leave the librarylogger.py alone?

@pekkaklarck I believe I have implemented changes for points (1) and (2) except for the fact that the actual report being generated does not show the keyword linking for error log levels like it does for warn. So clearly I am missing something. :)
Note I am using the builtin log keyword to test this -- I am assuming that should not be a problem.
When I looked through through the jsmodelbuilders.py and jsbuildingcontext.py, the only place where I could see the linking being done is MessageBuilder.build function. So that's where I added the simple if block like so. And yet its still not working.
I'll pop into the IRC robotframework channel this week...I suspect it would just be easier if we chatted.
   def build(self, msg):
        if msg.level == 'WARN':
            self._context.create_link_target(msg)
	if msg.level == 'ERROR':
             self._context.create_link_target(msg)
	self._context.message_level(msg.level)
        return self._build(msg)

Yes, src/robot/api/logger.py and src/robot/output/librarylogger.py are pretty identical. The former is the public API we expose to test libraries and the latter pretty much an implementation detail. IIRC the latter was needed to avoid cyclic imports. Anyway, both need to be updated.
I expected some other changes to be needed to get errors linked in addition to modifying MessageBuilder.build. I don't remember exactly how the logic goes, but there's a change that javascript code that generates log files need to be updated too. That would mean updating either src/robot/htmldata/rebot/log.html or some javascript files included into it. Log file generation is somewhat complicated because it is so much performance, memory and file size optimized that understanding it can be difficult. If you cannot figure it out, chatting more on IRC is probably a good idea.

Any progress here? Can I help somehow?

@pekkaklarck I'll look at the log.html and javascript and get back to you this week. I may need to chat in the IRC based on whether I can figure out the log.html on my own.
Apologies for the slowness -- day job keeps intruding.

@pekkaklarck
Actually, I am thinking it may not require any changes to the javascript. In the output.xml, there is an  element. If  manually place a message generated with log keyword using error loglevel, then the log.html automatically places it in the test execution errors section and links it to the keyword. So I am thinking I just need to figure out how to get messages logged at error put in the output.xml...any pointers there?
Btw, any day or time you'd be around IRC? I am in central standard time. I'd really like to wrap this up in the next day or two and figure chatting there would bring this to a close pretty quick.

@pekkaklarck  Once I got around the need  to hack javascript :), everything else fell into place. I believe I have implemented all the changes that have been outlined, including the test cases and documentation.
I am almost ready to submit the PR on this branch https://github.com/jaloren/robotframework/tree/ExposeErrorLog
However, there's one sticking point. I wasn't sure how to deliver the acceptance test cases. I simply wrote a robot test suite, where each test case demonstrates that error log level works in particular context. Is that sufficient? And if so, where do I put them in the src?

I decided to play with this myself and noticed that just changing if msg.level == 'WARN' to if msg.level in ('WARN', 'ERROR') in output/logger.py and in reporting/jsmodelbuilders.py was enough to get errors logged with the Log keyword shown correctly. On a quick look at your branch, it seems you've done the same, and also exposed the new ERROR level in other places too. Good job!
You are right that this new functionality needs tests. They belong to the atest directory where we have suites using certain functionality under atest/testdata and suites running them and verifying results under atest/robot. See atest/README.rst and look at some of the existing suites for more information.
Notice that Robot has three different ways to log (stdout, robot.api.logger and Python's logging) that all need to support the ERROR level. That obviously means they all need to also be tested. I'd prefer adding the tests in suites already testing these logging approaches, but haven't looked at the existing suites does that actually make sense. If not, having a new suite just testing logging errors is fine too.

@pekkaklarck  For the api logger and python logging, i have modified the test suites

test_libraries/logging_api.robot
testdata/test_libraries/logging_with_logging.robot

I have pushed that to my private branch.
However, I didn't write a test case for the log keyword. In testdata/test_libraries/builtin/log.robot, there is a ‘Log with different levels’ test case. There is already a log keyword with the error log level. I am not sure what to make of that since presumably that level wasn’t exposed until the changes I put in. However, since that seems to be the test case I was going to write, can you please take a look and let me know if there needs to be a separate test case on top of that?
Assuming that's all good, I should be ready to submit a PR.

Interesting that Log is already tested with the ERROR level. Shows how this level hasn't been handled consistently and why it is a good idea to change the behavior as proposed here.
That existing test should nevertheless be updated to verify that the logged error ends up to execution errors too. Basically that requires adding something like this to the test case:
    Check Log Message    ${ERRORS[<index>]}    The message    ERROR

Obviously also other places where the new(ly enhanced) ERROR level is tested should verify the message both under the test and in execution errors. You apparently have already added tests for the programmatic logging APIs, but remember that  print "*ERROR* The message" needs to be tested too.
Please go ahead and submit a PR. We can continue discussion there once I've reviewed it. If more changes are needed, you can squash the commits when you are ready. For that to be easy, it's recommended to have a branch for the PR.

I was looking where adding a link from a warning in a log file to a keyword causing the warning is tested and only found unit test test_message_linking in test_jsmodelbuilders.py. Probably it's enough to either test also errors in that test or rename it to, for example, test_warning_linking and add new test_error_linking.

Note I added test case Log Error Using Print in robot/test_libraries/logging_with_logging.robot.
I took a look at the unit testcase. I originally attempted to modify the unit test to verify the linking however it kept failing. I then checked out git revision 495befccb8e which was prior to any of my commits. However, that unit test kept on failing, which leads me to believe someone else's commit caused a regression.  I decided not to try to modify the test case until whatever this issue is resolved.
======================================================================
FAIL: test_linking (test_jsmodelbuilders.TestBuildErrors)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/Users/jaloren/gitrepos/robotframework/utest/reporting/test_jsmodelbuilders.py", line 409, in test_linking
    (0, 3, 'Linkable', 's1-t1-k1')))
  File "/Users/jaloren/gitrepos/robotframework/utest/../src/robot/utils/asserts.py", line 182, in fail_unless_equal
    _report_unequality_failure(first, second, msg, values, '!=')
  File "/Users/jaloren/gitrepos/robotframework/utest/../src/robot/utils/asserts.py", line 245, in _report_unequality_failure
    _report_failure(msg)
  File "/Users/jaloren/gitrepos/robotframework/utest/../src/robot/utils/asserts.py", line 231, in _report_failure
    raise AssertionError(msg)
AssertionError: ((-1L, 5, 'Error'), (41L, 3, 'Warning'), (0L, 3, 'Linkable', 's1-t1-k1'), (-1L, 5, 'Linkable')) != ((-1, 5, 'Error'), (41, 3, 'Warning'), (0, 3, 'Linkable', 's1-t1-k1'))

That test is passing for me in updated master.


Problem Coding Music!

Hello,
I've been trying for a few days to code this song into my sketch, but it renders the sketch invalid and I don't know why. I am using a local server too. Here is the code:

var song;
var deltaX = 0;

function preload() {
song = loadSound("Nuori prinssi 2.mp3");
}
function setup() {
createCanvas(windowWidth, 500);
song.play();
}
function draw() {
background('lightblue');
translate(deltaX, 0);
deltaX++;
// cloud
noStroke();
fill( 255 );
ellipse(100,100,150,50);
ellipse(75,125,150,50);
ellipse(200,125,150,50);
ellipse(150,135,150,50);
if (deltaX + 85 > width){
deltaX = -145;
}
}
Thanks in advance!

I did not see anything immediately wrong, assuming that the there is no extra character in front of the variable initializations fro song and deltaX.
Did you load teh sound library by uncommenting line seven in index.html from:
<!-- <script src="p5_lib/p5.sound.min.js"></script> -->
to:
<script src="p5_lib/p5.sound.min.js"></script>


How to infer on self-defined transformer structure?

Hi, I find that the examples in the project is on the basis of common transformer structure in the pytorch python package. I am wondering what i can do if i want to do inference on a model which has a variant structure of transformer block.

You can modify the source codes (c++ and cuda) to support variant structure.


WIP: set ownerreference for policy reports

Signed-off-by: Issif<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area config

/area outputs

/area tests

What this PR does / why we need it:
Set the OwnerReference to the Namespace for PolicyReports
Which issue(s) this PR fixes:
#331

Fixes #
Special notes for your reviewer:

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

Needs approval from an approver in each of these files:

OWNERS [Issif]

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

LGTM label has been added.  Git tree hash: 2c68c4ed34178fa865746a5dcc69fffe7d9c8d4d

New changes are detected. LGTM label has been removed.

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

Needs approval from an approver in each of these files:

OWNERS [Issif]

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

LGTM label has been added.  Git tree hash: 8000cdb43711b32b862cb110779c9076da3f6ae7

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

Needs approval from an approver in each of these files:

OWNERS [Issif,fjogeleit]

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


Unable to retrieve keypair

Seems I can't dispatch tx :|
But I guess it's related https://docs.kodadot.xyz/tutorials/how-to-fix-error-keypair.html#_1-are-you-using-a-ledger-hardware-wallet :|
https://kodadot.xyz/transfer/?target=G29NScLSew5zqwmJAPupvJWDCDkpxKUhDnMeVdD2BBcnHar&usdamount=500

no update here

@cryptodamsky 🥺

confirm error also when minting new NFT
@yangwao
https://user-images.githubusercontent.com/81104034/159170191-c891835e-aca5-4b00-bf30-faa2667c24bc.mp4

this is the same problem on my report  #2594

same here. just tried to create my first nft 😢

@vikiival is investigating it already, somehow merging something broke it.
I heard it's a special time to highlight, that we need better QA on the pipeline, writing tests &c, as we are adding way too much stuff without robust testing  😅 🫶
https://docs.kodadot.xyz/incentives/incentivized-qa.html#code-of-conduct

@vikiival is investigating it already, somehow merging something broke it.
I heard it's a special time to highlight, that we need better QA on the pipeline, writing tests &c, as we are adding way too much stuff without robust testing 😅 🫶
https://docs.kodadot.xyz/incentives/incentivized-qa.html#code-of-conduct

Sir @yangwao why u not label my report at #2594 It seems this is the same case :)


Edit "Implement Angular Guard" Step 2

Please edit Code block for Step 2. "Set the MsalGuard on the routes you wish to protect in src/app/app-routing.module.ts:" to include
import { BrowserUtils } from "@azure/msal-browser";

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 28c13084-08f6-c015-7769-c2ab9170c1d2
Version Independent ID: 44d5c264-8c95-0ae5-be9c-8982cea33223
Content: Tutorial: Create an Angular app that uses the Microsoft identity platform for authentication using auth code flow - Microsoft Entra
Content Source: articles/active-directory/develop/tutorial-v2-angular-auth-code.md
Service: active-directory
Sub-service: develop
GitHub Login: @henrymbuguakiarie
Microsoft Alias: henrymbugua

@dwinsi
Thanks for your feedback! We will investigate and update as appropriate.


SECURE: File not allowed to be loaded - dbghelp.dll

Describe the bug
Ghidra debugger throws exceptions
To Reproduce
Steps to reproduce the behavior:

Drag the exe file on the debugger icon (from the project view). It's an x86 application.
Click on the debug button and choose "debug ... in dbgeng locally IN-VM" or "debug ... dbgeng locally via GADP" (both behave in the same way)
See error

Expected behavior
No errors, debugging works as expected.
Attachments

Environment (please complete the following information):

OS: Windows 10 Pro N
Java Version: JDK-21.0.1
Ghidra Version: [e.g. 10.4]
Ghidra Origin: Running ghidraRun.bat (downloaded from the github)

Additional context
Running ghidra as administrator
I tried copying all the dlls from system32 to the jdk/bin folder and get basically the same error. (Note: I only copied the dlls, no subfolders).

I also get an error at the end, but I'm unsure if it's related

I have no JAVA_HOME env variable setup, in case that matters

@monochromatic-space  Going to probably need a little more info to troubleshoot, but you may actually be pretty close to having the debugger working correctly, so would like to verify that.  Will start with the second error first as that is, I think, the more significant issue....
That error appears when the name of the program as loaded statically into Ghidra does not match the name of the executable in the operating system's running processes list.   For example, I have program loaded right now into Ghidra as WinHelloCPP.exe (one of the programs from GhidraClass/ExerciseFiles we include in the distro).  When I run that from the debugger, I will get the same error with "WinHelloCPP.exe" in place of "pluginbasededitor.exe".  If I hit OK and look in the "Modules" tab in the pane on the right in the default tool, I will see one executable listed marked "WinHello.exe".   If I select that entry, right-click, and choose "Map Module to WinHelloCPP.exe", the problem will be resolved, and I can continue debugging.
Why does this happen?  Well, a number of possibilities actually.  For example, I could write a test program and call it TestA, compile it, rename the resulting executable TestB, copy that to a new location as TestC, load it into Ghidra and rename it TestD.  Ghidra really has no way of making the necessary connections to tie TestD, the program in Ghidra, with TestA, the name which will appear in an execution, without intervention from me.
Am hoping this is your main problem, and that getting past it will allow you to do what you wish.  That said, let me throw out a few other possibilities....  We have in the past had issues running executables with spaces in their paths.  This is a Windows versus Java thing.  I really had hoped we had fixed this in the current release, but I've been wrong before.
The "SECURE: File not allowed" message also comes up over and over again.  It may not be important, but then again.  I think you said that you copied dbgeng, dbgmodel, dbghelp , etc. directly from System32, which I'm guessing just includes the versions from the default Windows 10 Pro load.  I would very seriously recommend downloading the Windows Debugger Tools ([https://learn.microsoft.com/en-us/windows-hardware/drivers/debugger/debugger-download-tools#install-debugging-tools-for-windows]) and copying those versions into the jdk/bin directory.  The versions that come with the OS are a little underpowered and frequently result in consistency issues when used with Ghidra.  They should work, but our mileage has varied in the past.  If you care, there are a number of post in our GitHub/Issues section related to this.  I think #3479 contains me latest thinking, but I honestly don't think I truly understand all the corners of this issue, sadly.
In any case, let us know if any of the above works or doesn't, and we help you get past whatever's next!

Thanks! That kind of helped.
Some of the breakpoints are getting hit now, but then it seemed to have crashed.
I got this in the interpreter window:
CommandLine: D:\Battleforge\Skylords Reborn\pluginbasededitor.exe
Unable to add extension DLL: ntsdexts
Unable to add extension DLL: uext
Unable to add extension DLL: exts
SECURE: File not allowed to be loaded - C:\WINDOWS\SYSTEM32\dbghelp.dll
Error code: Win32 error 0n5
The call to LoadLibrary(ext) failed, Win32 error 0n2
"The system cannot find the file specified."
Please check your debugger configuration and/or network access.

Additionally, I also saw this:

...

Although I'm not very happy about those null pointer exceptions in the Debug Console, I'm pretty sure they are because your target has already died.  (At least, I'm going with that for the moment.)
When you get an exception running under a debugger in Windows, the OS hands the exception info back to the debugger, and allows you a chance to do something about it.  You can basically either fix it, ignore it, and tell the OS it's been handled (even if it hasn't).  If you just continue, the OS will try to handle it on its own.  If it fails, you get a second chance in the debugger.  Ignoring the exception after a second chance notification "usually" leads to the target terminating,
By default, most exceptions are marked unhandled when a debugger starts on the assumption you might be actually looking for those bugs, but very frequently that's not the case.  For example, page faults are usually handled by the OS, by paging in the relevant memory.  You basically don't want to mess with them.
In your case, the "C++ EH exception - code e06d7363" message is telling you that you've hit a C++ structured exception handler.  Am guessing you want to ignore these.  To do that, in the Objects window, open Sessions[0].Processes[xxx].Debug.Exceptions.C++ EH exception, and select either Continue or Execute.  T will toggle you through the options.  At a guess, you'll want "Continue:Not Handled" (i.e. let the OS do its thing) and "Execute:Output-only" (tell me, but don't stop).  If those don't do the trick, well, you may have to play around a bit.  (Also, depending on your target, it's possible this is an anti-debugging trap, in which case the correct action may be position-dependent.)

I decided to give this another try in Ghidra 11, but get this error when trying to launch the debugger.
First I get this popup, which points correctly to the exe file:
, when choosing launch I see the following error in the interpreter:

OS: Windows 10 Pro N
Java Version: JDK-21.0.1
Ghidra Version: 11.0
Ghidra Origin: Running ghidraRun.bat (downloaded from the github)

I think there's a ticket already in for this, but, unfortunately, it's a slightly more complicated fix than you might expect.  Problem is "/D:/whatever.exe" is the name associated with your imported program in the database, despite the fact you want "D:/whatever.exe" for the command line.  We could probably introduce logic that says effectively "if the pattern is "/%x:/something" and %x is a single letter, get rid of the initial slash", but, given this is a generic component, not obvious this is the right solution.  For now, the workaround - delete the leading slash.

Thanks, that caused that error to go away, but instead I'm getting an error when trying to debug.

No breakpoints are hit, it runs for a couple of seconds, and then I get this error

I also saw this error in the interpreter

Hmmm - the "Callback failed" errors are being generated by the Microsoft DLL, not the Ghidra code, and unfortunately that's one of those errors that has a couple dozen possible causes.  I don't have a good guess as to what's causing it (much less a solution).  That said, I don't think it should affect your debugging session.
The stack trace, on the other hand, is definitely ours, and I'll need to ask one of the other guys about the cause.  At a guess, the selected thread has disappeared or is in an odd state.  If you click "OK", is the debugger now in a bad state?  i.e. can you continue stepping?  And does it continue to re-pop the same error if you can step?

Oh, just noticed one thing:  I would definitely NOT work with two connections open, i.e. you have two entries in "Debugger Targets" marked DBGMODEL  While theoretically possible and something we intend to be possible, there are known issues running more than one connection.  If you restart, I would kill/disconnect the previous session using the pull-down in "Debugger Targets".  I might also consider using dbgeng IN-VM (vs. dbgmodel IN-VM) as it's considerably more well-tested.

Trying to run dbeng locally in VM results in this when adding breakpoints:

when switching back to the other debug mode, I get this error:

Without seeing your entire tool, I can't be sure, but it sounds like you've done something to move off the current snap.  What do I mean by that?  The debugger keeps a running tally of the state as you debug, which we call the trace.  You usually move only forward within the trace, typically by stepping or running to breakpoints, but you can also move backwards to see where you've been.
If the Dynamic listing window is grey (vs white), you are not at the most recent point in time (the most recent snap).  You can do this a number of ways, but using the double-arrow buttons in the Toolbar is the most common way.  (You can also move the little arrow in the Threads window, or you can switch the debugging mode in the Toolbar off of "Control Target".)  Are any of these the case?
The message regarding dbghelp.dll in most cases doesn't matter and as before is autogenerated by the underlying engine, based on the path used to load DLLs.

I'm running into this issue where a part of the code is being debugged at the beginning that doesn't get decompiled:

Stepping through the code gives me these errors:

It finally crashes with this error:

I can also see this other error:

After that it continuously throws the 1st exception if I try to continue stepping through the code

On the dbghelp.dll errors, if these are really getting in your way, I would go the route of duplicating the dbgXxxx.dll's from Windows Kit/10/Debuggers in the bin directory of your JDK.  Not ideal, but, as discussed elsewhere, seems to be the only thing that works.
Re the lack of decompiler output, probably not relevant. If there's no function, there's no decompiler output.
The last two errors are probably really the same error, and that definitely looks like a bug.  Not sure why exactly the error is being thrown (deliberate code obfuscation, maybe), but the error should probably result in a
console message, not a pop-up/stack trace.

I already have those dlls in the JDK folder

Interesting - have you verified that those are actually the loaded DLLs?

how do I do that?

I usually use SysInternal's procexp64, find the entry for javaw.exe, and enable the DLLs pane, but am guessing there are other ways to do the same, say from PowerShell.


Project filter toolbar not responsive (if too much filters)

In the content/home/project.md file, when "too much" filters are set, then the button bar could be no more responsive (and be hidden from the screen).

I guess it comes from file layouts/partials/widgets/projects.html, line 31:

The group of buttons become too long for the screen. We could add "flex-wrap"

This seems to work but we lose some nice round corner for the last and first button for respectively the first and second line. Any idea?

Thanks! :)

However, we're unlikely to convert the actual widget to use a dropdown box.

I prefer the actual widget, too, but if I list too many filters, I'll consider it :-)

I think this should be explained in the Managing Content page, section 'Create a project' as I have been banging my head for the past 12 hours thinking why my filters are showing in the Project section but when I clicked them nothing appear on my page! Especially for people not familiar yet with html like me; I keep tinkering the TOML and nothing happens LOL


[Feature] Move Actions.md documentation into classes

For each action, there should be a description in the source file.
This would enable a cool new command: https://twitter.com/KrauseFx/status/586912096769744897

Related commit: https://github.com/KrauseFx/fastlane/commit/debbeb7983e58facd87bf2891ca01aa763873519

Basically done, just need the actual documentation.


Add healthchecks, switch from Docker Compose V1 to V2

As per Docker documentation:

From July 2023 Docker Compose V1 stopped receiving updates. It’s also no longer available in new releases of Docker Desktop.

Therefore, the first patch replaces all uses of Docker Compose V1 (docker-compose command) with Docker Compose V2 (docker compose command) and changes installed version of Docker Compose in main.yml.
This gives a possibility to replace manual checking the health of the containers with healthchecks and using --wait flag in docker compose. --wait flag makes Docker wait for all healthchecks to be in "healthy" state. Since docker-compose.yml files in this repository use cqlsh to determine the health of a container, this means that Docker will wait for all nodes of the cluster to be fully bootstrapped and their CQL port ready.
All of this makes adding new nodes and making them boot in order easier, it is just adding new healthchecks and dependencies.

LGTM, I agree that the healthcheck timeout could be lowered (to 1 minute/1.5 minutes?)

Done


Session not persistent when using database as session driver

Hi there,
Because of my production environment I need to use the database as a session driver. You do this by setting the SESSION_DRIVER in the .env file to database. Unfortunately whenever I try to login the Authentication with Auth0 wil succeed, but the Authentication inside of Laravel is not working. l keep getting redirected to the login page. Works fine when using "file" as SESSION_DRIVER.

@nielsvandoorn did you find a solution for this?

@mchainanicimpress Sorry for the late response. We fixed this by using redis as the session driver

@nielsvandoorn I did something similar by using cookie

So this has been plaguing me for a few days.
Why does the database session driver not load a just-saved session correctly?


feature: build static task

Could we implement a task for the Makefile to do something like CGO_ENABLED=0 GOOS=linux GOARCH=amd64 go build -a -installsuffix cgo .?
This helps to build a static binary for a docker image that has no dynamics.

Not sure if this is good enough, but seems to do the job for me...
# Build Static
.PHONY: static
static: fmt lint vendor | $(BASE) ; $(info $(M) building static executable…) @ ## Build static program binary
	$Q cd $(BASE) && CGO_ENABLED=0 $(GO) build \
		-a \
		-installsuffix cgo \
		-tags release \
		-ldflags '-X $(PACKAGE)/cmd.Version=$(VERSION) -X $(PACKAGE)/cmd.BuildDate=$(DATE)' \
		-o bin/$(PACKAGE) main.go

I think this is good enough. I don't add it directly because adding features goes against the "minimal" Makefile I try to provide here. But people interested in such a rule are likely to find this issue.


Add DragonFly umtx_{sleep, wakeup}.

This adds DragonFly's futex functions: https://man.dragonflybsd.org/?command=umtx&section=2

@bors r+

:pushpin: Commit 9579c1dc5878abe52e7aeef229d69284cdc1543c has been approved by Amanieu

:hourglass: Testing commit 9579c1dc5878abe52e7aeef229d69284cdc1543c with merge 01b49024c26378a78e48358645a119595518e707...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: Amanieu
Pushing 01b49024c26378a78e48358645a119595518e707 to master...


[Enhancement] Use Windows 11 style settings menu

Make the setting menu looks like this:

To achieve this, we should use Community Toolkit Lab.


Renew/replace certs affected by LE CAA Bug

Per this bug, LE will start revoking certificates. They have advised to renew/replace certificates that may have been affected. I am using mod_md, so how can I force a renew?

I don't think there's a friendly way.  What I did: delete the "domains" directory (or move it elsewhere if you need to keep the history).  Restart httpd.  mod_md will see that it has no certificates, so it will request new ones.
The downside is that your site will be off-line until the new certificates arrive - visitors will get a 503 - service unavailable.   Depending on what validation method you use, this could be a few seconds or a few minutes...
I put a better way to force renewal  (using this example) on a wish list earlier today...

What I did: delete the "domains" directory (or move it elsewhere if you need to keep the history). Restart httpd. mod_md will see that it has no certificates, so it will request new ones.

Other than having to restart about 3 times to resolve issuance errors, it worked like a charm.

Glad you were successful.
One restart should have done it if you were patient.  What were the errors?
P.S. The reason I just delete domains instead of the whole tree is to re-use the LE account.


Sqlserver upsert with schema fail

GORM Playground Link

https://github.com/go-gorm/playground/pull/665
Description
const (
	skema = "my_schema"
)
...
nameStrategy := schema.NamingStrategy{
TablePrefix: skema + "." + "custom__",
}
...
db, err = gorm.Open(sqlserver.Open(dbDSN), &gorm.Config{
	NamingStrategy: nameStrategy,
})
db.Exec("CREATE SCHEMA my_schema")

	user := User{
		Model: gorm.Model{
			ID: 1,
		},
		Name: "jinzhu",
	}

	DB.Clauses(clause.OnConflict{
		Columns:   []clause.Column{{Name: "id"}},
		UpdateAll: true,
	}).Create(&user)

When try to upsert to sqlserver with schema, the script generated is not correct to sqlserver.
generated
MERGE INTO "custom__users" USING (VALUES('2023-11-24 10:17:52.412','2023-11-24 10:17:52.412',NULL,'jinzhu',0,NULL,NULL,NULL,0,1)) AS excluded ("created_at","updated_at","deleted_at","name","age","birthday","company_id","manager_id","active","id") ON "custom__users"."id" = "excluded"."id" WHEN MATCHED THEN UPDATE SET "updated_at"='2023-11-24 10:17:52.412',"deleted_at"="excluded"."deleted_at","name"="excluded"."name","age"="excluded"."age","birthday"="excluded"."birthday","company_id"="excluded"."company_id","manager_id"="excluded"."manager_id","active"="excluded"."active" WHEN NOT MATCHED THEN INSERT ("created_at","updated_at","deleted_at","name","age","birthday","company_id","manager_id","active") VALUES ("excluded"."created_at","excluded"."updated_at","excluded"."deleted_at","excluded"."name","excluded"."age","excluded"."birthday","excluded"."company_id","excluded"."manager_id","excluded"."active") OUTPUT INSERTED."id";

should be
MERGE INTO "my_schema"."custom__users" USING ...

Issue still exists. Why hasn't this been merged?


Review Java callbacks and threading

The CompletionStage returned by future methods will continue execution on a Tokio thread if any of the async combinators are used, e.g. CompletionStage.theAccept(...).
This can cause problems if the chained computation calls anything that might use block_on.
Java doesn't have a global executor thread poll like C# does. We need to do one of the following:

Carefully review our usage of block_on in each library (or remove it altogether).
Initialize a static thread poll on which the Java backend can complete futures.

This isn't a panacea, but the various combinators allow you to specify an executor on which to resume:
https://docs.oracle.com/javase/8/docs/api/java/util/concurrent/CompletionStage.html#thenApplyAsync-java.util.function.Function-java.util.concurrent.Executor-


Code actions support

Hi there!
I'm looking to replace a plugins use of the Facebook package with this suite and I noticed that when declaring the hyperclick usage the behaviour seems slightly different.
With that version of hyperclick it seems that it also handles the code-actions support too, I've played around trying to see if I can get code-actions to work based on the docs and some other packages like ide-typescript but I think ultimately this isn't supported in atom-ide-base/related packages?
Is the code-action support planned (maybe related to https://github.com/atom-community/atom-languageclient/pull/139) or is the recommendation to use https://github.com/steelbrain/intentions instead?

Thanks for the issue. I have been working on many different things to make this possible. The last piece of the puzzle is implementing the UI for code actions.
https://github.com/steelbrain/linter/issues/1705

See https://github.com/steelbrain/intentions/issues/84 for some new discussion on the topic.


Ionic 2 - Angular 2 | The Website doesn't load with [(ng-Model)] (with brackets) in the input

Ionic version:  (check one with "x")
[x ] 3.7.0
I'm submitting a ...  (check one with "x")
[x ] bug report
Current behavior:
If I enter "ionic serve", the website only loads, if I leave in the import the braces around the ng-model
Expected behavior:
The website should also load with the brackets around ng-Model, because as far as I know, this is the correct spelling
Related code:

Other information:
I am currently working in Ionic and Angular. I would just like a simple program that reads two values, adds and outputs again.
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
No Errors or warnings

It's because there shouldn't be a hyphen. It's just  [(ngModel)]

Hello! Thanks for opening an issue with us! As this seems like more of a support question I will urge that you ask this question on our forum or on our slack channel. Thanks for using Ionic!


newly successfully pushed images still not appear in Recent activities feeds.

version: Portus-2.0.1
also could not find error msg in log/production.log

here is the log before, this seems to be the normal behavior.

how can I debug this to find out what's wrong?

the portus interface return 301

updates:
after I sync manually, the images can be searched.
but newly pushed images still not appear in Recent activities feeds.

Which version of Docker are you using, and which version of docker distribution ? As stated in the README file, not all versions are supported in 2.0.1. Maybe you should be considering updating to 2.0.4 ?

I have the same problem and I am using Docker 1.9.1, Portus 2.0.5, Registry 2.4.0

@boyand could you provide more information on this ? I'm asking because this is one of those issues that are really hard to debug because there are a lot of factors that can trigger this issue, and it may very well be a problem in your current setup (not a bug in Portus itself).

I did some more debugging and in the (current, 8532207a390d2ddb73cff59cf6385e31cab63210) code, after changing this I saw every (new) push to an existing tag in the activity feed:
diff --git a/app/models/repository.rb b/app/models/repository.rb
index 0491214..057bc35 100644
--- a/app/models/repository.rb
+++ b/app/models/repository.rb
@@ -142,7 +142,13 @@ class Repository < ActiveRecord::Base
     elsif repository.tags.exists?(name: tag)
       # Update digest if the given tag already exists.
       _, digest = Repository.id_and_digest_from_event(event, repository.full_name)
-      repository.tags.find_by(name: tag).update!(digest: digest)
+      repository_tag = repository.tags.find_by(name: tag)
+      if repository_tag.digest == digest
+        return
+      end
+
+      repository_tag.update!(digest: digest)
+      repository.create_activity(:push, owner: actor, recipient: repository_tag)
       return
     end

This should've been fixed by #1031. If not, feel free to re-open.

hey guys ,the case is closed but the problem still exists with me .. i am using docker 1.12.6 , i deployed the development way in docker containers and pushed after all the successful configurations i assume ., i was able to push and pull the image to and from the registry , but i can not see the pushed images on the portus dashboard.


L10n for Portuguese

Description
This PR adds the Localization for Portuguese.
OBS:
It can be reviewed/merged after this PR (#22) be merged

Thank you so much! 🎉


Is there a good way to support multiple embeddings tied to a particular record?

Let's say you have a collection of Documents and you store embeddings for these documents, but due to token constraints, you have to store multiple embeddings tied to each document.
Do you have a recommended way for supporting this with Neighbor?
Current ideas:

Have an Embeddings model that has an embeddable--Not sure Neighbor would support this out of the box because it would have to support filtering on embeddable_type
Have a DocumentEmbeddings model and run nearest neighbor searches on that with some deduplication logic

Hi @weilandia, either approach should work.
For 1, nearest_neighbors returns a scope, so you can add where(embeddable_type: ...).
Here's a full example for 2:
require "bundler/inline"

gemfile do
  source "https://rubygems.org"

  gem "activerecord", require: "active_record"
  gem "neighbor"
  gem "pg"
end

ActiveRecord::Base.establish_connection adapter: "postgresql", database: "neighbor_example"

ActiveRecord::Schema.define do
  enable_extension "vector"

  create_table :documents, force: :cascade do |t|
    t.text :name
  end

  create_table :document_chunks, force: :cascade do |t|
    t.belongs_to :document
    t.column :embedding, :vector, limit: 3
  end
end

class Document < ActiveRecord::Base
  has_many :document_chunks
end

class DocumentChunk < ActiveRecord::Base
  belongs_to :document

  has_neighbors :embedding
end

def generate_embedding
  3.times.map { rand }
end

d1 = Document.create!(name: "Doc 1")
3.times { d1.document_chunks.create!(embedding: generate_embedding) }

d2 = Document.create!(name: "Doc 2")
3.times { d2.document_chunks.create!(embedding: generate_embedding) }

chunks = DocumentChunk.includes(:document).nearest_neighbors(:embedding, generate_embedding, distance: "cosine").first(5).uniq(&:document_id)
pp chunks.map(&:document).map(&:name)

Thanks @ankane!


This repository should point to azureml-examples for a more interesting set of use cases

This repo gives a great, clean, example of how to use GitHub actions to automate MLOps.
Users will be interested in more interesting / deep examples (e.g., something more interesting than scikit-learn). This repo should point to azureml-examples for those.

it's in the Reference section, should it be somewhere else in the README?

Oops, my bad for not noticing that.


How to hide single column in column selection dropdown?

i tried Data-Switchable="false" , but still it is not working

I will upload a new version today which includes that feature.

Please go here!


Flake Unx

About the PR

Media

[ ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

add: Added fun!
remove: Removed fun!
tweak: Changed fun!
fix: Fixed fun!

Please make a proper name and description, read the PR guidelines https://docs.spacestation14.io/en/getting-started/pr-guideline


Made profile names visible

Fixes Issue
#1928
The text is now visible on hovering over the profiles.
Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[X] All new and existing tests passed.
[X] This PR does not contain plagiarized content.

Screenshots
Previous

Now

Please do not delete the package-lock json

Hey @schmelto I retrieved the package-lock.json file. You can review it now.

Still it's showing changes in the package-lock.json file, You can follow this short guide  if you're facing some issues in reverting the changes!
Let us know if you need any help with this! :)

Okay. I'll let you know if I face any issues.


Add dags.git.gitSync.maxFailures

What does your PR do?

add dags.git.gitSync.maxFailures value
update git-sync.sh to allow up to maxFailures once the initial sync has succeeded

Checklist

[X] Commits are signed
[X] Commits are squashed (if appropriate)
[X] Chart version bumped
[X] Chart documentation updated
[X] Passes linting

Signed-off-by: Nathan J. Mehl<EMAIL_ADDRESS>

(the lint failure appears to be a failure to run the test?)

Okay, now that the lint test is passing --- @gsemet ? :)

I am ok. @thesuperzapper do you object ?

@gsemet this has been implemented in the new 8.0.0 already
(Sorry about that taking so long, it is finished... I just keep getting tied down with Kubeflow. I hope to merge it tomorrow.)

No problem! So we can close this PR when yours is released

oh nice.  is there an open PR for the other implementation?

@n-oden I ended up intentionally not implementing this in 8.0.0.
Here is the code for the new git-sync container:
https://github.com/airflow-helm/charts/blob/airflow-8.0.0/charts/airflow/templates/_helpers/pods.tpl#L82-L160
Was there a specific reason why you wanted this behaviour (as opposed to the git-sync container just restarting, and doing a CrashLoopBackoff)?

@thesuperzapper we have internal monitoring for pods in CrashLoopBackoff because this usually signals a failed CI/CD deploy.  The goal here was to make it so that a brief network glitch between the git-sync pod and github would not trigger alarms.

Looking at the 8.0.0 chart, it seems like you're now using https://github.com/kubernetes/git-sync? If so we can achieve the same effect by passing the GIT_SYNC_MAX_SYNC_FAILURES environment variable -- I'll put up a PR to do that.


fix: broken noop case for default params.

Ouch sorry, previous release includes a break on the following case:
function(bar = 'foo) {
  return bar;
}

I've added a "noop" set of tests.

#31

My fault, I should have added more tests after the "param defaults" PR ;-)


Specify openAPI for MVP

Fill out the v1.yaml openapi spec enough to cover the use cases of the internally-defined MVP.

This was done in the referenced PRs.


Add special time for Tokyo Disney

At OpeningTime Object, add two type of special time for Tokyo Disney: Character Greeting and Parades and Shows.

Ignore the failed check.
Will look at this sometime in the next few days :)
Glancing at it though, could we use the "supports_ride_schedules" style properties on parks for this, so we can programatically determine which parks support such features.

But both of them are not showing at ride list, because they are in the event group not in attraction, so that if put them into ride schedules, they can't match the ride list in the code process now.
Or, redesign the structure of rides info?

I'm not entirely sure what this commit does, can you explain what this adds to the dataset, update the README with any new variables that are available, and if this is something that needs to be extended to other parts of the API?

This commit implement the "special" property of API GetOpeningTimes, Tokyo Disney lost this data, so I put the Greeting and Show schedule into here.

I've checked this out and ran it, I don't think this fits into what "special schedules" means for other parks. This is being used to represent extended opening times for Extra Magic Hours and such.
At the moment, the API does not support returning greets/shows/parades for any parks. See issue #9 for a request to add entertainment to the API. If you want to add this, please extend parkBase with a new method for returning entertainment and implement it for the Tokyo parks.

I'm afraid I'm closing this pull request as it's not had any activity for a while.
If you wish to have entertainment times, we need to implement park entertainment as a separate feature to the project, as schedules for parades etc. doesn't belong in the park's opening times.
Please continue any work in #9 if we want this in the project.
Thanks


Security changes whitelisting

Work in progress

(this addresses #20)

This one is ready to go in, waiting on review

Could we check for the presence of whitelistedKeys and use that as the trigger to filter keys instead of having to use a separate boolean?

So you need atleast one of those keys commented out for the server to start up, else it gives parse errors.

@zackbloom could you review this and check if its ok to merge this?

@zackbloom I have uncommented the default keys sent by bucky in the yaml. It should work in its current form. Please check and merge if everything looks right.

@zackbloom wanted to check with you if you need anything from us after the last comment? Or is this ready to merge?

Friendly ping on the mergeability of this one.


overlord should purge old users with unverified emails

 Issue by sym3tri
Monday Aug 03, 2015 at 23:21 GMT
Originally opened as https://github.com/coreos-inc/auth/issues/352

Consider this case:

Some joker named Mr. Bar tries to register with<EMAIL_ADDRESS>He is not the rightful owner of the email address so he can't verify it. Mr. Foo (the rightful owner of<EMAIL_ADDRESS>ignores the email.

Some time later Mr. Foo tries to register with<EMAIL_ADDRESS>but cannot register b/c that email is already registered. The email will stay in the unregistered state forever. Mr. Foo is sad b/c he cannot register.

Currently there is a workaround which Mr. Foo may not know about:

click "login" instead of "register"
click "forgot password"
send reset password link
click link & reset password
hopefully the client app prompts him to verify his email before continuing, if not his email will never be verified.

Also might be nice for users to only be created once the email verification link has been visited.

There's no registration for the current dex. This would be handled by an app talking to the gRPC API,


New version: uazo.cromite version 129.0.6668.101

Created by 🥟 Dumplings in workflow run #4840.
Logs
Updated: 129.0.6668.90 → 129.0.6668.101
Cannot validate argument on parameter 'ChatID'. The argument is null, empty, or consists of only white-space characters. Provide an argument that contains non white-space characters, and then try the command again.
Submitting WinGet manifests
Cannot validate argument on parameter 'ChatID'. The argument is null, empty, or consists of only white-space characters. Provide an argument that contains non white-space characters, and then try the command again.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

No errors to post.

(Automated response - build 896.)

Automatic Validation ended with:
(Automated response - build 896.)


bundle update API is confusing

Hi!
Let's say there's a gem I'm using in my rails app, let's call it "Slugger". Slugger (2.0.0) depends on another gem, let's call it "Mugger" (spec.add_dependency 'mugger', '~> 7.0.0'). Mugger just released a new version (7.1.0) and it seems to have introduced some changes that break my app.
Slugger also released a new version (2.1.0).
Now I want to update Slugger to the latest version, but NOT update Mugger. Slugger didn't change its dependency version specification for Mugger (spec.add_dependency 'mugger', '~> 7.0.0'), hence old Mugger version should/would work just fine.
So, I tried:

bundle update slugger => mugger got updated
(git reset --hard HEAD)
bundle update --source slugger => mugger got updated
(git reset --hard HEAD)
bundle update --conservative slugger => mugger got updated

NOTE: I do have the newer version of mugger locally on my machine, but I think it would be an unexpected behavior if bundler would use it for that reason, especially in the case of --conservative.
What am I doing wrong?
The only way for me to update slugger to the latest version was to manually edit Gemfile.lock and change only that gem's version number, after which bundle install confirmed that all the dependencies are met.
I think bundle update API is confusing/obscure. I'm doing ruby/rails development for almost 10 years now and I still don't know how bundle update works, besides the "default" bundle update with no extra arguments.
I checked the documentation, but it's still not clear:
https://bundler.io/man/bundle-update.1.html (not sure why the page has .1. in its name)

--source=<name> - does something completely different from what I need (I saw this as a suggested option on stackoverflow: https://stackoverflow.com/questions/11263150/update-just-one-gem-with-bundler
and on this blog: https://ilikestuffblog.com/2012/07/01/you-should-update-one-gem-at-a-time-with-bundler-heres-how/
This blog post actually says that the documentation is wrong and misleading 🤷♂)
--conservative

Use bundle install conservative update behavior and do not allow shared dependencies to be updated.

Hmmmm, but it didn't work. Am I misunderstanding something?
Also, this documentation page is slightly misleading: https://bundler.io/v2.0/guides/updating_gems.html#updating-a-gem-without-modifying-the-gemfile

It will, however, update dependencies of other gems if necessary. For instance, if the latest version of rack-cache specifies a dependency on rack >= 1.2.2, bundler will update rack to 1.2.2 even though you have not asked bundler to update rack.

I think it would be clearer if it was written that "if there's newer version available, it would get updated, no matter that the existing version is compatible". For example, app uses rack 1.2.2 and if there's rack 1.2.3, after bundle update, the rack gem will get updated".

If bundler needs to update a gem that another gem depends on, it will let you know after the update has completed.

I don't think this is the case anymore. Maybe it's just unclear wording, but I don't understand what exactly does this say that bundler does?
I can try to create a repo that will reproduce this issue. Can you please confirm that what I'm seeing is unexpected behavior?
Thanks!
Environment
$ cat ~/.bundle/config

BUNDLE_BUILD__NOKOGIRI: "--use-system-libraries"
BUNDLE_GEM__COC: "true"
BUNDLE_GEM__MIT: "true"
BUNDLE_GEM__TEST: "rspec"
BUNDLE_GITHUB__HTTPS: "true"

$ bundler --version
Bundler version 1.17.1
$ ruby --version
ruby 2.5.3p105 (2018-10-18 revision 65156) [x86_64-darwin18]
$ gem --version
2.7.8

@vfonic I agree with your expectations. It looks like a bug in the --conservative option to me.
Regarding all the confusing docs, how about creating a PR to improve them?

@deivid-rodriguez I'm not sure I understand 100% what's the desired functionality. The reason I opened this issue was to kick off a discussion about possibly changing the functionality.
I think that the ideal solution would be:
bundle update - updates all gems (keeping in mind any dependency gems version restrictions)
bundle update <gem_name> - update the gem and all of its dependencies
bundle update <gem_name> --conservative - update the bare minimum of the gem and its dependencies. For example, if the dependency requirements/restrictions haven't changed, don't update dependencies.

As far as I know, bundler is a sponsored project. Is this correct?
Perhaps some of the development time could go into documentation improvements? I think having an incorrect/misleading documentation should be quite high priority for fixing.


How to install just Volto?

Is your feature request related to a problem? Please describe.
The pnpm install command installs many non-Volto dependencies.
Describe the solution you'd like
Is there a command that installs only Volto's dependencies?

@wesleybl By default pnpm installs dependencies of all packages that are part of the monorepo. To only install the dependencies of @plone/volto, use the --filter option:
pnpm --filter @plone/volto install

@davisagli can we have an "installvolto" script?

@wesleybl I am not clear why that would be useful. Can you elaborate upon your use case?
If the developer knows to read pnpm documentation, why do we need to document it or add a Makefile command in Volto?

@stevepiercy I believe most people using this repository are interested in Volto. Installing the dependencies of all packages takes much longer. Another thing is that searches become slower with so many packages. It would be nice to have a command that installs only Volto. It wouldn't be in make, but in package.json. It would be a "shortcut", a practicality, like the other package.json scripts.

@wesleybl that sounds reasonable to me, but I don't develop Volto. I would suggest that you join the next Volto Team Meeting to advocate for your feature request. Or I can reopen this issue for further discussion. Please let me know.

@wesleybl actually, there are these bits in the documentation:

Running commands
Developing Volto

If you can suggest how to improve that text to what you want, I think there would be no objection. However I don't think it is possible, now that Volto is a monorepo, and packages/volto is one of the packages that comprise Volto the product.

@stevepiercy It's possible; I mentioned how to do it in my comment above. I'm not sure about adding another command to the Makefile or package.json, because the more commands we add, the harder it is to find the right one. If it becomes a common request, we can add it. @wesleybl In the meantime you can add a personal alias in your shell, if it's something you need to do often.

@wesleybl @davisagli I have to admit coming from 17.x.x and lower Volto it's really strange to go inside packages/volto, run pnpm install thinking it's the yarn install equivalent and to have remix and nextjs downloading alongside all of those extra packages that come from those libraries.
Those packages should be example packages that could be worked on but when you want to interact with Volto you don't want to have the extra actions that you didn't even think you would be asking.
I will try to have a doc pull request next week mentioning the filter install given by David in
https://github.com/plone/volto/issues/5559#issuecomment-1866662070
making it more obvious for others like me and Wesleybl on how to get started the quickest with Volto 18 development.

@ichim-david It is mentioned already here: https://6.docs.plone.org/volto/contributing/developing-core.html#running-commands
But it might make sense to also mention it here: https://6.docs.plone.org/volto/contributing/developing-core.html#set-up-the-environment
/cc @sneridagh


System library added to a group has empty path

Short description 📝
Adding system library to Frameworks group using PBXGroup.addFile() method causes BPFFileReference to have empty path
Solution 📦
Edit PBXGroup.addFile method to correctly create a file with path that was passed with sourceTree type sdkRoot and developerDir

Thanks for the quick review @pepibumur ! I've updated the changelog as well.

Thank you for fixing it! We can merge it as soon as CI is happy.


new doc

It is an amazing job, however, I have some questions

what's the difference between doc directory and docs directory? why do we remain the doc?

For first question, there are two main reasons...

In Standard Go Project Layout, they use docs instead of doc. So, this pr is kind of correct it I think.
Because we use github to display our document, so there are important links we should keep when update doc. For example, in kubernetes ingress controller, we link easegress ingress controller to https://github.com/megaease/easegress/blob/main/doc/reference/ingresscontroller.md then we need to keep the structure in doc. And that will influence the new document.


Fomgroup fails, AppComponent_Host.html:1 ERROR Error: Expected validator to return Promise or Observable.

Bug, feature request, or proposal:
I cant use FormGroup on the components, this is not allowed ?
What is the expected behavior?
set disable: false on the first entry in the formgroup in app.component.ts
Providing a Plunker (or similar) is the best way to get the team to see your issue.
Plunker template: http://plnkr.co/edit/TqYYyhKerILQNyt7BFjQ?p=info
What is the use-case or motivation for changing an existing behavior?
using the standard angular form groups
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
@angular/cli: 1.2.7
node: 8.4.0
os: darwin x64
@angular/animations: 4.3.6
@angular/cdk: 2.0.0-beta.10
@angular/common: 4.3.4
@angular/compiler: 4.3.4
@angular/core: 4.3.4
@angular/forms: 4.3.4
@angular/http: 4.3.4
@angular/material: 2.0.0-beta.10
@angular/platform-browser: 4.3.4
@angular/platform-browser-dynamic: 4.3.4
@angular/router: 4.3.4
@angular/cli: 1.2.7
@angular/compiler-cli: 4.3.4
@angular/language-service: 4.3.4

Ok my Bad sorry


Output tab with stranges squares

Hello,
Using sybase, and defncopy to get a procedure definition, I see a square on empty lines in the output tab.
It does not matter, but when I try to copy the entire script to the clipboard, I only get back to the first square.
Do you see this squares ?

Thank you for dbeaver, a very useful tool.

Most likely this is an issue in Sybase driver. According to reported behavior these special characters have zero bytes inside - that's why Windows copies only the first part.
I'll add some additional output text processing to avoid such problems.


pull request for e2e test at 12/18/2016 20:45:18

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


Add 'today' as a valid input for Date parameter

Fixes #64
Background
Dates must currently be input in YYYY-MM-DD format. It can be a hassle to have to remember today's date as well as to manually calculate dates relative to today's date. To deal with this, the user can opt to input today or today [long] instead for any date parameter. The former will make the date parameter today's date, while the latter will make it today's date + [long] number of days.
Let's

delete the old DateMatchesKeywords file
add today and today [long] as valid inputs for the Date parameter.

Codecov Report

Merging #91 (44e59fa) into master (0b79a7b) will decrease coverage by 0.39%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master      #91      +/-   ##
============================================
- Coverage     47.18%   46.78%   -0.40%
  Complexity      451      451
============================================
  Files           104      104
  Lines          2238     2257      +19
  Branches        326      330       +4
============================================
  Hits           1056     1056
- Misses         1104     1123      +19
  Partials         78       78

Impacted Files
Coverage Δ

...in/java/seedu/address/commons/util/StringUtil.java
75.00% <0.00%> (-19.74%)
:arrow_down:

...in/java/seedu/address/logic/parser/ParserUtil.java
72.13% <0.00%> (ø)

src/main/java/seedu/address/model/entry/Date.java
0.00% <0.00%> (ø)

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 0b79a7b...44e59fa. Read the comment docs.